Feature request: ignore secondary alignments (at least for bam_vs_bed)

Hi,
I've only used a couple of your tools a few times, but it would be helpful to have an option in bam_vs_bed to ignore secondary alignments. I'll take a peek at the code and see if I can do it real quick, otherwise I'll just remove them from my bams.
Thanks!

Thanks, unfortunately that is not easily done. The script is very simplistic and simply runs bedtools multiple times and bedtools itself does not permit flag-filtering of the bam-file. The script could pre-filter, generating a temporary file, but I think that is best done by the user in order to avoid unexpected complications such as 60Gb of /tmp being filled up for a large BAM file.

Closing issue, please re-open if you think the issue is not solved


[Playground] Replace Google Analytics with Matomo (#21763)

This PR:

Adds MatomoAnalyticsService and uses it by default.
Configures it to submit data to https://analytics.apache.org/index.php?module=CoreHome&action=index&date=yesterday&period=day&idSite=19
Adds AnalyticsService as an abstract superclass for faster swaps in the future and to decrease coupling. The old AnalyticsService is renamed to GoogleAnalyticsService for possible future use.
Adds an action to track: click_cancel_run which had been left off before.
Fixes the bug of not tracking changing examples.
Fixes the bug of misreporting open_link event.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

R: @damondouglas

.take-issue

@alexeyinkin can you please rebase this PR?

hm we're waiting before making the matomo change

Per @pabloem we are not going to migrate to Matomo at the moment.


Dev site not processing any CSV imports

The last several imports (https://geomgdev.lib.umn.edu/imports) have not worked:

There is a record of uploading the CSV, but no records are listed as Success or Failed.
The last successful import was 2022-09-29.
The same CSV files import fine into the prod site
Exports work fine. (Exports were broken last week, but fixed by restarting Puma; see #232 )

Sounds like background job processing is down again? @mberkowski

Hmmmm. We just patched the dev host. I will look.

System patches earlier meant that Gems had to be rebuilt on redeploy. Nothing put any errors into logs to this effect.


⚠ AnimeBytes has degraded performance

In 45f7486, AnimeBytes ($AB) experienced degraded performance:

HTTP code: 200
Response time: 4160 ms

Resolved: AnimeBytes performance has improved in d52fec9 after 5 minutes.


Fix spaces, commas, quotes, brackets where needed

Make easier to use rest-api for beginners
Signed-off-by: Fedor Dikarev<EMAIL_ADDRESS>

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"


Help button

Rather than linking directly to DRM, LibRef, etc, put a Help button with a menu of choices.

Why did I want this? It seems unimportant now. Closing.


html content assets are not imported

Describe the bug

Assets in content attribute aren't turned into the production urls. Content is used in image meta tags like twitter:image.
<meta property="twitter:image" content="/screenshot.png" />

System Info
Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:
 System:
    OS: Windows 10 10.0.19042
    CPU: (8) x64 AMD Ryzen 5 3500U with Radeon Vega Mobile Gfx
    Memory: 3.35 GB / 9.95 GB
  Binaries:
    Node: 14.16.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.11 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 90.0.4430.93
    Edge: Spartan (44.19041.906.0), Chromium (90.0.818.49)
    Internet Explorer: 11.0.19041.1
  npmPackages:
    vite: ^2.1.5 => 2.1.5

Used package manager: npm
Before submitting the issue, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the docs.
[x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[x] Provide a description in this issue that describes the bug.
[x] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[x] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

Actually they have to be a direct path.


Add option to build Time Impedance as needed

Original issue https://github.com/oliversalzburg/cbc-kitten-scientists/issues/108 by @hickeys

For example, build first copy at year 40,000. build Nth copy at year 40000 + (N-1 * temporal penalty delay)

Closing Kitten Engineer candidate tickets for the time being. See https://kitten-science.com/engineers/ for details.


slow-odgi: Make a little top-level CLI

This is purely a quality-of-life thing, but it would be cool to make a top-level CLI entry point for the slow-odgi commands. In particular, what is currently python3 slow_odgi/flip.py could become slow-odgi flip, i.e., we'd make an installable executable called slow-odgi with several subcommands.
The concrete steps for this would be:

Add an __init__.py to the slow_odgi directory to make it into a Python package.
Adjust the imports in slow_odgi/*.py to use package-relative imports. For example, use from . import mygfa instead of import mygfa.
Somewhere (in __init__.py or a new __main__.py), make a new main function that dispatches to all of the various algorithms. argparse supports subcommands; we could also try Click instead.
Add metadata to make the slow_odgi package installable (with an executable). As you probably know, my favorite tool for this is Flit, but there are other options available.

Probably best to ignore this until after #29 and #31 are merged.

Other high-level things that should also be done after merging:

[ ] Since beginning to write slow-odgi, I have discovered how to pipe odgi commands. Do that more; cut down on the number of temp files. Maybe this will let me avoid the temp storm I sometimes have, where temp files created when testing one command then become targets for the next command.
[ ] Test against more of the graphs that are already in the odgi repo. I presume the present selection that we fetch with make fetch have been chosen because they are of a reasonable size? @susan-garry?
[ ] The graphs that we fetch frequently fail to actually test odgi commands rigorously. I have in the back of my mind a bit of a zoo of small graphs that could be added, either to the odgi repo or elsewhere, that, when tested against, would run the commands through their paces.
[ ] Refactor! There is some known duplication across branches, and sometimes I have done the same thing in different ways in different commands.

Going to wait until #48 lands, since this will create a large diff and create further opportunities for tidying!


User can't see the whole history in wallet

Expected behavior
User should be able to see whole history of his transactions.

Actual behavior
Wallet shows a lot of records but they are not all.

How to reproduce

Go to wallet.
Scroll to the end. It won't fetch other records.
Compare with Steemit.

Browser: Chrome
Operating system: Windows 10

Fixed in https://github.com/busyorg/busy/pull/976. Closing this but If you re-test this issue, and there is a bug, you can re-open or create a new one.


How to convert musicgen model to ONNX for deploy it on mobile devices

i want to running musicgen model on model devices, then i have to convert musicgen model to onnx format, i split musicgen to language model and compression model, i got errors when tried to convert language model to ONNX format, May i know if any suggestion for help convert it to onnx?

The same problem. Did you solved it ?

any update on this? Im trying to run convert a trained model to onnx and run it like this:
`from audiocraft.data.audio import audio_write
from audiocraft.models import musicgen
import torch
model = musicgen.MusicGen.get_pretrained('medium', device='cuda')
model.set_generation_params(duration=8)
model.lm.load_state_dict(torch.load('lm_final.pt ONNX MODEL HERE'))`

i also desperately need to make this happen... anyone solve this yet?


Fix issue with apphooks

We need to prevent the copy mechanism when a page has an apphook attached to it. This will now throw an error, notifying the user that this action is prohibited:

:+1:

what about pasting / dragging into a page with apphook?

@vxsx this is not part of this PR, I dunno what the correct behaviour should there be yet


chore(dev): release 0.16.12

:robot: I have created a release beep boop
0.16.12 (2024-12-13)
Bug Fixes

fix channel_programs update query in updateLineup (ea73a5d)
fix ProgramGroupingMinter to correctly assign type season for seasons instead of show (0b92fa3)
re-enable fixers (3fab90f)
request additional jellyfin user view types by default (a4cb229)
time slots - always use the currently saved lineup state for slot generation (04e58b6)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.16.12
:sunflower:


lcov.dat not being generated for 0.6.2+ version

I am using ember-cli 2.3.0, not sure if that has got to do anything with the issue

upgrade to ember-cli-blanket 0.9.1 - be sure to rerun ember g ember-cli-blanket to get updated blanket.js install

when I run ember test --test-page='tests/index.html?coverage',  I can't find the lcov file anywhere....
here is my blanket-options.js:
/* globals blanket, module */

var options = {
  modulePrefix: 'aeonvera',
  filter: '//.*aeonvera/.*/',
  antifilter: '//.*(tests|template).*/',
  loaderExclusions: [],
  enableCoverage: true,
  cliOptions: {
    reporters: ['json', 'lcov'],
    autostart: true,
    lcovOptions: {
      outputFile: 'lcov.aeonvera.info',
      excludeMissingFiles: true,

      // provide a function to rename es6 modules to a file path
      renamer: function(moduleName) {
        // return a falsy value to skip given module
        if (moduleName === 'unwanted') { return; }

        var expression = /^APP_NAME/;
        return moduleName.replace(expression, 'app') + '.js';
      }
    }
  }
};
if (typeof exports === 'undefined') {
  blanket.options(options);
} else {
  module.exports = options;
}

@jschilli  trying out new version

I get the file now but it has 'undefined' in it and nothing else

Seeing the same thing, the lcov file is generated but contains no information. If I switch from lcov to json I see coverage in the file.

any updates on this? :(

We fixed this in our projects - you can check https://github.com/ciena-frost/ember-frost-button for an example.  I can't recall exactly what was changed, but @sandersky may remember.

thanks a lot @sglanzer
I am guessing the solution is in this:
https://github.com/ciena-frost/ember-frost-button/pull/46/commits/a77ee9e4b8685325bbbfde70fa8d325597436ef8
I will take a look :)
@sandersky , would be great if you could let me know any other insight.
Appreciate your help

Might be related to #112

@jschilli This has been fixed with new version and can be closed

@svkangal thx


feat: added tags : cloud and glare

Quick PR to add the missing tags we talked about recently ! Clouds and glare !
Also in some of the pictures, there is something like "smog" /fine particles.
Do you think this is the right term to use ? Let's add it in this case !

Thanks for the review !
Good point for the illustrations!
Yes, a scrolling section on this topic with examples would be great!
I'll open an issue soon


Update ManualTests.md

Added manual test cases for Graphing mode in Calculator.

Added 4 cases to the hot keys section
Added 1 case to the UI scaling section
Added all manual functional tests for graphing mode, including special cases verifying each Key Graph Feature category
Had to renumber all the cases for hot keys to put Graphing mode cases in an orderly location

@Daniel-Parker - Right now, this is targeted at master, however it should be targeted as an updated to feature/GraphingCalculator.  Since the target can't be reassigned in a PR once opened, I am closing this one down.  Please re-open the PR, targeted at feature/GraphingCalculator.  Thanks!


Log full cell address if Debug

resolves: https://github.com/Siemienik/XToolSet/issues/134

released in xlsx-renderer v2.3.4


[Task] Adding example for hyperparameter tuning

Description

It would be good to add the scope/outline for that example

addressed by https://github.com/NVIDIA-Merlin/models/pull/786


docs: mention where to find the sitemap

So that users know where to locate their sitemaps

Motivation
I didn't see the URL to the sitemap so I took a guess. For the other users who don't know where to locate the sitemap, this is handy.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
I did not made any code changes.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Thanks :)


Android ~4.4 Webplatform Data Subscribe/Publish not working

Any type of subscribe/publish in android 4.4 webplatform (specifically appgyver build 4.0.6) does not work for me.
I am unable to run crosswalk due to crashes/slowness on 4.4 (5+ it's fine).
Anything about 4.0.6 does not allow inAppBrowser windows to open without crashing. Crosswalk and Web Platform.
Not sure where to go from here.

The InAppBrowser crash is something that's being worked on right now.
Is the subscribe/publish working on earlier build versions?


remove unneeded references

The C# solution doesn't seem to reference these packages at all. If they do become needed, they should be replaced with the latest packages referenced here https://docs.microsoft.com/en-us/azure/key-vault/general/client-libraries

cc @ianisms


redshift sequence

use redshift output to set color; not all lights can be supported. enable and disable with "t".

Despite shellcheck-issue for $redshift_options this issue is solved.


When hovering the mouse on Dorrclave, the page gets blank.

When hovering the mouse on Dorrclave, the page gets blank.

Fixed by #272


Update release script and add .gitatrributes for minified assets

A .gitattributes is added to prevent merge conflicts when merging
the javascript.min.js. Also updated the makeRelease script to
generate the assets before releasing to have the latest version without
a merged minified js file which could be damaged.

https://www.pivotaltracker.com/story/show/165081728


Runaway Change Detection with Info Window

When opening an info window without a marker, you get a runaway set of change detection.
Steps to reproduce and a minimal demo of the problem
https://stackblitz.com/edit/agm-change-detection-bug
What steps should we try in your demo to see the problem?

Check out the link above, open up console, right-click to open the info window. I have added the following for visualisation purposes.

  ngDoCheck(){
    console.log('Check was Run')
  }

Current behavior
runaway change detection.
Expected/desired behavior
change detection should stop
angular2 & angular-google-maps version
1.0.0-beta.3

I edited original issue with updated link

note: having parent <agm-marker> does not seem to matter.

This should be re-opened. Still a bug.


Classes, modules, fixing behaviour and scope

At present there are multiple open issues (inc #6652, #2963, and #2964, #2449) which overlap with this one, but don't completely cover it and if my understanding of the facts is badly lacking, or  I haven't found the key item among the issues I'm more than than happy for the powers that be to close this one.
Scenario. I want to implement a Parameter validation attribute class to support functions in a module.
If I write the class in C# and load it with add-type my understanding is that it is compiled to a temporary DLL and loaded and available system wide, whatever method is used to load it .
If I write the class in PowerShell it is not exported by the module if it is placed in the PSM1 file, or a file dot sourced by the PSM1.  Here is the class
using namespace System.Collections
using namespace System.Collections.Generic
using namespace System.Drawing.Printing
using namespace System.Management.Automation
#Validator as PS Class [ValidatePrinterExistsAttribute()]
class ValidatePrinterExistsAttribute : ValidateArgumentsAttribute {
    [void] Validate([object]$Argument, [EngineIntrinsics]$EngineIntrinsics) {
        if(-not ($Argument -in [PrinterSettings]::InstalledPrinters)) {
          Throw [ParameterBindingException]::new("'$Argument' is not a valid printer name.")
        }
    }
}

and here is a related class in C#
Add-type  -ReferencedAssemblies "System.Drawing.Common", "System.Linq","System.Management.Automation"  @"
using System.Drawing.Printing;
using System.Linq;
using System.Management.Automation;
public class ValidPrinterSetGenerator : IValidateSetValuesGenerator
{
    public string[] GetValidValues()
    {
        return PrinterSettings.InstalledPrinters.Cast<string>().ToArray();
    }
}
"@  -WarningAction SilentlyContinue -CompilerOptions "/nowarn:CS1701,CS1702"

(I found I had to use /NoWarn to supress issues which suggested some .NET Core parts were mismatched) .
After loading these via the PSM file  the C# class is available and   [ValidPrinterSetGenerator]::new().GetValidValues() returns a the list of printers, but  a function in the module which uses the  PS class for one of its parameters gives this error
#501 PS7 C:\Users\mcp\Documents\PowerShell\classTest>.\load2.ps1
InvalidOperation: C:\Users\mcp\Documents\PowerShell\classTest\classtest.psm1:19:9
Line |
  19 |          [ValidatePrinterExistsAttribute()]
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find the type for custom attribute 'ValidatePrinterExistsAttribute'. Make sure that the assembly that contains this type is loaded.

As I understand it the class might be available to things wholly within the module, but the validation attribute needs be accessible outside the module's scope. (Parameter stuff being done outside the module)
The is a workaround: add the classes to the ScriptsToProcess in the module manifest.
There is a secondary issue even with workaround. If the module is loaded usingimport-modulein a script the validation works but the PS class is invisible thereafter unless the script is dot-sourced. However if the script uses using module  or #requires -module the class is available without dot-sourcing.  I think it is broadly OK to say load modules which require at least V5 because they have classes with the using  syntax.
Given the length of time some of issues have been open, I'm wondering if the intent is to

Make import and using/requires consistent  and / or
Provide a way to export PS classes from a module, rather than pre-loading them and/or
Provided some more detailed documentation of the status quo ?

Add-Type uses "Microsoft.PowerShell.Commands.AddType.AutoGeneratedTypes" as namespace. So "[Microsoft.PowerShell.Commands.AddType.AutoGeneratedTypes.ValidatePrinterExistsAttribute()]" could work.

Add-Type uses "Microsoft.PowerShell.Commands.AddType.AutoGeneratedTypes" as namespace. So "[Microsoft.PowerShell.Commands.AddType.AutoGeneratedTypes.ValidatePrinterExistsAttribute()]" could work.

Sadly, not
>[Microsoft.PowerShell.Commands.AddType.AutoGeneratedTypes.ValidatePrinterExistsAttribute]
InvalidOperation: Unable to find type [Microsoft.PowerShell.Commands.AddType.AutoGeneratedTypes.ValidatePrinterExistsAttribute].

But it is present to do the validation
>test2 fax
fax
>test2 foobar
test2: Cannot validate argument on parameter 'name'. 'foobar' is not a valid printer name.

@jhoneill Please add more strong repo steps for both cases including the module sources - it would safe time of reviewers. Thanks!

@jhoneill Please add more strong repo steps for both cases including the module sources - it would safe time of reviewers. Thanks!

Simple demo. There are comments in the PSM1 / PSD1 to show which lines to enable to change where the loading happens.
You can import the module and run test2 <> or try doing that with load.ps1 or by dot sourcing load.ps1 and comparing the results.
As well as test2.ps1  You can compare the c# class behaviour e.g.
[ValidPrinterSetGenerator]::new().GetValidValues()
and ps class behaviour e.g.
[ValidatePrinterExistsAttribute]
classtest.zip
Thanks !!

Make import and using/requires consistent  and / or

They've mentioned a few times that requiring using was a design decision because classes are largely a parse time concept, so a parse time import was needed.

Provide a way to export PS classes from a module, rather than pre-loading them and/or

If the classes are defined within the psm1, using module will import them.  This is currently the only "proper" way to define classes to be exported.

This looks like dup #1762.

Make import and using/requires consistent  and / or

They've mentioned a few times that requiring using was a design decision because classes are largely a parse time concept, so a parse time import was needed.

Provide a way to export PS classes from a module, rather than pre-loading them and/or

If the classes are defined within the psm1, using module will import them. This is currently the only "proper" way to define classes to be exported.

Well, partly. If the classes are in the PSM1 file, instead of Dot-Sourced into I can have this as my PSM
using namespace System.Drawing.Printing
using namespace System.Management.Automation
using namespace System.Management.Automation.Language
class ValidatePrinterExistsAttribute : ValidateArgumentsAttribute {
    [void] Validate([object]$Argument, [EngineIntrinsics]$EngineIntrinsics) {
        if(-not ($Argument -in [PrinterSettings]::InstalledPrinters)) {
          Throw [ParameterBindingException]::new("'$Argument' is not a valid printer name.")
        }
    }
}

function test2 {
param (
    [ValidatePrinterExistsAttribute()]
    $name
)
$name
}

Then
ipmo .\classtest.psm1 ; test2 fax ; test2 foo; [ValidatePrinterExistsAttribute]
validates but can't find the attribute class and as you say
using module .\classtest.psm1
test2 fax ; test2 foo; [ValidatePrinterExistsAttribute]
Validates and does find the attribute class
OK. So 2 learnings

Put the classes directly into the PSM . Dot Sourcing them in does not work.  Those people who like a minimal PSM1 file will have to learn to cope.
Using loads more than import-module does (and I thinkrequires`  is like using, not like import

Pasting the using ... & test... lines into load2.ps1 and running that ...  validates and finds the class but after loading
>test2 foo
test2: Cannot validate argument on parameter 'name'. 'foo' is not a valid printer name.
> [ValidatePrinterExistsAttribute]
InvalidOperation: Unable to find type [ValidatePrinterExistsAttribute].

The  module's function is there and calls the class, but the class isn't visible outside the script which loaded the module... unless I dot source load2.ps1.  So Dot sourcing is required in one place, and fails in another.

This looks like dup #1762.
Overlaps but not exactly a dup :-)
The difference by adding classes to the PSM that @SeeminglyScience  called out is new (and very useful) information.

This looks like dup #1762.
Overlaps but not exactly a dup :-)
The difference by adding classes to the PSM that @SeeminglyScience  called out is new (and very useful) information.
[Might have to blog that]

Nah it's the same thing I think.  My guess is that the bug is related to when the resolution of the ITypeName.  It applies to most instances of trying to use an attribute that is loaded (even with Add-Type -TypeDef, even with Add-Type -Path to a dll).  Type resolution for attributes outside of the built in ones is sorta borked.

Can confirm, I've seen a lot of funkiness when using custom attributes. Sometimes it's fine to call them NameAttribute and PS picks it up fine with [Name()], other times it refuses to find it and you've gotta just call the class itself [Name].
Among many other oddities that are frustrating as all heck to reliably reproduce.

I set "answered" since we have a workaroung and cross-reference with meta-issue #6652.

@vexx32  yes saw your comments on 1762 , and had the same with where you can omit 'attribute' and next thing you find you can't  - and agree it's full of oddities.
@iSazonov  Makes sense :-)

@vexx32 It might amused you that after this a penny dropped that one of the problems I was getting might be because I'd removed "attribute" ... just put it back in into close to 100  places where I'd replaced a dynamic parameters with static ones and a validate class and now things behave (as above) without . sourcing

If the classes are defined within the psm1, using module will import them. This is currently the only "proper" way to define classes to be exported.

From what I can tell, this is apparently still true. But this doesn't make any sense to me. I download a psm1 file from my repo as pure text over TLS, and I convert  it to a dynamic module via New-Module and then import it into memory with Import-Module. Now I can access the functions as I wish, and when I am done, I have nothing to cleanup on the file system. Why cannot I do the same thing with a class? I should be able to download the class file, whether in a psm1 or ps1 file, as pure text over TLS and then import it easily into memory for instantiation. Logically, it makes sense to just import it with a module OR import it with another such command. For example:
$fooClass = New-Class -ScriptBlock $fooClassAsText
$fooClass | Import-Class
$myFooObj = [Foo]::new()  #If the class import was unsuccessful, I'll get an exception here telling me that Foo doesn't exist


Place code from difference deps into their own folders

We do not support deps (for now), but we can still work on this. Standard library compiled code should go into an elixir folder, while app code should either use the current or given app name, or default to just app
├── app
│   └── Elixir.App.js
└── elixir
    ├── Elixir.js
    ├── Elixir.ElixirScript.Agent.js
    ├── Elixir.ElixirScript.Atom.js
    ├── Elixir.ElixirScript.Base.js
    ├── Elixir.ElixirScript.Bitwise.js
    ├── ...

es6 import paths will need to be updated as well

Done with #201


[editor] making absolute area for drag and drop of sql file in editor

What changes were proposed in this pull request?

making absolute area for drag and drop of SQL file in the editor

PS: nit: description "making absolute area for drag and drop of SQL file in the editor" not really adding more info, more interesting would have been what was the problem and why switching to absolute fixed it

When we drag and drop a file to left assist (when editor is opened) then it always goes inside the editor. But we want file to drop in left assist. By making the area absolute for editor, files goes into editor when we drop into the editor only.


Fix pipe use in new composer constraint

Single pipes aren't defined in the composer spec,
and cause issues when installing other dependencies like silverstripe/behat-extension.
See https://getcomposer.org/doc/articles/versions.md#range

Closing in favour of https://github.com/silverstripe/silverstripe-framework/pull/6590


Fix double inset issue

Fix issue where bottom contentInset is added more than once when user taps on first responder view again

 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.

Note: This issue was seen in iOS 11.4 simulators.


Update regex.yaml

Type: Missing
Summary:
regex 2020.7.14
Details:
Add MIT
Resolution:
License Url:
https://github.com/gitgik/regex/blob/master/LICENSE
Description:
License is included in GitHub Repo

@dciborow The pypi you are curating has a different source location. Are you wanting to curate this package: https://pypi.org/project/regex/?

My internal system thinks this is what i need to update... let me debug internally.

@dciborow I'm going to go ahead and close this PR. This pypi version of regex has been curated as Apache. If you need to curate a different component, please resubmit.


Gentler text-protection scrim

Using fancy new things like blend-modes on the gradient and CSS filters to help readability while making the scrim subtler.
Scrim falls back to just use rgba gradient for browsers that don't support this.

manually pulled in changes with 08f6a1c284bb132cdbb15c8b83f0c909f1162c90 in release/2.0.0


Automatic update not working

Steps to reproduce
open app
Expected behavior
App rafresh and update library
Actual behavior
Nothing i have to refresh manually my library
Crash logs
No response
Mihon version
0.16.3
Android version
Android 13
Device
S20FE 5G
Other details
All my library is on a folder name "update".
Automatic update is enable and set to 12 hours.
In smart update all is unchecked
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.3.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Holy shit, please enable at least the second and third parameter for smart updates and change the update frequency to at least one day.

I'll try that and see tomorrow if it updates automatically.

Nope its still not working chapter not update automatically.

Kind of in the same boat, have issue where update tab won't update anything but when refreshing the manga when on the overview page it works, don't know if this should be a new thread

Kind of in the same boat, have issue where update tab won't update anything but when refreshing the manga when on the overview page it works, don't know if this should be a new thread

https://mihon.app/docs/faq/updates/smart

Since they un-necro'd the issue, going to do a summary & close out this issue as user-error/not understanding app features.
For the author's issue, since you adjusted Smart Updates & it still wasn't updating, sounds like either you were having source-related issues (dead/down source), extension-related issues (check with whoever maintains your extensions instead for these), or internet-related issues (maybe blocked site, only check when WiFi, etc).
For anyone else, this sounds like you don't understand how Smart Updates work, so here's a semi-quick summary.

Smart Updates, see link above from Xori71, manages what gets checked during Automatic Updates & Library/Category Updates (including the refresh button in the Updates page)
This reduces unnecessary requests sent to sources & spreads out your updates over per-series predicted release patterns.
You can view the status of prediction/when it'll check for a new update if you go to the manga entry, and tap on the hourglass icon.

Make sure Automatic Updates is enabled in your Mihon Settings > Library page, then leave it alone, let it update by itself & adapt over time.
Otherwise, if there's something you really want to read, and know it has a new chapter, then refresh the entry yourself.


Running "top" command inside docker container

It may not be an issue with Docker, though, I'm confused about the output of "top" command running inside a docker container -
Tasks:   2 total,   1 running,   1 sleeping,   0 stopped,   0 zombie
%Cpu(s): 10.6 us,  5.6 sy,  0.0 ni, 83.8 id,  0.0 wa,  0.0 hi,  0.0 si,  0.0 st
KiB Mem:   2040836 total,  1905172 used,   135664 free,   126276 buffers
KiB Swap:   522236 total,       24 used,   522212 free.  1195708 cached Mem

  PID USER      PR  NI    VIRT    RES    SHR S %CPU %MEM     TIME+ COMMAND
    1 root      20   0   18180   3272   2784 S  0.0  0.2   0:00.21 bash
   15 root      20   0   19880   2400   2044 R  0.0  0.1   0:00.80 top

The %Cpu(s) does not match with the total %CPU (all are 0's) being used by each process inside the container.
I would like to know what is this %Cpu(s) at the 2nd line of output of "top"command.
Thanks & Regards,
Sourav

Sorry... Issue not with docker-registry. Regret the inconvenience.


Merge Hebrew Translation

Is your feature request related to a problem? Please describe.
Users should be able to switch to Hebrew Translation.
Please take a look and merge when possible.
https://github.com/JoinSEEDS/wallet_translation/tree/Hebrew_he_il

This person contacted me
What's happening with this @JGDoh @gguij004

This person contacted me
What's happening with this @JGDoh @gguij004

Hi Nik, we talk about this one in one of our team meetings, when I go to the files the translation are in Spanish for some reason.

What happens with those translations I don't see yet the Hebrew lang in the app?????

The hebrew translations were done, but there was some issue with branches and such - they are in Github though
The problem is they are outdated by now and a lot of text has changed.
Maybe best for @buildc0de to connect with the guys who did the translation and get them to use weblate?

Bela Litman - one of the translators - sent me this
https://github.com/andarianlb/wallet_translation/tree/hebrew_translation
That's the recovered translation

Made a note of this. I will circle back to this ticket once we have a working translation process 🤓

@buildc0de I think we can close now this ticket since we already start using weblate

Yup, good to close.


HTMLField() from tinymce.models : font color

Is it possible to change the font color using the HTMLField() from tinymce.models?
I could not find this information in the docs ?

Added this
TINYMCE_DEFAULT_CONFIG = {
    'format': { 'title': 'Format',
    'items': 'bold italic underline strikethrough superscript subscript codeformat | formats blockformats fontformats fontsizes align | forecolor backcolor | removeformat' }
}

in my django site settings.py provides the following UI which gives me the possibility I was looking for :

Not perfect since not integrated nicely in the toolbar. If someone has the perfect solution...


Resize Post page

Hi Cotes Chung,
I like the Jekyll Theme you created but I would like to have the post pages a bit wider.

Is this something I can easily change myself?
Thanks for your help.
Stefan

When it comes to modifying post width, you can use DevTool to look for CSS variables that need to be modified, which are defined in the file main.scss and post.scss of assets/css/, but please be careful to modify them as this will affect the layout of adjacent elements.


Source Files Missing from Generated Eclipse Project

It seems this is due to the fact that for yotta, the CMake "source" directory is technically the build directory (as this is where yotta generates the top-level CMakeLists.txt file).
http://forums.mbed.com/t/support-for-eclipse-ide/861/3?u=jamcro01

+1


Add coverage reporting.

Since this is a native module, coverage reports will probably be hard. There should be a way though.

Looked into it some, looks like with Mocha this could be possible. So I'd probably have to rewrite the tests.

Looked into it some more, looks like it's not possible. :(


cannot change point size when rviz2 is auto-sized to fit screen

similar to this discussion over on rviz for Noetic....when I run rviz2 on my machine it works great until I auto-expand it to fit the screen (either half the screen or full screen). once this is done I cannot change the size of the pointcloud topic I am displaying.
Full screen auto-sized (cannot change size of points):

non-auto sized (can change size of points):

Tested on:

Intel x86
Ubuntu 20.04
Foxy built from source

[INFO] [1591020425.764452318] [rviz2]: Stereo is NOT SUPPORTED
[INFO] [1591020425.764746767] [rviz2]: OpenGl version: 4.6 (GLSL 4.6)
[INFO] [1591020425.817653351] [rviz2]: Stereo is NOT SUPPORTED

Thanks, we'll keep an eye on that other issue (https://github.com/ros-visualization/rviz/issues/1508) and investigate here if needed, but I assume it's the same issue (likely with mesa).

Since it came up in the other thread: RViz 2 uses Ogre 1.12.1 out of the box, so I guess recent Ogre won't help. If you want to give it a try, I could update to 1.12.6 and push a branch for you to try out (I have neither upgraded to 20.04 nor do I have an Intel x86 at the moment).

more than happy to help with testing

Based on the conversation in https://github.com/ros-visualization/rviz/issues/1508, this seems to have been a bug in mesa.  And it now seems to be fixed.  Given that, I'm going to close this out.  But feel free to reopen if you still see this issue.


Can I use my custom ID3D11Device pointer?

Hello,
Can I use my custom ID3D11Device pointer?  I want to use a exsited ID3D11Device pointer to replace the default device pointer in bgfx. Is it possible? And how to do?
Thanks.

Yes, see bgfx::setPlatformData: https://bkaradzic.github.io/bgfx/bgfx.html#_CPPv2N4bgfx15setPlatformDataERK12PlatformData
https://bkaradzic.github.io/bgfx/bgfx.html#_CPPv2N4bgfx12PlatformData7contextE
Make sure you understand bgfx internal threading before using this:
https://bkaradzic.github.io/bgfx/internals.html

OK, thank you bkaradzic .
It is just what I need.


Documentation: Fix links that weren't updated after last architecture change.

/lgtm

@chancez I saw that Documentation/reportqueries.md referenced string as an input type instead of Presto's varchar. Was this a correct change to make?

@timflannagan1 hmm it should use varchar probably. Whoops.


Document recoverable versus nonrecoverable contract errors

What
Document recoverable versus nonrecoverable contract errors to assist contract developers. We can provide some guidance of which scenarios one would be beneficial than the other.
This can possibly go in Errors page

We can provide some guidance of which scenarios one would be beneficial than the other.

Contract developers don't have control over which errors are not recoverable. Only the resource errors and internal errors are not recoverable. I don't think this information belongs to the example above. I agree that it would be nice to document somewhere, but I'm not sure where. This seems pretty low-level and I think most of the users shouldn't encounter this at all. But it is indeed pretty confusing for those who might encounter it. Maybe we need something like 'common gotchas' page?


Slight clarification in the installation docs.

Hi, @skoruba !
I would like to propose a change in the project setup documentation.
It doesn't mention changing the directory to src/ before doing yarn install and so on, which might be slightly misleading to newcomers.
BTW, Your client is awesome!

Hi @Icebreaker454 - thank you. :)


remove token refresh process for keycloak admin client

Description
authentication between our app and keycloak admin service is a client_credentials grant and doesn't require a refresh token.
I'm not sure but I think we remove that requirement like this.
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[X] I have read the CONTRIBUTING doc
[X] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

PR Deployed at: https://comfort-dev.apps.silver.devops.gov.bc.ca/pr-162

not currently required as we can enable refresh tokens for client_credential grants in the console settings.
we should look at doing this soon to improve security.


test: fix example-recording failure reporting status

closes #1072

Changes
This PR corrects an issue where the example-recording.yml workflow passes back a success status to GitHub in the parallel job whether or not the Cypress tests from the project examples/recording pass or fail.
The code example in README > Outputs is also corrected.
Verification

Modify examples/recording/cypress/e2e/spec-a.cy.js to introduce a deliberate failure condition by adding the following to the test spec:

assert.fail('This is a deliberate error')

Manually trigger a run of workflows/example-recording.yml by clicking on "Run workflow".

After the workflow run has completed, examine the logs and confirm that one of the jobs parallel (1) or parallel (2) is marked with a red cross to indicate failure.

Confirm that the step "Print Cypress Cloud URL" was executed for both parallel (1) and parallel (2), and that it reports the correct status of either "failure" or "success". One job should succeed, the other should fail.

Revert the deliberate error and push the correction as a new commit to the repo.

Run the workflow again via "Run workflow". (Don't use "Re-run jobs" as this will run against the original commit, not the corrected one.)

Confirm that all jobs pass, that the step "Print Cypress Cloud URL" was executed in the parallel jobs and that it reports "success".

This PR is not logged to the Triage board

@jennifer-shehane
Since you added me as an "outside collaborator" in this repo, my issues and PRs are not added to the Triage board anymore.
Could you please review and merge this PR?
The PR corrects a long-standing bug which I recently noticed.

@MikeMcC399 Hmm, true. I'll have to look into that.


Restore "Conan Package Management" menu

there is "Conan Package Management" menu, but it currently just displays and empty Windows.
I think it should do something useful (e.g. show HTML of conan graph, for instance).

@solvingj do you remember what was planned for this?

I'm not sure, I think this was intended to be the top level menu. Currently, we have a top level called "Conan".
If it's not too much trouble, I would prefer it be under:
Tools -> Conan Package Manager
Just like the Nuget Package Manager toolbar is today.


Add option to set --no-sandbox in puppeteer

Not sure if this really deserves to be merged in, but this is a quick fix to work-around this problem: https://github.com/GoogleChrome/puppeteer/issues/290
In my case, I'm trying to use cucumber-mink with one of the CircleCI pre-built images, and running into this problem.

On further testing I see that the image I'm using also has upstream issues with Puppeteer, so for now I'm using a branch of my fork to hardcode --no-sandbox and downgrade Puppeteer. I feel like this PR should not need to be merged, so I'll close this.


Add support to run Docker Compose v2

We need the e2e-action to be able support running Docker Compose v2.  The github runner image supports both v1 and v2 https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#tools.
Our current situation is that we are using docker-compose instead of docker compose which causes that Docker Compose v1 is used. In order to fully update to run the v2, we need to perform the following tasks:

[ ] Update the e2e-action to use docker compose instead of docker-compose e.g. https://github.com/newrelic/newrelic-integration-e2e-action/blob/main/newrelic-integration-e2e/pkg/dockercompose/docker.go#L10.
[ ] Update the e2e specs to use docker compose instead of docker-compose. We also need to check at the time of doing this task on each repository whether the docker-compose command it called within the action itself. Add the repository to the list as the task is finished:

[ ] ...

Closing it as it will be tracked in internally in JIRA.


Change the map view stats layout in desktop & mobile

Push the stats on top like this.

Done in
https://github.com/covid19india/covid19india-react/pull/92#issue-393477338
Commit: 7e5637a36d8748f0da49feb6a86f4243db28f0d1

@sirajalam049 Cool. When it will be pushed?

IDK, I have made PR, and also posted in the group. It might be done within a day I guess.

@sirajalam049 can you rebase this over master?

I think it would make sense to increase the width of the map along with this.

Fixed!


✨Add low-level context operations right onto the context.

Motivation
Reading context is doable, writing context is extremely awkward. This was by design, but it is increasingly becoming a stumbling point. I'm finding that we have to lug around snippets of code to act as "context helpers" to make the task less arduous.
This is because when creating effects like loggers, template outlets, dispatch channels, etc... it is often necessary to pair some state with the operations that run on that state. Among other things, this is the general pattern for implementing algebraic effect handlers which is something we're going to be doing more and more of going forward.
As such, we need simple, type-safe, yet low-level wrappers around context.
Approach
In the same way we went with the simplest possible thing that could work with the original context reading (just making it an opaque operation), I'm proposing a set of low-level helpers attached directly to the context that let you manipulate it for the current scope. These are get(), set(), and expect(). Currently, reading from a context behaves like expect() where an exception is raised if the context is not present. However, sometimes you may want to check if the context is set or not, and so you can use get() which will return undefined if not present.
/**
 * `Context`` defines a value which is in effect for a given scope which is an
 * (action, resource, call, or spawn)
 */
export interface Context<T> extends Operation<T> {
  /**
   * Set the value of the Context for the current scope.
   */
  set(value: T): Operation<T>;

  /**
   * Get the value of the Context from the current scope. If it has not been
   * set, and there is no default value, then this will return `undefined`.
   */
  get(): Operation<T | undefined>;

  /**
   * Gets the value of the Context from the current scope. If it has not been
   * set, and there is no default value, then raise a missing context error.
   */
  expect(): Operation<T>;
}

TODOs and Open Questions
Other options considered, but which I think we should hold off on for now (but add them if they become useful):
with() that calls an operation with a context value.
yield* Context.with(value, function*() {
  yield* Context.get() //=> returns value;
});

yield* Context.get() //=> undefined

However, I found that in general, it was just as easy to use Context.set() and it had less rightward drift because contexts only set for the current scope, they will disappear after the current operation.
consider deprecating bare operation context, and instead force you to use expect().
I think this conveys intent much better.
before:
import { Logger } from "./logger.ts";

export function* log(message) {
  let logger = yield* Logger;
  logger.write(message);
}

after:
import { Logger } from "./logger.ts";

export function* log(message) {
  let logger = yield* Logger.expect();
  logger.write(message);
}

Overall this feels like a nice improvement.  I really liked the ergonomics of being able to simply yield to a context.  Yielding a function on the context is less than ideal.  Otherwise, I like the changes and no blockers from me

@neurosnap Ok, I'll mark as ready for review. Are you saying you're in favor of not deprecating the current behavior in favor of Context.expect()? In that case, should we have Context.expect() at all?


customTypeMapping['JSON'] - Not Working properly in Apollo Graphql Android

My build.gradle file is
customTypeMapping['JSON'] = "kotlin.Any"
My graphql file is
mutation UpdatePropertyInfo(
    $id: ID
    $extra_beds: [JSON])
{
    update_property_info(
        id: $id
        extra_beds: $extra_beds
}

Auto generated file graphql model file
data class UpdatePropertyInfoMutation(
  val id: Input<String> = Input.absent(),
  val extra_beds: Input<List<Any?>> = Input.absent(),
) : Mutation<UpdatePropertyInfoMutation.Data, UpdatePropertyInfoMutation.Data, Operation.Variables>
    {}

When I pass a JSON list the format is like this
[["BedsModel(beds_count=0, bedroom_bedtype=[Bedroom_bedtype(_id=5ec23a7221866f101a184868, name=Sofa bed,
status=active, updated_at=0, created_at=0, __v=0, id=5ec23a7221866f101a184868, count=0),
Bedroom_bedtype(_id=5ec23a7e21866f101a184869, name=Couch, status=active, updated_at=0, created_at=0, __v=0,
id=5ec23a7e21866f101a184869, count=0), Bedroom_bedtype(_id=5ec23a8921866f101a18486a, name=Floor mattress,
status=active, updated_at=0, created_at=0, __v=0, id=5ec23a8921866f101a18486a, count=0)])"]

But i need format like this
[[{"bedroom_bedtype":[{"__v":0,"_id":"5ec23a7221866f101a184868","count":0,"created_at":0,"id":"5ec23a7221866f101a184868","name":"Sofa
bed","status":"active","updated_at":0},
{"__v":0,"_id":"5ec23a7e21866f101a184869","count":0,"created_at":0,"id":"5ec23a7e21866f101a184869","name":"Couch",
"status":"active","updated_at":0},
{"__v":0,"_id":"5ec23a8921866f101a18486a","count":0,"created_at":0,"id":"5ec23a8921866f101a18486a","name":
"Floor mattress","status":"active","updated_at":0}],"beds_count":0}]]

please help me to get this format

Thanks for reaching out! Can you share the code that creates the extra_beds input parameter? Is it a Input<List<Map<*,*>>> or some other json object ?
Also, how do you log it? The BedsModel especially looks weird, I can't see that anywhere in your mutation?

Is it ok to share the code in your email or in this chat?
Thanks

Whichever you prefer. You can write at martin at apollographql dot com if you prefer to not send this publicly

Hi Martin,
From your guidelines,I can be update some other steps,Its works now.
override fun encode(value: Any): CustomTypeValue<*> {
           when (value) {
               is MutableList<*> -> {
                   val str: String = Gson().toJson(value)
                   val type: Type = object : TypeToken<List<Map1<String, Any>>>() {}.type
                   val myMap: List<Map1<String, Any>> = Gson().fromJson(str, type)
                   return CustomTypeValue.GraphQLJsonList(myMap)
               }
               else -> {
                   return CustomTypeValue.fromRawValue(value)
               }
           }
       }

Thanks


Different outputs for Firefox and Chrome pupeeters

When I use queue multiple URLs for Firefox, I see them open in multiple tabs in same window.
The same test for Chromium gives multiple windows, one with each URL.
I agree that this question would be out of scope of this project. However shouldn't both outputs be consistent?

Can you show a minimal code sample?
But I think that, yes, this question is out of scope for the project and better belongs on the puppeteer repository itself. Keep in mind that puppeteer-firefox is still experimental AFAIK.


V3 of AzureRmWebAppDeployment does not allow you to deploy to a slot for a API app.

The deploy to slot option defined in https://github.com/Microsoft/vsts-tasks/blob/fe3532c4dee9c9e0a882d6d621055777366e9dae/Tasks/AzureRmWebAppDeployment/task.json#L81-L89
is only visible when WebAppKind = app || WebAppKind = functionapp || WebAppKind = applinux, if you have a WebAppKind = api the slots option will not be visible.
This does not appear to be intended behavior as this behavior allows for slots in the v2 version of the task.

Is there any progress on this issue? Having to downgrade all my releases is becoming tedious.

@grantwar, currently we are working on this issue.
As a workaround, Can you please replace the App Name with some Release variable (say $(appname)) and set the release variable to your App Service name?
In this way, WebAppKind is changed to 'app' and slot option will be visible.

Fix was checked-in. This will be deployed to production in next 3-4 weeks


Try to fix memory leakage.

In my server program, the following error will appear.
==15801== 49 bytes in 2 blocks are definitely lost in loss record 1 of 1
==15801==    at 0x483B7F3: malloc (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==15801==    by 0x167DE7: uv__malloc (uv-common.c:77)
==15801==    by 0x167D16: uv__strdup (uv-common.c:57)
==15801==    by 0x16810D: uv__unknown_err_code (uv-common.c:170)
==15801==    by 0x16A286: uv_strerror (uv-common.c:216)
==15801==    by 0x13D25C: tunnel_dump_error_info (tunnel.c:694)

If this error occurs very quickly, the server program will crash soon for OFM.
Please accept my correction.
Because libuv only runs in a single thread, there is NO need to consider the issue of multi-thread synchronization.

I agree the memory leak is unfortunate but this...

Because libuv only runs in a single thread

...unfortunately isn't a valid assumption. A libuv event loop is single-threaded but programs can have multiple event loops.

Is there a better solution?

I don't think so. I put it in only because the alternatives felt worse.

Well, I hope you will mark this function abandoned in the documentation and let users only use uv_strerror_r.
I think it's better.

I agree it's good to call out this sharp edge in the documentation, assuming it's not already mentioned. Want to send a PR for that?

I don't know how to do it. please do it by yourself. please.

Do we want to eliminate uv_strerror in v2?

Yes, good idea.


Testing to see if a simple rerender works for OSX arm

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #53.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@conda-forge-admin please rerender


Dynamical type (string) union

Is it possible to dynamically do string intersection as shown below?
export class Foo<K extends string> {
    addKey<L extends string>(key: L): Foo<K | L> {
        return this;
    }
    getKey(key: K) {

    }
}

const x = new Foo().addKey('a').addKey('b');

// need to constraint this to 'a' | 'b'
x.getKey('')

It works, but this is needed:
export class Foo<K extends string = never> {


Make SDL3Clipboard support images

This PR supersedes https://github.com/ppy/osu-framework/pull/6151 which aimed at adding support for copying and pasting images from the clipboard on Linux, but had the wrong approach of calling into an external executable (xclip for X11).
In the mean time I had actually started a native clipboard library in Rust for applications that would support any mime types and be given arbitrary data (the X11 implementation was somewhat finished, but not Wayland), but since osu!framework now uses SDL3, I can just use SDL_GetClipboardData and SDL_SetClipboardData as suggested back then by @Susko3.
This is a first attempt at implementing this, and on my Linux machine, the visual tests for copying images are all green! :D
Edit: actually doesn't work every time! D: I'll have to really refactor this code and make sure I understand what's going on.
Edit 2: bug identified, it seems to come from the given image itself...
That said, I really don't know what to think of this code, [...] I think the code looks ok now :)

There's also the caveat that I only implemented a way to copy PNGs. Wouldn't it also need to be able to copy JPGs depending on some setting? I know osu! lets you choose the image format for screenshot savings.

What you have looks pretty standard for the new age of unmanaged->managed callbacks. That being said, this PR now competes with https://github.com/ppy/osu-framework/pull/6223 and we should probably make sure that this implementation is going to easily lend itself to other mime types in the future.

I debugged a lot today (both with a debugger and with prints) and concluded that pngData was fine as a static due to how the API works. It only calls for a cleanup once if we're requesting for an image again.

It seems that sometimes the test gives a completely empty screenshot. I was worried that my implementation had some weird undefined behavior thing, but it seems like that bug is happening elsewhere...?

It seems I'm missing something related to the unmanaged call convention

I'm testing this on windows (by commenting out the WindowsClipboard override) and I can't get images from osu!framework into external programs (and vice versa). The clipboard tests pass fine, but applications complain about there being no image in the clipboard. And trying to paste images into framework doesn't work: either I get the stale image (last one copied from framework) or no image at all (even when copying a sample PNG from the internet).

Does copying images to/from framework into external programs work on Linux?

It does work on Linux (at least on my machine on X11), as long as there is a clipboard manager present. If there is no clipboard manager, the selection will disappear once the application closes, which is expected behavior.
I don't know why it doesn't work on Windows though. :/

Works perfectly fine on windows if the mime type is changed to image/bmp and the image is saved with .SaveAsBmp(). Do bitmaps work on Linux too?
I wonder if it works on macOS (which currently uses MacOSClipboard) and iOS (which uses SDL3Clipboard).

Works perfectly fine on windows if the mime type is changed to image/bmp and the image is saved with .SaveAsBmp(). Do bitmaps work on Linux too?
I wonder if it works on macOS (which currently uses MacOSClipboard) and iOS (which uses SDL3Clipboard).

At least on Wayland, the mime type also has to match. If I e.g. copy a PNG from my local storage, the current code works fine. If I use a Paint-like program to scribble around, Ctrl-A and Ctrl-C, then it's also saved as an image/bmp and needs that exact mime type to be retrieved from SDL.
The obvious solution would be to just have a list of image mimetypes that are tried one after another. FWIW, SDL itself also does this for the built-in GetClipboardText function.

I'll try to implement a multi-mime-types solution.

We should probably ask ImageSharp about it's supported decoder formats, see https://github.com/SixLabors/ImageSharp/discussions/1353 (but with IImageFormat.MimeTypes).
This is what I get on windows:
image/png
image/apng
image/jpeg
image/pjpeg
image/gif
image/bmp
image/x-windows-bmp
image/x-portable-pixmap
image/x-portable-anymap
image/x-tga
image/x-targa
image/tiff
image/tiff-fx
image/webp
image/qoi
image/x-qoi
image/vnd.qoi

Works perfectly fine on windows if the mime type is changed to image/bmp and the image is saved with .SaveAsBmp().

@Susko3 Can you tell me if this works on Windows now?

In any case, I can confirm that it works on MacOS.

Thanks for the PR,


"Brăzești" is listed in the drop-down menu

Description:
"Brăzești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Brăzești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Brăzești, Alba (Baia de Arieș)".


Possible to load old value in the content?

When there is error in validation, I want to show the old value inside the trix content. Is there any way to do this?

Trying to do the same.

i made a pull request https://github.com/amaelftah/laravel-trix/pull/48

closed in #48


Do not fork repositories on programming exercise import

Describe the bug
Programming exercises can't be imported because the maximum depth of the fork hierarchy exceeds 5.
Exemplary error message:
"message":"GBS2021NTHREADS/gbs2021nthreads-exercise cannot be forked. For performance reasons, fork hierarchies are limited to a maximum depth of 5 beneath a given root repository. Please create your fork from a repository higher up in the hierarchy, such as GBS2021NPROZESSE/gbs2021nprozesse-exercise."
The error and possible solutions are documented in the Bitbucket documentation.
As a workaround, programming exercises higher in the fork hierarchy should be imported.
To prevent this error in the future, we should stop using forks to import programming exercises with Bitbucket.
To Reproduce

Import a programming exercise
Import the imported programming exercise
Repeat step 1. and 2. five times to produce the error.

It seems like the api only supports creating forks instead of a clone. So we might need to find a way to only fork from the root?


Fix #408: Response.age is semantically a timedelta, not a datetime

First time I've tried to contribute to Werkzeug; criticism is welcome.
Fixes #408.

FWIW, I've decided to delay this to 1.0, because we can cleanly break the API in that release. The patch itself seems to be fine and can be applied as-is.

@untitaker can we re-schedule this to 0.13 since it's been 2 years since 1.0 was added? I think this is ok for a feature release.

Sure, but technically feature releases don't allow breakage at all. But 0.13 is not a feature release (pre-1.0 different rules apply in semver)
On 12 April 2017 17:12:03 CEST, David Lord<EMAIL_ADDRESS>wrote:

@untitaker can we re-schedule this to 0.13 since it's been 2 years
since 1.0 was added? I think this is ok for a feature release.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/werkzeug/pull/414#issuecomment-293609297

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Ah, I didn't know we were following semver, I just figured we were ok making big changes in 0.x releases. I'll merge this into master then.

Pre-1.0 we would comply with semver anyway because semver allows for a lot at this stage.
On 12 April 2017 17:14:53 CEST, David Lord<EMAIL_ADDRESS>wrote:

Ah, I didn't know we were following semver, I just figured we were ok
making big changes in 0.x releases. I'll merge this into master then.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/pallets/werkzeug/pull/414#issuecomment-293610281

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

#1104 rebases this against current master so that it can be cleanly merged.


Added Preview widow accessible from Settings for changing game art po…

Added Preview widow accessible from Settings for changing game art positions and layers easier.
I added some changes to GameMediaGeneration to be able to create previews without being connected to a device.
Be free to ask me to delete these changes if they are not wanted.

https://github.com/prosthetichead/GarlicPress/assets/8648447/e5761c0b-4fcc-4dfb-8588-8e6883286eee

This is great! thanks so much for the contribute! I'm working on a update at the moment to the skinning part that I want to release soon so I will get this added to main and bring it into my branch also. Thanks!

@prosthetichead
Thanks!
I'm working on a new medialayer editor also see video below.
It's not done yet but it's in a this branch on my fork https://github.com/00anwi/GarlicPress/tree/EnhancedPreview
It's in blazor hybrid so I had to change alot to get it to work.
I don't have my RG35XX yet so I can't test everything, but it should work.
https://github.com/prosthetichead/GarlicPress/assets/8648447/2477079c-5fd1-41ca-a25d-29e5314fd532

Nice that's awesome!!


Ldap active users bug fix

Bug fix. I got an error today when im on my client, if useraccountcontrol value is none then its crashing. Now it's checking.

Updated now and working again.

Thanks for the fix. In which instances is user_account_control = None?

I encountered msExchUserAccountControl. When i research it, it's Indicates that the user is associated with the Exchange server. This property specifies the status of the Exchange account and indicates whether there is a mailbox associated with the user.
If the ms Exch User Account Control property is assigned and its value is None, it causes an error.
But i didnt get it, why it is get the msExchUserAccountControl value. If im not busy, i would have setup on local exchange server.


Installation on Windows requires admin privileges

Hi,
LogoRRR is a quite valuable tool - thanks for sharing this!
Unfortunately the installation process on Windows has s severe limitation.
Current behavior
However, on Windows, especially in restricted environments, an installation using the provided installation won't work.
The installer requests admin privileges and if those are not granted, installation will abort.
The installer downloads the possibly required C++ runtime in background.
Installation is directly performed in c:\Program Files\LogoRRR.
Expected behavior
Installer will run without admin privileges.
Ideally installer will ask if C++ runtime libs shall be installed. Download and installation of C++ runtime shall only be performed when user permits this.
Installation is directly performed in c:\Users\<username>\AppData\Local\Programs\LogoRRR.

I could provide a zip file with all required dlls which could be used as a 'portable' solution for LogoRRR for your usecase maybe?
One problem I have here is to give the exe file an appropriate Icon (this was one motivation to create an installer in the first place ;-)
I'm not sure if installing applications without admin rights is supported by the installer tooling I'm using. I have to try it out.
At least Chrome installs itself to AppData\Local\Google if it doesn't get admin rights during installation, but is it common to install applications to this directory? It doesn't show up neither in 'add/remove programs...' as opposed to if you do install it with admin rights.
I would be happy to distributed logorrr with a single exe file, but it is an uphill battle against various security mechanisms from the OS which guard against such a use case ;-).

I wonder if it would be possible to use jpackage for that. I have not played with the build process yet but isn't there just a JAR file at the end? If so, it should be possible to let jpackage do the job. I'll play with that. With jpackage one can configure if this requires admin or non-admin rights on Windows. Its also quite easy to assign an application icon etc.

Ah, I see the problem - jpackage does not seem to support native images. Well... but may be reading the jpackage MSI code in OpenJDK provides some understanding on how to package an application using WIX toolset.

Well, I actually created a ZIP on my own. Nevertheless, I'll also have a look into the installer topic and the icon question. Possibly we'll find a solution for this together.

Actually, after thinking about it, you can just copy logorrr.exe to your system and start it from anywhere you like or have access to. It is completely self contained and will initialise itself from scratch if it finds the necessary vs2019 redist libs. As mentioned, odds are high that this is not possible due to some IT security restriction. If you are able to circumvent those issues in your usecase, you should be good to go.
Scenario 2 is to create a simple installer which installs LogoRRR to c:\Users<username>\AppData\Local\Programs\LogoRRR - alongside necessary redist dlls. The question here is if the installer still needs admin rights (i do not know atm).
logorrr.exe is created via GraalVM and gluon plugin tooling. logorrr.exe still has default meta data (see #20) which I would love to see resolved -> and actually by chance there has been some activity around this topic gluonhq/substrate/pull/1090 in the recent days 👍 I have to look into this in detail.
jpackager is in use for macos builds. it seemed to be the easiest solution to get to a pkg file. builds on windows use graal native image + advanced installer tech, but could be replaced via jpackager/jlink + wix instead. ideally i would like to have both solutions on both systems (native builds for mac/win and jpackager as well - since LogoRRR is also a playground for those technologies for me.
bottom line for this issue for me is: i try to create a non admin installer without the need to install redists explicitly with a nice user experience.

as of creating windows installers with jpackage - I'm not 100% sure but I think in the meantime gluonfx-maven-plugin can do this.

@Oliver-Loeffler I have prepared a 'slim' installer, which

should be able to be run as normal user
installs to Temp/Roaming/... directory
doesn't install visual studio runtimes (it assumes they are here - if not it just doesn't start)

please try it out, you can download the snapshot installer here:
https://www.logorrr.app/downloads/LogoRRR-22.1.0-installer-slim.exe
the plan for now is to distribute two versions for windows - one installer which installs everything, another one which is more friendly to be installed in restricted environments.

Great, I've tried out and it installs into C:\Users\<username>\AppData\Roaming\LogoRRR\ which is fine.
Unfortunately, don't know why, Microsoft defender complains it. It does not happen when packaging with JPackage. But one can click the Defender warning away. It does the job! Will try tomorrow in another environment and give you feedback.

windows defender complains about it since the installer is not signed (would require an ev certificate which costs about 600$/year or something #64). If you are comfortable with this solution I would close the issue.

Hi, so this works great. Issue solved! Thanks for the work!

Don't think its worth 600$ / year to sign it.

will be part of 22.1.0 release.


Adding stream and revokestream commands

These commands would help in moderation by allowing mods to give temporary screen sharing permissions with one command
stream {duration default 1} {time unit default hour} - gives user video role for specified duration
revokestream - takes away video role

Coverage increased (+0.09%) to 56.592% when pulling 25fe0c919edfffbca5a73554853d076455e2d997 on Harbys:master into ed8160f3e73e2060630b10e1fb0b762e0a51294a on python-discord:master.

I think I fixed all pointed out issues and implemented requested changes. Thx for pointing out the Scheduler as it actually made it much simpler

Hi, thank you for contributing!
The thing with changes to moderation features, is that they need follow internal discussions to decide on whether we want it, and how we would use it - which in turn affects the implementation.
That is not to say that you can't PR such a feature, but rather that the right course of action here would be to open an issue first to allow us to discuss it properly, and to prevent a case where you would do a lot of work that would end up not being used.
I will put this PR on hold for now until we can discuss it properly among the mods.

@Harbys Mods and admins team have made decision that we should have this feature, so I take this out from stalled status and you can continue your work. But please read #1343 issue, because implementation should be little bit different.

@MrHemlock I think it's all done as requested

@MrHemlock I think it's all done as requested

@Harbys are you planning on continuing this PR?
I'd like to see these changes get implemented.

@Xithrius Yes, I want to finish this pr, but I have very little spare time recently so I'm sorry it will be probably another week until I get it done.

@Xithrius Yes, I want to finish this pr, but I have very little spare time recently so, I'm sorry it will be probably another week until I get it done.

Take your time, I'm just making sure this will eventually get done.
Have a good one! :heart:

still need to do unit test, but I think that the cog is good for review

@Harbys will you be completing this PR? If not, it can be handed off to someone else if you don't have enough time.

Continued in #1493


Botched Integer Interpolation Expansion

Describe the bug
MontePy expands shortcuts for data entry:

The linear interpolation shortcut gets botched upon expansion, being converted to floating point and dropping necessary padding.
To Reproduce
Just load an MCNP input file with the shortcut and write it back out.
import montepy
montepy.read_input("testi.i").write_to_file("testo.i")

MCNP input file snippet
If applicable, please include a small section of the input file you were working on. If it includes any specific values please change or remove them. For example:
MCNP Test Model
C Cells
C Seven spheres within a sphere
1 1 -10 -1001  imp:n,p=2
2 1 -10 -1002  imp:n,p=2
3 1 -10 -1003  imp:n,p=2
4 1 -10 -1004  imp:n,p=2
5 1 -10 -1005  imp:n,p=2
6 1 -10 -1006  imp:n,p=2
7 1 -10 -1007  imp:n,p=2
C This cell breaks MontePy 0.3.3
8     0 -1010  1001  5i  1007  imp:n,p=1
C Graveyard
99    0 +1010     imp:n,p=0

C surfaces
1001 SO      0.1
1002 SX -1.1 0.1
1003 SX +1.1 0.1
1004 SY -1.1 0.1
1005 SY +1.1 0.1
1006 SZ -1.1 0.1
1007 SZ +1.1 0.1
1010 SO 3

C data
C materials
C UO2 5 atpt enriched
m1        92235.80c           5
          92238.80c          95
C execution
ksrc 0 0 0
kcode 100000 1.000 50 1050
mode n p

Error Message (if any)
MCNP Test Model
C Cells
C Seven spheres within a sphere
1 1 -10 -1001  imp:n,p=2
2 1 -10 -1002  imp:n,p=2
3 1 -10 -1003  imp:n,p=2
4 1 -10 -1004  imp:n,p=2
5 1 -10 -1005  imp:n,p=2
6 1 -10 -1006  imp:n,p=2
7 1 -10 -1007  imp:n,p=2
C This cell breaks MontePy 0.3.3
8     0 -1010  1001  1002.01003.01004.01005.01006.01007  imp:n,p=1
C Graveyard
99    0 +1010     imp:n,p=0

C surfaces
1001 SO      0.1
1002 SX -1.1 0.1
1003 SX +1.1 0.1
1004 SY -1.1 0.1
1005 SY +1.1 0.1
1006 SZ -1.1 0.1
1007 SZ +1.1 0.1
1010 SO 3

C data
C materials
C UO2 5 atpt enriched
m1        92235.80c           5
          92238.80c          95
C execution
ksrc 0 0 0
kcode 100000 1.000 50 1050
mode n p

Version

Version 0.3.3

Additional context
MontePy currently expands interpolation shortcuts and does not recompress them (#397).

Oh yikes.  I suspect that #397 is a common-mode failure. Part of this is we need to use context to change a shortcut to int from float.


How to get http.ResponseWriter in the generated code

In some of my API implementation, both the ResponseWriter and http.Request are needed.
I know that I can get the (r *http.Request) by
_, err := myFunc(params.HTTPRequest)

However, I could not find where to get the instance of  http.ResponseWriter in the generated code. Can anyone help? Thanks.

Implement a responder, by default we generate a few we can infer from the swagger spec but you can easily implement your own.
it's essentially  a func(http.ResponseWriter, runtime.Producer)
https://github.com/go-openapi/runtime/blob/master/middleware/context.go#L56-L68

Thanks @casualjim

Hi @casualjim
I think this is problematic if you're using something like Gorilla Sessions, where you need to invoke a session.save(rw, r), where rw is an http.ResponseWriter and r is an http.Request, before you return your responder.
In my case, I call session.save(rw, r) to get the session ID and then I need to save that value to the database. I don't see how I can do that with responder implementation, since the responder is invoked only after the route returns...

the responder executes  in the context of a web request. You can do it in a responder because it's essentially a call back. Afaict gorilla sessions wants you to call it before exiting ServeHTTP which it will.
api.StaticSayHelloHandler = static.SayHelloHandlerFunc(func(params static.SayHelloParams) middleware.Responder {

  originalResponder := getSomeResponder()

  return middleware.ResponderFunc(func(rw http.ResponseWriter, p runtime.Producer){

    session.save(rw, params.HTTPRequest)
    originalResponder.WriteResponse(rw, p)
  })
})

Hmm, I see, thanks. That seems to work.
I also tried modifying the template to include the http.ResponseWriter within the input parameter object of every request, just like how http.Request is stored.
This strikes me as having more symmetrical semantics and usage. Unless there's some obvious downside I'm unaware of...


paid post with email_recipient_filter=all delivers content to everyone (3.38.2)

Not sure this is by design, and I believe the behavior used to be different with send_email_when_published=true
There is an inconsistency of sorts because paid post with email_recipient_filter=all seems to deliver the post content (not just the title) to everyone, not just paid subscribers. I believe that wasn't the case earlier
Related conversation
https://github.com/TryGhost/Ghost/pull/12343
https://forum.ghost.org/t/posting-via-api-does-not-send-email-notification/18139

This is expected behaviour, if you would like to send an email only to paid members you can use email_recipient_filter=paid


Update parent version

The newest version of hotels-oss-parent includes the sonatype-oss-release profile with the maven-gpg-plugin, so it can be removed from this project.
Acceptance Criteria;

Update to version 2.0.5 of hotels-oss-parent.
Remove the sonatype-oss-release profile in pom.xml.

Done and merged


Unpublish edition edit

Trello
Add unpublish functionality for new design system edit page to allow unpublishing of published edition.
When edition is draft, does not shows unpublish tab

When edition is published, shows unpublish tab

Confirm unpublish

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Just added a couple of small comments regarding the UI but otherwise this looks good to me.
I would prefer someone with more Rails knowledge to look at it as well before approving for best practice etc. though it seems fine to me as well.


[Public Catalog] Align our external catalog update action on the same interval as the in product update for Quickstarts content refresh

This isn't 100% in sync, so there could be some differences, we'll want to try to align our external catalog update action on the same interval as the in product.
Currently set to 12AM PT, for install library service
Currently set to daily at 12AM PT for the external site
Acceptance Criteria

[ ] Confirm with @jbeveland27 / @JuliaNocera  what the interval is for refresh data for the in product I/O experience.
[ ] set the same time interval in a GH action to refresh the external site at the same cadence.
[ ] stagger time by 1 hr
[ ] Add this information to the contributor guide.

the decision was to leave this set to 1 time a day and we will monitor if that will change in the product and align as needed.

@aswanson-nr mentioned he wanted to change this to an hourly interval


Upstream Update (shared-ratelimit-patch)

The shared-ratelimit-patch branch is 8 commits behind dev.
This pull request was created automatically. If you wish to be excluded from automatic pr creation, open an issue here. Contact DonovanDMC if you find any other issues.

upstream branch name change


Update README.md

Add Gradients - curated splendid gradients made in Kotlin (port of https://webgradients.com for Android)

@bakhtiyork  😎 Thank you for your PR


Fix a minor typo in the docstring

this an an approximation -> this an approximation

hmmm, I think it should actually be this is an approximation

oh, you're right. I only focused on just removing duplicated an lol


Bump version to 0.7.0

Codecov Report
Patch and project coverage have no change.

Comparison is base (e00a195) 70.29% compared to head (e404cf0) 70.29%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #297   +/-   ##
=======================================
  Coverage   70.29%   70.29%
=======================================
  Files          67       67
  Lines        2461     2461
  Branches      214      214
=======================================
  Hits         1730     1730
  Misses        658      658
  Partials       73       73

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


Incorrect AutoHeight in Thumbs Gallery With Two-way Control

active index of controlled swiper is set after autoheight calculation instead of before

Here's a JSFiddle that shows the issue: https://jsfiddle.net/Wehrdo/nLqcts8e/1/
When using the navigation arrows in the main area, everything works fine. If you click the thumbnails, then the height doesn't update until the slide is being switched away from, causing it to be incorrect on the next slide.

Seems i have to fix it myself


stats.js error 500, when styles are provided

Issue description
When [styles] are given - occurs an error.
Steps to reproduce and a minimal demo of the problem
_Use stackblitz.com or similar -- try this template as a starting point: https://stackblitz.com/edit/angular-google-maps-demo-kt4nvu
What steps should we try in your demo to see the problem?
Reload the page and see the console
Current behavior
Map is styled, but this error occurs:
js?v=quarterly&callback=agmLazyMapsAPILoader&key=mykey:78 GET https://maps.googleapis.com/maps-api-v3/api/js/39/10/intl/ru_ALL/stats.js net::ERR_ABORTED 500

Unhandled Promise rejection:
Could not load "stats".
; Zone:

; Task:
Promise.then
; Value:
Error: Could not load "stats".
Error: Could not load "stats".
at HTMLScriptElement.__zone_symbol__ON_PROPERTYerror (https://maps.googleapis.com/maps/api/js?v=quarterly&callback=agmLazyMapsAPILoader&key=AIzaSyAvcDy5ZYc2ujCS6TTtI3RYX5QmuoV8Ffw:80:236)
at HTMLScriptElement.wrapFn (https://angular-google-maps-demo-kt4nvu.stackblitz.io/turbo_modules/zone.js@0.8.18/dist/zone.js:1166:39)
at ZoneDelegate.invokeTask (https://angular-google-maps-demo-kt4nvu.stackblitz.io/turbo_modules/zone.js@0.8.18/dist/zone.js:425:31)
at Zone.runTask (https://angular-google-maps-demo-kt4nvu.stackblitz.io/turbo_modules/zone.js@0.8.18/dist/zone.js:192:47)
at ZoneTask.invokeTask [as invoke] (https://angular-google-maps-demo-kt4nvu.stackblitz.io/turbo_modules/zone.js@0.8.18/dist/zone.js:499:34)
at invokeTask (https://angular-google-maps-demo-kt4nvu.stackblitz.io/turbo_modules/zone.js@0.8.18/dist/zone.js:1540:14)
at HTMLScriptElement.globalZoneAwareCallback (https://angular-google-maps-demo-kt4nvu.stackblitz.io/turbo_modules/zone.js@0.8.18/dist/zone.js:1566:17)
Expected/desired behavior
Map is styled with no errors in console
angular-google-maps, Angular, & any other relevant dependency versions
the latest versions were used
Other information

Fixed! AdBlock utility was blocking to load this file on my Mac.


README edit

The README gives an example of creating a custom pseudo selector :foo(bar), but no custom pseudo selectors can be added to qwery unless they are already supported by default in querySelectorAll.  You do say later that querySelectorAll is used where supported, but it isn't very clear that custom pseduo selectors will not be supported in that case.  If you wanted to implement this, I think you could with no performance loss and probably no more than three lines (I'm just picturing in my head).  But if not, the README should probably be updated.

I just banged around trying to get a replica of jQuery's :visible pseudo working only to realize it's not gonna work with querySelectorAll (as mentioned in the jQuery docs, it's not part of the CSS spec http://api.jquery.com/visible-selector/).
I agree that this becomes a pretty nasty thing to deal with when you start trying to determine what to pass off to querySelectorAll and what not to or throwing a try, catch around the whole thing. Kind of a bummer because some of those pseudos are really helpful but it can definitely be worked around.
PS. thanks @ded for all your awesome work on ender, qwery and all this other awesome JS!

again, many apologies that you went through all the trouble, then found out it wouldn't be supported. it's about time i update the readme on this stuff


Documentation about k8s on a origin

Sorry, this topic should be supervisord
--Fábio Andrijauskas
On Tue, Jun 28, 2022 at 11:03 AM Brian Lin @.***> wrote:

@.**** commented on this pull request.
In docs/data/stashcache/run-stash-origin-container.md
https://github.com/osg-htc/docs/pull/10#discussion_r908793063:

+### Running on origin kubenetes with systemd

+TODO
Right, that's within the container using supervisord. Systemd would only
really be used on bare metal to manage services, which is unnecessary with
k8s
—
Reply to this email directly, view it on GitHub
https://github.com/osg-htc/docs/pull/10#discussion_r908793063, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACEEH76H5GN7AECTCBYJP5DVRM46HANCNFSM5Z7ROIVQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Should we add some instructions for the monitoring configuration?
On Tue, Jun 28, 2022 at 12:12 PM Brian Lin @.***> wrote:

@.**** commented on this pull request.
In docs/data/stashcache/run-stash-origin-container.md
https://github.com/osg-htc/docs/pull/10#discussion_r908846904:

+### Running on origin kubenetes with systemd

+TODO
Sorry, this topic should be supervisord
I don't think this is necessary: the container should start supervisord
and thus the relevant services. The operator should ideally not have to
interact with supervisord at all!
—
Reply to this email directly, view it on GitHub
https://github.com/osg-htc/docs/pull/10#discussion_r908846904, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACEEH7ZELNSSJA3PBIOQITDVRNFDRANCNFSM5Z7ROIVQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
--Fábio Andrijauskas


#6 Added copyrights

Codecov Report

Merging #7 (bc03df4) into main (d63dc68) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main       #7   +/-   ##
=========================================
  Coverage     98.62%   98.62%
  Complexity      174      174
=========================================
  Files            37       37
  Lines           363      363
  Branches         27       27
=========================================
  Hits            358      358
  Misses            5        5

Impacted Files
Coverage Δ

.../java/com/github/viise/stena/ProtectException.java
50.00% <ø> (ø)

.../java/com/github/viise/stena/wal/WalAtmIntArr.java
100.00% <ø> (ø)

...java/com/github/viise/stena/wal/WalAtmLongArr.java
100.00% <ø> (ø)

...ain/java/com/github/viise/stena/wal/WalAtmRef.java
100.00% <ø> (ø)

.../java/com/github/viise/stena/wal/WalAtmRefArr.java
100.00% <ø> (ø)

...com/github/viise/stena/wal/WalAtmRefArrStrict.java
100.00% <ø> (ø)

...ava/com/github/viise/stena/wal/WalAtmStampRef.java
100.00% <ø> (ø)

...c/main/java/com/github/viise/stena/wal/WalDir.java
100.00% <ø> (ø)

.../java/com/github/viise/stena/wal/WalDoubleEps.java
100.00% <ø> (ø)

.../main/java/com/github/viise/stena/wal/WalFile.java
80.00% <ø> (ø)

... and 27 more

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 f934b4e...bc03df4. Read the comment docs.


UI Upgrade to Cloudscape & other customizations

Very happy with the UI now looking at the next stage, curious if it would be in a good interest to migrate to cloudscape UI that is created, maintained, and used by the AWS console team. For customers writing new features in the future, the plugins and components would allow them to build rich user interfaces without having to add external packages.
Curious to hear thoughts on this.

Also interested in hearing thoughts on using the Cloudscape components. https://cloudscape.design/demos/overview/

We will close this item as it is not a priority at the moment. It does not mean it is closed forever, if users show interest, we can open it again.


document launch schema version 3

follow-up https://github.com/jupyterhub/binderhub/pull/872

Thank you for catching up the docs.


can support Chinese? Can training to identify local languages?

First of all, thank you for your contribution to open source. I would like to ask if the project can support Chinese? Can training to identify local languages?

Hello @deepdream88,
Thanks for opening that issue and for your support.
Today Leon support English and French languages but I plan to also let him support other languages such as Chinese in the future. So stay tuned 😉
Does it reply to your question?

你好@ deepdream88，
感谢您打开该问题并获得您的支持。
今天Leon支持英语和法语，但我计划将来也支持其他语言，如中文。所以请继续关注
它回复了你的问题吗？

oh,thank you for your reply

You are very welcome. I'm going to close this issue as the answer has been provided.
Do not hesitate to reopen it or open a new one if necessary.


CDPX Localization

Both Powertoys Run and Keyboard Manager require localization efforts.
Here are are the instructions to enable loc efforts: (visible only to Microsoft employees) https://onebranch.visualstudio.com/Pipeline/_wiki/wikis/Pipeline.wiki/290/Localization

@ryanbodrug-microsoft @crutkas @saahmedm Created #6055 and #6056 to track the work pending for localization

Moving to 2009. Needs loc team interaction to be complete.

Pending dev work on this is to modify the installer project so that the new resource dlls are included. This might be easier to do once we receive the translated resources.

In the 0.25 release.  https://github.com/microsoft/PowerToys/releases/tag/v0.25.0


assert on windows

With every new release I still have to change this line in pattern.cpp:
  if (c <= 255 &&        std::isalpha(c) && (opt_.i || is_modified('i', modifiers, loc)))
because isalpha asserts in debug mode.
You said you didn't want to change it because of optimization, but the extra comparison is nothing compared to the function calls and their contents.

Ah, I think the problem is the c = compile_esc(loc + 1, chars) that may return c>256. I don't recall saying that it wasn't necessary to change because of optimization, but rather I may have looked at the wrong line that looks the same as you report but which doesn't have this issue.

Fixed.


To-Many Relationships and authorization

I recently came across jsonapi.org and this got me very excited. One thing that puzzles me though, is the preferred way of authorization for to-many relationships, where the parent object determines whether the client is authorized to read or write objects to the collection of child objects (for individual child objects the child object id can be used for authorization of course, the issue is only with collections of child objects where only the parent id is available for authorization).
Example to make it easier to explain: suppose I'm building a simple task manager application where i have a User model and a Task model. The User has many Tasks. A client acting on behalf of a User is only authorized to read this User's Tasks (not those of other Users) and may only create new Tasks for this user (not for other Users). With all requests, the client includes token for authentication where the token can be used to identify the User the client acts for.
The first issue is with reading the Tasks that belong to the a User. In my understanding there are 2 general approaches for reading a relationship (not including 'linked' side loading). The first involves first obtaining a list of ids of Tasks that belong to a User and then making a GET request to /tasks/first_task_id,second_task_id,third_task_id,..., which I'll call the id-list method. The second consists of using the 'links' semantic and making a GET request to /users/user_id/links/tasks, which I'll call the links method.
It seems to me that the links method would be preferable in the context of authorization, because I believe that the error semantics are simpler and more clear. If I make a request to /users/user_id/links/tasks and I get a 403 Forbidden response, this means to me: I am not allowed to read the tasks of this user. It arguably does not need an error message to explains this in more detail. If i make a request to /tasks/first_task_id,second_task_id,third_task_id and I get a 403 Forbidden response, this is much more ambiguous to me. In the links method I viewed it as a request for a single collection resource, and even though strictly speaking this is also a single collection resource with a compound key, intuitively for me, this id-list method is a request for multiple individual resources in a single request. My mind doesn't process this as "I'm not allowed to read the collection resource /tasks/first_task_id,second_task_id,third_task_id", but as "I'm not allowed to read one or more of /tasks/first_task_id, /tasks/second_task_id, /tasks/third_task_id". For the id-list method, I would have to add an error message to tell me exactly which of the tasks I'm trying to read is causing this 403 Forbidden response. Conceptually, authorization and the links method seems much clearer than authorization and the id-list method.
The second issue is with creating a new Task for a user. It seems to me that there may be 2 approaches that the current spec allows. The first would be to use a 2-step approach. One would make the following request:
POST /tasks
Content-Type: application/vnd.api+json
Accept: application/vnd.api+json

{
  "tasks": {
    "description": "Some task"
  }
}

The API responds with the created resource. The client uses the created resource's id and makes a second request:
POST /users/user_id/links/tasks
Content-Type: application/vnd.api+json
Accept: application/vnd.api+json

{
  "tasks": "task_id"
}

For this to work though, I would have to remove the "presence" validation on the Task's User property. If the client fails to make the second request, I may end up with Tasks in my DB that don't belong to any user. I'd prefer not to remove my "presence" validation.
I'm actually not sure that the spec allows this second approach, but let's assume that it does. The client would make the following request:
POST /tasks
Content-Type: application/vnd.api+json
Accept: application/vnd.api+json

{
  "tasks": {
    "description": "Some task",
    "links": {
      "user": "user_id"
    }
  }
}

It seems to me that authorization would now involve validating the task.links.user property (which I guess would deserialize to the 'task.user_id' property). This feels like mixing authorization with validation (and would in e.g. Rails probably be most easily handled through adding a custom validation, instead of doing controller 'action-based' authorization). I would rather not mix validation with authorization.
The approach that seems most intuitive to me, isn't actually currently allowed by the spec as I understand it. The client would make the following request:
POST /users/user_id/links/tasks
Content-Type: application/vnd.api+json
Accept: application/vnd.api+json

{
  "tasks": {
    "description": "Some task",
  }
}

I wont have to remove my "presence" validation and I'm also able to do 'action-based' authorization without any mixing of validation and authorization logic. However, the current spec states: "A to-many relationship MAY be added by sending a POST request with a single resource ID to the URL of the relationship." It does not mention anywhere that posting a resource object to a links URL is allowed, so I take it that this approach is not allowed and that instead approach 1 needs to be followed.
As mentioned, there's no issue with updating, or deleting Tasks, or reading an individual Task, because in these cases the Task's id can be used for authorization, akin to "shallow routing".
I dont think "to-many relationships with authorization" is a very uncommon use case. I may have completely missed the point somewhere here. What would be the preferred way to handle authorization and to-many relationships with jsonapi?

Not sure whether this was ever proposed, so I post it here. Please correct me if anything.
I'm thinking how to distinguish representation of relations of same-to-same type when there's difference of meaning in parent-child relationship. This edge case comes up only when objects of the same type are being linked, e.g. categories: category has parent category and can have children categories.
I have strong opinion on these links are being mixed right now according the specs without having the link type different. I mean users linked under authors attribute under post.links. This approach is useful, since user can be linked also as owner, editor implying meaning of a link.
You could create category.links.parentCategories and category.links.childCategories to distinguish same-to-same relations. Yet it does not self-describes which set of links is considered as the objects object being linked to, or objects being linked to the object if you omit the parent, child prefixes. You can only sense it because you as a human being understand the words.
I'm pretty sure it would break/breaks the specs, but here's the thing:

Let's call to-many relationships as linked (object has linked objects)
Let's call to-one relationships as links (object links to objects)

Example response to GET /collections/android-phones" (for simplicity and relation dependency, the id's are unique texts here):
{
  "links": {
    "collections: "http://example.com//collections/{collections.id}"
  },
  "collections": {
    "id": "android-phones",
    "name": "Collection of Android phones",
    "links": {
      "collections": ["phones"]
    },
    "linked": {
      "collections": ["samsung-android-phones", "sony-android-phones", "other-android-phones"]
    }
  }
}

The top level would still use links attribute as proposed by specs, only the results would distinguish between the parent and child links using links and linked attributes.
Note: The meaning could be swapped as well for object has links and object is being linked to. It's a matter of agreement on specs.

In the future, discussions like this would be fantastic for http://discuss.jsonapi.org. For now, I'm giving this a close. JSON API has reached 1.0 and we cannot make backwards incompatible changes.