Vault Chat isn't working with Ollama

Hello, when selecting Vault chat it seems to reference some of my files while repeating some of them and i see this after the referencing "Understood! I will only include `
tags for existing markdown blocks you provide withstartLineandendLine` attributes when referencing them. For newly generated markdown content, line numbers won't be included.
I'm ready to assist with your Obsidian editing and organization questions using markdown format as specified. Please ask away!  "
After that whatever i prompt i get that 'I don't have access to your obsidian vault'...
I am using gemma 2-9b from ollama with bge-m3 for embedding...
Any thoughts why this is happening?
Also the button vault chat is doing the above in a started conversation, in a new conversation in the input box if i press it, it doesnt do anything...

Thank you for reporting this issue. This behavior is likely related to the limitations of smaller models when handling complex contexts and longer prompts. I will add this to our tracking issue #146.

Also the button vault chat is doing the above in a started conversation, in a new conversation in the input box if i press it, it doesnt do anything...
About this, could you provide more details about what you're experiencing?

https://imgur.com/a/w7rAPi7
Actually if i write  a prompt and press it works. But the first problem mentioned that it doesnt actually do anything with the referenced files is shown here...
Does the ai have context for file structure, tags, links etc? What i am after is to be able to talk and query our assistant more generally about our vault as a whole...
I tried making my own solution with training models on my own data, but it is more complex to get right...
FYI: Today i will fork your plugin to add a amazon bedrock integration, which has most models with 0 data retention...

Could you elaborate on which specific methods you'd like to see exposed? I'm not familiar with how other local models work.

I haven't taken a closer look to what exactly you are passing to each model call you make and where. But generally the idea would be to make an interface that has all the logic of the actions the plugin does in relation to ai calling, where its internal model API implementations could also pass through, and this way we could make external model implementations more easily as well...

I didn't manage to make it work with AWS Bedrock... Hope you implement it some day so will be able to use full functionality privacy free!

If AWS Bedrock supports OpenAI-compatible APIs, you should be able to use it by selecting 'Custom (OpenAI Compatible)' as your Chat model.
For more details on setting this up, you can check our documentation:
https://github.com/glowingjade/obsidian-smart-composer/wiki/1.4-Advanced-Configuration#custom-openai-compatible-models

Thanks for following up! If i do come up with a working solution i will share it but Amazon Bedrock uses a different API structure and authentication system (AWS credentials) that isn't compatible with that structure.

Thanks for sharing. While we don't have immediate plans to implement Amazon Bedrock integration, we're keeping track of user requests. If more users express interest in this support, we'll definitely consider prioritizing it in our roadmap. Would it be okay to close this issue?

Yes please do


add owner and comment to a telemetry event

@lramos15 ok hopefully I got it right now?

gah ok there we go

Ok @lramos15 how is this?

hopefully we're good here?


basic scrolling

add basic scrolling
render border-radius (no clip)
add react ScrollView

closes #21

merged this, BTW: scrolling looks choppy but it's actually not an issue with webrender or that we would have anything slow, it's simply because we are polling for events in setInterval() - this will get fixed but it's a big change


Filter Pull Up

Hey all,
This is an initial implementation of Filters Pull UP as a separate optimizer.

Hey Mark,
I think I implemented everything that was suggested / commented.
Let me know if I forgot something.

Thanks for the PR, apologies for the delayed merge. Looks great now!

Thanks for the PR, apologies for the delayed merge. Looks great now!


The benchmark results in an error but the check shows success

swift package --disable-sandbox \
      benchmark thresholds check \
      --path $PWD/Benchmarks/Thresholds/ \
      --format markdown

succeeds when the actual benchmark has resulted in an error.

You mean if the benchmark crashed, or?

No the benchmark run doesn't crash. It simply throws an error.
I noticed the benchmark says Checking ["benchmark name 1", "benchmark name 2"] around the end, but if the benchmark run has resulted in an error, it doesn't mention that benchmark's name in there at all.
Probably all benchmark runs for a specific benchmark must fail for this to happen.
Then the Benchmarking package doesn't have any results for that certain benchmark to compare to, so it simply skips the comparison instead of complaining that where are the results that I'm supposed ro compare to or something. It should just fail the benchmark comparison if any of the benchmarks resulted in an error.


Show Fishing spot tooltip when mousing over in World Map

Mining spots has specific ores in the world map when you hover over it

Can you please add the specific type of fish at the fishing spots?

Duplicate of #7317


Cannot fingerprint input property 'kotlinVersionCodeCustomizer': value '(GitTag, ProviderFactory, VariantInfo) -> kotlin.Int' cannot be serialized.

I'm getting the following error when running ./gradlew generateAppVersionInfoFor<Variant> with gradle configuration cache disabled.
> Cannot fingerprint input property 'kotlinVersionCodeCustomizer': value '(io.github.reactivecircus.appversioning.GitTag, org.gradle.api.provider.ProviderFactory, io.github.reactivecircus.appversioning.VariantInfo) -> kotlin.Int' cannot be serialized.

And I'm getting the following 2 errors when running ./gradlew generateAppVersionInfoFor<Variant> with gradle configuration cache enabled. the first error is the aforementioned one, and the the second one is:
1 problem was found storing the configuration cache.
- Task `:app:generateAppVersionInfoFor<Variant>` of type `io.github.reactivecircus.appversioning.tasks.GenerateAppVersionInfo`: cannot serialize Gradle script object references as these are not supported with the configuration cache.
  See https://docs.gradle.org/8.2.1/userguide/configuration_cache.html#config_cache:requirements:disallowed_types

I thought that configuration cache problems were addressed at #24, but even when the configuration cache is disabled, the build still fails with the first error above.
Steps to reproduce:
app-versioning: 1.3.1
Gradle: 8.2.1
AGP: 8.1.0
appVersioning extension configuration in my app's build.gradle.kts:
appVersioning {
    overrideVersionCode { _, _, _ ->
        extensions.getByType<ApplicationExtension>().defaultConfig.versionCode!! + 1
    }
}

Run generateAppVersionInfo for any variant, for example debug: ./gradlew generateAppVersionInfoForDebug
Additional Context:
I reproduced it on my side project https://github.com/mr3y-the-programmer/Ludi as I was trying to automate the release process for it.

My Use case is increasing the versionCode property by 1 on each release, this is why I used extensions to get the current versionCode but this is not strictly required, if that defeats the purpose of computing the version code lazily, I can avoid it and I think I will compute the versionCode using only the 3 parameters as you mentioned Thanks.

Follow up: After computing versionCode lazily, running generateAppVersionInfoFor<Variant> completes successfully without errors.

Could you show a code snippet how did you "compute versionCode lazily"?


allow for customisation for blog background COLOR (likely per single post) and more simple styles

dunnit (just post-wise bg colour)


Bug: Workspace Memory consumption

worked in beta 7, only occurs in 1.0.0-M1 now (Win and native Backends)
Following test crashes cause it's allocating 16 bytes instead of 8 bytes for a double.
    @Test
    public void testWorkspace(){

        Nd4j.setDefaultDataTypes(DataType.DOUBLE,DataType.DOUBLE);
        var wsMgr = Nd4j.getWorkspaceManager();
        var initialSize = 1 * Double.BYTES;
        var learningConfig = WorkspaceConfiguration.builder()
                .policyAllocation(AllocationPolicy.STRICT)
                .policySpill(SpillPolicy.FAIL)
                .initialSize(initialSize)
                .policyLearning(LearningPolicy.NONE)
                .policyReset(ResetPolicy.BLOCK_LEFT)
                //.policyLocation(LocationPolicy.RAM)
                .build();
        var workspace = wsMgr.getWorkspaceForCurrentThread(learningConfig, "test");
        workspace.enableDebug(true);
        workspace.notifyScopeEntered();

        var matrix = Nd4j.zeros(1,1,'c'); // 1 Element in our matrix
        matrix.migrate(); //supposed to alloc 1 element = 1 Double = 8 Bytes, but it's taking 16

    }

Leads to this exception
00:34:21.769 [INFO ] [Nd4jWorkspace alloc]: Workspace [test]: Allocating array of 16 bytes, capacity of 1 elements, prevOffset: 0; currentOffset: 16; address:<PHONE_NUMBER>792
00:34:21.910 [INFO ] [Nd4jWorkspace alloc]: Workspace [test]: step: 0, spilled  16 bytes, capacity of 1 elements

org.nd4j.linalg.exception.ND4JIllegalStateException: Can't allocate memory: Workspace is full

	at org.nd4j.linalg.api.memory.abstracts.Nd4jWorkspace.alloc(Nd4jWorkspace.java:440)
	at org.nd4j.linalg.api.memory.abstracts.Nd4jWorkspace.alloc(Nd4jWorkspace.java:320)
	at org.nd4j.linalg.cpu.nativecpu.buffer.BaseCpuDataBuffer.<init>(BaseCpuDataBuffer.java:485)
	at org.nd4j.linalg.cpu.nativecpu.buffer.DoubleBuffer.<init>(DoubleBuffer.java:74)
	at org.nd4j.linalg.cpu.nativecpu.buffer.DefaultDataBufferFactory.create(DefaultDataBufferFactory.java:327)
	at org.nd4j.linalg.factory.Nd4j.createBuffer(Nd4j.java:1455)
	at org.nd4j.linalg.api.ndarray.BaseNDArray.migrate(BaseNDArray.java:5363)
	at org.nd4j.linalg.api.ndarray.BaseNDArray.migrate(BaseNDArray.java:5342)

However 2 doubles will be allocated with 16 bytes. Some off by one issue?

@sascha08-15  thanks for the reproducer. We'll look in to this. We'll be turning around a follow up release pretty quickly (this week or next week) to address some issues the community has raised.

@sascha08-15 actually this isn't a bug. We pad the memory for alignment reasons. If you look at the actual required memory we actually require that for performance reasons. (We force alignment which causes the padding), you can see that here: https://github.com/eclipse/deeplearning4j/blob/4766032444de8e0c2c3389270576bb6e7c466211/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/memory/abstracts/Nd4jWorkspace.java#L345
Migrate() also allocates more memory. You only need to call notify scope entered and the create to show this case. I'm closing this.


Added example for hoverColorFrom and hoverColorTo in README

I nearly didn't use this library because I thought this option didn't exist.

Thanks


OCPBUGS-30672,OCPBUGS-32842: bump go-jose to fix CVE-2024-28176

bump github.com/go-jose/go-jose/v3 to v3.0.3, refer https://github.com/go-jose/go-jose/releases/tag/v3.0.3
bump gopkg.in/go-jose/go-jose.v2 to v2.6.3, refer https://github.com/go-jose/go-jose/releases/tag/v2.6.3

Signed-off-by: Avinal Kumar<EMAIL_ADDRESS>

/retest

/lgtm

/cherrypick release-4.15

/cherry-pick release-4.15

/cherrypick release-4.15 release-4.14 release-4.13 release-4.12

/cherrypick release-4.15 release-4.14 release-4.13 release-4.12

/cherrypick release-4.15


Update GHSA-3885-8gqc-3wpf.json

Add CVE-2022-30184 fix diff

Hi @JarLob, could you explain how you determined the patch fixes CVE-2022-30184?

The fix is mentioned here and it makes sense.


IGAPP-693: Add urdu as language

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

Testable here: http://<IP_ADDRESS>:9000/testumgebung/ur

Testable here: http://<IP_ADDRESS>:9000/testumgebung/ur

Does not work for me...

Testable here: http://<IP_ADDRESS>:9000/testumgebung/ur

Does not work for me...

@Taggotty sorry, I should have mentioned it is the malteser testumgebung, so https://malteapp.de/testumgebung/ur


Add Clown Corps comic source

Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[x] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[x] Have removed web_hi_res_512.png when adding a new extension

I forgot the dates
Will add those tomorrow!

I've been thinking... Wouldn't it be better if I stored the chapter cache outside the main reader app, and in the extension's own SharedPreferences instead?
That way, if the app gets reinstalled, the cache gets cleared.
I could have used this feature once or twice during development.
It's probably not very important, though.
Just a thought.

I've been thinking... Wouldn't it be better if I stored the chapter cache outside the main reader app, and in the extension's own SharedPreferences instead?

It's not possible. The code is loaded and run in the app's context, so you don't have access to the extension's shared preferences.


Adding initial support for curves to RadeonRays

For now only implemented in the "skip links" (aka "bvh") intersector.

Remarks:

Please convert tabs to spaces
Make sure to use correct naming conventions for both variables and methods/functions
Check performance since the pull request increases ray structure size? (or can we avoid size increase?)
Remove debug std::cout <<
Not sure why RadeonRays/src/intersector/intersector_skip_links.cpp is modified, it should not be touched.
Don't like putting shape type into the shape class, can we refactor it out somehow?

Yep, sorry.
Will do.
Will do perf checks as discussed (with old ray struct and no self-collision check, with new ray struct and self-collision check, then with extra ray data separated into a different buffer.  All with and without hair).
Yup.
As discussed, intersector_curves should be removed, and intersector_skip_links handles the logic for any combination of geo and curves in the scene. But this logic should explicitly define out any curve logic in the CL code if the scene is geo only.
Will take a look.

Fixed issues, ready to merge.
Disagree somewhat on the ShapeType enum, which is just analogous to e.g. the BxdfType enum -- if there are more than two derived classes, using RTTI to dynamic cast between them is more cumbersome that a switch (IMO).   In any case, easy enough to discuss and clean up later as needed.


Improve ocaml-formatter web-ui styles

Issue: #2
This PR consists of web-ui improvements:

Styles have been moved to separate files:

reset.css - сommonly used reset browser styles https://meyerweb.com/eric/tools/css/reset/
index.css - base and layout styles

Changed flex to grid. Result is the same but with fewer styles.
TBD

Before:

After:
TBD

what's the situation here?


Not working with create-react-app using Tailwind CLI

Trying to follow guide @ https://tailwindcss-custom-forms.netlify.com/, but for some reason the form components aren't ending up in my CSS file. Can't figure out what's wrong.
Using the CLI version of tailwind (version 1.1.2), since we're on create-react-app -> tailwind build ./src/tailwind.input.css -c ./src/tailwind.config.js -o ./src/index.css.
Happy to provide any other information that could be of interest :)

Can you share your tailwind.config.js and tailwind.input.css?

Wow, I goofed on this one. Forgot to import components via @tailwind components – thanks!


Design Assets produced and reviewed

logo and design assets are created
website is refined
accent colour is incorporate
will update discord later

Had a session with @boucherbox yesterday:

branding assets for mixer done
logo done
stickers done
after today's UI session portal is in focus


Improve process & documentation for non-k/k projects

decide what the PSC's role & involvement should be
write prescriptive checklist that can be followed by subproject owners without incident response experience
decide how vulnerabilities are communicated

/help

/remove-lifecycle stale
The bug bounty puts core Kubernetes in Tier 1, and GA subprojects in Tier 2. We'd like to make this explicit on each project, and also formalize what that means for PSC support (possibly with more tiers).
/cc @cjcullen @joelsmith

/remove-lifecycle stale
/lifecycle frozen


Update documentation on behaviour of destroy()

Update documentation of destroy() to clarify that the container is only removed if it wasn't manually set via the container property.
This brings the documentation in line with the inline code comment.

Merged, thanks!


Using grpc in an old C procject?

Hello,
I have an old C project and there I would like to add a small grpc server. So I would be great if there is any possibility to build grpc as a windows DLL. So I see there is a solution a DLL in the master. Is there any documentation available how to use that.
Or does someone can give me a little hint how to build grpc as a windows DLL?
many thanks
mathias

There's a 'vsprojects' folder in the git repo. Have you tried these?

Hello yes I  tried the visual Studio solutions.  I build the whole solution.  But I do not know how to use the grcp_dll from plain C.  Is there any example around how to use it.

The problem is I have an old project which is only compilable via mingw32. So I do need static libs for mingw32. So are there any plans that grpc is usable on mingw32 in the near future.


Spike: How to read barcode from identity cards

Description

Identity Documents and their respective barcode format
Which formats are supported in the library
Identify each document layout/guide reticle
Priority given to BCSC

Nice to Have: BC Driver's license

BC Driver's license may have 2 codes

Notes:
There are two barcodes on the BC Services Card (same as on the BC DL). The serial number is a 1-D barcode on the right hand side of the back. While the 2D barcode is on the bottom side of the back. See this sample DL: https://images.ctfassets.net/nnc41duedoho/7za52YKEvPiRnuoArZz07B/d76195920ba339c03c2be814f5442090/DL-sample.jpg?
The 1-D barcode is pretty straightforward and translates to raw text. Any barcode scan library will give you something like the type of barcode and the raw text.
The 2-D barcode is more complicated, and may contain more information than needed. By complicated, I mean that it's a PDF417 barcode, but doesn't follow the AAMVA standard for BC issued identity docs. The barcode for BC issued documents actually contains text in the same format as the magstripe.
APD has some server side code that converts both types of PDF417 barcodes into a "canonical" format for our needs
In my opinion, our priority is the 1-D barcode for current capabilities, but the stretch goal is to extract both barcodes at the same time from the same frame (same UI interaction for the user). We can deal with the raw barcode text on the backend.
For test and sample IDs, you don't need to make any API calls yet, and you can generate as many as you need using tools like https://barcode.tec-it.com/en

Open an incognito browser to https://barcode.tec-it.com/en

(may need to Accept Cookies)

Choose 2D Codes > PDF417
Enter the desired content in the text box (see next section)
Click on Create Sequence
Download or take a screen shot of the resulting barcode

Here's a sample of the PDF417 for BC docs (line breaks for readability):
%BCVICTORIA^SURNAME,$GIVENONE GIVENTWO^123 FIRST ST$VICTORIA BC  V8X 2S8^
?;6360281679897=260419900403=
?_%0AV8X2S8                     M183 73BRNBRN9876543212                $B\"/G:HV76W?

That same site that I list above can generate the 1D serial number barcodes. The serial numbers are currently 9 characters. 1 alpha followed by 8 digits. Like T00252783.

With some minor tweaks the react native vision library supports the 1D and 2D barcodes on BC Services card and drivers license. I was able to read the two barcodes on my driver's license without any issue. Here's the code I used:
const codeScannerExperimental = useCodeScanner({
    codeTypes: ['qr', 'code-128', 'pdf-417'],
    onCodeScanned: (val)=>{console.log(val)}
  })
  return (
    <View style={[StyleSheet.absoluteFill, { transform: [{ rotate: orientationDegrees[orientation] ?? '0deg' }] }]}>
      {device && (
        <Camera
          style={StyleSheet.absoluteFill}
          device={device}
          torch={torchActive ? 'on' : 'off'}
          isActive={cameraActive}
          codeScanner={codeScannerExperimental}
          format={format}
        />
      )}
    </View>


South African public holidays

Thank you very much for your contribution!
Do you have a link to a kind of official resource where people can check where you got the information from? It wod be great if you could add that either here to the PR or (even better) as content if the git commit message.
Thank you!

Hey @heiglandreas
Thank you :) I have amended the latest commit with the appropriate source. For reference, here it is: https://www.gov.za/about-sa/public-holidays
Regards,
Sarel

Thank you!
I merged this even though I've seen that the Public holidays Act defines that each time a holiday falls on a sunday the following Monday shall be a holiday. So instead of using the date tag, we probably should use the datefollowup-tag for at least those that might occur on different weekdays.

Though Ive just seen that i need to make the FollowUp-IteratorItem more configurable as it currently would move a holyday that falls on a saturday onto the following specified day as well. Which is not correct for south africa....
So it looks like I will have to modify that a bit :-) Thanks for letting me know

Ah ok yes I was wondering about that. Ok let me know when you have the tweak in for the datefollowup-tag, and I'll make a PR to fix the XML up.
Thank you for this library though, it's going to be super useful to us!


Explanation of run/render [1-43]

[id: 1-43], [MAJOR]: Sorry if it is just me but the example is not educative enough. While calling the
render function fits my expectation as a user which would like to create (i.e. render) a cwl workflow
form a defined pipeline as code workflow I don't understand the purpose of the run function. To my
opinion the documentation lacks here explanations and background.

the run method has been changed to construct


Text node transform to 3d

Is there text node in pixi3d like Sprite3D？

There is not anything built in, but I just realise you can do this:
let text = new Text("test")
text.updateText()

let sprite = app.stage.addChild(new Sprite3D(text.texture));

Will create the text texture and then use that with a sprite 3d.


The shop will not open. Any Ideas?

I can not open the shop. When I do I only get an infinite connecting message and it never opens. I then have to ALT + F4 to close the title since I have no control. Any ideas on how to fix this? I am on the latest.

Follow same guidance as here https://github.com/pixeltris/YgoMaster/issues/263#issuecomment-1804834086


ChroniquesGalactiques Version 4.2.0

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Removed the buggy auto-calculation of hit-points when changing level or CON
Added an optional field for Psy Instability (psionics)
Added a new tab for charsheet version with release notes and a read-only log of data migrations if any

Character Sheet Info Roll20 Internal Use only.


Got error: multipart: NextPart: EOF

I got multipart error at server side.
code=400, message=multipart: NextPart: EOF, internal=multipart: NextPart: EOF

I made code sets to reproduce error
https://github.com/atsushi-ishibashi/runn-repro-326

I detected the below reset body caused it and when you set header again, you will get success.
https://github.com/k1LoW/runn/blob/e43ef04aa7b5c09c9a1542a3ad339730fdab8d55/http.go#L228

Considering this PR and #310 , I think no one has used it yet so can I make breaking change for multipart body format to simplify implementation?
（このPRの不具合を鑑みるとまだ僕以外は誰も使ってないと思うので #310 の修正で互換性維持するために難しい実装になるくらいなら互換性壊す修正にしても良いですか？）

I cannot imagine how to support map and slice in a simple way.
values, ok := r.body.(map[string]interface{})
&&
values, ok := r.body.([]interface{})

I think no one has used it yet so can I make breaking change for multipart body format to simplify implementation?

OK 👍

I cannot imagine how to support map and slice in a simple way.

I often use type-switch to solve that problem.
https://go.dev/tour/methods/16


Select no coins to spent which invalidate Omni transactions

I noticed this quite some time ago, when writing bitcoin-spock tests, and we finally found sneaky ways to work around it, but just right now I fully realized the actual issue is part of Omni Core, and not something that should have been addressed in bitcoin-spock.
The current wallet/spending code does not check, if the coins to spent violate consensus rules.
The consensus rules are pretty simple: only pay-to-pubkey-hash and pay-to-script-hash outputs qualify as valid inputs.
The issue while testing was that the coin selection (used by sendrawtx_MP or send_MP) often selects coinbase transaction coins, which result in invalidated Omni transactions, because coinbase transaction coins are no pay-to-pubkey-hash or pay-to-script-hash coins, and thus not valid.

Hey bud, sorry for delayed response - was a busy weekend with family...  I just reviewed your PR for this issue and support it.
Thanks
Z

Was addressed by #7.


Workaround: 'as:Public' not recognized by many implementations

Many implementations, e.g. Pleroma and many relays, do not recognize as:Public, but only know https://www.w3.org/ns/activitystreams#Public in to/cc for public posts. This causes public posts from Takahe being rejected or ignored.
This patches outbound json to use full uri instead of as:Public. This is more of hack than a proper implementation. Suggestions and improvements are welcomed.

The correct way to do this is to instead change the context that the JSON-LD canonicaliser uses, to remove the "as:Public" short identifer - that way the JSON-LD library will leave it expanded.


Collections back-end: Tech spec review

Review tech spec - https://docs.google.com/document/d/1-ATD0ykdvSa1NTUL8XeE4FiHFe4jDGcFapzaNnQgi6s/edit# - and come up with implementation plan.
Deliverable of this spike should be to fill in more implementation details into this ticket https://github.com/chanzuckerberg/napari-hub/issues/506 or break it out into a few smaller implementation tickets.

@richaagarwal and @klai95 will have a working session on the tech spec

I have updated the tech spec with an additional section, "Alternative to S3 cache", viewable here - https://docs.google.com/document/d/1-ATD0ykdvSa1NTUL8XeE4FiHFe4jDGcFapzaNnQgi6s/edit#heading=h.day8153r64bf
I have updated two tickets, https://github.com/chanzuckerberg/napari-hub/issues/506 and https://github.com/chanzuckerberg/napari-hub/issues/507, with the relevant information.
I still need to create an additional ticket to reflect a caching approach (via possibly CloudFront), and then we should have the tech spec re-reviewed by additional colleagues. I'll leave this open for now, but the above two tickets are considered "Ready" as they will help inform our proof of concept regardless of if they are the final approach.
@neuromusic How does that sound?

@richaagarwal perfect, thank you!!

Closing this as I've made some revisions to the tech spec that have informed the work in #506 and #507 and will be sending out for broader review/alignment.


Added HTTP(S) Basic Authentication

Thanks for creating this.  While simple, this is probably currently the most professional Micropython OTA update facility online.
To enhance it, I've added HTTP(S) Basic Authentication capability in code and documentation.  While I haven't done a huge amount of testing, I have tested it on an RPi Pico W and with Apache/2.4.54 server using a Let's Encrypt SSL cert.

Hi @LexxM3 thanks for your contribution!
I will have a look at this shortly.
Could you please in the meantime add some tests?

Could you please in the meantime add some tests?

I am not an expert with Python unittest, but welcome this opportunity to learn. So yes, will definitely do as quickly as time allows.
I can figure out most things, but can you please confirm one observation/assumption: your test framework is intended to be set up and run on a "regular" (non-Micro) Python environment with a regular filesystem, correct? As otherwise, setting that up in MicroPython on bare metal hardware seems to be cumbersome.

@LexxM3 Correct, simple unit tests using mocks.
I just thought about providing unit tests for your feature, but I'm not sure if it would be that trivial to implement. In particular it should be checked if the desired authorization-header is sent to the server if credentials are provided.

@LexxM3 Hi, are you still on it? Otherwise I would do some changes and finish it.

Due to some merge conflicts I created a new branch/PR and replayed your changes there and did some optimizations (#13)


The -i flag has confusing documentation

Calling manim with the -i flag saves the video as .gif IN ADDITION to also saving it as .mp4. I was expecting it to be saved as .gif INSTEAD of as a .mp4.
The fix would be to have the documentation and the manim --help text document this fact.
Also, in the following output, it is specified that the .mp4 file has been saved, but not that the .gif file has been saved:

The fix would be to have the documentation and the manim --help text document this fact.

Mmmmh, why not fixing -i instead of its documentation? We could make -i save only .gif

Either is fine by me, as long as (at least) it's very clear what is happening. Right now the output shown to the user is incomplete


Installing debugger on windows

I am trying to install debugger for dbeaver in Windows..
Can anybody help me in proper steps please?

Please see the instruction https://github.com/dbeaver/dbeaver/wiki/Optional-extensions

Hi,
Sorry for the late reply.. I have installed plugin in dbeaver but still
getting the same error as debugger plugin is not found..can you help me
with this, please?
On Tue, 17 Nov 2020, 1:00 pm Kseniia<EMAIL_ADDRESS>wrote:

Closed #10293 https://github.com/dbeaver/dbeaver/issues/10293.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/10293#event-4003974714, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIYSLZT4CWZZYDMKE5EL3ILSQIQ7XANCNFSM4TPY7Z6A
.

Please send log files https://github.com/dbeaver/dbeaver/wiki/Log-files
What is your DBeaver version?

My dbeaver version is 7.2.5 which I recently updated..
Is it possible that you provide me baby steps to install debugger in
dbeaver like from start what I have to do and what tools needed if any,
please?
It will be actually a great help for me..I want to make the best use of
dbeaver in our project and we are in desperate need of debugger like the
one you have shared on twitter..
Thanks..
On Wed 25 Nov, 2020, 4:23 PM Kseniia<EMAIL_ADDRESS>wrote:

Please send log files https://github.com/dbeaver/dbeaver/wiki/Log-files
What is your DBeaver version?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/10293#issuecomment-733631692,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIYSLZWCR54CZALL4ZWNIG3SRTO25ANCNFSM4TPY7Z6A
.

Waiting for your reply..
Thanks..
On Thu, 26 Nov 2020, 1:43 am Shraddha Pol<EMAIL_ADDRESS>wrote:

My dbeaver version is 7.2.5 which I recently updated..
Is it possible that you provide me baby steps to install debugger in
dbeaver like from start what I have to do and what tools needed if any,
please?
It will be actually a great help for me..I want to make the best use of
dbeaver in our project and we are in desperate need of debugger like the
one you have shared on twitter..
Thanks..
On Wed 25 Nov, 2020, 4:23 PM Kseniia<EMAIL_ADDRESS>wrote:

Please send log files https://github.com/dbeaver/dbeaver/wiki/Log-files
What is your DBeaver version?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/10293#issuecomment-733631692,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIYSLZWCR54CZALL4ZWNIG3SRTO25ANCNFSM4TPY7Z6A
.

duplicates #10192 .
issue will be resolved in the scope of the mentioned ticket. Thanks for the report.
Best regards


react/jsx-pascal-case flags Styled.h1 as not being in pascal case

While using the Theme-UI Styled Component <Styled.h1> in gatsby, this plugin fires off a console warnings about not being pascal case

the lint rule wants that to be <Styled.H1>, yes.

Is there a reason then why when testing <Styled.p> does not raise the same errors? Is this intentional?

I understand the reasoning behind the other error. Just trying to grasp the larger picture.

"p" is one thing, where "h1" is two things ("h" and "1").
I can see the argument where if "p" is ignored, dot-anything should be ignored too. I suspect that this is more the rule overlooking the (rare, unidiomatic) use case of namespaced components, rather than an intentional decision.
The "ignore" option that will be included in the next release may help you here; perhaps also a separate option to control the behavior on namespaced tag names?

Is it ok if I create a PR then for this feature?

Sure. Let's first maybe discuss what the option would be named, and what its semantics would be :-)

I guess most easily, the option could be named namespace and automatically set to false (?) where if set to true, it ignores all case issues after the dot. Perhaps that isn't rigorous enough in which case, do you have any suggestions as to what would be a better approach?

Or perhaps the rule should not warn on components that start with a lowercase after the dot?

Also, looking through the current logic, it seems that if the type is JSXNamespacedName or JSXMemberExpression, it does not verify at all the the string before the . or :.
For example, <$Test.P> or <$Test:P> are valid according to the rules.


Thanks @Random-Liu, I have signed the commit, kindly review and merge.


Fix typo in using with css in js libs docs

Fix typo in 'Using with CSS-in-JS libs' section.

Thanks!


Add lint for BR <IP_ADDRESS>.2a mailbox-validated

The changes to san.go are copied from #800.

Thanks for the review.
Yes, I intended to check only the mailbox-validated certificates with this PR (as indicated by the title and lint filename). I updated the description.


fix: support injectScriptsAt: 'STREAM'

Hi @brillout, What do you think of this PR?

In principle, yes, that's what we want. Although maybe the implementation could be made easier, let me have a look.

You're injecting chunks at any arbitrary position, does that work?
This doesn't work with React, see https://github.com/brillout/react-streaming/tree/main/src#readme. But maybe this works with Solid?

Although maybe the implementation could be made easier, let me have a look.

I think the implementation can be made a bit simpler? (Although it may require a little bit of refactoring of the existing code.) I can take over the implementation, as you wish and whichever you prefer.
But, before that, it's probably worth it to dig a bit whether injecting at any arbitrary position in the steam works.

I think it's not entirely at any arbitrary position. I injected it after the first stream is written, and I make sure it's only injected once.
https://github.com/vikejs/vike/pull/1740/files#diff-6ed114a319272b2469f4a1bab05dab39abcac240cb4a52a61666abd117f84975R349-R352
I've tried it and it works, here is the test video
https://github.com/vikejs/vike/assets/39896528/d5a934ba-270f-424f-8fbc-f57f8fc72c97

I believe solidjs is okay with it, also from the test I've commented in vike-solid issue https://github.com/vikejs/vike-solid/issues/95#issuecomment-2212395950, we can even inject all of that scripts in HTML_BEGIN.

I can take over the implementation, as you wish and whichever you prefer.

Sure, I really appreciate that :pray: . I had thought of a few implementations before I finally made this PR.

We could have a look at the chunks Solid writes to the stream

So I've looked at it, and here are the sequences:

If there is a promise inside a page or component with <Suspense> :

3:10:22 PM [vike][request(3)] HTTP request: /

fetchMovies start
chunk 1: Non promise chunk, until onCompleteShell()

Show

    WelcomeTodoData Fetching

My Vike app

      This page is:

Rendered to HTML.

          Interactive.
          <button data-hk="0-0-0-0-2-1-0-2-0" type="button">
            Counter
            0</button
          >

Star Wars Movies

      <template id="pl-0-0-0-0-2-1-0-4-0"></template>

Loading ...

3:10:25 PM [vike][request(3)] HTTP response / 200

fetchMovies done
chunk 2 : template tag

Show
<template id="0-0-0-0-2-1-0-4-0">

      A New Hope
      (1977-05-25)

      The Empire Strikes Back
      (1980-05-17)

      Return of the Jedi
      (1983-05-25)

      The Phantom Menace
      (1999-05-19)

      Attack of the Clones
      (2002-05-16)

      Revenge of the Sith
      (2005-05-19)

</template>

chunk 3 : Long Replace script

Show

Last chunk 4 : onCompeteAll()

If there is an async component without using <Suspense>, then there is no template tag chunk:

chunk 1: Non promise, until onCompleteShell()
chunk 2: Long Script
chunk 3: onCompleteAll()

If there is no promise inside a page or component, then there are only 2 chunks:

3:02:54 PM [vike][request(1)] HTTP request: /todo

chunk 1 : non promise chunk until onCompleteShell()
chunk 2 : onCompleteAll()

Even with this fix, there is one condition: the script tags will still be injected at the end before the closing  tag, which occurs if we return a pageContext promise in onRenderHtml()

Indeed, it seems like inserting chunks in-between wouldn't cause any issue.

Even with this fix, there is one condition: the script tags will still be injected at the end before the closing  tag, which occurs if we return a pageContext promise in onRenderHtml()

I think that's fine, it's an inherent limitation (independently of Solid).

Alternatively, how about we create a new Vike setting injectVikeScriptsAtHtmlBegin: boolean? This would be the easiest solution I guess.

If we're still unsure about this fix, then I agree with your idea :+1:. Since what we want to achieve is to be able to inject all those script tags (especially the client entry script) before all promises are resolved

In principle, I think it's fine and we can go either way (STREAM or HTML_BEGIN).
It's just that I've quite a lot on my plate, so that's why I'm inclined to go for the HTML_BEGIN approach for now, as it's probably much easier to implement.
That said, if you want you can give this PR another try: I'll happily merge it if becomes simpler and more straightforward. (I'm reluctant to merge the PR as-is because it would make the source code a bit harder to understand.)
I'm fine either way, whichever you prefer.

Awesome :100: , thanks.
I can try improve this code and also try create a new Vike setting injectVikeScriptsAtHtmlBegin: boolean. Let's not rush.
Update :

If there is an async component without using , ...

I think this is a rare use case, but It turns out, if we inject the client script tag after the stream end (injectStringAtEnd). Then we'll get Error: Hydration Mismatch.
And it doesn't happen either with this fix or HTML_BEGIN :+1:

Sounds good!

I'll happily merge it if becomes simpler and more straightforward.

Can we add support for 'STREAM' to our new setting injectScriptsAt in a fairly straightforward way? I'm inclined to think so, but maybe I'm missing something.
If it's too complex, then let's close this PR and postpone it. Especially given that I ain't sure how much performance gain there is to inject at beginning of the stream VS inject at the beginning of the HTML.

Yes, we can try that. Let me see if it works fine with Vue streaming.

https://github.com/vikejs/vike/pull/1740/commits/ba46b32c4f504683bb53065e7f07d658a8fc3ad0

injectScriptsAt: 'STREAM' doesn't play nice with Vue streaming at all (with or without async components & <Suspense>).
injectScriptsAt: 'HTML_BEGIN' with Vue streaming only works when there are no async component & <Suspense>.
SolidJs Streaming works fine :+1:

Especially given that I ain't sure how much performance gain there is to inject at beginning of the stream VS inject at the beginning of the HTML.

I'm not sure either, but I think vike-solid streaming and vike-react streaming are kinda similar or identical now (except that vike-solid flushes the first stream and the script tags in one go).
Below is the result with DEBUG=vike:stream pnpm dev. Here, I'm showing only from fetchMovies start to fetchMovies done.

vike-react streaming

Show
2:22:47 AM [vike][request(1)] HTTP request: /streaming
fetchMovies start
vike:stream onRenderHtml() hook returned react-streaming result
vike:stream onRenderHtml() hook returned Node.js Stream Pipe
vike:stream stream promise resolved
2:22:48 AM [vike][request(1)] HTTP response /streaming 200
vike:stream original Node.js Writable received
vike:stream stream begin
vike:stream data written (Node.js Writable)
            <!DOCTYPE html>

                    <link rel="stylesheet" type="text/css" href="/pages/style.css?direct">
                <meta charset="UTF-8" />
                <title>My Vike + React App</title><meta property="og:title" content="My Vike + React App" />
                <meta name="viewport" content="width=device-width, initial-scale=1"/><meta name="description" content="Demo showcasing Vike + React"/><link rel="icon" href="/ass
            ets/logo.svg"/>

vike:stream data written (Node.js Writable)

WelcomeData FetchingHTML StreamingWithout SSRNested LayoutClient Only

Star Wars Movies

Same as /star-wars page, but showcasing HTML Streaming and Progressive Rendering. (Note how the interactive counter works before the data finished loading.)
<button type
            ="button">Counter 0</button><template id="B:0"></template>
Loading...

vike:stream stream flushed
vike:stream data written (Node.js Writable)

vike:stream stream promise resolved
vike:stream stream flushed
fetchMovies done
...

vike-solid streaming

Show
2:30:43 AM [vike][request(1)] HTTP request: /
fetchMovies start
on server
vike:stream onRenderHtml() hook returned Node.js Stream Pipe
vike:stream stream promise resolved
2:30:43 AM [vike][request(1)] HTTP response / 200
vike:stream original Node.js Writable received
vike:stream stream begin
vike:stream data written (Node.js Writable)
            <!DOCTYPE html>

                    <link rel="stylesheet" type="text/css" href="/layouts/style.css?direct">
                    <meta charset="UTF-8" />
                    <title>My Vike App</title>
                    <meta data-hk="0-0-0-0" name="viewport" content="width=device-width, initial-scale=1"><meta data-hk="0-0-0-1" name="description" content="Demo showcasing Vike"><link data-
            hk="0-0-0-2" rel="icon" href="/assets/logo.svg">

vike:stream data written (Node.js Writable)

WelcomeTodo

My Vike app
T
            his page is:

Rendered to HTML.

Interactive. <button data-hk="0-0-0-0-2-1-0-2-0" type="button">Counter 0</button><!
            --/-->

Star Wars Movies
<template id="pl-0-0-0-0-2-1-0-4-0"></template>
Loading ...

vike:stream data written (Node.js Writable)

vike:stream stream flushed
fetchMovies done
...

I think we can ignore the fact that it doesn't work with Vue for now?
FYI the PR can be made easier to read by targeting the other PR instead of main (it's a GitHub PR setting).

I think we can ignore the fact that it doesn't work with Vue for now?

Yes, I think so

FYI the PR can be made easier to read by targeting the other PR instead of main (it's a GitHub PR setting).

Noted, thanks

Let's rebase.

I think we can merge this after we removed the hack and updated the docs.

What about these two options?

use injectHtmlTagsUsingStream() :

--- a/vike/node/runtime/html/injectAssets/injectHtmlTags.ts
+++ b/vike/node/runtime/html/injectAssets/injectHtmlTags.ts
@@ -25,13 +25,14 @@ function injectHtmlTags(htmlString: string, htmlTags: HtmlTag[], position: Posit
 function injectHtmlTagsUsingStream(
   htmlTags: HtmlTag[],
   streamFromReactStreamingPackage: null | StreamFromReactStreamingPackage
-): void {
+): void | string {
   const htmlFragment = joinHtmlTags(htmlTags.filter((h) => h.position === 'STREAM'))
   if (htmlFragment && streamFromReactStreamingPackage) {
     assert(streamFromReactStreamingPackage)
     assert(!streamFromReactStreamingPackage.hasStreamEnded())
     streamFromReactStreamingPackage.injectToStream(htmlFragment, { flush: true })
   }
+  return htmlFragment
 }

--- a/vike/node/runtime/html/injectAssets.ts
+++ b/vike/node/runtime/html/injectAssets.ts
@@ -71,7 +71,8 @@ function injectHtmlTagsToStream(

   function injectAtStreamMiddle(): string {
     assert(htmlTags)
-    const htmlMiddle = injectHtmlTags('', htmlTags, 'STREAM')
+    const htmlMiddle = injectHtmlTagsUsingStream(htmlTags, null)
+    assert(htmlMiddle)
     return htmlMiddle
   }

export and use joinHtmlTags()

--- a/vike/node/runtime/html/injectAssets/injectHtmlTags.ts
+++ b/vike/node/runtime/html/injectAssets/injectHtmlTags.ts
@@ -3,7 +3,7 @@
 export { injectHtmlTags }
 export { injectHtmlTagsUsingStream }
 export { createHtmlHeadIfMissing }
-
+export { joinHtmlTags }

--- a/vike/node/runtime/html/injectAssets.ts
+++ b/vike/node/runtime/html/injectAssets.ts
@@ -6,7 +6,12 @@ export type { PageContextPromise }
 import { assert, isCallable, isPromise } from '../utils.js'
 import type { PageAsset } from '../renderPage/getPageAssets.js'
 import { assertPageContextProvidedByUser } from '../../../shared/assertPageContextProvidedByUser.js'
-import { injectHtmlTags, createHtmlHeadIfMissing, injectHtmlTagsUsingStream } from './injectAssets/injectHtmlTags.js'
+import {
+  injectHtmlTags,
+  createHtmlHeadIfMissing,
+  injectHtmlTagsUsingStream,
+  joinHtmlTags
+} from './injectAssets/injectHtmlTags.js'
 import type { HtmlPart } from './renderHtml.js'
 import { getHtmlTags, type PreloadFilter, type HtmlTag } from './injectAssets/getHtmlTags.js'
 import type { StreamFromReactStreamingPackage } from './stream/react-streaming.js'
@@ -71,7 +76,7 @@ function injectHtmlTagsToStream(

   function injectAtStreamMiddle(): string {
     assert(htmlTags)
-    const htmlMiddle = injectHtmlTags('', htmlTags, 'STREAM')
+    const htmlMiddle = joinHtmlTags(htmlTags.filter((h) => h.position === 'STREAM'))
     return htmlMiddle
   }

Personally I prefer 2. much more. (But maybe I'm missing something.)
This will likely make it LGTM.

Nice, I just want to make sure if it's okay to export joinHtmlTags() for a single use

I think we can merge after [...] updated the docs

:)
I think the docs changes are easy, but let me know if you want me to help.

Merged :rocket:

FYI, this assert will fail when using injectScriptsAt: 'STREAM'
https://github.com/vikejs/vike/blob/b0d2d3a435025aad94e12d51d68e6de676de15a7/vike/node/runtime/html/injectAssets/injectHtmlTags.ts#L31

Indeed, I believe it's fixed now.
I've pre-released it as 0.4.179-commit-9822036. Does it work now?

Awesome :100:
Confirmed it works as expected. Thanks

FYI I'm starting to think that, maybe, we should remove the whole STREAM position. It doesn't seem to justify the added complexity.
It boils down to this: injecting scripts right before  VS right after the first stream chunk. It doesn't seem to make much of a difference. I'm even inclined to think that injecting before  is more performant.
Anyways, let's keep it for now and remove it if it causes another issue in the future.
I should have thought this through when I first implemented the STREAM position. My bad.

It was also due to my lack of knowledge. We're all doing our best; let's hope for the best.


chore(cherry-pick): cherry-pick #7562 #7539 #7482

cherry-pick #7562 #7539 #7482

Just want to make sure there is no conflict happened due to cherry pick?

Just want to make sure there is no conflict happened due to cherry pick?

Is to allow ci to pass.


Update bug_report.yml to remove shell attribute in logs

Logs are saved as shell input, which means they get rendered directly as a code block in the final output.  Let's change this to not be rendered.
The motivation for this is to enable something like this issue's logs, where we can use a <details> ... </details> block to hide the logs in an expandable text block.  If the logs are rendered as shell, the user cannot do this.

Thanks @ca-scribner , this change does not apply anymore. We are now using a different template. Closing.


#125 add toggle events

Events - minimized, restore and `toggle for both;
add event example;
update readme.

