Mattermail is not working

Hi,
I want Mattermail to automatically send incoming messages to a  Mattermost channel , but it doesn't work for me. My Mattermost server runs locally and is not accessible from the internet.
I get the following log outputs:
Mattermail Server Version: 4.0-dev
INFO Forumposts	09:07:03 Using Mattermost Api Version 4
DEBG Forumposts	09:07:03 Debug mode on
INFO Forumposts	09:07:03 Checking new emails
DEBG Forumposts	09:07:03 MailProviderImap.CheckNewMessage
DEBG Forumposts	09:07:03 MailProviderImap.CheckConnection: Dial
INFO Forumposts	09:07:03 Connected with "XXXXXXXXXXX:143"
DEBG Forumposts	09:07:03 ecoL7A LOGIN XXXXXXX XXXXXXX

DEBG Forumposts	09:07:03 MailProviderImap.selectMailBox: Select mailbox: INBOX
DEBG Forumposts	09:07:03 CuzPfQ EXAMINE INBOX

DEBG Forumposts	09:07:03 * 3 EXISTS
* 0 RECENT
* OK [UNSEEN 1] Message 1 is first unseen

DEBG Forumposts	09:07:03 * OK [UIDVALIDITY 1]
* FLAGS (\Answered \Seen \Recent \Flagged \Deleted \Draft)
* OK [PERMANENTFLAGS (\Answered \Seen \Recent \Flagged \Deleted \Draft \*)]
CuzPfQ OK [READ-ONLY] EXAMINE completed

DEBG Forumposts	09:07:03 5v1OGg CAPABILITY

DEBG Forumposts	09:07:03 * CAPABILITY IMAP4rev1  AUTH=CRAM-MD5 NAMESPACE ACL UIDPLUS ENABLE LIST-EXTENDED CHILDREN SPECIAL-USE CREATE-SPECIAL-USE METADATA
5v1OGg OK CAPABILITY completed

DEBG Forumposts	09:07:03 Im7c0Q CLOSE
DEBG Forumposts	09:07:03 Im7c0Q CLOSE succeeded
imap/client: 2018/10/12 09:07:03 response has not been handled: &{Im7c0Q [CLOSE succeeded]}

My config.json File looks like this:
{
	"Directory": "./data/",

	"Profiles": [
		{
			"Name": "Forumpost",
			"Channels": ["#CHANNELNAME"],

			"Email": {
				"ImapServer": "XXXXXXX:143",
				"Username": "USERNAME",
				"Password": "PASSWORD"
			},

			"Mattermost": {
				"Server": "http://XXXXXXXX:8065",
				"Team": "test",
				"User": "XXXXXXXX",
				"Password": "XXXXXXXXX"
			}
		}
	]
}

The data directory exists but nothing happens.
Can anyone help me with this problem?

Doesn't anyone have any idea why it doesn't work? I'm thankful for any help.


Support for async spans?

One way async span is supported in brave - https://github.com/openzipkin/zipkin/issues/1243
Are there plans to support this any time soon?

Thanks for asking, Sunil.
We are likely to pull out zipkin-go which could include the "flush"
operator on span present in Brave (which makes this work). It is possible
some custom hooks could be done in the mean time to accomplish the same.
On zipkin-go: there's currently no tracking issue for this, but it is
dependent on the "simplejson" effort: The simplified data structure has
been piloted in Brave 4+ as the primary span api, and that's been
successful. When we extract zipkin-go, we'd not only want the same
approach, but also a direct mapping to the model to make mapping less
indirect: https://github.com/openzipkin/zipkin/issues/1644

PS the async as described in brave is SPSC or really one-way RPC. Multiple consumers are being elaborated here https://github.com/openzipkin/zipkin/issues/1654


Latest Firmware has wider voltage range

Just a note for your Readme: whatever the latest firmware is for this CM1620 seems to allow a much wider voltage range. Using the Android App all cell chemistry 'options' allow changing the cell voltage in a much wider range, so it doesn't seem to matter what chemistry you set you can use it to charge any other chemistry assuming the voltage is correct.

I was almost certain that I could answer "yes I tried it and it worked"
But thought I would double check. So I plugged the same power source I had plugged in before back in and it burned out and let out the magic smoke. Second one to do this. It's going back to Amazon... there are some serious problems with the design of the hardware for this to happen.
I had assumed the first one I had was faulty, so with this one I was tentative and plugged in a low voltage (12v) supply first, which allowed it to boot fine. It also booted with a PC plugged into the micro USB port. I subsequently plugged in a 58v power supply which worked so I thought for sure the first one was faulty. But now I just plugged this one back in and poof!
 I have a feeling iSDT abandoned it becauase it's riddled with issues.

Yea as it says in the readme: I've also had one release its magic smoke (though in my case it was in a weird configuration of parallel connection of multiple units, which might've been the issue).
I guess there's a reason they are so cheap.
Next time I'm using mine I'll try to remember to upgrade the firmware and check if other cell voltages work then.


unirest  so many AsyncIdleConnectionMonitorThread  running !!!!!!!!!!!!!!!!!!!!!!!!!

if i use unirest asyn, when many requests,  my server will happen heap outofmermory beacuase  so
many AsyncIdleConnectionMonitorThread  running backround! how i use unirest .shutdown() to sovle this prolem！
if my code like this :
Unirest.setTimeouts(0, 0);
Future<HttpResponse> future = Unirest.post(url)
.header("accept", "application/json")
.fields(map)
.asJsonAsync(new Callback() {
public void failed(UnirestException e) {
try {
Unirest.shutdown();
} catch (IOException e1) {
e1.printStackTrace();
}
}
public void completed(HttpResponse response) {
try {
Unirest.shutdown();
} catch (IOException e) {
e.printStackTrace();
}
}
public void cancelled() {
try {
Unirest.shutdown();
} catch (IOException e) {
}
}
                });

Please don't use so many exclamation marks.

Hello, this should be resolved or addressed in Unirest 2. Please see the upgrade guide for important changes.


refactor(element-templates): create maven plugin to generate element templates on build

Issue
Element templates are generated on runtime.
Description
The current element template generation is refactored and moved to a maven plugin that is executed during the build. The element template generation can be further customized by specifying an outputDirectory and targetPlatforms (eg. camunda7).
Checklist

Code is easy to understand
No obvious errors or bugs
CI/CD Pipelines did run successfully
Tests were implemented
Documentation was created

Codecov Report
Patch and project coverage have no change.

Comparison is base (730797d) 13.90% compared to head (46d2322) 13.90%.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #122   +/-   ##
=========================================
  Coverage     13.90%   13.90%
  Complexity       24       24
=========================================
  Files            74       74
  Lines           453      453
  Branches          6        6
=========================================
  Hits             63       63
  Misses          389      389
  Partials          1        1

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

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

Good first version @rami-nk!
Let's talk about the things I found.
One more question.
Is it possible to use the plugin in gradle?

Great Work 👍

Thought about the version again and again... and added it back to the ElementTemplate Annotation.
I guess we should release it that way and gain some experience with it.


[Backfill corrections] Account for differences in fields in daily input files; convert date fields

Description
Daily and rollup (covering ~4 weeks) input files are formatted slightly differently. Daily input files don't contain lag or issue_date fields that are necessary for data filtering and modeling. In the pipeline, we combine rollup and daily files using bind_rows, whose output includes the union of all fields seen in component dfs. If a given field is missing from one of the component dfs, those entries are filled with NA. This happens to the lag and issue_date fields for daily files.
In the current version of the pipeline, we check if the lag and issue_date fields are entirely missing. However, even if this check passes, the missing values from daily files cause problems later in the pipeline.

So, add issue_date field to daily dfs on read. From this, derive the lag field.

