Help: System.Exception: The specified service does not exist as an installed service.

How to run this lib through Task Scheduler?
It works perfectly if starting manually via compiled exe or calling the exe via cmd/powershell.
But in Task Scheduler Returns below error: "mediaManager.Start()"
System.Exception: The specified service does not exist as an installed service.
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at SystemWebMonitorInterface.MediaManager.Start() in C:\Projects\SystemMonitorWebInterface\bin\SystemWebMonitorInterface\SystemWebMonitorInterface\MediaManager.cs:line 63
at SystemWebMonitorInterface.Program.<>c.<b__12_2>d.MoveNext() in C:\Projects\SystemMonitorWebInterface\bin\SystemWebMonitorInterface\SystemWebMonitorInterface\Program.cs:line 213
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.QueueUserWorkItemCallback.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem()
at System.Threading.ThreadPoolWorkQueue.Dispatch()

I found a solution.
If the app started hidden, the above message happens, the workaround is to start the app and then hide (eg running on the web server).

I know exactly what you ran into. And yes, the only solution I know so far is to run it starting out in an 'interactive' mode (aka having a window). I don't believe there's any other workaround for it, if you do figure out something else let me know.
I'll look into specifying that into the requirements section. Thanks~

Thanks for your reply, the 'interactive' mode is a solution!
Now i can forward the info to my web socket


metrics: Be able to tell how much time each StoreAPI took time (actually)!

