[$1000] mWeb - Videos starts playing instead of being downloaded on production

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/16492
Action Performed:

Open https://staging.new.expensify.com/
and login
Send a video in any chat
Open same chat https://new.expensify.com/
Click download icon to download the video

Expected Result:
Video should be downloaded to device
Actual Result:
Video starts playing in another tab
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/228405746-d061e821-e19b-45fb-ac6c-56eee11ffd7f.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011fb594eef7b19cdb
Upwork Job ID:<PHONE_NUMBER>253399040
Last Price Increase: 2023-04-04

Triggered auto assignment to @adelekennedy (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@adelekennedy Huh... This is 4 days overdue. Who can take care of this?

Current assignee @adelekennedy is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak (External)

Triggered auto assignment to @tgolen (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

moved to external

This is still waiting for proposals or more confirmation that it cannot be reproduced. @kbecciv Are you able to verify that it is still reproducible?

@tgolen Checking with team, update you shortly if issue is still reproductible.

Still looking into if the issue is reproducible.

Issue is not reproductible.
https://user-images.githubusercontent.com/93399543/231000977-611b1447-dc83-4f78-a598-ef8a4418b772.mp4


5.2 compile fails, and Market compatability

SubsystemBrowserUtils.cpp
// flags no longer need to be added in 5.1.1, throws warning, so the Engine just refuses to enable the plugin.
ADD_FLAG(CLASS_NoExport);
ADD_FLAG(CLASS_CustomConstructor);
Market lists plugin as compatible up to 5.0 so is out of step.

Fixed in 1.2.1


Bio Survey Summary - Table Order

Please fix the order of Table 1 and Table 2 in the biospecimen Survey Summary Statistics.
Requestor(s): Bio Team
Table or Figure title: Table 2: Symptoms in the 24 hours Prior to Sample Donation; Table 1: Biological Sex at Birth
Metrics to be updated or created: Update
Variables needed (include variable name and Concept ID): N/A
Specify numerator and denominator by variable name where applicable: N/A
Mock-up table if applicable: N/A
Notes: N/A
Approved by: Domonique

Issue completed in #40


Can I convert to Windows authentication with Identity?

hi @MohammadYounes. I have one intranet web form application with Identity to authenticate/authorize users in Visual Studio 2015. Now I tried to convert it from "Individual User Account" to Windows authentication to use company AD windows to automatically login if a user click a url.
Can I use the way you mentioned for user DabPFW in Question #52 https://github.com/MohammadYounes/OWIN-MixedAuth/issues/52). Highly appreciate your guidelines.

@zbx888 This adds windows authentication as an external provider, you need to keep "Individual User Account" then follow the steps to add auto windows login.
Thanks.

Yes, But if all users are under AD, why don't just use windows authentication without using individual user accounts ?

hi @MohammadYounes.  Good morning!
In this intranet application, Not all users are the application users and also we want to use Roles saved in the database table AspNetUserRole to authorize the access.
I followed your steps you recommended

Startup.Auth.cs: Change LoginPath to point to Windows Login Action
WindowsLogin.aspx.cs: On PageLoad request the windows challenge.
401.html: Add 401.html error page to your app root that will redirect to the normal login page
Web.config: Use 401.html to respond to 401 HTTP error (in case of login failure)

Also I changed the Web.Config and project properties to get user's AD UserName by User.Identity.Name

<authentication mode="Windows" />

Project Properties
Anonymouse Authentication =false
Windows Authentication = true

When I run the application, I got the error  "404 - File or directory not found."  and the returnURL over the character limit by repeating again and again.
http://localhost:9797/Account/WindowsLogin.aspx?ReturnUrl=%2FAccount%2FWindowsLogin.aspx%3FReturnUrl%3D%252FAccount%252FWindowsLogin.aspx%253FReturnUrl%253D%25252FAccount%25252FWindowsLogin.aspx%25253FReturnUrl%25253D%2525252FAccount%

@zbx888 Mixed authentication uses cookies and relies on Windows Authentication for the MixdAuth endpoint only. You need to enable Anonymous Authentication to avoid having the redirect loop (see this comment).

Even I enabled Anonymous Authentication, still get error 404 - File or directory not found.
The resource you are looking for might have been removed, had its name changed, or is temporarily unavailable.

Which file? Can you share the dev tools network log?

@MohammadYounes  Really appreciate your help!!!!
The included is network log file by F12 Developer Tool. Thank you!
NetworkData.zip

@MohammadYounes Even I enabled the Anonymous Authentication  the redirect URL still keeps loop.

@zbx888 The log is insufficient, as it shows only 1 entry.
Can you join me here https://gitter.im/MohammadYounes/OWIN-MixedAuth ?

@MohammadYounes  Thank you so much for your invitation but our security policy blocks instant messaging unless special approval.
I will try to build a new one again for testing on my developer computer. Before that please help me confirm that I had to follow the steps mentioned in Readme to configure.

On IIS set Authentication-Windows to Read/Write
On IIS Express set windowsAuthentication = "Allow" in ApplicationHost file
Reference MohammadYounes.Owin.Security.MixedAuth.dll
Register MixedAuth in Global.asax
Use MixedAuth in Startup.Auth.cs
Enable Windows authentication in Web.config

Then
1.Startup.Auth.cs: Change LoginPath to point to Windows Login Action
2.WindowsLogin.aspx.cs: On PageLoad request the windows challenge.
3.401.html: Add 401.html error page to your app root that will redirect to the normal login page
4.Web.config: Use 401.html to respond to 401 HTTP error (in case of login failure)
BEST REGARDS!

@zbx888 Step 4 - Windows Authentication should be enabled for the MixedAuth endpoint only:

<location path="MixedAuth">
  <system.webServer>
    <security>
      <authentication>
        <windowsAuthentication enabled="true" />
      </authentication>
    </security>
  </system.webServer>
</location>

hi @MohammadYounes Thank you so much for your help. Finally my application basically is working now.
I still have two issues so hopefully you can give me further help

I set ExpireTimeSpan = TimeSpan.FromMinutes(30) in Startup.Auth.cs
var cookieOptions = new CookieAuthenticationOptions
{
AuthenticationType = DefaultAuthenticationTypes.ApplicationCookie,
LoginPath = new PathString("/Account/WindowsLogin.aspx"),
ExpireTimeSpan = TimeSpan.FromMinutes(30),
Provider = new CookieAuthenticationProvider
{
OnValidateIdentity = SecurityStampValidator.OnValidateIdentity<ApplicationUserManager, ApplicationUser>(
validateInterval: TimeSpan.FromMinutes(20),
regenerateIdentity: (manager, user) => user.GenerateUserIdentityAsync(manager))
}
};
But I tested it never expires. How to fix it?

I try to assign a Role from database for the user by HttpContext.Current.User = new System.Security.Principal.GenericPrincipal(id, myRoles). Which is the best event to assign it? Application_PostAuthenticateRequest or other event?

@MohammadYounes Sorry bother you again and really appreciated your help.
Due to Application pool recycle I use StateServer on same server to keep session data. The current issue is if I keep the application running for several hours or overnight, sometimes got a poping up window for security credential and sometimes not but clicking a link to another page or button to load data got blank page. I checked IIS log file and found that is 401 error.
How to fix this issue?  Thank you so much!

@zbx888 This means your cookie expired due to user inactivity, and clicking a link to another page should redirect you to the auto login end point. What is this blank page you are referring to ?

@MohammadYounes Thank you so much!
YES, the blank page is I refer to.  Because of blank page I only click Back button on the IE browser to auto login.
In IIS log I found 401 error when cookie expired. Why did it not redirect to auto log as set in webconfig:

If I can check cookie expiration in the Master page, does it work? How do I check the cookie expire in this case.

@MohammadYounes Thank you so much!
YES, the blank page is I refer to. Because of blank page I only click Back button on the IE browser to auto login.
In IIS log I found 401 error when cookie expired. Why did it not redirect to auto log as set in webconfig:
<system.webServer>

If I can check cookie expiration in the Master page, does it work? How do I check the cookie expire in this case.

@MohammadYounes Thank you so much.
YES. The blank refers to what is set.
I found the 401 error in IIS log when I click the link. How can following setting not work to auto login?

How do I check cookie expiration in this case?

@MohammadYounes Thank you so much.
YES. The blank refers to what is set.
I found the 401 error in IIS log when I click the link. How can following setting httpErrors errorMode="Custom" not work to auto login?
httpErrors errorMode="Custom"
remove statusCode="401"
error statusCode="401" path="401.html" responseMode="File"

How do I check cookie expiration in this case?

@MohammadYounes Thank you so much.
YES. The blank refers to what is set. Due to blank I only can clickBACL button on IE browser to auto login.
I found the 401 error in IIS log when I click the link getting blank. How can following setting not work to auto login?
httpErrors errorMode="Custom"
remove statusCode="401"
error statusCode="401" path="401.html" responseMode="File"
httpErrors
How do I check cookie expiration in this case?

@MohammadYounes Highly appreciated if you can give me some ideas to fix this issue. Thank you.

@zbx888 The 401.html performs the redirect via a meta tag, check if the blank page contains this tag? may be try increasing the number of seconds.

@MohammadYounes Thank you!
Copy the View-> Source On blank page I got following source codes which totally is nothing with my code, looks like system automatically generated and the 401.html not called.

``
<META http-equiv="Content-Type"  content="text/html; charset=windows-1252">``
````

@MohammadYounes Thank you !
Got source code from View_Source of the blank page as following and it is generated by system, nothing with my application page which contains Master page. The 401.html file not called and no exception error is caught.
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">

<META http-equiv="Content-Type"  content="text/html; charset=windows-1252">

Very strange!

@MohammadYounes I am still struggle with the application one issue. If you can give some ideas to fix?
s1ppwebdev1 is a testing server name hosting several sites for testing and PayrollReportSSO is my web site folder under which I deployed the application also set StateServer to keep session data for application recycle pool.
The application runs fine at first several hours or even overnight on IE after starting then if I click one of links I got a blank page.
I used F12 developer tool to trace the network I found at that moment the page request favicon.ico under server root http://s1ppwebdev1/favicon.ico instead of own application root http://s1ppwebdev1/PayrollReportSSO and HTTP 1.1 404 not found error.

Following is working log but when getting a blank page the page didn't finish first step HTTP 302 transfer.

@zbx888 The favicon.ico is requested by the browser for each page, if it can't find it it will try to load it from the root. As you have this deployed as a web application (not a website), the browser is requesting http://s1ppwebdev1/favicon.ico instead of http://s1ppwebdev1/PayrollReportSSO/favicon.ico.
The logs you shared aren't relevant, as they are not showing any 401's!


RFC: Exit with code 3 when paths don't exist during scanning?

This is currently a bit open for debate, based on this forum thread.
Currently, restic returns exit code 3 for paths that it was told to back up, and that existed during its scanning phase, but then for some reason couldn't be read at the time that it would try actually back the path up. It currently does not return exit code 3 when there are paths it was told to back up, that did not exist during scanning either - it outputs a warning about these paths, but they don't cause an exit code 3 (this is true both for paths given on the command line and paths given in a --files-from file):
$ cat list.txt
foo
foo2
foo3

$ ./restic -r apa backup --files-from list.txt bar
pattern "foo3" does not match any files, skipping
bar does not exist, skipping
enter password for repository:
repository ba330420 opened successfully, password is correct
using parent snapshot 9af99ec5

Files:           0 new,     0 changed,     3 unmodified
Dirs:            0 new,     0 changed,     6 unmodified
Added to the repo: 0 B

processed 3 files, 494 B in 0:00
snapshot e1f007b5 saved

$ echo $?
0

The question is; Should restic return exit code 3 for files that it was told to back up but that it can't read at the time of scanning too (e.g. paths it was told to back up that doesn't even exist)?
A reason to do this would be to make sure that the user gets a non-zero exit code whenever there's any problem backing up the paths they supplied (not only when the paths became unreadable between scanning and backing up).
A counter-reason would be when you have a script that tells restic to back up your regular stuff as well as some mounted device, e.g. a hard drive, but where you don't have that hard drive mounted all the time. As it is now, you will not get a non-zero exit code when this happens, and if we change this you will. But perhaps that's a good reason to modify your script so it doesn't ask restic to back up that path if the drive isn't mounted..
Thoughts?

Personally, I'm leaning towards making this change, so that restic returns exit code 3 (unless a different exit code would be more suitable) for all paths it was asked to back up but couldn't. It would be more on the side of caution than it currently is, and there's nothing worse than having backups where you thought something was backed up, but it wasn't (besides being unable to restore them for some reason).

Backup paths that don't exist also don't end up in the list of paths belonging to a snapshot, such that only including existing paths in the call to the backup command won't lead to different snapshots.
I wonder whether someone is depending on restic silently ignoring non-existing paths. Returning exit code 3 when some files cannot be read already caused quite some confusion.

Related issue: Return different exit codes for different failures #956


Name format on the creditcard

Creditcard with dot in Name (LINCOLN/ABRAHAM J.)
Creditcard with only a full name (A. J. LINCOLN) - Name not separated by /

I had the same issue. I changed the RegExp patterns a bit.
This is pattern for VISA, but it's same part that parses the name that goes for all, allowing commas and periods inside the name:
var pattern = new RegExp("^%B(4[0-9]{12,18})\^([a-zA-Z0-9,. ]+)/([a-zA-Z0-9,. ]+)\^([0-9]{2})([0-9]{2})");


Added support for 'community-points-channel-v1'

Even though 'community-points-channel-v1' is not described on twitch api it seems to work fine and other libs are using it! It does not require token and read other events that 'channel-points-channel-v1' like custom-reward-created, custom-reward-updated, etc.
I've added the events individually and community-points-all that exports all events with the type value. I'm exporting the all object instead of splitting it the different values like channel_id, title, etc, since in the other events I end up splitting it later in my code instead using the ones that are exported from this lib.
As I said, this endpoint seems to have more "feature" than channel-points-channel-v1 and does not need token, print below to confirm (I was in debug: true)

Here is a video of me trying the endpoint with debug mode: https://cdn.dinip.pt/Dmp01/SIcIliyE06.mp4
Also a pastebin with the output jsons: https://pastebin.com/PezACb8T

Looks good but i wouldn't use the same event name as the other topic -- channel-points-channel-v1. I can't think of an example where someone would want to use both topics at the same time but should be able to handle both separately in case that changes in the future.


[Proposal] Extending Sequence to stride over a sequence

A Swift Sequence may represent many kinds of sequential elements: natural numbers, word from a source text, hues in a color wheel, bytes in a memory allocation, and so forth. It is natural and common to stride through sequences, especially those that aren't necessarily numerical in nature. A stride allows you to collect or consume spaced-out values at a faster pace. For example, you might want to look at "every 4th byte" to process alpha channels, "every other row" for spreadsheet highlights, or "every two hundredth word" for frequency analysis.
This proposal extends Sequence to introduce strides across any base.

This proposal should be updated to match the current style of the standard library, which is to conditionally extend base types with progressive capabilities (such as random access) using conditional conformance. It should also remove the implementation detail and focus more on the interface and use. Closing this PR for now pending those updates.


Clippy wont copy anymore after trying a multi-target delete

I tried something like  "clippy chuck 2 and 4" .It seemed  to work, removed the items, ... but after  was broken. Now clippy wont show any new clipboard activity in the panel. Can't copy anything into clippy now.
I tried deleting  node modules folder and restarting clippy, and restarting talon. Not getting any errors on bootup.
Osx Sonoma on M1

I just tried the same thing on windows and I didn't get any problems.
Do you have this same problem even after restarting clippy? That sounds like something is wrong with the storage then. You could of course try to remove the config file and/or the openAtLogin items on disk. They are stored in the user data folder four clippy. On windows that would be AppData\Roaming\clippy.
You could add a debug print here to find the folder
https://github.com/AndreasArvidsson/clippy/blob/b53611f0a0126c98f9a8ba0ac82508483b2b3ed5/src/util/storagePaths.ts#L12

I used that to find the path, it as /Users/<username>/Library/Application Support/clippy.
I cleared that folder. And also the "out" folder  in clippy repo, and reinstalled.
Still no luck. It's like it suddenly can't see or wont show the clipboard history.
Still need to restart entire comptuter to check that.

Ah you probably need to update the talon files.

I restarted computer, cleared those directories again. Still no luck.
I had previously update the userland code from your repo into my talon folder. Checked it again. I don't think that would be the culprit,  because  even if talon is off, when clippy starts up,  it should see  hand show  the current clipboard history..?

Very strange.
Yes should work without talon.
Let's have a look at one of the community meetups.

Ahh, I reverted commit, and it still wasn't working. But then I saw and remembered the notes to set permissions for mac with chmod +x node_modules/clipboard-event/platform/clipboard-event-handler-mac. Since those dissapeared from readme I didn't think they were needed anymore. But when I reinstalled, ran than, and then started up, it is working!

The prepare assets file should do that for you now, but might have broken. Really hard when I can't test stuff like that for myself.
Good catch. I will have a look at it.


Rendering engine without view files

Hello! I want to write an express rendering engine which uses views retrieved from a database, not the file system. I've checked this repo's issue history and done a lot of googling but I've found nothing...
It seems express requires a view file to exist on disk before passing the req/res to a rendering engine. Is there a way to prevent this behaviour?
Ideally, when I invoke res.render(name)my rendering engine is called with (name, options, callback) without /[name].html necessarily existing on disk.
I could overwrite express' res.render with my own renderer. However in order to handle errors properly it seems I'd have to pass in next each time I called res.render. That seems repetitive.

@davglass Express' view rendering is based on the file system. There is no inherent feature to support what you are looking for.
Mind explaining why you want to do what you are trying to do?

@hacksparrow I think you meant @davidmerfield 😄

Err yes, indeed.

Thank you for your response. I'm not quite sure what other information I can provide.
It seems a strange choice to prevent the rendering engine from being invoked if a local view file doesn't exist. As it stands, the rendering engine is passed a file path (not the file's contents) and is responsible for reading that file from disk.
Why not just pass the view's name to the rendering engine? (Let the rendering engine resolve the path to the view and check if a file exists).
Rendering Engine Foo could use that view name to look up a template on disk, returning an error if the template doesn't exist.
Rendering Engine Bar could use that view name to look up a template in a database, returning an error if the template doesn't exist.
However, I'm sure there's a lot of context I'm missing for these decisions. Anyway, thank you for getting back to me. I'll work something out myself.

