[feat] Is it possible to conditionally open dev-tools in production mode

Describe the problem
Some problems are too difficult to locate without dev-tools.
For example but not limited to:

The pictures are not displayed because of the wrong path is dynamically set in the production mode.

When the map is introduced,it is displayed in development mode but not displayed in production mode (very confused😢. google map is ok but baidumap fail. I'll try amap later)

CSP is configured incorrectly (didn't realize this at beginning but It is also one of the possible reasons why the picture is not displayed)

Describe the solution you'd like
By adding startup parameters or startup password, or simply let the developer be responsible for whether to open dev tools
Alternatives considered
No response
Additional context
No response

yes, add devtools feature flag in your Cargo.toml and call window.open_devtools whenever you want

you can also build a debug version of you app tauri build --debug which will builds your app in production mode but keeps access to some debug features including devtools


release-1.0 branch refers to missing go-control-plane branch

The revert_grpc_version_update branch was deleted from the go-control-plane repo, so dep can't download the dependencies: https://github.com/istio/istio/blob/release-1.0/Gopkg.toml#L115-L117
Only master exists here: https://github.com/envoyproxy/go-control-plane/branches
This was fixed in Istio master: https://github.com/istio/istio/issues/7985

Sorry if this was never fixed, 1.0 has reached EOL now though so I don't think this will be fixed


MERGE and REPLACE problem/question in ConflictResolution  or bug with REPLACE (JAVA)

Simple Storage 0.14.0
I have two folders
/movies/
/app/movies/
They have a lot of identical files, well, for example, there
/movies/filmAboutLove.mp4
/movies/horrorFilm.mp4
/app/movies/filmAboutLove.mp4
/app/movies/horrorFilm.mp4
I need update /app/movies/ folder and all files inside
That is, I need to replace, or add new files from the folder /movies/ to the folder /app/movies/
`
        DocumentFile f = DocumentFileCompat.fromFullPath(layoutContext, storage + "/movies");
        DocumentFile f2 = DocumentFileCompat.fromFullPath(layoutContext, storage  + "/app/movies");
        DocumentFileUtils.moveFolderTo(f, layoutContext, f2, true, null, new FolderCallback() {

        @Override
        public long onStart(DocumentFile param1DocumentFile, int param1Int, Thread workerThread)
        {
            return 1000L;
        }

        @Override
        public void onFailed(ErrorCode errorCode)
        {
        }

        @Override
        public void onParentConflict(DocumentFile param1DocumentFile, ParentFolderConflictAction action, boolean canMerge)
        {
            action.confirmResolution(ConflictResolution.MERGE);
        }

        @Override
        public void onContentConflict(DocumentFile destinationFolder, List<FileConflict> conflictedFiles, FolderContentConflictAction action)
        {
            handleFolderContentConflict(action, conflictedFiles);
        }

        public void handleFolderContentConflict(FolderContentConflictAction action, List<FileConflict> conflictedFiles)
        {
            askSolution(action, conflictedFiles, new ArrayList<FileConflict>(conflictedFiles.size()));
        }

        public void askSolution(FolderContentConflictAction action, List<FileConflict> conflictedFiles, List<FileConflict> newSolution)
        {
            FileConflict currentSolution = CollectionsKt.removeFirstOrNull(conflictedFiles);
            if (currentSolution == null)
            {
                action.confirmResolution(newSolution);
                return;
            }
            currentSolution.setSolution(FileCallback.ConflictResolution.REPLACE);
            conflictedFiles.forEach (it -> it.setSolution(currentSolution.getSolution()));
            newSolution.addAll(conflictedFiles);
            action.confirmResolution(newSolution);
        }

        @Override
        public void onReport(Report rep)
        {
            Log.d("DEBUG", String.format("Progress %f", rep.getProgress()));
        }

    });
}

`
Valid if you use code above you will have in folder /app/movies/ this list of files

filmAboutLove.mp4
filmAboutLove (1).mp4
horrorFilm.mp4
horrorFilm (1).mp4

When it was necessary to simply replace existing files, or add new folder or files
But if we think about that example, then we should get

filmAboutLove.mp4 (REPLACED)
horrorFilm.mp4 (REPLACED)

In my code, a function with a replacement, namely askSolution not work.
I guess it's because of ConflictResolution.MERGE, but while it says in the description to this, we should have skipped duplicate files rather than doing ConflictResolution.CREATE_NEW.
But what worries me more is that I can't. Replace files in identical folders, while being able to create new files and folders.

Will be fixed in 1.1.0. For now, just use 1.1.0-SNAPSHOT.
FYI, you are trying to replace all conflicted files without asking the user. Your askSolution() should be:
public void askSolution(
        FolderCallback.FolderContentConflictAction action,
        List<FolderCallback.FileConflict> conflictedFiles,
        List<FolderCallback.FileConflict> newSolution
) {
    newSolution.addAll(conflictedFiles);
    newSolution.forEach(it -> it.setSolution(FileCallback.ConflictResolution.REPLACE));
    action.confirmResolution(newSolution);
}

Also, on version 0.14.0 of the library, I noticed a bug, if you use moveFolderTo and specify a custom new folder name, then only files without directories will be transferred. Tried to contact you and inform through the social. networks. about it but couldn't.
Perhaps this has already been fixed in the new versions, but I decided to inform you all of a sudden that
 BUG DocumentFileUtils.moveFolderTo(f, layoutContext, f2, true, {CUSTOM_NAME}, new FolderCallback() { }
Now I will try updates with unpacking archives

Hmm, i can't use snapshot, repo it's not work

Wrong implementation of the Maven snapshot URL. Please read this: https://github.com/anggrayudi/SimpleStorage#overview

if you use moveFolderTo and specify a custom new folder name, then only files without directories will be transferred.

Also fixed in 1.1.0-SNAPSHOT. Please re-download/refresh your dependencies.

1.1.0 has been released.

1.1.0 has been released.

I described u maybe new bug in tg, with copyFile

The bug in copyFile() also fixed in 1.0.0

The bug in copyFile() also fixed in 1.0.0

1.1.0-SNAPSHOT don't work copyFile (without errors)
I will send you code. I try to backup file, if backup file it's not exist
            public void handleFolderContentConflict(FolderContentConflictAction action, List<FileConflict> conflictedFiles) {
                askSolution(action, conflictedFiles, new ArrayList<FileConflict>(conflictedFiles.size()));
            }

            private void askSolution(FolderContentConflictAction action, List<FileConflict> conflictedFiles, List<FileConflict> newSolution) {
                FileConflict currentSolution = CollectionsKt.removeFirstOrNull(conflictedFiles);
                if (currentSolution == null) {
                    action.confirmResolution(newSolution);
                    return;
                }

                DocumentFile BackUpFileFolder = DocumentFileUtils.findParent(currentSolution.getTarget(), layoutContext);
                if ((DocumentFileUtils.findFileLiterally(BackUpFileFolder, currentSolution.getTarget().getName() + ".bm")) == null)
                {
                    Log.d("zxc", "This works");
                    DocumentFileUtils.copyFileTo(currentSolution.getTarget(), layoutContext, BackUpFileFolder, new FileDescription(currentSolution.getTarget().getName() + ".bm"), new FileCallback() { });
                    Log.d("zxc", "This not work, onConflict empty");
                }
                currentSolution.setSolution(FileCallback.ConflictResolution.REPLACE);
                newSolution.add(currentSolution);

                askSolution(action, conflictedFiles, newSolution);
            }
        });
    }

nothing happens on the copyFile To function and the next log stops
as if something does not let him through, waiting for Promise
I tried to display onConflict, onFailed but it is not even called
maybe a new asynchronous task is needed not to write everything in one

I have tested copyFileTo() and it works. Your implementation is wrong. Putting DocumentFileUtils.copyFileTo() in the DocumentFileUtils.moveFolderTo()'s onContentConflict() may suspend the thread forever.


build(circleci): Save and link to CircleCI artifact

Summary
Deploy storybook as an artifact in CircleCI and link to it from a GitHub deployment
Related Issues or PRs

How To Test
Look at the PR and follow the link to the deployment
Screenshots (optional)

The Happo task also builds storybook, but I'm not sure how to save the copy it builds, so we're doing 2x the work for now. If it becomes a bigger deal, we can look into fixing/optimizing things

The Happo task also builds storybook, but I'm not sure how to save the copy it builds, so we're doing 2x the work for now. If it becomes a bigger deal, we can look into fixing/optimizing things

Just linking this open issue, which is relevant and I remembered seeing before: https://github.com/happo/happo.io/issues/86

My limited CI knowledge says this looks good. I'd recommend getting 👀 from someone more well versed in this if time permits, but I won't hold it up.

I worked with @eeeady on this, so I think we're good to go


When listing a Component's model, show the function signature of a value

If the value of a property of a Vue data model is a function. Vue Devtools currently just shows "Function" as the value. It would be neat if it showed the function signature instead (eg. "function someFunction(arg)").
You can get the signature by calling .toString() on the function and getting everything up to the closing parenthesis:
if (typeof value === "function") {
    const valueStr = value.toString()
    return valueStr.slice(0, valueStr.indexOf(")") + 1)
}

Definitely. PR are welcome!


Add tutorial for setting up GridEngine with Typescript

An example repo has been created: https://github.com/Annoraaq/grid-engine-ts-example


Option in version() to choose querystring or file name with hash

Hello,
It would be nice to be able to choose how mix version function handle the cache-busting.
Looks some people prefer querystring, but file names with the hash works very well for me.
What do you think?
Thanks.

