Using the Pandoc User Data Directory

In your instructions, you suggest putting logging.lua in the same directory as the pandoc filters. In my case this is the standard path ~/.local/share/pandoc/filters/.
So I have:
▶ ll .local/share/pandoc/filters/logging*
-rw-r--r--@ 1 ian  staff   9.1K  5 Oct 12:26 .local/share/pandoc/filters/logging.lua
-rw-r--r--@ 1 ian  staff   588B  5 Oct 14:59 .local/share/pandoc/filters/loggingTest.lua

However if I call local logging = require 'logging' in loggingTest.lua then I get the following error:
pandoc -t html5 --verbose -L loggingTest.lua

---
title: My Title
---

Here.
[INFO] Running filter /Users/ian/.local/share/pandoc/filters/loggingTest.lua
Error running filter /Users/ian/.local/share/pandoc/filters/loggingTest.lua:
/Users/ian/.local/share/pandoc/filters/loggingTest.lua:2: module 'logging' not found:
	no field package.preload['logging']
	no file '/usr/local/share/lua/5.4/logging.lua'
	no file '/usr/local/share/lua/5.4/logging/init.lua'
	no file '/usr/local/lib/lua/5.4/logging.lua'
	no file '/usr/local/lib/lua/5.4/logging/init.lua'
	no file './logging.lua'
	no file './logging/init.lua'
	no file '/usr/local/lib/lua/5.4/logging.so'
	no file '/usr/local/lib/lua/5.4/loadall.so'
	no file './logging.so'
stack traceback:
	/Users/ian/.local/share/pandoc/filters/loggingTest.lua:2: in main chunk

I'm not a Lua person, but I understood that ./logging.lua should have found it?

Ah, I think I've omitted a step. Sorry about that. .local/share/pandoc/filters will be searched by pandoc but here we're in the lua world, so the lua search path will need to be set. In fact I always run pandoc as follows, where XXX is the location of any required lua files (and indeed for me is also the location of the pandoc lua filters).
LUA_PATH="XXX/?.lua;;" pandoc

So you'll need something like this.
LUA_PATH="$HOME/.local/share/pandoc/filters/?.lua;;" pandoc

(You could also define the LUA_PATH environment variable statically.)
Does this work?

OK, thanks, LUA_PATH should work for most of my workflows. In some cases pandoc is run by other tools (that may or may not respect ENV), and so in these cases using PANDOC_STATE.user_data_dir and dofile inside the lua file should work better I assume... Perhaps you can mention at least LUA_PATH in the readme? I'll close this and thanks for a very helpful script!