Hi @davidmerfield, I don't believe @hacksparrow was saying n to you, only trying to ask for more clarification on your request. Perhaps I can provide some better context on the current views system. I think it's fair to say this views system was written prior to any of the current maintainers joined Express, and as such, none of us were actually involved with the creation & definition of this part of Express. Essentially "it is what it is".
BUT that's absolutely not to say it cannot be different.
I understand where you are coming from, but of course can also say that's just not where the Express view engine is today. I think that possibility morphing the view engine into being able to work as you describe may be a better place.
So, @davidmerfield, to jump off this discussion, would you be up for formulating a proposal on how this view system should work, how it differs from the current Express view system, and perhaps even what it would take for us to get from the current to the proposal and what kinds of impact it would have to the current view engine ecosystem and perhaps even what the upgrade paths would be?

I could overwrite express'  res.render  with my own renderer. However in order to handle errors properly it seems I'd have to pass in  next  each time I called  res.render . That seems repetitive.

As for this specific, technical point, you can use the same plumbing Express uses that prevents you from passing in next to res.render: the next function is actually always available as req.next, which you can call instead of requiring a callback.

Maybe something to be discussed at https://github.com/pillarjs/discussions/issues/2

The other angle here is 404s. To my knowledge there's two ways for a matched route's handler to 404: it can call next() to let the following routes/handlers try (this isn't an immediate 404, but can become one based on whether the other handlers can serve the request), or responding with a custom 404 from that handler. (In the latter case, a custom 404 may or may not be handler-specific; I could have one 404 function and both install it as my final fallback route handler and allow it to be called by other handlers to bypass trying any more routes, or any individual handler could create a 404 response in any way appropriate.) Supposing I had an app.findView function to perform the same lookup that the view engine currently uses so I don't have to duplicate the view lookup logic to get in ahead of the view engine's 500 error, I could check that function and enact the 404 strategy of my choice instead of calling response.render. However, that doesn't solve letting the rendering engine do something other than file lookups with the view name/path, and getting rid of the 500 error to let the rendering engine use non-filesystem resources would be just as broken by such a pre-emptive 404 check as it is by the builtin 500 error.
Whatever solution is worked out here should really have the following characteristics:

Whatever calls response.render should be able to choose whether a failure to find the view is a 500 error, a call to next() to try more route handlers, or a call to an arbitrary function that can terminate the handling in a custom 404.
The view engine should be able to choose whether the path is resolved against the filesystem or against something else, yet defer to the calling handler's choice of 500, next() or custom 404 handler when the expected resource is missing (whatever "missing" means in the place where the resource is looked up).

An extra perk to such a system would be that the rendering engine also could theoretically 404 if any resources other than the named view are missing, e.g. if the view includes a partial and the partial is missing (analogous to how if a static HTML page links to a page that's missing that link is considered a 404 even though it's an error in the site's content).

While looking for next("route") in the documentation, I noticed something in the documentation for express.static that led me to some further experimentation, and I've discovered that the simpler way to force a 404 and bypass the remaining routes is to pass next an error with the property status set to 404. This means my initial strategy for handling 404s in general needs a small adjustment, but it also means I can simplify my desired ideal for how the rendering engine allows for custom 404s. I've updated my preceding comment.

Hi @ScottFreeCode great comments, though I'm unclear how the 404 discussion relates to having the views use something other than the file system (the topic of this issue). The means that it is likely that we'll close this issue once non-fs views are implemented, and I don't want your comments to get lost in the shuffle.

Hi @dougwilson; thanks! Mainly I am figuring that, if Express were to cease sending 500 errors when the view file isn't found so that rendering engines can look for stuff in places other than the filesystem, then it would be up to the rendering engine to send an error such as 404 if the file isn't found (or, if it's not using the filesystem, if whatever else it's looking for isn't matched) -- because right now the 500 error is basically a 404 (the file wasn't found) but with arguably the wrong number, and that automatic error is what's in the way of non-fs views. So, if as I expect the view engines of the future end up responsible for sending the not-found error, I'd like to not end up in a situation where every engine is making its own choices about how those errors are reported, but rather have a fairly standard way for the calling program to tell the engine whether it should be a 500 error as it is now, a 404 error, or a call to next(). Maybe that's better suited for discussion at wherever the plans are for view system improvement, though; or maybe it's not something Express can control anyway and just needs to be documented for developers of future rendering engines?

I'm still not clear on what you mean, I'm sorry. I can say yes, we give a 500, but only if you are not passing a callback to the render function to do any other behavior. We only 500 if you don't write the code to do something else...?

Aaah, you're right; I had forgotten about the callback argument because I initially avoided it in order to avoid having to send the rendered response myself. My bad.

I workaround could be make and express extencion that rewrite the default render behabior using a blank template that render just a single variable, once you've retrieved the database view content (You can use you preferred template engine to flat your variables on the database string that will be your template) you can render this template with
.render('generic-template', {
'content': stringWithYourContent
})


[docs] migration guide: rename visuallyhidden incorrect description

Related page
https://mui.com/material-ui/migration/v5-component-changes/#rename-visuallyhidden
Kind of issue
Unclear explanations
Issue description
It says

Rename visuallyhidden to visuallyHidden for consistency.

But visuallyHidden is the same word twice...

Context
No response
Search keywords: rename visuallyHidden migration

@aarongarciah I feel so dumb. Classic developer oversight 101 here...  💥😃🔫


Add support for transforming Doxygen discussion/note tags

Bug/issue #, if applicable: Related to rdar://116077012
Summary
This note adds support for formatting Doxygen @discussion and @note tags. Discussion tags are turned into normal paragraphs, while @note tags are turned into “note” blocks.
Dependencies
https://github.com/apple/swift-markdown/pull/159
Testing
Create a documentation comment containing @note or @discussion tags:
This is an overview.

@discussion This is a discussion.

@note Please take note.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[ ] Added tests

I couldn’t figure out how the test fixture files were generated. But I would be happy to add some tests!

[x] Ran the ./bin/test script and it succeeded
[ ] Updated documentation if necessary — I don’t think this is needed, right?

@swift-ci please test

Does the reference resolver already visit this content? A good way to check would be to add unresolvable links in both places and look for warnings:
This is an overview.

@discussion This is a <doc:NotFound> discussion.

@note Please take <doc:NotFound> note.

Does the reference resolver already visit this content? A good way to check would be to add unresolvable links in both places and look for warnings:

Added that doc comment to a test project and got warnings on both links.

@swift-ci please test

@swift-ci please test

What do you mean by “verify the content of the in-memory node?” Does that mean I should be introspecting symbol.discussion?.content and not just comparing the formatted strings in the test? Added your test + some links in the meantime.

@swift-ci please test

@swift-ci please testr

@swift-ci please test


Blaze Join order

This is my code and I'm not finding any way to force blaze to first join with the CTE before doing implicit joins from the entity view :

 final BlazeCriteriaBuilder blazeCriteriaBuilder = BlazeCriteria.get(this.criteriaBuilderFactory);
            final BlazeCriteriaQuery<Product> blazeCriteriaQuery = blazeCriteriaBuilder.createQuery(Product.class);

            final Root<Product> root = blazeCriteriaQuery.from(Product.class);

            final var criteriaBuilder = blazeCriteriaQuery.createCriteriaBuilder(this.entityManager);

            criteriaBuilder
                    .innerJoinOnSubquery(CoPurchasesCTE.class, "CoPurchasesCTE")
                    .from(CoPurchasingPair.class)
                    .bind(CoPurchasesCTE_.PRODUCT_ID).select("product2.id")
                    .bind(CoPurchasesCTE_.FREQUENCY).select("frequency")
                    .where("product1.id").eq(productId)
                    .orderByDesc("frequency")
                    .setMaxResults(limit)
                    .end()
                    .on("id").eqExpression("CoPurchasesCTE.productId")
                    .end();

            final var entityViewSetting = EntityViewSetting.create(ProductCardView.class);

            final var productCardViewCriteriaBuilder = this.entityViewManager.applySetting(entityViewSetting, criteriaBuilder);

            return productCardViewCriteriaBuilder
                    .getResultList();

Result query :
select
      *
   from
       product p1_0
   left join
       product_tag t1_0
           on p1_0.id=t1_0.product_id
   left join
       tag t2_0
           on t2_0.id=t1_0.tag_id
   join
       (select
           cpp1_0.product2_id, cpp1_0.frequency
       from
           co_purchasing_pair cpp1_0
       where
           cpp1_0.product1_id=?
       order by
           cpp1_0.frequency desc
       fetch
           first 10 rows only) cpc1_0(product_id, frequency)
           on (p1_0.id=cpc1_0.product_id)

How can i please force Blaze to first join with CTE sub query then move on to the next joins.
Thank you :)

SQL is declarative, so the order or joins in this particular case shouldn't matter. What problem are you trying to solve?

Thank you for your response.
In my query, I use both INNER JOIN and LEFT JOIN. If I perform the INNER JOIN between the product and my CTE first, there will be fewer rows for the subsequent joins, leading to better query optimization. For example, if we have three tables A, B, and C:
Performing A INNER JOIN B LEFT JOIN C will generally be more efficient than A LEFT JOIN B INNER JOIN C.

Can you try adding the following before calling this.entityViewManager.applySetting(...):
criteriaBuilder.leftJoinDefault("tags", "t")

I think that this might change the join order at least in the HQL that is produced, though Hibernate ORM could still re-order joins potentially.

It worked as expected. I just need to arrange all the LEFT JOIN clauses in the desired order within my query.
Thank you so much for your help; you saved me a lot of time!

Glad it worked out for you!


macOS shared_lib_extension wrong

Bug summary
On macOS, the backend tries to load librt-backend-omp.so which does not exist.
Fix
Edit line 196 in src/runtime/backend_loader.cpp
#ifndef _WIN32
#ifdef __APPLE__
  std::string shared_lib_extension = ".dylib";
  #else
  std::string shared_lib_extension = ".so";
  #endif
#else
  std::string shared_lib_extension = ".dll";
#endif

solves it!

Already on it :)
also adding running (and fixing) tests on MacOS ;-)