I have this planned already.

Thanks Jeffrey.
@ricardogreis FYI - there's been plenty of tickets where this has been discussed - I've been guilty of bringing it up in another ticket before realising it's already been discussed as well.  Chrome seems to be so stuborn somehow and I have issues with cache busting nearly every time I deploy new js assets

When should we expect? https://github.com/JeffreyWay/laravel-mix/issues/1241#issue-262719770

+1 - any ETA on this at all?

+1

+1


Use std::promise for omp_node_event instead of busy wait

The old implementation of omp_node_event was using a spin lock. For CPU backends, this is not a good strategy as it can lead to reduced performance for simultaneously running kernels. This PR uses std::promise/future instead as the next best thing in order to mitigate such performance issues.

@hipsycl-ci-bot test as well please to be sure :)


Linting

Make sure all code that is already written passes the linter test.
Configuration on Codacy.

What linter do we want to use for this project?
eslint is configured in package.json and I added markdownlint in #72 enabling Travis for both of them.  I see accesslint is also being run on Pull requests and there's also a config file for Prettier in the repo that does'nt look like its being used. What about Codacy?
eslint is for JavaScript linting and is currently returning ALOT of errors.  Looking at the JS files in the repo, they contain HTML and that might be why there are so many errors..
Happy to try and clean it up if I can understand what we need

That would be great! @piet0024 and @ethanWallace can you connect with @smellems on this?


Review citation methods

In prep for a presentation, we should really look at the candidates from https://github.com/codemeta/codemeta and pick one schema that seems to be getting the most traction (check appearances on GitHub?)

Results in https://github.com/common-workflow-language/common-workflow-language/pull/364


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-07-31)
Features

replace tbl_islist to islist (f1e66c7)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


[$500] Web - ave the world: Hmm... it's not here when reloading the page after creating the #teacher-unit room

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the website
Click the + icon from LHN
Click Save the world
Click I know a teacher
Create a teacher unit
Click the header from the teacher unit
Reload the page
Wait and observe the result

Expected Result:
The Hmm... It's not here page shouldn't appear
Actual Result:
The Hmm... It's not here page appears
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/e5f58999-05ad-44d0-bac7-966dbad77ce8
https://github.com/Expensify/App/assets/93399543/dbc8a4a4-2e44-4a7f-b3a5-81bcb00dc74d
Expensify/Expensify Issue URL:
Issue reported by: https://expensify.slack.com/archives/C049HHMV9SM/p1694061131439169
Slack conversation: @misgana96
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0183fa65ac686f749f
Upwork Job ID:<PHONE_NUMBER>868602368
Last Price Increase: 2023-09-26

Issue OwnerCurrent Issue Owner: @marcochavezf

Proposal
Please re-state the problem that we are trying to solve in this issue.
Hitting refresh triggers the "The Hmm... It's not here page appears" from the teacher unit.
What is the root cause of that problem?
When you refresh the page being on the Teacher Unit Details Sidebar, An "OpenReport" write API is called. On this API being called, the backend is returning 403, which means '403 forbidden — you don't have permission to access this resource'  . [ See attached screenshot ]

On receiving this error, the function applyHTTPSOnyxUpdates handles it as a failure condition, hence the frontend considers that the user does not have access and shows the "The Hmm... It's not here page appears" page.
https://github.com/Expensify/App/blob/954c66ded309b1a11f87424b77b9a451ff338010/src/libs/actions/OnyxUpdates.ts#L35-L37
What changes do you think we should make in order to solve the problem?
The backend must not return a 403 for this request.
What alternative solutions did you explore? (Optional)
On frontend we can better manage the call of the OpenReport API.

Assigning myself here to track this bug

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/gianluca

@eVoloshchak have you had a chance to review the proposal?

@abekkala, I agree with @iamuddeshya's assessment, this is a back end issue
Could you make this internal so an engineer is assigned?

@abekkala, I agree with @iamuddeshya's assessment, this is a back end issue Could you make this internal so an engineer is assigned?

Is there anything I need to do? What's next for me?
Can help with backend if required. 🙂

@eVoloshchak and @iamuddeshya
@marcochavezf is an internal engineer already assigned
@marcochavezf do you agree this should be moved to internal for you to fix?

@abekkala Is the bounty claimable if this issue is to be moved to backend to be fixed?

@iamuddeshya

Is the bounty claimable if this issue is to be moved to backend to be fixed?

Yes, the [$50] reporting payout still applies!

No update

No update, focused on wave7 and traveling back home

I'm swamped with other high-priority items, I will come back to this one tomorrow

No update, but prioritizing this one for this week

Still wrapping up some priorities for wave7, no update

No update, tackling one issue by one

No update, focused on TU launch

No update

No update

No update

No update

No update

No update

No update

No update

No update

Moving this to weekly, swamped with other dailies.

No update

I'm heading out for the Thanksgiving holiday and will be back bright and early Monday morning.

No update

