Chat mode not working: TypeError: can only concatenate str (not "NoneType") to str

I installed LLM Graph Builder App and running Backend and Frontend separately (dev environment). I added an article from Wiki: King Kong (1933 film) - Wikipedia.
Got "TypeError: can only concatenate str (not "NoneType") to str" when trying Chat mode. Here is the log in backend:
2024-12-14 15:16:42,439 - QA_RAG called at 2024-12-14 15:16:42.439451
2024-12-14 15:16:42,467 - Checking access for database: neo4j
2024-12-14 15:16:42,475 - Read access count: 0
2024-12-14 15:16:42,475 - The account has write access.
2024-12-14 15:16:42,476 - Chat Mode: graph_vector_fulltext
2024-12-14 15:16:42,768 - Chat mode settings: {'retrieval_query': '\nWITH node as chunk, score\n// find the document of the chunk\nMATCH (chunk)-[:PART_OF]->(d:Document)\n// aggregate chunk-details\nWITH d, collect(DISTINCT {chunk: chunk, score: score}) AS chunks, avg(score) as avg_score\n// fetch entities\nCALL { WITH chunks\nUNWIND chunks as chunkScore\nWITH chunkScore.chunk as chunk\n\n    OPTIONAL MATCH (chunk)-[:HAS_ENTITY]->(e)\n    WITH e, count(*) AS numChunks \n    ORDER BY numChunks DESC \n    LIMIT 40\n\n    WITH \n    CASE \n        WHEN e.embedding IS NULL OR (0.3 <= vector.similarity.cosine($embedding, e.embedding) AND vector.similarity.cosine($embedding, e.embedding) <= 0.9) THEN \n            collect {\n                OPTIONAL MATCH path=(e)(()-[rels:!HAS_ENTITY&!PART_OF]-()){0,1}(:!Chunk&!Document&!Community) \n                RETURN path LIMIT 20\n            }\n        WHEN e.embedding IS NOT NULL AND vector.similarity.cosine($embedding, e.embedding) >  0.9 THEN\n            collect {\n                OPTIONAL MATCH path=(e)(()-[rels:!HAS_ENTITY&!PART_OF]-()){0,2}(:!Chunk&!Document&!Community) \n                RETURN path LIMIT 40 \n            } \n        ELSE \n            collect { \n                MATCH path=(e) \n                RETURN path \n            }\n    END AS paths, e\n\n    WITH apoc.coll.toSet(apoc.coll.flatten(collect(DISTINCT paths))) AS paths, \n         collect(DISTINCT e) AS entities\n\n    // De-duplicate nodes and relationships across chunks\n    RETURN \n        collect {\n            UNWIND paths AS p \n            UNWIND relationships(p) AS r \n            RETURN DISTINCT r\n        } AS rels,\n        collect {\n            UNWIND paths AS p \n            UNWIND nodes(p) AS n \n            RETURN DISTINCT n\n        } AS nodes, \n        entities\n}\n\n// Generate metadata and text components for chunks, nodes, and relationships\nWITH d, avg_score,\n     [c IN chunks | c.chunk.text] AS texts, \n     [c IN chunks | {id: c.chunk.id, score: c.score}] AS chunkdetails,\n     [n IN nodes | elementId(n)] AS entityIds,\n     [r IN rels | elementId(r)] AS relIds,\n     apoc.coll.sort([\n         n IN nodes | \n         coalesce(apoc.coll.removeAll(labels(n), ['Entity'])[0], "") + ":" + \n         n.id + \n         (CASE WHEN n.description IS NOT NULL THEN " (" + n.description + ")" ELSE "" END)\n     ]) AS nodeTexts,\n     apoc.coll.sort([\n         r IN rels | \n         coalesce(apoc.coll.removeAll(labels(startNode(r)), ['Entity'])[0], "") + ":" + \n         startNode(r).id + " " + type(r) + " " + \n         coalesce(apoc.coll.removeAll(labels(endNode(r)), ['Entity'])[0], "") + ":" + endNode(r).id\n     ]) AS relTexts,\n     entities\n\n// Combine texts into response text\nWITH d, avg_score, chunkdetails, entityIds, relIds,\n     "Text Content:\n" + apoc.text.join(texts, "\n----\n") +\n     "\n----\nEntities:\n" + apoc.text.join(nodeTexts, "\n") +\n     "\n----\nRelationships:\n" + apoc.text.join(relTexts, "\n") AS text, \n     entities\n\nRETURN \n    text, \n    avg_score AS score, \n    {\n        length: size(text), \n        source: COALESCE(CASE WHEN d.url CONTAINS "None" THEN d.fileName ELSE d.url END, d.fileName), \n        chunkdetails: chunkdetails, \n        entities : {\n            entityids: entityIds, \n            relationshipids: relIds\n        }\n    } AS metadata\n', 'top_k': 5, 'index_name': 'vector', 'keyword_index': 'keyword', 'document_filter': False, 'node_label': 'Chunk', 'embedding_node_property': 'embedding', 'text_node_properties': ['text'], 'mode': 'graph_vector_fulltext'}
2024-12-14 15:16:42,769 - Error during chat setup: can only concatenate str (not "NoneType") to str
Traceback (most recent call last):
File "/Users/pchan3/llm-graph-builder/backend/src/QA_integration.py", line 406, in setup_chat
llm, model_name = get_llm(model=model)
^^^^^^^^^^^^^^^^^^^^
File "/Users/pchan3/llm-graph-builder/backend/src/llm.py", line 19, in get_llm
env_key = "LLM_MODEL_CONFIG_" + model
~~~~~~~~~~~~~~^
TypeError: can only concatenate str (not "NoneType") to str
2024-12-14 15:16:42,774 - Error processing chat response at 2024-12-14 15:16:42.774582: can only concatenate str (not "NoneType") to str
Traceback (most recent call last):
File "/Users/pchan3/llm-graph-builder/backend/src/QA_integration.py", line 423, in process_chat_response
llm, doc_retriever, model_version = setup_chat(model, graph, document_names, chat_mode_settings)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/pchan3/llm-graph-builder/backend/src/QA_integration.py", line 406, in setup_chat
llm, model_name = get_llm(model=model)
^^^^^^^^^^^^^^^^^^^^
File "/Users/pchan3/llm-graph-builder/backend/src/llm.py", line 19, in get_llm
env_key = "LLM_MODEL_CONFIG_" + model
~~~~~~~~~~~~~~^
TypeError: can only concatenate str (not "NoneType") to str
2024-12-14 15:16:42,775 - Total Response time is  0.34 seconds
[INFO]{'api_name': 'chat_bot', 'db_url': 'bolt://localhost:7687', 'userName': 'neo4j', 'database': 'neo4j', 'question': 'who is the director', 'document_names': '[]', 'session_id': 'e5976d9a-83b7-4651-a814-dab5aec2330d', 'mode': 'graph_vector_fulltext', 'logging_time': '2024-12-14 23:16:42 UTC', 'elapsed_api_time': '0.34'}
INFO:     <IP_ADDRESS>:54270 - "POST /chat_bot HTTP/1.1" 200 OK

Hi @peterchanws ,
Can you check configuration in frontend .env file. (Refer )
the llm model needs to be defined in following format
VITE_LLM_MODELS="diffbot,openai_gpt_3.5,openai_gpt_4o"
and in backend .env
LLM_MODEL_CONFIG_openai_gpt_4o="gpt-4o-2024-11-20,openai_api_key"
model format in frontnend & backend should match otherwise it throws TypeError.

Thanks, for your reply.
I test Chat mode using LLM "Diffbot". I have this in frontend :
VITE_LLM_MODELS="diffbot,openai_gpt_3.5,openai_gpt_4o,ollama_lamma3"; VITE_LLM_MODELS_PROD="diffbot,openai_gpt_3.5,openai_gpt_4o,ollama_lamma3"
and this in backend env:
LLM_MODEL_CONFIG_diffbot="diffbot,diffbot_api_key"
Frontend and backend LLM model format is the same, and still get the "TypeError: can only concatenate str (not "NoneType") to str" error.

this seem weird as your configuration is correct.
Can you check what value is getting passed from frontend for model in api payload.
Also add some logs in get_llm function in llm.py to check what model backend is receiving.

Check this PR.
#946
this will probably give better understanding of error you are facing.  If this doesn't solve the issue please share latest logs after this code deployment.


Added Angr Tool

Added another binary analysis tool called angr into the list.

Thanks @farif, I've extended the description a bit. This helps the tools to be more discoverable on the website. The length of the description is not really a limitation anymore as it was in the past.

Thanks @farif, I've extended the description a bit. This helps the tools to be more discoverable on the website. The length of the description is not really a limitation anymore as it was in the past.


bundle list will print message in random order when no specs found

Saw this when working with local openshift registry before I had any images with "apb" in the name.
(apbs) [dwhatley@precision-t mssql-apb]$ sbcli bundle list --refresh
Getting specs for registry: [docker]
INFO == REGISTRY CX ==
INFO Name: docker
INFO Type: local_openshift
INFO Url: docker.io
WARN Failed to create a InternalClientSet: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined.
INFO Validating specs...
INFO All specs passed validation!
INFO Registry docker has 0 bundles available from 13 images scanned
 BUNDLE     IMAGE     REGISTRY
 ------ -+- ----- -+- --------
INFO APBs filtered by white/blacklist filter:-> openshift/httpd-> openshift/jenkins-> openshift/mariadb-> openshift/mongodb-> openshift/mysql-> openshift/nginx-> openshift/nodejs-> openshift/perl-> openshift/php-> openshift/postgresql-> openshift/python-> openshift/redis-> openshift/ruby
(apbs) [dwhatley@precision-t mssql-apb]$

versus
(apbs) [dwhatley@precision-t mssql-apb]$ sbcli bundle list --refresh
Getting specs for registry: [docker]
INFO == REGISTRY CX ==
INFO Name: docker
INFO Type: local_openshift
INFO Url: docker.io
WARN Failed to create a InternalClientSet: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined.
INFO Validating specs...
INFO All specs passed validation!
INFO Registry docker has 0 bundles available from 13 images scanned
 BUNDLE     IMAGE     REGISTRY
 ------ -+-INFO APBs filtered by white/blacklist filter:-> openshift/httpd-> openshift/jenkins-> openshift/mariadb-> openshift/mongodb-> openshift/mysql-> openshift/nginx-> openshift/nodejs-> openshift/perl-> openshift/php-> openshift/postgresql-> openshift/python-> openshift/redis-> openshift/ruby
 ----- -+- --------
(apbs) [dwhatley@precision-t mssql-apb]$

Worth noting that I ran across this while trying out a PR to bundle-lib, so this may not be a bug in sbcli.


🐛 [BUG] - The Image at the Tourist Website Project is not Showing its broken

Description
Here The Image is not showing in the card its broken

Reproduction URL
https://zerooctave.co/ZeroOctave-Javascript-Projects/
Reproduction steps
1. Go to 'https://zerooctave.co/ZeroOctave-Javascript-Projects/'
2. Scroll down to 'Tourist Website'
3. See error