Changing the Autofocus checkbox in the channel of a Protocol Step and clicking Change, crashes LVP

AF cannot be changed in a Step for a Protocol

can you create a step-by-step on this and include a log file with stack trace.

yes, will do for three or four old issues like AG and Step 0 Add


Fix text-file type=password behaviour

#44 #46

Coverage increased (+0.003%) to 92.872% when pulling 4e476345730072ee11bb4839fb07882e76a9a065 on textFieldPassword into 5adb881a9a021fc4d7dcfa7c960d72a2b16bf295 on master.

@netmenya just noticed that there is another pr #48 for this. Ping me and i will close this.

Coverage increased (+0.003%) to 92.872% when pulling f52ab017863f188639a1a5de57830c0c0847cd0f on textFieldPassword into 5adb881a9a021fc4d7dcfa7c960d72a2b16bf295 on master.


window::Window::is_maximised does not return the current state of the window

Describe the bug
window::Window::is_maximised does not return the current state of the window, it instead returns a value that it set on creation of the Window object.
Steps To Reproduce
Run the is_maximized, change from maximized to normal, or from normal to maximized and run the is_maximized method again. the function will give back the same value.
Expected behavior
for the is_maximized function to instead run self.window.is_maximized. as of now it gives back a stored value on initialization.
Platform and Versions (please complete the following information):
OS: Linux
Rustc: rustc 1.78.0 (9b00956e5 2024-04-29)
Additional context
This makes it impossible to track whether or not the window is maximized effectively. If there is a reason for it to give back the value on initialization, please provide.
edit: there are a few more functions that have the same behaviour, for example: is_minimized, inner_position, outer_position, scale_factor, inner_size, outer_size.