Thanks. Yes I'll definitely mention LUA_PATH. I'd be happy to mention alternatives such as dofile (which I haven't used) but perhaps it's more useful just to ensure that the issue is explained. I'll reopen this issue for now, and will close it when I've updated the docs (or created an issue to update the docs).


Fix Markdown

Hello! I hope you are having a lovely day. Thank you for writing this codebase.
There is a small markdown error in the readme. This is a fix for it.

Oops, you are right. Thanks!

Well it didn't fix the problem yet.. Let me check


peer-to-peer

Hi @ShaliniNayan, good job so far on the to-do list project. Below are some ideas and suggestions about the project, cheers! 👍🏽
HIGHLIGHTS 🎯

Good implementation of guidelines🥇

RECOMMENDATIONS 🤔
-deployment: **npm install gh-pages, open package.json scripts object add "deploy": "gh-page -d dist", "homepage": "https://github.com/supreeti/ToDoList", npm run deploy **

Thankyou & Noted.


Support STOMP 1.2 ACK and NACK headers

Original issue: https://github.com/jmesnil/stomp-websocket/issues/111

Most of your PR are in https://github.com/JSteunou/webstomp-client

Thanks, that is good to know.

Hi,
Any progress in supporting 1.2?
Thanks!


Fix signup, login and logout logic in the NavBar

This logic was not properly working before, so users could not smoothly sign up, log out, back in, clear progress, etc.
How to test:
Sign up, log out, log back in, log back out, and then clear your user data again.

Clear your local storage (user and loggedIn items)
Go to the home page
Click account button in the top-right corner
In the sign-up modal, pick an avatar, copy your code and click the button at the bottom to close it
The account button in the nav bar should now should your avatar
Click it again to see the login modal stating that you are logged in
Click "Sign out"
The account button in the nav bar should now should the outline icon again
Click it again to see the login modal in the login state
Log in with the code you copied previously
The modal should show the "You're logged in now" state - click "Continue"
The account button in the nav bar should now should your avatar
Click it again and sign out again, then click the avatar icon again
Click "Clear saved progress" at the bottom
The modal disappears and the account icon switches to the outline icon again

Good luck 😀

While you're at it, the private key is the LoginModal is not copiable because it should be:
diff --git a/components/chapters/LoginModal.tsx b/components/chapters/LoginModal.tsx
index 0e22032..c1bec3a 100644
--- a/components/chapters/LoginModal.tsx
+++ b/components/chapters/LoginModal.tsx
@@ -64,7 +64,7 @@ export const LoginModal = (props) => {
   }

   function copy() {
-    navigator.clipboard.writeText(userPrivateKey)
+    navigator.clipboard.writeText(user.privateKey);
     setCopied(true)

     setTimeout(() => {

Thank you. Just added that fix.


Add formatted field to JSON if possible.

I used String.format which expects args like %s ... I haven't given this a lot of thought. Some (all?) logging frameworks use {} in Java.
If anything this is the scaffolding and we just have to decide which formatting to support?

I think a different approach probably makes more sense here that makes it logger-aware rather than trying to pack it into the MessageInterface class.
When recording from an appender,

The top level message attribute should include the formatted message.
We should add formatted as a constructor parameter to MessageInterface when passing parameters. (In my opinion, it should be required, but we should probably allow it to be null for backwards compatibility and use that as the default.) Then, we should use this in the appenders, passing the message template as the "message", the parameters as "parameters", and the formatted message as the "formatted" parameter. Essentially, what we've been doing with the Log4j2 appender with the addition of the formatted constructor parameter.

If we're not going through an appender, I think we need to just expect the client to do the right thing (which right now is probably just using the withMessage method on the EventBuilder and forgoing attaching a MessageInterface unless they know what they are doing.)

Yeah, I think that all makes more sense, will update this soon.

Cleaned up PR here: https://github.com/getsentry/raven-java/pull/220


support versions of python earlier than 3.10

tenforty currently uses a few features, i.e. match/case pattern matching, and union-style type declarations, that were introduced to the language in 3.10.
I'm not sure this should be done. Python 3.10 was introduced in October 2021, and we're now on 3.12, so it seems like there's been a reasonable amount of time to upgrade. And the smaller the number of interpreters, the less to test. This said, it's a small package and it doesn't seem either than onerous or that dangerous to use more universal constructs.
Adding this issue to capture discussion.

Thanks to @ekerstein for raising: https://github.com/mmacpherson/tenforty/issues/18


Enable Stargate Query with Json Marshalled Response

Background
The current implementation of enabling stargate queries(https://github.com/osmosis-labs/osmosis/pull/2190) proto marshals the response. Since we do not have set standards on how we want to support stargate queries in the contract side, either way of returning the response is possible: either by proto marshalling the response OR by json marshalling the response.
The advantage of Json Marshalling though, is that the cosmwasm std querier for stargate expects the response to be json marshalled by default. It'd be way easier for contracts to use stargate querier if the response from go side be json marshalled. Also, the process of unmarshalling proto marshalled responses that comes from the go side would be gnarly, having to query it in raw bytes, and having to find out a way to proto unmarshal it ( afaik, the proto library in rust is pretty heavy, we don't know if we want to include the heavy proto library in all contracts we have).
Suggested Design
We change https://github.com/osmosis-labs/osmosis/pull/2190 to return responses json marshalled, not proto marshalled.
Acceptance Criteria

stargate query responses return json marshalled responses.
We have a test case to prove that json marshalling the response is safe

I believe this should be closed


follow codemirror6 focused specific changes

close https://github.com/craftzdog/cm6-themes/issues/3


[Snyk] Security upgrade openjdk from 8u131-jre to 8u332-jre

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

java/rmi-registry-bind-deserialization-bypass/Dockerfile

We recommend upgrading to openjdk:8u332-jre, as this image has only 172 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

729
Out-of-bounds Write SNYK-DEBIAN9-FREETYPE-1019584
Mature

786
Out-of-Bounds SNYK-DEBIAN9-GLIBC-356506
Mature

786
Out-of-Bounds SNYK-DEBIAN9-GLIBC-356506
Mature

786
Out-of-Bounds SNYK-DEBIAN9-GLIBC-356506
Mature

714
CVE-2017-10087 SNYK-DEBIAN9-OPENJDK8-383687
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Checkmarx One – Scan Summary & Details – cb7416d5-6e38-435b-ae70-5b62790f3aec
No New Or Fixed Issues Found


Possible conflict - OnFilterScriptExit

I'm not sure it's about FCNPC plugin or MapFix library but found this:
D:\FCNPC.inc(423) : error 021: symbol already defined: "SSCANF_OnFilterScriptInit"
D:\dynamicgz.inc(447) : error 021: symbol already defined: "OnFilterScriptExit"
MapFix library: http://forum.sa-mp.com/showthread.php?p=3962807
MapFix's OnFilterScriptExit:
public OnFilterScriptExit()
{
	if(MapFixIsEnabled < 2)
	{
		printf("  MapFix by Nexius v%s unloaded (filterscript version).", MAPFIX_VERSION);
		DestroyMapFixObjects();
		#if defined DeleteSVar
			DeleteSVar("MapFixIsEnabled");
		#endif
	}
	return 1;
}

FCNPC include version: 1.7.4
MapFix library version: 3.2.1

Try to use include version of MapFix.
On Dec 4, 2017 14:11, "Yousha Aleayoub"<EMAIL_ADDRESS>wrote:

I'm not sure it's about FCNPC or MapFix library but:
D:\FCNPC.inc(423) : error 021: symbol already defined:
"SSCANF_OnFilterScriptInit"
D:\dynamicgz.inc(447) : error 021: symbol already defined:
"OnFilterScriptExit"
MapFix's OnFilterScriptExit:
public OnFilterScriptExit()
{
if(MapFixIsEnabled < 2)
{
printf("  MapFix by Nexius v%s unloaded (filterscript version).", MAPFIX_VERSION);
DestroyMapFixObjects();
#if defined DeleteSVar
DeleteSVar("MapFixIsEnabled");
#endif
}
return 1;
}
MapFix library: http://forum.sa-mp.com/showthread.php?p=3962807
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ziggi/FCNPC/issues/117, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-Qw1vgUnvdxSPCM8hyRXh_iA23w5Uzks5s89NdgaJpZM4Q0fC1
.

Thanks, solved.
NOT a bug.


Updated branding.yml and respective files to help docs to use the term "Confluent Cloud" correctly.

Description
I have made the changes in branding.yml and branding-warning-temp.yml files to help docs use the term "Confluent Cloud" correctly.
Notes for Reviewers
Test:   npm run spell: error was green, no spelling errors found.
npm run spell:error

<EMAIL_ADDRESS>spell:error
vale --glob='!.github/vale/styles/*' . --minAlertLevel=error
✔ 0 errors, 0 warnings and 0 suggestions in 0 files.

Please check the above changes if I did anything wrong please let me know.
This PR fixes #100

Hi @csengineer1990
As far as I see, everything should be correct. The rule you added flags wrong spellings correctly, I checked by adding some. You may not see an error when running npm run spell:error because you fixed the one case already.
I just ran the pipeline and all works. I'll go on an approve and merge your PR now!
Thanks for your contribution 🎉  and have fun with Hacktoberfest!

Thank you so much mam for your support and guidance .


[Multiple Ordering Providers] Integration into Test Card Creation

Why should we do this?

Currently, users are unable to select the appropriate ordering provider when creating a new test card in SimpleReport.
Integrating the ordering provider selection functionality into the test card creation workflow will ensure that the correct ordering provider information is captured and associated with each test record.

User story
As a SimpleReport user, I want to be able to select the appropriate ordering provider when creating a new test card so that the correct ordering provider information is associated with the test record.
Action Requested

Integrate the ordering provider selection functionality into the existing test card creation workflow.
Ensure the correct ordering provider information is captured and associated with the test record, including:

Displaying the list of available ordering providers for the user's facility
Allowing the user to select the appropriate ordering provider
Correctly storing the selected ordering provider information with the test record

Acceptance Criteria
Design

[ ] Design is ready for handoff (i.e. dev ready)
[ ] Design includes latest content updates
[ ] Capture questions that come up during the content and design work

Implementation

[ ] Integrate the ordering provider selection UI into the test card creation workflow
[ ] Ensure the selected ordering provider information is correctly captured and associated with the test record
[ ] Validate the integration works as expected through comprehensive testing

Review

[ ] Reviewed with design and/or design lead
[ ] Reviewed with product and engineering team

Additional Context

Figma: https://www.figma.com/design/LglkDsGZDKMZm3XCkJPhfR/SimpleReport-2023%2B?node-id=5984-397&t=gcK2LTCUEPhQIxxY-0

blocked by #7982


Sub-10-minute Multi-Blind results unexpectedly round to nearest second

Title is self-explanatory.

Welp, never mind. I did not realize the rounding rule was different for multi-blind.


test issue

Bug Description
this is a test
To Reproduce
test
Environment
test
Relevant log output
test

Additional context
No response

issue used to test the automation to jira. Now it's fixed


fix: Updating logic to push blank key value params based on manage access for APIs

Description

Updating logic to push blank key value params based on manage access for APIs.

Fixes #19253
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Followed the steps in the linked issue and it works now as expected.

Manual

Test Plan

Add Testsmith test cases links that relate to this PR

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=8b1a652

/ok-to-test sha=d9fff56

/ok-to-test sha=94a58fc


ServerIncompatibilityError with version 5.20.0

With cave-client version 5.20.0 I get a new ServerIncompatibilityError:
>>> client = cv.CAVEclient('flywire_fafb_public')
>>> client.chunkedgraph.level2_chunk_graph(720575940614131061)
ServerIncompatibilityError: Use of keyword argument `bounds` in `level2_chunk_graph` is only permitted for
server version >=2.15.0, your server version is None. Contact your system
administrator to update the server version.

Same code works like a charm with 5.18.0. I assume this is a configuration issue?

hi @schlegelp - this is related to a new scheme we implemented for comparing the compatibility of the client and the version of the service running remotely, but this shouldn't be erroring here. will look into it!

We have a new system in place that should allow features to be introduced to caveclient but not passed along to a server deployment if it is not yet compatible. It apparently is too aggressive — I propose a fix in #179.


The futex facility returned an unexpected error code.Aborted

OS
linux
VERSION
latest
ISSUE DESCRIPTION
snap install
The futex facility returned an unexpected error code.Aborted

ref https://github.com/electron/electron/issues/24211


Allow describing an index from a given Type.

Here is an example
public class PropertyDynamicIndexProvider : IndexProvider<Property>
{
    public override void Describe(DescribeContext<Property> context)
    {
        context
            .For(typeof(PropertyIndex))  // in addition to the existing ".For<PropertyIndex>()"
            .Map(property => new PropertyIndex
            {
                Name = property.Name,
                ForRent = property.ForRent,
                IsOccupied = property.IsOccupied,
                Location = property.Location
            });
    }
}

Hi @MikeAlhayek  ， All problems resolved

@MikeAlhayek Thanks for your help .
I didn't notice that this test should be moved to CoreTests because I copied it from one of the Sqlite Index tests above

I guess since it's just to ensure the index is created using type, we could also have left it in the SQLite only. It does not matter either way. It does not hurt running it on all databases either

@MikeAlhayek I reupdated the code
        protected static PropertyInfo[] TypePropertiesCache(Type type)
        {
            if (TypeProperties.TryGetValue(type, out var pis))
            {
                return pis;
            }

+            // If the same name type is already cached, it should be removed from the cache.
+            var existsingType = TypeProperties.Keys.FirstOrDefault(x => x.FullName == type.FullName);
+            if (existsingType != null)
+            {
+                var existsingProps = TypeProperties[existsingType];
+                foreach (var prop in existsingProps)
+                {
+                    PropertyAccessors.Remove(prop, out _);
+                }
+
+                TypeProperties.Remove(existsingType, out _);
+            }

            var properties = type.GetProperties().Where(IsWriteable).ToArray();
            TypeProperties[type] = properties;
            return properties;
        }

@hyzx86 why do you need to remove a previously store type?

@hyzx86 why do you need to remove a previously store type?

Because my dynamic index is generated according to the TypeDefinition in the OC, it may be updated many times during the run of the program, and if allowed for a long time, this static collection will cache many useless index types
        public override void Describe(DescribeContext<ContentItem> context)
        {
            var contentTypes = _contentDefinitionManager.ListTypeDefinitionsAsync().GetAwaiter().GetResult();
            foreach (var contentType in contentTypes.Where(x => !x.HasStereotype() && x.Parts.Any(x => x.Name == nameof(DIndexPart))))
            {
                var config = _dynamicIndexAppService.GetDynamicIndexConfigAsync(contentType.Name).GetAwaiter().GetResult();
                if (config != null)
                {
                    // get or create a dynamic index type
                    var indexType = _dynamicIndexAppService.GetOrCreateDynamicIndexTypeAsync(config).GetAwaiter().GetResult();
                    context.For(indexType)
                        .When(contentitem => contentitem.ContentType == contentType.Name)
                        .Map((contentItem) =>
                    {
                        if (!contentItem.Published || !contentItem.Latest)
                        {
                            return null;
                        }
                        var table = _fsql.CodeFirst.GetTableByEntity(indexType);
                        var entity = contentItem.ToModel(config, indexType, table);
                        return (EocContentItemMapIndex)entity;
                    });
                }
            }
        }
    }

If the type already exists, it would just return it. Why would you need to remove it. Not sure I understand. Maybe you want to add a new test case to represent the problem.

First of all, the type name doesn't change, and according to the rules in my code it might generate one of these type names
public class {TenantName}.DynamicIndex.{ContentTypeName}
However, whenever a field changes, a new c# type is generated based on this content type definition
At this point, the newly generated type is exactly the same as the type name cached in Yessql, but they are no longer the same type.
So the following exception is thrown when the property lookup is performed
InvalidCastException System.InvalidCastException:
[A]EasyOC.DynamicIndex.AmisSchemaDIndex cannot be cast to
[B]EasyOC.DynamicIndex.AmisSchemaDIndex.
Type A originates from 'DynamicTypesAssembly, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
in the context 'Default' in a byte array. Type B originates from 'DynamicTypesAssembly, Version=<IP_ADDRESS>, Culture=neutral,
PublicKeyToken=null' in the context 'Default' in a byte array.

Full log :
2023-12-29 10:42:33.4084xxxx|ERROR|IContentHandler thrown from OrchardCore.Contents.AuditTrail.Handlers.AuditTrailContentHandler by InvalidCastException System.InvalidCastException: [A]EasyOC.DynamicIndex.AmisSchemaDIndex cannot be cast to [B]EasyOC.DynamicIndex.AmisSchemaDIndex. Type A originates from 'DynamicTypesAssembly, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' in the context 'Default' in a byte array. Type B originates from 'DynamicTypesAssembly, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' in the context 'Default' in a byte array.
at YesSql.Serialization.PropertyInfoAccessor.Invoker`2.Invoke(Object target)
at YesSql.Commands.IndexCommand.GetProperties(DbCommand command, Object item, String suffix, ISqlDialect dialect)
at YesSql.Commands.CreateIndexCommand.AddToBatch(ISqlDialect dialect, List`1 queries, DbCommand batchCommand, List`1 actions, Int32 index)
at YesSql.Session.BatchCommands()
at YesSql.Session.FlushAsync()
at YesSql.Session.FlushAsync()
at YesSql.Services.DefaultQuery.CountAsync()
at YesSql.Services.DefaultQuery.QueryIndex`1.YesSql.IQueryIndex<T>.CountAsync()
at OrchardCore.Contents.AuditTrail.Handlers.AuditTrailContentHandler.RecordAuditTrailEventAsync(String name, IContent content)
at OrchardCore.Modules.InvokeExtensions.InvokeAsync[TEvents,T1](IEnumerable`1 events, Func`3 dispatch, T1 arg1, ILogger logger) at YesSql.Serialization.PropertyInfoAccessor.Invoker`2.Invoke(Object target)
at YesSql.Commands.IndexCommand.GetProperties(DbCommand command, Object item, String suffix, ISqlDialect dialect)
at YesSql.Commands.CreateIndexCommand.AddToBatch(ISqlDialect dialect, List`1 queries, DbCommand batchCommand, List`1 actions, Int32 index)
at YesSql.Session.BatchCommands()

Type generated part of the code for reference
  private Type CreateDynamicType(DynamicIndexConfigModel config)
  {
      // Define the namespace and assembly for the dynamic type
      string nameSpace = config.EntityInfo.NameSpace;
      string assemblyName = "DynamicTypesAssembly";
      // Create the dynamic assembly
      AssemblyBuilder assemblyBuilder = AssemblyBuilder.DefineDynamicAssembly(
          new AssemblyName(assemblyName), AssemblyBuilderAccess.Run);

      // Create a dynamic module in the assembly
      ModuleBuilder moduleBuilder = assemblyBuilder.DefineDynamicModule(nameSpace);
      // Define the type
      TypeBuilder typeBuilder = moduleBuilder.DefineType(config.EntityInfo.FullName, TypeAttributes.Public);

      typeBuilder.SetParent(typeof(EocContentItemMapIndex));

      // Add the attributes to the type
      var indexCols = config.Fields.Where(x => !x.Disabled && x.IsDefaultIndex)
                                   .Select(x => x.DbFieldName).ToList();
      var indexNames = new List<string>()
      {
          "ContentItemId", "DocumentId"//, "Published", "Latest"
      };
      if (indexCols.Any())
      {
          indexNames.AddRange(indexCols);
      }
      typeBuilder.SetCustomAttribute(new CustomAttributeBuilder(
                  typeof(EOCIndexAttribute).GetConstructor(new[] { typeof(string), typeof(string[]) }),
                  ["IDX_{tableName}_DocumentId", new[] { string.Join(",", indexNames) }])
              );
...

Another option is to make the type cache of Yessql replaceable, so external code can define how to cache these types
This is better because external code knows exactly when it needs to clean them up; for example, the current scheme removes a ContentType definition and the dynamic index type associated with that type is cached in the collection forever

Well, I reproduced this problem through unit tests, because in OrchardCore, the storage can be initialized multiple times

@hyzx86
This is where we initialize the store in OC. Maybe try investigating why it is called more than once. Or add a state if the store in initialized or not before initializing again.
https://github.com/OrchardCMS/OrchardCore/blob/250c5ff4285c82e4e3ed6be49f7429005d0b0e11/src/OrchardCore/OrchardCore.Data.YesSql/OrchardCoreBuilderExtensions.cs#L125

@hyzx86 you may also want to open an issue in OC if you think that the store is being initialized more than once. That is something we would need to address in OC not in YesSql.
If you are sure that multi-initialization happens, please provide a reproduction steps and if possible a small repo that would help debugging the issue. Ping me then, and I'll carve out time to investigate it.

@MikeAlhayek OrchardCore Is a multi-tenant system, it will certainly initialize IStore many times, which is not the problem of OC.
I think the problem now is how to abstract the type cache, and then we can modify or define its implementation externally.

Format

@hishamco  Thanks, I think you sent a link to a code format specification, but I can't find it. Can you send it again? I must print it out 😁

Can you send it again?

Hope I can find it and send it back to you

Dears ， I added a configuration class, IndexTypeCacheProvider  to abstract the implementation of the index type cache
It seems to be working well so far
I found another way to repeat or duplicate registration, which can also trigger this problem  with session.RegisterIndexes

@hyzx86

OrchardCore Is a multi-tenant system, it will certainly initialize IStore many times, which is not the problem of OC.

OC registers a singleton IStore for each tenant. During registering the IStore we register all indexes as you can see here
https://github.com/OrchardCMS/OrchardCore/blob/ef1eaab35f571fe9500dfe9099a6154fef0a4c9e/src/OrchardCore/OrchardCore.Data.YesSql/OrchardCoreBuilderExtensions.cs#L108-L114
so it wouldn't be creating multiple IStore instances. Maybe the issue is in something else.

@hyzx86

OrchardCore Is a multi-tenant system, it will certainly initialize IStore many times, which is not the problem of OC.

OC registers a singleton IStore for each tenant. During registering the IStore we register all indexes as you can see here https://github.com/OrchardCMS/OrchardCore/blob/ef1eaab35f571fe9500dfe9099a6154fef0a4c9e/src/OrchardCore/OrchardCore.Data.YesSql/OrchardCoreBuilderExtensions.cs#L108-L114
so it wouldn't be creating multiple IStore instances. Maybe the issue is in something else.

@hyzx86

OrchardCore Is a multi-tenant system, it will certainly initialize IStore many times, which is not the problem of OC.

OC registers a singleton IStore for each tenant. During registering the IStore we register all indexes as you can see here https://github.com/OrchardCMS/OrchardCore/blob/ef1eaab35f571fe9500dfe9099a6154fef0a4c9e/src/OrchardCore/OrchardCore.Data.YesSql/OrchardCoreBuilderExtensions.cs#L108-L114
so it wouldn't be creating multiple IStore instances. Maybe the issue is in something else.

Not really, pls see  session.RegisterIndexes

@hyzx86 have you tried implementing IIndexProvider interface for indexes that use generic type instead of ‘IIndexProvider`?

@hyzx86 have you tried implementing IIndexProvider interface for indexes that use generic type instead of ‘IIndexProvider`?

Yes, but eventually I gave up
The TypeProperties cache actually holds The result type of the map function
IndexDescriptor<T, TIndex, TKey>.IndexType
It's only used when you delete an index, because the map index returns null when you delete an index, so you can't get its type
in Store.cs
Clearly limit the use of  DescribeContext<> Maybe we can use a factory method here and implement it in the interface
        private static Func<IDescriptor> MakeDescriptorActivator(Type type)
        {
            var contextType = typeof(DescribeContext<>).MakeGenericType(type);
            return Expression.Lambda<Func<IDescriptor>>(Expression.New(contextType)).Compile();
        }

If you use IIndexProvider then you would provide the type of index using ForType() I never tried it. But, this should work. From there you can caching the dynamic type not the type provided in IIndexProvider<T>

If you need to create IndexTypeCacheProvider, the default implementation should solve the problem you are facing. Meaning your comment have to be addressed by default otherwise you're code can't be accepted since you are adding API that allows generic,

This interface needs to be reimplemented when dynamic type and multi-tenant scenarios are used,

You may want to add interface to the abstraction project maybe like this
public interface IIndexTypeCacheProvider
{
    PropertyInfoAccessor GetPropertyInfoAccessor(PropertyInfo property);

    PropertyInfo[] GetProperties(Type type);

    void Update(Type type);
}

The IndexTypeCacheProvider can go into the Core project.
But, I am not sure that the complexity added by IndexTypeCacheProvider is needed. Maybe @sebastienros can provide more feedback on the approach you taken.

@MikeAlhayek  updated the default index cache key
Now , you can modify the test PopulateIndexUsingGenericType
Comment on the following line
            // update index type cache
            store1.Configuration.IndexTypeCacheProvider?.UpdateCachedType(changedType);

And  run the test ，Errors will be reproduced
There is nothing wrong with using the full name of the type as the Key for the cache.
We can call it after regenerating the type
IndexTypeCacheProvider. UpdateCachedType(Type) updates the cached type
However, in an OC or other project , it may be better to replace it with an in-memory cache or a distributed cache, since both of these are tenant isolated in an OC.
And we can define the full name of the class to start with the tenant name when using dynamic type generation

After a period of test run, it is found that this method is not reliable
Next I'm going to try to implement  IIndexCommand through external code


Add survey link to readme

Signed-off-by: leonardpahlke<EMAIL_ADDRESS>/cc @mkorbi @catblade

Given as this will likely close in the next ~2 weeks, probably not?

@amye  we had discussed closing on the 25th of November.  Temporary-but may get us traffic.  The three of us met this morning and agreed.

LGTM

I think we can temporarily add it, there will be enough updates soon so at one or the other we can take it out than


fix(documentation): remove misplaced plugin

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/scullyio/scully/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?
The ContentFolder plugin (a router plugin) is included in the list of the built-in render plugins.
Issue Number: N/A
The plugin is removed from the list.
He's already inside the list of built-in router plugins
Does this PR introduce a breaking change?

[ ] Yes
[x] No

i close it as i noticed in the recommanded plugins doc that the plugin is stated as both render AND router plugin


[WIP] Members configuration settings

Pick members configuration from settings, sets default
Updates member flag based on settings
Dynamically update subscription info

Closed in favor of https://github.com/TryGhost/Ghost/pull/10537


Custom game with "Character Spell Lists" option disabled starts characters without their first spell

Console or Emulator: emulator- snes9x v1.53
If emulator state which emulator and version.  If console, state which flash cart firmware.
Flags (paste flag string if available):
Description of problem: When creating a custom randomization game without the characters' spells randomized, Mario begins without the Jump spell; in the one game I've gone as far as the second character, Mallow didn't have the Thunderbolt spell, either. Both characters gain their second spell at the appropriate level, but still missing the first.

This is an issue with the old version 7.  Current beta version 8 has this fixed, among numerous other things.
https://randomizer-beta.smrpgspeedruns.com/


[Suggestion]Update the returned result of ServiceMonitor in monitor.md

The result should be
kubectl get servicemonitor
NAME                           AGE
my-release-milvus              54s

PR #435


Selecting the 'Preferences: Color theme' in the palette menu don't work.

By doing Ctrl+Shift+P to enter Command Palette and selecting Preferences Color theme, the action doesn't  do anything.

unexpectedly autoresolved


[Feature request] Broken link detection in dev Server

https://github.com/vuepress/vuepress-next/discussions/780

Try to use vuepress-plugin-md-enhance (remember install with @next flag. It provides an option called linkCheck which is enabled in dev mode by default.


Jackson convertor

It would be fantastic if there were a convertor to go between Spray JSON's JsValue and Jackson's JsonNode. That way, Spray JSON could be more efficiently leveraged in traditional containers (with json2xml a one-way bridge to JAXB). Currently, in order to use Spray JSON in traditional containers requires marshalling to/from String

oh damn, I wrote one of these at my last gig and never OSSd it when I had the chance. Bugger!

I am very interested in this task. Would you consider merging it if I implement it? Or is there cleaner way to make jackson <--> spray interop?
A bit of context. I want to wrap MLeap library as a Finatra web service. Finatra is built with jackson support by default and I want to stick to it but MLeap object could be easily serialised only to JsValue tree. So I thought about writing a JsNode (jackson) <--> JsValue (spray) bridge. I am not sure I could convert all objects given that jackson has some weird nodes like POJO, Binary and Missing. But even partial marshaling should be fine given that in most cases one would not encounter these node.
What do you think?


More UI fixes

Fixes #352
Fixes Seqr Links
Fixes #366

Codecov Report
Base: 67.91% // Head: 67.91% // No change to project coverage :thumbsup:

Coverage data is based on head (e5ee4ea) compared to base (22434e2).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #353   +/-   ##
=======================================
  Coverage   67.91%   67.91%
=======================================
  Files          71       71
  Lines        5620     5620
=======================================
  Hits         3817     3817
  Misses       1803     1803

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Updated ReadME's

Summary of the pull request
Updated the ReadMe for the SampleConfigurations to provide links to more information, and additional guidance on what will happen to their device.
PR checklist

[x] Documentation updated

Made requested changes.


Navigation Toggle buttton not visible but it's there . Colour needs to be changed.

Description
Navigation Toggle buttton is not visible but it's there . It's colour has to be changed.
Relevant files
Mention the files that are relevant to this issue
Screenshot
Please attach screenshots for the same, if applicable

References
Add file reference link here

I am working on it.

#42 #43


Update location.yaml

Included request parameter “MaxAge’ to give the developer option to mention the maximum aging of the location info, accepted by the application.
If the developer wants only real-time or near real-time information, we as operators then can avoid validating against stale location info resulted from network reasons such as device switched-off, not in servicing area, device issues etc.

Hello @jlurien
Regarding your question, my understanding is:

value 0 mean 'now' (real time localisation)
it should be the value default.
mandatory or not? I haven't a strong opinion but I understood that https://github.com/patrice-conil has a preference for the mandatory version.
as we use 3GPP Monitoring event, the age could be useful when the location type is "LAST_KNOWN_LOCATION" (and not "CURRENT_LOCATION").  Indeed if I got from my back end system a localisation with a  "ageOfLocationInfo"= 19, if you asked me with maxAge = 0  I cannot localize you while if you asked me with maxAge = 20 I send back  "verificationResult": true. I'm not sure about the maximum ageOfLocalisation we are able to manage in the BE.
Completely agree with your point about defining also the expected behavior when the requested age cannot be fulfilled. In my previous example, if I got from my back end system a localisation with a  "ageOfLocationInfo"= 19, if you asked me with maxAge = 0  I should probably sent an explicit 'unable to localize' instead of  "verificationResult": false. This could need additional discussion because from a boolean we can switch to a string with an enum (true/false/unableToProvide) .

Hello @jlurien Regarding your question, my understanding is:

value 0 mean 'now' (real time localisation)
it should be the value default.
mandatory or not? I haven't a strong opinion but I understood that https://github.com/patrice-conil has a preference for the mandatory version.
as we use 3GPP Monitoring event, the age could be useful when the location type is "LAST_KNOWN_LOCATION" (and not "CURRENT_LOCATION").  Indeed if I got from my back end system a localisation with a  "ageOfLocationInfo"= 19, if you asked me with maxAge = 0  I cannot localize you while if you asked me with maxAge = 20 I send back  "verificationResult": true. I'm not sure about the maximum ageOfLocalisation we are able to manage in the BE.
Completely agree with your point about defining also the expected behavior when the requested age cannot be fulfilled. In my previous example, if I got from my back end system a localisation with a  "ageOfLocationInfo"= 19, if you asked me with maxAge = 0  I should probably sent an explicit 'unable to localize' instead of  "verificationResult": false. This could need additional discussion because from a boolean we can switch to a string with an enum (true/false/unableToProvide) .

My concerns here about API usability and "dev friendliness":
If default is 0, meaning "current location" (I prefer this term instead of real-time), and we make maxAge required, we are forcing clients to explicitly provide a maxAge. So if default is the best service, then I wonder in which cases a client would ask for anything worse on purpose. Only maybe if clients get a first response of type "unable to provide" and then ask again for "last know location". Then, a better design approach would probably be to provide the maxAge indication as part of the response, instead of providing it in the request. Clients may not be familiar with typical ages in our networks to decide which threshold in minutes is enough to get the last known location if available.

@bigludo7 , @jlurien PR is approved now and I do not have write access. can one of you merge this PR now?

@bigludo7, @ukkaps,
Sorry, but I had some pending comments, and there is still not agreement about how to add this new parameter. I will revert this commit and we can discuss further on this on tomorrow's meeting. For other PRs, please let's wait until we have at least the approval of at least 2 people.
In the meantime, if you could read my latest comments and follow the discussion, it would be great. Thanks

Hi @bigludo7,

The current proposal considers maxAge as an input parameter, so the response would depend on the requested threshold about age. But maybe we could think if it fits better as part of the response. Quoting myself :) "Clients may not be familiar with typical ages in our networks to decide which threshold in minutes is enough to get the last known location if available."
For the second point I guess you assume that network location is fresh, not stalled at A,B. In that case is clearly false, yes.

The concept of considering the age makes sense but we have to decide how to fit it in the API design, request or response, optional or required, default or not, any associated error if needed, etc. We are not in a hurry for this so let's try to find the best approach.


IE9 - performance problems

There are some serious performance issues when scrolling in IE9 http://facebook.github.io/fixed-data-table/example-object-data.html -> probably because of  how IE9 fetches asynchronously the images?

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.


Improved Error Handling for SDK tools

On Windows, the Viewer & DQT can silently fail in certain situations. This is clearly very bad user experience, hence this PR.
List of changes:

Get detailed error message out of GLFW, like in cases of broken OpenGL
Display top level exception as a popup message with an option to open new GitHub issue
Catch unhandled exceptions from all threads within the process
Catch SEH exceptions (similar to signals), covering Segmentation Faults, Illegal Instructions, and more cases not covered by regular C++ exception handler
Make all 3rd-party dependencies of the Viewer / DQT delay-loaded. This way when some DLL is missing, the error message is more detailed with an option to report the issue

What about unit-testing this? Can we potentially set up a unit-test where the .exe is run without any DLLs and somehow catch that the error is out? Something for Aseel?

Also, what happens under Linux? Not meant to be addressed here at all?

Failure to load will go through SEH and land in line 68 of the bootstrap.
Regarding unit-test, I don't know - these are very much application level guards. What type of test would you recommend?


:bug: Fix resetting bm servers

What this PR does / why we need it:
The bare metal servers are supposed to be resetted when a HetznerBareMetalMachine object is deleted. At the moment, there are two problems. First, the SSH secret reference was deleted before it was used and second the SSH command didn't work.
TODO: Update SSH command
TODOs:

[x] squash commits
[ ] include documentation
[ ] add unit tests

Current failure:
"[preflight] Running pre-flight checks\n[reset] No etcd config found. Assuming external etcd\n[reset] Please, manually reset etcd to prevent further issues\n[reset] Stopping the kubelet service\n[reset] Unmounting mounted directories in "/var/lib/kubelet"\n[reset] Deleting contents of directories: [/etc/kubernetes/manifests /etc/kubernetes/pki]\n[reset] Deleting files: [/etc/kubernetes/admin.conf /etc/kubernetes/kubelet.conf /etc/kubernetes/bootstrap-kubelet.conf /etc/kubernetes/controller-manager.conf /etc/kubernetes/scheduler.conf]\n[reset] Deleting contents of stateful directories: [/var/lib/kubelet]\n\nThe reset process does not clean CNI configuration. To do so, you must remove /etc/cni/net.d\n\nThe reset process does not reset or clean up iptables rules or IPVS tables.\nIf you wish to reset iptables, you must do so manually by using the "iptables" command.\n\nIf your cluster was setup to utilize IPVS, run ipvsadm --clear (or similar)\nto reset your system's IPVS tables.\n\nThe reset process does not clean your kubeconfig files and you must remove them manually.\nPlease, check the contents of the $HOME/.kube/config file.\n"
"W0710 17:39:29.701352    7648 removeetcdmember.go:85] [reset] No kubeadm config, using etcd pod spec to get data directory\n"

New idea: we de-provision via installing a vanilla-ubunt image.

New idea: we de-provision via installing a vanilla-ubunt image.

No, de-provision via installing a vanilla-ubunt image would need new states in the state-machine.
kubeadm reset was writing to stderr which made the command fail. But overall it did not fail.
Solution: redirect stderr to stdou and ignore return code.


[DO NOT MERGE] Define OpAMP Protobuf schema stability guarantees

The guarantees mostly mirror the proposal for OTLP (https://github.com/open-telemetry/opentelemetry-proto/pull/432), the primary difference being that OpAMP has no gRPC services defined and does not need any guarantees for service definitions.
Resolves https://github.com/open-telemetry/opamp-spec/issues/134

@andykellr I am opening this draft so that we can discuss what else is necessary to resolve https://github.com/open-telemetry/opamp-spec/issues/134

I think this looks good.

@andykellr please review.


Is this package maintened by newrelic ?

Hi,
I'm starting to newrelic but it seems this react-native module isn't maintened. Is there any work planned by NewRelic on improving react-native support ?

See https://discuss.newrelic.com/t/experimental-react-native-support/115888 and https://docs.newrelic.com/docs/mobile-monitoring/new-relic-monitoring-react-native/monitor-your-react-native-application/, but note that javascript errors are currently only reported as custom events and don't support automatic symbolication (you have to do that manually in your local env).
I think it's safe to say this library is deprecated.

Thank you very much !


Feature/rewind multipart deux

Merge in isakovp's fix for multipart. This should fix our 'infinite hang on multipart post (uploading a recording)' issue
See https://github.com/lostisland/faraday/pull/347 for some more.
This is a different PR so we don't have a merge commit.

@sfgeorge

@runningferret Just noticed 0.9.1 just came out.  Shall I update our base to that before merging this in?

@sfgeorge Up to you... neither of those things in the changelog look particularly relevant to our usage so I doubt it's an issue either way.

Reason I really mention is because lostisland/faraday#347 is no longer mergeable, and I'm afraid that 0.9.1 might be why.  Thinking on whether it may be good to resolve the conflict now, as supposed to having the pain later when it's no longer fresh in our minds.


Bugfix/ip 238 angular2 check

Concerning #238 I switched out some imports to require statements for conditional loading. You check if your window object has the ng attribute.
If so you're probably running Angular2 so you can start your adapter and hook into the DOM.
I don't feel it's the cleanest fix in the world, since I prefer ES6 imports, but for the sake of low coupling I felt that backend.ts should be responsible for only loading IF window.ng is defined.

@mobinni We can wrap second if inside the first one we don't need it separate other than that looks good. Also, Can you add a comment there why we are doing it like this?

@sumitarora done!

Hey @mobinni, thanks a lot for the contribution! After reviewing your solution, I agree with your stance on preferring import over require. Here's what I have in mind in terms of a cleaner approach:

Have content-script be responsible for determining whether backend.js should be injected.
Create a separate script with a simple function to determine whether ng is available (similar to what you have already), if so, post a message for content-script.
Inject said script to the page batarangle is trying to debug, just like how backend.js is injected.

This way does require touching more parts of the application, so please DM me on slack if you have any question. :)


techynroll.com is not bypassed

Domain
Techynroll.com
Link
https://techynroll.com/does-refinancing-hurt-your-credit/
Version
0.0.0
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
No response
(Optional) Anything else?
Original link: https://defaultfreeshort.in/DSqw
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Done. Thanks for a good report  : )
domainBypass("techynroll.com", () => {
    awaitElement("a#enablebtn", a => {
        safelyAssign(a.href);
    })
})


fix make install target

What is the purpose of this change? What does it change?

Was the change discussed in an issue?
fixes #1037

How to test changes?
make install
make bin
make cross

to clean up after make install run rm $GOPATH/bin/odo

closing this as i just noticed that this is also fixed in #1038

Yeah, I didn't open an issue for it since I just now noticed it as part of my attempt to run the e2e tests locally :)


Missing Kotlinc

Type: Bug
I have Intellij IDEA CE installed, and I can't seem to find the kotlinc compiler in the setup.  What can I do, and if possible show me where I can get a working copy of kotlinc.
VS Code version: Code 1.75.1 (441438abd1ac652551dbe4d408dfcec8a499b8bf, 2023-02-08T21:32:34.589Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz (8 x 2419)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.77GB (6.89GB free)

Process Argv
--crash-reporter-id 878a6b85-9049-465e-849e-1cc9b3b95d57

Screen Reader
no

VM
0%

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Disable hw error recovery, add logging

Disable hw error recovery. Add additional logging to help determine the hardware error cause
Disable hw error recovery by commenting the line wrapping uod.hwl
Set log level for UaClient to Warning
Subtask of #401

Immidiate analysis conclusion:
The cause of hwl.is_connected being False is that subclasses of HardwareLayerBase were not required to make calls to super() in connect and disconnect which means the field was not set. Furthermore, because the field was not set, ErrorRecoveryDecorator would not initialize in the correct state (OK for connected hardware)


Refactor Partial Path Queries

Refactor the partial path query templates to use the latest version of the partial flow interface.

Closing in favor of #33


Blog post: Planning project sprint notes 12 Feb

@mattlucht -- Next sprint notes for the Planning Permissions project blog.

👍


CentOS虚拟机中启动OOM

There is insufficient memory for the Java Runtime Environment to continue.
Native memory allocation (mmap) failed to map<PHONE_NUMBER> bytes for committing reserved memory.
Possible reasons:
The system is out of physical RAM or swap space
In 32 bit mode, the process size limit was hit
Possible solutions:
Reduce memory load on the system
Increase physical memory or swap space
Check if swap backing store is full
Use 64 bit Java on a 64 bit OS
Decrease Java heap size (-Xmx/-Xms)
Decrease number of Java threads
Decrease Java thread stack sizes (-Xss)
Set larger code cache with -XX:ReservedCodeCacheSize=
This output file may be truncated or incomplete.

Out of Memory Error (os_linux.cpp:2673), pid=8434, tid=139895182022400

JRE version:  (8.0_40-b25) (build )
Java VM: Java HotSpot(TM) 64-Bit Server VM (25.40-b25 mixed mode linux-amd64 compressed oops)
Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again

启动成功以后立即挂掉，OOM的问题怎么解决？

Ensure that the Linux OS has enough memory for Kafka-Eagle applications (default is 2G)

ke.sh
# Default Memory
export KE_JAVA_OPTS="-server -Xmx2g -Xms2g"


ISPN-5153 CacheEntryInvalidatedEvent is not raised

Added back in invalidation event
Fixed issue where remove events were being sent as post event
** Affected Invalidation Events
** Affected Eviction Events

https://issues.jboss.org/browse/ISPN-5153

Pushed, thanks


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #29.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #29

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Didn't see that 0.5.0 was released already 🤦


Error runnig getAllPingableDevices inside snap confinement

As I am using this package inside of a snap there can be new limitations that can happen.
Snap is a very confined environment and the developer needs to specify in advance (and sometimes get permission) in order to use different systems calls and info.
This leads me to a new bug that I am not sure how much it is common to get and it is permission denied on using the avahi-browse command.
This is my log.
  Check that /usr/bin/timeout exists
    command: timeout 2s avahi-browse --all -p
  Check that /usr/bin/timeout exists
    command: timeout 2s mdns-scan
You can make the mdns process better by installing `mdns-scan`
Unhandled exception:
ProcessException: Permission denied
  Command: ping -O -n -W 1 -i 1 -t 255 -c 1 <IP_ADDRESS>
#0      _ProcessImpl._start (dart:io-patch/process_patch.dart:401)
#1      Process.start (dart:io-patch/process_patch.dart:38)
#2      BasePing.platformProcess (package:dart_ping/src/ping/base_ping.dart:71)
#3      BasePing._onListen (package:dart_ping/src/ping/base_ping.dart:86)
#4      _runGuarded (dart:async/stream_controller.dart:814)
#5      _StreamController._subscribe.<anonymous closure> (dart:async/stream_controller.dart:692)
#6      _BufferingStreamSubscription._guardCallback (dart:async/stream_impl.dart:417)
#7      _StreamController._subscribe (dart:async/stream_controller.dart:691)
#8      _ControllerStream._createSubscription (dart:async/stream_controller.dart:827)
#9      _StreamImpl.listen (dart:async/stream_impl.dart:473)
#10     _StreamIterator._initializeOrDone (dart:async/stream_impl.dart:1028)
#11     _StreamIterator.moveNext (dart:async/stream_impl.dart:1004)
#12     HostScanner._getHostFromPing (package:network_tools/src/host_scanner.dart)
#13     HostScanner.getAllPingableDevices (package:network_tools/src/host_scanner.dart:41)
<asynchronous suspension>

To fix that and still allow a better way of finding all the SRV lists in Linux if it is possible I am suggesting the following.

Short fix for most cases - by default it will not use the Linux-specific implementation, a boolean value can be passed if needed.
We find a way to catch the error and in case of missing permission it calls the non-Linux specific implementation.

I think 2 will be a little difficult since it is hard to debug the program when it runs as a snap so we can just level it open and not touch it at all.
We can add to it  p4 after completing 1 for now.

Opened the issue wrong.
Will close it.


Starting a scene reveals a 'duration' datetime picker in the scene

Missing break statement. Fixed and waiting for release of v3.0.1


Clean-Up Button

Give users the option to delete all of the resources made after deployment via a UI button in Portal.

Engineering work needed is small, but this requires a change in design.

Closing for now as we will wait on usability study findings to figure out what to do in regard to resource deletion.


ARROW-8937: [C++] Implement strptime scalar string to timestamp kernel

For now, this is not exposed in any public APIs beyond using CallFunction to invoke it.

+1. Will merge once CI is green

The ARM failure looks like a flake
/usr/bin/ccache /usr/bin/c++  -DARROW_EXPORTING -DARROW_HAVE_NEON -DARROW_HDFS -DARROW_JEMALLOC -DARROW_JEMALLOC_INCLUDE_DIR="" -DARROW_NO_DEPRECATED_API -DARROW_USE_GLOG -DARROW_WITH_BACKTRACE -DARROW_WITH_BROTLI -DARROW_WITH_BZ2 -DARROW_WITH_LZ4 -DARROW_WITH_SNAPPY -DARROW_WITH_TIMING_TESTS -DARROW_WITH_ZLIB -DARROW_WITH_ZSTD -DGTEST_LINKED_AS_SHARED_LIBRARY=1 -DURI_STATIC_BUILD -Isrc -I/arrow/cpp/src -I/arrow/cpp/src/generated -isystem /arrow/cpp/thirdparty/flatbuffers/include -isystem jemalloc_ep-prefix/src -isystem googletest_ep-prefix/include -isystem /arrow/cpp/thirdparty/hadoop/include -isystem orc_ep-install/include -Wno-noexcept-type  -fdiagnostics-color=always -ggdb -O0  -Wall -Wno-conversion -Wno-sign-conversion -Wno-unused-variable -Werror -march=armv8-a  -g -fPIC   -std=gnu++11 -MD -MT src/arrow/CMakeFiles/arrow_objlib.dir/compute/kernels/vector_take.cc.o -MF src/arrow/CMakeFiles/arrow_objlib.dir/compute/kernels/vector_take.cc.o.d -o src/arrow/CMakeFiles/arrow_objlib.dir/compute/kernels/vector_take.cc.o -c /arrow/cpp/src/arrow/compute/kernels/vector_take.cc
c++: fatal error: Killed signal terminated program cc1plus
compilation terminated.

This may be related to code size -- some near-term refactoring I am undertaking in this file should address that


Pipeline uploading full model after first run

Once a pipeline that contains a model has been run, the full initialised model is then uploaded. This is particularly inconvenient for huggingface transformer models.
Example:
with Pipeline("HF pipeline") as builder:
    input_str = Variable(str, is_input=True)
    builder.add_variables(input_str)

    hf_model = TransformersModelForCausalLM(
        model_path="EleutherAI/gpt-neo-125M",
        tokenizer_path="EleutherAI/gpt-neo-125M",
    )

    output_str = hf_model.predict(input_str)

    builder.output(output_str)

output_pipeline = Pipeline.get_pipeline("HF pipeline")

# Run the model
output_pipeline.run("Hello there")

print("Now uploading GPTNeo pipeline")
# The next line will upload the full 1GB model
uploaded_pipeline = api.upload_pipeline(output_pipeline)```

Rather than uploading whatever object is passed to upload_pipeline we could check whether it is an instance or not. AFAIK only the class is needed rather than the instance; the builder knows the __init__ arguments so can take the class, re-initialise it on the server, and then call the load/predict functions. (This does assume the user calls no other instance methods, e.g. hf_model.some_other_method(), which mutate self.)
To extract the class from an instance:
import inspect

def class_from_object(instance_or_class):
    if inspect.isclass(instance_or_class):
        return instance_or_class
    else:
        return type(instance_or_class)

Sounds good - we can try this out post API v1 migration. I guarantee people will upload after a local run so this will def have to be in there post public launch.


JDBC Sink Leaks DB Credentials

Describe the bug
The JDBC sink leaks db credentials (the in contents of 'configs') into logs and API results.
To Reproduce
Steps to reproduce the behavior:
0. Start Pulsar standalone

Register a JDBC sink
Observe the log output on a standalone pulsar instance
It contains the credentials in a line starting with 'ProcessBuilder starting the process with' in a JSON object called '--function_details'
Fetch the sink details using 'bin/pulsar-admin sinks get'
Observe that the result also contains the credentials

Expected behavior
Secrets should not be logged or returned by the API.

@tuteng Would you please help on this issue?

Thank you for reporting this problem. The current configuration is printed in function_worker. We may need to plan how to replace this sensitive information. Please let us know if you have any good suggestions. @jeloba

Maybe storing the password in the 'secrets' map instead of 'configs' would be an option?

+1 we should pass in the credentials through secrets map.

cc @nicoloboschi as you recently touch JDBC sink component, do you think this issue is still relevant and we should pick it up?

Closed as stale. Please open a new issue if it's still relevant to the maintained versions.


Add JWT_VERIFY_EXPIRATION option for jwt.decode

resolves #107

Just adding this to option to the app.config isn't enough. You need to update the calls to the underlying pyjwt to disable it. Per their docs here: https://pyjwt.readthedocs.io/en/latest/usage.html

Expiration time is automatically verified in jwt.decode() and raises jwt.ExpiredSignatureError if the expiration time is in the past .... You can turn off expiration time verification with the verify_exp parameter in the options argument


Error: @module-federation/nextjs-mf tried to access tslib

Summary
If you encounter the error Error: @module-federation/nextjs-mf tried to access tslib in your yarn workspaces environment, please install the tslib in project(monorepo)'s root. It should run fine.
Version
yarn: 3.5.0
next: 13.2.4
@module-federation/nextjs-mf: 6.2.2
tslib: 2.5.0
Error report
error - Failed to load next.config.js, see more info here https://nextjs.org/docs/messages/next-config-error
Error: @module-federation/nextjs-mf tried to access tslib, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

Required package: tslib
Required by: @module-federation/nextjs-mf@virtual:808e5805779f65c511d5d05f8eeefd3fb378684c1abe9523d005e11e76d684acd99ad6083e178e1a0a410520e5e58ad5ee8f8fd188cad32ba4f1169348538ab1#npm:6.2.2 (via /Users/kiyung/vscode/playground/mfa-test/.yarn/__virtual__/@module-federation-nextjs-mf-virtual-37d2e681ab/0/cache/@module-federation-nextjs-mf-npm-6.2.2-5046ff5f86-c1a4903ed9.zip/node_modules/@module-federation/nextjs-mf/src/plugins/)

Require stack:
- /Users/kiyung/vscode/playground/mfa-test/.yarn/__virtual__/@module-federation-nextjs-mf-virtual-37d2e681ab/0/cache/<EMAIL_ADDRESS>- /Users/kiyung/vscode/playground/mfa-test/.yarn/__virtual__/@module-federation-nextjs-mf-virtual-37d2e681ab/0/cache/<EMAIL_ADDRESS>- /Users/kiyung/vscode/playground/mfa-test/services/main/next.config.js
    at Function.require$$0.Module._resolveFilename (/Users/kiyung/vscode/playground/mfa-test/.pnp.cjs:13159:13)
    at Function.mod._resolveFilename (/Users/kiyung/vscode/playground/mfa-test/.yarn/__virtual__/next-virtual-fea21aff0c/0/cache/next-npm-13.2.4-eedd00f0a6-8531dee41b.zip/node_modules/next/dist/build/webpack/require-hook.js:23:32)
    at Function.require$$0.Module._load (/Users/kiyung/vscode/playground/mfa-test/.pnp.cjs:13009:42)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/Users/kiyung/vscode/playground/mfa-test/.yarn/__virtual__/@module-federation-nextjs-mf-virtual-37d2e681ab/0/cache/@module-federation-nextjs-mf-npm-6.2.2-5046ff5f86-c1a4903ed9.zip/node_modules/@module-federation/nextjs-mf/src/plugins/NextFederationPlugin.js:8:17)
    at Module._compile (node:internal/modules/cjs/loader:1103:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1157:10)
    at Object.require$$0.Module._extensions..js (/Users/kiyung/vscode/playground/mfa-test/.pnp.cjs:13203:33)
    at Module.load (node:internal/modules/cjs/loader:981:32)

If i need to add it as a peer dep or dependency, feel free to open PR adjusting as needed

Thank you @ScriptedAlchemy :)

If i need to add it as a peer dep or dependency, feel free to open PR adjusting as needed

In my opinion, Adding The peer deps or deps doesn't matter of this issue.
I installed the tslib on my workspaces directories but it doesn't work.
Installing the tslib on the root directory works only.
Also, I tried to add tslib dependency on @module-federation/nextjs-mf's package.json and installed it but it doesn't work too.
I created project you can test 😄  (link)

If you add it to package.json in your node modules, then run install - it kills it off and reinstalls it fresh haha. You could run yarn pack on my repo and install the tarball, or make edit, run yarn pack then install from tarball to see if that helps


Fluent Details List- Manual Sorting

Im having an issue with manual sorting of a SQL database. Visually they are not sorting, but if I check the record items after sorting I can see the sorting take place, the only thing that is visually happening is the Sort Arrow changing direction

@denise-msft I have followed those directions exactly, I just went back into the app and now it wont even display the records at all..
I even changed the datasource from SQL to Dataverse to rule out the manual sorting logic and same thing it wont even show the data at all
where would you like for me to pull snippets from?? The OnChange, Items and Columns??
Also my version of the component is 1.1.29


Switched to short_help kwarg from help for command help text.

The truncated help text for the "Commands:" section (on pipenv) threw me off some while first looking at and learning Pipenv.
I made a quick PR to see how it would be with displaying the full help text.
Eliminating the auto-truncation by the click library was achieved by switching from the help kwarg to short_help in the click.command decorator.

This seems sensible to me. Thanks @timworx, great first contribution!


Background operator doesn't work correctly

The statement of the process start in the background doesn't work correctly.
Example:
(*_*)$>sleep 100 &

[1] 3455

[1] Done sleep 100
(*_*)$>

The process is started and immediately terminated.
This process isn't displayed in the work list after that.

Also, although it is printed that the task is completed, it remains in the background.
You can check it using the ps command.


TextConverter output wrong line with text

Hi@pietermarsman,
I am trying to convert  a PDF to text and have come across a output problem.
TextConverter generates the following:
Education Background
Work Experience
2010.09 -2014.07
Xian Engineer University
Art Design
Bachelor
Degree
2012.09 -2013.11
The result I excepted:
Education Background
2010.09 -2014.07
Xian Engineer University
Art Design
Bachelor
Degree
Work Experience
2012.09 -2013.11
Here is my code.
def pdf_to_detail(path):
    rsrcmgr = PDFResourceManager()
    retstr = StringIO()
    codec = 'utf-8'
    laparams = LAParams()
    device = TextConverter(rsrcmgr, retstr, codec=codec, laparams=laparams)
    fp = open(path, 'rb')
    interpreter = PDFPageInterpreter(rsrcmgr, device)
    maxpages = 0
    pagenos=set()
    for page in PDFPage.get_pages(fp, pagenos, maxpages=maxpages, check_extractable=True):
        interpreter.process_page(page)
    fp.close()
    device.close()
    detail = retstr.getvalue()
    print(detail)
    return(detail)

the version of pdfminer.six is 20191110
test (1).pdf

Hi @Tam678, thanks for your question. Could you try with the newest version of pdfminer.six and see if the problem persists? A lot of small things changes around the turn of the year.

Hi @pietermarsman ,thanks for you help.I'm trying to use the newest version of pdfminer.six.
Before using the newest one,I've changed line_margin in LAParams().It works,but there still exited something contents incorrect.Anyway,thank you very much for your response！


Recordset doesn't handle url-encoded table names in fragment ID

I try to visit https://synapse-dev.isrd.isi.edu/chaise/recordset/#1/Zebrafish:Brain%20Anatomy and get stuck with a spinner and a javascript error like Error: table is undefined

So what's happening is that when we get the window.location.hash it has encoded the space   as %20, so the app looks for a table named Brain%20Anatomy.

fix is committed to master. can be viewed here now:
https://synapse-dev.isrd.isi.edu/~schuler/chaise/recordset/#1/Zebrafish:Brain%20Anatomy
However,... the record app has the same bug. :-(

confirmed https://synapse-dev.isrd.isi.edu/chaise/recordset/#1/Zebrafish:Brain%20Anatomy now works


[Enhancement] Fix unstable ut caused by missing one invocation

What type of PR is this：

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

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

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

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

Bugfix cherry-pick branch check:

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

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

[FE PR Coverage Check]
:heart_eyes: pass : 4 / 5 (80.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/system/ComputeNode.java
0
1
00.00%
[425]

:large_blue_circle:
com/starrocks/system/HeartbeatMgr.java
4
4
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

[FE PR Coverage Check]
:disappointed: fail : 3 / 4 (75.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/system/ComputeNode.java
1
2
50.00%
[437]

:large_blue_circle:
com/starrocks/system/HeartbeatMgr.java
2
2
100.00%
[]

run starrocks_admit_test


Extension errors out with an MSBuild crash due to trying to load files it shouldn't

Type: Bug
Issue Description
Extension errors out with an MSBuild crash due to trying to load an old unsupported vcproj file and predictably failing, instead of ignoring it because its a C++ project.
Should be known that the vcproj files in question are part of a project outside of my control, and their existence in the same folder as the C# project is due to an automated process that will overwrite any changes made. Also should be known that Omnisharp handles this scenario just fine.
Steps to Reproduce
Open a .NET SDK C# project that's in a folder containing an old vcproj.
Note how none of the intellisense stuff works.
Expected Behavior
The C# extension's LSP server should not be trying to load C++ projects.
Actual Behavior
The C# extension's LSP server tries to C++ projects and crashes.
Logs
C# log
Using dotnet configured on PATH
Dotnet path: /home/rstat1/dotnet/dotnet
Activating C# standalone...
waiting for named pipe information from server...
[stdout] {"pipeName":"/tmp/1f6e96be.sock"}
received named pipe information from server
attempting to connect client to server...
client has connected to server
[Info  - 3:22:56 PM] [Program] Language server initialized
[Info  - 3:22:56 PM] [LanguageServerProjectSystem] Loading /home/rstat1/code/volt2/out/update_engine-bin/jobcache/36be5d2c-673c-47a2-9e13-588339114bc2/Tests/MFC/mfc1/mfc1.sln...
[Error - 3:22:57 PM] [LanguageServerHost] Microsoft.CodeAnalysis.MSBuild.Rpc.RemoteInvocationException: An exception of type Microsoft.Build.Exceptions.InvalidProjectFileException was thrown: The project file "mfc1.vcproj" is in the ".vcproj" file format, which MSBuild no longer supports. Please convert the project by opening it in the Visual Studio IDE or running the conversion tool, or use MSBuild 3.5 or earlier to build it.  /home/rstat1/code/volt2/out/update_engine-bin/jobcache/36be5d2c-673c-47a2-9e13-588339114bc2/Tests/MFC/mfc1/mfc1.sln
   at Microsoft.CodeAnalysis.MSBuild.Rpc.RpcClient.InvokeCoreAsync(Int32 targetObject, String methodName, List`1 parameters, Type expectedReturnType, CancellationToken cancellationToken) in /_/src/Workspaces/Core/MSBuild/Rpc/RpcClient.cs:line 148
   at Microsoft.CodeAnalysis.MSBuild.Rpc.RpcClient.InvokeAsync[T](Int32 targetObject, String methodName, List`1 parameters, CancellationToken cancellationToken) in /_/src/Workspaces/Core/MSBuild/Rpc/RpcClient.cs:line 114
   at Microsoft.CodeAnalysis.LanguageServer.HostWorkspace.LanguageServerProjectSystem.OpenSolutionAsync(String solutionFilePath) in /_/src/Features/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/HostWorkspace/LanguageServerProjectSystem.cs:line 109
   at Microsoft.CodeAnalysis.LanguageServer.HostWorkspace.LanguageServerProjectSystem.OpenSolutionAsync(String solutionFilePath) in /_/src/Features/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/HostWorkspace/LanguageServerProjectSystem.cs:line 116
   at Microsoft.CommonLanguageServerProtocol.Framework.QueueItem`3.StartRequestAsync(TRequestContext context, CancellationToken cancellationToken) in /_/src/Features/LanguageServer/Microsoft.CommonLanguageServerProtocol.Framework/QueueItem.cs:line 136

C# LSP Trace Logs
lsp-trace.txt
Environment information
VSCode version: 1.84.2
C# Extension: 2.12.19
Using OmniSharp: false
Dotnet Information
.NET SDK:
 Version:           8.0.100
 Commit:            57efcf1350
 Workload version:  8.0.100-manifests.6c33ef20
Runtime Environment:
OS Name:     fedora
OS Version:  39
OS Platform: Linux
RID:         linux-x64
Base Path:   /home/rstat1/dotnet/sdk/8.0.100/
.NET workloads installed:
Workload version: 8.0.100-manifests.6c33ef20
There are no installed workloads to display.
Host:
Version:      8.0.0
Architecture: x64
Commit:       5535e31a71
.NET SDKs installed:
8.0.100-rc.2.23502.2 [/home/rstat1/dotnet/sdk]
8.0.100 [/home/rstat1/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 8.0.0-rc.2.23480.2 [/home/rstat1/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.0 [/home/rstat1/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 8.0.0-rc.2.23479.6 [/home/rstat1/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.0 [/home/rstat1/dotnet/shared/Microsoft.NETCore.App]
Other architectures found:
None
Environment variables:
DOTNET_ROOT       [/home/rstat1/dotnet]
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Visual Studio Code Extensions

Extension
Author
Version
Folder Name

ayu
teabyii
1.0.5
teabyii.ayu-1.0.5

coder-remote
coder
0.1.28
coder.coder-remote-0.1.28

codespaces
GitHub
1.16.2
github.codespaces-1.16.2

cpptools
ms-vscode
1.19.1
ms-vscode.cpptools-1.19.1-linux-x64

csharp
ms-dotnettools
2.12.19
ms-dotnettools.csharp-2.12.19-linux-x64

ecdc
mitchdenny
1.8.0
mitchdenny.ecdc-1.8.0

EditorConfig
EditorConfig
0.16.4
editorconfig.editorconfig-0.16.4

firefly
ankitcode
4.0.0
ankitcode.firefly-4.0.0

format-json
ClemensPeters
1.0.3
clemenspeters.format-json-1.0.3

go
golang
0.40.0
golang.go-0.40.0

json-to-go
maracko
0.3.1
maracko.json-to-go-0.3.1

kotlin
fwcd
0.2.32
fwcd.kotlin-0.2.32

mongodb-vscode
mongodb
1.3.1
mongodb.mongodb-vscode-1.3.1

ng-template
Angular
17.0.2
angular.ng-template-17.0.2

ninja
surajbarkale
0.0.1
surajbarkale.ninja-0.0.1

nix-develop
jamesottaway
0.0.1
jamesottaway.nix-develop-0.0.1

ols
DanielGavin
0.1.22
danielgavin.ols-0.1.22

one-dark-darker
JoelCrosby
1.0.4
joelcrosby.one-dark-darker-1.0.4

prettify-json
mohsen1
0.0.3
mohsen1.prettify-json-0.0.3

remote-containers
ms-vscode-remote
0.321.0
ms-vscode-remote.remote-containers-0.321.0

remote-explorer
ms-vscode
0.4.1
ms-vscode.remote-explorer-0.4.1

remote-ssh
ms-vscode-remote
0.107.1
ms-vscode-remote.remote-ssh-0.107.1

remote-ssh-edit
ms-vscode-remote
0.86.0
ms-vscode-remote.remote-ssh-edit-0.86.0

render-crlf
medo64
1.7.1
medo64.render-crlf-1.7.1

rest-client
humao
0.25.1
humao.rest-client-0.25.1

rust-analyzer
rust-lang
0.3.1748
rust-lang.rust-analyzer-0.3.1748-linux-x64

shader
slevesque
1.1.5
slevesque.shader-1.1.5

theme-monokai-pro-vscode
monokai
1.2.1
monokai.theme-monokai-pro-vscode-1.2.1

url-encode
flesler
1.1.0
flesler.url-encode-1.1.0

vsc-material-theme
Equinusocio
34.3.0
equinusocio.vsc-material-theme-34.3.0

vsc-material-theme-icons
equinusocio
3.2.0
equinusocio.vsc-material-theme-icons-3.2.0

vscode-antlr4
mike-lischke
2.4.3
mike-lischke.vscode-antlr4-2.4.3

vscode-clangd
llvm-vs-code-extensions
0.1.25
llvm-vs-code-extensions.vscode-clangd-0.1.25

vscode-css-formatter
aeschli
1.0.2
aeschli.vscode-css-formatter-1.0.2

vscode-dotnet-runtime
ms-dotnettools
2.0.0
ms-dotnettools.vscode-dotnet-runtime-2.0.0

vscode-gn
npclaudiu
1.1.0
npclaudiu.vscode-gn-1.1.0

vscode-icons
vscode-icons-team
12.6.0
vscode-icons-team.vscode-icons-12.6.0

vscode-proto
sankethdev
0.0.6
sankethdev.vscode-proto-0.0.6

Extension version: 2.12.19
VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:50:47.800Z)
OS version: Linux x64 6.5.12-300.fc39.x86_64
Modes:

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 4100)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 0, 0

Memory (System)
31.12GB (24.18GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome

XDG_SESSION_TYPE
wayland

So the exception seems to be coming from MSBuild here:
https://github.com/dotnet/msbuild/blob/f97064d3b83485628dec0769659e786b55f6835a/src/Build/Construction/Solution/SolutionFile.cs#L1297
It doesn't appear there's any public way to bypass that; there's an internal field but that's not settable via a Public API.


Locations for Valorant Maps

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Need to find a way to get all locations for valorant maps, perhaps manually adding them

i was going to work on this as part of refactoring some parts of the code if thats fine

Ok


Error occurred when executing CLIPTextEncode (NSP): not enough values to unpack (expected 2, got 1)

Error occurring after git pull this morning.
WAS Node Suite: CLIPTextEncode Prased Prompt:
 (abstract organic architecture:1.23), Sky blue, James Stokoe
!!! Exception during processing !!!
Traceback (most recent call last):
  File "C:\Users\somebody\ComfyUI\ComfyUI\execution.py", line 145, in recursive_execute
    output_data, output_ui = get_output_data(obj, input_data_all)
  File "C:\Users\somebody\ComfyUI\ComfyUI\execution.py", line 75, in get_output_data
    return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
  File "C:\Users\somebody\ComfyUI\ComfyUI\execution.py", line 68, in map_node_over_list
    results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
  File "C:\Users\somebody\ComfyUI\ComfyUI\custom_nodes\was-node-suite-comfyui\WAS_Node_Suite.py", line 8299, in nsp_encode
    embeddings, pooled = clip.encode(new_text)
ValueError: not enough values to unpack (expected 2, got 1)

Worked ok before, also all fine swapping it out with default CLIP Text Encode.

Can be resolved by reverting changes in commit 7cc8c03.

Should be patched.

confirmed as fixed. cheers.


Download link for VPN client via MSIX is older than that available from the App Store.

[Enter feedback here]
Download link for VPN client via MSIX is older than that available from the App Store.

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

ID: e9ef898a-0b72-e018-8a3d-6f14f62e1119
Version Independent ID: b62b5b8f-bc88-0868-a2f6-1020a5097460
Content: Configure Azure VPN Client - Microsoft Entra authentication - Windows - Azure VPN Gateway
Content Source: articles/vpn-gateway/openvpn-azure-ad-client.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

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

@oobedan
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Yes, this sometimes happens because two different team push the download versions. Both versions should be at the latest version now. Also, we've had some new functionality added for P2S Entra authentication. The product team just released a new version of the Azure VPN Client. #please-close


feat: add compat with pgvector

Fix #227
Feat

Add new Gucs SET vectors.pgvector_compat=on to enable/disable statement rewrite
Function pgvector_compat_rewrite would be used to edit these elements

accessMethod -> from hnsw/ivfflat to vectors
opclass -> from vector_ip_ops/vector_cosine_ops to vector_dot_ops/vector_cos_ops
options -> from m: int/ef_construction: int/list: int to options: serialized toml

Test file compact.slt for pgvector-style SQL statements

Examples
These statments would be acceptable from now:
CREATE INDEX hnsw_l2_index ON t USING hnsw (val vector_l2_ops);
CREATE INDEX hnsw_ip_index ON t USING hnsw (val vector_ip_ops) WITH (ef_construction = 80);
CREATE INDEX hnsw_cosine_index ON t USING hnsw (val vector_cosine_ops) WITH (m = 12, ef_construction = 80);
CREATE INDEX ivf_l2_index ON t USING ivfflat (val vector_l2_ops) WITH (lists = 100);

Limitation
There is still a bit difference at Gucs, including:

hnsw.ef_search at pgvector vs vectors.hnsw_ef_search at pgvecto.rs
ivfflat.probes at pgvector vs vectors.ivf_nporbe at pgvecto.rs

Document
In progress...

test& add: schema check

Thanks @cutecutecat and @usamoi . Lots of reviews and commits to make this possible!


[FLINK-24509][docs] Add kafka serializer to producer examples

What is the purpose of the change
This pull request fixes documentation examples for the kafka producer by replacing the SimpleStringSchema with KafkaSerializationSchema
Brief change log

A new KafkaSerializationSchema is created with basic topic and record content filling
A newly created KafkaSerializationSchema is passed as the serialization schema to the FlinkKafkaProducer constructor

Verifying this change
This change does not affect any tests since it is an update to the documentation.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

LGTM. @alibahadirzeybek Can you rebase this on release-1.13 please?

Done


Devops Daycamp 2015 beginner track update

This is not quite finished but should point us in the right direction. Feedback is encouraged. If you want to make changes after discussing push to the branch as I can't guarantee I'll be able to respond and make changes for the next few weeks.
@dspt @jordane @lucywyman @iankronquist @Phillels @ramereth
The repository I am going to use for the git walk-through will be a simple flask app, probably this gist.
NOTE This has a lot of changes that are unrelated to the restructuring of the daycamp beginner track slides, those unrelated changes were because sphinx was complaining about the build process, those problems are now fixed thankfully.
I'm not sure how it was compiling before...

Fixed @lucywyman 's suggestions.  As far as the test app goes, what would be good changes to make that would teach version control and text editing (in vim?)?

I think one year we just had a student repository, where people would add a file with their name and some information about them.  We could do something similar, but maybe more fun (incentives?), and maybe less personal (not really sure what this would be either...)?   I think keeping the actual content of the activity simple is a good idea, since the point of the exercise isn't writing $language, which is why just plain text seems good, but there are certainly other possibilities.
@dspt Should that go in a separate PR?  Totally fine if it belongs here, I just thought this was mostly spelling/grammatic/style fix-ups.

+1 on this


chore: bdd-spring-1575991235 to 0.0.1

chore: Promote bdd-spring-1575991235 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Image editor doesn't open when clicking the palette icon in monaco

Describe the bug
The palette icon appears when creating a new sprite, but nothing happens when you click on it
To Reproduce

Open a new project in JS/Python
Drag out a sprite creation snippet
The palette icon will appear - if you click on it, nothing happens :(

Expected behavior
You should be able to edit the image.
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: Chrome 87

dup of https://github.com/microsoft/pxt-arcade/issues/2721?

dup of https://github.com/microsoft/pxt-arcade/issues/2721?

yup, a dupe! I missed it >>

yup, a dupe! I missed it >>


Have now rm check now.json for name

Currently if I do now rm -s -y, I get Error! now rm expects at least one argument.
I would love for that command to read the now.json file for the name property. This makes it easier to have a single source of truth for the app name when you have multiple now.jsons with different names.

Sounds like a very good idea!

Thank you for creating this issue!
Please note that – in order for us to be able to handle each feature request with care – we have to consume all of them through a unified pipeline that makes it easier for us to prioritize, track and progress on the features our user base is interested in.
In turn, the Issues tab on this repository is best only be used for reporting bugs, which we can immediately act on.
For well-explained feature requests (like the one you just posted), please contact us at<EMAIL_ADDRESS>so that we can consider adding it to our roadmap.


Teku:Nethermind on gnosis: Nethermind returned twice the block for the same slot

Description
Nethermind returned a block from the same slot twice, with a 3s delay. Teku clearly knows/shows in logs the block is empty.
We are not sure if this is an issue on Teku or Nethermind, we'd like to understand what happened and whether there is anything that can be done to improve it.
Desktop (please complete the following information):
Please provide the following information regarding your setup:

Debian 12
eth-docker
Nethermind/v1.28.0+9c4816c2/linux-x64/dotnet8.0.8
teku/v24.8.0/linux-x86_64/-eclipseadoptium-openjdk64bitservervm-java-21

Additional context
This is on the gnosis chain, where the time to get a block in is 1.66s max.
Logs
https://gnosischa.in/slot/17052573
consensus-1  | 2024-08-21 16:03:20.413 INFO  - Calling local execution layer to start block production (block slot: 17052573)
consensus-1  | 2024-08-21 16:03:22.000 INFO  - ESC[37mSlot Event  *** Slot: 17052572, Block: 16ae7cca982b24cb7276ef61ec80084b6597f20eba9df4251be0df4638ef75a3, Justified: 1065784, Finalized: 1065783, Peers: 110ESC[0m
consensus-1  | 2024-08-21 16:03:27.001 INFO  - ESC[37mSlot Event  *** Slot: 17052573, Block:                                                        ... empty, Justified: 1065784, Finalized: 1065783, Peers: 110ESC[0m
consensus-1  | 2024-08-21 16:03:32.000 INFO  - ESC[37mSlot Event  *** Slot: 17052574, Block: fbc06d932b621859b8e63a60d20729d7882b4da8f3c0c4fe3b2c9e660aad27d8, Justified: 1065784, Finalized: 1065783, Peers: 110ESC[0m
consensus-1  | 2024-08-21 16:03:37.001 INFO  - ESC[37mSlot Event  *** Slot: 17052575, Block: 9d452f290922d5e747fb3a3f5a16bc49629f12192997d1b249821ee2f039c3ee, Justified: 1065784, Finalized: 1065783, Peers: 110ESC[0m
consensus-1  | 2024-08-21 16:03:40.001 INFO  - ESC[32mEpoch Event *** Epoch: 1065786, Justified checkpoint: 1065785, Finalized checkpoint: 1065784, Finalized root: 9a5c72a4f4c33f66206af288b3e9c281b0d8fa23b790b81aeadc6325a6a61597ESC[0m
consensus-1  | 2024-08-21 16:03:40.090 INFO  - Attestation performance: epoch 1065784, expected 15364, produced 15327, included 15326 (100%), distance 1 / 1.00 / 4, correct target 15327 (100%), correct head 15297 (100%)
consensus-1  | 2024-08-21 16:03:40.090 INFO  - Block performance: epoch 1065785, expected 1, produced 1, included 1 (100%)
consensus-1  | 2024-08-21 16:03:40.090 INFO  - Sync committee performance: epoch 1065785, expected 448, produced 352, correct 352, included 334 (95%)

execution-1  | 21 Aug 16:03:20 | Produced block 35596759 (0xd8ced6...15e666), diff: 0, tx count: 1
execution-1  | 21 Aug 16:03:20 | Improved post-merge block 35596759 (0xd8ced6...15e666), diff: 0, tx count: 1
execution-1  | 21 Aug 16:03:23 | Produced block 35596759 (0x823107...0f014b), diff: 0, tx count: 7
execution-1  | 21 Aug 16:03:23 | Improved post-merge block 35596759 (0x823107...0f014b), diff: 0, tx count: 7

Hi, thanks for reaching us.
Checking the logs it seems a pretty standard behaviour on the EL side: when CL asks for a block production (execution payload) the EL immediately prepare a block and then starts improving it over time (Improved post-merge block ..) several times before CL asks for the final one when it's time to propose.
What happened instead is that it seems that the block production has been slow (or failed for some reason but there is no evidence in logs) and you missed it.
looking at the history of your validator it seems to perform well overall. Are you experiencing similar issues often?

Hi @tbenr, thanks for your feedback!
The validator performs well, but we did miss that block and we're not clear why. Should we probably enable DEBUG logs everywhere to get a better idea?
Another couple of questions would be:

Why does Teku think the block was included when it clearly wasn't?
Is Teku using the same timing in Gnosis as in Ethereum? Consider that gnosis timing is considerably shorter

Going to close this. Opened https://github.com/Consensys/teku/issues/8552

Hi @galaxy-mario would you be able to change this class to use debug log so if it happens again we can track the state and blocks you had as you canonical?
After starting teku you can run the following command on a terminal to change the log level for a specific class.
curl --location --request PUT 'http://<IP_ADDRESS>:5051/teku/v1/admin/log_level' \
--header 'Content-Type: application/json' \
--data '{
  "level": "DEBUG",
  "log_filter": [
    "tech.pegasys.teku.validator.coordinator.performance.DefaultPerformanceTracker"
  ]
}'

Note that <IP_ADDRESS> refers to the localhost ip and 5051 is the default port of the REST API. You might need to change those according to the way your clients is setup


Fix typo in title of welcome slideshow

Description
Thank you for all your hard work on the awesome devtools!
This is a minor edit to fix a typo in the title of the welcome slide show. This is missing an "a" before "little tour". Also, I would say it's more common (in American English, at least) to say "take a tour" vs. "have a tour", though they may both be correct.
Additional context
n/a

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ x Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).

Thanks!


Bug: pull-requests-history doesn't work

500 Internal Server Error
{'message': "Internal : Can't find a proper physical plan for this query"}

@liuyangc3, could you mind provide the link of the page you encounter this error?

@Mini256

link https://api.ossinsight.io/q/pull-requests-history?repoId=15452919
code https://github.com/pingcap/ossinsight/tree/main/api/queries/pull-requests-history