Screenshots
![DESCRIPTION](LINK.png)
https://github.com/ZeroOctave/ZeroOctave-Javascript-Projects/assets/125110272/d5712836-ad10-44ec-af15-f8c9f32c2b52

Logs
No response
Browsers
Chrome, Firefox, Safari, Microsoft Edge, Brave
OS
Windows
Additional Context
No response
Sections Completed

[X] Description
[X] Reproduction URL
[X] Reproduction steps
[X] Screenshots
[X] Logs
[X] Browsers
[ ] OS
[X] Additional Context

Code of Conduct

[X] I follow Contributing Guidelines of this project.
[X] I have checked the current issues for duplicate problems.
[X] I have read the readme of this repository.

Hi @gopiexplore, I would love to resolve this issue, please assign it to me.

Please merge with Hacktoberfest-accepted


Change windows versions to 1.9.0 due to unstable build

Changed references of Windows release to point to 1.9.0 as 2.x builds are currently unstable on Windows

as discussed with tomas, lets communicate this via github release page


How do I download your software?

Hello, I need your software urgently, how do I download it?
@krlvm thank you

@krlvm krlvm

https://github.com/krlvm/ClassicSearch/releases


bug:

Describe the bug
avante.nvim 0.0.10 version
press <leader>ae to enter edit mode and submit any questions, it will return the error below:
Error occurred while processing the response: {
message = "post https://api.anthropic.com/v1/messages - curl error exit_code=nil stderr={}",
stderr = "{}"
}
To reproduce
same as above
Expected behavior
no error should return, this issue is introduced by avante.nvim 0.0.10 version
Installation method
Use lazy.nvim:
{
  "yetone/avante.nvim",
  event = "VeryLazy",
  lazy = false,
  version = false, -- set this if you want to always pull the latest change
  opts = {
    -- add any opts here
  },
  -- if you want to build from source then do `make BUILD_FROM_SOURCE=true`
  build = "make",
  -- build = "powershell -ExecutionPolicy Bypass -File Build.ps1 -BuildFromSource false" -- for windows
  dependencies = {
    "nvim-treesitter/nvim-treesitter",
    "stevearc/dressing.nvim",
    "nvim-lua/plenary.nvim",
    "MunifTanjim/nui.nvim",
  },
}

Environment
neovim v0.10.1
Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    -- add any other plugins here
  },
})

happens in ubuntu 22.04 and ubuntu 24.04
looks like good in macos


Switch to InMemoryDocumentStore as much as possible

We can have some tutorials around other DocumentStores, but to show the other features, we should start using InMemoryDocumentStore

@bilgeyucel can this issue be closed?

No, it is still a valid issue


Serializable structs with explicit layout

