QueenParseTreeVisitor.java: Implement ConditionalTernaryE...

The puzzle 737136575 originating from #63 has to be resolved:
https://github.com/jvmqueen/queen-of-java/blob/07a7f854199c9ba95dbb9f74ad0cdd0ec7b2f041/src/main/java/org/queenlang/transpiler/QueenParseTreeVisitor.java#L2163-L2163
"Implement ConditionalTernaryExpression node for Queen.".
The puzzle was created by amihaiemil at 2023-01-31 22:14:08 +0200.
Estimation is 60 minutes.
If you have any technical questions, don't ask me, I won't be able to help. Open new issues instead.

@amihaiemil this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-02-10T20:20:41.049700.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@amihaiemil Don't forget to close this ticket before the deadline (2023-02-10T20:20:41). You are past the first half of the allowed period.

@amihaiemil Looks like you've missed the task deadline (2023-02-10T20:20:41). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@amihaiemil I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.

Puzzle disappeared from the code, that's why I closed this ticket.


No response when using the "Ask Open Canvas" feature for editing.

When using the "Ask Open Canvas" feature to request changes in the document editor, there is no response returned by the system. The issue persists even after attempting to make further changes via the chat interface.
Steps to Reproduce:

Open the document editing interface.
Request a change to a highlighted section using the "Ask Open Canvas" button.
Observe that no response is given by the system.
Attempt to make additional requests using the chat input field.
Notice that still no response is provided.

Expected Behavior:
The system should provide a response to the user's request, either allowing edits or providing guidance, similar to the cloud version.
Observed Behavior:

No response is given when using "Ask Open Canvas".
No response is provided even after repeated requests via the chat.

Environment:

Version: Local server (localhost:3000)
Document Title: "Whispers of the Sea"
Specific section attempted for edit: The final paragraph, highlighted in blue.

Screenshots:
(Attach the provided screenshot showing the issue)
Additional Information:
This issue does not occur in the cloud version, where the requested edits are processed successfully.
Severity:
High - Blocks editing functionality, disrupting user workflow.
Potential Cause:

The issue may be related to differences in the backend processing logic between the local and cloud environments.

Suggested Label:

Bug
Editing Functionality

This detail for errors:
langgraph-api-1       | FAILED TO GENERATE TOOL CALL {
langgraph-api-1       |   styleRules: [
langgraph-api-1       |     'Use clear and concise language to explain complex concepts, ensuring accessibility for a broad audience.',
langgraph-api-1       |     'Structure written content with clear headings and subheadings to enhance readability and organization.',
langgraph-api-1       |     'Incorporate historical context and practical applications to provide depth and relevance to the topic being discussed.'
langgraph-api-1       |   ],
langgraph-api-1       |   content: [
langgraph-api-1       |     'The user is interested in writing articles, particularly on technical subjects like Design of Experiments (DoE).',
langgraph-api-1       |     "The user values detailed and thorough explanations, as indicated by their request to 'chi tiết hoá' (detail) the article.",
langgraph-api-1       |     'The user is open to further adjustments and additional information regarding the content they are working on.'
langgraph-api-1       |   ]
langgraph-api-1       | }
langgraph-api-1       | Error: Reflection tool call failed.
langgraph-api-1       |     at RunnableLambda.reflect (/Volumes/DATA/open-canvas-main/src/agent/reflection/index.ts:80:11)
langgraph-api-1       |     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
langgraph-api-1       |     at async /Volumes/DATA/open-canvas-main/node_modules/@langchain/core/dist/runnables/base.cjs:1531:34 {
langgraph-api-1       |   pregelTaskId: '15c3f8ee-7a7e-5ec3-96c7-3dddfb405f73'
langgraph-api-1       | }

@sonnt-dna that's the reflection graph, which is not blocking the open canvas graph from executing. I also made a change sometime next week which should mitigate those errors in the reflection graph. I'm going to close this issue now, please ping me if you encounter another error.


Violation of PRIMARY KEY constraint on Drives data import

Discussed in https://github.com/trimble-oss/dba-dash/discussions/134

Originally posted by BrubiSQL April 12, 2022
Hello,
I'm getting weird error on one of my SQL server instances, to be precise on the instance where I have placed DBA dash.
I have made a separate VM for DBA Dash testing, and on that VM I have only C drive, and I have installed SQL Server 2019 developer on it.
I have connected one more SQL instance to be monitored by DBA Dash, and on that instance, everything is working just fine.
My current working theory is, that issue is SQL Server placement on C drive.
Thank you for any help,
B.

Error is:
Message
Microsoft.Data.SqlClient.SqlException (0x80131904): Violation of PRIMARY KEY constraint 'PK__#AE2608F__737584F723D0E7AC'. Cannot insert duplicate key in object 'dbo.@Drives'. The duplicate key value is (C:\).
The data for table-valued parameter "@Drives" doesn't conform to the table type of the parameter. SQL Server error is: 3602, state: 30
The statement has been terminated.
   at Microsoft.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at Microsoft.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString, Boolean isInternal, Boolean forDescribeParameterEncryption, Boolean shouldCacheForAlwaysEncrypted)
   at Microsoft.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean isAsync, Int32 timeout, Task& task, Boolean asyncWrite, Boolean inRetry, SqlDataReader ds, Boolean describeParameterEncryptionRequest)
   at Microsoft.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, TaskCompletionSource`1 completion, Int32 timeout, Task& task, Boolean& usedCache, Boolean asyncWrite, Boolean inRetry, String method)
   at Microsoft.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean& usedCache, Boolean asyncWrite, Boolean inRetry, String methodName)
   at Microsoft.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at DBADash.DBImporter.update(String tableName) in D:\a\dba-dash\dba-dash\DBADash\DBImporter.cs:line 377
   at DBADash.DBImporter.<>c__DisplayClass11_1.<Update>b__5(Context context) in D:\a\dba-dash\dba-dash\DBADash\DBImporter.cs:line 219
   at Polly.Policy.<>c__DisplayClass110_0.<Execute>b__0(Context ctx, CancellationToken _)
   at Polly.Policy.<>c__DisplayClass138_0.<Implementation>b__0(Context ctx, CancellationToken token)
   at Polly.Retry.RetryEngine.Implementation[TResult](Func`3 action, Context context, CancellationToken cancellationToken, ExceptionPredicates shouldRetryExceptionPredicates, ResultPredicates`1 shouldRetryResultPredicates, Action`4 onRetry, Int32 permittedRetryCount, IEnumerable`1 sleepDurationsEnumerable, Func`4 sleepDurationProvider)
ClientConnectionId:8ce1e050-afe1-4ee1-ac44-06a279b73519
Error Number:2627,State:2,Class:14

