tag 0.12.3

Purpose

tag bi-weekly release

@kmdeck CI error from the snow model - is that an easy fix? (I'll have a look too)
  1 dependency had output during precompilation:
┌ KrylovKit
│  WARNING: using ChainRulesCore.add!! in module KrylovKit conflicts with an existing identifier.
└
     Testing Running tests...
Test Summary: | Pass  Total   Time
Aqua tests    |   10     10  11.5s
Test Summary:                              | Pass  Total   Time
Richards model implicit timestepping tests |   52     52  52.8s
Test Summary:        | Pass  Total   Time
Domains module tests |   59     59  14.5s
Test Summary:           | Pass  Total   Time
General utilities tests |   43     43  13.8s
Test Summary:        | Pass  Total  Time
Variable types tests |   42     42  0.6s
Test Summary: | Pass  Total  Time
Driver tests  |   51     51  2.2s
Test Summary:             | Pass  Total   Time
Bucket albedo types tests |   17     17  26.9s
Test Summary:     | Pass  Total     Time
Bucket snow tests |   76     76  1m07.1s
Test Summary:     | Pass  Total  Time
Bucket soil tests |   52     52  7.5s
Snow Parameterizations, FT = Float32: Test Failed at /Users/runner/work/ClimaLand.jl/ClimaLand.jl/test/standalone/Snow/parameterizations.jl:81
  Expression: all(q_l[T_bulk .< _T_freeze] .< eps(FT))

@kmdeck CI error from the snow model - is that an easy fix? (I'll have a look too)
  1 dependency had output during precompilation:
┌ KrylovKit
│  WARNING: using ChainRulesCore.add!! in module KrylovKit conflicts with an existing identifier.
└
     Testing Running tests...
Test Summary: | Pass  Total   Time
Aqua tests    |   10     10  11.5s
Test Summary:                              | Pass  Total   Time
Richards model implicit timestepping tests |   52     52  52.8s
Test Summary:        | Pass  Total   Time
Domains module tests |   59     59  14.5s
Test Summary:           | Pass  Total   Time
General utilities tests |   43     43  13.8s
Test Summary:        | Pass  Total  Time
Variable types tests |   42     42  0.6s
Test Summary: | Pass  Total  Time
Driver tests  |   51     51  2.2s
Test Summary:             | Pass  Total   Time
Bucket albedo types tests |   17     17  26.9s
Test Summary:     | Pass  Total     Time
Bucket snow tests |   76     76  1m07.1s
Test Summary:     | Pass  Total  Time
Bucket soil tests |   52     52  7.5s
Snow Parameterizations, FT = Float32: Test Failed at /Users/runner/work/ClimaLand.jl/ClimaLand.jl/test/standalone/Snow/parameterizations.jl:81
  Expression: all(q_l[T_bulk .< _T_freeze] .< eps(FT))

I have seen this too. It seems intermittent, though.... I just retried the job in CI, let's see if it passes. we should fix this at some point...

@kmdeck there is also an error to build the snow tutorial in the docs (see CI) - did you see that before?

@kmdeck there is also an error to build the snow tutorial in the docs (see CI) - did you see that before?

I have not seen it, but perhaps @a-charbon has? https://github.com/CliMA/ClimaLand.jl/actions/runs/9473604094/job/26101550233?pr=653#step:5:48

By the way, I was trying to run a coupled run with the latest climaland, and there is a bug.
@Sbozzolo added the thermo_params to the PrescribedAtmosphere object: https://github.com/CliMA/ClimaLand.jl/blob/97f06a17e349b8666b6b73d89657b4bdaae03c4a/src/shared_utilities/drivers.jl#L89
and since we effectively specify this multiple times in the model, there is also this check:
https://github.com/CliMA/ClimaLand.jl/blob/97f06a17e349b8666b6b73d89657b4bdaae03c4a/src/shared_utilities/models.jl#L391
which will fail when we use the CoupledAtmosphere atmos type.
I am trying to fix this ahead of time before we do this release

@kmdeck there is also an error to build the snow tutorial in the docs (see CI) - did you see that before?

I have not seen it, but perhaps @a-charbon has? https://github.com/CliMA/ClimaLand.jl/actions/runs/9473604094/job/26101550233?pr=653#step:5:48

@kmdeck @AlexisRenchon The lack of a "Date" column comes when the data scraping portal called in the code block via the sitedata_daily() or sitedata_hourly() functions doesn't return a webpage at all or returns a data error page, such as trying to scrape too large of a file (which shouldn't happen since the size of the csv I scrape in the tutorial doesn't change over time) or the website is down. This is usually transient and an issue on the data-website side from it being down temporarily, and goes away by running again (just trying to scrape the information again).
I would assume this is the case since this code hasn't been touched in months and works on other builds? I just checked the data portal site and requested a CSV, and received what should come through.

Thanks for the update, Andy!


feat(formatting/#2464): Initial format-on-save implementation

This implements format-on-save, adding an "editor.formatOnSave" configuration setting that can be set per-filetype, and then actually implementing the format-on-save behavior.
TODO:

[x] Fix up notification message (when applying edits due to save, update to say something like "Saved with X formatting edits from ")
[x] Move the write command to Feature_Vim.Effects.save(~bufferId)
[x] Investigate adjusting cursor position in response to formatting edits (#3263)
[x] Investigate limits for large files
[x] Add test case

Fixes #2464

Works like a charm ! Thank you very much !

Awesome @ronangaillard , great news! Let me know if you see any issues 👍

Awesome @bryphe ! Thank you so much! One question though, I have updated OniVim2 on startup but it seems that I don't have this feature yet, do I have to build it from source?


build: Fix an incorrect make variable passed to goreleaser

What problem does this PR solve?:
The make variable reference was replaced recently in https://github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pull/704/files#diff-fa98b7f1d4a049ad777fb9453f6657c512c8b715a9f8bc47dfd57f1143bbb930L14.
The new reference is to GORELEASE_VERBOSE, which is unset, and goreleaser fails:
> make build-snapshot
▶ running go generate
make[1]: Entering directory '/home/dlipovetsky/nutanix/capi-runtime-extensions'
▶ configuring supported csi providers
make[1]: Leaving directory '/home/dlipovetsky/nutanix/capi-runtime-extensions'
▶ building snapshot
  ⨯ command failed                                   error=invalid argument "" for "--verbose" flag: strconv.ParseBool: parsing "": invalid syntax

Not sure how CI passed for #704, because this issue affects main.
Which issue(s) this PR fixes:
Fixes #
How Has This Been Tested?:

Special notes for your reviewer:

@dlipovetsky Thank you for fixing this! I renamed the variable to GORELEASER_VERBOSE to be inline with the flag name - hope that's ok with you.


Contributors and Maintainers welcome

Hi @cbm755
I would like to thank you for contributing to this project in your past pull requests.
Would you like to co-author and co-maintain this project?, you'd be more than welcome!
...then I would add you on github as a collaborator.
thx
Johannes

Not sure I'll always have time, but happy to help.
In general, I think many maintainers is good, as folks can be more or less active over time.

awesome, thank you! I added/invited you as collaborator in this project. (and yes, we can all be more or less active over time, I guess we are all juggling multiple projects and responsibilities).


Add Futhark

Also, maybe we should remove/somehow indicate abandoned packages or similar? E.g. nimterop isn't really maintained anymore.

Yeah, I agree.

I think marking packages as abandoned but keeping them on the list if they're still relevant would be good: so people don't try resubmitting nimterop, for example.


CI for Node

Added Node.js workflow

The following tests are failing:
Sets the current player to be whoever it is not
Contains the compiled JavaScript
Initializes with two players
Starts the game with a random player


Added the option to run the demo app locally via Docker

Please accept this ridiculously late contribution to the Swarm.
This adds the option to run the Conversations demo app locally using Docker and Compose V2.
Why is this beneficial?
It eliminates the overhead of maintaining a local Node environment necessary to run the demo app. Maintaining means running compatible versions of Node, NPM and Yarn.
This may sound trivial, but the number of times I've had to switch between different Node versions, clear Node modules, reinstall and clear the cache of Yarn is higher than I'd ever expected myself.
This small addition should help alleviate similar frustrations of others.
Risks
Starting the container does not automatically open the app in the browser, this has to be done manually.
The Webpack Dev Server's hot reload function still works as expected, as the src directory from the host machine is mounted into the container. The same can be done for other directories/files if deemed useful.

@ipletnjov-tw fix the commit message then too, to match the guidelines.


Refactor misspelled word Resouce to Resource

Am using vim, this took almost an hour to figure out why.

Thanks! Could you please run make fmt in your branch? Looks like CI is failing because of this.


EKS cleanup workflow

Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

/ci-awscni

/ci-awscni

/ci-awscni

/ci-awscni

/ci-awscni


Issue#52 minimize toolbox

Blockly does not allow manipulation of the toolbox except updating the toolbox' xml definition.
You also cannot inject your own elements inside the blockly workspace so the button to toggle min/max has to stay outside
You can see an example on "open roberta" how the toggle between different toolboxes. There the tab-buttons also stay outside and in fact that's the only other example I found where someone tries to change toolbox categories dynamically.
You also can't have more than one toolbox.

Ah, that makes sense. Would it make sense to have the "current" toolbox XML
string in the reducer (which might be the empty one) and have the component
update it when it changes?
On Mon, Jun 29, 2020, 11:54 AM Matthias<EMAIL_ADDRESS>wrote:

@eurocoinsalbum commented on this pull request.
In src/BlocklyInterface/BlocklyEditor.tsx
https://github.com/FRUK-Simulator/Simulator/pull/69#discussion_r447076417
:

@@ -130,6 +133,25 @@ export const BlocklyEditor: FunctionComponent = () => {
}
}, [highlightedBlock, currentBlockSelection, executing]);

// show/minimize the toolbox. Blockly does not allow the manipulation of the toolbox
// in any way except updating the xml definition of it.
useEffect(() => {
if (blocklyRef.current) {

 const workspace = Blockly.getMainWorkspace() as WorkspaceSvg;

 const blocklyXml = showToolbox ? getToolbox() : getEmptyToolbox();

blockly allows only ONE toolbox :-(
To have multiple ones you have to change this one to whatever it should
display right now. In fact it's just an illusion to the user that he has
multiple toolboxes (or collapsed/expanded)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/FRUK-Simulator/Simulator/pull/69#discussion_r447076417,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEZSQPUYKL4J65C7FSTI5ALRZC2J3ANCNFSM4OKOD65Q
.

Holding the xml definition in the state is not enough for the toggle-visibilty-feature. The onClick has to invert the visible flag. By knowing the current xml-definition it can't really know if the state is collapsed or expanded.
But additional to the visibility flag I added now also the xml definition to the state to support already "multiple toolboxes" and integrated that with the current implementation.


Undefined variable: self

https://github.com/hprose/hprose-php/blob/2468699a4fbfbfdaca39119e41389de713b3e4a0/src/Hprose/Client.php#L101

Thank you!


Migrate to Swift 3

I migrated the code to the stable version of Swift 3.
Also migrated the tests and they all pass (both on iOS and on OS X). 👍

Looks good 👍. I've added you as a collaborator on the repo and registered you as an owner of the cocoapod (trying the Artsy model of more free access). Feel free to merge and release :)
Thanks very much!

That sounds great, thanks! A few points that I would like to make clear before proceeding:

Why is the file AVAssetImageGeneratorTimePoints.swift not in the project? I spent a while trying to figure out why Carthage wasn't building and it turns out that file is not included in the project and so the Xcode migrator didn't convert it to the last syntax. Can I insert it into the project to keep all the files in a single place?
I was thinking to branch off the current master to keep a swift2.3 branch. Then I could merge this PR into master that would support 3.0 by default in CocoaPods and Carthage.

Let me know your thoughts about this process :)

Yeah I think so--no idea what's happened there.
Sounds perfect 👌
Thanks so much for your help!

On 19 Sep. 2016, at 4:47 pm, Patrick Balestra<EMAIL_ADDRESS>wrote:
That sounds great, thanks! A few points that I would like to make clear before proceeding:
Why is the file AVAssetImageGeneratorTimePoints.swift not in the project? I spent a while trying to figure out why Carthage wasn't building and it turns out that file is not included in the project and so the Xcode migrator didn't convert it to the last syntax. Can I insert it into the project to keep all the files in a single place?
I was thinking to branch off the current master to keep a swift2.3 branch. Then I could merge this PR into master that would support 3.0 by default in CocoaPods and Carthage.
Let me know your thoughts about this process :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Nice! I just updated Travis to use Xcode 8 and the build passed at the first attempt 🎉
I also branched off swift-2.3 for those still using it. I will soon try to release the pod. Do we release the current master and then people will be able to choose a specific branch for any Swift version?

Yep—release master as the latest and then people can use either the version
tags or the :branch specifier in their podfile.
Thanks again!
On Wed, Sep 21, 2016 at 4:42 PM, Patrick Balestra<EMAIL_ADDRESS>wrote:

Nice! I just updated Travis to use Xcode 8 and the build passed at the
first attempt 🎉
I also branched off swift-2.3
https://github.com/matthewpalmer/Regift/tree/swift-2.3 for those still
using it. I will soon try to release the pod. Do we release the current
master and then people will be able to choose a specific branch for any
Swift version?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/matthewpalmer/Regift/pull/33#issuecomment-248525156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB1CwBO3VKOM9-S3zQ4L3ODE5wlJGkMvks5qsNG5gaJpZM4J_-MJ
.


Configure Azure Pipelines

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

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: PandaPerio sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Azure OpenAI: Upgrade TestFramework to STJ 6.0.9

The Azure.AI.OpenAI library is not able to upgrade to Azure.Core 1.43.0 or SCM 1.1.0 because those bring in a transitive dependency on STJ 6.0.9, where some APIs from STJ 4.7.2 have been obsoleted, and so the library will not build.
https://github.com/Azure/azure-sdk-for-net/pull/45961 pins the Azure.Core version and opts AOAI CI's out of pipeline stages that replace the Azure.Core and SCM package references with project references.
Once SCM 1.1.0 has released, as part of the upgrade for the GA release of OAI and AOAI, we'll need to remove the version overrides and project reference exclusions added in https://github.com/Azure/azure-sdk-for-net/pull/45961, https://github.com/Azure/azure-sdk-for-net/pull/44896, and https://github.com/Azure/azure-sdk-for-net/pull/44825.
Related to https://github.com/Azure/azure-sdk-for-net/issues/44899

Once this item is complete, we can merge https://github.com/Azure/azure-sdk-for-net/pull/46011 as well.


Error refreshing token: invalid_grant

Hi
Description of issue :
I created an addon and using oauth2 lib.
At first connection user grant permission for addon
Addon display link to grant oauth2 access.
User validate access, and re open addon.
Access is granted and user can use addon.
User go in his account and revoke access for the add-on.
User open again addon, permission pop up for addon appears
User validate permission.
After validation user get issue : Error refreshing token: invalid_grant
Now each time user try to open addon he get this error.
Details of error : Error refreshing token: invalid_grant (line 379, file "Service", project "OAuth2")
Way to solve the issue : Have access to the script and run PropertiesService.getUserProperties().deleteAllProperties()
After deleting all properties issue above is not displayed.
In the function which open the sidebar for addon I try to catch this error but does not work. I do
try{
var service = getService();
}catch(e){
.....
}
I use version 13 of the lib.
Scope of oauth 2 : http://www.google.com/m8/feeds/
Code of lib is too complex for me so I can't give be more helpful.
Stéphane

I'm sorry but does not work properly. I no longer get the error message but nothing happens. I tested and from hasaccess I received a true.
As a quick fix I will test token by myself.
try{
var fetch = UrlFetchApp.fetch("https://www.googleapis.com/oauth2/v1/tokeninfo?access_token="+TOKEN);
}catch(e){
PropertiesService.getUserProperties().deleteAllProperties();
}
May that can help you ?
After this try/catch I use the hasaccess.
Stéphane

Can you provide a more detailed set of steps and sample code that I can use to replicate the problem?

Basically it is what I do and don't get a false.
var service = getSharedService();
if (!service.hasAccess()) {
Display screen with authorization url.
} else {
Display app screen
}
hasAccess() send me a true so I don't display the screen with authorization url.

My fix only makes it so that errors during token refresh return false. If the access token isn't expired but was revoked, there is no way to determine that without making a request (either to the API itself of the token validation endpoint). My guess is that your request to the API is returning an error response (like 403 or 401) and you should catch that error and re-show the authorization prompt. That is not something the library can handle for you, as the process varies from API to API.

OK so I will keep this check before to display user interface
Thank you for support.
Stéphane


Add a button of "Stop Code run" on the OUTPUT panel

Just like this, sometimes I want to stop some running code, but I had to right click and press "Stop code run" on the right menu (may be hard for some users to find this). But why not just add some button on the top panel, so that I can directly click it. (Just like "Clear Output")

This is possible with the new task system (version 2.0.0) that executes in the terminal. A task can be stop by pressing the garbage bin button there.
We don't have any plans to back port this to the old task runner executing in the Output panel.


fixed a wrong calculation about volumeinfo

rebase by development branch

What this PR does / why we need it:
The value of 'CapacityRangeBytes' of volumeinfo is in bytes , but the 'v.size' is in Gigabytes.
my mistake, last pr is base on master branch , leading to unuseful commit records.Sorry for that.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

LGTM~

Confirmed volume size needs to be converted to bytes for CapacityBytes:
https://github.com/opensds/opensds/blob/8e4865fa4377e95957b81ea953c3b268255e8b95/pkg/model/volume.go#L49


Adding everyday mood

Adding everyday mood would help in self realization that what's going inside. Everyone is busy and we hardly get time to analyze ourselves.

Hello @riyajaiswal25 , thanks for the issue. We are currently working on the ideation of our pre-existing self-care feature, where we are thinking to use this feature.


My flutter app crash when I'm trying to run it, It was working fine before that

ProcessException: ProcessException: Process "C:\Projects\flutterExcersice\NestedTabs-master\flutternestedtabsapp\android\gradlew.bat" exited abnormally:
Exception in thread "main" java.lang.NullPointerException
at org.gradle.wrapper.BootstrapMainStarter.findLauncherJar(BootstrapMainStarter.java:34)
at org.gradle.wrapper.BootstrapMainStarter.start(BootstrapMainStarter.java:25)
at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:129)
at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)

Hi, @leoula This has nothing to do with Flutter, it's caused by gradle. Search google to see if any approaches might help you solve this problem.
Let me know

I couldn't get anything on google. It was working fine last week. I opened the project to edit some staff. But couldn't run at the first place.

Try running flutter clean

I just did, nothing is changed.

Is it happening only with the current project or also with fresh new project?

It's working fine with other projects. Just this project.

@leoula have you changed something in the android folder?

I'm having the same error. Just started after updated new version of Flutter, I guess. Had no problems what so ever before weekend.

I revise my comment. I was nothing wrong with flutter after update.

Hi @leoula
can you please provide your flutter doctor -v ,
your flutter run --verbose
and your pubspec.yaml
or if possible a reproducible minimal code sample.
Please refrain from posting pictures if not necessary.
Thank you

Hi @leoula
Without additional information, we are unfortunately not sure how to resolve this issue. We are
therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the
bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Working Dockerfile and slightly adjusted Procfile.

This Dockerfile is the simplest way to get slackin running on Docker.
It's running here: https://slackin.octohost.io/
You can try the Docker build with:
docker pull octohost/slackin
docker run -d -P -e SLACK_CHANNEL=channel -e SLACK_SUBDOMAIN=subdomain -e SLACK_API_TOKEN=api-token-goes-here octohost/slackin:0.4.0

It will be listening on port 3000 - Docker will likely assign a random 49xxx port on the outside.

miraculous


Convert Storage::entries's max_size argument to Option<u64>

Currently the value is just a u64 with u64::MAX defined as NO_LIMIT.
This no limit is not mentioned in the documentation. Something like the following enum would suite better:
enum Size {
    Fixed(u64),
    NoLimit,
};

I could provide a PR, if wanted.
Edit (brson): per discussion, let's change this to Option<u64> with a doc-comment explaining that None means "no limit". The Storage trait is in src/storage.rs.

Maybe adding a comment is better.
/cc @Hoverbear

@bkchr Would adding a comment about this limit be satisfactory? Since NoLimit == Fixed(u64::MAX)

Perhaps Option<u64> is appropriate. A None value correctly communicates that The max size is none.

@bkchr Would you still be interested in making a PR, or is this open for others to tackle?

Open for others :)

Id like to work on this!

dang, too slow :/

Sure thing @pwoolcoc, you're too quick on the draw :)

@estk Aw, thanks so much! Do you think you might be interested in this one instead? https://github.com/pingcap/raft-rs/issues/70

@Hoverbear haha, you really gotta be quick here! Thanks for pointing that one out, but it looks like I'm too late again. I'll be away for the weekend but I'll swing back next week and try to pick up an issue. Thanks for the hospitality @Hoverbear

🤦♀ Ha, I'm sorry. :)


feature request: dashboard config option to set dark mode by default

Please add an option to the dashboard config node in the Node-RED editor to specify dark / light mode as the default. Not only would this be a desirable enhancement in itself, it would also provide a work around for the issue preventing ?theme=dark query string parameters from having an effect when using the Node-RED web server to host FlexDash; i.e. the original URL is rewritten by the Node-RED web server such that the query string is eliminated, apparently before FexDash acts on it.

It's on the list... BTW, as long as you have a trailing slash on the URL there is no redirect and thus the query string is not eliminated. E.g. http://example.com/flexdash/?theme=dark, note the / before the ?.
Please use the Node-RED forum for features/discussions/problems/...


New curse version

Will the version listed in #1 be published? this is important as our build is published there and will use your mod

Hey @Xikaro , sorry it seems are time zone is almost opposite of each other.
As of sending this message, It has been uploaded to Curse Forge and awaiting there ususal approval.

Thanks, I'm looking forward to combining other AE2 Wireless Terminal mods))


Add whisperx package

Adding whisperx, faster-whisper and ctraslate2 packages.

whisperx package

which depends on faster-whisper package

which depends on  ctraslate2 package

@dusty-nv I've check it and works well. Only one thing is that it has been carefully with ctranslate2 build python packages with the directory to build it.
Or fixed with:
include_dirs = [pybind11.get_include(), '/usr/local/include/ctranslate2']
library_dirs = ['/usr/local/lib']

Or fixed with change:
CMAKE_INSTALL_PREFIX
from https://opennmt.net/CTranslate2/installation.html#install-from-sources

Hi @johnnynunez
Thank you for checking.
Will you explain to me what you are proposing regarding the CTranslate2 build?
I believe it found cmake fine, and it also built the Python wrapper.

Hi @johnnynunez Thank you for checking. Will you explain to me what you are proposing regarding the CTranslate2 build? I believe it found cmake fine, and it also built the Python wrapper.

Yes because you use containers. If it's compile outside using for example miniconda, there is an error, from Ctranslate2 repository.

ok, thank you.


[FEATURE] Reactive (reactor) support

Is your feature request related to a problem?
I would like to be able to use client in a reactive way
What solution would you like?
Provide a module that has a reactive version of the client. Preferably using reactor/WebFlux

Do you want to propose how this will look? Also, have you looked at the async capabilities?

This sounds like a positive direction. Are you up for starting work on a PR? Does anyone else have thoughts?


⚠ ClamAV has degraded performance

In d236e73, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 210 ms

Resolved: ClamAV performance has improved in ccd4e4d.


Invalid cast when trying access _reddit.User.Inbox

To duplicate:
Given the following scenario with regards to the user's inbox:

Add user as a moderator of a sub and do not read the notification.
Reply to a post with the user.
Reply to the above comment with a different user.
As a different user, toss out a username mention.

And the following code:
_reddit.InitOrUpdateUser();
var x = _reddit.User.Inbox.ToList();

The following error is produced:
System.InvalidCastException: 'Unable to cast object of type 'RedditSharp.Things.Comment' to type 'RedditSharp.Things.PrivateMessage'.'

at RedditSharp.Listing`1.ListingEnumerator`1.get_Current() in D:\temp\b263835\RedditSharp\Listing.cs:line 177
at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)

I suspect the issue is line 128 of AuthenticatedUser.cs and should look something like the following:
public Listing<RedditSharp.Things.Thing> Inbox
Though admittedly I haven't had the opportunity to dig through everything yet.

What version are you encountering this in?

The current code tries to return a listing of private messages for the inbox. The Reddit API returns both comments and private messages for the inbox. This causes the exception when it tries to cast a comment to a private message. I was experiencing the issue in version 1.1.12.
I added a pull request to fix the issue. I just made the inbox return a listing of things instead of a listing of only private messages.


chore: fix typo in intended

inteded -> intended

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@all-contributors add eltociear for documentation

Thank you @eltociear - every detail matters 😋


[BUG] NCFModel throws an error when used with mm.RegressionTask

Bug description
An assert gets triggered which doesn't seem to apply to a regression task and stops the training.

Steps/Code to reproduce bug
import os
import numpy as np

from nvtabular.loader.tf_utils import configure_tensorflow

configure_tensorflow()

import nvtabular as nvt
from nvtabular.ops import *
from merlin.models.utils.example_utils import workflow_fit_transform, save_results

from merlin.schema.tags import Tags

import merlin.models.tf as mm
from merlin.io.dataset import Dataset

import tensorflow as tf

from merlin.datasets.entertainment import get_movielens

input_path = os.environ.get("INPUT_DATA_DIR", os.path.expanduser("~/merlin-framework/movielens/"))
get_movielens(variant="ml-1m", path=input_path);
train, valid = get_movielens(variant="ml-1m", path=input_path);

model = mm.benchmark.NCFModel(
    train.schema.select_by_name(['userId', 'movieId', 'rating']),
    embedding_dim=64,
    mlp_block=mm.MLPBlock([128, 64]),
    prediction_tasks=mm.RegressionTask('rating'),
)

batch_size = 16 * 1024
LR = 0.03

opt = tf.keras.optimizers.Adagrad(learning_rate=LR)
model.compile(optimizer=opt, run_eagerly=False, metrics=[tf.keras.metrics.AUC()])
model.fit(train, validation_data=valid, batch_size=batch_size)

Expected behavior
Training runs successfully. You can train a regression model with NCF.
Environment details
current main on merlin-models

@radekosmulski do you mind to give more context for what's the motivation to use AUC() metric with a regression task? Would RMSE or MSE be not more proper metric for this task?

@rnyak I missed that AUC was being used there, I just copied the code 🙂 But I also suspect this issue is not related to the metric being used, there is an error earlier in the model stack, AFAICT. There was a discussion on slack where I think I understand the situation a bit better now -- that these tasks are not meant to be interchangeable. If we want to move from classification to regression then we might need to modify the base architecture a little bit.
Will close this issue as I don't believe it as a bug anymore.


[lambda][output] adding incident_key for PD output to support de-duping

to @airbnb/streamalert-maintainers
size: small
resolves: #58
changes
Super tiny change that adds an incident_key for pagerduty output to support de-duplication of incidents on the PD side. This incident_key will contain the triggering rule_name.

@ryandeivert - can you include testing steps going forward in your PR description?

@ryandeivert - what happens if there is a rule name that is the same for diff clusters/rules files?

@mimeframe great point and question. Working on a way to determine that now.

On Mar 17, 2017, at 11:02 AM, mimeframe<EMAIL_ADDRESS>wrote:
@ryandeivert - what happens if there is a rule name that is the same for diff clusters/rules files?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@jacknagz merged in master to resolve conflicts. PTAL and approve.


Supporting ShardKey = xyz in parenthesis : vitess issue 2109

This Pull Request tracks the issue https://github.com/youtube/vitess/issues/2109
@sougou : PTAL

LGTM


Application uses hardcoded path to /home

This assumption breaks our deployment and we cannot test the application.
[15:02][root@cbox-ocisdev-03 (ocis_dev:box/ocis) ocis-wopiserver]# grep -ri '/home' .
./ui/app.js:      axios.get(tokenUrl, { params: { filePath: '/home' + filePath, fileId: fileId } })

The request should be send to: /eos/user/g/gonzalhu/... but is sent to /home/eos/user/g/gonzalhu/...

This applies to any application/extension.
The application needs to use the configured webdav namespace.
The UI knows the good namespace as when clicking on "All files" the request is correct.
This boils down to harcoded paths in the app.
@kulmann could you point @wkloucek to the magic variable that encapsulate the files namespace so is used instead of /home?

I didn't find any place where Web needs the namespace setting since it only uses the WebDav API - which is aware of the namespace. Web only offers to configure the homeFolder, which is the Path where the user is brought to when he clicks the home icon. But this doesn't need to be qual to the namespace.
Therefore I switched to File IDs wich is easier -> see #https://github.com/owncloud/ocis-wopiserver/pull/10

@wkloucek let me take a look at that PR


Webfinger

Is there any intent to support Webfinger with this library, since Mastodon uses it? Or is the library strictly ActivityPub-as-advertised?

Will do all that.
If we're delegating to the caller to canonize usernames, then it might also make sense to have the caller responsible for validating / checking for the existence of usernames. In which case, the json builder could be a static function (maybe in the Utils namespace). If it's static that wouldn't work with the callable being set to an ActivityPubConfig field but food for thought?
It's difficult for me to anticipate any more WebFinger functionality being necessary unless at some point in the future this library takes charge of canonizing usernames.

I think it would make sense for the JSON builder to at least check for the existence of the canonical actor in the database (as it does in the current iteration of your code), which would mean it couldn't be static. I do agree that we probably won't need much more WebFinger functionality beyond this though.


Integration Test - Release - 2020/12/14 19:42:33

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'


toml::to_string serializes -0.0 to --0.0 which toml::from_str can't parse

#[derive(serde::Deserialize, serde::Serialize)]
struct Foo {
    a: f64,
}

fn main() {
    let foo = Foo { a: -0.0 };
    let s = toml::to_string(&foo).unwrap();
    println!("{}", s);
    toml::from_str::<Foo>(&s).unwrap();
}

a = --0.0

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error { inner: ErrorInner { kind: Wanted { expected: "newline", found: "a period" }, line: Some(0), col: 7, at: Some(7), message: "", key: [] } }', src/main.rs:10:31

Thanks for the heads up, I didn't realize this hadn't been published yet.  I just published 0.5.9 which should include the fix for this from #426.


stacked borrows: ensure array-to-elem casts behave correctly

When "as"-casting an entire array, that should create a raw ptr usable for all elements in the array, even if we immediately cast to the element type.

@bors r+

:pushpin: Commit a2b227f95aca2897c983df327932d0fbdf3c13c1 has been approved by RalfJung

:hourglass: Testing commit a2b227f95aca2897c983df327932d0fbdf3c13c1 with merge ee8dbbc40f8d1fe2805fd62b5cf32b9d3e0b62ac...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing ee8dbbc40f8d1fe2805fd62b5cf32b9d3e0b62ac to master...


Fix filter_pre_get_posts overrides existing value in meta_query

It is also possible that a query does not have the index meta_query so this code can return a notice about undefined index.


Update snforge template before precalc_addr release

Has to be done on the release day. Consider making manual template releases

Can you elaborate on that @piotmag769?

@cptartur Basically we need to update the template since we did a breaking change in how the standard deploy flow has to be written (contract.deploy(@calldata)), but it has to be done just before the release to not confuse users


SpiritParser not properly implemented yet

The SpiritCalculation always terminates with exit code 100 (ERROR_MISSING_OUTPUT_FILES) because the SpiritParser is not properly implemented yet.

The comparison to the expected list of files in https://github.com/JuDFTteam/aiida-spirit/blob/a928f18646f5c0149ed606285ff79464f3de0ac1/aiida_spirit/parsers.py#L45 does not work properly
no parsing is done but the template code from the cookiecutter is still present

We should define which information should be parsed from the output files of spirit.

Ideas for things that can be parses / should appear in the database:

[ ] initial spin configuration
[ ] final spin configuration
statistics of the spirit run

[ ] total number of iterations
[ ] convergence information

I think the relevant outputs depend strongly on the method that is being run. So far I agree with Philips list and I think these should be the first things to be implemented. For most methods, the energy of the initial/final/ground state would also be interesting.

  * [ ]  total number of iterations
  * [ ]  convergence information

Just as a heads up: It is currently not so straight forward to get these from the spirit python API. This is because the functions that query the simulation state (like e.g. spirit.simulation.get_max_torque_norm) need to be called from a separate thread, while the simulation is running. I will implement a better method on a branch soon.
Other useful outputs might be:
For LLG (Dynamics) (meaning LLG with any solver that is not VP/LBFGS   and  direct_minimization=false)

[ ] Snapshots of the spin configuration at different time intervals
[ ] Energies at different time intervals

For LLG (Minimization) (So we probably expect some metastable configurations)

[ ] The profile/radius (thinking of Skyrmions)
[ ] Topological charge

For GNEB

[ ] The saddle point configuration/energies
[ ] Energy barrier

For HTST

[ ] All of the quantities in htst_info
[ ] Eigenvalues/Eigenvectors (if non sparse calculation)

MC

[ ] Critical temperatures (requires post processing, so maybe a bit out of scope)
[ ] Ground state spin configuration (simulated annealing + final minimization with LLG) (How to control convergence?)

Just as a heads up: It is currently not so straight forward to get these from the spirit python API. This is because the functions that query the simulation state (like e.g. spirit.simulation.get_max_torque_norm) need to be called from a separate thread, while the simulation is running. I will implement a better method on a branch soon.

As long as we can write it to a file or the stdout we are good. The parsing will be done after the spirit calculation has finished and there we extract data from the output files. Therefore we can also look for keywords in the stdout that spirit produces and take the information (e.g. number of iterations) from there.

Since the basic parser works I move this to the new issue #8


Add the CHANGELOG.md to falcosidekick

What to document
Currently, the falcosidekick chart has no changelog. It would great if we introduced it.

Ok, the changelog is only for what concern the helm chart?

Ok, the changelog is only for what concern the helm chart?

Yes, exactly.
BTW: Unfortunately, it's a manual process - I know. Anyway, it's not needed to be too precise, but at least it will help folks to understand what has changed. Also, I'd like to automatize that in the future, any idea about that is welcome!

Ok, will try to do so asap


Added the feature of auto commenting when a PR is merged

Description
I have added auto-comment-on-pr-merge.yml in .github/workflows directory. This file will add the feature of auto commenting when a pr is merged.
Related Issues
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]