Hey, this is great! There are two things I would like you to change:

The automatic check failed because your changes break the linting rules some places. Here is the output from npm test:

happiness: Use Happiness (https://github.com/JedWatson/happiness)

  /home/travis/build/Norkart/Leaflet-MiniMap/src/Control.MiniMap.js:324:35: Missing space before function parentheses.

  /home/travis/build/Norkart/Leaflet-MiniMap/src/Control.MiniMap.js:335:31: Strings must use singlequote.

  /home/travis/build/Norkart/Leaflet-MiniMap/src/Control.MiniMap.js:335:45: Strings must use singlequote.

  /home/travis/build/Norkart/Leaflet-MiniMap/src/Control.MiniMap.js:336:13: Strings must use singlequote.

Please change the event name for minimising to minimize so that it is consistent with the other event names.

After that I will be happy to merge this. 👍

Fabulous, thank you for your contribution! 🥇


feature/my-space

personal folder

your README file is not in UPPERCASE. Please change this before!


Instance resets to its default values when focused on through the search results list

Current behaviour:
When a node is selected from a search result list, its generics, fields, and properties all return to the default values for that node's type.
Desired behaviour:
When a node is selected from a search result list, no changes should be made to that node.

Fixed in v1.6.1. The resetting behaviour was caused by a node reloading the default model of its type after being focused on by clicking on its search result.
As of v1.6.1, nodes keep track of all previously loaded models and their latest generics, fields, and properties, and will always defer to a cached model over a default model for a given type. This means that a node will never be able to load its type's default model after it has already been loaded once, thereby fixing this issue.


Installing lit from esm.sh provider is failing

Installation of lit package is failing right now when using esm.sh as a provider.
Expected
Installs the package and updates the import-map
Result
Fails with a error in resolving a import.
file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:1697
            throw new JspmError(`No '${subpath}' exports subpath defined in ${pkgUrl} resolving ${originalSpecifier}${importedFrom(parentUrl)}.`, "MODULE_NOT_FOUND");
                  ^

JspmError: No './css-tag' exports subpath defined in https://esm.sh/*<EMAIL_ADDRESS>resolving @lit/reactive-element/css-tag imported from https://esm.sh/v135/<EMAIL_ADDRESS>    at throwExportNotDefined (file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:1697:19)
    at Resolver.resolveExport (file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:1726:17)
    at async TraceMap.resolve (file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:2915:59)
    at async TraceMap.visit (file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:2719:26)
    at async file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:2756:13
    at async Promise.all (index 0)
    at async TraceMap.visit (file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:2751:9)
    at async file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:2756:13
    at async Promise.all (index 0)
    at async TraceMap.visit (file:///workspaces/sandbox/node_modules/@jspm/generator/dist/generator-387fd949.js:2751:9) {
  jspmError: true,
  code: 'MODULE_NOT_FOUND'
}

The resolution of the path /css-tag seems to failing. Could it be because the pacakge is exposing the modules with the file extension.
https://unpkg.com/browse/<EMAIL_ADDRESS>"./css-tag.js": {
      "types": "./development/css-tag.d.ts",
      "node": {
        "development": "./node/development/css-tag.js",
        "default": "./node/css-tag.js"
      },
      "development": "./development/css-tag.js",
      "default": "./css-tag.js"
    },

