A way to update the TxtInOut files without updating the SQL database?

Hey,
I know we at OPTAIN/CS10 use our Database+TxtInOut a little differently than you do (ie. after running buildR we don't modify the SQL anymore, just edit the text files it gave us). But for this reason most of the prepR functions only half work for us. Such as add_atmo_dep. It does work as it writes our text file which we want, but then the function gets an error trying to write to an SQL which we don't use (or need). An easy way for us to get around this is to just run the function, let it write our text file, and then catch the error.
An example is below: where the db_path does not contain an SQL. It works as atmodep.cli is wrriten out, however the error must be caught.
hacked_atmodep <- function(df, db_path, t_ext) {
+     tryCatch(
+         {
+             SWATprepR::add_atmo_dep(df,db_path, t_ext)
+         },
+         error = function(e) {
+             message('catching the error')
+             print(e)
+         }
+     )
+ }

>
> hacked_atmodep(df, "model_data/cs10_setup/scenarios/baseline/", "annual")
[1] "Atmospheric deposition data were written into  model_data/cs10_setup/scenarios/baseline/atmodep.cli"
catching the error: : Could not connect to database: unable to open database file

So a suggestion I would like to make is to just add an extra flag of whether or not to update the SQL. I could implement it myself and add a pull request if you would like?

Hi Moritz, very good suggestion. I put this on a list to fix this sometime, but you are very welcome to update this function.


libswift: fix a few bugs in StackList

Use MemoryLayout.stride instead pf MemoryLayout.size. This fixes a buffer overflow bug in case of unaligned elements.
Plus some other bug fixes for stacklists which get larger than a single slab.
Also, add the append(contentsOf:) method.
Unfortunately l cannot provide a test case right now. But it will be tested when I land another change, which uncovered these bugs.

@swift-ci smoke test


Urlencoded routes with query in TreeRouteStack being assembled wrong

I have a route that contains an encoded forward slash (%2F). Assembling the route with a query string returns an unexpected string.

Expected
Actual

/this%2Fthat
/this%2Fthat

/this%2Fthat?foo=bar
/this/that?foo=bar

I've written a failing test on a forked repo. Below is an excerpt of running phpunit on TreeRouteStackTest.
$ phpunit ZendTest/Mvc/Router/Http/TreeRouteStackTest
PHPUnit 4.1.4 by Sebastian Bergmann.
..............SS..F................
There was 1 failure:

1) ZendTest\Mvc\Router\Http\TreeRouteStackTest::testAssembleWithEncodedPathAndQueryParams
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'/this%2Fthat?foo=bar'
+'/this/that?foo=bar'

I believe the issue is caused by the call here to normalize()

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


Server wither cat crash

I'm using a custom modpack, forge .1240, and EnderZoo-1.7.10-<IP_ADDRESS>.jar. A player on my server hit a wither cat, and the server crashed. It was a seemingly random occurrence, but I'd appreciate it if you could take a look, so it doesn't happen again.
Here is the crash report: http://pastebin.com/AFKJA8SL
Please take a look at this whenever you can, thank you :)

I'm getting this as well. Using direwolf20 1.0.3.. tried updating to the latest enderzoo to no avail.


Revert "Support for zk:// for mesos_master_url"

Reverts memsql/memsql-mesos#5
Fails with "ImportError: No module named cli"

I've re-pushed the Docker image with this change.


Unit used for distances?

Global world coordinates, that is.
Currently the distance unit is (as far as I know) not actually defined as meters or such. To make sure we're all consistent about everything we should obviously define it as something.
First idea of obviously meters. Problem with that is, grid tiles wouldn't be 1x1 meter large (that's too large to work with, SS13 looks really ugly because of it). They'd be closer to like, 0.5 or 0.33.
Now the problem with that is, you can't just "fetch the next tile" anymore by adding 1 to your x axis. It's a minor usability concern but I figured it'd be worth making a thread about.
I feel that we should still go with meters and just live with it that one tile is NOT one meter.

Are we associating world units with tile indices? When thinking about the speed of something, would you say "I want the object to move 2 tiles/sec.", which would depend on knowing the size of tiles, or would you want to say "I want the object to move 2wu/sec".
Dissociating tile indices from world units would allow the scaling between tile indices and world units to be completely handled by the render or map manager, and transparent to all other code.

Personally I would like it if grids could define their own tile size.

There is no problem with grids providing their own constants for converting between world units and tile indices, because there will be code that needs to convert between the two. The main point is that code needs to not make the assumption that 1 tile = 0.5wu or 1 tile = 0.33wu. Or worse, what is happening in the codebase right now, with code assuming 1wu = 32px.


Need Rest API to get the availability zones information on the basis of Object selected in a particular region

This is a question/issue transferred from https://github.com/MicrosoftDocs/feedback/issues/544, please help whether there's an answer for that.
Is your feature request related to a problem? Please describe.
It would be better if we can get the availability zones available in a particular region based on object selected.
Right now it is mentioned that only below regions support the availability zones:
Central US
France Central
East US 2 (Preview)
West Europe
Southeast Asia (Preview)
If we can get the information using Rest API that weather selected region supports the availability zone or not? If it does, how many zones are there.
Basically to get the supported availability zones based on region and selected resource.
For example:
While creating the VM using Portal, if we select the region as Central US which supports availability zones, it does enable the Availability Zone option in Portal UI.
And If user selects the Size as "Standard_DS3" which supports only Zone "3".
Similar support required from Rest API, is there a way we can get the supported list of Availability Zone(s) based on Region, Resource etc.
Describe the solution you'd like
Need a Get API call to get the list of Supported Availability Zones based on Region and Resource selected.
Describe alternatives you've considered
NA
Additional context
NA

