I/O race condition exception

Type (choose one): Bug
NLog version: 4.3.5
Platform: .Net 4.5
Current NLog config (xml or C#, if relevant)
<?xml version="1.0" encoding="utf-8"?>

<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
      internalLogFile="C:\Logs\nlog-svc-errors.log"
      internalLogLevel="Error"
      throwExceptions="false"
      autoReload="true">

  <variable name="logPath" value="C:\Logs"/>
  <variable name="fileName" value="XactPrmSvc"/>

  <variable name="timeJson" value='"Time":"${longdate:jsonEncode=true}"'/>
  <variable name="levelJson" value='"Level":"${level}"'/>
  <variable name="machineJson" value='"Machine":"${machinename:jsonEncode=true}"'/>
  <variable name="loggerJson" value='"Logger":"${logger:jsonEncode=true}"'/>
  <variable name="callSiteJson" value='"CallSite":"${callsite:jsonEncode=true}"'/>
  <variable name="messageJson" value='"Message":"${message:jsonEncode=true}"'/>
  <variable name="generalInfoJson" value ='${var:timeJson},${newline}${var:levelJson},${newline}${var:machineJson},${newline}${var:loggerJson},${newline}${var:callSiteJson},${newline}${var:messageJson}'/>

  <variable name="exceptionType" value = '"ExceptionType"\:"${exception:format=type:jsonEncode=true}"'/>
  <variable name="exceptionMessage" value = '"ExceptionMessage"\:"${exception:format=message:jsonEncode=true}"'/>
  <variable name="exceptionMethod" value = '"ExceptionSourceMethod"\:"${exception:format=method:jsonEncode=true}"'/>
  <variable name="exceptionStackTrace" value = '"ExceptionStackTrace"\:"${exception:format=stacktrace:maxInnerExceptionLevel=15:innerFormat=type,message,method,stacktrace:jsonEncode=true}"'/>
  <variable name="excJsonBlock" value = '${var:exceptionType},${newline}${var:exceptionMessage},${newline}${var:exceptionMethod},${newline}${var:exceptionStackTrace}'/>

  <!--
  See http://nlog-project.org/wiki/Configuration_file
  for information on customizing logging rules and outputs.
   -->
  <targets>
    <target xsi:Type="AsyncWrapper" name="log-file" queueLimit="5000" overflowAction="Discard">
      <target xsi:type="File" fileName="${var:logPath}\${var:fileName}.log" createDirs="true"
              autoFlush="true"
              layout="{${var:generalInfoJson}${onexception:,${newline}${var:excJsonBlock}}}${newline}"
              maxArchiveFiles="5"
              archiveNumbering="Date"
              archiveEvery="Day"
              archiveDateFormat="yyyy-MM-dd"
              archiveFileName="${var:logPath}\${var:fileName}.{#}.log"/>
    </target>

    <target xsi:type="Console"
            name="console"
            layout="${longdate} | ${level:uppercase=true} | ${message} ${exception:format=tostring:maxInnerExceptionLevel=10}" />
  </targets>

  <rules>
    <logger name="*" minlevel="Info" writeTo="log-file"></logger>
    <logger name="*" minlevel="Debug" writeTo="console">
      <filters>
        <when condition="level == LogLevel.Debug and starts-with('${message}', 'InactivityMonitor')" action="Ignore" />
        <when condition="level == LogLevel.Debug and starts-with('${message}', 'Got ping response for sessionid')" action="Ignore" />
        <when condition="level == LogLevel.Debug and starts-with('${message}', 'Shutdown of Consumer')" action="Ignore" />
        <when condition="level == LogLevel.Debug and starts-with('${message}', 'Remove of Consumer')" action="Ignore" />
        <when condition="level == LogLevel.Debug and ends-with('${message}', 'No Active TX closing normally.')" action="Ignore" />
        <when condition="starts-with('${message}', 'Closing The Session with Id')" action="Ignore" />
        <when condition="starts-with('${message}', 'Closed The Session with Id')" action="Ignore" />
        <when condition="starts-with('${message}', 'Executing Shutdown on Session with Id')" action="Ignore" />
      </filters>
    </logger>
  </rules>
</nlog>

In case of a BUG:

What is the current result?

Every so often we are seeing the following error:
Probable I/O race condition detected while copying memory. The I/O package is not thread safe by default. In multithreaded applications, a stream must be accessed in a thread-safe way, such as a thread-safe wrapper returned by TextReader's or TextWriter's Synchronized methods. This also applies to classes like StreamWriter and StreamReader.

What is the expected result?
Did you checked the Internal log?

Yes, and nothing

Please post full exception details (message, stacktrace, inner exceptions)

Here are a couple of them:
Void InternalBlockCopy(System.Array, Int32, System.Array, Int32, Int32)
    ExceptionStackTrace:     at System.Buffer.InternalBlockCopy(Array src, Int32 srcOffsetBytes, Array dst, Int32 dstOffsetBytes, Int32 byteCount)
   at System.IO.StreamWriter.Write(Char[] buffer, Int32 index, Int32 count)
   at System.IO.TextWriter.SyncTextWriter.WriteLine(String value)
   at NLog.Targets.Target.Write(AsyncLogEventInfo logEvent)
   at NLog.Targets.Target.WriteAsyncLogEvent(AsyncLogEventInfo logEvent)
   at NLog.LoggerImpl.WriteToTargetWithFilterChain(TargetWithFilterChain targetListHead, LogEventInfo logEvent, AsyncContinuation onException)
   at NLog.LoggerImpl.Write(Type loggerType, TargetWithFilterChain targets, LogEventInfo logEvent, LogFactory factory)

Error during session close: System.IndexOutOfRangeException: Probable I/O race condition detected while copying memory. The I/O package is not thread safe by default. In multithreaded applications, a stream must be accessed in a thread-safe way, such as a thread-safe wrapper returned by TextReader's or TextWriter's Synchronized methods. This also applies to classes like StreamWriter and StreamReader.
   at System.Buffer.InternalBlockCopy(Array src, Int32 srcOffsetBytes, Array dst, Int32 dstOffsetBytes, Int32 byteCount)
   at System.IO.StreamWriter.Write(Char[] buffer, Int32 index, Int32 count)
   at System.IO.TextWriter.SyncTextWriter.WriteLine(String value)
   at NLog.Targets.Target.Write(AsyncLogEventInfo logEvent)
   at NLog.Targets.Target.WriteAsyncLogEvent(AsyncLogEventInfo logEvent)
   at NLog.LoggerImpl.WriteToTargetWithFilterChain(TargetWithFilterChain targetListHead, LogEventInfo logEvent, AsyncContinuation onException)
   at NLog.LoggerImpl.Write(Type loggerType, TargetWithFilterChain targets, LogEventInfo logEvent, LogFactory factory)

Are there any work arrounds? yes/no

No

Is there a version in which it did worked?

We never saw this error in NLog 3.x

Can you help us by writing an unit test?

Not sure how to reproduce consistently. But, I'd certainly love to help out anyway I can.

Thanks for the info!
Do you know in which target the problem is? It's not fully clear to me. I never have seen this error before, it's a BCL error? Also an idea where/how to fix this?

Googling the I/O race condition brings up quite a few things, mostly people suggesting to not share StreamWriters or TextWriters amongst threads. My guess has been the file target because of this call: "NLog.Targets.Target.WriteAsyncLogEvent" in the stack trace since the file target is the only one wrapped in an async wrapper.

yes the "async" name is here a bit misleading, we always do WriteAsyncLogEvent for each target. (as we only know at target level if it's async or not).
I pretty sure the issue is in the console target, as we don't use text of streamwriters directly AFAIK.
        private void Output(string textLine)
        {
            if (this.Error)
            {
                Console.Error.WriteLine(textLine);
            }
            else
            {
                Console.Out.WriteLine(textLine);
            }
        }

Console.Error and Console.Out are TextWriters
Can you confirm this is in the console target? Otherwise it would be difficult to fix this one if we dunno where the problem is.

Sure. I will mess with the targets this week to see what I can find.
On Sun, Sep 11, 2016 at 10:07 AM Julian Verdurmen<EMAIL_ADDRESS>wrote:

yes the "async" name is here a bit misleading, we always do
WriteAsyncLogEvent for each target. (as we only know at target level if
it's async or not).
I pretty sure the issue is in the console target, as we don't use text of
streamwriters directly AFAIK.
    private void Output(string textLine)
    {
        if (this.Error)
        {
            Console.Error.WriteLine(textLine);
        }
        else
        {
            Console.Out.WriteLine(textLine);
        }
    }

Console.Error and Console.Out are TextWriters
Can you confirm this is in the console target? Otherwise it would be
difficult to fix this one if we dunno where the problem is.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NLog/NLog/issues/1631#issuecomment-246188115, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABu24Zp_1ed1ujqLRDxfQGzDNRbJ-RR4ks5qpCcngaJpZM4J5ZU7
.

Thanks!

Looks like this confirms your theory a little bit:
http://stackoverflow.com/questions/33915790/console-out-and-console-error-race-condition-error-in-a-windows-service-written

Thanks for the info. I think we could detect that's writing to StreamWriter.Null and then disabling logging?
You be sure, this won't break you application when thrown exceptions is false, right?

So, I already have thrownExceptions set to false and this is making it past that.

yes, but are those written to the internal log only or is the exception thrown in to your own program (the latter should be impossible...)

It's being thrown in my program as that is where those stack traces came from above.

that's bad!

I thought so as well :)

        /// <summary>
        /// Writes the log to the target.
        /// </summary>
        /// <param name="logEvent">Log event to write.</param>
        public void WriteAsyncLogEvent(AsyncLogEventInfo logEvent)
        {
            lock (this.SyncRoot)
            {
                if (!this.IsInitialized)
                {
                    logEvent.Continuation(null);
                    return;
                }

                if (this.initializeException != null)
                {
                    logEvent.Continuation(this.CreateInitException());
                    return;
                }

                var wrappedContinuation = AsyncHelpers.PreventMultipleCalls(logEvent.Continuation);

                try
                {
                    this.Write(logEvent.LogEvent.WithContinuation(wrappedContinuation));
                }
                catch (Exception exception)
                {
                    if (exception.MustBeRethrown())
                    {
                        throw;
                    }

                    wrappedContinuation(exception);
                }
            }
        }

Dunno why this would thrown the exception

I'll see if I can come up with a simple project to consistently reproduce this consistently based off the bug mentioned in the Stackoverflow article.

What is the type of the outer most exception? IndexOutOfRangeException?

Yes

yes will write an unit test. But I don't see the flawn. I exception should be catched and not rethrown.

Here is a link to the filed Microsoft bug:
https://connect.microsoft.com/VisualStudio/feedback/details/2057284/console-out-probable-i-o-race-condition-issue-in-multi-threaded-windows-service

I can reproduce the exception while debugging, but the error isn't rethrown when throwExceptions=false.
Are you sure you have posted the correct config?

full exeption:
System.IndexOutOfRangeException occurred
  HResult=-2146233080
  Message=Probable I/O race condition detected while copying memory. The I/O package is not thread safe by default. In multithreaded applications, a stream must be accessed in a thread-safe way, such as a thread-safe wrapper returned by TextReader's or TextWriter's Synchronized methods. This also applies to classes like StreamWriter and StreamReader.
  Source=mscorlib
  StackTrace:
       at System.Buffer.InternalBlockCopy(Array src, Int32 srcOffsetBytes, Array dst, Int32 dstOffsetBytes, Int32 byteCount)
       at System.IO.StreamWriter.Write(Char[] buffer, Int32 index, Int32 count)
       at System.IO.TextWriter.WriteLine(String value)
       at System.IO.TextWriter.SyncTextWriter.WriteLine(String value)
       at NLog.Targets.ConsoleTarget.Output(String textLine) in X:\nlog\NLog\src\NLog\Targets\ConsoleTarget.cs:line 153
  InnerException:

BTW, The outer exception should be of type nlogruntimeexception.

I'm pretty sure I did, but I will double check.

I'm not seeing that ConsoleTarget line in my stack traces

I kind of trimmed it before, but here is an example complete stack trace:
at System.Buffer.InternalBlockCopy(Array src, Int32 srcOffsetBytes, Array dst, Int32 dstOffsetBytes, Int32 byteCount)
   at System.IO.StreamWriter.Write(Char[] buffer, Int32 index, Int32 count)
   at System.IO.TextWriter.SyncTextWriter.Write(Char[] buffer, Int32 index, Int32 count)
   at System.Diagnostics.TextWriterTraceListener.WriteLine(String message)
   at System.Diagnostics.TraceInternal.WriteLine(String message)
   at Telerik.Reporting.Processing.TraceTimer.WriteLine(String message)
   at Telerik.Reporting.Processing.Timer.Dispose()
   at Telerik.Reporting.Processing.ReportProcessor.ProcessReport(ReportSource reportSource, Hashtable deviceInfo, IRenderingContext context)
   at Telerik.Reporting.Processing.ReportProcessor.ProcessAndRender(String format, ReportSource reportSource, Hashtable deviceInfo, IRenderingContext renderingContext, CreateStream createStreamCallback)
   at Telerik.Reporting.Processing.ReportProcessor.ProcessAndRenderStateless(String format, ReportSource reportSource, Hashtable deviceInfo, IRenderingContext renderingContext, CreateStream createStreamCallback)
   at Telerik.Reporting.Processing.ReportProcessor.RenderReport(String format, ReportSource reportSource, Hashtable deviceInfo)
   at XactPrm.Services.ReportGenerator.GeneratePdf(ReportProcessor reportProcessor, ReportSource instanceReportSource) in D:\BuildAgent2\work\b2c0b1bf5309070d\Services\XactPrm.Services\ReportGenerator.cs:line 552
   at XactPrm.Services.ReportGenerator.RenderReport(BaseGenerateReport arg, ReportProcessor reportProcessor, InstanceReportSource instanceReportSource) in D:\BuildAgent2\work\b2c0b1bf5309070d\Services\XactPrm.Services\ReportGenerator.cs:line 417
   at XactPrm.Services.ReportGenerator.GenerateReport(GenerateReport arg) in D:\BuildAgent2\work\b2c0b1bf5309070d\Services\XactPrm.Services\ReportGenerator.cs:line 202

Here is different one.
at System.Buffer.InternalBlockCopy(Array src, Int32 srcOffsetBytes, Array dst, Int32 dstOffsetBytes, Int32 byteCount)
   at System.IO.StreamWriter.Write(Char[] buffer, Int32 index, Int32 count)
   at System.IO.TextWriter.SyncTextWriter.WriteLine(String value)
   at NLog.Targets.Target.Write(AsyncLogEventInfo logEvent)
   at NLog.Targets.Target.WriteAsyncLogEvent(AsyncLogEventInfo logEvent)
   at NLog.LoggerImpl.WriteToTargetWithFilterChain(TargetWithFilterChain targetListHead, LogEventInfo logEvent, AsyncContinuation onException)
   at NLog.LoggerImpl.Write(Type loggerType, TargetWithFilterChain targets, LogEventInfo logEvent, LogFactory factory)
   at XactPrm.Services.XactNet.XactNetInstanceState.<GetSmartObjectFileRecords>d__68.MoveNext() in D:\BuildAgent2\work\b2c0b1bf5309070d\Services\XactPrm.Services\XactNet\XactNetInstanceState.cs:line 350
   at System.Linq.Buffer`1..ctor(IEnumerable`1 source)
   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
   at XactPrm.Services.XactNet.XactNetInstanceState.PreLoadSmartObjectFileRecords() in D:\BuildAgent2\work\b2c0b1bf5309070d\Services\XactPrm.Services\XactNet\XactNetInstanceState.cs:line 298
   at XactPrm.Services.XactNet.XactNetUploadService.OnMessageReceived(UploadAssignment obj) in D:\BuildAgent2\work\b2c0b1bf5309070d\Services\XactPrm.Services\XactNet\XactNetUploadService.cs:line 61

Looks like the same thing happens with the Debug target as well.

My operations team just sent me, the NLog.config. What I sent you was not correct. Apparently the configuration they have is with throw exceptions set to true.

Thanks for the confirm!

I could reproduce this in an unit test :)
Too bad I cannot recognize the .Null writers (would expect that as it should be a singleton)
 var output = GetOutput();
            var isNull = (output == TextWriter.Null || output == StreamWriter.Null);

