Update README.md to fix CI badge

CI badge keeps showing failing cause the test_depr one has failures - trying to be specific which test we want to show on the badge

I think we might want to split out the strict deprecation test to a different yml file.


Change thumbnail image from SF program

The thumbnail image that shows up for the LGROW site is SF's image. Can we also change the text that goes along with it?
See James' link to the program in slack...SF's logo pops up, with canned text that is incorrect.
We need this fixed before we can share the program anywhere online.

duplicate of 196


Counter transmission format has changed since tasmota v5.14.0

Counters are not displayed.
Since version 5.14.0 the transmission format has changed: "COUNTER":{"C1":251,"C2":0,"C3":0,"C4":0}
In version 5.11.0: "Counter1":0,"Counter2":0,"Counter3":0,"Counter4":0
Can you please update the code

did you enable auto detection of unknown keywords in adapter setup ?
this should solve it

no. where can I find this option?
under "Instances -> sonoff -> Settings" I could not find this option.

you must use the latest version of the adapter => 2.2.2 for auto creation of unknown objects

Am 26.07.2018 um 21:06 schrieb Tingeltangelbobi<EMAIL_ADDRESS><EMAIL_ADDRESS>no. where can I find this option?
under "Instances -> sonoff -> Settings" I could not find this option.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ioBroker/ioBroker.sonoff/issues/51#issuecomment-408202857, or mute the thread https://github.com/notifications/unsubscribe-auth/ALG4Y2LCdobwAWPHXznqdj8k9v_TRP6Rks5uKhNPgaJpZM4VP-d7.

2.2.2 is already installed
admin adapter v 2.4.9
js-controller v 1.4.2
but I can't find the option

ok, the new auto create option is only supported with admin 3!

Am 27.07.2018 um 08:03 schrieb Tingeltangelbobi<EMAIL_ADDRESS>2.2.2 is already installed
admin adapter v 2.4.9
js-controller v 1.4.2
but I can't find the option
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ioBroker/ioBroker.sonoff/issues/51#issuecomment-408320577, or mute the thread https://github.com/notifications/unsubscribe-auth/ALG4Y85B64A9PZkc6CgMqUP9E48HTUKhks5uKq1LgaJpZM4VP-d7.

sorry, I mistyped, I mean ioBroker.admin 3.4.9

ok, then you should see in adapter instance => sonoff settings
"automatische Erstellung von Zuständen"

Am 27.07.2018 um 08:34 schrieb Tingeltangelbobi<EMAIL_ADDRESS>sorry, I mistyped, I mean ioBroker.admin 3.4.9
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ioBroker/ioBroker.sonoff/issues/51#issuecomment-408325478, or mute the thread https://github.com/notifications/unsubscribe-auth/ALG4Y0v1vUqKq15Bj-3G95uoqwxCgZhhks5uKrRcgaJpZM4VP-d7.

the option is not displayed

ok, reupload the adapter, if still not working delete and reinstall

Am 27.07.2018 um 09:12 schrieb Tingeltangelbobi<EMAIL_ADDRESS>the option is not displayed
https://user-images.githubusercontent.com/35076365/43306904-ff268084-917c-11e8-9aef-f2c5cbc74349.PNG
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ioBroker/ioBroker.sonoff/issues/51#issuecomment-408332900, or mute the thread https://github.com/notifications/unsubscribe-auth/ALG4Y59ynUYvdxbB9_fp-oikZXxdJ3Vuks5uKr1dgaJpZM4VP-d7.

ok now the option is there.
but all objects and their history have been deleted.
The records of one year have been deleted.

I think the records are still there. When you still know the old ID names then update your history/sql/influxdb adapter to latest version and use alias feature to store new alues at the old ids. works also for selections

all objects will be created again, BUT some as the ip are only created on a new start of the tasmota devices.
history will reappear if the objects a recreated and you set the history flag again.
All history files remain there and are NOT deleted

Am 27.07.2018 um 10:13 schrieb Tingeltangelbobi<EMAIL_ADDRESS>ok now the option is there.
but all objects and their history have been deleted.
The records of one year have been deleted.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ioBroker/ioBroker.sonoff/issues/51#issuecomment-408346304, or mute the thread https://github.com/notifications/unsubscribe-auth/ALG4Y40RDF87bqcZW90nCNHxALkTXVm4ks5uKsurgaJpZM4VP-d7.

thank you very much. It's working.
the history data has been transferred.


update package config for py3.12 compatibility

Currently this package isn't installable on Python 3.12, due to the setup.py config having some dependency that uses a function that has been removed from the standard library. (See error trace below). There are probably multiple ways of fixing this...
I took the opportunity to migrate the setup from the old-style setup.py to the new/current recommended pyproject.toml. (See here.) I moved all the contents from setup.py to the equivalent within pyproject.toml.
I've verified that with these changes, installing works fine on py3.12 using pip install.  No change should be required in any user code or docs. This is just a backend infrastructure update.

Thanks @mperrin! I tried creating a new python 3.11 environment, and the installation went smoothly there as well.


about nnUNetTrainerV2BraTSRegions_moreDA.py

your work is so great!!
i am try to deal with my own dataset,and, i wrote these:

but as soon as i run these,the error came as :

so i m wondering where do i need to edit to let nnunet recongnize my nnUNetTrainerV2XXXXRegions_moreDA.py

i also tried to add this in nnUNetTrainerV2BraTSRegions_moreDA.py:

but still:

just dont know how to make the changes recongnized by nnunet

Hi @zhao9chen99999, what is the command that you used for training?

Hi @zhao9chen99999, what is the command that you used for training?

nnUNet_train 3d_fullres nnUNetTrainerV2BraTSRegions_DA4_BN_BD_lymphomaUnet 567 0 --npz # BL +lymphomaunet config

Have you tried the other trainer classes in the file?
If other trainer classes cannot be found as well, you might want to check this function, which looks for the trainer class:
https://github.com/rixez/Brats21_KAIST_MRI_Lab/blob/65284cc87718c0e64e3271d0ff43b945174de41c/nnunet/training/model_restore.py#L23

recursive

this one can work fine:

but this one can't:

next i changed nnUNetTrainerV2BraTSRegions_moreDA.py -nnUNetTrainerV2BraTSRegions_DA4_BN_BD from512 to 1024：

but the output net architect results is still 512，not 1024，so i wonder maybe， the edition on nnUNetTrainerV2BraTSRegions_moreDA.py doesn't work？if that’ s true，which file do i need to edit ？thx！！！

i common out the nnUNetTrainerV2BraTSRegions_DA4_BN_BD in nnUNetTrainerV2BraTSRegions_moreDA.py :

i found the nnunet still could work with the nnUNetTrainerV2BraTSRegions_DA4_BN_BD in nnUNetTrainerV2BraTSRegions_moreDA.py :

in fact, i tried delete the nnUNetTrainerV2BraTSRegions_moreDA.py and the nnunet can still run:

that confuse me,i dont know what to do next

One possible reason is that you are running the training command with a different copy of the nnUNet library. Please check which file the nnUNet_train command runs from.

One possible reason is that you are running the training command with a different copy of the nnUNet library. Please check which file the nnUNet_train command runs from.

thx for your anwser,it really helps a lot!


Homebrew entrypoints for downgrading

supermysterychunkhax is listed as a entrypoint for homebrew, but downgrading does not work with it. possibly add a disclaimer stating such

Do you have a source for this?

If you remember, a few days ago, I was talking on irc about how i could not get your downgrader to progress past HAX INIT.

I have added a notice for this, although I cannot fix it as this appears to be an issue with svchax.


Fix llvm.getelementptr offset bug.

Fix llvm.getelementptr offset bug
Description
We were computing a byte offset on a pointer taking into account the actual type, causing the offset to be off by a factor of the actual type's length.
In other words, we were calculating the offset as if it were a C pointer, whereas we needed a byte offset. Example:
my_type_t *ptr;

// Wrong:
my_type_t *offset_ptr = ptr + offset;

//  Right:
my_type_t *offset_ptr = (my_type_t *) (((void *) ptr) + offset);

fixes #328
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

fixes #328

We're currently processing your upload.  This comment will be updated when the results are available.


[10.x] Replace deprecated Rule class with ValidationRule in Enum validation

This PR updates the Enum validation rules in Laravel to use the new ValidationRule class instead of the deprecation Rule class. This is important because the Rule class has been removed in the new version of Laravel.

The Rule class isn't being removed? It's just no longer the preferred choice.


KAFKA-10635: Improve logging with throwing OOOSE

We need to have the full information on how the seq / epochs are inferred with logging.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

ping @nicktelford , have you been able to try out this patch and collect the improved logs?

@nicktelford ping


Remove unnecessary code for conftest.py

in order to avoid redundant code, the Factory was created and all the logic for creating the driver was transferred to driver_util, you need to remove the redundant code from conftest.py

Done


Support cargo rustdoc

Thanks for the pull request, and welcome! You should hear from @huonw (or someone else) soon.

Thanks!


Remove repeated key

Changes
PR fixes below error
  File "./gitea/deploy.py", line 110
    verify=False,
    ^^^^^^^^^^^^
SyntaxError: keyword argument repeated: verify

which is coming while running deploy.py python script

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

Codecov Report

Merging #886 (73bbe3d) into main (2e0f0be) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #886      +/-   ##
==========================================
+ Coverage   63.58%   63.62%   +0.03%
==========================================
  Files          78       78
  Lines        5012     5022      +10
==========================================
+ Hits         3187     3195       +8
- Misses       1520     1521       +1
- Partials      305      306       +1

Impacted Files
Coverage Δ

pkg/reconciler/status.go
74.50% <0.00%> (+1.33%)
:arrow_up:

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

🤦🏻


Prevent double imports when registering a plugin

@passsy Big things take time. Ready for a rereview.

@passsy took my time to carefully think about a good variable name

@passsy took my time to carefully think about a good variable name

This variable name is superior, will name my first child After


feat: LP performance reward metric

closes https://github.com/vegaprotocol/specs/issues/2254
Summary
PR adds a new reward metric for reward Liquidity Providers for providing liquidity within a specified range.
To make the rewards independent of the liquidity fees accumulated on a market, the transfer implements the concept of "virtual fees". That is, the fees that would have been distributed for a fixed amount of accumulated fees (1 quantum).
This reward metric will require core to compute SLA performance for a number of price ranges and as such a new network parameter is added to restrict the minimum transfer size of transfers to reward pools.

Blocked until we finalise https://github.com/vegaprotocol/specs/pull/2299

https://github.com/vegaprotocol/specs/pull/2299 has been merged now, but lets leave this one open until the work's been prioritised.

Closing PR as ticket closed - https://github.com/vegaprotocol/specs/issues/2305


error while deploying to azure - deploy-azure.sh

Describe the bug
A clear and concise description of what the bug is.
I am trying to deploy my chatbot to azure. Using {root}/scripts/deploy/deploy-azure.sh
My run command is
bash ./deploy-azure.sh --subscription SUB \
--deployment-name NAME \
--ai-service AzureOpenAI  \
--ai-service-key API \
--ai-endpoint ENDPOINT  \
--client-id CLIENT_ID \
--tenant-id TENANT \
-rg RESOURCE_GROUP

