graphic window, fill outside bounds of window

Studying the JSON output of Alabaster on Glulxe interpreter, I noticed it seems to be giving fill numbers that would go outside the window if extra calculation aren't done to prevent that. is it possible that RemGlk do the maths on these?
Story data file: http://mirror.ifarchive.org/if-archive/games/glulx/Alabaster.gblorb
If you make the random window match up to 24 for input, JSON input:
{ "type": "init", "gen": 0, "metrics": { "width":720, "height":400, "charwidth":9, "charheight":16 }, "support": [ "timer", "hyperlinks", "graphics", "graphicswin", "sound" ] }

{"type":"char","gen":1,"window":24,"value":" "}

{"type":"line","gen":2,"window":24,"value":"go west"}

Here is what you get out RemGlk to define the left graphics window:
{"type":"update", "gen":1,
 "windows":[
 { "id":27, "type":"graphics", "rock":200,
   "graphwidth":180, "graphheight":400,
   "left":0, "top":0, "width":180, "height":400 },
 { "id":24, "type":"buffer", "rock":201,
   "left":180, "top":0, "width":540, "height":400 }
 ]

Later, down in update 3, you get graphics JSON like this:
{"type":"update", "gen":3,
 "content":[
 {"id":27, "draw": [
{"special":"fill", "x":0, "y":0, "width":180, "height":400, "color":"#FFFFFF"},
{"special":"image", "image":20, "width":60, "height":156, "x":0, "y":0, "url":"http://localhost:4000/static/resource/pict-20.png"},
{"special":"image", "image":20, "width":60, "height":156, "x":60, "y":0, "url":"http://localhost:4000/static/resource/pict-20.png"},
{"special":"image", "image":20, "width":60, "height":156, "x":120, "y":0, "url":"http://localhost:4000/static/resource/pict-20.png"},
{"special":"image", "image":23, "width":90, "height":234, "x":0, "y":0, "url":"http://localhost:4000/static/resource/pict-23.png"},
{"special":"image", "image":23, "width":90, "height":234, "x":90, "y":0, "url":"http://localhost:4000/static/resource/pict-23.png"},
{"special":"image", "image":3, "width":142, "height":220, "x":0, "y":180, "url":"http://localhost:4000/static/resource/pict-3.png"},
{"special":"image", "image":14, "width":162, "height":400, "x":9, "y":0, "url":"http://localhost:4000/static/resource/pict-14.png"},
{"special":"fill", "x":0, "y":0, "width":9, "height":400, "color":"#FFFFFF"},
{"special":"fill", "x":0, "y":0, "width":180, "height":0, "color":"#FFFFFF"},
{"special":"fill", "x":171, "y":0, "width":180, "height":400, "color":"#FFFFFF"},
{"special":"fill", "x":0, "y":400, "width":180, "height":400, "color":"#FFFFFF"}
 ] }

The first "fill" is a clear of the canvas.  It is these last two "fill" that are the topic. If the window has a bounds of "graphwidth":180, "graphheight":400, these fills are basically saying width/height of the full window, but using x and y offsets.

The original Glk spec allows for negative position coordinates. GlkOte accepts drawing commands that fall outside the window coordinates, either because X/Y are negative or W/H are too large. And, as you note, games send such commands.
Therefore, it makes sense to treat such commands as legal.
(Fills and images are clipped to the graphics window size.)


Request timed  out when adding a document to the watson collection

I am trying to upload a sample document from Salesforce Apex class to IBM Watson. I am trying with the following snippet but am getting this error
{
"code" : 500,
"error" : "Request timed out, please try again."
}
Somehow it is getting timed out.
            Blob fileBlob = Blob.valueOf('This is a sample pdf file');
            IBMDiscoveryV1 discovery = new IBMDiscoveryV1('2017-11-07');
            discovery.setEndPoint('https://gateway.watsonplatform.net/discovery/api');
            discovery.setUsernameAndPassword('<<userId>>', '<<password>>');
            IBMDiscoveryV1Models.AddDocumentOptionsBuilder builder = new IBMDiscoveryV1Models.AddDocumentOptionsBuilder(
                    '<<environmentId>>,
                    '<<collectionId>>');

            IBMWatsonFile.FileBuilder fileBuilder = new IBMWatsonFile.FileBuilder();
            fileBuilder.name('Sample.pdf');
            fileBuilder.body(fileBlob);
            IBMWatsonFile file = fileBuilder.build();

            builder.file(file);
            IBMDiscoveryV1Models.AddDocumentOptions options = builder.build();

            docResponse = discovery.addDocument(options);
            System.debug(docResponse.getDocumentId());

How big is your document? Right now the timeout is 120 seconds. See https://github.com/watson-developer-cloud/salesforce-sdk/blob/master/force-app/main/default/classes/IBMWatsonClient.cls#L2

Hey @Nabarun. Thanks for submitting this issue.
I'm seeing similar errors when trying it myself, so I'm going to be looking into it soon. I'll report back with any findings.

Thanks @lpatino10  for verifying this

I think I've found the issue @Nabarun and I'm working on a patch for it now. I'll update again when it's released and ready to use.

The fix is in review now. I detailed the problems in the PR if you're curious @Nabarun.
Once it's merged you should be able to upload files fine, although uploading a PDF won't quite work the way you've detailed in this issue. For a PDF, you're going to need to create a file in your Salesforce UI and query for it in your Apex code to then send it to the Discovery service. Here's an example I used while testing for this patch:
// finding PDF file uploaded to Salesforce with the name "sample_pdf"
String fileTitle = 'sample_pdf';
SObject[] queryResult = Database.query('Select VersionNumber, VersionData, Title, Origin, FileType, FileExtension, Description, ContentUrl, ContentSize, ContentLocation, ContentDocumentId From ContentVersion WHERE Title = :fileTitle');

// the VersionData property has the Blob we want to use for our file
Blob fileBlob = (Blob) queryResult[0].get('VersionData');
IBMWatsonFile.FileBuilder fileBuilder = new IBMWatsonFile.FileBuilder();
fileBuilder.body(fileBlob);
IBMWatsonFile file = fileBuilder.build();

// we need to specify the filename and content type - validators have been added to check for this
IBMDiscoveryV1Models.AddDocumentOptions options = new IBMDiscoveryV1Models.AddDocumentOptionsBuilder()
  .environmentId('<environment_id>')
  .collectionId('<collection_id>')
  .file(file)
  .filename('sample_pdf')
  .fileContentType(IBMWatsonHttpMediaType.APPLICATION_PDF)
  .build();

IBMDiscoveryV1Models.DocumentAccepted resp = discovery.addDocument(options);

However, for something simpler, you can just use a JSON string in pretty much the same way you were doing:
IBMWatsonFile file = new IBMWatsonFile.FileBuilder()
  .body(Blob.valueOf('{ "key": "value" }'))
  .build();

IBMDiscoveryV1Models.AddDocumentOptions options = new IBMDiscoveryV1Models.AddDocumentOptionsBuilder()
  .environmentId('<environment_id>')
  .collectionId('<collection_id>')
  .file(file)
  .filename('sample_json')
  .fileContentType(IBMWatsonHttpMediaType.APPLICATION_JSON)
  .build();

IBMDiscoveryV1Models.DocumentAccepted resp = discovery.addDocument(options);

Hopefully the patch clears everything up for you. Feel free to open this issue back up or create a new one if there are still problems.

Thanks @lpatino10 let me try the solution and respond back to you.

I merged the changes and confirm I am no longer getting the timeout issue. But still the file is not getting inserted in my collection. I will debug more and update with the response.

I am able to get the file upload working for me. Thanks for all your support.

Thanks @Nabarun for your patience and @lpatino10 for your help to get this resolve


Added tests for logic

Math Magicians: Tests
In this PR, we did the following:

Set up React Testing Library.

Wrote unit tests for the files operate.js and calculate.js using Jest.

Created unit tests for all React components:

Used Jest snapshots to test the components.

Used React Testing Library to simulate user interaction.

Okay, thanks. we will walk through it.


Remove backtrace from ECR errors

Beautify ECR errors for issues like #4577, by removing the stacktrace.
Before:

After:

Maybe add a comment to explain the purpose of the rescue?

I've restricted the rescue, to rescue only Errno errors, so other errors will have their backtrace for debugging if needed.

I think we should only rescue some Errno's, specifically ENOENT.

@RX14 done. I rescued ENOENT & EISDIR, I think thoses two will be the most common in this case.

@RX14 I've updated to use long-form if in the rescue block.

Ready to 🚢


Disease mapping

Work on creating a disease map for the data processing pipeline to simplify id matching.
For example, in clinvar data, disease names are very specific, such as "esophageal squamous cell carcinoma" and "pancreatic cancer, susceptibility to 1", these should be mapped to "esophageal cancer" and "pancreatic cancer".

Wrote a quick script to capture all the disease name mappings. It captures most of the mappings but isn't aligned perfectly towards the later indices. During the creation of the original script, there was manual data changes done in excel. At some point during the manual code alterations, 144 rows of data was added (or incorrectly split from the raw data).
In the original jupyter notebook, ClinVar.ipynb, you can see there is 220,938 rows in the dataset before the manual mapping done outside of the code. When the data is read back in after the manual excel changes, there is 220,926 rows. Some splitting is done and the resulting number of rows is 221,082.
Daniall will have to help me correct some of the later mappings in the csv file output from my script. The map can be found in the rework branch (connected to this issue) of the repository in data/disease_maps.csv.

Finished first manual pass through of the disease map csv, updated file was pushed to rework branch. @DaniallMasood will have to go in and confirm/change some of the disease mappings.


Odin causing weird graphical issues

I have these weird graphical issues when using the legit mod. When taking this picture I had no other mods in my mod folder and all odin features disabled.

To clarify:
In the picture you can see entities rotated onto there heads and roatated around some midpoint. And this problem only occurs with other player around.
Hope you can help me,
Thanks in advance

It is a problem with essential cosmetic rendering either enable the setting or update to latest essential version

thx fixed it

thx a lot, fixed it


It do not work in Chrome 47?

It has problem in chrome 47
ERROR Error: Uncaught (in promise): TypeError: Cannot set property style of [object Object] which has only a getter(…)
ERROR CONTEXT DebugContext_ {view: Object, nodeIndex: 4, nodeDef: Object, elDef: Object, elView: Object}

this might be the issue
https://github.com/angular/zone.js/issues/427

I think we can close this issue, I had the same error in the past.


Update paths for our 2f_85 gripper URDF

These were updated in: https://github.com/PickNikRobotics/ros2_robotiq_gripper/commit/8e4025879957a7568afd870c1063010464bd3249
, but not in our gripper URDF.
Needed for properly rendering grasp previews:

Why are we keeping a copy of the grippers URDF here? Is there a reason we can't use the xacros included in robotiq_description (like this example here)?

There's a longer historical answer here, but the short answer is because the UI can't parse nor process the xacro macros in the gripper description package. We still require a standalone, fully parsed URDF for rendering grasp previews. I think we can push in a change to handle processing xacros now that some blocking work has been completed. Hopefully we won't need to maintain this for too much longer...

Tested this with the gazebo config.
Without these changes, the rest API process crashes.


Parsing errors should show the name of the tiddler containing the bad markup

This would be an extension of #44. See example of an issue that should be covered.

In next version


Added debug cmd to print UUID

Added a command that prints the UUID
@alfredh What do you think about this?

great, thanks !


Allow using AP when ps-max-modem is enabled

Contains #281 and includes a change that now prints which expression returned an ESP error.
AP and STA need different beacon intervals so I've added a separate config value for this.

but it contained #281 not #282

Pardon, my bad :)


Distinguish between source lists directives and non-source list directives

The prior behavior tends to default treat directives that are lists as "source lists". But, a source list kind of has specific behavior (ex. they support * to mean "all"). So, there are not meaningful for other lists such as media-type and sandbox. These directives have "lists", but they are unique to their usage. So, we shouldn't treat them like a source list. This PR adds support for media-type and sandbox. In particular it treats them as follows:

sandbox - Can set a maximally strict policy using either sandbox: true or sandbox: []. Any other policy requires each element in the list to be of the form allow-*.
media-type - Requires that each element in the list be of the form type/subtype.

In addition the the above, I also (I think) simplified some of the surrounding logic. I bumped into places where the existing logic had to be adjusted to account for the new top level directive types anyway. And, as I was refactoring, I think I was able to remove some of the nestedness of what was there.

@oreoshake - Took a pass at the changes you asked for in https://github.com/twitter/secureheaders/pull/354/commits/36a22c82dcf6cf5dc0ab4a719785516d8fedfbba.


[Hydroshare GIS] You can add files that are not tif or shp files that don't do anything on the map

I can add .ipynb, .pdf files and so forth that do not change anything on the map into the layers. Although it does not do anything to map, there should be a way not to add those files. It makes no sense to have them in the layers list.
There should be a way that resources with no files that can be loaded onto Hydroshare GIS won't be displayed.

To add to this:

The "Add Resource" button should only search GIS capable resource.  This would drastically speed up the query
You can also add non-gis files using the "Add Local" button.  This file browser should filter to only show GIS capable files as well.

This issue was moved to CUAHSI-APPS/tethysapp-hydroshare_gis#20


[GCB] Enable integration tests when run in --docker

Currently the GCB builds can only run integration tests when run in GCB with --project cloud-cpp-testing-resources, because that's the project and account w/ all the right permissions. However, our dev cycle is slow if we're iterating on an integration test and always waiting 5+ minutes for the integration tests to run in that account.
We want to be able to run integration tests locally in docker.
Idea:

We could create a separate service account (in cloud-cpp-testing-resources) that has all the right permissions to run integration tests.
We create some keys on that account that we can download and use locally
We somehow (I think this may work, but need to tests) enable our gcloud ADC use the SA credential
We need to make sure this SA credential works w/in the docker container
voila, integration tests work w/in a local docker container ???

Note:

We'll need to rotate the SA keys periodically. That's OK.
This SA will only be used for local docker tests -- nothing else.

Not sure if this works, but worth a try maybe. Mount $HOME/.config/gcloud into the corresponding Docker image directory. Then the build scripts could run with whatever ADC the developer has stored there.  If they are an owner for the integration test project (and all of us are) they should have all the privileges needed to run the integration tests.  I think maybe the Bazel builds would need some tweaks to get access to those files, not sure?
That would eliminate the need to download service account keys, create separate accounts, etc.

I love that idea. I actually tried this a few weeks ago and couldn't get it work. I came to the conclusion that our "owner" accounts still didn't have the right permissions. But maybe I was confused maybe something else was broken at the time.
I'll look into this again, because it would be the easiest and nicest solution if it works. Thanks for suggesting!

When trying to run the bazel integration tests as me in docker, I get the following error:
================================================================================
FAIL: //google/cloud/iam/samples:grpc_credential_types (see /h/.cache/bazel/_bazel_jgm/eab0d61a99b6696edb3d2aff87b585e8/execroot/com_github_googleapis_google_cloud_cpp/bazel-out/k8-fastbuild/testlogs/google/cloud/iam/samples/grpc_credential_types/test.log)
INFO: From Testing //google/cloud/iam/samples:grpc_credential_types:
==================== Test output for //google/cloud/iam/samples:grpc_credential_types:

Running UseAccessToken() example
Standard exception raised: Error in non-idempotent operation GenerateAccessToken: The caller does not have permission
2021-04-29T13:26:50.278632325Z [INFO] <140353785537728> Enabled logging for gRPC calls (google/cloud/iam/internal/iam_credentials_stub_factory.cc:48)
2021-04-29T13:26:50.279812706Z [DEBUG] <140353785537728> GenerateAccessToken() << name<EMAIL_ADDRESS>scope: "https://www.googleapis.com/auth/cloud-platform" lifetime { seconds: 60 }  (./google/cloud/internal/log_wrapper.h:86)
2021-04-29T13:26:50.513507172Z [DEBUG] <140353785537728> GenerateAccessToken() >> status=The caller does not have permission [PERMISSION_DENIED] (./google/cloud/internal/log_wrapper.h:89)
terminate called after throwing an instance of 'std::runtime_error'
  what():  Error in non-idempotent operation GenerateAccessToken: The caller does not have permission
================================================================================

The permissions to do that are documented here:
https://cloud.google.com/iam/docs/reference/credentials/rest/v1/projects.serviceAccounts/generateAccessToken
That permission is included in the roles/iam.serviceAccountTokenCreator role.  Try giving yourself that role for the service account in question and see if that works.

To my surprise that one credential seems to do the trick. I'm surprised because we, with "Owner" role, seem to be able to do everything, like access GCS, spanner, etc, but why did "Owner" not give us that one "generateAccessToken" permission?  Not sure.
But anyway, yay! This seems like an excellent solution to running integration tests in docker.


#label-issue does not properly update the "View" category

When I click "x Issues Reported" in kata search, this happens:

Comments are filtered to show issues only, but the selected "View" category is still "All".

Thanks


Merge OnSceneLoaded into FinishLoadScene

Better support support of additive scene loads.
The only breaking change is in OnServerSceneChanged that needs a new argument : LoadSceneMode but i think it's needed for full support of additive scene loads.

interesting.
@MrGadget1024 I assume you tested it in your scene load example?

@tomkrikorian could you please keep the name OnSceneLoaded instead of FinishLoadScene in this PR?

Done :)

Waiting on MrGadget to complete testing and if it passes I think we'll green light this PR.

This cannot be approved at this time.  It breaks the Additive Scenes example, and unnecessarily changes the signature of OnServerSceneChanged.

@tomkrikorian @MrGadget1024 does this work with the latest commits here, or not?

Closing this in favor of #972 and because it's gone stale.


How to retrieve message from postMessage ?

Hello there,
I'm new user of Retool (just a week ago), and I want to display a custom loading component, while the react-retool is loading. To do this, I'm looking to send a message when all the requests are loaded, so my react application, can change a loadingState from true to false or something else
The postMessage seems working, but I can't retrieve the message on my react application
  onRetoolDataReceived = event => {
    console.log('Data from Retool');
    console.log(event.data);
  };

<Retool
  url=[APP]
  onData={this.onRetoolDataReceived}
  data={{
    from: 'YYYY-MM-DD',
    to: 'YYYY-MM-DD',
  }}
/>

In this case, the function is called, because I see the first console.log, but for the second one, nothing is displayed
The message I send in postMessage function is
parent.postMessage(
     {
          type: "REQUEST_FINISHED",
          name: 'request_1',
          id: 'request1_loaded',
          loadingTime: '12ms',
     },

     [MY_APP_PUBLIC_URL]
)

Any ideas ?

I'm struggling with the same problem. Did you manage to find a solution?

@FlorianRuen as a test, can you try posting your message with a wildcard as the second argument?
parent.postMessage(
     {
          type: "REQUEST_FINISHED",
          name: 'request_1',
          id: 'request1_loaded',
          loadingTime: '12ms',
     },
    '*'
)

If that doesn't do it, would you please share the version of Retool and the version of react-retool you're using?

@ftoyoshima I haven't found a solution to date, so I was doing with a classic loader on the iFrame but the rendering was not terrible, because the loader disappeared before the end of the iFrame loading
@kentwalters It's not working,
I add a new query that run a JS script (with content in your message)
But in my app console with onData, I can see every message when an iFrame param change

But my custom message from postMessage isn't displayed here (and a notification display query successfully run, so I can confirm the postMessage is executed on Retool side)
For the react-retool version, I'm using v1.4.0, which seems to be the latest

@FlorianRuen @ftoyoshima Can you please confirm that in /settings/beta you've enabled "Enabled parent window communication"?

That's it! This solved the problem! Thanks for the support @kentwalters!

@kentwalters it's already enabled on my side. So the problem persist

@ftoyoshima maybe you can post your code or something to try to find the source of my problem ?

@FlorianRuen, I created a minimal working example here:
https://github.com/ftoyoshima/react-retool-sandbox
The linked Retool App, is basically a button which triggers a JS Query with the following code:
console.log('(Retool internal log is working)')
parent.postMessage('hi!!', '*')

@FlorianRuen you can also clone and run this project, which includes a working sample application. Can you try that and let us know what you see?

@kentwalters not working for me, I got the same : one button that refresh all requests and after trigger a query
console.log('(Retool internal log is working)')
parent.postMessage('hi!!', '*')

And on my app side, I got only the message generic from retool (same as my previous screenshots), but I can't see the "Hi" message appear in my console ...

@FlorianRuen, did you try it with React application? Maybe it is an issue with some React configurations.

@ftoyoshima Seems working now!
I think it was always working, I thought the postMessage message would appear in the console, along with all the other messages with "Received message from parent" but actually not at all...
maybe I was looking in the wrong place,
but by the way, maybe the text in the console "Received message from parent" doesn't make sense in a sense, if the messages from postMessage are not listed the same way?
This text in the console isn't from my code, but from Retool (can't find occurence in the library), can you see it in the console while the iframe is loading ?


