2.40.0 failure on atlantis_repo_whitelist

Description
In 2.40.0 I'm getting the below error doing an init on the new module.  According to the docs, atlantis_repo_whitelist is still a valid variable.  Going back to 2.39.0 fixed this issue for me.  Is this variable going away in 2.40.0?
⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions
2.40.0

Terraform:
0.14.10
Provider(s):
Module:

Reproduction
Use v2.40.0, and try to init a project that is using the atlantis_repo_whitelist variable
Steps to reproduce the behavior:
No Workspaces
Yes I have cleared the cache
Went from 2.39.0 to 2.40.0 and the issue started
Expected behavior
A successful init
Actual behavior

  on main.tf line 137, in module "atlantis":
 137:   atlantis_repo_whitelist     = ["github.com/${var.github_organization}/*"]

An argument named "atlantis_repo_whitelist" is not expected here.

Terminal Output Screenshot(s)

Additional context
In 2.40.0 the docs still reflect that atlantis_repo_whitelist is a valid variable.  Going back to 2.39.0 works.

This is a documentation issue, change the value to atlantis_repo_allowlist to use v2.40.0 and beyond.


[WIP] Enable ERC20

This PR adds OBToken as one of the erc20 coins.

Can one of the admins verify this patch?

Coverage decreased (-0.04%) to 41.465% when pulling 516e879d50939c19d3582c424d23a6438367c220 on erc20 into fb404a2601add9260dbd9d81efcdac5c07dbd596 on master.


[RodeoFX] Fix Marker repr and str #592

Fixes: #592
This PR fixes an AttributeError when calling
repr(otio.schema.Marker()) or str(otio.schema.Marker())
Without the fix:
>>> repr(otio.schema.Marker())
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/devLocal/elabrosse/rdoenv/auto_lineup/otio/lib/python2.7/site-packages/opentimelineio/schema/marker.py", line 25, in __repr__
    repr(self.media_reference),
AttributeError: 'opentimelineio._otio.Marker' object has no attribute 'media_reference'

>>> str(otio.schema.Marker())
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/devLocal/elabrosse/rdoenv/auto_lineup/otio/lib/python2.7/site-packages/opentimelineio/schema/marker.py", line 9, in __str__
    self.media_reference,
AttributeError: 'opentimelineio._otio.Marker' object has no attribute 'media_reference'

With the fix:
>>> repr(otio.schema.Marker())
"otio.schema.Marker(name='', marked_range=otio.opentime.TimeRange(start_time=otio.opentime.RationalTime(value=0, rate=1), duration=otio.opentime.RationalTime(value=0, rate=1)), metadata={})"

>>> str(otio.schema.Marker())
'Marker("", TimeRange(RationalTime(0, 1), RationalTime(0, 1)), {})'

The otio.schema.Marker object's __repr__ and __str__ methods are referencing a 'media_reference' and a 'source_range' attribute.
however Markers do not have those attributes. They do however have a 'marked_range' attribute that is not referenced in the __repr__ / __str__

Awesome, thanks!
If you say:
Fixes: #592
instead of
Issue: #592
Then when this PR is accepted github will automatically close the issue for us, too.

Accidentally re-resolved in #740 (even though this was the original fix).


Ng-zorro-issue: The font style in modal

What problem does this feature solve?
In "nzContent   : 'Some descriptions' ",
the font-style-color is invalid！

https://angular.io/guide/component-styles#deprecated-deep--and-ng-deep


Can't  serialize dynamic object in actor

Expected Behavior
Actual Behavior

Steps to Reproduce the Problem
actor code

when I beg that Actor

habs this error
Dapr.Actors.ActorMethodInvocationException: Remote Actor Method Exception
---> Dapr.Actors.ActorInvokeException: Type '<>f__AnonymousType0`2[System.Int32,System.Int32]' cannot be serialized. Consider marking it with the DataContractAttribute attribute, and marking all of its members you want serialized with the DataMemberAttribute attribute. Alternatively, you can ensure that the type is public and has a parameterless constructor - all public members of the type will then be serialized, and no attributes will be required.
RELEASE NOTE:1.4

Actors use DataContractSerializer and inherit all of its behavior and limitations. This is not something we control inside of Dapr.
I don't suggest trying to use dynamic with actors.


Make Select-String more intelligent for non-primitive types

Summary of the new feature/enhancement
As a user,
I want to be able to easily use Select-String to find string data in formatted output,
so that I can find and process data in PowerShell more quickly.
By default, when you pass non-string data into the Select-String cmdlet, it will find matches based on the object's ToString() method results. This is great when the ToString output actually represents the data you want to match, but very often ToString results do not represent the data that you see in PowerShell, and this can result in incorrect or failed matches.
For example, consider this script:
# First, capture the date:
$date = Get-Date

# Now, get the current month in string format:
$month = $date.ToString("MMMM")

# Now, look at how the date renders in PowerShell, showing the month as a string
$date

# Now, try to match the month using Select-String. This returns nothing.
Get-Date | Select-String -Pattern $month

That script shows the current date, including the month in string format, but if you try to select that string based on the actual string month, there are no matches. Why does this happen? Because ToString() on DateTime objects returns the date time with a numeric month, not the string month.
Now let's look at a more realistic example:
# Get some services, including the Windows Update service, and filter output on the
# string "Update"
Get-Service wuauserv,bits | Select-String Update

That returns nothing. Why? Because the ToString method on service objects returns the name of service, so you can't filter output based on a partial match of a display name string this way.
Here is one more example:
Get-Process -Id $PID | Format-List * | Select-String Memory