I think I will pause logging to the console when the error arises (IndexOutOfRangeException). Do you think that's OK?

I think that will be okay.

OK will be 4.3.9! This week release!

NLog 4.3.9 is live! https://www.nuget.org/packages/NLog/4.3.9


The mods don't work for me, I read them in the game but they don't work

I have installed: TestMod - StarPanda-Challenge. I can't with the first mod move the soap to test the mods and second, I can't configure the game with the second mod to make it more difficult, any help?

if you are playing with BRML 2.X this is normal, as stated in the first line of the "Avaliable Mods" section.

and how could I fix it?


Make the Histoire component center

Closes #239
The commit looks like a lot of changes, but basically, I just added this wrapper div to make the component center.
<div class="min-h-screen w-full flex items-center justify-center">

I have added this wrapper to AlertDialog, Avatar, and Checkbox components, if you think this is good, I will add this wrapper to other components as well.

Let's not do this.
As I've suggested in Discord, it is better to look at how we implement Accordion's story, which is using type:grid.
The reason is we are going to populate more story in the future, which your approach above would be eventually replace. I wouldn't want your effort to go to waste, could you try changing to use type: grid for the layout? 😁

Okay I will do that instead


Carry #27834 — Do not require .git in the build context

Carry #27834 and hopefully fixes builds :angel:.

Add .git to .dockerignore
export DOCKER_GITCOMMIT in the Makefile
prioritize DOCKER_GITCOMMIT against the git command in
./hack/make.sh
Also add integration-cli/bundles to gitignore (it's generated when
using integration-cli shell)

Note that, on my laptop for example, I'm sending 30M of context with this, and 150M without :scream_cat:, so I really think this can be a huge win :angel:
/cc @tianon @jhowardmsft @icecrime @thaJeztah @cpuguy83
:frog:

Had to re-open because I force-pushed the branch before re-opening the PR... 😅
Now that 1.13.0 is out, I'll update the Janky builds to take care of that

@vdemeester - Think I've got all the remaining bits for Windows done. Can you cherry-pick https://github.com/Microsoft/docker/commit/e969eabd7faa19335fe58e7dc0f255923166a674 from https://github.com/Microsoft/docker/tree/jjh/gitignore?
I am just doing the final validations to the CI script itself so that it works in the current mode (with .git present in the docker image) and this new mode (without .git in the docker image). Once I've done that, I'll update production Jenkins and things should be good :)

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "carry-pr-27834" git@github.com:vdemeester/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842353340064
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Updated.. Something bad though, is that some of the validate scripts (lots) are using git to validate only changes 😓 I wonder what to do there /cc @tiborvass @tianon

Bind mount .git! Really we should be bind mounting . I think.

Ok so it's green, I did 2 things:

On the CI, I am binding .git when validate (and only when validate)
On the Makefile I always bind .git (not a huge fan be…) — this means validation won't work if remote daemon. I should probably make it "conditionnal" but …

SGTM :+1:

ping @cpuguy83

I'll go ahead and merge this, looks like we have enough LGTM's 🏎 🐎


Home - Connect to Game

Button Connect + code field
Try to connect to existing game with same code
Error if game not exist

Done


TypeError in RNNs tutorial

URL(s) with the issue:
https://www.tensorflow.org/tutorials/keras/basic_regression
Description of issue (what needs changing): https://www.tensorflow.org/tutorials/keras/basic_regression#build_the_model
Clear description
The model = build_model() function results in a TypeError and gives the following trace:
TypeError                                 Traceback (most recent call last)
<ipython-input-19-671884cecb64> in <module>
----> 1 model = build_model()

<ipython-input-18-d3556ed39f80> in build_model()
      3     layers.Dense(64, activation=tf.nn.relu, input_shape=[64]),
      4     layers.Dense(64, activation=tf.nn.relu),
----> 5     layers.Dense(1)
      6   ])
      7

~/anaconda3/lib/python3.7/site-packages/keras/engine/sequential.py in __init__(self, layers, name)
     91         if layers:
     92             for layer in layers:
---> 93                 self.add(layer)
     94
     95     @property

~/anaconda3/lib/python3.7/site-packages/keras/engine/sequential.py in add(self, layer)
    130             raise TypeError('The added layer must be '
    131                             'an instance of class Layer. '
--> 132                             'Found: ' + str(layer))
    133         self.built = False
    134         if not self._layers:

TypeError: The added layer must be an instance of class Layer. Found: <tensorflow.python.keras.layers.core.Dense object at 0x7efe86908f28>

