[BUG] Azure OpenAi with Azure Search with Semantic search

Library name and version
Azure.AI.OpenAI
Describe the bug
I am using Azure.Ai.OpenAi with Azure search to get answers on my documents with a semantic configuration. I have the feeling when the answer is too big, I get a a 207 HTTP error code _message.
I tried with different AI Model : Gpt-4 1106-Preview;0613 AND  Gpt-35-turbo 0301
tentative messages:
Depending of the question, I get the error 207 error code or not
Fail: HttpStatus 207 error code

What are my obligration if I want to distribute the Nozzles B04614 ???
What is the B0701 ???
How to protect my kitchen ?

Success

Hi

Success but no data found

What is the B04614 ???

Azure Search configuration
Index

Semantic configuration

Expected behavior
I should get a GPT answer to my question
Actual behavior
207 HTTP error code message
Reproduction Steps
    internal class Program
    {
        private const string OpenAiEndpoint = "{{SetValues}}";
        private const string OpenAiAzureKeyCredential = "{{SetValues}}";
        private const string OpenAiModelName = "{{SetValues}}";  // I tested Gpt-4 1106-Preview;0613 AND  Gpt-35-turbo 0301

        private const string AzureSearchEndpoint = "{SetValues}}";
        private const string AzureSearchIndexName = "{{SetValues}}";
        private const string AzureSearchAuthenticationKey = "{{SetValues}}";
        private const string AzureSearchSemanticConfigurationName = "{{SetValues}}";

        static void Main(string[] args)
        {
            OpenAIClient client = new OpenAIClient(
                new Uri(OpenAiEndpoint),
                new AzureKeyCredential(OpenAiAzureKeyCredential)
            );

            ChatCompletionsOptions chatCompletionsOptions = new ChatCompletionsOptions()
            {
                DeploymentName = OpenAiModelName,
                Temperature = (float)0,
                MaxTokens = 800,
                NucleusSamplingFactor = (float)1,
                FrequencyPenalty = 0,
                PresencePenalty = 0,
                AzureExtensionsOptions = new AzureChatExtensionsOptions()
                {
                    Extensions =
                    {
                        new AzureSearchChatExtensionConfiguration()
                        {
                            SearchEndpoint = new Uri(AzureSearchEndpoint),
                            IndexName = AzureSearchIndexName,
                            Authentication = new OnYourDataApiKeyAuthenticationOptions(AzureSearchAuthenticationKey),

                            // I have the bug only when using semantic search
                            QueryType = AzureSearchQueryType.Semantic,
                            SemanticConfiguration = AzureSearchSemanticConfigurationName,
                        },
                    }
                }
            };

            // Fail: HttpStatus 207 error code
            // chatCompletionsOptions.Messages.Add(new ChatRequestUserMessage("What are my obligration if I want to distribute the Nozzles B04614 ???"));
            // chatCompletionsOptions.Messages.Add(new ChatRequestUserMessage("What is the B0701 ???"));
            // chatCompletionsOptions.Messages.Add(new ChatRequestUserMessage("How to protect my kitchen ?"));

            // Success
            chatCompletionsOptions.Messages.Add(new ChatRequestUserMessage("Hi"));

            // Success but no data found
            // chatCompletionsOptions.Messages.Add(new ChatRequestUserMessage("What is the B04614 ???"));

            Response<ChatCompletions> responseWithoutStream = client.GetChatCompletions(chatCompletionsOptions);

            ChatCompletions completions = responseWithoutStream.Value;
            string content = completions.Choices[0].Message.Content;
        }
    }

Environment
Visual studio 2022:
Version 17.9.2
Azure.Ai.OpenAi 1.0.0-beta.15
Console Application
.NET SDK:
Version:           8.0.201
Commit:            4c2d78f037
Workload version:  8.0.200-manifests.e575128c
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19045
OS Platform: Windows
RID:         win-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.201\

Here is an error message I have

