Add multi_line_output mode for more compact formatting

Could we add a mode similar to 0/grid mode, but that uses the fixed indent width for subsequent lines? Mode 0 looks like this:
from third_party import (lib1, lib2, lib3,
                         lib4, lib5, ...)

I would like this (where indent=2):
from third_party import (lib1, lib2, lib3,
  lib4, lib5, ...)

This would be more compact, and can reduce line count. I would be happy to attempt a PR; pointers on where to look would be welcome. isort has been a great improvement to my workflow; thank you for building it!

FYI, the current mode 0 matches the pep8 style guide whereas the suggestion for indent does not.

While I don't have the bandwith to add support for this requested feature myself, I will accept a pull request that adds it. The correct place to add this functionality is here: https://github.com/timothycrosley/isort/blob/develop/isort/wrap_modes.py#L46

Thanks @timothycrosley. I'll try to find time to take a stab at this. Line 36 says that name and order are significant; should I add a new @_wrap_mode decorated function to the end, after noqa?

@gwk,
Glad to hear! Look forward to the pull request. Yes that's correct, right after noqa.
Thanks!
~Timothy

Hi there,
I looked into this one a bit and it can be almost achieved with the HANGING_INDENT or 2 mode. The only problem is that HANGING_INDENT hard-codes the usage of \ and ignores the --use-parentheses completely.
This problem seems to occur in the other modes as well, but the other way around ie. most of them hard-code the usage of parentheses.
I'm guessing that people don't notice this, because they probably use a profile and the profiles tend to use use_parentheses:  True or they just prefer using parentheses. However, turning off use_parentheses isn't possible for most modes.
I made some modifications in my fork that makes HANGING_INDENT mode "respect" the --up option.
With this change, and an indent of 2, you can achieve what the OP wanted:
$isort -d test.py -m HANGING_INDENT -i 2 --up
from third_party import (lib1, lib2, lib3, lib4, lib5, lib6, lib7, lib8, lib9,
  lib10, lib11, lib12, lix1, lix2, lix3, lix4, lix5, lix6, lix7, lix8, lix9,
  lix10, lix11, lix12)

Please let me know what you think about this approach. If it is OK I would prefer to add some additional tests and/or update some tests to reflect the changes.
It is also probably a good idea to update the other modes to use backslash-joining by default and use parentheses only if use_parentheses is True. Would you agree?
Maybe that could be done as a separate issue and PR?
Thanks,
Tamas

This looks like what I was hoping for. My only question is, does this insert parentheses unconditionally or only when the line needs to wrap?

Will put more details here later, but, the use_parenthesis is not meant for wrap modes but only meant for single line wrapping with one import. isort has a release gaurentee that the same options will return the same output, other than fixing large errors. We could think about changing that in the future for 6.x.x but for the meantime code can be refactored to reuse logic, but a different wrap mode rather then utilizing the --up would be needed for this ticket

Some additional information:
Here is the official release policy: https://timothycrosley.github.io/isort/docs/major_releases/release_policy/#formatting-guarantees
Here is the updated documentation on the use_parentheses option to clarify its intended and effective usage: https://timothycrosley.github.io/isort/docs/configuration/options/#use-parentheses
If you look at where it's used, it only gets used for individual line wrapping, but never effects wrap modes: https://github.com/timothycrosley/isort/search?q=use_parentheses&unscoped_q=use_parentheses
Thanks!
~Timothy

Hi @timothycrosley
All that makes a lot of sense.
What would be a good name for the new wrap mode?
I assume it will have to be added as the last mode to keep things backward compatible.
As a side note, it might be just me, but the mode names don't really tell me what the difference is between them.
Wouldn't it make sense to have a generated doc page when you show the modes with some examples to see what the difference is without having to try them out one-by-one?
Thanks,
Tamas

This looks like what I was hoping for. My only question is, does this insert parentheses unconditionally or only when the line needs to wrap?

Hi @gwk
It inserts the parentheses just when the lines wraps.
Thanks,
Tamas

What would be a good name for the new wrap mode?

Open to any name that makes sense, your initial observations make me think HANGING_INDENT_WITH_PARENTHESES might be a good candidate - though it is very verbose.

I assume it will have to be added as the last mode to keep things backward compatible.

Yes that's correct! If that assumption isn't commented in the source file via comments and/or via tests, another improvement that could be made is clarifying it.

As a side note, it might be just me, but the mode names don't really tell me what the difference is between them.
Wouldn't it make sense to have a generated doc page when you show the modes with some examples to see what the difference is without having to try them out one-by-one?

:100: % This. It would be really nice to have autogenerated docs, under the config tab of the new documentation website. Maybe pick a long example multi import statement and auto run it through all modes. Again this is just a bonus improvement that could be done.
Thanks for all the thought put into this!
~Timothy

I like HANGING_INDENT_WITH_PARENTHESES and although is long, there is precedent, it comes only as shared  :2nd_place_medal:  :smile:
VERTICAL_PREFIX_FROM_MODULE_IMPORT
VERTICAL_HANGING_INDENT_BRACKET
HANGING_INDENT_WITH_PARENTHESES
VERTICAL_GRID_GROUPED_NO_COMMA
VERTICAL_HANGING_INDENT
...

We have the numeric short-hands OFC, maybe worth introducing short-hands based on the first character of the modes that have more than 1 word? Would be similar to what you have for normal options already, backward compatible and there are no conflicts ATM:
 0, GRID
 1, VERTICAL
 2, HANGING_INDENT, HI
 3, VERTICAL_HANGING_INDENT, VHI
 4, VERTICAL_GRID, VG
 5, VERTICAL_GRID_GROUPED, VGG
 6, VERTICAL_GRID_GROUPED_NO_COMMA, VGGNC
 7, NOQA
 8, VERTICAL_HANGING_INDENT_BRACKET, VHIB
 9, VERTICAL_PREFIX_FROM_MODULE_IMPORT, VPFMI
10, HANGING_INDENT_WITH_PARENTHESES, HIWP

That's another idea that can result in another issue/PR though :smile: , going back to this issue there is still one related problem left.
The current hanging indent mode doesn't take --tc into consideration. That must be a bug!
Am I fixing it while I'm at it or would you prefer that to be separated into another separate issue?
Thanks,
Tamas

OK, I think this is done in the PR linked above.
Usage:
$ isort -d test.py -m HANGING_INDENT_WITH_PARENTHESES -i2
from third_party import (lib1, lib2, lib3, lib4, lib5, lib6, lib7, lib8, lib9,
  lib10, lib11, lib12, lib13, lib14, lib15, lib16, lib17, lib18, lib19, lib20,
  lib21, lib22, lib23, lib24, lib25, lib26, lib27, lib28, lib29, lib30)

I think wrap_modes.py could be refactored a bit, to remove some of the duplication and make the intent a bit clearer, but I refrained, in order to get the new feature in.

This was just deployed with version 5.2.0 of isort (thanks @sztamas!).
~Timothy

@sztamas & @timothycrosley, I just want to say thank you for implementing this. I finally got back to working on my library and this did the trick!


Update teams-for-vdi.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/5440
Added note.

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

@LanaChin PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Hi @lindspea - The PM says that this feature is not available yet for VDI so am closing this PR without merging.


⚠ Samsung has degraded performance

In ef17b6b, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2604 ms

Resolved: Samsung performance has improved in ecb8fae.


Fastest way to get an ISeachResult ?

Do some controlled experiments testing this. Have used this:
$Searcher = New-Object -ComObject Microsoft.Update.Searcher
$ISearchResult = $Searcher.Search($Criteria)

and https://gist.github.com/bklockwood/716d9f582dd78a9e6e9e which is really the above but using a different search service (download wsusscn2.cab rather than connecting directly to windows update servers).
Time to take a much closer look at the methods & properties of IUpdateSearcher

Searcher.Online = $false

seems to speed things up. ~9 sec without this; ~4 secs with (when running remotely)


[feature request] Promote host network feature annotation as the spec fields

What would you like to be added:
we can promote the hostnetwork feature annotation as the spec field from annotation
Why is this needed:

/assign


Add style to white section

Let me know what you think about that have never work with .tmTheme file always change everything inside the base.less file

The updated atom version can be found at https://github.com/SintrumIT/oceanic-next-italic-white-atom

Yep I just updated atom and working fine ;)


adapt David Hill (TRU)'s randomForest example to RS data

comparison:
https://cran.r-project.org/web/packages/randomForest/randomForest.pdf


[Bug]: Update from 2.x to 3.x: Errors if you do not set any custom preferences

Bug Description
Hi there,
fyi, i just updated from a '2.x' version to the latest version, and then for example the 'o' keyword to search notes, 'osetup', etc. etc. didn't work anymore;
Turns out, if the Alfred prefs.plist is missing because you did not set any custom preferences in the Alfred workflow, you get all kinds of errors.
Solution: set at least 1 Alfred Workflow setting, so prefs.plist will be created.
Hope that helps :-)
Relevant Screenshot
No response
To Reproduce
See description and debug log...

Workflow Configuration 2
Not relevant...
Version & System Information

version info
-------------------------------
Metadata JSONs: 3/3
Metadata String Length: 32075
-------------------------------
WORKSPACE DATA
'workspace' exists
'workspace.json' exists
-------------------------------
INSTALLED VERSION
macOS: 13.0.1
Alfred: 5.0.5
---
Obsidian: 1.0.3
This Workflow: 3.2.3
Advanced URI Plugin: 1.31.1
Metadata Extractor: 1.0.2
-------------------------------
LATEST VERSION
Obsidian: 1.0.3 (Insider: 1.0.3)
This Workflow: 3.2.3
Advanced URI Plugin: 1.31.1
Metadata Extractor: 1.0.2
-------------------------------
INTERNAL WORKFLOW CONFIGURATION
Vault Path: /Users/david/Documents/Obsidian/Obsidian.David
Vault Name (encoded): Obsidian.David
-------------------------------

Make sure you have done the following

[X] I have updated to the latest version of this workflow.
[X] I have read the Troubleshooting Documentation.
[X] If the o… main search does not work, I have tried rerunning osetup and it still does not work.
[X] If I updated to version 3+ for the first time, I have disabled migrating workflow cutomizations. (If you haven't, you can simply re-download and install the latest release from GitHub and uncheck Migrate my workflow customizations.)

That is indeed weird. I can only guess that this is some sort issue with how Alfred migrates to Alfred 5. (Therefore most likely unrelated to this workflow, so I am closing this, but thx for the report.)

hmm, will have to look into it at some point then

Could you try a "dummy" prefs.plist with the following save it the workflow folder after installation?
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict/>
</plist>

and have a look whether this will make it work?

tested and does not occur anymore with recent versions. :)


Update make_timeseries for pandas deprecation

In [3]: dd.demo.make_timeseries()
/Users/taugspurger/sandbox/dask/dask/dataframe/io/demo.py:91: FutureWarning: Creating a DatetimeIndex by passing range endpoints is deprecated.  Use `pandas.date_range` instead.
  freq=partition_freq))
