Add fn/end and do/end as brackets in language configuration

Fixes #83
Adds fn/end and do/end as brackets in language configuration. This enables bracket colorization, and enables other bracket-related features (e.g. "Go to Bracket").
Screenshot:

Hi @ollien. Thanks a lot for taking the time to open a PR!
I'll admit I'm a little on the fence about this. I'm not entierly opposed to this change, but I would instinctively expect keywords to always be the same color. I'm also slightly worried that fn, do and end might be considered as brackets in some unexpected scenarios, though I can't think of any off the top of my head.
Looking at the corresponding issue in vscode-elixir-ls it looks like Lukasz has run into and fixed some issues so I'll have to take a better look at how he did it.

Totally makes sense. One note, though

I'm not entierly opposed to this change, but I would instinctively expect keywords to always be the same color

FWIW, the colorization still has to be enabled within the VSCode settings. You could always disable this feature for your editor as a whole, or for elixir specifically. Having the option is nice, though, IMO.

Looking at the corresponding issue in vscode-elixir-ls it looks like Lukasz has run into and fixed some issues so I'll have to take a better look at how he did it.

Definitely agreed with this @Blond11516. Something to do with unbalancedBracketScopes in the syntax, it looks like.

You could always disable this feature for your editor as a whole, or for elixir specifically. Having the option is nice, though, IMO.

While I personally dislike colorized do/end, I definitely do want colorized brackets. What I think we may want to document is that users can override the colorized brackets in their settings if they wish. I haven't tested it with this PR specifically, but if I add this to my VS Code settings.json, it correctly stops colorizing everything except for parens:
  "[elixir]": {
    "editor.language.colorizedBracketPairs": [
      ["(", ")"]
    ]
  }

So, I presume that people could revert to the old colorizing behavior while still having bracket matching (which is nice), by using:
  "[elixir]": {
    "editor.language.colorizedBracketPairs": [
      ["(", ")"],
      ["[", "]"],
      ["{", "}"]
    ]
  }

That's all to say, if we can make sure that this doesn't run into the same issue that ElixirLS did initially, I'm for it, and perhaps we can document that colorized do/end and fn/end can be removed by overriding editor.language.colorizedBracketPairs in the README.

Thanks guys. I've used this configuration for some time and am indeed running into some edge cases. I'll take a look soon and what can be done

@zachallaun Thanks for the added context, I looked a bit for a way to disable specific bracket pairs but couldn't find it after a quick search!
Documenting this would probably be good enough, but ideally I'd like to do this through a proper configuration option, which I feel would be more idiomatic. With that said I doubt there's a way to configure colorized bracket pairs programatically, so docs might have to be it.
One last thing @ollien, if you could update the changelog that would be great! Given this is the first change since the last release, you can add an ## [Unreleased] section and add the change there.

From what I can quickly find there seems to be two ways to go about controlling this programatically:

Update the workspace configuration to change the editor.language.colorizedBracketPairs setting. This would mean binding two settings together and keeping them in sync (or handling the cases when they inevitably will not be) feels like a rat's nest I don't want to get into. This would also run the risk of overriding user-made custom changes to the config, which would be really bad.
Update the language configuration to remove fn/end and do/end from the list of brackets. This removes the coloring, but it also means we lose bracket pair highlighting and the ability to use features like "Select to bracket" which would be unfortunate.

Neither of these sound very appealing to me, so I think we're simply going to document it.
@ollien I'm thinking of adding a ## Configuration > ### Manual configuration section to the README, like so:
## Configuration

### lexical.server.releasePathOverride

...

### lexical.notifyOnServerAutoUpdate

...

### Manual configuration

Would you be willing to add that if it looks like a clear and easily discoverable place to you?

Sorry for the delay here, I've just found the chance to work on this. While my preliminary testing shows great improvement (thank you for linking that ElixirLS issue!), I'm going to run this for a couple of days to see if any other major issues come up.

Great! Seeing as this is basically the same configuration as Elixir LS I'm pretty confident it should work well.

Feel free to merge this when you're confident there aren't any obvious issues left.