Describe the bug
Can`t edit area of effects values
Expected behavior
able to edit area of effect values
Code Sample

[Serializable]
[StructLayout(LayoutKind.Explicit)]
public struct GameplayAction
{
    [FieldOffset(0)]
    public GameplayActionType Type;

    [FieldOffset(4)]
    [ShowIf(nameof(Type), GameplayActionType.SpawnProjectile)]
    public SpawnProjectileAction SpawnProjectile;

    [FieldOffset(4)]
    [ShowIf(nameof(Type), GameplayActionType.ApplyEffect)]
    public ApplyEffectAction ApplyEffect;

    [FieldOffset(4)]
    [ShowIf(nameof(Type), GameplayActionType.ApplyAreaOfEffect)]
    public AreaOfEffectShape AreaOfEffect;

    [FieldOffset(4)]
    [ShowIf(nameof(Type), GameplayActionType.SpawnAreaOfEffect)]
    public SpawnAreaOfEffectAction SpawnAreaOfEffect;
}
//

Screenshots

Desktop: Windows 11
**Unity version:  2020.3.35f1

Tri Inspector version: 1.1.01

TL;DR: You can easily solve this by adding [NonSerialized] attribute to all but one field with [FieldOffset(4)]

I know this is a year-old issue, but I just did some digging.
This is actually a Unity behavior due to how Unity Serialization works, and this is also the behavior in vanilla Unity.
Consider the following struct:
[Serializable, StructLayout(LayoutKind.Explicit)]
public struct ExampleStruct
{
    [FieldOffset(0)] public ActionEnum Type;
    [FieldOffset(4)] public EnumA A;
    [FieldOffset(4)] public EnumB B;
}

If you create a serialized object that contains this struct, Unity will save the object as such:
--- !u!114 &2043024539519176523
MonoBehaviour:
  StructValue:
    Type: 0
    A: 0
    B: 0

Unity Inspector works by updating the value to a serialized representation then deserialize it to apply to the instance. You can probably see the problem - Unity will apply A, then B. But A and B are the same field!, which means any changes to A will be immediately overwritten by the already serialized B value. This is also why we can still edit B in the Inspector.
Now, despite being able to hide fields that are not relevant using Tri-Inspector, Unity Serialization stands true always. However, we can leverage the [ShowInInspector] feature to make a non-serialized field editor...
This is what I would do:
[Serializable, StructLayout(LayoutKind.Explicit)]
public struct ExampleStruct
{
    [FieldOffset(0)] public ActionEnum Type;

    [NonSerialized, ShowInInspector, ShowIf(nameof(Type), ActionEnum.TypeA)]
    [FieldOffset(4)] public EnumA A;

    [NonSerialized, ShowInInspector, ShowIf(nameof(Type), ActionEnum.TypeB)]
    [FieldOffset(4)] public EnumB B;

    [SerializeField, HideInInspector]
    [FieldOffset(4)] private int _internalField;
}

This way, you explicitly tell Unity to only maintain one true value, and only use other fields as an Editor provided by Tri-Inspector:
--- !u!114 &4228018463491459332
MonoBehaviour:
  StructValue:
    Type: 0
    _internalField: 0

I believe this issue can be closed due to not being a bug on the Tri-Inspector side, if it is a bug at all.


test(middleware): get rinkeby private key from env var

Motivation
Test is failing because account needs to have some ether in it and we got no ether in it, sad. If we'll send some ether to the address used in the test, it'll be drained quickly, no bueno.
Solution
Use private key for rinkeby account from environment variable (@gakonst needs to set it for GitHub Actions and send some rinkeby ether to this account)

Gonna set this up today


Buffering unknown topics

By default, producing records to an unknown topic will indefinitely append to the client's "unknown topic buffer" without any feedback or a way to flush the buffered records outside of closing the client.
I see the unknown topic buffer has a capacity of 100 (https://github.com/twmb/franz-go/blob/master/pkg/kgo/producer.go#L678), but it does not get respected by appends (https://github.com/twmb/franz-go/blob/master/pkg/kgo/producer.go#L683). Therefore, it looks like the unknown topic buffer will grow indefinitely in size.
With auto topic creation not enabled and no timeouts configured, client.ProduceSync on an unknown record will hang indefinitely and the callback function forclient.Produce will not be called. I understand that this would not be an issue with AllowAutoTopicCreation enabled, but we do not wish to enable auto topic creation. My workaround at the moment is simply to set a client timeout through RecordDeliveryTimeout, but the growing buffer issue still exists especially during high throughput.
It would be useful if there were an error specifically for records written to unknown topics when auto topic creation is not enabled. Alternatively, it would be nice if errRecordTimeout were exposed so I can easily and cleanly determine if an error from client.ProduceSync or client.Produce is a timeout error and therefore potentially an "unknown topic" error. Perhaps a ProductOpt config such as OnRecordDeliveryTimeout could be helpful as well.

Good note! Per our discussion on slack, I've decided to change producing to unknown topics to failing after 5 load failures. This should cover the problem you're running into.
I've also gone ahead and exported ErrRecordTimeout and ErrRecordRetries.
Thanks for the report, this is a good idea. I've tagged v1.1.0 with this new behavior, which closes this issue. Please let me know if you have more good ideas, thank you!


🛀🏾  Include core/built task in grunt build/init

refs #8162
Within the grunt init and grunt build task we'll clean our core/built directory, so they don't build up over time as we're now delivering client
assets with cache-busting hash in the filename.

Closing because we're trying a different attempt


Fix :line:col in anything quick access when @ is in path

Fixes #185139

@Tyriar can you update the test?

@TylerLeonhardt will do

Thanks!


Fix pagination for looped or grouped slider.

On the example app of the repository if we change the swiper configuration with
loop: true
on line 36 of file app.component.ts, the pagination gets broken. It does not go to the correct slide.
The culprit is method onIndexSelect() of swiper.component.ts of ngx-swiper-wrapper.
Inspecting the function onClickIndex() on core.js of Swiper package I've come to the present commit which solves the problem.
Regards

Thanks, I will merge this and include in the upcoming 4.5.0 release.


"more" to "less" on line 29.

The example is setting access token lifetime to 2 hours. An access token's default lifetime is assigned a random value ranging between 60-90 minutes (75 minutes on average). https://learn.microsoft.com/en-us/azure/active-directory/develop/active-directory-configurable-token-lifetimes#access-tokens. So it is actually extending the lifetime from the default lifetime and users will authenticate less frequently.

@rwike77
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks.  #sign-off


⚠ PrivateHD has degraded performance

In 29ff51e, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2487 ms

Resolved: PrivateHD performance has improved in 4177de8.


[TC-291] GenIso can't handle concurrent requests

Because GenIso always changes the same files (no temporary directories created), it can't handle concurrent requests. We sometimes send commands to create 10 Edge Caches at the same time and we found out that the wrong IP address shows up on servers.
I've got the code to fix it up (but need to sign our paperwork). But our GenIso.pm now deviates from what's in Traffic Ops because we wanted to keep the 1.0.0 functionality (don't use the install_cfg directory, keep using the simple network line).
Suggestion: We should have a parameter to call different ISO generator scripts. Seems like this is implementation specific and not really a core function of Traffic Ops.
Basic changes :
use File::Temp;
use File::Copy::Recursive;
my $tmpdir = File::Temp::tempdir(CLEANUP => 1);
...
GenIso.pm:        my $tmp = File::Copy::Recursive::dircopy($dir,$tmpdir);
GenIso.pm:  my $cfg_dir = "$tmpdir/$install_cfg";
GenIso.pm:  print STUF "Dir== $tmpdir\n";
GenIso.pm:  my $cmd = "mkisofs -joliet-long -input-charset utf-8 -b isolinux/isolinux.bin -c isolinux/boot.cat -no-emul-boot -boot-load-size 4 -boot-info-table -R -J -v -T $tmpdir";
GenIso.pm:  open(IN, "<$tmpdir/ks.src") || die("$tmpdir/ks.src:$!");
GenIso.pm:  open (OUT, ">$tmpdir/ks.cfg") || die("$tmpdir/ks.cfg:$!");
GenIso.pm:        File::Path::rmtree $tmpdir;
Author: Dewayne Richardson
JIRA Link: https://issues.apache.org/jira/browse/TC-291
Found Version: 2.1.0

Still an issue


Object services

I have a curiosity because using the services object of the webapp I get encrypted results?

Can you show us a screenshot or something? Does it also happen if you disable the Tampermonkey script?

if I use the command from the console the result is encrypted with your script no

Which command do you execute from the console? Please be more clear, or better yet show a screenshot.

Example:
var searchCriteria = getAppMain().getRootViewController().getPresentedViewController().getCurrentViewController().getCurrentController()._viewmodel.searchCriteria;
    searchCriteria.maxBid =10000000;
    services.Item.clearTransferMarketCache();

    services.Item.searchTransferMarket(searchCriteria, 1)

Well that's because services.Item.searchTransferMarket returns an observable. However this isn't actually an issue but more help with coding, I'll have to refer to Gitter. These issues are meant for bugs or feature requests.

ok sorry,
where can I post these question ?


Proposed fix for double-borders in justified button groups

This fix works for all the examples in the docs (and my own application) in IE9+. The fix is null in IE8 dues to lack of :not() support – I though't I'd better put this out there in case I was missing an obvious caveat before working on IE8-compatible code (which would involve resetting all justified button left borders and re-applying them in the case of the first child).

Here's the components page with links to the compiled version of this fork. The relevant stuff can be accessed via Buttons > Justified button groups

Fixes #13830.

I have tried this fix, it is ugly if you hover over or click a button as that border will not be the color of the left button.

This was my fix, it works beautifully, but it's quite a bit of css: #13831

Nice one @Jelte – personally I find the double border far more repulsive than the barely noticeable lack of stateful inset. Seeing as the core team seem to prefer the status quo, perhaps we should create a bootstrap-single-border-justified-buttons repo with multiple branches depending on user preference? ;)

Why not just do this?
.btn-group-justified .btn + .btn {
  margin-left: 0;
  border-left-width: 0;
}

@MarkMurphy first option solves the IE8 case — nice (if you put in a PR for that I'll close this one)! Second option won't work when dealing with nested structures.

Second option won't work when dealing with nested structures.

@barneycarroll Can you elaborate on this? The source is already the border-collapse property except with the "seperate" value instead of "collapse". See: https://github.com/twbs/bootstrap/blob/master/less/button-groups.less#L204

@MarkMurphy if you check out the 2nd and 3rd demos in the docs, there are numerous situations where the actual button-like element might be a descendant of the .btn-group, at which point border-collapse no longer takes effect.

@barneycarroll That link doesn't point to the right place but I see what you mean now. There still seems to be a double border even with option 1 when when it comes to having a nested button group. (and yes I made sure I updated the selector to target that aswell)

Ok so this seems to do the job:
.btn-group-justified .btn,
.btn-group-justified .btn-group {
  & + .btn,
  & + .btn-group .btn {
    border-left-width: 0;
  }
}

which translates into this:
.btn-group-justified .btn + .btn,
.btn-group-justified .btn + .btn-group .btn,
.btn-group-justified .btn-group + .btn,
.btn-group-justified .btn-group + .btn-group .btn {
  border-left-width: 0;
}

Can't go this route unfortunately—there are a number of border inconsistencies with this. Our general approach has been to avoid removing borders.


Rule for Func and Lazy to be resolved even without requested service registered

Currently, the service even being a lazy resolved / injected, should be registered.
It should enable more lazy scenarios. Given that it is a default Autofac behavior, the rule will provide an easier migration.
Btw, may be the rule should work more generally for any asResolutionCall dependencies?

This test program should work without RegisterPlaceholder and IfAlreadyRegistered.Replace:
using System;
using DryIoc;
public class Program
{
	public static void Main()
	{
		var c = new Container();
		c.Register<A>();
		c.RegisterPlaceholder<B>(); // fix part 1

		var a = c.Resolve<A>();
		c.Register<B>(ifAlreadyRegistered: IfAlreadyRegistered.Replace); // fix part 2

		a.LateResolve();

		Console.WriteLine("Hello World");
	}

	class A
    {
        private readonly Func<B> _func;

        public A(Func<B> func)
        {
            _func = func;
        }

        public void LateResolve()
        {
            _func();
        }
    }

    class B
    {
    }
}

Funny that now when DryIoc has rule WithFuncDoesNotNeedRegistration, I cannot come-up with working Autofac example:
using System;
using Autofac;

public class Program
{
	public static void Main()
	{
		var b = new ContainerBuilder();
		b.RegisterType<A>();

		var c = b.Build();

		var a = c.Resolve<A>();

		var b2 = new ContainerBuilder();
		b2.RegisterType<B>();

		b2.Update(c);

		a.LateResolve();
	}
}

class A
{
	private readonly Func<B> _func;

	public A(Func<B> func)
	{
		_func = func;
	}

	public void LateResolve()
	{
		_func();
	}
}

class B
{
}


Example in README indented incorrectly

It's a little hard to read this way becuase the indenting is not correct. I think this is the correct one:
on: [push]

jobs:
  jbang:
    runs-on: ubuntu-latest
    name: A job to run jbang
    steps:
    - name: checkout
      uses: actions/checkout@v1
    - uses: actions/cache@v1
      with:
        path: /root/.jbang
        key: ${{ runner.os }}-jbang-${{ hashFiles('*.java') }}
        restore-keys: |
          ${{ runner.os }}-jbang-
    - name: jbang
      uses<EMAIL_ADDRESS>      with:
        script: createissue.java
        args: "my world"
      env:
        JBANG_REPO: /root/.jbang/repository
        GITHUB_TOKEN: ${{ secrets.ISSUE_GITHUB_TOKEN }}

You probably should stop using TABs in files that indent-sensitive ;-)

wasn't actually problem of tab or spaces but that spotless plugin for gradle can't handle yaml style indention in markdown ;/

fixed indention and ignoring the file in validation step.


Use open LLMs instead of openai

Is your feature request related to a problem? Please describe.
It is expensive to use openai credentials in an open source project. Moreover, getting free credits is also not viable.
Describe the solution you'd like
I propose to use open LLMs like mistral,llama,gemma etc. We can use ollama to deploy these LLMs.
Would like to start working on the same.

Hey,
I will try to solve this by using a fine-tuned, lightweight, open-source LLM. Will create a pull request once completed.

hey @denxxs
I have already created a pull request #18 for the same.
You can though suggest changes in the same . If required, we can work on something together.


[New] include Natural Earth data in covsirphy library

Summary of this new feature
As discussed in #1173,  Natural Earth published under Public Domain, will be included in covsirphy as geopandas does.

[ ] Add a script of downloading/updating the dataset to stable version publish workflow
[ ] Save the file in covsirphy/gis/natural_earth directory with README.md (to explain the data source)
[ ] Ensure covsirphy to include the dataset with pyproject.toml (using "include" option?)
[ ] covsirphy.gis.geometry._Geometry._natural_earth() will read the saved files internally (with relative path from covsirphy directory?)
[ ] Explain the workflow and data source in documentation

CovsirPhy version I used: 2.26.1

Because the dataset of the data source will be updated irregularly, the dataset will not be included in CovsirPhy. As a quick solution, the directory to save the files will be changed to "covsirphy/gis/Natural_Earth" directory.


image-filters doc missing

"(see the image-filters doc for how they work on a separate canvas)"
Yeah, but where?

I've also ran into this problem. Specifically I've been trying to figure out what the parameter order of scale-hsla is.

Related, duplicate of #285.


Check If Package Is Installed false positive

Device
V540TND, V560TNE
RTE version
v0.9.1-rc1
OSFV version
fix-novacustom-v5xtnx-configs-powerctrl
Affected component(s) or functionality
Tests which require additional software fail if it is not installed in advance
Brief summary
The keyword wrongly interprets dpkg output and thinks the package is installed
How reproducible
100%
How to reproduce
run for example scripts/run.sh dasharo-compatibility/usb-camera.robot which requires ffmpeg and libinput, only reproducible if these are not already installed
Expected behavior
packages should be detected as missing and installed
Actual behavior
packages are detected as installed and then the test fails because it is actually not
Link to screenshots or logs

log: https://cloud.3mdeb.com/index.php/s/GZde3NXCN9TKFM9
Additional context
It may be an issue in SSHLibrary, because the output of Execute Command In Terminal shoul not be empty there.
Probably related to https://github.com/Dasharo/open-source-firmware-validation/issues/439 and https://github.com/Dasharo/open-source-firmware-validation/issues/438
Solutions you've tried
No response

Fixed with this change https://github.com/Dasharo/open-source-firmware-validation/pull/455/files


Use squash merge to keep the git history cleaner

Can I suggest that we use squash merge when merging PRs to keep the history cleaner? I was just going over the changes between 1.4.1 and 1.4.2 and it is so much more difficult to figure out what has changed based on the individual commit messages compared to if the merged PRs were squashed into a single commit. Where commits like e.g. "fix a typo" or "make it work" make sense in the local context of the feature branch, once the feature is complete in my opinion there's no need to include those in the main history as those commits end up being interleaved with commits from other PRs which makes the history in master quite confusing and a lot of commits are out of context.
@mccartney @sksamuel do you have any thoughts on this?

Yeah. Sure. I am just not used to it (at work).
I think we can configure to enable only Squash merges for PRs: (i.e. @sksamuel can)

Done.


Scroll collapsed file into view

Fixes #23701, #23515.
Alternate approach to #23604 using CSS scroll-margin-top, which is also taken into account for direct links to files in a diff:

On the PR diff, this currently shows the previous file first: https://try.gitea.io/jpraet/test/pulls/13/files#diff-b94d08b409f9d05fb65b6cccaf7b3e4ecc7cc333
On the commit diff, the first line of the linked file is currently under the sticky header: https://try.gitea.io/jpraet/test/commit/1a19e6b14e31d295b7372f3346580c3e85690ff5#diff-b94d08b409f9d05fb65b6cccaf7b3e4ecc7cc333

Codecov Report

Merging #23702 (a08b6c1) into main (f521e88) will increase coverage by 0.03%.
The diff coverage is 41.40%.

:exclamation: Current head a08b6c1 differs from pull request most recent head 37da638. Consider uploading reports for the commit 37da638 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main   #23702      +/-   ##
==========================================
+ Coverage   47.14%   47.18%   +0.03%
==========================================
  Files        1149     1155       +6
  Lines      151446   152396     +950
==========================================
+ Hits        71397    71901     +504
- Misses      71611    72015     +404
- Partials     8438     8480      +42

Impacted Files
Coverage Δ

cmd/dump.go
0.66% <0.00%> (-0.01%)
:arrow_down:

cmd/web.go
0.00% <0.00%> (ø)

models/actions/run.go
1.64% <0.00%> (-0.08%)
:arrow_down:

models/actions/runner.go
1.44% <ø> (ø)

models/packages/package.go
45.45% <0.00%> (-1.13%)
:arrow_down:

models/user/search.go
77.50% <0.00%> (-6.29%)
:arrow_down:

modules/actions/workflows.go
0.00% <0.00%> (ø)

modules/context/context.go
64.54% <0.00%> (-3.53%)
:arrow_down:

modules/doctor/storage.go
30.65% <0.00%> (-1.29%)
:arrow_down:

modules/setting/git.go
45.45% <ø> (ø)

... and 37 more

... and 47 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I think this might have a dependency/interaction on https://github.com/go-gitea/gitea/pull/23553, so would wait until that one is in.

I think this might have a dependency/interaction on #23553, so would wait until that one is in.

seems to be OK


Lots of problems installing extensions

Also posted this at jupyter/notebook#389 since I wasn't sure where it belongs...
Apologies for this somewhat long issue. It describes my progress (and ultimate failure) to install jupyter extensions.
I just upgraded from ipython to jupyter and am trying to get my environment working properly. I upgraded using conda (conda install jupyter) and have jupyer 1.0.0 installed. I am trying to get extensions working, following these instructions:
https://github.com/ipython-contrib/IPython-notebook-extensions/wiki/Home_Jupyter
First I was stuck on installing extensions:
https://github.com/ipython-contrib/IPython-notebook-extensions/wiki/Home_Jupyter#1-installing-an-extension
When I first tried
 jupyter nbextension install usability/codefolding/main --user

I got a long stack trace ending with
IOError: [Errno 2] No such file or directory: u'usability/codefolding/main

It seemed like it was looking for an extension in my local file path, so I surmised I was supposed to clone the IPython-notebook-extensions repo and install the extensions from the base directory.
git clone git@github.com:ipython-contrib/IPython-notebook-extensions.git
cd IPython-notebook-extensions
jupyter nbextension install usability/codefolding/main --user

Still got the IOError. However, examining the repo files, I was able to finally get something to work
jupyter nbextension install usability/codefolding/main.js --user

This command completed without an error, so I assumed it was working. In fact, it put the file main.js into the path /Users/rpa/Library/Jupyter/nbextensions, which I presume is not the desired behavior.
After futher trial and error, I ended up with
$ jupyter nbextension install usability/codefolding --user

which gave me
copying /Users/rpa/RND/open_source/IPython-notebook-extensions/usability/codefolding/codefolding.py -> /Users/rpa/Library/Jupyter/nbextensions/codefolding/codefolding.py
copying /Users/rpa/RND/open_source/IPython-notebook-extensions/usability/codefolding/codefolding.yaml -> /Users/rpa/Library/Jupyter/nbextensions/codefolding/codefolding.yaml
copying /Users/rpa/RND/open_source/IPython-notebook-extensions/usability/codefolding/firstline-fold.js -> /Users/rpa/Library/Jupyter/nbextensions/codefolding/firstline-fold.js
copying /Users/rpa/RND/open_source/IPython-notebook-extensions/usability/codefolding/foldgutter.css -> /Users/rpa/Library/Jupyter/nbextensions/codefolding/foldgutter.css
copying /Users/rpa/RND/open_source/IPython-notebook-extensions/usability/codefolding/icon.png -> /Users/rpa/Library/Jupyter/nbextensions/codefolding/icon.png
copying /Users/rpa/RND/open_source/IPython-notebook-extensions/usability/codefolding/magic-fold.js -> /Users/rpa/Library/Jupyter/nbextensions/codefolding/magic-fold.js
copying /Users/rpa/RND/open_source/IPython-notebook-extensions/usability/codefolding/main.js -> /Users/rpa/Library/Jupyter/nbextensions/codefolding/main.js

Looked good. I now have all the extension files in /Users/rpa/Library/Jupyter/nbextensions/codefolding. So I did
jupyter nbextension enable usability/codefolding/main.js

As far as I can tell, all this does is put the following into /Users/rpa/.jupyter/nbconfig
{
  "load_extensions": {
    "usability/codefolding": true
  }
}

Now I start jupyter in my home directory with jupyter notebook. After creating a new python 2 notebook, I see the followin in the log
[I 12:54:30.137 NotebookApp] Kernel started: 35af37d8-82ec-4899-9b08-633f69adc7e8
[W 12:54:30.185 NotebookApp] 404 GET /nbextensions/usability/codefolding.js?v=20150903125424 (::1) 14.70ms referer=http://localhost:8888/notebooks/Untitled.ipynb?kernel_name=python2

which makes me think that the extension is not being found. Sure enough, the extension doesn't work. The following cell
import notebook
print(notebook.nbextensions.check_nbextension('usability/codefolding', user=True))
print(notebook.nbextensions.check_nbextension('usability/codefolding/main.js', user=True))

gives me False False.
I am using the most standard platform possible (anaconda on mac) and have tried to follow the instructions carefully, but I really don't see how to move forward. So there seems to be a bug in either the code or the documentation.
Any advice appreciated.
p.s. The output of my config / path info is
from __future__ import print_function
from jupyter_core.paths import jupyter_config_dir, jupyter_config_path
print(jupyter_config_dir())
print(jupyter_config_path())

/Users/rpa/.jupyter
['/Users/rpa/.jupyter', '/Users/rpa/anaconda/etc/jupyter', '/usr/local/etc/jupyter', '/etc/jupyter']

from __future__ import print_function
from jupyter_core.paths import jupyter_data_dir, jupyter_path
print(jupyter_data_dir())
print(jupyter_path())

/Users/rpa/Library/Jupyter
['/Users/rpa/Library/Jupyter', '/Users/rpa/anaconda/share/jupyter', '/usr/local/share/jupyter', '/usr/share/jupyter']

I am having very similar problems with the javascript not loading in my notebooks, but I was able to get the install checks to work. From the repository usability directory (for me: path toIPython-notebook-extensions/usability/) use command line python to execute the following:
import notebook
notebook.install_nbextension('usability/python-markdown',user=True)
E=notebook.nbextensions.EnableNBExtensionApp()
E.enable_nbextension('python-markdown')

For me this passes the check:
notebook.nbextensions.check_nbextension('python-markdown', user=True)
I hope this helps you, but if anyone else has further advice to why I might not be getting the extensions to load that would be appreciated. I am seeing the same 404 error in my javascript console.

Installing notebook extensions can be really confusing. I struggled a lot, too.
@rabernat : I think in your case the reason is that the extensions gets copied into codefolding/... instead of usability/codefolding/...
So if you try print(notebook.nbextensions.check_nbextension('codefolding', user=True)) it most likely will work.
Hopefully I will get to making a conda recipe for notebook extensions in the next days, so it will be much easier installing extensions, as well as activating/deactivating them.

@juhasch -- thanks for the tip! Yes, in fact, notebook.nbextensions.check_nbextension('codefolding', user=True) does return True.
However, I still can't actually get the extension to work. I am still getting the 404 errors in my log. I tried changing my ~/.jupyter/nbconfig/notebook.json file to read
{
  "load_extensions": {
    "codefolding": true
  }
}

to match the grammar from your check_extension suggestion, but that doesn't work either. The errors are like
[W 13:38:27.146 NotebookApp] 404 GET /nbextensions/codefolding.js?v=20150904133822 (::1) 16.28ms referer=http://localhost:8888/notebooks/lectures/test_extensions.ipynb

and the extension is not loading.
Any idea what I have to do to make it work?

Try "codefolding/main": true, instead of "codefolding": true in the json file.

Yes! It works! Thanks so much.
It seems like the wiki is inaccurate / incomplete, no? Is it out of date?

IPython 4.x changed a lot of things. Please update the Wiki if you find it inaccurate.
We will change from the Wiki to having the documentation directly in the repository. With many changes in each IPython/Jupyter version it is hard to keep the Wiki correct.

I am similarly lost... don't see a recipe for simple install of JS extensions that works, start to finish I did an upgrade from Ipython to Jupyter in my base Python install, most everything works except this, I just went through trying:
tom@tom-sam:~$ sudo jupyter nbextension install usability/codefolding/main
/usr/local/lib/python2.7/dist-packages/IPython/utils/path.py:258: UserWarning: get_ipython_dir has moved to the IPython.paths module
warn("get_ipython_dir has moved to the IPython.paths module")
copying usability/codefolding/main -> /usr/local/share/jupyter/nbextensions/main
Traceback (most recent call last):
File "/usr/local/bin/jupyter-nbextension", line 11, in
sys.exit(main())
File "/usr/local/lib/python2.7/dist-packages/jupyter_core/application.py", line 267, in launch_instance
return super(JupyterApp, cls).launch_instance(argv=argv, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/traitlets/config/application.py", line 592, in launch_instance
app.start()
File "/usr/local/lib/python2.7/dist-packages/notebook/nbextensions.py", line 401, in start
super(NBExtensionApp, self).start()
File "/usr/local/lib/python2.7/dist-packages/jupyter_core/application.py", line 256, in start
self.subapp.start()
File "/usr/local/lib/python2.7/dist-packages/notebook/nbextensions.py", line 317, in start
self.install_extensions()
File "/usr/local/lib/python2.7/dist-packages/notebook/nbextensions.py", line 305, in install_extensions
nbextensions_dir=self.nbextensions_dir,
File "/usr/local/lib/python2.7/dist-packages/notebook/nbextensions.py", line 216, in install_nbextension
_maybe_copy(src, full_dest, verbose)
File "/usr/local/lib/python2.7/dist-packages/notebook/nbextensions.py", line 53, in _maybe_copy
shutil.copy2(src, dest)
File "/usr/lib/python2.7/shutil.py", line 130, in copy2
copyfile(src, dst)
File "/usr/lib/python2.7/shutil.py", line 82, in copyfile
with open(src, 'rb') as fsrc:
IOError: [Errno 2] No such file or directory: u'usability/codefolding/main'
tom@tom-sam:~$
I look forward to the better docs! If I need to clone the repo let me know,, also it sort of seems that
This form may be depreciated? or will it work once installation is sorted out?
%%javascript
IPython.load_extensions('IPython-notebook-extensions-master/usability/python-markdown/main');

@dartdog looks to me from https://github.com/jupyter/notebook/issues/389 like this is ready to close too?

absolutely closed as far as I'm concerned, I still can't get some JS
extensions working but that is other topic (now that I have the base
configuration and base extensions working correctly)
On Wed, Sep 9, 2015 at 3:17 PM, jcb91<EMAIL_ADDRESS>wrote:

@dartdog https://github.com/dartdog looks to me from
jupyter/notebook#389 https://github.com/jupyter/notebook/issues/389
like this is ready to close too?
—
Reply to this email directly or view it on GitHub
https://github.com/ipython-contrib/IPython-notebook-extensions/issues/282#issuecomment-139034372
.

--
Tom Brander
http://tombrander.com -Real Estate
http://oswco.com -Open Software
3763 West Jackson Blvd.
Birmingham, AL 35213
Phone<PHONE_NUMBER>
@dartdog

OK, @juhasch, could you close this?
@dartdog, feel free (of course) to open separate issues for each of the extensions you find which don't work :)


Add comment highlight when target from url

Currently this refresh page when click outside if highlighted. Is that acceptable?

Codecov Report

Merging #9047 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #9047      +/-   ##
=========================================
+ Coverage   41.09%   41.1%   +<.01%
=========================================
  Files         549     549
  Lines       71488   71488
=========================================
+ Hits        29380   29387       +7
+ Misses      38392   38385       -7
  Partials     3716    3716

Impacted Files
Coverage Δ

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

routers/repo/view.go
39.47% <0%> (+0.87%)
:arrow_up:

modules/process/manager.go
78.37% <0%> (+4.05%)
:arrow_up:

models/unit.go
67.56% <0%> (+5.4%)
:arrow_up:

Continue to review full report at Codecov.

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

Please resolve conflicts.

Dont merge please. There is a issue

You can review now.
As you can see # is still staying when click.
Last code:

@silverwind please review

Needs one more rebase (sorry my other JS PR landed).


[Backdrop] WIP, basic functionality

This will close #11546 by implementing the Backdrop spec
Still a WIP, but I figured given it's size I should PR early to get feedback on style, organization, etc.
This PR implements:

[X] Backdrop container component

[X] implicit BackdropFront and BackdropBack layout handling via flexbox

[X] BackdropBack

[X] BackdropBackSection with transitions

[X] BackdropBackMenuItem for navs (see "Active back layer")

[X] StackedFade for header transitions (a bit fragile, a BackdropHeaderStack is probably better)

[X] BackdropFront

[X] BackdropFrontSubheader

[X] BackdropFrontContent

[X] disabling / "scrim" state

[X] minimization state and controls

Future:

[ ] Backdrop control over and shortcuts for BackdropFront and BackdropBack props
[ ] Some more Backdrop variants of selection controls and input fields
[ ] Maybe generalize the custom close icon functionality and make use of it

Here's what the current implementation looks like:

Alright, the implementation is stable enough, so I'm going to put this down while waiting on some feedback.
As an aside, this codebase is a joy to work with. Very well designed.

@micimize This is a fantastic first PR - thanks for taking it on, and I'm sorry it has taken so long to review it!
I have rebased and updated the demos for changes in one of the imports, and while I need to give it a more thorough review, at first glance, it looks like a very clean implementation. 👍

It just occurred to me that if this component was ever to get promoted to core in the future, the name will conflict with the existing Backdrop component. Not sure how we'd handle that, or whether to account for it now...
@oliviertassinari, thoughts?

@mbrookes We could rename the existing Backdrop component Overlay?

@oliviertassinari Same name that came to mind for me .
Definitely a breaking change, but not one we have to worry about immediately – although we should probably allow some time between renaming the current Backdrop, and promoting this one if / when the time comes

@mbrookes Perfect, sounds like a plan :).

@mbrookes good to hear! Hopefully soon I'll have the time to add unit tests and typings. Been pretty focused on the react-native fork and related work


Added tag support.

Added tag fields to the preamble section of the general template.
Added tag fields to the preamble section of the technical template.
Added tag fields to the preamble section of each MIP.
Added appropriate tags to each MIP in the tags fields of the preamble sections.

@LongForWisdom If we do add tag support, I think we need to discuss the initial tags more thoroughly. Additionally,  we will need to formally define tags, how to use of them, the initial tags themselves, etc. This would need to be included in MIP0 or at least in the README.

Aight. So actually the confusion here may be my fault. I intended to setup the templates such that the 'Tags' field had the  field on. Buuuut it looks like I forgot to do that.
If the MIP Editor sets these then I don't think they need to be formally defined, I don't think we need to detail their use, I think they can purely be used by MIP Editors to aid discoverability in front-ends. I did consider adding a statement somewhere that tags should be used for that purpose, but there isn't really a good component in which to do that. I guess it could go in the MIP Editors roles / responsibilities.
Regarding the initial tag set, I mostly just went with what made sense to me. I'll provide justifications here for the tags used:
core: Defines MIPs that define core governance processes. Highlights MIPs that have a direct bearing on the governance process.
'Role' tag (mip-editor, governance-facilitator, various domain teams, etc): Highlights MIPs that have a direct bearing on how that role operates.
Process: Anything with a Process Component in it, useful if someone wants to understand the different ways they can interact with the protocol.
problem-space: Admittedly this one is more speculative, anything that relates to the problem-space definition. Changes, a new problem-space, whatever. Might be less useful.
collateral-onboarding: Anything related to the onboarding of new collateral. Useful if you want to onboard new collateral.
oracles: Anything related to oracles, if you want to become an oracle or join/create an oracle team.
risk: Anything under the domain of risk, useful if you want to understand what risk teams do.
interim: Anything that is expected to at some point become obsolete. Useful if you want to understand what is impermanent.
governance-cycle: Anything relating to the governance cycle. May be less useful.
emergency: Anything related to emergency handling.
application: Anything that involves applying for something. Collateral, Oracles, etc. Useful if you want to apply for things? May not be necessary.
active-list: Any list of ratified/active roles/entities. Domain teams, General Risk Models, etc.

Made some additional minor fixes during the merge.

Changed MIP5 such that it's type is 'General' (no sub proposals / process components)
Fixed whitespace formatting in MIP10 to match the rest.


$sailsSocket.get is not a function

I'm getting this error for anything related to the $sailsSocket eg .on, .subscribe. I've included in my main module angularSails.io and $sailsSocket in my controller.
I'm using the latest version 0.12.1. Do I need to include sails.io too?

yes, for the moment you need to include sails.io

Hey @robwormald  sorry to add up. Apparently, I'm getting this because I'm actually wish to listen from my other app at 8080. So I'm looking for ways to declare this default URL, and saw #30 and tried that, it works. Awesome.
But on production, when my code was compiled and serve up on S3, how do I get this URL works again? At the moment, we bundle all vendor stuff into one file so, I don't see a way to put the io.sails.url = "http://url" anywhere.
Can you guys give some suggestion on this?

Ugh, I really need to pull sails.io and replace it, shouldn't be this difficult. Apologies.
Best thing to do meanwhile would be to create a file (sailsconfig.js or w/e) and concat that in.

alright. no worries.


is there a way to have as possible values of a parameter the branches of another repository

if i want to create another parameter that have as possible values the list of branches of the current repo. how to do that?

i want to let the user choose the branch to clone of another repo for example

resources:
repositories:

repository: mytest
type: git
name: mytest
ref: $(BranchBinariesName)

parameters:

name: BranchBinariesName
type: string

variables:

name: BranchBinariesName
value: "${{ parameters['BranchBinariesName'] }}"

stages:

stage: test1
displayName: "test"
pool:  myBuilder
jobs:

job: display
displayName: 'display'
timeoutInMinutes: 210
steps:

checkout: mytest

so i need that the possible values of the parameter  BranchBinariesName be the available branches of the repo mytest

@yazbekhe - Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow


⚠ Bofrost.es has degraded performance

In bb0802d, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1703 ms

Resolved: Bofrost.es performance has improved in 4bdf664 after 13 minutes.


CLI tool for static analysis

It would be great if a cli tool could be built which is using the same logic/code then the admission controller which could be used for static analysis of manifests in CI pipelines to verify if they meet the requirements of the different Policy levels.
We are using GitOps for deployment so our pipeline cannot authenticate to the clusters. This prevents us to use apply dry-run.

On slack I've received some solutions for the problem so I thought I will share them here:

@liggitt shared a branch in his fork that can be used to build the psscheck cli tool which does this:

git clone --branch psscheck-example https://github.com/liggitt/kubernetes.git
cd kubernetes
make psscheck
./_output/local/go/bin/psscheck --help
./_output/local/go/bin/psscheck ~/tmp/priviledged_pod.yaml
evaluating pod security standards level=restricted, version=latest

/home/zachar/tmp/priviledged_pod.yaml: Pod "priviledged-pod-1" failed:
   privileged: container "centos" must not set securityContext.privileged=true
   allowPrivilegeEscalation != false: container "centos" must set securityContext.allowPrivilegeEscalation=false
   unrestricted capabilities: container "centos" must set securityContext.capabilities.drop=["ALL"]
   runAsNonRoot != true: pod or container "centos" must set securityContext.runAsNonRoot=true
   seccompProfile: pod or container "centos" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost"

got the following tools listed which can do it:
checkov, regula, terrascan, kyverno (plocies here)

/transfer kubernetes


set announce to empty array if not present

This patch prevents torrent-discovery from crashing if there is no announce property defined on the torrent object.

This was a recent regression - good catch! Released as 3.3.1.


Showdown crashes if a safe string is passed as an argument

I'm trying to use ember-cli-showdown together with ember-i18n. ember-i18n wraps the resulting string in Ember.String.htmlSafe.
The Showdown library is not aware of Ember's safe strings so it crashes with text.replace is not a function.

Responded with my concerns and an alternative in the PR

@jasonmit, I cant find the PR to review said alternative. Im having the same issue as @lolmaus lolmaus. Could you please help by pointing me in the right direction?

@javogel https://github.com/gcollazo/ember-cli-showdown/pull/35


Auto Complete With Snowflake Not Working With Create table

Description
Hello, I am new to DBeaver but I absolutely love it, with one exception. The Auto Complete for Tables/Column Names does not seem to work when the SQL is:
CREATE TABLE USER_DB.USER_NAME.RESULT_SET AS
SELECT A.KEY
FROM
PRD_CL_DB.APP_CLDH_DLV.TABLE_A AS A
, USER_DB.USER_NAME.TABLE_B AS B
WHERE
A.KEY = B.KEY;
If I comment out the CREATE TABLE AS then the auto complete works!
It is a simple work around but hoping this could get fixed in a future version.
Before commenting out the CREATE TABLE line if I press CTRL-Space I get "no proposals"
and the messages from  Query Manager look like this:
-- Load procedures [USER_DB, null, A]
-- Load tables [USER_DB, null, A, null]
-- Load procedures [USER_DB, null, A]
-- Load tables [USER_DB, null, A, null]
If I have to guess the Meta grab is only taking into account the USER_DB that is listed in the CREATE line to then try and lookup the tables, alias.
DBeaver Version
Community Edition <IP_ADDRESS>404071654
Operating System
Windows 10
Database and driver
Snowflake (DBeaver net.snowflake.client.jdbc.SnowflakeDriver)
Steps to reproduce
No response
Additional context
No response

Thank you for bug report


Delete removed commit history

Gogs version (or commit ref): <IP_ADDRESS>3
Git version: 1.9.1
Operating system: Ubuntu 12.04
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

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

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

Log gist:

Description
I accidentally added some sensitive data into the source files of one of my repositories ( you can think that I hard-coded my password in the source file). And I removed them from the history by using bfg (it's the same as git-filter-branch as far as I can tell), and then I did a git push -f to my gogs site. After that I cannot find any trace of the removed data from my repository, so far so good.
But then I noticed the old commits are still available on the home page, and by click on the link of these commits, I can still browse the full file tree at that time. By doing so I can still see the removed data. I suppose this is because these data was updated to gogs' database upon each git push, and would not be altered afterwards.
Now how can I totally remove these links so that no one can see the removed data from the web interface?
Additional information: when I remove the sensitive data from git commit history, I followed the guide on Github help documentation here and bfg guide here.

Run git gc in the repos settings and it should go byebye :)
On Aug 23, 2016 06:56, "Joker Qyou"<EMAIL_ADDRESS>wrote:

Gogs version (or commit ref): <IP_ADDRESS>3
Git version: 1.9.1
Operating system: Ubuntu 12.04
Database (use [x]):

PostgreSQL
MySQL
SQLite

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

Yes (provide example URL)
No
Not relevant

Log gist:

Description
I accidentally added some sensitive data into the source files of one of
my repositories ( you can think that I hard-coded my password in the source
file). And I removed them from the history by using bfg
https://rtyley.github.io/bfg-repo-cleaner/ (it's the same as
git-filter-branch as far as I can tell), and then I did a git push -f to
my gogs site. After that I cannot find any trace of the removed data from
my repository, so far so good.
But then I noticed the old commits are still available on the home page,
and by click on the link of these commits, I can still browse the full file
tree at that time. By doing so I can still see the removed data. I suppose
this is because these data was updated to gogs' database upon each git
push, and would not be altered afterwards.
Now how can I totally remove these links so that no one can see the
removed data from the web interface?
Additional information: when I remove the sensitive data from git commit
history, I followed the guide on Github help documentation here
https://help.github.com/articles/remove-sensitive-data/ and bfg guide
here https://rtyley.github.io/bfg-repo-cleaner/.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gogits/gogs/issues/3504, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEgdoyl15durzdcPy2EumtpZIjaHaaObks5qin1dgaJpZM4JqjVJ
.

Yes, we have gc option in admin panel.

No. I did run that git gc task, actually twice. Maybe I did not make myself clear. I'm not saying that the sensitive data is still in my repo, I'm saying it's still on the web interface.
I can see all the old commits from this page:

The commits in the green outline are these I pushed with git push -f, which are new ones. These in the red outline are the old commits. (Yes I did it twice to use bfg tool right)

Although I can see these old commits I don't think it's a problem, I thought they are no longer valid links. But when I click on these links, the page is actually accessible, and I can just browse all the source code at that time point. The sensitive data I removed is still there, not in the repo, bu on the web interface of gogs.
Now this is what I was asking, I want to remove these from the web interface so that no one could ever see them.

You can delete records in action table, but Gogs does not have such ability to detect such force push changes.

So I can manually delete records in action table? I wonder how GitHub
behave if I did a force push.
On Thu, Aug 25, 2016, 01:51 无闻<EMAIL_ADDRESS>wrote:

You can delete records in action table, but Gogs does not have such
ability to detect such force push changes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gogits/gogs/issues/3504#issuecomment-242151962, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABZbs6Klf-eGcSqxkYT-z-2wteX57YuLks5qjISIgaJpZM4JqjVJ
.


Voting ajax

This PR is intended to solve the issue https://github.com/luanfonceca/speakerfight/issues/46.

The "view" do not use "django messages" anymore because all communications are done asynchronously. The same messages were preserved, but now they are sent in a json response.
All client-side code to deal with ajax vote was created in a file called voting.js.
Looking for a more semantic api, the http method used to vote now is POST instead GET.
The flow to a non-logged User to vote is:

He try to vote > He is redirected to login form > He is sent back to the Event Detail Page (allowed to vote)

ps.: As we was talking about this issue in chats, I decided to open this PR and continue any conversation here for the records.

Coverage decreased (-0.21%) to 91.21% when pulling 79a397414f67dbf39c6f6b6e3b3d742f102ba7aa on cacarrara:voting-ajax into ab83389449d26ef2fdb3d036a2b6f55f1d8f17be on luanfonceca:master.

@cacarrara, i dupped the code to keep the non-ajax requests work, there are more ups than downs on this solution, so it is more good for us, really thanks for fixing this issue, it will make a huge difference on events with Call for papers and Voting process in two steps.


stream remains active after pump callback when error does not terminate stream

If a readable stream in a pump chain emits an error but continues to produce data the pump callback will be invoked at the point of error and the stream will continue to run.  The callback will not be re-invoked when/if the streams actually close.  This appears to be due to internal use of end-of-stream which assumes "failure" and closure are coincident.
Sample code here.
A stream that continues after an error is certainly unusual, but it's legal, and pump does not fulfill its responsibility by invoking the callback and shutting down the remaining pipe stages when the stream actually closes or errors at dest in this situation.  I'm not going to argue strongly that it should, but IMO saying it shouldn't isn't trivially justifiable either.  So, your call.

it will call destroy on all streams in this case. try adding that method to your streams in your example.


error install pod file

hi when install pod file
xcode7.3 and swift2.3
[!] There are multiple dependencies with different sources for `ImageSlideshow` in `Podfile`:

- ImageSlideshow/Alamofire (from `https://github.com/zvonicek/ImageSlideshow.git`, branch `swift2.3`)
- ImageSlideshow (~> 0.5)