/Users/taugspurger/sandbox/dask/dask/dataframe/io/demo.py:45: FutureWarning: Creating a DatetimeIndex by passing range endpoints is deprecated.
Dask Name: make-timeseries, 11 tasks

Switching to pd.date_range should be fine.

The demo module should also be deprecated.  The main API entry point for this function is now dask.datasets.timeseries.

Can I work on it?

@mrocklin i resolved the warning part. Can you guide me what you mean by deprecating the demo module. Does it need to be shifted somewhere else?

@mrocklin i resolved the warning part. Can you guide me what you mean by deprecating the demo module. Does it need to be shifted somewhere else or do i only change the main entry point of the API in setup.py. But as i see it should be dataframes as it is named that.
Can you guide me more on this? I would be more than thankful.

Can you guide me what you mean by deprecating the demo module

Ideally, a warning would be emitted when any of the functions in dask.dataframe.demo is called. For some, like make_timeseries, the message would instruct people to use dask.datasets.timeseries. Others, like make_string, may just be removed (or perhaps moved to datasets.py, not sure).

And we can leave the deprecation to a separate module.

Apologies for missing your post @liabifano, it looks like @HSR05 has started on this.
Let me know if you want help finding / starting on another issue!


OPENTOK-32575: Electron app

What is this PR doing?

Electron app project at /electron
New npm scripts npm run electron, npm run electron-build
Opens the meet:// protocol in a hidden iframe for safari users which launches the app if it is installed
Redirects safari users to /electron/download which links to the .dmg installer at https://s3.amazonaws.com/artifact.public.tokbox.com/meet/meet-electron.dmg

How should this be manually tested?

Use the official signed build from s3 via the app
a. Run the app locally
b. Open a room in safari
c. Download and install the dmg from the link
Test a manual launch of the app
a. Launch the app manually
b. Check you can publish and subscribe, and that screensharing works (screensharing will only work when https://github.com/opentok/webrtc-js/pull/2130 has merged and deployed to tbdev)
Test launching the app via safari
a. Navigate to a room with safari, ensure the app launches
Build the app locally
a. Do a quick run with npm run electron
b. Uninstall the official build you may have installed earlier
c. Build with npm run electron-build
d. Install using the locally built dmg at /electron/build/
e. Ensure you can start the locally built app

What are the relevant tickets?
OPENTOK-32575

This appears to be affected by: https://github.com/travis-ci/travis-ci/issues/3477
Already tried closing & re-opening. I might start a fresh PR later if this doesn't clear up, hard to say whether that would make a difference though :/.


AlertDialog properties not found and crash

Not all of AlertDialog properties are found:

Type '{ children: Element; leastDestructiveRef: MutableRefObject; isOpen: boolean; onClose: () => void; isCentered: true; }' is not assignable to type 'IntrinsicAttributes & IAlertDialogProps & { ref?: any; }'.
Property 'isCentered' does not exist on type 'IntrinsicAttributes & IAlertDialogProps & { ref?: any; }'.ts(2322)
Me imports:
import { Pressable, AlertDialog, Text } from 'native-base';
Also I have a crash after close AlertDialog and moving by stack:

but maybe it is same problem.
Android_Pixel_4_API_30
"native-base": "3.0.7",
"react-native": "0.64.2",
node v14.15.4
yarn v1.22.10

Steps to reproduce crash:

Go to N screen in stack (react-navigation)
Open AlertDialog
Go back
Return to N screen

@ugar0ff  @kytosai This issue has been resolved in version 3.3.0 or later. Thanks for reporting.


Replace useless NilAccessDefect with useful LibraryError

First of all, thank you for yet another great library!
I wanted to contribute at least a bit, by fixing an issue I found when testing this library. 😃

Thank you for this PR! My questions:

How do you let LoadLibraryA(cname) raise a NilAccessDefect? bad DLL?
Is there other codes that cause useless exception instead of LibraryError for bad DLL?

Thank you for this PR! My questions:

How do you let LoadLibraryA(cname) raise a NilAccessDefect? bad DLL?
Is there other codes that cause useless exception instead of LibraryError for bad DLL?

To be honest, I'm still puzzled. After this Patch, it's telling me that it can't find "KERNEL32.dll" -- even though, this should be impossible, as far as I understand.
Yes, there are other places in the code that could be improved by replacing NilAccessDefect with something meaningful, but I did not know about the others yet, when I opened this PR.

If you think this method for making more useful exceptions is helpful, perhaps it would be even better if all places where a NilAccessDefect could possibly be found would be improved by a more meaningful exception, in the same way as done in this commit (try catch all the possible failures)?
To put all this in context: I was playing around with the openssl example, because I wanted to test if I can embed the library into the executable with memlib. However, I am still struggling to get it to work.


Publishing on PyPI

Adds a workflow to publish allowing pip install ebcc

Codecov Report
Patch and project coverage have no change.

Comparison is base (7a232dc) 99.49% compared to head (a8ef707) 99.49%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   99.49%   99.49%
=======================================
  Files          51       51
  Lines       76248    76248
  Branches      709      627   -82
=======================================
  Hits        75860    75860
  Misses        265      265
  Partials      123      123

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


How do we reset the query builder to default value?

Is there any helper method to reset/clear the query builder?

They only two method allower are removeRule and removeRuleSet, you could make a reset/clear button with click event like this.
HTML Example
<query-builder [formControl]='queryCtrl'></query-builder>
queryCtrl is a FormControl
Example of reset :
reset() {     this.queryCtrl.setValue({       condition: 'and',       rules: []     })   }


FB volume shoots to 100%.
btw, if left in layout, Template.html continues to trigger max volume at points while playing.
really not cool.

I commented it out in the next version,


Enhance 72 about results

content updates
error message updates
htmlmin for build

@wpears - once this one is merged I'll add the next, which switches to Mapbox geocoder.

Just that tiny typo, fix it in the next PR, eh? ;)


[Columns] Fix check for child as lazy node Column

Closes #835
Fix based on this comment. As the check now uses internals from a canary version of React any future updates to the payload format could break the check.

Good enough! Thanks


Add documentation for HTTP proxies

This provides documentation for #9951, already merged in master.
Replaces #10133

Ping @jfrazelle!

Yay thank you! LGTM

LGTM - @fredlf @jamtur01

LGTM

Well that was fast, thanks! :+1:

@icecrime @jfrazelle I think we should add the help message in the CLI return message when client can't connect to the daemon due to the proxy and suggest then what to do.Maybe something like this FATA[0000] Error response from daemon: 404 page not found, if you set http_proxy or https_proxy, you can add /var/run/docker.sock to no_proxy and try again. Because most people don't view the docs often especially for those have the experience in using docker, and in this way they konw what to do when meet this situation.

See the only problem is not everyone is going to want to setup a proxy so
pushing them towards that in an error message can be misleading... I'm not
sure exactly what to do tho because I agree many might not see the docs,
but I don't know how we can know for sure they want a proxy
On Mon, Jan 19, 2015 at 5:21 PM, leijitang<EMAIL_ADDRESS>wrote:

@icecrime https://github.com/icecrime @jfrazelle
https://github.com/jfrazelle I think we should add the help message in
the CLI return message when client can't connect to the daemon due to the
proxy and suggest then what to do.Maybe something like this FATA[0000]
Error response from daemon: 404 page not found, if you set http_proxy or
https_proxy, you can add /var/run/docker.sock to no_proxy and try again.
Because most people don't view the docs often especially for those have the
experience in using docker, and in this way they konw what to do when meet
this situation.
 So you can add the help message in the CLI, Thank you!

 So
you can add the help message in the CLI, Thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10192#issuecomment-75486807.

@jfrazelle
What version of docker is this patch going to be released in?

the one released today, it might have also been in 1.5.0 i cant remember
off the top of my head
On Thu, Apr 16, 2015 at 6:46 AM, Piotr Dobrogost<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle
What version of docker is this patch going to be released in?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10192#issuecomment-93737819.

@jfrazelle
Do you refer to v1.6.0-rc7? If so then I should have been more precise and specify that I'm concerned with stable releases.

@piotr-dobrogost you're in luck; 1.6 was just pushed out the door


What happened to old templates?

I have been using rx resume for quite a while now, but my resume is built on the older version of the template which is not available in the current verison and it breaks up the layout. Is there any way we can still use old templates? please let me know if someone has information about this.
Thanks!

Having similar problems importing from an old .json file

You can make use of https://v2.rxresu.me for the time being :)

Thanks!

Hi! Could anyone tell me if there is a way i can use the previous templates? i have the same problem like @kameshkotwani
Thanks in advance.


Burlap incompatibilities

https://classroom.udacity.com/courses/ud600/lessons/3780788560/concepts/40374085350923
At the above link it refers that in order to create the initial state of a graph node you execute this command:
GraphDefinedDomain.getState(domain, 0)
But getState does not exist as a static method at the current Burlap library.
What is the version I am seeing? How much Burlap has changed since then?
And where could I find a migration guide?
Thank you

It's probable that the Udacity videos are on version 2. If you want to roll back, check the github branches for a version 2 to use. A list of many of the big changes between 2 and 3 can be found on this BURLAP update: http://burlap.cs.brown.edu/updates.html
The biggest change is probably that the State interface was changed and greatly simplified to support a much larger range of types of states/observation datatypes. In the case of GraphDefinedDomain, rather than use a static-class constructor (which set up and initialized OO-MDP classes and attributes), you can now just instantiate the specific graph state as follows:
State s = new GraphStateNode(0);

(for graph node 0)
You can find a simple example usage in the domain's source file (most domains have a simple example in their main source file). Lots of other information is also in its Java doc.
I will close this, but feel free to ask other questions.


fixed spelling 'drectory' -> 'directory'

see https://github.com/microsoft/azure-pipelines-extensions/issues/898

see https://github.com/microsoft/azure-pipelines-extensions/pull/852
this PR only fixes the spelling, no functional change


Add Thermal Thevenin Example (Remove Params from Others)

This issue is to remove the thermal parameters from non-thermal Thevenin examples: https://github.com/pybop-team/PyBOP/blob/9e15f90b75c4f5fb04cb17c0807dd0b5e1a2e6fb/examples/scripts/ecm_CMAES.py#L24
At the sametime, let's add a Thevenin with thermal submodel example with those corresponding parameters. For more information on PyBaMM's thermal submodel see: https://docs.pybamm.org/en/latest/source/examples/notebooks/models/thermal-models.html

@Dibyendu-IITKGP, are you be able to take this one on?

Not yet. I'll do it.


v0.8.0: Update default cache-control headers

I'm improving the default cache-control headers to:

Provide better performance by continuing to serve (possibly stale) pages straight out of the cache for a full day after they were first cached
Provide better freshness by starting to refresh caches only 1 minute after they were first cached
Provide protection from temporary errors for 5 minutes after the cache expires

