Feature to score team dashboards

Feature to score team dashboards

Overall score for a dashboard between 0-5
Score for individual widget data: build, scm, quality, deploy

Changes include

Score collector to calculate score for enabled dashboards (collectors/misc/score-collector)
UI widget to display and configure score
Core models, repositories & API changes

Score Collector:

Collector will run periodically to calculate score for enabled dashboards
Configurations for score calculation can be set at default and individual widgets
Configurable criteria to set score if it meets a threshold.

Flow to view score for a dashboard

Enable score widget by clicking on configure and save

After score collector has run, you should see overall score for the team dashboard

Details for score can be viewed by clicking View Details

Individual widgets will have score view, and clicking on it will show details of the score

@satishc1  @tabladrum  fyi, the PR to score team dashboards.

+1

@nireeshT Please pull this pr down and lets discuss.

@satishc1  @tabladrum  looking into it. @sidbha007 Good stuff !!

@sidbha007   when I click on configure (gear button) on score widget, modal is displayed with save button. Will there be any configure steps listed or am I missing something or is it expected behavior.

@nireeshT yes, the configure screen currently just enables scoring for the dashboard by clicking on Save button.
I think adding some explanatory text might help "Click on save to enable scoring for your dashboard". If you have any ideas let me know, will fix it. thanks
We had a way to configure scoring using custom Or default configuration.
The default was picked up from configuration in scoring collector, and custom from configure screen.
We removed custom option from configure screen, as while comparing dashboard scores default configurations made configs same for all.
I did check-in the custom-config html file we had, but its not referenced in config.html : https://github.com/walmartlabs/Hygieia/blob/0c021beb2f9651de12aa38b3040a0f0e8a93fecd/UI/src/components/widgets/score/custom-config.html

@sidbha007  - yep, got it. Scoring makes more sense when it is configured at the instance level. Just a thought, can we have score widget as view only on dashboard and enable widget on Administer dashboard modal by creating a new tab "Settings" -> Score (checkbox) on/off.

Also can we persist all scoring properties into Mongo via JSON  when ever collector starts and read the criteria from Mongo to calculate score. You have to setup rules in json and read through them to calculate metrics. Check out these files.
https://github.com/capitalone/Hygieia/blob/master/UI/src/test-data/cicd-gates.json
https://github.com/capitalone/Hygieia/blob/master/UI/src/app/dashboard/core/data-factories/cicd-gates-data.js

@nireeshT having it enabled in administer dashboards sounds like a good idea. Thanks, will look into it changing to this.
For your second point, I am not sure if I fully understand it.
Are you suggesting that scoring configuration is persisted in mongo, and score is calculated on UI based on the config? Like, we fetch the config and use the data in the dashboard to calculate score. If you can explain more, it would help. Thanks. I'll also look at cicd gates
In current implementation, score is calculated and persisted for each dashboard and available via apis for each dashboard

The score apis for dashboard can be used in other places like Product dashboard, and we have internal team dashboards where the scoring is displayed. So, it doesn't rely on dependent data used to calculate score
The time range could be flexible, e.g we can use 30 days instead of 14 days to calculate score without changing in UI

@nireeshT thank you for reviewing 👍
Cool, so you are suggesting to have widget id in scoreWidgetMetrics instead of name.
So, reason had kept id as widget name, as was thinking the score type could be for Dashboard, Team & User in future. With ids as "build, deploy, quality, scm", it works for any type. But, having widget id might be good for dashboard. Am, looking at it on how to change it to have widget.
So, for the other point to calculate scores by persisting in mongo, are you thinking something like this

Collector starts and persists configuration
The same configuration is used to calculate score for dashboards in collector
have a api to view configuration
S0, by saving configuration we can read it from mongo using a api, instead of looking at config file. Just wanted to be clear, if thats the intent.
will look at these, thanks

@sidbha007  - yep, that is what I asked for, when I meant persisting config into Mongo.

@nireeshT i have added few changes, if you could have a look. thanks

Score metrics has field refId which will have widget id. In UI, refId is used to attach score for a widget
Scoring settings are now saved in collection score_criteria_settings and used to calculate score + API to access it
https://github.com/walmartlabs/Hygieia/blob/cd9133d7ae2d28e976ecaf4407361c370ff5ab31/api/src/test/resources/score-criteria-settings.json
refactoring & cleanup

@sidbha007  cool, will take a look. Can you ping me on https://gitter.im/capitalone/Hygieia, I can brief you about the change which I am working on and we can collaborate.

@sidbha007  - I pulled the latest code base, I see github metrics are missing. Is this expected

@nireeshT i updated the property name in collector from githubScmWidget to scmWidget to be more generic.
Srry, should have added in comments.
Could you check if you are using updated scm property name score.scmWidget
https://github.com/walmartlabs/Hygieia/blob/cd9133d7ae2d28e976ecaf4407361c370ff5ab31/collectors/misc/score/docs/3_widget_all_category_settings.md
thanks

@sidbha007  check these changes for getting score next to dashboard title.
https://github.com/nireeshT/Hygieia/commit/74f09fa52df41e6e052f7e9d95e25a47027f4198
When I remove score widget from Capone template, I don't see individual widget scores. Anyways, these changes are rough draft though. If you see an option of moving the toggle to enable score to Administer- dashboard screen, this kind of display is possible and can get rid of score widget. Let me know your thoughts.

@nireeshT  I thought about it, IMO score as a widget has its advantages in usability
It follows conventions of other widgets which makes it easy to answer some questions

What is this widget for ->  Score (the header says it)
When was it last updated -> DateTime in upper right
How to see details -> "View Details" link is little more obvious
it could be activated/deactivated
The score itself stands out as it has big size, with text for the score say "3.5". It grabs attention
It is little easier to relate that the other widgets that show "star rating" are for the score.

The disadvantage is the cost of space it takes on UI.
I was thinking about the idea to put it besides title.
In the dashboard page, the header has a bit of

Primary navigation (Hygieia Icon)
Secondary Navigation (Widget, Pipeline, Cloud) links
Title of Dashboard
Putting score beside the title makes it bit restrictive in horizontal space. we could only put this much
It is difficult to add text "score" or add few other details.
Comparatively, it makes a bit harder to figure out.

I thought about moving title on the page with score and tabs.
Please check this.

It follows the same conventions and might be little easier to understand but with same space problems, and not as clean when its shown in a widget.
Another option could be to reduce space used by score widget.

I do think it does look cleaner in the widget.
But, I might be biased with the score in a widget as we have used it for a while internally and it has worked.

@sidbha007  - I like both the options but I am some how inclined to first option. Reason being user can create a dashboard with variety of templates. Infact they can create a custom template. If we put the score widget in the existing standard template (Capone.html), they could have a score widget which they might not need on dashboard and there is no way to hide it. I am thinking if we could bring out both the flavors to ground.  With the first option, if you could keep everything intact on header (Hygieia logo, dashboard -title and Widget, Pipeline, Cloud tabs) , use the space where you added title to display - "

Dashboard Score i - "score value
ng-rateit(your version of stars) ". This could be enabled/disabled with toggle from administer dashboard. Also I like the idea of last updated next to score.
With the widget option, we can have a separate standard template - score**Template.html and you can add score widget in the layout. This way we can keep the existing templates intact and can have both the options.

@nireeshT awesome, that's a great suggestion, thanks. i'll work on it 👍

@nireeshT based on our discussion have added changes to make score as a dashboard level configurable setting, with options to select the display in header or widget.
Now, dashboard has new fields for scoreEnabled, scoreDisplay (HEADER, WIDGET)
The score when enabled creates a collector item, and when disabled makes the collector item inactive. it will not be added as a widget in dashboard model

Create Dashboard has option to enable score

The display options

Display in Header : This will display score in a separate header
Display in Widget: This option will display score as a widget in templates. It will be shown as first widget in top-left

Administer Dashboard has tab to view Score settings

Score Display in Header

When score display is selected as header, it will add a header section to display score.
The updated time is shown in right

Score Display in Widget

When score display is selected as Widget, it will show score as a widget in dashboard.
It looks like a widget, but doesn't have other options to configure and delete

I'll do some cleanup today, but let me know your thoughts on this. thanks

@sidbha007  I looked into it, all good. Awesome work!! While you are at cleanup fix this undefined check.

Thank you @nireeshT !!
I have added a fix to show "No Details Found!" message in details pop-up when score data is not available.
If you find any other issues, pls let me know, will fix them today. thanks for looking

@tabladrum @satishc1 LGTM

Looking good. Merge it!


CDK Drag and Drop not working perfectly in multiple rows

What is the expected behavior?
Drag is not working perfectly when items has wrapped.
What is the current behavior?
Drag and drop should work perfectly when items are to be wrapped.
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
7, 7, linux
Is there anything else we should know?
No

Duplicate of https://github.com/angular/material2/issues/13372.


[Bug]: Stretchly is intercepting Play/Pause media key

Version

[X] I'm using version 1.8.1

Known issues

[X] I've checked Known issues

Existing issues

[X] I've checked Existing issues

What operating system are you using?
Windows
Operating System Version
Windows 10 v1809
Reproduction steps
When trying to register a global hotkey in Winamp, it says my keyboard's Play/Pause media key is already registered to another application. I used Microsoft's Spy++ following this guide, and it showed me that Stretchly is using that hotkey, even though these settings are left on default:
"pauseBreaksShortcut": "",
"resumeBreaksShortcut": "",

Expected Behavior
To not use a hotkey it's not set to use.
Actual Behavior
See above. Using the hotkey doesn't seem to do anything in Stretchly itself.
Relevant log output
No response
Preferences
No response
Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Nvm, it stopped doing it O.o

Ok so apparently the media keys only pause/play Winamp, while the Stretchly break window is active.