Hi, please try to remove the ImageSlideshow, '~> 0.5' from your Podfile. ImageSlideshow/Alamofire should also install the ImageSlideshow itself.

remove  ImageSlideshow, '~> 0.5'
and install again but error:
[!] Unable to satisfy the following requirements:

- `AlamofireImage (~> 3.0)` required by `ImageSlideshow/Alamofire (1.1.0)`

None of your spec sources contain a spec satisfying the dependency: `AlamofireImage (~> 3.0)`.

You have either:
 * out-of-date source repos which you can update with `pod repo update`.
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

Note: as of CocoaPods 1.0, `pod repo update` does not happen on `pod install` by default.

thank you update pod repo and The problem was solved

I install when I want pod 'ImageSlideshow/Alamofire', :git => 'https://github.com/zvonicek/ImageSlideshow.git'
Install the latest version of them But I would like to install for swift 2.3
I thank you guide me how can i do it

Hello @zvonicek
I'm trying to use   pod "ImageSlideshow/Alamofire", :path => "../" in my Project but CocaPods reply : [!] No podspec found for ImageSlideshow in ../
this my podFile
Pods for ......
pod 'IQKeyboardManagerSwift'
pod 'CHIPageControl/Jaloro'
pod 'Cosmos', '~> 20.0'
#pod 'ImageSlideshow', '~> 1.8.3'
pod 'ARSLineProgress'
pod 'ReachabilitySwift'
#pod 'DropDown'
pod 'Kingfisher'
pod 'DLRadioButton'
pod 'MessageKit'
pod 'BetterSegmentedControl', '~> 1.3'
pod 'Firebase/Core'
pod 'Firebase/Messaging'
pod 'FirebaseStorage'
pod 'Firebase/Firestore'
pod 'Alamofire', '~> 5.1'
pod "ImageSlideshow/Alamofire", :path => "../"
end
How Can I Solve it ?