Message
Microsoft.Data.SqlClient.SqlException (0x80131904): Violation of PRIMARY KEY constraint 'PK__#AE2608F__737584F723D0E7AC'. Cannot insert duplicate key in object 'dbo.@Drives'. The duplicate key value is (C:\).
The data for table-valued parameter "@Drives" doesn't conform to the table type of the parameter. SQL Server error is: 3602, state: 30
The statement has been terminated.
   at Microsoft.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at Microsoft.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString, Boolean isInternal, Boolean forDescribeParameterEncryption, Boolean shouldCacheForAlwaysEncrypted)
   at Microsoft.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean isAsync, Int32 timeout, Task& task, Boolean asyncWrite, Boolean inRetry, SqlDataReader ds, Boolean describeParameterEncryptionRequest)
   at Microsoft.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, TaskCompletionSource`1 completion, Int32 timeout, Task& task, Boolean& usedCache, Boolean asyncWrite, Boolean inRetry, String method)
   at Microsoft.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean& usedCache, Boolean asyncWrite, Boolean inRetry, String methodName)
   at Microsoft.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at DBADash.DBImporter.update(String tableName) in D:\a\dba-dash\dba-dash\DBADash\DBImporter.cs:line 377
   at DBADash.DBImporter.<>c__DisplayClass11_1.<Update>b__5(Context context) in D:\a\dba-dash\dba-dash\DBADash\DBImporter.cs:line 219
   at Polly.Policy.<>c__DisplayClass110_0.<Execute>b__0(Context ctx, CancellationToken _)
   at Polly.Policy.<>c__DisplayClass138_0.<Implementation>b__0(Context ctx, CancellationToken token)
   at Polly.Retry.RetryEngine.Implementation[TResult](Func`3 action, Context context, CancellationToken cancellationToken, ExceptionPredicates shouldRetryExceptionPredicates, ResultPredicates`1 shouldRetryResultPredicates, Action`4 onRetry, Int32 permittedRetryCount, IEnumerable`1 sleepDurationsEnumerable, Func`4 sleepDurationProvider)
   at Polly.Retry.RetryPolicy.Implementation[TResult](Func`3 action, Context context, CancellationToken cancellationToken)
   at Polly.Policy.Implementation(Action`2 action, Context context, CancellationToken cancellationToken)
   at Polly.Policy.Execute(Action`2 action, Context context, CancellationToken cancellationToken)
   at DBADash.DBImporter.Update() in D:\a\dba-dash\dba-dash\DBADash\DBImporter.cs:line 218
ClientConnectionId:8ce1e050-afe1-4ee1-ac44-06a279b73519
Error Number:2627,State:2,Class:14
```</div>

As discussed, this error occurs because a DISTINCT is used which is over all columns.  If the free space or capacity (likely free space) we will end up with duplicate rows.
The simplest fix is to use GROUP BY and AVG the free space and capacity columns.  MIN could be used instead to show a worst-case scenario.  The problem would be if the drive also expanded - though it's unlikely the query would capture a drive expansion in flight. A better option to show the worst case would be to de-duplicate using a ROW_NUMBER function and sort by free space.
Comparison:
--original
SELECT DISTINCT dovs.volume_mount_point AS Name,
	dovs.total_bytes as Capacity,
	dovs.available_bytes as FreeSpace,
	dovs.logical_volume_name as Label
FROM sys.master_files mf
CROSS APPLY sys.dm_os_volume_stats(mf.database_id, mf.file_id) dovs;

--avg
SELECT dovs.volume_mount_point AS Name,
	AVG(dovs.total_bytes) as Capacity,
	AVG(dovs.available_bytes) as FreeSpace,
	dovs.logical_volume_name as Label
FROM sys.master_files mf
CROSS APPLY sys.dm_os_volume_stats(mf.database_id, mf.file_id) dovs
GROUP BY dovs.volume_mount_point,
		dovs.logical_volume_name;

-- row number de duplicate
WITH T AS (
	SELECT dovs.volume_mount_point AS Name,
		dovs.total_bytes as Capacity,
		dovs.available_bytes as FreeSpace,
		dovs.logical_volume_name as Label,
		ROW_NUMBER() OVER(PARTITION BY dovs.volume_mount_point ORDER BY dovs.available_bytes) rnum
	FROM sys.master_files mf
	CROSS APPLY sys.dm_os_volume_stats(mf.database_id, mf.file_id) dovs
)
SELECT Name,
	Capacity,
	FreeSpace,
	Label
FROM T
WHERE rnum=1

Performance is very similar between all 3 - this is a lab server with 1024 databases:

Interestingly the row number function has a higher estimated cost. With DISTINCT and with GROUP BY we are estimating to return far fewer rows.  The actual costs are about the same for all 3 - with the query taking around 2 seconds of CPU/duration.  This is on a lab server with 1024 databases.   On a server with 50 databases, the execution time is ~100ms.
With the default settings, this query is run once per hour so query performance isn't a concern.
The expectation is that there will be very small differences in free space in the duplicate rows.  The simplest solution using AVG should be sufficient.

Fix to be included in the next release

Fix is now in the latest release


autoPlay={false} doesn't work

Issue description

components: UncontrolledCarousel
reactstrap version #5.0.0-alpha.3
import method es
react version #16.0.0
bootstrap version #4.0.0-beta

What is happening?
If I set autoPlay={false} then:

Carousel still switch slides
I got a warning in the DevTools console:

proxyConsole.js:54 Warning: Failed prop type: Invalid prop `ride` of value `false` supplied to `Carousel`, expected one of ["carousel"].
    in Carousel (created by UncontrolledCarousel)
    in UncontrolledCarousel (at Slider.jsx:70)
    in div (at Slider.jsx:69)
    in Slider (at PageHome.jsx:35)
    in div (at PageHome.jsx:35)
    in div (at PageHome.jsx:34)
    in PageHome (created by Route)
    in Route (at App.jsx:17)
    in Switch (at App.jsx:16)
    in main (at App.jsx:15)
    in div (at App.jsx:11)
    in App (at index.js:10)
    in Router (created by BrowserRouter)
    in BrowserRouter (at index.js:10)

What should be happening?
Carousel should only display the first slide all the time, unless the user change slides manually.

const Example = () =>
set interval is "false"  in uncontrollable carousel.
Its Working..............................................................

const Example = () => ;

interval = {false}


Checking for existing uProxy cloud server

Refactored common code into getDroplet_()
Checking for existing uproxy cloud server in start() and returning unique error if server already exists
Using this.waitDigitalOceanActions_() in destroyServer() to determine when the DELETE action has been completed (so that we can then create a new server if necessary)

This change is

Nice! One comment.

Better I think! A couple of suggestions then we'll be good to go.

:+1: !


Prevent uncontrolled exec config growth

Fixes #14444
This includes two commits:

This removes the exec config from the container after the command exits
so that dead exec commands are not displayed in the container inspect.
The commands are still kept on the daemon so that when you inspect the
exec command, not the container, you are still able to get it's exit
status.

This also changes the ProcessConfig to a pointer.

This adds an event loop for running a GC cleanup for exec command
references that are on the daemon.  These cannot be cleaned up
immediately because processes may need to get the exit status of the
exec command but it should not grow out of bounds.  The loop is set to a
default 5 minute interval to perform cleanup.

It should be safe to perform this cleanup because unless the clients are
remembering the exec id of the process they launched they can query for
the status and see that it has exited.  If they don't save the exec id
they will have to do an inspect on the container for all exec instances
and anything that is not live inside that container will not be returned
in the container inspect.

@bprashanth and @thockin do you think this fix will address your specific issues around CPU and memory?
Thanks for checking it out.

To be clear, the GC will nuke old exec info after O(minutes)?  For example, we run a health-check every 10 seconds by exec.  That means the daemon will hold (60/10)*5 = 30 old execs and that's all?  That seems workable, I think.
Thanks for jumping on it.

Prashanth has better hands-on view than I do right now.

@thockin yes and the exec id list on the container inspect will be immediately removed whenever the exec process dies to cover that path as well.  Do you wont see large lists of exec ids even if they are all running.

LGTM

Thanks for the quick turn around on this!
Suggest clarifying the remote api docs around execids returned on inspect:  https://docs.docker.com/reference/api/docker_remote_api/

@duglin I changed it to a mark and sweep model for the final removal.  See the last commit for my changes.

ping @jfrazelle @LK4D4

LGTM


Doc improvements

As mentioned in a previous PR (and as @rmanoka mentioned wrt Value) our docs are extensive but unclear, because we're trying to address both high-level (with a growing list of convenience functions and API improvements), and lower-level (e.g. geojson as a computation pipeline component / FFI component) use-cases. We have lots of docs and examples, they're just intermingled in ways that make certain aspects of the crate difficult to discover.
I would dearly like to come up with a (hopefully simple) reorganisation strategy that splits these concerns and addresses them in a way that makes it more obvious where one has to "go" to figure out how to use the crate, depending on one's use case.
Apart from the motivation laid out above, my sense is that geojson is one of the more (if not the most) popular crates in the geo ecosystem, and public-facing improvements to it will hopefully benefit the project (such as it is) more generally.
What do people think?

Giving the docs some much needed attention would be great!

splits these concerns and addresses them in a way that makes it more obvious where one has to "go" to figure out how to use the crate

I'm not sure I entirely understand what the two separate use cases are. I do agree that the crate seems to be harder to pick up than I'd like (especially for those new to rust), and no doubt the documentation could be better organized.

Apart from the motivation laid out above, my sense is that geojson is one of the more (if not the most) popular crates in the geo ecosystem, and public-facing improvements to it will hopefully benefit the project (such as it is) more generally.

💯

Yes, I too agree that the docs could be a bit more streamlined.  For concreteness, here are the specific points I would like to see addressed:

Clarify integration with serde_json:  we only advertise the str::parse and ToString::to_string methods, but typically the {from,to}_writer and related methods are more useful and provide greater flexibility.  Also, the crate doc. suggests that serde is optional, but we rely quite heavily on it.  Is this an artifact from a previous version of the crate?

Modularize top-level crate doc:  we have covered most of the crate functionality, but it is all mentioned at the top-level.  This makes it difficult to quickly figure what to do for a specific use-case which is typically what users want out of the docs.  Would be better to be move things around so the top-level doc is concise, and points to the specific structs and their docs for the respective functionality.

I could work on a PR for this, but docs are also a matter of taste, so wanted to get some more opinions before refactoring things around.

AFAIK there was an effort to keep things non-serde-specific at one point, but we do already depend on it quite heavily, as you say, so maybe it's OK to have it as a dep? @frewsxcv ISTR this was discussed before (a long time ago, though…)

I'm very happy for examples / docs to be moved around and only mentioned at the top level where relevant – I agree that the docs are quite top-heavy.

Also, the crate doc. suggests that serde is optional, but we rely quite heavily on it. Is this an artifact from a previous version of the crate?

My read is that the docs are just out of date.
see here for a similar issue: https://github.com/georust/geojson/issues/78
Looks like it was originally dropped around 2017: https://github.com/georust/geojson/pull/58


first pass at my_resources - enough implemented to start CLI work

@theborch anything else for a rc4? if not let's merge this then update changelog/version in the 2.25.0rc4 branch and push? I want to get this available so we can reference it via the CLI.


SqlConnection pooling is sometimes preventing test databases from being deleted.

Description
Clears the sql connection pool before attempting to delete the integration test database.
Related issues
I've been noticing a bit of a build up...

Testing
Integration tests can exit immediately instead of waiting for exclusive database access that they don't get
FHIR Team Checklist

[ ] Update the title of the PR to be succinct and less than 50 characters
[ ] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[ ] Tag the PR with the type of update: Bug, Dependencies, Enhancement, or New-Feature
[ ] Tag the PR with Azure API for FHIR if this will release to the Azure API for FHIR managed service (CosmosDB or common code related to service)
[ ] Tag the PR with Azure Healthcare APIs if this will release to the Azure Healthcare APIs managed service (Sql server or common code related to service)
[ ] CI is green before merge
Review squash-merge requirements

Semver Change (docs)
Patch|Skip|Feature|Breaking (reason)

            command.CommandText = $"DROP DATABASE IF EXISTS {databaseName}";

FYI
I had it fixed by changing this line to
IF db_id('{databaseName}') IS NOT NULL ALTER DATABASE {databaseName} SET READ_COMMITTED_SNAPSHOT ON WITH ROLLBACK IMMEDIATE
DROP DATABASE IF EXISTS {databaseName}

Refers to: test/Microsoft.Health.Fhir.Shared.Tests.Integration/Persistence/SqlServerFhirStorageTestHelper.cs:123 in 66b641e. [](commit_id = 66b641e443de17ec27a6ae829c662680c51e516e, deletion_comment = False)


Update design details of Ordinals inscription send flow

To be consistent with BTC experience UI needs to be adapted to the fullpage when on a popover

Designs: https://www.figma.com/file/OSQocEEJnfcMkniEl8u3U8/%231483---Initial-Bitcoin-support?node-id=1540%3A73103&t=FNRziBxZd0VBTFeU-4

@mica000 Are the fixes in this issue still relevant?

@markmhendrickson yes, will be solved in https://github.com/leather-wallet/extension/issues/4371

It seems there are more fixes in this particular issue than #4371 per se – in particular the copy changes from your earlier comments?

I'm noticing some differences between the Figma and implementation during usage day:

Top box has black background in prod vs. gray in Figma
The "Ordinals inscription" unit below is joined with the address field visually in Figma vs. not in prod
The address field doesn't have a "To" label in production as seen in Figma
We say "Inscription 2652322" up top in prod but it's "#2652322" in Figma
We have both back and close buttons in the modal nav in prod but only back in Figma
There's a "View details" link in Figma but not in prod

@mica000 are these all worth tackling still?

@mica000 assigning you since these details will be resolved with standardization work in progress


Espresso test suite for this application will help us learn test automation on firebase apps

From @sillysachin on January 9, 2017 4:42
Currently I am struggling with automating test for a firebase app . It would be of tremendous help to have a integrated sample of test suite with both debug and release modes etc.
Copied from original issue: firebase/friendlypix#72

From @yofu1234 on March 5, 2017 7:57
seconded!

Cleaning up issue fixed in upcoming v2.


To be Deleted

Sorry

If this happens again you can close the issue by yourself, don't worry about it.


Fix AlterAdd Test Failure

Codecov Report
Patch and project coverage have no change.

Comparison is base (0b47bdd) 93.29% compared to head (7653706) 93.29%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   93.29%   93.29%
=======================================
  Files          77       77
  Lines        4576     4576
  Branches      423      423
=======================================
  Hits         4269     4269
  Misses        185      185
  Partials      122      122

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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Fabric 1.20.1] All fish except fish bones have same look (and chicken?)

