Feature/Buffer Memory SessionId

Existing buffer memory nodes are just in-memory, data will be wiped once app restarted. Hence it requires user to pass in history array when using API.
This PR improve this scenario.

All buffer memory nodes now retrieve chat messages from database using sessionId
Add conversation summary buffer memory

Related issue - https://github.com/FlowiseAI/Flowise/issues/1666

Hi @HenryHengZJ , how do we use this in the API or we do not need to have additional coding to specify the sessionId?

Hi @HenryHengZJ , how do we use this in the API or we do not need to have additional coding to specify the sessionId?

@VincentEmmanuel you just need to specify the sessionId in overrideConfig https://docs.flowiseai.com/integrations/langchain/memory


Add useApplications to the UI exports

What changed?
The UI is also exporting the useApplications hook and LoadingPage now.
Updated Loader style to match what we do in WE in terms of wrapping content.

Why?
The export is needed in WE to have access to the number of applications.
The LoadingPage will be used in WE instead of what we have atm to ensure consistency.

Sorry @jpellizzari realised today that there's a loading page that comes with the apps which looks a bit different to what we have. I've made some adjustments and we'll use this one (added to the exports as well). Thanks!


Question: how to make bottomSheet to peek->expand to full screen->scrollable

I'd like to be able to do something similar to this:
http://i.stack.imgur.com/3czMy.gif
I think it also needs to tell me how much it got expanded, so that I could animate stuff as it expands.

Well, @AndroidDeveloperLB, Looking at the gif, it seems to me that the bottom sheet expands to fill the screen (on expand), but the bottom sheet is composed of a Collapsing toolbar and Appbar.
To get how much the bottom sheet has expanded, in your callback's onSlide() method, use the slideOffset, or get the bottomSheet's position in window (locationInWindow()).

Can you please add this sample here (of course, without all the graphics ... :)  ) ?


[Docs feature] Add "Copy to clipboard" button on code samples

Hey! 👋
I was going through the docs today and thought that maybe adding a "Copy to clipboard" button on the code samples could make it easier for people to go through the "Get started" tutorials! 🙂
People can still manually copy and paste if they want but personally, I sometimes think I copied a whole code sample and actually I didn't so I thought a "Copy" button could be helpful.
Here's a quick demo:

Example in the deploy preview
I'm not super familiar with Docusaurus so I just followed the code sample in this doc.
No worries at all if you think that's not a good idea! 🙂

Hey @charliegerard!
This is so awesome, thank you so much!
Shipping it 🚀


Need to have ability to configure dataloader

Hi, based on the example here, I've set up the following Spring config in the context of a Spring Boot app levering the present project:
@Configuration
@Slf4j
public class DataLoaderConfig {

    @Autowired
    private Dao<Author>    authorDao;

    @Autowired
    private Dao<Publisher> publisherDao;

    @Autowired
    private Dao<Book>      bookDao;

    @Bean
    public Instrumentation instrumentation() {
        log.info("Creating GraphQL execution instrumentation");

        final AuthorBatchLoader    authorBatchLoader    = new AuthorBatchLoader(authorDao);
        final BookBatchLoader      bookBatchLoader      = new BookBatchLoader(bookDao);
        final PublisherBatchLoader publisherBatchLoader = new PublisherBatchLoader(publisherDao);

        final DataLoader<Integer, Author>    authorDataLoader    = new DataLoader<>(authorBatchLoader);
        final DataLoader<Integer, Book>      bookDataLoader      = new DataLoader<>(bookBatchLoader);
        final DataLoader<Integer, Publisher> publisherDataLoader = new DataLoader<>(publisherBatchLoader);

        final DataLoaderRegistry registry = new DataLoaderRegistry();
        registry.register("authors", authorDataLoader);
        registry.register("books", bookDataLoader);
        registry.register("publishers", publisherDataLoader);

        return new DataLoaderDispatcherInstrumentation(registry);
    }
}

And BTW, I'm using the resolver support, as in the following:
@RequiredArgsConstructor
public class AuthorResolver implements GraphQLResolver<Author> {

    private final Dao<Author> authors;

    public Author author(int id) {
        return authors.findById(id);
    }

    public List<Author> authors(int index, int maxResults) {
        return authors.paginate(index, maxResults);
    }

}

There seems to be a bit of abstraction involved, and I'm uncertain about how resolvers correspond to data fetchers (I suspecting that my resolvers are invoked by some generic that fetcher wrapping them). So I'm not clear as to how this type of abstraction may play into the data loading I've configured being used or not...
At runtime, I can see the the SimpleGraphQLServlet instance uses the DataLoaderDispatcherInstrumentation created above (added traces that show the servlet holding the instrumentation instance), but the instrumentation is not being used for loading. Not sure how to wire things up in the context of the Spring Boot starter so that Dataloader is used, there's no doc I could find on the topic. It would be very useful to be able to benefit from the Dataloader support when using the Spring Boot starter.
Can you provide steps to move forward? Thanks.

Would love to have first-class support for DataLoaders, as well, since they're the only non-deprecated way to do batching.

Closing, since support for this has been added in the meantime. See also https://github.com/graphql-java-kickstart/graphql-spring-boot/tree/master/example-request-scoped-dataloader.

This example is perfect, just one thing it needs more is to explain on how additional parameters received in a Resolver would be passed to the DataLoader.
Consider the example schema
type Query {
   products(productId: [Int]!): [Product]
}

type Product {
    productId: Int
    name: String
    transactions(fromDate: String, toDate: String): [Transaction]
}

type Transaction {
    transactionId: String
    value: Float
    quantity: Int
}

and the query that I would want to send is to get all transactions that have happened between 20-MAR and 22-MAR for products 3,5 & 7.
products ( productId: [3,5,7] ){
    productId
    name
    transactions(fromDate:"20-MAR-2020", toDate: "22-MAR-2020"){
        transactionId
        quantity
    }
}

Using DataLoader I am able to reduce the trips to the transaction data source(db, api)
I don't do 3 trips, but only one with all 3 product Ids passed as the List.
However the DataLoader, BatchLoader does not have provision to pass the parameters fromDate & toDate.

Closing, since support for this has been added in the meantime. See also https://github.com/graphql-java-kickstart/graphql-spring-boot/tree/master/example-request-scoped-dataloader.

Hi @oliemansm How I can use CustomGraphQLContextBuilder  for multiple registry


Height doesn't adjust for each month with wrap_content on horizontal paged calendar

Steps to reproduce

I've added calendar view with wrap_content

<com.kizitonwose.calendarview.CalendarView
            android:id="@+id/cv_custom_calendar"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:layout_marginStart="8dp"
            android:layout_marginEnd="8dp"
            app:cv_dayViewResource="@layout/calendar_day"
            app:cv_hasBoundaries="true"
            app:cv_outDateStyle="none"
            app:cv_monthHeaderResource="@layout/calendar_header"
            app:cv_orientation="horizontal"
            app:cv_scrollMode="paged"
            app:layout_constraintEnd_toEndOf="parent"
            app:layout_constraintStart_toStartOf="parent"
            app:layout_constraintTop_toBottomOf="@+id/tv_month_year" />

Open fragment for the first time, scroll to the previous month and height of the CalendarView stays the same even though the number of days and rows increased
Open the same fragment again, resize works.

Note:
Is this an edge case because February starts on Monday with 28 days and it has 4 rows?
With fixed android:layout_height it shows all dates, but doesn't look nice with extra space at the bottom. Also, I can't find the right height that will show all days for every screen size.
Is there a possibility to have fixed size and spacing between rows to be auto-adjusted?
Device:
Samsung S8,
Samsung A5 2017
Video:
https://vimeo.com/user133404315/review/511093300/5daa85a221

Video: vimeo.com/user133404315/review/511093300/5daa85a221

This link is not working.

Is there a possibility to have fixed size and spacing between rows to be auto-adjusted?

I do not understand, could you explain a bit more?

@kizitonwose thanks for the reply.
Can you see this video? https://drive.google.com/file/d/11p5Bm1wIHH9PUsaAtnryVOL4nOX-qUVg/view?usp=sharing
I want to have CalendarView that will adjust its height based on the number of rows. That doesn't work on the video that I sent you. Do you know what could be the reason for this bug? Do you have some suggestions that I could try?
In case that you don't have suggestions for the problem mentioned above, I could try a workaround if you know the answer for that.
Create CalendarView that will have fixed height. Based on that fixed height CalendarView would adjust its row spacing.

Ok, that's not possible. The calendar adjusts its height from the bottom when a month has fewer rows than the previous one. The spreading behavior in the second video looks rather odd to me, but not possible anyways.

The library uses a linear layout internally, so if you really want this, willing to fork the project, and the calendar has a fixed height, then in the binding logic, you'd just divide the remaining bottom space by the number of visible rows and set it as a divider on the linear layout.

Hi @kizitonwose, sorry for the confusion, please don't focus on my second question about the spreading of rows. It was just an alternative that I mentioned if we don't solve the first problem.
Focus on the problem in the first video where you can see that I can't see all the rows because they are cut off. CalednarView is not resized automatically with wrap_content so it shows all rows.

Ok, please check with version 1.0.2 which was released yesterday, this issue is likely fixed.

Hey @kizitonwose, thanks for keeping the library up to date.
I now have a different problem, and I think it's similar to issue #283
Take a look at a video, you can see that some days are missing.
https://drive.google.com/file/d/1_D8Z81k-GF8zecjNH78AxBvMIdclg_n1/view?usp=sharing

Are you reloading the calendar every time it scrolls to a new month?

Yes, I call cv_custom_calendar.notifyCalendarChanged()

Why?

Every time I scroll I have to pull data from the server, after that I have to show that data on the calendar for that month.
In bind method I keep logic that will render a calendar based on that list from the server, so I have to recall that.
I tested now without calling notifyCalendarChanged() and the same problem is still happening. Some days start to be invisible after scrolling back to that month.

Hi @kizitonwose, a reminder in case you missed my last comment.  Do you have an idea what could go wrong, maybe we can take a look at the library together to find the source of the bug and to fix it?

Are you sure this is not an issue with your binding logic? If you can set up one of the examples to reproduce this, then I can take a look directly.

@kizitonwose You solved the first problem with the library, the second one with disappearing dates was on my side in binding logic, it's fixed now.
Thanks for your help!


Update API reference docs for TaskRun