(Also note that, if there is an issue after merging, it'll be okay 🙂)

Been using this since Monday and had no complaints. Merge away!


Improve PDFDocument.checkLastPage/Catalog.getAllPageDicts for documents with corrupt XRef tables (PR 14311, 14335 follow-up)

Improve PDFDocument.checkLastPage for documents with corrupt XRef tables (PR 14311, 14335 follow-up)
Rather than trying, and failing, to fetch the entire /Pages-tree for documents with corrupt XRef tables, let's fallback to indexing all objects before trying to invoke the Catalog.getAllPageDicts method.

Update Catalog.getAllPageDicts to always propagate the actual Errors (PR 14335 follow-up)
Rather than "swallowing" the actual Errors, when data fetching fails, ensure that they're always being propagated as intended to the call-site instead.
Note that we purposely handle XRefEntryException specially, to make it possible to fallback to indexing all XRef objects.

/botio test

Thank you for doing this!


Update Ryan K CAC card

Description
CAC card update!
Reviewer Notes
Is there anything you would like reviewers to give additional scrutiny?
Setup
Instructions are here

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 7809db99322bb9de66111ba674320ddd8095f5b7


Error when try to show previous answer from slide with multiple answers

Is there a way we can recall a specific answer from a previous slide where that slide had multiple answers?
Error showing for that use-case right now with [--prev|1.1--]
Fatal error: Uncaught Error: Call to a member function bindParam() on boolean in /var/www/html/app/model/slide.model.php:82 Stack trace: #0 /var/www/html/lib/functions.php(197): Slide->findPreviousAnswer('', '1', '1') #1 /var/www/html/app/view/project/slide.php(65): injectPrevAnswer('<h2>\nExplain ho...') #2 /var/www/html/lib/template.class.php(52): include('/var/www/html/a...') #3 /var/www/html/lib/controller.class.php(26): Template->render() #4 /var/www/html/lib/main.php(76): Controller->render() #5 /var/www/html/lib/main.php(112): init() #6 /var/www/html/public/index.php(11): require_once('/var/www/html/l...') #7 {main} thrown in /var/www/html/app/model/slide.model.php on line 82

Actually this seems to be broken on recalling an answer from a slide with only one answer as well
Fatal error: Uncaught Error: Call to a member function bindParam() on boolean in /var/www/html/app/model/slide.model.php:82 Stack trace: #0 /var/www/html/lib/functions.php(197): Slide->findPreviousAnswer('', '2', '3') #1 /var/www/html/app/view/project/slide.php(65): injectPrevAnswer('[--prev|2.3--]') #2 /var/www/html/lib/template.class.php(52): include('/var/www/html/a...') #3 /var/www/html/lib/controller.class.php(26): Template->render() #4 /var/www/html/lib/main.php(76): Controller->render() #5 /var/www/html/lib/main.php(112): init() #6 /var/www/html/public/index.php(11): require_once('/var/www/html/l...') #7 {main} thrown in /var/www/html/app/model/slide.model.php on line 82

This can also be caused by an expired session.  I've rewritten quite a large part of this code, so could you re-test?


Added more fields to SBAudio

I'm not sure if I'm doing this right...

For conversion between the JSON strings we receive and the native types we declare them as, transformers must be implemented. See editURLJSONTransformer for how to do a URL conversion for streamURL.

:cake:


Removes StoredAccountMeta::set_meta()

🚧 builds on #702 🚧
Problem
We are trying to move account storage files away from using mmaps internally. The function StoredAccountMeta::set_meta() updates the underlying mmap, which will not be possible once mmaps are removed. (It's also not supported for Tiered Storage at all.)
There are no callers of set_meta().
Summary of Changes
Removes StoredAccountMeta::set_meta()

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.8%. Comparing base (c0be86d) to head (36d05d8).
Report is 20 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master     #703     +/-   ##
=========================================
- Coverage    81.9%    81.8%   -0.1%
=========================================
  Files         851      851
  Lines      230475   230487     +12
=========================================
- Hits       188785   188768     -17
- Misses      41690    41719     +29


--completion flag not recognized in bash

The pipenv --completion command is not working:
$ pipenv --completion
Usage: pipenv [OPTIONS] COMMAND [ARGS]...
Try 'pipenv -h' for help.

There is NO WARRANTY, to the extent permitted by law.

@ttecles I am having trouble finding any reference to a --completion flag in the code base.  Can you reference me to the docs where it is specified?

@ttecles I am having trouble finding any reference to a --completion flag in the code base. Can you reference me to the docs where it is specified?

@matteius there you go: https://pipenv-fork.readthedocs.io/en/latest/advanced.html#shell-completion
I found a commit that removed that flag: https://github.com/pypa/pipenv/commit/267ba191a5cb0623bc99e60b44e99045a11e07cd

@ttecles I am having trouble finding any reference to a --completion flag in the code base. Can you reference me to the docs where it is specified?

@matteius there you go: https://pipenv-fork.readthedocs.io/en/latest/advanced.html#shell-completion
I found a commit that removed that flag: 267ba19

I see the docs in github do not have that anymore:
https://github.com/pypa/pipenv/blob/main/docs/advanced.rst#-shell-completion

@ttecles Thanks, and just noting that the correct page to access the pipenv docs is here:  https://pipenv.pypa.io/en/latest/

@ttecles Thanks, and just noting that the correct page to access the pipenv docs is here: https://pipenv.pypa.io/en/latest/

thanks @matteius

@ttecles you probably did the same as me - upgrade pipenv and notice your shell completion broke. It looks like this breaking change was released in 2021.11.9, and the changelog doesn't say anything about it directly. I had to create a version check in my .profile to determine how to install pipenv completion, like so:
    # Set up magic shell completion for pipenv (works in bash and zsh)
    # Have to check version bc pipenv changed the implementation
    if version_greater_equal "$(pipenv --version)" "2021.11.9" ; then
        eval "$(_PIPENV_COMPLETE=zsh_source pipenv)"
    else
        eval "$(pipenv --completion)"
    fi

It would have been nice if this sort of thing had been updated in the docs along with this sort of breaking change. I'm also now going to have to make it shell-specific in my .profile...

I added some tags to the ticket since it seems like there is a chance to improve something here.  I don't have the bandwidth to directly make changes at the moment, but I could advise and review PR if there were an improvement to be made.  My thoughts are we are trying to reduce the number of flags supported in some cases, where it makes sense.  I am speculating that the change away from --completion is that shell completion is supposed to work by default, so passing a flag to enable it seems like we want to move away from that.
TLDR:  Enhancement potential here if we can make it work nicer -- I didn't experience the issue directly, but also let's not add back --completion flag, let's try to make it work well out of the box.

I think this can be closed.  The last maintainer had dropped the --completion flag and we don't intend to add it back.  We also recently fixed a different bug around completion being broken.

I am speculating that the change away from --completion is that shell completion is supposed to work by default, so passing a flag to enable it seems like we want to move away from that.

Reducing the number of flags is a very admirable goal, but unless a completion script has been sourced by bash, I'm curious how this is supposed to work by default.
(note, also came here since my shell suddenly started shooting out this error—think a deprecation plan should have been considered in this case)

@dlo - The flag as removed before I started working on the project; we have no plans to add it back in to add a deprecation warning and then removing it again.

unless a completion script has been sourced by bash, I'm curious how this is supposed to work by default.

Either export

I just found this and for completeness, version_greater_equal is
referenced here: https://unix.stackexchange.com/a/567537

    # Set up magic shell completion for pipenv (works in bash and zsh)
    # Have to check version bc pipenv changed the implementation
    if version_greater_equal "$(pipenv --version)" "2021.11.9" ; then
        eval "$(_PIPENV_COMPLETE=zsh_source pipenv)"
    else
        eval "$(pipenv --completion)"
    fi

https://pipenv.pypa.io/en/stable/shell/#shell-completion

https://pipenv.pypa.io/en/stable/shell/#shell-completion

@Guang1234567 The link is broken.


module not found error

while running the inference of the project the following error is encountered in colab
Traceback (most recent call last):
File "fakeout/inference.py", line 36, in
from models import mm_embeddings
File "/content/drive/MyDrive/Media_Forensics/Fakeout/FakeOut/fakeout/models/mm_embeddings.py", line 25, in
from fakeout.models import normalization
ModuleNotFoundError: No module named 'fakeout'

where as the files also available in the folder

This is a scope issue. It can be solved in a few ways:

Add the following lines to inference.py:

import sys
sys.path.append('/content/drive/MyDrive/Media_Forensics/Fakeout/FakeOut')

Probably replacing from fakeout.models import normalization with from models import normalization should work too.

I'll be working on finding a solution (without any "scope patches") for this in an upcoming pull request.
Thank you for bringing this up, let me know if it helped you in the meantime.

Fixed in master branch source code.


Arcade

New language request ArcGIS Arcade is an expression language based on JavaScript

Is there anything I need to do to get this into the next release?

@marcoscaceres after almost 2 years it is nice to have someone finally review this request. Thank you!
I updated the test to include a markup test for Arcade.

Note to self: update CHANGES.md


Update to version 1.39.6

https://github.com/WikiTeq/Taqasta/pull/131 should be closed when this PR merged


added location+time end point

-usage

make secret.py
python3 crawler.py location -l washignton
added metros.py and dateToId.py

Feature implemented in https://github.com/asmitks/insta-graphql-scraper


Questions about encoding/decoding and network architecture

Hi,
First of all, thank you for this repository, and please excuse me if the following questions are very basic.
I have started studying the splitbrain network model and reading the paper, but I have not yet understood where the encoding part ends and the decoding part starts in deploy_lab.prototxt. It seems to me that the output of the network gives a feature representation, and not the recreated image I expected to find. Could you please point me in the right direction?
Also, it seems to me that if you wanted to replicate the architecture described in your paper, you would need to use two networks. Is it correct?
Thank you in advance.

Hi, I stripped away the last layer, and left the rest of the network as a feed-forward feature extractor.
Yes, when I trained it, I trained 2 separate networks. Then I put them together afterwards into a single network.

Hi,
Just to understand, if I have to implement this then following steps need to do

L_channel (Input) => Network 1 => ab_channel (output)
ab_channel (Input) => Network 2 => L_channel (output)

finally merge both networks along channels for each layers to test different Classification/Detection network comparisons. e.g. Classification network
3) RGB => (Network 1 + Network 2) => fc5 features => classifier
Please let me know If my understanding is correct or not?
if yes, then I would request you to please let me know how to merge each layer along channel. ( I do agree it is a library/caffe question but if you can help me to with this.)
Thanks

I have the same doubts. And I see the architechture for pre-training has only feature extractor. So during the trainning, is there decoder for recontructed images?

Hi, thanks for the question. The ground truth is heavily downsampled.


ollama support (Local LLMs)

Correct me if I'm wrong but it seems the cog doesn't support locally hosted LLM models. Most notable of which would be something like Ollama, or possibly something like h2ogpt.
It'd be really nice to avoid having to pay for the usage of this cog via hosting locally.

The aiuser cog does supports setting alternative OpenAI API servers. (as long as they are compatible with the more recent OpenAI clients packages)
h2ogpt seems to provide a OpenAI proxy so it might work out of the box, but Ollama doesn't and probably needs something like litellm.


Render Error - Planets

Beyond a certain distance (about 1100 "km" in the build as of 2022/03/02), planets (and, presumably, stars if we implement those) stop displaying. As the large but distant object is moved to the center of the view, first its edge (e.g., atmosphere) stops being drawn, then the circular exterior edge gets smaller and smaller until the object is no longer drawn at all. Changing zFar and zNear (in Camera.lua) have had no obvious effect so far.

I can no longer reproduce this.
When I saw it, it was definitely a graphical glitch. The planet did not just suddenly disappear. It would be modestly sized and I can't remember, but either turning away, or turning towards it would make it shrink to a singularity or expand to its correct size.
Maybe your Z changes have helped.
If you are able to reproduce it in LTheoryRedux (or camera/renderer etc), then please let me know what tweaks I need in order to see it again.

The size of the galaxy in the experimental build was changed back to 2e4, near Josh's original value of 1e4. This value is currently controlled in LTheoryRedux for Background Mode by the variables ./script/Config/App.lua:scaleSystemBack , and for Flight Mode by ./script/Config/App.lua:scaleSystemReal . Also, ./script/Config/App.lua:scalePlanetReal controls the size of planets. Making planets larger than 1e6 (to try to approximate real object sizes in our actual Solar system) also contributes to the problem of planets demonstrating the rendering problem.
The change back to a smaller star system was made in response to the concern you expressed that testers would be bothered by ships being "jiggly" in a larger-scale star system (see https://discord.com/channels/695088786702336000/695723947307827222/1083111699105730623). Ships being jiggly and planets that stop being rendered are both artifacts of having a star system in which objects can fit into an extent larger than about 5e4.
To reproduce the "planet disappearing" rendering problem -- which necessarily brings the "jiggly ships" problem back as well -- edit ./script/Config/App.lua and change the value for scaleSystemReal to 1e5 or 1e6 or anything larger.

I just spotted this. By eye, it seems to me that the planet is intersecting with the nebula / stars, which I'm guessing is being rendered using a skybox or skysphere?
I suspect the issue is that the skybox/sphere is writing to the zbuffer, which means that the skybox/sphere is actually rendering on top of the planet when it's sufficiently far away. That would explain why modifying znear/zfar had no effect.
If that's the case, the solution would be to disable zwrite when rendering the skybox/sphere.


Add Embark to the list of supported libraries

Added at https://docs.ens.domains/dapp-developer-guide/ens-libraries


Process database permissions in SSMS

This article indicates that the user must be in the Analysis Services Admins group (server admin) to connect to Azure Analysis Services via SSMS. Does that mean that a uses who is assigned permissions of "Process database" through an AS database role cannot connect to AS with SSMS and process a database?

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

ID: f0514bd5-d82f-702c-c975-caeacd7bdae6
Version Independent ID: de5bd2fe-0b80-2b2a-bc96-76337baebcf2
Content: Manage Azure Analysis Services
Content Source: articles/analysis-services/analysis-services-manage.md
Service: azure-analysis-services
GitHub Login: @Minewiskan
Microsoft Alias: owend

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

I have tested with a coworker.  I assigned my coworked a role with "Process database" permission, and no other permission.  My coworker was able to log in with SSMS, but was unable to see any databases or anything else of note.

Apologies if my question wasn't clear. If they have "Process database" permissions, wouldn't you expect them to see the database in SSMS and be able to process the database? Wondering if they need Azure IAM permissions.
Thanks
On Thursday, April 4, 2019 04:34:01 PM PDT, MartinJaffer-MSFT<EMAIL_ADDRESS>wrote:

I have tested with a coworker. I assigned my coworked a role with "Process database" permission, and no other permission. My coworker was able to log in with SSMS, but was unable to see any databases or anything else of note.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

What permissions does a uses need to see the database in SSMS and be able to process the database?

@chrisjrisch - A user account must have at least Read permissions to see server databases in Object explorer. If you want a user to be able to both see databases in Object explorer and process databases or tables from Object explorer (not scripted), give the account Read and Process.

That's not the behavior I'm experiencing. I can give members "Read", "Process database", and "Full Control" permissions in SSMS and they still cannot see the AAS database. They can only see the databases if I make them a server admin.


table row updated auto if we have datatable within angular form.. can u have a fix for this

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior

Expected behavior

Reproduction of the problem

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Table version: 0.8.x

Angular version: 2.0.x

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript X.X | ES6/7 | ES5]

Try to reproduce it in a stackblitz, also try upgrading your version.


Image changes size when loading single result page

Description

When viewing a single image result, the image thumbnail will load first, then the fullsize image will load. This frequently causes a smaller image to load momentarily, then flashes to the larger image.
Should the thumbnail not be loaded in this context? Should it be loaded at the same size as the original image so there's no size discrepancy, even if there is a quality discrepancy?
Reproduction

Visit https://search-staging.openverse.engineering/image/0439408c-11ae-490e-a10a-8139766f68fb
Refresh the page
Observe that the thumbnail shows up momentarily before the full image loads

Screenshots

https://user-images.githubusercontent.com/10214785/165646181-e8a89b82-36df-4368-b9d9-da731ed60a85.mp4
Environment

Device: Desktop
OS: Ubuntu
Browser: Firefox

Additional context

Resolution

[ ] 🙋 I would be interested in resolving this bug.

This is indeed caused by the thumbnail being used as a placeholder.
If we have the image dimensions in the API data then we can use it to stretch the thumbnail to size until the actual image loads.

This can be fixed pretty easily. One question I have before doing so requires help from @panchovm:
Currently, single result images have a max-height of 500px. This means that on my 24" 5k monitor, a single result never looks larger than this:
https://cln.sh/fQbQIZ
Do we want to increase this size as part of this PR?

It sounds like a nice improvement defining a relative height for different screen sizes. My only concern is showing a tall image on many devices where the title, author, and CTA are hidden below the visible area.
Do you have an idea for tackling this @zackkrida? I am very open to try it


Clear up files between queries to help avoid running out of disk space

We create a new working directory for each repository that we analyse, but we do not clear up these directories between queries. This can lead to running out of disk space if four queries use a lot of space. We can help avoid this and thus we'll only fail if a query uses all the space on its own.

Here's my suggestion for deleting the work directory between queries.
Unfortunately I haven't been able to reproduce the out-of-disk error. I tried creating a custom instructions.json file that would run the mp911de/logstash-gelf, RestExpress/RestExpress, poetix/protonpack, Blazebit/blaze-persistence repositories in one job, and these are the repos that caused the error in https://github.com/dsp-testing/qc-controller/actions/runs/2344503810. Unfortunately after trying it a few times it always completed with loads of free space. I added in a call to df and it showed 31GB of free space. This leads me to think there was something else going on which caused the disk space issues earlier, but I'm not sure what.
However, I think doing this change is likely still beneficial. Just from a logical perspective it makes sense to delete these files once we're done with them and prevent them from interfering with future queries.
I did some integration test runs (e.g. https://github.com/github/codeql-variant-analysis-action/actions/runs/2534864625) and you can see that the artifact uploads are still present, so deleting the files after uploading them is no problem.


TIM-227 Likes per Comment

#143  👈
main

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

Join @tamrivera05 and the rest of your teammates on  Graphite


Give Json Monster an online status, explaining what he does.

No idea how discord API handles this, but it should say something like "Auto formats your JSON"

#5
Fixed in my pr


consistency in logging images between multiple logging backends -- wandb and tensorboard

🐛 Bug
I am still new to PL so perhaps there is a better way to do this. But there seems to be an issue with consistency when logging
model performance to different backends. I have a simple sequence-to-sequence model and I wanted to log some initial checks on the training data, to make sure that the data and labels line up correctly. I was logging to both tensorboard and weights and biases (wandb), through a custom callback. However I found that I needed to manually change the image logging call, depending on which backend I was using. That creates really fragile code, since if I change the logging backend callback--or even the order of callback for logging, then the code will break.
Please reproduce using the BoringModel
I have my model and a simple trainer class:
trainer = Trainer(logger=[tb_logger, wandb_logger])

Then I created a custom callback to the initial batch of images to both loggers. But as you can see I have to refer to experiment[0] and experiment[1] to log correctly. I think the correct way would be to use just experiment.log() and this should be agnostic of which backend is used.
class TimeSeriesDataSanityCheckCallback(Callback):

    def on_train_batch_start(self, trainer, pl_module , batch, batch_idx, dataloader_idx) -> None:
        source, target = batch
        plots = []
        for i in range(len(source)):
            fig, (ax1, ax2) = plt.subplots(1, 2)
            fig.suptitle(f'Sensor data time-series {i}')
            ax1.plot(range(len(source[i, :, 0])), source[i, :, 0])
            ax2.plot(range(len(target[i, :, 0])), target[i, :, 0])
            im = plot_to_image(fig)
            plots.append(im)

        grid = torchvision.utils.make_grid(plots, nrow=1)
        pl_module.logger.experiment[0].add_image('generated', grid, 0)
        pl_module.logger.experiment[1].log({"val_input_image":[wandb.Image(grid.cpu(), caption="val_input_image")]})

I have that auxiliary function plot_to_image to simple take matplotlib plots and convert them to tensors. Here is that code if it helps.
class TimeSeriesDataSanityCheckCallback(Callback):

    def on_train_batch_start(self, trainer, pl_module , batch, batch_idx, dataloader_idx) -> None:
        source, target = batch
        plots = []
        for i in range(len(source)):
            fig, (ax1, ax2) = plt.subplots(1, 2)
            fig.suptitle(f'Sensor data time-series {i}')
            ax1.plot(range(len(source[i, :, 0])), source[i, :, 0])
            ax2.plot(range(len(target[i, :, 0])), target[i, :, 0])
            im = plot_to_image(fig)
            plots.append(im)

        grid = torchvision.utils.make_grid(plots, nrow=1)
        pl_module.logger.experiment[0].add_image('generated', grid, 0)
        pl_module.logger.experiment[1].log({"val_input_image":[wandb.Image(grid.cpu(), caption="val_input_image")]})

To Reproduce

Expected behavior
The expected behavior is to have a consistent logging interface and not need to specialize the call depending on which loggers are used. That would be a more robust method.
Environment
wget https://raw.githubusercontent.com/PyTorchLightning/pytorch-lightning/master/tests/collect_env_details.py
# For security purposes, please check the contents of collect_env_details.py before running it.
python collect_env_details.py

PyTorch Version (e.g., 1.0): 1.9
OS (e.g., Linux): ubuntu linux 20.04.2
How you installed PyTorch (conda, pip, source): conda
Build command you used (if compiling from source):
Python version: 3.9
CUDA/cuDNN version: cudatoolkit 11.1
GPU models and configuration: 1 gpu
Any other relevant information:

Additional context

Seems related to https://github.com/PyTorchLightning/pytorch-lightning/issues/3635

Hi @00krishna thanks for reporting this issue. It may also he related to #6720 and #7920 - please feel free to shar your thoughts on those as well.
Thanks :smiley:

Dear @00krishna,
Definitely a feature on our roadmap ! Would you be interested in making a PR following chat in  #6720 ?
Best,
T.C


Limit json version to 1.8.x for Ruby1.8.x and 1.9.x

Latest json gem no longer supports Ruby 1.8.x and 1.9.x .

Coverage decreased (-0.04%) to 96.129% when pulling b374f21db6f7542524f9afc74b070fc9a73bfd8f on ppworks:limit_json_version into fb502c755ed8801aa5d4575f18b86a9b7ecf9f5e on intridea:master.

:eyes:

Hey @sferik @Motoshi-Nishihira @paulwalker could someone take a look at this? This is fixing an issue that is causing all of the other PRs builds to fail.

Resolved by https://github.com/intridea/oauth2/pull/271.


[ISSUE#7629] typo about mQ -> mq and getmQ -> getmq

Which Issue(s) This PR Fixes

Fixes #7629
Brief Description

How Did You Test This Change?

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

Comparison is base (c2c29c2) 43.20% compared to head (650a512) 43.22%.
 Consider uploading reports for the commit c2ad253 to get more accurate results

Files
Patch %
Lines

...lient/impl/consumer/DefaultMQPushConsumerImpl.java
34.61%
33 Missing and 1 partial :warning:

...a/org/apache/rocketmq/client/impl/MQAdminImpl.java
10.00%
27 Missing :warning:

...mq/client/impl/producer/DefaultMQProducerImpl.java
57.50%
16 Missing and 1 partial :warning:

...lient/impl/consumer/DefaultMQPullConsumerImpl.java
31.81%
15 Missing :warning:

...e/rocketmq/client/impl/consumer/RebalanceImpl.java
46.15%
14 Missing :warning:

.../rocketmq/client/impl/consumer/PullAPIWrapper.java
46.15%
7 Missing :warning:

...rocketmq/client/impl/factory/MQClientInstance.java
72.22%
5 Missing :warning:

...apache/rocketmq/store/timer/TimerMessageStore.java
20.00%
4 Missing :warning:

...apache/rocketmq/broker/slave/SlaveSynchronize.java
0.00%
3 Missing :warning:

...client/consumer/store/RemoteBrokerOffsetStore.java
76.92%
3 Missing :warning:

... and 6 more

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #7630      +/-   ##
=============================================
+ Coverage      43.20%   43.22%   +0.01%
- Complexity      9796     9801       +5
=============================================
  Files           1164     1164
  Lines          84420    84428       +8
  Branches       10961    10963       +2
=============================================
+ Hits           36474    36490      +16
+ Misses         43420    43411       -9
- Partials        4526     4527       +1

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

看起来CI不怎么稳定


feat: characterizations of monotonicity in locally finite orders

if preferred I can split this into multiple PRs, just indicate how you would prefer it be split.

I think a good split would be the finset/covby lemmas, the relation lemmas (in parallel), and then the trivial consequences of the two together.
Having the analogous results for multiset would be great too.

@eric-wieser I split it, but I'm not sure the split was quite along the lines you suggested.

It seems to be shorter to use well-founded induction than to use strong induction.  I pushed a commit doing this, feel free to ditch it if you don't like it.

bors d+

@hrmacbeth thanks for the nice golf!
bors merge

In principle Fin.liftFun_iff_succ is a special case of this, but this PR doesn't include an IsTrans version (only PreOrder as far I see), and applying this PR may not lead to a shorter proof ...


Test consistency of hashes between versions

Currently there is no way to test if a change in the hashing code unintentionally changes the result. We should fix that.
One way to do this:

Hash some values now and include them in a test as constants
Test hashes for equality with those constants

Aha! Link: https://csiro.aha.io/features/ANONLINK-30

We now have a few regression tests for this.


Refactoring Gulp file

Hi,
nice boilerplate ;)
Maybe you should use gulp-load-plugins for having a cleaner gulp file.

Hi, thanks. That's true, to be honest I wanted to rewrite it but was lazy :) But maybe I will do it in the future.

Closing this one.


Unable to validate server certificates during MTLS truststore file in karate

Trying to test MTLS verification.The self signed server certificates needs to be validated and CA associated with these self signed certificates are available in the truststore.p12 file.
When making the call with this truststore.p12 file with ssl ={ trustAll: false}, expecting that the server side cert is validated and should pass the call as the correct truststore is given but its failing with this exception:
Invalid parameter algorithm exception

follow this process to reopen please: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue


📖 Update root README file

What this PR does / why we need it:
Updates root readme file with latest changes since release v1.1.0 is out and CAPI release v1.1.2 is being used as of now.
Also, updates a bit deployment process workflow

/test-v1b1-ubuntu-integration

Thanks for the reviews all! I have addressed all of them, take a look again.

/test-v1b1-ubuntu-integration


ENH: Add gradient encoding direction angle computation utils

Add gradient encoding direction angle computation utils: compute pairwise angles across gradient encoding directions.
Add the corresponding test.

Need to go through https://github.com/nipreps/eddymotion/pull/188#issuecomment-2118215168 and incorporate https://github.com/jhlegarreta/eddymotion/pull/1.

Went over @yibeichan's related https://github.com/nipreps/eddymotion/pull/188#issuecomment-2118215168. For now, assuming it is an ndarray, so no need for the checks.

@oesteban ready to be merged.

@esavary would be great if you could have a quick look. Thanks.

@esavary would be great if you could have a quick look. Thanks.

I made a few comments on your questions inside the code, but it looks ready to merge to me.


[5.10] Fix SILGen to emit fix_lifetime for inout-to-pointer conversion.

This fixes use-after-free miscompilation bugs that can occur when a lifetime-optimized standard library type, like Dictionary or String is converted to an UnsafePointer using implicit inout-to-pointer conversion:
func ptrToDictionary(_: UnsafePointer<Dictionary<K, V>>) {}

func testDictionary() {
  var d: Dictionary = ...
  ptrToDictionary(&d)
}

func ptrToString(_: UnsafePointer<String>) {}

func testString() {
  var s: String = ...
  ptrToString(&s)
}

Address to pointer conversion must always be guarded by either a mark_dependence or a fix_lifetime. Use fix_lifetime for call emission in SILGen to limit the optimization impact to the narrow scope of the pointer conversion.
This could negatively impact performance simply because SIL does not provide a way to scope pointer conversion. fix_lifetime is unnecessarilly conservative and prevents the elimination of dead allocations.
(cherry picked from commit c1d465303d97a09f10bca0b7762643c1d998e6e4)
Risk: Low. The fix adds a conservative lifetime marker to SIL. When changing the SIL pattern produced by SILGen, there is a risk that some poorly written SIL pass will assert because it had never "seen" this pattern, or that a poorly written optimization will regress.
Scope: This fix causes SILGen to emit an extra lifetime marker whenever a non-trivial type is passed 'inout' to a function that takes an Unsafe[Raw]Pointer.
Original PR: https://github.com/apple/swift/pull/69476
Reviewed By: Nate Chandler @nate-chandler
Testing: Added several unit tests.
Resolves: rdar://117807309 (Fix SILGen to emit fix_lifetime for inout-to-pointer conversion)

@swift-ci test

@swift-ci test

@swift-ci please test windows platform


M10spg

Adds m10spg functionality (hopefully)

I beg to holier powers that this may work, and if it does not I will rewrite the entire thing in NMEA format as pennance

Not working yet! I was using this command to try and get a better sense of what was going on today and thought I should keep it.

Not working yet! I was using this command to try and get a better sense of what was going on today and thought I should keep it.

UBLOX hates to see us winning

Resolves #8.
We'll need new issues for all the improvements that can be made to this work. I also expect we might need to talk about the data types again. Converting altitude to a float could be more agnostic of the sensor, but would hurt our performance on smaller processors for not much gain. I would prefer just to use mm and a unsigned integer, and maybe we can just be more careful about our math for the MS5611 and change that too. This also works better for the packager format, since it uses unsigned integers for GPS data.
I think there's a decent amount of duplication the collector that could be fixed too.

Merging for now, added some review comments to work on while implementing more features though.


Fixing import and export

fixing issue
https://github.com/threefoldtech/jumpscaleX_core/issues/596

code refactor change property name threebot_zdb_sonic_start to start_servers_threebot_zdb_sonic
Fixing an issue related to storclient
Save schema before exporting

Thanks !


Public key size

Hello,
how can i get the size of public key ?
is there a function that does that ?
Best regards

Thank you, it works fine :)


UI Refactor

User Story
UI Refactor

User Story Title
UI Refactor

T-shirt size: S(mall), M(edium), L(arge)
S

User Story
As a user, I benefit from an agile team that sustains its velocity.

Description
Refactor minor areas where technical debt has accrued.

Acceptance Criteria/Tests
The list of tests or criteria to be used to validate the story has been completed. May still need UAT or SIT / integration testing with other stories.

not specific enough to be actionable....  we should specify areas that need the work so we can size it.


Make proxiing of invalid urls automatically return client app

if e.g. /test is called the client representation should be returned

Way to go would be something along the line if nginx from proxy returns 404 return redirect to /


Support Custom Extensions to Cyber Observables

+1 to this issue - We are doing this by dumping to JSON, adding our properties in, and re-wrapping as ObservedData

Just to add to this - you currently cant have a custom observable in ObservedData - working on a patch.

`import json
from stix2 import bundle
from stix2 import ObservedData
Custom STIX Observables
@stix2.observables.CustomObservable(type='x-location', properties={
'country': stix2.properties.StringProperty(required=True),
'city': stix2.properties.StringProperty(),
})
class CustomLocation():
def init(self, **kwargs):
pass
location = CustomLocation(country="England",city="Birmingham")
obsd = ObservedData(first_observed=timezone.now(),last_observed=timezone.now(), number_observed=1,objects={
"0":json.loads(str(location))
},allow_custom=True)`

Thank you for the feedback, @AdamTheAnalyst! I've renamed this issue since the bug you found is unrelated to the original issue, and made a new issue for the original feature this one was referring to.
I'm also looking into your error and adding a unit test for it.


[FEATURE] Test progress counter

One of my friends wanted to check the program and gave him quite a lot of assignments because he has 77GB (2227 files) of mods and the program ran for 2 hours and still did not finish work and it is not known whether it crashed or still tesed, so it would be good if there was a progress counter, e.g. on bottom of the window:
Tested 245/2227
He only saw these log entries:
NOTICE: 2021-06-22T20:18:37.067Z (collection) Finished read mods on disk (test pending)
NOTICE: 2021-06-22T20:18:38.107Z (collection) Finished read save games
FILE-ERROR: 2021-06-22T20:18:38.225Z (FS19_barbedWire_fenceKit) Zip missing file: modDesc.xml: Error: Entry not found
FILE-ERROR: 2021-06-22T20:18:38.621Z (JohnDeere_Series_8R) Zip missing file: modDesc.xml: Error: Entry not found
FILE-ERROR: 2021-06-22T20:18:38.765Z (aaa_FS19_SDC_Flatbed) Caught XML Parse error: Error: Invalid character in entity name
Line: 2
Column: 24
Char:
FILE-ERROR: 2021-06-22T20:18:39.386Z (do tego sie kurwa nie przyznaje wiec usunalem nazwe zipa) Caught XML Parse error: Error: Malformed comment
Line: 277
Column: 22
Char: -
FILE-ERROR: 2021-06-22T20:18:39.414Z (FS19_CM_CR9000Series) Caught XML Parse error: Error: No whitespace between attributes
Line: 27
Column: 42
Char: i
FILE-ERROR: 2021-06-22T20:18:39.558Z (FS19_JohnDeere6R_2014-2018) Caught XML Parse error: Error: Invalid character in entity name
Line: 2
Column: 18
Char:
FILE-ERROR: 2021-06-22T20:18:39.567Z (FS19_JohnDeere_6M_6R_by6195rpower) Caught XML Parse error: Error: Invalid character in entity name
Line: 2
Column: 17
Char:
FILE-ERROR: 2021-06-22T20:18:39.592Z (FS19_kroneBiGX_Series) Caught XML Parse error: Error: Invalid character in entity name
Line: 6
Column: 34
Char:
FILE-ERROR: 2021-06-22T20:18:39.845Z (FS19_Zetor_56xx_pack) Caught XML Parse error: Error: Invalid character in entity name
Line: 32
Column: 42
Char: "
NOTICE: 2021-06-22T20:18:40.198Z (ipcProcess) Could not get "broken list" : Error: ENOENT: no such file or directory, open

This should be doable.  I'll dig a bit on that log - I can tell you that the scan did error out - that "ipcProcess" at the end is it asking for the first of the lists to populate the UI and failing.  But that is a catch all at the end that should have been unreachable, clearly it is not.

Counter is about done.  Trying to figure out how it got the that unreachable portion.
(pardon the noise, I don't want to lose my notes)
Things it is not:

key length (mod name).  Valid keys can be ~134M characters long (2^27).  Can't possible hit that on the filesystem.
key value (mod name) - it's always escaped. Even if it were not, there is no character that is valid on the filesystem that isn't in a string key name.
number of files - there is a key limit, but someone testing it 5 years ago found it to be in the neighborhood of 400k before things got wobbly.

BUT
there is a memory limit. 2^32 bytes per object. I suppose it is possible we are hitting this.
Moving forward - any chance when I drop the new version your friend can re-run the scan with some sort of profiler open? (even just task manager) and look at the memory footprint - if it starts going way, way up, this would be where I look next.  I'm hesitant to start here, because I'm really not sure where I could trim.  I'm loading the bare basic details, and keeping nothing open.
Oh.  one last bit - are any the mods in their folder overly large (maybe 2+ Gb).  I don't know that my zip library in particular will cope well with that.
Thanks!

He tested on the current version (he will also do it on the new one), here is a screan before starting the program:

after starting on:

and a video while working:

This is super helpful.  That drop at about 0:03 - 0:05 in usage looks a whole lot like a out-of-memory error garbage collection to me.  Not sure why or where, but it gives me a bit more to go on.  Thank you!

Here is a test of the new version, unfortunately the test counter does not work properly.
https://user-images.githubusercontent.com/2678653/123527765-57dd3580-d6e2-11eb-940b-a3f9b60967da.mp4
log.txt

Version 1.9.25 works fine.


SunmiScan module not scanning barcodes ( only scanning qr codes)

@Surile  I am trying to scan with SunmiScan.scan() but it only scans the qr code, and not scanning the barcodes.
also how can we come to know that what is the type of qr or barcode because right now it only sending the data of qr.

@Rehankhalil462 https://developer.sunmi.com/docs/en-US/xeghjk491/ciceghjk502

@Surile I am using in expo react native , it scans other barcodes , but i have code128 barcode which is not being scanned by this. Can you please tell how can i check which type of barcode was scanned by sunmi v2s. because right now i am just getting the data of code.

@Surile also is there any way to get the high quality result of receipt images while using printBitmap method?

I don't think I have the function to support barcode. I read that the document vendor has updated the sdk

---- Replied Message ----

From
Rehan<EMAIL_ADDRESS>

Date
10/14/2022 21:54

To

Cc
、

Subject
Re: [Surile/react-native-sunmi-printer] SunmiScan module not scanning barcodes ( only scanning qr codes) (Issue #18)

 I am using in expo react native , it scans other barcodes , but i have code128 barcode which is not being scanned by this. Can you please tell how can i check which type of barcode was scanned by sunmi v2s. because right now i am just getting the data of code.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


⚠ Bofrost.at has degraded performance

In ddb6ce4, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 932 ms

Resolved: Bofrost.at performance has improved in e3d814b after 6 minutes.


Screen shots and work around out dated

Hi
The screenshots for Ionos are outdated and they have changed the layout of their DNS records. Also clicking on the loop for the video continually reloads this page rather than showing the video.

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

ID: 60ec04cf-93af-b216-4290-c2d0ff79878d
Version Independent ID: e7fd186e-7efd-711e-3f29-e318b2100083
Content: Create DNS records at 1&1 IONOS for Office 365
Content Source: Office365-Admin/dns/create-dns-records-at-1-1-internet.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

@pebaum, is this the video referred to in this article? https://www.youtube.com/watch?v=H7KLpm50kv8

@pebaum - we would like to follow up and get your insights on the query from @jamesrhea. Thank you.

removed the video link - working on removing outdated screenshots across this set.


Feat: Add more unsupported acceptance tests

Signed-off-by: Daniel Ivanov<EMAIL_ADDRESS>Description:
Related issue(s):
Fixes #
Notes for reviewer:
Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report

Merging #191 (250ee46) into main (a7455a6) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #191      +/-   ##
==========================================
- Coverage   68.91%   68.86%   -0.06%
==========================================
  Files           7        7
  Lines         608      607       -1
  Branches       87       87
==========================================
- Hits          419      418       -1
  Misses        162      162
  Partials       27       27

Impacted Files
Coverage Δ

packages/relay/src/lib/eth.ts
54.45% <ø> (-0.15%)
:arrow_down:

Continue to review full report at Codecov.

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


Add support for Tuya LH-961ZB motion sensor

This quirk fixes the problem there the iaszone state is never reset.

Codecov Report
Patch coverage: 72.72% and project coverage change: -0.05 :warning:

Comparison is base (fa60b9d) 83.92% compared to head (d440a84) 83.88%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2289      +/-   ##
==========================================
- Coverage   83.92%   83.88%   -0.05%
==========================================
  Files         260      261       +1
  Lines        8219     8252      +33
==========================================
+ Hits         6898     6922      +24
- Misses       1321     1330       +9

Impacted Files
Coverage Δ

zhaquirks/tuya/sm0202_motion.py
72.72% <72.72%> (ø)

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.

Thanks for the information, I have updated it


Cannot deploy perseus using beta.19

This issue is reporting a bug in the code of Perseus. Details of the scope will be available in issue labels.
The author described their issue as follows:

Cannot deploy an app in the beta.19 due to missing item HSR_IGNORE on generic type S in functions of package perseus:

get_frozen_global_state_and_register
get_frozen_state_and_register

The steps to reproduce this issue are as follows:

I'm not sure whether my app has something special that makes the deployment fail. It might just fail trying to deploy any app.

A minimum reproducible example is available at <>.

Hydration-related: false
The author is willing to attempt a fix: true

Tribble internal data
dHJpYmJsZS1yZXBvcnRlZCxDLWJ1ZyxBLWRlcGxveW1lbnQsYXV0aG9yLXdpbGxpbmctdG8taW1wbA==

I'm willing to open a PR for this, I want to make sure I have the right idea because I didn't spend enough time in the project to understand what this code does. Here's my idea:
The trait MakeRx looks like this (without the doc comments):
pub trait MakeRx {
    type Rx: MakeUnrx;
    #[cfg(debug_assertions)]
    const HSR_IGNORE: bool = false;
    fn make_rx(self) -> Self::Rx;
}

Meaning HSR_IGNORE is available on types that implement the MakeRx trait only on debug builds.
But the functions get_frozen_state_and_register and get_frozen_global_state_and_register both have this condition:
if *is_hsr && S::HSR_IGNORE {
    return Ok(None);
}

Which has no guard to not be compiled in non-debug builds.
I would propose to simply guard either the full condition, or split it in order to prevent the compiler from tying to find HSR_IGNORE on non-debug builds.
Or perhaps HSR_IGNORE should be defined on all builds and in this case we would just remove it from the trait definition.
Please let me know, I'm willing to open a PR if it is a simple change (again because of my light understanding of what is actually going on there) or feel free to fix it quickly.
PS; I'm surprised something like this would not be picked up while trying to build the library in release mode, I haven't tried though.

 PR would be great! (I'll also check up on the CLI tests, which should have caught this...)


fix log at analyzer detection

get_binary_in_path can return False in that case the debug log fails

We should not throw an exception because python could not concatenate a boolean value with a string in the debug log. (printing out false does not give us more information)
The case if there was no analyzer binary was found is handled in the analyzer_types, but the execution does not get there because of the exception before.
https://github.com/Ericsson/codechecker/blob/8d5089709133e59e7ca035e07bc7d2b79b53c330/libcodechecker/analyze/analyzers/analyzer_types.py#L99-L122

I knew one day we should turn every log print into .format instead of simple concatenation...


Does anyone know how to help?

Beginner here. While installing I came across an error and I'm wondering if anyone knows a solution any help would grateful. It comes when i put in the line "python3 tookie-osint". I'm using a command prompt window and have python installed on it but idk

pip install alive-progress
https://pypi.org/project/alive-progress/

Thank you so much I'll try later.

pip install alive-progress
https://pypi.org/project/alive-progress/

This should the answer. If you have issues with it, try pip3 install alive-progress

Thank you so much!!


firefox addon no have hcapcha solver?

firefox addon no have hcapcha solver?

I loaded version 0.4.12 from Chrome's local extension cache and the hCAPTCHA solver is still working fine, so I am guessing this change was unintentional.

Apparently Hcaptcha sent a bunch of cease and desist letters to all the major solvers, that's probably why. The extension failed at solving basically all hcaptcha stuff for me anyways.

I loaded version 0.4.12 from Chrome's local extension cache and the hCAPTCHA solver is still working fine, so I am guessing this change was unintentional.

but there is no solution anymore, and it still doesn't work

I loaded version 0.4.12 from Chrome's local extension cache and the hCAPTCHA solver is still working fine, so I am guessing this change was unintentional.

is it still working? i also installed 0.4.12, hcaptcha is displayed in the extension interface but not resolved. I looked at the traffic and saw that the api responds that the type is invalid, I also checked the documentation and the request is sent correctly

I also tried to send the same request but specifying recaptcha in the type, I got a different error in the response, I guess they completely cut hcaptcha from the api

I would also like to know from those who bought a subscription, is this service able to solve enterprise hcaptcha? in the documentation it says that it can do it, but as I have already seen, the documentation should not be trusted. And support has been down for 2 weeks now and they keep promising to bring it back

le epic exit scam. if yall paid by card, chargeback. this will get their stripe accounts closed in no time. also do paypal chargeback if paid by paypal


Mod suggestion

Mod name
Clumps
Curseforge link
https://www.curseforge.com/minecraft/mc-mods/clumps
Other links
https://github.com/jaredlll08/Clumps
What it does
Clumps groups XP orbs together into a single entity to reduce lag when there are many in a small area. On top of this, it also makes the player immediately collect the orbs once they touch the player, so you are not stuck with a bunch of orbs in your face.
Why should it be in the modpack
Could be useful in mob grinders
Why shouldn't it be in the modpack
I don't know... it's another mod
Categories
Performance optimization, New feature
Additional details
No response

Thanks for the suggestion! However, a similar mod was already rejected for the following reasons: https://github.com/Fabulously-Optimized/fabulously-optimized/issues/71#issuecomment-864748974.


ValueError: The number of classes has to be greater than one; got 1

I am trying to train my data using spaCy + scikit-learn because MITIE is taking too long time to train my data. But when I run python -m rasa_nlu.train -c config_spacy.json it produces error saying
ValueError: The number of classes has to be greater than one; got 1

And I tried to search the solution on web but didn't find anything very useful. Can you please what is possibly I am doing wrong here.
My data is something like this.
{
  "rasa_nlu_data": {
    "common_examples": [
      {
        "text": "Dear Customer, You have made a Debit Card purchase of INR 962.00 on 26 Oct. Info.VPS Brown House. Your Net Available Balance is INR 5,584.58.",
        "intent": "parseSms",
        "entities": [
          {
            "start": 31,
            "end": 41,
            "value": "Debit Card",
            "entity": "cardType"
          },
          {
            "start": 42,
            "end": 53,
            "value": "purchase of",
            "entity": "transactionType"
          },
          {
            "start": 54,
            "end": 57,
            "value": "INR",
            "entity": "spendCurrency"
          },
          {
            "start": 58,
            "end": 64,
            "value": "962.00",
            "entity": "spendAmount"
          },
          {
            "start": 85,
            "end": 96,
            "value": "Brown House",
            "entity": "merchantsName"
          },
          {
            "start": 128,
            "end": 131,
            "value": "INR",
            "entity": "balanceCurrency"
          },
          {
            "start": 132,
            "end": 140,
            "value": "5,584.58",
            "entity": "balanceAmount"
          }
        ]
      },
     // Similar entries
}

P.S : I tried to run the same training data using MITIE backed on aws instance type m4.4xlarge, and process existed successfully without generating any error or directory.

Without knowing your training data, that's hard to guess. But judging by the error message I'd assume all your training examples are labeled with the same intent. Can you please check that. Adding examples with a different intent should solve the issue then.

@tmbo : I have already given the my training data sample. And yes all my examples of the same intent. When i added examples with another intent it solves my existing problem. But a new error has come now.
/usr/local/lib/python2.7/dist-packages/sklearn/metrics/classification.py:1113: UndefinedMetricWarning: F-score is ill-defined and being set to 0.0 in labels with no predicted samples.
  'precision', 'predicted', average, warn_for)
/usr/local/lib/python2.7/dist-packages/sklearn/metrics/classification.py:1115: UndefinedMetricWarning: F-score is ill-defined and being set to 0.0 in labels with no true samples.
  'recall', 'true', average, warn_for)


Fix for Image dropdown not working

tracked down to a change in b05349838e0ea074ec0bdd49e785e63c0372e81d using git bisect :)

Thanks!


Release/session

Add examples
Use requests sessions
Add more logging

Will get the release out but someone needs to fix the indents @mnemonicflow

@geeknam As per https://www.python.org/dev/peps/pep-0008/#code-lay-out "continuation lines with more indentation included to distinguish this from the rest" is totally alright and this is a reformatting rule that I've run the code through inside PyCharm, but I'm not sure this project adheres to PEP8 coding style standards.
After a second look there are other cases where more spaces on continuation lines can be added by the above rule. If you want I can undo this or I can reformat again to apply the rule across the whole code.


Create OccurenceOfCharInString.java

Contributing
If you want to contribute to Algorithmics and make it better, your help is very welcome. Contributing is also a great way to learn more about social coding on Github, new technologies and and their ecosystems and how to make constructive, helpful bug reports, feature requests and the noblest of all contributions: a good, clean pull request.
How to make a clean pull request
Look for a contribution instructions.

Create a personal fork of the Algorithmics on Github.
Clone the fork on your local machine. Your remote repo on Github is called origin.
Add the original repository as a remote called upstream.
If you created your fork a while ago be sure to pull upstream changes into your local repository.
Create a new branch to work on! Branch from develop if it exists, else from master.
Implement/fix your feature, comment your code.
Follow the code style of the project, including indentation.
If the project has tests run them!
Write or adapt tests as needed.
Add or change the documentation as needed.
Squash your commits into a single commit with git's interactive rebase. Create a new branch if necessary.
Push your branch to your fork on Github, the remote origin.
From your fork open a pull request in the correct branch.
And that's it, we will take care of the rest.

 Your commit message should describe what the commit, when applied, does to the code – not what you did to the code.

Thanks for contributing, please refer the contributing guidelines (contributing.md) to make a valid contribution

Closing this PR for now, feel free to reopen if you'd like to work on it


[Bug]: The config file contains multiple bars, but no bar name was given

Checklist

[X] I have read the appropriate section in the contributing guidelines
[X] I believe this issue is a problem with polybar itself and not a misconfiguration on my part
[X] I have searched for other open and closed issues that may have already reported this problem
[X] I have checked the known issues page for this problem.
[X] I have followed the debugging guide to narrow down the problem to a minimal config.

Steps to reproduce

Just running polybar

Minimal config
[colors]
disabled = #707880
Rosewater = #f5e0dc
Flamingo = #f2cdcd
Pink = #f5c2e7
Mauve = #cba6f7
Red = #f38ba8
Maroon = #eba0ac
Peach = #fab387
Yellow = #f9e2af
Green = #a6e3a1
Teal = #94e2d5
Sky = #89dceb
Sapphire = #74c7ec
Blue = #89b4fa
Lavender = #b4befe
Base = #1e1e2e
Mantle = #181825

[bar/left]

width = 16%
offset-x = 1%
offset-y = 1%
height = 22pt
fixed-center = true

background = ${colors.Mantle}
foreground = ${colors.Lavender}

line-size = 1pt

font-0 = "RobotoMono Nerd Font:weight=bold:size=9;2"
font-1 = "RobotoMono Nerd Font:size=10;3"
font-2 = "RobotoMono Nerd Font:size=11;3"

modules-center = power xworkspaces

enable-ipc = true

wm-restack = generic

[bar/middle]

width = 18%
offset-x = 41%
offset-y = 1%
height = 22pt
fixed-center = true

background = ${colors.Mantle}
foreground = ${colors.Lavender}

line-size = 1pt

font-0 = "RobotoMono Nerd Font:weight=bold:size=9;2"
font-1 = "RobotoMono Nerd Font:size=10;3"
font-2 = "RobotoMono Nerd Font:size=11;3"

modules-center = day space sep space date space sep space time

enable-ipc = true

wm-restack = generic

[bar/right]

width = 21%
offset-x = 78%
offset-y = 1%
height = 22pt
fixed-center = true

background = ${colors.Mantle}
foreground = ${colors.Lavender}

line-size = 1pt

font-0 = "RobotoMono Nerd Font:weight=bold:size=9;2"
font-1 = "RobotoMono Nerd Font:size=10;3"
font-2 = "RobotoMono Nerd Font:size=11;3"

modules-center = update space sep space pulseaudio space sep space backlight space sep space battery space sep space wlan space

enable-ipc = true

wm-restack = generic

[module/xworkspaces]
type = internal/xworkspaces

label-active = 
label-active-padding = 2
label-active-foreground = ${colors.Mauve}
label-active-font = 2

label-occupied = 
label-occupied-padding = 2
label-occupied-font = 2

label-empty = 
label-empty-foreground = ${colors.disabled}
label-empty-padding = 2
label-empty-font = 2

[module/pulseaudio]
type = internal/pulseaudio
format-volume-prefix = "墳 "
format-volume-prefix-foreground = ${colors.Rosewater}
format-volume = <label-volume>
format-volume-prefix-font = 3

label-volume = %percentage%%

label-muted = muted
label-muted-foreground = ${colors.disabled}

[network-base]
type = internal/network
interval = 5
format-connected = <label-connected>
format-disconnected = <label-disconnected>
label-disconnected = disconnected
format-connected-foreground = ${colors.Lavender}

[module/wlan]
inherit = network-base
interface-type = wireless
label-connected-font = 3
label-connected = 
label-connected-foreground = ${colors.Rosewater}

[module/day]
type = internal/date
interval = 1

date = %A

label = %date%
label-foreground = ${colors.Lavender}

[module/date]
type = internal/date
interval = 1

date = %d-%m-%Y

label = %date%
label-foreground = ${colors.Lavender}

[module/time]
type = internal/date
interval = 1

date = %H:%M:%S

label = %date%
label-foreground = ${colors.Lavender}

[module/battery]
type = internal/battery
poll-interval = 5
full-at = 99
format-full-prefix = "  "
format-charging-prefix = "  "
format-discharging-prefix = "  "
format-full-prefix-foreground = ${colors.Rosewater}
format-charging-prefix-foreground = ${colors.Rosewater}
format-discharging-prefix-foreground = ${colors.Red}
label-charging = %percentage%%
label-discharging = %percentage%%
label-full = %percentage%%

[module/backlight]
type = internal/backlight
card = intel_backlight
format-prefix = " "
format-prefix-foreground = ${colors.Rosewater}
format-prefix-font = 3

[module/update]
type = custom/script
exec = /home/meledo/.config/polybar/update.sh
interval = 960
format-prefix = "ﰵ "
format-prefix-font = 3
label = %output%
format-prefix-foreground = ${colors.Rosewater}
click-left = "kitty yay -Syu"

[module/power]
type = custom/text
content = 
content-font = 3
content-foreground = ${colors.Red}
content-margin = 1
click-left = "sh ~/.local/bin/powermenu"

; decor

[module/sep]
type = custom/text
content = "|"
content-foreground = ${colors.disabled}

[module/space]
type = custom/text
content = " "

[settings]
screenchange-reload = true
pseudo-transparency = true

Polybar log
[aesth@localhost ~]$ polybar
notice: Parsing config file: /home/aesth/.config/polybar/config.ini
error: Uncaught exception, shutting down: The config file contains multiple bars, but no bar name was given. Available bars: left, middle, right

Expected behavior
To show the bar
Actual behavior
It gives me this error instead and my config file seems correct
Window Manager and Version
i3-gaps 4.20.1-2
Linux Distribution
Arch Linux 5.18.2-arch1-1
Polybar version
polybar 3.6.3

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags: -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions         -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security         -fstack-clash-protection -fcf-protection -Wp,-D_GLIBCXX_ASSERTIONS -g -ffile-prefix-map=/build/polybar/src=/usr/src/debug -flto=auto -O3 -DNDEBUG -Wall -Wextra -Wpedantic -Wsuggest-override
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now -flto=auto -Wall -Wextra -Wpedantic -Wsuggest-override  -Wall -Wextra -Wpedantic -Wsuggest-override

Additional Context / Screenshots
There is another issue that was like this when I tried to do what the answer told me to do it still didnt work

Thanks, But how would I be able to start all the bars at once

You don't. You run one polybar command per bar.

Thanks


figure out a way around docker0 bridge not connecting to DNSmasq.

Currently on both OPi-lite and Nanopi the docker0 bridge always seems to attach to <IP_ADDRESS> mean while the DNSmasq resolver is listening on <IP_ADDRESS> so no containers are able to resolve DNS.
A currently work around is to add --bip <IP_ADDRESS>/16 to the command that starts the docker daemon in meta-resin.

relates to https://github.com/resin-os/resinos/issues/229

fixed in meta-resin


Integrating Kaguya as a GPTs Action Due to HTTP/HTTPS Mismatch

I am attempting to integrate Kaguya with GPTs as an action, but I am encountering an issue related to the HTTP and HTTPS protocols.
Steps to Reproduce:

I ran the Kaguya plugin and successfully accessed it at http://localhost:3000.

I copied the contents from http://localhost:3000/openapi.yaml and pasted it into GPTs actions.

Upon doing so, I received the following warnings:
Warnings
None of the provided servers is under the root origin https://localhost
Server URL http://localhost:3000 is not under the root origin https://localhost; ignoring it

Issue:
It appears that there is a protocol mismatch issue, as Kaguya is running on HTTP and GPTs actions seem to require HTTPS. As a result, the server URL is being ignored due to not being under the root origin https://localhost.
Questions:

Has anyone successfully integrated Kaguya as an action in GPTs?
Although Kaguya is originally intended as a ChatGPT plugin, I believe there should be a close similarity in its integration with GPTs. If it is indeed feasible to run it as a GPTs action, how should the yaml be modified to address the HTTP/HTTPS protocol mismatch?

Any guidance or suggestions on resolving this issue would be greatly appreciated. Thank you for your support and for the development of this innovative tool.
Best regards,

You should deploy your service to the public internet


fix: Drop trailing hex characters from version header values

Fixes #636

Marked as "do not merge" while we wait for confirmation that this won't break anything.


luci-app-statistics: add ipstatistics plugin

@hnyman Can we merge this, too?

You are not going to format label display units first?
I would assume that something like number_format: "%5.1lf%SB/s" might make sense.
(or %s instead of %S if the values magnitudes differ much)
E.g.
https://github.com/openwrt/luci/blob/e46c96750aa8b5a4f9d3ed1cd7909485a26c1353/applications/luci-app-statistics/htdocs/luci-static/resources/statistics/rrdtool/definitions/iptables.js
Docs at:
https://oss.oetiker.ch/rrdtool/doc/rrdgraph_graph.en.html

@PolynomialDivision
Hmm. I didn't test this before merging, as it seemed to run for you.
But when I try to enter the LuCI statistics config section with that collectd plugin installed, I get now an error, likely due to the missing configSummary section.
Does the config part really work for you in LuCI?

No it did not work.
Sry, I did not try on a fresh install. I wrote the collectd config manually, so I did not notice and deleted the configSummary Part since I thought it was useless. Next time I will be more careful.


todayonline.com

www.todayonline.com
todayonline.com
Its a news site. Not malicious

@funilrys  move this one to us please as further discussion is required.
@p1r473  it's indeed false-positive,it might take a while but it will be whitelisted one way or another.
Thanks for the report 👍

/move to Ultimate-Hosts-Blacklist/whitelist


Bug/scroll top update

Re-arrange and re-implemented props.scrollTop, props.scrollLeft, props.scrollToRow, and props.scrollToColumn handling.
#202
Updates to these props would now take effect.

this is fixed in https://github.com/schrodinger/fixed-data-table-2 😻


setting the pooling strategy - supported models?

I receive the error below when trying to use the "hkunlp/instructor-xl" . It seems related to selection of the pooling mode.
Also, when running WhereIsAI_UAE-Large-V1 it seems to auto-select "MEAN" pooling whereas the model authors recommend "cls"
Are these models currently supported?
===traceback
INFO:     Started server process [1]
INFO:     Waiting for application startup.
INFO     2023-12-06 19:53:55,770 infinity_emb INFO:           select_model.py:17
model=hkunlp/instructor-xl selected, using
engine=<class                                                                   'infinity_emb.transformer.sentence_transformer.Sente                             nceTransformerPatched'> and device=None
INFO     2023-12-06 19:53:55,772                       SentenceTransformer.py:66
sentence_transformers.SentenceTransformer
INFO: Load pretrained SentenceTransformer:
hkunlp/instructor-xl
ERROR:    Traceback (most recent call last):
File "/app/.venv/lib/python3.10/site-packages/starlette/routing.py", line 677, in lifespan
async with self.lifespan_context(app) as maybe_state:
File "/app/.venv/lib/python3.10/site-packages/starlette/routing.py", line 566, in aenter
await self._router.startup()
File "/app/.venv/lib/python3.10/site-packages/starlette/routing.py", line 654, in startup
await handler()
File "/app/infinity_emb/infinity_server.py", line 182, in _startup
model, min_inference_t = select_model_to_functional(
File "/app/infinity_emb/inference/select_model.py", line 21, in select_model_to_functional
init_engine = engine.value(model_name_or_path, device=device.value)
File "/app/infinity_emb/transformer/sentence_transformer.py", line 52, in init
super().init(*args, **kwargs)
File "/app/.venv/lib/python3.10/site-packages/sentence_transformers/SentenceTransformer.py", line 95, in init
modules = self._load_sbert_model(model_path)
File "/app/.venv/lib/python3.10/site-packages/sentence_transformers/SentenceTransformer.py", line 840, in _load_sbert_model
module = module_class.load(os.path.join(model_path, module_config['path']))
File "/app/.venv/lib/python3.10/site-packages/sentence_transformers/models/Pooling.py", line 120, in load
return Pooling(**config)
TypeError: Pooling.init() got an unexpected keyword argument 'pooling_mode_weightedmean_tokens'

Closing because of https://huggingface.co/hkunlp/instructor-base/discussions/6 - Instructor is just not compatible with sentence-transformers


Update README.md

Is there a reason to change the formatting?

Not really. Just thought it looked better :smile:

It does look better

Eh whatever :smile:
Assigning to @Tratcher .


Optimize scoped tag behavior

Right now we append whole object as the source of data. We could be smarter about it and analyze the template to see which variables are being used.
const { root } = STORE

in the case above, we're sure that only root is accessed so we can fill up the store with this property only
The behavior above is "static", but in case of dynamic behavior which can not be analyzed, e.g.
STORE[Math.random()]
we'd need to include whole store as we do now.

In the following example, we know that the value is used in a static way only:
<script>
const { root } = scope
</script>

By walking through the tree, we could create a list of used identifiers and then only inject this data into the template.
<script>
const { root } = { root: "abcd1" }
</script>

There is an exception, where the developer wants to take the value dynamically, e.g.:
<script>
  const x = scope[some_magic_value]
</script>

in such case we want to keep the current behavior


trigger meet kostas does not always fire

The trigger fires once the whole team is mounted in thehelicopter (code). However, if one of the team members mounts as commander, he apparently is not a member of the crew.
see also related issue #54.
Maybe switch to using fullCrew instead of crew.

On second thought, maybe it has something to do with the order in which the units are listed by crew and by units group player

Order was to blame, changed the condition in e4ed65b. Closing the issue.


Add a publish-build-info command for reporting build results to crates.io

We're working on reviving this old thing :) The associated crates.io PR should go in first.
This goes with https://github.com/rust-lang/crates.io/pull/764. @shepmaster worked on these too :)
This PR adds a cargo command, publish-build-info, which allows crate owners to report crate version, rust version, target, and pass/fail to crates.io.
The purpose is to enable crate authors to automatically report to potential crate users on the Rust version and platform compatibility of each version of the crate.
The idea is that this command would be added to CI, and an example of configuration for Travis is in the documentation we added. We haven't actually tested that configuration out yet; we wanted to get these PRs in for feedback before we set up crates.io instances that weren't local but that we could publish to :) We want to add Appveyor too, but we want to test the travis configuration first.
We also thought about, but didn't implement:

An --overwrite flag to allow editing of an already-reported-on crate version+rust version+target combination, to allow for correcting errors if reporting manually Overwriting is now the default behavior.
Adding flags to enable manual specification of rust version and crate version, right now cargo just detects what it's running with. Target can be overridden with --target since there was already support for that flag :)

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Like https://github.com/rust-lang/cargo/pull/3627 looks great to me! I think we can mege whenever crates.io is ready

:umbrella: The latest upstream changes (presumably #5152) made this pull request unmergeable. Please resolve the merge conflicts.

I'm gonna close this for now as it's pretty out of date at this point, but it can of course be resubmitted!


No need for .nvmrc, since we mention engine into packge.json

I want to merge this change because...

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

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

Sure, what do you think about lts/* instead of specifying a specific version since we don't care much here about the version?
ref https://github.com/nvm-sh/nvm#nvmrc @maarcingebala

Sounds good, but the current LTS is 16.13.2, so we should also update versions of node and npm defined in package.json.

You don't need both, .nvmrc file and engine in package.json ... just decide to use any of them instead of changing both every time
https://stackoverflow.com/questions/61403815/js-nvmrc-vs-package-json-engines
I've no problem to update both by the way

OK, so let's stick with one version defined in package.json, anyone actually needing to use any of the npm commands with this repo should also understand how to set the correct version of Node.

@HazemKhaled Could you rebase the PR against the main branch? We push all changes related to Saleor 3.1 to main, later during releases we move them to the 3.1 branch.

I just stumbled over this a bit. It's unclear if this project requires Node v10.15.2 or Node >=16 and <17.


pretty_print while also sorting based on outer object name

PLEASE DON'T FORGET TO "STAR" THIS REPOSITORY :)
When reporting a bug please include the following:
Version of Boost
1.81.0
You can find the version number in the file <boost/version.hpp>
Steps necessary to reproduce the problem
The pretty print example https://www.boost.org/doc/libs/1_75_0/libs/json/doc/html/json/examples.html would work.
All relevant compiler information
MSVC 14.2 also g++ on ubuntu.

This is not a bug report. It is just a query about whether there is a way to have pretty_print also sort (ascending or descending) its output based on the outermost object name?
I am aware that there are separate command line tools that act on a pre-existing json file and produce an output that is sorted. E.g., jq. I had raised a query regarding exactly this on SO here for which the solution was to use jq.
I'd rather not want to run jq separately -- is it possible to enhance pretty_print to make such modifications on the fly within boost::json itself?
Thank you.

You need to extract the keys from the json::object first, sort them, then output the values using the sorted keys. Like this: https://godbolt.org/z/TEe1d3Y4Y

You need to extract the keys from the json::object first, sort them, then output the values using the sorted keys. Like this: https://godbolt.org/z/TEe1d3Y4Y

Thank you.
In trying to replicate this at my end, I have the following Isource8.json file with unprettified raw content:
{"1":{"height":120,"metrics":[[-3,-2,-1],[0,1,2,3]]},"2":{"height":130,"metrics":[[-3,-2,-1,-4],[0,1],[5,7]]}}
Then, the following code reads this content and default pretty_prints into an Osource8-1.json file based on the pretty_print function available at : https://www.boost.org/doc/libs/1_75_0/libs/json/doc/html/json/examples.html
Then, the code pretty_prints into an Osource8-2.json file based on the pretty_print function you have provided at Godbolt and sorts it in descending order.
Everything works fine as expected except that in the latter case, there is an additional unnecessary , after the last entry pertaining to key "1". This may be okay from a JSON format point of view. However, my text editor (VSCode) highlights this as a squiggly red error (Trailing comma json 519, which could be a VSCode internal error number) and I am unsure if this will break something else down the line.
The entire code is:
#include <boost/json.hpp>
using namespace boost::json;
#include <boost/filesystem.hpp>
using namespace boost::filesystem;
#include <vector>
#include <iostream>
#include <fstream>

void pretty_print(std::ostream& os, boost::json::value const& jv, std::string* indent = nullptr)
{
    std::string indent_;
    if (!indent)
        indent = &indent_;
    switch (jv.kind())
    {
    case boost::json::kind::object:
    {
        os << "{\n";
        indent->append(4, ' ');
        auto const& obj = jv.get_object();
        if (!obj.empty())
        {
            auto it = obj.begin();
            for (;;)
            {
                os << *indent << boost::json::serialize(it->key()) << " : ";
                pretty_print(os, it->value(), indent);
                if (++it == obj.end())
                    break;
                os << ",\n";
            }
        }
        os << "\n";
        indent->resize(indent->size() - 4);
        os << *indent << "}";
        break;
    }

    case boost::json::kind::array:
    {
        os << "[\n";
        indent->append(4, ' ');
        auto const& arr = jv.get_array();
        if (!arr.empty())
        {
            auto it = arr.begin();
            for (;;)
            {
                os << *indent;
                pretty_print(os, *it, indent);
                if (++it == arr.end())
                    break;
                os << ",\n";
            }
        }
        os << "\n";
        indent->resize(indent->size() - 4);
        os << *indent << "]";
        break;
    }

    case boost::json::kind::string:
    {
        os << boost::json::serialize(jv.get_string());
        break;
    }

    case boost::json::kind::uint64:
        os << jv.get_uint64();
        break;

    case boost::json::kind::int64:
        os << jv.get_int64();
        break;

    case boost::json::kind::double_:
        os << jv.get_double();
        break;

    case boost::json::kind::bool_:
        if (jv.get_bool())
            os << "true";
        else
            os << "false";
        break;

    case boost::json::kind::null:
        os << "null";
        break;
    }

    if (indent->empty())
        os << "\n";
}

void pretty_print_new(std::ostream& os, boost::json::value const& jv, std::string* indent = nullptr)
{
    std::string indent_;
    if (!indent)
        indent = &indent_;

    switch (jv.kind())
    {
    case boost::json::kind::object:
    {
        os << "{\n";
        indent->append(4, ' ');

        std::vector<boost::json::string> keys;

        for (auto const& v : jv.get_object())
        {
            keys.push_back(v.key());
        }

        std::sort(keys.begin(), keys.end(), std::greater<>());

        for (auto const& k : keys)
        {
            os << *indent << boost::json::serialize(k) << ": ";
            pretty_print(os, jv.at(k), indent);
            os << ",\n";
        }

        indent->resize(indent->size() - 4);
        os << *indent << "}";
        break;
    }

    case boost::json::kind::array:
    {
        os << "[\n";
        indent->append(4, ' ');

        for (auto const& v : jv.get_array())
        {
            os << *indent;
            pretty_print(os, v, indent);
            os << ",\n";
        }

        indent->resize(indent->size() - 4);
        os << *indent << "]";
        break;
    }

    default:
    {
        os << boost::json::serialize(jv);
        break;
    }
    }

    if (indent->empty())
        os << "\n";
}

int main() {
	std::string fname = "Isource8.json";
	//Read JSON file and store in json::object
        const path fileName(fname);
	object obj;
	std::ifstream file(fname);
	std::string content(std::istreambuf_iterator<char>{file}, std::istreambuf_iterator<char>{});
	value jv = parse(content);
	obj = jv.as_object();
	file.close();

        //Pretty print (original source code) to OSource8-1.json
        fname = "Osource8-1.json";
        std::ofstream ofile1(fname);
        pretty_print(ofile1, obj);
        ofile1.close();

        //Pretty print (New source code) to OSource8-2.json
        fname = "Osource8-2.json";
        std::ofstream ofile2(fname);
        pretty_print_new(ofile2, obj);//Sorts in descending order
        ofile2.close();//There is an unnecessary extra , after "1"
}

(apologies for the wall of code text here. I am trying to figure out how to load Isource8.json over at GodBolt and trying to see if it can output Osource8-1.json and Osource8-2.json in a Godbolt project on the website itself.

As a matter of principle you should want to get rid of that extra comma :)

Instead of outputting the comma after each element and special-casing the last comma, output the comma before each element but the first. So conceptually, instead of this:
1,
2,
3,
4,
output this instead:
,1
,2
,3
,4
,5
It is easier to unroll the first iteration through the loop to leave out the first comma than it is to detect when you are at the last loop iteration (and faster too).

Instead of outputting the comma after each element and special-casing the last comma, output the comma before each element but the first. So conceptually, instead of this:
1, 2, 3, 4,
output this instead:
,1 ,2 ,3 ,4
It is easier to unroll the first iteration through the loop to leave out the first comma than it is to detect when you are at the last loop iteration (and faster too), leaving you with this:
1 ,2 ,3 ,4

Yes, I was able to make a change along the lines you suggested and it works fine. The change was
        int nokeys = static_cast<int>(keys.size());

        int count = 0;
        for (auto const& k : keys)
        {
            os << *indent << boost::json::serialize(k) << ": ";
            pretty_print(os, jv.at(k), indent);
            if(count < nokeys - 1)
                os << ",\n";
            else
                os << "\n";
            count++;
        }

Possibly not the most elegant way to code this, but it seems easier for me to maintain and understand what is going on. I run this part of my code only once and hence speed is not of the utmost importance -- although I understand your idea of unrolling the first iteration and then loop through the remaining.

As a matter of principle you should want to get rid of that extra comma :)

As a matter of principle, https://godbolt.org/z/44nYjnjeY


Peer to Peer Code Review

Leave your feedback here.

It looks great 👍 💯 , I think it's better if you decrease the animation font size for headline part in the mobile version because it scales out a little bit from the mobile screen.

You did a great job 👍 💯 , i think that would be better if you let your works section centralized in the desktop version because its letting a gap in the right side


Feature: Moving a tab to another split

There seems to be no way to move a tab to another currently existing group. :sp and :vsp always introduce a new split which doesnt help with this use case.
Proposal: Introduce a new keybinding just like <C-W> with hjkl which addionally moves the tab to the next group in that direction. If there is no existing group it should be created.

I like this idea - we could use the <C-W>H/J/K/L bindings for moving the window to an adjacent split, creating if necessary.
Some interesting discussion here, too: https://stackoverflow.com/questions/2228353/how-to-swap-files-between-windows-in-vim (Having a <C-w>y and a <C-w>p - a way to yank and paste buffers between windows)


merge package template from arlac77/npm-package-template

Updated standard to latest version

Codecov Report

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

@@          Coverage Diff           @@
##           master    #610   +/-   ##
======================================
  Coverage    62.5%   62.5%
======================================
  Files           1       1
  Lines           8       8
======================================
  Hits            5       5
  Misses          3       3

Continue to review full report at Codecov.

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


Checkbox selectable flag

The selectable flag should not be supported in tree-view with checkbox mode.
/kind feature

Selectable flag now only ignored in recursive checkbox mode. See commit


[backport] Relax tolerance in TestArrayElementwiseOp::test_doubly_broadcasted_pow

Backport of #3758

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 9413f747453174c7610d1a141bc84d5e7b02785b, target branch v7) succeeded!

This is part of CUDA 11 support

LGTM.


Assertion in clang when referring to a structure type that doesn't exist

Trying to compile the following code:
kernel void test(global struct nope* arg) {}

leads to the following assertion
clspv: clspv/third_party/clang/lib/AST/RecordLayoutBuilder.cpp:2987: const clang::ASTRecordLayout& clang::ASTContext::getASTRecordLayout(const clang::RecordDecl*) const: Assertion `D && "Cannot get layout of forward declarations!"' failed.
Abandon

This is coming from the layout checks in the frontend. If I remove those clspv aborts later in the flow.

Still failing here. Why did you close this?

hum, I was compiling with a clvk-like compile line, and it was not failing.

This is passing compiling with clspv -rewrite-packed-structs

This is because of the code in FrontendPlugin.cpp that is only checking the support for the layout when this pass is not used.


fill in GOVERNANCE.md and MAINTAINER.md appropriately

I just put placeholders in these based on the template.

Ack. Thanks.

Updated by PR #212