Hi, can you show me what that app says? Stretchly only registers shortcuts if they are set in preferences. If something gets registered, then it must be some weird Electron.js thing

Sorry, I can't reproduce it right now, it sort of comes and goes 🤔 I can take a screenshot of it the next time it happens.

Got it:

Thanks! So what are those screenshots saying? Was it again when break was active?

Those are the screenshots I got when following through the steps of this guide: https://stackoverflow.com/questions/829007/find-out-what-process-registered-a-global-hotkey-windows-api/43645062#43645062
As far as I understand, in the last window it identifies Stretchly as the process that registered the "Media Play Pause" global hotkey.
The break wasn't active at that time.

I am not setting such a hotkey in my code, but looks like there is an issue in Electron: https://github.com/electron/electron/issues/32385
I don't have youtube videos playing, like in that issue, but I play sounds (ie. Break finished).
Can you try disabling sounds from Preferences -> Theme and see if issue is still there? Restart of Stretchly or maybe better whole computer would be needed I guess to make sure OS doesn't hold that shortcut after Stretchly has been restarted

Stretchly doesn't seem to intercept the hotkey if I disable sounds.
(Not 100% sure it's a consistent behaviour, as the bug hasn't appeared consistently either.)

:+1: Thanks for trying! I guess we'll have to wait for Electron issue to be fixed. Can't think of any workaround right now

So the electron issue has been fixed, so hopefully next version of Stretchly will work fine.

@kazerniel did you have a chance to check new version?

I turned back the sound now and Stretchly seems to not intercept the hotkey anymore. The bug wasn't consistent, so can't be 100% sure it's gone, but for now it seems fixed, and I'll comment here if that changes :)


only phone number can check?

Could you add telegram ID to check it out with phone number and so on   ???

Hi @yanzhang4013 👋 Would you like to check the information associated with Telegram IDs (without having the phone number available)?

Hi @yanzhang4013 👋 Would you like to check the information associated with Telegram IDs (without having the phone number available)?

I think it's a query by ID to get the cell phone number.

Thanks! I think this falls outside the main scope of the tool. Though we would welcome a Pull Request that implemented this feature.


Stack Driver Propagate traceId From Upstream Apps

We have some upstream apps which are coded in Dot Net and we have some other services coded in Spring Boot which runs on top of GKE.
Now the users will interact with Upstream Apps coded in dotnet which in-turn rely on Spring Boot Services for the task to complete.
For Spring boot Apps, am using Spring Cloud Sleuth tracing to trace the service calls flow/journey, but how do I stitch the entire journey right from the users interaction with Dot Net Apps till the final request is complete.
Say for example, if the dotnet apps are using this library https://cloud.google.com/trace/docs/setup/aspnet to instrument tracing and push the same to stackdriver , when the dot net service hit our Spring Boot service, does the trace_id/parent_id will remain the same or since the apps are coded in different languages and are using different client libraries , a new trace_id will be generated when the Spring boot service is called ?
And if yes, how do I avoid this scenario and propagate the same trace_id so that I can see the entire service call journey in stack driver trace UI irrespective of language barrier

The stitching is done via trace propagation in the HTTP headers.
We rely on Zipkin with Brave (B3-) headers.
Take a look at https://github.com/openzipkin/zipkin4net to make your .NET application use Zipkin for trace propagation.

@meltsufin Any idea, If this library http://googleapis.github.io/google-cloud-dotnet/docs/Google.Cloud.Diagnostics.AspNet/index.html internally uses the one you are suggesting https://github.com/openzipkin/zipkin4net , so that by using the google-cloud-dotnet library we can push all the traces to stackdriver

The library you're referring to doesn't seem to be using Zipkin. Nonetheless, depending on how it propagates traces across requests, it may interoperate with Zipkin GCP.
cc: @SurferJeffAtGoogle


No "main" entry point

I can't compile this under visual studios for some reason. It's giving me a no main entry point error. Project apitest, file FXC

i have the same problem did you fix it.

https://stackoverflow.com/a/20365225/1047713


Add a option to show the ship name on the Presence text and add pt-br support

Added related code, and edited the language templates (needs fine tuning).

Hi, apologies for looking at this so late. I will quickly take a look at this.

I May have accidentally mixed the 2 PR that I created here. that's why this one has a ship name field.
If it's a problem, I can try to separate the 2.

It is fine if you submit a single PR with both the ship name stuff and the new translation. Just close the redundant PR to make things clear.

It is fine if you submit a single PR with both the ship name stuff and the new translation. Just close the redundant PR to make things clear.

OK.

I've implemented the changes, but I cannot test it yet, because my dev machine doesn't have EDMC set up. will try to test it later today.

It did not work.
How can I trace errors with EDMC?

Tested today, has some ruff edges, but it seems to work.
The main things  would be:

Unaligned checkbox for the "show ship name"  option
that the ship name displays as None right after logging in and th

Nice work mate! Let me know when you the PR is ready to me merged.

seems to work fine, fixed some dangling {ship} fields
Examples (my friend that had his system in pt-BR tested for me):

From my testing, the ship field makes the text a little too wide, do you think it's a good idea to keep it?
(the images won't show it, because if it sees that the name is "None" it hides the field, maybe it's something with inara)

Maybe put the ship name in a new line?

It seems that it doesn't work:
shipt = " \nwith {ship}".format(ship=state['ShipName'])

It seems that discord prevents you from inserting newlines.
later this week I'll try to will remove the ship text code for now, and push only the translation.
Maybe using the image field I can try to implement this later (in another PR).

Later this week I'll try to remove the ship text code, and push only the translation.

Alright!

colsing this and opening one only for the translation.

colsing this and opening one only for the translation.


Grammar rules for Fountain

Hi. I'd like to create Abstract Syntax Trees out of Fountain documents.
Is there a grammar spec or a reference compatibility document (kind of like a TCK) so that I can know that I'm parsing documents correctly?
Thanks!

The Fountain syntax spec is at http://fountain.io/syntax, and this repo has a set a tests along with test files you can use. It's not great, but it might be sufficient for what you're trying to do.

How about a BNF grammar ?

I know this is an old thread, but I've written a PEG for Fountain:
https://github.com/nyousefi/Fountain/issues/53
If people are interested, I can clean it up and add some docs and maybe a CLI parser/linter.


Set a default limit for all strings, opaques and arrays

@2opremio Is this change is abandoned?

Yes!


Update sbt-scalafmt to 2.4.6

Updates org.scalameta:sbt-scalafmt from 2.4.3 to 2.4.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "sbt-scalafmt" } ]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #225 (9c9acbe) into master (e11f720) will decrease coverage by 5.55%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
- Coverage   41.66%   36.11%   -5.56%
==========================================
  Files           1        1
  Lines          36       36
  Branches        1        1
==========================================
- Hits           15       13       -2
- Misses         21       23       +2

Impacted Files
Coverage Δ