Changes
This commit adds missing optional fields to "Configuring a TaskRun".
Other than these changes, the TaskRun documentation looks up to date. Closes #4656.
/kind documentation
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Docs included if any changes are user facing
n/a Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
NONE

thank you @lbernick 👍
/lgtm

/test pull-tekton-pipeline-alpha-integration-tests


Pretty printer for AST

Encoder is lovely, but it puts everything on one line. For debugging, it would be nice to have a pretty printer.

Yes, please! :pray: Otherwise, how do we write integration tests?


New SlackWebookHandler and long context/extras problem

Hi,
I just got around to test the new cool SlackWebhookHandler. It works nice and fixes problems I had before, there is however one annoyance I found. @an1zhegorodov I didn't want to continue discussion in your original PR though.
I use attachments and include context and extra data. Depending on the length of the context, it's either formatted well or cut:
Looking good:
$nestedArray = [];
for ($i = 0; $i < 27; $i++) {
    $nestedArray["foo $i"] = str_repeat($i, $i * 2);
}
$context = ['nested' => $nestedArray];
$this->logger->warning('Warning test', $context);

the whole nested array is displayed in code block (using "```").
If I, however, make it bigger:
$nestedArray = [];
for ($i = 0; $i < 30; $i++) {
    $nestedArray["foo $i"] = str_repeat($i, $i * 2);
}
$context = ['nested' => $nestedArray];
$this->logger->warning('Warning test', $context);

The context output is truncated and the whole code block is broken. Below, it's how it looks like at the beginning of the msg (I don't include full msg cause it's big):

You can see at the beginning that there is no closing "```" of code block (probably because truncation). I don't have time to debug this now, it could be as well a limitation of Slack Webhook formatting.
What I'm trying to do:
I want to include a stacktrace (not based on exception) in a context so that it's prettified and displayed in code block on Slack. The NormalizerFormatter and "stringification" get the job done, but only for small contexts.

AFAIK there are no limits when using the normal message field instead of attachments, which is the weirdest thing ever...

Maybe you could truncate the output by hand to make sure the backticks is present at the end, or at least report it to slack so that they fix their truncation to not rip apart a code block..

@Seldaek I hope by you, you mean me - the user of the library. Then - yes, I could, I already do that since I found out about the limitation.
I didn't have time to look into it but I'll get around to reporting it someday ;)

@mirfilip By you I mean anyone running into this issue that feels like fixing it in monolog, or in slack :)

Because I had to deal with the same issue I have created this package. You can find it also in wiki of monolog.


Fineract 614: Rates module

Previously defined sub-rates can be added to set the nominal interest rate of a loan product. When a loan account is created you can choose which rates from the associated rates to the loan product should be applied to the new loan account. This is an optional way of defining the nominal rate for loans.

@angelboxes thank you very much for your interest in and contribution to Fineract/Mifos! I'm an (old) volunteer who got reminded of Mifos/Fineract this week-end at the FOSDEM conference, and I'm taking a moment to help the project clean up some of it's old issues and pull requests etc.
We were wondering whether you are still alive, and would be willing to git rebase develop this PR and take any previous review feedback into account? If we don't hear from you within 2 weeks, we will take the liberty to close this PR without merging it, to avoid confusion for future new contributors looking at this repository - hope you understand and that's fair.
/Cc FYI @apache/fineract-committers

Hi, I pulled the latest changes from the dev repository and rebased my branch.

@ShruthiRajaram will you re-review this and merge if OK for you? (From a quick look I've just had, it seems to me that @angelboxes has addressed your review comments, but you'll want to have a closer look I guess.)

@angelboxes would you mind to re-rebase this, so that Travis CI will run on it?

@angelboxes would you mind to re-rebase this, so that Travis CI will run on it?

I just rebased my brach with the latest changes from develop

@angelboxes did you notice that the (new) Travis CI build failed? Note:
Building 95% > :integrationTest.googlecode.flyway.core.api.FlywayException: Found more than one migration with version '350' (Offenders: SQL '/home/travis/build/apache/fineract/build/tmp/tomcatRunWar/work/Tomcat/localhost/fineract-provider/WEB-INF/classes/sql/migrations/core_db/V350__rates.sql' and SQL '/home/travis/build/apache/fineract/build/tmp/tomcatRunWar/work/Tomcat/localhost/fineract-provider/WEB-INF/classes/sql/migrations/core_db/V350__email_from.sql');

SQL script needs to be re-numbered

Hi, I didn't noticed there was a more recent Travis build, I've changed the name of the script and rebased my branch. I will keep an eye on the travis build result if it fails again.

OK for me now! @ShruthiRajaram ok for you as well as-is - merge?

@vorburger , we had a call with this team last year. And we requested some change in functionality. So I prefer Ed Cable or Santosh Math to review the functionality before merging it. I have informed Santosh already and once they confirm we can merge this

@ShruthiRajaram my question was just if the points which YOU (earlier, above) raised are now adressed? Voice Calls without any public written record here and no concrete concrete code review feedback which the contributor could react to and rectify, and mention of (nice!) other people who have not commented on this PR in 7 months (!) isn't really how the open source is supposed to work... ;-) Unless you (@ShruthiRajaram) have strong objections, I am inclined to merge this. My main goal isn't this particular functionality, but wanted to encourage contributors like @angelboxes who clearly are reactive to concentrate feedback here - we should not discourage and demotivate such contributions by taking SOOO long...

Hey @vishwasbabu , Please assign @angelboxes to this issue on JIRA: https://issues.apache.org/jira/projects/FINERACT/issues/FINERACT-614
He is the reporter on the issue.
I tried doing that but for some reason It was failing. Thanks

Hi @angelboxes, Can we have one quick functionality demo regarding the ticket?

I am unable to select him from the typeahead . I shall reach out to infra
for help. Meanwhile , can @angelboxes try assigning the issue to himself ?
On Friday, March 8, 2019, Awasum Yannick<EMAIL_ADDRESS>wrote:

Hey @vishwasbabu https://github.com/vishwasbabu , Please assign
@angelboxes https://github.com/angelboxes to this issue on JIRA:
https://issues.apache.org/jira/projects/FINERACT/issues/FINERACT-614
He is the reporter on the issue.
I tried doing that but for some reason It was failing. Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/fineract/pull/465#issuecomment-470847270, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAhH6YLI6j6d_KaW6Cdh6n6-El-gZhESks5vUh9agaJpZM4VnVLM
.

--
Sent from my iPad

That will be a good way to fix it. @angelboxes , please assign the issue to yourself on JIRA.
@angelboxes I appreciate your patience on this issue, you are doing great work.

@santoshconflux Why do you need a call or demo? Will pulling this code in and testing the feature on your environment not validate if the issue has been solved?
Also if there needs to be a demo, does it mean a UI is needed because that will mean we accepting this PR only if there is a UI for it already given Fineract does not have a UI. Do you want him to show you a postman call to an endpoint so you see the response. It looks like the issue is not well understood.
If that is the case, then we need to create a confluence page or this or update the JIRA ticket and even discuss on the mailing list. I think we can merge this code and do the UI tests and confluence pages later given @angelboxes has been so patient with us.

@awasum , We know @angelboxes did great including this feature.   Since  UI is not yet developed in Community app level and we still have open ticket https://github.com/openMF/community-app/issues/2997  which GSOC student can work upon.
All we can check it with Postman only.
@angelboxes can walk me through Postman or share some use cases. We had demo long time back and in order to just refresh our memory I asked for quick demo , not detailed one, so that I can figure out whether it could create any regressions.
We should not forget that loan 'product' is very base of whole loan module and any new inclusions in the loan product demands to test whole or at least to some extent of loan module thoroughly for regressions.

@vishwasbabu , @ShruthiRajaram , if this PR is merged and since it's UI version is not yet implemented, https://github.com/openMF/community-app/issues/2997
Do you think it won't be any problem for existing functionality  if I fetch latest code from develop branch of Community App to work with latest code from develop branch of Fineract?

Santosh ,  I haven’t reviewed the code .
-> The code dosen’t contain integration tests , so we cannot determine if
it actually works without manually testing it out .
-> the code does not break any existing integrations tests , so , we can
tell that the most common loan scenarios do not break
-> The integration tests themselves have very low code coverage, so the
possibility that it might break something does exist (we have tickets on
Jira for improving code coverage and hope to make some progress on the same
in this GSOC)
-> To my knowledge, there isn’t any API breaking change introduced in the
develop branch since 1.2 release , so the community-app should continue to
work.
Apologies if my responses aren’t particularly helpful.
On Friday, March 8, 2019, Santosh Math<EMAIL_ADDRESS>wrote:

@vishwasbabu https://github.com/vishwasbabu , @ShruthiRajaram
https://github.com/ShruthiRajaram , if this PR is merged and since it's
UI version is not yet implemented, openMF/community-app#2997
https://github.com/openMF/community-app/issues/2997
Do you think it won't be any problem for existing functionality if I fetch
latest code from develop branch of Community App to work with latest code
from develop branch of Fineract?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/fineract/pull/465#issuecomment-470906245, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAhH6QtJyub7-o1tu6R_Ce8TbeJylp88ks5vUlHFgaJpZM4VnVLM
.

--
Sent from my iPad

@vishwasbabu those replies were very helpful and clearly establishes the need for either a demo or for @santoshconflux to merge the UI code and manually test the functionality from his end:
@angelboxes
The purpose of the demo is to demonstrate end to end the rates module work you've done since we don't have full integration test coverage in place. Santosh wanted to see a demo or manually test the code via the UI locally to ensure no regressions were introduced.

@awasum Hi, I'm trying to assign the ticket to myself in Jira but I don't seem to find an option to do it.
@santoshconflux This feature was designed for us as a way to configure multiple rates to a loan product, since our nominal interest rate is actually composed of two different rates, we needed an option for people that configure the loan products could choose between rates that were previously defined in the rates configuration module as the nominal interest rate used for the product. It just affects the UI as the rates chosen are added in the nominal interest rate field.
About the demo, we'll work on it so you can test the changes by yourself and give us better feedback, we actually have a dev environment but we are not allowed to share it.
Thanks for all your  comments