However, I get the error
{"code": "InvalidTemplate", "message": "Deployment template validation failed: 'The template parameters 'azureAdInstance, azureAdTenantId, webApiClientId' in the parameters file are not valid; they are not present in the original template and can therefore not be provided at deployment time. The only supported parameters for this template are 'name, webAppServiceSku, packageUri, aiService, completionModel, embeddingModel, plannerModel, aiEndpoint, aiApiKey, webApiKey, deployNewAzureOpenAI, deployCosmosDB, memoryStore, deploySpeechServices, deployWebApiPackage, location, webappLocation, sqlAdminPassword'. Please see https://aka.ms/arm-pass-parameter-values for usage details.'.", "additionalInfo": [{"type": "TemplateViolation", "info": {"lineNumber": 0, "linePosition": 0, "path": ""}}]}

Specifically, azureAdInstance, azureAdTenantId, webApiClientId in the parameters file are not valid.
I think probably there is an issue with scripts/deploy/main.bicep
Platform

OS: MacOS
IDE: VSCode
Language: C#
Source: main branch

Additional context
Add any other context about the problem here.

@ekdnam - thanks for the submission, we will get this fixed.

@ekdnam Can you confirm that your versions of deploy-azure.sh and main.bicep both match the latest versions in the repo?  The error seems to indicate that main.bicep template doesn't contain those parameters, but it should.
It would also help me to know if this this a new deployment, or are you updating a pre-existing one?

I don't have the recent version, its old.
I am updating a pre-existing one

As @gitri-ms mentioned, it seems as though you have something out of sync - like you're running a relatively new version of deploy-azure.sh with an older version of main.json and the two are not compatible.
If you do not want to take the latest changes to the webApi and webApp then I suggest you try to revert the updates you have to the deploy-azure.sh script. If you're ok with updating to the latest then that should work also.
Closing this for now, please feel free to create a new issue if you still have issues deploying after resolving the out-of-sync issue.


FR: Expose record error with additional user info

Feature proposal

Firebase Component: Crashlytics

Currently there is only the -recordError: method exposed which calls the appropriate underlying implementation with additionalUserInfo dictionary set to nil.
- (void)recordError:(NSError *)error {
  FIRCLSUserLoggingRecordError(error, nil);
}

Is there a reason that there is no second method which passes the additionalUserInfo through? I.e.:
- (void)recordError:(NSError *)error withAdditionalUserInfo:(NSDictionary<NSString *, id> *)additionalUserInfo {
  FIRCLSUserLoggingRecordError(error, additionalUserInfo);
}

I'm going through the migration guide and there is no explanation as to why it was removed. Is it welcomed if I submit a PR?

Just a note on this for the maintainers... The email we received around a month ago mentions that crash reporting will stop working using the deprecated SDK in November so we need to migrate to Firebase Crashlytics ASAP but this is hard if there is missing functionality from the replacement.
I raised a support ticket for this hoping that it gains a higher priority but please could you let us know if this will be addressed soon? I see it more as missing functionality than a feature request since we need this feature for tracking issues and we need to be able to provide support for older releases of our app and the longer that I hold off switching to FirebaseCrashlytics, the more more builds that will be out there that will stop reporting crashes in November.
I'll hold out a little longer but it would be great to get a documented response so that I can start on an alternative approach if this doesn't score a high enough priority to be tackled soon, cheers. 🙏

I can't test this at the moment, but could this give the same results in the meanwhile?
let tempError = myError as NSError
let recordableError = NSError(domain: tempError.domain, code: tempError.code, userInfo: additionalUserInfo)
Crashlytics.crashlytics().record(error: recordableError)

I can't test this at the moment, but could this give the same results in the meanwhile?
let tempError = myError as NSError
let recordableError = NSError(domain: tempError.domain, code: tempError.code, userInfo: additionalUserInfo)
Crashlytics.crashlytics().record(error: recordableError)

I am running into the same migration problem. Wouldn't you lose data about the original error myError, because you only record the domain, code, and additionalUserInfo, not myError itself

I am running into the same migration problem. Wouldn't you lose data about the original error myError, because you only record the domain, code, and additionalUserInfo, not myError itself

The data you could lose is in tempError.userInfo, so you could merge that with additionalUserInfo.
Here it says we should append additional info in NSError.userInfo like so:
let userInfo = [
  NSLocalizedDescriptionKey: NSLocalizedString("The request failed.", comment: ""),
  NSLocalizedFailureReasonErrorKey: NSLocalizedString("The response returned a 404.", comment: ""),
  NSLocalizedRecoverySuggestionErrorKey: NSLocalizedString("Does this page exist?", comment: ""),
  "ProductID": "123456",
  "View": "MainView"
]

let error = NSError.init(domain: NSCocoaErrorDomain,
                         code: -1001,
                         userInfo: userInfo)

Thanks for pointing that out @mcontin & @zachcristol-heb, I think this would work in our use case so I just need to verify it but can't right this second due to the ongoing incident.
If this does prove to be successful, then maybe a good action point from this ticket for the Firebase team would be to include such instructions in the migration guide 🙏

I can confirm that the approach above works both for Fabric and Firebase SDKs, thanks!


Use "owner" to extract provider for Sharekit in products

Closed in favour of: https://github.com/surfedushare/harvester/issues/347


NixOps deploy build fails when using +1000 keys - bash argument list too long

Issue description
When deploying a machine with lots of keys, the deploy fails with a bash "argument list too long" error.
Steps to reproduce
Create 2 nix files as explained in the sample from the manual:
test.nix:
{
  network.description = "Web server";
  webserver =
    { config, pkgs, ... }:
    { services.httpd.enable = true;
      services.httpd.adminAddr =<EMAIL_ADDRESS>      networking.firewall.allowedTCPPorts = [ 80 ];
    };
}

test-servers.nix:
webserver =
  { config, pkgs, ... }:
  {
    deployment.targetEnv = "virtualbox";
    deployment.virtualbox.memorySize = 1024; # megabytes
    deployment.virtualbox.vcpu = 2; # number of cpus
    };
  }

Add a lot of keys to test-servers.nix, in the form below (example)
    deployment.keys."phpmyadmin.password" = {
      text = fileContents (./keys/phpmyadmin.password);
      group = "keys";
      permissions = "0640";
    };

Create a new deployment:
$ nixops create -dtest test.nix test-servers.nix

The build never succeeds
$ nixops deploy -dtest --build-only:
building path(s) ‘/nix/store/lv40g4brdfa187x3h08jbxgwrs12gmpm-nixos-system-webserver-17.03.890.ce3ab704b2’
while setting up the build environment: executing ‘/nix/store/86blj9iqyxwmdgkn3dyrpib1gkbmz91v-bash-4.4-p5/bin/bash’: Argument list too long
builder for ‘/nix/store/apmw3a1pb7imlw7p6vls9hs6glj4n44h-nixos-system-webserver-17.03.890.ce3ab704b2.drv’ failed with exit code 1

Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
17.03.890.ce3ab704b2 (Gorilla)
Nix version: (run nix-env --version)
nix-env (Nix) 1.11.8
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
17.03.890.ce3ab704b2

Apologies for butting in as I know next to nothing about nixops but could you confirm how many keys are you using? I'm on 17.03 as well and tried to reproduce with 10000 keys (phpmyadmin1 - phpmyadmin10000)  and, while very slow, nixops deploy -dtest --build-only has been happily chugging along for the last 40 minutes whereas it seems to abort immediately for you.

The issue is that our builder is bash and it the file used for the builder is too long, you'll get this error.
Here is the derivation that fails to build: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/system/activation/top-level.nix#L106
Try adding somewhere below (inside the mkDerivation call):
passAsFile = [ "buildCommand" ];

This will pass the builder as a file, instead of a string.

Hi @pbogdan,
I ran the deploy task several times, so it doesn't build all the derivations anymore (it takes some time indeed as nixops seems to generate derivations and services for every single key file)
So it goes immediately to the last step (the failing bash call).
@domenkozar How would I go about testing your addition to the mentioned file?  Should I fork this repo, adjust the file and use that as -I parameter to the nixops deploy call?
Currently I'm using
nixops deploy -dtest --build-only -I nixpkgs=https://github.com/NixOS/nixpkgs-channels/archive/nixos-16.09.tar.gz

As a side note: I'm still using 16.09 as somehow phpfpm is no longer available as package in 17.03 (or I haven't figured out what to do to use phpfpm in 17.03)
Kind regards,
Erik

Fork, change the file, use -I nixpkgs=/path/to/nixpkgs, profit.

Hi Domen,
I'm just from trying this out: I forked the nixpkgs-channels repo, adjusted the file in https://github.com/4levels/nixpkgs-channels/blob/nixos-16.09/nixos/modules/system/activation/top-level.nix#L105
and used -I nixpkgs=https://github.com/4levels/nixpkgs-channels/archive/nixos-16.09.tar.gz but to no avail :-(
I verified the contents of the tar.gz archive and the file inside nixos/modules/system/activation/top-level.nix is the correct one (with the passAsFile addition).
So either I'm still doing something wrong, or the passAsFile command doesn't seem to help

Hi @domenkozar,
even after running nix-collect-garbage -d (it took about 20 minutes to regenerate all direvations) the same error still pops up :-(
Am I doing something wrong or is the passAsFile option not the correct one?
Kind regards and thanks again already!

Can you make a diff how you added the flag?

I just added 1 single line:
passAsFile = [ "buildCommand" ];

here: https://github.com/4levels/nixpkgs-channels/blob/nixos-16.09/nixos/modules/system/activation/top-level.nix#L105
Or was I supposed to replace "buildCommand" with something else?  I'm on totally uncharted territory here ;-)

Hi @domenkozar,
I'm just trying now with changing buildCommand to buildCommandPath as I found references of one of your PR's adding a check for this in setup.sh - https://github.com/NixOS/nixpkgs/pull/15803/files
So I commented out the previous passAsFile line and changed buildCommand to buildCommandPath here: https://github.com/4levels/nixpkgs-channels/blob/nixos-16.09/nixos/modules/system/activation/top-level.nix#L108
I did another nix-collect-garbage -d but after rebuilding all derivations, this fails again with the exact same error..
Thanks again..

I'm giving it another try, this time with
passAsFile = [ "buildCommand" "activationScript" ];

after removing the previous try with buildCommandPath
https://github.com/4levels/nixpkgs-channels/blob/nixos-16.09/nixos/modules/system/activation/top-level.nix#L105
New test running...

@domenkozar
Adding the activationScript to the passAsFile list seems to work!
I'm trying this asap in my "real" deployment setup and report back here ;-)
If this actually solves my issue, I'm definitely buying you a beer (or a coffee or a tea)!
Fingers crossed!

Hi @domenkozar,
So the build runs fine now, but the copying closures call seems to fail now:
secure........> copying closure...
error: [Errno 7] Argument list too long

It's failing eithere here https://github.com/NixOS/nixops/blob/master/nixops/backends/init.py#L336 or a few lines lower.

Hi @domenkozar,
Exactly what I thought as well! I just found the same line in the same file by searching the NixOps repository..
But how to resolve?  It seems like all closures are joined as a string, but how to work around that?
Fiew, now I'm diving into Python code as well, more uncharted territories..

Hi @domenkozar,
All I can see with my limited python skills is that the function logged_exec seems related as well:
https://github.com/NixOS/nixops/blob/master/nixops/util.py#L46
But I'm sure you've already seen that as well.  I hope there is a similar workaround for this!
Kind regards,

I think it's better to fix this in nixops by instead of generating a huge system.activationScripts.nixops-keys script, rather put the huge part into its own file and source it from the activation script.

Hi @dezgeg,
How would I go about that? I obviously have no idea how to put this part into a file and source it :-(
Kind regards,

Something like this to nixops (completely untested):
diff --git a/nix/keys.nix b/nix/keys.nix
index b6a1e6b..65c55bc 100644
--- a/nix/keys.nix
+++ b/nix/keys.nix
@@ -123,33 +123,35 @@ in
       "things to break."
     )];