Azure.RequestFailedException: 'Service request failed.
Status: 207 (Multi-Status)
Content:
{"id": "5415d5ba-35cc-4700-a636-ae5fe6c6216f", "model": "gpt-4", "created":<PHONE_NUMBER>, "object": "extensions.chat.completion", "choices": [{"index": 0, "finish_reason": "stop", "message": {"role": "assistant", "content": "The B0701 refers to a series of ILP (Integrated Life Protection) valves used in fire suppression systems. These valves control the release of extinguishing agents, such as ABC dry chemical powder, from pressurized cylinders in the event of a fire. The ILP Powder System, which includes the B0701 valve, is designed for rapid fire suppression to minimize property damage. The B0701 valve is equipped with features such as a pressure gauge, a ball valve, and multiple outlet connections for the discharge network [doc2][doc5]. The system is intended for use in various applications, including protecting engine compartments and adjacent auxiliary aggregates in vehicles [doc5].", "end_turn": true, "context": {"citations": [{"content": "5  Solenoid actuator\nThe solenoid actuator is an electromechanical device that allows the ILP Powder System to be actuated electrically \nfrom a control unit or other electric signal.The solenoid actuator is equipped with a supervisory switch which must be \nelectrically connected to the supervisory circuit of the control unit.The solenoid actuator is only permitted for use in the temperature range from -40 \u00b0C to +50 \u00b0C.Article number 029900234\nInlet connection (C) M16 x 2 Minimess with non-return valve\nOutlet connection (A) M10 x 1 male with O-ring\n\n6.9.7  Manual release device with diaphragm\n The manual release device with diaphragm is an inline device.When the ILP valve is actuated it releases the nitrogen in  \n the FireDETEC\u00ae  pneumatic detection tube allowing the ILP valve to open and release the extinguishing agent into the  \n hazard area.The warning labels provide information on how \nto use a component or that a ILP Powder System is installed for fire protection.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.THIS EQUIPMENT IS PROTECTED AGAINST FIRE BY:THIS EQUIPMENT IS PROTECTED AGAINST FIRE BY: WARNING\nDO NOT USE THIS EQUIPMENT WHEN THE FIRE \nPROTECTION SYSTEM IS DISABLED.MANUAL ACTUATION\n\nPull the yellow safety \ndevice\n\nPush the red \nhandle\n\n1 21 2\n\nCAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.CAUTION\nTHIS TUBE IS UNDER PRESSURE.Article number 026300506\n\nConnection ISO 8434-1 M18x1,5 with 24\u00b0 cone\n\nStandard ISO 8434-1\n\nWorking pressure PN 315\n\nSurface Electro galvanized\n\nMaterial Steel\n\nPlacing the double swivel nut directly between nozzle fitting and T-fitting\nThe double swivel nut is used to connect the nozzle fitting 026300012 to the T-fitting 026600017.Page 26 of 62 027650086 - 2023/09/20\n\nSecuring the nozzle with the nozzle bracket\nThe double swivel nut is used to connect the nozzle fitting 026300012 to the bulkhead connector 026300507 when using \nthe nozzle with a designated nozzle bracket 024900495.Nozzle fitting\n\nNozzle\n\nDouble swivel nut\n\nNozzle bracket\n\nBulkhead connector \n\n6.9.10  Nozzle bracket\nThe nozzle bracket is used to fix the bulkhead connector in place when securing the nozzle as detailed in chapter 6.9.9", "title": null, "url": null, "filepath": null, "chunk_id": "2"}, {"content": "....................................................................................  11\n5.5 Pressure ..........................................................................................................................................................................  11\n5.6 Safety precautions ..........................................................................................................................................................  11\n 5.6.1 General ...............................................................................................................................................................  11\n 5.6.2 Pressure cylinders including transportation ........................................................................................................  11\n\n6 Components ..................................................................................................................................................................  12\n6.1 ILP valve series B0701.Pressurized extinguishing agent pressure cylinder\nThe ILP Powder System uses a pressurized type B0701 cylinder valve assembly..The ILP Powder System combines an environmentally safe extinguishing agent and specially developed components for \na powerful and rapid fire suppression, which reduces property damage to the lowest possible extent.\n\n3.1 Illustrations\n3.1.1 ILP valve\n\nG\n\n1\n\nA\n\nA\n\nC\n\n2\n\n3\n\n4\n\n5\n\nFigure <IP_ADDRESS> B0701 Valve with ball valve\n\n  Legend\n1 Pressure gauge 5 Safety screw\n2 Ball valve A Outlet connection\n3 Safety clip C Inlet connection\n4 FireDETEC\u00ae connection G Dip tube thread\n\n\n\nInstruction manual\nFireDETEC\u00ae ILP System for ABC powder - UNECE R107 - B07508\n\nCEODEUX Extinguisher Valves Technology S.A..Page 12 of 62 027650086 - 2023/09/20\n\n6 Components\nFor detailed product information on the respective components refer to their respective instruction \nmanuals.\n\n6.1 ILP valve series B0701\nThe ILP valve controls the release of the ABC powder from the ILP pressure cylinder.", "title": null, "url": null, "filepath": null, "chunk_id": "0"}, {"content": "The caution sign must be placed in a conspicuous location at the entrance to the \nprotected hazard and or on the cabinet or machine that is being protected.Instruction manual\nILP fire suppression system for FM-200TM B0701\n\nCEODEUX Extinguisher Valves Technology S.A", "title": null, "url": null, "filepath": null, "chunk_id": "4"}, {"content": "1 Isometric diagram of FK-5-1-12 pressurized with nitrogen to 240 psi at +68 \u00b0F\n\nInstruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.\u00b0F)\n\nHeat of vaporization (kJ/kg \u00b0C) at boiling point 37,8\nThermal conductivity (W/m \u00b0C) of liquid 0,034\n\nInstruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Instruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Instruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Article number 029900168\n\nSwitch type SPST (auto-reset pressure switch)\n\nFixture 1/8\u201d NPT\n\nProof pressure 400 psi\n\nType Normally open (N/O) at atmospheric conditions (0 psi)\n\nElectrical rating 120 / 240 V - 375 VA\n\nConformity UL listed\n\nInstruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Article number B07800002 Article number B07800200\n\nOuter diameter 0,234\u201d / 6 mm Outer diameter 0,234\u201d / 6 mm\n\nInternal diameter 0,156\u201d / 4 mm Internal diameter 0,156\u201d / 4 mm\n\nColor Red RAL 3000 Color Black RAL 9005\n\nMinimum bending radius 4\u201d / 100 mm Minimum bending radius 5,9\u201d / 150 mm\nBurst temperature +320 \u00b0F at 240 psi\n\n+160 \u00b0C at 16,5 bar\nBurst temperature +284 \u00b0F at 240 psi\n\n+140 \u00b0C at 16,5 bar\n\nConformity UL listed Conformity\n\nInstruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Through-panel \nfastener\n\nThrough-panel \nprotector\n\nArticle number B07830005 Article number B07860001\nOrifice diameter 0,236\u201d / 6 mm Orifice diameter 0,25\u201d / 6,35 mm\nMaximal panel thickness 0,95\u201d / 2,4 mm Maximal panel thickness 0,07\u201d / 1,8 mm\nLength 1,18\u201d / 30 mm Length 0,20\u201d / 5 mm\nMaterial Polyamide Material Rubber\n\nInstruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Pre Engineered Fire Suppression Systems - FK-5-1-12 \nFire Extinguishing Agent\n\n028000493\n\nARTICLE NUMBER\n\nYEAR OF MANUFACTURING\n\nFILLING LOCATION\n\nPRESSURE (bar / psi)\n\n/\n\nGROSS WEIGHT (Kg / Lbs)\n\n/\n\nAGENT WEIGHT (Kg / Lbs)\n\n/\n\nTARE WEIGHT (Kg / Lbs)\n\n/\n\nChemical Under Pressure, N.O.S., (Nitrogen)\nFK-5-1-12 Fire Protection Fluid\n\nUN 3500\n\nArticle number 028000492 (for FM/UL) 028000493\nKey features self-adhesive self-adhesive\nDimensions 135 x 175 mm (5,31\u201d x  6,88\u201d) 135 x 175 mm (5,31\u201d x  6,88\u201d) \n\nInstruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Article number B04420127\nTube connection 6 mm quick connect fitting\nLength 3,9\u201d / 100 mm\nHEX flats 9W / 24 V\nPressure gauge scale 0 to 400 psi\nCharging port YES\n\nInstruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Article number B07850006\nAudible signal 87d\nPower supply 110-220 V AC / 6mA\nFrequency 3 kHz\n\nInstruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A.Instruction manual\nILP total flooding system for FK-5-1-12 B0701\n\nCEODEUX Extinguisher Valves Technology S.A", "title": null, "url": null, "filepath": null, "chunk_id": "1"}, {"content": "11\n5.5 Pressure .  11\n5.6 Safety precautions .  11\n 5.6.1 General .  11\n 5.6.2 Pressure cylinders including transportation .  11\n\n6 Components .  12\n6.1 ILP valve series B0701.The product described in this instruction manual is intended exclusively for specialists with experience in installation and \nmaintenance and specifically with the product described herein. \n\n1.1 General understanding of safety\nThe product has been safely constructed and poses no danger if it is operated in a technically correct manner.Obey all instructions and warnings in this instruction manual when assembling, installing and operating this product.\n\n1.2 Distributor obligations\nThe distributor of the product described herein must ensure the following:\n\u2022 All warnings, instructions and product labels for end-users are in the responsibility of the distributor.\n\u2022 Check whether the product is subject to European Regulations, Directives or other acts and, if necessary, whether it \n\nis marked accordingly (for example  or ) and, if required, whether it is supplied with the declaration of conformity.\n\u2022 Protection from unauthorized access.\n\u2022 Compliance with European and national regulations.\n\u2022 Awareness and comprehension of the product specifications of all affected personnel.\n\u2022 Avoidance of non-intended use.\n\u2022 When the product is installed, all warnings, instructions and product labels for the end-users are in the responsibility \n\nof the operator.\n\n1.3 Symbols used and their meanings\nThe warnings are scaled according to the.Indicates useful tips and recommendations as well as information for efficient and trouble-free \noperation.\n\n2 Intended use\nThe FireDETEC\u00ae ILP System for ABC dry chemical powder \u2013 UNECE R107 is a pre-engineered fire fighting system \nintended for use on category M2 and M3 vehicles, as detailed by UN Regulation No 107.It utilizes dry chemical ABC \npowder FUREX ABC 770, which is effective on:\n\u2022 Class A fires  -  surface fires of celluloid materials (wood, paper, cloth \u2013 anything that leaves an ash residue after \n\ncombustion)\n\u2022 Class B fires  -  fires of flammable liquids and gases\n\u2022 Class C fires -  fires that involve energized electrical equipment\nThe FireDETEC\u00ae ILP System with ABC dry chemical powder \u2013 UNECE R107 must be designed and installed in \naccordance with the requirements outlined in this instruction manual.The product is defined for the European market/countries according to European regulations and directives.\u2022 Retrofits and modifications are prohibited\n\u2022 All guarantees/warranties are void in the event of non-intended use.\n\n3 Functional description\nThe FireDETEC\u00ae ILP System for ABC dry chemical powder \u2013 UNECE R107 is a pre-engineered automatic fire fighting \nsystem intended for use on category M2 and M3 vehicles, as detailed by UN Regulation No 107.The FireDETEC\u00ae ILP System for ABC dry chemical powder \u2013 UNECE R107 is abbreviated as \u201cILP \nPowder System\u201d and the extinguishing agent dry chemical ABC powder FUREX ABC 770 as \u201cABC \npowder\u201d in the following chapters of this instruction manual.The ILP Powder System is used to protect the engine compartments and adjacent auxiliary aggregates of category M2 \nand M3 vehicles.The ILP Powder System is made up of 3 basic functional units:\n\n1.The FireDETEC\u00ae pneumatic detection tube is pressurized with dry nitrogen when it is put into service and \nits service pressure is sustained by the pressurized extinguishing agent pressure cylinder.The FireDETEC\u00ae pneumatic detection tube is installed in the hazard area as a continuous linear detector that will rupture \nfrom flame impingement or when the temperature reaches the release point.The ILP Powder System is developed for use in total flooding applications, where \n\u2022 the hazard area is either not normally occupied or the fire fighting system is shut-off in presence of personnel, as in \n\nthe common case of maintenance work being performed.\n\u2022 upon rupture of the FireDETEC\u00ae pneumatic detection tube, the valve of the pressurized extinguishing agent pressure \n\ncylinder opens automatically and the ABC powder is released.\n\u2022 in addition to the automatic actuation via the FireDETEC\u00ae pneumatic detection tube, an additional mode of actuation \n\nvia a manual release device or an electrical actuation via a solenoid valve is also possible.\n\n2.Pressurized extinguishing agent pressure cylinder\nThe ILP Powder System uses a pressurized type B0701 cylinder valve assembly.The ABC powder is pressurized with nitrogen, stored in ILP pressure cylinders made of steel, that are located in a safe \nand accessible location.Mounted upon is an ILP valve, connecting to the FireDETEC\u00ae pneumatic detection tube and the discharge network.The ILP valve is made of nickel plated brass, equipped with a pressure gauge to monitor the ILP Powder System and \nFireDETEC\u00ae pneumatic detection tube pressure, a ball valve that is the interface and isolation shut-off to the FireDETEC\u00ae \npneumatic detection tube.Each ILP valve is equipped with 2 outlet connections (A).Each outlet connection (A) is fitted with a plug that must be \ninstalled in the outlet connection (A) whenever the ILP pressure cylinder is not safely fixed in place, being transported or \nis not in service.Each main \nbranch line is connected to one of the 2 outlet connections (A) of the ILP valve.Upon actuation, the ABC powder is discharged into the area affected by fire through the discharge network.A secondary branch line is usually used to protect adjacent auxiliary aggregates and varies depending on the \napplication.The ILP Powder System combines an environmentally safe extinguishing agent and specially developed components for \na powerful and rapid fire suppression, which reduces property damage to the lowest possible extent.\n\n3.1 Illustrations\n3.1.1 ILP valve\n\nG\n\n1\n\nA\n\nA\n\nC\n\n2\n\n3\n\n4\n\n5\n\nFigure <IP_ADDRESS> B0701 Valve with ball valve\n\n  Legend\n1 Pressure gauge 5 Safety screw\n2 Ball valve A Outlet connection\n3 Safety clip C Inlet connection\n4 FireDETEC\u00ae connection G Dip tube thread\n\nInstruction manual\nFireDETEC\u00ae ILP System for ABC powder - UNECE R107 - B07508\n\nCEODEUX Extinguisher Valves Technology S.A.This is possible within the limitations detailed in chapter 8.FUREX ABC 770 appears as a fine-grained and dusty powder and is off-white to yellowish in color.In addition, it is  \ncoated in silicone to enhance water repellence and flow properties.Unlike water, FUREX ABC 770 does not conduct electricity and is not sensitive to cold temperatures.FUREX ABC 770 is \neasy to store and empty pressure cylinders can be easily refilled.FUREX ABC 770 is propelled from the ILP pressure cylinder by nitrogen (N2).Refer also to the MSDS in chapter 17.1.\n\n5.1  Physical properties of FUREX ABC 770\nActive ingredient Ammonium Dihydrogen Phosphate\n\n91 % \u00b1 3 %\nBulk density 87 \u00b1 5 g / 100 ml\nHumidity < 0,25 %\nColor Off-white / Yellow\nSuitable for Class A, B and C fires\n\n5.2  Extinguishing mechanism\nFire suppression with ABC powder products works by smothering the fire in a layer of inert solid, in case of FUREX ABC \n770 this is Ammonium Dihydrogen Phosphate", "title": null, "url": null, "filepath": null, "chunk_id": "0"}], "intent": "["What is B0701", "Define B0701", "B0701 meaning"]"}}}], "usage": {"prompt_tokens": 6562, "completion_tokens": 154, "total_tokens": 6716}, "system_fingerprint": "fp_8abb16fa4e"}
Headers:
x-envoy-upstream-service-time: REDACTED
apim-request-id: REDACTED
x-ms-client-request-id: 0ee933d6-293c-4319-bcec-b9c6c810dd1e
Strict-Transport-Security: REDACTED
X-Content-Type-Options: REDACTED
x-ms-region: REDACTED
x-ratelimit-remaining-requests: REDACTED
x-ratelimit-remaining-tokens: REDACTED
Date: Wed, 10 Apr 2024 11:56:49 GMT
Content-Length: 17198
Content-Type: application/json; charset=UTF-8
'

Could the Free semantic ranker tier of Azure search be the problem ?


Prestwich/betterer deser

Supersedes #56
Motivation
Json representations are more complex than they need to be. We can simplify them and improve the logic that loads them from environment variables
Solution
Simplify Json representation of signers and connections by eliminating the type: fields and compressing them into single strings where possible.
@arnaud036 after this PR we can dispense with the _TYPE env vars for signers and connections
PR Checklist

[x] Added Tests
[x] Updated Documentation
[x] Updated CHANGELOG.md for the appropriate package

Awesome. I'd like to get sign-off from Arnaud on changes real quick. make sure we don't unintentionally break something


Gogs does not wrap Repository and User Names properly

Gogs version (or commit ref): <IP_ADDRESS>9
Git version: latest
Operating system: Ubuntu 16.04
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist (usually found in log/gogs.log): //

Description
If you go to the Adminstration Panel in Gogs and switch to the User view or the Repostory view, then you can notice that long names without spaces and dashes will not wraped if they are to long.
Because of not wrapping those long names the complete table exiting the right edge.

In my test the Repository is named "testtesttesttesttesttesttesttesttesttesttesttesttest" and causes that the complete layout bugs around.
Maybe the Name schould wrapped with "..." or in two lines.

I think the solution is change CSS word-warp to break-all