Closes #276

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist

[x] I have gone through the contributing guide
[x] I have updated my branch and synced it with project main branch before making this PR
[x] I have performed a self-review of my code
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Thankyou for your contribution!!

Please add the label of gssoc and appropriate level as per gssoc guidelines on this pull request as well.


AWS Parameter Store JSON number results in "unable to unmarshal secret" when using dataFrom

Issue:
Same problem as in issue 510, but with parameter store. So when JSON parameter has numerical value without double quotes, it will end up in ESO error.
Example:
parameter store secret:
{
  "DB_HOST": "myrds.eu-north-1.rds.amazonaws.com",
  "DB_PASSWORD": "verysecret",
  "DB_PORT": 5432,
  "DB_USERNAME": "seppo_hovi"
}

dataFrom is being used:
  dataFrom:
  - extract:
      key: /path/to/some/secret

Ends up in error:
external-secrets-operator-79b8c57f87-m6s8b external-secrets {"level":"error","ts":1660907739.6617749,"logger":"controllers.ExternalSecret","msg":"could not get secret data from provider","ExternalSecret":"namespace/secret","SecretStore":"/cluster-param-store","error":"unable to unmarshal secret secret: json: cannot unmarshal number into Go value of type string  --omitted--
Observations:
On a quick glance it looks like parameterstore.go is handling this similarly as the secretsmanager.go pre Fix 545

Ahh sorry, forgot to mention that! We are using 0.5.9
    Image:         ghcr.io/external-secrets/external-secrets:v0.5.9
    Image ID:      ghcr.io/external-secrets/external-secrets@sha256:8aacccd4efd705565ad627bd2c300b70e2834b181a9b5b1952eb123419018e2d

Getting the same error in the same situation when using AWS Secrets Manager


Helm version 0.3.2

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #207 (17a347b) into main (f2a8f35) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #207   +/-   ##
=======================================
  Coverage   53.15%   53.15%
=======================================
  Files           8        8
  Lines         730      730
=======================================
  Hits          388      388
  Misses        332      332
  Partials       10       10

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


forcing exact dictionary lookup

I have trained with the dictionary, but since it learns to generalize it gets some words that are in the training data wrong for example "the", "to", letters of the alphabet. Is there a way to force if the word is from the training to match it completely? and only generalize for new unseen wodds

Yes it makes errors like these.  You will get a better general result [and quicker] if you use it in combination with your reference lexicon.
There is a small example g2p servlet that shows a simple integration for this:

https://github.com/AdolfVonKleist/Phonetisaurus/tree/openfst173/python/script

there are also some better alternatives to this out there these days.  I have a transformer-based solution that will be released here before end of year.  I'll continue to fix bugs here, but 'the future is coming' and this is getting pretty dated.

This is great for performance, accuracy seems good enough, and the question is how much more ram, cpu, gpu you need to just increase accuracy a little bit by using neural method

@AdolfVonKleist: For the transformer-based solution you mentioned, did you mean end of year 2019 or 2020? Or academic year? Where would I subscribe to make sure I don't miss this upcoming release?


QuickSilver hangs all the time with Sierra

Hi,
Ever since I upgraded to Sierra, QuickSilver takes a long time to load (pressing cmd+space)
If I hover over the QuickSilver icon on menu bar during this time, the spinning disc is observed.
I checked that I am using the latest version of QuickSilver already.
Is this a common issue? have you any suggestion? should I de-install + reinstall?
This is my machine spec
MacOS Sierra 10.12.4
MacBoo Pro Ret 15in
2.3Ghz i7
8GB DDR3
Intel Iris Pro 1536mb

It’s been a common complaint since the Safari plug-in was fixed. You might want to disable the Safari History preset for now.
It can be found under Preferences → Catalog → Plugins → Safari
See #2331

Perfect!  Problem solved.
Thank you so much for such prompt response, I was lost without Quicksilver.
And thank you for such wonderful product!
Have a good day.
Philomena.

Thank you! I have had the same issue and this solved it for me as well.


ng new: "Please choose a version of [angular tool] from this list"

Versions
Angular CLI: 6.0.1
Node: 8.11.1
OS: win32 x64
Angular:

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.6.1
@angular-devkit/core         0.6.1
@angular-devkit/schematics   0.6.1
@schematics/angular          0.6.1
@schematics/update           0.6.1
rxjs                         6.1.0
typescript                   2.7.2

Repro steps

ng new mynewfoldername

That's it!
Observed behaviour
After doing some initial installs, the CLI will prompt me for which version of some tool do I want to use, like so:
Please choose a version of "@ngtools/webpack" from this list: (Use arrow keys)
> 6.0.1
  6.0.0
  6.0.0-rc.11
  6.0.0-rc.10
  6.0.0-rc.9
  6.0.0-rc.8
  6.0.0-rc.7
  6.0.0-rc.6
  6.0.0-rc.5
  6.0.0-rc.4
  6.0.0-rc.3
  6.0.0-rc.2.4
  6.0.0-rc.2.3
  6.0.0-rc.2.2
  6.0.0-rc.2
  6.0.0-rc.1
  6.0.0-rc.0
  6.0.0-beta.9
  6.0.0-beta.8
  6.0.0-beta.7
  6.0.0-beta.6
  6.0.0-beta.5
  6.0.0-beta.4
  6.0.0-beta.3
  6.0.0-beta.2
  1.10.2
  1.10.1
  1.10.0
  1.10.0-rc.0
  1.10.0-beta.3
  1.10.0-beta.2
(Move up and down to reveal more choices)

Sometimes it's @ngtools/webpack that it asks about, as above.  Other times it's @angular-devkit/core.  Another time, I was asked this question about a dozen times in a row, each one for a different constituent Angular component.
Desired behaviour
Should it not just install the latest stable versions of each component for a new project?  It was doing this for a while, and I wasn't seeing any of these prompts.  But then they came back.
Mention any other details that might be useful (optional)

That's pecurial, never seen this reported before. It seems related to Yarn. Do you have any custom yarn configuration, or some kind of proxy?

Yes, I am behind a proxy at work.  In fact, we use an internal Nexus mirror server for our NPM packages, so this may well be an issue with that.
I will investigate further...

It's working for me now, after  I made a config change to my .yarnrc file.
So here's the entries for our Nexus mirror server that were in my .npmrc file:
registry=http://acme.com/content/groups/public-npmjs/
@angular:registry=http://acme.com/content/groups/public-npmjs/
@types:registry=http://acme.com/content/groups/public-npmjs/

When I checked my .yarnrc file, however, I saw that I only had one of the three equivalent entries there:
registry "http://acme.com/content/groups/public-npmjs/"
So, I updated my .yarnrc to add the missing entries for Angular, like so:
registry "http://acme.com/content/groups/public-npmjs/"
"@angular:registry" "http://acme.com/content/groups/public-npmjs/"
"@types:registry" "http://acme.com/content/groups/public-npmjs/"

That appears to have fixed things for now.  But as I said in the original report, the problem was intermittent anyway.  I'll close this one for now.


Hide "Staff" flags in UI and API plus documentation of configuration permissions

This PR does 3 things:

Hiding the Staff flag when viewing or editing of users, if FEATURE_CONFIGURATION_AUTHORIZATION is set to True.
Documentation of configuration permissions
Removing a few is_staff flags that have been overseen before (in dojo/templates/base.html and  dojo/templates/dojo/rules.html)

This is the final PR for configuration permissions and is_staff removal for the release 2.7.0. It is feature complete, but I maybe its better to leave the default of FEATURE_CONFIGURATION_AUTHORIZATION set to False for now. Then nothing changes for the users for now, unless they change the flag it. Or shall we get live with it right away?

Might be best to leave it off by defaul tin 2.7.0 to allow ourselves to test it in real life? Also, should we add some docs about staff users and superusers and what they mean and don't mean in the new setup?

It is off by default at the moment exactly for that reason 😃 I will turn it on in my productive installation to see how it works.
I would like to change the documentation anyway, when the configurable permissions will be the default. Then the staff flag won't have no meaning anymore and won't be visible in the UI or the API. At that point I will add some integration tests as well.

I know not everyone agrees, but I think we should allow people to access the /admin interface, which needs the staff flag?

I know not everyone agrees, but I think we should allow people to access the /admin interface, which needs the staff flag?

I agree with you for superusers. For them it might make sense to access the /admin interface. Not so much for non-superusers, for them the admin interface is empty anyway.
I would change this PR to make sure a superuser is always staff as well. Would that work for you @valentijnscholten?

yeah, was thinking about that as well.

With the latest commit superusers will always be staff as well.

Should it be added to initialize_permissions or some migration that does a one time updates of superusers? Will it also work if someone creates a superuser via manage.py?

Should it be added to initialize_permissions or some migration that does a one time updates of superusers?

That would only be needed if an instance already has superusers which are not staff. They would have a lot problems in the current version because of all the is_staff flags. With the configuration permissions this problems will be gone anyway. If there is a case, you can do a save of that superusers and he will be made staff by the signal. The only difference will be if a superuser can access the admin interface. I don't see a migration necessary for this case.

Will it also work if someone creates a superuser via manage.py?

Yes it does, I have tested it. The signals work there as well.


"&" in team names brings some trouble

Description
I created a team with the name: first & second.
Then I would create a namespace. In the dropdown to select a team for the namespace there stood this content: first & second
If I select this team name and click create, I get the error message: Not found.
Portus version: 2.4.3@5a616c0ef860567df5700708256f42505cdb9952

This issue still exists.

This issue still exists.

bump

unstale

unstale

unstale


Ignore posts with publication date in future

Hello,
we just added this package to our apostrophe installation. Almost everything worked out of the box, it skipped unpublished blogPosts, but published blogPosts with publicationDate and publicationTime in the future are shown in the sitemap.
Do we use this wrong or is there a bug?
Thanks again :)