On Azure hosted webapp logs are not getting through Seq, I had to set ServicePointManager.Expect100Continue to false

I had to set ServicePointManager.Expect100Continue = false although this isn't good, because it forces the whole app to use this since this is a global setting.
This sink should propose a setting to set ServicePointManager only for seq  (https://stackoverflow.com/a/15341369/2047306)

Thanks for the heads-up, @silversens :+1:
It might be possible to work around this by using an HttpClientHandler with the sink, e.g.:
https://msdn.microsoft.com/en-us/library/system.net.http.webrequesthandler.continuetimeout(v=vs.110).aspx
var handler = new WebRequestHandler();
handler.ContinueTimeout = TimeSpan.Zero;

Log.Logger = new LoggerConfiguration()
    .WriteTo.Seq("http://localhost:5341", messageHandler: handler)
    .CreateLogger();

I can't confirm that this works, but if it does, it might be a quicker workaround than going through the process of adding some kind of direct support here. Are you in a position to spin it up and give it a try?
Best regards,
Nick

@nblumhardt thanks for the quick feedback.
It implies stop using appsettings and those keys :
  <add key="serilog:using:Seq" value="Serilog.Sinks.Seq" />
  <add key="serilog:write-to:Seq.serverUrl" value="http://localhost:5341" />
  <add key="serilog:write-to:Seq.restrictedToMinimumLevel" value="Verbose" />

So it might work (I still didn't test it) but I'd rather keep having everything in config.

Thanks for the reply 👍
The problem with adding features directly to the sink is that there's a very broad surface area on HttpClient, the handlers for various platforms, and network infrastructure in the middle. Supporting them all specifically would make the maintenance and testing of the sink much harder.
If the workaround works, you could make a slight modification to your config so that it's not picked up by the default ReadFrom.AppSettings():
  <add key="serilogx:write-to:Seq.serverUrl" value="http://localhost:5341" />
  <add key="serilogx:write-to:Seq.restrictedToMinimumLevel" value="Verbose" />
  <add key="serilogx:write-to:Seq.expect100Continue" value="False" />

and then in code use:
.WriteTo.Seq(ConfigurationManager.AppSettings["serilogx:write-to:Seq.serverUrl"] /* etc */)

What do you think?

Thanks for the reply.
This is a solution I'll consider.
Although, even if you're right "exposing every configurable thing in settings is not a solution", I still believe Azure won't be a rare platform.
We'll see if supports for this gets requested a lot with time.
Regards,
Sÿl

Thanks, yes, this makes sense 👍
A lot of Seq customers currently use Seq successfully with Azure - I wonder if it's possibly something to do with your virtual network configuration? I haven't personally published an Azure web app for quite a while, though, so given an opportunity I'll spin one up and see if I hit the same problem. Thanks for the feedback!
Nick

Nick,
Just to be clear, I meant Azure in PaaS, so no virtual network configuration.
You can try with free hosting.
 I haven't seen any further reports of this, so closing, but keen for anyone else who hits it to chime in here :-) Cheers!


Flutter doctor does not recongizes the SDK

I am having problems passing the flutter doctor test.
I have all the env paths correctly in addition to both ANDROID_HOME and JAVA_HOME variables.
The output is the following:
C:\Users\Moises Rodam>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.0.5, on Microsoft Windows [Version 10.0.19042.928], locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.
[√] Chrome - develop for the web
[√] Android Studio (version 4.1.0)
[√] VS Code (version 1.55.2)
[√] Connected device (2 available)

! Doctor found issues in 1 category.

Then:
C:\Users\Moises Rodam>flutter doctor --android-licenses
Error: Could not find or load main class Rodam\AppData\Local\Android\Sdk\cmdline-tools\latest\bin\\..

You can notice that it Could not find or load main class Rodam\AppData\... because the folder name is C:\Users\Moises Rodam\ (Moises Rodam).
The ' ' is running the folder as two different commands. I believe Moises is ran before.
How can I fix it?
It should be referring to Moises Rodam\AppData\...
Thank you!

@mrodan It is better to separate the name with _ or - instead of using space. Space will create issue further when you run your app too. That's also one of the reason, why Flutter apps don't have space in the name.

@mrodan
Can you provide complete error log ? This seems to be incomplete Error: Could not find or load main class Rodam\AppData\Local\Android\Sdk\cmdline-tools\latest\bin\\..

@mrodan
Can you provide complete error log ? This seems to be incomplete Error: Could not find or load main class Rodam\AppData\Local\Android\Sdk\cmdline-tools\latest\bin\\..
Answer: @darshankawar that was the complete log error.

@mrodan It is better to separate the name with _ or - instead of using space. Space will create issue further when you run your app too. That's also one of the reason, why Flutter apps don't have space in the name.
Answer: @AbhishekDoshi26 I know. Next time, ill name my user with no space.
Also, you were right, it gave me errors when running. I had to create a joint and change env var path.

Thank you both

C:\Users\Avneesh Singh Yadav>flutter doctor --android-licenses
Error: Could not find or load main class Singh
Caused by: java.lang.ClassNotFoundException: Singh
could you help please


Resolve Gradle Conflicts

Remove Double Declarations of dependencies and Resolve Conflicts on Master Branch

@chepsi check whether the Gradle conflicts has been resolved by pulling the latest changes  on both master and develop branch

@wangerekaharun the issue has been resolved.

Okay. Closing the Issue now


Update README

Updating the README on point 2 and 8.
Point 2 now asks the user to run eval $(docker-machine env default) first as this won't do any harm and will remove some friction with the install (better to just do this first than error and then run this, then go back).
Point 8 actually tells the user to type start once the fuge shell opens!

a new release?
that hasn't been published yet ?


Screen mirrored

I tried to run the m5stack example on an m5stack core. Using rotation=0 shows the text from right to left. Also width and height are reversed. rotation= 6 shows the text in the upper left corner from left to right, as expected, but width and height are reversed (I set w=320 and h=240, but i need to set w=240 and h=320 to get it right)
Thank you

Glad you got it to work. I have not tried it in an MStack since the rotation stuff was incorporated. I can update the example or happy to accept a Pull Request if you have it working.


Fix Travis build

Summary
Travis has been failing with 1 scenario, and occasionally with 3.
Scenario: Asking the command line tool for help

Looks like an expected newline has been removed, which I assume is down to a new release of commander (the module we use to provide CLI help). I've removed a newline from the expectation. It doesn't matter much from a readability perspective.
The other two failures are random and tend to be to do with timeouts. So I've made the callbacks explicit, should be more robust now.

@andymsuk or someone else more familiar with the Travis stuff should probably review. Though it looks fine to my less experienced eyes.

@ChrisBAshton - Other than the changes to command_line_help.feature, the rest seem redundant and like they should not have made any difference. Returning a promise should have the same affect as calling done() at the end and returning a Promise that resolves shouldn't do anything either (unless it's something to do with an extra tick?) What happens if you revert them? Does it fail again? If so, I feel like there might be an underlying issue.

Looks like you're right, @andymsuk ! Shall I squash and merge?


UEKR6 repo metadata not found for AArch64 target

Even with a -k flag, the script tries to fetch repo metadata for ol8_UEKR6, which failes for AArch64 targets.
$  ./centos2ol.sh -k -V
<...snip...>
Errors during downloading metadata for repository 'ol8_UEKR6':
  - Status code: 404 for https://yum.oracle.com/repo/OracleLinux/OL8/UEKR6/aarch64/repodata/repomd.xml (IP: <IP_ADDRESS>)
Error: Failed to download metadata for repo 'ol8_UEKR6': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried

https://yum.oracle.com/oracle-linux-8.html suggests that there is only an x86 build of the ol8_UEKR6 repository available, and sure enough:
$ curl -I  https://yum.oracle.com/repo/OracleLinux/OL8/UEKR6/x86_64/repodata/repomd.xml
HTTP/1.1 200 OK
$ curl -I  https://yum.oracle.com/repo/OracleLinux/OL8/UEKR6/aarch64/repodata/repomd.xml
HTTP/1.1 404 Not Found

I'd guess that this wants to be conditionally enabled based on basearch:
		[ol8_UEKR6]
		name=Latest Unbreakable Enterprise Kernel Release 6 for Oracle Linux $releasever ($basearch)
		baseurl=https://yum.oracle.com/repo/OracleLinux/OL8/UEKR6/$basearch/
		gpgkey=file:///etc/pki/rpm-gpg/RPM-GPG-KEY-oracle
		gpgcheck=1
		enabled=1

Though that would cause this later logic to fail as yum-config-manager won't know anything by the name of UEK to disable them.
# Disable UEK repos if UEK kernel is not being installed
if ! "${install_uek_kernel}"; then
  echo "Disabling UEK repositories since we are not installing the UEK kernel"
  yum-config-manager --disable \*UEK*
fi

So probably the logic needs a more detailed overhaul for missing ol8_UEKR6 builds for AArch64.

Thanks for reporting this. UEK6 for aarch64 is in the BaseOS Latest repo, I suspect because the minimum kernel release we have for aarch64 is UEK6. Meanwhile, we are look at fixing this logic, unless you want to submit a PR. Be sure to check our Contribution guide first, if you do.

If I'm you understanding right, does that mean an AArch64 upgrade is always going to change the kernel around? That would mean that this workaround later in the script:
          if [ -d /sys/firmware/efi ]; then
              grub2-mkconfig -o /boot/efi/EFI/redhat/grub.cfg

is likely to need to be unconditionally applied for an AArch64 upgrade. I do find myself with a system stuck at the grub command line after the A64 upgrade process, and need to manually set my linux/initrd to get back up and running (after which, I regenerate the grub config as above, of course).

If I'm you understanding right, does that mean an AArch64 upgrade is always going to change the kernel around?

Correct.


add enum entry for ARCHIVED because mailchimp uses it

Thanks for your wrapper library, as it helped me interface with the pretty confusing MailChimp API (in hindsight mainly because of multiple names for objects in the model). Got an exception during my testing so I added this missing status, and then looked for any others mentioned in documentation but this seems to be only one AFAIK.

Thankyou for your contribution. This change has been added to the develop branch and will be included in the next release.


bindings in

const who = 'world';
render(html`<template id="res">hello ${who}</template>`, document.body);

// `hello {{}}` instead of `hello world` :(
console.log(document.getElementById('res').innerHTML);

Oh, interesting...
I suppose we have to walk into nested template contents when preparing a template.

@valdrinkoshi #50 starts us on the path to being able to fix this. Because it does a recursive walk on the template, rather than using a TreeWalker, we can choose to special-case
How critical is this?

The fact that we're using a template literal with a template element within it it's a bit strange, I know, but nevertheless I think it's important we support it for inter-op.
I've been updating iron-overlay to use lit-html.
<iron-overlay> gets a template via distribution, stamps it and hosts it in a stacking-context-safe node (e.g. <body>). It relies on <template> so that the overlay content gets created and appended only when needed.
Another similar example is <iron-list>, which gets the item template via distribution.
If I want to use <iron-list>, I have to pass a template:
const templateResult = html`
<iron-list items=${props.items}>
  <template>
    <div> lit-html binding: ${props.userName}, polymer binding: [[item]] </div>
  </template>
</iron-list>`;

Well, with #50 I think we can do this, but it might be a bit confusing to deal with two separate data-binding systems. The lit-html bindings are going to change the <template>'s content. You have that right above, but I wonder how many would get that distinction immediately.
It's be nice to have an iron-list that took something even a bit more abstract than template elements, maybe a function that produces DOM (something like (item, index) => Node).

What about event binding?
<iron-list items="${items}">
  <template>
    <div on-click="${(e) => this.onItemClick(e)}">
      [[item]]
    </div>
  </template>
</iron-list>

Does not work in #50


Building docs locally not working with latest commit #32e7866

I am getting below error in console when I tried to build docs locally.
Uncaught ReferenceError: angular is not defined

I've change some things around. Can you do a pull and test it out?

I had the same issue, it was resolved by a bower install

This error doesn't seem to be cropping up anymore, but if you have any more issues compiling the docs locally, let us know.


providers.gradleProperty() doesn't read value from module's gradle.properties file

I have gradle.properties in the root project dir and subproject dir. And I define the same property in both of them
Expected Behavior
When I read  property via providers.gradleProperty() in subproject build.gradle I should see the value from subproject gradle.properties file.
Current Behavior
I see value from root gradle.properties
Context
I have plugin applied to each module in my project. I try to change property value seen by some of subprojects by creating gradle.properties in those modules
Steps to Reproduce
I've created a repo with a repoducer and details https://github.com/sboishtyan/Gradle-read-property-question
Your Environment
Gradle 7.5.1
Java 11

Thank you for providing a valid reproducer.
The issue is in the backlog of the relevant team and is prioritized by them.

Duplicate of #13302

Ref https://github.com/gradle/gradle/issues/13302#issuecomment-637272694, workaround:
// TODO remove this once https://github.com/gradle/gradle/issues/23572 is fixed
fun Project.localGradleProperty(name: String): Provider<String> = provider {
    if (hasProperty(name)) property(name)?.toString() else null
}

As that one was closed due to frustration, this is more a follow-up, than a duplicate. :-D
Just mentioning so that none has the idea to close this as duplicate. :-)