Parameters defined
Are all parameters defined and formatted correctly?
Yes. This method does not use any parameters apart from train_dataset which has been replaced with a constant in my above example.
Raises listed and defined
This raises a TypeError.

@chennakeshava1998 I ran the tutorial without any issues. Here is the gist. Can you create a gist or share code to reproduce the issue? Thanks!

Here is the gist of a reduced version of the tutorial. I am using it for a different application, so I only needed the model creation parts. Gist: https://gist.github.com/chennakeshava1998/e38e8da0a6adf0640d8b5640095f3b83
The only change I have made is in specifying the input dimension to the RNN. I have used a constant of [64] instead of the length of an array.

@chennakeshava1998 Here is the working version. keras.Sequential and  tensorflow.keras.Sequential are different where one is implemented inside other framework. When model was built entirely on tensorflow framework there is no issue. Thanks!
I am closing this issue. Thanks!

okay, thanks a lot!


Added a new domain

Nevermind, I was searching in the list on my phone (in alphabethic order), but didn't see it. Now I am on laptop and saw it was already added.


Fixing typos

and fixing link

Hi @4c74356b41, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@4c74356b41 : Thanks for your contribution to the Azure documentation! The author, @rgardler, has been notified to review your proposed change.

#sign-off

@rgardler - Hi Ross, could you resolve this merge conflict. Thanks!

This document has been edited elsewhere with similar updates. This PR can be closed.


How about turning off the default build reports generation feature

That report generation takes time, slowing down the overall build. Reports are definitely useful, but do you need them every time you run the build. We can conditionally disable generating test reports by setting <disableXmlReport>true<disableXmlReport>. If you need to generate reports, just add -DcloseTestReports=false to the end of mvn build command.
=====================
If there are any inappropriate modifications in this PR, please give me a reply and I will change them.

-1: If you don't want reports, don't run them for your invocation on the command line.


Feature request: Environments

Hey guys, what do you think about adding support for environments?
https://www.simoahava.com/analytics/better-qa-with-google-tag-manager-environments/
Would love to contribute for that!
Basically it boils down to parsing gtm_auth and gtm_preview params in a slightly different snippet:
<script>(function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
new Date().getTime(),event:'gtm.js'});var f=d.getElementsByTagName(s)[0],
j=d.createElement(s),dl=l!='dataLayer'?'&l='+l:'';j.async=true;j.src=
'https://www.googletagmanager.com/gtm.js?id='+i+dl+ '&gtm_auth=<token>&gtm_preview=env-3&gtm_cookies_win=x';f.parentNode.insertBefore(j,f);
})(window,document,'script','dataLayer','GTM-XXXXXX);</script>

Hey :)
Yeah, you're welcome, feel free to open a PR and I'll look at it in the next days or I can do it next week probably


Documentation: Custom PDF Layout

How to test the new PDF generator in the browser
This method offers a (relatively) fast way of previewing the PDFs that an app will output using the new PDF generator.
Note: As of writing, this is available in the browser only, and has not yet been implemented in the App backend. I would very much appreciate receiving feedback on potential issues or bugs on Slack (or comment here) before this goes live!
Prerequisites

At least version 3.56.0-alpha2 of app-frontend.
Google Chrome. Print-to-pdf functionality differs widely between browsers, and the new PDF-service uses Chromium so only Chrome will look correct. There are many known issues in other browsers that may never be fixed as it will only ever run on Chromium, so please only report issues found in Chrome!

How to view PDF

In any app, fill in some data until you are satisfied.
Add ?pdf=1 to the end of the URL and hit enter.
Should look something like: http://local.altinn.cloud/bjosttveit/newcomer-sogndal/#/instance/501337/b37e7676-5ba1-4e82-91d2-69e4a2ba8bcc?pdf=1.
Use Chrome's built in print function. (Ctrl+P/⌘+P)
You can optionally choose to save to a PDF file or simply view the result in the preview pop-up.

Customizing the PDF
The new PDF generator offers two ways of configuring PDFs.

Automatic method: This works just like the old method, documented here.
Custom layout method: Allows you to specify a custom layout file to use when generating the PDF instead of automatically including components from your form layouts.

Automatic method
This reads your layout files and will automatically include your components unless they are removed using excludeFromPdf at either the page or component level. The existing documentation should still apply, and existing apps using this method should work out-of-the-box. One difference is that more components are included now like e.g. headers and paragraphs, unless they are excluded. The first page will have the name of the app, name of the organization owning the app, and instance information (sender, receiver, date sent...).
Custom layout method
This is a new method of configuring PDFs which allows you to fully customize the content by creating a new layout file for the PDF generation. The excludeFromPdf options above have no effect when using a custom layout. In order to use this you need to add the following to your Settings.json file:
{
    "pages": {
        "pdfLayoutName": "<name of pdf layout file>"
        ...
    }
    ...
}

The layout file is just like any other layout file, and can include the following type of components:

Summary
Group
Header
Paragraph
Image
Panel
InstanceInformation

New options
There are some new options available for customizing the PDF:
PageBreak: You can specify that a page break should occur either before or following a component. This can e.g. be used to have a header start on a new page. This is done in the following way:
{
    "id": "pdf-header",
    "type": "Header",
    "textResourceBindings": {
      "title": "This is a custom PDF"
    },
    "dataModelBindings": {},
    "size": "L",
    // Add the "pageBreak" object with either "breakBefore" or "breakAfter"
    "pageBreak": {
      "breakBefore": true, // can be either a boolean or an expression
      "breakAfter": false
    }
}

Excluded children: A summary of a group component can be specified to exclude specific children from view. This is done in the following way:
{
    "id": "pdf-group-summary",
    "type": "Summary",
    "componentRef": "some-group-component",
    "pageRef": "Form",
    // Add "excludedChildren"
    "excludedChildren": ["some-child-component"]
},

Including instance information
The automatic PDF-layout includes instance information on the first page of the PDF. In order to get this in the custom PDF, you need to add the following as the first component in the layout file:
{
    "id": "pdf-instance",
    "type": "InstanceInformation",
    "elements": {
      "dateSent": true,
      "sender": true,
      "receiver": true,
      "referenceNumber": true
    },
    "pageBreak": {
      "breakAfter": true
    }
},

https://orange-bay-0b940ce03-795.westeurope.1.azurestaticapps.net/app/development/ux/pdf/


Add cloudflare plugin

WHY are these changes introduced?
We want to support cloudflare tunnels
Contributes to https://github.com/Shopify/internal-cli-foundations/issues/307

WHAT is this pull request doing?

Add a new plugin: Cloudflare tunnels
Use cloudflare tunnels by default
A new way to opt-in to use ngrok will be added in a future PR
For now, only trycloudflare tunnels are supported, but we will work on support custom domains as well

How to test your changes?

Just app dev, and you should see a cloudflare link

Post-release steps

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
[ ] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

Great, thanks. Yeah, the post-release label is the way to release both things together. But we need to prepare the documentation before. Once the next PR is ready and everything is clear, we should ping Shaina.


2 Instances of one consumer group processing one event twice

Package Name: @azure/event-hubs
Package Version:
Operating system: Windows
[ ] nodejs16

version:

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
I wrote a simple consumer without checkpointstore, ran it in two cmd. Both are using the same group name. I produce a single message, Both the consumers are processing the event even though I am using one group name in both the consumers
To Reproduce
Steps to reproduce the behavior:

Write a simple consumer to process events, Use same group name, Execute it in 2 cmd windoes

Expected behavior
Only one consumer should process the event because they are using the same group name
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Do i need chexkpointstore. Could you point me to the documentation page where I can find information on multiple instances and single consumer group scenario.

@selvaganesh22 Thanks for reaching out! As you noticed, by default, a consumer will read all events in the partition(s) it subscribes to, and it doesn't matter if those events are being read by another consumer.
If you pass in a checkpoint store, the internal load-balancing mechanism will use it to keep track of which partition is subscribed to by which consumer. You can read more about the architecture of event hubs in https://docs.microsoft.com/en-us/azure/event-hubs/event-hubs-scalability#partitions


File updating

It can be useful to be able to append edit existing files.

append
update, base on a start point and a small content into a byte slice
insert, this can be a heavy work with the way file chunks are manage

A basic use case for append is saving video or audio streams.

The interface is not

append
update
insert

But it use an interface of io.ReaderAt, io.WriterAt.


ParserError: Expected ',' but got identifier   --> project:/contracts/Date.sol:44:34:

Hi,
I followed along on the YouTube video to learn how to create Dapps but got an error while running "truffle test" for the Date contract. Out put below
PS C:\DPUMD\FirstTruffle> truffle test
Using network 'test'.
Compiling your contracts...

Compiling .\contracts\Date.sol
Compiling .\contracts\Date.sol
Compiling .\contracts\Migrations.sol

ParserError: Expected ',' but got identifier
--> project:/contracts/Date.sol:44:34:
|
44 |         require(msg.value == (10 finney), "claiming a date costs 10 finney");
|                                  ^^^^^^
Compilation failed. See above.
Truffle v5.4.3 (core: 5.4.3)
Node v14.17.3
Is there a declaration for "10 finney" missing?
Your correspondence is appreciated

Hello, the support for "finney" was dropped in newer solidity versions. Which version are you using in your truffle-config.js?

0.8.0

Is there an alternative I could use or should I just get started on another tutorial series?

Instead of using 10 finney you can use 0.01 ether.

I hope that has helped. If you have further questions please don't hesitate to ask. Cheers!

Certainly helpful, thank you


Loadable Components with react-redux connect error

💬 Questions and Help
if i loadable(() => import("pages/redux/Count"), options),
and in Count.tsx
i use connect(mapStateToProps)(Count)
typescript submit error such as
Type 'Promise<typeof import("/Users/zhengxianfeng/zxf4399/github/learn-react/src/pages/redux/Count")>' is not assignable to type 'Promise<DefaultComponent<{}>>'.

The issue is on the next line.
Type 'Promise<typeof import("/Users/zhengxianfeng/zxf4399/github/learn-react/src/pages/redux/Count")>' is not assignable to type 'Promise<DefaultComponent<{}>>
>> HERE IS WHY

Probably you have to export default something, and so on. And that's a normal TypeScript case, which has nothing with Redux or Loadable.
If you want help please provide enough information for us to understand the whole picture. Please value our time over your time.


Git push error for azdo

PR 743 build -
azd version 0.2.0-beta.3-pr.1877365 (commit 33710f9beab18fc32168ae2e02768f2697bdc164)