njk filter returns empty if there's another async filter

here's the case.
If you have a config: site.filter('x', async () => "hello", true);
and a page A that uses {{ x }}.
If you need to render content from page A into page B: {{ A.page.content }}  (pre rendered content) will work. But {{ A.page.content | njk }} will return empty.
This is useful to render content of RSS feeds, where you don't want A.content (the full rendered page) but still want to resolve the templates.
I think the problem is that the njk plugin/filter should either assume it's async by default or have an option to become so.

I've made some changes and now njk filter is async. Can you test it now?

Yep! It works now! :)


Adds Amazon S3 support to heroku instances

resolves #1210

We need to add the new env variables to the heroku buildpack, is that on your own personal repository @orenyk ?

Yea it is, I'll get those set up. Could you add a short page to the Reservations wiki explaining how to set this stuff up, and also add the new configuration params to the .env.example file and the configuration wiki page, and instructions to the Heroku deployment guide? Thanks!

Edited .env.example file. I'll edit the wiki once we've merged

Looks good, I actually haven't used S3 before but I'll throw something together to test this quickly. I'll also go and add these to the buildpack. I'll also squash this; there might be some issues given your merge commit but I think I can just skip it and everything will work fine fingers crossed.

Oh wait, the buildpack stuff is in this repository (specifically in app.json), whoops! I'll add it to this branch.