Is there any new developments on this issue? I also have the ui problem at version 1.12.7:


support psuedo-column that is an inbound table

This will allow the related table to be displayed as part of the regular columns..
sketch out what need to be done

Reference.columns and Reference.related
If visible-columns annotation is not defined, we should not add any Inbound Foriegn Key to the list of columns. If this annotation is defined and includes inbound foreign keys, we should remove them from the Reference.related. These two attributes will co-exist, but should not include duplicate values.
Inbound Foreign Key Pseudo Column
ReferenceColumn is a class that is used for Reference.columns. It wraps Columns. But in this case what we are wrapping is the related Reference, therefore the ReferenceColumn cannot be reused. I should create a new class that wraps Reference object. It should have the same attributes as ReferenceColumn, which are:

name: Reference's name.
displayname: Reference's displayname.
type: We are using markdown for pseudo-columns. This one should be markdown too.
nullok, default, inputDisabled: These attributes only make sense in entry contexts. Since we are not using this pseudo-column in entry contexts, they are not needed for this class.
formatValue and formatPresentation: Chaise is not directly using these function. They are being used internally by Tuple.values to generate the values. But in this case, we don't have directly access to values. Instead there should be a attribute that returns the Reference. Then chaise has to call read on that and use it as the value.

** I might be able to reuse the ReferenceColumn class, I can decide more about this when I get to actually implementing it`

This was implemented in #438 PR. This functionality has not been implemented in chaise yet.


pre-commit warning

[WARNING] repo https://github.com/gitguardian/ggshield uses deprecated stage names (commit, push) which will be removed in a future version.

as far as I can tell, this is just a doc bug. The full warning is
[WARNING] hook id `ggshield` uses deprecated stage names (commit) which will be removed in a future version.  run: `pre-commit migrate-config` to automatically fix this.

If I run the suggested command pre-commit migrate-config, it makes the following changes:
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -14,10 +14,10 @@ repos:
     hooks:
     -   id: ggshield
         language_version: python3
-        stages: [commit]
+        stages: [pre-commit]
 -   repo: https://github.com/gitguardian/ggshield
     rev: v1.32.0
     hooks:
     -   id: ggshield-push
         language_version: python3
-        stages: [push]
+        stages: [pre-push]

So, the stages just need to be modified in these 2 doc pages:

https://docs.gitguardian.com/ggshield-docs/integrations/git-hooks/pre-commit
https://docs.gitguardian.com/ggshield-docs/integrations/git-hooks/pre-push

Hi, thanks for the suggestion, this change should appear in the public doc soon.

The changes have been deployed.

I believe this should be reopened.
I just received the same warning on a repository using ggshield's pre-commit hook.
The full warning message is:
[WARNING] repo `https://github.com/gitguardian/ggshield` uses deprecated stage names (commit, push) which will be removed in a future version.  Hint: often `pre-commit autoupdate --repo https://github.com/gitguardian/ggshield` will fix this.  if it does not -- consider reporting an issue to that repo.

Running the suggested command doesn't fix the issue.
The stages should be modified in the hooks file so that a very minimal local config doesn't report a warning:
  - repo: https://github.com/gitguardian/ggshield
    rev: v1.34.0
    hooks:
      - id: ggshield


Fortran fix

After updating DYNAMITE to version 3, while running dev_tests/test_nnls.py I got the error that you can see at the bottom of this message. I took a closer look at this error, and I think that I found a way to solve it. Please check it out to see if it works also on other operating systems. I tested it on Ubuntu 22.04 and Fedora 36, and it seems to be working (even though for Fedora I have some other issues, see below). I tested this only by running dev_tests/test_nnls.py for the moment, but I will try to run some more tests with the other tutorial notebooks.
On Fedora however I have the problem that the computation hangs indefinitely at some point: the code gets past the point where I was getting the error before, but it hangs without arriving at the end. I seem to remember that this is a problem that came up at some point for someone else, but I forgot (and cannot find here) what the solution was: any idea?
By comparing the log file with the one from Ubuntu, which gets to the end of the calculation, it seems that on Fedora my run stops before spitting out this line in the log file:
[DEBUG] Jul 25 12:15:35 - dynamite.physical_system.System - physical_system.py:get_component_from_class:162 - Checking for 1 and only 1 component of class <class 'dynamite.physical_system.TriaxialVisibleComponent'>...
Actually, while looking for a clue on how to solve the hanging issue, I took a look at the open issues and saw that @FabioRigamonti had already found a solution for the Fortran problem when running one of the test files. My implementation is different from the one he proposes, but has the same effect (I think!). Closes #162.

[ERROR] 16:00:39 - dynamite.weight_solvers.LegacyWeightSolver - cmd_nnls_5.0 exit code 1. ERROR. Message: At line 168 of file triaxnnls_CRcut.f90 (unit = 27, file = 'datfil/orblib_5.0.dat')
Fortran runtime error: I/O past end of record on unformatted file

Error termination. Backtrace:
#0  0x57756a in ???
#1  0x5777c7 in ???
#2  0x5782bf in ???
#3  0x582ad9 in ???
#4  0x43d3c0 in ???
#5  0x44033c in ???
#6  0x40279c in ???
#7  0x7faaed42954f in ???
#8  0x7faaed429608 in ???
#9  0x402864 in ???
#10  0xffffffffffffffff in ???
Logfile: NGC6278_output/models/orblib_000_000/ml5.00/nnls.log
multiprocess.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/multiprocess-0.70.13-py3.10.egg/multiprocess/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/usr/local/lib/python3.10/site-packages/multiprocess-0.70.13-py3.10.egg/multiprocess/pool.py", line 48, in mapstar
    return list(map(*args))
  File "/usr/local/lib/python3.10/site-packages/dynamite-3.0.0-py3.10.egg/dynamite/model_iterator.py", line 376, in create_and_run_model
    weight_solver = mod.get_weights(orblib)
  File "/usr/local/lib/python3.10/site-packages/dynamite-3.0.0-py3.10.egg/dynamite/model.py", line 666, in get_weights
    weights, chi2_tot, chi2_kin = weight_solver.solve(orblib)
  File "/usr/local/lib/python3.10/site-packages/dynamite-3.0.0-py3.10.egg/dynamite/weight_solvers.py", line 230, in solve
    raise RuntimeError(text)
RuntimeError: cmd_nnls_5.0 exit code 1. ERROR. Message: At line 168 of file triaxnnls_CRcut.f90 (unit = 27, file = 'datfil/orblib_5.0.dat')
Fortran runtime error: I/O past end of record on unformatted file

Error termination. Backtrace:
#0  0x57756a in ???
#1  0x5777c7 in ???
#2  0x5782bf in ???
#3  0x582ad9 in ???
#4  0x43d3c0 in ???
#5  0x44033c in ???
#6  0x40279c in ???
#7  0x7faaed42954f in ???
#8  0x7faaed429608 in ???
#9  0x402864 in ???
#10  0xffffffffffffffff in ???
Logfile: NGC6278_output/models/orblib_000_000/ml5.00/nnls.log
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/alice/dynamite/dev_tests/test_nnls.py", line 141, in <module>
    run_user_test()
  File "/home/alice/dynamite/dev_tests/test_nnls.py", line 62, in run_user_test
    smi = dyn.model_iterator.ModelIterator(c)
  File "/usr/local/lib/python3.10/site-packages/dynamite-3.0.0-py3.10.egg/dynamite/model_iterator.py", line 86, in __init__
    status = model_inner_iterator.run_iteration()
  File "/usr/local/lib/python3.10/site-packages/dynamite-3.0.0-py3.10.egg/dynamite/model_iterator.py", line 515, in run_iteration
    output = p.map(self.create_and_run_model, input_list)
  File "/usr/local/lib/python3.10/site-packages/multiprocess-0.70.13-py3.10.egg/multiprocess/pool.py", line 364, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/usr/local/lib/python3.10/site-packages/multiprocess-0.70.13-py3.10.egg/multiprocess/pool.py", line 771, in get
    raise self._value
RuntimeError: cmd_nnls_5.0 exit code 1. ERROR. Message: At line 168 of file triaxnnls_CRcut.f90 (unit = 27, file = 'datfil/orblib_5.0.dat')
Fortran runtime error: I/O past end of record on unformatted file

Error termination. Backtrace:
#0  0x57756a in ???
#1  0x5777c7 in ???
#2  0x5782bf in ???
#3  0x582ad9 in ???
#4  0x43d3c0 in ???
#5  0x44033c in ???
#6  0x40279c in ???
#7  0x7faaed42954f in ???
#8  0x7faaed429608 in ???
#9  0x402864 in ???
#10  0xffffffffffffffff in ???
Logfile: NGC6278_output/models/orblib_000_000/ml5.00/nnls.log

This works for me, running GCC/gfortran 9.4.0 under Arch Linux. I tried both test_nnls.py and a longer run (~50 models) on an Atlas3D galaxy.

Tested on Mac OS 11.6.7 Big Sur and gcc/gfortran 10.3.0: test_nnls.py and test_all.sh both work ok :-)

I also did some further checks: the tutorial notebooks (in docs/tutorial_notebooks) all work fine on Ubuntu (some problems on Fedora for number 3 and 4, but not related to the orbit calculation, which is totally fine there as well!). Moreover, test_orbit_losvds.py works nicely on both Ubuntu and Fedora! :)

Also works with GCC 12.1.0 under Arch Linux (after I removed the static-libs compiler flags in Makefile.linux, which I think are unnecessary)

Also works with GCC 12.1.0 under Arch Linux (after I removed the static-libs compiler flags in Makefile.linux, which I think are unnecessary)

Interestingly, the same is the case for Mac OS 11.6.7 / gcc 10.3.0: commenting out the static-libs compiler flags in Makefile compiles successfully and test_nnls.py, test_orbit_losvds.py, and test_all.sh execute successfully. Perhaps we can delete the static-libs part from both Makefiles?


Same sample name several times.

I see that although the primary key name changes, it can be a bit confusing if the sample name can appear several times. I would suggest, that the website only allows users to input each sample name one time in this week.

resolved by #27


unable to find lib.exe path  in windows

I am also set the path in directory of roaming/npm/node_modules/rapidgame/libexepath.txt as
C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\bin
but it gives the following error.

Are you sure you got those paths correct? It looks like it finds your MSBuild.exe in the 14.0 folder which is VS 2015. But you have specified the 10.0 folder in the above path which corresponds to VS 2010.
If you have further troubles I recommend taking a look at cocos2d-x new gen-libs. It might have already made RapidGame obsolete.

Yes it is correct.Still it does not take the path.
I see in updates as ,
August 11, 2015: On Windows, the path to MSBuild.exe, Lib.exe and VCTargetsPath can be set manually in case they cannot be automatically located.
How can I set these paths manually.is there any information to that.
Because I already set path for lib.exe in libexepath.text in rapidgame node_module,but not works.