This is partially true, the Window struct contains a clone of an AtomicBool which will be updated in another callback in https://github.com/tauri-apps/tao/blob/b1b8b3ffac3dcd98c92e0d0010a49c5dc43e40c0/src/platform_impl/linux/window.rs#L264, same is done for other getters as well.
Can you show a reproduction where it is not behaving as expected? Also what Desktop environment you're running?

Im sorry for the false report, it seems that the way i tried to retrieve the state was not functioning correctly and wasnt outputting the current value, which led me to looking at the code, missing the callback and coming to the conclusion that there is nothing that updates the state after initialization. I hope this wasn't a big inconvenience, but may i ask, are there events for when these values change? I have been looking for a way to do this without retrieving the data at some set intervals.

Without a code example of what you're trying to do, I can't really suggest that much but generally Resize event will be triggered when maximizing a window, so maybe check there.

Ah, so there isnt a concrete event, like WindowMaximized or anything simmilar. Well i think im going to close this issue here.


Fix showcase and page quick delete buttons

The showcase and page quick delete buttons didn't check if the item could be deleted, which would break the site path. This no longer happens.

Coverage increased (+0.007%) to 95.74% when pulling bd97249521509057bd015330cc8163ad7bcb3883 on quickDeleteFix into c37d604b57155ea822a494ccc2d2f4dc02415578 on master.

Coverage increased (+0.008%) to 95.741% when pulling 9ef0d39a665f9a8a2508e9eff876b4ed4ed0b627 on quickDeleteFix into c37d604b57155ea822a494ccc2d2f4dc02415578 on master.

General question on all of the Destroy services: what should the behavior be when it cannot destroy? In the past we've allowed it to fall through to FKC checks at the data layer, which would throw an exception. Since we're handling checks at the app layer to see if there is an association in site_objects, I wonder if it shouldn't behave the same. Ie, if we get to the point where its actually trying to Destroy an object and it fails a CanDelete? check, raise an exception.

@jgondron I don't know that the FKC have actually worked for these cases in the past. I had to change how these delete utilities work because I was able to delete things that should not have been deletable and which, when deleted, broke the site path. I agree that we should stay consistent, but I also think changing these to be exceptions will be a considerable change and should be put into it's own task.

Coverage increased (+0.01%) to 95.745% when pulling eadb7896482596d88220c612b2b7a5bc51c3c582 on quickDeleteFix into c37d604b57155ea822a494ccc2d2f4dc02415578 on master.

Coverage increased (+0.01%) to 95.745% when pulling 194da83736569af01172f832df1d7cfe25d8c27f on quickDeleteFix into c37d604b57155ea822a494ccc2d2f4dc02415578 on master.


Set Uncaught Exceptions' JSON Responses

If nerium can capture uncaught exceptions, most often seen as "Internal Server Error" messages, that would allow the client to manage these errors like any other, by looking at the error property. As is, the JS will need to take this potential response into account repeatedly, at the expense of maintainability and, potentially, performance.
http 'http://nerium-takei:8081/v1/takei/takei-orders/affix/?client_id=361&advertiser_name=Visa'
HTTP/1.1 500 INTERNAL SERVER ERROR
Connection: close
Content-Length: 37
Content-Type: application/json
Date: Mon, 22 Jan 2018 17:38:02 GMT
Server: gunicorn/19.7.1

{
    "message": "Internal Server Error"
}

I'm not sure we've got handling for every contingency, but have added 400s for everything that we've seen in practice, and a catchall that reformats any remaining 500s to JSON. If anything else pops up please start a new issue, but closing this for now.

https://github.com/OAODEV/nerium/pull/22


Add Severity bar in html report

As proposed by @Quentin-M, it could be nice to have a severity bar in place of

More or less,

Ok, I think the code is done, but I removed it, I don't know why, it will be quick !
Little precision : Should I include safe features in the total or not ?
Example
data: 8 features, 4 safe, 2 low, 1 medium 1 high
result: 50% low - 25% Medium - 25% medium (without) vs 50% save - 25% low - 12.5% medium - 12,.5 % High (with)

You can included for now

@jgsqware pull request created :)


Add custom server example using Hapi

This is really cool. Very very neat example! Thanks for taking the time

No problem :-).  Updated with nextWrapper.js -> next-wrapper.js


Issue with compiling

ive been trying to compile for like a week straight and before this error i was getting a video.cpp error. i fixed that by changing my hxcodec version to 2.6.0 but now when i compile or open the exe i get a new error:

Note: my friend @boohglas DID use hxvlc to make a intro cutscene, and it compiled and worked for him, but it wont for me. am i missing something here?

Remove the export folder and hxcodec library from the project and then it should work fine.

source/states/PlayState.hx:44: characters 14-28 : Type not found : vlc.MP4Handler
source/states/PlayState.hx:847: characters 13-25 : Type not found : VideoHandler
new error

That's not from within hxvlc

i looked it up and it has to do with me removing hxcodec

im still confused as to what to do cause once i removed hxcodec and deleted the export like you said and the compiled, compilation started fighting for its life cause i removed hxcodec. do i change anything in playstate.hx?

im still confused as to what to do cause once i removed hxcodec and deleted the export like you said and the compiled, compilation started fighting for its life cause i removed hxcodec. do i change anything in playstate.hx?

Yes, hxcodec and hxvlc aren't the same libraries

well i know that much. what else do i change though


An example for WAIT task

Hi, I am currently working on some prototypes using conductor orchestrating workflow. The prototype workflow has a http task to download an image and the next task's input param need to be the downloaded image. Since download takes time, I am planning to use WAIT. Could you provide an example WAIT task? Kitchensink example does not have a reference  to WAIT task.
Another option I am trying to use is - Fork and Join. Fork on one task and join on the task. Fork example has a sub workflow within it. Is subworkflow necessary for a Fork task?
Thank you,
Sri

WAIT task can be added by creating a task with taskType as WAIT.  WAIT task acts as a gate and does not move forward until mark completed either by posting a message to SQS or manually invoking REST endpoint.
https://netflix.github.io/conductor/metadata/systask/#wait
Subworkflow is not necessary for the fork task.  Fork can create any task of tasks.

Closing the issue.  Please re-open/create if you need further help.


Device index not passed for openmm engine

When running the openmm engine with {'OpenCLDeviceIndex' : '1'} set, it always runs on device 0 instead of the given DeviceIndex. This is the full engine generating script.
from simtk.openmm import app
from simtk import unit
import openmmtools as omt
import simtk.openmm as mm
import openpathsampling as paths
import openpathsampling.engines.openmm as peng_omm

template = peng_omm.snapshot_from_pdb('input.pdb')
topology = template.topology
gro = app.GromacsGroFile('input.gro')
box_vectors = mm.Vec3(*gro.getPeriodicBoxVectors())

forcefield = app.GromacsTopFile('topol.top', periodicBoxVectors=box_vectors)

system = forcefield.createSystem(
        nonbondedMethod=app.PME,
        nonbondedCutoff=1.1*unit.nanometers,
        constraints=app.HBonds,
        rigidWater=True,
        ewaldErrorTolerance=0.0005
)

system.addForce(
        mm.MonteCarloBarostat(
                    1*unit.atmospheres,
                    310*unit.kelvin,
                    25
                )
)

integrator = omt.integrators.VVVRIntegrator(
        310*unit.kelvin,
        5.0/unit.picoseconds,
        2.0*unit.femtoseconds
)

integrator.setConstraintTolerance(0.00001)

openmm_properties = {'OpenCLDeviceIndex': '1'}

options = {'n_frames_max': 4000,
           'n_steps_per_frame':2500,
           'platform':'OpenCL'}

engine = peng_omm.Engine(
        template.topology,
        system,
        integrator,
        openmm_properties = openmm_properties,
        options=options
)

This option does work when using pure openmm, so I think there's a strange issue passing the openmm_properties

Okay, I see the problem and checked that the options are passed to the engine. The problem is in the way you initialize the engine. We changed this a little so let me explain with the example that should work. Keep all you had except
openmm_properties = {'OpenCLDeviceIndex': '1'}

options = {'n_frames_max': 4000,
           'n_steps_per_frame':2500}

engine = peng_omm.Engine(
        template.topology,
        system,
        integrator,
        openmm_properties = openmm_properties,
        options=options
)

engine.initialize(platform='OpenCL')

if you do not specify the platform when initializing it will use the fastest available and this could be CUDA.
At least check that my example works.
Now the bigger problem I see here or maybe you (@sroet and @dwhswenson) have an idea:
The issue
We agreed that we did not want to fix completely the platform, which allows to restart the same saved engine on another platform. Hence the initialisation with the platform needed. The current problem is that we store platform specific options which render the approach unfeasible. Fortunately the CUDA platform does not care about non-CUDA options and will ignore them. The other platforms will complain (this is to be changed in the next OpenMM release).
I think the current way does not make (complete) sense we either

stick to the old way and really create an engine with all the options and recreate the engine anytime we want to run on another machine with another platform, etc... This is the easiest way, but it will also mean to recreate movers that use the engine.

We separate the engine description and the actual instance of the engine. I had the idea before and it would mean the the Engine is only a kind of factory which the instance independent properties and running the initialization can set properties of the engine for a particular instance. If you use the engine it will create an EngineInstance object that contains the additional information like machine, time of creation, additional properties, etc... and this is what is stored with the snapshots instead of Engine itself. This could in particular be useful when using Gromacs or a general ExternalEngine to store information on the specific run.

@jhprinz

At least check that my example works.

This works! Thanks!
I'm not entirely sure why it wasn't before though. I understand that the "fastest" platform is always OpenCL device 0 (CUDA isn't an available platform for the OpenMM build that I'm currently using) , but the OpenCLDeviceIndex should be able to overwrite this, right?

That is strange actually. The only thing I can imagine is that Openmm will ignore platformProperties in case you do not specify a platform. This would make sense from the point that properties are not compatible between platforms.

