Prevent user groups creation

"In our experience, user education helps ensure responsible Teams usage".
Yeah. I'm managing Office 365 and Teams on a secondary school. It's not even possible to get the teachers educated on correct usage, let alone the students.
Teams Education will be a lot more adoptable once Microsoft acknowledges that.

Documentdetails
⚠ Dit gedeelte niet bewerken. Het is vereist om problemen te koppelen tussen docs.microsoft.com en GitHub.

ID: 815d1ce1-cb68-25b5-6a59-435461559495
Version Independent ID: 1f6ac386-4ca4-1cde-bae0-ed6df8c94d61
Content: Microsoft Education governance Veelgestelde vragen voor IT-professionals - Microsoft-Teams
Content Source: Teams/plan-teams-governance-edu.md
Service: msteams
GitHub Login: @rmw2890
Microsoft Alias: MyAdvisor

@rboterenbrood Thank you for this feedback
I will get this informaiton over to the Teams/Skype for Business team for investigation and update.
I was able to get this into the backlog for the Teams/Skype for Business team. The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Thank you for this feedback and sorry for the delay in response. This was passed to the team handling education and I heard this discussed in meeting back when it was brought up. If you are stilling seeing issues that can be addressed in the content please go ahead and re-open.


TypeReferenceTypeDescriptor should only be used for references of type definitions

Currently, a TypeReferenceTypeDescriptor applies to any named type. But, going by the spec definitions, a type reference is only applicable for identifiers which refer to a type definition. The issue here is that we don't have a way to identify types that were defined using a type definition.

With the improvements done to type refs in the previous releases, this seems to have been addressed. Hence closing this.


feat: add string quote style options to YamlEmitOptions

gives greater flexibility in determining the default string quotes when values require it.
fixed a broken reference to char in EmitStringAnalyzer.Analyze()

👍 Makes sense, thanks.


Bug Report: Private DNS zone link in setting.connectivity.tf

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform:  1.0.10
azure provider: 2.82.0
module:   1.0
Description
In setting.connectivity.tf file the Private DNS zone are linked to the VNET's even when the bellow boolean's were false. this happens for the hub vnet and for the spoke vnet
      enable_private_dns_zone_virtual_network_link_on_hubs   = false
      enable_private_dns_zone_virtual_network_link_on_spokes = false

Steps to Reproduce

Create hub vnet and attach the spoke vnet (spoke_virtual_network_resource_ids)  in the setting.connectivity.tf

put the following values to false:
enable_private_dns_zone_virtual_network_link_on_hubs   = false
enable_private_dns_zone_virtual_network_link_on_spokes = false

in the TF plan the link between VNET and private DNS zone is created.

Looks like the root cause of this was down to selecting the wrong scope when determining whether these resources should be managed_by_module.
We were pulling the managed_by_module value from the underlying azurerm_private_dns_zone configuration and not considering the individual azurerm_private_dns_zone_virtual_network_link configuration.
Have updated the code to consider both as part of determining whether to create the azurerm_private_dns_zone_virtual_network_link resource or not.


Change configs while running

Commands to set config values while the bot is running, as well as to reload the backup list and main config.

Easy to do, no problem.

+1 :joy:


Working with multiple subscriptions

Hi Team,
I'm trying to understand how to use this to implement terraform code with multiple subscriptions.
In plain terraform, we would have the following to define the aliases of our subscriptions, say Azure for argument's sake:

# Configuration for our "main" subscription
provider "azurerm" {
  tenant_id       = var.TENANT_ID_MAIN
  subscription_id = var.SUBSCRIPTION_ID_MAIN
  client_id       = var.SERVICE_PRINCIPAL_ID_MAIN
  client_secret   = var.SERVICE_PRINCIPAL_SECRET_MAIN
  features {}
}

# Configuration for the "secondary" subscription
provider "azurerm" {
  alias           = "secondary"
  tenant_id       = var.TENANT_ID_SECONDARY
  subscription_id = var.SUBSCRIPTION_ID_SECONDARY
  client_id       = var.SERVICE_PRINCIPAL_ID_SECONDARY
  client_secret   = var.SERVICE_PRINCIPAL_SECRET_SECONDARY
  features {}
}

However, since TerraformTasks expects a parameter called backendServiceArm, where we would have defined the service connection to the cloud provider, this is how we define them inside terraform:
provider "azurerm" {
  features {
  }
}

Hence, it's not clear to me how we define the already implemented secondary service connections in Azure DevOps for multiple subscriptions. Could you shed some light on how to implement this?

hi, if the service principal can access both subscriptions I think it should be enough to just specify the subscript_id and use backendServiceArm as normal.  Have you tried that?

I haven't. I'll try and get back to you.
In a scenario where that's not possible, how can we access multiple subscriptions using service connections only?

Well, this extension is basically just setting up the calls to terraform so how would you do it if you were running the commands manually in the agent?

just want to state i have this issue too
if i setup with the main subscription for my remote state backend and my tf files have this
provider "azurerm" {
  subscription_id = "xxxxxxxx-xxxx-xxxx-xxxx-5af85f7fb97c"
  features {}
}

provider "azurerm" {
  alias = "it-shared"
  subscription_id = "yyyyyyyy-yyyy-yyyy-yyyy-c00645d69e56"
  features {}
}

provider "azurerm" {
  alias = "ct-identity"
  subscription_id = "zzzzzzzz-zzzz-zzzz-zzzz-ecfc60d3ba7e"
  features {}
}