Date fields time_value and issue_date, when available, are read in as datetime class with the local timezone. We expect these to be dates. All of the datetimes correspond to UTC midnight datetimes (such that the date is one day later). In the Python pipelines that produce input files, these date fields are actually formatted as datetime64[ns] (timezone-naive datetimes). It appears that R's arrow::read_parquet is assuming these are in UTC, and converting to the host's timezone.

Convert these back to UTC and then to dates for appropriate handling.

The pipeline expects a field call geo_value; input files call it fips instead, so rename.

Changelog

NAMESPACE
io.R
main.R
utils.R
documentation files

Superseded by https://github.com/cmu-delphi/covidcast-indicators/pull/1760


Monoid instance for ClassName

When you have dynamic classes we have couple options to encode them:
<> and guard
[ ...
, HP.classes $
  [ HH.ClassName "baz"
  , HH.ClassName "foo"
  ]
  <> (guard isBar $> HH.ClassName "bar")
  <> (guard isFiz $> HH.ClassName "fiz")
]

fold and guard
[ ...
, HP.classes $ fold
    [ [ HH.ClassName "baz"
      , HH.ClassName "foo"
      ]
    , guard isBar $> HH.ClassName "bar"
    , guard isFiz $> HH.ClassName "fiz"
    ]
]

STArray
unArr do
  push $ HH.ClassName "baz"
  push $ HH.ClassName "foo"
  guard isChecked $> HH.ClassName "bar"
where
unArr = unsafeFreeze >>> runST >>> runPure
push = pushSTArray

mguard - needs Monoid instance
[ HH.ClassName "baz"
, HH.ClassName "foo"
, mguard isChecked $ HH.ClassName "bar"
]
where
mguard b m = if b then m else mempty