@marcochavezf  I'm going ooo though Jan 03.
I don't think there will be much movement through then so I'm keeping myself assigned as by the time I return this probably won't be at the payment stage for me to take action.
However, if things progress quickly you can reapply the Bug label to get someone one this to submit payment.
(I'll also be checking in on my GH's while I'm ooo)

No update

No update

Focused on wave issues, no update

not overdue. Marco is prioritizing wave issues.

No update

No update, focused on wave work

There were more changes to the public rooms, I'm going to apply the retest-weekly label to ensure if it's still an issue

Issue not reproducible during KI retests. (First week)

No longer reproducible, closing it out


Process to ID and review TAZs for possible splits for high growth areas

Requested by: CAMPO - Mike
Date: 08/2022
Priority: L
Additional details: ID by FY growth in Pop/HHs and possible average trip productions using a default average trip rate per HH (the last part of this is from Leta)

We believe this is better implemented as a manual process using the FY trip production output. You can select your upper limit and identify all TAZs above that limit.

I agree.


[$500] Text with single backtick code block is not rendered correctly

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.40-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Go to any chat.
Send a text with single backtick code block.

Expected Result:
The text with single backtick code block is rendered correctly.
Actual Result:
The text with single backtick code block appears with two boxes outside the text.
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/a4745a2f-0db6-4613-9ab5-8a99bdba96c6

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c1b63e23f853fcb5
Upwork Job ID:<PHONE_NUMBER>711943680
Last Price Increase: 2024-02-12

Proposal
Please re-state the problem that we are trying to solve in this issue
The text with single backtick code block appears with two boxes outside the text.
What is the root cause of that problem?
https://github.com/Expensify/App/blob/39ac35ad7b57007829b8d2768ef23b3b02ffea99/src/components/HTMLEngineProvider/HTMLRenderers/CodeRenderer.tsx#L46
It seems to be caused by the boxModelStyle styles passed to the InlineCodeBlock component.
What changes do you think we should make in order to solve the problem?
Remove the boxModelStyle styles passed to the InlineCodeBlock component.
Screenshots

MacOS: Chrome / Safari

Before
After

PROPOSAL
Please re-state the problem that we are trying to solve in this issue
The text with single backtick code block appears with two boxes outside the text. This is happening on all of the below platforms:

Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop

What is the root cause of that problem?
This is happening because 'backtick' is an essential part of most coding languages. Either when a backtick is typed with a text and passed to the backend code, or when the text with a backtick is printed on the screen, at any one or both of these instances, backtick is getting mixed with the code resulting in unexpected results. It's difficult to come up with a definite reason without debugging on all of the platforms.
What changes do you think we should make in order to solve the problem?
At the time, the text with backtick is passed on to the backend script, the backtick may be replaced by any other key. Similarly, when it is printed, this other key may be replaced by a backtick again.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~018df9103ffcdb09b7?viewMode=1

@kadiealexander I would like to suggest a $250 (half of $500) compensation for @ikevin127 since he helped by providing the correct RCA and pointing out the offending PR.
I'll remove @Santhosh-Sellavel since we didn't require a C+ review.

@kavimuru @m-natarajan can someone please test https://github.com/Expensify/App/pull/36360 on staging and confirm if this is resolved?

@Beamanator Tested just now on latest staging (v1.4.40-2).

MacOS: Chrome
https://github.com/Expensify/App/assets/56457735/4dd0009e-7270-41cf-bff3-7f5a3f192ddb

Yes agreed, it looks like it's working well on staging macos & iOS!

[x]  [@ikevin127] Determine if we should create a regression test for this bug.
[x]  [@ikevin127] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

No need for regression tests here as this was caused by TS migration https://github.com/Expensify/App/pull/35462 which was meant to only convert JS to TS, not change existing logic. More context here.

@ikevin127 is paid!


Use new release of WPILib

WPILib broke reverse compatibility (again) . This pull request updated the jar files and ported the code so it can be used with the new version of WPILib. Although this pull request can't automatically be merged, there are no actual merging conflicts.

Merge conflict


added the qualimap package links

Please don't remove the qt entry....
Thanks for your contribution!


How to use React to send a POST request to a Laravel application

Article checklist
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Have you placed your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines
This fixes #1620

Hello @kayere .
Thank you for the review. Am working on the conclusion.

@kayere done. Please check.

plagiarism check done

@Agusioma Great work overall - please go over the changes as there were some parts I found taking a double look (or difficult to understand).
Please include any applicable changes to future articles.
cc @kayere


chore: Release version 3.0.1

Hi
it seems https://github.com/callstack/repack/pull/255 was not released in this version, care to take a look ?

@meypod My bad, I forgot to release debugger-app package when I was publishing 3.0.1 release of repack. It should be available on npm now

Thank you


Upgrading existing installation

Sorry if this has been brought up before....
I used git clone initially to install, and as you can not use that on an already existing directory, what is the best way to upgrade to a new/beta version? My artisan has no update command.
Thanks

Usually:

git pull origin master
composer install -o --no-dev
php artisan migrate
rm -rf ./bootstrap/cache/*
Done :)

Lucky I cp'd the directory beforehand...

Generating optimized autoload files
php artisan optimize --force
PHP Fatal error:  Class 'McCool\LaravelAutoPresenter\LaravelAutoPresenterServiceProvider' not found in /var/www/Cachet/vendor/laravel/framework/src/Illuminate/Foundation/ProviderRepository.php on line 146
[Symfony\Component\Debug\Exception\FatalErrorException]
Class 'McCool\LaravelAutoPresenter\LaravelAutoPresenterServiceProvider' not found

This is the bootstrap/cache/* thing.


Change to RTM VS2017

/fyi @KevinRansom

@dotnet-bot test ci please

@dotnet-bot test ci please

This is good to merge.

@mmitche this broke all of our ci builds that depended on VS.


unable to hit api

When i launch the app, it's just showing empty screen on click of any icon, no data or loading is there, on android phones.

Please let me know, how to fix this

Thank you for reporting.
Probably caused by the following problem, but we do not currently know how to deal with it.
This problem may be improved by restarting the application.
https://github.com/DroidKaigi/conference-app-2021/issues/93

If restarting the app doesn't work, I would appreciate it if you could provide the logcat log for investigation. 🙏

I will close this issue once, so please let me know if there is anything else at any time.


Add checkbox to use JupyterLab

Probably should update the README.md and manifest.yml to clarify that JupyterLab can be used in addition to Jupyter Notebook, as well as CHANGELOG.md

Welp, I'm sold. Jupyter Lab has a dark theme. Serious review tomorrow.


Ensure resulting cube has correct dtype after maths operations

Closes https://github.com/SciTools/iris/issues/2528

@djkirkham - does this need targeting against master, or closing?


Smooth Scrolling looks terrible

Long story short, I'm casting a page I built (I'm self taught so please don't be too critical) to a Chromecast using Cast All The Things It contains the following....

And references:

It looks great in a browser, but when I cast it using Cast All The Things, it takes forever to load and looks terrible. Any suggestions? I've googled until I'm blue in the face, however, because I'm self taught, I'm not even sure if I'm googling the right things to be able to smooth scroll and see something quickly without really long load times.
Any alternate suggestions? Or any suggestions to make this work seamlessly? Thanks in advance!

Unfortunately we can't help because this isn't a CATT issue, since it appears that it casts correctly.


support IE 11 input type='color'

What problem does this feature solve?
a color picker support IE 11
What does the proposed API look like?
it is a COMPONENT or DIRECTIVE

This is the default behavior of HTML, and you can use angular color picker  instead.


document the code of the poroject to make it more readable

We need documentation for our code.
I propose following the google styleguide:
https://google.github.io/styleguide/cppguide.html
specially just go to how to comment functions. Most functions are NOT SELFEXPLANATORY. And i spent now hours over hours to find out what just single functions do. Let alone a set of functions, called in a specific order. This must stop.
The bare minimum for me would be: to add a comment, that defines what inputs the function can take (YES also those from inherited classes, why would you inherit multiple classes and then loose track of possibilities for your functions by searching code around because you cant find out what each subclass now has?)  and also some basic explanation of what the functions is supposed to do.
Obviously you don't write down, what the function does, which is already immediately obvious in the code. You more so write down what the meaning of the function is.
So for example for a function that says:
public void Activate(ChangeArgs args)
{
  this.labels.Clear();
  if (!editList.TrueForAll(e => e.Initialize(args)))
    return;
  editList.ForEach(e => e.Activate(args));
}

you would not need to explain that it looks for elements that are initialized and then activates them, you would instead explain waht activation means. What is an activated element and in what case would i call this function?
Ok, so i will add some comments after this release version, as i don't find time to now go through the code base and add a bunch of coments everywhere, but i will probably tidy up the code base a bit after the next release.
Hope to see more people following the basic guidelines of multiperson projects :)
lets do this!

Hahaha, I just think that's sooo ironic.
At work we have projects with thousands and thousands of lines of code, but only about a hundred lines of comments.

How is that ironic? Or does krarilotus work with you?
On Tue, 31 Mar. 2020, 5:11 pm Kimberly<EMAIL_ADDRESS>wrote:

Hahaha, I just think that's sooo ironic.
At work we have projects with thousands and thousands of lines of code,
but only about a hundred lines of comments.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Sh0wdown/UnofficialCrusaderPatch/issues/564#issuecomment-606421361,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALGGWTQYNVUDBM6Y5HGXYBLRKGCPLANCNFSM4LWHLL2A
.

We need to discuss this for the backend changes with the new Gui. So there will be no methods like this in future. More simple stuff you understand easily.

We are a developer team with no guidelines really right now, which is just not good for a project of this scale, which gets bigger and bigger. Having no guidelines or documentation would be detrimental for the future development of the patch.
Good code speaks for itself, but for good code, you need to have a common ground of guidelines, which i would just default to the google stileguide. The google stile guide covers alot of ground and in case of questions of how to write code or comments, its an easy thing to look it up.
@ByBurton maybe your work should then take some pages out of the book of our development, or maybe even with that amount of code, your projects have other forms of organization to keep track of everything. Either way, we need in and out of project doumentation.
I am happy to discuss constructive opinions on this topic and take all the developers comments into consideration to form a common ground.

@Krarilotus
For Clean code we should use the Code convention for C#:
https://docs.microsoft.com/de-de/dotnet/csharp/programming-guide/inside-a-program/coding-conventions
https://www.dofactory.com/reference/csharp-coding-standards
For Documentation of the Code we can use Comments that will get parse to xml, and with the xml we can create Document Files:
https://visualstudiomagazine.com/articles/2011/07/21/wcovb_xml-comments-part-1.aspx

Yes thats perfect, exactly like that! I vote for this @PodeCaradox


Make AccountHandle::get_alias_output() public

Description of change
modified AccountHandle::get_alias_output() visibility from pub(crate) to pub
Links to any relevant issues
Closes #214
Type of change

Enhancement (a non-breaking change which adds functionality)

How the change has been tested
Change checklist
Tick the boxes that are relevant to your changes, and delete any items that are not.

[ x] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that new and existing unit tests pass locally with my changes

This doesn't really look like an API that should be exposed tbh... Maybe we should add another one that just returns an Option output.

I could use client functions and retrieve Alias information from a node, the idea was to reuse code


Did my flutter upgrade fail?

I just tried to run flutter upgrade and I got this message:
sethladd-macbookpro:flutter_gallery sethladd$ flutter upgrade && flutter run --release
fatal: no upstream configured for branch 'package-assets'
Unable to upgrade Flutter: no upstream repository configured.
sethladd-macbookpro:flutter_gallery sethladd$

As a user, I'm left wondering:

What state am I in?
Did it fail?

Suggestion:
Assuming that the whole upgrade did indeed fail, let's print out how to mitigate this particular error (I'm actually not sure what to do here?) and at least print out "Upgrade failed".
Thanks!

You're also in a funny setup, whereby you've switched your git branch to one which does not have an upstream branch configured. :)  Normal flutter users won't have switched their git branch on their flutter checkout.

You can restore peace and justice using git checkout -b master upstream/master.

Thanks!
Would still love for the tool to print out an acknowledge if the command itself failed.

flutter git:(c59966290) $ flutter upgrade
Downloading Dart SDK from Flutter engine 520529dbba9b3048f5cc0e0268a274911f3887e6...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 44.3M  100 44.3M    0     0  15.9M      0  0:00:02  0:00:02 --:--:-- 15.9M
Building flutter tool...
Unable to upgrade Flutter: no upstream repository configured.

I just encountered this same message, as I had pulled down an older commit on master to test it (not master). Would be great to have more text in the tool to notify users about the state :)


Add pattern support to tag rules

Resolves #227
This adds a "patterns" configuration property to the allowed-tags and no-restricted-tags rules, which can be combined with the existing "tags" property to allow or forbid a list of patterns. This also changes the description of the allowed-tags in the README, since it was erroneous.

Thanks for the changes!


urllib.error.HTTPError: HTTP Error 403: rate limit exceeded

A header error occurs when the command is executed using a VPN network.

Hi, @tusharsadhwani
Isn't this a big problem?

Can you share the terminal output? I'll look into this.

❯ yen create venv38 -p 3.8
Traceback (most recent call last):
  File "D:\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "D:\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "D:\CIAN_proejcts\Github\httpfpt\venv\Scripts\yen.exe\__main__.py", line 7, in <module>
  File "D:\CIAN_proejcts\Github\httpfpt\venv\lib\site-packages\yen\cli.py", line 38, in cli
    python_version, python_bin_path = ensure_python(args.python)
  File "D:\CIAN_proejcts\Github\httpfpt\venv\lib\site-packages\yen\__init__.py", line 34, in ensure_python
    python_version, download_link = resolve_python_version(python_version)
  File "D:\CIAN_proejcts\Github\httpfpt\venv\lib\site-packages\yen\github.py", line 82, in resolve_python_version
    pythons = list_pythons()
  File "D:\CIAN_proejcts\Github\httpfpt\venv\lib\site-packages\yen\github.py", line 52, in list_pythons
    python_releases = get_latest_python_releases()
  File "D:\CIAN_proejcts\Github\httpfpt\venv\lib\site-packages\yen\github.py", line 36, in get_latest_python_releases
    with urlopen(GITHUB_API_URL) as response:
  File "D:\Python310\lib\urllib\request.py", line 216, in urlopen
    return opener.open(url, data, timeout)
  File "D:\Python310\lib\urllib\request.py", line 525, in open
    response = meth(req, response)
  File "D:\Python310\lib\urllib\request.py", line 634, in http_response
    response = self.parent.error(
  File "D:\Python310\lib\urllib\request.py", line 563, in error
    return self._call_chain(*args)
  File "D:\Python310\lib\urllib\request.py", line 496, in _call_chain
    result = func(*args)
  File "D:\Python310\lib\urllib\request.py", line 643, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: rate limit exceeded

❯ yen create venv38 -p 3.8
Traceback (most recent call last):
  File "D:\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "D:\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "D:\httpfpt\venv\Scripts\yen.exe\__main__.py", line 7, in <module>
  File "D:\httpfpt\venv\lib\site-packages\yen\cli.py", line 38, in cli
    python_version, python_bin_path = ensure_python(args.python)
  File "D:\httpfpt\venv\lib\site-packages\yen\__init__.py", line 34, in ensure_python
    python_version, download_link = resolve_python_version(python_version)
  File "D:\httpfpt\venv\lib\site-packages\yen\github.py", line 82, in resolve_python_version
    pythons = list_pythons()
  File "D:\httpfpt\venv\lib\site-packages\yen\github.py", line 52, in list_pythons
    python_releases = get_latest_python_releases()
  File "D:\httpfpt\venv\lib\site-packages\yen\github.py", line 36, in get_latest_python_releases
    with urlopen(GITHUB_API_URL) as response:
  File "D:\Python310\lib\urllib\request.py", line 216, in urlopen
    return opener.open(url, data, timeout)
  File "D:\Python310\lib\urllib\request.py", line 525, in open
    response = meth(req, response)
  File "D:\Python310\lib\urllib\request.py", line 634, in http_response
    response = self.parent.error(
  File "D:\Python310\lib\urllib\request.py", line 563, in error
    return self._call_chain(*args)
  File "D:\Python310\lib\urllib\request.py", line 496, in _call_chain
    result = func(*args)
  File "D:\Python310\lib\urllib\request.py", line 643, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: rate limit exceeded

@wu-clan this issue has been fixed in the 0.4.2 release. Can you check?

I've built and tested it locally and it looks fine, it's working!
But the json file adds 1MB directly to the package 😱

Yeah, unfortunate, but I think it's fine.


ESP 协议实现服务端

Is your feature request related to a problem? (你需要的功能是否与某个问题有关?)
Describe the solution you'd like (描述你期望的解决方法)
Describe alternatives you've considered (描述你想到的折衷方案)
Additional context/screenshots (更多上下文/截图)

属于内部私有协议，暂时没开源的必要


INT-B-20477

Agility ticket
Summary
Created a new endpoint to cancel moves.
How to test

Navigate to http://officelocal:3000/swagger-ui/ghc.html#/move/moveCancellation
Grab a move id from the db and execute the endpoint
Check database that move status is now cancelled

So silly question, and I know technically it can be canceled or cancelled.  But did we ask the PO which they prefer?  I feel like I'm used to seeing Canceled with 1 'l' more, but maybe it's me.  According to google, the 1 'l' is he American preference, so maybe that's why.

So silly question, and I know technically it can be canceled or cancelled. But did we ask the PO which they prefer? I feel like I'm used to seeing Canceled with 1 'l' more, but maybe it's me. According to google, the 1 'l' is he American preference, so maybe that's why.

And interestingly enough, google says although canceled with 1 'l is preferred, cancellation, with 2 'l's is standard.

So silly question, and I know technically it can be canceled or cancelled. But did we ask the PO which they prefer? I feel like I'm used to seeing Canceled with 1 'l' more, but maybe it's me. According to google, the 1 'l' is he American preference, so maybe that's why.

And interestingly enough, google says the spelling for cancellation, with 2 'l's is standard.

@loganwc ^^^ 😬

Logan, don't kill me, but 'cancellation' was actually correct, with 2 'll's.  I was concerned about 'cancelled' having 2 'll's.  Don't blame me!  That's the english dictionary for you, I even googled it several times before I first commented.  Sorry, maybe it wasn't clear.

What am I doing wrong


Adding HierarchyNodeExporter to fix XF's not being editable

This fixes the XF page not being editable in the site admin for SPA pages.

@niekraaijmakers Can u please explain why HierarchyNodeExporter is required ?

the WCM editor requires “:hierarchyType”:“page” to be there in the model JSON for some reason. without it, the experience fragment editor in SPA context won’t start without that property for some reason.. why I don’t know exactly


Update to the name function urllib.urlopen

According to the https://docs.python.org/3/library/urllib.request.html#module-urllib.request the name of the function has been changed
to urllib.request.urlopen

Instead of use urllib, just use requests as in the other pieces here.


Create Generic Pages “recommended-products”

Automatically generated by Netlify CMS

👷 Deploy Preview for ltcblog processing.

Name
Link

🔨 Latest commit
13651f66544d91300d17d55cc83dbcc04b055601

🔍 Latest deploy log
https://app.netlify.com/sites/ltcblog/deploys/65e740c040a17700089e2d83


Add release link to readme

replace the link with this, it will install the latest on click https://github.com/JensvandeWiel/ArkAscendedServerManager/releases/download/latest/Ark-Ascended-Server-Manager-amd64-installer.exe

Are you sure? :P

Used this link
https://github.com/JensvandeWiel/ArkAscendedServerManager/releases/latest/download/Ark-Ascended-Server-Manager-amd64-installer.exe


use simpler bean registration pattern for extensions

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Aries-cdi-pr/37/


Improve documentation for the tests section on the "Writing A Plugin" page

Based on the location of the test folder in the source code of the project, the path should be test/plugins/cool-linter.test.ts instead of tests/plugins/cool-linter.test.ts.

Thanks for reporting this!

:rocket: This issue has been resolved in v3.12.0. See Release 3.12.0 for release notes.

:rocket: This issue has been resolved in v4.0.0. See Release 4.0.0 for release notes.


[ko] Update deprecated kubectl command in apparmor doc

This updates example kubectl exec commands to avoid warnings. It is a follow-up change to kubernetes#37221.
Running kubectl exec <pod_name> <command> is deprecated:
kubectl exec [POD] [COMMAND] is DEPRECATED and will be removed in a future version. Use kubectl exec [POD] -- [COMMAND] instead

It should be: kubectl exec <pod_name> -- <command>

Thanks @tbalthazar
Sorry for this late approval. Korean localization uses own development branch (dev-1.25-ko.1) and milestone. So, some PRs to main branch for Korean contents can be delayed.
/approve


Support For Multiple Embeds

Hello all,
This PR adds support for multiple embeds as discussed in #25. I've rolled the changes I've made for this into the original shortcode, and added a new partial responsible for loading pdf.js and the shortcode styles.
Give it a test and let me know what you think.

@anvithks Any thoughts on this?

@anvithks Any thoughts on this?

I am sorry but I haven't been able to test this yet.
I will try to spend some time over the weekend and merge the PR if all OK.
Thanks @ctrezevant .

Hi @ctrezevant , thanks for sharing this code!
PR #32 is a proposal mixing your code and this little trick: splitting the code into multiple files can be avoided. See 0b75911 and this discussion on discourse.gohugo.io.
I've tested #32 and it works fine. If it's not working for you, please double-check the commits https://github.com/anvithks/hugo-embed-pdf-shortcode/pull/32/commits/f7a8fecb847c1845d35683c7a85eb82f751aabd3 and https://github.com/anvithks/hugo-embed-pdf-shortcode/pull/32/commits/e3642f0d37a806d5735e5f670e3e31e1f4da6841. They are required to make this shortcode work with my theme (Papermod) but your setup may be different

Glad to see this moving forward! Everything looks solid after a cursory review.

@chtzvt Closing this as it was added to #32 and merged with your commits.


[image_picker] Unable to pick a WebP image on iOS

Steps to Reproduce

Run plugin example application on iOS.
Try to pick a WebP image.

Nothing happens. The image is not loaded. Neither Flutter not Xcode provide useful output in the logs.
No exceptions. The future is not completed.
I would expect PlatformException if we cannot process this kind of images on iOS.

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.2 20G95 darwin-arm, locale en-UA)
    • Flutter version 2.2.3 at /Users/oandreyev/Tools/flutter
    • Framework revision f4abaa0735 (6 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/oandreyev/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] IntelliJ IDEA Community Edition (version 2021.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 59.0.4
    • Dart plugin version 212.4746.57

[✓] Connected device (2 available)
    • Oleg’s Dev iPhone (mobile) • 46424eb83323aec8a7172c9bbe20446bb493b4d2 • ios            • iOS 14.6
    • Chrome (web)               • chrome                                   • web-javascript • Google Chrome 92.0.4515.131

I'm having the same issue. Is there some workaround?

@TrGiLong Hi, did you solve this problem? It's still reproducable.

Hi, @burcus
I have fixed it but the branch isn't been merged into main branch. I have to write some extra code to pass their requirements so it will last time. The PR is here
You can also use my branch for your project while waiting. As far as i know there aren't any important changes in package :D

Thank you for feedback @TrGiLong. How long do you think it would take to merge with the main branch?

@burcus
I can't say how long it will exactly take because it depends on the reviewer. I will try to fix the code today and request a new review for it 😅

Hope this error will fix immediately. I think it's critical because there's no chance for catching the error. Waiting for review good luck.


chore(stencil): change reloadStrategy to pageReload

:tada: This PR is included in version 1.13.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Resolve sale_price_effective_date for Commerce's Scheduled Update (Staging)

The field sale_price_effective_date for the period of validity for a product's special price is currently set if the product attributes special_from_date and special_to_date are set:
https://github.com/magmodules/magento2-channable/blob/bc3fa007b6949f871ead088fb1a2c086b9f645af/Helper/Product.php#L953-L957
Adobe Commerce replaces this mechanism by its Scheduled Update (magento_staging) functionality. The mentioned attributes are not set, since it is solved differently.
This leads to unavailability of sale_price_effective_date. We need this date range in Channable.

Thank you for your patience in awaiting our response but we couldn't reproduce this issue with sale_price_effective_date issues on the Commerce edition. Unfortunately, we were unable to reproduce the issue. Could you please reach out to us through our contact form so that we can work with you to resolve this matter further?

Hello @nige-one, any updates on this issue? Alternatively, can we consider it closed? Thank you!

Appreciate your message, @nige-one . Since we couldn't reproduce this issue, I'm closing it. Thank you!


Update alsa to 0.5.0

Since nix that is dependency crate is old version, midir cannot build for riscv64gc.
riscv64gc support is added by nix 0.18. So I would like to update alsa to 0.5.0

moving to https://github.com/Boddlnagg/midir/pull/101


Missing data in Automate UI

Cookbook version
2.3.5 and 2.4.0
Chef-client version
12.13.37
Platform Details
Ubuntu
Scenario:
Demonstrate Chef Automate.  No data shows up in the UI for chef runs.
Steps to Reproduce:
Contact SA team in Chef Slack.  We have an environment quarantined that is showing the problem.
Expected Result:
You'll see the UI has no data.
Actual Result:
No Data in the UI

issue is intermittent.
scar-pto-in-nyc [1:06 PM]
@vj actually it looks like both 2.3.5 and 2.4.0 are intermittently broken

[1:06]
we've had four instances of 2.3.5 break in the past two days

[1:06]
i think i'm the only one who tested 2.4.0 so far, also fails

[1:09]
what's really odd is it's intermittent

[1:09]
we haven't tracked down what is choking up the data collector

vj [1:10 PM]
huh.

scar-pto-in-nyc [1:11 PM]
it's really weird

vj [1:11 PM]
do you have a sample role you’re using so we can try to reproduce?  is it always on the same certain profiles?

scar-pto-in-nyc [1:11 PM]
so with the one i just did upgraded to 2.4.0, ran chef client over, still no data.

[1:11]
spun up a fresh environment, worked out of the box

[1:11]
they are bit-for-bit identical

vj [1:11 PM]
so weird. ok.

scar-pto-in-nyc [1:12 PM]
i suspect some kind of variable data

[1:12]
a date stamp or random number perhaps

[1:12]
is sneaking into the data stream

[1:12]
or i could just be talking out of my butt, i have no idea

[1:12]
thats where we could use some help

Sean, my research showed no issue in the audit cookbook here.
I added more details in the internal ticket(CMP-166), but this issue is related to ElasticSearch missing our defined mapping for index insights-2017.03.10.


Powershell  in Visual code 1.33

Issue Type: Bug
Powershell doesn't work with VC 1.33
VS Code version: Code 1.33.0 (0dd516dd412d42323fc3464531b1c715d51c4c1a, 2019-04-04T15:14:28.026Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5820K CPU @ 3.30GHz (12 x 3300)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.92GB (27.49GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

vscode-color
ans
0.4.5

vscode-django
bat
0.19.0

lit-html
bie
1.11.0

flask-snippets
cst
0.1.0

vscode-eslint
dba
1.8.2

code-runner
for
0.9.7

vscode-firefox-debug
hbe
1.7.9

python
ms-
2019.3.6352

html5-boilerplate
sid
1.0.3

vscodeintellicode
Vis
1.1.5

jinja
who
0.0.8

html-css-class-completion
Zig
1.18.0

Are you using Powershell in the editor or from the terminal?

I'm using it in the editor.

From the extension list, it doesn't look like you have powershell installed: https://marketplace.visualstudio.com/items?itemName=ms-vscode.PowerShell
Did you install it previously?
Also, what exactly doesn't work?

I got it working, don't know what happened :(    Sry


Add files via upload

@danokn I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host ckpsiedlce.onmicrosoft not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


How proxy for Clickhosue

That's an interesting project!

When I use clickhouse, how do I add the username and password in the config file ? or other place?
Thanks!

Hi @tangxueming Trickster is a reverse proxy cache, so the originating request that is proxied through trickster should include the credentials, and they will be passed through. Trickster only needs to be configured with the upstream clickhouse endpoint URL.


Add CHANGELOG to repository

Summary
Nowadays, most open-source projects keep a changelog in the repository for easy accessing and searching for changes between versions.
Gridsome is currently missing any form of a changelog. There is no changelog in the releases on GitHub and not in the repository
Basic example
https://keepachangelog.com/en/1.0.0/
Motivation
Git commit history alone isn't easy enough to read and understand changes made between releases. Users of a library or framework want to easily scan for new features, fixes and breaking changes.

Yeah it'd be great if the changelog could be made more obvious, ideally in the root of the repo.
Thanks


The @actions action is running twice, even for non-auth requests

It seems that we are calling it from Toolbar and View at the same time.

@sneridagh Is this still an issue?

Is this still an issue?

yes


Problem with file video

After selecting the video quality, the download starts;
After downloading all the files I get this message and in my folder videos I have only a lot of small file, but not an mp4 video

Use the nightly build of ffmpeg:
https://ffmpeg.zeranoe.com/builds/win64/static/ffmpeg-20200309-608b8a8-win64-static.zip

solved, thanks


Tanishq ppp

Codecov Report

Merging #300 (0973463) into master (52dced5) will increase coverage by 5.64%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #300      +/-   ##
============================================
+ Coverage     65.70%   71.35%   +5.64%
- Complexity      837      913      +76
============================================
  Files           135      135
  Lines          2782     2789       +7
  Branches        340      340
============================================
+ Hits           1828     1990     +162
+ Misses          837      676     -161
- Partials        117      123       +6

Impacted Files
Coverage Δ

...ss/model/order/OrderContainsKeywordsPredicate.java
76.92% <0.00%> (-7.70%)
:arrow_down:

...seedu/address/logic/commands/ListOrderCommand.java
100.00% <0.00%> (ø)

.../address/logic/parser/SortOrdersCommandParser.java
91.66% <0.00%> (+0.75%)
:arrow_up:

.../seedu/address/logic/commands/AddOrderCommand.java
86.66% <0.00%> (+0.95%)
:arrow_up:

src/main/java/seedu/address/model/Date.java
89.28% <0.00%> (+3.57%)
:arrow_up:

...a/seedu/address/model/person/UniquePersonList.java
93.02% <0.00%> (+4.65%)
:arrow_up:

...ava/seedu/address/model/order/UniqueOrderList.java
69.23% <0.00%> (+5.76%)
:arrow_up:

...rc/main/java/seedu/address/model/order/Amount.java
78.57% <0.00%> (+7.14%)
:arrow_up:

src/main/java/seedu/address/model/AddressBook.java
88.46% <0.00%> (+7.69%)
:arrow_up:

src/main/java/seedu/address/model/order/Order.java
72.72% <0.00%> (+9.65%)
:arrow_up:

... and 11 more

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 52dced5...0973463. Read the comment docs.


try to get 1 million data from cassendra but only able to fetch 5000

getRepositoryType().dsl().select().allColumns_From(provider).without_WHERE_Clause().getList();
this code show only 5000 data

You are very likely using cassandra auto-paging option (Ref)

You should use an Iterator<???> to fetch all rows rather than using getList() because there is a default limit of 5000 or something similar set at the driver side, which corresponds to 1 page worth of data

Iterator<Entity> iterator = getRepositoryType().dsl().select().allColumns_From(provider).without_WHERE_Clause().iterator()
while(iterator.hasNext()) {
 Entity entity = iterator.next();
 ...

}


nx: init at 1.0.0

Description of changes

Initialized nx at version 1.0.0

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

error: builder for '/nix/store/zj2hwwzdlm4wgdqy169f71ss7ldkwbqx-python3.10-randomname-0.1.5.drv' failed with exit code 2;
       last 10 log lines:
       >   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
       >   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
       >   File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
       >   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
       >   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
       >   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
       >   File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
       > ModuleNotFoundError: No module named 'setuptools'
       >
       >
       For full logs, run 'nix log /nix/store/zj2hwwzdlm4wgdqy169f71ss7ldkwbqx-python3.10-randomname-0.1.5.drv'.
error: 1 dependencies of derivation '/nix/store/q92k9gh82vk303wcikw6vcjsd8yb63lp-python3.10-nx-1.0.0.drv' failed to build

error: builder for '/nix/store/zj2hwwzdlm4wgdqy169f71ss7ldkwbqx-python3.10-randomname-0.1.5.drv' failed with exit code 2;
       last 10 log lines:
       >   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
       >   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
       >   File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
       >   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
       >   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
       >   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
       >   File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
       > ModuleNotFoundError: No module named 'setuptools'
       >
       >
       For full logs, run 'nix log /nix/store/zj2hwwzdlm4wgdqy169f71ss7ldkwbqx-python3.10-randomname-0.1.5.drv'.
error: 1 dependencies of derivation '/nix/store/q92k9gh82vk303wcikw6vcjsd8yb63lp-python3.10-nx-1.0.0.drv' failed to build

That's... strange? That's the behaviour I've seen with Wheels disabled, which is why I've set preferWheels in the derivation. It's built fine on my PC and OfBorg seems to be happy too... 🤔

error: builder for '/nix/store/zj2hwwzdlm4wgdqy169f71ss7ldkwbqx-python3.10-randomname-0.1.5.drv' failed with exit code 2;
       last 10 log lines:
       >   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
       >   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
       >   File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
       >   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
       >   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
       >   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
       >   File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
       > ModuleNotFoundError: No module named 'setuptools'
       >
       >
       For full logs, run 'nix log /nix/store/zj2hwwzdlm4wgdqy169f71ss7ldkwbqx-python3.10-randomname-0.1.5.drv'.
error: 1 dependencies of derivation '/nix/store/q92k9gh82vk303wcikw6vcjsd8yb63lp-python3.10-nx-1.0.0.drv' failed to build

That's... strange? That's the behaviour I've seen with Wheels disabled, which is why I've set preferWheels in the derivation. It's built fine on my PC and OfBorg seems to be happy too... 🤔

Also to note - this has been tested on both standard NixOS aarch64 and nix-on-droid

Note that there is another nx that is reasonably popular, especially compared to the nx in this PR which is only a week old. May cause confusion.

Indeed, I knew about this one only.
Could we get a namespacing of the form nix-nx ? And people can alias nx if they wish for.

@not-my-segfault is this still something you're interested in getting along? There may yet be work to be done, but if not we can just close the PR.

This is fine to be closed for now. I might re-submit sometime into the future


BUG: fail to build the docs in Chinese

Describe the bug
fail to build the docs in Chinese，but can  build the docs in English.

To Reproduce
Python version: Python 3.9
Xorbits version: 0.3.0
Crucial packages:None
Full stack of the error:

Sphinx error:
Builder name html_zh_cn not registered or available through entry point

@huangyong2002 Thanks for reporting this issue!
Looks like there's something wrong with the 'make.bat'. Are you interested in solving this problem?

Okay, I'll try to fix it.

@huangyong2002 Thanks for reporting this issue!
Looks like there's something wrong with the 'make.bat'. Are you interested in solving this problem?

Okay, I'll try to fix it.

Great! You can replay take to this issue and GitHub will assign this issue to you.

take

@huangyong2002 Thanks for reporting this issue!
Looks like there's something wrong with the 'make.bat'. Are you interested in solving this problem?

Hi, we can replace the "make html_zh_cn" command with the "sphinx-build -b html -D language=zh_CN source build/html_zh_cn" command to build Chinese documentation #521


Get R check passing

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.21 :tada:

Comparison is base (486605b) 72.80% compared to head (c31d3e3) 73.02%.

:exclamation: Current head c31d3e3 differs from pull request most recent head 999e692. Consider uploading reports for the commit 999e692 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
+ Coverage   72.80%   73.02%   +0.21%
==========================================
  Files          11       11
  Lines         923      923
==========================================
+ Hits          672      674       +2
+ Misses        251      249       -2

Impacted Files
Coverage Δ

R/attributes-concept.R
65.90% <ø> (ø)

R/attributes-nested.R
30.00% <ø> (ø)

R/collectCodesetId.R
87.96% <ø> (ø)

R/criteria.R
62.65% <ø> (ø)

R/exit.R
84.44% <ø> (ø)

R/query.R
63.15% <ø> (ø)

R/cohort.R
77.57% <100.00%> (+2.09%)
:arrow_up:

R/conceptSet.R
70.00% <100.00%> (-0.12%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


            (
                ActionChains(self.parent.driver)
                .move_to_element_with_offset(self._element, 2, 2)
                .perform()
            )

        # Apply the monkey patch for Firefox WebDriverElement
>       firefox.WebDriverElement.mouse_over = mouse_over
E       AttributeError: module 'splinter.driver.webdriver.firefox' has no attribute 'WebDriverElement'

.direnv/python-3.9.12/lib/python3.9/site-packages/pytest_splinter/splinter_patches.py:21: AttributeError
==================================================================================== short test summary info ====================================================================================
ERROR test_file.py::test_file - AttributeError: module 'splinter.driver.webdriver.firefox' has no attribute 'WebDriverElement'
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
======================================================================================= 1 error in 0.63s ========================================================================================

Temporary workaround
patch_webdriverelement can be disabled by overriding fixture
from pytest_splinter.webdriver_patches import patch_webdriver

@pytest.fixture(scope="session")
def browser_patches():
    patch_webdriver()

Patch works! @kalekseev , you're a life-saver @kalekseev , thanks!
It looks like WebDriverElement was moved to splinter.driver.webdriver module and BaseWebDriver uses it as self.element_class. I don't think that monkey-patching BaseWebDriver is the way to go - on the other hand if there's a limitation "single test session == single web driver" it should be okay.
If any pytest-dev is willing to provide me support on this, I can provide a PR. For the simpler case :-)

@mpasternak you can try and test if that monkeypatch is still needed for firefox https://github.com/pytest-dev/pytest-splinter/blob/0ee5590c469b27221e7a93980cc163a195a42014/pytest_splinter/splinter_patches.py#L12-L18
It's probably not needed because https://github.com/cobrateam/splinter/blob/master/docs/news/0.7.0.rst#features

Support for mouse_over, mouse_out in Firefox driver.

In that case you can safely remove patch_webdriverelement

This should be fixed for now in trunk. Unfortunately ATM #163 I'm unable to run test, because all I get is "flask" webriver...

@kalekseev as the CI is working again -- please check if the recent changes in the codebase fixed this problem. Is your patch still needed?

pip install https://github.com/pytest-dev/pytest-splinter/archive/master.tar.gz works

@kalekseev thank you for both the problem report and the initial patch idea, as it saved my life. Looks like it is no longer relevant - closing it. Thanks!


Possible error in the calculation of the log probabilities in SAC

https://github.com/openai/spinningup/blob/038665d62d569055401d91856abb287263096178/spinup/algos/pytorch/sac/core.py#L53-L67
In line 64 of the code above, the squashing function (tanh) is applied to pi_action, producing an action pi_action which lives in the interval [-1,1]. In line 60, logp_pi is set to the corresponding log probability of such action.
However, in line 65 pi_action is multiplied  by self.act_limit, which is a scaling factor. Now pi_action lives in the interval [-self.act_limit,self.act_limit]. According to the change of variables formula, also the corresponding logp_pi should change. Assuming that self.act_limit > 0, I think that the correct result would be obtained by adding, afterline 60, the following:
logp_pi -= np.log(self.act_limit)

I believe most environments tested have act_limit=1, so it would be the same. If act_limit is not one, then you are correct.


Microsoft.NetworkCloud Public Preview for 2024-05-01-preview API version

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[X] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been created in adherence to OpenAPI specs PR creation guidance).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Private azure-rest-api-specs-pr related PRs:
This version (2024-05-01-preview):
https://github.com/Azure/azure-rest-api-specs-pr/pull/18113
Intermediate private preview versions:
(2024-03-01-preview):
https://github.com/Azure/azure-rest-api-specs-pr/pull/17224
(2023-12-01-preview):
https://github.com/Azure/azure-rest-api-specs-pr/pull/16516,
https://github.com/Azure/azure-rest-api-specs-pr/pull/16856,
https://github.com/Azure/azure-rest-api-specs-pr/pull/17104
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[X] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[X] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

Closing as this public preview is not being pursued at this time.


ENH: Docs aligned with updated PsychoJS demo

The changes to the docs reflect these updates to the demo:

Import MouseView ES6-style
Separating drawing the overlay from showing/hiding it via opacity. See https://github.com/u01ai11/MouseView.js/issues/17

Thanks! No problems at all with importing via ES6. Which is quite
unexpected TBH: I just tried it out and yeah, worked everywhere.
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Mon, Apr 5, 2021 at 8:46 PM Alex Anwyl-Irvine @.***>
wrote:

Merged #19 https://github.com/u01ai11/MouseView.js/pull/19 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/u01ai11/MouseView.js/pull/19#event-4553119749, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACESG4EVSJN5S2X2R2XIXVDTHIHS3ANCNFSM4Z7ZYGXA
.


Add support for component ipfs

Blocked by https://issues.apache.org/jira/browse/CAMEL-13173

@tdiesler FYI for when this comes to be merged to master - I had to add a little fix to the IPFS integration tests on the prod side:
https://issues.jboss.org/browse/ENTESB-10509


Support client certificate authentication

Modified the openssl wrapper to add functionality for authenticating
client certificates.  Added to the example set for how to use certain
functions.

I'm not so sure about this. OpenSSL has a very long list of functions, and I do not want to add every single one of them to the AMQP-CPP library.
The TcpHandler::onSecured() method is called when the connection has been setup and can be overridden in user space to check if the connection is secure enough for your liking. Is this interface currently not powerful enough for you? Maybe we need some extra virtual methods in the TcpHandler class before the connection is set up to allow applications to further initialize the SSL structure?

I need to do both server and client authentication, and the call to onSecured() in the handler occurs too late in the process (after the TLS handshake).  This means the client will need a key/cert pair during the handshake, and be able to load a CA certificate for verification purposes.  It also needs access to set the verification mode, supply the callback, load the peer cert, and get the verification results.
Currently, I have supplied these functions via the OpenSSL wrapper (which seemed like the logical place), and am using this functionality at the first point in which it would be available for use - right after the SSL object is created in the SslWrapper.  This follows the example left in the comment in the SslWrapper constructor.  The LibEvHandler onSecured() method has been overridden to get the peer certificate and then check the verify result using the SSL object.
Somehow the application will need access to these lower level OpenSSL functions.  Whether that functionality is supplied via the OpenSSL wrapper itself, or via a modified handler, is a good question.  Having the ability to initialize the SSL structure from outside the library, before the TLS handshake occurs, would be of value.

I am not going to merge this one. But what could be interesting, is to add an additional method to the TcpHandler class that allow user space code to create and initialize their own SSL_CTX and SSL structures. This would give maximum flexibility to setup secure connection to your own liking:
/**
 *  Method that is called by AMQP-CPP that allows user space applications to create
 *  and initialize their own openssl SSL* structure that is going to be used to secure
 *  the connection. The default implementation simply calls the SSL_CTX_new() and
 *  SSL_new() functions, but applications that want to (for example) share contexts
 *  between connections, or add their own initialization routinges, can override this
 *  method and do their own initialization.
 *  @param  connection   The connection to be initialized
 *  @return SSL*         The SSL* structure to be used
 */
SSL *TcpHandler::onSecure(TcpConnection *connection)

Do you think this would be useful, and is it possible to send in a pull request for this?

Not merging this one, waiting for a better solutions.


[BUG] Evalating many small expression uses too much memory

Bug description
Evalating many small expression uses too much memory, is there a memory leak?
Steps to reproduce
The following code uses more than 10GB memory, why? In ideal case, it only allocates 1 variable.
#include "pocketpy.h"
using namespace pkpy;

char buf[1024];

int main(){
    // Create a virtual machine
    VM* vm = new VM();

	vm->exec("a = 0");
	for (int i = 0; i < 200 * 1000 * 1000; ++i)
		vm->exec("a = a + 1 # intersting loooooooooooooooooooooooooooooooong comment");

    // Dispose the virtual machine
    delete vm;
    return 0;
}

Compiled with code from v2.0 branch with command g++ -std=c++17 -O1 -frtti test.cpp $(find src/ -name "*.cpp") -Iinclude -DPK_ENABLE_OS=1
Environment information
You need to provide these strings:

v2.0 branch (a53d6f1621afebd6828301463d0e9d787ae51ba5)
linux
64-bit

Thanks for your report. I will investigate this soon.
May I know what kind of project you are working on with pkpy in use?

Hi. Currently, GC triggers on function calls. There is no function calls in your code so memory won't be freed.

I noticed a leak of PoolExpr.
PoolExpr: 8.00 KB (used) / 8.00 KB (total) - 14.25 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 total arenas
  - 0x7f5861dfd010: 0.07 MB (used) / 0.25 MB (total)

PoolExpr: 8.00 KB (used) / 8.00 KB (total) - 125014.25 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 total arenas
  - 0x7f5861dfd010: 0.07 MB (used) / 0.25 MB (total)

PoolExpr: 8.00 KB (used) / 8.00 KB (total) - 250014.25 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 total arenas
  - 0x7f5861dfd010: 0.07 MB (used) / 0.25 MB (total)

PoolExpr: 8.00 KB (used) / 8.00 KB (total) - 375014.25 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 total arenas
  - 0x7f5861dfd010: 0.07 MB (used) / 0.25 MB (total)

May I know what kind of project you are working on with pkpy in use?

No specific project for now, I'm just a random CS student interested in compilers and tries to learn something from pkpy. I noticed this incredible work from DBinary's QQ group, where you mentioned it.

There is no function calls in your code so memory won't be freed. You can invoke GC manually via vm->heap.collect().

I updated the code to:
#include "pocketpy.h"
using namespace pkpy;

char buf[1024];

int main(){
    // Create a virtual machine
    VM* vm = new VM();

    vm->exec("a = 0");
    for (int i = 0; i < 200 * 1000 * 1000; ++i){
        vm->exec("a = a + 1 # intersting loooooooooooooooooooooooooooooooong comment");
        if (i % 200'000 == 0) {
            vm->heap.collect();
            printf("GC at i = %d\n", i);
        }
    }
    // Dispose the virtual machine
    delete vm;
    return 0;
}

It does not help much, it uses more than 10GB memory at i = 71000000, where I interrupted it.
I investigate the problem a little by attaching GDB to it. It looks like for some reason (maybe stacktrace?), pkpy keeps a clone of the source evaluated and never frees it, which costs up to 4.7GB memory at i = 71000000. If this is intend behavior, this issue shall be transformed into a feature request instead of bug report (because the memory is still kept by the VM object and will be free after delete vm).
In application scenarios such as game developing, it is extremely easy for pkpy users to write something like this:
// Game loop
while (true) {
    // other logic
    ...
    // run script
    vm->exec("emit_event('tick')");
}

This will leak memory slowly and finally causing the game to crash. Maybe we should mention this pattern in the document and warn users not to wrote code like so.

This is not a feature. Source is expected to be freed after exec.

Well, I have pushed a fix for expr leak.
PoolExpr: 8.00 KB (used) / 8.00 KB (total) - 6.88 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 arenas
  - 0x7ff6bc5bb010: 0.07 MB (used) / 0.25 MB (total)

PoolExpr: 8.00 KB (used) / 8.00 KB (total) - 6.88 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 arenas
  - 0x7ff6bc5bb010: 0.07 MB (used) / 0.25 MB (total)

PoolExpr: 8.00 KB (used) / 8.00 KB (total) - 6.88 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 arenas
  - 0x7ff6bc5bb010: 0.07 MB (used) / 0.25 MB (total)

PoolExpr: 8.00 KB (used) / 8.00 KB (total) - 6.88 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 arenas
  - 0x7ff6bc5bb010: 0.07 MB (used) / 0.25 MB (total)

I am refactoring the module. And there are other known leaks about expr. However, I think it won't leak anymore for your case.
Could you please pull the latest and try again your code to see whether it still leaks or not?

Ok. It's fixed.

I think there are no leaks now. You can use Pools_debug_info(buffer, size); to get an overview of current memory pools.

#include "pocketpy.h"
using namespace pkpy;

char membuf[2048];

int main(){
    // Create a virtual machine
    VM* vm = new VM();

	vm->exec("a = 0");
	for (int i = 0; i < 200 * 1000 * 1000; ++i){
        vm->exec("a = a + 1 # intersting loooooooooooooooooooooooooooooooong comment");
        if(i % 1000000 == 0){
            Pools_debug_info(membuf, 2048);
            printf("%s\n", membuf);
        }
    }

    // Dispose the virtual machine
    delete vm;
    return 0;
}

PoolExpr: 0.00 KB (used) / 8.00 KB (total) - 0.00 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 arenas

0x7f96aaab8010: 0.07 MB (used) / 0.25 MB (total)

PoolExpr: 0.00 KB (used) / 8.00 KB (total) - 0.00 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 arenas

0x7f96aaab8010: 0.07 MB (used) / 0.25 MB (total)

PoolExpr: 0.00 KB (used) / 8.00 KB (total) - 0.00 KB (exceeded)
PoolFrame: 0.00 KB (used) / 10.00 KB (total) - 0.00 KB (exceeded)
PoolObject: 0 empty arenas, 1 arenas

0x7f96aaab8010: 0.07 MB (used) / 0.25 MB (total)


⚠ Bofrost.nl has degraded performance

In 38c3b95, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1055 ms

Resolved: Bofrost.nl performance has improved in c9ee8aa.


Force Union Field can't change a field back and forth

Describe the bug
When I change a union field to a different member, I can't change it back.
To Reproduce

Download binary null.sys from https://msdl.microsoft.com/download/symbols/null.sys/01FE6381a000/null.sys, and null.pdb from https://msdl.microsoft.com/download/symbols/null.pdb/DFC419D36E2BB141A3B5F9A3EEA4BD341/null.pdb
Download Windows WDK gdt file: windows_ntddk_x64_wdk_10_22000.zip Unzip it to get file windows_ntddk_x64_wdk_10_22000.gdt.
Import null.sys to Ghidra.
Data Type Manager -> Open File Archive ...: Open windows_ntddk_x64_wdk_10_22000.gdt
In Data Type Manager, right click windows_ntddk_x64_wdk_10_22000. Click "Apply Function Data Types".
File -> Load PDB File: load the downloaded null.pdb
Analysis -> Auto Analyze null.sys: use the default analyzers.
The decompile of function NlsRead looks like this:

undefined8
NlsRead(undefined param_1,undefined param_2,undefined param_3,undefined param_4,undefined param_5,
       undefined param_6,undefined4 *param_7)
{
  *(undefined8 *)(param_7 + 2) = 0;
  *param_7 = 0xc0000011;
  return CONCAT71((int7)((ulonglong)param_7 >> 8),1);
}

Retype param_7 to PIO_STATUS_BLOCK. Then the second line of the function body becomes: (param_7->field0_0x0).Status = -0x3fffffef; Right click Status -> Force Field: Change it from Status to Pointer
This line becomes: *(undefined4 *)&(param_7->field0_0x0).Pointer = 0xc0000011;
Right click Pointer -> Force Field: change it back to Status. Click OK. Nothing happens. The field remains as Pointer.

Expected behavior
I should be able to change the field between Status and Pointer back and forth.
Screenshots

Attachments

null.sys: SHA256: 876185b0fc0e802ff7954d84d6255fd61eeff217ace56e61510e1fa1f56cdef1
null.pdb: SHA256: d3e5facdaf7c8c33af5f7b11f8b35cf057ae27d6f038f0a670ac4c369ad23aee

Download links are in the previous section.
Environment (please complete the following information):

OS: Kali Linux x64
Java Version: 17.0.2
Ghidra Version: 10.2 master, 47f76c7
Ghidra Origin: locally built

This seems to be working properly on latest master build.
At least i haven't got any probems changing it that way.


Add DictBatcherIterDataPipeCreator

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

src/gravitorch/creators/datapipe/__init__.py
100.00%

src/gravitorch/creators/datapipe/base.py
100.00%

src/gravitorch/creators/datapipe/chained.py
100.00%

src/gravitorch/creators/datapipe/dictbatcher.py
100.00%

src/gravitorch/creators/datapipe/random.py
100.00%

src/gravitorch/creators/datapipe/sequential.py
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


Add Test Summary to Lint + Test workflow

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@2ff2263). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #257   +/-   ##
=======================================
  Coverage        ?   90.76%