when the terraform plan attempts to run i get multiple errors like this:
Error: Error ensuring Resource Providers are registered.
│
│ Terraform automatically attempts to register the Resource Providers it supports to
│ ensure it's able to provision resources.
│
│ If you don't have permission to register Resource Providers you may wish to use the
│ "skip_provider_registration" flag in the Provider block to disable this functionality.
│
│ Please note that if you opt out of Resource Provider Registration and Terraform tries
│ to provision a resource from a Resource Provider which is unregistered, then the errors
│ may appear misleading - for example:
│
│ > API version 2019-XX-XX was not found for Microsoft.Foo
│
│ Could indicate either that the Resource Provider "Microsoft.Foo" requires registration,
│ but this could also indicate that this Azure Region doesn't support this API version.
│
│ More information on the "skip_provider_registration" flag can be found here:
│ https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs#skip_provider_registration
│
│ Original Error: populating Resource Provider cache: listing Resource Providers: loading the initial page: providers.ProvidersClient#List: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="AuthorizationFailed" Message="The client '<CLIENTGUID>' with object id '<CLIENTGUID>' does not have authorization to perform action 'Microsoft.Resources/subscriptions/providers/read' over scope '/subscriptions/yyyyyyyy-yyyy-yyyy-yyyy-c00645d69e56' or the scope is invalid. If access was recently granted, please refresh your credentials."
│
│   with module.core-connectivity.provider["registry.terraform.io/hashicorp/azurerm"].it-shared,
│   on modules/connectivity/main.tf line 21, in provider "azurerm":
│   21: provider "azurerm" {
│

once i re-read this as i pasted this here i realised my SPN needed permissions to the additional subs - applied the appropriate permissions to the subs manually and it succeeded just fine
what might be good is some documentation notes for this use case

Looks like this is not an issue with this extension.  Closing.


Trigger Dropdown/Focus with JavaScript

Is it possible to trigger the rendered dropdown with JavaScript?
I have tried several ways for trigger the visibility, but I guess there is one flag that won't be set. If I then click outside the dropdown, it won't be closed until I clicked one time on the rendered div-container.
Which event set the component state for example to isOpen, so that the outside click closed the dropdown automaticly?
The original -Tag doesn't triggered the dropdown's visibility as well.

Try updating to version 2.1.0, I've added new methods: show, hide, toggle

Thank you! I will try it today.
I found a typo in my last post. The last sentence should read: The original -Tag doesn't triggered the dropdown's visibility as well. It set only the focus to the rendered dropdown.

the new functions only trigger the visibility from the dropdown. However, the focus is not set on the input field. The focus is only on the input field when you click again. Is it possible to set the focus as well?

I've fixed it, please try version 2.1.1

Thank you very much. It works well.


Windows: Broken file output because of newline translation

Hi,
First off thank you for providing this demo program!
I've noticed that on Windows, it produces skewed images such as this one:

This is because your demo writes binary pixel values to stdout, which typically contain some 0x0A (Line Feed) bytes here and there.
Because Windows uses CR LF newlines, and stdout is usually in text mode (think fopen(,"w"), as opposed to fopen(,"wb")),
these 0x0A bytes will be replaced by a 0x0D 0x0A (CR LF) sequence during fwrite().
Consider writing images to a file instead of to the console.
Alternatively, you could use something like setmode(fileno(stdout), O_BINARY) on Windows to force stdout into binary mode.

Thanks for trying my demo program. And for your work on libschrift!
I had only tested my program on macOS and Linux, where they work fine. I never tried Windows.
There doesn't seem to be a portable way to put stdout in binary mode.
I'll probably end up writing to a binary file, which is sure to be portable.

It's fixed. Thanks for the report.


ci: add license check using github actions

Use Giithub for license checks.

DNM while the action is adapted and tested for the main tree.


If centerMode is true, no longer sets scrollToShow to 1.

Hoping to resolve https://github.com/kenwheeler/slick/issues/541, but would like to confirm that this doesn't affect any other settings.

@eschow, @kenwheeler... this fix only works if there are an even number of slides. If there are a number of slides that are not divisible by an odd number it does not behave as expected.
For example try this fix with 6 slides (works), and then try with 7 slides (broken). I'm trying to track down where this is failing, but in the meantime if someone else can figure it out IT WOULD HELP IMMENSELY. THANKS!!!!

From my understanding, centerMode only works with an odd number of slides because there's nothing to center in an even set of slides. Correct me if I'm wrong, @kenwheeler. That was just the assumption I made when I was using this library.

That seems very inconsistent and restrictive. When working with a variable amount of data this would be unacceptable. If centerMode is to work with slidesToMove it should work with both odd and even number of slides. You would have to assume that infinite was true, but if you scrolled (by 3) past the number of slides available, then it should go back to the beginning. ~ Thoughts?

Center mode works with any number

Thats not what I'm seeing, unless I'm missing something.
With @eschow fix and an odd number of slides, it breaks. And without the fix, no matter what number is set for slidesToScroll, it only jumps one. Try applying the proposed fix above, with slidesToScroll && slidesToShow = 3, and different numbers of slides. It seems to break when numberOfSlides is not divisible by slidesToScroll.

I'm saying in general, not this fix

Thanks again for the help and quick responses. True, in general. Although, when centerMode == true && slidesToScroll == 3, this does not work. Is that a valid bug that has been logged? If not, could we consider this a bug?
To that point, I know this fix above is an attempt to fix this, however all I'm pointing out is that this fix does not fix it for -n number of slides, which is what I would expect. I'll still be digging in too and see If I can find a solution, but please keep me informed as well....

@eschow BTW the title of this issue is totally misleading. There is no property "scrollToShow", it's either slidesToShow, or slidesToScroll. In your fix you remove _.options.slidesToScroll = 1; and then set the slidesToShow so I'm completely sure what you are trying to accomplish.
@kenwheeler To be clear, The issue I'm having is when centerMode == true && slidesToScroll == 3.

My apologies. Multitasking is not my strong suit and I filed it while working on something else.

On Jan 15, 2015, at 12:55 PM, Burton Podczerwinski<EMAIL_ADDRESS>wrote:
@eschow BTW the title of this issue is totally misleading. There is no property "scrollToShow", it's either slidesToShow, or slidesToScroll. In your fix you remove _.options.slidesToScroll = 1; and then set the slidesToShow so I'm completely sure what you are trying to accomplish.
@kenwheeler To be clear, The issue I'm having is when centerMode == true && slidesToScroll == 3.

Reply to this email directly or view it on GitHub.

This will partially fix the issue I'm seeing. If in centerMode, if the slideCount is divisible by the slidesToScroll property, allow it. Otherwise, set slidesToScroll = 1. However, the case where slideCount % slidesToScroll !== 0, is still not accounted for. <- which is what it breaking for me. Pull Request??

ok @befreestudios I'm gonna review this and try and get it in the next release. Been busy with the kid and the holidays, just getting back into the swing of things.

@kenwheeler, awesome. Thank you. I completely understand (have three three kids as well). Thanks again and let me know if you need anything else from me. Thx.

:+1:  on this fix.
When centerMode is enabled it appears as though slidesToScroll gets fixed to 1.

This is a bit of a mess... and very old... I'll close it and try to note down this issue for the upcoming version 2.


Cooking component class creation

Description
Task: Cooking Component Class
Feature: #9
The cooking component class will handle changing the state of an item and transforming it to a new item after being passed through a station. This will include constructors, simple variables and functions to handle changing the state of an item etc. Additionally, this class will inherit some functions from the station branch to assist in this implementation.
Example
Some of the methods and variables will be;

create:
update:

Dependencies

[x] Need to use item handling component to handle items (#3 )

Milestones
List of steps that need to be completed for this task.

[x] Create basic constructors (Aug. 25)
[x] Implement Item handling component (Aug. 26)
[x] Create functions to change state of item (Aug 27)
[x] Implement Testing (Aug. 27)
[x] Documentation Updated (Aug. 27)

Completion Deadline: Aug. 27
Documentation

Main description of feature
JavaDoc
...

Member

Willem Bor @willembor
Alexander Shomer @Ashom2

There seems to be quite an overlap with the CookingComponent and the CookIngredientComponent, could y'all rectify this by merging these functions so we have one function for handling cooking components? Doesn't seem like the cookingComponent is being used anyways.


HystrixDashboard - doesn't seem to display the right TPS

I am using the version 1.1.2.RELEASE of turbine stream and I think the dashboard has an issue. Let me know, if I am wrong or not.
Here is an example of the data that I got from the turbine stream
data: {"rollingCountFallbackFailure":0,"rollingCountFallbackSuccess":0,"propertyValue_circuitBreakerRequestVolumeThreshold":"200","propertyValue_circuitBreakerForceOpen":false,"propertyValue_metricsRollingStatisticalWindowInMilliseconds":"10000","latencyTotal_mean":55,"type":"HystrixCommand","rollingCountResponsesFromCache":0,"TypeAndName":"TypeAndName=>HystrixCommand_lcs.aurora-read","rollingCountTimeout":0,"propertyValue_executionIsolationStrategy":"THREAD","instanceId":"lcs:08fe0c944c1c:8080","rollingCountFailure":0,"rollingCountExceptionsThrown":0,"latencyExecute_mean":55,"isCircuitBreakerOpen":false,"errorCount":0,"group":"AuroraLcsCacheDao","rollingCountSemaphoreRejected":0,"latencyTotal":{"0":35,"25":52,"50":69,"75":69,"90":85,"95":87,"99":143,"99.5":143,"100":441},"requestCount":2597,"rollingCountCollapsedRequests":0,"rollingCountShortCircuited":0,"latencyExecute":{"0":35,"25":52,"50":68,"75":69,"90":85,"95":87,"99":144,"99.5":144,"100":441},"propertyValue_circuitBreakerSleepWindowInMilliseconds":"5000","currentConcurrentExecutionCount":1,"propertyValue_executionIsolationSemaphoreMaxConcurrentRequests":"10","errorPercentage":0,"rollingCountThreadPoolRejected":0,"propertyValue_circuitBreakerEnabled":true,"propertyValue_executionIsolationThreadInterruptOnTimeout":true,"propertyValue_requestCacheEnabled":true,"rollingCountFallbackRejection":0,"propertyValue_requestLogEnabled":true,"rollingCountSuccess":2612,"propertyValue_fallbackIsolationSemaphoreMaxConcurrentRequests":"50","InstanceKey":"InstanceKey=>lcs:08fe0c944c1c:8080","propertyValue_circuitBreakerErrorThresholdPercentage":"50","propertyValue_circuitBreakerForceClosed":false,"name":"lcs.aurora-read","reportingHosts":17,"propertyValue_executionIsolationThreadPoolKeyOverride":"null","propertyValue_executionIsolationThreadTimeoutInMilliseconds":"10000"}

Here is the extracted interesting data:
propertyValue_metricsRollingStatisticalWindowInMilliseconds: 10000
reportingHosts: 17 -- As it is described this will get summed across instances in a cluster
requestCount: 2597
If I am looking the code in the hystrixCommand.js file I can find the following method
function preProcessData(data) {
...
   // do math
   convertAllAvg(data);
   calcRatePerSecond(data);
}

The convertAllAvg method updates the propertyValue_metricsRollingStatisticalWindowInMilliseconds value in the following way
    // the following will break when it becomes a compound string if the property is dynamically changed
   convertAvg(data, "propertyValue_metricsRollingStatisticalWindowInMilliseconds", false);

....

    function convertAvg(data, key, decimal) {
    if (decimal) {
       data[key] = getInstanceAverage(data[key], data["reportingHosts"], decimal);
    } else {
       data[key] = getInstanceAverage(data[key], data["reportingHosts"], decimal);
    }
   }

After this call, the  propertyValue_metricsRollingStatisticalWindowInMilliseconds is set to Math.floor(10000 / 17) = 588
So now, the TPS is starting to be wrong.
function calcRatePerSecond(data) {
  var numberSeconds = data["propertyValue_metricsRollingStatisticalWindowInMilliseconds"] / 1000; (1)
  var totalRequests = data["requestCount"]; (2)
  if (totalRequests < 0) {
    totalRequests = 0;
  }
  data["ratePerSecond"] =  roundNumber(totalRequests / numberSeconds); (3)
  data["ratePerSecondPerHost"] =  roundNumber(totalRequests / numberSeconds / data["reportingHosts"]) (4)
}

(1)  numberSeconds = 588 / 1000 = 0.588
(2) totalRequests = 2597
(3) ratePerSecond = 2597 / 0.588 = 4416
(4) ratePerSecondPerHost = 2597 / 0.588 / 17 = 259
It should be
(1)  numberSeconds = 10000 / 1000 = 10
(2) totalRequests = 2597
(3) ratePerSecond = 2597 / 10 = 259.7
(4) ratePerSecondPerHost = 2597 / 10000 / 17 = 15.27
In conclusion we should remove the line
 convertAvg(data, "propertyValue_metricsRollingStatisticalWindowInMilliseconds", false);

Let me know if I am wrong.

Maybe you could check with the original source code in netflix/hystrix?

It is a gub. Confirmed on https://github.com/Netflix/Hystrix/issues/1251

Thanks for the update. We can attempt a merge when it is fixed in Hystrix I guess.

Netflix has merged the fix.

This fix introduces new issue. Turbine returns propertyValue_metricsRollingStatisticalWindowInMilliseconds (e.g. 10 000 per host) as a sum (170 000) for all hosts (17) which make no sense for this property. Before this fix this value was re-calculated as average value per host by Hystrix dashboard (170 000 / 17 = 10 000).
Calculation described in https://github.com/Netflix/Hystrix/issues/1251 looks now as following:
(1) numberSeconds = 170000 / 1000 = 170  // should be 10
(2) totalRequests = 2597
(3) ratePerSecond = 2597 / 170 = 15.27 // should be 259.7
(4) ratePerSecondPerHost = 2597 / 170 / 17 = 0.98 // should be 15.27
Looks like Hystrix Dashboard is fine now but we have to align Turbine module? Do not sum propertyValue_metricsRollingStatisticalWindowInMilliseconds ?

This is Turbine 1.0 and 2.0 incompatibility. See: https://github.com/Netflix/Turbine/issues/100

Which versions of hystrix and which versions of turbine (as spring cloud release) are not compatible?
What is the way forward, what libs version do i have to get rid of?

Please see: #1629. There is not easy fix. We have a custom patch for this.


fix typo

Describe the bug

To Reproduce
No response
Expected behavior
No response
Diagnostic report (REQUIRED)
No response
Screenshots and additional context (optional)
No response

Fixed for the next version.


Sketching support for TestStream on FnApiRunner

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

FnApiRunner manual: https://docs.google.com/document/d/18-AhNn1VG134ti1A7A__BtLQQg-99VmjRoNlcWg-8Qg/edit


Having trouble getting this working

First I just want to say that this is a fantastic idea and I'm really happy you developed the package. Pretty crazy that something like this isn't included by default in chartJS. I am unfortunately having some trouble getting it working within my application though.
I am loading your package via jsdelivr CDN, and using it along with chartJS within a basic jQuery document ready function. I am not a JS expert so I guess I could be setting something up wrong here, but it's not apparent to me what it is. My code seems to match your examples, at least for setting up the config parameter.
Here is what my chart currently looks like:

Here is my loading of your package & chart js in my base html file:
    <script
      type="text/javascript"
<EMAIL_ADDRESS>    ></script>
    <script
      type="text/javascript"
      src="https://cdn.jsdelivr.net/npm/chartjs-plugin-colorschemes"
    ></script>

And here is my .js file containing the chartJS and your package code. Note that this file is called at the very end of the body of my base HTML file.
$(function() {
  /// code goes here that sets up data variables below (cacc, rsvc, models)
  /// not posted to keep post short

  let ctx = document.getElementById("stacked_chartjs");
  let chart = new Chart(ctx, {
    type: "line",
    data: {
      labels: ["January", "February", "March", "April", "May", "June", "July", "August"
      ],
      datasets: [
        {
          label: "Corporate Access",
          data: cacc
        },
        {
          label: "Research Services",
          data: rsvc
        },
        {
          label: "Models & Content",
          data: models
        }
      ]
    },
    options: {
      plugins: {
        colorschemes: {
          scheme: "brewer.Paired12"
        }
      },
      maintainAspectRatio: true,
      responsive: true,
      tooltips: {
        mode: "index",
        intersect: false
      },
      hover: {
        mode: "index"
      },
      scales: {
        xAxes: [
          {
            scaleLabel: {
              display: true,
              labelString: "Month"
            }
          }
        ],
        yAxes: [
          {
            stacked: true,
            scaleLabel: {
              display: true,
              labelString: "Consumpion"
            }
          }
        ]
      }
    }
  });
});

The only other thing I thought could possibly be interfering is that I am using a custom Bootstrap theme, which has its own .JS file that contains chartJS (among other things), but I confirmed that it's using chartJS version 2.8.

It turns out that your plugin was indeed not working because of the .JS file from the custom bootstrap theme I mentioned. I have not dug into what specifically caused it yet, but the plugin started functioning once I commented out the reference to that file.
Going to close this, but I will report back my findings and post them here for anyone who runs into a similar issue.


docs: Add fal-specific README and reduce root README

Our builds are broken because ../../README is not valid.

Approved on a Huddle.


[feature request] Disable Autocompletion for specific target

Hi,
it would be great if you have an option to ignore some Std libraries packages in the AutoCompletion.
So you if you write an Javascript Applikation and write something like:
var h:File + (hit AutoCompletion) should only valid Javascript classes appear and no Flash classes.

Hi,
Cross platform projects need completion. Maybe we can set higher priority for completion item if it's "marked" for current target.
For example, if project target is JavaScript, then move completion for "js" package to the top of completion list.
What do you think about it?
But again, this is more like exception for JavaScript classes declaration, like "js.Browser" and "js.html.*" and other.
Or maybe we can change icon or add additional text to completion item, like "Browser (JavaScript)", or something like that to make items which can be used only using currently active target.
Also, it may be tricky to detect if class is exclusive for some target.

Hmm i thing the perfect solution is when you have an tab in the sdk or module window where you can activate or deactivate the haxe std folders so you can decide if you need php flash java and so on its the result should be the same if you delete e.g. std/php/. Perhapse you can drag and drop the items so you can set the Priority?!
I create a little mockup perhaps it helps

Good idea, perhaps I can implement it with checkbox list as you suggested.

:+1: Good Idea.


inotify is leaking a file descriptor

Reported by @alberts at https://codereview.appspot.com/8202043/#msg6

I noticed that TestFsnotifyDeleteWatchedDir is leaking a file descriptor:

strace -f -q -e inotify_init,open,close ./fsnotify.test
-test.cpu=1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1
-test.v -test.run=TestFsnotifyDeleteWatchedDir

> SIGABRT shows that all the readEvents goroutines (which are in charge of closing
the inotify fd) are still stuck in their read syscall.
> This seems like a problem with the current design of Watcher. readEvents is
blocked and can't see the closed done channel.

See #5 regarding blocking behaviour.

I don't know what I'm doing :-1:
strace: Can't stat './fsnotify.test': No such file or directory

GNU gdb (Ubuntu 7.7.1-0ubuntu5~14.04.2) 7.7.1
./fsnotify.test: No such file or directory.

go test -c


chore: step end time

Original calculation for end_time_for_step was not good because LAMBDA_ms is not constant which could make several steps move very quickly and then some steps taking a very long time.
New calculation only uses current LAMBDA to increase the time spent in the current step.
Reason for this:
loop_back_finish_state_ = elapsed_time_in_step > (int64_t)(2 * (LAMBDA_ms - POLLING_INTERVAL_ms));
being two times lambda is because this covers two steps since the delay is only added in second finish.
Besides this vote_accepting_steps is increased to 100 to prevent issues when nodes are down for a while and steps are not synchronized.

prnet cleaned-up successfully


Add more html error codes

Add more html error codes; add decline method

Seems legit. Did you test the decline user?

Yes I did. I tested it and I got a http 200 OK reply, and I think I also looked at the account on the phone after this but I'm not sure.


clean up block access api

other modules should Never know about chunks. Just get and set blocks

From the outside chunks should be fairly opaque. The chunk manager will handle all location and de-allocation. It can optionally handle caching the data. It should be given a callback to fill a chunk in with data. The function should not have to create it’s own structure, but instead fill in the structure of the chunk.  Since the most common case is to fill a chunk entirely from either a saved state or a function, the block setting api should be fast and support setting Lotts of blocks at the same time.
Proposal:
Create a chunk manager with parameters.  The texture manager is internal to the chunk manager, but can be overridden if desired.  Caching is internal to the chunk manager.  the mesher is internal to the chunk manager. Draw distance is internal as well.
When a new chunk must be allocated it calls the generate function to fill in the chunk. It will first create an empty chunk at the desired location. Then call the function to fill in the chunk. Then it will run the mesher. Then it will convert the mesh data into an actually threeJS mesh. Then it will add the mesh to the threeJS scene.  It will also place the mesh data into a cache to be used when
The outside should provide a threeJS group for the chunk manager to put meshes into. Alternatively you could pass in a callback for when a mesh should be added or removed from the scene. These could also be events.
The chunk api now looks like this:
Chunk {
	isEmpty() // returns if chunk is completely empty. true by default.
	getRawData() // returns a flat array of block values
	setBlockAtLocalCoords(x,y,z,value) //set using raw integer coords
	setBlockAtLocalPosition(pos, value) //set using Position object
	setBlockRange(start,dim,array)
	// set a rectangular range of block values
  // starting position
  // size of the range in x, y, z dimensions (a Position used a w,h,d)
  // flat array of values to be copied in.
  getChunkCoordId()
  //gets the string used for caching
	getChunkSize()
	// gets the size of the chunk in blocks. assumes cube shaped chunks.
}

The chunk manager has the following init parameters
new ChunkManager({
	chunkSize: 16, // size of the chunks. assumes cube shaped
	blockSize: 1, //size of a single block in meters
        chunkDistance: 3,
           // draws chunks within a particular radius of the camera,
           // in chunks. uses manhattan distance.
	generateChunkData: (chunk) => {
		//a callback function. should fill the chunk with data.
        },
        textureManager: new TextureManager(),
             //replace tm with one preloaded with the textures you want,
	mesher: new CulledMesher(), //replace with the custom mesher
	container: new Three.Group(),  // a threejs object that all meshes will be placed in
})

cm.setBlockAtWorldPosition(pos,value)
     // sets block value at the global position. will look up the correct chunk then set it there.
cm.textureManager
    // reference to the texture manager
cm.updateCenterPosition(pos)
    // used to update the center of the world and trigger loading new chunks and removing old chunks.
cm.rebuildAllChunks()
    // marks all chunks as dirty, rebuilds them.

Internally the chunk manager will re-mesh and rebuild chunks when their data changes. The outside code never has to mess with this (since it’s 99% always the same)

fixed with 5f86d040786681dd8c9e93b2cdfd380207100483


Split Getting Started into Installing Kubeflow and Concepts

The Getting Started section has become very small.
I think there is significant benefit if we do the following:

Move the Installing Kubeflow page to the root of the docs

this will let users immediately see where they can learn how to install Kubeflow

Create a new Concepts section at the root

this section will cover core principals/features which span across all components

Move the Getting Started/Kubeflow Overview page to Concepts
Move the Components/Multi-Tenancy page to Concepts

NOTE: Obviously we will update the Kubeflow Overview and Multi-User pages, but we should do this AFTER we move them to their new home.

/priority p1

@RFMVasconcelos if you agree do you have capacity to make this change?

@shannonbradshaw thoughts?

@shannonbradshaw I think you might have misunderstood what I was proposing.
My proposal would result in the following structure:
About (NO CHANGE)
- Introduction to Kubeflow (NO CHANGE)
- ...

Concepts (NEW SECTION)
- Conceptual Overview (MOVED OUT OF: "Getting Started")
- Multi-Tenancy (MOVED OUT OF: "Components")

Installing Kubeflow (MOVED OUT OF: "Getting Started")

Components (NO CHANGE)
- Central Dashboard (NO CHANGE)
- ...

External Add-Ons (NO CHANGE)
- Istio (NO CHANGE)
- ...

...

NOTE: This is an iterative step, we should re-factor/split-up the About section next
Also, I strongly disagree with putting Multi-Tenancy at the root, and believe we need to create a Concepts section, as there will certainly be more high-level design concepts we want to highlight to users.
Also, what are your thoughts about this @RFMVasconcelos?

As a side note, I think we should rename "Installing Kubeflow" to "Deploying Kubeflow" as this better covers managed services, (which aren't "Installed")

Do feel free to ping me on any priority thread that you need my eyes on :)

@thesuperzapper I did not misunderstand. I simply disagree and provided a rationale as to why. It sounds like you disagree with what I'm proposing, but I have not seen you state a rationale as to why you oppose placing multi-tenancy at the top level. Also, I'm not sure I've seen a rationale as to why you believe started/kubeflow-overview should be placed in the same section. Why would we place a conceptual overview together with a topic like multi-tenancy in the same section?

@shannonbradshaw my assumptions/logic is:

We want to provide a "feature" view of Kubeflow in the docs, rather than only a component view like right now

Therefore, we should create a new top-level section of the docs called Features (or Concepts)

The Features section should have an "overview" page

Therefore, we can use started/kubeflow-overview as a stating point for this "overview"

Multi-Tenancy is a "feature" of the platform  (which cut across multiple components)

Therefore, Multi-Tenancy should live in the Features section

Note, because we will have more "features" than just Multi-Tenancy we dont want to put it at the top level on its own.
What part are you disagreeing with?

@RFMVasconcelos did you get a chance to look at this split, as described in: https://github.com/kubeflow/website/issues/2671#issuecomment-829723462

It looks like we are tackling 3 problems here:

Installing Kubeflow to top level. > everyone seems to agree with the move
What to do with multi-tenancy? > no consensus yet > Started issue #2736
Where to move Getting started/Kubeflow overview to remove Getting Started

I would suggest actioning move (1), discuss (2) in #2736 and move (3) under about so we can remove Getting Started and then we refactor all things under about.
WDYT?

@RFMVasconcelos sounds good if you want to make PR's, I will review.

@RFMVasconcelos  I actually think we can keep Installing Kubeflow under Getting Started, allowing us to add other pages to Getting Started as well.
For example, we could add an Getting Started / Examples page, like I proposed in https://github.com/kubeflow/website/issues/2759

I am going to close this in preference of leaving the "Getting Started" section, so https://github.com/kubeflow/website/issues/2760 and https://github.com/kubeflow/website/issues/2759 can add things to the Getting Started section.
Regarding "Concepts" I think we can place them under "Getting Started" also, as proposed in: https://github.com/kubeflow/website/issues/2803


Key protected API

Hi,
I'm trying to use your client for creating dataverses. I'm making a script which work with your lib destinated to final users.
So they can't go to the production server to execute the script on localhost.
I'm trying to open the API like said here: http://guides.dataverse.org/en/latest/installation/config.html#blockedapipolicy
I've set a key and the "unblock-key" policy. But I don't find how to specify the key for the python library.
Could you help me ? :/
Regards

More explanations given by @pdurbin on IRC.
I think I was in a wrong way, my problem seems to be elsewhere.
IRC Log: http://irclog.iq.harvard.edu/dataverse/2016-08-16#i_39461

The problem is probably that the Dataverse docs aren't clear enough! @jri-sp please feel free to make a pull request! http://guides.dataverse.org/en/4.4/developers/documentation.html 😄


Update dependencies

There has been a bug fix in https://github.com/sile/thrift_codec :
https://github.com/sile/thrift_codec/pull/10

ping @inikulin

This was merged upstream at https://github.com/sile/rustracing_jaeger/pull/37


Add initial flux deployment

/kind enhancement
What this PR does / why we need it:
We want to deploy prow and the additional components in this repository via flux.
This PR adds the initial flux deployment, i.e., only the flux components themselves without any workload application.
Flux manages itself, i.e., the flux-system kustomization contains the flux manifests and the flux Kustomization for itself as well.
Later on, we can add more Kustomizations and HelmReleases to deploy additional components and remove them step by step from deploy.sh.
Which issue(s) this PR fixes:
Part of https://github.com/gardener-community/hackathon/tree/main/2024-12_Schelklingen
Special notes for your reviewer:
Manual steps for rollout:
After merging this PR, we need to bootstrap the flux self-management by applying the flux-system bootstrap kustomization once:
# g target --shoot prow-trusted
k apply --server-side -k clusters/prow-trusted
# g target --shoot prow-build
k apply --server-side -k clusters/prow-build

Awesome 🚀
/lgtm
/approve


Documentation is not correct

Under configure hybrid specific settings, the steps are:

In Central Admin, under system settings, click "Manage services in this farm".  There is no such option.
For the Microsoft SharePoint Foundation Subscription Settings Service, click Enable Auto Provision - WHERE IS THIS OPTION?  I see nothing similar to enable auto provision.
[Enter feedback here]

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

ID: 1671c351-27e4-e726-858d-c3d660777977
Version Independent ID: 6cbafe83-90b9-1113-a49b-9212505338b5
Content: Set up SharePoint services for hybrid environments
Content Source: SharePoint/SharePointServer/hybrid/set-up-sharepoint-services-for-hybrid-environments.md
Product: sharepoint-server-itpro
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@nancywagar Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello @nancywagar
I found that those directions are for 2019 menu options, that's probably you are using SharePoint 2013.
I will create a pull request to add the SharePoint 2013 alternatives.
Thank you

@nancywagar Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Support react v16.11 (#96)

Description
Support react and react-dom 16.11.
References
https://github.com/auth0/auth0-react/issues/96
Testing
I tested by changing these lines
https://github.com/auth0/auth0-react/blob/master/package.json#L75-L76
to
    "react": "~16.11.0",
    "react-dom": "~16.11.0",

and running npm test (these all pass)
I reverted this because I figured it's better to test with 16.13 normally. Is there a different approach you'd like to take?
I tried setting up the integration tests but I ran into these kinds of errors:
Module not found: Can't resolve '@auth0/auth0-react' in ' ...

[x] This change adds test coverage for new/changed/fixed functionality

Checklist

[x] I have added documentation for new/changed functionality in this PR or in auth0.com/docs
[ ] All active GitHub checks for tests, formatting, and security are passing
[x] The correct base branch is being used, if not master

Thanks @tiagob

@adamjmcgrath I'm using this in a full stack developer tool I made. It generates the boilerplate for a full-stack. It includes Auth0 with RN/Expo and React web. I'd love to hear any advice or feedback you could provide
https://github.com/tiagob/create-full-stack


Add an RSS feed.

First pass at this is available at https://web3isgoinggreat.com/feed

@molly It's good! I'd recommend that you also add images.


fix: update method now uses findOneAndUpdate instead updateOne

Proposed Changes

update method now uses findOneAndUpdate instead updateOne

Readiness Checklist
Author/Contributor

[ ] If documentation is needed for this change, has that been included in this pull request
[ ] Remember to check if code coverage decrease, if so, please implement the necessary tests

Reviewing Maintainer

[ ] Label as breaking if this is a large fundamental change
[ ] Label as either automation, bug, documentation, enhancement, infrastructure, or performance

:tada: This PR is included in version 2.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


error in codelist-definition for BT-01-notice in fields.json

The current codelist-definition for BT-01-notice is:
"codeList" : {
"value" : {
"id" : "eforms-legal-basis",
"type" : "flat"
},
"severity" : "ERROR"
}
the codelist-Folder contains no codelist with that name - only legal-basis and legal-basis_eforms-legal-basis.
In accordance to other codelist-defintions in fields.json, should the codelist-definition for BT-01-notice be changed into
"codeList" : {
"value" : {
"id" : "eforms-legal-basis",
"type" : "flat"
"parentId" : "legal-basis"
},
"severity" : "ERROR"
}
?

The codelist property in the fields.json doesn't support a parentId.
It is defined in the coselists.json where a mapping is defined for the 'eforms-legal-basis', so this is a valid entry.

The filenames in the codelists/ folder do not always correspond to the identifier of the codelist. See: https://docs.ted.europa.eu/eforms/latest/codelists/index.html#_filenames_of_tailored_codelist
You should use the content of the codelists/codelists.json to determine the filename corresponding to a codelist from an identifier.
Please see the section on the "codelist" property in the following page for explanations on the use of "parentId".
https://docs.ted.europa.eu/eforms/latest/fields/index.html#_dynamic_properties
The content of fields.json is correct for this aspect, so I'm closing this, as it's not an issue in the SDK.


"Install kumactl" page should be easier to find

What happened?
At this moment the page that aggregates all the installation info https://kuma.io/install/2.8.x/ apparently doesn't have link to https://kuma.io/docs/2.8.x/production/install-kumactl/#install-kumactl

Triage: the proper fix is to have a page like we used to:

but without the repetition (kumactl on macos installs the same as on debian)
and to have this page be linked on every guide that requires kumactl as a prerequisite


provide fake dns for perf test

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: utka
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

prow/OWNERS [utka]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: icygalz, utka
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

prow/OWNERS [icygalz,utka]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, icygalz, utka
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

prow/OWNERS [icygalz,utka]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@utka: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
1e9c9cf368b6ea991625bdaa26edbb8b36a56428
link
/test integ-pilot-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@utka: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-mixer-k8s-presubmit-tests.sh
1e9c9cf368b6ea991625bdaa26edbb8b36a56428
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@utka: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-mixer-k8s-presubmit-tests.sh
1e9c9cf368b6ea991625bdaa26edbb8b36a56428
link
/test integ-mixer-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
1e9c9cf368b6ea991625bdaa26edbb8b36a56428
link
/test integ-pilot-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Fix phpType Handling for Big Integer Columns

After initiating this discussion on GitHub, it has become clear to me that this is indeed a bug. A big integer should be considered an integer, not a string, for the phpType.
This PR addresses and corrects that issue.

Thank you!


no pref/alt labels after drag&drop

drag a concept from the master to the project tree. in the concept details, no detail is shown (broaders, narrowers, alt, pref all empty)

Works fine for me on master. Maybe this has been fixed in one of the other PRs?

👍

Now it appeared again, try on demo/atlantis/thesaurex, which is on master branch. After refresh children are gone


how to skip hevc h265 files

hi
I am wondering how i can pass  h265 hevc file into the scripts but have it just skip them. My system completely crumbles trying to process these so if i choose to download one i wouldn't want the script to do much of anything to it other than maybe tagging.

Add the following the the video codec: h265, x265, hevc
The script will then copy h265/hevc. So it will skip video conversion. However, it may still convert audio or do other functions if you have it setup for that.

awesome. thanks you. i will give that a go and see if it helps. I don't have too much else configured so it should work in most cases i think


added HTML and CSS for header of page

Added in HTML and CSS for the header of the page. It includes just the page title, and then a background image with a search field within it. In theory, the Teleport API will generate the image for us based on the searched city. These are just placeholders for now just to get the structure of the page.

I also added a possible search history area and the area where the information will generate. Again, everything is just placeholders for now, just like to see the structure of the page until we start adding in the good stuff.

Got nitpicky and fixed the Plan Your Trip button to be right aligned in the section.

Added second HTML page for TripAdvisor information to be generated into. All placeholders!! Also added responsive breakpoints in CSS.


Could not load OpenSSL. with jruby <IP_ADDRESS>-SNAPSHOT

This issue has been created at https://github.com/rsim/oracle-enhanced/issues/1614

What did you do?
Execute CI using Travis CI using jruby-head

What did you expect to happen?
It should run CI.

What happened instead?
It gets Could not load OpenSSL.

Fetching https://github.com/rails/arel.git
Fetching https://github.com/rsim/ruby-plsql.git
Fetching https://github.com/colszowka/simplecov.git
The dependency ruby-oci8 (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for java but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock --add-platform ruby`.
The dependency byebug (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for java but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock --add-platform ruby`.
Fetching https://github.com/kubo/ruby-oci8.git
Could not load OpenSSL.
You must recompile Ruby with OpenSSL support or change the sources in your
Gemfile from 'https' to 'http'. Instructions for compiling with OpenSSL using
RVM are available at rvm.io/packages/openssl.
The command "bundle install" failed and exited with 17 during .

Here is the entire output

Have you tried any solutions posted on similar issues in our issue tracker, stack overflow, or google?

I tried replacing https with http as it may be related with SSL by this commit.
Then it gets this error.
NameError: uninitialized constant Bundler::RubygemsIntegration::YamlLibrarySyntaxError
Did you mean?  Bundler::YamlSyntaxError

Here is the entire output.

The last successful build https://travis-ci.org/rsim/oracle-enhanced/builds/320343001 at 2017-12-22 18:55:45
The first failed build https://travis-ci.org/rsim/oracle-enhanced/builds/320733240 at 2017-12-23 18:52:58
Between these two builds no commits made to the rsim/oracle-enhanced repository. There is a daily build scheduled.

Can you please share the output of bundle env? Thank you

Here is the output of bundle env
Environment
Bundler       1.16.1
  Platforms   ruby, universal-java-1.8
Ruby          2.4.1p0 (2017-12-26 revision 58053) [java]
  Full Path   /home/travis/.rvm/rubies/jruby-head/bin/jruby
  Config Dir  /home/travis/.rvm/rubies/jruby-head/etc
RubyGems      2.7.4
  Gem Home    /home/travis/.rvm/gems/jruby-head
  Gem Path    /home/travis/.rvm/gems/jruby-head:/home/travis/.rvm/gems/jruby-head@global
  User Path   /home/travis/.gem/jruby/2.4.0
  Bin Dir     /home/travis/.rvm/gems/jruby-head/bin
Tools
  Git         2.15.1
  RVM         1.29.3 (latest)
  rbenv       rbenv 1.1.1-25-g6aa70b6
  chruby      not installed
Gem.ruby      /home/travis/.rvm/rubies/jruby-head/bin/jruby
bundle #!     /usr/bin/env jruby_executable_hooks

Bundler Build Metadata
Built At          2017-12-21
Git SHA           0034ef341
Released Version  true

Bundler settings
gemfile
  Set via BUNDLE_GEMFILE: "/home/travis/build/yahonda/oracle-enhanced/Gemfile"

Gemfile
Gemfile
# frozen_string_literal: true
source "https://rubygems.org"
git_source(:github) { |repo| "https://github.com/#{repo}.git" }
group :development do
  gem "rspec"
  gem "rdoc"
  gem "rake"
  gem "activerecord",   github: "rails/rails", branch: "master"
  gem "arel",   github: "rails/arel", branch: "master"
  gem "ruby-plsql", github: "rsim/ruby-plsql", branch: "master"
  platforms :ruby do
    gem "ruby-oci8",    github: "kubo/ruby-oci8"
    gem "byebug"
  end
  platforms :jruby do
    gem "pry"
    gem "pry-nav"
  end
end
group :test do
  gem "simplecov",  github: "colszowka/simplecov", branch: "master", require: false
end

Gemfile.lock
<No /home/travis/build/yahonda/oracle-enhanced/Gemfile.lock found>

Entire output of the failed build https://travis-ci.org/yahonda/oracle-enhanced/jobs/322171825

@yahonda Can you reproduce with your local env?

@hsbt So far I have not reproduced this in my local environment.  Let me try again.

It reproduces locally using Vagrant.
Install software

Vagrant 2.0.1
VirtualBox Version 5.2.4
Oracle Database 11g Express Edition for Linux x64. The file name is oracle-xe-11.2.0-1.0.x86_64.rpm.zip.

http://www.oracle.com/technetwork/database/database-technologies/express-edition/overview/index.html
Steps to reproduce:

At Vagrant guest

host $ git clone -b runs_oracle https://github.com/yahonda/rails-dev-box.git
host $ cp oracle-xe-11.2.0-1.0.x86_64.rpm.zip rails-dev-box/puppet/modules/oracle/files/.
host $ cd rails-dev-box
host $ vagrant up
host $ vagrant ssh

it would take 10 minutes to complete.

At rails-dev-box

$ sudo apt-get install python-software-properties -y
$ sudo apt-add-repository ppa:webupd8team/java
$ sudo apt-get update
$ sudo apt-get install oracle-java8-installer -y
$ sudo apt-get install oracle-java8-set-default -y
$ export JRUBY_OPTS='--debug --dev -J-Xmx2048M'
$ export JAVA_HOME=/usr/lib/jvm/java-8-oracle

$ rvm install jruby-head
$ rvm --default use jruby-head
$ gem update --system
$ gem install bundler

$ git clone https://github.com/rsim/oracle-enhanced.git
$ cd oracle-enhanced/
$ bundle env
$ bundle install

Actual result:
vagrant@rails-dev-box:~/oracle-enhanced$ bundle install
Fetching https://github.com/rails/rails.git
Fetching https://github.com/rails/arel.git
Fetching https://github.com/rsim/ruby-plsql.git
Fetching https://github.com/colszowka/simplecov.git
The dependency ruby-oci8 (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for java but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock --add-platform ruby`.
The dependency byebug (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for java but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock --add-platform ruby`.
Fetching https://github.com/kubo/ruby-oci8.git
Could not load OpenSSL.
You must recompile Ruby with OpenSSL support or change the sources in your
Gemfile from 'https' to 'http'. Instructions for compiling with OpenSSL using
RVM are available at rvm.io/packages/openssl.
vagrant@rails-dev-box:~/oracle-enhanced$

Here is the entire output at Vagrant guest

Added DEBUG_RESOLVER=true bundle install --verbose output below. Not sure if it would help.
vagrant@rails-dev-box:~/oracle-enhanced$ DEBUG_RESOLVER=true bundle install --verbose
Starting resolution (2017-12-27 19:04:45 UTC)
User-requested dependencies: [#<Gem::Resolver::DependencyRequest:0x5db45159 @dependency=<Gem::Dependency type=:runtime name="did_you_mean" requirements="= 1.0.1">, @requester=nil>, #<Gem::Resolver::DependencyRequest:0x6107227e @dependency=<Gem::Dependency type=:runtime name="executable-hooks" requirements="= 1.3.2">, @requester=nil>, #<Gem::Resolver::DependencyRequest:0x7c417213 @dependency=<Gem::Dependency type=:runtime name="bundler-unload" requirements="= 1.0.2">, @requester=nil>, #<Gem::Resolver::DependencyRequest:0x15761df8 @dependency=<Gem::Dependency type=:runtime name="rubygems-bundler" requirements="= 1.4.4">, @requester=nil>, #<Gem::Resolver::DependencyRequest:0x6ab7a896 @dependency=<Gem::Dependency type=:runtime name="bundler" requirements="= 1.16.1">, @requester=nil>]
Resolving dependencies...Creating possibility state for did_you_mean (= 1.0.1) (1 remaining)
 Attempting to activate did_you_mean-1.0.1
 Activated did_you_mean at did_you_mean-1.0.1
 Requiring nested dependencies ()
 Creating possibility state for executable-hooks (= 1.3.2) (1 remaining)
  Attempting to activate executable-hooks-1.3.2
  Activated executable-hooks at executable-hooks-1.3.2
  Requiring nested dependencies ()
  Creating possibility state for bundler-unload (= 1.0.2) (1 remaining)
   Attempting to activate bundler-unload-1.0.2
   Activated bundler-unload at bundler-unload-1.0.2
   Requiring nested dependencies ()
   Creating possibility state for rubygems-bundler (= 1.4.4) (1 remaining)
    Attempting to activate rubygems-bundler-1.4.4
    Activated rubygems-bundler at rubygems-bundler-1.4.4
    Requiring nested dependencies (executable-hooks (>= 1.3.2), bundler-unload (>= 1.0.2))
    Creating possibility state for executable-hooks (>= 1.3.2) (1 remaining)
     Attempting to activate executable-hooks-1.3.2
     Found existing spec (executable-hooks-1.3.2)
     Creating possibility state for bundler-unload (>= 1.0.2) (1 remaining)
      Attempting to activate bundler-unload-1.0.2
      Found existing spec (bundler-unload-1.0.2)
      Creating possibility state for bundler (= 1.16.1) (1 remaining)
       Attempting to activate bundler-1.16.1
       Activated bundler at bundler-1.16.1
       Requiring nested dependencies ()

Finished resolution (7 steps) (Took 0.063195 seconds) (2017-12-27 19:04:45 UTC)
Unactivated:
Activated: did_you_mean, executable-hooks, bundler-unload, rubygems-bundler, bundler
Running `bundle install --verbose` with bundler 1.16.1
Fetching https://github.com/rails/rails.git
Fetching https://github.com/rails/arel.git
Fetching https://github.com/rsim/ruby-plsql.git
Fetching https://github.com/colszowka/simplecov.git
Found changes from the lockfile, re-resolving dependencies because the list of sources changed, the dependencies in your gemfile changed, you added a new platform to your gemfile
The dependency ruby-oci8 (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for java but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock --add-platform ruby`.
The dependency byebug (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for java but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock --add-platform ruby`.
Fetching https://github.com/kubo/ruby-oci8.git
Could not load OpenSSL.
You must recompile Ruby with OpenSSL support or change the sources in your
Gemfile from 'https' to 'http'. Instructions for compiling with OpenSSL using
RVM are available at rvm.io/packages/openssl.
vagrant@rails-dev-box:~/oracle-enhanced$

Using rubygems 2.6.4 changes some bahavior. It does not show Could not load OpenSSL. but failed to install ffi.
Installing ffi 1.9.18 (java)
Gem::InstallError: invalid gem: package metadata is missing in
/home/vagrant/.rvm/gems/jruby-head/cache/ffi-1.9.18-java.gem
An error occurred while installing ffi (1.9.18), and Bundler cannot
continue.
Make sure that `gem install ffi -v '1.9.18'` succeeds before bundling.

In Gemfile:
  pry-nav was resolved to 0.2.4, which depends on
    pry was resolved to 0.10.4, which depends on
      spoon was resolved to 0.0.6, which depends on
        ffi
vagrant@rails-dev-box:~/oracle-enhanced$ gem -v

Entire output:

vagrant@rails-dev-box:~/oracle-enhanced$ bundle install
Fetching https://github.com/rails/rails.git
Fetching https://github.com/rails/arel.git
Fetching https://github.com/rsim/ruby-plsql.git
Fetching https://github.com/colszowka/simplecov.git
The dependency ruby-oci8 (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for java but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock --add-platform ruby`.
The dependency byebug (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for java but the dependency is only for ruby. To add those platforms to the bundle, run `bundle lock --add-platform ruby`.
Fetching https://github.com/kubo/ruby-oci8.git
Fetching gem metadata from https://rubygems.org/........
Fetching gem metadata from https://rubygems.org/..........
Fetching gem metadata from https://rubygems.org/..........
Resolving dependencies................
Using rake 12.3.0
Fetching concurrent-ruby 1.0.5 (java)
Installing concurrent-ruby 1.0.5 (java)
Fetching i18n 0.9.1
Installing i18n 0.9.1
Fetching minitest 5.10.3
Installing minitest 5.10.3
Fetching thread_safe 0.3.6 (java)
Installing thread_safe 0.3.6 (java)
Fetching tzinfo 1.2.4
Installing tzinfo 1.2.4
Using activesupport 5.2.0.beta2 from https://github.com/rails/rails.git (at master@17ad4eb)
jar-dependencies: spec must be either String or Gem::Specification
Using activemodel 5.2.0.beta2 from https://github.com/rails/rails.git (at master@17ad4eb)
jar-dependencies: spec must be either String or Gem::Specification
Using arel 9.0.0 from https://github.com/rails/arel.git (at master@bb0e60f)
jar-dependencies: spec must be either String or Gem::Specification
Using activerecord 5.2.0.beta2 from https://github.com/rails/rails.git (at master@17ad4eb)
jar-dependencies: spec must be either String or Gem::Specification
Using bundler 1.16.1
Fetching coderay 1.1.2
Installing coderay 1.1.2
Fetching diff-lcs 1.3
Installing diff-lcs 1.3
Fetching docile 1.1.5
Installing docile 1.1.5
Fetching ffi 1.9.18 (java)
Installing ffi 1.9.18 (java)
Gem::InstallError: invalid gem: package metadata is missing in
/home/vagrant/.rvm/gems/jruby-head/cache/ffi-1.9.18-java.gem
An error occurred while installing ffi (1.9.18), and Bundler cannot
continue.
Make sure that `gem install ffi -v '1.9.18'` succeeds before bundling.

In Gemfile:
  pry-nav was resolved to 0.2.4, which depends on
    pry was resolved to 0.10.4, which depends on
      spoon was resolved to 0.0.6, which depends on
        ffi
vagrant@rails-dev-box:~/oracle-enhanced$ gem -v
2.6.14
vagrant@rails-dev-box:~/oracle-enhanced$ ruby -v
jruby <IP_ADDRESS>-SNAPSHOT (2.4.1) 2017-12-27 7bd10a6 Java HotSpot(TM) 64-Bit Server VM 25.151-b12 on 1.8.0_151-b12 [linux-x86_64]
vagrant@rails-dev-box:~/oracle-enhanced$ bundler -v
Bundler version 1.16.1
vagrant@rails-dev-box:~/oracle-enhanced$

Does running ruby -ropenssl -e 'p OpenSSL' succeed for you? Because if not, that would imply that OpenSSL is actually not available to load

It shows  OpenSSL
$ ruby -ropenssl -e 'p OpenSSL
OpenSSL
Refer line 860.
https://travis-ci.org/yahonda/oracle-enhanced/jobs/323192740


Gcs repository refactoring

Gcs repository refactoring:

GCS now aligned to the common plugin model
Bump Up ES to version 7.9.1

@ivanyu could you plz do not merge it so far ... it seems that we have concurrency issue

@ivanyu there is no issue ... everything is ok

@ivanyu done


Version change to 2.0.0.

What is the necessity of this update? What is updated?
I changed the version of pom.
How did you do the test? (Not required for updating documents)

LGTM


[BUG] 'find_element_by_name'

Traceback (most recent call last):
File "C:\Users\User\Downloads\Microsoft-Rewards-Farmer-master\Microsoft-Rewards-Farmer-master\ms_rewards_farmer.py", line 773, in
login(browser, account['username'], account['password'])
File "C:\Users\User\Downloads\Microsoft-Rewards-Farmer-master\Microsoft-Rewards-Farmer-master\ms_rewards_farmer.py", line 46, in login
browser.find_element_by_name("loginfmt").send_keys(email)
AttributeError: 'WebDriver' object has no attribute 'find_element_by_name'

You're using the wrong version of the web driver.
Try selenium 3.1.4 and also ensure you're using the appropriate chrome version for whatever you have downloaded on your PC

You must have downloaded the GitHub release, you need to clone the latest instead. (for some reason the "release" uses find_element_by_name which no longer exists)

You must have downloaded the GitHub release, you need to clone the latest instead. (for some reason the "release" uses find_element_by_name which no longer exists)

can you please tell how do i clone the latest release

i have a fix for this. it worked for me. all you have to do is go to requirements.txt and replace the selenium text with selenium==3.14.1

someone re created the bot https://github.com/MehdiRtal/MicrosoftRewardsBot.git

Fixed with v3.0 #188


timezone issue of get_bars/get_bars_batch

when get_bars/get_bars_batch called on minute level frames, the timezone of returned frames is set to Asia/Shanghai, however, the value of frame is 8 hours great than expected.
Reason:  bars returned by jqdatasdk are naive datetime object. get_bars use datetime.astimezone() to add timezone info to records, this caused frame has been treaded as UTC, then localized as 'Asia/shanghai', thus the value is not right.

fixed in commit cc0a1d5eb9c0fb6508a76c55b06517b462d757d9


feat(vitest): add @suspensive/vitest generating named config automatically by packageJson.name

@suspensive/vitest's forPackage
I added @suspensive/vitest generating named config automatically by packageJson.name
So we don't have to name manually. It will find package.json's name field automatically and use it as vitest's name.
AS IS
import { defineConfig } from 'vitest/config'

export default defineConfig({
  test: {
    name: '@suspensive/react', // manually we have to add this in every package
    dir: './src',
    watch: false,
    environment: 'jsdom',
    globals: true,
    setupFiles: './test.setup.ts',
    coverage: { provider: 'v8' },
  },
})

TO BE
import { forPackage } from '@suspensive/vitest'
import { defineConfig } from 'vitest/config'

export default defineConfig(forPackage()) // forPackage will find package.json's name field automatically

But it's extendable
import { forPackage } from '@suspensive/vitest'
import { defineConfig } from 'vitest/config'

export default defineConfig(
  forPackage({
    test: {
      name: 'suspensive renamed test name',
    },
  })
)

PR Checklist

[x] I have written documents and tests, if needed.

@okinawaa I will change base branch


[App]: Parallel coordinates chart tooltip shows 1e-1 instead of 1e-10

Current Behavior
You can see on the left, and from the position on the chart, that the value for weight_decay is 1.693e-10, but the tooltip drops that last 0.

Expected Behavior
No response
Steps To Reproduce
No response
Screenshots
No response
Environment
Browsers: Chrome 113
Additional Context
No response

Hello @davidgilbertson !
Thank you for reporting this! I was able to reproduce the error and will be making a bug report for this.

WandB Internal User commented:
rsanandres-wandb commented:
Hello @davidgilbertson !
Thank you for reporting this! I was able to reproduce the error and will be making a bug report for this.


[feature request] define disk temp path

Thanks for such amazing tool!
I'm scanning a huge organization and not sure if the space in my disk will be enough to store everything under /tmp/ directory.
So my suggest is include an optional parameter to define the temp directory when --disk is used.
I'd like to help implement that but I don't know golang enough to contribute.

hey @riramar great suggestion. Would a separate --disk-path= option work?

yes... it should work! thanks @zricethezav

Closed this prematurely, sorry. Still have not implemented this.

Use --clone-path in v7


Attachment in Response

Some commands deal with files, so the user should be able to attach the file with the command and we send FormData to the handler.

Explores multipart/related or similar so we can send both text and files.


WIP: Fix insert figure

Why
Inserting a figure is broken since introducing a custom figure model.
What

moves the original helper for inserting a figure into ArticleAPI,
fixes related implementations
fixes regressions in commands for inline-formula and ext-link

Codecov Report

Merging #702 into master will decrease coverage by <.01%.
The diff coverage is 20.51%.

@@            Coverage Diff             @@
##           master     #702      +/-   ##
==========================================
- Coverage   64.98%   64.97%   -0.01%
==========================================
  Files         280      279       -1
  Lines        7156     7169      +13
==========================================
+ Hits         4650     4658       +8
- Misses       2506     2511       +5

Impacted Files
Coverage Δ

src/kit/ui/ToggleTool.js
73.91% <0%> (ø)
:arrow_up:

src/article/editor/InsertNodeCommand.js
0% <0%> (ø)
:arrow_up:

src/article/editor/DropFigure.js
0% <0%> (ø)
:arrow_up:

src/article/editor/InsertInlineFormulaCommand.js
26.08% <0%> (ø)
:arrow_up:

src/article/editor/InsertExtLinkCommand.js
0% <0%> (ø)
:arrow_up:

src/article/editor/InsertFigureCommand.js
0% <0%> (ø)
:arrow_up:

src/article/metadata/MetadataEditor.js
96.1% <100%> (ø)
:arrow_up:

src/article/reader/ArticleReader.js
60.81% <100%> (ø)
:arrow_up:

src/kit/app/CommandManager.js
90.9% <100%> (ø)
:arrow_up:

src/article/ArticleAPI.js
35.29% <5.55%> (-3.53%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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


Can't match lambda parameter in setup

Hi there,
I'm trying to mock ngrx/store, specifically the select method. The select method takes a function, so I pass in a lambda like state => state.compensation.filters. TypeMoq doesn't seem to recognize this as a match when I use the same function in my real code to call select. Am I doing something wrong? When I use It.isAny(), everything works fine, but I actually need TypeMoq to respond differently based on different input parameters, so that doesn't work in my case.
const store = TypeMoq.Mock.ofType<Store<AppState>>();
const filtersStateStore = TypeMoq.Mock.ofType<Store<FiltersState>>();

filtersStateStore.setup(f => f.subscribe(TypeMoq.It.isAny())).callback(subscribe => {
  subscribe(filters);
});

this.store.setup(s => s.select(val => val.compensation.filters))
  .returns(() => filtersStateStore.object);

The last line is the important one. I provide the rest simply for context.

To be able to match the select method based on its input parameters, you should create the setup using a predicate. E.g.
this.store.setup(s => s.select(TypeMoq.It.is(val => val.compensation.filters !== null)))
  .returns(() => filtersStateStore.object);

Alternatively you may create your own custom matcher to better suit your scenario (see #79 ).


Use the same python interpreter as yum when possible

We rely on a system python installation with the correct python
libraries.  To improve the chances that the python interpreter we call
has the yum libary available, we use the same interpreter as yum uses.
Fixes #3143

:+1:

:+1:
im thinking of /usr/bin/env python style shebang lines, should be fine though

Can one of the Enterprise Linux maintainers take a quick look? cc @jonlives @lamont-granquist @jaymzh

:+1:

:+1:

:+1:


buildEmberPlugins breaks embroider

In addon, with embroider enabled, and using the recent "babel.config.js" + buildEmberPlugins( feature, build fails with a message, like:

Build Error (PackagerRunner) in assets/test.js
Module not found: Error: Can't resolve '../../../../..$TMPDIR/embroider/b3ee60/node_modules/@ember/test-helpers/-internal/build-registry' in '$TMPDIR/embroider/b3ee60/tests/dummy/assets'
Stack Trace and Error Report: /var/folders/d9/kk8qd5p12j7_zmxkffkcslr80000gn/T/error.dump.6513f604a00cb2fe5ae748a9cc36f5c6.log

I've tried to find a root cause in embroider itself, but turns out the issue appears once I add
     plugins: [
      ...buildEmberPlugins(__dirname),
    ],

to the "babel.config.js", with embroider build is enabled. If I have "embroider" + plain "babel.config.js", build works just fine. So I guess, ec-babel, might be a good starting point to explore the issue..
Note, it's reproducible, even if the useBabelConfig is not set.
I've created a reproduction here https://github.com/ro0gr/ec-babel-emroider-buildEmberPlugins-issue
Please let me know, if I can help somehow to track down the root cause.

it's worth noting, if I leave the plugins as an empty array, w/o buildEmberPlugins( being passed, build starts working again.

hm, turns out passing the disableModuleResolution: true, fixes the build issue reported here. I'm fine to close this, cause it seems to be solved now for me. However, I'm curious if this is an intentional behavior. Can/should it be improved?


Angular Material vs Angular MDC

What is difference between both of this and which is preferable to use?
Angular Material - Material Design components for Angular - https://material.angular.io
Angular MDC - Material Web Components for Angular -  https://github.com/trimox/angular-mdc-web
How it is different from implementation and contributors point of view?
Are they both maintained by google ?

In a really shallow description, both are component librarys that use material design as guideline to build it's components.
The first one is maintained by Angular team (Google).
The second is a third-party integration incorporating Google Material Components library
Also, you should avoid opening issues for support questions. More info at https://github.com/angular/components/blob/master/CONTRIBUTING.md#-got-a-question-or-problem

The second library you link there is not an official Angular library; it's a community project based on MDC Web. Here's the gist:

Angular Material today is a standalone implementation of Material Design components built by the Angular team at Google
MDC Web is a separate library of component primitives built by the Material Design team at Google

The Angular team has been working on refreshing the existing Angular Material components to incorporate MDC's primitives internally, these new versions of the components are available for testing in the @angular/material-experimental package today, but they're not yet stable. In a future version of Angular, they will replace the existing components.
See the Angular roadmap for more information: https://angular.io/guide/roadmap


StubRunner with repoRoot and workOffline=false is still looking for stubs in local repository

Hey guys,
for my scenario I need to work with remote repositories to retrieve the stubs. There is a case that didn't work out as I expected and I wanted to discuss that here.
So, there is the workOffline Flag and the URL repoRoot.
If workOffline is true and no repoRoot is defined, the stubs will be searched in the local maven repository.
If workOffline is true and a repoRoot is defined, there will be an error.
If workOffline is false and no repoRootis defined, the classpath will be scanned.
The last case is if workOffline is false and a repoRoot is defined. My expectation was that the stubs will be searched in this repository and there will be an error if they are not found there.
What really happens is that the stubs will be searched in the defined repository, but as a fallback will also be searched in the local repository.
So, to reproduce it, just use the samples and change the paramters. Since I use the JUnit rule mechanism, I changed BeerControllerWithJUnitTest that way:
@Rule public StubRunnerRule rule = new StubRunnerRule()
			.repoRoot("http://artifactory.example.com")
			.downloadStub("com.example","beer-api-producer")
			.workOffline(false);

As I said, when the stub is in the local maven repository, this code just runs. If there is no stub in the local repository, I get this error message, which additionally confirms that the local repository is used:
java.lang.IllegalArgumentException: For groupId [com.example] artifactId [beer-api-producer] and classifier [stubs] the version was not resolved! The following exceptions took place [org.eclipse.aether.transfer.MetadataNotFoundException: Could not find metadata com.example:beer-api-producer/maven-metadata.xml in local (/Users/xxxx/.m2/localRepository), org.eclipse.aether.transfer.MetadataNotFoundException: Could not find metadata com.example:beer-api-producer/maven-metadata.xml in remote0 (http://artifactory.example.com)]

What I really want to do is to ensure that only the most recent stubs are used, both on our build server and on local developer machines. That's the reason why this fallback kinda bothers me. On the build server, it feels not right if there was no stub in the given remote repository, so I expect an error.
Maybe I'm too anxious because there is a lot of magic happening. Would be great to get some feedback about my thoughts. :)
Cheers, Gonzo

As far as I see this behaviour is not documented: https://cloud.spring.io/spring-cloud-contract/spring-cloud-contract.html#_stub_downloading

If you provide the stubrunner.repositoryRoot or stubrunner.workOffline flag will be set to true then Stub Runner will connect to the given server and download the required jars. It will then unpack the JAR to a temporary folder and reference those files in further contract processing.

So if this behaviour is correct, it should at least be mentioned there, I guess.

Yeah that looks bad. I'll take another look in the code why we are doing the fallback (cc @mariuszs - do you remember any particular reason?) and if there's no concrete reason then it should be fixed as you mentioned. Thanks a lot for checking it out.

if (!workOffline) {
  session.setUpdatePolicy(RepositoryPolicy.UPDATE_POLICY_ALWAYS);
}

Local repository is always used, but when workOffline flag is disabled, then we are always asking remote repository for changes.

@Gonzo17 go ahead and check out latest snapshots. Now you should start getting an IllegalStateException

@marcingrzejszczak Yeah that works, thanks!
One last question to this. If the artifact is found in both remote and local repository, it will take the artifact from the remote? Or does it take from the local repository then? And how does it decide if one artifact is newer than the other?

It picks from the remote first and if it finds it then that version will be used regardless of what you have locally.

Alright, thank you very much! Case closed :)


Move and simplify activity rules

Codecov Report
Base: 74.28% // Head: 72.93% // Decreases project coverage by -1.35% :warning:

Coverage data is based on head (d9e0b37) compared to base (a8d4cf3).
Patch coverage: 17.10% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #614      +/-   ##
==========================================
- Coverage   74.28%   72.93%   -1.35%
==========================================
  Files          17       18       +1
  Lines        6517     6322     -195
==========================================
- Hits         4841     4611     -230
- Misses       1676     1711      +35

Impacted Files
Coverage Δ

src/Enzyme.jl
83.86% <ø> (-0.16%)
:arrow_down:

src/internal_rules.jl
0.00% <0.00%> (ø)

src/compiler/interpreter.jl
66.66% <50.00%> (-15.28%)
:arrow_down:

src/compiler/validation.jl
60.84% <58.33%> (-1.23%)
:arrow_down:

src/compiler.jl
73.81% <100.00%> (-0.64%)
:arrow_down:

src/pmap.jl
83.33% <0.00%> (-1.29%)
:arrow_down:

src/compiler/pmap.jl
76.37% <0.00%> (-1.17%)
:arrow_down:

src/typeanalysis.jl
90.00% <0.00%> (-0.91%)
:arrow_down:

... and 5 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


UNABLE TO INSTALL THIS EXTENSION IN YII2

I ma unable to install this extension in yii2. composer shows error when i run this command

composer require "nex/yii2-chosen" "*"
Deprecation Notice: The Composer\Package\LinkConstraint\MultiConstraint class is
deprecated, use Composer\Semver\Constraint\MultiConstraint instead. in phar://C
:/ProgramData/ComposerSetup/bin/composer.phar/src/Composer/Package/LinkConstrain
t/MultiConstraint.php:17
Deprecation Notice: The Composer\Package\LinkConstraint\LinkConstraintInterface
interface is deprecated, use Composer\Semver\Constraint\ConstraintInterface inst
ead. in phar://C:/ProgramData/ComposerSetup/bin/composer.phar/src/Composer/Packa
ge/LinkConstraint/LinkConstraintInterface.php:17
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

Problem 1
- The requested package dbtek/chosen-bootstrap could not be found in any version, there may be a typo in the package name.
Problem 2
- The requested package nex/yii2-chosen could not be found in any version, there may be a typo in the package name.
Potential causes:

A typo in the package name
The package is not available in a stable-enough version according to your minimum-stability setting
see https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion for more details.

Read https://getcomposer.org/doc/articles/troubleshooting.md for further common problems.
Installation failed, reverting ./composer.json to its original content.
please help ASAP
i need it in yii2 else show me the process for manual installation of this great extension.

This is not an extension problem. First of all, you have an old version of composer (look at deprecation notices). Run composer self-update to renew it.
Second: I can not reproduce your problem. Just installed my package on fresh system:
php composer.phar require "nex/yii2-chosen" "*"
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
  - Installing nex/yii2-chosen (dev-master 881afa0)
    Cloning 881afa090ff11183052aebeb61d3dacd54933da9

Writing lock file
Generating autoload files

Third: I see you have errors not only for my package, but also for dbtek/chosen-bootstrap. It can be some common problem.

Thanks for your reply,
Sir i already updated my composer but still i am unable to install this extension. ? :( :(
I really need it. please help me
my composer command shows -
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- The requested package dbtek/chosen-bootstrap could not be found in any ver
sion, there may be a typo in the package name.
Problem 2
- The requested package nex/yii2-chosen could not be found in any version, t
here may be a typo in the package name.
Potential causes:

A typo in the package name
The package is not available in a stable-enough version according to your min
imum-stability setting
see https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion f
or more details.

Read https://getcomposer.org/doc/articles/troubleshooting.md for further commo
n problems.
Installation failed, reverting ./composer.json to its original content.

Did you read the mentioned discussion? What is the minimum-stability setting of your project? If it is not dev (stable by default), it can be the reason.


Mention Office.onReady as Office.initialize alternative

Source https://github.com/OfficeDev/office-js/issues/5#issuecomment-382189046

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

ID: af6ccc36-8ea8-af6e-8731-059ce81d8628
Version Independent ID: 2d98dd01-b671-d532-3bcb-88d40d6ef3d1
Content: Loading the DOM and runtime environment
Content Source: docs/develop/loading-the-dom-and-runtime-environment.md
Product: office
GitHub Login: @o365devx
Microsoft Alias: o365devx

We are working on this.

onReady is now documented here: https://docs.microsoft.com/en-us/office/dev/add-ins/develop/understanding-the-javascript-api-for-office


Blog search demo is batch oriented, allow 1 second visibility-delay w…

…ith improved througput.
Tutorial also updated.

👍


Publish private builds

Allow users to publish private builds
Reviewers:
@jinruh
@sbaogang
@eweizman

Pop up modal/dialog UI:

Flagged private build UI:

This is considered a Pinterest specific feature because the specific code to upload artifact to Pinterest specific repo. Can we hide this feature in the UI?

Pop up modal/dialog UI:

Cool, since we are not going to need the repo name etc., and already we hard coded the branch name, we could just hardcode everything and make the dialog extremely simple - just ask for SHA and file path, and we will fill out everything else.

Pop up modal/dialog UI:

r+, if you can resolve the conflicts, squash all the intermediate commits before you merge.


Roadmap to Symfony

This is the roadmap about this process:
Initial setup and codebase reorganization:
If we decide to refactor, we need to create a new Symfony project and reorganize the existing codebase to fit the standard Symfony directory structure.
Refactor PHP classes:
Refactoring the PHP classes in the system/classes directory to use Symfony's services, dependency injection, and autowiring.
Admin backend refactoring:
Refactoring of the admin backend (admin/includes) to use Symfony's controllers, routing, and Twig templating system. Make use of newest Version of AdminLTE or re-write a complete custom solution, maybe using react as frontend helper.
Security:
Implement Symfony's security component for authentication and authorization, replacing the existing security system in YaWK.
Plugins and widgets:
Refactor the existing plugins and widgets to work as Symfony bundles or create new ones as Symfony bundles. This will make them more maintainable and extendable.
Templates:
Refactor the template system to work with Symfony's Twig templating engine and asset management.
Translations:
Implement Symfony's translation component for multi-language support in both the frontend and backend.
Event handling:
Use Symfony's event dispatcher and event subscribers to handle various events in the CMS, such as page creation, content updates, and more.
Automated testing:
Implement automated testing using PHPUnit and Symfony's WebTestCase to ensure code quality and prevent regressions.
Performance optimization:
Optimize and improve the performance of the CMS using Symfony's built-in tools like the profiler and caching system.

This topic has moved to: https://github.com/YaWK/yawk.symfony
(symfony dev playground, not in use yet)
This issue here will be closed.


added PreUpdate for ModWorld

does not return a boolean, not sure if that was supposed to be done.

Also  added PreSaveCustomData hooks for Items, Players and Worlds

sorry for da wrong tabs boiis


input/output stream

Definitely be sure to re-write the tests before merging, if I need to help there, poke me. 90% test coverage and 100% documentation are the goal, if possible :)

Looks good to me! ✅


compose ui

Has Blessed been tested on Compose UI?

Not sure what you mean. This is a BLE library that has nothing to do with any UI framework....

Thanks, I mean, I tried to work with the library and everything went very well, but when I changed just the UI from view to Compose, the changes of the data from the device(onCharacteristicChanged) didn't receive. tried a couple of ways(lifecycles stuff mainly), and because I didn't really try to dive into the library, I wondered if this strange case, as u said, tested.

I can't think of a reason why it wouldn't work in Compose. Just make sure you don't mess up the threading part...

thanks. appreciate your work!


Plugin is not working with Webpack 1.12.14

Hello,
This plugin seems not to be working. It's showing me something like following, when I console.log this:
code:
const StyleLintPlugin = require('stylelint-webpack-plugin');

plugins: [
        new webpack.HotModuleReplacementPlugin(),
        new webpack.NoErrorsPlugin(),
        new StyleLintPlugin({configFile: '.stylelintrc', context: 'app/styles/', files: '**/*.scss', failOnError: true }),
        new webpack.DefinePlugin({
          __DEVCLIENT__: true,
          __DEVSERVER__: false
        })
    ]

result:
plugins:
   [
       HotModuleReplacementPlugin {},
       NoErrorsPlugin {},
       { apply: [Function: bound apply] },
       DefinePlugin { definitions: [Object] },
   ]

Even if I console.log only this plugin then it's again showing me the same result:
code: console.log(new StyleLintPlugin({configFile: '.stylelintrc', context: 'app/styles/', files: '**/*.scss', failOnError: true }));
result: { apply: [Function: bound apply] }
I also looked into the implementation and found it's as per what Webpack plugin doc explains, at https://webpack.github.io/docs/plugins.html. Could you please let me know what is the fix for this?

@ManvendraSK sorry for the delay, were you able to find out what the problem is?

Closing for now, @ManvendraSK if you're still seeing issues please feel free to reopen with a repo that reproduces the problem. Thanks.


image/gif: decoding of some GIFs fails with error gif: invalid pixel value yet they open alright.

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go version devel +ea306ae Fri Apr 1 04:13:44 2016 +0000 darwin/amd64

What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN="/Users/emmanuelodeke/go/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/emmanuelodeke/go"
GORACE=""
GOROOT="/Users/emmanuelodeke/go/src/go.googlesource.com/go"
GOTOOLDIR="/Users/emmanuelodeke/go/src/go.googlesource.com/go/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"

What did you do?
If possible, provide a recipe for reproducing the error.

A link on play.golang.org is best.
Tried to decode a gif https://brobible.files.wordpress.com/2016/03/joker1.gif but it failed with error gif: invalid pixel value.
Complete runnable program here http://play.golang.org/p/_8PzZ6RffI

What did you expect to see?
Successful GIF decoding

What did you see instead?
error gif: invalid pixel value

It is indeed an invalid GIF image: the frame's palette only has 128 entries (i.e. valid values are from 0 to 127 inclusive), but the 'transparent index' is given as 128, which is outside the range [0, 127]. The GIF spec at https://www.w3.org/Graphics/GIF/spec-gif89a.txt says in section 22, Table Based Image Data, "Each index must be within the range of the size of the active color table".
Imagemagick similarly complains:
$ convert joker1.gif x.png
convert.im6: invalid colormap index `joker1.gif' @ error/colormap-private.h/ConstrainColormapIndex/34.
Nonetheless, it treats it as a warning, not an error. Furthemore, joker1.gif is viewable (with no visible error message) in both Firefox and Google Chrome, and by Eye-of-Gnome (which uses gdk-pixbuf). I suppose that it's a case of reality trumping the standard, and we should 'fix' this for Go.

Thank you @nigeltao, this helps me quite a lot as I was encountering a lot of such gifs on the web and needed to import them


Feature/#35

Description: 概要

Changes: 変更内容

Changes

feat: add prev and next post (334e866d3ce0804f2808131caacd1b48d5cd9ddd)
feat: implement links (ab41a6f3d5b68c452a8a03c36176eb2d6b4a26b3)
fix: storybook test (ed2f59568a55d2901503dec96736d27946849d3b)

Expected Impact: 影響範囲

Operating Requirements: 動作要件

Additional context: 補足

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/next-v0.8.2@8a31ea3). Click here to learn what that means.
The diff coverage is n/a.

@@                  Coverage Diff                   @@
##             release/next-v0.8.2     #103   +/-   ##
======================================================
  Coverage                       ?   43.64%
======================================================
  Files                          ?      156
  Lines                          ?     1274
  Branches                       ?      230
======================================================
  Hits                           ?      556
  Misses                         ?      693
  Partials                       ?       25

Continue to review full report at Codecov.

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


Added test for query string without path

/cc @halter73 @cesarbs

@cesarbs Turns out this explodes on the current kestrel 😄

This is one of the issues I'll address in https://github.com/aspnet/KestrelHttpServer/issues/1279

@davidfowl Do you want someone to update the davidfowl/added-bad-target-test branch so this test passes and then merge?

@halter73 I think it depends on how complicate the fix it. Goal was to reduce number of changes to this branch unless necessary. I found this while writing the state machine for dev. @natemcmaster claims that it'll be fixed with other scheduled work so I'm not sure if there's a short term thing we should do here.

Closing for now


Fix a typo in CHANGELOG.md

Landed in https://github.com/metarhia/common/commit/4632ee5683eedd515ddb6521b97ab1d6419d792c.


opt opt const expr exec in set operator

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


gateway changes

About the PR
stuff useful for downstream:

for destinations with closeable: true alt clicking an open destination gateway, with the correct access, will close it
added popup and sound for access denied
support different portal closing sound

also cleaned the code up a bit
Why / Balance
good
closing destinations might let you counter "ddosing" a portal idk
Technical details
m
Media
this gateway opened the destination

this one didnt

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
no
Changelog
no cl no fun

it was very fast, wow.  Thanks a lot!


B 17849 restrict records returned by prime simulator

Agility ticket
Summary
Prime Simulator is pulling too many records and is bogged down. Added filter on API call to pull only recent moves.
Urgent change requested by Dan Schuster
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the office app
Login as a prime
Page should now load only 20 records at a time

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Closing this PR while Logan and myself work on using a GHC endpoint for listing the moves instead of changing the return data structure of the prime listMoves endpoint. Stay tuned!

Test coverage on server failed.
Last run: Mon 11 Dec 2023 05:09:30 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.


modify the webpack and eslint

I think the webpack config and eslint is not good enough, I can modify it, but it can make the project's code change to lot, Do you want I do this?

I modified the code using es6 and make this project become beautiful. But I think I modified too much, so I created a new project to save it.
but in my opinions, I want to pr, so I think you can review in here. And if you think this is good enough, I will come up with pr.
Sincerely, I want you can accept my code.

Thanks for your interest!
You're absolutely right that es6 modules look a lot better and have advantages over require. Initially I made it with es6 modules, but babel uses Object.defineProperties when it transpiles the code, which breaks in IE8. Since this is a library and I can't control what other people need to support, I decided not to break IE8. Most of the other things that are ugly come from performance tweaking, some of which was made to apply to a previous generation of browsers.
For the whitespace there's a utility called smudge that you can add to your git attributes file to change the indentation on your working copy. This gist sets up your git configs to use it. https://gist.github.com/stutrek/5ab23a0e05ce391e251e83b769c4079e

If you clean things up and add some comments I'd really appreciate that, but you'll have to work from a fork. Github won't let you make a PR if the source and target don't share a common ancestor

@stephenkingsley if you'd like a simple contribution that you can do without having to reformat the code, etc. you can add this: stutrek/scrollMonitor#56
There's a good method of feature detection here:
https://github.com/WICG/EventListenerOptions/blob/gh-pages/explainer.md#feature-detection

actually, I fork a branch to do this at first. But I think that is not a simple contribution, so I save in a new project, and not come up with pr. I think I will contribution in this project, and meanwhile maintain in my project.


Changelog

Solves #64

Could you wrap the form in Filament in a Card? (Like the others, so it has a white BG)