`grpc_server_handling_seconds is not helpful, because of how Querier works: It uses TCP flow-control for incoming stream. This means that slower Store e.g:

Does not really mean that this server is slow, it might also mean that:

Querier is processing (merging) series in mean time from other stores slowly
Another StoreAPI is slow and Queriers is waiting for its result.

The fact that StoreAPI is streaming also is important: The first frame might with lower latency, next might be slower etc
Some ideas:

Measure idle/wait time (spent on grpc send) - even better:  handling time WITHOUT waiting time (edited)
Buffer more on Querier (rather poor idea
?

Help wanted (:

Still applicable.
Kind Regards,
Bartek Płotka (@bwplotka)
On Tue, 27 Oct 2020 at 18:35, stale[bot]<EMAIL_ADDRESS>wrote:

Hello 👋 Looks like there was no activity on this issue for the last two
months.
Do you mind updating us on the status? Is this still reproducible or
needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be
closed (we can always reopen an issue if we need!). Alternatively, use
remind command https://probot.github.io/apps/reminders/ if you wish to
be reminded at some point in future.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/3097#issuecomment-717407397,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3OZIHLHWO3YA5ZV3EK3SM4AHPANCNFSM4QOC6ZLA
.


fail to load kinetics 400 video

Hello, I am deeply impressed by your excellent research and am eager to conduct some experiments. However, I am unable to proceed with experiments on the Kinetics 400 due to the same issue mentioned in this link (https://github.com/OpenGVLab/UniFormerV2/issues/13). Could you possibly share the dataset?

Please check
URL: https://pan.baidu.com/s/1W--PcVfY6zX8xhwgomO5nA
Password: 8lni


axis in negative values

Hi,
I have created a sample. The application is working but the axis goes in negative values.
Iam sending a plunker sample. Please help me.
http://plnkr.co/edit/e5UvCSAxLg93WSMURZG3?p=preview
Thanks & Regards,
Govardhan

It seems you committed an error in your jQuery selector line 9:
- var result = $("<div>").css({
+ var result = $("div").css({

This way, drops are drawn correctly. Moreover, the stylesheet is missing to get separator lines. Don't forget to add the https://unpkg.com/event-drops/dist/style.css style.


Fail to save tags for one emulator blob if use spaces as the key/value

Storage Explorer Version: 1.33.0-dev (96)
Build Number: 20240104.6
Branch: main
Platform/OS: Windows 10
Emulator Version: Azurite 3.28.0
Architecture: x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Install and run Azurite 3.28.0 -> Launch Storage Explorer.
Expand 'Local & Attached -> Storage Accounts -> (Emulator - Default Ports) (Key)'.
Blob Containers -> Open one blob container.
Select one blob-> Click 'Edit Tags...'.
Type spaces as the key or value -> Click 'Save'.
Check whether succeeds to save tags.

Expected Experience
Succeed to save tags.
Actual Experience
Fail to save tags.

Additional Context
This issue doesn't reproduce for regular blobs.

@JasonYeMSFt confirm if this is an Azurite bug, and if so, open an issue against them with details and then move this to Future.

Opened an Azurite bug https://github.com/Azure/Azurite/issues/2336


--no-firsth1 option causes "undefined" words to appear in the toc

Example without the option:
$ npx markdown-toc README.md
- [my-project](#my-project)
- [Getting started](#getting-started)
- [Usage](#usage)
  * [Example](#example)
- [API](#api)
  * [The `` element](#the--element)
    + [Attributes/Properties](#attributesproperties)
    + [Methods](#methods)
    + [Events](#events)
- [More Examples](#more-examples)
  * [Usage with [`@lume/element`](https://github.com/lume/element)](#usage-with-lumeelementhttpsgithubcomlumeelement)
  * [Usage with React](#usage-with-react)
  * [Usage with React and TypeScript](#usage-with-react-and-typescript)
  * [Usage with other view libs/frameworks](#usage-with-other-view-libsframeworks)
- [Development](#development)
  * [Test it in the examples](#test-it-in-the-examples)
  * [Test it in another project](#test-it-in-another-project)

Example with the option:
$ npx markdown-toc --no-firsth1 README.md
undefined [Getting started](#getting-started)
undefined [Usage](#usage)
- [Example](#example)
undefined [API](#api)
- [The `` element](#the--element)
  * [Attributes/Properties](#attributesproperties)
  * [Methods](#methods)
  * [Events](#events)
undefined [More Examples](#more-examples)
- [Usage with [`@lume/element`](https://github.com/lume/element)](#usage-with-lumeelementhttpsgithubcomlumeelement)
- [Usage with React](#usage-with-react)
- [Usage with React and TypeScript](#usage-with-react-and-typescript)
- [Usage with other view libs/frameworks](#usage-with-other-view-libsframeworks)
undefined [Development](#development)
- [Test it in the examples](#test-it-in-the-examples)
- [Test it in another project](#test-it-in-another-project)

@trusktr thank you for the issue and the example. Will you also include the source README.md to help determine where the bug is occurring?

It happens only when the file contains <!-- toc -->. Sample file : README.md.txt.
Without <!-- toc --> :
mycomputer$ markdown-toc --no-firsth1 README.md
- [About](#about)
- [Requirements & installation](#requirements--installation)
  * [Python package installation](#python-package-installation)
  * [Installation from source](#installation-from-source)
  * [Docker usage](#docker-usage)
...

With <!-- toc --> :
mycomputer$ markdown-toc --no-firsth1 README.md
undefined [Requirements & installation](#requirements--installation)
- [Python package installation](#python-package-installation)
- [Installation from source](#installation-from-source)
- [Docker usage](#docker-usage)
...

Also it's probably a duplicate of #42 and #131


[Metric Framework] Adds support for Histogram metric

Description
As part of the https://github.com/opensearch-project/OpenSearch/issues/10238 we have added the Metrics Framework and support for just the counter type of metrics. We also need to add Histogram support to record the latency metrics also.
Related Issues
Resolves #12061
[#11884]
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta, Please take a look. I will add tests in the next commit.

Thanks @Gaganjuneja , please:

add CHANGELOG entry
add documentation issue (I think we have none for metrics part right but we should document that, please correct me if I am wrong)

Thanks @Gaganjuneja , please:

add CHANGELOG entry
add documentation issue (I think we have none for metrics part right but we should document that, please correct me if I am wrong)

Yes, Documentation, I am doing and will add in the another commit.

Thanks @Gaganjuneja , please:

add CHANGELOG entry
add documentation issue (I think we have none for metrics part right but we should document that, please correct me if I am wrong)

Create the documentation issue - https://github.com/opensearch-project/documentation-website/issues/6338

@Gaganjuneja please resolve the conflicts, thank you


Check for previous/existing GitHub issues/module proposals

[x] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[x] I have checked for that this module doesn't already exist in the module indexes.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/insights/auto-scale-setting
Module Details
Main goal is to introduce the AVM Insights/Autoscalesettings will allow VMSS to use this module to configure autoscale.
The Microsoft.Insights/autoscalesettings resource type in Azure is used to configure autoscale settings for various Azure resources. Autoscale settings help ensure that you have the right amount of resources running to handle the fluctuating load of your application. You can configure autoscale settings to be triggered based on metrics that indicate load or performance, or triggered at a scheduled date and time.
Key Properties:

name: The name of the autoscale setting.
location: The location where the autoscale setting is deployed.
tags: A list of key-value pairs that describe the resource.
properties: This includes several sub-properties such as:

enabled: A boolean indicating whether the autoscale setting is enabled.
notifications: An array of notification settings, including email and webhook configurations.
predictiveAutoscalePolicy: Settings for predictive autoscale, including look-ahead time and scale mode.
profiles: An array of profiles that define the autoscale rules. Each profile includes:

capacity: Defines the minimum, maximum, and default number of instances.
fixedDate: Specifies the start and end date for the profile.
recurrence: Defines the recurrence schedule for the profile.
rules: An array of rules that define the conditions for scaling. Each rule includes:

metricTrigger: Defines the metric that triggers the scaling action.
scaleAction: Defines the action to take when the metric trigger condition is met.

Example Bicep Template:
resource symbolicname 'Microsoft.Insights/autoscalesettings@2022-10-01' = {
  name: 'string'
  location: 'string'
  tags: {
    tagName1: 'tagValue1'
    tagName2: 'tagValue2'
  }
  properties: {
    enabled: bool
    name: 'string'
    notifications: [
      {
        email: {
          customEmails: ['string']
          sendToSubscriptionAdministrator: bool
          sendToSubscriptionCoAdministrators: bool
        }
        operation: 'Scale'
        webhooks: [
          {
            properties: {
              {customized property}: 'string'
            }
            serviceUri: 'string'
          }
        ]
      }
    ]
    predictiveAutoscalePolicy: {
      scaleLookAheadTime: 'string'
      scaleMode: 'string'
    }
    profiles: [
      {
        capacity: {
          default: 'string'
          maximum: 'string'
          minimum: 'string'
        }
        fixedDate: {
          end: 'string'
          start: 'string'
          timeZone: 'string'
        }
        name: 'string'
        recurrence: {
          frequency: 'string'
          schedule: {
            days: ['string']
            hours: [int]
            minutes: [int]
            timeZone: 'string'
          }
        }
        rules: [
          {
            metricTrigger: {
              dimensions: [
                {
                  DimensionName: 'string'
                  Operator: 'string'
                  Values: ['string']
                }
              ]
              dividePerInstance: bool
              metricName: 'string'
              metricNamespace: 'string'
              metricResourceLocation: 'string'
              metricResourceUri: 'string'
              operator: 'string'
              statistic: 'string'
              threshold: int
              timeAggregation: 'string'
              timeGrain: 'string'
              timeWindow: 'string'
            }
            scaleAction: {
              cooldown: 'string'
              direction: 'string'
              type: 'string'
              value: 'string'
            }
          }
        ]
      }
    ]
    targetResourceLocation: 'string'
    targetResourceUri: 'string'
  }
}

This template provides a comprehensive way to define autoscale settings for your Azure resources, ensuring that your applications can handle varying loads efficiently. For more detailed information, you can refer to the official documentation https://learn.microsoft.com/en-us/azure/templates/microsoft.insights/autoscalesettings
Do you want to be the owner of this module?
Yes
Module Owner's GitHub Username (handle)
FallenHoot
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

Hi @FallenHoot,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER

Hi @FallenHoot,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
Before starting development, please ensure ALL the following requirements are met.
Please use the following values explicitly as provided in the module index page:

For your module:

ModuleName - for naming your module
TelemetryIdPrefix - for your module's telemetry

For your module's repository:

Repo name and folder path are defined in RepoURL
Create GitHub teams for module owners and contributors and grant them permissions as outlined here.
Grant permissions for the AVM core team and PG teams on your GitHub repo as described here.

Check if this module exists in the other IaC language. If so, collaborate with the other owner for consistency. 👍
You can now start the development of this module! ✅ Happy coding! 🎉
Please respond to this comment and request a review from the AVM core team once your module is ready to be published! Please include a link pointing to your PR, once available. 🙏
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team


querier for RestoreForState not closed.

Not confirmed, but most likely the cause for: https://github.com/prometheus/prometheus/issues/4842
this single line consumed 3 days of troubleshooting and drilling down of all layers.
cc @gouthamve, @codesome
Signed-off-by: Krasi Georgiev<EMAIL_ADDRESS>

Will this fix be in 2.4.x version?

@AndorCS can you please confirm that it fixes your issue as well? I pinged you on IRC that I have pushed a new image for you to test with tag "close-querier"


Updated groupwise registration docs.

Hi Kasper,
I've made some small changes to the groupwise registration docs. This is my first pull request on github so let me know if there is anything I can do differently.
Thanks,
Anthony.

Hi Anthony, the changes are perfect, good job! One thing you could do is change the commit message to match the style of this repository and be a little more specific. This basically boils down to "DOC: Updated groupwise registration docs with note on coordinate tolerance error" or similar. You can reword the commit message using interactive rebasing as described here. Also, if you point the pull request to "develop" branch that would be great (more control over what goes in to mainline SimpleElastix etc and readthedocs.org are actually watching the develop branch).


Add method on Activity to create a reply

Creating a reply requires the developer to set a specific set of properties on an activity.
I figured it out the hard way, by looking at the C# implementation of Activity.CreateReply.
I would be nice to have a createReply method on the Activity class that I can use instead.

Review C# implementation and either copy or remove both.


Re-write code base

The code has grown dramatically compared to the initial days of this script. I think we will be better off splitting the code into multiple files before it grows enormous and becomes hard to understand.
See #72 for progress.

Merged #72


IntentDialog session.send message don't appear to reach Slack

I recently upgraded my bot from a BotConnectorBot to a UniversalBot. I used to have good connectivity with Slack but since upgrading I'm not getting replies to show. I'm using an intentDialog which abbreviates like this:
intents.matches('WhatVersion', [
  function(session, args) {
    console.log("What version is invoked");
    session.send("I'm running version %s", version);
  }]);

I'm hosting in Azure (WebApp) and after turning NODE_DEBUG to "botbuilder" I get logs which say:

restify listening to http://undefined:undefined
ChatConnector: message received.
ChatConnector: message received.
session.beginDialog(/)
/ - IntentDialog.begin()
OnBeginIntent
/ - session.sendBatch() sending 0 messages
/ - IntentDialog.matches(WhatVersion)
/ - waterfall() step 0 of 1
What version is invoked
/ - session.send()
/ - session.sendBatch() sending 1 messages

There are no message appearing in Slack at all.
I see the events coming from AppInsights which show "success" which are all of the type SendChannelActivityToBot - I'm not sure if this is expected or we should also expect replies.

Really I'm a little stuck; can anyone suggest a path for further debugging or instrumentation?

When upgrading - there's  a very subtle change to how connectors are instantiated; the property appSecret becomes appPassword.
This corrected my error, which is thus about authentication, but there are no errors reported anywhere.


Rename fc.util.dataJoin().attrs to attr

In keeping with d3.selection().attr

As attr supports (key, value), is it worth updating the single-valued usages to this syntax?

I can definitely do that
On Fri, 7 Aug 2015 06:51 Colin Eberhardt<EMAIL_ADDRESS>wrote:

As attr supports (key, value), is it worth updating the single-valued
usages to this syntax?
—
Reply to this email directly or view it on GitHub
https://github.com/ScottLogic/d3fc/pull/512#issuecomment-128606224.

Done :)

Done :)

awesome


Better error message on foreign export

According to @NCrashed:

Just to note, the current implementation fails on the:
import Primes
import GHC.Base
foreign export java takePrime :: Int -> Java a Int
takePrime i = return . fromIntegral $ primes !! i
primes = filterPrime [2..]
where filterPrime (p:xs) =
p : filterPrime [x | x <- xs, x mod p /= 0]
With:
ghcvm: panic! (the 'impossible' happened)
(GHCVM version 0.0.1):
No type variables for the Java foreign export!

The panic should be replaced with a notice that you cannot have a quantified variable in the tag position of the Java monad for exports (yet). This should be done in GHCVM.TypeCheck.TcForeign.

why can't it be done in compiler/ETA/Utils/Panic.hs just by changing the panic output or maybe by giving the sorry output instead[1]? "sorry! (unimplemented feature or known bug)\n"

@tibi77 So the idea is to give a "human-readable" error that clearly states what mistake the programmer made and how they can fix it. I think you might have misunderstood it.

Did this get fixed in #341 ? :smile:

@rahulmutt yeah, it seems it can be closed

Great, closing!


AlignTemplatedTestCases doc page uses incorrect transformer name

Page uses incorrectly:  AlignTestCases  and AlignSettingsSection
In all cases it should be AlignTemplatedTestCases

I've read documentation at least few times yesterday and still somehow missed it.. thanks for being the second pair of the eyes :)


Unique email for 2 roles differently

Hi,
I want add a new role Blogger and want to have 2 users (one with role User and one with role Blogger) with same email id but password's different, how to obtain that?
Thanks,
Duc

Hi @ducnguyenhuy, This looks like a question about Devise, please search the Wiki or use the Mailing List or Stack Overflow. If this is a bug report then please provide a detailed report and full backtrace. Thanks!

ok, thanks for mailing list. will post now and close this.

I just posted: https://groups.google.com/forum/#!topic/plataformatec-devise/OABANrOST48. Please reply if you can help. Thanks.


Fix memory leak caused by multiple redirect.

When processing multiple redirects Request will init more than once.
the end event register every time when the Request.prototype.start, also
the same with pipe event on Request.prototype.init.
you'll see the test scripts here: https://github.com/belltoy/request-redirect-test

@nylen We probably could make your example a private function instead of on the prototype.
Unless we intend to expose that functionality we could just do:
function addHandlerOnce(ev, cb) {
  var self = this
  if (!self._addedHandler[ev]) {
    self.on(ev, cb)
    self._addedHandler[ev] = true
  }
}

addHandlerOnce.call(self, ev, cb)

or
function addHandlerOnce(context, ev, cb) {
  var self = context
  if (!self._addedHandler[ev]) {
    self.on(ev, cb)
    self._addedHandler[ev] = true
  }
}

addHandlerOnce(self, ev, cb)

@nylen We probably could make your example a private function instead of on the prototype.
Unless we intend to expose that functionality we could just do:
function addHandlerOnce(ev, cb) {
  var self = this
  if (!self._addedHandler[ev]) {
    self.on(ev, cb)
    self._addedHandler[ev] = true
  }
}

addHandlerOnce.call(self, ev, cb)

or
function addHandlerOnce(context, ev, cb) {
  var self = context
  if (!self._addedHandler[ev]) {
    self.on(ev, cb)
    self._addedHandler[ev] = true
  }
}

addHandlerOnce(self, ev, cb)

Overall I would say that the init method should be refactored more.
Not this PR's concern but it would make these changes more maintainable as well as others.

Overall I would say that the init method should be refactored more.
Not this PR's concern but it would make these changes more maintainable as well as others.

@nylen @seanstrom Can we have redirects clear all listeners before calling init again? That way we would consolidate cleanup in one place, instead of scattering it around the module.
@seanstrom +1000 :)

That would break any consumers who are listening for those events.  request-debug doesn't use these 2 events but it uses a similar approach, and we encourage wrapping request like this.

Any status update about this bug? It is still exists on the latest stable.

This PR has been closed as "abandoned" as a part of an automated cleanup. If this is a mistake and you are still interested in merging this PR, please do the following:

[ ] Rebase your changes onto the master branch (even if there are no merge conflicts, we want to make sure your change is under the most recent tests).
[ ] Push your updated branch to Github to update this PR.
[ ] Make sure this PR's Travis tests are still passing. (Run npm test locally if tests are still failing).
[ ] Comment here to let us know that you've updated the PR and are still interested in getting it merged. Somebody from the project will respond and work with you to review.

Thank you for your understanding. If you have any questions, please check out https://github.com/request/request/issues/2556 and feel free to comment / ask anything there.


655 migration override userprofile

This PR updates the ingest code so that after resource creation, we overwrite the owner's creator information for the resource with what was in the bag (back when the resource was last edited), rather than what is in the user profile (at the time of ingest).

@pkdash @hyi Can you review?  @mjstealey Can we try this in another dry run tomorrow (Friday the 9th)?

@hyi Thanks for the suggestion to use filter instead of a linear search.  @pkdash Can you review when you have a chance?  We'd like to do another dry run this afternoon.  Thanks.

@pkdash The ingestion code does not currently try to read researcherID or researchGateID from the XML because I didn't have an example resource with these fields to test with.  I don't believe it is currently possible to create these fields in the web interface, but if it is possible, please let me know how and I will add this.

@pkdash I fixed the researchGageID typo.  I also fixed a limitation in the SAX parser (used to ensure some elements are parsed in the order encountered in the resourcemetadata.xml).  Please let me know if this can be merged.  Thanks.

+1


[WIP][SPARK-3359][BUILD][DOCS] More changes to resolve javadoc 8 errors that will help unidoc/genjavadoc compatibility

What changes were proposed in this pull request?
This PR only tries to fix things that looks pretty straightforward and were fixed in other previous PRs before.
This PR roughly fixes several things as below:

Fix unrecognisable class and method links in javadoc by changing it from [[..]] to `...`
[error] .../spark/sql/core/target/java/org/apache/spark/sql/streaming/DataStreamReader.java:226: error: reference not found
[error]    * Loads text files and returns a {@link DataFrame} whose schema starts with a string column named

Fix an exception annotation and remove code backticks in @throws annotation
Currently, sbt unidoc with Java 8 complains as below:
[error] .../java/org/apache/spark/sql/streaming/StreamingQuery.java:72: error: unexpected text
[error]    * @throws StreamingQueryException, if this query has terminated with an exception.

@throws should specify the correct class name from StreamingQueryException, to StreamingQueryException without backticks. (see JDK-8007644).

Fix [[http..]] to .
-   * [[https://blogs.oracle.com/java-platform-group/entry/diagnosing_tls_ssl_and_https Oracle
-   * blog page]].
+   *
+   * Oracle blog page.

[[http...]] link markdown in scaladoc is unrecognisable in javadoc.

It seems class can't have @return annotation. So, two cases of this were removed.
[error] .../java/org/apache/spark/mllib/regression/IsotonicRegression.java:27: error: invalid use of @return
[error]    * @return New instance of IsotonicRegression.

Fix < to < and > to > according to HTML rules.

Fix
 complaint

Exclude unrecognisable in javadoc, @constructor, @todo and @groupname.

How was this patch tested?
Manually tested by jekyll build with Java 7 and 8
java version "1.7.0_80"
Java(TM) SE Runtime Environment (build 1.7.0_80-b15)
Java HotSpot(TM) 64-Bit Server VM (build 24.80-b11, mixed mode)

java version "1.8.0_45"
Java(TM) SE Runtime Environment (build 1.8.0_45-b14)
Java HotSpot(TM) 64-Bit Server VM (build 25.45-b02, mixed mode)

Note: this does not yet make sbt unidoc suceed with Java 8 yet but it reduces the number of errors with Java 8.

I should double check the built javadoc. Will leave some images in the changes. BTW, this still does not fully resolve the problem (cc @srowen).

Test build #69105 has started for PR 15999 at commit cfce0e8.

Test build #69105 has finished for PR 15999 at commit cfce0e8.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

retest this please

Test build #69109 has started for PR 15999 at commit cfce0e8.

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

Merged build finished. Test FAILed.

retest this please

Test build #69119 has started for PR 15999 at commit cfce0e8.

Test build #69122 has started for PR 15999 at commit b0c805c.

Test build #69119 has finished for PR 15999 at commit cfce0e8.

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

Merged build finished. Test PASSed.

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

Oh thank you! Will double check and change the title tomorrow.

Test build #69122 has finished for PR 15999 at commit b0c805c.

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

Merged build finished. Test PASSed.

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

Let's get this in for 2.1, even if it doesn't fix 100% of the errors. This looks like a big step towards making it work with javadoc 8. Let me know when ready @HyukjinKwon

I just double-checked and I think it is ready. Thank you @srowen.

Test build #69153 has started for PR 15999 at commit 297783d.

Test build #69153 has finished for PR 15999 at commit 297783d.

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

Merged build finished. Test PASSed.

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

Merged to master/2.1


Check and show health state of K8s Namespace resource

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #1610
Does this PR introduce a user-facing change?:

NONE

Just left the last nit.
/lgtm

/lgtm

Way to go!
/approve


Bug when not connected to Redis

I found a little bug.. when using the RedisBackend as a backend class, and when Redis is not running then no error will be raised in the app, but worst is that the rate limiter has no effect at all! All requests come through. Imagine my local Redis going down unexpectedly and my API becomes unlimited for everyone :scream:
I looked at aredis constructor for StrictRedis and it does not look like one can pass anything to check for available/existing connection.
Redis's official way of ensuring for connection is the ping command.
I think the best solution here is to allow users to pass already established Redis connection to backend for RateLimitMiddleware ?
Or do you have a better idea?
Something like:
from aredis import StrictRedis
from fastapi import FastAPI
from ratelimit import RateLimitMiddleware

app = FastAPI()
my_redis_conn = StrictRedis()

@app.on_event("startup")
async def startup():
    # either command will raise ConnectionError
    await my_redis_conn.get(None)
    # or await my_redis_conn.ping()

app.add_middleware(
    RateLimitMiddleware,
    authenticate=my_auth,
    backend=RedisBackend(conn=my_redis_conn),
    config={
        ...
    },
)

With passing my_redis_conn we're reusing the connection - no need to make another one.
Obviously, there's some little edge case scenario. E.g. Redis & app is running, and then Redis goes down after X days & the rate limiter will again allow all requests.
So maybe it would be better to overwrite allow_request for Redis Backend?

Okay, got thinking about it again... maybe it's not a bug and somehow an intended behaviour, however if I decide to rate limit my API then I would rather expect the service to throw error & go down as a whole than allow all requests without limits.
Which brings me to the point that this package does not need extra checks etc. But I still think that sharing a Redis connection (by passing it down to asgi-ratelimit) would be a nice addition.

Okay, got thinking about it again... maybe it's not a bug and somehow an intended behaviour, however if I decide to rate limit my API then I would rather expect the service to throw error & go down as a whole than allow all requests without limits.
Which brings me to the point that this package does not need extra checks etc. But I still think that sharing a Redis connection (by passing it down to asgi-ratelimit) would be a nice addition.

In my test, after the crash of the contract, any request to pass through the flow limiter should be thrown an exception.
Can you give me a snippet of code that reproduces the bug?

In my test, after the crash of the contract, any request to pass through the flow limiter should be thrown an exception.
Can you give me a snippet of code that reproduces the bug?

I'm sorry - looks like I had a bug in my own code :facepalm: It throws ConnectionError nicely. Closing down.

I'm sorry - looks like I had a bug in my own code :facepalm: It throws ConnectionError nicely. Closing down.


[e2e]: container tag branch1 default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v27.0.396
Run: https://github.com/slsa-framework/example-package/actions/runs/8422592333
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.tag.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.tag.branch1.default.slsa3.yml
Trigger: push
Branch: v27.0.396
Date: Mon Mar 25 15:33:10 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v27.0.397
Run: https://github.com/slsa-framework/example-package/actions/runs/8430174842
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container.tag.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container.tag.branch1.default.slsa3.yml
Trigger: push
Branch: v27.0.397
Date: Tue Mar 26 03:06:33 UTC 2024
Tests are passing now. Closing this issue.


Basic Recursive algorithms in C++

Is your feature request related to a problem? Please describe.
The folder for Recursive algorithm in C++ language does not contain this basic algorithms
Describe the solution you'd like
I would like to add the following algorithms which are the fundamentals of Recursion:

Recursively finding Fibonacci Numbers
Recursively finding Greatest Common Divisor (GCD) of given two numbers
Recursively finding Factorial of a number
and many others....

Describe alternatives you've considered
Additional context
@Kumar-laxmi  As I am doing Competitive coding for quite some time now and have solved many question based on this algorithm so I request you to kindly assign me this under SSOC'23.

4 Star on Codechef
Specialist on Codeforces

Request for adding Java implementation for algorithms mentioned above in SSOC-23 to enhance the aesthetic appeal of the Algorithm's repository.
Admin- @Kumar-laxmi

@AK3847 Kindly open one issue per algorithm, you have mentioned 3 algorithms in same issue


suggestion: use a random AppName on 'odo create'

odo create picks a random name for the component, but it uses a fixed name for the AppName of app.
Is there a reason the name doesn't get randomized?
I'm trying out odo create when I push a second unrelated component, I get an error because they are both using the same AppName:
Error: Failed to create the component: Failed to update config to component deployed.: url named "http-8080" already exists in the same app named "app"

cc @kadel

Is there a reason the name doesn't get randomized?

We want to make sure that the URL names are somewhat predictable. When you do odo create, odo push, odo delete odo push, the second odo push should create the same resources as the first one.

Error: Failed to create the component: Failed to update config to component deployed.: url named "http-8080" already exists in the same app named "app"

This is a problem, odo needs to make sure that the url names don't collide
/kind bug
/priority medium

Closing this since it's not relevant anymore.
/close


Azure Functions from VSCode with a Linux Consumption Plan not adding Remote build App Settings on Linux

Azure Functions from VSCode with a Linux Consumption Plan and deploy the code, it does not create those settings automatically. It only create these:

When I perform the same operations with a Premium Consumption Plan It creates those settings automatically and everything works great.

Ensure the when using  Linux Consumption Plan adds the App Settings during deployment.
•	ENABLE_ORYX_BUILD=true
•	SCM_DO_BUILD_DURING_DEPLOYMENT=true

I'm pretty sure remote build is the default for Linux Consumption, meaning we don't need those settings. Does it actually fail to build? If I create a consumption app from VS Code, I only see these settings:

But when I deploy, I see logs like "Running oryx build..." and "Running pip install..." implying it is doing a build.

For a internal MSFT they were experiencing this issue the build failed to load binaries during the deployment.
From my research :
I would verify if you have these app settings in your function app.
Deployment technologies in Azure Functions | Microsoft Docs -
Remote build on Linux
To enable remote build on Linux, the following application settings must be set:
•	ENABLE_ORYX_BUILD=true
•	SCM_DO_BUILD_DURING_DEPLOYMENT=true
By default, both Azure Functions Core Tools and the Azure Functions Extension for Visual Studio Code perform remote builds when deploying to Linux. Because of this, both tools automatically create these settings for you in Azure.
When apps are built remotely on Linux, they run from the deployment package.
Their findings Notes:
We’re performing the deployment from Visual Studio Code with this configuration:
{
"azureFunctions.deploySubpath": ".",
"azureFunctions.projectLanguage": "JavaScript",
"azureFunctions.projectRuntime": "~3",
"debug.internalConsoleOptions": "neverOpen",
"azureFunctions.preDeployTask": "npm prune",
"azureFunctions.scmDoBuildDuringDeployment": true
}
It works with Premium Consumption Plan (EP1) but It doesn’t with Linux Consumption Plan (Y1)
I tried with GitHub Actions and a linux runner and It works for a Linux Consumption Plan. So, I assume it’s a bug in Visual Studio Code/Azure Function tools…
Another MSFT Feedback:
I don't believe VS Code does remote build by default. It does, however, initialize a settings.json file with a default scmDoBuildDuringDeployment value based on language. I think Python is automatically configured for remote build. For Node, you need to opt in.
Try removing the pre/post deployment tasks in settings.json and ensure node_modules is in .funcignore.
Take a look at this sample: functions-headless-chromium/puppeteer at main · anthonychu/functions-headless-chromium (github.com) The remote build from VS Code worked for me, and the resulting app does not have the SCM do build setting.
Now not sure if it supposed to be by default or not.

Can you show logs of where the build failed?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Pvp Doesnt work

Since the new update im having some problems on my server with the pvp, some players sometime can kiis others and sometimes not...

Yup the same is happening to my server. some players are taking damage while others don't take any kind of damage, while other take damage,
im not sure if its the rocket problem or unturned patch problem.

Can u please test this on vanilla before you report this, to make sure if its a rocket problem!?

i think its not a rocket problem, beacaused i talked with otheres host server, and everyone is gettung this, so i think i really a unturned bug with the new version <IP_ADDRESS>, almost sure that isnt rocket but is a unternd bug

im having the same issue, but if the player dies or suecide they go back to normal until reconect


Fixes the unbounded read test for the nightly job.

The current solution os the use a KeyedProcessFunction and use the processElement() method to count the number of records that have been processed.
Whenever the expected count is reached, the method logs. Count exceed is also logged.
The job is terminated after a timeout of 18 minutes.
Since KeyedProcessFunction is used for the counting, same key is used i.e a key named "commonKey" is added to the records to make sure they are received together.
To solve the problem of parallel operator, parallelism and number of task slots have been set as 1.

To check for the job success:

We check if there are any logs pertaining to exceeded count. In such a case a failure is reported.
In case of absence of "count-exceeded" logs, we search for the actual count read. We then check this count with the number of rows in the BQ table. In case of a match, the job succeeds.
In case a less number of records are read:

We check if the records are actually absent by retrying to search for the logs.
In case after 5 retries the records are still absent, then we raise an exception indicating that the logs are actually not present.

/gcbrun

1. We check if the records are actually absent by retrying to search for the logs.
2. In case after 5 retries the records are still absent, then we raise an exception indicating that the logs are actually not present.

We don't need this if the wait has been increased to 1 minute. I believe the logs are found after a minute, and retries are not required. Please correct me if I'm wrong.

1. We check if the records are actually absent by retrying to search for the logs.
2. In case after 5 retries the records are still absent, then we raise an exception indicating that the logs are actually not present.

We don't need this if the wait has been increased to 1 minute. I believe the logs are found after a minute, and retries are not required. Please correct me if I'm wrong.

Yes, this is fixed now.


Fix workflow

Fixes the nightly and merge workflows from not running.
Proof here: https://github.com/huggingface/accelerate/actions/runs/5338157693
Which also, we currently have a failure on multi-GPU on both main and pypi versions of accelerate, which I'll look at this week.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


vUSD

We would like to extend the use case of PT as well as allow staking SUI with stablecoins and issue vUSD that earns staking rewards without the risk of volatile SUI price.
At a high level, a portion of the input will be used for delta hedging on a third-party external platform. This feature may need to set a cap.

On hold until the public alpha version is completed


What is CURL_1P3A

After digging into your code, I am unsure of where you obtain CURL_1P3A env var from.
I know there is an api.1point3acre[s].com, but without any documentation it seems unhelpful on how to decipher the proper headers to use.

It requires some MITM techniques to extract the CURL_1P3A data from the app. :-P

I see. I've followed others' suggestions to get the cookies once you login and view the network requests
Do we simply need to recreate this get request using curl?

it's not the same. It should be acquired in the app not the web.

There is a curl request sent by the app labeled "CURL_1P3A" that we have to intercept and copy?
I haven't ever used the app so I will need to dig into that.

Gotcha haha. If you update this repo I will be excited!!


If Expanded is a child of Padding it breaks

Broken example
class UserText extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Padding(
      padding: EdgeInsets.all(10.0),
      child: Expanded(
        child: Column(
          crossAxisAlignment: CrossAxisAlignment.start,
          children: <Widget>[
            Text("@username",
              style: TextStyle(
                fontSize: 24.0,
                fontWeight: FontWeight.w500
              )
            ),
            Text("I love tokenized assets, this is probably too long!",
              overflow: TextOverflow.clip,
              softWrap: true,
              style: TextStyle(fontSize: 18.0)
            )
          ],
        )
      )
    );
  }
}

Working example
class UserText extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Expanded(
      child: Padding(
        padding: EdgeInsets.all(10.0),
        child: Column(
          crossAxisAlignment: CrossAxisAlignment.start,
          children: <Widget>[
            Text("@username",
              style: TextStyle(
                fontSize: 24.0,
                fontWeight: FontWeight.w500
              )
            ),
            Text("I love tokenized assets, this is probably too long!",
              overflow: TextOverflow.clip,
              softWrap: true,
              style: TextStyle(fontSize: 18.0)
            )
          ],
        )
      )
    );
  }
}

What does "it breaks" mean exactly?
Can you please also add the output of flutter doctor?

Padding parent of Expanded

Expanded parent of Padding

Adding the output of flutter doctor is requested in the issue template and it's always a good idea to do so ;-)

Lukes-iMac:~ lukepighetti$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, v0.0.0-unknown, on Mac OS X 10.13.5 17F77, locale en-US)
[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.
[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
[✗] Android Studio (not installed)
[!] VS Code (version 1.24.1)
[✓] Connected devices (2 available)

! Doctor found issues in 3 categories.

@HansMuller

I don't believe Expanded is meant to work when it isn't a direct descendant of Row, Column, or Flex. From the DartDoc:

Using an Expanded widget makes a child of a Row, Column, or Flex expand to fill the available space in the main axis (e.g., horizontally for a Row or vertically for a Column). If multiple children are expanded, the available space is divided among them according to the flex factor.

If you look at the yellow text in the red widgets in that screenshot, I believe that's what it's telling you (should also be in the logs).

Sounds like a won't-fix? :)

@RedBrogdon  is correct, the framework is working as intended in this case. It's doing its best to tell you that Expanded must be a child of a Row, Column, or Flex.

FWIW, the message is displayed (more readably) in the console, too.
If the message wasn't clear, please let me know.

So, one question: If I am not allowed to use Expanded() in a padding widget, how can I expand my Row inside a padding Widget?

I got a notification on this for some reason and I just want to confirm that this definitely is working as expected and should be left closed.


Release 0.84.0

Adding automating GitHub releases here.
Add a develop branch after this is merged to master.

skip_tags enabled in Appveyor settings


Add full service url on deployment to home page demo

For a newbie, it's nice to know the full url to the service I just deployed. Unless I am missing something [that already exists in the demo], I have to know how to translate "{username}/hello@dev" to "https://{username}.stdlib.com/hello@dev/".

We'll be overhauling the onboarding demo shortly, and will keep this in mind. :) Thanks!


Rename default branch from secp256k1-zkp to master

IIRC that's a remnant of the time when secp256k1-zkp was rebased on upstream. But I think it's confusing now.

+1

+1

Done. Deleted secp256k1-zkp to make clear that this isn't going to be updated.

Done. Deleted secp256k1-zkp to make clear that this isn't going to be updated.

We need to update travis: https://travis-ci.org/github/ElementsProject/secp256k1-zkp/jobs/754182998#L214
(Well, we anyway need to switch away but let's update it as long as it's still up.)

We need to update travis: https://travis-ci.org/github/ElementsProject/secp256k1-zkp/jobs/754182998#L214
(Well, we anyway need to switch away but let's update it as long as it's still up.)

We need to update travis: https://travis-ci.org/github/ElementsProject/secp256k1-zkp/jobs/754182998#L214
(Well, we anyway need to switch away but let's update it as long as it's still up.)

Ah no, it just works: https://travis-ci.org/github/ElementsProject/secp256k1-zkp/builds/754177884 The hiccup was just due to the branch being deleted during the travis run...

We need to update travis: https://travis-ci.org/github/ElementsProject/secp256k1-zkp/jobs/754182998#L214
(Well, we anyway need to switch away but let's update it as long as it's still up.)

Ah no, it just works: https://travis-ci.org/github/ElementsProject/secp256k1-zkp/builds/754177884 The hiccup was just due to the branch being deleted during the travis run...


Fix docs

saw a small mistake in the docs, here's the pr

If i find any small problems in the docs, would you like 1 pr containing them all or multiple small pr's based on the file in the docs?

If it are smallish fixes, (grammar, wording, spacing, ...) feel free to send them all in one PR.


app_store_connect_api_key doesn't work with key_content

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
When trying to create an app_store_connect_api_key using key_content upload_to_testflight will result in an error.
However, providing the key with key_filepath does work.
Command executed
app_store_connect_api_key(
  in_house: true
)

The rest is set with ENV VARS
Complete output when running fastlane, including the stack trace and command used
/usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/spaceship/lib/spaceship/connect_api/token.rb:57:in `initialize': [!] invalid curve name (OpenSSL::PKey::ECError)
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/spaceship/lib/spaceship/connect_api/token.rb:57:in `new'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/spaceship/lib/spaceship/connect_api/token.rb:57:in `create'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/pilot/lib/pilot/manager.rb:33:in `api_token'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/pilot/lib/pilot/manager.rb:20:in `login'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/pilot/lib/pilot/manager.rb:16:in `start'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/pilot/lib/pilot/build_manager.rb:17:in `upload'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/fastlane/lib/fastlane/actions/upload_to_testflight.rb:27:in `run'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/fastlane/lib/fastlane/runner.rb:263:in `block (2 levels) in execute_action'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/fastlane/lib/fastlane/actions/actions_helper.rb:69:in `execute_action'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/fastlane/lib/fastlane/runner.rb:255:in `block in execute_action'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/fastlane/lib/fastlane/runner.rb:229:in `chdir'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/fastlane/lib/fastlane/runner.rb:229:in `execute_action'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/fastlane/lib/fastlane/runner.rb:157:in `trigger_action_by_name'
	from /usr/local/lib/ruby/gems/2.6.0/gems/fastlane-2.161.0/fastlane/lib/fastlane/fast_file.rb:159:in `method_missing'
	from Fastfile:28:in `block (2 levels) in parsing_binding'

##[error]Bash exited with code '1'.
Finishing: Run Fastlane Deploy

 I've tried providing the key with and without line breaks but still it didn't seem to make a difference

Environment

 I don't have this, it's running on Azure DevOps

@SEGVeenstra I’m glad key_filepath works but key_content should be working too! How are you trying to load it? Are you trying to load it from an environment variable or a string? One thing you will need to make sure that you do is that you keep the new lines (\n) in there.
We should actually probably catch this error and spit out something a bit more useful 🤔

How are you trying to load it? Are you trying to load it from an environment variable or a string?

I'm setting an environment variable. The response (in 1.60) returned the key and it looked correct, but somehow didn't work.
Loading the same key with the path did.
I found some other suggestions like explicitly putting the \n in there after copying it from the file. Someone suggested to escape the \n by doing \\n, but that didn't help either.
The way I got the key in the env var was with Azure DevOps variables.
I just opened the file, copied the content, and pasted it in there.
It's a single line field so I thought the \n would be removed so I've also tried it with putting them manually.

@SEGVeenstra Thanks for reply! I haven’t tested it on Azure DevOps myself yet 🤔 Putting \n in there should work as of 2.161.0 (which it looks like you are on).  I’m not sure how Azure DevOps handles the insert of multi-line test.
I don’t know if this is how you added the \n but I would maybe deleting the lines in a text editor and replacing them with \n before copying them into Azure.
Maybe we should also support a Base64 encoded key_content to prevent this issue from happening 🤔

Putting \n in there should work as of 2.161.0 (which it looks like you are on).

Correct, I saw the change that was made recently so I tried that.

I don’t know if this is how you added the \n but I would maybe deleting the lines in a text editor and replacing them with \n before copying them into Azure.

This is exactly how I did it ;)

Maybe we should also support a Base64 encoded key_content to prevent this issue from happening

That sounds like a solid solution.

I had this issue.. I just solved it by doing:
app_store_connect_api_key(
  key_id: "#{ENV["KEY_ID"]}",
  issuer_id: "#{ENV["ISSUER_ID"]}",
  key_content: "#{ENV["KEY_CONTENTS"]}".gsub('\n', '\\n'),
  in_house: false,
)

That gsub function replaces all \n with an escaped \\n.. I haven't tested with Azure, but for Github secrets, we pasted the key exactly as we got it from Apple's developer portal, and I passed it as an ENV variable to Fastlane from Github secrets..
Then I invoke the above as the very first thing before my build runs. Finally, I do:
upload_to_testflight(
      username: "#{ENV["APPLE_ID"]}",
      app_identifier: "#{ENV["APP_IDENTIFIER"]}",
      app_platform: "ios",
      apple_id: "#{ENV["APPSTORE_APP_ID"]}",
      ipa: "#{ENV["BUILD_NAME"]}.ipa",
      skip_waiting_for_build_processing: true,
      team_id: "#{ENV["TEAM_ID"]}"
    )

@Brandon-T Thank you, but I already have a work around.
It would be nice if it gets a proper solution though.

@SEGVeenstra This should have a proper solution in 2.162.0 - https://github.com/fastlane/fastlane/blob/master/fastlane/lib/fastlane/actions/app_store_connect_api_key.rb#L22
The app_store_connect_api_key has that same gsub that @Brandon-T mentioned ☝

Oh wait... I did already comment on this about that OMG
I think I will add a base64 encoded option though because why not

Added new is_key_content_base64 in #17401 which should fix some of their issues with setting the key content in ENV and CI 🤞

@joshdholtz I'm getting invalid curve name error when calling upload_to_testflight.
Prior to calling upload_to_testflight I get a valid api_key from
api_key = app_store_connect_api_key(
        key_id: "#{ENV["KEY_ID"]}",
        issuer_id: "#{ENV["ISSUER_ID"]}",
        key_content: "#{ENV["KEY_CONTENT"]}",
        duration: "#{ENV["DURATION"]}",
        in_house: "#{ENV["IN_HOUSE"]}",
        is_key_content_base64: "#{ENV["IS_KEY_ENCODED"]}"
      )

I'm using Fastlane 2.166.0.
Do I need to manually remove the \n from the AppStore Connect Private Key before encoding it to a base64 string ?
Any advise would be highly appreciated.

@tasvir-rohila-carsales Hey! My first suggestion would be to use the environment variables that app_store_connect_api_key has built in so you don't need to manually have all the options being interpolated 😊  Below is the table with a column of the environment variables names and this was generated by running fastlane actions app_store_connect_api_key
Secondly if you are Base64 encoding the the key you should not remove the \n or \r. The issue may have been that you are technically passing a string in to is_key_content_base64_encoded.
Try changing to the using the environment variables that the action attempts to load from by default and let me know how it goes!
+-----------------------+-----------------------------------+-----------------------------------+---------+
|                                    app_store_connect_api_key Options                                    |
+-----------------------+-----------------------------------+-----------------------------------+---------+
| Key                   | Description                       | Env Var                           | Default |
+-----------------------+-----------------------------------+-----------------------------------+---------+
| key_id                | The key ID                        | APP_STORE_CONNECT_API_KEY_KEY_ID  |         |
| issuer_id             | The issuer ID                     | APP_STORE_CONNECT_API_KEY_ISSUER  |         |
|                       |                                   | _ID                               |         |
| key_filepath          | The path to the key p8 file       | APP_STORE_CONNECT_API_KEY_KEY_FI  |         |
|                       |                                   | LEPATH                            |         |
| key_content           | The content of the key p8 file    | APP_STORE_CONNECT_API_KEY_KEY     |         |
| is_key_content_base64 | Whether :key_content is Base64    | APP_STORE_CONNECT_API_KEY_IS_KEY  | false   |
|                       | encoded or not                    | _CONTENT_BASE64                   |         |
| duration              | The token session duration        | APP_STORE_CONNECT_API_KEY_DURATI  |         |
|                       |                                   | ON                                |         |
| in_house              | Is App Store or Enterprise (in    | APP_STORE_CONNECT_API_KEY_IN_HOU  |         |
|                       | house) team? App Store Connect    | SE                                |         |
|                       | API cannot not determine this on  |                                   |         |
|                       | its own (yet)                     |                                   |         |
+-----------------------+-----------------------------------+-----------------------------------+---------+

Hey @joshdholtz Thanks for your quick reply.
I tried your suggestions (of using built in APP_STORE_CONNECT_API_KEY_* environment variables). However the api_key returned by app_store_connect_api_key() works well with match() action which is performed before upload_to_testflight in my Fastfile, it fails with the same invalid curve name error.
FYI the output of api_key which gets passed to match() has boolean value for :is_key_content_base64=>true
Does the api_key get reset after match() action ?

@tasvir-rohila-carsales
It does not get reset so I'm not sure what going on 🧐 Can you post what your Fastfile looks like so I can see what is all going on?
Thanks!

Hi @joshdholtz
Here is a basic version of my Fastfile
require 'fileutils'

fastlane_version "2.144.0"

default_platform :ios

build_dir = "../../build"

# default version range of xcode to build with if one is not passed in
xcode_version = "~> 12.0"

# global variable that stores api_key returned by app_store_connect_api_key()
api_key = app_store_connect_api_key()

private_lane :set_xcode_version do
  xcode_version = ENV["XCODE_VERSION"]
  UI.user_error!("No XCODE_VERSION environment variable specified") if xcode_version == nil

  xcversion version: xcode_version
end

platform :ios do

    set_xcode_version
    dev_setup

    export_method = "app-store"

    # Create the archive
    build_ios_app(
      scheme: scheme,
      derived_data_path: build_dir,
      output_directory: build_dir,
      clean: true,
      xcargs: "BUILD_NUMBER=#{build_number} #{marketing_version} #{swift_conditions} #{gcc_definitions} \"-quiet\" USER_INTERFACE_STYLE=#{user_interface_style}",
      output_name: ipa_name,
      export_team_id: "SQH3BRZ9MZ",
      export_method: export_method,
      codesigning_identity: "Apple Distribution: Carsales.com Ltd. (SQH3BRZ9MZ)"
    )

    changelog = changelog_from_git_commits(
      commits_count: 5,
      pretty: "[%an] %s"
    )

    upload_to_testflight(
        api_key: api_key,
        app_identifier: bundle_identifier,
        skip_submission: true,
        skip_waiting_for_build_processing: true,
        ipa: ipa_path,
        changelog: changelog
      )
  end

  desc "Setup match on developer machine"
  lane :dev_setup do |options|
    readonly = options[:readonly]
    readonly = true unless readonly != nil
    force_for_new_devices = options[:force_for_new_devices]

    do_match(readonly: readonly, force_for_new_devices: force_for_new_devices, type: "development")
    do_match(readonly: readonly, force_for_new_devices: force_for_new_devices, type: "adhoc")
  end

  private_lane :do_match do |options|
    # the MATCH_PASSWORD env var is used by Match. We're just reading it here to make sure the user specified it

    git_encryption_key = ENV["MATCH_PASSWORD"]
    UI.user_error!("MATCH_PASSWORD environment variable must be specified") unless git_encryption_key

    keychain_params = get_keychain_params
    keychain_path = keychain_params[:keychain_path]
    UI.user_error!("keychain_path is missing") unless keychain_path

    # Get/create the required certificates and provisioning profiles for development
    match(
      api_key: api_key,
      type: options[:type],
      keychain_name: keychain_path,
      readonly: options[:readonly],
      force_for_new_devices: options[:force_for_new_devices]
    )
  end

  private_lane :recreate_keychain do
    # set up login keychains
  end

  private_lane :get_keychain_params do
    # gets keychain params
  end

  lane :unlock_my_fastlane_keychain do
    options = get_keychain_params
    keychain_path = options[:keychain_path]
    keychain_password = options[:keychain_password]

    unlock_keychain(
      path: keychain_path,
      password: keychain_password
    )
  end
end

Hey @joshdholtz
Just a FYI, upload_to_testflight() works well if I provide a plain-text for APP_STORE_CONNECT_API_KEY_KEY environment variable.
Also under the hood, when does Fastlane generate the JWT ?
I ask this because I got this failure even though the duration APP_STORE_CONNECT_API_KEY_DURATI was 1200 and the script took less than 20 minutes to run.
[09:48:27]	[Step 1/1] [!] The request could not be completed because:
[09:48:27]	[Step 1/1] 	Authentication credentials are missing or invalid. - Provide a properly configured and signed bearer token, and make sure that it has not expired. Learn more about Generating Tokens for API Requests https://developer.apple.com/go/?id=api-generating-tokens


Brainstorming some ideas

Hi Giulio!
I wrote a big article / proposition about validation here:
https://github.com/ansman/validate.js/issues/50
I think it should be interesting to you as well because Tcomb-validation has a lot of the same weaknesses as ValidateJs and Joi. No separate parsing step (e.g. no idiomatic way to decipher L10n of input data, no way to "jump through" formats (like splitting tags into array), etc.). You may answer there or here if you wish, I will pick up the discussion anyway.

Hi Ivan,
I'm glad to see you again.
I'm going to read your article, thanks for pointing out.
p.s.
As you suggested 7 months ago, I'm digging into semantic ui for my tcomb-form project: great framework.
https://github.com/gcanti/tcomb-form/blob/master/src/templates/semantic.js

I'm glad to see you again.

I'm suprised you remembered me :smile:

As you suggested 7 months ago, I'm digging into semantic ui for my tcomb-form project: great framework.

It changed very much since then. For better I believe.

I'm going to read your article, thanks for pointing out.

Yeah, I want to discuss this with someone experienced. Half-year ago I was super newbie in the frontend development. Now I've got some experience. I'm excited about your tcomb project but I wonder how well it suits for ideas I put in those article.
I'm considering to create a library with described behavior on my own, but if it's possible to reuse or update tcomb-validation to cover those issue cases it will be much better. I'm not chasing the glory of library creator, I'm a practical man :wink:

I managed to combine Tcomb validation with proper I18n handliing and I'm pretty happy with results so forth. I'm going to write an article after some trial and post a link here.

I'm going to write an article after some trial and post a link here

Great! Looking forward to your post.
Btw, for i18n, have you noticed this issue? https://github.com/gcanti/tcomb-validation/issues/27

Btw, for i18n, have you noticed this issue?

Yes. My concerns were about lower level stuff like converting dates in string format to Date objects to make them possible to validate with Tcomb. I'm going to mess with validation messages customization a bit later :smile: Thank you!


Migration - Universal mixtures

This PR migrates the Universal mixtures example to the latest version of GraphPPL.

Here we gooo Thanks @bartvanerp


Added constants for hierarchical naming

Added constants for hierarchical naming.
Reduced dependency surface of the AdapterUtilities package.

We're adding a TestProperty to store multiple levels of a name. I am calling it HierarchicalName. Any recommendations about the naming?
HierarchicalName property is an array, which holds these for a TestCase:

Namespace
Class
TestGroup
DisplayName
Theory

HierarchicalName property is an array, which holds these for a TestCase:

Namespace
Class
TestGroup
DisplayName
Theory / DataRow

I'm ok with the name HierarchicalName or HierarchyName. We shouldn't need an entry for Theory/DataRow. That is captured by TestGroup & DisplayName.
In the case of a "Fact" test, you would have a Namespace/Class//DisplayName and in the case of a "Theory" test you would have a Namespace/Class/TestGroup/DisplayName where TestGroup captures the "test method" level and DisplayName would be different for each data driven test case. The way we handle this today in the test hierarchy is if there is only one DisplayName in a TestGroup, then the TestGroup level is removed.

What about using just the name Hierarchy for the property.

What about using just the name Hierarchy for the property.

Sounds better.


[FEA] Update Transformers4Rec to use merlin core schema

🚀 Feature request
Remove merlin-standardlib dependency and use the merlin-core schema instead.
Motivation
The merlin-standard lib has an old schema, that is not maintained. As the Merlin framework has now a merlin-core library that provides a Schema class and the related Tags. The latter schema is the one produced by Merlin NVTabular and also used by Merlin Systems for serving. So to maintain a seamless integration between T4Rec, NVTabular, and Systems, we should update T4Rec to use the same schema class used by NVTabular and Systems.
Your contribution

Scope

[ ] Change notebooks to use merlin core schema
[ ] Deprecation warnings

I see these 3 PRs related to this. Can we close this ticket?

https://github.com/NVIDIA-Merlin/Transformers4Rec/pull/642
https://github.com/NVIDIA-Merlin/Transformers4Rec/pull/634
https://github.com/NVIDIA-Merlin/Transformers4Rec/pull/632


Crash when Player 1 and Player 2 JSON is missing

The program expects that player 1 and player 2 json files are available.  If not, then the program hangs and the user can't star the program.
Allow the program to continue loading and not crash.  Include sample player1.json and player2.json files to help prevent this in the future.

This is fixed now that the app is using Eclipse RCP.  Nothing is loaded on startup.


xformers use

Hello author, thank you very much for publishing such an excellent paper. I would like to inquire where you started xforms?

Because I need about 30 seconds to perform a fusion, but you said it only takes 3 or 4 seconds

Because I need about 30 seconds to perform a fusion by a100, but you said it only takes 8 seconds

Thank you for your interest. This code does not use the xformer. The running time depends on the size of the composition area, so larger cases may take longer to process.

I understand, thank you for your response

Dear author, I have another question. Which image will be used as the guide image when calculating CLIPimage? Is it the original background image?

Hi, you may refer to this issue to see if it is clear. Thank you.

Dear author, I understand how to calculate, but I have a question about how to process the results because after calculating the cosine similarity, the value is around 0.5 instead of 70 or 80. So I would like to ask you. In addition, when I chose to calculate CLIPimage, I used the fused image and background image for calculation.

Dear author, I understand how to calculate, but I have a question about how to process the results because after calculating the cosine similarity, the value is around 0.5 instead of 70 or 80. So I would like to ask you. In addition, when I chose to calculate CLIPimage, I used the fused image and background image for calculation.

Please multiply the cosine similarity by 100 to obtain a percentage rather than a decimal. Additionally, the method used to calculate CLIPimage appears to be incorrect. Could you please follow the guidelines outlined in Issue 20 for calculating the metrics accurately? Thank you for your attention to this matter.
The information is copied below for your convenience:

When calculating the LPIPS for the background, we mask out the foreground region (according to the user mask) on both the generated composition image and the provided foreground image, i.e., set those pixels to 0. Then calculate the LPIPS on masked images.

When it comes to the foreground, we crop the generated composition image (according to the user mask), and only compare the foreground image cropped out from the generated image with the provided reference image.

Dear author, hello, thank you for your prompt response amidst your busy schedule. I have looked at issue 20 and calculated the LPIPS index according to the method you mentioned, but is the same method used to calculate CLIP? When calculating CLIPtext, I used fused images and prompt words to calculate image text correlation. However, when calculating image text correlation, I don't know which two images to use because there is only one image text correlation in the paper, and there is no distinction between foreground and background regions. I don't know if I have expressed my question clearly

Dear author, hello, thank you for your prompt response amidst your busy schedule. I have looked at issue 20 and calculated the LPIPS index according to the method you mentioned, but is the same method used to calculate CLIP? When calculating CLIPtext, I used fused images and prompt words to calculate image text correlation. However, when calculating image text correlation, I don't know which two images to use because there is only one image text correlation in the paper, and there is no distinction between foreground and background regions. I don't know if I have expressed my question clearly

The CLIP_text is computed between the generated composite image and the text prompt. The CLIP_image is calculated between the cropped region of the generated image, according to the user’s mask, and the reference foreground image. I hope this clarifies things.

I understand, thank you for your patient reply. Wishing you a happy life and smooth work


[Feature] Zendesk Integration

Zendesk is a service-first CRM company that builds software designed to improve customer relationships. As employees, they encourage each other to grow and innovate.
Zendesk API Hierarchy
The API Structure is roughly divided like this:
Ticketing API #13841

Tickets
Users
Organizations
Groups
Apps
Targets
Side Conversation

Help Center API #13842

Articles
Article Comments
Article Labels
Categories
Search
Sections
External Content Records
External Content Sources
External Content Types

Live Chat API #13843

Chats
Agents
Visitors
Departments
Real Time Chat

Voice API #13844

Phone numbers
Digital lines
Availabilities
Greetings
Stats

Sales CRM API
Pending analysis
Custom Data API #13845

Object Types
Object Records
Relationship Types
Relationship Records
Custom Objects Events
Custom Objects Jobs
Profiles
Events

Below is the status (as of 2022-05-14) of the endpoints of each section (at the moment the documentation uses v2 as point of reference).
References (Useful Links)

Zendesk API reference documentation

@felixsuarez0727 please keep this in consideration before starting with zendesk

not a priority at present, closing this issue


Revert "[CWS] Disable unused registry event types (#31047)"

What does this PR do?
This reverts commit 846e066983d9fd5ffc333e5fdc6d22a5fb8cb669 (#31047).
Motivation
This PR broke the new-e2e-windows-service-test, new-e2e-windows-service-test-nofim, and new-e2e-cws: [--run TestAgentWindowsSuite] jobs.
Describe how to test/QA your changes
Run full pipeline on the revert PR: https://gitlab.ddbuild.io/DataDog/datadog-agent/-/pipelines/49098843
Possible Drawbacks / Trade-offs
n/a
Additional Notes

Force-merging the revert PR to fix main.


Add helm charts for lws.

What type of PR is this?
/kind feature

What this PR does / why we need it
Added helm charts for lws.
Which issue(s) this PR fixes

Fixes #260
Special notes for your reviewer
Does this PR introduce a user-facing change?

/ok-to-test

/hold
Put my concern here: https://github.com/kubernetes-sigs/lws/issues/260#issuecomment-2497316210

/hold cancel

Hi @wqlparallel could you solve the comments? And does this support install lws in a different namespace rather than lws-system?

I have committed the code to address the comments.  And this Helm chart supports installing the lws in a different namespace when referencing the main image. However, it may not work as expected in the latest release (v0.4.2) because the commit that deployed in the specified namespace hasn’t been merged yet.

Please squash, I'm ok with the PR. Leave to @ahg-g for final approval.


Design of Crowdfund dapp

Make a design for the main page and 2 variants of the donation company page.
Set of colors and style - Beam

@VikMaevskaya  please have a look and share your comments
Main screen
https://www.figma.com/file/YpEP6ZX9xNZcblyGUkBo0O/Gallery-v2-%2F-CROWDFUND?node-id=1072%3A1220
Project screen 1
https://www.figma.com/file/YpEP6ZX9xNZcblyGUkBo0O/Gallery-v2-%2F-CROWDFUND?node-id=1091%3A1897
Project screen 2
https://www.figma.com/file/YpEP6ZX9xNZcblyGUkBo0O/Gallery-v2-%2F-CROWDFUND?node-id=1072%3A1580

for this step enought. Thanks!


API for arbitrary repo

Could you provide an API for using agentless on an arbitrary repo, not just ones in the SWE bench?

Hi @yubinhu, we are working towards that


Issue with orientations != RAS

When loading an image the Subject instance keeps a copy of the volume and density where volume is a ScalarImage and density is a torch.Tensor.
This means that density itself does not have any concept of it's own coordinate system. When canonicalize  is called only volume is modified in the loop inside ToCanonical. This function may re-order the underlying tensor e.g. if orientation is LPS.
This means that the volume tensor is re-ordered by the density tensor is not.
I think a reasonble fix for this is to make density into a ScalarImage.
This would also allow you to remove volume from the subject instance saving some memory.

I've noticed you have another fix for this in https://github.com/eigenvivek/DiffDRR/commit/feb0fd9b17590386013c1780f942fb0062b7b64c so I will close this issue.
I still think it's worth considering removing volume and turning density into a ScalarImage.


case sensitivity bug with multiselect

When there are 2 items in multiple select with the same label but with different case, selecting one will remove both in multiselect.
I have 2 items in the dropdown:
<MultiSelectBox options={[{label: "element type 1", value: "3899"},{label: "Element type 1",value: "3872"}]} label="Element Types" placeholder="Select Element Types" value="" field="elementTypes" formName="createView" />

fixed in v0.3.11

Thanks for the quick fix!


Fix Usage of Bootsel Button for Pico 2

function board_button_read() always returned TRUE on Pico 2.
this fix uses code from
https://github.com/raspberrypi/pico-examples/blob/master/picoboard/button/button.c
That fix the problem and you can use the button on Pico2 again ;-)

Fix of usage function board_button_read() with pico 2.


Dormant test

Currently the CppUTests are all either "opt-out" (can be filtered to be not run with command line) or they are disabled ("IGNORE_TEST").
I am looking for a way to have CppUTests "opt-in", means that they are included in the testing executable, but only run if I explicitely chose them.
So I created the macro DORMANT_TEST that declares an "opt-in" test dormant until it is awaked using strict name filtering using e.g. "-sg  -sn " . Like IGNORE_TEST it is not executed by default, so I DORMANT_TEST is an apoption of IGNORE_TEST, with key change of DormantUtestShell::shouldRun and DormantUtestShell::willRun .
--> Please pull my changes to cpputest master.
About IGNORE_TEST:
First I thought this could be done using IGNORE_TEST, but it looks like there is no way to run an IGNORE_TEST compiled into a testing executable even not using command line option "IGNORE_TEST" (only sets up filtering for the named test, but cannot execute it -> why sets it up filtering though?) or a custom derived runner class.
The current behavior of "IGNORE_TEST" is useless to me, I can easily surround a test with "#if 0 ... #endif" if don't like it to be included in the testing executable.
IMHO it would be better to be used for tests that are executed, but not counted as failures if they fail (if all TEST(...)s pass, the IGNORE_TEST()s won't make the test run FAIL, but I can see from the verbose logs if the test was successful or not).

I like this. I have a test in the demo executable (examples) that demonstrates an (intentional) crash upon floating point error. Right now, a user has to comment in a #define in order to run it, and of course it mustn't run by default. An opt-in test would be the most elegant way to handle this situation, also.

IGNORE_TEST has its own merits though. One of the being that the test is still run past the compiler, which is not the case if you #if 0 them.

Can we add this behavior to IGNORE_TEST instead ? It seems useful :) At least, with the name explicitly.

I would prefer the name OPT_IN_TEST to DORMANT_TEST
@basvodde - not quite sure what you mean with "add this behavior to IGNORE_TEST" ?

@martiert You want to have a look at this?

The functionality of running an ignored test if you specifically call it via -sn.

Ah. Sounds good to me.

In fact a IGNORE_TEST should IMHO be run if called with -sn && -sg . Which would be the perfect solution for me!
This is independent of my proposal -> are you awaiting more from me? I’d like to trash it and awaiting your fix to being able to call IGNORE_TEST with -sn && -sg.
Von: Bas Vodde<EMAIL_ADDRESS>Gesendet: Donnerstag, 18. Februar 2016 09:40
An: cpputest/cpputest
Cc: PaulBussmann
Betreff: Re: [cpputest] Dormant test (#880)
The functionality of running an ignored test if you specifically call it via -sn.
—
Reply to this email directly or view it on GitHub https://github.com/cpputest/cpputest/pull/880#issuecomment-185598491 .  https://github.com/notifications/beacon/AKas0ujcjDSWlhuTgLwex-pDtRVuBbUKks5plXrDgaJpZM4Hc0RI.gif

Why is it independent? If IGNORE_TEST can be called with -sn, then DORMANT_TEST is not needed anymore, right?

Yes, if I would be able to call IGNORE_TEST with -sn && -sg (only -sn is insufficient IMHO), then I have my OPT-IN and don’t need DORMANT_TEST anymore.
Von: Bas Vodde<EMAIL_ADDRESS>Gesendet: Donnerstag, 18. Februar 2016 09:54
An: cpputest/cpputest
Cc: PaulBussmann
Betreff: Re: [cpputest] Dormant test (#880)
Why is it independent? If IGNORE_TEST can be called with -sn, then DORMANT_TEST is not needed anymore, right?
—
Reply to this email directly or view it on GitHub https://github.com/cpputest/cpputest/pull/880#issuecomment-185605418 .  https://github.com/notifications/beacon/AKas0rMXtpHlSV99VZ36MtVPLrgeVcf4ks5plX4VgaJpZM4Hc0RI.gif

Why is -sn insufficient?

Tests with exactly the same name can live in different groups, no?

Example (not really good, but demonstrates my needs):
IGNORE_TEST(PPC, ValgrindReportsNoPossibleLeaks)
IGNORE_TEST(PPC, ValgrindReportsNoDefinitiveLeaks)
IGNORE_TEST(PPC, Valgrind)
TEST(x86, ValgrindReportsNoPossibleLeaks)
TEST(x86, ValgrindReportsNoDefinitiveLeaks)
TEST(x86, Valgrind)
TEST(x64, ValgrindReportsNoPossibleLeaks)
TEST(x64, ValgrindReportsNoDefinitiveLeaks)
TEST(x64, Valgrind)
I want to run all tests that are named exactly “Valgrind”. I want to filter TESTs, to speed up my test run).
The problem is that “-sn”, “-sg” are currently used only for filtering, not for activation, therefore the user’s intention must be figured out (-> does the user want to filter or activate tests?).
Giving only “-sn” without “-sg” should not run the “IGNORE_TEST”, because in the example above I don’t want to activate IGNORE_TESTs from PPC which are currently having issues, I want to filter the others to speed up my tests.
Therefore I suggest to only activate IGNORE_TESTs with -sn && -sg. Giving both there is no doubt that activation instead of filtering was my intention.
Von: arstrube<EMAIL_ADDRESS>Gesendet: Donnerstag, 18. Februar 2016 10:04
An: cpputest/cpputest
Cc: PaulBussmann
Betreff: Re: [cpputest] Dormant test (#880)
Tests with exactly the same name can live in different groups, no?
—
Reply to this email directly or view it on GitHub https://github.com/cpputest/cpputest/pull/880#issuecomment-185608741 .  https://github.com/notifications/beacon/AKas0qaIUpw2jITopk5CoLMoIkaFV-P5ks5plYCIgaJpZM4Hc0RI.gif

The problem is that I can assume people explicitly stating a test, they want to run it. But for a group, I don't think we can assume that.
This makes me wonder whether it is not better to add a parameter for "run ignored tests" ?

Why not use the existing syntax:
$ ./CppUTestExtTests.exe -v "IGNORE_TEST(FE__with_Plugin, should_crash___when__feenableexcept_was_called)"

This test will run with "IGNORE_TEST" or "TEST" on the command line. Currently, it will just be ignored. The new feature would actually run it.

It is the same as -sg -sn but more explicit.

$ ./CppUTestExtTests.exe -v "IGNORE_TEST(FE__with_Plugin, should_crash___when__feenableexcept_was_called)"
This test will run with "IGNORE_TEST" or "TEST" on the command line. Currently, it will just be ignored. The new feature would actually run it.

Perfect! It was in fact the first thing I tried when I saw this this command line option to get the ignored test running.

How about just adding a command line option to run ignored tests? That way it can be combined with all the other filtering and selection methods ?

IGNORE_TEST is supposed to be ignored, why not let user to explicitly rename IGNORE_TEST to TEST to run them?

IGNORE_TEST is supposed to be ignored, why not let user to explicitly rename IGNORE_TEST to TEST to run them?

@Liu Nick: thanks for your answer and sorry for not previously explaining my motivation:

I am looking for a way to have CppUTests "opt-in", means that they are included in the testing executable, but only run if I explicitely chose them.

This is because building the testing executable is part of a build process taking some hours overall. After this, tests are executed in my CppUTest-base product test binary which also contains functional tests and takes up to hours of complete execution.
After the build is done, it is quite inconvenient to re-build the testing executable for a released product with changed sources. Unfortunately this isn’t easy to handle. Although I am about to improve the build infrastructure to accomplish this, right now, it is simply not possible.
It would be really, really comfortable, to not have to re-compile the testing executable which would be possible if CppUTest provides “OPT-IN” tests.

IGNORE_TEST is supposed to be ignored

I agree that IGNORE_TEST is not an OPT-IN test (although it could be and I’d be very happy having this! : ) ), so I suggested to have an additional macro on initial pull request.
Finally after this discussion, I think it would be very, very smart to have IGNORE_TEST as OPT-IN test, so CppUTest accepts that that I change my mind if I want to run a test or not after compilation … : )

@PaulBussmann I would like to have this feature, too.

Hi,
If the reason is that re-compile take too long time, then I can not concur with this.
Try to think, if compilation takes long time which might as long as hours, then how do you get real time feed back from unit test?
One distinct character of good unit test is the short build-and-execute cycle time.
I raise this because i'm also suffer from long build-and execute cycle time in my unit test environment, and I'm finding methods to try to accelerate it.

For our unit tests, we get the response in real-time within seconds and this is not the motivation for my question.
My use case is that I have another CppUTest based testing executable as artefact from product release building process. This process takes hours and as already stated I am currently not able to easily change sources and recompile just in order to run a dormant test. Changing and recompile from a sealed build is furthermore not good in respect of source code versioning from QA…
Therefor I agree with you that for development unit tests from within Visual Studio it’s not needed.
But in the test harness where you don’t even have the compiler matching the target’s OS, etc. it is simply not possible to recompile. Therefor I just kindly ask you to add this feature.
If my programming as appreciated, please let me know.

I would like to have this, both for running automated tests as part of a build process, and for ad-hoc testing. Also, this would make things much better for CppUTests own testing.
Other Use Cases

Some tests can be successful only under certain conditions, or on specific platforms and are therefore implemented as IGNORE_TEST, possibly with #ifdef. We could now do away with the #ifdef and run the tests as opt-in tests, where appropriate.

Some tests inCppUTest examples are to demonstrate certain failures or even a (intended) crash. A user ought to be able to run these easily as opt-in tests, but they should be ignored during regular testing.

Personally, I have always disliked the fact that I had to change / recompile the code in order to run an ignored test (something I can just as easily do with #if, which can be commented in / out easily, whereas to change TEST to IGNORE_TEST and vice versa, I have to edit the actual code).
The only time when IGNORE_TEST in its current form is really useful is when I want to run a test by the compiler, but it can't possibly run (because a feature is not implemented) and probably never will. IGNORE_TEST tends to be, well, ignored forever..... ;-)
For me, the current implementation of IGNORE_TEST is useful only if I intend to ignore a test "permanently" (like: I know I should test this, but I can't and I won't).

@basvodde I think a command line option, which can be combined with any filtering, would be nice.

Ok, so when we move to a command line option, then I won't merge the DORMANT tests as a command line option that runs ignored tests will do the same. However, a lot of the code in this PR can be re-used. Can we change it to that?

Now the -ri option is implemented. Does that make this pull request obsolute?

Yes, thank you very much! :)))
So it'll be part of V3.7.3, right?
Btw. how can I prepare my code for transitioning to V3.7.3?
I noticed round about 100 compiler errors in my test program due to parameter changes of TestRunner and CHECK... macros since V3.7.2.

Wow. Did we change all that :-/ ? Most changes I recall shouldn't have broken any tests. Would you care to elaborate (perhaps an other issue, or Google Groups)?
Is it possible that perhaps you are using mismatched headers?

Thank you very much for your support!
I'll summarize the conflicts of my code with V3.7.2 API against current sources on "master" and open another issue if needed. IMHO this pull request can be closed.

hi, I also encountered the compatible issue when I upgrade my CppUTest Framework from an older version to a newer one, including compile error, crash error.

Interesting. Especially crashes. Would love to know where they come from.


xgboost installation for R with clear instructions

Hi,
Devtools link is not working and i'm not able to follow the work around given,
please help to give step by step instructions for non technical people to follow.
Thanks in Advance
Alben

Please give more details about what happens in your devtool installation, the output of the command, and the environment you are installing the toolkit

Hi Tqchen,
Thank you very much for the response
After installing devtools and Rtools (setting the path), it is still
throwing an error saying..
"Error in Curl :: curl_fetch_memory(URL, handle=handle)"
Moreover, when i tried workaround, I am not able to follow this work around
given

clone the repository and set the workspace to ...
run R cmd install ....
run install.packages('path...

Regards
Alben
On Jul 17, 2015 9:22 PM, "Tianqi Chen"<EMAIL_ADDRESS>wrote:

Please give more details about what happens in your devtool installation,
the output of the command, and the environment you are installing the
toolkit
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/xgboost/issues/394#issuecomment-122322724.

You can try the alternative,

download https://github.com/dmlc/xgboost/archive/master.zip
unzip it on the root of the folder
run R cmd install R-package

there might be error message reporting some packages missing, please install these packages and try again

Seems related:
http://stackoverflow.com/questions/31311872/error-while-using-install-github-devtools-timeout-issue
options(download.file.method = "wininet")
library("devtools")
...

You can now install xgboost from CRAN https://github.com/dmlc/xgboost/tree/master/R-package


Registering middleware with HotChocolate v11

I have a middleware that I used on V10 that checks if IP is whitelisted on the server. If it's not then it returns QueryException which in turn is serialized into nice GraphQL error.
public class IpFilterMiddleware
    {
        private readonly FieldDelegate _next;
        private readonly ApiOptions _options;

        public IpFilterMiddleware(FieldDelegate next, IOptions<ApiOptions> options)
        {
            _next = next ?? throw new ArgumentNullException(nameof(next));
            _options = options?.Value ?? throw new ArgumentNullException(nameof(options));
        }

        public async Task InvokeAsync(IMiddlewareContext context)
        {
            var httpContext = context!.ResolveContextData<HttpContext>(nameof(HttpContext));
            var ip = httpContext.Connection.RemoteIpAddress;
            if (ip != null && _options.WhiteList != null && _options.WhiteList.Any()
                && !_options.WhiteList.Contains(ip.ToString()))
            {
                throw new QueryException(InvalidIp(ip));
            }

            await _next(context);
        }
    }

The middleware was registered like this
.AddGraphQL(
  sp =>
    SchemaBuilder.New()
      .Use<IpFilterMiddleware>()
      .AddServices(sp)
      .AddQueryType<MyQueryType>()
      .Create())

Now, this doesn't work on V11 as the registration is using a different APIs. What would be the proper way of registering my middleware on this version?

In v11 this method is name UseField<>()

In v11 this method is name UseField<>()

But actually I would not implement this on a field middleware.
Field middleware really means it is executed in parallel on every field. Better use a request middleware and throw a GraphQLException.
services
    .AddGraphQLServer()
    // add your middleware here
    .UseRequest(next => context =>
    {
         // do your thing
         next(context);
    })
    // next chain in the default request pipeline of hot chocolate
    .UseDefaultRequestPipeline();

But actually I would not implement this on a field middleware.
Field middleware really means it is executed in parallel on every field. Better use a request middleware and throw a GraphQLException.
services
    .AddGraphQLServer()
    // add your middleware here
    .UseRequest(next => context =>
    {
         // do your thing
         next(context);
    })
    // next chain in the default request pipeline of hot chocolate
    .UseDefaultRequestPipeline();

Please close the issue if this solved your problem.

Please close the issue if this solved your problem.


spam open boxes is broken

It says "Can not find that box" when I type any box

Fixed

thank you


[Bug]: HttpContent.SetHeader failing when secret header already exists

Describe the issue
The "Http Content" codeunit throws an error message when content is defined from SecretText.
Error message:
Cannot add value because the header 'Content-Type' does not support multiple values.

AL call stack:
"Http Content Impl."(CodeUnit 2355).SetHeader line 12 - System Application by Microsoft
"Http Content Impl."(CodeUnit 2355).SetContentTypeHeader line 4 - System Application by Microsoft
"Http Content Impl."(CodeUnit 2355).SetContent line 6 - System Application by Microsoft
"Http Content"(CodeUnit 2354).SetContent line 2 - System Application by Microsoft
"Http Content"(CodeUnit 2354).Create line 2 - System Application by Microsoft\"Http Content"(CodeUnit 2354).Create line 2 - System Application by Microsoft

This happens because inside "Http Content Impl.".SetHeader procedure Headers.Contains(Name) condition will return false for a secret header, which will be automatically created when we set a content from the SecretText value.
Expected behavior
Content will be defined with a new header without error.
Steps to reproduce
codeunit 50000 SetHeader
{
    trigger OnRun()
    var
        HttpContent: Codeunit "Http Content";
    begin
        HttpContent.Create(SecretStrSubstNo('anything'));
    end;
}

Additional context
No response
I will provide a fix for a bug

[X] I will provide a fix for a bug

@ajkauffmann has agreed to include this in a pull request he is working on anyway 🥳
I cannot assign it to him (seems I can only assign to people who have been active on the issue), but consider it assigned 😉

I'm active now. 😊
Yes, happy to take this one!

And now I could assign you 🥳

@ajkauffmann thank you for picking this one. We tried to use it internally to promote open-source contribution within an organization but it was pushed down in priority.

Is this issue fixed, because i have the same issue when i try to use a secretbody with Application/JSON as content-type

Yes, a PR will be created next week!

Thank you for the Solution, this is just a major issue because i have a Customer who needs this solution ASAP and security is key.

You can always use a workaround by creating a HttpContent object with the correct and use it with HttpContent.Create(HttpContent). Then you don't have to wait for the PR to be approved and backported to v25.


PlotHeatmap::with_label_format is (probably) unsound

Allowing user control of the label format string is unsound, as implot passes it directly into sprintf with a 32 char buffer. (See https://github.com/epezent/implot/issues/310)
Plot heatmap uses the provided format string here:
https://github.com/epezent/implot/blob/4fcc6e01aca406ef17d5a2dabdcbc9e1bd962c0d/implot_items.cpp#L2063
Setting the format string to "%100f" overflows the 32 character buffer and writes into stack memory (though on my system, glibc detects it and aborts).
The same issue is present in PlotPieChart, but that does not currently have a binding in this library.
For now, I think the safest options would be either removing it or manually generating it on the rust side (ie. generate a string `"%.{n}f" with a given level of precision, while ensuring that it is still safe)

Good find, I agree that this is a problem. Probably disabling custom formatting is the best bet at the moment - likely the best API to have would be to let the user provide a formatter callback, but this won't be possible with the way implot exposes the formatting functionality.

I've now marked the label-setting function as unsafe and added a note about the buffer. This does not remove functionality, but makes the danger of setting a wrong format string much clearer and has people explicitly opt into it with unsafe.