=======================================
  Files           ?        4
  Lines           ?      433
  Branches        ?        0
=======================================
  Hits            ?      393
  Misses          ?       40
  Partials        ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


missing "floor_colors.json" file

Hi,
When I prepare my own dataset following your instruction, I meet the error that "No such file or directory: './prepare/floor_colors.json'". I think this file may be missing in this repository.
Thx

How to address this issue?

It is not used; ignore it.


added support for haproxy group per service port

Added support for haproxy group per service port. Currently both service port had to be part of same hagroup group. Now similar to Vhost, we can have different haproxy group per service port. This allows us to have external and internal services running on same box on different service ports, as they can be part of different haproxy group. This supports,
HAPROXY_0_GROUP
HAPROXY_1_GROUP
and maps them to corresponding service ports.
It is backward compatible as well, so it supports HAPROXY_GROUP.
If both HAPROXY_0_GROUP and HAPROXY_GROUP are specified, the more specific definition takes preference and the service group will be added to HAPROXY_0_GROUP and not HAPROXY_GROUP.
If there are 2 services HAPROXY_0_PORT and HAPROXY_1_PORT and if HAPROXY_0_GROUP and HAPROXY_GROUP labels are defined. HAPROXY_0_PORT service is added to HAPROXY_0_GROUP and HAPROXY_1_PORT is added to HAPROXY_GROUP since HAPROXY_1_GROUP is not specified for it.