Big fan of the concept, but for some reason the items look like this while in inventory. (They look fine in-hand)
This even after generating packs.
Please help?

Hello,
That's weird.
Are you playing on a server or on singleplayer? If on a server, did you make sure to apply the resource pack after generating it? Also, what minecraft version are you on?
I'll look into that. Thanks for letting me know.

Big fan of the concept, but for some reason the items look like this while in inventory. (They look fine in-hand)
This even after generating packs.
Please help?

Moreover, what is the other fishing mod on the screenshot? Makbet its compatibility issue

Hi, sorry! I'm on Fabric, 1.20.1, singleplayer. The other fishishing related things in the screenshot, the first row up to the fish bones are all vanilla items and fro the burger onwards, which is from better mcdonalds foods, the rest are simply decorative items from chipped and handcrafted.
Hope this helps!

There seems to be an issue with multiple resource packs interactions. Your other resource pack is overriding tropical fish's texture (and chicken). I'll check whether it's something on my end.
In the meantime, try putting Polymer Resources to the top of resource packs list. That shuld fix it.

Hi again, thanks for the tip! I've tried it again and, nope, nothing. :(

Ok, I'll try to fix it in a few days

You can try one more thing - download the latest, updated version of the mod from Modrinth. It should work, but at this point I'm not entirely sure. Thanks for patience - in advance.

Hello again! Thank you for your time on this!
I've downloaded the newest version, generated packs again, moved pack in the resource list making sure it stays at the top, and still nothing, unfortunately.
Not going to lie, this is a bit convoluted, I don't understand why there's a need for this extra step when other mods load their textures without issues, is there any particular reason as to why Fishing 101 does it this way?
I have tried making a new blank instance with just Fishing 101 and its dependency, generate pack, and it does work, so I can confirm it's not an underlying issue with the mod itself rather some compatibility issue as first thought.
The only issue now is narrowing it down to the other mod or resource pack that causes this, since my minecraft is HEAVILY loaded lol.
I'm guessing there's really no easy fix to this?

I did some educated guessing and - that was actually easy enough to figure out!
So the culprit is apparently the mod "Eating Animations" which reverts your foods to the vanilla look when in inventory.
I'll remove Eating Animations from my list of mod because I'd enjoy your fishing and foods variety, but I guess that's something worth knowing, since the Eating Animations mod is pretty popular, fixing the compatibility issue would be a good thing!
Most people would just assume fishing 101 doesn't work and not bother - But they'd be missing out!
Anyway thanks for your time and patience! Keep rocking!

Hi again! Glad you were able to figure it out. I'll make sure to check for compatibility errors with the Eating Animations mod.
Fishing101 behaves differently to other mods. It relies on a special library that allows the mod to work on multiplayer servers without the need for players to download it (hence the resourcepack).
Thanks for the kind words! I'm glad you're enjoing my mod!
Have a good time!


upower: upstream 0.99.5 version has fixed this issue.

The upstream latest 0.99.5 version has included this fix. So need to
remove this bbappend to avoid the mismatch recipes version issue.
meta-overc build with the latest meta-openembedded layer will report
the following WARNING:
WARNING: No recipes available for:
meta-overc/meta-cube/recipes-support/upower/upower_0.99.4.bbappend
Signed-off-by: Guojian Zhou<EMAIL_ADDRESS>

merged


[BUG] Upload file

Hi,
Trying to figure out how to upload a file from Kaffy, could you give some insight what should be the best way to achieve that!
I tried with before_create callback but it's not working for me.
Here how my upload work from the frontend:
https://github.com/areski/ex-chitchat/blob/master/lib/chit_chat_web/controllers/upload_controller.ex
Here the schema:
https://github.com/areski/ex-chitchat/blob/master/lib/chit_chat/documents/upload.ex
and Kaffy upload admin:
https://github.com/areski/ex-chitchat/blob/master/lib/chit_chat_web/admin/upload_admin.ex
I have the feeling that somehow I should cancel the insert from Kaffy and take the hand but not sure how I should do that?
Have you tried uploading files with Kaffy?

I think since Kaffy is handling the actual database operation, you might need to refactor a few functions you have.
You could use before_create but you have to return a changeset and not the {:ok, upload} value returned from Repo.insert.
In your Documents.create_upload_from_plug_upload(upload) function, if you could make this return a changeset and have a different function save that changeset for you in your create action, it might work.
Example:
# you would have 2 functions instead of the create_upload_from_plug_upload one
def prepare_upload(%Plug.Upload{filename: fn, path: p}) do
  # do some processing
  changeset = Upload.changeset(some_params)
  {:ok, changeset}
end

# you would use this in your own controller, Kaffy doesn't need this.
def save_upload(changeset), do: Repo.insert(changeset)

# Kaffy callbacks
def before_create(conn, changeset) do
  # do some processing with conn, get the upload
  upload = conn.params["upload"]
  prepare_upload(upload)
end

Let me know if this still doesn't work for you.

I think this is done with your PR 🙏🏻


Bower version