azd init -t Azure-Samples/todo-nodejs-mongo -b pr/743

Scopes of PAT
Agent Pools (read, manage)
Build (read and execute)
Code (full)
Project and team (read, write and manage)
Release (read, write, execute and manage)
Service Endpoints (read, query and manage)
D:\codes\test\node927>azd pipeline config --provider azdo
Using pipeline provider: Azure DevOps
You need an Azure DevOps Personal Access Token (PAT). Please create a PAT by following the instructions here https://aka.ms/azure-dev/azdo-pat
? Personal Access Token (PAT): xxx

? Personal Access Token (PAT): xxx
? Save the PAT to the node927 environment file (.env)? Yes
? Please enter an Azure DevOps Organization Name: puichan0401

? Please enter an Azure DevOps Organization Name: puichan0401
Creating or updating service principal az-dev-09-27-2022-19-45-02.

? Initialize a new git repository? Yes
? A remote named "origin" was not found. Would you like to configure one? Yes
? How would you like to configure your project? Create a new Azure DevOps Project
? Enter the name for your new Azure DevOps Project OR Hit enter to use this name: (node927)

? Enter the name for your new Azure DevOps Project OR Hit enter to use this name: node927
using default repo (node927) in newly created project(node927)
? Would you like to commit and push your local changes to start the configured CI pipeline? Yes
Pushing changes
fatal: Authentication failed for 'https://dev.azure.com/puichan0401/node927/_git/node927/'
Error: git push: pushing changes: failed to push: exit code: 128, stdout: , stderr: : exit status 128

I ran this again with --debug. This time, I am prompted for credential. A new PAT entry is added:

End of the debug output
...
Service Connection azconnection already exists. Updating endpoint
? Would you like to commit and push your local changes to start the configured CI pipeline? Yes
2022/09/27 12:54:54 command_runner.go:95: Run exec: 'git -C D:\codes\test\node927 branch --show-current'
2022/09/27 12:54:54 command_runner.go:95: Run exec: 'git config credential.helper'
2022/09/27 12:54:54 command_runner.go:95: Run exec: 'git -C D:\codes\test\node927 add .'
2022/09/27 12:54:54 command_runner.go:95: Run exec: 'git -C D:\codes\test\node927 commit --allow-empty -m Configure Azure Developer Pipeline'
Pushing changes
2022/09/27 12:54:54 command_runner.go:95: Run exec: 'git -C D:\codes\test\node927 push --set-upstream origin main'
Enumerating objects: 182, done.
Counting objects: 100% (182/182), done.
Delta compression using up to 8 threads
Compressing objects: 100% (170/170), done.
Writing objects: 100% (182/182), 685.24 KiB | 7.29 MiB/s, done.
Total 182 (delta 19), reused 0 (delta 0), pack-reused 0
remote: Analyzing objects... (182/182) (48 ms)
remote: Storing packfile... done (91 ms)
remote: Storing index... done (35 ms)
To https://dev.azure.com/puichan0401/node927/_git/node927
 * [new branch]      main -> main
branch 'main' set up to track 'origin/main'.
2022/09/27 12:55:06 command_runner.go:95: Run exec: 'git -C D:\codes\test\node927 remote set-url origin<EMAIL_ADDRESS>
Successfully configured Azure DevOps Repository https://dev.azure.com/puichan0401/node927/_git/node927

Error: post git push hook: Could not queue the build because there were validation errors or warnings.

Just checked the project. After running azd pipeline config --debug (and I suspect with the Git PAT),

code is pushed and
pipeline run is a success!

Not sure why the "Error: post git push hook: Could not queue the build because there were validation errors or warnings."

Removed the blocker label from this issue.
The root cause is related to how git credentials are persisted and not related to azd.
Uses must be aware than rotating a PAT would affect any store credentials using that PAT, so they need to refresh the stores.
We can invest some time in the future on how to provide good error handling or messages about wrong PAT

We also encounter the same issue on Mac OS, but we created a master  branch to push code, and it passes.

@vhvb1989 what if I work on multiple repo and use diff PAR for my projects? This is not about rotating PAT. Previous credential is cached. How do we prevent that?

what if I work on multiple repo and use diff PAR for my projects?

git-credential-manager automatically stores one credential per domain.  This is because you would use the same PAT for all the projects/repos/pipelines within an Azdo Server. The PAT can't not be scoped to one project or repo only, as it is created for the entire Organization (or even across organizations).
The PAT is equivalent to your user-log-in credentials. You don't need a different user-account for each project.
Using one PAT per repo (within the same organization) is equivalent to have multiple user accounts (one account per repo)
It is more likely to think about a relation between a PAT and a tool.  You might have one PAT for different tools. For example, one PAT for azd cli, another PAT for az cli, another PAT for some scripts, etc.  Then you can control the access that each tool can have with the PAT.
The PAT is usually regenerated or expired when the tool is no longer in use, or after a time period.  After that, people usually know that they need to update the PAT for that tool


% Reads mapped > 100

Hi, in this entry: DRX011916 the processing logs are as follows:
52549766,158.0,19.3,1716
How is it possible that 158.0% of the reads were mapped?  Does this have some other meaning?

This is clearly our mistake. We'll improve it by the next update.

It could be that for paired end results you treat them differently

Understood. Thank you for pointing that out.


Xamarin.Android project fails to compile with Linker enabled after updating to 2.4 from Nuget

I have a Xamarin.Android project that was running with ModernHttpClient 2.3 and worked perfectly.
After updating to 2.4, if I build with Linking set to "SDK Assemblies Only" it fails to build with this error:
Error	358	The "LinkAssemblies" task failed unexpectedly.
Xamarin.Android.XamarinAndroidException: error XA2006: Reference to metadata item 'System.IntPtr Android.Runtime.JNIEnv::StartCreateInstance(System.Type,System.String,Android.Runtime.JValue*)' (defined in 'OkHttp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null') from 'OkHttp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' could not be resolved. ---> Mono.Cecil.ResolutionException: Failed to resolve System.IntPtr Android.Runtime.JNIEnv::StartCreateInstance(System.Type,System.String,Android.Runtime.JValue*)
at Mono.Linker.Steps.MarkStep.MarkMethod(MethodReference reference)
at Mono.Linker.Steps.MarkStep.MarkInstruction(Instruction instruction)
at Mono.Linker.Steps.MarkStep.MarkMethodBody(MethodBody body)
at Mono.Linker.Steps.MarkStep.ProcessMethod(MethodDefinition method)
at Mono.Linker.Steps.MarkStep.ProcessQueue()
at Mono.Linker.Steps.MarkStep.Process()
at Mono.Linker.Steps.MarkStep.Process(LinkContext context)
at Mono.Linker.Pipeline.Process(LinkContext context)
at MonoDroid.Tuner.Linker.Process(LinkerOptions options, LinkContext& context)
at Xamarin.Android.Tasks.LinkAssemblies.Execute()
--- End of inner exception stack trace ---
at Xamarin.Android.Diagnostic.Error(Int32 code, Exception innerException, String message, Object[] args)
at Xamarin.Android.Tasks.LinkAssemblies.Execute()
at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
at Microsoft.Build.BackEnd.TaskBuilder.d__20.MoveNext()	C:\Program Files (x86)\MSBuild\Xamarin\Android\Xamarin.Android.Common.targets	1017	5
If I set Linking to None it works fine.

When I say if Linking is set to None it works fine, what I mean is that it builds, it doesn't actually make successful network requests.

It was working for me, even on device. I suspect we've got different versions of Xamarin.Android maybe?

You are right, Ill try upgrading to latest and try again.

Sure enough upgrading to Xamarin.Android 5.1 worked. Perhaps its worth adding a note that ModernHttpClient 2.4 requires Xamarin.Android 5.1 or greater to build? I did notice in the release notes of 5.1 there are a number of breaking changes: https://forums.xamarin.com/discussion/37277/stable-release-xamarin-android-5-1-0-breaking-changes-new-features-and-bug-fixes

I've noticed the same. Unfortunately we can't upgrade to Xamarin Android 5.1 yet because it breaks using Android Support Library resources. Maybe it's possible to require a certain version in the nuget package? Using TargetFramework MonoAndroid51 in the nuspec might already be enough. (Wishful thinking: Maybe include a version compiled against 4.x :wink:)

@Qonstrukt Just use allowedVersions to pin yourself at the previous ModernHttpClient version


docs: Merge sway reference section into the sway book

Description
Checklist

[ ] I have linked to any relevant issues.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).

[ ] If my change requires substantial documentation changes, I have requested support from the DevRel team

[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[ ] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[ ] I have requested a review from the relevant team or maintainers.

We should think about deleting the reference once the merge is fully done.

We should think about deleting the reference once the merge is fully done.

Agreed @IGI-111 : Will add more sections from the reference section within the book before deleting the section

@laishawadhwa  You'll need to add Namespaces to docs/book/spell-check-custom-words.txt to fix the spellcheck CI, as well as fix these lint errors:
docs/book/src/reference/known_issues_and_workarounds.md:11 MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]
docs/book/src/reference/known_issues_and_workarounds.md:20:1 MD004/ul-style Unordered list style [Expected: asterisk; Actual: dash]
docs/book/src/reference/known_issues_and_workarounds.md:21:1 MD004/ul-style Unordered list style [Expected: asterisk; Actual: dash]
docs/book/src/reference/known_issues_and_workarounds.md:22:1 MD004/ul-style Unordered list style [Expected: asterisk; Actual: dash]
docs/book/src/reference/known_issues_and_workarounds.md:23:1 MD004/ul-style Unordered list style [Expected: asterisk; Actual: dash]
docs/book/src/reference/known_issues_and_workarounds.md:39 MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]


Make it possible to change folders' color

What feature or improvement do you think would benefit Files?
Presonalize the look and feel via changing folders' color
e.g. set it blue to look like iPadOS' file manager app
Requirements

This proposal will give an option to set folder color

Files Version
<IP_ADDRESS>
Windows Version
windows11 22623.1245
Comments
No response

Thank you for your feedback, you can customize folder thumbnails from the custimization tab in the properties window.

Thank you for your feedback, you can customize folder thumbnails from the custimization tab in the properties window.

Oh, I mean changing the UI color. the folder element, in other words, this widget, is yellow. I hope we can select other color for this element, like a pink folder

This picture, the folder widget, you can give some different styles to choose, just like changing app theme

You mean like across the board? We're tracking this in #10941.
There are some programs in Windows that can already do this but I don't know them offhand and they aren't recommended since they make modifications to the system.