Thank you for the PR!
Can you please add a test that tests the correct config generation with two services in two different groups? One using your newly implemented HAPROXY_0_GROUP and one falling back to HAPROXY_GROUP.

Added test cases for

Fallback to HAPROXY_GROUP label for 2 services(on same hosts) when HAPROXY_{n}_GROUP is not specified for any of the services.
Using new HAPROXY_{n}_GROUP label with 2 different services(on same hosts), with each service being associated with a different haproxy_group.
Hybrid Config where HAPROXY_0_GROUP is specified for first service and HAPROXY_GROUP is also specified without specifying HAPROXY_1_GROUP for second service.

Added documentation as well.

Is anything else required here?

@ajays20078 thank you! Can you rebase on current master and squash into a single commit please? I'll merge it then.

Done

@ajays20078 Github is still showing me This branch has conflicts that must be resolved, also there's five commits in this PR now instead of one. If your local branch is clean try something along these lines: git reset --soft HEAD~5 && git commit -am 'added support for haproxy group per service port' && git push --force.

That dint work ,as my local branch was not clean, have forked off  a fresh branch again and committed into that. Opening a different PR for that and closing this. Sorry for too much of confusion.


Do not raise when manifest digests mismatch

This check may result in false positives if the manifest lists are
re-ordered and pushed. For now, we do not want a build to fail in such
cases.

