File input no longer posting

From @MihaiBratulescu on July 4, 2016 7:10
I have a few of these file inputs:
<input<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>hidden />

<input<EMAIL_ADDRESS><EMAIL_ADDRESS>                                   type="file" multiple style="display:inline-block">

The keys get posted but not the files. In Request.Form.Keys the Value keys are missing for the file inputs but if I change to text it's working all the sudden.
This has been happening since I upgraded to RTM and worked fine in RC2.
My project.json :
{
  "userSecretsId": "aspnet-WNCT.Web-f9ad84ae-9b95-4000-ab80-3ecdf4814421",

  "dependencies": {
    "Microsoft.AspNetCore.Session": "1.0.0",
    "Microsoft.AspNetCore.Hosting": "1.0.0",
    "Microsoft.AspNetCore.Identity.EntityFrameworkCore": "1.0.0",
    "Microsoft.AspNetCore.Mvc": "1.0.0",
    "Microsoft.AspNetCore.Razor.Tools": "1.0.0-preview2-final",
    "Microsoft.AspNetCore.Server.IISIntegration": "1.0.0",
    "Microsoft.AspNetCore.Server.Kestrel": "1.0.0",
    "Microsoft.AspNetCore.StaticFiles": "1.0.0",
    "Microsoft.AspNetCore.Diagnostics": "1.0.0",
    "Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore": "1.0.0",
    "Microsoft.EntityFrameworkCore.SqlServer": "1.0.0",
    "Microsoft.Extensions.Configuration.EnvironmentVariables": "1.0.0",
    "Microsoft.Extensions.Configuration.Json": "1.0.0",
    "Microsoft.Extensions.Configuration.UserSecrets": "1.0.0",
    "Microsoft.Extensions.Logging": "1.0.0",
    "Microsoft.Extensions.Logging.Console": "1.0.0",
    "Microsoft.Extensions.Logging.Debug": "1.0.0",
    "Serilog": "2.0.0-rc-602",
    "Serilog.Sinks.RollingFile": "2.0.0-rc-703",
    "Serilog.Extensions.Logging": "1.0.0-rc2-10110",
    "HtmlAgilityPack": "1.4.9",
    "LDAP": "1.0.0",
    "WNCT.BL": "1.0.0-*",
    "WNCT.Domain": "1.0.0-*",
    "WNCT.EF": "1.0.0-*"
  },

  "tools": {
    "Microsoft.AspNetCore.Razor.Tools": "1.0.0-preview2-final",
    "Microsoft.AspNetCore.Server.IISIntegration.Tools": "1.0.0-preview2-final"
  },

  "frameworks": {
    "net451": {}
  },

  "buildOptions": {
    "platform": "x86",
    "emitEntryPoint": true,
    "preserveCompilationContext": true
  },

  //"runtimes": { "win": [ "win7-x86" ] },

  "publishOptions": {
    "include": [
      "wwwroot",
      "Views",
      "appsettings.json",
      "web.config"
    ]
  },

  "scripts": {
    "prepublish": [ "npm install", "gulp clean", "gulp min" ],
    "postpublish": [ "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%" ]
  }
}

Is there a solution out there?
Copied from original issue: aspnet/Home#1631

From @MihaiBratulescu on July 5, 2016 16:55
@Tratcher Yes, I now see it's posting but not binding to the dictionary (IDictionary<string, IEnumerable>) from my model.

From @Tratcher on July 5, 2016 16:32
Is it in Request.Form.Files?

From @MihaiBratulescu on July 5, 2016 17:3
Sorry I swear I wrote something else, my model is:
public class ScopeModel
{
public IDictionary<string, IEnumerable<IFormFile>> ZoneOfAnalysisFiles {get;set;}
....
}

From @Tratcher on July 5, 2016 16:58
@rynowak @kichalla ?

From @MihaiBratulescu on July 6, 2016 13:17
@prilutskiy Yes, like I said it worked in RC2. I have the same problem on a different machine at home.

From @prilutskiy on July 5, 2016 23:14
@MihaiBratulescu, Do you have enctype attribute of the form set to multipart/form-data?

Could you share how your raw request looks like?(ex: browser developer tools) I am not interested in the files being uploaded but the individual parts' headers(because for model binding purpose how the key names are important). So following is the model you are binding to?
public class ScopeModel
{
public IDictionary<string, IEnumerable<IFormFile>> ZoneOfAnalysisFiles {get;set;}
....
}

I am looking at it now.

From @MihaiBratulescu on July 6, 2016 21:27
@kichalla not sure what you ask for but here is a more simple example that also does not work:
Model:
    public class UploadModel
    {
        public IDictionary<string, IEnumerable<IFormFile>> Files { get; set; }
    }

View:
<form method="post"  enctype="multipart/form-data" action="/Home/Upload">

   @for (int i = 0; i < 3; i++)
   {
       <input type="text" hidden<EMAIL_ADDRESS><EMAIL_ADDRESS>value="@(i)File" />
        <input type="file" multiple<EMAIL_ADDRESS><EMAIL_ADDRESS>/>
   }
    <button type="submit">Submit</button>
</form>

Controller:
[HttpGet]
        public IActionResult Upload()
        {
            return View();
        }

        [HttpPost]
        public IActionResult Upload(UploadModel form)
        {
            return View();
        }

As for the headers:
Request URL:http://localhost:54821/Home/Upload
Request Headers
Provisional headers are shown
Content-Type:multipart/form-data; boundary=----WebKitFormBoundaryB2Y9SIARh3WgqkPB
Origin:http://localhost:54821
Referer:http://localhost:54821/Home/Upload
Upgrade-Insecure-Requests:1
User-Agent:Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36
Request Payload
------WebKitFormBoundaryB2Y9SIARh3WgqkPB
Content-Disposition: form-data; name="Files[0].Key"
0File
------WebKitFormBoundaryB2Y9SIARh3WgqkPB
Content-Disposition: form-data; name="Files[0].Value"; filename="somefile.torrent"
Content-Type: application/x-bittorrent
------WebKitFormBoundaryB2Y9SIARh3WgqkPB
Content-Disposition: form-data; name="Files[1].Key"
1File
------WebKitFormBoundaryB2Y9SIARh3WgqkPB
Content-Disposition: form-data; name="Files[1].Value"; filename=""
Content-Type: application/octet-stream
------WebKitFormBoundaryB2Y9SIARh3WgqkPB
Content-Disposition: form-data; name="Files[2].Key"
2File
------WebKitFormBoundaryB2Y9SIARh3WgqkPB
Content-Disposition: form-data; name="Files[2].Value"; filename=""
Content-Type: application/octet-stream
------WebKitFormBoundaryB2Y9SIARh3WgqkPB--

Thanks! for the simple repro. cc @dougbu

ac984173980e3386371af6b4ddf269d8e9db1318


Allow batch launch in the "launch" panel

User story
A user wants to test a hypothesis. So they set a number of different configurations for one or multiple parameters.
And then launch all the corresponding studies.
It would be nice to have a "table mode" in the "Launch" window for that. Something like that

Parameter
Scenario 1
Scenario 2

area1.cluster2.min_power
1100
1200

area1.cluster2.max_power
1200
1400

@flomnes I think what the user should do is create a variant of the base study with the 2nd scenario and launch the 2 studies.
This even allows him to remember what were his hypotheses when looking at the commands list of the variant.
This is what Alexander does for instance.
If this suits you, you can close this issue 🙂


Add knowledge of cooperative matrices

Some optimizations are not aware of cooperative matrices, and either do
nothing or assert. This commits fixes that up.

Shouldn't this PR include new tests for this extension (especially enabled optimzations)?

Not sure what our stance is on trying to include more extensions, but the Nvidia version (SPV_NV_cooperative_matrix) would be equally supportable and might be worthwhile to cover here too.

Shouldn't this PR include new tests for this extension (especially enabled optimzations)?

Done, I had this sitting for a while, and forgot I did not add the tests yet.

Not sure what our stance is on trying to include more extensions, but the Nvidia version (SPV_NV_cooperative_matrix) would be equally supportable and might be worthwhile to cover here too.

Done. I did not add extra testing for NV because it did not make for a meaningfully different tests than the KHR examples.


[keyvault] Remove timeout tests

Packages impacted by this PR

@azure/keyvault-certificates
@azure/keyvault-keys
@azure/keyvault-secrets

Issues associated with this PR

Fix #26601
Fix https://github.com/Azure/azure-sdk-for-js/issues/26229
Fix https://github.com/Azure/azure-sdk-for-js/issues/26644
Fix https://github.com/Azure/azure-sdk-for-js/issues/26157
Fix https://github.com/Azure/azure-sdk-for-js/issues/25853
Fix https://github.com/Azure/azure-sdk-for-js/issues/26182
Fix https://github.com/Azure/azure-sdk-for-js/issues/23997
Fix https://github.com/Azure/azure-sdk-for-js/issues/24755
Fix https://github.com/Azure/azure-sdk-for-js/issues/23580

Describe the problem that is addressed by this PR
See the issue, but basically these tests are super flaky and don't add any value -- they're testing the transport, not Key Vault. Removing them will make the pipelines much greener (and make rerecording much much easier).

API change check
API changes are not detected in this pull request.


[bug] lease vehicle

If player lost leased vehicle, and take lease it again in garage - all cargo is save in vehice. It is possible option dont save cargo in vehicle if player place vehicle in garage?
Thank you.

I have the same problem


Investigate I/O memory leak in "docker stats"

When using docker stats after a while running, it shows an increase in BLOCK I/O memory.
Confirmed for all three components.

Strangely enough, after I changed the host to a better service, the issue stopped.


Package Requested

[X] I would like someone else to build the manifest.
[ ] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[ ] I have checked that the installer for this package is not a standalone or portable executable or a .zip file
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Publisher: https://github.com/Alia5
Publisher Url: https://github.com/Alia5/GlosSI
Package Name: GlosSI
Description:  Tool for using Steam-Input controller rebinding at a system level alongside a global overlay
Package Version: <IP_ADDRESS>
Installer URL: https://github.com/Alia5/GlosSI/releases/download/<IP_ADDRESS>/GlosSI-<IP_ADDRESS>.zip
Current Package Identifier (for new versions):

Blocked by microsoft/winget-cli#140
@soredake - This isn’t the first time I’ve mwntioned that .zip files are not yet supported
https://github.com/microsoft/winget-pkgs/issues/61766#issuecomment-1134994407
I’m asking that you refrain from opening issues requesting packages distributed as .zip files until the client supports them, as creating these issues now is needlessly clogging up the issues queue

