Build warning on latest Next.js

I'm getting the following build warning when running github.com/ada-url/website
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Parsing of /Users/yagiz/Developer/ada/ada-url.com/node_modules/.pnpm/@contentlayer+core@0.3.3_esbuild@0.17.19_markdown-wasm<EMAIL_ADDRESS>for build dependencies failed at 'import(`file://${compiledConfigPath}`)'.
<w> Build dependencies behind this expression are ignored and might cause incorrect cache invalidation.
Generated 10 documents in .contentlayer
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Parsing of /Users/yagiz/Developer/ada/ada-url.com/node_modules/.pnpm/@contentlayer+core@0.3.3_esbuild@0.17.19_markdown-wasm<EMAIL_ADDRESS>for build dependencies failed at 'import(`file://${compiledConfigPath}`)'.
<w> Build dependencies behind this expression are ignored and might cause incorrect cache invalidation.
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Parsing of /Users/yagiz/Developer/ada/ada-url.com/node_modules/.pnpm/@contentlayer+core@0.3.3_esbuild@0.17.19_markdown-wasm<EMAIL_ADDRESS>for build dependencies failed at 'import(`file://${modulePath}?x=${Date.now()}`)'.
<w> Build dependencies behind this expression are ignored and might cause incorrect cache invalidation.
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Parsing of /Users/yagiz/Developer/ada/ada-url.com/node_modules/.pnpm/@contentlayer+core@0.3.3_esbuild@0.17.19_markdown-wasm<EMAIL_ADDRESS>for build dependencies failed at 'import(`file://${modulePath}?x=${Date.now()}`)'.
<w> Build dependencies behind this expression are ignored and might cause incorrect cache invalidation.
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Parsing of /Users/yagiz/Developer/ada/ada-url.com/node_modules/.pnpm/@contentlayer+core@0.3.3_esbuild@0.17.19_markdown-wasm<EMAIL_ADDRESS>for build dependencies failed at 'import(filePathJoin(generatedPkgPath, 'generated', 'index.mjs'))'.
<w> Build dependencies behind this expression are ignored and might cause incorrect cache invalidation.
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Parsing of /Users/yagiz/Developer/ada/ada-url.com/node_modules/.pnpm/@contentlayer+core@0.3.3_esbuild@0.17.19_markdown-wasm<EMAIL_ADDRESS>for build dependencies failed at 'import(filePathJoin(generatedPkgPath, 'generated', 'index.mjs'))'.
<w> Build dependencies behind this expression are ignored and might cause incorrect cache invalidation.

Thanks for reporting this issue. I think it's a duplicate of https://github.com/contentlayerdev/contentlayer/issues/272


Can't import non-libraries from server.ts

Have you experienced this bug with the latest version of the template?
yes
Steps to Reproduce

run npx create-remix --template remix-run/blues-stack
create app/foo.ts in base directory with contents export const bar = 'baz'
add import { bar } from './foo' to server.ts
run npm run setup && npm run build && npm run start

Expected Behavior
✅ app ready: http://localhost:3000
Actual Behavior
Error: Cannot find module '~/foo'

See https://github.com/remix-run/blues-stack/pull/84 (cc @mcansh )


Request: Provide kubernetes deployment yaml file

It would be better to have kubernetes deployment yaml file

You can feel free to implement that part and make a pull request.


Can I use this bot for my project?

This bot is great for welcoming first timers. Can I integrate this bot into my project ? If it does not cost you anything.
I have no idea "How to create a github bot". So, I want to use this one.

I am closing this issue. because. This project has MIT License. So, I can use it. if you don't want me to use it. Just reopen this issue.
Thanks for your time

Yes no cost to me or you for public projects. Example usage in the README.md, should mostly be copy and paste 🤓


[ci] Divide .gitlab-ci.yml into several files

Attempt to divide .gitlab-ci.yml into several files.
Part of https://github.com/paritytech/ci_cd/issues/325

Merge after https://github.com/paritytech/substrate/pull/11384


Java program to left rotate an array

Added a java program to left rotate an array by certain value taken as input.
#294
Please make it under Hacktober Fest 2021 .

I want this to be assigned. Please make it. @Astrodevil @harshitsrivastava1608


How to debug the built version with angular-cli

Versions
Angular CLI: 1.5.0
Node: 9.0.0
OS: win32 x64
Angular: 5.0.1
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.5.0
@angular-devkit/build-optimizer: 0.0.32
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.35
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0
@schematics/angular: 0.1.2
typescript: 2.4.2
webpack: 3.8.1

Repro steps

ng build

Observed behavior
All files are minified and cannot debug the production version

Desired behavior
Is there an option to map ts files in order to debug the prod version?
The desired behavior is that it would be great if we can track bugs in the production version.

 Thank you! that is the trick!


Display referenced non-exported type symbols

<EMAIL_ADDRESS><EMAIL_ADDRESS>https://deno.land/x/masx200_leetcode_test@8.2.0/design-skiplist/index.ts?code
interface Skiplist {
    search: (target: number) => boolean;
    add: (num: number) => void;
    erase: (num: number) => boolean;
}

function Skiplist(): Skiplist {
    const storage = new Map<number, number>();

    function search(target: number): boolean {
        return storage.has(target) && (storage.get(target) || 0) > 0;
    }

    function add(num: number): void {
        storage.set(num, 1 + (storage.get(num) || 0));
    }

    function erase(num: number): boolean {
        if (storage.get(num) === 1) {
            storage.delete(num);

            return true;
        }
        if (search(num)) {
            storage.set(num, -1 + (storage.get(num) || 0));
            return true;
        } else {
            return false;
        }
    }
    return { search, add, erase };
}

export default Skiplist;

Interfaces can't be exported as a default export of a module, and because interface SkipList is not exported from the source module, the documentation does not display it, as we only display exported symbols.
We need to consider how to deal with "private" type annotations though when dealing with doc node entries, because TypeScript and language services would allow going to those definitions even if they don't exist.  In the view above, we would expect to be able to link the the SkipList as a symbol, even though it isn't exported.
The easiest current work around would be to export the interface, and then it would be available in the documentation.

Another example:
declare module Test {
  export interface Options {
  }
}

export function test(options: Test.Options): void;

Outputs:
Defined in file:///.../mod.d.ts:6:0

function test(options: Test.Options): void


Update for Tiled API change.

In the latest Tiled release, get_data_sources() changes its return type from list[dict] to list[DataSource] (a dataclass). Correspondingly, here in the TiledWriter, we need to update the usage from data_source["id"] to data_source.id. With the change, tests pass.

Interesting, CI passed here but failed on main.

very interesting indeed

the reason is in this snippet: `    def stream_datum(self, doc: StreamDatum):
# Get the Stream Resource node and the associtaed handler (consolidator)
sres_node, handler = self.get_sres_node(doc["stream_resource"], desc_uid=doc["descriptor"])
handler.consume_stream_datum(doc)
    # Update StreamResource node in Tiled
    # NOTE: Assigning data_source.id in the object and passing it in http
    # params is superfluous, but it is currently required by Tiled.
    sres_node.refresh()
    data_source = handler.get_data_source()
    data_source.id = sres_node.data_sources()[0]["id"]  # ID of the existing DataSource record
    endpoint = sres_node.uri.replace("/metadata/", "/data_source/", 1)
    handle_error(
        sres_node.context.http_client.put(
            endpoint,
            content=safe_json_dump({"data_source": data_source}),
            params={"data_source": data_source.id},
        )
    ).json()`

    `
    the `sres_node.data_sources()` does not return a `list[DataSource]`, even if the `handler.get_data_sources()` does.

    @danielballan what to do about this? this is blocking #1636 , I'd like to open a PR based on this issue and fix this, just what is the desired behavior?


Error with permissions

kubectl get hpa
sqs-consumer-scaler       Deployment/<>   /30 (avg)             1         10        1          4h
Warning   FailedComputeMetricsReplicas   horizontal-pod-autoscaler                               failed to get <<name>> external metric: unable to get external metric default/<<name>>/nil: unable to fetch metrics from external metrics API: <<name>>.external.metrics.k8s.io is forbidden: User "system:anonymous" cannot list resource "<<resource>>" in API group "external.metrics.k8s.io" in the namespace "default"

looks like it is missing some permissions, did you deploy the full adapter.yaml? There are service account, role bindings defined.

Yes, I did.
Here are role bindings and roles:
k8s-cloudwatch-adapter-resource-reader                 6d
k8s-cloudwatch-adapter:crd-metrics-reader              6d
k8s-cloudwatch-adapter:external-metrics-reader         6d
k8s-cloudwatch-adapter:system:auth-delegator           6d

We experienced this issue as well. EKS with K8s 1.14. We reverted to d385146da902433cb0b8466658f943728e3a6bce as that had worked for us in the past with the same setup and it resolved the issue.

Hi @joesbigidea do you mean it worked fine with v0.8.0?

That's correct, the head version with 0.9.0 gave us this issue:
  Warning  FailedComputeMetricsReplicas  1s    horizontal-pod-autoscaler  failed to get external metric <<name>>: unable to get external metric custom-metrics/<<name>>/nil: unable to fetch metrics from external metrics API: <<name>>.external.metrics.k8s.io is forbidden: User "system:anonymous" cannot list resource "<<name>>" in API group "external.metrics.k8s.io" in the namespace "custom-metrics"
Reverting to the version with v0.8.0 resolved the issue. It's working fine with that version.

CreationTimestamp:                                                     Fri, 07 Aug 2020 13:13:42 -0700
Reference:                                                             Deployment/server-events
Metrics:                                                               ( current / target )
  "name1-test-lowpriority-q-length" (target average value):  <unknown> / 30
Min replicas:                                                          1
Max replicas:                                                          10
Deployment pods:                                                       1 current / 0 desired
Conditions:
  Type           Status  Reason                   Message
  ----           ------  ------                   -------
  AbleToScale    True    SucceededGetScale        the HPA controller was able to get the target's current scale
  ScalingActive  False   FailedGetExternalMetric  the HPA was unable to compute the replica count: unable to get external metric default/name1-test-lowpriority-q-length/nil: unable to fetch metrics from external metrics API: MissingEndpoint: 'Endpoint' configuration is required for this service
Events:
  Type     Reason                   Age                From                       Message
  ----     ------                   ----               ----                       -------
  Warning  FailedGetExternalMetric  31m (x9 over 35m)  horizontal-pod-autoscaler  unable to get external metric default/name1-test-lowpriority-q-length/nil: unable to fetch metrics from external metrics API: InvalidEndpointURL: invalid endpoint uri
caused by: parse https://monitoring.request blocked by whitelist-route-regexp "^$": /latest/meta-data/placement/availability-zone/.amazonaws.com/: invalid port ": " after host
  Warning  FailedComputeMetricsReplicas  31m (x9 over 35m)  horizontal-pod-autoscaler  failed to get name1-test-lowpriority-q-length external metric: unable to get external metric default/name1-test-lowpriority-q-length/nil: unable to fetch metrics from external metrics API: InvalidEndpointURL: invalid endpoint uri
caused by: parse https://monitoring.request blocked by whitelist-route-regexp "^$": /latest/meta-data/placement/availability-zone/.amazonaws.com/: invalid port ": " after host
  Warning  FailedComputeMetricsReplicas  29m (x4 over 31m)   horizontal-pod-autoscaler  failed to get name1-test-lowpriority-q-length external metric: unable to get external metric default/name1-test-lowpriority-q-length/nil: unable to fetch metrics from external metrics API: name1-test-lowpriority-q-length.external.metrics.k8s.io is forbidden: User "system:anonymous" cannot list resource "name1-test-lowpriority-q-length" in API group "external.metrics.k8s.io" in the namespace "default"
  Warning  FailedGetExternalMetric       10m (x36 over 28m)  horizontal-pod-autoscaler  unable to get external metric default/name1-test-lowpriority-q-length/nil: unable to fetch metrics from external metrics API: the server is currently unable to handle the request (get name1-test-lowpriority-q-length.external.metrics.k8s.io)
  Warning  FailedGetExternalMetric       5m (x14 over 31m)   horizontal-pod-autoscaler  unable to get external metric default/name1-test-lowpriority-q-length/nil: unable to fetch metrics from external metrics API: name1-test-lowpriority-q-length.external.metrics.k8s.io is forbidden: User "system:anonymous" cannot list resource "name1-test-lowpriority-q-length" in API group "external.metrics.k8s.io" in the namespace "default"
  Warning  FailedGetExternalMetric       34s (x9 over 4m)    horizontal-pod-autoscaler  unable to get external metric default/name1-test-lowpriority-q-length/nil: unable to fetch metrics from external metrics API: MissingEndpoint: 'Endpoint' configuration is required for this service
```

Digging deeper looks like kiam is blocking the requests. I have created image with curl which gave me hints on what's happening. Now taking it again from the top.


How to create video fact sheets ?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I use --print-to-file to create a fact sheet for each downloaded video.
Is there currently any more efficient way to achieve this than repeating the "really-long-filename-template" over and over ?
--print-to-file "title:                  %(title)s" ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"
--print-to-file "fulltitle:              %(fulltitle)s" ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"
--print-to-file "alt_title:              %(alt_title)s" ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"
--print-to-file "format:                 %(format)s" ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"
--print-to-file "vcodec:                 %(vcodec)s" ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"
--print-to-file "resolution:             %(resolution)s" ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"
--print-to-file "duration:               %(duration>%H:%M:%S)s" ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"
--print-to-file "filesize:               %(filesize)s" ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"
etc.
etc.
...

Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
n/a

You could combine them into a single command.

--print-to-file 'title:                  %(title)s
...
filesize:               %(filesize)s' ".\%(playlist|)s...].factsheet.nfo"

Or, you could save the filename template in an alias

--alias meta-name ".\%(playlist|)s...].factsheet.nfo"

--print-to-file "title:                  %(title)s" --meta-name
...

Config files are parsed like a POSIX shell. So the same syntax for continuations work.
The extra new lines look like some issue with CRLF settings. Difficult to say exactly without access to the files.

alternatively you could do:
--alias print-meta '--print-to-file {} ".\%(playlist|)s\%(playlist_index|)02d%(playlist_index& = |)s%(release_date>%Y-%m-%d,release_timestamp>%Y-%m-%d,upload_date>%Y-%m-%d,timestamp>%Y-%m-%d|#nodate#)s %(alt_title,title)s\!nfo\%(title)s [%(extractor)s.%(extractor_key)s.%(id)s] [%(format_id)s].factsheet.nfo"'
--print-meta "title:                  %(title)s"
--print-meta "fulltitle:              %(fulltitle)s"
# etc...

Ah, my bad. Aliases are not parsed inside the arguments. Use bashonly's solution

https://github.com/yt-dlp/yt-dlp/issues/6808#issuecomment-1509361107 I can't seem to reproduce this.

Aliases are not parsed inside the arguments.

Does that mean, that aliases can only be used at the start of an instruction, i.e. at the beginning of a line ?

Yes. Once defined, an alias is just like any other switch.

#6808 (comment) I can't seem to reproduce this.

nvm, I can repro and I'll push a fix.

You could use --parse-metadata to save it into a field, but that won't work for -o with subdirs

That is not a supported %() syntax. I think this is what you want:
"%(playlist_title&playlist_title:         |)s%(playlist_title)s
%(playlist&playlist                |)s%(playlist)s
%(playlist_id&playlist_id:            |)s%(playlist_id)s"

The problem is not the new lines, but that what you are giving is not a valid output template. There is no support for nested %(... %(...)s)s

I can't test atm. But I think this should work?
%(playlist&
playlist:          |)s%(playlist)s

You still need to do line by line, but now newlines can be properly removed. Something like:
%(playlist_index&--------------- PLAYLIST -------------------------------|)s
%(playlist&
playlist title:        {}|)s
...

a) I have not changed the right column, so why all of a sudden the ~~ occur unexpectedly in the second setup, while they don't appear in the first? I would expect null output ("") in both setups due to the | override.

the ~~ appears in the second example because the left column does not have a | override:
%(playlist&playlist_title:       )s

b) In the second setup in the last 2 lines, why are the ~~ before the slash / but not after ?

same reason

try this
--print "%(playlist&--------------- PLAYLIST -------------------------------|)s%(playlist&
playlist:             {}|)s%(playlist_id&
playlist_id:          {}|)s%(playlist_uploader&
playlist uploader:    {}|)s%(playlist_uploader_id&
playlist uploader id: {}|)s%(playlist_index&
playlist index:       {}|)s%(playlist_count& / {}|)s%(playlist_autonumber&
playlist autonumber:  {}|)s%(n_entries& / {}|)s%(playlist&
---------------------------------------------------------|)s"


Upgrade lodash to 3.2.0

Are you sure we can merge it? It seems there are some breaking changes. We should check if our apps don't break first.


TechDocs: Improve techdocs-common APIs for it to be used by techdocs-cli

Generator should optionally take an output directory as argument.
Generator may not get techdocs-ref annotation since that is not critical for the step.

Needed for the incoming improvements in TechDocs CLI https://github.com/backstage/techdocs-cli/pull/20

@backstage/maintainers I would like to merge this one before the release this week. It's a dependency for the techdocs-cli work. Thank you!

@backstage/maintainers I would like to merge this one before the release this week. It's a dependency for the techdocs-cli work. Thank you!


Update requirements.txt for event_handler

The requirements file for event_handler includes the lines:
pytest==5.3.0; python_version > "3.0"
pytest==4.6.6; python_version < "3.0"

...since we require python >= 3.6, and since pytest is now at version 6, I propose we update this to:
pytest~=6.0.0

Does that sound okay to you, @dinagraves
(this is what I'm using for the event_handler tests)
Also, we could get faster installs and slimmer prod builds if we isolate testing dependencies. AIUI, PIP doesn't support "dev dependencies" like e.g. npm, but I did find this: https://stackoverflow.com/questions/17803829/how-to-customize-a-requirements-txt-for-multiple-environments

Excellent point.  In python-docs-samples, we have two files: requirements.txt and requirements-test.txt.  I propose we do the same for fourkeys.


dbeaver/pro#2303 Turn off validation for nosql databases

Nothing should be highlighted with red in sql editor for nosql databases (for example, dynamodb, mongodb).
No error should be thrown while editing script.

verified


[Question]:  I have issues when query with Knowledge Graph

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I have code below:
storage_graph = StorageContext.from_defaults(persist_dir = '/content/graph_chatbot',graph_store = graph_store)
kg_index = load_index_from_storage(
storage_context = storage_graph,
service_context = service_context,
max_triplets_per_chunk = 10,
verbose = True
)
kg_query_engine = kg_index.as_query_engine(
include_text=False,
response_mode="tree_summarize", #test response !!!
#embedding_mode="hybrid", #
similarity_top_k=5,
)
query_str = "According to labor code tell me about DOLISA"
response = kg_query_engine.query(query_str)
when it's generate response, it's take too long I use the index previously saved at '/content/graph_chatbot' created and saved from KnowledgeGraphIndex, when I load_index_from_storage right after getting the index, the query at kg_query_engine = kg_index.as_query_engine(
include_text=False,
response_mode="tree_summarize", #test response !!!
#embedding_mode="hybrid", #
similarity_top_k=5,
) didn't take that long but after a while when I tried again it took longer to generate the answer, why ????
Llama.generate: prefix-match hit
Llama.generate: prefix-match hit
Llama.generate: prefix-match hit
Llama.generate: prefix-match hit
Llama.generate: prefix-match hit
Llama.generate: prefix-match hit
Llama.generate: prefix-match hit
Llama.generate: prefix-match hit
It's still like this

LlamaCpp is just pretty slow in general. The more context received and sent to the LLM, the slower it will take.
If I had to guess, it's retrieving quite a lot of text to send to the llm.
You can see what is being sent to the LLM here:
https://docs.llamaindex.ai/en/stable/module_guides/observability/observability.html#simple-llm-inputs-outputs
You can also test and see what is retrieved like this
retriever = index.as_retriever(include_text=False, similarity_top_k=5)
nodes = retriever.retrieve("test")

@dosu-bot when I use from llama_index.memory import ChatMemoryBuffer
memory = ChatMemoryBuffer.from_defaults(token_limit=1500)
chat_engine = ReActAgent.from_tools(
query_engine_tools, llm=llm, memory=memory, verbose=True
)
chat_engine = kg_index.as_chat_engine(
chat_mode="react",
memory=memory,
verbose=True,
) it's still take a long time, it's return too much thing,  it stops at llama_decode() for a long time and returns things like " Relationship properties are the following:\n\nThe relationships are the following:\n(:Entity)-[:IS_INTENDED_FOR]->(:Entity),( :Entity)-[:DESIGNED_TO_ORIENT]->(:Entity),(:Entity)-[:DESIGNED_TO_PROVIDE_GOOD_BASIC_UNDERSTANDING_OF]->(:Entity),(:Entity)-[:WRITTEN_BY]->(:Entity),(:Entity )-[:CURRENT_AS_OF]->(:Entity),(:Entity)-[:WAS_WRITTEN_BY]->(:Entity),(:Entity)-[:REFERS_TO_AMOUNTS_IN]->(:Entity),(:Entity)- [:ISTYPEOF]->(:Entity),(:Entity)-[:COVERS]->(:Entity),(:Entity)-[:SPECIFIES_RIGHTS_AND_DUTIES_OF]->(:Entity),(:Entity)-[: IS_A_SOURCE_OF]->(:Entity),(:Entity)-[:SETS_OUT]->(:Entity),(:Entity)-[:HAS_PROVISION]->(:Entity),(:Entity)-..... "


Update README for setup dev environment with docker

Thank you for the wonderful conference the other day!
For now, I have written the minimum method to get the application up and running. Feedback would be appreciated.

Thank you!


Can't extract pre-train model

Hi, I just follow the instruction to download two pre-train models from Onedrive. However, I can't extract it. And the link from Baidu doesn't exist. Could you reupload the pre-train models again?

I just check that the link from Baidu exists. Also, do you mean extract model.pth.tar?
I am sorry that I name it *.tar, which follow the name habit of many torch code. But the model is not tar compressed file. Just use torch.load('model.pth.tar') can load the model.

@Wilfred-Yang
Merry Christmas! I recently update the Emotion-FAN, new features include data process, environment install, CK+ code, and Baseline code. Also, you can find the old version directory of Emotion-FAN in the README.md. I hope my new updates can help you greatly. Please see the Emotion-FAN for more details.


Blocked calling api from code playground

Hey there.
I am attempting to call gutendex.com using elm in the "Ellie" playground. You can experiment with going to ellie-app.com. Here is the code:
module Main exposing (main)

import Browser
import Html exposing (Html, button, div, text)
import Html.Events exposing (onClick)
import Http
import Json.Decode exposing (Decoder, field, string)

type alias Model =
    { title : Maybe String }

initialModel flags =
    ( { title = Nothing }, getBook )

getBook : Cmd Msg
getBook =
    Http.get
        { url = url
        , expect = Http.expectJson GotBook bookDecoder
        }

bookDecoder : Decoder String
bookDecoder =
    field "title" string

type Msg
    = GotBook (Result Http.Error String) -- start with just title

update : Msg -> Model -> ( Model, Cmd msg )
update msg model =
    case msg of
        GotBook (Ok title) ->
            ( { model | title = Just title }, Cmd.none )

        GotBook (Err e) ->
            ( model, Cmd.none )

view : Model -> Html Msg
view model =
    case model.title of
        Just title ->
            div [] [ text title ]

        Nothing ->
            div [] [ text "Looking..." ]

url =
    "http://gutendex.com/books/2591/"

main : Program () Model Msg
main =
    Browser.element
        { init = initialModel
        , view = view
        , update = update
        , subscriptions = \_ -> Sub.none
        }

However, in dev tools, I see I get a 'Blocked' request. I'm not sure what this means. Is your server setup to block external requests like this?
Thanks.

Hi Ethan, I think the issue is that Gutendex is not an https site, but Ellie is, so you browser blocks the request for security. Sorry! I will look into supporting https requests this weekend.

OK, I think HTTPS and CORS are working now. Would you mind letting me now if it works for you when you get a chance?
By the way, if you want to do more than playground work, I recommend hosting your own version of Gutendex. I might have to restrict access to gutendex.com if I get too much traffic to handle.

Sweet! I'll check it out. Yeah - restricted/throttled access is all I need since it's just for me at the moment. I'll close this issue if I find that it's working for me.

Looks great, thanks a lot!


Ability to search orders by ref and subtotal

This adds the ability to search orders by partial reference and subtotal (net total cost) either using the query param or using the field specific param.

Codecov Report

Merging #564 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #564   +/-   ##
========================================
  Coverage    96.41%   96.41%
========================================
  Files          123      123
  Lines         3631     3631
  Branches       319      319
========================================
  Hits          3501     3501
  Misses          90       90
  Partials        40       40

Impacted Files
Coverage Δ

datahub/search/omis/views.py
100% <ø> (ø)
: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 03014f8...9a2c24b. Read the comment docs.


Update URLs to HTTPS

THis PR updates URLs to HTTPS for security reasons.
P.S. Please update this one.

LGTM, all links still work


Urgent: Follow-up on High-Severity Vulnerabilities Reported

Hello, amit-laish and I reported two high-severity vulnerabilities over a month ago. Please review the submitted vulnerabilities and contact us so we can remediate them.

Who are you talking to? Read our policy on security vulnerability reporting: https://github.com/open-webui/open-webui/blob/main/docs/SECURITY.md#reporting-a-vulnerability
If you're being ignored, you probably didn't follow policy.


fix: PATH_PREFIX not replaced when using user directive

Files were owned by node:node with 755. If you override the user then they won't have permission to replace __PATH_PREFIX__ on startup. This change makes the UI folder writable by anybody. At first I didn't like this, but as it was 755 before then the node user could already write files. Really this is no worse as we're in a container.

/release-pr


Include some sort of showcase list

I find this framework quite cool, but I really wished there was some showcase section on the landing page or readme. It would be interesting to see how it goes out in the wild and how people customize it 😉

Absolutely! After the holidays I'll go about setting that up.

It's up here: https://github.com/andybrewer/mvp#showcase
Feel free to create a PR with any more you'd like to see on there


Add Advance Toolchain (IBM Power cross compiler) - gcc based

Let me know if this is too big and I'll see what I can cut down.
Seems to need the following build dependencies (16.04) but assuming 18.04 is the same:

apt-get install -y lsb-release  autoconf  automake libtool m4  bison sed flex perl \
        cvs debhelper dpkg-sig gawk libncurses5-dev libxml2-utils ncurses-term subversion texinfo xutils-dev zlib1g-dev \
        autoconf-archive dh-systemd docbook2x pkg-config \
        wget git

closes #20
closes #mattgodbolt/compiler-explorer/issues/205

Hi @grooverdan
Thanks for this! I'd very much prefer not building compilers in the install_compilers.sh script: especially unconditionally. It would mean the compiler update would take a very long time! :)
Instead we build the compilers separately, then tar them up and upload to S3 storage. This is how all the GCC and clang builds are done. These builds are done from within a Docker container that has all the necessary prerequistites (see for example https://github.com/mattgodbolt/compiler-explorer-image/blob/master/gcc/Dockerfile for some context).
This would be the first non-GCC or clang build and so it wasn't at all obvious that this is the way to proceed. My thoughts would be:

copy the gcc directory to be e.g. ibm-power. Make the necessary edits to the Dockerfile to install all the build dependencies.
Update the build.sh inside that docker image to build the compiler, installing it like the GCC ones do to a known good spot, then tar.xz-ing them. Ideally s3 uploading but I appreciate that's impossible to test...
change the install_compilers.sh to [a] check to see if the compiler's already installed, and if not [b] install it like the other compilers (with a fetch and an untar).
Make a separate PR to the main compiler-explorer repository to configure using the IBM power. Note that this PR does not close https://github.com/mattgodbolt/compiler-explorer/issues/205 as that needs changes in the main repo too.

I appreciate this is a ton of extra work! I can probably use what you've done so as a template and try it out myself, but I thought in the spirit of sharing I'd explain as best as I can how I'd make the change and see if it's something you can do. I realise not everyone is a Docker user or knows anything about s3 and so on!

That sounds workable. I can work with docker, haven't done much with s3 however it looks like just a push so that should be manageable. As long as you're ok for  the docker to clone https://github.com/advancetoolchain/advance-toolchain (its light and just a build system) and I don't need to reinvent a cross compile build script from scratch I should be right.

I know this initially only mentioned C++ and Go, but am I right in guessing Fortran compilers would/could be included in this as well?

Did start down this path, however after talking to AT maintainers the gcc branches/ibm/gcc-8-branch built using ct-ng in #127 seems easier.


Changed endpoint configuration url from dev to prod for ai features

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


Updated xmldao

Created XMLDAO and XMLSupport for writing from FObject to XML and vice versa
Modified foam.core.Date and foam.core.DateTime to incorporate various date formats in the future.
For now, it is set to ISO8601. Property for these types is called dateFormat

Confirmed.


Uncaught DeveloperError thown

The following error is getting thrown on screen movements.   This was present in 1.20, and is still in 1.21.
Reverting to cesium 1.19 doesn't thrown the error.
the following is the console output in google chrome, Version 50.0.2661.94 m (64-bit) Windows 7 64 bit.
Cesium.js:24158 Uncaught DeveloperError: This object was destroyed, i.e., destroy() was called.
Error
at new DeveloperError (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:1321:19)
at VertexArray.throwOnDestroyed (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:24158:19)
at continueDraw (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:126700:12)
at Context.draw (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:126737:9)
at DrawCommand.execute (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:63127:17)
at executeCommand (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:155472:21)
at executeCommands (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:155699:17)
at executeCommandsInViewport (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:155949:9)
at updateAndExecuteCommands (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:155840:17)
at Scene.pick (http://localhost/cds/Scripts/_Libs/cesium/1.21/Cesium.js:156357:9)throwOnDestroyed @ Cesium.js:24158continueDraw @ Cesium.js:126700Context.draw @ Cesium.js:126737DrawCommand.execute @ Cesium.js:63127executeCommand @ Cesium.js:155472executeCommands @ Cesium.js:155699executeCommandsInViewport @ Cesium.js:155949updateAndExecuteCommands @ Cesium.js:155840Scene.pick @ Cesium.js:156357(anonymous function) @ cesium.js:334handleMouseMove @ Cesium.js:56624listener @ Cesium.js:56449

Hello @bmckilligan,
Have you tried clearing your cache?  Sometimes that can cause weird side effects in chrome when switching between versions.
Otherwise, do you have a code example that causes this to happen?  We haven't seen this on our end.
Thanks!

Are you using terrain and/or picking?  I believe this may be a dupe or related to #3779 (which we should fix ASAP)

http://cesiumjs.org/Cesium/Apps/Sandcastle/?src=Hello World.html&label=Showcases&gist=f1bf37574ba19d084c372b5a6d09466f
I was able to replicate the error in a simple example.  I am using the draw helper plugin (http://cesiumjs.org/plugins)  and it was throwing the error on event listeners.
I have modified the plugin for my own purposes to work with recent versions of cesium and to draw on 3d terrain, but the bug is being thrown in an event listener
If the user zooms in and moves around the globe to where the billboard goes off screen, the error is thrown

@bmckilligan This was fixed in #3779 and will be in the next release.


Redefining constant Struct::ColInfo

After the upgrade to the new 1.11.0 release (thanks!), I see a lot of warnings like these in our CI logs:
/drone/src/vendor/bundle/ruby/3.2.0/gems/write_xlsx-1.11.0/lib/write_xlsx/worksheet.rb:360: warning: redefining constant Struct::ColInfo

The line in question also creates the same struct over and over again in a loop - this could probably be optimized:
https://github.com/cxn03651/write_xlsx/blob/38f2ab620e44653e4ea68eda19b7b87977d1379b/lib/write_xlsx/worksheet.rb#L360-L361

Thank you for your comment.
How about https://www.dropbox.com/s/xe7hqh6xnpnc3tn/write_xlsx-1.11.0.a.gem?dl=0

Sorry, I found another redefinition.
try https://www.dropbox.com/s/1hclmi09m5o2xpk/write_xlsx-1.11.0.b.gem?dl=0

@cxn03651 Thanks for the quick response!
Are these packaged gems based on a public branch on GitHub? I'd feel a bit more comfortable trying that out on a company computer. 🙈

try issue_110 branch.
https://github.com/cxn03651/write_xlsx/tree/issue_110

@cxn03651 I checked the branch on the same codebase @franzliedke was referencing and this solves the issue, tests pass and we no longer get the warning output. Thanks for working on the fix!

@cxn03651 when we expecting this to be released to main branch

sorry for the delay of fixed version's release.
v1.11.1 has been just released.


Odd looking/broken chunks with OTG 8 and Farseek 2.4

OTG Plugin or Mod
[Please specify if it's about Forge Mod or Spigot server Plugin placing "x" into []]

[x] OTG Mod for Forge
[ ] OTG Plugin for Spigot

OTG Version
[Please specify the full OTG version that you are running by placing "x" into []]

[ ] 1.12.2_v8_r1
[ ] 1.12.2_v7_DEV_r26
[ ] 1.12.2-v6
[ ] 1.12.2-v5
[ ] 1.11.2-v8
[ ] 1.11.2-v9
[ ] 1.10.2-v21
[x] Other - 1.12.2_v8_3_r4 (also tested using 8.2)

Forge or Spigot Versions
Forge <IP_ADDRESS>47
What other plugins are you running?
Biome Bundle v6.1, Journey Map 5.5.5, Farseek 2.4, Streams 0.4.7
Description
When I move around in a new world while the latest version of Farseek (and Streams) is installed, some of the chunks that are generated are kinda "broken".
This happens both if you just fly into new terrain or if you teleport using the /tp command (/tp makes it easier to see the "broken" chunks as they will usually be near your target-loacation).
You can see the effect in this picture:

I tested Farseek with other mods that mess with world-generation (BoP, Traverse, etc.) and it didn't seem to happen then, so I guess it's an OTG-problem.
From what I can tell, the terrain itself generates just fine - it won't suddenly change elevation or anything like that -> the "structure" of the terrain seems to be correct. Ores, Caves, etc. that are undergorund also seem to be fine (no sudden gaps or anyhting like that). Apparently the only thing going wrong is the "ground cover". So instead of a layer of grass on the surface you end up getting a layer of stone.
In some places it looked as if the ground-cover from the neighbouring biome was used instead of the correct one. For example I had a situation where some biome with a lot of podzol was next to a mountainous biome and the "broken" chunk was in the mountainus part. The surface of the "broken" chunk was completely covered in podzol instead of stone.

Hey @Tiscan76 , thanks for reporting this. Looks like a problem with blocks not being replaced properly. Not sure this is mod-compatibility issue, this has been reported more often. Will group issues/reports and look at this as soon as it has prio.

Moving to support for now, need a minimal test setup for this. Preferably without any other mods (or only minimal mods to recreate the issue). Need a preset, seed and coordinates where the problem is reproduced so I can debug it.

I can't guarantee that it will always happen but what I did is this:
Create a modpack with just OTG, BB, Farseek 2.4 and Streams 0.4.7
Create a new world using the seed "Farseek"
Teleport to 11000 150 -11000 -> i did 3 tests and in all 3 cases my Minecraft lagged like hell and when it finally showed something, there were 2 messed up chunks (funny thing is, at 11000 150 11000 there is another very funky piece of terrain-gen -> 1 chunk of what looks like Mesa in an Ice-Plains biome - this will also always happen on my test-setup but IDK if it's the same bug)
The problem is: while i can always reproduce the bug by teleporting around using /tp ~1000 ~ ~1000, it usually happens at different coords. Only the 11000/-11000 and 11000/11000 have been consistent so far.
Hope this helps.


Very slow in R seurat implement

Recently, I have tried this method to infer gene co-expression in single-cell RNA-seq data, but I found it was too slow in R, could it be possible to implement it in Python with the input of anndata.
Thank you very much

Hi Yuanlizhanshi,
Thanks for your interest. This repo you are commenting on actually provides an implementation of CS-CORE in Python. We will provide a tutorial that takes the input of anndata in the next couple of days. Meanwhile, feel free to install it with
pip install git+https://github.com/ChangSuBiostats/CS-CORE_python.git

and try CSCORE_IRLS.py, where X is a cell by gene count matrix, seq depth is the sequencing depth of cells. We recommend subsetting the full cell by gene matrix X from anndata to focus on only the genes of interest (for example, the top highly expressed genes), as computation involving the full matrix with 20k genes will be slower, and co-expressions are notoriously hard to estimate for extremely sparse genes.
A tutorial / vignette of this Python version is coming soon! Will update here when is available.

@ChangSuBiostats is there an update on the vignette?

Hi @yuanlizhanshi and @Zethson ,
Thank you both for your interest in our work!
Here is a vignette for running CS-CORE with AnnData input in Python:
notebook. We have also benchmarked the time and memory usage of this implementation at this notebook.
Feel free to leave a message if you have any questions!
Best,
Chang


Error: pad method not found

https://discord.com/channels/915186263609454632/1137543980293292083/1154591575440109659

When using an ability or attacking a creature

Addressed in v0.5.6. (Issue with parameters passed to the calculation, I think.)


Ability to multi-select and download attachments from the chat details page

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @zach in #android-requests.

done


Class constructor IndexedDBLocalPersistence cannot be invoked without 'new'

TypeError: Class constructor IndexedDBLocalPersistence cannot be invoked without 'new'
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 462:37   _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 555:39   dcall
https://www.gstatic.com/firebasejs/10.7.0/firebase-auth.js 1:48595                 _getInstance
https://www.gstatic.com/firebasejs/10.7.0/firebase-auth.js 1:144495                _initializeAuthInstance
https://www.gstatic.com/firebasejs/10.7.0/firebase-auth.js 1:144643                instanceFactory
https://www.gstatic.com/firebasejs/10.7.0/firebase-app.js 977:39                   getOrInitializeService
https://www.gstatic.com/firebasejs/10.7.0/firebase-app.js 921:31                   initialize
https://www.gstatic.com/firebasejs/10.7.0/firebase-auth.js 1:68877                 initializeAuth
dart-sdk/lib/_internal/js_shared/lib/js_util_patch.dart 110:66                     callMethod$
packages/firebase_auth_web/src/interop/auth.dart 24:40                             getAuthInstance
packages/firebase_auth_web/firebase_auth_web.dart 92:43
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54                 runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 127:5                 _async
packages/firebase_auth_web/firebase_auth_web.dart 91:33
packages/firebase_core_web/src/firebase_core_web.dart 323:41
dart-sdk/lib/internal/iterable.dart 403:20                                         moveNext
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 1140:20  next
dart-sdk/lib/async/future.dart 516:16                                              wait
packages/firebase_core_web/src/firebase_core_web.dart 315:18                       initializeApp
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 45:50
dart-sdk/lib/async/zone.dart 1661:54                                               runUnary
dart-sdk/lib/async/future_impl.dart 162:18                                         handleValue
dart-sdk/lib/async/future_impl.dart 838:44                                         handleValueCallback
dart-sdk/lib/async/future_impl.dart 867:13                                         _propagateToListeners
dart-sdk/lib/async/future_impl.dart 643:5                                          [_completeWithValue]
dart-sdk/lib/async/future.dart 524:22
dart-sdk/lib/async/zone.dart 1661:54                                               runUnary
dart-sdk/lib/async/future_impl.dart 162:18                                         handleValue
dart-sdk/lib/async/future_impl.dart 838:44                                         handleValueCallback
dart-sdk/lib/async/future_impl.dart 867:13                                         _propagateToListeners
dart-sdk/lib/async/future_impl.dart 643:5                                          [_completeWithValue]
dart-sdk/lib/async/future_impl.dart 713:7                                          callback
dart-sdk/lib/async/schedule_microtask.dart 40:11                                   _microtaskLoop
dart-sdk/lib/async/schedule_microtask.dart 49:5                                    _startMicrotaskLoop
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 181:7

related to #12344

I encounter the same issue after upgrading to 3.19. Please fix asap.

Same issue after 3.19 upgrade.

I am facing the same issue after the 3.19 upgrade. Any solutions ?

encountered issue after 3.19 upgrade , please fix asap

flutter run -d chrome --release
flutter run -d chrome --profile
You can run if you don't want to downgrade.

flutter run -d chrome --release flutter run -d chrome --profile
You can run if you don't want to downgrade.

what is the root cause of this issue?

Looks like the new Flutter version is not working properly with the Firebase package. Waiting for update.
https://github.com/firebase/flutterfire/issues/12291

It's been 3 days waiting ... our projects have pending bugs and features to be implemented, please fix this asap.

It's been 3 days waiting ... our projects have pending bugs and features to be implemented, please fix this asap.

same here...

i think.  they need more time than expected.

waiting for the new release !

flutter run -d chrome --release flutter run -d chrome --profile
You can run if you don't want to downgrade.

flutter run -d chrome --release flutter run -d chrome --profile
You can run if you don't want to downgrade.

Thanks, It's a good solution while I wait for an update :D

flutter run -d chrome --release flutter run -d chrome --profile
You can run if you don't want to downgrade.

flutter run -d chrome --release flutter run -d chrome --profile
You can run if you don't want to downgrade.

Thanks, It's a good solution while I wait for an update :D

Update your fibrease_auth to the latest version - 4.17.5

Same isso

same here

please help i am also in this condition

To solve this error, we need to downgrade


fix(match-making/types): Add missing docs for NEX 1.x and older

Resolves #27
Changes:

In NEX 1.x and older, two string fields are missing from MatchmakeSessionSearchCriteria. Guessing from the values of the other two string fields on Street Fighter IV 3D Edition (the first one is empty and the second one is m_MatchmakeSystemType), the missing fields are m_MinParticipants and m_MaxParticipants. Document this version difference appropiately on the wiki.

[x] I have read and agreed to the Code of Conduct.
[x] I have read and complied with the contributing guidelines.
[x] What I'm implementing was an approved issue.
[x] I have tested all of my changes.

LGTM


Supporting esbuild-plugin-pipe

Hi 👋, I have been loving using unplugin mostly for vite and esbuild plugins in parallel.
One snag that took a while to get my head around is using multiple esbuild plugins to transform some code, this works in piped bundlers like rollup, vite etc. but from what I understand not in esbuild as it exposes onLoad internally which runs once per file, and unplugin uses that when using transform meaning any subsequent plugins won't do anything.
A potential solution seems to be esbuild-plugin-pipe which adds support for compatible plugins to pipe from one to another, passing along the output. Would support for this  be considered as part of unplugin?

The esbuild-plugin-pipe repository is now archived. The unplugin library has an internal onTransform hook, allowing all unplugin plugins to be chained together on the user side. See https://github.com/vue-macros/vue-macros/blob/5fd78c159fd0cd8162bc6d990a3eb0c516216ee0/tsup.config.ts#L57


fix typos in comments

Working on this 👍

you dont have any
most of the "typos" are because github and words like that are not in the English dictionary...

i only found one typo

@mangafas Yeah now that I am looking I don't see any typos either

I know of at least one: authenting


DynamicCompositeAgent remove can cause race between onClose and doWork

see relevant TODO here: https://github.com/real-logic/Agrona/blob/master/agrona/src/main/java/org/agrona/concurrent/DynamicCompositeAgent.java#L214
Essentially, remove is threadsafe in it's list manipulation, but the old list may be iterated through by an AgentRunner which will call doWork after onClose has happened or as it is happening. A possible solution is to have a work counter (incremented by the doWork loop on execution) used to delay the close effect until a further iteration has run.

There is also a race between onClose() and remove(Agent)

Putting a work counter in here would be bad for performance. We need a different solution.

There is no race with doWork() and onClose() as onClose() should not be called until the doWork() method has returned and will not be invoked again. Look to he standard AgentRunner. Same as onStart() should be called before doWork() is ever called. The only true race is onClose and remove(Agent).
We can address all of this more simply. We can specify that administrative action must be called from a single thread and clarify the other behaviour in documentation.

It is also worth considering that onStart() and onClose() are specified to be called on the same thread as doWork(). This would help with issues like biased locking. This makes the whole thing much cleaner.

I think I know a better way to do this. I'll pick it up today.

There is no race with doWork() and onClose() as onClose() should not be called until the doWork() method has returned and will not be invoked again. Look to he standard AgentRunner. Same as onStart() should be called before doWork() is ever called. The only true race is onClose and remove(Agent).

From a single Agent point of view, sure. For the sake of this discussion we must consider a DynamicCompositeAgent (lets call it parent) executed by an AgentRunner. Its spinning in doWork, while other threads are adding/removing agents. Adding will imply onStart should be executed, removing will imply onClose needs to run. So for parent there's no race between doWork and onClose, but for children there's a risk. Similarly onClose for the parent can race with remove to trigger onClose on the child concurrently.

It is also worth considering that onStart() and onClose() are specified to be called on the same thread as doWork().
onStart() and doWork() are to be called from the executing thread, onClose is called by the closing thread, at least in the current AgentRunner.

I've changed the implementation. @RichardWarburton it would be good if you can confirm this would work for you?

@mjpt777 I think overall it looks like it work work well. I've got a few thoughts:

You check that the Agent elements of the array aren't null, but you don't do this for the List based constructor. ArrayList can contain nulls, shouldn't they be consistent?
Thinking about it do we want the add and remove methods to just yield for idle, or should they take an idle strategy?
I kind of felt that an array was a better choice than an arraylist. Iterating over it is a common operation, whilst adding/removing are uncommon.
Regardless of container choice if you're iterating over it using a for loop isn't it better to load the agents field into a local variable and use that. I think Nitsan benchmarked it.
IMHO The add method should document that if the onStart() method throws an Exception that the agent isn't added.
And least importantly, is there any reason to manually null check and throw the exception rather than use Agrona's Verify.notNull(); method or even Objects.notNull() in the JDK?

@RichardWarburton Thanks. I've applied your feedback. The only thing is the idle strategy which I think does not make sense. This is highly unlikely to contend and if so then a Thread.yield() should be fine.

@RichardWarburton I've addressed your concern about IdleStrategy by making the methods tryAdd(Agent) and tryRemove(Agent) so that they are non-blocking. You can spin or failure or employ your own IdleStrategy.

I think that's a good approach.


rte manifests: update sample-devices image

Consume the enhanced sample device plugin image that allows reregistration of the devices in case of kubelet restart.
P.S: there is no failure noted before this change but seems more right to consume the enhanced version.

/retest

/approve

let's not retest: let's actually check the setup is correct with the new sample device plugin. Perhaps some other updates are due.

let's not retest: let's actually check the setup is correct with the new sample device plugin. Perhaps some other updates are due.

d/s change is also failing, I'll check the reason later today

we need to update the daemonset manifests (like https://github.com/openshift-kni/numaresources-operator/blob/main/rte/hack/manifests/sample-devices/daemonset-dpa.yaml)
adding
        command:
          - /bin/deviceplugin
        args:
          - --alsologtostderr
          - --config-dir=/etc/devices

into the container template so it reads like
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: device-plugin-a-ds
  namespace: sample-dev-ns
spec:
  revisionHistoryLimit: 10
  selector:
    matchLabels:
      name: device-plugin-a
  template:
    metadata:
      labels:
        name: device-plugin-a
    spec:
      containers:
      - args:
        - --alsologtostderr
        - --config-dir=/etc/devices
        command:
        - /bin/deviceplugin
        env:
        - name: DEVICE_RESOURCE_NAME
          value: example.com/deviceA
        image: XXX
        imagePullPolicy: IfNotPresent
        name: device-plugin-a-container
        securityContext:
          privileged: true
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /var/lib/kubelet/device-plugins
          name: kubeletsockets
        - mountPath: /etc/devices
          name: device-a-config-vol
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      schedulerName: default-scheduler
      serviceAccount: sampledevices-sa
      serviceAccountName: sampledevices-sa
      terminationGracePeriodSeconds: 30
      volumes:
      - hostPath:
          path: /var/lib/kubelet/device-plugins
          type: Directory
        name: kubeletsockets
      - configMap:
          defaultMode: 420
          name: device-a-configmap
        name: device-a-config-vol
  updateStrategy:
    rollingUpdate:
      maxSurge: 0
      maxUnavailable: 1
    type: RollingUpdate

/hold cancel


Change test naming convention for services

At the moment our test naming convention for services is inconsistent. This PR will fix this issue.

Current coverage is 100% (diff: 100%)

Merging #90 into master will not change coverage

@@           master   #90   diff @@
===================================
  Files          10    10
  Lines          69    69
  Methods        14    14
  Messages        0     0
  Branches       10    10
===================================
  Hits           69    69
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 54b728d...aea5c33


Using packaged functions with Oracle Object Types and Oracle Object Collections as parameter

I can call functions in a package with simple parameters as in and out arguments, but until now I struggle invoking a package function having complex types, such as Oracle Object Collections or Types as in and/or out params (out as return). I can do it with a huge dynamic SQL block, but until now I did not figure out how to do it through this library. Is this possible? Can you please share examples? Many thanks!

Many thanks. That looks great. I will go through the examples. You're my today-hero!

Many thanks. That looks great. I will go through the examples. You're my today-hero!

LOBs still not support in UDT
I will try to add it and inform you here

In the mean time I made some further tests. Can it be, that you not yet support collection types (create or replace CollType as table of ObjType)? It comes with "unsupported array type" when executing a statement directly or with "unsupported type object: Slice" when trying to register the collection type (declared as var out []MyType{}).

you should register the type only not its collection MyType{} then you can use in query, exec and bulk insert/merge
but inside oracle collection i didn't test it

We, in our situation, have mostly use cases, where we call functions in packages. These packages have object types and object collections as input, returning primitives, types or even again collections. Here an example:
create or replace type MY_ID_OBJ force as object(ID number(22,0));
create or replace type MY_ID_COLL as table of MY_ID_OBJ;
create or replace type MY_TEST_OBJ force as object(ID number(22,0), NAME varchar2(2000));
create or replace type MY_TEST_COLL as table of MY_TEST_OBJ;

create or replace package MY_TEST_PCK as
   function F_GetData(P_Ids MY_ID_COLL) return MY_TEST_COLL;
end;

create or replace package body MY_TEST_PCK as
   function F_GetData(P_Ids MY_ID_COLL) return MY_TEST_COLL is
      L_Coll MY_TEST_COLL := MY_TEST_COLL();
   begin
      for r in (with data as(
                  select 1 as ID, 'Data1' as DATA from dual union all
                  select 2      , 'Data2'         from dual
               )
               select MY_TEST_OBJ(ID, DATA) as MY_DATA
                 from data
                where ID in (select ID from table(P_Ids))) loop
         L_Coll.Extend;
         L_Coll(L_Coll.LAST) := r.MY_DATA;
      end loop;
      return L_Coll;
   end;
end;

select * from table(MY_TEST_PCK.F_GetData(MY_ID_COLL(MY_ID_OBJ(1),MY_ID_OBJ(2))));

Many thanks for your investigations!

OK i will test all and publish new release

Hi Samy
Do you probably have any idea when you plan to publish the new release with
enabled Oracle collection types?
Thanks
Meinrad
Am Di., 16. Mai 2023 um 15:36 Uhr schrieb Samy Sultan <
@.***>:

OK i will test all and publish new release
—
Reply to this email directly, view it on GitHub
https://github.com/sijms/go-ora/issues/366#issuecomment-1549689551, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALBTNCCLBWHM2DKQM5SLOYDXGN7EXANCNFSM6AAAAAAYCASCNA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

--
Meinrad Hermanek
Oberdorf 25a
5102 Rupperswil
+41 (0)79 773 14 71
@.***

sorry for late
I actually recode all functions related to parameter encoding/decoding so I can implement Lobs in UDT
I will finish within days

finally I fix the issue


pin e-h alpha

Alphas dont follow semver and theres a new e-h alpha now which has breaking changes and is automatically being picked up in 10.0.
Obviously can update for the new e-h hal alpha as well, but seems best to get this patch and a new 10.1 release in first.
error[E0107]: this trait takes 0 generic arguments but 1 generic argument was supplied
  --> /home/jacob/.cargo/registry/src/github.com-1ecc6299db9ec823/radio-0.10.0/src/blocking.rs:97:30
   |
97 |     T: Transmit<Error = E> + DelayUs<u32>,
   |                              ^^^^^^^----- help: remove these generics
   |                              |
   |                              expected 0 generic arguments
   |
note: trait defined here, with 0 generic parameters
  --> /home/jacob/.cargo/registry/src/github.com-1ecc6299db9ec823/embedded-hal-1.0.0-alpha.6/src/delay.rs:14:15
   |
14 |     pub trait DelayUs {
   |               ^^^^^^^

error[E0107]: this trait takes 0 generic arguments but 1 generic argument was supplied
   --> /home/jacob/.cargo/registry/src/github.com-1ecc6299db9ec823/radio-0.10.0/src/blocking.rs:180:39
    |
180 |     T: Receive<Info = I, Error = E> + DelayUs<u32>,
    |                                       ^^^^^^^----- help: remove these generics
    |                                       |
    |                                       expected 0 generic arguments
    |
note: trait defined here, with 0 generic parameters
   --> /home/jacob/.cargo/registry/src/github.com-1ecc6299db9ec823/embedded-hal-1.0.0-alpha.6/src/delay.rs:14:15
    |
14  |     pub trait DelayUs {
    |               ^^^^^^^

error[E0107]: this trait takes 0 generic arguments but 1 generic argument was supplied
   --> /home/jacob/.cargo/registry/src/github.com-1ecc6299db9ec823/radio-0.10.0/src/blocking.rs:223:38
    |
223 |     T: State<State = S, Error = E> + DelayUs<u32>,
    |                                      ^^^^^^^----- help: remove these generics
    |                                      |
    |                                      expected 0 generic arguments
    |
note: trait defined here, with 0 generic parameters
   --> /home/jacob/.cargo/registry/src/github.com-1ecc6299db9ec823/embedded-hal-1.0.0-alpha.6/src/delay.rs:14:15
    |
14  |     pub trait DelayUs {
    |               ^^^^^^^

For more information about this error, try `rustc --explain E0107`.
error: could not compile `radio` due to 3 previous errors
warning: build failed, waiting for other jobs to finish...
error: build failed

hey thanks for the PR! the reason i didn't have this is that it means we can slide over alpha releases so long as they're not breaking for us (and you can pin the version at the project level), but, given this release has broken our API it seems like a good addition for now ^_^


[Enhancement] Option for horizontal StartPage title

Hi, I like the plugin!
A request regarding the StartPage view. The title is vertical. If the option was available, I'd prefer it horizontal in order to match new games shelf and also hopefully fix the slight misalignment:

Thanks in advance.

Great catch! Will implement this, but probably not very soon. Quite busy atm!

The recent StartPage addon added the option to show a view label, so hiding it entirely could be another option. Thanks. :)

Hey @chocolatechipcats! I've released an update with an ability to show horizontal label, or hide it completely. Tried solving the alignment issues the best I could, but it can still be wonky sometimes depending on the situation.
Hope this works for you! :)


Can't find where the creditentials e.g. log in stored

Anyone know the directories?

Should be in ~/.messer/ directory: https://github.com/mjkaufer/Messer/blob/master/config/index.js


nextcloud-client segmentation fault on nixos-unstable

Describe the bug
After #118084 was merged nextcloud-client no longer fails to build, but now it crashes with SIGSEGV.
To Reproduce
Steps to reproduce the behavior:

nix-shell -p nextcloud-client
nextcloud

Expected behavior
nextcloud-client starts and works.
Additional context
gdb core dump
warning: Can't open file /memfd:JSGCHeap:QtQml (deleted) during file-backed mapping note processing
warning: Can't open file /memfd:JSVMStack:QtQml (deleted) during file-backed mapping note processing
warning: Can't open file /SYSV51260028 (deleted) during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file /memfd:xshmfence (deleted) during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
warning: Can't open file /run/nscd/dbe8hReO (deleted) during file-backed mapping note processing
warning: Can't open file anon_inode:i915.gem which was expanded to anon_inode:i915.gem during file-backed mapping note processing
[New LWP 76379]
[New LWP 76363]
[New LWP 76369]
[New LWP 76371]
[New LWP 76374]
[New LWP 76370]
[New LWP 76381]
[New LWP 76375]
[New LWP 76373]
[New LWP 76372]
[New LWP 76380]
[New LWP 76377]
[New LWP 76376]
[New LWP 76378]
[New LWP 76362]
[New LWP 76382]
Core was generated by `/run/current-system/sw/bin/nextcloud'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x00007f7ba9a6d590 in ?? ()
[Current thread is 1 (LWP 76379)]

Notify maintainers
@caugner
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.27, NixOS, 21.05.20210408.9e377a6 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.4pre20210326_dd77f71`
 - nixpkgs: `/nix/store/mmws9c6185xn3wvlzxa4a4h68x9q26yw-source`

nixpkgs rev: 9e377a6ce42dccd9b624ae4ce8f978dc892ba0e2
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
- nextcloud-client
# a list of nixos modules affected by the problem
module:

I'm able to run this on unstable, and have the gui launch. I did notice
/run/current-system/sw/bin/nextcloud

You are likely running nextcloud from your system, and not from nix-shell, try nix-shell --pure -p nextcloud-client --run nextcloud

Still the same.
Output of nix-shell --pure -p nextcloud-client --run nextcloud:
QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/run/user/1000/runtime-lycheese'
/tmp/nix-shell-1814-0/rc: line 1:  3909 Segmentation fault      (core dumped) nextcloud

But if it works on your end, it must be something in my config. Not sure what could cause this though. Is there a way to prevent a nix-shell from accessing /run?

Is there a way to prevent a nix-shell from accessing /run?

I think that's okay, and probably not affect the behavior.

Or force rebuilding of all dependencies?

nix-build -E 'import <nixpkgs> {}; pkgs.nextcloud-client' --check  for nix <2.4
nix build .#nextclout-client --rebuild nix >=2.4

I am on nix version 2.4pre20210326_dd77f71 and nix build .#nextclout-client --rebuild is giving me an error that my path is not a flake. I am using flakes for my system configuration and do not have any nix channels, if that is relevant.

Since it works on your end, should we just close this issue? I don't really want to take up your time with fixing my configuration…

giving me an error that my path is not a flake.

oh right, it would be nix build nixpkgs#nextcloud-client --rebuild, when outside of the tree

giving me an error that my path is not a flake.

oh right, it would be nix build nixpkgs#nextcloud-client --rebuild, when outside of the tree

The command worked, but when following the result link and executing bin/nextcloud it still throws a segmentation error.

FWIW it's segfaulting for me too, on nixpkgs f5e8bdd with nextcloud-client installed in environment.systemPackages, though I don't have the first line your logs show and I directly get a segfault. Haven't had the time to investigate yet though, but I don't think this should be closed yet. Maybe it's related to not using KDE as a DE?

I managed to fix it on my side.
After confirming that I managed to run it successfully on a new install with the same versions I suspected it had something to do with remaining configuration.
Running nix-shell --pure -p strace nextcloud --run 'strace -fte trace=file nextcloud' showed that immediately before segfaulting it tried to open a file in an existing directory. After moving the Nextcloud folders in ~/.cache, ~/.config and ~/.local/share the nextcloud client started normally.
@Ekleog does moving/deleting those directories work for you as well?

I had the same issue. Deleting ~/.cache/Nextcloud fixed it for me. it contains files like these:
$ ls ~/.cache/Nextcloud/qmlcache/
01567610c4e844cecc63495b5844e454d8fc94ee.qmlc  4da21d846270ee9247c9b9821dfa3afbbbb52097.qmlc [...]

So likely triggering some Qt version mismatch?

@Ekleog Are you still having this problem? If not, I would like to close this issue.

Oh. I'm sorry, I wanted to actually dive some more into this and figure out a way to make the migration to 21.05 more seamless, but it looks like I'm way late and 21.05 already forked anyway so it's too late to add this kind of workarounds to the build…
I can confirm that the rm -r ~/.cache/Nextcloud workaround worked for me too, so let's close as you were suggesting.
Thank you for the solution!

I recently had the same issue, with the same fix, on 22.05.
Maybe https://github.com/NixOS/nixpkgs/commit/788119f78101a10b2c58de0690c23572ac348076 got reverted?


no deps file in master branch

i want to copy deps to use master branch, but there is no deps file in master branch

This file doesn't exists anymore. See #267

we are now using Composer for the master branch.

tnx)


Fix "aggressive" StringConcatenation offenses in classes

This PR fixes Style/StringConcatenation offenses in app/classes in "aggressive" mode.
It follows up on  PR #1017 (which fixed only the offenses that could be found in "conservative" mode).
I'm breaking the fix of remaining offenses into (artificially) small PRs because to facilitate review and because (per Rubocop) the cop is less safe in "aggressive" mode.

Coverage remained the same at 85.78% when pulling<PHONE_NUMBER>217b7178ff276b497cb086c2383146 on jdc-string-concat-classes-aggressive into df199c641f67027b73c51f1608f04b9e730dd0e2 on master.

That Query::LocationPatternSearch#initialize_flavor CONCAT COALESCE is too mind-boggling, but the rest of 'em seem a-OK!

Thanks @nimmolo!
The change I made to that query does seem wrong. I'll have a closer look.

I don't believe it's necessarily wrong, I just don't get it!

I have some extra newlines in the SQL, but they shouldn't make a difference.
However, to be on the safe side, I'll get rid of them.

It was right, but still mind-boggling.
I added some line breaks and spacing, which hopefully make it more readable (but the boggle factor is unchanged).

This all looks correct and safe and proper.  I only wish it were possible to make correct code more readable!  But I think you've done as good a job as possible.

@pellaea: Thanks very much for taking the time to review.
I agree that the interpolation format is sometimes ugly. But I'd like to leave he cop on, and would rather have an ugly format than the awkward enabling/disabling of the cop.

Agreed that lots of disabling/enabling cops is a bad idea!  I can live with
the cop.
On Sat, Jun 18, 2022 at 7:20 PM Joseph D. Cohen @.***>
wrote:

@pellaea https://github.com/pellaea: Thanks very much for taking the
time to review.
I agree that the interpolation format is sometimes ugly. But I'd like to
leave he cop on, and would rather have an ugly format than the awkward
enabling/disabling of the cop.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1022#issuecomment-1159580773,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNOUANVXUEYTG6HZZI3VPZKSPANCNFSM5YX44NKQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


profile banner image

picking this one up 👍

@bennyhodl great! FYI I just created this https://github.com/neb-b/daisy/issues/34
This issue doesn't need to do that but just know it will use nostr.band in the future

@bennyhodl are you still working on this? was hoping to get it in the next release and can pick it up if you don't plan to finish it soon

got a little carried away with the profile UI changes and started working on this https://github.com/neb-b/daisy/pull/53. Sorry if you were almost ready with it.


Add Rain/Change/Fair labels to pressure graphs

My analog barometer has nice text with Rain between 970 and 980, Change centered around 1000 and Fair between 1020 and 1030.

playing with styles and clip-path
<style>
	.barometer-label {
		font-family: Georgia, serif;
		font-style: italic;
		font-size: 24px;
		opacity: 0.5;
		clip-path: polygon(40px 12px, 70px 12px, 70px 100px, 40px 100px);
		}
</style>
<text class="barometer-label" x="300" y="90">Rain</text>
<text class="barometer-label" x="300" y="60">Change</text>
<text class="barometer-label" x="300" y="20">Fair</text>

the clip-path isn't working correctly yet. https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/clip-path and https://www.w3schools.com/cssref/css3_pr_clip-path.php have been my reference.

I seem to be able to clip the text to the extents of my graph with this code:
<style>
	.barometer-label {
		font-family: Georgia, serif;
		font-style: italic;
		font-size: 24px;
		opacity: 0.5;
		clip-path: polygon(
			58px 14px,
			486px 14px,
			486px 121px,
			58px 121px) view-box;
	}
</style>
<text class="barometer-label" x="200" y="130">Rain</text>
<text class="barometer-label" x="455" y="60">Change</text>
<text class="barometer-label" x="200" y="20">Fair</text>

The important thing for me to figure out was the view-box part of the clip-path.
Next, to implement the details in the code, making sure that the clip path polygon is properly defined based on the actual graph being drawn.


Synchronize SSH keys on login with LDAP

This PR synchronises SSH keys on login with LDAP.
It could do with a test from an LDAP user.

Should fix #5545

I did a quick test and removed my user from gitea and then logged in. The ssh key was added.
Then I removed the key from my account and relogged and it was added again.
Also works with multiple keys. It looks to be doing everything I want. Thanks for the fast response.
Maybe add the 'sshPublicKey' attribute to the log.Trace("Fetching attributes '%v', '%v', '%v', '%v', '%v' with filter %s and base %s", ...) call

@NomAnor cool.
It looks like the changes have broken some tests so I need to look at those and re-push.
I'll look at changing the debug note too.

OK, problem 1. is definitely a bug with the current implementation - Any sqlite installation will hanging if they have to remove keys during a synchronisation.

I changed the full name of my account an relogged and it was not overwritten. Then I started the synchronization as administrator and it was changed. So the current behaviour of gitea seem to be that account details are only overwritten when the synchronization runs and initially filled out on first login without an existing account (Maybe document that relogging does currently not synchronize account information).
The current ssh key implementation does not adhear to this behaviour and synchronizes on every login. I think it would be better if all external authentication plugins and all account attributes work the same way.
If someone uses external account management then all information available should be overwritten so the account management is the definite source of information. From a user perspective I think it would be good that this synchronization also happens on every login (that might have performance issues on big installations; make it configurable?).  In this case the overwritten user settings should be disabled (like the username for external accounts).
I'm not in a corporate environment so their requirements might be different.
Another problem might occour if someone does not want to store ssh keys in the ldap directory. Then the keys are deleted by the synchronization because the sshPublicKey Attribute is absent. Only delete the keys if the sshPublicKey Attribute is present but empty?

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8dc09ed). Click here to learn what that means.
The diff coverage is 62.5%.

@@            Coverage Diff            @@
##             master    #5557   +/-   ##
=========================================
  Coverage          ?   37.64%
=========================================
  Files             ?      321
  Lines             ?    47206
  Branches          ?        0
=========================================
  Hits              ?    17769
  Misses            ?    26886
  Partials          ?     2551

Impacted Files
Coverage Δ

modules/auth/ldap/ldap.go
53.91% <100%> (ø)

models/ssh_key.go
40.43% <100%> (ø)

models/user.go
48.03% <40%> (ø)

models/login_source.go
26.3% <50%> (ø)

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 8dc09ed...c8d4f8b. Read the comment docs.

Hi @NomAnor. I think you're right - if you're going to have externally managed sshkeys then they should be externally managed.  And so on, and so on...
I guess the thing is, although this PR doesn't actually change the current policy it certainly makes it more apparent...
(I've fixed the problem with the test cases and the deadlock bug in sqlite now.)


Add support for asp-page to URL/form taghelper

See #5921 for the examples with asp-page

Should get covered by https://github.com/aspnet/Mvc/pull/6053

@pranavkm is this still being covered by #6053 now that we punted page naming?

I think we should add asp-page-handler too, which would set the handler (formerly formaction). You'd be able to use it by itself to generate a URL for a handler on the current page too.

I'll do the page-handler as a follow up PR. (just to keep this one simple).

Handler supported added in https://github.com/aspnet/Mvc/commit/649ee4d28d7cf46401c09ad0108a4f23b6d7d513


fix Modal scrolled position

Resolves: https://github.com/web3connect/web3connect/issues/54

Thanks! I'm going to wait for beta.20 to be merged to include this fix together with the other fixes

Fixed on https://github.com/web3connect/web3connect/pull/72/commits/92852619cfdf3eb59f2468d8f4a4ad9cedf86b0d


Enable useImage, useSvg, useFont to use direct http URLs instead of require()

We want to be able to use http(s) URIs directly when creating image, SVG, fonts etc (through an SkData instance).

Would the URL be an url to a bundle resource and require be implemented inside the hooks? Or would it support external URLs like https etc? Could you elaborate on why we should do this and some more details? :)

I'm trying to use a local file url (of the form 'file:///var/mobile/Media/......') with an ImageShader component by first resolving my file url with useImage. So far my code looks exactly like the example in the ImageShader documentation except instead of requiring an image at a local path in the project directory, I am passing my file url directly which is causing my project to crash. Doing some logging in JSISkImageFactory shows that my file is being read 'correctly' but that it is crashing when trying to read SkImage->width in SkImage::MakeFromEncoded.


Bug: lens didn't retry / restart or go in crashloopbackoff when was unable to subscribe with NATS topic

📜 Description
error logs :
[Jul 22 2024 22:01:22 GMT+0530] lens-dcd-6d894948f-fwsnx: {"level":"info","ts":1721665882.957072,"caller":"pubsub-lib/PubSubClientService.go:139","msg":"Subscribed to pubsub client","topic":"CD.TRIGGER"} [Jul 22 2024 22:01:23 GMT+0530] lens-dcd-6d894948f-fwsnx: 2024/07/22 16:31:23 No stream was created already. Need to create one. Stream name: ORCHESTRATOR [Jul 22 2024 22:01:23 GMT+0530] lens-dcd-6d894948f-fwsnx: 2024/07/22 16:31:23 Error while creating stream.  stream name:  ORCHESTRATOR error:  nats: stream name already in use [Jul 22 2024 22:01:23 GMT+0530] lens-dcd-6d894948f-fwsnx: {"level":"error","ts":1721665883.9656017,"caller":"pubsub-lib/PubSubClientService.go:305","msg":"unable to retrieve consumer info from NATS-server","stream":"ORCHESTRATOR","consumer":"CD-TRIGGER-DURABLE1","err":"nats: consumer is offline","stacktrace":"github.com/devtron-labs/common-lib/pubsub-lib.PubSubClientServiceImpl.updateConsumer\n\t/go/src/github.com/devtron-labs/lens/vendor/github.com/devtron-labs/common-lib/pubsub-lib/PubSubClientService.go:305\ngithub.com/devtron-labs/common-lib/pubsub-lib.PubSubClientServiceImpl.Subscribe\n\t/go/src/github.com/devtron-labs/lens/vendor/github.com/devtron-labs/common-lib/pubsub-lib/PubSubClientService.go:161\ngithub.com/devtron-labs/lens/client.NewNatsSubscription\n\t/go/src/github.com/devtron-labs/lens/client/NatsSubscription.go:71\nmain.InitializeApp\n\t/go/src/github.com/devtron-labs/lens/wire_gen.go:55\nmain.main\n\t/go/src/github.com/devtron-labs/lens/main.go:28\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:250"} [Jul 22 2024 22:01:24 GMT+0530] lens-dcd-6d894948f-fwsnx: {"level":"error","ts":1721665884.4666336,"caller":"pubsub-lib/PubSubClientService.go:170","msg":"error while subscribing to nats ","stream":"ORCHESTRATOR","topic":"CD.TRIGGER","error":"nats: consumer is offline","stacktrace":"github.com/devtron-labs/common-lib/pubsub-lib.PubSubClientServiceImpl.Subscribe\n\t/go/src/github.com/devtron-labs/lens/vendor/github.com/devtron-labs/common-lib/pubsub-lib/PubSubClientService.go:170\ngithub.com/devtron-labs/lens/client.NewNatsSubscription\n\t/go/src/github.com/devtron-labs/lens/client/NatsSubscription.go:71\nmain.InitializeApp\n\t/go/src/github.com/devtron-labs/lens/wire_gen.go:55\nmain.main\n\t/go/src/github.com/devtron-labs/lens/main.go:28\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:250"} [Jul 22 2024 22:01:24 GMT+0530] lens-dcd-6d894948f-fwsnx: {"level":"error","ts":1721665884.466705,"caller":"client/NatsSubscription.go:73","msg":"Error while subscribing to pubsub client","topic":"CD.TRIGGER","error":"nats: consumer is offline","stacktrace":"github.com/devtron-labs/lens/client.NewNatsSubscription\n\t/go/src/github.com/devtron-labs/lens/client/NatsSubscription.go:73\nmain.InitializeApp\n\t/go/src/github.com/devtron-labs/lens/wire_gen.go:55\nmain.main\n\t/go/src/github.com/devtron-labs/lens/main.go:28\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:250"} [Jul 22 2024 22:01:24 GMT+0530] lens-dcd-6d894948f-fwsnx: {"level":"info","ts":1721665884.4668856,"caller":"lens/App.go:58","msg":"starting server on ","port":8080}
Affected areas
Other CRITICAL functionality
Additional affected areas
Other CRITICAL functionality
Prod/Non-prod environments?
Prod
Is User unblocked?
Yes
How was the user un-blocked?
TEMPORARILY - By doing some changes from the backend/DB
Impact on Enterprise
User was not able to see deployment metrics for the past couple of days
👟 Steps to replicate the Issue

bounce nats and lens, and make sure lens comes before NATS does.

👍 Expected behavior
lens should have been retried subscribing to topic : CD-TRIGGER-DURABLE-1
👎 Actual Behavior
lens did not retry.
☸ Kubernetes version
NA
Cloud provider

NA

🌍 Browser
Chrome
✅ Proposed Solution
NA
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find any similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

https://github.com/devtron-labs/lens/pull/29


get_available_approvals() does not have destination and source state references

Hello!
the documentation of get_available_approvals suggests the object returned after the execution has destination and source state references, but they are through the transition object.

I made the change on my side. Please see this diff
https://github.com/rupin/WorkflowEngine/commit/11aa951cbcd1e110dfe008e6c524c03311ea4172#diff-2a5d907555493e832fb037724823fdd0


[Bug]: SQL server unable to connect if password has special characters ';/>

Is there an existing issue for this?

[X] I have searched the existing issues

Description

Error occurred when the password begins with ';/>
Error thrown was Driver com.microsoft.sqlserver.jdbc.SQLServerDriver claims to not accept jdbcUrl,
Escaping characters also did not work

Steps To Reproduce

Try to connect to a SQL server with password having some special characters like ';/>
Unable to connect

Public Sample App
No response
Version
cloud
 Front conversations

Conversation from the user

I'm currently using Appsmith v1.7.1.
I think the issue is with the semi-colon (;) in the password.
After inputting the password I get this error message (I'm changing the server and credentials):
Driver com.microsoft.sqlserver.jdbc.SQLServerDriver claims to not accept jdbcUrl,
jdbc:sqlserver://IP:PORT;user=user;password=<masked>;/>;
So it only recognizes the first character before the semi-colon as the password, because that's the only part that's "masked"

Closing because fixed here.


Last Child does not work as per example in documentation

The example is located here:
https://tailwindcss.com/docs/pseudo-class-variants/#last-child-v1-1-0
You can see the broken functionality here using the latest CDN resource:
https://jsfiddle.net/FoomFries/vgse0k2b/2/

The CDN build doesn't include last-child variants for anything out of the box. You have to enable it for the utilities you want it for in your tailwind.config.js file 👍

Ah, I had missed that statement. Enabling them on my installation via the variants in the tailwind.config.js resolved the issue. Thanks!


Allow distinction between port to bind to and port to announce

Docs for druid.port say:
This is the port to actually listen on; unless port mapping is used, this will be the same port as is on druid.host

I want to run druid in container with virtualized network and fixed port, but expose it on different random ports to host (think mesos). I was unable to find documentation about port mapping, it looks like it only appears in druid.port description and code prohibits any port mappings.

Related discussions on ports:
https://github.com/druid-io/druid/issues/1124
https://github.com/druid-io/druid/pull/1125

@bobrik Druid currently always announces itself on the port it binds to. It would require some work to make those two distinct. Druid announces itself externally, but also uses some of those mechanisms internally to talk between nodes, so it would require clarifying whether internal communication between nodes should go over the announced port or the port it binds to.
I will update the title of the issue to make it more clear what you are referring to.

it would require clarifying whether internal communication between nodes should go over the announced port or the port it binds to.

Announced port should be used for communication between nodes. My use-case is docker image where druid always binds to port 8080 in private net, but exposed port can be arbitrary (e.g. assigned by mesos). Nodes can only see each other on announced (exposed) port.


Ensure the file array is not null

Ensure the git diff file array is not null to not broke the UI

If the UI is fixed, let’s close this

Ok, closing it.


Add thumbv7 and thumbv8 CI targets

The build-no-alloc job has been expanded to be built for
thumbv7m-none-eabi and thumbv8.main-none-eabi

Motivation

Future Work

Current dependencies on/for this PR:

main

PR #247   👈

This comment was auto-generated by Graphite.


[ISSUE #1004]🚀optimize and improve consume logic⚡

Which Issue(s) This PR Fixes(Closes)

Fixes #1004
Brief Description

How Did You Test This Change?

Summary by CodeRabbit
Release Notes

New Features

Introduced asynchronous methods for batch locking and unlocking message queues, enhancing efficiency.
Added new modules and data structures for handling batch request and response bodies in the messaging protocol.
Enhanced the ProcessQueue struct with methods for better tracking of unlock attempts and timestamps.

Bug Fixes

Updated locking mechanisms for improved reliability and performance in message queue management.

Documentation

Expanded protocol capabilities with new request and response modules, improving clarity and functionality.

These updates aim to enhance the overall user experience by improving the efficiency and reliability of message queue operations.

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥


Fix error type inference bug

Purpose
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/16900
Also fixes https://github.com/ballerina-platform/ballerina-lang/issues/16917

Codecov Report

Merging #16902 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #16902   +/-   ##
=======================================
  Coverage   68.85%   68.85%
=======================================
  Files           5        5
  Lines         122      122
  Branches       21       21
=======================================
  Hits           84       84
  Misses         38       38

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 ac59230...e55fef1. Read the comment docs.


net/http: TestTransportMaxConnsPerHost still flaky

What version of Go are you using (go version)?

$ go version
go version devel +83f205fa88 Sun May 12 02:55:19 2019 +0000 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/cuonglm/.cache/go-build"
GOENV="/home/cuonglm/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/cuonglm/go"
GOPROXY="direct"
GOROOT="/home/cuonglm/sources/go"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/home/cuonglm/sources/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build766721718=/tmp/go-build -gno-record-gcc-switches"

What did you do?
From git root repo
$ cd src/net/http
$ go test -run TestTransportMaxConnsPerHost -count=100 -race
What did you expect to see?
Test passed
What did you see instead?
--- FAIL: TestTransportMaxConnsPerHost (0.07s)
    transport_test.go:661: Too many get connections (http2): 2
    transport_test.go:676: Too many get connections (http2): 3
--- FAIL: TestTransportMaxConnsPerHost (0.03s)
    transport_test.go:661: Too many get connections (http2): 2
    transport_test.go:676: Too many get connections (http2): 3
--- FAIL: TestTransportMaxConnsPerHost (0.08s)
    transport_test.go:661: Too many get connections (http2): 2
    transport_test.go:676: Too many get connections (http2): 3
--- FAIL: TestTransportMaxConnsPerHost (0.06s)
    transport_test.go:661: Too many get connections (http2): 2
    transport_test.go:676: Too many get connections (http2): 3
--- FAIL: TestTransportMaxConnsPerHost (0.03s)
    transport_test.go:661: Too many get connections (http2): 2
    transport_test.go:676: Too many get connections (http2): 3
FAIL
exit status 1
FAIL	net/http	4.433s

This is a bit interesting because we get the right number of dials and TLS handshakes but the connection returned is supposed not marked reused.

The problem is that the GotConn for http2 (h2_bundle:6878) is called before we know whether we are creating a new stream (h2_bundle:  7449) which is what determines reuse.
This is not a flaky test but a real bug.

Using the nextStreamID is flawed anyhow. If you allow HTTP, we start at 3 which means its immediately reused.

If we move the GotConn call into getClientConn(h2_bundle:749), we can distinguish between a new connection and a used connection.

➜  http  go test -run TestTransportMaxConnsPerHost -count=100 -race
PASS
ok  	net/http	1.641s

by the way,
➜  http go version
go version go1.12.5 darwin/amd64

I submitted a fix which does highlight the issue with http2. I did some other testing and found some of the other ClientTrace calls like TLSHandshakreXXX are missing.

The darwin-amd64-race builder caught this too:
https://build.golang.org/log/06b60fdfdf4a88e04ae4181d80ddd39db9aaafa0

I just got two TryBot flakes in a row on this:
https://storage.googleapis.com/go-build-log/06afa4b7/freebsd-amd64-12_0_63953b9e.log
https://storage.googleapis.com/go-build-log/433caf31/windows-amd64-2016_dcea98e4.log

CC @bradfitz

Whoops, I'd forgotten to submit 177037. Done. Hopefully it's better now.

In the other direction, it appears that TestTransportMaxConnsPerHost occasionally deadlocks: see #32336.

Closing as duplicate of #32336 now that we understand how severe that one is.


Feature/setup backend tables

In this PR, we did-

Recreate Users table: This table stores the details of all the users who register on the website, such as username, password, and email.
Recreate Doctors table: This table stores the details of all the items in the website, such as
classes It includes the item's name, bio, specialization,consultation_fee and photo.
Recreate Appointments table: This table stores the details of all the appointments made by the users, such as the item's name, the user's username, the date, and the city.

STATUS: APPROVED :heavy_check_mark: :tada:
Hi @Enning94 , @ankitt26   :wave:
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

[x] Created Tables  :+1:
[x] Nice Setup of the database.

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and :computer: Happy coding :computer:
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.400

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.400


switched from 2.8.1 to 4.3.1. Made no code migration. Why is it working?

I have used 2.8.1 version and now switched to 4.3.1 . I made no code migration6 just updated npm package, why is it still working? Should i wander? Or everything is ok?

There might be some things working, but you should look at the migration guide to be sure: https://github.com/ReactTraining/react-router/blob/master/packages/react-router/docs/guides/migrating.md