OSBS-7663

Signed-off-by: Athos Ribeiro<EMAIL_ADDRESS>Maintainers will complete the following section:

[x] Commit messages are descriptive enough
[x] "Signed-off-by:" line is present in each commit
[x] Code coverage from testing does not decrease and new code is covered
[x] JSON/YAML configuration changes are updated in the relevant schema
[x] Changes to metadata also update the documentation for the metadata
[x] Pull request includes link to an osbs-docs PR for user documentation updates

Waiving 'Code coverage from testing does not decrease and new code is covered' as the relevant tests are currently skipped.

Re-enabled one of the tests changing a few checks

RN added:

Do not raise when manifest digests mismatch - Manifest lists may have been re-pushed


Read the Docs is not building

We need to fix the Read the Docs link as it is not building due to python 2.7 not being supported

Fixed https://github.com/tomturner/django-tenants/commit/5128d9e9d2409f3cd0b09c0bd574170ff657725d


Buycraft buy sign layout

I'd like to be able to customize how the buycraft buy signs looks in the config.

plz add this!! i want to make other colors and signs on my sign


Scrolling issue (not scroll or touch button while scrolling)

Description
Hi guys, I have been using Detox for 2 years now and I have been staying at version 14 for a long time and they have been okay. But as we are thinking of testing on newer OSes and devices we think we might need to update the detox testing version as well. And so far the speed of testing has been great.
However, I ran into this situation constantly where the screen supposed to scroll but it is either not scroll or the scroll animation will also trigger the button pressing.