I think last one looks better, also it should be better in terms of performance.
So as we already have Semigroup (#451), what if we added Monoid instance to ClassName?
only drawback might be having extra spaces in dom, here is definition of classes:
classes = prop (PropName "className") <<< joinWith " " <<< map unwrap

If spaces become an issue we can update classes function to just fold over array and use spaces only when element is not empty string (it might even be faster then join)

Btw, that mguard exists as Data.Monoid.guard now, but a new release hasn’t been tagged yet

@spicydonuts didn't knew about that, Thanks! updated description

How about this?
className :: String -> Maybe ClassName
className = Just <<< ClassName

classes = prop (PropName "className") <<< joinWith " " <<< catMaybes <<< map unwrap

[ HH.className "baz"
, HH.className"foo"
, guard isChecked *> HH.className"bar"
]

I'm not totally sure about semantics of monoid's mempty but pure mempty === empty === mempty in this case seems a bit dodgy, doesn't it?

pure mempty === empty === mempty  is not true:

pure mempty :: Monoid a => Maybe a which is Just mempty
empty :: Maybe a and mempty :: Maybe a both are Nothing.

That's  an option too, but ClassName has empty value which we could use instead of extra wrappers.

Sorry, I meant evaluation time and ClassName monoid w/o Maybe
[ ClassName "" ] =evaluates=> ""
[ ] =evaluates=> ""

[ ClassName "foo", ClassName "bar" ] =evaluates=> "foo bar"
[ ClassName $ "foo" <> " bar" ] =evaluates=> "foo bar"

That's not good at all (looks like javascript, right?) :(
Providing smart constructor forall m. Alternative m => String -> Alternative ClassName and removing class_ (or classes_) will help
className "" =ev=> [ ] =ev=> ""
className "foo" =ev=> ["foo"] =ev=> "foo"
className "foo" <> className "bar" =ev=> ["foo", "bar"] =ev=> "foo bar"
-- shouldn't have spaces in body
className $ "foo" <> " bar" =ev=> [ ] =ev=> ""

@cryogenian We could ask for Symbol and make sure it's not empty + has no space at compile time.
so you are thinking of something like this?
newtype ClassNames = ClassNames (Array String)

derive newtype instance Monoid ClassNames

classNames :: forall x
 . IsSymbol x
  => SymbolNotEmpty x
  => SymbolHasNoSpace x
  => SProxy x -> ClassName
classNames = ClassName [reflectSymbol :: SProxy x]

or
class ValidClassName x
instance (IsSymbol x, SymbolNotEmpty x, SymbolHasNoSpace x) => ValidClassName x

classNames :: forall x
 . ValidClassName x
  => SProxy x -> ClassName
classNames = ClassName [reflectSymbol :: SProxy x]

and use it like this:
HP.classes
  $  classNames @"foo"
  <> classNames @"baz"
  <> Data.Monoid.guard isFiz (classNames @"fix")

@safareli This is even better! :+1:

update:
<> and Monoid.guard
it's nicer as you do not need pares over guard statements
[ ...
, HP.classes $
  [ HH.ClassName "baz"
  , HH.ClassName "foo"
  ]
  <> guard isBar [HH.ClassName "bar"]
  <> guard isFiz [HH.ClassName "fiz"]
]


Error when creating a pod which was previously deleted

Describe the bug
When this endpoint POST https://fairos.fairdatasociety.org/v1/pod/new is called for a pod which was previously deleted, the response is:
{"message":"pod new: error uploading data","code":500}

This should be gone once #118 is resolved

duplicate of #117


Dashboard ide

Hi @jambulud, @sjimenez77,
Please review the code for the tasks - Deleting a project and opening a project path location.
Thanks,
Dattu Chillal


new-post: snakemake support

Closes #15
Co-authored-by: Maria Fernando<EMAIL_ADDRESS>Co-authored-by: Tibor Simko<EMAIL_ADDRESS>Preview available on Netlify: https://deploy-preview-16--flamboyant-wing-4d6caf.netlify.app/

Thanks, I reread it again and it seems ready to be merged.
Blocked as we need to update the date to match the date of the publication.


windows terminal breaks on ctrl+C

Describe the bug
https://discord.com/channels/1212416027611365476/1317009958113644554
Reproduction
I don't have Windows, see discord.
Steps to reproduce
pnpm dev

System Info
TODO

Used Package Manager
pnpm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Here's the loom video from my window's 10 system.
I don't think so it seems to be the case with readline (still skeptical tho).
Cause in the video you can see after terminating server. My powershell seems to be loading properly.
But that is also very inconsistent.
But after running pnpm test command it crashes.

@nishu-murmu So with my readline change, you can press ctrl+C twice and everything works just fine? Still need to figure out why the prompt get's messed up, but that's progress, right?

@aklinker1
Yeah, in my powershell it seems to be working.
In windows it does works but very inconsistent, sometimes I also face that crash issue where powershell gets useless and am not able to enter any command.
In linux it is showing something like this.

Windows is more problematic.
Maybe we need to handle that exit code 130 thing? I saw some similar code in vitest's codebase. They were programmatically handling that case.
What do you think?

They were programmatically handling that case.

Can you share where they are doing that?

Here,
https://github.com/vitest-dev/vitest/blob/main/packages/vitest/src/node/stdin.ts#L49-L68
But they seem to be using raw input mode.
Can you also look around a bit and see if it's possible to use raw input mode.

Yeah, we specifically chose not to use raw input mode because then you have to manually handle all keyboard shortcuts for interrupting the process.

Yeah I know, I'm not suggesting raw input mode. I'm pitching maybe we can do something for windows specifically.
They also OS specific code.

Oh I see, sorry yes, that makes sense. Though it means we have to manually handle an edge case, which I wanted to avoid :/
@nishu-murmu Did you ever explore older versions of WXT and find the version that introduced this behavior? I don't think I ever heard confirmation from anyone that v0.19.19 actually introduced this.

This is powershell terminal
https://www.loom.com/share/f914b2f34f994551b630b231708be473?sid=f9191019-8916-4064-b9e9-72fbd2df767b
Here is the tested video for version 0.19.18
It seems to be working fine.
After pressing Ctrl + C the powershell terminal is working fine.
But yeah pressing twice works.

Does the "-> wxt-demo" usually show up when asking to confirm stopping the process? I haven't used powershell in a while but previously, that prompt would show on its own line.

Yes,
It shows up when asking for confirmation.
This works fine initially.
For any version.
But doing this more than twice or thrice breaks the windows terminal
https://www.loom.com/share/f914b2f34f994551b630b231708be473?sid=34c3b762-0b10-447f-9d9e-033fbbb074e9

Doesn't look like its work to me. When you type "Y", shouldn't it stop the process and give you a new blank line to type on? I understand that pressing Ctrl+C again makes it work, but you shouldn't have to press it again after typing "Y"


Add sender name for mails

Sent mails currently don't have a sender name and it will display the mail address / part before the @ instead (depending on the mail client)
What we can do is adding a sender name for sent mails.

Can be solved by adding the info in the EMAIL_FROM variable like:
EMAIL_FROM='Don't Reply<EMAIL_ADDRESS>Needs documentation tho


Game stats

Codecov Report

Merging #23 (6a9637d) into develop (16a2179) will increase coverage by 0.09%.
The diff coverage is 92.90%.

@@             Coverage Diff             @@
##           develop      #23      +/-   ##
===========================================
+ Coverage    94.08%   94.18%   +0.09%
===========================================
  Files           42       45       +3
  Lines          575      705     +130
  Branches        84      109      +25
===========================================
+ Hits           541      664     +123
- Misses          29       35       +6
- Partials         5        6       +1

Impacted Files
Coverage Δ

src/components/Button.jsx
100.00% <ø> (ø)

src/components/ErrorMessage.jsx
100.00% <ø> (ø)

src/components/ServicesLink.jsx
100.00% <ø> (ø)

src/modals/LeaderboardDetails.jsx
100.00% <ø> (ø)

src/pages/PlayerProps.jsx
100.00% <ø> (ø)

src/modals/StatDetails.jsx
89.41% <89.41%> (ø)

src/components/TextInput.jsx
97.29% <96.87%> (+17.29%)
:arrow_up:

src/components/Modal.jsx
100.00% <100.00%> (ø)

src/components/RadioGroup.jsx
100.00% <100.00%> (ø)

src/utils/nullableNumber.js
100.00% <100.00%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


Added ocpv_lab and members to validated-content-curators team

Added ocpv_lab and members to validated-content-curators team to resolve #668

@garethahealy conflicts have been resolved


LLM-based agents for multi-step tasks and dynamic environments

Description
Set up LLM agent to do open-vocabulary manipulation
Checklist

[ ] I have performed a self-review of my code
[ ] If it is a core feature, I have added thorough tests
[ ] I have added documentation for the changes
[ ] I have updated the README file if necessary
[ ] I have run on hardware if necessary

Screenshots (if applicable)
Add any relevant screenshots or screen recordings to help reviewers understand the changes.
Additional context
Add any other context or information about the pull request here.

Tested 10-23; ai_pickup demo works; need audio fix in docker


feature request: host completion improvements

[X] I have read through the manual page (man fzf)
[X] I have the latest version of fzf
[X] I have searched through the existing issues

Info

OS

[X] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[X] bash
[ ] zsh
[ ] fish

Hey.
I think the current host completion has a number of areas where it could be improved:

It's not (easily) configurable, e.g. it would be nice to have a user switch that disables v4 and/or v6 addresses, and perhaps options that do sorting
It lacks many names that are shown by bash-completion’s _known_hosts_real(), which is quite powerful and IIRC even checks ssh_config for other configured GlobalKnownHostsFile and UserKnownHostsFile. fzf’s current completion seems to ignore any host (which are more or less all of them) that I have in the GlobalKnownHostsFile. _known_hosts_real() also has many more features, I think it can optionally use Avahi and ruptime.

For (2), I think it would be best if fzf would simply use _known_hosts_real() - why re-inventing the wheel, if bash-completion guys already maintain that properly?
Based on the 0.42.0 fzf, I tried roughly the following:
_fzf_host_completion() {
  _fzf_complete +m -- "$@" < <(
        _known_hosts_real ''
        printf '%s\n' "${COMPREPLY[@]}" | sort -u -V
  )
}

and at a first glance it seemed to work:

The sort-u is necessary as COMPREPLY contained many duplicates (no idea why).
-V is just for sorting ... ideally this would be configurable,... plus I'd like a sort mode that sort by domains[0].

Some open points:

Is _known_hosts_real() available for sure? I mean bash-completion does load completions for commands dynamically (i.e. only when the command is actually completed), but I'd blindly guess that _known_hosts_real() is always available and we don't need to "import" it first.
Do we have any side-effects? At least COMPREPLY is set. Haven't checked whether _known_hosts_real() uses other non-local variables. As long as the function is called within process substitution <(…) this shouldn't be a problem, but I've seen that in edfdcc8cee31e227a5f7eda9c1faa4893757645c you placed the printing code in a separate function, so if that would ever be called outside process substitution it might cause unwanted side effects.
What about the config env vars recognised by _known_hosts_real(), like COMP_KNOWN_HOSTS_WITH_AVAHI? Should we just use them directly, or clean them and initialise them only by our own e.g. FZF_KNOWN_HOSTS_WITH_AVAHI counterparts (so that people could configure bash-completion and fzf differently)?

[0] With sorting by domains I mean e.g.
9.aq
5.a.com
6.a.com
1.b.com

I.e. sorted reversely by each domain component, and within the same component one should again do alphabetical (or better natural) sort.

Thanks for the suggestion, but before we discuss the topic in detail I have to mention that I'm not particularly looking to improve/expand the shell features (key bindings and fuzzy completion) we provide here. They are intentionally kept simple (with minimal configuration knobs)

to reduce the maintenance burden of keeping three implementations (bash, zsh, fish) in sync
to serve as the basic reference implementation with no bells and whistles for those who are trying to learn how to write fzf-powered shell functionality on their own.

I believe extended versions of the scripts should be maintained as separate projects.

https://github.com/junegunn/fzf/issues/2805#issuecomment-1110366399
https://github.com/junegunn/fzf/issues/1761#issuecomment-555581153

https://github.com/Aloxaf/fzf-tab is a good example.
A separate project focused on a specific shell doesn't have the problem of maintaining the same functionality across three different shells, so it should be easier to add more advanced features. It's going to be used by advanced users who know what they're doing so the maintainer can experiment, break, and fix things with more confidence.

Thanks for the suggestion, but before we discuss the topic in detail I have to mention that I'm not particularly looking to improve/expand the shell features (key bindings and fuzzy completion) we provide here. They are intentionally kept simple (with minimal configuration knobs)

Well, while on one side I can see your points (especially keeping fzf itself clean and simple and also not having to add functionality for numerous shells at the same time), in practise this poses IMO several problems.

Generally it's very bad to distribute software outside some form of distribution or other package management system (no security support, no systematic way for regular updates, dependency mess, breaking any form up centralised management (checking for updates, automatic deplyoment, etc. pp.). While this seems to have become a new fashion, one should still try to avoid it as best as one can.
If any such functionality is now put in 3rd party repos, there's in practise little chance that the respective code gets ever packaged, especially if there are many (like a separate fzf-* for every little thing). By keeping at least generic stuff in the main fzf project, platform that packages this, should at least sooner or later get the new functionality.
In practise, such 3rd party repos are also rather prone to get outdated or have other issues that are never fixed. This is even more likely when the 3rd party project would expand features that are already in the main fzf. Consider there was now another project that extends host completion by overwriting _fzf_host_completion() it's all too likely that sooner or later you make a change that doesn't work with their state anymore.

I think bash-completion might serve as a good model:

it tries to include base functionality of generic use
it ships the "plugins" for major programs or such of generic use, if the respective project doesn't ship the completions by itself
it still easily possible for some actual project (like git or) so to maintain and ship its own set of completions

Another possible way, that would help you to keep the fzf repo clean from all to many shell stuff, could be that you generally outsource all shell integration into separate fzf-<shell> repos.
I guess these would have at least some chance to get packaged, and there would no longer be so many possible areas of conflict between shell integration code shipped by the main fzf package and such from 3rd party project, since, that would be again in one repo.
That would be again similar to how things are handled in the bash world:

there's bash itself, just the shell - which would be fzf - just the fuzzy finder
bash-completion, generic completion infrastructure and support for some major programs - which would be fzf-bash, fzf-zsh, etc.
downstream projects like git shipping their own completions - which would be 3rd party fzf-something projects

What's your opinion on that?
Any beyond that, would you still accept code for the actual feature idea itself?

btw: (Your) current implementation of __fzf_list_hosts() also gives errors, if ~/.ssh/known_hosts or /etc/hosts doesn't exist. But TBH, I'd very much prefer to go the "use _known_hosts_real() way" proposed here, rather than fixing this. ;-)
So would be happy if you'd agree to get something like that merged.


