Default value for train args.

https://github.com/fastnlp/fastNLP/blob/8a87807274735046a48be8eb4b1ca10801875039/fastNLP/core/trainer.py#L42-L45
Should we set some default value for train_args? Otherwise we will pass all these args every time, which is very redundant.

i think we should set some default value
if user need to change the default value, they can pass their own args

I deleted the default value in trainer. One reason is that those are important arguments that I want users to set them explicitly, otherwise ValueError will remind users. Another reason is that I want the definition of trainers clean and concise, without if-else statements.
However, one possible solution is to pre-define those default value in train_args, rather than in trainers.

We can have a default .cfg file to be loaded before the user defined .cfg in the ConfigLoader. So user can easily check what kind of args that have default, and can overwrite them by writing these args in there .cfg

so different trainer should use different default values? maybe we can provide the default value what we suggest
if the user forget load train_args or set values from train_args, should we remind them?

so different trainer should use different default values?
maybe we need that, and even for different model we have different default values.
if the user forget load train_args or set values from train_args, should we remind them?
I think we can set as many default args as we could, let user to run the code instantly without setting anything.

For the long-term goal,  we wish that the users may build their models without considering their hyperparameters, like AutoML.
Current, it's better to reduce the user's efforts.
Consider this, a dictionary of args with default value is preferred. But I suggest to put these args in formal args list directly. Thus, the user can easily know which argument has what default value.
Besides, the following codes are somewhat redundant and fragile.
`
self.n_epochs = train_args["epochs"]
self.batch_size = train_args["batch_size"]
self.pickle_path = train_args["pickle_path"]
    self.validate = train_args["validate"]
    self.save_best_dev = train_args["save_best_dev"]
    self.model_saved_path = train_args["model_saved_path"]
    self.use_cuda = train_args["use_cuda"]

`

I would say we could pass args like:
    # a trainer method need args to pass
    def __int__(self, epochs, batch_size, validate, save_best_dev, use_cuda)
          pass

# pass args
train_args = ConfigLoad.load() # load from cfg file
Trainer(**train_args)

So we can know the args without look into the method body, and write the default cfg file. user also can set there own args changing default values


fix: verify if self.bufnr is not nil before save

I reproduce the error and see that when you don't change the ui Harpoon try to save but buffer number don't exist, so this cause an error.
Issue: https://github.com/ThePrimeagen/harpoon/issues/385

Better approach: https://github.com/ThePrimeagen/harpoon/pull/392


chore[python]: apply license headers to all the source files

Summary of Changes
this pr only touches the file written in python. Will have another pr for JS/CSS/HTML
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes, including screenshots of any UI changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes all tests documented in the developer guide

Codecov Report

Merging #513 into master will increase coverage by 2.58%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #513      +/-   ##
==========================================
+ Coverage   81.04%   83.63%   +2.58%
==========================================
  Files         227      185      -42
  Lines        5683     3868    -1815
  Branches      675      501     -174
==========================================
- Hits         4606     3235    -1371
+ Misses       1009      615     -394
+ Partials       68       18      -50

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 3a8ecdb...b0039f0. Read the comment docs.

Same question here I'm wondering about why we have to add this to every file.

@ttannis cc you in the same email so that you could clarify.
cc @markgrover

But I think it would be a good practice as many OSS projects did that. And I have already merged all the backend prs.


mAP on COCO

Hi, Thanks for your repo.
I want to test this repo for my project.
Have you tested on coco. What's map in this repo?

Hi, Thanks for your repo. I want to test this repo for my project. Have you tested on coco. What's map in this repo?

I'm afraid this repo does not support testing for the time being. This repo is only a backup of my code for developing this network. It only supports training and rest. If you have an emergency, you can temporarily test your dataset. However, I haven't tested it on the coco dataset, and.If you are not in a hurry, you can wait for the following. It is expected that it will be completed by the end of September with full functions and merged to the mmlab/mmdetection. At that time, you can directly install mmdetection_ Dev3. X framework runs SparseInst.

Hi, Thanks for your repo. I want to test this repo for my project. Have you tested on coco. What's map in this repo?

I'm afraid this repo does not support testing for the time being. This repo is only a backup of my code for developing this network. It only supports training and rest. If you have an emergency, you can temporarily test your dataset. However, I haven't tested it on the coco dataset, and.If you are not in a hurry, you can wait for the following. It is expected that it will be completed by the end of September with full functions and merged to the mmlab/mmdetection. At that time, you can directly install mmdetection_ Dev3. X framework runs SparseInst.

Thanks. Will you pull MR to mmdetection ?  I used to work in sensetime. It may be need some time to merge in.

Hi, Thanks for your repo. I want to test this repo for my project. Have you tested on coco. What's map in this repo?

I'm afraid this repo does not support testing for the time being. This repo is only a backup of my code for developing this network. It only supports training and rest. If you have an emergency, you can temporarily test your dataset. However, I haven't tested it on the coco dataset, and.If you are not in a hurry, you can wait for the following. It is expected that it will be completed by the end of September with full functions and merged to the mmlab/mmdetection. At that time, you can directly install mmdetection_ Dev3. X framework runs SparseInst.

Thanks. Will you pull MR to mmdetection ?  I used to work in sensetime. It may be need some time to merge in.

Yes I am a mmlab member now.And this network will be merged in mmdetection in the future.

Thanks a lot

Hi, Thanks for your repo. I want to test this repo for my project. Have you tested on coco. What's map in this repo?

I'm afraid this repo does not support testing for the time being. This repo is only a backup of my code for developing this network. It only supports training and rest. If you have an emergency, you can temporarily test your dataset. However, I haven't tested it on the coco dataset, and.If you are not in a hurry, you can wait for the following. It is expected that it will be completed by the end of September with full functions and merged to the mmlab/mmdetection. At that time, you can directly install mmdetection_ Dev3. X framework runs SparseInst.

Thanks. Will you pull MR to mmdetection ?  I used to work in sensetime. It may be need some time to merge in.

Yes I am a mmlab member now.And this network will be merged in mmdetection in the future.

Hi， How's it going now. I try to use this repo but can't obtain a worked ap.

really sorry to reply so late, I am currently bussy with my courseworks in school, maybe It seems this repo have some issues,  especially with the data preprocessing. maybe I can fix them in the future.


Feature request: Excluding actions by org in action_ref_should_be_full_length_commit_sha

You can exclude an action from checking action_ref_should_be_full_length_commit_sha, but I would appreciate if we can exclude all actions from a certain organization.
I want to allow using action refs like @v1 if the action is provided by a "trusted" organization, such as our own one.

Thank you for your feedback.
How about supporting glob patterns?

https://github.com/suzuki-shunsuke/ghalint/pull/650