.../main/scala/io/circe/bson/BsonCodecInstances.scala
36.11% <0.00%> (-5.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 e11f720...9c9acbe. Read the comment docs.


auto_import: fix bug when the common path was shorter than both the c…

Fix bug when the common path was shorter than both the current use and target path.
Wrong 1 (unnecessary self)
use std::fmt::nested::Debug;
std::fmt::Display<|>

--->
use std::fmt::{ self, Display, nested::Debug};
Display<|>

Wrong 2 (unnecessary, Debug disappear!!)
use std::fmt::Debug;
std::fmt::nested::Display

-->
use std::fmt::Debug::{ self, nested::Display, };
Display<|>

bors r+


use redpanda.yaml for rpk commands instead of flags

configure the rpk section of redpanda.yaml to satisfy the configuration
needs of rpk to avoid using flags everywhere.
Add instructions for creating an rpk profile so external rpk users can
also run commands without having to set flags.

2023-09-26T00:02:23.3555120Z >>> kubectl --request-timeout=30s logs redpanda-3rrmbectg8-test-kafka-produce-consume --namespace redpanda-3rrmbectg8 --container redpanda
2023-09-26T00:02:23.4348794Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?
2023-09-26T00:02:23.4350581Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?
2023-09-26T00:02:23.4351873Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?
2023-09-26T00:02:23.4353120Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?
2023-09-26T00:02:23.4354381Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?
2023-09-26T00:02:23.4355928Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?
2023-09-26T00:02:23.4357154Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?
2023-09-26T00:02:23.4358380Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?
2023-09-26T00:02:23.4359611Z unable to create topics [produce.consume.test.redpanda-3rrmbectg8-test-kafka-produce-consume]: broker closed the connection immediately after a dial, which happens if the client is using TLS when the broker is not expecting it: is TLS misconfigured on the client or the broker?

It seems like TLS is not propagated to rpk.yaml configuration file.

https://github.com/redpanda-data/helm-charts/actions/runs/6315915913/job/17149494209?pr=741#step:20:8661
  rpk topic delete test-topic-506 source.test-topic-506 mm2-offset-syncs.test-only-redpanda.internal
  TOPIC                                         STATUS
  mm2-offset-syncs.test-only-redpanda.internal  UNKNOWN_TOPIC_OR_PARTITION: This server does not host this topic-partition.
  source.test-topic-506                         OK
  test-topic-506                                OK

That test seems to not working as rpk should return none zero code ...

https://github.com/redpanda-data/helm-charts/actions/runs/6315915913/job/17149494209?pr=741#step:20:8661
  rpk topic delete test-topic-506 source.test-topic-506 mm2-offset-syncs.test-only-redpanda.internal
  TOPIC                                         STATUS
  mm2-offset-syncs.test-only-redpanda.internal  UNKNOWN_TOPIC_OR_PARTITION: This server does not host this topic-partition.
  source.test-topic-506                         OK
  test-topic-506                                OK

That test seems to not working as rpk should return none zero code ...

I'll open an issue for rpk.
The test did succeed, it's just the cleanup that's failing.

https://github.com/redpanda-data/helm-charts/actions/runs/6315915913/job/17149494209?pr=741#step:20:8661
  rpk topic delete test-topic-506 source.test-topic-506 mm2-offset-syncs.test-only-redpanda.internal
  TOPIC                                         STATUS
  mm2-offset-syncs.test-only-redpanda.internal  UNKNOWN_TOPIC_OR_PARTITION: This server does not host this topic-partition.
  source.test-topic-506                         OK
  test-topic-506                                OK

That test seems to not working as rpk should return none zero code ...

I'll open an issue for rpk.
The test did succeed, it's just the cleanup that's failing.

Looking deeper, I think this is correct. It succeeded because the desired effect is that the non-existant topic be deleted. If it doesn't exist then if deletion is idempotent, it's a success. The topic is, indeed, deleted.
  NAME                          PARTITIONS  REPLICAS
  _internal_connectors_configs  1           1
  _internal_connectors_offsets  25          1
  _internal_connectors_status   5           1
  _schemas                      1           1
  source.test-topic-506         1           1
  test-topic-506                1           1
  + rpk topic delete test-topic-506 source.test-topic-506 mm2-offset-syncs.test-only-redpanda.internal
  TOPIC                                         STATUS
  mm2-offset-syncs.test-only-redpanda.internal  UNKNOWN_TOPIC_OR_PARTITION: This server does not host this topic-partition.
  source.test-topic-506                         OK
  test-topic-506                                OK

Looking deeper, I think this is correct. It succeeded because the desired effect is that the non-existant topic be deleted. If it doesn't exist then if deletion is idempotent, it's a success. The topic is, indeed, deleted.

When I was looking at the managed connectors that topic should be created by mm2 connector. I understand that rpk did the right thing.

Looking deeper, I think this is correct. It succeeded because the desired effect is that the non-existant topic be deleted. If it doesn't exist then if deletion is idempotent, it's a success. The topic is, indeed, deleted.

When I was looking at the managed connectors that topic should be created by mm2 connector. I understand that rpk did the right thing.

Maybe it cleaned up the topic itself when the config was deleted?

tested this locally, any further issues can be addressed, afterwards, this is a very clean experience and users will love it.


High Memory and CPU Overload on VPS During Bulk Updates with Nested Relations

PGSync version: 3.2.0
Postgres version: 15.1
Elasticsearch/OpenSearch version: 7.17.23
Redis version: 7.2.5
Python version: 3.9
Problem Description:
I recently added PGSync to my docker-compose.yml for Supabase with the following setup:
pgsync-redis:
    container_name: pgsync-redis
    image: redis
    command: redis-server --requirepass ${PGSYNC_REDIS_PASSWORD}
    ports:
      - 6379:6379

pgsync:
    container_name: pgsync
    build: ./requirements/pgsync
    depends_on:
      db:
        condition: service_healthy
      pgsync-redis:
        condition: service_healthy
    environment:
      - PG_USER=supabase_admin
      - PG_PASSWORD=${POSTGRES_PASSWORD}
      - PG_HOST=${POSTGRES_HOST}
      - PG_PORT=${POSTGRES_PORT}
      - PG_DATABASE=${POSTGRES_DB}
      - REDIS_HOST=pgsync-redis
      - REDIS_PORT=6379
      - REDIS_AUTH=${PGSYNC_REDIS_PASSWORD}
      - ELASTICSEARCH_SCHEME=${ELASTICSEARCH_SCHEME}
      - ELASTICSEARCH_HOST=${ELASTICSEARCH_HOST}
      - ELASTICSEARCH_PORT=${ELASTICSEARCH_PORT}
      - ELASTICSEARCH_USER=${ELASTICSEARCH_USER}
      - ELASTICSEARCH_PASSWORD=${ELASTICSEARCH_PASSWORD}
      - ELASTICSEARCH=True
      - LOG_LEVEL=INFO

Everything works perfectly during initial synchronization with Elasticsearch, and updates to individual entries in my Postgres database are also synced correctly.
However, I encountered a severe issue when running a Python script that updates data for around 10,000 movies daily at 10 AM. Since adding PGSync to my stack, this bulk update caused my VPS to overload in terms of RAM usage, which I noticed through the extremely slow performance of my Supabase API and even SSH access to my VPS. This overload persisted for over 10 hours after the script had finished running.
Here are the relevant details:

Postgres contains two tables:

tmdb_movie (columns like id, original_title, etc.)
tmdb_movie_translation (linked to tmdb_movie with a movie_id foreign key)

PGSync configuration:

[
    {
        "database": "postgres",
        "index": "movies",
        "mapping": {
            "translations": {
                "type": "nested"
            }
        },
        "nodes": {
            "table": "tmdb_movie",
            "schema": "public",
            "columns": [
                "id",
                "adult",
                "backdrop_path",
                "budget",
                "homepage",
                "imdb_id",
                "original_language",
                "original_title",
                "popularity",
                "release_date",
                "revenue",
                "runtime",
                "status",
                "vote_average",
                "vote_count",
                "collection_id"
            ],
            "children": [
                {
                    "table": "tmdb_movie_translation",
                    "columns": [
                        "title",
                        "overview",
                        "language_id"
                    ],
                    "label": "translations",
                    "relationship": {
                        "variant": "object",
                        "type": "one_to_many"
                    }
                }
            ]
        }
    }
]

The first synchronization goes smoothly, and individual updates are synced correctly. But when the bulk update occurs, it seems that the numerous triggers, combined with the nested relationship in tmdb_movie_translation, are causing a massive overload.
Observed behavior:

Last logs from PGSync were repeating the following message in a loop:

pgsync  | Sync postgres:movies Xlog: [0] => Db: [4] => Redis: [0] => Elasticsearch: [958,552]...

The overload continued even after stopping PGSync and restarting the database. Supabase containers were continuously overloaded in terms of CPU and RAM.

Possible Related Issues:
I suspect this might be related to how PGSync handles the large number of triggers during bulk updates, especially with nested relationships : https://github.com/toluaina/pgsync/issues/88
Any insights or solutions would be greatly appreciated! 😁

Well I've upgrade my VPS with more vCPU and RAM and seems to work. I close this issue for now 😁


Delete videos endpoint

From Full-Stack-Project-Assessment created by Dedekind561: CodeYourFuture/Full-Stack-Project-Assessment#467
Description
HTTP Method: DELETE
Route: /api/videos/{id}
This endpoint is used to delete a specific video from the database. DELETE endpoints don't require a request body, and they don't even need to return a response body! While you can return a confirmation message: however, if you don't want to, you can use the HTTP status code 204 to tell the client that the action was successful, but there is no response body.
To make the endpoint nicer, you can also change it so if you try to call this endpoint on an id that doesn't exist, the system may respond with a HTTP 404 status code designating that the video did not exist in the first place or was already deleted.
Acceptance Criteria

[ ] A DELETE /api/videos/{id} backend endpoint is implemented to remove specific videos.
[ ] The endpoint handles non-existent video IDs gracefully, returning an appropriate HTTP status code (e.g., 404).
[ ] The endpoint returns a confirmation message or uses an HTTP status code 204 to indicate successful deletion.

https://github.com/fhkahin/Full-Stack-Project-Assessment/pull/32


Create a title filter component on the Home page

Hi @ali1373107, are you working ok with the filter component or is there any blockers?

Hi @kahei-li I have created SerachForm.jsx component in #116 and closing this thicket now.


beta.6 release branch

Merge the release branch to master.
Make sure it's merged by 'Create a merge commit' option.

The failing test 'WiX file should have documentation about Visual Studio C++ redistributables' should be disabled for now due to #4665. @mirichmo will submit a PR to disable it later targeting the master branch.

PR #4673 for reference


susan quiz april 20

Legal actions in which the alleged injured party sues for monetary damages are
*a. Civil actions
b. Criminal actions
c.  Malpractice
d. Vicarious liability

Significant gaps in the quality of waived testing practices in that positions offices resulted in a recent development of
a. Behavioral Competencies
*b.  good laboratory practices
c.  quality indicators
d . test threshold values

The level of care that a person of ordinary intelligence and good sense would exercise under the given circumstances is the definition of
*a. Due care
b. Quality care
c.  critical care
d. Standard of care

Which of the following is a phlebotomy QA procedure?
*a. Checking needles for blunt tips and barbs
b. keeping a record of employee sick leave
c. recording any instrument maintenance
d. tracking all lab OSHA violations

Which preanalytical factor that can affect validity of test results is not always under the phlebotomist’s control?
a. Patient ID
*b. Patient preparation
c. specimen collection
d. Specimen handling

After a failed venipuncture attempt and before deciding to switch to the patient’s other arm, the phlebotomist moves the needle around several times in search of the vein without knowing exactly where the needle is in relation to the desired vein.  No blood was obtained.  At the new site, blood easily flows and the venipuncture is completed.  The patient later complains of pain in the antecubital area where the phlebotomist was unsuccessful.  What claim can be made against the phlebotomists?
a. Assault
b. Fraud
*c. Negligence
d. Res ipsa loquitur

Which laboratory document describes in detail the steps to follow for specimen collection?
a. OSHA safety manual
b. policy guidelines
*c. Procedure manual
d. safety manual

Drawinga patient's blood without his or her permission can result in ca charge of
*a assault and battery
b. breach of confidentiality
c. Malpractice
d. Negligence

Which of the following would violate the patient’s right to confidentiality?
*a. Discussing the nature of a patient’s test results  with the family
b. fibing the patient a physician name that you know and trust
c. sharing information on a difficult draw with a coworker
d. showing a patient his or her lab results when requested

A phlebotomist explains to an inpatient that he has come to collect a blood specimen.  The patient extends his arm and pushes up his sleeve  the is an example of
a. Expressed consent
*b. Implied consent
c. malpractice
d. negligence

How do QC and QA differ?
a. QA aims to prevent future problems
b. QA is a component of a QC program
c. QC attempts to prevent future problems
*d. All of the above.

Hi @nwhitmore66 , I see that you're trying to build a quiz. In order to use this tool, you can fork the repository and follow the installation instructions.

What?  I am just a teacher, I do not know these terms
Sent from my iPhone
On Apr 21, 2020, at 9:37 AM, Jonathan Rasmussen<EMAIL_ADDRESS>wrote:

CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Hi @nwhitmore66https://github.com/nwhitmore66 , I see that you're trying to build a quiz. In order to use this tool, you can fork the repository and follow the installation instructions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/college-of-southern-idaho/blackboard-quiz-generator/pull/1#issuecomment-617220775, or unsubscribehttps://github.com/notifications/unsubscribe-auth/API6KA6A7T2NOXIGTEUB2XDRNWVS3ANCNFSM4MNLFD5A.


Publish v0.1.0

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/115545
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 33c5ff96be03d83489a3a8bc003006016482d9b9
git push origin v0.1.0


[Remove] Unused components and icons from dashboard

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4552  👈
#4498
#4499
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jnsdls and the rest of your teammates on  Graphite


Fix stack trace printing in node 4.

Fixes the failing snapshot test on node 4. Through the snapshot test I learned that stack traces on node 4 and node 6 can be different; the node 6 ones have more information.
This made me realize the node 4 stack trace matching was actually broken and that the two regexes we had could easily be rolled into a single one. Finally, to make the test pass, I created a custom Error class to make sure that the stack trace is definitely the same on all node versions.

Current coverage is 90.27% (diff: 100%)

Merging #1549 into master will not change coverage

@@             master      #1549   diff @@
==========================================
  Files            28         28
  Lines          1121       1121
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1012       1012
  Misses          109        109
  Partials          0          0

Powered by Codecov. Last update 913b8a3...95c598c


[Codegen] overloaded fromRawValue across modules for ArrayEnum props with type alias uint32_t cause runtime crash

Description
codegen generates duplicate fromRawValue for  array enum type props across different modules in same namespace facebook::react with different type alias for uint32_t
In the reproducer  app i used react-native-webview as an example to crash with Modal component from react-native
Modal generates node_modules/react-native/ReactCommon/react/renderer/components/rncore/Props.h from line 295 for parsing supportedOrientations prop
at line 323
static inline void fromRawValue(const PropsParserContext& context, const RawValue &value, ModalHostViewSupportedOrientationsMask &result)

WebView generates /ios/Pods/Headers/Private/React-Codegen/react/renderer/components/RNCWebViewSpec/Props.h from line 110 for parsing dataDetectorTypes prop
at line 142
static inline void fromRawValue(const PropsParserContext& context, const RawValue &value, RNCWebViewDataDetectorTypesMask &result) {

both function only differ in result parameter but they same at runtime because of same type alias
using RNCWebViewDataDetectorTypesMask = uint32_t;
using ModalHostViewSupportedOrientationsMask = uint32_t;

In the stack trace of crash we can see it starts to parse ModalHostViewProps then call fromRawValue from RNCWebViewSpec/Props.h with method signature facebook::react::fromRawValue(facebook::react::PropsParserContext const&, facebook::react::RawValue const&, unsigned int&)
One possible solution i see is using strong warapper class/struct for alias types, Or is there any compiler flag we can use to fix function overloading?
Note:- I haven't used react-native-webview in the js side, it is just for the codegen. If i remove supportedOrientations prop completely app wont crash
Steps to reproduce
clone repo https://github.com/netmaxt3r/rn-codegen-runtime-crash

yarn install
bundle install
RCT_NEW_ARCH_ENABLED=1 bundle exec pod install
yarn start
On another console yarn ios

React Native Version
0.73.6
Affected Platforms
Runtime - iOS
Areas
Codegen
Output of npx react-native info
System:
  OS: macOS 14.4.1
  CPU: (12) arm64 Apple M2 Max
  Memory: 86.72 MB / 32.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.11.1
    path: ~/.nvm/versions/node/v20.11.1/bin/node
  Yarn:
    version: 1.22.22
    path: /opt/homebrew/bin/yarn
  npm:
    version: 10.2.4
    path: ~/.nvm/versions/node/v20.11.1/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.4
      - iOS 17.4
      - macOS 14.4
      - tvOS 17.4
      - visionOS 1.1
      - watchOS 10.4
  Android SDK:
    API Levels:
      - "23"
      - "28"
      - "29"
      - "30"
      - "31"
      - "33"
      - "33"
      - "34"
    Build Tools:
      - 28.0.3
      - 29.0.2
      - 30.0.3
      - 33.0.0
      - 33.0.1
      - 33.0.2
      - 34.0.0
    System Images:
      - android-33 | Google Play ARM 64 v8a
      - android-34 | Android TV ARM 64 v8a
      - android-34 | Google TV ARM 64 v8a
      - android-34 | Google APIs ARM 64 v8a
    Android NDK: 22.1.7171670
IDEs:
  Android Studio: Not Found
  Xcode:
    version: 15.3/15E204a
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 21.0.1
    path: /usr/bin/javac
  Ruby:
    version: 2.7.6
    path: /Users/maxter/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.6
    wanted: 0.73.6
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: true

Stacktrace or Logs
#2	0x00000001801655c0 in abort ()
#3	0x00000001046c6f74 in facebook::react::fromRawValue(facebook::react::PropsParserContext const&, facebook::react::RawValue const&, unsigned int&) at /Users/maxter/tmp/p/rn-codegen-runtime-crash/ReproducerApp/ios/Pods/Headers/Private/React-Codegen/react/renderer/components/RNCWebViewSpec/Props.h:181
#4	0x00000001046bece4 in unsigned int facebook::react::convertRawProp<unsigned int, unsigned int>(facebook::react::PropsParserContext const&, facebook::react::RawProps const&, char const*, unsigned int const&, unsigned int const&, char const*, char const*) at /Users/maxter/tmp/p/rn-codegen-runtime-crash/ReproducerApp/ios/Pods/Headers/Public/React-Fabric/react/renderer/core/propsConversions.h:132
#5	0x00000001049285fc in facebook::react::ModalHostViewProps::ModalHostViewProps(facebook::react::PropsParserContext const&, facebook::react::ModalHostViewProps const&, facebook::react::RawProps const&) at /Users/maxter/tmp/p/rn-codegen-runtime-crash/ReproducerApp/node_modules/react-native/ReactCommon/react/renderer/components/rncore/Props.cpp:152
#6	0x0000000104b560b0 in facebook::react::ModalHostViewProps* std::__1::construct_at[abi:ue170006]<facebook::react::ModalHostViewProps, facebook::react::PropsParserContext const&, facebook::react::ModalHostViewProps const, facebook::react::RawProps const&, facebook::react::ModalHostViewProps*>(facebook::react::ModalHostViewProps*, facebook::react::PropsParserContext const&, facebook::react::ModalHostViewProps const&&, facebook::react::RawProps const&) at /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator17.4.sdk/usr/include/c++/v1/__memory/construct_at.h:41

Reproducer
https://github.com/netmaxt3r/rn-codegen-runtime-crash
Screenshots and Videos
No response

Thanks for the reproducer @netmaxt3r

One possible solution i see is using strong warapper class/struct for alias types, Or is there any compiler flag we can use to fix function overloading?

Type wrappers could be a solution here, or we could qualify the fromRawValue function with the module name so they won't clash at runtime.
Let me get back to you on potential solution here.

we could qualify the fromRawValue function with the module name

we need to consider single module having multiple array enum props as well , which could clash each other in this case

we could qualify the fromRawValue function with the module name

we need to consider single module having multiple array enum props as well , which could clash each other in this case

Ah that's also a really valid scenario

same problem, need help

@cortinico  any news on the possible approach to fix this ? If i get direction i can start working on a PR

One possible solution i see is using strong warapper class/struct for alias types

@netmaxt3r  I believe the best solution would be the one you suggested, to wrap the types into a struct and use them so the overload resolution won't be ambiguous.
If you could send over a PR to draft this fix, we can look into merging it (or see if there are any blockers)

@cortinico  please see PR #44123 , I am pretty new to codegen, i need to figure out the call stack for static inline std::string toString(const ${enumMask}Wrapped &wrapped) function and check other platforms + update jest snapshots. Please review changes with internal team if i missed any scenario.

Any ideas how to apply the PR to an existing 0.74.1 project? Seems the PR code is typescript while the code in node_modules/@react-native/codegen is javascript
I feel I'm missing a step here?

@knro try @react-native/codegen version 0.75.0-nightly-20240429-b7de91666 might work with 0.74

Thank you @netmaxt3r.  It's still crashing (with the same backtrace that you have). I noticed we have two codegens apparently.
node_modules/@react-native/codegen which is updated to 0.75 night above
node_modules/react-native-codegen which is v0.71
I tried deleting the react-native-codegen but after yarn install it gets added again to node_modules, so not sure how to make it select the first one?

I noticed we have two codegens apparently.

please add using yarn resolution example

I noticed when I install the bundle

that is native code , we just need to bump the js package for this fix

I noticed the app is still crashing

I tested with latest rn app see reproducer app. could you try rebuilding app from clean Pods

@netmaxt3r Thank you!!! The resolutions field did the trick and now it no longer crashes.

Closing as https://github.com/facebook/react-native/pull/44123 got merged which fixes this issue

@cortinico
is it possible keep this open  till fix land on release version or create fix release for 0.74.x (current) rn version?

is it possible keep this open till fix land on release version or create fix release for 0.74.x (current) rn version?

We generally close issues once they're fixed to keep track of the remaining actionable work. As there is nothing actionable here, I'd rather keep it closed.
We'll be cutting the 0.75 release branch really soon, and this change will go out in 0.75.0-rc.0


Properly document elvis.config

Add samples and proper descriptions explaining what you can do and how to do it with that file.

Where do you want this? In the README.md?

Hi @JohanVikman !
README.md would be great but, if it's too limited, maybe you can write a wiki page and link to it from the README.md file.

As a matter of fact, the wiki (from this repo and elvis_core as well) is in desperate need for some clean-up/update.

Please take a look at my pr @elbrujohalcon . Is it ok?

I believe this might be related to https://github.com/inaka/elvis/issues/438, but this issue, itself, is already dealt with (though still open), right?

Yeah. I'm closing it.


Android-Java: 1.9 How onclicklistener works and handled in the Android studio.

How click is been handled in android.
What is OnClickListner and how to use it.
Also add a small project to showcase how OnClickListner works.

Sir, @VikasPandey121 can I work on this issue?

Can you assign this task to me @VikasPandey121

Please keep these steps in mind while creating a pull request:

use git fetch and git pull commands before pushing commit it to the remote repository.
Your Contributing file should be in this directory Android_Development/Java/ with the file name starting from 2.2 for instance 2.2 Android UI Widgets
Note:- If you have multiple files to commit then add a tag at the beginning of your file name like 2.2.1, 2.2.2 etc
Create a new branch and push your PR on Android_Development_With_Java branch.
Pull request should contain only one commit
Note: If you have multiple commits then squash your commit.

Sir, @VikasPandey121 can I work on this issue?

@Pradumnasaraf  Yes sure, I have assigned this issue to you.
@shekhars08 Please look into other issues which are still not been assigned to others.

Thank You for assigning it to me.

I am interested (docum..)


TA-30 Improve readme

Card TA-30 is about making a deployment to the SAML mock, whilst this is
an important part of the shared mock IDP instance, the main use of this
is for developers in other teams to set themselves up with the mock with
minimum effort from the get access team.  These readme changes will help
clarify what and how the mock IDP can be used.

Codecov Report

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

@@            Coverage Diff            @@
##             master      #30   +/-   ##
=========================================
  Coverage     94.59%   94.59%
  Complexity      187      187
=========================================
  Files            28       28
  Lines           555      555
  Branches          7        7
=========================================
  Hits            525      525
  Misses           27       27
  Partials          3        3

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 9718822...4c90398. Read the comment docs.


Implement Agent

Implement new concurrent datatype Agent mostly like RefM from ZIO and looking like Ref + Semaphore (Atom + Gatekeeper) for everything else
It should have at least following methods
trait Agent[F[_], A]{
  // return current value, this will never block
  def get: F[A]
  // update value with effectful transformation, wait for the new result
  def updateM(f: A => F[A]): F[A]
  // enqueue transformation, return immediately
  def fireUpdateM(f: A => F[A]): F[Unit]
  // modify value with effectful transformation, wait for the new result
  def modifyM[B](f: A => F[(A, B)]): F[B]
}

Working on this

PR
https://github.com/TinkoffCreditSystems/tofu/pull/107

Closing for now, since simple version has been delivered.
We'll wait for your more advanced implementation @susliko


bug: cortex run: Error: Please delete model before downloading again

Cortex version
v1.0.0-141
Describe the Bug
Issue faced when running an already downloaded huggingface model author/model_id
> cortex-nightly run bartowski/Meta-Llama-3.1-8B-Instruct-GGUF
...
Selected: Meta-Llama-3.1-8B-Instruct-Q2_K.gguf
Model already downloaded: bartowski:Meta-Llama-3.1-8B-Instruct-GGUF:Meta-Llama-3.1-8B-Instruct-Q2_K.gguf
Error: Please delete the model before downloading again

Error: Model not found

Expected behaviour:

I expect it to run the downloaded model
I expected the model alias to be the huggingface path bartowski/Meta-Llama-3.1-8B-Instruct-GGUF

Screenshots / Logs
Mac:

What is your OS?

[x] MacOS
[ ] Windows
[ ] Linux

What engine are you running?

[X] cortex.llamacpp (default)
[ ] cortex.tensorrt-llm (Nvidia GPUs)
[ ] cortex.onnx (NPUs, DirectML)

Retrying this with v172, redownloading a already downloaded HF model bartowski/Meta-Llama-3.1-8B-Instruct-GGUF
Getting an uncaught error, but cortex chat successfully runs
Selected: Meta-Llama-3.1-8B-Instruct-Q2_K.gguf
Model already downloaded: bartowski:Meta-Llama-3.1-8B-Instruct-GGUF:Meta-Llama-3.1-8B-Instruct-Q2_K.gguf
libc++abi: terminating due to uncaught exception of type cpp::bitwizeshift::bad_result_access<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>>: error attempting to access value from result containing error
[1]    28174 abort      cortex-nightly run bartowski/Meta-Llama-3.1-8B-Instruct-GGUF

Proposing:

Verifiable Bug
-> Planning

Closing the original issue

Create a new issue later


Fix ResponsiveDataTable bug

Notes for Reviewers
This PR fixes the bug if ResponsiveDataTable it has a required prop as columns and option props as tableCols( both the prop type is same) which is displayed in MUI datatable, if someone doesn't pass tableCols the table breaks.
So I fixed this by passing the both columns or tableCols, if the optional prop is not passed it wont break anymore. Altough the any big props changes may lead to make the chages all over where it is been used.
This PR fixes #635
Signed commits

[x] Yes, I signed my commits.

The MUIDatatable documentation  specifies that there is no tableCols prop. However, in the codebase, there are inconsistencies: some components pass both columns and tableCols with same arguments, while others use only columns.
To eliminate this ambiguity, it’s important to note that columns is designed to define the table headers and manage the visibility of specific columns. Standardizing the use of columns across the codebase would avoid confusion.

You are completly right @capricorn-32 but sistent compoent are the base component whihc is been used across all the projects of Layer5, making any major change in the prop type or removing any prop, will lead to a mojor break and leads to breaking change across all over the places it is been used.
Although all the components is been documneted.
Just to avoid all this pain, I tried to make a change in a way a that, it should be fixed without changing the behavoiur or the current component.
I can remove the tableCols props as there is no use of it, but it will lead us to major change in all over the code bases.

@mdkaifansari04 I am pointing this because in future it will more pain to work around this. Lets see, what decision should be made to prevent this.

One way to fix this would be change in component and update the docs,
After just the release we have to make changes in the corresponding projects as I've access to all the projects eventually.
I can make the changes but I'm busy in some school stuff might not be possible now.
If anyone shows up we can create issues for each repo.


Simple fullscreen support

VSCode Version: 1.0
OS Version: OS X 10.10

I could be missing this in an option somewhere, but if not, it would be awesome if there was an option for "use_simple_full_screen" (like in Sublime Text) which just full screens VS Code in the current window and not create a new space like it does currently.

Did you try View>Toggle Full Screen

@egamma yes, this works for full screening VS Code into a new space, but it's not the same a 'simple full screen' where VS Code would full screen without entering a new space on the desktop.

that is one of the things I really miss compared to Sublime as well. 👍

+1

+1

+1

All you have to do is hold down the option key when you click the full screen button and it changes to the old plus button. Is that what you are looking for?

@richarddavenport nope, that does maximize the editor and is what I currently do. But, it doesn't get rid of the menu/window chrome and actually put the app into "full-screen" mode :(

+1

+1

I thought I'd post an update to this thread, since it has had some eyes on it for a while now. Simple (pre-Lion) fullscreen support has just been added in electron/electron#10254 and will (probably) be released in Electron 1.8.x. While I'm sure it will be a little while before vscode fully tests and upgrades to that version of Electron (1.8.0 is still in beta, after all), some of you will be happy to hear that there is at least a light at the end of the tunnel ☺

+1

Any news on this one guys?

Yeah if you have iTerm for example they have the option:

Which is really great cause it allows to have a window on top.
The thing I hate the most about the native fullscreen with vscode is that when I run my code it goes away to another window. Therefor I can not test my app and see the console at the same time.

Going fullscreen means that if you move the cursor too far to the top of the screen, you Mac menu bar scrolls down and covers VSCode's tab bar. Really annoying when using the mouse to change tabs.

Any update on this? I'm very nearly considering switching back to Sublime for this. I don't want to. 🙍♂

@o8e This functionality is possible in Electron v1.8.*, but VSCode is currently on Electron v1.7.12. As soon as VSCode upgrades its Electron version, then a patch can be submitted that will provide pre-Lion fullscreen. Unfortunately, I'm not sure when that is expected to happen.

👍 This is my hangup here.
Native fullscreen is extra clunky for people with multiple monitors or people who need to frequently switch apps.
So, most developers.

iTerm has excellent non-native full screen support, but idk how accessible
that would be. Plus the global keyboard shortcut means super fast context
switching. I favor the iTerm model vs osx spaces.
On Sun, Jun 10, 2018, 10:45 AM Len Smith<EMAIL_ADDRESS>wrote:

👍 This is my hangup here.
Native fullscreen is extra clunky for people with multiple monitors or
people who need to frequently switch apps.
So, most developers.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/5344#issuecomment-396067644,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYvGgKb4d-GHbFkTN06ZRdQMjLLuUuDks5t7VtGgaJpZM4IIV_1
.

@reedspool go to System Preferences -> Accessibility -> Display -> [x] Reduce motion.

+1

I see no reason why we could not support this feature now that Electron 2.0.x landed.

@bpasero let me know if I can help in any way ✌

Blocked by https://github.com/electron/electron/issues/13135

https://github.com/electron/electron/pull/14646
It's already fixed. Hope you guys will add this option soon.

We are close, but there is https://github.com/Microsoft/vscode/pull/55267#issuecomment-427870573

I just want to update from the original post, this is also relevant on OS Version: macOS 10.14

This landed on master. To enable, configure window.nativeFullscreen: false

@bpasero I get "Unknown configuration setting" error when I try to add that to my config file.

@mesqueeb you will have to wait until tomorrow and then get our insiders build: https://code.visualstudio.com/insiders/

@a-eid it's not in the insider build yet. looks like the current commit is 4a29bd3731e40c07622bae and this is a few commits after that. hopefully tomorrow.

We had to stop the insiders build due to a bug, so you will probably only see this next week Monday.

it's in insiders now!
@bpasero there's a 1 or 2 second pause when i switch to simpleFullScreen now that wasn't on my custom build. is that from https://github.com/Microsoft/vscode/pull/55267/commits/9667519b858db6076f800590f3d433fb51ba6149#diff-7c4f2973d00eb042eb19c3c3260ae078R523 ? also i don't quite understand what that was fixing.

@ignu seems quite fast for me, I do not think my change has an impact. That was to fix an issue (that is still there actually) where Electron would return a 0px width for the document while entering or leaving fullscreen. But if you think your delay is caused by it, just comment it out and try it.

Fyi the setting got renamed from window.nativeFullscreen to window.nativeFullScreen

I am seeing some weird behavior when using multiple windows. Windows no longer rendering but still appearing in the Window menu, or windows completely closing without warning.
I'll see if I can get some solid repro steps, but if it's not something with my specific setup then I bet you can repro easily regardless. I had no extensions enabled, and my settings file isn't too crazy.
settings.json.txt (Github won't upload JSON files?)

@kibiz0r if you can distill steps, file a separate issue please. if this feature is too broken, I will take it out again...

@kibiz0r @bpasero cmd+` is fine for me on multiple monitors. i even put instances on three monitors and even tried a combination of some in fullscreen and some windowed and can't reproduce. I'm on High Sierra (10.13.2)

I'm just on one monitor. I suppose it could potentially be affected by multiple monitors. I'll do a repro today and try it with two monitors as well.

Install VSCode Insiders (bdfe4fd41f6a5fe538197b3cba937230249a563a)
Save this as ~/Library/Application Support/Code - Insiders/User/settings.json:

{
    "window.nativeFullScreen": false,
    "window.restoreFullscreen": true
}

Open VSCode Insiders
cmd+ctrl+f to fullscreen the window
cmd+n to create a new file, type some stuff in it
cmd+shift+n to open a new window
cmd+ctrl+f to fullscreen this new window
cmd+n to create a new file, type some other stuff in it
cmd+` to switch between windows; see that this works fine
Quit VSCode Insiders
Re-open VSCode Insiders
See that only one window from the last session is re-opened
cmd+shift+n to open a new window
cmd+ctrl+f to fullscreen this new window
cmd+n to create a new file, type some more stuff in it
cmd+` to switch between windows
See that the app alternates between switching windows immediately after pressing cmd+`, then the next time requiring the user to press cmd+` twice before it will switch
Open the Window menu; see that there is only one entry, "Welcome", despite having two windows open
cmd+shift+w to close the first window; see that the "Do you want to save?" dialog slides down from the top of the screen and the window closes
cmd+shift+w to close some invisible second window; see that the "Do you want to save?" dialog slides up from the bottom of the screen; see that this didn't close any visible window
cmd+shift+w to close a third window (the second visible window); see that the "Do you want to save?" dialog slides down from the top of the screen and the window closes

@ignu I (and others: https://github.com/Microsoft/vscode/issues/63291) are seeing a strange behaviour where when entering simple fullscreen, the mouse acts up: it no longer changes its cursor when moving over UI elements, such as the explorer divider to resize it. I also see no hover showing up in the editor when hovering over code.
Given these issues (also https://github.com/Microsoft/vscode/issues/5344#issuecomment-436264187) I think I will have to disable this feature until the underlying issues (possibly in Electron?) are fixed.

Because it's not enabled by default and at least for me - it works well - can we keep the feature? This has been one of my most anticipated features yet.

I found a way around it and the feature will remain, however we have to disable the custom titlebar on macOS if this feature is enabled due to issues with the mouse interaction.

For me it is bugged but I still love it.
The bugged fullscreen is still better then apple's horrible native fullscreen.

when opening vscode the green plus sign does not work, only the shortcut does.
when I use a second monitor and have my laptop closed and then unplug the monitor and open my laptop the window is in the wrong size (too large). Maybe other apps have this as well, I need to check that when I'm in the office.
the red orange green buttons dissapear after exiting fullscreen, but who uses those anyway...

But you all can just double click on title bar on VSCode and the result is the same. So why even bother to add bugged feature.

How is the result the same?

It's not the same. I mean, it might seem silly to obsess over the few pixels the menu bar takes up "maximized" in macOS, but I find the aesthetics awful and I'm in my editor too many hours a day.
Anyway @clankill3r I'm curious why that happens to you. I fixed that very issue and Simple Fullscreen VS Code should resize itself whenever the resolution changes. (I also seem to have the same flow as you when I take my MacBook home for the night.)

For me it's not the few pixels the menu bar takes up, for me the important
thing is that when I run my code a new window can come one top off VSCode.
With native fullscreen it goes to another desktop and opens the app there,
and then when I close that app it stays at that desktop. So I have to
switch manually to VSCode, where command + tab often does not work cause
that will go to whatever sceen I don't have in fullscreen. Neither does
command + ~ work cause that never goes to fullscreen applications.
Op wo 28 nov. 2018 om 00:44 schreef Len Smith<EMAIL_ADDRESS>
It's not the same. I mean, it might seem silly to obsess over the few
pixels the menu bar takes up "maximized" in macOS, but I find the
aesthetics awful and I'm in my editor too many hours a day.
Anyway @clankill3r https://github.com/clankill3r I'm curious why that
happens to you. I fixed that very issue and Simple Fullscreen VS Code
should resize itself whenever the resolution changes. (I also seem to have
the same flow as you when I take my MacBook home for the night.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/5344#issuecomment-442261580,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtFWlFY0XE7JGpS8HlbQ6ky1nEUW958ks5uzc53gaJpZM4IIV_1
.


Jaspr Community on Discord

Hello @schultek, @snehmehta and others,
For better communication during contributing I created Discord server here: https://discord.gg/G8pH8X4u
Where you can join and we can communicate on next progress. ;-)

Hi @mjablecnik, great idea! Can you send a new link, the current one is expired.

@schultek https://discord.gg/yuBS8jPP3K

This is now linked in the readme


Scrcpy closes after a few seconds

scrcpy-server.jar: 1 file pushed. 0.6 MB/s (19342 bytes in 0.032s)
INFO: Initial texture: 720x1280
Invalid return value 0 for stream protocol
Invalid return value 0 for stream protocol
[h264 @<PHONE_NUMBER>61cd00] corrupted macroblock 5 31 (total_coeff=-1)
[h264 @<PHONE_NUMBER>61cd00] error while decoding MB 5 31
[h264 @<PHONE_NUMBER>61cd00] concealing 2249 DC, 2249 AC, 2249 MV errors in P frame
WARN: Cannot terminate server

Having this issue.

Immediately? Or after some time?
Related: https://github.com/Genymobile/scrcpy/issues/281.

After a few seconds, I can control it for a few seconds 2-3 seconds. Then It closes and all errors are
adb: error: failed to get feature set: device offline
ERROR: "adb push" returned with value 1

OK, so you lost the adb connection. What is the result of adb devices when this happens?

I think i found out the issue. There was not enough power to the USB.

Try another resolution, I got the same error using the default one (1080x2336 on my phone), it works when then I tried scrcpy -m 1024  which lower the resolution to 472x1024.

going through some trouble
INFO: scrcpy 1.16 https://github.com/Genymobile/scrcpy
C:\Users\arino\Downloads\scrcpy-win64-v1.16\scrcpy-server: 1 file pushed, 0 skipped. 7.0 MB/s (33622 bytes in 0.005s)
[server] INFO: Device: HUAWEI INE-LX2r (Android 9)
[server] ERROR: Exception on thread Thread[main,5,main]
android.media.MediaCodec$CodecException: Error 0xfffffc0e
at android.media.MediaCodec.native_configure(Native Method)
at android.media.MediaCodec.configure(MediaCodec.java:1967)
at android.media.MediaCodec.configure(MediaCodec.java:1896)
at com.genymobile.scrcpy.ScreenEncoder.configure(ScreenEncoder.java:205)
at com.genymobile.scrcpy.ScreenEncoder.internalStreamScreen(ScreenEncoder.java:84)
at com.genymobile.scrcpy.ScreenEncoder.streamScreen(ScreenEncoder.java:55)
at com.genymobile.scrcpy.Server.scrcpy(Server.java:76)
at com.genymobile.scrcpy.Server.main(Server.java:225)
at com.android.internal.os.RuntimeInit.nativeFinishInit(Native Method)
at com.android.internal.os.RuntimeInit.main(RuntimeInit.java:371)
[server] ERROR: The hardware encoder is not able to encode at the given definition.
[server] ERROR: Try with a lower definition:
[server] ERROR:     scrcpy -m 1024
Exit due to uncaughtException in main thread:
INFO: Renderer: direct3d
INFO: Initial texture: 1080x2336
WARN: Device disconnected
anyone knows how to fix this/

anyone knows how to fix this?

Yes, your error message says:
[server] ERROR: The hardware encoder is not able to encode at the given definition.
[server] ERROR: Try with a lower definition:
[server] ERROR: scrcpy -m 1024

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#exception

anyone knows how to fix this?

Yes, your error message says:
[server] ERROR: The hardware encoder is not able to encode at the given definition.
[server] ERROR: Try with a lower definition:
[server] ERROR: scrcpy -m 1024

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#exception

so wait how do i fix it?

anyone knows how to fix this?

Yes, your error message says:
[server] ERROR: The hardware encoder is not able to encode at the given definition.
[server] ERROR: Try with a lower definition:
[server] ERROR: scrcpy -m 1024

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#exception

how do i change it to a lower definition

Execute scrcpy -m 1024: https://github.com/Genymobile/scrcpy/blob/wincli/FAQ.md#command-line-on-windows

Execute scrcpy -m 1024: https://github.com/Genymobile/scrcpy/blob/wincli/FAQ.md#command-line-on-windows

i still cant figure it out it says this commmand cannot find opperatble program or some stuff can you give me a step by step on how to do this

it says this commmand cannot find opperatble program or some stuff

Copy the exact command you typed and the exact error message.

Microsoft Windows [Version 10.0.18363.1256]
(c) 2019 Microsoft Corporation. All rights reserved.
C:\Users\arino>...\scrcpy-win64-xxx
'...\scrcpy-win64-xxx' is not recognized as an internal or external command,
operable program or batch file.

No but in this guide, at step 3 you have to adapt the path. Where is your scrcpy folder? For example, if its name is scrcpy in your download folder, type cd C:\Users\arino\Downloads\scrcpy.

tried em all didnt work
C:\Users\arino>\downloads\scrcpy
The system cannot find the path specified.
C:\Users\arino>\downloads\scrcpy-win64-v1.16
The system cannot find the path specified.
C:\Users\arino>\Downloads\scrcpy-win64-v1.16
The system cannot find the path specified.
C:\Users\arino>C:\Users\arino\Downloads\scrcpy
'C:\Users\arino\Downloads\scrcpy' is not recognized as an internal or external command,
operable program or batch file.
this is the file name scrcpy-win64-v1.16

But you absolutely don't type the command I pasted. cd then space, then C:\............
cd C:\Users\arino\Downloads\scrcpy-win64-v1.16

Sorry, I can't help more.

im new to these commands thats why i asked for steps like 1. do this , 2. type this

hi guys i am new to scrcpy 2.1.1
i want to ask when i connect my NOKIA 3.4 phone via wirelessly.
i got screecopy on my PC wirelessly in same network (like HOME is my wifi name) of my NOKIA phone.
but once my phone disconnected from HOME wifi and get reconnect within same network again
i cannot able reaccess or screencopy my phone
however, i have to make all process again from the 1st step like this.
1==> go to phone settings/developer mode/usb debugging mode=ON
2==> connect phone to PC / allow alwaz this mobile
3==> open scrcpy terminal cmd
4==> adb devices (device serial number shown under)
5==> next
6==> adb tcpip 5555
7==> restarting in TCP mode port: 5555
8==> adb connect <IP_ADDRESS>:5555
9--> connected to <IP_ADDRESS>:5555
I have to remake all these processes again n again after once device disconnected from Wi-Fi and reconnected again in same network.
after reconnected if i make again this command
8==> adb connect <IP_ADDRESS>:5555
it's given me error.
cannot connect to <IP_ADDRESS>:5555: No connection could be made because the target machine actively refused it. (10061)
can somebody have a proper solution for this?
that i don't want to connect my phone again n again
Whynot it can save always for once.
Thanks in advance.
slemon slim


Stop duplicating data in private map for public prefixes

Currently all key-values are stored in the private map, and if they have a public prefix then they are additionally stored in the public map. This duplicates data and would be much nicer if we could iterate over both maps when performing range queries in order to get the results out in-order efficiently whilst reducing storage use.

c.f. https://github.com/microsoft/LSKV/pull/188#issuecomment-1321801117


Coordinate(s) serializing as string type

The adapter(s) MUST cast the ordinate values to a float. In most cases they are being serialized as a JSON string.
{
    "coordinates": ["-85.1122", "2945444"]
}

MUST be
{
    "coordinates": [-85.1122, 2945444]
}

Committed in the main branch


Prompt error：KeyError: 'diff_decoder_type'

#
Hello, sir
I enter: Python information / TTS / gradient / infer Py, prompt error: diff_ decoder_ type.
I want to try to convert text into voice, but I don't know the location of text input and voice output.
Thank you for your teaching.

inference/gradio/infer.py is used for huggingface demo. If you want to infer in the terminal, you can run this: CUDA_VISIBLE_DEVICES=-1 python inference/tts/ps_flow.py --exp_name ps_normal_exp. The input text can be changed in the  example_run scripts.


Add Marko to wiki::Home.md ## Template Engines

wiki::Home.md ## Template Engines >>
 - [Marko](https://github.com/marko-js/marko/blob/master/docs/express.md) -- renders React components on server. It renders static markup and *does not* support mounting those views on the client.

I already see Marko listed in the Templates engines section you linked to.

oh, sorry, nvm

Looking at the link you gave above, it seems like http://markojs.com/docs/express/ is the canonical link, right?

yes, but in the list everything is linked to github repos
so i assume the gh-repo-link is preferred

Well, the preference is to list the module that plug into res.render through app.engine and friends. Mark seems to not do any of that at all, so I would prefer to simply remove it all together.

there is a way to use it as proper engine (http://stackoverflow.com/a/29732769), but it seems to be slower than used directly, so.. idunno

Hi @noformnocontent are you sure that the description you posted is the right description? Perhaps can you summarize again what change I should make from this conversation?


Add-AzureRMServiceEndpoint does not verify ServicePrincipalKey

When calling Add-AzureRMServiceEndpoint with an invalid parameter for -servicePrincipalKey, the function does not verify the connection.
I am able to create an endpoint in my on-prem TFS with a bad password, so the verification fails:

I think this should be a switch on the function because even on the dialog you are not forced to verify the connection.  Something like Add-AzureRMServiceEndpoint -Verify. Comments?

How would the return code disambiguate failing to add the endpoint with failing to verify? Maybe it should be two different functions, i.e. Verify-AzureRMServiceEndpoint ?

I will have to give that some thought. My knee jerk response is we define the error codes or it throws and exception if it could not verify.

What do you think about using the -WhatIf for verify?

I suppose that would be okay, so long as the behavior is documented.

@mrichman + @DarqueWarrior :
Sorry guys, I'm in clean up mood.
Can this issue be closed? I fail to comprehend where this issue relates to this project VSTeam.

Feel free to close. I no longer use TFS/VSTS.


Electron-19 (Frameless Window)

Added Frameless Window support

@lneir Please review. Thanks

@KiranNiranjan You need to merge master to this branch and push a new update. There are conflicts.

@VishwasShashidhar I'm waiting for the design team suggestion on this and also need to make some more changes on the client app.

Closing this as it is not currently required.


Fix typo in package:objective_c topics

coverage: 90.815%. remained the same
when pulling a43ab2f2f5d7885f8122d77d86555cfca7af7abe on quickfix
into c837526da8808501b53255caea0364eaa2c84afb on main.


After adjusting the space at the top, the reset button was obscured and could not be accessed.

Thanks for the report, I just pushed an update that allows you to open the settings panel with alt+g as a temporary fix to this issue so that you can then set the "SPACE BOTTOM"
As soon as I have time I will then try to make the bar calculate the bottom space automatically


Do deviantart have stash folder size?

From api when i list inside a folder with the following way
"https://www.deviantart.com/api/v1/oauth2/stash/metadata?list=1&folderid=" + srFolderid + "&access_token=" + srAuthCode;
I don't see offset. So does this mean there is no folder size limit and api displays all results ?
So far i tried with 132 items and it listed all items

Yeah there is no limit on that at the moment. We want to go back at some point and update the entire stash api so that its a bit more sane, we are just waiting for time to do it.


chore(master): release 1.0.14

:robot: I have created a release beep boop
1.0.14 (2023-02-22)
Bug Fixes

added link to BLE fw #36 (b91b9f8)
aligned controls (6f8d371)
colour+ui tweaks (1d60d6f)
minor ui cleanup on mobile, fix GH link on ios #15 (77b61c7)
trying to improve #40 (175309b)
was an easy fix for #41 (1e3a629)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/builder555/PineSAM/releases/tag/v1.0.14 :sunflower:


chore: bump to nightly-2023-08-23

nightly-2023-08-19 is now the tentative release candidate for the first official release of Lean 4, so it would be nice to try it out explicitly everywhere!

I'll merge this when #15 is merged :)

Is this still the current release candidate?

It is now! :-)


Phase 2 - Extract, prepare and publish npm package

Description:
Phase 2 is the core of the Heatmap project to publish the npm package with tests and documentation

closing as this is the sds related work that is not prioritized
replacing with https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/gh/chanzuckerberg/single-cell-data-portal/4666


Should work with bytes?

Hmm just noticed ahocorasick.unicode in the docs  and TRIE_LETTER_TYPE in the source. Is there no way to support both bytes and strings in python 3?

Constantly segfaults when I disable AHOCORASICK_UNICODE, even when not using pyahocorasick objects.

Bytes are supported. Could you please run python unitests.py? There're many tests that use bytes.

test (unittest.loader._FailedTest) ... ERROR

======================================================================
ERROR: test (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: test
Traceback (most recent call last):
  File "/usr/lib64/python3.5/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib64/python3.5/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "<...>/pyahocorasick/test.py", line 16, in <module>
    a.add_word(w, (i, w))
TypeError: string expected

----------------------------------------------------------------------
Ran 1 test in 0.001s

FAILED (errors=1)
Test failed: <unittest.runner.TextTestResult run=1 errors=1 failures=0>
error: Test failed: <unittest.runner.TextTestResult run=1 errors=1 failures=0>

Hello,
Is the fix to this bug is planned for the near future?
I have the same problem :(
Unfortunately cannot work in bytes mode

Hi,
It seems like adding bytes as words haven't been implemented yet. Any updates?

Unfortunately I cannot devote as much time as previously. Waiting for contributors.

could we at least flip the default and make a 1.5 bump ?  it's easy to convert strings->bytes, but hard to convert bytes->strings reliably.

could we at least flip the default and make a 1.5 bump ? it's easy to convert strings->bytes, but hard to convert bytes->strings reliably.

Not exactly the best solution, but you can still use a codec which cannot fail to do the bytes to string conversation like latin-1.

@vapier hey :wave:
re:

could we at least flip the default and make a 1.5 bump ? it's easy to convert strings->bytes, but hard to convert bytes->strings reliably.

Can you be more explicit? what would need to be fixed/changed and where?

https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/refs/heads/release-R100-14526.B/dev-python/pyahocorasick/files/pyahocorasick-1.4.0-bytes.patch

@vapier you rock! .. I guess there is an interesting derived tidbit: pyahocorasick may be used in chromium OS? :innocent:
And I see you are also the author to credit for the patch https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+log/refs/heads/release-R100-14526.B/dev-python/pyahocorasick/files/pyahocorasick-1.4.0-bytes.patch
I also now recall of so many of your awesome contribs like on strace and so many other places. I am not worthy.

@WojciechMula this deserves a 1.5 bump indeed. I reckon I am not impacted in most cases as usually stuff only integers in automatons. But I wonder if we should not publish two builds now that I started pushing pre-built wheels for all OSes: one for bytes and one for unicode?

Oh man, it's really great news about usage!
@pombredanne I think it's better to finish releasing and then add patch. More work,   but a bit more logical. Of course IMHO it's up to you.
Anyway, it would be great if we provided specialisations for each type and then have some lightweight factory for this, even in pure python. This would be pretty easy if we have some code generator underneath.

I pushed some tests in this branch https://github.com/WojciechMula/pyahocorasick/tree/release-2.0.0

all the tests have been adapted or duplicated with a condition on the build type to run on unicode and bytes builds
there are new environment variable to drive the build as unicode or bytes See https://github.com/WojciechMula/pyahocorasick/commit/f6a5d20df0c6903da3c98b2bcf9d33c0c2fba036

Define either AHOCORASICK_UNICODE=yes or AHOCORASICK_BYTES=yes to build one or the other such as:
with AHOCORASICK_UNICODE=yes pip install -e .  or AHOCORASICK_BYTES=yes make
There are some issues that show up:

Some tests fails on bytes, so this was a good idea to add these tests ;)
There is also some weird issue that shows up in bytes build and also on Windows unicode builds (see  #165 )
Some memory leaks on unicode and not on bytes

@vapier the new environment variable should help you at the minimum get rid of your setup.py patch
Now to support strings & bytes simultaneously, we could either build both and manage a factory in Python as @WojciechMula  suggested above... or find a way to deal with both together.
I have been toying with the idea of an alternative unified way (on the C side) where the number of symbols in some "alphabet" would be the key input as opposed to the implied bytes = length 1 (e.g. 256 symbols in alphabet) and unicode = variable length based on build.
Also re: https://github.com/WojciechMula/pyahocorasick/issues/65#issuecomment-1046271016

I reckon I am not impacted in most cases as usually stuff only integers in automatons.

Actually I usually use sequences of integers as keys with ahocorasick.KEY_SEQUENCE
I could very much benefit from less memory usage if the KEY_SEQUENCE integers could be 16bits rather 32bits... My set of unique values are in a well defined range roughly under 15 bits long.

You can see the runs failures there https://github.com/WojciechMula/pyahocorasick/actions/runs/1952208882

all bytes builds have at two tests failing, both linked to the Automaton.items() method
the windows unicode builds have also the same tow tests failing as if they were built always a bytes and not unicode.

At this stage all the bytes-based tests pass.
I wonder if it would be useful to build two different packages like pyahocorasick and pyahocorasick-bytes to make it easier for users to pick a byte build? In this case the wheels would be built for bytes and the sdist would be by default building as bytes. This would be fairly simple.

We have #165 as a related issue

personally i'd rip the bandaid off, but failing that, publishing two wheels with the diff names sounds OK assuming they can be installed in parallel
i guess there's no way to build 2 C components into a single package and pick the right one based on the inputs ?


@W-13974769 - Show Color Attribute on ProductTile 🀄

Description
In this PR we enhance the ProductTile component to show a selectable attribute in the form of a swatch group. By default this attribute is "color" but can be changed by updating the constant file. This will allow the customer to view different colors of any given master/variant product type before clicking into the product detail page.
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

Render swatches for selectable attribute
Ensure that links are clickable and user can go directly to variant on the PDP

How to Test-Drive This PR

Checkout code and run npm run start
Visit any PLP (mens ties is a good on), you should see tiles rendered with variation attributes if they are some (when the product is a master/variant)
Hover over the swatch to change the image, click it to go directly to the PLP for the clicked variation

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

None of swatch images is shown in Product tile on PDP/homepage, but they are showing on product tile on PLP

None of swatch images is shown in Product tile on PDP/homepage, but they are showing on product tile on PLP

Thanks.. I've addressed this.


Not working when phone is idle

It working perfectly when app close, some time delay,it not a matter but
job schedular not working when phone going to idle state

I checked the Battery Optimisation option on my phone options and see that my application is optimized by the phone. So, background services have been having problems with waking up. I set the "not optimize" option and see services are working fine.

FJD is one of the things deferred during doze mode. More info: https://developer.android.com/training/monitoring-device-state/doze-standby


README updates

This updates the README to include stuff from @jimmynotjim's recent pull request and includes mention of Capital Framework to close #27.

:+1: Looks good

Thanks! Does someone mind merging? @Scotchester @jimmynotjim @anselmbradford


AAD support

Article needs updating to reflect AAD support https://docs.microsoft.com/en-us/azure/storage/files/storage-files-active-directory-overview

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

ID: c12f1ea9-6446-be01-22fe-885fb945d22c
Version Independent ID: 634f8ace-1a8c-6811-627b-fdfbdd1942a3
Content: Use an Azure file share with Windows
Content Source: articles/storage/files/storage-how-to-use-files-windows.md
Service: storage
GitHub Login: @RenaShahMSFT
Microsoft Alias: renash

@patpicos Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@RenaShahMSFT @wmgries Can you please take a look? Thanks!

@yuemlu ping

#reassign: @roygara

@patpicos Thanks for reaching out, we apologize for the delayed response. This article has been updated with AAD DS support. Accordingly, I'm closing out this issue.
#please-close


Add shorter options for standard parameters

-u for username
-p for password
-c for config file
-m for message
-t for topic
-q for queue
-cn for client name

Resolved in release 0.0.33


Remove eaa-kbc

Remove eaa-kbc from tests and CI files.
Fixes #160

These https://github.com/confidential-containers/enclave-cc/blob/3cb8dfd33c12b52b9f9252473429aa9bac8f0c0b/tools/packaging/build/agent-enclave-bundle/enclave-agent-sample-kbc.yaml#L21-L29
were also added because of RATS-TLS and can be removed from the Occlum instance(s)


Windows download links of HP 7.10.3 lead to 403

Both Windows Haskell platform links on the download page (64bit and 32bit) currently lead to a 403 HTTP error.

whoops! fixed. Thanks for the report.


[RadzenPanelMenuItem] Selected state based on pattern

Is your feature request related to a problem? Please describe.
Building a Menu on a Application that offers "types" on the menu and details navigation not on the menu (e.g. having Customers as menu point, but navigating to a Customer with Id n in the url) skips the selected state of the menu item when in a sub state.
Describe the solution you'd like
Introduce a PathMatch Property that takes an enum:

Full default
Beginning
Contains (not sure about the use of this)
End (not sure about the use of this)
Pattern uses a second new property PathPattern with a regex pattern
Based on the PathMatch Property the Selected of the item is calculated using the Path and the current Uri.

Describe alternatives you've considered
Setting the selected state on base of the path using a pattern or similar works but is a lot of effort.
Additional context
Please let me know if this would work for you and I will prepare a Pull Request with implementation.

We should better emulate the existing behavior of NavLink.

Implemented the NavLink behavior in https://github.com/radzenhq/radzen-blazor/commit/73a3a881caec5f30f7b84916853ca7c91fc8d097


Needs example of  on usage and example page.

Initially this post was about there being NO demo on the page for the project.
I was mistaken and there is. It's NOT intuitive enough to find.
A play button and demo indicates (to most developers/users) that its a video that will likely pop up. which is bad when trying to quickly evaluate a library.
Consider removing the "Play Demo" to reveal the sample and make it Just have a series of examples of it in action without any coxing.
When developers are assessing which library to use they hop through dozens of projects and just want to see two things:

How is it used.
What does it look like.

Most jQuery based plugins are consistent on the first part, but the second part is incredibly important as each project has different ideas about what is a sapid UI.
Additionally: Consider Making buttons bigger on slideshow or add side arrows to go between the different examples. Its kind of a challenge to click on a desktop.

Totally agree!

it would be great to an example of changing the configuration object in javascript as well.


Issue 333 - Trim outer space for folder name

Issue that this pull request solves
Closes: #333
Proposed changes
Brief description of what is fixed or changed
Trim input of folder_name (For adding new folder and edit)
Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface
Other information
Any other information that is important to this pull request

Hello @Kevan-Y,
Thanks for the update. The same issue is appearing in files also. Could you check that out.
route: /folder/

Thanks!

Hello,
LGTM! Thanks for the enhancement. :)