It also behaves unexpected when I try to restart. Running the following code defaults to device 0, even when the original engine ran on device 1 (in a different script).
def next_steps(old_storage, current_storage, n_steps):
    tps_sampler = old_storage.pathsimulators[0]
    tps_sampler.storage = current_storage
    tps_sampler.restart_at_step(old_storage.steps[-1])
    tps_sampler.run(n_steps)

Its been a while. Any news here? I did not have much time to look into this. There is still the issue that when restarting after loading you need to initialise an engine using .initialize()

Current status of this is:
if I load the engine and do analysis with it, without initializing it explicitly it runs on device 0. With explicit .initialize(platform='OpenCL') it seems to run on the original device index of 1. I didn't have the time yet to look how this behaves in the case of me trying to restart a tps simulation


Improvement for error message "Counting invalid without valid specification does not make sense"

It would be great if there was a reference to which check or line number this refers to, with larger agreements its tricky to find.

SODA-925


[d.gg utilities] Bot Flair Missing from "Hide Individual Flairs"

This 'Bot' flair is not an option under Settings -> Hide Individual Flairs
Here are the options I do see: https://i.imgur.com/bvz8H3R.png

Flairs are normally called flair1, flair2, flair3, etc....
Looks like the bot's flair is just called "Bot".
We need to manually add to the list of flairs that is loaded there an item for "Bot".

Fixed in the dev branch (@Voiture-0 ❤ PepoTurkey)


Release 0.11.5

This pull request introduces new features/fixes including:

Support for the native Boomi JMX decommissioning of nodes on scale-down
Support for automounted volumes
Support for external NFS drives
Automatic PCF registration
Support for private Docker registries

@aanelli Merged PR. Thanks!


Error when searching

I have tried searching from two different networks (in case twitch banned my ip address) and both networks can not search. It could be a twitch problem and will resolve its self. But here is the error when searching
System.AggregateException: One or more errors occurred. ---> System.Net.WebException: The remote server returned an error: (500) Internal Server Error.
at System.Net.WebClient.DownloadDataInternal(Uri address, WebRequest& request)
at System.Net.WebClient.DownloadString(Uri address)
at TwitchLeecher.Services.Services.TwitchService.SearchChannel(String channel, VideoType videoType, LoadLimitType loadLimit, DateTime loadFrom, DateTime loadTo, Int32 loadLastVods)
at TwitchLeecher.Services.Services.TwitchService.Search(SearchParameters searchParams)
at System.Threading.Tasks.Task.Execute()
--- End of inner exception stack trace ---
---> (Inner Exception #0) System.Net.WebException: The remote server returned an error: (500) Internal Server Error.
at System.Net.WebClient.DownloadDataInternal(Uri address, WebRequest& request)
at System.Net.WebClient.DownloadString(Uri address)
at TwitchLeecher.Services.Services.TwitchService.SearchChannel(String channel, VideoType videoType, LoadLimitType loadLimit, DateTime loadFrom, DateTime loadTo, Int32 loadLastVods)
at TwitchLeecher.Services.Services.TwitchService.Search(SearchParameters searchParams)
at System.Threading.Tasks.Task.Execute()<---

An internal server error 500 happens when the request gets processed by the server and the executed code throws an unexpected error that does not get caught by an error handling mechanism. In other words: Twitch messed up 😄


possible typo - counting bin index for range join

Hi - thanks for the writeup! I had a question about the bin trick to optimize range joins.
     WITH prep AS (

   SELECT user_id
        , event_at
        , event_name
        , FLOOR(DATE_PART(epoch_second, event_at) / ($bin_size_sec)) AS bin
     FROM events
    WHERE event_name IN ('A', 'B', 'C')

    UNION ALL

   SELECT user_id
        , event_at
        , event_name
        , FLOOR(DATE_PART(epoch_second, event_at) / ($bin_size_sec)) AS bin
     FROM events
    WHERE event_name IN ('A', 'B', 'C')

      AND DATE_PART(epoch_second, event_at) % $bin_size_sec >= $bin_size_sec - $sequence_duration_sec

)

I understood we want to carry forward the last two hours from the previous bin to the next one. Should the second view in this UNION query not compute the bin index as:
1 + FLOOR(DATE_PART(epoch_second, event_at) / ($bin_size_sec)) AS bin

instead of,
FLOOR(DATE_PART(epoch_second, event_at) / ($bin_size_sec)) AS bin

Fixed, thank you!


RuntimeError: can't start new thread

I use the mask2former to train the semantic segmentation on ADE20K.
When the training process turns to evaluation, the error occurs:
RuntimeError: can't start new thread. RuntimeError: DataLoader worker (pid 59861) exited unexpectedly with exit code 1. Details are lost due to multiprocessing. Rerunning with num_workers=0 may give better error trace.
But if I directly run with evaluation mode, there is no error.
My env is as follows:
`----------------------  -----------------------------------------------------------------------------------
sys.platform            linux
Python                  3.8.13 (default, Mar 28 2022, 11:38:47) [GCC 7.5.0]
numpy                   1.23.1
detectron2              0.6 @/home/huang/code/detectron2-main/detectron2
Compiler                GCC 5.5
CUDA compiler           CUDA 11.1
detectron2 arch flags   8.6
DETECTRON2_ENV_MODULE
PyTorch                 1.9.0 @/home/huang/.conda/envs/mask2former/lib/python3.8/site-packages/torch
PyTorch debug build     False
GPU available           Yes
GPU 0,1,2,3,4,5,6,7     GeForce RTX 3090 (arch=8.6)
Driver version          455.23.04
CUDA_HOME               /usr/local/cuda
Pillow                  9.2.0
torchvision             0.10.0 @/home/huang/.conda/envs/mask2former/lib/python3.8/site-packages/torchvision
torchvision arch flags  3.5, 5.0, 6.0, 7.0, 7.5, 8.0, 8.6
fvcore                  0.1.5.post20220512
iopath                  0.1.9
cv2                     4.6.0

`

Looks like a PyTorch or Detectron2 data loader problem.


Consider maintaining pyuv

I ran into https://github.com/saghul/pyuv/issues/271 when compiling pyuv for CPython 3.10.  It should be a simple fix, but it seems like the project is effectively unmaintained and looking for new maintainers.  Since pynvim is the only package I use that (optionally) depends on pyuv, I thought I would make a note here to see if we are interested in getting maintainer status over there to at least keep it functional.  Alternatively, we could drop pyuv altogether but based on the setup.py it seems that we would have to drop support for older Pythons on windows.
I'm not familiar enough with either code base to volunteer myself as a proper maintainer, though in a pinch I could PR + merge the necessary fixes to get it working.

I take this back after looking at the code.  Since the PY3 code path doesnt even consider pyuv we should just EOL that dependency at some point.


[BugFix][Cherry-Pick][Branch-2.5] Make Struct create all subfield columns to prevent BE crash

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Cherry-pick #14593
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_be_unittest


Problem loading npm-packaged vue-component

Description
I wanted to use vue-google-autocomplete via vue-brunch. vue-google-autocomplete declares a .vue file as its main in package.json.
When adding it to the project and integrating it as per its readme, everything looks fine until the moment one opens the browser:
Uncaught Error: Cannot find module 'vue-google-autocomplete' from 'main.js'

Actually this module should be compiled and present, but can not be found anywhere in the bundled file.
Here's a pretty minimal example to play around with:
git clone https://github.com/pierrebeitz/vue-brunch-google-autocomplete-problem
cd vue-brunch-google-autocomplete-problem
npm install
npm start
# open http://localhost:3333

I also asked at vue-brunch. But after reading a bit through the issues here, i think you folks could have some input to guide me to the right direction as well. I already found out that everything works, once the path of the file to include does not start with node_modules - but marking it as "not vendor" via conventions.vendor is not sufficient.
Thanks for your time and input (and this brilliant piece of software - straying through the source was a pleasure). :)
Environment

Brunch: 2.10.12
Node.js: v8.4.0
NPM: 5.4.2
Operating system: MacOS 10.12.6
Editor: Spacemacs

package.json
{
  "name": "blorgh",
  "description": "blorgh",
  "version": "0.0.1",
  "scripts": {
    "start": "brunch watch --server",
    "build": "brunch build --production"
  },
  "dependencies": {
    "vue": "^2.4.4",
    "vue-google-autocomplete": "^1.0.13"
  },
  "devDependencies": {
    "babel-brunch": "^6.1.1",
    "babel-eslint": "^7.2.3",
    "babel-plugin-transform-runtime": "^6.23.0",
    "babel-preset-es2015": "^6.24.1",
    "brunch": "^2.10.10",
    "vue-brunch": "^2.0.1"
  }
}

brunch-config.js .
module.exports = {
  files: {
    javascripts: { joinTo: 'app.js' },
    templates: { joinTo: 'app.js' },
  },
  plugins: {
    babel: {
      presets: ['es2015'],
    },
  },
};

seems to work via
  npm: {
    compilers: ['babel-brunch', 'vue-brunch'],
  },

in brunch-config.js.


[feature request]: Optionally disable tests during configuration

It would be great if there was a way to disable the compilation of the unit tests during cmake's configuration. Specifically for CI. When we include Catch via ExternalProject_Add, the unit tests are compiled during each CI test which wastes time. A simple -DDISABLE_UNITTESTS would go a long way to shaving off a minute from each test for people using Catch.hpp.

FWIW is there a reason you can't just run:
curl -L https://github.com/philsquared/Catch/releases/download/v1.9.3/catch.hpp -o catch.hpp

first?

Do you mean Catch's own selftests?

@kirbyfan64 curl is not present on all systems.
@horenmar yes, Catch's own selftests. A lot of people use CMake's ExternalProject_Add to get Catch instead of using git submodules, or maintaining a copy in their repo, but Cactch's CMakeList.txt is currently setup so that a make install first builds the self tests, so this is being done on every CI test for projects that use Catch.

This is a good example of what I'm talking about.
https://travis-ci.org/Microsoft/GSL/builds/226877346?utm_source=github_status&utm_medium=notification
For our hypervisor project we maintain a patched version of Catch to fix this, but I couldn't do that for Microsoft's GSL patch to switch to Catch, so at the moment, its always building the self tests.