PlasticGui.EventTracking is causing build to fail

Specifically for linux builds and webgl builds:
Library\PackageCache\com.hdyar.osu-importer@6a84470af9\Runtime\Scripts\SupportTypes\osu-event-types\OSUEvent.cs(4,7): error CS0246: The type or namespace name 'PlasticGui' could not be found (are you missing a using directive or an assembly reference?)

Fixed with https://github.com/hunterdyar/osu-importer-unity/commit/faac92baa8e67bbeeb96c8f636813a30fd159c94


WIP: Broke FileUtils to simple warpers and use as context managers

Issue #235

Coverage decreased (-0.09%) to 94.275% when pulling 9b907f984e9ada8fb9d673016d31e2d601b52f5c on mimi1vx:ctx_managers into 646005e74c55278b89e8def60152cf27c92b0255 on openSUSE:master.


Expose currentUser

Expose the user object currently logged in with Azure authentication.
Updated README.md with info and example
This PR may address issue #23

Thanks!


Add ExceptionResponder

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues
-

See https://github.com/yiisoft/yii-web/pull/332

👍

👍


fix: MD006/ul-start-left

Consider starting bulleted lists at the beginning of the line

Maybe I have misunderstood the rule MD006/ul-start-left, but looking at the beginning of your changes, it looks like the intention is to remove white space before any bullet list points. Further down, at https://github.com/MicrosoftDocs/windows-itpro-docs/pull/4630/files#diff-6db9d1a344f92fbadf8af29c51da8b32, it looks like you have added extra space in front of the bullet list points. Doesn't that go against the rule you refer to? Or have I misunderstood the format?

It does flag lists that aren't left aligned, but this has the side-effect of highlighting list items that aren't indented enough under things like ordered list to render correctly

Very well. I trust your changes are well founded and that you have verified the results. The rich text diff does not show any changes, so I presume all is well here.

Most of them should show no diffs (when leading space was removed), but it will for the case where the ordered list was re-indented like in that one file you linked to

@AndreaBarr The PR that introduces Markdown formatting fixes is available for review. Thanks.

@AndreaBarr Following up for updates.

@nschonni please resolve the conflicts for merging.

@AndreaBarr rebased and resolve the conflicts


Consider ways to maintain a method-chain when transitioning between pages

Something like

                context.Page<Homepage>()
                       .SearchFor("Game of thrones")
                       .ArriveAt<SearchResultsPage>()
                       .SearchResultSomething();

Right, got it.
Now supporting methods on page object models that transition to other page objects.
The syntax looks like this:
        [Uri("/")]
        public class Homepage
        {
            public virtual RemoteWebDriver Driver { get; set; }

            [Id("twotabsearchtextbox")]
            public virtual IWebElement SearchBox { get; set; }

            [CssSelector("nav-searchbar")]
            public virtual IWebElement SearchForm { get; set; }

            public virtual SearchResultsPage SearchFor(string thing)
            {
                SearchBox.Click();
                Driver.Keyboard.SendKeys(thing);
                SearchForm.Submit();

                return Arrives.At<SearchResultsPage>();
            }
        }

You need to use the Arrives.At helper to transition in your page objects. This means we can support method chaining across page boundaries.

Now on nuget - docs updated.


chore: bump barretenberg dependency to v0.3.6

Description

Problem*

Resolves
Summary*
This PR gets us onto a proper tag of barretenberg so we can start pulling down build artifacts rather than building ourselves.
Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

v0.3.6 is a version using master, shouldn't this fail since we are still on the forked branch in barretenberg?

Oof, we're still on a fork? :face_exhaling:

Yeah this is pending the dynamic backend stuff at the moment


GUVNOR-2173 - Change Impact: Modifications for use in for drools-wb

This is a small method that's useful in a number of the drools-wb indexers.
@manstis Would you mind reviewing?

@mrietveld other than code formatting, first parameter being non-final (the others are) and lack of tests it looks fine.. I assume they'll be a test or two coming? (You can mock the hell out of it... so it'll be simply).

@manstis Thanks for harping about tests -- I'll add one and clean up the code and let you know when I've done that.


Move is_dense() from Fetch to FetchState

The is_dense trait method should be cacheable, as the dense vs sparse nature of the component storage should be nearly static.
We can do so by moving this function from Fetch to FetchState.
As part of this change:

Implement a handy StorageType::is_dense() -> bool method
Benchmark the performance changes use ecs_bench_suite.

This is now const. https://docs.rs/bevy/latest/bevy/ecs/query/trait.Fetch.html#associatedconstant.IS_DENSE


⚠ Fetchers has degraded performance

In acc905f, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 441 ms

Resolved: Fetchers performance has improved in 6711eb0.


[EasyCodingStandard] loading two config files