Hey y'all as per our discussion here<EMAIL_ADDRESS>I've accepted angelboxes PR as an unmerged branch with the name FINERACT-614.  In the process, I resolved the merge conflict.  I have not resolved the findbugs warnings there.  Unfortunately github doesn't seem to automatically bring this (amazing!) conversation along for the ride, so those of you with outstanding comments will have to take them to the dev list.  Consider tagging your subject lines with [FINERACT-614] to make them easier to filter and find.
@angelboxes you should be able to continue working on your branch, but there's no need to edit your monocommit any longer.  Just add new commits for the adjustments you'd like to make, and continuously send PRs to this repo.  We will be able to accept most, perhaps even all of them now.  When you are done and everyone here is happy we can merge the branch to develop.


python312Packages.distutils: unbreak on Darwin

see https://github.com/NixOS/nixpkgs/issues/356030#issuecomment-2482641338

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Can be good to go if tested.

I successfully built it on aarch64-darwin but I want to wait for @ofborg before merging.


Signature Error

I tried to generate an invoice using the examples/full.ts provided in the git but from yesterday onwards the following error has been shown when trying to TestXML in https://sandbox.zatca.gov.sa/TestXML
category : SIGNATURE_ERROR code :X509IssuerName message : wrong X509IssuerName   category : SIGNATURE_ERROR code :X509SerialNumber message : wrong X509SerialNumber

Look at previous issues.
For ZATCA Web portal to work you need to not generate certificates. Use the Java SDK provided certificates. Otherwise use APIs.


Refactor autoagora

Successful test in prod

Regarding the comment I just added extra assertion where it checks if inside the model the queries can be found, since in other tests. Could make a literal check if the whole model looks good but I think it would be too much. Since it would be having there all the model ina variable, if you consider its better then sure could add it in the constant file for tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 84.444%

Files with Coverage Reduction
New Missed Lines
%

autoagora/main.py
1
71.93%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
342

Relevant Lines:
405

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

15 of 15 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 84.807%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
307

Relevant Lines:
362

💛  - Coveralls


What are the specific label IDs used in the PanNuke dataset masks?

I am working with the PanNuke dataset and trying to understand the label IDs used in the masks. Could you provide clarification on what each label ID corresponds to in the dataset's masks?

Are PanNuke and GlaS the only pathology datasets used? Why are there more pathology data mentioned in the paper?

All mask corresponds to the name of the target. The targets in our processed datasets are: neoplastic cells, epithelial cells, inflammatory cells, connective tissue cells. If you are working on the original version of PanNuke, you can find in the description to match the target names to the ID number.
PanNuke and GlaS are the datasets used for model training and evaluation. We curated on additional pathology dataset, but due to data size and quality we didn't use that for model development, which was described in Supp Table 1. We are currently holding those per request.

The PanNuke dataset webpage is currently inaccessible. Is there any alternative source for the dataset description?

You can check out this one. https://www.kaggle.com/datasets/andrewmvd/cancer-inst-segmentation-and-classification

We processed the dataset from https://www.kaggle.com/datasets/andrewmvd/cancer-inst-segmentation-and-classification, together with Part 2 and 3. The label IDs corresponding to each class are: {0: 'Neoplastic cells', 1: 'Inflammatory', 2: 'Connective/Soft tissue cells', 3: 'Dead Cells', 4: 'Epithelial'}. We used all the images from the source dataset but the dead cell annotations.
Hope this helps!

Have you merged the labels of the oriHave you merged the labels of the original PanNuke dataset? Is the released data the processed complete dataset or just a part of it?ginal PanNuke dataset? Is the released data the processed complete dataset or just a part of it?
Thank you!

We processed the full dataset from the Kaggle version.

Is the PanNuke dataset released in your repository the processed version of all the data, or is it only a portion of the processed data?
Thank you！

It is the processed version of all the data.


BUG pick up config file from project root

Currently, if you run nbqa isort it'll only pick up a .nbqa.ini file if it's in the current working directory. It should be able to pick it from the project root, regardless of where nbqa is being run from

Just traverse up the path till you find a matching file.
It could look something like this:
def get_config_path(config_file_name:str=".nbqa.ini"):
    current_path = Path(os.getcwd()).resolve()
    # recurse up until you are in the root dir
    while current_path != current_path.parent:
        config_file = current_path / config_file_name
        if config_file.is_file():
            return config_file
        current_path = current_path.parent

Some care is needed though, because the config file specifies other config files (e.g. setup.cfg) relative to the repository root

Maybe the art student approach ("copy from the masters") would fit here.
Just having a peek how flake8 and so on do it ^^

Sure, thanks - I almost have a PR ready though ("inspired" by how black do it), so don't waste too much time on this :smile:


Doctrine DBAL 2.5 Breaks Symfony if DB Doesn't Exist

This is a problem with the most recent symfony-standard.
With the newest Doctrine DBAL 2.5, Doctrine introduced Doctrine\DBAL\Connection::detectDatabasePlatform() which is different than 2.4. Now in 2.5, any time a repository is instantiated (not even used) by the DI, we get a PDOException if the DB doesn't already exist.
Imagine trying to run app/console doctrine:database:create and getting a "Database doesn't exist!" error. Repositories as services are very common in most projects, so this is a huge issue.
To reproduce quickly, create a fresh Symfony project, letting composer install the AcmeDemoBundle. Use all default parameters. Now try to generate an entity. You'll get a PDOException. This is a new regression.
composer create-project symfony/framework-standard-edition symfony-standard
cd symfony-standard
app/console doctrine:generate:entity --entity=AcmeDemoBundle:Foo

You get error this on every OS/distro/platform:
  [Doctrine\DBAL\Exception\ConnectionException]
  An exception occured in driver: SQLSTATE[HY000] [1049] Unknown database 'symfony'

  [Doctrine\DBAL\Driver\PDOException]
  SQLSTATE[HY000] [1049] Unknown database 'symfony'

  [PDOException]
  SQLSTATE[HY000] [1049] Unknown database 'symfony'

is solved in DBAL 2.5.1?

no. Read the release announcement. They explicitly mention that they don't have an appropriate fix yet

thanks @stof still waiting then

Just wanted mention this issue still exists and the pull request associated with the above closed ticket did not get accepted, so the issue is still present. Thank you guys for opening the issue and bringing it to the attention of the appropriate devs. Found the page as I also was experiencing the same.

Hi all. Has the issue already been resolved or is there a workaround. Actually I'm following a tutorial with and facing the same error...

See https://github.com/doctrine/DoctrineBundle/issues/351#issuecomment-65771528 for the workaround

Ok, thank you !!!

Closing as the issue is resolved. I'm not getting that error anymore when using the latest versions.
I guess the underlying problem is still open http://www.doctrine-project.org/jira/browse/DBAL-1057 but that doesn't affect the use-case of this ticket anymore.

@Tobion Currently, we explicitly forbid doctrine/dbal 2.5.x because of that issue. I suggest to keep this open as a reminder to allow it again once the referenced issue has been solved.

I think we can allow dbal 2.5 again now. The common case to use repositories as services doesn't cause any problems anymore. DBAL-1057 might only affect some edge cases.

Which explicit version should be avoided then?

I think it's a combination of DBAL and ORM. The following should be fine

