add min(1) on schema and reference to the docs for NEXTAUTH_SECRET on .env.example

Is your feature request related to a problem? Please describe.
Some users are not too familiar with Next-Auth to know that the NEXTAUTH_SECRET needs to be a valid string. Unfortunately, the error logs on the terminal are not so clear.
[next-auth][error][SIGNIN_OAUTH_ERROR]
https://next-auth.js.org/errors#signin_oauth_error "ikm" must be at least one byte in length {
  error: {
    message: '"ikm" must be at least one byte in length',
    stack: 'TypeError: "ikm" must be at least one byte in length\n' +
      '    at normalizeIkm (/mnt/repos/website/testing/node_modules/@panva/hkdf/dist/node/cjs/index.js:26:15)\n' +
      '    at hkdf (/mnt/repos/website/testing/node_modules/@panva/hkdf/dist/node/cjs/index.js:47:60)\n' +
      '    at getDerivedEncryptionKey (/mnt/repos/website/testing/node_modules/next-auth/jwt/index.js:114:34)\n' +
      '    at Object.encode (/mnt/repos/website/testing/node_modules/next-auth/jwt/index.js:48:34)\n' +
      '    at createState (/mnt/repos/website/testing/node_modules/next-auth/core/lib/oauth/state-handler.js:29:34)\n' +
      '    at getAuthorizationUrl (/mnt/repos/website/testing/node_modules/next-auth/core/lib/oauth/authorization-url.js:68:53)\n' +
      '    at processTicksAndRejections (node:internal/process/task_queues:96:5)\n' +
      '    at async Object.signin (/mnt/repos/website/testing/node_modules/next-auth/core/routes/signin.js:37:24)\n' +
      '    at async NextAuthHandler (/mnt/repos/website/testing/node_modules/next-auth/core/index.js:238:26)\n' +
      '    at async NextAuthNextHandler (/mnt/repos/website/testing/node_modules/next-auth/next/index.js:23:19)',
    name: 'TypeError'
  },
  providerId: 'github',
  message: '"ikm" must be at least one byte in length'
}

Describe the solution you'd like

Add some reference link in the .env.example pointing to the NEXTAUTH_SECRET docs in the Next Auth documentation.
Add .min(1) in the .env schema on the NEXTAUTH_SECRET variable, to make the error log more clear.

Not sure why you are getting this error.
In dev, NextAuth will automatically hash your options in order to generate a secret.
In prod, NextAuth will throw an error if the NEXT_AUTH_SECRET is not set.
Link to relevant docs: https://next-auth.js.org/configuration/options#secret

since i didn't see anyone else complaining about this, i'm closing the issue


Remove space and version specifier from result

There are some projects, such this one, that define ruby version in Gemfile with ~> 1.2.3 specifier.
In this PR I've just used tr to remove them (and blank chars).

If a version range is specified in Gemfile, would it be more correct to check Gemfile.lock for the actual locked Ruby version?

@aripollak Nice catch, maybe would be better to check Gemfile.lock to identify this pattern:
RUBY VERSION
   ruby 2.3.7p456

So we'll ensure that is the correct version, but doing that will change all sed patterns defined. Maybe we can open an issue for that and for this one I can just implement a test scenario, WDYT?

Hm, I don't think I understand what you're asking. I do think it's time to deprecate Ruby pre-2.1 and Bundler pre-1.13, so the patchlevel and Gemfile can be ignored. By looking at the machine-generated Gemfile.lock instead of Gemfile, I think it will be easier to recognize all bundler-supported scenarios, including version ranges.

I do think it's time to deprecate Ruby pre-2.1 and Bundler pre-1.13, so the patchlevel and Gemfile can be ignored.

Agreed!
Now about changes: what I mean was to just implement tests scenario for this PR (reading from Gemfile) then in a separate PR we can refactor sed scripts to identify pattern reading  Gemfile.lock. So we can discuss and implement behavior such this one that you described above, WDYT?

I'd prefer not to include these changes as is. It would choose the earliest allowed version of Ruby, which might be counterproductive in some cases since this plugin takes precedence over the per-shell or global version.

No problem, so we can close this one.


Complete implementation fo dev signatures

@dr-orlovsky this isn't listed in the rgb 0.11 board (https://github.com/orgs/RGB-WG/projects/17), do you still plan to do it in a 0.11 release or should we move the task to a 0.12 milestone?

Moved


internal: React types to 18

Codecov Report

Merging #1887 (282b871) into master (a57b9fa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1887   +/-   ##
=======================================
  Coverage   98.56%   98.56%
=======================================
  Files         101      101
  Lines        1885     1885
  Branches      393      393
=======================================
  Hits         1858     1858
  Misses         13       13
  Partials       14       14

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


show round time in effort field

Describe the problem
When I try to import my effort from Jira to ADO, then If It's 10m then it's converting value something like 0.3333333 which is but obvious weird to show, and I am using  MapRemainingWork as mapper. Please find my config json code of importing effor below.
Please attach the following files:
{
"source": "TimeSpent",
"target": "Microsoft.VSTS.Scheduling.Effort",
"mapper": "MapRemainingWork"
}

Check this section in our FAQ and come back if that still did not solve your problem: https://github.com/solidify/jira-azuredevops-migrator/blob/master/docs/faq.md#7-how-to-migrate-the-work-log-time-spent-remaining-estimate-fields

Do I need to shift at latest 3.0.407? right now I am at 3.0.307.

New version does not address the effort fields. ^^
Maybe try to migrate the field to a custom integer field for the purpose of testing?
I don't know if maybe the VSTS fields could have automation behind them or not.
Here is a config sample that has worked for me before:
      {
        "source": "timespent$Rendered",
        "target": "Custom.TimeSpentHours"
      },
      {
        "source": "timespent",
        "target": "Custom.TimeSpentSeconds"
      },

Maybe the MapRemainingWork mapper could help you as well:
        {
          "source": "timeestimate",
          "target": "Microsoft.VSTS.Scheduling.RemainingWork",
          "mapper": "MapRemainingWork",
          "for": "Bug,Task"
        },

I don't think the tool can convert hours to minutes as of today, because only seconds and hours is available in jira. If a field has decimal value for the number of hours, then the same value will be migrated to ADO.

Thanks for response this does work for me but let's say I've one task which have time spent 25.666666667 then It's showing 25.6666667 in my effort field in ADO, is there any way that I use so I get round figure value or 25.7 something because 25.66666667 is very weird to show. @Alexander-Hjelm

Not to my knowledge, at the moment. We would need to update the MapRemainingWork mapper, or write a new mapper entirely.
One workaround could be to write an automation in powershell that accomplishes the same thing between your export and import steps.

Ok, and I noticed one thing that my history which in Jira work item which isn't fully showing in my ADO work item.

The usually means there is data which you have not mapped, or there were errors during the export or import and some revisions were not fully migrated. Check the log files!

Thanks mate!! for help, last thing do you have any reference where I can take help from to achieve the idea you suggested about automation in powershell?

OpenAI and general scripting knowledge probably ^^ I can check if we have any sample scripts that you can base your own work on. Stand by

fix-field.py.txt
Here is a fairly undocumented python script which we have used to fix fields post-migration!

Thanks Mate !!! Appreciate


chore(main): release 0.3.0

:robot: I have created a release beep boop
0.3.0 (2023-06-19)
Features