Well, i think this is not relevant anymore :).

Thanks for the follow up, @matthaias. 😄


BMS Client fails to start

Hi @crowbarmaster, its been a while, but I finally had some time to revisit this software and try it out again.
Unfortunately, the client won't even start on my Windows server.  Looking at event viewer, I see these errors:
Faulting application name: BedrockService.Client.exe, version: 2.6.8093.31951, time stamp: 0x61e1d8d9
Faulting module name: KERNELBASE.dll, version: 10.0.17763.2452, time stamp: 0xad1c2e55
Exception code: 0xe0434352
Fault offset: 0x0000000000039319
Faulting process id: 0x58c
Faulting application start time: 0x01d832879e26a029
Faulting application path: C:\Minecraft\BDServer\BedrockService.Client.exe
Faulting module path: C:\Windows\System32\KERNELBASE.dll
Report Id: 9e37a0f5-d59a-4fd1-9d90-9d29f6ff0ff5
Faulting package full name:
Faulting package-relative application ID:
Application: BedrockService.Client.exe
CoreCLR Version: <IP_ADDRESS>210
.NET Version: 6.0.0
Description: The process was terminated due to an unhandled exception.
Exception Info: System.IO.FileLoadException: Could not load file or assembly 'System.Windows.Forms, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'. The located assembly's manifest definition does not match the assembly reference. (0x80131040)
File name: 'System.Windows.Forms, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'
I tried both of these versions, with the same error:
BedrockService.Client.exe
2.6.8093.31951
BedrockService.Client.exe
2.6.8089.1944
I thought it might be something to do with .NET, but I did a Windows update and after the update for .NET, it didn't resolve the error.
Trying to start the 2.6.8093.31951 service, and I get this error on first run:

It looks like some of the files were moved around to their new locations, but something died in the migration.  I attempted to start the service again, and it finished creating some of the files, and started the service.  Manually looking at the service log, and it did upgrade the Minecraft server version, even though I set the SelectedServerVersion=<IP_ADDRESS> in both the Service.conf file and the BmsConfig\ServerConfigs\world.conf.  But perhaps I need to really make those changes via the client.
Anyway, I wonder if anyone else is having problems. Perhaps it's just a Windows 2019 server problem and works ok on Windows 10/11?

I did try running the client in a new folder to simulate a clean install, same error.

Hey man! Hope things are well!
I started by downloading a copy locally to my laptop right from GH. It seems to launch as expected.... I will fire up my Server 2019 VM in a bit and give it a go! As for the service, Sounds like perhaps it was something that happened launched as a service. I had done testing in this manner, but perhaps I missed testing something near the end. I should have time Thursday to pinpoint that issue!

Update: After a small fight getting VBox to work on my main machine, I gave up and installed a fresh copy of the latest vbox, followed by a fresh copy of server 2019, .NET Desktop and hosting runtimes. I grabbed the latest copy of the client, and it fired right up. I am at a loss here, unfortunately. NET Core and WinForms is getting worse to work with.

Let me try looking at the .NET versions I have installed tomorrow and if they need to be updated.  I can live with a “just me” problem and try to figure it out.