-    system.activationScripts.nixops-keys = stringAfter [ "users" "groups" ]
-      ''
-        mkdir -p /run/keys -m 0750
-        chown root:keys /run/keys
-
-        ${optionalString config.deployment.storeKeysOnMachine
-            (concatStrings (mapAttrsToList (name: value:
-              let
-                # FIXME: The key file should be marked as private once
-                # https://github.com/NixOS/nix/issues/8 is fixed.
-                keyFile = pkgs.writeText name value.text;
-              in "ln -sfn ${keyFile} /run/keys/${name}\n")
-              config.deployment.keys)
-            + ''
-              # FIXME: delete obsolete keys?
-              touch /run/keys/done
-            '')
-        }
-
-        ${concatStringsSep "\n" (flip mapAttrsToList config.deployment.keys (name: value:
-          # Make sure each key has correct ownership, since the configured owning
-          # user or group may not have existed when first uploaded.
-          ''
-            [[ -f "/run/keys/${name}" ]] && chown '${value.user}:${value.group}' "/run/keys/${name}"
-          ''
-        ))}
-      '';
+    system.activationScripts.nixops-keys =
+      let script = ''
+          mkdir -p /run/keys -m 0750
+          chown root:keys /run/keys
+
+          ${optionalString config.deployment.storeKeysOnMachine
+              (concatStrings (mapAttrsToList (name: value:
+                let
+                  # FIXME: The key file should be marked as private once
+                  # https://github.com/NixOS/nix/issues/8 is fixed.
+                  keyFile = pkgs.writeText name value.text;
+                in "ln -sfn ${keyFile} /run/keys/${name}\n")
+                config.deployment.keys)
+              + ''
+                # FIXME: delete obsolete keys?
+                touch /run/keys/done
+              '')
+          }
+
+          ${concatStringsSep "\n" (flip mapAttrsToList config.deployment.keys (name: value:
+            # Make sure each key has correct ownership, since the configured owning
+            # user or group may not have existed when first uploaded.
+            ''
+              [[ -f "/run/keys/${name}" ]] && chown '${value.user}:${value.group}' "/run/keys/${name}"
+            ''
+          ))}
+        '';
+      # Generate the script in a separate file to avoid a "Argument list too long" with a huge number of keys.
+      in stringAfter [ "users" "groups" ] "source ${pkgs.writeText "setup-keys.sh" script}";

     systemd.services = (
       { nixops-keys =

Hi @dezgeg,
Thank you so much for this patch.
Could you be so kind to instruct me how to use my own forked version of the NixOps package in my deploy machine?  I'm not too sure how I can tell nix to use my forked version of NixOps instead of the standard one.
I have already forked the nixpkgs repo as well and I'm using that for now..
Kind regards,
Erik

Hi All,
I've been trying really hard to get my version of Nixops in my local nixos machine running, but I'm failing miserably.  Everything I try results in failures, wrong packages being downloaded and what more, sigh
Even when I try this (from the manual):
file: my-nixops.nix
with import <nixpkgs> {}; # bring all of Nixpkgs into scope

stdenv.mkDerivation rec {
  name = "nixops";
  src = fetchurl {
    url = "https://github.com/4levels/nixops/archive/master.tar.gz";
    sha256 = "02p19r970dg3xyskicml5v2vps2r5mms2s8sv596ij3g33ym7jhc";
  };
}

Then this fails:
$ nix-build my-nixops.nix`
these derivations will be built:
  /nix/store/7am2z9w20b0ryp751sb8nziy8y8lzk38-master.tar.gz.drv
  /nix/store/17vnz9and9qii2rkqbm8vdlf3ysxlhkq-nixops.drv
building path(s) ‘/nix/store/jibakjw535nxgqwz2krcik8xfyvh5ky0-master.tar.gz’

trying https://github.com/4levels/nixops/archive/master.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   122    0   122    0     0    198      0 --:--:-- --:--:-- --:--:--   198
100  268k  100  268k    0     0   102k      0  0:00:02  0:00:02 --:--:--  220k
building path(s) ‘/nix/store/kwdfrr02pzyyvkaxmx6a6gfsdd5x14g1-nixops’
unpacking sources
unpacking source archive /nix/store/jibakjw535nxgqwz2krcik8xfyvh5ky0-master.tar.gz
source root is nixops-master
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file nixops-master/tests/unit/test_split_openssh_args.py
warning: file nixops-master/tests/unit/test_split_openssh_args.py may be generated; SOURCE_DATE_EPOCH may be non-deterministic
patching sources
configuring
no configure script, doing nothing
building
no Makefile, doing nothing
installing
install flags: install
make: *** No rule to make target 'install'.  Stop.

The documentation in the Nixops readme seems very outdated as well: eg, the installation instructions link leads to a 404 and the Google groups seems very dead.
Hope someone can help, I'm really close to solving this issue!
As a sidenote: isn't it a good change to have the nixops-keys script always use a file based process as suggested by @dezgeg? If this change would make it in the official repo, I wouldn't even have to go through this struggle..
Kind regards,
Erik aka 4levels

https://nixos.org/nixops/manual/#chap-hacking
On Tue, Apr 25, 2017 at 11:33 AM, Erik Van Kelst<EMAIL_ADDRESS>wrote:

Hi All,
I've been trying really hard to get my version of Nixops in my local nixos
machine running, but I'm failing miserably. Everything I try results in
failures, wrong packages being downloaded and what more, sigh
Even when I try this (from the manual):
file: my-nixops.nix
with import  {}; # bring all of Nixpkgs into scope
stdenv.mkDerivation rec {
name = "nixops";
src = fetchurl {
url = "https://github.com/4levels/nixops/archive/master.tar.gz";
sha256 = "02p19r970dg3xyskicml5v2vps2r5mms2s8sv596ij3g33ym7jhc";
};
}
Then this fails:
$ nix-build my-nixops.nix`
these derivations will be built:
/nix/store/7am2z9w20b0ryp751sb8nziy8y8lzk38-master.tar.gz.drv
/nix/store/17vnz9and9qii2rkqbm8vdlf3ysxlhkq-nixops.drv
building path(s) ‘/nix/store/jibakjw535nxgqwz2krcik8xfyvh5ky0-master.tar.gz’
trying https://github.com/4levels/nixops/archive/master.tar.gz
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   122    0   122    0     0    198      0 --:--:-- --:--:-- --:--:--   198
100  268k  100  268k    0     0   102k      0  0:00:02  0:00:02 --:--:--  220k
building path(s) ‘/nix/store/kwdfrr02pzyyvkaxmx6a6gfsdd5x14g1-nixops’
unpacking sources
unpacking source archive /nix/store/jibakjw535nxgqwz2krcik8xfyvh5ky0-master.tar.gz
source root is nixops-master
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file nixops-master/tests/unit/test_split_openssh_args.py
warning: file nixops-master/tests/unit/test_split_openssh_args.py may be generated; SOURCE_DATE_EPOCH may be non-deterministic
patching sources
configuring
no configure script, doing nothing
building
no Makefile, doing nothing
installing
install flags: install
make: *** No rule to make target 'install'.  Stop.
The documentation in the Nixops readme seems very outdated as well: eg,
the installation instructions link leads to a 404 and the Google groups
seems very dead.
Hope someone can help, I'm really close to solving this issue!
As a sidenote: isn't it a good change to have the nixops-keys script
always use a file based process as suggested by @dezgeg
https://github.com/dezgeg? If this change would make it in the official
repo, I wouldn't even have to go through this struggle..
Kind regards,
Erik aka 4levels
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/issues/25057#issuecomment-296974048,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHtg1GuwOcaU69C5ZTAnaSZcJ6E8y1iks5rzb3kgaJpZM4NCq-N
.

Hi @domenkozar, thanks a lot, I totally overlooked this appendix in the manual..
Maybe this should go in the Nixops readme instead of the 404 link ;-)

Hi @dezgeg,
your fix doesn't seem to change anything, the copying closure call still fails with
[Error 7] Argument list too long
I'm currently checking if I can adjust the copy closure part just as you did adjusted the activationScript
Kind regards,
Erik

Fixed: https://github.com/NixOS/nixops#nixops
On Tue, Apr 25, 2017 at 12:06 PM, Erik Van Kelst<EMAIL_ADDRESS>wrote:

Hi @domenkozar https://github.com/domenkozar, thanks a lot, I totally
overlooked this appendix in the manual..
Maybe this should go in the Nixops readme instead of the 404 link ;-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/issues/25057#issuecomment-296983466,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHtg1M-CAwkizdB8oKQO2U66xY_t0sHks5rzcW0gaJpZM4NCq-N
.

Hi @domenkozar, you fixed the documentation links, right?  Or did you also manage to fix this actual issue?
As this is all so confusing and new for me, I'm giving up to see this resolved anytime soon.  If no-one else cares to fix this, I guess it will remain broken untill someone with the needed skills runs into this limitation and creates a fix.
I'll resort to ecnrypting and packaging the key files and try to get them in the keystore myself :-(
Too bad though as I felt like I was almost there to workaround this bug!
Thanks again for all your support!
Erik

I wish I had time to help you. Add some print statements around that code so that you can see how the code is being invoked.

Hi Domen, like I said before, I have never written a single line of python code.  Besides that, I don't even understand how nixops / nix works / is built.  So this is clearly out of my leage, let alone me trying to debug this..

If you can create a gist that I can simply run that reproduces the issue I can look into it soon :)

Hi Domen,
I can create a gist, but I cannot add +1000 key files to it, right?
I can create a tar archive with the 2 forementioned nix expressions (test.nix and test-server.nix) and the related key files.

Would be useful to have the full Python stack trace as well.

Hi Domen,
I wouldn't know how to get to that point.  It seems like @pbogdan found a quick way to test this with 10k files, maybe he can shed some light how he set this up...

@4levels can you reproduce nixops error by passing --debug?

HI @domenkozar ,
I collected all garbage so now the rebuild takes a while.  As soon as it's done, I'll create a gist with the results.  You ment adding "--debug" to the file init.py right? (not the nixops --debug switch)
Or do I need to do both?  Adding --debug to the nixops deploy call generates a huge XML file.

nixops --debug is what I meant.