I took a look at this now that .zip is supported.
Unfortunately the newest releases use a token which is unique when the download link is clicked and the file will not download without it.  https://glossi.flatspot.pictures/api/download/GlosSI-<IP_ADDRESS>?token=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJqdGkiOiJmODg3NGJmMC04ZDUxLTRjMjYtOGY4ZC01N2M3NmI3YWZkMDIiLCJpYXQiOjE3MDYyMzY1MjMsImV4cCI6MTcwNjIzNjY0M30.r5WNPJrz8iWa3FVYGR19OAYzJAdy1gjbjX4ZOhLgP7A&installer=1


[Q&A + HELP] How to make k8s coredns work with OVN DNS

Hello team,
We have a DNS specific problem. What we want is making K8S coredns can forwarding the POD dns request traffic into the OVN world, and OVN can give back the openstack DNS records.
We have an openstack-based Cloud with OVN, enable domain and DNS in neutron. The VMs can be pinged by its domain name with the correct FQDN. Then we deploy a K8S env on the said VMs with coredns, the PODs on the K8S can work well with the internal DNS name, but can not reach the VMs DNS world. Both of openstack and K8S deployments were configured with the same internal DNS service.
The DNS request of VMs is "hijacked" by OVN, but the coredns will forward the DNS request from POD to outside of K8S, it will be the similar network frame just like the VM sent the DNS request itself.  I saw the very strange network packages on the compute node, they were generated by the VM nic tcpdump capture.
VM ping the domain name in openstack world. (Work well)
23:52:35.428669 fa:16:3e:74:c8:b8 (oui Unknown) > fa:16:3e:d3:4a:85 (oui Unknown), ethertype IPv4 (0x0800), length 96: (tos 0x0, ttl 64, id 43939, offset 0, flags [DF], proto UDP (17), length 82)
    <IP_ADDRESS>.41224 > cedar.internal.com.domain: [bad udp cksum 0x9a35 -> 0x16d5!] 30257+ A? k8s-tt-new-bz.zonea.teststack. (54)
23:52:35.429924 fa:16:3e:d3:4a:85 (oui Unknown) > fa:16:3e:74:c8:b8 (oui Unknown), ethertype IPv4 (0x0800), length 148: (tos 0x0, ttl 64, id 43939, offset 0, flags [DF], proto UDP (17), length 134)
    cedar.internal.com.domain > <IP_ADDRESS>.41224: [no cksum] 30257- q: A? k8s-tt-new-bz.zonea.teststack. 1/0/0 k8s-tt-new-bz.zonea.teststack. [1h] A <IP_ADDRESS> (106)

Pod ping the domain name in openstack world.(NOT WORK)
23:47:50.668761 fa:16:3e:74:c8:b8 (oui Unknown) > fa:16:3e:d3:4a:85 (oui Unknown), ethertype IPv4 (0x0800), length 107: (tos 0x0, ttl 63, id 58906, offset 0, flags [DF], proto UDP (17), length 93)
    <IP_ADDRESS>.5448 > cedar.internal.com.domain: [bad udp cksum 0x9a40 -> 0x79bd!] 30058+ [1au] A? k8s-tt-new-bz.zonea.teststack. ar: . OPT UDPsize=2048 DO (65)
23:47:50.669640 fa:16:3e:d3:4a:85 (oui Unknown) > fa:16:3e:74:c8:b8 (oui Unknown), ethertype IPv4 (0x0800), length 159: (tos 0x0, ttl 63, id 58906, offset 0, flags [DF], proto UDP (17), length 145)
    cedar.internal.com.domain > <IP_ADDRESS>.5448: [no cksum] 30058- q: A? k8s-tt-new-bz.zonea.teststack. 1/0/0 . OPT UDPsize=2048 DO (117)

The last package VM received looks strange, as we got a zero A record. I'm not sure whether OVN is failed to dns_lookup in somehow and return the wrong DNS resp.
Could any one can help to leave a suggest about how to make them work as wish in our situation? Thank you.

@bzhaoopenstack thanks for the bug report!  It really looks like ovn-controller can't handle DNS requests that include OPT UDPsize=X.  I did a quick test:
$ ovn-nbctl list dns
_uuid               : 7d2b8afe-7d6f-4745-a28d-3aaf2a4cbe5d
external_ids        : {}
records             : {google.com="<IP_ADDRESS>"}

$ ovn-nbctl show
switch 1949fdae-2502-4b81-8e4d-d739a1bdf906 (ls)
    port vm2
        addresses: ["00:00:00:00:00:02"]
    port vm1
        addresses: ["00:00:00:00:00:01"]

Without bufsize as option:
$ ip netns exec vm1 dig google.com +bufsize=0

; <<>> DiG 9.11.36-RedHat-9.11.36-11.el8_9 <<>> google.com +bufsize=0
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 18412
;; flags: qr rd ad; QUERY: 1, ANSWER: 1, AUTHORITY: 0, ADDITIONAL: 0
;; WARNING: recursion requested but not available

;; QUESTION SECTION:
;google.com.                    IN      A

;; ANSWER SECTION:
google.com.             3600    IN      A       <IP_ADDRESS>

;; Query time: 0 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Tue Dec 19 10:44:06 CET 2023
;; MSG SIZE  rcvd: 54

With bufsize set:
$ ip netns exec vm1 dig google.com +bufsize=2048

; <<>> DiG 9.11.36-RedHat-9.11.36-11.el8_9 <<>> google.com +bufsize=2048
;; global options: +cmd
;; connection timed out; no servers could be reached
[...]
10:44:46.428097 00:00:00:00:00:01 > 00:00:00:00:01:00, ethertype IPv4 (0x0800), length 93: (tos 0x0, ttl 64, id 32910, offset 0, flags [none], proto UDP (17), length 79)
    <IP_ADDRESS>.48523 > <IP_ADDRESS>.53: [bad udp cksum 0x6396 -> 0xd37f!] 64569+ [1au] A? google.com. ar: . OPT UDPsize=2048 (51)

Ah, this is due to https://github.com/ovn-org/ovn/commit/4b10571aa89b226c13a8c5551ceb7208d782b580
@brianphaley do you think there's a safe way to ignore additional (EDNS) records or accept some of the common ones?  In this case dig seems to set +bufsize by default (I see it set at 4K on my system).

@dceara Thanks so much for your verify. And I also see the associated bug issue in openstack https://bugs.launchpad.net/neutron/+bug/2030294 .
And we are using the OVN based openstack, so we are confused that why the behavior is different than linux bridge/ovs based deployment.
During we deep into the DNS resp nw package, we found the package is very strange and wrong. Such as,

In our product deployment, we use CoreDNS for K8S deployment, and K8S runs on the openstack based Cloud. But seems CoreDNS enable the EDNS by default, we are considering to change the CoreDNS to other DNS service. If you could provide more advice towards this issue, that's really helpful to us. Thank you.

Disabling EDNS would be a way forward for now.  However, I really think OVN should try to handle this and ignore unknown resource requests of unknown type.

@bzhaoopenstack Would you be able to test the following change?

@dceara Thank you so much for such quick fix. Let me check whether there is any resource to verify this.

@brianphaley @bzhaoopenstack OK, I'll wait a few days from confirmation from you guys before posting this as a formal patch for review on ovs-dev.
Thanks,
Dumitru

@brianphaley @bzhaoopenstack Did you happen to have time to try out the potential fix?
Thanks!

I went ahead and posted the patch on the dev mailing list:
<EMAIL_ADDRESS>

