Investigate including 'descheduler' on management clusters

We've had several ManagementClusterPodPending alerts due to pods not being able to schedule on nodes due to resource constraints. This usually occurs as Falco (daemonset) being unable to schedule on a control plane node because other pods are using up all available resource requests.
Often there are pods running on the node can be scheduled successfully elsewhere which would better distribute the pods.
It could be worth us running descheduler on the MCs to help avoid this issue going forward.

app is ready, Currently unable to test because of a problem with github pages

App is ready and seems to work just fine. Will wait for one occurrence of ManagementClusterPodPending to deploy the app and see if it helps.

The descheduler app I created works, but unfortunately it does not help us with the use case Marcus described.
A new strategy that would probably solve our problem is in the roadmap with no ETA.
I tried to implement a simple strategy myself but it's not a quick thing I can do in my spare time, so for now I think there is nothing we can do with descheduler.


just read notifications that have sounds.

Current coverage is 25.37%

Merging #9 into develop will increase coverage by +0.75% as of 49dcaa3

@@            develop      #9   diff @@
=======================================
  Files            13      14     +1
  Stmts           199     201     +2
  Branches         47      50     +3
  Methods           0       0
=======================================
+ Hit              49      51     +2
  Partial           9       9
  Missed          141     141

Review entire Coverage Diff as of 49dcaa3

Powered by Codecov. Updated on successful CI builds.


Django 3.2 support

Various improvements/fixes, mostly relating to version upgrades & improving tests but also some new features (see CHANGELOG for the full list)
Can be reviewed commit-by-commit.

MySQL test cases are failing because it looks like they use a more recent mysql version that validates constraint length (maybe this is a mariadb thing?) -- locally with mysql 5.7 it doesn't do that.
I think will have to just skip those tests for mysql


[BUG-python/deployment] RemoteFeedbackDataset pull does not convert record to local

Describe the bug
We were doing a demo and saw this bug where the records are still RemoteFeedbackRecords.
Stacktrace and Code to create the bug
N.A.
Expected behaviour
N.A.
Additional context
N.A.

Hi @davidberenstein1957 could you elaborate a bit on this? Which argilla version are you using? I've checked since 1.17.0 and we are calling to_local before returning those to the local instance, and before we were just dumping the Python dict so IMO the RemoteFeedbackDataset shouldn't have been returned ever on the pull call. Anyway, feel free to provide more context and I'll look into it, thanks!

No worries, feel free to ping once updated, thanks 🤗


Use u8 instead of Encoding.UTF8.GetBytes

@stephentoub i pulled these into https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/2214


Mission statement

Following up on Ondřej's suggestion from June's monthly call (see https://fortran-lang.discourse.group/t/fortran-monthly-call-june-2021/1334/14) we want put together a long-term mission statement for Fortran and where we want to be in three years from now.
The actual mission statement can be created collectively in the fortran-lang.org wiki here.

Surely, a mission statement should be clear, inspirational, and actionable, guiding the community towards a common vision for Fortran's advancement and growth. Collaboration  in this process can ensure that it reflects the collective aspirations and values of the Fortran community.  Invite key stakeholders, contributors, and community members involved in Fortran development to participate in this mission statement creation. Would love to be part of this mission.


TypeError: stickyManager.registerStickyItem is not a function

Hi!
I'm just paste code from https://ownego.github.io/polaris-vue/components/Filters?example=filters-with-a-resource-list# and catch error. Please help!

UPDATE!
Error only if it in modal.

Fixed in https://github.com/ownego/polaris-vue/pull/397

Close due to fixed at v2.1.9.


update the demo to use Mobilenet v2

This change is

Also, sharing output screens.
MobilenetV1:

MobilenetV2:

I haven't read on architectural changes in v2 but seeing output, it looks like it doesn't make use of softmax. Please correct me if I'm wrong.

added softmax to the logits output, also updated the readme file. Thanks!

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Nice getting MobileNetV2 supported! Couple comments:

I don't see the softmax and the README changes? Perhaps forgot to push?
Just curious what a single inference takes in V2 vs V1 in the browser? Can you do quick measurements?

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

they are compatible both at around 30ms on my desktop.


Bump Kubernetes dependencies to v1.23

This PR bumps Kubernetes dependencies to v1.23, as well as controller-runtime and controller-tools which require Kubernetes v1.23:

sigs.k8s.io/controller-runtime
sigs.k8s.io/controller-tools

Ideally, we should have a tag version for each version of Kubernetes dependency, but for now, we can have a solid commit for it. So that no matter the consumers use which version of Kubernetes, they can easily fetch code from this repo(by a commit ID).

cc @qiujian16 @mikeshng


Add textBuilder param

Adds ability to supply our own TextWidget.
Addresses: https://github.com/leisim/auto_size_text/issues/84

Would you be able to land this?? It's extremely non-invasive and a nice add.

Yes looks good. Would you mind adding a small section to the readme? And maybe document the new property. Thanks :)

Should be good to go

I tweaked it a bit to include the size in the Style, and add the Data param for more flexible usage.

Yes looks good. Would you mind adding a small section to the readme? And maybe document the new property. Thanks :)

Hi can we land this pretty-please!? Would love to remove this fork dep from my yaml file

It's probably a better idea to create a dedicated widget for this purpose. TextFields have margin and padding that would result in incorrect font sizes.

Not if you zero them out... which you would likely do in this use case.

Also this is just one example use case, basically in any use case where we want to lean on your widget to calculate the target fontSize, but then do "whatever else" after that point, this would enable that.

Here you can see we used it to do some quite advanced logic, it worked awesome. We switch between a regular Text and a TextField when the thing is tapped, and have styled the TextField to exactly match the padding you get with Text:
https://github.com/gskinnerTeam/flutter-folio/blob/f1a42df97609e0b28b03422a02b5b7a4cb04745b/lib/views/editor_page/placed_scrap_renderer.dart#L132

@esDotDev That's really cool thanks for sharing. I started working on a AutoSizeBuilder widget that can be used for anything and also AutoSizeTextField and AutoSizeSelectableText. What do you think about that?

Sounds cool! So that would really just focus on producing double size value, and then we can do anything we need with it?

Yes exactly. It's mostly used as a common builder for all the other AutoSize widgets and it should communicate with the AutoSizeGroup


[7.x] Fix problems with dots in validator

Backport for https://github.com/laravel/framework/issues/34263

@driesvints Do you know when we can expect this release to be tagged?

@BenWalters probably next tuesday.

@driesvints  no chance of it being patched sooner as it's a bug? Sorry to be a pain, it's just without it we cannot push out our upgrade.

@BenWalters we patched this yesterday btw 👍

Any chances this fix will be ported to 6.x "LTS" ?


Filtering with diacritics is case sensitive in DIbiFluentDataSource

Using collate utf8_bin is breaking filtering with words containing diacritics - utf8_bin is not case sesnitive, but utf8_general_ci is. And by the way, why is there this line after all? Why not just use utf8_general_ci even for words containing non-ASCII characters?
https://github.com/ublaboo/datagrid/blob/master/src/DataSource/DibiFluentDataSource.php#L183

Related #428


Feature Request: Make service selectable

I think the development for Intervals.icu apps is great!
For me it would be more useful if i could choose the services that are used.
For example, I don't use Wahoo (anymore). Therefore I would like the option to be able to select this in the env.env.
Maybe this could be further developed. I'm thinking, for example, of a website where I can specify a few parameters (fatigue, stress, mood).
On submit, the automation process (get and distribute weight from Withings) is triggered.
For the automation it would also be interesting, for example, to get the steps of the previous day from Garmin and enter them in Intervals.icu the day before.
Maybe I can find some time to develop it too.
Great Project!

Thank you!
What you suggest is along the same ideas I'm having. I'm (very) slowly working towards that and still a beginner in coding in Python. So your request will probably be open for a long time, but is very much along the lines I'm thinking as well.

@Tronje-the-Falconer  done some research on fitness and fatigue past summer and now remember that it wouldn't be a good thing to past them from one system to another since they all do their own calculations that are slightly different in result.
Sleep for example, would be something for which you could say read it from xxx and write it to yyy. And I think heart rate, weight, body fat as well.
But maybe it is even better to read all this from the various systems and write them to your own database, add Grafana to it for nice graphs.
What would you think?

I like to use intervals.icu as my main plattform. So currently i use as main services:

strava (activities, for Zwift-sync to intervals )
garmin (activities)
withings (weigh and bodyfat)
matlab (ImReady4 Script)
Zwift

The Withings synchronization is currently very complex (Withings -> Fitbit -> via Appsync to googlefit -> intervals).
The morning routine is measure HRV with GarminWatch, weigh with withings, open app for syncing weight, open app for analyse HRV, open Website intervals to update Steps from Garmin (Day before), and set Fatigue, Stress and Mood for current day.
current optimum would be: measure hrv with watch, weigh, open Website set fatigue, stress and Mood, click send.
On Buttonclick Values are send to intervals and weigh and bodyfat is synced from withings, matlabscript is fired, steps from day before are synced to intervals.
i try to realise it. Have to set up a raspi to have a development environment. you got already withings- and intervals-part.
fatique, stress and mood are some Wellnessfields in intervals. So not calculated but personal feeling. This Part (wellnessdata on intervals) could be extended.


PermissionError on "Removing git created by copier" on windows if "git add ." is in the _tasks

Hi there, first of all thank you very much for this tool!
Describe the bug
I'm on a windows 10 machine, and in my copier.yaml I initialize the git repository and stage the files:
_subdirectory: template
# questionnaire here...
_tasks:
  - git init
  - git add .

The template renders correctly, the tasks run fine, but at the end I got this error:
Removing git created by copier git_path=<project dir>\.ctt\defaults\.git
Traceback (most recent call last):
  File "%USERPROFILE%\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "%USERPROFILE%\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "%USERPROFILE%\.local\bin\ctt.exe\__main__.py", line 8, in <module>
  File "%USERPROFILE%\AppData\Roaming\uv\tools\copier-template-tester\lib\site-packages\copier_template_tester\main.py", line 67, in run_cli
    run(base_dir=args.base_dir, check_untracked=args.check_untracked)
  File "%USERPROFILE%\AppData\Roaming\uv\tools\copier-template-tester\lib\site-packages\copier_template_tester\main.py", line 44, in run
    write_output(src_path=input_path, dst_path=base_dir / output_path, data=defaults | data)
  File "%USERPROFILE%\AppData\Roaming\uv\tools\copier-template-tester\lib\site-packages\copier_template_tester\_write_output.py", line 144, in write_output
    shutil.rmtree(git_path)
  File "%USERPROFILE%\AppData\Local\Programs\Python\Python310\lib\shutil.py", line 750, in rmtree
    return _rmtree_unsafe(path, onerror)
  File "%USERPROFILE%\AppData\Local\Programs\Python\Python310\lib\shutil.py", line 615, in _rmtree_unsafe
    _rmtree_unsafe(fullname, onerror)
  File "%USERPROFILE%\AppData\Local\Programs\Python\Python310\lib\shutil.py", line 615, in _rmtree_unsafe
    _rmtree_unsafe(fullname, onerror)
  File "%USERPROFILE%\AppData\Local\Programs\Python\Python310\lib\shutil.py", line 620, in _rmtree_unsafe
    onerror(os.unlink, fullname, sys.exc_info())
  File "%USERPROFILE%\AppData\Local\Programs\Python\Python310\lib\shutil.py", line 618, in _rmtree_unsafe
    os.unlink(fullname)