DBAL 2.5 with ORM 2.5
DBAL 2.5 with ORM >= 2.4.8 (which has not been release yet, it requires https://github.com/doctrine/doctrine2/commit/e05930e71482df06982c332b8820a1ae51bd7f28 to be tagged I guess)

It merits some testing, but I agree with Tobion - last time I checked, I couldn't repeat the issue. It would be great to not restrict 2.5 again.

See #800


concat operator optimization

It seems the current implementation of concat operator is based on mshadow. And if the input of concat has multiple NDArray, on gpu, it will launch kernel for many times. Tensorflow has customized kernel for concat operator, it will do kernel launch only once.
Any plan to optimize this?

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!
Also, do please check out our forum (and Chinese version) for general "how-to" questions.


ox.add_edge_grades() fails when network has zero length edges

I was using the add_edge_grades() function and got the division by zero error.

ZeroDivisionError                         Traceback (most recent call last)
<ipython-input-165-f0944a5b715b> in <module>()
      4 # Get the directed grade (ie, rise over run)--slope for each edge in the network and add it to the edge as an attribute.
      5 # keeps downhil change negative
----> 6 bx_streets = ox.add_edge_grades(bx_streets, add_absolute=False)

~/anaconda/envs/gep662/lib/python3.6/site-packages/osmnx/elevation.py in add_edge_grades(G, add_absolute)
    114     for u, v, data in G.edges(keys=False, data=True):
    115         elevation_change = G.nodes[v]['elevation'] - G.nodes[u]['elevation']
--> 116         grade = elevation_change / data['length']
    117         data['grade'] = grade
    118         if add_absolute:

ZeroDivisionError: float division by zero

It turned out that the network had few edges with zero length attribute; I identified and removed them. It would be helpful to incorporate the workaround for such situations into the add_edge_grades() function, maybe as an additional parameter like drop_zero_len=True
The network I used was for the Bronx using this line of code:bx_streets = ox.graph_from_place('Bronx, New York, USA', network_type='bike'). Not sure how common zero length edges in other networks.

@anastasiaclark that's interesting. Do you have a sense of why those edges had length zero? What kind of edges were they (ie, what was the highway type)? How many were there?

The network type is 'bike'. There were four edges with zero length; all in the same location. I mapped them over the OSM and they appear to be very tiny. Attached is the code that reproduces the error and examines the zero length edges.
Archive.zip

I just ran the code you uploaded and I'm not getting any errors.
import geopandas as gpd
import osmnx as ox
ox.config(use_cache=True, log_console=True)
bx = ox.graph_from_place('Bronx, New York, USA', network_type='bike')
bx = ox.add_node_elevations(bx, api_key='YOUR-KEY-HERE')
bx = ox.add_edge_grades(bx, add_absolute=False)

That all runs fine. What version of OSMnx are you using? Make sure you're upgraded to the latest. If you keep getting the error, please post your full python environment info, all package versions, and the full code to reproduce the error.

 Updating to the latest version (from 0.6 to 0.8) of osmnx fixed the error!

G = ox.graph_from_bbox(40.68, 40.19, -79.68, -80.37, network_type='drive', simplify=False)
G = ox.elevation.add_node_elevations(G, elevAPIkey, max_locations_per_batch=512, pause_duration=0.02)
G = ox.elevation.add_edge_grades(G, add_absolute=True)

ZeroDivisionError: float division by zero

Rtree-0.9.3 geopandas-0.6.2 networkx-2.4 osmnx-0.11


[BUG]: 'docker' not found in agent version 3.218.0

What happened?
##[error]Unhandled: Unable to locate executable file: 'docker'. Please verify either the file path exists or the file can be found within a directory specified by the PATH environment variable. Also verify the file has a valid extension for an executable file.
2.217.2 agent version build worked ok. Same Docker@2 version as the one that failed: 2.214.0
Our agent pool updated automatically to 3.218.0 today, and my theory is that this is what broke this.
This is breaking our windows pipelines.
Versions
Windows-2022 3.218.0
Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[X] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Windows 2022
Version controll system
No response
Relevant log output
No response

Duplicate: https://github.com/microsoft/azure-pipelines-tasks/issues/18039


Ability to lock or not as easily move a pool?

The way pools are moved as of 1.6.0 mean that whenever you are zoomed in on a process and you need to move on the canvas, you end up moving the pool rather than transversing the canvas.  This can be avoided by using the hand tool, but a user can to manually switch into this.  Is there a way to avoid this by allowing a user to click and drag on white space of a pool?  Maybe only allow a pool to be moved if you are selected on the left side in the pool name?  Would be a huge convenience
Thanks

There is the h shortcut that allows you to toggle the hand tool quickly.
Other than that, yes, having some modifier to use the hand would definitely be a big convenience.

ya have been using h, but it is not intuitive once you are "trained" to click and drag on white space ;P

This is also a common problem when dealing with large pools.  you quickly forget that you are inside of a pool when zoomed in.

@nikku any further thoughts on this?  It is a MAJOR ux problem when working with larger pools and you zoom in.  Having to remember to use h is a constant problem.

I would also love to have this feature. Gonna look into this.

I guess this is addressed via https://github.com/camunda/camunda-modeler/issues/629? People should not be required to drag the canvas on pools anymore.


fix: Prevents "stuck" joycon when going from single to paired or vice…

… versa AND have active joystick input when doing so.
Known bug on MacOS where a "stuck" active joystick won't recover before re-pairing (going to L+R-mode) again.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fix: Solves an edge case where a joycon can get "stuck" and continue to scroll when going from a L+R pair to a single controller, without any way of stopping it. It is not stop-able.

What is the current behavior? (You can also link to an open issue here)
When having a "stuck" joycon input, nothing can be done to stop it before a L+R pair is connected again.

What is the new behavior (if this is a feature change)?
When having a "stuck" controller, any movement/input on any other joycon unit will take over the control

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[x] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report

Merging #747 (a5e5e4f) into release44 (ee2811f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           release44     #747   +/-   ##
==========================================
  Coverage      68.23%   68.23%
==========================================
  Files            279      279
  Lines          33774    33774
  Branches        4516     4516
==========================================
  Hits           23046    23046
  Misses         10270    10270
  Partials         458      458

Continue to review full report at Codecov.

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


Add steps to enable building

Step 1 is sign into Travis, only then we can use the service actually.
Step 2 is flip the switch

Done


[stable/airflow] enable database properties configuration for external database

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Add "properties" field for AIRFLOW__CORE__SQL_ALCHEMY_CONN which is generated in _helpers.tpl to support database connection properties such as TLS/SSL
Which issue this PR fixes

fixes #23436

Special notes for your reviewer:
TLS is general requirement for external database hosted in cloud provider, this PR could help to enable the airflow hosting on cloud based Kubernetes @thesuperzapper
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@catweisun  Why on earth did you make a new PR?
Either way, this one is missing the changes in values.yaml

@thesuperzapper  Add the values.yaml and squash commits into single one.

/lgtm


Edit metadata

Someone online is wanting to edit MP4 metadata:
https://stackoverflow.com/questions/67349984/edit-metadata-from-mp4-with-go
I recommended this package, but I dont actually see any documention on how to edit metadata, or if its even possible

Thanks for the recommendation, but unfortunately, that type of metadata asked for is not supported.
This library focuses on the mp4 boxes that are part of the MPEG standards and on extracting/packing data. There is some metadata there, but not the type of metadata that iTunes/Apples has defined for .mp4/.mov files which are in Apple-specific boxes, with fields like title.
There is a list of ffmpeg-supported metadata boxes in mp4 files at https://kdenlive.org/en/project/adding-meta-data-to-mp4-video/. None of these boxes are supported in mp4ff.
It should be rather easy to add them, but there is a potential issue when box names start with ©,  e.g.   ©nam for title, which does not map to 4 bytes in Go due to UTF-8 being used to represent strings.  To make it 4 characters, another character encoding like iso/iec 8859-1 (latin1)  needs to be used.

wouldnt it be better to define a box name as [4]byte rather than string? As youve discovered, using string assumes each rune is one byte, which that doesnt have to be the case at all.

I checked a file I recently made with iMovie, and that had other metadata boxes according to my mp4ff-info tool:
  [udta] size=115776
    [clfn] size=14
     - not implemented or unknown box
    [dscp] size=41
     - not implemented or unknown box
    [kywd] size=14
     - not implemented or unknown box
    [thmb] size=115663
     - not implemented or unknown box
    [auth] size=15
     - not implemented or unknown box
    [titl] size=21
     - not implemented or unknown box

Here there seems to be titl box inside a udta (user data) box, which is different from the case above.
There is quiet some information at https://developer.apple.com/library/archive/documentation/QuickTime/QTFF/QTFFChap2/qtff2.html.
it explains that boxes (atoms as they are called in QuickTime lingo) that starts with © contain data in a "foreign language" and starts with a language code.
The metadata possibilities seem to be quite rich, so it is quite a challenge to make an editor for them.

@89z I exaggerated a bit, it actually works with to read the four bytes as a string, as long as you don’t try to get runes and that isn’t done anywhere in the code.
The return of the Type() method does not need to be 4 bytes string, but could be a 5-byte utf-8 string.
The file format spec ISO 14496-12 says that “boxtype is an uint32, which is normally four printable characters”. All boxes in the standard only use printable ascii characters.
I just added this comment about utf-8  here so that any potential implementer would spend some extra thoughts about that aspect.

There is actually already a test for handling an unknown box with a non-utf-8 type as a child of an udta box at
https://github.com/edgeware/mp4ff/blob/8fe1228bb1b78604a67f243d2af97961dd656f7f/mp4/udta_test.go#L8

I saw that the original ticket on StackOverflow has been closed due to lack of focus.
This project is open to PRs in this area, but the metadata boxes are not core to this project and there is no planned work in the area.
This ticket will be closed soon.


console transport: scrambled Log messages on console

I send log messages from the browser (frontend) by Winston Http Transport to the backend. Every 10 seconds, 4 messages are fired. On backend, I collect the log messages by a express middleware which forwards the message by frontendLogger.log(log.level, log.message, callback).
The messages logged to file are all correct, but on the node console, the messages are scrambled.
This is the backend winston configuration:

const winston        = require("winston");

var frontendLogger= new (winston.Logger)({
    transports: [
        new (winston.transports.Console)({
            level: "silly",
            colorize: true,
            label: "BL",
        }),
        new (winston.transports.File)({
            filename: "log/Frontend.log",
            maxsize: 500000,
            maxFiles: 10,
            tailable: true,
            zippedArchive: true,
        }),
    ],
});

The logs should be like this:
info: [BL] my repeated info message.
warn: [BL] my repeated warn message.
debug: [BL] my repeated debug message.
error: [BL] my repeated fatal message.
...

but it shows like that:
info: [BL] my repeated info message.
warn: [BL] my repeated warn message.
debug: [BL] my repeated debug message.
error: [BL] my repeated fatal message.
infodebug: [BL] my repeated info message.
: [BL] my repeated debug message.
warnerror: [BL] my repeated warn message.

: [BL] my repeated debug message.
warn: [BL] my repeated warn message.
error: [BL] my repeated fatal message.
infodebug: [BL] my repeated info message.
: [BL] my repeated debug message.
warnerror: [BL] my repeated warn message.
: [BL] my repeated fatal message.
...

I guess there is a timing problem, when the messages are received nearly at same time. What can I do?

Note:
I remarked, that only stderr messages scrambles stdout messages. Never a stderr scrambles a stderr or a stdout a stdout message.
info and warn are on stdout level, error on stderr level.
from winston console transport code:
if (this.stderrLevels[level]) {
   process.stderr.write(output + this.eol);
} else {
   process.stdout.write(output + this.eol);
}
 ```


Animation disappears when computer is locked and then unlocked

Describe the bug
After an app with an animation is started, if the user locks the computer, when the computer is unlocked (even after a few seconds) the animation is not shown anymore.
To Reproduce
Steps to reproduce the behavior:

Start an app with animation (check if animation is visible and running)
lock your computer
unlock your computer
the animation will is not showed anymore

Expected behavior
The animation would be displayed.
Screenshots
Before locking the computer:

After unlocking the computer:

Desktop (please complete the following information):

OS: Windows 10 Pro
Browser : N/A
Version: v4.0.30319

Additional context
The animation has no state where nothing is visible.

Hello,
I was able to reproduce this same issue regarding the animation for an application we've been working on as well.  To expand on the described bug (the animation disappearing after a user locks/unlocks the computer screen), it looks like any screen overlay (i.e. in our case, a Windows UAC prompt) causes the animation to disappear as well.
Is there a plan or roadmap in the works to address this bug?

@Crayjin WindowChrome bug was fixed too.


Please add the feature to upload settings and installed plugins to the cloud

Please add the feature to upload settings and installed plugins to the cloud. Cuz after reinstall os, all settings and plugins is missing, :(

Duplicate of #2743


prepare next version: update regexes

Objective

The post release version bump job failed: https://github.com/bevyengine/bevy/actions/runs/9799332118
This is because main didn't update to 0.14 as that happened in a branch

Solution

Update the regexes to work with the -dev suffix

Hold up, how does this work in the first place when \+ is escaped in [0-9]\+? Am I missing something?


Typo in this sentence

Similarly for changing the settings of a signle service within a specific resource

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

ID: d6f46cd1-caaa-99ac-1098-f15552369f22
Version Independent ID: 34ec4839-afba-f57e-1a6d-bc453e70b9ce
Content: Configure deployments - SQL Server big data clusters
Content Source: docs/big-data-cluster/deployment-custom-configuration.md
Product: sql
Technology: big-data-cluster
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@rgward -- Bob, thank you for your feedback.  I am looking into it.

Thank you for your keen eye.  I have updated the documentation and it should be live by tomorrow.


Content

Is this project an evolution of autovc?
can I transfer only rhythm or tiber or pitch from audio1 to audio2 where they have different contents?

I have the same question. I want to transfer rhythm &  pitch from audio1 to audio2,  and audio 2 retains the original tiber and content. So, i think this project can be combined with this repo.


Mistake on Decimal data type page re. bytes

Hello @Linda-Editor,
On Decimal data type page, please change:

Decimal variables are stored as 96-bit (12-byte) signed integers scaled by a variable power of 10. The power of 10 scaling factor specifies the number of digits to the right of the decimal point, and ranges from 0 to 28. ...

to:

'Decimal variables are stored as 96-bit (12-byte) unsigned integers, together with a scaling factor (used to either indicate a whole-number power of 10 to scale the integer down by, or that there should be no scaling), & also together with a value indicating whether the Decimal number is positive or negative. The scaling factor is the number of digits to store to the right of the decimal point, and ranges from 0 to 28. ...'

This correction can be verified by consulting the OLE Automation specification on the DECIMAL type that VBA's Decimal type maps to, here.

@MarkFern Thanks. I made this update to the page (with minor edits to your submission); please confirm that it's okay or let me know if you want updates; changes will go live later today:
Decimal variables are stored as 96-bit (12-byte) unsigned integers, together with a scaling factor (used to indicate either a whole number power of 10 to scale the integer down by, or that there should be no scaling) and a value (used to indicate whether the decimal number is positive or negative).
The scaling factor is the number of digits to store to the right of the decimal point, and ranges from 0 to 28.

@Linda-Editor, thanks for looking at this.
The following text:

... (used to indicate whether the decimal number is positive or negative) ...

shouldn't have brackets because it is of primary importance and not simply extra information.
Other edits seem fine.
Your ability to fathom these other edits, is probably the reason why you're the editor.  👍  😊

@MarkFern Thanks, Mark. I'll make those changes. :-)


Large Data

Sorting items in grid with large data (more than 500 items) makes drag and drop operation very slow

I just have the same problem with large data, and the react-infinity was the best solution for me, here a link of an example with more of 3000 items, point_right: https://codesandbox.io/s/exciting-snow-uehlk

I want to apply for a grid that means both directions x and y.
not only for rows, So how can we virtualize or use infinity here?


option for sealos login to increase the mirror pull limit

Detailed description of the question.
Hello,
I am trying to run - sealos run labring/kubernetes:v1.24.0 labring/calico:v3.22.1 --masters $IP_ADDR -p $PASSWORD.
However I am getting an error like - " error Applied to cluster error: initializing source docker://labring/kubernetes:v1.24.0: reading manifest v1.24.0 in docker.io/labring/kubernetes: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit".
Is there a way, I could pass the credentials of docker hub?
Thanks for all your help in advance
Padmini
Some reference materials you see.
No response

You can use our cloud public registry, doc is here: https://www.sealos.io/docs/cloud/apps/appstore/use-sealos-registry
Currently, you need do login action to pull images, as we are focusing provide a way to pull public images anonymous asap.

You can use our cloud public registry, doc is here: https://www.sealos.io/docs/cloud/apps/appstore/use-sealos-registry
Currently, you need do login action to pull images, as we are focusing provide a way to pull public images anonymous asap.

Thank You


How to report lyrics ?

I'm still actively working on this issue. I'm closing this and will track this at #16.


docs: Add code signing doc, update welcome to reference it

Added a new document for setting up yubikey and ssh code signing
Moved existing gpg code signing instructions to that same doc
Removed code signing details from welcome doc
Added reference to code signing doc from welcome doc

@ckhrysze can / should we merge this?


[Cherry-pick] [3.0.0] If the single log length is long, it will causes the View…

… Log page to hang (#10612)

Purpose of the pull request
fix #10609

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Hi @zhuxt2015 I had cherry-pick this pr into 3.0.0-prepare.
see https://github.com/apache/dolphinscheduler/commit/1b0f8c04d3e419320e30254393a8592ed52065a4

Hi @zhuxt2015 I had cherry-pick this pr into 3.0.0-prepare.
see 1b0f8c0 please take a check.

@zhuxt2015 will be close this weekend if no further information


Yesterday it worked, today: Image already shrunk to smallest size?!

hi there,
Yesterday i shrunk an Image from my selfmade Raspberry Pi with Batocera from 128GB down to 17GB with "PiShrink"
this was my FIRST test and i was very Impressed!!!
Today i added some new files because i forgot something (was about 1GB) to the SD-Card and made a new clone of the SD-Card again.
I've used 1:1 same commands, same SD-Card, same Raspberry...
the only thing i did was...
i copied the PIShrink shrunken image to the SD-Card
sudo umount /dev/sdb1
sudo umount /dev/sdb2

and then
sudo dd if=/dev/sdb of=disk.img status=progress

and finally
sudo pishrink.sh disk.img

But today it says:
fuba@fuba-Ubuntu:~$ sudo pishrink.sh disk.img
pishrink.sh v0.1.2
pishrink.sh: Gathering data ...
pishrink.sh: /etc not found, autoexpand will not be enabled ...
pishrink.sh: Checking filesystem ...
SHARE: 5009/884736 files (1.9% non-contiguous), 3099161/3518419 blocks
resize2fs 1.45.5 (07-Jan-2020)
pishrink.sh: ERROR occurred in line 335: Image already shrunk to smallest size
fuba@fuba-Ubuntu:~$

here is the fdsik info about the img file:
fuba@fuba-Ubuntu:~$ sudo fdisk -l disk.img
Disk disk.img: 119,9 GiB,<PHONE_NUMBER>92 bytes, 249737216 sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes
Disklabel type: dos
Disk identifier: 0x224a7b5b

Device     Boot   Start      End  Sectors  Size Id Type
disk.img1          2048  6293503  6291456    3G  c W95 FAT32 (LBA)
disk.img2       6293504 34440856 28147353 13,4G 83 Linux

any idea why it worked yesterday but not today?!

ah...
i think i found the problem..

seems the "rest" of the disk is unallocated because of the "shrunken" image just used ~14GB on second partition :/
any idea who to prevent this when use a "shrunken" image?


Allow longer S3 Access Key and Secret Key

My actions before raising this issue

[X] Read/searched the docs
[X] Searched past issues

Expected Behaviour

I would like to connect to an AWS S3 compatble service (Cloud Storage) served from an on-premise storage.
Current Behaviour

Both AccessKey and SecretKey fields are truncated to match AWS's number of characters.
Our AWS S3 compatible service uses 129 character long AccessKey and SecretKey.
Possible Solution

Modify Acces Key and Secret Key fields to allow entering 129 characters long keys.
Steps to Reproduce (for bugs)

Click on Cloud Storage
Click on Add (+)
Enter Bucket Name, Endpoint, Access Key and Secret Key (both are truncated)

Context

Your Environment

Git hash commit (git log -1):
Docker version docker version (e.g. Docker 17.0.05): Version:  20.10.12
Are you using Docker Swarm or Kubernetes? No
Operating System and version (e.g. Linux, Windows, MacOS):  macOS 12.0.1
Other diagnostic information / logs:
Logs from `cvat` container
  [Wed Apr 20 22:39:10.737344 2022] [wsgi:error] [pid 555:tid<PHONE_NUMBER>00] [remote <IP_ADDRESS>:35558] ERROR - 2022-04-20 22:39:10,735 - serializers - Cannot create resource datacenter with specified credentials. Access forbidden.

Next steps
You may join our Gitter channel for community support.

@sebagarayco , thanks for the report. Could you please contribute and send us a PR with necessary changes? It should not be a complex change.


Add date-time support for forecasting framework

Reference Issues/PRs
#347, #220
What does this implement/fix? Explain your changes.

Support for pd.DatetimeIndex, pd.PeriodIndex in data and forecasting horizon, including helper functions to patch mathematical operations on pandas date/time objects for our purposes (mostly returning integers for known time unit, rather than time delta objects)
Included forecasting data sets are now returned with pd.PeriodIndex
Support for absolute forecasting horizon specification via refactored ForecastingHorizon class
Easy importing via sktime.forecasting.all module enabling from sktime.forecasting.all import * following fastai
A few minor changes (typos, formatting, etc)
Corresponding updates to forecasting notebook
New unit tests for supported combinations of data index types and forecasting types

Does your contribution introduce a new dependency? If yes, which one?

seaborn (soft dependency, for plotting)

Key files to review
Since the forecasting horizon was used in basically every forecasting algorithm, this PR makes changes to many files. Key changes include:

sktime/forecasting/base/_fh.py (implementation of forecasting horizon class)
sktime/forecasting/base/tests/test_fh.py (unit tests for forecasting horizon class)
sktime/forecasting/tests/_config.py (check SUPPORTED_INDEX_FH_COMBINATIONS)
sktime/forecasting/tests/test_all_forecasters.py (unit testing framework for all forecasters)
sktime/utils/datetime.py (date/time helper functions)

Comments

Supporting only pd.PeriodIndex and not pd.DatetimeIndex would make the code a bit simpler as pd.DatetimeIndex requires additional patches to return integers for durations/time deltas and additional type distinctions (dispatch), however it seems like pd.DatetimeIndex supports some functionality which is not supported by pd.PeriodIndex (e.g. is_month_end)

Future work

refactor BaseLastWindowForecaster and reduce inheritance as much as possible

@fkiraly I've made some changes after our conversation, mostly refactored the helpers functions, and added a list of key files to review.

Thanks @fkiraly for the review.

Re adding support for more types like pd.TimeDelta: (1) it would add more case distinctions (e.g. because pandas does not consistently handle collections of duration types (see below)), (2) specifying a forecasting horizon with durations seems redundant as the time unit will always be specified in the data, (3) I don't think using duration types is very intuitive from a user's perspective and can't think of a situation where I would prefer passing duration types rather than integers
We need _coerce_duration_to_int() because some of our algorithms internally and other package we interface require integer forecasting horizons, not duration types.
One reason why we need so many case distinctions is that pandas doesn't seem to be very consistent with the typing. For example, multiple pd.TimeDelta (duration for timestamp type) are collected in a pd.TimeDeltaIndex, but multiple pd.DateOffsets (from duration for pd.Period type) are collected in a simple pd.Index with individual pd.DateOffsets inside.
I'm going to move __getitem__ and __len__ to DELEGATED_METHODS
Re downward compatibility, at this stage I prefer consistency of method names over compatibility, adding deprecation cycles will complicate and slow dev work considerably I think, also the ForecastingHorizon was mainly used internally, it was included in the notebooks for example
I'm going to add some unit tests for the date-time utility functions and better docstrings
I'm going to add something like has_out_of_sample_values and has_in_sample_values and caching to avoid computing the same things twice

I've implemented a few suggested changes. Let me know if there's anything else you would definitively change.


After upgrade, brew update would install curl automatic.

brew doctor output
Your system is ready to brew.

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 3.6.15-53-g4534748
ORIGIN: https://github.moeyy.xyz/https://github.com/Homebrew/brew
HEAD: 4534748f29758a8964688d202705f6f66deadfee
Last commit: 17 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 1637e66b57a7c334f2cc95e449e2331fccb08d38
Core tap last commit: 49 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_EDITOR: nvim
HOMEBREW_FORCE_BREWED_CURL: set
HOMEBREW_MAKE_JOBS: 8
HOMEBREW_NO_AUTO_UPDATE: set
HOMEBREW_NO_ENV_HINTS: set
HOMEBREW_NO_INSTALL_CLEANUP: set
all_proxy: socks5://<IP_ADDRESS>:7890
http_proxy: http://<IP_ADDRESS>:7890
https_proxy: http://<IP_ADDRESS>:7890
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit arm_blizzard_avalanche
Clang: 14.0.0 build 1400
Git: 2.37.1 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.79.1 => /usr/bin/curl
macOS: 12.6.2-arm64
CLT: <IP_ADDRESS>.1.1668646533
Xcode: N/A
Rosetta 2: false

What were you trying to do (and why)?
Try to run brew update, but it install curl automatic even I never installed it via brew.
What happened (include all command output)?
==> Fetching dependencies for curl: brotli, gettext, libunistring, libidn2, libnghttp2, ca-certificates<EMAIL_ADDRESS>libssh2, openldap, rtmpdump, lz4, xz and zstd
==> Fetching brotli
==> Downloading https://ghcr.io/v2/homebrew/core/brotli/manifests/1.0.9
^C
Error: 'curl' must be installed and in your PATH!

What did you expect to happen?
curl is in my PATH, and I don't want to manage it via brew .
Step-by-step reproduction instructions (by running brew commands)
Just `brew update`.

This is your problem:
HOMEBREW_FORCE_BREWED_CURL: set

Do
unset HOMEBREW_FORCE_BREWED_CURL

and Homebrew should stop installing curl automatically. You may need to remove this setting from your .zprofile/.zshrc/equivalent.

This is your problem:
HOMEBREW_FORCE_BREWED_CURL: set

Do
unset HOMEBREW_FORCE_BREWED_CURL

and Homebrew should stop installing curl automatically. You may need to remove this setting from your .zprofile/.zshrc/equivalent.

Thanks.
I‘m so sorry that I overlook env variables.


Okapi Dependency Update from V. 0.36 to 1.43.0

Updating Okapi dependency to the latest version which supports java 8. (1.43.0)
Currently the latest version of Okapi is 1.45.0

Thank you @hylstonn! this is promising. Would you mind updating the commit message with more detail about each change required?


SA1642: Language ignored in .NET 6.0 SDK RC2

In .NET 6.0 SDK RC2, the language is not being taken into account when reporting SA1642
The language is setup correctly in stylecop.json
{
  "settings": {
    "documentationRules": {
      "documentationCulture": "en-GB"
    }
  }
}

And the constructor comment uses Initialises rather than Initializes
/// <summary>
/// Initialises a new instance of the <see cref="Class1"/> class.
/// </summary>

Expected:
No warning is issued
Actual:
SA1642 warning is issued
Remarks:
This seemed to work on preview versions of .NET 6.0 SDK, and continues to work through Visual Studio 2022.

This issue is now in the stable version of Visual Studio 2022 17.4.1.

Also in 17.5.5

Is there a workaround for this?
Just migrated from VS2019 where everything worked. Should / could I install an earlier version of StyleCopAnalyzers from NuGet to get the localisation fixed, or it a VS2022 issue and I'm stuck with the warnings?
Currently using v<IP_ADDRESS>5

No work around at the moment, just suppress those warnings, which defeats the purpose of the warnings. Downgrade will not work, as the issue is with Roslyn.

@sharwell, is there anything that could be done here? For example, are the satellite assemblies required or could the resource strings be embedded in the main assembly somehow?


[Image_picker] Constant fields should be private (Breaking change)

The constant in https://github.com/flutter/plugins/blob/master/packages/image_picker/lib/image_picker.dart#L11-L15 were meant to be private.
Although it is an easy fix, it is a breaking change nonetheless.

Hi @cyanglaz
What are those constants? I've checked the source and I just found the platform static field, is he the one that must be private?
https://github.com/flutter/plugins/blob/3e547d9a808c35ea81b32bb225cfac1098607672/packages/image_picker/image_picker/lib/image_picker.dart#L29-L30

[triage-team]
Without additional information, I am therefore reluctantly going to close this bug for now. Please feel free to reopen if necessary.


Problem for linking the example 6: "undefined reference to `omp_get_num_threads'"

Hi, I'm trying to use your GPU FHE lib, while I meet some problem on compiling the examples. I set DHEonGPU_BUILD_EXAMPLES=ON  and have passed all the other steps. But when it reach the step to link the lib for exp6(only this exp use openmp and it occurs the error) I get the error:
tmpxft_003e9f9a_00000000-7_6_multi_stream_usage_way1.cudafe1.cpp:(.text+0x9c5): undefined reference to 'omp_get_num_threads'
/usr/bin/ld: tmpxft_003e9f9a_00000000-7_6_multi_stream_usage_way1.cudafe1.cpp:(.text+0x9cd): undefined reference to 'omp_get_thread_num'

I didn't modified the CMakeFIle while I don't understand what happens since all the things seems should work but not. Can you help to figure out what's the problem?

I will check it and let you know.

I've made an edit now, if the problem still persists, can you share with me the nvcc, gcc, cmake and openmp versions you are using?
Just put this to your terminal.

nvcc --version
gcc --version
cmake --version

For checking openMP version, use them one by one;

echo '#include <omp.h>
#include <stdio.h>
int main() {
printf("OpenMP version: %d\n", _OPENMP);
return 0;
}' > check_openmp.c
gcc -fopenmp check_openmp.c -o check_openmp
./check_openmp

Thx, it now works for me :)