Bookmark Cards Can Cause Security Warning

Issue Summary
Bookmark cards aren't defaulting to HTTPS for the img src tag. This may cause browsers to give a security warning like: "Parts of this page are not secure (such as images)."
To Reproduce

Create a new post and insert a bookmark card for a Squarespace hosted website

The site that produced this issue for me is https://www.theblinkdate.com/

The published post will use http:// for the bookmark card img src.

The image src tag should be relative and use // instead of hard coding http or https. Squarespace's CDN supports https and redirects all http request to https.
Technical details:

Ghost Version: 3.38.3
Node Version: 12.18.0
Browser/OS: Firefox 84.0.2 on macOS Catalina
Database: MySQL

This is a duplicate of https://github.com/TryGhost/Ghost/issues/11870 - TLDR the bug lies with Squarespace, not Ghost.

@ErisDS, I'm not too familiar with how the Ghost cards are created; so I apologize if my question below is obvious.
When I look at the problematic post  in the posts table in the MySQL database, I see that it the post html was inserted with the img src being "http://static1.squarespace.com/static/..."
I modified the html column for that post so the img src was "//static1.squarespace.com/static/..." and it fixed the security warning.
Would a possible solution be to search post content for any "http://" references  in src tags and replace it with protocol relative URLs before inserting into the database?

Please refer to & use the original issue - duplicates create huge overhead for us as maintainers.


Add to the README.md how to customize site for your city

I started work on customizability on a branch in my fork.
Here's the first commit: https://github.com/specious/adopt-a-drain/commit/cbbf0cd6a5f513ea60d1474a35ec8e11e814317e


Use default behavior to open external link

Use default behavior to open external link
Why not window.open? Because some browsers block it
If we know it is just a simple normal external link, why not just open it using the default behavior?

Merged, thanks

Do you have plan to release a new version of V2?


are recurring_tasks ripe for use?

ran bundle add mission_control-jobs and then
#/config/routes.rb
Rails.application.routes.draw do
  mount MissionControl::Jobs::Engine, at: "/solid_queue_jobs"

...
end

Going through the 'tabs' revealed nothing out of the ordinary - except for the last tab:
https://localhost:3000/solid_queue_jobs/applications/mortimer/recurring_tasks?server_id=solid_queue

This tab returns
NameError in MissionControl::Jobs::RecurringTasksController#index
uninitialized constant SolidQueue::RecurringTask

 def recurring_tasks
    tasks = SolidQueue::RecurringTask.all
    last_enqueued_at_times = recurring_task_last_enqueued_at(tasks.map(&:key))

    tasks.collect do |task|

Where did I take a wrong turn?

Hey @wdiechmann! Can you update Solid Queue?

Certainly - but I'm on 0.3.1?!?

Could you update it to 0.5.0? It was released last week. I added this in the requirements section but it's easy to miss.

I'm aware this may muddle the issue - but my reason for installing mission_control-jobs in the first place is that I'm struggling with
default: &default
  dispatchers:
    - polling_interval: 1
      batch_size: 500
      recurring_tasks:
        background_job_scheduler:
          interval: 60
          class: BackgroundManagerJob
          queue: default
          schedule: every minute
          enabled: true
  workers:
    - queues: "*"
      threads: 5
      processes: 1
      polling_interval: 0.1

my attempts to configure a background_manager_job looking like this
class BackgroundManagerJob < ApplicationJob
  queue_as :default

  #
  # step 1 background jobs will be running 3 jobs
  #
  #   1: employee_state_job
  #   2: employee_eu_state_job
  #   3: employee_auto_punch_job
  #
  def perform
    Rails.logger.info "Performing BackgroundManagerJob"
      dt = DateTime.current
      prepare_state_job if dt.hour == 5 and dt.min == 0
      prepare_eu_state_job if dt.hour == 5 and dt.min == 5
      prepare_auto_punch_job if dt.hour == 23 and dt.min == 50
  rescue => error
      UserMailer.error_report(error.to_s, "BackgroundManagerJob.perform").deliver_later
  end

but it keeps logging this in my development.log
19:33:00 web.1  | [ActiveJob] Enqueued BackgroundManagerJob (Job ID: cf530c52-8758-49c1-9b7c-a94b63f4c018) to SolidQueue(default)
19:33:00 web.1  | SolidQueue-0.3.3 Skipped recurring task – already dispatched (4.6ms)  task: :background_job_scheduler, at: 2024-08-20 17:33:00 UTC
19:33:00 web.1  | SolidQueue-0.3.3 Error in thread (0.0ms)  error: "ActiveRecord::RecordInvalid Posten findes ikke"

My googling has so far let me utterly down - so I reasoned that the mcj might shed some light on the issue

Could you update it to 0.5.0? It was released last week. I added this in the requirements section but it's easy to miss.

dang - obviously I did just that! Miss out on the requirement - and then again - I fail to find the tag in the list when selecting between branches and tags (button above the initial file list - and the README)

now somehow all of a sudden the tags list displays a few tags between 0.5.0 and 0.3.1 - my bad by the looks of it!
thx for the ultra-fast reply
❤

Don't worry, that's my fault 😅  It should fail loudly when it doesn't have the version of the adapter that it needs, but I was in a hurry 😅 I'll make sure it fails loudly for v1.0.

ehhh - how do I circumvent this?
?7 bellis % bundle update mission_control-jobs
Fetching https://github.com/rails/rails.git
Fetching gem metadata from https://rubygems.org/........
Could not find gem 'mission_control-jobs (~> 0.5.0)' in rubygems repository https://rubygems.org/, cached gems or installed locally.

The source contains the following gems matching 'mission_control-jobs':
  * mission_control-jobs-0.1.0
  * mission_control-jobs-0.1.1
  * mission_control-jobs-0.2.0
  * mission_control-jobs-0.2.1
  * mission_control-jobs-0.2.2
  * mission_control-jobs-0.3.0
  * mission_control-jobs-0.3.1
?7 bellis %

Oh, no, mission_control-jobs doesn't have a version 0.5.0 yet, only solid_queue. For mission_control-jobs you're good in version 0.3.1.

😅
okay - now I seriously need a fresh cup'o'joe 😆
thx again


⚠ Bofrost.at has degraded performance

In 366f426, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1243 ms

Resolved: Bofrost.at performance has improved in 48c8ac7 after 31 minutes.


Added Portuguese Translation.

adding Portuguese Translation in lesson 5-browser-extension.

Google Chrome Very good Bat Translation ,portuguese, seach plugins ex; /. managed to solve? comment there close as resolved if yes 🎉✨


Workaround for Avahi target name with current Conan recipe