curl is not present on all systems.

Yes, but it's available:

Specifically for CI

;)
FWIW there's other stuff you can do too, but in general, curl is present on most systems. Heck, Git uses curl for HTTPS transport, so if someone's downloaded the source code, there's a good chance they also have curl installed.

For CI, yes we probably could make that work. It's not as elegant though as we would have two ExternalProject_Add blocks, one for CI, and one for everything else. Plus, I think in general when someone wants to say build and test the GSL, they don't want build and test the GSL's unittest selftests. I think it would be a useful option for projects using Catch, and it would be a pretty simple mod to the CMakeLists.txt file to support.

While I still think that either getting the single-include directly, or vendoring it, is the better option, especially if you are using tagged version and don't want automatic update, there is no reason not to support this.
Anyway, just sticking if (!NO_SELFTEST) and endif() around defining the SelfTest and Benchmark targets should be enough, correct?

This is my patch. I don't think you need all of this. You do at least have to disable setting the compiler options for these targets if they do not exist.
https://github.com/Bareflank/Catch/commit/61080717bf7706a94ab115441d5357911c02907d

👍 Thanks a ton


Import Custom Data with XLS file in 3.1 Version

How you import some custom data with xls file.  I am using 3.0
like  in this code Excel::import(new UsersImport, 'users.xlsx'); I also want to send some form data.

You can just pass the additional information as parameter(s) to your Import class and use it over there.
E.g. see this topic for an example.

@GlennM Thank you Very much for your help. It's work


An Error Occured When Clicking Generate [Ubuntu 22]

Hi there,
Before opening this i went through the closed issues section and found one more famiIiar issue
https://github.com/FujiwaraChoki/MoneyPrinter/issues/21   ,
but no solution was there so opening this
have set this up on Ubuntu VPS running version 22. Whenever i click generate , it shows
"An error occurred. Please try again later."
Below are my screenshot
The error

Running Backend

Running Frontend

My .env file

replace localhost with vps ip , in index.html


Fixes #470. Support legalLabelInsets on Apple Maps

Fixes #470. This will add support for legalLabelInsets prop on iOS with Apple Maps. Code is adapted from RN core.

Thanks! We should probably avoid -> notation. Would you mind adding an example for this?

@gilbox Yeah, let's keep it consistent. I've removed it.

@scarlac will you be able to add an example?

@gilbox Ah! I am assuming you mean updating the documentation. Yes, I'll get right on it tomorrow and update this PR :)

@scarlac I was talking about adding an example in the example/examples dir. That's how I smoke-test PRs since we don't have specs.

@gilbox I've added an example to demonstrate the usefulness of the legalLabelInsets. I think it's fairly simple while showing a very realistic scenario of when it could be of use. Personally, I'm using it in an advanced way but opted for simplicity in the example, for the sake of understanding.
Let me know if there's more you need before it's ready for a merging.

lgtm but I'd like to get a second opinion re: the async op

thanks @scarlac ( ◠‿◠ )

My pleasure! 👍


jackett: 0.21.1915 -> 0.21.2090

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jackett is: API Support for your favorite torrent trackers
meta.homepage for jackett is: https://github.com/Jackett/Jackett/
meta.changelog for jackett is: https://github.com/Jackett/Jackett/releases/tag/v0.21.2002
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.21.2002 with grep in /nix/store/w3g9lhsq1b7li6bkr3mqdacxq0hjxn77-jackett-0.21.2002
found 0.21.2002 in filename of file in /nix/store/w3g9lhsq1b7li6bkr3mqdacxq0hjxn77-jackett-0.21.2002

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jackett

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w3g9lhsq1b7li6bkr3mqdacxq0hjxn77-jackett-0.21.2002 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jackett https://github.com/r-ryantm/nixpkgs/archive/14fd5d684d1736cacfaf977c7100c96e679cb5b5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/14fd5d684d1736cacfaf977c7100c96e679cb5b5#jackett

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w3g9lhsq1b7li6bkr3mqdacxq0hjxn77-jackett-0.21.2002
ls -la /nix/store/w3g9lhsq1b7li6bkr3mqdacxq0hjxn77-jackett-0.21.2002/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

jackett

Maintainer pings
cc @edwtjo @nyanloutre @purcell for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jackett is: API Support for your favorite torrent trackers
meta.homepage for jackett is: https://github.com/Jackett/Jackett/
meta.changelog for jackett is: https://github.com/Jackett/Jackett/releases/tag/v0.21.2031
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.21.2031 with grep in /nix/store/jkx9ckqaphfxzrya303ii1jh1j172c7x-jackett-0.21.2031
found 0.21.2031 in filename of file in /nix/store/jkx9ckqaphfxzrya303ii1jh1j172c7x-jackett-0.21.2031

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jackett

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jkx9ckqaphfxzrya303ii1jh1j172c7x-jackett-0.21.2031 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jackett https://github.com/r-ryantm/nixpkgs/archive/a5f612c3f2b7ec5b13e566e4466bc2bdc762f69e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a5f612c3f2b7ec5b13e566e4466bc2bdc762f69e#jackett

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jkx9ckqaphfxzrya303ii1jh1j172c7x-jackett-0.21.2031
ls -la /nix/store/jkx9ckqaphfxzrya303ii1jh1j172c7x-jackett-0.21.2031/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

jackett

Maintainer pings
cc @edwtjo @nyanloutre @purcell for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jackett is: API Support for your favorite torrent trackers
meta.homepage for jackett is: https://github.com/Jackett/Jackett/
meta.changelog for jackett is: https://github.com/Jackett/Jackett/releases/tag/v0.21.2090
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.21.2090 with grep in /nix/store/q9kav02b11jdfcm4956y9v457r0frw4z-jackett-0.21.2090
found 0.21.2090 in filename of file in /nix/store/q9kav02b11jdfcm4956y9v457r0frw4z-jackett-0.21.2090

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jackett

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q9kav02b11jdfcm4956y9v457r0frw4z-jackett-0.21.2090 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jackett https://github.com/r-ryantm/nixpkgs/archive/e5cb1131f25018a4ce6679af8433a1bf212704c1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e5cb1131f25018a4ce6679af8433a1bf212704c1#jackett

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q9kav02b11jdfcm4956y9v457r0frw4z-jackett-0.21.2090
ls -la /nix/store/q9kav02b11jdfcm4956y9v457r0frw4z-jackett-0.21.2090/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

jackett

Maintainer pings
cc @edwtjo @nyanloutre @purcell for testing.

x86 darwin failure is expected: https://hydra.nixos.org/build/253835890


RC-93: finished refactoring UI tests, code coverage is 75%, and Travi…

Fixes issue: RC-
Summary of fix

Add summary here

Checklist:

[ ] Builds on your computer
[ ] Merging to dev and NOT master
[ ] Travis CI passes
[ ] Created new tests if needed
[ ] Modified tests if needed

…s won't run UI tests

Need to mock the podcasts and its episodes. Will do it after Travis finishes processing


update line-arrows example

as same behavior as  #13074
Co-Authored-By: Andreas Hocevar<EMAIL_ADDRESS>

perhaps it will be good to modify some other example who show some drawing actions
as https://openlayers.org/en/latest/examples/measure.html and more ... ?


nipapd: mangle IPv4 address

This mangles IPv4 address in search queries through IPy to get a
properly formatted IPv4 address before we send it to Postgres. We do
validation of IPv4 addresses but once they are verified we send the
original value, so for example <IP_ADDRESS> is recognized as IPv4 but
Postgres doesn't like the leading 0 in the third octet. By running it
through IPy we get rid of that leading 0 and all is fine.
Fixes #860.

Looks nice, merging!


please provide me full working code of cumulative filter, the code which has been given at showcase is not working properly

i searched for cumulative filter  example, but i doesn't get any working code.

Question should be asked on stackoverflow and tagged with yadcf - http://stackoverflow.com/questions/tagged/yadcf , that way more people
will be able to help you and your questions will be easier to find for others
Use github for bug reports or new features request only
In case of bug always post a code sample that resembles your issue or event better provide a link to your page


Problem with getting logout to work properly