This also returns nothing, because of the ToString method on Format cmdlet output objects returns their type name, none of which match Memory.
It is reasonable for a user to expect to be able to easily and consistently parse/filter output that is rendered in the PowerShell console, but this is not possible unless they pipe to Out-String -Stream before they then pipe that streamed result to Select-String.
Proposed technical implementation details (optional)
[EDIT]: Updated to reflect the parameter name we decided to move forward in the discussion below.
I want to make Select-String better by adding a new -ConsoleOutput switch parameter (or some better parameter name: suggestions welcome) -FromFormattedOutput switch parameter that indicates that you want to select a string based on the console output of the data that is piped into Select-String, which would automatically take care of the formatting and output of non-value and non-string types (and non-MatchInfo, but that's a special case internally for Select-String), and select string matches based on that output rather than based on the ToString method output of individual objects.
Additional details
Personally I would prefer if Select-String worked this way by default for non-value and non-string types, but that would be a breaking change at this point, so that's not an option; however, users who want it to work this way by default can use $PSDefaultParameterValues['Select-String:FromFormattedOutput'] = $true, and that will have the same result.

From examples above it is not clear that is desired output. Can we get it with a workaround like Out-String or anything like?

I both love the idea and agree that it should have been the default behavior.
@iSazonov, I think of it as the equivalent of piping to Out-String -Stream first, and then performing the usual line-by-line search.
In fact, in my profile I have the following simple function:
Set-Alias slsd Select-DisplayString
function Select-DisplayString
{
  # Note: Since we want to use $Input for simplicity, we canNOT
  #       make this an advanced function.
  param([string[]] $Pattern)

  if ($args.Count) { Throw "Unexpected arguments specified: $args" }

  if ($Pattern) {
    $input | Out-String -Stream | Select-String -Pattern $Pattern
  }
  else {
    $input | Out-String -Stream
  }

}

Example use, in a directory that has files / directories that are symlinks, such as / on macOS:
PS> Get-ChildItem | Select-DisplayString ^l
l-r--            7/2/2012  3:31 PM                User Guides And Information -> /Library/Documentation/User Guides And Information.localized
l-r--          12/20/2012  5:55 PM                User Information -> /Library/Documentation/User Information.localized

That is, the for-display output of Get-ChildItem was filtered by the first character on each line, which is the Mode column, whose first character being l indicates a symlink.

I wonder if we could do a simple check... Something like:
string objectString = LanguagePrimitives.ConvertTo<string>(InputObject);
if (objectString == InputObject.BaseObject.GetType().Fullname)
{
    // Use out-string on InputObject
}
else
{
    // Use objectString
}

That way we only have the object being resolved this way if there are no other conversion paths available.

To get straight to bike-shedding regarding what to name the new switch:
We already have switches that modify output behavior with prefix As, such as -AsHashtable and -AsString for Group-Object.
As for input treatment, the only instance I could find that somewhat fits is Copy-Item's -FromSession
From as the prefix strikes me as reusable, so  perhaps -FromDisplay [representation] or -FromDisplayOutput.
In any event, incorporating the word display strikes me as preferable to console.

@vexx32: That test isn't sufficient, I'm afraid, because there are cases with different .ToString() behavior that is still distinct from PowerShell's output formatting; e.g., on Windows:
PSonWin> (Get-Process)[0].ToString()
System.Diagnostics.Process (ApplicationFrameHost)

However, I also think that (perhaps as an experimental feature), Select-String could (should?) attempt to detect when its InputObjects are being converted into useless data and attempt to automatically compensate, using the behaviour that can also be forced with a switch.

Maybe. That might just be another source for confusion though since the way an object renders as string is 100% dependent on how the author decided to implement it, and there's really no model for consistency there.
I'd just like to see it work this way by default. Select-String is often associated with grep, but users of grep expect to select information that they would see in a console if they were to output it there, not information that comes from a method and varies from object to object. If you're getting into object methods or properties, there are better commands for that.

That's a good point... Yeah, I can't really see any reason I'd pipe complex objects into Select-String without Out-String at present. 🤔

Note to self: when implementing this, consider #8963 (i.e. What happens if you're selecting text that is formatted and possibly colored/highlighted already -- will the highlighting still work properly and provide a good experience for users)?

Seems kind of wordy... hmm. What do you think of something like -FromFormat?

My original thought was that display was a less formal term to guide those potentially not even aware of PowerShell's output formatting system.
However, format to refer to the formatting system is technically more accurate, so I like the idea, and it would also give us as nicely alliterative alias name, -ff.
I'm still tempted to be a little more wordy - -FromFormattedOutput - and then rely on prefix name matching (-FromFormat) or the alias name for brevity.

I think the value proposition here is clear, and I like the ideas around the parameter name (thanks for participating in that discussion @vexx32 and @mklement0!), so I'll probably move forward with -FromFormattedOutput for now in a PR and then change it later if the PowerShell Committee has something else they would prefer.

@mklement0 I like the -ff alias' similarity to -f for -Force, as though it's an "I really don't care how you do it, just make it a string and work with it! 💢" kind of switch, which is kind of fitting. 😂

I was just watching a 30-minute demo of Docker streamed from Ignite, when the presenter ran this command:
Get-Process | findstr smss

This is exactly what Select-String should be able to do by default, rather than switching to findstr or grep. In fact, consulting the Select-String documentation shows that it is even documented to work similarly to both grep and findstr, yet as demonstrated by the difference in output from gps | findstr smss and gps | sls smss, you can see that Select-String does not function like findstr or grep, which work against the textual output of a command.
Given that's the case, I'm hopeful the PowerShell Committee votes in favor of not having an additional parameter to get this behavior.

Yeah, since this is about selecting strings, you'd think that when the command doesn't get string input, it would automatically run the object through the formatting engine to get the formatted string to search.

Good points, @KirkMunro  and @rkeithhill; I think changing the default behavior makes perfect sense and strikes me as  a bucket 3 change:

On the  input side, there are probably not too many people who rely on the current behavior, as it is either near-useless or hard to predict (you'd have to know what a particular input type stringifies to with .ToString(), which is often just the full type name, and otherwise non-standardized and not typically visible in PowerShell).

I wish we could also change the default behavior on the output side - have Select-String output strings by default - but that would obviously be a prohibitive breaking change; at least we have the -Raw switch now, however.

at least we have the -Raw switch now, however.

It looks very specific and Out-String works.

@mklement0 If I understand you say that no need to have FromFormattedOutput parameter?

It looks very specific and Out-String works.

The -Raw switch is already implemented - even though you could always do Out-String -Stream or (...).Line, -Raw is clearly an improvement in terms of bith convenience and performance.
I simply pointed -Raw out, because it isn't well-known yet - your response being a case in point.

If I understand right you say that no need to have FromFormattedOutput parameter?

Yes, we are now advocating simply changing the default behavior for non-string input.

Heading into @PowerShell/powershell-committee meeting in a few mins, just got through this thread and wanted to get a few thoughts out:

I absolutely agree that it's a valuable scenario to want to filter the formatted received at the console. I use findstr and grep all the time (and if I ever get around to setting up this, I'll just use grep everywhere).

My inclination in reading through the thread is actually to lean towards an entirely new cmdlet with the reasoning that it sounds like there's multiple behaviors one might want, and an expanding matrix of parameters on Select-String doesn't sound awesome. Don't get hung up on the name, but I think something like a Select-FormattedOutput or Select-FormattedString would be helpful.

I'm very worried about breaking changes to Select-String. Not picking on you at all, @mklement0 (I saw this sentiment throughout the thread), but with regards to this statement:

On the input side, there are probably not too many people who rely on the current behavior, as it is either near-useless or hard to predict

The issue with breaking changes is that often folks have hardcoded themselves to work around "hard to predict" behaviors. Take @KirkMunro's Get-Date example: the default ToString() doesn't return what's output by the default formatter. Maybe I did it once, realized that the string name of the month wasn't in the part parsed by Select-String, and now I'm actually trying to parse "12/18/2019 2:52:23 PM" as a mm/dd/yyyy string and hand it to some native utility. This breaks me catastrophically.

To that end, I propose that one or two people actually build this, throw it up on the Gallery, and then we can actually play with an implementation and decide what fits right across multiple platforms (I have a strong feeling we won't get it right on the first try).
Might it actually just be a wrapper on findstr and grep depending on platform? Or is there a reason we would want this to be fully implemented in PowerShell land?

@PowerShell/powershell-committee discussed this, and we agree it should be started in the Gallery by folks in the community. Primarily, we question the overall usefulness of this given the existing findstr and grep on platforms. If there's an elegant design that proves to be popular, we can consider including it in the future.
We do agree that we should stop telling new users that Select-String/sls is our grep equivalent and document the existing utilities and foo | Out-String | Select-String 'bar' workaround better.
Oh, and we bikeshedded on the name of the cmdlet for a while. No one's in agreement, so I threw every possible one that folks threw out into my previous comment

@joeyaiello What is the perceived value in adding another cmdlet to do essentially the same job? :confused:
The way I see it at the moment is:

We have a Select-String cmdlet,
for which a lot of potential piped input is effectively useless, because
most objects' ToString() values are significantly less useful than their Out-String values.

Adding another cmdlet effectively only duplicates Select-String (and likely quite a lot of code if we're not very careful and do a lot of refactoring of the original cmdlet as well), and would effectively make Select-String a second-class citizen -- we introduce a new cmdlet that does everything Select-String does, and more, and then Select-String becomes effectively useless, since the new cmdlet would already do what Select-String does.
I don't see a particular need for this to be a whole extra cmdlet, and nor do I think adding one additional parameter would be a huge change for this cmdlet. If we're talking adding parameters to Invoke-WebRequest, sure, cause for caution. Here? Not so much, in my opinion.

Some small comments:
@KirkMunro

Now, try to match the month using Select-String. This returns nothing.

Get-Date | Select-String -Pattern $month

This seems rather artificial to me given that PowerShell is an object-based shell. Why are you not doing:
Get-Date | where Month -match $month

And if you really want to grep against formatted output, it's simple:
Get-Date | Out-String -Stream | Select-String -Pattern $month

@mklement0

On the input side, there are probably not too many people who rely on the current behavior,

On what basis are you making this claim? It would certainly break my daily workflow.  Oh and BTW, Select-String works against files too so that has to be taken into account.
Finally

Going through the formatter is an order of magnitude slower than where or sls (as it is today).
Formatted output has no guarantee of compatibility. Changing how objects are rendered is one of the things that is explicitly called out as being completely open to change. So writing scripts against formatted output is discouraged.

There's scripting in scripts, where you want performance and you want to use .NET objects, and there's ad-hoc use where you are pulling data and you want to get chunks of it quickly and easily. That's why grep or findstr are used even by folks on the PS Team or PMs not on the PS team in place of Select-String, and in place of using Out-String -Stream, Where-Object, etc. Being able to get information quickly and easily is highly useful, which is the motivation here. The motivation here, at list for me, isn't to use Select-String to filter output in scripts.
My only argument for updating Select-String vs a new command is to have a single command to do easy filtering with highlighting of what it found in-box so that I don't have to worry about what system I'm on when I use it. Given PowerShell is a shell, this seems like solid value to offer across the board. I can use grep or findstr, but I tend to stick to native, in-box PS that just works everywhere. That's where I think augmenting Select-String vs using @SteveL-MSFT's Select-Text in his upcoming module adds value.
Given the pushback on augmentation of Select-String and the concern around breaking changes, I'll just hope that a successor gets enough traction to replace Select-String with a suitably short alias in-box, and then just switch to that.

In addition to the points made by @vexx32  and @KirkMunro:

And if you really want to grep against formatted output, it's simple:

Select-String's sole purpose is to search through strings (and being able to do that "quick and dirty", as @KirkMunro explains, is an invaluable interactive tool).
Now, if the input isn't composed of strings, searching what string representation of the input makes more sense?

What you see in the console (host), i.e. the formatted representations?
Or  the result from a .ToString() call which produces a near-useless and hard-to-predict stringification you don't typically get to see elsewhere (which answers the on what basis question).

Piping to | Out-String -Stream isn't simple: it's an obscure, cumbersome workaround for something the cmdlet should have done automatically to begin with.

Oh and BTW, Select-String works against files too so that has to be taken into account.

Yes, the -LiteralPath binding via Get-Item / Get-ChildItem output for file-content searching would have to be retained, which makes for an inconsistency - but an easily explained one.
That is, if you really wanted to search a directory listing as printed to the screen, then - and only then - would you need | Out-String -Stream.
Apparently we have an oss function wrapper for that now - that such a function was created speaks to how often you currently have to resort to that workaround.

It would certainly break my daily workflow.

Assuming the -LiteralPath binding is retained, what would break?
Other than someone inappropriately using Select-String with non-string input in  a script (the scenario that's @joeyaiello's concern) - which is where you should definitely check object properties instead - nothing should break, and much is gained.
To me, that makes it a bucket 3 change, which spares us the confusion of introducing another cmdlet.

Revisiting some of the earlier points:

Primarily, we question the overall usefulness of this given the existing findstr and grep on platforms.

The different syntax and varying capabilities between these utilities , let alone having to use different utilities on different platforms to begin with is reason alone to make Select-String support the intuitively expected behavior of searching the formatted representations.

Going through the formatter is an order of magnitude slower than where or sls (as it is today).

If the intent is to search the formatted representations, that cost must inevitably be paid  - and using findstr or grep already incurs that cost.
Again, the primary use case is quick-and-dirty interactive search.
In programmatic use, Where-Object should be used instead.
Providing such guidance - along with conversely saying: in scripts, use Select-String only with string input - as part of the documentation should be sufficient.

throw it up on the Gallery, and then we can actually play with an implementation

It's not in the Gallery, but I've created a Select-StringFormatted wrapper function (which will be slow compared to a proper implementation as part of SelectString) in this Gist (also part of this SO answer), which has Out-String -Stream built in - this is what Select-String itself should do.
I recommend defining scs as its alias, which is what Select-String's alias should always have been, had its name followed the naming conventions.
Assuming you have looked at the linked code to ensure that it is safe (which I can personally assure you of, but it's always good to check), you can install it as follows:
irm https://gist.github.com/mklement0/46fea9e6e5ef1a3ceaf681c976cb68e3/raw/Select-StringFormatted.ps1 | iex

Sample call (assumes Set-Alias scs Select-StringFormatted):
PS> Get-Process | scs service

     16     3.74      14.09       0.00    5664   0 SecurityHealthService
     10     3.61       6.84       0.00     632   0 services
     12     3.21       7.15       0.00    2388   0 VGAuthService

While this is already very convenient, there's room for improvement, given that simply string-filtering the lines doesn't include the header line with the column names; for (implicitly to be) Format-Tabled input, the header line could be included for display, analogous to the existing for-display enhancement of coloring the matching part of the line.

I think that having Select-String search against the formatted output of objects would be a great idea.  As it is, I only use it to search files or plain text as it otherwise acts odd.  Same thing with Set-Clipboard.  If a cmdlet exists to do things with strings, then it should handle it in a way that makes the most sense.  If I manually format something, I don't want Select-String to be searching the FormatEntryData string that it gets from ToString.  I would expect it to search the formatted output.

Hear, hear, @FireInWinter, but note that - as with Set-Clipboard - it is the - often useless - .ToString() return value applied directly to the input object that is used:
PS> @{ foo=1 } | sls hash   # @{ foo=1 }.ToString() -> 'System.Collections.Hashtable'

System.Collections.Hashtable


Typo in HTML ids vs classes lecture

Describe the Issue
Hello,
I noticed a typo in one of the HTML lectures.
One of the questions to verify the user watched the video is as follows:
What happens if you try to use duplicate an id in your HTML?
Affected Page
https://www.freecodecamp.org/learn/full-stack-developer/lecture-html-fundamentals/what-are-ids-and-classes
Steps to Reproduce
The error is on question #2 of the page linked above.
Recommended fix or suggestions
I believe what the author was going for was:
What happens if you try to duplicate an id in your HTML?
But, I think it could be better posed as:
What happens if you use the same id more than once in your HTML?
Screenshots
No response

There are a few other changes that need to be made for the questions following this lecture video
For the feedback for the first question, it should be changed to this
### --feedback--

Remember that an `id` name is unique and should only be used once per HTML document.

It needs to be changed in all areas.
Also, for that second question, I don’t think the feedback is all that helpful.
Let’s change it to this instead
### --feedback--

Think about how using duplicate `id` names in your HTML can affect your CSS and JavaScript.

For interested contributors, here is the file you need to update
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/25-front-end-development/lecture-html-fundamentals/6708382cf088b216580a9ff1.md


vault-secrets-webhook: Allow users to specify resources for init-containers

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
fixes #1328

License
Apache 2.0

What's in this PR?

Introduced environment variables ( VAULT_ENV_CPU_REQUEST, VAULT_ENV_MEMORY_REQUEST, VAULT_ENV_CPU_LIMIT, VAULT_ENV_MEMORY_LIMIT) for vault-secrets-webhook. Instead of applying hard-coded values, this provides the users an option to specify resource requests and limits for the init-containers being added during pod mutation for secrets viz vault-env and copy-vault-env. Without this change, it is not possible to run mutated pods with "guaranteed" QoS using the default settings - they always run with "burstable".
Hint: Set VAULT_ENV_CPU_LIMIT=50m to be able to run the mutated pod with guaranteed QoS.
Checklist

[x] Code meets the Developer Guide
[n] User guide and development docs updated (if needed)
[x] Related Helm chart(s) updated (if needed)

Can you fix the tests? You can check it in the build logs.


[PT] add 5.1 and 5.2

#51

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Remove FluentAssertions

This testing style and library are inconsistent with our other projects.

Do you accept PR for this issue?

Start with one test class? Some if this will be mechanical, but another part will be about establishing the desired style for the tests. I'd also like to address the over use of Mock.

Well I have already removed the FluentAssertion from all projects. I sent a PR.


[SMALLFIX] Refactor zookeeper code to inject leader path

Generalize the utility to take the leader path as an argument

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11930/Build result: ABORTED[...truncated 2877 lines...]	at org.codehaus.plexus.classworlds.launcher.Launcher.launchStandard(Launcher.java:330)	at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:238)	at jenkins.maven3.agent.Maven31Main.launch(Maven31Main.java:181)	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)	at java.lang.reflect.Method.invoke(Method.java:606)	at hudson.maven.Maven3Builder.call(Maven3Builder.java:136)	at hudson.maven.Maven3Builder.call(Maven3Builder.java:71)	at hudson.remoting.UserRequest.perform(UserRequest.java:120)	at hudson.remoting.UserRequest.perform(UserRequest.java:48)	at hudson.remoting.Request$2.run(Request.java:326)	at hudson.remoting.InterceptingExecutorService$1.call(InterceptingExecutorService.java:68)	at java.util.concurrent.FutureTask.run(FutureTask.java:262)	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)	at java.lang.Thread.run(Thread.java:745)[INFO] [INFO] --- maven-jar-plugin:2.4:jar (default-jar) @ alluxio-tests ---[WARNING] JAR will be empty - no content was marked for inclusion![INFO] Building jar: /home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/target/alluxio-tests-1.4.0-SNAPSHOT.jar[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: java.io.IOException: Backing channel is disconnected.[INFO] [INFO] >>> maven-source-plugin:2.3:jar (attach-sources) @ alluxio-tests >>>
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11933/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11934/
Test PASSed.

LGTM


ATS lack of API to temporarily reload remap file from specified file

Currently, ATS reload remap file by calling reloadUrlRewrite(), the filename is fixed (either default "remap.config" or configured by "proxy.config.url_remap.filename").  But if ATS crashes after reloading remap.config (due to some bug in some remap plugin), ATS will keep crashing since the remap.config is always bad.  We have a module which can periodically detect if there's new remap.config available and reload that file.  We would like to do is saving the new file as remap_temp.config and reload that; if after a period of time and the ATS is running normal, we rename remap_temp.config to remap.config.   In this case, if remap_temp.config has issue, ATS crashes; ATS can start up normal by using the old remap.config.
Currently, ATS doesn't have API to reload a specified remap file.  I would like to add another API - reloadUrlRewrite(const char *filename) to reload a remap file to achieve this goal.

If this is still an issue please reopen.


Add onboarding instructions/resources for being on-call

This should likely be co-located with the other support docs, but we should make sure to link to it from the onboarding checklist.
This doc may include (brainstorming):

Link to the video that @izakp did explaining kubernetes/monitoring
"What you should to have set up" when you start being on-call (i.e. hokusai, heroku CLI, etc.)
Additional information on how to debug issues/monitor
Maybe links to past notable incidents and their resolution

Closing this as we now link to the onboarding video in our docs.


Add PR template

@eddiebergman I think your merge was a bit too quick for me 😅

We can keep the current template and ignore my suggestions above. It was only minor stuff.

I'll add a new PR with those changes, apologies for being a bit fast with it


Gallery: Some drawer item actions are not accessible

In the gallery, when the following items from the drawer are selected there is no feedback for screen reader users about the result of the action:

Animate Slowly
Checkerboard Offscreen Layers
Checkerboard Raster Cache Images
Performance Overlay

(This was discovered in an a11y review)

Don't we announce that the checkbox is now checked, in each case?

You're right. That change was probably not included in the gallery version that was used for the review.


Reverse occupancy sensor

What do you think about make an option for reverse occupancy sensor. Then we can make “native” notification when the robot starts cleaning.

+1 for this. I think it makes more sense for an occupancy sensor to be active when there is movement/activity in the Home.


[todo] update claude-2  to claude-2.1

Is your feature request related to a problem? Please describe.
新功能是否与解决某个问题相关, 请描述
claude-2 not work now
Describe the solution you'd like
你期望的新功能实现方案
update to claude-2.1
Additional context
其他
Add any other context or screenshots about the feature request here.

@joostshao I just did a quick test(not in debug mode yet), mine is infinite Waiting for response..., is your the same?

PR #573 prepared, may need some tests.

@joostshao I just did a quick test(not in debug mode yet), mine is infinite Waiting for response..., is your the same?

worked for me , and wait 10 seonds.

I will handle the version update later today


Can't connect to locally running Kafka

I'm trying to connect Kafka-ui to a Kafka cluster that is running locally on my machine, NOT via a Docker container.  It will connect fine to other clusters that I am monitoring, but not to my local cluster.  No matter what I try it reports the local cluster as offline.
In my Kafka config/server.properties I have the following entries:
listeners=PLAINTEXT://localhost:9092
advertised.listeners=PLAINTEXT://localhost:9092
I'm currently using the following docker-compose file:
version: '3.7'
services:
kafka-ui:
image: provectuslabs/kafka-ui
ports:
- "9080:8080"
environment:
KAFKA_CLUSTERS_0_NAME: Local
KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS: host.docker.internal:9092
KAFKA_CLUSTERS_0_ZOOKEEPER: host.docker.internal:2181
KAFKA_CLUSTERS_0_KAFKACONNECT_0_NAME: Local
KAFKA_CLUSTERS_0_KAFKACONNECT_0_ADDRESS: http://host.docker.internal:8093
I see the following in the log output:
kafka-ui_1  | 17:02:27.776 [kafka-admin-client-thread | adminclient-60] ERROR com.provectus.kafka.ui.service.KafkaService - Failed to collect cluster Local info
kafka-ui_1  | org.apache.kafka.common.errors.TimeoutException: Call(callName=listNodes, deadlineMs=1636563747774, tries=1, nextAllowedTryMs=1636563747875) timed out at<PHONE_NUMBER>775 after 1 attempt(s)
kafka-ui_1  | Caused by: org.apache.kafka.common.errors.TimeoutException: Timed out waiting for a node assignment. Call: listNodes
This is what I get in the ui:

As you can see, I can connect to other clusters on my network, just not my local cluster.
I've tried replacing host.docker.internal with my the local IP address of my machine and the IP address assigned by my VPN, and  receive the same problem.
Is there some other configuration that I'm missing?
Thanks

Yes, that is correct.

@GloWorm0
Which platform are you running the app on?
How did you try passing the host.docker.internal?

I'm running on macOS (10.15.7).
I'm not sure I understand.  I just tried to use it when setting the environment variables for the container.  So instead of KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS: localhost:9092 (which obviously won't work because it's not running in the container) I tried KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS: host.docker.internal:9092 (as well as actually using the IP.

It's not just that.
First of all, your docker-compose should have this:
services:
  kafka-ui:
    extra_hosts:
      - host.docker.internal:host-gateway
[...]

Secondly, your KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS should look like this: host.docker.internal:9092.
Moreover, take a look at your KAFKA_ADVERTISED_LISTENERS, it should be like this: PLAINTEXT://host.docker.internal:9092.

I just tried that setup and it worked for me.
KAFKA_ADVERTISED_LISTENERS is kafka's variable.
Yeah, that's probably right. I tried that setup with kafka running in a container but I've accessed it externally via port forwarding, so, something close to your setup.

Adding extra-hosts didn't change anything.  Very frustrating.  I can access other clusters, just not my local Kafka.  I'm trying to evaluate this tool and would rather do some of the tests just on my local cluster, but I guess I won't be able to do that.
I'm sure it's something with my setup, but have no idea what it could be.  Kafka is up and running and accessible, just not from within Docker.
I've run other simpler tests with containers not involving Kafka and can use hosts.docker.internal to access the local machine, just not with this container.
I'll have to see if I can get another container to interact with my local Kafka.

I'd also try to run a 'fair' instance of kafka on bare metal to try that thing out. I suppose the trick might be about KAFKA_ADVERTISED_LISTENERS.

OK, I figured it out.
In my Kafka server.properties I had advertised.listeners=PLAINTEXT://localhost:9092.  This only allow access via localhost.  Once I changed it to my machine's IP address everything worked fine.
Thanks for the help.

You're welcome :) I'll convert this to a discussion to make it easily discoverable for others.


Show release notes automatically after an install @joaomoreno

@egamma isn't this #12982, fixed yesterday?

created by accident.


variantProperties can also be a string

It can be set to Component set for node has existing errors. This happens when you have invalid/duplicate variant(names).
The type definition should be something in the line of:
  interface VariantMixin {
    readonly variantProperties: { [property: string]: string } | null | string
  }

or:
  // I am unsure if this is allowed in a type definitions file
  const VariantErrorMessage = "Component set for node has existing errors";
  interface VariantMixin {
    readonly variantProperties: { [property: string]: string } | null | typeof VariantErrorMessage;
  }

It seams everytime we have an error from Figma, variantProperties (that is deprecated btw) changes its type to string to bring error messages.
I saw this occuring this week, when trying to get instances with conflict errors.

Because this question is related to a now deprecated API, I'm going to close this issue out. If you have issues with the new API componentProperties, please feel free to open a new issue.


Do we have a global state in MvRx?

Hello,
My issue is probably close to this issue: https://github.com/airbnb/MvRx/issues/174
However, I don't think the answer is generic enough (combining two ViewModel classes into one class).
My case is like this:

I have an activity that contains two fragments, FragmentA and FragmentB.
On TV app both of them are visible at the same time while on the Mobile app, FragmentA displays first and then FragmentB displays if the user clicks on a button.
FragmentA contains a RecyclerView that displays a list of items.
FragmentB is a normal view that displays an object that FragmentA sends to it.

Based on my understanding of what has recommended in the above issue, the solution is to have one ViewModel class for both of Fragments. I don't like this approach because I don't want to have a big ViewModel class. Also, I think it is against the SoC.
A good approach, in my opinion, is what Redux does by having a global state for the whole of the app.
As I am very new to MvRx, do we have such a thing(global state)? If yes, would you please show me a doc or a sample app that I refer to? If not, what is your recommendation?
Thanks again for your awesome job 👍

@Hesamedin We don't have anything like that in MvRx but what you could do is create some sort of singleton to hold that shared state that emits some sort of observable. You could then use MvRxViewModelFactory.create to inject it as a constructor dependency of both ViewModels. Then each ViewModel could call functions on it/observe its state and both would be kept in sync. Would that work?

Thanks @gpeal for your answer. It makes sense. Please let me get more info about MvRxViewModelFactory.create from the wiki and I'll come back in case I have another question.
I close this ticket for the meantime.

@Hesamedin https://github.com/airbnb/MvRx/wiki#view-model-creation


Include IOException error message when response entity write fails

 Issue by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT
Originally opened as https://github.com/twogee/missing-link/issues/26

If response cannot be written to disk, the IOException error message is
currently buried.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 13 Sep 2011 at 3:45

 Comment by GoogleCodeExporter
Monday May 11, 2015 at 22:05 GMT

Original comment by<EMAIL_ADDRESS>on 13 Sep 2011 at 3:46
Attachments:

ML26_Include_IOException_error_message_when_response_entity_write_fails.patch


Cant import Icon in sveltekit

Package

[ ] lucide
[ ] lucide-angular
[ ] lucide-flutter
[ ] lucide-preact
[ ] lucide-react
[ ] lucide-react-native
[ ] lucide-solid
[X] lucide-svelte
[ ] lucide-vue
[ ] lucide-vue-next
[ ] Figma plugin
[ ] source/main
[ ] other/not relevant

Version
0.454.0
Can you reproduce this in the latest version?

[X] Yes
[ ] No

Browser

[ ] Chrome/Chromium
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] iOS Safari
[ ] Opera
[X] Other/not relevant

Operating system

[ ] Windows
[ ] Linux
[ ] macOS
[ ] ChromeOS
[ ] iOS
[ ] Android
[X] Other/not relevant

Description
tried all of these
import Home from 'lucide-svelte/icons/home';
import Home from 'lucide-svelte/dist/icons/home.svelte';

is taking from website
https://lucide.dev/guide/packages/lucide-svelte#typescript-example

Failed to resolve import "lucide-svelte/icons/home" from "src/lib/components/Icon.svelte". Does the file exist?

[plugin:vite:import-analysis] Missing "./dist/icons/home.svelte" specifier in "lucide-svelte" package

Steps to reproduce
see above
Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)