[x] I have tested this issue on the latest Detox release and it still reproduces

Reproduction
Just any scrollview with a lot of buttons and selections on the scrolling touch area.
Expected behavior
As you can see in the gif below the screen suppose to keep scrolling but it hung here and there and also the selection supposed to be "AFL" but after the whole animation it changed into "Martial Art". Which was not what we wanted to test and this affected the whole test case.
Screenshots

Environment (please complete the following information):

Detox: 17.3.4
React Native: 0.61.2
Node: v13.7.0
Device: iPhone 11 (simulator)
Xcode: 11.3 (11C29)
iOS: 13.3
macOS: Mojave 10.14.6 (18G6020)

Try adding a start point to your scroll action, so it starts a bit above the bottom. See if that helps.

Try adding a start point to your scroll action, so it starts a bit above the bottom. See if that helps.

Hi, @LeoNatan thanks for responding. Forgive me if I miss the information but I can't find how to 'add the starting point' as you suggested. Is there document about this or a workaround? Thanks


Missing 3.1.5 tag

Current behavior
Ecto have a 3.1.5 on https://hexdocs.pm/ecto/Ecto.html but missing in https://github.com/elixir-ecto/ecto/tags and https://github.com/elixir-ecto/ecto/releases.
Jumping from hexdocs to code may be broken.
Expected behavior
3.1.5 should be in tags and releases

