How to save the model

I have successfully run graphcast_demo.ipynb locally, but how do I save the model parameters after the final training using the sample dataset?
I try using code:
# @title Autoregressive rollout (keep the loop in JAX)
print("Inputs:  ", train_inputs.dims.mapping)
print("Targets: ", train_targets.dims.mapping)
print("Forcings:", train_forcings.dims.mapping)

predictions = run_forward_jitted(
    rng=jax.random.PRNGKey(0),
    inputs=train_inputs,
    targets_template=train_targets * np.nan,
    forcings=train_forcings)
predictions

with open(f"dm_graphcast/params/test.npz", "wb") as f:
    checkpoint.dump(f, graphcast.CheckPoint)

But it doesn't work.

What about trying this
ck_file='Gcloud/params/GraphCast_small - ERA5 1979-2015 - resolution 1.0 - pressure levels 13 - mesh 2to5 - precipitation input and output.npz'
print("Reading Checkpoint=", ck_file)
ckpt = checkpoint.load(ck_file, graphcast.CheckPoint)
print("Model description:\n", ckpt.description, "\n")
params = ckpt.params
model_config = ckpt.model_config
task_config = ckpt.task_config
Perform optimisation
ck_file2=f"GC_new.npz"
print("Saving Checkpoint=", ck_file2)
with open(ck_file2, "wb") as f:
        checkpoint.dump(f, ckpt)

You need to pass a graphcast.CheckPoint object, not the class definition. In the colab that is the ckpt variable, but if you're not changing it, you can also just download the file directly. If you want to change it (eg fine tune it), you'll need to construct a new object to pass to checkpoint.dump.

You need to pass a graphcast.CheckPoint object, not the class definition. In the colab that is the ckpt variable, but if you're not changing it, you can also just download the file directly. If you want to change it (eg fine tune it), you'll need to construct a new object to pass to checkpoint.dump.

ok, thank you, i have got it.

You need to pass a graphcast.CheckPoint object, not the class definition. In the colab that is the ckpt variable, but if you're not changing it, you can also just download the file directly. If you want to change it (eg fine tune it), you'll need to construct a new object to pass to checkpoint.dump.


open 2 Ps4 macro || noticed wrrong PID of ps4 remote play

i managed to open 2 PS4 remote play app, now i wanna control them with each PS4 macro, however it seems the second PS4 macro that i opened to connect to the second PS4 remote play app failed because the second PS4 macro was getting wrong PID of the second PS4 remote play app - so how can i specify the PID of ps4 remote play app for PS4 macro?

Hi, did you figure out the solution?


Improve error from Toolbox\Blueprints\BlueprintForm->doLoad()

With the newer, explicit erroring out in Toolbox, which was implemented from static analysis, the error - \InvalidArgumentException(__METHOD__ . '(): Parameter #1 does not contain array of filenames'); - is rather unspecific.
It would be beneficial to include the $filename to tell the developer/user what caused this error specifically. As shown in a recent error-report, this can be confusing and obscure.
Tagging @mahagr by request. Feel free to move to Toolbox-repo if desirable, just thought it more pertinent and potentially time-sensitive for Core if it is a pervasive error.

I receive this error when clicking "Add" from the Plugins page in Dashboard. The Plugins page itself renders properly, listing all the installed plugins, but the page that should show all available plugins delivers the Crikey! page, stating there are no filenames in the array.


async function () {} syntax causes parser to throw

This expression parses correctly using babel-eslint:
 const foo = async () => {};
However this one does not:
 const foo = async function () {};
Both expressions are valid, so I'm fairly sure this is a bug. Here's my configuration:
{
  "env": {
    "node": true,
    "es6": true
  },
  "parser": "babel-eslint",
  "parserOptions": {
    "ecmaVersion": 6,
    "sourceType": "module",
    "codeFrame": false
  },
  "rules": { ... }
}

babel-eslint 6.1.2, eslint 3.19.0

It works fine on the repl, and astexplorer so it shouldn't be happening? Maybe update to babel-eslint 7 then and you should be good


[AVM Question/Feedback]: Using variables in the custom library files

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Description
In the Azure/terraform-azurerm-caf-enterprise-scale module we have defined the module in the root directory and use the same configuration for different environments referencing root in subdirectories. This approach ensures that all environments have the same policies and so on. We rely on the template_file_variables variable to pass different values to the various environments.
I wonder if it will be a similar approach here, making variables available in the custom library files? If so, great! If not, should I put it in a feature request or has it already been considered?

Hi!
You can still use a templating approach but this needs to be external to the provider.

I had the idea of using a separate Terraform state file to generate the custom lib from templates.
./lib
  - my.alz_architecture_definition.json.tftpl
  - main.tf

You could then consider generating the custom lib every time you run the pipeline. Do not store the resultant architecture definition files in Git, rely instead on the Terraform to generate them.
Then the steps of your pipeline would be:

Pre-reqs, auth, etc
Run tf init, apply in the ./lib dir to generate the artifacts
Run tf init, apply in the root dir to deploy

Thoughts?

So anyway, we will have to generate the json files upfront somehow for them to be "sent" to the provider.
We will try it and see if that will work for us. Thanks

Would using policy_default_values & policy_assignments_to_modify work for this?
I'm starting to use this module to manage 3 environments and I can customize the policy assignment parameter values for each environment using the above functionality, whilst only defining the policies once. No external templating needed.
Mainly I'm only updating the parameter values using this method, with a couple of instances of the non-compliance message for specific policy assignments. If you've got a more complex templating requirements then I guess this might not work.
I'm also referencing 2 separate libraries for the ALZ provider, one in the parent for any common policies / archetypes, and one in the environment folder for the architecture definition and anything custom that might be needed for that environment (not had to do that yet though).
provider "alz" {
  library_references = [
    {
      path = "platform/alz",
      ref  = "2024.07.02"
    },
    {
      custom_url = "${path.cwd}/../alz_lib_common"
    },
    {
      custom_url = "${path.cwd}/alz_lib"
    }
  ]
}

That might also work, I will definitely give that a try! Thanks for the suggestion @JWilkinsonMB :-)