Lol, okay the docs are wrong. I'm able to import, but actually there is no home icon

Do we not generate the alias files here?
For a quick fix replace the path home with house.

at least there is no home.svelte for me

at least there is no home.svelte for me

I implemented a fix for this in lucide-react a long time ago, looks like we should also fix this here.

This is where I fixed it in lucide-react https://github.com/lucide-icons/lucide/pull/1486

I've something working for this!
@jguddas Yess that's correct, I've used part of this, but it needed some extra code to have .svelte files as output.

@Kapsonfire-DE I've created a test release for this on<EMAIL_ADDRESS>Can you maybe check if everything is working file on your side?

Im sorry, I didnt see this yet. im Gonna test tomorrow

Working, thanks!


charts getting blurred when zooming out and then zooming back in

when i zoom the window to extreme minimum i..e., 25% and then come back to 100% 2-3 times then the charts get blurred

Could you please give us some details ?
On what browser have you experienced this behaviour and which version of chart.js are you using ?

i am using chart.js version 2.0 and i have experienced the issue in both chrom and firefox, the issue arrises when the zoom is reduced to minimum and then increased back to 100%

@EshaanSharma711 could you create a js fiddle that reproduces the bug ? I can't reproduce that bug myslef, nothing get blurred (with firefox 59.0.3 and chrome 66.0.3359.139).