Hi,
The bower support, as requested in https://github.com/peterolson/BigInteger.js/issues/78, works!
Unfortunately I am not able to specify a version number. This means I cannot use a "release" of big-integer but either have to build with the latest commit, which might mean my builds might unexpectedly break in the future, or depend on a git commit hash, which makes it difficult and error-prone to depend on the correct version.
While reading the bower docs (I'm not an expert on creating packages) it seems that all that is required is pushing semver git tags. Would it be possible to add a 1.6.17 git tag to the latest release and use git tags to tag future versions?
I would love to send you a pull request but I don't think one can make PRs for git tags. Please let me know if there is anything I can do to make this happen.
Thanks for all the great work on this library! ❤
Regards,
Dennis

Thanks for the suggestion. I had actually been meaning to add git tags for a while, this was a good reminder.
I just added tags for all version from 1.6.0 and up. Let me know if that resolves your issue.

Works like a charm, thanks @peterolson!
$ bower install<EMAIL_ADDRESS>bower not-cached    https://github.com/peterolson/BigInteger.js.git#1.6.17
bower resolve       https://github.com/peterolson/BigInteger.js.git#1.6.17
bower download      https://github.com/peterolson/BigInteger.js/archive/v1.6.17.tar.gz
bower extract       big-integer#1.6.17 archive.tar.gz
bower resolved      https://github.com/peterolson/BigInteger.js.git#1.6.17
bower install       big-integer#1.6.17

big-integer#1.6.17 bower_components/big-integer


Bug: Release process fails if PR approvals are required

Describe the bug
Currently, to use this action you must disable required approvals for pull requests. If pull requests are required, the following error occurs
remote: error: GH006: Protected branch update failed for refs/heads/beta.
remote: error: Changes must be made through a pull request.

You can allow certain users/apps to bypass this the repository settings, but that has no effect here, even with adding the OpenSauced GitHub app.
A workaround is to install a GitHub app and use an app token
https://github.com/semantic-release/github/issues/175#issuecomment-1489689561
Steps to reproduce

Install and setup the release action - https://github.com/open-sauced/api/blob/beta/.github/workflows/release.yml#L132
Enabled required approvals on the repository - 2
Merge a pull request
Note that the semantic release job fails with the error

Browsers
No response
Additional context (Is this in dev or production?)
No response
Code of Conduct

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

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

@brandonroberts I just made you a GitHub App admin for the org if you want to test a fix on a benign repo.

Adding a note here that GitHub Packages does not currently support deployments using tokens from GitHub Apps, so including a Dockerfile at the root of the project will fail to deploy.
The workaround is to rename it to something.Dockerfile and point to that Dockerfile as the input for the container to be built. This skips the deployment to the GitHub Registry, and you would have to set up a Personal Access token to deploy to ghcr.io.


Panics when applying 1.10.0 to contracts-parachain

When applying #21 to contracts-parachain some unexpected panics appear in the logs. The behavior is weird, as the runtime produces blocks regardless of these panics.
Needs research to find the cause of such logs.
More context on this issue was posted in StackExchange: https://substrate.stackexchange.com/questions/11527/unexpected-panics-after-upgrading-template-to-v1-10-0
Blocks: https://github.com/r0gue-io/contracts-parachain/pull/5

Solved by #25


Github MacOS Release blocked by Gatekeeper on Catalina

Hello,
I finally upgraded to Mac OS Catalina, and due to the changes in Gatekeeper the version of ninja on the Github releases page is now blocked from running (v1.9.0). There doesn't appear to be a way to verify command line apps individually on Catalina, you can only disable it entirely (I may have misunderstood the instructions on the web-page I found).
I installed via brew instead, and that version runs fine. I am mostly reporting this issue for anyone else who has a problem with this, as I couldn't find mention of it already.

This came up on another project -- it seems like the Gatekeeper check for downloaded binaries may involve some form of flag that is set when they get downloaded by a browser. Apparently using curl to download the exact same file does not get blocked by Gatekeeper.

Would codesigning the binary be enough to fix this? Or do I also need to notarize it? Can you even notarize raw binaries on macOS?

This came up on another project -- it seems like the Gatekeeper check for downloaded binaries may involve some form of flag that is set when they get downloaded by a browser. Apparently using curl to download the exact same file does not get blocked by Gatekeeper.

xattr -c /path/to/ninja has always worked for me.

@jkuebart is right, this is doe to the quarantine bit. Browsers set that when downloading files:
% ls -l@ ~/Downloads/ninja
-rwxr-xr-x@ 1 thakis  staff  628848 Nov 28 12:04 /Users/thakis/Downloads/ninja
	com.apple.macl	    72
	com.apple.quarantine	    57

If you remove that with xattr -d com.apple.quarantine ~/Downloads/ninja (or -c, which deletes all attributes), then the ninja binary from the Releases section runs just fine on an M1 mac:
%  ~/Downloads/ninja --version
1.10.2

@jkuebart is right, this is doe to the quarantine bit. Browsers set that when downloading files:
% ls -l@ ~/Downloads/ninja
-rwxr-xr-x@ 1 thakis  staff  628848 Nov 28 12:04 /Users/thakis/Downloads/ninja
	com.apple.macl	    72
	com.apple.quarantine	    57

If you remove that with xattr -d com.apple.quarantine ~/Downloads/ninja (or -c, which deletes all attributes), then the ninja binary from the Releases section runs just fine on an M1 mac:
%  ~/Downloads/ninja --version
1.10.2

Is this still a problem with Nina 1.11?


[BUG] Accept appointment failed if entering a room or recource

Hello,
I have the following issue with DOMI.
HCL Product Version
HCL Notes 12 with German Notes 12 Mailtemplate
Describe the bug
If I create a meeting with DOMI and append a room or recourse the meeting will create successfully. But after sending the appointment I cannot change it and every customer using also a Notes Client will get the following message if they want to accept the meeting:
"Bitte wählen Sie einen Online-Besprechungsbereich für diese Online Besprechung"
To Reproduce
Steps to reproduce the behavior:

Create an appointment
Create DOMI Meeting (e.g. Teams, GoTo Meeting)
Select an internal room or recource
Send the appointment
try to change it or try to accept the meeting with a Notes Client

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
The error message:

The problem might be in this LS part:

Desktop (please complete the following information):

OS: Windows 10

Kind regards
Karsten Poschmann

Hi @KarstenPoschmann:   This seems similar to issue #104, which has been resolved.  Are you sure you are running the latest release code?

This may be fixed in 12.0.1 Beta 1.  I recall working on it but I forget if it got fixed for Beta 1 or after.

Thank you, I have implement the new code and now it works


ttl oddities

https://raw.githubusercontent.com/geneontology/noctua-models/master/models/591f58ab00000003.ttl

The triple <http://model.geneontology.org/591f58ab00000003> a <http://www.w3.org/2002/07/owl#Ontology> is there twice
Classes are injected into the abox
oio:id injected everywhere

Hypothesis: some kind of invalid triple was added onto http://model.geneontology.org/591f58ab00000003, causing the owlapi to inadvertently pun. The imports is on the punned IRI, which renders it invisible to the owlapi. this in turn lead to existing class declarations leaking in.... I dunno.
@balhoff any ideas?
This model was created 2 days ago.
We should at least test adding some new models, make sure nothing odd has happened

my hypothesis is not quite correct, the OWLAPI is reading the imports.. but it doesn't rule out that it was in an odd state pre-save.

Hmm, direct output from OWLAPI via minerva:
http://noctua.berkeleybop.org/download/gomodel:591f58ab00000003/owl

The injected oio:id is present on the Ontology nowhere else
We have class declarations but the label tbox axioms are gone (good)
there is only one owl:Ontology triple, as expected

I can comment on one part so far: the <...> a owl:Ontology triple is seen twice because I had put in a hack to workaround the RDF parsing bug in OWL API. I manually injected the owl:Ontology declaration ahead of any other triples so that ontology annotations would be correctly parsed. This can probably be removed now.

I don't think it's just this model. I looked at a few others and see oio:id and rdfs:label on gene product classes, but not on GO terms.

Current guess: this stems from adding classes to the model that aren't already in the tbox. This will happen for every gene product that has the IRI mismatch described here. In this model you can see the oboInOwl:id isn't getting stored for the PRO IRIs, which have OBO PURLs that match what's in NEO.

But why is the OWL API direct ttl different?


Setup monitoring and reporting for clusters in PSI

We need monitor PSI clusters so we can catch issues early

/lifecycle frozen

We are not going to use PSI. SO closing it
/close


Selected option disappear

Microsoft PowerToys version
0.45.0
Running as admin

[ ] Yes

Area(s) with issue?
General
Steps to reproduce
Hey guys,
any selected option just disappear.  see the screenshot here for an example:

I have a 4k monitor, not sure if this might be the problem with resolution etc.
✔ Expected Behavior
To see the option! :-)
❌ Actual Behavior
no option highlighted
Other Software
Windows 10, 4k resolution monitor

@neuropass If you change the display resolution to 1920x1080 or something else, can you see the options?

I just tried, nope same thing they just disappear when I click on them.

cc: @niels9001

Hmm strange, can't replicate it:

What's the Windows version you are on?

@neuropass
What color settings have you set in windows and PT settings?

@neuropass

What's the Windows version you are on?

Running the command winver can do this.

Hey guys,
some more info:
https://gyazo.com/3e04696249fe1f3d7c3fe73786a0ff7e.png
and:
https://gyazo.com/589b36ad7ea8bd9438417330347be901.png
and:
https://gyazo.com/5749a89e8b57b8f480e0582ff595aa25.png
Microsoft Windows 10 - 21h1 OS build: [Version 10.0.19043.1202]

This option is changed to ComboBox so this doesn't apply anymore


Each time SMS Backup+ is updated, causing Textra to generate MMS error

Each time SMS Backup+ is updated automatically and there's a "what's new" popup in SMS Backup+.  This popup seems to cause Textra to generate an error about MMS.  I have to go into SMS Backup+ and close the "What's new" popup, and then Textra will work.  Textra states that this is not something they can fix, but rather something wrong with another application (in this case SMS Backup+).   Seems like that popup is causing SMS Backup+ to put a lock on something that is needed for texting.
I'm using SMS Backup+ 1.5.10 and Android 5.1.1 (Nexus 6).

Sorry, the popup is just informational, don't see how this would interfere with Textra.


Remove the MIT license from tools jar[skip ci]

we have moved tools to spark-rapids-tools/core
These MIT licenses are only for the UI of tools jar
Contribute: https://github.com/NVIDIA/spark-rapids-tools/issues/46
Signed-off-by: gashen<EMAIL_ADDRESS>

build

There's a number of things in NOTICE and NOTICE-binary that also need to be removed (Bootstrap, DataTablesSrc, jQuery, Mustache, and Spur).

+1
Since we do not include those binaries into the JAR, then NOTICE and NOTICE-binary also need to be updated.

Updated NOTICE and NOTICE-binary

build

build

build

build


Question: How to filter the results by peptide FDR?

Hi and thanks ahead for reply:
I wonder if I can filter the results by peptide-level FDR, since there is only a 'Qvalue' column in the result 'oglyco.psmtsv' and 'single.psmtsv'. And I can't find the relative explanations (for peptide FDR) in the wiki... Thanks again for response!

Not sure what you want. FDR is a group parameter (not for individual assignments). i think you may want the q-value, which is simply the FDR of all peptides beginning with the highest scoring peptide to the peptide in question. Every peptide with q-value < 0.01 has 1% chance of being incorrect (1% FDR). Let us know if you have more questions.

Thanks for your explanation!
What I want to express is the individual peptide q-values. So is there any information about the q-values of peptide identifications given in the result? Or how should I do to keep the peptide-level FDR of the result lower than 1%?

It looks like the 'Qvalue' column in the 'oglyco.psmtsv' represents the q-values of PSMs, so I wonder if there are any information about the q-values of peptide identifications?

Hi,  the FDR is only controlled in PSM level in the current release version. The way the peptide FDR is calculated under development is to obtain the best score PSM for each peptide and order all peptides from high to low to obtain peptide FDR.
Thanks for asking and this is an important feature to include. Please let us know if you have any suggestions.

Thanks for all these answers. Looking forward to seeing the new features and thanks again!

check boxes added in next release


fix: upgrade deps

and support the new cluster.chartOCI flag to detect OCI helm releases
also handle bad : characters in resource names

/approve

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
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 [jstrachan]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
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 [jstrachan]

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


WIP: Adopt Kubernetes Community Membership governance

Add member/reviewer/approver/owner process by lightly modifying the
Kubernetes Community Membership process.
Attempt to make as few changes to the K8s version as possible, to ease
cross-contribution.
Changes include:

Removing references to SIGs, as Submariner only has subprojects.
Update names, contact information and links for Submariner
parallels where possible.
Use explicit, vs relative, links where necessary.

Closes: #70
Signed-off-by: Daniel Farrell<EMAIL_ADDRESS>

I guess we can now close this one @dfarrell07 ? :)

I guess we can now close this one @dfarrell07 ? :)

Yep, thanks for the merge!


[buildmgr] Add C pre-processor secure flags

Fixes https://github.com/Open-CMSIS-Pack/devtools/issues/1056


[Build] Cache test requirements locally

Running ctest on an existing and previously built and run codebase
checkout fails if there is no internet connection because pip will
attempt to look online for the packages. The pip documentation
recommends caching the archives locally and then referring to them in
the install step:
https://pip.pypa.io/en/stable/user_guide/#installing-from-local-packages
For unknown reasons setuptools and wheel are not archived
automatically, so I had to manually store them in an additional step.
The archives are stored in BUILD/downloads.
Closes #300
Signed-off-by: Raihan Kibria<EMAIL_ADDRESS>

For info, good explanation here: https://stackoverflow.com/a/67595330/535103
TL;DR it's because of pip's relatively new "build isolation" default, where build dependencies (i.e. setuptools and wheel) are installed in some temporary build directory.
This can be disabled (--no-build-isolation) but doing that has it's own problems (e.g. requires wheel to be separately installed in the virtualenv, adding unnecessary pollution).
I think on balance the approach taken in this PR seems like the best option. Though I'm happy if you want to experiment with that flag in case it simplifies things @fnRaihanKibria

@feltech Squashed and ready to merge.


Enable Mutating and Validating webhooks by default