HIVE-24167: TPC-DS query 14 fails while generating plan for the filter

What changes were proposed in this pull request?
Make it possible for PlanMapper to link related Operators or RelNodes correctly.
For your reference, I made a Gist page explaining the behaviors of PlanMapper and alternative approaches in my mind.
https://gist.github.com/okumin/b111fe0a911507bdf6a7204f49b9cb72
Why are the changes needed?
No. 14 of TPC-DS or other complex queries with CTE materialized fail due to this bug.
https://issues.apache.org/jira/browse/HIVE-24167
Does this PR introduce any user-facing change?
No. This is a bug fix.
Is the change a dependency upgrade?
No.
How was this patch tested?
I added a new test case, which fails with the current master branch, and restored No. 14 of TPC-DS.

just seen the details you linked to the PR.
I believe the link should be allowed - but in this the merge should be allowed due to the fact that these things represent the same thing.
I see now why the signature become an obstacle - I believe the SWO was able to match the same plan-parts to only scan the table once; if you don't have link calls there to explain the situation -> that could cause equiv violations  here...could you please check that?

I believe the link should be allowed - but in this the merge should be allowed due to the fact that these things represent the same thing.

I see now why the signature become an obstacle - I believe the SWO was able to match the same plan-parts to only scan the table once; if you don't have link calls there to explain the situation -> that could cause equiv violations here...could you please check that?