support iceberg offset parsing (#9) (41510fd)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/contiamo/spark-prometheus-export/releases/tag/v0.3.0 :sunflower:


Exception when using FluentFTP inside Docker Container

FTP OS: Unix
FTP Server: Pure-FTPd
Computer OS: Unix Container
FluentFTP Version: 35.0.5
Description:
I have a Application written in NET Core 3.1, when I execute the Application on my Windows Machine, Files can be uploaded as expected. When I build a Docker Container from exectly the same Source Code, I am unable to upload Files correctly.
The first File seems to be uploaded when I read through the logs, but in fact it is just a 0 Byte File on the FTP Server. The second upload fails with an Exception.
As I am starting both the plain Windows Version and the Docker Version on my Windows Machine, the FTP Server in general is reachable and answering Requests from my Machine. Either there is a Bug in FlutenFTP when executed in a Docker Container, or I am having Issues with Passive FTP from my Docker Container.
Already spent various hours debugging this Problem, I am unable to find the Issue, any help on this Topic is highly appreciated.
Dockerfile:
FROM mcr.microsoft.com/dotnet/aspnet:3.1
COPY bin/Release/netcoreapp3.1/publish/ app/
WORKDIR /app
EXPOSE 80
ENTRYPOINT ["dotnet", "xxx-xxxxx.dll"]

Code:
public static void RunExport()
{
	FtpClient ftpClient = new FtpClient("xxx.xxx.xxx",xx,"xxx","xxx");
	ftpClient.AutoConnect();

	foreach (String Path in Directory.GetFiles(Path.GetDirectoryName(Path.Join(AppDomain.CurrentDomain.BaseDirectory,"Queries/"))).OrderBy(x => x))
	{
	  ParseAndExecuteSQL(Path, ftpClient);
	}
	ftpClient.Disconnect();
}

public static void ParseAndExecuteSQL(String SQLPath, FtpClient ftpClient)
{
	//Some other Code here

	DataTable CSVData = Generic.ExecuteSelectQuery(SQLQuery);
	byte[] FTPFile = Generic.GetCSV(CSVData);
	Console.WriteLine("Database Export | Path:" + FTPPath + " Rows:" + CSVData.Rows.Count + " Size:" + (FTPFile.Length/1024).ToString() + "kb");
	ftpClient.Upload(FTPFile,FTPPath);

	//Some other Code here
}

Exception:
FluentFTP.FtpException
Error while uploading the file to the server. See InnerException for more info.

FluentFTP.FtpException: Error while uploading the file to the server. See InnerException for more info.
 ---> System.NullReferenceException: Object reference not set to an instance of an object.
   at FluentFTP.FtpClient.CheckFileExistsBySize(FtpSizeReply sizeReply)
   at FluentFTP.FtpClient.FileExists(String path)
   at FluentFTP.FtpClient.UploadFileInternal(Stream fileData, String localPath, String remotePath, Boolean createRemoteDir, FtpRemoteExists existsMode, Boolean fileExists, Boolean fileExistsKnown, Action`1 progress, FtpProgress metaProgress)
   --- End of inner exception stack trace ---
   at FluentFTP.FtpClient.UploadFileInternal(Stream fileData, String localPath, String remotePath, Boolean createRemoteDir, FtpRemoteExists existsMode, Boolean fileExists, Boolean fileExistsKnown, Action`1 progress, FtpProgress metaProgress)
   at FluentFTP.FtpClient.Upload(Byte[] fileData, String remotePath, FtpRemoteExists existsMode, Boolean createRemoteDir, Action`1 progress)

Logs:

# AutoConnect()

# AutoDetect(True, False)

# Connect()

# AutoConnect()

# AutoDetect(True, False)

# Connect()
Status:   Connecting to ***:21
Response: 220---------- Welcome to Pure-FTPd [privsep] [TLS] ----------
Response: 220-You are user number 2 of 100 allowed.
Response: 220-Local time is now 16:12. Server port: 21.
Response: 220-This is a private system - No anonymous login
Response: 220-IPv6 connections are also welcome on this server.
Response: 220 You will be disconnected after 15 minutes of inactivity.
Status:   Detected FTP server: PureFTPd
Command:  AUTH TLS
Response: 234 AUTH TLS OK.
Status:   FTPS Authentication Successful
Status:   Time to activate encryption: 0h 0m 0s.  Total Seconds: 0.2250653.
Command:  USER ***
Response: 331 User *** OK. Password required
Command:  PASS ***
Response: 230 OK. Current directory is /
Command:  PBSZ 0
Response: 200 PBSZ=0
Command:  PROT P
Response: 200 Data protection level set to "private"
Command:  FEAT
Response: 211-Extensions supported:
Response: EPRT
Response: IDLE
Response: MDTM
Response: SIZE
Response: MFMT
Response: REST STREAM
Response: MLST type*;size*;sizd*;modify*;UNIX.mode*;UNIX.uid*;UNIX.gid*;unique*;
Response: MLSD
Response: AUTH TLS
Response: PBSZ
Response: PROT
Response: UTF8
Response: TVFS
Response: ESTA
Response: PASV
Response: EPSV
Response: SPSV
Response: 211 End.
Status:   Text encoding: System.Text.UTF8Encoding+UTF8EncodingSealed
Command:  OPTS UTF8 ON
Response: 200 OK, UTF-8 enabled
Command:  SYST
Response: 215 UNIX Type: L8

# Upload("/system/export/DailyPeak.csv", Overwrite, False)

# FileExists("/system/export/DailyPeak.csv")
Command:  SIZE /system/export/DailyPeak.csv
Response: 213 0

# DeleteFile("/system/export/DailyPeak.csv")
Command:  DELE /system/export/DailyPeak.csv
Response: 250 Deleted /system/export/DailyPeak.csv

# OpenWrite("/system/export/DailyPeak.csv", Binary)
Command:  TYPE I
Response: 200 TYPE is now 8-bit binary

# OpenPassiveDataStream(PASV, "STOR /system/export/DailyPeak.csv", 0)
Command:  PASV
Response: 227 Entering Passive Mode (138,68,96,224,117,201)
Status:   Connecting to ***:30153
Command:  STOR /system/export/DailyPeak.csv
Response: 150 Accepted data connection
Status:   FTPS Authentication Successful
Status:   Time to activate encryption: 0h 0m 0s.  Total Seconds: 0.0786551.
Status:   Disposing FtpSocketStream...

# Upload("/customer/export/DailyPeak.csv", Overwrite, False)

# FileExists("/customer/export/DailyPeak.csv")
Command:  SIZE /customer/export/DailyPeak.csv

Just set the remoteExists mode to NoCheck or something and see if it works. If it works, its probably a bug inside FileExists. Best way to debug this is to download the source and add it to your project, so you can hit breakpoints and debug cleanly.

We moved our application to Docker we can now interact directly with the Files on Disc, we have already removed FluentFTP from that application, uploading Files via FTP is currently not necessary anymore,
We currently won't spend Time and Ressources debugging this Issue. If you need more details regarding our Environment or anything else, feel free and reach out to me. You might also close this Issue if you won't further investigagte.
Kind regards and thanks for your work on this Project!

Thanks!

Thanks!


HSEARCH-3323 Search 6 groundwork - Restore support for scrolling

https://hibernate.atlassian.net/browse/HSEARCH-3323
@yrodiere here is the draft for the feature, fell free to complete / fix it if you want provide this feature before July.
if the CI passes, this is ready for review
draft again

Hi @yrodiere I hadn't finished with the changes actually :P I'll let you know when I done.

Hum ok, I got a notification with the description being edited to tell me it's ready for review... Something must have gone wrong. Sorry.

@yrodiere I've updated the branch adding a test the loading behavior on scrolling... it fails now for the Lucene backend for the reasons you expressed above... Next days I'll finish it, doing all the staff we agreed to do.

Thank you @yrodiere for completing the job.
+1 to merge from me

Merged, thanks!


TPAT and TRT - no kernel image is available for execution on the device

Hi,
I've successfully converted a model into TensorRT using TPAT generated plugin using the following command:
/usr/src/tensorrt/bin/trtexec --onnx=model_batch1_tpat.onnx --saveEngine=model.plan --buildOnly --verbose --fp16 --workspace=6000 --explicitBatch --noTF32 --plugins="tpat_onehot.so"

but after running trtexec test using this command:
/usr/src/tensorrt/bin/trtexec  --loadEngine=model.plan --verbose --workspace=6000  --plugins="./tpat_onehot.so"

I'm getting the following errors:
[06/02/2022-02:55:34] [E] [TRT] ../rtExt/cuda/cudaPluginV2DynamicExtRunner.cpp (108) - Cuda Error in execute: 209 (no kernel image is available for execution on the device)
[06/02/2022-02:55:34] [E] [TRT] FAILED_EXECUTION: std::exception

I managed to get one TPAT plugin for tpat_onehot.so which doesn't throw this error, but I don't see any difference in the way I generated the plugins. Is there something about the non-deterministic process of generating a plugin using TVM that can cause this behavior?
Thank you!

It looks like the error is caused by your environment.
https://forums.developer.nvidia.com/t/tensorrt-no-kernel-image-is-available-for-execution-on-the-device-error-48-hex-0x30/62307
Maybe you should modify the 'sm' in TPAT/python/trt_plugin/Makefile. you can learn from this issue:
https://github.com/Tencent/TPAT/issues/7


Pre-rendered links in code get converted to Markdown on render

A pre-rendered link in either a inline code or a code block will look like a raw Markdown link after rendering.
This is a result of the fix to #49: "Retain pre-formatted links" -- rev a13073c. In the current repo head the code can be found here.
I don't have a good idea for fixing this at this time.
I will add a test for this.

Is this fixed?
This is how the test cases appear in gmail in Firefox:

Not fixed yet. The tests just test the brokenness.
I am planning on looking at this again in the next couple days. It's really annoying.

I added a partial fix: Inline code (in backticks) and code blocks that use backticks are fixed. Code blocks that are indented are not fixed.
Going to leave this open and update the description.


Optimize BucketMetadata parsing

Without wanting to derail you any further, it seems like we could still go from 2 lookups to 1 using json.find("owner") (not that I'm familiar with the API).
Originally posted by @devbww in https://github.com/googleapis/google-cloud-cpp/pull/9886#discussion_r976948979

The same comment probably applies to any other parsing of JSON objects.

We do not have time to work on this. Closing for now.


Polygon outline flicker in the GeoJSONLayer

The filled part and stroked outline of polygons in GeoJSONLayer flickers really bad due to z-fighting. It's a small issue technically, but visually very unpleasing.

Thinking about a generic solution: In v4 we have added a layerIndex uniform. Perhaps we could use this in the project function to add a tiny z offset in clip space (i.e. layerIndex * EPSILON) after rendering?

Let's discuss this to see which is better - do the offset in the world space or in the clip space

We now have the polygonOffset prop to combat this. We should make sure we use it in the examples.

@shaojingli @ibgreen
We are applying polygon offset for this example  (all layers get applied by an offset of -(layerIndex100) by default). I tried to increase the offset to -(layerIndex5000), the flickering is gone to some extent, but at higher zoom levels it is still present. If I enable fp64 prop, with existing polygon offset there is not flickering, so should I just enable fp64 on this example by default?

should I just enable fp64 on this example by default?

I think that is fine for now.


Metrics not showing

I'm running into an issue with the metrics tab of the web app.
Following the documentation (https://www.kubeflow.org/docs/external-add-ons/kserve/webapp/#metrics), I can confirm that Grafana is accessible and the metrics tab is displayed. However, no request metrics are displayed for the InferenceService.

CPU and Memory metrics are eventually shown but with a noticeable delay.
The same setup was working fine in Kubeflow 1.5 (KServe 0.7). The issue only occurs with Kubeflow 1.6 (KServe 0.8). Is anyone able to confirm these observations?

Looking at the network traffic, I can see that the web app sends requests to the grafana endpoint. For example (shortened):
https://example.com/grafana/.../knative-serving-revision-http-requests?var-namespace=undefined&var-configuration=mymodel-predictor-default
Notice how the var-namespace parameter is set to undefined which leads to an empty response from grafana. The link is working correctly when properly setting the namespace.

The issue seems to be the order of execution:
https://github.com/kserve/models-web-app/blob/a3a8e80492a81cfe307299efd926513663f60146/frontend/src/app/pages/server-info/metrics/metrics.component.ts#L18-L31
The graphs are generated in the @Input() of the InferenceServiceStatus. However, the @Input() of the namespace is only executed afterwards. Therefore, the namespace is undefined for the generation of the graphs.

Same issue here.


Node protocols: Client and Server

I temporary asked hydra to stop trying to build this PR since it's evaling to an error repeatedly: https://hydra.iohk.io/jobset/Cardano/plutus-pr-2054#tabs-errors.  Feel free to remove the label once the eval error is addressed.


Remove comment only ProductInfoHeaderValue

Version
all, but particularly 4.11.+
Describe the bug
In 4.11, we have added a constructor to ConnectorClient which takes an HttpClient.  We then pass that HttpClient down to the underlying ServicClient's constructor.  Due to the fact that we have a ProductInfoHeaderValue which has no Product (comment only), the underlying ServiceClient will throw and catch 4 exceptions.
I've created a PR to address this within ServiceClient, but they have recommended we remove the comment only ProductInfoHeaderValue:
https://github.com/Azure/azure-sdk-for-net/pull/16252
To Reproduce
Steps to reproduce the behavior:
Run this test:
        [Fact]
        public void ConnectorClient_CustomHttpClient_UserAgentHeaderAdds()
        {
            var httpClient = new HttpClient();
            ConnectorClient.AddDefaultRequestHeaders(httpClient);

            using (var connector = new ConnectorClient(new Uri("http://localhost/"), new MicrosoftAppCredentials(string.Empty, string.Empty), customHttpClient: httpClient, disposeHttpClient: false))
            {
                // Use the connector
            }

            Assert.Equal(6, httpClient.DefaultRequestHeaders.UserAgent.Count);
        }

Notice, none of the ServiceClient UserAgent headers have been added.
Expected behavior
Underlying ServiceClient UserAgent headers are added.
Remove the comment only product info header, and re-run the above test.  It will pass.
https://github.com/microsoft/botbuilder-dotnet/blob/main/libraries/Microsoft.Bot.Connector/ConnectorClientEx.cs#L200

The recommendation for anyone seeing a null reference exception due to this is to ignore.

After I upgraded VS2019 Version 16.8.2 and .NetCore3.1 version, I started seeing this error. What I actually recognized is the error was related to Microsoft.Rest.ServiceClient and even earlier versions was giving the error. But somehow suppressed.
I dig into the reason for couple of days. Here is what I found.
Null reference error is thrown on AddUserAgentEntry method in Microsoft.Rest.ServiceClient (I tried for both Microsoft.Rest.ClientRuntime.dll v2.3.20 and v2.3.21). The caller method SetUserAgent has a try catch around it. So the error is suppressed and only thrown in debug mode.
The Stack trace of the error is "   at Microsoft.Rest.ServiceClient`1.<>c.b__55_0(ProductInfoHeaderValue left, ProductInfoHeaderValue right)"
Despite of "left.Product" returned null, I found out the root cause of the error is HttpClient.DefaultRequestHeaders.UserAgent field which is filled in  AddDefaultRequestHeaders method in Microsoft.Bot.Connector.ConnectorClient.
       private void AddUserAgentEntry(ProductInfoHeaderValue pInfoHeaderValue)
        {
            lock (lockUserAgent)
            {
                if (!HttpClient.DefaultRequestHeaders.UserAgent.Contains(pInfoHeaderValue, new ObjectComparer<ProductInfoHeaderValue>((ProductInfoHeaderValue left, ProductInfoHeaderValue right) => left.Product.Name.Equals(right.Product.Name, StringComparison.OrdinalIgnoreCase))))
                {
                    HttpClient.DefaultRequestHeaders.UserAgent.Add(pInfoHeaderValue);
                }
            }
        }

In my case, the UserAgent collection has these values --> "Microsoft-BotFramework/3.1" , "BotBuilder/<IP_ADDRESS>",
"(.NETCoreApp,Version=v3.1; Microsoft Windows 10.0.18363; X64)"
In the code below,  ProductInfoHeaderValue parsedValue is not parsed correctly for this value "(.NETCoreApp,Version=v3.1; Microsoft Windows 10.0.18363; X64)". So the Product field becomes null.
 public static void AddDefaultRequestHeaders(HttpClient httpClient)
        {
            lock (httpClient)
            {
                ProductInfoHeaderValue item = new ProductInfoHeaderValue("Microsoft-BotFramework", "3.1");
                if (!httpClient.DefaultRequestHeaders.UserAgent.Contains(item))
                {
                    httpClient.DefaultRequestHeaders.UserAgent.Add(item);
                }
                ProductInfoHeaderValue item2 = new ProductInfoHeaderValue("BotBuilder", GetClientVersion());
                if (!httpClient.DefaultRequestHeaders.UserAgent.Contains(item2))
                {
                    httpClient.DefaultRequestHeaders.UserAgent.Add(item2);
                }
                if (ProductInfoHeaderValue.TryParse("(" + GetASPNetVersion() + "; " + GetOsVersion() + "; " + GetArchitecture() + ")", out ProductInfoHeaderValue parsedValue) && !httpClient.DefaultRequestHeaders.UserAgent.Contains(parsedValue))
                {
                    httpClient.DefaultRequestHeaders.UserAgent.Add(parsedValue);
                }
                httpClient.DefaultRequestHeaders.ExpectContinue = false;
            }
        }

I can provide more info if you need.

Hi @sebnema
I'm sorry you've spent time investigating this.  The root cause is a known issue.  You are correct, that the problem is in ServiceClient and has been present for years now.  They are apprehensive to take a fix at this level of the core product: https://github.com/Azure/azure-sdk-for-net/pull/16252
The reason the Bot Builder sdk has not had this issue until 4.11.0 is because we were not using the ServiceClient constructor with HttpClient param, until 4.11.0:  https://github.com/microsoft/botbuilder-dotnet/commit/918c54778e0c0e48187560661f208e034ceffb31#diff-914b91edd6e7e20fb74559dea5488a0b0ee7d63a197cd703135fb1adf44d079a
Our plan to resolve this in 4.12.0 is to replace the comment only ProductInfoHeaderValue in the Bot Builder SDK with one containing a product.

Thanks @EricDahlvang.

hi @EricDahlvang I encountered this issue when I am trying to send a message from a context. I cannot ignore this exception, else I wont be able to send a message to user like below. This worked in 4.9, but not in 4.11
    await dialogContext.Context.SendActivityAsync(MessageFactory.Text(formattedText, formattedText));

https://github.com/saikumaru/botframework-timer/blob/41a49d9e5d5c9e89b3f8fdc4de84bc5f37b376e5/Dialogs/TopLevelDialog.cs#L135
Please advice if this is related, else I can open a new one to track.

hi @EricDahlvang I encountered this issue when I am trying to send a message from a context. I cannot ignore this exception, else I wont be able to send a message to user like below. This worked in 4.9, but not in 4.11
    await dialogContext.Context.SendActivityAsync(MessageFactory.Text(formattedText, formattedText));

https://github.com/saikumaru/botframework-timer/blob/41a49d9e5d5c9e89b3f8fdc4de84bc5f37b376e5/Dialogs/TopLevelDialog.cs#L135
Please advice if this is related, else I can open a new one to track.

@sahithikkss the exception is caught internally.  You can ignore the exception, and the message will be sent as expected.

@sahithikkss the exception is caught internally.  You can ignore the exception, and the message will be sent as expected.

 It dint work for me. Appreciate if you can please share an example.  It dint work for me. Appreciate if you can please share an example. I have even attached my repo.
            try
            {
                await dialogContext.Context?.SendActivityAsync(MessageFactory.Text(formattedText, formattedText));

            }
            catch { }

No, what i'm suggesting is that you do not need to do anything with this exception.  The SDK is referencing ServiceClient, and that is where the exception is thrown and caught.  The null reference exception from ProductInfoHeaderValue will not affect your code.  If send message is not actually sending the message, then something else must be wrong.
Just hit F5 in the debugger when you see this exception, and everything will continue without issue.

No, what i'm suggesting is that you do not need to do anything with this exception.  The SDK is referencing ServiceClient, and that is where the exception is thrown and caught.  The null reference exception from ProductInfoHeaderValue will not affect your code.  If send message is not actually sending the message, then something else must be wrong.
Just hit F5 in the debugger when you see this exception, and everything will continue without issue.


Starting network with large numbers of gas objects causes sync errors

Repro Steps:

Load up experimental-rest-api => only using this because it has custom genesis. Theoretically you could do this on main with sui cli, but I have not tested that.
Boot up local rest server
Run /sui/genesis with single account containing > 1000 gas objects (use genesis.conf)
Run /sui/start and you will get this error

2022-03-20T01:29:31.744669Z  INFO rest_server: Started 4 authorities
2022-03-20T01:29:31.744761Z  INFO sui_network::transport: Listening to TCP traffic on <IP_ADDRESS>:10000
2022-03-20T01:29:31.744842Z  INFO sui_network::transport: Listening to TCP traffic on <IP_ADDRESS>:10001
2022-03-20T01:29:31.744868Z  INFO sui_network::transport: Listening to TCP traffic on <IP_ADDRESS>:10002
2022-03-20T01:29:31.744891Z  INFO sui_network::transport: Listening to TCP traffic on <IP_ADDRESS>:10003
Mar 20 01:29:31.829 INFO request completed, error_message_external: Sync error: Too many authority errors were detected: [ClientIoError { error: "Message size exceeds buffer size" }, ClientIoError { error: "Message size exceeds buffer size" }, ClientIoError { error: "Message size exceeds buffer size" }], error_message_internal: Sync error: Too many authority errors were detected: [ClientIoError { error: "Message size exceeds buffer size" }, ClientIoError { error: "Message size exceeds buffer size" }, ClientIoError { error: "Message size exceeds buffer size" }], response_code: 424, uri: /sui/start, method: POST, req_id: fc8b3740-d5d2-44e4-8735-fc5fd9277ad6, remote_addr: <IP_ADDRESS>:57533, local_addr: <IP_ADDRESS>:5000

genesis-1024.txt

#1297 will solve this issue. Closing this one.


Fix cubemap loading

Do not throw an error if there are extra files in the .zip file for cubemap texture.
Throw an error if any one of the six cubefaces do not have a texture.

Looks good. Will merge after lunch if there are no comments.


added updated approval/gasselection modals

This pull request introduces 1 alert and fixes 2 when merging d67394ddb4ae79eb77921c4afba681bb6a68e572 into 3cee54cbb972dd32ce63a4849f58788b32e8d575 - view on LGTM.com
new alerts:

1 for Potentially inconsistent state update

fixed alerts:

1 for Superfluous trailing arguments
1 for Potentially inconsistent state update

Comment posted by LGTM.com


Investigate all methods of censoring users' access to Internet services due to US sanctions/embargo/boycott.

Lack of transparency in censorship and human rights violations by companies such as Google, which by default and without prior notice, it prevents many users from US-sanctioned countries from accessing many sites and services, including non-financial sites, human rights organizations, and security tools, makes conditions for protesting, analyzing and reporting human rights issues very difficult and sometimes impossible in a country like Iran, which, on a par with China, denies people access to many Internet sites and services:

(Is this other than an irony of human rights?)
The most common type of blocking users' access due to their IP geolocation is HTTP 403 error. But this error may be caused by the server for any reason, but it is also irrelevant to the embargo. In Google services, differences may be detected through Tor. In the following case, the upper case received a HTTP 403 error as a result of a boycott by Google, and the lower case same error due to an internal server issue:

But it still looks different on different servers:

In the following, users see that the  boycott by Google is not only with HTTP 403 error, and in some cases users may receive HTTP 404 error:

Or in some services, users get HTTP 504 error instead of HTTP 403:

Distinguishing censorship due to sanctions and censorship due to prohibition in the user network is very easy in a successful HTTPS communication. But Google has different ways of boycotting users. Such as blocking TLS communication by the server (after TCP handshake):
From a server in Germany that is now geolocated by Google as Iran:
# curl -v https://test.firebaseio.com/
*   Trying <IP_ADDRESS>:443...
* Connected to test.firebaseio.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*  CAfile: /etc/pki/tls/certs/ca-bundle.crt
*  CApath: none
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to test.firebaseio.com:443
* Closing connection 0
curl: (35) OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to test.firebaseio.com:443

Use Firebase IP but use example.com for SNI:
# curl -v --resolve 'example.com:443:<IP_ADDRESS>' https://example.com/
* Added example.com:443:<IP_ADDRESS> to DNS cache
* Hostname example.com was found in DNS cache
*   Trying <IP_ADDRESS>:443...
* Connected to example.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*  CAfile: /etc/pki/tls/certs/ca-bundle.crt
*  CApath: none
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to example.com:443
* Closing connection 0
curl: (35) OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to example.com:443

Use tor proxy:
# torsocks curl -v https://test.firebaseio.com/
*   Trying <IP_ADDRESS>:443...
* Connected to test.firebaseio.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*  CAfile: /etc/pki/tls/certs/ca-bundle.crt
*  CApath: none
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.2 (IN), TLS handshake, Certificate (11):
* TLSv1.2 (IN), TLS handshake, Server key exchange (12):
* TLSv1.2 (IN), TLS handshake, Server finished (14):
* TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
* TLSv1.2 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.2 (OUT), TLS handshake, Finished (20):
* TLSv1.2 (IN), TLS handshake, Finished (20):
* SSL connection using TLSv1.2 / ECDHE-RSA-CHACHA20-POLY1305
* ALPN, server accepted to use http/1.1
* Server certificate:
*  subject: CN=firebaseio.com
*  start date: Jul  1 20:16:52 2021 GMT
*  expire date: Sep 29 20:16:51 2021 GMT
*  subjectAltName: host "test.firebaseio.com" matched cert's "*.firebaseio.com"
*  issuer: C=US; O=Google Trust Services LLC; CN=GTS CA 1D4
*  SSL certificate verify ok.
> GET / HTTP/1.1
> Host: test.firebaseio.com
> User-Agent: curl/7.76.1
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 404 Not Found
< Server: nginx
< Date: Sat, 18 Sep 2021 11:05:02 GMT
< Content-Type: text/plain
< Content-Length: 82
< Connection: keep-alive
< Access-Control-Allow-Origin: *
< Strict-Transport-Security: max-age=31556926; includeSubDomains; preload
<
* Connection #0 to host test.firebaseio.com left intact
Firebase error. Please ensure that you spelled the name of your Firebase correctly

Or like blocking TCP communication to some servers:
# traceroute -n ooni.org --tcp -m50 -O info --port=443 -A
traceroute to ooni.org (<IP_ADDRESS>), 50 hops max, 60 byte packets
 1  <IP_ADDRESS> [*]  192.811 ms  205.170 ms  203.302 ms
 2  <IP_ADDRESS> [*]  214.085 ms  219.588 ms  235.012 ms
 3  <IP_ADDRESS> [AS24940]  223.928 ms  223.910 ms  234.972 ms
 4  * * *
 5  <IP_ADDRESS> [AS24940]  235.876 ms  235.871 ms <IP_ADDRESS> [AS24940]  235.653 ms
 6  <IP_ADDRESS> [AS24940]  247.224 ms <IP_ADDRESS> [AS24940]  203.301 ms <IP_ADDRESS> [AS24940]  228.067 ms
 7  <IP_ADDRESS> [AS24940]  228.000 ms <IP_ADDRESS> [AS24940]  225.673 ms  240.931 ms
 8  <IP_ADDRESS> [AS15169]  239.249 ms <IP_ADDRESS> [AS15169]  307.365 ms <IP_ADDRESS> [AS15169]  240.167 ms
 9  * * *
10  <IP_ADDRESS> [AS15169]  315.941 ms <IP_ADDRESS> [AS15169]  252.236 ms  215.712 ms
11  <IP_ADDRESS> [AS15169]  299.810 ms  299.787 ms <IP_ADDRESS> [AS15169]  290.106 ms
12  <IP_ADDRESS> [AS15169]  246.007 ms * <IP_ADDRESS> [AS15169]  192.436 ms
13  <IP_ADDRESS> [AS15169]  204.691 ms <IP_ADDRESS> [AS15169]  206.092 ms *
14  <IP_ADDRESS> [AS15169]  259.016 ms * *
15  <IP_ADDRESS> [AS15169]  300.078 ms * *
16  * <IP_ADDRESS> [AS15169]  334.793 ms <IP_ADDRESS> [AS15169]  386.770 ms
17  * <IP_ADDRESS> [AS15169]  349.484 ms *
18  <IP_ADDRESS> [AS15169]  349.450 ms <IP_ADDRESS> [AS15169]  348.811 ms <IP_ADDRESS> [AS15169]  437.533 ms
19  <IP_ADDRESS> [AS15169]  437.534 ms <IP_ADDRESS> [AS15169]  441.481 ms <IP_ADDRESS> [AS15169]  443.749 ms
20  * * *
21  * * *
22  * * *
23  * * *
24  * * *
25  * * *
26  * * *
27  * * *
28  * * *
29  * * *
30  * * *
31  * * *
32  * * *
33  * * *
34  * * *
35  * * *
36  * * *
37  * * *
38  * * *
39  * * *
40  * * *
41  * * *
42  * * *
43  * * *
44  * * *
45  * * *
46  * * *
47  * * *
48  * * *
49  * * *
50  * * *

# traceroute -n ooni.org --icmp -m50 -A
traceroute to ooni.org (<IP_ADDRESS>), 50 hops max, 60 byte packets
 1  <IP_ADDRESS> [*]  192.712 ms  202.952 ms  202.969 ms
 2  <IP_ADDRESS> [*]  215.994 ms  215.978 ms  215.988 ms
 3  <IP_ADDRESS> [AS24940]  215.990 ms  216.967 ms  225.389 ms
 4  * * *
 5  <IP_ADDRESS> [AS24940]  234.141 ms  235.204 ms  235.846 ms
 6  <IP_ADDRESS> [AS24940]  235.206 ms  211.323 ms  210.436 ms
 7  <IP_ADDRESS> [AS24940]  209.800 ms  210.381 ms  209.806 ms
 8  <IP_ADDRESS> [AS15169]  296.548 ms  296.543 ms  296.527 ms
 9  * * *
10  <IP_ADDRESS> [AS15169]  299.174 ms  298.391 ms  290.207 ms
11  <IP_ADDRESS> [AS15169]  290.171 ms  290.168 ms  291.686 ms
12  <IP_ADDRESS> [AS15169]  282.384 ms  281.110 ms  280.447 ms
13  <IP_ADDRESS> [AS15169]  293.000 ms  293.691 ms  293.679 ms
14  <IP_ADDRESS> [AS15169]  303.117 ms  313.463 ms  314.110 ms
15  <IP_ADDRESS> [AS15169]  375.580 ms  375.537 ms  375.580 ms
16  <IP_ADDRESS> [AS15169]  394.030 ms  371.614 ms  377.574 ms
17  <IP_ADDRESS> [AS15169]  391.349 ms  380.651 ms  386.538 ms
18  <IP_ADDRESS> [AS15169]  419.132 ms  420.290 ms  420.783 ms
19  <IP_ADDRESS> [AS15169]  420.196 ms  415.617 ms  418.509 ms
20  <IP_ADDRESS> [AS15169]  428.967 ms  423.064 ms  407.262 ms
21  * * *
22  * * *
23  * * *
24  * * *
25  * * *
26  * * *
27  * * *
28  * * *
29  <IP_ADDRESS> [AS396982/AS15169]  436.369 ms  450.152 ms  428.301 ms

# traceroute -n ooni.org --udp -m50 -O info --port=443 -A
traceroute to ooni.org (<IP_ADDRESS>), 50 hops max, 60 byte packets
 1  <IP_ADDRESS> [*]  196.894 ms  210.786 ms  210.780 ms
 2  <IP_ADDRESS> [*]  254.024 ms  255.827 ms  253.965 ms
 3  <IP_ADDRESS> [AS24940]  239.107 ms  239.076 ms  239.014 ms
 4  * * *
 5  <IP_ADDRESS> [AS24940]  264.865 ms <IP_ADDRESS> [AS24940]  240.069 ms  254.244 ms
 6  <IP_ADDRESS> [AS24940]  254.224 ms <IP_ADDRESS> [AS24940]  212.183 ms  246.152 ms
 7  <IP_ADDRESS> [AS24940]  246.123 ms <IP_ADDRESS> [AS24940]  246.100 ms <IP_ADDRESS> [AS24940]  246.053 ms
 8  <IP_ADDRESS> [AS15169]  246.058 ms  248.892 ms <IP_ADDRESS> [AS15169]  316.783 ms
 9  <IP_ADDRESS> [AS15169]  248.839 ms <IP_ADDRESS> [AS15169]  249.109 ms <IP_ADDRESS> [AS15169]  248.730 ms
10  <IP_ADDRESS> [AS15169]  260.839 ms <IP_ADDRESS> [AS15169]  280.100 ms <IP_ADDRESS> [AS15169]  211.933 ms
11  <IP_ADDRESS> [AS15169]  212.686 ms <IP_ADDRESS> [AS15169]  290.697 ms <IP_ADDRESS> [AS15169]  211.847 ms
12  <IP_ADDRESS> [AS15169]  208.306 ms * <IP_ADDRESS> [AS15169]  267.057 ms
13  <IP_ADDRESS> [AS15169]  219.506 ms *  310.886 ms
14  <IP_ADDRESS> [AS15169]  300.270 ms <IP_ADDRESS> [AS15169]  300.234 ms  301.500 ms
15  <IP_ADDRESS> [AS15169]  310.781 ms <IP_ADDRESS> [AS15169]  368.404 ms <IP_ADDRESS> [AS15169]  382.030 ms
16  <IP_ADDRESS> [AS15169]  371.146 ms * *
17  <IP_ADDRESS> [AS15169]  359.696 ms  341.801 ms *
18  <IP_ADDRESS> [AS15169]  353.315 ms <IP_ADDRESS> [AS15169]  432.218 ms <IP_ADDRESS> [AS15169]  351.941 ms
19  <IP_ADDRESS> [AS15169]  351.295 ms <IP_ADDRESS> [AS15169]  361.653 ms  362.233 ms
20  * * *
21  * * *
22  * * *
23  * * *
24  * * *
25  * * *
26  * * *
27  * * *
28  * * *
29  <IP_ADDRESS> [AS396982/AS15169]  342.698 ms  358.024 ms  358.215 ms

And this ban is bidirectional. This means that in some Google servers, you can not communicate with any Iranian IP. Like Google Cloud Shell:
$ traceroute --tcp --port=443 -O info -A -n iran.ir
traceroute to iran.ir (<IP_ADDRESS>), 30 hops max, 60 byte packets
 1  <IP_ADDRESS> [*]  0.059 ms  0.014 ms  0.012 ms
 2  * * *
 3  * * *
 4  * * *
 5  * * *
 6  * * *
 7  * * *
 8  * * *
 9  * * *
10  * * *
11  * * *
12  * * *
13  * * *
14  * * *
15  * * *
16  * * *
17  * * *
18  * * *
19  * * *
20  * * *
21  * * *
22  * * *
23  * * *
24  * * *
25  * * *
26  * * *
27  * * *
28  * * *
29  * * *
30  * * *

$ ping -c9 iran.ir
PING iran.ir (<IP_ADDRESS>) 56(84) bytes of data.

--- iran.ir ping statistics ---
9 packets transmitted, 0 received, 100% packet loss, time 232ms

In all of the above tests, censorship has not been imposed by the user government. However, in some cases, the user network may also block some of these addresses, which are blocked due to sanctions. The above tests are performed in a way that does not involve this censorship of the user government.

In such a situation, it is very difficult to distinguish between these two types of Internet censorship. As we at WikiCensorship strive to increase transparency, we will Investigate the various methods of censorship due to sanctions here, and if we have enough data, knowledge and techniques to distinguish them, we will publish it in a new post on Wikicensorship.

Another example of 404 error due to sanctions:
# curl -I https://dl.google.com/go/go1.17.4.linux-amd64.tar.gz

HTTP/2 404
content-length: 1449
content-type: text/html; charset=utf-8
server: downloads
x-content-type-options: nosniff
x-frame-options: SAMEORIGIN
x-xss-protection: 0
date: Thu, 09 Dec 2021 08:26:28 GMT
alt-svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43"

curl -I -x socks5://<IP_ADDRESS>:9150/ https://dl.google.com/go/go1.17.4.linux-amd64.tar.gz

HTTP/2 200
accept-ranges: bytes
cache-control: public,max-age=86400
content-disposition: attachment
content-length: 134807371
content-security-policy: default-src 'none'
content-type: application/x-gzip
etag: "b1d61f"
last-modified: Thu, 02 Dec 2021 23:31:19 GMT
server: downloads
x-content-type-options: nosniff
x-frame-options: SAMEORIGIN
x-xss-protection: 0
date: Thu, 09 Dec 2021 08:29:12 GMT
alt-svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43"

This ban was imposed by the Chinese government, but it is interesting to note:
HTTP 451 : Unavailable For Legal Reasons

> ./curl -v https://www.tiktok.com/
*   Trying <IP_ADDRESS>:443...
* Connected to www.tiktok.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
*  CAfile: C:\shared\curl-7.82.0-win64-mingw\bin\curl-ca-bundle.crt
*  CApath: none
* TLSv1.0 (OUT), TLS header, Certificate Status (22):
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.2 (IN), TLS header, Certificate Status (22):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.2 (IN), TLS header, Finished (20):
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Certificate (11):
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Finished (20):
* TLSv1.2 (OUT), TLS header, Finished (20):
* TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* TLSv1.3 (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / TLS_AES_256_GCM_SHA384
* ALPN, server accepted to use h2
* Server certificate:
*  subject: CN=*.tiktok.com
*  start date: Dec 13 00:00:00 2021 GMT
*  expire date: Jan 13 23:59:59 2023 GMT
*  subjectAltName: host "www.tiktok.com" matched cert\'s "*.tiktok.com"
*  issuer: C=US; O=DigiCert Inc; CN=RapidSSL TLS DV RSA Mixed SHA256 2020 CA-1
*  SSL certificate verify ok.
* Using HTTP2, server supports multiplexing
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* h2h3 [:method: GET]
* h2h3 [:path: /]
* h2h3 [:scheme: https]
* h2h3 [:authority: www.tiktok.com]
* h2h3 [user-agent: curl/7.82.0]
* h2h3 [accept: */*]
* Using Stream ID: 1 (easy handle 0x2491f36f870)
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
> GET / HTTP/2
> Host: www.tiktok.com
> user-agent: curl/7.82.0
> accept: */*
>
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* old SSL session ID is stale, removing
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
< HTTP/2 451
< content-type: text/html
< content-length: 29
< expires: Mon, 05 Sep 2022 09:19:51 GMT
< cache-control: max-age=0, no-cache, no-store
< pragma: no-cache
< date: Mon, 05 Sep 2022 09:19:51 GMT
< x-cache: TCP_MISS from a96-17-105-219.deploy.akamaitechnologies.com (AkamaiGHost/10.9.3-44045506) (-)
< x-tt-trace-tag: id=16;cdn-cache=miss;type=dyn
< server-timing: cdn-cache; desc=MISS, edge; dur=0, origin; dur=0
< x-akamai-request-id: 16af8fa0
< set-cookie: _abck=[SNIP]~-1~-1~-1; Domain=.tiktok.com; Path=/; Expires=Tue, 05 Sep 2023 09:19:51 GMT; Max-Age=31536000; Secure
< set-cookie: bm_sz=[SNIP]~4277317~4339511; Domain=.tiktok.com; Path=/; Expires=Mon, 05 Sep 2022 13:19:51 GMT; Max-Age=14400
<
Unavailable For Legal Reasons* Connection #0 to host www.tiktok.com left intact


hwmv2: Build with ARC MWDT became broken after HMWv2 was implemented.

Describe the bug
Build with ARC MWDT became broken after HWMv2 was implemented.
ARC MWDT toolchain doesn't contain it's own DTC-preprocessor. It uses preprocessor, which is a part of Zephyr-SDK.
In current implementailon ${ZEPHYR_SDK_INSTALL_DIR}/cmake/zephyr/target.cmake file should be included on very early
build stages to obtain the preprocessor via CROSS_COMPILE variable (it defined inside pointed .cmake file).
Modules call sequence:
zephyr_default->dtc->FindHostTools->toolchain/arcmwdt/generic.cmake->${ZEPHYR_SDK_INSTALL_DIR}/cmake/zephyr/target.cmake
The point is target.cmake requires ARCH variable to be defined. New HWMv2 defines ARCH variable much more later in arch_v2 module via CONFIG_ARCH.
In case of MWDT such construction as set(ARCH ${CONFIG_ARCH}) is not applicable as Kconifg module was not yes initialized. And it at the same time requires dtc, and we get circullar dependence.
So, here I'd like to ask for a advice how to fix the issue in the correct way.
To Reproduce
1. export ZEPHYR_TOOLCHAIN_VARIANT=arcmwdt
2. west build -b nsim/nsim_em samples/hello_world/
3.
-- Board: nsim, identifier: nsim_em
-- ARCMWDT_TOOLCHAIN_PATH is not set, use default toolchain from METAWARE_ROOT: '/home/user/ARC_202312/MetaWare'
-- Found host-tools: zephyr 0.16.3 (/home/user/sandbox/zephyr-sdk-0.16.3)
-- Found dts preprocessor: CMAKE_DTS_PREPROCESSOR-NOTFOUND (Zephyr SDK 0.16.3)

Expected behavior
Build not fails:
...
-- Found host-tools: zephyr 0.16.3 (/home/user/sandbox/zephyr-sdk-0.16.3)

CMake Debug Log at /home/user/sandbox/zephyr-sdk-0.16.3/cmake/zephyr/target.cmake:32 (if):
  Variable "ARCH" was accessed using READ_ACCESS with value "arc".
Call Stack (most recent call first):
  /home/user/sandbox/zephyrproject/zephyr/cmake/toolchain/arcmwdt/generic.cmake:26 (include)
  /home/user/sandbox/zephyrproject/zephyr/cmake/modules/FindHostTools.cmake:103 (include)
  /home/user/sandbox/zephyrproject/zephyr/cmake/modules/dts.cmake:9 (find_package)
  /home/user/sandbox/zephyrproject/zephyr/cmake/modules/zephyr_default.cmake:129 (include)
  /home/user/sandbox/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:66 (include)
  /home/user/sandbox/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:97 (include_boilerplate)
  CMakeLists.txt:5 (find_package)

-- Found dts preprocessor: /home/user/sandbox/zephyr-sdk-0.16.3/arc-zephyr-elf/bin/arc-zephyr-elf-gcc (Zephyr SDK 0.16.3)
...

Impact
Showstopper
Logs and console output
-- Application: /home/user/sandbox/zephyrproject/zephyr/samples/hello_world
-- CMake version: 3.25.1
-- Found Python3: /home/user/sandbox/zephyrproject/.venv/bin/python3 (found suitable version "3.11.2", minimum required is "3.8") found components: Interpreter
-- Cache files will be written to: /home/user/.cache/zephyr
-- Zephyr version: 3.6.99 (/home/user/sandbox/zephyrproject/zephyr)
-- Found west (found suitable version "1.1.0", minimum required is "0.14.0")
-- Board: nsim, identifier: nsim_em
-- ARCMWDT_TOOLCHAIN_PATH is not set, use default toolchain from METAWARE_ROOT: '/home/user/ARC_202312/MetaWare'
-- Found host-tools: zephyr 0.16.3 (/home/user/sandbox/zephyr-sdk-0.16.3)
CMake Debug Log at /home/user/sandbox/zephyr-sdk-0.16.3/cmake/zephyr/target.cmake:32 (if):
  Variable "ARCH" was accessed using UNKNOWN_READ_ACCESS with value "".
Call Stack (most recent call first):
  /home/user/sandbox/zephyrproject/zephyr/cmake/toolchain/arcmwdt/generic.cmake:26 (include)
  /home/user/sandbox/zephyrproject/zephyr/cmake/modules/FindHostTools.cmake:104 (include)
  /home/user/sandbox/zephyrproject/zephyr/cmake/modules/dts.cmake:9 (find_package)
  /home/user/sandbox/zephyrproject/zephyr/cmake/modules/zephyr_default.cmake:132 (include)
  /home/user/sandbox/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:66 (include)
  /home/user/sandbox/zephyrproject/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
  CMakeLists.txt:5 (find_package)

-- Found dts preprocessor: CMAKE_DTS_PREPROCESSOR-NOTFOUND (Zephyr SDK 0.16.3)
-- Found Dtc: /home/user/sandbox/zephyr-sdk-0.16.3/sysroots/x86_64-pokysdk-linux/usr/bin/dtc (found suitable version "1.6.0", minimum required is "1.4.6")
-- Found BOARD.dts: /home/user/sandbox/zephyrproject/zephyr/boards/synopsys/nsim/nsim_nsim_em.dts
CMake Error at /home/user/sandbox/zephyrproject/zephyr/cmake/modules/extensions.cmake:4205 (message):
  failed to preprocess devicetree files (error code No such file or
  directory)

Environment (please complete the following information):

OS: Linux
Toolchain: arcmwdt
Commit SHA or Version used: 1a55caf8263e11bc22f93b88e2ccf99eb30f690f

Additional context

@tejlmand or @nordicjm , @57300  can you please take a look at this?

@kokas-a After https://github.com/zephyrproject-rtos/zephyr/pull/71163 was merged, can this be closed?

Fixed in https://github.com/zephyrproject-rtos/zephyr/pull/71163


Token replacement should not be hard coded

IMHO, the hard coded string in
[Route("[controller]/[action]")]
is not type safe or error prone.
[Route("[controller]/[action]")]
public class ProductsController : Controller
{
    [HttpGet] // Matches '/Products/List'
    public IActionResult List() {
        // ...
    }

    [HttpGet("{id}")] // Matches '/Products/Edit/{id}'
    public IActionResult Edit(int id) {
        // ...
    }
}

There should be any better way to avoid this.

cc @JamesNK

Hi @sadqiang
.NET attributes allow a very small subset of primitive types as arguments. That limits the options for creating a strongly typed API at this point. If you don't like having that route pattern in every controller you have a couple of options:

Use conventional routing instead of attribute routing. That centralizes routing rules. Example: https://docs.microsoft.com/en-us/aspnet/core/mvc/controllers/routing?view=aspnetcore-2.2#setting-up-routing-middleware
Have a const string and use it with RouteAttribute. That would reduce the chance of a typo in a controller file. Example: [Route(Constants.ControllerBaseRoute)]


Cannot load texture imported as TextureAtlas

Godot version:
Godot Engine v3.2.alpha2.official
OS/device including version:
Windows 10 (1709)
Issue description:
It exists in filesystem, but can't be used
After opening the project:
ERROR: get_multiple_md5: Condition ' !f ' is true. Continuing..:
   At: core/os/file_access.cpp:667
ERROR: Cannot open file 'res://.import/icon.png-487276ed1e3a0c39cad0279d744ee560.res'.
   At: core/io/resource_format_binary.cpp:1035

When trying to select (double click) it:
ERROR: Cannot open file 'res://.import/icon.png-487276ed1e3a0c39cad0279d744ee560.res'.
   At: core/io/resource_format_binary.cpp:986
ERROR: Failed loading resource: res://.import/icon.png-487276ed1e3a0c39cad0279d744ee560.res.
   At: core/io/resource_loader.cpp:278
ERROR: Failed loading resource: res://icon.png.
   At: core/io/resource_loader.cpp:278
ERROR: load_resource: Condition ' !res.is_valid() ' is true. returned: ERR_CANT_OPEN
   At: editor/editor_node.cpp:656

Steps to reproduce:
Image from https://godotengine.org/article/atlas-support-returns-godot-3-2

Minimal reproduction project:
BrokenAtlas.zip

Ubuntu 18.04 bionic
x86_64 Linux 5.0.0-31-generic

Opening the Minimal reproduction project gives me those errors
If I enter another project and make an atlas with my own textures and Import Mode Mesh it works fine
If I do the same but with Import Mode Region the console spams different errors
...
ERROR: get_pixel: Index p_x=200 out of size (width=200)
   At: core/image.cpp:2389.
ERROR: get_pixel: Index p_y=200 out of size (height=200)
   At: core/image.cpp:2390.
... // hundreds of these

I have the same problem texture-atlas region. (I'm using ubuntu). the references to the new file are created but the file itself is not. this mostly happens if there are nodes already using the image, and usually works if nothing is using the existing image. A work-around is to create the file in gimp then open the single images can then adjust the region used.
I've tried this in 3.2.2 and opened the same project with 3.2.3 RC6 with the same results.


Stressng

Description
Adding stressng label according to new label convention for VM and Pod
Fixes

The following labels were added for VM and Pod
benchmark-operator-uuid: {{ uuid }}
benchmark-operator-workload: uperf
benchmark-operator-role: server

Change pin-client to meaningful name: pin-node.

Can one of the admins verify this patch?

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_stressng
Fail
3
00:39:15

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_stressng
Pass
0
00:02:27

LGTM!


[master] Upgrade to latest dependencies

Cron. -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced via:
./hack/update-deps.sh --upgrade --release v0.21
./hack/update-codegen.sh

Details:
bumping knative.dev/reconciler-test fa758f8...28c3852:
  > 28c3852 DSL to perform assertions on eventshub store (# 62)
bumping knative.dev/pkg 6040b3a...0d31134:
  > 0d31134 Fix nil pointer panic in kvstore (# 2002)

/lgtm
/approve

/lgtm
/approve


layout of Candrakirana

Something is going wrong with the layout of the Candrakirana in html. See screenshot below: the sreen is much too wide, so one needs to set a very small letter size to be able to to see left and right ends of the screen. This is not happening in the other html displays. Do you understand the cause, Axelle?

even two days later, the layout is still not as expected. see screenshot. by the way, the display of the colophon isn't as we expect it either. is it the same problem, or is it a different one?

Finally, I can join the discussion. Something wrong also happened in section number 12 about Chanda. It seems that paragraphs consisting of lists of meters cannot be placed between two metrical texts.

The display is as expected with the current encoding.

Please explain more clearly what you mean, @ajaniak :
do you mean the encoding is wrong?
If not, it must be the display that is wrong, because we don't have a rule (as far as I know) that any prose passages will be displayed only in the space between the width of verse passages and the corresponding apparatus entries.
If such a rule is in place, it should be removed. The display of verse vs. prose should be as it is in other editions, e.g. Sarasamuccaya:

It seems the problem is solved now. Great! Just one small infelicity remains for me at this time, concerning display of <ab type="colophon">. See screenshot:

It would be nicer if the displayed heaidng "colophon" were separated by some vertical space from the preceding <lg> and if both the heading and the actual colophon stood at the same width as 10.2 and Sūtrasandhi.

Like this?

The new view of colophon seems very nice to me @ajaniak.

yes, that looks nice! thanks. I am closing the issue now.


Retry inserts in integration tests.

Attempt to fix #1008

The context @vjeffrey mentioned above:

I think the problems with the flaky integration tests are happening because we do a lot of async elastic operations for performance reasons and I think we have real concurrency bugs. Some of the failures we've been seeing lately are because of how the code handles the daily_latest: a new document is inserted and then every other document belonging to that node is updated to have daily_latest set to false, both of these asynchronously. When this is done in a tight loop, like in the tests, there is a lot of opportunity for conflicts.
If we were using a relational database I'd say we should order by a date column rather than using that extra flag, but I don't really know enough about how we're doing that to say whether it's possible here.
I have a PR to add some retries in the test code but I'm not really comfortable modifying the actual service code for this because I'm not sure how I'd know if I was introducing a performance regression.

Running these tests in a loop on my local machine I am still seeing failures, but they are all failing in TestReportingListSuggestions which is most likely related to these ES issues but would be a totally separate debugging session.

ya, the flaky reporting list suggestions test is documented here: https://github.com/chef/automate/issues/599


Xamarin.Android : Could not load assembly 'CodeGeneration.Roslyn.Attributes" Compile Error

Updating dependancies installed by S3 Nuget causes the following compiler error in Xamarin Studio for Android Applications :
/Library/Frameworks/Mono.framework/External/xbuild/Xamarin/Android/Xamarin.Android.Common.targets: Error: Exception while loading assemblies: System.IO.FileNotFoundException: Could not load assembly 'CodeGeneration.Roslyn.Attributes, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=ec41cf4c17a5eaf3'. Perhaps it doesn't exist in the Mono for Android profile?
File name: 'CodeGeneration.Roslyn.Attributes.dll'
at Xamarin.Android.Tuner.DirectoryAssemblyResolver.Resolve (Mono.Cecil.AssemblyNameReference reference, Mono.Cecil.ReaderParameters parameters) <0xb0f04f8 + 0x00423> in :0
at Xamarin.Android.Tuner.DirectoryAssemblyResolver.Resolve (Mono.Cecil.AssemblyNameReference reference) <0xb381528 + 0x0001f> in :0
at Xamarin.Android.Tasks.ResolveAssemblies.AddAssemblyReferences (ICollection`1 assemblies, Mono.Cecil.AssemblyDefinition assembly, Boolean topLevel) <0xb380bd8 + 0x0046b> in :0
at Xamarin.Android.Tasks.ResolveAssemblies.Execute () <0xb380000 + 0x00577> in :0  (Cura.UserApp.Android)
After trying back and fourth with Xamarin guys, it turned out that the latest update of S3 Nuget installs a dependency (PInvoke and Windows.Core and PLCrypto) and updating those would install this weird dll  :
/packages/PInvoke.Windows.Core.0.2.8/lib/portable-net45+win+wpa81+MonoAndroid10+xamarinios10+MonoTouch10/CodeGenerationAttributes.dll
Reverting back to old version of S3 with it's dependancies resolved the issue.
Please consider fixing this in your upcoming version update.
Thanks

Which specific version of the AWSSDK.S3 NuGet package is causing this issue? Are you running Xamarin studio on Windows?


Fix desktop integration in Firefox Recipe

To get the dialog asking to install the .desktop file I had to fox the following (like discussed in the firefox-dev PR):
Fix firefox.desktop (add Type=Application), or the .desktop is not parsable
Set AppRun to firefox.wrapper to invoke the correct executable

Thanks


Malformed AML in AddActionToItemTypes.xml

Hello,
there is a minor error and I am too lazy to create a fork...
The error is in AddActionToItemTypes.xml line 10

Thanks for bringing this to our attention. We'll fix it!
Eli

Fixed in version 2.3.1. Closing out this issue.
Eli


Update data-broker/scripts/readme

The readme in the scripts directory has outdated instructions based on Redis < 5.0

Assigning to Bruce since I think he did the latest updates to the scripts. Feel free to reassign as needed if that's incorrect.


Fix workload test do not cleanup

Please provide a description of this PR:

PR needs rebase.

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


Fixed path to patch file

This script has a bad path to the patch file, which causes the script to fail.
Benjamin Hirashima
I agree to the terms of the Intel® RealSense™ Cross Platform API CLA.

I created a pull request with a better approach: #369

Your proposal is appreciated, but I agree that using dirname $0 is a bit more robust

Agreed!


fix: QA categories were being used only by the CLI engine

Updated the QA http rest engine of Move2Kube required for adding QA category enable/disable support to Move2Kube API https://github.com/konveyor/move2kube/issues/1155, https://github.com/konveyor/move2kube/issues/1151#issuecomment-1993816895. For the disabled categories, the QA engine should automatically consider the default answer as the input answer.
Signed-off-by: Akash Nayak<EMAIL_ADDRESS>

Related https://github.com/konveyor/move2kube/issues/1154


gha: Add ingress conformance test

This commit is to add workflow for running Ingress conformance test.
Minikube cluster (with multiple nodes enabled) is used due to the
native support for LB services (with minikube tunnel) without the
complicated steps to install metalLB like kind.
Upstream PR: https://github.com/cilium/cilium/pull/19742
Signed-off-by: Tam Mach<EMAIL_ADDRESS>

The related GHA test is passed here https://github.com/cilium/cilium/runs/7130120318?check_suite_focus=true, full CI test is not required.

amazing test coverage

This test is an invisible friend in 1.12, but sometimes it's hard for me to find her, so I need to make it visible again.

3rd time should be a charm I hope, marking this ready to merge.


[test] Initial workspace definition

First version only has a dedicated task for type-checking utils. Over time we'll consolidate dependencies to make it clearer what utils you should import from where (e.g. fireEvent from @testing-library/react or test/utils?).

e.g. fireEvent from @testing-library/react or test/utils?

We have banned some import paths in material-ui-x to only allow test/utils: https://github.com/mui-org/material-ui-x/blob/f11f26659f873ef86bf06ae556aa23dcea5c8f1f/.eslintrc.js#L42. We had bugs because we were importing from @testing-library/react in some places.

Ultimately a package manager would do this job (e.g. yarn v2).


Need to migrate RASA intents and entities from one server to another server.

I have created the intents and entities which are already created in on RASA server and trained the NLU on that server.
Now I want to migrate these intents and entities from this server to another server and want to modify the user experience text for each intent.
Please help

@neerajkandwal - did you use Rasa UI on the source server as well or just Rasa?


Question - targeting child nodes best practise

If I have, for example, a list:

item 1

item 2

item 3

Normally I would create a class for the
 and then target the items with .list > li.
With this library - would I be better off using globalStyle('${list} > li', {}) or applying a class to each
 ?

My personal preference would be to use a class for the li elements since there's less indirection, makes it easier to see which elements are styled when looking at the markup, and makes it easier to detect unused styles too. That said, it's really up to you.


error

Hello, may I ask if there is an error in the code in tools/gen_ In data.py, there are：
full_seq_x.append(measurement['y'])
full_seq_y.append(measurement['x'])

The correct one should be the following：
full_seq_x.append(measurement['x'])
full_seq_y.append(measurement['y'])

right?

We convert the coordinate here. It is not an error, sorry for the confusion.


component-dep-issues//for_release

SonarQube analysis reported 14 issues

 10 blocker
 2 critical
 1 major
 1 info

Watch the comments in this conversation to review them.
10 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 BlackListWhiteListDAO.java#L127: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListDAO.java#L340: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListDAO.java#L419: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListDAO.java#L478: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListDAO.java#L513: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListDAO.java#L547: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListDAO.java#L582: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListDAO.java#L613: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListDAO.java#L646: Use a variable binding mechanism to construct this query instead of concatenation.
 BlackListWhiteListService.java#L133: Define and throw a dedicated exception instead of using a generic one.


chore: delete useless gitutil types

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
delete useless gitutil types
Which issue(s) this PR fixes:

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

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.2%) to 74.78%

Files with Coverage Reduction
New Missed Lines
%

pkg/cmd/destroy/options.go
7
82.13%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
7146

Relevant Lines:
9556

💛  - Coveralls


Elb swapping

@afred : please review. The ec2_elb_starter script is still a work in progress, but there's enough here to look at. Thanks.

@afred, please review.
ruby scripts/ec2_elb_start.rb abc.wgbh-mla-test.org
ruby scripts/elb_swap.rb abc.wgbh-mla-test.org

Does it work for you?
Does it look ok?
What next?

@mccalluc went ahead and merged, but I'm still wondering about the desirability to block an ELB swap if DNS has not propagated.


Add JDBC instrumentation

Other than add JDBC instrumentation, may be can intercept Spring Data Repositories and Templates.

You can already use the Spring Data Repositories together with Sleuth's annotations.

But cannot add annotations on methods like save(), findOne()

Why? You can inherit a method in your interface and annotate it

Yes, but a little ugly.

I have created a library that is proxying your DataSource with one of the jdbc instrumentation libraries such as p6spy.
It as well integrates with sleuth and registers listener that is very similar to the one from brave-instrumentation-p6spy, but mine is also creating span for each connection taken from the DataSource and query span includes time to close ResultSet if query returns it.
In zipkin it looks great:

You can find more screenshots and try it on library page.

That looks awesome @gavlyukovskiy . Let's do it like this. Everyone who is interested in this feature please try the https://github.com/gavlyukovskiy/spring-boot-data-source-decorator library. Once we get more parties involved and happy with the library we'll think of putting it in the core of Sleuth.

Closing this in favour of Brave's p6spy instrumentaiton - https://github.com/openzipkin/brave/tree/master/instrumentation/p6spy#brave-instrumentation-p6spy


chore(package): update babel-jest to version 23.0.1

Closes #12

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BUG] Units for fake regions are incorrect

Bug Description
In Stepmania, fake regions are defined in units of beats.  GrooveAuthor, however, interprets the value of a fake region in units of seconds, and will incorrectly render fake regions as a result.
Expected Behavior
GrooveAuthor uses units of beats for fake regions.
Screenshots
In this example, I have a 2-beat fake region.  In Stepmania, only the mines are fakes, the tap notes are real.  But in GrooveAuthor, the fake region incorrectly extends to cover several notes.

Version
<IP_ADDRESS>

Thank you for finding and reporting this! It should be fixed in https://github.com/PerryAsleep/GrooveAuthor/commit/05c2b5c7386240eafae2a4e877a42038dafe1d1b. I will make a new release with this bug fix sometime soon.

The fix for this is available in v0.2.2.


High cpu usage

Is that the problem?
System Specs:
Windows 10.
Win32 Application
VS2017
Ram 8G.
CPU: Core i5-3330 3.00GHz (4CPUs)

IIRC, examples do not throttle or do vsync so they just render as fast as they can.

All examples should vsync, which corresponds to that graph on the right. Otherwise usage would be higher.

With BGFX_RESET_NONE result:

With BGFX_RESET_VSYNC result:

I'm sorry. It may be a problem due to the operating system.
I will investigate further

WaitForInputIdle should prevent spinning inside while loop, but didn't look at that on Win10:
https://github.com/bkaradzic/bgfx/blob/master/examples/common/entry/entry_windows.cpp#L513

I used the sample code of Microsoft and added some commands:
bgfx::setPlatformData
bgfx::init
bgfx::reset
...
And the result was unexpected
Bgfx run with ~0% cpu

Can we re-open this issue ?
It is still happening, and the reason is that the examples update() is being called in a tight loop and never sleeps.
https://github.com/bkaradzic/bgfx/blob/master/examples/common/entry/entry.cpp#L523

Perhaps it'd be nice to show the name of the GPU used in the examples

Press F1.

Press F1.

Oh, I didn't know one could do this ! An imgui button to trigger this display might be nice to have.

Once I finish example restart you'll be able to switch renderer in runtime: https://github.com/bkaradzic/bgfx/blob/15feda2ff5106c4f2145098164104b0c8cdda665/examples/common/example-glue.cpp#L177


Release with jhipster-lite-sample-app

Like other sample apps, it could be nice to have this
We need to think which modules should be included:

PostgreSQL
Liquibase
all dummies
between JWT and OAuth2, I don't know yet

Maybe it's closed to the full app https://github.com/jhipster/jhipster-lite/blob/main/tests-ci/generate.sh#L105

The work is done.
The sample app will be available at https://github.com/jhipster/jhipster-lite-sample-app after the next release v0.30.0 :-)
cc @renanfranca as you wanted to have this one
So closing this

The work is done.
The sample app will be available at https://github.com/jhipster/jhipster-lite-sample-app after the next release v0.30.0 :-)
cc @renanfranca as you wanted to have this one
So closing this

I remember when I asked if this up-to-date sample exists! That's really cool because anyone can try to use a project generated by jhlite without the need to learn how to use the generator! If someone enjoys the sample, they will be motivated to dig into the project 👏👏👏👏


through-model 'saving' listener not firing on .attach()

I have models A, B, and C, where C is a through-model between A and B:
var A = bookshelf.model.extend({
  table: 'as',
  ...
  bRelation: function() {
    return this.belongsToMany('B').through('C');
  }
  ...
});

var B = bookshelf.model.extend({
  table: 'bs',
  ...
  aRelation: function() {
    return this.hasMany('A').through('C');
  }
  ...
});

var C = bookshelf.model.extend({
  table: 'as_joined_to_bs',
  ...
  initialize: function() {
    this.on('saving', this.doSomething);
  },
  aRelation: function() {
    return this.belongsTo('A');
  },
  bRelation: function() {
    return this.belongsTo('B');
  },
  doSomething: function() {
    console.log('did something');
  }
  ...
});

When I try to insert into the as_joined_to_bs join table via:
instanceOfA.bRelation().attach([ instanceOfB ], { transacting: t })

the insert succeeds, but doSomething never gets called. From what I can tell this is because Bookshelf's pivotHelpers._processPivot() inserts into the join table directly with a Knex query, rather than by trying to save an instance of through-model C. Is that correct? Is there any existing Bookshelf-ic way to make doSomething get called on .attach() (before the row has been inserted)?
Thanks!

This is a bug. Join tables should be handled different when there's a through model.

+1 - I've been looking around the open issues but haven't seen much movement on this topic, are there plans to address this in an upcoming release?

This should be something that someone might be able to do a PR for. Otherwise it goes on a very long backlog of issues.

Got it, thank you for the update!

@rpnzl, thanks for taking on the PR. @bendrucker, what are the next steps for reviewing this and hopefully merging it before the next release?

Awesome! Thanks @rpnzl and @bendrucker!


@conda-forge-admin, please add bot automerge

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/update_checker-feedstock#7.

Lgtm!
On Fri, Jun 12, 2020, 15:18 Automatic conda-forge linter <
<EMAIL_ADDRESS>wrote:

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in #7
https://github.com/conda-forge/update_checker-feedstock/pull/7.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/update_checker-feedstock/issues/6#issuecomment-643443158,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALCRDT6MQDOCS5MCRHG33RWJ5RVANCNFSM4N4SUA6A
.


Can't search item with higher stat than the item i owned

For example, when i crt+D on my Lion's Roar and checked at "21%more melee physical ...", i can see the price in trade. I can search for "15-30% more"  but i can't search for "22-30% more" or "15-18% more.
SR for my bad English.

This is an intended behaviour. The min value will never be greater than the actual value and the max value will never be less than the actual value. This was built in to keep the scrolling mechanism simple. But there is a "Enable input of numeric values with keyboard" #152 planned which will not have this saftey mechanism.
I'll close this issue. But feel free to reopen if this is not what you want/expect.

I'd love for it to be possible with scrolling too, more handy than typing a number for me.
Maybe only allow it to go above the current value if holding CTRL while scrolling, for example?
Or adding it as a setting that's off by default.
I don't use max range, and it's possible to go back to the original value by pressing the scroll wheel, so it would still be simple without the limit, for my use case anyway.

I'd love for it to be possible with scrolling too, more handy than typing a number for me.
Maybe only allow it to go above the current value if holding CTRL while scrolling, for example?
Or adding it as a setting that's off by default.
I don't use max range, and it's possible to go back to the original value by pressing the scroll wheel, so it would still be simple without the limit, for my use case anyway.


Rename the train method to fit to avoid confusion with PyTorch's built-in train method.

The train method in the KAN class is used to define the main training logic for the model. However, in PyTorch, the name "train" is typically used for switching the model between training and evaluation states, such as model.train(). Also, in this implementation of code, if ther user calls the model.train() explicitly, it would raise error. To avoid confusion and improve code clarity, this PR renames the train method to fit.
The fit method better conveys the purpose of the function, which is to train the model using the provided dataset and hyperparameters. This change ensures that the method name does not clash with PyTorch's built-in functionality and makes it clear that it is a custom model training function.
The renaming is a minor change and does not affect the functionality of the method. All occurrences of train within the method have been replaced with fit, and any calls to the method in other parts of the codebase have been updated accordingly.
This change improves the readability and maintainability of the codebase by following a more consistent and intuitive naming convention.

tried loading the device file on colab didn't run

Not sure about this, but you can patch this function as a temporary solution like(this function should be called before you use the kan lib):
from kan import KAN
def patch_kan_train_function():
    def patch_train(*args, **kwargs):
        pass

    KAN.train = patch_train

and then use it as model.fit moving on right?

and then use it as model.fit moving on right?

In my specific use case, the KAN model is integrated as a component within my larger neural network model. I don't directly utilize the train method provided by the KAN class. Instead, I opt for a more customized approach, explicitly managing the training process using PyTorch's loss.backward() and optimizer.step() functions.
class MyNet(nn.Module):
    def __init__(self):
        ...
        self.output_module = KAN(...)  # Integrate KAN as a component

# Training loop (custom, explicit control)
for train_batch in train_dataloader:
    ...
    loss.backward()  # Compute gradients
    optim.step()     # Update model parameters
    ...

If you prefer to use the standard Keras model.fit() API, you would typically rename your custom train function to fit. However, this isn't necessary in my current approach as I'm directly controlling the training loop.

This is me trying to run this code on Google Colab with their device example

Their Code Example:
https://github.com/KindXiaoming/pykan/blob/master/tutorials/API_10_device.ipynb

@KindXiaoming  Could you please review this? I think this PR is somehow important for those who want to integrate the KAN into their modules.

Hi @linkedlist771 , thank you for your message. As I explained in an issue (forgot which), tutorials use train() all the time, so it would be too confusing to switch to another API. Also, one can say this is even deliberate to be designed as such, since you need to manually update grid, not just plug-and-play (will improve the error message though). I do understand that users want plug-and-play, I think any KAN variants that do not require grid update can be safely and directly plug-and-play.

Could u also check the gpu issue @KindXiaoming

hi @ChrisD-7, can you pull the lastest version and see if gpu issue still persists. Apple GPU (MPS) seems not solved yet, but I don't see more issues regarding cuda.

Hi @linkedlist771 , thank you for your message. As I explained in an issue (forgot which), tutorials use train() all the time, so it would be too confusing to switch to another API. Also, one can say this is even deliberate to be designed as such, since you need to manually update grid, not just plug-and-play (will improve the error message though). I do understand that users want plug-and-play, I think any KAN variants that do not require grid update can be safely and directly plug-and-play.

This concern is valid, but for plug-and-play scenarios, I believe a warning would be sufficient instead of raising an exception. It might complicate the train function if we check the invoking function to determine the user's intent (whether to switch to the model's training mode or simply train the model). However, if necessary, I can create a slightly more complex train function to handle this issue.

@KindXiaoming tried pulling it again is it something with the colab env issue? I'm trying to run it for a classification model and faced a RAM issue thought of loading the GPU and faced this issue.

Closed as completed.


Not able to fetch OTC timeseries quotes

Hi,
I recently noticed that an ValueError is raised when fetching OTC tickers. The exception is raised by the _handle_api_call and the message is:
'Error getting data from the api, no return was given.'
Ticker examples: "AYSI", "SCGLY"
This used to work fine. Does anyone know what may have changed recently?
Thanks!

I concur as well, I don't have the problem with GOOG, but have it with GBTC or ETHE which are both OTC.
It has been happening since October 19, 2019.
I think this is the second time it happened, it happened for a couple of days in august 2018 if I am not mistaken.
It is obviously an alpha vantage issue and not an issue of this library, but maybe the error returned by this library could be improved.
I will contact Alpha Vantage right away and forward if I have more info.
ts = TimeSeries(key='MY_KEY', output_format='pandas')
gbtc, meta_gbtc = ts.get_intraday(symbol=ticker,
                                  interval='1min',
                                  outputsize='full')

Traceback (most recent call last):
  File "./av_intra_histo.py", line 40, in <module>
    outputsize='full')
  File "/usr/local/lib/python3.5/dist-packages/alpha_vantage/alphavantage.py", line 164, in _format_wrapper
    data = call_response[data_key]
KeyError: 'Time Series (5min)'

Hey Emmanuel,
Thank you for the info and contacting AV! It does seem to be a AV issue because I was calling a different endpoint than ts.get_intraday and got the same issue with OTC tickers.
Thanks!

I concur as well, I don't have the problem with GOOG, but have it with GBTC or ETHE which are both OTC.
It has been happening since October 19, 2019.
I think this is the second time it happened, it happened for a couple of days in august 2018 if I am not mistaken.
It is obviously an alpha vantage issue and not an issue of this library, but maybe the error returned by this library could be improved.
I will contact Alpha Vantage right away and forward if I have more info.

https://www.alphavantage.co/query?function=TIME_SERIES_INTRADAY&symbol=GBTC&interval=1min&apikey=MY_KEY
Returns {}

ts = TimeSeries(key='MY_KEY', output_format='pandas')
gbtc, meta_gbtc = ts.get_intraday(symbol=ticker,
                                  interval='1min',
                                  outputsize='full')

Traceback (most recent call last):
  File "./av_intra_histo.py", line 40, in <module>
    outputsize='full')
  File "/usr/local/lib/python3.5/dist-packages/alpha_vantage/alphavantage.py", line 164, in _format_wrapper
    data = call_response[data_key]
KeyError: 'Time Series (5min)'

If you upgrade to the newest version of the alpha_vantage package, the error returned is:
ValueError: Error getting data from the API, no return was given.
The OTC data has changed it's formatting as of Q3 2019

As of Q3, 2019, this data is available monthly and historically in the Metrics.
Check this out

We (AV) are working on a fix, thanks guys.

Thanks Patrick, looks like your support team didn't get the memo or chose a different answer, this is what I got from them:

Hi Emmanuel,
We do not support the OTC markets.
Thanks
Team Alpha Vantage

@ecourreges-orange - technically that is that the "more correct" answer. But we want to be able to support them as well.

Has this ever happened. Does AV support OTC tickers?

Has this ever happened. Does AV support OTC tickers?

@ecourreges-orange - technically that is that the "more correct" answer. But we want to be able to support them as well.
And are working on a fix

Has this ever happened. Does AV support OTC tickers?

@ecourreges-orange - technically that is that the "more correct" answer. But we want to be able to support them as well.
And are working on a fix

Has this ever happened. Does AV support OTC tickers?

@ecourreges-orange - technically that is that the "more correct" answer. But we want to be able to support them as well.
And are working on a fix

Has this ever happened. Does AV support OTC tickers?

I can't tell you anymore, I switched to yfinance which does realtime but with 15 minute delay, good enough for my needs.
@PatrickAlphaC can you tell us if OTC markets are now working? Thanks.

@ecourreges-orange - technically that is that the "more correct" answer. But we want to be able to support them as well.
And are working on a fix

Has this ever happened. Does AV support OTC tickers?

I can't tell you anymore, I switched to yfinance which does realtime but with 15 minute delay, good enough for my needs.
@PatrickAlphaC can you tell us if OTC markets are now working? Thanks.


Legend Placement

Closing this since I haven't gotten any response. Let me know if you'd like to revisit


[Feature Request] Integration to MLOPS System

Is your feature request related to a problem? Please describe.
Sometimes we may need to record the run in Promptflow for later evaluation using other MLops systems (e.g. MLFlow/ClearML). But I found it was quite difficult to integrate. E.g. In ClearML, a global experiment object need to be maintained during the experiment. Therefore, adoption to ClearML need to pass the experiment object to each node
Describe the solution you'd like
Integration supports to other MLops system. Or new MLops for promptflow?
Describe alternatives you've considered
I finally integrated promptflow to ClearML, but many tricks need to be applied.
Additional context
Add any other context or screenshots about the feature request here.

Hi @sparkdemotivator, "It seems like your main concern currently is the need to individually pass the experiment object to each node using flow input and output linking, which you think is a cumbersome process. It appears you might prefer a global variable setting, allowing downstream nodes to access this object without manually passing it across nodes. Is that correct?

Yes. It was the pain point of integrating to my MLops system (ClearML). If you have better ideas to integrate with other opensource MLops system without using much effort in tackling those cumbersome process, it is also welcome to suggest.


feature request: support pods in systemd generate --new

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind feature
Description
podman generate systemd --new pod currently fails with:
Error: error generating systemd unit files: cannot generate generic files for a pod

Steps to reproduce the issue:

Run podman generate systemd --new pod on a pod

After talking to @mheon on IRC he suggested to open a feature request since it should be technically possible to create such a systemd file.

@vrothberg hasn't some of this been implemented?

It has been for containers but not for pods. I will tackle this after my PTO. The v2 API (and the bugs) had a higher priority in past weeks.

No problem,

Any news on this? I'd love not having to keep around the container config files when creating systemd units for pods :)

Not yet. I expect to be working on this in May. Note that you can create systemd units for pods but not (yet) with the --new flag.

I going through issues at the moment and noticed that this one is covered mutliple times. Given https://github.com/containers/libpod/issues/5605 has a more recent conversation, I am going to close this one here and ask to move comments over to https://github.com/containers/libpod/issues/5605.


Job IDs are not loaded

Describe the bug
Job IDs are not loaded when loading an experiment from the service.
Steps to reproduce
from qiskit_experiments.library import FineFrequency
from qiskit_ibm_experiment import IBMExperimentService
from qiskit_ibm_provider import IBMProvider

service = IBMExperimentService()
provider = IBMProvider(instance="XXX")
backend = provider.get_backend("XXX")

exp = FineFrequency(physical_qubits=[0],delay_duration=160,backend=backend)
expdata = exp.run().block_for_results()
print(expdata.jobs()) # Job ID exists.

expdata.save()
load_expdata = ExperimentData.load(expdata.experiment_id, service)
print(load_expdata.jobs()) # Prints an empty list

Expected behavior
The Job ID should be loaded correctly from the service (the ID appears correctly in IQX)
Additional Information

Qiskit IBM Runtime version: 0.11.2
Python version: 3.9
Operating system: Windows

(I am not sure if this bug report really belongs here or in Qiskit Experiments)

The current behavior is that jobs will only be loaded if a provider is given, like load_expdata = ExperimentData.load(expdata.experiment_id, service, provider), but this is confusing. We already recommend in the documentation that users do service = ExperimentData.get_service_from_backend(backend) to get the service, in which case there's already a provider involved. Perhaps we should just require a provider when loading.

Thanks @coruscating !
Adding the provider argument raised an error for some reason, but the get_service_from_backend(backend) did the trick.


How to get totalRows as a props in cellRenderer function

I am trying to get following things in cellRenderer function:

Total rows count
Current number of rows(rows that are currently present in the table ), so that I can get the rowIndex for the newly added row in the table

Any suggestions will be helpful

hi @ac13793 I don't fully understand your question.
usually in your component you specify a rowCount prop to rv components to tell it how many components it will have. and cellRenderer is automatically called with those indices to render the corresponding rows.
I would encourage you to checkout out some of the demos on codesandbox.
hope that helps!


The latest version (3.0.0) is not working in Cordova on Android 7

Probably this issue is on our side, sorry to bothering.


change testing glob to be windows compatible

The previous glob failed on windows machines.

Codecov Report

Merging #784 into master will decrease coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #784     +/-   ##
=========================================
- Coverage   91.26%   91.07%   -0.2%
=========================================
  Files         235      100    -135
  Lines       10882     4942   -5940
  Branches     1027      498    -529
=========================================
- Hits         9932     4501   -5431
+ Misses        950      441    -509

Impacted Files
Coverage Δ

...-api/test/noop-implementations/noop-tracer.test.ts
60% <0%> (-40%)
:arrow_down:

...ry-api/test/noop-implementations/noop-span.test.ts
63.63% <0%> (-36.37%)
:arrow_down:

.../opentelemetry-api/src/trace/NoopTracerProvider.ts
40% <0%> (-40%)
:arrow_down:

...ntelemetry-exporter-stackdriver-trace/src/types.ts
100% <0%> (ø)
:arrow_up:

...s/opentelemetry-plugin-xml-http-request/src/xhr.ts
79.41% <0%> (ø)
:arrow_up:

...ntelemetry-exporter-stackdriver-trace/src/trace.ts
95.83% <0%> (ø)
:arrow_up:

...entelemetry-plugin-dns/src/enums/AttributeNames.ts
100% <0%> (ø)
:arrow_up:

...ry-plugin-dns/test/functionals/dns-disable.test.ts
100% <0%> (ø)
:arrow_up:

packages/opentelemetry-plugin-dns/src/version.ts
100% <0%> (ø)
:arrow_up:

...emetry-plugin-xml-http-request/src/enums/Format.ts
100% <0%> (ø)
:arrow_up:

... and 203 more