I posted a v2 (it's the same fix as in v1 it just adds a test case):
<EMAIL_ADDRESS>


WebIDE documentation pages needing removal

This bug tracks pages from the WebIDE documentation that are no longer valid/need to be removed. This is work that is needed in preparation for FF #68.
Acceptance Criteria

[x] The page: Running and debugging apps has been removed.
[x] The page: Creating and editing apps has been removed.
[x] Debugging Firefox OS over USB has been removed.
[x] Debugging Firefox OS over Wifi has been removed.

Now that I have deleted these pages and their translations, please don't add new pages to this issue. Start a new issue instead and assign it to me, @irenesmith


Multicheckbox fails with no options selected

Getting some trouble with a manyToMany relationship. I have a Form with an ObjectMulticheckbox (from DoctrineORMModule, it extends Zend\Form\Element\MultiCheckbox) and this element do not allow to have no options selected.
Same problem here : https://github.com/zendframework/zf2/issues/2761#issuecomment-9709091
There is a temp fix so please don't yell at me, it needs some improvements but for now, it does the trick.
File \Zend\Form\Form.php
/**
 * Set data to validate and/or populate elements
 *
 * Typically, also passes data on to the composed input filter.
 *
 * @param  array|\ArrayAccess|Traversable $data
 * @return Form|FormInterface
 * @throws Exception\InvalidArgumentException
 */
public function setData($data)
{
    if ($data instanceof Traversable) {
        $data = ArrayUtils::iteratorToArray($data);
    }
    if (!is_array($data)) {
        throw new Exception\InvalidArgumentException(sprintf(
            '%s expects an array or Traversable argument; received "%s"',
            __METHOD__,
            (is_object($data) ? get_class($data) : gettype($data))
        ));
    }

    $this->checkExpectedFields($data);

    $this->hasValidated = false;
    $this->data         = $data;
    $this->populateValues($data);

    return $this;
}

/**
 * Check if all fields that we wanted are setted, very usefull
 * for instance of MultiCheckbox form element because we may
 * need them to be setted in our form in case of an empty selection
 *
 * @param array $data
 * @param array $validationGroup
 */
private function checkExpectedFields(array &$data, $validationGroup = null) {
    if (null === $validationGroup) {
        $validationGroup = $this->validationGroup;
    }

    foreach ($validationGroup as $subFieldName => $value) {
        if (is_int($subFieldName)) {
            $subFieldName = $value;
        }

        if (is_array($data[$subFieldName])) {
            $this->checkExpectedFields($data[$subFieldName], $validationGroup[$subFieldName]);
        } elseif (!isset($data[$subFieldName])) {
            $data[$subFieldName] = null;
        }
    }
}

I assume that we need all fields that are described in the validationGroup so if a field si in the validationGroup and not in the posted data we need to set it to default value (null here).

How about this?
https://github.com/zendframework/zf2/pull/2395

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


How can I overlay one image on another with a transparent background using Slab.Image?

I studied Wiki (Image, Layouts and Window), but dont understand how i can do this. Please explain to me

Hi BestHatzker,
Correct me if I am misunderstanding this, but from your question it seems you want to draw two different images on top of each other. To do this, you just need to place the cursor back at the position of the first image drawn.
local X, Y = Slab.GetCursorPos()
Slab.Image('Image_Bottom', {Path = "image1.png"})
Slab.SetCursorPos(X, Y)
Slab.Image('Image_Top', {Path = "image2.png"})

If this isn't what you are looking for, please elaborate what it is you are trying to accomplish and possibly provide a screenshot.

Thank you! It's really work :)


Fixed typos

Fixed two very minor typos I noticed while reading the documentation.

I'm not positive that the current version is incorrect. Can you cite a grammar rule?

My line of thinking was that the verbs "implement" and "accept" should be in third-person singular form.
Since "it" is a singular pronoun and "init() method" is a singular noun, the corresponding verbs (implement, accept) should have an s appended to them satisfy this rule.

I'm pretty sure the current wording is correct and that the tense (or mood?) is called the subjunctive. For example, this grammar site provides the following example:

I suggest that he implement a budget cut in March.

Hey, that's something I had actually never heard of! Didn't even think that far, just assumed it was a typo because it didn't sound "right" to me. Thanks for the lesson in English grammar.
I was a bit over-eager to contribute something (no matter how trivial), I guess.


IncludeFilter not working in specific situation

Trying to use something like this (using EF Core 3.0):
_context.CustomerLocations.IncludeFilter(c => c.Zones.Where(x => x.Polygon.Contains(c.Point)));
I get:
System.Exception: Oops! Select projection are not supported in EF+ Query IncludeFilter. For more information, contact us<EMAIL_ADDRESS>  at Z.EntityFramework.Plus.QueryIncludeFilterProvider1.CreateQuery(Expression expression)
at System.Linq.Dynamic.Core.DynamicQueryableExtensions.Where(IQueryable source, ParsingConfig config, String predicate, Object[] args)
at System.Linq.Dynamic.Core.DynamicQueryableExtensions.Where[TSource](IQueryable1 source, ParsingConfig config, String predicate, Object[] args)    at System.Linq.Dynamic.Core.DynamicQueryableExtensions.Where[TSource](IQueryable1 source, String predicate, Object[] args)
at CustomerApi.Controllers.Customer.CustomerLocationsController.GetLocations(Int32 page, Int32 limit, String filter, String order, Boolean expand, Boolean ignoreQueryFilters) in /Users/usr/Documents/GitHub/api/CustomerApi/Controllers/Customer/CustomerLocationsController.cs:line 61
at Microsoft.AspNetCore.Mvc.Infrastructure.ActionMethodExecutor.TaskOfIActionResultExecutor.Execute(IActionResultTypeMapper mapper, ObjectMethodExecutor executor, Object controller, Object[] arguments)
at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.g__Awaited|12_0(ControllerActionInvoker invoker, ValueTask1 actionResultValueTask)    at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.<InvokeNextActionFilterAsync>g__Awaited|10_0(ControllerActionInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)    at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Rethrow(ActionExecutedContextSealed context)    at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)    at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeInnerFilterAsync() --- End of stack trace from previous location where exception was thrown ---    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeFilterPipelineAsync>g__Awaited|19_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Logged|17_1(ResourceInvoker invoker)    at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)    at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)    at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)    at Microsoft.AspNetCore.Localization.RequestLocalizationMiddleware.Invoke(HttpContext context)    at CustomerApi.Startup.<>c.<<Configure>b__5_0>d.MoveNext() in /Users/usr/Documents/GitHub/api/CustomerApi/Startup.cs:line 122 --- End of stack trace from previous location where exception was thrown ---    at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)

Just worked without EF Plus:
.Include(c => c.Zones.Select(x => x.Polygon.Contains(c.Point)));

Hello @mhosman ,
It looks you use LINQ Dynamic which makes a projection under the hood which is not compatible with our features.
Unfortunately, there is nothing we can do here as we are not compatible with projection ;(
However, it's possible for you to replace all your LINQ dynamic call by the C# Eval Expression library. The library is not free but you can replace all your dynamic call by a library that really evaluates and compile code at runtime.
Let me know if that answers correction to your issue.
Best Regards,
Jonathan

Thanks Johnathan!


Add winget support

Add support for distribution via winget

Great idea!
Someone suggested this on Reddit too during v0.2 release

Hey @Samuel12321, Could you take care of this task?
I'll focus on coding and bug fixes in the mean time 😅😁

sure, already started

Thanks pal you've been so helpful along the way. If you weren't here to help I would've dropped this project 👍❤

I have submitted the required files to the winget repo. just waiting for them to review.
https://github.com/microsoft/winget-pkgs/pull/3946

That's awesome 👍
What about the certificate issue?

still working on that issue

done

What about the two yaml files in two different directories?

what?

See this https://github.com/ShankarBUS/ModernFlyouts/commit/49b249a798c6f4aca4443e46a816aa5f593b89c8#commitcomment-42990220


[Bug] Webpack plugin sample corrupted font file

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[X] Not reproducible in the monaco editor playground

Monaco Editor Playground Code
No response
Actual Behavior
When running the browser-esm-webpack-monaco-plugin sample the icons are failing to load, I'm having the same issue when running monaco editor with the webpack plugin from my project that uses Webpack 4, monaco editor v0.30.1 and webpack plugin v6.0.0.
This happens in all browsers.

Expected Behavior
The icons are loading.
Additional Context
Removing file-loader from webpack seems to solve this issue, whenever file-loader is included the font file is corrupted. Does anyone know of any workarounds for this issue, I can't drop the use of file-loader in my project.
const path = require('path');
const MonacoWebpackPlugin = require('monaco-editor-webpack-plugin');

module.exports = {
	mode: process.env.NODE_ENV,
	entry: './index.js',
	output: {
		path: path.resolve(__dirname, 'dist'),
		filename: '[name].bundle.js'
	},
	module: {
		rules: [
			{
				test: /\.css$/,
				use: ['style-loader', 'css-loader']
			},
			// This is causing the issue
			// {
			// 	test: /\.ttf$/,
			// 	use: ['file-loader']
			// }
		]
	},
	plugins: [
		new MonacoWebpackPlugin({
			languages: ['typescript', 'javascript', 'css']
		})
	]
};

I had the same issue. I found this: https://github.com/microsoft/monaco-editor/issues/2742 . I downgraded to<EMAIL_ADDRESS>and now it's working.


combines parameterized query phases

[x] fixes #24
[x] previously parameterized queries happened in 2 phases to allow for dynamic result format code decisions based on type (that way timestamp could continue to be sent as text while other formats were sent as binary). this PR reverts the ability to make those decisions dynamically based on type. instead, they must be sent as either unspecified, all text, all binary, or a pre-determined array of formats. this makes the parameterized query much simpler and faster.

This PR is highlighting a couple of problems with the way PostgreSQLTableNameCache is working. I think this is something that might need to be handled at a higher layer (in FluentPostgreSQL instead).


Update LICENCE

See https://trello.com/c/nhcs2ylM/48-add-mit-header-to-the-licenc-se-files

There's a lot of redundancy here.
Maybe we can centralise our licence in another app and pull it into the repos that need it?

Sounds like your Firebreak idea sorted.


[SPARK-36505][PYTHON] Improve test coverage for frame.py

What changes were proposed in this pull request?
This PR proposes improving test coverage for pandas-on-Spark DataFrame code base, which is written in frame.py.
This PR did the following to improve coverage:

Added unittest for untested code
Remove unused code
Add arguments to some functions for testing

Why are the changes needed?
To make the project healthier by improving coverage.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unittest.

Test build #142763 has started for PR 33833 at commit 21b2900.

Test build #142763 has finished for PR 33833 at commit 21b2900.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142763/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47263/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47263/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47263/

Test build #142768 has started for PR 33833 at commit a05ad0b.


Update search field styles on iOS devices

Issue Description

The search field text boxes have rounded corners on iOS 10.2
Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior

Current Behavior

iOS 10.2

Take a look at: https://css-tricks.com/webkit-html5-search-inputs/

This currently has the following styles applied, though it is still rendering oddly in Mobile Safari.
    -webkit-appearance: textfield;
    -moz-appearance: textfield;
    appearance: textfield;

Tech Design
I plan to switch the appearance: textfield; line to appearance: none;. This will allow the input to render like other text based inputs.


🔄 Synced file(s) with jhudsl/DaSL_Course_Template_Bookdown

Synced local file(s) with jhudsl/DaSL_Course_Template_Bookdown.

Changed files

Created local  .github/workflows/render-preview.yml  from remote  .github/workflows/render-preview.yml

This PR was created automatically by the repo-file-sync-action workflow run #1417888960

Please carefully review these changes and decide which are useful for your course.
See the release notes: https://github.com/jhudsl/DaSL_Course_Template_Bookdown/releases

If you don't want the changes from a particular file, you can always revert that particular commit before merging the sync PR.
If you will not want any updates on this file in the future, you may want to remove a file from being synced in your repo by reconfiguring the sync file.

If you want only some changes, but they are not on a whole file basis, you could check out the branch and make manual edits. To checkout the branch, navigate to your own repository you should be able to run:

git checkout repo-sync/DaSL_Course_Template_Bookdown/default

If you don't want any of the changes you can close the PR entirely.

You may want to unenroll your repository from the sync GitHub actions by filing a PR on DaSL_Course_Template_Bookdown
to delete your repo name from this file if this will continue to be the case.


Passing props down to SVG elements

I've encountered a really funny issue which is described in this Stack answer - https://stackoverflow.com/a/41164571 and I've put together a demo at - https://codepen.io/jjmax/pen/MqEmMa
In the demo if you remove hidden from the first SVG's class property you will see three SVG rendered. Add it back and you get one hidden and two squares (SVG 2 & 3).
The SVGs here are identical and the issue seems to be caused by the identical mask ids of each SVG. (aside - remove hidden from the first SVG and add it to the second one.....)
This demo doesn't use SVGR or SVGO but these SVGs are ones I am using in the app I'm building.
I have two courses of action -

Using SVGR and SVGO set the SVG elements to accept a prop, which will be a unique id when the SVG is loaded in the app. This way all SVGs will use different ids and will work as needed. But I don't know if this is possible. Is it?
Ask the designer to redo the SVGs to not use masks.

Any input on this would be great, thanks.

Just an update before I close this.
We've chosen to ask the designer to redo the SVGs without using mask. I'm not sure yet what work this will entail but t is the best answer for us as the project is a longterm one which will have multiple deployments and different SVGs per customer. The first solution adds a bit of overhead and potential to haunt us down the road.
I've done a bit more work on the idea and I think it should be possible to accomplish using SVGR - maybe replace-attr-values and template options or perhaps a script to modify the React components once they are generated.


Can not pass AVP_SECRET as env var to generate command in ConfigManagementPlugin

Describe the bug
I am trying to use (to pass AVP_SECRET env from ArgoCD UI) the following config map:
apiVersion: v1
kind: ConfigMap
metadata:
  name: cmp-plugin
  namespace: argocd
data:
  avp.yaml: |
    apiVersion: argoproj.io/v1alpha1
    kind: ConfigManagementPlugin
    metadata:
      name: argocd-vault-plugin
    spec:
      allowConcurrency: true
      discover:
        find:
          command:
            - sh
            - "-c"
            - "find . -name '*.yaml' | xargs -I {} grep \"<path\\|avp\\.kubernetes\\.io\" {} | grep ."
      generate:
        command: ["sh", "-c"]
        args: ['argocd-vault-plugin generate -s "$AVP_SECRET" ./ --verbose-sensitive-output']
      lockRepo: false

From the documentation -> https://argocd-vault-plugin.readthedocs.io/en/stable/config/#multitenancy:
This is the secret that I am trying to read:
apiVersion: v1
kind: Secret
metadata:
  name: vault-configuration-test01
  namespace: argocd
type: Opaque
data:
  AVP_AUTH_TYPE: XXX
  AVP_KV_VERSION: XXXX
  AVP_TYPE: XXX
  VAULT_ADDR: XXX
  VAULT_TOKEN: XXX

Notice that the secret name is parametrized via an environment variable. This means each Argo app manifest can set AVP_SECRET to be the name of the Kubernetes secret that contains the configuration for the backend needed to generate its secrets.

I installed the plugin by following the install instructions from here(init Container and sidecar) -> https://argocd-vault-plugin.readthedocs.io/en/stable/installation/
Expected behavior
Define env var AVP_SECRET in ArgoCD Application (UI) and use it with flag -s $AVP_SECRET in the generate command.
Screenshots/Verbose output

time="2023-10-09T15:33:43Z" level=error msg="`sh -c \"argocd-vault-plugin generate -s \\\"$AVP_SECRET\\\" ./ --verbose-sensitive-output\"` failed exit status 1: 2023/10/09 15:33:43
reading configuration from environment, overriding any previous settings
2023/10/09 15:33:43 AVP configured with the following settings:

2023/10/09 15:33:43 avp_kv_version: 2

2023/10/09 15:33:43 avp_secret: vault-configuration-test01

Error: Must provide a supported Vault Type, received
Usage:
  argocd-vault-plugin generate <path> [flags]\n\nFlags:\n  -c, --config-path string         path to a file containing Vault configuration (YAML, JSON, envfile) to use\n  -h, --help                       help for generate\n  -s, --secret-name string         name of a Kubernetes Secret in the argocd namespace containing Vault configuration data in the argocd namespace of your ArgoCD host (Only available when used in ArgoCD). The namespace can be overridden by using the format <namespace>:<name>\n      --verbose-sensitive-output   enable verbose mode for detailed info to help with debugging. Includes sensitive data (credentials), logged to stderr" execID=b0155

Secret has to be defined as ARGOCD_ENV_AVP_SECRET in configMap so you can use it in UI as AVP_SECRET.


Alterntives for named individuals

Is your feature request related to a problem? Please describe.
I tried to generate artifacts for https://github.com/solid/authorization-panel/blob/main/proposals/acp/solid-acp.ttl but the named individuals were absent.
Describe the solution you'd like
The generator uses owl:NamedIndividual but this vocab used rdfs:Resource. Could the generator work with either?
Describe alternatives you've considered
Extend the vocab or if owl:NamedIndividual is better we could suggest that for the ACP vocab.

Good observation @edwardsph - yeah, I do think owl:NamedIndividual would be the better option in the vocabulary itself, as I think that represents the intended semantics far better than the generic rdfs:Resource (which doesn't really convey any specific semantics at all). @acoburn - have you any thoughts on this - should we suggest this change to the panel do you think...?

Actually @edwardsph - I just added support for treating rdfs:Resourceinstances the same way we do owl:NamedIndividual anyway, as there's no harm in doing so (I still think the vocab itself should be updated though).
Let me know if you'd like a new release of the Artifact Generator, or if running locally yourself you can just update from main once this PR is merged: https://github.com/inrupt/artifact-generator/pull/580

For vocabs, I tend to avoid the more powerful owl-related semantics when they are not required. Here, rdfs:Resource conveys the intended meaning

Adding both does not hurt; then simple RDFS clients can get their thing, and more advanced OWL clients get theirs.


Introduce core-execution/execution-integ-tests project

And move Execution engine related tests out of integ-tests to execution-integ-tests.
Fixes https://github.com/gradle/gradle-private/issues/4135

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle clean

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test QFL

I've triggered the following builds for you. Click here to see all build failures.

QuickFeedbackLinux build

@bot-gradle test and merge


PowerApp Shoutouts_20201204001138.zip Import issue

PowerApp Shoutouts_20201204001138.zip Import doesn't have data connection to configure while importing , so unable to perform Step 7 .

But I think we can add our data source created when we edit the app after /importing so we can added needed data sources.

Am I right in saying that the Allow in step 11 creates that SharePoint connection?


fix includes for https://github.com/fritzsedlazeck/SURVIVOR/issues/75

PR fixing https://github.com/fritzsedlazeck/SURVIVOR/issues/75

Thanks.
Fritz


Use parser generator for parsing contracts

This is about 5 times faster than the standard parser

The parser generator isn't quite suitable for us so we use a hacked version
of nearley-webpack-loader to compile it
Given the above we need to use webpack to compile the tests
We still have the old parser as it is used by Blockly, we should be
able to remove this in another PR

Deployed to https://david.marlowe.iohkdev.io/
Had to do some nix rearranging to get things working, @krisajenkins best to check purescript.nix

@krisajenkins I will change some FIXMEs to TODOs, they are mostly issues that already existed that came up when playing around with this. I intend for more PRs to fix these problems after the re-design work is finished.

@krisajenkins I am confident that the quickcheck tests exercise the parser well, not least because they showed up quite a few issues :-)

@krisajenkins I am confident that the quickcheck tests exercise the parser well, not least because they showed up quite a few issues :-)