Either, We can get rid of these enablewebhook flags, Or, default value should be true.
Conversation: https://github.com/kubedb/cli/pull/304#issuecomment-425520417
bash script: https://github.com/kubedb/cli/blob/7d7e249253a778c0dbb2e1d44881ab789ba953a2/hack/deploy/kubedb.sh#L116-L117
Chart: https://github.com/kubedb/cli/blob/7d7e249253a778c0dbb2e1d44881ab789ba953a2/chart/kubedb/values.yaml#L69-L71

These are already enabled by default in bash script.

https://github.com/kubedb/cli/pull/313 and https://github.com/kubedb/operator/pull/197 fix remove the need to enable* and apiserver.ca variables in chart (hopefully for ever).


Make fedpkg clone work in openshift

Fixes https://github.com/packit-service/packit/issues/431
Edit: description on Rado's request

passwd: fedpkg needs that otherwise it complains that current user doesn't have an entry in /etc/passwd - SCL uses this method in their images
.ssh - ssh command is very picky about perms of .ssh and id_rsa; stolen from Betka
I also added a new command which accepts events via CLI and processes them directly for easier local development
and a gitconfig which makes a nice commit author instead of (not using git config b/c we change users and it likely ends up in /opt and not in /home/packit): https://src.fedoraproject.org/rpms/python-ogr/c/07797442e8b27e6ea785b39b4b6d5cfc5f6ef0a1?branch=master

Rerady for review, I added a description.


Update Composer dependencies (2023-10-16-00-14)

Visual regression test passed!
View the visual regression test report


Add Markov chain-based gibberish detection

Most of the code has been sto- I mean, adapted from https://github.com/amitness/Gibberish-Detector
Read more about Markov chains: https://en.wikipedia.org/wiki/Markov_chain
gadjubas pidor

e7bab603c6ee223fd157e62d8855e398071d3bbe 🤗


docker-compose: starting Neo4j as non-root user

Neo4j documentation describes how to run a docker container with neo4j as non-root, but it does not state how to do this with docker-compose.  How can this be done?

You can check the docker compose documentation to find out how to convert docker run commands into compose files, here:
https://docs.docker.com/compose/compose-file/#domainname-hostname-ipc-mac_address-privileged-read_only-shm_size-stdin_open-tty-user-working_dir
For the --user flag you should add a user: <VALUE> parameter under your neo4j service.
Here's an example of a neo4j compose file used for some tests (it has some variables that need replacing so won't work without modification, but you can see how it specifies a non-root user):
https://github.com/neo4j/docker-neo4j/blob/master/src/test/resources/causal-cluster-compose.yml


Python functions not being detected

VSCode Version: 1.46.0
OS Version: Windows 10

Steps to Reproduce:

def c_to_f(a):
""" Converts temperature in Celcius to Faranhite """
answer = (a * 9/5) + 32
return answer

c_to_f(40)

c_to_f(40)
Traceback (most recent call last):
File "", line 1, in
NameError: name 'c_to_f' is not defined

Does this issue occur when all extensions are disabled?: Yes

/extPython


No file in data/glove?

Hi!
I am training the Sequicity model (one of the e2e models).

I download the given data.

But when I run:  python model.py -mode train -model tsdf-multiwoz
There is a FileNotFoundError: [Errno 2] No such file or directory: '/convlab/ConvLab/convlab/modules/e2e/multiwoz/Sequicity/data/glove/glove.6B.50d.txt'
So, should there be a txt file in the folder?
Thank you!

Thanks for pointing out. There should be such a file in data folder.
Please download the embedding file at https://nlp.stanford.edu/projects/glove/, and then move it to /convlab/ConvLab/convlab/modules/e2e/multiwoz/Sequicity/data/glove


Playlist don't show immediately after it is created. Until I exit the app and launch again.

Describe the bug
After I created a playlist, I accessed the playlist tab and I didn't see any of the created playlists there.  They would only show when I exit the app and re-launch
To Reproduce

Go to the "songs" page
Highlight songs you wish to add to a playlist
Create playlist
Access the playlist page
See error. (It shows empty playlist)

Expected behavior
I expected that after I had created a playlist, I would see the playlist on the designated page and be able to use it.
Screen recording
https://youtu.be/b3vBhc--WXc
Test Environment

Device: Redmi Note 5 Pro
OS: Android 8.1
Software version: Bop music player v1.1.2

Hey @HarryHeights  , will work on it asap.

Okay. That's great. I'm still working on the FAQs. Would be done soon.

@HarryHeights ,I have added swipe to refresh layout to refresh the playlists list. Will soon update in releases

Oh okay. That's great


Add support for alternate pronunciations from cmu_dict & custom_dict

So after some refactoring in PR #68, the only thing I pull from CMU dict are the phones.
Currently, when I do, I use the first item in the list and ignore any others, and in the custom dict I only store one.
So to support multiple pronunciations, I would need to:

[ ] Make the 'phones' key in custom dict point to a list containing the current string.

[ ] Add a new property which cannot exceed the length of the _phones list, and represents the index into that list.

[ ] Update all the consumers of _phones to use the index, not zero.

For example, the first result in CMU_dict for "and" or "in" has a stress of "0", rather than "1", which is the second result.
Makes a rather big difference which one you pick!

Once this is in place, this is a good candidate for the first thing a user could change on the page.
Alternatively, could be the basis for some automated logic to pick which pronunciation to match the expected meter or rhyme?


Binding services to the module context breaks static properties on methods.

Hi there,
This was working in 10.4, but in 11, as of 97861aa30da56eef2907516010ac31157e8c3abe there is a use case that is broken.
Let's say we have EmailService.js
var EmailService = function(){
// Do some stuff
};
EmailService.foo = 'Some Value';
module.exports = EmailService;

When the module loader is calling _.bindAll, it's replacing all methods on the object with a "bound" version, but it completely wipes out any existing properties on that function.
It could be all well and good to say "Don't export functions", but 1. it does it to every function on the object, not just the exported one (so if EmailService.foo were a function, and had properties, those would be wiped out as well), and 2. we may be exporting a third party library, where we don't control the kind of object that's being exported.
Since _.bindAll is destructive, I would suggest making this something the service can opt into or control themselves.

When the module loader is calling _.bindAll, it's replacing all methods on the object with a "bound" version, but it completely wipes out any existing properties on that function.

That would be terrible!  Fortunately the exact code you posted works fine for me on a fresh install of v0.11 (tried EmailService.foo in sails console, as well as console.log(EmailService.foo) inside of the function), so I suspect the issue is elsewhere.

Sorry, for some reason I thought I had tested the example thoroughly, but testing it, you are correct. Where it fails is the following example:
var EmailService = function(){
// Do some stuff
};
EmailService.SomeFn = function() {
	// Do some other stuff
};
EmailService.SomeFn.foo = 'Some Value';
module.exports = EmailService;

So basically, if the property on the function is also a function that contains properties, those props are lost.
Are you able to reproduce the same issue?
Thanks again for looking at it :)

Ok, that I can reproduce, and it makes sense.  I suppose this is more an issue with Lodash's _.bindAll function than with Sails core, but that's neither here nor there.  Can you work around it?  Is it imperative that your service functions have their own properties?  Or could you just store those properties in a hash at the top level of the service, or better yet in a private var that doesn't get exported at all?

Thanks for the response, but I don't know if I can work around it for all of my scenarios, as it's a third party lib that's exposing utility methods on a function that also does some work on it's own.
For the immediate case, I'm pretty sure I can work around this one method, but not the others, and I have a feeling I'll hit it again.
That being said, please feel free to close this out if you feel so inclined, though it may be useful to keep it open so I don't reopen another one when I run into an issue that is a permanent blocker :)
Thanks again for looking at this!

Actually, it seems to me that we don't need to be running _.bindAll on modules that export functions.  The intention is to make sure that any methods exposed by the service (or controller, model, etc.) have access to the sails instance using this.sails.  If you're exporting a function, we can just use regular _.bind on it, which should leave its properties alone.  If any of those properties happen to be functions themselves, they won't have this.sails, but that's okay because 1) you can add it yourself, and 2) in a lot of cases (yours included) they'll be from a 3rd party that doesn't need a reference to Sails anyway.  Sound right?

I think it sounds right, except for _.bind portion leaving alone properties, unless there is something else going on that I'm not aware of.
_.bind should return an entirely different function instance, and as far as my understanding of underscore and lodash go, they don't copy properties of the function over, for instance here:
var foo = {
  bar: function(){console.log('foo.bar', this, arguments);}
};
foo.bar.baz = 1;
foo.bar = _.bind(foo.bar, g);
foo.bar.baz; // is undefined

But it's possible I don't know the whole picture, since AFAIK, calling _.bindAll should technically do the same, but it seems it's not from what I can tell.
Out of curiosity, do you know why it's ignoring leaving properties alone in these cases?
Thanks again for your responses :)

Hi there,
I have same problem but I think i'll end it up by appending all to an extra object (mongoose in my case):
module.exports = {
  mongoose: {
    User: require('../models/User.js')(mongoose),
  }
};

Any info on why this was closed? It doesn't appear to be actually fixed.

@natecavanaugh sorry about that!  I just stumbled across this issue- sorry for the slow reply.
@natecavanaugh I appreciate you taking the time to get to the bottom of what's breaking.  Could you provide more details about your use case?  In order to make a change like this, it's helpful to know more about how you're using your services (see    https://github.com/balderdashy/sails/blob/master/CONTRIBUTING.md#v-proposing-features-and-enhancements for more background)
@lmacsen would you mind posting more about your use case?  Also, re mongoose:  you might want to check this out

@natecavanaugh By the way, to give my question some more context-- my hope is that this is something we can address via a configuration option (philosophically, you should be able to tell Sails to leave your services alone, but binding is also useful in the general case).  What I'm looking for is stuff like "Where are you calling your service from?" , "What is the function that's getting its winged clipped supposed to do?", and "Why would a workaround not be appropriate in your case?" Thanks again!

Not sure why this got closed either.  The reasoning behind binding module methods to the module is to make sure they have access to the Sails instance they're running in, even if globals are turned off.  This allows Sails apps to be included as dependencies (and just allows you to turn sails.config.globals off if you need to).
The flipside is that if you have a Sails service like:
module.exports = {
  someSweet3rdPartyToolMethod = require('tool')
};