Thanks, fixed!


pub build fails in security_monkey v0.3.2

As in https://github.com/akserg/angular.dart.ui/issues/142 , pub build fails on v0.3.2.
Seems to even fail with code_transformers: '0.2.3+2'
Fails on the .deb and homebrew version: https://github.com/dart-lang/homebrew-dart/issues/18
Some discussion: http://stackoverflow.com/questions/27308501/since-when-has-the-angular-transformer-required-the-dartsdk-path-be-specified-in
This is a blocking failure and must be corrected ASAP.

If using the .deb package, setting up the symlink corrects the problem:
ln -s /usr/lib/dart /usr/lib/dart/bin/dart-sdk

If using the homebrew package, it's not so easy.  Adding the dart_sdk location to the angular transformer allows pub build to build the security_monkey codebase, but it still fails when building the angular dart ui package
transformers:
 - angular:
    dart_sdk: "/usr/local/opt/dart/libexec"

Output:
(securitymonkey)➜  dart git:(develop) ✗ pub build
Loading source assets...

Error on line 21, column 5 of ../../../.pub-cache/hosted/pub.dartlang.org/angular_ui-0.6.3/pubspec.yaml: Error loading transformer: Invalid arguments(s): sdkDirectory must be provided.
    html_files:
    ^^^^^^^^^^^^^

If you're on a mac, you can download the Dart Editor and perform a pub build from there.  That way you can avoid the homebrew package until it's fixed.
I really hope these packages (homebrew and deb) are fixed so they can perform a pub build without the user modifying the codebase or adding symlinks post installation.  It is super hacky at the moment.

Do you have any suggestions when the pub build appears to fail even after setting the sym link to the dart-sdk?  I keep running the command and it runs for ~minute and then just stops.  When I look in the build directory there is nothing...
Thanks

@applesauce49 You're using the .deb install of the dart tools?  Does pub build provide any output before it stops?  What is the output of  pub --version ?  If it's older than v1.8.*, you may want to do a sudo apt-get update before running apt-get install -y dart
I may upload a .tar file containing the build output.  Others have requested that.  Would that help?

I've got dart 1.8.5.
I think part of my problem appears to be that I'm using the t1.micro ubuntu instance type at AWS.  as soon as I tried to do the build on my local machine it worked fine.  based on your comment, i can assume it's safe for me to just tar up the results of that and send it to my aws ubuntu instance?
Thanks

The homebrew issue has been fixed with code_transformers 0.2.5.  Huge thanks to @sethladd.
I've updated the pubspec.yaml in commit e66463fa4b3fb4994f4226f129e9f997b84cf1aa in both the master and develop branches.

code_transformers 0.2.5 also removes the requirement to have the symlink in the .deb installs.

To be fair, huge thanks to @pq and @jakemac53


Can have a better arrangement

try this docs : https://reactnative.dev/docs/flatlist

@amanabhaysinghh which component you're referring to? what do you mean by better arrangement exactly?