Adding Validation Loss to detect Overtraining

I saw that this repo has added the ability to use a validation loss to help figure out the optimal amount of training. Might be an interesting addition.
https://github.com/victorchall/EveryDream2trainer/blob/main/doc/VALIDATION.md

I this idea is good.
I believe that it is not enough to simply observe loss/average, since the weights are continually being updated.

I think this would be a good means to determine learning rates. There's a lot of speculation as to the proper rate, but with validation, we should be able to prove appropriate learning rates. The question then would be if that value would be universally good when training something specific (e.g. style), or good for everything. I think there's a lot of potential to be gained with this.

So... is this still being considered or was it scraped? @kohya-ss

I have implemented this in a fork I originally created to implement the new noise scheduling functions. I won't create a PR because the massive refactoring I have done is largely experimental, but you can check it out here. I'm doing a run with wandb right now to test LoRA settings.
https://github.com/AMorporkian/kohya_ss/tree/hypertune
Sorry for the absolutely terrible commits, I was just messing around and the messages are not at all descriptive. If there isn't any movement on this by next week, I'll take some time to actually make some proper code that would integrate better.

This seems like a really cool idea. I have to make a lot of different training runs to figure out what the optimum settings are for a given training set. Even doing things in epochs, number of training images seems to still affect the results somewhat.

I have implemented this in a fork I originally created to implement the new noise scheduling functions. I won't create a PR because the massive refactoring I have done is largely experimental, but you can check it out here. I'm doing a run with wandb right now to test LoRA settings.
https://github.com/AMorporkian/kohya_ss/tree/hypertune

I borrowed some of your ideas (random_split and collate updates) and put a copy/paste validation loss together. Helped to not have to create a custom dataset to get this ball rolling.
If anyone else is interested in testing, see #914 . Expanding into validation datasets would be the next step but this works well with minimal changes.

any updates on this :D


Get contents of DataGrid from DevExpress

Hello,
Im using the DataGrid from DevExpress and I would like to read from and write to the cells.
At the moment the Cells contain Text and Checkboxes, but when inspecting with FlaUInspect the elements are just listed as "custom" type.

This is what I have tried to access the checkboxes:
dataItem.FindFirstChild("IsSelected").FindFirst(TreeScope.Descendants, new ConditionFactory(new UIA3PropertyLibrary()).ByControlType(ControlType.CheckBox)) returns null
tableItem.FindFirstChild("IsSelected").FindAllDescendants()[0].AsCheckBox() returns a CheckBox but accessing IsChecked throws System.InvalidOperationException: 'Native pattern is null'
I have been able to toggle the checkbox using tableItem.FindFirstChild("IsSelected").Click() which is unfortunately not suited for my usecase because i would like to check the current state or explicitly set it to true or false.
What would you suggest in that case?
regards

Fid you enable ClearAutomationEventsHelper.IsEnabled = false; as stated in the https://github.com/FlaUI/FlaUI/wiki/Common-Issues`?

Yes, I have set the ClearAutomationEventsHelper.IsEnabled = false; in the OnStartup(StartupEventArgs e) in App.xaml.cs.
When i use FlaUInspect with the CTRL selection function it just crashes at random locations with HRESULT: 0x80040201 An event was unable to invoke any of the subscribers. Though sometimes it manages to display that the element Custom "" "" has a CheckBox as a child.
That happens for example in MainViewModel.cs ElementToSelectChanged:

Though sometimes it manages to display that the element Custom "" "" has a CheckBox as a child when i click into the checkbox fast while holding ctrl.

Searching in the code it looks like i can only find the Custom "" ""

I was able to use the Grid pattern and get the value by specifying row and column. I had to implement my own grid to get non localized values for the checkboxes.

👍


Thanks!


Error happens with egs/librispeech/ASR/prepare_mmi.sh

I got following error when just running ./prepare_mmi.sh
./prepare_mmi.sh --stage 6
2024-05-16 11:27:09 (prepare.sh:81:source) Running prepare.sh
2024-05-16 11:27:09 (prepare.sh:83:source) dl_dir: /data/store3/kamo/icefall/egs/librispeech/ASR/download
2024-05-16 11:27:09 (prepare.sh:86:source) Stage -1: Download LM
2024-05-16 11:27:09 (prepare.sh:95:source) Stage 0: Download data
2024-05-16 11:27:09 (prepare.sh:117:source) Stage 1: Prepare LibriSpeech manifest
2024-05-16 11:27:09 (prepare.sh:128:source) Stage 2: Prepare musan manifest
2024-05-16 11:27:09 (prepare.sh:139:source) Stage 3: Compute fbank for librispeech
2024-05-16 11:27:09 (prepare.sh:173:source) Stage 4: Compute fbank for musan
2024-05-16 11:27:09 (prepare.sh:182:source) Stage 5: Prepare BPE based lang
2024-05-16 11:27:09 (prepare.sh:210:source) Stage 6: Prepare phone based lang
2024-05-16 11:27:09 (prepare_mmi.sh:11:main) Running prepare_mmi.sh
2024-05-16 11:27:09 (prepare_mmi.sh:17:main) Stage 0: Prepare bigram token-level P for MMI training
Traceback (most recent call last):
  File "/data/store3/kamo/icefall/egs/librispeech/ASR/./local/convert_transcript_words_to_tokens.py", line 103, in <module>
    main()
  File "/data/store3/kamo/icefall/egs/librispeech/ASR/./local/convert_transcript_words_to_tokens.py", line 84, in main
    assert Path(args.lexicon).is_file()
AssertionError
./prepare_mmi.sh --stage 6
2024-05-16 11:27:09 (prepare.sh:81:source) Running prepare.sh
2024-05-16 11:27:09 (prepare.sh:83:source) dl_dir: /data/store3/kamo/icefall/egs/librispeech/ASR/download
2024-05-16 11:27:09 (prepare.sh:86:source) Stage -1: Download LM
2024-05-16 11:27:09 (prepare.sh:95:source) Stage 0: Download data
2024-05-16 11:27:09 (prepare.sh:117:source) Stage 1: Prepare LibriSpeech manifest
2024-05-16 11:27:09 (prepare.sh:128:source) Stage 2: Prepare musan manifest
2024-05-16 11:27:09 (prepare.sh:139:source) Stage 3: Compute fbank for librispeech
2024-05-16 11:27:09 (prepare.sh:173:source) Stage 4: Compute fbank for musan
2024-05-16 11:27:09 (prepare.sh:182:source) Stage 5: Prepare BPE based lang
2024-05-16 11:27:09 (prepare.sh:210:source) Stage 6: Prepare phone based lang
2024-05-16 11:27:09 (prepare_mmi.sh:11:main) Running prepare_mmi.sh
2024-05-16 11:27:09 (prepare_mmi.sh:17:main) Stage 0: Prepare bigram token-level P for MMI training
Traceback (most recent call last):
  File "/data/store3/kamo/icefall/egs/librispeech/ASR/./local/convert_transcript_words_to_tokens.py", line 103, in <module>
    main()
  File "/data/store3/kamo/icefall/egs/librispeech/ASR/./local/convert_transcript_words_to_tokens.py", line 84, in main
    assert Path(args.lexicon).is_file()
AssertionError

data/lang_bpe_500/lexicon.txt doesn't exist, but I have data/lang_phone/lexicon.txt instead.
Could you help me?

@pkufool  Could you have a look? I think prepare_mmi.sh is refactored from the original prepare.sh.

hi, i'm looking into it, will return to you by tomorrow

dear naoyuki,
I ran the preparation script end to end and this PR should fix the issue.
The problem was introduced when we was trying to strip some unnecessary parts from the main prepare.sh script, the part for generating lexicon.txt was also stripped and moved to prepare_lm.sh.
This PR should fix the problem, please check, thanks.
https://github.com/k2-fsa/icefall/pull/1626￼
Fix an error occured during mmi preparation by JinZr · Pull Request #1626 · k2-fsa/icefall
github.com
best
jin

On May 16, 2024, at 10:33, Fangjun Kuang @.***> wrote:
@pkufool https://github.com/pkufool Could you have a look? I think prepare_mmi.sh is refactored from the original prepare.sh.
—
Reply to this email directly, view it on GitHub https://github.com/k2-fsa/icefall/issues/1624#issuecomment-2113902350, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOON42ARVBHKJGYJHH3F6ILZCQLHJAVCNFSM6AAAAABHZIXYEWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJTHEYDEMZVGA.
You are receiving this because you are subscribed to this thread.


Fix- Overexposed shadow blur radius of cards

Issue Number
fixes #225 - Fix- Overexposed shadow blur radius of cards

Describe the changes you've made
i have added css style under index.js of SmallCourseCard - boxShadow: "0px 0px 5px rgba(0,0,0,0.2)",

[x] My code follows the code style of this project.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

Hey @naveen8801  you have not made changes to all the cards which have this issue, you can go for making changes to the rest of the cards too :)

Hey @naveen8801 you have not made changes to all the cards which have this issue, you can go for making changes to the rest of the cards too :)

sure @adarsh-technocrat 👍

Hi @Abhishek-kumar09  @adarsh-technocrat , i found className={classes.li} in NonHero.js file in props passed to ListItem component , i think there is no use of this line because no such class exist , it may confuse some new comers so should i remove it ?   :)

@naveen8801  you can remove that!