You are right, I've re-edited it to "Links".

I just tested it works correctly when you pass properties from the command line or from $GRADLE_USER_HOME/gradle.properties. I'll just leave this here for myself.

Stored in a gradle.properties file in a project directory, then its parent project’s directory up to the project’s root directory.

Source: https://docs.gradle.org/current/userguide/build_environment.html#sec:gradle_configuration_properties
I want to force android.injected.testOnly=false on a specific app module but having it globally doesn't break anything, so it will have to do, for now.

Here is a workaround I use to access gradle.properties from a plugin. Visually, it should be compatible with isolated projects and configuration cache, but I haven't tested it extensively yet.
/**
 * A utility class providing access to project properties declared in `gradle.properties`
 * files from the project directory up to the root project. The properties closer to the
 * current project directory take precedence over properties in parent directories.
 *
 * It is a workaround for the issue when `ProviderFactory.gradleProperty` and `GradleProperties`
 * take into account only root `gradle.properties`.
 * TODO: Remove when the issue is fixed
 *   https://github.com/gradle/gradle/issues/23572
 */
abstract class ProjectGradleProperties(
    private val layout: ProjectLayout,
    private val environment: Environment,
    private val rootDir: File,
) {

    @Suppress("unused")
    @Inject
    constructor(
        layout: ProjectLayout,
        environment: Environment,
        project: Project,
    ) : this(layout, environment, project.rootDir)

    val properties: Map<String, String> by lazy {
        val properties = mutableMapOf<String, String>()
        layout.projectDirectory.asFile
            .walkUpToRoot(rootDir)
            .map { it.resolve(Project.GRADLE_PROPERTIES) }
            .mapNotNull(environment::propertiesFile)
            // Properties closer to the current project take precedence
            .forEach(properties::putAllAbsent)

        properties
    }

    fun byNamePrefix(prefix: String): Map<String, String> {
        return properties.filterKeys { it.startsWith(prefix) }
            .mapKeys { (key, _) -> key.removePrefix(prefix) }
    }
}