PermissionError: [WinError 5] Access is denied: '<project dir>\\.ctt\\defaults\\.git\\objects\\0a\\e48258d27f73e6d23157bc56d1e943e87a07b4'

The problem is specific to windows, and there's a rmtree example in the shutils documentation that explains how to solve that:
import os, stat
import shutil

def remove_readonly(func, path, _):
    "Clear the readonly bit and reattempt the removal"
    os.chmod(path, stat.S_IWRITE)
    func(path)

shutil.rmtree(directory, onexc=remove_readonly)

To Reproduce

Use a windows machine
create a template with _tasks: ['git init', 'git add .']
run ctt

Expected behavior
the .git directory should be removed without errors.

Thanks for trying out CTT and leaving such a clear bug report!
I have added a Windows test to CI and cut a new release with the suggested fix (2.1.4). Please let me know if you run into any additional issues or if you have other feedback.


Throws error of not found email templates when using with Nuxt Layers

Hi there! I want to create a Nuxt Layer that holds in a few email templates and the apps consuming this layer should be able to render these templates.
This is currently not possible though. The issue is, that vue-email searches for the ~/emails directory in the root of the consuming apps, not of the original layer app where the ~/emails directory is.
Reproduction
https://github.com/madebyfabian/nuxt-mail-layer-reprod
If you clone this repo, install dependencies and start the playground with pnpm start, you see the effect.

@madebyfabian yes, this is issue, i'll solve it as soon possible

@Dave136 Thanks, let me know if I can help 🙏🏻

I think you should loop through the nuxt.options._layers property to check for each layer

Hi @Dave136, unfortunately, this does not work in beta-5.
I see that is the added resolve has been removed here: https://github.com/Dave136/vue-email/commit/0658bf42458af54ea505615bf1860e7b74911694#diff-be7ecc6bc17358942938e025d26bee5e5c3664db77e2be717b644d96d3222172L130
Was this on purpose? I am currently working on creating a PR that uses nuxt.options._layers to fix this.

@madebyfabian sure!


Builds new tkr bom for v0.12.0

What this PR does / why we need it
New TKr for 0.12.0. Built off of projects-stg.registry.vmware.com/tkg/tkr-bom:v1.22.8_vmware.1-tkg.1-tf-v0.11.4
Which issue(s) this PR fixes
Fixes: #3996
cc @joshrosso

Still waiting on user package repo to be available:
❯ crane ls projects.registry.vmware.com/tce/main | rg 0.12.0
❯

Also @joshrosso - did repo-12 get pushed?
❯ crane ls projects.registry.vmware.com/tce/repo-12
Error: reading tags for projects.registry.vmware.com/tce/repo-12: GET https://projects.registry.vmware.com/v2/tce/repo-12/tags/list?n=1000: NOT_FOUND: repository tce/repo-12 not found

Still waiting on user package repo to be available:
❯ crane ls projects.registry.vmware.com/tce/main | rg 0.12.0
❯

Also @joshrosso - did repo-12 get pushed?
❯ crane ls projects.registry.vmware.com/tce/repo-12
Error: reading tags for projects.registry.vmware.com/tce/repo-12: GET https://projects.registry.vmware.com/v2/tce/repo-12/tags/list?n=1000: NOT_FOUND: repository tce/repo-12 not found

I named it wrong :smile: (core-12)!
Renaming now for consistency!


VHX.tv support please

https://buythecoffeeman.vhx.tv/packages/the-coffee-man-film-1/videos/tcm-online-master-web
https://embed.vhx.tv/videos/106682