May resolve #298.
avahi/0.8@#10417ed9b1382e1c2b6916db4247dc40 uses CMakeDeps and therefore the target namespace is avahi:: not Avahi::.
@will-leathers-smith-sony, can you try this out? Unfortunately, this won't be tested by GitHub Actions, since the local conanfile.txt doesn't require the Conan avahi package, instead the GitHub Actions build-test workflow uses apt-get to install it.

I have tested this by deleting avahi from my local conan cache and deleting the package for nmos-cpp, then applying the fix to the relevant nmos-cpp cmake file inside Conan. Seems good to me.


Updates for reflex-dom-0.5

Most of of the changes in this PR are small adjustments for changes in the ghcjs-dom API - a lot of functions take ToJSVal a => a rather than JSVal, or return a instead of Maybe a.
The hard part, which I haven't figured out, is how to deal with the onSuccess and onUpgradeNeeded callbacks. I just don't understand this part of the reflex host yet.

The test app starts up well, and it creates the todo table.
Putting 12 into the text box and hitting the + button, the debugging string says:
Waiting..Upgarding..Trans err: "Closed"

It maybe due to the fact that onSuccess callback is commented out.
Some of the event trigger code was written with respect to old reflex.
I will look into that.

Pushed a commit which fixes the issue.
Now the sample code in the test, insert values as expected.

@mageshb Excellent, thank you!


Update meeting-policies-in-teams.md

This setting does not only apply for dial-in users, but also for external users that are joining using Teams client.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Update elixir maker: --one-line is deprecated in favor of --format=on…

Update elixir maker: --one-line is deprecated in favor of --format=oneline.

Ohoh, a PR from master.. ;)
I assume that https://github.com/neomake/neomake/pull/379/commits/c24de2a95f711e4a6fba69da64ea7b3ce314efc5 should be picked from here?  (I can do this)


Cannot fully programatically maximize window.

Hi,
Having this live of code called: this.WindowState = System.Windows.WindowState.Maximized;
does change the size of the window, however there are borders all around the window, where the window isn't touching the edge of the screen or the taskbar. Here is my opening window tag:
    <Controls:MetroWindow x:Name="GameWindow" x:Class="WpfApplication1.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:Controls="clr-namespace:MahApps.Metro.Controls;assembly=MahApps.Metro"
                      ShowMinButton="False"
                      ShowMaxRestoreButton="False"
                      ResizeMode="NoResize"
                      SaveWindowPosition="True"
        Title="MainWindow" Height="350" Width="525" GlowBrush="#FF07908E" Background="#FF252525"
                      WindowState="Maximized"
                      IgnoreTaskbarOnMaximize="True">

Effectively, I want to come out of fullscreen as it is set to when it launches, and have it take up all the screen but not cover the taskbar... Which it does apart from it leaves this strange gap around....

@pingu2k4 which version?

Apologies. Version <IP_ADDRESS>.
As can probably be seen by the simplicity of the program so far I am kind of new to this framework, but I can't find how to do this anywhere I search online, and I'm sure that it should work correctly with what I have...

Just an update - I am now at home where I have 2 monitors and not just 1.
I get the exact same result here with the same project. however, I created a new project with mahapps.metro and a simple button. I used the same line of code to maximise, and on the new project it worked fine...

OK, so I have found out where the problem lies. I had set
ResizeMode="NoResize"
And this is where the problem lies. I don't know whether it is expected behaviour to act as it does when this is set to noresize, but I can get this working properly with the following:
this.ResizeMode = System.Windows.ResizeMode.CanResize;
this.IgnoreTaskbarOnMaximize = !this.IgnoreTaskbarOnMaximize;
this.WindowState = System.Windows.WindowState.Maximized;
this.ResizeMode = System.Windows.ResizeMode.NoResize;

I am happy with using this whether I should have to or not really... I'll just keep this open for time being in case you want to do anything with this still. :) Feel free to close though.

@pingu2k4 this is fixed with 39a7e05a512f21d6f3fa55ec363665b3429b021f

hi everyone,
i think this issue is still open in version 1.2.4 (from nuget).
please check the attached project
thanks
Davide
mahappsdemo.zip

@davidefu Should be fixed now in latest pre-release. As a workaround (with latest stable 1.2.4) set the ResizeMode in loaded event
this.Loaded += (sender, args) => this.ResizeMode = ResizeMode.NoResize;

thanks

hi,
strange issue in the last alpha version <IP_ADDRESS>
if i create a window with style: none, resizemode: noresize, state: Maximized the window ignore the window taskbar and overlap.
but there is a way to resize the window with the shortcut windows + down arrow
if i force in the windowstatechanged event the maximized state, the window is not fully maximized.
By the way wich is the correct way to create a full (with no taskbar overlapping) non resizable window?
please check the attached project

mahappsalphaissue.zip
Thanks

@davidefu Is this a normal running Win10? or is it inside a VM?
You can use IsWindowDraggable="False" instead doing this
        void MainWindow_StateChanged(object sender, EventArgs e)
        {
            if (this.WindowState != System.Windows.WindowState.Maximized)
            {
                this.WindowState = System.Windows.WindowState.Maximized;
            }
        }

Here is the working xaml (without any code behind stuff) that works here on my win 7 prof 64 bit machine
<Controls:MetroWindow x:Class="WpfApplication2.MainWindow"
                      xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
                      xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
                      xmlns:Controls="clr-namespace:MahApps.Metro.Controls;assembly=MahApps.Metro"
                      Title="MainWindow"
                      Width="525"
                      Height="350"
                      ResizeMode="NoResize"
                      IsMinButtonEnabled="False"
                      IsWindowDraggable="False"
                      Topmost="True"
                      WindowState="Maximized"
                      WindowStyle="None">
  <Grid Background="Red">
    <TextBlock><Run Text="<IP_ADDRESS>" /></TextBlock>
  </Grid>
</Controls:MetroWindow>

@davidefu by the way, the shortcut windows + down arrow is normal behavior in win 10, just tested it out with normal window instead MetroWindow.
And, I take your sample with my changes and run it inside VMWare with Win10... and it works :-(

hi,
i'm on a normal windows 10 on a surface pro 4 (so with 175% or 200% of zoom)
i tried with your code, but i don't understand how to make the window maximized without overlap the taskbar.
so is there a way to override the behavior of the windows + down arrow?
if i force (with the code you saw on statechanged event) the maximized state the window appears with the border around it.
same issue if i set the maximized state on windows loaded insted of one the xaml
thanks
davide

i'm on a normal windows 10 on a surface pro 4 (so with 175% or 200% of zoom)

ok, i'll try on my VM and PC this setting with Win 10, but not with a surface, that's not my price class :-)

i tried with your code, but i don't understand how to make the window maximized without overlap the taskbar.

taskbar issue is (should be) fixed with MahApps.Metro 1.3.0-ALPHA137

so is there a way to override the behavior of the windows + down arrow?

i don't know, maybe you can prevent the keys in preview key events

if i force (with the code you saw on statechanged event) the maximized state the window appears with the border around it.

setting the maximized state in the state event is not a good idea

same issue if i set the maximized state on windows loaded insted of one the xaml

yeah maybe this is an issue with your settings of the dpi zoom...

great. l'll be happy to test the next release

@davidefu by the way ResizeMode="NoResize" and WindowState="Maximized" are not good friends! doesn't working with normal windows too!

@punker76 now in the alpha 137 resizemode="NoResize" and WindowState="Maximized" seems to be good friends also with high dpi monitor. Thanks a lot for the support.
Extra chapter:
with this code i was able to override the windows + down arrow
`private const int WM_SYSCOMMAND = 0x0112;
private const int SC_RESTORE = 0xF120;
    protected override void OnSourceInitialized(EventArgs e)
    {
        base.OnSourceInitialized(e);
        HwndSource source = PresentationSource.FromVisual(this) as HwndSource;
        source.AddHook(WndProc);
    }

    private IntPtr WndProc(IntPtr hwnd, int msg, IntPtr wParam, IntPtr lParam, ref bool handled)
    {
        if (msg == WM_SYSCOMMAND && wParam.ToInt32() == SC_RESTORE)
        {
            Debug.WriteLine(msg);
            handled = true;
        }

        return IntPtr.Zero;
    }`

extra detail here https://msdn.microsoft.com/en-us/library/windows/desktop/ms646360(v=vs.85).aspx

@punker76
hi, i've found another really strange issue.
it's happening only with a .NET 4.0 apps (not in .NET 4.5)
and it's not reproducible with the visual studio debugger attacched, so only running from the bin folder directly
you can see that the window is not fully maximized

i've tested it on windows 10 with 100% of zoom and 175% of zoom, save behavior
please find the attacched project
WpfApplication2.zip

can be that a possible related issue? http://stackoverflow.com/questions/19280016/windowchrome-resizeborderthickness-issue

@davidefu please try again with <IP_ADDRESS>, thx!!!

@davidefu any news?

@punker76 hi, the test app attacched is working fine.
tomorrow morning i'll make a more deep test and on a different machine.

i notice a strange issue.
with noresize in xaml i can still resize with win+arrow
this is not possible if i set noresize in loaded event but the window overlap the taskbar

I think we can't fix this in a good way, NoResize and maximized window is
always a bad combination ( also without MahApps ). And I don't get your
issue with the overlapping taskbar. resizing with win+arrow should be
possible, so the window can be moved to another monitors without the sizing
bug.
Am 05.05.2016 9:10 vorm. schrieb "davidefu"<EMAIL_ADDRESS>
i notice a strange issue.
with noresize in xaml i can still resize with win+arrow
this is not possible if i set noresize in loaded event but the window
overlap the taskbar
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/MahApps/MahApps.Metro/issues/1823#issuecomment-217090962

the win+arrow was not possible in previous alpha
the window overlap the taskbar only if i the the resize mode in loaded event and nothing in the xaml

the alpha141 has good bahavior except for the runtime issue regarding the borderthickness

maybe a typo but in BorderlessWindowBehavior.cs at line 84
if you change the line to this.AssociatedObject.ResizeMode = ResizeMode.NoResize;
the behavior seems to be correct


Unable album zip download

Dear friends,
After L-L upgrade (2019-11-03) to the latest version (4.0.0-beta.1),
Im not able to download zipped album or selected photos.
Im getting just browser response "file not found" - links are like the following
https://www.mydomain.com/album/api/Album::getArchive?albumIDs=1572468403
The download links before and after upgrade are similar.
Lychee, Apache and PHP logs are clean.
Seems to me like a probelm with permissions or so, not sure...
Is there any TEMP location?
Im creating bulletproof backups, when doing upgrades etc.
When I returned back to previous version from backup, everything was worked normally.
But there is many uploads, so I need current version working...
Please any idea?
Many thanks!
P.S.: Running Win10x64, Apache 2.4.41, MariaDB 10.4.8, PHP 7.3.11,