//region Utils
private fun File.walkUpToRoot(rootDir: File): Sequence<File> = sequence {
    var current = absoluteFile
    while(true) {
        yield(current)
        if (current == rootDir || current.parent == null) break
        current = current.parentFile
    }
}

private fun <K, V> MutableMap<K, V>.putAllAbsent(other: Map<K, V>) {
    for ((key, value) in other) {
        if (key !in this) this[key] = value
    }
}
//endregion

Usage:
val gradleProperties = objects.newInstance<ProjectGradleProperties>()
gradleProperties.byNamePrefix("myplugin.")

It should be easy to adapt this snippet to use Provider API if needed.


Variables for top level wftemplate/cronwf name

if i have a cronwf or wftempl i can't seem to find variable for the top level object name of each of those? ie i want mutex but {{workflow.name}} becomes the generated name if submitted manually ie mycron-123 vs mycron-124 so its not effective as those names are unique. want to have variable of mycron dynamically in the mutex (without hardcoding mycron)

Related to #7611

i wonder if these work: {{workflow.labels.workflows.argoproj.io/workflow-template}} or
{{workflow.labels.workflows.argoproj.io/cron-workflow}}


Graphics out

Hello!
I need to add an HDMI output to the pocketbeagle. (even if it requires a chip like the NXP TDA)
How can I go about that and how could I make it work with linux?
Thank you so much!

Are you designing a new board based on the SiP or just trying to design an adapter for the PocketBeagle?
PocketBeagle does not break out all of the SiP's LCD display lines to headers you would need a display adapter from the available peripherals. USB might be able to do it but it won't be particularly performant.
If you are making a new board based on the SiP you can leverage the design from other Beagles (if you can source the chips):

I want to design a new board based on the SiP and reflow the chip from a pocket beagle
Thank you so much for the provided schematic!


zimme:collection-behaviours

I'm not currently using your package, but I intend to in the future.  Real nice work and definitely needed.
I just ran into another collection-behaviors, apparently quickly gaining traction.  Realized it was a different one than the one you had on your list and thought you might want it on your TODO list, at the very least.  I think the zimme package has autoform support which might be the upper hand for it right now.
Anyway, just an FYI ticket. :)

Yes. At the time that I wrote this, sewdn's version had more downloads than zimme's so I thought that was the dominant one, but looks like zimme's is more active. I will put it in.

I won't be able to use this package until you put it under it's own namespace, won't use a package which adds new functionality under Meteor (#2).
I think I could use this package to add attachBehaviour to Mongo.Collection.prototype.
I think I could also use this package to automatically add behaviours to all collections if a package setting was available on Meteor.settings.
I'm currently working on a es2015 re-write of all my packages so this would be a good time to try and use this package.
I think the idea of this package is nobel, but it needs work and I may be able to help with that later on.

I will put it on my todos.

@zimme, I've deprecated the Meteor namespace for the functionality. It's now on the CollectionExtensions namespace.

Oh wait, hold on, I got the naming wrong.

Okay it should be good now.

Nice, I'm doing a es2015 rewrite of zimme:collection-behaviours will be releasing it under collectionbehaviours:core too. I'll switch over to use this package for the Mongo adding stuff.


service configuration:node port should can not repeat

In GitLab by @cuiaq123 on Mar 5, 2018, 17:17
If you are reporting a problem, please make sure the following information are provided:

Version of board (last git commit number) , docker engine and docker-compose.
Config files of board, you can get them by packaging "board.cfg" and files in the same directory, including subdirectory.
Log files, you can get them by package the /var/log/board/.
Other necessery information.

In GitLab by @luotian-github on Nov 8, 2019, 07:46
will consider checking for duplicate node port in system

In GitLab by @yhua123 on Nov 8, 2019, 07:46
duplicate #771

In GitLab by @sokril on Nov 8, 2019, 08:18
closed

In GitLab by @sokril on Nov 8, 2019, 08:18
assigned to @luotian-github


changed button style for widget button

styling separated for widget button so it won't interfere with start button.