Tested this with our Heroku demo site and my newly minted AWS account (with a separate managed user's credentials) and uploading an image worked perfectly. However, removing an image from an equipment model gave the following 500 error:
Errno::ENOENT (No such file or directory @ unlink_internal - /app/public/attachments/equipment_models/photos/31/original/):
  app/controllers/equipment_models_controller.rb:99:in `block in delete_files'
  app/controllers/equipment_models_controller.rb:93:in `each'
  app/controllers/equipment_models_controller.rb:93:in `delete_files'
  app/controllers/equipment_models_controller.rb:105:in `update'

Can you test all of our various uploads for both uploading, replacing, and removing files? I believe it's just equipment model image, equipment model documentation, and app settings favicon but I could be missing something. Once we've resolved the deletion issue and tested all the cases successfully we should be good to merge and update the wiki. Thanks!

Looking into it

Should be fixed now. It looks like the old file deletion code was hard coded with the file system paths (???)

This doesn't actually delete the files, it only removes the association. According to the README, the correct way is to set the attribute to nil and save instead. I'm going to see if that works...

I think this is because we set preserve: true to account for soft-deletion / deactivation of equipment models; it's not really a big deal, it just means that you might have files left publicly accessible after you expect them to be destroyed, but that's probably true regardless of whether or not we're using S3 (edit: verified locally) so I think this is good to go. I'm going to rebase onto the latest master just to make sure everything's happy and then merge.

Ok, tested and merging!


Re-adjust window when line is no longer visible

When you uncheck a line that was very big, and the ones left are much smaller the window stays at the size to include the largest. Would it be possible to have the graph window re-adjust to the changes to better view the smaller line.

Just changed the way the checkboxes work. Graph resizes now. Lines are removed from the dataset and replaced when the checkbox is rechecked.


add system.linux.memory.slab to system metrics

Fixes  https://github.com/open-telemetry/semantic-conventions/issues/531
Related to https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/7417
Changes
Adds the slab memory metric that is only available on Linux systems as system.linux.memory.slab.
Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[x] Change log entry added, according to the guidelines in When to add a changelog entry.

If your PR does not need a change log, start the PR title with [chore]

[x] schema-next.yaml updated with changes to existing conventions.

For the attributes, it references the system.memory.state which will only match the total well-known value.
reclaimable and unreclaimable are additional custom state attributes normally associated with slab memory. For example, the current (0.101.0) version of the opentelemetry-collector-contrib uses system.memory.slab_reclaimable and system.memory.slab_unreclaimable to indicate those states.
Should this metric reference the system.memory.state attributes or create a new registry (e.g system.linux.memory.slab.state) that explicitly indicates the total, reclaimable and unreclaimable states?

Thank's @rogercoll for this PR!

Should this metric reference the system.memory.state attributes or create a new registry (e.g system.linux.memory.slab.state) that explicitly indicates the total, reclaimable and unreclaimable states?

I think it makes sense to have a specific *.state attribute which would include the reclaimable and unreclaimable.
As it was pointed out at https://github.com/open-telemetry/semantic-conventions/issues/522 (from the usage naming convention):

Where appropriate, the sum of usage over all attribute values SHOULD be equal to the limit.

Agreed with @ChrsMark!

I agree with adding a new attribute for this, and even though the best name for that attribute would be system.linux.memory.slab.state, that name may be restricted by #50 (a metric name can't also be a namespace). I'm not sure what other name to give this though, especially because it has to go in the registry when I have a hard time imagining it will ever end up on another metric.
Not sure what to recommend. I'll check in on #50 again to get clarification that this restriction is definitely in place; it's hard to tell from the discussion history exactly where it ended up.

I think it makes sense to have a specific *.state attribute which would include the reclaimable and unreclaimable.

Sounds good to me. To be aligned with the system.memory model, I think the following could work:
Metric name: system.linux.memory.slab.usage
Attributes: system.linux.memory.slab.state (reclaimable and unreclaimable)

Metric name: system.linux.memory.slab.limit

One question: do we actually plan to implement the system.linux.memory.slab.limit as well in the Collector?

I would say so, the current sum of all system.memory.usage states do not equal to the limit which is needed for v1. If this PR moves forward, I can open an issue in the collector to track the implementation.
cc @dmitryax

@rogercoll there are still some failing checks that prevent from merging

@joaopgrassi could you check this one as you have requested changes but I think it's resolved now

@rogercoll there are still some failing checks that prevent from merging

I think the failing checks are unrelated to the PR but to endpoints blocking the CI IP. Error message:
ERROR: 1 dead links found in ./docs/resource/k8s.md !
  [✖] https://www.itu.int/ITU-T/studygroups/com17/oid.html → Status: 0

I can reach the failed endpoint on my end.


Support implicit instantiation of UIWindow and its root view controller from "Main" storyboard

Support implicit instantiation of UIWindow and its root view controller from "Main" storyboard.
If the feature is supported, the following cases are perfectly resolved.

Issue #11
StackOverflow question

Implemented as a new feature for version 0.3.

Nice Yoichi. Will you be adding a code example?

Documented here :)