@kgyrtkirk

I believe the link should be allowed - but in this the merge should be allowed due to the fact that these things represent the same thing.

I guess this mentions the link on creating a FilterOperator?

I see now why the signature become an obstacle - I believe the SWO was able to match the same plan-parts to only scan the table once; if you don't have link calls there to explain the situation -> that could cause equiv violations here...could you please check that?

I guess the SWO stands for SharedWorkOptimizer. With CTE materialization, each CTE is compiled separately. It means materialized_cte, another_materialized_cte, and the body part are compiled one by one in a dedicated manner. So, SharedWorkOptimizer has no chance as it's invoked in analyzeInternal.
Just for a note. We use CTE because we need to support some cases which SharedWorkOptimizer doesn't support. An example is complex CTE + UNION ALL. We may replace CTE materialization with SharedWorkOptimizer when it becomes more mature.

each CTE is compiled separately
got it now - I was  thinking its the other way around

I was thinking about this a bit in the last couple days; here is a summary of my thoughts:

to have this feature cause trouble the following is needed:

a failing query
an incorrectly taken stat entry
the bad stat entry causes further issues
....not very likely :D

keeping multiple equiv groups which should be merged is not fortunate - but may not cause trouble
at the end of the day all a Map<Signature, RuntimeStat> will be created from all these details

if there is are 2 groups which have the same Signature that will mean there will be multiple candidates for the key in the map => which means those should have been in a single group...that's why I didn't really like to relax the check on the signature

I believe lookup-by-signatue will either return one a representative group ; even when there are more

from the listed approaches in the doc:

I liked the best the RelTreeSignature related; I think that would be the best if possioble

this might be a way to achieve a merge between the 2 groups
have you tried somehow relax the alias name of HiveTableScan in the RelTreeSignature ?

that class creates the signature of the relnodes 1-by-1 ; so you could access/customize/etc based on the type

possibly use the @Signature annotation
use something to filter out poisonous stuff @IgnoreSig or something
by-hand stuff

on the other hand after thinking it thru what might be the outcomes - I find that the idea to not pull the signatures during linking might actually work correctly...

@kgyrtkirk
I tried the option using RelTreeSignature in the following Pull Request. I still have some concerns, but it will likely work from my perspective.
https://github.com/apache/hive/pull/5077
Could you please take a look and see if it seems feasible? If we prefer #5077 , I will void this PR and try to get it merged.

Let me close this one because we will likely go with #5077 or another approach.