@JWilkinsonMB Do you experience any trouble when it comes to roleDefinitions with your setup?
I have set up 3 different environments and testing them locally to simulate a pipeline run by triggering plan and apply simultaneously in the 3 environments.
However I seem to hit an error quite often and the error is any of or multiple of these role definitions.
Example:
╷
│ Error: Missing Resource State After Create
│
│   with module.alz_architecture.module.role_definitions["new3/Subscription-Owner"].azapi_resource.this,
│   on .terraform/modules/alz_architecture/modules/azapi_helper/main.tf line 1, in resource "azapi_resource" "this":
│    1: resource "azapi_resource" "this" {
│
│ The Terraform Provider unexpectedly returned no resource state after having no errors in the resource creation. This is always an issue
│ in the Terraform Provider and should be reported to the provider developers.
│
│ The resource may have been successfully created, but Terraform is not tracking it. Applying the configuration again with no other
│ action may result in duplicate resource errors. Import the resource if the resource was actually created and Terraform should be
│ tracking it.
╵
╷
│ Error: Missing Resource State After Create
│
│   with module.alz_architecture.module.role_definitions["new3/Application-Owners"].azapi_resource.this,
│   on .terraform/modules/alz_architecture/modules/azapi_helper/main.tf line 1, in resource "azapi_resource" "this":
│    1: resource "azapi_resource" "this" {
│
│ The Terraform Provider unexpectedly returned no resource state after having no errors in the resource creation. This is always an issue
│ in the Terraform Provider and should be reported to the provider developers.
│
│ The resource may have been successfully created, but Terraform is not tracking it. Applying the configuration again with no other
│ action may result in duplicate resource errors. Import the resource if the resource was actually created and Terraform should be
│ tracking it.
╵
╷
│ Error: Failed to create/update resource
│
│   with module.alz_architecture.module.role_definitions["new3/Security-Operations"].azapi_resource.this,
│   on .terraform/modules/alz_architecture/modules/azapi_helper/main.tf line 1, in resource "azapi_resource" "this":
│    1: resource "azapi_resource" "this" {
│
│ creating/updating Resource: (ResourceId
│ "/providers/Microsoft.Management/managementGroups/new3/providers/Microsoft.Authorization/roleDefinitions/d3584a79-4f0d-5980-aa3c-7a76ba783b76"
│ / Api Version "2022-04-01"): GET
│ https://management.azure.com/providers/Microsoft.Management/managementGroups/new3/providers/Microsoft.Authorization/roleDefinitions/d3584a79-4f0d-5980-aa3c-7a76ba783b76
│ --------------------------------------------------------------------------------
│ RESPONSE 404: 404 Not Found
│ ERROR CODE: RoleDefinitionDoesNotExist
│ --------------------------------------------------------------------------------
│ {
│   "error": {
│     "code": "RoleDefinitionDoesNotExist",
│     "message": "The specified role definition with ID 'd3584a79-4f0d-5980-aa3c-7a76ba783b76' does not exist."
│   }
│ }
│ --------------------------------------------------------------------------------

I suspect that it primarily has to do with the simultaneous plan/apply operations and possibly the "common" library, but I'm not sure. I have met other errors as well for management groups and missing permissions on these.
Example:
Error: Failed to create/update resource
│
│   with module.alz_architecture.module.management_groups_level_1["new3-landingzones"].azapi_resource.this,
│   on .terraform/modules/alz_architecture/modules/azapi_helper/main.tf line 1, in resource "azapi_resource" "this":
│    1: resource "azapi_resource" "this" {
│
│ creating/updating Resource: (ResourceId "/providers/Microsoft.Management/managementGroups/new3-landingzones" / Api Version
│ "2023-04-01"): GET https://management.azure.com/providers/Microsoft.Management/managementGroups/new3-landingzones
│ --------------------------------------------------------------------------------
│ RESPONSE 403: 403 Forbidden
│ ERROR CODE: AuthorizationFailed
│ --------------------------------------------------------------------------------
│ {
│   "error": {
│     "code": "AuthorizationFailed",
│     "message": "The client 'XYZ' with object id 'aaaaaaaa-bbbb-cccc-dddd-eeeeeeeeeeee' does not have authorization to perform action 'Microsoft.Management/managementGroups/read' over scope '/providers/Microsoft.Management/managementGroups/new3-landingzones' or the scope is invalid. If access was recently granted, please refresh your credentials."
│   }
│ }
│ --------------------------------------------------------------------------------
│
╵

I have not been able to find out exactly why this happens, setting the timeouts and delays variables haven't had any effect either. Have you experienced anything like this?

@chrholt I've not had the issue getting 'Missing Resource State After Create', but several times I've had the 403 error. A subsequent re-run or two of the deployment fixed it.
I think changing the timeouts and delays should in theory fix it, but on the Azure side it says changes can take up to 10 minutes and setting the delay to 10 minutes for everything is going to lead to a very slow deployment.
Looking at https://github.com/Azure/terraform-azurerm-avm-ptn-alz/pull/103 it seems the intent is to move away from fixed delays to use a retry feature with exponential backoff that's being introduced in v2.0.0 of the azapi provider (https://github.com/Azure/terraform-provider-azapi/pull/392).
Hopefully that will fix these issues, but @matt-FFFFFF will be far better qualified to comment.

AzAPI v2 makes the above much easier to deal with. As you can see on the PR we are able to retry on these common errors...
Hold fire until the next release as we also deprecate the helper module.

Also it's great to see folks coming up with innovative ways to use this! Awesome 🙌

The missing resource state after create errors have been fixed and will be released in azapi v2.0

Adding #RR to see if there is anything further required here

Not the original author, but from my perspective v0.9.0-beta with v2 AzAPI has been working really well. Much faster to deploy and so far at least, without the intermittent errors described above.

Excellent! Glad to hear it

When it comes to variables I was able to achieve what we need using default_policy_values and policy_assignments_to_modify, as suggested by JWilkinsonMB :-)
Good to hear about the retry feature too, I have not yet been able to test it though.
For our use case we need to have the ability to assign notScopes dynamically, but for that I have created a separate issue in terraform-provider-alz so it's not really relevant to this particular issue.

We will follow with this feature after GA!


Add Carton local to Perl ignore

Add local/ directory of Carton dependency manager
Pull Request
Thank you for contributing to @toptal/gitignore and https://www.gitignore.io.
New or update
Select the appropriate check box for this pull request. This helps when merging to ensure there are no conflicts with other templates or misunderstandings of how thee template list works.
New

[ ] Template - New .gitignore template
[ ] Composition - Template made from smaller templates
[ ] Inheritance - Template similar to an existing template
[ ] Patch - Template extending functionality of existing template

Update

[x] Template - Update existing .gitignore template

Details
Carton is a dependency manager for Perl projects (similar to NPM for Node). The manpage for it specifically says to run echo local/ >> .gitignore:
Ref: https://manpages.ubuntu.com/manpages/impish/man3/Carton.3pm.html

@alexmcode @yusufzmly Please review

@yusufzmly Please review


Move build products out of the ghc tree

We've discussed this in #32 already when debating how to properly reset the build. So far we have depended on the build products being built inside of the ghc tree, such that we could potentially use makes build artifacts during the build. As we now can build on Windows, OS X and Linux.

Agreed, it's time to start moving things around. Let me take care of this.

I moved most build artefacts to buildRootPath = ".build" which can be changed in Settings/User.hs. An example command invocation:
/-----------------------------------------------\
| Run Ghc Stage0 (package = ghc)                |
|     input: compiler/main/GHC.hs               |
| => output: .build/stage0/compiler/build/GHC.o |
\-----------------------------------------------/

This is still work in progress and there are a few temporary hacks.
The remaining bits are: gmp, program binaries, package configuration folders such as libraries/bootstrapping.conf, generated files in includes, and files produced by ghc-cabal (e.g. package-data.mk). Should be able to finish this tomorrow, although the last one seems tricky. We may first need to tackle the ghc-cabal issue #18.

Coming across some limitations unfortunately. For example:
Exit code: 1
Stderr:
In file included from compiler\nativeGen\AsmCodeGen.hs:14:0:

compiler\HsVersions.h:23:0:
     fatal error: ../includes/ghcautoconf.h: No such file or directory
compilation terminated.

For some reason compiler\HsVersions.h includes a generated file directly ../includes/ghcautoconf.h, not relying on -I paths. This forces me to keep a copy of generated file in includes folder.
Does anyone know the reason for #include "../includes/ghcautoconf.h"? Shall this be fixed in GHC?

And another one :(
Exit code: 1
Stderr:
compiler\main\DynFlags.hs:127:0:
     fatal error: ../includes/dist-derivedconstants/header/GHCConstantsHaskellExports.hs: No such file or directory
compilation terminated.

Another mystery. When invoking the following command:
inplace/bin/ghc-stage1.exe
-hisuf hi
-osuf o
-hcsuf hc
-static
-hide-all-packages
-no-user-package-db
-this-package-key rts
-i
-i.build/stage1/rts/build
-i.build/stage1/rts/build/autogen
-I.build/stage1/rts/build
-I.build/stage1/rts/build/autogen
-optP-include
-optP.build/stage1/rts/build/autogen/cabal_macros.h
-odir .build/stage1/rts/build
-hidir .build/stage1/rts/build
-stubdir .build/stage1/rts/build
-rtsopts
-H32m
-O2
-c rts/PrimOps.cmm
-o .build/stage1/rts/build/PrimOps.o
-Irts
-Iincludes
-I.build/includes

I get the following error:
ghc-stage1.exe: ghcversion.h missing

Even though folder .build/includes contains ghcversion.h. The command goes ahead happily when I copy ghcversion.h to includes directory. No idea why this happens despite -I.build/includes.

ghc-stage1.exe: ghcversion.h missing

I guess you are hitting this:
-- | Find out path to @ghcversion.h@ file
getGhcVersionPathName :: DynFlags -> IO FilePath
getGhcVersionPathName dflags = do
  dirs <- getPackageIncludePath dflags [rtsUnitId]

  found <- filterM doesFileExist (map (</> "ghcversion.h") dirs)
  case found of
      []    -> throwGhcExceptionIO (InstallationError ("ghcversion.h missing"))
      (x:_) -> return x

from compiler/main/DriverPipeline.hs

ghc-stage1.exe: ghcversion.h missing

I guess you are hitting this:
-- | Find out path to @ghcversion.h@ file
getGhcVersionPathName :: DynFlags -> IO FilePath
getGhcVersionPathName dflags = do
  dirs <- getPackageIncludePath dflags [rtsUnitId]

  found <- filterM doesFileExist (map (</> "ghcversion.h") dirs)
  case found of
      []    -> throwGhcExceptionIO (InstallationError ("ghcversion.h missing"))
      (x:_) -> return x

from compiler/main/DriverPipeline.hs

@angerman Thanks! I think you are right.

OK, it looks like moving generated files from includes to .build/includes is just too cumbersome due to the above issues. I don't think this this is worth it. We'll keep mutating includes folder just like the old build system did. I suggest we do the same with `inplace' for now, to avoid further complications.
The only remaining bit then is to move files produced by ghc-cabal (e.g. package-data.mk). Hopefully I'll have more luck with this.

@hvr, just mentioned https://ghc.haskell.org/trac/ghc/ticket/8040 on irc.

@angerman Thanks. Maybe we need to add the above to that ticket.

@snowleopard done. Just linked this issue from trac.
@snowleopard could you leave a compiled list of all the files we had to leave in the ghc tree and could not move out of it? So we can track that separately and potentially fix it upstream?

@angerman Thanks! Here is the list of what will stay in the tree for now:

All generated files in includes.
All binaries/generated files placed into inplace. I'm not sure we can change any conventions here, because people may depend on them.

@snowleopard, I think I have fixes for most of the issues above in D2530. This leaves only the ghcversion.h issue. It seems like the correct way to deal with this would be for Hadrian to ensure that the build directory is included in the rts library's include path. Would this be possible?

@bgamari Great, many thanks!

This leaves only the ghcversion.h issue.

Just to make sure I understand: are you saying we no longer need to generate files into includes and can in fact move all generated files from there to _build/includes?
If yes, I'll give it a try! Adding build directory to rts's include path should be easy (although judging by this comment: https://github.com/snowleopard/hadrian/issues/113#issuecomment-170422165, this has already been attempted).

Just to make sure I understand: are you saying we no longer need to generate files into includes and can in fact move all generated files from there to _build/includes?

I believe so. D2530 has been merged so you should now be able to move ahead with this. Let me know if you have any trouble.

@bgamari Thanks! The above commit moves all generated includes into _build/generated.
There are still a few remaining bits, see the todo list in https://github.com/snowleopard/hadrian/issues/113#issuecomment-170315446. The biggest one is the files generated by ghc-cabal. As we discussed previously, the easiest solution would be to modify ghc-cabal so it is possible to specify a folder where to put generated files (currently it creates them in the package directory).

@bgamari Ah, there is still one remaining relative path in DynFlags.hs:
#include "../includes/dist-derivedconstants/header/GHCConstantsHaskellExports.hs

Andrey Mokhov<EMAIL_ADDRESS>writes:

@bgamari Ah, there is still one remaining relative path in DynFlags.hs:
#include "../includes/dist-derivedconstants/header/GHCConstantsHaskellExports.hs

Bah, sorry about that. Opened D2559 with a fix.

@bgamari Thanks for fixing!
There is one more problem: ghcversion.h seems to be baked in somehow -- if I move it to _build/generated, I get ghc-stage1.exe: ghcversion.h missing from GHC even though I pass the directory via -I. Looking at the code, it seems that GHC looks for ghcversion.h in RTS package include directories for some reason, which point to includes.
I temporarily copy ghcversion.h to includes, but I hope there is a better solution.

There is one more problem: ghcversion.h seems to be baked in somehow

Yes, it is. In an earlier message I suggested you deal with this by adding your build directory to the include-path of the rts package.

@bgamari Ah, indeed, I confused filenames -- sorry!
As I understand, you refer to the following line in RTS package.conf.in file:
include-dirs:           TOP"/rts/dist/build" TOP"/includes" TOP"/includes/dist-derivedconstants/header" FFI_INCLUDE_DIR

In a way, we'd like to change it to
include-dirs:           TOP"/rts/dist/build" TOP"/includes" TOP"/_build/generated" FFI_INCLUDE_DIR

This is a bit tricky, because _build is not fixed but is a user setting. Anyway, the above commit seems to do the trick.

Anyway, the above commit seems to do the trick.

Heh, oh that is devious ;)

What's the progress on this?

The above commits move the directories generated by ghc-cabal into the build directory.
I believe we can now close this issue, because moving program binaries outside inplace/bin and Stage1 package database outside inplace/lib is probably beyond the scope of Hadrian (and it can be easily done if/when we decide to do that).
Note: ghc-cabal still runs the configure scripts locally for some packages, generating some files in the tree, but I don't think we can do anything about it before tackling #18.
If you spot any other build artefacts that should be moved outside of the GHC tree, feel free to reopen this issue, or even better create a new one.


ChunkedAssociativeLongArray re-use expired nodes

This change gives ChunkedAssociativeLongArray ability to store chunks that are expired and removed from main data structure.
Then instead of allocating new Chunk immediately we are trying to poll one from this cache.
So if you have constant or slowly changing load ChunkedAssociativeLongArray will never throw away old chunks or allocate new ones which makes this data structure almost garbage free. Also chunks stored in cache as SoftReferences so this memory can actually be freed if nesesary.

Numbers:
In ReservoirBenchmark: updates are 5.9x faster and ~370x less time spent in GC than SlidingTimeWindowReservoir.
In SlidingTimeWindowReservoirsBenchmark: updates are 3.2x faster, reads - 2.7x faster, 37x less time spent in GC. (here we see small difference because most time we spent in cleaning snapshots)

Graphs:
Comparison of implementations under constant load [duration 1h]

Comparison of implementations under changing load [duration 1h]

Hi @arteam,
It would be nice if you have a chance to look at this PR before release.

Here is great illustration of chunks reuse. link

Hi @ryantenney thank you for merge.
🎉

I didn't merge, but only because I didn't want to screw up the release @arteam was planning! Just waiting for him to review and merge!

Hi @arteam,
Thank you for review. All comments fixed.


Release file handle after moving direct downloaded application file

Possible fix for https://github.com/hydephp/cli/issues/192

Did not solve https://github.com/hydephp/cli/issues/192


MapView.Marker onSelect support for Android

Is MapView.Marker onSelect support for Android on your short-term roadmap?

https://stackoverflow.com/questions/29345928/changing-the-icon-of-google-maps-marker-while-its-selected
https://stackoverflow.com/questions/29740839/how-to-select-and-deselect-a-marker-in-google-maps-in-android

Just ran into this issue. When you say those events are not supported on Android, does that actually mean Google Maps? As it does not seem to work for me on iOS using Google Maps.


How do I prevent UVR from using OpenGL as It does not work for me and keep it only using Direct3D9 or CPU?

You might want to check
https://github.com/pal1000/mesa-dist-win/releases
to wrap OpenGL to Vulkan on Windows, but I'm not sure if OGL is the culprit of this error at all.

You might want to check https://github.com/pal1000/mesa-dist-win/releases to wrap OpenGL to Vulkan on Windows, but I'm not sure if OGL is the culprit of this error at all.

I tried mesa drivers but It does not work.
This error happens when I select High Performance mode for my switchable graphics AMD Radeon 7000m.

Are you sure that's the problem bound to OGL?
Some games or apps using OGL don't work for you?
Maybe just use DDU, go to safe mode, uninstall all the GPU drivers (Intel,
and then AMD), and reinstall them from scratch.
niedz., 21 maj 2023 o 00:39 Mohammed Mehdi TBER @.***>
napisał(a):

You might want to check https://github.com/pal1000/mesa-dist-win/releases
to wrap OpenGL to Vulkan on Windows, but I'm not sure if OGL is the culprit
of this error at all.
I tried mesa drivers but It does not work.
This error happens when I select High Performance mode for my switchable
graphics AMD Radeon 7000m.
—
Reply to this email directly, view it on GitHub
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556032997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJ3EHAV4PUGQAUWJXWMHYLXHFBYNANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you commented.Message ID:
@.***>

I suppose the issue is related to compatibility. The legacy graphics cards
Intel HD Graphics 3000 and AMD Radeon 7400m do not work out of the box
in Windows 10+; therefore, there is no OpenGL service.
A technical, translator, and moderate Sunni Muslim. The antidote of
hypocrites, disbelievers and grooming perisher.
On Sun, 21 May 2023 at 18:43, deton24 @.***> wrote:

Are you sure that's the problem bound to OGL?
Some games or apps using OGL don't work for you?
Maybe just use DDU, go to safe mode, uninstall all the GPU drivers (Intel,
and then AMD), and reinstall them from scratch.
niedz., 21 maj 2023 o 00:39 Mohammed Mehdi TBER @.***>
napisał(a):

You might want to check
https://github.com/pal1000/mesa-dist-win/releases
to wrap OpenGL to Vulkan on Windows, but I'm not sure if OGL is the
culprit
of this error at all.
I tried mesa drivers but It does not work.
This error happens when I select High Performance mode for my switchable
graphics AMD Radeon 7000m.
—
Reply to this email directly, view it on GitHub
<
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556032997
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AIJ3EHAV4PUGQAUWJXWMHYLXHFBYNANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556237767,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASU2BOGSLTOD3RMS47L3Q7DXHJH5HANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Try out reading about Amernime Drivers. Maybe there are still modded
drivers for your card.
Also, the fact you somehow installed the drivers from e.g. Windows 7 or 8
on Windows 10, doesn't necessarily mean that OGL doesn't work, if you
actually installed the drivers.
Can you run any other OpenGL applications or games?
pon., 22 maj 2023, 10:30 użytkownik Mohammed Mehdi TBER <
@.***> napisał:

I suppose the issue is related to compatibility. The legacy graphics cards
Intel HD Graphics 3000 and AMD Radeon 7400m do not work out of the box
in Windows 10+; therefore, there is no OpenGL service.
A technical, translator, and moderate Sunni Muslim. The antidote of
hypocrites, disbelievers and grooming perisher.
On Sun, 21 May 2023 at 18:43, deton24 @.***> wrote:

Are you sure that's the problem bound to OGL?
Some games or apps using OGL don't work for you?
Maybe just use DDU, go to safe mode, uninstall all the GPU drivers
(Intel,
and then AMD), and reinstall them from scratch.
niedz., 21 maj 2023 o 00:39 Mohammed Mehdi TBER @.***>
napisał(a):

You might want to check
https://github.com/pal1000/mesa-dist-win/releases
to wrap OpenGL to Vulkan on Windows, but I'm not sure if OGL is the
culprit
of this error at all.
I tried mesa drivers but It does not work.
This error happens when I select High Performance mode for my
switchable
graphics AMD Radeon 7000m.
—
Reply to this email directly, view it on GitHub
<

https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556032997

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AIJ3EHAV4PUGQAUWJXWMHYLXHFBYNANCNFSM6AAAAAAXRB43MU

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556237767
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ASU2BOGSLTOD3RMS47L3Q7DXHJH5HANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556775972,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJ3EHCSD5XU6JDXT2RFISTXHMP3BANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you commented.Message ID:
@.***>

Amernime Drivers do not differ from Windows Update drivers. All OpenGL apps
do not work if I set the power plan for them to High Performance. Also,
when I set the Switchable-Graphics power plan to Maximum Performance, I
faced red artifacts, and the computer threw a blue screen after that. I can
only choose High Performance but still, it does not affect OpenGL apps
until I manually choose the power plan per app, and when that happens, It
throws that previous error.
A technical, translator, and moderate Sunni Muslim. The antidote of
hypocrites, disbelievers and grooming perisher.
On Mon, 22 May 2023 at 14:28, deton24 @.***> wrote:

Try out reading about Amernime Drivers. Maybe there are still modded
drivers for your card.
Also, the fact you somehow installed the drivers from e.g. Windows 7 or 8
on Windows 10, doesn't necessarily mean that OGL doesn't work, if you
actually installed the drivers.
Can you run any other OpenGL applications or games?
pon., 22 maj 2023, 10:30 użytkownik Mohammed Mehdi TBER <
@.***> napisał:

I suppose the issue is related to compatibility. The legacy graphics
cards
Intel HD Graphics 3000 and AMD Radeon 7400m do not work out of the box
in Windows 10+; therefore, there is no OpenGL service.
A technical, translator, and moderate Sunni Muslim. The antidote of
hypocrites, disbelievers and grooming perisher.
On Sun, 21 May 2023 at 18:43, deton24 @.***> wrote:

Are you sure that's the problem bound to OGL?
Some games or apps using OGL don't work for you?
Maybe just use DDU, go to safe mode, uninstall all the GPU drivers
(Intel,
and then AMD), and reinstall them from scratch.
niedz., 21 maj 2023 o 00:39 Mohammed Mehdi TBER @.***>
napisał(a):

You might want to check
https://github.com/pal1000/mesa-dist-win/releases
to wrap OpenGL to Vulkan on Windows, but I'm not sure if OGL is the
culprit
of this error at all.
I tried mesa drivers but It does not work.
This error happens when I select High Performance mode for my
switchable
graphics AMD Radeon 7000m.
—
Reply to this email directly, view it on GitHub
<

https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556032997

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AIJ3EHAV4PUGQAUWJXWMHYLXHFBYNANCNFSM6AAAAAAXRB43MU

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<

https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556237767

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ASU2BOGSLTOD3RMS47L3Q7DXHJH5HANCNFSM6AAAAAAXRB43MU

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1556775972
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AIJ3EHCSD5XU6JDXT2RFISTXHMP3BANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1557223853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASU2BOB7XRTYDTGQQAYJWSLXHNSW5ANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you authored the thread.Message ID:
@.***>

So when you set your Radeon switchable iGPU to Maximum performance, OGL works, but artefacts you encounter then, causes blue screen after some time.
Didn't you try to completely disable your Radeon card instead so only Intel HD Graphics 3000 would be used. Maybe it doesn't have OGL support broken, as for your Radeon 7400m.

So you've fixed your issue by this hook and only GPU processing doesn't
work now?
It's normal, it only works with Nvidia cards (min. maybe GTX 6XX, I'd need
to check) so you're force to use CPU at this point.
czw., 25 maj 2023, 14:59 użytkownik Mohammed Mehdi TBER <
@.***> napisał:

Intel HD Graphics does not have OpenGL support by default in Windows 10,
and I am using it alone, actually. However, to make OpenGL works, I hooked
some apps using 'WIN81RTMVersionLie' yet It seems that the OpenGL (maybe
version 3x) of this card is older than Radeon's one which is version 6x and
still It does not support your app (GPU button is greyed out) Also, it was
extremely slow in Topaz Photo AI let alone Video AI. I also still face
YouTube frame drops while using Chrome (with OpenGL as forced backend and
VP9 as the codec).
A technical, translator, and moderate Sunni Muslim. The antidote of
hypocrites, disbelievers and grooming perisher.
On Thu, 25 May 2023 at 11:32, deton24 @.***> wrote:

So when you set your Radeon switchable iGPU to Maximum performance, OGL
works, but artefacts you encounter then, causes blue screen after some
time.
Didn't you try to completely disable your Radeon card instead so only
Intel HD Graphics 3000 would be used. Maybe it doesn't have OGL support
broken, as for your Radeon 7400m.
—
Reply to this email directly, view it on GitHub
<
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1562670356
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ASU2BOG5GUO36TSP2XISTGLXH4YMHANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1562866052,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJ3EHF6VHJFRK4RCM5IPJ3XH5JTZANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you commented.Message ID:
@.***>

Actually, it already functions without hooking, but disabling Radeon is
necessary to ensure that. I was curious if Radeon can enhance the
separation process, but it appears that your software only supports CUDA,
as you mentioned. It also lacks support for OpenGL, despite encountering
the typical OpenGL error that occurs with OpenGL programs.
A technical, translator, and moderate Sunni Muslim. The antidote of
hypocrites, disbelievers and grooming perisher.
On Thu, 25 May 2023 at 23:49, deton24 @.***> wrote:

So you've fixed your issue by this hook and only GPU processing doesn't
work now?
It's normal, it only works with Nvidia cards (min. maybe GTX 6XX, I'd need
to check) so you're force to use CPU at this point.
czw., 25 maj 2023, 14:59 użytkownik Mohammed Mehdi TBER <
@.***> napisał:

Intel HD Graphics does not have OpenGL support by default in Windows 10,
and I am using it alone, actually. However, to make OpenGL works, I
hooked
some apps using 'WIN81RTMVersionLie' yet It seems that the OpenGL (maybe
version 3x) of this card is older than Radeon's one which is version 6x
and
still It does not support your app (GPU button is greyed out) Also, it
was
extremely slow in Topaz Photo AI let alone Video AI. I also still face
YouTube frame drops while using Chrome (with OpenGL as forced backend and
VP9 as the codec).
A technical, translator, and moderate Sunni Muslim. The antidote of
hypocrites, disbelievers and grooming perisher.
On Thu, 25 May 2023 at 11:32, deton24 @.***> wrote:

So when you set your Radeon switchable iGPU to Maximum performance, OGL
works, but artefacts you encounter then, causes blue screen after some
time.
Didn't you try to completely disable your Radeon card instead so only
Intel HD Graphics 3000 would be used. Maybe it doesn't have OGL support
broken, as for your Radeon 7400m.
—
Reply to this email directly, view it on GitHub
<

https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1562670356

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ASU2BOG5GUO36TSP2XISTGLXH4YMHANCNFSM6AAAAAAXRB43MU

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1562866052
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AIJ3EHF6VHJFRK4RCM5IPJ3XH5JTZANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Anjok07/ultimatevocalremovergui/issues/527#issuecomment-1563602245,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASU2BOEDWJLXQA56BT2KJK3XH7OZLANCNFSM6AAAAAAXRB43MU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Refactor Configuration

Note:

https://github.com/graphprotocol/indexer/pull/659 refactors some changes from this PR, most notably the input validation module.
Reviewing https://github.com/graphprotocol/indexer/pull/659 instead of this PR for the latest refactorings is advised.

Refactors configuration to accept the following options as lists instead of single values:

--network-provider (new name for ethereum, which is now an alias)
--epoch-subgraph
--network-subgraph-endpoint
--network-subgraph-deployment

Introduces:

a validation function validateNetworkOptions to sanitize the new cardinality of the network options.
a input-parsing.ts module with parsers for the new network identification format.

To ensure compatibility, this PR restricts its scope to argument parsing and does not incorporate the network multiplexing changes expected from #652. which will resolve all the FIXMES introduced here.

New dependencies:

parsimmon: for the new input-parsing.ts module.
lodash-countby: used in the new validateNetworkOptions function.

Closes #650

looks pretty good! how do you think untaggedUrl would be used?

untagged inputs will be used if an indexer prefers to stick to a single network configuration.
Indexers will likely use untagged and single network configurations for the upcoming release, as using multiple networks will block the next database migration.

https://github.com/graphprotocol/indexer/pull/659 refactors some changes from this PR, most notably the input validation module.
Reviewing https://github.com/graphprotocol/indexer/pull/659 instead of this PR for the latest refactorings is advised.

Resolved by https://github.com/graphprotocol/indexer/pull/668


pull request for e2e test at 01/28/2017 05:26:25

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[WIP] Separate HTML element into their own package

Closing this for now... not comfortable this is the right separation.


Running kitchen test managed should do something

Per the README.md kitchen test managed should be able to run.  This is not the case.
(chef) .../effortless/effortless-infra-migration git:master $ kitchen test managed
-----> Starting Kitchen (v1.23.2)
>>>>>>
No instances for regex `managed', try running `kitchen list'

Usage:
  kitchen test [INSTANCE|REGEXP|all]

Options:
  -c, [--concurrency=N]                        # Run a test against all matching instances concurrently. Only N instances will run at the same time if a number is given.
  -p, [--parallel], [--no-parallel]            # [Future DEPRECATION, use --concurrency] Run a test against all matching instances concurrently.
  -d, [--destroy=DESTROY]                      # Destroy strategy to use after testing (passing, always, never).
                                               # Default: passing
      [--auto-init], [--no-auto-init]          # Invoke init command if .kitchen.yml is missing
  -D, [--debug], [--no-debug]                  # Run the converge and verify with debugging enabled.
  -t, [--test-base-path=TEST_BASE_PATH]        # Set the base path of the tests
  -l, [--log-level=LOG_LEVEL]                  # Set the log level (debug, info, warn, error, fatal)
      [--log-overwrite], [--no-log-overwrite]  # Set to false to prevent log overwriting each time Test Kitchen runs
      [--color], [--no-color]                  # Toggle color output for STDOUT logger

Description:
  The instance states are in order: destroy, create, converge, setup, verify, destroy. Test changes the state of one or more instances to destroyed, then executes the actions for each state up to destroy. At any sign of failure, executing the actions stops and the instance is left in the last
  successful execution state.

  There are 3 post-verify modes for instance cleanup, triggered with the `--destroy' flag:

  * passing: instances passing verify will be destroyed afterwards.

  * always: instances will always be destroyed afterwards.

  * never: instances will never be destroyed afterwards.
(chef) .../effortless/effortless-infra-migration git:master $

The README was written optimistically -- that recipe hasn't actually been written.  It's very much a more complex target.

@ncerny given the immaturity of code and documentation in this repo, we've chosen to go in another direction with our migration solution.  I'll close this and my other issues.


fetch from kitsu api if osu!api key is not defined

If the user has no osu!api key defined, then use the Kitsu API.

You can get and use your api key even if you has been restricted, there's no reason to relay on 3rd party service

You can get and use your api key even if you has been restricted, there's no reason to relay on 3rd party service

also even if your account is disabled

Overall, relaying on 3rd party service can't be same 1:1 to regular osu!api and can lead to unexpected problems, outage or maintenance of the service which can negatively reflect on user expirience.

I bet you didn't even try it, and yet you are judging about it.
I have this running with USSR on my personal Server since about 1 Month and i don't experience any issues.

Overall, relaying on 3rd party service can't be same 1:1 to regular osu!api and can lead to unexpected problems, outage or maintenance of the service which can negatively reflect on user expirience.

I bet you didn't even try it, and yet you are judging about it. I have this running with USSR on my personal Server since about 1 Month and i don't experience any issues.

Yes, you're right i did not try it but i just spitting my opinion on using 3rd party services at all, however in this situation using of 3rd party services it's absolutelly unnecessary

Overall, relaying on 3rd party service can't be same 1:1 to regular osu!api and can lead to unexpected problems, outage or maintenance of the service which can negatively reflect on user expirience.

I bet you didn't even try it, and yet you are judging about it. I have this running with USSR on my personal Server since about 1 Month and i don't experience any issues.

Yes, you're right i did not try it but i just spitting my opinion on using 3rd party services at all, however in this situation using of 3rd party services is absolutely unnecessary

But though this is an option for users who don't want to enter their osu!api key, or may experience issues with the main api.
If you don't want to use the Kitsu api then just enter the api key and it's all good.
This is just an alternative.

Overall, relaying on 3rd party service can't be same 1:1 to regular osu!api and can lead to unexpected problems, outage or maintenance of the service which can negatively reflect on user expirience.

I bet you didn't even try it, and yet you are judging about it. I have this running with USSR on my personal Server since about 1 Month and i don't experience any issues.

Yes, you're right i did not try it but i just spitting my opinion on using 3rd party services at all, however in this situation using of 3rd party services is absolutely unnecessary

But though this is an option for users who don't want to enter their osu!api key, or may experience issues with the main api. If you don't want to use the Kitsu api then just enter the api key and it's all good. This is just an alternative.

If you had written this originally in the description of the pr, there would be no questions to you because the concept of "if not defined" is quite vague. But anyway, i'll stay stick to my opinion on that because i can't think of real world reason why you would not use regular osu api.

I understand where alo is coming from, the plan with bpy really is to have as little reliance as possible on any external sources - if someone doesn’t provide an API key and kitsu is offline or shut down or something else which prevents it from working, what do we do now? bancho will be running for aslong as the game does, which is aslong as we need to guarantee that we have a working API. that is why it is better to force an API key and use bancho, it is a safety guarantee.

hm tbh i think it's a valid usecase for people that aren't comfortable hooking their osu api key up to a server where any user can abuse it, but i might add a warning or something at startup to people are aware they'll have slightly less-reliable beatmap info

also, perhaps i can add option to hook up multiple osu api keys and ratelimits on a per-key basis to help with abuse potential in general

if someone doesn’t provide an API key and kitsu is offline or shut down or something else which prevents it from working, what do we do now?

they provide an osu!api key and have the same functionality as now - this is only an additive layer for if one is not provided

if someone doesn’t provide an API key and kitsu is offline or shut down or something else which prevents it from working, what do we do now?

they provide an osu!api key and have the same functionality as now - this is only an additive layer for if one is not provided

my hypothetical was if they didn't provide one either though, we should still force an api key either way if the user wants any reliable beatmap info


Mast: Watch iconic.com.bd

Mast requests the watch of the watch_keyword iconic\.com\.bd. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by tripleee in Charcoal HQ


Update Action to Node 20

I saw node16 in action.yml and that made me create this PR.

Will cut a release for this soon!


Digitally sign the PowerRename shell extension dll

Currently it's not signed.

@crutkas
this should be done in this milestone as part of the work to update the release pipeline.

@enricogior i'm totally not aware how to do this "CI" part and would like to work on this. "Digitally sign" i have no clue, would please give me some more detail

This is something we have to do, not something you can.

@nishp77
I'm taking care of this, thanks,

Fix available in 0.15.0 https://github.com/microsoft/PowerToys/releases


Add localisation support

Would it be possible to add localisation for all the different hardcoded elements?
Like the the clock and texts like "Feels like" and "chance of rain" in the weather tile and other hardcoded text strings.

It will be better to make config more custombizable and let define all "hardcoded" string right into tiles.

Agreed :)

But i have know idea how to do it without mess of code. Weather tile already has a lot of fields. IMO it should has custom lines like text_list tile, but some fields are still predefinded, like temperature, summary.


Identify AWS Services

I am thinking Elastic Beanstalk, RDS, Route53, IAM, TurnKey CouchDB, Cloudwatch, CloudFormation, and SES... is this a good selection? Am I missing anything?

Other services we should probably have:

Certificate Manager (makes deploying SSL and managing your certs easy)
WAF (Layer 7 firewall)

Some nice-to-have-but-not-necessary services:

Cloudtrail (audit trail of every api call made)
Glacier (cold off-line storage, like tape, very cheap)

This is some of the info that CloudTrail gives you btw:

Nice!

For Glacier, would that be for document backups? Is it HIPPA friendly?
RDS does backups on its own... don't need Glacier for that, right?

Glacier would just be for long term retention, like a lot of documents, such as my university health records, are kept for 7 years. I can only imagine this being a US / Western Europe requirement.
So Glacier is not for backups, but it's for data you just can't delete but will probably never use.
Also I don't know a lot about healthcare (although I try!), so if you're ever like "wtf are you saying Daniel no one does that in healthcare" just let me know...
If we used that CouchDB EC2 images...you would do backups by taking snapshots of the EBS volumes which then get sent to S3. You can then delete this or also tier them into Glacier (so after 30 days you delete the backup in S3 and then move it to Glacier, where you keep it forever or maybe delete it from even Glacier after 90 days).
[CouchDB-EBS Volume]--->S3 Snapshot--->Glacier--->Delete?
RDS does backups as well to S3 like above.

Thanks for the detailed answer. Although I do work in healthcare, I can't claim to know all of the fine print details of every regulation... however, my understanding is that it depends on where you live in terms of max retention times. I would say, if possible, we should keep every document for each patient around forever. While this may be a bit aggressive, having a centralized place that captures a patient's entire medical history is rather important.
Maybe I'm naive, but can't we just write a node cron job that backs up CouchDB to S3 every night (limiting the backup count to 3)?


return type Subscription instead of Observable - typescript

Hi,
I'm using ionic 2 and in this example from docs is probably a mistake if one uses typescript. this.meteorSubscription returns Subscription type instead of Observable
  this.meteorSubscription = MeteorObservable.subscribe<any>("myData").subscribe(() => {
          // Subscription is ready!
       });
    }

@msio777 , can you please open an issue in https://github.com/Urigo/meteor-rxjs repository?
Thanks!

@msio777
The return value of MeteorObservable.subscribe is Observable, and then you need to subscribe on the return value to get Subscription.
The docs are correct.


make nyc cover all source files including non-strict mode agents

move nyc and mocha config from long cli args to inside package.json configs

Code coverage is decreasing because I fixed the config to ensure all source files are measured. A lot of them (e.g. the agents in agent/) don't have tests yet.


Add Blog Post on Programming Language Selection for DSA

📥 Pull Request
Description
As a web developer with a growing skill set, I am excited to make my first contribution to this project. In this pull request, I have added a new blog post to the blog section of the web app for algorithms. This blog discusses the importance of selecting the right programming language for Data Structures and Algorithms (DSA), providing valuable insights and guidance for users on their learning journey.
I believe this addition will enhance the educational resources available to users and improve their overall learning experience. I am also actively working on additional features for this project.
Fixes #
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Just a Glimpse of Blog Page

Thank you so much


#feature; Add liquidation revenue to aave

What do you think is missing?
Aave liquidations have a liquidation revenue. That should be considered protocol revenue.
Describe the solution you'd like
Start tracking liquidation revenue. Ideally we upgrade to schema 3.0.0 and then we can also see the source of all revenue.

@dmelotik , I am assigned both tasks - this one and aave-forks upgrade. do you prefer to do this one first before updrading?

The upgrade will not make it unnecessary. Regardless of the upgrade we don't currently have logic in the subgraph to calculate liquidation revenue. Something I missed when I first did the aave forks directory.
I think this can be done separately or concurrently with the upgrade to 3.0.0. Whatever makes more sense to do in your workflow, so it is up to you!

I see, that's helpful! I prefer to add liquidation revenue concurrently with the upgrade if that's OK.


YUV support

And other pixel formats too I guess

I suppose 10 bit colour support could also be part of this.

See https://github.com/swaywm/wlroots/pull/724

Mesa provides YUV support for linux-dmabuf.


[terminal completions]: broken in bash/zsh on Unix

Follow up on: https://github.com/microsoft/vscode/issues/235021
The completions seem to be broken on Unix machines, in bash/zsh terminals. Please see https://github.com/microsoft/vscode/issues/235021#issuecomment-2515627451 for more info.

Broken how? Do they ever show up?

@meganrogge I think they just didn't work at all, see https://github.com/microsoft/vscode/issues/235021#issuecomment-2515627451

This will be hard to investigate because will need to use a vm

Can reproduce on windows using WSL, nothing works at all

Interestingly, it doesn't show the No suggestions widget either, though I see the command is correctly being invoked. It's like the widget doesn't get created 🤔

wondering if maybe this will fix it https://github.com/microsoft/vscode/pull/236630


Update DataframeBuilder.scala

Add _platform_ingested_at column to DataframeBuilder. It add current date to DateFrame.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@6f3caec). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             develop      #19   +/-   ##
==========================================
  Coverage           ?   47.65%
==========================================
  Files              ?       22
  Lines              ?      405
  Branches           ?       33
==========================================
  Hits               ?      193
  Misses             ?      212
  Partials           ?        0

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


Refactore code by modules

Refactore Product/Schedule module functions
Detail:

Product

create imageService:

implement saveImage()
implement getImageUrls()
implement getMainImageUrl()
implement deleteImage()

refactor create():

validation update

check if start_at > end_at

integrate imageService->saveImage()
enable receive of empty start_at and end_at variable
enable forced data of status variable push

refactor read():

validation update

require id path-param
check if id exists in the db at the first point

integrate imageService->getImageUrls() method from the service

refactor index():

add condition search features:

add keyword search
add price range search
add publishDate(start_at and end_at) range search
add status search
add custom_field search

integrate imageService->getMainImageUrl() method from the service

refactor update():

validation update

require id path-param
check if id exists in the db at the first point
check if start_at > end_at

enable updates for the following fields:

name
description
price
start_at
end_at
custom-fields
dataUrls (array format should be as it is now, require all fields with the "https" URL if no update)

Schedule

search(): no refactoring for now :+1:
create():

validation update:

require product_id param
check if product_id exists in the db at first point

update request data format:

param->userInfo

name
email
customFields (JSON format only)

param->scheduleInfo

product_id
start_at
end_at
total_fee
customFields (JSON format only)

create slackService file

create slackService->sendAdmin()
call slackService->sendAdmin() in the Schedule::create()

note:
during the integrate imageService->getImageUrls() method from the service update, image link became inaccessible for some reason.
solution:
recreate storage link:
rm public/storage
php artisan optimize:clear
php artisan storage:link


Add tests exercising fundchannel

https://github.com/lightninglabs/lightning-terminal/pull/865 corrects the fundchannel functionality to allow a channel to be created of an amount that is less than the cumulative available asset amounts.
MVP TODO

[x] test whether multi-input asset funding of a custom channel works

Test-dimension scope to consider:
Positive and negative tests exercising this functionality shall be added asserting channel funding given:

Single, Multiple asset UTXOs
A fund_channel size less than, equal, or greater than, the input amounts

#882 fixes this. I'm closing this issue.


disabled property / readonly tree???

Hi,
I did not found any resource to disable/enable/read only entire tree and i need to suggest you to implement this feature on the new release.
If my object was disabled (property disabled), the entire tree disabled and any callbacks are ignored.
I developed new features to improve my use:
-On checked (or clicked) on the item with children and this item is expand automatically or retracts when is unchecked;
I hope you enjoy my suggestions
Kiosaninsl

Hi,

In zTree, the checkbox or radio is not belong the  edit model.

When you want to edit, please see: 'setting.edit.enable'.
When you want to not allow check the node, please see: 'setting.callback.beforeCheck'

When you want to click the node and it was expand automatically, please use 'setting.callback.onClick'  && 'treeObj.expandNode()'

Hello,
I think I expressed myself poorly. It would be interesting to have a disabled throughout the tree, preventing any event the same (callbacks, expands, clicks, etc) without the need to monitor all events tree.
The mode "read only" would expand, reduce and roll the tree but not allow any type of change (edit, check or radio item changed, etc.) without the need to monitor all events manually.
Being that I could set the disabled property [jquery prop ('disabled, true)] in the div tag and the component would modify the object's behavior.
I managed to do this with the events but the work was very manual and I would like to help implement these features.
Thank you

you can add this, by your self.
example:

you make some function to check the readonly/edit mode,  it will return true or false.  the  rule is any thing by your need.

you can add the function to the all callback 'before...' what you want to controll.

so, when you change the mode to readonly,  the all callback 'before' will return false.


View/Template Helper

Hey guys,
It would be great if there was a way to blend in some view/template helpers.
myView = Backbone.View.extend({
template : "my_view",
templateHelper : someHelperFunctions
...
If the templateHelper attribute is defined, extend the template's data with this object literal.
Would be really handy, rather than having to normalize your view data elsewhere.
Thoughts?
Thanks!
Dan

Not all template engines work this way, but I'm glad you found a way to extend LayoutManager to work for your use case!


Fix/add ability to set pub sub decorators together

PR Checklist
Please check if your PR fulfills the following requirements:

[-] The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
[+] Tests for the changes have been added (for bug fixes / features)
[-] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?
This PR address this issue, that was initially evaluated as feature. https://github.com/flamewow/nestjs-asyncapi/issues/465
[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?
Issue Number: 465
What is the new behavior?
AsyncApiPub and AsyncApiSub decorators now creates method decorator with different meta keys
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@MaksimKiselev thanks for handling that thing
https://www.npmjs.com/package/nestjs-asyncapi/v/1.1.1

Any idea why my subs are returning an object instead of an array here after this change?
https://github.com/flamewow/nestjs-asyncapi/pull/473/files#diff-bc32787f3b434f34f1cf6480e670abbcd25092915b971c0f234579c24c8379b6R16
{
  '0': {
    channel: 'test',
    message: {
      payload: [Object],
      name: 'HealthController#getHealth',
      headers: undefined
    },
    type: 'sub'
  }
}

as a result I get this error:
const metadataCombined = [...metadataOperations || [], ...metadataSubs || [], ...metadataPubs || []];
                                                                           ^
TypeError: (metadataSubs || []) is not iterable
    at exploreAsyncApiOperationMetadata (/nestjs-asyncapi/dist/lib/explorers/asyncapi-operation.explorer.js:23:76)

I tried reproducing the same behaviour in the sample app but I couldn't, it returns an array as it should.

@Pedr0Rocha are you build your nest app with tsc compiler?
Can you please provide your tsconfig* files, package.json (deps, devDeps)?

It was fixed here: https://github.com/flamewow/nestjs-asyncapi/pull/494/commits/5069e2210f5ecd8e87c35e4615c2a066b298252b
Thanks


Use MAL to calculate CLR metrics, remove OAL dependency

More metrics analysis should be moved into MAL's responsibility.

Closed by https://github.com/apache/skywalking/pull/7091


ROS node setup error

I believe that python 2.7 is standard in ROS melodic, but pip install asks for pip3.
Please let me know if you have any ideas.
Environment

ubuntu 18.04
ros melodic

~/workspace/catkin_ws/src/SFA3D master
❯ pip install .
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. A future version of pip will drop support for Python 2.7. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
Defaulting to user installation because normal site-packages is not writeable
Processing /home/yukihiro/workspace/catkin_ws/src/SFA3D
ERROR: Package 'sfa' requires a different Python: 2.7.17 not in '>=3.6.0'

hi @yukkysaito
Actually, I have created a virtual environment with only python3.6, so I can use pip install .
You can try pip3 install .

Hi @yukkysaito
How is it going? If it works, please close the issue. Otherwise, we can help you to solve it. @AhmedARadwan

Hi @yukkysaito, Can you try using pip3 install . after the installation of pytorch?

It's not an issue of the source code.

This might not be a problem with the source itself, but with the README of the ROS part and therefore it is still a bug and should be reopend:
The ros README states that on Ubuntu 18.04 and with rosmelodic you would only need to run pip install . in order to install the dependencies. But in 18.04 the standard python interpreter is 2.7 and therefore pip tries to install dependencies for  2.7. but fails as setup.pyof SFA3D requires >=3.6
Ubuntu 18.04 ships with python 3.6 as python3 with pip3. Simply using pip3 install . works for installing the packages but it would be still not possible to run it with standard ROS Meldoic (which is by design meant to be python2 only). There are hacky ways on how to execute a python3 code within ros. But those sould be explained in the README. I have setup a virtualenv trying to solve the problem. It kinda worked but I got not objects published (might be another bug/misconfiguration by myself). Anyway, the README is missing what @AhmedARadwan actually did from a clean system to get this working. It would be very  great if you could check that.

Thanks @nullket
@AhmedARadwan Can you check the issue and update the instructions in README?

@maudzung okay I'll edit the README asap


ApineFileImage is broken

ApineFileImage does not follow the same structure as ApineFile, and the functions used in ApineFileImage do not work anymore with its parent. Therefore, the class should use a rewrite in my opinion.

What do you mean by "should use a rewrite"? And also can you give us more details on errors you are getting with ApineFileImage?

We indeed found many issues inside ApineFileImage. We found that it wasn't saving properly modifications, cropping was actually resizing and there was issues with load the resource.

Considering there's no further comments, I close this issue.


Would need to be very strict about picture size. Lets say 120x120, no more no less to keep load down.
Picture attached is 120x120.

I'll second this. To have a basic image of the item.

If someone wants to try in the pre-release demo:

A picture can be uploaded on the product edit screen
On the stock overview page there is a new button next to consume which shows the picture of the product (only enabled when the product has a picture)

No demo data so far, I uploaded the picture above for product "Cucumber".
Still needs some polish, that's the first time we handle files, so also security things, file size checks, and so on (please don't overload my demo server :D).
The files are saved in data/storage/productpictures.

Does not like long file names. Gives you the error of maybe duplicate.
But I don't consider this a bug. Just a thing to note.
Maybe more accurate error messages?

Changes since yesterday:

Better file name validation
Finished product edit page (added also possibility to remove a picture)
On the stock overview page there is no separate button to show the picture, instead a little image icon next to the product name apperas and the whole name cell is clickable to show the picture, if the product has one
The picture is also shown in the productcard (purchase/consume/etc. pages)

To workaround duplicate file names, just some random characters are prepended to the file name during the upload - I think that should be enough...
Let me know if you find any bugs or have any further/other ideas on this. :)

Put picture in the center maybe?

Sure ... I'm happy when the rest seems to work... :)
Maybe I'm just a little worried about the (security) of the file upload API...

Should not be possible to serve file from outside the data/storage directory (should already be fine because / is disallowed in the requested file name)
The file is handled on both server and client side completely in RAM, no chunking or something and also no file size checks (only configured limits from PHP or the webserver will apply)

=> So means is only good for small files as of now, so for product pictures it will be ok I think...

I added a button in the dialog to jump to the product edit screen (and back on saving the product)...
Centering the title I did not get to work...

Thanks, yes that would affect every modal/dialog - I limited it to that one in https://github.com/berrnd/grocy/commit/ae58606d04f90be687b00ee1fb23fe3fc71ac116.

Something to note about the picture upload feature. It will accept any file. Not sure about the file size, but tried 25mb and that worked fine.

The file API accepts all file types, but preselcting only image files in the file picker is possible - done in https://github.com/berrnd/grocy/commit/f90faca62ee0a3b346cb1b7f51d8722971313edf.

I have a small addition to this. When editing products it would be useful with a small icon that shows if the product actually has a picture or not.
Please have a look at my example.


Fixing typo in make debug

PR #1294 actually has a fix for this. There is one more typo in make debug than what is shown here.


Phaser stops working when the browser tab is inactive for a while

I noticed in OSX Google Chrome (version 44.0.2403.155 (64-bit)) that Phaser stops working with the following error when the browser tab is inactive for a while. (The screen shows only a blank black screen.)
Uncaught TypeError: Cannot read property 'image' of undefined

The line with the error is in clearGLTextures, "for (var key in this.cache.image)"
Is there anyone that has come across the same issue?

Sounds like the WebGL context is being destroyed by Chome. Not that I've ever seen it do this - anyone else?

Yes! I was run only:
new Phaser.Game(800, 200, Phaser.Canvas, 'game');

next day I seen the same error message occur.

@piecioshka I have a question. Was your disableVisibilityChange set to true or false?
E.g. Did you set
game.stage.disableVisibilityChange = true;

I have not seen it for a while since I set disableVisibilityChange to false (I think that it is set to false by default).

No, I run only one line that I mention.

@BeFiveINFO I was used your line, but it doesn't help me, a have the same error:
phaser.js:71831 Uncaught TypeError: Cannot read property 'image' of undefinedPhaser.Cache.clearGLTextures @ phaser.js:71831Phaser.Game.contextRestored @ phaser.js:38874

Thank you for the feedback. Then disableVisibilityChange might have little or nothing to do with it.

So how long does it take before it destroys the context? If we're talking a day here then I'm honestly not sure it matters.

It varies. Sometimes it took only half an hour or maybe less. I notice that the problem can be seen also in the example demos of the phaser site.
I often keep example pages in a tab while searching for the forum on google and so on. I noticed that the phaser screen in the example pages sometimes (not always) goes black and stops working while it was inactive in background.

It would be nice if it handled it gracefully, but I'm still not that bothered about the fact that it does actually happen. It's an edge case for sure.

Thank you for paying attention!
I will post findings as soon as I find anything else about the problem.

Thank you for spending time to open this issue. I believe it has been discussed as far as it can be, and no immediate code change will be involved, so I'm closing it. Feel free to submit a PR if you feel you can resolve it in a clean and API friendly way.

I believe that have the same problem. And i found a way how i can reproduce it.
I open my website where my app  on Mobile Chrome.
Then i switch applications on mobile device and return.
For example:

Open website
Switch app to Skype, switch to Camera then to .... (3 apps can be enough for my mobile)
Return to Chrome

If I switched too many apps and return, i see Chrome Refreshing.
If I switched not enough apps everything's okay
But between this conditions i see a bug with rendering.
I believe that it can be because of cleaning of memory by browser (but before browser automatic unloading from memory).
I can't find a way how to solve this problem...
Can you help me? ))
Thanks!


[IOTDB-1174] optimize mergechunks thread number

if merge path is smaller than mergeChunkSubTaskNum, will use merge path number.
so threads are not wasted.

@zhanglingzhe0820 can you take a look?

Thanks lingzhe review.
hi @HTHou , linzhe has approved,  could you please reivew and merge?


Black formatter fails if environment has an older version of black installed

My Python environment has black version 19.10b0 installed, which is less than the minimum 22.3 stated in the black formatter extension docs. But since the newer version is bundled with the extension, the 22.3 version should be used if the environment installed version is too low. Here is the black formatter output when I try formatting the document:
Formatter Name: Black
Formatter Module: black
CWD Format Server: /Users/aldcroft/git/sparkles
sys.path used to run Formatter:
    /Users/aldcroft/.vscode/extensions/ms-python.black-formatter-2022.1.11291006/bundled/formatter
    /Users/aldcroft/miniconda3/envs/ska3/lib/python38.zip
    /Users/aldcroft/miniconda3/envs/ska3/lib/python3.8
    /Users/aldcroft/miniconda3/envs/ska3/lib/python3.8/lib-dynload
    /Users/aldcroft/miniconda3/envs/ska3/lib/python3.8/site-packages
    /Users/aldcroft/.vscode/extensions/ms-python.black-formatter-2022.1.11291006/bundled/libs

Settings used to run Formatter:
[
    {
        "workspace": "file:///Users/aldcroft/git/sparkles",
        "trace": "error",
        "args": [],
        "severity": {},
        "path": [],
        "interpreter": [
            "/Users/aldcroft/miniconda3/envs/ska3/bin/python"
        ]
    }
]

black --stdin-filename /Users/aldcroft/git/sparkles/sparkles/core.py -
CWD Formatter: /Users/aldcroft/git/sparkles
[Error - 8:20:56 AM] Usage: black [OPTIONS] [SRC]...
Try 'black -h' for help.

Error: No such option: --stdin-filename

Same error here:
Formatter Name: Black
Formatter Module: black
CWD Format Server: c:\tmp\jdrive
sys.path used to run Formatter:
    c:\Users\jens.humrich\.vscode\extensions\ms-python.black-formatter-2022.1.11301002\bundled\formatter
    C:\Users\jens.humrich\Anaconda3\python39.zip
    C:\Users\jens.humrich\Anaconda3\DLLs
    C:\Users\jens.humrich\Anaconda3\lib
    C:\Users\jens.humrich\Anaconda3
    C:\Users\jens.humrich\Anaconda3\lib\site-packages
    c:\users\jens.humrich\dvl-core
    c:\users\jens.humrich\dvl-wind
    C:\Users\jens.humrich\Anaconda3\lib\site-packages\locket-0.2.1-py3.9.egg
    C:\Users\jens.humrich\Anaconda3\lib\site-packages\win32
    C:\Users\jens.humrich\Anaconda3\lib\site-packages\win32\lib
    C:\Users\jens.humrich\Anaconda3\lib\site-packages\Pythonwin
    c:\Users\jens.humrich\.vscode\extensions\ms-python.black-formatter-2022.1.11301002\bundled\libs

Settings used to run Formatter:
[
    {
        "workspace": "file:///c%3A/tmp/jdrive",
        "trace": "error",
        "args": [],
        "severity": {},
        "path": [],
        "interpreter": [
            "C:\\Users\\jens.humrich\\Anaconda3\\python.exe"
        ]
    }
]

black --stdin-filename c:\tmp\jdrive\12_analyze_onedrive.py -
CWD Formatter: c:\tmp\jdrive
[Error - 11:14:44 AM] Usage: black [OPTIONS] [SRC]...
Try 'black -h' for help.

Error: No such option: --stdin-filename

After uninstalling black via

pip uninstall black
Found existing installation: black 19.10b0
Uninstalling black-19.10b0:

This issue was resolved.

@karthiknadig - can't you set the import path so that the bundled version of black is always imported?
I'm running in a configured production environment with an older version of black and cannot just update to the latest.

@taldcroft We took this mode of allowing using environment version first after discussing with tool owners. We will discuss internally with the team and see if we can provide an around this.

There is a setting now that allows oyu to control where black is imported from. In the latest version, black is by default used from the bundle. If you need to you black from the environment. Then you can set black-formatter.importStrategy to fromEnvironment to get black from the environment.
It may show some error message about incompatibility, but it will use the version available as best as possible.


add features for custom types and custom modules

Hi all,
As discuted in issue: https://github.com/puppetlabs/puppetlabs-dsc/issues/224
Here my pull request with custom module building features.
Most of the changes can be understood in the README_BUILD.md file.

Hello @msutter ,
Apologies for the late reply and thank you for your contribution! It seems like all these features were added by this PR. In order to be absolutely sure, please rebase it with the latest code version and fix all the conflicts. :)
Cheers!

@msutter I'm sorry to bother you but I was wondering whether you had any intention of returning to work on this PR at any point within the future?

@msutter
Apologies but we will now be closing this PR, however we would like to thank you on behalf of Puppet for the work that you put into both it, and all other work you have done within the Puppet modules and express our hope to see you contribute once more in the future.
Best Wishes


Fix generic issue with SelectAria to follow useSelect

Closes #3061
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
You can use same Code Sample from the original issue.
🧢 Your Project:
No

@fitiskin mind closing and reopening this PR again? Not sure why the commit status is stuck...

I'm sorry. Fixed a problem with the docs build.

GET_BUILD


[8.x] Change the order of the bindings for a sql server query with a limit

As SQL Server moves the OrderBy part to be part of the "select" part of
the query when using a limit, we need to move the Order bindings to be in the right place.
Example
When using a subquery in the OrderBy
User::query()
    ->where('email', ':emailbinding:')
    ->orderBy(Login::select('created_at')->where('created_at', ':datebinding:')->limit(1))
    ->skip(10)->take(10),

this in the following query
select * from (select *, row_number() over (order by (select top 1 [created_at] from [logins] where [users].[name] = ? and [user_id] = [users].[id]) asc) as row_num from [users] where [email] = ?) as temp_table where row_num between 11 and 20 order by row_num

Notice the that [users].[name] = ? (part of the original order part) comes before the [email = ?] (part of the where clause) part.
The normal order of the bindings is:
    public $bindings = [
        'select' => [],
        'from' => [],
        'join' => [],
        'where' => [],
        'groupBy' => [],
        'having' => [],
        'order' => [],
        'union' => [],
        'unionOrder' => [],
    ];

When getting the bindings, this results in the order bindings coming after the where bindings.
This results in the following query:
select * from (select *, row_number() over (order by (select top 1 [created_at] from [logins] where [users].[name] = ':emailbinding:' and [user_id] = [users].[id]) asc) as row_num from [users] where [email] = ':namebinding:') as temp_table where row_num between 11 and 20 order by row_num

This pull requests changes the order of the bindings by moving order to come right after selectwhen a limit is used

Previous attempts at this over the past week have been either causing breaking changes or have been refused. So please make 100% sure that this actually fixes things without breakages on other apps.

100% is ambitious but I'll do my best.
@driesvints are there already particular cases you are worried about?
@spawnia, @Javdu10, @madbarron, @arjenvanoostrum  @Danil42Russia  I noticed you already submitted SQL Server related PR's.
Could you give this patch a try in your application and see if it causes any issues?

The currently failing test looks unrelated to this change:
 1) Illuminate\Tests\Integration\Queue\ThrottlesExceptionsWithRedisTest::testCircuitResetsAfterSuccess
Mockery\Exception\NoMatchingExpectationException: No matching handler found for Mockery_149_Illuminate_Contracts_Queue_Job::release(599). Either the method was unexpected or its arguments matched no expected argument list for this method

Without any more proof, this is my current rationale:
The possibilities of breaking anything is more limited then the other PR's this week:

this code only applies to SQL Server
this code only applies to queries with a limit

The behavior with this fix is better then the current behavior:

this only moves the position of the order bindings to be congruent with the move of the order query part
if there are no order bindings, nothing happens
if there are order bindings, the old situation

breaks obviously with an exception because of a mismatch in field types (int field gets a string binding)
or results in the wrong results because the wrong bindings are used for the wrong fields

The currently failing test looks unrelated to this change:

Think you hit a false negative. Restarted them and they look fine now.

Your PR mentions "when a limit is used" but this change is in the "offset" method? Also, does this not change the logic for all order by's when an offset is used? Were all order by clauses that also had an offset broken prior to this PR? This seems geared to a particular cause of sub-query order by's that use a limit.

@taylorotwell

Your PR mentions "when a limit is used" but this change is in the "offset" method?

My bad. This is indeed related to the "offset" part. I first noticed it when using pagination and attributed it to the limit which is clearly wrong.

Also, does this not change the logic for all order by's when an offset is used?

It does change the logic for all order by's of the "offsetted" query as all the order query logic is moved to the select.

Were all order by clauses that also had an offset broken prior to this PR?

No, only the order by clauses that had bindings (subqueries).

In other words, I'm pretty apprehensive this is going to break other more common queries that aren't currently broken.

In understand your hesitance.
Would it help if we would limit this logic to only apply when at least one of the order columns is a subquery?

I do think limiting the logic to order bys that are subqueries is safer.

👍 I've updated this PR to check for subqueries in the $query->orders first.

Hi,
I've looked at the files and tested some scenarios. The fix solves the issue mentioned (though your example in the first comment mentions :datebinding: which is not found in the resulting query, a bit confusing).
One thing I noticed is that the new binding order is not applied when getBindings()  is called before toSql(). So:
$query = User::query()
    ->where('email', ':emailbinding:')
    ->orderBy(Login::select('created_at')->where('name',':namebinding:')->where( 'user_id', 'users.id')->limit(1))
   ->skip(10)->take(10);

$query->getBindings(); // [':emailbinding:', ':namebinding:']
$query->toSql();
$query->getBindings(); // [':namebinding:', ':emailbinding:']

No issue when fired at a database by the framework, however it is confusing when debugging the bindings.

Thanks for taking a look @arjenvanoostrum

I've looked at the files and tested some scenarios. The fix solves the issue mentioned (though your example in the first comment mentions :datebinding: which is not found in the resulting query, a bit confusing).

I've fixed the example to use :namebinding: consistently.

One thing I noticed is that the new binding order is not applied when getBindings() is called before toSql(). So:
$query = User::query()
    ->where('email', ':emailbinding:')
    ->orderBy(Login::select('created_at')->where('name',':namebinding:')->where( 'user_id', 'users.id')->limit(1))
   ->skip(10)->take(10);

$query->getBindings(); // [':emailbinding:', ':namebinding:']
$query->toSql();
$query->getBindings(); // [':namebinding:', ':emailbinding:']

No issue when fired at a database by the framework, however it is confusing when debugging the bindings.

Good point. This could lead to confusion.
This could be fixed by adding logic to orderBy() and orderByRaw() or by duplicating logic to getBindings().
orderBy() already contains logic for detecting subqueries.
This would mean we would have to touch code outside of the SQL Server grammar which is unfortunate.
A reason not to go down that road is that the most common ways of debugging queries (debugbar, telescope, ray) would not be suffering from this confusion. They listen to the QueryExecuted event and toSql() has already taken place by then.
My suggestion is:

to keep it in the SQL Grammar for now
accept the possible confusion when manually debugging to be of a lesser evil then the bug
consider the more risky fix for a major release


Titanfall 2 DDS.

Tacent View can't view Titanfall 2 DDS files, 'Image Failed to Load'.

Can youpost an example dds file that fails to load.

dds_example.zip

No problem! Glad it's getting used.


change MAX_LEVEL

change MAX_LEVEL to 16
#168 shows that Level 16 is possible.

@callmenemo491 i'm lvl 18 :3

@callmenemo491 I set the max level to 25, that is what i believe is the max level what you can hit.

updating the PR to have MAX_LEVEL to 25
cheers for the info @SuicidalcookieS

PR updated.

i don't know if we actually NEED MAX_LEVEL anymore. any thoughts?

thinking about it now... i think we dont need MAX_LEVEL...
i dont know much about javascript, to be able to come up with a PR to code to remove/disable the MAX_LEVEL functionality.
if you want to go ahead with the 'disable' then i'm ok with U closing this PR.

I made a PR #174 that removes the leveling mode all together and prefers lower progress planets to play on higher difficulty zones.  Did a few zones testing and it seems to work properly, but I am having some latency issue with steam servers resulting in the script refreshing the page.


Add SameConditionalType config to Style/SoleNestedConditional

Is your feature request related to a problem? Please describe.
Add config to Style/SoleNestedConditional to check for same conditional type only. When SameConditionalType: true don't report offenses to merge if and unless.
Describe the solution you'd like
current:
# bad
if foo || !check
  unless bar && !baz
    do_something
  end
end

Such conditons are difficult to merge & I would like to avoid refactoring them if I don't have tests.
# SameConditionalType: true (default false)
# bad
if foo
  if bar
    do_something
  end
end

# good
if foo
  unless bar
    do_something
  end
end

Describe alternatives you've considered
Nothing

Even if I don't agree with your reasoning (for me it's all the same if I have to combine if and unless) I'm not opposed to making this configurable, although I'd probably wait a bit to see if someone else is interested in such functionality before adding it.


Allow assuming an AWS role in AWS MSK IAM flow

I'm integrating your tool into my AWS Kafka infrastructure, and so far it looks like a great fit, but I've stumbled upon a (seemingly) missing feature that I think could be useful.
We have a bunch of Kafka clusters in different regions, so instead of creating separate local AWS profiles for them we use a unified profile with explicit --assume-role for each of the regions/accounts. I can't make it work with topicctl though: AWS_ROLE_ARN is not documented in the AWS Go SDK and doesn't seem to do anything. The only way I could find was generating a bunch of profiles for each of the roles, which quickly gets out of hands.
There appears to be a way to assume a role programmatically in the SDK, so I'm wondering if you could offer an extra configuration option for mechanism: AWS-MSK-IAM, such as assume-role? It'd help a lot.

Yup, this shouldn't be too bad. I can take a look in a few days unless someone else wants to help and do it before then.

Ok, I have a draft of a fix in https://github.com/segmentio/topicctl/pull/67. Can you give it a try and let me know whether it works for you? Unfortunately, I don't have access to an MSK cluster at the moment to test it on. Thanks.


avoid needing a namespace cleaner by not importing anything

Using namespace::autoclean in a prerequisite of Moose is at the very
least unexpected.  With older versions, it may also explode.
To keep the namespace clean, yet avoid extra prerequisites, avoid
importing anything in the first place.

It's a lot easier to fix this by just removing the Test::CleanNamespaces plugin. As was pointed out in #moose-dev, it's not really that important for this module to have a clean namespace. I'll release a new version once Travis passes.


ONT clean minimap2/samtools error "query name too long"

command:
 nextflow run clean/clean.nf --nano data/raw/fastq_pass/2021-05-11_histoplasma_ont.fastq.gz --host hsa --control dcs --cores 36 --max_cores 72 --output clean_gridion-basecalled_fastq -profile local,singularity --singularityCacheDir singularity_images -resume

error:

Error executing process > 'clean_nano:minimap2_nano (1)'

Caused by:
  Process `clean_nano:minimap2_nano (1)` terminated with an error exit status (1)

Command executed:

  PARAMS="-ax map-ont"
  if [[ false != 'false' ]]; then
    PARAMS="-ax splice -k14"
  fi

  minimap2 $PARAMS -N 5 --secondary=no -t 36 -o 2021-05-11_histoplasma_ont.sam db.fa.gz R.fastq

  samtools fastq -f 4 -0 R.clean.fastq 2021-05-11_histoplasma_ont.sam
  samtools fastq -F 4 -0 R.contamination.fastq 2021-05-11_histoplasma_ont.sam

  samtools view -b -F 2052 2021-05-11_histoplasma_ont.sam | samtools sort -o 2021-05-11_histoplasma_ont.contamination.sorted.bam --threads 36
  samtools index 2021-05-11_histoplasma_ont.contamination.sorted.bam
  samtools idxstats  2021-05-11_histoplasma_ont.contamination.sorted.bam > idxstats.tsv

Command exit status:
  1

Command output:
  (empty)

Command error:
  [M::mm_idx_gen::82.008*2.02] collected minimizers
  [M::mm_idx_gen::93.121*2.84] sorted minimizers
  [M::main::93.121*2.84] loaded/built the index for 195 target sequence(s)
  [M::mm_mapopt_update::95.299*2.80] mid_occ = 705
  [M::mm_idx_stat] kmer size: 15; skip: 10; is_hpc: 0; #seq: 195
  [M::mm_idx_stat::96.508*2.78] distinct minimizers: 100159343 (38.79% are singletons); average occurrences: 5.540; average spacing: 5.586
  [M::worker_pipeline::107.705*4.83] mapped 236008 sequences
  [M::worker_pipeline::114.048*6.56] mapped 213499 sequences
  [M::worker_pipeline::119.300*7.87] mapped 212885 sequences
  [M::worker_pipeline::126.440*9.50] mapped 198236 sequences
  [M::worker_pipeline::132.862*10.43] mapped 208394 sequences
  [M::worker_pipeline::137.395*11.30] mapped 198667 sequences
  [M::worker_pipeline::142.667*12.23] mapped 209456 sequences
  [M::worker_pipeline::149.662*13.36] mapped 198352 sequences
  [M::worker_pipeline::153.842*13.82] mapped 208217 sequences
  [M::worker_pipeline::159.454*14.61] mapped 199491 sequences
  [M::worker_pipeline::164.832*15.32] mapped 208509 sequences
  [M::worker_pipeline::170.117*15.98] mapped 199400 sequences
  [M::worker_pipeline::175.694*16.62] mapped 208714 sequences
  [M::worker_pipeline::180.648*17.16] mapped 200316 sequences
  [M::worker_pipeline::186.048*17.72] mapped 211112 sequences
  [M::worker_pipeline::191.191*18.22] mapped 215964 sequences
  [M::worker_pipeline::199.764*18.88] mapped 227796 sequences
  [M::worker_pipeline::200.759*18.79] mapped 166159 sequences
  [M::main] Version: 2.17-r941
  [M::main] CMD: minimap2 -ax map-ont -N 5 --secondary=no -t 36 -o 2021-05-11_histoplasma_ont.sam db.fa.gz R.fastq
  [M::main] Real time: 200.978 sec; CPU: 3772.524 sec; Peak RSS: 12.177 GB
  [E::sam_parse1] query name too long
  [W::sam_read1] Parse error at line 197
  [bam2fq_mainloop] Failed to read bam record.

Work dir:
  /scratch/hoelzerm/projects/2021-05-10_BI116_Histoplasma_assembly/work/0a/fa9afda443b96162b47a043d51b748

I think the problem is that the read names are too long for samtools? But if so, why did we not discover that before?
head /scratch/hoelzerm/projects/*_assembly/work/51/d2f4a7b54b7c814bdf573db09af375/R.fastq
@11dbdf9c-e46f-4afd-b1b1-b7c3cdd2e938DECONTAMINATErunid=8842aa4e3c221ff265498dfdfdfe183253ab4499DECONTAMINATEread=9DECONTAMINATEch=247DECONTAMINATEstart_time=2021-04-28T12:20:06ZDECONTAMINATEflow_cell_id=FAP92766DECONTAMINATEprotocol_group_id=210428_GI5_Run21-086DECONTAMINATEsample_id=21-03679
GTTATGCCGTTCAGTTGCGTATTGCTAATAAGATTGGTAGAGAAAATGGAACATAAGCAACATTCTTCAAAATCACTTTGAACTGCTTGCCAACCTCATCAATTATATAGGTCATCACTGTTCCATATTCTTCAACCTAGTTGTTGCATCTTCAGTATACACTTGTTCTTCTGCAGATTTGAATAATAGAAATCTTGATCAATTATTGCATATATAGCAATACGTAAC
+
&$($&%$%-A6E);'+*&&%%$&-;3E=6:84114;2.'&*(+2B@11$0521/26*8A)$$%507<CFA@-)1))39<<:A<</..6>?=>4-*14:8B2515538486<8@4/6$"),%7$4,065:9>?D5.=?7)*.13=:,+8?FK6D.01)$'2757FBMEKRH9<;:==<@E/75412,))%).;=><<=;9@<45C)),96689%,**',8656-,-,%#

I think I stumbled across this some time ago.
The thing is, that the length of read names in the SAM format specification is limited to 254.
So samtools does basically the right thing, cause the format is incorrect.
If we want to avoid this we'd have to check every input read... Not sure if it's possible to cut the read names only if this error occures.

Do you actually remember why we do this renaming in the first place? :)
We insert DECONTAMINATE strings into the read ID to replace whitspaces I think. Likely, to restore the full read ID after mapping/read extraction?
see:
@11dbdf9c-e46f-4afd-b1b1-b7c3cdd2e938DECONTAMINATErunid=8842aa4e3c221ff265498dfdfdfe183253ab4499DECONTAMINATEread=9DECONTAMINATEch=247DECONTAMINATEstart_time=2021-04-28T12:20:06ZDECONTAMINATEflow_cell_id=FAP92766DECONTAMINATEprotocol_group_id=210428_GI5_Run21-086DECONTAMINATEsample_id=21-03679

Maybe, during the step where we anyway rename reads, we could check for the length and cut the string of at 254 characters?
Or we should really think if this renaming is necessary at all... I think the idea was to have the original (and complete) read IDs in the decontaminated FASTQ files.

Yeah, I also think it was because of the whitespaces and the mapper messing with the readnames!

Maybe, during the step where we anyway rename reads, we could check for the length and cut the string of at 254 characters?

and we could use something shorter than DECONTAMINATE ^^

and we could use something shorter than DECONTAMINATE ^^

haha, true ^^ we just need to be sure that it can be safely replaced... what we could also do: rename to something like
read1
read2
...
readXYZ
and save a map tsv between original name and re-name and finally use that to restore the original names.
But we should also not blow up this task...
here is a simple rename/restor py script for FASTAs:
https://github.com/EBI-Metagenomics/emg-viral-pipeline/blob/master/bin/rename_fasta.py
but maybe that's also easily doable w/ seqkit or so...  https://bioinf.shenwei.me/seqkit/

So Basti says minimap does the right thing, because only the first part (til the first whitespace) is the actual read name, the rest is just additional information.
From this point of view, we could remove all the renaming :sweat_smile:

yeah... basically true. I think I wanted to keep all information bc/ e.g. I am not sure what happens if someone uses then the cleaned FASTQ w/ the missing description in e.g. a QC tool like NanoPlot etc...
But basically right... we can also skip the renaming and see if this causes trouble... this will also save time and disk usage. I'm fine w/ that


label/milestones buttons are messed up

(Newest Refined GitHub as of 2019-05-06)

The label overview is messed up. When you disable the extension it looks nicer actually. The new button position is different than for milestones.
I like the wider search box though 👍
It looks to be all about the New Label button position.
Example: https://github.com/Cockatrice/Cockatrice/labels
label:

milestone:

Due to:
https://github.com/sindresorhus/refined-github/pull/1886
https://github.com/sindresorhus/refined-github/commit/ca8b1e13f78c50a5a3283982fba41e8be49e12c0
Previous issues/PRs for this bug:
https://github.com/sindresorhus/refined-github/issues/1959
https://github.com/sindresorhus/refined-github/pull/1961