cp_built_image_to_nfs: changes to allow for separate nodes

Is the pod's status.hostIP sufficient, or should we maybe use the pod's spec.nodeName and look up the node's status.addresses of type Hostname or ExternalIP.

Dropping.


Build failure: pipewire with bluezSupport=false

Steps To Reproduce
Steps to reproduce the behavior:

set bluezSupport to false and try to build the pipewire package

Build log
error: builder for '/nix/store/nvkxdnzcr6bf3gwydrp112w5sk6czqa3-pipewire-0.3.85.drv' failed with exit code 1;
       last 10 log lines:
       > Run-time dependency libavcodec found: YES 60.3.100
       > Run-time dependency libavformat found: YES 60.3.100
       > Run-time dependency libavutil found: YES 58.2.100
       > Did not find CMake 'cmake'
       > Found CMake: NO
       > Run-time dependency opus found: NO (tried pkgconfig)
       >
       > meson.build:303:11: ERROR: Dependency "opus" not found, tried pkgconfig
       >
       > A full log can be found at /build/source/build/meson-logs/meson-log.txt
       For full logs, run 'nix log /nix/store/nvkxdnzcr6bf3gwydrp112w5sk6czqa3-pipewire-0.3.85.drv'.
error: 1 dependencies of derivation '/nix/store/k5ml31v3wp9mdsc1f3mn2crsx4lxhmhs-SDL2-2.28.4.drv' failed to build
error: 1 dependencies of derivation '/nix/store/xzin0rppjcapjw31x8ljqv3sdz80fs7k-gnome-shell-45.2.drv' failed to build
error: 1 dependencies of derivation '/nix/store/i4n30yl81sg3yylrhr3vqdpldv0x9ypa-google-chrome-120.0.6099.109.drv' failed to build
error: 1 dependencies of derivation '/nix/store/7ykmviy8km51fswx954h5sdxsaq7a751-hwdb.bin.drv' failed to build
error: 1 dependencies of derivation '/nix/store/zqzbbsfj8s3w70rmlk6yk80rxy6ywljl-openal-soft-1.23.1.drv' failed to build
error: 1 dependencies of derivation '/nix/store/9j5p858g9qkyx77rjsx1r5sfv1g054jw-system-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/0g6j0ima4giqwfyyslb0viickl46ivn0-nixos-system-heliodor-23.11.20231220.d65bcea.drv' failed to build