I'm having difficulties with implementing the logout functionality of your otherwise awesome IDP.
Currently the result of calling the IDP#logout action with SP SLO object results in this error:
undefined method `authn_request?' for nil:NilClass at the method "encode_response":
def idp_make_saml_response(found_user)
encode_response found_user
end
I've tried to put in a debugger to get a look at what's going on, and currently I'm confused how come the before_filter "before_filter :validate_saml_request, only: [:new, :create]" in saml_idp/app/controllers/saml_idp/idp_controller.rb should not be applied for the logout action as well.
As far as I see, the method 'validate_saml_request' is the only one which calls 'decode_request' in saml_idp/app/controllers/saml_idp/idp_controller.rb which results in setting self.saml_request.
And it seems like the method "encode_response" depends on it being set.
Currently my method "idp_logout" is plain empty, but I don't think that should be a problem, should it?
Thank you in advance.

@LalaDK, I think you're right about validate_saml_request applying to the logout action.
FWIW, if you attempted this now, saml_response would be set per this code which was committed after this issue, but without applying the before_filter the request would be considered an authentication request.
The idp_logout theoretically can be empty if the IdP only does authentication and doesn't store a session with the user being logged in. Otherwise the idp_logout should destroy the current session (or at least log the user out).


flexbox icons

Icon Request

Icon name: flexbox
Use case: page builder
Screenshots of similar icons:

I take these icons from the codepen https://codepen.io/wstone/pen/mOONLK.
flex direction column

flex direction row

wrap for row

justify content for row

align items for row

wrap for column

justify content for column

align items for column

Thanks.

@trungdo-epi Nice request!

@ericfennis Thanks. I am very appreciate lucide team's work.

@trungdo-epi Allright I this is my result. A lot of icons.
Beside flesbox these type of icons are also really usefull for design applications. So I took some inspiration from illustrator, figma and webflow. Webflow had specific flexbox interface like things.

After I designed this Chromium Devtools release a new interface tool for flexbox. I really like these, it also fits the way I designed these I think. Only missing icons are: flex-direction and flex-wrap icons, plus baseline. But baseline icon can be useful for other things as well.

@ericfennis These icons look beautiful and easy to understand.

Fixed with all the PRs above this message.


feat: get batch run result

What I change
Add command to get batch run result
Resolves #7 for the endpoint /v1.0/{organization_name}/{project_name}/batch-run/{batch_run_number}/.
What I checked
Manually checked if the response body obtained by GET request using curl and the one by this command are identical.

@tj-ko is a co-author. I had a pair-programming with him.

Thanks for the review. Fixed with 41e5b8a10b229ebd6d904c718147d2d432366e7f.

API doc should be fixed, while it's not an issue of magicpod-api-client

I agree with it. Created internal issue.


compat sync

Here is a set of small compat syncs. None of them should make a functional difference, most actually just adjust comments.
I was reviewing them and checking that they aren't missing any fixes so I thought I might as well just sync them while I'm there.
One thing missing here is arc4random.c which seems to have diverged quite a bit from all shipped versions. Would there be interest in having that synced to an upstream version too or is there a reason for the divergence?

arc4random has some changes by me to support privsep, fixing some compiler warnings and sweeping some coverity (which I don't really use anymore) under the table.
I'm happy to review any sync changes you want to bring in though.
Thanks for your work so far!


mistral-demo $M7B_DIR issue

Hi I am using Google colab and when i run this command "mistral-demo $M7B_DIR" I use T4 GPU i got this error any solution for that plz
Traceback (most recent call last):
File "/usr/local/bin/mistral-demo", line 8, in
sys.exit(mistral_demo())
File "/usr/local/lib/python3.10/dist-packages/mistral_inference/main.py", line 183, in mistral_demo
fire.Fire(demo)
File "/usr/local/lib/python3.10/dist-packages/fire/core.py", line 143, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "/usr/local/lib/python3.10/dist-packages/fire/core.py", line 477, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "/usr/local/lib/python3.10/dist-packages/fire/core.py", line 693, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/mistral_inference/main.py", line 157, in demo
generated_tokens, _logprobs = generate(
File "/usr/local/lib/python3.10/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/mistral_inference/generate.py", line 49, in generate
prelogits = model.forward(
File "/usr/local/lib/python3.10/dist-packages/mistral_inference/model.py", line 314, in forward
h = self.forward_partial(input_ids, seqlens, cache=cache)
File "/usr/local/lib/python3.10/dist-packages/mistral_inference/model.py", line 296, in forward_partial
h = layer(h, freqs_cis, cache_view)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/mistral_inference/model.py", line 188, in forward
r = self.attention.forward(self.attention_norm(x), freqs_cis, cache)
File "/usr/local/lib/python3.10/dist-packages/mistral_inference/model.py", line 128, in forward
output = memory_efficient_attention(
File "/usr/local/lib/python3.10/dist-packages/xformers/ops/fmha/init.py", line 268, in memory_efficient_attention
return _memory_efficient_attention(
File "/usr/local/lib/python3.10/dist-packages/xformers/ops/fmha/init.py", line 387, in _memory_efficient_attention
return _memory_efficient_attention_forward(
File "/usr/local/lib/python3.10/dist-packages/xformers/ops/fmha/init.py", line 403, in _memory_efficient_attention_forward
op = _dispatch_fw(inp, False)
File "/usr/local/lib/python3.10/dist-packages/xformers/ops/fmha/dispatch.py", line 125, in _dispatch_fw
return _run_priority_list(
File "/usr/local/lib/python3.10/dist-packages/xformers/ops/fmha/dispatch.py", line 65, in _run_priority_list
raise NotImplementedError(msg)
NotImplementedError: No operator found for memory_efficient_attention_forward with inputs:
query       : shape=(1, 28, 32, 128) (torch.bfloat16)
key         : shape=(1, 28, 32, 128) (torch.bfloat16)
value       : shape=(1, 28, 32, 128) (torch.bfloat16)
attn_bias   : <class 'xformers.ops.fmha.attn_bias.BlockDiagonalCausalLocalAttentionMask'>
p           : 0.0
decoderF is not supported because:
attn_bias type is <class 'xformers.ops.fmha.attn_bias.BlockDiagonalCausalLocalAttentionMask'>
bf16 is only supported on A100+ GPUs
<EMAIL_ADDRESS>is not supported because:
requires device with capability > (8, 0) but your GPU has capability (7, 5) (too old)
bf16 is only supported on A100+ GPUs
cutlassF is not supported because:
bf16 is only supported on A100+ GPUs
smallkF is not supported because:
max(query.shape[-1] != value.shape[-1]) > 32
dtype=torch.bfloat16 (supported: {torch.float32})
attn_bias type is <class 'xformers.ops.fmha.attn_bias.BlockDiagonalCausalLocalAttentionMask'>
bf16 is only supported on A100+ GPUs
unsupported embed per head: 128

Same issues here when running locally with a Nvidia Tesla V100. I wonder if there are any ways to change bf16 to fp32 since this might be the culprint (older Tesla GPUs like T4 don't support bf16).

you need to have a NVIDIA A100 machine to fine tune. I faced this earlier with V100. After switching to A100, it worked.


[Feature Request] KEEP_ALIVE_CONNECTIONS

It will be nice if a keepalive config similar to what nginx upstream has gets added to sanic. Basically, it limits the number of idle connections, and closes the least recently used one when the limit is exceeded. I am not entirely sure, but I think it will be more useful than the current KEEP_ALIVE_TIMEOUT config.
reference: http://nginx.org/en/docs/http/ngx_http_upstream_module.html#keepalive

Hi! Is there anyone working on that? I'd like to try to implement that, but not sure if that's desired by core developers.
Also, extra thoughts on realisation appreciated.

@LittleLampLight It does sound like something that should not be too difficult to add.
Take a look at https://github.com/huge-success/sanic/blob/master/sanic/server.py, and you will see that connections currently are handled as a set(). I guess we would need to store them in some sort of an LRU cache instead. Using the standard library module would likely not be enough because we need to remove them ONLY if they are idle, and it probably would not be efficient to maintain both idle and active connections in a separate structure.
I am also curious to see if this is something that any of the ASGI frameworks have already implemented. We plan to make a push towards becoming ASGI compliant after the 1st of the year.


Pull Request : Add SSL support

I ended up not actually needing this, but someone else might want it, so I'm throwing this code over the wall.  One thing to note is that this adds a dependency on the openssl cookbook.  Not sure if that's a deal-breaker or not.

Quick question:

Should it be enabled by default?
Only self-signed supported?

Good idea though, maybe we can expand the support to full certificates :-)

And check the build errors and fix as required :-)

So, funny story.  I needed an SFTP server setup and for whatever reason it didn't occur to me, until right after I added the signed cert, that I was building an FTPS server. :)
I'll add full-ish support for SSL, fix those build errors and send a new pull request later today or tomorrow.  Do you have a preference if I close this PR and open a new one or just commit over this open one?

Feel free to update this one :-)

Just a heads up, this is still on my radar but I've got a fire to put out at work, so it'll probably get done during the weekend.

I've still got to write some integration tests, but how does commit https://github.com/Gazzonyx/chef-vsftpd/commit/906dc73d524d76bf2b7a99d94f7b576599c68541 strike your fancy?

That looks pretty good!

Completely forgot about this, so sorry.


Validation errors for TypeOfFrameRef

I think these errors should be 'corrected', and ideally some external entities should be made present. I also wonder how an ensemble of files could be validated, allowing these external entities to match.
   {
    "name": "xsd",
    "valid": false,
    "error_count": 6,
    "errors": [
     {
      "message": "Element '{http://www.netex.org.uk/netex}TypeOfFrameRef': No match found for key-sequence ['epip:EU_PI_LINE_OFFER', 'epip:1.0'] of keyref '{http://www.netex.org.uk/netex}TypeOfFrame_AnyKeyRef'.",
      "line": 21,
      "type": "xsd"
     },
     {
      "message": "Element '{http://www.netex.org.uk/netex}TypeOfFrameRef': No match found for key-sequence ['epip:EU_PI_STOP', 'epip:1.0'] of keyref '{http://www.netex.org.uk/netex}TypeOfFrame_AnyKeyRef'.",
      "line": 46,
      "type": "xsd"
     },
     {
      "message": "Element '{http://www.netex.org.uk/netex}TypeOfFrameRef': No match found for key-sequence ['epip:EU_PI_COMMON', 'epip:1.0'] of keyref '{http://www.netex.org.uk/netex}TypeOfFrame_AnyKeyRef'.",
      "line": 163,
      "type": "xsd"
     },
     {
      "message": "Element '{http://www.netex.org.uk/netex}TypeOfFrameRef': No match found for key-sequence ['epip:EU_PI_NETWORK', 'epip:1.0'] of keyref '{http://www.netex.org.uk/netex}TypeOfFrame_AnyKeyRef'.",
      "line": 191,
      "type": "xsd"
     },
     {
      "message": "Element '{http://www.netex.org.uk/netex}TypeOfFrameRef': No match found for key-sequence ['epip:EU_PI_TIMETABLE', 'epip:1.0'] of keyref '{http://www.netex.org.uk/netex}TypeOfFrame_AnyKeyRef'.",
      "line": 428,
      "type": "xsd"
     },
     {
      "message": "Element '{http://www.netex.org.uk/netex}TypeOfFrameRef': No match found for key-sequence ['epip:EU_PI_CALENDAR', 'epip:1.0'] of keyref '{http://www.netex.org.uk/netex}TypeOfFrame_AnyKeyRef'.",
      "line": 1005,
      "type": "xsd"
     }
    ]
   }

Hi guys,
I'm very new to NeTEx / EPIP, so be gentle.
The problem
I was using you validation tool - good work BTW, to check validity of experimentally generated timetables and I assume I get the same issues reported as in the OP. XML file attached bellow.
The issues reported are in the same vein as this one:
     {
      "message": "In violation of key-ref constraint, missing key reference \"TypeOfFrame_AnyKeyRef\" (@ref=\"epip:EU_PI_LINE_OFFER\")",
      "line": 7,
      "type": "consistency"
     },

It refers to this part of the XML:

I believe the issues are being reported incorrectly.
Expected behavior
Here is a snapshot from the document: "Public transport - Network and Timetable Exchange (NeTEx)— Part 4: Passenger Information European Profile", chapter 9.7 "Referential integrity".

If I understand it correctly, external references, marked with attribute versionRef shall be ignored by the checker.
I believe this issue (#67) is actually an error in the validation tool and not just an enhancement.
I would very much appreciate if it could be fixed. More so that I’ve noticed that there has been some work done recently in the key ref constraint evaluation JS rule.
If I can be of any help, let me know.
Regards, Roman
Support data
Sample file: netex_epip_rhorenov_02.zip
DATA4PTTools version: 57aa7b7, tag:1.0.4, "Merge pull request #87 from ITxPT/fix/keyref", locally built docker image
OS: Microsoft Windows [Version 10.0.19045.3086]
Command:
docker run --rm -it -v C:\Work\TTR\2023-NeTEx\DATA4PTTools\mytestdata:/usr/local/greenlight/mytestdata netex validate --output json --schema<EMAIL_ADDRESS>--input mytestdata

Result:
[
 {
  "name": "mytestdata/netex_epip_rhorenov_02.xml",
  "valid": false,
  "validations": [
   {
    "name": "everyLineIsReferenced",
    "valid": true
   },
   {
    "name": "everyStopPlaceHasAName",
    "valid": true
   },
   {
    "name": "everyStopPlaceHasACorrectStopPlaceType",
    "valid": true
   },
   {
    "name": "stopPlaceQuayDistanceIsReasonable",
    "valid": true
   },
   {
    "name": "frameDefaultsHaveALocaleAndTimeZone",
    "valid": true
   },
   {
    "name": "everyScheduledStopPointHasAName",
    "valid": true
   },
   {
    "name": "everyStopPointHaveArrivalAndDepartureTime",
    "valid": true
   },
   {
    "name": "locationsAreReferencingTheSamePoint",
    "valid": true
   },
   {
    "name": "everyStopPlaceIsReferenced",
    "valid": true
   },
   {
    "name": "passingTimesIsNotDecreasing",
    "valid": true
   },
   {
    "name": "netexUniqueConstraints",
    "valid": true
   },
   {
    "name": "netexKeyRefConstraints",
    "valid": false,
    "error_count": 6,
    "errors": [
     {
      "message": "In violation of key-ref constraint, missing key reference \"TypeOfFrame_AnyKeyRef\" (@ref=\"epip:EU_PI_CALENDAR\")",
      "line": 63,
      "type": "consistency"
     },
     {
      "message": "In violation of key-ref constraint, missing key reference \"TypeOfFrame_AnyKeyRef\" (@ref=\"epip:EU_PI_STOP\")",
      "line": 110,
      "type": "consistency"
     },
     {
      "message": "In violation of key-ref constraint, missing key reference \"TypeOfFrame_AnyKeyRef\" (@ref=\"epip:EU_PI_NETWORK\")",
      "line": 206,
      "type": "consistency"
     },
     {
      "message": "In violation of key-ref constraint, missing key reference \"TypeOfFrame_AnyKeyRef\" (@ref=\"epip:EU_PI_TIMETABLE\")",
      "line": 2242,
      "type": "consistency"
     },
     {
      "message": "In violation of key-ref constraint, missing key reference \"TypeOfFrame_AnyKeyRef\" (@ref=\"epip:EU_PI_LINE_OFFER\")",
      "line": 7,
      "type": "consistency"
     },
     {
      "message": "In violation of key-ref constraint, missing key reference \"TypeOfFrame_AnyKeyRef\" (@ref=\"epip:EU_PI_COMMON\")",
      "line": 29,
      "type": "consistency"
     }
    ]
   },
   {
    "name": "xsd",
    "valid": true
   }
  ]
 }
]

Hi @skinkie,
and thanks a lot for your feedback regarding the different problematic aspects of the data file in question.

I think your point make sense, but is there any example where typeOfFrameRef with versionRef="any" is used? Because... it really does not make sense to do so in that way.

Do you mean any example of TypeOfFrameRef element with versionRef attribute (disregard the value "any", it's not the issue)?
There are many examples of that in the "Public transport - Network and Timetable Exchange (NeTEx)— Part 4: Passenger Information European Profile (2019)" document.
For example in the chapter "8.4.4 Identifier structure for EPIP version frames" is the following example:

Also the sample EPIP XML file from data4pt.org (is it still valid???) Example XML-file for the EPIP profile
They all use typeOfFrameRef with versionRef to identify the type of the frame (e.g. epip:EU_PI_LINE_OFFER).
As I pointed out in my previous post, how I understand it, the presence of the versionRef in the element (as opposite to the version attribute - they are mutually exclusive) means, that the object it refers to, does not live in the same XML file and shall be ignored by the validator - as stated in the chapter 9.7 in the docs.
Would you agree?
Note, that I have the EPIP specification from 2019, so the state of affairs might have changed since...
Regards, Roman

Do you mean any example of TypeOfFrameRef element with versionRef attribute (disregard the value "any", it's not the issue)?

versionRef is used to refer to an external version, because the version attribute is not present there is no key identity constraint matching going on, and NeTex' special "any" value does not have to be used. So 1.0 it would make sense, any does not.

Also the sample EPIP XML file from data4pt.org (is it still valid???) Example XML-file for the EPIP profile

That example was taken from a national access point, which in effect had many NeTEx issues. My guess is that this example was actually fixed to overcome these issues, it does validate against the full XSD.

They all use typeOfFrameRef with versionRef to identify the type of the frame (e.g. epip:EU_PI_LINE_OFFER).

Yes, but not with any.

As I pointed out in my previous post, how I understand it, the presence of the versionRef in the element (as opposite to the version attribute - they are mutually exclusive) means, that the object it refers to, does not live in the same XML file and shall be ignored by the validator - as stated in the chapter 9.7 in the docs.
Would you agree?

General correct interpretation. I don't know about being mutually exclusive (at least not being enforced in the schema).

Note, that I have the EPIP specification from 2019, so the state of affairs might have changed since...

Documentation could always get better ;-)

I get it, you don't like "any" :-) But it really has absolutely nothing to do with this issue.
The issue is that the following element (and other TypeOfFrameRef/@versionRef):
<TypeOfFrameRef ref="epip:EU_PI_LINE_OFFER" versionRef="1.0" />

causes the validation to fail, although the documentation says it shall pass.
Doc snapshot re. external reference:

Doc snapshot re. checking (more like not checking) the external references (chapter 5.3.3 Integrity Checking of XML Documents):

Updated EPIP XML data, removed "any" from versionRef: netex_epip_rhorenov_03.zip

I get it, you don't like "any" :-)

I like any, but only on version ;)

But it really has absolutely nothing to do with this issue.

I absolutely agree, if versionRef is checked, it is a bug in the validator.


Fixed random typos in the README, readability

Found some typos in the readme, improved readability a tiny bit.

Thank you!


pull-packages: make sure to also update pip packages

This is ready for review, the CI failures are unrelated to this PR.


CU-1p1h2e9 [Wallet] Support two types of data structure deposit trans…

…actions at the same time

Task linked: CU-1p1h2e9 [Wallet] Support two types of data structure deposit transactions at the same time


makepqx should exclude connector.query.pq

I couldn't find a better place to file an issue regarding to MakePQX.exe, so I am submitting an issue here.
MakePQX.exe includes connector.query.pq in the finale .pqx, where Power Query SDK excludes it from .mez. I think MakePQX.exe should exclude it as well.

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

ID: 7cd6b034-1753-6368-19d9-bb699a24b768
Version Independent ID: 5b80c524-3870-f774-e4cd-950e56da9653
Content: Power Query feedback - Power Query
Content Source: powerquery-docs/feedback.md
Service: powerquery
GitHub Login: @bezhan-msft
Microsoft Alias: bezhan

Thanks for looking into this.


Intégrate

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Not a real issue, closing.

How do I fix all the security issues I’m facing from licenses and other contributors ? Sent from my iPhone. El 11 dic 2024, a la(s) 10:25, Steven Woolgar @.***> escribió:
Is there an issue I can help with?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

All my information is exposed in GitHub!

Not quite sure why you are adding issues to this repo. This is clearly unrelated.

You are working over my intellectual property ! Infringement and Security damages Sent from my iPhone. El 11 dic 2024, a la(s) 13:25, Steven Woolgar @.***> escribió
Not quite sure why you are adding issues to this repo. This is clearly unrelated.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Add Page for Debugging / Printing in Move

Url
https://aptos.dev/en/build/cli/working-with-move-contracts#printing-debugging-information
Describe the content issue
We have this page: https://aptos.dev/en/build/cli/working-with-move-contracts#printing-debugging-information, however, i think we would benefit from having a dedicated page for just how to print.
From observing new Move users it seems like they really struggle to do things like debug print and are going to other sites like https://medium.com/cryptocurrency-scripts/aptos-move-strings-and-essential-debugging-tips-850984d5fd66 instead. The fact that this even exists I think shows the need for a page like this on docs.
Section
move-and-smart-contracts

cc @aptos-labs/move-eng


Ensure white background is applied to just the plot and not the entire output area for matplot lib plots

Refs: https://github.com/microsoft/vscode-jupyter/issues/7470

[x] anyOS @misolori

Complexity: 3
Create Issue
Create Issue

Retina display option for Matplotlib does not work as intended
Testing

Install Python
Install Python & Jupyter extension
Change theme to dark (anything other than light/white)
Install Matplotlib

Open an ipynb file
Create a Python cell with the following code & run it

%pip install -U matplotlib

Or use the following documentation (https://matplotlib.org/stable/users/installing.html)

Create a Python cell with the following code & run it

import matplotlib.pyplot as plt
plt.figure()
plt.plot([1,2], [1,2])
plt.show()

The background of the entire output area should not be white (only the plot should be white)

Heres a sample of how it used to render (bug):

Heres a sample of how it should render (after the fix):

Verified that this works as described:

And also with multiple outputs in the same cell: