Remove generated comment blocks

Hi, thoughts on removing the generated comments in inject? Or can this be configurable?
Is there a standard within Avaje on if a generated java file should have comments? I noticed that http $Route and JsonComponent don't but $DI do.
The comments in DI take up 10-30% of most files (DI files are small) and it takes up processing time when there's a lot of small files, e.g. the case when you follow the pattern of controller -> service -> repository -> model / dto mapper and more.
Thanks!

@rbygrave what say you

Well, the method comments in the DI classes don't really add much value, so we could remove those.

it takes up processing time

I'm surprised that we think this takes a material amount of time.

Is there a standard within Avaje on if a generated java file should have comments?

Not a standard per say but the comments are there to assist people in reading the generated code and looking to gain understanding of what is going on. For example, I think the comments in the AvajeModule are good value in that regard and I'm not keen to remove any of those.
We could remove the method comments on $DI though, those comments don't add much value.


[Telegram.WebApp] Changing swipes behavior is not supported in version 7.6

Yes, because this feature was added in Bot API 7.7. There's nothing we can do.

yeah, but you updated your package, why it still showing a warn?

Telegram client also needs to support Bot API 7.7 for this to work.


There is no way to construct a sigevent

There appears to be no way to construct a sigevent outside of the libc crate.  As far as I can tell, you cannot use a struct literal format--because of inaccessible fields-- and there is no constructor, and it does not derive Default.
To illustrate the problem, I have this simple contrived example:
fn main() {
    let s = libc::sigevent {
        sigev_value: libc::sigval {
            sival_ptr: std::ptr::null_mut(),
        },
        sigev_signo: 0,
        sigev_notify: 0,
        sigev_notify_thread_id: 0,
    };
    std::process::exit(s.sigev_signo);
}

Which fails with
error: cannot construct `sigevent` with struct literal syntax due to inaccessible fields
 --> src/main.rs:2:13
  |
2 |     let s = libc::sigevent {
  |             ^^^^^^^^^^^^^^

You can use mem::zeroed for unused fields. See https://github.com/rust-lang/libc/pull/2346#issuecomment-899226540 for a similar discussion.


[T3A4][W09-A3]Xiao Pu

Ready for review

Ready for review

Your code interrupts the program if the file is deleted. While your way of handling is definitely accepted, you may also consider showing an error message and re-create the file for the user

@xpdavid Some comments added. Please close the PR after reading comments


<EMAIL_ADDRESS>hash check failed

Installing '7zip' (18.05) [64bit]
7z1805-x64.msi (1.7 MB) [=====================================================================================] 100%
Checking hash of 7z1805-x64.msi ... ok.
Extracting 7z1805-x64.msi ... done.
Linking ~\scoop\apps\7zip\current => ~\scoop\apps\7zip\18.05
Creating shim for '7z'.
Creating shortcut for 7-Zip (7zFM.exe)
'7zip' (18.05) was installed successfully!
Installing 'curl' (7.61.0) [64bit]
Downloading https://dl.bintray.com/vszakats/generic/curl-7.61.0-win64-mingw.tar.xz (0 B)...

Checking hash of curl-7.61.0-win64-mingw.tar.xz ... Exception calling "Join" with "2" argument(s): "Value cannot be null.
Parameter name: values"
At C:\Users\laura\scoop\apps\scoop\current\lib\install.ps1:656 char:13
+             $hexbytes = [string]::join(' ', $hexbytes).ToUpper()
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : ArgumentNullException

ERROR Hash check failed!
App:         curl
URL:         https://dl.bintray.com/vszakats/generic/curl-7.61.0-win64-mingw.tar.xz
First bytes:
Expected:    623a06a6815ce17743d457c2168d9ea91900459674e68196b484e8e9079660c4
Actual:      e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855

Please run scoop update and try again. Curls checksums change quite frequently.


Migrating cerulean theme from v2 to v3 - variables.scss

What's the difference of #2351 and this?

Sorry I didnt make a single PR , I forgot to change the two files variables.scss and theming.scss
In this , I added variables.scss file and in other I added theming.scss

Please make them as a single PR, because merging them individually never makes sense.

it's done.

I cannot reproduce this error. It looks like a syntax error.
can your upgrade your saas gem version
my gems :
sass (3.4.15, 3.4.14, 3.4.13)  sass-rails (5.0.3, 5.0.1)
Cerulean Theme look without border-radius  :

Of course I can update sass, but the users may not be able to.
(RailsAdmin officially supports Rails 4.0, which has sass-rails(~> 4.0.0) and sass(~> 3.2.0) dependency by default.)
Can you make this compatible with sass 3.2.x?

I'll try this weekend.

it's better with the old syntax 😉 and some colors adapted

with reset filters button
before click on remove filters button

init state or after click on remove filters button

Merged in, thanks! :smiley:


Example for optimistikit + superforms

Describe the problem
The docs already mention: "enhance: some libraries like superforms provide a custom enhance function that is different from the one provided by SvelteKit. To allow you to use this libraries together with optimistikit you can pass a custom enhance function. It's important for this function to have the same signature as the sveltekit one."
I honestly didnt manage to make this work. I also asked on the svelte discord. generally i dont see much content about how the enhance of sveltekit works and how to combine them without having 2 requests send or w/e.
So could we maybe get an exmaple on how to do this? Dosent need to be detailed imo.
Describe the proposed solution
Very simple exmaple with superforms doing a simple create action combining both enhance functions

Yeah I should probably fix the README...super forms does things slightly different and it doesn't really works as of today. I'll check with Andreas to make it work tho


Support react 17

Added support for react 17 as peer dependency.

Hey just wondering what this PR is waiting for? Thanks

Hey just wondering what this PR is waiting for? Thanks

It looks like this repo is not active currently. As a workaround you can install this patched version.

Hey @AXeL-dev, i tried installing the patched version and it gives me this error<EMAIL_ADDRESS>is not a module

.

Hey @AXeL-dev, i tried installing the patched version and it gives me this error<EMAIL_ADDRESS>is not a module  .

Did you keep both modules? 'cause i see in the bottom an import line with react-nice-dates/build/style.css. You should only keep one module & replace the css import with import<EMAIL_ADDRESS>(if it still doesn't work, try to reinstall the module from npm).