Full log
Additional context
I think we should add the -Dopus=false and -Dbluez5-codec-opus=false meson options when the bluez support is disabled to avoid the Opus dependency.
Notify maintainers
ping @Kranzes @K900

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[garry@heliodor:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.68, NixOS, 23.11 (Tapir), 23.11.20231220.d65bcea`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`
---

Add a :+1: [reaction] to [issues you find important].

[reaction]: https://github.blog/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/
[issues you find important]: https://github.com/NixOS/nixpkgs/issues?q=is%3Aissue+is%3Aopen+sort%3Areactions-%2B1-desc

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/a-sad-state-of-nixpkgs/49397/1


Add keyvalue

Quick and dirty. Should fix #8.

I'm going to decline this as plugins should now go to https://github.com/seabird-irc/plugins


Review the ALB layer design Architecture with Warren

Deploy the Alb ip forwarding functions from the SEA config file
Create a Core Alb layer, that can be used to deploy the automation that takes care of the External load balancer creation and Dynamo Db modifications
Automation should also create the Required Route53 resources.
Automation is triggered based on Internal load balancer creation with particular tags


Completion is not working well

The Flutter plugin is getting multiple reports that completion is not working or not working well. Unfortunately, I cannot transfer them, so I'll add links here.
https://github.com/flutter/flutter-intellij/issues/6177
https://github.com/flutter/flutter-intellij/issues/6176

Dup, sorry, I'm having network issues.


New package: FWFTables v0.2.0

Registering package: FWFTables
Repository: https://github.com/HenricoWitvliet/FWFTables.jl
Created by: @HenricoWitvliet
Version: v0.2.0
Commit: 115ce6d75eec9a03bfdf039270d3a2ca2856a4f9
Git reference: master
Release notes:

First public version to read fixed width ascii files using the Tables-interface

I fixed my errors and a new version is here: https://github.com/JuliaRegistries/General/pull/21055. So this request can be closed.


Handle the case when the type is null

When by mistake the type of the action is null, The execution breaks at the library.
This used to work before https://github.com/reduxactions/redux-actions/commit/bea186f6b2467d3f706b54649b1dd4d6f53ffc98#diff-52ace21827ea0130a413a22cd1697dc8

I can take a look at this this weekend. I need a little time to familiarize myself with the codebase since I've only just joined as a maintainer.

We are aligning with FSA, which says the type must be a string.
https://github.com/redux-utilities/flux-standard-action#type

@timche Thats correct, But if mistake an action is called with null, It should not break the application, It should only log a warning message, because it wont have any side effects


Questioning use of _window

We are using the variable _window instead of window to reference the global object.
Using this one liner, we make sure that _window is referencing the real global object.
const _window = 'undefined' == typeof unsafeWindow ? window : unsafeWindow;
Problem is that this one liner is not included in the library and have to be placed manually.
Is it possible to include that one liner in the src folder so that it is included in the built library?

we could also overwrite window to unsafeWindow at the start of the userscript.
if("undefined" != typeof unsafeWindow) {
    //trust me bro
    window = unsafeWindow;
}

we could also overwrite window to unsafeWindow at the start of the userscript.
if("undefined" != typeof unsafeWindow) {
    //trust me bro
    window = unsafeWindow;
}

this actually works

seems to be fixed in a new tampermonkey version


Add reviewer if the PR has a label

In our workflow, we want to assign a reviewer to a PR only if it has a certain label.
It would be cool to have this option as well, so that reviewer will be assigned if the Pr has the specified label.

So I got this workflow working by using the event labeled instead of opened
and adding an if condition if: contains(github.event.pull_request.labels.*.name, '<label_name>') for the job. So for all other labels, the action is skipped and only runs if the PR has a certain label.
Closing the issue as this is what I was expecting!


Multiple shapes disabled in objects crash the engine

Godot version:
All 3.0 branches (stable/master)
OS/device including version:
All
Issue description:
When you make a Scene with some disabled shapes and instance in a main scene, the engine crashes.
Steps to reproduce:

Open the Project provided as attachment
Click on "Pool Initialize" button
Click on ">" many times
The engine crashes

Minimal reproduction project:
RigidBenchmarkOMatic.zip
My opinion:
The disabled shapes are acting like live physic objects (not really disabled) and crashes the engine due to a lot of physics processing. It must be really disabled, like there is no shape when you click on "Disabled" property.

The engine crashes because the Asteroid.tscn scene in the example project contains an active and empty CollisionPolygon2D - Shape.
Open the Asteroid.tscn scene and check the last CollisionPolygon2D. It is set as active and does not contain any data.
When the engine starts this results in a shape with segments etc. set to NULL and segment size zero. When as asteroid collides / comes close to the player, the shape is tested and an exception is thrown.
If the shape is removed or disabled, the game runs fine.

Yes, @ShyRed, it was a shape uninitialized, i removed it, but the engine must not crash for this, there needs to add a if checking in the engine code to avoid crashing. But the main problem is that the disabled shapes are treated as active. I added 2 types of asteroids, Asteroid1 with 1 active shape and many disabled shapes, and Asteroid2 with only 1 active shape. When you check on "Asteroid1" checkbox the FPS goes down to 7 fps, but if you uncheck (use Asteroid2) there is 400 fps. Disabled shapes must be treated as disabled, not enabled in the physics system.
This is the example modified:
RigidBenchmarkOMatic.zip

Is this problem solved? I can't reproduce it

The backtrace seems similar to that in #17789 which was fixed by #18144.


how to make chromedp undetectable

What versions are you running?

$ github.com/chromedp/chromedp v0.3.1
$ chrome browser 75.0.3770.142
$ go version go1.12 darwin/amd64

What did you do? Include clear steps.
I want to crawl user comments from this page https://hotels.ctrip.com/hotel/436458.html?isFull=F#ctm_ref=hod_sr_lst_dl_n_1_1
As I know the website will detect whether a visit is from crawler or not, I should use chromedp with headless false mode and make chromedp as real browser completely.
here is my code:

allocCtx, cancel := chromedp.NewExecAllocator(context.Background(), []chromedp.ExecAllocatorOption{
		chromedp.NoFirstRun,
		chromedp.NoDefaultBrowserCheck,
		chromedp.DisableGPU,
		chromedp.UserDataDir(dir),

		chromedp.NoSandbox,
		//chromedp.Headless,

		// After Puppeteer's default behavior.
		chromedp.Flag("disable-infobars", true),
		chromedp.Flag("excludeSwitches", "enable-automation"),
		chromedp.Flag("disable-background-networking", true),
		chromedp.Flag("enable-features", "NetworkService,NetworkServiceInProcess"),
		chromedp.Flag("disable-background-timer-throttling", true),
		chromedp.Flag("disable-backgrounding-occluded-windows", true),
		chromedp.Flag("disable-breakpad", true),
		chromedp.Flag("disable-client-side-phishing-detection", true),
		chromedp.Flag("disable-default-apps", true),
		chromedp.Flag("disable-dev-shm-usage", true),
		chromedp.Flag("disable-extensions", true),
		chromedp.Flag("disable-features", "site-per-process,TranslateUI,BlinkGenPropertyTrees"),
		chromedp.Flag("disable-hang-monitor", true),
		chromedp.Flag("disable-ipc-flooding-protection", true),
		chromedp.Flag("disable-popup-blocking", true),
		chromedp.Flag("disable-prompt-on-repost", true),
		chromedp.Flag("disable-renderer-backgrounding", true),
		chromedp.Flag("disable-sync", true),
		chromedp.Flag("force-color-profile", "srgb"),
		chromedp.Flag("metrics-recording-only", true),
		chromedp.Flag("safebrowsing-disable-auto-update", true),
		chromedp.Flag("enable-automation", true),
		chromedp.Flag("password-store", "basic"),
		chromedp.Flag("use-mock-keychain", true),
	}...)

What did you expect to see?
I expect to see user comments display below 住客点评

What did you see instead?
below 住客点评 no user comment display

Sorry, but I don't see anything resembling comments or reviews in that page.
If you can reproduce this with a site that shows English, that would be helpful.

Having this issue with recaptcha too.
Tested on: https://patrickhlauke.github.io/recaptcha/

chromedp works just the same as any other library that drives Chrome using the DevTools Protocol. Anything that, eg, puppeteer or Selenium or any other package is capable of doing, can be done using chromedp. It's a cat and mouse game trying to stay ahead of sites detecting scrapers, and is somewhat of a silly exercise. I'm closing this issue, because this "issue" comes down to a "how does one use the Chrome DevTools Protocol". The short answer is "you can never fully stop sites from detecting a scraper", and the long answer is "be more clever / faster than the other site."


supershape is counterclockwise

Supershape output should be clockwise for consistency.

This is definitely tied to the orientation decision in #154

Related to #154

glued_circles() also appears to be CCW.

Actually, I verify supershape() as already being clockwise.  However, it starts somewhat clockwise of X+, instead of on X+.

Uh, I'm confused:
s = supershape(step=0.5,m1=16,m2=16,n1=0.5,n2=0.5,n3=16,r=50);
echo(cc = polygon_is_clockwise(s));
echo(s[0]);

That was the code I wrote before posting this issue.  And the output I get:
ECHO: cc = false
ECHO: [31.8073, 0]

Output says it's NOT clockwise and it starts on the X+ axis.
Does the direction it turns depend on parameter choice?

Huh.  I have no idea.  Maybe it does, because the polygon_is_clockwise() test I ran with it returned true.

path = supershape(m1=4, n1=1/2, n2=1/2, n3=4);
stroke(path,width=0.01);
echo(polygon_is_clockwise(path));
// ECHO: true

Nevermind.  Examination shows that I already applied a fix for it in my code, and forgot I had.

What about the business of not starting on the X+ axis?

That was a side effect of my original fix, and has been corrected.

supershape() and glued_circles() are both fixed for CW from X+ standardization, as is everything in shapes2d.scad.


Add optional symbol to cancel_all_orders

Coverage decreased (-0.3%) to 38.484% when pulling 8e61e5fae3c7823d6947d55ac43f9b220e12c0fd on nielskool:cancel_all_orders-optional-symbol into a511b2b2dd157d23d61b3df6af004ddd8b30c45c on sammchardy:develop.

Thanks @nielskool, find me on Telegram '@sammchardy'


[flutter roll] Revert "Remove single-view assumption from ScrollPhysics"

Reverts flutter/flutter#117503, context: https://b.corp.google.com/issues/263555070.
umm I was looking at a g3 fix for the breakages but I am not sure what would be the default value to use for devicePixelRatio. Looks like Michael will come back in January. Putting up a revert for now.

This  reverted CL also breaks the analysis of Flutter gallery, since it uses the deprecated ScrollPhysics.tolerance method, and the test fails if gallery has any lint failures.
The failures can be seen at https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/flutter-analyze/18257/overview
Could the new method be added, so that gallery and others could migrate to it, and then the old method be marked deprecated in a later commit?


Missing JSON Schema properties for Permission Scopes

This PR adds some missing schema properties expected in the various protocol scopes.
I wrote tests that validate that the messages are processed, but I think there need to be some additional testing regarding the functionality of various scopes in a subsequent PR/Issue.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.69%. Comparing base (6dc12ce) to head (5295f99).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #781   +/-   ##
=======================================
  Coverage   98.69%   98.69%
=======================================
  Files          73       73
  Lines       11264    11264
  Branches     1609     1609
=======================================
  Hits        11117    11117
  Misses        141      141
  Partials        6        6

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


Use patched aeson from hackage

Now that the index state has been updated, we can use the patched aeson-<IP_ADDRESS> from hackage instead of using our fork.
This patch contains a bug fix/memory optimisation.

bors r+


add fail-on-empty (from dorny/test-reporter)

I like markdown output but I miss fail-on-empty from dorny/test-reporter. So here is a PR to add this feature.

Useless, back on dorny/test-reporter since same option exist (use-actions-summary)


Convert TabBar to TabBar and TabBarSelection to TabBarSelection

The TabBarSelection constructor no longer has the odd maxIndex parameter and the selection is initialized by value rather than the index of the selected tab.
TabBar has a tabLabelBuilder parameter (was List) and uses the TabBarSelection's values.
For now, TabBarView does not use the TabBarSelection's values. It wasn't clear how to do that.

LGTM but travis is unhappy


Enable multicluster controller on arm64 architecture

Describe what you are trying to solve
There was a troubleshooting meeting with @jainpulkit22 , and we found that the MC deployment can't be deployed successfully. It turns out there is no Antrea Multi-cluster controller image for arm64 architecture.
User is unable to deploy Antrea Multi-cluster controller on the system with arm64 architecture
Describe the solution you have in mind
Provide a new Multi-cluster controller image and multi-arch manifests for arm64 architecture.

Hi @jianjuns, do you have any comment for this issue?

I feel while arm64 support for any feature is preferrable, it is not high priority. If the efforts is trivial we can support it, if not maybe let us do later when we see a real requirement.

I have tested on this issue, I think from my side, it can be make, the only error is antrea/golang have no arm64 image, so if we use the golang image directly, it will be good.
Another issue is since lots of Arm64 is on Mac, the command sed -i should be changed into sed -i .bak
At last, seems like another hidden danger here: seems like none of our image have a specific arm64 version, maybe most of them are good now, not sure if they are good in the future.
My tools' versions:
kind version                                       kind v0.17.0 go1.19.2 darwin/arm64
`
docker version
Client:
Cloud integration: v1.0.29
Version:           20.10.22
API version:       1.41
Go version:        go1.18.9
Git commit:        3a2c30b
Built:             Thu Dec 15 22:28:41 2022
OS/Arch:           darwin/arm64
Context:           default
Experimental:      true
Server: Docker Desktop 4.16.2 (95914)
Engine:
Version:          20.10.22
API version:      1.41 (minimum version 1.12)
Go version:       go1.18.9
Git commit:       42c8b31
Built:            Thu Dec 15 22:25:43 2022
OS/Arch:          linux/arm64
Experimental:     false
containerd:
Version:          1.6.14
GitCommit:        9ba4b250366a5ddde94bb7c9d1def331423aa323
runc:
Version:          1.1.4
GitCommit:        v1.1.4-0-g5fd4c4d
docker-init:
Version:          0.19.0
GitCommit:        de40ad0
`


[FEATURE] Handle Global Menu depth independently from toc

If the toctree on the root page is limited to a certain level count, like :maxdepth: 1 still the complete page tree is displayed in Sphinx.
However as not ever project needs a full document tree and the creation of the document tree is time-consuming, an option for the guides.xml, max-menu-depth, was introduced. max-menu-depth = 0 disables the global menu all together.

Suggestion: It is a bit odd to me to set the max-depth to 0 to disable the global menu. I would rather see 0 as unlimited which is the default. if you want to have it different you can increase it to 100.
Other way of disabling the menu:
Disable the menu could be a different option. however from a user point of view, people could also remove the compilerpass when they setup their application. Which might be an option you could provide in your typo3 app? On create of the DI container we know the settings so we could remove the GlobalMenuPass from the collected list of compiler passes. This way the compiler pass doesn't have to disable itself.
I'm open for a discussion about my suggestions.

@jaapio I applied your suggestions. We dont need to disable this so whoever wants to disable it probably can remove the pass like you said or introduce the option


Update README.md for nuget documentation

resolve https://github.com/openexr/openexr/issues/345
edit: preview here
https://github.com/openexr/openexr/tree/8ad94f6b4c20c5fb2831d0cd083158613acd802b

@coast77777777777, this is great, thanks for contributing. Could we invite you to the next steering committee meting to discuss in person? Thursday, August 8, 1pm PDT via Zoom, the meeting info is on the ASWF calendar here: https://lists.aswf.io/g/openexr-dev/viewevent?repeatid=15006&eventid=494941&calstart=2019-08-08

Greetings Cary,
I’m happy to contribute as needed and I should be able to attend the zoom meeting on August 8.
Thanks,
Rob

On Jul 29, 2019, at 12:07 PM, Cary Phillips<EMAIL_ADDRESS>wrote:
@coast77777777777, this is great, thanks for contributing. Could we invite you to the next steering committee meting to discuss in person? Thursday, August 8, 1pm PDT via Zoom, the meeting info is on the ASWF calendar here: https://lists.aswf.io/g/openexr-dev/viewevent?repeatid=15006&eventid=494941&calstart=2019-08-08
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Checking back on this PR, the nuget instructions are a good addition, but it seems that they'd go better as a section in INSTALL.md, rather than in the FAQ.

Closing this for now, it seems to have gone stale.


Hide built hideout modules

Item detail pages currently show all hideout modules regardless of whether the user has already built the module. This changes it so that built modules are hidden by default, but the user can display them with the toggle similar to how the crafts and barters tables work:

Also allows for showing and hiding of quests requiring or providing items, depending on quest completion:

.deploy

.deploy

.deploy


Tooltip not working

When I click on a event, the tooltip does not appear. I see the following error in the console.
Any ideas?

Tooltip does not appear, too.

When I click on a event, the tooltip does not appear. I see the following error in the console.
Any ideas?

Having the same issue of the tooltip not opening but with no errors in the console.


editor.requestExit() not called when closing the editor frame

Hi,
When i close VisEditor with close button in window decoration, it doesn't call dispose()  method nor show a "confirm exit" dialogue.
But when i go to file->exit button, it works properly.
I think the problem is here
https://github.com/kotcrab/VisEditor/blob/master/Editor/src/com/kotcrab/vis/editor/ui/EditorFrame.java#L164

That's interesting. Looks like the LWJGL Timer thread is created only on Windows. I'll search for some other method. Thanks for reporting.


Visualisation 10 get data and create json from it.

Need to visit data source page, download the data (.csv or some other format) and create json from it. Consider if data source provides api => less work for us. Json could be created using some small script tai some web app that converts csv to json.
Acceptance criteria: json file with data in comment for the task. Later these would be used to create UI without back-end and to initialize database

Global-GHG-Emissions-by-sector-based-on-WRI-2020.txt


[Task] JSON Routes are undocumented

What should be done
We should add a clear documentation on how to use JSON Routes.
We should also add example with Query.

:tada: This issue has been resolved in version 1.0.0-beta.25 :tada:
The release is available on:

GitHub release
v1.0.0-beta.25

Your semantic-release bot :package::rocket:


Small typo

In the sentence "but they most both run on the same operating system (Windows or Linux)", I think "most" should be "must".

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

ID: 3847e16b-92ae-0202-c7e1-ca9a8474e65f
Version Independent ID: a25b6b5b-05ab-b3b2-616c-5d06eadb928d
Content: Azure Functions Premium plan
Content Source: articles/azure-functions/functions-premium-plan.md
Service: azure-functions
GitHub Login: @cachai2
Microsoft Alias: cachai

@RichG36 - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


How to make the acquisition function work?

https://github.com/fmfn/BayesianOptimization/blob/master/examples/visualization.ipynb
Looking at the example here, I tried to create my target function and visualize bayesian optimization.
However, looking at the results, the acquistion function does not work at all.

What should I do to plot acquistion function graph?
I'll put the code below.
import pybamm
import numpy as np
import matplotlib.pyplot as plt
from matplotlib import gridspec
from bayes_opt import BayesianOptimization
from bayes_opt import UtilityFunction
def fortemplimit(x,k,a) :
model = pybamm.lithium_ion.DFN({"thermal": "lumped"})
parameter_values = model.default_parameter_values
experiment = pybamm.Experiment(
[
(
"Charge at " + str(x) + " C for " + str(a) + " seconds",
"Rest for " + str(k) + " seconds",
"Hold at 4.1 V until 50 mA",
)
]
,
termination="80% capacity",
)
sim = pybamm.Simulation(
model, experiment=experiment, solver=pybamm.CasadiSolver("fast with events"),parameter_values=parameter_values,
)
sim.solve()
solution = sim.solution
return (-1) * solution["Time [s]"].entries[-1]
def experiment_func(x):
model = pybamm.lithium_ion.DFN({"thermal": "lumped"})
parameter_values = model.default_parameter_values
#fast_solver = pybamm.CasadiSolver(atol=1e-3, rtol=1e-3, mode="fast")
vmax = 4.1
experiment = pybamm.Experiment(
[
(
"Charge at " + str(x) + " C until 4.1 V",
"Hold at 4.1 V until 50 mA",
        )
    ]
    ,
    termination="80% capacity",
)
sim = pybamm.Simulation(
    model, experiment=experiment, solver=pybamm.CasadiSolver("fast with events"),parameter_values=parameter_values,
)
sim.solve()
solution = sim.solution
Times=[]

for t in range(int(solution["Time [s]"].entries[-1])) :
    if solution["Volume-averaged cell temperature [K]"](t) >= 298.66154 :
        Times.append(t)

if not Times :
    return (-1) * (solution["Time [s]"].entries[-1])

else :
    k=max(Times)-min(Times)
    a=min(Times)
    return fortemplimit(x,k,a)

#print(experiment_func(2.7))
x = np.linspace(1, 3, 30)
real_x=x.reshape(-1,1)
Y=[]
for i in x :
Y.append(experiment_func(i))
real_Y=np.array(Y).reshape(-1,1)
optimizer = BayesianOptimization(
f=experiment_func,
pbounds={"x": (1, 3)},
verbose=1,
random_state=987234,
)
optimizer.maximize(init_points=7, n_iter=10, acq="ei", xi=1e-01)
print(optimizer.max)
def posterior(optimizer, x_obs, y_obs, grid):
optimizer._gp.fit(x_obs, y_obs)
mu, sigma = optimizer._gp.predict(grid, return_std=True)
return mu, sigma

def plot_gp(optimizer, x, y):
fig = plt.figure(figsize=(16, 10))
steps = len(optimizer.space)
fig.suptitle(
'Gaussian Process After {} Steps'.format(steps),
fontdict={'size': 25}
)
gs = gridspec.GridSpec(2, 1, height_ratios=[3, 1])
axis = plt.subplot(gs[0])
acq = plt.subplot(gs[1])

x_obs = np.array([[res["params"]["x"]] for res in optimizer.res])
y_obs = np.array([res["target"] for res in optimizer.res])

mu, sigma = posterior(optimizer, x_obs, y_obs, x)
axis.plot(x, y, linewidth=3, label='Target')
axis.plot(x_obs.flatten(), y_obs, 'D', markersize=8, label=u'Observations', color='r')
axis.plot(x, mu, '--', color='k', label='Prediction')

axis.fill(np.concatenate([x, x[::-1]]),
          np.concatenate([mu - 1.9600 * sigma, (mu + 1.9600 * sigma)[::-1]]),
          alpha=.6, fc='c', ec='None', label='95% confidence interval')

axis.set_xlim((1, 3))
axis.set_ylim((None, None))
plt.gca().invert_yaxis()
axis.set_ylabel('Objective value of charging time (second)', fontdict={'size': 15})
axis.set_xlabel('Applied Current rate (C)', fontdict={'size': 15})
plt.show()
utility_function = UtilityFunction(kind="ei", kappa=0, xi=1e-01)
utility = utility_function.utility(x, optimizer._gp, 0)
acq.plot(x, utility, label='Utility Function', color='purple')
acq.plot(x[np.argmax(utility)], np.max(utility), '*', markersize=15,
         label=u'Next Best Guess', markerfacecolor='gold', markeredgecolor='k', markeredgewidth=1)
acq.set_xlim((1, 3))
acq.set_ylim((0, np.max(utility) + 0.5))
acq.set_ylabel('Utility', fontdict={'size': 20})
acq.set_xlabel('x', fontdict={'size': 20})

axis.legend(loc=2, bbox_to_anchor=(1.01, 1), borderaxespad=0.)
acq.legend(loc=2, bbox_to_anchor=(1.01, 1), borderaxespad=0.)
plt.show()

plot_gp(optimizer, real_x, real_Y)

Hi, I'm happy to help but I'm not going to spend 30 minutes getting this example to run.
can you please place the code you would like us to look at between three back ticks like this:

If you give us an example that we can quickly run, we will be happy to help.

Looking at the graph, the acquisition function is not drawn at all.
If you look at the graph I uploaded, the range of the axis of the acquisition function is from 0 to 1.
This is precisely why I believe there is an error.
I set the range of the axis of the acquisition function from 1 to 3
-> axis.set_xlim((1, 3)).
But the graph is from 0 to 1. What should I do to resolve this error?

First of all, thank you so much for watching. Sorry for taking so long to execute the code. Reducing the init points and n_iter in the optimization process will shorten the code execution time, but it will still take about 10 minutes. I'm so sorry.
I think an error appears in the axis setting problem of the acquisition function. I definitely set the x-axis from 1 to 3, but in the actual graph, the x-axis from 0 to 1 is set. It would be nice if this part could be solved.

Like I said: I cannot run the code you posted. That makes it difficult for me to help you.
Please post the code the way I suggested, maybe have a read of this github article on posting code.
The acquisition function is working perfectly, notice in your top points that you are getting an increasing number of samples near the maximum. What is not working is your plot; if you post useable code I will be able to tell you why.

Thanks to your answer, it was solved. Thank you vert much!


⚠ Turnos 24x7 has degraded performance

In c5e5489, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 956 ms

Resolved: Turnos 24x7 performance has improved in 987e206.


Request fails when adding the top_n_tokens parameter

System Info
Adding the top_n_tokens parameter to a request results in an error.
Traceback (most recent call last):
  File "/opt/conda/bin/text-generation-server", line 8, in <module>
    sys.exit(app())
  File "/opt/conda/lib/python3.10/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/opt/conda/lib/python3.10/site-packages/typer/core.py", line 216, in _main
    rv = self.invoke(ctx)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/typer/main.py", line 683, in wrapper
    return callback(**use_params)  # type: ignore
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/cli.py", line 89, in serve
    server.serve(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 228, in serve
    asyncio.run(
  File "/opt/conda/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 636, in run_until_complete
    self.run_forever()
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 603, in run_forever
    self._run_once()
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 1909, in _run_once
    handle._run()
  File "/opt/conda/lib/python3.10/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
  File "/opt/conda/lib/python3.10/site-packages/grpc_interceptor/server.py", line 165, in invoke_intercept_method
    return await self.intercept(
> File "/opt/conda/lib/python3.10/site-packages/text_generation_server/interceptor.py", line 21, in intercept
    return await response
  File "/opt/conda/lib/python3.10/site-packages/opentelemetry/instrumentation/grpc/_aio_server.py", line 82, in _unary_interceptor
    raise error
  File "/opt/conda/lib/python3.10/site-packages/opentelemetry/instrumentation/grpc/_aio_server.py", line 73, in _unary_interceptor
    return await behavior(request_or_iterator, context)
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 100, in Prefill
    generations=[generation.to_pb() for generation in generations],
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 100, in <listcomp>
    generations=[generation.to_pb() for generation in generations],
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/types.py", line 89, in to_pb
    return generate_pb2.Generation(
TypeError: Argument for field generate.v2.Generation.top_tokens is not iterable

Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction

Start any model: docker run --gpus all --shm-size 1g -p 80:80 ghcr.io/huggingface/text-generation-inference:1.3.3 --model-id bigscience/bloom-560m --max-top-n-tokens 16
The request with the json body

{
	"inputs": "1 + 2 = "
}

works, when the requests json body is changed to
{
	"inputs": "1 + 2 = ",
	"parameters":
	{
		"top_n_tokens": 4
	}
}

one gets 424 Failed Dependency with the error Request failed during generation: Server error: Argument for field generate.v2.Generation.top_tokens is not iterable.
Expected behavior
The second request works (it did with 1.1.0).

Fixed #1497

Fixed in this PR: https://github.com/huggingface/text-generation-inference/pull/1497

Hi!
This is still broken. If you launch TGI and run the default example request via swagger api, you will see a http 424.
I suggest to re-open the ticket, @Narsil  and @OlivierDehaene
Related: https://github.com/huggingface/text-generation-inference/issues/1340

The issue is that according to https://github.com/huggingface/text-generation-inference/blob/17345402114e3d4a1f1c0a7fd650c734f7a992f9/proto/generate.proto#L159 Generation expects top_tokens to be of type Tokens, but in https://github.com/huggingface/text-generation-inference/blob/17345402114e3d4a1f1c0a7fd650c734f7a992f9/server/text_generation_server/models/types.py#L98 a list[Tokens] is used.


Atelier File Manager copying files instead of moving them + error message not resized for the window

I've been using Atelier File Manager for Fantia to keep all the files within a single folder for each month to find faster the files from each post.
When I first used it a some months back, I believe I randomly selected a few authors and used the settings Mode: Move to each month folder and I did not select the options below.
Just recently when I was trying to do the same for all authors, I selected some which I moved the files already however the program gave me an error message saying the files were already existing in the main month folder.
I reviewed this and confirmed the files were duplicate (a copy in the month folder and the original file still inside in each post folder)
I understand that this error is saying this because the files already exists and that isn't an issue to me. However, I would like to request if its possible to add an option to overwrite them in case the program accidentally copied the files instead of moving them to the folder I selected.
Also, this error message is not shown properly on the program window, which it may be something that you might need to fix even without the above issue.
Here is a screenshot

Oh, Sorry for my delay.
I think it's a mistake in my design.
If you move file without title, there is a high probability that the same file name will appear, e.g cover or comment.
So I think, I will remove "Write title into filename" option, and fix the error message.
About overwrite, maybe It will take some time to prepare.

Sorry, I forgot resize error message.


view doesn't show weddings person is involved in

This is just a suggestion to add, but i think its a feature that makes sense and quite easy to implement

Maybe will do for assign weddings as well?


Meteor <IP_ADDRESS> can't import materialize.scss

Hi,
I've upgraded to 3.3.3_1 and suddenly this happened when starting up my app.
=> Errors prevented startup:

   While processing files with fourseven:scss (for target web.browser):
   /client/index.scss: Scss compiler error: File to import: {poetic:materialize-scss}/bower_components/materialize/sass/materialize.scss not found. Import origin:
   /client/index.scss

I am trying to use materialize css via poetic:materialize-scss
If I remove the new package syntax I get this error
=> Errors prevented startup:

   While processing files with fourseven:scss (for target web.browser):
   /client/index.scss: Scss compiler error: File to import:
   ../.meteor/local/build/programs/server/assets/packages/poetic_materialize-scss/bower_components/materialize/sass/materialize.scss not found. Import origin:
   /client/index.scss

This error only started when switching to meteor <IP_ADDRESS>. I am going to go research the change log in meteor itself now so I can get back to building my app as soon as possible.

This is correct behavior and in line with what MDG does for less, see: https://github.com/meteor/meteor/issues/5242
Files that are intended for to be imported should be prefixed with an underscore or have the isImport flag set to true in the package.js file.

Suggested fix is ask the materialize package owner to change his package.
It's how the new build system works.
Temporary workaround for you might be to specify all the imports independently: https://github.com/poetic/meteor-materialize-sass/blob/master/bower_components/materialize/sass/materialize.scss
These imports all have the correct behavior.

Btw, If you import materialize.scss and materialize.scss is also processed by meteor, you have all the scss files twice


Update grpc-netty-shaded to 1.40.0

Updates io.grpc:grpc-netty-shaded from 1.39.0 to 1.40.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.grpc", artifactId = "grpc-netty-shaded" } ]

labels: library-update, semver-minor

Superseded by #177.


Merged versions from CY49T2 and CY49R1 for CY50

This series of commits results from changes commited in ectrans for CY49T2, merged with the version of ectrans from CY49R1 (in the contrib/ directory of ifs-source).
The final commit 00cdb6ea, tagged for-CY50 in ACCORD-NWP fork of ectrans, is the version bundled for CY50 at first (a version merged with a more recent version of ectrans, could be later bundled).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 5 committers have signed the CLA.:x: Daan Degrauwe:x: AlexandreMary:x: RyadElKhatibMF:x: ddegrauwe:x: walidchikhiDaan Degrauwe seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

OK Sam, thanks


Could not get work for some events

Hello,
is there any documentation fpr the eufy-config fileds ?
I filled in and got some events, but "motion detection events does not work."
Used a solo cam "Eufy security indoor 2K"
Thanks

Hello,
I solved the connection


Crash scarecrow:arm it is not allowed

Getting the following when client crashes.
java.lang.IllegalArgumentException: Cannot set property PropertyDirection{name=facing, clazz=class net.minecraft.util.EnumFacing, values=[north, south, west, east]} to up on block scarecrows:arm, it is not an allowed value
Full log
crash-2018-10-23_20.28.05-client.txt

Already fixed by 0308c4f6ff7f2535aea5782643e6ce145a1564ae and 8f38b6b834fefbdb2296d7a8b3bd54829acefcfc in  v1.0.3.


User-defined app execution aliases

Currently, packaged apps can declare an app execution alias for the process, which enables them to be launched from the command line as it was in the Path environment variable.
I think this is a much superior model because:

The user can turn on or off which ones they want
It doesn't need to be in a particular folder that's in the Path environment variable
The actual executable name doesn't need to match the app execution alias

However, it is not possible to define an app execution alias for any executable yourself. Most utilities don't come packaged so this is really a significant limitation. Basically, provide a box in the "app execution aliases" settings page where you can type the alias and the executable location and it's possible to type the alias in a terminal to launch the program, without even adding the folder in which its located to the Path environment variable.

A package can declare 0-100 <Application>s and each can be declared with AppListEntry="none" so it doesn't show up in applists (e.g. StartMenu). If you do this and define an app execution alias for each does it accomplish your goal?

A package can declare 0-100 <Application>s and each can be declared with AppListEntry="none" so it doesn't show up in applists (e.g. StartMenu). If you do this and define an app execution alias for each does it accomplish your goal?

I'm aware of that but that's not what I meant. I want to be able to define arbitrary app execution aliases manually for any unpackaged app.


[5.9] Fix a few test failures on macOS 14 Beta versions.

Cherry-pick of #586

Explanation: Update tests and logic for what are considered valid URLs to fix test failures when running on macOS 14 Beta versions.
Scope: Unit tests and handling of what's considered a valid URL.
Issue: rdar://108237260
Risk: Low.
Testing: This is mostly a test change. On the "main" branch, we verified these changes on macOS 13 and macOS 14 and on Linux (using the Swift CI).
Reviewer: @franklinsch

@swift-ci please test

Since 5.9 is only taking major showstoppers now I don't think this qualifies. Also, if we don't have any other 5.9 changes anymore, this won't matter.


Cucumber fixes

Should fix https://github.com/webdriverio-community/wdio-video-reporter/issues/453

rewrote cucumber demo, since existing demo didn't work anymore
Added hooks that cucumber framework uses
External lib updates

@brickfungus thanks for raising the PR. It seems that there is still 1 test failing, let me know when I can review this work.

@christian-bromann is there any way to re-run that without a push? These tests pass locally for me, and they tend to be a bit flakey for me, too
I can push an empty commit if that's easier

Just retriggered the test

okay, failed again... i'll dig into that


Only include values files if they do exist in preview.sh

This is because in helm we use "ignoreMissingValueFiles: true". I.e. we
just ignore non existing value files. Let's do the same for the
preview.sh script.
Before:
❯ make preview-all
make -f common/Makefile preview-all
make[1]: Entering directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
Error: open /home/michele/Engineering/cloud-patterns/multicloud-gitops/overrides/values-None.yaml: no such file or directory
Error: open /home/michele/Engineering/cloud-patterns/multicloud-gitops/overrides/values-None.yaml: no such file or directory
Error: open /home/michele/Engineering/cloud-patterns/multicloud-gitops/overrides/values-None.yaml: no such file or directory
Error: open /home/michele/Engineering/cloud-patterns/multicloud-gitops/overrides/values-None.yaml: no such file or directory
Error: open /home/michele/Engineering/cloud-patterns/multicloud-gitops/overrides/values-None.yaml: no such file or directory
common/scripts/preview.sh: eval: line 79: unexpected EOF while looking for matching `"'
common/scripts/preview.sh: eval: line 79: unexpected EOF while looking for matching `"'
common/scripts/preview.sh: eval: line 79: unexpected EOF while looking for matching `"'
make[1]: *** [common/Makefile:59: preview-all] Error 2
make[1]: Leaving directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
make: *** [Makefile:12: preview-all] Error 2

After:
❯ make preview-all > /dev/null ; echo $?
0

/lgtm


Add Polish translation

Adds polish translations

Hey @tonybaloney 👋  I've created PR with Polish translation, have a look please 🙂

Codecov Report

Merging #262 (e006f2d) into master (abdb9fd) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #262   +/-   ##
======================================
  Coverage    8.25%   8.25%
======================================
  Files           3       3
  Lines         642     642
  Branches       99      99
======================================
  Hits           53      53
  Misses        589     589

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

Thank you for your contribution


cellState.selectedPosition() returns wrong position for May 1st, 2017

Sorry for creating another issue.. but I probably found a 'bug' (?):
I have the range of dates selected 18th April through May 11th (dates doesn't matter, the main thing is that May 1st should be in the range):

I have a rule that if  the cellState.selectedPosition() == .left and cellState.selectedPosition() == .right then it should has a dark blue view (start and end dates). But as you can see the May 1st also is selected (has this dark view). And as I see it falls into the if selectedPosition() == .left
And this happens to all the dates that are in the left top corner of the month (for example Jan 1st 2018).
Does it reproduces on your side?

Ah yes.
this one was already logged here -> https://github.com/patchthecode/JTAppleCalendar/issues/280
Its a small fix i need to do on my end.
I'll up this over the weekend.
I'll close the issue here and flag it as a duplicate.

@patchthecode got it! thanks a lot. sorry for duplication :)

It cool man. :]


Add Project Guide To Installation

Need to add project installation guide in the form of code snippets and GIFs. Easy to use and well documented.

Guide To Installation Has Been Added And Documented

Task Has Been Completed


[FEATURE] Functional zkforge test

🚀 Feature

Implementation of foundry tests.
Motivation

The test capabilities of foundry are, in my opinion, one of the best features compared to hardhat. Having foundry tests in foundry-zksync would be extremely helpful to any developer.
Implementation

It seems like this feature is already a WIP, the command zkforge test can be run but the Rust implementation is pointing to a specific contract (a Greeter contract). Ideally the implementation would just be to complete this, assuming that the foundry testing already works with the Greeter contract.
Additional context
zkforge test pointing to the greeter contract:

Hey! Yes, this feature is on WIP right now. It's a complex feature, because we need to plug a special version of the zkevm into the foundry codebase, but it is something that we need to do in order for this project to be really useful.


exp: misc. guide and ref. updates

Extracted from https://github.com/iterative/dvc.org/pull/2909

More concrete description of what exp apply does
Remove some implementation details from exp run (now in the UG)
Move How-to after Exp Mgmt in UG
Single place for info. about "persisting" experiments
Reword/ clarify https://dvc.org/doc/user-guide/troubleshooting#git-auth
Other misc. edits (clean Markdown, add tooltips, links to guide, etc.)

Mind double checking the DVC behavior explanations when you can @dberenbaum ?