More details in https://github.com/canonical-web-and-design/canonicalwebteam.flask-base/issues/41, https://github.com/canonical-web-and-design/maas.io/issues/559 and this comment on an internal doc.
The new default cache-control instruction will be:
max-age=60, stale-while-revalidate=86400, stale-if-error=300

Each can be overridden individually, leaving the others intact, or each can be set to None to remove it.
Rationale
max-age=60
Hard-cache for a minimal amount of time so content can be easily refreshed.
The considerations here are as follows:

To avoid caching headaches, it's best if when people need to   refresh a page (e.g. after they've just done a release) they
can do so by simply refreshing their browser.
However, it needs to be long enough to protect our applications from excessive requests from all the pages being generated (a factor of the number of cacheable pages served by the application and the frequency with which we generate new caches)

1 minute seems like a good compromise.
stale-while-revalidate=86400
stale-while-revalidate defines a period after the cache has expired (max-age) during which users will get sent the stale cache, while the cache updates in the background. This means that users will continue to get speedy (potentially stale) responses even if the refreshing of the cache takes a while. The content could be up to a day out of date, if traffic is incredibly low, but simply refreshing the page again should yield the fresh content.
We want this to be pretty long, so users will still get a quick response (while triggering a background update) for as long as possible after the cache has expired.
An additional day will hopefully be long enough for most cases.
stale-if-error=300
stale-if-error defines a period of time during which a stale cache will be served back to the client if the cache observes an error code response (>=500) from the backend. When the cache receives a request for an erroring page, after serving the stale
page it will ping off a background request to attempt the revalidate the page, as long as it's not currently waiting for a response.
We set this value to protect us from transitory errors. The trade-off here is that we may also be masking errors from ourselves. While we have Sentry and Greylog to potentially alert us, we might miss these alerts, which could lead to much more serious issues (e.g. bringing down the whole site). The most clear manifestation of these error would be simply an error on the site.
So we set this to 5 minutes following expiry as a trade-off.
Fixes https://github.com/canonical-web-and-design/canonicalwebteam.flask-base/issues/41
QA
I wrote a fairly thorough test of this functionality, which seems to pass, so if you read the test you may be satisfied.
You can also go to https://ubuntu-com-9425.demos.haus/ (from this PR) and inspect the headers that are returned, check they behave as expected.
$ curl -I https://ubuntu-com-9425.demos.haus/  # The homepage sets all its own cache instructions
...
cache-control: max-age=61, stale-while-revalidate=90, stale-if-error=300

$ curl -I https://ubuntu-com-9425.demos.haus/openstack  # The openstack page doesn't
...
cache-control: max-age=60, stale-while-revalidate=86400, stale-if-error=300

@jpmartinspt can you have a look at this please

@nottrobin if you go to a page where we set the cache headers so the response is not cacheable like /advantage we are now appending max-age=60, stale-while-revalidate=86400, stale-if-error=300 so for the advantage endpoint we end up with no-cache, no-store, must-revalidate, max-age=60, stale-while-revalidate=86400, stale-if-error=300 is this expected?

@jpmartinspt that's a good point. No that's not ideal. I guess we should test for cacheability before setting the default times.

@jpmartinspt did that change

All good just needs the linting fixed.


Can debian:publish

Since the upgrade of sbt-native-packager: #110
Complains about the lack of a debian changelog, ie. http://stackoverflow.com/questions/38393297/how-to-solve-changelog-error-for-building-deb-with-sbt-native-packager
Once you give it a changelog it complains "dpkg-genchanges: error: source package has two conflicting values - sbt-launcher-packaging and sbt"

Fixed by https://github.com/sbt/sbt-native-packager/pull/961


fix(client): ensure presetEnv is loaded

Prior to this, if a user first loaded a React challenge and then navigated to a JS challenge, they would see
TypeError: Cannot read property 'presets' of undefined
in the console and be unable to run tests or evaluate code until they reloaded the page.

@ojeytonwilliams Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


CMake Tools uses too much status bar space, especially with long kit names like  "Visual Studio Enterprise 2019 Preview - x86"

See the screenshot:

Can something be done to reduce the status bar space?
It's a problem for me because I can't see the C/C++ database or flame icons anymore after switching to a git branch with a long name. Also, the GitLens status bar items disappears as well.
If the "Visual Studio Enterprise 2019 Preview - x86" kit name could be reduced to something small that would be good enough for me. That is only useful for clicking to change the kit, so if it just showed an icon or something, I could click it or hover to see the longer kit name if I wanted to know.

Certain kits are even worse:

It's a problem on my laptop because the C/C++ flame/database icons are no longer viewable.

I'd prefer a better UI too, something else than the status bar... maybe a drop-down in the menu bar, like in Visual Studio.

I don't believe VS Code yet has an API for extensions to add items to the main menu bar.

It would be nice to be able to choose which widgets to display. I, for example, never uses the Run CTest button.

@bobbrow How do you change active target in "icon" mode?

@ghuser404 You could use the CMake: Set Build Target command or use the settings:
    "cmake.statusbar.visibility": "icon",
    "cmake.statusbar.advanced": {
        "buildTarget": {
            "visibility": "compact"
        }
    }

The build target doesn't have an icon, so an alternative design would be to give it an icon. Maybe we someone could open a new issue for that.

@sean-mcmanus A "configure" icon/button would be great too. I'll create a ticket for that later.


DateTime UTC stored incorrectly - 0.1.49-beta

If datetime value is of type UTC it is converted to Local when storing using AddRow.

Timezone issues are my favorite. Can you give a lot more detail on what you are seeing. It’s pretty tricky in Postgres and we’ve been through hell getting this right so want to be really clear about what’s not working. The biggest mistake is usually the incoming datetime value not having right type (eg must be utc or local). Also read carefully what Postgres does with timestamp with tz columns. It’s a bit counterintuitive
Von meinem iPhone gesendet

Am 29.02.2020 um 7:44 PM schrieb Sandeep Chandra<EMAIL_ADDRESS>
If datetime value is of type UTC it is converted to Local when storing using AddRow.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

The column in table is timestamp without time zone.
Using AddRow, when the value has DateTimeKind of UTC it is converted to local.
Even if I make the column timestamp with time zone the value is converted to local.
If I specify DateTime value's DateTimeKind to Unspecified then it is persisted correctly. But I still have to set DateTimeKind after reading from database.

I think we should follow this
https://www.npgsql.org/doc/types/datetime.html

In Table Detailed Behavior: Sending values to the database the second case is incorrect IMO.

@sandeepc24 These lines are responsible: https://github.com/demetrixbio/FSharp.Data.Npgsql/blob/8976abdfec7bb07f63b261700b7dba429cbcd89a/src/DesignTime/InformationSchema.fs#L204-L213
based on this comment https://github.com/npgsql/npgsql/issues/1076#issuecomment-355400785. It seems it's been put in place to enforce point number 2 in Detailed Behavior: Reading values from the database, but the following comment suggests that it's been fixed, meaning the explicit DataSetDateTime might not be needed anymore. This'll be tricky to fix as the slightest oversight has the potential mess up clients.

Looks like I was wrong. @melanore added those lines fairly recently. Maybe he can chime in on this.

Have a look at this https://github.com/npgsql/npgsql/issues/2881#issuecomment-594392589

Any update on this issue, I am held up because of this issue.

Conversion to local kind was redundant in case of timestamp column. We are in progress of switching type provider to new SDK template / optimizing design time performance. Fix will be released some time this week.

Issue should be gone in version 0.2.0-beta. https://github.com/demetrixbio/FSharp.Data.Npgsql/pull/70/files#diff-03428eaeb9f35fb62752267fbaa39f96L211

That fixed the issue for me, thanks a lot.


[BUG] Un-triggered tabstops create a backlog

Description
When using snippets with tabstops, it seems like un-triggered tabstops begin to pile up into a queue that can trigger later. For example, if you have a tabstop at the end of an expression but use arrow keys to step past it, pressing tab at any time after the expression will cause the cursor to jump backwards to the tabstop you have already passed.
To Reproduce
As mentioned above, if you use arrow keys to move the cursor after a tabstop, pressing tab will jump the cursor backwards to the stop.
Expected Behavior
I would suggest that, if the cursor's location is past the location of the tabstop, the stop should be removed from the tabstop queue. This would mean there would be no unexpected jumps backwards when pressing tab. If this backwards jumping is considered intended behavior, then at least having a checkbox in the plugin's settings to only allow tab to jump to stops that are in front of the cursor would be a much appreciated capability.

I experience this frequently aswell


[5.3] Hide empty paginators

Fixes #15124 correctly. It should now have the same behavior as 5.2.
Also, see #15125 for further discussion.

Looks like I just beat you to it :smile:
I noticed there is a hasPages() method already on the paginator which 5.2 probably used internally. #15246

Looks like @garygreen beat me to the punch.

@garygreen Haha, yeah. Good find on using hasPages().


👾 Questionnaire

The following tasks are related to the questionnaire:

[x] Solve Bug where Loading Screen is visible indefinitely.
[x] Change UI/UX of Quiz Panel to new updated design.
[x] Change UI/UZ of the Answer panels to updated design.
[x] Implement Responsive behaviour.
[x] Solve bug where first answer option is not selectable.
[x] #28

Completed in #50.


Create junit5 extention library

This library should provide a test extension that will build up the report and titles based on test names etc.

This has been started - a new project called lsd-junit5 has been created (WIP)

Project has been started so this issue can be closed.


Enter the path of the audio folder to be processed:

how to remove the default let on the "Vocals/Accompaniment Separation & Reverberation Removal" tab
in the parameter "Enter the path of the audio folder to be processed:"
Right now it's E:\codes\py39\test-20230416b\todo-songs\todo-songs.
I want it to be empty when I start localhost!

Find it. File: infer-web.py. line - 1572


ModDevTool - Import Files Improvements

ModDevTool - Import Files Improvements

[ ]  Tag file as modified so it can be skipped next time
[x]  Instead of just bending the paths to the _import folder

[x]  get the original path
[x]  move to file to the mod _import folder
[x]  Bend the path

.import Folder at the root of the Mod Folder

[x]  Remove the rewriting of the .import files

[ ]  Check if the .import folder in the mod folder exists

[ ]  If not create one

[ ]  Copy the imported files into the .importfolder of the mod folder

[x]  Don’t copy the files inside the mod folder

[x]  Just add the paths to the imported files to the mod_file_paths so the they get added only to the final zip file

[x]  make sure they are added to the root of the .zip
mods-unpacked/
**mods-unpacked/.import**
mods-unpacked/ModAuthor-ModName
mods-unpacked/ModAuthor-ModName/ModMain.gd
mods-unpacked/ModAuthor-ModName/_meta.json
mods-unpacked/ModAuthor-ModName/...

