Ensures all questions are answered before submitting.

Context
The access to data questionnaire needs all questions to be answered before submitting.
Changes proposed in this pull request

Makes all questions required
Verifies only the questions relevant - only checks the government questions when the visitor has said they work for government, and only checks the non-government question when the visitor has said that they don't work for government.
Shows error messages when required fields have not been filled in

Guidance to review
The questions are still skippable by clicking either the ‘Skip this step’ or the ‘I'd prefer not to say’ step.

Do not merge label added, as I haven't run it through any accessibility or browser testing yet.

All testing done; ready for review.


SRE Testing: Emergency upgrade with mainnet snapshot network stall

Performed emergency upgrade with latest TTL snapshot and network was able to load up  and create around 20 blocks - but then all nodes went into CatchUp and network stalled.
continous messages: event took very long to dispatch - StorageRequest, ReactorCrank
Dumps (TTL Snapshot): [(http://genesis.casperlabs.io/ajith-mainnet-clone-1/casper-node-dumps/ajith-mainnet-clone-1/18072024_0156/dump_download_list.html)]
Slack: https://casper-labs-team.slack.com/archives/C01ULKL8G8J/p1721268082469159?thread_ts=1720821743.218009&cid=C01ULKL8G8J

Received the following update from @fizyk20 ;

I'm still doing some work that came out of this ticket, but it's not really directly related to it (I'm porting some changes I made to the global-state-update-gen tool over to the feat-2.0 branch).

As per the chat with Bart and based on his above updates, closing this issue.


DRILL-6084: Show Drill functions in WebUI for autocomplete

Building on top of DRILL-3988 and leveraging DRILL-5868, this allows support for Drill functions to be now available in the WebUI.
If users wants UDFs to show up, they should place the UDF jars in the $DRILL_HOME/jars/3rdparty directory so that this can be loaded during the Drillbit's startup.
Here is a snapshot of the WebUI leveraging the list of functions dynamically generated in the internal javascript and available in the WebUI's auto-complete:

@arina-ielchiieva / @ilooner please review only the last commit, as this is rebased off commit #1483 , which (in turn) is rebased off commit #1484 .
Much of the underlying javascript location, etc is identical to what was done for #1279

@arina-ielchiieva  / @ilooner  can one of you do a review of this minor PR?

That's a good point. Under the sys.functions implementation, there were 2700+ built in functions (when accounting for the different signatures), while  there are over 450 unique function names. From these, I've pruned out non-alphabetical functions, but I am certain there are many functions like converttonullabledecimal18.
While I don't know why such functions exist in the registry if they are not expected to be used, my thinking is that I don't think users will use such functions without knowing what they do. We could define a 'black-list' of such function names, but that adds the task of having to maintain this in the long run.

@kkhatua what if we add one more parameter to the function template, similar to null handling? This parameter will indicate if function is internal or public. Default is public. This would allow you to filter out all internal functions. You would have to add one more column to the sys.functions table as well.
By the way, does sys.functions table have null handling column? I believe it could be useful.

@arina-ielchiieva  made some additional changes:

Introduced the boolean internal field in the sys.functions table. For the WebUI, these functions are not available.
Added tests to validate the internal field.


ch21 full code

just need to put one together.

Appreciate that; as you mention in the book, "transfer learning...is outside of the scope of this book" but it would be nice to play around with the code without having to go back to ch19 and integrate all the changes from that point on.


Oracle 12c EE - ORA-27106: system pages not available to allocate memory

Using Oracle 12c EE version throws ORA-27106: system pages not available to allocate memory due missed parameter in sysctl.conf file. According to Oracle documentation this parameter should be:
vm.hugetlb_shm_group=dba_gid
for example, if groupadd -g 500 dba is Dockerfile.ee:
vm.hugetlb_shm_group=500
at docker run time running as root is fixed by calling:
#!/bin/bash
# fix ORA-27106: system pages not available to allocate memory
sysctl vm.hugetlb_shm_group=500

At Dockerfile.ee script the effective user is oracle, so I propose fixed that using another script running as root user which call finally runOracle.sh script, for example:
Dockerfile
USER	root

VOLUME ["/opt/oracle/oradata"]
EXPOSE	1521 5500

CMD	[ "sh" , "-c" ,  "/home/oracle/manage-oracle.sh" ]

where manage-oracle.sh call sysctl command and then runOracle.sh as user oracle.
sysctl vm.hugetlb_shm_group=500

echo "export ORACLE_SID=$ORACLE_SID" >>/home/oracle/.bashrc
echo "export ORACLE_PDB=$ORACLE_PDB" >>/home/oracle/.bashrc
su - oracle -c "$ORACLE_BASE/runOracle.sh"

I don't think that it is good practice to explicitly modify kernel parameters inside a container as they may affect the unix host as well. And it may require to run the container in privileged mode which is also not desirable.
Docker 1.12 has introduces a parameter for setting sysctl parameters inside the container at runtime: Configure namespaced kernel parameters (sysctls) at runtime. But it appears that the parameter posted above is not (yet) white listed:
[oracle@localhost dockerfiles]# sudo docker run --name oracle-test --sysctl vm.hugetlb_shm_group=500 -p 1521:1521 -v /home/oracle/oradata:/opt/oracle/oradata oracle/database:<IP_ADDRESS>-ee
invalid argument "vm.hugetlb_shm_group=500" for --sysctl: sysctl 'vm.hugetlb_shm_group=500' is not whitelisted
See 'docker run --help'.

I think for now the preferred method is to set host kernel parameters specifically to a value that is desired inside a container.
@Djelibeybi, do you have more input on this?

Can we confirm if setting this inside the container actually changes the host? To be honest, my DB knowledge is pretty slim, so I'm wondering if there is another workaround that doesn't require making this change.
The simplest change is to do at the host level though.

Changing this parameter at the container level change the host level value, here the test case:
root@localhost:/home/data/db# sysctl -a|grep vm.hugetlb_shm_group
vm.hugetlb_shm_group = 0
    -- start a container in another terminal using a regular user --
root@localhost:/home/data/db# sysctl -a|grep vm.hugetlb_shm_group
vm.hugetlb_shm_group = 54322

because by adding --privileged parameter we allow that. This could be a problem in a host with multiple containers requiring this parameter, if there is a workaround for this ora-error rather than changing hugetlb_shm_group any tip will be great.
--sysctl parameter is not allowed at Oracle Container cloud service by now.
Marcelo.

@marcelo-ochoa out of curiosity, what is the value of running Oracle DB on a Docker container inside Container CS, instead of DBCS ?

Well as in my blogpost a Docker image running in Oracle CS service run
faster than in DBAAS, but is only just for fun.
I am preparing another post with my personal test of the CS.
El 12/12/2016 21:39, "Bruno Borges"<EMAIL_ADDRESS>escribió:

@marcelo-ochoa https://github.com/marcelo-ochoa out of curiosity, what
is the value of running Oracle DB on a Docker container inside Container
CS, instead of DBCS ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oracle/docker-images/issues/222#issuecomment-266599763,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQNVhhFHU39pnOOiulqjwzhmB1CX1sGJks5rHeldgaJpZM4K_sk5
.

mind to share the URL to your blog post?

Sure.
El mar., 13 dic. 2016 a las 5:54, Bruno Borges<EMAIL_ADDRESS>escribió:

mind to share the URL to your blog post?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oracle/docker-images/issues/222#issuecomment-266681566,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQNVhvLrPI4hOXrTqmLf5OmmmanY97Trks5rHl1LgaJpZM4K_sk5
.

@marcelo-ochoa, can you also please share a test case and/or background to this?

Yes, my idea is to write another blog post similar to this test including other topics such as private registry usage.

I still struggle to understand the problem here. The container is created with use_large_pages=TRUE (the default) which means that the database may use large pages if available but doesn't have to, see the documentation.
That being said, the documentation for enabling HugePages support does not even mention the parameter vm.hugetlb_shm_group to be changed. So at this point I struggle to understand what would be accomplished by changing this parameter or whether this change is even needed.

OK, so as a workaround the instance created by this Docker image should be changed the dbca.rsp.tmpl file with a parameter  USE_LARGE_PAGES=FALSE, because by default is TRUE.
And telling users that this parameters will affect the performance when using large SGA values.

No. If you follow the link to the documentation you will find that TRUE means that the database can use huge pages if available and can even mix between huge pages and regular pages if there aren't enough huge pages available. However, if there are no huge pages available the database will use regular pages which means that huge pages are not mandatory to be there. Hence why TRUE is the default parameter. If there are huge pages available, use them, otherwise continue with regular pages.
Only if the parameter is set to ONLY the database will accept huge pages and huge pages only. In the case that huge pages aren't available the database will raise an error on startup.
If you read my response above a second time you will also find that all of that has nothing to do with the Linux parameter vm.hugetlb_shm_group as per the second documentation link. So whatever issue you face is not related, at least not directly, to the parameter vm.hugetlb_shm_group.
But given that you still have not shared under what circumstances you do get this error it is hard for me to figure out why you get that error and whether it is legitimate. So please share a test case here in this issue thread and some explanation what you are trying to do that causes this error.

@gvenzl I remember the exception when trying to configure a large SGA and using it, I'll try to reproduce it with a simple test case.

FYI

Some conclusion of 1 month usage of OCCS
How to provisioning latest Oracle Apex distribution on Docker and 11gXE for course setup, development, testing and production environments
Update of the previous post with Apex 5.1 support
Comparing DBAAS with IAAS (Docker on Ubuntu) using 12cEE Docker images
Best regards, Marcelo.

Thanks @marcelo-ochoa,
I will close this issue until you can reconfirm with a test case that this is indeed an issue.


Error 30006: Release version 'service-monitor/latest' doesn't exist

I'm developing bosh development with bosh-lite. When I update the bosh version from 1.3012.0 to 1.3262.4.0, I get this message when run "bosh deploy" commond:
Deploying
Are you sure you want to deploy? (type 'yes' to continue): yes
Director task 19
Started preparing deployment > Preparing deployment. Failed: Release version 'service-monitor/latest' doesn't exist (00:00:00)
Error 30006: Release version 'service-monitor/latest' doesn't exist
The release section is defined in my manifest file as：
release: {name: service-monitor, version: latest}
When I changed the ‘latest’ to actual  version ：
release: {name: service-monitor, version: 0+dev.37}
It works fine.  But I should change the manifest file every time when I update the release, it's hard to use in developing.

@willwillgo should use releases (plural) key instead of release in your manifest.


ACT-695: Add config for slack notification on deployment

What is the Purpose?
Add config to notify slack upon deployment
What was the approach?
Add bash script that sends payload via curl to a Slack channel dedicated for slack notifications
Are there any concerns to addressed further before or after merging this PR?
None will be run by CI tools
Mentions?
@andrewtpham  @odenypeter @nasirhjafri
Issue(s) affected?
ACT-695

Sorry, but why don't we use circle CI orb for slack?
https://circleci.com/orbs/registry/orb/circleci/slack

Why don't we use travis notifications?
https://docs.travis-ci.com/user/notifications/

@nasirhjafri  Good suggestion let me test it out.

@nasirhjafri Good suggestion but Travis has a bit of a bottleneck. They offer notifications per build basis rather than per stage. If I configure it the way you suggest we will have to send only one notification per the whole build. I would prefer using the script method since we have to do another notification for the dev branch for test-coverage on the build and test stage. This is for feature ACT-680.

@Darius-Ndubi Let's use travis notifications for builds. We won't be sending slack notifications for code coverage and will be posting on PR instead. I'll update ACT-680.
cc: @andrewtpham

Okay @nasirhjafri, switching the implementation to use Travis CI slack notation.

@nasirhjafri I have applied the feedback. Kindly review.

Codecov Report

Merging #337 into dev will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #337   +/-   ##
=======================================
  Coverage   50.18%   50.18%
=======================================
  Files          91       91
  Lines        8243     8243
=======================================
  Hits         4137     4137
  Misses       4106     4106

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9782f9e...11d16d8. Read the comment docs.


Important warning in Google Play Console

Can someone explain what currently I should do with my app for solving this problem?
flutter_segment: ^3.5.0

@danielgomezrico Thank you for help 😊

The mesage you show is weird, the version the library use is:
https://github.com/la-haus/flutter-segment/blob/master/android/build.gradle#L37

Ok found out what happened, we merged a dependabot PR that moved the segment version back, my bad, nothing to see here.


Providing sndcpy automotor

Really like your article on Medium and your script are super useful 🎉.
There is sound support also, here Genymobile/scrcpy.
Could you make about this automotor too?
Appreciate your work. Thanks a lot!

I checked  it a bit and it seems that this is not directly supported by scrcpy but by a different tool called sndcpy.
According to sndcpy page it can start from console via sndcpy <serial> so feel free to fork the repository and modify this
line https://github.com/madlymad/mac-scrcpy-app/blob/main/scripts/run_scrcpy.sh#L16.
In that line $d is the device serial number so changing this line to sndcpy $d will start the other application.
In case you want to start both apps then just add the new line and also keep the old one.
If you do not start from scratch you can open the macOS Application with the Automator app and do the modifications there.
In case you need further help let me know.


update fedora version to 34

Signed-off-by: xuezhaojun<EMAIL_ADDRESS>Related issue: https://github.com/open-cluster-management-io/api/issues/104

/assign @qiujian16

/approve
/lgtm


Edges of sites off screen

Operating system: Windows 10
This is:

[x] a bug
[ ] general feedback

The edges of a website are off the edge of the screen.
Here's the same site using Brave and Beaker
Beaker

Brave

test-site.zip

I've noticed this too, on the 0.8.8 release (in Windows 10 as well).

Interesting, thanks. I'll check into it.

Confirmed to occur on Windows (not macOS) when fullscreen. Investigating causes...

Okay thanks for reporting. Looks like a quirk of windows that I was able to solve by using a different bounds-fetching function. Fixed in master!


Skeleton swagger document (no paths, no definitions), no errors: how to troubleshoot

Did a merge to our master branch for a major release, now swashbuckle is generating an "empty" swagger.json document like this:
{"swagger":"2.0","info":{"version":"v1","title":"My API"},"paths":{},"definitions":{}}

There's hundreds of commits here, and no errors, except for the message about "No operations defined in spec!".
How does one troubleshoot when there's no errors or logs?  I have dozens of endpoints that swashbuckle is ignoring.

sure, but most stuff is abstracted out:
	public class Startup
	{
		public Startup(IConfiguration configuration)
		{
			Configuration = configuration;
		}

		public IConfiguration Configuration { get; }

		// This method gets called by the runtime. Use this method to add services to the container.
		public void ConfigureServices(IServiceCollection services)
		{
			services.AddLogging();
			services.AddOData();
			services.ConfigureCors(Configuration);
			//services.AddSignalR(options => { });
			services.ConfigurePostgreSqlDatabaseContext(Configuration);
			services.ConfigureContextAwareness(Configuration);
			services.ConfigureIdentityServices(Configuration);
			services.ConfigureAuthenticationServices(Configuration);
			services.ConfigureCommonServices();
			services.ConfigureCacheSets();
			services.ConfigureSwagger(Configuration);
			services.ConfigureWebApplication();
			services.ConfigureKeyCdn(Configuration);
			services.ConfigureEmailServices(Configuration);
			services.ConfigureMappingProfiles(Configuration);
			services.ConfigureMvc(Configuration);
		}

		// This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
		public void Configure(IApplicationBuilder app, IHostingEnvironment env)
		{
			app.StartLogging(Configuration);
			app.InitializeCacheSets();
			app.StartSwagger();
			app.UseAuthentication();
			app.UseStaticFiles();
			//app.UseSignalR(routes => { routes.MapHub<FeedHub>("/api/feed/live"); });
			app.StartMvc(env);

			if (env.IsDevelopment())
			{
				using (var scope = app.ApplicationServices.GetService<IServiceScopeFactory>().CreateScope())
				{
					var provider = scope.ServiceProvider.GetService<IApiDescriptionGroupCollectionProvider>();
					var loggerFactory = scope.ServiceProvider.GetService<ILoggerFactory>();
					var logger = loggerFactory.CreateLogger<Startup>();
					var sb = new StringBuilder();
					foreach (var descriptionGroup in provider.ApiDescriptionGroups.Items.OrderBy(x => x.GroupName))
					foreach (var descriptionItem in descriptionGroup.Items.OrderBy(x => x.RelativePath,
						StringComparer.OrdinalIgnoreCase))
					{
						sb.AppendLine($"{descriptionItem.HttpMethod} {descriptionItem.RelativePath}");
					}
					logger.LogInformation($"Registered Routes: \r\n{sb}");
				}
			}
		}
	}

swagger config looks like this:
    public static class Swagger
    {
        public static IServiceCollection ConfigureSwagger(this IServiceCollection services, IConfiguration config)
        {
            services.AddSwaggerGen(c =>
            {
                c.IncludeXmlComments(Path.Combine(AppDomain.CurrentDomain.BaseDirectory,
                    "MyApi.App.xml"));
                c.DescribeAllEnumsAsStrings();
                c.SwaggerDoc("v1", new Info {Title = "My API", Version = "v1"});
                c.DescribeAllParametersInCamelCase();
                c.DescribeStringEnumsInCamelCase();
                c.IgnoreObsoleteProperties();
	            c.CustomSchemaIds(t => t.FullName);
            });

            return services;
        }

        public static IApplicationBuilder StartSwagger(this IApplicationBuilder app)
        {
            app.UseSwagger(options => { });
            app.UseSwaggerUI(options =>
            {
                options.SwaggerEndpoint("/swagger/v1/swagger.json", "My API Documentation");
            });
            return app;
        }
    }

mvc config looks like this:
	public static class Mvc
	{
		public static IServiceCollection ConfigureMvc(this IServiceCollection services, IConfiguration config)
		{
			services
				.AddMvc(options => {
					options.Conventions.Add(new ApiExplorerVisibilityEnabledConvention());
				}).SetCompatibilityVersion(CompatibilityVersion.Version_2_1);

			services.AddOData();

			// Workaround: https://github.com/OData/WebApi/issues/1177
			services.AddMvcCore(options =>
			{
				foreach (var outputFormatter in options.OutputFormatters.OfType<ODataOutputFormatter>().Where(_ => _.SupportedMediaTypes.Count == 0))
				{
					outputFormatter.SupportedMediaTypes.Add(new MediaTypeHeaderValue("application/prs.odatatestxx-odata"));
				}
				foreach (var inputFormatter in options.InputFormatters.OfType<ODataInputFormatter>().Where(_ => _.SupportedMediaTypes.Count == 0))
				{
					inputFormatter.SupportedMediaTypes.Add(new MediaTypeHeaderValue("application/prs.odatatestxx-odata"));
				}
			});
			return services;
		}

		public static IApplicationBuilder StartMvc(this IApplicationBuilder app, IHostingEnvironment env)
		{
			if (env.IsDevelopment())
			{
				app.UseDeveloperExceptionPage();
				app.UseCors("AllowAll");
			}
			else
			{
				app.UseExceptionHandler("/Home/Error");
				app.UseCors(options =>
				{
					options.WithOrigins("http://localhost:5000", "https://dev.myapi.com", "https://myapi.com",
						"https://www.myapi.com");
				});
			}

			var builder = new ODataConventionModelBuilder();
			var inflector = new Inflector.Inflector(CultureInfo.CurrentCulture);

			var entityType = typeof(IEntity);
			var types = AppDomain.CurrentDomain.GetAssemblies()
				.SelectMany(s => s.GetTypes())
				.Where(p => entityType.IsAssignableFrom(p) && entityType != p)
				.ToList();

			foreach (var type in types)
			{

				builder.AddEntitySet(inflector.Pluralize(type.Name), builder.AddEntityType(type));
			}

			app.UseDefaultFiles();
			app.UseStaticFiles(new StaticFileOptions{
				ServeUnknownFileTypes = true,
				}
			);

			app.UseMvc(routes =>
			{
				routes.MapODataServiceRoute("odata", "api/odata", builder.GetEdmModel());

				// Workaround: https://github.com/OData/WebApi/issues/1175
				routes.EnableDependencyInjection();

				routes.CreateRoutes();
			});
			return app;
		}

		private static void CreateRoutes(this IRouteBuilder routes)
		{
			routes.MapRoute(
				"default",
				"{controller=Home}/{action=Index}/{id?}");

			//more routes go here.
		}
	}

Swashbuckle is built on top of ApiExplorer. I see a class there called ApiExplorerVisibilityEnabledConvention that looks suspicious. Can you share the contents?

I also see some odd ordering based on ApiDescription GroupName which makes me think you might be setting custom GroupNames in controllers. By default Swashbuckle will use that piece of metadata to determine which actions should be included in a given doc. So, this might also be the cause. As you appear to be doing quite a bit of ApiExplorer customization, I suggest you read the following which describes how SB selects actions for a given Swagger document:
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/blob/master/README.md#generate-multiple-swagger-documents

By default, Swashbuckle uses GroupName to determine how actions should be grouped into different Swagger documents - if it’s null OR equal to the requested documentName (“v1” in your example) it will be included, otherwise it’s omitted. So, by setting GroupName to ControllerName for all actions you’re omitting all actions from the “v1” Swagger document.
May I ask why you’re doing this?

We were having some problems with 404's on some routes in conditions where we wouldn't have access to swagger.json, and needed to document/ verify that routes were properly registered.  This was code suggested by SO to log that info.  Looks like we'll need to rethink that strategy...
Thanks for your assistance.

In my case it was missed [ApiController] at controller definition.
It was work without it in version 2.x and it does not work in version 3.x of lib https://github.com/Microsoft/aspnet-api-versioning
I found it after debug at this line https://github.com/domaindrivendev/Swashbuckle.AspNetCore/blob/28e8673e2579b406bc5120b89589aee3b5b79341/src/Swashbuckle.AspNetCore.SwaggerGen/Generator/SwaggerGenerator.cs#L48


Chore: how dark mode works

Description
This pr adds to the readme.md of the repo, a short explanation of the light and dark mode on the website
This PR fixes #
fixes issue #922
Notes for Reviewers
@leecalcote did I get it right, or I still need to change some things.
Signed commits

[x] Yes, I signed my commits.

Video of the changes I made
screen-capture (2).webm

Thanks. @eeshaanSA @UtkarshMishra12 @Krishna2323 @tinniaru3005 @debo19

Lgtm

LGTM ✅

@Musab1258 DCO is failing. Can you please follow these steps Link
or
To add your Signed-off-by line to every commit in this branch:
Ensure you have a local copy of your branch by checking out the pull request locally via command line.
In your local branch, run: git rebase HEAD~8 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin chore-how-dark-mode-works

@UtkarshMishra12, DCO checks had passed.

@Musab1258 you'll need to only include your commits. Resync your fork in order to avoid having commits from other people show up in your PR.

@Musab1258 you'll need to only include your commits. Resync your fork in order to avoid having commits from other people show up in your PR.

@Musab1258 Please follow these steps 🚀

@leecalcote, I am not getting this right.
I guess I will have to delete this fork, create another fork and start making the pull requests again.

@Musab1258, yes, perhaps so. Although, before you do, please understand that what you've outlined in this PR is how a user can switch between light and dark mode. If we have to explain this to the user, all is lost.
Rather, what the original issue is requesting is developer-facing documentation of how to contribute (code) to the meshery.io website under the newly created dark mode theme. This entails describing how CSS works on the site and how to ensure that style considerations are made for both light and dark mode.
I'll let you close this PR and rethink your approach to the original issue. If it seems daunting or the request remains a bit confusing, don't sweat it. Unassign yourself from the issue, and let's find you another issue to work on. You can do a PR review when another contributor opens a PR against the original issue, if you do choose to remove assignment.

Thanks @leecalcote. Please, let me give it a last shot, if I don't get it by the 31st of december, I will unassign myself.


stucks when power plug insertion ACPI event happens

The following string is printed on the screen and then ly freezes.
[ 9910.556531 ] acpi INT3400:00: Unsupported event [ 0x86 ]
Then the only way to make ly work again is to restart the relative unit via another tty.
Other events also write on the screen but doesn't affect ly.

Could you try compiling the latest version and report if this is still happening? Thank you for your help!

Installed latest version using ly-git aur package after a full system upgrade.
Now the acpi string doesn't show anymore, but when i insert the power plug the cursor goes in low left of the screen and then ly stop reacting to keypresses.

I think it's not Ly's fault, can you check your /etc/systemd/logind.conf and try again after setting the event to "ignore" ?

This is xev's output when the plug is inserted, maybe it can be useful:
MappingNotify event, serial 58, synthetic NO, window 0x0,
    request MappingKeyboard, first_keycode 8, count 248

KeyPress event, serial 58, synthetic NO, window 0x3200001,
    root 0x199, subw 0x0, time 9313421, (106,89), root:(157,498),
    state 0x0, keycode 248 (keysym 0x0, NoSymbol), same_screen YES,
    XLookupString gives 0 bytes:
    XmbLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyRelease event, serial 58, synthetic NO, window 0x3200001,
    root 0x199, subw 0x0, time 9313421, (106,89), root:(157,498),
    state 0x0, keycode 248 (keysym 0x0, NoSymbol), same_screen YES,
    XLookupString gives 0 bytes:
    XFilterEvent returns: False

I think i've managed to fix this.
To confirm my assumptions can you explain what's the purpose of the condition in
src/main.c, line 95
while((input_key = wgetch(win.win)))
Other than acquiring user input it is meant to terminate the program if an input error occurs, right ?


open in app with nothing when f7-view set browserHistory to true

Framework7 version: latest
Vue.js version: latest
Platform and Target: [Android]

Describe the bug
load pages in android device with webview, but there is show nothing on webview when "f7-view" set browserHistory to true
Expected behavior
show pages correct
Actual Behavior
show pages incorrect

Your issue is closed because it is not filled correctly. Before you open an issue please review the contributing guideline.
Please fill the issue template correctly and:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with isssue

It will save a lot of time for other contributors to check your issue and fix it.


Apply for publisher verification

https://docs.github.com/en/apps/github-marketplace/github-marketplace-overview/applying-for-publisher-verification-for-your-organization
My organization requires third parties actions to be from verified publishers. This currently blocks me from using the promptfoo-action in my repos. It would help us if promptfoo would apply for publisher verification.

+1 we also have the same problem, would love to use this GitHub Action, but our organization requires verified publishers. Is there anyway we could help get this verified?
Thanks


Add Article for Hello World in ALGOL 68

Merged #138. Thanks, @trevershick.

Looking to write this one up next. I currently have an article going out tomorrow and Friday, so I'm shooting for a Sunday release of this article.
May 20th, 2018

Almost done with this article. I just need to write up some background and find an image. Should be edited and complete tomorrow--published Sunday.

Added link to README. Done! See ya tomorrow.


Tag regex with match groups and reformatting

Hey,
Great tool!  I've been using this to recapitulate the results in the bioRxiv paper and to perform similar multiplexing-via-tag experiments.
One limitation that I ran into with the initial implementation was that the script expected the tag sequence to start at position 0 of read 2.  For some experiments we performed, the variable portion of the tags were downstream of position 0.  To get around this, I changed the regex from matching to searching using regex groups.  The current implementation isn't super robust, but will check that exactly one match group is supplied in the compiled tag regex.
Otherwise, I just reformatting things so that pep8 wouldn't complain.
I tested this implementation versus the original on some test data, but it may need to be further validated before merging.
Edit:  You will also want to change the Running-the-script wiki page from:

[Required] Structure of the sequenced antibody-oligo tag in regular expression. This filters potential PCR artifacts (without a polyA tail) from true ADT reads (containing a polyA tail). For 15 nucleotide antibody barcodes and a read 2 of at least 21 nucleotides this would be "^[ATGC]{15}[TGC][A]{6,}"
-tr TAG_REGEX, --TAG_regex TAG_REGEX
Example: "^[ATGC]{15}[TGC][A]{6,}"

to

[Required] Structure of the sequenced antibody-oligo tag in regular expression. This filters potential PCR artifacts (without a polyA tail) from true ADT reads (containing a polyA tail). For 15 nucleotide antibody barcodes and a read 2 of at least 21 nucleotides this would be "^([ATGC]{15}[TGC])[A]{6,}"
-tr TAG_REGEX, --TAG_regex TAG_REGEX
Example: "^([ATGC]{15}[TGC])[A]{6,}"

Hey @wflynny
I have a question for you. Would you be able to send me a sample (10k reads is enough) of your data with the associated tags?
I want to test it out directly with the updated script. I'm also implementing an automatic regex generation based on the tags. So maybe I have to adjust it to your type of samples.
Thanks

Hey @Hoohm.  Let's follow up via email: bill dot flynn at jax dot org?
I have permission to selectively share fastqs from some experiments, but don't yet have disclosures yet for the experiments you're probably most interested in, e.g. tags with variable portions far down on read2, though I'm working on getting permission to share those too.


Most tox environments are broken

__________________________________________________ summary __________________________________________________
ERROR:   py27-WTForms1: could not install deps [WTForms==1.0.5, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError('/home/jugmac00/Projects/flask-admin/.tox/py27-WTForms1/bin/python -m pip install WTForms==1.0.5 -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt', 1)
ERROR:   py27-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py27-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
SKIPPED:  py35-WTForms1: InterpreterNotFound: python3.5
SKIPPED:  py35-WTForms2: InterpreterNotFound: python3.5
ERROR:   py36-WTForms1: could not install deps [WTForms==1.0.5, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError('/home/jugmac00/Projects/flask-admin/.tox/py36-WTForms1/bin/python -m pip install WTForms==1.0.5 -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt', 1)
ERROR:   py36-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py36-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
ERROR:   py37-WTForms1: could not install deps [WTForms==1.0.5, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError('/home/jugmac00/Projects/flask-admin/.tox/py37-WTForms1/bin/python -m pip install WTForms==1.0.5 -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt', 1)
ERROR:   py37-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py37-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
ERROR:   py38-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py38-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
  flake8: commands succeeded
  docs-html: commands succeeded

That is kinda surprising as the tests on Travis seem to work fine.

Still broken
ERROR:   py27-WTForms1: could not install deps [WTForms==1.0.5, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError('/home/jugmac00/Projects/flask-admin/.tox/py27-WTForms1/bin/python -m pip install WTForms==1.0.5 -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt', 1)
ERROR:   py27-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py27-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
ERROR:   py35-WTForms1: could not install deps [WTForms==1.0.5, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError('/home/jugmac00/Projects/flask-admin/.tox/py35-WTForms1/bin/python -m pip install WTForms==1.0.5 -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt', 1)
ERROR:   py35-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py35-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
ERROR:   py36-WTForms1: could not install deps [WTForms==1.0.5, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError('/home/jugmac00/Projects/flask-admin/.tox/py36-WTForms1/bin/python -m pip install WTForms==1.0.5 -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt', 1)
ERROR:   py36-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py36-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
ERROR:   py37-WTForms1: could not install deps [WTForms==1.0.5, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError('/home/jugmac00/Projects/flask-admin/.tox/py37-WTForms1/bin/python -m pip install WTForms==1.0.5 -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt', 1)
ERROR:   py37-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py37-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
ERROR:   py38-WTForms2: could not install deps [WTForms>=2.0, -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt]; v = InvocationError("/home/jugmac00/Projects/flask-admin/.tox/py38-WTForms2/bin/python -m pip install 'WTForms>=2.0' -r/home/jugmac00/Projects/flask-admin/requirements-dev.txt", 1)
  flake8: commands succeeded
  docs-html: commands succeeded

After some debugging I found out that - at least on Ubuntu - you need to sudo apt install libpq-dev in order to run the test suite.
That is still not enough to run all tests successfully.
Besides other messages I also see psycopg2.OperationalError: fe_sendauth: no password supplied. Any help on this?

CI flow is moved to Github Actions and pipelines are green now. @mrjoes we can close this issue.


X Ray Image Query: Change location of units in bp output

Description

Shifting things around before release. I changed where units show up in the blueprint output; it should make more sense now.

Type of change

[ ] Bug fix
[ ] New feature
[ ] Documentation update
[x] Other~~  a change

How Has This Been Tested?

built ran rztopaz; all tests new and old pass
Checklist:

[x] I have followed the style guidelines of this project.~~
[x] I have performed a self-review of my own code.~~
[x] I have commented my code where applicable.~~
[ ] I have updated the release notes.
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[x] I have added tests that prove my fix is effective or that my feature works.~~
[x] I have confirmed new and existing unit tests pass locally with my changes.~~
[ ] I have added new baselines for any new tests to the repo.
[x] I have NOT made any changes to protocol or public interfaces in an RC branch.~~
[x] I have assigned reviewers (see VisIt's PR procedures for more information).~~

merge w/ develop in #18342


JOSS Review: Local installation hangs

Related to the JOSS Review (openjournals/joss-reviews#2105). When I want to run the application locally, the application gets stuck loading a network.

These are the commands I entered in order to run the application:
git clone https://github.com/ToyotaResearchInstitute/materialnet.git
cd materialnet
yarn install
yarn start

Any ideas on how I can get this to work?

Sorry about this. I will update the instructions and provide a data bundle suitable for local builds. I will report back here (via a pull request) when it's ready.

@rcannood, please try the build one more time using this branch: https://github.com/ToyotaResearchInstitute/materialnet/tree/data-bundle.
The README on that branch contains updated build instructions, and a pointer to a data bundle you can use to resolve this issue. If you run into trouble, please let me know and I will edit that branch until things are working.

It works perfectly now, thanks!


merge from template-sync/template-arlac77-github

package.json

chore(deps): add ^15.5.4 remove ^15.5.3 (devDependencies.npm-pkgbuild)

.github/pr_labeler.yml

chore(labels): merge from template .github/pr_labeler.yml

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

npm package (@latest dist-tag)
v1.11.63
GitHub release

Your semantic-release bot :package::rocket:


preview.png has outdated copyright year

See: https://github.com/denoland/deno_blog/blob/4bbfc34de59e7648c909207b9687f7f959fb506d/.github/preview.png

I think this file may not be used now 😄


Unit tests for the executor

Fixes #6

[ ] I have added the tests to cover my changes.
[ ] I have updated the documentation and CHANGELOG accordingly.
[ ] I have read the CONTRIBUTING document.

@venkatBala Could you also run a workflow to do a manual functional test? Other than that this is ready to go.


Can I get g:dein#plugin in hooks?

Since 19e721a, build dict is disabled in plugin setting. I wrote setting to build vimproc by hook_post_update with referring to doc.
[[plugins]]
repo = 'Shougo/vimproc.vim'
hook_post_update = '''
  if dein#tap('vimproc.vim')
    if dein#util#_is_windows()
      let cmd = 'tools\\update-dll-mingw'
    elseif dein#util#_is_cygwin()
      let cmd = 'make -f make_cygwin.mak'
    elseif executable('gmake')
      let cmd = 'gmake'
    else
      let cmd = 'make'
    endif
    let g:dein#plugin.build = cmd
  endif
'''

I could get the desirable result, but this if dein#tap() block seems redundant to me.
Can I use the plugin dict with more simple way? Such as...
# local variable `plugin` has the `g:dein#plugin` value
hook_post_update = '''
  let plugin.build = 'make'
'''

I think this can be useful in other cases.

Implemented.

thx a lot!
ref 7679fc0479381fada06425c2efd8197e215510e0


Begin #12704 Update jnr-ffi to 2.2.9

What changes are proposed in this pull request?
This fixes AlluxioFuseFileSystemTest on Linux ARM64
Also see https://github.com/jnr/jnr-ffi/pull/284
While here also update jnr-fuse to 0.5.7
Why are the changes needed?
Adds a dependencyManagement for jnr-ffi.
jnr-ffi:2.2.9 brings a fix for read/write int64_t on Linux ARM64.
Does this PR introduce any user facing changes?
No user facing changes!

Any feedback ?


How to map from Alamofire JSON response AnyObject?

As the title says. How do I map from Alamofire JSON response AnyObject?
Alamofire.request(.GET, URL + "Timetable").responseJSON { (_, _, JSON, _) in
let timeTable = Mapper().map(JSON, toType: Timetable.self)
}
JSON is AnyObject? but the map function does not accept that. How can I proceed?

Thank you :+1:


Issue while installing pytorch-forecasting through pip

PyTorch-Forecasting version: Can't Install due to wheel issue.
PyTorch version: Can't install same wheel issue
Python version: 3.11.5
Operating System: Windows

Issue faced while installing pytorch-forecasting

I have already installed Microsoft Visual C++ 14.0
Issue faced while installing pytorch

Cannot debug this issue.

@DS-mehul try the --no-cache-dir option of pip install.

Similar issue here. For me it fails on the scikit-learn build, which appears to be an older version.

Same.

See https://pytorch.org/get-started/locally/. You install pytorch by pip install torch.

@DS-mehul, we released a new version of pytorch-forecasting on pypi. Can you check if with the new version this issue still exist?


python311Packages.yalexs-ble: 2.2.2 -> 2.2.3

Diff: https://github.com/bdraco/yalexs-ble/compare/refs/tags/v2.2.2...v2.2.3
Changelog: https://github.com/bdraco/yalexs-ble/blob/v2.2.3/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 245701 run on x86_64-linux 1

6 packages built:

home-assistant-component-tests.august
home-assistant-component-tests.yalexs_ble
python310Packages.yalexs-ble
python310Packages.yalexs-ble.dist
python311Packages.yalexs-ble
python311Packages.yalexs-ble.dist


Storybook docs height with Firefox

The problem :
When I'm on Firefox, Storybooks docs height looks minimal. In comparison at Chrome, docs height fit the content.
With Firefox :

With Chrome :

I don't have any extensions, and my cache is clear 😞

Same for me

This line in preview.js is not applied on firefox

I did reproduce, but I don't think padding is the cause of the problem. I agree that DevTools are saying so, but we can clearly see the padding by inspecting the computed tab. (Also, on localhost, DevTools are not saying padding have problems)

I did test that in production (https://demo.start-ui.com/storybook), it looks like it has been fixed, I'm closing this issue. Feel free to reopen if it happens again.


General changes to make the cookbook to work on chef12

Chef12 loads the resources in a different way. So, these are minor changes to make sure it works fine on chef12

Thanks again @alejandrod. We'll review and publish your PR as soon as possible.

Hey, I'll be great if you can merge this. Otherwise is not possible to use the cookbook

I'm sorry this issue has lagged so far behind. Changes to our infrastructure have left our own Chef usage a bit in the stone-age. I'm happy to merge this pull request, but I'd feel more confortable doing so with the knowledge that it's still desired and verified from multiple sources. So anyone interested (and especially who have tested this patch). Please comment.

As I said. I've been using this for a while, no problem. The only problems I had when upgrading Chef. So, that's the reason of the changes.

Could you please merge? thanks!


Download via WebUSB feature

Description
Automatically use WebUSB to download your code to the device via WebUSB
How to use it?
In Chrome, click on Download to initiate the download and follow the pairing instructions.
If you cannot see that button and you editor supports this experiment, go to About, then click Experiments and Download WebUSB. If you cannot see this card, your editor does not support yet this feature.
Feedback
Do you use this feature? Is there something you'd like to change? Did you encounter issues? Please drop a comment on this issue to send us your feedback.

i am using the latest firmware version( 0250), it works on the normal version, but not the /beta version.

Oh, and the "console" function does not work properly
I saw it could sync with my device but in reality, no.

Editor supports this now.

@abchatra  Remove experiment from experimet.ts when closing these.


rename NoteCommitmentProof to StateCommitmentProof

Now that the TCT will store not just note commitments but swaps also, and in the future other bits of state, we should rename the NoteCommitmentProof to StateCommitmentProof everywhere to avoid confusion.

Should we also rename "note commitment tree" to "state commitment tree"?


add the flow to the Node-RED Library

https://flows.nodered.org/

Done:
https://flows.nodered.org/node/node-red-contrib-enocean


Added yet another Dribbblish option

active_control_fg codes the color of the shuffle and repeat button once activated.

@khanhas how can we access the colors of the scrollbar ? I couldn't find it in user.css.

@spoutn1k use DevTool to find it


⚙ latest release broke the site

Describe the bug
Unable to render site
To Reproduce
update to latest release
run hugo server
you will get
themes/blowfish/layouts/_default/list.html:23:19": execute of template failed: template: _default/list.html:23:19: executing "main" at <.File.Path>: error calling Path: runtime error: invalid memory address or nil pointer dereference
Built in 172 ms
Error: error building site: render: failed to render pages: render of "section"
themes/blowfish/layouts/_default/list.html:23:19": execute of template failed: template: _default/list.html:23:19: executing "main" – File is nil; wrap it in if or with: {{ with .File }}{{ .Path }}{{ end }}
Expected behavior
It should render site
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Linux

Hugo & Blowfish versions
hugo 124, blowfish v2.62.0,

I get the same thing.
ERROR render of "section" failed: "[REDACTED]/themes/blowfish/layouts/_default/list.html:23:19": execute of template failed: template: _default/list.html:23:19: executing "main" at <.File.Path>: error calling Path: runtime error: invalid memory address or nil pointer dereference
Built in 1060 ms
Error: error building site: render: failed to render pages: render of "section" failed: "[REDACTED]/themes/blowfish/layouts/_default/list.html:23:19": execute of template failed: template: _default/list.html:23:19: executing "main" – File is nil; wrap it in if or with: {{ with .File }}{{ .Path }}{{ end }}

I have the same issue

@BabaRagnarok @monstermuffin I am doing a release right now that includes a fix... (or potential fix as I never eas able to experience the issue) could you help with testing it?

@BabaRagnarok @monstermuffin I am doing a release right now that includes a fix... (or potential fix as I never eas able to experience the issue) could you help with testing it?

I can confirm the issue is fixed for me with this release, thanks!

@monstermuffin @nunocoracao Yes now is working thanks

Closing the issue thanks


Stringify renamed_from value

Fix for https://github.com/winebarrel/ridgepole/issues/125

Coverage increased (+0.007%) to 96.663% when pulling 59c2b9af1c5431308c20cdbeaad5b733b608d80f on stringify_renamed_from into be840bed5d210ad2f165be041d76151b291b1e8f on 0.6.

Coverage increased (+0.007%) to 96.663% when pulling 59c2b9af1c5431308c20cdbeaad5b733b608d80f on stringify_renamed_from into be840bed5d210ad2f165be041d76151b291b1e8f on 0.6.


Question : How to create the AST for typescript file content passed as string

I have an usecase where the ts file is in server and I get the string content of this. Is it possible to build an AST using the string content instead of the file path or tsconfig ?

I do this all the time in tests.
export function createProjectFromString (fileContent: string): {
  project: Project,
  sourceFile: SourceFile,
} {
  const project = new Project({
    useVirtualFileSystem: true,
    manipulationSettings: { indentationText: IndentationText.TwoSpaces },
  })
  const sourceFile = project.createSourceFile('__temp__.ts', fileContent)
  return { project, sourceFile }
}

Thanks @lazarljubenovic! That's exactly right. You can use the createSourceFile function on an instance of the default export—Project.

Thanks a lot @lazarljubenovic  & @dsherret . It worked. My suggestion if this option can be mentioned in https://dsherret.github.io/ts-simple-ast/details/source-files it will be helpful.

@SanthoshHari912 thanks, that makes sense. I will provide a link there to https://dsherret.github.io/ts-simple-ast/setup/adding-source-files -- Opened #387.


WIP implementation for CRI using rkt cli / systemd-run

Pulled from #6 , made some small changes, and added code to run rkt commands under systemd-run.
Depends #15 to get in first.
cc @s-urbaniak @lucab

@yifan-gu  I think we can merge it so we can move forward. We can just create issues for the things I mentioned

@yifan-gu I think we can merge it so we can move forward. We can just create issues for the things I mentioned

OK, let me fix things you mentioned in the comments shortly. I actually pushed more commits on implementing other interfaces. But we can split them into follow up PRs.

hmm, gihub needs to add removing approval option ;)
I took a look on new commits and the are looking OK.

OK, merging this.


Update requirements for teams property

Make it clear that if your organisation has enabled teams, that the property becomes required for pipeline creation, not just that it becomes available.
Trying to create a pipeline without passing the teams property in an org with teams enabled results in a 422 error.

Hi @fd-jonathanlinn, thanks for taking the time to put this together! Can you please share a bit more about the error you received? I tried to confirm this case internally but was told that you can create a new pipeline without the teams property, even with teams enabled.

Hey @mbelton-buildkite no problem, so when trying to create a pipeline within our org via the UI or API it would refuse to create until we selected/provided a Team.  I've just tried to create a pipeline again via the UI without a team and that works fine:

But through the API I still see the same problem:
{
    "message": "Validation Failed",
    "errors": [
        {
            "field": "teams",
            "code": "Please ensure at least one team is added to this pipeline",
            "value": []
        }
    ]
}

This happens both when I omit the teams key, and when I include it as an empty object.  It works fine if I provide a team UUID and access level within the teams object.

We've actually just had a user experience the same issue through the UI where they could not create a pipeline without selecting a team - let me know if it you would like me to share details about this to yourselves via the support/hello email addresses

Thanks for the extra context @fd-jonathanlinn! I've looked into this more and learned that there's another permission involved. If you've enabled teams and are an administrator, you don't need to provide the teams. Otherwise, you do. I'll suggest how we can word that in this PR.
To sort out the inconsistent behavior you're seeing between the UI and the API, it would be best for you to email Support<EMAIL_ADDRESS>

Both changes look great, thanks for clarifying the permissions, all makes sense now 😁

Hmmm the check isn't showing the build result, so the merge conditions aren't met. I'll override and merge it for you.


BUG-000103133  The Public Notification template incorrectly prints Avery labels for 'Property Occupants'

Salesforce ID: BUG-000103133
Salesforce Submitter: Bradley Snider
Salesforce Submit Date: 2/23/2017 11:35 AM
Salesforce Bug Type: Usability
Salesforce Severity: Medium
Repro Data: (n/a)
Work Around: (n/a)
Product: ArcGIS for Local Government
Functional Category: ArcGIS for Local Government
Client Platform: (n/a)
Version Found: 10.5
Planned Version Fixed: (n/a)
Comment: (n/a)

Duplicate of #74


graph/prelude.resolvers.go:19:44: __DirectiveResolver not exported by package generated

What happened?
When I do fastgql generate I get the error: graph/prelude.resolvers.go:19:44: __DirectiveResolver not exported by package generated
What did you expect?
Files be generated
Minimal graphql.schema and models to reproduce
versions

gqlgen version? v0.13.0-dev
go version? 1.17
dep or go modules? modules

How did this issue solve for you?


fix(core): use caret version for swc helpers

Summary
Using ^ for deduplication
Checklist

[x] Tests updated (or not required).
[x] Documentation updated (or not required).

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


Build: Fix SDK for nuget publish

Description

Currently dotnet pack fails on SDK 8 (MSBuild 17.8.3+195e7f5a3) in the CI environment (Linux problem?)
This PR fixes the dotnet pack to use SDK 7.0.403 in the CI environment as a temporary fix.
All local builds and pack work OK for me on mac M2 so I didn't see the need to fix the whole solution.
How Has This Been Tested?

Run github action on my fork
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

OK I see that the build failed on this too.  To me this seems like a MSbuild on Linux issue.  Maybe we should add the workaround to the build as well?  Should we just do it on CI?

Is it possible to downgrade to net7?


feat: otel oathkeeper tracing

@krtk6160 ,
Today the oathkeeper codebase got a PR to fix OTEL tracing here https://github.com/ory/oathkeeper/pull/1049 . I compiled the source and tested it locally and was able to see tracing flowing thru honeycomb!

I pushed a docker image here: https://hub.docker.com/layers/ntheile/oathkeeper/v0.40.1/images/sha256-2aced8081b5fc8648bf9d48affb59fe4245c64b449ca82a81e10f93474331f14?context=explore
@nicolasburtey  wanted to test this in staging. can you add this to the staging deployment and see if tracing work?
Thanks,

@ntheile https://github.com/GaloyMoney/charts/pull/2330 overrides the oathkeeper image to use your custom image


Ask for default when adding new required property

EdgeDB Version: 1-beta.1
EdgeDB CLI Version: beta.1
OS Version: macOS Big Sur

Steps to Reproduce:
I made a minimal repro of how I got my schema into an inconsistent state.

Create an initial schema

module default {
    type BlogPost {
        property title -> str;
    }
};

Add some data;

INSERT BlogPost;

Update

module default {
    type BlogPost {
        property title -> str;
        required property name -> str;
    }
};

Run create-migration (works fine). If you make an existing non-required property required, the migration tool prompts you for a default, but not if you define a new required property.

Inconsistent state:

Running migrate:
$ edgedb -I inconsistent migrate
edgedb error: ERROR: MissingRequiredError: missing value for required property 'name' of object type 'default::BlogPost'

Runing create-migration (e.g. after setting a default for .content):
$ edgedb -I inconsistent create-migration
edgedb error: Database must be updated to the last migration on the filesystem for `create-migration`. Run:
  edgedb migrate

I have fixes for the updated test case where we add a new required property, but frustratingly, the original issue of adding a property via a new parent class might be a bit trickier. I'm going to spend a couple hours looking at it, but I might put up a partial fix without handling that case yes.


V2 - Company Not Found - Not able to proceed with the flow after creating company due to CORS issue

Prerequisite

Create a Foundation Level CLA group with CCLA enabled. Foundation has 2 suborojects but only 1 subproject is enrolled in the CLA Group
Add github repository for the CLA group
Generate template

Steps to Follow

Login to Github as corporate contributor
Navigate to the repository used in prerequisite
Create PR
CLA check fails
Click on CLA NOT SIGNED
Contributor is redirected to Contributor Console
Click on Proceed as a Corporate Contributor
Click on building icon and click on +Click Here
Enter new company details
Click Next
Click Proceed
Observed no action is performed and CORS issue is displayed in console

Verified that the issue is fixed


fix: TS1479 error

Fixes https://github.com/ivanhofer/typesafe-i18n/issues/756.
Key Change:

Addition of "type": "module" in package.json.

Issue Background:

The problem arises due to TypeScript's reliance on the package.json type field to determine the module format (CommonJS or ECMAScript Module, ESM) of a project.
In the absence of the type field, TypeScript, along with Node.js, defaults to interpreting modules as CommonJS. This becomes an issue when dealing with .d.ts files, as they do not specify the module system (like .mjs or .cjs would).
Consequently, when an ESM is imported in such a scenario, TypeScript incorrectly assumes it to be a CommonJS module, leading to module resolution conflicts described in https://github.com/ivanhofer/typesafe-i18n/issues/756.

Thank you for considering this pull request!

Ivan Hofer passed away. Im not sure there is any one that can approve this PR

Actually, it's incorrect solution as well.
Read more here:

https://github.com/unjs/unbuild/issues/238
https://twitter.com/atcb/status/1634653474041503744
https://github.com/microsoft/TypeScript/issues/50466


feat: Add scrape config for self-monitor

Description
Changes proposed in this pull request (what was done and why):

Add scrape-config for self-monitor
Add the missing servicer for self-monitor

Changes refer to particular issues, PRs or documents:

https://github.com/kyma-project/telemetry-manager/issues/822

Traceability

[x] The PR is linked to a GitHub issue.
[ ] New features have a milestone set.
[ ] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[ ] The corresponding GitHub issue has a respective area and kind label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] Adjusted the documentation if the change is user-facing.
[x] The feature is unit-tested
[ ] The feature is e2e-tested

/unhold


Refactor ChatGPT function calling

Problem:
The implemented functions are not providing the information to the AI model.
Changes:
Based on https://platform.openai.com/docs/guides/function-calling ChatActivity and AssistantFagments were refactored, so the system message calling for the function and the result of the functions are added to the chatMessages
A new function, getCurrentTime added.
The result

Other changes will be merged manually soon.


How do I log into my own codeforces account and show the information that I have completed problems or contests?

You cannot log into the Codeforces account using this site. It will show you the publicly available information available on CF API. Currently, you can't mark any problem or contest as completed unless it is completed in CF.


Integration needs reloading

Checklist

[X] I have updated to the latest available Proxmox VE Custom Integration version.

Describe the issue you are experiencing
Following setup :
Home Assistant running on a Rpi4
Proxmox running on an old PC
The integration works but if I power down the proxmox server and power it back on the integration needs reloading in order to get it to work again. Otherwise all the sensors and buttons are stuck in unavailable mode.
In which version of Home Assistant Core do you have the problem?
2023.10.3
What version of Proxmox VE Custom Integration has the issue?
2.0.4
What version of Proxmox VE do you have the problem?
8.0.4
Additional information
Only tested with this version so I have no idea if this was an issue before.

Duplicated with #93.


[11.x] Fix getDefaultUserProvider docblock

Currently, creating user provider using createUserProvider without explicitly passing its name will always return null unless auth.defaults.provider config entry is defined (which is not defined out-of-the-box btw).
The reason is, getDefaultUserProvider method - contrary to its name and typehint - returns null when said config entry is missing.
This PR changes that to fall back to default 'users' provider instead of null.

Not sure, I follow the reasoning for this change.
Maybe the auth.defaults.provider key is left blank intentionally, and thus no provider should be returned.
What are the benefits of having an out-of-the-box user provider? Granted that each shipped guard defines their own provider.
What issues does this change solve?
Also, defaulting to some hard-coded value doesn't seem intuitive to me, if someone does not expect it to return anything, hunting for this value could be cumbersome.
As this is also a change in default behavior of a public method, maybe targeting 12.x and adding a note to the upgrade guide is a better idea.

Maybe the auth.defaults.provider key is left blank intentionally, and thus no provider should be returned.

I found original PR (#18856) and I think your point is valid.
My use case was to use CreatesUserProviders trait outside auth subsystem as thin abstraction layer over Eloquent to find users by their credentials. Maybe that's not intended use case, but anyway, anyone who want to define default user provider may override getDefaultUserProvider method.
I've reverted my changes and instead fixed getDefaultUserProvider docblock to reflect that null is allowed return value.
I also changed GuardHelpers::setProvider method so it is now chainable.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!


Don't log fatal after one try and write logs to a file on client side

This change is

We should do similar handling for sendAnswer too. Try a few times, if it doesn't work show him final box and ask him to send over the log file to us.

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

client/main.go, line 86 [r1] (raw file):

		&interact.Req{Repeat: false, Sid: s.Id, Token: *token})

	for err != nil {

We should probably have a limit on the number of tries here. If there isn't any response after a while from the server, we should display him the final screen box that we do if his token is invalid for e.g. We can ask him to contact us in that box.
Also did you test it out by say disconnecting the server for a sec and see if this works.

client/main.go, line 384 [r1] (raw file):

	try := 0
	conn, err = grpc.Dial(*address, grpc.WithInsecure())
	for err != nil {

Does this work
for err != nil || try <10 {

}

If it does we can move the log.Fatal  outside the loop. Infact instead of log.Fatal we could again show the box with a error message.
The user won't have any visual indication during this period. So its better if we keep the sleep down to 1 second? If he can't connect in 10 seconds I am sure there is a problem at our end.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

client/main.go, line 86 [r1] (raw file):
Previously, pawanrawal (Pawan Rawal) wrote…
> We should probably have a limit on the number of tries here. If there isn't any response after a while from the server, we should display him the final screen box that we do if his token is invalid for e.g. We can ask him to contact us in that box.
>
> Also did you test it out by say disconnecting the server for a sec and see if this works.

The client doesn't crash if the server crashes, but if the stream is broken, the client has to re-establish it and not the server. We need it handle it separately. This is only for temporary network errors.

client/main.go, line 384 [r1] (raw file):
Previously, pawanrawal (Pawan Rawal) wrote…
> ```
> Does this work
> for err != nil || try <10 {
>
> }
> ```
> If it does we can move the `log.Fatal`  outside the loop. Infact instead of `log.Fatal` we could again show the box with a error message.
> The user won't have any visual indication during this period. So its better if we keep the sleep down to 1 second? If he can't connect in 10 seconds I am sure there is a problem at our end.
>

It should be an &&.  Sure, we can show a screen with some info, can you create a new screen to take care of this?

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

client/main.go, line 86 [r1] (raw file):
Previously, ashwin95r (Ashwin Ramesh) wrote…
> The client doesn't crash if the server crashes, but if the stream is broken, the client has to re-establish it and not the server. We need it handle it separately. This is only for temporary network errors.

Hmm, so there isn't any way we can test this? Also do we need this for `sendAnswer`?

client/main.go, line 384 [r1] (raw file):
Previously, ashwin95r (Ashwin Ramesh) wrote…
> It should be an &&.  Sure, we can show a screen with some info, can you create a new screen to take care of this?

It's already there, the new screen which is the instructions termui element below this. I will link it with this.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

client/main.go, line 86 [r1] (raw file):
Previously, pawanrawal (Pawan Rawal) wrote…
> Hmm, so there isn't any way we can test this? Also do we need this for `sendAnswer`?

Yeah, we could have this for send answer too.

client/main.go, line 384 [r1] (raw file):
Previously, pawanrawal (Pawan Rawal) wrote…
> It's already there, the new screen which is the instructions termui element below this. I will link it with this.

Okay

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 4 unresolved discussions.

client/main.go, line 384 [r1] (raw file):
Previously, ashwin95r (Ashwin Ramesh) wrote…
> Okay

Done.

client/main.go, line 93 [r6] (raw file):

		try++
		if try > 10 {
			log.Fatal(err)

Change this to show Error page(). Also add this for sendAnswer()/other places we do log.Fatal. See if you can simulate and test it out. Though my guess is the ping would be trigerred first.

client/main.go, line 164 [r6] (raw file):

				}

				if count > 2 {

Maybe instead of having different number of retries at every place. Make this a constant and reuse it?

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

client/main.go, line 93 [r6] (raw file):
Previously, pawanrawal (Pawan Rawal) wrote…
> Change this to show Error page(). Also add this for sendAnswer()/other places we do log.Fatal. See if you can simulate and test it out. Though my guess is the ping would be trigerred first.

Done. and yeah time based ping is more likely to be triggered

client/main.go, line 164 [r6] (raw file):
Previously, pawanrawal (Pawan Rawal) wrote…
> Maybe instead of having different number of retries at every place. Make this a constant and reuse it?

This would be once every 5 seconds. So 2 would be good as we don't want to wait for more than 10 seconds. At other places we use time.Sleep

Comments from Reviewable

Merged!


As a user, I want to see the output more clearly so i can distinguish…

… between the outcome of each command

I did an npm install on all the packages. package.lock.json was updated for vscode-extension package.

Please also ignore the Pre-publish Build - master check, this is an error in our ci configuration.


Fixed background and footer gap

Fixed the styling gap issues between the background image and footer. Closes #53. Also made some additional styling on footer's content.
Before:

After:

Great job!


Fix Escaping and Like Operator Prefix

improve error messages
prefix like condition with operator to match other conditions
escape all string input parameters
break up tests for readability
add escape and like test cases

This PR covers more than one thing, which I know is not ideal but I got fixing and creating tests and wanted the code to be fixed to ensure all the tests passed.

I checked quickly and LGTM. Thanks a lot for your work on that. Didn't even know it was being used :p


v4 fluent naming config

@yungezz @ChenTanyi @xccc-msft
Note this will force it for all places, regardless what casing used in spec.

ManagedClusterAPIServerAccessProfile -> ManagedClusterApiServerAccessProfile
More in network since this is for all.

Sample javadoc
    /**
     * De-serializes the instance to int value.
     *
     * @return the int value.
     */
    @JsonValue
    public int toInt() {
        return this.value;
    }


Added Favourite contacts feature

Contacts will be highlighted in red if favourited

You may want to run command "gradlew clean checkstyleMain checkstyleTest" in intellij idea terminal for local style check


Make obvious why parsing fails for multipart

Possibly also for urlencoded bodies. It's confusing when something fails and there's no pointer to the issue.

I don't know about obvious but the crash report is more precise now. We still need an option to enable additional reporting on errors that end in a 400 but that's a separate concern.


adding AFD to the list of unsupported services

adding AFD to the list of unsupported services

@omartin2010 : Thanks for your contribution! The author, @jaredr80, has been notified to review your proposed change.

#sign-off


test(e2e): Fix e2e tests increase timeout

Reason for Change

#TICKET_NUMBER
If the reason for this PR's code changes are not clear in the issue, state value/impact

Changes

add
remove
modify

Testing steps

Either list QA steps or reasoning you feel QA is unnecessary
Describe how you made sure to know that your changes worked. Should allow someone else to go verify your code without in depth knowledge.
"Unit tested only", "tested in rdev by a, b, c, verifying feature worked by... ", "manually ran pipeline locally with these results: ..."

Notes for Reviewer

Turned out that FF and Edge tests were failing because we need to increase timeout from 3 to 5 mins, so it has enough time to tear down context. Chrome seems to tear down faster, so we never needed to do this


RX label on aux channel moves to main channel when PTT on TX-DISABLED frequency

Hi there!

Let's have, on main channel, a tx-disable frequency (AM freq for example) (VFO or memory)
Let's listen, on auxiliary channel, to a continuous transmission (VFO or memory) : RX blinks on this channel
During reception, press PTT: RX moves to main channel, still. At the end of transmission, RX is replaced by >> on main channel.

Hi,
Could you make a short video ?
Armel F4HWN.

https://github.com/armel/uv-k5-firmware-custom-feat-F4HWN/assets/891923/97323ed1-2a7b-402c-b0b3-a09bff0995c3

Ok, thank you. It's certainly because VFO A is on TX DISABLE.
I'm not sure I can simply fix it. But I'll take a look when I have time.
Armel F4HWN.

Ok. Could you try this firmware and test it ?
I'm not quite sure if my solution is the right one to fix this issue... So test very carefully.
You need to unzip firmware before flash it...
Armel F4HWN.
f4hwn.packed.bin.zip

I will test it as soon as I can (probably during next WE).

So I tested it. It looks to work fine. 👍
Tested all combinations of (A/B channels) (TD = main TX-DISABLE channel) :
(MR-TD/MR) (MR-TD/VFO) (VFO-TD/MR) (VFO-TD/VFO)
(MR/MR-TD) (MR/VFO-TD) (VFO/MR-TD) (VFO/VFO-TD)
One point of discussion : should it interrupt reception on aux channel (for 4 sec) as the main channel cannot be conversational due to tx-disabling ? Is there a use case where the radio should stay on tx-disable channel for 4 sec ?
Another point I have noticed :

press PTT : TX-DISABLE is displayed immediately BUT RX continues on aux channel
release PTT : RX is stopped, TX-DISABLE remains for 2 sec, then another 2s of silence before returning to RX channel.

Fixed in v2.7


change url to node

I have made the smallest possible change.
is L127 okay ? (occurance of containerUrl)
I have used vscode.dev and followed "goto Definition".
what else needs to be done ? (occurances of variables that need to be replaced)
please understand this is my first PR of actual Solid Code since late 2018.
don't know how to request review in github.
we do teamwork...

old (Promise string was passed)
new (NamedNode is passed, maybe introduce Promise in ff line)
I was not sure, if we completely can leave out Line 127.
somewhere the construction of the Promise must be introduced.
still get used to it (coming from procedural).
please review and stay patient.

materialized it out with @timea-solid .
why do the checks fail ?
what next ?

changed L137 (call of the changes function)
am not sure if containerNode is there already in the scope...

I'm only ninetyeight point something sure about this

url => containerNode

fixed one error.
another one popped up.

advancing slowly.
changing any to NamedNode.
need review, if I am on the right path, @timea-solid

Well done so far!
I made comments on the lines of code.
You have completed the  and changed recursiveDelete function which was using

reverted to any in two files...

from 11/12 errors to 2/11 errors - progress...

now I have a strange effect:
I have done 1. https://github.com/SolidOS/solid-logic/pull/38/commits/8e8d9661dae105b58ef5638eb644fee7d2ff8fb4 2. revert https://github.com/SolidOS/solid-logic/pull/38/commits/10ccdd3e88e0bbdac0d00ae072e273cdf09b98bd
and the builld run shows different results

https://github.com/SolidOS/solid-logic/actions/runs/2242340555
https://github.com/SolidOS/solid-logic/actions/runs/2242381396

a) a heap of internal errors - fetcher issue
b) inbox - array prob
c) containerNode.doc() is not a function

Your investigation and actions are correct!
To have an easier time to get to the problems you can locally, on your code run npm run test. I believe it might give you better error messages than what you see here in git.
So please try npm run test.

@timea-solid
I omitted the 4th parameter, which is the document itself and I thought won't be necessary.
npm run test showed progress, only 2 errors instead of 4.
now
a) the package json and the package lock json was modified cause I needed to install jest locally.
b) the utilityLogic.test.ts now throws the following
Utility logic › getContainerMembers › When container has some containment triples › Resolves to an array with some URLs

    expect(received).toEqual(expected) // deep equality

    - Expected  - 4
    + Received  + 1

    - Array [
    -   "https://container.com/bar/",
    -   "https://container.com/foo.txt",
    - ]
    + Array []

      56 |       });
      57 |       it("Resolves to an array with some URLs", () => {
    > 58 |         expect(result.sort()).toEqual([
         |                               ^
      59 |           'https://container.com/foo.txt',
      60 |           'https://container.com/bar/'
      61 |         ].sort());

      at Object.<anonymous> (test/utilityLogic.test.ts:58:31)

I don't know, how to proceed, cause it's in another file now and the test should work, shouldn't it ?
but overall I slowly gain debug security in touching the code ;-)
ideas ?

I checked your code locally.
The 2 test fails you see are because of the use of the .getContainerMembers function. For example in in the line of code in the InboxLogic: https://github.com/ewingson/solid-logic/blob/c2225db9d0a1f1253cb67372d125b3a61d9d5693/src/inbox/InboxLogic.ts#L29 -> there is a problem there see screenshot:

It has to do with exactly what you changed, string to NamedNode.
You need to find all functions that make use of getContainerMembers  and make sure to pass a NamedNode and NOT a string anymore.
hope this helps 🤗

I think the test tests    L139 return nodes.map(node => node.value);
and receives other than expected. Now I gonna understand  that ...

Yes, looks good to me. Try to understand return nodes.map(node => node.value); and fix it.
Unfortunately, there will be more lines coming your way. Do not get discouraged. You are on the right path.
You can also run npm run build and you will see what issues are left to solve.

fixed another one, @timea-solid !
5 left ...

caught another one, @timea-solid !
4 left ...

3 ...

@timea-solid
I have changed the fetch type:
L16 constructor(store: LiveStore, ns: SolidNamespace, underlyingFetch: { fetch: (url: NamedNode, options?: any) => any }) {
now the errors have increased again.
would like to synch with you, if I'm still on the right path...

@timea-solid
I have changed the fetch type: L16 constructor(store: LiveStore, ns: SolidNamespace, underlyingFetch: { fetch: (url: NamedNode, options?: any) => any }) {
now the errors have increased again. would like to synch with you, if I'm still on the right path...

The Fetcher should not be changed. So you need to revert your code and find a way to use string from namedNode here.

reverted to pre-fetch-changes.
3 errors left.

UtilityLogic.ts L 97-101
async createContainer(url: string) {
    const stringToNode = url._what_here_?()
    if (!this.isContainer(stringToNode)) {
      throw new Error(`Not a container URL ${url}`);
    }

I need to do just the opposite of what you did in the function above this snippet.
createContainer seems to stay passed a string, cause the fetch requires it.
so, how do I convert a string to a named node ? I think in the direction of first declare a new named node......

solved it like this
async createContainer(url: string) {
    const stringToNode = new NamedNode (url);
    if (!this.isContainer(stringToNode)) {
      throw new Error(`Not a container URL ${url}`);
    }

2 left

fixed another one.
only one left.
src/util/UtilityLogic.ts:146:41 - error TS2552: Cannot find name 'url'. Did you mean 'URL'?

146       return this.underlyingFetch.fetch(url, { method: "DELETE" });
                                            ~~~

  node_modules/typescript/lib/lib.dom.d.ts:14005:13
    14005 declare var URL: {
                      ~~~
    'URL' is declared here.

Found 1 error in src/util/UtilityLogic.ts

not sure how to proceed...

yippie !!!

solved it like this
async createContainer(url: string) {
    const stringToNode = new NamedNode (url);
    if (!this.isContainer(stringToNode)) {
      throw new Error(`Not a container URL ${url}`);
    }

2 left

This fix is breaking some tests. So it is not correct just yet.

Ok I looked at the code and there are some things I needed to check and also learn.

What is a NamedNode -> https://linkeddata.github.io/rdflib.js/doc/classes/NamedNode.html
From the test fails we can see a NamedNode looks like:

 NamedNode {
    "classOrder": 5,
   "termType": "NamedNode",
    "value": "https://container.com/bar/",
    },
    ```
    This means the code at [L93](https://github.com/ewingson/solid-logic/blob/6feb1cef67833fd36d35ae685b45607b11d01dc0/src/util/UtilityLogic.ts#L93) is not correct.

 So what can we use instead of url.toString() ? from the NamedNode functions?

I tried String(url.value) but this was not the final solution.
maybe another step in the right direction ?
can you teach me how to read the test results (how did you find L93 is the problem?)

I've modified a bit and tried the named node method value
isContainer(url: NamedNode) {
    const nodeToString = url.value;
    return nodeToString.charAt(nodeToString.length-1) === "/";
  }

negative.

tried url uri() , url.uri and url.doc() . negative.
I reverted to url.value cause that way at least the npm run build passes.

Tests should be fixes now. It made sense to fix them since we needed to propagate in tests the fact that we use NamedNodes and not strings.

@timea-solid superb !
what is https://github.com/SolidOS/solid-logic/runs/6964470253?check_suite_focus=true ?
this whole CI deploy workflow still is a land with hidden doors to me ;-(

@timea-solid superb !
what is https://github.com/SolidOS/solid-logic/runs/6964470253?check_suite_focus=true ?
this whole CI deploy workflow still is a land with hidden doors to me ;-(

You can ignore the npm publish. That step should upload the artifact to npm which because we are on a dev branch it can be ignored. It would only be needed if you further need this dev artifact to integrate into another and develop cross modules (which is something I do often).

@timea-solid
I'm working myself "back-to-forth". I hit two other variables, that need to be NamedNode, otherwise the function in question is not callable. it's ourInbox and options.target

@timea-solid a good scientist communicates in the openness and peer review is a commonly wide accepted method.
my npm run build locally only shows one remaining error.
I have converted options.target to a NamedNode.
I wanted to synch with you the further processing.
a) can I try to convert ourInboxto a node, too ? I would need to import sym therefor to InboxLogic.ts .....
b) my first and mostdesired aim is to make npm run build without errors.
then, in a second step, we can care about the CI build. momentary it complains about that it wants absolute URIs.
what do you think ? my main work happens between 17 and 00 o'clock.
there are more things (an ocean) that I do NOT know opposed to a drop I understood ;-)

I htink 'ourInbox' should stay a string for 'createContainer' call but for 'this.util.findAclDocUrl(ourInbox)' is should be a NamedNode.
Also 'this.util.setSinglePeerAccess' could do some change on 'ownerWebId', 'peerWebId'.
Also in the 'markAsRead' url can be changed.

shall I PR to resolve conflicts ?

@timea-solid !!!!!
I solved that merge thang ! so I understood more than nothing !
hope that helps !

changed as in https://github.com/SolidOS/solid-logic/pull/38#issuecomment-1161354841

@timea-solid
npm run build throws no more errors. that's progress.
can you help me interpret the CI tests as follows:
Test Suites: 2 failed, 7 passed, 9 total
Tests:       3 failed, 2 skipped, 64 passed, 69 total
Snapshots:   0 total
Time:        6.561 s
Ran all test suites.
Jest did not exit one second after the test run has completed.
This usually means that there are asynchronous operations that weren't stopped in your tests. Consider running Jest with `--detectOpenHandles` to troubleshoot this issue.
npm ERR! Test failed.  See above for more details.
Error: Process completed with exit code 1.

I'll wrap my head upon jest a while, as suggested.
maybe @SharonStrats can help.
can we jump on a call during this week ?

obsolete because of https://github.com/SolidOS/solid-logic/pull/45


Update kubernetes.mdx

SUMMARY
Update the Helm Kubernetes instruction to use bootstrapScript. In the helm values file, additionalRequirements currently referenced in the spec, does not exist.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

sorry this took a while @mvoitko, I made your requested changes, but this PR may be moot per my comment above. Thanks for this awesome lib!


[WIP] Change meaning of UUID field, add VPNKitKey field

We have missused UUID filed for VPNKit, and not the -U flag which it naturally maps to.
It seems reasonably to leave it up to the caller how they wish to generate UUIDs.
This is a breaking change, hence I marked it WIP (so it doesn't gets merged too soon), and the biggest question question is how should we communicate this change to those who depend on these bindings?

It seems reasonably to leave it up to the caller how they wish to generate UUIDs.

LGTM, these bindings are pretty new so no need to worry about making breaking just yet.
In the future it might be nice to generate a UUID if the caller doesn't supply one.


🐛 [firebase_ui_auth] SignInScreen error messages can be hidden by the in-screen keyboard on some devices

Bug report
Describe the bug
In SignInScreen, when the user clicks the Sign In button and some error occurs, since the in-screen keyboard is not removed, the error message appears hidden behind it so it might look to the user that nothing has happened.
(1) Before clicking the sign-in button,  (2) after clicking, and (3) after dismissing the keyboard manually:

Steps to reproduce
Steps to reproduce the behavior:

Create a flutter app with a SignInScreen, with an EmailSignInProvider and a GoogleProvider
Open the app on a Nokia 6 or other physical or emulated android device whose display characteristics make the sign-in button appear just above the keyboard, as shown in the screenshots above
Enter some non-existing user data
Without dismissing the keyboard (by clicking elsewhere), click Sign In
Dismiss the keyboard to check that the error message has appeared behind the keyboard

Expected behavior
The error message is highlighted clearly, for example by automatically hiding the keyboard when the user clicks sign in.

Flutter doctor
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.3.7, on Microsoft Windows [Version 10.0.19044.2364], locale en-GB)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.18)
[√] Android Studio (version 2021.2)
[√] IntelliJ IDEA Ultimate Edition (version 2021.3)
[√] IntelliJ IDEA Ultimate Edition (version 2022.2)
[√] Connected device (5 available)
[√] HTTP Host Availability

• No issues found!

Flutter dependencies
Click To Expand
Dart SDK 2.18.4
Flutter SDK 3.3.7
commitments_app 1.0.0+1

dependencies:
- cloud_firestore 4.3.1 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- collection 1.16.0
- cupertino_icons 1.0.5
- date_time_picker 2.1.0 [flutter intl]
- firebase_auth 4.2.4 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 2.4.1 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_dynamic_links 5.0.11 [firebase_core firebase_core_platform_interface firebase_dynamic_links_platform_interface flutter meta plugin_platform_interface]
- firebase_ui_auth 1.1.6 [email_validator firebase_auth firebase_core firebase_dynamic_links firebase_ui_localizations firebase_ui_oauth flutter flutter_localizations flutter_svg]
- firebase_ui_oauth_google 1.0.13 [firebase_auth firebase_ui_oauth flutter google_sign_in]
- flutter 0.0.0 [characters collection material_color_utilities meta vector_math sky_engine]
- flutter_riverpod 1.0.4 [collection flutter meta riverpod state_notifier]
- flutter_speed_dial 6.2.0 [flutter]
- google_sign_in 5.4.2 [flutter google_sign_in_android google_sign_in_ios google_sign_in_platform_interface google_sign_in_web]
- intl 0.17.0 [clock path]
- url_launcher 6.1.7 [flutter url_launcher_android url_launcher_ios url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_launcher_windows]

dev dependencies:
- flutter_lints 2.0.1 [lints]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters collection matcher material_color_utilities meta source_span stre
am_channel string_scanner term_glyph]

transitive dependencies:
- _flutterfire_internals 1.0.12 [collection firebase_core firebase_core_platform_interface flutter meta]
- async 2.9.0 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- characters 1.2.1
- clock 1.1.1
- cloud_firestore_platform_interface 5.10.1 [_flutterfire_internals collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 3.2.1 [_flutterfire_internals cloud_firestore_platform_interface collection firebase_core firebase_core_web flutter flutter_web_plugins js]
- crypto 3.0.2 [typed_data]
- desktop_webview_auth 0.0.11 [crypto flutter http flutter_web_plugins plugin_platform_interface]
- email_validator 2.1.17
- fake_async 1.3.1 [clock collection]
- firebase_auth_platform_interface 6.11.7 [_flutterfire_internals collection firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 5.2.4 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core_platform_interface 4.5.2 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 2.1.0 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_dynamic_links_platform_interface 0.2.3+26 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_ui_localizations 1.0.2 [flutter flutter_localizations intl path]
- firebase_ui_oauth 1.1.6 [desktop_webview_auth firebase_auth firebase_ui_auth flutter_svg flutter]
- flutter_localizations 0.0.0 [flutter intl characters clock collection material_color_utilities meta path vector_math]
- flutter_svg 1.1.6 [flutter meta path_drawing vector_math xml]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta vector_math]
- google_sign_in_android 6.1.4 [flutter google_sign_in_platform_interface]
- google_sign_in_ios 5.5.1 [flutter google_sign_in_platform_interface]
- google_sign_in_platform_interface 2.3.0 [flutter plugin_platform_interface quiver]
- google_sign_in_web 0.10.2 [flutter flutter_web_plugins google_sign_in_platform_interface js]
- http 0.13.5 [async http_parser meta path]
- http_parser 4.0.2 [collection source_span string_scanner typed_data]
- js 0.6.4
- lints 2.0.1
- matcher 0.12.12 [stack_trace]
- material_color_utilities 0.1.5
- meta 1.8.0
- path 1.8.2
- path_drawing 1.0.1 [vector_math meta path_parsing flutter]
- path_parsing 1.0.1 [vector_math meta]
- petitparser 5.1.0 [meta]
- plugin_platform_interface 2.1.3 [meta]
- quiver 3.2.1 [matcher]
- riverpod 1.0.3 [collection meta state_notifier]
- sky_engine 0.0.99
- source_span 1.9.0 [collection path term_glyph]
- stack_trace 1.10.0 [path]
- state_notifier 0.7.2+1 [meta]
- stream_channel 2.1.0 [async]
- string_scanner 1.1.1 [source_span]
- term_glyph 1.2.1
- test_api 0.4.12 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.1 [collection]
- url_launcher_android 6.0.22 [flutter url_launcher_platform_interface]
- url_launcher_ios 6.0.17 [flutter url_launcher_platform_interface]
- url_launcher_linux 3.0.1 [flutter url_launcher_platform_interface]
- url_launcher_macos 3.0.1 [flutter url_launcher_platform_interface]
- url_launcher_platform_interface 2.1.1 [flutter plugin_platform_interface]
- url_launcher_web 2.0.13 [flutter flutter_web_plugins url_launcher_platform_interface]
- url_launcher_windows 3.0.1 [flutter url_launcher_platform_interface]
- vector_math 2.1.2
- xml 6.1.0 [collection meta petitparser]

Thanks for the report. This is replicable using iOS simulator as well like below:


Fixed the crash issue when a pointer is null.

When using libcurl, at the time the process is about to exit, openssl will invoke the err_cleanup function, setting err_string_lock to NULL. If other threads are still calling libcurl at this moment, it will cause a crash in the int_err_get_item function.
当我使用libcrul，在进程要退出的时候，openssl会调用err_cleanup将err_string_lock赋值为NULL,此时其他线程还在调用libcurl，在int_err_get_item函数中会产生崩溃

This PR has the label 'hold: cla required' and is stale: it has not been updated in 30 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

This PR has the label 'hold: cla required' and is stale: it has not been updated in 61 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

This PR has been closed.  It was waiting for a CLA for 90 days.


Update npm package url-parse & fs-path

Codecov Report

Merging #9255 (18a85b1) into main (22c7ab6) will decrease coverage by 0%.
The diff coverage is n/a.

:exclamation: Current head 18a85b1 differs from pull request most recent head 0217109. Consider uploading reports for the commit 0217109 to get more accurate results

@@          Coverage Diff          @@
##            main   #9255   +/-   ##
=====================================
- Coverage     71%     71%   -1%
=====================================
  Files        383     383
  Lines      24803   24803
  Branches    3986    3986
=====================================
- Hits       17762   17752   -10
- Misses      5445    5456   +11
+ Partials    1596    1595    -1

Impacted Files
Coverage Δ

src/client/datascience/dataScienceSurveyBanner.ts
69% <0%> (-7%)
:arrow_down:

src/client/datascience/jupyter/kernels/helpers.ts
54% <0%> (-1%)
:arrow_down:

...atascience/interactive-window/interactiveWindow.ts
70% <0%> (-1%)
:arrow_down:

...lient/datascience/jupyter/kernels/cellExecution.ts
73% <0%> (+<1%)
:arrow_up:


[feat] Allow transformPage to return a promise

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Closes #3967 by allowing the transformPage resolve option to return a promise.

thank you!

Thank you!


Backend API Changes to rename attachment  for ministry request (#779)

The system should allow renaming of attachment already uploaded.
DOD

[x] Service tier to support renaming a file by creating new version
[x] Provide new endpoint to allow renaming of file
[x] Unit Testing

Note on URL format =  http://localhost:15000/api/foidocument/ministryrequest//documentid//rename
POST http://localhost:15000/api/foidocument/ministryrequest/1/documentid/1/rename
Http Body:
{
"filename":"newname.docx"
}
Response:
{
"id": 1,
"message": "New Document version created",
"status": true
}