@surajmuthreja to get update here.

@hellosnow Is there a particular resource (Compute/Network, etc) for which you'd like to get the AvailabilityZones?

@dsgouda  Ideally I am looking for all the resources which will be there as part of VM/Instance creation on Azure.
E.g. We would need to provide this zone information for Public IP as well, instance size, region (wherever it is supported).

@hyonholee Would you be able to take a look at this or suggest someone who can?

Hi @hellosnow  @dsgouda ,
Can we please have some update on this?

@surajmuthreja please contact @hyonholee directly, he handles the Compute SDKs

@hellosnow , @dsgouda , @hyonholee  Any update on this request.

Currently, a user can use Get-AzComputeResourceSku to get the result.  For example, the following will show zones for disks.
Get-AzComputeResourceSku | ? {$.LocationInfo[0].Zones -ne $null -and $.ResourceType -eq 'disks' }
But, the request is for more convenient API that shows zones based on input.  We will discuss regarding the suggested API.

There is an API call to get this info
https://docs.microsoft.com/en-us/rest/api/compute/resourceskus/list
Returns a huge response that you can filter and look for the zones allowed per resorce or region
{             "resourceType": "disks",             "name": "Premium_LRS",             "tier": "Premium",             "locations": [                 "southeastasia"             ],             "locationInfo": [                 {                     "location": "southeastasia",                     "zones": [                         "3",                         "2",                         "1"                     ]                 }             ],             "restrictions": []         },         ...
    ` {
        "resourceType": "virtualMachines",
        "name": "Standard_F8",
        "tier": "Standard",
        "size": "F8",
        "family": "standardFFamily",
        "locations": [
            "eastus2"
        ],
        "locationInfo": [
            {
                "location": "eastus2",
                "zones": [
                    "1",
                    "2",
                    "3"
                ]
            }
        ],
        ...`

Thanks Daniel!
This is really huge response and needs filtration. But for the time being we can live with it I guess.
It will be more convenient if we can fulfill the original request also.
Thanks in advance!
Thanks,
Suraj Muthreja.
From: Daniel Estévez<EMAIL_ADDRESS>Sent: Friday, December 21, 2018 3:33 AM
To: Azure/azure-rest-api-specs<EMAIL_ADDRESS>Cc: Suraj Muthreja<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-rest-api-specs] Need Rest API to get the availability zones information on the basis of Object selected in a particular region (#3594)
There is an API call to get this info
https://docs.microsoft.com/en-us/rest/api/compute/resourceskus/list
Returns a huge response that you can filter and look for the zones allowed per resorce or region
{ "resourceType": "disks", "name": "Premium_LRS", "tier": "Premium", "locations": [ "southeastasia" ], "locationInfo": [ { "location": "southeastasia", "zones": [ "3", "2", "1" ] } ], "restrictions": [] }, ...
` {

    "resourceType": "virtualMachines",

    "name": "Standard_F8",

    "tier": "Standard",

    "size": "F8",

    "family": "standardFFamily",

    "locations": [

        "eastus2"

    ],

    "locationInfo": [

        {

            "location": "eastus2",

            "zones": [

                "1",

                "2",

                "3"

            ]

        }

    ],

    ...`

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-rest-api-specs/issues/3594#issuecomment-449149685, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AlvCg6q45gnXd6CVXcApVo3LZ3YLat3sks5u7AkVgaJpZM4Vxh5G.

Cleaning up old issues from 2016 - 2018. Please reopen this issue if it is still a concern.

@Drewm3  -- This issue is still not fixed. Can you please reopen this and assigned to relevant team.
I could not reopen this issue.

@surajmuthreja , the SKUs API for VMs provides the zones where each VM size is available for any specific subscription. It seems like this was the original request which is resolved. Could you describe why you feel this is not resolved at this point?

@Drewm3  We are looking for APIs through which we should be able to get supported zones once user selects any Object.
Apart from VM size, there are other resources also where zone dependency is there. e.g. Public IP,

@surajmuthreja, as far as I know there is no API that will indicate if an arbitrary resource type and SKU is available in a specific zone. The reason for this is that most resources either support zones in a region or not. For example with managed disks or standard public IP addresses, they are supported in all zones if the region supports zones. This works for those resources because they are basically software constructs.
VM sizes are different because they are tied to physical hardware, and that physical hardware may not be available in all zones. So with VM sizes we need to have the specific API support to expose which zones support which sizes.
Please let me know if this answers your question, and if it does then I will close this issue.

hitting the same brick wall as @surajmuthreja
We are trying to deploy virtual machines (and every other related resource - disks, IPs) in an automated fashion and highly-available with the usage of Availability Zone.
I'm aware of Availability Sets but in this case, it's not enough - we are planning to use AS only if a region does not support AZ design.
So back to the problem -

first we need to get an API response if the region/location supports AZ design or not
if yes, we can expect 3 zones - well true for disks/public IPs but false for VM tiers

It would be great if there would be a simple API call that would take the following parameters (location, virtual machine tier) and return all available zones as an array

I have answered this question from the compute/VM case. We provide an API to determine exactly which VM SKUs are available in AZs in each region.
@armleads-azure, could you comment on the possibility of a generic API, possibly as part of list locations, which can indicate if the location supports AZs or not?

I realize this thread has been open for a while, but I want to share how we handle this in the Azure portal for Networking resources (e.g. Public IP Addresses).
The portal makes a call to the Providers API.
GET /subscriptions/<subId>/providers/Microsoft.Network?api-version=2020-06-01
In the response body you can find zoneMappings which have the display name for the region and the valid availability zones. A sample (trimmed) response is below so you can see an example:
{
  "id": "/subscriptions/<subId>/providers/Microsoft.Network",
  "namespace": "Microsoft.Network",
  "authorizations": [
...  ],
  "resourceTypes": [
    {
...
      "resourceType": "natGateways",
      "locations": [
        "West US",
...      ],
      "apiVersions": [
        "2020-11-01",
...      ],
      "zoneMappings": [
        {
          "location": "East US 2",
          "zones": [
            "3",
            "2",
            "1"
          ]
        },
        {
          "location": "Central US",
          "zones": [
            "3",
            "2",
            "1"
          ]
        },
...      ],
      "capabilities": "CrossResourceGroupResourceMove, CrossSubscriptionResourceMove"
    },
    {

It's been 3 years since this feedback was provided.
Is there still no API that one can call to programmatically determine for a given "Resource type and region" if Zone redundancy is supported?

It's been 3 years since this feedback was provided. Is there still no API that one can call to programmatically determine for a given "Resource type and region" if Zone redundancy is supported?

Agree. My use case is also very simple: "List Azure regions which support Availability Zones"
How does this docs page get its list? https://docs.microsoft.com/azure/availability-zones/az-overview#azure-regions-with-availability-zones

Hi,
I happen to have the same requirement where I would want to list the locations and corresponding availability zones for the VMs. I use the REST endpoint (https://docs.microsoft.com/en-us/rest/api/compute/resourceskus/list )as suggested by Daniel in one of the comments. However, I would like to seek your advice in interpreting the output which looks like this -
{'resourceType': 'virtualMachines', 'name': 'Standard_PB6s', 'tier': 'Standard', 'size': 'PB6s', 'family': 'standardPBSFamily', 'locations': ['westeurope'], 'locationInfo': [{'location': 'westeurope', 'zones': ['2'], 'zoneDetails': []}], 'capabilities': [{'name': 'MaxResourceVolumeMB', 'value': '344064'}, {'name': 'OSVhdSizeMB', 'value': '1047552'}, {'name': 'vCPUs', 'value': '6'}, {'name': 'MemoryPreservingMaintenanceSupported', 'value': 'False'}, {'name': 'HyperVGenerations', 'value': 'V1,V2'}, {'name': 'MemoryGB', 'value': '112'}, {'name': 'MaxDataDiskCount', 'value': '12'}, {'name': 'CpuArchitectureType', 'value': 'x64'}, {'name': 'LowPriorityCapable', 'value': 'False'}, {'name': 'PremiumIO', 'value': 'True'}, {'name': 'VMDeploymentTypes', 'value': 'IaaS'}, {'name': 'EphemeralOSDiskSupported', 'value': 'False'}, {'name': 'EncryptionAtHostSupported', 'value': 'False'}, {'name': 'CapacityReservationSupported', 'value': 'False'}, {'name': 'AcceleratedNetworkingEnabled', 'value': 'True'}, {'name': 'RdmaEnabled', 'value': 'False'}, {'name': 'MaxNetworkInterfaces', 'value': '1'}], 'restrictions': [{'type': 'Location', 'values': ['westeurope'], 'restrictionInfo': {'locations': ['westeurope']}, 'reasonCode': 'NotAvailableForSubscription'}, {'type': 'Zone', 'values': ['westeurope'], 'restrictionInfo': {'locations': ['westeurope'], 'zones': ['1', '2', '3']}, 'reasonCode': 'NotAvailableForSubscription'}]},
As I understand it, the information provided in this line 'locationInfo': [{'location': 'westeurope', 'zones': ['2'], 'zoneDetails': []}], indicates that the VM type resources of StandardPBSFamily can only be deployed in a single zone (2). Is this interpretation right?
What does the last line  'restrictionInfo': {'locations': ['westeurope'], 'zones': ['1', '2', '3']}, 'reasonCode': 'NotAvailableForSubscription'}] indicate?


Tcl App: unknown color name "$colors(-selfg)"

Dark Theme on Mac (macOS Sonoma), getting error unknown color name "$colors(-selfg)" when selecting row in a table view. App written in Tcl.

The following might fix it in ...sv_ttk/theme/dark.tcl on line 441:
#ttk::style map Treeview -background {selected "#292929"} -foreground {selected "#ffffff"} or better yet
ttk::style map Treeview -background {selected "#292929"} -foreground [subst {selected $colors(-selfg)}]

Hi, sorry for the delay. I didn't notice this bug unfortunately, because although it was broken on my Linux machine either, it did so silently, and have never thrown an error because of the invalid color string. Fortunately the fix was as simple as replacing the curly braces around the string with double quotes.
Also, this issue didn't affect the sv-ttk Python package, as it's currently not up-to-date with the main branch.
Anyway, it's been fixed now with bc81227

Great. THANK YOU!
On Sun, Nov 5, 2023 at 8:23 PM Benedek Dévényi @.***>
wrote:

Closed #122 https://github.com/rdbende/Sun-Valley-ttk-theme/issues/122
as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/rdbende/Sun-Valley-ttk-theme/issues/122#event-10865647478,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIWLMM4XEI7SMCLYP67NQDYC7RSJAVCNFSM6AAAAAA6GYDRBOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHA3DKNRUG42DOOA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


limit to 2 arguments for function OR

Why in the doc here is not said that function OR has limitation of max 2 arguments:
@or(True,False,not(False))
function 'or' does not accept 3 argument(s)
in your doc is seems that there is no such a limitation
or(, , ...)
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 76e31411-f1a2-710b-94c0-39acf9e7314b
Version Independent ID: d5cefffe-dee0-2d2e-f3cb-8eefd5d40d17
Content: Expression and functions in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/control-flow-expression-language-functions.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

@Rutikhal, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@Rutikhal Since you can nest "OR" functions any number of times, it essentially means that you can have any number of arguments, just that at one instance of the function call, there need to be two parameters passed at max.
If you were to use OR with more than 2 inputs, you can nest the OR function as shown below :
@or(or(false, false), true)
Hope this helps.

@Rutikhal I will now proceed to close this thread. However, feel free to still comment and tag me if you have any queries.


feat(caller): watch queue status changes

I have added watching the "status" event for checking if the prompt is still in the queue, and calling onFailed if not.
I have also by necessity refactored the event handling of run(), it now relies on promises, this had the side effect of removing the delay calls, causing noticeable speedup at job startup.
I've also added custom error classes for call wrapper, so the library's end users can distinguish between failed jobs' errors if need be. It should resolve issue #9.

Added an execution_interrupted handler to handleJobExecution and the appropriate key to TComfyAPIEventMap, and a new Error class. Execution interruption didn't seem to trigger either execution_error or the status event, so it should further increase the coverage of ways prompts can leave.

Import styling fix.

lgtm🙌


Commcell object names - letter case

Object names like for example agent and backupset names are returned in lowercase.

Since a Commcell cannot have clients with same names, or backupset with the same name of different case, we chose to only use lowercase, to avoid the complexities, and ease of understand-ability.


[Proposal] New environments

Proposal
I would like to know what's the policy/procedure to add new environments or to be referenced in the third-party environments page. The page says to reach out on Discord, but the link is broken (it says "invite invalid").
I made this collection of gridworlds based on Gymnasium. The environments are fully discrete and are a nice easy benchmark (even though some are more challenging) before moving to pixel-based or continuous environments. AFAIK, there are no popular gridworld implementations, and MiniGrid does not have native spaces.Discrete observations (so, to use tabular methods once have to implement some workarounds).
These gridworld environments have been used in AAMAS 2024 and NeurIPS 2024 papers, and I plan to use for future work. I would really like to have them either integrated in Gymnasium, or referenced in the third-party environment page.
Motivation
No common gridworld repository exists, AFAIK. Mine has been used in AAMAS and NeurIPS paper, and I'd like to make it more popular.
Pitch
Have it referenced in the third-party page, or make it integrated by default in Gymnasium.
Alternatives
No response
Additional context
No response
Checklist

[X] I have checked that there is no similar issue in the repo

We would be happy to include your project within the third-party page, just make a PR adding the project.


Deprecate dictionary.of list

Addresses #2297
The initial reasoning was when I compared performance of those snippets of code:
let inline ofList l =
    // original code in compiler
    let dict = new System.Collections.Generic.Dictionary<_,_>(List.length l, HashIdentity.Structural)
    l |> List.iter (fun (k,v) -> dict.Add(k,v))
    dict

let inline ofSeqWithKnownCount (n: int) s  =
    let dict = new System.Collections.Generic.Dictionary<_,_>(n, HashIdentity.Structural)
    for kvp: KeyValuePair<_,_> in s do
      dict.Add(kvp.Key, kvp.Value)
    dict

let makeDict1 (n:int) =
  [for i in 0 .. n do yield i,i ] |> ofList

let makeDict2 (n: int) =
  // winner
  let dict = Dictionary<_,_>(n)
  for i in 0 .. n do
    dict.Add(i,i)
  dict

let makeDict3 (n: int) =
  seq { for i in 0 .. n do yield KeyValuePair<_,_>(i,i) } |> ofSeqWithKnownCount n

let makeDict4 (n: int) =
  [ for i in 0 .. n do yield KeyValuePair<_,_>(i,i) ] |> ofSeqWithKnownCount n

let makeDict5 (n: int) =
  [| for i in 0 .. n do yield KeyValuePair<_,_>(i,i) |] |> ofSeqWithKnownCount n

I compared with populating dictionaries of same size I've noted from evaluating a small set of scripts.
Using FSI and #time, the procedural code doesn't generate much garbage and ran about 10x faster of worst one, but this is totally out of context of the compiler workload.
I think it is good to get rid of leaky abstraction:

call to List.length, some lists were about 1k elements in my test
procedural code replacing the old code isn't much longer or less readable
with such helpers available, there is opportunity for getting into pitfalls in new places

The end result of those changes is basically 0% performance change, I've noticed 0.3% variation between master and this (mostly in favor of master) but this was on underpowered laptop (i7 2.1ghz), if you have feeling there is opportunity to do better measurement or think there is a risk with proposed changes, please let me know.

The helper used HashIdentity.Structural for the comparer, but the new code doesn't. Is that an issue?

@FunctionalFirst thanks for the review, it is my mistake and I'm going to fix it.

@smoothdeveloper is this still a live PR?
thanks
Kevin

@KevinRansom there is no visible performance gain in this PR, but the deprecated construct is quite wasteful (we now save a call to List.length + all allocations done to build the throwaway list).
I prefer the construct not be available in case it would be used in place where it could actually hurt the performance.
The only place I wonder if I made the correct choice is in src/fsharp/MethodCalls.fs where I pass paramVars.Length, but I think that default is fine (there shouldn't be more than few dozens of parameters).
I have no objection at discarding the PR if this doesn't meet requirements for visible improvement or other reason.

@dotnet-bot test this please

The CI error are getting myself concerned... What build commands should I issue to get a good grasp of what is going wrong locally?

@smoothdeveloper Look at the raw logs in each of the CI runs

@dsyme / @KevinRansom I fixed issues it is green on Jenkins, but AppVeyor is queued for a long time now.
Note that this PR contains #2296 that I closed.
Thanks for bearing with me.


[feature request] Add google earth API externs file (attached)

This issue was imported from Closure Compiler's previous home at http://closure-compiler.googlecode.com
The link to the original issue is:
https://blickly.github.io/closure-compiler-issues/#866

An externs file for the Google Earth API is available at: https://github.com/vicb/VisuGps3/blob/f2e563c2ff5d51091843e01fb7dac1071372cac7/src/vgps3/plugins/earth/externs/google_earth_api.js


Could the _dist_from_source method be made more efficient?

Either supplying an array of region central coords as well as an array of positions, or creating an inverse function of dist_from_region where a single position is passed and an array of regions?

That could be helpful for the BaseSource _generate_interloper_mask method efficiency drive.


 Or git reset --hard our master branch to upstream.
Otherwise the two repo's aren't really identical, at least not in terms of commits.


Adds reminder to check nightly builds before release day

Changes
Adds items to check the nightly builds at 7 days out and 1 day out as part of the timeline (lesson learned from the 1.2 release).
Fixes #84
/assign @dprotaso

/lgtm
/approve


Traversing example in Scala, Groovy, Kotlin

Add example of using this library API in Scala, Groovy, Kotlin as test or sample code or code snippet.
There is no required minimum, even one example from list is good to have:

iterators using DFS/BFS pre/post order
DFS pre-order and custom visitor
DFS post-order and custom visitor
BFS pre-oreder and custom visitor
BFS post-order and custom visitor

The description is intentionally vague to test existing documentation.
Feel free to open issues against documentation and samples.

@amatiushkin Would like to pick this up. Please confirm.
One query I will look to complete listed traversals with one of Scala/Kotlin. What is completion criteria for this issue ?

I'd be willing to take this on and implement using Scala

@stong2351 thanks for signing up! Let me know if you need a dedicated issue to track (or feel free to create one).

Thanks!

I would like to add in Groovy. Would like to check if it's not picked up yet.


Request arbitrary names for connections

SQL Operations Studio Version: Version 0.27.0-insider (0.27.0-insider)

Steps to Reproduce:
Connections in the servers tab show up by host name or IP address. If you have servers named with some algorithmic number or IP address its difficult to remember which is which without referring to documentation. It would be really useful if we could set an arbitrary label for a server. This, for example, is how Navicat works. It has a Connection Name field which defaults to the host name but can be customized and appears in the list of saved connections.
Servers in sqlops

Servers View in Navicat

Connection Editor in Navicat

this would be useful not only for IP addresses, but for long fqdn's too. For example, a url to an Amazon RDS server can get way too long, and may be meaningless unless you've got a friendly name option available.

I also miss this. It helps a lot when you have many connections and you need find the right one for your job.

This would be very usefull. I have many connections, so its very dificult to remember all of them

Also leaving a mark as I am in dire need of this functionality.
I have lot's of SQL Servers that I am currently dealing with that have non-logical names so an option to rename the connection to some user defined alias would be greatly appreciated.

This appears to be a duplicate of #66.


Browser version of Safari User Agents

Consider the following Safari User Agent strings:
Standard
Mozilla/5.0 (Macintosh; Intel Mac OS X 13_0_1) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.1 Safari/605.1.15
iPhone
Mozilla/5.0 (iPhone; CPU iPhone OS 16_1_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.1 Mobile/15E148 Safari/604.1
The current implementation returns 605.1.15 (or 604.1) as the version of the User Agent. I would argue that in the case of Safari the version 16.1 would be the correct value (in both cases). For example the MDN Web Docs and Can I Use work with this versioning.
I guess this would require to change this line from CreateDefaultBrowserRegex("Safari"), "Safari" } to CreateDefaultBrowserRegex("Version"), "Safari" }.
What do you think?

Thanks for pointing out the (new?) version behavior.
I'll have a look at it the days and what effects it has (also for consumers of the lib).

As note https://regex101.com/r/gRLirS/1 cc @gfoidl

Small update: I've been looking into this and I'm still looking for a way to efficiently evaluate both options.

Thanks for the update and for your work!

Looking at AppleWebKit version is not very useful, because this part of user agent is frozen from Safari ±11.1 and all newer desktop versions report the same "605.1.15" version.

Yes, this is the case for privacy reasons.
I've been researching this issue on and off, and haven't come to a conclusion on what the best way is.
Therefore I have not posted a status.

What problems do you see with reporting "Version" part of UA as Version other than this being possibly a breaking change? The Version string seems to be consistently reported from Safari 3.


Debug change detection in images swiper

Seems that it runs too often after some quick tests. It shouldn't. Should it?

Probably it's Swiper.js running outside of Zone.js
https://angular.io/guide/change-detection


Java >= 9 compatibility

Currently, when executing the plugin in an JDK 9 environment, it fails to unpack the neo4j archive properly:
[INFO] Extracting /var/folders/q2/5yhk0jjx5pnd45xtjjbs90cc0000gn/T/neo4j-server-maven-plugin/downloads/server/3.3.1/neo4j-server-unix.tar.gz
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.rauschig.jarchivelib.FileModeMapper$FileSystemPreferencesReflectionChmodCommand (file:~/.m2/repository/org/rauschig/jarchivelib/0.7.1/jarchivelib-0.7.1.jar) to method java.util.prefs.FileSystemPreferences.chmod(java.lang.String,int)
WARNING: Please consider reporting this to the maintainers of org.rauschig.jarchivelib.FileModeMapper$FileSystemPreferencesReflectionChmodCommand
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

# Followed by warnings for each file:
März 23, 2018 9:20:21 VORM. org.rauschig.jarchivelib.FileModeMapper$UnixPermissionMapper chmod
WARNUNG: Could not set file permissions of ~/projects/neo4j-server-maven-plugin/integration-tests/target/neo4j-server/neo4j-community-3.3.1. Exception was: null```

I am having similar issues with neo4j server. I am using neo4j to work with BloodHound. The server is running well but when I try to upload files to BloodHound it says unrecognized file format. When I see the log file for neo4j it is having following contents
nohup: ignoring input 2018-09-20 10:21:23.891+0000 WARN  You are using an unsupported version of the Java runtime. Please use Oracle(R) Java(TM) Runtime Environment 8, OpenJDK(TM) 8 or IBM J9. 2018-09-20 10:21:23.916+0000 INFO  ======== Neo4j 3.4.7 ======== 2018-09-20 10:21:23.969+0000 INFO  Starting... WARNING: An illegal reflective access operation has occurred WARNING: Illegal reflective access by org.neo4j.unsafe.impl.internal.dragons.UnsafeUtil (file:/home/shiva/BloodHound/neo4j-community-3.4.7/lib/neo4j-unsafe-3.4.7.jar) to constructor java.lang.String(char[],boolean) WARNING: Please consider reporting this to the maintainers of org.neo4j.unsafe.impl.internal.dragons.UnsafeUtil WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations WARNING: All illegal access operations will be denied in a future release

fixed with #10


Tutorial update to clarify that we use the letter O

PR Checklist
If you are contributing to Javis.jl, please make sure you are able to check off each item on this list:

[x] Did I update CHANGELOG.md with whatever changes/features I added with this PR?
[x] Did I make sure to only change the part of the file where I introduced a new change/feature?
[x] Did I cover all corner cases to be close to 100% test coverage (if applicable)?
[x] Did I properly add Javis dependencies to the Project.toml + set an upper bound of the dependency (if applicable)?
[x] Did I properly add test dependencies to the test directory (if applicable)?
[x] Did I check relevant tutorials that may be affected by changes in this PR?
[x] Did I clearly articulate why this PR was made the way it was and how it was made?

Link to relevant issue(s)
Closes #332
How did you address these issues with this PR? What methods did you use?
The problem was still that O can be confused with 0 easily. Now it should be very clear due to both the word letter as well as mentioning that it stands for origin.

@TheCedarPrince maybe you have an idea on why the format task is failing. When I run it on my machine it failed the first time but passed the second without changing the file which is weird.


[Bug]: Inconsistent error code when user doesn't exist

What happened?
Corso attempts to verify if a users exists by querying the users mailbox and drive. It seems that we're getting inconsistent results from the query. Requests to one datacenter are returning a user not found error while requests to another datacenter are returning an invalid user error
Corso should handle both errors and report the user as unavailable for backup
Corso Version?
all
Where are you running Corso?
N/A
Relevant log output
No response

After some digging around it seems like there's a few odd things with the current code and test case. The failing test seems to imply we shouldn't be returning an error. The code for getting users is setup to return an error saying the user doesn't exist in the tenant. Previously we were matching the case where we check if the mail folder is found (thus avoiding the if-block) and just marking the user as not having an inbox and continuing. With the new error we hit the mail folder if-block and return the error causing the test to fail
After discussion we've decided that we should actually return an error if the user doesn't exist


pipx run -> _download_and_run: error branch can never execute properly

Describe the bug
In commands.py, in function run():  run() calls _download_and_run() with the variable package_or_url where _download_and_run() is expecting argument package.
The result is that if the if branch executes for this statement in _download_and_run():
    if not (venv.bin_path / app).exists():

_download_and_run() will try and execute venv.get_venv_metadata_for_package(package).
If the package argument to _download_and_run() is in fact a url (as it could be if run() is called with a --spec argument) then venv.get_venv_metadata_for_package(package) will fail because it is expecting a package name and not a url.
How to reproduce
One would need to call pipx run with a --spec argument with a url and a non-existent app.  For example by modifying an example from the pipx makefile:
pipx run --spec=git+https://github.com/cs01/nox.git@5ea70723e9e6 notnox
Unexpected behavior
Because notnox is not an app in the package, the error if statement will execute.  However venv_metadata_inspector.py will find no package named git+https://github.com/cs01/nox.git@5ea70723e9e6 and so will return empty information about the venv.  Thus the printed error message has no apps to show as available or associated with the package in the venv.

After mergin #222, I cannot reproduce the bug (pipx works fine now.)


Diff?

So where is the patch generator method aka diff? Seems like this library does only half of the process.

Hm don't see a diff method in the spec.


How to give brigade-worker more compute

I am using dind to build my projects. As it takes a lot of time for my project to build, I was thinking I can either set a a node affinity or make brigade.js ask for more resources for the dind deployment. But I was not able to see where I can do these changes. Is this feature supported? If yes can someone tell me where I should do this change?

@iamnewspecies there currently is a way to set resource requests on a given Job pod, though it looks like some documentation is missing.  On that note, check out following PR that both documents use of said resourceRequests as well as adding/documenting the ability to set resource limits: https://github.com/Azure/brigade/pull/693

Looks like this was merged less than a day ago. I have to build my image from master and put it in some registry.

We did the above mentioned change. But it is not quite helping. We also want to add "Taints and Tolerations" so that we can have a dedicated machine. Is there a way to mention this somewhere?

Found the solution. Jobhost and #693 helped.


Add page insets to CarouselGridLayout

Enables carousel collection views to be edge-to-edge in a future revision
This is currently a breaking change that will be rectified as a byproduct of the forthcoming refactor work

Test approved. breaking the app only on carousels in the expected way


minimal ubuntu - game changer?

https://blog.ubuntu.com/2018/07/09/minimal-ubuntu-released
Does this make this project obsolete?

Definitely not.  This project has a ton of features, such as an init process, daemons, syslog and cron
http://phusion.github.io/baseimage-docker/
^ Start reading here.  If you are using baseimage-docker like a regular docker image that hosts one process, you might be missing out on a lot of niceties that make it unique.
In fact:

Using Minimal Images from Dockerhub
On Dockerhub, the new Ubuntu 18.04 LTS image is now the new Minimal Ubuntu 18.04 image. Launching a Docker  instance with docker run ubuntu:18.04  therefore launches a Docker instance with the latest Minimal Ubuntu.

https://github.com/phusion/baseimage-docker/blob/master/image/Dockerfile#L1

FROM ubuntu:18.04

...the latest baseimage-docker, released 9 days ago, is already built atop this new minimal image.

...the latest baseimage-docker, released 9 days ago, is already built atop this new minimal image.

I'm thrilled you guys put this together and naturally accrue benefits from Canonical's improvements.


Add a function to coerce Cmd Never to Cmd msg

This is more a suggestion than a pull request but it seemed easiest to illustrate the idea by writing the code.
There should be an idiom for expressing the idea of a sink effect - i.e. one that isn't expected to emit any value when it is called. A possible example might be writing to the storage API. The obvious way to do this is to use a Cmd Never. But as things stand there isn't an idiomatic way to incorporate this into an application loop.
Implementing the idiom is trivial but might be surprising if it appeared naked in application code. It seems like it might be better to wrap it up in a core function to create a common idiom for these cases.

The strict (instead of lazy) semantics of Elm will hit you in the face as soon as you try to use the sink-function in your program. Hint: the Debug.crash call there is not a good idea.

Actually, as it is, it won't even type-check. It was written in haste at work while waiting for something to compile.
How about...
map (\_ -> Debug.crash "Cmd Never passed to Cmd.sink resulted in a value")

?

Actually, as it is, it won't even type-check.

Oh, of course it would have type-checked. The type of Debug.crash is String -> a, polymorphic in a, so it would have been instantiable to an arbitrary function type as return type well enough.

How about...

Yes, that's better in that it won't immediately crash. Whether it does exactly what you want depends on the definition of Cmd.map, which I haven't looked into.

Yep, should work.

What do you think about adding absurd : Never -> a (#593), and solving this with map absurd? It seems safer than making the user write a Debug.crash call.

I'd rather not add something like this. There is no motivating example. No reason to add it then.

I added my motivating example to #593.


Error in latest version (v1.3.0)

After an update to this action earlier today (v1.3.0) we started getting the following error in our builds:
+ install commenter-linux-amd64 /usr/local/bin/commenter
commenter-linux-amd64: OK
+ '[' -n /github/workspace ']'
+ cd /github/workspace
+ '[' -n '' ']'
+ TFSEC_FORMAT_OPTION=json
+ TFSEC_OUT_OPTION=results.json
+ '[' -n '' ']'
+ tfsec --out=results.json --format=json --soft-fail '' .
Usage:
  tfsec [directory] [flags]

...

Error: accepts between 0 and 1 arg(s), received 2

For now we've version locked to v1.2.0 which works fine.

Pinning to v1.2.0 was a workaround for us as well.


setFragment is missing

as there is setQuery and setSearch, but missing setFragment in the fragment plugin. it is useful when I want to change the value of a fragment, while addFragment can't do it. At the moment, I need to removeFragment and then addFragment to work like setFragment.

are you looking for uri.fragment('new-hash')?

Not exactly. I am looking for setFragment of the Fragment plugin. It woild setting a data in fragment but not setting the whole fragment.

https://www.domain.com/#?a=1
uri.setFragment("a", 2)
the result will be
https://www.domain.com/#?a=2


chore(csp): enable google doubleclick in img-src

Issue
Fixes - https://jira.sdlc.merative.com/browse/MERATIVE-869
Description
Changed

This PR updates both CSPs on Franklin to allow for googleads.g.doubleclick.net and fix *.bing.com load error in the console.

Design Specs

Figma Link - N/A

Test URLs

Before (Changes from main): https://main--merative2--hlxsites.hlx.page/
After (Changes from this PR): https://chore-csp-869--merative2--proeung.hlx.page

Testing Instruction

Not sure how to test this as the script looks to be loaded just for the LIVE domain name (https://www.merative.com/thank-you). Perhaps, we need to adjust the domain listing via Google Ads to point to (https://chore-csp-869--merative2--proeung.hlx.page/thank-you) to ensure that CSP is not blocking anything? @scottbridgeman Let me know what you think.

@proeung  - When I tried to the form, https://chore-csp-869--merative2--proeung.hlx.page/contact, it passed me to the production server and blocked the event. We may have to update the CSP on the live server and re-test. Unless you have other ideas on how we might be able to test this. Totally trust your expertise here :-)

@scottbridgeman I don't know if something got changed, but I'm no longer seeing doubleclick error in the console log here (https://chore-csp-869--merative2--proeung.hlx.page/thank-you). I think we can try and merge the changes and see if anything else comes up on the main/live domain name side.

Thank you, @proeung - I will monitor Google Analytics to see if any events are firing/being captured.


avoid get_export every time when guest code call into host function.

Wasmer has a WasmerEnv design to lazy init ctx when initialize instance. Is it possible to add this feature?
This approach can avoid lookup realloc every time when guest code call into host function.
For example
      let func = get_func(&mut caller, "canonical_abi_realloc")?;
      let func_canonical_abi_realloc = func.typed::<(i32, i32, i32, i32), i32, _>(&caller)?;
      let memory = &get_memory(&mut caller, "memory")?;

could be
let lazy_ctx = get_lazy_ctx(caller)?; // get_func/get_memory only be called once when linking
let func_canonical_abi_realloc = lazy_ctx.func_canonical_abi_realloc;
let memory = lazy_ctx.memory;

Thanks for the report!
FWIW the codegen here is still, uh, very early-days. AFAIK nothing's actually using this in production yet.
Have you tested this out locally and seen issues with the get_func calls in profiles?

I'm going to close this for now since AFAIK we don't have a concrete reason to not do this, but if it's an issue please feel free to reopen or submit a new issue!


New text field is being displayed above the menu

New text field is displayed above the menu
Description
New text field is being displayed above the menu (while adding it) whereas other tools insert object behind menu. Afterwards, it works the same as other tools.
Steps to reproduce

start a new text field before a menu
insert a long text that overlaps menu

Screenshots
text tool:

other tools (eg. free draw):

Possible solution

maybe higher z-index of the menu?

I think we'd have to render the textarea as a sibling of the UI.


Problems with the test_ratio

Thanks for your very nice code.
I have a doubt, is setting the test_ratio to 0.9 too large?

No, the lower number of training data would illustrate the efficiency of the method while having a low number of training data. Most DCNNs and ViTs require a significant number of reference (ground truth) data.

Thanks for your reply!


Update guava from 33.2.1-jre to 33.3.0-jre

Update guava to 33.3.0-jre
Revert some formatting
Set test env locale to "C"

CI always fails in ClientServerTests, but it looks like main isn't building clean, too.

https://github.com/com-lihaoyi/mill/issues/3387


chore: remove AllowUpdateAfterProposal

Description

closes: #1906

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

I think this PR needs to be re-opened. I'm guessing you branched off of the 02-client branch before it was merged to main? If we merge this in the git history on main will have an additional 72 commits.

I think this PR needs to be re-opened. I'm guessing you branched off of the 02-client branch before it was merged to main? If we merge this in the git history on main will have an additional 72 commits.

the commits get squished tho don't they?

I think this PR needs to be re-opened. I'm guessing you branched off of the 02-client branch before it was merged to main? If we merge this in the git history on main will have an additional 72 commits.

the commits get squished tho don't they?

Yeah they will. I think the commit would just have lots of co-authors and the repo will count all the commits into the contributions again. I think it's still cleaner to branch off of main

I think this PR needs to be re-opened. I'm guessing you branched off of the 02-client branch before it was merged to main? If we merge this in the git history on main will have an additional 72 commits.

the commits get squished tho don't they?

Yeah they will. I think the commit would just have lots of co-authors and the repo will count all the commits into the contributions again. I think it's still cleaner to branch off of main

I think this PR needs to be re-opened. I'm guessing you branched off of the 02-client branch before it was merged to main? If we merge this in the git history on main will have an additional 72 commits.

the commits get squished tho don't they?

Yeah they will. I think the commit would just have lots of co-authors and the repo will count all the commits into the contributions again. I think it's still cleaner to branch off of main

i dont think squishing commits retains co-authors/commits unless you specifically leave them in there (don't delete the coauthors in the commit comment)

Yeah they will. I think the commit would just have lots of co-authors and the repo will count all the commits into the contributions again. I think it's still cleaner to branch off of main

i dont think squishing commits retains co-authors/commits unless you specifically leave them in there (don't delete the coauthors in the commit comment)

That might work, but I am not sure.
I would prefer to have a cleaner git history but no stress.

Agreed clean git histories are nice, I would likely reopen the pr if it was mine,  I think the consequences will just be how 02-client was merged in https://github.com/cosmos/ibc-go/commit/f57170b1d4dd202a3c6c1c61dcf302b6a9546405. Happy to let @charleenfei decide what do here

i'm going to try to merge and squish and see if i can apply the strategy i think will work here with regards to the commit history... 😅

i'm going to try to merge and squish and see if i can apply the strategy i think will work here with regards to the commit history... sweat_smile

It worked. Nice one!


DARKMODE_[Feature]:

Describe the feature
Adding darkmode feature to our current website can make our website to look more good and can provide better user friendly experience @trishanu-init assign this issue to me I will add this feature .
Add ScreenShots
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

assign-me

@Meetjain1 Follow the issue template before creating an issue

@Meetjain1 Follow the issue template before creating an issue

I just removed the non-required field.

can you assign me this issue under gssoc 24.

/assign-me

/assign-me
@Ujjwal-2910
You were not asked to write this ok !
Why are u doing these type of cheap practices ??

@Meetjain1  First create PR's for the issues assigned to you , don't spam by claiming multiple issues.


Net::HTTP instrumentation enhancements

Currently we only patch the request method, but it would be good to add visibility around connect.
It would also be nice to be able to allow / deny list functionality for context propagation. This would likely be something we'd like to apply across the board to other instrumentation if we go this route.

Connect method instrumentation support was added as part of https://github.com/open-telemetry/opentelemetry-ruby/pull/608


Enable a MathML option in the KaTeX demo.

I think this may enable a MathML option in the demo page. I'll have to see a deployed version to check.
Addresses #2221.

You should also add output to katexOptions:
https://github.com/KaTeX/KaTeX/blob/903b459ca42f6689fa960bc3bc1d38b16b57dbed/website/static/js/index.js#L50-L51
You can run the website locally by cd website && yarn start.

You should also add output to katexOptions:

Done.

You can run the website locally by cd website && yarn start.

When I try that, I get an error message:

'docusaurus-start' is not recognized as an internal or external command, operable program or batch file.

Oh, I see. First I need to yarn install from the website subdirectory. It works! Thank you.


Make address mandatory for professors and optional for other roles

UG not yet updated

I think you can fix the line length issues as reported by the Java CI failing tests.