[x]  Add the file paths inside the .import folder to the mod_file_paths

The Current Code is cursed 🧙♂

I have to rewrite the zipping part

Let’s add an array with

src path
dst path

Open the file from the src path

add it to the dst path ( should be the local zip path )

Add an zip dst path as a separate thing

Maybe also zip name?

Done with - https://github.com/GodotModding/godot-mod-tools/commit/624571603c27ece02122c8fa9392659e3ebc9358


clarify maxOpenShiftVersion upgrade error message

Signed-off-by: Ankita Thomas<EMAIL_ADDRESS>

@ankitathomas thanks for the quick turnaround on this, one suggestion on the test, looks good otherwise.

/retitle Bug 1992677: clarify maxOpenShiftVersion upgrade error message

/retitle clarify maxOpenShiftVersion upgrade error message


Are binary releases avaible?

Would be awesome to test this out.

Hi - no release as of yet - but I plan on making a release soon. Although only for 1.7 and 1.8 jdk versions. I have a few critical bugs that are preventing a release at the moment - but once I solve those I should be able to release


direct index array access is not typesafe

Note Duplicate of closed issue: https://github.com/Microsoft/TypeScript/issues/13778
However, after reading the discussion there, I still believe it warrants discussion. I  more or less echo the sentiment expressed by @OliverJAsh in this comment.

TypeScript Version<EMAIL_ADDRESS>
Search Terms
array access string number undefined
Code
index.ts
{
	const test: string[] = [];
	// expected [ts] error:
	// Type 'string | undefined' is not assignable to type 'string'.
	// Type 'undefined' is not assignable to type 'string'. [2322]
	const result: string = test[0];
}

{
       // here is an imagined case where we want the
       // first result of a server response. Typescript
       // tells us we can use our result however we wish.
	const response: () => string[] = () => [];
	const test = response();
	// expected [ts] error:
	// Type 'string | undefined' is not assignable to type 'string'.
	// Type 'undefined' is not assignable to type 'string'. [2322]
	const result: string = test[0];
}

tsconfig.json
{
  "compilerOptions": {
    "target": "es5",
    "module": "commonjs",
    "strict": true,
    "esModuleInterop": true
  }
}

Expected behavior:
	// Type 'string | undefined' is not assignable to type 'string'.
	// Type 'undefined' is not assignable to type 'string'. [2322]

Actual behavior:
No error on assignment of undefined | string to string
Playground Link:
I will create a playground shortly, for now, here is the entire issue in a github repo: https://github.com/prmichaelsen/ts-array-access
Related Issues:
https://github.com/Microsoft/TypeScript/issues/13778
Note
I understand there is a closed issue on this topic, but I believe that it is popular enough a request to warrant discussion. Here is my case.
The primary concern seems to be breaking existing for-loops where the index is clearly assigned. However, instead of:
const array = [1, 2, 3];
for (var i =0; i< array.length; i++) {
    array[i]+=1; // array[i] is possibly undefined
}

You can write:
const array = [1, 2, 3];
const result = array.map(value => value + 1);
// which is arguably an increasingly common way
// of doing things, anyway

Then, when doing random index accesses, you are required to check if the result is undefined, but not while iterating an enumerated collection.
So:
const array = [1, 2, 3];
// good
const result = array.map(value => value + 1);
// or for in-place manipulation
array.forEach((value, i) => array[i] = value + 1);

// bad
for (var i =0; i< array.length; i++) {
    array[i]+=1; // array[i] is possibly undefined
}

// errors, which is good
const result: number = array[50]; // array[50] is possibly undefined

Please let me know what you think.

I suggest closing this since #13778 is now open.


bootstrapped ci (shows no variance) vs std (shows high variance)

Hey folks!
I frequently follow rliable's guidelines to plot sample efficiency curves. I came across results now where 5 seeds of one experiment had large variance, but the bootstrapped confidence interval suggests little to no variance. Here are two plots to visualize my issue:

The number of bootstrap replications is set to 50000.
Here is a colab notebook to reproduce these plots:
https://colab.research.google.com/drive/1hFtmCX-TLUcPuDKZZlTPq34R7bDz_NWI?usp=sharing
It would be great to hear your intuitions about this. Do you think this is just a coincidence or a bug?
edit:

Lowering the reps to 3000 did not affect the plot
Reshaping from 750 episodes, 101 checkpoints to 5 runs, 150 episodes, 101 checkpoints did not affect the plot

I'd have to take a closer look sometime next week but usually this issue happens due to not bootstrapping over the correct axis (The readme specifies shape of the data expected). I think you want to switch the task and seed axis to fix this.
If you have a single task, then you can turn on task_bootstrap=True to not worry about shape related issues. https://github.com/google-research/rliable/blob/master/rliable/library.py#L215

Thanks for your reply @agarwl
My current take is to have the data in the shape of (5 runs, 150 episodes, 101 checkpoints).
Compared to your terms: checkpoints = frames, episodes = games, runs = training repetitions = tasks
This is the result if task_bootstrap = False

and if task_bootstrap = True. The intervals with task bootstrapping are more pronounced.

With task_bootstrap = True and a shape of (750 episodes, 101 checkpoints), the CIs are messed up.

Given the same shape of (750, 101) and task_bootstrap = False, the plot seems equivalent to the first one.


Error removing tmp extension from file

I'm running this on Arch.
The ffmpeg command that is created when the service starts is /usr/bin/ffmpeg -nostdin -rtsp_transport tcp -t 600 -i rtsp://user:password@<IP_ADDRESS>:8554/unicast -c copy /var/rec/poop/2024/04/27/2024-04-27_22-40-04
No temporary video file is created upon service start. However, if I run the ffmpeg command manually, it creates a .mkv video file from the stream without any issues.
Output from "journalctl | grep rtsp-recorder"
Apr 27 23:01:22 TurboEncabulator rtsp-recorder_$i[3649944]: [2024-04-27 23:01:22.017018697] ERROR: error removing tmp extension from file '/var/rec/poop/2024/04/27/2024-04-27_23-01-20': rename /var/rec/poop/2024/04/27/2024-04-27_23-01-20.tmp.mkv /var/rec/poop/2024/04/27/2024-04-27_23-01-20.mkv: no such file or directory
I don't know if this is of any relevance, but the file in the ffmpeg command was 2024-04-27_22-40-04. Whereas the file mentioned in the journalctl error output is always 2 seconds behind, (e.g. 2024-04-27_22-40-02.mkv).
Any idea what is going on here?  Thanks!!

I'll look into it. This project needs some improvement (I didn't knew anyone but me was using it). I'll get you back in a week or two with a version that has better debug options.

I would appreciate that so much!!
Your project is actually exactly what I was looking for. I have several Wyze cams whose RTSP streams I would like to save to a NAS on my LAN.
Truth be told, I wasn't counting on a reply here so I provided a pretty rough sketch of my particular issue. If you do release something with more debug options in the future I'll make sure to provide more thorough feedback.

I've just found I had two commits I didn't push since last August 😅 can you check if has been fixed?

I've just found I had two commits I didn't push since last August 😅 can you check if has been fixed?

Hey! Yes for sure! Thank you so much for looking into this!!
And apologies for the late reply. I was traveling internationally for business for the past two weeks, but I'm home now so I'll give this a shot today and let you know what I get on my end.
Cheers!

Same thing is happening to me. Also running Arch Linux. @hsg120 did you do anything else for it to work?

I got it to work with ffmpeg with following command:
ffmpeg -i 'rtsp://admin:<passwd>@<IP_ADDRESS>:554/onvif1' -vcodec copy -acodec copy -map 0 -f segment -segment_time 300 "ffmpeg_capture-%03d.mkv"
I'd like to know if it is possible to make necessary changes for rtsp-recorder to work with my cameras. Or at least have some guidance so I can fork the project myself and do the changes.

@icaroerasmo
I ran into an issue that I believe has nothing to do specifically with this program, and I've been meaning to spend more time on it sooner so I could come back with useful feedback for @Miguel-Dorta. I already spent a few hours last month trying to correct this to no avail...
My steps--

I downloaded the updated version of rtsp-recorder, built the .bin file and put it into /opt/bin.
I created a temporary service file called poop.conf and saved it in /opt/etc/rtsp-recorder.
I put the rtsp-recorder service file into /etc/systemd/system
File save location is /var/rec.

When start the service I get no output. If I check the status of the service (i.e. sudo systemctl status rtsp-recorder@poop), it says the service failed.
If I search the system logs for "rtsp" I get a systemd error that says, "rtsp-recorder@poop: Changing to the requested working directory failed: Permission denied". And then<EMAIL_ADDRESS>Main process exited, code=exited, status=200/CHDIR"
My user account has read/write access to the working directory (/var/rec). I also tried logging in as root and running the service and I get the same error. I've done a lot of googling and I'm sort of at a loss as to what I'm doing wrong.

@icaroerasmo I have not gotten it to work on my end, but I don't think it's any fault of this particular program. I'm currently stuck with a read/write permission issue. I'd intended to come back with some kind of useful feedback for @Miguel-Dorta but I spent a few hours last month trying to fix my particular issue and I'm failing miserably.
I downloaded the updated version of this program and installed it. I created a service and configuration file, and designated my working directory (/var/rec).
When I start the rtsp-recorder service it fails immediately. When I check the systemd log file it says--

Jun 15 20:10:35 TurboEncabulator systemd[1]: Started RTSP Recorder for poop.
Jun 15 20:10:35 TurboEncabulator (recorder)[250749]<EMAIL_ADDRESS>Changing to the requested working directory failed: Permission denied
Jun 15 20:10:35 TurboEncabulator systemd[1]<EMAIL_ADDRESS>Main process exited, code=exited, status=200/CHDIR
Jun 15 20:10:35 TurboEncabulator systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
Jun 15 20:10:35 TurboEncabulator systemd[1]<EMAIL_ADDRESS>Scheduled restart job, restart counter is at 1.

I'm at a loss though because my user account has read/write permission to that directory. I also tried logging in and running the service as root, but still got the same error. I've also done a lot of googling, but to no avail. I'm probably doing something dumb, but the solution alludes me.

@icaroerasmo
I can confirm it works!
Now, I'm going to further configure it to my liking, and try adding more cameras.
Thank you @Miguel-Dorta so SO much for getting this working!!!!! I am so appreciative.

@hsg120 it seems it is something with my cameras that stream videos in a format which is not expected by rtsp-recorder. I decided to implement a solution myself in shell script.

A lot of messages during the weekend! Let me reply one by one.

I can confirm it works!  Also, I had to take ownership of /var/rec.
Now, I'm going to further configure it to my liking, and try adding more cameras.
Thank you @Miguel-Dorta so SO much for getting this working!!!!! I am so appreciative.

 Thank you for your feedback! 🙂

Edit: Is there a way to limit the number of videos saved in order to prevent filling the hard disk?