Haha. Fair enough. :-D


allow domain suffix update in shared networks

Description
This PR...

Fixes: #9405

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 10783

@blueorangutan test keepEnv securityGroups

@blueorangutan test keepEnv

@DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests

[SF] Trillian test result (tid-11176)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 57074 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9585-t11176-kvm-ol8.zip
Smoke tests completed. 139 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

needs the following (but on hold now) : test in advanced zone with security groups, add UI part, check question by @weizhouapache in https://github.com/apache/cloudstack/pull/9585#discussion_r1730859388 (and probably more)

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11007

[LL]Trillian test result (tid-7016)
Environment: kvm-alma8 (x2), Advanced Networking with Mgmt server a8
Total time taken: 50402 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9585-t7016-kvm-alma8.zip
Smoke tests completed. 139 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_mount_shared_fs
Failure
93.86
test_sharedfs_lifecycle.py

test_resize_shared_fs
Error
0.23
test_sharedfs_lifecycle.py

@blueorangutan package

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11056

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11060

@blueorangutan test keepEnv securityGroups

@blueorangutan package

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✖ debian ✔ suse15. SL-JID 11079

@blueorangutan package

@weizhouapache a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11084

@blueorangutan LLtest keepEnv securityGroups

@DaanHoogland a [LL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests


RuntimeError: Cannot add watch, already scheduled

It seems that when running cachier, I randomly get these errors:
2021-09-17 09:05:32,357 - fsevents - ERROR - Unhandled exception in FSEventsEmitter
Traceback (most recent call last):
  File "/Users/lorenz/.../signals/.venv/lib/python3.9/site-packages/watchdog/observers/fsevents.py", line 315, in run
    _fsevents.add_watch(self, self.watch, self.events_callback, self.pathnames)
RuntimeError: Cannot add watch <ObservedWatch: path=/Users/lorenz/.cachier/, is_recursive=True> - it is already scheduled

My current solution is to delete the cache, assuming it is corrupt, and then start from scratch.
The most similar error I found on the net was the one reported in https://github.com/spyder-ide/spyder/issues/14803. There, the solution was to unwatch on switching directories/projects.

Hey,
That's look very relevant! Thank you for reaching out.
Anyway to produce the full stack trace, pointing to a code line in cachier itself?
Or is this all you're getting?
It sounds like this will be hard to reproduce to test correctly.

I get that error with my flask server from time to time, I don't have more details:
ERROR:fsevents:Unhandled exception in FSEventsEmitter
Traceback (most recent call last):
  File "/Users/louismartin/miniconda3/envs/dev3.9/lib/python3.9/site-packages/watchdog/observers/fsevents.py", line 315, in run
    _fsevents.add_watch(self, self.watch, self.events_callback, self.pathnames)
RuntimeError: Cannot add watch <ObservedWatch: path=/Users/louismartin/.cachier/, is_recursive=True> - it is already scheduled

This seems to pop-up every now and then when I use multiple process to access the same cache value, but unable to reproduce consistently.

I can always reproduce the error on mac intel 12.7 using the latest cachier 2.2.2 and Python 3.11.1 with this script:
import logging
import threading
import time
from cachier import cachier

@cachier()
def func(x):
    logging.info("Calculating %s", x)
    time.sleep(1)
    return x * x

def thread_function(name):
    logging.info("Thread %s: starting", name)
    func(1234)
    logging.info("Thread %s: finishing", name)

def main():
    format = "%(asctime)s: %(message)s"
    logging.basicConfig(format=format, level=logging.INFO, datefmt="%H:%M:%S")

    func.clear_cache()

    threads = []
    for index in range(3):
        logging.info("Main    : create and start thread %d.", index)
        x = threading.Thread(target=thread_function, args=(index,))
        threads.append(x)
        x.start()

    for index, thread in enumerate(threads):
        logging.info("Main    : before joining thread %d.", index)
        thread.join()
        logging.info("Main    : thread %d done", index)

if __name__ == "__main__":
    main()

Output (the script hangs and I need to hit Ctrl-C):
11:30:25: Main    : create and start thread 0.
11:30:25: Thread 0: starting
11:30:25: Main    : create and start thread 1.
11:30:25: Thread 1: starting
11:30:25: Main    : create and start thread 2.
11:30:25: Thread 2: starting
11:30:25: Main    : before joining thread 0.
11:30:25: Calculating 1234
11:30:25: Unhandled exception in FSEventsEmitter
Traceback (most recent call last):
  File "/private/tmp/venv/lib/python3.11/site-packages/watchdog/observers/fsevents.py", line 324, in run
    _fsevents.add_watch(self, self.watch, self.events_callback, self.pathnames)
RuntimeError: Cannot add watch <ObservedWatch: path='/Users/ficarell/.cachier/', is_recursive=True> - it is already scheduled
11:30:26: Thread 0: finishing
11:30:26: Main    : thread 0 done
11:30:26: Main    : before joining thread 1.
11:30:26: Thread 2: finishing
^CTraceback (most recent call last):
  File "/private/tmp/try_cachier.py", line 40, in <module>
    main()
  File "/private/tmp/try_cachier.py", line 35, in main
    thread.join()
  File "/Users/ficarell/.pyenv/versions/3.11.1/lib/python3.11/threading.py", line 1112, in join
    self._wait_for_tstate_lock()
  File "/Users/ficarell/.pyenv/versions/3.11.1/lib/python3.11/threading.py", line 1132, in _wait_for_tstate_lock
    if lock.acquire(block, timeout):
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
KeyboardInterrupt

Packages installed in the virtualenv:
❯ python -V
Python 3.11.1
❯ pip freeze
cachier==2.2.2
portalocker==2.8.2
watchdog==3.0.0

I don't get the error in any of these cases:

if running on Linux
if time.sleep(1) is removed
if the disk cache is reused (not executing func.clear_cache())

Killing (CTRL-C) a long running cached function mid-miss can readily reproduce this.

Thank you both for chiming in, and @GianlucaFicarelli especially for the very detailed way to reproduce this!
This dramatically increases the chance that a past active contributor or a new one will take this on and try to solve this bug.


Updating named pipe parsing

Type of Change

[x] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Changes
Updates named pipe parsing to understand both \sql\query and \tsql\query trailing segments

I think that fail cannot legitimately be blamed on my change ;)

thank you! 🙇


CS has been damaged

CS decompression failed and the file is damaged

我这边测试了是正常解压的

运行也没有问题

能提供更加详细些的信息吗，或者您可以运行下面的命令来进行测试
cd /tmp
wget -O CobaltStrike4.3.zip https://raw.githubusercontent.com/No-Github/Archive/master/CS/CobaltStrike4.3.zip
unzip CobaltStrike4.3.zip

It could be network fluctuation


Fix/incoming track status

It fixes incoming track status recvonly/sendonly on sdp offers request.

@HoseinHaqiqian Oh one thing before I merge. Can you update CHANGELOG.md and pubspec.yaml


python3Packages.vllm: relax deps, add cuda stdenv

Description of changes
Dependent on PRs: https://github.com/NixOS/nixpkgs/pull/324672, and https://github.com/NixOS/nixpkgs/pull/324674

The latter adds a new version of cutensor, which is required to build cupy
The formers fixes cupy which is broken, and a dependency of vllm.

The changes made here:

remove hardcoded package version setting in requirements.txt (makes vllm not break if someone updates torch or cupy.)
adds a conditional setting of stdenv for building using the special env required to build cuda code.

Happy to make an overall joint PR to show the fixes work all together if that would make things easier. Have tested locally and they do in fact all build together happily.
@SomeoneSerge @samuela @happysalada (not sure Lach's handle)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The current package has an infinite recursion issue because of rec, and also I see there's a broken = !rocmSupport && !cudaSupport - is this intended to mean "at least one back-end has to be enabled"?

hmm I haven't encountered the infinite recursion, it builds successfully for me. How did you trigger it?
I didn't author the broken line, but I would assume yes it wants you to have either cuda or rocm (can confirm it reports itself as broken if I don't enable cudaSupport).

Huh, did you manage to get it to work without updating triton?
https://github.com/NixOS/nixpkgs/pull/292996

I think we should've first merged these:

Dependent on PRs: https://github.com/NixOS/nixpkgs/pull/324672, and https://github.com/NixOS/nixpkgs/pull/324674

...and got vllm back into a buildable state

Ah my bad, the build was green and the changes were minimal, i realise now it had to wait.


Rename: browser -> release; vendor -> browser

Based on discussion of #57 we should rename classes and variables named browser* (except browserName and browserVersion) to release*, because we intend a particular release of a browser. Similarly, comments and variable names that refer to vendor actually refer to a particular browser (which has multiple releases).
@arobins does that sound right?

sgtm

Fixed in 4f4d92b2983e3f8b4f1025069d85f0ca350c31e0


struct of file key

I know it has three parts volume id, file id and file cookie. but what's the exact format of file key?
for example 7,06600d5ead3144      7,066006f8e1da58
volume id is 7,  what about file id and file cookie?
thx

cookie is 5ead3144
file id is 06600d
cookie is fixed size, 32 bits.

if I use custom file key, is it needed the file id or cookie has the same size as the example above?

no limit to file id.
cookie should be fixed size, 32 bits.

got it . thx!


Fix misspelling in deprecation warning that was leading to a "Constant not Found" error

I happened to hit while upgrading from Bulkrax 1 :)


Bugfix/bug 7301 import parameter wizard error ms acess file

merged and deleted branch, please update ticket.


Add support reading orc data through the index of the column

Description
Currently, velox reads the data in orc/parquet/dwrf files through column names, This should be fine in most cases. However, in some scenarios, we need to support reading data by the index of the column.
E.g. When writing ORC files with hive versions earlier than 2.0.0, the output column name is similar to _col0 instead of the actual column name. When we read the orc file, we get column names from orc file like _col0 and _col1 (struct<_col0:string,_col1:string,_col2:string,_col3:string,_col4:string>). But the schema defined by our table is struct<server_ip:string,header:string,reqtimestamp:string,idfa:string,idfv:string>. In this way, the data we read through velox is all null.
By default, Presto reads data through the column index. For orc and Parquet, hive.orc.use-column-names and hive.parquet.use-column-names control the reading of data through column names respectively.
Similarly, spark also supports reading orc data by column index.
Therefore, it is necessary to read data through column index in velox.
Internally, I have implemented this function, but I feel that the code is not very well written, and want to ask the community's opinion.
CC: @Yuhta @mbasmanova

CC: @spershin Sergey has been looking into this.

This is scoped to be addressed with schema evolution support in Velox (#6065).  In the meantime, @spershin will have a solution to just replace the column names in file with names in table schema.

Going to roll this out in the next day or two.

PR with the proposed fix:
https://github.com/facebookincubator/velox/pull/6531

PR with the proposed fix:
https://github.com/facebookincubator/velox/pull/6531


Packed unsigned int to float conversion

These two failures (doc build and check) I've seen intermittently. Not sure the cause (maybe a pinned version of rust that's now out of date??) but will try to retrigger and see if it successes as first steps just to pass the tests. These failed tests succeed on my system.

Hi @jlb6740, I'm away until Mon Oct 26; @julian-seward1 might be able to review this PR sooner (otherwise I'm happy to look at it then!).

Closing this issue but Chris had some comments that I am responding to. I will reference this issue in the new PR.


Features page

Making /features have a complete set of links.
Testing plan:
Manually tested in dev environment.
GIFs or screenshots:

Merged, after squashing the commits, thanks @alya!


Fully qualified class name

Is there any way to access the fully qualified class name from an Automation Element? For example, 'MyProject.MyModule.MyWPFControl' rather than just 'ComboBox'.  We have a number of custom controls that all appear with ClassName=ComboBox but need to be automated differently. It would be nice to have a generic method which can perform the necessary automation based on the fully qualified class names without needing to mess around with any AutomationProperties to determine which control we're dealing with. Or is there an alternative way to approach this problem? I have worked a bit with Squish, which does have this feature, so I'm assuming there is some method I can use here.
Thanks for your time,
-B

Not directly. The ClassName is a .NET / WPF value, FlaUI only has access to what is exposed to UIA. What you can do in your WPF control is use a custom AutomationPeer and probably override the GetClassNameCore and return your class name there.

Closing as a solution is proposed.


java.lang.NoClassDefFoundError: shapeless/HListerAux

elasticmq version: 0.8.8
Not sure if this is an issue and I cannot see anything obviously wrong with my dependencies. I am getting the following exception:
java.lang.NoClassDefFoundError: shapeless/HListerAux$
at org.elasticmq.rest.sqs.directives.ElasticMQDirectives$class.action(ElasticMQDirectives.scala:24)
at org.elasticmq.rest.sqs.TheSQSRestServerBuilder$$anon$1.action(SQSRestServerBuilder.scala:89)
at org.elasticmq.rest.sqs.CreateQueueDirectives$class.$init$(CreateQueueDirectives.scala:19)
at org.elasticmq.rest.sqs.TheSQSRestServerBuilder$$anon$1.(SQSRestServerBuilder.scala:89)
at org.elasticmq.rest.sqs.TheSQSRestServerBuilder.start(SQSRestServerBuilder.scala:89)
when I use this line in my test:
val server = SQSRestServerBuilder.start()
The below is how the shapeless library is imported via spray-routing-shapeless2_2.11:1.3.3. There is no other mention of shapeless in the dependency tree except here:
[info]   +-io.spray:spray-json_2.11:1.3.2 [S]
[info]   +-io.spray:spray-routing-shapeless2_2.11:1.3.3 [S]
[info]     +-com.chuusai:shapeless_2.11:2.1.0 [S]
[info]     +-io.spray:spray-http_2.11:1.3.3 [S]
[info]     | +-io.spray:spray-util_2.11:1.3.3 [S]
[info]     | +-org.parboiled:parboiled-scala_2.11:1.1.7 [S]
[info]     |   +-org.parboiled:parboiled-core:1.1.7
Im using Spray 1.3.3, akka 2.3.11, aws-sdk 1.10.0.
Is this something that anyone has experienced?

hmm, "interesting" :) Just in case, I uploaded 2.3.9-SNAPSHOT which depends on akka 2.3.11 and aws-sdk 1.10.1. Can you try that?
This shouldn't really change anything as spray is the same, but who knows ;)
There's no other root cause exceptions further down the stack trace?

So I have managed to work around the problem by not using spray-routing-shapeless, and just using spray-routing. This seems to remove the shapeless 2.1.0 dependency and I end up with shapeless 1.2.4 on my classpath (transitive dependency of spray-routing (non shapeless)). This seems to work.

Ah, so 1.2.4 was evicted by 2.1.0. Soon I hope to move to akka-http which I think doesn't depend on shapeless, or at least on a newer one :)

0.9.0-beta1 now uses akka-http (https://s3-eu-west-1.amazonaws.com/softwaremill-public/elasticmq-server-0.9.0-beta1.jar)


Kill adb.exe on exit

Really hard to search for this so sorry if it's a duplicate.
Issue: adb.exe is still in task manager after program exits/closes.
Solution: Kill adb.exe on program exit.
Not a big deal but it would be nice.

There is a possible that other processes are using adb. If scrcpy kill adb when it exists, it will cause some problem in other processes.

How about the option to automatically kill it then?

why not just run it like scrcpy && adb kill-server, so adb will be killed the moment scrcpy successfully finishes its work?

Implemented option --kill-adb-on-close: a3cdf1a6b86ea22786e1f7d09b9c202feabc6949


Fix lock file

Why:

#305 added gatsby-remark-images but used NPM instead of Yarn, adding
the wrong lock file instead of changing the correct one. And the
dumbass reviewer missed that.

This change addresses the issue by:

Installing the missing dependency using bin/setup, updating the
correct lock file;
Removing the wrong lock file.

I miss one PR and everything goes into flames :joy:

I miss one PR and everything goes into flames joy

That's how much I miss you.


Not to be that guy, but...

You behind this project call yourself for programmers and developers etc, but you can not even fix such simple things as the leaderboard after the new Agario update? A bit funny (:-D) but tragically... Not to mention the ''ERTP'', when the minion feed you it stop the player and it starts lagging. And the game's physics in general are not the best and could be improved. A joke when compared to the original Agar.io. But but...
Please fix the leaderboard!

Why do we have to do the work for you? Learn javascript and do it yourself if it is so called 'easy'
And I don't really understand how this is an issue...

You behind this issue call yourself "not that guy" but you're behaving like one.
We're not connected with Moneyshit or Zeach in any way and we still don't know the new protocol's changes.
Intercepting raw bytes and figuring out what they do isn't easy at all. Try it yourself. Even though forairan and other people have already done this in the past it's not trivial to sit and look at numbers for a few hours all the time.
We're also doing this in our free time, as a hobby, and as you might have seen already nobody has time.
If the performance of this project isnt satisfying your demand then you should make an agar server emulator from scratch.
Please stop whining about our inactivity and try actually doing something productive.
@MastaCoder please close

This is why we need more collaborators, #MakeLukaCollaborator
Honestly issues are just getting lame these days

#700

#700

@AgarXTRA What's wrong with the physics too.

@Tyler3D kek'd

In fact, I have an idea
If it's so easy to do, do a pull request on it instead of whining to us.

The O.P apparently has a lot of money -> https://github.com/Megabyte918/MultiOgar-Edited/issues/621
He should contribute to MIT making a 1,000,000,000 camera lol -> https://www.wired.com/2011/12/is-this-really-one-trillion-frames-per-second/

triple kek

quad kek

why dis not closed?
@Megabyte918 @MastaCoder

Sorry I was at school.
My response: Do it yourself or don't say anything at all.


Azure Arc Kubenetes connection to Azure Monitor Managed Grafana

Is it possible to create a step by step how to connect Azure Arc Kubenetes cluster to Azure Monitor Managed Grafana?

@arthursilvany are you planning to use Prometheus also or do you wanna to connect on azure monitor?

I have checked this link Collect Prometheus metrics with Container insights - Azure Monitor | Microsoft Learnhttps://learn.microsoft.com/en-us/azure/azure-monitor/containers/container-insights-prometheus?tabs=cluster-wide#send-data-to-azure-monitor-managed-service-for-prometheus and it's not possible yet
@.***
Arthur de Oliveira Silvany (HE/HIM)
Cloud Solution Architect -  Engineering - Brazil
Support_COGSBrazil_1057
Mobile: +55 (11) 95081-4442
<EMAIL_ADDRESS>From: Junior @.>
Sent: sexta-feira, 17 de fevereiro de 2023 13:40
To: microsoft/azure_arc @.>
Cc: Arthur Silvany (HE/HIM) @.>; Mention @.>
Subject: Re: [microsoft/azure_arc] Azure Arc Kubenetes connection to Azure Monitor Managed Grafana (Issue #1687)
@arthursilvanyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Farthursilvany&data=05|01|Arthur.Silvany%40microsoft.com|1d0aef3b02f1470ee5c408db11058f6c|72f988bf86f141af91ab2d7cd011db47|1|0|638122487800529808|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=UDDOoG1%2FyB34%2BdlV5VioknPsc1PH8GHMz8Mb%2B9uNKGk%3D&reserved=0 are you planning to use Prometheus also or do you wanna to connect on azure monitor?

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fazure_arc%2Fissues%2F1687%23issuecomment-1434893972&data=05|01|Arthur.Silvany%40microsoft.com|1d0aef3b02f1470ee5c408db11058f6c|72f988bf86f141af91ab2d7cd011db47|1|0|638122487800529808|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=M6F%2FAfrm%2BSqwlClsA%2BhxzHXoAZnsw%2Fo3zjct3GNmGcI%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHPFGC5RKSR3QBKE6DEKYA3WX6SUTANCNFSM6AAAAAAU6E3J24&data=05|01|Arthur.Silvany%40microsoft.com|1d0aef3b02f1470ee5c408db11058f6c|72f988bf86f141af91ab2d7cd011db47|1|0|638122487800529808|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=qDNNFz3PivPofYbolh0hVqUYBgQ1IbRC6x%2FgKCqgi2A%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Is your request to have a scenario to show Managed Grafana working against Azure Arc-enabled k8s?
What did you confirm is not supported yet?

From this link in the purple note
https://learn.microsoft.com/en-us/azure/azure-monitor/containers/container-insights-prometheus

@arthursilvany, i found an away to expose the metrics from the cluster to a prometheus workspace in azure


Fix link to docs

Closing this since the link was fixed by https://github.com/zeebe-io/zeebe-get-started-go-client/commit/315fc46b9f547421a0e02e65ca912608097238c9


[Idea]: Folia Support

Describe the feature
Folia Support
Granted at the beginning some features won't be available
How is the feature useful to all Multiverse users?
Because Folia is much more efficient and it is good to do work early so you won't have to do work later.
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.

Duplicate of #2901


SummarizeOp fails if using core.NameScope

core.NameScope adds a '/' to blobnames.
SummarizeOp tries to create a file for each blob.
In Linux, filenames can't have '/' in them.
Thus, SummarizeOp fails when creating a file.

The error is like this:
RuntimeError: [enforce fail at summarize_op.h:32] log_file_->good(). Failed to open summarize file for tensor gpu_0/conv_w. rdstate() = 4
We may need a regex solution for this operator as well.

Oh shoot, you are right - things get wrong because the "/" slash needs to be replaced. Maybe we can do a find and replace with "#"? I hope no one uses # in their variable names.

I guess we want to keep the name scope in filename because different GPUs could have different stats.
So, I agree we should replace '/' with something in SummarizeOp
Or simply change the separator '/' with another char.


Added environment variable AS_ROOT

Just thought that running from root could be useful.

How is this useful? You are running lots of untrusted code and opening hole if AS_ROOT is set.

I agree with this PR. It's not wlc's responsibility to babysit users, and there are often legitimate reasons to run it as root or not care about the security implications (i.e. working in a throwaway VM or something).

What are these legimate reasons, give me some real life use cases so I can consider this?

If this was added the correct way to do it would be to simply make wlc either warn or do nothing when running as root, not to add any kind of mechanism to request it.
I like tools which attempt to run as they're told and any errors and warnings should come from actual attempts to access resources any such which fail.  I dislike UID checks, such as in other tools like pacman which requires fakeroot hacks.

Here's an example: I need to go from a fresh install of Arch to a working desktop ASAP to test something on a virtual machine. I don't care about the security of this machine, it's going to be destroyed in a few minutes anyway. The overhead of creating a user and setting things up for it is just an annoyance. (this scenario has happened to me when makepkg dropped support for being run as root, very annoying, especially because in that case I had to set up sudo and such)
Write software as if your users are smart enough to make their own informed decisions about it.

This is a tangent, but to be fair the nobody user is available for this reason.

Getting an interactive session as nobody requires some setup on most distros, though. You'd get the same amount of setup necessary to just create a new user.

Maybe just remove the root check if there is need for this. Env variable for this seems funny.

Agreed.

you can do what makepkg used to do (still is?) in arch, were you could run as root if you run with --as-root

@nicman23: Not since https://projects.archlinux.org/pacman.git/commit/?id=61ba5c961e

I've always disagreed with that change, it's pretty far removed from the arch way imo.

Maybe it should be included in the API itself so compositors can handle it differently? I'd prefer the --as-root flag but let the compositors handle how that works.

This issue isn't about makepkg.
The proper solution here is to not have any root checking at all thus because pull request adds unnecessary code/complication, it likely won't be merged.
Try removing the UID checks instead, that would likely have more success.

https://github.com/Cloudef/wlc/commit/46c44f763acb47ef08a3daafbca33ea909b00574


Model Nav Bar

Description
Added Model NavBar
Fixes #23
Type of change
Please delete options that are not relevant.

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

https://user-images.githubusercontent.com/96647299/196059731-fa6ff7b4-0f02-48d8-8c1e-4f965a2e19c3.mp4

@Debajyoti14
check this #44 for issue #23 and guide me if anything update is required

Congrats @Perumallpardhiv 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐
SignIn / SignUp in ClueLess Website .
Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


crash on moving block directly infront of source

upon moving a block that is directly infront of the optical source (or moving a block to be directly infront) with a piston and possibly other methods while the source is emitting a beam causes the game to crash
crash-2024-10-28_20.55.44-client.txt
crash-2024-10-27_22.06.54-client.txt
crash-2024-10-27_22.24.24-client.txt

after a little bit more testing it actually seems to be when a change to the active beam occurs when an indicator that is not in digital mode is involved with the beam in some way

same issue


models and transactions | question

Hi, first of all thanks for this library. I used bookshelf and it was nightmare.
Anyway I have worked with models and transactions little bit different and therefore I am trying to do same thing with objections as well. I am not sure if it is OK and therefore I am here :)
Here is what I am trying to add

I am registering all models because I would like to get any model in any function without "import  User from ...";

class Test extends Model {
  myFunction() {
    const { User, Photo } = this.$models;
  }
}

The main reason is that I will get automatically models with binded transaction inside my transactions. Here is example

class Test extends Model {
  myFunctionWithTransaction() {
    this.transaction(async () => {
      const { User, Photo } = this.$models;

      const user = await User.query().where({ id: 123 }); // User is extended Model with trx from latest transaction created on demand

       // this.$query() is binded to transaction as well because I called transaction on this object
       this.$query().where({ ... })
    });
  }
}

I am able to use it with constructor as well.

class Test extends Model {
  myFunctionWithTransaction() {
    this.constructor.transaction(async (models) => {
      const { User, Photo } = models;

      const user = await User.query().where({ id: 123 }); // User is extended Model with trx from latest transaction created on demand
    });
  }
}

Can you tell me what do you think about it?
Here is my plugin for model register
const models = {};

export default Model => class Models extends Model {
  static extend() {
    const currentKnex = this.knex();
    class NewModel extends this {}

    return NewModel.bindKnex(currentKnex);
  }

  static get models() {
    if (!this.$modelsHandler) {
      const knex = this.knex();
      const handler = {
        get(target, name) {
          const model = models[name];
          if (!model) {
            throw new Error(`Model ${name} was not registered. Use Model.register`);
          }

          // use transaction from current model
          return model.extend().bindKnex(knex);
        },
      };

      this.$modelsHandler = new Proxy({}, handler);
    }

    return this.$modelsHandler;
  }

  get $models() {
    return this.constructor.models;
  }

  static register() {
    if (models[this.modelName]) {
      throw new Error(`Model ${this.modelName} is already registered`);
    }

    models[this.modelName] = this;
  }
};

Here is my plugin for transactions
import objection from 'objection';

const TRANSACTION_TIMEOUT = 60000;

export default Model => class Models extends Model {
  static transaction(fn) {
    return objection.transaction(this, (ModelTrx, trx) => new Promise(async (resolve, reject) => {
      const NewModel = ModelTrx.extend(); // extended model has different dataloaders

      let timerId = setTimeout(() => {
        if (timerId) {
          timerId = null;
          reject(new Error(`Transaction timeout after ${TRANSACTION_TIMEOUT}`));
        }
      }, TRANSACTION_TIMEOUT);

      try {
        const result = await fn(NewModel.models, trx);
        if (timerId) {
          clearTimeout(timerId);
          timerId = null;
          resolve(result);
        }
      } catch (error) {
        if (timerId) {
          clearTimeout(timerId);
          timerId = null;
          reject(error);
        }
      }
    }));
  }

  async transaction(fn) {
    const trxBefore = this.$transaction();

    const result = await this.constructor.transaction(async (models, trx) => {
      this.$transaction(trx);

      // update $models
      this.$pushModels($models);

      try {
        const result = await fn(models, trx);
        this.$popModels();

        return result;
      } catch (error) {
        this.$popModels();

        throw error;
      }
    });

    // todo use trxBefore after transaction
    this.$transaction(trxBefore);

    return result;
  }

};

I don't know what $pushModels and $popModels do, but you do realize that there if you call objection.transaction twice there is absolutely no quarantee that the first call's callback gets called first. I hope$pushModels doesn't modify the shared model constructor. Also there is no guarantee which transaction takes how much time, so $popModels calls an happen in any order, not necessarily in the order $pushModels happened.

Other than that, this seems like an interesting approach! Can be very useful if you write alot of your code in model methods.

Thank you @koskimas for fast response.
Yes all my logic is in the models. This was the main reason why I did it in this way.
Order is guaranteed  because I am waiting for response with "await". If you are in the transaction you are able to create only nested transaction.
All models are extended when you use transaction (it is same like bindTransaction, I am using own extend method from models plugin)
In the feature:
I can skip extending if I am currently in the nested transaction but I am not doing it in this moment.
I can see just one problem. I totally fine with this.constructor.transaction (static method) but I am not sure with this.transaction (instance method)
From my point of view I am able to set transaction to "this" via this.$transaction(trx);. But I need to use new $models which are connected to new transaction as well.
Here is updated transaction plugin. I did not test it. It is just idea for objection. I have working model in bookshelf.
import objection from 'objection';

const TRANSACTION_TIMEOUT = 60000;

export default Model => class Models extends Model {
  constructor(...args) {
    super(...args);

    this.$currentModels = [];
  }

  static transaction(fn) {
    return objection.transaction(this, (ModelTrx, trx) => new Promise(async (resolve, reject) => {
      const NewModel = ModelTrx.extend(); // extended model has different dataloaders

      let timerId = setTimeout(() => {
        if (timerId) {
          timerId = null;
          reject(new Error(`Transaction timeout after ${TRANSACTION_TIMEOUT}`));
        }
      }, TRANSACTION_TIMEOUT);

      try {
        const result = await fn(NewModel.models, trx);
        if (timerId) {
          clearTimeout(timerId);
          timerId = null;
          resolve(result);
        }
      } catch (error) {
        if (timerId) {
          clearTimeout(timerId);
          timerId = null;
          reject(error);
        }
      }
    }));
  }

  async transaction(fn) {
    // const trxBefore = this.$transaction();

    const result = await this.constructor.transaction(async (models, trx) => {
      this.$transaction(trx);

      // update $models
      this.$currentModels.push(models);

      try {
        const response = await fn(models, trx);
        this.$currentModels = this.$currentModels.slice(0, -1); // immutable pop
        this.$popModels();

        return response;
      } catch (error) {
        this.$currentModels = this.$currentModels.slice(0, -1); // immutable pop

        throw error;
      }
    });

    // todo use trxBefore after transaction
    // this.$transaction(trxBefore);

    return result;
  }

  get $models() {
    if (this.$currentModels.length) {
      return this.$currentModels[this.$currentModels.length - 1];
    }

    return this.constructor.models;
  }
};


iOS client data sources

Adds client data source for points, polygons, and polylines under TGMapData.
Update demo app to fix warnings.
Update umbrella header to include all public header files to fix warnings.

See demo app for sample usage.

Good to go when CI finishes.


[AutoDiff upstream] Upstream @derivative attribute serialization.

Upstream @derivative attribute serialization/deserialization.
Test all original declaration kinds and various wrt: parameter clauses.
Resolves TF-837.

@swift-ci Please test

I believe this patch is straightforward and non-controversial, so I'll merge it now.
Happy to address any feedback later!


PDF JS Firefox extension unsigned again (v1.4.141)

Everything was just fine until today. v1.4.141 broke signing, v1.4.139 was just fine so it's #7093. Obviously it doesn't make much sense as it's just a fix for a translation engine issue.

v1.4.143 wfm -- probably AMO was temporary out of service. Closing as resolved.


Is there a sidebar in this theme?

Hi @freeze-dolphin, there is no sidebar yet. I intend to create one according to my needs and also if someone asks, of course. I am available and happy to accept project pull requests and updates.
So we don't miss this opportunity, I'm going to open a task for improvement. Thanks!

Is the sidebar available now?


Don't explode in IE8

This wont make it work in IE8, but it makes the syntax valid, which allows me t exclude it with a runtime check, instead of passing it through es3ify during build

@chibicode ping

@SimenB published as 0.6.3!


Consume Input if enabled

Added a configuration to consume the input events when the console is focused.
That prevents actions being triggered by typing in the console.

Thanks for your contribution, @Japortie!
This issue was known for a long time but since I don't do godot that much lately I did not know how to fix this issue.
PR Looks good. No time to test right now, so I'll take another look at your PR a little more closely later.

I'm no Godot expert so there might be a better way, but this is what worked for me so far.

Hello again!
I might be doing something wrong or I don't understand this, but this doesn't seem to work for me.
I have this Demo project installed locally with console to run tests on.
Player is able to move character with keyboard arrow keys in this demo. I have console open, input box has focus, when I press arrow keys character moves.
Tested on

v3.2.2.beta.custom_build.757d8b567
v3.2.1.stable.mono.official

@all-contributors please add @Japortie for code

Thanks for adding me to the Contributors ;)
The problem occurs due to how Godot handles Input. The demo you referenced checks in the _process() method if the key for right or left is pressed (polling mode) which can't be prevented by the console.
If a game handles Input using Events via "_unhandled_input()", it won't receive the event anymore because the gui input is handled before that. The in detail explanation for Input handling in Godot can be found here
I also forked the 2d platformer demo and made a small showcase where the left and right input is not triggered when the console is open, because it's processed in via unhandled_input:
https://github.com/Japortie/godot-demo-projects/tree/feature/use-unhandled-input-for-move-left-right
Best Regards