Here is a reproduction for the bug
https://codesandbox.io/p/devbox/list-esm-install-bug-y72sjm?file=%2Findex.js%3A4%2C18

I've disabled the esm.sh provider test for now. I think this is an esm.sh issue actually //cc @ije


up disparity

Using your code I'm able to compute the disparity Left, I'm wondering if your algorithm support also up or down disparity?
Thank you

It should, yes. You may need to rotate/flip the images so that the disparity is always negative, which is how our model was trained.


TileLayer - all tiles in MemoryCache

Hi,
referring to https://github.com/BruTile/BruTile/issues/106
How can I get all tiles cached in TileLayer (Viewport)?
From this MemoryCache.TileCount I know how many there are, but I don't know what the max and min indexes of columns and rows are (to use in MemoryCache.Find).
Maybe there is a way to get the col/row indexes in the Map.Viewport (.Extent) ?
Thanks,
Lukasz

I'm doing:
`var tileCount = tileLayer.MemoryCache.TileCount;
var minCol = 0;//?
var maxCol = tileCount;//?
var minRow = 0;//?
var maxRow = tileCount;//?
var zoomLevel = "4";
for (int colIdx = minCol; colIdx < maxCol; colIdx++)
for (int rowIdx = minRow; rowIdx < maxRow; rowIdx++)
{
var feature = tileLayer.MemoryCache.Find(new BruTile.TileIndex(colIdx, rowIdx, zoomLevel));//above 8 (zoomLevel>8) it's very long process
if (feature != null)
{
    //using the tile from MemoryCache
}

}`
It would be nice to have something like this:
var allTiles = baseLayer.MemoryCache._bitmaps;