:rocket: deployment of nhsuk/register-with-a-gp-beta-web succeeded (http://register-with-a-gp-beta-web-pr-218.dev.beta.nhschoices.net)


Output in 0xRRGGBB format?

Hi, I am using PixiJs and it uses the '0xFFFFFF' notation. Do colord have this output format? I will .replace('#', '0x') for now, but a built-in way would be interesting.

Hi! I opened #109, I hope that solves your issue!
I peeked around because of https://github.com/discordjs/discord.js/pull/9756, which saw some demand for a color library, and base10 is just the best format since it's what Discord natively uses and it's what the library can process the fastest.

Awesome, Kyra!


clean up schema-diff

By making the payday tables permanent in #3441, we introduced schema drift. make schema-diff now fails.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

+1 from @rohitpaulk in https://github.com/gratipay/gratipay.com/issues/3471:

make schema-diff errors out when deploying code. It complains about differences between the local and production DB, and all of them are related to payday. This happens because we don't delete all payday related tables and functions in our production DB until the next payday starts. This change was intentional - it helps us debug issues related to payday. I can't find the exact PR where the change was made, if someone does please link it here.

One solution that I see here is to run payday.sql against the test DB before generating local.sql.

Me over there:

We could also make the payday_ tables permanent instead of temporary.

This change was insufficient:
diff --git a/recreate-schema.sh b/recreate-schema.sh
index 2492343..0846bd8 100755
--- a/recreate-schema.sh
+++ b/recreate-schema.sh
@@ -24,6 +24,7 @@ echo "Applying sql/schema.sql ..."
 echo

 psql $DATABASE_URL < sql/schema.sql
+psql $DATABASE_URL < sql/payday.sql

 echo "=============================================================================="

I'm seeing a few differences - one is that the production database has a table called 'affected'. Looks like this was used for sending a temporary email or so? Checking issues..

Hmm, doesn't look like it was under https://github.com/gratipay/gratipay.com/issues/4393

I don't see any code anywhere that uses it, going to delete.

Deleted.

One more: payday_subscriptions is present on production, but not local. I don't see code that uses it either, maybe part of the old payday script?

Yep, removed in https://github.com/gratipay/gratipay.com/commit/151bda0a2855715d46ab246fbad35b57b35b1b49.
I'm deleting the table on production

One old function lying around from when we were using usernames instead of IDs. I didn't know that Postgres supported functions with the same name but different signatures...

Down to a single statement now:

^ Fixed that by adding --no-privileges when calling pg_dump

Closed in #4450.

Closed in #4450.


Improve error handling, add audio input select

Make the program display a serious error message if the retry client fails
Allow the user to select an audio input source

I meant to make this PR on my fork, sorry.


execute examples\css\build.js unresponsive if i didn't install webpack

Bug report

What is the current behavior?
execute examples\css\build.js unresponsive
If the current behavior is a bug, please provide the steps to reproduce.
cd .\examples\css
node .\build.js（do not install webpack）

What is the expected behavior?
should show install webpack tips in terminal

Other relevant information:
webpack version:  5.88.2
Node.js version:16.16.0
Operating System:window10
Additional tools:

We have docs here https://github.com/webpack/webpack/tree/main/examples#building-an-example


Example app doesn't exit on OS X

Shoudn't != 'darwin' really be == 'darwin' ?

Read the comment above that line of code
app.on('window-all-closed', function() {   // On OS X it is common for applications and their menu bar   // to stay active until the user quits explicitly with Cmd + Q   if (process.platform != 'darwin') {     app.quit();   } });
On OS X it is common for applications and their menu bar to stay active until the user quits explicitly with Cmd + Q

Yep, ditto @santiq, this is mirroring the native behavior on OS X.

@noseglid we opened #26 to create a more expected OS X behavior of being able to restore the app once all the windows are closed by clicking the doc icon.

I must admit i misunderstood the comment. I've always hated this behavior in OS X and i just assumed Electron was trying to mitigate it, not simulate it :). My bad. Thanks for response


Feature request [rust]: allow specifying TLS flavour to use

Feature Request
Motivation
We are building our application against musl, cross-compiling it from regular libc Ubuntu. However, we are having issues with svix client, because it depends on system OpenSSL, which in our case would be linked against libc. There are workarounds, but they are a bit involved.
Proposal
Allow user to choose whether openssl or rustls version of reqwest will be used by the svix client
Alternatives
Alternative is to either vendor openssl, comiling it each time, or to use a pre-made Docker image with openssl compiled against musl included. However, we are already using reqwest with rustls, and would like to avoid having both flavours in the dependency tree, as well as extra complexity in the build pipeline.

Thanks @msdinit!
We actually explicitly switched away from rustls to openssl because it was causing issues. Take a look at #704, and let me know your thoughts.

Ah, I see. Did not notice that as I only searched through the issues, not PRs :)
Regardless though, the PR you linked deals with the server, my request is about the client.
Would you be open to having rustls as a non-default option for the client only? I believe the overhead should be minimal (rename dependencies as reqwest-openssl and reqwest-rustls and conditionally importing from them, though would need to actually try it out)

Oh, sorry, I misread your request.
100%, we can easily change that, and potentially even make rustls the default. I didn't realize we were using openssl for that. Yeah, should be trivial.

Awesome, thank you!

The following patch seems that it does what is necessary, I have then ran three commands making sure the dependencies did or did not appear on the dependency tree as expected:
 # we have only the rustls-tls
 cargo tree --features rustls-tls --no-default-features

 # we have both since features are additive
 cargo tree --features rustls-tls

 # we have only default-tls
 cargo tree

commit a212a162747e79cc6e814caa29764c15f9b0c65f
Author: Marcos Antonio Lopes<EMAIL_ADDRESS>Date:   Fri Apr 14 10:15:50 2023 +1000

    Feature request [rust]: allow specifying TLS flavour to use

    towards issue https://github.com/svix/svix-webhooks/issues/889

diff --git a/rust/Cargo.toml b/rust/Cargo.toml
index a083cdd..bb118eb 100644
--- a/rust/Cargo.toml
+++ b/rust/Cargo.toml
@@ -25,5 +25,9 @@ serde = "^1.0"
 serde_derive = "^1.0"
 serde_json = "^1.0"
 url = "^2.2"
-reqwest = { version = "^0.11", features = ["json", "multipart", "default-tls"], default-features = false }
+reqwest = { version = "^0.11", features = ["json", "multipart"], default-features = false }
+
+[features]
+default = ["reqwest/default-tls"]
+rustls-tls = ["reqwest/rustls-tls"]

Oh awesome @marcoslopes, mind opening a PR? We would love to merge it.


Hadamard bottleneck

Hi thanks for your work!
Since the Hadamard matrix can only have a matrix size 4nx4n all the model dimension (or intermediate dim) that are not multiple of 4 cannot use the Hadamard rotation right?
For example qwen 0.5B 2.5 cannot use this roation?
Thanks in advance

yeah indeed...
thanks!


Some content in index.md should use the remove-from-content-only class

This is used on FUN:
https://lms.fun-mooc.fr/courses/course-v1:inria+41026+session02/courseware/b59d1c0260ce40b28f145f4200780333/fe43500ef1c84d1b80d6a95d1ae8b275/
At least the things highlighted in this screenshot should be hidden on FUN by using the remove-from-content-only CSS class, there may be more in other places:

OK not important enough.


Failed to Install Provider - SHA256 checksum failed

Unable to initialize Terraform due to a SHA256 checksum error.
Terraform Version
Terraform v1.2.5 on darwin_arm64
Terraform Configuration Files
terraform {
  required_providers {
    multipass = {
      source = "larstobi/multipass"
      version = "1.3.0"
    }
  }
}

provider "multipass" {
  # Configuration options
}

Steps to Reproduce
terraform init
Error  Info
Initializing the backend...

Initializing provider plugins...
- Finding larstobi/multipass versions matching "1.3.0"...
- Installing larstobi/multipass v1.3.0...
╷
│ Error: Failed to install provider
│
│ Error while installing larstobi/multipass v1.3.0: checksum list has unexpected SHA-256 hash
│ eb5cd9b4fb82076db54b054d30c8cee3ac49f2e7f1b8f5bb1554661788890846 (expected
│ e80e4a4a70e074c58083c1f52ad510918c3b7d305beb878e841d1cba45040e3e)
╵

Thanks for reporting! I wasn't able to fix it, so I released a new patch version with an empty commit and tested myself. Was able to install. Try v1.3.1 instead, please.
Thanks again!


Decode Seq2Seq with unexpected  hidden clear

Bug Description
I noice a commit try to reduce memory usage by clear preState.hidden in advance. But it will cause unexpected crash in following decode step
https://github.com/facebookresearch/flashlight/blob/712c9a3fd0a01d1ca7f8b6f49487ea155fa0bcda/flashlight/app/asr/criterion/TransformerCriterion.cpp#L387
In normal case, buf->prevStates is a vector of TS2SState. Every TS2SState->hidden should have same size (Maybe 6 in my case?). But sometimes TS2SState->hidden is empty, that will cause crash in decode step(Because in decode step it will concatenate buf->prevStates)
I wrote a little code to demonstrate
// flashlight/app/asr/criterion/TransformerCriterion.cpp
      std::stringstream ss;
      for(int ii=0;ii < buf->prevStates.size();ii++) {
          auto &state = buf->prevStates[ii];
          ss<<" "<<state->hidden.size();
      }
      FL_LOG(fl::INFO) << "decode call input.dims ="<<buf->input.dims()<<", ys.size()="<<buf->ys.size()
       <<" prevStates.hidden size =["<< ss.str() << "]prevStates.size()=" <<buf->prevStates.size();

      std::tie(amScores, outStates) = criterion->decodeBatchStep(
          buf->input,
          buf->ys,
          buf->prevStates,
          buf->attentionThreshold,
          buf->smoothingTemperature);

The output will be like
I1113 09:47:35.609973 82176 TransformerCriterion.cpp:394 decode call input.dims =512 83 1 1, ys.size()=1 prevStates.hidden size =[ 0]prevStates.size()=1
I1113 09:48:48.601410 82176 TransformerCriterion.cpp:394 decode call input.dims =512 83 1 1, ys.size()=4 prevStates.hidden size =[ 6 6 6 6]prevStates.size()=4
I1113 09:48:49.637162 82176 TransformerCriterion.cpp:394 decode call input.dims =512 83 1 1, ys.size()=9 prevStates.hidden size =[ 6 6 6 6 6 6 6 6 6]prevStates.size()=9
I1113 09:48:49.658872 82176 TransformerCriterion.cpp:394 decode call input.dims =512 83 1 1, ys.size()=9 prevStates.hidden size =[ 6 6 6 6 6 6 6 6 6]prevStates.size()=9
I1113 09:48:49.684775 82176 TransformerCriterion.cpp:394 decode call input.dims =512 83 1 1, ys.size()=9 prevStates.hidden size =[ 6 6 6 6 6 6 6 6 6]prevStates.size()=9
I1113 09:48:49.727832 82176 TransformerCriterion.cpp:394 decode call input.dims =512 83 1 1, ys.size()=10 prevStates.hidden size =[ 6 6 6 6 6 6 6 6 6 6]prevStates.size()=10
I1113 09:48:49.759795 82176 TransformerCriterion.cpp:394 decode call input.dims =512 83 1 1, ys.size()=8 prevStates.hidden size =[ 0 0 6 6 6 6 6 6]prevStates.size()=8
F1113 09:48:49.763934 82176 Decode.cpp:655 Exception in thread 0

Note the second line from the bottom output prevStates.hidden size =[ 0 0 6 6 6 6 6 6]. First two prevStates's hidden vector is empty
Decode is fine in old code
Reproduction Steps
Run master code after this commit use decode_transformer_s2s_ngram.cfg
Platform and Hardware
V100 16G * 8
Ubuntu with Docker

Hey, thanks for reporting. I have several comments.
Here https://github.com/facebookresearch/flashlight/blob/712c9a3fd0a01d1ca7f8b6f49487ea155fa0bcda/flashlight/app/asr/criterion/TransformerCriterion.cpp#L371 we full the buffer with necessary batch of previous states. Then we call decode https://github.com/facebookresearch/flashlight/blob/712c9a3fd0a01d1ca7f8b6f49487ea155fa0bcda/flashlight/app/asr/criterion/TransformerCriterion.cpp#L384 for the full batch and then we clean up the previous state for which we computed the outputStates https://github.com/facebookresearch/flashlight/blob/712c9a3fd0a01d1ca7f8b6f49487ea155fa0bcda/flashlight/app/asr/criterion/TransformerCriterion.cpp#L398. We don't use the previous state anywhere else. So it should not cause the problem of null pointers for the previous states. However with this fix now you can run with larger beams / larger input sequences.
Could you run with commented part of code on cleaning the previous states? Do you have another behaviour?
Also cc @xuqiantong if I missed something.

I dig a little deeper into code. Here's what I found:
The root case is because prevStates.hidden does not has the same dims. Function input parameter rawPrevStates is vector<TS2SState*>. But these TS2SState* seems share the same  pointer. That means, rawPrevStates has a lot identical TS2SState* pointer.(Is this as design or a pointer bug?)
In my case, I got 2 loop here
https://github.com/facebookresearch/flashlight/blob/712c9a3fd0a01d1ca7f8b6f49487ea155fa0bcda/flashlight/app/asr/criterion/TransformerCriterion.cpp#L363
so in the first loop end, we clear hidden
https://github.com/facebookresearch/flashlight/blob/712c9a3fd0a01d1ca7f8b6f49487ea155fa0bcda/flashlight/app/asr/criterion/TransformerCriterion.cpp#L401
will clear more than we expected.
Add a little code to demonstrate here
https://github.com/facebookresearch/flashlight/blob/712c9a3fd0a01d1ca7f8b6f49487ea155fa0bcda/flashlight/app/asr/criterion/TransformerCriterion.cpp#L398
for (int i = start; i < end; i++) {
        TS2SState* prevState = static_cast<TS2SState*>(rawPrevStates[i].get());
        if (prevState) {

          if (testFlag) {
            FL_LOG(fl::INFO) <<"Before clear "<<i<< " "<< rawPrevStatesDetail(rawPrevStates);
          }

          prevState->hidden.clear();
          if (testFlag) {
            FL_LOG(fl::INFO) <<"After clear "<<i<<" "<<rawPrevStatesDetail(rawPrevStates);
          }
        }
      }

We will get
TransformerCriterion.cpp:439 Before clear 0 [ 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6 6]
38368 TransformerCriterion.cpp:444 After clear 0 [ 0 0 0 0 0 0 6 0 6 0 0 0 6 6 6 6 6 6]

So in second loop run, we will get prevStates.hidden size =[ 0 0 6 6 6 6 6 6]

I suggest a fix for this
https://github.com/facebookresearch/flashlight/blob/712c9a3fd0a01d1ca7f8b6f49487ea155fa0bcda/flashlight/app/asr/criterion/TransformerCriterion.cpp#L398
      std::unordered_set<TS2SState*> validPrevStateSet;
      for (int i = end;i < rawPrevStates.size();i++){
        validPrevStateSet.insert(static_cast<TS2SState*>(rawPrevStates[i].get()));
      }

      for (int i = start; i < end; i++) {
        TS2SState* prevState = static_cast<TS2SState*>(rawPrevStates[i].get());
        if (prevState && validPrevStateSet.cend() == validPrevStateSet.find(prevState)) {
          prevState->hidden.clear();
        }
      }

Thanks for your detailed investigation, we will have a look and will try to repro and then come back to you =).

By the way, I think there's a bug here
https://github.com/facebookresearch/flashlight/blob/75c241bede3e3c1441078c2958338351955fe6e7/flashlight/app/asr/criterion/TransformerCriterion.cpp#L362
t will increase during decoding, so if t get larger than 999,  step will be 0, then the while loop will become infinite loop.

Hi @zjuturtle, thanks for spotting the bugs!

For the state.hidden bug, I think we can simply move https://github.com/facebookresearch/flashlight/blob/master/flashlight/app/asr/criterion/TransformerCriterion.cpp#L398-L403 out of this loop https://github.com/facebookresearch/flashlight/blob/master/flashlight/app/asr/criterion/TransformerCriterion.cpp#L363 and place it at the end.

For the step=0 bug, we should add something like step = max(step, 1).

Please let me know your thoughts!

For state.hidden bug, if we just move clear code out of the loop, I think we will lose the opportunity clear hidden in advance in the loop. Add some valid check code, it will not cost much extra time.
Add following code before loop
https://github.com/facebookresearch/flashlight/blob/94480042ce95f49eb6067253caf39a7e6c2e4d20/flashlight/app/asr/criterion/TransformerCriterion.cpp#L363
    // A simple bimap to recode valid clear hidden
    std::map<TS2SState*, int> validClearPtr2Index;
    std::map<int, TS2SState*> validClearIndex2Ptr;
    for (int index=0;index<rawPrevStates.size();index++){
      TS2SState* ptr = static_cast<TS2SState*>(rawPrevStates[index].get());
      auto iter = validClearPtr2Index.find(ptr);

      // Pointer exists in earlier rawPrevStates
      if (iter != validClearPtr2Index.end()){
        validClearIndex2Ptr.erase(iter->second);
        validClearPtr2Index.erase(iter);
      }
      validClearPtr2Index[ptr] = index;
      validClearIndex2Ptr[index] = ptr;
    }
    validClearPtr2Index.clear();

Then in the original clear code,
https://github.com/facebookresearch/flashlight/blob/94480042ce95f49eb6067253caf39a7e6c2e4d20/flashlight/app/asr/criterion/TransformerCriterion.cpp#L398-L403
modify to
      // clean the previous state which is not needed anymore
      // to prevent from OOM
      for (int i = start; i < end; i++) {
        auto iter = validClearIndex2Ptr.find(i);
        if (iter != validClearIndex2Ptr.end()) {
          TS2SState* prevState = static_cast<TS2SState*>(rawPrevStates[i].get());
          if (prevState) {
            prevState->hidden.clear();
          }
        }
      }

This fix is better than my last suggestion for speed

Let us discuss your solutions and then I will send the fix (or you could do PR) =) Thanks again for reporting the issue!