I find the issue,in rapidgame.js file a function for getting libpath as "getLibExePath".In that checking file,but we provide path only as " if (fileExists(libExePath)) " as passed whole file path it is executed successfully.Thats fine,but after completing the project it shows the error as follows,
Build FAILED.
   "C:\Users\johnramesh\AppData\Roaming\npm\node_modules\rapidgame\src\cocos2d-x\build\cocos2d-js-win32.sln" (libcocos2d;libjscocos2d target) (1) ->
   "C:\Users\johnramesh\AppData\Roaming\npm\node_modules\rapidgame\src\cocos2d-x\cocos\2d\libcocos2d.vcxproj" (default target) (2) ->

     C:\Users\johnramesh\AppData\Roaming\npm\node_modules\rapidgame\src\cocos2d-x\cocos\2d\libcocos2d.vcxproj(36,3): error MSB4019: The imported project "C:\Microsoft.Cpp.props" was not found. Confirm that the path in the <Import> declaration is correct, and that the file exists on disk.

0 Warning(s)
1 Error(s)

I found "prebuildWin" function in rapidgame.js file in line no 754 as
// set VCTargetsPath
// (overcome error MSB4019: The imported project "C:\Microsoft.Cpp.Default.props" was not found.)
process.env["VCTargetsPath"] = vcTargetsPath;
is there any issue here or in my System.
Thanks for looking...

Sorry, can you post your code for fixing getLibExePath?

In regards to the props issue, you either need to let it run the complete git diff when downloading cocos2d-x or else manually change your cocos\2d\cocos2dx.props file like this.

I installed VS 2015 community version it solves the problem.
But android prebuild fails,It show the following error
MSBUILD: C:\Program Files (x86)\MSBuild\14.0\bin\amd64\MSBuild.exe
LIB: C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\lib.exe
VCTARGETS: C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V140
Happily prebuilding android
Building Android libraries for all platforms...
Spawning:
make
Current dir:
C:\Program Files\nodejs\node_modules\rapidgame\src\proj.android
./build.sh armeabi Debug
PLATFORM=android-14
ARCH=armeabi
CONFIG=Debug
CORES=4
CC_ROOT=/c/Program Files/nodejs/node_modules/rapidgame/src/cocos2d-x
NDK_MODULE_PATH=/c/Program Files/nodejs/node_modules/rapidgame/src/cocos2d-x:/c/Program Files/nodejs/node_modules/rapidgame/src/cocos2d-x/external:/c/Program Files/nodejs/node_modules/rapidgame/src
NDK_TOOLCHAIN_VERSION=4.9
/c/cygwin64/bin/awk: /c/cygwin64/bin/awk: cannot execute binary file
c:/CoCoS-Devlopment/android-ndk-r10e/build/core/init.mk:373: *** Android NDK: Aborting.    .  Stop.
/usr/bin/which: no Android NDK: Host 'awk' tool is outdated. Please define NDK_HOST_AWK to point to Gawk or Nawk !    ar in (/usr/local/bin:/usr/bin:/cygdrive/c/ProgramData/Oracle/Java/javapath:/cygdrive/c/Windows/system32:/cygdrive/c/Windows:/cygdrive/c/Windows/System32/Wbem:/cygdrive/c/Windows/System32/WindowsPowerShell/v1.0:/cygdrive/c/Program Files (x86)/Microsoft SQL Server/100/Tools/Binn:/cygdrive/c/Program Files/Microsoft SQL Server/100/Tools/Binn:/cygdrive/c/Program Files/Microsoft SQL Server/100/DTS/Binn:/cygdrive/c/Program Files (x86)/Microsoft SQL Server/100/Tools/Binn/VSShell/Common7/IDE:/cygdrive/c/Program Files (x86)/Microsoft SQL Server/100/DTS/Binn:/cygdrive/c/Program Files (x86)/Microsoft Visual Studio 9.0/Common7/IDE/PrivateAssemblies:/cygdrive/c/Program Files (x86)/Microsoft ASP.NET/ASP.NET Web Pages/v1.0:/cygdrive/c/Program Files/Microsoft SQL Server/110/Tools/Binn:/cygdrive/c/Program Files (x86)/Git/cmd:/cygdrive/c/Program Files/Microsoft/Web Platform Installer:/cygdrive/c/Program Files (x86)/GtkSharp/2.12/bin:/cygdrive/c/Program Files (x86)/Skype/Phone:/cygdrive/c/Program Files/Git/cmd:/cygdrive/c/msys/1.0/bin:/cygdrive/c/Program Files/nodejs:/cygdrive/c/Users/johnramesh/.dnx/bin:/cygdrive/c/Program Files/Microsoft DNX/Dnvm:/cygdrive/c/Program Files (x86)/Windows Kits/10/Windows Performance Toolkit:/cygdrive/c/CoCoS-Devlopment/cocos2d-js-v3.6.1/cocos2d-js-v3.6.1/tools/cocos2d-console/bin:/cygdrive/c/Python27:/cygdrive/c/Ruby21/bin:/cygdrive/c/Program Files/Java/jdk1.8.0_60/bin:/cygdrive/c/apache-ant-1.9.4-bin/apache-ant-1.9.4/bin:/cygdrive/c/adt-bundle-windows-x86_64-20140702/sdk/platform-tools:/cygdrive/c/adt-bundle-windows-x86_64-20140702/sdk/tools:/cygdrive/c/adt-bundle-windows-x86_64-20140702/sdk/build-tools/19.0.0:/cygdrive/c/CoCoS-Devlopment/android-ndk-r10e:/cygdrive/c/Users/johnramesh/AppData/Roaming/npm)
Missing the 'ar' NDK build tool. Please install the Android NDK and toolchains.
make: *** [all] Error 1
In stackoverflow thread
http://stackoverflow.com/questions/8384213/android-ndk-revision-7-host-awk-tool-is-outdated-error
I did like the above thread but not worked.
Guide me to further.Thanks

android-ndk-r10e  is the problem.I installed android-ndk-r9d.It works.

2.In cygwin , white spaces in path does not take so I put environment variables for path which contains "Program Files" as "Progra~1", because My Machine is 64-bit version.
3.Prebuild is succeeded.
4.After creating the project,with out any modifications I fired the command "make",build fails.
The Following error came.

I am awaiting for your reply.
Thank you very much.

Sorry man, I really haven't tried the Windows + Android build. Another developer got that all working. Maybe try gen-libs instead of RapidGame?

@johnramesh, I was having some trouble making it work on Windows too, tried the gen-libs and it worked on first. This guy wrote a quick guide on how to make it work for the win32 VS 2015 build, worked for me ;) http://discuss.cocos2d-x.org/t/prebuilt-for-win32-cocos-gen-libs-p-win32/24497.
The bad part, is that it doesn't have a template project (yet), so you have to create a new project, delete the cocos2d folder inside it, and make the necessary changes for each platform specific project for targeting the prebuilt lib.

The new Android prebuild command is greatly simplified in RapidGame 1.0.0. It can even be run with a command prompt instead of cygwin. Hopefully this issue is solved. If not, go ahead and re-open or open a new issue.


Heightmap mesh

Hi Sergi, can you please review the heightmap_mesh branch, and merge it to the master if it is working fine. Thanks

I wouldn't merge this PR because it contains a lot of commits that are not related to the heightmap meshes. I would therefore suggest to close it and opening a new one after instructing @ibrahimhroob how to do it correctly ;)


Support installation mechanism using Kubernete's etcd

We are using Kubeadm to provision cluster. The current installer scripts for Calico creates its own unauthenticated. We would like to use the same etcd used by Kubernetes for Calico. The benfits we see are:

We have one etcd to deal with. We can backup that, etc.
If HA setup (coming in Kubeadm 1.9), we don't have to manually manage a separate etcd.
We provision cluster in DigitalOcean. We need to use authentication with etcd to avoid potential issues. It will get better to just let Kubeadm deal with all that.

My questions are:

Are there any potential issues with using the same etcd as Kubernetes?
Can such installer scripts be added to this repo and add to the doc? We are happy to test/contribute in this regard.

cc: @sanjid133

@tamalsaha I think there are a few things here.
Firstly, Calico certainly supports etcd auth using certificates. See ETCD_KEY_FILE and friends here
This manifest is more generic, and includes etcd auth as well as the ability to specify your own etcd endpoints.
Now, when deploying on kubeadm, the k8s apiserver's etcd will by default be installed for access on the master only. This means that in the default configuration Calico can't use the etcd created by kubeadm.
However, if you deploy your own etcd cluster and point both Calico and Kubernetes at it, it should work fine. Many users do this. The main thing to be aware of is sizing your cluster appropriately to make sure it meets the performance requirements of your cluster.
Hopefully that helps!

@tamalsaha I'm going to close this issue as we have not heard from you for some time. If @caseydavenport 's response does not adequately resolve your issue please reopen.


Create issue template to transfer repo to xarray-contrib org

Make it easier for people to transfer their repository to the xarray-contrib organization via an YAML issue template.
Preview at https://github.com/weiji14/xarray-contrib/blob/issue-template/transfer-repo/.github/ISSUE_TEMPLATE/transfer-repo-to-this-organization.yaml
Fixes #3
References:

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-githubs-form-schema#checkboxes

Ccing @keewis / @jhamman  for any additional feedback


kafka: prefer roundrobin rebalance strategy

By default librdkafka uses the "range" strategy which (from what I understand), given two consumers C1 and C2, and four topic/partitions A0,A1,B0,B1 will assign them as C1:{A0,B0}, C2:{A1,B1}. This is nice when your topics have multiple partitions (the normal case), but OpenBMP uses topics with a single partition to preserve message ordering and so the range strategy with two consumers and two topics A0,B0 will assign them both C1 and leave C2 with nothing (as soon as C1 shuts down then C2 will get the full set of topics).
Instead we want all partitions to be evenly distributed across consumers which is what the round-robin strategy does.
Eventually we'd probably want a cooperative strategy that only changes assignments incrementally (to avoid massive rebalances when there are many consumers), and we want to use the new static consumer groups feature in Kafka 2.3+, but this is good enough for now.

Should fix https://github.com/CAIDA/pybgpstream/issues/7 (finally...)


nimony: implement when

