IAC-1328: support refresh-state

Describe your changes
Comment Triggers

PR Check triggers

Build: trigger build
Sub Category Field Check: trigger subcategoryfieldcheck
gitleaks: trigger gitleaks

This pull request has been automatically closed due to 30 days of inactivity. If you believe this was done in error, please feel free to reopen the PR or reach out to the BT team for further assistance. Thank you for your contributions!


[google_sign_in] Make the Delegate non-final to allow overriding

Description
Make the Delegate non-final to allow overriding.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@cyanglaz Hi Chris, could you help me to merge? I don't have the team role yet.

Exception has occurred.
MissingPluginException (MissingPluginException(No implementation found for method share on channel plugins.flutter.io/share))


Use craco instead of CRA-rewired

Switching to craco to replace react-app-rewired, as suggested in https://github.com/CartoDB/carto-react-template/pull/334#issuecomment-1082695416
For now just in template-sample-app-3 to see how it works (spoiler, it's pretty simple :slightly_smiling_face:)

This fails in CI, but I would suggest dealing with it in this same PR: basically switching (extending?) carto-2 version with carto-3 counterparts

This fails in CI, but I would suggest dealing with it in this same PR: basically switching (extending?) carto-2 version with carto-3 counterparts

@VictorVelarde Sorry, I'm not sure what you mean. Do you propose to write also a Cypress test-suite on v3?
It seems to me the CI currently fails on v2 because tests depend on external data:
getAggregationSumWidget().findByText('411M').should('exist');

If carto-demo-data.demo_tilesets.osm_buildings has been updated since the test spec was written, this 411M value has probably changed. IMO we should rewrite this test to check something else.

Notice that today, craco is not still officially supported with cra-v5 (they are working on it for v7.0.0, currently on alpha)

Done in #356


Add new sample: tab-adaptivecards-graph-search

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

What's in this Pull Request?
Add a new sample showing how to implement a tab with adaptive cards and a bot as a back-end code, and also using Graph api to search different resources across office 365

Great sample - first one showing adaptive cards as tabs.
Thanks!

Hi @AhmadiRamin,
this is nice sample, and it would be great to demo this in the community calls. We are currently fully booked until end of the year but would have again free spots starting from January if you'd be interested.
To avoid your response to get hidden on the GitHub noise, please submit following form if you'd be available for one of the bi-weekly calls on Thursday at 3 PM UK time - https://aka.ms/m365pnp/request/demo
Hopefully works - let us know. Thx 🙏🚀👏

Hi @VesaJuvonen
Thank you, I would love to demo this in one of the calls, I've submitted the form :)
Many thanks


[QoL] Visualizer Bitmap missing high-bit palette control

The visualizer is checkbox that adds user control of the high-bit for Apple ][ graphics:
[x] High-bit set. This palette bit controls green/magenta (off) orange/blue (on)

Most things that want the high bit to be set will set it in the bitmap data itself.  I take it you've found a counter-example?

Yeah, Lode Runner has all of its sprite data @ $AD00 with the low bit set.  It uses "Colorization Tables" at @ $A200, $A280, $A300, ... $A8FF to colorize the monochrome pixels. See #74 for details.
Column2Pixel .dd1 $a2
         .dd1    $a3
         .dd1    $a4
         .dd1    $a5
         .dd1    $a6
         .dd1    $a7
         .dd1    $a8

I haven't (yet) seen the reverse of the high-bit being forced off, but might need to consider that edge case as well.

Lode Runner isn't the only game.  I've seen others but their names escape my memory at the moment.  i.e. Font Data.

Would this make the most sense as a "flip high bit"?  Seems like it works for the case of "all bits are low and should be set".  I'm wondering what it means if something has high bits set, or has some bytes each way.

That might work.  There are a couple of different ways of handling this:

Force high bit.
Force low bit
Toggle high bit

Sadly we need more data before we can make an informed choice that covers all cases (instead of just the common ones.)

Did the "flip bit" thing.  Anything fancier should probably use a custom converter that knows how to set the bits correctly for each byte.

👍 Verified fixed.

As I reverse engineer more games I'll make notes of any failing edge cases.


Running script on SQL 2022; "Cannot find data type dbo.ipaddress."

Trying to run these scripts on SQL server 2022.  The IP Address fields cannot be created.  The error is:
Cannot find data type dbo.ipaddress.
Is there a different data type that could be used?

@ItAdmin712 - you may want to look elsewhere for the script to setup the SQL database, although I'm sure that this script works well for some, I also had trouble with it. I would look further at the official Microsoft documentation specifically this web page - https://learn.microsoft.com/en-us/windows/win32/nps/sql-programmability#sample-stored-procedure I used the script there to create my database.

Thats where I originally created this from, but 2016 days, you need to run for newer NPS versions, I havent had time to even look at updates in so long bc we no longer use NPS, using a docker freeradius mod bc it can do a little more we needed it to do
if exists (select * from dbo.systypes where name = N'ipaddress')
exec sp_droptype N'ipaddress'
GO

EXEC sp_addtype N'ipaddress', N'nvarchar (15)', N'not null'
GO`

and change columns from
[dbo].[ipaddress] NULL ,

to
[ipaddress] NULL ,


merge from arlac77/template-arlac77-github,arlac77/template-esm-only

package.json

chore(deps): add ^7.3.2 remove ^7.3.1 (devDependencies.c8)

:tada: This PR is included in version 1.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Two sum problem

Given an array of integers numbers and an integer sum, return indexes of the two numbers such the sum of two numbers is equal to sum.
Assumpttion

Each input would have exactly one solution,
You should not not use the same element twice.
Return the indexes in any order.

Explain different approaches in Go Language

I would like to take this issue up.

I can work on this.

@rakshitgondwal Sure
Read the contributing guidelines before proceeding further

@rakshitgondwal Sure
Read the contributing guidelines before proceeding further

@rakshitgondwal let me check give me some time

@rakshitgondwal delete your node_modules folder and get latest.. The issue should be fixed.. Let me know if you face any issues..

Hey can I also work on this issue?

@sambhavgupta0705 Can you work on [/issues/25] ?

Yes I can

@rakshitgondwal delete your node_modules folder and get latest.. The issue should be fixed.. Let me know if you face any issues..

I'm still getting the same error

@rakshitgondwal It should be a problem actually. Even I am using windows.
Fork the repository again. Instead of old fork.

@rakshitgondwal Your forked branch is behind of 5 commits update and then take latest in your local


⚠ RPC server - Conflux has degraded performance

In 4cfc66c, RPC server - Conflux ($CONFLUX_RPC_URL) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: RPC server - Conflux performance has improved in 8677ecf after 4 minutes.


fix(tidb): archive log for all unsuccessful task

Currently, only when the task fails will it be archived. If the task times out or fails unstably, no logs can be collected.
The logs generated by parallel tasks will overlap with each other, so here we first use tar to package the logs and then archive them.

replay test for timeout pipeline  https://do.pingcap.net/jenkins/blue/organizations/jenkins/pingcap%2Ftidb%2Fghpr_check2/detail/ghpr_check2/34412/artifacts

/hold

/unhold


Updated Mapbox Studio to 0.3.3

Mapbox Studio is now at version 0.3.3. Upon installing the current version via Cask, I discovered the discrepancy, so I updated the Cask to reflect the latest release.

Thank you for the contribution. It was merged directly as 642c7441dabd002342e98743945014e2d3c098bf to keep commit history cleaner. Your contribution is still credited to you.


Maximum Append Size of 1048576 Exceeded

Hi,
We have been receiving below exception on appending to stream now. From docs i can see, events size that can be appended to stream is approx 16 MB. But from below exception, it looks like we cannot append more than 1 MB. Is there a buffer limit on streaming to event store?
java.util.concurrent.ExecutionException: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: Maximum Append Size of 1048576 Exceeded.
at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:395)
at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1999)
... 7 more
Caused by: io.grpc.StatusRuntimeException: INVALID_ARGUMENT: Maximum Append Size of 1048576 Exceeded.
at io.grpc.Status.asRuntimeException(Status.java:533)
at io.grpc.stub.ClientCalls$StreamObserverToCallListenerAdapter.onClose(ClientCalls.java:478)
at io.grpc.internal.ClientCallImpl.closeObserver(ClientCallImpl.java:434)
at io.grpc.internal.ClientCallImpl.access$500(ClientCallImpl.java:66)
at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl$1StreamClosed.runInternal(ClientCallImpl.java:763)
at io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl$1StreamClosed.runInContext(ClientCallImpl.java:742)
at io.grpc.internal.ContextRunnable.run(ContextRunnable.java:37)
at io.grpc.internal.SerializingExecutor.run(SerializingExecutor.java:123)
... 3 more
On further investigation, it seems to be coming from netty client, used by grpc to connect to server and i can see there is windows frame flow control for http/2, set at netty channel, which has default value of 1048576. is there a way we can override that?
Server version : <IP_ADDRESS>
Client : 1.0.0 (java)
Thanks.

Hey @rohsin47, and thanks for reporting this. I wasn't aware of that default limitation with the Java gRPC implementation. We can definitely use the maxInboundMessageSize internally. Expect a patch landing this week.
By the way, version 2.0.0 of the java client is available on Maven Central!

@YoEight Seems like that error is from https://github.com/EventStore/EventStore/blob/3108c3d26e753770778af04f1916b104105261ed/src/EventStore.Core/Services/Transport/Grpc/RpcExceptions.cs#L87

@ahjohannessen You are the MVP! I was looking at the wrong location. Let's fix this 💪

I think it is a server settings, MaxAppendSize, for this?

There is. @rohsin47 any chance you changed that value without knowing?

I think the default value is 1MB

@rohsin47 After conducting some research, this change has been going on since 20.6. The DB still allows an event with a payload as large as 16MB but you still have to compose with the max append size too. You can obviously increase that value on your own.
A bit of history: https://discuss.eventstore.com/t/how-to-change-max-append-size/2810/3
@ahjohannessen Yeah, I totally forgot about that setting, to be honest.


Smart Charging - limit charging periods according ocpp paramater

Test: limt with a charging plan. It must not have more than 50 items in the charging plan table.


Missing plugin exception

Hello.
I launched on real android device the packages\gamepads\example\lib\main.dart but got on Gamepads.list() call:
MissingPluginException (MissingPluginException(No implementation found for method listGamepads on channel xyz.luan/gamepads))
How can I make it works?

Hello. I launched on real android device the packages\gamepads\example\lib\main.dart but got on Gamepads.list() call: MissingPluginException (MissingPluginException(No implementation found for method listGamepads on channel xyz.luan/gamepads))
How can I make it works?

Did you modify your application's MainActivity as per the Android integration instructions in the README?

How are you referencing gamepads in your pubspec.yaml file? The latest published version of gamepads on pub.dev (0.1.1) does not support Android, so if you are depending on gamepads from pub, gamepads_android simply will not be included in your build.
Everything should be in sync on git at the moment.

Closing due to lack of response

Hello, thanks for your answer.
I creaed new flutter project,

added 'gamepads: ^0.1.1' string to pubspec.yaml
replace code in main.dart with https://github.com/flame-engine/gamepads/blob/main/packages/gamepads/example/lib/main.dart
replace code in MainActivity.kt with mentioned android integration code from readme.md

And got follow in debug console on build:
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (9, 12): Unresolved reference: flame_engine
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (11, 40): Unresolved reference: GamepadsCompatibleActivity
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (23, 5): 'registerInputDeviceListener' overrides nothing
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (29, 5): 'registerKeyEventHandler' overrides nothing
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (33, 5): 'registerMotionEventHandler' overrides nothing

Hello, thanks for your answer.
I creaed new flutter project,

added 'gamepads: ^0.1.1' string to pubspec.yaml

replace code in main.dart with https://github.com/flame-engine/gamepads/blob/main/packages/gamepads/example/lib/main.dart

replace code in MainActivity.kt with mentioned android integration code from readme.md

And got follow in debug console on build:
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (9, 12): Unresolved reference: flame_engine
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (11, 40): Unresolved reference: GamepadsCompatibleActivity
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (23, 5): 'registerInputDeviceListener' overrides nothing
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (29, 5): 'registerKeyEventHandler' overrides nothing
e: \gamepadtest\android\app\src\main\kotlin\com\example\gamepadtest\MainActivity.kt: (33, 5): 'registerMotionEventHandler' overrides nothing

This interface referred to in the integration code is introduced in gamepads 0.1.2, please upgrade the package, that should resolve the issue.


Admin user browsing

Please review new branch

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: kosecki123:x: m0azamYou have signed the CLA already but the status is still pending? Let us recheck it.


TACE 16 to UTF8 and back converter

Convert UTF8 input and respond back with a TACE16 Encoded String.
Revert the conversion in another method.
Unit test cases

This work is already present in Open-Tamil due to encoding work by @arulalant
We can convert back/forth using tamil.txt2unicode and tamil.unicode2txt module functions as also demonstrated in http://tamilpesu.us/tsci/ website.


github.CommitFile is missing URL fields

The github.CommitFile struct is missing a few fields. For example if we look at (shortened) file struct from api.github.com/repos/google/go-github/commits/b9698164669e386680621e2006cab02271257611:
{
  "sha": "916a2dc2a1a4699bf677505e0caad928ac5e0b6d",
  "filename": "github/search.go",
  "status": "modified",
  "additions": 12,
  "deletions": 8,
  "changes": 20,
  "blob_url": "https://github.com/google/go-github/blob/b9698164669e386680621e2006cab02271257611/github/search.go",
  "raw_url": "https://github.com/google/go-github/raw/b9698164669e386680621e2006cab02271257611/github/search.go",
  "contents_url": "https://api.github.com/repos/google/go-github/contents/github/search.go?ref=b9698164669e386680621e2006cab02271257611",
  "patch": "..."
},

Notice the blob_url, raw_url, and contents_url have no matching struct field in github.CommitFile. This is particularly evident when no patch value is returned (for example on binary files) and the file contents must be retrieved by the client (which requires the contents_url)
I didn't want to just submit a pull request adding the fields as there seems to be a pattern within go-github to avoid raw URLs and wanted to double check all the missing fields should be added or only some before attaching a pull request.

Early in the development of the library, I was intentionally avoiding those raw URL fields since they are mostly useful if you are relying on the hateoas design of the GitHub API.  go-github does not, so most of the time these URLs aren't needed.  There have been a few cases where people have needed them for various purposes, so I'm fine with them being added at this point.

In pursuit of closing non-actionable issues, now that #484 has been merged, is there anything else left to be done for this issue? Or shall we mark it as a question and close?

Sorry for the delay. I believe this has been fixed by #484!


prisma 1 nexus doesn't expose all fields. aggregate fields on connection.

I found the reported issue in the forum but it was closed with no answers hence opening an issue here.
https://v1.prisma.io/forum/t/nexus-not-expose-all-functions-not-able-to-call-aggregation-from-nexus-interface/7543/5
it throws the exact same error :
{
  "data": null,
  "errors": [
    {
      "message": "Unknown prisma-client function for field UserConnection.aggregate",
      "locations": [
        {
          "line": 9,
          "column": 5
        }
      ],
      "path": [
        "meta",
        "aggregate"
      ]
    }
  ]
}

please provide a way to expose this. my use-case is I'm exposing my whole prisma api via nexus in the graphql-server with graphql-shield on it.
My current work-around for this is to add a resolver for the same and which would require me to write resolver for all the connections like this:
import { prismaObjectType } from 'nexus-prisma'

export const UserConnection = prismaObjectType({
  name: 'UserConnection',
  definition(t) {
    t.prismaFields(['edges', 'pageInfo'])
    t.field('aggregate', {
      type: 'AggregateUser',
      resolve: async (parent, args, ctx) => {
        const count = await ctx.prisma
          .usersConnection()
          .aggregate()
          .count()
        return {
          count,
        }
      },
    })
  },
})

Also the type param shows an error but the server complies and starts successfully:

package.json:
{
    "nexus": "0.11.7",
    "nexus-prisma": "0.3.8",
    "prisma-client-lib": "1.34.10"
}

I found this in your code. why are we not exposing the aggregate?
    if (utils_1.isConnectionTypeName(typeName) && fieldName !== 'aggregate') {
        // returns `pageInfo` and `edges` queries by the client
        return true;
    }

Closing this cause it belongs to nexus-schema-plugin-prisma repo. sorry.
https://github.com/graphql-nexus/nexus-schema-plugin-prisma/issues/12
https://github.com/graphql-nexus/nexus-schema-plugin-prisma/issues/184


Automatic grouping of duplicated layers annoys some users

The question we need to consider here is what would we replace it with. Although not perfect, it is a really easy way to cut down the amount in this list is an easy to understand way (well hopefully anyway).
Sounds like a feature for later as we have quite a few other bits to do first. Lets leave it open to discuss though.

After reading this request/complaint the first time, I understood that the automatic grouping in general annoys some users. Now, re-reading it, I think it is only regarding the grouping of a layer which was just duplicated by using the duplicate icon:

Options to fix this:

Open the group where the duplication took place automatically.

or

Use an uncommon prefix for the ID of the new layer, e.g. add copy- before the duplicated ID instead of appending it.

Regarding automatic grouping in general:

Although not perfect, it is a really easy way to cut down the amount in this list is an easy to understand way

:+1:
Later there could be a setting panel, where beside other things the user would be able to turn automatic grouping on/off.

So question should we remove the grouping from the Maputnik layers list? It would make a few of the other tickets easier if it was an non-grouped list

#320 - we could use a vertical tab panel (see https://github.com/davidtheclark/react-aria-tabpanel)
#431 - becomes really simple because it's just reversing the order of an array, reversing a tree makes a lot less sense form a UI perspective
#437 - we could add a 'filter input' at the top of the layers list, which would make finding things a lot easier.

So shall we remove the grouping all together?

Personally, I use the grouping to collapse a long layer list into a more manageable form. I am constantly expanding and collapsing as I work. A filter input is likely to be slower for someone (like me)  who knows exactly where in the layer list a layer is, and just needs to get there quickly.

Hi @kateler I have a few more questions

...someone (like me) who knows exactly where in the layer list a layer is, and just needs to get there quickly

What is the advantage for you of

scroll layer list
click to expand group
click layer

Over scrolling a long list and clicking the layer?

I use the grouping to collapse a long layer list into a more manageable form

What if you could filter by multiple things, so for example currently grouping is by the prefix_, so if you have the layers

road_motorway
road_minor
bridge_motorway
bridge_minor
waterway_tunnel
waterway_river
And lots more...

In the filtered version you would enter the filter road, bridge which would filter the layer list to

road_motorway
road_minor
bridge_motorway
bridge_minor

My stylesheets have something like 150 layers, so it's hard to scroll precisely and quickly to the right place. When layers are collapsed into groups, the list is a little more than one screen high. Most of it is visible at once so I don't often have to scroll, just expand.
If filtering existed in addition to the grouping, I would use it sometimes. But probably not most of the time, because cartography is such a mouse-heavy activity and I'd avoid the extra step of moving my hands back and forth between mouse and keyboard.

If you are having many layers in a style I  also see the advantages of the current grouping as described by @kateler:

better visual overview of the layer tree
less scrolling (I also prefer to expand and collapse groups compared to scrolling)

But there are also a lot of drawbacks of the current implementation:

all the points @orangemug outlined in https://github.com/maputnik/editor/issues/428#issuecomment-619150204.
automatic grouping is certainly confusing for some users (as reported in the survey), because it's not entirely clear why it's happening. It's especially confusing if you duplicate a layer.
also the current implementation isn't "complete":

it's not possible to show/hide a layer group (https://github.com/maputnik/editor/issues/376)
it's not possible to drag a layer group

the automatic grouping is a hack using the layer names, so it could result in ambiguous group names. E.g. for the OSM Liberty style there are two groups named road and a group and a layer named water:

Because it looks like currently there are far more drawbacks than advantages, I think we should remove the grouping all together for now.
Later it would be nice to implement a new grouping which:

allows manually grouping/ungrouping of layers instead of automatic grouping to avoid confusing the user and give precise control over which layers should be grouped
allows to edit group names
allows to show/hide whole groups
allows to drag groups

I want to add that we should directly implement a filter function for the layer list (https://github.com/maputnik/editor/issues/437, and good suggestion in https://github.com/maputnik/editor/issues/428#issuecomment-619352557 regarding filter by multiple things :+1:) if we decide to remove the current automatic grouping.
I suppose this would help users with long layer lists a lot until there would be a new grouping implemented, because it would be possible to "simulate" the automatic grouping to only show all layers which were automatically grouped before in one certain group.

Thanks @pathmapper that's a really good overview of the problems.
So I think before we do anything we should try and reach out to a few more users to get some more feedback. Whatever we end up doing, it's going to be a big workflow change to the editor for regular users. Hopefully we can get a solution that users, like @kateler, are really happy with. However we're obviously going to need to make some compromises and accessibility should definitely be a key aim here. If we do impact users workflow lets try and come up with a solution that minimises that impact, although I still hopeful with can make Kate's experience better.
Here's how I suggest we proceed

Wait for a few more days see if we get any other commenters to this ticket
Start to reach out to users that we know use Maputnik regularly to collect more feedback
Once we have some more voices, build a proof of concept from that feedback. Hopefully we can add this into the editor behind one of our debug flags in release v1.8.0 so users can try it out first.

Does that sound like a good approach?

@orangemug makes totally sense :+1:
Another drawback of the current implementation:
Because it's not possible to drag a group, it's also not possible to drag a layer between groups:

Notifying a few people that have been active with Maputnik in the past/present, for some feedback.
@nspringer-trimble @fredj @kylebarron @keum @jonahadkins @BergWerkGIS @robert-claypool
If anybody/everybody has some time to give feedback on this ticket, it'd be much appreciated. As it's pretty core to the editing experience it's something we want to get right.

Ping @lukaswelte @sk1p

As a UI/UX designer and frequent Maputnik user I see a few different issues here.

Not all users like automatic grouping: but... many do.  I agree with @pathmapper that it took me a little while to figure out why things were grouping, but one I did I liked it and started naming layers on purpose for grouping. It is a bit arbitrary since Mapbox uses layer order for draw order (not a z-index like Mapzen), so the grouping is irrelevant if the layers are not next to each other in the stack. I suggest the following fixes (and forgive me if some of these are already in place):

Groups are expanded by default (but show the headers)
Add a toggle or setting to not show groups
Add a hover message on the toggle explaining how groups are formed
Use LocalStorage to save which groups are expanded (this will need to be done per style file)
Redesign the headers a little to make them more obvious and clickable

New layers appear outside their group: this we should just fix so the group expands when a layer is duplicated.
Can't drag between groups: expand groups on hover when dragging (with a 500ms debounce so things don't jump around too much)

Thanks @nspringer-trimble would you still allow searching/filtering of layers based off of name?
Also a major concern is accessibility, writing a stable TreeView (https://www.w3.org/TR/wai-aria-practices/examples/treeview/treeview-2/treeview-2a.html) with drag/drop reordering isn't easy. I'm unable to find one that seems to meet our needs. We could write our own, but I'm concerned with the effort/time that would take. Is there

A re-orderable treeview component we can use that you've seen?
Any tweak to the UI you can think of to make that less effort?

I have never used searching or filter myself. The grouping essentially takes care of this.  Our style files typically have 110+ layers and finding one has never been a problem.
I have never researched React tree controls, I am mostly an Angular developer :)
You could allow dropping on a group header (hightlighting it on hover) and then expanding the group after dropping it.

I have never researched React tree controls, I am mostly an Angular developer :)

Ok thanks, if there is anything ARIA compliant in angular/vanilla web please also send that my way. The aria TreeView gets pretty complex spec wise 😬, anything would be good.
I think lets give a bit of time for others to respond, thanks for all the feedback @nspringer-trimble some very good points 👍


openssl 1.1.1 can not find sm.h

When I compiled by source files,I could not find the sm*.h.
Compile command：
"./config shared && make && make install"

it's private, you can only access it with EVP*, EC*, etc #5670 #7075


Hiding the Buffer type (far right) on tabline

Hi,
Thanks for a great plugin. I'm using the tabline extensively and am wondering if there's anyway to hide the "buffers" string (indicating tabtype?) on the far right of the tabline.
I've tried setting:
let g:airline#extensions#tabline#show_tab_type = 0

to no avail.
Thanks.

It works for me to disable the buffer, but when i have tabs, there is always the filename shown:

I've read :help airline-tabline but could not find any option beside 'show_tab_type', but the filename is not the type I guess ...
Can anyone point out how to disable the filename showing and, for example display the contents of
airline#extensions#tabline#tabs_label
for example, or disable it completely? Thanks


Feature/add comments

@HenrikFrtz You are on the right track.
Your last PRs looked quite good.
May you check the merge conflict of this PR?
I will give check your single examples and open new issues for further improvements.


Rename main branch to main

Codecov Report

Merging #11854 (7632757) into main (e0b33b7) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #11854      +/-   ##
==========================================
- Coverage   68.95%   68.94%   -0.01%
==========================================
  Files         312      312
  Lines       16400    16400
  Branches     4750     4750
==========================================
- Hits        11308    11307       -1
- Misses       5065     5066       +1
  Partials       27       27

Impacted Files
Coverage Δ

packages/expect/src/utils.ts
95.58% <0.00%> (-0.56%)
:arrow_down:

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 e0b33b7...7632757. Read the comment docs.

Bro can you sent me the special hack facebook i want to hack a facebook plz
help me...!
On Wed, Sep 8, 2021, 9:21 PM Codecov Comments Bot @.***>
wrote:

Codecov
https://codecov.io/gh/facebook/jest/pull/11854?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook
Report
Merging #11854
https://codecov.io/gh/facebook/jest/pull/11854?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook
(7632757
https://github.com/facebook/jest/commit/76327574b941af87b4e2079552d5a08697ddcd9d)
into main
https://codecov.io/gh/facebook/jest/commit/e0b33b74b5afd738edc183858b5c34053cfc26dd?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook
(e0b33b7
https://github.com/facebook/jest/commit/e0b33b74b5afd738edc183858b5c34053cfc26dd)
will decrease coverage by 0.00%.
The diff coverage is n/a.
[image: Impacted file tree graph]
https://codecov.io/gh/facebook/jest/pull/11854?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook
@@            Coverage Diff             @@
main   #11854      +/-
==========================================

Coverage   68.95%   68.94%   -0.01%

==========================================
Files         312      312
Lines       16400    16400
Branches     4750     4750
==========================================

Hits        11308    11307       -1

Misses       5065     5066       +1
Partials       27       27

Impacted Files
https://codecov.io/gh/facebook/jest/pull/11854?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook Coverage
Δ
packages/expect/src/utils.ts
https://codecov.io/gh/facebook/jest/pull/11854/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook#diff-cGFja2FnZXMvZXhwZWN0L3NyYy91dGlscy50cw== 95.58%
<0.00%> (-0.56%) ⬇
Continue to review full report at Codecov
https://codecov.io/gh/facebook/jest/pull/11854?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook
.
Legend - Click here to learn more
https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook
Δ = absolute  (impact), ø = not affected, ? = missing data
Powered by Codecov
https://codecov.io/gh/facebook/jest/pull/11854?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook.
Last update e0b33b7...7632757
https://codecov.io/gh/facebook/jest/pull/11854?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook.
Read the comment docs
https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=facebook
.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/jest/pull/11854#issuecomment-915402947, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APAJXWGQHVJMD6EVFICKFPDUA6IAXANCNFSM5DVH3NCA
.


[FEATURE] Add support for inclusive naming

Add support for inclusive naming by deprecating the master nomenclature.
See meta issue https://github.com/opensearch-project/opensearch-clients/issues/16 for details.

@VachaShah I'm working on this. You can assign it to me.


Zoom event is triggered when dragging layer around

🐛 Bug
Zoom event is triggered when dragging layer around
To Reproduce
Steps to reproduce the behavior:
import napari
import numpy as np
v = napari.Viewer()
il = v.add_image(np.random.rand(10, 10), colormap='reds')
il2 = v.add_image(np.random.rand(10, 10), colormap='reds')
def zoom(event):
    print('ZOOM')
v.camera.events.zoom.connect(zoom)

Expected behavior
This zoom event should only be triggered when the camera zoom value is really changed. I noticed the zoom level is modified slightly when the layer is dragged around, so that might be the reason!
Environment
napari: 0.4.15
Platform: Linux-5.17.4-200.fc35.x86_64-x86_64-with-glibc2.33
System: Freedesktop.org 21.08.12 (Flatpak runtime)
Python: 3.9.9 (main, Nov 10 2011, 15:00:00) [GCC 11.2.0]
Qt: 5.15.2
PyQt5: 5.15.6
NumPy: 1.22.3
SciPy: 1.8.0
Dask: 2022.04.1
VisPy: 0.10.0
OpenGL:

GL version: 4.6.0 NVIDIA 510.47.03
MAX_TEXTURE_SIZE: 16384

Screens:

screen 1: resolution 1920x1200, scale 1.0

Plugins:

console: 0.0.4
scikit-image: 0.4.15
svg: 0.1.6

Guh. Yeah so I think the issue is that the zoom is actually a computed feature of the viewport rectangle, as discussed in #3723. This means that changing the rectangle will result in a float computation, and, floats being what they are, the zoom will vary a tiny bit and get triggered. I improved your script a little bit with:
import napari
import numpy as np
v = napari.Viewer()
il = v.add_image(np.random.rand(10, 10), colormap='reds')
il2 = v.add_image(np.random.rand(10, 10), colormap='reds')
def zoom(event):
    print(f'{v.camera.zoom=}')
v.camera.events.zoom.connect(zoom)

And this gets me:
v.camera.zoom=88.82499999999999
v.camera.zoom=88.82499999999997
v.camera.zoom=88.82499999999999
v.camera.zoom=88.825
v.camera.zoom=88.82500000000002
v.camera.zoom=88.82500000000005
v.camera.zoom=88.82500000000006
v.camera.zoom=88.82500000000007
v.camera.zoom=88.82500000000009
v.camera.zoom=88.8250000000001
v.camera.zoom=88.82500000000012
v.camera.zoom=88.8250000000001
v.camera.zoom=88.82500000000009
v.camera.zoom=88.82500000000007
v.camera.zoom=88.82500000000006
v.camera.zoom=88.82500000000007
v.camera.zoom=88.82500000000009
v.camera.zoom=88.8250000000001
v.camera.zoom=88.82500000000012
v.camera.zoom=88.8250000000001
v.camera.zoom=88.82500000000012
v.camera.zoom=88.8250000000001
v.camera.zoom=88.82500000000009
v.camera.zoom=88.8250000000001
v.camera.zoom=88.82500000000012
v.camera.zoom=88.82500000000013
v.camera.zoom=88.82500000000014

ie you can see that the zoom has changed on every trigger.
We could change equality to some tolerance, but that requires fiddling with pydantic data models. The implementation is here:
https://github.com/napari/napari/blob/eea4117525bed8fb11cd0ba13aba4e9a7c83b848/napari/components/camera.py#L35
You can see that we don't actually do anything specific with the zoom: the EventedModel decides to fire when we change the value. One thing might be to create some kind of "LooseFloat" type that only updates if it changes by more than some epsilon. That seems like a can of worms though.
Thoughts welcome!

thanks for the clear report @cnstt - is this actively causing you a problem in your plugin, or is it a minor annoyance?
If it's causing you a problem because you trigger a slow-ish computation on zoom change, perhaps @tlambert03's qthrottler will be useful for you!


I am unable to get nodes to joing microk8s cluster on raspberry pi 4

Please run microk8s inspect and attach the generated tarball to this issue.
We appreciate your feedback. Thank you for using microk8s.
All rasps
Ubuntu-20.10-preinstalled-server-arm64+raspi.img.xz   installed as image
root@ubuntu:/home/ubuntu# cat /proc/cpuinfo
processor	: 0
BogoMIPS	: 108.00
Features	: fp asimd evtstrm crc32 cpuid
CPU implementer	: 0x41
CPU architecture: 8
CPU variant	: 0x0
CPU part	: 0xd08
CPU revision	: 3
processor	: 1
BogoMIPS	: 108.00
Features	: fp asimd evtstrm crc32 cpuid
CPU implementer	: 0x41
CPU architecture: 8
CPU variant	: 0x0
CPU part	: 0xd08
CPU revision	: 3
processor	: 2
BogoMIPS	: 108.00
Features	: fp asimd evtstrm crc32 cpuid
CPU implementer	: 0x41
CPU architecture: 8
CPU variant	: 0x0
CPU part	: 0xd08
CPU revision	: 3
processor	: 3
BogoMIPS	: 108.00
Features	: fp asimd evtstrm crc32 cpuid
CPU implementer	: 0x41
CPU architecture: 8
CPU variant	: 0x0
CPU part	: 0xd08
CPU revision	: 3
Hardware	: BCM2835
Revision	: d03114
Serial		:<PHONE_NUMBER>b14ab3
Model		: Raspberry Pi 4 Model B Rev 1.4
root@ubuntu:/home/ubuntu#
master=============
root@ubuntu:/home/ubuntu# microk8s.kubectl get all --all-namespaces
NAMESPACE     NAME                                          READY   STATUS    RESTARTS   AGE
kube-system   pod/coredns-86f78bb79c-l5snv                  1/1     Running   1          2m16s
kube-system   pod/calico-kube-controllers-847c8c99d-vwv98   1/1     Running   0          9m55s
kube-system   pod/calico-node-6v54d                         1/1     Running   0          9m55s
kube-system   pod/hostpath-provisioner-976f6d665-j26m9      1/1     Running   0          96s
kube-system   pod/metrics-server-7b7db5984b-7jdxd           1/1     Running   0          90s
ingress       pod/nginx-ingress-microk8s-controller-jbzxh   1/1     Running   0          107s
NAMESPACE     NAME                     TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)                  AGE
default       service/kubernetes       ClusterIP   <IP_ADDRESS>            443/TCP                  10m
kube-system   service/kube-dns         ClusterIP   <IP_ADDRESS>           53/UDP,53/TCP,9153/TCP   2m16s
kube-system   service/metrics-server   ClusterIP   <IP_ADDRESS>            443/TCP                  90s
NAMESPACE     NAME                                               DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR            AGE
kube-system   daemonset.apps/calico-node                         1         1         1       1            1           kubernetes.io/os=linux   9m59s
ingress       daemonset.apps/nginx-ingress-microk8s-controller   1         1         1       1            1                              117s
NAMESPACE     NAME                                      READY   UP-TO-DATE   AVAILABLE   AGE
kube-system   deployment.apps/coredns                   1/1     1            1           2m16s
kube-system   deployment.apps/calico-kube-controllers   1/1     1            1           9m59s
kube-system   deployment.apps/hostpath-provisioner      1/1     1            1           98s
kube-system   deployment.apps/metrics-server            1/1     1            1           90s
NAMESPACE     NAME                                                DESIRED   CURRENT   READY   AGE
kube-system   replicaset.apps/coredns-86f78bb79c                  1         1         1       2m16s
kube-system   replicaset.apps/calico-kube-controllers-847c8c99d   1         1         1       9m55s
kube-system   replicaset.apps/hostpath-provisioner-976f6d665      1         1         1       98s
kube-system   replicaset.apps/metrics-server-7b7db5984b           1         1         1       90s
root@ubuntu:/home/ubuntu# microk8s add-node
From the node you wish to join to this cluster, run the following:
microk8s join <IP_ADDRESS>:25000/d17ab5558429f412843c2e5ccd2dd9a4
If the node you are adding is not reachable through the default interface you can use one of the following:
microk8s join <IP_ADDRESS>:25000/d17ab5558429f412843c2e5ccd2dd9a4
microk8s join <IP_ADDRESS>:25000/d17ab5558429f412843c2e5ccd2dd9a4
root@ubuntu:/home/ubuntu#
Worker ==================
ubuntu@ubuntu:~$ sudo microk8s join <IP_ADDRESS>:25000/d17ab5558429f412843c2e5ccd2dd9a4
Contacting cluster at <IP_ADDRESS>
Traceback (most recent call last):
File "/snap/microk8s/2157/scripts/cluster/join.py", line 967, in
join_dqlite(connection_parts)
File "/snap/microk8s/2157/scripts/cluster/join.py", line 900, in join_dqlite
update_dqlite(info["cluster_cert"], info["cluster_key"], info["voters"], hostname_override)
File "/snap/microk8s/2157/scripts/cluster/join.py", line 818, in update_dqlite
with open("{}/info.yaml".format(cluster_backup_dir)) as f:
FileNotFoundError: [Errno 2] No such file or directory: '/var/snap/microk8s/2157/var/kubernetes/backend.backup/info.yaml'
ubuntu@ubuntu:~$
on this try i ran microk8s start on worker(i dont think this needs to be done, though)
inspection-report-20210519_003531.tar.gz
ubuntu@ubuntu:~$ sudo microk8s join <IP_ADDRESS>:25000/d17ab5558429f412843c2e5ccd2dd9a4
Contacting cluster at <IP_ADDRESS>
Failed to join cluster. Error code 500.
ubuntu@ubuntu:~$
inspection-report-20210519_003531.tar.gz

Hi @eddiechristian
In the first attempt that failed with FileNotFoundError: [Errno 2] No such file or directory: '/var/snap/microk8s/2157/var/kubernetes/backend.backup/info.yaml' the MicroK8s node probably did not start correctly. You would better do a microk8s status --wait-ready.
On the second attempt you used the IP <IP_ADDRESS>, why? All rpi are on the same network right? Which network is this? Is it 172.17.0.X or 192.168.1.X?
Also please set the different hostname to each node. You do not want to get into the unpleasant situation where ubuntu is resolved to a different IP based on the node you are on.

also  , do i need to do a microk8s start on each node?

also , do i need to do a microk8s start on each node?
MicroK8s needs to be running on each node, but its state should be persistent - i.e if you left it started, it should start again if you reboot etc

i tried wait-ready
ubuntu@ubuntu:~$ microk8s status --wait-ready microk8s is running high-availability: no   datastore master nodes: <IP_ADDRESS>:19001   datastore standby nodes: none addons:   enabled:     ha-cluster           # Configure high availability on the current node   disabled:     dashboard            # The Kubernetes dashboard     dns                  # CoreDNS     helm                 # Helm 2 - the package manager for Kubernetes     helm3                # Helm 3 - Kubernetes package manager     host-access          # Allow Pods connecting to Host services smoothly     ingress              # Ingress controller for external access     linkerd              # Linkerd is a service mesh for Kubernetes and other frameworks     metallb              # Loadbalancer for your Kubernetes cluster     metrics-server       # K8s Metrics Server for API access to service metrics     rbac                 # Role-Based Access Control for authorisation     registry             # Private image registry exposed on localhost:32000     storage              # Storage class; allocates storage from host directory ubuntu@ubuntu:~$ microk8s join <IP_ADDRESS>:25000/8d999fd16c06c51144a8c80804d1c4d9 Contacting cluster at <IP_ADDRESS> Waiting for this node to finish joining the cluster. ..   ubuntu@ubuntu:~$ k get no k: command not found ubuntu@ubuntu:~$ microk8s kubectl get no The connection to the server <IP_ADDRESS>:16443 was refused - did you specify the right host or port? ubuntu@ubuntu:~$ microk8s status microk8s is not running. Use microk8s inspect for a deeper inspection. ubuntu@ubuntu:~$
master hung after worker trys to join
`ubuntu@ubuntu:~$ sudo microk8s add-node
From the node you wish to join to this cluster, run the following:
microk8s join <IP_ADDRESS>:25000/8d999fd16c06c51144a8c80804d1c4d9
If the node you are adding is not reachable through the default interface you can use one of the following:
microk8s join <IP_ADDRESS>:25000/8d999fd16c06c51144a8c80804d1c4d9
microk8s join <IP_ADDRESS>:25000/8d999fd16c06c51144a8c80804d1c4d9
ubuntu@ubuntu:~$
ubuntu@ubuntu:~$
ubuntu@ubuntu:~$ k get no
Error from server (Timeout): the server was unable to return a response in the time allotted, but may still be processing the request (get nodes)
ubuntu@ubuntu:~$
ubuntu@ubuntu:~$
ubuntu@ubuntu:~$
ubuntu@ubuntu:~$ k get no
Error from server (Timeout): the server was unable to return a response in the time allotted, but may still be processing the request (get nodes)
ubuntu@ubuntu:~$ sudo microk8s kubectl get no
^C
ubuntu@ubuntu:~$ ls /var/snap/
core/     core18/   lxd/      microk8s/ snapd/
ubuntu@ubuntu:~$ ls /var/snap/microk8s/
2157/    common/  current/
ubuntu@ubuntu:~$ ls /var/snap/microk8s/2157/
.kube/                                    certs/                                    inspection-report-20210519_003531.tar.gz  opt/
args/                                     credentials/                              juju/                                     tmp/
bin/                                      inspection-report/                        juju-home/                                var/
ubuntu@ubuntu:~$ ls /var/snap/microk8s/2157/credentials/
callback-token.txt         cluster-tokens.txt         controller.config          kubelet.config             scheduler.config
client.config              cluster-tokens.txt.backup  known_tokens.csv           proxy.config
ubuntu@ubuntu:~$ ls /var/snap/microk8s/2157/credentials/client.config
/var/snap/microk8s/2157/credentials/client.config
ubuntu@ubuntu:~$ cat  /var/snap/microk8s/2157/credentials/client.config
apiVersion: v1
clusters:

cluster:
certificate-authority-data: LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSURBVENDQWVtZ0F3SUJBZ0lKQUx5Rmx0NVA5dW5mTUEwR0NTcUdTSWIzRFFFQkN3VUFNQmN4RlRBVEJnTlYKQkFNTURERXdMakUxTWk0eE9ETXVNVEFlRncweU1UQTFNVGd5TXpRM016TmFGdzB6TVRBMU1UWXlNelEzTXpOYQpNQmN4RlRBVEJnTlZCQU1NRERFd0xqRTFNaTR4T0RNdU1UQ0NBU0l3RFFZSktvWklodmNOQVFFQkJRQURnZ0VQCkFEQ0NBUW9DZ2dFQkFNaVNmRk9xMkhRMDFteUdwekxOWGxZb29FcG1Xd2h4ZWVuMUlnUHhrWjVlcHRhREwvM0sKMlVtVG5CSnQvSHpFdUszOWw4emloUHU2dGJnb0RrbEtVTTIwbGRvZ0tiR2piaFFqZGx6RHB5Y2pvZ1F6d3U5ZQpRdGVXOVR5Skh0dnhPbWF4cm1acXN3OVNTRUhaUnMrV044RnRPZHdWSUsxNm1tMHoyVHdzcFdBTnl4U1g3OFNOCit6aUgxY3NTQnFUekMrSk5IcEtacUlkVkx0QXBWWGhrRHgrOFZocmc5ekR1ZmI4MlNlS1BaR3Flbmh6MGgyZ0oKOVZQNWxRbFlBNXVtNVk3bEtFTmRRd0I0WFdoR3RVNHpWRTYvQzd1aSthS3lRRnlJZkRzUFdWVXYyTFlKQzVscAp3S3l1dXduRDFNUnRyREQ4TE9hUnBYVVJWZ2Jod3BCUk9mVUNBd0VBQWFOUU1FNHdIUVlEVlIwT0JCWUVGRDVXCm5VdVFUbm9mR0N2UThOU0kwMnMwRlErSU1COEdBMVVkSXdRWU1CYUFGRDVXblV1UVRub2ZHQ3ZROE5TSTAyczAKRlErSU1Bd0dBMVVkRXdRRk1BTUJBZjh3RFFZSktvWklodmNOQVFFTEJRQURnZ0VCQUdEWWY5TmpzUzB6K0p4SQpwQXRVRFBrdDllQjV2cEtKYzY2cjUvcDM0VTM3K0FtVjVqTThhQ29YQmM2TzJpTHo1UGtzbzlSb1NocHp3UUc2CjJaeUk2bEJ4bkxqWTYvS2Nta09qUkZSVUsvM0lnNitqZElUenF1di80OUVVeUVqZGk5WlU4R2l0RW1uZGx6cUoKSHIxOFdyS1BYTEF2MzEvZkM4SzZrc2pYVDR6TytxbHJiSTEvY3lsSHN3TVFNdG1JaDg5WVA4Y1J3bGV6ZlRnVwpBa1lOcFc1c0lRTGZXYWJhaDY5YlJGNHUyWTY0NE8ycDRZdXJvTHhDcHJJZ1JIblI2Z2hJZTYyUE5rbHJnUXBICnJFU2RNZkJpQml6aTFvZ1B1RW5zcGNTbkRMZStaZExFMGpudGdsNmNSQ0JhZG00MG51NGJoUGVJVmtiU3UzcUIKS3hZNjFvYz0KLS0tLS1FTkQgQ0VSVElGSUNBVEUtLS0tLQo=
server: https://<IP_ADDRESS>:16443
name: microk8s-cluster
contexts:
context:
cluster: microk8s-cluster
user: admin
name: microk8s
current-context: microk8s
kind: Config
preferences: {}
users:
name: admin
user:
token: UWR2RW9aSHpBMGk5YnYwa3NjaTZBbUl4SzFPYTRBSyt4bHo2eEl2RGxMQT0Kubuntu@ubuntu:~$ sudo bash
root@ubuntu:/home/ubuntu# cat ~/.kube/cache/
discovery/ http/
root@ubuntu:/home/ubuntu# cat ~/.kube/cache/
discovery/ http/
root@ubuntu:/home/ubuntu# microk8s statu
`

Could you please set a different hostname to each node.

The master is hung, so i cant get that report(it hangs in inspect)
inspection-report-20210519_194451.tar.gz

now iam trying ubuntu21.04 and getting root@new-hostname:/home/ubuntu# microk8s join <IP_ADDRESS>:25000/48220197b6c261522543ec20ab11615e Contacting cluster at <IP_ADDRESS> Waiting for this node to finish joining the cluster. ..   Traceback (most recent call last):   File "/snap/microk8s/2157/scripts/cluster/join.py", line 967, in <module>     join_dqlite(connection_parts)   File "/snap/microk8s/2157/scripts/cluster/join.py", line 900, in join_dqlite     update_dqlite(info["cluster_cert"], info["cluster_key"], info["voters"], hostname_override)   File "/snap/microk8s/2157/scripts/cluster/join.py", line 856, in update_dqlite     restart_all_services()   File "/snap/microk8s/2157/scripts/cluster/join.py", line 784, in restart_all_services     stderr=subprocess.DEVNULL,   File "/snap/microk8s/2157/usr/lib/python3.5/subprocess.py", line 581, in check_call     raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command '['/snap/microk8s/2157/microk8s-stop.wrapper']' returned non-zero exit status 1 root@new-hostname:/home/ubuntu# microk8s join <IP_ADDRESS>:25000/48220197b6c261522543ec20ab11615e
hopefully my network config is not affecting it? `root@new-hostname:/home/ubuntu# cat /etc/netplan/50-cloud-init.yaml
This file is generated from information provided by the datasource.  Changes
to it will not persist across an instance reboot.  To disable cloud-init's
network configuration capabilities, write a file
/etc/cloud/cloud.cfg.d/99-disable-network-config.cfg with the following:
network: {config: disabled}
network:
version: 2
wifis:
wlan0:
access-points:
Fios-G2NCh:
password: sou35pea63unhand
dhcp4: true
optional: true
root@new-hostname:/home/ubuntu#`

Can you give me some more direction? I dont think the rasp pi rebooted, but i will do what it says above and re-try the join? My colleagues and i work with k8s everyday(kubeadm to join) and 3 of us have struggled to get microk8s up... it should not be that hard , especially for those who have used k8s on a daily basis.

Hi guys, I am trying to add nodes in ARM64 Ampere, buy it doesn't work as well.
Unfortunately [https://microk8s.io/docs/install-alternatives#heading--arm] has no information, because it seems to be different to add cgroup for example.


[ feature ] group selection via tabs

I noticed we don't have an issue for this.
The controller window could look similar to the mockup, but with:

round tabs
a tab for nodes that are Not in group ( https://github.com/chrisgoringe/cg-controller/issues/237 )
maybe no refresh button ( https://github.com/chrisgoringe/cg-controller/issues/236 )
maybe a bypass button ( https://github.com/chrisgoringe/cg-controller/issues/89 )

The icon set ( https://www.primefaces.org/diamond/icons.xhtml ) does not have a bypass icon,
( https://www.svgrepo.com/svg/503241/bypass ).
So maybe we could use of these:
( Because we are "skipping" over the bypassed nodes ).

Other alternative icons could be:

Yeah, a round icon may look better next to the bolt.
But power off would not make sense ( it's the oposite ).
The ban icon could work.
And instead of getting blue, it could get the bypass color !

Trying to think through groups and multiple controllers and tabs...
I think the proposal is to eliminate the current drop-down element, and instead select the group using tabs.
Question: what tabs should be available on a controller?
Option 1 would be to have all of them (like the current drop-down list). That seems to go a little against the idiom of tabs (especially if you have two controllers - they all show all the tabs) and is also a waste of space (again, especially with multiple controllers all showing all tabs)
Option 2 would be that each controller only has tabs that have been added to it somehow. I'm thinking maybe something like this:

a new controller gets the "All" tab
the '+' creates a new tab by popping up a drop down menu to choose which group it should be for
each tab has an 'x' on it (like browser tabs) to close that tab
close the last tab and the controller closes (so maybe no need to have the controller 'X' any more)
when minimised, the currently selected tab forms the title, and only the 'expand' (and bypass) icons are available

thoughts?

There's an initial version of this (option 1, because that's simpler to do first), on the 1.3 branch

Love the design!

Definitely option 2. Doesn't make sense to have all tabs open. People should be able to keep specific tabs open on each Controller windows.

Looks great !
Yeah, option 2 would be the best.

I would also make them rearrangeable by adding a small handle (3 vertical dots) to the left of each tab, to grab them individually (and perhaps move them from one window to another).

I would also make them rearrangeable by adding a small handle (3 vertical dots) to the left of each tab, to grab them individually (and perhaps move them from one window to another).

I'd like that.
But, if we have multiple rows, rearranging tabs will not be very useful.
Because tabs change positions when they are selected.
We could have an user option to always keep tabs in a single row:

the text would be partially hidden, if there is not enough space ( like on the browser )

And to save tab space, ( for both multi-row and single-row modes ):

Instead of 3 dots, we can grab the whole tab to move it.
To move a Controller, grab it by the buttons area.

and maybe:

No the "x" button. To remove a tab, drag it out of the Controller ( on to the graph canvas area )

Don’t think I’m going to make the tabs draggable as a priority - seems
pretty low benefit/cost…
On Fri, 1 Nov 2024 at 8:28 PM, JorgeR81 @.***> wrote:

I would also make them rearrangeable by adding a small handle (3 vertical
dots) to the left of each tab, to grab them individually (and perhaps move
them from one window to another).
I'd like that.
But, if we have multiple rows, rearranging tabs will not be very useful.
Because tabs change positions when they are selected.
We could have an user option to always keep tabs in a single row:

the text would be partially hidden, if there is not enough space (
like on the browser )

And to save tab space, ( for both multi-row and single-row modes ):

Instead of 3 dots, we can grab the whole tab to move it.
To move a Controller, grab it by the buttons area.

and maybe:

No the "x" button. To remove a tab, drag it out of the Controller (
on to the graph canvas area )

—
Reply to this email directly, view it on GitHub
https://github.com/chrisgoringe/cg-controller/issues/249#issuecomment-2451582193,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBLMA5WVH4XYJJZLB6KHGDZ6NCSHAVCNFSM6AAAAABQ4GC5I2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRGU4DEMJZGM
.
You are receiving this because you commented.Message ID:
@.***>

Don’t think I’m going to make the tabs draggable as a priority - seems pretty low benefit/cost…

And what do you think about a "single row" mode ?

Yeah, that's fairly simple. Might be best just to make it always like that (having lots of tabs probably isn't going to be normal)

Might be best just to make it always like that (having lots of tabs probably isn't going to be normal)

Yeah, and also having tabs changing places is bad for "muscle memory" when we use the GUI.
And with group colors, we probably just need to see the first letter of a group name to recognize it.

Remember this old Chrome tab design:
https://www.reddit.com/r/chrome/comments/11sfwca/remember_this_layout/
I realise this is probably not possible to do here.
Just wanted to share the idea / mockup.
It would make the active tab more noticeable, and it would look cool :)

I prefer the current design (color highlighting + horizontal line are more than enough to notice them)

I prefer the current design (color highlighting + horizontal line are more than enough to notice them)

Yeah, the current tab design is functional.
And I suspect the other design will be very hard to do.
But, if it's easy, we could have it as an option.

To improve the current design, the bottom corners could be made smoother between the tabs.

Yeah, outflares like that turn out to be very non trivial...

To improve the current design, the bottom corners could be made smoother between the tabs.

Yeah, outflares like that turn out to be very non trivial...

Yeah, that's why I thought the other design would be hard to make.

What if the non-selected tabs were "floating tabs" ?
This design would be easier to do, and it would make the selected tab more noticeable.

That looks possible... I'll take a look.
But I also want to get into the snapping :)

Looks good !

Now even the multi row layout seems easier to understand !
Perhaps we should keep it as an option, if it does not make the code music more complex.

Group buttons are back! 😆

Yeah, with buttons maybe we don't even need the colored line at the bottom.
This just distracts from the active button.

There are too many icons in my opinion, especially if you want to keep a single row.
I would do like Chris said:

each tab has an 'x' on it (like browser tabs) to close that tab
close the last tab and the controller closes (so maybe no need to have the controller 'X' any more)

But I would make the “x” appear only on mouseover (or even when the mouse hovers on the right side of the tab/button?)

There are too many icons in my opinion, especially if you want to keep a single row.

each tab has an 'x' on it (like browser tabs) to close that tab

Yeah, but if we make all tabs larger to fit the 'x' buttons, we have even less space for a single row.

But I would make the “x” appear only on mouseover (or even when the mouse hovers on the right side of the tab/button?)

That could be a good solution, but when we mouse hovers the tab it would need to increase size to fit the button.
And the button text may also need to shift to accommodate the "x" button.
That may somewhat confusing ... I would need to try it ...

Yeah, but if we make all tabs larger to fit the 'x' buttons, we have even less space for a single row.
That could be a good solution, but when we mouse hovers the tab it would need to increase size to fit the button.
And the button text may also need to shift to accommodate the "x" button.

I would simply make the x appear on top of the title, no need to reserve a space for it

I would simply make the x appear on top of the title, no need to reserve a space for it

That could work !
And it could also show the button to bypass.
The omitted group tab could use the bypass color  ( on the whole tab or maybe just as highlight ) ( not pictured ).
If mouseover it's not possible, we could use Ctrl+click to show it, like the slider edit menu.

I would simply make the x appear on top of the title

Honestly, I meant another thing with my comment, I should have said "over the title" (the x still inside the tab)
But I like the concept you made (and removed 😆 )! More space and less visible icons!

What if all icons (except for +) are moved to a new bar right under group tabs/buttons?

What if all icons (except for +) are moved to a new bar right under group tabs/buttons?

I think a row just for buttons would waste a lot of space.
I would probably need 3 rows, just to have 3 tabs.

Well, it's not that bad! Buttons would waste space anyways, at least this way they are more accessible.
P.S. You forgot the X in the mockup

P.S. You forgot the X in the mockup

Ah, OK, so this would replace all overlaid buttons.
Here it is:

Here is a single row, with all 5 buttons.

Nice!
Maybe in the future there will be space for other buttons, like show/hide hidden widgets!

https://github.com/chrisgoringe/cg-controller/issues/143

(Pythongosssss made a PR for this feature in ComfyUI)

https://github.com/Comfy-Org/litegraph.js/pull/250

Yes.
Hopefully Pythongosssss's PR will get adopted and we can just riff off that!

I implemented (just visually) and I find the minimise and remove ( _ X ) buttons being on the second row really feels wrong (they are window actions, they are always top right).
So it's slightly less tight use of space, but this just feels better:

will see what it's like with multiple tabs...

So it's slightly less tight use of space, but this just feels better:

Yes, it does look better.
I think it's worth it to lose that extra space.

There is enough space for other buttons in the future, like show/hide hidden widgets!

Do we really need a separate button for that?
If you want to see "advanced nodes", you would probably also want the "advanced widgets".
The bolt icon could enable both at the same time.
If we had 2 buttons I feel like I would just mistake one for the other, all the time.

OK, this is now on 1.3

One thing I've added having used it:
if there is only one tab, clicking on it allows you to select a different tab
This is because I find I kept on adding a tab and then switching back to remove the old one (esp. on new controllers)

Awesome! 🎉

if there is only one tab, clicking on it allows you to select a different tab

I'm not able to do this.
Clicking on the ( single ) tab, has no effect.

Same as @JorgeR81

But a better solution could be to have a button to replace tabs.
It could work even if there are multiple tabs.

Yeah, that would be nice, especially if you opened many tabs and you want to replace the first one

I found a bug: the image in previous Controller get resized when you create and click on a new Controller window
https://github.com/user-attachments/assets/6cd1b050-c119-4119-a765-a9622432c43f

Sorry - I hadn't pushed my latest changes.... updated now!

Raised #264 for the image resize bug

if there is only one tab, clicking on it allows you to select a different tab

This works very well. We could keep this.
And the 'replace' button will only show up, if there is more than 1 tab, in the Controller.
( Just the same way as the "Advanced Controls" button only shows up, if there are advanced control nodes, in the group ).


[Scenario] Manor of Ravens campaign (Descent)

What kind of mission do you want to add (mark with an X, e.g. [X] Descent)?

[ ] Mansions of Madness
[x] Descent

Please enter the GitHub Url to your scenario here
https://github.com/ValkDes/ValkyrieMoR
@scrubbless Can you please add it to the mission list?

I've added this, should be listed within the hour


feat(#37): Implement entire optimization pipeline

Implement entire optimization pipeline: bytecode -> xmir -> bytecode -> run optimized bytecode
Closes: #37

PR-Codex overview
Focus of this PR:
This PR focuses on making changes to the code in order to improve the functionality and readability of the project.
Detailed summary:

Updated the BytecodeIrTest class to use the correct XPath expression.
Added the Base64 import in the XmirIr class.
Modified the verify.groovy file to invert the assertion.
Updated the XmirFootprint class to use stream operations.
Modified the OptimizationTest class to use Paths instead of File.
Added the Base64Bytecode class.
Updated the BytecodeIr class to use byte[] instead of Input.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 2min)


ram consumption

Hi Michael
A collaborator has to analyze 3000+ files ( dda orbitrap so around 700MB - 1.5GB per file )
FragPipe ran out of memory with 1TB on a VM during quant....
I was wondering wherher SAGE might be an option here.... I am confident speed would not be an issue :)
As I understand, SAGE does not write intermediate results, but rather stores everything in memory. Would this cause proble,s with so many files or would you recommend a specific procedure how to deal with this?
I quickly went over the intro and the github issues but did not see anything related to ram requirements for large scale analyses.
Any help would be appreciated!
best Klemens

1TB might work if you're not doing anything crazy with semi-enzymatic/HLA/etc... I think the largest I have done in a single search was ~2k files.
Are they wanting to run a label-free quantitative analysis where MBR is needed, or is it possible that the files could be processed in batches? If you don't need MBR, you could search each file individually or in batches (e.g. 96 at a time) and then use Mokapot or percolator to control FDR across the experiment. Sage will handle predicting/aligning RTs (used for rescoring PSMs) & global FDR for you, but there aren't any other substantial differences between searching all files at once and individually - and I imagine that the difference between searching 512 files and all 3000 at once will be very minor and within acceptable error.
Even with MBR, I would suggest assessing whether moderately sized batches are OK (I would imagine very few peptides are lost going from 3000 -> batches of 512, for example)

Hi Michael
Thanks for the swift reply and the support. Really means a lot!!
I think if possible they would love to do standard tryptic no crazy modification and MBR quant.
That being said I would propose that they stay with the 1TB setup for now and just try the 3000 with MBR.
If it crashes then all files without MBR
And additionally 512 batches with MBR including some same files in all batches as reference.
Then simply check both the reference files and the nonMBR quantization s whether the 512 batch MBR runs are behaving as expected.
If this is the case then stitch everything together with additional tools for adjusting FDR.
Would you think this would be viable as a strategy? I would report back once we have the results as this might be of interest to others.
Best and thanks again for your time and this cool tool.
Klemens

I think it would be viable! If it doesn't work, please reach out and I can try to implement some other kind of solution


Udacity examples should be improved (/tensorflow/examples/udacity)

https://www.udacity.com/course/viewer#!/c-ud730 is a really cool course in the context of Machine Learning and TensorFlow. After completing Coursera Andrew Ng course, they give valuable reasoning why and how something should be done in TensorFlow.
But, there is an issue.
Theory part in videos and implementing it in TensorFlow do not click together. Forum is full of questions in the format "how should it be done?", "am I doing it correctly?".
Yes, I can spend 2 days on a one assignment, search internet and take parts from manuals that are on the tensorflow.org page, but still I am not convinced that I am doing it correctly.
Please improve https://github.com/tensorflow/tensorflow/tree/master/tensorflow/examples/udacity so that best practice, how previous task would have been correctly solved, is included in the next task.

+1

I agree.

There isn't going to be any updates to the course from the instructors beyond fixes at this point.


Failing acceptance tests

Pended the acceptance test SmokeTestLong - Verify that gitops can deploy an app with long name, which was failing here with:
• Failure [302.259 seconds]
Weave GitOps Add App Tests
/home/runner/work/***/***/test/acceptance/test/add_tests.go:29
  @skipOnNightly SmokeTestLong - Verify that gitops can deploy a single workload to multiple clusters with app manifests in config repo (Bug #810) [It]
  /home/runner/work/***/***/test/acceptance/test/add_tests.go:1404

  Unexpected error:
      <*errors.errorString | 0xc000139ab0>: {
          s: "Failed to delete app test-app-dvnup739",
      }
      Failed to delete app test-app-dvnup739
  occurred

  /home/runner/work/***/***/test/acceptance/test/add_tests.go:1522

Its failure was blocking one of the checks that are required for branch merging, which was in turn blocking https://github.com/weaveworks/weave-gitops/pull/1445 from being merged.
Seems related to #1449 and other v2 work.
Edit : [Gini]
Another test seems to be failing when I tried to fix the broken main https://github.com/weaveworks/weave-gitops/pull/1458
https://github.com/weaveworks/weave-gitops/runs/5221157992?check_suite_focus=true
• Failure [155.910 seconds]
Weave GitOps Add App Tests
/home/runner/work/***/***/test/acceptance/test/add_tests.go:29
  @gitlabTests Test3 - Verify that gitops can deploy and delete a gitlab app in a subgroup [It]
  /home/runner/work/***/***/test/acceptance/test/add_tests.go:1169

  Unexpected error:
      <*errors.errorString | 0xc00043f260>: {
          s: "Failed to delete app test-app-idejv7a5",
      }
      Failed to delete app test-app-idejv7a5
  occurred

  /home/runner/work/***/***/test/acceptance/test/add_tests.go:1236
------------------------------

This ticket needs some investigation on why these tests fail. There's a suggestion about increasing wait time by @rokshana-b  https://github.com/weaveworks/weave-gitops/pull/1458#issuecomment-1042002113 , should be considered as part of this investigation. These all seem to be failing in the same file/suite.

Closing this issue since these tests no longer exist.


feat: Implement Python client code generator

Closes #894
This adds a custom code generator for the Python client. The mapping between GraphQL types, model types, and fields is somewhat explicit, but doesn't require too much code. The code generation parses the core GraphQL python library (which we already depend on) and uses Jinja2 templates to populate the models module.
I tried some existing packages for generating Python GraphQL clients, and my sense was that writing something specific for this client is probably easier than learning how to configure those tools for this particular client. This approach also has the advantage not imposing its own client and query style, which makes this compatible with the existing API by design.
I have done a little manual testing, but testing is mostly through the newly modified or added Python tests. These include checking that missing item relationships are handled gracefully by the client, which is at least a temporary requirement as the ingestion runs and may be a more permanent one.
As the prod GraphQL server has been updated ahead of the Python client, these changes also include the corresponding changes to the models module and GraphQL schema file. As well as the associated test infrastructure changes.

A few things are blocking this from be reviewable and thus mergeable.

I should increase code coverage by covering the code generator. I think we should be able to reuse the existing test infrastructure to do this in a fairly meaningful way without needing to make new complex mocks or stubs.

It seems like the deposition_id can be None in production. Assuming this is not temporary, I need to add some logic to the relationship field types to handle that case. We should also have at least one test to cover this too.

I need to determine what the type and value of Annotation.method_links should be (i.e. str vs. dict) and if that needs any special handling for the dict.

Ideally this PR would be much smaller, but I'm not sure how easy that is given that this mostly generates code/data and quite a few things have changed on prod.

This is ready for review now.
The PR is much larger than I'd like (though almost all the delta is auto-generated in _models.py, schema.graphql, and up.sql), but I am unsure how to split this up without setting up new fakes/stubs that may take a significant amount of time and probably are not worth it in the long term.
There are two practical issues blocking this from being merged.

The tests I added exacerbate the flakiness described in #942, so we should almost certainly fix that before merging this.
One of my fancy tests that programmatically imports a temp module seems to break codecov. I should either change the test or find the right way to make codecov ignore that module.

This looks great! I think we'll need to update the docs templates to add depositions / deposition authors when we get a chance but that can be handled in a later PR.

If there's no existing issue I'll add an one after merging this to make sure that doesn't get lost.

One of my fancy tests that programmatically imports a temp module seems to break codecov. I should either change the test or find the right way to make codecov ignore that module.

I added a .coveragerc file to the python client directory and explicitly ignored the temp module I create. That's possibly sensitive to changes, but I don't want to ignore things that should not be ignored.


Add New Categories & Words

This fix the issue #13.
I have added

[x] new category K-POP as you inspired from Red Velvet
[x] words in food.
[x] words in transportation.
[x] words in place.

I am a little bit confused about the players' familiarity with some words & K-POP. Feel free to ask to make changes 😁. I have no problem.

Welcome. Sure tell me the changes!


Fixups for modern go

This bring the repo up to speed for modern golang:

add go.mod
switch to new-style go:build tag

Corresponds to / based on https://github.com/syndtr/gocapability/pull/22

https://github.com/syndtr/gocapability/pull/22 was previously LGTMed by @avagin and @hallyn


Bittrex Insufficient Funds

System information

Have I written custom code (as opposed to using zenbot vanilla): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Debian 9 Stretch
Zenbot version: 4? Master branch latest version
NodeJS version: 8.9.1
Python version: ?
Exact command to reproduce: ./zenbot.sh trade bittrex.BCC-USDT --strategy atr --period 4h --buy_pct 30

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in Zenbot or a feature request.
Selling on bittrex results in "INSUFFICIENT FUNDS" message, and sell does not go through.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

I know @KryptoNova made a fix before that did help with this, which worked almost all the time. However it has failed a few times and I was using sell_pct 100. Thanks.

I noticed that the problem occurs after the bot put up a buy order and couldn't purchase at the required price. In the end, the bot does not cancel the order and you receive this error.

@brucetus, unfortunately, the problem that I had was related to the fee not reducing the Size of the asset purchased.  That caused an insufficient funds message, because there was not enough BTC to cover the exchange fee.
It could be that cancels and sells are happening so fast that bittrex cannot handle the encumbering and unencumbering of BTC.  That would be controlled by the order_poll_time and wait_for_settlement values that you have setup in your conf.js.  The "selling" being in a less than a second interval, is an indicator that this may be the case.
Could you provide what values you have for  in your order_poll_time and  wait_for_settlement  conf.js as that may be a clue?

Yes, my order_adjust_time, order_poll_time, and wait_for_settlement are all at 5000 ms. I'm trading on 4h period with poll_trades at 60000.

When you logged into Bittrex, was there still a pending sell order that needed to be cleaned up manually?

I wish I could say I remember for sure, but I believe there was no open order on Bittrex.

By chance, are you monitoring more than one trading pair on Bittrex, but only one of them is doing live trading?
For example, you may be trading a selector, but monitoring several other selectors on concurrent running Zenbots from the same public IP address.  That can lead to Bittrex starting to return blank documents as that is how they handle too many request errors, I suspect.  That is what I was finding with #659.
The next time you get one of these, could you check to see if there is a pending order out there?  That would help us determine if @kazdimenator's comment is where the problem lies.
I have some ideas how to handle both, but would only like to introduce code to fix one or the other situation.

Yes I should've mentioned that I am running multiple trade pairs, my bad. At times some are doing live trading, and some are in manual mode just to get data in case I want to turn them on (since you can't backfill with Bittrex). If I get the error again I'll check the site to see if there are any open orders. I'm running on both sell_pct 99 and 100, to see if it is related to that. Thanks.

I think the multiple trading pairs is contributing to this, unfortunately.  That is what I was getting with the fix I put in for #659.
I may have to think about this one a bit to see if there is a way that there is a way to recover these.  It may be possible to introduce some way of recovering.  Your findings on the pending sell order is going to be most valuable in determining the place where the code needs to be addressed.

Hey KryptoNova, I can confirm that no orders are open after the error occurs.

Interesting.  I wonder if it is almost something on Bittrex's end.  It seems that the cancel order is being registered, but Bittrex is not disencumbering the funds quick enough.  Either that or the rounding is 1 satoshi off causing the error.  It may be that a debug message needs to be output to see what the price and size of the sale is prior to issuing to the exchange.

However, I think I am no longer going to be working with Bittrex and someone else may need to pickup on a PR for this.  I just tried to withdraw my coins to avoid Zenbot from investing a windfall sale I made on a long held asset.  When I did, they said I can not withdraw due to not being validated and I have had an account there without problems for over two years.  Essentially, they have stolen my coins unless I give them my personal information.  To get a withdrawal cap of 0.025 per day with .001 transfer fee, I have to give them my full name, address, and birth date (Basic Validation).  To get a 100 Btc cap, I have to give a selfie with government issued picture ID (Enhanced Validation).  I will probably do the basic, if I pass, then take a beating on transfer fees and get my coins.  The enhanced is a hacker's dirty dream, to get a picture of me with personal information holding my government picture ID in a selfie.
I would recommend that @DeviaVir really look into the negative comments associated with Bittrex within the last 30 days and see if it is something that Zenbot should really be supporting.  Good luck to all that continue to work with Bittrex as I am not getting warm fuzzy feelings from them.

Fair enough. I don't think we should stop supporting Bittrex considering it is one of the only exchanges we have that lists as many coins as it does, and it works better than many other exchanges in Zenbot. Do you think you can give me an idea on what this problem might be due to so that I can try to fix it?

Oh yeah and another idea if you want to fix the HitBTC API, it has similar coin selection to Bittrex, but the API still needs fixes, I know a bit about it so I might work on that instead.

I really think the problem with Bittrex may actually be related to some server problems they are having. I saw on twitter they are dealing with infrastructure problems.
That in mind, I think that they are still locking the funds of the previous cancel. I think you may need to increase wait_for_settlement to higher than 5000. This does have the side effect of increasing the chance for slippage.
If I can get my coin from Bittrex, currently holding 99 percent of my BTC due to the windfall, I will look into HitBTC. However, it will be some time, if at all, as I hear basic validation always fails and I will have no way to get my coin from them. So depressed.

This issue was moved to highvelocityspace/zenbot#43


footer added icons

Closes: #364

Title : Adding more social media icons
Name: : Jayanth Thalla
Idenitfy yourself: :GSSOC Contributor

Describe the add-ons or changes you've made 📃
I have added the social media icons like Instagram, discord, and Facebook.
How Has This Been Tested? ⚙

I tested on the localhost on the browser
Screenshots 📷

Type of change ☑

What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x ] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x ] This PR does not contain plagiarized content.
[ x] I have performed a self-review of my own code.
[x ] I have commented my code, particularly wherever it was hard to understand.
[x ] My changes generate no new warnings.

Note to reviewers 📄

@jayanththalla you have search my profiles to add the proper links.

in angular webmaster log icons are not there

@jayanththalla footer section is single so resolve it by yourself


Client side validation with zod: Errors don't get cleared when using 'regex' and 'optional' together

Description
Once you inserted an invalid value into a field with regex and optional in the schema, the error does not get cleared when you empty the field and thus the form is invalid until you insert a valid value.
If applicable, a MRE
https://stackblitz.com/edit/superforms-2-testing-dmk3cn?file=src%2Froutes%2F%2Bpage.svelte

An empty field is the same as an empty string. Use a stringProxy to set it to undefined when empty: https://stackblitz.com/edit/superforms-2-testing-ns3m6n?file=src%2Froutes%2F%2Bpage.svelte

Shouldn't that be the default, when using optional?

There is no way to make it default if you use bind:value, as it will bind to the input field value, which is an empty string when empty.


Export tikv status port in CI

What problem does this PR solve?
Issue Number: close https://github.com/pingcap/tiflash/issues/8023
Problem Summary:
All CI failed after https://github.com/tikv/pd/pull/6945, the pd coordinator can not start because the pd can not connect to tikv status port.
What is changed and how it works?
Export the status port of tikv in CI
Check List
Tests

[ ] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-integration-test

/run-integration-test

/run-all-tests

/run-integration-test

/run-all-tests

/run-unit-test

/run-unit-test

waiting https://ci.pingcap.net/blue/organizations/jenkins/tiflash-ghpr-unit-tests/detail/tiflash-ghpr-unit-tests/11180/pipeline


Make edit buttons more visible and consistent

Add yellow color to edit button in event list
Add edit button to event detail view
Maintain consistent styling across both views

This pull request improves the visibility of the edit button.
Closes #35
Previous Button
https://github.com/DormSoup/dormsoup/issues/35

After change

@gabrc52
I am not an MIT student, but I decided to address this issue. Since I only read the code and implemented the changes, there may be some issues with the actual behavior on the screen. I wish your project great success.

I'm really appreciative of your quick response to my issue and still impressed that you came upon our repository. My main feedback would be to choose a color with more contrast rather than yellow (based on the first screenshot), and to add the word "Edit" like the other buttons (in the second screenshot). Thanks a lot!
After that, if @lad6558 or @mitabreu don't have other comments, we can merge 🎉

Thanks for testing it in your local environment.
I have tried to accurately reflect your feedback in my development.
Improved Event Detail Bottom Bar Layout
Changes Made
1-1. Modified the bottom bar layout in EventDetail.tsx to:

Add "Edit" text to detail view button
Use w-1/3 width for buttons when edit access is available (3-way split)
Use w-1/2 width for buttons when no edit access (2-way split)
Adjust border and border-radius styles conditionally
1-2.
Change edit button color to red for better contrast
(I'm not sure if this is the right color.)

Improved UI consistency:

Removed empty space when edit button is not present

Technical Details

Added conditional classes based on event?.editable
Adjusted border styles using border-x-[1px] or border-l-[1px] based on layout

Visual States

With edit access: Like | Add to Calendar | Edit
Without edit access: Like | Add to Calendar
Both layouts support liked/unliked states with proper styling

Also @gabrc52 thanks for testing locally and providing feedback

And thanks for the support @maurayonori

Thanks so much again! I've merged your PR 🎉. And good luck in your future endeavors!


chore(main): release 0.20.0

:robot: I have created a release beep boop
0.20.0 (2024-12-12)
✨ New Features

Created Range Selector (7257c40)
UX-881: Range Selector (#217) (7257c40)

🪲 Bug Fixes

Fixed issue where avatar lower badge was not showing (#218) (a1742a3)
removed defaults to true on showValues (7257c40)
removed large thumbs (7257c40)
removed rounded from widgetbook (7257c40)
Top app bar padding (#215) (7b374b7)

📈 Documentation

widgetbook and example for range selector (7257c40)

⛓ Dependencies

automated: Update icons (#219) (1bd8bfa)
automated: Update icons (#221) (10cc97c)

🧪 Tests

adjusted golden tolerance (7257c40)
created interaction tests (7257c40)
testing range selector (7257c40)
wrote 1 test for range selector (7257c40)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.20.0
:sunflower:


mass upload via client produces MKCOL errors

Test setup via https://gitea.owncloud.services/jw/hetzner/src/branch/master/make_ocis_test.sh
git clone https://github.com/owncloud-docker/compose-playground.git
cd compose-playground/compose/ocis
sed -i -e 's/your-url/$ipaddr/g' config/identifier-registration.yml
echo >> .env OCIS_BASE_URL=$ipaddr
echo >> .env OCIS_HTTP_PORT=9200
echo >> .env OCIS_DOCKER_TAG=1.0.0-beta4
docker-compose -f ocis.yml -f ../cache/redis-ocis.yml up

connect user einstein with the desktop client.
wait until the initial sync is done.
Then copy a folder with ca 30 jpeg photos into the sync folder of the desktop client.
cp -a ~/testpics ~/ownCloud2

client log
  5/12/20 3:29:18 PM,                          testpics/photo_2020-05-11_18-10-59.jpg,                     ownCloud2,Server replied "404 Not Found" to "MKCOL https://<IP_ADDRESS>:9200/remote.php/dav/uploads/einstein/1650630572" (skipped due to earlier error, trying again in 1 minute(s))
  5/12/20 3:29:18 PM,                          testpics/photo_2020-04-29_17-21-20.jpg,                     ownCloud2,Server replied "404 Not Found" to "MKCOL https://<IP_ADDRESS>:9200/remote.php/dav/uploads/einstein/3339200750" (skipped due to earlier error, trying again in 1 minute(s))
  5/12/20 3:29:18 PM,                          testpics/photo_2020-04-29_17-21-15.jpg,                     ownCloud2,Server replied "404 Not Found" to "MKCOL https://<IP_ADDRESS>:9200/remote.php/dav/uploads/einstein/3029869760" (skipped due to earlier error, trying again in 1 minute(s))
  5/12/20 3:29:18 PM,                          testpics/photo_2020-04-29_17-21-03.jpg,                     ownCloud2,Server replied "404 Not Found" to "MKCOL https://<IP_ADDRESS>:9200/remote.php/dav/uploads/einstein/31011025" (skipped due to earlier error, trying again in 1 minute(s))
  5/12/20 3:29:18 PM,                          testpics/photo_2020-04-29_17-20-50.jpg,                     ownCloud2,Server replied "404 Not Found" to "MKCOL https://<IP_ADDRESS>:9200/remote.php/dav/uploads/einstein/2735690951" (skipped due to earlier error, trying again in 1 minute(s))
  5/12/20 3:29:18 PM,                          testpics/photo_2020-04-29_17-20-21.jpg,                     ownCloud2,Server replied "404 Not Found" to "MKCOL https://<IP_ADDRESS>:9200/remote.php/dav/uploads/einstein/1239652929" (skipped due to earlier error, trying again in 1 minute(s))
....

After a few seconds the red error messages are removed from the client settings window, and the green checkmark indicates, that all was synced to the server:

This indicator is wrong. Only the folder was created at the server. Nothing is uploaded. Checked with web UI.
(@TheOneRing the client should not show the green checkmark, when the uploads actually fail)
Ater a few minutes, the same uploads are attempted again. Again failing, reported with a larger retry timeout then.

@jnweiger any file > 10MB? I'd guess https://github.com/owncloud/ocis/issues/194

no. my jpegs are 3 to 4 mb. There was one 14MB .mp4 in the folder when I started, But I carefully removed that, to not trigger too many bugs at once :-)

@jnweiger can you check the ocis logs if you are also running into #268 ?

I am getting multiple different errors now....

Will try again later, to reprodce correctly, then I'll attach a log.

@jnweiger can you still reproduce with the latest 2.7 daily builds?
Please always specify the exact client build information.

Retested with rc1 and desktop client beta4: upload of 1000 random files succeeds. closing here.


Score rendering error

Error with files: A26, A27, A28

Issue is related to missing files on server (deploying deletes user uploads)


Fail to add a Team-Repo

While trying to assign a repository to a team I got the following error:
{"err":"Team does not exist","ok":false}
Enviroment:
Ubuntu 14.04
Gogs installed from binary Version: <IP_ADDRESS>7 Beta
Log:
2014/10/06 19:44:32 [T] Organization created: blub
2014/10/06 19:44:53 [T] Team created: blub/bla
2014/10/06 19:45:14 [T] action.NewRepoAction: blub/blahaaaa
2014/10/06 19:45:14 [T] Repository created: blub/blahaaaa
2014/10/06 19:45:27 [teams.go:136 TeamsRepoAction()] [E] Action(add): Team does not exist

I run in to this trying to add an existing repository.
Using: gogs_v0.5.9_linux_amd64.zip

@dromer what if you create a new organization?

Hah. Indeed now it does. That's odd.
To recap:
So I created a new orga. Made a repo for this orga. Then created the team, and added the existing repo to the team.

Ok, I found out how to do it: it's (currently) only possibly to add repos to empty teams.
So, removing all users from a team, then adding an existing repo, then adding members. Works.

@dromer great! Glad to hear it!

Well, it still is a bug.
Hope to see it resolved in 0.6.0 ;)

I see what you mean now... So if you try to add repository to non-empty teams, this bug will be reproduced?

Yup, that seems to be the current behaviour in 0.5.9

https://github.com/gogits/gogs/issues/831 I created a new issue here to keep a record. @dromer


gflags can not be used by a cmake project

gflags does not properly fix up its cmakelists file, so it can't be used by a downstream cmake project. Additionally it deletes the debug cmake config file, so debug builds will not link to the debug version.

I though issue with debug build was already fixed

The debug build does work, the issue is that there isn't a share/gflags/glfags-config-debug.cmake, so find_package(gflags) won't work very well in debug mode.


"Awake" taskbar tray menu is not translated into Turkish

Microsoft PowerToys version
0.74.0
Utility with translation issue
Awake
🌐 Language affected
Turkish
❌ Actual phrase(s)

Taskbar tray menu is not translated into Turkish.
✔ Expected phrase(s)
Translate the Wake Up taskbar tray menu correctly.
ℹ Why is the current translation wrong
No translation!

dup 11456


fix: Fix ssl certificate recreation

fix: Fix ssl certificate recreation

The apply pointing to the branch with the effective change
https://app.terraform.io/app/weights-and-biases/workspaces/customer-merantix/runs/run-mLk6GepMnB3P44QC
The new run without any change to validate the change will not recreate the cert repeatedly, only if the subdomain changes.
https://app.terraform.io/app/weights-and-biases/workspaces/customer-merantix/runs/run-QqhbxuCRBxMxcYpU

Here is a new run changing the subdomain because it was incorrect was incorrect.
https://app.terraform.io/app/weights-and-biases/workspaces/customer-merantix/runs/run-6eUp7jxoKEHquamm

This PR is included in version 1.19.2 :tada:


DBZ-848 Preparing Docker example image for MongoDB for a replica set

Basis for MongoDB replica set example

Yes, that's of course an option. I kinda hoped there was a way to do it
just within Dockerfile, but I'll go for the script then.

Closing as this might be slightly outdated now in 2024 :D


Get the check-in count from the BatchRegistry contract

Goal
There is a "To Be implemented" string in packages/nextjs/app/page.tsx that should show the number of builders that have checked in to the contract.
We want to show the real count (reading from the BatchRegistry contract)
For more info about the contract: #10

Is Someone working on this one ?

This is only to get the Checkin Count .
So maybe it should be added to the main page tweaks only, where we are listing all the team members that checkin ?

ok

so this checkin counter is an open issue right ?

Yes definately!

Can I take this on? I see it's still open.

Yes, go for it!


BugFix #276 [Pharmacy Services] Negative values for Dose and Duration

BugFix #276 [Pharmacy Services] Negative values for Dose and Duration

Coverage increased (+0.1%) to 33.907% when pulling ce5d0794885da32ad7b29396e2168c56b3ab2553 on Guimino:master into 4b19aa49fd3403f55140a85a00d82fc50eba7558 on esaude:master.


[ReadMe]: Change transformer to transform prop name

Small correction to the docs here, looks like the property name should be transform here, otherwise the registerTransform function throws transform must be a function error.

Thanks, yes, this has changed since Style Dictionary v4.

Thanks!


Workspace settings "cmake.configureSettings" not recognized

Brief Issue Summary
User settings:
"cmake.configureSettings": {
	"FUNK" : "meh"
}

Workspace settings:
"cmake.configureSettings": {
	"FUNK" : "yasss"
}

And inside cmake file:

message( "Do you FUNK? ${FUNK}" )

Expected:

Do you FUNK? yasss

Apparent Behavior:

Do you FUNK? meh

Platform and Versions

Operating System: Windows 10
VSCode Version: 1.30.2
CMake Tools Extension Version: 1.1.3
Compiler/Toolchain: Visual C++ 2017

Other Notes/Information
Moreover, if I clear user settings I get no FUNK at all. It's not a critical issue for me (because what I want to put into configure options - OpenCV, boost, other libs paths and include dirs - is shared and of the same version for all the projects), but is worth looking into, as I gather.
Thank you for making and maintaining this extension! It's such a relief to escape from cumbersome, heavy full-blown Visual Studio into Code :)

I wasn't able to reproduce the problem with the latest version of VS Code and 1.1.3 of the extension. Please let us know if you continue to have a problem with this.

I'm having exactly the same issue a year since this one was closed, so I suspect it still exists. Just like @jclay mentioned, only resetting state or reloading window helps CMake to pick up the new configureSettings - otherwise they seem to be cached and any changes are ignored when running "CMake - Configure".

We're tracking this with #1127

Thanks, I'll subscribe there.


[docs] binarycaching.md: NuGet requires to set API Key for source

Describe the pull request

What does your PR fix?
Fixes #21851

Which triplets are supported/not supported? Have you updated the CI baseline?
No

Does your PR follow the maintainer guide?
Yes

Can you please sign CLA first?

I am waiting my company approval

Ping for response.

Closing this PR since it seems that no progress is being made. Please reopen if work is still being done.

Please click "Ready for review" if this PR is ready.

Ping for response again.

Thank you!


how to disable broker ?

i dont want use broker when create micro service , how to do it?

There's no way to explicitly disable the broker. It's part of the tooling. If you don't want it to start a http server just set it to the memory broker.