I'm using a simple program I made myself for that. It limits the recordings by disk space and time. If you are interested I can publish it. It'd probably take me a few weeks tho.

I got it to work with ffmpeg with following command: ffmpeg -i 'rtsp://admin:<passwd>@<IP_ADDRESS>:554/onvif1' -vcodec copy -acodec copy -map 0 -f segment -segment_time 300 "ffmpeg_capture-%03d.mkv"
I'd like to know if it is possible to make necessary changes for rtsp-recorder to work with my cameras. Or at least have some guidance so I can fork the project myself and do the changes.

Sure! Can you investigate further what does it make your command work? My guess is the -map 0, but you need to test it with your cameras to be sure. If it's that, I can look into adding more parameters to the ffmpeg subprocess.
You can also 100% fork this repo (the freedom of open source). If you wanna some guidance, if we simplify this program, it's just the following:
while (!SIGINT) {
  exec("ffmpeg", ...)
  sleep(time)
  exitFfmpeg()

  if (reportStatus) {
    http(method, url)
  }
}

It's obviously more complex than this, because it needs to handle signals, ffmpeg hangs, report endpoint timeouts, etc. but this is the basic behaviour.
If you want to modify the ffmpeg parameters, it's here
And this is the main loop

What I saw is that each camera has its own output format. I had to discover mine searching on Google. I will fork the repository and try to implement a profile feature so each format will generate a specific ffmpeg command.


feature for sequence diagrams

I would really love a process box.
I realize we're strapped for space in this field. I just want a way to contain a bunch of items without them being alt or opt. Thoughts?

proc: create customer

end

What about even doing a "box" clause so you can create a box with any name.
box=proc: create customer

end

I'd also just be ok with being able to wrap stuff in a box without any text like this.

I'm simply looking for a way to label a bunch of things together.
Would love some help.

Hi! Sorry for the slow response.
The project has got good traction but it also means that it is very hard to manage all issues on your spare time. I am trying to get a bigger team together for faster development and better support #866.
This is a super old issue and you are proabably not thinking about it anymore. However, this is a good idea and I will mark this as an enchancement and put it in the backlog of suitable tasks.


[BUG] "clear" not recognized as an internal or external command, operable program or batch file. python rich module

Every time i wanna print table, it writes me:
""clear" not recognized as an internal or external command, operable program or batch file. python rich module"

What can I do with that?

I doubt this is anything to do with Rich. If this is a Windows batch file, you want cls not clear


How to apply a new theme using the MaterialThemeBase

Hey there, I'm updating my code from using the old PaletteHelper to the new MaterialTheme but cannot find a way to update the current theme.
I simply want to toggle between dark/light, everything worked as expected previously...
My App.xaml:
  <Application.Styles>
    <themes:MaterialThemeBase />
    <StyleInclude Source="avares://Material.Avalonia/Material.Avalonia.Templates.xaml" />
    <StyleInclude Source="avares://Material.Icons.Avalonia/App.xaml" />
    <StyleInclude Source="/Styles/Styles.xaml" />
  </Application.Styles>

My OnFrameworkInitializationCompleted:
  var theme = Theme.Create(Theme.Light, Primary, Accent);
  var themeBootstrap = this.LocateMaterialTheme<MaterialThemeBase>();
   themeBootstrap.CurrentTheme = theme;

My toggle theme code:
var materialTheme = Application.Current.LocateMaterialTheme<MaterialThemeBase>();
materialTheme.CurrentTheme.SetBaseTheme(Theme.Dark);

Hello, @teocomi
Sorry for the long wait for an answer
materialTheme.CurrentTheme is a property that generates a new theme object for you each time, so that users cannot change the internal theme from the outside (since changing the theme will not change the styles). So when you do materialTheme.CurrentTheme.SetBaseTheme(Theme.Dark); you change an object that has nothing to do with what's inside MaterialThemeBase. You need to change the properties you want in the theme object and then assign it back, you do that:
var materialTheme = Application.Current.LocateMaterialTheme<MaterialThemeBase>();
var theme = materialTheme.CurrentTheme.SetBaseTheme(Theme.Dark);
materialTheme.CurrentTheme = theme;

Does this solve your problem?

Hey @SKProCH thanks for your support! I just tried unfortunately this doesn't seem to work either.
I can put together a quick sample app if that helps...
And thanks for the great library!

@teocomi
Yes, I think it would make it easier to find the problem if you gave me an sample app. Or you can just share link to the actual one.

Hello, @teocomi
I took a look at your project. Apparently this problem was fixed in the nightly builds. At least with 3.0.0-rc0.90-nightly everything works.
I hope that soon I will have enough time to post a stable rc, in the meantime, use the nightly build.

It does indeed work! Sorry for opening a non-issue & thanks for checking 💥


Backport Apple Silicon support to 6.9

Context
Fixes #15751 and #16448

@bot-gradle test this

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

@bot-gradle test this

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

@bot-gradle test this

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

@bot-gradle test this

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

@bot-gradle test this

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

@bot-gradle test and merge

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


Support cyclic dependencies in uv tree

Summary
Closes https://github.com/astral-sh/uv/issues/8551.

Okay this isn't quite right -- it doesn't handle inversions correctly.

I have a version that does handle cycles via SCCs but it doesn't preserve alphabetical ordering of package names.


Build error in android

Purpose

Approach

Open Questions and Pre-Merge TODOs

[ ] upgrade flutter version.

Learning

Blog Posts

How to Pull Request Github Repo with Learning focused Pull Request Template.

Memo
Simulator running is clear
Pixel6


Refactor Storage Classes

Refactor Storage classes to work with new Models

Done by GH on 13/3/20 PR#33


Enable STModification for getSTForExpression endpoint

Purpose

$Subject

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

Related PR: https://github.com/wso2-enterprise/ballerina-low-code-editor/pull/1241.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

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


Cannot get inline math formula in webpage

Always get block version instead of inline version using the examples in the following, but it is rendered correctly in the gitbook editor.
Inline math: $$\int_{-\infty}^\infty g(x) dx$$

Block math:

$$
\int_{-\infty}^\infty g(x) dx
$$

Yes, please fix this! I have the same problem.

+1 Fix please


CalculateCoverage: Fix file not foundexception

Check if the file HitsFilePath exists
--
I have the same issue as https://github.com/tonerdo/coverlet/issues/8
This simple check fix my problem

Pull Request Test Coverage Report for Build #19

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 82.92%

Files with Coverage Reduction
New Missed Lines
%

/src/coverlet.core/Coverage.cs
7
35.71%

Totals

Change from base Build #18:
-0.2%

Covered Lines:
301

Relevant Lines:
363

💛  - Coveralls

Hi @sbalant, thanks for your contribution. I have some thoughts:
This approach just ignores the problem and it's not obvious to the user that something went wrong. A better approach will be to figure out why that error is even being thrown in the first place and try to prevent that. Possibly the file isn't being created in the first place? (https://github.com/tonerdo/coverlet/blob/master/src/coverlet.core/CoverageTracker.cs#L31)

No problem ;)

Any idea when this will be released to Nuget. I'm hitting this issue also.

Splendid! Thanks!


pod securityContext: runAsNonRoot

Events:
Type     Reason     Age                             From               Message

Normal   Scheduled  26m                             default-scheduler  Successfully assigned default/test-k8tz to node2
Warning  Failed      (x12 over )  kubelet            Error: container has runAsNonRoot and image has non-numeric user (k8tz), cannot verify user is non-root (pod: "test-k8tz_default(4308d622-b565-4067-a243-15bb7897a89f)", container: k8tz)
Normal   Pulled      (x94 over )  kubelet            Container image "quay.io/k8tz/k8tz:0.5.0test" already present on machine
pod status
root@master ~]# kubectl get pod
NAME        READY   STATUS                            RESTARTS   AGE
centos      1/1     Terminating                       0          4h33m
test-k8tz   0/1     Init:CreateContainerConfigError   0          26m
pod yaml
`apiVersion: v1
kind: Pod
metadata:
name: test-k8tz
spec:
securityContext:
runAsNonRoot: true
containers:

env:

name: TZ
value: Asia/Shanghai
image: nginx
name: nginx
volumeMounts:
mountPath: /etc/localtime
name: data
readOnly: true
subPath: Asia/Shanghai
mountPath: /usr/share/zoneinfo
name: data
readOnly: true
volumes:

hostPath:
path: /usr/share/zoneinfo
name: data
`

Sidecar container should be started with non root to avoid conflict with securitycontext

Hi @chenhuazhong,
this bug already fixed one version later (v0.5.1).
try to upgrade and let me know if it solves your problem.

It's OK after upgrading

It's OK after upgrading

excellent!


Comparison of Potential and NLFFF Extrapolation Results for 64x64x64, 96x96x96, and 128x128x128 Dimensions

I've compared the results of potential and NLFFF extrapolations for data cubes with dimensions of 64x64x64, 96x96x96, and 128x128x128. The z-slices of these 3D cubes at various heights are shown in the Jupyter notebook: test_compare_b3d.ipynb.
The results for these three data cubes were produced using the following input parameters:

64x64x64:
--time 2023-10-14T16:00:00 --coords -560 90 --hpc --box_dims 64 64 64 --box_res 3.0

96x96x96:
--time 2023-10-14T16:00:00 --coords -560 90 --hpc --box_dims 96 96 96 --box_res 2.0

128x128x128:
--time 2023-10-14T16:00:00 --coords -560 90 --hpc --box_dims 128 128 128 --box_res 1.5

Explanation of the Inputs:

time: Observation time in ISO format
coords: Center coordinates [x, y] in arcsec if HPC or deg if HGC or HGS
hpc: Use Helioprojective coordinates
box_dims: Box dimensions in pixels as three integers
box_res: Box resolution in Mm per pixel

Key Details:

The z-slice plots are provided for z=0, z=1, z=3, z=5, z=10, z=30, and z=60. For the larger data cubes (96x96x96 and 128x128x128), the corresponding z indices are scaled as z*96/64 and z*128/64, respectively.

You can access the data cubes via this link: Google Drive. Download the three .h5 files and place them in the folder defined by the datapath variable in the notebook. You can then run the notebook on your local machine.

To read the .h5 files directly, you can use the read_b3d_h5 function provided in the pyampp.gxbox.boxutils module. Full instructions and documentation for this function can be found here: read_b3d_h5 documentation.

Summary of Results:

The potential field results for the 64x64x64, 96x96x96, and 128x128x128 data cubes are nearly identical.
The NLFFF results for the 96x96x96 and 128x128x128 cubes exhibit nonlinear force-free characteristics, whereas the 64x64x64 NLFFF result closely resembles the potential field result.
Artifacts are observed in the NLFFF results for the 96x96x96 and 128x128x128 data cubes, particularly around regions with strong magnetic fields. A zoom-in of these artifacts is provided in the notebook. These artifacts are more prominent in regions below a height of 20 (in the 64x64x64 scale, equivalent to 40 in the 128x128x128 scale).