I need this
new System.Collections.Generic.Mscorlib_DictionaryDebugView<BruTile.TileIndex, Mapsui.Providers.Feature>(tileLayer.MemoryCache._bitmaps).Items :(

//using the tile from MemoryCache
var imgData = ((Mapsui.Geometries.Raster)feature.Geometry.Clone()).Data;
imgData.Position = 0;

using (System.IO.FileStream file = new System.IO.FileStream("tile_" + zoomLevel + "_" + colIdx + "_" + rowIdx + ".png", System.IO.FileMode.Create, System.IO.FileAccess.Write))
    imgData.CopyTo(file);

imgData.Dispose();
imgData = null;

You want to create a cache on the file system? Be careful, the most tile providers will not allow this.
I would say, the suggested approach with the own MemoryCache would be the best.

I know that they may not allow this (a ban, block IP etc.). This will be a temporary, occasional, experimental solution. Not for permanent use. I'm careful. ;)
"own MemoryCache"... as System.Collections.Generic.Dictionary<BruTile.TileIndex, Mapsui.Providers.Feature>... I'm not sure how to do this... inherit TileLayer in own TileLayer?
Thanks.

The only thing, you have to do, is, when creating the TileProvider/TileSource for your layer, to add your own cache. That’s it.

Sorry, I don't know (yet) how to do this. Any sample?

		public TileLayer CreateTileLayer()
		{
		  TileLayer tileLayer = new TileLayer((ITileSource) CreateTileSource(), 200, 300, 2, (IFetchStrategy) null, (IRenderGetStrategy) null, -1, -1);
		  return tileLayer;
		}

		private HttpTileSource CreateTileSource()
		{
		  return new HttpTileSource((ITileSchema) new GlobalSphericalMercator(), "https://some_provider.png", (IEnumerable<string>) new string[3]
		  {
			"a",
			"b",
			"c"
		  }, (string) null, "name", (IPersistentCache<byte[]>)new MemoryCache2<byte[]>(), (Func<Uri, byte[]>) null, OSMAttribution);
		}

It seems that GetTileInfos is the solution I needed. :)
var area = MapControl.Map.Viewport.Extent;
var theTails = tileLayer.TileSource.Schema.GetTileInfos(new BruTile.Extent(area.MinX, area.MinY, area.MaxX, area.MaxY), zoomLevel).Select(a => a.Index);
Without own MemoryCache<byte[]>. but PersistentCache in TileSource of TileLayer can't be null.

But that only provides the tiles currently visible, not all tiles, that are in the MemoryCache.

And this is exactly what I need, visible tiles in current view (Viewport.Extent).
Not all tiles from MemoryCache, only from specific area&zoom on demand.
Sorry, it seems my question was not clear enough.

BTW

Be careful, the most tile providers will not allow this.

We use own GeoServer and don't cache tiles from web providers (OSM etc.).
Sometimes a local map is required (on the same machine where is app, without lan, without internet access) and the map doesn't have to be of the whole world. So we need create an offline map of a part of the glob, some specific area and provide it to the app (e.g. mbtiles).


Run simplifier before generating ByteCode

Running the simplifier is necessary to do things like inline data
constructor wrappers.
Fixes #256 and #393

Added a test now which I verified fails before this patch but succeeds with the patch.

At long last with this branch I can almost load our main API server into ghcide! Only 14 files left to go (273 files now work), but that's a different bug :tada: Great work @mpickering!


Close support

Behavior on closed channels is likely to be inconsistent.  Need to fix this.

96ce95f is a partial fix.

7cc4390 should take care of this.  Check README for details.


Add background color to ChatRoom separator

@RocketChat/android

Closes #2109
Changes: [Add background color on ChatRoom separator when be selected group by type.]
Screenshots or GIF for the change:

 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.

 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.

@eliezer-of Here is a reference on how this screen should looks like.

@filipedelimabrito  the changes are correct?

@filipedelimabrito the changes are correct?

Could you provide a screenshot with your most recent changes @eliezer-of ? So I can see how it looks like.

Could you provide a screenshot with your most recent changes @eliezer-of ? So I can see how it looks like.

sure :sweat_smile:

Could you provide a screenshot with your most recent changes @eliezer-of ? So I can see how it looks like.

@filipedelimabrito sure! :sweat_smile:

Could you provide a screenshot with your most recent changes @eliezer-of ? So I can see how it looks like.

@filipedelimabrito sure! :sweat_smile:


Tendermint 0.34.12

Codecov Report

Merging #435 (0c35a4a) into main (66405b6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #435   +/-   ##
=======================================
  Coverage   18.50%   18.50%
=======================================
  Files         142      142
  Lines       22035    22035
=======================================
  Hits         4078     4078
  Misses      17273    17273
  Partials      684      684

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 66405b6...0c35a4a. Read the comment docs.

Wow, I'm surprised this doesn't cause issues with the SDK having a different tendermint version. But you added tests that show that this builds correctly, so should work!

Seems to also fix state sync.

Actually I'm really confused why this isn't using a replace directive. From what I can gather, the SDK is still going to be using the older tendermint version (https://stackoverflow.com/questions/57952397/how-to-resolve-conflicting-go-module-dependencies-when-a-top-level-module-and-on )
Can this be changed to add a replace line, I don't want to risk having complex bugs with the osmosis repo and SDK repo's TM versions being out of sync.

Sure, we can do that.

all set.


Analyzer GetAllRecognizers endpoint

This PR closes task 1844 and adds the following to analyze.proto:
Analyzer.GetAllRecognizers method - returns all (predefined and custom) recognizers

c struct: represents a predefined or custom recognizer in the GetAllRecognizers response. it holds the following properties:

name
entity
language

RecognizersAllRequest message: empty message sent to Analyzer.GetAllRecognizers
RecognizersAllResponse message: an array of GetAllRecognizers returned from Analyzer.GetAllRecognizers

looks good. there is an open discussion with omri374. we can approve after its resolved


WFLY-10651 Upgrade to Infinispan 9.3

PR for issue: https://issues.jboss.org/browse/WFLY-10651

Can one of the admins verify this patch?

I'm closing this based on the comments above.


feat:[PIPE-201741]: Add info regarding github connectors usage in V2 repo listing

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66d86ff1176d69d5164113a6--harness-developer.netlify.app


Make 'freeze' and 'seal' do nothing

Sanity check needed

Well, side effects that I can think of:

Object.isFrozen will always return false
When a property is (over)written directly (obj1.prop = {}), it can fail without throwing any error regardless of it being frozen or not. So, with all objects being unfrozen, there could be any internal code assigning a property to a (supposedly) frozen object and success.

Whether to keep it temporary is on you. I personally think monkey patching them is hacky but fine, as long as this is a client mod. Well yeah, it could cause issues in the future but I doubt that (hence the "sanity check needed")


Query Params and Route matching

Is there a way for us to specify in our router.config query parameters?
I would like to hit something like the following URL
#/information/42/wisdom?isUser='car'

If I map my router like the following my information controller get's triggered, but the $routeParams only has {id: 42, type: 'wisdom'}. No isUser property is in the object.
  $router.config([
    { path: '/information/:id/:type', component: 'information' }
  ]);

If I map my router like the following my information controller get's triggered, but the $routeParams has {id: "42", type?isUser: "wisdom"}. The type and isUser properties are not separated and still no query parameter
  $router.config([
    { path: '/information/:id/:type?isUser', component: 'information' }
  ]);

If I map my router like the following my information controller doesn't get triggered.
  $router.config([
    { path: '/information/:id/:type?isUser', component: 'information' }
  ]);

Doesn't seem supported yet. But I think it's essential that query params are either put into $routeParams or that $location.search() changes triggers view updates.
Right now I have to do this:
home.js
function HomeController($routeParams, $location, $scope){
  var self = this;
  this.params = $routeParams;

  // set query when coming to this route anew
  refreshQuery();
  function refreshQuery(){
    self.query = $location.search();
  }
  // set query when only the $location.search value changes, also deregister on destroy
  $scope.$on("$destroy", $scope.$on("$locationChangeSuccess",refreshQuery));
}

home.html

Home

{{ home.params | json }}

{{ home.query | json }}

In index.html
Go Home
Go Home with Query
Go New

When you go from '/home' to '/home?foo=5...' the controller is not reloaded, so that's why I need to listen for $locationChangeSuccess.
If we don't have built in support, everyone is going to have to add some custom code like this and that seems unnecessary.

Route recognizer is definitely solid in its support for query params. https://github.com/tildeio/route-recognizer/blob/master/tests/recognizer-tests.js
Hopefully that means adding support will be easier.

I had a look into this and actually it is possible and it isn't possible at the same time.
With the ng-link directive, you can specify your query string parameters.
Go Home
Go Home With Query

The link is parsed by Route Recognizer and they use the queryParams property to add the query parameters.
But if you do this, it will result in your html as home?foo=5&bar=10, but when you click on it, the question mark is URL encoded wich results in home%3Ffoo=5&bar=10. I tracked down the reason why it got url encoded and found that in dist/router.es5.js, the URL is changed by using $location.path(). You can only change the path with it and not the query params. This should be changed to $location.url() and then it works.
I'll add a pull request for this issue. Not sure if it will be accepted. Maybe there is a reason why they want it to be with $location.path() and they want to handle the query params somewhere else.

+1 - I would hate to see functionality lost for this and making it complex to do things like bookmark or tweak things from the URL during testing. Right now it causes fits because it sees a duplicate route if you use two routes with the same path and different query strings, and the message isn't very useful when debugging:
{ path: '/gallery/:id', component: 'gallery' },
{ path: '/gallery/:id/edit', component: 'editGallery' }

leads to
router.es5.js:953 Could not instantiate controller EditGalleryController
2angular.js:11768 TypeError: Cannot read property '0' of undefined
    at componentToCtrlDefault (router.es5.js:1126)
    at componentLoader (router.es5.js:1138)
    at Object.router.registerViewport.instantiate (router.es5.js:937)
    at router.es5.js:789
    at processQueue (angular.js:13988)
    at angular.js:14004
    at Scope.$get.Scope.$eval (angular.js:15211)
    at Scope.$get.Scope.$digest (angular.js:15026)
    at Scope.$get.Scope.$apply (angular.js:15316)
    at done (angular.js:9804)

I've been thinking about this as well, but I don't feel like I have an especially strong feeling of the best way to solve it. I'm totally open to suggestions though.

Tried to fix it in router.es5.js by using location.url() instead of location.path(). At first it seemed to work when adding the queryParams property in the ng-link directive.
Go Home With Query

But when I wrote tests for this, calling router.navigate('/home?foo=5&bar=10'), it didn't work. I get really strange behaviour.
This is what I wrote as a test
  it('should work with querystrings', inject(function ($router) {
    $router.config([
      { path: '/user/:name', component: 'userAge' }
    ]);
    compile('<ng-viewport></ng-viewport>');

    $router.navigate('/user/brian?age=10');
    $rootScope.$digest();
    expect(elt.text()).toBe('hello brian (10)');

    $router.navigate('/user/igor');
    $rootScope.$digest();
    expect(elt.text()).toBe('hello igor (unknown)');
  }));

This is the result of running that test
Chrome 41.0.2272 (Mac OS X 10.10.2) ngViewport should work with querystrings FAILED
	Expected 'hello brian (unknown)' to be 'hello brian (10)'.
	Error: Expected 'hello brian (unknown)' to be 'hello brian (10)'.
	    at Object.<anonymous> (/Users/sam/Documents/Projects/opensource/router/test/router-viewport.es5.spec.js:64:24)
	    at Object.invoke (/Users/sam/Documents/Projects/opensource/router/node_modules/angular/angular.js:4185:17)
	    at Object.workFn (/Users/sam/Documents/Projects/opensource/router/node_modules/angular-mocks/angular-mocks.js:2364:20)
	Expected 'hello igor (10)' to be 'hello igor (unknown)'.
	Error: Expected 'hello igor (10)' to be 'hello igor (unknown)'.
	    at Object.<anonymous> (/Users/sam/Documents/Projects/opensource/router/test/router-viewport.es5.spec.js:68:24)
	    at Object.invoke (/Users/sam/Documents/Projects/opensource/router/node_modules/angular/angular.js:4185:17)
	    at Object.workFn (/Users/sam/Documents/Projects/opensource/router/node_modules/angular-mocks/angular-mocks.js:2364:20)
Chrome 41.0.2272 (Mac OS X 10.10.2): Executed 21 of 21 (1 FAILED) (0.215 secs / 0.207 secs)

So although I pass the querystring to the controller when I use Brian, the controller doesn't receive it. But when navigating away to igor (with no queryparams), then the params are being received by the controller.
Didn't have time to look more into this behaviour.

I think the ui-router had a simple enough way of dealing with query parameters https://github.com/angular-ui/ui-router/wiki/URL-Routing. It allows us to specify query parameters directly in the URL path.
$router.config([
    { path: '/information/:id/:type?isUser', component: 'information' }
  ]);

To interact with the parameters then I could utilize $routeParams to grab the parameter as:
$routeParams.isUser

If we want to pass query parameters I like the simplicity of ui-router also. It doesn't make me specify queryParams like @SamVerschueren pointed out. Instead, it is smart enough to know if they are query parameters or URL parameters:

    Information

& programmatically:
$router.navigate('/information/42/wisdom?isUser=false');

The new router uses Route Recognizer for this so I guess we should follow their rules... It is that library that forces you to add the queryParams property.

+1
http://geekswithblogs.net/shaunxu/archive/2015/03/30/horror-migrating-angular-ui-router-to-angular-new-router.aspx

I don't like that you have to specify query params:
Go Home With Query

@btford Why can't we just pass the object?

@niemyjski good on you for writing up such a detailed migration experience!! :clap: :+1: That will be incredibly valuable to @btford and the rest of the new router team!
I also agree that I'd rather just pass query params along with the other params and not on a special queryParams object, if possible.

@niemyjski My fix works for the ng-link directive. But when I tried to create tests for that functionality by calling $router.navigate() directly with query parameters, I really got weird results. So this needs further investigation. I will investigate if I find more time.
Passing query params directly instead of via the queryParams property should be changed in Route Recognizer as that one is used to parse the routes. Don't think this change can be made on the routers side.


crashes when first indexing

hello, I try to Index a huge forum, but at some point it fails and can't go on indexing.
it seems to start from the beginning when I launch it again...
PHP Fatal error:  Uncaught TypeError: Blomstra\Search\Seeders\Seeder::groupsForDiscussion(): Argument #1 ($discussion) must be of type Flarum\Discussion\Discussion, null given, called in /opt/flarum/vendor/blomstra/search/src/Seeders/CommentSeeder.php on line 69 and defined in /opt/flarum/vendor/blomstra/search/src/Seeders/Seeder.php:27
Stack trace:
#0 /opt/flarum/vendor/blomstra/search/src/Seeders/CommentSeeder.php(69): Blomstra\Search\Seeders\Seeder->groupsForDiscussion()
#1 /opt/flarum/vendor/blomstra/search/src/Jobs/SavingJob.php(18): Blomstra\Search\Seeders\CommentSeeder->toDocument()
#2 [internal function]: Blomstra\Search\Jobs\SavingJob->Blomstra\Search\Jobs\{closure}()
#3 /opt/flarum/vendor/illuminate/collections/Collection.php(695): array_map()
#4 /opt/flarum/vendor/illuminate/database/Eloquent/Collection.php(350): Illuminate\Support\Collection->map()
#5 /opt/flarum/vendor/blomstra/search/src/Jobs/SavingJob.php(24): Illuminate\Database\Eloquent\Collection->map()
#6 /opt/flarum/vendor/illuminate/container/BoundMethod.php(36): Blomstra\Search\Jobs\SavingJob->handle()
#7 /opt/flarum/vendor/illuminate/container/Util.php(40): Illuminate\Container\BoundMethod::Illuminate\Container\{closure}()
#8 /opt/flarum/vendor/illuminate/container/BoundMethod.php(93): Illuminate\Container\Util::unwrapIfClosure()
#9 /opt/flarum/vendor/illuminate/container/BoundMethod.php(37): Illuminate\Container\BoundMethod::callBoundMethod()
#10 /opt/flarum/vendor/illuminate/container/Container.php(653): Illuminate\Container\BoundMethod::call()
#11 /opt/flarum/vendor/illuminate/bus/Dispatcher.php(128): Illuminate\Container\Container->call()
#12 /opt/flarum/vendor/illuminate/pipeline/Pipeline.php(128): Illuminate\Bus\Dispatcher->Illuminate\Bus\{closure}()
#13 /opt/flarum/vendor/illuminate/pipeline/Pipeline.php(103): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline\{closure}()
#14 /opt/flarum/vendor/illuminate/bus/Dispatcher.php(132): Illuminate\Pipeline\Pipeline->then()
#15 /opt/flarum/vendor/illuminate/queue/CallQueuedHandler.php(120): Illuminate\Bus\Dispatcher->dispatchNow()
#16 /opt/flarum/vendor/illuminate/pipeline/Pipeline.php(128): Illuminate\Queue\CallQueuedHandler->Illuminate\Queue\{closure}()
#17 /opt/flarum/vendor/illuminate/pipeline/Pipeline.php(103): Illuminate\Pipeline\Pipeline->Illuminate\Pipeline\{closure}()
#18 /opt/flarum/vendor/illuminate/queue/CallQueuedHandler.php(122): Illuminate\Pipeline\Pipeline->then()
#19 /opt/flarum/vendor/illuminate/queue/CallQueuedHandler.php(70): Illuminate\Queue\CallQueuedHandler->dispatchThroughMiddleware()
#20 /opt/flarum/vendor/illuminate/queue/Jobs/Job.php(98): Illuminate\Queue\CallQueuedHandler->call()
#21 /opt/flarum/vendor/illuminate/queue/SyncQueue.php(43): Illuminate\Queue\Jobs\Job->fire()
#22 /opt/flarum/vendor/illuminate/queue/Queue.php(57): Illuminate\Queue\SyncQueue->push()
#23 /opt/flarum/vendor/blomstra/search/src/Commands/BuildCommand.php(131): Illuminate\Queue\Queue->pushOn()
#24 /opt/flarum/vendor/illuminate/container/BoundMethod.php(36): Blomstra\Search\Commands\BuildCommand->handle()
#25 /opt/flarum/vendor/illuminate/container/Util.php(40): Illuminate\Container\BoundMethod::Illuminate\Container\{closure}()
#26 /opt/flarum/vendor/illuminate/container/BoundMethod.php(93): Illuminate\Container\Util::unwrapIfClosure()
#27 /opt/flarum/vendor/illuminate/container/BoundMethod.php(37): Illuminate\Container\BoundMethod::callBoundMethod()
#28 /opt/flarum/vendor/illuminate/container/Container.php(653): Illuminate\Container\BoundMethod::call()
#29 /opt/flarum/vendor/illuminate/console/Command.php(136): Illuminate\Container\Container->call()
#30 /opt/flarum/vendor/symfony/console/Command/Command.php(298): Illuminate\Console\Command->execute()
#31 /opt/flarum/vendor/illuminate/console/Command.php(121): Symfony\Component\Console\Command\Command->run()
#32 /opt/flarum/vendor/symfony/console/Application.php(1042): Illuminate\Console\Command->run()
#33 /opt/flarum/vendor/symfony/console/Application.php(299): Symfony\Component\Console\Application->doRunCommand()
#34 /opt/flarum/vendor/symfony/console/Application.php(171): Symfony\Component\Console\Application->doRun()
#35 /opt/flarum/vendor/flarum/core/src/Console/Server.php(42): Symfony\Component\Console\Application->run()
#36 /opt/flarum/flarum(24): Flarum\Console\Server->listen()
#37 {main}
  thrown in /opt/flarum/vendor/blomstra/search/src/Seeders/Seeder.php on line 27

I must have missed this in my notifications.. Did you ever fix this?


Rework conversion pipeline

The whole conversion pipeline is in a single file src/generate_bindings.nim plus some templates in src/templates/.

@planetis-m would you like to review this or do you have any comments?

If you want please copy the wrappers and destructors from my PR. Else I could do it later.

If you want please copy the wrappers and destructors from my PR. Else I could do it later.

I will do it in a separate PR shortly. There could be changes to examples and readme additions


Scroll broken in firefox

Description
Scrolling don't work on pages except the home screen. For e.g. https://sukoon-stress-free.netlify.app/html/yogatherapy.html. This issue is related to firefox only.
Screenshots
No response
Additional information
No response

Hello @shivanshu-semwal Kindly attach an ss related to the problem

There is no issue in my case.


$ pnpm build  ><EMAIL_ADDRESS>build C:\Users\xxx\ChatGPT-Admin-Web > turbo run build  (node:19552) NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.  Please migrate your code to use AWS SDK for JavaScript (v3). For more information, check the migration guide at https://a.co/7PzMCcy (Use node --trace-warnings ... to show where the warning was created)

本地build会报这个错，怎么处理呢？
`$ pnpm build

<EMAIL_ADDRESS>build C:\Users\xxx\ChatGPT-Admin-Web
turbo run build

(node:19552) NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.
Please migrate your code to use AWS SDK for JavaScript (v3).
For more information, check the migration guide at https://a.co/7PzMCcy
(Use node --trace-warnings ... to show where the warning was created)`

很抱歉，v2版本将不再维护，v3版本可关注 https://github.com/AprilNEA/ChatGPT-Admin-Web/releases/tag/v3.0.0-alpha


[QUERY] Azure Schema registry + Spark Structured Streaming + Kafka/EventHub compatibility

Is there a way to use AZURE Schema Registry with Spark Structured Streaming? (not the opensource schema registry).
Ideally I would use some high level library like Abris (https://github.com/AbsaOSS/ABRiS), to read and write avro from/to Kafka with automatically registering/ reading the df schema in the schema registry. However, I have not managed to find anything at all that could enable spark to do to_avro or from_avro against an Azure Schema Registry.
There is an open PR that could add the functionality for some column types https://github.com/Azure/azure-event-hubs-spark/pull/615
There seems to be an unmainained repo that at least tried to do the from_avro part (https://github.com/arerlend/azure-schemaregistry-spark)
Anybody knows about a good way how to interact with Azure Schema Registry with Spark Structured Streaming?

@conniey could you please follow up with @scaladevspark?

Unfortunately, I have no insights into Spark Structured Streaming.  Pinging @hmlam who may have some insights.


ZipperList for parent nodes

It seems reasonable to store parent nodes as a ZipperList since we always have some parent node in focus (current context). That could allow some memory model simplification and more efficiency.

See src/ZipperList.hs


Bluetooth: HCI: Add discard mechanism to avoid waiting timeout

The Bluetooth receiving thread may not be able to process broadcast
packets because the system API(bt_hci_cmd_send_sync) is in block state.
If HCI driver is still waiting buffer for adv report, an assertion will
be triggered.
Fixes: #30955
Signed-off-by: LingaoM<EMAIL_ADDRESS>

Could you rename the title/commit summary, since this isn't about HCI in general but just about the userchan HCI driver? I.e. something like "Bluetooth: userchan: ..."


Add support for pushing to container registries different than Dockerhub

Signed-off-by: Andre Riesco<EMAIL_ADDRESS>

For tcb it was still wrong. I fixed it and rebased the code, now it is working also for the tcb-platform-publish and tcb-platform-publish-ostree commands

LGTM


[Debugger Expression Evaluation] Disallow negation with numeric literals

Description:
Debugger expression evaluation engine should be updated to disallow negation with numeric literals, which was introduced with Swan Lake Alpha 5 release.
Describe your task(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Closing since this has been already handled by the debugger runtime helper methods.


Text mode: announce builder starting port

Hello!
Firstly, the 1.52 release of Synth has me pretty happy on all of my machines, thanks! =)
The only nit that I have remaining is that while building in text mode, I generally have to run synth (upgrade-system or install) and then tail -f /var/log/synth/00_last_results.log for the sole purpose of keeping up with what the builders are currently working on, with the occasional switching to watching specific build logs if something seems to be taking a bit too long.
Would you consider having the text mode announce the ports it begins to build? I've been running recently with this change: https://github.com/jrmarino/synth/compare/master...kevans91:portname_announce -- but I don't like the formatting that I've used. This takes a standard run from something along the lines of:
00:01:08 => [01] Builder launched
00:01:08 => [02] Builder launched
00:04:11 =>      Left: 4  Succ: 0  Fail: 0  Skip: 0   Ign: 0
00:04:38 => [01] 00:03:24 Success graphics/opencv-core
00:05:36 => [02] 00:04:24 Success editors/vim-lite
00:05:36 => [02]          Shutting down
00:07:15 =>      Left: 2  Succ: 2  Fail: 0  Skip: 0   Ign: 0
00:10:18 =>      Left: 2  Succ: 2  Fail: 0  Skip: 0   Ign: 0
00:10:19 => [01] 00:05:38 Success multimedia/ffmpeg
00:11:32 => [01] 00:01:10 Success net/freerdp
00:11:32 => [01]          Shutting down

into
00:01:08 => [01] Builder launched
00:01:08 => [02] Builder launched
00:01:08 => [01] graphics/opencv-core
00:01:08 => [02] editors/vim-lite
00:04:11 =>      Left: 4  Succ: 0  Fail: 0  Skip: 0   Ign: 0
00:04:38 => [01] 00:03:24 Success graphics/opencv-core
00:04:38 => [01] multimedia/ffmpeg
00:05:36 => [02] 00:04:24 Success editors/vim-lite
00:05:36 => [02]          Shutting down
00:07:15 =>      Left: 2  Succ: 2  Fail: 0  Skip: 0   Ign: 0
00:10:18 =>      Left: 2  Succ: 2  Fail: 0  Skip: 0   Ign: 0
00:10:19 => [01] 00:05:38 Success multimedia/ffmpeg
00:10:20 => [01] net/freerdp
00:11:32 => [01] 00:01:10 Success net/freerdp
00:11:32 => [01]          Shutting down

I did consider using the spacing-padded line format that the 'Shutting down' messages use, but that would seem to cause me more parsing when quickly scanning lines to get an idea of the status.

with the occasional switching to watching specific build logs if something seems to be taking a bit too long.

That shouldn't be required; it's what the watchdog is doing
The text mode is basically a developer test mode that I left in place because it was sort of useful.  I'm not really looking to beef it up because it's basic use case is set and forget.  What you're really asking for is the "web interface" (on the to-do list) or a way to query synth about the current state (something I added to DragonFly poudriere that never made it to FreeBSD version)

I'm considering this, but I would pad the starting lines to maintain pseudo columns.

enhancement published.


Add  Rate Us Page

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Hello, my name is A.A.D.Sailaja. I am a Tech Enthusiastic and i'm looking forward to contribute in open source projects. Here in this project i would like to add a rate us option it will allow the user to rate our website and also he can suggest us to improve if any
Can you please assign me this issue under gssoc'24
Use Case
It will Increase User Experience and also Website Responsiveness
Benefits
No response
Add ScreenShots

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I'm a SSOC'24 contributor
[X] I want to work on this issue

@0xabdulkhalid Can you Assign me this issue I will Make the Page in an Attractive Way

Sorry @sailaja-adapa, We can't assign more than 1 issue at a time.
You can work on #386 for now, Once you finished it then we can work on this. Until that this issue will be on hold.

Yeah it's okkk @0xabdulkhalid

@sailaja-adapa Provide a prototype of how the changes will look.

Yeah sure @0xabdulkhalid

I will update you by this evening @0xabdulkhalid

I would like to make like this according to our website @singodiyashubham87

@sailaja-adapa It seems good, but how will the user get to this pop-up?
Are you thinking of adding a button Rate Us to display this Modal/Pop-Up?

@singodiyashubham87   i would like add rate us on footer so when user click on that they will head to new page as i kept an screenshot before like that page will be appear in an attractive way according to our website still i didn't improved that much only i added rate us option on footer

@sailaja-adapa SSOC or GSSOC?

@singodiyashubham87 GSSOC'24


[Event Request] Codeunit 6500 Item Tracking Management - OnGetWhseExpirationDate

Hi Team,
In codeunit 6500 we need possibility to replace and avoid using standard code. Having event with handling in procedure GetWhseExpirationDate, would be a perfect solution :

[IntegrationEvent(false, false)]
local procedure OnGetWhseExpirationDate(var ExpDate: Date; ItemNo: Code[20]; LotNo: Code[50]; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


defaultFile option doesn't work with **

What should I put to work with any file in my project's path?
"todomd.defaultFile": "**/{todo,someday,todo.archive}.md"

2023-02-09 12:42:45.868 [error] Activating extension usernamehw.todo-md failed due to an error:
2023-02-09 12:42:45.868 [error] Error: cannot open file:///%2A%2A/%7Btodo%2Csomeday%2Ctodo.archive%7D.md. Detail: Unable to read file '/**/{todo,someday,todo.archive}.md' (Error: Unable to resolve nonexistent file '/**/{todo,someday,todo.archive}.md')
    at a.$tryOpenDocument (vscode-file://vscode-app/Applications/Visual%20Studio%20Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:2779:76347)

macos
VScode Version: 1.75.0

todomd.defaultFile is an absolute path to a file that is always available in running commands or populating tree views & webivew.
It's a SINGLE file, not a glob.

I don't get it. But thanks. I just removed the todomd.defaultFile setting.
Unfortunately, you need a todomd.defaultArchiveFilesetting.
The issue with an absolute path is that if you use VScode's built-in sync mechanism. This will likely break for the other machine


replace append by concat for flag generation

Hello,
This is my first participation in an open source project, so please tell me if I'm doing anything wrong :)
I tracked this bug from this (see my answer):
https://stackoverflow.com/questions/59058854/intero-error-wrong-type-argument-stringp-nil
NB I didn't quite solve the original problem (intero-ghc-version not being set immediately), but this one was easier to pinpoint.
The bug was introduced by 0a87ecaf98950eae7516613ffc84f901bbf48f54
I hope this helps!
Best,
Mikael

Hi again,
I think I found out the reason why intero-ghc-version was nil,
see my second commit, and also the consolidated answer I posted in the SO thread
(split-string intero-ghc-version "\\.")

should really be
(split-string (intero-ghc-version) "\\.")

so that the local variable is set properly, right ?
With these two fixes I am back with a running intero :)

Seems that #643 contains all the commits from this Pull Request, so this can be closed. Also - @chrisdone would have to do something with the failing CI on master first, but it was 2 months since the last build :/


List devfile component without connecting cluster if experimental flag is enabled

Signed-off-by: jingfu wang<EMAIL_ADDRESS>What type of PR is this?
/kind bug
What does does this PR do / why we need it:
If the experimental flag is enabled, user should be able to list devfile components even if user doesn't connect to OpenShift cluster. Instead of failing the odo catalog list components, we output the following warning to user:
odo catalog list components                                                                                 ✔  17:00:33 
 ⚠  Please log in to an OpenShift cluster to list OpenShift/s2i components

Odo Supported Devfile Components:
NAME                 DESCRIPTION
java-spring-boot     Spring Boot® using IBM Java
openLiberty          Open Liberty microservice in Java

Which issue(s) this PR fixes:
Fixes #2740
Fixes #2706
How to test changes / Special notes to the reviewer:

/assign

@kadel Please review this PR, thanks!

@cdrage @johnmcollier Comments addressed, please view again, thanks!

/retest

/retest

/retest

comments addressed, and this is a small change
/approve
/lgtm


chore: import repository https://github.com/q1323829945/test6.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

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

Needs approval from an approver in each of these files:

OWNERS

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

@q1323829945: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
7dd880c4a3e1de2ece1b0aad0c48169ba009b238
link
/test verify

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


[BEAM-11731][BEAM-10582] Allow pyarrow<4,numpy<1.21.0, improve pyarrow verification

A few simultaneous changes here:

Allow pyarrow up to 3.x, numpy up to 1.21.x
Modifies CI to use numpy <1.20.0 when testing pyarrow <3 to workaround ARROW-11450
Adds a new uses_pyarrow mark, so that we can test both parquetio_test and dataframe.io_test against multiple pyarrow versions

Note this does not protect our users from using numpy 1.20.0 and pyarrow <3. Unfortunately it doesn't look like there's a way to achieve that in setup.py.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python PreCommit

Thank you!


added airbridge_generator logic

What are the issues this pull addresses (issue numbers / links)?
Did you add tests to cover your changes (yes/no)?
Did you update the documentation accordingly (yes/no)?
Did you read the CONTRIBUTING document (yes/no)?
Summary
Adds a tool to generate auto generate airbridges!
Details and comments
Example usage:
Airbridge_Generator(design, [cpw], crossover_length=0.010 + (2 * 0.006) , BRS1=0.005)
where cpw is an TLine in QComponents

HI @clarkmiyamoto! Thank you for your contribution! It seems that you need to format your code with yapf in order to pass the lint job on the pipeline.


Styles: Ensure (automatic) variable fallbacks can render

Originally discussed in the stylis-plugin-css-variables repo:
https://github.com/ItsJonQ/stylis-plugin-css-variables/issues/2

I explored this a bit this morning. I think I may have an idea of how to solve this 🙏 .
It involves creating a Map that lives in memory where the custom variables (declared at :root) are stored.
That way, variables can be grabbed from this Map (which should work in IE) vs. window.getComputedStyle(...).getPropertyValue('--css-variable'), which does not.

Ohhh boy! Exciting update for this.
I have a working solution ✨
Tested on IE11 via VirtualBox:

(Note the fallback values on the right)


[SPARK-42125][CONNECT][PYTHON] Pandas UDF in Spark Connect

What changes were proposed in this pull request?
Support Pandas UDF in Spark Connect.
Since Pandas UDF and scalar inline Python UDF share the same proto message, ScalarInlineUserDefinedFunction is renamed to CommonUserDefinedFunction.
Why are the changes needed?
To reach parity with the vanilla PySpark.
Does this PR introduce any user-facing change?
Yes. Pandas UDF is supported in Spark Connect, as shown below.
>>> from pyspark.sql.functions import pandas_udf
>>> import pandas as pd
>>> @pandas_udf("double")
... def mean_udf(v: pd.Series) -> float:
...     return v.mean()
...
>>> df = spark.createDataFrame([(1, 1.0), (1, 2.0), (2, 3.0), (2, 5.0), (2, 10.0)], ("id", "v"))
>>> type(df)
<class 'pyspark.sql.connect.dataframe.DataFrame'>
>>> df.groupby("id").agg(mean_udf("v")).show()
+---+-----------+
| id|mean_udf(v)|
+---+-----------+
|  1|        1.5|
|  2|        6.0|
+---+-----------+
>>>

How was this patch tested?
Existing tests.

I would prefer to file a follow-up PR to reuse pyspark.sql.tests.test_pandas_udf test cases.

@HyukjinKwon @zhengruifeng @grundprinzip May I get a review, please?

Merged to master and branch-3.4. Thanks all!

Merged it to unblock follow-up PRs. Feel free to leave comments, I will address them in the follow-ups.


Feature reconfigurable base

Allowing encoding base to be reconfigurable. This should help interoperability between javallier and python-paillier. Default encoding base is 16 (the same as python-paillier).

Not related to this pull request, but I've noticed PrivateSum project in examples/ folder is not compatible with java 1.7 and just works with version 1.8.


chore(swamp): replace cosmos prefix with celestia in swamp tests

Overview
Use "celestia" prefix for accounts instead of "cosmos" in swamp tests
Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Does it fixes tests on mac? Also the question why didn't it work before

Codecov Report

Merging #1583 (86a56fa) into main (311d9dc) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1583   +/-   ##
=======================================
  Coverage   54.91%   54.91%
=======================================
  Files         230      230
  Lines       14983    14983
=======================================
  Hits         8228     8228
  Misses       5874     5874
  Partials      881      881

Impacted Files
Coverage Δ

cmd/celestia/main.go
47.05% <ø> (-15.45%)
:arrow_down:

core/fetcher.go
50.42% <100.00%> (+3.09%)
:arrow_up:

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


Interface to run raster-to-vector attribution

Input:

vector geometries (LineStrings), pre-split to segments within the raster grid
raster

Output:

as "one-to-one": raster value for a vector feature
as "vectorised" input Vector<Geometry> output Vector<T: RasterValueType> of the same length

Possible optimisation: a Geometry or Feature could keep track of its row, col index in the raster grid, to avoid recomputing transforms when comparing geometries with multiple rasters.
Useful to expose at:

CLI level, input/output files
Python wrapper level, if/when we go for that
C++ library level, public function/method

In terms of GeoDataFrames:

start with splits which has index and geometry
compute a new column for each raster that we intersect with

Closed by #18


Add support for Cortex-A/R AArch32 cache management APIs

Add support for Cortex-A/R AArch32 (L1) cache management APIs through CMSIS. Cortex-M cache implementation is moved to its own directory as it's not generic for all Arm architectures.
Also enabled caches on fvp_baser_aemv8r_aarch32 to have a Cortex-R AArch32 FVP board with caches enabled.

@manuargue Seeing some CI failures, though only one related to Arm.

@manuargue Seeing some CI failures, though only one related to Arm.

yes, logged this in https://github.com/zephyrproject-rtos/zephyr/issues/56188 and https://github.com/zephyrproject-rtos/zephyr/issues/56189, but both are failing in main. The first is being addressed already, for the second I may have time today to look at it (if not fixed already).

rebased to pickup unrelated fixes, hoping for successful builds 🤞