and that third-party function has properties, or expects its context to be that of the 3rd party module, you're in trouble.  And since it's very unlikely that the third-party library would expect to have access to Sails as this.sails, it'd be perfectly safe for it not to be bound.  As @mikermcneil hinted at, a configuration setting to toggle this behavior could be the right answer, but it would probably have to apply to every method in the module--having it be more selective would make things pretty complicated and entangled.  In the worst case scenario where you've cooked up some service that needs to have untouched, 3rd party methods mixed with methods that need access to Sails, you can do something like:
module.exports = function(sails) {

   return {

      myMethod: function(foo) {...},
      3rdPartyMethod: require('tool');

   };

};

and call methods with MyService(sails).myMethod();

@sgress454 Good points!

and call methods with MyService(sails).myMethod();

@natecavanaugh for the record, this is what Sails core does internally in many cases.

Unless @natecavanaugh or anyone else has any immediate objection or is interested in writing up a full proposal for the config option to control this pretty quickly, let's plan to leave this behavior as is for 0.12, but then tentatively plan to remove the automatic binding of context for Sails v1.0.
cc @sgress454

see also https://github.com/balderdashy/sails/pull/2910  (this was merged, then later reverted- not sure what the intent was there.  Either way, it would to be rebased, and it would need accompanying changes to docs.  It does contain tests!)

@natecavanaugh Went ahead and opened a very simplistic proposal for this to make sure it stays on our radar: https://github.com/balderdashy/sails/pull/3536
@everyone:  We could really use help to get this done for Sails v1.0, so if you're interested in contributing, please check out that pull request above.  Thanks!

@mikermcneil Sorry for my late reply :)
But I think that's fine for now (I was able to switch out some of the functionality for another lib), but I'm happy to hear it's on the radar :)
Thanks again!


Update DG for undo command

Codecov Report

Merging #41 (c89e75a) into master (8193392) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage     41.48%   41.48%
  Complexity      266      266
=========================================
  Files            75       75
  Lines          1357     1357
  Branches        142      142
=========================================
  Hits            563      563
  Misses          764      764
  Partials         30       30

Continue to review full report at Codecov.

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


calculate cartline unit price based on cart user, rather than Auth::user

Currently, the GetUnitPrice class that's used to calculate CartLine unit prices in the Cart Pipeline relies on Auth::user() to fetch the CustomerGroups used to calculate the CartLine's pricing.
However, there are instances where Auth::user() would not yield the correct CustomerGroups that apply to the cart (when running a queued job, for example), as there is no current authenticated user in that context.
Instead, this Pull Request relies on the cart's user to get the applicable CustomerGroups.

Thanks @kylekanderson I think this would be useful in 0.2, can we target that branch please.

Thanks @kylekanderson I think this would be useful in 0.2, can we target that branch please.

Sure thing. I just updated this to target 0.2.

@alecritson I'm happy with this, can you sort the merge conflict and merge?


Normal offset application in vertex shader?

This line should be in the fragment shader.
vec3 invertedNormals = a_normals + (u_normals.x < 0.0 ? calculateOffset(u_normals) * 2.0 - 1.0 : vec3(0.0));

Which requires some refactors to the vertex shader.
This will apply the normal texture on a per fragment basis versus a per vertex basis, which makes a lot more sense for textures.
TLDR: If you don't have enough vertices it looks like this:

When it should look like this:

+1

What's the status on this fix @redwoodfavorite?

I'm close to finishing this, I have a working version but need to make some refactors.  I'm thinking I'll get the PR in by Friday.

@redwoodfavorite update please

This PR should addresses this issue, just need to provide perf examples -> https://github.com/Famous/engine/pull/315


exclude const generic arguments from generic_types

resolves #626
nvm got help from @aumetra and solved it :)

Just wondering whether it actually should work with const as well if so, then the fix would be more evolved. Yet at the moment it is not working with consts so I guess it is just fine to ignore them though. But most likely users with a struct having generic const variable it would still fail but with a different error than now.
However that can be another PR later on. I'll create an issue to the kanban board to track this if I ever get to test the cons support.

sounds good :)
I was actually trying to somehow change the code to make it also support consts (not just filter it out), but I didn't even know what the idea of it would be, or where it can be used in the generated docs

or where it can be used in the generated docs

Yeah, whatever can be done with const-generics (at the moment) seems somewhat irrelevant to OpenAPI docs (I could also just not be aware of use-cases).

But most likely users with a struct having generic const variable it would still fail but with a different error than now.

Do you mean with a constant associated type? Something like Foo<BAR = 2>? These will receive the same errors presumably, since they fall under the branch GenericArgument::AssocConst, which isn't covered by utoipa.
If you mean something like Foo<2>, it would fall under the same branch (GenericArgument::Const) and would be ignored.

Do you mean with a constant associated type? Something like Foo<BAR = 2>? These will receive the same errors presumably, since they fall under the branch GenericArgument::AssocConst, which is covered by utoipa to fail with the same error.
If you mean something like Foo<2>, it would fall under the same branch (GenericArgument::Const) that was patched with this PR and would be ignored.

Either of those, frankly I haven't looked in to the details how or what could be done with a const generic type. For utoipa OpenAPI docs it indeed does not make much sense and even a generic ToSchema type should be used with #[aliases] that tell the actual types of the generic arg anyway.


feat(providers): Upstream Transaction Conversion Util

Description
https://github.com/paradigmxyz/op-rs/pull/15 introduces the InMemoryChainProvider in the kona-providers crate.
Since the Chain state provides blocks and its associated transactions to commit to the internal provider, the types are coerced into kona-derive's TxEnvelope type.
This ticket is to add a utility into reth's rpc-types-compat crate, following discussion here to handle these type conversions.

Updated the issue to make it clear that this should live in reth's rpc-types-compat crate and not elsewhere


Fix: Issue #32

This PR resolves issue #32, where I was to add an about us and explore section
the about us section I added:

the explore section I added:
img-1

img-2

@khaulanauman  see it is looking  cool,

can resolve this   error .

@AkshitLakhera is this error on vs code? Try using npm run dev . I dont understand what ur trying to say

fixed the error-kindly review


fix(firebase_ui_localizations): Update Japanese translation