excludes:
  - policy_name: action_ref_should_be_full_length_commit_sha
    action_name: suzuki-shunsuke/* # glob pattern

@suzuki-shunsuke
Looks great! Thank you for your quick response.

🎉 https://github.com/suzuki-shunsuke/ghalint/releases/tag/v1.1.0


Fix type in Time Entry link

Coverage decreased (-38.2%) to 40.764% when pulling 67944ce1bdbcede343e95dea7f7355f499249756 on blachniet:patch-1 into 5c0fcfe0179ca82087a462a88b842f564eed4198 on dougEfresh:master.


Add make targets for starting/killing selenium in e2e-targets.mk

Actually, let's not do this, we're moving away from having the selenium stuff in beaker, opting for keeping it in webdriverio-server instead.


Choose layer order for collage pictures

Is your feature request related to a problem? Please describe.
Im working on a collage setting, where it looks like pictures lay above each other.
For this effect to look nice, it would be nice to be able to select the order of the individual images.
Describe the solution you'd like
Have the option, wheather picture should be inserted to the top or background of other images.
(Now, images will be inserted on top all of the time)
Describe alternatives you've considered
Change the settings order of the individual pictures, so that every picture is in the right layer.
This way, the first picture (and in my experience main picture) is not in the first place.
Additional context
Actual implementation:
Image order: Top left, top right, bottom left, bottom right.

Possible improvement.

I am afk right now but I think you can move the position of each image in the merge definition where you could delete one set also.
If you change the order, you also change the sequence to merge.

That‘s the Alternative I referred to, but this will also change the image order.
I would like to preserve the image order, but change the Layer order.

Perfect.
I think before/behind merge would bring a great benefit already - An explicit layer order with numbers would be the full solution.
Looking forward to this.


x/pkgsite: too many users are filing package removal requests

Say I mistakenly pushed a git tag on a Go module of mine, and now I can see it on pkg.go.dev. I only know the basics about Go modules, so I am not yet familiar with retractions. From the home page of pkg.go.dev, I find https://pkg.go.dev/about#removing-a-package:

If you would like to hide versions of a module on pkg.go.dev, as well as from the go command, you should retract them. Retracting a module version involves adding a retract directive to your go.mod file and publishing a new version. See the Go blog post New module changes in Go 1.16 and the modules reference for details.
If you cannot retract your module, you can file a request for the pkgsite team to remove your package.

Presumably, this should lead me to publish a module retraction, not to file a request. However, plenty of requests are filed regularly. I've always had the suspicion that the majority of those should be fixed with retractions instead, so I engaged with one of them. And indeed I think this particular user wants to use a retraction; they want to entirely take back a v1.0.0 tag from their users, not simply hide the documentation.
Assuming that all of these users are filing issues by following the pkg.go.dev docs as shown above, I think the docs are confusing. They should better clarify that:

A retraction is almost always what the user should do. In fact, I struggle to imagine what the site means by "If you cannot retract your module". If we have an edge case in mind, I would call it out explicitly. If not, I would remove the alternative entirely.

Removing a package from pkg.go.dev only hides the documentation. The Go module and version will still work for go get pkg@latest requests, which I think is a pretty common concern amongst users. For example, the particular issue above is about having pushed a v1.0.0 tag when the module is meant to be a v0. The wording should be clearer.
It's true that the link points to an issue template which explains this in an HTML comment, but by that point the user has already made the choice to not retract. Plus, can we trust the user to read the entire template carefully before filling the form? :)

How to retract the latest version in a clean way. https://go.dev/ref/mod does say:

To retract a version, a module author should add a retract directive to go.mod, then publish a new version containing that directive.
A version containing retractions may retract itself.

That is, to retract a v1.0.0, it is possible to publish a v1.0.1 which retracts both versions. The user can follow the links to the blog post or docs, but they have to do quite a lot of reading to find the few important sentences.
In fact, I would remove the link to the blog post, because the link to the modules reference already overlaps with its information. Plus, the example shown in the blog post is rather simple - it does not cover how to retract the latest version, for example.

--
I propose new wording. Lengthier, but hopefully clearer.

If you would like to hide versions of a module on pkg.go.dev, as well as from the go command, you should retract them. Retracting a module version involves adding a retract directive to your go.mod file and publishing a new version. See the modules reference for details.
Note that it is possible to retract the latest version of a module; the link above includes an example. Also note that published versions cannot be reused or modified, and this includes retracted versions.
If you cannot retract your module (TODO: why?), you can file a request for the pkgsite team to hide your package documentation. Note that the package will continue to be available via go get or go install unless its module is retracted.

Note the TODO per the point 1 above.
Also note that I explicitly call out that retracted versions cannot be used again. I think this is a common misunderstanding: inexperienced Go users think that they can remove or retract a Go module or version, to then "start fresh" as if that package or version was never published. That is usually not true. Perhaps this should also be called out in https://go.dev/ref/mod#go-mod-file-retract.
cc @bcmills @seankhliao @julieqiu

We've seen the case where the repository is already gone so there is no place to publish version for retraction. (e.g. github id change, domain name change, ...) Other than that I don't see the case.
Good idea. Please send a PR.
In the second paragraph, I'd suggest explicitly call out the fact that removal from pkgsite doesn't remove the versions or modules from the module proxy.

Note that it is possible to retract the latest version of a module; the link above includes an example. Also note that published versions cannot be reused or modified, and this includes retracted versions.

Perhaps there should be separate section headings for "Removing a package" and "Removing a version of a package", with the latter giving the exact steps to take in the common case of recovering from an accidentally pushed version tag.

We've seen the case where the repository is already gone so there is no place to publish version for retraction.

I see, thanks. I'll amend the text.

In the second paragraph

Did you mean the third paragraph, where we talk about go get continuing to work?

Perhaps there should be separate section headings for "Removing a package" and "Removing a version of a package"

Personally, I wouldn't do that split. The difference between what a user sees on pkg.go.dev and what they get via go get is already confusing enough. I think we should just aim to solve the common problem that keeps coming up again and again: I published a module or version by mistake and I want to take that back.
If anything, we could call the two sections "removing a package's documentation" and "removing versions of a package", but I think the vast majority of people will want to do more than just hiding the documentation pages.

Maybe it's worth mentioning that you should immediately delete the tag from the remote if you realize you pushed the wrong tag, and realize that very soon after. If you catch it before anyone downloads it, you can actually delete it.

At some point I wrote a blog post trying to enumerate the common cases. I've since learned about shorter starting pseudoversions and that retractions work even without matching module names (??).
I get the feeling that the only removal requests that need manual action should be when repos have been removed.


Prometheus review

Tested and reviewed Prometheus guide.
Cosmetic changes for consistency with existing guides.

Nice work @pdaigle. Can you squash these commits into one?
It will be something like
git rebase -i HEAD~4 in your case here.
Here's a good article on how to rebase

Will re-open a separate PR.


Game: Update GameUserStateManager

As mentioned in the PR #1437 , the retrieval of the user's achievement state requires changes to meet the new goal system.
https://github.com/source-academy/cadet-frontend/blob/0204b1fa9936fc59b8cd6584bfa1bbd4b2b4a449/src/features/game/state/GameUserStateManager.ts#L91-L97

Will update this as soon as the backend integration is ready

#1436 centralised the stubbing to Game.tsx. Please replace the following lines instead.
https://github.com/source-academy/cadet-frontend/blob/master/src/pages/academy/game/Game.tsx#L15-L19

Any update on this?

Still blocked by backend integration

Resolved by #1986


Cannot use custom report output files as inputs to other tasks

Expected Behavior
Config like the following should work:
val reportMerge by tasks.registering(io.gitlab.arturbosch.detekt.report.ReportMergeTask::class) {
    output.set(project.layout.buildDirectory.file("reports/detekt/merge.xml"))
}

subprojects {
        tasks.withType(io.gitlab.arturbosch.detekt.Detekt::class.java).configureEach {
            reports {
                custom {
                    reportId = "csv"
                    outputLocation.set(layout.buildDirectory.map { it.file(name + "report.csv") })
                }
            }
        }
        reportMerge.configure {
            val reports = tasks.withType(io.gitlab.arturbosch.detekt.Detekt::class.java).map { it.reports.custom.first { it.reportId == "csv"}.outputLocation }
            input.from(reports)
        }
}

Observed Behavior
Property 'outputLocation' is declared as an output property of CustomDetektReport(reportId=csv, outputLocation=property 'outputLocation') (type CustomDetektReport) but does not have a task associated with it.

Steps to Reproduce
Failing test:
    @Test
    @Suppress("LongMethod")
    fun `custom report output locations can be used by other tasks without declaring explicit task dependencies`() {
        val builder = DslTestBuilder.kotlin()
        val buildFileContent = """
            ${builder.gradlePlugins.reIndent()}

            allprojects {
                ${builder.gradleRepositories.reIndent(1)}
            }

            val reportMerge by tasks.registering(io.gitlab.arturbosch.detekt.report.ReportMergeTask::class) {
                output.set(project.layout.buildDirectory.file("reports/detekt/merge.xml"))
            }

            subprojects {
                ${builder.gradleSubprojectsApplyPlugins.reIndent(1)}

                tasks.withType<io.gitlab.arturbosch.detekt.Detekt>().configureEach {
                    reports {
                        custom {
                            reportId = "csv"
                            outputLocation.set(layout.buildDirectory.file(name + "report.csv"))
                        }
                    }
                }

                reportMerge {
                    input.from(tasks.withType<io.gitlab.arturbosch.detekt.Detekt>().map { it.reports.custom.first { it.reportId == "csv"}.outputLocation })
                }
            }
        """.trimIndent()

        val projectLayout = ProjectLayout(numberOfSourceFilesInRootPerSourceDir = 0).apply {
            addSubmodule(
                name = "child1",
                numberOfSourceFilesPerSourceDir = 2,
                numberOfCodeSmells = 2
            )
            addSubmodule(
                name = "child2",
                numberOfSourceFilesPerSourceDir = 4,
                numberOfCodeSmells = 4
            )
        }

        val gradleRunner = DslGradleRunner(projectLayout, builder.gradleBuildName, buildFileContent)
        gradleRunner.setupProject()
        gradleRunner.runTasksAndExpectFailure("detekt", "reportMerge", "--continue") { result ->
            assertThat(result.output).contains("FAILURE: Build completed with 2 failures.")
            assertThat(result.output).containsIgnoringWhitespaces(
                """
                    Execution failed for task ':child1:detekt'.
                    > Analysis failed with 2 issues.
                """.trimIndent()
            )
            assertThat(result.output).containsIgnoringWhitespaces(
                """
                    Execution failed for task ':child2:detekt'.
                    > Analysis failed with 4 issues.
                """.trimIndent()
            )
            assertThat(projectFile("build/reports/detekt/detekt.xml")).doesNotExist()
            assertThat(projectFile("build/reports/detekt/merge.xml")).exists()
            assertThat(projectFile("build/reports/detekt/merge.xml").readText())
                .contains("<error column=\"31\" line=\"4\"")
            projectLayout.submodules.forEach {
                assertThat(projectFile("${it.name}/build/reports/detekt/detekt.xml")).exists()
            }
        }
    }

Context
Your Environment

Version of detekt used:
Version of Gradle used (if applicable):
Gradle scan link (add --scan option when running the gradle task):
Operating System and version:
Link to your project (if it's a public repository):

@3flex was this fixed by #6964 also ?

Unfortunately not - it was found and verified to be an existing issue while I worked on that PR https://github.com/detekt/detekt/pull/6964#pullrequestreview-1875817690

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.


Flutter Desktop support using a Webview.

Because we cant use ion on Flutter Desktop we hit on the idea of actually just opening ion in a new Windows that is a Desktop Webview.
The main FLutter app then opens a new window on the desktop just to play ion functionality.
There are 2 desktop webviews we are experimenting with.
https://github.com/zserge/webview/tree/webview-x

golang wrapping c.
this branch is for supporting Edge and modern webviews. Because Edge is based on Chromium now it should be easier.

The other one is a rust based one.

What if we make this a go-flutter WebView plugin for everyone ?
At the moment the go-flutter team only support Url Launcher which works but breaks the UX experience.

See: https://github.com/go-flutter-desktop/go-flutter/issues/324

SO i propose:

Make it a plugin, but then we have to problem of compositing.

Make it a plugin, but it opens a new GLFW Window and plays there to avoid the compositing issues.


[requires.io] dependency update on master branch

@eirsyl: Can you please take a loook at this? :)
ImportError: No module named 'runner'


lib,src: fixes for Windows and error output

On Windows the node executable needs to be explicitly invoked.
Make sure the data property exists before trying to use it when
printing error output.

This was also fixed back then in #178, shebang lines (of unix style) does not work on windows.

Merged, Thanks @kfarnung!


fix: set public access to enable npm public release

From https://semantic-release.gitbook.io/semantic-release/support/faq#how-can-i-set-the-access-level-of-the-published-npm-package

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


IndexError: list index out of range

Describe the bug
code:
!python train_speaker_embeddings.py hparams/train_x_vectors.yaml --data_folder= /content/speechbrain/recipes/VoxCeleb/SpeakerRec/your_data_folder/RIRS_NOISES/pointsource_noises --number_of_epochs=15 --device='cpu'

error:
Downloading https://www.dropbox.com/scl/fi/a5zg4588mefxzq2cleuux/RIRS_NOISES.zip?rlkey=01ymylca0an1nc7tlpdt2bjif&dl=1 to /content/speechbrain/recipes/VoxCeleb/SpeakerRec/your_data_folder/RIRS_NOISES/pointsource_noises/rirs_noises.zip
RIRS_NOISES.zip?rlkey=01ymylca0an1nc7tlpdt2bjif&dl=1: 816MB [00:12, 64.0MB/s]
Extracting /content/speechbrain/recipes/VoxCeleb/SpeakerRec/your_data_folder/RIRS_NOISES/pointsource_noises/rirs_noises.zip to /content/speechbrain/recipes/VoxCeleb/SpeakerRec/your_data_folder/RIRS_NOISES/pointsource_noises
/content/speechbrain/recipes/VoxCeleb/SpeakerRec/your_data_folder/RIRS_NOISES/pointsource_noises/rirs_noises.zip exists. Skipping download
/content/speechbrain/recipes/VoxCeleb/SpeakerRec/your_data_folder/RIRS_NOISES/pointsource_noises/rirs_noises.zip exists. Skipping download
results/xvect_augment/1986/save/veri_test2.txt exists. Skipping download
Getting file list...
0it [00:00, ?it/s]
0it [00:00, ?it/s]
Traceback (most recent call last):
  File "/content/speechbrain/recipes/VoxCeleb/SpeakerRec/train_speaker_embeddings.py", line 235, in <module>
    train_data, valid_data, label_encoder = dataio_prep(hparams)
  File "/content/speechbrain/recipes/VoxCeleb/SpeakerRec/train_speaker_embeddings.py", line 140, in dataio_prep
    train_data = sb.dataio.dataset.DynamicItemDataset.from_csv(
  File "/content/speechbrain/speechbrain/dataio/dataset.py", line 409, in from_csv
    return cls(data, dynamic_items, output_keys)
  File "/content/speechbrain/speechbrain/dataio/dataset.py", line 153, in __init__
    static_keys = list(self.data[self.data_ids[0]].keys())
IndexError: list index out of range

Expected behaviour
I'm trying to use pre-trained model via Colab. However, it did not work as expected.
To Reproduce
No response
Environment Details
No response
Relevant Log Output
No response
Additional Context
No response

can you provide the code to reproduce this error

It seems like it's failing to find any file in the data folder you specified. To my understanding, the VoxCeleb dataset should exist there, but it looks like it isn't, or is not placed correctly.
Did you follow the VoxCeleb SpeakerRec recipe README and follow the steps (specifically preparation)?

Oh, thank you for reminding. I also thought that was the issue but the dataset is no longer available. I'm using another model in CommonLanguage now.


Screen reader does not announce errors when reading line by line.

Issue Type: Performance Issue
For example writing code in Python:
print(var)
And I navigates line by line with up or down key arrows, the screen reader just reads the code.
But when I go to the toolbar buttons pressing F5 I found:
1 Warnings
Warning generated by Pylance  var   is not defined at line 1 and character 7.
Detecting an error navigating line by line or word by word is an essential need for productivity in programmers, the same needs the people who uses a screen reader have.
It would be great if screen reader can read lines and word attributes as underlined or can detect if a word is marked with some colour as occur with the IDE eclipse or as Microsoft word Spelling errors  works.
VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:28:07.755Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-2500 CPU @ 3.30GHz (4 x 3300)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: unavailable_offopengl: enabled_onrasterization: unavailable_offskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: unavailable_off

Load (avg)
undefined

Memory (System)
7.92GB (3.67GB free)

Process Argv
--crash-reporter-id a872c7cd-9a7a-4263-b751-d751b55c52f7

Screen Reader
yes

VM
67%

Process Info
CPU %	Mem MB	   PID	Process
    0	   105	  6252	code main
    0	    79	  1908	   shared-process
    0	    57	 13040	     ptyHost
    0	   108	  5092	   window (Getting Started - Visual Studio Code)
    0	    78	  3156	     extensionHost
    0	    84	  6560	   window (Issue Reporter)
    0	    22	  8728	   crashpad-handler
    0	    93	  9360	   gpu-process
    0	    38	  9400	   utility
    0	   162	 11212	   window (● probando.py - Visual Studio Code)
    0	    91	 13340	     extensionHost
    0	    67	 10412	       "C:\Users\Alfred\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\Alfred\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\node\jsonServerMain" --node-ipc --clientProcessId=13340
    0	    90	 13476	       electron_node bundle.js
    0	   133	 13284	   window (Python - Get Started - Visual Studio Code)
    0	    96	  4268	     extensionHost
    0	    62	  5464	       "C:\Users\Alfred\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\Alfred\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\node\jsonServerMain" --node-ipc --clientProcessId=4268
    0	     7	  9072	       python c:\Users\Alfred\.vscode\extensions\ms-python.python-2021.6.944021595\pythonFiles\lib\python\debugpy\adapter
    0	     5	 15004	         console-window-host (Windows internal process)
    0	    64	 14164	       electron_node bundle.js
    0	    74	 13976	   window (undefined)

Workspace Info
;

Extensions (3)

Extension
Author (truncated)
Version

python
ms-
2021.6.944021595

vscode-pylance
ms-
2021.6.3

jupyter
ms-
2021.6.999662501

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
pythonvspyt678:30270856
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551cf:30311713
vspre833:30321513
pythonptprofiler:30281270
vshan820:30294714
pythondataviewer:30285071
vscus158:30321503
pythonvsuse255cf:30323309
vscorehov:30309549
vscod805cf:30301675
pythonvspyt200:30331937
vscextlangct:30322307
binariesv615:30325510
vsccppwt:30329788
aatestflightcf:30331712
bridge0702:30333155

@aefren thanks for filling this and sorry for the slow response, I was on vacation.
This is the same problem as https://github.com/microsoft/vscode/issues/90842
Until we improve our editor decoration accessibility I suggest that you either use the status bar for the problem detection (F1 > focus status bar) or the Problems panel Ctrl + Shift + M.

Quick update: we are looking into doing quick announcements on the vscode side when a decoration is encountered (like a breakpoint). More details in this issue https://github.com/microsoft/vscode/issues/130565#issuecomment-900246306
Feedback welcome.
fyi @hediet


Update power-setting-guids.md

Fix GUIDs

@nnpcYvIVl : Thanks for your contribution! The author(s) have been notified to review your proposed change.

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

@mcleanbyron : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add Random Password Generator Snippet

Hello maintainers,
I'm excited to contribute a new code snippet to the "30-seconds/30-seconds-of-code" repository. This pull request adds a snippet for generating random passwords with customizable lengths and character sets.
Changes:
Added password-generator.md with the new snippet.
Included an explanation of how the function works.
Provided example usages and outputs.
This snippet can be a valuable addition to the repository, offering a secure way to generate strong passwords. I've tested the snippet locally to ensure its functionality.
I'm looking forward to hearing your feedback and any suggestions for improvement.
Best regards,
Johar Khan

Thank you for your contribution. However, as mentioned in numerous issues and PRs in the past, security is besides the scope of this repository and I don't want to attempt putting out any content that might end up causing security breaches or problems for people. I believe it's better to leave security matters, no matter how trivial, to the experts and avoid giving advice or providing code for such things. Cheers! 🍻


Crashes after update UI

I liked your library because ti's too simple. But it runs on the main thread and you can not update the UI after get the response.

Not a bug. It runs on the thread where you start it.
Try to use AsyncTask or IntentService or your own Thread management.
Or you can use a higher-level library like Volley.

I used AsyncTask. You library is easier than Volley

Thanks for contributing. I'm going to add documentation so everybody can see how to avoid doing I/O on Main-Thread.


version bump for Hackage

I pulled bef35c8 and pushed it to Hackage, so I had to bump the version.

Cool, thanks.


[Snyk] Upgrade io.grpc:grpc-netty-shaded from 1.59.1 to 1.60.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade io.grpc:grpc-netty-shaded from 1.59.1 to 1.60.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 23 days ago, on 2023-12-05.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Closing this PR bc it's already outdated. Will prepare a separate with all of the dependencies update.


Preserve tags fields simple selector

Fixes #371

This pull request and #422 are failing because of:
Downloading vet tool...
[DEBUG] go get golang.org/x/tools/cmd/vet

Executed command failed!
-- Command run was: go get golang.org/x/tools/cmd/vet
-- Failure was: package golang.org/x/tools/cmd/vet: cannot find package "golang.org/x/tools/cmd/vet" in any of:
	/usr/local/go/src/golang.org/x/tools/cmd/vet (from $GOROOT)
	/root/go/src/golang.org/x/tools/cmd/vet (from $GOPATH)

If I am not mistaken the "go tool vet" is baked in go and there is no need to "go get" it.

@yosiat Other than those two small issues this looks good to go. Thanks!

@nathanielc , I added a simple condition to fetch from the tags from the right place (aux/emitter's tags).
But I found out that Aux is empty for this test: TestStream_InfluxQL the Percentile test case specifically.
I see that in floatPointBulkAggregator.AggregateBatch, I do put the tags in the aux fields, but on floatPointEmitter.EmitPoint the Aux is empty.
Do you know why?

@yosiat  I took a look at the percentile case. Looks like a bug on the InfluxDB side https://github.com/influxdata/influxdb/blob/master/influxql/call_iterator.go#L991
The returned point does not preserve Aux fields or Tags. I'll ping them and see what they say. It should be an easy fix on their end of things.

This is the PR for fixing percentile in InfluxDB https://github.com/influxdata/influxdb/pull/6293

@nathanielc I tried to run the tests after doing "git pull" in $GOPATH/github.com/influxdata/influxdb but I am getting those errors:
pipeline/influxql.go:89: undefined: "github.com/influxdata/influxdb/influxql".FloatDistinctReduceSlice
pipeline/influxql.go:93: undefined: "github.com/influxdata/influxdb/influxql".IntegerDistinctReduceSlice

I did something wrong, or should I fix those errors?

I did something wrong, or should I fix those errors?

Nope, InfluxDB core changed again and broke things. I am working on setting up vendoring so that things like this don't keep breaking Kapacitor builds.

@yosiat Vendoring complete, a rebase should fix the build.

@nathanielc for some reason I fail to run the tests:
λ ~/code/go/src/github.com/influxdata/kapacitor/ preserve-tags-fields-simple-selector go test -v -run='TestStream_InfluxQL' ./integrations/streamer_test.go ./integrations/helpers_test.go
# command-line-arguments
integrations/streamer_test.go:20:2: cannot find package "command-l/vendor/github.com/influxdata/influxdb/client" in any of:
	/Users/yosi/.gvm/gos/go1.6/src/command-l/vendor/github.com/influxdata/influxdb/client (from $GOROOT)
	/Users/yosi/code/go/src/command-l/vendor/github.com/influxdata/influxdb/client (from $GOPATH)
FAIL	command-line-arguments [setup failed]

do you have any idea why?

@yosiat Not sure, what is command-l?

@nathanielc I have no idea as well.. I will try to check why TopSelectorTests are not passing and will fix the TestStream_InfluxQL tests to contain the tags in their expectations tomorrow.

@nathanielc Does those are simple selectors as well?

sum
count
distinct
mean
median

@nathanielc are we good to go? If so I will squash

@yosiat Just those minor changes and a squash and this should be good to go. Thanks!

@yosiat  Also can you add a CHANGELOG entry? And mark it as BREAKING? Since is possible that someone was relying on the previous incorrect behavior.

@nathanielc Done, can you make sure the changelog/indentation formatting is ok?

@nathanielc the tests are passing and the formatting is ok :+1:


RHIROS-1048 add a check to process only aws archives

Documentation update? :memo:

[ ] Yes
[X] No

Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[X] Bugfix
[ ] New Feature
[ ] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added
[ ] DB Migration Added

Additional :mega:
Feel free to add any other relevant details such as links, notes, screenshots, here.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.04 :tada:

Comparison is base (6651b74) 72.97% compared to head (b1eec28) 73.01%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #317      +/-   ##
==========================================
+ Coverage   72.97%   73.01%   +0.04%
==========================================
  Files          27       27
  Lines        1321     1323       +2
==========================================
+ Hits          964      966       +2
  Misses        357      357

Flag
Coverage Δ

unittests
73.01% <100.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

ros/lib/constants.py
100.00% <100.00%> (ø)

ros/lib/utils.py
88.27% <100.00%> (+0.07%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Thank you @r14chandra for this change. As it is already addressed in another PR, closing this PR on behalf of #316


nm-applet and blueman applet not showing icons with Statusnotifier

For some reason there is no icons displayed for blueman applet or statusnotifier under qtile wayland
OS: Arch Linux
Qtile extras git AUR ( r252.395320f-1 )

 $  qtile --version
0.22.2.dev113+g1102157a

(qtile-wayland git from aur)
In the logs there is only this line
libqtile statusnotifier.py:start():L160 Cannot find icon in current theme and no icon provided by StatusNotifierItem

I already have a bunch of icons installed and tried setting it to the icon
themes (as under /usr/share/icons)
On Thu, 12 Jan, 2023, 4:22 pm elParaguayo, @.***> wrote:

Do you have any icon themes installed and have you set icon_theme when
configuring the widget?
Also, this is really one for the main qtile repo as qtile-extras just adds
the context menus to the widget.
—
Reply to this email directly, view it on GitHub
https://github.com/elParaguayo/qtile-extras/issues/229#issuecomment-1380146591,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH22JW4UMYKYCSUFAMXD5Y3WR7O5PANCNFSM6AAAAAATZCYRE4
.
You are receiving this because you authored the thread.Message ID:
@.***>

one small question, how do I check which icon themes are actually accessible by qtile?
( I think that could be the issue where StatusNotifier is somehow not able to find the icon themes)

Can you run the following commands in a python terminal session please and paste the output here:
>>> from xdg.IconTheme import getIconPath
>>> getIconPath("blueman-tray", theme="Papirus-Dark")

(also yes I need the context menu so that eventually I can use the drop down for nm-applet, blueman, discord etc

qtile-extras adds the code for the context menu but does nothing with the icons. All the icon code is inherited from the main qtile version of the repo. So, any issues with icons should be posted to the qtile repo. Any issues with the context menu are dealt with here.
However, no need to create a separate issue for now. I'll deal with this one here.

Can you run the following commands in a python terminal session please and paste the output here:
>>> from xdg.IconTheme import getIconPath
>>> getIconPath("blueman-tray", theme="Papirus-Dark")

(also yes I need the context menu so that eventually I can use the drop down for nm-applet, blueman, discord etc

qtile-extras adds the code for the context menu but does nothing with the icons. All the icon code is inherited from the main qtile version of the repo. So, any issues with icons should be posted to the qtile repo. Any issues with the context menu are dealt with here.
However, no need to create a separate issue for now. I'll deal with this one here.

>>> from xdg.IconTheme import getIconPath
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'xdg.IconTheme'

WTH :eyes:
I seem to have python-pyxdg 0.28-1 installed...Interesting
Seems like newer versions might have removed this? or I am missing some other dependency?

I think I just realised my mistake, there was python xdg installed on my home path using pip, while at the same time  I also had python-pyxdg installed systemwide
Removed the one from the user side and now the issue seems to have been resolved (can now load blueman applet and nm applet without issue
Btw, is this menu without  icons normal?

That's correct for now. The only icon support in menus at the moment is when the menu provides icon data rather than an icon name.
However, that's only because I built the code using an app that provided icons that way. I didn't realise icon names could also be provided.
I should be able to add that relatively easily. Thanks for pointing it out!

That's correct for now. The only icon support in menus at the moment is when the menu provides icon data rather than an icon name.
However, that's only because I built the code using an app that provided icons that way. I didn't realise icon names could also be provided.
I should be able to add that relatively easily. Thanks for pointing it out!

Ah ok. So should I keep the issue open for fixing the menu icons?
Or maybe open a new issue for it?

We can keep this one open and I'll close it once I've got the change working.

Have submitted a PR with the relevant changes. Funnily enough, all the code was already there, I just left out the part that told the menus to draw icons when it received an icon name. As a result, this was pretty quick to do.

Ok. That fix is merged so, if you download the latest git version, you should, hopefully, get icons in your menus (provided your icon theme provides the relevant icons).

We now have another issue: the popup menus are cut:

I wouldn't call that an issue as the submenus work, they're just not displaying as you expected them to.
The way it's shown is how I've implemented it as I wanted basic functionality to display menus. It should be possible to have them display as you show but it's just not a priority for me at the moment.

👍


He3 bar on miners

Similar to the Shield and Hull statusbars that appear above ships, a 3rd bar should be added that displays the status of the Helium cargohold within the miner.
The code for this bar is currently commented out because no messages exist to pass the new He3 amount between the server/client as a miner harvests He3.
New messages would need to be added, or existing messages would need to be altered in order to ensure the client is up to date with the server regarding He3 quantities within the Miner's cargo hold.
Commented code is at radarimage.cpp line 963
Imported from trac ticket #156,  created by Xynth on 07-19-2010 at 19:05, last modified: 07-23-2010 at 01:43

I would ask that you consider;  When you reveal the miners payload, that this information be only available to that miners team.  You remove the decision making process from the attacker as to which target to select, if you reveal the miners closed cargohold, to the enemy.  Miner hunting is a very deep aspect of this game, I'm afraid this could take away some of that depth/art.
Trac comment by Big_Beta_Tester on 07-19-2010 at 21:29

Don't debate in trac.
http://www.freeallegiance.org/forums/index.php?showtopic=58176
Trac comment by Xynth on 07-19-2010 at 22:07

Got it, thanks KGJV for pointing me in the right direction.
r788
Trac comment by Xynth on 07-23-2010 at 01:43

The bugs in the latest patch are that the client isn't updating the ore in the miner to 0 when it offloads so it launches with a full bar again.  It updates once it gets to the rock.  This can probably be fixed with a message, but seemingly could be fixed in the client code but my attempts failed at this.
The other problem is the bars aren't updating when you aren't watching the sector they are mining in so you send a miner to the next sector he comes back full but with an empty bar.  That one will probably be tougher.
Trac comment by Xynth on 07-21-2010 at 01:55


URW Bookman and Century Schoolbook are no longer valid

The fonts are not proper rendered by text2image anymore.

any prove /example/details to you statement?

Since pango 1.44.0 (Released 2009):

Use harfbuzz for shaping on all platforms
Stop using freetype for font loading; this
drops support for type1 and bitmap fonts

And the Century fonts are type1 fonts. The resulting tif file generated was a blank canvas with no glyph in it. Therefore 0byte .box file,  then no .lstmf file were generated.
-rw------- 1 1000 1000    0 Aug  1 15:03 eng.Century_Schoolbook_L_Bold.exp0.box
-rw------- 1 1000 1000 3184 Aug  1 15:03 eng.Century_Schoolbook_L_Bold.exp0.tif
-rw------- 1 1000 1000    0 Aug  1 15:03 eng.Century_Schoolbook_L_Bold_Italic.exp0.box
-rw------- 1 1000 1000 3184 Aug  1 15:03 eng.Century_Schoolbook_L_Bold_Italic.exp0.tif
-rw------- 1 1000 1000    0 Aug  1 15:03 eng.Century_Schoolbook_L_Italic.exp0.box
-rw------- 1 1000 1000 3184 Aug  1 15:03 eng.Century_Schoolbook_L_Italic.exp0.tif
-rw------- 1 1000 1000    0 Aug  1 15:03 eng.Century_Schoolbook_L_Medium.exp0.box
-rw------- 1 1000 1000 3184 Aug  1 15:03 eng.Century_Schoolbook_L_Medium.exp0.tif
-rw------- 1 1000 1000    0 Aug  1 15:03 eng.URW_Bookman_L_Bold.exp0.box
-rw------- 1 1000 1000 3184 Aug  1 15:03 eng.URW_Bookman_L_Bold.exp0.tif
-rw------- 1 1000 1000    0 Aug  1 15:03 eng.URW_Bookman_L_Bold_Italic.exp0.box
-rw------- 1 1000 1000 3184 Aug  1 15:03 eng.URW_Bookman_L_Bold_Italic.exp0.tif
-rw------- 1 1000 1000    0 Aug  1 15:03 eng.URW_Bookman_L_Italic.exp0.box
-rw------- 1 1000 1000 2390 Aug  1 15:03 eng.URW_Bookman_L_Italic.exp0.tif

This  can be verified with
text2image --fonts_dir /usr/share/fonts --list_available_fonts

Issue #2593 was caused by this.
Actually, there are
2111: URW Bookman Light
2112: URW Bookman Light Italic
2113: URW Bookman Semi-Bold
2114: URW Bookman Semi-Bold Italic

But not URW Bookman L Bold etc.
Googling return a post said that the
 14: C059
 15: C059 Bold
 16: C059 Bold Italic
 17: C059 Italic

were replacement for the Century Schoolbooks, though.
C059's were in the fonts-urw-base35 package.

https://packages.debian.org/buster/fonts-texgyre

BTW, the font list in language-specific.sh should use for training the legacy engine.
For LSTM see https://github.com/tesseract-ocr/langdata_lstm/blob/master/eng/okfonts.txt.

Why is this not being merged?

Instead of just removing the old ones, it might be reasonable to add the replacements, too.

I reject this PR.
Shree, you can open a new PR which replace the font names with their compatible font name from the TeX Gyre collection.


GUI: QObject::connect: Cannot connect (null)

My debug.log full of this erro logs:
2017-10-31 13:55:01 UpdateTip: new best=ff1d11550b1e17818d766c62794750ada3ad17fd0224b657e8775cda7c9735d4  height=1532  log2_work=57.758819  tx=2491  date=2017-10-31 13:54:56 progress=0.999966  cache=0.1MiB(119tx)
2017-10-31 13:55:01 ProcessNewBlock : ACCEPTED
2017-10-31 13:55:06 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:11 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:16 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:21 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:26 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:31 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:36 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:41 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:46 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:51 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:55:56 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:56:01 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:56:06 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:56:11 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:56:16 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:56:21 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)
2017-10-31 13:56:26 GUI: QObject::connect: Cannot connect (null)::stateChanged(QNetworkSession::State) to QNetworkReplyHttpImpl::_q_networkSessionStateChanged(QNetworkSession::State)

Please re-compile windows wallet with QT 5.6 or newer to avoid this bug.

Thanks , noted. Currently trying to adjust build system and start using 5.9. Problem is Typical debian distros are still at 5.5.1 so something developed on linux may eithr not wok, or look different in windows.

@BlockMechanic do you have build script for windows environment? Trying to build with MXE on ubuntu but failed.


handled non-exiting crumb

The app crashes when crumb does not exist. Looks like Yahoo Finance has changed the response a few days ago.

Hi!
Sorry to hear about crashes in production. I've just started autotests. If they wouldn't fail, I'll merge this

Looks like something has failed. Do you know why?

Hey! Got pretty busy due to exams and New Year hussle. I'm merging this with a follow up tests fixing commit.


Added Tower Of Hanoi

Can you please add some descriptions to other algos you contributed to. I really like your contributions thanks 👍

Will be done ASAP.

@sherxon - Description to all the algos have been added.


Cross Origin Isolation breaks ffmpeg.wasm

Describe the bug
Everything runs great on local chrome with shared array buffer flags set. When I deploy to my server, I get a 'Shared Array Buffer not support' error. To fix that I use Cross Origin Isolation, but then I get resource blocking (The resource at “https://unpkg.com/<EMAIL_ADDRESS>cannot be loaded ... for example).
To Reproduce
Steps to reproduce the behavior:

Copy the code at (https://unpkg.com/@ffmpeg/ffmpeg@0.10.0/dist/ffmpeg.min.js), save and reference it locally.
Deploy website to a server that has Cross Origin Isolation set.
Load web page.

Do I go through and download each module references in the obfuscated code? I just can't get to 'Hello World' on the server. I'm good once I get words on a screen, but I just feel like jumping off the building until I can get words on the screen. I'm in a "Google It" > post an issue > Stackoverflow > Google more .... death spiral : (
I'm pretty sure what I'm doing here is tripping over webpack or something like that. Any way to grab everything bundled in one min.js file?

Can you post the code you are using for context?

Are you sure Cross Origin Isolation has been setup correctly?
An easy way to check is to open the console and check the value of crossOriginIsolated

Seems to be working now with these three headers:
      key: 'Cross-Origin-Embedder-Policy',
      value: 'require-corp',

      key: 'Cross-Origin-Opener-Policy',
      value: 'same-origin',

      key: 'Cross-Origin-Resource-Policy',
      value: 'same-origin',


Figure out Haul Dashboard

We were chatting today that it would be great to do Ken Wheeler like webpack-dashboard but for React Native, showing you logs, network stuff, performance (frames per second, CPU, etc).
We will be investigating that and will try to come up with mockups. Ideas appreciated.
CC: @Kureev for your idea on network logging

Any interest collaborating on Reactotron? We’re ramping up on v2 shortly. Could even resurrect our original CLI if there was interest.

Hah, interesting. My idea of network logging was born after taking under the hood of reactotron. I think if we can merge some features of Reactotron into Haul Dashboard, that would be awesome.
Regarding the idea itself (Haul Dashboard):

Great, I was thinking about to build a UI for Haul/Metro as well 🙌
I'd rather focus on RN-specific things: managing packages (so we can pause packager, install/remove npm package, link/unlink it, unpause packager), provide network throttling options and some more (let me know if you're interested) 🤔
I'm all for cooperation here. Community will benefit from moving packager to the new level 🚀

Should we discuss it on Slack and document our conclusions here?

Yeah, I believe this is fantastic idea @Kureev. There's that #haul channel on our open source Slack. I believe we can sit down and discuss it at some point.
We are having weekly meetings on Wednesdays - 12pm -> how about you join next one on Slack and we discuss it there?

@skellock haven't played with it much yet so hard to tell. Happy to check it out!

Reactotron TL;DR

There's two parts.  client and server and they talk via web sockets passing json objects of a "known" protocol. [1]

The client comes as a library [2] that plugs into the RN or web app you're making.

The server also comes as a library that plugins into an Electron app [3]. This used to be a CLI (r.i.p.) but i "upgraded" it when i launched 1.0.

It has many great features [4] for RN dev.  From basic logging [5], to redux plugins, to tracking network requests, and async storage, etc.

enabling reactotron in your app is pretty easy [6]

The Pitch
Let's make a new shared lib that both haul and reactotron use to communicate from app-to-server.
If haul had the ability to inject express middleware, I suspect we could use that channel with websockets to host the reactotron-core-server functionality.
With that in place, you could tap into this (event emitter & a few functions?) to build your own baked-in haul-cli dashboard, but start out of the gate with a bunch of features.
Next?
I can put together a proof-of-concept together inside haul (i wouldn't have time until this weekend) if you kinda need to see this action first. brace yourself, it won't be pretty, but it'll be awesome!
Then, if you're in, we could firm up the reactotron client/server protocol [8].
My game plan (and @rmevans9 as well) for 2.0 is to focus on building a plugin system for the Electron app & to bring more RN things online.
Others Who Want To Help
/cc @rmevans9 - he's helping me built v2 and built many features in v1
/cc @GantMan - huge evangelist and also contributor
/cc @jamonholmgren - my employer who goes out of his way to let me work on OSS
Others Who Might Be Interested?

Ram from Microsoft has a lib that would be a good fit to use this...
https://github.com/nparashuram/maya-kai
the storybook crew?
any else who needs to communicate between their RN app and a "server/app".

Finally
💩  That wasn't TL;DR at all.

current socket.io, i have a branch where i've converted this to websockets.  ws on the server, and straight up websockets on the front.
yarn add -D reactotron-react-native or npm i -D reactototron-react-js -- we support both platforms
Enhanced copy & paste, dragging images, better keystroke functionality, a lot more.
Some links:
4a. https://infinite-red-staging.herokuapp.com/reactotron
4b. https://www.youtube.com/watch?v=tPBRfxswDjA
4c. brew cask install reactotron
4d.  https://twitter.com/reactotron <-- lots of pics
Reactotron.log({ a: 1, b: 2, wtf: this.props })
example:

// on startup
import Reactotron from 'reactotron-react-native'

Reactotron
  .configure() // can change things like ip address, port, etc.
  .use(plugin) // can use plugins like `redux`, `redux-saga` or make your own
  .useReactNative() // or use a nice preset to give you a lot
  .connect() // connect to the app (server)

// then elsewhere, you use it within your app to do things like:

Reactotron.clear() // clears the app's logs
Reactotron.log('hi') // prints strings or objects
Reactotron.display({ name: 'HAUL', value: this.props, preview: 'wtf' }) // prints nicer stuff

// ghetto benchmarks
const bench = Reactotron.benchmark('please go faster')
await slowThings()
bench.stop()

For example:
7a.  you: tap s to save a copy of the redux state tree to disk
7b.  us: click 📷 to save a copy of the redux state tree to disk
A few things would have to change from it's current implementation. A few examples:
8a. * during handshake I'd like the clients to identify capabilities (device/plugins/etc)
8b. i'd like to have the notion of "active" and "passive" clients for replaying certain types of messages (great for automation, remote control, and for remote pair programming)
8c. this wouldn't have to happen immediately, we could absolutely thin slice this to start.

Great proposal, @skellock!
I only have one concern regarding coupling Haul with Reactotron. I think it would be great if we can conform to Reactotron protocol so we can use its plugins etc, but depending on the package itself would be an overkill for me (I'd like to hear your opinion, @grabbou).

Yeah, I totally agree with you @Kureev. I still haven't had enough time to dive into Reactotron much yet, but I quite like it. However, I think that using its API and ecosystem but with a different frontend (Haul Dashboard via CLI) would make a lot of sense and resolve quite a few problems when it comes to writing such integration.

@Kureev, are you still interested in doing it? I saw you writing this idea at Metro Bundler discord. I just wanted to express that I am extremely interested in helping you with this (maybe by providing some UX, however I know you are awesome at this as well)

Yeah, I am. Just need to see someone really aims for building it. I don't have enough resources to handle it on my own. Will be nice to plan something serious there.
In my mind, UI should be pluggable. That said, I think we need to:

Create a spec/contract that packagers should comply with in order to be able to use it (UI)
Create a PoC UI and bridge create adapter for Haul so it complies to the standard and therefore can be used with the UI

If you wonder which functionality for the PoC I have in mind:

Start/Stop/Restart packager
Bundle information

I'm very open for collaboration if by any chance you have similar vision.

Any updates/thoughts so far?

Thanks for the write-up. In general, I would love to see this sort of functionality in the wild, compatible with Haul.
Unfortunately, we (Haul team at Callstack) are busy as well in the upcoming quarter and it's unlikely we will pick this up ourselves. However, if there's any interest from outside collaborators to help to land this, I would be more than happy.

@grabbou I can pick it up concept-wise, but for the UI solution I would love to pair up with someone who has UI design skills. Do you know someone who may be interested?

Unfortunately, I don't. Let's tag this issue as help wanted to see if there's any interest in the future.

cc @zamotany who's been working on something similar (CLI) recently.


Uptime kuma: init package and module

Description of changes
This adds the uptime-kuma package and module. Uptime-kuma is a fancy self-hosted monitoring tool.
This PR also contain a nixos test for the uptime-kuma module, testing basic functionality (getting http response).
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1174

@RaitoBezarius @SuperSandro2000 thanks for your reviews! I have implemented your suggestions. I have also bumped the package to version 1.18.0 (released yesterday).

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/617

How about we use a vendorHash isntead of 29k lines of lock file?

@SuperSandro2000 I'm not sure to know what you're talking about. I've seen references about vendorHash only in the context of go modules.

@SuperSandro2000 I've made the changes you've suggested, I think we can merge if everything look fine to you

Could you squash into 2 commits? Otherwise the large files are still part of the history, which I'd like to prevent

Could you squash into 2 commits? Otherwise the large files are still part of the history, which I'd like to prevent

@Mindavi I'll squash, I'm not completely done taking into account the remarks from @06kellyjac
Do you see any solution to prevent committing the big files ? I guess it's the problem with nodejs projects...

Oops, I totally misread and thought those files were not needed anymore :). Then it doesn't matter too much. Just make sure to adhere to guidelines regarding commit messages then, then it's all good. Nothing to do about the large files I guess

Oops, I totally misread and thought those files were not needed anymore :). Then it doesn't matter too much. Just make sure to adhere to guidelines regarding commit messages then, then it's all good. Nothing to do about the large files I guess

No problem, I'll finish taking into account the last comments that I got and then squash the commits :)

I think I addressed all the remarks. @Mindavi @06kellyjac if you want to have another look !


Add sort by activation time for extensions

Hello, I thought it would be nice to be able to filter by activation time for extensions, for example, so you could more easily tell what extensions are the slowest.
It could be added to the already existing menu!

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Deregister engine by itself by onJobFailed

1. Describe the feature
Introduce exception handling through JobFailed events to delete the engine by itself from the engine space.
2. Motivation
This is useful for USER mode engines when some particular error occurred, such as token expiry, hardware failures, jar conflicts, which might lead to late coming queries doomed to fail. In this case, the engine should offline itself, and the new coming queries should be able to start a new engine to run and the old running queries also might be able to succeed via client retry.
3. Describe the solution
kyuubi.engine.exception.class.for.deregister - a seq for particular exception classes to notify engine to deregister
kyuubi.engine.exception.message.for.deregister -  a seq for particular exception messages to notify engine to deregister
kyuubi.engine.exception.stacktrace.for.deregister -  a seq for particular exception stacktraces to notify engine to deregister
Any of them matched the engine will deregister itself from the engine space, for example, kyuubi.engine.exception.stacktrace.for.deregister contains org.apache.spark.executor.Executor.$anonfun$updateDependencies
4. Additional context

WDYT, @pan3793 @turboFei @ulysses-you?

Good feature, it is good for a long running service.

Quite useful feature. And for the configuration keys, how about the following ones?

kyuubi.engine.deregister.exception.classes
kyuubi.engine.deregister.exception.messages
kyuubi.engine.deregister.exception.stacktraces

The same prefix indicates that affects the same stuffs, and end with s to match seq

I guess we can also add another conf like spark does for stages and task e.g. spark.task.maxFailures, kyuubi.engine.job.max.failures to deregister

Quite useful feature. And for the configuration keys, how about the following ones?

kyuubi.engine.deregister.exception.classes
kyuubi.engine.deregister.exception.messages
kyuubi.engine.deregister.exception.stacktraces

The same prefix indicates that affects the same stuffs, and end with s to match seq

make sense

I guess we can also add another conf like spark does for stages and task e.g. spark.task.maxFailures, kyuubi.engine.job.max.failures to deregister

In further, is there a TTL for a failure.
For example, if an exception has been disappeared for 30 minutes, we think it has been recovered.

I guess we can also add another conf like spark does for stages and task e.g. spark.task.maxFailures, kyuubi.engine.job.max.failures to deregister

In further, is there a TTL for a failure?
kyuubi.engine.deregister.exception.ttl?
For example, if an exception has been disappeared for 30 minutes, we think it has been recovered.

Make sense too


Fix ClusterStatsChangeTracker to handle case where current node is no…

…t found in previous state.
Cluster stats may have changed in this case.
@vekterli please review
@toregge FYI

Had to kick the CLA bot in the shins first, but now good to go.


Add connect metrics

First attempt to add Kafka Connect metrics
Prometheus:

Grafana:

Current dashboard:

Awesome work @vdesabou !


Don't cache children of Numaflow Controller Rollout twice

Currently, we are calling Watch() on Resources in NumaflowControllerRollout here but we also have these same resources being cached by the gitops-engine.
There is a call to clusterCache.OnResourceUpdated() in sync/cache.go. This uses a callback. We would need to make sure that we safely call the same reconciliation code that is being used by Reconcile(), such that only one NumaflowControllerRollout is being processed at a time. Thus, we would probably need to use a workqueue.RateLimitingInterface where both Reconcile() and this callback add to that queue (similar to what we're planning for PipelineRollout).

FYI @afugazzotto - there was an issue here before, but I updated it to better describe the issue


Replace internal isFunc function by @untemps/utils/function/isFunction, #70

Closes #70

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ArrowDataSource for Spark

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

unit test passed, will add ci support later


notifications and read callbacks Fixes #490

I'm not sure this is still backward compatible, or not, or what should it be backward compatible with.

@zkiiito thanks!


Got access violation when using -ras command line option

Brief description of your issue
I downloaded the latest code from https://github.com/winsiderss/systeminformer, and then using VS2022 attach the service process and debug, here is what I got

the error is happend in https://github.com/winsiderss/systeminformer/blob/master/SystemInformer/runas.c#L1973
according to the Microsoft doc: https://learn.microsoft.com/en-us/windows/win32/api/winsvc/nf-winsvc-startservicectrldispatchera
the parameter of StartServiceCtrlDispatcherA should be an array and terminated with NULL values
here is a piece of code from https://learn.microsoft.com/en-us/windows/win32/services/writing-a-service-program-s-main-function:
SERVICE_TABLE_ENTRY DispatchTable[] =
{
	{ SVCNAME, (LPSERVICE_MAIN_FUNCTION) SvcMain },
	{ NULL, NULL }
};

// This call returns when the service has stopped.
// The process should simply terminate when the call returns.

if (!StartServiceCtrlDispatcher( DispatchTable ))
{
	SvcReportEvent(TEXT("StartServiceCtrlDispatcher"));
}

and I modified runas.c like this:
SERVICE_TABLE_ENTRY DispatchTable[] =
{
	{ ServiceName->Buffer, (LPSERVICE_MAIN_FUNCTION)RunAsServiceMain },
	{ NULL, NULL }
};
StartServiceCtrlDispatcher(DispatchTable);

the issus was solved, and the service for systeminformer.exe running successfully
Steps to reproduce (optional)
No response
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
No response

Good catch, fixed 👍


chore(dependencies): upgrade @carbon/icons-react

Changelog
Changed

@carbon/icons-react dependency from alpha 26 to alpha 32.

@asudoh I think that yarn.lock and .yarn-offline-mirror changes after the proper yarn add<EMAIL_ADDRESS>should also be committed to solve the CI problem.

Updated offline folder 👍 also moved to es directory. We should be safe including it because es is transpiled outside of ESM. Down to move back to lib though if we feel that's safer 👍

Ah just kidding, will move back to lib since Jest will complain about es.

Thanks @joshblack for the updates!!

Withdrawing as it appears that a similar change has got into master - Don't hesitate to speak up if anybody thinks otherwise - Thanks!

sorry @asudoh I forgot about this open PR, part of it ended up overlapping with #1838 🙇 I needed the updated dependency to fix our experimental tooltips (#1883) since the tooltip icons were not forwarding refs (https://github.com/IBM/carbon-elements/pull/231)

No worries @emyarod!! Though I know people tend to be frustrated by this kind of things, I 💯 understand this kind of thing may happen and it sometimes is a good thing. We can communicate, but don't be hesitant to do the right thing as you did for tooltip!


push a release with updated bindgen?

Hi all,
The current version of mpi-sys v0.2.0 on crates.io has bindgen v0.60 set as a dependency.
Whereas latest master has bindgen v0.65.
Would it be possible to push a new release of mpi-sys to crates.io to pickup this change?
The bindgen update fixes an issue with Clang versions above v14
(see here for reference https://stackoverflow.com/questions/76443280/rust-bindgen-causes-a-is-not-a-valid-ident-error-on-build)
Thanks!

Thanks, this is coming in #168.


Tidy imports

This pr is an offspring of my work on policies. It simply make imports a bit tidier.

Does it mean that we should now put imports from crate and self after other imports? I thought that our convention was to put use crate-s first, but I actually prefer to put them last (more precisely, 1. external crates, 2. crate, 3. super, 4. self).

@honzasp my reference here would be burntsushi, and it goes like this:

std imports
external imports
crate imports (I don't distinguish between self, super and crate tbh, sorting kinda takes care of that itself)

for ref:

https://github.com/rust-lang/regex/blob/master/src/compile.rs
https://github.com/BurntSushi/ripgrep/blob/master/crates/cli/src/wtr.rs


default api timeout setting is not proper on startup

default is 1000 milli, better make it longer.
although users can configure it themselves, but a proper default is better.

because the shell not read the wrong config file, i think it the shell bug.
you can use the command 'cat' to check the kafka-manager shell,the config file is set belowdeclare -r script_conf_file="/etc/default/kafka-manager"

Default has been increased to 5 secs.  You can set it yourself in conf:
kafka-manager.api-timeout-millis=10000


bump deps

This will close #31 #48 #52 #54 and #55. I didn't include #56 because we need to figure out why the tests are failing first.


Broken the project

currently I have broken the project :-1:
will fix it today. sorry!

major issues have been fixed


fix(recorder): don't mess retries order any more

Motivation/Description of the PR

the retries order was turned around each time a promise was recorded
this was a problem because retryOpts are extracted from the last element in the array
have a look at the code and attach a debugger, while running a test with special retriy()-options

Applicable helpers:

[x] WebDriver
[x] Puppeteer
[x] Nightmare
[x] REST
[x] FileHelper
[x] Appium
[x] Protractor
[x] TestCafe
[x] Playwright

Applicable plugins:

[ ] allure
[ ] autoDelay
[ ] autoLogin
[ ] customLocator
[ ] pauseOnFail
[ ] puppeteerCoverage
[ ] retryFailedStep
[ ] screenshotOnFail
[ ] selenoid
[ ] stepByStepReport
[ ] wdio

Type of change

[ ] :fire: Breaking changes
[ ] :rocket: New functionality
[ ] :bug: Bug fix
[ ] :clipboard: Documentation changes/updates
[x] :hotsprings: Hot fix
[ ] :hammer: Markdown files fix - not related to source code
[ ] :nail_care: Polish code

Checklist:

[x] Tests have been added
[ ] Documentation has been added (Run npm run docs)
[ ] Lint checking (Run npm run lint)
[ ] Local tests are passed (Run npm test)

I don't know what's wrong with the CI.

I don't know what's wrong with the CI.

did you rebase with branch 3.x?

I don't know what's wrong with the CI.

did you rebase with branch 3.x?

yes. the commits of this branch are on top of the latest 3.x commit fc80c1b . you may check this at https://github.com/nitschSB/CodeceptJS/commits/3.x-fix-retry-order

this may have impact on actor.retry() implementation

I have added more tests and cleaned the actor.retry()-method. Please merge this soon.


Eliminate PublicKey from isAddressLookupTableAccount

We don't need an entire PublicKey instance just to assert on the address of the owner.

Current dependencies on/for this PR:

master

PR #303

PR #304   👈

This comment was auto-generated by Graphite.

Merge activity

Oct 17, 8:49 PM: @steveluscher started a stack merge that includes this pull request via Graphite.


How to change the name of usb devices?

I wanna change the external usb devices, which helps me to specify them by unique names.

It may be impossible to change the usb device name.1
[1]: https://stackoverflow.com/questions/31408879/usb-devices-where-do-their-names-come-from


Wider Code Blocks

I love this theme. My only concern is that the code block is far too narrow. So is the post content to be honest but it's more acute with the code blocks.
Here is an example:

I tracked it down to this .kg-canvas class in the kg.css file.
Changing the value of 640 within min(640px, calc(100% - 8vw)) works within the inspector live-view. However when I change the actual file in the theme folder and restart ghost, it reverts to 640 again. Apparently, something was cached and I saw the change once the cache was cleared.
I'm not very CSS/HTML savvy so I'm sure I am missing a basic web development concept here
Could you please point me to where I would need to go in order to change the width of code blocks and posts in general?
I am using the prism syntax highlighter with standard site header and site footer code injections.
This is the code block I am looking at and changing line 136 in particular
https://github.com/TryGhost/Dawn/blob/fdb1ee25303290f265969f020075f61172006c3d/assets/css/misc/kg.css#L128-L154

Hi @frakman1
You can try to add custom css for the <pre> tag:
grid-column: wide-start/wide-end;

You can try that by going to your admin panel: Settings - Code injection, and then add this to the Site Header field:
<style>
    pre {
        grid-column: wide-start/wide-end;
	}
</style>

You can even be more specific by using a better css selector
<style>
    .single-content > pre {
        grid-column: wide-start/wide-end;
	}
</style>

I am also not an expert on HTML/CSS, so let me know if this worked for you.

Hi @frakman1
You can try to add custom css for the <pre> tag:
grid-column: wide-start/wide-end;

You can try that by going to your admin panel: Settings - Code injection, and then add this to the Site Header field:
<style>
    pre {
        grid-column: wide-start/wide-end;
	}
</style>

You can even be more specific by using a better css selector
<style>
    .single-content > pre {
        grid-column: wide-start/wide-end;
	}
</style>

I am also not an expert on HTML/CSS, so let me know if this worked for you.

Thank you. That seems to work.
I don't understand where you see .single-content > pre I looked through the inspect fields but didn't see it.

Thank you. That seems to work.
I don't understand where you see .single-content > pre I looked through the inspect fields but didn't see it.

Yeah, @mvolpato's solution is the most straightforward way to make the code blocks or any other element inside the post content wider. Dawn has 3 grid columns, main (640px), wide (920px) and full (full width), and you can use them like:
grid-column: wide-start/wide-end;

/* Or */