Performance Issue:
As mentioned previously, the NLFFF code takes significantly longer to complete for larger data cube dimensions than windows machines. In some cases, the extrapolation does not complete even after an overnight run on a Mac with an Intel chip.
I conducted additional tests using the same NLFFF code on both an Intel-based Mac and an Apple Silicon Mac, with identical Python environments (Python 3.10.13). On the Apple Silicon Mac, the code completes the extrapolation within tens of seconds, whereas on the Intel Mac, it runs much slower (hundreds of seconds or never completes).
Need help! @vit1-irk @Alexey-Stupishin

Please could you provide date, time and AR number (or coordinates)?

Please could you provide date, time and AR number (or coordinates)?
(UPD: date and time can be taken from file name). (UPD: it is a AR13465).
@sageyu123 @Alexey-Stupishin @vit1-irk

@Alexey-Stupishin I’ve updated the issue text with the coordinates and resolutions. Let me know if anything else is needed.


CoreFoundation: enforce 8-byte alignment on CFRuntimeBase

The atomic typed reference count is a 64-bit value.  We need to ensure
that any statically allocated structure is 8-byte aligned.  This comes
up in the case of CFString where CFConstantString types are emitted into
__cfstring on ELF, which can end up 4-byte aligned on ARMv7.  When
accessed at runtime, the offset becomes underaligned, resulting in a bus
error due to the unaligned access.
Although the attribution of CFRuntimeBase to 8-byte alignment should
enforce the alignment of __CFString due to the member alignment
requirement, just be extra cautious and mark both as requiring 8-byte
alignment.

@swift-ci please test Linux platform


hard coded vs usr defined strlength

... make foot note clearer

We had a similar formulation previously. But I don't remember exactly what the issues with "If not set" was. Are there restrictions wrt to length so that not every specified option is a valid option?
Might be, that back then the confusion was db layer vs. service layer. Does anyone remember? Most probably it's not an issue and nothing speaks against the way Hans-Joachim formulated it, just wanted to throw into the pot what I faintly remember :)

I can't remember...


CDC_ACM + Hid keyboard issue

My setup:
MCU stm32f103c8
CubeMX v.6.3.0
AL94.I-CUBE-USBD-COMPOSITE v.1.0.3
app.c
#include "main.h"
#include "usbd_def.h"

extern USBD_HandleTypeDef hUsbDevice;

uint8_t kbdPress[8] = {0,0,4,0,0,0,0,0};
uint8_t kbdRelease[8] = {0,0,0,0,0,0,0,0};

#define LED_ON() LL_GPIO_ResetOutputPin(LED_GPIO_Port, LED_Pin)
#define LED_OFF() LL_GPIO_SetOutputPin(LED_GPIO_Port, LED_Pin)
#define LED_TOGGLE() LL_GPIO_TogglePin(LED_GPIO_Port, LED_Pin)
#define GET_PTT() LL_GPIO_IsInputPinSet(PTT_GPIO_Port, PTT_Pin)

void print(char *str)
{
    uint16_t c = 0;
    while (str[c] != '\0')
        c++;
    CDC_Transmit(0, (uint8_t *) str, c);
}

void KeyDown(void)
{

    uint8_t res = USBD_HID_Keybaord_SendReport(&hUsbDevice, kbdPress, 8);
    if(res)
        print("KEY DOWN FAIL\r\n");
    else
        print("KEY DOWN OK\r\n");
}

void KeyUp(void)
{
    uint8_t res = USBD_HID_Keybaord_SendReport(&hUsbDevice, kbdRelease, 8);
    if(res)
        print("KEY UP FAIL\r\n");
    else
        print("KEY UP OK\r\n");
}

void app_init(void)
{
    LED_OFF();
}

void app_loop(void)
{
    KeyDown();
    LED_ON();
    LL_mDelay(1000);
    KeyUp();
    LED_OFF();
    LL_mDelay(3000);
}

LED blinks, both devices HID and COM7 appear in device manager, serial terminal prints "KEY DOWN OK/KEY UP OK" but no input comes from keyboard
Issues #17 and #19 are kind of similar to mine, but both them resolved without clear explanation.

More information about my changes for HID keyboard added.

@nckpln  : It was not working for me. So, I used a working  Keyboard Report Descriptor extracted from here:
https://www.instructables.com/STM32-As-HID-USB-Keyboard-STM32-Tutorials/
and it works.   Receiving chars.


Backport for Mediawiki < 1.25.0?

Hi kulttuuri,
I'm currently stuck on Mediawiki 1.16.5 due to some extensions I need that have never been updated to support later versions.  Is there any chance of backporting this extension for older versions of Mediawiki?

Unfortunately there are no plans on doing this at the moment. I actually first started the extension codebase with older mediawiki and then ported to latest version.
Older extensions should also have some backwards compatibility, have you checked with later mediawiki version to see if the old extensions would run?


add ext-json to composer.json

Description of change:
add json extension dependency to composer.json
Risks & Mitigation:
low - none

The Lint Wizard redacts anything he may have said before! :sparkles::sparkles::sparkles:
Lint Wizard Approved :heavy_check_mark:

The Lint Wizard redacts anything he may have said before! :sparkles::sparkles::sparkles:
Lint Wizard Approved :heavy_check_mark:


feat: add GetMapByName

Currently, the library allows you to get a map from an eBPF program loaded by themselves. If a map is already created, you cannot get it. This function aims to get a map already created.

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

Selftest is also failing, could you please address those issues ? Thanks for your contribution, I'll take a better look on the next iteration. Cheers!

Sorry for the convenience... I hope the test will works 🙏

Any updates? 🔍

Any updates? 🔍

Sorry for the delay, been out sick, slowly catching up on things. Will check this soon.

Thanks! 👍  No problem! I hope you feel better 🙏 Cheers!

Is it possible to release a new version of this library? It will help me 🙏

Is it possible to release a new version of this library? It will help me 🙏

It's being baked. It'll be available asap.

@aymericDD https://github.com/aquasecurity/libbpfgo/releases/tag/v0.5.0-libbpf-1.2

Thanks @geyslan
After updating my vendor I have the following error when I try to compile the program:
cgo: ../vendor/github.com/aquasecurity/libbpfgo/libbpfgo.go:1077:18: unexpected: -1-byte enum type - enum bpf_cgroup_iter_order {}

@aymericDD I haven't detected that error (yet).

Could you try to change that assignment to cgroupIterOrderC := C.enum_bpf_cgroup_iter_order(opts.CgroupIterOrder) and check it again.

Thanks for your help. I am still debugging, but I have a question. Which version of the kernel are you using? I am using the v5.15.x, but I supposed you use v6.x.

Which version of the kernel are you using? I am using the v5.15.x, but I supposed you use v6.x.

I'm testing it on the amd64-vm (Vagrantfile) which uses 5.15.0-78-generic - without errors.
Note that the only use of AttachIter in selftests is passing a zeroed value IterOpts: https://github.com/aquasecurity/libbpfgo/blob/c11365cbf2c67619350e72e3f3c67e01082780fb/selftest/iter/main.go#L39


inflated ops count for a-b compared to a+b

ops not reflecting the underlying compute if underly instructions are fused
from tinygrad import Tensor, GlobalCounters
a = Tensor.rand(3, 4).realize()
b = Tensor.rand(3, 4).realize()
GlobalCounters.reset()
(a+b).realize()
print(f"a+b {GlobalCounters.global_ops=}")
GlobalCounters.reset()
(a-b).realize()
print(f"a-b {GlobalCounters.global_ops=}")

a+b GlobalCounters.global_ops=12
a-b GlobalCounters.global_ops=24

is this still happening or has it been solved already? I ran your code and got
a+b GlobalCounters.global_ops=12 a-b GlobalCounters.global_ops=12


Linked dependencies resolves to file system path

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:  1.58.0
OS Version: MacOS 11.4 (Big Sur)

Obs: Not related to OS, as other OSs have the same problem
Steps to Reproduce: (A reproducible cenario can be found here)

Use the link feature of npm to link a dependency to a local copy by using a symbolic link

yarn link SOME_DEPENDENCY

Now when using the intellisense feature to auto import, it resolves the dependency to the relative filesystem path instead of the node_modules require path.
For example:

// Instead of resolving as a require path
import {something} from "dependency/something";

// It resolves to the relative path
import { something } from "../../../../some/unrelated/path/dependencyFolder/something"

I have this issue too! Any updates?

I'm having this issue either.

There are so many reasons why this doesn’t work, and only one of them is something we can easily fix. I think the easiest way to explain the problems is by explaining how we would generally expect this to work under today’s architecture, and then point out all the reasons why this example code doesn’t work that way. Here’s what should happen, at a high level:

You open the vscode_linked_dependency_bug folder in VS Code and open the Dependant/index.js file
We see Dependant/package.json and use it to scan node_modules for auto-imports
We see Dependant/node_modules/dependency_bug and add it to the auto-import list, discovering that it’s a symlink as we go
You ask for completions and we query the auto-import list, preferring the symlink because it gives us a nice node_modules-style module specifier

Here are all the reasons why that doesn’t work:

(2) We don’t see/process Dependant/package.json for auto-imports, because it’s not at the project root. When you don’t have a tsconfig/jsconfig file, we assume the project root is the folder you opened in VS Code. So, the only place we look for a package.json to scan for auto-imports is vscode_linked_dependency_bug itself, which doesn’t have a package.json. You can fix this yourself by adding a Dependant/jsconfig.json file. But JS users usually don’t know to do this, and it would be nice if we could infer the monorepo structure of the folder you’ve opened and set up multiple inferred projects with their root directories set appropriately. But this would be a huge, scary change that could break a ton of stuff.
(2) Even if we did find that package.json, it doesn’t specify dependency_bug as a dependency. This purely a user configuration error; we would never offer auto-imports for a node_module that’s not listed in your package.json.
(3) Even if we discovered Dependant/node_modules/dependency_bug/package.json, its main field points to index.js which does not exist. The file you wanted to import was called soSomething.js, which can’t be found from its package.json file. This is probably part user error and part design limitation for us. It would be nice if we could just scan the whole directory for files to auto import, but that can get really expensive really fast. Maybe it would be reasonable to do when we discover that it’s a symlink to a local project, but this would actually be a pretty big change.
(3) Even if the main field of the dependency package.json was set correctly, the auto-import provider currently only resolves type definition files, not JS files, even for inferred JS projects. This is the one small, easy fix for us.

So, this issue is not just a bug, it’s one bug, one or two feature requests, and one or two user config errors wrapped up in one report. I can fix the bug as part of TS 4.4, but (@DanielRosenwasser) it sounds like we need to have a larger discussion around how to be smarter about unconfigured JS projects, or at least how we can nudge the authors of those kinds of projects toward successful configurations.

