Problem to include vegas on joomla

hi
I'm tring to include your plugin into a website made with joomla
but I can't see any background image
all seems ok
I tried to include it on my static website and it worked well
I try to associate it on a div but nothing
$doc->addStyleSheet(JURI::base() . 'modules/mod_maohomegallery/css/vegas.css', 'text/css', 'all');
$doc->addScript(JURI::base(true) . '/modules/mod_maohomegallery/js/vegas.js', 'text/javascript');
$doc->addSCriptDeclaration("
jQuery('#mainarticle').vegas({
slides: [
{ src: '".JURI::base()."modules/mod_maohomegallery/images/prova.jpg' },
{ src: '".JURI::base()."modules/mod_maohomegallery/images/prova1.jpg' }
]
});
");
the scripts are loaded correctly
thanks

Hello,
I'm not a Joomla expert, but your slide paths must be wrong.
Try something like:
slides: [
{ src: 'images/
or
slides: [
{ src: '/images/

Hi
I think the problem is not due from the path, because if I set src: '/images/ from firebug I get an error of wrong url, instead if I set  src: 'images/ all seems ok but the images don't appear. I try to change the div but with the same result.
try to have a look
http://maofree.changeip.net/travaglio/osteria-gelateria/
at first login set two times "maofree"
thanks

Hi
after some test I've solved that problem, now it works very well
thanks
bye

Share for other devs please.

ok
in the main file of the module it is necessary to use these instructions
for my site it was necessary also to add a line of css code to that div
height: auto !important;
to avoid a problem with the fixed height set from the plugin to it,
necessary like when I load the page when firebug is open, when I close firebug the height doesn't change, the images cover only a part of that div
 $doc->addStyleSheet(JURI::base() . 'modules/mod_maohomegallery/css/vegas.min.css', 'text/css', 'all'); $doc->addScript(JURI::base(true) . '/modules/mod_maohomegallery/js/vegas.min.js', 'text/javascript'); $doc->addSCriptDeclaration("     jQuery(document).ready(function(){         jQuery('#homegallery-background').vegas({             delay: 9000,             preload: true,             timer: false,             transition: 'zoomOut',             transitionDuration: 2000,             slides: [                 { src: '".JURI::base() . "modules/mod_maohomegallery/images/piazza.jpg' },                 { src: '".JURI::base() . "modules/mod_maohomegallery/images/giardino.jpg' },                 { src: '".JURI::base() . "modules/mod_maohomegallery/images/pasta.jpg' },                 { src: '".JURI::base() . "modules/mod_maohomegallery/images/bruschetta.jpg' },                 { src: '".JURI::base() . "modules/mod_maohomegallery/images/salumi.jpg' }             ]         });     }); ");
bye


UI for the notebook "old" client

As suggested here: https://discourse.jupyter.org/t/jupyter-archive-make-download-and-extract-archive-files/2363/13?u=hadim

I don't really have the time neither the need for this feature so I don't think I'll work on it in the near future. That being said I will be happy to review a PR.

As the classical notebook is retiring from active development, I let this issue open only if somebody is interested in implementing the feature.
Here are two repositories that could provide a code basis for this feature:

https://github.com/data-8/nbzip
https://github.com/ryanlovett/jupyter-tree-download

I'm closing this one as the classical notebook is retiring. For those interested by using this extension with a UI close to the classical notebook, I suggest using JupyterLab in simple mode.


Upgraded kruntime to beta3

Everything seems to work for me in windows both in cli + visual studio
My only concern is the nulls passed around in UpdateBufferFilterFacts.cs, if anyone has input on what should go in there, it would be much appreciated :)

The build is failing because of how Travis is handling c# builds. It can't deal with beta3 yet.
We can either wait until they fix Travis, or use our own build script (similar to https://github.com/OmniSharp/omnisharp-server/blob/master/.travis.yml)

Created an issue on travis-ci
https://github.com/travis-ci/travis-ci/issues/3273

@nosami Is this really Travis and not our build.sh script which still had references to ~/.kre in it?

well, our build.sh doesn't have a line 41, so that is a quick telltale sign that something is falling over that is out of our control.

Our code is cloned like this git clone --depth=50 git://github.com/OmniSharp/omnisharp-roslyn.git OmniSharp/omnisharp-roslyn
Our build.sh doesn't go into the home folder
curl -sSL https://raw.githubusercontent.com/aspnet/Home/master/kvminstall.sh | sh && source ~/.kre/kvm/kvm.sh && kvm upgrade
Downloading kvm as script to '/home/travis/.k/kvm'
Downloading kvm from 'https://raw.githubusercontent.com/aspnet/Home/master/kvm.sh'
Appending source string to /home/travis/.bashrc
Appending source string to /home/travis/.zshrc
Type 'source /home/travis/.k/kvm/kvm.sh' to start using kvm
/home/travis/build.sh: line 41: /home/travis/.kre/kvm/kvm.sh: No such file or directory

Just in case there was any doubt, I reran the last good beta2 build https://travis-ci.org/OmniSharp/omnisharp-roslyn/builds/51087932

https://github.com/Yantrio/omnisharp-roslyn/blob/beta3/scripts/Omnisharp#L13

@jrieken do you happen to know which branch of XRE was used for beta3? We need to copy the DTH models.

there's a tag https://github.com/aspnet/XRE/tree/1.0.0-beta3

https://github.com/OmniSharp/omnisharp-roslyn/blob/master/src/OmniSharp/AspNet5/AspNet5ProjectSystem.cs#L272
This line is failing with a json conversion error :(
A first chance exception of type 'Newtonsoft.Json.JsonSerializationException' occurred in Newtonsoft.Json.dll

Additional information: Cannot deserialize the current JSON object (e.g. {"name":"value"}) into type 'System.Collections.Generic.IEnumerable1[System.Collections.Generic.KeyValuePair2[System.String,Microsoft.CodeAnalysis.ReportDiagnostic]]' because the type requires a JSON array (e.g. [1,2,3]) to deserialize correctly.

To fix this error either change the JSON to a JSON array (e.g. [1,2,3]) or change the deserialized type so that it is a normal .NET type (e.g. not a primitive type like integer, not a collection type like an array or List) that can be deserialized from a JSON object. JsonObjectAttribute can also be added to the type to force it to deserialize from a JSON object.

Path 'CompilationOptions.CompilationOptions.SpecificDiagnosticOptions.CS1702'.

and the message payload appears to be correct to me
  "Framework": {
    "FrameworkName": "Asp.Net,Version=v5.0",
    "FriendlyName": "ASP.NET 5.0",
    "ShortName": "aspnet50",
    "RedistListPath": "C:\\Program Files (x86)\\Reference Assemblies\\Microsoft\\Framework\\.NETFramework\\v4.5.1\\RedistList\\FrameworkList.xml"
  },
  "CompilationOptions": {
    "LanguageVersion": 6,
    "Defines": [
      "DEBUG",
      "TRACE",
      "ASPNET50"
    ],
    "CompilationOptions": {
      "AllowUnsafe": false,
      "Usings": [],
      "OutputKind": 2,
      "ModuleName": null,
      "ScriptClassName": "Script",
      "MainTypeName": null,
      "CryptoKeyFile": null,
      "CryptoKeyContainer": null,
      "DelaySign": null,
      "CheckOverflow": false,
      "Platform": 0,
      "OptimizationLevel": 0,
      "GeneralDiagnosticOption": 0,
      "WarningLevel": 4,
      "ConcurrentBuild": true,
      "SpecificDiagnosticOptions": {
        "CS1702": 5,
        "CS1705": 5
      },
      "MetadataReferenceResolver": null,
      "XmlReferenceResolver": null,
      "SourceReferenceResolver": null,
      "StrongNameProvider": null,
      "AssemblyIdentityComparer": {},
      "Errors": []
    }
  }
}}```

Ignore that, Fixed that error :D

:+1:

Awesome PR

:clap:


why is it taking a 3 second delay to navigate to other pages on button click? help needed ,suggest solutions

Hi 👋 Do you use Kitten Tricks as a starter app?
Since this is not UI Kitten issue, I suggest you looking through react-navigation guidelines in case this is a framework of your choice

may be  related  to https://github.com/akveo/react-native-ui-kitten/issues/613#issue-493615327


[sram_ctrl/rtl] Use flop instead of prim_sparse_flop

This commit changes back the PRIM_FLOP_SPARSE_FSM to an ordinary flop.

The only failure here is the known broken spi_passthru_test_fpga_cw310_rom_with_fake_keys . Merging.


@conda-forge-admin, please add user @kvnkho

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @kvnkho in conda-forge/fugue-sql-antlr-feedstock#5.


Updated Readme.md

Updated the readme file and added the perfect link to the folder structure

updated the Navigation for the ease of User

Hi Sahil! The Folder Structure is indeed well formed but the Main Branch is Few Commits Forward than your commit and it will disturb the entire documentation of the Repo! This cant be merged due to certain issues! Thank you :)

What's the issue can you elaborate.
On Fri, Dec 22, 2023, 6:12 PM Prasanna Muppidwar @.***>
wrote:

Closed #21 https://github.com/ML-Nagpur/Cool-Notebooks/pull/21.
—
Reply to this email directly, view it on GitHub
https://github.com/ML-Nagpur/Cool-Notebooks/pull/21#event-11326231600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3JSSLJ6L6DUIAEEBWU7IMLYKV54XAVCNFSM6AAAAABA7YHJNCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGMZDMMRTGE3DAMA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Basically the Readme and Folder Structure is Updated but your Pr does not meet the Current Updated Folder Structure, Firstly you have to Sync your fork and then clone it to add your changes before merging the request to full fill our requirement for a merge.


Incorrect huber loss calculation

Thanks for sharing the code for your amazing work!
I have a question about your huber loss implementation. Shouldn't you take the sqrt of the data_dim to calculate huber_c?
Your code:
def loss_func_huber(x, y):
    data_dim = x.shape[1] * x.shape[2] * x.shape[3]
    huber_c = 0.00054 * data_dim
    ...

Corrected code:
def loss_func_huber(x, y):
    data_dim = x.shape[1] * x.shape[2] * x.shape[3]
    huber_c = 0.00054 * math.sqrt(data_dim)
    ...

Ah, thank you for catching this! We will reflect this in our new version of the paper.


[Match Solver] ETH Unwrap happens with sell amount on buy order

For this order: https://gnosis-protocol.io/orders/0x7d71e36b822483a3ac2f19a457262a194ee46f2dff13332499af1f975f834e465fb8b9512684d451d4e585a1a0aabfb48a253c6760e8be11
See this simulation: https://dashboard.tenderly.co/gp-v2/staging/simulator/1ed3a7fa-e32a-4137-a18d-cc3b8f15f84f
It's failing because it's trying to unwrap the sell amount (~420 xSushi) to ETH which doesn't work since the contract only holds the buy amount of ~1.9 WETH

The order cleared instead with the HTTP solver (https://logs.gnosis.io/goto/97ca9682a77631a91f78cb22bed0dd7a)
My hunch is that it's because here we use the sellAmount as execution regardless of whether or not it's a buy order (in which case we should use the buyAmount.

This should be fixed with https://github.com/gnosis/gp-v2-services/pull/821


Correct the Xcode fixits for dispatch_async() and dispatch_group_async()

The same change was already made in the swift repo (https://github.com/apple/swift/pull/6958)
rdar://problem/27915073

@swift-ci Please test

Test failure is unrelated to this change.

@swift-ci Please test

@dgrove-oss could you please take a look at the build output for Linux? It's failing with a timeout in the SwiftPM tests.

I think there is something broken in how the libdispatch PR testing is being done.  It has been habitually failing on this swiftPM subtest (and we have been ignoring the failure, merging the changes, and not seeing the failure in the main runs).    I'll ask for help on swift-dev list to see if anyone has an idea of what could be wrong.

@dgrove-oss thanks. The failure is clearly not related to the change, so I am going to go ahead and merge.


How can I include the debug version of ZED SDK libraries?

Hi All,
I am trying to create a test project which is from https://www.stereolabs.com/blog/how-to-set-up-a-multi-camera-network/ But when I include "sl/Camera.hpp (#include <sl/Camera.hpp>)", I am getting the below build (Debug build) errors:
/usr/local/zed/include/sl/Camera.hpp:95: error: #warning ("'Debug' builds are not supported since this library was built in 'Release', 'RelWithDebInfo' should be preferred to avoid crashes and memory issues") [-Werror=cpp]

So how can I include the debug version of ZED SDK libraries?
Regards,
Rajesh

This should be a warning, not an error.
In any case, try to call cmake with -DCMAKE_BUILD_TYPE=Release argument to see if it solves

Hi @obraun-sl,
Thanks for your reply. The release build works without any issue, but I wish to make the debug build for the same.
If I define "ALLOW_BUILD_DEBUG", then I can take the debug build. Is there any other option ?
Regards


Is LMv2 using resnext101 or just res101?

Describe
Model I am using (LayoutLMv2):
Hi, I'm doing some works based on LMv2, but I found FUNSD's F1 score is lower than that reported in paper.
So, I want to know the open source code using which visual backbone now?
It seems that the provided code about detectron2 config is using Res101 rather X101, so the f1 score is lower.
I am not sure my option is right or not.
BTW, I want to know that if I train CORD dataset based on BERT-base with longer epoches, BERT-base f1 score will higher than BERT-Large?
(When I reproduced CORD dataset on LMv2, I trained LMv2 10epoch rather than 5epoch, I found my BERT-base test f1 score = 96.3)

Hi @Senwang98 , we use resnext101 as the visual backbone, I think loading the weight from the pre-trained model directly is ok, just make sure parameters are loaded correctly.
For the 2nd question, I think the first step is checking the evaluation and data process code,  if there is nothing wrong in the code, try to make sure whether this phenomenon could be reproduced on other models(such as Bert).

@Dod-o
I trained LMv2(layoutlmv2-base-uncased) with CORD, 5 epoch achieve 95.4 and 10 epoch got 96.03
***** eval metrics *****
  epoch                   =       10.0
  eval_accuracy           =     0.9776
  eval_f1                 =     0.9741
  eval_loss               =     0.1345
  eval_precision          =     0.9756
  eval_recall             =     0.9725
  eval_runtime            = 0:00:02.73
  eval_samples_per_second =     36.594
  eval_steps_per_second   =      4.757
***** Running Prediction *****
  Num examples = 100
  Batch size = 8
 92%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████▎          | 12/13 [00:02<00:00,  4.59it/s]***** test metrics *****
  test_accuracy           =     0.9728
  test_f1                 =     0.9603
  test_loss               =      0.162
  test_precision          =     0.9592
  test_recall             =     0.9614
  test_runtime            = 0:00:02.70
  test_samples_per_second =     36.974
  test_steps_per_second   =      4.807

hi @Senwang98, The CORD code is similar to FUNSD, refer to the FUNSD code is ok.
And I'd like to know how did you calculate the F1?  Did you use the token-level F1 rather than entity-level?

@Dod-o
I copy the metric codes from run_dunsd.py

@Senwang98 I think we use entity-level metrics, maybe you can double-check about this.


Server crash (1.12.2)

crash-2020-09-19_16.52.11-server.txt
latest.log

Hi there! Thanks for reporting this and apologies for the delay, this should now be fixed in the latest version. Thank you for your patience :)


How to set values from HttpContext when doing BulkUpload

Hi,
I have implemented some custom EventHandlers that log certain information to the DB using this function:
 public void AddAuditEntry(UserAccountEvent<CustomUser> evt, string additionalDetail = null)
        {

            using (var db = new CustomMembershipRebootDatabase())
            {

                var audit = new AuditTrail
                {

                    Date = DateTime.UtcNow,
                    Activity = GetTypeNameWithoutGenericArity(evt),
                    Detail = additionalDetail,
                    ClientIP = HttpContext.Current.Request.UserHostAddress,
                    Username = evt.Account.Username,
                    Browser = HttpContext.Current.Request.Browser.Browser,
                    UserAgent = HttpContext.Current.Request.UserAgent,
                    Source = Constants.ApplicationName,
                    UpdatedBy = GetUpdatedBy(evt)
                };
                db.Audits.Add(audit);
                db.SaveChanges();
            }
        }

This works great from all my synchronous calls in my site.
However, when I introduced this new logging functionality, it breaks my BulkUserUpload. My bulkuser upload loads a file from the html file and processes it in its own Task so that it can return a result to the user right away. It then emails the user upon completion.
Task.Run(() => bulkDS.ParseCSVFile(model.File.FileName, csvStreamReader, client, app, emails,  model.SendWelcomeEmail));

Because the task runs in its own thread, it doesn't have access to the HttpContext because it is null.
I know I could pass the values of UserHostAddress, Browser, UserAgent etc, through my Task.Run, but I don't know how I could get them to be used by my event handler.
All I am wanting to do, is have the user get a response back ASAP after they submit their file to let them know it has been received and is being processed. I don't want them to have to wait until its completed.
Do you have any suggestions on how I could rework things to allow this to work?

Yea, you're going to have to copy those values. Or pass the context... but not sure what sort of trouble that might get you into.

So I'm thinking the best way to pass these through would be updating the eventHandler and updating my config like so:
  Config.AddEventHandler(new AuditEventHandler(HttpContext.Current.User as ClaimsPrincipal));

This would be ok, but I'm using a static MembershipRebootConfiguration as per the samples in the Customizations Sample. Does the MembershipRebootConfiguration have to be static? Will there be any odd effects if its non-static?
My other thought, is that I could remove the Task.Run from my controller action and make the call synchronous. Then in the View, I would call it using Ajax, instead of a full post back. This might give me the desired result. The only concern here, is if the user navigates away from the page, the ajax response won't be received. This is ok from a user perspective because they'll be sent an email on completion of the process.


[CDEC-416] Sinbin Test.Pos.Infra.Arbitrary.Slotting

Description
Free up infra-test by moving Test.Pos.Infra.Arbitrary.Slotting to sinbin-test.
Linked issue
https://iohk.myjetbrains.com/youtrack/issue/CDEC-416
Type of change

[~] 🐞 Bug fix (non-breaking change which fixes an issue)
[~] 🛠 New feature (non-breaking change which adds functionality)
[~] ⚠ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
[~] 🔨 New or improved tests for existing code
[~] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

[x] I have read the style guide document, and my code follows the code style of this project.
[x] If my code deals with exceptions, it follows the guidelines.
[x] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.

Testing checklist

[~] I have added tests to cover my changes.
[x] All new and existing tests passed.

Ignore the shellcheck failure.

@erikd I just tried to here but the original version (on develop) actually doesn't even run for me. It just creates copies of all the cabal files, but with -e as a suffix 🤷♂
Are you able to run the script from develop successfully on your machine? If so you could test if my modified script works 🙂


One more please

Good Alberto! I'll rephrase my question)
If I load 50 keys into BAGS mode, can the software find any of these 50 keys? Can the software find key #48 or key #35?
Thanks Alberto! please

Are you becoming so annoying. If the keys are in the given range it will find it.
Did you already did some test?


Add empty gaslighting tag file to fix error when hexdoc-hexcasting is not present

wait, i already did this.
https://github.com/hexdoc-dev/hexdoc/blob/21ce01237097c5caa11685fec2202a11fc18467c/src/hexdoc/_export/resources/data/hexdoc/tags/items/gaslighting.json


feat(router) expose router as a singleton

Exposing the router as a singleton allows for more introspection of the defined api routes programmatically

Manually merged with minor edits!


test and add builds for windows

Well I just successfully used this on windows. Still totally failing on the test side of things though (I don't have any).

capture adding tests in #2


dependency tree

to help users with debugging, let's create a visual dependency tree with "major" dependencies, with comments about what each one does. Not sure how to approach this yet, but i think it would help

Don't think so.. For that is the stacktraces? It is something usual issues to accept and use these stacktraces.
Or okey, think it. if we create it that way, it won't help to some newcomers or whoever, they must know the logic and cycles behind, core api and etc. These one-lines comments in form of dependency tree, won't help. It is easier for them to just paste the stacktraces which in turn are more useful for us, instead of force them to think so much.
Hm, or maybe to show by default your idea as dependency tree, and with --verbose to show only stacktraces.

Maybe you misunderstand (or I do). I want to add something that looks like this to the docs:
generate
├─┬ assemble-core
│ ├── assemble-fs
│ ├── assemble-render-file
│ ├── assemble-streams
│ ├── base-task
│ └─┬ templates
│   ├── base
│   ├── base-data
│   ├── base-engines
│   ├── base-helpers
│   ├── base-option
│   ├── base-plugins
│   └── base-routes
├── assemble-loader
├── base-fs-conflicts
├── base-fs-rename
├── base-generators
├── base-npm
├── base-npm-prompt
├── base-pipeline
├── base-questions
├── base-runner
├── base-runtimes
├── base-store
├── generate-collections
└─┬ generate-defaults
  └── base-pkg

btw, it's not about stack traces, it's also about finding the documentation you need or even where to start

Oh yea yea. Don't know why I was thinking for output/commands/options, lol :D
So yea, it would be in some help. :)

lol np.  fwiw I already started working on something to do the tree part if you want to play around with it. It's not wired up anywhere, so you'd need to do node lib/tree from the root of generate.

closing since this was addressed


Handle OperationCancelled when peer disconnects during DAO fork check

Closes: #1234

Alternative solutions I considered:
In order to perform the dao check in Peer._run() we'd need to run it in the background
and use a notification mechanism other than an Event so that we can tell the PeerPool
whether the check was successful or not. That is very error prone, though, because a
callsite may very easily forget to wait on that condition and start to use the peer straight
away. And we have no way of putting checks on peer methods to avoid them being called before
the DAO fork check has completed because we need to use those methods in order to perform
the DAO fork check.
Also thought about having the DAO fork check in the handshake() func, but then we'd have to
duplicate some code from _run() to read messages, and it'd also have to return the messages
received during the DAO check so that the PeerPool can re-add them to its subscribers. In
fact it'd require a lot of changes to make that work because the peer is not running at that
time and the new ExchangeManager (which is used during the DAO check) stuff requires the
peer to be running.

Alright, seeing this I want to take a stab at an alternate idea.

Alright, it's really messy but I think it actually attacks the root of the problem as well as doing some cleanup migrating things that still live in the p2p module to be under trinity.
see https://github.com/ethereum/py-evm/pull/1260 as my proposed alternative to this PR

#1260
#1259 I think

#1265 is now the correct link.


Increment version for identity releases

Increment package version after release of com.azure azure-identity

Closing this PR in favor of another release for Identity lib https://github.com/Azure/azure-sdk-for-java/pull/22821


REST API Create variable group uses wrong property for name

https://docs.microsoft.com/en-us/rest/api/azure/devops/distributedtask/variablegroups/add?view=azure-devops-rest-6.1
api-version=6.1-preview.2
Unless I am completely mis-reading the documentation, the JSON below uses variableGroupProjectReferences.name for the variable group name.
If this is the intended behaviour, what is the purpose of the root name property, and how can variableGroupProjectReferences be used to link/permit the variable group to be used within a project?
It would be useful to provide some snippets alongside the REST documentation.
POST https://dev.azure.com/{{organization}}/8a217fcd-ca15-4def-9e86-b42fe27d5fcd/_apis/distributedtask/variablegroups?api-version=6.1-preview.2
Authorization: Basic {{fullToken}}:{{userId}}
Content-Type: application/json

{
    "name" : "phil-var-group-1",
    "description" : "phils var group",
    "variableGroupProjectReferences" : [
        { "name" : "proj-ref", "projectReference" : {"id" : "8a217fcd-ca15-4def-9e86-b42fe27d5fcd" } }
    ],
    "variables" : {
        "key-secret-01" : { "isReadOnly" : false, "isSecret" : true, "value" : "abc"},
        "key-normal-01" : { "isReadOnly" : false, "isSecret" : false, "value" : "def"},
        "key-readonly-01" : { "isReadOnly" : true, "isSecret" : false, "value" : "ghi"}
    }
}

Resulting in a variable group that looks like
{
  "count": 1,
  "value": [
    {
      "variables": {
      },
      "id": 1,
      "type": "Vsts",
      "name": "proj-ref",
      "createdBy": {
        "displayName": "xxx",
        "id": "1f70b7e8-0393-4e44-94cb-1f43efbcf17f",
        "uniqueName"<EMAIL_ADDRESS>      },
      "createdOn": "2021-08-03T19:01:52.0833333Z",
      "modifiedBy": {
        "displayName": "xxx",
        "id": "1f70b7e8-0393-4e44-94cb-1f43efbcf17f",
        "uniqueName"<EMAIL_ADDRESS>      },
      "modifiedOn": "2021-08-03T19:01:52.0833333Z",
      "isShared": false,
      "variableGroupProjectReferences": null
    }
  ]
}

@Philcartmell - It looks like your have a product question. Consider these resources:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow

As for your documentation suggestion, you're referring to a REST API topic, which isn't part of this repo. I've flagged an author who may be able to address code snippets in the REST API.
@wnjenkin for awareness.


Sorting problem

Picture tells everything.
This is situation, when I open page. So, no sortings done before that...

Look at Id.

the setArray wont trigger sorting, have you chacked the array you are passing into the grid?
ps!
on booleans and numbers you need go use value converter to keep them from turning into text
because that will mess up future sorting and filtering

Converter didn't help (converter copied from you git).
Rows will be sorted only after: if I click column "sort uparrow/downarrow".
{colField:"Id", colField:"Id | numberFormatter", colSort:"field:Id;asc:true", colHeaderName:"Id", colType:"number", colWidth:100, colFilter: "field:Id;" , colFilterMenu:"filter:Id",colAddRowAttributes:"readonly"},
Data, what is coming in to client from server is ALWAYS min-id ---> max-id order. The way, how the sql server saves data to table. I do not use/accept reusing deleted record slots in table.

Yep, as I said. Record with id 79 is the first one on datastream incoming to client...

So the grid isnt the issue, its the data you are getting, or are I missing something?

Grid IS the issue. Grid shows wrong record on first line (Picture above).

k, Ill have a new look at it later today when I get a chance to se if I manage to reproduce it

Rechecked data
Yep, as I said. Record with id 79 is the first one on datastream incoming to client...

Before I get a chance to test this later
Can you do me a favor and do console.log(yourData[0]) right before and right after setting data to array
console.log(yourData[0])
datasource.setArray(yourData)
console.log(yourData[0])

Not at my computer so cant test, but looking at my code, having a hard time seeing how and why, but maybe Im blind today, happen before 😄
Btw, are you doing any filtering after setting the data ?
Because that will mess up order if you havent set any.

I'm doing first filtering instantly after setting data.
Original data order remains same as after setting data/filtering, but
order in grid is wrong after first "empty" filtering...
I set only couple filter equations on "initially filtering" like this:
this.ds.query([{attribute:"TargetCondition_s", operator:'', value:""},{attribute:"TargetAssemblerName", operator:'', value:""}]);
This was only for testing to get operators ready. But I need to set * operator to every columns
at start anyway...

I set only couple filter operators on "initially filtering" like this:
this.ds.query([{attribute:"TargetCondition_s", operator:'', value:""},{attribute:"TargetAssemblerName", operator:'', value:""}]);

This is the cause of the issue then, just need to set sort order right after, maybe I should add a de
When you do filter in js the result isnt sorted.
maybe I should use the key I set in each object as default if no sort order exsist, so filtering always reproduces same order when no column is sorted

@JOKaija
I pushed out a new build
When setting the array with "setArray() it will set the internal key as sort order by default when no other sort is set, that way result for a filter will come back in same order thay came in, until user sorts in some other way
Some of these parts will evolve over time in the default datasource (array only)
Later I want to create a easy to use with remote data for paging, or caching that tracks the entities so you can define a better schema and have undo options etc, skiped that parts because if you feed the grid with 50 rows and want to created tracking entities it will take a few seconds in slow hardware.
Reopen the issue if this new build did not help you

Now it works. Great!

@JOKaija
Info about sorting:
Added option to set locleCompare to use, since so many uses english browser settings but need sorting to be able to sort locale language
Sample I used to get it to sort Norwegian (my main language)
this.datasource.setLocaleCompare('nb-NO'); //switch out with you locale if you need something else
More about localCompare here:
https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/String/localeCompare
List of codes
http://stackoverflow.com/questions/3191664/list-of-all-locales-and-their-short-codes


Duplicate welcome message for Direct Channel

Sample information

Sample type: samples
Sample language: nodejs
Sample name: 03.welcome-users

Describe the bug
The problem described at https://blog.botframework.com/2018/07/12/how-to-properly-send-a-greeting-message-and-common-issues-from-customers/ occurs when using the sample code with the example provided at https://github.com/Microsoft/BotFramework-WebChat/tree/master/samples/15.d.backchannel-send-welcome-event
The effect of this is that the bot issues two welcome messages.
The problem is because DirectChannel sends two ConversationUpdate messages, which results in two welcome messages.  The local emulator squashes this.
To Reproduce
Steps to reproduce the behavior:

Install the sample at https://github.com/Microsoft/BotFramework-WebChat/tree/master/samples/15.d.backchannel-send-welcome-event
Install 03.welcome-users
Connect via ngrok
See that you get two welcome messages when you load the html page

Expected behavior
I expect to see just one welcome message displayed when loading the html page
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
I asked for help on StackOverflow (https://stackoverflow.com/questions/54443597/microsoft-botframework-webchat-is-getting-two-welcome-messages)
The advice given there is very difficult to fit into the pattern supplied in the samples.  I couldn't work out how to catch the event since my bot class does not have an "on" method.
Workaround
Check for the presence of entities in the conversation update
if (typeof turnContext.activity.entities != 'undefined') {
Doing this breaks the view for the emulator, so you also need to check for an emulator specific channel:
if (typeof turnContext.activity.entities != 'undefined' || turnContext.activity.membersAdded[0].name == 'User') {
In context this looks like:
if (turnContext.activity.membersAdded.length !== 0) {
    // Iterate over all new members added to the conversation
    for (var idx in turnContext.activity.membersAdded) {
        if (typeof turnContext.activity.entities != 'undefined' || turnContext.activity.membersAdded[0].name == 'User') {
            if (typeof turnContext.activity.entities != 'undefined') {
                // only users have got the entities key
                await dialogContext.beginDialog(MASTER_DIALOG)
            }
        }
    }
}

[bug]

Anyone free to pick up this one?
Investigate how our 15.d interact with Bot Builder sample 03.
Because we have recently going back-and-forth about the topic (dupe conversationUpdate or not), so it might already fixed or not. If not, we probably need to update 03 to match the latest behavior (Web Chat should send an event to the bot to indicate Web Chat is connected, then the bot send welcome message, once.)

We believe this is addressed in https://github.com/Microsoft/BotFramework-WebChat/issues/1371. Please re-open if this is still an issue on WebChat >4.3.3


Automatically set relativePaths to false when publicPath is used

Automatically set relativePaths to false when publicPath is used (non empty string).

Fixed in v4.


EventPipe session was not disposed upon stopping exception

Describe the bug
Event pipe session isn't disposed when there's exception.
To Reproduce
Steps to reproduce the behavior:
For whatever reason, when there's exception, the event pipe session should be disposed upon stopping.
Expected behavior
Dispose being called upon exception.
...
                try
                {
                    if (_currentSession != null)
                    {
                        _...

                        if (disposeEventSessionImmediately)
                        {
                            DisposeEventPipeSession();
                        }
                    }

                    _logger.LogTrace("[{typeName}] Disabled.", _typeName);
                }
                catch (Exception ex)
                {
                    _logger.LogError(ex, "Unexpected exception stopping session.");
                    // !!!!!! Dispose should have been called here !!!!!!!
                    throw;
                }
                finally
                {
                    _singleTraceSessionHandle.Release();
                }

The same should be done for start logic.

Checked the code carefully and disposing is actually happening in the caller. Close this one.


Fix stop-bitcoin-stats

Typo in clearing populate.connections.
Missing clear for populate.addresses.

@stridentbean @tmconnors I see this bug is still present in #30


v3rpc: don't elide next progress notification on progress notification

Fixes #5878
Clocked parts of the failing test, noticed the wait time was about 6s instead of 3s.

@heyitsanthony Great!!!! My double mistakes finally caused problem to myself... I failed to figure out why after quite a while investigation.

lgtm


Exclude lesioned areas from confound masks.

Segmentation algorithms are often confused by lesioned areas, resulting in incorrect automated segmentation results in these regions. As such, it is good practice is to subtract the lesion mask from these tissue masks before calculating confound time series (e.g. mean WM/CSF signals, CompCor), which rely on tissue masks.
When FMRIPREP detects a lesion mask and uses it during spatial normalization, it should also consider this mask when calculating confounds.
[I'm happy to implement this at some point. I'm mostly adding this ticket as a reminder to myself and so that others are aware of this issue.]

This might be of interest to @rciric. Thanks!!

Speaking of @rciric: For now, I am working on a hack that re-runs the aCompCor node (outside of fmriprep) with a modified mask and then pumps the corrected regressors into xcpEngine. Will post a note here to the code when it's ready.

Hi, I am also very interested in this - so thank you for working on it! Quick question - is the lesion mask used in registration of EPI to T1w? I'm not sure what the best method is to improve this step...

Hi @mitchellxh, it is not. I suspect that in general, the boundary-based registration will perform poorly with lesioned data, so the rigid coregistration will be used.

Thanks @effigies. That's last part of the puzzle for my data.
Also, I've had success using enantiomorphic normalization to getting normalized T1w images and allowing for Freesurfer to run. Is this functionality too niche in potentially including as an option in the pipeline? Thanks!

It doesn't strike me as overly niche, but I don't know this space very well. Is the community working with lesion data converging around a few core algorithms at this point, or is it still largely exploratory? The reason I ask is that it will affect our approach to inclusion.
If this is one of the "standard" algorithms for lesioned data, then it makes sense to add it, and use it opportunistically when a lesioned brain is detected. sMRIPrep is where our structural pipelines now live, so if this is the route we go down, that's where you should open an issue and we can start orienting you to the code and work out how best to incorporate this improvement.
If there's still a lot of controversy over the best way to do things, then it may make sense to go with more of a plugin architecture, which @oesteban, @satra and I have been talking about a bit. The idea there is to allow users to utilize the core of fMRIPrep, while keeping modifications in a public repository that can also be audited. This should help us turn fMRIPrep from a tool for users who want bog-standard fMRI preprocessing to one that can enable development of new methods or analysis of new imaging types. We don't really have a process or template for fMRIPrep plugins yet, but if that's what you're interested in, we can figure it out as we go.

@effigies  - perhaps this is a use case of an fmriprep plugin?

@satra @effigies Agree that the enantiomorphic registration sounds like a good use case for a plugin.
The cost-function masking was already built-in to ANTs, so enabling it in fmriprep was simply a matter of adding a new input (the lesion mask) and tweaking a few things in the existing registration workflow.
My understanding of enantiomorphic registration is that it involves replacing lesioned voxels with homotopic voxels from the undamaged hemisphere, and so adding this to fmriprep would likely require building that new functionality.

Thanks, and yes - there is not a converged method in terms of lesioned fmri preprocessing. That said, I have had good results doing a combination of this enantiomorphic registration for T1w correction and applying a T2w intermediate between the EPI and T1w image (lesions will have same contrast as EPI, yet same spatial-resolution as T1w). Much of the method is described by the Yourganov/Rorden paper here.
Having a plugin for this would be great! There is an existing SPM toolbox (developed by the authors) that I've used that requests at minimum the T1w image and lesion-map in T1w space. However, it will also accept and transform a T2w lesion-map and T2w image (which is what we tend to use for lesion mapping), helping automate getting the lesion-map in T1w-space.
Let me know how I can help facilitate creating such a plugin. Thanks again!

here is a general plan for plugins, with the current assumption that plugins are sequential:
inside fmriprep
# first create the fmriprep wf based on cmdline specs
wf = ...
for plugin in plugins:
    plugin_func = load_plugin(plugin, fmriprep_version)
    wf = plugin_func(wf)

wf.run(...)

for a plugin:

create a function that takes a versioned fmriprep workflow as input and returns a modified workflow
for the plugin creator, this will tantamount to changing nipype workflow using disconnect and connect methods, and in some settings modifying the data grabbing nodes.

I'm back to working on lesion data after a bit of a break and figured I'd update this thread. I actually ended up creating a NiPype workflow that piggybacks on an existing FMIRPEP working directory to re-run aCompCor and generate regressors after subtracting lesioned voxels. The vast majority of the workflow is just nodes copied from FMRIPREP with an extra customsubtract_roi node dropped in between the acc_tfm_lf node and the acompcor_lf node. In this sense, it sounds a lot like the plugin architecture that @satra described.
I haven't tested it with any version of FMRIPREP other than the one I developed it with (1.3.2), so it would likely need to be updated to work with the current version. It's designed to be run from within the FMRIPREP singularity container.
It would be nice to get this properly integrated into FMRIPREP (as it builds directly on the work we did to implement constrained cost function masking), but I don't have the bandwidth to tackle that right now myself.

I realize this is a very out-of-date question but just wanted to loop back and ask about some of the existing denoising methods in place for when a lesion-mask is detected for clarity sake.
Is the lesion-mask currently transformed to our requested standardized space? If so, where might they be saved? And is it in-fact subtracted from the workflow that calculates csf, wm, and aCompcor confounding time series as described here?
And if we were to add the --use-aroma flag, is the lesion mask subtracted before melodic is run? I ask because I'm interested in running melodic on the normalized bold without the lesion subtracted, so that I can manually denoise components that overlap with the lesion-mask.


I want to add a weather app made using HTML, CSS , JAVASCRIPT and API.

Describe the project
A clear and concise description of the project you are going to add to this repository.
I want to add a weather app made using HTML, CSS , JAVASCRIPT and API.
Issue Type:

[ ] Bug Report
[ ] Feature Request
[ ] Documentation
[x] New Project

Additional context
Add any other context or screenshots about the project here.
Screenshots

If applicable, add screenshots to help explain your problem.

@TusharKesarwani kindly assign me this project and mention if any changes needs to be made in the UI or API fetch or anything.

@ashutoshgithubs Proect already exists


New: Email Column Shorcut Creation

This PR introduces the ability to create email columns via a direct shortcut, optionally setting the column to unique and also controlling the size via the length parameter, this saving time in writing email columns using the string method (the method known and used until today)
Use Cases
Before:
$table->string('email')->unique();

With the PR proposal:
$table->email();

Alternatives (based on PHP 8+):

Setting a different name for the column:

$table->email('destination_email');

Disabling the unique:

$table->email(unique: false);

Controling the length to the column:

$table->email(length: 50);

I don't think we should be mixing in user land concepts with database schema's. Email isn't a real database column type in any engine afaik? Maybe I'm wrong.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Update sbt-unidoc to 0.4.0

Codecov Report

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

@@           Coverage Diff           @@
##           master     #745   +/-   ##
=======================================
  Coverage   79.69%   79.69%
=======================================
  Files          35       35
  Lines         655      655
  Branches       23       23
=======================================
  Hits          522      522
  Misses        133      133

Continue to review full report at Codecov.

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

Thanks @n4to4!


How to use this plug-in?

Hi, I see that this plug in has been updated from time to time, but the sound-driver.plg remained the same.
May I know how to use this plug-in? I want to try to install it on my unraid, and use it for my docker instances. Thank you.

This plugin is not active supported because it mainly supports USB Sound devices.
Just install it, if you see your sound devices you are in luck if not try using a USB Sound device <- I can't do much about the fact that most built in sound cards (AMD/Intel) systems won't work since I would have to rebuild the whole Kernel and provide it through the plugin which I won't/can't do.

@dwong33 do you still need help or is your question answered?

Closing this issue due to no response. If you still having trouble running the plugin then please feel free to open up a new issue or reopen this issue.

(I replied the email but not appearing here, strange)
Thank you so much for your follow up! I am using a Asus VC 65, which has a built in sound card. So in this case, it is not likely to make it works?
Thank you.

Thank you so much for your follow up! I am using a Asus VC 65, which has a built in sound card. So in this case, it is not likely to make it works?

Sadly enough no. I would recommend that you buy some cheap USB adapter, these will usually work with the plugin.

Sure, let me get one from aliexpress, and try that! If I have further configuration problem. I will submit a ticket here again. Thank you!


Docs: Updated page about the Python Driver

@rhsimplex Do you know why the Travis CI build failed on this pull request? It only changed some docs; it didn't change any code.

Current coverage is 96.36% (diff: 100%)

Merging #667 into master will increase coverage by 0.51%

@@             master       #667   diff @@
==========================================
  Files            21         21
  Lines          1155       1319   +164
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1107       1271   +164
  Misses           48         48
  Partials          0          0

Powered by Codecov. Last update 73ed9c4...79bf698


Perform tests

Test both python and cpp server on:

Messages sending/receiving with packet loss
Server disconnect
Enormous amount of users

And fix the problems if possible.

The problem with users was caused by weak webserver package. Other bugs were fixed.


Order of settings in CLI help is not intuitive

For some reason, quiet and verbose are randomly in the middle (and non-consecutive).

The help menu is also generally way more verbose than I need. I end up seeing options that are very advanced. I wonder if there's anything to be done about that?

@zanieb - Did this also get fixed, by chance?

Nope. I'll take care of it though.

I did this now


This should fix #311
Added handling for absolute Windows file paths.
Added some additional tests around the url.fromFileSystemPath() to help catch this error in the future as no existing test was failing.

Kind of wondering, why is this library doing all this weird stuff with checking for windows and manual path fixing when there's available libraries that do this all for us? For example, node's in-built path library

Hi please merge this if possible

@jonluca anything preventing this from being merged?

Kind of wondering, why is this library doing all this weird stuff with checking for windows and manual path fixing when there's available libraries that do this all for us? For example, node's in-built path library

Yeah I agree - I think I'd prefer to just go through and fix all these issues directly. Will take a look right now, and if I don't see a fast way of fixing it I'll merge in this PR.

Thanks for the PR - I'm going to release a breaking change soon with changes that should fix what this is targeting. In general I think we should revamp the windows logic, like @meepen said.

https://github.com/APIDevTools/json-schema-ref-parser/pull/321


uv add hangs with circular dependencies in the lock file.

Originally posted in #4599
I've encountered a consistent hang when there's a circular dependency in the lock file.
name = "foo"
version = "0.1.0"
requires-python = ">=3.9"
dependencies = ["poetry"]

When I run uv lock followed by uv add flask, the process hangs indefinitely.
Did a little digging myself and I found a loop in the lock.rs file that appears to cause the issue. Adding a check to skip the loop iteration if the key already exists in the map seems to fix the problem for me.
https://github.com/astral-sh/uv/blob/796171e1e6d55dae095effebb6d815ac91727c2c/crates/uv-resolver/src/lock.rs#L354-L380

Thanks!

I shall fix.


Question - Razor Pages -  What is the CSS in Head?

I created a default Razor Pages project. When I run the page, there are two style tags in the head.
<style type="text/css">embed[Height="175"][Width="175"],... , <style type="text/css">.ggbox,....

I've stripped out everything I could from the app (the Analyzers are left) and all of file contents but they are still there. What are they?

Razor pages along with tag helper functionality have the ability to add additional content to certain tags, such as the <head> and <body> tags.
I'm guessing this is maybe injected by the Visual Studio Application Insights feature, which automatically lights up some diagnostic features when running locally in Visual Studio. A similar bit of functionality can be easily enabled when running on Azure Application Services.
But, without seeing more of the specific HTML or JS, it's hard to say for sure.
Does anything else within the content seem to indicate that it's somehow related to Application Insights?

It's not a Razor thing. It looks like the browser or an ad blocker is putting that in there. I don't look in head that often, but since I was look at something new (Razor Pages) I was looking at everything. Slipped my mind that the browser is external and not part of the dev env... sry.

Ah good to know! That CSS did look a bit odd to me! 😄


Missing props in mapStateToProps

Version
4.1.1
Steps to reproduce
This component is querying a db via GraphQL thanks to Apollo. It is a GraphQL container.
It uses compose from react-apollo library in order to use multiple enhanchers at the same time, in a readable way.
Example component:
import React, { Component } from 'react';
import { connect } from 'react-redux';
import { graphql, compose, withApollo } from 'react-apollo';
import { myQuery } from './graphqlQuery';

class MyComponent extends Component {
  render() {
    const { exampleProp } = this.props;
    return null;
  }
}

function mapStateToProps(state, ownProps) {
  // I expect to see ownProps.exampleProp here but it is undefined
  console.log(ownProps.exampleProp);
}

export default compose(
  withApollo,
  connect(mapStateToProps),
  graphql(myQuery),
)(MyComponent);

Expected Behavior
ownProps should contain the props passed to the component as stated here
like:
Object = {
  client: ApolloClient
  exampleProp: "propcontent",  // <-- this is going to be lost
  history: Object
  location: Object
  match: Object
  staticContext: undefined
  __proto__: Object
}

Actual Behavior
instead ownProps contains only this:
Object = {
  client: ApolloClient
  history: Object
  location: Object
  match: Object
  staticContext: undefined
  __proto__: Object
}

All the props that the component should have had from parents and from GraphQL response are missing, including, in this case, exampleProp.

This doesn't have anything to do with React Router.

It happened just when I updated it from react-router v3.0.2, so I think it has.
Couldn't it be the connection with redux?

Then it's most likely cause by this: https://github.com/ReactTraining/react-router/blob/master/packages/react-router/docs/guides/blocked-updates.md

I read your link, thanks, but it doesn't seem the cause to me first because my component is actually re-rendered and second because is not location aware (I don't even try to read the location).
I tried to modify the export part like this:
import { withRouter } from 'react-router-dom'

// ...

export default compose(
  withApollo,
  withRouter(connect(mapStateToProps)),
  graphql(myQuery),
)(MyComponent);

but it won't work neither


QueryEditor: Adds inline query editor scene object

Factored PanelChromeRenderer out of VizPanelRenderer to be used with  QueryEditorRenderer.
Added generic utility function getClosest for finding the closest scene object that matches user-supplied criteria. Refactored existing sceneGraph functions to use this utility.

Leaving this PR as a draft for now in case there's any feedback or I've made any major mistakes 😅

Why would we use PanelChromeRenderer when rendering QueryEditor ?

To summarize what we just discussed:

Let's change the implementation slightly so that it allows editing closest's SceneQueryRunner queries.
No add/remove button - basically one can only change what's defined in SQR.
Add demo in scenes-app demos.

I've implemented the changes that have been suggested and added a small demo.
One issue I've left unsolved for now is how to handle multiple queries (for now the editor component just deals with one query). Should we just have a scrollable list of query editors, or a tabbed view or something? What do you think @dprokop @torkelo

One issue I've left unsolved for now is how to handle multiple queries (for now the editor component just deals with one query).

@kaydelaney I've just tested this, and also looking at the code multiple queries are supported. I'm confused :)

Added support after I left that comment 😅

Implemented some of the suggested changes @torkelo

@kaydelaney I've pushed sync with main and a minir layout fix


Run script from Portainer

Is your feature request related to a problem? Please describe.
is it a possible to add functionality in which:
administrator can chose sh/cgi script (not upload, script on server) and put it under the button in the Portainer (and optionally specifies number of parameters). Whereas a normal user only has the option of clicking on this button and entering a variable, the script result would be presented to the user. This would give greater possibilities of using the Portainer.

Not something I have an immediate need for but I can see it would help to provide some custom self-servicing to the users. I'd want it to be able to run Powershell as well however as we're on Windows.

@krystian-er is this actually duplicate with #2181 which is already implemented but not released yet?

@olljanat not really
#2181 gives the ability for a user to execute a job on a Docker host via a privileged container. This allows you to execute commands that are available on the hosts and thus would probably partially solve the OP issue.
Regarding the ability to run pre-configured scripts for standard users, this is something that we'll have to discuss.
cc @ncresswell

I do want to see our “job/task execution” evolve to include common tasks as simple “click to execute” buttons (eg upgrade docker to latest version, backup all volumes etc).
We will collate a list of frequent tasks and evolve this.
From: Anthony Lapenna<EMAIL_ADDRESS>Sent: Thursday, 1 November 2018 9:31 AM
To: portainer/portainer<EMAIL_ADDRESS>Cc: Neil Cresswell<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [portainer/portainer] Run script from Portainer (#2409)
@olljanathttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_olljanat&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=NvaeurP8KxDVPnVJnDfRCw8mtieXRFXN1cdQNPQFX6g&s=MVGEFUsVsXhjy-zA3FToOrMfsDf4eU9BzX6CYtA3cyU&e= not really
#2181https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_2181&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=NvaeurP8KxDVPnVJnDfRCw8mtieXRFXN1cdQNPQFX6g&s=33ziHgtCHGnzpxuN3LW7NXEZIu69990ocfz2mekvuM0&e= gives the ability for a user to execute a job on a Docker host via a privileged container. This allows you to execute commands that are available on the hosts and thus would probably partially solve the OP issue.
Regarding the ability to run pre-configured scripts for standard users, this is something that we'll have to discuss.
cc @ncresswellhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_ncresswell&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=NvaeurP8KxDVPnVJnDfRCw8mtieXRFXN1cdQNPQFX6g&s=R1aF_us8zgyJ3Zu9fAoyhlagqWLK0Ybg_U-Ek0UFxLc&e=
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_2409-23issuecomment-2D434836780&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=NvaeurP8KxDVPnVJnDfRCw8mtieXRFXN1cdQNPQFX6g&s=4gfyA75L2qS6xdJlhL65SwpsrnogUCOlMbk2InlFyA8&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlcgknXGCPnXZHWZ-5FoaZGmKbtPvpiks5uqgh0gaJpZM4X9-2Dik&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=NvaeurP8KxDVPnVJnDfRCw8mtieXRFXN1cdQNPQFX6g&s=QCJz68xIF0JnjL07TqOHjlJ7HJFmKvY0KPSFCOwE5RI&e=.


_standardize_user_data may return None for sample_weights

The tensorflow branch the keras engine was modified such that model._standardize_user_data will now return None instead of an empty list when sample_weights are not defined.
Although the code states:
    Returns:
      A tuple of 3: inputs (arrays or dicts, depending on whether `x` was a dict
      or not), target arrays, sample-weight arrays.
      If the model's input and targets are symbolic, these lists are empty
      (since the model takes no user-provided data, instead the data comes
      from the symbolic inputs/targets).

The following commit seems to have changed the default sample weights return from [] to None.
https://github.com/tensorflow/tensorflow/commit/41abdd1212777a60e0d97ada4a39d13f2b9014a1
This in turn causes the list add operation in the following return statement to fail:
def _evaluate(model: Model, nodes_to_evaluate, x, y=None):
   [...]
    x_, y_, sample_weight_ = model._standardize_user_data(x, y)
    return f(x_ + y_ + sample_weight_)

Thanks for reporting @pedro-r-marques do you have any ideas to fix that?

The following check after the call to _standardize_user_data fixes it for me.
    x_, y_, sample_weight_ = model._standardize_user_data(x, y)
    if sample_weight_ is None:
      sample_weight_ = []

Apologies but I've not had time recently to submit a pull request.

@pedro-r-marques no problem. What you're doing already helps a lot :)

Fixed in https://github.com/philipperemy/keract/pull/65.
I used a try catch but your solution also works.


Add COLLATE to indexes

https://www.sqlite.org/lang_createindex.html

@dankochetov could this also be expanded to include COLLATE on other areas, like column definitions in CREATE TABLE?


Consider adding a lossy deserialization method

The idea is from matthieum on reddit. It might be possible to add a Deserialize::deserialize_lossy that opts in at a call-site to ignore missing fields. Maybe suppress other errors too.

Nobody answered on reddit, so I'll ask this question here: will serde on decode set "none" to the field of structure, if type if this field is Option and field does not exist in input string?

@e-oz No, Serde the framework does not deserialize missing Option fields to None. That sort of thing is up to the individual Serde formats to handle. The Serde format for YAML does have this behavior - see the relevant code here. Same thing in the JSON deserializer here.

@dtolnay so deserialization will just fail if partial structure will be send to input? I can't believe it, it's too brittle to survive. I'm not trying to say you are lying, of course, I just really can't believe it. But if it's so, then you are saving tone of my time (I just will not even try to switch to serde).

Without this behaviour you would not be able to specify a format that's supposed to error on missing fields. Note that it's trivial to enable this behaviour in your own format.

In other words, the decision of whether to fail on a "partial structure" depends on the format that is being deserialized. Concretely, for those formats that serialize structs as maps (JSON, YAML, some others), the behavior is controlled by this missing_field method, which may return either an error or a default value for the missing field. Both JSON and YAML Serde formats return a default of None when the type of the missing field is Option.
There are other formats in which this is not possible. In particular, some formats are not self-describing so we can't even tell whether a field is missing. Serde the framework attempts to be general across all of these different formats.

@oli-obk easy, just make them non-Optional.
@dtolnay

Both JSON and YAML Serde formats return a default of None when the type of the missing field is Option.

In your first response I thought you are saying exactly opposite thing. Sorry for misunderstanding, glad to hear it works this way.

I deleted an irrelevant discussion.
@e-oz

Nobody answered on reddit, so I'll ask this question here: will serde on decode set "none" to the field of structure, if type if this field is Option and field does not exist in input string?

@dtolnay

@e-oz No, Serde the framework does not deserialize missing Option fields to None. That sort of thing is up to the individual Serde formats to handle. The Serde format for YAML does deserialize missing Option fields to None - see the relevant code here. Same thing in the JSON deserializer here.

@e-oz

@dtolnay so deserialization will just fail if partial structure will be send to input? I can't believe it, it's too brittle to survive. I'm not trying to say you are lying, of course, I just really can't believe it. But if it's so, then you are saving tone of my time (I just will not even try to switch to serde).

@oli-obk

Without this behaviour you would not be able to specify a format that's supposed to error on missing fields. Note that it's trivial to enable this behaviour in your own format.

@dtolnay

In other words, the decision of whether to fail on a "partial structure" depends on the format that is being deserialized. Concretely, for those formats that serialize structs as maps (JSON, YAML, some others) the behavior is controlled by this missing_field method which may return either an error or a default value for the missing field. Both JSON and YAML Serde formats return a default of None when the type of the missing field is Option.
There are other formats in which this is not possible. In particular, some formats are not self-describing so we can't even tell whether a field is missing. Serde the framework attempts to be general across all of these different formats.

@e-oz

@oli-obk easy, just make them non-Optional.
@dtolnay

Both JSON and YAML Serde formats return a default of None when the type of the missing field is Option.

In your first response I thought you are saying exactly opposite thing. Sorry for misunderstanding, glad to hear it works this way.

Does anyone have real examples where they need to deserialize the same struct with and without ignoring unknown field / missing field errors?
Also are there any libraries in other languages that handle this that we could use for inspiration?


Line2D/Polygon2D rotates wrong direction using vertex shader with batching enabled

Godot version:

3.x d08666f999f3912e2528994a7dad42eada236b5f
OS/device including version:

Windows 10
GLES3
Issue description:

This appears to be the same issue as https://github.com/godotengine/godot/issues/46895
Adding the following shader to a Line2D or Polygon2D makes them rotate the wrong way:
shader_type canvas_item;
render_mode blend_mix;

void vertex() {
    VERTEX;
}

With batching OFF:

With batching ON:

Steps to reproduce:
See above.
Minimal reproduction project:

BatchingRotationReprod.zip

cc @lawnjelly

FYI I'm working on the fix right now.

Fixed by #48457.


VCPP: Change instr encoding: cmd bits in LSB:s

By moving the command bits (4 MSB) and some other bits (e.g. reg number in SETREG) to the least significant bits, many instructions can be created with one ldi MRISC32 instruction instead of two (ldhi + or).

Never mind...


some annotations miss

a good job, but some annos miss,  i have download the version 9 from the kaggle
for example, the image id<PHONE_NUMBER>11 miss the person

you can print the info from json for check
{'category_id': 199, 'iscrowd': 0, 'isthing': 0, 'image_id': 272411, 'area': 47638, 'bbox': [0.0, 6.0, 444.0, 222.0]}
{'category_id': 138, 'iscrowd': 0, 'isthing': 0, 'image_id': 272411, 'area': 44094, 'bbox': [0.0, 508.0, 444.0, 132.0]}
{'category_id': 185, 'iscrowd': 0, 'isthing': 0, 'image_id': 272411, 'area': 8111, 'bbox': [22.0, 253.0, 422.0, 289.0]}
{'category_id': 43, 'isthing': 1, 'iscrowd': 0, 'image_id': 272411, 'area': 11328, 'bbox': [240.0, 387.0, 164.0, 159.0]}

a good job, but some annos miss, i have download the version 9 from the kaggle for example, the image id<PHONE_NUMBER>11 miss the person  you can print the info from json for check {'category_id': 199, 'iscrowd': 0, 'isthing': 0, 'image_id': 272411, 'area': 47638, 'bbox': [0.0, 6.0, 444.0, 222.0]} {'category_id': 138, 'iscrowd': 0, 'isthing': 0, 'image_id': 272411, 'area': 44094, 'bbox': [0.0, 508.0, 444.0, 132.0]} {'category_id': 185, 'iscrowd': 0, 'isthing': 0, 'image_id': 272411, 'area': 8111, 'bbox': [22.0, 253.0, 422.0, 289.0]} {'category_id': 43, 'isthing': 1, 'iscrowd': 0, 'image_id': 272411, 'area': 11328, 'bbox': [240.0, 387.0, 164.0, 159.0]}

thanks for catching this problem, such noise of annotation do exist. If you go to check our paper, our expert rater will sample 50% from the annotations queues to check if all the masks meet expectation. But our raters should do 100% final check on COCO-val set (5000 images). If this id is in val set, could you please provide the image id, so that we could fix it really quick.

thanks ...


Intermittent AutoCompactTest.ReadHalf failures

We're observing intermittent AutoCompactTest.ReadHalf failures. The failures occur with the latest (1.22) and prior (v1.18) releases, and can be observed on multiple architectures (x86-64 + aarch64), kernels and filesystems. E.g.
[  239s] ==== Test AutoCompactTest.ReadAll
[  239s] iter   1 => 100.040 MB [other   0.000 MB]
[  239s] iter   2 => 100.040 MB [other   0.000 MB]
[  239s] iter   3 => 100.040 MB [other   0.000 MB]
[  239s] iter   4 => 100.040 MB [other   0.000 MB]
[  239s] iter   5 => 100.040 MB [other   0.000 MB]
[  239s] iter   6 =>  79.524 MB [other   0.000 MB]
[  239s] iter   7 =>  59.008 MB [other   0.000 MB]
[  239s] iter   8 =>  38.492 MB [other   0.000 MB]
[  239s] iter   9 =>   4.103 MB [other   0.000 MB]
[  239s] ==== Test AutoCompactTest.ReadHalf
[  239s] iter   1 =>  50.017 MB [other  50.023 MB]
[  239s] iter   2 =>  50.020 MB [other  50.021 MB]
[  239s] iter   3 =>  50.020 MB [other  50.021 MB]
[  239s] iter   4 =>  50.020 MB [other  50.021 MB]
[  239s] iter   5 =>  50.020 MB [other  50.021 MB]
[  239s] iter   6 =>  50.020 MB [other  50.021 MB]
[  239s] iter   7 =>  29.504 MB [other  29.505 MB]
[  239s] iter   8 =>   8.988 MB [other  29.505 MB]
[  239s] iter   9 =>   4.103 MB [other   0.000 MB]
[  239s] /home/abuild/rpmbuild/BUILD/leveldb-1.22/db/autocompact_test.cc:103: failed: 0 >= 9442061

 99   // Verify that the size of the key space not touched by the reads
100   // is pretty much unchanged.
101   const int64_t final_other_size = Size(Key(n), Key(kCount));
102   ASSERT_LE(final_other_size, initial_other_size + 1048576);
103   ASSERT_GE(final_other_size, initial_other_size / 5 - 1048576);

The AutoCompactTest.ReadHalf test fills a DB with 512 keys (200KB per record) and then subsequently deletes them. Following deletion, iteration of half of the key space is used to trigger compaction via:
  DBIter::FindNextUserEntry
        -> DBIter::ParseKey
                -> DBImpl::RecordReadSample
                        -> DBImpl::MaybeScheduleCompaction

The background worker thread may concurrently (re)trigger compaction via:
  DBImpl::BGWork
        -> DBImpl::BackgroundCall
                -> DBImpl::MaybeScheduleCompaction

Following each DoReads() iteration loop, the estimated size of each half of the key space is obtained via:
  DBImpl::GetApproximateSizes
        -> VersionSet::ApproximateOffsetOf
                -> Table::ApproximateOffsetOf

The approximate offset is determined by walking each file at each level
until the key space boundary is found.
The failure is due to the estimated size of the non-iterated key space shrinking more than expected:
  // Verify that the size of the key space not touched by the reads
  // is pretty much unchanged.
...
  ASSERT_GE(final_other_size, initial_other_size / 5 - 1048576);

Given that all keys are deleted, I don't think this test should make any assertions on how compaction affects the non-iterated key space (aside from it being >= 0), but I'm probably missing something here.
I don't see how the iterated key space has any affect on which SST ranges are compacted. Manual compaction could do this, but it shouldn't be triggered in this case.

Yes, I got this failure too.


Fix minor mistakes

Pull Request Test Coverage Report for Build b0679d58e2ee81e9514af7b386c6fad165748cac-PR-859

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.937%

Totals

Change from base Build da059e05ff0b15867785ab77b93503d576808367:
0.0%

Covered Lines:
992

Relevant Lines:
1439

💛  - Coveralls


Logging is blocked when pytuya is imported

Pytuya is a part of my Snips Homeautomation.
I tried to implement logging for several times now and today I found out that the pytuya module somehow blocks logging.
Here is a simple example to reproduce the problem:
import pytuya
import logging

logging.basicConfig(
        level=logging.INFO,
        filename='logs.log',
        filemode='w'
)

logger = logging.getLogger('Test')
logger.addHandler(logging.StreamHandler())
logger.info('Test')

is not working.
#import pytuya
import logging

logging.basicConfig(
        level=logging.INFO,
        filename='logs.log',
        filemode='w'
)

logger = logging.getLogger('Test')
logger.addHandler(logging.StreamHandler())
logger.info('Test')

by commenting the pytuya module the logger is working again...
Is there a simple way to fix that?
Thank you!

@LucidSkyWalker I think the issue here is that both the library and your calling app both used basicconfig, and pytuya called it first. One workaround would be to import pytuya AFTER you setup logging.
Thanks for reporting this, I think the correct solution is for pytuya to not to setup any config (and then for callers to choose). I'll get that done.


演示站点证书过期

Laravel Version: #.#.#
PHP Version:
Laravel-admin: #.#.#

Description:
有效时间：2022-03-07 至 2022-06-05
Steps To Reproduce:

https://laravel-admin.org/docs/zh/
文档站点的证书也过期了

This package may have been abandoned

updated.

@jxlwqq  Thanks Bro,
Please don't give up on this project, because it's so good  :)


knitr engine adjusments for xml2 behaviour change

also removed shiny widget as it's superseded by open_workspace()

Codecov Report

Merging #15 (1c7cab8) into main (7397d0b) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
- Coverage   11.08%   11.08%   -0.01%
==========================================
  Files          22       22
  Lines        2670     2671       +1
==========================================
  Hits          296      296
- Misses       2374     2375       +1

Impacted Files
Coverage Δ

R/knitr-engines.R
0.00% <0.00%> (ø)

R/request-helpers.R
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Decision on adopting type hints

What would you like changed/added and why?
Some parts of the coder use type hints, whereas others do not. It would be beneficial to make a decision and to keep the code consistent.
Do you have any interest in helping improve the documentation?
Yes
Do you have any suggestions for the new documents?
Adopting type hints is probably good to be able to detect errors earlier.

I'm :+1: and try to incorporate type annotations when I have the time or need to better interpret the code with the hopes that one day all the code will have type hints.

An unchecked type hint is just a note to your future self, but it also triggers additional checks by the type checker. Adding hints without checking them just makes the task of adopting a type checker more involved.
I would suggest adopting a type checker first, add type checks to your CI (and pre-commit, if it's fast enough), and then begin annotating your project.

Adding hints without checking them just makes the task of adopting a type checker more involved.

Didn't know that it would make the adoption of a checker more involved.

I would suggest adopting a type checker first, add type checks to your CI (and pre-commit, if it's fast enough), and then begin annotating your project.

Looks a reasonable plan. @effigies please feel free to downvote the adoption of type hints (and checkers) if you do not see any benefit to them; then it is more a decision about consistency in the code.


User-defined &mut-returning functions

Hax doesn't support any kind of &mut-returning functions, except a fixed list of functions from core with special treatment.
Supporting &mut-returning functions would require the use of Aeneas's backward functions.

@W95Psp this should probably go on the triage/priority list.

We need to have a concrete plan for this before deciding exactly how broadly we want to support that


Make allow_blank false when the type is defined

Hey, we're working on a project and we have a lot of required parameters with their type defined but allow_blank is true by default so we should make allow_blank false for each one and that's bad (DRY). is there any option we can set to make it false when at least the type is defined or just for the requires method?

+1

I would take a pull request that configures global defaults for various validators.

@azbshiri @intuxicated Could you confirm this pull request?

I really appreciate your great work. I think it does the job 💯

Thanks for confirming the issue. Closed.


GatewayAPI topology (DAG)

https://github.com/Kuadrant/kuadrant-operator/pull/447 introduced the GatewayAPI topology (DAG) implementation
This (epic) issue tracks the usage of GatewayAPI topology (DAG) implementation by the different kuadrant controllers

[X]  Rate limiting wasmplugin controller #447
[ ]  Rate limiting limits controller #527
[ ] AuthPolicy controller
[ ] DNS policy controller
[ ] Gateway Kuadrant controller
[ ] Kuadrant controller
[ ] Limitador Cluster Envoy controller  #527
[ ] RateLimitPolicy controller
[ ] TLS Policy controller

The GatewayAPI Topology
The controller builds what it is named "The GatewayAPI Topology". It reads ALL Gateways, HTTPRoutes and policies (only those the controller cares about) in the cluster. With all that info, the controller creates a new instance of the Gateway API topology as well as a set of indexes (another "view" of the topology) that can be queried. Interesting to note here that field indexers can optionally be added to the manager to build internal index caches. One example being implemented is an index to query "give me all the HTTPRoutes parenting Gateway A".
The current implementation of the "GatewayAPI Topology" does not use a k8s client to query Gateway API and kuadrant policies. Instead, fetching resources must be done externally and then the GatewayAPI topology receives a list of resources. It could be done otherwise and have generic function with a k8s client to build topology for a given policy of one kind. The main reason for the current implementation is to decouple GatewayAPI topology and the cluster. Meaning that I can build GatewayAPI topology from a set of resources filtered with some criteria.
The implementation assumes "cheap" operations of reading ALL the gateways, routes and policies. That assumption comes from the controller runtime k8s client that holds a cache. At least for list and get operations. That cache is being kept up to date  in the background when there are updates/deletes in the objects triggered by kuadrant controllers or any other third party controllers or any other k8s API server user. Check for this comment with a verification of this "cheap" operation assumption. Indeed, the resource lists are cached across reconciliation events and controllers.
The topology implementation is based on a pure  Directed Acyclic Graph data structure implementation that is totally decoupled from the GatewayAPI relations. On top of that, there is an implementation of the GatewayAPI topology which builds a graph from Gateways and HTTPRoutes. In the future, it can be extended for additional GatewayAPI API types. On top of the GatewayAPI topology, a number of indexes are created which are tailored for the needs of the kuadrant controllers. Thus:
Kuadrant controller
       |
       |
       V
Custom topology indexes (views)
       |
       |
       V
GatewayAPI topology (focused on one gateway)
       |
       |
       V
     DAG

I like the direction this is going really. In fact, I now wonder if the approach adopted for the WasmPlugin (#447) and (ongoing) for the Limitador CR (#527) shouldn't be a model for all internal resources managed by the Kuadrant Operator, and not resources that map 1:1 to the number of gateway objects only.
What I mean by that is no longer having any controller that reconciles a policy. Instead, policies would be treated as just another source of events that triggers reconciliation through mapping. All events would map to the affected gateways, and therefore the input to the reconciliation function becomes the gateway object always.
The kinds of resources related to policies to actually reconcile are:

WasmPlugin
EnvoyFilter
Limitador
AuthConfig
AuthorizationPolicy
DNSRecord
Certificate

The reconciliation of each of these resources can be independent event from each other, by being handled by dedicated reconcilers. Within each reconciliation process, we could leverage concurrency in Go to compute the desired states of resources known upfront to be truly independent from each other (if there is any, given by the number of branches down from a gateway node until that kind of resource.)
In the future, if we decide to grow the hierarchy upwards and, say, turn the GatewayClass the new root (thus replacing the Gateway), it should be easy to adapt the mappers and wrap every reconciliation function within another loop. Arguably, this could be done in a generic way from day 1.
I realise this approach increases the computation load, compared to the current attempt to isolate the blast by focusing on each individual policy object. On the other hand, I believe it's more consistent with the expectation of a "state of the world" kind of reconciliation. Plus, computing a bigger desire state does not mean necessarily more requests to the API server.
The bigger challenge IMO is status updating, but I believe even on this one we can win with this approach. The status of the reconciled objects (when available) is straightforward as it can be set at each reconciliation function. The status of the policy objects, on the hand, depends on gathering the final states of all reconciliation functions to only then to be set. If every step (i.e. every internal resource kind, every reconciliation function) has a corresponding status condition added to the policy, the final one should be a no brainer. Finally, the state the gateway objects and routes can wait for the policy and/or be partially and incrementally set by the reconciliation functions.
At the level of the Kuadrant instance, the kinds of resource to reconcile are:

IstioOperator
(Istio) ConfigMap
ServiceMeshControlPlane

I'd like to propose modifying the list of controllers in the description to the list of resource kinds above. I.e., no more "AuthPolicy controller" or "RateLimitPolicy controller", but "AuthConfigReconciler", "AuthorizationPolicyReconciler", "WasmPluginReconciler", "LimitadorReconciler", etc.

The controller reconciliation function input cannot be always Gateways (or gateway classes). One example is Limitador's limits controller. There is one limitador object to be reconciled per kuadrant instance (currently only one allowed per cluster). So for this particular case, the controller reconciliation function input is the kuadrant CR.

This makes sense and it's a more balanced view than mine from before actually. Depending on the kind of object, we should pick the correct root of the tree from where to reconcile. And I guess the "correct" one is the first common node that would require no jumping to sibling nodes to compute the desired state – as you exemplified for the Limitador CR, whose root cannot be the Gateway.
IOW, if you can create a Go routine that reduces the problem to the size of the subtree rooted by the object passed as input to the reconciliation function, then you're good. But you shouldn't have to query anything outside this subtree.
I could argue that today the Kuadrant CR is the tip and true root of the entire tree.

The main motivation of moving logic from "policy controller" to specific resource controller is that each controller can define their own mappers and predicates to filter events. This is a powerful capability that totally affects the controller's logic. Additionally, the reconciliation of some elements are not coupled with others. Responsibility is decoupled by design. But that also comes with a price. And that brings me to "policy controllers". They still need to exist, as far as I can see now. The policy controllers main responsibility would be basically maintain updated status. Plus other features centered on the policy itself. For instance the constraint of network resources being targeted by a single kuadrant policy (per type). That functionaliy is owned, IMO, to the "policy controller".

Here I see almost no conflict with what I said actually. What you call "policy controller" is what I implied as needed for the status reconciliation. I would call those "xPolicyStatusReconciler" and stick with the pattern of a reconciler is named after the resource kind it reconciles, not the resource kind it reconciles from.
As for the validation, I argue it does not belong to this controller. If you deem a policy to be invalid at this stage, it's already too late as the other reconcilers may have already created resources as if the policy were valid. All validation should be performed at the API level IMO, which is not the case today, I know.


Unable to run docker example

I'm trying to run the docker example as mentioned in README_DOCKER.md. However, I'm running into the following error when I run the test script:
docker run -it --rm --name=cam1 --link=voctocore:corehost local/voctomix gstreamer/source-videotestsrc-as-cam1.sh
error:
Failed to connect to host 'localhost:10000': Connection refused
I have been trying to resolve it for the past week but have had no success. Would really appreciate the help, thanks in advance.

Given the date that documentation was last touched, i suspect the documentation was written for voctomix1.
I believe noone has ever tested docker with voctomix 2.x before.
I will remove the docker readme and the Dockerfile from the repository. If you want to provide us with a working Dockerfile and docker readme, i'd be very happy to merge a pull request for that.


bugfix: reduce error logs when download home dir not exist

…me dir not exist
Signed-off-by: Starnop<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
At present, gc disk will produce error logs continuously when download home dir not exist. And in this case, there is no need to do the gc job. So just return  directly in this PR.
Ⅱ. Does this pull request fix one issue?

NONE.
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #960 into master will decrease coverage by 0.04%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master     #960      +/-   ##
==========================================
- Coverage   46.46%   46.42%   -0.05%
==========================================
  Files         116      116
  Lines        6910     6910
==========================================
- Hits         3211     3208       -3
- Misses       3441     3445       +4
+ Partials      258      257       -1

Impacted Files
Coverage Δ

supernode/daemon/mgr/cdn/cdn_gc.go
0% <0%> (ø)
:arrow_up:

supernode/daemon/mgr/cdn/cache_detector.go
7.24% <0%> (ø)
:arrow_up:

supernode/store/local_storage.go
62.94% <100%> (ø)
:arrow_up:

cmd/dfdaemon/app/root.go
63.29% <0%> (-3.38%)
:arrow_down:

supernode/daemon/mgr/scheduler/manager.go
22.6% <0%> (+0.68%)
:arrow_up:

Continue to review full report at Codecov.

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


Branching Strategy Implementation

Is your feature request related to a problem? Please describe
Coming from the initial [Proposal] Branching strategy for build and the solution proposed as part of this proposal would be the closest implementation for the build repo.
To summarize the solution:

The main branch is used for active development and targets the upcoming release version.
A <MAJOR_VERSION>.x branch is cut for every major release version, example as 2.x, 3.x. This is inclined with the Core branching and with planned release cycles.  Considering the next major release is 3.0.0, the main would have active development, new features and everything compatible for 3.x and above. A 2.x branch should have everything related to 2.x series. Now when the next upcoming release is 4.0.0, main would shift everything compatible for 4.x and above and 3.x branch will be added to the repo.
During distribution jobs execution, for 2.2.1 release, the pipelines/jobs would use an input BRANCH_NAME and use the 2.x and reference.

NOTE: The on going activity to separate out the Jenkins shared libraries  would indirectly influence this branching strategy. The jenkinsfiles should be modified to fetch the shared libs from external repo, example https://github.com/prudhvigodithi/opensearch-build-pipelines/blob/main/opensearch/distribution-build.jenkinsfile#L1-L4. With this, the jenkins shared libraries and its associated tests need not be part of this branching implementation.
There is other Open issue related to this topic to server the entire build repo as framework with right versioning, which can be later fetch during runtime using the right version.

[ ] https://github.com/opensearch-project/opensearch-build/issues/2490

Describe the solution you'd like

[ ] The Jenkinsfiles cannot be branched, hence the parking place for all the jenkinsfiles would be the main branch. The public Jenkins jobs will use Jenkinsfiles fetching from main branch.
[ ] Add an input BUILD_BRANCH to the Jenkinsfiles so that jobs can clone the build repo with right BUILD_BRANCH used for the release.
[ ] Use gitScm within the Jenkinsfiles to clone to build repo BUILD_BRANCH to run the distribution workflows, so that the code is available to running workspace. Once the repo is branched the distribution workflows also are part of each branch, so when jenkins clones the right BUILD_BRANCH the job workspace will have all the workflows part of that specific BUILD_BRANCH.
https://github.com/prudhvigodithi/opensearch-build-pipelines/blob/main/opensearch/distribution-build.jenkinsfile#L78
[ ] Modify/Ensure Jenkins job configuration points to the main branch of the build repo, where the jenkinsfiles are hosted.
[ ] Create new branch 2.x and 3.x from main ,  main will be having the active development and 2.x, 3.x will have frequent backports. https://github.com/prudhvigodithi/opensearch-build/tree/2.x.
NOTE: At this point the <MAJOR_VERSION>.x branch is expected to cut manually from main until an automation is derived.
[ ] Configure the backport workflows to the build repo.
[ ] In check for build Jenkins job, add the respective BRANCH_NAME to make sure the distribution jobs uses the right branch in build repo. Modify the workflow that raises an auto PR https://github.com/opensearch-project/opensearch-build/pull/2544 to update the check for build Jenkins job with BRANCH_NAME as well.
[ ] Manifest workflow (that adds the manifests for every version) will always raises a PR to main, need to curate and backport the right manifests to the right <MAJOR_VERSION>.x branch.
[ ] Update the README file with the branching solution incorporated.

Describe alternatives you've considered
No response
Additional context
No response

This branching strategy seems to assume that major version numbers are in sync. This is not the case.
The build repo builds many products that have significantly diverging build numbers, starting with OpenSearch and OpenSearch Dashboards that are going to be increasingly diverging as independent products. We follow semver, so there's no need to increment a major version number of OpenSearch unless there's a breaking change, and we avoid making breaking changes. Thus, within a year I suspect we'll be releasing OpenSearch 3.0 and OpenSearch Dashboards 5.0 at the same time.

Hey @dblock I would like to move forward with branching based on existing setup where we publish OpenSearch, OpenSearch Dashboards together for every release. I'm not sure if we have a target date to separate out OpenSearch and OpenSearch Dashboards into a separate release channels. However if that is the long term goal, we should be going with OpenSearch Build repo own way of versioning, example the branch still be as 1.x, but this can support the upcoming releases like may be OS 3.3 and OSD 5.1 and we should be add a compatibility matrix for this.  IF there is any breaking change for the build repo, we can go with 2.x which can support may be next OS 4.0 and OSD 6.0, in this scenario main would be still for active development, wdyt @dblock @bbarani any suggestions?

Hey @dblock thanks for bringing this up,  just to add we can start to cut the branch <MAJOR_VERSION>.x following the core, once the OS and OSD are diverged, we can keep the track using a compatibility matrix.

@prudhvigodithi Branching the build repo in its own version and following semver makes sense to me. I would start with a versioning scheme that clearly distinguishes it from OpenSearch or OpenSearch Dashboards (e.g. 0.1.0).

@dblock just a thought, this would end up having lot of semVer branches created to manage both OpenSearch and OpenSearch Dashboards, also still main would be having both OpenSearch and OpenSearch Dashboards development included that needs to be backported to OpenSearch and OpenSearch Dashboard release semVer branches.

Yes, this plan is complicated. That's why we never branched this repo in the first place. But you seem to want to do it at all costs.

Keeping this implementation on hold until a better approach is taken to address the scenario (future implementation), where the build repo handles multiple products and each of its version is offset with other. With this a single branch as <MAJOR_VERSION>.x cannot be the source of truth for a product which has an ongoing release <MAJOR_VERSION>+2.x
Example:
Lets say the build repo handles 2 products OpenSearch and  OpenSearch Dashboards, OpenSearch 2.3.4 is about to get released and at the same time OpenSearch Dashboards 4.1.1 is bing released, then working on 2.x or 4.x branch does not solve the branching requirement and it would end up creating lot of branches per product as os-2.x, osd-4.x, etc
This implementation is more friendly to the existing approach (as of today September 12th 2022) where the build repo builds OpenSearch and  OpenSearch Dashboards with same release version and are shipped together (Sample 2.3.0 release).
Related PR's and issues that are on hold:
https://github.com/opensearch-project/opensearch-build/pull/2559
https://github.com/opensearch-project/opensearch-build/issues/2558
As mentioned above the build repo can be branched as <PRODUCT>-<MAJOR_VERSION>.x, but with this keeping main as source of truth for all products does not solve the proper branching strategy as the <PRODUCT>-<MAJOR_VERSION>.x is cut from main, which means again every <PRODUCT>-<MAJOR_VERSION>.x has all product build code.
@bbarani @dblock

Needing branches such as product-2.x should be telling us that we’re thinking about it wrong. The goal of this repo was to map 1:1 to what’s in Jenkins for all products and versions. That’s why we built backwards compatible manifest support. If a tool needs to be different across versions, it doesn’t belong in this repo but in its own. If a script needs to be different across versions, either both must be checked into this repo under different folders (like multiple manifests are) or live in the product repo that is branched already.

@prudhvigodithi Can we close this issue for now and revisit later when we have sorted out the gaps?

Hey @bbarani yes I'm ok to close this and re-open a new one that address the points we discussed above. I'm going to close this as not planned.


Not Working In Live Server And Github-Page

jekyll-paginate-v2 plugin working properly in local server but not working in live server and github pages.

This gem is unfortunately not supported yet by github.com


Add deprecation warnings to magphase and ComplexNorm

@anjali411 I noticed that there are missing deprecation warnings. The style of message is same as the other ones in #1445.

thanks!


Use a loop to ping Guacamole

Fixes #1425
What is being addressed
Guacamole ping depends on Azure deployment and App Service startup times which can vary. So a strategy of waiting X number of seconds doesn't always work.
How is this addressed

Convert to a loop that tests if the endpoint is ready

Will bypass the PR deployment tests since the change is at the E2E ones which are only run after merge to main. A run with the change can be viewed here: https://github.com/microsoft/AzureTRE/runs/5440389315

/test-force-approve


Support for private bitbucket repo.

Hi,
Is it possible to use private instance of RTD with private bitbucket repo?
Also is there some option to use RTD without bitbucket or github repo ?
Thanks,
Makailol

+1

We aren't looking to support private repositories with readthedocs.org, we only offer hosting for open source projects. Github and Bitbucket private repository support is built into readthedocs.com however.
There isn't a way currently to use RTD without a VCS backing, but we've thought about bringing the feature back. This would likely be a manual zip/tarball upload.

Would you help me with this statement: "Bitbucket private repository support is built into readthedocs.com however."
We have a bitbucket repository. When it was in private status, RTD would not accept a repository URL in this format<EMAIL_ADDRESS>When we switched to a public repository and used this URL: https://bitbucket.org/name/sphinx_1.git the build was successful. Is there a method where we can build an RTD instance from a bitbucket access level is set to "This is a private repository"?

We don't support private repositories on readthedocs.org. We believe the value of RTD comes from projects being open source. You're welcome to host anything for free, as long as its liberally licensed.
Readthedocs.com is a separate product, focusing on companies that require private hosting. It has support for private repositories on github and bitbucket.

Got it - missed the .com part of readthedocs. Glad to see the .com service is up and running - we'll use it.

For anyone who may come across this ticket, I was able to solve this problem by creating a public github repo dedicated to hosting the docs of my project while keeping the repo containing the proprietary code private. It's not ideal but considering RTD.org does not plan to support private repos and RTD.com charges very large fees it seems to be a good enough solution.

Readthedocs.com does seem very expensive.  We would definitely use for our small-scale but commercial project if it were 1/5 of the price.  For now we will be forced to use @KylePiira 's approach.

For anyone considering working around our restriction of private repositories to host your commercial documentation, please consider at least donating money through a gold subscription for you project. It's entirely unfair to offload your commercial hosting on to volunteer labor without supporting our efforts in some fashion.
We don't offer a small paid plan on our commercial hosting as we tend to offer a lot of custom guidance to customers. If you're okay with a plan with no email support or a smaller amount of pageviews, feel free to contact us at<EMAIL_ADDRESS>and we can perhaps put together a small plan to get you started.


Several "Priority one" website fixes

Make the / URL perform a timed redirect to /2020/
Fix text of the agenda related to it being not finished
The iCal link is now a button in /agenda/
There are no longer multiple h1 tags in a single page anymore (I think)
There is an additional page on "how to join" to explain how to use BBB (which is empty at the moment; it's expected to be contributed to as other PRs)

Nice work!