grid-column: full-start/full-end;

Yeah, @mvolpato's solution is the most straightforward way to make the code blocks or any other element inside the post content wider. Dawn has 3 grid columns, main (640px), wide (920px) and full (full width), and you can use them like:
grid-column: wide-start/wide-end;

/* Or */

grid-column: full-start/full-end;

Yeah, @mvolpato's solution is the most straightforward way to make the code blocks or any other element inside the post content wider. Dawn has 3 grid columns, main (640px), wide (920px) and full (full width), and you can use them like:
grid-column: wide-start/wide-end;

/* Or */

grid-column: full-start/full-end;

I have added the CSS style in my site header injection but seems nothing has been changed since that.
By the way, is that possible to make the whole post area wider? for browsing pages on a large-size screen, the margins left and right waste too much space.


nftport

Why did nftport change their free (1000 nfts) and community (10-50k nfts) plan to only 100 free?

This is not the forum for that question.


Added note about EXO support in New/Suspend/Resume/Remove-MailboxImportRequest

https://github.com/MicrosoftDocs/office-docs-powershell/issues/5714
@chrisda please double check it. Thanks!

Docs Build status updates of commit 1917081:
:clock10: Full build: the average full build time is 18 min(s) 8 sec(s), this is based on the last 1000 build(s) for this repository.

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-MailboxImportRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-MailboxImportRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Resume-MailboxImportRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Suspend-MailboxImportRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

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