@andrewbranch you are right about the user configuration, I will update the example repo with the dependency listed on package.json.
Really, how would TS know that it's a dependency if it's only on the folder.
But this cenario occurs on other projects that do have the dependency on package.json, and I think that would be 99% of the cases.
I will look into the points you described, looking into our real use case scenario, and update this issue with what I discover.
For last, it really is a change that was made on a recent release from TS and not with VSCode I believe. Because I've set it to use a previous TS version, and the imports worked correctly again.

What version of TS works? It’s probably working by accident 😬
The other thing to point out is that if you already have another file that imports "dependency_bug/soSomething", that’s another way that TS can find that file and it should become available as an auto-import to other files in the project.
However, without having a jsconfig.json or tsconfig.json file anywhere in here, a lot of features are going to work very poorly. TS will only be aware of files that are currently open or imported by files that are currently open. In general, your experience will improve a lot if you add jsconfig.json files at the root of each project. That, combined with the config changes I mentioned, along with the bugfix I mentioned in my last bullet point, are enough to fix the issue.


Update the guide to use current Az module (instead of AzureRM)

This page should urgently updated to Az PowerShell Module. The AzureRM module is quite long time ago replaced by the Az Module.
Furthermore is ADAL actually also outdated and MSAL should be used by now:
https://github.com/AzureAD/azure-activedirectory-library-for-dotnet#update-to-msalnet-now
And last but not least the second script works only with Windows PowerShell (5) and not with the current cross platform PowerShell (7). This  means also that, what you are describing here, does not work in Azure Cloud Shell. Which is, in my opinion, really bad for tutorial for an Azure Service.
Especially as the page is marked as "public preview" which seems that you are describing here a quite new functionality, I think, it is really bad practice to referencing such outdated tools.
So please update the page and use at least the Az Module and PowerShell 7. Thx!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2b0443f7-69c6-1e5d-cb14-7876595b8b54
Version Independent ID: a0fd8e29-b356-6e36-3a8a-8ec5fb68bb26
Content: Create Azure AD users using service principals - Azure SQL Database
Content Source: articles/azure-sql/database/authentication-aad-service-principal-tutorial.md
Service: sql-database
Sub-service: security
GitHub Login: @GithubMirek
Microsoft Alias: mireks

@J0F3 We are checking this and will get back to you.

@GithubMirek Please check the feedback from the user and please help to enhance the document.

We are also attempting to use this functionality and would like to see the documentation updates.  Thank you!  Eric

Following up with @GithubMirek

Following up with @GithubMirek

@J0F3 - Hey Jonas, thanks for the feedback! We really appreciate it. This is on our list to update, and we will do our best to update as soon as possible. We are committed to updating the documentation before the feature becomes GA, but can't commit to a timeline right now. Since I am tracking this internally, I will go ahead and close this issue. However, please let us know if there are further concerns. We thank you for your attention on our documentation.
#please-close

@J0F3 - Hey Jonas, thanks for the feedback! We really appreciate it. This is on our list to update, and we will do our best to update as soon as possible. We are committed to updating the documentation before the feature becomes GA, but can't commit to a timeline right now. Since I am tracking this internally, I will go ahead and close this issue. However, please let us know if there are further concerns. We thank you for your attention on our documentation.
#please-close

Script updated in https://github.com/MicrosoftDocs/azure-docs-pr/pull/146624


Custom Buttons

Just wondered, is there any way that you could add the ability to send commands to an api or even add the ability to make a button be able to send a command to a php website etc ?
My robot has an api that i can pre-set waypoints etc that can be called by simply calling a webpage and it runs the php code on that page , so is there a way to make the button widget be able to do something like that ?
Thanks
K
Cool interface btw :-)

Thanks! It might be possible, providing CORS doesn't cause any issues if the API is on HTTPS which iirc would be blocked.
There is also the slight problem that one would need to use XHR or fetch requests for the actual calls, and those are blocked on Safari under HTTP, so it wouldn't work at all on iOS and only on Chrome/Firefox in MacOS.
I think the easier way might be to have an extra backend node that handles this part and isn't constrained by browser shenanigans so it can send the requests when it gets some service call.

Hmm , so i guess running some kind of node on the ros server that will actually do the api calls would be the way to go ? i wonder if there is one that can be used / manipulated ?

Yeah I think it's the way to go to get the most done for the least hassle :smile:
Unless you're planning on using cpp, a simple python node with flask should be all you need.
I'm not sure if anything like that exists already, but here's another chatgpt example as a rough starting point:
#!/usr/bin/env python

import rospy
from std_msgs.msg import Empty
import requests
from flask import Flask, jsonify

app = Flask(__name__)

# URL of the REST API endpoint
API_URL = "https://api.example.com/data"

data_cache = None

def fetch_data():
    global data_cache
    try:
        response = requests.get(API_URL)
        response.raise_for_status()
        data_cache = response.json()
        rospy.loginfo("Data fetched successfully: %s", data_cache)
    except requests.exceptions.RequestException as e:
        rospy.logerr("Failed to fetch data: %s", e)
        data_cache = None

def callback(msg):
    rospy.loginfo("Received a message on the Empty topic, fetching data...")
    fetch_data()

@app.route('/data', methods=['GET'])
def get_data():
    if data_cache is not None:
        return jsonify(data_cache)
    else:
        return jsonify({"error": "No data available"}), 500

if __name__ == '__main__':
    rospy.init_node('flask_fetch_node')

    # ROS subscriber
    rospy.Subscriber('fetch_topic', Empty, callback)

    # Run Flask app in a separate thread
    from threading import Thread
    flask_thread = Thread(target=app.run, kwargs={'host': '<IP_ADDRESS>', 'port': 5000})
    flask_thread.start()

    rospy.spin()

You'd need to subscribe to one of these and then make the call once you get the message.


Add pagination support in clubs resource

Some methods have a page parameter (e.g. /api/clubs/xxx/animes, /api/clubs/xxx/mangas etc.), but they don't seem to work yet (no matter how I try to call them as a query (?page=xxx) or in body), API returns only the first 20 results and that's it. When this is fixed, I'll add support right away. In the meantime, this issue will stick around as a reminder for the future
The source file of the club's API controller: https://github.com/shikimori/shikimori/blob/master/app/controllers/api/v1/clubs_controller.rb

Yay https://github.com/shikimori/shikimori/commit/e8c12520544ef58716ff7765bd16ba00b6f0dbf3

Closed in a25e15


Lot of orphaned galops processes in the dashboard background jobs VM

The dashboard VM is getting restarted every few hours and this issue doesn't repro anymore.


Add popular datasets and tutorials

Thank you @ArushiSinghal !
As this is your first contribution to AlgoWiki, have some 🍰 🌮 🌯 from AlgoWiki team.
Keep Contributing. 👍
~
Vikesh


Rejected Proposals: Unable to Open proposal

Having posted a rejected proposal.
Then accessing Rejected Proposals.
Click view to drill into proposal. = Page not found

Example proposal to demonstrate the issue below (currently all proposals in fairground suffer this issue):
{"rationale":{"title":"An example freeform proposal1693037184","description":"I propose that everyone evaluate the following IPFS document and vote Yes if they agree. [bafybeigwwctpv37xdcwacqxvekr6e4kaemqsrv34em6glkbiceo3fcy4si](https://dweb.link/ipfs/bafybeigwwctpv37xdcwacqxvekr6e4kaemqsrv34em6glkbiceo3fcy4si)"},"terms":{"newFreeform":{},"closingTimestamp":1693037184}}

Fixed in https://github.com/vegaprotocol/frontend-monorepo/issues/1152


Make 0 elevation the default

Need a way to set the default elevation for a map, to avoid having to cover the entire map just to get to a minus elevation.

Macro and method added in v0.1.5 to allow user to change pixel elevation values. E.g., change all -10 pixels to 0.


Allow different base die for core d20 rolls

For homebrew purposes, allow modules or world scripts to modify the number of faces of the base die through hooks such as dnd5e.preRollInitiativeDialog (added with this PR) or dnd5e.dnd5e.preRollAbilitySave. Otherwise, default to the expected d20.
This change adds a faces property to rollData (or rollOptions in the case of initiative), which defaults to 20.
I did consider whether this meant needing to rename all the 'D20' references to something like 'AdvDisRoll', as this change is primarily to allow the base die to be changed for all rolls that use the Advantage/Disadvantage mechanic and all associated features therein. But, I think it's okay to keep the naming around the d20 and this is just a slight allowance for homebrew within that core functionality.

I've just realised I've introduced this change right when a major overhaul of the d20 roll code is happening, so if it can be baked into those changes, or I can reintroduce the change once the overhaul is complete.


Failed to detect a version allowing to call tofu

Describe the bug
whenever I switch to version 0.58.0 or above I get
Failed to detect a version allowing to call tofu I don't want to use opentofu I want to use terraform.
it works no problem in any version smaller then 0.58.0
Steps To Reproduce

use tenv to specify using terragrunt 0.58.0 or above
run terragrunt run-all plan
get error.

Expected behavior
should use terraform and not get Failed to detect a version allowing to call tofu : no compatible version found
Nice to haves
terragrunt run-all plan
INFO[0000] The stack at /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev will be processed in the following order for command plan:
Group 1
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/config
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/ecs_cluster
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/vpc_subnet_module
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/ecs_application/cloud_trail

Group 2
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/nacls
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/security_groups

Group 3
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/aurora_rds
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/alb

Group 4
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/secret_manager
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/waf

Group 5
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/ecs_application/ecs_application

Group 6
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/interface_vpc_endpoints
- Module /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/ecs_application/alarms_and_metrics

Failed to detect a version allowing to call tofu : no compatible version found
time=2024-05-23T14:22:48+01:00 level=error msg=tofu invocation failed in /home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/ecs_cluster/.terragrunt-cache/KQIqexa2ITiHIWfaGUR5Zni6sKw/BW7VBkPiNY16vnYOMAYEEEN7uiY/terraform/ecs_cluster prefix=[/home/work/repos/ibi-keycloak-terraform/terragrunt-v2/dev/base_infrastructure/ecs_cluster]

Versions

Terragrunt version: >=0.58.0
OpenTofu/Terraform version: 0/1.8.2
Environment details (NixOS SMP PREEMPT_DYNAMIC Fri May 17 10:18:09 UTC 2024 x86_64 GNU/Linu):

Additional context

I suspect it is the issue of Nix, Terragrunt try to execute tofu and terraform and looks like it is not permitted to run tofu
Most probably can be implemented workaround with TERRAGRUNT_TFPATH https://terragrunt.gruntwork.io/docs/reference/config-blocks-and-attributes/#terraform_binary

Hay thanks for bringing this to my attention yes doing a export TERRAGRUNT_TFPATH=~/.tenv/Terraform/1.6.4/terraform does fix the problem.


Difficult to debug hooks