I have the same problem than @ZilRahman :(

I have the same problem than @ZilRahman :(
I have this issue<EMAIL_ADDRESS>is not a module when I import like this import { DateRangePickerCalendar, DateRangePickerCalendarProps, DateChangeCallBack, } from '@axel-dev/react-nice-dates';
I uninstall react-nice-dates

You're getting this error because the library is written in javascript & you're using it on a typescript project, the best solution would be to install a types declarations module (something like @types/@axel-dev/react-nice-dates), but i don't really have enough time to create that module & publish it, so for now here is a quick solution:

Create a file in the root of your project & name it declarations.d.ts.
Open that file & paste the following line into it (don't forget to save):

declare module '@axel-dev/react-nice-dates';

Go to your tsconfig.json file & add the file that we just created to the include entry:

{
   ...,
   "include": [
     ...,
+    "declarations.d.ts"
   ]
}

Your error should be fixed after that.

@danieltoro, the types declarations issue was fixed in version 3.1.2. Simply update the library using:
npm i @axel-dev/react-nice-dates@latest


CAMPD Design Waiver Meeting Materials

🧭 Objective
For the exemption template, we’re looking to present both the data type picker screen and the filter screen. We only need a snapshot of both screens. The only things we’re looking to add to the existing wireframes is the EPA logo in the top left corner, color, and breadcrumbs on the filter screen. No need to have the wireframe “working”. We’re mostly concerned on making the argument that we need to use a large amount of screen width, and a banner/header that won’t disrupt the breadcrumbs. We are still deciding on whether we want a compressed footer (similar to the ECMPS template), so please have snapshots with and without the footer on both screens (so four snapshots total). The compressed footer does not need to work.
📋 Tasks

[x] Add EPA logo to existing wireframes
[x] Add compressed footer placeholder to a new set of screens
[x] Export four screens: datatype intro, datatype intro w/ footer, filters, filters w/ footer

🤝 Stakeholders
@kstetsyuk @tlageman-epa @MoAdeyoju @JorjaComer

@kstetsyuk @sgillEPA @tlageman-epa Here are four screens to use as talking points during the waiver discussion next week. Some things to note:

I didn't include color on purpose. I didn't want to distract from the purpose of this waiver request with anything outside of grayscale wireframes.
The header and footer (where applicable) have a fixed width, while the rest of the content on the Filters screens spans from edge to edge. This will hopefully make the case for the wide layout.
The waiver documentation (below) is very clear about where there is wiggle room, and where EPA sites and web applications need to align with the EPA style guide and with USWDS. I'm sure you already have, but please take a look at the documentation.

Applications and the One EPA Web Standalone Template
Procedure: Complying with EPA.gov "Look andFeel"

@cvp-alexrobinson Thanks for getting these to us! After talking with some folks, we'd like to have these snapshots in color (header, footer, and content). It would be helpful for when we are presenting these so we can show what our color scheme is, and what color buttons we plan on using, etc. The blue used on the regular epa.gov pages should work.
Let me know if you have any questions! Thanks!

@kstetsyuk — Based on the Look and Feel documentation, really all you need—if anything at all—are grayscale wireframes. With that being said, I'm here to support you and the team with whatever you need so I've attached some colorized versions of the wireframes.

Landing Page and Data Preview Screens
Guidance from Karina, 3/17/2021:
After some conversation, we need to refocus on materials for the Look & Feel waiver. It has come to our attention that a landing page would be huge aid in our justification, so we need something for that done as soon as possible. We are not looking for anything too complex – something simple that won’t take up much time. We just need to capture the basic idea of what that page will do/look like. We need these screenshots with the same header and footer as used in the “Custom Data Download” screenshots.
We also need a screenshot of the data preview for the “Custom Data Download” page. This will aid our argument that we need to use a wide area. Same header and footer.
We’d like to have these by COB tomorrow and can use ticket #892. If there are any questions, please let me know.

@kstetsyuk — I still need to flesh out the preview data table, but am I getting close?

Hey Alex,
Please use the same header/footer (colors) as in https://xd.adobe.com/view/e1a73023-0085-4e20-aa61-73e0e85cb064-b4f4/
Know it might be a bit awkward with colors, but maybe you have idea for an easy fix.
I'll pass the landing page to the team now to take a look.
Thanks!

Sorry - scratch the previous message for color change. We'll keep this header and footer (the ones in the images you just posted) and see what Kevin has to say. Hoping it won't be too much hassle to change if need be?
Getting good feedback about the landing page, so at this point we just need the data preview.
Great work! Thanks again!

@kstetsyuk — It's not difficult changing the header and footer, I just swap them out. But it sounds like we can stay with the light version for now. I'll keep working on the preview screen.

@kstetsyuk — Here's the data preview screen. Let me know if this works.

@cvp-alexrobinson
Here are some edits for the landing page.
•	Title: CAMPD: Clean Air Markets Program Data
•	Remove top nav
•	Data: Create custom queries, download bulk datasets and use the CAMD API to retrieve emissions, allowance and compliance data.
•	Analysis with “Coming Soon” disclaimer: View and download common analyses, reports, insights and other information using CAMPD data.
•	Viz with “Coming Soon” disclaimer: Visualize and interact with CAMPD data graphically and through maps.
Need “Coming Soon” because analysis and viz won’t be rolling that out in the first release.
Thanks!

@kstetsyuk — The site navigation should always be present, no matter which page you're on. I understand that we've changed the architecture a bit though. I've updated the main navigation to be Home, Data, Analysis, and Visualization. Data, Analysis and Visualization would be drop downs that would house the other pages in that section. If for the demo tomorrow, you do want to remove the CAMPD navigation, please let me know.
Let me know if you have questions or concerns.

@kstetsyuk — Here's the edit with no navigation:


Revamp the introduction page

This isn't API feedback, just intro page feedback.
I am still not sure how much value the introduction page actually has since it mostly shows off random pieces of code in the Quickstart section without any context, justification or story.
Goals that I want to achieve here:

Add at least one sentence of context at the very top to justify why the hell lamindb exists in the first place. Inspired by latch.
Reduce the Design section which is IMO too long.
Simplify the language. chatGPT didn't know what a mesh of databases is so why would random people?

chatGPT didn't know what a mesh of databases is so why would random people?

We're not targeting random people here, but data architects in biotech. Among those, it's a well known fact that the industry moved away from centralized to mesh/distributed systems. The word appears in the 5th bullet and is the only one that tries to target this highly technical audience. The fact that lamindb is a distributed/mesh system is very unique. Previously we used the word "distributed", but because "mesh" came up a few times in conversations, I changed it.
Quilt has it even on the repo-level description (https://github.com/quiltdata/quilt):

And Quilt is so far the most popular solution for data lakes in the whole industry.
So, why can't we have this word (or "distributed") in bullet 5?

So, why can't we have this word (or "distributed") in bullet 5?

We can and totally should use distributed, but mesh in my opinion is not a common enough term to describe distributed in this context.
I'll happily argue that Quilt is doing themselves a disservice by using mesh.

Besides the details, is this PR generally going into a direction that you agree with or would you rather not have it?

Add at least one sentence of context at the very top to justify why the hell lamindb exists in the first place. Inspired by latch.

I don't like adding this or probably any sentence.
We have a whole landing page at lamin.ai that explains why Lamin exists.

These are the 6 reasons what we should iterate on (Maybe there should be 3,4,5, or 7 reasons? Are these really the perfect words that nail the problems we solve? ...). There is a ton of improvement potential on the landing page.
Everbody knows that data in bio is mismanaged, and I know enough people who get annoyed when this gets pointed out in generic ways:

I think this is a hella long paragraph that would be suited for a paper about lamindb, but not for somebody who just came from the landing page at lamin.ai to lamin.ai/docs.

We can and totally should use distributed, but mesh in my opinion is not a common enough term to describe distributed in this context.
I'll happily argue that Quilt is doing themselves a disservice by using mesh.

Happy to agree on this! Mesh is weaker than distributed, but I agree this nuance is for deep down in the docs.

I now made it until down here:

I liked everything including moving the "Assumptions" further up!

I wouldn't say this:

We're not open-sourcing almost all our work because at least a comparable amount of work is done for the hub, and that's closed source.
I'd also say that "pride" rings an ideological bell, which I'd like to avoid. I think it's perfectly OK to simply use plain language (as it was) and then people can decide what they think about it. We're open-sourcing the core data access & data modeling parts to

put users in full control of their data
help defragmenting data around the world

These are very pragmatic reasons.

Add at least one sentence of context at the very top to justify why the hell lamindb exists in the first place. Inspired by latch.

I don't like adding this or probably any sentence.
We have a whole landing page at lamin.ai that explains why Lamin exists.

These are the 6 reasons what we should iterate on (Maybe there should be 3,4,5, or 7 reasons? Are these really the perfect words that nail the problems we solve? ...). There is a ton of improvement potential on the landing page.
Everbody knows that data in bio is mismanaged, and I know enough people who get annoyed when this gets pointed out in generic ways. I fear the following risks annoying readers, too:

I think this is a hella long paragraph that would be suited for a paper about lamindb, but not for somebody who just came from the landing page at lamin.ai to lamin.ai/docs.

All right, fair.
I took this as an opportunity to check how a few other tools are doing this:
MLFlow:

MLflow is an open-source platform, purpose-built to assist machine learning practitioners and teams in handling the complexities of the machine learning process. MLflow focuses on the full lifecycle for machine learning projects, ensuring that each phase is manageable, traceable, and reproducible.

W & B:

Weights & Biases is the machine learning platform for developers to build better models faster. Use W&B's lightweight, interoperable tools to quickly track experiments, version and iterate on datasets, evaluate model performance, reproduce models, visualize results and spot regressions, and share findings with colleagues.

Latch bio:

It takes months to build infrastructure with the compute, storage, and user-friendly interface necessary to run bioinformatics pipelines at scale.

The Latch SDK is an open-source toolchain to define serverless bioinformatics workflows with plain python and deploy associated no-code interfaces using single command. [...]
Neptune:

Neptune is the MLOps stack component for experiment tracking. It offers a single place to log, compare, store, and collaborate on experiments and models.

DVC:

DVC is a tool for data science that takes advantage of existing software engineering toolset. It helps machine learning teams manage large datasets, make projects reproducible, and collaborate better.

I could continue. I think one trend that I notice is that they justify in their 1-2 intro sentences why they exist. We don't:

LaminDB is an open-source data framework for biology.

Therefore, I propose that we slightly adapt our "LaminDB is an open-source data framework for biology." sentence to include a justification and remove the other fluff that I added.

I agree that this here doesn't have to be on the intro page, but it gotta move somewhere:

Can you move it somewhere so that the information doesn't get lost?

I made it to the end! I like all the changes except the long sentence at the very top. :D
Thanks, Lukas! 🙏

Thanks!
Curious about your thoughts on https://github.com/laminlabs/lamindb/pull/1514#issuecomment-2017918774. I'll make the other suggested/requested changes of course.

I agree that this here doesn't have to be on the intro page, but it gotta move somewhere:

Can you move it somewhere so that the information doesn't get lost?

It's still there though under repositories. I wanted to avoid that people find guides and tutorials and then click on the links there and think this is how they're supposed to look at them.

Re the "intro sentence".
I randomly picked "W&B". After coming from their landing page, I end up on their github readme and find

Use W&B to build better models faster. Track and visualize all the pieces of your machine learning pipeline, from datasets to production machine learning models.

Then I go to the docs, and I find the beast of a sentence that you reference:

Weights & Biases is the machine learning platform for developers to build better models faster. Use W&B's lightweight, interoperable tools to quickly track experiments, version and iterate on datasets, evaluate model performance, reproduce models, visualize results and spot regressions, and share findings with colleagues.

Compare this with our GitHub Readme that also appears on the docs landing page:

LaminDB is a data framework for biology:

Manage storage & databases with a unified Python API ("lakehouse").
Track data lineage across notebooks & pipelines.
Integrate registries for experimental metadata & in-house ontologies.
Validate, standardize & annotate.
Collaborate across distributed LaminDB instances.

I'd say you get the simpler and more consistent developer-oriented messaging on our GitHub/docs compared to W&B.
If we don't like bullets, we can try putting the bullets into a sentence that'd be similarly long as the sentence of W&B or scanpy. I prefer the 5 bullets instead of an equivalent long sentence. There are a few tools out there that list bullets as the first thing rather than a long sentence.
Modal:

Chroma:

Etc.

I could continue. I think one trend that I notice is that they justify in their 1-2 intro sentences why they exist.

So, I think you're somehow not acknowledging that we have 5 concise bullets that explain why LaminDB exists, and you seem to prefer to put the content of these bullets into a single sentence. And we can debate that! But adding a mammoth sentence in addition to the 5 bullets doesn't make sense to me.

The Latch SDK is an open-source toolchain to define serverless bioinformatics workflows with plain python and deploy associated no-code interfaces using single command.

Neptune is the MLOps stack component for experiment tracking. It offers a single place to log, compare, store, and collaborate on experiments and models.

Re content: Just like Latch's & Neptune's & W&B's and a few more of your sentences, our 5 bullets describe what LaminDB does. There is no additional attempt to describe a "why". The other examples you mention try to mix in a bit of a problem statement. I'd stay away from this in our case, because our product is even more complicated than literally all of the examples your giving in the sense that LaminDB's scope is broader than each of the tools that you're listing. This makes messaging so hard because we can't just write down a single sentence that everybody understands. (It was really simple to come up with the sentence of Scanpy.)

It's still there though under repositories. I wanted to avoid that people find guides and tutorials and then click on the links there and think this is how they're supposed to look at them.

OK! All good then! It's not so easy to follow the changes in the design section.

Discussion continued here: https://laminlabs.slack.com/archives/C04FPE8V01W/p1711372317778849

One thing I only spotted now, where I'm also not clear on the value add, is this:

Isn't it clear that we'll show off "high level features"? I mean, compare this to say, Chroma. Can we just get going and show what you can do with LaminDB rather than announcing that we'll show what we're going to do?
I think that in this instance you made a simple sentence that has the same information in it more complicated. Am I off?

Sorry, I don't get my mails properly at Helmholtz (it blocks my email provider).
Since this introduction page in general doesn't really have any story and reads a bit "list-like" instead of "motivated-by-like", I think that specifying that this is just showing off the high level features and the rest can be found somewhere else, can be useful.
However, I don't hold this change dear to my heart at all so if you disagree feel free to revert this. No problem!


Segmentation fault on OnePlus 7 Pro and Samsung Galaxy S10e

Have I written custom code?:
No, i used the android facemesh example in the mediapipe repo:
https://github.com/google/mediapipe/tree/master/mediapipe/examples/android/solutions

OS Platform and Distribution:
Build on Win 10 19042.1237

Mobile device:
Executed on OnePlus 7 Pro with Android 11. Reproduced on Samsung Galaxy S10e

MediaPipe version:
v0.8.8

Bazel version (if compiling from source):
3.7.2

Solution:
FaceMesh

Android Studio, NDK, SDK versions:
SDK Version: 30

Describe the current behavior:
When using the OnePlus 7 Pro or the Samsung Galaxy S10e a segmentation fault occurs and the app will be closed or the screen freezes, after clicking on "Start Camera", with the following error:
A/libc: Fatal signal 11 (SIGSEGV), code 2 (SEGV_ACCERR), fault addr 0x6ddb5bb998 in tid 1837 (Thread-8), pid 3843 (e.apps.facemesh)
This does not happen on my Xiaomi Mi Mix 2 under the same conditions.
Describe the expected behavior:
FaceMesh can be used without crashing
Standalone code to reproduce the issue:
https://github.com/google/mediapipe/tree/master/mediapipe/examples/android/solutions
Other info / Complete Logs :
I added the complete debug log and parts of the logcat log as attachments (The logact log contains a more detailed error message):
logcat.txt
log.txt

Duplicate https://github.com/google/mediapipe/issues/2604#issuecomment-961309080

Hi @Leander250, Duplicate #2604 Is fixed in the latest release. Could you please verify the above issue with latest release.

I tried it a few times with the latest Version (commit: cf101e62) and it seems to work now.


Shell script json inteface

Currently the shell script works as:
julia --project=. runfit.jl -p model_spec.json

Thank you. I'll review asap.

can you try two more things please?

merge main into your branch and resolve conflicts so it can be merged
see if you can write functionality to recover the project and manifest

I can help to wrap this up?

That would be helpful. I can move the PR to the branch inside of ACEsuit.


Option to write dashboard console logs in the host console

When users run an Aspire app, they run the host. Output from the host is written to the console. If there is an error in the host, such as from the resource service, these errors are written to the console.
The dashboard is started with the host, but it runs out of process. The dashboard is launched by the DCP. Its logs aren't written to a console the user can see. If an error occurs in the dashboard, then there is no way to access the details. Errors reported to us won't have any logging information.
There should be a host config option to output dashboard logs to the console.

@JamesNK  - assuming this needs to move to preview 6 (prior to GA)

I'll take this one @JamesNK


Does not handle local URIs with spaces

The library seems not to work for me for images that have spaces in their file names (and are therefore URI encoded). I think I've tracked this to the following line: https://github.com/davemorrissey/subsampling-scale-image-view/blob/master/library/src/com/davemorrissey/labs/subscaleview/decoder/SkiaImageRegionDecoder.java#L61
If this were instead:
ContentResolver contentResolver = context.getContentResolver();
decoder = BitmapRegionDecoder.newInstance(contentResolver.openInputStream(Uri.parse(uriString.substring(FILE_PREFIX.length()))), false);

I think there would be no issue.

Just confirmed the issue is URL encoding by passing a the URI as a string to ImageSource, but using URLDecoder beforehand.

PR: https://github.com/davemorrissey/subsampling-scale-image-view/pull/115

Same problem here. Please apply PR.

I can't reproduce this at the moment. This works:
imageView.setImage(ImageSource.uri("file://sdcard/DCIM/IMAG 0210.jpg"));

This also works:
imageView.setImage(ImageSource.uri(Uri.parse("file://sdcard/DCIM/IMAG 0210.jpg")));

Your suggested in PR #115 breaks both of these cases. Could you provide some sample code I could use to reproduce the problem you're seeing?

Right, so I think the issue is that android can return a URI with URI encoding in it.
The chain of events to arrive at this situation was to iterate files in a directory and form a URI like this:
Uri.fromFile(file)
Which results in something like:
file:///storage/emulated/0/Pictures/MyApplication/My%20Image.jpg  which is why I tried to make sure Uri.parse was called on the string.
So the actual problem here is that file URI's need to be decoded as URI encoded objects...
Also worth noting, the same URI worked fine with the Glide framework... I'm not sure what they do differently.

I've tried to find a workaround for this, but the difficulty is that the view has no way of knowing whether the URI string it's given is URL encoded, or just contains %xx escape sequences deliberately. Only the app code will know that, based on how it generated the URI. The app code can use URLDecoder.decode to decode the URI if it's from Uri.fromFile or any other source that encodes the URI.
Supposing my file is named Imag 001.jpg and I use Uri.fromFile to get the URI, it will be encoded to Image%20001.jpg. Now if I have a file named Imag%20001.jpg (unlikely I know) and I use some method that doesn't encode it, the view can't distinguish between them and whatever fix I put in for one will break the other.
I think it will have to be the app developer's responsibility to check whether their URLs are getting encoded, and decode them if necessary.

Hmmm, so maybe I need to do a test with Glide with an image named Imag%20001.jpg and loaded differently. I suppose it assumes that all URIs must be URI encoded.

Let me know what you discover. I don't have much time to set up test projects with other libraries like this, but if there's a fix I can add without breaking anything for existing users I'll do it.

I've put in a workaround for Uri.fromFile URL encoding the path. The ImageSource constructor will check whether the file exists, and if not it will URL decode the path and try again. This works for Uri.fromFile provided the path you used in the File constructor was not URL encoded, because Url.fromFile would then double encode it. This will be in the next release.

does this have a solution? still i can't access files with "space" in it.


AzureDevOps does not exist as an option for Network Security Group service tags

AzureDevOps does not exist as an option for Network Security Group service tags.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dc5a4cfa-aa84-9f0f-55f4-71ec4861c070
Version Independent ID: d621620a-c924-03b1-8b43-f99c33e58bb8
Content: Add IP addresses and URLs to allow list - Azure DevOps
Content Source: docs/organizations/security/allow-list-ip-url.md
Product: devops
Technology: devops-security
GitHub Login: @chcomley
Microsoft Alias: ChComley

Azure DevOps service tag is released.
https://devblogs.microsoft.com/devops/azure-devops-service-tag-released/

AzureDevOps is still not available as a Service Tag in the Network Security Group Editor.

https://docs.microsoft.com/en-us/azure/virtual-network/service-tags-overview
AzureDevOps | Azure Dev Ops.Note: This tag is not currently configurable via Azure Portal | Inbound | No | Yes


current login not global, but explicit per session

Both in the current situation as with #358 the current login (master and grid) is determined by a file.
That means its the same in any session for that user. With the multi master setup it would be nice if the current master/grid is not global, but session specific, like docker-machine.
This allows:

running separate session point at different master/logins, eg to compare stuff, trailing logs, running scripts
on a new session no default master/grid active, so you always have to explicitly set this (pre-caution).

nice gist to update your command line: https://gist.github.com/jakolehm/0edf6b90b205aa34d895 (requires jq)


4090 opt

tests passing on h100 and 4090, 50% faster on 4090, should have modest perf gains on h100, too.


Download logs in a date range for workflows generated from the same wftmpl or cron workflow

Summary
Add the capability to download logs for all workflows run in a time range that were created from a workflow template or cron workflow.
Motivation
We run a lot of workflows that were generated from a workflow template.  Generally our developers care about logs in a date range for all workflows of a specific workflow template.  We aren't aware of any easy way to get just those logs in a timely way.
Proposal
Not sure the best approach, or if it's a need for other users.  We could write a script that queries the workflow archive and then downloads the necessary files directly from minio/s3.  Figured I'd make a ticket just in case it's a reasonable thing for argo to support natively (potentially through the argo cli?).

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

This could be achieved by adding the workflow template in the log context. We should also label the pods with template.


Focus on Search bar while pressing K

Make the focus on to search bar input while pressing 'k' so user can afterwards starts writing search term

👋


Is there a grammar / keyword / more than just the basic examples?

This is great, but I'd really love to know the full possibilities when specifying diagrams.
Is there a grammar published anywhere, or a list of all the keywords and things that can be used for each diagram?
Because I'm not sure if the examples are extensive to all the possibilities.
Just curious if there is more.

nevermind.  I was skipping over a particular documentation page


SiteCore demo

Work with SiteCore architect for creating a SiteCore demo of mixed workloads.

Presentation was given by Bart at https://europe.sugcon.events/.


Error in document

Hi there,
There is a defect in following mentioned link.
https://msdn.microsoft.com/en-us/library/microsoft.owin.security.oauth.oauthauthorizationserverprovider(v=vs.113).aspx
Please scroll to "Properties" and go to "OnGrantCustomExtension".
I believe that word "andpoint" in line "Called when a request to the Token andpoint arrives with a "grant_type" of any other value" should be written as "endpoint".
Thanks,
Jai

Unfortunately the file isn't under our control so we can't fix it.


Enter tab issues

#483
Fixed issues with workaround (don't create new tabs on button press - only influence visibility).
Did not determine why we could un-focus.
Removed async commands for confirmation dialogs.
@devoncarew @lukechurch

lgtm w/ the noted changes - thanks for tackling this!

Merging


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 All CLA requirements met.


Fix formatting of GraphiQL errors

Fixes https://github.com/graphql/graphiql/pull/636
Fixes https://github.com/graphql/graphiql/pull/722
This approach allows overriding the error handler. By default it uses JSON.stringify on the error; if the result is '{}' then it will instead create a {message,stack} error to stringify (since JSON.stringify(new Error("Something")) is '{}').

this works with subscriptions?

It should do; we had to hack around this issue in PostGraphile:
https://github.com/graphile/postgraphile/blob/7979cfb10f3246c7c198be8db2c929e82e0e4323/postgraphiql/src/components/PostGraphiQL.js#L320-L333
I'll give it a try.

Yes, definitely works with subscriptions. Took me a while to get our dev server up and running with subs!

@benjie awesome, lgtm!


Docs

Here is the latest version of the docs. So far all integrators were documented as well as all systems. Functions within integrators must still be properly documented, but there is a python script in the docs folder that can help with automatically doing this.

Thanks, @JLMR29! I will review this throughout the week :)

It looks great, however I need to go through it file by file.
Can you again make a merge request to the branch "docs_comments". Once I have verified everything there, I merge it to the master branch.


Feat/#180 android aab

specifying aab: true in buildSchemes does the job.
"buildSchemes": {
        "debug": {
          "id": "renative.helloworld.debug",
          "signingConfig": "Debug",
          "bundleAssets": true,
          "aab": true
        }
      }

hi  there,
have been trying with the aab tag but its only generating an APK not the AAB or am I looking in the wrong location?

@MwareSolutons what version of RNV you running the project?
latest 0.29.0 should support it. just tested
rnv run -p android with
{
   "platforms": {
        "android": {
            "aab": true
        }
    }
}

generates:
./platformBuilds/helloworld_android/app/build/outputs/bundle/debug/app.aab

ah that must be it I am on 0.28.0


Error on a simpe query

I am running the following code :
import xlsxwriter
import xlsxwriter as xl
from capiq_excel import download_data
download_data(
['MSFT','AAPL'],  # Any id type. Ticker, name, CUSIP, ISIN, etc.
financial_data_items=['IQ_REV', 'IQ_EBIT'], # Financial data variable names from Capital IQ
freq='Q',
num_periods=4
)
And getting the following error :
com_error                                 Traceback (most recent call last)
 in
7 from capiq_excel import download_data
8
----> 9 download_data(
10     ['MSFT','AAPL'],  # Any id type. Ticker, name, CUSIP, ISIN, etc.
11     financial_data_items=['IQ_REV', 'IQ_EBIT'], # Financial data variable names from Capital IQ
c:\users\kapis\appdata\local\programs\python\python38-32\lib\site-packages\capiq_excel\main.py in download_data(company_ids, financial_data_items, market_data_items, ids_folder, data_folder, data_outpath, ids_outpath, restart, timeout, run_failed, **financial_command_kwargs)
36
37     if restart or not os.path.exists(ids_outpath):
---> 38         capiq_ids = download_capiq_ids(
39             company_ids,
40             outpath=ids_outpath,
c:\users\kapis\appdata\local\programs\python\python38-32\lib\site-packages\capiq_excel\ids.py in download_capiq_ids(ids, outpath, folder)
24
25     print('Populating XLSX files for ids')
---> 26     populate_all_ids_in_folder(folder)
27
28     print('Combining all ids into a single CSV file')
c:\users\kapis\appdata\local\programs\python\python38-32\lib\site-packages\capiq_excel\ids.py in populate_all_ids_in_folder(folder, restart)
37
38     for file in file_tracker.file_generator():
---> 39         populate_capiq_ids_for_file(file, excel)
40
41
c:\users\kapis\appdata\local\programs\python\python38-32\lib\site-packages\capiq_excel\workbook\populate\main.py in populate_capiq_ids_for_file(filepath, excel)
75     wb = excel.Workbooks.Open(filepath)
76     successful = _wait_for_capiq_result(excel)
---> 77     _copy_paste_values(excel, wb, cell_range='A:Z')
78     excel.ActiveWorkbook.Close(SaveChanges=True)
79     return successful
c:\users\kapis\appdata\local\programs\python\python38-32\lib\site-packages\capiq_excel\workbook\populate\main.py in _copy_paste_values(excel, wb, cell_range)
83     ws = wb.Sheets('Sheet')
84     ws.Range(cell_range).Copy()
---> 85     ws.Range(cell_range.split(':')[0]).PasteSpecial(Paste=constants.xlPasteValues, Operation=constants.xlNone)
86     excel.CutCopyMode = False
87
~\AppData\Local\Temp\gen_py\3.8\00020813-0000-0000-C000-000000000046x0x1x9_Worksheet.py in Range(self, Cell1, Cell2)
237         # The method Range is actually a property, but must be used as a method to correctly pass the arguments
238         def Range(self, Cell1=defaultNamedNotOptArg, Cell2=defaultNamedOptArg):
--> 239 		ret = self.oleobj.InvokeTypes(197, LCID, 2, (9, 0), ((12, 1), (12, 17)),Cell1
240 			, Cell2)
241                 if ret is not None:
com_error: (-2147352567, 'Exception occurred.', (0, None, None, None, 0, -2146827284), None)
Can you tell me what am I missing here?

Same issue here...

Same issue... Does anyone know how to fix it?

Same issue... Does anyone know how to fix it?

Thanks for the prompt reply. Yes, this issue happens repeatedly.
Running code :
from capiq_excel import download_data
download_data(
['MSFT', 'AAPL'],  # Any id type. Ticker, name, CUSIP, ISIN, etc.
financial_data_items=['IQ_TOTAL_REV', 'IQ_COST_REV'],  # Financial data variable names from Capital IQ
market_data_items=['IQ_FLOAT_PERCENT'],  # Market data variable names from Capital IQ
freq='Q',
num_periods=6
)
Trackback:
Creating XLSX files with commands to get ids
Populating XLSX files for ids
Traceback (most recent call last):
File "D:/OneDrive/Programming/documents_python/Entrepreneur/PoodleFiance/poodle-data/src/capitaliq/capiq.py", line 18, in
restart=False
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\main.py", line 41, in download_data
folder=ids_folder
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\ids.py", line 26, in download_capiq_ids
populate_all_ids_in_folder(folder)
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\ids.py", line 39, in populate_all_ids_in_folder
populate_capiq_ids_for_file(file, excel)
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\workbook\populate\main.py", line 77, in populate_capiq_ids_for_file
_copy_paste_values(excel, wb, cell_range='A:Z')
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\workbook\populate\main.py", line 85, in _copy_paste_values
ws.Range(cell_range.split(':')[0]).PasteSpecial(Paste=constants.xlPasteValues, Operation=constants.xlNone)
File "C:\Users\gaoha\AppData\Local\Temp\gen_py\3.7\00020813-0000-0000-C000-000000000046x0x1x9_Worksheet.py", line 240, in Range
, Cell2)
pywintypes.com_error: (-2147352567, 'Exception occurred.', (0, None, None, None, 0, -2146827284), None)
Version:
Excel Version 2011
Windows 10 1909 (18363.900)

Thanks for the prompt reply. Yes, this issue happens repeatedly.
Running code :
from capiq_excel import download_data
download_data(
['MSFT', 'AAPL'],  # Any id type. Ticker, name, CUSIP, ISIN, etc.
financial_data_items=['IQ_TOTAL_REV', 'IQ_COST_REV'],  # Financial data variable names from Capital IQ
market_data_items=['IQ_FLOAT_PERCENT'],  # Market data variable names from Capital IQ
freq='Q',
num_periods=6
)
Trackback:
Creating XLSX files with commands to get ids
Populating XLSX files for ids
Traceback (most recent call last):
File "D:/OneDrive/Programming/documents_python/Entrepreneur/PoodleFiance/poodle-data/src/capitaliq/capiq.py", line 18, in
restart=False
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\main.py", line 41, in download_data
folder=ids_folder
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\ids.py", line 26, in download_capiq_ids
populate_all_ids_in_folder(folder)
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\ids.py", line 39, in populate_all_ids_in_folder
populate_capiq_ids_for_file(file, excel)
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\workbook\populate\main.py", line 77, in populate_capiq_ids_for_file
_copy_paste_values(excel, wb, cell_range='A:Z')
File "D:\Applications\Anaconda3\envs\poodlefinance\lib\site-packages\capiq_excel\workbook\populate\main.py", line 85, in _copy_paste_values
ws.Range(cell_range.split(':')[0]).PasteSpecial(Paste=constants.xlPasteValues, Operation=constants.xlNone)
File "C:\Users\gaoha\AppData\Local\Temp\gen_py\3.7\00020813-0000-0000-C000-000000000046x0x1x9_Worksheet.py", line 240, in Range
, Cell2)
pywintypes.com_error: (-2147352567, 'Exception occurred.', (0, None, None, None, 0, -2146827284), None)
Version:
Excel Version 2011
Windows 10 1909 (18363.900)

Thanks @VincentGaoHJ for the additional information. I spun this project off from a larger project years ago, and I guess I never tested the download_data function as I already had Capital IQ ids so was using download_data_for_capiq_ids, and back then I wasn't yet regularly including tests in all my libraries. When I ran the download_data function now, I faced the same issue.
I believe the bug should be fixed now in 0.4.2, so please upgrade and give it a try. I have also added a test so that I know it is all working. Let me know if you run into any further issues.

Thanks @VincentGaoHJ for the additional information. I spun this project off from a larger project years ago, and I guess I never tested the download_data function as I already had Capital IQ ids so was using download_data_for_capiq_ids, and back then I wasn't yet regularly including tests in all my libraries. When I ran the download_data function now, I faced the same issue.
I believe the bug should be fixed now in 0.4.2, so please upgrade and give it a try. I have also added a test so that I know it is all working. Let me know if you run into any further issues.

Thanks! The project is really helpful~

Thanks! The project is really helpful~


Add idlastro license

Add idlastro license

Looks like I missed or forgot about the action for me in https://github.com/sot/Ska.Sun/pull/6#issuecomment-331152317 .  I think this is all I had in mind (though I don't know if you prefer the comment someplace else @taldcroft ).


Adding inline rolls to where the roll is and multiple inline rolls.

Hey. So if i have a text like "Something something (roll: 3d6), then something something (roll: 1d20+3).", it only rolls the first one. I wonder if it possible to make the bot roll the second one too (and the third one, so on)? Also would it be possible to add the result of the inline roll to the place it was called? For example:
"Something something (roll: 3d6)"
magic happens
"Something something (roll 3d6, Result: 10)"
I wonder if this is possible via the manage messages permission?

Editing the original message is not possible, no. Nobody (including bots) has the ability to edit other peoples' messages.

So, I would like it if the inline message was displayed near the result of the die roll. What if the message was just displayed in the bot's response? For example:
Sifaus: Something something (Roll: 3d6)
Bot: @Sifaus rolled 6 (3 + 1 + 2 = 6) for Something something
I'm trying to keep the rolls as organized as possible.

Aye, if the bot re-printed the message it would be handy, was about to ask for that even for normal rolls.
So when I say !roll D20+5
the answer is:  @Author rolled D20+5 and got 17 (12, 5)
And in the case of an inline roll it would just reprint the message but with the resulted inserted where the roll command used to be.


DCEModel tests for OG and LCB code run very slowly

LCB's code in particular is quite simple.
Does anyone have a guess as to why this is?
Thanks.

@mjt320 I notice multiple test files have OG. Can you point to the exact files to be examined?

Thanks @sunyu0410
test_DCEmodels_etofts_OG_MO_AUMC_ICR_RMH
test_DCEmodels_tofts_LCB_BNI
Looking at this again, perhaps the slow execution is just down to the implementations, particularly for the convolution step. Maybe it's not so surprising...

@mjt320 Thanks for the info. If it generates the expected outcome, then it probably is not a bug. I've removed the bug label.

Issue resolved


feat(no-path-verbs): prohibit use of verbs in api paths

PR for issue #3.
Description of changes
Mature RESTful designs represent

HTTP paths as resources and
HTTP methods as operations available for those resources.

Traditional RPC and SOAP calls invoke object methods for CRUD operations. This rule aims to prevent RPC and SOAP operation name "leakage" into Swagger API paths.
Pull request (PR) check-list

:white_check_mark: Please review and check the appropriate items.

Code standards compliance

[x] Yes, ESLint passes.

Code quality. Do the quality gateways pass with an "A" grade?

[x] Complexity
[x] Duplications (0.00%)
[x] Issues (0 issues or documentation for "won't fix" cases)
[x] Maintainability/Technical Debt
[x] Reliability
[x] Security

Test coverage

[x] The source code is 100% covered with passing specs.

:information_source: These tasks are not required to open a PR and can be done afterwards, or while the PR is open.

Coverage remained the same at 100.0% when pulling 82f8a027c9526b031818247ee91b37006325a4ea on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

Coverage remained the same at 100.0% when pulling 82f8a027c9526b031818247ee91b37006325a4ea on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

Coverage remained the same at 100.0% when pulling c6c035c2808285497c7e1049765da2e02db63578 on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

Coverage remained the same at 100.0% when pulling c6c035c2808285497c7e1049765da2e02db63578 on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

Coverage remained the same at 100.0% when pulling 6c45785ea4aa7c4e887d3ce17c79ea01b4816742 on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

Coverage remained the same at 100.0% when pulling 81063fa7320a4b0b42e9cbb6a3c6c28654b5acb4 on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

Coverage remained the same at 100.0% when pulling 81063fa7320a4b0b42e9cbb6a3c6c28654b5acb4 on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

Coverage remained the same at 100.0% when pulling db3326c9fcebeba0544f1a82ec2803057ccc13cf on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

Coverage remained the same at 100.0% when pulling db3326c9fcebeba0544f1a82ec2803057ccc13cf on feat/#3-no-path-verbs into b6dcd9342d4b724eb26bf3f8d36f7b22032bb759 on develop.

chore(scm): merge #9.
Closes #3.


Bords with integrated ethernet

Hi, I have already successfully used this great library with unos and megas....
But now I got an arduino Leonardo ETH with integrated W5500 ethernet and I can not get it to work with the RC522 reader which Works fine with the UNO and the Standard-W5100 ethernetshield for example.
I also tried another board (http://tronixlabs.com/arduino/boards/ethernet/dfrobot-arduino-compatible-with-onboard-w5500-ethernet-and-poe-australia/) that seems to be somewhat similar.
Both boards Do Not recognize the RC522 reader during setup.
Even if I deactivate the CS from ethernet and SD, the reader can not be accessed.
There must be something general that Blocks the Icsp or so?
I have used the mosi, miso and sck from the 6 pin icsp header of the board because they do not have them on the GPIO Pins like the UNOs or Megas and also tried different Pins as the CS pin for the reader.
No success on getting the reader to respond.
Is there anything else I can try?
Thanks in advance and kind regards
Gerald

Check #97


Roll Flutter Engine from 538975f2511b to ebee709bc549 (2 revisions)

https://github.com/flutter/engine/compare/538975f2511b...ebee709bc549
2024-01-19<EMAIL_ADDRESS>[Fuchsia] Redo - Use chromium test-scripts to download images and execute tests (flutter/engine#49847)
2024-01-19<EMAIL_ADDRESS>Roll Skia from fb3f61d932e6 to 2536dc6fef1d (2 revisions) (flutter/engine#49892)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jonahwilliams@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Added Note for New Contributors

Summary
Added Note for New Contributors
Full changelog

[Documentation Update]
[Linked the respected files as hyperlinks

 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.

just pushed a new commit with what I meant it to be, wdyt?

@Tieske Thank you for the approval. I am looking forward to more such contributions. Could you please tell me when will it be merged? The merger will act as a motivation for me.

merging is not up to me, but on @Kong/team-core

Thank you for your reply @Tieske


Do not enlarge images

Trello Card
When the Image component is provided with a maxWidth and maxHeight, we would calculate an aspect ratio for the image that would fit within those bounds. This would create values that we could multiple the width and height by to get new widths and heights. The problem was in doing this when the width and height were less than the maxWidth and maxHeight. In such a case we would multiple the width and height resulting in larger numbers. In these cases, we should not apply a multiplier, setting the width and height to their original values.

@knicklabs Amazing!! Thank you for this 🙏


[VS Code] Request was cancelled

Error:
Request was cancelled
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.1011

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.29.1

platformName
Linux

platformVersion
4.15.0-42-generic

@ashr123 Hey! thank you for reporting the issue. We’re tracking this issue via #469, so I’m going to close this as a duplicate.  It would be great if you send your logs for investigation. You can run "Live Share: Export logs" rom VS Code command palette , and add your comments / logs under #469


puller(cdc): more tests and logs for frontier and kv-client

What problem does this PR solve?
Issue Number: close #10370
What is changed and how it works?

add more details about tableID and tableName into kv-client logs;
add more details for check locked regions;
add a log for region becoming initialized;
add more unit tests for RegionRangeLock and Frontier.

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

/cherry-pick


Notice manual update

Used artifacts from: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2609001&view=artifacts&pathAsName=false&type=publishedArtifacts
Build from: Mar 6, 2023 at 9:15 AM PST
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2609001&view=results

What tool(s) are we using to generate this?

I believe this is generated by this internal pipeline.  There is some common automation that looks at all your dependencies and then generates this document.

/checkenforcer override

/check-enforcer override


Kotlin 1.9.x AtomicReferenceFieldUpdater was not found

I recently updated two of my projects to Kotlin 1.9.10 and I found out that they no longer work with TeaVM because of:
#################################################################
|
| Compiler problems
|
#################################################################
| Class: kotlin/LazyJVM.kt:124
| Method: kotlin.SafePublicationLazyImpl.<clinit>()V
| Text: Class java.util.concurrent.atomic.AtomicReferenceFieldUpdater was not found

I use 1.0.0-b7 version and I can confirm that everything works with Kotlin 1.8.x (1.8.22 for example).
Any idea why I get this problem in Kotlin 1.9.10 and how to fix it?
I tried the following without success:

Upgrade to 1.9.20-RC
Downgrade to 1.9.0
Change TeaVM to 1.0.0-SNAPSHOT (that actually adds a second compiler error| Class: -
| Method: com.github.xpenatan.gdx.backends.teavm.TeaGL20.compareObject(Lorg/teavm/jso/JSObject;Lorg/teavm/jso/JSObject;)Z
| Text: Native method com.github.xpenatan.gdx.backends.teavm.TeaGL20.compareObject(Lorg/teavm/jso/JSObject;Lorg/teavm/jso/JSObject;)Z has no implementation

was not found means that when teavm does not have the class implemented.
I guess kotlin 1.9.10 is using it and teavm for some reason is trying to parse is now.
TeaGL20 you can ignore, it should work with this error. I think the problem started when I added @Emulate anotation in GL20.  I'll try a different approuch.

Hey @Quillraven.
Can you try the snapshots? I just updated to teavm version 0.9.0 and from the release note it now supports AtomicReferenceFieldUpdater. https://github.com/konsoletyper/teavm/releases/tag/0.9.0

I have good and bad news!
Good: both compilation errors are gone and one of the games is working again using the SNAPSHOT version!
Bad: the other game still does not work. I can see the LibGDX logo and the first few frames of the game, which is better than before. I also see some debug messages in the browser console. But after a few frames it simply freezes and I get the browser popup that the page is not reacting and if I want to close it.
Working: https://github.com/Quillraven/MysticGarden/tree/kotlin
Not-working: https://github.com/Quillraven/MysticWoods/tree/fleks-2.0

For mysticwoods. Its a deadlock somewhere.  Its getting inside a loop and cant escape.

I sent you video on discord

Cool - will have a look in more detail tomorrow. I know that it worked before with Fleks 2.4 (ECS) and TeaVM 1.0.0.b7. I think the normal desktop backend is also still working. That is what I used to check if Fleks 2.5 has any sideeffects.
I will see if something has changed in findNearbyEnemies or if I can rewrite it to avoid the deadlock.
fyi: before this project did not work because it used GdxAI with some special annotations to pass info from a .tree file to a AI task. I updated that recently as well to the latest LibKTX version which supports a DSL for behavior tree creation. With that change, this project was then successfully running as well in the browser because the annotations + reflection stuff was no longer needed :)

I quote myself from Discord:

The problem is using "mutableSetOf()". For whatever reason this causes the deadlock. Using "hashSetOf()" is working. The difference is, that mutableSetOf is using a LinkedHashSet while hashSetOf uses a HashSet.
No idea why this is a problem since Kotlin 1.9.x but it is rather simple to fix 😉

Thank you for pointing me to the function that caused the issue!


Network Calls resulting in "Synchronous remote object proxy error"

Describe the bug
Each time I attempt to make a network call with the AWSMobileClient SDK, my console is flooded with error messages that look like so:
Synchronous remote object proxy returned error: Error Domain=NSCocoaErrorDomain Code=4099 "The connection to service on pid 0 named com.apple.commcenter.coretelephony.xpc was invalidated." UserInfo={NSDebugDescription=The connection to service on pid 0 named com.apple.commcenter.coretelephony.xpc was invalidated.

To Reproduce

Import AWSMobileClient in a Swift file
Make a network call (e.g., AWSMobileClient.default().signUp(username: email, password: password)
Notice the flood of messages received in the console in Xcode

Expected behavior
A console that doesn't have these errors.
Screenshots
N/A
Environment(please complete the following information):

Amplify Framework Version: The latest version of AWSMobileClient
Dependency Manager: Cocoapods
Swift Version : 5.0

Device Information (please complete the following information):

Device: Simulator (iPhone 11 Pro)
iOS Version: iOS 13
Specific to simulators:

Additional context
The network call actually succeeds, and the app doesn't crash. Removing the Cocoapod results in the console errors not appearing.

Hi @rogerogden
This looks like Xcode related issue - https://forums.developer.apple.com/thread/108883.
Maybe try the workaround mentioned in stack overflow? https://stackoverflow.com/questions/52455652/xcode-10-seems-to-break-com-apple-commcenter-coretelephony-xpc

@royjit Thanks for getting back to me. I'll run the Terminal command. It just seemed interesting that it only happened when I installed the Amplify Cocoapod, and not before.

Hi @rogerogden
This looks like Xcode related issue - https://forums.developer.apple.com/thread/108883. Maybe try the workaround mentioned in stack overflow? https://stackoverflow.com/questions/52455652/xcode-10-seems-to-break-com-apple-commcenter-coretelephony-xpc

Solved the issue, thx.


[test] expectCrashLater(withMessage:) reliability workaround

Don't check for the message using expectCrashLater(withMessage:). It is not reliable. Just check for a crash.

Works around SR-16103, and a similar issue when testing on simulators.

@swift-ci please smoke test

@swift-ci please test macOS platform

@swift-ci please test macOS platform

@swift-ci please test

Good idea!


Cooldown between toggles

You should add a 60 second cooldown between the toggle to prevent players from abusing it and getting out of scenes easily.

There is now the simplepassive_cooldown convar, this sets the number of milliseconds seconds between each toggle.


adding blockProgress capability for checkout ui extensions

resolves https://github.com/Shopify/checkout-web/issues/12795
we've added a new block_progress flag to our checkout ui extension configuration

@cpeddecord we are archiving this repo - can you open this PR in the cli repo instead?


Don't gather data in export_scaling_file

Fix #1400
Purpose
This code is causing the long runs to fail, and doesn't appear to be used.
To-do

Content

[ ] I have read and checked the items on the review checklist.

bors r+

bors r+


Problem Json File

In the web browser , the console : EXCEPTION: SyntaxError: JSON.parse: unexpected character at line 2 column 1 of the JSON data , is there any problem with the json file ????

Fixed here https://github.com/Vintharas/angular2-step-by-step-06-consuming-real-data-with-http by adding 'accept' header to http request ^_^

Good morning Jaime ,
I have tried this example and it works fine , thank you very much .
One other question , is example is valid for the current version of angular
or not , in the future i will face some problems or not ????
2016-07-31 22:33 GMT+01:00 Jaime<EMAIL_ADDRESS>
Closed #2
https://github.com/Vintharas/ngstockholm-angular2-workshop/issues/2.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Vintharas/ngstockholm-angular2-workshop/issues/2#event-740366539,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATHVT08h3PUV-07U1krfF3oBG4yYN6Bjks5qbRS8gaJpZM4JW7-p
.

This example is in the beta version but I have written new samples in the latest version here:
https://github.com/Vintharas/angular2-step-by-step-01-your-first-component
https://github.com/Vintharas/angular2-step-by-step-02-refactoring-to-services
https://github.com/Vintharas/angular2-step-by-step-03-your-second-component-and-data-bindings
https://github.com/Vintharas/angular2-step-by-step-04-routing
https://github.com/Vintharas/angular2-step-by-step-05-forms-and-validation
https://github.com/Vintharas/angular2-step-by-step-06-consuming-real-data-with-http


the cardholder was not authorized

the cardholder was not authorized. this is used in 3-d secure authentication. I was not allowed to process a transaction when the plugin popped up another dialog telling me to wait for verification, I was expecting it to show me what to do, but nothing.
Any help will be useful please

Did this happen in staging? If yes, what card details did you use?

For all I know I have read through the Paystack docs for that, saw that it's to verify if the transaction is coming from the owner of the card. I think ave seen this a lot which paying using a website it redirects to Visa or Master card verification page
i think that was what wanted to happen but did not.

Did this happen in staging? If yes, what card details did you use?

card details?

I used the card number, expiring date and cvv number

Yes, if this happened in staging, it means you might have used one of the Paystack test cards.

Any way I can reproduce this?

Yes, if this happened in staging, it means you might have used one of the Paystack test cards.

Nop..... I used my bank card for this. and I was threw that error

thanks for the response bro... appreciated

The dialog in question does it have any private information? If no, can you send a screenshot?

Or you asking for the other dialog... wait let me run the process again and screenshot it

After this.... it shows me "the cardholder was not authorized"

Please, can you try a payment in staging and let me know if this also happens? Please use the  test card titled "Bank Auth Simulation".

Okay it did that but... the transaction was successful... I used my Visa card on a live account of PayStack but it will just say processing and fail the transaction...
but with this test card it just did processing and display a msg saying "close tap to continue" after closing tab, transaction was made successful

I'm having the same problem. On the dashboard on paystack, the transactions are marked as ongoing

I'm having the same problem. On the dashboard on paystack, the transactions are marked as ongoing
EDIT:This seems to be a problem only when other authentification methods are required

so you got your way around it?

@quiet-programmer So did you get it to work or is it a limitation of the plugin?

Later worked perfectly fine... when I removed the version name number of the plugin in pubspec.yaml
From-> flutter_paystack: version number
To -> flutter_paystack:
Clean flutter build and rerun build
That worked for me...

I will be closing this issue since it appears to have been resolved.


Add statement normalization before execution.

Overview
This commit adds a call to Server.NormalizeStatement() immediately before execution. It defers normalization just before execution in case statements leading up it will alter the normalization process.

lgtm, :ship:


Add 'undocumented internals' section to 5.2 upgrade considerations

While it's good for us to err on the side of more information in our upgrade notes (because there are always people doing hacks that rely on internals and then complaining that it wasn't mentioned...), the ones for 5.2 are getting pretty big. I therefore propose moving the ones relating to undocumented functionality into their own section - essentially to communicate "you can stop reading here, but if you stumble on something breaking in this release that isn't covered above, here is some stuff that will show up in a documentation search" :-)

Thanks for this, much better.


[BUG] Websockets appear nonfunctional when Rancher is installed in an existing k8s cluster with an alternative ingress controller

Rancher Server Setup

Rancher version: 2.7.0
Installation option (Docker install/Helm Chart): Helm Chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): K8S 1.24.9

Proxy/Cert Details: No proxy.
Other: Helm chart installed with:

      helm install rancher rancher-stable/rancher
      --wait
      --kubeconfig=/home/ansibleadmin/.kube/config
      --version {{ ansiblevars['rancher-ver'] }}
      --namespace cattle-system
      --create-namespace
      --set hostname={{ rancher_host }}
      --set bootstrapPassword={{ temppassword }}
      --set ingress.enabled=false
      --set ingressClassName=nginx
      --set tls=external

Created nginx-ingress with yaml:
---
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: rancher-lb
  namespace: cattle-system
spec:
  ingressClassName: nginx
  rules:
  - host: {{ rancher_host }}
    http:
      paths:
      - path: /
        backend:
          service:
            name: rancher
            port:
              number: 80
        pathType: ImplementationSpecific
  tls:
    - hosts:
      - {{ rancher_host }}
      secretName: rancher-lb-cert

Information about the Cluster

Kubernetes version: K8S 1.24.9
Cluster Type (Local/Downstream): Local

User Information

What is the role of the user logged in? Admin

Describe the bug

Cannot view pod logs or open shells.  Attempts all fail to connect / show "disconnected" in the bottom status bar.  Apparently no websocket connectivity.
To Reproduce
Install Rancher in a vanilla k8s cluster running metallb as the cluster provided LoadBalancer, with nginx-ingress (note: not ingress-nginx) as the ingress controller as above.
Result
A mostly functional Rancher installation.  The dashboard and other features work and the cluster can be explored, but some functionality is missing as described above.
Expected Result
A functional Rancher installation.
Additional context
I believe this is simply some missing configuration for the ingress.  The nginx-ingress documentation mentions that to support websockets, you need to add the annotation nginx.org/websocket-services to the ingress configuration, but this requires service details that aren't documented anywhere I have been able to find.

  annotations:
    nginx.org/websocket-services: rancher

Was the magic sauce.


bump cloud.google.com/go/spanner from v1.55.0 to v1.64.0

WHAT

SSIA
WHY

Support new statements such as RENAME.

📝 Since the spanner-emulator could not use RENAME statement, I confirmed that RENAME can be used with the actual Spanner instead of in tests.


Make SSPL service less chatty.

Created by: qnikst
Do not resent to halon messages that halon is not interested in.
Do not log too much information in stdout, unless tracing is
enabled.

Closing as an obsolete


Missing value although using Batch

I am using batch node to query data from InfluxDB over a specified period. The following is a short example:
 batch

                |query(
                        '''
                       SELECT "timeUntilEndOfLicence",

                       FROM "artifactory__systems"."autogen".Artifactory
                        '''
                        )
                        .period(9m)
                        .every(20s)

                  |last('timeUntilEndOfLicence')
                         .as('timeUntilEndOfLicence')
..............

In this Tickscript I do calculations in  an eval-node with the field timeUntilEndOfLicence. The result field called "licenceStatus" is pushed to InfluxDB to the same database with the same measurement (artifactory__systems"."autogen".Artifactory). In another Tickscript I do the same calculating for another field  and push the result field called "storageStatus" to InfluxDB. All works fine.
In another Tickscript I want to combine "licenceStatus" and "storageStatus" to do further processing. The way I access these fields is exactly the same as stated above using batch, period and last node. Nevertheless kapacitor log shows that the fields licenceStatus and storageStatus are missing. Checking the logs and InfluxDB entries these fields seem to be available though.
Here https://github.com/influxdata/kapacitor/issues/1217 they had a similar problem and suggest using batch instead of stream as solution. It seems that there still are some problems with batch.

@ss682 I think this may be better suited for https://community.influxdata.com/. Can you please ask the question there so that we can answer it appropriately.


How to stop current request without using exit(0)

I am creating a simple PHP module with PHP-CPP library.
I make this to limit process per IP address per php script file. I have found a solution for everything I needed, except 1 thing : I cant stop the script cleanly.
If you look in the script it have exit(0); .. If I use for example Php::echo(); for writing some headers and HTML, that result doesn't get returned to the web browser (nor telnet).
Do you know a method to echo some headers and body then kill the script without executing the final script ?
Thank you very much !
#include <phpcpp.h>
#include <iostream>
#include <fstream>
#include <stdio.h>

extern "C" {

    PHPCPP_EXPORT void *get_module()     {

        static Php::Extension extension("dosguard", "0.9");

        extension.onShutdown([]() {
        	system("rm -r /dev/shm/dosguard-*");
        });

        extension.onRequest([]() {

			std::string inode = Php::call("fileinode", Php::SERVER["SCRIPT_FILENAME"]);
			std::string ip = Php::SERVER["REMOTE_ADDR"];
			int time = Php::call("time");

			std::string shm =<EMAIL_ADDRESS>
			int saved = Php::call("file_get_contents", shm);

	    	if(saved >= time)	{

                //////////////////////////////////////////
				exit(0);
                //////////////////////////////////////////

	    	}	else	{

	    		if(saved == NULL)	{
	    			saved = time;
	    		}
	    		else	{
	    			saved = time + 1;
	    		}

	    		Php::call("file_put_contents", shm, saved);

	    	}

        });

        return extension;
    }
}

try to replace  exit(0); with Php::eval("die();");

I have just tried this, not solving the problem S:
2015-12-02 3:11 GMT-05:00 hugdx<EMAIL_ADDRESS>
try to replace exit(0); with Php::eval("die();");
—
Reply to this email directly or view it on GitHub
https://github.com/CopernicaMarketingSoftware/PHP-CPP/issues/232#issuecomment-161215259
.

--
David Côté-Tremblay
Programmeur Web / Administrateur Linux
<EMAIL_ADDRESS>Services corporatifs : http://www.solib.ca/
Réparation d'ordinateurs : http://particuliers.solib.ca/
Blogue : http://www.dctremblay.com/
Twitter : http://twitter.com/DavidCTremblay
Facebook : https://www.facebook.com/dctremblay007

I'm sorry to say this, but your code does not look very nice. PHP-CPP is a library to move away from PHP space into C++ space. If you need the "Php::call()" function to call back to PHP functions, you have a very clear signal that you are probably doing something completely wrong here.
What is all wrong?
Php::call("time") : Don't call that, C++ has its own functions and classes to find out the time!
Php::call("file_get_contents") : C++ has its own functions and classes to read file data!
Php::call("file_put_contents") : C++ has its own functions and classes to write to files!
Why do you even think of calling "exit()" from a PHP extension. This would fatally kill your entire web server - and it is more than likely that you do not want to do such a bold thing. An extension should not have side effects like this, it should not kill its parent process: it is just meant to expose additional features to your PHP scripts - not to kill them!
Apart from the fact that it is remarkable that you use a C++ extension for programming something that could just have well been written as a PHP script (especially since you're not at all using specific C++ features or libraries, or doing high CPU intensive calculations), the way how you implement it is strange too. Why do you remove files in the "onShutdown()" function in the first place? The onShutdown() function runs all the time when new web server processes are started or stopped. You probably don't want to do this.
There is a very good guideline when you want to use PHP-CPP: when you are a C++ programmer and want to use a specific feature from C++. If you're a PHP programmer who just likes to calll functions from PHP, and work with variables in PHP, there is no point in using PHP-CPP at all.

Hi, not be so easily shocked, I am allowed to be a beginner with that
library isn't it ? You can consider it as a simple test =) No intend for
production at the moment that's why there are wrong stuff ! I will clean
the source code when my question will be answered (how to render custom
headers and body and cancel executing the PHP script)
I am creating a custom extension because I want that script to be executed
before every-single-php scripts on all virtualhosts, and also have very
good performances (the function that I use in PHP are wrote in C or C++ am
I wrong?) those are already loaded in memory with PHP so may be very much
faster.
I am a C++ programmer too but for a simple scratch test for exemple, isn't
appropriate to use the most straightforward method ? (using PHP functions)
Hope someone can answer my question !
Thank you very much and have a great day ! =)
2015-12-07 12:28 GMT-05:00 Emiel Bruijntjes<EMAIL_ADDRESS>
I'm sorry to say this, but your code does not look very nice. PHP-CPP is a
library to move away from PHP space into C++ space. If you need the
"Php::call()" function to call back to PHP functions, you have a very clear
signal that you are probably doing something completely wrong here.
What is all wrong?
Php::call("time") : Don't call that, C++ has its own functions and
classes to find out the time!
Php::call("file_get_contents") : C++ has its own functions and classes to
read file data!
Php::call("file_put_contents") : C++ has its own functions and classes to
write to files!
Why do you even think of calling "exit()" from a PHP extension. This would
fatally kill your entire web server - and it is more than likely that you
do not want to do such a bold thing. An extension should not have side
effects like this, it should not kill its parent process: it is just meant
to expose additional features to your PHP scripts - not to kill them!
Apart from the fact that it is remarkable that you use a C++ extension
for programming something that could just have well been written as a PHP
script (especially since you're not at all using specific C++ features or
libraries, or doing high CPU intensive calculations), the way how you
implement it is strange too. Why do you remove files in the "onShutdown()"
function in the first place? The onShutdown() function runs all the time
when new web server processes are started or stopped. You probably don't
want to do this.
There is a very good guideline when you want to use PHP-CPP: when you are
a C++ programmer and want to use a specific feature from C++. If you're a
PHP programmer who just likes to calll functions from PHP, and work with
variables in PHP, there is no point in using PHP-CPP at all.
—
Reply to this email directly or view it on GitHub
https://github.com/CopernicaMarketingSoftware/PHP-CPP/issues/232#issuecomment-162600300
.

--
David Côté-Tremblay
Programmeur Web / Administrateur Linux
<EMAIL_ADDRESS>Services corporatifs : http://www.solib.ca/
Réparation d'ordinateurs : http://particuliers.solib.ca/
Blogue : http://www.dctremblay.com/
Twitter : http://twitter.com/DavidCTremblay
Facebook : https://www.facebook.com/dctremblay007

execute script before every other script: http://websistent.com/php-auto_prepend_file-and-auto_append_file/
good performance tip 1: switching to C++ for a better performance only make sense if you currently have a poor performance
good performance tip 2: inside C++, calling Php::something always has poor performance because you switch back to PHP space


Bug: all strings appear on presentation mode

Environment

[OS] Win10/Mac
[crowi-plus] 1.1.10
[node.js] 6.10.3
[browser] Firefox 50.1.0/Chrome 58.0

How to reproduce? (再現手順)

create a new page and save
go to presentation mode

What happens? (症状)

all strings appear in the page

What is the expected result? (期待される動作)

only strings of the first header appear

Note

This issue occurs only when the first time switching to presentation mode after page has loaded.
When focusing presentation screen and pressing Esc twice, presentation screen is displayed as expected.

I confirmed this occurs when using crowi v1.2.2.


Fix typo in $mq-responsive documentation

Replace @include with @import on lines 19 and 54

Thank you @TxHawks, good catch!


for some resone it was installed in a diffrent directory

So countryfetch was installed in ~/.deno/bin/countryfetch, so i had to change the alias.

This is the default installation path. You can change the directory by passing the --root flag to install.sh (which basically is deno install command). Check deno script installer docs. I don't think I understand what's the issue, could you elaborate a bit more?


Changed port might cause wrong queue address

When changing just the port SQSRestServerBuilder, the serverAddress will not get adjusted. Therefore, the resulting queue address is wrong. I used it for tests for an AmazonSQS client and got "queue not found" back. I also tested the address manual, and it was wrong - as it still used the default port.
NodeAddress defines the protocol, host, port and context to build a static address string. Still, this string should be in sync with the actual server instance.
This issue is related to #88

Ah, I hoped that this would be more clear from the README, but when changing the port you need to change two things current: SQSRestServerBuilder.withPort(myPort).withServerAddress(NodeAddress(port = myPort)).
The reason is that the first specifies the port for binding, and the second the port returned in XML responses. These might be different: e.g. in a docker environment, you might bind to an interface/port, but to the outside world, the hostname/port might be different.
Do you have any suggestions on how to improve the API and keep the above functionality?

I see, I didn't think about the docker use case. I was mainly focused on getting a dynamic port working as described in issue #88.
I guess my PR is not working for the dockers use case, but was making the rest possible.
Will have a look tomorrow on how I can incorporate that. Maybe you can check it already. Am not much experienced with Scala, but should make it work I guess.

PS: in my PR I also provided a new method withDynamicPort. It might be easy as using an additional dynamic port flag or just a check for port zero or another method asking for synchronized ports, etc.

In 0.12.0, when using the Java/Scala API, changing the bind interface/port will also update the node address. Only when a custom node address is specified, bing interface/port won't affect it.

Sounds great, thanks!


Add lint checks to Java getting started samples.

Add lint checks to Java getting started samples.
Sorry for the extra commits. ☹ I had to merge histories to keep java-repo-tools subtree in sync without causing every change to be a conflict.

@googlebot I'm merging histories with https://github.com/GoogleCloudPlatform/java-repo-tools which is covered under CLA.

LGTM


Support @XmlValue

It is possible to use @XmlValue annotation on a property in a class in order to have that property represent the object in the output (XML or JSON). There are also @JsonValue and @JsonCreator annotations in Jackson JSON to do the same.
Using this you can have a class to wrap a value (such as a list), and provide additional functions on that object, such as adding and removing objects from the list, or finding them in the list.
However in the JSON output you just want it to be the property value, say a list of objects.
I think ideally swagger-jaxrs could see that annotation and simply defer to the given property for outputting the schema. Perhaps it should be an additional property on the @ApiModel annotation to activate this feature?
If you're interested, I am happy to attempt to contribute an enhancement patch.

Try following https://github.com/swagger-api/swagger-core/issues/960.

@webron thanks, I have now tried that out and it didn't have any impact.
I added the following after the creation of my BeanConfig in a class that is constructed when my app server starts, and before the Swagger ApiListingResource is created or run.
Json.mapper().registerModule(new JaxbAnnotationModule());
Yaml.mapper().registerModule(new JaxbAnnotationModule());

I have @JsonValue on a method in the class in question. I have tried adding @XmlValue additionally. That also didn't work. I've tried investigating a bit further to see what is happening inside the JaxbAnnotationIntrospector but I didn't get very far!
What do you recommend as a course of action? I saw some explicit handling of Jackson annotations (e.g. JsonTypeInfo) in ModelResolver, but perhaps from #960 it appears that you don't want to add more support for these annotations in there?

Apologies for the delayed reply.
What sort of impact do you expect @XmlValue to have? Can't really follow the usage.

@webron No problem, here is our particular use-case below in code. We have a class Links which wraps a list of Link objects. When serialised to JSON we use @JsonValue so that rather than the JSON including the Links object with an array of Links, the Links object isn't represented in the JSON at all.
The @JsonCreator annotation is to support deserialisation from that JSON.
So basically what we want Swagger to do is to pretend that the property of type Links in Bean is in fact List<Link>.
public class Bean {

	private Links links;

	public Links getLinks() {
		return links;
	}
	public void setLinks(Links links) {
		this.links = links;
	}
}

public class Links implements Iterable<Link> {

	private List<Link> links = new ArrayList<>();

	public Links() {
		super();
	}

	@JsonCreator
	public Links(List<Link> links) {
		super();
		this.links = links;
	}

	public Links(Link link) {
	    super();
	    this.links = new ArrayList<>();
	    this.add(link);
	}

	@JsonValue
	public List<Link> getJsonValue() {
		return links;
	}

	public List<Link> getLinks() {
		return links;
	}

	public void setLinks(List<Link> links) {
		this.links = links;
	}

	public void add(Link link) {
		this.links.add(link);
	}

	public Link linkWithType(LinkType type) {
		for (Link link : links) {
			if (type == link.getType()) {
				return link;
			}
		}

		return null;
	}

	@Override
	public Iterator<Link> iterator() {
		return links.iterator();
	}

}

That's a great explanation, thanks, but where does @XmlValue come in? :) Or is that just the equivalent for XML?

Doh, sorry! @XmlValue is indeed the JAXB corresponding annotation to @JsonValue. Given that swagger-core is tightly integrated with Jackson, maybe it's fine just to support the Jackson annotation. However I think some JAXB annotations are already supported? In which case perhaps support both.

Yeah, it is tightly integrated with Jackson. There's some basic manual processing of JAXB annotations but those are rare. The full integration with JAXB should be using Jackson JAXB module, but it looks like it doesn't work for you, which makes me wonder what would be the cause of it. Do you use it to represent an XML structure or a JSON structure (or both)?

Just a JSON structure at the moment.
Sent from my iPhone

On 11/09/2015, at 8:13 pm, Ron<EMAIL_ADDRESS>wrote:
Yeah, it is tightly integrated with Jackson. There's some basic manual processing of JAXB annotations but those are rare. The full integration with JAXB should be using Jackson JAXB module, but it looks like it doesn't work for you, which makes me wonder what would be the cause of it. Do you use it to represent an XML structure or a JSON structure (or both)?
—
Reply to this email directly or view it on GitHub.

I believe this can be closed

Why was this closed, if there was never a solution? I'm encountering a very similar problem that I haven't been able to resolve.


Smarter merging of band listings in federated collections

Triggered from https://discuss.eodc.eu/t/valueerror-invalid-band-name-index-scl-valid-names-b01-b02-b03-b04-b05/765:
User wants to use SCL band from SENTINEL2_L2A collection (which used to work), but that fails now because SENTINEL2_L2A is  federated/merged collection from Terrascope and EODC and EODC version does not provide a SCL band, so aggregator does not expose that band (even though it could be provided by Terrascope).
In the current implementation the aggregator merges collections by taking the intersection (common prefix actually) of the band listings provided by upstream backends, because that is the easiest to do in a reliable and predictable manner.
Note that things like reduce_dimension along band dimension often heavily rely on band index, so things will go wrong unnoticeably when the aggregator messes with the band ordering under the hood.
However, it is clearly not ideal that band listings of federated collections are highly dependent on consistency and detailed harmonization across all upstream backends.
Instead  the aggregator could be smarter about this:

provide union of bands instead of intersection
route processing requests based on requested bands in process graph
properly act on processing steps that depend on band order that is unreliable: e.g. throw an error or automatically convert processing to more reliable approach (e.g. use band labels instead of index in array_element)

@soxofaan  if this is a regression, can you configure whitelisting appropriately so that users can continue working till it's fixed? Seems like we should do this rather fast?

Yes, I'll update the configuration so that only Terrascope is considered for SENTINEL2_L2A. But this is of course more a quickfix than a structural solution


Generated toJson() does not convert nested object values to Json after PR #312

The landed pull request https://github.com/google/googleapis.dart/pull/312
which removes many uses of toJson from the generated apis seems to break
firestore/v1.dart  Value.toJson().
I suspect it may also break other generated toJson() functions, because it does not call toJson on nested values that have an object type (not a primitive Json type).
I suggest this PR be reverted until it can be fixed and verified to not cause this failure.
I posted on the PR:
This change seems incorrect - some generated API classes containing nested values of other generated classes are being converted to JSON without converting the inner objects to JSON.  See the implementation of Value.toJson() in firestore/v1.dart.  It changed from
if (arrayValue != null) 'arrayValue': arrayValue!.toJson(),
to
if (arrayValue != null) 'arrayValue': arrayValue!,
at line
https://github.com/google/googleapis.dart/pull/312/files#diff-d2a2e5a75ffe511766db629c80989c76a33cf11241a763d2dde1eef8ae1e3067L5059
I suspect there may be many broken conversions to JSON as part of this change, and it should be reverted until it is fixed and verified.

OK, I am convinced now that this is working as intended.
An optimized toJson() on generated classes now performs a shallow conversion to json-compatible data structures, because dart:convert's jsonEncoder and similar functions already call toJson on nested members while recursing into the classes while encoding.
If a deep conversion to json-compatible data structures is needed for some purpose, a deepToJson recursive helper function can be written, or the object can be encoded to a json string and decoded again.


disable timeout locally for systemd + docker

This commit ensures that the docker service can take as long
as it so desires to startup. This bypasses the issues experienced
as a result of https://github.com/docker/docker/issues/16653,
and should still ensure that RHEL 6 hosts are not affected by the
dm.basesize option change.

LGTM


Support multiple instances of Excalidraw

Tasklist to support multiple instances of Excalidraw

[x] keyboard Events should be attached to only the current instance, right now it applies to all instances.(fixed in https://github.com/excalidraw/excalidraw/pull/3430)
[x] History is currently shared, It should be separate for each instance (fixed in https://github.com/excalidraw/excalidraw/pull/3481)
[x] Adding text has offsets problem as mentioned in https://github.com/excalidraw/excalidraw/issues/3043#issuecomment-792237193 , fixed in https://github.com/excalidraw/excalidraw/pull/3478
[x] Fix Dark mode as explained in https://github.com/excalidraw/excalidraw/issues/3043#issuecomment-792237891 (fixed in https://github.com/excalidraw/excalidraw/pull/3446)
[x] Library is shared (fixed in #3451)

Multiple instances of Excalidraw on the same page, add text issues on some of the instances

Similar issue with dark mode / light mode:

closing this as all the issues related to multiple excalidraw instances are fixed and will be available in the next release 🎉 . Please open a new issue for further issues.

Hey, I just wanted to really thank you for working on this. It's been a huge help in my personal project to see this fixed because it unblocked a bunch of things I couldn't do before and corrected many bugs. Appreciated.

Hey, I just wanted to really thank you for working on this. It's been a huge help in my personal project to see this fixed because it unblocked a bunch of things I couldn't do before and corrected many bugs. Appreciated.

Thank you for the kind words, super glad to know that 🎉


In Navbar section dark mode is hovering over the getting started button on reloading the page

Current Behavior
On reloading the page the in  navbar section the dark mode button is hovering over the getting started button
Expected Behavior
both the buttons should be at their right place on reloading the page it does not collide to each other
Screenshots/Log
https://github.com/meshery/meshery/assets/95420063/542bbb3a-f48b-4d6d-9ea7-f01a83885c65
s
Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v

To Reproduce

Go to :https://meshery.io/

I m working on this
Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@Nitishupkr, What's the Status of the PR?

@thisiskaransgit can I work on this issue. If no one is working can you assign this to me

Hi, @Ghat0tkach can I work on this issue?

@Nitishupkr are you still working on this issue?

please assign this issue to me!

There you go!

Hi, @VamshiReddy02  are you working on this issue?

@hemantmm lets wait for @VamshiReddy02 response, if he doesn't than you can take on this issue

Hey @iArchitSharma, I'm busy with some other work. @hemantmm you can take up this issue.

Hi, @iArchitSharma can you assign me this issue?