@SimonLoir i can provide you with  the image of the error ,as you can see in the image the widget along with the tooltip got blurred

@SimonLoir i can provide you with an image of the error, as you can see in the image, the chart along with the tooltip got blurred, this happened when i zoomed out to 25% and then zoomed back in to 100% two times

I can still reproduce this on master. It looks like resize events fire when increasing the zoom after getting down to 25%. Perhaps something is wrong in the scaling that takes place during the resize event


Reverse sorting backtraces in the debugger...

I can't think of the last time the top few frames of a backtrace were the interesting ones. In a terminal window, it makes sense to that the deepest frames are also the most interesting, and closest to your cursor at the bottom of the screen. On the web, however, that means that the most interesting frames are at the bottom of your browser window, which means scrolling.
It'd be EXCELLENT if the backtrace in werkzeug was reverse sorted so that the deepest frames are at the top of the page.
In an ideal world, I think this would be a tunable, something like:
WERKZEUG_BACKTRACE_ORDER = 'descending' # vs 'ascending'
Even if it wasn't a tunable, having it reversed by default makes more sense.
Thanks in advance. -sc

I'd rather not automatically scroll, but maybe clicking the error message at the top of the page will scroll to its counterpart at the end of the traceback.


PIC flag for Makefile-built Android Static Library

Currently if we build a shared lib with ndk = r12b and android-api = 21, and the shared lib is linking Makefile-built tensorflow android static library, the android ndk hints that the tensorflow and nsync lib is not complied with -fPIC option, this PR fix the problem. It only impacts the android build.

Can one of the admins verify this patch?

@petewarden please review

Jenkins, test this please!

(CLA is fine, it's just me)

Can one of the admins verify this patch?


Remove section renderer

Fixes #1021
This PR removes the SectionRenderer class, putting the responsibility instead inside the Section::PreviewService. I think this is an improvement as it moves more of the functionality inside the service objects, but the fact that this service is called "Preview" and is used in the SectionPublishingAPIExporter makes me think it is either doing too much, or needs a different name.

@floehopper yes, I started thinking something along those lines too as I made the changes. I'll see what that looks like in a branch.

@chrislo: It think it might make sense to close this PR while you investigate the alternative approach?

I'll have a think and reopen another one later.


D2IQ-69177 fix: restore limits when editing a service

while the form would show you the resource limits you set for a service, the
JSON editor is not picking those up. this ultimately leads to those values being
forgotten if you just hit "edit service" -> "save".
this change helps the JSON editor pick up the current values.

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

v5.0.52
GitHub release

Your semantic-release bot :package::rocket:


'vlc_windows_fix.h': No such file or directory

idk when I'm complied the game this error pop up (even I recompiled so I still don't know what is issue)

Do a clean build, remove the export


This PC shortcut open old explorer.exe when Files is set as default explorer app.

What's the Problem?
This PC shortcut open old explorer.exe whe Files is set as default explorer app.
Solution/Idea
Replace Files as default explorer everywhere. When I open This PC shortcut or if try attach file to Outlook email message it open small window of old explorer. Is it possible?
Alternatives
Make some link between explorer.exe and new explorer if is it possible.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
<IP_ADDRESS>
Windows Version
22000.348
Comments
No response

This is currently by design. To open files when clicking on This PC or recyclebin, the reg files included below should do the trick but there may be undesired consequences.
To do:

Extract the two files from this archive SetUnsetFilesAsDefault_Bad.zip and place them on the desktop
Run SetFilesAsDefault_Bad.reg

Closing this for now, this is something we're working on but still too experimental to ship publicly :)

Great it works :-) Thanks

@koubafire glad it works. If you could provide feedback about how well it works if would be great. E.g if you find it causes issues opening the Control Panel etc.

Control Panel works, but it crash when I try open Network drives. I have Czech version of windows 11


replace the css`` path using constructed ast to avoid plugin conflicts

All of the examples of this would need to be updated as well. I just changed this one to get the ball rolling.
As a bonus, this change allows for attr css value support in tkh44/emotion.

You should now have full commit access. Have fun :)

I don't think it worked 😐

There should be an invite in your email, you need to accept it


Syntax error in polymer-micro