Update:
When leaving latest version of Laravel and roll back just "public" folder (uploads untouched) ZIP download stareted to work.
But I have old Lychee containing sorting bug etc :( Unhappy :(

During your upgrade, did you run the migration again ?
php artisan migrate

Thanks ildyria for your response.
Yes I did.
5 minutes ago I found the right solution. My headache is away now.
Needed dependecies fix/update. Fixed by "composer install --no-dev".
Now working like a charm!
Many thanks all you guys for L-L existence, I really like it.

The best way to upgrade is directly to use git instead of the releases. :)
But it is surprising you needed the composer install as those files are supposed to be provided by the .zip

I remmeber, yes you said this a few weeks ago.
Can you please help with a good (free is the best) git piece of soft for Win?
I saw a few of them, but I can not decide which one :(

Personally, the basic Git BASH was good enough for me: https://git-for-windows.github.io/
As long as you know the git commands (git pull etc...) then you are good to go. :)

Many thanks, I'll give them a try :)


Please add a default for "Telescope vim_bookmarks"

Please add a default option to "Telescope vim_bookmarks" and make it search starting from the current directory down, recursively.
Alternatively, make the default option configurable, with the most sensible options be:

from current directory down
inside "this project" directory, where project is found by navigating up to the ".git", ".svn" ... - I believe there are already plug-ins for finding the project root.

I typically launch all my editor windows from the top of the tree so this would not make much of a difference to me, but maybe some users prefer to cd several layers deep.
Thank you!

For now, I changed the behavior so that :Telescope vim_bookmarks is equivalent to Telescope vim_bookmarks all.
I think the behavior you suggested of getting the bookmarks for the current directory or project root (via .git etc.) should be implemented in vim-bookmarks itself. @MattesGroeger what do you think?

Hey @tom-anders, thanks for all your work integrating vim-bookmarks with Telescope 🎉 👏
I have to admit that I'm quite out of the loop regarding the plugin as I have been absent for many years 😅 . From just quickly going through my own documentation I found this part which sounds like what you and @0x8000-0000 are looking for, no?

@MattesGroeger Yeah, I'd also say this looks like what we want. @0x8000-0000 If I understand this correctly, if you use this code snippet from the docu
" Finds the Git super-project directory.
function! g:BMWorkDirFileLocation()
    let filename = 'bookmarks'
    let location = ''
    if isdirectory('.git')
        " Current work dir is git's work tree
        let location = getcwd().'/.git'
    else
        " Look upwards (at parents) for a directory named '.git'
        let location = finddir('.git', '.;')
    endif
    if len(location) > 0
        return location.'/'.filename
    else
        return getcwd().'/.'.filename
    endif
endfunction

then :Telescope vim_bookmarks should do exactly what you want, i.e. only show you the bookmarks for the current project.

@tom-anders - confirmed! It works great, thank you!


To much versions places...will be often inconsistent

http://www.angusj.com/clipper2/Docs/Overview.htm

each source file

clipper.engine.h https://github.com/AngusJohnson/Clipper2/tree/main/CPP/Clipper2Lib/include/clipper2/clipper.engine.h

I think the versioning is not that important.

I think the versioning is not that important.

Thats correct. One place is enough


Add the ability to self-host via pre-built container image for docker-compose stack

Congrats on open-sourcing the raycast ecosystem!
Publishing pre-built container images on docker-hub or ghcr would be useful for self-hosters to host ray.so - especially the Code Images generator - on their own terms and server. Accompanying the container image with an example docker-compose yaml stack would also be useful to facilitate self-hosting.

Hi @jo-chemla, thanks for the suggestion! We're currently focusing on improving the ecosystem itself, so we won't be prioritizing docker images at this time.

Thanks for getting back and understand the reply.
Would you be wiling to let this issue open in case anyone wants to tackle that challenge and eventually do a PR?
Facilitating self-hosting setup might as well result in some traction for rayso itself.
There are multiple guides (eg 1, 2), maybe someone will want to give it a hand at some point.


Check for google* foreigns installed before downloading

This helps to avoid unnecessary downloading if these libs are already installed.

Hmm, (https://github.com/google/googletest/blob/master/googletest/docs/FAQ.md#why-is-it-not-recommended-to-install-a-pre-compiled-copy-of-google-test-for-example-into-usrlocal)[seems] it's discouraged.

 I'll do it anyway.


Ice dracolily spawning

I've been unable to track down ice lilies in the game. Looking at mapgen.lua, line 853, it appears that the game tries to place them on blocks that don't exist - I can't find "default:dirt_with_snowy_grass" anywhere in Minetest documentation. Is it supposed to be "default:dirt_with_snow"?
I might suggest also placing them on "default:permafrost_with_moss", just for the sake of balance since the fiery lilies have multiple options.

I also they have issues with the textures as well, I found some untextured ones that has spawned previously:

Dracolilies have been removed as of 1.2, so this is no longer an issue.


[Bug]: No activity or message if .git/index.lock exists

Describe the bug
I noticed that my git "backups" weren't happening.
I saw no output at all in Obsidian.  Strangely, my obsidian-git settings had returned to their defaults???
Relevant errors (if available)
Running a git command manually generated this message:
fatal: Unable to create '/home/ravi/doc/obsidian/.git/index.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.

Steps to reproduce
I'm guessing a:
touch .git/index.lock

Would do it.  But I didn't create this file it the first place.  Maybe it happened because git died ungracefully?
Expected Behavior
A warning message that the operations of obsidian-git couldn't proceed, mentioning .git/index.lock.
Addition context
No response
Operating system
Linux

I just noticed this in the console:
Uncaught (in promise) Error: fatal: Unable to create '/home/ravi/Documents/obsidian/.git/index.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.

    at Object.action (eval at <anonymous> (app.js:1), <anonymous>:2120:29)
    at PluginStore.exec (eval at <anonymous> (app.js:1), <anonymous>:2155:29)
    at eval (eval at <anonymous> (app.js:1), <anonymous>:2594:43)
    at new Promise (<anonymous>)
    at GitExecutorChain.handleTaskData (eval at <anonymous> (app.js:1), <anonymous>:2592:16)
    at GitExecutorChain.eval (eval at <anonymous> (app.js:1), <anonymous>:2576:44)
    at Generator.next (<anonymous>)
    at fulfilled (eval at <anonymous> (app.js:1), <anonymous>:2496:28)

It would be great to have a popup message on error :)

I can't reproduce. Which command do you call? Does commit all changes work? (Show the error message). When I add the file via touch I get an error notification.


Review data being injected into the datalake

We need to follow what's specified:

raw data
signed data

we're currently having a mix of both

Also, besides the raw data and the signed data, we need to send the operator ID. Furthermore, the monitoring data (json field) must include the raw data, the resource ID and the referenceID.

Concerning the Kafka-python library, Kafka Producer has certain restrictions on the parameters to be sent. For example, the arguments must be of type bytes or serializable to bytes via value_serializer or key_serializer. Due to these impositions, we were unable to send everything in a field value: the monitoring data (raw data), the operatorID, and the hashed Data (already encrypted). Therefore we send in 2 different fields, a key with the date signed and encrypted, and a value with the following data:
data = {
    "monitoringData" : {
        "metricName" : "cpu_utilization",
        "metricValue" : "0.453",
        "resourceID" : "X",
        "referenceID" : "Y",
        "timestamp": "the time which the metric was measured"
    },
    "operatorID" : "id_example"
}

With the producer command as:
producer.send('topic_test', key=hashDataEncrypted, value=data)

@aviweit what you think about the last comment? It is agreeable for you to receive it in this format?

[PoS] The last version, updated today, sends to a Kafka topic the agreed fields: 1. the raw data with an associated operator ID, business ID, and network ID; 2. the encrypted data. We will close this issue but we still need to address where the MDA obtains the secret or pair keys to encrypt and finally how DL will decrypt the data send via Kafka producer.


Expected to mark variable ready only once

Hi
Im using the finetuning notebook example to try to fine-tune llama2.But I am getting this error.
Expected to mark a variable ready only once. This error is caused by one of the following reasons: 1) Use of a module parameter outside the forwardfunction. Please make sure model parameters are not shared across multiple concurrent forward-backward passes. or try to use _set_static_graph() as a workaround if this module graph does not change during training loop.2) Reused parameters in multiple reentrant backward passes. For example, if you use multiplecheckpoint functions to wrap the same part of your model, it would result in the same set of parameters been used by different reentrant backward passes multiple times, and hence marking a variable ready multiple times. DDP does not support such use cases in default. You can try to use _set_static_graph() as a workaround if your module graph does not change over iterations. Parameter at index 559 has been marked as ready twice. This means that multiple autograd engine  hooks have fired for this particular parameter during this iteration. You can set the environment variable TORCH_DISTRIBUTED_DEBUG to either INFO or DETAIL to print parameter names for further debugging.
Interesting that this only happens when I enable gradient checkpointing. But if I don't use gradient checkpointing it will run OOM.

@nivibilla Hi, did you solve it?

I ended up using TRL instead


Conntrack memory leak?

Hi,
We're seeing an issue on Lighthouses where conntrack appears to be consuming more and more memory which eventually crashes the host.  We're seeing this pattern on non-Lighthouse nodes too but at a much smaller scale (~30k conntrack size vs 13 million).
Here's a heap profile:
go tool pprof heap-profile                                                                                1 ↵
File: nebula
Type: inuse_space
Time: Aug 14, 2020 at 12:55pm (IST)
Entering interactive mode (type "help" for commands, "o" for options)
(pprof) top5
Showing nodes accounting for 1728.58MB, 98.92% of 1747.47MB total
Dropped 57 nodes (cum <= 8.74MB)
Showing top 5 nodes out of 9
      flat  flat%   sum%        cum   cum%
 1319.06MB 75.48% 75.48%  1726.08MB 98.78%  github.com/slackhq/nebula.(*Firewall).addConn
  407.01MB 23.29% 98.78%   407.01MB 23.29%  github.com/slackhq/nebula.(*TimerWheel).Add
    2.50MB  0.14% 98.92%       16MB  0.92%  github.com/slackhq/nebula.ixHandshakeStage1
         0     0% 98.92%  1726.08MB 98.78%  github.com/slackhq/nebula.(*Firewall).Drop
         0     0% 98.92%  1721.07MB 98.49%  github.com/slackhq/nebula.(*Interface).decryptToTun

And graph of conntrack size:

The steep drop-offs are coming from restarting the VM.
Some other info:

Nebula version 1.2.0
CentOS 7 running on VM
memory: 4GB
~2k hosts are running health checks against the lighthouse once a minute

After adding a list-conntrack sshd command and inspecting the contents of the conntrack table I was able to see the cause. Our nodes' healthchecks are using netcat without a source port specified meaning the node will use a random port. So each time the healthcheck is run, a potentially new source port is chosen which results in a new entry in conntrack.