The architecture of the hooks we use to trim unnecessary files makes it difficult to work out what's wrong when template generation does not go as expected.
Cookiecutter provides a --verbose argument which enables much more logging. We should make sure this can be enabled by our scripts, and we should also make sure it enables logging within our template utilities.
For example I would like to know which files it removes/leaves when executing keep_files(), if I had access to this information I would have noticed the missing comma in my DESIRED_STRUCTURE list

Thanks for opening this @da1910. I agree with having the --verbose option and a logger for better debugging the whole rendering process.

Renaming this issue...


Possible bug in parsing URL's that are created with a Captial letter - Empty BUO

I'm using version 5.0.6 on latest iOS (12.2) and Android api level 28 on a pixel.
Consider the below urls, both have different id's  but the first resolves correctly  while the second doesn't. The only difference at creating time is that the "J" is capitalized.  I tried this with a couple of others with similar results. .

https://kaji.app.link/juneteenth - works (BUO, and properties are populated)
https://kaji.app.link/Juneteenth - doesn't work (BUO and other properties are empty)

This is an initial data point so more testing may be required to fully uncover the issue but so far I can reproduce this consistently in XF iOS and Android.

@kwacks sorry for delay, ping please our support team, it is not a xamarin sdk issue because all data sdk receives from backend

@kwacks were you able to file a support ticket for this issue?
https://support.branch.io/support/tickets/new

Yes, but no response from your team so far.

	Happy to - what's your email or should I add it here?

	+bdm

On Tue, Aug 27, 2019 at 5:38 PM -0700, "csalmi-branch"<EMAIL_ADDRESS>wrote:
@kwacks would you be willing to email me your support ticket information? I can look into the status of this ticket for you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@kwacks have you heard anything back from the support team ticket you filed?

I've emailed you the ticket and response.

I've emailed you the ticket and response.


Add test and fix "undefined" for scope not set

In past action card 24

@bahmutov I agree...

I think once we get over DNS and deploys are automatic after the tests pass, it will be smooth sailing in fixing issues and updating features


Update mrlearning-base-ch1.md

removing high pri loc tag

Docs Build status updates of commit a1d8dd8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/mrlearning-base-ch1.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[HUDI-3304] Allow selective partial update

Change Logs
Expand on the Hudi partial update payload to create one that will allow "null" to be updated also. This payload will use "_hoodie_change_columns" to select which specific columns will be updated. It also will sort during precombine phase to ensure merges are not lost if precombined out-of-order.
Impact
Using this payload will have some small performance impact because of the sorted precombine phase. This will not affect standard operation.
Risk level (write none, low medium or high below)
low
Documentation Update
This feature would require detailed documentation to explain its usage.
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[x] CI passed

check this https://issues.apache.org/jira/browse/HUDI-4882
you  can also specify select columns in Multiple ordering fields for partial update

After looking at RFC-46, this needs to be rebased to follow the new "merge" interface instead of using payload classes

@bschell are you still working on this? the title says WIP

Yes, no longer a WIP. Let me update

Hi, I've cherry-picked this commit and created a new PR to continue the work: #9979


Add a flag to the party_status to look at historic battles

./dl_habitica_info.sh | ./party_status.py -b 10 would give results from 10 bosses ago. ./party_status.py -b 0 is default.
This is extra nice as, when the battle finishes, it would be nice to be able to run ./party_status.py -b 1

Bangin'


remaining issues on "mingw" Ruby

see https://gist.github.com/jmettraux/e373c9978613c3bd8834

Probably due to lacking TZ info on mingw platform.


Document Microsoft SQL Server support in sql.from

Microsoft SQL Server support has been merged in:
https://github.com/influxdata/flux/pull/2847
... but is not yet in our documentation:
https://v2.docs.influxdata.com/v2.0/reference/flux/stdlib/sql/from/
Customers are asking about this, so it'd be good to have in the doc. Thanks!
Relevant URLs
https://v2.docs.influxdata.com/v2.0/reference/flux/stdlib/sql/from/

SQL Server support has been merged, but it has not been released. It will be part of the next Flux release (likely 0.70). Once that release is cut, the docs for SQL Server will go live.


Is grid calendar supported?

Library information:

Version: 2.0.0
View or Compose module: View

Question:
Does this library have support for grid looking calendar like this? If so can you help me how to implement that grid looking calendar?

Duplicate https://github.com/kizitonwose/Calendar/issues/535#issuecomment-2241054447


WeaveNet Grafana Dashboards and Prometheus Alerts

What is missing?
Weave Net grafana dashboards and weave net alert
Why do we need it?
Many people spinning up prometheus with weave as the CNI needs weave prometheus monitoring setup and grafana dashboards with alerts configured.
Environment
Kubernetes
I have made the Grafana dashboards for weave and also the alerts.
Please let me know if i can integrate it here.
https://stackoverflow.com/questions/60232516/weavenet-grafana-dashboards-and-alerts/60447864#60447864

I think a patch to integrate them would be very cool!

Can you share an example PR on how to do it..


Publish v7.0.0

Publishes react-themes v7.0.0

Working fine 👍 .

Working fine 👍 .

Can you put an approval on it?


Documentation on the integral operator

Hi,
As per my previous issue, I want to start contributing to Pybamm, but do not have any experience contributing to open-source projects so I am looking for an easy entry point.
In my work I have been using pybamm's integral operator. I find the documentation a bit misleading here. Currently it describes the integral operator as follows,
pybamm.Integral(function, integration_variable)
$I = \int_{a}^{b}\!f(u)\,du$
"where $a$ and $b$ are the left-hand and right-hand boundaries of the domain respectively, and $u\in\text{domain}$."
However, I don't think that the differential element actually used in calculating the integral is necessarily du (where u is the integration variable). For example,
$\int_{0}^{R}\!1\,dr = R$
But,
pybamm.Integral(1, r) $\neq R$ (at least in certain cases, as shown below)
It looks like the integral operator actually performs,
$I = \int_{u_{min}}^{u_{max}}\!f(u)\,dq$
Where $u$ is some spatial variable, $u_{min}$ and $u_{max}$ are set by the geometry of the spatial variable, and $dq$ is given by,
$dq = du$ for cartesian coordinates.
$dq = 2\pi u du$ for cylindrical coordinates.
$dq = 4\pi u^2 du$ for spherical coordinates.
I show this in the code below for the cylindrical case, where,
$\int_{0}^{R}\!1\,dq = \pi R^2$
In the code, if you change coord_sys="cylindrical polar"  to coord_sys="cartesian", it will give,
$\int_{0}^{R}\!1\,dq = R $
and if you change it to coord_sys="spherical polar" it will be
$\int_{0}^{R}\!1\,dq = \frac{4}{3} \pi r^3 $
As mentioned above, I'm looking for a contribution to make myself - I am just starting an issue first to follow the contribution guidelines. Please let me know your thoughts on this suggestion.
Steps to Reproduce
import pybamm
import numpy as np

model = pybamm.BaseModel()

r_p = pybamm.SpatialVariable("r", domain="positive particle", coord_sys="cylindrical polar")

R_p = 2

one = pybamm.PrimaryBroadcast(1, "positive particle")
one_int = pybamm.Integral(one, r_p)

geometry = {"positive particle": {r_p: {"min": 0, "max": R_p }}}

var_pts = {r_p: 50}

# the PDE describing c is irrelevant, it's just to run the solver

c = pybamm.Variable("c", domain="positive particle")
D = pybamm.Scalar(1)
dcdt = pybamm.div(D * pybamm.grad(c))
model.rhs[c] = dcdt

model.boundary_conditions = {
c: {"left": (0, "Neumann"),
"right": (1, "Neumann")}, }
model.initial_conditions = {c: 1}

model.variables = {"c": c, "Integral of 1": one_int}

submesh_types = {"positive particle": pybamm.Uniform1DSubMesh}
mesh = pybamm.Mesh(geometry, submesh_types, var_pts)
spatial_methods = {"positive particle": pybamm.FiniteVolume()}
disc = pybamm.Discretisation(mesh, spatial_methods)
disc.process_model(model)
sim = pybamm.Simulation(model)
sim.solve([0, 1])

integral_result = sim.solution["Integral of 1"].entries[0]

pi_r_squared = np.pi * R_p **2
v_sphere =  4 / 3 * np.pi * R_p **3

print("The integral calculated by pybamm is ", integral_result)
print("The value of R is ", R_p)
print("The value of pi * R^2 is ", pi_r_squared)
print("The value of 4/3 * pi * R^3 is ", v_sphere)

thanks @isaacbasil! It would indeed be helpful to provided this additional detail in the docstring wrt the different coordinate systems. Please do make a PR for this

I saw the PR for this was merged, so I will close it for now


java.lang.UnsupportedOperationException: Transaction is managed by Spring, please use Spring's @Transactional annotation instead.

java.lang.UnsupportedOperationException: Transaction is managed by Spring, please use Spring's @Transactional annotation instead.
when I post http quest to localhost:8080/departments/create?name=aa&location=111

Thank you for your feedback. This is a bug. I've upgraded Ktorm to version 2.7.2 to fix it.


[Before release] Security audit

Make sure that it's not possible construct URLs that crash the browser.

No exploits were indentified


Executing node fails on Linux Aarch64

Hi,
i'm using version 6.0.0 on a linux running on a ARM64 (aarch64) architecture:
╰─$ uname -a
Linux fedora 6.1.6-200.fc37.aarch64 #1 SMP PREEMPT_DYNAMIC Sat Jan 14 16:36:00 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux

When running the build, it fails with:
./gradlew build --console=plain
> Task :assessment-ui:installNode

> Task :assessment-ui:installYarnGlobally FAILED
/home/moe/Projects/project-assessment/assessment-ui/node/bin/node: /home/moe/Projects/project-assessment/assessment-ui/node/bin/node: cannot execute binary file

Using file on the path returns:
/home/moe/Projects/project-assessment/assessment-ui/node/bin/node: ELF 64-bit LSB executable, x86-64, version 1 (GNU/Linux), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 2.6.32, BuildID[sha1]=7c488e62cc870eb3b205a3f7f578e28ed9b973eb, with debug_info, not stripped

It looks like it downloaded node for x64, not arm64.
Workaround:
frontend {
	nodeDistributionUrlPathPattern = 'vVERSION/node-vVERSION-linux-arm64.TYPE'
}

Hi @mhalbritter,
Thank you for reporting this issue, and sorry for the late answer.
Can you execute command java -XshowSettings:properties -version and provide the value of system properties os.name and os.arch?
Thanks for your help!
BR

Sure:
    os.arch = aarch64
    os.name = Linux

Thank you very much for your work on that one! ❤


[New autocomplete] automatically pops up a join hint when a join is already typed

Description
https://github.com/user-attachments/assets/f89b8299-69c9-47eb-92ff-b15fe49db514
DBeaver Version
DBeaver 24.2.4
Operating System
windows 10
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Also applying schema proposal after database and dot should not replace already written part of fully qualified name