Hi Domen, I added --debug to the ssh call in the file mentioned before (the __ init __ script) with the default nixops (so not the updated version with your passAsFile addition):
building path(s) ‘/nix/store/bmaiaw7pxawc8ndq1k9c219ma3wqjdqy-nixos-system-secure-16.09pre-git’
while setting up the build environment: executing ‘/nix/store/gabjbkwga2dhhp2wzyaxl83r8hjjfc37-bash-4.3-p48/bin/bash’: Argument list too long
builder for ‘/nix/store/61jrly4lw1yfxbyp8cvhvwaachbdnmzq-nixos-system-secure-16.09pre-git.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/c2va29jcnaja43g534kd7n56hqhsdlfr-nixops-machines.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/c2va29jcnaja43g534kd7n56hqhsdlfr-nixops-machines.drv’ failed
Traceback (most recent call last):
  File "/nix/store/pw63rhnxibm3n4wdvkyzla20n8ydjlbs-nixops-1.5/bin/..nixops-wrapped-wrapped", line 951, in <module>
    args.op()
  File "/nix/store/pw63rhnxibm3n4wdvkyzla20n8ydjlbs-nixops-1.5/bin/..nixops-wrapped-wrapped", line 379, in op_deploy
    repair=args.repair, dry_activate=args.dry_activate)
  File "/nix/store/pw63rhnxibm3n4wdvkyzla20n8ydjlbs-nixops-1.5/lib/python2.7/site-packages/nixops/deployment.py", line 989, in deploy
    self._deploy(**kwargs)
  File "/nix/store/pw63rhnxibm3n4wdvkyzla20n8ydjlbs-nixops-1.5/lib/python2.7/site-packages/nixops/deployment.py", line 957, in _deploy
    self.configs_path = self.build_configs(repair=repair, include=include, exclude=exclude)
  File "/nix/store/pw63rhnxibm3n4wdvkyzla20n8ydjlbs-nixops-1.5/lib/python2.7/site-packages/nixops/deployment.py", line 633, in build_configs
    raise Exception("unable to build all machine configurations")
Exception: unable to build all machine configurations

I'll add the output of the other calls in separate comments below

I meant that you test with passAsFile change since that fixes the first bug :)

This is the output of the nixops deploy call with the passAsFile additions and the extra --debug in the __ init __.py script in nixops/backends
secure........> copying closure...
Traceback (most recent call last):
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/bin/..nixops-wrapped-wrapped", line 951, in <module>
    args.op()
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/bin/..nixops-wrapped-wrapped", line 379, in op_deploy
    repair=args.repair, dry_activate=args.dry_activate)
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/lib/python2.7/site-packages/nixops/deployment.py", line 989, in deploy
    self._deploy(**kwargs)
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/lib/python2.7/site-packages/nixops/deployment.py", line 964, in _deploy
    max_concurrent_copy=max_concurrent_copy)
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/lib/python2.7/site-packages/nixops/deployment.py", line 656, in copy_closures
    tasks=self.active.itervalues(), worker_fun=worker)
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/lib/python2.7/site-packages/nixops/parallel.py", line 41, in thread_fun
    result_queue.put((worker_fun(t), None))
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/lib/python2.7/site-packages/nixops/deployment.py", line 652, in worker
    m.copy_closure_to(m.new_toplevel)
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/lib/python2.7/site-packages/nixops/backends/__init__.py", line 336, in copy_closure_to
    ssh.run_command("nix-store -j 4 -r --debug --ignore-unknown " + ' '.join(closure), check=False)
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/lib/python2.7/site-packages/nixops/ssh_util.py", line 286, in run_command
    return nixops.util.logged_exec(cmd, self._logger, **kwargs)
  File "/nix/store/v5wgxmfhawxq725ny2fx2hrh4z3yhd6i-nixops-1.5pre0_abcdef/lib/python2.7/site-packages/nixops/util.py", line 76, in logged_exec
    stderr=subprocess.STDOUT)
  File "/nix/store/acsc2d6r6x6z9cdnks4ff5n0n84sw96z-python-2.7.13/lib/python2.7/subprocess.py", line 390, in __init__
    errread, errwrite)
  File "/nix/store/acsc2d6r6x6z9cdnks4ff5n0n84sw96z-python-2.7.13/lib/python2.7/subprocess.py", line 1024, in _execute_child
    raise child_exception
OSError: [Errno 7] Argument list too long

If I could only figure out how to rewrite this line:
ssh.run_command("nix-store -j 4 -r --debug --ignore-unknown " + ' '.join(closure), check=False)

to use a file containing the joined closures...

Maybe we should move this issue to the NixOps repo instead?  Its's getting quite long already, what do you suggest?

You can workaround this by setting deployment.hasFastConnection = true in nixops expression

OMG! THAT WORKS!