[MWPW-142625] PEP relative to App Switcher

This makes the necessary changes to:

position the PEP prompt relative to the App Switcher;
focus the App Switcher if the PEP prompt is dismissed;
cancel displaying the PEP prompt if the App Switcher is open.

This is safe to be merged in the feature branch, but we'll need to have a few things on our radar:

we need to get confirmation from the UNav team on the 1.1 version release that contains the updates to interact with the App Switcher, currently this is available just on production; we do have safety mechanisms is place in case their methods aren't available;
currently the Profile menu adobeProfile.getUserProfile() public method is throwing errors, thus we can't retrieve profile data, I've reported the issue on the appropriate Slack channel.

This PR also addresses a bunch of leftover TODOs from the code.
Resolves: MWPW-142625
Test URLs:

Before: https://main--cc--adobecom.hlx.page/drafts/ramuntea/firefly?milolibs=project-pep--milo--adobecom
After: https://main--cc--adobecom.hlx.page/drafts/ramuntea/firefly?milolibs=project-pep--milo--overmyheadandbody

@overmyheadandbody I see that some universal navigation unit tests are failing

@overmyheadandbody - I see some tests are failing

The unit tests will be addressed by @robert-bogos as part of the overall unit tests work for PEP. Since this is open against a feature branch, I will merge for now.

Unit tests are added through this PR.


MolFromMolFile error -> old mol file format?

$ python
>>>from rdkit import Chem
 >>>m = Chem.MolFromMolFile('50-21-5.mol')
error message is "[14:02:18] CTAB version string invalid at line 4"
where 50-21-5.mol is:
  NIST<PHONE_NUMBER>2D 1   1.00000     0.00000
NIST Standard Reference Data
  6  5  0     0  0              1 v2000
   32.0000   18.0000    0.0000 C   0  0  0  0  0  0           0  0  0
   11.0000   18.0000    0.0000 C   0  0  0  0  0  0           0  0  0
    0.0000    1.0000    0.0000 C   0  0  0  0  0  0           0  0  0
    1.0000   35.0000    0.0000 O   0  0  0  0  0  2           0  0  0
   42.0000   34.0000    0.0000 O   0  0  0  0  0  2           0  0  0
   41.0000    0.0000    0.0000 O   0  0  0  0  0  2           0  0  0
  1  2  1  0     0  0
  1  5  1  0     0  0
  1  6  2  0     0  0
  2  3  1  0     0  0
  2  4  1  0     0  0
M  END

Hi,
The RDKit is pretty strict about what it accepts as input. The specification for mol files says that the version tag should be either V2000 or V3000. Your file says "v2000". The quickest fix would be to correct the file(s), but I can add support for this type of thing as part of the "strictParsing=False" mode for MolFromMolFile().

Great, that works, thanks for taking to time to explain!


Some ideas about the contribution on docs-cn repo

We need a clear standard for the condition under which  a PR could get merged，say a certain number of guys have reviewed and approved.
Commit records might need to be squashed before PR get merged, as we do in k8s repo.
We may need to add a list to the CI-robot, which includes the guys who are active in the field of translation. Thus, a new PR could be assigned to one of the guys in the list for review automatically or manually.

Please feel free to give your opinions:)

agree +1
And, any certain message should be send to the guy who will merge this PR(like you, ^_^)?

say a certain number of guys have reviewed and approved.

CI-robot is a good idea

When exactly should we squash the commits? If after lgtm then this label will be removed after squash, causing another lgtm again.

When exactly should we squash the commits? If after lgtm then this label will be removed after squash, causing another lgtm again.

IMHO, it is not have to be a label, it would be okey if only the author and reviewer reach an agreement on the change, and the reviewer give a clear response that implies his approval of the change.
And when a certain number of guys have reviewed , @chentao1596 ,I think it's the author himself should be the one who ping admin for a merge :)

I think if labels like lgtm or others would help the admin to merge, otherwise that would be mountains of works waiting for the admin to update. If lgtm is labeled then the admin just has to merge without taking look into every pr. WDYT?

What I'm concerned about is if a contributor have the right to add the label in this repo.
If so, we can certainly make things this way, if not, it means we might need to give lots of guys read/write access to the repo, which is quite dangerous and may drive things out of control.
And no matter which way we take, an admin' task are the same: to receive a signal that implies the PR is ready, do some necessary check, and then a merge. so I think there is no essential difference between the two

we may create a project to track the process of first chinese localization version. and put related pr in it. =)

and if necessary some idea about this repo could be issued by Chinese or bilingual

@lichuqiang Oops, I mistakenly deleted a comment from Chuqiang - I meant to say that I agree - the article translator is like a "manager" who should take care of the life cycle of the piece. He/She should be the one who contacted the admin for a merge.


Mulesoft integration broken after JDK upgrade to 8_352

Question
Mule Integration is broken after upgrading to Adopt 8_352.
Currently, the application uses mulesoft as an integration framework used to connect to many endpoints via different transport types.
The mmc (mule core) internally connects to mmc-console (it just manages and monitors functions for all your on-premise Mule ESB Enterprise deployments and provides a green status if that endpoint is healthy.) and gets the status of the endpoint.
The connection b/w them was all fine till AdoptJDP 8_345 and less, but the moment the upgrade happened the communication broke with below exceptions:
com.xx.yy.server.manager.MMCException: HTTP response error for MMC request List Servers; response code=401 : reason=<!doctype html>HTTP Status 401 – UnauthorizedHTTP Status 401 – UnauthorizedType Status ReportMessage Bad credentialsDescription The request has not been applied because it lacks valid authentication credentials for the target resource.Apache Tomcat/7.0.109
And we also see few exceptions like :
2023-01-09 05:18:55.562 [INFO] org.apache.commons.httpclient.HttpMethodDirector:193 - Failure authenticating with BASIC 'Galaxy Realm'@localhost:8585
After going through the RN, we don't see any specific change which may have caused this, but the issue is not seen in Oracle 8_351(i understand the delta difference b/w these two branches.)
Any thoughts would be useful.
Context
Java version:
openjdk version "1.8.0_352"
OpenJDK Runtime Environment (Temurin)(build 1.8.0_352-b08)
OpenJDK 64-Bit Server VM (Temurin)(build 25.352-b08, mixed mode)
Your operating system and platform:
SUSE Linux Enterprise Server 12 (x86_64)
VERSION = 12
PATCHLEVEL = 5

The 8u352 update switched on TLS v3 for client roles. See: https://bugs.openjdk.org/browse/JDK-8245263


Contributing.md addition

Contribution Checklist
What is the purpose of the change
Write a contributing.md for Osmosis that explains the branching structure, how we use the SDK fork, best practices in contributing to the main chain repo, and how to make / test updates to SDK branches.
Brief change log
Added:
How to update dependencies for builds: https://github.com/osmosis-labs/osmosis/issues/456#issuecomment-908860526
How to change things in vendor for local builds / local testing
Branch structure of releases on v7, v6, v4, and using the mergify labels for backporting.
How to build proto files. (rm -rf vendor/ && make build-reproducible once docker is installed)
And general best practices for submitting contributions
Verifying this change
This change has been tested locally by rebuilding the doc website and verified content and links are expected

It looks like the dist folder got pushed, we might need to add it to gitignore

We need to remove these before merging this PR.

Will do, I didn’t see that, my bad!
On Fri, May 6, 2022 at 12:13 PM Daniel Farina @.***>
wrote:

It looks like the dist folder got pushed, we might need to add it to
gitignore
[image: image]
https://user-images.githubusercontent.com/13665117/167171619-d3d5e258-cf7a-469c-b7c7-6e546b8bb74b.png
We need to remove these before merging this PR.
—
Reply to this email directly, view it on GitHub
https://github.com/osmosis-labs/docs/pull/80#issuecomment-1119779996,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASH4FPTBM4NS36MTPXVQNC3VIVAMBANCNFSM5VFHD3TQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


[bug]: hugo server error

Steps that lead to the issue

I build a new site with hugo new site xxx
Then I follow the doc

cd ${PROJECT_DIRECTORY}
git submodule add https://github.com/jgazeau/shadocs.git themes/shadocs
git submodule update --init --recursive

Next I use themes\shadocs\exampleSite\config.toml as site config
Finally run hugo server error

Expected behavior
I don't konw what's wrong?
How to solve these problems?
Please help me. Thank you!
Reproducer

hugo.exe server
Start building sites …
hugo v0.111.3-5d4eb5154e1fed125ca8e9b5a0315c4180dab192+extended windows/amd64 BuildDate=2023-03-12T11:40:50Z VendorInfo=gohugoio
ERROR 2023/04/21 17:01:50 render of "taxonomy" failed: "E:\web\c\themes\shadocs\layouts_default\terms.html:1:4": execute of template failed: template: _default/terms.html:1:4: executing "_default/terms.html" at <partial "main.html" (dict "root" . "contentTemplate" "theme/terms-content.html")>: error calling partial: "E:\web\c\themes\shadocs\layouts\partials\main.html:5:8": execute of template failed: template: partials/main.html:5:8: executing "partials/main.html" at <partial "theme/includes-head.html" .root>: error calling partial: "E:\web\c\themes\shadocs\layouts\partials\theme\includes-head.html:68:36": execute of template failed: template: partials/theme/includes-head.html:68:36: executing "includeSass" at <append (resources.Get .)>: error calling append: reflect: call of reflect.Value.Type on zero Value
ERROR 2023/04/21 17:01:50 render of "home" failed: "E:\web\c\themes\shadocs\layouts_default\index.html:1:4": execute of template failed: template: _default/index.html:1:4: executing "_default/index.html" at <partial "main.html" (dict "root" . "contentTemplate" "theme/content.html")>: error calling partial: "E:\web\c\themes\shadocs\layouts\partials\main.html:5:8": execute of template failed: template: partials/main.html:5:8: executing "partials/main.html" at <partial "theme/includes-head.html" .root>: error calling partial: "E:\web\c\themes\shadocs\layouts\partials\theme\includes-head.html:68:36": execute of template failed: template: partials/theme/includes-head.html:68:36: executing "includeSass" at <append (resources.Get .)>: error calling append: reflect: call of reflect.Value.Type on zero Value
Error: Error building site: failed to render pages: render of "taxonomy" failed: "E:\web\c\themes\shadocs\layouts_default\terms.html:1:4": execute of template failed: template: _default/terms.html:1:4: executing "_default/terms.html" at <partial "main.html" (dict "root" . "contentTemplate" "theme/terms-content.html")>: error calling partial: "E:\web\c\themes\shadocs\layouts\partials\main.html:5:8": execute of template failed: template: partials/main.html:5:8: executing "partials/main.html" at <partial "theme/includes-head.html" .root>: error calling partial: "E:\web\c\themes\shadocs\layouts\partials\theme\includes-head.html:68:36": execute of template failed: template: partials/theme/includes-head.html:68:36: executing "includeSass" at <append (resources.Get .)>: error calling append: reflect: call of reflect.Value.Type on zero Value
Built in 28 ms

Solution?
No response

Hi @9201314 ,
Sorry for the delay,
Actually if you reuse the config.toml from the exampleSite you will have to delete the params.includes section to make it work. This section is used to declare the resource files to include during the build, which in your case doesn't exists.
After that it will work. But I also invite you to check the other parameters. They might not all be useful in your case.

Hi @9201314 ,
Sorry for the delay, Actually if you reuse the config.toml from the exampleSite you will have to delete the params.includes section to make it work. This section is used to declare the resource files to include during the build, which in your case doesn't exists. After that it will work. But I also invite you to check the other parameters. They might not all be useful in your case.

Thank you for your reply.It's very useful.


Private messages get mangled

Links in private messages show up strange: http://hwraid.le-vert.net turns into http://hwraid.le$6ert.net.
Lorem ipsum gets mangled too every other line, no matter which language you test (I tested original, english, kyrillic, arabic and chinese).
Are these errors connected to rot47?
Postgres db is globally set to utf-8.
We have no idea what bug/misconfiguration could cause this problem, so please help.

I can't duplicate. I've thrown in Lorem ipsum and foreign text of all kinds, the 'dann tested....' string mentioned above and it is all reproduced perfectly at both ends of the private message.

Then this is either a locale problem or another postgres issue - or both :smiling_imp:
Strange how the "u" is primarily affected.

But it's OK before it enters the database?
In any event, I've got a postgres test site now, but it's behind a firewall and I can't access it until Monday. I'll have a look then unless we come up with any brilliant ideas in the meantime.

I can enter the message correctly - after I hit submit, the page gets reloaded and the message displays mangled. When I send a message to myself from Michael's hub (mysql based) write and reload work ok but the messages arrives mangled at our (postgres based) hub.
Would it help to increase loglevel? It's set to 2 atm.

This needs more investigation, and we intentionally don't do a lot of logging of the contents of private mail. To verify that it either is or isn't a postgres issue you may need to insert a log statement to see what the text is just prior to storage - and compare that with what you see in the database.

Just tested on postgres and still cannot duplicate the issue.

We inserted log statements but couldn't find anything abnormal. And it seems that the problem fixed itself. Whatever happened - we don't know.
We usually test on Wednesday and Friday but not much in between and we always pull from git dev branch before.

FYI - I did test with and without the PDO merge.
I'm for leaving this open a while just in case it pops up again.