I don't really know what other url's to supply (some may not be accessible without my account, i'm not sure?) I have paid for a rental but youtube-dl does not recognize the host.
I was surprised by this because as far as I can tell it's essentially the same site as vimeo on demand. But I could be wrong.
I've successfully retrieved rental videos from on demand before, so I just assumed it would work here too, This was my mistake.
I'd love to see support for this as I have come across VHX in the past and they host some very interesting independent documentaries.
Thanks for any help.

Duplicate of #14835.


TerraformTaskV1: Terraform plan executed twice

Hello,
I recently stumbled upon this Terraform extension to Azure Pipelines and thought I would try implementing it in my deployments. I was surprised to find that the terraform plan command is executed twice instead of just once, which increases the time it takes to run the pipeline.
Would it be possible to modify the task so the plan command is executed once? For example if the -out parameter is present in the first plan command, the existing terraform plan can be used to produce the JSON output and therefore eliminating the need for a second plan step?

should work now.  Still facing issues? reopen the issue if you do.

@mericstam Hello. Please re-open this issue. I have managed to reproduce using both TerraformTaskV1@0 (version resolved to 0.203.0) and TerraformTaskV2@2 (version resolved to 2.203.0).
Repro pipeline:
name: $(Rev:rr)
trigger:
- main
variables:
  system.debug: true
  tf_dir: $(Build.SourcesDirectory)/terraform
  azure_connection: <redacted>

steps:
- task: TerraformTaskV2@2
  displayName: init
  inputs:
    provider: azurerm
    command: init
    workingDirectory: $(tf_dir)
    backendServiceArm: $(azure_connection)
    backendAzureRmResourceGroupName: <redacted>
    backendAzureRmStorageAccountName: <redacted>
    backendAzureRmContainerName: terraform-state-files
    backendAzureRmKey: omtest.tfstate

- task: TerraformTaskV2@2
  displayName: plan
  inputs:
    provider: azurerm
    command: plan
    workingDirectory: $(tf_dir)
    environmentServiceNameAzureRM: $(azure_connection)

Task log output (V2):
7.txt
You can see in the debug log that terraform plan command runs twice with different parameters:
1:
2022-04-29T04:19:31.0918453Z ##[debug]exec tool: /usr/local/bin/terraform
2022-04-29T04:19:31.0919238Z ##[debug]arguments:
2022-04-29T04:19:31.0920008Z ##[debug]   plan
2022-04-29T04:19:31.0921103Z ##[debug]   -detailed-exitcode
2022-04-29T04:19:31.0921798Z [command]/usr/local/bin/terraform plan -detailed-exitcode
2022-04-29T04:19:32.9006741Z
2022-04-29T04:19:32.9035932Z No changes. Your infrastructure matches the configuration.
2022-04-29T04:19:32.9036513Z
2022-04-29T04:19:32.9037379Z Terraform has compared your real infrastructure against your configuration
2022-04-29T04:19:32.9038182Z and found no differences, so no changes are needed.

2:
2022-04-29T04:19:33.0580368Z ##[debug]exec tool: /usr/local/bin/terraform
2022-04-29T04:19:33.0581372Z ##[debug]arguments:
2022-04-29T04:19:33.0582132Z ##[debug]   plan
2022-04-29T04:19:33.0583386Z ##[debug]   -out=/home/vsts/work/1/s/plan-binary-d799e9be-e8f4-4f0a-bb9f-f607e169b205.tfplan

Initially I thought it's just a mistake in the logs so I decided to test how many times plan gets called by using side-effects.
I added a http data resource to terraform file to call an Azure Function and log the number of requests during pipeline by Application Insights.
data "http" "example" {
  url = "https://<redacted>.azurewebsites.net/api/HttpTrigger1?code=<redacted>==&name=test2"
}

Hi, V3 release is out in version 0.1.17 on the Marketplace.


PostgreSQL support

Hello,
I would like to have the possibility to use PostgreSQL because my entire information system is only with PostgreSQL.
I'm looking to use OpenERP/Odoo/Tryton because they use PostgreSQL but in same time it's very time and money consuming to adapt them to be similar as Corteza.
Thank you

Hey @colundrum! PostgreSQL (multi-db) support is planned in near future. Can not say antything about the timeline at the moment but there are some changes that need to be done (the db layer) and more thorough integration/e2e testing.

I am still interested in this and cockroachdb as a replacement for postgresql.

@fire if you have time & energy to tackle this, let me know

Can you write a document or link me how the code is organized. Also mention how the db is accessed.

should be quite clear if you inspect the code.
things to do:

refactor repository packages to allow multi-db support
migrate current db-layer to jmoiron/sqlx
add ability to configure which db to use

Integration tests have been implemented and cover most of the code that touches the db.

Do you use triggers or sql function-likes?

no.


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit e3197b6:
:clock10: Pending: waiting for processors (28 builds ahead of you)
:warning: Docs Build is busy, currently there are 28 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit e3197b6:
:clock10: Full build: the average full build time is 7 min(s) 54 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit e3197b6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Anti-frost signal

During the winter when the temperature drops to +/- 2 degrees or lower, the AC unit sometimes needs a period (+/- 15 min) to perform some sort of anti-frost process. During the process the AC heating unit is not working (disabled), thus the temperature drops very fast in the room.
During that process the AC its led keeps flashing. Is there a way to capture this process/signal in HA? I would like to know when it's busy with this anti-frost process to automate other climate tasks in the house.

@vdende unfortunately, Panasonic does not expose this information in the API.
I'm not even sure they send this info to their servers... they don't even include info about hardware errors in the API. Or if the device is actively heating/cooling.

That's a bummer, but thanks for checking @sockless-coding

That's really a shame, I was wondering the same thing. There is no way to detect if the AC is defrosting or if the power consumption is just high atm...


Handle multiple FSMs

I have the need to have, in parallel, several FSM that are exactly the same, but that will be in different states as they respond to a button, but that button is different for all of them.
Considering tinyfsm FSMs are not instances, what is the best way to handle this?

add a context as a container to hold the instance replace the global static pointer

Hello,
I have a need to create a vector of FSMs so that I can maintain individual states. The size of the vector is dynamic and will be defined during runtime. Any idea how I can do this??
I would want something like this std::vector vec;
vec[0]::dispatc(event0); vec[1]::dispatch(event1);

Joining the question as well.
I like the tinyFSM concept, but because the state is static, it makes doesn't seem to allow a "clean" way to create instances.
@digint  would like to see an example for that if exists?


Project excluded from Hacktoberfest 2021

My project also excluded.why?

I think author is not adding aur pr's to project

How to find a good repository to contribute to?

Maybe I think ...

well, we r not adding names here. We are adding projects or algorithms or etc. How can this repo be blocked?

Maybe I think ...

well, we r not adding names here. We are adding projects or algorithms, etc. How can this repo be blocked?

The system for hacktober doesn't check what you are adding, it checks the amount of PRs coming in and since the PRs coming into this repositories were a lot, the Hacktober System considered it to be spam and excluded it, there is nothing we can do about it.

How to find a good repository to contribute to?

Go through the resources tab of the hacktoberfest website, they have a huge list of websites that help you find good repos to contri in

Just want to mention I'm well aware of the issue and as you mentioned @SamikshyaMstar , I've contacted them through multiple media over this issue, I'll let you guys know if I receive any reply. Keep an eye on Discussions. Thanks for all of your contribution and support! 😊

add a hacktoberfest-2021 tag to your repo

HI @abhpd, i contributed two code work in your repo, and you have already marked them as accepted, but i am seeing on mail status as.

PR #  Vaccine finder application #1390
TreeSort algorithm #1393
Please approve the PR. Thanks

I haven't recieved any reply will update you guys on discussions.

Same is happening with me also. I raise 2 PR's and they too got excluded. Does any1 have idea about the same?

I haven't recieved any reply will update you guys on discussions.

@abhpd the Hacktoberfest website is very clear that repositories that encourage simple PRs as a way to quickly win a shirt will be excluded. It's likely you have not heard a response because this repository so clearly falls under that category, and they have so many other similar repositories to deal with that replying to every single one of you individually would be a huge waste of time.
See the bottom section of https://hacktoberfest.digitalocean.com/resources for the full details where it says:
Bad repositories will be excluded.
We've seen many repositories that encourage participants to make simple pull requests – to quickly gain a pull request towards winning. While these projects may be a valuable learning tool for new contributors, they often aren't valuable and high quality contributions to open source projects, and go against one of our core values for Hacktoberfest.

Some examples of projects that don’t follow the values of Hacktoberfest include:

Projects asking contributors to add their name or profile information to a list

Projects asking contributors to submit assorted data structures and algorithms (DSA)

Projects asking contributors to curate arbitrary lists of quotes, interview questions, etc.


Generated fonts not valid on Windows?

Hi, I'm currently using FontTools' varLib to generate static instances of variable fonts in my project VFIT.
The fonts produced with varLib are not recognized by Windows. Attempting to open or install them yields the following error:

Inspecting the properties of the font file in explorer also shows (seemingly nonsensical based on Google Translate) Chinese text instead of the actual properties defined in the name table.

Bear in mind, this issue ONLY occurs on Windows. On macOS, I have no issues at all. Any idea why this is happening?

Upon running the varLib.mutator command, I have determined this must an error in my code. I don't know what I'm actually doing wrong, but I know that it is my problem and not FontTools'.


error while build app

i got this error while running the app , thats the problem logs and my project numbers :
yaml :
environment:
  sdk: ">=2.3.0 <3.0.0"

...

  images_picker: ^0.0.9

logs
AndroidManifest.xml:194:9-65 Error:
Attribute activity#com.yalantis.ucrop.UCropActivity@theme <EMAIL_ADDRESS>from AndroidManifest.xml:194:9-65
	is also present at [com.github.LuckSiege.PictureSelector:picture_library:v2.6.0] AndroidManifest.xml:66:13-58 value=(@style/Base.Theme.NoActionBar).
	Suggestion: add 'tools:replace="android:theme"' to <activity> element at AndroidManifest.xml to override.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:processDebugManifest'.
> Manifest merger failed : Attribute activity#com.yalantis.ucrop.UCropActivity@theme <EMAIL_ADDRESS>from AndroidManifest.xml:194:9-65
  	is also present at [com.github.LuckSiege.PictureSelector:picture_library:v2.6.0] AndroidManifest.xml:66:13-58 value=(@style/Base.Theme.NoActionBar).
  	Suggestion: add 'tools:replace="android:theme"' to <activity> element at AndroidManifest.xml to override.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 27s
Exception: Gradle task assembleDebug failed with exit code 1

did you use image_cropper or other picker plugin?
They are same use ucrop.
you must choose one of them and delete others

can u tell how to do this should i not use images_picker with image_cropper


lv_imgbtn_set_action LV_BTN_ACTION_PR not working

I am using pc keyboard and created a lv_imgbtn , if I set callback on click action and press Enter key then callback is working fine but when i set action to pressed and pressing enter key then its not working.
Can anyone help me in this?
Also I am using keyboars keys in pressed state , following is my keyboard key state code,
if(in.type == EV_KEY)
 {
            printf("key code %d\n",in.code);
            if(in.code == BTN_MOUSE || in.code == BTN_TOUCH)
            {
 	        if(in.value == 0)
       		{
                    evdev_button = LV_INDEV_STATE_REL;
                }
                else if(in.value == 1)
                {
                    evdev_button = LV_INDEV_STATE_PR;
            	}
            }
	    else
            {
 		//keyboard key
              	if(in.value == 0)
                  {
                     data->state = LV_INDEV_STATE_REL;
                  }
                else if(in.value == 1)
                 {
                     data->state = LV_INDEV_STATE_PR;
                     data->key = keycode_to_ascii(in.code);
	   	}
		return false;
	   }

 }

It triggers the CLICK action. You can also use the LONG_PRESS action.
PRESS action was not added, but I've just added it to dev-5.3 branch. Does it work well?

In your comments please use the
```c
code
code
...
```
to add sytax highlight to your code.

LONG_PR also not working if I am presseing enter key

LONG_PR should work. Have you already set up the PC simulator? A test in the simulator would be great to be sure the library works, and the issue is in the driver.

may be the issue is different what I did , I created three imgbtn and added them to the group but initially there is home screen where I have only labels , so when i pressed enter key it hide all the labels and unhide all the imgbtns so instead of showing just three buttons , the first button which by default focused in group get also clicked by that enter key and callback function called.
correct me if my explanation is wrong.

No I am using on resberry pi like board , and all the function key which i assigned works fine , even enter key working as I want it to be except in this imgbtn case.
Can you tell me another way to use CLICK action as its work when enter key press and then released , here in my above mentioned code I am using only pressed mode of keyboard keys.

I see, but please make a test only with a singe imgbtn to see if the object itself is working or the issue is somewhere else.

its working when I first pressed the enter key to display all three btn and hide labels, it directly call the callback function of first imgbtn then I exit from that function and after that it works fine I press enter callback function called . The Only issue is when I press first time then it should not click the button instead wait for me to press the enter key again.

I see. Maybe set the action function of the imgbtn when you unhide it. When you hied it set the actions to NULL

still the same issue.

I solve my problem with a different logic.
I now don't return LV_GROUP_KEY_ENTER when I am going in main menu.
Thanks for all your time and support.


PDF text extraction can fail in complex shaping scenarios

Description
Compiling a pdf with non-latin (in this case specifically devanagari) text in it can sometimes result is strange text encoding. This results in text that is not properly selectable. The part of the font that is selectable differs across pdf viewers, and sometimes introduces non-existent characters into the selection.
Minimal Working Example:
#set text(font: "Siddhanta")

आ रु॒क्मैरा यु॒धा नर॑ ऋ॒ष्वा ऋ॒ष्टीर॑सृक्षत ।

अन्वे॑नाँ॒ अह॑ वि॒द्युतो॑ म॒रुतो॒ जज्झ॑तीरिव भा॒नुर॑र्त॒ त्मना॑ दि॒वः ॥

(explicitly states font for repeatability)
typst output
This results in the following chunk of selectable text in Ubuntu's Document Viewer:
अा
ैरा युध
ा नर॑ ऋ॒ ष्वा ऋ॒ ीर॑ सृक्षत ꠰
अन्वे॑नाँ॒ अह॑ व॒ ुताे॑ म॒ ताे॒ जज्झ॑ती रव भा॒नुर॑त॒ त्मना॑ द॒वः ꠱
In other pdf viewers the output may be different, e.g. firefox gives:
अा ˳॒ƨैरा यु॒धा नर॑ ऋ॒ष्वा ऋ॒ʆीर॑सृक्षत ꠰
अन्वे॑ नाँ॒ अह॑ Vव॒ȭुताे॑ म॒˳ताे॒ जज्झ॑तीRरव भा॒नुर॑तA॒ त् मना॑ Tद॒वः ꠱
(notice the introduced latin characters)
It seems that this is not an inherent limitation of pdf itself, LuaLaTeX can generate properly functioning pdfs. A minimal working example is provided for LaTeX as well:
\documentclass[a4paper]{article}

\usepackage{fontspec}
\setromanfont{Siddhanta}

\begin{document}

आ रु॒क्मैरा यु॒धा नर॑ ऋ॒ष्वा ऋ॒ष्टीर॑सृक्षत ।

अन्वे॑नाँ॒ अह॑ वि॒द्युतो॑ म॒रुतो॒ जज्झ॑तीरव भनर॑र्त॒ त्मना॑ दि॒वः ॥

\end{document}

LaTeX output
Because of this it seems that it is possible to do this in such a way that makes the text encoding work in pdfs.
Reproduction URL
No response
Operating system
Linux
Typst version

[ ] I am using the latest version of Typst

Wow! This must have been one of the weirdest things I've ever debugged.
Okay so basically, when you turn some text into glyphs, OpenType allows you to specify a "script". Depending on which script is activated and on the font, the same set of text can result in a different set of glyphs.
So basically, in the Latex snippet you posted, what happens is that Latex does not automatically set the script that the text should be shaped with, so none is selected. However, in the case of Typst, the deva script will be detected and selected automatically. If this script is activated, then there are a couple of places where two glyphs correspond to a single codepoint, and in PDF there does not seem to be a proper way of encoding this. Try the following Latex snippet, and you will see that you will also get issues when copy-pasting from a PDF viewer:
\documentclass{article}
\usepackage{fontspec}
\pagenumbering{gobble}

\setmainfont{siddhanta.ttf}[Script=Devanagari]

\begin{document}

आ रु॒क्मैरा यु॒धा नर॑ ऋ॒ष्वा ऋ॒ष्टीर॑सृक्षत ।

अन्वे॑नाँ॒ अह॑ वि॒द्युतो॑ म॒रुतो॒ जज्झ॑तीरव भनर॑र्त॒ त्मना॑ दि॒वः ॥

\end{document}

If you remove the script part, then you get the expected result. So let's do the same for Typst:
// Some gibberish script that doesn't exist in the font.
#set text(font: "Siddhanta", script: "none")

आ रु॒क्मैरा यु॒धा नर॑ ऋ॒ष्वा ऋ॒ष्टीर॑सृक्षत ।

अन्वे॑नाँ॒ अह॑ वि॒द्युतो॑ म॒रुतो॒ जज्झ॑तीरिव भा॒नुर॑र्त॒ त्मना॑ दि॒वः ॥

and now copy-pasting works as expected... So I don't think there is much we can do to change the behavior in PDF, as you can see this can also be reproduced in Latex, it's just this quirk in how OpenType features are selected that causes that difference to appear... So I'll go ahead and close this issue (feel free to comment again if you have another issue, though).

Try Renderer=HarfBuzz and LuaTeX it should give bettter results here (it basically applies the solution I suggested in my comment on #526).

Thanks, I'll look into it!

Try Renderer=HarfBuzz and LuaTeX, it should give bettter results here (it basically applies the solution I suggested in my comment on #526).

I honestly thought this was the default when using LuaHBTeX

Indeed, seems like at least in Chrome and Acrobat it works when changing to renderer (not in Preview and Firefox, but that's presumably on them), so I guess we will still have to figure something out.. Thanks for the hint!

I honestly thought this was the default when using LuaHBTeX

Unfortunately it is not. Other than using HarfBuzz for shaping, this renderer/mode also tries hard to make text extraction from PDF more reliable, for cases like the one here.
LibreOffice is another application to check for PDF text extraction issues, it uses a similar mechanism (I wrote the code in both cases).
Pango with cairo backend should be checked as well as I believe it does something similar (I was not involved with that one and I haven't tested it in a long time, so my information might be outdated).

I can replicate this with Arabic with the following snippet
#set text(lang: "ar")
== صحفيّة - دَوليَّة
<صحفية---دولية>
  \"صحفيّة\" \(نسبة إلى \"صحيفة\" مفرد) و\"دَولي\". بل الأصح ان تستخدم
  \"صَحفي\" و\"دَولي\"

It renders correctly,

The default font on macOS produces almost correct text extraction, it chokes on he diacritics and the parenthesis looks wrnog.
"صحف ّية" )نسبة إلى "صحيفة" مفرد( و" َدولي". بل الأصح ان تستخدم " َصحفي" و" َدولي"

However, if you sent the font to another like Amiri or Scheherazade New, you get even worse extraction
For Amiri:
"􏰟حفيّة" )نسبة إلى "􏰟حيفة" مفرد( و"دَولي". بل الأصح ان تستخدم " َ􏰟حفي" و"دَولي"

And for Scheherazade New:
"􏰗حف ّية" )نسبة إلى "􏰗حيفة" مفرد( و" َدولي". بل ا􏰘􏰗ح ان تستخدم " َ􏰗حفي" و" َدولي"

Just like OP, I get correct text extraction from LibreOffice with the fonts above

Just like OP, I get correct text extraction from LibreOffice with the fonts above

Out of curiosity, do you only get correct extraction when opening it in Chrome, or also in other viewers like Firefox or Apple Preview? From my experiments, the ActualText workaround only works in Chrome/Acrobat...

For the LibreOffice export, I got mostly correct extraction from Apple Preview. The first line is Amiri, second is Scheherazade New.
نسبة إلى "صحيفة" مفرد
نسبة إلى "صحيفة" مفرد

Other than the tofu block, it's correct
As for Chromium,
نسبة إلى "صحيفة" مفرد
نسبة ٕالى "صحيفة" مفرد

It's similar to Apple preview, except the إ was decomposed into ا and ٕ  (combining mark).
Firefox  does the worst
نسبة إلى"صحيفة"مفرد
لىٕا نسبة"صحيفة"مفرد

Note there is no spacing around the quotes and "إلى" has been completely broken in Scheherazade New.
For context, i'm using the versions of the font from nixpkgs  rev c3aa7b8938b17aebd2deecf7be0636000d62a2b9.
Specifically, Amiri 1.000 and Scheherazade New 4.000

Thanks!


Async form of MQTTClient.publish does not respect retain parameter

The async function MQTTClient.publish accepts a parameter for message retention, but that parameter is not passed along to the EventLoop variant. The code at issue is reproduced below.
    public func publish(to topicName: String, payload: ByteBuffer, qos: MQTTQoS, retain: Bool = false) async throws {
        return try await self.publish(to: topicName, payload: payload, qos: qos).get()
    }

Recommend change:
    public func publish(to topicName: String, payload: ByteBuffer, qos: MQTTQoS, retain: Bool = false) async throws {
        return try await self.publish(to: topicName, payload: payload, qos: qos, retain: retain).get()
    }

Hi Ross
Do you want to make a PR with those changes

Fixed in v2.8.0


[labs/analyzer] Support for BYO TypeScript version

Should this be an RFC?

[X] This is not a substantial change

Which package is this a feature request for?
Other/unknown (please mention in description)
Description
The analyzer currently uses its own version of TypeScript (currently 4.7.4). This is a problem when users are running the analyzer against a TS project that is being built against a different version of TS, since we currently stop analysis if there are semantic diagnostic errors (e.g. type errors), which often happens when running an older tsc against a codebase updated to a newer version of TS.
We should probably consider:

Requiring that the ts module be passed into the Analyzer constructor rather than importing it directly. We could then consider having the CLI / gen-manifest support finding or pointing to a user-provided TS installation
Update the analyzer to accept some number of different ts versions. e.g. build a union type of multiple versioned ts libs, and add the required duck-typing to choose correct API for where they vary. For example node.decorators for pre-4.8 vs. ts.getDecorators(node) for 4.8+
Adding multiple builds / tests to the monorepo for building the analyzer and testing it against multiple versions

Alternatives and Workarounds
Not possible today.

Isn't this approach applicable to @lit/localize-tools and @lit-labs/compiler as well?


Add two new options in the package

Inverted exclamation mark (¡) and inverted question mark (¿) match very well in lowercases. But in all caps must be raised to the baseline (Harmony, Robert Bringhurst dixit).

This example is only applied in spanish (castilian, asturian, etc.), portuguese, italian, and so on. We need add two new options:

raisecapitalquestion=<dimen>, and
raisecapitalexclamation=<dimen>.

and macros:

\capitalquestion, and
\capitalexclamation.

Minion Pro was well crafted by Robert Slimbach even in small caps:

The example is useful in headings and body text with all caps. It's just another suggestion. Best wishes ;-)

I confirm the need for y-adjusted inverted exclamation marks and
inverted question marks looking at your examples and explanation.
However, your option and macro names are misleading (at least to
a person who does not speak Spanish) as they imply manipulations
of the question mark or exclamation mark.  If I have understood
correctly, you are interested in shifting the upside-down versions
not the regular ones.
Moreover, I doubt whether two separate adjustment dimensions
are required.  Both, inverted question mark and inverted
exclamation mark are designed so similarly that a single vertical
shift possibly fixes both.  Show me three examples of fonts
where different values are required and I'll agree with your
suggestion.  FF TheSans seems to be a good start.  :wink:

Yes, only upside-down versions (¿, ¡) not the regular ones (?, !). English speakers only use one (?) at the end of sentences, but spanish speakers we use two; one in the beginning (¿, ¡) and one at the ending of sentences (?, !). Example:

¿Cómo estás, Chris? (How are you, Chris?)
¡Bien! ¿Y tú? (Fine! and you?)
¡Bien también! (Fine too!)
Fue un gusto verte, Chris. (It was nice to see you, Chris)
A mí también, Tuxkernel (Me too, Tuxkernel).

The dialogue was typeset mostly in lowercases (match very well with ascenders and descenders), but in all caps they need to be elevated to the baseline. I never I've used an american keyboard layout, but in my spanish keyboard layout, I'm using SHIFT + ¡ to produce inverted exclamation mark; and SHIFT + ? to produce inverted question mark. I repeat, those marks only are useful in all caps (spanish, portuguese, italian, and so on).
Now, related to option names, they were named just to illustrate the problem (suggested). The names of new options could be:

raisecapitalinvertedquestionmark=<dimen>. Probably not a value or dimen, just aligned to the baseline.
raisecapitalinvertedexclamationmark=<dimen>. Probably not a value or dimen, just aligned to the baseline.

Maybe we don't even need new options in the package, just macros for it:
-\capitalinvertedquestionmark, and
-\capitalinvertedexclamationmark.
Babel package (spanish) includes two macros for this: "! (¡) and "? (¿).  Look:

These macros are useful for TeX spanish users, but not for a english speaker typesetting paragraphs in different languages. TeX hackers can easily do the trick (rotate those marks) using graphicx (\rotatebox command) or rotating packages. I repeat, perhaps we don't need new options in the package, just simple macros. I was clear?


Prettied all files to same indentation. Wanna work on some page. Can u assign me a task

Please fix the merge conflicts and give another PR


fix recipes for Makie 0.20

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (7e6ca7c) 81.43% compared to head (6d1f326) 81.17%.

Files
Patch %
Lines

ext/VLBISkyModelsMakieExt.jl
60.00%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
- Coverage   81.43%   81.17%   -0.26%
==========================================
  Files          24       24
  Lines        1524     1530       +6
==========================================
+ Hits         1241     1242       +1
- Misses        283      288       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


please support point type for postgresql

Since point type is not supported by Doctrine core. So I edit the
TableFieldsGenerator.php
and add a line
$platform->registerDoctrineTypeMapping('point', 'string');
after json type.
It works. Please support the point type on next release.
thx

@unizhu support added. Ref: https://github.com/InfyOmLabs/laravel-generator/issues/593#issuecomment-471154604


[🚀for esl-utils]: groupBy utility

Request for groupBy separate utility for esl-utils/misc module.
groupBy function should transform an iterable to the pairs object with a criteria value as a key and an array of original items that belongs to the current criteria value.
Expected API:
 groupBy<T, V>(array: T[], group: (item: T) => V): Record<V, T[]>
Examples:
expect(groupBy([1, 2, 10, 4, 3], (i) => i % 2 > 0 ? 'odd' : 'even')).toBe( { odd: [1, 3], even: [2, 10, 4] })
expect(groupBy([100, 20, 10, 101, 300], (i) => String(i)[0] ).toBe( { '1': [101, 10, 101], '2': [20], '3': [300] })
expect(groupBy(['cat', 'fat', 'kitten'], (s) => s.length)).toBe( { 3: ['cat', 'fat'], 6: ['kitten']})
expect(groupBy([{ a: 2, b: 1 }, {}, {a: 1}, {}, {b: 2}], (s) => s['a'])).toBe( { 2: [{a: 2, b: 1}], undefined: [{}, {}, {b: 2}], 1: [{a: 1}] })
Priority: Normal
NTH
Support for iterables instead of an array as an input.

:tada: This issue has been resolved in version 3.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CRC check failed! with JK_B2A8S20P and SW V11.262

Hi,
i'm allready using an JK_B2A8S20P with an ESP8266 over UART. It runs since several weeks without any issues on SW V11.17.
Now i'm building a new unit with an other BMS with SW V11.262 and getting several failure messages.

the new ESP8266 runs on my first BMS without issues.
I've allready checked the wiring, but without success.
jk-bms-kopie.yaml.txt
any ideas, then changing to an ESP32?
Best regards
Christoph

additional info:
my "old" unit based on version 1.2.0 is running with the new BMS with SW V11.262 without any problems
This old unit was build with ESPHome 2022.4.0
regards
Christoph

Are you able to downgrade your ESPHome installation? Could you try component version 1.5.0 plus ESPHome 2022.4.0?

i tested on an different PC, which i've used to setup the first unit.
Python version is 3.11.0
esphome version is 2022.10.2
unfortunately i get the same result with "CRC check failed!" and "Invalid header"

I've ordered some new ESP and will test my luck with new hardware.
i will keep you informed

I am now completely confused
This afternoon I flashed the 2nd ESP8266 with the old program and connected it to my first battery with BMS (SW V11.17). There it works without any errors or abnormalities.
Then I flashed the previous ESP with the new program and connected it to the new BMS (SW V11.262).
And there it runs without any problems. No CRC or header errors.
Only sometimes there is an error message in the log that took a little too long.

Very confusing.
now that I don't know how to narrow down the cause further, I suggest closing here.
regards
Christoph

A CRC error every now and then is fine. Let's close this issue.

Hi,
May I open it again?
I'm having the same problem with the JK_B2A8S20P with Hardware version V11.XW and software V11.287
I'm using:
project:
name: "syssi.esphome-jk-bms"
# version: 2.0.0
version: 1.5.0
Always i have this messages and the information is not updated:

Only restarting the ESP8266 and a few minutes after I receive the cell state once:

Could you help me?
Using the BlueTooth connection the ESP8266 show everything but the connection is not stable and the JK is bipping all the time, so i want to have it connected with the UART port (GPS connector)
Kind regards
Luis.


Added invariant culture for stringify type serialization

If the end users switches culture during the install of the app, it may crash since decimal types will be either comma or period seperated depending on the culture.
The additional change the null check on the event, is a favor for my cross compilation issues, what does the $ mean on the strings?

The $ is c# 6 string.format as is the null check with ?
I'll commit this one, but I usually stick to the latest syntax.


Locators version 0.1.10 dont have ServiceProperty

ServiceProperty restored in version 0.1.11 (pull request #25)


UI Base - Update how we configure client data inclusion and injection of "auth" plugins

Description

Migrate the client constraints/data configuration sidebar from the FF Auth plugin to core
Introduces socket constraints/control to all widget types, including third-party widgets
Introduce a "client data providers" list in the sidebar to make it clear what data is appending to msg._client

Todo

Update documentation for plugins
Add documentation for how to use the client configuration (and update the link in the Editor)

Related Issue(s)
Closes #853

Screenshot of the sidebar:


Updating collect-timeout will not change scrape_timeout configuration

The scrape_timeout for Prometheus is set as part of the charm's init method when instantiating the CosAgentProvider class. This value is obtained from the charm's collect-timeout config.
But the problem is that once the charm is running and the user changes the value of collect-timeout, this will not change the scrape_timeout value again since the charm's init method won't be called. So Prometheus will always be configured with the default collect-timeout value of 10s.

Since we're updating the relation data when config change event occurs (due to the refresh_events configuration), it would look like the scrape_timeout would also be updated along with that.
This would require some manual verification to be 100% certain. Maybe we could have a new test for this as part of the COS integration tests?


Smart edit prevents nvim -d from working

Hello !
When let g:suda_smart_edit = 1 is set in my init.vim file, I can't use nvim -d to display a diff between 2 read-only files (or 1 read-only file, and 1 writable file).
Without the option set, everything is fine.
Thanx for the help,
Best regards,
Dupond

I have no idea about nvim -d so please provide details.

nvim -d is basically vimdiff equivalent.

Well, I'm sorry but I'm not esp so closed. Please feel free to re-open or create a new one with detailed information about the issue and reproduction procedure.

This is not my business at all (in fact, I just came across this issue by accident), but shouldn't this be marked as 'help wanted' instead of just closing?


Circular dependency resulting in infinite loop

I am working on a server with roughly 400 files, using GraphQL which lends itself to circular dependencies. I'm trying to use madge to prevent this as best I can, but I'm running into an issue where it appears madge ends up in an infinite loop. I've traced the issue, and fixed it for my purposes so I wanted to bring it to light here and see if it's a bug or if I'm doing something wrong.
My project is structured so that I have a bunch of component directories, each with an index.js that exports all the modules in the component. So something like this:
- component
  - controller.js
  - model.js
  - index.js
  - graphql
    - type.js
    - mutation.js
    - query.js
    - index.js

I've traced the issue down to tree.js#L169. It appears in the case of my project depTree (which is the result of the call to dependency-tree has in itself circular references. I'm not sure if this is intended by dependency-tree, but I'm assuming it is. This would be fine if the convertTree function had a short-circuit in the event that there was a circular reference, but I can't find where it does that. I'm assuming that it is intended for tree to be that object. When I change the function to be the following it fixes the issues:
convertTree(depTree, tree, pathCache) {
    for (const key in depTree) {
        const id = this.processPath(key, pathCache);

        if (!tree[id]) {
            tree[id] = [];

            for (const dep in depTree[key]) {
                tree[id].push(this.processPath(dep, pathCache));
            }

            this.convertTree(depTree[key], tree, pathCache);
        }
    }

    return tree;
}

Basically, I only call to convertTree in the event that id isn't found in the flat tree. This seems like it might be the intended functionality, but I'm not sure exactly so I thought I would post here rather than make a PR.
Has anyone else run across this same issue?

It solves me too! Thank you for sharing your solution.
(I use exactly same structure like @wjohnsto , export every module from a folder via index.js)

I have this problem as well. It currently takes madge 12.5 minutes to check our codebase for circular dependencies. After making the change @wjohnsto suggested, it takes only 22 seconds. For what it's worth, we also have quite a few index.js files that re-export things within their folder, though I didn't verify that that was the cause of the problem.

I do think this is the intended functionality. I've been trying to think of a case where you would want it the other way, but I'm unable to find one. Maybe @pahen understands the logic a little more, it seems like he has done the bulk of the work in tree.js.

Where you able to verify that the tree from dependency-tree had a circular reference in your project? @wjohnsto I'm curious how it looked. I'm looking at writing a test for the fix you provided ..

I was able to verify that there were multiple circular dependencies. I can try to work up the simplest example I can, as what I have currently is a little bit complex for a gist.

Thanks for all your effort @wjohnsto :) I was able to verify the problem and your fix now. Good job! I've added your fix and released it in Madge v3.1.0 now.

Awesome! 👍


chore: release 1.2.83

:robot: I have created a release *beep* *boop*
1.2.83 (2022-04-04)
Bug Fixes

cleanup: manage custom tags that do not have associated existent stages (ef6efc3)
ignoring broken config in container registry (50ed5c7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/werf/releases/tag/v1.2.83 :sunflower:


Mark as unread missing

The ability to mark articles as unread seems to be missing from the latest update

Fixed


Refactor test suite to use mock data instead of the database

Currently, much of the existing test suite is using a test database and is consistently deleting all data in the database before each test case. This can cause conflict between test cases executed in parallel sharing the same test database resources. This is why the CI workflow is sequentially executing automated testing.
To remove the requirement of a database from the test suite would allow parallel workflow testing, would optimise test performance time, and would resolve the chance of push and pull request event driven CI workflows failing because of conflict between workflows executed by separate events that could share the same resources, such as a commit push event starting a new workflow before another workflow from a prior push event to the same repository branch has completed.
To remove sequential CI testing following removing the requirement of a database for testing, remove occurences of
max-parallel: 1 from the .github/workflows/node.js.yml

this will be partially fixed in #90 and will be finished in a later ticket. The preferred solution is to use an in-memory database


[DOC] Unattend README misleading/incorrect

If adding additional user content, place an unattend.xml to be imported into the image here.
This unattend.xml answer file acts indentically to an autounattend.xml answer file placed in the root of the installation media.

This isn't entirely true, any windowsPE or offlineServicing configuration pass (i.e. phases 1 & 2) must be contained within autounattend.xml, as the 'Panther' unattend.xml cannot apply those settings. This should be properly documented.

OfflineServicing does not require an autounattend.xml and also why the module directly applies the unattend.xml directly to the offline image as well as copying it to the Panther directory.

@DrEmpiricism What about windowsPE? Because the only reason I came to this conclusion is that only phases 3-7 were working when I used OO's method, only by applying it to the root system drive as autounattend.xml could I get phases 1 and 2 to work.


[openfbx] Miniz dep was removed in latest version and replaced by libdeflate

Miniz dep was removed in latest version and replaced by libdeflate.
So latest version vcpkg package is not working because it still have miniz as dependency instead of libdeflate now.
I have to fix the port version 2024-05-08.

The library has a proper cmake support. The port should use it instead of the vendored one.

The library has a proper cmake support. The port should use it instead of the vendored one.

Ah ok, so what should i do ?
I'm not familiar with updating vcpkg ports :(

Please run command ./vcpkg x-add-version openfbx after finish your changes, and command agagin.

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.

Ok, just did it...
Now i have this error :
Error: PRs must add only one version, and must not modify any published versions.
When making any changes to a library, the version or port-version in vcpkg.json must be modified, and the version database updated.
Making the following changes will fix this problem:```diff
error: checked-in files for openfbx have changed but the version was not updated
version: 2024-05-08#1
old SHA: 0044aa1e6ff32e6287babe258cab76c7f7bf8bf7
new SHA: 92185e9f54b9847f4c4b87cf7ffbe4f9522f4dda
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
See https://learn.microsoft.com/vcpkg/commands/add-version?WT.mc_id=vcpkg_inproduct_cli#--overwrite-version for more information.
No files were updated

This is ci error log, please take a look:
CMake Error at CMakeLists.txt:12 (target_link_libraries):
  Target "openfbx" links to:

    libdeflate::libdeflate

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

Please run command ./vcpkg x-add-version openfbx --overwrite-version and commit again.

Tested usage successfully by openfbx:x64-windows


Peer dependency issue with npm 7

Problem or feature description
Using this package with npm v7 in a repository with Rect 17 will cause a peer dependency error. A suggested fix by @wildpow has been proposed in https://github.com/sghall/react-move/pull/71.

Can we please get this merged in as it breaking dependencies down the line. Thanks.

The increased peer range was released in version 6.5.0 👍  Please re-open this if you encounter any issues...


JSON object parsing "problem" in launch.json

Issue Type: Bug

Create the ill-foramted json object in launch.json as below:

{
    "version": "0.2.0",
    "configurations": [

        {
            "name": "Test training",
            "type": "python",
            "request": "launch",
            "program": "${cwd}/main.py",
            "console": "integratedTerminal",
            "env":{
                "LANG": "C.UTF-8"
            },
            "args": [
                "--mode", "train"
                "--data-dir", "./data/msg",
                "--output-dir", "./tmp_output",
                "--dataset", "some_dataset"
            ]
        }
    ]
}

(Note that a comma is missing after "train",, which means that the json object is invalid. I verify this with json.loads in python and got the following error: json.decoder.JSONDecodeError: Expecting ',' delimiter: line 16 column 17 (char 388))

Execute the debugger

The debugging process runs without problem, instead of failing due to json parsing error. The args for debugger turn out to be correct too, given the terminal output like
root@10d189da27e6:/home/polonsky/Documents/EntityRelationExtraction# cd /home/polonsky/Documents/EntityRelationExtraction ; env LANG=C.UTF-8 PYTHONIOENCODING=UTF-8 PYTHONUNBUFFERED=1 /usr/bin/python3 /root/.vscode-server/extensions/ms-python.python-2020.2.64397/pythonFiles/ptvsd_launcher.py --default --client --host localhost --port 44049 /home/polonsky/Documents/EntityRelationExtraction/main.py --mode train --data-dir ./data/msg --output-dir ./tmp_output --dataset some_dataset

Please veryfy if this is a bug or some auto-correction feature.
Extension version: 2020.2.64397
VS Code version: Code 1.42.1 (c47d83b293181d9be64f27ff093689e8e7aed054, 2020-02-11T14:50:36.977Z)
OS version: Linux x64 5.3.0-40-generic snap
Remote OS version: Linux x64 5.3.0-40-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7300HQ CPU @ 2.50GHz (4 x 3100)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: disabled_softwareviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
2, 2, 1

Memory (System)
7.66GB (1.23GB free)

Process Argv
--force-user-env --no-sandbox --unity-launch --no-sandbox

Screen Reader
no

VM
0%

Item
Value

Remote
Container tensorflow/tensorflow:1.15.2-…

OS
Linux x64 5.3.0-40-generic

CPUs
Intel(R) Core(TM) i5-7300HQ CPU @ 2.50GHz (4 x 3100)

Memory (System)
7.66GB (1.23GB free)

VM
0%

Please note that by adding : instead of ,, after "train", the debugger will run too.

We actually don't parse the launch.json, VS Code does on our behalf. It's also extended JSON which is much more lenient than standard JSON. Your best option would be to report this on the library that VS Code uses for parsing.


Update teams-standalone-static-tabs-using-spo-sites.md

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LanaChin PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


CB-7862: FileReader reads large files in chunks with progress

I added support in FileReader for reading large files in chunks and reporting progress for each chunk. Previously, the Cordova FileReader would always read files as a single chunk (resulting in OOM exceptions for files > 10 MB or so), and would NEVER invoke the onprogress callback. With this change, it's possible to read files with sizes up to the actual available application memory: I have verified with files over 500 MB.
The default chunk size of 256 KB is based on some non-scientific timing tests I did: it yields only minor overhead from multiple exec() calls while still providing frequent-enough progress reports for large files. Much smaller chunk sizes could cause large file reads to take significantly longer due to the numerous exec() calls (and progress callbacks).
The FileReader.READ_CHUNK_SIZE value is exposed and may be adjusted by app code, though I don't expect that is something that would need to be changed normally so I didn't add it to the plugin documentation. The added test case however does use that capability so that it can verify chunking and progress behavior without having to generate a very large file.

@rakatyal or @riknoll please review

LGTM. Tested on Marshmallow device and KitKat emulator


User profile images + additional information

Would be nice to have this, or, have some predefined ones available.
When in doubt, gravatar is always works.
I know it seems silly but in the realm of multi-tenancy (#16) it might apply to have a company logo for a profile, etc
Would be nice to have a phone number in there too or support for #158

Where would user profile images be displayed?

I think to honestly answer that we'd need to see the initial roll out of #16 and what it introduces into the UI. If the only place it might appear would be in the profile then it's usefulness is limited.
If "Recent Activity" is going to be logging actions of other multi-tenant users, it might be nice to identify these actions by a profile image to the left (or at least the profile name, etc)
Don't have a great answer =)

I don't see significant value in supporting user avatars since the concept of a user in NetBox is mostly just for authentication and authorization purposes. We can possibly extend the tenant model to include additional info, but that would be a separate issue (maybe covered by #503). I'm going to close this out due to lack of activity.


Error in an accepted Push request

I'm submitting a ...
[ ] bug report
[ ] feature request
[x] question about the decisions made in the repository
[ ] question about how to use this project

Summary
Hi there, I have made a pull request to add the "Samoyed Coin" token, and it was accepted, thank you ! However, I didn't put a direct link for the image of token, so the image does not appear in the Solana Explorer, is it possible to update the logo link in solana.tokenlist.json with this one please ? https://i.ibb.co/tLGpvNf/samo.png

Also, I don't know if it's linked but, but nothing appear on the Solana Explorer https://explorer.solana.com/address/7xKXtg2CW87d97TXJSDpbD5jBkheTqA83TZRuJosgAsU/largest?cluster=mainnet-beta
The name of the token should appear right ? Only the logo was not the good one in the PR
Thank you in advance for your help ! :)

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

I believe Solana Explorer repo needs to be updated to use latest token-list version and then deployed live. We have the same problem with STEP token not displaying meta-data yet.

Oh okay, thank you for your comment ! So If a dev can just update the logo link, it would be perfect thank you :)


Migrating

I have been trying to apply the same base code for my project but the registration page gives an integrity error. I'm not using sqlite but postgres. Is there something I'm supposed to change besides the database URI?

Hi. I'm sure there's nothing else you need to do. The integrity error is
typically caused by a duplicate record in the database.
On Sun, 25 Dec 2022 at 19:41, DAHCROWN @.***> wrote:

I have been trying to apply the same base code for my project but the
registration page gives an integrity error. I'm not using sqlite but
postgres. Is there something I'm supposed to change besides the database
URI?
—
Reply to this email directly, view it on GitHub
https://github.com/Dev-Elie/User-Authentication-in-Flask/issues/4, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANXIOXDH47U6KUH52NA53NLWPB2MZANCNFSM6AAAAAATJCCSXA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


FE-1092 Feature/stations rewards screen

Why?
Implementation of new Stations Rewards screen.
How?

Created the respective ApiModels which are needed in order to parse the new responses in the 2 new get calls in ApiService (which are used to populate this screen)
Created the getDevicesRewardsByRange and getDeviceRewardsByRange in DataSource & Repository which respectively use these new API calls in order to get the data
Created the respective UIModels which are used to populate the UI and are passed to the view layer.
These UIModels are created in RewardsUseCase and DevicesRewardsViewModel using the API models.
Created DevicesRewardsActivity which is the activity of the screen along with its DevicesRewardsViewModel
Created RangeSelectorView component which is the "7D/1W/1Y" selector above the charts in order to reuse it anywhere we want.
Created TooltipMarkerView component which is the tooltip in the charts and is currently used in both "Total Earned" and "Rewards Breakdown" chart.
Created DeviceRewardsAdapter along with its view which is the list item for every "Device Reward Summary" card in this screen
Created DeviceRewardsBoostAdapter along with its view which is the list item for every "X boost reward details" table of information
Created initTotalEarnedChart and initRewardsBreakdownChart functions in util/Charts in order to initialize, populate and show these 2 new charts properly.
Created ZonedDateTime.getFormattedMonthDate() function which returns a ZonedDateTime in DD/MM or MM/DD depending on user locale.
Created RetryCardView which is a reusable card as per the design, which is used in the case of errors in this screen inside each card respectively.

Testing
For now it can be tested only via local mock flavour, the respective files which can be edited are get_user_device_rewards_summary.json and get_user_devices_rewards.json

This is ready for review @pantelisss and includes the latest iteration of the error state.


Page completely reloads when opening modal

Anytime I open a modal the page beneath completely reloads and resets it's state. Is there a way to prevent that?

Now that you mention it, I'm experiencing the same behavior.

Can't confirm without an example code. Please share an example gist or a repo.
Make sure that:

You're triggering a modal with an Inertia Link instead of a regular a tag.
In case your app has axios installed, upgrade it to the same version Inertia uses. npm i<EMAIL_ADDRESS>

I have the same issue. It becomes clear when I setup page transitions. Then, whenever I open a modal, the page behind it transitions out and in again, because it is completely re-rendered as mentioned.
Already tried npm i<EMAIL_ADDRESS>with no luck.

@MtDalPizzol please set up an example repository with the issue reproduction.

There you go: https://github.com/prontostack/pronto-fuel/tree/momentum-modal
I've been working on this starter kit since before Laravel adopted Vite as the default bundler and now I'm adding some stuff I always end up using and I think momentum-modal would be a great improvement. I've added a momentum-modal branch which contains the exact behavior I described. Be aware that I have upgrade axios though...

After further investigation, it seems the problem isn't related to momentum-modal, rather that Inertia is triggering a layout to re-render even if using the persistent layouts feature.

I had a look at the PingCRM project that uses modals (https://modal.advanced-inertia.com/) and it looks like you need to add :only=['modal'] to the Inertia link. Once I do that, the page doesn't reload and only the queries necessary for the modal get run. Without only, I have the same problem as described in this issue.

@ksassnowski Tried that. Still the same problem...

I've also upgraded the inertiajs/inertia-laravel to 0.6.3 so it correctly set the inertia assets version.

Found it! What did the trick for me was adding :preserve-state="true" to the Inertia Link. Wouldn't this be the default expected behavior though?

The slideover page doesn't use partial reloads: https://github.com/lepikhinb/momentum-modal-example/blob/master/resources/js/Pages/Organizations/Index.vue

Well... I did dig into this for the last few days, and what worked (and made more sense to me), was to create a wrapper DialogLink component that simply sets preserve-state, preserve-scroll and :only=['modal'].
And this is important: :only=['modal'] DOES CHANGE what comes back from the server. So, it seems reasonable to me to always set this, since when it is not present, the JSON that comes back from the server also contains all the data from the baseRoute, which doesn't make sense to me, since the baseRoute is already rendered and should not change when the dialog opens.
Also, I've upgrade axios to the latest version (0.27.2) and everything is working.
<template>
  <Link
    preserve-state
    preserve-scroll
    :only="['modal']"
  >
    <slot />
  </Link>
</template>

@MtDalPizzol

And this is important: :only=['modal'] DOES CHANGE what comes back from the server.

It does since that's how Inertia works. You should load partial data like this (even though it's not necessary), but don't forget making parent page's props Lazy.


Save dashboard state

As a user I can save the current state of my CapitalConnect dashboard so that I do not have to create a new dashboard everytime.

Implemented in AB3 by default


🐛  s3 certs + hoot [b]

whoooo000000000ps! been a bit busy as of late 🥴
aws certs expired 2024-09-25 🤣

This pull request has been deployed to Vercel.

Latest commit:
478a6cd

✅ Preview:
https://jeromefitzgerald-84h0js5l3-j-e-r-o-m-e.vercel.app

🔍 Inspect:
https://vercel.com/j-e-r-o-m-e/jeromefitzgerald/BqNkEmUsry25Fpm1XdXz1HduJRCk

View Workflow Logs


Add support to export as LaTeX and Tikz

Are there any plans to support exporting diagrams to LaTeX (with math and Tikz)?
Example of an implementation: https://www.mathcha.io/

Sure, that could make it easier for a latex input and latex output design. Let me know if there are some low hanging fruits for which the core team doesn't have time.
I am just making sure this has eyeballs since the collab+open source nature of excalidraw is unbeatable (except perhaps text -> image of plantuml).


Improve and test optional imports

AC - tested function that does optional imports is in place

@pavelkrizek have a look please

Codecov Report

Merging #34 into master will increase coverage by 1.56%.
The diff coverage is 98.63%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
+ Coverage   91.65%   93.21%   +1.56%
==========================================
  Files          55       55
  Lines        2456     2448       -8
==========================================
+ Hits         2251     2282      +31
+ Misses        205      166      -39

Impacted Files
Coverage Δ

tests/integration/test_data_unchanged.py
97.61% <90.90%> (+17.26%)
:arrow_up:

...rc/hcrystalball/model_selection/_model_selector.py
95.40% <100.00%> (ø)

src/hcrystalball/utils.py
84.12% <100.00%> (+1.07%)
:arrow_up:

src/hcrystalball/wrappers/__init__.py
100.00% <100.00%> (+52.83%)
:arrow_up:

tests/unit/model_selection/test_model_selector.py
100.00% <100.00%> (ø)

tests/unit/test_utils.py
95.14% <100.00%> (+1.72%)
:arrow_up:

...talball/feature_extraction/_holiday_transformer.py
97.22% <0.00%> (+2.77%)
:arrow_up:

Continue to review full report at Codecov.

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


unable to add password

Output of restic version
restic 0.8.1
compiled with go1.9.2 on windows/amd64
How did you run restic exactly?
Powershell> restic -r C:/path/to/repo key add

What backend/server/service did you use to store the repository?
test on local drives
Expected behavior
add new password

Actual behavior
enter password for repository:
password is correct
read /dev/stdin: Descriptor not valid
ReadFull
main.readPassword
        src/github.com/restic/restic/cmd/restic/global.go:249
main.ReadPassword
        src/github.com/restic/restic/cmd/restic/global.go:285
main.ReadPasswordTwice
        src/github.com/restic/restic/cmd/restic/global.go:302
main.getNewPassword
        src/github.com/restic/restic/cmd/restic/cmd_key.go:68
main.addKey
        src/github.com/restic/restic/cmd/restic/cmd_key.go:74
main.runKey
        src/github.com/restic/restic/cmd/restic/cmd_key.go:155
main.glob..func10
        src/github.com/restic/restic/cmd/restic/cmd_key.go:22
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).execute
        src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:698
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).ExecuteC
        src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:783
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).Execute
        src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:736
main.main
        src/github.com/restic/restic/cmd/restic/main.go:69
runtime.main
        /usr/local/go/src/runtime/proc.go:195
runtime.goexit
        /usr/local/go/src/runtime/asm_386.s:1635
unable to read password
main.ReadPassword
        src/github.com/restic/restic/cmd/restic/global.go:289
main.ReadPasswordTwice
        src/github.com/restic/restic/cmd/restic/global.go:302
main.getNewPassword
        src/github.com/restic/restic/cmd/restic/cmd_key.go:68
main.addKey
        src/github.com/restic/restic/cmd/restic/cmd_key.go:74
main.runKey
        src/github.com/restic/restic/cmd/restic/cmd_key.go:155
main.glob..func10
        src/github.com/restic/restic/cmd/restic/cmd_key.go:22
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).execute
        src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:698
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).ExecuteC
        src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:783
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).Execute
        src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:736
main.main
        src/github.com/restic/restic/cmd/restic/main.go:69
runtime.main
        /usr/local/go/src/runtime/proc.go:195
runtime.goexit
        /usr/local/go/src/runtime/asm_386.s:1635

Steps to reproduce the behavior
issue with restic-0.8.1, but works with restic-0.8.0

Do you have any idea what may have caused this?
no
Do you have an idea how to solve the issue?
no
Did restic help you or made you happy in any way?
just testing.

Thanks for a good report. May I ask if you compiled both 0.8.0 and 0.8.1 yourself? If yes, did you check out the release tags or some other commit? What command did you use to compile them?

Hi,
I've just downloaded the versions from releases, tested with x86 and x64 versions with not change.

Well, the issue is not well described.
I replaced the downloaded restic-0.8.1 by the compiled restic to and run into the same problem!?
In one case, I had registered the RESTIC_PASSWORD as an environment variable and then removed it. As soon as I removed it, the problem reoccured.
When there's no RESTIC_PASSWORD, we're prompted for the repo password, and then it crashes.
I reproduced it hereafter:
PS>$env:RESTIC_PASSWORD='pass'
PS>restic -r path/to/repo key add
password is correct
enter password for new key:
enter password again:
saved new key as...
PS>$env:RESTIC_PASSWORD=$null
PS>restic -r path/to/repo key add
enter password for repository:
password is correct
read /dev/stdin: Descriptor not valid
ReadFull
main.readPassword[...]

Sorry if this feature was documented somewhere and that I hadn't seen it.
Thanks

Thanks for the report!
It is related to the issue, it looks like asking for a new password is broken on Windows when using PowerShell. The workaround is passing the password to restic via the environment variable.
Somebody with a Windows development environment and knowledge of Go is needed to help debugging this.

I’m new to github and reporting issues, so I hope this can help.
I’m also getting the same issue as originally described, however I can also add the following:

While testing with the latest version of the restic_0.8.2_windows_amd64 (from the releases page) I have seen the issue on both Windows 7 and Windows 2016 server

The issue also does not seem to be restricted to PowerShell environment, I have been able to duplicate the error on both these platforms in the Windows Command Line

Downgrading to restic_0.8.0_windows_amd64 the issue goes away. I am able to add keys in either PowerShell or Command line environments

One final thing, all my testing involved “one liners” with no environmental variables set, and being prompted for password.

I used  the following without powershell and still errors :-
restic 0.8.3
compiled with go1.10 on windows/amd64

hi
i had the same problem and no enviroment was set with the binary 0.8.3 download
but after compiling from the current source, i have no problems now
so i think it is fixed now?
restic 0.8.3 (v0.8.3-221-g26757ae) compiled with go1.10.1 on windows/amd64

Can someone please confirm what's the current status here? Is this still an issue? I don't have a Windows machine at hand...

The downloaded version has still this issue:
> restic version
restic 0.8.3
compiled with go1.10 on windows/amd64
> restic key add -r data
enter password for repository:
password is correct
read /dev/stdin: Descriptor not valid
ReadFull
main.readPassword
        src/github.com/restic/restic/cmd/restic/global.go:251
main.ReadPassword
        src/github.com/restic/restic/cmd/restic/global.go:287
main.ReadPasswordTwice
        src/github.com/restic/restic/cmd/restic/global.go:304
main.getNewPassword
        src/github.com/restic/restic/cmd/restic/cmd_key.go:72
main.addKey
        src/github.com/restic/restic/cmd/restic/cmd_key.go:78
...

SHA1 hash of restic.exe: CAD7174FC452429705B2468ABCFB5C2D07E75FF6
Is it maybe a bug fix in the Go toolchain? cd1188 is using Go 1.10.1. Is the Go version of the download binary 1.10.0?

download 0.8.3 as binary 64bit has this error by me in cmd and powershell on windows 10
(compiled with go1.10 on windows/amd64)
SHA1 CAD7174FC452429705B2468ABCFB5C2D07E75FF6
read /dev/stdin: Descriptor not valid -> is that not a linux path or is that also valid in windows?
compiling from source has no error...
I do not know more.. sorry

Hello
i downloaded now the old 0.8.3 source from the homepage and compiled it with the new go-compiler 1.10.1 and the old error is present...
so i can say this error is in the old source-code and not in the go-toolchain
i used this old-src
restic-0.8.3.tar.gz
SHA1 4617F0FAC9CC804C46465A17F569ED237566472A
C:\restic_src\restic-0.8.3>restic version
restic 0.8.3
compiled with go1.10.1 on windows/amd64
C:\restic_src\restic-0.8.3>restic -r C:\repo key add
enter password for repository:
password is correct
read /dev/stdin: Das Handle ist ungültig.
ReadFull
main.readPassword
src/github.com/restic/restic/cmd/restic/global.go:251
main.ReadPassword
src/github.com/restic/restic/cmd/restic/global.go:287
main.ReadPasswordTwice
src/github.com/restic/restic/cmd/restic/global.go:304
main.getNewPassword
src/github.com/restic/restic/cmd/restic/cmd_key.go:72
main.addKey
src/github.com/restic/restic/cmd/restic/cmd_key.go:78
main.runKey
src/github.com/restic/restic/cmd/restic/cmd_key.go:159
main.glob..func11
src/github.com/restic/restic/cmd/restic/cmd_key.go:22
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).execute
C:/Users/XXXX/AppData/Local/Temp/restic-build-134152615/src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:698
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).ExecuteC
C:/Users/XXXX/AppData/Local/Temp/restic-build-134152615/src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:783
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).Execute
C:/Users/XXXX/AppData/Local/Temp/restic-build-134152615/src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:736
main.main
src/github.com/restic/restic/cmd/restic/main.go:69
runtime.main
C:/Go/src/runtime/proc.go:198
runtime.goexit
C:/Go/src/runtime/asm_amd64.s:2361
unable to read password
main.ReadPassword
src/github.com/restic/restic/cmd/restic/global.go:291
main.ReadPasswordTwice
src/github.com/restic/restic/cmd/restic/global.go:304
main.getNewPassword
src/github.com/restic/restic/cmd/restic/cmd_key.go:72
main.addKey
src/github.com/restic/restic/cmd/restic/cmd_key.go:78
main.runKey
src/github.com/restic/restic/cmd/restic/cmd_key.go:159
main.glob..func11
src/github.com/restic/restic/cmd/restic/cmd_key.go:22
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).execute
C:/Users/XXXX/AppData/Local/Temp/restic-build-134152615/src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:698
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).ExecuteC
C:/Users/XXXX/AppData/Local/Temp/restic-build-134152615/src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:783
github.com/restic/restic/vendor/github.com/spf13/cobra.(*Command).Execute
C:/Users/XXXX/AppData/Local/Temp/restic-build-134152615/src/github.com/restic/restic/vendor/github.com/spf13/cobra/command.go:736
main.main
src/github.com/restic/restic/cmd/restic/main.go:69
runtime.main
C:/Go/src/runtime/proc.go:198
runtime.goexit
C:/Go/src/runtime/asm_amd64.s:2361
C:\restic_src\restic-0.8.3>

Ok, thanks for the confirmation. We can then close this issue, as it is not present in the code in the master branch any more (we're using a library for reading the password from the user, and that has been updated since 0.8.3, I think this solved it).
I'm closing this issue for now, please feel free to add further comments. Thanks!


No Full List of Commands

There is no comprehensive list of TP commands anywhere. Could someone update the wiki with more updated commands?

The recommended list of TP commands is here: https://samfun123.github.io/KtaneTwitchPlays/ It's not comprehensive yet but I've been the only one who has been working on documenting commands. If anyone wants to help, just let me know and I explain how to.
I still need to make sure everything from the wiki has been moved to the new documentation but I'll at least put up that this is the recommended place to look.

@samfun123 that documentation is great so far. Could you consider adding other bits of info such as configuration options, modes, (like what they do) etc?

I think I want to keep that documentation for the players of TP. So modes could go on there but stuff for streamers is probably better on the wiki.

Since pretty much every command has been documented now, I'm going to close this.


Use token logos under Pool Catalyst

Problem
Within the Pool Catalyst section of a pool's page (/pool/###), the current colored circles with bubbles in them don't serve much value, and misses an opportunity to make it quicker and easier to visually understand which tokens make up the pool.
Solution
Replacing the generic bubbling liquid with the token logo will vastly improve readability. See image below for an example of current vs proposed.

Hi Jeremy, we're currently working on similar UX solutions in #152. Shipping soon:)


PDF invalid font name used in dictionary (very small fix)

Bug Report
Description of the problem
Corrupted PDF files get generated if the font name includes a space. To be more precise, this is the postscript name extracted from the font file itself, not the font family given by the user. The invalid dictionary structure produced gets ignored by browser PDF viewers but programs like Adobe Acrobat Reader DC don't even open the PDF file.
Font: Uni Ila.Sundaram-01
Corrupted.pdf
Fixed.pdf
Code sample
The following changes generated a correct PDF file. Just replaced spaces with an _ for the font name
class EmbeddedFont extends PDFFont {   constructor(document, font, id) {     ...     this.name = this.font.postscriptName.replace(' ', '_');     ...   }   ...   ...   embed() {     ...     const name = tag + '+' + this.font.postscriptName.replace(' ', '_');     ...


Preview is styled by other pages in browser

When using 'More Browser Options/Open In Browser', the resulting page can become styled by whatever is in the Chrome browser, losing its original styling which is still visible in VS Code. This can be got round by closing all tabs and clearing the cache in Chrome, but still leaves the question of why other styling overrides what is shown originally in VS Code. 'Open in browser' does not necessarily mean 'open in browser with the same styling'.

Thanks for pointing that out! I added a FAQ question for that, but I'll keep the phrasing the same in the extension so that it isn't too long of a title.


gridcoin-research: <IP_ADDRESS> -> <IP_ADDRESS>-hotfix-1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/gridcoin-community/Gridcoin-Research/releases.
meta.description for gridcoin-research is: A POS-based cryptocurrency that rewards users for participating on the BOINC network
meta.homepage for gridcoin-research is: https://gridcoin.us/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/fcj53rc6pfvzpsaiv9zdb5ghycs9g3sg-gridcoin-research-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/fcj53rc6pfvzpsaiv9zdb5ghycs9g3sg-gridcoin-research-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
gridcoin-research

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fcj53rc6pfvzpsaiv9zdb5ghycs9g3sg-gridcoin-research-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gridcoin-research https://github.com/r-ryantm/nixpkgs/archive/90f7f3a0240addb891d4930c8e38cca720b0dc0f.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/90f7f3a0240addb891d4930c8e38cca720b0dc0f#gridcoin-research

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fcj53rc6pfvzpsaiv9zdb5ghycs9g3sg-gridcoin-research-<IP_ADDRESS>
ls -la /nix/store/fcj53rc6pfvzpsaiv9zdb5ghycs9g3sg-gridcoin-research-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

gridcoin-research

Maintainer pings
cc @gigglesquid for testing.

Do not merge yet. Actual testing required

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/gridcoin-community/Gridcoin-Research/releases.
meta.description for gridcoin-research is: A POS-based cryptocurrency that rewards users for participating on the BOINC network
meta.homepage for gridcoin-research is: https://gridcoin.us/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS>-hotfix-1 in filename of file in /nix/store/m0qgg5zh5n3hyibfmwi1h4iql34g6szx-gridcoin-research-<IP_ADDRESS>-hotfix-1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
gridcoin-research

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/m0qgg5zh5n3hyibfmwi1h4iql34g6szx-gridcoin-research-<IP_ADDRESS>-hotfix-1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gridcoin-research https://github.com/r-ryantm/nixpkgs/archive/ed9afea01260da46a169105421489f24e26ce96e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ed9afea01260da46a169105421489f24e26ce96e#gridcoin-research

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/m0qgg5zh5n3hyibfmwi1h4iql34g6szx-gridcoin-research-<IP_ADDRESS>-hotfix-1
ls -la /nix/store/m0qgg5zh5n3hyibfmwi1h4iql34g6szx-gridcoin-research-<IP_ADDRESS>-hotfix-1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

gridcoin-research

Maintainer pings
cc @gigglesquid for testing.

Added backport to release-23.11
This version bump (at least <IP_ADDRESS>) is now required by the blockchain. Older versions of the wallet will not function on the current blockchain


DCOS-14010: Add container to pick logging component and API

Note: DCOS-14078 will take care of the breadcrumb bug. It is not handled in this PR!
~Dependent on this PR: https://github.com/dcos/dcos-ui/pull/1907~ (merged)

This PR adds functionality to choose one of the existing logging APIs. To use Mesos API set this variable in your Config.dev.js:
mesos: {
  'logging-strategy': 'logrotate'
}

To use journald API set this variable in your Config.dev.js:
mesos: {
  'logging-strategy': 'journald'
}

It will limit the files to view to be only stdout and stderr, when using Mesos API. Other files will still be available to view using the file viewer under the files tab.
Using Mesos API:

Using journald API

Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[x] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future? (Not applicable)

For reviewers: View diff between PRs here Merged!

Run integration tests

When I set it to logrotate it seems that there are some problems with going backwards in the log viewer. Also the "Go to bottom" button doesn't ever go away, but I'm not sure that's related to this PR.
Here's a screen recording: https://cl.ly/2O0R2Z3C2G16

I ran into the following errors while testing this and don't feel comfortable green-lighting this one without the ability to properly test it esp. as the loading was somewhat flaky and is very slow.

https://jira.mesosphere.com/browse/DCOS_OSS-715
https://jira.mesosphere.com/browse/DCOS_OSS-718
https://jira.mesosphere.com/browse/DCOS_OSS-717
https://jira.mesosphere.com/browse/DCOS_OSS-716

FYI: Number 1. and 4. is because this backend fix has not merged yet: https://github.com/mesosphere/dcos-enterprise/pull/654

NB: I changed the base of this point to a feature branch! Also, I put up a PR for the MesosLogView fixes here: https://github.com/dcos/dcos-ui/pull/1974
I still need to fix these two issues:

https://jira.mesosphere.com/browse/DCOS_OSS-718
https://jira.mesosphere.com/browse/DCOS_OSS-717

And as mentioned before these have been fixed in the backend:
https://jira.mesosphere.com/browse/DCOS_OSS-715
https://jira.mesosphere.com/browse/DCOS_OSS-716

When using the mesos API I get this warning when I scroll in the log window:

The Go to bottom button seems a little flaky (sometimes shows, sometimes doesn't, sometimes shows when at bottom already) but I don't see it as a blocker.

@MatApple The scrolling setState error will be fixed by updating to the next release of reactjs-components

We also loose the log tab highlighting sometimes -

@MatApple that scrolling error isn't related to this task. If you simply shrink your browser to a smaller size this error will happen to any container that has a scrollbar.
This is in reference to this comment.

Looking through the remaining open PRs that follow this one I can see @mlunoe has fixed the bugs that are noted as outstanding on this PR (Tab highlight and Go to bottom flakiness) https://github.com/dcos/dcos-ui/pull/1974#issuecomment-285096797

@orlandohohmeier - @rcorral updated this PR based on your requested changes so I'm going to merge 👍


NPC is invulnerable when is set to false

Like title say FNPC is invulnerable even when is set to be off. Also notice because of this you can't set FNPC health and aways returning 100.000. I was using Bodyguards filterscript when i found it.

FCNPC_OnTakeDamage should return 1 for take effect.

Everything work fine now. Thanks again :)