There seems to be a problem with polymer-micro file after this addition 2 days ago
https://github.com/alchapone/polymer-rails/commit/aea1b2996224b148cf7987d99d00bcc65a357cf1
It throws a syntax error ln:16
Currently i changed polymer-micro resolve function to:
function resolve() {
document.body.removeAttribute('unresolved');
}
if (window.WebComponents) {
addEventListener('WebComponentsReady', resolve);
} else {
if (document.readyState === 'interactive' || document.readyState === 'complete') {
resolve();
} else {
addEventListener('DOMContentLoaded', resolve);
}
}
}());

Just to get things trough this seems to work, but i do believe there was a problem when the change happened 2 days ago.
Edit: Created a pull request for convenience https://github.com/alchapone/polymer-rails/pull/53
Cheers

thank you


Convert evaluation to file (noch mal darüber reden)

User story

As a consultant
I want the evaluation as a file
So that I can present it to customers

Acceptance criteria

[ ] Evaluation can be downloaded as file

Definition of done

Approved by product owner
Code has been peer reviewed and approved
No syntax or runtime errors emerged
Code has to be included in the release candidate

Backend will es löschen, frontend fragen


ISPN-7398 Fix AbstractProtocolServerTest.testStartingWithoutTransport

https://issues.jboss.org/browse/ISPN-7398

Looking...

I think we also need to clean up the getCache(String, String) method a bit
Maybe we should have it do
if (DEFAULT_CACHE_NAME.equals(cacheName)) {          log.deprecatedDefaultCache();          return getCache();       }
This way we also get the default cache exception thrown.

Hm, that would create an infinite loop, but yes we need a null handler there too. Let me fix it.

@wburns fixed

Ah yes in the case when default cache name is set.  I was trying to trust
the of block instead of copying, but I guess copying is required :+1:
On Fri, Jan 27, 2017, 6:11 AM Tristan Tarrant<EMAIL_ADDRESS>wrote:

Hm, that would create an infinite loop, but yes we need a null handler
there too. Let me fix it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/infinispan/infinispan/pull/4802#issuecomment-275642653,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnj7zOxDrmADPUxS3ZPZqoG3Nev2qhmks5rWdDPgaJpZM4Lti-V
.

Pulling...

Integrated into master, thanks @tristantarrant !


Drop Python3.8, support Pandas v2.2 [do not merge]

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #1268
This PR fixes #1534

[ ] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[ ] CHANGES.rst has been updated (with summary of main changes)

[ ] Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

Drops Python3.8 references and marks Python3.9 as base requirement.
Drops older xarray and pandas support.
Updates documentation and CI for dropped Python3.8.
Raises required versions of scipy and numpy (1.9.0 and 1.20.0, respectively).
Removes the winter_storm indice and indicator (deprecated in xclim v0.46.0).

Does this PR introduce a breaking change?
Yes, Python3.8 is no longer supported and base dependency versions have been augmented:

numpy>=1.20.0
pandas>=2.2.0
scipy>=1.9.0
xarray>=2023.11.0