(Related to https://github.com/nette/application/pull/182/files)
ECS is installed via composer create-project and runned by:
temp/nette-coding-standard/vendor/symplify/easy-coding-standard/bin/ecs check src tests --configuration temp/nette-coding-standard/easy-coding-standard.neon

So there is common easy-coding-standard.neon (in fact, there will be versions for PHP 7.1, PHP 7.0 and PHP 5.6) and in some packages will be local coding-standard.neon with parameter skip etc.
It would be ideal if I could load both configuration files in the command line.

I got confused by example that loads only 1 file.
Do you mean something like this?
temp/nette-coding-standard/vendor/symplify/easy-coding-standard/bin/ecs check src tests --configuration php7-checkers.neon php71-checkers.

Closing as "too hard to implements propperly", based on https://github.com/Symplify/Symplify/pull/219#issuecomment-312622155


TO Postinstall: AttributeError: module 'hashlib' has no attribute 'scrypt'

This Bug Report affects these Traffic Control components:

Traffic Ops - Postinstall

Current behavior:

Python Postinstall fails on CentOS 7 if Python 3 is installed (but succeeds if Python 3 is not installed, because it will use Python 2).
Originally reported by @rsidhaarth in the first draft of #6236.
Expected behavior:

the Python Postinstall script should succeed on CentOS 7, with or without Python 3.
Steps to reproduce:

<<'DOCKER_COMMANDS' docker run --rm -iv$(pwd):/opt -w/opt centos:7 bash
yum -y install python3 &&
<<'PYTHON_COMMANDS' PYTHONPATH=traffic_ops/install/bin python3
import _postinstall
print(_postinstall.hash_pass('mysecretpassword'))
PYTHON_COMMANDS
DOCKER_COMMANDS

Output:
[...]
Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
  File "/opt/traffic_ops/install/bin/_postinstall.py", line 433, in hash_pass
    hashed = hashlib.scrypt(passwd.encode(), salt=salt, n=n, r=r_val, p=p_val, dklen=dklen)
AttributeError: module 'hashlib' has no attribute 'scrypt'

You're right that the opportunity to include hashlib.scrypt() exists as of Python 3.6, but that doesn't mean that the compiled Python version necessarily includes it. As you can see from the Python 3.6.8 source, it won't include hashlib.scrypt() if the OpenSSL version is under 1.1 (CentOS 7 uses OpenSSL 1.0.2k + security patches):
#if (OPENSSL_VERSION_NUMBER > 0x10100000L && !defined(OPENSSL_NO_SCRYPT) && !defined(LIBRESSL_VERSION_NUMBER))

PyDoc_STRVAR(_hashlib_scrypt__doc__,
"scrypt($module, /, password, *, salt=None, n=None, r=None, p=None,\n"

Because Python 3.6 does not assume that hashlib.scrypt() will exist, we shouldn't, either.

Oh, okay, that makes more sense, since I was pretty sure it was Python 3.6 since the last time I looked - which was a while ago so it would be weird for the version to have gone down in the meantime.
Honestly supporting unmaintained versions of OpenSSL doesn't sound any better (potential security problem?), and OpenSSL versions prior to 1.1.1 are already unmaintained. It does seem based on that CPP directive that a vendor could specifically disable SCRYPT anyway, so that's enough to warrant this PR IMO, but if CentOS7 only uses encryption libraries that aren't maintained and recieving security fixes then we may need to seriously consider dropping support for it. Not sure offhand what the entire footprint of implications of that is, but at the least it seems difficult to trust the security of the user passwords created and inserted by postinstall.


Scenario results are reported incorrectly when an Around hook does not call its block

I'm working with a set of tests that can be run in parallel on multiple client machines, and I have a tag that can conditionally skip a scenario (represented by the @die tag here) based on whether the environment is appropriate for that particular test. The tag is implemented as an Around hook that only calls block.call when the client's environment (e.g., OS version) matches the Scenario's expectation. There are a couple of cucumber bugs I've noticed: the counts at the end are off, and the previous scenario is reported incorrectly if it uses any After hook. An example:
Feature file:
Feature: Some feature
  Scenario: A scenario before a dead scenario
    Given something happens

  @die
  Scenario: The first dead scenario
    Given something happens

  @after
  Scenario: A scenario with an after hook before a dead scenario
    Given something happens

  @die
  Scenario: Another dead scenario
    Given something happens

Step file:
Given(/^(.*)$/) do |step_text|
  puts step_text
end

After('@after') do
  puts 'After hook'
end

Around('@die') do |scenario, block|
  puts 'Not running this scenario!'
end

Output:
Feature: Some feature

  Scenario: A scenario before a dead scenario # test.feature:2
    Given something happens                   # test.rb:1
      something happens

  @after
  Scenario: A scenario with an after hook before a dead scenario # test.feature:10
      Not running this scenario!
    Given something happens                                      # test.rb:1
      something happens
      After hook

  @after
  Scenario: A scenario with an after hook before a dead scenario # test.feature:10
 Not running this scenario!

4 scenarios (2 passed)
2 steps (2 passed)
0m0.046s

Note that the scenario "A scenario with an after hook before a dead scenario" is reported twice, but the second time it has no steps. Also notice that the output of the Around hook is attached to the wrong scenario each time. In addition, observe that it correctly reports 4 scenarios, but the breakdown only includes the 2 passing scenarios and ignores the skipped ones, where I would expect them to be marked "skipped."
The output is similar in other formats (e.g., -f json).

Hi, thanks for reporting this.
While we work on your issue, I suggest you work around it for the time being by using a Before hook that calls skip_this_scenario under the conditions where you don't want to run it. You should get the reporting you expect.

Thank you, I've already modified the hook very similarly to your suggestion, though I wasn't aware of the specific skip_this_scenario function.

I'm going to close this because:

it's probably really complex to fix
it should get resolved naturally as the formatters are re-written
there's a workaround.


Document docker swarm version

docs: Document docker version for swarm
Docker swarm support currently requires a back-level version of Docker
so add details to the limitations doc.

Fixes #781.

@mcastelino, @GabyCT, @grahamwhaley, @iphutch  - please can you review?

lgtm

Hi @iphutch - good catch! Branch updated.

Hi @iphutch. Thanks fixed.
Note that I have had to add some slightly odd refs into the TOC to make this work for github (not just space -> dash: it seems the github md->xhtml code doesn't work exactly as we thought, so we'll have to keep an eye on this).


fix: home server card

Closes #87

#89 might be the actual fix for this - now that a new 0.22.0 version is out, I can see the Server "Usage" section from the menu that this card was meant for

Closing in favor of #89


Impl Error for SendError and TrySendError

This would be more consistent and would make it easier to wrap them with custom error types.

Thanks for the report. The channel in Mio has been deprecated. Given this, I'm not going to make any further changes to it. Someone else may decide to take the channel and maintain it in a separate crate, at which point they can decide if they want to implement Error or not.


Update librsync-go to v0.8.5, circbuf to v0.1.3

Notable improvements these new versions bring:

Optimized code path for generating deltas with blocks that are power-of-two-sized.
Avoid allocating unbounded amounts of memory when the target differs completely from the source.
Several bugfixes in edge cases that shall not affect balenaEngine.

- What I did
- How I did it
- How to verify it
- Description for the changelog

LGTM


docs: correct code example (modular-forms integration)

Overview

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests / types / typos

Description
In the code snippet that shows how to use onSubmit$ handler, the handler function is typed incorrect. Handler function's type must have been enclosed in QRL or, as has been done in this particular case, passed as a type parameter to the $ function.
Checklist:

[x] I have made corresponding changes to the documentation

Thank you for fixing this!

approved by @fabian-hiller 🙏
thanks a lot for adjusting this one @uceumice 🙏


feat: foxy multi-account

⚠  Blocked by https://github.com/shapeshift/lib/pull/1016
Description
This adds multi account support for FOXy:

Overview
Deposit
Withdraw

Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes https://github.com/shapeshift/web/issues/2611

Risk
FOXy modal should be fully regression tested - in theory, this brings no changes for now and uses programmatic accountNumber vs. hardcoded accountNumber, but this requires a breaking(ish) change in the lib PR.
Testing

With the MultiAccounts flag on, the account dropdown should be present in FOXy opportunity Overview, Deposit, and Withdraw steps
With the MultiAccounts flag off, there should be no user-facing changes nor regressions with the current flow

Engineering

Refer to the top-level testing notes
console.log or debug accountNumber props in components to ensure the right account number is passed down

Operations

Refer to the top-level testing notes
Regression test the whole Yearn modal flow, including making deposit and withdraws

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2642

PR #2644

PR #2651

PR #2652
PR #2661

PR #2680   👈
PR #2669

This comment was auto-generated by Graphite.


is java_package option's value from .proto file ignored in ScriptHost.withTypes call?

Hi,
I'm trying to use protobuf messages in the cel-java and my code fails with an exception
"Caused by: org.projectnessie.cel.tools.ScriptCreateException: check failed: ERROR: :1:9: [internal] unexpected failed resolution of 'com.xyz.Customer'".
I looked at the code and I think it ignores the value of java_package option in the ScriptHost.withTypes method.
Please let me know if you want me to fix this issue.
Thanks!

Sure, feel free!


[Offline area] Refactor code related to loading local/remote tile overlays

Fixes https://github.com/google/ground-android/issues/1790

Things done:

Convert TileSource enum to sealed data class
Simplify OfflineAreaRepository
Replace List<TileSource> with TileSource when adding overlay for simplicity. If we need to support multiple tile sources, then we can change it back later.

This is a pure refactor and doesn't change app behavior.
Verified for regressions by testing that remote mog overlay is loaded in offline area screen and locally downloaded tiles are loaded on home screen.

@gino-m  PTAL?

/gcbrun


Adds hack for doc members fix

Fixed

Provides a workaround for JSDoc issue https://github.com/jsdoc3/jsdoc/issues/1301 causing member properties to be omitted.

Preview:
http://pixijs.download/fix-docs/docs/index.html
Example:

DisplayObject w/o hack
DisplayObject w/ hack

Addresses https://github.com/pixijs/pixi.js/issues/3416


Add a --force-install command to tkn pac bootstrap to update to latest

even if it's alreayd installed on the cluster,

this is done right?
https://github.com/openshift-pipelines/pipelines-as-code/blob/34bc00f2baaabfaa8e3f9d04c22908a79fa8a818/pkg/cmd/tknpac/bootstrap/bootstrap.go#L143

yes can be closed!


Can't install on Linux Mint 19.2

Hello dear pyansys team.
I've just used bash command: python3.7 -m pip install pyansys on Linux Mint 19.2 in order to install pyansys package. On the course of installation I received the following messages:

psutil/_psutil_common.c:9:10: fatal error: Python.h: No such file or directory
#include <Python.h>
^~~~~~~~~~
compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

Failed building wheel for psutil

psutil/_psutil_common.c:9:10: fatal error: Python.h: No such file or directory
#include <Python.h>
^~~~~~~~~~
compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

Command "/usr/bin/python3.7 -u -c "import setuptools, tokenize;file='/tmp/pip-build-otkmj7c7/psutil/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-hx9e23sf-record/install-record.txt --single-version-externally-managed --compile --user --prefix=" failed with error code 1 in /tmp/pip-build-otkmj7c7/psutil/
And ultimately, pyansys hasn't bee installed.
When I was installing pyansys on my windows machine - I didn't encounter any errors and it has been working fine.
I was wondering if you could fix the issue for linux please.
Thank you in advance.
Ivan

Hello,
Looks like you're missing python-dev libraries on your linux install.  Run:
sudo apt-get install python3-dev

I'm surprised you're getting this bug as pip should be installing the wheel from PyPi.

That worked, thank you very much. The exact command I used is:
sudo apt-get install python3.7-dev


Why the image_AUROC of ganomaly is changed so much every time

when i use this command  python tools/train.py --model ganomaly --config src/anomalib/models/ganomaly/config.yaml to training, the image_AUROC is changed so much every time. sometimes is 0.51... sometimes 0.23... I noticed that the random seed is set to 42. So, shouldn't the result be the same every time?

and can you please to explian why the anomalous score is over 100% for both good images and broken images

I have the same problem about the score over 100 and wrong labels. Have you found the problem?

Hi,
I was doing some tests on the MVTEC dataset with ganomaly and had similar anomaly scores of more than 100% (sometimes up to 1500-ish scores).
Searching into the code, I found that the Ganomaly model compute some form of normalization based on all the predictions.
Basically, in the test_step function of the model, the min and max anomaly values of the current batch are compared with previous min and max from other batches and stored.
Then, in the test_epoch_end function, the normalization is performed with regards to these min and max.
Now, the problem is that the callback handling visualization is called BEFORE the normalization happens (test_epoch_end), causing the display of un-normalized (and multiplied by 100) values.
The quick fix I tried was to re-use the min and max values from the validation step and to normalize in the test_step function. Then I got values within the 0-100% range.
    def test_step(self, batch: dict[str, str | Tensor], batch_idx: int, *args, **kwargs) -> STEP_OUTPUT:
        """Update min and max scores from the current step."""
        super().test_step(batch, batch_idx)
        logger.info(f"{self.max_scores=}/{self.min_scores=}")
        # self.max_scores = max(self.max_scores, torch.max(batch["pred_scores"]))
        # self.min_scores = min(self.min_scores, torch.min(batch["pred_scores"]))
        batch["pred_scores"] = self._normalize(batch["pred_scores"])
        return batch

    def test_epoch_end(self, outputs: EPOCH_OUTPUT) -> EPOCH_OUTPUT:
        """Normalize outputs based on min/max values."""
        # for prediction in outputs:
        #     prediction["pred_scores"] = self._normalize(prediction["pred_scores"])
        super().test_epoch_end(outputs)
        return outputs

That being said, this is 100% not properly tested, the prediction are still all labeled as anomalies and the network has a very low AUROC score (~0.3). I'll look more into it tomorrow if I have the time.

@D3lt4lph4 have you solve this problem？


zoomDelta misbehaving in FF

It appears on FF, the zoomDelta is 0.1 not 10%
behaves as expected in IE, Chrome
Ideally, it would be nice to set the zoomDelta as a potential option ... I don't know what the browser deltas are doing ...

wheel: function (event) {
var e = event.originalEvent,
msDeltaY = 117.25, // IE
mozDelatY = 5, // Firefox
webkitDelatY =<PHONE_NUMBER>9414062, // Chrome, Opera
zoomDelta = 0.1, // 10%
delta;
  if (this.disabled) {
    return;
  }

I had improved this, you may try the v0.7.8 now.


Nested route rendering

export default
class AppRouter extends React.Component {
	render() {
		return (
			<Router history={history}>
				<Route path="/" component={R1}>
					<Route path="/r2" component={R2}>
						<Route path="/r3" component={R3}>
							<Route path="/r4" component={R4}/>
						</Route>
					</Route>
				</Route>
			</Router>
		);
	}
}

v1.0.0 beta3

User navigates to /r2/r3
*(R1 is rendered then R2 and then R3 which is fine.)

User then navigates to /r2/r3/r4.
*(I've noticed that the whole chain of routes is rendered again! not just R3, R4 , although they are the    only components that are about to be changed.
Meaning that the router will call render method for R1 & R2 components as well..)

Is this the desired behavior?

Is this the desired behavior?

Yeah, if a route component (and all its parents) is the same between pages, it'll be re-rendered. You need this if your render output depends on route params, pathname or other active routes.
Think of the router component as a React component that first renders <R1><R2><R3 /></R2></R1> and then changes its state in response to a URL change to render <R1><R2><R3><R4 /></R3></R2></R1>. The resulting behavior is the same.

First of all thank you for your answer :).
I understand what you are saying, but I must say I still don't see it as the best approach on this case.
I'll explain:
Suppose I'm on a given route A/B/C/123
lets say A needs to know about the url param change down below in the route chain.
if the route changes to A/B/C/12345,
I can't see how re-render by default A component is the solution, rather then agreeing on a function (callback) that can be set up on a component,
the callback would then be fired(if exists) on A & B & C(not necessarily in that order :P) and A will change it's state and if needed be re-rendered.
class A extends React.Component{
//will be fired when a route involving A as a component is changed.
    onRouteChanged(): {
        if(...) {
            this.setState(..)
        }
    }
}

***I might be totally wrong here, I did just start working with react about a month ago, but for the chance my thoughts about this matter are making sense, here they are :)

This is the entire selling point of react, "always rerender" when data changes and quit worrying about render getting called.


csauto: Add version 2.0.9

Add CSAuto on version 2.0.9 (gh repo)

[x] I have read the Contributing Guide.

/verify
Sorry, I kinda forgot about it


NLA collection support

Changes to provide collection support for NLA, closes #22

These changes specifically allow Hypercane to process NLA collections in Trove.


feat(tests-integration): add a naive region migration integration test

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?

Fix heartbeat handler ignore upgrade candidate instruction
Fix handler did not inject wal options
Expose RegionMigrationProcedureTask
Add a naive region migration integration test

Checklist

[x]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.
[x]  This PR does not require documentation updates.

Refer to a related PR or issue link (optional)
#2700

@killme2008 PTAL


No way to unhide hidden stock metadata columns

Describe the bug
If a "stock" metadata column (such as "Created" or "Modified") is hidden using the "Hide" feature, there is no way to unhide it using the UI.
To Reproduce

Click Modified column header
Select "Hide"
Observe that there is no way to reverse no. 2

Expected behavior
When adding a new column, in Columns Menu, switches to show/hide stock metadata columns should be displayed
Screenshots
N/A
Desktop (please complete the following information):

OS: All
Version 2.2.2

added to the roadmap, thanks for the report =)