The conditions are evaluated as constant expressions, if they are true or false then only the passing branch is semchecked. However if one of the conditions fail to compile, the entire when statement is left unresolved.
There is an important limitation in that the semchecking of the chosen branch has to be delayed until the body phase of semchecking for it to be possible to compile the condition (in the system module at least). This means symbols declared in when branches can't be used before they're declared. We could maybe relax this if the condition is constant over phases (and we aren't in the system module) i.e. true or defined(abc) but expressions like declared(abc) could go either way.
Test for defined is disabled until untyped args/magics after overloading work.

Fine for now but in principle within a generic you must not evaluate the when statement directly, but after instantiation. Well it is not clear how it interacts with checked generics.


How are registries expected to behave when a subject is deleted?

I see:

When deleting a manifest that has an associated referrers tag schema, clients MAY also delete the referrers tag when it returns a valid image index.

...but I don't see any guidance for registries who support the referrers API.

Are we leaving this up to registry providers discretion?
I believe the justification for not allowing an index to have a subject was due to garbage collection implementations, so it feels incomplete not to have a recommendation for registries. Given that this is a new kind of relationship between artifacts, it would be great to standardize on the behavior rather than let it diverge again.

I'm not a maintainer, but as an interested party, my understanding is that the link between a manifest and the blobs it contains is a "hard" link and the link from things that refer to a manifest is "soft" -- the former is something I as the manifest author control, the latter might have been created by another party who will be surprised if it goes away (even if the thing it points to does).
As a concrete user example, there are some regulated industries I've worked with where the regulators themselves would be very interested in having the ability to keep a copy of all the SBOMs of the images, but don't want to store the images themselves for various reasons (storage, IP, etc).

Like what Tianon said. My feeling is that this is a policy for the registry. Allow the user to configure. I might imagine that some regulation may want signatures/attestations on an object have to stick around for audit sake, even if the object is now on a denylist or deleted due to security or whatever. If they have to pay for the storage, then be up front. They may prefer the option to delete-when-dangling.
They're decoupled.

put another way..
Should client users push artifacts before or after pushing the container images said artifacts refer to?
If we support both there are timing windows associative with the OP question for garbage collection.. and again that would be a registry decision point. I can see pros/cons either way (strict/loose or strict/loose with a time window), and I can see registries choosing one or the other by customer type and use desired use cases.  Maybe someone want's to keep an artifact around for a currently deleted image, where the artifact certifies a prior version of the image still in use on a cloud account in some remote location :-)

At some point in time.. we are going to need to cover gc policies, IMO.


Remove backend settings

I am not sure of your working system. I had several problems with plotting the graph with TkAgg. It seems the problem related to these issues:

https://matplotlib.org/faq/osx_framework.html
https://github.com/matplotlib/matplotlib/issues/12074

Finally, I ran the plotting successfully with Qt5 in a virtual environment. However, I didn't find the solution for the backend MacOSX and TkAgg.

If you use Linux, there may be no problems with backend settings.  In theory all the backends are available.
But for macOS, there is a significant bug that invalidates the macOS default backend.
The reason for this bug is that python is installed as a framework on the macOS system.
In the early stages of the development of this library, in the process of desperate trial and error: I spent most of the night discovering that only the mandatory use of the TkAgg backend would make this library work on macOS.
We need more testing to test whether the back end works reliably on every system.


add support for Windows prefix delegation

Issue #, if available:
Description of changes:

This PR adds support for Windows prefix delegation to increase pod density per node. Instead of assigning secondary IPv4 address, this feature assigns IPv4 prefix to primary ENI on Windows node.
Can enable the feature by adding an entry "enable-windows-prefix-delegation: true" in amazon-vpc-cni config map after enabling IPAM.
Can set either warm-ip-target and/or minimum-ip-target, or warm-prefix-target in amazon-vpc-cni config map   to fine-tune the warm pool configuration used in prefix delegation.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@haouc The PR still shows all the commits from the feature branch, but I would like to squash all commits into one. Should I squash them locally and then force-push to the feature branch, and then submit the PR to merge into master? Please advise how to merge the feature branch. Thanks a lot!


Suggest gems crashes the whole desktop wowsim

I downloaded the desktop version of the program.
When I click on the suggest gems button - does not matter if it's the feral or warrior, the whole program crashes.
Sim Links and Screenshots
Attached below are screenshot of error I get before the whole thing crashes.
Desktop (please complete the following information):

OS: Windows 10
Browser: Firefox, Chrome
Version: v0.1.46

happened to me at random times but only rarely on the previous version. now it happens every time you press the suggest gem button

happening with stat weights too, everytime

cant reproduce this, post a link export -> link otherwise #3663 may fix it but could only get a stat weights crash

Link to sim, but happens in any, no matter what set I'm editing

ok was able to trigger it a few times just spamming the button. crash points to same issue as stat weights. should be fixed with #3663 as well, wasnt able to trigger after, if it still happens after we'll need locks around backend db access

Also happening with calculate stat weights, server just closes immediately no errors


Problem inputs specifications

Hi,
Is there somewhere the input specifications or tests inputs to feed to the submitted solution to run them locally ?

In the dataset go to Project_CodeNet/derived/input_output. The README explains how to get the test inputs, which are in that directory also


As a user, I don't want to see a repository that has empty issues

https://telegram.me/teknologi_umum_v2/110543
If the repository doesn't have issues or doesn't have the label hacktoberfest just hide it

hello, can you assign to me please?


KeyError related to holdout indexing in modelhub._fit_model()

If modelhub._fit_model is called without passing a data frame, the data frame is loaded from from the input path and then filtered to remove holdout data. However, the line to filter the data frame is written incorrectly and throws a KeyError. Compare the line in _fit_model to the line in _run_model.
Additionally, if a data frame is not passed, a str col_holdout should be passed, because model.specs.col_holdout is supposed to be a tuple of str and not a str (though there isn't a check when creating an instance of modelhub), so it is not clear which holdout set to filter out. Alternatively, you could just make the data frame a required argument. It only gets called from modelhub._run_model, which does pass a data frame).

Currently when we fit a model we will always pass in a dataframe
https://github.com/ihmeuw-msca/modrover/blob/main/src/modrover/learner.py#L265
And the holdouts are used outside of the _fit function to subset the training data
https://github.com/ihmeuw-msca/modrover/blob/main/src/modrover/learner.py#L128-L136


BHPhoto Error Message

Hi. I am using the Discord webhook method and when I start the script and look at the logs, the same error message keeps looping for all of the out of stock products:
caught exception during request: got response with status code 429 for <product link>
followed by:
scrape failed
Some lines also say:
caught exception during request: HTTPSConnectionPool(host='www.bhphotovideo.com', port=443): Read timed out. (read timeout=1)
Is this error occurring because the website is detecting the bot, or is it something else? I am using the config file bhphoto_rtx_3080.yaml

That 429 error code means you're making too many requests: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/429.
If you're using a 2 second interval time I would recommend upping it to 4 seconds to avoid being detected.

I just committed a change to increase the read timeout on faster refresh intervals: https://github.com/EricJMarti/inventory-hunter/commit/654af6f881c6bfbebf4d439de71111cd89cf21ef
Please pull the latest image using:
$ docker pull ericjmarti/inventory-hunter:latest


question about update query frequency in Interactive workload

This is a handsome graph database benchmark and it is useful in many ways.
I am testing Interactive workload,  how can I do to make it to be a write-heavy workload to test writing ability of neo4j? can I increase the update query frequency? but I see the Table B.1 in ldbc-snb-specification document says that just Q1 to Q14 are mixed,  the update queries did not mention.

@xspeng95 thanks. You can increase the frequency of the update queries by reducing the TCR (Time Compression Ratio). I've recently updated the specification to cover this, see e.g. Section <IP_ADDRESS> "Measurement window".

Yeah!!! I have reduced the TCR and the frequency of the update queries increased up to 30%.
and I have another question bothered me for a long time, the thread_count  maybe not works when I set thread_count=16 to run ./interactive-create-validation-parameters.sh. it spent one day to generate database validation parameters when I set SF=10 and only one CPU work actually.
Can this command be executed in multiple threads?
Machine: Intel(R) Xeon(R) Silver 4110 CPU @ 2.10GHz with 32 CPUs

Hi @xspeng95, creating validation parameters is intended to be executed on a single thread to ensure determinism. We are aware that it often takes a long time with the current reference implementations but we expect this to improve in the coming years (as more and more vendors pick up the LDBC benchmarks and optimize their databases).

@szarnyasg Thank you very much. In fact, more and more people in Industry and academia choose the LDBC benchmarks indeed !!

We hope so. Best of luck with your experiments and feel free to open new issues if you have questions.


ACC-260.2 (Level AA)Kyma-dashboard_Edit Field Suggestion pop up is away from the edit field.

1.1 Internal Message Details
Internal Message:
Test Plan:	ACC SAP Kyma runtime 2024
Message Priority:	Medium
Affected Check Point:	ACC-260.2
Affected UI:	Edit Field
Short text:	ACC-260.2 (Level AA)Kyma-dashboard_Edit Field Suggestion pop up is away from the edit field.
1.2 Application Details
Application Name	Kyma-dashboard
Application URL:	https://canary.cockpit.btp.int.sap/cockpit#/globalaccount/7304343e-1ce0-412b-a955-5948bab9d2c8/subaccount/e0ec1134-2770-40a2-910b-2b8e32ec8f60/subaccountoverview
User:	SSO
Technology:	SAP UI5
UI5 Version:	1.124.1
1.3 Test Environment Details:
GitHub	https://github.com/kyma-project/busola/issues
Web browser:	Google Chrome - Version 127.0.6533.73 (Official Build) (64-bit)
Operating System:	Windows 11 Enterprise Version 22H2
Screen Reader:	JAWS 2024.2405.83 ILM 64 bit for Windows
2. Screen Flow
•	Login into the application using the URL provided.
•	URL: https://canary.cockpit.btp.int.sap/cockpit#/globalaccount/7304343e-1ce0-412b-a955-5948bab9d2c8/subaccount/e0ec1134-2770-40a2-910b-2b8e32ec8f60/subaccountoverview.
•	Activate ‘Console URL:Link to dashboard’ in overview page.
•	Activate ‘Extension’ item.
•	Activate ‘Create’ button.
Settings for Text resize:
•	Open developer tool by pressing F12.
•	Activate customized and control dev tools ().
•	Activate settings Activate Devices.
•	Add custom device.
•	Set it to 1280 * 1024 and save.
•	Resize the browser into 200%.
3. Issue
Synopsis: Suggestion pop up is away from the edit field.
Observed Behavior:
Suggestion pop up is away from the ‘Resource’ edit field within ‘Extension’ page.
Expected Behavior:
On resizing the text to 200%, all UI elements need to be fully recognizable and operable.
The screen layout must be scaled so that there is no loss either content or function when text is resized to 200%.

Kindly refer to attached error and issue processing document for more details.
Regards,
Huda

ACC-260.2 (Level AA)Kyma-dashboard_Edit Field Suggestion pop up is away from the edit field.docx
Issue processing-09.2021 (1).docx


Login des

@CodiumAI-Agent /review

PR Review

 ⏱ Estimated effort to review [1-5]
3, because the PR includes multiple files with significant changes, including CSS for styling, JavaScript for functionality, and integration with Auth0 for authentication. The complexity lies in understanding the logic for image carousel in the login component, the integration with Auth0, and ensuring the CSS changes achieve the desired layout without affecting other components.

 🧪 Relevant tests
No

 🔍 Possible issues

Possible Bug: The CarouselData array in Carouseldata.jsx may contain items without a title property, which could lead to displaying undefined on the UI.

Code Duplication: The Signup component includes a button for signing up with Google using Auth0, but there's no equivalent option in the Login component. This inconsistency might confuse users.
 🔒 Security concerns
No

 Code feedback:
relevant fileClient/Client/src/components/Login.module.css
suggestion

Consider using responsive units (e.g., em, rem, %, vw) for widths, heights, and font sizes to improve the responsiveness of the login page across different screen sizes. [important]

relevant linewidth: 400px;
relevant fileClient/Client/data/Carouseldata.jsx
suggestion

Ensure all items in the CarouselData array have a title property to prevent displaying undefined for images without titles. If a title is intentionally omitted, handle this case in the UI logic. [important]

relevant lineimg : LE
relevant fileClient/Client/src/components/Login.jsx
suggestion

Add a mechanism to prevent rapid, repeated clicks on the forward and backward buttons in the image carousel, which could lead to a poor user experience or UI glitches. Throttling or debouncing the click events could be effective. [medium]

relevant line
relevant fileClient/Client/src/components/Signup.jsx
suggestion

Consider adding a loading state to the "Signup using google" button to provide feedback to the user during the authentication process, improving the user experience. [medium]

relevant line loginWithRedirect()}>Signup using google

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.
The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.


SELECT statement increasing uncommitted transactions

Dbeaver version:  7.0 Windows 64
Database: Oracle

ROLLBACK increasing uncommitted transactions too

thanks for the bug report

It is not a bug. Once you are in transaction mode every SQL query is transactional. SELECT query locks resources/tables/rows. That's why we show it there.
You can use Smart-commit mode. In this case dbeaver will switch back to auto-commit after transacton end.

@serge-rider with smart-commit i should the get the same feature as before right? (eg show uncomitted transactions) and it seems that its not working, can anybody confirm this? that is a big feature for me ;)