Very nice Yoichi, thanks.


Update metrics.md

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Changes:

updated translation
fixed translation

Thanks again @ViktorOn!


Fix propagation of NOTIFICATION_VISIBILITY_CHANGED

For CanvasItems, hiding a previously visible node's parent (direct or indirect) currently does not propagate a NOTIFICATION_VISIBILITY_CHANGED notification to the child. This change propagates the notification whenever the child's actual visibility changes as a result of it's parent's change in visibility.

you mean that if a node is visible and has visible children, changing
visibility does not propagate to children?
On Fri, Sep 25, 2015 at 9:43 PM, eska014<EMAIL_ADDRESS>wrote:

For CanvasItems, hiding a previously visible node's parent (direct or
indirect) currently does not propagate a NOTIFICATION_VISIBILITY_CHANGED
notification to the child. This change propagates the notification whenever
the child's actual visibility changes as a result of it's parent's change
in visibility.
You can view, comment on, or merge this pull request online at:
https://github.com/okamstudio/godot/pull/2533
Commit Summary

Fix propagation of NOTIFICATION_VISIBILITY_CHANGED

File Changes

M scene/2d/canvas_item.cpp
https://github.com/okamstudio/godot/pull/2533/files#diff-0 (2)

Patch Links:

https://github.com/okamstudio/godot/pull/2533.patch
https://github.com/okamstudio/godot/pull/2533.diff

—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/pull/2533.

--
OkamStudio


Unclear how to use Search in headless mode

Issue Summary
Just set up Ghost and search is working well in the managed mode, but the content API documentation doesn't seem to reference searching—I'm integrating this with my React site and want to provide a search feature. An old issue about search was closed as complete (and all the other references to search I can find are marked as closed/done) but I can't find out how to access it, what's the standard way to use search for a headless usage of Ghost?
Thanks for the help, and for offering this open source!
Steps to Reproduce
Search for info on how to use search headless
Ghost Version
5.32.0
Node.js Version
16.19.0
How did you install Ghost?
Ubuntu 22.04, docker compose
Database type
MySQL 8
Browser & OS version
No response
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

The documentation doesn't mention this, at least that I can find. Documentation can be buggy; I searched for a while outside of it too, found various discussions and issues that all seemed closed/resolved. There are two old issues that mention it, both are closed, and one is locked.
I'll ask in the forum, but @ErisDS it really would be nice if the docs were clear on this. If the feature isn't supported and there's not parity here, that's fine, but it'd be good to know when evaluating this for headless use.

If you have less than about 10,000 posts, you could have a template that writes out the titles you want to search during build time. The search engine is then just essentially searching and filtering through that data structure of titles and possibly posts.
You could also use the Content API to grab all the titles and cache for a period on the client and again, the search function could essentially become a client-side filtering function.
I just tested keeping the Developer Tools open while using the on-site search and... it makes no network requests. That suggests, the post titles are already there on the client side, so Ghost itself appears to be working like that-- filtering through content that's already on the client.
The beauty of this design is that's a perfect match for a "headless" blog, because no API calls back to the Ghost server are actually needed. It could be used even on a statically hosted blog.
Even if search can be implemented using the existing Content API, I agree it could helpful to provide some guidance there, since people might be looking for a "Search API".


GooglePlus decoder needed

I believe the default decoder provided by RAuth.utils is incompatible with GooglePlus via OAuth2 decoding schema.  Default RAuth decoder when used by GooglePlus, results in an empty dict {}.
I used this following decoder which seems to work just fine as I passed this into RAuth's get_auth_session() and got the desired tokens.
@staticmethod
def decoder(data: bytes, encoding: str='utf-8') -> dict:
    """
    Provide a decoder for RAuth to invoke to unpack the response and decode accordingly

    :param data: bytes
        Byte data stream that needs to be decoded (handled by this decoder)
    :param encoding: str
        Encoding for the bytes

    :return:
        Dictionary with the values returned by Google Plus

    :rtype: dict
    """
    result = data.decode(encoding)
    if result:
        json = loads(result)
        return json
    else:
        return {}

Seems to be the same for Yahoo OAuth2. I will try this one and keep you posted


[BAAI] support framework flagscale-2409 & support case llava1.5-7B-flagscale

Replace mirror.baidu.com with tuna.tsinghua.edu.cn: at Aug. 24, most libs are not available via mirror.baidu.com


Grafana API does not export panel annotations

After creating a panel that consumes this datasource, when I use the Grafana API and call /api/dashboards/uid/{uid}, I get everything dashboard details, but I don't get annotations for any panels that are using this data source, so if I delete thedashboard and want to recreate it, the JSON API panels won't have the field extractions set, I have to manually do it afterwards.

Turns out after updating both the plugin and Grafana, the issue solved itself. lol


Use CTRL+SHIFT+P then search for the command name instead of using shortcuts

I am used to CTRL+SHIFT+P then search for the command name instead of using shortcuts.
Also this way, I can still use the default shortcut bindings of vscode

Duplicate of #2


Fix court string matching with whitespace

As discussed in https://github.com/freelawproject/eyecite/issues/135#issuecomment-1386402484, there is presently a bug where court strings without whitespace are not properly matched. 3b2fe0972ebb86fad1a0beaa9942980398f38cbc implements a failing test for this bug. 94b1e2f16ddbd8005a6c5905a1621850c4adac9c implements a simple fix.
This PR is also related to the changes proposed in https://github.com/freelawproject/eyecite/pull/129, but I think that that proposal has been made obsolete with the removal of all the duplicate citation strings by @flooie (https://github.com/freelawproject/eyecite/issues/135#issuecomment-1404139512). In any case, this PR addresses a different problem re whitespace.
Note that this PR is based off of #143 (needed to update black to make GitHub Actions happy), so that should be merged first.

@flooie looks like this one never got your approval. Mind taking a look, please?

That'd be great, thanks @mattdahl

Thanks for merging that other PR, @flooie. I just rebased this one as well.
N.B., I previously suggested that #129 had been made obsolete by intervening changes. This is false. More notes over there.

Thanks.  I'll take a look soon.

I added a few lines because if you test it with "Commonwealth v. Muniz, 164 A.3d 1189 (Pa. 2017)" as mentioned in the issue, it now returns "njcirctpassaic" instead of "pa", this happens because the first court in the list returned by courts-db that starts with "Pa" is "Passaic Cty. Cir. Ct., N.J."
I added a check to look for an exact match before try the startswith approach, for this specific case this approach works.

Thanks, I agree that looking for an exact match first is good (cf. https://github.com/freelawproject/eyecite/pull/129#issuecomment-1731981318).
But can we refactor this so we only iterate over courts once?
Something like:
def get_court_by_paren(paren_string: str) -> Optional[str]:
    """Takes the citation string, usually something like "2d Cir", and maps
    that back to the court code.

    Does not work on SCOTUS, since that court lacks parentheticals, and
    needs to be handled after disambiguation has been completed.
    """
    court_str = strip_punct(paren_string)
    court_str = court_str.replace(" ", "")

    court_code = None
    if court_str:
        for court in courts:
            s = strip_punct(court["citation_string"]).replace(" ", "")

            # Check for an exact match first
            if s == court_str:
                return court["id"]

            # If no exact match, try to record a startswith match for possible eventual return
            if s.startswith(court_str):
                court_code = court["id"]

        return court_code

Thanks, I agree that looking for an exact match first is good (cf. #129 (comment)).
But can we refactor this so we only iterate over courts once?
Something like:
def get_court_by_paren(paren_string: str) -> Optional[str]:
    """Takes the citation string, usually something like "2d Cir", and maps
    that back to the court code.

    Does not work on SCOTUS, since that court lacks parentheticals, and
    needs to be handled after disambiguation has been completed.
    """
    court_str = strip_punct(paren_string)
    court_str = court_str.replace(" ", "")

    court_code = None
    if court_str:
        for court in courts:
            s = strip_punct(court["citation_string"]).replace(" ", "")

            # Check for an exact match first
            if s == court_str:
                return court["id"]

            # If no exact match, try to record a startswith match for possible eventual return
            if s.startswith(court_str):
                court_code = court["id"]

        return court_code

great, thanks, it looks more readable, I'll push this change

There is a new PR for all checks to pass correctly: https://github.com/freelawproject/eyecite/pull/188


[keyauth] Add auto expiration to keys

For example, to auto-expire in 2d or 1y since the creation date.

+1

+1

+1

+1

+1

Hi,
As a company looking into api gateways, this would much improve our use case for kong.
Is there any progress on this?

+1
Is there any progress on this?

+1

+1
I know that on OAuth plugin there is an expiration, but it could be very usefull on other Authentications

+1

An implementation of this has been proposed at #4239. It leverages the native TTL support of our new DAO implementation, which means that TTLs are specified in seconds with this first pass.

+1

I am glad to close this PR now on behalf of @eskerda who fixed it by submitting a PR #4984. It took some four and half years to get this in, but we finally did, ;-). Perhaps it was worth to wait, like all the good stuff. It feels better if it just doesn't come given. Haha... well thanks again @eskerda! Closing this now.

@bungle is this feature documented somewhere? I can see the ttl field present in Kong API response but don't know how to set it.


Put robots.txt back in eyre :/

Upon review, /robots.txt is actually a magic web filename, which shouldn't interact with the tree frontend etc.
This PR keep the config change from "DENY" to "ALLOW"

Don't we have, like, tree hiding magic? This seems dubious without some other way for a user to edit their robots.txt

looked around for tree hiding magic, couldn't find it


feat(blog): adding systemd blog

Signed-off-by: Kris Nóva<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface

/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog
/area documentation

/area videos

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
had this question a lot recently - just adding my way of doing things if we are interested in accepting

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign kris-nova after the PR has been reviewed.
You can assign the PR to them by writing /assign @kris-nova in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leodido]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leodido]

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