@naveen8801 you can close this PR now as this issue is resolved from this PR (#327)


Using the Theme component on Android may lead to a blank page.

Description
I had multiple stack Navigators and required dark mode adaptation for my application. To achieve this, I utilized gluestack with react navigation v6. According to the documentation at https://gluestack.io/style/docs/configuration/themes, when using tokens within the theme, it is necessary to utilize the Theme component in conjunction with the StyledProvider component. However, when I imported the Theme component, the navigation page ended up blank.
CodeSandbox/Snack link
No response
Steps to reproduce

npx gluestack-ui-scripts eject-theme，export config and configure theme
Configure any Stack navigator pages， and use the gluestack provider with Theme components

gluestack-ui Version
1.0.3
Platform

[X] Expo
[ ] React Native CLI
[ ] Next
[ ] Web
[X] Android
[ ] iOS

Other Platform
No response
Additional Information

Hi @Vvaraz , this is on us we will fix this with the next release. Theme Component renders a View, because of which your layout might be breaking and we are adding the support for passing styled props to that View.

Will be fixed with this PR: https://github.com/gluestack/gluestack-style/pull/500

Hi @Vvaraz , this is on us we will fix this with the next release. Theme Component renders a View, because of which your layout might be breaking and we are adding the support for passing styled props to that View.

Ok,Based on my current actions, I have commented out the View component of ThemeContext.Provider in Theme.tsx and uses patch-package,looks like this:
`diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 4a3bae2..5e0fefc 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -22,7 +22,8 @@ export const Theme: React.FC<{
return (
<ThemeContext.Provider value={contextValue}>
{/* @ts-ignore */}

 <View dataSet={{ 'theme-id': name }}>{children}</View>

 {/* <View dataSet={{ 'theme-id': name }}>{children}</View> */}

 {children}

</ThemeContext.Provider>
);
};`

Hey @Vvaraz, commenting out View will break the Theme in the Web application.


snippets orderable not working after upgrading to wagtail 4.0

Issue Summary
Ordering in snippets not working after updating to wagtail 4
Steps to Reproduce
I have a menu system implemented as following

@register_snippet
class Menu(TranslatableMixin, ClusterableModel):
    """The main menu clusterable model."""

    title = models.CharField(_("link_title"), max_length=100)
    slug = AutoSlugField(populate_from="title", editable=True)

    panels = [
        MultiFieldPanel(
            [
                FieldPanel("title"),
                FieldPanel("slug"),
            ],
            heading="Menu",
        ),
        InlinePanel("menu_items", label="Menu Item"),
    ]

    def __str__(self):
        return self.title

class MenuItem(TranslatableMixin, Orderable, ClusterableModel):
    """The First level menu of the Navigationbar."""

    link_title = models.CharField(_("link_title"), blank=True, null=True, max_length=50)
    link_page = models.ForeignKey(
        "wagtailcore.Page",
        null=True,
        blank=True,
        related_name="+",
        on_delete=models.CASCADE,
    )
    open_in_new_tab = models.BooleanField(default=False, blank=True)

    page = ParentalKey("Menu", related_name="menu_items")

    panels = [
        MultiFieldPanel(
            [
                FieldPanel("link_title"),
                PageChooserPanel("link_page"),
                FieldPanel("open_in_new_tab"),
            ],
            heading="MenuItem",
        ),
        InlinePanel("sub_menu_items", label="Sub Menu Item"),
    ]

    class Meta(Orderable.Meta):
        unique_together = ('translation_key', 'locale')

class SubMenuItem(TranslatableMixin, Orderable):
    """The Second level menu of the Navigationbar."""

    link_title = models.CharField(_("link_title"), blank=True, null=True, max_length=50)
    link_page = models.ForeignKey(
        "wagtailcore.Page",
        null=True,
        blank=True,
        related_name="+",
        on_delete=models.CASCADE,
    )

    open_in_new_tab = models.BooleanField(default=False, blank=True)

    page = ParentalKey("MenuItem", related_name="sub_menu_items")

    panels = [
        MultiFieldPanel(
            [
                FieldPanel("link_title"),
                PageChooserPanel("link_page"),
                FieldPanel("open_in_new_tab"),
            ],
            heading="SubMenuItem",
        ),
    ]

    class Meta(Orderable.Meta):
        unique_together = ('translation_key', 'locale')

I can see menu ordering perfectly when in wagtail 3
After I update to wagtail 4 and apply default migrations, I am not able to change menuItem order in the admin. Whatever the changes I do to ther order are lost once I save the snippet.
Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: (yes / no)

Technical details

Python version: 3.8.10
Django version: 4.0.7
Wagtail version: 4.0, 4.0.1
Browser version: Chrome 105

looks like this is the same issue as https://github.com/wagtail/wagtail/issues/9145. Please close this if it is the same.


fix: docs/sharepoint/walkthrough-profiling-a-sharepoint-application.md

Adjust for skipped heading levels

@nschonni : Thanks for your contribution! The author, @John-Hart, has been notified to review your proposed change.

@John-Hart - Could you review the proposed changes?
Also, could you follow the instructions in Link your GitHub and Microsoft accounts to link your Microsoft account with your GitHub user account? Thanks.

Verified the updates; merging.


Make Report.getOutputLocation return a Property instead of a Provider

Fixes #22230

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

Discussion in progress on Slack.

For consideration for Gradle 8.0.

Given that it's a pretty minor breaking change, I'd be okay if we break in 8.0 as long as we include a section in the upgrade guide.

I'll add the note to the upgrade guide.  I wouldn't want this to be the only breaking change in 8.0, but we have others, so I think we should use the opportunity to fix this.
@big-guy - I don't understand why we wouldn't want that warning to remain after making this change?

@big-guy - I don't understand why we wouldn't want that warning to remain after making this change?

I meant if instead of making a breaking change we put the new method in ConfigurableReport, we shouldn't tell people to not use ConfigurableReport.

@bot-gradle test this

@bot-gradle test BD

OK, I've already triggered the following builds for you:

BuildDistributions build

@nathan-contino - Can you please take a look at this upgrade note: https://builds.gradle.org/repository/download/Gradle_Master_Check_BuildDistributions/58826074:id/distributions/gradle-8.0-docs.zip!/gradle-8.0-20221128132035%2B0000/docs/userguide/upgrading_version_7.html#report_getoutputlocation_return_type_changed

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Add fetch_owner hive script

Sorry if this is dumb question, but I can't find the fetch_owner.hql script. I've searched git, I cloned the repo, I've searched this pull request....

here is the scripts. But it may not be useful for you. This change was push into 'linkedin' branch just because it is 'linkedin special'. We have another project/procedure that store the dataset owner info into a hive table, that's why we can fetch ownership info from a hive table. Take a look of comments here : https://github.com/linkedin/WhereHows/commit/0c0355f9703a65bae6bc16f53ee0b38a3e133543


Extension loads, but doesn't do much else

Bug report
Extension loads, but no other functionality works
Describe the bug
The extension installs without issue.  I get a warning that I don't have GitHub CoPilot which I'm assuming is not an issue.  The Supabase panel loads fine, clicking Connect has no error, but the Database view doesn't show any of the database.  It's like it wasn't able to connect, though I don't see an error.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Install extension
Launch extension by clicking on Supabase icon in left gutter
Click 'connect' button to connect to the database
See the Database view with sub-menu for Tables, Views, etc.
None of the sub-menu items have any functionality.   I.e. Clicking Tables doesn't do anything.

Expected behavior
I expected to see the sub-menu items under Database populated so when I click Tables, I should see the tables
Screenshots

System information

OS: macOS Sonoma 14.6.1
Browser (if applies) [e.g. chrome, safari]
Version of supabase-js: [e.g. 6.0.2]
Version of Node.js: [e.g. 10.10.0]

Additional context
Add any other context about the problem here.
Am I missing any steps?  I don't see any settings for this extension so not sure if I missed something.  Also, not sure how to find the logs to diagnose it.  Appreciate any guidance.

It's impressive that they managed to release this without adding a field where you can write the address of your local supabase AND used <IP_ADDRESS>:80 as the default?! How's this supposed to work when the defaults are that the local supabase api is exposed on port 54321?

Sorry for the delay, this is is very much an MVP for now we've shipped for Launch Week 12 and we're currently exploring where to go from here, so your feedback and feature requests are very much appreciated 🙏
You can see the default ports that are being used here: https://github.com/supabase-community/supabase-vscode-extension/blob/main/src/features/database/classes/supabase-api.ts#L8-L11
These are the default ports that the supabase CLI uses when you run supabase start.  I believe not having GitHub Copilot might be the issue here. I'll investigate how we can unbundle it.


update argument name in async example

It seems like the message local used later in sleepy_hello_world isn't declared. I guess that the async version takes message like the sync version in the first example?

Apologies, I missed this! Good catch yup, tho all sorted now, happily.


Publish version 17 to npm

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Problem Description
The electron-packager package is marked as deprecated, with the following message:

Please use @electron/packager moving forward. There is no API change, just a package name change

Unfortunately, you can't just change the package name since there is no overlap in released versions. also involves a major version bump. In my case, that version bump also came with a breaking regression #1700.
Proposed Solution
It would be nice to retroactively release @electron/packager 17 to npm so one can merely replace the name of electron-packager in package.json.
Alternatives Considered
An alternate (and more confusinger) way to switch over package names is to add a package alias to the consuming package like:
npm i --save-dev electron-packager@npm<EMAIL_ADDRESS>-E
Additional Information

This is neither easy nor ideal, I'll take a look at #1700

This is neither easy nor ideal

How so?
In particular, why would it not suffice to take v17.1.2, change only the package name to @electron/packager and publish the result to npm?

@rotu For security reasons publishing of npm packages that Electron maintains is strictly controlled and mostly automated, though we have break-glass tools for certain scenarios in this case there is no reason to go outside our standard publishing practices.

@rotu For security reasons publishing of npm packages that Electron maintains is strictly controlled and mostly automated, though we have break-glass tools for certain scenarios in this case there is no reason to go outside our standard publishing practices.

Okay. So how does this publishing process foreclose on making a 17-series release? (One would think pushing a v17.1.2-0 tag would cause a release, but I don't understand the CI scripts). And how did a hard breaking change get through this process on a patch version?

One would think pushing a v17.1.2-0 tag would cause a release

One would be wrong, I'm not going to give you an essay on how Electron's npm publishing system works. You can look up various components of it, e.g. CFA or read up on why the practice of publishing a semanticly versioned package out-of-order is considered bad. But that's all moot as the answer was no, we won't be publishing a v17 of the scoped package name.

And how did a hard breaking change get through this process on a patch version?

It didn't, this change was published as version 18 and therefore is allowed to have breaking changes, in this case this was an unintentional side effect of a build system change (from cjs to compiled esm typescript) which changed the export semantics without folks realizing.

One would be wrong, I'm not going to give you an essay on how Electron's npm publishing system works. You can look up various components of it, e.g. CFA

Thanks for the refs re: CFA! I'll study up!

or read up on why the practice of publishing a semanticly versioned package out-of-order is considered bad. But that's all moot as the answer was no, we won't be publishing a v17 of the scoped package name.

That's exactly backwards. Semantic versioning is specifically designed so that fixes can be added to previous major branches without upsetting the order. Only bumping the most significant (major) number will cause that release to take priority over the current latest release.

And how did a hard breaking change get through this process on a patch version?

It didn't, this change was published as version 18 and therefore is allowed to have breaking changes

Yes, it did. As written up in #1700, the API change happened in 18.1.2 vis-a-vis 18.1.1, NOT the bump from major version 17 to 18. You can see the sizable diff with npm diff --diff=<EMAIL_ADDRESS>--diff=<EMAIL_ADDRESS>old src/index.js:
module.exports = async function packager (opts) {

new dist/index.js:
Object.defineProperty(exports, "__esModule", { value: true });
exports.serialHooks = exports.packager = exports.allOfficialArchsForPlatformAndVersion = void 0;
const hooks_1 = require("./hooks");
Object.defineProperty(exports, "serialHooks", { enumerable: true, get: function () { return hooks_1.serialHooks; } });
const packager_1 = require("./packager");
Object.defineProperty(exports, "packager", { enumerable: true, get: function () { return packager_1.packager; } });
const targets_1 = require("./targets");
Object.defineProperty(exports, "allOfficialArchsForPlatformAndVersion", { enumerable: true, get: function () { return targets_1.allOfficialArchsForPlatformAndVersion; } });
exports.default = packager_1.packager;
__exportStar(require("./types"), exports);

Okay, so I think I understand the build system a little better.

semantic-release doesn't actually check that changes are semantic. It only aggregates commit messages and bumps the version according to the level-of-impact described by the commit message.
CFA is merely semantic release where a human must approve part of the CI process.
Releases for this repo are run from the main branch. So to backport a hotfix, you'd have to expand that filter to include maintenance branches (?).
Since the pre-name-change tags are formatted as expected, it should be as simple as creating a maintenance branch after the v17.1.2 tag and pushing a fix commit.

Additionally, something should probably be done about the broken releases in 18.1.2, 18.1.3, 18.2.0, 18.3.1. I think these are supposed to be deprecated but maybe @MarshallOfSound has opinions on how seriously we take semver 'round these parts.


Set LastDesignTimeBuildSucceeded = false by default, so we don't show…

… any diagnostics until the first design time build succeeds

Tagging @dotnet/project-system for review.

Tagging @MattGertz for ask mode approval.

Approved pending signoffs.


#210

try #210 again

Codecov Report

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

@@            Coverage Diff            @@
##             master     #216   +/-   ##
=========================================
  Coverage          ?   24.11%
=========================================
  Files             ?       23
  Lines             ?     1518
  Branches          ?        0
=========================================
  Hits              ?      366
  Misses            ?     1152
  Partials          ?        0

Impacted Files
Coverage Δ

src/deprecated/util.jl
0% <ø> (ø)

src/deprecated/cindex/types.jl
0% <ø> (ø)

src/deprecated/cindex/base.jl
0% <ø> (ø)

src/deprecated/wrap_cpp.jl
0% <ø> (ø)

src/deprecated/cindex.jl
1.41% <ø> (ø)

src/show.jl
0% <0%> (ø)

src/compat.jl
0% <0%> (ø)

src/deprecated/wrap_c.jl
0% <0%> (ø)

src/cltypes.jl
100% <100%> (ø)

src/LibClang.jl
100% <100%> (ø)

... and 13 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 1c5f425...75628a5. Read the comment docs.

rebasing in gitkraken takes no effect, but manually rebasing in CLI works, not sure why.


How to use 'removePragmas' in 'import' mode

How to remove warnings in 'import' mode？

Hi @huoxiaoliang, this will depend on how you are importing CesiumJS and any build tools you are using. For example, if you are using webpack, this is how we recommend you remove pragmas.
Cesium is a big project, so we use GitHub for feature requests and bug tracking exclusively. I would ask that you take this to the Cesium Forum where there are members of the community and developers from the team who can help.


PyPi Release Behind But??

Seems like what is available on PyPi (0.7.1) is months behind this one here which is 0.7.0 but has recent commits? (got here from the PyPi -> Bitbucket -> Migrated Repo Link
And sublt name differance
pywhois (here)
vs python-whois (PyPi)

whoops, must have been something missed when migrated from bitbucket to github. Have pushed 0.7.2 with latest merges.


Update windows to 0.58

@BenjaminBrienen Thank you for your contribution. Unfortunately, this needs more work. The accesskit_windows crate doesn't compile. Also, looking at the diff of Cargo.lock, how did libloading's windows_targets dependency end up getting downgraded?

@BenjaminBrienen Thank you for your contribution. Unfortunately, this needs more work. The accesskit_windows crate doesn't compile. Also, looking at the diff of Cargo.lock, how did libloading's windows_targets dependency end up getting downgraded?

No idea, haha!
I'll fix the compile errors soon.

I'm trying to figure out what's going wrong with the implement macro, so I used cargo expand and cleaned up the output. The only real errors are related to ITextProvider and PlatformNode:
error[E0277]: the trait bound `PlatformRange_Impl: Accessibility::ITextRangeProvider_Impl` is not satisfied
    --> platforms\test_output\src\text.rs:203:73
     |
203  |         <ITextRangeProvider as windows::core::Interface>::Vtable::new::<Self, -1>(),
     |                                                                         ^^^^ the trait `Accessibility::ITextRangeProvider_Impl` is not implemented for `PlatformRange_Impl`
     |
     = help: the trait `Accessibility::ITextRangeProvider_Impl` is implemented for `PlatformRange`
note: required by a bound in `ITextRangeProvider_Vtbl::new`
    --> C:\Users\BenjaminBrienen\.cargo\registry\src\index.crates.io-6f17d22bba15001f\windows-0.58.0\src\Windows\Win32\UI\Accessibility\impl.rs:3246:19
     |
3244 |     pub const fn new<Identity: windows_core::IUnknownImpl, const OFFSET: isize>() -> ITextRangeProvider_Vtbl
     |                  --- required by a bound in this associated function
3245 |     where
3246 |         Identity: ITextRangeProvider_Impl,
     |                   ^^^^^^^^^^^^^^^^^^^^^^^ required by this bound in `ITextRangeProvider_Vtbl::new`

Hello @BenjaminBrienen,
Looking at the changelog for the windows crate, I see that there is at least one breaking change in the COM interface authoring process. See https://github.com/microsoft/windows-rs/pull/3065.

I took a step back to look at the compile error that @mwcampbell mentioned, and I'm not actually getting one:
PS C:\Users\BenjaminBrienen\source\accesskit> cargo clean
     Removed 0 files
PS C:\Users\BenjaminBrienen\source\accesskit> cargo build -p windows
   Compiling proc-macro2 v1.0.66
   Compiling unicode-ident v1.0.8
   Compiling windows_x86_64_msvc v0.52.6
   Compiling windows-targets v0.52.6
   Compiling windows-result v0.2.0
   Compiling windows-strings v0.1.0
   Compiling quote v1.0.33
   Compiling syn v2.0.32
   Compiling windows-implement v0.58.0
   Compiling windows-interface v0.58.0
   Compiling windows-core v0.58.0
   Compiling windows v0.58.0
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 15.15s
PS C:\Users\BenjaminBrienen\source\accesskit> git status
On branch update-windows-dep
Your branch is up to date with 'origin/update-windows-dep'.

nothing to commit, working tree clean

@BenjaminBrienen You need to compile the accesskit_windows crate, not the windows one.

I need some help with how to fix the error with HWND not being Send.

Relevant upstream issue: https://github.com/microsoft/windows-rs/issues/3093
If we are sure that our use of HWND is thread-safe, then we'll probably have to wrap it in a new type which is Send, for internal use.

It compiles but the test fails

Tests pass on my machine. We always recommend to launch them from a regular command line window, otherwise they may fail.
I can see these warnings though:
warning: unused `BOOL` that must be used
   --> platforms\windows\examples\hello_world.rs:355:14
    |
355 |     unsafe { ShowWindow(window.0, SW_SHOW) };
    |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: `#[warn(unused_must_use)]` on by default
help: use `let _ = ...` to ignore the resulting value
    |
355 |     unsafe { let _ = ShowWindow(window.0, SW_SHOW); };
    |              +++++++                              +

warning: unused `BOOL` that must be used
   --> platforms\windows\examples\hello_world.rs:359:18
    |
359 |         unsafe { TranslateMessage(&message) };
    |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
help: use `let _ = ...` to ignore the resulting value
    |
359 |         unsafe { let _ = TranslateMessage(&message); };
    |                  +++++++                           +

warning: unused `Foundation::BOOL` that must be used
   --> platforms\windows\src\tests\mod.rs:176:18
    |
176 |         unsafe { ShowWindow(self.window.0, SW_SHOW) };
    |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: `#[warn(unused_must_use)]` on by default
help: use `let _ = ...` to ignore the resulting value
    |
176 |         unsafe { let _ = ShowWindow(self.window.0, SW_SHOW); };
    |                  +++++++                                   +

warning: unused `Foundation::BOOL` that must be used
   --> platforms\windows\src\tests\mod.rs:177:18
    |
177 |         unsafe { SetForegroundWindow(self.window.0) };
    |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
help: use `let _ = ...` to ignore the resulting value
    |
177 |         unsafe { let _ = SetForegroundWindow(self.window.0); };
    |                  +++++++                                   +

warning: unused `Foundation::BOOL` that must be used
   --> platforms\windows\src\tests\mod.rs:217:26
    |
217 |                 unsafe { TranslateMessage(&message) };
    |                          ^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
help: use `let _ = ...` to ignore the resulting value
    |
217 |                 unsafe { let _ = TranslateMessage(&message); };
    |                          +++++++                           +

warning: `accesskit_windows` (example "hello_world") generated 2 warnings
warning: `accesskit_windows` (lib test) generated 3 warnings

Also, I don't think we need to change the public API and expose WindowHandle. We don't want to explicitly allow convertion between WindowHandle and *mut c_void or vice versa, *mut c_void could be anything.

Also, I don't think we need to change the public API and expose WindowHandle.

The hello_world example needs to use Adapter::new, so how should we handle that interface?

I think Adapter::new should expect an HWND. The convertion to WindowHandle should probably be done inside Adapter::with_wrapped_action_handler.

I think Adapter::new should expect an HWND. The convertion to WindowHandle should probably be done inside Adapter::with_wrapped_action_handler.

And what about the handler parameter? It expects + Send and the struct the example uses isn't Send (I added an unsafe impl for now)

I'm not sure if SimpleActionHandler should be Send or if it should store a WindowHandle. The later would be more correct I guess but then we have to export the type just for the example.

The whole crate builds for me now. Sorry for not checking before.

@BenjaminBrienen I have pushed further changes to your branch, I hope it is OK with you?
@mwcampbell The only open question for me now is what do we want to do with the action handler in the example.

@BenjaminBrienen I have pushed further changes to your branch, I hope it is OK with you?
@mwcampbell The only open question for me now is what do we want to do with the action handler in the example.

Awesome, thanks for the help!

In the example, I think manually implementing Send on SimpleActionHandler is the best we can do. We shouldn't need to implement Sync though.
Also, because the change to windows-rs's HWND crate is an API-breaking change (as shown by the required change to accesskit_winit), we're going to need to commit this as a semver-breaking release of accesskit_windows. But we don't want to do a semver-breaking release of accesskit_winit, since the API of that crate didn't break, and such a release would be more disruptive for our users. So unfortunately, what we'll need to do is withdraw the accesskit_winit changes from this PR, temporarily break the CI, and do another PR with those changes. @DataTriny and I have done this kind of thing before to work around the limitations of release-please, the release management tool we're using.

We're trying something different this time to work around the limitations of the tool we use to manage releases. As part of this I had to rebase your work @BenjaminBrienen. Your name disappeared from the second commit because of this. Sorry for hijacking your work, this is not my intention.

We're trying something different this time to work around the limitations of the tool we use to manage releases. As part of this I had to rebase your work @BenjaminBrienen. Your name disappeared from the second commit because of this. Sorry for hijacking your work, this is not my intention.

It doesn't bother me. 😄 Recognition isn't why I want to help! Thanks for bringing it to the finish line.


Route based on hostname

So I got http-proxy-middleware and http-proxy-agent working well now. However, to make my life easier :) I'll need to be able to route traffic based on hostname rather than just paths. It's because we have several backend api endpoints that are on different servers.
Basically, I want to do something similar to my config code below but I'm sure it might not work. So any HTTP requests that goes to http://server-api-a will be forwarded to http://<IP_ADDRESS>:3128. Any HTTP requests that goes to http://server-api-b will be forwarded to http://<IP_ADDRESS>:8888
var HttpsProxyAgent = require('https-proxy-agent');
var proxy = require("http-proxy-middleware");

var optionsServerA = {
    target: 'http://thatserver',
    agent: new HttpsProxyAgent('http://<IP_ADDRESS>:3128')
};

var optionsServerB = {
    target: 'http://anotherserver',
    agent: new HttpsProxyAgent('http://<IP_ADDRESS>:8888')
};

var apiProxy = proxy('http://server-api-a', optionsServerA);
var apiProxy = proxy('http://server-api-b', optionsServerB);

Any ideas how this can be achieved?

I can add each /path and it works but more prone to error especially if I miss one or few more paths.

The router option is maybe something you're looking for.
https://github.com/chimurai/http-proxy-middleware/blob/master/recipes/router.md

WOW! Looks like that could be it! :D I'll play with it tonight. Thanks a lot! :)

I'm very close but I need to bypass the http-proxy-agent when my request is onlylocal.com. What do I need to change?
var customRouter = function(req) {
  let host = '';
  switch  (req.headers.host) {
    case 'cs.video.example.com': host = 'http://cs.video.exampe.com'; break;
    case 'onlylocal.com': {
        if (req.originalUrl.indexOf('/app') !== -1) {
          host = 'http://localhost:9990';
        }
        if (req.originalUrl.indexOf('/static') !== -1) {
          host = 'http://localhost:8888';
        }
        break;
    }
  }
  return host;
};

var options = {
    agent: new HttpsProxyAgent('http://<IP_ADDRESS>:2301'),
    target: 'http://www.example.com',
    router: customRouter
};

var myProxy = proxy(options);

var app = express();
app.use(myProxy);

I got this working now

Is there any way to use the router based on X-Forwarded-Host and not the Host header?

Is there any way to use the router based on X-Forwarded-Host and not the Host header?

OK, we have managed this using a higher priority express middleware to overwrite req.headers.host.


question about filling scoring matrix

Thank you for this excellent work! I have a question about filling the scoring matrix after reading the article and source code: in https://github.com/labstructbioinf/pLM-BLAST/blob/886a8fbee3b66c76141ed0580444caf25176e5b9/alntools/numeric.py#L131, pLM-BLAST local used the maximum value from 1 to i+1 as the candidate filling value, not as in pLM-BLAST global or traditional SW algorithm, which just use i as the candidate. What is the purpose of this design?

another question is that, in the article it claimed that "...the above equation does not truncate the values to zero", however, both i+1 in the https://github.com/labstructbioinf/pLM-BLAST/blob/886a8fbee3b66c76141ed0580444caf25176e5b9/alntools/numeric.py#L131, j+1 in the https://github.com/labstructbioinf/pLM-BLAST/blob/886a8fbee3b66c76141ed0580444caf25176e5b9/alntools/numeric.py#L133, and line https://github.com/labstructbioinf/pLM-BLAST/blob/886a8fbee3b66c76141ed0580444caf25176e5b9/alntools/numeric.py#L123 will cause the value to be truncated to zero. Did I missunderstand something？

Hi, thanks for intresting questions.

The global mode is more like NW approach, local uses SW-like filling, where off diagonal elements h_tmp[1] and h_tmp[2] are selected by scanning row/column values. We use python range function so (1, i+1) becomes (1, i).
Thanks for pointing this, you are correct, later we discovered that values below zero are very rare and only slightly negative (probably as a consequence of network activation functions). And if they appear in one point, all neighboring points are ~0 or negative. This is because substitution matrix of p5 is "smooth" - values are gradually changing. Hence paths derived from scoring matrix are omiting low similarity regions, when we do clip them to 0, they are omited anyway. At the beginning we believed that negative values will act similar to gap penalty.  Anyway I will change this back to match publication text :)

Thanks for your explanation!

Thanks for your explanation! I will close this issue.


Create reusable Saga compliance test suite

Try to centralize what's already done today. Move this to the new compliance nuget. Get ready for RavenDb / CosmosDb / DynamoDb usage

Was pretty well already done anyway


fix: 修复auto add to project action 语法错误

👀 PR includes

✨ Feature

[ ] New feature

🎨 Enhance

[ ] Code style optimization
[ ] Refactoring
[ ] Change the UI
[ ] Improve the performance
[ ] Type optimization

🐛 Bugfix

[ ] Solve the issue and close #0

🔧 Chore

[ ] Test case
[ ] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

📝 Description
🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self Check before Merge

[ ] Add or update relevant Docs.
[ ] Add or update relevant Demos.
[ ] Add or update relevant TypeScript definitions.

这种应该用 chore: xx 吧 不然 changelog 也带上了，还是英文的 commit 。。。

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ES query builder

Coverage remained the same at 100.0% when pulling 612a40bca8ed361a3d69467a3efb40b9b24d85c4 on es-query-builder into 94350ea002e84ad474c36b3811c2d15c02619c11 on master.

Coverage remained the same at 100.0% when pulling d3147cd15cc53c0c6ab40830c8af93ef5e84cfab on es-query-builder into 94350ea002e84ad474c36b3811c2d15c02619c11 on master.

Coverage remained the same at 100.0% when pulling b7cf78e7ab6c703ff9f79525cd92d0c68cc68174 on es-query-builder into 94350ea002e84ad474c36b3811c2d15c02619c11 on master.

Coverage remained the same at 100.0% when pulling c85946a71554d28fb20a8d89cc2608f009b3f4bf on es-query-builder into 94350ea002e84ad474c36b3811c2d15c02619c11 on master.

rebase por favor


As of version 0.38.4 volar Formatter conflicts with ESLint

Test in ESLint Standard mode
Volar formatting automatically removes Spaces before function argument parentheses when adding project folders to vscode workspace,This causes a ESLint error: Missing space before function parentheses
The following Settings have been set
"javascript.format.insertSpaceBeforeFunctionParenthesis": true
data() {   !!Missing space before function parentheses.
    ...
}

This doesn't happen when you open the project folder directly and will work with ESLint
data () {   \\It's Ok.
    ...
}

javascript.format.insertSpaceBeforeFunctionParenthesis works for me.
In case you're using lang="ts", please make sure you also have config typescript.format.insertSpaceBeforeFunctionParenthesis.

This is fine if you open the project folder directly, but adding the project to the vscode workspace will cause the above problems.

Maybe you have workspace VSCode settings which override insertSpaceBeforeFunctionParenthesis value.

same issue here
insertSpaceBeforeFunctionParenthesis does not resolve
volar v1.2.0


added snap support

maybe it's useful to combinate it with apt/dnf -> apt/dnf/snap

Sorry for late reply. Got caught up in exams. Shall review and merge next week.

No problem. Here is an updated version.
https://github.com/tuxfy/ProxyMan/blob/master/dists/snap.sh
Added missing touch command for initial creating the conf file


Feature to calculate db_payload_size and response_payload_size

Hello, @DmitryTsepelev !
#1
Here I add railties for adding logs about Db payload size and response payload size. And we`ll see in logs like this
Completed 200 OK in 114ms (Views: 44.4ms | ActiveRecord: 13.8ms | DB Payload: 69.3kb | Body Payload: 29.3kb | Allocations: 12382)

Also, this information (db_payload_size and response_payload_size) added to 'process_action.action_controller' of ActiveNotification
I patched ActiveRecord::Relation and ActiveRecord::Calculations, for calculate bytes of result (AR extension)
DB payload size
For response_body I added railtie ControllerResponsePayloadSize and calculate response_body before render
For calculating I used Marchal.dump(some_object).size
If result contains AR object (#<Transaction id: ..., token: ....>), that means more allocated memory, and we will calculate this, not only rows data.
Difference between (for example)
Transaction.select(:token)

and
Transaction.pluck(:token)

Before, I also tried to use ObjectSpace.memsize_of, but this is not performance, because we need to find all related objects and caculate them
if ratio less than threshold, will send notificaion (of log, dependencies configuration) with message 'I/O to response payload ratio is ratio, while threshold is threshold'
Also, we can configure own notifications, by using add_notification, for example
IoToResponsePayloadRatio.configure do |config|
  config.add_notification(slack: SomeClass::Slack)
end

For monitoring data, include to controller
class TransactionsController < ApplicationController
  include IoToResponsePayloadRatio::Controller
end

And, I am ready for fixing code, if needed)

Got it! Thanks for the hard work again, and please let me know if you have any other ideas for the lib!


python312Packages.pynmea2: add patch to remove imp

Description of changes
Fix build (https://hydra.nixos.org/build/259361996)
ZHF: #309482

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 312718 run on x86_64-linux 1

6 packages built:

python311Packages.pynmea2
python311Packages.pynmea2.dist
python312Packages.pynmea2
python312Packages.pynmea2.dist
satellite
satellite.dist


Improve parent-organizations-team. Optionally can also create parent teams

Add an optional parameter that creates parent teams that don't exist at the target.
This is recursive so all team parents all the way up to the root are created
If this parameter is not set, it reverts to the old behavior of issuing an message the parent doesn't exist.
Uses a internal script called __copy_team_and_parents_if_not_exists_at_target.sh since it's recursive

🎉


Fix code scanning alert - Server-side requests should not be vulnerable to forging attacks

"this code carries the risk of SQL injection. Parameters should be securely inserted when adding data to the database."
Tracking issue for:

[ ] https://github.com/delacortezz/juice-shop/security/code-scanning/139

this code is vulnerable to SSRF as it performs an HTTP request to a URL defined by untrusted data.
Some websites I use as resources :
https://rules.sonarsource.com/csharp/RSPEC-5144/
https://brightsec.com/blog/ssrf-server-side-request-forgery/

Code fixed ....


Two Audio Triggers Volume Sliders Do Not Work Independently

I added two audio triggers (with 5 audio spectrum bars each).
Each audio trigger shows a volume slider on the right,
If both audio triggers are activated at once and I change the volume slider for the first, it affects the 2nd audio trigger as well. But the volume slider in the 2nd audio trigger does not change.

OS: Windows 11
QLC+ Version 4.12.7

Volume faders are now in sync. Thanks for reporting.
Please keep in mind that there is only one audio source so multiple audio triggers can't have independent volume levels (it's like changing the Windows microphone input level)


Client/Admin: Geo-Cord search set to DD and enter a DMS coordinates

In the client and had the Geo-Cord search set to DD and enter a DMS coordinates. It move me to the wrong location but if you have the Geo-Cord set to DMS at the time of the search it will work correctly. Here is the DMS coordinates that I enter which is one of the Melbourne image.
285959.75N 0805959.75W

Did a major refactor on the zoomTo module.  Needed to change some of the regular expressions that were being used for the DD, DMS, and MGRS inputs.
Changed the way the RegExp object was being used from RegExp.$N to pulling the actual parsed items from the array returned in the regular expressions per: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/n
Pulled in the mgrs.js library (https://github.com/proj4js/mgrs) to parse the coordinates coming in from the input.  Also moved the dmsToDd function from coordinate.js library to zoomTo.js.  This removes the dependency on that library for the DMS to DD functionality.
Hopefully this module can now easily be reused in our other OL3 projects.


Cannot create multiple instances of the read journal

JdbcReadJournal starts an actor when a query for events by tag is issued:
https://github.com/dnvriend/akka-persistence-jdbc/blob/ca058d607fae67758bce37c9d1b1002114b9f2f0/src/main/scala/akka/persistence/jdbc/query/scaladsl/JdbcReadJournal.scala#L95-L96
Akka Persistence allows for the creation of multiple instances of a read journal, with different configuration paths. Because the JournalSequenceActor is created with a hardcoded name, as a top-level actor, this causes a conflict when there is more than one JDBC read journal created:
[error] akka.actor.RepointableActorRef [sourceThread=engage-employee-service-application-akka.actor.default-dispatcher-18, akkaTimestamp=18:46:12.462UTC, akkaSource=akka://engage-employee-service-application/user/StreamSupervisor-1/flow-8-0-ignoreSink, sourceActorSystem=engage-employee-service-application] - Error in stage [akka.stream.impl.UnfoldAsync@440425b8]: actor name akka-persistence-jdbc-journal-sequence-actor] is not unique!
akka.actor.InvalidActorNameException: actor name [akka-persistence-jdbc-journal-sequence-actor] is not unique!
at akka.actor.dungeon.ChildrenContainer$NormalChildrenContainer.reserve(ChildrenContainer.scala:129)
at akka.actor.dungeon.Children.reserveChild(Children.scala:134)
at akka.actor.dungeon.Children.reserveChild$(Children.scala:132)
at akka.actor.ActorCell.reserveChild(ActorCell.scala:370)
at akka.actor.dungeon.Children.makeChild(Children.scala:272)
at akka.actor.dungeon.Children.attachChild(Children.scala:48)
at akka.actor.dungeon.Children.attachChild$(Children.scala:47)
at akka.actor.ActorCell.attachChild(ActorCell.scala:370)
at akka.actor.ActorSystemImpl.actorOf(ActorSystem.scala:718)
at akka.persistence.jdbc.query.scaladsl.JdbcReadJournal.journalSequenceActor$lzycompute(JdbcReadJournal.scala:97)
at akka.persistence.jdbc.query.scaladsl.JdbcReadJournal.journalSequenceActor(JdbcReadJournal.scala:95)
at akka.persistence.jdbc.query.scaladsl.JdbcReadJournal.retrieveNextBatch$1(JdbcReadJournal.scala:196)
at akka.persistence.jdbc.query.scaladsl.JdbcReadJournal.$anonfun$eventsByTag$1(JdbcReadJournal.scala:228)

The ReadJournalProvider implementation can take the config path as an optional third parameter, which can be used to namespace the actor name.

Good find, I did not know it was possible to create multiple instances. We should definitely include a test for this scenario


Use HDK class in examples + some new examples

This PR introduces the following example usages:

Change basic_python_usage.ipynb to use HDK class (now used in all examples).
Taxi example simplified due to HDK class usage.
Manually construct query graphs.

Showcase the benefit of compilation.

Show how one can run a query graph on CPU/GPU/some part on CPU and some part on GPU.

Allow relatively easy table generation for heterogeneous aggregation performance analysis (with plots).

Some minor updates in HDK class in hdk.py: allow table refragmenting and easy gpu memory cleanup.

Looks good. Please check the debug_logs usage.


Reuse provider instance within session

Hi. We've found bug in session providers.
Please check out test. If I define foo before bar it works as expected.
If bar before foo then foo provider is called twice.

Hey! Sounds good to me, thanks for this.
I made an experiment some time ago and master is currently not ready to accept big fixes for 1.x, so I need to move things around first and then I’ll merge and release this. :)

Thanks @dizballanze!

1.2.9 released. 😉


Add moth plushie

Moth plushie. It's cute, don't mind, merge it.
About the PR
Add moth plushie.
Why / Balance
Why is it must be in the game? It's cute.
Technical details
It's my first PR, pls, don't kill me.
Media
https://github.com/space-wizards/space-station-14/assets/106442259/661423c0-5fb1-4488-aeea-a642b53be291

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
add: Added moth plushie. Enjoy it.

you should give it one of the moth sounds for it's interaction sound :D

Uh... I will try, okay.

https://github.com/space-wizards/space-station-14/assets/106442259/9de6e6c2-62f3-492f-8bbc-1782a5da4e49
Now good?

cl is wrong
:cl:

add: Added sus.

The changelog should be fixed, but other than that it looks good.

I hope now it's fixed..


Instances

Instances within Lunar describes an interior that is networked. For example rooms, interiors, dungeons, etc. They exist within a world.
As such a few issues come up:

worldFrom should it still be loaded and receiving/updating data?

If not, when entering the instance it could be slow to load back in. This could be combated with a loading screen but may not be desirable.

Should probably intiliaze a new LunarWorld but a very barebones implementation to prevent too many resources being used from a simple interior.

Client Design

Simple implementation of LunarWorldSkeleton.

Separate box2d world and entity/player lists.
worldFrom should probably stop updating itself while in interior, implement some kind of lock/pause state for that.

(unless desired)

Network server should send packets indicating a player has enter the interior, not the parent world.
Possibly include a max player capacity for interiors (configurable) (eg. "This room is already full")
Will need interior spawn flag and events.
Once player leaves interior, keep the interior around in memory until they walk X distance away? (configurable)

Alt, X seconds after destroy, or, destroy immediately depending on how large the interior is.

If player gets within X distance of interior, load it in depending on how large it is? (configurable)
Remove player from parent world, once left, add them back.

Server Design

remove player from parent world. Once left, add them back.
Max player capacity for interiors. (configurable)
Only send the player updates from within the interior, not from whole world.
...

Current progress as of dcd18f6

Added base Instance classes that extend their respective worlds from client and server
Added protocol infrastructure to support entering instances
Added a new method for handling creating new Box2d bodies (EntityBodyHandler) and (EntityBodyCreator)

This allows for multiple bodies to be created for when entering instances and worlds

TODO:

We need to separate worlds and instances more, its all too cluttered and confusing to understand where data is going.
Server needs to be heavly refactored, janky and cluttery mess of code.
Instances need more work and implement the configuration values for them
Leaving instances, and other actions within them.
...

Old.


unused: sumnode has its own lt

This is unused code, since the SumNode has it's own __lt__ here https://github.com/tinygrad/tinygrad/blob/master/tinygrad/shape/symbolic.py#L273

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/shape/symbolic.py                 2              16
------------------------------------------------------------
lines added in the tinygrad folder: -14


317 questionnaire settings

What is the context of this PR?

Moves questionnaire creation and updating of questionnaire meta data to a modal.
Removes redundant route (/create), and unused code (QuestionnaireProperties).
Refactors Modal component to be more useful on its own
Refactor QuestionPageToolbar to use ModalTrigger component

How to review
Check it matches design, and ensure code is good

Closing this PR, as it got a bit of a mess after rebasing. Will make a fresh PR


Table header displayed twice

The table column names are displayed twice, yet I don't think I'm doing anything weird in my code:
I have a container component with the following render function
render() {

      {data && !res &&
        data.map(function mapData(places, index) {
          return <PlaceTable key={index} places={places}/>;
        })}

}

And here is my render function of the PlaceTable element:
    return (

{name}

        <BootstrapTable data={places} striped={true} hover={true} pagination={true} search={true} exportCSV={true} selectRow={selectRowProp} ref={name}>
          <TableHeaderColumn dataField="name">Name</TableHeaderColumn>
          <TableHeaderColumn dataField="address" isKey={true} dataAlign="center" >Address</TableHeaderColumn>
        </BootstrapTable>

    );

Am I missing something?

Include css/react-bootstrap-table-all.min.css will solve this issue :)

Thank you for helping out, indeed my CSS inclusion wasn't working, see this issue explaining why and how to make it work.
https://github.com/erikras/react-redux-universal-hot-example/issues/750

I'll check this out. Thanks

I'm pretty sure react-bootstrap-table had nothing to do with the issue, just the way css are included in this boilerplate can be a bit confusing ;)

ok, I'm not familiar with that repo, but this is a issue first time that I see, so how did you solve it? follow this thread?

Yeah, I tried this solution and it works too.
Thank you for your time :)

Great :)  Thanks for these feedback :+1:


Fix onnx_chainer.replace_func.fake_as_funcnode to reconstruct return value structure

With the current implementation, alt_func is allowed to return a dict, but it is flattened into a tuple. For example,
def f(x):
    return {"a": x+1, "b": x-1}

returns a dict, while onnx_chainer.replace_func.fake_as_funcnode(f, 'F') returns a tuple, so that the consumer of the result who expects dict does not work.
This PR fixes the problem by explicitly saving the structure as a skeleton and reconstructing original structure from the skeleton and flattened values.
As a side effect, not only a single level of tuple/list/dict but also arbitrary nesting of tuple/list/dict is allowed in return value.

Jenkins, test this please!

Jenkins CI test (for commit 374525ac00e77d1e850f5013ec9ca22957b4de4f, target branch master) failed with status FAILURE.

Jenkins. test this please!

Jenkins CI test (for commit 374525ac00e77d1e850f5013ec9ca22957b4de4f, target branch master) succeeded!

Jenkins CI test (for commit 374525ac00e77d1e850f5013ec9ca22957b4de4f, target branch master) succeeded!


HTTP Batching

Requests that go the same node could be collected and sent in one http-request to improve performance. The node's API endpoint would need to be able to take a list as parameter. Alternatively HTTP/2 should be a more standard-conformous way to do this. On the client, the linkeddata-service would need to collect all requests over a certain period of time: e.g. wait for 200ms after the latest request with max 1000ms after the first in the batch, then send everything.

Falcor's batching might be interesting as reference: http://netflix.github.io/falcor/video-tutorials/batching-requests.html

Cleaning up old issues.


Apply orthologues species filters

Description

Added the functionality to filter the orthologues downloads based on the filters applied on the table

Views affected
http://ves-hx2-75.ebi.ac.uk:42229/Mus_musculus/Gene/Compara_Ortholog?g=ENSMUSG00000017167;r=11:101170523-101190724
Related JIRA Issues (EBI developers only)
https://www.ebi.ac.uk/panda/jira/browse/ENSWEB-6112

@ens-ap5  - Sorry, I missed that bit. I have updated the description now.

@ens-ap5 - Sorry, I missed that bit. I have updated the description now.

Thanks! I just want to check that I understood what was going on and that the Perl changes didn't conflict with the fix that I put in before the 103 release


Quick start problem of getting result from MMS

Hello, I have been trying to set up MMS on Ubuntu machine but encountered problem. It is appreciated if you could offer help in solving such problem

Ubuntu Server on AWS
I was using Deep Learning AMI (Ubuntu 16.04) Version 26.0 on AWS, while I was following the instructions on https://github.com/awslabs/multi-model-server/blob/master/README.md, it was smooth. One difference is that I have python 3 on my server so I used python 3 instead of python 2.7. But when I try to get the result from localhost:8080, with the command you have provided on the tutorials, it turned to be an infinite wait.
curl -X POST http://<IP_ADDRESS>:8080/predictions/squeezenet -T kitten.jpg
The command above never returns the result. When I try to access the server from a new terminal without terminating this process, and use the top command, I came to see that the CPU was 0% utilization. I am wondering what was happening.

As I have checked the AWS EC2 security policies, I have found out that this was because the default security policy of my server has disabled the 8080 port. So I will never get response from the 8080 port. Therefore, in order to circumvent this problem, I have to change the listening port of MMS. Changing method follows:

Navigate to the folder where you wish to start the MMS. I recommend using a fixed folder to start the MMS service.
Create a new file called config.properties.
Modify the content of the file and add the following line:
inference_address=https://<IP_ADDRESS>:80
Save this file and then start the MMS server from this path.

Please also note this method might not work due to the location of configuration file might vary. This part of the documentation describes about where to locate the configuration file. I tried the method above on native AWS EC2, but if your environment has other configurations, please change it accordingly.
For more information about listening port configuration, please refer to the MMS configuration documentation page
@eric-haibin-lin @leezu FYI

Currently, as I try out using the starting MMS from a clean virtual environment, I have run into two new problems. Maybe you can help about it. I am running on a AWS EC2 GPU instance.
Preparation work: I created a new virtual environment, installed the necessary packages and installed MMS, the command lines attached in the attachment below:
Environment Preparation.docx

If I don't specify the port, using the default one, I will never finish starting MMS. That is, the starting will continue to run all the time, with continuous outputs, rather than halting at an error or saying a successful start. The output of this part is shown in the attachment below:
Infinite Start.docx

If I specify the port, using port 80, and then granting this process elevated privilege, it will give an error without successful start. The output of this part is shown in the attachment below:
Specify Port Start Error.docx


GIS needs POPS as zipped shapefile and csv

a GIS script pulls POPS data from edm-recipes/datasets/dcp_pop/latest/ and expects to find dcp_pops.shp.zip and dcp_pops.csv.zip
however we stopped archiving source data to those formats, so there is currently only dcp_pops.csv, dcp_pops.parquet, dcp_pops.sql in that folder. the last time we archived a POPs shapefile was version 20240131

Is there a reason they need both the csv and the shapefile?
Also, is this need driven by ArcPy, or more that this actual shapefile needs to be distributed? If the former, could arcpy just pull in the geoparquet file instead?

@caseysmithpgh I used the csv from dcp_pop/20240814/ to generate dcp_pops.shp.zip and dcp_pops.csv.zip and put them in both dcp_pop/20240814/ and dcp_pop/latest/. I ran the code below in a jupyter notebook to create the shapefile:
import geopandas as gpd
gdf = gpd.read_file(r'/Users/damonmccullough/Downloads/dcp_pops.csv')
gdf.to_file("dcp_pops.shp/dcp_pops.shp")

If either of those files are giving your scripts issues, happy to troubleshoot

I need to take a closer look at the utils script this is referencing--but tagging @jackrosacker just for an FYI

@fvankrieken
Is there a reason they need both the csv and the shapefile?
Also, is this need driven by ArcPy, or more that this actual shapefile needs to be distributed? If the former, could arcpy just pull in the geoparquet file instead?

I'm not sure, haven't see their script(s) yet but would love to change this dependency on a recipes shapefile. seems like two long-term options are:

DE publishes POPS to edm-publishing/ for GIS scripts to get a "transformed" version of POPS
GIS script pulls in dcp_pops.csv from edm-recipes and generates the shapefile

Our script expects to take a csv and shapefile, and output a csv, shapefile, and geodatabase for publication. The codeas written takes the shapefile, converts to feature class, changes dataset schema, and then exports a fresh shapefile. This results in an unchanged csv, and a feature class and shapefile each with the new dataset schema.
If I'm remembering correctly, we chose this route to avoid using geopandas, as there are occasionally some conda environment issues on our end when we introduce geopandas. We can happily work around that now/soon.
For this release, I'm hoping that changing the zipped shapefile on your end solves the immediate problem so that we can publish this release without changing code. Then for next release we can use geopandas to ingest csv/geoparquet > apply schema changes to the geodataframe > write to shapefile and feature class.

Regenerated zipped csv and shapefile using data library to ensure access level is correct

Issue was resolved once access level was adjusted. Thanks!


update checkbox/radio documentation with block example, fixed failing builds

love you dudes

@sukhrajghuman Finally able to rebase this branch properly after temporarily turning off branch protection for develop. I will turn it back on if @alex-page prefers it but I found it immensely difficult to successfully rebase with it on.


Problem with auto completion

The 'auto completion' is not 'to complete', but to replace. If I type 'fs', then the suggestion is 'fs.close()', I hit tab(or any other key I set), the result will be 'close()', not 'fs.close()'.
sublime version: v3 3143 (x64)
And it makes Javascript completion plugin can not  auto-complete,  I am not sure whether this is nodejs's problem or  that plugin's.

Could you try install last version of the plugin from the refactor branch ?

i have the same problem

@varp Yeah, the problem fixed. Thanks.


New feature? Event Callback functions

Great work with this plugin.
I would love a way to listen to events, I've written an example of what I would like to achieve.
<ReactRotatingText
    items ={['events', 'would', 'be', 'awesome']}
    onTypingStart={console.log('Typing has started')}
    onTypingEnd={console.log('Typing has finished')}
    onDeletingStart={console.log('Deleting has started')}
    onDeletingEnd={console.log('Deleting has finished')}
/>

Could anybody help steer me in the direction of how I would achieve something like this?
My use case is that I would like to update an image onTypingEnd().
Obviously the names of the events can change, it's just an example at this point.
Thanks.

To be quite honest, even I don't really recall how I wrote this component. I'm thinking I'm going to do a re-write with RxJS some time in the future. But until then, if anyone else has an idea of where to inject those callback props, that would be great.

I have some ideas, but there's some weird stuff going on with props.items.
Its like the default values ['first',second','third'] are still in use.
Will provide some examples in due course.

#18


Add validation requirements for attachments

Currently file attachments can be uploaded without any checks on the file.The FileAttachmentUploadChecker has been created to check the validity of the file attempted to be uploaded.
There are two input types, one being the file itself and the other the file's title. The file's title has two checks being its presence and has a max length of 255 chars. The file check itself validates its presence and whether its of a supported format and if it's a zip that it only contains files of supported formats.
Trello:
https://trello.com/c/EL9jPXIl/802-validation-requirements-for-uploading-file-attachments

Nice one @1pretz1 👍 . I forgot to suggest maybe editing the commit with the MIME type stuff to mention that we also disallow files with no extension where this is required to infer the type (e.g. text files), as only a few files seem to lack an extension in Asset Manager.

@kevindew ready for review no.2! :)

@kevindew I've made a few small changes since you last reviewed the pr, one being making sure the file_attachment_upload_service creates a blob with the same content-type as whats been checked in the checker.
Two, I've used file.path instead of just file for the arg when opening the zip as this previously was causing errors when running the app locally (however not in the test suite as the file upload object was different between environments).


Duplicate ActiveJob log entries

Hello, thanks for the hard work on this gem! It's been very useful. I was hoping if you could help with the following issue.
Gem Versions
rails (<IP_ADDRESS>)
activejob (<IP_ADDRESS>)
activejob-uniqueness (0.3.1)
rails_semantic_logger (4.14.0)
sidekiq (7.2.4)
sidekiq-pro (7.1.3)

Rails Logging Config
  config.log_level = :info
  config.i18n.fallbacks = true

  # Send deprecation notices to registered listeners.
  config.active_support.deprecation = :notify

  config.colorize_logging = false

  config.rails_semantic_logger.started    = true
  config.rails_semantic_logger.processing = true
  config.rails_semantic_logger.rendered   = false

The Issue
We are getting duplicate ActiveJob log entries when activejob-uniqueness is present in the Gemfile. The issue resolves as soon as we remove the gem from our code. Please note how both the jobs below have two log entries each for Performed.
Neither of these jobs use unique. Logs for all of our jobs are getting duplicated as soon as we include the gem.
2024-05-01 19:58:22.544696 I [1:sidekiq.default/processor] [PullLookalikePercentilesJob] [cd86b4d4-433f-4a2f-9584-de34a232f1e0] Rails -- Performed PullLookalikePercentilesJob (Job ID: cd86b4d4-433f-4a2f-9584-de34a232f1e0) to Sidekiq(default) in 5.89ms -- {:event_name=>"perform.active_job", :adapter=>"Sidekiq", :queue=>"default", :job_class=>"PullLookalikePercentilesJob", :job_id=>"cd86b4d4-433f-4a2f-9584-de34a232f1e0", :provider_job_id=>"51558e2c9e7da60762338772", :duration=>5.89, :arguments=>"[\n\n]"}

2024-05-01 19:58:22.544717 I [1:sidekiq.default/processor] [PullLookalikePercentilesJob] [cd86b4d4-433f-4a2f-9584-de34a232f1e0] Rails -- Performed PullLookalikePercentilesJob (Job ID: cd86b4d4-433f-4a2f-9584-de34a232f1e0) to Sidekiq(default) in 5.93ms -- {:event_name=>"perform.active_job", :adapter=>"Sidekiq", :queue=>"default", :job_class=>"PullLookalikePercentilesJob", :job_id=>"cd86b4d4-433f-4a2f-9584-de34a232f1e0", :provider_job_id=>"51558e2c9e7da60762338772", :duration=>5.93, :arguments=>"[\n\n]"}

2024-05-01 19:58:26.132405 I [1:sidekiq.default/processor] [ImportInventoryPackageSiteListJob] [a7018f7b-15b6-4e93-a4d3-d868a24cd91c] Rails -- Performed ImportInventoryPackageSiteListJob (Job ID: a7018f7b-15b6-4e93-a4d3-d868a24cd91c) to Sidekiq(default) in 1397.53ms -- {:event_name=>"perform.active_job", :adapter=>"Sidekiq", :queue=>"default", :job_class=>"ImportInventoryPackageSiteListJob", :job_id=>"a7018f7b-15b6-4e93-a4d3-d868a24cd91c", :provider_job_id=>"859c39a03444316f10b93ce5", :duration=>1397.53, :arguments=>"[\n\n]"}

2024-05-01 19:58:26.132448 I [1:sidekiq.default/processor] [ImportInventoryPackageSiteListJob] [a7018f7b-15b6-4e93-a4d3-d868a24cd91c] Rails -- Performed ImportInventoryPackageSiteListJob (Job ID: a7018f7b-15b6-4e93-a4d3-d868a24cd91c) to Sidekiq(default) in 1397.89ms -- {:event_name=>"perform.active_job", :adapter=>"Sidekiq", :queue=>"default", :job_class=>"ImportInventoryPackageSiteListJob", :job_id=>"a7018f7b-15b6-4e93-a4d3-d868a24cd91c", :provider_job_id=>"859c39a03444316f10b93ce5", :duration=>1397.89, :arguments=>"[\n\n]"}

Each enqueued, performing, performed etc log entry is repeated twice.

I think I have figured out what's causing this behaviour.
Rails semantic logger replaces the default ActiveJob::LogSubscriber with it's own custom log subscriber here.
activejob-uniqueness then inadvertantly reopens the ActiveJob::LogSubscriber class and adds it back as a subscriber in this file.

The issue is actually related to Rails 6.0.x.
Rails 6.0.x has ActiveJob::Logging::LogSubscriber.attach_to :active_job.
activejob-uniqueness has ActiveJob::LogSubscriber.attach_to :active_job_uniqueness.
rails_semantic_logger replaces both of these  with it's own logger here. And thus duplicating logging.

Version 0.3.2 with the fix has been shipped https://rubygems.org/gems/activejob-uniqueness/versions/0.3.2


⚠ SkipTheTrailers has degraded performance

In ff40eae, SkipTheTrailers ($STT) experienced degraded performance:

HTTP code: 200
Response time: 4129 ms

Resolved: SkipTheTrailers performance has improved in 16d96db.


Update provisioner to use ACM

This will update the application provisioner to use the ACM certificate created. Also fixed few of the chart issues.

LGTM


docs: use strict repo names in sample config

Aligns the provided sample config with the feature added in github/safe-settings#371 so it continues to match the intended repositories.
@decyjphr @martinm82 The change in #371 had some surprising side effects for me. Any repo containing the word admin became excluded and I also had my test repository named safe-settings-test excluded by the string 'safe-settings'.

Yes, I agree with this change. I hit this issue as well. Sorry that I missed that in the PR.


The project's Babel config is invalid: Cannot read properties of undefined (reading 'uid')

Hey everyone,
Describe the bug
I tried to start the blank page default project but I'm getting bable errors during building stage saying that

The project's Babel config is invalid: Cannot read properties of undefined (reading 'uid')

This is breaking the build and I can't start the project. Strange enough, If I start the helloworld-template it works just fine - same setup.
To Reproduce
Setup new project with blank page template (v.0.34) and try running it with rnv run -p web
Expected behavior
A clean build / nextjs startup which actually shows up in the browser
Screenshots or copy&paste

Desktop (please complete the following information):

OS: windows 7 10.0.19044
Node Version v16.13.1
RNV Version 0.34.0

Maybe someone can help?
Thanks and cheers,
Ben
Update
Recreated the project with node 14.18.2 and the error goes away. Now I can't seem to run builds for any targets involving webpack like webos throwing this error:

Recreated the project with node 14.18.2 and it works just fine. So, we can't have node 16?

+1

@r0oto0r @lednewcrk can you please try the latest version? I just tested this and I can't replicate it. Created a new project, picked template-blank and ran it on web, seems to work just fine

@r0oto0r hello, did latest versions fix your issue?

Hey, sorry I moved away from this project in the end, so we can close this issue. Thanks for your help!


erro in use without '_id'?

preuse collections with col1, it's ok and then wrong without '_id':
helpers.js?debc:139 Uncaught (in promise) TypeError: Cannot read property '_id' of undefined
    at addPersonalSigningKey (helpers.js?debc:139)
    at eval (user.js?6e98:144)

but change collection to new col2, it's ok again.
what's wrong on col1?

For me, I see the error when I use the same radiks server for localhost and production domain. If you remove the BlockstackUser in the mongo db the app should work again.

That worked for me thanks a lot!

Yes, you need a different radiks-server for apps on different domains. This is due to having different appPrivateKeys from Blockstack auth on different domains. Seems important to add to the documentation.


Meta Issue: make sure that skore brings benefits / is usable in plain vscode / jupyter

Is your feature request related to a problem? Please describe.
Right now, the whole user experience of skore assumes that we are going to use the skore UI (I don't remember how its called). But this is not currently how data scientists work, and they need some where to start.
Data scientists do interactive work in notebooks or vscode (or other environments, those are just the two most popular). We need to find a way for the usage of score to be beneficial very quick in such an environment.
Here is a screenshot of me trying to get some useful information from the CrossValidationReporter:

I struggle to get what I want, which is the report on my cross validation. I have instantiated a skore project to try to see if it would give me a report, though this is not what I was interested in in the short term.
Knowing the discussions around skore, I understand that I probably need to start the server, but nothing on the screen tells me to do it. I am not guided through the process.
Describe the solution you'd like
As a user, I need to find easy ways of getting benefit (eg having the report), without reading the docs.
We should probably consider giving value (showing easily the report) without the skore UI, so that people install us and start using the tool
Describe alternatives you've considered, if relevant
No response
Additional context
No response

If I put together:

the current issue
the issue mentioned in: https://github.com/probabl-ai/skore/issues/949
the issue mentioned in: https://github.com/probabl-ai/skore/issues/1002

I would be really happy as a user to write:
# %%
proj = skore.create()
reporter = proj.CrossValidationReport(estimator, X, y)

# %%
reporter.plot.roc()

and get the reporting activity happening that means:

# %% should trigger the equivalent of the skore.show_activity() from @GaelVaroquaux or the a kind of skore.launch() that start the UI within VS Code or a browser next to my notebook
since the call done with proj, having a callback to get sync information between the UI windows and the activity happening the reporter.
I can continue using my reporter object afterwards if I don't want to use the UI.


Support custom cargo check/run/build/test commands

Sometimes we don't just want to run cargo check/run/test/build commands, but also cargo xrun/xcheck/xtest/xbuild, the purpose of this pr is to support customization of these commands

:+1:


[Package Issue]: Henry++.simplewall cannot be updated

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
Winget detects a new release of simplewall, but running the upgrade throws an error.
Steps to reproduce
Have an older version of simplewall installed, try to upgrade it via WinGet
Actual behavior
> winget upgrade simplewall
Found simplewall [Henry++.simplewall] Version 3.7.5
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Successfully verified installer hash
Starting package install...
Installer failed with exit code: 2

Expected behavior
The package upgrades
Environment
Windows Package Manager v1.6.3133
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.3693
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.21.3133.0

Screenshots and Logs
2023-11-16 11:46:00.851 [CORE] WinGet, version [1.6.3133], activity [{91F9DDD9-AE63-4058-B6F9-E5CD7C10F317}]
2023-11-16 11:46:00.851 [CORE] OS: Windows.Desktop v10.0.19045.3693
2023-11-16 11:46:00.851 [CORE] Command line Args: "C:\Users\nikkeh\AppData\Local\Microsoft\WindowsApps\winget.exe" upgrade simplewall
2023-11-16 11:46:00.851 [CORE] Package: Microsoft.DesktopAppInstaller v1.21.3133.0
2023-11-16 11:46:00.851 [CORE] IsCOMCall:0; Caller: winget-cli
2023-11-16 11:46:00.861 [CLI ] WinGet invoked with arguments: 'upgrade' 'simplewall'
2023-11-16 11:46:00.862 [CLI ] Found subcommand: upgrade
2023-11-16 11:46:00.865 [CLI ] Leaf command to execute: root:upgrade
2023-11-16 11:46:00.867 [CLI ] Executing command: upgrade
2023-11-16 11:46:00.869 [REPO] Default source requested, multiple sources available, adding all to source references.
2023-11-16 11:46:00.869 [REPO] Adding to source references msstore
2023-11-16 11:46:00.869 [REPO] Adding to source references winget
2023-11-16 11:46:00.890 [CORE] Examining extension: PFN = Microsoft.Winget.Source_8wekyb3d8bbwe, ID = IndexDB
2023-11-16 11:46:00.890 [CORE] Found matching extension.
2023-11-16 11:46:00.894 [REPO] Source `winget` after auto update time [5 mins]; it has been at least 12 mins
2023-11-16 11:46:00.900 [CORE] Examining extension: PFN = Microsoft.Winget.Source_8wekyb3d8bbwe, ID = IndexDB
2023-11-16 11:46:00.900 [CORE] Found matching extension.
2023-11-16 11:46:01.002 [REPO] Multiple sources available, creating aggregated source.
2023-11-16 11:46:01.002 [REPO] Adding to aggregated source: msstore
2023-11-16 11:46:01.002 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/information
2023-11-16 11:46:01.120 [REPO] Response status: 200
2023-11-16 11:46:01.121 [REPO] Sending http GET request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/information
2023-11-16 11:46:01.137 [REPO] Response status: 200
2023-11-16 11:46:01.138 [REPO] Adding to aggregated source: winget
2023-11-16 11:46:01.144 [CORE] Examining extension: PFN = Microsoft.Winget.Source_8wekyb3d8bbwe, ID = IndexDB
2023-11-16 11:46:01.144 [CORE] Found matching extension.
2023-11-16 11:46:01.163 [REPO] Opening SQLite Index for ImmutableRead at 'C:\Program Files\WindowsApps\Microsoft.Winget.Source_2023.1116.933.355_neutral__8wekyb3d8bbwe\Public\index.db'
2023-11-16 11:46:01.163 [SQL ] Opening SQLite connection #1: 'C:\Program Files\WindowsApps\Microsoft.Winget.Source_2023.1116.933.355_neutral__8wekyb3d8bbwe\Public\index.db' [1, 40]
2023-11-16 11:46:01.164 [REPO] Opened SQLite Index with version [1.7], last write [2023-11-16 09:32:38.000]
2023-11-16 11:46:01.248 [REPO] Creating new SQLite Index with version [Latest] at ':memory:'
2023-11-16 11:46:01.248 [SQL ] Opening SQLite connection #2: ':memory:' [6, 0]
2023-11-16 11:46:01.254 [REPO] Reading MSI UpgradeCodes
2023-11-16 11:46:01.354 [FAIL] C:\__w\1\s\external\pkg\src\AppInstallerRepositoryCore\Microsoft\Schema\1_0\Interface_1_0.cpp(204)\WindowsPackageManager.dll!00007FF9FF9554E5: (caller: 00007FF9FF95B959) Exception(1) tid(38ec) 800700B7 Nie można utworzyć pliku, który już istnieje.

2023-11-16 11:46:01.354 [REPO] Ignoring duplicate ARP entry Machine|X86|7-Zip [7-Zip 22.01]
2023-11-16 11:46:01.430 [REPO] Reading MSI UpgradeCodes
2023-11-16 11:46:01.614 [SQL ] Opening SQLite connection #3: ':memory:' [6, 0]
2023-11-16 11:46:01.623 [REPO] Finding available package from installed package using system reference search: Query:[none] Include:ProductCode='simplewall'[Exact] Include:NormalizedNameAndPublisher='simplewall'+'henry'[Exact]
2023-11-16 11:46:01.631 [REPO] Opening SQLite Index for ReadWrite at 'C:\Users\nikkeh\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\StoreEdgeFD\installed.db'
2023-11-16 11:46:01.631 [SQL ] Opening SQLite connection #4: 'C:\Users\nikkeh\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\StoreEdgeFD\installed.db' [2, 0]
2023-11-16 11:46:01.632 [REPO] Opened SQLite Index with version [1.5], last write [2023-02-15 16:46:35.000]
2023-11-16 11:46:01.638 [REPO] Opening SQLite Index for ReadWrite at 'C:\Users\nikkeh\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db'
2023-11-16 11:46:01.638 [SQL ] Opening SQLite connection #5: 'C:\Users\nikkeh\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db' [2, 0]
2023-11-16 11:46:01.639 [REPO] Opened SQLite Index with version [1.5], last write [2023-11-16 11:34:28.000]
2023-11-16 11:46:01.660 [REPO] Sending http POST request to: https://storeedgefd.dsx.mp.microsoft.com/v9.0/manifestSearch
2023-11-16 11:46:01.680 [REPO] Response status: 200
2023-11-16 11:46:01.703 [REPO] Opening pinning index
2023-11-16 11:46:01.703 [REPO] Opening existing pinning index
2023-11-16 11:46:01.703 [REPO] Opening SQLite Index for ReadWrite at 'C:\Users\nikkeh\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\pinning.db'
2023-11-16 11:46:01.703 [SQL ] Opening SQLite connection #6: 'C:\Users\nikkeh\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\pinning.db' [2, 0]
2023-11-16 11:46:01.703 [REPO] Opened Pinning Index with version [1.0], last write [2023-07-15 19:44:33.000]
2023-11-16 11:46:01.706 [CLI ] Found one app. App id: Henry++.simplewall App name: simplewall
2023-11-16 11:46:01.708 [REPO] Downloading manifest
2023-11-16 11:46:01.708 [CORE] WinINet downloading from url: https://cdn.winget.microsoft.com/cache/manifests/h/Henry++/simplewall/3.7.5/c931
2023-11-16 11:46:02.305 [CORE] Download hash: 90bd6af642b38cdf8c071c46b62685d31504462636748e354dcb126efd952566
2023-11-16 11:46:02.305 [CORE] Download completed.
2023-11-16 11:46:02.306 [CLI ] Starting installer selection.
2023-11-16 11:46:02.306 [CLI ] Completed installer selection.
2023-11-16 11:46:02.306 [CLI ] Manifest fields: Name [simplewall], Version [3.7.5]
2023-11-16 11:46:02.320 [CLI ] Found existing installer file at 'C:\Users\nikkeh\AppData\Local\Temp\WinGet\Henry++.simplewall.3.7.5\simplewall-3.7.5-setup.exe'. Verifying file hash.
2023-11-16 11:46:02.324 [CLI ] Existing installer file hash matches. Will use existing installer.
2023-11-16 11:46:02.324 [CLI ] Installer hash verified
2023-11-16 11:46:02.324 [CORE] Started applying motw to C:\Users\nikkeh\AppData\Local\Temp\WinGet\Henry++.simplewall.3.7.5\simplewall-3.7.5-setup.exe with zone: 2
2023-11-16 11:46:02.327 [CORE] Finished applying motw
2023-11-16 11:46:02.327 [REPO] Creating new SQLite Index with version [Latest] at ':memory:'
2023-11-16 11:46:02.327 [SQL ] Opening SQLite connection #7: ':memory:' [6, 0]
2023-11-16 11:46:02.333 [REPO] Reading MSI UpgradeCodes
2023-11-16 11:46:02.470 [FAIL] C:\__w\1\s\external\pkg\src\AppInstallerRepositoryCore\Microsoft\Schema\1_0\Interface_1_0.cpp(204)\WindowsPackageManager.dll!00007FF9FF9554E5: (caller: 00007FF9FF95B959) Exception(2) tid(38ec) 800700B7 Nie można utworzyć pliku, który już istnieje.

2023-11-16 11:46:02.470 [REPO] Ignoring duplicate ARP entry Machine|X86|7-Zip [7-Zip 22.01]
2023-11-16 11:46:02.541 [REPO] Reading MSI UpgradeCodes
2023-11-16 11:46:02.719 [CLI ] Installer args: /S
2023-11-16 11:46:02.720 [CLI ] Starting: 'C:\Users\nikkeh\AppData\Local\Temp\WinGet\Henry++.simplewall.3.7.5\simplewall-3.7.5-setup.exe' with arguments '/S'
2023-11-16 11:46:04.681 [CLI ] ShellExecute installer failed: 2
2023-11-16 11:46:04.682 [CLI ] Terminating context: 0x8a150006 at C:\__w\1\s\external\pkg\src\AppInstallerCLICore\Workflows\InstallFlow.cpp:1fe

On a clean install, I don't get any errors. However, I did try running simplewall and then upgrading and saw the behavior. I believe exit code 2 means that it couldn't access the file. The specific error I get is Can't write: C:\Program Files\simplewall\simplewall.exe.  Try making sure that simplewall is closed and then running the upgrade again.

Closing the program before upgrade fixes the issue. I hope it's possible to make winget automatically close the process and relaunch it after upgrade, or show a warning instead of just failing.


How can I set js_errors to false ?

Anyway, great library :+1:

Hey @changwuf31 sorry for the late reply, I got no notifications for this. I have worked with documentation, i think the answer is here:
http://gastonjs.readthedocs.org/en/latest/api/commands/javascript/set_js_errors/

Arr.. thank u, but i'm using it with the php driver.. Guess I should open the issue there ?

Yeah, open it there.

@jcalderonzumba what about being able to preconfigure this on phantomjs boot script? See https://github.com/jcalderonzumba/gastonjs/pull/17


Fix Calendar Date Validation to Restrict Past Dates

Related Issue
None
Fixes:  #857
Description
Updates the calendar date validation to prevent users from selecting dates prior to the current date, ensuring valid and up-to-date submissions.
Type of PR

[X] Bug fix

Screenshots
Before

After

Checklist:

[x] I have made this change from my own.
[x] My code follows the style guidelines of this project.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@akash70629 Your pull request has been successfully merged! Don't stop here—continue contributing by addressing more issues and aiming to become a top contributor. Exclusive swag awaits for those who go the extra mile!
Great contribution ! 🎉

@akash70629 Due to a conflicting issue, some code was removed from the repository. We are investigating the matter. Meanwhile kindly create the PR again for merging.Please Do Sync fork before then add your part in code  BuddyTrail. Thank you for your understanding!

@akash70629 Due to a conflicting issue, some code was removed from the repository. We are investigating the matter. Meanwhile kindly create the PR again for merging.Please Do Sync fork before then add your part in code BuddyTrail. Thank you for your understanding!

@PriyaGhosal It's strange 😕😕 !!! Anyway Here is the same PR #912 merge it .


Add pin to Taskbar on context menu

Add pin to Taskbar on context menu for windows 11

Sorry, but could provide more details? I'm not quite understand.


Please remove the QR code step

When browsing and downloading a iOS test builds on a iOS device on App Center the flow always interrupts with a popup with a QR code before actually going to the download page.
This QR code step is completely pointless.
I am already on the mobile device... what would I need the QR code for?
Do you expect me to do a screen capture or print it on paper? And then download it? On PC this step does not popup (here it actually would make more sense as your NOT on mobile.).
As an alternative, for those rare cases where user might actually like to distribute the app using QR codes. (I.e. on a physical paper print.) Then add an option on the actual download page to share a link to the app with QR code. In example in the dropdown menu with the three dots "..." that exists on that page.
I have no idea what the UI designer was thinking, but they really did not have the most obvious and common use case in mind.
Thanks.

Dennis Belfrage / Supercell

Ahaa... looks like changeing the url from https://appcenter.ms/... to https://install.appcenter.ms/... makes this step go away.