Description
Update Japanese translation
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] All unit tests pass (melos run test:unit:all doesn't fail).
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

@kahdojay could you please find some other dev who speaks Japanese to proof-read this PR and approve?

@KoheiKanagu - do you mind merging with latest on main branch, please?


feat: add /videos

Notes for Reviewers
This PR fixes #
/videos

a particular category's videos in grid

individual vidoe page ( customizable)

Signed commits

[ ] Yes, I signed my commits.

@leecalcote and others need to deliver videos.


buildseries.com fails to find video file

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.07.10. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.07.10

Before submitting an issue make sure you have:

[ ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

This site does not download auy video just small file, it fails to get the video
youtube-dl --no-check -v https://www.buildseries.com/video/5b43b33fdbaab32daa090d27/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--no-check', '-v', 'https://www.buildseries.com/video/5b43b33fdbaab32daa090d27/']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2018.07.10
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] Proxy map: {}
[generic] 5b43b33fdbaab32daa090d27: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 5b43b33fdbaab32daa090d27: Downloading webpage
[generic] 5b43b33fdbaab32daa090d27: Extracting information
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'http://delivery.vidible.tv/video/redirect/5b43b3
3fdbaab32daa090d27.mp4?bcid=56e9e5c0ff690c1d9008d4e6&w=1280&h=720&en
c=mp4&domain=cdn.vidible.tv'
[download] Destination: Jake Weary Discusses TNT's 'Animal Kingdom'-5b43b33fdbaa
b32daa090d27.mp4
[download] 100% of 1.11KiB in 00:00
same also for uk site https://www.buildseries.com/uk/

HINT:
It's probably videos from AOL, embed is:

and second for this same video:

@dstftw time to add AOL to generic: https://github.com/rg3/youtube-dl/search?q=vidible&unscoped_q=vidible

redirects to yahoo.com.

redirects to yahoo.com.


Added user agent show and cookie format based.

I did some modifications.

@aeri resolve the conflicts ASAP and I will merge your code. You can try to get the :gift: in a future pull request. The challenge proposed is optional and you can bring your own ideas.
The current conflicting files are:
README.md
src/main/java/es/unizar/webeng/hello/HelloController.java
src/main/webapp/WEB-INF/jsp/wellcome.jsp
src/test/java/es/unizar/webeng/hello/HelloControllerUnitTest.java

Fixed tests.


Problem with WindowsTerminal/CMD - differences

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: Version: 1.87.2
Commit: 863d2581ecda6849923a2118d93a088b0745d9d6
Date: 2024-03-08T15:20:17.278Z
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22631

OS Version: W11

Steps to Reproduce:

I'm following the https://www.youtube.com/watch?v=RtY5FV5TrIU NES Development Environment

Want compile demo from https://github.com/NesHacker/DevEnvironmentDemo

Using https://sourceforge.net/projects/cc65/files/cc65-snapshot-win32.zip

I have errors:

 *  The terminal process "C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -Command cl65 "e:\roms\DevEnvironmentDemo-main\demo.s" " terminated with exit code: 1.
 *  Terminal will be reused by tasks, press any key to close it.

 *  Executing task: cl65 "e:\roms\DevEnvironmentDemo-main\demo.s"

cl65 : The term 'cl65' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1
+ cl65 e:\roms\DevEnvironmentDemo-main\demo.s
+ ~~~~
    + CategoryInfo          : ObjectNotFound: (cl65:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

 *  The terminal process "C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -Command cl65 "e:\roms\DevEnvironmentDemo-main\demo.s" " terminated with exit code: 1.
 *  Terminal will be reused by tasks, press any key to close it.

The reason for this due "Terminal" not like cmd but interactive. It's looks Windows problem around where Ineractive terminal is not this same as CMD..
Eg. when doing command in cmd: cl65.exe  --verbose --target nes demo.s -o demo.nes its working
Also I noticed simial problem around CategoryInfo          FullyQualifiedErrorId  when using simple PIP in some cases, but not in CMD, by this powershell.exe doing problems around VSC. Not sure how to solve.. but maybe should be option how to force to CMD not powershell/WindowsTerminal

Also trying experiments to omit Task build by "play" btn / debugger but no positive results.
        {
            "name": "ca65 NES",
            "type": "cl65",
            "request": "launch",
            "program":"${file}",
            "console": "integratedTerminal",
            "executable": "E:\\cc65-snapshot-win32\bin\\cl65",
            //"input": "demo.s",
            //params": "--verbose --target nes -o demo.nes",
            "args": ["--verbose --target nes -o demo.nes"],
            "sourcePath": "${workspaceFolder}/",
        },

Regarding python and WindowsTerminal:
>setup
The Setup command may only be used inside a Describe block.
At C:\Program Files\WindowsPowerShell\Modules\Pester\3.4.0\Functions\Describe.ps1:125 char:9
+         throw "The $CommandName command may only be used inside a Des ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (The Setup comma...Describe block.:String) [], RuntimeException
    + FullyQualifiedErrorId : The Setup command may only be used inside a Describe block.

It's something like this:
activate
activate : The term 'activate' is not recognized as the name of a cmdlet, function, script file, or operable program. C
heck the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1
+ activate
+ ~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (activate:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Suggestion [3,General]: The command activate was not found, but does exist in the current location. Windows PowerShell does not load commands from the current location by default. If you trust this command, instead type: ".\activate". See "get-help about_Command_Precedence" for more details.

Then ok .\activate but if exist something like cl gcc msvc etc. giving problems.
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "ca65 NES compiler XXX",
            "command": "C:\\cc65\\bin\\cl65 --verbose --target nes ${file} -o demo.nes",,

        },

I'm getting Cannot find a rom path in launch.json

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!

It's problem around WindowsPowerShell use in case it's not this same as cmd around execute commands.


[BUG] kim mouse atlas coordinate space

Describe the bug
kim_mouse_100um mesh and annotations are not in same coordinate system?
To Reproduce
Display the root mesh and the annotations image in brainrender-napari
Expected behaviour
Root mesh and annotations images roughly overlap
Log file
n/a
Screenshots
If applicable, add screenshots to help explain your problem.

Computer used (please complete the following information):

OS: Ubuntu
Version: 22.04.2 LTS

This is now fixed (I assume due to https://github.com/brainglobe/brainglobe-atlasapi/pull/271).


Support for rules to run system commands

Add support to run various miscellaneous system commands not covered by existing object types.   For example:
CRTSBSD.FILE:dummy.pf
system -i "CRTSBSD SBSD(BATCHWL/BATCHSBSD) POOLS((1 *SHRPOOL3 *N *MB))"
CHGSBSD.FILE:dummy.pf
system -i "CHGSBSD SBSD(BATCHWL/BATCHSBSD) POOLS((3 8000 999 *MB))"
CRTJOBQ.FILE:dummy.pf
system -i "CRTJOBQ JOBQ(BATCHWL/BWLJOBQ)"
CRTJOBD.FILE:dummy.pf
system -i "CRTJOBD JOBD(BATCHWL/BWLJOBD) JOBQ(BATCHWL/BWLJOBQ) LOG(4 40 *MSG)"
ADDJOBQE.FILE:dummy.pf
system -i "ADDJOBQE SBSD(BATCHWL/BATCHSBSD) JOBQ(BATCHWL/BWLJOBQ) MAXACT(*NOMAX) SEQNBR(15)"
CRTCLS.FILE:dummy.pf
system -i "CRTCLS CLS(BATCHWL/BWLCLASS) RUNPTY(30) TIMESLICE(5000)"
ADDRTGE1.FILE:dummy.pf
system -i "ADDRTGE SBSD(BATCHWL/BATCHSBSD) SEQNBR(9999) CMPVAL(*ANY) PGM(QSYS/QCMD) CLS(BATCHWL/BWLCLASS) POOLID(3)"

I did have tabs, but after this comment I tried with tabs, spaces, and no spaces.   Can't seem to get it to run (haven't looked too deeply, working on other tasks at the moment).

let me investigate

OK I got it working with a few tweaks on your Rules.mk
-bash-5.1$ makei b -t CRTSBSD.FILE -e lib1=BOBBUILD
Set variable <lib1> to 'BOBBUILD'
> /QOpenSys/pkgs/bin/make -k BUILDVARSMKPATH="/tmp/tmppyno8ixq" -k BOB="/QOpenSys/pkgs/lib/bob" -f "/QOpenSys/pkgs/lib/bob/src/mk/Makefile" CRTSBSD.FILE
=== Creating [CRTSBSD.FILE] from custom recipe
system -i "CRTSBSD SBSD(BATCHWL/BATCHSBSD) POOLS((1 *SHRPOOL3 *N *MB))"
CPD000D: Command *LIBL/CRTSBSD not safe for a multithreaded job.
CPD9801: Library BATCHWL not found.
CPF1696: Subsystem description BATCHSBSD not created.
make: *** [/home/REINHARD/bob-recursive-example/susan/.Rules.mk.build:7: CRTSBSD.FILE] Error 255
Objects:             0 failed 0 succeed 0 total
Build Completed!

I changed it to
CRTSBSD.FILE: SAMREF.FILE
	system -i "CRTSBSD SBSD(BATCHWL/BATCHSBSD) POOLS((1 *SHRPOOL3 *N *MB))"

Notice that I had to give it a real file that already exists
I put a space after the colon of the target and a tab before the system command

We will keep this issue because you shouldn't have to make a dummy source

https://vscode.dev/github/IBM/ibmi-bob/blob/b816e8bd912fb37a78896a24e01e88de2559fc60/src/makei/rules_mk.py#L120
Problem was address with this regex to parse a target line with no dependencies, but now it breaks if line is followed by a space
target: dep<\n>
<space><\n>


[BUG] Nothing works for me

helix-editor version
23.10
helix-gpt version
31.0
Describe the bug
Nothing seems to work.
:lsp-workspace-command

Nothing happens if I press  C-x nor on space-a

languages.toml
[[language]]
name = "tsx"
language-servers = [ "eslint", "emmet-ls", "typescript-language-server", "tailwindcss-ls", "gpt" ]
formatter = { command = "prettier", args = [ "--parser", "typescript" ] }
auto-format = true

[language-server.gpt]
command = "helix-gpt"
args = ["--logFile", "/tmp/helix-gpt.log"]

helix-gpt logs
tail -F /tmp/helix-gpt.log
APP 2024-03-26T09:14:27.113Z --> triggerCharacters: | ["{","("," "]

helix logs
This is the only reference to gpt I see on the logs
2024-03-26T10:14:27.077 helix_lsp::transport [INFO] gpt -> {"jsonrpc":"2.0","method":"initialize","params":{"capabilities":{"general":{"positionEncodings":["utf-8","utf-32","utf-16"]},"textDocument":{"codeAction":{"codeActionLiteralSupport":{"codeActionKind":{"valueSet":["","quickfix","refactor","refactor.extract","refactor.inline","refactor.rewrite","source","source.organizeImports"]}},"dataSupport":true,"disabledSupport":true,"isPreferredSupport":true,"resolveSupport":{"properties":["edit","command"]}},"completion":{"completionItem":{"deprecatedSupport":true,"insertReplaceSupport":true,"resolveSupport":{"properties":["documentation","detail","additionalTextEdits"]},"snippetSupport":true,"tagSupport":{"valueSet":[1]}},"completionItemKind":{}},"hover":{"contentFormat":["markdown"]},"inlayHint":{"dynamicRegistration":false},"publishDiagnostics":{"versionSupport":true},"rename":{"dynamicRegistration":false,"honorsChangeAnnotations":false,"prepareSupport":true},"signatureHelp":{"signatureInformation":{"activeParameterSupport":true,"documentationFormat":["markdown"],"parameterInformation":{"labelOffsetSupport":true}}}},"window":{"workDoneProgress":true},"workspace":{"applyEdit":true,"configuration":true,"didChangeConfiguration":{"dynamicRegistration":false},"didChangeWatchedFiles":{"dynamicRegistration":true,"relativePatternSupport":false},"executeCommand":{"dynamicRegistration":false},"inlayHint":{"refreshSupport":false},"symbol":{"dynamicRegistration":false},"workspaceEdit":{"documentChanges":true,"failureHandling":"abort","normalizesLineEndings":false,"resourceOperations":["create","rename","delete"]},"workspaceFolders":true}},"clientInfo":{"name":"helix","version":"23.10 (f6021dd0)"},"processId":1720,"rootPath":"/Users/adrian/Developer/e-id","rootUri":"file:///Users/adrian/Developer/e-id","workspaceFolders":[{"name":"e-id","uri":"file:///Users/adrian/Developer/e-id"}]},"id":0}

Anything else to look for?

see https://github.com/leona/helix-gpt/issues/49


Changing a switch state when pressed

Hi,
This may have been answered already but I was not able to find anything, so if it was sorry.
I have implemented UPB switches using HAP-NodeJS and everything is working well. The next task is to change the state of the switch in Homekit when a switch is pressed manually. UPB provides a event message that contains the details of the event that can be decoded and then used to set the state of the appropriate switch in HAP-NodeJS. My question is how do I do this and where would the code be? The event needs to be decoded first then the appropriate switch state would need to be changed.
Update: The way in which I communicate with UPB is through an RS232 interface module so the events are sent when the switch is pressed and I already know how to decode the message. I am just thinking that the event handler needs to be in the main code and not in the switch code since it could happen at any time. I think at one time KNX was supported in a similar way but that was a while back and the code structure has changed since then. Just looking for pointers as this is the last part of my implementation.
I have a true HomeKit switch that works in the way I want it to so I am sure it is possible.
Thanks in advance for any pointers.
Andy

Hi I would like to implement accessory light so that switch status is changed to Homekit when a switch is manually pressed.
A solution could be adaptable as for the garage sensor, in input on the gpio of raspberry with wirinpi.
Someone has solutions to create this accessory?
thank you


Add BuildProgressPlugin

I figured I'd take a stab at #935 since it seemed relatively easy.
I do have a concern though!
I created the plugin in react-dev-utils per suggestion in #935, however this resulted in adding webpack as a dependency 😱 -- it was listed as a peerDependency previously (which fails to resolve post lerna/lerna#318).
Would it be better to implement it in react-scripts and add progress as a dependency? Or would you rather explore a solution which doesn't use progress?
edit: read below

Using the same technique used in https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/config/paths.js#L74, I changed webpack back to a peerDependency in ee9ca5e7bce53d7c71321ae07ed84e26066b79b4.

Closing this as the solution doesn't seem appropriate given the webpack peerDependency was removed in #963 / 9d53d22115a67a3fae36a0f1a08aaeb67953973f.
It seems the solution for this will be implemented elsehow (not in react-dev-utils, but perhaps in react-scripts).

Sorry, I had no time to look at this yet. I will, can you please reopen?
require('webpack').ProgressPlugin should be fine. Peer dependency was only removed because now that we declare compat with Webpack 2, it will be annoying to bump it every few weeks. If npm provided a way to declare compat with all beta versions, I would've kept it. But


[bug]: Installation fails at "Preparing metadata (setup.py)"

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
cuda
VRAM
12GB
What happened?
I was trying to install InvokeAI with the provided install.sh script, but the installer quits with error code 1 and prints this error
Collecting basicsr>=1.4.2
  Downloading basicsr-1.4.2.tar.gz (172 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 172.5/172.5 kB 6.2 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [33 lines of output]
      /home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/installer.py:27: SetuptoolsDeprecationWarning: setuptools.installer is deprecated. Requirements should be satisfied by a PEP 517 installer.
        warnings.warn(
      WARNING: The wheel package is not available.
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/tmp/pip-install-1jv5lczb/basicsr_cdab16a383a747ca93d00fb420ff69eb/setup.py", line 147, in <module>
          setup(
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/__init__.py", line 86, in setup
          _install_setup_requires(attrs)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/__init__.py", line 80, in _install_setup_requires
          dist.fetch_build_eggs(dist.setup_requires)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/dist.py", line 874, in fetch_build_eggs
          resolved_dists = pkg_resources.working_set.resolve(
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/pkg_resources/__init__.py", line 789, in resolve
          dist = best[req.key] = env.best_match(
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/pkg_resources/__init__.py", line 1075, in best_match
          return self.obtain(req, installer)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/pkg_resources/__init__.py", line 1087, in obtain
          return installer(requirement)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/dist.py", line 944, in fetch_build_egg
          return fetch_build_egg(self, req)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/installer.py", line 87, in fetch_build_egg
          wheel.install_as_egg(dist_location)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/wheel.py", line 110, in install_as_egg
          self._install_as_egg(destination_eggdir, zf)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/wheel.py", line 118, in _install_as_egg
          self._convert_metadata(zf, destination_eggdir, dist_info, egg_info)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/wheel.py", line 139, in _convert_metadata
          _unpack_zipfile_obj(zf, destination_eggdir)
        File "/home/toasterbirb/.local/share/invokeai/.venv/lib/python3.10/site-packages/setuptools/archive_util.py", line 130, in _unpack_zipfile_obj
          f.write(data)
      OSError: [Errno 28] No space left on device
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Screenshots
No response
Additional context
Some python version information in-case that matters
pip 22.2.2
Python 3.10.9
Also, I have 49G left in the partition I'm installing into, so the OSError: [Errno 28] No space left on device error is a bit odd
Contact Details
No response

OSError: [Errno 28] No space left on device means you've run out of space on the drive where you're trying to install InvokeAI. If you have a drive with more space available, you can select that path when the installer asks you to "Select your preferred location for the 'invokeai' directory".
I also see paths like /home/toasterbirb/.local/share/invokeai/.venv/ in the log, which is strange because the /invokeai/ directory should not end up under ~/.local/share. Could you please verify the path you're installing to?

I was installing to ~/.local/share on purpose. Is there some reason why it shouldn't be put there?
I have 49 gigabytes of storage space available in the partition I'm trying to install InvokeAI into

I was installing to ~/.local/share on purpose. Is there some reason why it shouldn't be put there?

No reason - if that's intended, go for it. Just didn't expect to see that location in the logs.
To confirm, you're selecting this location when prompted by install.sh?

I have 49 gigabytes of storage space available in the partition I'm trying to install InvokeAI into

That's very odd indeed, but the message clearly states that there's no space left on the device....
perhaps you could try to start the installation again, and run watch df -h in another terminal to pinpoint the device whose free space begins to dwindle?

Yes, I was specifying the location with the installation script. The total size of the installation directory was about 22 megabytes in size.
When installing and monitoring df -h, it looks like my /tmp gets full. I have mounted it as tmpfs and 4 gigabytes of storage. I'll try giving it temporarely a bit more space and see if that helps

Yup, that was the issue. Looks like you need about 6 gigabytes of space in /tmp to get trough the installation.

Yup, that was the issue. Looks like you need about 6 gigabytes of space in /tmp to get trough the installation.

Thank you! we can add this to the documentation.


Split pattern parsing and text matching for better performance

This commit creates two new functions, compile_pattern() and grok_search(), which simply split grok_match() into compiling and searching. This greatly improves performance when matching large amounts of text to the same pattern repeatedly, since the pattern can be expanded by compile_pattern() once and reused many times by calling grok_search, typically in a loop over all input lines.
In my own testing this improved the performance of PyGrok by about 70%. The performance now matches re.search() provided by Python, since that is basically all that's going on in grok_search().
Name of new functions can be changed, compile_pattern() and grok_search() are just suggestions. I wanted to avoid changing grok_match() to avoid breaking existing code.

+1 on this change. very good results regarding parsing speed

Any hope of this getting merged?

I would really love to also have this feature. I have a very bad performance without this and this really helps

@garyelephant any plans to implement #20 or simply take this PR as is ? otherwise I will simply take this PR off to my own fork of pygrok and then upload to pypi under a slightly different name.

https://pypi.python.org/pypi/pygroker temporary package until this becomes part of pygrok itself.

Thanks @rlgomes. I will implement #20 this week

So did anything happen on #20?

pygrok 1.0.0 is available in pip now


Missing $HOME/.ssh causes gopass decryption to error out

Summary
If ~/.ssh is missing from the user's home directory and the store uses the age encryption backend, showing a secret will fail citing inability to open a file in $HOME/.ssh.
Steps To Reproduce

In a testing environment, rm -rf $HOME/.ssh or rm -rf ~/.ssh or attempt from within a containerized environment missing this directory by default.
Run gopass show -f secret_name
Enter password/PIN if asked
Results:[root@3737b5f66ec4 ~]# gopass show -f secret_name
Enter your PIN:
❌ Decryption failed: open /root/.ssh: no such file or directory

Error: failed to retrieve secret "secret_name": failed to decrypt

Recipients info:[root@3737b5f66ec4 ~]# gopass recipients
Hint: run 'gopass sync' to import any missing public keys
gopass
└── age1<...>

Expected behavior
gopass gracefully skips over the missing file in $HOME/.ssh and moves on to attempt decryption with age keys, only producing an error after and if all decryption methods have been exhausted.
Environment

OS: Fedora 36 host, Fedora 36 container
OS version: Linux 3737b5f66ec4 6.1.6-100.fc36.x86_64 #1 SMP PREEMPT_DYNAMIC Sat Jan 14 17:00:40 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
gopass Version: gopass 1.14.3 go1.18.3 linux amd64
Installation method: Fedora package repositories

Additional context

Without evaluating the code at play here, it is possible the root cause of this issue lies with an upstream go package.

Oh, this is an old version.
We already fixed this.


.IPScan accuracy needs NSAppTransportSecurity

The URL call for .IPScan points to an http address which is prohibited by default. We should enable such calls in the app to be able to call the API and get the results.
I'll make a PR to point this out in the README file.

Upon further reading, I'm now concerned about anybody using this library in a commercial application. ip-api.com clearly states that there is a usage limit (150 request per IP per minute) and commercial applications need to go pro, in which case they can use SSL (which is slower and I don't recommend for location lookup)

Please explain, what do you mean by "prohibited by default" ?
I don't share your commercial concern. I can imagine many apps which does not make 150 request per minute from the same IP address. Their PRO feature seems reasonable priced for people who do.

On iOS 9+, you need special setting in info.plist to be able to call any URL with http protocol. (Read Here)
About the commercial concern, I'm not concerned for myself, but for anybody who'd use this library. You see, this library uses an API which might have consequences for anybody who is using it, you should at least warn them. That's all I'm saying. And yes, their pricing is quite reasonable, and I think you should allow settings for pro users (just like Google's API key) in the framework.

Gotcha. iOS 9 SDK requirement. Agree on the commercial comment as well. Thanks for your contributions.

Thank you for your PR. I've ported them to Swift 2.2 branch


Multiple output prediction is possible  in cog?

i want to return two output prediction in below code..
        if tables_crops:
            temp_dir = tempfile.mkdtemp()
            cropped_table_path = Path(temp_dir) / f'cropped_table.jpg'

            # Save the cropped table image in the temporary directory
            cropped_table = tables_crops[0]['image'].convert("RGB")
            cropped_table.save(cropped_table_path)
            print(f"File is successfully saved at {cropped_table_path}")

            bbox_info = [obj['bbox'] for obj in objects if obj['label'] in ['table', 'table rotated']]
            bbox_json = json.dumps(bbox_info, indent=4)

            # Save JSON data to a file
            json_path = Path(temp_dir) / 'bbox_info.json'
            with open(json_path, 'w') as json_file:
                json_file.write(bbox_json)

            # Return the paths of the saved image and JSON file
            return cropped_table_path, json_path

my run comment is : $ cog predict -i<EMAIL_ADDRESS>but i got error..when i use single prediction it worked smoothly..(ex. return json_path )
my cog version is 0.8.6
can anyone fix this..thx in advance!!! @mattt @zeke

Yep, you wanna do something like this:
from cog import BasePredictor, BaseModel, File, Path

class Output(BaseModel):
    cropped_table: Path
    json: Path

class Predictor(BasePredictor):
    def predict(self) -> Output:
         ...
        return Output(cropped_table=..., json=...)

Bear in mind that cog has a Path that differs from (extends?) the built-in Python path.
See https://github.com/replicate/cog/blob/main/docs/python.md#returning-an-object