Unfortunately my computer is not allowed upload to GitHub. So PR is currently unavailable for me.

@zjuturtle
Thanks again for reporting/debugging/solving the issue. Here the fix https://github.com/facebookresearch/flashlight/pull/280, could you try it and tell if it works now for you?

The PR is merged, closing the issue for now.
Feel free to reopen if there is still any problem, and please report if everything works for you now @zjuturtle


Initial check-in of amppackager helm chart

Initial PR for creating click-to-deploy deployer for amppackager.
This includes the helm chart for amppackager: https://github.com/ampproject/amppackager/tree/master/deploy/gcloud

Thanks for the suggestions. Will add the application manifest and schema.yaml in subsequent PRs, this PR is just dedicated to the helm chart for now.


Add a checkbox to link Topmate profile

PS: By default, it should be checked

are you saying you want a checkbox for each of the links


fix for cp4na issues 277 and 258

Added support for multiple value files and --setfiles in helm client.
Kubedriver with this change can support multiple value files which are passed as an array of strings to the helmclient.
The change is backwards compatible with existing assemblies.
Example -
compose:
  - name: Create
    deploy:
      - helm:
          chart: apache-7.3.15.tgz
          name: r{{ system_properties.resource_id_label }}
          namespace: default
          values: [ "apache-helm-chart-values.yaml", "apache-helm-chart-values2.yaml" ]

For backwards compatibility the code also supports value files to be set to a single string like it is being used as.
compose:
  - name: Create
    deploy:
      - helm:
          chart: apache-7.3.15.tgz
          name: r{{ system_properties.resource_id_label }}
          namespace: default
          values: "apache-helm-chart-values.yaml"

Also added is a support for passing --set-file (refer Helm install) as part of the deploy configuration by setting a parameter setfiles to a dictionary mapping keys to file names
Example -
compose:
  - name: Create
    deploy:
      - helm:
          chart: apache-7.3.15.tgz
          name: r{{ system_properties.resource_id_label }}
          namespace: default
          values: [ "apache-helm-chart-values.yaml", "apache-helm-chart-values.yaml" ]
          setfiles:
               service.nodePorts.https : "httpsPort.data"

For a resource tree which looks like -
helm/
├── apache-7.3.15.tgz
├── apache-helm-chart-values2.yaml
├── apache-helm-chart-values.yaml
└── httpsPort.data

Tested with TNC v2.1 and kubedriver built with this change on the scenarios shown above.

Also, could we add documentation for setfiles and update the docs for values at https://github.com/IBM/kubernetes-driver/blob/master/docs/reference/kegd/helm.md#arguments

Fixes #47

@dvaccarosenna updated the commit with requested changes.
Simple unit tests for install and upgrade
Updated docs.
And applied the small fixes.


Firmware for libsigrok compatibility

Hi Benjamin,
Someone created this PR for libsigrok: https://github.com/sigrokproject/libsigrok/pull/239
Do you happen to know anything about it? I wonder if someone wrote a firmware for the WCH-CH569 that emulates the fx2lafw protocol and I figured you'd be the most likely person to know since the PR author isn't responding.
Either way, do you know of someone writing a firmware that produces a logic analyzer functionality? Might be worth writing a compatible driver for libsigrok if so.

Unfortunately I'm not aware of that PR and I do not have any contact with the author of that PR.
I'm not aware of anyone writing a firmware the produce a logic analyzer functionality for WCH CH569 to be clear there was pratically no any feedback / PR for hydrausb3_fw (except the amazing work done by https://github.com/kauwua on the lib and hydradancer_fw...

Thanks for your feedback, unfortunately I have no option but to cancel the PR then. I do really wonder how much work it would be to create a firmware with this functionality and whether we'd have the same issues here as we have with the FX3. The biggest problem was the renumeration where the FX3 showed up on a different USB bus with a whole different VID/PID after the firmware was uploaded in USB2 mode. This issue would be the same here, right?

Thanks for your feedback, unfortunately I have no option but to cancel the PR then. I do really wonder how much work it would be to create a firmware with this functionality and whether we'd have the same issues here as we have with the FX3. The biggest problem was the renumeration where the FX3 showed up on a different USB bus with a whole different VID/PID after the firmware was uploaded in USB2 mode. This issue would be the same here, right?

I'm not aware of that issue with FX3 as I have mainly worked on WCH CH569 which probably will do not have such limitation especially as kauwua has wrote a full low level USB2 / USB3 stack maybe to be cross checked with kauwua
The main issue anyway to do a good Logic Analyzer with WCH C569 is there is no DMA on GPIO so it will be pretty slow to read the GPIO with the CPU fast enough to have something like > 10MSPS
The only other way to sample the GPIO quickly (up to 120MHz in fact linked to MCU speed) will be to use HSPI but it have mandatory framing with header... so not usable for LA ...

Thanks, that is important to know and a bummer indeed. With DMA for the I/O, this could've been a serious FX3 competitor.


Duplicate icons on File history and line history

GitLens Version: 11.1.3

Git Version: git version 2.19.2.windows.1

VSCode Version: 1.52.1
ea3859d4ba2f3e577a159bc91e3074c5d85c0523
x64

OS Version: Windows 10

Steps to Reproduce:

Select any file and go to File History or Line History, each icon is repeated twice.
Here is the screenshot:

Do you possibly have both the stable and insiders editions of GitLens installed?

Do you possibly have both the stable and insiders editions of GitLens installed?

yeah.. you're right.. I've installed insiders edition too and it resulted in duplicate icons.
Actually I was using remote explorer extension on VSCode and opened another remote instance of VSCode. There I've installed GitLens insider edition. So now in my local VSCode instance it shows both stable and insider editions resulting in duplicate icons.

yeah.. you're right.. I've installed insiders edition too and it resulted in duplicate icons.
Actually I was using remote explorer extension on VSCode and opened another remote instance of VSCode. There I've installed GitLens insider edition. So now in my local VSCode instance it shows both stable and insider editions resulting in duplicate icons.


Bug with error_on_unknown_keys triggers syntax_error

It appears there is a bug when the JSON string to be parsed has an unknown key, even when error_on_unknown_keys=false.
A syntax_error will get triggered whenever the unknown key is prefixed by one of the valid key names.
// test.cpp
#include <iostream>
#include "glaze/glaze.hpp"

struct my_struct
{
    int port;
};

int main() {
    // The JSON string below has an unknown key "portmanteau"
    // This should be fine because we parse with .error_on_unknown_keys
    // But instead of getting an unknown_key error we get a syntax_error occurring
    // on the letter "m" in "portmanteau"
    // I believe this occurs whenever an unknown key contains a valid key name from my_struct as a prefix
    // For example, parsing will be successful if the "portmanteau" is replaced with any word that does not
    // begin with "port"
    std::string buffer = R"({"portmanteau":14,"port":17})";
    my_struct s{};

    auto err = glz::read<glz::opts{.format=glz::json,.error_on_unknown_keys=false}>(s, buffer);
    if (err)
    {
        std::string errMsg = glz::format_error(err, buffer);
        std::cout << errMsg << "\n";
    }

    std::cout << s.port << std::endl;
}

Output:
g++ -std=c++20 -I./glaze/include test.cpp; ./a.out
1:7: syntax_error
   {"portmanteau":14,"port":17}
         ^

0

Thanks so much for pointing out this bug. It will be a high priority to fix, but I will be preoccupied for the next few weeks before I can get to it. L

Thanks again for pointing out this bug, it has been fixed with the merge of #737.


Increase the width of the new node modal

I made the "new node" modal dialog a little wider. It is now wide enough that scrolling is no longer
necessary when adding attributes to new nodes.
It does not seem to have messed up any of the other modals and seems fine in FF/Chrome.

Thanks. Added those changes to the right position (scss). Now only affecting the graph viewer modals and not every modal.
https://github.com/arangodb/arangodb/commit/e2a9198e6d03a432dd747bd6d9ceb309cc8d3d0b


Add option user-name=username

This makes it trivial to integrate into any OS's init system.
GoAccess should not run as root when producing real time reporting.

This is great. Thanks for submitting this! Merged.


Scalable Broadcasting.

So with the boadcaster using Chrome, it seems all combinations of viewers using either Firefox or  Chrome browser works for me.
However, if the broadcaster uses Firefox, there are issues with Chrome in the mix of viewers.  Like if the broadcaster uses Firefox and all viewers are using Chrome, the third viewer to join has a black screen for video?
Has anyone else seen this?   Is there a fix?

So I've been testing and fixing stuff as I go along to see if we can get this working good enough for smaller stream groups. From testing there are a few features that need to be addressed: Adding connection quality metrics and super noding.
Super noding would allow a healthy client in the chain to server up more than one additional client (making it into more of a tree structure).
The metrics for connection quality would be used to push people with poor connections towards the end of chain, or closer towards being leaf's in the stream tree so you don't have them uploading/resharing choppy video.
I have fixed/added some features to my current version I've been working on. Like higher quality audio, the audio doubling issue (there is another background media element with no controls that plays by default in the demo)
@rayj00 have you still been looking at using/playing with this?