Thank you for your contribution.
I've cloned the fork and tested it, works smoothly.
Thanks
Best


Use ipinfo.MyIPWithOptions instead of ipinfo.MyIP. Timeout in 5 seconds.

LGTM!

Hey @gkodali-zededa looks like you need to update this PR since there are now conflicts with what's in master. Could you please do that?
Also, while doing it -- you may want to get rid of the parasitic merge commits sitting in this PR by just squashing everything


no equal distance between points on x-axis

I couldn't find if this is possible:
If I have, for example, 3 data-entries with time = 1,2 and 10
What I want is a line graph with distances between the points on the x-axis according to the time of the data entries. So that the distance between points 1 and 10 = 10x the distance between points 1 and 2.
But when I add these points to a LineChartView, I need to give them xIndex 0,1,2
and the distance between 1, 2 and 10 is equal everywhere.
Is it possible what I want or is this not a feature (yet)?

I have asked for such a feature in #176 (#194 the same) a long time ago. But it needs lot of rewrite and even before start writing, we need to think about a lot of things... So it is a big change and no ETA yet.
I suggest you workaround it buy inserting the rest xIndex but no y value provided

@liuxuan30 你意思是ChartDataEntry初始化的时候value参数传0？


GOGRepo not working with new installs

Hi,
When setting up the gogrepo-addon, I cannot login
attempting gog login as 'email' ...
cannot continue, gog is asking for a reCAPTCHA :(  try again in a few minutes.
exiting...
Googling the error, gives me some hints on creating the gog-cookies.dat manually, but no real info on how to do that. Should perhaps be outlined in the doc?
The Chrome-addon "Cookies.txt" was supposedly an easy way to get the needed info, but just pasting that info into the /home/pi/.gogrepo/gog-cookies.dat was not the solutiuon.

yeh also see the same but this is not a RetroNAS issue to fix. gogrepo application needs to understand/fix why it is being captcha'd

potential fix https://github.com/eddie3/gogrepo/issues/65#issuecomment-1250761919
is it possible to implement in the fork that retronas uses?

workaround added to fork, closing this for now


Location maps

Finished adding maps for every pokemon

@l-marazovic-HTL please review the changes to your manually added descriptions before approving this pr.

@l-marazovic-HTL please review

The most Location are right and some have to be fixed (for example snorlax or butterfree)

I have already tracked down the bug and will try to fix it right away

Fixed in this commit!


Syntax error when starting the script

I have just installed the script following the instruction and, whenever I start it from Photoshop, it just pop s up an error (see the screenshot attached for all the details).
Thank you for your support.

@Onsyzygy thanks for the issue.
Usually this kind of issue occurs when downloaded ZIP is being renamed to jsxbin. So please be sure you have unarchived the downloaded ZIP and the Size Marks.jsxbin within that ZIP is in use.

Hi there!
Sorry for the super-late reply but yes, I solved the problem!
Thanks a lot for your help, I will close the ticket now.

Hi again.
After some time, the error is again there. I don't know what happened. I unzipped the file in the script directory but to be sure, I put a copy in its folder too, in that directory. The name of the app is still unchanged, and that is: Size Marks.jsxbin.
I don't know why this error is happening again.
Thanks a lot for your help in advance!

Hi again.
After some time, the error is again there. I don't know what happened. I unzipped the file in the script directory but to be sure, I put a copy in its folder too, in that directory. The name of the app is still unchanged, and that is: Size Marks.jsxbin.
I don't know why this error is happening again.
Thanks a lot for your help in advance!


Signs display toggle

I was wondering if it was possible to be able to toggle the display signs on the fly, without having to change the configuration and restart Neovim. There were no indicators in the README or source that this functionality currently exists. I opportunistically (poorly) tried accomplishing this by changing lua/todo-comments/highlight.lua (see snippet below). This sort of works, however after setting the global variable todo_comments_disable_signs to true the signs only disappear after switching back and fourth with another buffer (redrawing issue ?).
Is it currently possible to toggle displaying signs on the fly ? If not, is that a feature request to make ? 👀
--- a/lua/todo-comments/highlight.lua
+++ b/lua/todo-comments/highlight.lua
@@ -126,12 +126,14 @@ function M.highlight(buf, first, last)
       end

       -- signs
-      local show_sign = Config.options.signs
-      if opts.signs ~= nil then
-        show_sign = opts.signs
-      end
-      if show_sign then
-        vim.fn.sign_place(0, "todo-signs", "todo-sign-" .. kw, buf, { lnum = lnum + 1, priority = 8 })
+      if not vim.g.todo_comments_disable_signs then
+        local show_sign = Config.options.signs
+        if opts.signs ~= nil then
+          show_sign = opts.signs
+        end
+        if show_sign then
+          vim.fn.sign_place(0, "todo-signs", "todo-sign-" .. kw, buf, { lnum = lnum + 1, priority = 8 })
+        end
       end
     end
   end

It would be nice to be able to toggle highlighting as well 👁👁
Great work, thank you!

Good idea. I'll see what I can do!

I just pushed some changes:

you can now simply run setup again with different options. the options override the options from previously calling setup
you can also call require("todo-comments").disable() to disable everything

so for @lcrockett:
-- disable signs
require("todo-comments").setup({signs = false})

-- enable signs again
require("todo-comments").setup({signs = true})

@edshamis
-- disable highlighting, but keep signs
require("todo-comments").setup({highlight = {before ="", keyword = "", after = ""}})

disable completely:
require("todo-comments").disable()

Thank you for the suggestion!

Tested -> this works great, excellent. In addition to this change, would it also be possible to allow enabling / disabling signs to be buffer bound ?

@lcrockett that's not possible right now.
What would be your use case for this?
I could add some options to exclude certain filetypes etc, but not sure if that's what you want

I personally use lots of splits within Neovim and additionally move to other adjacent TMUX panes. When doing so, I currently use autocommands (WinEnter, WinLeave, WinNew, BufEnter, etc..) to hide / unhide visualisations when switching buffers, like LSP indicators, git signs, listchars, indent markers, etc. and on top of it use vimade to fade all but the active Neovim split. This way i'm not distracted by other buffers in sight that are screaming for attention.
What i'm after is having the active buffer show todo signs, highlights, or both, if it's the active buffer. While still maintaining the possibility to keep todo visualisations in sight for out of focus buffers, just in case it is deemed helpful at that particular time, if that makes sense. Hence the buffer specific todo visualisation toggle request.
For an LSP indicator example of what i'm currently doing to hide / unhide visualisations, see https://github.com/neovim/neovim/issues/13324#issuecomment-847353681.


feat(pdk) implemented kong.service.request.set_cert_key()

This feature allows plugins to override client certificate used while handshaking with services dynamically.

Also adding a note that we should bump the PDK version number when adding new features like this one


Update troubleshooting.md

Fix typo

@rolyon
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @andywcoder
Thanks for this update. We'll work on getting this merged.
thanks

#sign-off


feat(zeebe): extract connector mappings

This PR adds an extractor for the zeebe:TaskHeaders used by connectors. This will add variable names from connector mappings to the overall scope.
We could add this extractor as an additional module, but I decided to implement it here, as we can re-use the feel utilities needed for parsing the result expression.
related to https://github.com/camunda/camunda-bpmn-js/issues/265

To better understand what this PR does:

Result variable defined (if defined) is added to global scope
Result variables extracted (throught context) are added to global scope

Correct?

Correct, except that they are not added to the global scope but to the nearest scoped parent. In most cases this will be the process scope, unless you use subprocesses


Also allow for SVG images to be used.

Signed-off-by: Kurt Garloff<EMAIL_ADDRESS>

OK, it's already done ...


Maps disappearing

installed on the Forge
both foundry v10 and v11, pf2e games
certain maps are not loading even if I add the image again
Find the Culprit detected this module to be the issue

I too see this behavior; with V11 and DnD system. I did notice this only happens if I change the only setting in Super Select. If I switch the option to On, the problems starts.

Same in V12


Dogfooding Notes

Capturing some initial notes on the current UX:

Landing page lets you mint without logging in

After minting is finished, the button below indicates purchase, which then prompts for a login. This can be confusing because its not clear why a password was required to access the minting page (The button on the password prompt says "Log In", but doesn't actually log you into your wallet).
Suggestion: users should be prompted to log into the wallet first?

Unclear how to add funds to wallet

There is no way to access the "funding" page in the dev-wallet from the UI. User has to log out and log back in again. The funding page is also hidden behind "manage". Ultimately this is confusing.

Insufficent finds error is too small. It's not clear there is a problem. The dev-wallet closes and you're left wondering. This should be a modal, with instructions about how to add funds to the wallet?

After purchasing you're immediately prompted to list your items. This feels awkward. Should redirect user to their profile after purchase, with the purchased item highlighted in some way.

On testnet, after a user logs in using their wallet, the top banner reads "Log in to the Service Account and initialize it to get started." but this is impossible. More confusing messaging around the mock-login to the admin/minting page.

The title of the "Store" page says "Marketplace". This is confusing.

Clicking "purchase" if you're not logged in, bring up the login dialog, you log in, but have to click purchase again to complete the purchase.

adding some more notes from observations:

If minting fails, there is no error message. it seems like a glitch in the UI and could be handled better


Queries for graph generation

Resolves #30

Codecov Report
Patch coverage: 93.02% and project coverage change: -0.36% :warning:

Comparison is base (89dce07) 94.30% compared to head (286ee0a) 93.95%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
- Coverage   94.30%   93.95%   -0.36%
==========================================
  Files          10       13       +3
  Lines         246      331      +85
==========================================
+ Hits          232      311      +79
- Misses         14       20       +6

Files Changed
Coverage Δ

pandasaurus/graph/graph_generator.py
90.90% <90.90%> (ø)

pandasaurus/query.py
96.20% <92.85%> (-0.77%)
:arrow_down:

pandasaurus/utils/logging_config.py
94.73% <94.73%> (ø)

pandasaurus/graph/graph_generator_utils.py
100.00% <100.00%> (ø)

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


Duplicate keys detected not warned

Version
2.5.17
Reproduction link
https://codesandbox.io/s/zwm20ozlym
Steps to reproduce
check reproduction url
What is expected?
Duplicate keys detected to be warned
What is actually happening?
Duplicate keys detected not warned

when addVnodes is called, vnodes with same key are not warned

Hi, may I try to make this improvement my first contribution? I have already had some situations with duplicated keys in which a warning would have helped.

of course, nobody is working on this 🙂

Added a pull request (#8881) with a new test case and a solution.


[Snyk] Security upgrade aiohttp from 3.8.6 to 3.9.4

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

521/1000  Why? Recently disclosed, Has a fix available, CVSS 4.7
Cross-site Scripting (XSS) SNYK-PYTHON-AIOHTTP-6645291
aiohttp:  3.8.6 -> 3.9.4
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

Included in #169


fix(gatsby-plugin-nginx): remove trailing slash

What's the purpose of this pull request?
same as #1145 for 0.x branch
How it works?

How to test it?

base.store Deploy Preview

References

preview using this version https://sfj-446d5ed--base.preview.vtex.app/


Custom Logging for Team 2 CSV

Summary
Team 2 needs to export some CSV files at the end to evaluate their agents
I added a CustomLogs function to base agent that gets called at the end of the simulation only if you set flag --log to your agent type.
E.g.: go run main.go --log=Team1Agent1
I also fixed the config file because it seems like we allowed an old config file version to go through in the commits with the wrong key names

I hope this counts as testing :o


dosemu2: init at git 2020-11-04

Motivation for this change
I wanted to run dosemu2 on nixos so I made a package.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 102819 run on x86_64-linux 1

1 package built:

dosemu2

Result of nixpkgs-review pr 102819 run on x86_64-linux 1

1 package built:

dosemu2

I am not using nixos anymore, therefore I will not be maintaining this. Thanks for taking the time to review it so far.
If anybody wants to adopt it, of course feel free.


Unable to build the image

Correctly set up the env, and try to build one by myself, but seems like there is an error about the yarn package
#16 216.5 @calcom/web:build: Type error: Cannot find module 'node-mocks-http' or its corresponding type declarations.
#16 216.5 @calcom/web:build:
#16 216.5 @calcom/web:build: > 1 | import { createMocks } from "node-mocks-http";
#16 216.5 @calcom/web:build:     |                             ^
#16 216.5 @calcom/web:build:   2 |
#16 216.5 @calcom/web:build:   3 | import type {
#16 216.5 @calcom/web:build:   4 |   CustomNextApiRequest,
#16 216.9
#16 216.9  Tasks:    5 successful, 6 total
#16 216.9 Cached:    0 cached, 6 total
#16 216.9   Time:    3m34.399s
#16 216.9 Failed:    @calcom/web#build

Solution: cd to calcom folder and run yarn add node-mocks-http


feature request: font size

Hey, amazing project. Could you add the feature to change the font size?
On iPad its sometimes hard to read it (if you don't want to touch the display with your nose).
The Browser-Zoom is kinda weird when autoscroll in enabled.
Thanks :)

Hey @gasinkas thanks for your support !
I just added the feature on the tab page :

Hope you'll enjoy it :)