Other information:
Python3.12 support is still not possible due to missing Python3.12 builds for numba. (This is being addressed in #1613)

@aulemahal I've left a few "FIXME" notices since changes will ultimately involve us needing to make some API/UI decisions.

@Zeitsperre  My git-fu is weak so I pushed the relevant commit of the other branch here instead.

On my todolist, the only remaining issue here is the darn bug of lazy_indexingwith the newest map_blocks.

Fixed! My solution also works with 2023.11.0.

Adding the approved label so that we can run more tests. This PR will still require a proper review.

coverage: 90.156% (-0.1%) from 90.299%
when pulling 21c966d314f40328063bf5e55f20579ddf776c01 on drop-python38
into e1148b2e5b5423917efe35938022d79427231ad1 on master.

Sorry for the mammouth PR indeed. All those changes stem from the same two updates of python and pandas.
Xclim doesn't precribe what frequency code should be used, in most cases, it only passes them down to xarray. This PR changes all places where xclim was using the old codes (tests, checks, documentation). A user with pandas < 3 will stil be able to use "A" and should get a warning from pandas.
However, there a some instances where a frequency code is checked against the output of xr.infer_freq or parsed with pd.frequencies.to_offset. Changes there are breaking, which explains why we decided to pin pandas and xarray to the versions accepting the new codes.


'/app/wp-content': No such file or directory

Hello
I am trying to run using fig, but get the following error:

Step 4 : ADD wordpress.conf /etc/apache2/sites-enabled/000-default.conf
 ---> Using cache
 ---> 86541ebe4fb1
Step 5 : RUN rm -fr /app
 ---> Using cache
 ---> a66b310a5a36
Step 6 : ADD WordPress/ /app
 ---> Using cache
 ---> 675a2f9e1b52
Step 7 : ADD wp-config.php /app/wp-config.php
 ---> Using cache
 ---> 2cc18154b334
Step 8 : ADD run-wordpress.sh /run-wordpress.sh
 ---> Using cache
 ---> a2129b577d80
Step 9 : RUN chmod 755 /*.sh
 ---> Using cache
 ---> ec8b92f7fffd
Step 10 : RUN chmod -R 777 /app/wp-content
 ---> Running in 4da7fc7712da
chmod: cannot access '/app/wp-content': No such file or directory
2015/01/31 20:49:28 The command [/bin/sh -c chmod -R 777 /app/wp-content] returned a non-zero code: 1
anton@anton-magickpad:~/git/tutum-docker-wordpress-nosql$

Can you please tell me what I need to do to fix this?
Thanks

Would be nice with an answer here.

Cuz this repo uses wordpress as a sub-module, you have to run git submodule init and git submodule update before running fig/docker-compose.
I was thinking to remove the sub-module though

fixed in the latest commit


Dependency functionality is not able to detect outputs

Hi,
I realized that dependency is not working correctly.
Unfortunately, I can't find a similar already open issue.
Architecture:
<account_id>/config-rules/us-west-2/REQUIRED_TAGS/terragrunt.hcl
<account_id>/config-rules/us-west-1/REQUIRED_TAGS/terragrunt.hcl
<account_id>/iam_role/terragrunt.hcl

The module REQUIRED_TAGS needs the ARN of the IAM Role.
 <account_id>/config-rules/us-west-2/REQUIRED_TAGS/terragrunt.hcl:
locals {
  region_vars   = read_terragrunt_config(find_in_parent_folders("region.hcl"))
  aws_region    = local.region_vars.locals.aws_region
}

dependency "role" {
  config_path                             = "../../../iam_role"
}

terraform {
  source = "${get_terragrunt_dir()}/../../../../../../..//ccoe-modules-aws-config/modules/rules/REQUIRED_TAGS"

  extra_arguments "custom" {
    commands = [
      "plan",
      "import",
      "push",
      "refresh",
      "destroy",
    ]

    arguments = [
      "-var-file=${get_terragrunt_dir()}/../../../config.tfvars",
      "-var-file=${get_terragrunt_dir()}/../../../tags.tfvars",
    ]
  }

  extra_arguments "output_file" {
    commands = [
      "apply"
    ]
  }
}

include {
  path = find_in_parent_folders()
}

inputs = {
  output_role_arn = dependency.role.outputs.role_arn
  aws_region    = local.region_vars.locals.aws_region
}

<account_id>/iam_role/terragrunt.hcl
terraform {
  source = "${get_terragrunt_dir()}/../../../../..//ccoe-modules-aws-iam/modules/roles"

  extra_arguments "custom" {
    commands = [
      "plan",
      "import",
      "push",
      "refresh",
      "destroy",
    ]

    arguments = [
      "-var-file=${get_terragrunt_dir()}/../config.tfvars",
    ]
  }

  extra_arguments "output_file" {
    commands = [
      "apply"
    ]
  }
}

include {
  path = find_in_parent_folders()
}

IAM Role module:
outputs.tf
output "role_arn" {
  value = aws_iam_role.sns_role.arn
}

Issue:
Command:
terragrunt run-all plan \
--terragrunt-parallelism 5 \
-out=output.tfplan \
-lock=false \
--terragrunt-log-level 'error'

Changes to Outputs:
  + role_arn = "arn:aws:iam::<account_id>:role/NotificationAutomationExecutionRole"

You can apply this plan to save these new output values to the Terraform
state, without changing any real infrastructure.

Saved the plan to: output.tfplan

To perform exactly these actions, run the following command to apply:
    terraform apply "output.tfplan"
ERRO[0044] Module /Users/<account_id>/config-rules/us-west-2/REQUIRED_TAGS has finished with an error: /Users<account_id>/iam_role/terragrunt.hcl is a dependency of /Users/<account_id>/config-rules/us-west-2/REQUIRED_TAGS/terragrunt.hcl but detected no outputs.
Either the target module has not been applied yet, or the module has no outputs. If this is expected, set the skip_outputs flag to true on the dependency block.  prefix=[/Users/<account_id>/config-rules/us-west-2/REQUIRED_TAGS]

Environment

terragrunt version v0.36.1
Terraform v1.1.6
on darwin_amd64

I see that you are running run-all plan. Is this a fresh new deployment where nothing has been deployed yet? If so, then you are running into the known issues with plan-all, documented in the warning here.

@yorinasub17 Thank you for the answer.


dbs-upcall: Need to fix unit test failure.

03:09 [130] $ cargo test -p dbs-upcall -- test_upcall_client_send_request
    Finished test [unoptimized + debuginfo] target(s) in 0.05s
     Running unittests (target/debug/deps/dbs_upcall-be6e9a9c8aea1888)

running 3 tests
test tests::test_upcall_client_send_request_error ... ok
test tests::test_upcall_client_send_request ... FAILED
test tests::test_upcall_client_send_request_without_result ... FAILED

failures:

---- tests::test_upcall_client_send_request stdout ----
thread 'tests::test_upcall_client_send_request' panicked at 'called `Result::unwrap()` on an `Err` value: Kind(WouldBlock)', crates/dbs-upcall/src/lib.rs:698:51
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- tests::test_upcall_client_send_request_without_result stdout ----
thread 'tests::test_upcall_client_send_request_without_result' panicked at 'called `Result::unwrap()` on an `Err` value: Kind(WouldBlock)', crates/dbs-upcall/src/lib.rs:730:51

failures:
    tests::test_upcall_client_send_request
    tests::test_upcall_client_send_request_without_result

test result: FAILED. 1 passed; 2 failed; 0 ignored; 0 measured; 17 filtered out; finished in 0.00s

Fixed in #167.


Fix Xml Code Comments in Jsrt headers

Minor: There are a few cases in the Jsrt header files where the code comments don't adhere to Xml or Xml Documentation conventions
this non-code commit fixes those cases.

Hi @Oceanswave, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Hi, I am closing and re-opening this PR to bump the CLA bot. Sorry for the inconvenience!

Hi @Oceanswave, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

Looking good. Thanks @Oceanswave !


QGIS plugin forms

Here are a couple of form mockups for the first couple of processing steps in the QGIS plugin.
Build Network Tool form
Calculate Network Metrics form
Please review and let me know what you think.

Sorry to be late on this one. I think the forms look good. Obviously when you code them things will get tweaked based on what Qt forms can and can't do but I think the general idea is good.


mq(ticdc): fix send message time cost duration metrics inaccurate

What problem does this PR solve?

Issue Number: close #7577
What is changed and how it works?
fix incorrect time duration calculation on send message.
Check List
Tests

Manual test (add detailed scripts or steps below)

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

/run-integration-test
/run-kafka-integration-test


Cannot run a training because of the name

Describe the bug
Hi,
first, thank you for Stardist!
I got a bug with the training, I am using the tutorial with my own data, but when I have to train the model, it threw this error :
ValueError                                Traceback (most recent call last)
Cell In[11], line 1
----> 1 model.train(X_trn, Y_trn, validation_data=(X_val,Y_val))

File ~/miniconda3/lib/python3.11/site-packages/stardist/models/model2d.py:405, in StarDist2D.train(self, X, Y, validation_data, classes, augmenter, seed, epochs, steps_per_epoch, workers)
    399 [(p-2*b) % d == 0 or _raise(ValueError(
    400     "'train_patch_size' - 2*'train_completion_crop' must be divisible by {d} along axis '{a}'".format(a=a,d=d) if self.config.train_shape_completion else
    401     "'train_patch_size' must be divisible by {d} along axis '{a}'".format(a=a,d=d)
    402  )) for p,d,a in zip(patch_size,div_by,axes)]
    404 if not self._model_prepared:
--> 405     self.prepare_for_training()
    407 data_kwargs = dict (
    408     n_rays           = self.config.n_rays,
    409     patch_size       = self.config.train_patch_size,
   (...)
    416     sample_ind_cache = self.config.train_sample_cache,
    417 )
    419 # generate validation data and store in numpy arrays

File ~/miniconda3/lib/python3.11/site-packages/stardist/models/base.py:342, in StarDistBase.prepare_for_training(self, optimizer)
    340 self.callbacks = []
    341 if self.basedir is not None:
--> 342     self.callbacks += self._checkpoint_callbacks()
    344     if self.config.train_tensorboard:
    345         if IS_TF_1:

File ~/miniconda3/lib/python3.11/site-packages/csbdeep/models/base_model.py:194, in BaseModel._checkpoint_callbacks(self)
    192 ModelCheckpoint = keras_import('callbacks', 'ModelCheckpoint')
    193 if self.config.train_checkpoint is not None:
--> 194     callbacks.append(ModelCheckpoint(str(self.logdir / self.config.train_checkpoint),       save_best_only=True,  save_weights_only=True))
    195 if self.config.train_checkpoint_epoch is not None:
    196     callbacks.append(ModelCheckpoint(str(self.logdir / self.config.train_checkpoint_epoch), save_best_only=False, save_weights_only=True))

File ~/miniconda3/lib/python3.11/site-packages/keras/src/callbacks/model_checkpoint.py:183, in ModelCheckpoint.__init__(self, filepath, monitor, verbose, save_best_only, save_weights_only, mode, save_freq, initial_value_threshold)
    181 if save_weights_only:
    182     if not self.filepath.endswith(".weights.h5"):
--> 183         raise ValueError(
    184             "When using `save_weights_only=True` in `ModelCheckpoint`"
    185             ", the filepath provided must end in `.weights.h5` "
    186             "(Keras weights format). Received: "
    187             f"filepath={self.filepath}"
    188         )
    189 else:
    190     if not self.filepath.endswith(".keras"):

ValueError: When using `save_weights_only=True` in `ModelCheckpoint`, the filepath provided must end in `.weights.h5` (Keras weights format). Received: filepath=models/stardist_lebatard/weights_best.h5

To reproduce
Basic tutorial for training model : https://github.com/stardist/stardist/blob/master/examples/2D/2_training.ipynb
I tried with different version of stardist from (0.80 to 0.85)
Expected behavior
I expect the command to train the model
Environment (please complete the following information):
os: Linux-6.5.0-26-generic-x86_64-with-glibc2.38
stardist: 0.8.5
csbdeep: 0.7.4
tensorflow: 2.16.1
WARNING:tensorflow:From /tmp/ipykernel_292456/1014409130.py:12: is_gpu_available (from tensorflow.python.framework.test_util) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.config.list_physical_devices('GPU') instead.
tensorflow GPU: False
Do you have any solution to solve it ? Thank you for your time.
IGuy

Hi everyone,
it seems like the bug came from Keras, after downgrading it to an older version.
All seems to work !
IGuy

We've released StarDist 0.9.0, which now support Keras 3 and TensorFlow 2.16+.


Added 22 books

A small contribution from my side, I hope this may help the community.

Please follow the structure for GitHub readme.

sorry about that I will fix it

I am closing this PR, I will re-structure the whole thing, and I will make a new PR with new changes only when I completed my self-review checklist ty


Include browser global for 'use strict' environments (minifications)

The locale/*js files in moment.js are not 'use strict'. Some projects will only include some of the locale files (as opposed to loading all the 80 translations in moment-with-locales.min.js). When minified, the locale/*js files might often be placed in a 'use strict' environment resulting in the 'this' object to be undefined. This fix adds 'window' as a paramter to the IIFE's in locale/*js files.

+1. Combining/minifying the locale files with other scripts that use strict mode is impossible right now because of this.

In moment 2.10.0 and up the UMD is generated automatically, with a header file and some magic. Can you check if the new header works or we have to patch it with this. It takes this from the global scope, which I think works even in strict mode.

@abhustoft can you please check if the current version (2.10.3) works as you'd expect?

All locale files are use strict as of 2.10.

Great, thanks!


Add ROS2 timers

This PR adds ROS2 timers and enables them to be triggered by waitsets.

Note that the upstream rclrs repo has an open PR that introduces async executors, and it will make some significant changes to how wait sets work: https://github.com/ros2-rust/ros2_rust/pull/421
Be ready for some merge conflicts down the road. I do have a plan to introduce timers on top of the async execution PR, but I'm currently waiting until that PR gets merged first. If you have a need for timers and are interested in jumping forward to the async execution implementation (at the risk of some API instability), then I'd be happy to make a branch that includes timers.

@mxgrey I appreciate the heads up, that's very useful to know. It looks like you've done a lot of great work on the async executor, that's a nice improvement.
We'll go ahead with the sync version in the short term, as that unblocks our immediate need for timers, but the less time we spend diverged from upstream the better.
If you're willing to do an async timer implementation that would be amazing. You're more than welcome to copy anything here, I would assume the Timer struct will be largely transferrable. Once the async execution is merged into main I might be able to help port timers across too.


Less parsing

Feels a bit rushed (hardcoded values woo) but isolated well from the rest of the code.

What if we had ITransform, pulled in via MEF so that there weren't specifically modifying FileTransforms to add CSS or PNG or 'something' processing.

What about something like this?
[TransformInfo(Extension = ".less", Output = ".css")]
public sealed class CssTransform : ITransform

Where the only method on the ITransform is to Process(string content)
We could also do a ProcessMany to support bundling, but that introduces different problems.


Add support for afl-fuzz testing

I tested this library under afl last night: total of 400M executions, ~400 code paths found, no crashes or hangs. Stopped when it was >30mins that any of the fuzzers had found a new path. This is the code I used to support the fuzzing, in case you'd like to easily re-run against any future changes.
(I didn't test under ASAN, as cross compiling with -m32 didn't work out of the box)
"make fuzz" creates a simple executable that de-serialises stdin
and re-serialises to stdout.
"make fuzz_testcases" extracts the smaller json test cases into
a testcases directory.
The library can then be fuzzed as follows:
CC=afl-clang-fast make fuzz
make fuzz_testcases
mkdir out
afl-fuzz -i testcases -o out ./fuzz

Thanks a lot!


discord bot

the discord bot just typing but it doesn't replay

Sorry for the late reply, just seeing this now. Can you share the contents of your ".env" file?


ReadQuery implementing JsonApiQueryRulesInterface

do you have some hints on enabling sparse fieldset?
    /**
     * @return RuleInterface[]|null
     */
    public static function getFieldSetRules(): ?array
    {
        // no field sets are allowed
//        return [];
        return [
            Schema::RESOURCE_ID     => r::stringToInt(r::moreThan(0)),
            Schema::ATTR_NAME       => r::asSanitizedString(),
            Schema::ATTR_FIRST_NAME => r::asSanitizedString(),
            Schema::ATTR_LAST_NAME  => r::asSanitizedString(),
            Schema::ATTR_NAME       => r::asSanitizedString(),
        ];
    }

does not show sparse fieldset by fields[users]=id,name
outputting:

{
    "errors": [
        {
            "status": "422",
            "title": "The value is invalid.",
            "detail": "The value is invalid.",
            "source": {
                "parameter": "fields"
            }
        }
    ]
}

return null on getFilterRules(), getFieldSetRules() does allow all field to be filterable and selective
but how to define specific fields?

Oh. Yes. I misused the syntax filter query rules on
spare  fieldset rule.
And thanks for the update of multiple sparse fieldset rules!


Two instances on the same page

Great work!
I need to use the feature twice on the same page. How can I make only one of the sections scroll?

Hi @ptohy. Thanks! I think you can just try giving a different name to this one wrapper-nav like wrapper-nav-ns, and it should work.


Add docker runtime support.

Adds passthrough support for the docker run --runtime parameter, allowing pipeline-level specification of  the target docker runtime. This is primarily intended to provide support for nvidia-docker in contexts where the default docker runtime can not, or should not, be reconfigured, allowing the use of CUDA devices within pipeline steps. A minimal example configuration is given at uw-ipd/buildkite-cuda.

[ ] Expand --runtime documentation.
[ ] Provide tutorial for basic configuration of cuda-enabled docker-based buildkite agent?

Thanks @asford! Looks like a great addition, and also the buildkite-cuda example is great. There's definitely room for a tutorial, or guest blog post, about running cuda jobs too.

Sounds great. Do you think the current documentation for this plugin needs any expansion?

Not really! Perhaps just linking to the docs with a title, rather than inlining the entire link? And then a test (hopefully it’s mostly copy and paste?). Then it should be good to merge!
We don’t assume we’re teaching you much about Docker itself here. So linking to the Docker docs is enough I think.
Let me know if you need a help with the test or doc update, happy to help!

Great, I've taken care of the test. I inlined the documentation link to match the current format, perhaps you can take on the documentation reformatting and/or split to a follow-up PR if needed?
I'm happy to expand the buildkite-cuda example into a more full-fledged example and/or blob post. There are, obviously, a few different ways to configure the system around nvidia-docker and it might be nice to provide a single reference covering this plugin as well as the docker-compose plugin. What's a good communication avenue to follow up with you? The buildkite-community slack?
docker-compose run tests
 ✓ Run command
 ✓ Run command without a workdir should not fail
 ✓ Pull image first before running the command with mount-buildkite-agent disabled
 ✓ Runs the command with mount-buildkite-agent disabled
 ✓ Runs command with volumes
 ✓ Runs command with environment
 ✓ Runs command with user
 ✓ Runs command with additional groups
 ✓ Runs command with network that doesn't exist
 ✓ Runs with debug mode
 ✓ Runs with custom runtime

11 tests, 0 failures

Awesome, thanks so much @asford! This has been released in v1.2.0.
A guest blog post is probably the way to go? Only because the tools might get out of date, and if it were an official tutorial we might not have the ability to keep it up-to-date. But if we publish it on https://building.buildkite.com (similar to https://building.buildkite.com/continuous-delivery-with-maven-buildkite-and-packagecloud-fc8c86f1e9b1) it could still help plenty of people? Want send me an email (tim@)? And I can invite you as a contributor to the Medium publication.


Product page

[ ] add filterign and Sorting Functionality
[ ] start working on Medicin page
[ ] Start working on farmer Product page.
[ ] collect all medicine data
[ ] collect all farmer product data
[ ]add this data to backend .


RSS Feed isn't updating when a change happens

I've watched a website for about a month that changes twice a week and haven't gotten anything added to my RSS feed. Here's what the feed looks like:
<?xml version='1.0' encoding='UTF-8'?>
<rss xmlns:atom="http://www.w3.org/2005/Atom" xmlns:content="http://purl.org/rss/1.0/modules/content/" version="2.0"><channel><title>changedetection.io</title><link>https://changedetection.io</link><description>Feed description</description><docs>http://www.rssboard.org/rss-specification</docs><generator>python-feedgen</generator><lastBuildDate>Wed, 02 Jun 2021 20:04:28 +0000</lastBuildDate></channel></rss>****

Did I forget to set something up? Looks like RSS works for others

@twiclo I added a specific test to check the RSS output.. seems to work fine, can you load your RSS url in another browser or command line and tell me if it works? maybe its cached in your browser (and we need to add some fix to stop the caching)

But yes.. seems to work for others
Can you tell me what the URL is? then i'll add it to my watches aswell

I don't see how a cache could last months. I did just open it again on my phone and one of the two watched sites was in the RSS however the one I added first which had an update 10 hours ago doesn't appear in the feed

@twiclo unfortunately there's nothing I can do for you unless

you provide your backup .zip file
tell me the URL you're watching


[Event Parameter Change] page 5853 "Get Pst.Doc-RtrnRcptLn Subform" - OnBeforeFindRecord

I requested changes in https://github.com/microsoft/ALAppExtensions/issues/20397 that were completed in 21.02. One small change was not implemented in page 5853 "Get Pst.Doc-RtrnRcptLn Subform".
Parameter Which should be set as var to make it work and act like the other seven pages:
[IntegrationEvent(false, false)]
local procedure OnBeforeFindRecord(var Which: Text; var ReturnReceiptLine: Record "Return Receipt Line"; var ReturnValue: Boolean; var IsHandled: Boolean)
begin
    //A108F000.Event21.02 Parameter Which as var
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52027.


TAS rock classification

Adds a function to retrieve the igneous rock classification using a TAS diagram

Thanks a lot - can you add a test for this and also add a documentation page?

Yes, work in progress

Added test and documentation for the TAS classification. Although I am not sure how to compile the doc.

Thanks a lot!
You can compile it locally by running make.jl in /doc. Once merged it will be compiled automatically.
In addition it would be really cool if you can add a plotting routine that plots that TAS diagram so you can add your own data on top.
Plotting routines in GeoParams are only made available when the Plots package is loaded. For that reason you would have to add your plotting function in Plotting.jl.

you now added 46 files (most related to building the webpage) - can you please remove those?

Progress on plotting function (need a bit more work).
I still have a problem when I compile the doc locally:
"Missing docstring. Missing docstring for GeoParams.TASclassificationData. Check Documenter's build log for details."
It seems it cannot find the documentation of the function, but I do export them and I have the docstring in the TAS routine. Do you have an idea of what I do wrong here?

yes, this is quite typical. I think it only employs the GeoParams version that is online.
yet, the docs/build directory should not be added to the repo. We should add that to the .gitignore file

Ah ok, thanks!

thanks for the modifications - great addition!


routes in api.php only works when ajax request is made by jquery

When i try to make an ajax request with axios or fetch api gives a 404 error

Did you write request route into api.php?

Hi @s9ul3ss!
In adition to @tepeumut suggest, can you verify that the HTTP_X_REQUESTED_WITH HTTP header is sent with the value xmlhttprequest?

Hello. I wrote it in api.php

I tried to send the X-Requested-With  header with the value  XMLHttpRequest and it worked but in fetch it didn't

in fetch api gives a 200 but doesn't show the json in the response

Can you share both your route php code and the javascript code please?

This worked for me, just go to system/core/input.php and comment line 836 and write return true.

cuando intento hacer una solicitud ajax me esta generando el error 503 en mi servidor de producción, solamente el error ocurre desde que empecé a usar Luthier CI, al desactivar el plugin funciona sin problemas hay alguna configuración adicional que se debe realizar?

When I try to make an ajax request it is generating  503 error on my production server, only the error occurs since I started using Luthier CI, when deactivating the plugin it works without problems, is there any additional configuration that must be done?

Hi @jalidchaer, thanks for using Luthier CI.
Please provide more information in order to help you with your issue. 5xx errors are server-related and can occur by many causes such bad server configuration, syntax errors, out of memory, etc.
If you are using apache, try to find the error.log file in your hosting file system and open a new issue with the related stack trace.

Hi @andersonsalas,
Thanks for your answer, the 503 error occurs in ajax requests, as far as I could it is due to a problem in the memory limit, is there any memory or resource limit that must be configured on the server for the use of the plugin?

PHP execution and memory limits can be configured with memory_limit and max_execution_time php.ini directives. However, keep in mind that those directives may be locked/disabled on shared hostings, so I'm assuming that you're using a VPS/dedicated server for your application.


app: Dependency updates

Remove and update some app/ dependencies.

mkdirp 3.0.1, https://github.com/isaacs/node-mkdirp/blob/main/CHANGELOG.md
tar 7.4.0, https://github.com/isaacs/node-tar/blob/main/CHANGELOG.md
Remove unused @types/semver
dotenv 16.0.1 to 16.4.5
Add app/tsconfig.json to formatted file list
jest 27.4.7 to 29.7.0
Remove open dep, use builtin openExternal
yargs 16.2.0 to 17.7.2
i18next to 23.12.1 and i18next-fs-backend to 2.3.1
i18next-parser 7.7.0 to 9.0.0
Remove electron-log dependency
electron-wix-msi 4.0.0 to 5.1.3, https://github.com/electron-userland/electron-wix-msi/blob/master/CHANGELOG.md
@electron/notarize 2.1.0 to 2.3.2
@babel/* deps
typescript 4.5.5 to 5.5.3
tsconfig: Bump target from "es6" to "2023"
Remove regenerator-runtime dependency
octokit 2.1.0 to 4.0.2
app: winget-create: Remove node-fetch import
plugins: plugin-management: Remove node-fetch import
app: Add node and npm minimum requirements
app: Fix babelrc to target node 20
app: Add plugin-management and convert to ts

Testing done

[x] each commit builds on CI
[x] ran the app, did some smoke testing
[x] tested pushing assets using push-release-assets.js

@illume I have just tested push-release-assets.js:

Create a new version draft, call it e.g. v0.1234 (DO NOT PUBLISH)
Run the push-release-assets.js script with any file, like node push-release-assets.js v0.123 somefile.zip

Worked fine just by moving to this branch.

Thanks @joaquimrocha

Rebased against main, because merge conflict.

The "Build Container" job failure is unrelated to this.

Rebased against main to fix conflict.

Fixed a lint issue.


empty buffer on breakpoint

Following the installation (https://github.com/vim-vdebug/vdebug/issues/361), here's what I get when php reaches the xdebug_breakpoint()

@BlackIkeEagle any idea why I get an empty buffer ? instead of seeing the file where the breakpoint was ?
I'm so confused as to how we're supposed to use vdebug, and lot of errors keep popping up
For example:

I type :Breakpoint on a line
I type :BreakpointWindow to see the id of my breakpoint
I type :BreakpointRemove 11000 to remove the breakpoint
⇒ getting the following error

- [ERROR] {Thu 14 2018 10:39:21} An error occured: <class 'neovim.api.nvim.NvimError'>
Traceback (most recent call last):
  File "/home/dori/.vim/plugged/vdebug/pythonx/vdebug/event.py", line 524, in dispatch_event
    Dispatcher.events[name](self.__session_handler).run(*args)
  File "/home/dori/.vim/plugged/vdebug/pythonx/vdebug/event.py", line 422, in run
    self.session_handler.breakpoints().remove_breakpoint_by_id(id)
  File "/home/dori/.vim/plugged/vdebug/pythonx/vdebug/breakpoint.py", line 56, in remove_breakpoint_by_id
    self.breakpoints[id].on_remove()
  File "/home/dori/.vim/plugged/vdebug/pythonx/vdebug/breakpoint.py", line 107, in on_remove
    self.ui.remove_breakpoint(self)
  File "/home/dori/.vim/plugged/vdebug/pythonx/vdebug/ui/vimui.py", line 203, in remove_breakpoint
    self.windows.breakpoints().remove_breakpoint(id)
neovim.api.nvim.NvimError: b'Index out of bounds'

When I press  to continue the code execution, my page is shown. After typing  to close vdebug, the vdebug panes don't close. When trying to close them manually, I get

I had forgotten to set the let g:vdebug_options.path_maps to match remote and host files.


Commerce Settings

Are there any additional configuration to get this to work with commerce apart from adding:
services.AddSitemapsCommerce();
In the configuration docs (https://github.com/Geta/geta-optimizely-sitemaps/blob/master/docs/editor-guide.md) it talks about the format and how its currently standard or mobile. If I do check the "standard and commerce" option and run the scheduled job my commerce product pages still do not show in the sitemap.xml.
Is something missing?

@marisks Any input?

As I remember, it is enough to follow configuration documentation here: https://github.com/Geta/geta-optimizely-sitemaps/tree/master#configuration
So call AddSitemaps, AddSitemapsCommerce, and make sure that razor pages are mapped.

@marisks Yeah unfortunately its not working. I am guessing it could be due to it being multi site, multi language, multi catalog it can't get the correct one.
If I create my own sitemap and commerce generator by implementing the public virtual member, eg:
public class StandardAndCommerceSitemapXmlGenerator : CommerceSitemapXmlGenerator, ICommerceAndStandardSitemapXmlGenerator
{
//
}

And just use the default from the repo as an override:
        protected override IEnumerable<XElement> GetSitemapXmlElements()
        {
            var rootContentReference = _referenceConverter.GetRootLink();

            if (SitemapData.RootPageId != Constants.DefaultRootPageId)
            {
                rootContentReference = new ContentReference(SitemapData.RootPageId)
                {
                    ProviderName = "CatalogContent"
                };
            }

            var descendants = ContentRepository.GetDescendents(rootContentReference);

            return GenerateXmlElements(descendants);
        }

Then descendants is always empty.

Have you configured RootPageId for the catalog you want to generate a sitemap?


Fixes deformed emoji in pull request reviews

As title.
See bug on try.gitea.io: https://try.gitea.io/saitho/test2/pulls/2/files
I might need to recompile CSS... ^^

CI failed

No idea on that CI failure, but this looks like the proper place to add that rule:
https://github.com/go-gitea/gitea/blob/fe1a6a2d9ae8ecb2224b00dc8adc920c7388c28c/public/less/_markdown.less#L275

I didn't generate the CSS by command because I didn't have NPM installed
(for some reason) and I was on the train.
Might be some whitespace missing. Will take a look at that tomorrow and fix
that. ;)
silverwind<EMAIL_ADDRESS>schrieb am Mo., 2. Sep. 2019, 18:20:

No idea on that CI failure, but this looks like the proper place to add
that rule:
https://github.com/go-gitea/gitea/blob/fe1a6a2d9ae8ecb2224b00dc8adc920c7388c28c/public/less/_markdown.less#L275
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/go-gitea/gitea/pull/8047?email_source=notifications&email_token=ABXUS7WSKCVALZ6PYB4VXX3QHU4NZA5CNFSM4ISWKXKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5WGOHI#issuecomment-527197981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABXUS7VJCL2CGEIYO6GGJWDQHU4NZANCNFSM4ISWKXKA
.

silverwind<EMAIL_ADDRESS>schrieb am Mo., 2. Sep. 2019, 18:20:

No idea on that CI failure, but this looks like the proper place to add
that rule:
https://github.com/go-gitea/gitea/blob/fe1a6a2d9ae8ecb2224b00dc8adc920c7388c28c/public/less/_markdown.less#L275
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/go-gitea/gitea/pull/8047?email_source=notifications&email_token=ABXUS7WSKCVALZ6PYB4VXX3QHU4NZA5CNFSM4ISWKXKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5WGOHI#issuecomment-527197981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABXUS7VJCL2CGEIYO6GGJWDQHU4NZANCNFSM4ISWKXKA
.

Not related to @silverwind  's suggestion, but about the CI failing: try pulling from the latest changes in master and push this again.

Am I getting this right? The newline is missing from the original file?

Codecov Report

Merging #8047 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8047      +/-   ##
==========================================
- Coverage    41.6%   41.59%   -0.01%
==========================================
  Files         480      480
  Lines       64163    64163
==========================================
- Hits        26696    26691       -5
- Misses      34008    34011       +3
- Partials     3459     3461       +2

Impacted Files
Coverage Δ

models/pull.go
54.85% <0%> (-0.46%)
:arrow_down:

models/issue.go
49.49% <0%> (-0.24%)
:arrow_down:

models/repo_list.go
74.14% <0%> (+0.97%)
:arrow_up:

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 8ae96a3...189300d. Read the comment docs.

Am I getting this right? The newline is missing from the original file?

Looks right. It is a generated file that should not be edited by hand and from a minifier perspective, it makes sense to not include that newline.

@silverwind, about your suggestion: is this ready for approval?

make LGTM works


When the results will be out ?

Hi, @msaligane when the results will be out?
Is it already out?

I think it is closes by fca543a


[Bug]: Default QWEB Template not working Odoo 17

Description
I am experiencing an issue with the default QWEB templates in Odoo 17. When attempting to send an offer or invoice, the system fails, and the QWEB template does not render correctly.
Minimal Reproduction (if possible, example repository)

Install Odoo 17.
Go to the Sales or Invoicing module.
Create a new offer or invoice.
Attempt to send the offer or invoice via email.
Observe the error related to the QWEB template rendering.

Exception or Error
No response
Version
v4.0.0-beta.323
Cloud?

[ ] Yes
[X] No

Couldnt reproduce here. Whats the error looks like? Doesnt seem to be a Coolify problem

Odoo probably need SMTP configuration for sending emails. That is not set by default.


README: Add usage instructions for Jazzy

The syntax mentioned in the README no longer works there.

Thanks for updating the README!


dnd_start: data.event.target not always accurate

dnd_start.vakata has some kind of bug because data.event.target is not always accurate.
It is ok most of the time. but not always. User may clickdown on a node to drag it and in the end the dragged node may not be the one the event started with, but it's neighbour node

The dnd events are low level events emitted from the DND helper, which is not related to jstree in any way - its events do not have the concept of node ID, etc - they only work with what was clicked in the DOM. Usually you will not have to deal with those events when working with a tree only (move_node, copy_node and check_callback will suffice), but if you have to implement something that needs the low level events it is up to you to normalize the node (for example using get_node with the second param set to true).


__VERDACCIO_BASENAME_UI_OPTIONS is missing version field

Your Environment

verdaccio version: 5.22.1
os: [linux]
platform: [docker]

Describe the bug
After updating from 5.14.0 to 5.22.1, I noticed that verdaccio version is no longer displayed in the website footer. And __VERDACCIO_BASENAME_UI_OPTIONS global variable doesn't have a version field as well.
To Reproduce
Just run verdaccio via Docker image v5.22.1.
Expected behavior
Version is displayed in the footer and in the __VERDACCIO_BASENAME_UI_OPTIONS.

Thanks, yeah probably 😅 I screw up something else again ... refactoring not easy need more test 😬


Problem Running Container

I have followed the instructions in the readme, but I'm missing something. When the container(s) run, I get the following error.
occi-ay-worker | Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/compute-worker/archiyou-compute-worker/build/index.js' imported from /compute-worker/
occi-ay-worker |     at new NodeError (node:internal/errors:387:5)
occi-ay-worker |     at finalizeResolution (node:internal/modules/esm/resolve:315:15)
occi-ay-worker |     at moduleResolve (node:internal/modules/esm/resolve:907:10)
occi-ay-worker |     at defaultResolve (node:internal/modules/esm/resolve:1115:11)
occi-ay-worker |     at nextResolve (node:internal/modules/esm/loader:163:28)
occi-ay-worker |     at ESMLoader.resolve (node:internal/modules/esm/loader:841:30)
occi-ay-worker |     at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
occi-ay-worker |     at ESMLoader.import (node:internal/modules/esm/loader:525:22)
occi-ay-worker |     at node:internal/modules/run_main:58:28
occi-ay-worker |     at loadESM (node:internal/process/esm_loader:91:11) {
occi-ay-worker |   code: 'ERR_MODULE_NOT_FOUND'

I'm far from a Docker expert, so I'm not sure what to do next.
I also get an connection error for Celery. Do I need to set the default password somehow and then put that in the .env file to make it work?

It works now, thanks!


Beta server does currently not assign hybrid heatpumps when present in ETM scenario.

Summary
When creating a LES on the beta server that contains other technologies for space heating than just combi heating the other technologies are not represented in the technology profile.
The number of units for combi heating is decreased proportionally, but the remaining part of the heating demand is not represented in the technology profile.
On the live server hybrid technologies are assigned in ETMoses when they are present in ETModel.
The following might be a different issue:
However, when I select a technology that is not present in ETMoses (for example 50% Micro-CHP). The number of units of "combined network gas" goes down, but no other technology pops up.
Example
My scenario has 1657 households.
When I assign for space heating 100% "Condensing combi boiler" and for Hot water 100% Condensing combi boiler my technology profile contains

1657 units "Households water heater combined network gas"
1657 units "Households space heater combined network gas"

When I assign in beta ETModel for space heating 50% Condensing combi boiler and 50% hybrid heatpump and for hot water 100% condensing condensing combi boiler
I get:

1657 units "Households water heater combined network gas"
828 units "Households space heater combined network gas"
and no other heating technologies.

When I assign in ETModel for heating 50% Condensing combi boiler and 50% hybrid heatpump
And for Hot Water 50% condensing combi boiler and 50% fuel cells

828 units "Households water heater combined network gas"
828 units "Households space heater combined network gas"
and no other heating technologies.
So, the hybrid electrical and gas were not assigned.
And the fuel cells were not assigned

CC: @dennisschoenmakers

@antw can you look into this?


Turn off debugging

I started noticing a ton of random logging from babel-preset-env after updated the dependencies of an old project, and traced it down to a this line. I'm not too familiar with this preset, so I'm not sure if always-on debugging is a feature or a bug. In case it's an oversight, here's a quick PR fixing it (:

It's a feature, but this is also a duplicate of #18.

Welp, my bad, sifted through issues but not PRs before opening.
Thanks for the clarification!


[ODS-4583] Bring in MetaEd artifacts for API release version

@jamessmckay   I updated all 3 Extensions project output from Atom build .  Note there is no change in Homegraph project