So I've been testing and fixing stuff as I go along to see if we can get this working good enough for smaller stream groups. From testing there are a few features that need to be addressed: Adding connection quality metrics and super noding.
Super noding would allow a healthy client in the chain to server up more than one additional client (making it into more of a tree structure).
The metrics for connection quality would be used to push people with poor connections towards the end of chain, or closer towards being leaf's in the stream tree so you don't have them uploading/resharing choppy video.
I have fixed/added some features to my current version I've been working on. Like higher quality audio, the audio doubling issue (there is another background media element with no controls that plays by default in the demo)
@rayj00 have you still been looking at using/playing with this?

First, I am not a Javascript programmer. My "expertise" is in an ancient assembly language used on telephone switches. :)
That said, I have been "playing" around with RTCMulticonnection/Scalable Broadcasting code on and off, mostly off, for a couple years. I still don't have a good idea of how it all works.  But I got it working to some extent. I have added a simple authentication.  One for broadcaster and one for viewers.  My issue is testing with many viewers.  Right now I use multiple tabs in the browser,  but that slows connections down considerably if I get to 10 or so tabs (viewers).  What has been your experience with it?

Also incorporated a database to store roomID rather than using the local storage.  I also got recording the broadcast working.

All that said,  I'm now playing with conferencing.  Trying to get switch camera to work.

Good to know there are some people around with some technical chops like Assembler. I've a bit of a different background, lots of C/++/# python, js and I've dabbled in Assembler to debug things.
So I've been testing with a very small group of people I know. I've noticed that there seems to be some logic bugs in how the room management works. You can easily get into a situation where the room is hung open and you cannot start a stream until everyone has cleared that room. I've been looking at stringing this together with some more socket.io stuff to make a bare bones federated streaming site/system that can be ran with almost no infrastructure. But I've also done a similar mod to it, that allows authenticated streaming, so a user can start a stream, then other's can watch it.

I've been toying with this idea for almost two years and came back to this library after I was able to set it up and getting it working.
From talking to some of the people I know that want to use this, they would like to be able to record or clip things from their stream, they might also want to do something similar to conferencing.
Also to give you and idea, I've built some functional parts of what Mauz has done by hand, it's taken about 3 months of pouring over the client side to get a good idea of what is going on. The way that Webrtc works is just complicated enough to make it complicated to manage all of the moving parts, you could probably get all of this down to 2-4k lines of code from the almost 10k that it is counting the server side.
So are you trying to change from desktop capture, to camera, to tab while still broadcasting/sharing? Technically you can use a page element for a media source if I recall correctly from when I started playing with idea originally.

For conferencing, I'm considering the mobile user and his ability to change cameras.   I do have two USB cameras connected to my PC to emulate a cell phone...kinda. :)  I think I have the stream swapped, but  it's showing no video.  No errors in the console log either.  I truck on....

If you want to take a look at the monstrosity I published it as fedwave in my repo's. There is also a public instance of it running for testing.

I have not taken the time to learn git/github.   I do everything on my PC.

Yes.  Actually I have a Linode VPS.  I am using LXD to containerize and letsencrypt for SSL.  I have multiple containers, like one for scalable broadcast, one for conference calling, one for a plain website, etc.  I still would like to have maybe 50 viewers for my scalable broadcasting testing.

Nice, I'm in the process of getting a hold of people to do more testing, the big weakness I've see is getting slow users in the viewer chain. I'll probably be able to wrangle about 30 people to do testing.


feature request: React-router 4 and react-router-redux 5 support

Is there any plan to support those updated version of those two packages in the near future?

Looks like a dead project...