@domenkozar, it seems like the change in the nixpkgs repo (with the passAsFile statements isn't even required.  I did adjust the nixops repo with the change from @dezgeg and that seems to be sufficient.
This means I can use the official nixpkgs repo and only need to patch the nixops repo.
The change as proposed by @dezgeg seems to work the magic!
Thanks for your kind support and for bearing here with me!
Kind regards,
Erik

@dezgeg @4levels can you make a PR for the fix?

Hi @domenkozar,
All credit goes to @dezgeg so I consider it not fair creating a PR with his code change..
I don't mind if he agrees, I consider it an honor to have my first PR in a Nix repo ;-)
Kind regards,
Erik

@4levels since https://github.com/NixOS/nixops/pull/915 was merged, I presume this can be closed?


Connection termination is not handled correctly

Errors in the connection are not processed correctly. Connection termination cannot be handled correctly as a result

Fixed in 0.5.3


apply sdpa for mpt and internlm

What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@echarlaix could you please take a look?

Can we have a test for each model architecture that is updated in this PR?


This uitility does not work for me - very random

HI, when i run this program i always get 0 sols /s i dont understand whats up with it.
Every other CPU miner i use works fine.
I have an i7 2600k:
    ==================== www.nicehash.com ====================
            Equihash CPU&GPU Miner for NiceHash v0.5c
    Thanks to Zcash developers for providing base of the code.
        Special thanks to tromp, xenoncat and djeZo for providing
          optimized CPU and CUDA equihash solvers.
    ==================== www.nicehash.com ====================

Setting log level to 2
[10:28:36][0x00000fb8] Using SSE2: YES
[10:28:36][0x00000fb8] Using AVX: YES
[10:28:36][0x00000fb8] Using AVX2: NO
[10:28:36][0x00000fb8] api | Listening on port 3334
[10:28:36][0x0000068c] stratum | Starting miner
[10:28:36][0x0000068c] stratum | Connecting to stratum server europe.equihash-hu
b.miningpoolhub.com:17023
[10:28:36][0x0000068c] stratum | Connected!
[10:28:37][0x0000068c] stratum | Subscribed to stratum server
[10:28:37][0x0000068c] miner | Extranonce is 1c000068
[10:28:38][0x0000068c] stratum | Authorized worker crankyman.multipoolminer
[10:28:38][0x0000068c] stratum | Target set to<PHONE_NUMBER>00000000000000000000000
0000000000000000000000000000000
[10:28:38][0x0000068c] stratum | Ignoring non-clean job #58b
[10:28:51][0x00000fb8] Speed [15 sec]: 0 I/s, 0 Sols/s
[10:28:51][0x0000068c] stratum | Received new job #58c
No idea whats going on

What speeds do you get on the other miners? Are they greater than 1 sol/s? CPUs are not good for mining. They just can't do the math fast enough so your going to just end up spending more money on the electricity it uses that you'll ever get from what you earn from mining.


Add CLI for extracting percentiles from probabilities using ECC

As a IMPROVER developer, I want to modify the percentile CLI to support the conversion of probabilities to percentiles using ECC so that I can generate percentile output.
Related issues:  #54
The plugins to convert from probabilities to percentiles already exists. This issue just adds a call to this plugin from the improver-percentile CLI when it is provided with cube of probability. The plugin interface must be modified to accept a pre-defined list of percentiles, as well as providing the option to create a list of percentiles, if these aren't provided.
Acceptance criteria:

Need the ability to extract a single percentile and a list of percentiles.
Retain support for supplying the number of percentiles to the plugin.
The CLI will only support providing the plugin with a list of percentiles, where there may be a single percentile in the list, rather than the number of percentiles.
CLI tests.

Completed.


Delete a column, the operation record(+,-,~) related to this column has not been deleted

Hi,
If I delete a column in the table, the corresponding column in the record table is also deleted. But the operation records (+, -, ~) generated by this column are not deleted, which will cause some problems

That's on purpose. If you want to cascade deletion to historical records, you can do the following:
class MyModel(models.Model):
    history = HistoricalRecords(cascade_delete_history=True)


File opening is failing

Describe the bug
When the file contains hex encoded data:
    kwargs["text"] = kwargs["file_stdin"].read().decode("utf-8")
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa8 in position 0: invalid start byte

To Reproduce
Use a hex encoded file
Try to import it into Ciphey

Hey!
I would like to help fix this issue

Hey!
I would like to help fix this issue

Awesome Sashreek1! Please get to work fixing the issue and when your done do a pull request!

I am actually unable to recreate this issue,
I have used this file containing HEX encoded text and use
ciphey -f "hextest"
and it works. This is what the file contained :
68 65 6C 6C 6F 20 74 68 65 72 65
and here is the output
(venv) [user@localhost tests]$ ciphey -f "hextest"
Result 'hello there' (y/N): y
Format used:
 hex
Final result: "hello there"


Docker configuration

Is there any plan to also provide Docker configuration additionally to VirtualBox?
I would send a PR, if you'd be ok with this.

Heya, unfortunately not. Since this is an OS project for an app I'd like people to use the same environment when developing just like you'd do on any other web-app when working in large teams. Homestead comes closest to a Forge provisioned server so we'll stick with that.
Thanks for offering though! Feel free to ask any other question if you'd like 🙂


Add Selfhost

Adding self-hosting clients etc for all Fedora machines I'll also be using at the off-grid project site. TODO

corrected mup alias

updated aliases to be added, added asciinema, cowsay etc.


adding rst tables comparison

This adds a few tables in the syntax page to compare one vs the other...what do folks think? Good way to compare?

Here's how one of them looks - I'm not a huge fan of the CSS but I do think the layout is useful: https://49-240151150-gh.circle-artifacts.com/0/html/syntax.html#extra-markdown-syntax

Quick question first, how to you find out the links for CircleCI build artifacts?

Looks good to me though 👍

Mostly, click on the circle link, go to "Artifacts", then click index.html from the dropdown list. You may need CircleCI permissions for this though - let me know if I need to add you or something.
There are also a few bots that will auto-post a link to the documentation alongside your CircleCI link...I'll set that up at some point, just havne't gotten around to it. E.g.:
https://github.com/larsoner/circleci-artifacts-redirector-action

I can't see any "Artifacts" section?

Hmmm - it looks like you may not be logged in with your CircleCI account. Can you make sure it is connected w/ your GitHub account, and that you're logged in, and try again?

Sorted ta, for some reason I had to log out and log back in, and now it is showing up


MorseGraphDrawGraph(DomainGraph(parameter_graph.parameter(i))) and database.DrawMorseGraph(i) producing different graphs

I created a database and its parameter graph:
from DSGRN import *
database = Database("EMT.db")
parameter_graph = database.parametergraph
and then randomly picked an index 1000 and drew the corresponding Morse graph in two different ways:
database.DrawMorseGraph(1000)
and
MG1000 = MorseGraph(DomainGraph(parameter_graph.parameter(1000)))
DrawGraph(MG1000)
But they are giving me two different Morse graphs.  I am wondering whether the indexing schemes in database and in the corresponding parameter graph are the same.

I can see why this causes confusion. The indexing schemes are different as in fact it turns out two different things are being indexed: Morse Graphs, by "MorseGraphIndex", and combinatorial parameters, by "ParameterIndex". Why they are not the same is because more than one parameter can have the same Morse graph, so the database structure labels distinct Morse Graphs with "MorseGraphIndex" indices. The database has a table which gives the mapping from each ParameterIndex to the not-necessarily-unique "MorseGraphIndex".  (This design prevents duplicate storage of the same Morse Graph.) This table is accessible via database.__call__ (e.g. database(1000), which accepts ParameterIndex 1000 and returns the associated MorseGraphIndex). Another table in the database gives the mapping from MorseGraphIndex to actual Morse Graphs. This table is the one accessed via the database.DrawMorseGraph call, i.e. for
database.DrawMorseGraph(1000)

what is being retrieved is  a (picture of a) Morse Graph which has been assigned the "MorseGraphIndex" 1000, while in
MG1000 = MorseGraph(DomainGraph(parameter_graph.parameter(1000)))

what is being retrieved is the Morse Graph corresponding to "ParameterIndex" 1000.
What should work is
database.DrawMorseGraph(database(1000))

which will call the __call__ routine which retrieves from the database the MorseGraphIndex corresponding to a ParameterIndex, and then pass that to DrawMorseGraph.

Thank you very much!


Browsers Back Button does not work as expected

Browsers history is synced with all the iframes rendered in the page. This is not the correct behaviour

🙈

the comment was by mistake . sorry


mev boost holesky

mev-boost: add holesky support

@aldoborrero Do you know who can I contact about buildbot problem? Looks like something is broken there

@selfuryon let me talk with @Mic92 as he's the supreme god on buildbot.

~Fixed in #517


Scene does not need to extend from Node (same with HTML interface).

Scene can be a standalone thing that contains a single root Node within it, which is the top level Node of the world that the scene contains. Scene does not need transforms applied to it.
(This will also simplify module dependencies, reducing potential circular dependency problems.)

[ ] Refactor so that Scene does not extend from Node (possibly both Node and Scene might extend from a base class if any functionality should be shared, possibly not if that shared API surface area is small enough and would merit individual implementation for each class).
[ ] Do the same with motor-node/scene elements.

Work for #40 is waiting for this to be done first.

[ ] Nodes (the current Node class) will extend from a new Transformable class, which Scene won't extend from. Transformable will house the position, rotation, etc.
[ ] Both Scene and Node (Node's name might possibly change) will extend from a new TreeNode class. The TreeNode class makes it easy to construct a tree, where the root will be a Scene TreeNode and all other nodes are Node TreeNodes.

Completed in bf3a487d1d43616598b60635a50104a0c6b47ae6 (for now, will revisit in the future with a better multiple-inheritance scheme when Proxy is widely supported, and am using a mixin pattern for now)


Update Image of Fairy Tail(S1) - Sense Of Wonder

This issue is about Fairy Tail (OP/ED/OST).

[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate.
[x] This issue is for enhancement/bug fixing.
[x] This issue is an addition to JSON database.
[x] I have gone through Contribution Guidelines.
[x] Assign this issue to me.

Has been fixed in #149


Create Ingres template

What this PR does / why we need it: This is PR is to add ingress template that can be created by end user to have  sa-discovery ingress
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

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

Thank you @krimeshshah for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.


Security group ID is malformed (empty)

What happened?
I created a security group and an ingress rule and connected them. The ingress rule had a failing qualification Can create ingress which failed with this error:
Qualification Failures:

    An error occurred (InvalidGroupId.Malformed) when calling the AuthorizeSecurityGroupIngress operation: The security-group ID '' is malformed

Raw Output:

Running CLI command: "aws 'ec2' 'authorize-security-group-ingress' '--region' 'us-east-2' '--dry-run' '--cli-input-json' '{ "GroupId": "", "IpPermissions": [ { "FromPort": 0, "ToPort": 0, "IpProtocol": "All", "IpRanges": [ { "CidrIp": "<IP_ADDRESS>/0" } ] } ], "TagSpecifications": [ { "ResourceType": "security-group-rule", "Tags": [ { "Key": "Name", "Value": "si-8632" } ] } ] }'"

An error occurred (InvalidGroupId.Malformed) when calling the AuthorizeSecurityGroupIngress operation: The security-group ID '' is malformed

Output: { "protocol": "result", "status": "success", "executionId": "tomcruise", "data": { "result": "failure", "message": "\nAn error occurred (InvalidGroupId.Malformed) when calling the AuthorizeSecurityGroupIngress operation: The security-group ID '' is malformed" }, "unset": false }

Notably, the group ID is empty. This matches what we can see in the GUI:

I do not expect the ID to be empty because I connected the security group to the ingress, which should set the group ID:

In trying to reproduce this, I also found another scenario where the group ID appeared empty in the GUI but there were no failing qualifications:

How can we reproduce this?
This is somewhat reproducible. My general steps are:

Create the region
Create a security group
Create an ingress
Connect the security group and the ingress
Fill out an IpPermissions entry in the ingress
Wait

I do not wait between any of these steps; i.e. I perform them as fast as SI allows me to. Maybe this is related to these bugs?
I tried deleting and re-creating the edge between the security group and the ingress; this did not change the group ID on the ingress in the GUI, and the qualification function was still failing. Notably, nothing appears to happen when I create an edge. I would expect to see the ingress start qualification again, but it looks like nothing happens (no progress spinner, etc.).
Operating system
Linux
Architecture
x86_64 / amd64
What browsers are you seeing the problem on?
Firefox
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional context
Debug info for the group ID:

Thanks for this @Octogonapus
Looking into it now!

This is actually user error on my part, thanks for helping @stack72


Fatal error: Unexpectedly found nil while unwrapping an Optional value

[x] I have read the Getting Started section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
3.10.0

Plugin version
6.0.0-beta.25

Android version
21-34

iOS version
12-17

macOS version

Xcode version
15

Device information:
Description
Expected behaviour:
Current behaviour:
Steps to reproduce

private static func _setCookie(cookie: NSDictionary, result: @escaping FlutterResult) {
    let domain = cookie["domain"] as? String
    let expiresDate = cookie["expires"] as? Double
    let isSecure = cookie["secure"] as? Bool
    let isHttpOnly = cookie["httpOnly"] as? Bool
    let origin = cookie["origin"] as? String

    var properties: [HTTPCookiePropertyKey: Any] = [:]
    properties[.name] = cookie["name"] as! String
    properties[.value] = cookie["value"] as! String
    properties[.path] = cookie["path"] as? String ?? "/"
    if domain != nil {
        properties[.domain] = domain;
    }
    if origin != nil {
        properties[.originURL] = origin;
    }
    if expiresDate != nil {
        properties[.expires] = Date(timeIntervalSince1970: expiresDate!)
    }
    if isSecure != nil && isSecure! {
        properties[.secure] = "TRUE"
    }
    if isHttpOnly != nil && isHttpOnly! {
        properties[.init("HttpOnly")] = "YES"
    }

    let cookie = HTTPCookie(properties: properties)!   Thread 1: Fatal error: Unexpectedly found nil while unwrapping an Optional value

    httpCookieStore!.setCookie(cookie)
}

I am trying to run the web view code in iOS 17 with code 15, but every time I try to access the web view iOS app crashes. Then this error was produced "Thread 1: Fatal error: Unexpectedly found nil while unwrapping an Optional value". it targets this line "let cookie = HTTPCookie(properties: properties)!". you can see it on the above code.
Images
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.0, on macOS 14.0 23A344 darwin-arm64
(Rosetta), locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.83.0)

no answer till

anyone can help

Fixed on latest 6 version.

can you give me the link


Undo damage and remove redundant build action

This should at least restore the basic functions that I broke today, and also remove the redundant extra build that was occurring every time a PR was updated.


change3 in css leads to errors

Hey,
I tried to change part of the css source code as follow in order to locate the icon not in the bottom of the page.
.leaflet-iconLayers-layer {
cursor: pointer;
position: relative;
bottom:390px;       I added this line to the code*
width: 80px;
height: 80px;
background-color: #fff;
background-repeat: no-repeat;
background-size: cover;
text-align: center;
box-sizing: border-box;
box-shadow: 0 0 5px #000;
}
After doing so, the icon moves upper on the page but it does not function as before. When I click on the it, it will be collapsed but it does not let me choose other base-maps. In other words, when I try to hover over other base maps, the table containing base maps will be closed quickly and does not let me to choose my favorite base map.
any idea??
tnks,
Saeid

Add the following CSS rule instead of your midification:
.leaflet-left .leaflet-control.leaflet-iconLayers {
    margin-bottom: 390px;
}

It will shift the whole control 390 pixels up.

Thanks, Aparshin


【Comfyui】The output is garbled during the reversal process（不知道为什么反推一直是乱码，重装了好多次）

There are no errors reported in the terminal output：


No history state pushed when adding turbo frames

When adding <turbo-frame id="shell">...</turbo-frame>, HTTP requests are issued via Turbo, but no state is changed to the browser history, i.e. the contents of the address bar doesn't change.

Is this related to https://github.com/hotwired/turbo/issues/50?

Is this related to https://github.com/hotwired/turbo/issues/50?

Yes, perhaps it is. I was thinking #50 is different due to all the video-related things it mentions, but reading it again, this is exactly what I'm observing:

Use the new Turbo Frames feature (which works), but then the URL doesn't get updated

So is this a known limitation then? Using <turbo-frame> makes a huge difference for me with Firefox in terms of perceived performance (in fact, initially I thought Turbo isn't working at all when using it without <turbo-frame>  in FF, as it didn't seem to make a difference), so it'd be very desirable to use. But the lack of history update makes it not really usable atm.

Yes, perhaps it is. I was thinking #50 is different due to all the video-related things it mentions, but reading it again, this is exactly what I'm observing:

Use the new Turbo Frames feature (which works), but then the URL doesn't get updated

So is this a known limitation then? Using <turbo-frame> makes a huge difference for me with Firefox in terms of perceived performance (in fact, initially I thought Turbo isn't working at all when using it without <turbo-frame>  in FF, as it didn't seem to make a difference), so it'd be very desirable to use. But the lack of history update makes it not really usable atm.

So is this a known limitation then?

I believe this behavior is a design decision.
I'm closing this issue in favor of #50. If you have further thoughts, please share them there.

So is this a known limitation then?

I believe this behavior is a design decision.
I'm closing this issue in favor of #50. If you have further thoughts, please share them there.

Ok, thanks for clarifying.

I believe this behavior is a design decision.

I wonder why, though? This makes Turbo essentially unusable for me on
Firefox, as I have the choice between not using a frame, thus having the
correct history but same experience as without Turbo, or using Frames, much
better experience, but incorrect history :( In any case, thanks a lot for
your efforts on working on this library, appreciate it.

Ok, thanks for clarifying.

I believe this behavior is a design decision.

I wonder why, though? This makes Turbo essentially unusable for me on
Firefox, as I have the choice between not using a frame, thus having the
correct history but same experience as without Turbo, or using Frames, much
better experience, but incorrect history :( In any case, thanks a lot for
your efforts on working on this library, appreciate it.


Terminate all Foundation Zcash instances running pre-NU5 testnet versions

Depends on merging #2793 and #2794, and updating all pull requests to the latest main branch.

There's also a zcashd instance that we should terminate, because it will be outdated soon too.
Also we should probably shut down the old DNS seeder.

There's also a zcashd instance that we should terminate, because it will be outdated soon too.

This is shut down now.


Fix SERVICE_HOSTNAME in the example

# kubectl get inferenceservice ${MODEL_NAME} -o jsonpath='{.status.url}'
http://flowers-sample.kubeflow.example.com/v1/models/flowers-sample

# SERVICE_HOSTNAME=$(kubectl get inferenceservice ${MODEL_NAME} -o jsonpath='{.status.url}' | cut -d "/" -f 3)

# echo $SERVICE_HOSTNAME
flowers-sample.kubeflow.example.com

This change is

/cc @yuzisun @ellis-bigelow @animeshsingh

cool, thanks for fixing ! Do you think it is worth to move these into a script which can be shared between all these examples?

We probably don't want to generalize as it makes pathing a bit challenging. One option is we don't expose the protocol and just do the hostname in the spec. I think this is fine for now though.


broke test

Describe your problem and how to reproduce it:
I followed along with the video entered the code exactly and the first test fails second test passes the code is correct so the test I think is broke
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/responsive-web-design/responsive-web-design-principles/make-an-image-responsive
Tell us about your browser and operating system:

Browser Name: microsoft edge
Browser Version:
Operating System: windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

worked in chrome only test I have had trouble with so far

Please use our forum for debugging the code of your projects.  If the forum members determine there is nothing wrong with your code, you can request this issue to be reopened.  GitHub issues are meant for reporting bugs and not troubleshooting the code in your projects.
Thank you for understanding.

not n issue with code issue with test it worked in chrome and not edge test is broken on Microsoft edge

What code did you try on Microsoft Edge?
What errors (if any) did you see in the browser's console (not the FCC console)?

I followed the code in the video for help

Please post what you see in your browser's console.  Thank you.  Also, please post the code you tried.

@lasjorg I notice if I add the following console.log to the first test's testString (when testing locally):
    testString: console.log(JSON.stringify(getComputedStyle(document.querySelector('.responsive-img')))); assert(getComputedStyle($('.responsive-img')[0]).maxWidth === '100%');

I only see the following in Edge
{"WebkitAnimation":""}

vs the following in Chrome:
{
  "0": "animation-delay",
.
.
.

  "maxWidth": "100%",
.
.
.
  "zIndex": "auto",
  "zoom": "1"
}

What version of Edge are you running? @kevthew  and @RandellDawson
It passes for me in the Chromium-based version which is the only version I have, so I'd like to know what version you are running.
@RandellDawson Does it give you the same output if you run the log from the browser console? I get the full object when I run it in the console. Not sure where it is getting -webkit-animation from, I only see that on the fa icons (animated icons) and in the bootstrap file for the main site. I don't think it should show up in the output/test iframe context. I don't see in the object when I run the log in the browser console.
Side note, not really sure why the test is using getComputedStyle, as far as I know using $('.responsive-img').css('max-width') === '100%' should be exactly the same. It's a bit shorter and maybe less confusing than mixing Jquery and plain JS. Or just switching it to only plain JS for all of it.

@lasjorg I tested it using Edge version 44.18362.449.0 on Windows 10 Home.  It gives the same from the browser console. On Chrome it works as expected.

Well, I can't test it using that version (unless I set up a VM for it).
Not sure where it is getting the pixel value from, that seems very odd. I don't understand how max-width set to 100% would ever compute to a pixel value.
Using $(selector).css(property) should be the same as using getComputedStyle. Looking at the jQuery source I'm not sure why it wouldn't be the same.
Jquery source for reference
https://github.com/jquery/jquery/blob/master/src/css.js#L267
https://github.com/jquery/jquery/blob/master/src/css/curCSS.js
https://github.com/jquery/jquery/blob/master/src/css/var/getStyles.js

Can I take a look at this issue?

@Pallof sure, go for it.

For these challenges, are the helper-functions ready to be used in the tests? Or, can a simple fix be made like:
assert(code.match(/max-width:\s*100%;/));

For these challenges, are the helper-functions ready to be used in the tests? Or, can a simple fix be made like:
assert(code.match(/max-width:\s*100%;/));

That is what it was before https://github.com/freeCodeCamp/freeCodeCamp/pull/37934. So I don't see why not. As for the helper-functions, I don't know anything about that.
I'd still like to know why Edge is giving the result it is, but I guess it doesn't matter much.

Well, @moT01 , you were involved with the current change. Are we better leaving it as is, changing the test to use regex, or change it to what it was before #37934 ?

For these challenges, are the helper-functions ready to be used in the tests?

No, the helper functions aren't ready

FYI - When I console.log $('.responsive-img').css('max-width') in the browser console, I see 268px not 100% using Edge. For Chrome, I see 100%.

Looks like the getComputedStyle method won't work for edge. So yes, probably need to go back to regex here if we want it to work on edge.
Here's a modification of something I have used in the past to isolate the styles of a selector...
const resImg = code.match(/\.responsive-img\s*{[\s\S]+?[^}]}/g)[0]; assert(/max-width\s*:\s*22%\s*(;|})/g.test(resImg));
^ I modified without testing, hoping to get a helper function for isolating styles added to the PR for those.

This is still causing problems every now and then.
Have we abandoned the support of the older Edge versions (prior to the switch to Chromium), or should we fix this by switching back to a regex?


Change configs for "Run Spark Thriftserver on local" in ClickHouse.md

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

Temporarily close this pr.
will make a new one after some tests


sh:languageIn + runtime languageIn support

fromRdf:

take parameter { languageIn?: readonly string[] } that defaults to sh:languageIn value (if any) on destructure
given resource.values() on LiteralType/StringType, pick first literal value with languageIn

SPARQL:

take optional languageIn? parameter to SparqlGraphPatterns
add FILTER on literals (get this from kos-kit)

Didn't do the SPARQL part of this. kos-kit doesn't have it; its SPARQL retrieves all literals and then filters them in memory. Will do the SPARQL side if it becomes necessary.


[BUG] Fix bad channels error, better default

Fixes https://github.com/mne-tools/mne-python/issues/12351

Ok I good by me

Thanks @alexrockhill !
FYI to avoid the pre-commit bot having to do stuff you can do locall
pre-commit install --install-hooks

@alexrockhill I suspect this broke circle can you check?
https://app.circleci.com/pipelines/github/mne-tools/mne-python/22981/workflows/d074dd99-a4e4-4034-b5a9-90a4e21bbc60/jobs/63796/parallel-runs/0/steps/0-142


Use absolute paths to files from tests

As discussed in #266 I'm separating refactor on tests from original PR

Thanks!


Swipe gesture blocks range bar swipe

I have the react-native-video-controls over a video, i want to implement some swipe events to change the video source. I manage to make this work. However i am now unable to swipe the volume down or even move the seek bar. Hope i can find any help

+1

+1

I had a similar issue with this plugin. I understood that it catches the event at first, as it is higher in the component tree. Basically, every panResponder-based element nested into GestureRecognizer will fail at some point.
I made a workaround for this, it is a bit hacky but it works for my needs, maybe it could help you.
I modified the _handleShouldSetPanResponder function from source, to check the target of the catched event ; if it has the property ignoreSwipe, then I don't want to actually catch this event in GestureRecognizer
_handleShouldSetPanResponder(evt, gestureState) {
    if (evt._targetInst.memoizedProps.children) {
        if (evt._targetInst.memoizedProps.children.props) {
            if (evt._targetInst.memoizedProps.children.props.ignoreSwipe) { // you could change ignoreSwipe to whatever
                return false
            }
        }
    }

    return evt.nativeEvent.touches.length === 1 && !this._gestureIsClick(gestureState);
}

// example
<GestureRecognizer>
    <ComponentWithPanResponder ignoreSwipe={true} />
</GestureRecognizer>

But this component shouldn't be capturing the events unless we explicitly set on*ShouldSetPanReponderCapture, correct? And it doesn't appear to be doing that.


Adding selective 'plugins' to config

Would be nice if the plugins array could be specified. It'll most likely have to be done in the user's config/environment.js.

Yes. For now you can reopen the ember-selectize component and override the plugins array.
Question: to keep selectize configuration in a single place, could it be possible/desirable to set the environment from the Brocfile?

Not sure if it's possible, but it's possible to get the config in the addon's index.js from the user's config/environment.js. See here.

So, what do you think of centralizing all the configuration in config/environment.js?
Ember-cli configuration is confusing right now.
Quoting a tweet from @EmberSherpa:

TIL: EmberCLI has 3 levels of config
App: config/environment.js
Asset Compilation: Brocfile.js
CLI: Task parameters
Thanks @abuiles

So, I guess this is a matter of whether this is an app or asset compilation configuration.
Possible selectize configurations (at least for now):

Theme
Plugins

Changing theme does change the assets that are included, but not the app or the code itself (just css files) -> Brocfile
Changing plugins does not affect the assets that are included because all the selectize plugins are included in the same js file -> environment
We reach a conclusion that doesn't make me happy. Perhaps we should just use environment.js, considering the theme an app config, but just one that doesn't have any impact on code.

@knownasilya, I also don't know how this component would behave with plugins like "drag_drop" or "optgroup_columns", but nevertheless this configuration option should exist.

#15 c66898d118466a6285d4355f7a544577b3635803

Closed in PR #15


Create user stories for professional facing patterns and components

What
Create user stories from each of the components or patterns on from this mural board
Why
To help us understand what the user needs driving each component are and give us a greater idea of which items should be prioritised.
Done when

[ ] There are user stories for each of the elements on the mural board

Initial user stories for each component added to the mural to aid discussion and prioritisation needed for #1525


[ruff] Detect more strict-integer expressions (RUF046)

Summary
Part 2 of the big change introduced in #14828 and the follow-up to #14832.
Test Plan
cargo nextest run and cargo insta test.

Hmm, I can't change the base branch if the target is from a fork. I'll wait reviewing this PR until the target is merged.

Would you mind writing a short summary of what you're changing in the PR description? It's hard to review code if the change is unclear.
Did you review through the ecosystem changes? It's unclear to me why we're no longer flagging
https://github.com/RasaHQ/rasa/blob/b8de3b231126747ff74b2782cb25cb22d2d898d7/rasa/shared/core/training_data/loading.py#L86

I'm not suggesting that we should not improve the rule at all, but I'm wary of making rules unnecessarily complex by implementing too much ad hoc type inference. It results in inconsistent behavior between rules and makes Ruff harder to maintain.
Can we focus on the most important cases you outlined above: single numbers, mathematical expressions, and a few known function calls? It also makes me wonder if we could move some or most of the logic into typing::is_int (we can only move the "strict" part). The logic would then benefit all rules instead of just a few.

A is_int_expr next to is_int that returns true for expressions that are guaranteed to be int seems like a good outcome. The maybe-int cases can then be handled inside the rule itself.

@InSyncWithFoo
We shouldn't fix those cases for now. We can handle round calls directly in RUF046 and anything more involved using round won't be detected because ResolvedPythonType doesn't support it.

Perfect, thank you!


project - zip and src check if empty fix

Regression from #2179. Fixes #2184.
Before:
python tools\project.py -m KL25Z -p 0 -i gcc_arm

 * KL25Z::gcc_arm      C:\Code\git_repo\github\mbed\.build\export\.temp\MBED_A1

With this patch:
python tools\project.py -m KL25Z -p 0 -i gcc_arm

  * KL25Z::gcc_arm     mbed\.build\export\MBED_A1_gcc_arm_KL25Z.zip

@sarahmarshy @theotherjimmy @screamerbg

cc @ohagendorf

I also pushed a fix, but this looks good. Sorry about that :/

@sarahmarshy Please test , I checked the previous logic. if this not bool is correct, and works also for --source

LGTM

Tested with and without --source. Looks good.


Presets: Small cleanup

Remove some options that are implied by a base preset.
Remove unused preset for building with Xcode.

@shahmishal ping

Force-pushed to add a tag to the commit title.


Imgui not compatible with Mac OS 10.11.4 and LibGDX 1.9.9

Exception in thread “main” com.badlogic.gdx.utils.GdxRuntimeException: java.lang.Exception: ERROR: 0:3: ‘’ :  version ‘150’ is not supported

    at com.badlogic.gdx.backends.lwjgl3.Lwjgl3Application.<init>(Lwjgl3Application.java:117)
    at com.x.prototype.desktop.DesktopLauncher.main(DesktopLauncher.kt:21)
Caused by: java.lang.Exception: ERROR: 0:3: ‘’ :  version ‘150’ is not supported

    at gln.objects.GlShader$Companion.createFromSource-RHQEpZs(shader.kt:70)
    at imgui.impl.ImplGL3.createDeviceObjects(ImplGL3.kt:57)
    at imgui.impl.LwjglGlfw.newFrame(LwjglGlfw.kt:107)

This is not an ImGui issue, per se. You are running an old mac which does not support OpenGL 3.2, which is the minimum version of OpenGL we support. An OpenGL 2 backend might be considered, but is a small use case for most of us. I'll see what I can do

Hm how hard for one would be to make It compatible with opengl2? It is about changing this class https://github.com/kotlin-graphics/imgui/blob/master/src/main/kotlin/imgui/impl/ImplGL3.kt only?

For me myself, this would be hard as my experience is with OpenGL 3 and 4, not 2. It would require making a separate class, ImplGL2, and only using functions up to OpenGL 2.1 in it, and changing the shaders to be OpenGL 2.1 compatible. If you wish to do it, please send a PR when it gets close to workable and I'll see what I can do to help.

I'm gonna give this an attempt in the ImplGL2 branch. We'll see what comes of it

OpenGL2 support is live in the ImplGL2 branch. Give it a shot and see if it works for you.

Thanks for your effort!
So I have changed It like this:
LwjglGlfw.init(
GlfwWindow.from((Gdx.graphics as Lwjgl3Graphics).window.windowHandle),
clientApi_ = LwjglGlfw.GlfwClientApi.OpenGL2
)

ImplGL3.renderDrawData(this) => LwjglGlfw.instance.renderDrawData(this)
And It shows up this error:
0(12) : error C5060: out can't be used with non-varying color

It looks like we shouldn't even use shader at all for GL2

I've also tried the latest version from ImplGL2 (with libgdx 1.9.8 though) and got this stacktrace:
java.lang.Exception: ERROR: 0:7: Invalid storage qualifiers 'in' in global variable context
ERROR: 0:8: Invalid storage qualifiers 'in' in global variable context
ERROR: 0:9: Invalid storage qualifiers 'in' in global variable context
ERROR: 0:11: Invalid storage qualifiers 'out' in global variable context
ERROR: 0:12: Invalid storage qualifiers 'out' in global variable context
ERROR: 0:16: Use of undeclared identifier 'uv'
ERROR: 0:16: Use of undeclared identifier 'UV'
ERROR: 0:17: Use of undeclared identifier 'color'
ERROR: 0:17: Use of undeclared identifier 'Color'
ERROR: 0:18: Use of undeclared identifier 'Position'
�
	at gln.objects.GlShader$Companion.createFromSource-RHQEpZs(shader.kt:70)
	at imgui.impl.ImplGL2.createDeviceObjects(ImplGL2.kt:49)
	at imgui.impl.LwjglGlfw.newFrame(LwjglGlfw.kt:154)
	at imgui.impl.LwjglGlfw$Companion.newFrame(LwjglGlfw.kt:34)

I know what I did wrong, will be fixed in an hour or two
On Wed, Feb 13, 2019, 06:13 alex-dorokhov<EMAIL_ADDRESS>wrote:

I've also tried the latest version from ImplGL2 (with libgdx 1.9.8 though)
and got this stacktrace:
java.lang.Exception: ERROR: 0:7: Invalid storage qualifiers 'in' in global variable context
ERROR: 0:8: Invalid storage qualifiers 'in' in global variable context
ERROR: 0:9: Invalid storage qualifiers 'in' in global variable context
ERROR: 0:11: Invalid storage qualifiers 'out' in global variable context
ERROR: 0:12: Invalid storage qualifiers 'out' in global variable context
ERROR: 0:16: Use of undeclared identifier 'uv'
ERROR: 0:16: Use of undeclared identifier 'UV'
ERROR: 0:17: Use of undeclared identifier 'color'
ERROR: 0:17: Use of undeclared identifier 'Color'
ERROR: 0:18: Use of undeclared identifier 'Position'
�
at gln.objects.GlShader$Companion.createFromSource-RHQEpZs(shader.kt:70)
at imgui.impl.ImplGL2.createDeviceObjects(ImplGL2.kt:49)
at imgui.impl.LwjglGlfw.newFrame(LwjglGlfw.kt:154)
at imgui.impl.LwjglGlfw$Companion.newFrame(LwjglGlfw.kt:34)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kotlin-graphics/imgui/issues/72#issuecomment-463177231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOafQCdtyRvy9ZrTp2-k1G1QUlfLTR2uks5vNAFlgaJpZM4a1-Fb
.

Should be fixed in ImmplGL2 commit 4b38958

Ok now It works but there is still some issue when I resize window, does not happen on opengl3

I cannot reproduce this issue. Is there something specific I need to do?

ps: ImGui.drawData?.run(LwjglGlfw.instance::renderDrawData)

@Sylvyrfysh Try test lwjgl.kt and change to GlfwClientApi.OpenGL2 I am testing this on latest opengl version right now but this should be not a problem?

looks something wrong with the clearing, maybe it has something to do with the scissor?

Retina mac?

Right now I can not test it on Mac hardware. Currently I am using linux mint.

Was able to reproduce after changing the window size. Looking into it

Yes happens only when you resize the window :)

0f9c3ab

That one works correctly! :+1: I will try It on mac and let you know how It works there.

Ok It also works on Mac. Thanks for your help! I appreciate It :+1:

No problem!

org.lwjgl.opengl.GL11C.nglDrawElements(IIIJ)V (0 bytes) @ 0x00000001155416c4 [0x0000000115541600+0xc4]
J 1294 C1 com.badlogic.gdx.backends.lwjgl3.Lwjgl3GL20.glDrawElements(IIILjava/nio/Buffer;)V (124 bytes) @ 0x0000000115538e94 [0x0000000115537880+0x1614]
J 1273 C1 com.badlogic.gdx.graphics.Mesh.render(Lcom/badlogic/gdx/graphics/glutils/ShaderProgram;IIIZ)V (257 bytes) @ 0x000000011552707c [0x00000001155269e0+0x69c]
J 1243 C1 com.badlogic.gdx.graphics.g2d.SpriteBatch.flush()V (193 bytes) @ 0x00000001155165ec [0x00000001155159c0+0xc2c]
J 2121 C1 com.badlogic.gdx.graphics.g2d.SpriteBatch.end()V (90 bytes) @ 0x000000011598a034 [0x0000000115989f20+0x114]
J 2592 C1 com.x.prototype.screen.editor.TileEditorScreen.renderScene(F)V (607 bytes) @ 0x0000000115e2578c [0x0000000115e1fe40+0x594c]
J 2715 C1 com.x.prototype.screen.base.BaseScreen.render(F)V (33 bytes) @ 0x0000000115fe137c [0x0000000115fe0c60+0x71c]
J 2711 C1 com.badlogic.gdx.backends.lwjgl3.Lwjgl3Window.update()Z (213 bytes) @ 0x0000000115fdd1d4 [0x0000000115fdbfa0+0x1234]
j  com.badlogic.gdx.backends.lwjgl3.Lwjgl3Application.loop()V+101
j  com.badlogic.gdx.backends.lwjgl3.Lwjgl3Application.<init>(Lcom/badlogic/gdx/ApplicationListener;Lcom/badlogic/gdx/backends/lwjgl3/Lwjgl3ApplicationConfiguration;)V+257

This error happens only on opengl2 when I try to render something libgdx related in my render method:
app.batch.begin()
app.batch.draw(app.assetManager.get<Texture>(path), 0f, 0f , 10f, 10f)
app.batch.end()

Does your machine support opengl 4.3?

I am testing this on OpenGL 4.6 and switched Imgui to OpenGL2 version but also happens on mac with open gl 2.1

After your init, plug this code in:
`glEnable(GL43.GL_DEBUG_OUTPUT)
glEnable(GL43.GL_DEBUG_OUTPUT_SYNCHRONOUS)
    cb = GLDebugCallback_43()
    glDebugMessageCallback(cb, 1)
    glDebugMessageControl(GL11.GL_DONT_CARE, GL11.GL_DONT_CARE, GL11.GL_DONT_CARE, BufferUtils.createIntBuffer(0),
            true)`

and add this class somewhere
`
class GLDebugCallback_43 : GLDebugMessageCallback() {
override fun invoke(source: Int, type: Int, id: Int, severity: Int, len: Int, msg: Long, udata: Long) {
    if (id == 131185 || id == 131218)
    // || id == 131186
        return
    val _source: String = when (source) {
        GL_DEBUG_SOURCE_API -> "API"

        GL_DEBUG_SOURCE_WINDOW_SYSTEM -> "WINDOW SYSTEM"

        GL_DEBUG_SOURCE_SHADER_COMPILER -> "SHADER COMPILER"

        GL_DEBUG_SOURCE_THIRD_PARTY -> "THIRD PARTY"

        GL_DEBUG_SOURCE_APPLICATION -> "APPLICATION"

        GL_DEBUG_SOURCE_OTHER -> "UNKNOWN"
        else -> "UNKNOWN"
    }
    val _type: String
    val _severity: String = when (severity) {
        GL_DEBUG_SEVERITY_HIGH -> "HIGH"

        GL_DEBUG_SEVERITY_MEDIUM -> "MEDIUM"

        GL_DEBUG_SEVERITY_LOW -> "LOW"

        GL_DEBUG_SEVERITY_NOTIFICATION -> "NOTIFICATION"

        else -> "UNKNOWN"
    }

    when (type) {
        GL_DEBUG_TYPE_ERROR -> _type = "ERROR"

        GL_DEBUG_TYPE_DEPRECATED_BEHAVIOR -> _type = "DEPRECATED BEHAVIOR"

        GL_DEBUG_TYPE_UNDEFINED_BEHAVIOR -> _type = "UNDEFINED BEHAVIOR"

        GL_DEBUG_TYPE_PORTABILITY -> _type = "PORTABILITY"

        GL_DEBUG_TYPE_PERFORMANCE -> _type = "PERFORMANCE"

        GL_DEBUG_TYPE_OTHER -> _type = "OTHER"

        GL_DEBUG_TYPE_MARKER -> _type = "MARKER"

        else -> _type = "UNKNOWN"
    }

    println("ID $id: $_type of $_severity severity, raised from $_source: ${MemoryUtil.memASCII(msg)}")
    println("STACKTRACE:")
    Thread.currentThread().stackTrace.forEach(::println)
}

}`

stacktrace.txt
I have added but actually I am getting SIGSEGV

From that stacktrace, ImGui is not doing anything to cause that. Did nothing get printed to console?

new buffers sizes, vtx: 2048, idx: 1024
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x0000000041023e67, pid=3816, tid=0x00007f7e015a6700
#

See if 48c582f fixes it

It didn't help :-/

If you comment out imgui calls, does it work fine?

If I comment out everything from LwjglGlfw.newFrame(), ImGui.newFrame() to LwjglGlfw.instance.renderDrawData (and keep LwjglGlfw.init, context and all initialization) and draw something on screen using libgdx batch It works.

What about only commenting out only renderDrawData?

Same issue.

And keep in mind this problem happens only with OpenGL2. On opengl3 It works without problems.

I guess then the only way to find out is to comment out internally everything of createDeviceObjects, then uncomment step by step to find out the problematic call

guitest.zip
I can easily simulate It with this demo.

guitest.zip
I can easily simulate It with this demo.

btw, thanks for your donation, appreciated!

Tada! 2c1f58f
On a side note, I'll talk to the LibGDX guys about fixing stuff like that on their end too. Thanks for working with us through this

Good this one works correctly! :+1:

hs_err_pid19901.log
There is still one issue but this one will be probably harder. It does happen on Mac with OpenGL2. It doesn't happen for me on Linux. It happens randomly or with some interaction in UI (and libgdx combination).

This is a JVM bug specifically affecting Mac. Find it here

Hm :thinking:  but this does not happen on other screens where we do not use ImGui.

That's due to the compilation failing on getDrawData. I'll see if I can modify the code to fix it, but it's doubtful

May trying another jdk be an option for you, @yuraj11? Like jdk11?

Try d2c5266

But I second elect's trying a different JDK

Sorry I can not test It right now but I will check It tomorrow. I can also try with latest JDK after testing your change.

guitest.zip
Something happened and I tried reverting your last change in ImplGL2 branch but does not help. I see you have rebased changes from master which probably contained some change which broke opengl2. You can check demo. It cointains latest implgl2 branch library. Somehow UI will dimiss after libgdx is drawing something. Didn't happen previously and there is no problem on opengl3.

It is caused by this change https://github.com/kotlin-graphics/imgui/commit/5b5112c5d043abec2282180fd5cd98dc15a28067

Beside the mentioned issue It looks that d2c5266 fixes the crash on Mac (will report If something changes).

That is the change that fixes the memory leak. Try the latest changes, fixed again with a new memory eak workaround

Now everything works good now.

No issues with OpenGL2 on Mac so far :+1:

hs_err_pid5129.log
One more random crash on Mac (OpenGL2). Happens rarely comparing to previous issues.

Weird, that belongs to stb though, so it should be outside our reaching area
An update to stb is coming to lwjgl though

And I can not simulate It on linux on opengl2 or 3. So this could be again some mac issue (but I can not confirm for sure).

hs_err_pid12238.log
The last mentioned issue happens for me on OpenGL2 and Linux so It is not only mac specific but OpenGL2 in general.

Now this last issue happened also on OpenGL3+.

Ignore that last issue. It is probably on our side :man_facepalming: (I am calling multiple times LwjglGlfw.init, ImGui.addFont etc. in onShow method which is called when you come back from other screen but dispose is not called so that's why It happens only when you return from other screen) Anyway I think you can merge opengl2 branch to master. It is stable now.

And thanks for your help again!

Nice, ok then I'll merge later
Thank you, dear

hs_err_pid2411.log
hs_err_pid2428.log
I have found two random issues on Mac OpenGL2 :-/ The previous looks like fixed but this one happens on first time when opening the screen.


Trading system

first trade deals

[ ] Cosmic Custard Consortium - 64x white dye - 8x raw gold, random generating flower
[ ] Black Hole Bagels LLC - 64x wheat, 8x black dye -
[ ] Bob's Construction Fleet - 64x iron plates - 8x belts, 8 cogs
[ ] Inter Galactic Toiletries - 64x Paper - 8x bones, 4x red dye
[ ] Wood and such inc - 16x axes - 64x wood log, 16x saplings, water bucketdesign v

S weddings inc - 64 crossbows, 64 red dye > wolf in cage, 64x snow block, 16 bones

Doungen delver Dave
[ ] 32 leggings > dungeon loo
[ ] 256 torches > skeleton spaener

Need to think of things that can't be made by the player or are not practical to automate to be the payout for trades in the mid, late game. In the early game trades can be for what the player might need  at that moment

What if water will be very hard to get? To a point where it makes sense to make deals for it instead.

We can make batteries a bit more complex to make and sell them early on.

Maybe in general. Anything electronic will not be manufactured by the player but teaded for instead. Mostly that means me system things

#249


feat: Support micro-services application generation

Closes #530

Synchronize JHipster templates overridden in the blueprint
Correct relationships import path in svelte and cypress code
CI job to verify micro-services application generation

@mraible, would you be able to review the micro-services support?

Yes, I'll try to take a look in a few hours. I'm planning on using the following command to test:
jhipster jdl reactive-mf --blueprints svelte --client-framework svelte

Hopefully, this will work. You can see the reactive-mf JDL at https://github.com/jhipster/jdl-samples/blob/main/reactive-mf.jdl.

I tried the command above and was able to start everything successfully. However, after authenticating in the gateway, I don't see any menu items.

If I run npm start in the gateway, it doesn't help. I tried running it in the blog app too, but it fails there.
><EMAIL_ADDRESS>start
> svelte-kit dev --port 9000 --open true

Your jsconfig.json should extend the configuration generated by SvelteKit:
{
  "extends": "./.svelte-kit/tsconfig.json"
}
> Port 9000 is already in use
    at Server.onError (/Users/mraible/Downloads/smf/blog/node_modules/vite/dist/node/chunks/dep-e1fc1d62.js:47384:28)
    at Server.emit (node:events:527:28)
    at emitErrorNT (node:net:1399:8)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)

If I run npm run e2e in the gateway, it works:
    ✔  All specs passed!                        01:48       26       26        -        -        -

If stop the gateway's npm start and run npm start in the blog directory, the UI comes up and I'm able to see entities. When I try to create a new blog, it 404s.

If I run npm run e2e, it seems to run tests against the gateway, rather than the blog only.
    ✖  17 of 20 failed (85%)                    09:53       79       20       49        -       10

I believe @mshima implemented things with the expectation that e2e tests could not be run as a whole (like they can on the gateway with a monolith UI), but you can run them all using npm run e2e --workspaces.
See my previous comments here about testing microfrontends with Cypress.

It doesn't looks correct as blog should have been a microservice (no micro-frontend support) without UI. Can you test with local JDL removing clientFramework from each microservice application configuration and not passing client-framework as cli option

@vishal423 As far as I know, adding clientFramework to each microservice is the only way to currently trigger microfrontends. Are you using a different technique?
I tried the JDL below this morning and the results (no entities on gateway) are the same:
application {
  config {
    baseName gateway
    reactive true
    packageName com.okta.developer.gateway
    applicationType gateway
    authenticationType oauth2
    buildTool gradle
    clientFramework svelte
    prodDatabaseType postgresql
    serviceDiscoveryType eureka
    testFrameworks [cypress]
  }
  entities Blog, Post, Tag, Product
}

application {
  config {
    baseName blog
    reactive true
    packageName com.okta.developer.blog
    applicationType microservice
    authenticationType oauth2
    buildTool gradle
    clientFramework svelte
    databaseType neo4j
    devDatabaseType neo4j
    prodDatabaseType neo4j
    enableHibernateCache false
    serverPort 8081
    serviceDiscoveryType eureka
    testFrameworks [cypress]
  }
  entities Blog, Post, Tag
}

application {
  config {
    baseName store
    reactive true
    packageName com.okta.developer.store
    applicationType microservice
    authenticationType oauth2
    buildTool gradle
    clientFramework svelte
    databaseType mongodb
    devDatabaseType mongodb
    prodDatabaseType mongodb
    enableHibernateCache false
    serverPort 8082
    serviceDiscoveryType eureka
    testFrameworks [cypress]
  }
  entities Product
}

entity Blog {
  name String required minlength(3)
  handle String required minlength(2)
}

entity Post {
  title String required
  content TextBlob required
  date Instant required
}

entity Tag {
  name String required minlength(2)
}

entity Product {
  title String required
  price BigDecimal required min(0)
  image ImageBlob
}

relationship ManyToOne {
  Blog{user(login)} to User
  Post{blog(name)} to Blog
}

relationship ManyToMany {
  Post{tag(name)} to Tag{post}
}

paginate Post, Tag with infinite-scroll
paginate Product with pagination

microservice Product with store
microservice Blog, Post, Tag with blog

deployment {
  deploymentType docker-compose
  appsFolders [gateway, blog, store]
  dockerRepositoryName "mraible"
}

If I update this JDL to use clientFramework: angularX, and generate it with the main generator, the gateway shows entities, and everything works as expected.

@mraible , I think there is confusion. Svelte blueprint doesn't yet support micro-frontends. This PR only adds support for Reactive Gateway and micro-service applications. Can you please test if that works for you?

I tried with shipster jdl reactive-ms and it seems to have worked. I was able to build Docker images and run everything with docker compose up. When I first tried to run npm run e2e in the gateway app, it fails after a while.
Still waiting to connect to Chrome, retrying in 1 second (attempt 62/62)
Cypress failed to make a connection to the Chrome DevTools Protocol after retrying for 50 seconds.

This usually indicates there was a problem opening the Chrome browser.

The CDP port requested was 58637.

Error: connect ECONNREFUSED <IP_ADDRESS>:58637
    at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1142:16)