@ap-pauloafonso please check https://github.com/dbeaver/dbeaver/wiki/Auto-and-Manual-Commit-Modes

@serge-rider i still dont know why i'm getting empty transaction log screen ,even with pending transactions ;(

I think this is a regression in 7.0...
Some releases ago you modified the behavior so that commit and rollback were always enabled in manual commit mode, which is fine. However, select statements did not increase transaction count. 7.0 broke this, and now select is increasing count again.

Here is the relevant decision https://github.com/dbeaver/dbeaver/issues/6618#issuecomment-526191526

fixed in scope of #8416


Remove support for configuration resolution across projects

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@melix This PR now has a green build.
Is the reworked error message OK?

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

Closing as making this an error is postponed to Gradle 8


404 page

I'd like to have a 404 page that will point to the old wiki when a page isn't found. I have a static version of the old wiki downloaded (see README for how). We would like to add a link that depends on the link that was "wrong", but I'm not sure if that's possible.

Fixed by #56


Enable zip dependencies when using the Python requirements plugin

Hey, it is me again,
I ran into an issue while using zipping my dependencies using the python requirements plugin, as per their documentation we need to import a file created during the deploy process so we can use the zipped dependencies.
After getting tired of manually updating the wsgy.py on each deploy I decided to update my personal fork with the automated "solution" and open this PR so other people can benefit from that.
Cheers,
EM

Coverage increased (+0.03%) to 98.054% when pulling 94a51ecfd8c828812fc1f3b0b4e2879f692aa496 on ericovis:master into<PHONE_NUMBER>b9c9ad879a5112808f82c772e04117 on logandk:master.

Coverage increased (+0.03%) to 98.054% when pulling 94a51ecfd8c828812fc1f3b0b4e2879f692aa496 on ericovis:master into<PHONE_NUMBER>b9c9ad879a5112808f82c772e04117 on logandk:master.

Coverage increased (+0.03%) to 98.054% when pulling 94a51ecfd8c828812fc1f3b0b4e2879f692aa496 on ericovis:master into<PHONE_NUMBER>b9c9ad879a5112808f82c772e04117 on logandk:master.

Coverage increased (+0.03%) to 98.054% when pulling 94a51ecfd8c828812fc1f3b0b4e2879f692aa496 on ericovis:master into<PHONE_NUMBER>b9c9ad879a5112808f82c772e04117 on logandk:master.

Hi Eric,
Thanks for the PR! This looks very useful, I'll merge this and push a new release soon.


wip: Drop "invalid cast" error down to # typed: true

Motivation

Want to see how much havoc this causes.
Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_OtRErqhdSg7feH
→ https://go/builds/bui_OtREAr5C4qHNCb
→ https://go/builds/bui_OtREcaoqnCgvN9


Amazon API security with API Keys

I have deployed my rest API on amazon API gateway and I have a scenario in front of me with security concern. I am using an api key for all the api requests, I wanna know if the that api key is exposed somehow and as we know the same api key is being used by already published apps...Then what are my options?
Also as mentioned here I can have only 10000 API keys per AWS account if I want the api keys to be unique per user for it to be more secure but what if the number of user shoots out to be more than 10000.

I am using javascript sdk in mobile app development.

We do not recommend using API keys for security. API keys are used primarily to meter API access. You should rely on some form of stronger authentication such as Sigv4 or an Oauth flow with custom authorizers. The pet store example uses Sigv4.

Hi
I have api gateway /auth endpoint which talks to my lamda function written in java which generates  temporary credentials .
My question is that during Developement i use profile credential provider, so if depolyed what credential provider lamda function will use?how do i set env variable / property file or .aws file to provide credentials?
Thanks in advance
sirfak

The "profile" for the Lambda function is based on the execution role you've configured for the function itself. From the Lambda console you can see the name of the execution role, use the IAM console to modify the access policy for the role and configure which services the Lambda function is allowed to communicate with

If my java lamda function has code for eg
AWSSecurityTokenServiceClient sts_client = new AWSSecurityTokenServiceClient();
Which is suppose to read credendtials from profile/environment/property as  required in the credential provider chain during local developement(sirfak profile user for eg)
If i have understood your point, the api key and access key during lamda execution will be read from the execution role defined in lambda console and not for sirak user profile (the dev user) ?

correct

#23

#20


stat: Fix distmv to allow zero-matrices

For example in Normal.CovarianceMatrix

I'm repurposing this issue, because I think this is what I meant at that point and then forgot.
For []float64 methods, we typically have the receiver as the first argument, allow it to be nil, and allocate and return otherwise.
In the suite, in some places we do the same for matrix receivers, stat.CovarianceMatrix being another example. There is no need to do this. The convention is that zero-sized matrices get automatically adjusted in size to the right thing. Instead, the user should just allocate a zero-value and use it
var cov mat.SymDense
stat.CoviarianceMatrix(&cov, data, weights)

This is roughly as easy as the current cov := stat.CovarianceMatrix(nil, data, weights), but has a simpler signature.

SGTM


Publish to METADATA

Before this happens, there should be a discussion with https://github.com/zenna/AbstractDomains.jl about unifying, to avoid multiple packages that do the same thing.

It's fine as it is. Thanks!

I learned recently that following semver usually means starting at v0.1. https://semver.org

The Julia convention is to use 0.0.x to mean experimental (according to Stefan on Discourse the other day)


Divide operation problem

Hi, I was using this library for my project, and found out that divide operation is not supported in this library.
I thought it doesn't make sense since Element_div method is clearly defined in pbc.c.
So I double checked the source code over and over, and found this.
https://github.com/debatem1/pypbc/blob/c518d6ff074f538080c5794924f83ee441ed44ea/pypbc.c#L1338-L1341
In here, the function position is shifted- so Element_div is actually pointing nb_inplace_floor_divide (ifloordiv), not nb_true_divide.
I think the 0 in line 1338 was supposed to be a blank space between nb_inside_or and nb_floor_divide.
After I erased the 0 in line 1338 and installed the library once again, the divide operation works.
Hope this helps if you were using this library and have to use divide operation.

OHHHHHH，I really really want to give you a huge hug. This helps me a lot, i almost give up programming on python!

OHHHHHH，I really really want to give you a huge hug. This helps me a lot, i almost give up programming on python!


Update qr code

Build failed because of:
can not read a block mapping entry; a multiline key may not be an implicit key at line 1486, column 17:
                name: TarsJavaStart
                    ^


Listening on port 9000...

My Config.cson shows:
"php-debug":
{
ServerPort: 8080
}
Not 9000
Does this mean the Package is failing to pickup a None Default port?
Or is it not reading my settings?
Or can i simply not use a different port of than 9000
Confused, I use a PHPed, debugger works fine on port 8080
and all application have worked for years using this same setup.
Atom just does not want to play ball

I am unable to reproduce this, are you still experiencing this issue?

i actually give up in the end.
I completely forgot i was trying to use Debugger in Atom.
Let me update everything and revisit this.

I have the same issue, I know it's listening on the port because I checked to see if it is open.

Can you let me know what OS your're on? My guesses are that this is possibly a windows permission/firewall rule and Atom is not being allowed to bind to that port. A quick way to check would be to try running atom as an Administrator.

Nevermind I actually figured it out...My "php-debug" config wasn't tabbed under "*", it was just floating there. My guess is that's why it didn't get loaded

If port binding fails the package throws an exception, and it does that at 2 or 3 places. If you read OP's post carefully you'll realize that it's a matter of selecting the port to be listened on as opposed to receiving connections. The whole purpose of this test was to check if my config was loaded properly. I eventually figured out that my PathMaps also weren't loaded which prevented the package from functioning at all. This can be solved with a simple test.
I've written code that fixes this but the lack of activity on this repo discouraged me from contributing.
I also emailed you about this. (using the email visible on your profile)

Please feel free to submit pull requests.
This project is definitely not dead or unmaintained at this point. The unfortunate reality is that this and several other projects I maintain are done in my free time apart from my full time job, which occasionally results in larges gaps in me being able to contribute and answer questions. So pull requests are definitely appreciated, even if they can't be reviewed asap.
I'm going to close this issue for now, and I look forward to a pull request from you :)


fix: detect jest dependency when react-scripts is installed

related to https://github.com/nvim-neotest/neotest-jest/issues/60#issuecomment-1522414215

I patched locally and it worked for me!

After patching works for me too. Have spent so much time trying to figureout why I always getting - No test found in my project. Thank you.

This has been replaced by https://github.com/nvim-neotest/neotest-jest/pull/64


Use Paperclip to read the Paperclip attachments

We can't rely on the attachment path to be something we can just
File.read because it may not be a local file which can be opened.
Instead we can rely on Paperclip to give us an IO adapter which
is capable of reading an attachment independent from the backend.

Sadly, the specs are failing because our Circle CI orbs are creating docker containers without specifying the text encoding, so it's defaulting to ASCII-8BIT. The fixtures in this repository contain UTF-8, so when we use the correct Paperclip IO adapter (instead of just doing File.read) it will attempt to do the conversion between those encodings. This conversion fails because there are some characters with accents and non breaking spaces that aren't valid ASCII.
@benjaminwil is writing a ticket to fix this in the orb, rather than here.