Starting with this one. While is fixed, you can change the markdown property of that metadata column
isHidden: true => isHidden: false

easy one: Fixed modifying .filter(c => !c.isMetadata) per .filter(c => !c.skipPersist)


target.py: cope with non-root users and with non-standard home directories

proposed fix for https://github.com/ARM-software/devlib/issues/317

Would it make sense to keep the sub-directory devlib-target inside of the home directory?

Would it make sense to keep the sub-directory devlib-target inside of the home directory?

possibly - but i don't see how that has to do with this PR? from a quick glance, it seems, {/tmp/,<external_storage>,</mnt/stateful_partition>}/devlib-target is used as working_directory, if the home is not set.
am i misunderstanding what you're saying?

Sorry, I just meant that the previous behaviour was if the working directory was not explicitly set, it would use the devlib-target inside of the default home directory, however now the base home directory would be used, meaning anything that devlib deploys would be mixed along with any other files in the home directory rather then keeping them contained in a subdirectory.
So I was suggesting maybe doing something like:
self.working_directory = self.path.join(self.execute("pwd").strip() , 'devlib-target')

So I was suggesting maybe doing something like:
self.working_directory = self.path.join(self.execute("pwd").strip() , 'devlib-target')

i think you're absolutely right. sorry.


PowerRegisterSuspendResumeNotification failed with errno= 5 in azure function