Execution time: 8 min. 33 s.

I'll try again.

I dig the logo for this blueprint!

This time, all Cypress tests pass! 💯

I can't wait until Cypress run faster on M1.

Thanks for review @mraible
I see a few posts suggesting building local cypress binary; not sure if that's doable and gain.

@vishal423 : approved and good job :-)


Add activity feed API

#493  👈
main

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

Join @cameronvoell and the rest of your teammates on  Graphite


Leaflet >=1.1.0 L.DomEvent.fakestop()

L.DomEvent._fakestop() is L.DomEvent.fakestop() since Lealfet 1.1.0, which causes issues with opening bootstrap dropdowns in the sidebar.
This should fix and allow backward compatibility in addTo() and removeFrom() methods (synthax untested)
var fakeStop = L.version === '1.1.0' || '1.2.0' ? L.DomEvent.fakeStop || stop : L.DomEvent._fakeStop || stop;
will try to make a PR soon, but if someone does it before me - it would be nice to have the stop events listeners activated and deactivated from an external private function to addTo() and removeFrom() so they are all in one place.

why not just var fakeStop = L.DomEvent.fakeStop || L.DomEvent._fakeStop || stop;?

I've been using L.DomEvent._fakeStop() with 1.2.0 until I found out it just kept failing silently, but hopefully the evaluation dosen't fail silently?