Ah that would do it. One other thing that you can do is to set the conntrack expiration timers to be much more aggressive. At Slack, we actually used values based on what AWS does on their internal network. Iirc they are something like 60s for UDP, 12m for TCP, but I could be slightly off there. Depending on your particular needs, you could adjust these down aggressively.

Ours is set at 120h at the moment :p. Thanks!


Rename ComputeError exception

Same name as field on QCElemental FailedOperation object. Creates confusion.

Switched to qcio; no longer relevant.


[Bug] Stream-InitSink failed

What happened
I am trying to fetch data from cloud MySQL to cloud MySQL by Pulsar

add cluster

add data node

config data ingestion

What you expected to happen
the system show fail
How to reproduce
I've pasted the picture
Environment
No response
InLong version
master
InLong Component
InLong Manager
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

你好，mysql这个语法错误解决之后，还是出现config_failed的错误

这个该怎么查呢？
新手刚刚开始用InLong，想用InLong从腾讯云MySQL上拉数据到腾讯云MySQL，麻烦详情讲一下吧，非常感激！

please see the log details to get the specific cause.


Install.sh  fails on  Linux Mint 21.3

Describe the bug
Fresh install on Linux Mint 21.3,  sudo bash install.sh  fails with  cat: /etc/upstream-release: Is a directory
To Reproduce (MANDATORY, DO NOT REMOVE)
Explicit steps to reproduce the behavior:

Fresh install of Linux Mint 21.3
wget https://github.com/espocrm/espocrm-installer/releases/latest/download/install.sh
sudo bash install.sh

Expected behavior
The install script to recognize  that it is running on a Linux Mint 21.3 system and appropriately determine the information
it needs from the /etc/upstream-release directory
Screenshots
If applicable, add screenshots to help explain your problem.
EspoCRM version
Specify the EspoCRM version (or versions) on which the bug occurs.
Fresh pull of Espo 8.1.4

Additional context
Add any other context about the problem here.

As the Error states, OS is not supported. Not a bug.

Except that the documentation explicitly states that "Linux Mint" is a
supported OS.
The problem is  that this piece of code:
local linuxString=$(cat /etc/*release | grep ^NAME | tr -d 'NAME="' |
tr '[:upper:]' '[:lower:]')
declare -a linuxOsList=(centos redhat fedora ubuntu debian mint)
To me, it says that Linux Mint is is a supported OS ...
NAME="Linux Mint"   -- which is the right string

The  tr -d 'Name="'  returns the string  -- Linux int -- Which is too
long and int doesn't match any of the names in
the linuxOsList
Mike
On 2/14/24 10:16, Yuri Kuznetsov wrote:

As the Error states, OS is not supported. Not a bug.
—
Reply to this email directly, view it on GitHub
https://github.com/espocrm/espocrm-installer/issues/7#issuecomment-1944157234,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACOD7J2XLNMJWII7YF4MQETYTTPNFAVCNFSM6AAAAABDHFZAOSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBUGE2TOMRTGQ.
You are receiving this because you authored the thread.Message ID:
@.***>

@mhmoran after deep investigating we added support Linux Mint.
So, please use the documentation for the installation, https://docs.espocrm.com/administration/installation-by-script/ .


Radio

Miss the RadioGroup for don't allow to check multiples radio

Miss the RadioGroup for don't allow to check multiples radio

Need a selected_value props for the RadioGroup
https://github.com/palantir/blueprint/blob/f4cf4a70ec3a59c86f292d841fb0e1bc65237abe/packages/core/src/components/forms/radioGroup.tsx#L61

Need a selected_value props for the RadioGroup
https://github.com/palantir/blueprint/blob/f4cf4a70ec3a59c86f292d841fb0e1bc65237abe/packages/core/src/components/forms/radioGroup.tsx#L61


How to post data to comments

http://localhost:3000/posts/    post ok
http://localhost:3000/posts/1/comments/   post failed

Hey bud, we cannot see your local files. I think a screenshot and some source code would be better?

I guess you want to create a new comment using POST /posts/1/comments. Currently, the way to create a comment for post 1 is to POST to /comments with the desired postId:
// POST /comments
{
  body: 'a comment for post 1',
  postId: 1
}

It will then appear in /posts/1/comments.
Does that help?

Thx @typicode solve my problem.
I have use mysql yet , and json-server will be used in which circumstances?  (json-server is good)  :p bad english.

Thx @sciku1 replay issue. :)

Is this still an issue? That seems like a fix, unless you want to make the '/post/1/comments/' fully restful.


UX Refactor | Rethinking the graphic on the Credit Transaction View screen |3| [v0.2.0]

https://trello.com/c/qaUd7URs/585-ux-refactor-rethinking-the-graphic-on-the-credit-transaction-view-screen-3-v020
Past work:
A number of changes to the Credit Transaction View screen have already been completed, see Card 567.
This Card:
The current graphic on the View Transaction screen was designed specifically for Credit Transfers. This graphic is not appropriate for other balance change transactions (Validation, Part 3 Award, Reduction) as credits are only flowing to a fuel supplier (with no cash going back to the Government of BC).
Potential design solution(s):

For all non-credit transfer transactions (Validation, Part 3 Award, Reduction), remove the Government of BC "box" from the graphic. Also remove the credit value arrow (i.e. <--, left arrow) as these transactions do not involve monetary value.
For Validation and Part 3 Award, we could have an upward facing arrow with the quantity of credits shown on one side and the company shown on the other (i.e. in the existing "box" that is already in the graphic)
For Reduction, we could have a downward facing arrow with the quantity of credits shown on one side and the company shown on the other (i.e. in the existing "box" that is already in the graphic)

 UX Refactor | Rethinking the graphic on the Credit Transaction View screen |3| [v0.2.0]


fixcatch exception

Proposed changes
Capture exception data and return it to the user as a prompt.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


NoMethodError with mongoid

We are seeing lots of errors like these ones that are crashing our app:
** [NewRelic][2020-06-10 21:47:12 +0000 scheduler.* (4)] ERROR : NoMethodError: undefined method `reply' for #<Mongo::Monitoring::Event::CommandFailed:0x00007f3929cdaed8>
** [NewRelic][2020-06-14 01:18:02 +0000 worker.1 (4)] ERROR : NoMethodError: undefined method `descendant_complete' for nil:NilClass

The reply error seems to come from these files:
app/worker.1 /app/vendor/bundle/ruby/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>5/lib/new_relic/agent/instrumentation/mongodb_command_subscriber.rb:25:in `error_present?'
app/worker.1 /app/vendor/bundle/ruby/2.6.0/gems/newrelic_rpm-<IP_ADDRESS>5/lib/new_relic/agent/instrumentation/mongodb_command_subscriber.rb:34:in `completed'

Versions:

newrelic_rpm (<IP_ADDRESS>5)
mongoid (6.4.2)
mongo (2.12.1)

The mongo gem changes its implementation of error reporting at version 2.6.0.
The agent was recently enhanced to report errors at span event level.
The two together leads to this specific issue.  There are two quick fixes that will resolve this immediately:

downgrade mongo gem to 2.5.3
downgrade ruby agent to <IP_ADDRESS>3

Meanwhile, we will definitely be fixing the auto-instrumentation for mongo to support newer versions of the gem.

MongoDB instrumentation has been updated to cover 2.6 through 2.12
The error reported above has also been addressed.
These fixes will appear in the next release of the agent.

Until the fix is released, the following patch may be applied (typically by loading in a Rails initializers file:
module NewRelic
  module Agent
    module Instrumentation
      class MongodbCommandSubscriber

        def error_key_present?(event)
          if reply = event.reply
            ERROR_KEYS.detect{ |key| reply[key] }
          end
        rescue => error
          false
        end

        def completed(event)
          begin
            return unless NewRelic::Agent::Tracer.tracing_enabled?
            segment = segments.delete(event.operation_id)
            return unless segment

            # operations that succeed buy have errors return CommandSucceeded
            # with an error_key that is populated with error specfics
            if error_key = error_key_present?(event)
              # taking the last error as there can potentially be many
              attributes = event.reply[error_key][-1]
              segment.notice_error Mongo::Error.new("%s (%s)" % [attributes["errmsg"], attributes["code"]])

            # failing commands return a CommandFailed event with an error message
            # in the form of "% (%s)" for the message and code
            elsif event.is_a? Mongo::Monitoring::Event::CommandFailed
              segment.notice_error Mongo::Error.new(event.message)
            end
            segment.finish
          rescue Exception => e
            log_notification_error('completed', e)
          end
        end

      end
    end
  end
end


Bug Update Feedback.tsx component visiblity issue

Fixed the Feedback Component visibility issue. Now feedback component will become invisible after 3 seconds of submission.

Seems better than the current version, could you reduce the time to 1.5-2 seconds and add an exit animation?

Thank you for your feedback! I'm glad to hear that the changes are an improvement. I'll work on optimizing the performance to achieve a response time within the 1.5-2 seconds range. Additionally, I'll explore the possibility of adding an exit animation to enhance the user experience."

I've made the changes as per your suggestions. The exit animation is now set to 1.5 seconds, and I hope it provides a smoother user experience.

@Abdul-Rahman-E thanks for the update! It's not quite what I had in mind, because I was thinking that the "Thanks" text would show for a second, and then it would fade out. I pushed a tweak to the timings, and I also updated it to be CSS-only.
Before
https://github.com/reactjs/react.dev/assets/2440089/21e3c525-03f2-4fde-ad7c-c4557ec62164
After
https://github.com/reactjs/react.dev/assets/2440089/6d64174e-35bd-44a9-a600-e4a898f2fc21


[WIP] Phoenix Instrumentation

This PR replaces the plug-based instrumentation code with an alternative provided by Phoenix: Endpoint instrumentation.
By using Phoenix instrumenters we offload part of the work on calculating elapsed time to Phoenix itself, note that we no longer rely on the Transaction module for that and record straight to Collector instead.
From now on NewRelixir.Plug.Phoenix gets deprecated and users of Plug-based frameworks other than Phoenix will be able to instrument their endpoints with NewRelixir.Instrumenters.Plug.
There is still some work to do before closing this and cutting the next release:

[x] Track current transaction through conn to ensure Ecto instrumentation still works.
[ ] Add some unit tests.
[ ] Update the README with setup instructions for the new instrumenter.
[ ] Update the CHANGELOG.
[ ] Improve documentation.
[ ] Make the dependencies on Phoenix and Plug optional.

resolves https://github.com/TheRealReal/new-relixir/issues/12

👍 ✨ Very good implementation

@coburncoburn @ulissesalmeida Just found out Ecto's instrumentation depends directly on the plug-based Phoenix instrumenter, through conn.private.new_relixir_transaction. I added a task to fix that, but it will probably involve some changes to Ecto instrumentation too.

Finished working on the transaction tracking through the process dictionary, it is the same approach we've been using for external requests. Now we no longer need to rely on the conn struct.

@coburncoburn Finished doc updates ✅ 100% ready now.