In AzureRM we are creating a Function App for right_st.
This function is run as a HttpTrigger
and executing this:
using namespace System.Net
# Input bindings are passed in via param block.
param($Request, $TriggerMetadata)
# Write to the Azure Functions log stream.
Write-Host "PowerShell HTTP trigger function processed a request."
 & "D:\home\site\wwwroot\HttpTrigger2\right_st.exe"

We get the following error:  PowerRegisterSuspendResumeNotification failure
Here are the console logs from Azure Function App:
2020-03-17T18:39:21  Welcome, you are now connected to log-streaming service. The default timeout is 2 hours. Change the timeout with the App Setting SCM_LOGSTREAM_TIMEOUT (in seconds).
2020-03-17T18:39:29.439 [Information] Executing 'Functions.HttpTrigger2' (Reason='This function was programmatically called via the host APIs.', Id=60f154ce-6a13-4ec8-8b06-87688dbceba0)
2020-03-17T18:39:29.446 [Information] INFORMATION: PowerShell HTTP trigger function processed a request.
2020-03-17T18:39:30.557 [Error] ERROR: fatal error: PowerRegisterSuspendResumeNotification failure
Script stack trace:
at , D:\home\site\wwwroot\HttpTrigger2\run.ps1: line 8
System.Management.Automation.RemoteException: fatal error: PowerRegisterSuspendResumeNotification failure
Microsoft.Azure.WebJobs.Script.Workers.Rpc.RpcException : Result: ERROR: fatal error: PowerRegisterSuspendResumeNotification failure
Script stack trace:
at , D:\home\site\wwwroot\HttpTrigger2\run.ps1: line 8
System.Management.Automation.RemoteException: fatal error: PowerRegisterSuspendResumeNotification failure
Note:  If we run the following it works( This is using a 7za.exe file that is downloaded from the internet and uploaded to the function):
# Input bindings are passed in via param block.
param($Request, $TriggerMetadata)
# Write to the Azure Functions log stream.
Write-Host "PowerShell HTTP trigger function processed a request."
& "D:\home\site\wwwroot\HttpTrigger2\7za.exe"
Resolution:
Cherry picked:  https://go-review.googlesource.com/c/go/+/219657/

https://github.com/golang/go/issues/37149

Probably good to follow the one for Go 1.14.x as well (golang/go#37699); I have subscribed to both of them so I should get an email when something happens.


Slack notifications for auto approved requests just say "Movie Request Automatically Approved"

Description
Pretty much the title, but I'm setting up Jellyseerr to try it out and I use Mattermost which is compatible with Slack notifications.
I've setup a webhook and have it added and it can send the test and notifications to the server. However, all it says "Movie Request Automatically Approved".
I use auto approval, but I still want to know what was requested so I can keep an eye on it in case it isn't automatically retrieved for whatever reason. Is this intended for the Slack notifications to be vague?
Version
1.9.2
Steps to Reproduce

Create webhook in Mattermost/Slack
Enable Slack notifications in Jellyseer
Add the webhook link, select the options for "Request Automatically Approved" (There is no way to customize what this notification would look like.)
Request content
See notification just says "Movie Request Automatically Approved"

Screenshots

Logs
No response
Platform
desktop
Device
PC
Operating System
Ubuntu - Docker
Browser
Edge
Additional Context
Jellyseerr is installed through Docker on native Ubuntu 24.04, not using WSL. I'm configuring it through Windows 11 on a separate machine.
Code of Conduct

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

Are you using the slack agent or the webhook agent?

Are you using the slack agent or the webhook agent?

Slack agent. Webhook doesn't work with Mattermost, but in other services, including Radarr, Sonarr, Emby, etc, the Slack notification works with Mattermost.


Cannot load Packwiz modpack in prism

Packwiz: AUR packwiz-git r283.7de26ec-1
Prism: PrismLauncher 6.3
when I expory my pack with packwiz mr export and try to run it in prism, no mods are downloaded and it hangs untill either I abort it or it crashes with
Exception caught from launcher
java.lang.ClassNotFoundException: org.quiltmc.loader.impl.launch.knot.KnotClient
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520)
	at org.prismlauncher.utils.ReflectionUtils.findMainMethod(ReflectionUtils.java:151)
	at org.prismlauncher.launcher.impl.StandardLauncher.launch(StandardLauncher.java:87)
	at org.prismlauncher.EntryPoint.listen(EntryPoint.java:126)
	at org.prismlauncher.EntryPoint.main(EntryPoint.java:71)
Exiting with ERROR
Process exited with code 2.

modrinth.index.json.txt
is the index.json generated

tried to recreate the pack using fabric (which means I had to drop a mod)
prism still did not download the mods from the mrpack

quilt does not like tor proxy


test_small_test()

Hi, there is no issue for this PR. Please open an issue first to make it easier to understand what you are working on and match the issue with a PR. Thank you!

This PR was related to both:

#159 which should not have been closed because we only have one test that always passes.
#226 which is why the test passes on Python 2 but not on Python 3.  See Travis logs

Tests need to be thought about differently than production code.  It is OK for tests to fail.  It does not mean that the PRs are useless or that they need to be closed.  Instead, tests failing usually means that there is some discovery going on that will make the product more robust and provably more robust in the future.   I hope that we can find ways to add more tests to the project.


Delete Expired Subscriptions

Description
This enables an automatic deletion of expired subscritions.
Closes #12
subAutoRemove can read the subscriptions and delete those that are expired (created a week ago).

[1 ] New feature (non-breaking change which adds functionality)

Let's move this to an API endpoint so a CRON service can call it.


[ISSUE #2171] fix the spell error in ACL

What is the purpose of the change
Fix #2171
Brief changelog
fix the spell error
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.1%) to 51.196% when pulling 0b669544621f6ac5961b06c940b0ba42482e7f5d on fantow:fixSpellError#2171 into 9f95a972e10e0681bc3f2d00e9957aa212e897b5 on apache:master.

Sry,I want to change the submit branch.I forget change it.


Support for stylus conditional ?= assignments