Just as an update: the orb was not actually the issue, and no code was changed in the CircleCI orbs as a result of this issue.


Project maintenance

I just wanted to reach out and check if this project is still being (or will be) maintained.
I quite like it, but have noticed a few issues lately with compatibility of dependencies. The biggest being that it seems this library is incompatible with mobx 6. Also it seems the library only supports the deprecated version of the AsyncStorage library in React Native (https://reactnative.dev/docs/asyncstorage).
Thanks for all your work here!

This project has no update plan for the time being, I am very sorry.
The reason is that TC39 updated the decorators proposal again, and MobX deprecated the decorators API in v6.
It's waiting for the decorators proposal to stabilize enough.

Thanks for the update and all your work on this project! I’m sorry to hear it won’t be updated. I don’t use decorators currently so I may attempt to fork and remove that to get support for mobx 6.


UMD wrapper for twine

import Twine from './twine';

❤ enough said
@Shopify/tnt

Minor comments, otherwise looks awesome ❤

Good to 🐑  ?

Ok, ran make .uglify too and added the minified version in this PR

Nice. 👍

@owilliams Updated

👍

👏


[BUG] path_merge_collinear(path, closed = true) will not remove a collinear point if it is the first point in a path

Describe the bug
path_merge_collinear(path, closed = true) will not remove a collinear point if it is the starting point of the path, but it can be forced to remove the collinear point if the input path is reversed.
Code To Reproduce Bug
include <BOSL2/std.scad>

examplePath = [[10,0],[10,-10],[-10,-10],[-10, 10],[10, 10]];

// Current Merge Function Behavior

forwardPathMergeCollinearResult = path_merge_collinear(examplePath, closed = true);

backwardsPathMergeCollinearResult = path_merge_collinear(reverse(examplePath), closed = true);

xdistribute(spacing = 30) {
    color("red")
    VisualizePath(examplePath);

    color("green")
    VisualizePath(forwardPathMergeCollinearResult);

    color("blue")
    VisualizePath(backwardsPathMergeCollinearResult);
}

// Personal Solution Implementing Expected Behavior

customMergeCollinearFunction = ExhaustiveClosedPathMergeCollinear(examplePath);

backwardsCustomMergeCollinearFunction = ExhaustiveClosedPathMergeCollinear(examplePath);

fwd(30)
xdistribute(spacing = 30) {
    color("magenta")
    VisualizePath(customMergeCollinearFunction);

    color("cyan")
    VisualizePath(
    backwardsCustomMergeCollinearFunction);
}

// To hide file paths in console
echo("\n\n\n\n\n\n\n");

// ===== Helper Modules and Functions =====
// I took this directly from the pairs documentation
module VisualizePath(path) {
    for (p = pair(path, wrap=true))
        stroke(p, endcap2="arrow2");
}

// Functions I wrote
/*
    Performs the not operation on every element in a boolean list

    list : boolean list to invert
*/
function NotBooleanList(list) = [for (i = list) !i];
//

/*
    Extends the functionality of path_merge_collinear to verify that absolutely all
    co-linear points are removed (path_merge_collinear will not the start point of a closed
    path, even if it is collinear).

    path : path to perform collinearity check on
*/
function ExhaustiveClosedPathMergeCollinear(path) =
    let(
        allPointTriplets = [for (p = triplet(path, wrap=true)) p],

        collinearityBitMask = [for (triplet = allPointTriplets)
            if(is_collinear(triplet)) 1 else 0],

    collinearPath = bselect(path, NotBooleanList(collinearityBitMask))
    ) collinearPath;
//

Expected behavior
I don't know if this is a bug or intentional. I expected the function to remove all collinear points regardless of where they are in the path.
Screenshots

Additional context
Add any other context about the problem here.

OpenSCAD Version: OpenSCAD version 2023.12.09 (git 3d094bf87)
Other libraries used: None

That looks like a bug.  Does this work in all the cases?
function path_merge_collinear(path, closed, eps=EPSILON) =
    is_1region(path) ? path_merge_collinear(path[0], default(closed,true), eps) :
    let(closed=default(closed,false))
    assert(is_bool(closed))
    assert( is_path(path), "Invalid path in path_merge_collinear." )
    assert( is_undef(eps) || (is_finite(eps) && (eps>=0) ), "Invalid tolerance." )
    len(path)<=2 ? path :
    [
      if(!closed) path[0],
      for(triple=triplet(path,wrap=closed))
        if (!is_collinear(triple,eps=eps)) triple[1],
      if(!closed) last(path)

Note, your use of triplet was better than existing code, but you've overcomplicated things in various other ways.

That does appear to work in all the cases.
Testing Proposed Patch
include <BOSL2/std.scad>

examplePath = [[10,0],[10,-10],[-10,-10],[-10, 10],[10, 10]];

// Patched path_merge_collinear(path, closed = true) Behavior

forwardPathMergeCollinearResult = path_merge_collinear(examplePath, closed = true);

backwardsPathMergeCollinearResult = path_merge_collinear(reverse(examplePath), closed = true);

xdistribute(spacing = 30) {
    color("red")
    VisualizePath(examplePath);

    color("green")
    VisualizePath(forwardPathMergeCollinearResult);

    color("blue")
    VisualizePath(backwardsPathMergeCollinearResult);
}

// To hide file paths in console
echo("\n\n\n\n\n\n\n");

// Proposed patch
function path_merge_collinear(path, closed, eps=EPSILON) =
    is_1region(path) ? path_merge_collinear(path[0], default(closed,true), eps) :
    let(closed=default(closed,false))
    assert(is_bool(closed))
    assert( is_path(path), "Invalid path in path_merge_collinear." )
    assert( is_undef(eps) || (is_finite(eps) && (eps>=0) ), "Invalid tolerance." )
    len(path)<=2 ? path :
    [
      if(!closed) path[0],
      for(triple=triplet(path,wrap=closed))
        if (!is_collinear(triple,eps=eps)) triple[1],
      if(!closed) last(path)
  ];

// ===== Helper Modules and Functions =====
// I took this directly from the pairs documentation
module VisualizePath(path) {
    for (p = pair(path, wrap=true))
        stroke(p, endcap2="arrow2");
}

Screenshots

I'm still working on getting more proficient in OpenSCAD and with BOSL2, so the compliment and advice are appreciated.


Add support for showing view definition in BigQuery

# List information_schema.views
presto:default> select * from bigquery.information_schema.views;
 table_catalog | table_schema | table_name |                      view_definition
---------------+--------------+------------+-----------------------------------------------------------
 bigquery      | ebyhr        | test_view  | select 1 as c1, KEYS.NEW_KEYSET('AEAD_AES_GCM_256') as c2
(1 row)

# SHOW CREATE VIEW
presto:default> show create view bigquery.ebyhr.test_view;
               Create View
------------------------------------------
 CREATE VIEW bigquery.ebyhr.test_view AS
 SELECT
   1 c1
 , KEYS.NEW_KEYSET('AEAD_AES_GCM_256') c2
(1 row)

presto:default> show create view bigquery.ebyhr.test_view;
               Create View
------------------------------------------
 CREATE VIEW bigquery.ebyhr.test_view AS
 SELECT
   1 c1
 , KEYS.NEW_KEYSET('AEAD_AES_GCM_256') c2
(1 row)

SHOW CREATE VIEW output should be usable to re-create a view.
Currently the connector does not supports creating views.
However, when we add create view support, this won't work, as the view uses
BQ syntax.
I would expose this information via a something like $big_query_views
system table. @electrum?

@findepi Changed to $big_query_views system table.


Add people endpoint

People endpoint!
I added support for the people endpoints. All HTTP Get requests for the people TMDB endpoint are supported.
Additions

New types for people

People endpoint added to the TMDB class

All GET endpoints with /person: details, changes, movie_credits, tv_credits, combined_credits, external_ids, images, tagged_images, translations, latest, popular

I think since this introduces some changes to the types it may break some of the current implementations. I would be down to bump this to a major version to reduce that.

I am trying to add the currently missing endpoints in the near future. The old ones should be tested if they are still up to date.

I am trying to add the currently missing endpoints in the near future. The old ones should be tested if they are still up to date.

Yup understand that but the new addition on PaginatedResult type would cause a breaking change for anyone pulling in the latest minor/patch version. If you want to, you can add the people endpoint and in a subsequent request update the types.

Ok I am going to rewrite the endpoint to use the basic people endpoint and make a seperate pull request with the PaginatedResult

Ok I am going to rewrite the endpoint to use the basic people endpoint and make a seperate pull request with the PaginatedResult

Ok I am going to rewrite the endpoint to use the basic people endpoint and make a seperate pull request with the PaginatedResult

Sounds like a plan 😂


Update wide fibonacci to 256 columns

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#575  👈
#569
#568
#567
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @shaharsamocha7 and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.08%. Comparing base (5f9adce) to head (58ef499).

Additional details and impacted files
@@                    Coverage Diff                    @@
##           04-04-Remove_Mask_struct     #575   +/-   ##
=========================================================
  Coverage                     94.08%   94.08%
=========================================================
  Files                            66       66
  Lines                          8348     8348
  Branches                       8348     8348
=========================================================
  Hits                           7854     7854
  Misses                          432      432
  Partials                         62       62

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Merge activity

Apr 15, 6:45 AM EDT: @shaharsamocha7 started a stack merge that includes this pull request via Graphite.


Li2023 IMF

Suggest adding the metallicity dependent IMF from Li + 2023.

Actually, they don't seem to give a functional form or a table of values for the slopes so I'm not going to worry about it


Unserialize throws error

I was going to try out to parse a real-world example of a serialized object. However, I got the following output:
> php.unserialize
[Function]
> php.unserialize( 'a:2:{i:0;a:2:{i:0;a:3:{i:0;O:8:"CUrlRule":16:{s:9:"urlSuffix";N;s:13:"caseSensitive";N;s:13:"defaultParams";a:0:{}s:10:"matchValue";N;s:4:"verb";N;s:11:"parsingOnly";b:0;s:5:"route";s:17:"<controller>/view";s:10:"references";a:1:{s:10:"controller";s:12:"<controller>";}s:12:"routePattern";s:30:"/^(?P<controller>\w+)\/view$/u";s:7:"pattern";s:39:"/^(?P<controller>\w+)\/(?P<id>\d+)\/$/u";s:8:"template";s:17:"<controller>/<id>";s:6:"params";a:1:{s:2:"id";s:3:"\d+";}s:6:"append";b:0;s:11:"hasHostInfo";b:0;s:14:"}' );
TypeError: undefined is not a function
    at error (/Users/Ingwie/Work/BIRD3/node_modules/phpjs/build/npm.js:8548:13)
    at _unserialize (/Users/Ingwie/Work/BIRD3/node_modules/phpjs/build/npm.js:8676:11)
    at _unserialize (/Users/Ingwie/Work/BIRD3/node_modules/phpjs/build/npm.js:8655:22)
    at _unserialize (/Users/Ingwie/Work/BIRD3/node_modules/phpjs/build/npm.js:8655:22)
    at _unserialize (/Users/Ingwie/Work/BIRD3/node_modules/phpjs/build/npm.js:8655:22)
    at Object.exports.unserialize (/Users/Ingwie/Work/BIRD3/node_modules/phpjs/build/npm.js:8682:12)
    at repl:1:5
    at REPLServer.self.eval (repl.js:110:21)
    at repl.js:249:20
    at REPLServer.self.eval (repl.js:122:7)

Why am I getting that?

Thanks for reporting, and sorry for the long wait. This project and my motivation were in a bad spot. I recently did a lot of work to breathe new life into it (http://locutus.io/2016/05/announcing-locutus/).
It seems our unserialize function needs some love. @kukawski I noticed you self-assigned this one, did you have plans to attack it?

@kvz yes, I plan to work on it. Currently I'm on strtotime rewrite, but I will probably change focus to this one as this problem is simpler to fix.

Awesome 💯

I don't get the same error message about trying to call undefined as function with latest locutus release. The only problem with the example call is, that it has a serialized object ("O" type) which is currently unhandled. I will look at past commits to understand why we don't support this type and eventually try to implement it.

The problem with "O" type is, that it is followed by class name. The name may also include namespace. PHP can serialize and deserialize private and protected object properties, which then get special markup included in the serialized string. JS doesn't have private and protected properties nor real namespaces, thus implementing "O" is only partially possible. Just throwing error on "O" with the message we currently have is IMHO a better way than do lots of logic to be able to handle the type only partially.
Anyway, I can't reproduce the original issue, so I will close the ticket. If the issue still exists, please reopen and include a new example for reproduction.


Phone Volume sidebar

Steps to reproduce
When pressing the physical volume up or down
-While media(anime) is playing on- the actual phone volume sidebar shows up next to the player volume sidebar and this doesn't happen if you pause the media ,it has to be playing when you do that
Expected behavior
it shouldn't shows up
Actual behavior

Crash logs
No response
Aniyomi version
App version: <IP_ADDRESS>-7539 (standard, c91fb86e4, 123, 2024-05-10T10:30Z)
Android version: 13 (SDK 33; build RMX3630_11_C.24)
Android build ID: RMX3630_11_C.24
Device brand: realme
Device manufacturer: realme
Device name: RE8DDCL1 (RMX3630)
Device model: RMX3630
WebView: Android System WebView 124.0.6367.123
MPV version: c6ccabe
Libplacebo version: v7.349.0
FFmpeg version: n6.1.1
Android version
13
Device
Realme 10
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] If this is an issue with an official manga extension and this issue can be replicated in the Tachiyomi app, that I should be opening an issue in Tachiyomi's extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

type the exact version instead of latest beta or any similar prompts

type the exact version instead of latest beta or any similar prompts

Updated


version check makes it impossible to use the action with a certain commit

Kinda a follow-up to #112, I wanted to test the action and pin it to a commit. But it seems like the option is to run your actions with ...@vVERSION or ...@master. Neither might be desirable to test unreleased code.
The offending lines are here:
https://github.com/ariga/atlas-action/blob/master/shim/index.js#L18-L28
Is there a good reason to bail there?

Looks like the action version is used for the download, so I see why. It's just a bit unconventional to not allow this. Especially since many people prefer the commit vs a fluent version like v1, let alone master.

Because this action is written in Go, the option to checkout and compile at runtime was just too slow that we didn't want to ship it.
Instead, we build the binary for master and for tags and use the tag to decide what to only download the prebuilt binary.

Would it be possible to add a input variable to the action to specify which version of the go binary to use? I'd like to lock the github action to a specific SHA commit hash and I am hitting this issue.

Hey @sdemjanenko
Thanks for reaching out. If I understand correctly, you want to use a specific SHA commit id for the version when using it, i.e
- uses: ariga/atlas/action/migrate-push@abc123

Instead of using a specific tag:
- uses<EMAIL_ADDRESS>
Since the action shim, looks at version tags to decide what version of the atlas-action binary to download, that is failing.
Did I get that right?

@rotemtam yes that is correct. I wonder if something like
- uses: ariga/atlas/action/migrate-push@abc123
   with:
     version: v1

could be a solution.

I understand the need and the suggestion, but I don't love having two different ways of achieving the same thing.
Why is pinning the action version tag an issue?

For me, I'd like to have an option when I update to address maybe necessary changes vs getting failed builds etc. when I least expect or need it.
Same with all other software.

@till , I understand that, but you can pin a specific tag such as v1.2.3 which is bound to a specific build.

@rotemtam pinning the action is a security best practice. Here is a recent article talking about how github actions aren't pinned enough: https://devops.com/report-surfaces-thousands-of-potential-vulnerabilities-in-github-workflows/amp/. Also here is a tool that I am using to automatically flag if my github actions have this problem: https://github.com/zgosalvez/github-actions-ensure-sha-pinned-actions

@till , I understand that, but you can pin a specific tag such as v1.2.3 which is bound to a specific build.

Sorry for the late reply.
For me that includes the action itself. Rather have dependabot send me a PR, etc..


Errors when 'Allow Hyphens in Short URLs' is disabled

Reproducible bug summary

enable Allow Hyphens in Short URLs plugin
create short URL containing hyphen(s)
e.g. sho.rt/a-b
deactivate plugin;

hyphens not shown in Short URL column in admin/index.php
i.e. sho.rt/ab shown where sho.rt/a-b used to be
neither sho.rt/a-b nor sho.rt/ab successfully redirect to target
trying to delete the link results in the error
something wrong happened while deleting :/

Technical details

YOURLS version: 1.7.2
PHP version: 7.1.8
Ubuntu 16.04.3 LTS w/ Apache 2.4.27

You're surprised that deactivating the Allow Hyphens in Short URLs plugin breaks short URLS containing hyphens?

@dgw not surprised at all but thought that it might be confusing for other users 🤷♂

I agree there's something to be improved here: at least you should be able to delete such a link or do something with it. I'll look into things.

I have experimented with so many non-default characters, I am now wondering what I would do when a major new update is released!

@Sushubh - Wouldn't it be better to make a copy of the Allow Hyphens in Short URLs plugin?  Then adjust it to make your own plugin for your "many non-default characters."  In your config file add a Constant with a list of "non-default characters." separated by a specified standard character from the 36 base.  For example, using the number "8" as a separator:
define( 'NON_DEFAULT_CHARACTERS', '-8~8!8@8#8$8%8^8&8*8(8)8_8+' );
Note: I am not saying any of those "non-default characters" would actually work!
Then, within a function in your plugin, you can explode these into an array.
$extra = explode( "8", NON_DEFAULT_CHARACTERS );
$extra would then be a nice array of all your "non-default characters" you could use in your plugin.
echo $extra[0]; // -
echo $extra[1]; // ~
echo $extra[6]; // ^
You could then experiment by simply changing the config file.
Pop VeKind
PS: Did you consider the value of the extra characters?
Are you running a goo.gl or bit.ly where these may be important?  If you are you will likely need more resources an more knowledge to make it work.
Are you running a private YOURLS?  Likely the defaults are plenty.
Do you just want to see if it can be done?  Will it work?  Then you are near and dear to my heart.  Good luck, it is the journey that matters.  Make the above suggested any extra characters plugin and enjoy!

I am actually using this plugin:
http://github.com/josheby/yourls-additional-charsets
I was more concerned about a major update that would break compatibility with older plugins (something i am soon going to experience with xenforo2 release)!

@ozh - Just a flash idea...
Perhaps leave the Admin basically the same with the added ability to delete any entry.
Likewise, when the shortened URL was serviced, maybe we should check anything against the DB.
For example, we add the keyword 123-XYZ  to the Database.  Then remove the hyphen plugin.  123-XYZ could be allowed to work IF it is already in the DB, just no new hyphen keys woyld be allowe to be added.
Just a thought.

@Sushubh - Ah yes, that is always a problem.  How to "Future Proof" a program/plugin?  It cant be done.
yourls-additional-charsets looks good though. I especially like that you can supply your own Custom Charset.  Actually, I like simple things and that is all I would use.
I would not worry about this breaking in future releases of YOURLS.  It uses a standard hook - 'get_shorturl_charset' - which appears to be designed in the core files for just this sort of purpose.
Pop VeKind

An Evolving Idea...
I've been chewing on this problem for a while.  I'm working on a Settings Page to replace the constants in the config file.  It is basically the same as, "What if I change my settings from 62 to 36?" (uppercase, lowercase, and numbers, to only lowercase and numbers?) It also fails on the uppercase letters.
I believe setting changes should not cause failures.
DB Entries Are Legal
The solution is simple, make all entries in the DB legal. This would mean setting changes would not break anything. If you had settings of 62, plus hyphens, plus exclamation points and a keyword
Something-Special! Then changed to a strict 36 only, your keyword Something-Special! would continue to work.
If it is in the DB and you don't want it, just delete it.
Automatic Only
Use the restriction settings only in the making of short URLs during automatic mode.
Allow Manual Override
When adding or editing a short URL MANUALLY, you should have the power of adding characters outside of the restricted range prescribed in settings for automatic URLs. Perhaps with a warning!

@PopVeKind your 1200 word comment is equal to what I said earlier here and in other issues: one should be able to modify a link from a different charset, and link created with a once valid charset should remain valid. At this point, what we need is a PR, not opinions.

I am looking at how to make a PR for this problem.  I wondered if other characters may be involved?
To quote section 2.3 of RFC 3986: (http://www.ietf.org/rfc/rfc3986.txt) "Characters that are allowed in a URI but do not have a reserved purpose are called unreserved. These include uppercase(A-Z), lowercase(a-z), decimal digits(0-9), hyphen(-), period(.), underscore(_), and tilde(~)."
Periods(.) may be problematic in some advanced YOURLS as they could be confused with file names like somefile.php or robots.txt
There is a plugin that allows the use of the slash (/).  This is not even an unreserved character and does conflict with other plugins or colocated programs.
For redirection, I can see no reason why uppercase (A-Z) lowercase (a-z), decimal digits (0-9), hyphen (-), underscore (_) and tilde (~) could not just be passed to the DB and allowed to pass or fail based on being in the DB.
Anyone deciding to use a period (.) or slash (/) or any other character, should be advised to do so at their own risk as these are advanced options that may conflict with other plugins or programs.
What non-standard (a-z, A-Z, 0-9) characters do you use?
This should solve the 'Allow Hyphens in Short URLs' problem.

Created simple plugin to allow period in short URL period-in-urls.zip

Closing, superseded by 2646
@parisek how tf did you upload a zip file here? :)

Fixed in current master.

The plugin for allowing hyphens isn't listed in the curated YOURLS plugins list and its page can't be found anymore.
Is this still supported somehow in latest YOURLS?