@zhenyulin @Geczy There's a non-public branch with defaults for react-router v4 being tested and released soon. I'll be making an update in the near future as well.
It's worth noting that you can use the mapper object to add support for any router in the meantime...there's really nothing special about how we treat routers events (see "2. Optional, Access Third Party Redux Libraries" under Installation.

Any news on that?

@bertrandk Just checking in to see if that branch has been made public? Is there a recommended way to track routes if one is only using react-router 4 without react-router-redux?


There is a typo under "FHIR service" section

[Enter feedback here]
There is a typo under "FHIR service" section, the text of "For more information about MedTech" should be "For more information about FHIR" instead.
FHIR service

Azure Health Data Services includes support for FHIR service. The FHIR service enables rapid exchange of health data using the Fast Healthcare Interoperability Resources (FHIR®) data standard. For more information about MedTech, see Overview of FHIR.

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

ID: 84d91547-07fd-7147-8319-9457c9580aca
Version Independent ID: 4009af7a-bba4-4dd3-592a-c0d6d323d12d
Content: What is Azure Health Data Services?
Content Source: articles/healthcare-apis/healthcare-apis-overview.md
Service: healthcare-apis
GitHub Login: @mikaelweave
Microsoft Alias: mikaelw

@chiachang100
Thanks for your feedback! We will investigate and update as appropriate.

Hi @chiachang100 ,
Thanks for sharing this observation.
Changes have been made:


enhancement:improve messaging when adjusting upper and lower peer limits

PR Description
improve messaging when adjusting upper and lower peer limits
Fixed Issue(s)
https://github.com/ConsenSys/teku/issues/6992
Documentation

[ ] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[ ] I thought about adding a changelog entry, and added one if I deemed necessary.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Doc: Add a new Extention Feature(extention11.kt)

Description:

add a new extention11.kt

PR Instruction:

Issue will be assigned on a first come first serve basis.
If you are thinking to make a pr then at first be sure to get assigned.
before pushing your PR please make sure to fetch upstream.
your pr should be pushed on the develop branch only.
try to complete your assigned task within 3 to 5 days and if you need more time then please let me know. As the program has limited time only so please try to complete as fast as you can.
With your PR try to send a screenshot or screen recording of your work

I want to enroll


feat: add preview in dashboard using iframe

Changes:

♻ the useResumePreview hook is currently unused, because huge loading times
🚀 I refactored it to use the same functionality as [here](https://github.com/AmruthPillai/Reactive-

Resume/blob/main/apps/client/src/pages/public/page.tsx#L81-L86)

using iframe to render it and set resume in the useEffect((), [frameRef])

💪 It is fast and from the exploratory tests I've done locally, we can rely on it, as well.

Discussion in issue: https://github.com/AmruthPillai/Reactive-Resume/issues/2005#issuecomment-2319381861

@AmruthPillai please review and merge if it's okay. Thank you!

Though in case the page has a lot of content, the scrollbars will be visible.. 👇

This can be resolved with setting the iframe.scrolling to "no".
<iframe
  ref={frameRef}
  title={resume.title}
  src="/artboard/preview"
  scrolling="no"
  style={{
    height: "100%",
    width: "100%",
    zoom: 0.35,
  }}
/>

There is only one deprecation warning, related to iframe.scrolling.
Didn't find a viable alternative, but this one is working as expected.

the thing is that we need to set overflow: "hidden" in the iframe's document's HTML

https://github.com/AmruthPillai/Reactive-Resume/blob/main/apps/artboard/index.html#L2
or in the HTML's body: https://github.com/AmruthPillai/Reactive-Resume/blob/main/apps/artboard/index.html#L35

this would mean that we'd need to make this part of static HTML to dynamic code (react)..

⚠ 'scrolling' is deprecated.ts(6385)

Review please 🙏
Thank you! 😄
cc. @AmruthPillai

Review please 🙏 Thank you! 😄
cc. @AmruthPillai

@AmruthPillai What do you think?

Review please 🙏 Thank you! 😄
cc. @AmruthPillai

@AmruthPillai What do you think?

Small reminder 😄 Thank you!
cc. @AmruthPillai

@AmruthPillai Please review 😄

Hello @rinczefi-user !
I have tested this in my setup but the preview is zoomed in.

I have done all your changes as shown in your commits.

It seems that this happens on Firefox. On Google chrome it shows ok.

It seems that this happens on Firefox. On Google chrome it shows ok.

Hmm.. I see that it's supported only in the new Firefox versions.. 🤔

link: https://developer.mozilla.org/en-US/docs/Web/CSS/zoom#browser_compatibility 👇

I'll try this suggestion 👇 later today, thanks!
https://stackoverflow.com/questions/4049342/how-can-i-zoom-an-html-element-in-firefox-and-opera


web中报错

Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.0.0, on Microsoft Windows [Version 10.0.17134.984], locale zh-CN)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[√] Chrome - develop for the web
[√] Android Studio (version 4.1.0)
[√] VS Code (version 1.53.2)
[√] Connected device (2 available)

in dio 3.x not install 4.x


Fix newsflow

This is an attempt to fix #6085
I absolutely don't know if this approach is the right one.
In any case, a lot of styles are missing: here's what I've achieved so far:

Actually, I think we'd be more interested in removing it entirely – we don't actively maintain Newsflow anymore and the idea is that more content will simply get pushed into the Welcome screen that appears on the Dashboard home, rather than bringing back the newsflow overlay.

:+1: (I've always disabled it in all my instances :wink:)


No Audio Redrix

There is only dummy output for output devices in gnome and no input devices found in gnome setting on arch linux.
debug-logs.tar.gz
Boardname
redrix
Any ideas on how I might go about fixing this?

Possibly related to #70

I tried downgrading the kernel by compiling and successfully booting linux-6.1.78(it was the closest source I could find to what they mentioned in #70 ) and rerunning setup-audio but still no luck. Here are my debug new logs:
debug-logs.tar.gz

Oh right, alderlake on older kernels needs a modprobe config, see #100

Or #95 for instructions

I have no audio on my redrix too. I remember when I first got redrix @WeirdTreeThing did update the script and the sound and mic was working no problem. I just installed Fedora 39 and I'm running dnf upgrade. I'll see if that fixes the audio. Also @pacmacm how is your experience on redrix? Are you using RW_LEGACY or full rom?

Actually I think my issue might be from using an ipu6 kernel. I will retry with the latest fedora kernel and comment the result.

It still doesn't work. Info:
rajas@Redrix:~$ neofetch
             .',;::::;,'.                rajas@Redrix
         .';:cccccccccccc:;,.            ------------
      .;cccccccccccccccccccccc;.         OS: Fedora Linux 39 (KDE Plasma) x86_64
    .:cccccccccccccccccccccccccc:.       Host: Redrix rev3
  .;ccccccccccccc;.:dddl:.;ccccccc;.     Kernel: 6.7.7-200.fc39.x86_64
 .:ccccccccccccc;OWMKOOXMWd;ccccccc:.    Uptime: 48 secs
.:ccccccccccccc;KMMc;cc;xMMc:ccccccc:.          GPU: Intel Alder Lake-UP3 GT2 [Iris Xe Graphics]
:cccccccccccccccccccccccccccc:'.         Memory: 3317MiB / 31934MiB
.:cccccccccccccccccccccc:;,..
  '::cccccccccccccc::;,.
rajas@Redrix:~$ cat /etc/modprobe.d/sof.conf
options snd-intel-dspcfg dsp_driver=3

I am very happy with redrix overall, I've been using it without the camera, mic or speakers working and still tend to use it more than my pink pixelbook go. I am using the full rom firmware. Earlier when I opened to issue I tried installing fedora just out of curiosity but still couldn't get the audio script to fix it. I looked into compiling a kernel while on Fedora but I found it was going to be a lot more difficult and didn't see any benefits to staying on fedora. I actually had a hard time installing the packages I needed from their repo so I moved back to arch and have been using it ever since.
Just now I did a system upgrade, cloned the repo and Made a file /etc/modprobe.d/sof.conf with the line options snd-intel-dspcfg dsp_driver=3 Like Weirdtreething suggested, ran the script and rebooted
[riley@archlinux ~]$ ./setup-audio
WARNING: You may run into audio issues, even after running this script. Please report any issues on github.
Installing SOF
Installing UCM configuration
Cloning into '/tmp/chromebook-ucm-conf'...
remote: Enumerating objects: 1245, done.
remote: Counting objects: 100% (269/269), done.
remote: Compressing objects: 100% (183/183), done.
remote: Total 1245 (delta 141), reused 183 (delta 66), pack-reused 976
Receiving objects: 100% (1245/1245), 213.60 KiB | 354.00 KiB/s, done.
Resolving deltas: 100% (628/628), done.
Increasing alsa headroom (fixes instability)
Audio installed successfully! Reboot to finish setup.
after rebooting I am still having the same problem:
[riley@archlinux ~]$ wpctl status
PipeWire 'pipewire-0' [1.0.3, riley@archlinux, cookie:4209167127]
└─ Clients:
31. pipewire                            [1.0.3, riley@archlinux, pid:1778]
33. WirePlumber                         [1.0.3, riley@archlinux, pid:1777]
34. WirePlumber [export]                [1.0.3, riley@archlinux, pid:1777]
46. Firefox                             [1.0.3, riley@archlinux, pid:1907]
52. wpctl                               [1.0.3, riley@archlinux, pid:2629]
Audio
├─ Devices:
│      45. Alder Lake PCH-P High Definition Audio Controller [alsa]
│
├─ Sinks:
│  *   32. Dummy Output                        [vol: 1.00]
│
├─ Sink endpoints:
│
├─ Sources:
│
├─ Source endpoints:
│
└─ Streams:
47. Firefox
49. output_FL       > Dummy Output:playback_FL	[active]
51. output_FR       > Dummy Output:playback_FR	[active]
Video
├─ Devices:
│
├─ Sinks:
│
├─ Sink endpoints:
│
├─ Sources:
│
├─ Source endpoints:
│
└─ Streams:
Settings
└─ Default Configured Node Names:
debug-logs.tar.gz

@pacmacm how is suspend and battery usage while suspended on full rom? When I tried full rom suspend apparently didn't work at all in Windows 11 and it was inconsistent in Linux. Sometimes it worked, and other times it didn't, and there wasn't any way of telling because either way the power light would be white and unblinking.

@pacmacm do you also have issues with HDMI through Thunderbolt 4 ports?

@ChocolateLoverRaj Suspend works reasonably well. Id say that battery life is preserved as well as any other laptop ive tested. The only minor inconvinience is that when I open the lid of the laptop again the majority of the time I have to either hit a key on the keyboard or tap the trackpad to get it to come back out of suspend.
The hdmi port works without issue on wayland. I have not yet tested hdmi through the thunderbolt 4 port.

When I first got installed Linux on a Redrix, @WeirdTreeThing got audio and mic to work without issues. So today, I went back to a commit from around the last time I used redrix, and audio and mic worked today with the script and configs from November. I created a branch as a temporary alternative for Redrix until this issue is fixed - https://github.com/ChocolateLoverRaj/chromebook-linux-audio/tree/redrix-temp. @pacmacm see if it works for you.

That worked! Thank you very much. @ChocolateLoverRaj

what exactly is the issue here

I removed to modprobe config file and rebooted to see if it would make a difference and it doesn't seem to have removed any functionality. It seems as if something has changed since this older version of the script that has broken compatibility with redrix

send logs with the modprobe config

Logs when audio is working or not working? I guess I will fresh install fedora on an external SSD, run your latest script, and then send logs.

not working

send logs with the modprobe config

@WeirdTreeThing what command should I run for logs besides dmesg?

send logs with the modprobe config

I'm not sure what you mean by "with the modprobe config". Here are the logs after running the script without doing anything else, on a fresh install of Fedora 39 (KDE).
debug-logs.tar.gz

debug-logs.tar.gz
Here are my current logs running Arch

Rajas update your kernel and send logs again

pacmacm is the issue that audio never actually outputs?

i have located the issue, it ended up being a skill issue

So I don't have to update kernel and test?

You still do

You need at least 6.6 for my patch that uses the sof driver by default on ADL/RPL. Before that, it would use the hdaudio driver which won't work on chromebooks. Alternatively you can use the modprobe config that I mentioned before to force the sof driver.

You still do

Should I git pull and rerun the script after updating the kernel?

Just update your system until I tell you that I fixed the issue on my side

I have a new install just for testing. I already have audio working with the latest kernel on main install. Should I still update the test install and send logs? And if so, should I re run the script before sending logs?

Update your test system, wait for me to push my changes, test them and if it's still broken, send logs.

K, let me know when your changes are pushed.

changes have been pushed to the ucm repo

I tested it and the audio works now

@WeirdTreeThing u can close this issue now

How about I wait for OP to comment, it is their issue after all.

This all sounds good to me.  I'll do whatever I can to help solve this new suspend issue. Thanks everybody.


Web Stories Carousel Widget Not Clickable on AMP Pages with Legacy Theme

Bug Description
The Web Stories Carousel Widget is not clickable on an AMP page with the AMP legacy theme. The issue can be reproduced on AMP Legacy theme (with “Astra” theme activated). Console errors show amp-bind script is missing. Thanks to @milindmore22 for his findings that the amp-lightbox markup is not getting added on Legacy theme for some reason. His alternative considered is to add this code snippet add_action( 'amp_post_template_footer', [ $this, 'render_stories_lightbox' ] ); below this line
Expected Behaviour
For the Web Stories to open in a lightbox.
Steps to Reproduce

Enable AMP plugin and its Reader Mode
Activate Astra Theme
Add Web Stories Carousel Widget
Visit site in AMP version
Click on Story
Note behavior and console errors

Screenshots
https://user-images.githubusercontent.com/66372350/135308359-fe043df0-4d81-4082-a770-887b28db97e1.mov

Additional Context
Original report: https://wordpress.org/support/topic/web-stories-not-clickable-on-amp-pages/
Stories not clickable at https://www.thecurrentindia.com/govt/encumbrance-certificate-in-karnataka/?amp=1 , but clickable at https://www.thecurrentindia.com/govt/encumbrance-certificate-in-karnataka/ .

Plugin Version: 1.11.0
WordPress Version:
Operating System: macOS
Browser: Chrome

Yep, hooking into amp_post_template_footer would indeed be the correct solution here.

Verified in QA


Display is not initialized on i5 3gen - gtx960

Hi I wanted to install newest version of ChimeraOS, unfortunately it has some problem with (most likely) display initialization.
My setup:
i5-3450
GTX 960
I tried to do some troubleshooting but:
Web interface works, but since i have no luck with display init i cannot pass "Please enter the password shown on your display to continue". So i also cannot go straight into ssh because i cant add new ssh key.
I tried terminal directly on machine but it constantly switches me back to main screen (which has only blinking underscore) so its really hard to use it because i constantly need to switch back into TTY)
What i can do to grab some logs?

I used liveUSB to access files. There's messege that vulkan cannot be initialized. With this newest gamepadUI we need to stick with vulkan or there's way to go with openGL?

wlserver: [backend/headless/backend.c:68] Creating headless backend wlserver: [libseat] [libseat/backend/seatd.c:64] Could not connect to socket /run/seatd.sock: No such file or directory wlserver: [libseat] [libseat/libseat.c:76] Backend 'seatd' failed to open seat, skipping wlserver: [libseat] [libseat/libseat.c:73] Seat opened with backend 'logind' wlserver: [backend/session/session.c:109] Successfully loaded libseat session ATTENTION: default value of option vk_xwayland_wait_ready overridden by environment. ATTENTION: default value of option vk_xwayland_wait_ready overridden by environment. ATTENTION: default value of option vk_xwayland_wait_ready overridden by environment. vulkan: selecting physical device 'NVIDIA GeForce GTX 960': queue family 0 vulkan: physical device supports DRM format modifiers vulkan: vkCreateDevice failed (VkResult: -1000174001) Failed to initialize Vulkan

gamescope.txt
This is log mentioned above.

@alkazar Did we ever confirm if 900 series Nvidia worked? The last I tested with v37/v38 the GTX 970 had the same vulkan issue.

According to https://chimeraos.org/download/ that nvidia card is too old. Too bad, I'd also like to use a 960 for this.


SOURCE_DATE_EPOCH not working as expected

In https://github.com/tc39/proposal-pattern-matching, locally npm run build is now deterministic. It won't update the date until spec.emu is modified. However, PRs that do not update spec.emu - and builds on main that do not update it - still generate a date diff.
I suspect there's timezone issues here, and it's something about the value i'm passing to the env var, or, something about the modification dates locally vs in github actions, but I'm hoping the ultimate goal of a stable spec date can be achieved somehow :-)

Is there a way I can see what the diff is?

Whenever I've checked it, it's just been "the date" - if you mean which specific date, one way would be to add the logging to CI here: https://github.com/tc39/proposal-pattern-matching/runs/7272048687?check_suite_focus=true but another would be for ecmarkup to print out some debug info when SOURCE_DATE_EPOCH is passed.

I mean that I want to see what the diff is between one of the files locally and in CI, yes.

https://github.com/tc39/proposal-pattern-matching/runs/7273193396?check_suite_focus=true shows the diff is from July 9th to July 10th.

I'm pretty sure that's because you're deriving the timestamp from the file modification time on disk, which is not preserved in git. You can use the commit timestmap instead, if you want, but you expressed reluctance in the original thread.
I'm not sure what timestamp you're actually hoping to use, here; ecmarkup will let you specify whatever timestamp you want, and I am pretty sure that if you pass it the same timestamp it will give you the same out (everything is treated as UTC), but you do have to figure out some way to give it the same timestamp across builds for the builds to be reproducible.

aha, i didn't realize git didn't preserve file modification times. i'll look into that.

(either way, it still seems like it'd be useful to include the complexity in ecmarkup, rather than having to repeat it in every proposal repo)

I feel exactly the opposite: because it's a lot of complexity unrelated to the core job of transforming documents, it should not be included in ecmarkup.
(I dunno about "including it in every proposal repo"; I think you're the only delegate who has expressed this preference.)

I'm not the only one; every proposal repo based on the template auto-generates the HTML on every push, so every repo is affected by this.
Since ecmarkup outputs a date, it does seem related to the document it generates.

It is true that the date in the HTML will be set based on when the deploy job runs, but I don't think anyone else has said they consider that to be a problem. Also it's usually just going to be a short shell command (git log -1 --format=%at or =%ct), probably?), since CI runs on linux and unlike ecmarkup does not need to handle being run on non-linux platforms or outside of git repos; it's not like it's a whole script to copy around.
I agree that having a mechanism for setting the date to an explicit timestamp is in scope, which is why I added it, but I don't consider that the mechanism for deriving such a timestamp is in scope for this project.