I am trying to convert a stylus file to scss:
$myColor ?= #f00;

.color {
   color: $myColor;
}

Output is :
.color  {
  color: $myColor;
}

I am not a SCSS expert, but I guess the expected output is:
$myColor: #f00 !default;
.color  {
  color: $myColor;
}

@laure-at-irun I am sorry that I am on vacation, and I will deal with this issue when the holidays are over.

@laure-at-irun please upgrade stylus-converter to 0.6.7

Thanks for the quick fix!


memory leak when using List<T>

Goals
I have memory leak when use List in realm 2.5.0
class Horario: Object {    dynamic var Horario:String = "" }
class Dia: Object{
dynamic var Dia:String = ""
dynamic var Ordem:Int = 0
var horario = List<Horario>()

override static func primaryKey() -> String? {
    return "Dia"
}

Expected Results
I have use weak or unowned reference for list properties
Actual Results
When defined weak var horario = List<Horario>() i can't set value for list, always return nil after set value
Version of Realm and Tooling

Realm version: 2.5.0
Xcode version: 8.3
iOS/OSX version: 10.2
Dependency manager + version: pod 1.1.1

Don't use weak var or var for lists.


Please provide this list in RPZ format

This list is very useful; many thanks!
It would be even more useful if it were in RPZ format.  This (horrible) awk hack does exactly that:
awk '{ if ($0 !~ /^#/) print $0 " CNAME ." ; else { sub(/#/, ";"); print $0 }}' combined_disguised_trackers_justdomains.txt

Please consider providing an RPZ version of this list in addition to the AdBlock formatted version.

@ppaeps sed will do this in a instant on your end. No need to bother the @AdguardTeam fellas for that 😉 :
sed -i '/^[^#]/s/$/ CNAME \./g combined_disguised_trackers_justdomains.txt
Result(partial sample):
# Company name: A8.net
#
# a8.net disguised trackers
#
a8.01cloud.jp CNAME .
a8.cyclemarket.jp CNAME .
a8.denwa-hikari.com CNAME .
a8.eonet.jp CNAME .
a8.haptic.co.jp CNAME .

👍

Hah!  You are not wrong.  sed would be easier than awk.  I put together that one-liner from a larger (more gruesome) awk script that converts and combines a bunch of blocklists into an RPZ for me. :)
Since the nice folks of @AdguardTeam are already providing a list in AdBlock format, it shouldn't be too much of a hardship for them to add the additional one-liner to produce RPZ too.  More formats, more better.  The easier we can make it for everyone to block these cretins, the less incentivised more cretins will be to join the party and the slower these lists grow.

@ppaeps as i'm really not familar with RPZ format i see in your script you're substituting # with ;.
Is that how comments are marked there?
I agree with you about your request here,more formats are allways better 👍

An example in the RFC @ https://tools.ietf.org/id/draft-vixie-dnsop-dns-rpz-00.html#rfc.appendix.A does show such commenting.

Thanks @TPS 👍  For some reason i couldn't find it. Anyway,here is reworked final sed command(in case is needed):
sed -i '/^#$/d; s/^#/;/g; /^[^;]/s/$/ CNAME \./g' combined_disguised_trackers_justdomains.txt
Result(partial sample):
; Title: AdGuard CNAME disguised trackers list
; Description: The list of trackers that disguise the real trackers by using CNAME records.
; Homepage: https://github.com/AdguardTeam/cname-trackers
; Company name: A8.net
; a8.net disguised trackers
a8.01cloud.jp CNAME .
a8.cyclemarket.jp CNAME .
a8.denwa-hikari.com CNAME .
a8.eonet.jp CNAME .
a8.haptic.co.jp CNAME .
a8.lavie-official.jp CNAME .
a8.lens-labo.com CNAME .

@dnmTX You'll want to add a ;/^$/d;$G or something similar (;$a\, ;$s/$/\'$'\n'$'/, depending on your mood) to ensure there's a newline at the end of the output file.  Some DNS servers (e.g. NSD) will cry about files that don't end with a newline.
This is all academic though.  I just submitted pull request #14 which generates the RPZ formatted files directly.  It would be great if @AdGuardTeam could merge this. 🙏

Philip @ppaeps your wish came through(congrats 👏  😄 ) so you can close the issue here if you want 👍

Hooray!  Thanks for the merge @adguard!

what diff use IN and without IN in cname
sample
a.b.c.d  CNAME .
a.b.c.d  IN CNAME .

This list is very useful; many thanks!
It would be even more useful if it were in RPZ format. This (horrible) awk hack does exactly that:
awk '{ if ($0 !~ /^#/) print $0 " CNAME ." ; else { sub(/#/, ";"); print $0 }}' combined_disguised_trackers_justdomains.txt

Please consider providing an RPZ version of this list in addition to the AdBlock formatted version.

How about this command?
awk '{print $1" IN CNAME ."}' combined_disguised_trackers_justdomains.txt  >> combined_disguised_trackers_justdomains.zone

No difference.  Check section 5.1 of RFC 1035.  "Omitted class [...] default to the last explicitly stated values".  In practice, every DNS implementation I've encountered, defaults them to IN even when they don't appear anywhere in a zone file.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/93408

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/103816

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104028


new: Create a new shim executable.

Go Miles! I'm super excited to have a shim solution that's not specific to the shell!

@rotu So far this works perfectly for primary shims (npm, node, etc), but unfortunately doesn't work for secondary shims (npx, pip, etc).
The problem is that secondary shims add --alt bin flags and before/after args into the shell file, which are then passed to proto. When relying on argv[0], we have lost this information.
The only solution I can think of is having some kind of manifest like ~/.proto/shims/manifest.json that keeps a mapping of how everything should be executed.

Agreed, but I think secondary shims are a problem from the beginning. That's what I meant in my comment:

To that end, proto needs a way to map the name of a bin to the plugin/tool which provides it. I realize this is ambiguous (e.g. two plugins could declare a program with the same name) and proto needs to handle this either implicitly or by saving desired shims in the .prototools config.

I think the easiest thing is to add a section to the yaml, which maps the name to a $PROTO_HOME-relative path:
[shims]
npm = "tools/npm/10.2.4/bin/npm"

This even provides the ability to shim to the real bundled npm instead of one provided by node_depman_plugin:
[shims]
npm = "tools/node/20.10.0/bin/npm"

If these need to float with the version, the path could instead be, e.g. tools/npm/${PROTO_NPM_VERSION}/bin/npm or use some other placeholder like proto:npm/bin/npm.

Yup exactly. Will look into ways to easily solve this. This may land in 0.25 instead of 0.24. Want to ensure it all works correctly.

This also allows you to e.g. remap node like bunx --bun does:
[shims]
node = path/to/bun

or even possibly configure arguments on a per-project basis:
[shims]
node = ["path/to/node", "--loader", "ts-node/esm"]


[Footer]: Adds White Space Below

Detailed description
It looks like the footer is currently adding a large amount of white space to the bottom of the page below it on the smallest breakpoint due to the accordion not hiding its content properly.

I think if we can just add overflow: hidden; to the .bx--accordion class then it'll fix the issue.
Steps to reproduce the issue

Go to https://ibmdotcom-react.netlify.com/iframe.html?id=dotcom-shell--default
View at sm breakpoint.
Scroll to the bottom of page.

Additional information

We should also look into seeing if this is something we should contribute back to carbon as well.

@oliviaflory @annawen1 please triage accordingly.


Mod API research

This link https://tibcosoftware.github.io/spotfire-mods/docs/using-the-api/api-docs/ provides the API documentation which will help us understand the working of the API along with it lists all the interfaces and the functions in the API. Each interface and function has a short introduction and how to use them.

In order to use import and export functions in other files, we have used the Bundle.js library, Bundle.js Website This issue has been resolved now in my main template.