LGTM label has been added.  Git tree hash: 81a0bf5d38ca99ff0a7634f35c3caf170b8f78a8

LGTM label has been added.  Git tree hash: 81a0bf5d38ca99ff0a7634f35c3caf170b8f78a8

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, leodido, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leodido,leogr]

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

i am fairly certain my current twitter account settings do not let someone mention me in a post - but just in case - please do not mention my twitter in this post - thanks for the quick merge


Branch cleanup, chat example Dail error

I download the branch cleanup, run the chat example in internet server, the server have public internet ip, such as <IP_ADDRESS>:3000
and run another node in my local pc. chat <IP_ADDRESS>:3000
take a error:
panic: connection could not be identified: failed to ping peer: rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: authentication handshake failed: connected to peer with addr <IP_ADDRESS>:3000, but their id writes addr <IP_ADDRESS>:3000"

If I run the branch origin/master chat example, it will work ok.

In the intranet, no a same PC, also have this error.

When I turn on the grpc log(using environment variable GRPC_GO_LOG_VERBOSITY_LEVEL=99 GRPC_GO_LOG_SEVERITY_LEVEL=info),the detail information is:
Listening for peers on port: 34327
INFO: 2020/01/07 23:31:37 parsed scheme: ""
INFO: 2020/01/07 23:31:37 scheme "" not registered, fallback to default scheme
INFO: 2020/01/07 23:31:37 ccResolverWrapper: sending update to cc: {[{<IP_ADDRESS>:3000  <nil> 0 <nil>}] <nil> <nil>}
INFO: 2020/01/07 23:31:37 ClientConn switching balancer to "pick_first"
INFO: 2020/01/07 23:31:37 blockingPicker: the picked transport is not ready, loop back to repick
INFO: 2020/01/07 23:31:37 pickfirstBalancer: HandleSubConnStateChange: 0xc000126150, {CONNECTING <nil>}
WARNING: 2020/01/07 23:31:37 grpc: addrConn.createTransport failed to connect to {<IP_ADDRESS>:3000  <nil> 0 <nil>}. Err :connection error: desc = "transport: authentication handshake failed: connected to peer with addr <IP_ADDRESS>:3000, but their id writes addr <IP_ADDRESS>:3000". Reconnecting...
INFO: 2020/01/07 23:31:37 pickfirstBalancer: HandleSubConnStateChange: 0xc000126150, {TRANSIENT_FAILURE connection error: desc = "transport: authentication handshake failed: connected to peer with addr <IP_ADDRESS>:3000, but their id writes addr <IP_ADDRESS>:3000"}
panic: 2.connection could not be identified: failed to ping peer: rpc error: code = Unavailable desc = connection error: desc = "transport: authentication handshake failed: connected to peer with addr <IP_ADDRESS>:3000, but their id writes addr <IP_ADDRESS>:3000"


css variables don't work inline

Trying to use css variables inline doesn't work.
<link rel="import" href="../../bower_components/paper-styles/paper-styles.html">
...

The result I need to resort to making css classes which look like (taken from the neon-elements 'shared.css')
.red-100 {
  background: var(--google-red-100);
}

.yellow-100 {
  background: var(--google-yellow-100);
}

Which works but is kind of a let down because I thought variables and mixins where suppose to stop us from having to define classes like this.

Using variables directly in the style attribute is not something that's supported by Polymer's custom properties shim. Polymer's support for custom properties is primarily designed to provide a way for elements to expose a styling api which can be used in any styling scope independent of local dom boundaries.
This shim has other limitations as well, see Styling local DOM for more information.


Support turning caching off on custom_task

See: https://github.com/latchbio/latch/blob/f01232889928b1ad11430458239e6e7cb7e0e1b3/latch/resources/tasks.py#L365

makes sense, should be pretty quick to do. Want to make a PR?

I would if it was immediately obvious to me. But it’s not. Care to take a shot?


Pass -index-file-path as a primary file input

-index-file uses single compiles, but the -index-file-path needs to be passed as a -primary-file input. It looks like all the other indexing flags were already being handled correctly.
This fixes the remaining index store tests:
Index/Store/driver-index-with-auxiliary-outputs.swift
Index/Store/driver-index.swift
Index/Store/driver-multiple-inputs.swift
Index/Store/unit-one-file-multi-file-invocation.swift

@swift-ci test

@swift-ci test

I remember seeing some TODOs about handing index file paths when I was working on the file list stuff.

@cltnschlosser I just saw the FIXME in addPlatformSpecificLinkerArgs for relation to -index-file-path, but I'm not sure we need special handling there since, as far as I know, we'll never be doing any linking when indexing. It also looks like the cpp driver has some special case handling of file lists, but so far it doesn't seem to be necessary here, I'm looking into that more to see if there are any behavior differences.

@swift-ci test


[WIP] Hide the art container if images are disabled

Extremely minor code change, but it does look quite nice (in my opinion). Most of my app usage is on mobile data and browsing the forums is a bit of a pain because of all the wasted space in a post if avatars are disabled.
With this change, if you then enable images, it makes the text view feel very small again. I might experiment with text wrapping around the avatar, or starting the text below the avatar or perhaps moving the avatar entirely.
Let me know what you guys think.

Cool! Does this handle the case where images may be disabled/enabled while in the view? For example the user is viewing some comments with images disabled, goes into settings and enables images, do we properly re-show the view?
Having the text wrap around the avatar was the solution I was hoping to put together but didn't have time to come back to it. I think having the text below the avatar might leave too much empty space since next to the avatar will all be blank. If there's a better way to layout the comments I'm open to ideas though.

Since it only checks if (imagesEnabled) when the activity loads, the updated setting will only be visible when loading another activity or reloading the current one. So in the example of the forums, if they were in a thread and changed the setting, they would have to leave the thread and enter it again to see the change.
As for the wrapping, its likely we would have to use something like this. I do hate adding dependancies, but it looks reasonably promising. I'll give it a go anyway, to see how it looks. I'll post results :)

Manually merged in the remaining changes from this PR to avoid dealing with more merge conflicts :stuck_out_tongue: . Thanks @stuxo


Flatten arrays with one value when normalizing item

My usecase is the following:
I have a questionnaire which is translated using the PrezentDoctrineTranslatableBundle. My queryBuilder is the following:
$this->createQueryBuilder('q')
            ->select('q, qt')
            ->join('q.translations', 'qt')
            ->where('qt.locale = :locale')
            ->orderBy('qt.name')
            ->setParameter('locale', $locale);

The result, after 'getArrayResult()':
array (size=2)
  'id' => int 4
  'translations' =>
    array (size=1)
      'en' =>
        array (size=4)
          'name' => string 'asdfa' (length=5)
          'description' => string '' (length=0)
          'id' => int 6
          'locale' => string 'en' (length=2)

Note that the 'translations' array, is not numerical, but has the locale as its key. This is because the PrezentTranslatable uses an indexBy strategy on 'locale', on the relation between the translatable entity and its translations.
Because of this, the normalized item comes out like:
array (size=2)
  'q.id' => int 4
  'qt.en' =>
    array (size=4)
      'name' => string 'asdfa' (length=5)
      'description' => string '' (length=0)
      'id' => int 6
      'locale' => string 'en' (length=2)

instead of
array (size=5)
  'q.id' => int 4
  'qt.name' => string 'asdfa' (length=5)
  'qt.description' => string '' (length=0)
  'qt.id' => int 6
  'qt.locale' => string 'en' (length=2)

This causes the grid to not display a value for i.e. 'qt.name', and also makes it impossible to filter and sort on this field.
The reason for this is that the normalizeOneItem function, only flattens an array if its key is '0'. I thought about this, and I think the same can be done for all arrays with only one value, even if they have an alphanumerical key.
I'm just not sure if this has any side-effects. I haven't encountered any yet, but thats not saying anything, of course.

Hi,
@eliecharra is proposing a PR #47 with a solution much more elegant than my StandardNormalizer.
It builds the normalized array with a custom Doctrine hydrator instead of a normalizer after an array hydrator.
It solves your issue, cleans the code and makes it much more extendable.
It's PR is not finished, but I'm pretty sure it is the best way to solve the problem. Thanks a lot for you PR and your work to solve the problem, but I believe this PR is not useful anymore.
Best regards,
Philippe


Create migrations for Regulation

?? 🤔😅

?? 🤔😅

@danielmursa LGTM == Looks Good To Me, standard PR approval message 😛


Fix Promtail and Loki errors to get logging in grafana

Data source connected, but no labels received. Verify that Loki and Promtail is configured properly.

Fixed in this commit: https://github.com/jkulzer/kube/commit/d196ac32d91ba43d0f032c73285ed15ee1a94eb6


can't create certificate

Bugs should be filed for issues encountered whilst operating cert-manager.
You should first attempt to resolve your issues through the community support
channels, e.g. Slack, in order to rule out individual configuration errors.
Please provide as much detail as possible.
apiVersion: cert-manager.io/v1alpha2
kind: Certificate
metadata:
name: ag-cert
spec:
secretName: ag-cert-secret
dnsNames:
- ns15.xxx.com
- ns16.xxx.com
acme:
config:
- http01:
ingressClass: nginx
domains:
- www.xxx.com
- api.xxx.com
issuerRef:
name: letsencrypt-prod
# We can reference ClusterIssuers by changing the kind here.
# The default value is Issuer (i.e. a locally namespaced Issuer)
kind: ClusterIssuer

Describe the bug:
A clear and concise description of what the bug is.
kubectl logs certificate/ag-cert
error: no kind "Certificate" is registered for version "cert-manager.io/v1alpha2" in scheme "k8s.io/kubectl/pkg/scheme/scheme.go:28"
Expected behaviour:
valid certificate registration
A concise description of what you expected to happen.
Steps to reproduce the bug:
Steps to reproduce the bug should be clear and easily reproducible to help people
gain an understanding of the problem.
Anything else we need to know?:
Environment details::
Ubuntu 18.04

Kubernetes version (e.g. v1.10.2):
1.16.1
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc):
cert-manager version (e.g. v0.4.0):
0.11.0
Install method (e.g. helm or static manifests):
helm
/kind bug

@munnerz Yes. I did. I used commend from doc:
Install the CustomResourceDefinitions and cert-manager itself
kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v0.11.0/cert-manager.yaml
If this command is missed CRD creation that is a bug in script or in doc.
How to fix it?