Replace string interpolation with string.Format(CultureInfo.InvariantCulture

Looks fine to me.

Has already been fixed with commit 0e915f47b315628b7aeef3faabea1d75ca41dec5.


Question!

When I clone this code to a remote virtual machine for the web (in this case, a EC2 from amazon), the database does not work. Why would that be?
Instead, when I run react remotely, it still communicates with a local instance of a server (one running on my local machine and not on the EC2)

I got it... Change line 2 on the myAPI.js to the server, it works!


Use CloudFront (or another CDN)

S3 is not a CDN.
<link rel="stylesheet" href="https://s3-us-west-1.amazonaws.com/patterns.esri.com/files/calcite-web/1.0.0-beta.31/css/calcite-web.min.css">
<script src="https://s3-us-west-1.amazonaws.com/patterns.esri.com/files/calcite-web/1.0.0-beta.31/js/calcite-web.min.js"></script>

What you're looking for is CloudFront.  You'll want to put your bucket behind a sub-domain, like cdn.esri.com or some-such.
<link rel="stylesheet" href="https://cdn.esri.com/files/calcite-web/1.0.0-beta.31/css/calcite-web.min.css">
<script src="https://cdn.esri.com/files/calcite-web/1.0.0-beta.31/js/calcite-web.min.js"></script>

Let me know if you need assistance setting it up.  I'm an AWS Certified Solutions Architect.

this website repo doesn't reference any content on S3.

https://s3-us-west-1.amazonaws.com/patterns.esri.com/files/calcite-web/1.0.0-beta.31/css/calcite-web.min.css

I think you're looking for https://github.com/Esri/calcite-web/issues


resource(engine): fix skipping resource meta remove in gc retry

What problem does this PR solve?

Issue Number: close #5832, also close #5699
What is changed and how it works?
In #5832, r.client.DeleteResource failed because of database table is locked: resource_meta", which is acceptable because database operation could fail.
In the next retry of *DefaultGCRunner.gcOnce, handler(ctx, res) returns resource not found error because the local file resource had been removed, but resource meta in metastore is not removed.
In this PR, when removing resource rpc returns resource not found, ignore this error and continue to delete resource from resourcemeta
Check List
Tests

Unit test
Integration test

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

Codecov Report

Merging #5944 (69bab70) into master (6a451ea) will increase coverage by 0.1702%.
The diff coverage is 61.0169%.

Flag
Coverage Δ

dm
51.9694% <23.2142%> (+0.0392%)
:arrow_up:

engine
62.4552% <75.1533%> (+0.2705%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #5944        +/-   ##
================================================
+ Coverage   57.0764%   57.2466%   +0.1702%
================================================
  Files           682        678         -4
  Lines         80224      79975       -249
================================================
- Hits          45789      45783         -6
+ Misses        30146      29942       -204
+ Partials       4289       4250        -39


Custom validation

command
args

I will stick with kubeconform and only make its parameters configurable because other commands might output errors differently, leading to failure of my std_err and result logic


How can I even get started with say calc.exe using CodedUI

I've downloaded the projects and I already have a basic codedui project using a UIMap that launches the windows calculator. I can't seem to figure out how to begin to use your framework and extensions. Could you post a sample for CodedUI?

I've gone ahead and forked and added a sample for CodedUI with the calculator.
https://github.com/ianceicys/TestEssentialsSampleCodedUI
https://github.com/ianceicys/TestEssentialsSampleCodedUI/blob/master/TestAutomationEssentials.Samples/CodedUI/CalculatorUIWin7Example/CalculatorTests.cs
What I don't know how to do is NOT reference a UIMap. Here is the line of code I want to change so I don't reference a UIMap.
Mouse.Click(UIMap.UICalculatorWindow.Find<WinButton>(By.Name("1")));
If my application has multiple windows where the title bar changes how do I search and find the windows.

Hi Ian,
I apologize for not responding yet (I was very busy this week...). Anyway, you can get the calculator window like this:
            var calc = Process.Start("calc.exe");
           Wait.Until(() => calc.GetCurrentMainWindowTitle() == "Calculator");            var calcWindow = UITestControlFactory.FromWindowHandle(calc.MainWindowHandle);`
I'll try to send a complete example later on.
HTH,
Arnon.

Arnon,Thanks so much for the feedback and the code sample. I've update the code to use ApplicationUnderTest.Launch as that appears to be a best practice from Microsoft.
https://github.com/ianceicys/TestEssentialsSampleCodedUI/commit/005950bfa29c2dd3c1a2aa4458f87c9e2453a2b4
I am also looking to see how to enhance this example similiar to http://www.evoketechnologies.com/blog/windows-automation-testing-coded-ui/
Thanks again,Ian Ceicys
Date: Thu, 21 Apr 2016 04:42:41 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [arnonax/TestEssentials] How can I even get started with say calc.exe using CodedUI (#2)
Hi Ian,
I apologize for not responding yet (I was very busy this week...). Anyway, you can get the calculator window like this:
var calc = Process.Start("calc.exe");
Wait.Until(() => calc.GetCurrentMainWindowTitle() == "Calculator");
        var calcWindow = UITestControlFactory.FromWindowHandle(calc.MainWindowHandle);`

I'll try to send a complete example later on.
HTH,
Arnon.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Thanks a lot for helping me improve the project by adding samples! I appreciate it a lot!
Unfortunately I have a problem opening Coded UI projects on my laptop right now :-(
Anyway I took a glimpse at your code and I have few comments:

In the test method, instead of:
Mouse.Click(calcApp.Find<WinButton>(By.Name("1")));
You can write:
calcApp.Find<WinButton>(By.Name("1")).Click();
When using Assert.AreEqual, the first argument is the expected result and the second is the actual. This comes into affect when the assertion fails and the message that is written tells what was expected and what was the actual. If you switch them the error message can be very confusing!
I don't think that you should depend on an environment variable to know where Calc.exe is. First, it's normally in the PATH so if you just use "calc.exe" it should be found. If you want to be more precise you can use Path.Combine(Environment.SystemDirectory, "calc.exe")

Thanks again,
Arnon.

Hi Arnon,Thanks so much for the quick response and awesome feedback. I incorporated your feedback below:https://github.com/ianceicys/TestEssentialsSampleCodedUI/blob/master/TestAutomationEssentials.Samples/CodedUI/CalculatorUIWin7Example/CalculatorTests.cs#L38-L65
I genuinely appreciate your feedback so I've added 5 additional test examples.
I decided to expand a sample to work with the menuitems, but I am unable to get a working example with the menuitems.
https://github.com/ianceicys/TestEssentialsSampleCodedUI/commit/c2878cd8d2ff64548bf4b7b29d39c41a61268b39
If you have any sample that works with GetChildren and menuitems I'd love to build out several additional samples around the mortgage calculator.

While I don't think it's necessary to use an environment variable to know where calc is...for custom apps using an environment variable is desirable and when I want to extend the test to Windows 10 and UWP apps environment variables work nicely. https://www.benday.com/2012/10/06/coded-ui-tests-how-to-avoid-hard-coding-paths-to-exes-or-urls-using-applicationundertest-and-browserwindow/
Thanks again and I look forward to your feedback so I can get a working sample with the mortgage calculator put together.
Ian Ceicys

Date: Thu, 21 Apr 2016 07:46:34 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [arnonax/TestEssentials] How can I even get started with say calc.exe using CodedUI (#2)
Thanks a lot for helping me improve the project by adding samples! I appreciate it a lot!
Unfortunately I have a problem opening Coded UI projects on my laptop right now :-(
Anyway I took a glimpse at your code and I have few comments:

In the test method, instead of:

Mouse.Click(calcApp.Find(By.Name("1")));
You can write:
calcApp.Find(By.Name("1")).Click();

When using Assert.AreEqual, the first argument is the expected result and the second is the actual. This comes into affect when the assertion fails and the message that is written tells what was expected and what was the actual. If you switch them the error message can be very confusing!

I don't think that you should depend on an environment variable to know where Calc.exe is. First, it's normally in the PATH so if you just use "calc.exe" it should be found. If you want to be more precise you can use Path.Combine(Environment.SystemDirectory, "calc.exe")

Thanks again,
Arnon.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Hi Arnon,Last email of the day, I promise.
I made some good headway on getting a working example of the help dialog and window. But I have some questions in my code I'd love to get your feedback on.
https://github.com/ianceicys/TestEssentialsSampleCodedUI/commit/724ee5cea3c45ccb551078997e7cf3968d56dacb
Before I venture further on building out the mortgage calculator example I'd love to get some feedback on my questions so I can best highlight your extension methods in the examples.
Thanks again and I look forward to your feedback.
Ian Ceicys
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS><EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: RE: [arnonax/TestEssentials] How can I even get started with say calc.exe using CodedUI (#2)
Date: Thu, 21 Apr 2016 12:55:27 -0400
Hi Arnon,Thanks so much for the quick response and awesome feedback. I incorporated your feedback below:https://github.com/ianceicys/TestEssentialsSampleCodedUI/blob/master/TestAutomationEssentials.Samples/CodedUI/CalculatorUIWin7Example/CalculatorTests.cs#L38-L65
I genuinely appreciate your feedback so I've added 5 additional test examples.
I decided to expand a sample to work with the menuitems, but I am unable to get a working example with the menuitems.
https://github.com/ianceicys/TestEssentialsSampleCodedUI/commit/c2878cd8d2ff64548bf4b7b29d39c41a61268b39
If you have any sample that works with GetChildren and menuitems I'd love to build out several additional samples around the mortgage calculator.

While I don't think it's necessary to use an environment variable to know where calc is...for custom apps using an environment variable is desirable and when I want to extend the test to Windows 10 and UWP apps environment variables work nicely. https://www.benday.com/2012/10/06/coded-ui-tests-how-to-avoid-hard-coding-paths-to-exes-or-urls-using-applicationundertest-and-browserwindow/
Thanks again and I look forward to your feedback so I can get a working sample with the mortgage calculator put together.
Ian Ceicys

Date: Thu, 21 Apr 2016 07:46:34 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [arnonax/TestEssentials] How can I even get started with say calc.exe using CodedUI (#2)
Thanks a lot for helping me improve the project by adding samples! I appreciate it a lot!
Unfortunately I have a problem opening Coded UI projects on my laptop right now :-(
Anyway I took a glimpse at your code and I have few comments:

In the test method, instead of:

Mouse.Click(calcApp.Find(By.Name("1")));
You can write:
calcApp.Find(By.Name("1")).Click();

When using Assert.AreEqual, the first argument is the expected result and the second is the actual. This comes into affect when the assertion fails and the message that is written tells what was expected and what was the actual. If you switch them the error message can be very confusing!

I don't think that you should depend on an environment variable to know where Calc.exe is. First, it's normally in the PATH so if you just use "calc.exe" it should be found. If you want to be more precise you can use Path.Combine(Environment.SystemDirectory, "calc.exe")

Thanks again,
Arnon.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Hi arnonax,
I've incorporated your feedback and have the Help/About dialog working and with a proper assert.
Here is the commit:
https://github.com/ianceicys/TestEssentialsSampleCodedUI/blob/917acf491234300574e17fb7d84aabb380252550/TestAutomationEssentials.Samples/CodedUI/CalculatorUIWin7Example/CalculatorTests.cs#L188-L208
I've also added a mortgage monthly payment example.
Here is the commit:
https://github.com/ianceicys/TestEssentialsSampleCodedUI/commit/4f02f3eb7c34bc25b1cf55ab40f227ca76691580
I am going to continue to work on adding additional examples, but if there are extension methods you think I should highlight please let me know.
Thanks for the feedback, it's been a pleasure working on these examples!
Ian


Carbon UI project folder structure is not exactly as maven project

The carbon UI project folder structure is not exactly as maven project, so the default pom file in UI project will not include the i18n resource file.
i18n resource file resource file is not created when creating the UI project.
https://wso2.org/jira/browse/DEVTOOLEI-1106

The Carbon UI Bundle project is not supported in Integration Studio.


Added nested @if support

Added nested @if support
Improved test output
Added more test cases
Made indentation consistent

+1 please!

+1

Nice catch. Thanks.


Change/set check-in and check-out dates programmatically

Description
I want to open the date picker with a pre-selected date range. For example, I've two (home & show) pages. The user selects the date range from the home page and submits the form which redirects to the show page. Now I want to populate the date picker with that selected range on show page. It would be great if we can set check-in and check-out values with props.
Datepicker Version
I'm using 3.0.9 version.

Try to update to v3.0.10, https://github.com/krystalcampioni/vue-hotel-datepicker/releases/tag/v3.0.10

Thank you for your response. As per your suggestion, I've updated my version from 3.0.9 to 3.0.10 but I'm still not able to set the date range before opening the date picker. I want to change the check-in and check-out date programmatically. It would be great if you will provide me an example of how to it.

The props that you are looking for are "startingDateValue" and "endingDateValue"


Post videos instead of images on social media

[ ] figure out how to specify a thumbnail image for the video
[ ] Resize video in ffmpeg to be portable on Instagram and twitter

the thumbnail is a bit annoying if its the first frame since these  models start from noise

https://user-images.githubusercontent.com/378888/127774709-9125d5c4-3415-4a2d-a40a-f0d87d720b2c.mov
When navigated with a googlebot user agent, netlify doesn't render the site properly


ReactNativeConfigPackage is not abstract and does not override abstract method createJSModules() in ReactPackage

/.../node_modules/react-native-config/android/src/main/java/com/lugg/ReactNativeConfig/ReactNativeConfigPackage.java:13: error: ReactNativeConfigPackage is not abstract and does not override abstract method createJSModules() in ReactPackage
public class ReactNativeConfigPackage implements ReactPackage {
       ^
1 error
:react-native-config:compileReleaseJavaWithJavac FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':react-native-config:compileReleaseJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

This PR #144 fix problem but need to merge ASAP

Agreed. I am also waiting on this PR to be merged.

This issue was resolved when #144 was merged.

:+1:!

error: ReactNativeContacts is not abstract and does not override abstract method createJSModules() in ReactPackage
public class ReactNativeContacts implements ReactPackage {
^
1 error
:react-native-contacts:compileReleaseJavaWithJavac FAILED
On
react-native-cli: 2.0.1
react-native: 0.46.0

@saurabhspatil did you find a solution?

error: ReactNativeContacts is not abstract and does not override abstract method createJSModules() in ReactPackagepublic class ReactNativeContacts implements ReactPackage {^1 error:react-native-contacts:compileReleaseJavaWithJavac FAILED
Got this module working by adding the following to ReactNativeContacts.java.
import com.facebook.react.bridge.JavaScriptModule;
@Override public List<Class<? extends JavaScriptModule>> createJSModules() { return Collections.emptyList(); }

I am still getting this error
:react-native-contacts:compileReleaseJavaWithJavac - is not incremental (e.g. outputs have changed, no previous execution, etc.).
C:\Users\Oserlab\Desktop\YC_ReactNative\youngclay\node_modules\react-native-contacts\android\src\main\java\com\rt2zz\reactnativecontacts\ReactNativeContacts.java:13: error: ReactNativeContacts is not abstract and does not override abstract method createJSModules() in ReactPackage
public class ReactNativeContacts implements ReactPackage {
^
1 error
:react-native-contacts:compileReleaseJavaWithJavac FAILED
I am using:-
"react-native-contacts": "1.0.4",
"react-native": "0.46.0",
"react-native-cli": 2.0.1


Shows Pdf Conversion Completed but Reports Error in Obs

Successfully used this for a few weeks now, then over the last weekend it only generates this fail notice 100% of attempts:
(1) Plugin reports pdf conversion is completed, but also displays conversion failed alert.
(2) Marker API successfully reports API access and process.
I've tried uninstalling/reinstalling plugin, resetting Marker API key, turning off all other Obsidian plugins with no success.  My Datalab dev account is properly funded and no other errors being reported in the log.
Any suggestions / any more information I can provide / anything else I can try?

I'm sorry to hear that, can you please send me your logs from obsidian?
Here you can see how: https://help.obsidian.md/Help+and+support#Capture+console+logs
I updated the plugin last week, but I didn't change anything regarding the datalab api (at least I think so). Are you using the datalab API or the self-hosted version?
The connection to the datalab API seems to work fine, so this should not be the problem.
I need the logs to help you, because I don't have the API myself.

Never mind, I think I've found the issue. Please try Version 1.3.1 and tell me if this fixes your problem!

> Never mind, I think I've found the issue. Please try Version 1.3.1 and
tell me if this fixes your problem!
Sure thing - will give it a go in ~60 minutes time.
> I need the logs to help you, because I don't have the API myself.
Happy to set you up with an API key for a day if the 1.3.1 version a fix -
will let you know shortly.
On Tue, 3 Dec 2024 at 15:51, L3N0X @.***> wrote:

Never mind, I think I've found the issue. Please try Version 1.3.1 and
tell me if this fixes your problem!
—
Reply to this email directly, view it on GitHub
https://github.com/L3-N0X/obsidian-marker/issues/10#issuecomment-2515712443,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BNMZ4JGA6SEAUUJ27QWICAD2DYYWNAVCNFSM6AAAAABS6V24FOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJVG4YTENBUGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Unfortunately 1.3.1 didn't fix the issue.

(1) version 1.3.1
(2) error generated, folder is created (as per options) but nothing in the folder
(3) API records that conversion took place
(4) Obs log shows no errors or warnings.

Now it should actually be fixed, made a small mistake yesterday, sorry. Now it should actually be working again. Please check if 1.3.2 is working for you now.

Confirmed as working now - many thanks for your help!
On Wed, 4 Dec 2024 at 04:32, L3N0X @.***> wrote:

Now it should actually be fixed, made a small mistake yesterday, sorry.
Now it should actually be working again. Please check if 1.3.2 is working
for you now.
—
Reply to this email directly, view it on GitHub
https://github.com/L3-N0X/obsidian-marker/issues/10#issuecomment-2517069666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BNMZ4JBHV7SAR24P5FGVX5D2D3R3RAVCNFSM6AAAAABS6V24FOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJXGA3DSNRWGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Are static constructors thread-safe?

Would be nice if this article could call out thread-safety aspects of static constructors.  Thanks!

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

ID: aa3ed1c5-1f79-9234-beec-b5771c3fb3c0
Version Independent ID: ac4f6361-70a7-fb43-101e-c12044f60896
Content: Static Constructors - C# Programming Guide
Content Source: docs/csharp/programming-guide/classes-and-structs/static-constructors.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

/cc @jcouv to check this answer.
My interpretation of the C# spec section on static constructors, it states:

The static constructor for a closed class type executes at most once in a given application domain.

That would seem to imply that only thread will invoke the static constructor.
However, there are no restrictions on the code you write inside a static constructor. You could write code that starts new threads in the constructor. That would require your own thread synchronization.

Yeah, this is a similar discussion #10244, there's always unsafe things you can do anywhere in code, but it's always nice to know when you don't need an extra layer of mutexing around things or not or can use the inherit constructs in the language. :)

I'm not quite sure how you want to express this in this documentation, but C# static class constructors are thread-safe, and executed under a lock that is specific to the exact specific type of the class. (This is important for generics, i.e. the List static class constructor executes under a different lock than the List static class constructor.) Adding additional mutexes is both non-necessary, and possibly a mechanism to cause deadlock issues.
The precise behavior of the thread-safety of class constructors can be found in sections <IP_ADDRESS> and <IP_ADDRESS> in partition II of the ECMA 335 standard. See http://www.ecma-international.org/publications/files/ECMA-ST/ECMA-335.pdf
Here is a short, somewhat more easily readable description of the special behavior of the class constructor lock. For exact details see section 105.3.3 as referenced above.
This lock is a deadlock aware lock that will not deadlock if the class constructor for type A depends on the class constructor for type B, which in turn depends on the class constructor for type A. In that case, the class constructor lock will allow the class constructor for type B to observe the partially uninitialized state of type A, and would also allow type A to observe the partially initialized state of type B if initialization started with type B. This deadlock avoidance mechanism is robust, and works even in cases where the initialization of types A and B started on separate threads, but reasoning about the correctness of such class constructors is difficult, such that it is recommended that circular references in static constructors are to be avoided as a matter of good software engineering practice.
Standard thread synchronization primitives such as monitors, semaphores, mutexes, etc, do not participate in the deadlock avoidance scheme, and it is possible to build a deadlock in their presence, if they are not used in a safe manner.

I'll propose the following text based on the above comments:

The runtime calls a static constructor no more than once in a single application domain. Furthermore, that call is made in a locked region based on the specific type of the class. No additional locking mechanisms are needed in the body of a static constructor.

Please vote up or down, with additional suggestions.

As it's executed under a lock, might also be worth noting what to avoid:
Static constructor deadlocks:

So, to avoid the risk of deadlocks, avoid blocking the current thread in static constructors and initializers: don’t wait on tasks, threads, wait handles or events, don’t acquire locks, and don’t execute blocking parallel operations like parallel loops, Parallel.Invoke and PLINQ queries.