minor: strange ui layout :)

See attached screenshot. I have a suspicion it should show up a bit differently))
browser: Chrome 114 on macOS Ventura

You  must be running in --verbose=True mode.  We don't often maintain that.

3801bc1545ada42455373c851484307c0e0cdace

Ah i see. FYI the system info button on System tab doesnt work)

Your system doesn’t support some information


Using multiple db contexts with SQlite in-memory tests results in wrong result

I use Sqlite in-memory database to write tests for my application as described here and in the docs https://github.com/aspnet/EntityFramework.Docs/blob/master/samples/core/Miscellaneous/Testing/TestProject/SQLite/BlogServiceTests.cs
But my tests fails often when I open for every section a new database context with
// In-memory database only exists while the connection is open
var connection = new SqliteConnection("DataSource=:memory:");
connection.Open();

try
{
  var options = new DbContextOptionsBuilder<BloggingContext>()
      .UseSqlite(connection)
      .Options;

  // Save items
  using (var context = new BloggingContext(options))
  {
      .....
  }
  // Use a separate instance of the context to verify correct data was saved to database
  using (var context = new BloggingContext(options))
  {
      .....
  }

}
finally
{
    connection.Close();
}

when I collapse the propertys manually in the debug mode the data appears suddenly or sometimes not. When I don't using the seperate db context everything works fine.

In this variation the data is present in the second db context. I hope it helps.
// In-memory database only exists while the connection is open
var connection = new SqliteConnection("DataSource=:memory:");
connection.Open();

try
{
  var options = new DbContextOptionsBuilder<BloggingContext>()
      .UseSqlite(connection)
      .Options;

  // Save items
  .....

  // Use a separate instance of the context to verify correct data was saved to database
  using (var context = new BloggingContext(options))
  {
      .....
  }

}
finally
{
    connection.Close();
}

@StarpTech This looks like an issue that was fixed in EF Core 1.1.2. If you are already using 1.1.2 (or a pre-release of 2.0.0) then please open an issue on https://github.com/aspnet/EntityFramework. (This github repo is for EF6 issues.)


Update scorecard-action to v2:alpha

Signed-off-by: Azeem Shaikh<EMAIL_ADDRESS>
Summary
v2:alpha release fixes some bugs.

Release Note
NONE

Documentation
NONE

Codecov Report

Merging #987 (d545409) into main (4dd242d) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #987      +/-   ##
==========================================
- Coverage   48.10%   47.99%   -0.12%
==========================================
  Files          61       61
  Lines        5421     5421
==========================================
- Hits         2608     2602       -6
- Misses       2532     2536       +4
- Partials      281      283       +2

Impacted Files
Coverage Δ

pkg/types/alpine/v0.0.1/entry.go
53.52% <0.00%> (-2.49%)
:arrow_down:

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


Requesting additional accepted type 'list' to support reading from yaml-file

It would good to have list as accepted type here to support reading from yaml-file.

Doesn't it work already if you use from_yaml? https://oulu-imeds.github.io/solt/utils.html#solt.utils.from_yaml

I think I have found the bug. The fixed version is now in master.


What does the setting SHOW_TYPING do?

Summary

I set SHOW_TYPING and SHOW_READ to true, but I don't see any differences in the program output.  I thought maybe SHOW_TYPING would display some kind of status message like "<thread_name> is typing..." to STDOUT.

Motivation

I think it would be a great feature to have, if possible.

Additional note
I might not be understanding what those settings are supposed to do.  Not sure if this is a bug or a feature request...

@krunoPeric

SHOW_TYPING currently does nothing. The intention is that it will show the user your are typing to, that you are indeed typing (e.g. krunoPeric is typing..., in the FB UI)
SHOW_READ shows marks messages sent to you as READ when you use the clear command (see the docs)


Ability to host webpages using nginx-proxy-manager

Is your feature request related to a problem? Please describe.
Want to use nginx-proxy-manager to serve host dynamic or static webpages.
Describe the solution you'd like
Ability to configure to point to a directory to serve the static content in one of the server configuration options.
Describe alternatives you've considered
Alternative was to deploy nginx outside of the docker container to use the http server portion to serve static content.
Additional context
Add any other context or screenshots about the feature request here.

Nginx Proxy Manager is a great tool to perform the reverse proxy.  I don't see any ability to host other website using the nginx http server.   It would be good to somehow also support hosting static or dynamic webpages using this great tool.

I see there are already several feature request for this.  Closing this one.


The import com.google.maps.android.R cannot be resolved

Thanks for stopping by to let us know something could be better!

PLEASE READ
If you have a support contract with Google, please create an issue in the support console. This will ensure a timely response.
Discover additional support services for the Google Maps Platform, including developer communities, technical guidance, and expert support at the Google Maps Platform support resources page.
If your bug or feature request is not related to this particular library, please visit the Google Maps Platform issue trackers.
Check for answers on StackOverflow with the google-maps tag.

Please be sure to include as much information as possible:
Library is installed in android Studio.
In DefaultClusterRenderer the import statement - 'import com.google.maps.android.R' has error that 'R' cannot be resolved.
App appears to work, but how can i eliminate this error.
Summarize your issue in one or two sentences
Expected behavior
What did you expect?
Observed behavior
What did you see instead?  Describe your issue in detail here.
Environment details

Device make and model, or emulator details
Android version
Library version and other environment information

Steps to reproduce
If you can reproduce using the demo app in the project, it helps us troubleshoot faster.

?

Code example
# example

Stack trace
# example

Following these steps will guarantee the quickest resolution possible.
Thanks!

@G-Steve-E Have you tried Android Studio "File->Invalidate caches"?

Sean:
Sorry for the delay.
Invalidating the caches did not work.
Steve

On Jun 30, 2022, at 7:48 AM, Sean Barbeau @.***> wrote:
@G-Steve-E https://github.com/G-Steve-E Have you tried Android Studio "File->Invalidate caches"?
—
Reply to this email directly, view it on GitHub https://github.com/googlemaps/android-maps-utils/issues/1100#issuecomment-1171315733, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHG6VBEQM3734ULZXGEEAT3VRWXSRANCNFSM5Z4P3BOQ.
You are receiving this because you were mentioned.

Hi @G-Steve-E .
Sometimes Android Studio messes up the R.java and the indexes. The general way to fix this is through invalidating the caches and/or synching the Gradle file.
Can you please verify whether this still happens with the latest Android Studio version?
Thanks.


Added add, append, prepend, and touch functions.

Added add, append, prepend, and touch functions and some test cases for them.

Please ping if any changes are required, I'll do the necessary.

Thanks @vVv-AA for the contribution. I have merged the changes.


Added last played remover

Last Played Remover
https://github.com/GrodanBool/Steam-Deck-Tweak-Last-Played-Remover
The theme removes the Last Played time from the game pages.
Checklist
Failure to complete this checklist or deleting boxes from the checklist will result in the closing of your pull request unless this is a theme update. Please write any comments regarding this checklist at the bottom of your pull request.
Check every box.

[x] I am the original author of this theme or have permission from the original author to make this pull request.
[x] All copyright of this theme's contents belong to the listed author or is cited in the repository linked above.
[x] This theme's target has been marked appropriately and only styles said target.
[x] This theme works properly on the latest versions of SteamOS for Steam Deck, decky-loader and SDH-CssLoader.
[x] This theme only uses * or !important if absolutely necessary.
[x] This theme is under 4MB in size and uses the least disk space possible.
[x] This theme's preview image does not include text unless it is necessary to describe changes that can be made.
[x] This theme is safe for work and does not contain any sexual, drug-related, or profane content.
[x] This theme prefixes any CSS variables with a unique identifier.

Check one box.

[x] I am not bundling a part of another theme with this theme to encourage mixing and matching themes.
[ ] Themes included with this theme are toggleable using a patch.

Check one box.

[ ] This is a keyboard theme applied to the default keyboard.
[ ] This is a system-wide theme applied to the default keyboard. The keyboard is toggleable using a patch.
[x] This theme does not target the keyboard.

Good catch, will do.


Allow using RegExp for paths alongside strings

The regexparam library allows for passing RegExp through parse. I think it would be a useful feature to be able to use custom regular expressions for routes.
For example:
<Switch>
  <Route path="/:id">Hello</Route>
  <Route>404</Route>
</Switch>

id will match anything. But if we know that an ID will only be digits, it would be nice to use a regex to validate it:
const idRegex = /^[/](?<id>\d+)$/;

<Switch>
  <Route path={idRegex}>Hello</Route>
  <Route>404</Route>
</Switch>

If it doesn't match, it will continue on to the 404 page.
This uses a named capture group so it could be a direct drop in for "/:id".
Alternatively, a normal regex could be used:
/^[/](\d+)$/

// Then consumed like:
const params = useParams();
const id = params[1];

This allows for lifting validation out of routes.
Patterns like route prefixes (/@:username) and multiple params in a single segment (/:id-:opt) becomes possible.
And since it builds upon functionality that is already here, it won't cost much to implement.

Released this in v3.2.0


Using List View within tab view. (question)

Is there an example of using List view within this component? I wanted to render few hundred list items with avatars on each of the multiple tabs.  I am just starting out and looked at react-native-scrollable-tab-view  but it seems with the new verision of react-native 0.42 there is a significant lag at this moment.  So In my search I found this lib. Has anyone used listview within this component? Would very much love to hear your feedback. thanks.

You can use any type of view. Nothing specific with listview, though you might have to do removeClippedSubviews={false} due to an RN bug.

@satya164  i thought keeping removeClippedSubviews value to true improved performance. Why would in this case you would recommend.

I don't recommend it, but sometimes you'll face this bug https://github.com/facebook/react-native/issues/1831 and there's no other choice.

I am using listview in both the tabs and onPress when I am pushing the page on  I am getting this error :
this.props.navigator.push({id: 8,}); it giving me error "undefined is not object(evaluating 'this.props.navigator.push')"


Remove port range check for 30000-32767

Kubernetes by default has the range of 30000-32767 for NodePort assignment. However, it is not appropriate to ensure (and fail the install) if something is bound within this range, because Kubernetes will actually only choose open ports when assigning a NodePort. Additionally, this range can be changed via configuration, so hardcoding verification of it being unused with no way of changing it is problematic.
It would be great to backport this to 5.5.x as well, as currently we have to maintain a fork of Gravity.
See also https://community.gravitational.com/t/port-usage-and-verification/163/3

Resubmitted internally to have the CI execute successfully. CI is not yet set up to accommodate external PRs.


bookends: Add periods to stream status bookends.

Also change "deactivated"->"archived" for consistency.

Merged, thanks @alya!


As a Mentor / Mentee in the apply process, users are unable to open the PDF documents

Describe the bug
Users are not able to open the PDF documents after application process, and application approved by Admin
To Reproduce

Apply as Mentor user
Approve the application as Admin user
As a mentor user, go to application page, get the training page opened
Try to open any PDF document in the training page

Expected behavior
PDF document must be opened, error happening is that clicking the PDF document button, nothing happens
Screenshots/logs
2023-04-16T23:35:39.666573+00:00 heroku[router]: at=info method=GET<EMAIL_ADDRESS>host=mentee-dev.herokuapp.com request_id=c4b39816-0f15-49b9-908a-4c8ec8e43fe9 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=4ms status=200 bytes=324 protocol=https
2023-04-16T23:35:42.083622+00:00 heroku[router]: at=info method=GET<EMAIL_ADDRESS>host=mentee-dev.herokuapp.com request_id=6536cf0e-ad8b-4240-8ba6-ec3a98c40284 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=142ms status=200 bytes=252 protocol=https
2023-04-16T23:35:42.082759+00:00 app[web.1]: <IP_ADDRESS> - - [16/Apr/2023:23:35:42 +0000] "GET<EMAIL_ADDRESS>HTTP/1.1" 200 75 "https://mentee-dev.herokuapp.com/application-page" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.4 Safari/605.1.15"
2023-04-16T23:35:42.295673+00:00 app[web.1]: <IP_ADDRESS> - - [16/Apr/2023:23:35:42 +0000] "GET<EMAIL_ADDRESS>HTTP/1.1" 200 60 "https://mentee-dev.herokuapp.com/application-page" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.4 Safari/605.1.15"
2023-04-16T23:35:42.296129+00:00 heroku[router]: at=info method=GET<EMAIL_ADDRESS>host=mentee-dev.herokuapp.com request_id=37ac29a4-d70e-419d-ac55-0152eb3df491 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=5ms status=200 bytes=237 protocol=https
2023-04-16T23:35:42.512748+00:00 heroku[router]: at=info method=GET path="/api/training/1" host=mentee-dev.herokuapp.com request_id=5e26dedf-1b22-441a-9351-aaafdcee2985 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=4ms status=200 bytes=1743 protocol=https
2023-04-16T23:35:42.512288+00:00 app[web.1]: <IP_ADDRESS> - - [16/Apr/2023:23:35:42 +0000] "GET /api/training/1 HTTP/1.1" 200 1564 "https://mentee-dev.herokuapp.com/application-page" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/16.4 Safari/605.1.15"
2023-04-16T23:35:55.072762+00:00 heroku[router]: at=info method=GET path="/api/training/trainVideo/627d2b792720b6ebf28512dc" host=mentee-dev.herokuapp.com request_id=c8776e75-3b82-4c9c-929a-09d1cd5d5d53 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=2ms status=500 bytes=308 protocol=https
2023-04-16T23:35:55.071458+00:00 app[web.1]: [2023-04-16 23:35:55,071] INFO in require_auth: Error parsing JWT token -- not included in header or invalid token
2023-04-16T23:35:55.071480+00:00 app[web.1]: [2023-04-16 23:35:55 +0000] [8] [INFO] Error parsing JWT token -- not included in header or invalid token
2023-04-16T23:35:55.071630+00:00 app[web.1]: 2023-04-16 23:35:55,071 <IP_ADDRESS>: requested https://mentee-dev.herokuapp.com/api/training/trainVideo/627d2b792720b6ebf28512dc: INFO in [require_auth: 22]: Error parsing JWT token -- not included in header or invalid token
2023-04-16T23:35:55.071697+00:00 app[web.1]: [2023-04-16 23:35:55,071] INFO in require_auth: Illegal ID token provided: None. ID token must be a non-empty string.
2023-04-16T23:35:55.071722+00:00 app[web.1]: [2023-04-16 23:35:55 +0000] [8] [INFO] Illegal ID token provided: None. ID token must be a non-empty string.
2023-04-16T23:35:55.071742+00:00 app[web.1]: 2023-04-16 23:35:55,071 <IP_ADDRESS>: requested https://mentee-dev.herokuapp.com/api/training/trainVideo/627d2b792720b6ebf28512dc: INFO in [require_auth: 23]: Illegal ID token provided: None.


Virtual File System

Introduces a virtual file system, in the sense that it processes the textDocument/didOpen, textDocument/didChange and textDocument/didClose notifications to replicate the state of a file being edited in the IDE.
This is accesed in the Handler via a new field with signature
(J.Uri -> IO (Maybe VirtualFile))

where a VirtualFile is defined as
data VirtualFile =
  VirtualFile {
      _version :: Int
    , _text    :: Yi.YiString
    } deriving (Show)

One problem with depending on the yi-rope package is that it is GPL. IANAL, so I'm not sure if that means that this package must be GPL too. This: http://stackoverflow.com/questions/11819831/including-gpl-external-library-in-mit-code seems to suggest it does.

My personal, and entirely selfish preference, is to keep this library MIT so that I can use it in the VHDL language server I am building and sell it in the future.

Yes, I was wondering about that too.
I discussed it on IRC with some people, who seemed to think that BSD3 and GPL are compatible. I am not personally worried about licensing, so would gladly change the license here to match something.
@adamwalker what are your views on that? Given you are a contributor.

And BSD3? Because I am quite keen to have changes managed here, and YiRope is good for that.
I imagine it should be possible to pass the types/functions in as part of the initialization, if all else fails.
Except the change management is particularly easy with YiRope.

Yeah - anything that uses GPL code must be GPL. It also might be possible to not have the rope stuff as part of the framework but instead provide a separate library that the user calls from within the file opened/changed/saved callbacks that does all the work of keeping track of the contents of URIs.
But, thats not as nice as what you currently have. It's up to you really. Just consider that using yi-rope prevents commercial uses of this library.

Well, we have identified a workaround that could be made to enable commercial use and strip out the GPL part.
So I suggest we leave it as now and let a future commercial user do that. Unless you want to do a PR on it for that functionality now already.
It is no big deal for me, I am not personally planning commercial use, but I am not keen to do much work for the benefit of commercial users only :).
If it would help, I am happy to change the license too, if you have a preference.

Just consider that using yi-rope prevents commercial uses of this library.

And that means any company that wants to ship a language server will have to ignore this package and build their own from scratch.

And from GHC 8.2 we can make the Yi.Rope package a backpack specification, so you can plug in anything that meets the signature.

@adamwalker Are you happy with the proposed workaround, if commercial use is required? I would like to merge this.

Yep. The backpack proposal sounds good.

And I captured the required cleanup in an issue, for any future commercial users.
See https://github.com/alanz/haskell-lsp/issues/16


docs: Add 'how to request merging rights'

Linked also to the GitHub issue on how to become a committer

Motivation for this change
While looking for this information myself, I bumped into the TODO in the manual.
@cole-h provided the missing answer and the link to #50105
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Errmm, this makes me sad. Because it's the unsavory truth that I'm too proud to put "on paper".  Still to vague to actually be a "how to" though. Currently it really is just that thread.

So... scrap the numbered list and keep the link to the GitHub issue and original TODO?

I feel like that TODO is a bit damaging. Can I perhaps ping some of the folks in the linked issue.
Not looking for a verdict but more of a guideline. I believe new members to the community might seek the same.
At a personal level, while I don't think I qualify just yet I'd like be able to transform this guideline into some kind of milestone I could aim at.

I feel like that TODO is a bit damaging. Can I perhaps ping some of the folks in the linked issue?
Not looking for a verdict but more of a guideline. I believe new members to the community might seek the same.
At a personal level, while I don't think I qualify just yet I'd like be able to transform this guideline into some kind of milestone I could aim at.

I think it will be perfectly reasonable to at least put what happens currently. We owe it to people.

I've removed the points and left the link to the discussion.
Until there's an official word this is more helpful than the TODO :sweat_smile:

I agree that being honest about current state is better than a TODO.
Maybe this encourages someone to help out by improving the process, I'm unfortunately able to do only the minimal work to keep current status quo running.


list_queue needs lowest priority (applies after all other filter)

per discussion: http://discuss.flexget.com/t/movie-queue-migration-question/2248

We were discussing this in irc as well. It should do the removing from the list on the learn phase I think, rather than the filter phase, to ensure output went fine. We need to add some logic on filter phase then to make sure not to accept more than one entry based on the same queue item somehow though.


Allow to configure one or more images to display after every afk

Mainly important for shatters but can be used for requirements as well which is why multiple images should be allowed.
Shatters image in question: https://media.discordapp.net/attachments/893216268914679889/898057319944957963/patience.gif

Not needed anymore in my opinion


Run test-integration job on macOS

macOS support has been added to GitHub Actions runners with Colima: https://github.com/actions/runner-images/issues/6216.

docker commands are failing with exit status 125. Not sure what's going on here, and I can't reproduce locally. Closing for now. If anyone has any ideas, feel free to reopen.


Docdiff: investigate why it breaks other integrations

Docdiff works correctly currently. However, I had to disable it for now because it interferes with other integrations like EthicalAds and Search As You Type.
I assume that it breaks because it changes the DOM and it may be removing some things required for the other integrations? 🤷🏼

This is probably because doc-diff replaces the DOM at config.root_selector:
https://github.com/readthedocs/doc-diff/blob/main/src/extension.js#L166
It should probably be loaded last, after all other DOM manipulations are finalized.

Hrm... I just realized that all those promises are working in a sync way because I'm calling them directly. I have to create a new Promise() if I want them to be async.

Well, you have three layers of promises so far: the outside promise signalling loading state, the promises for each tool, and the return from Promise.all(...) (this is also a promise [^1]). You'd want to chain on to the return value from Promise.all(...), where you do have config scoped. So, something like:
return Promise.all(
  promises
).then(() => {
  return docDiff.initialize(config);
});

I have to create a new Promise() if I want them to be async.

I didn't look closely, but this is generally the issue that makes async code hard. Everything needs to be async to take advantage of async operations. For the purposes of this first iteration, it will be easier to avoid promises/async execution.

I updated the code to make it a little cleaner and show what I'm trying to do and how I'm chaining the Promises. It still doesn't work, because for some reason is not calling initializeDocDiff now, but 🤷🏼
https://github.com/humitos/readthedocs-client/blob/a78b2fd74b5b783cb6b5c326a70f0fe438f8169d/src/index.js#L73-L77

It looks like you're not resolving the promises here: https://github.com/readthedocs/readthedocs-client/blob/main/src/index.js#L67-L69
This would leave the Promise waiting for resolve/reject, and your then(...) will never be reached. You will need to call that resolve callback function to tell the Promise to resolve. Something like:
for (let fn of integrations) {
  promises.push(
    new Promise((resolve) => {
      resolve(fn(config));
    })
  );
}

Promise also gets a reject argument on the promise callback. This can be used to trigger error handling farther up on the Promise chain:
return new Promise((resolve, reject) => {
   ...
   reject(new Error("Oops!"));
});

Promises tend to have a steep learning curve. Native async code is a bit easier to understand in comparison, as there are lots of ways to shoot your foot with promises.


Use python slim to reduce docker build size

Description:
The current published docker image is built on top of python3.6 image and is 1.78GB. Using python3.6-slim I've got the size down to 1.47GB.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

Hello @vrih,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

63adc16831ac4bb4d146c0e4297a2834a00f2a7c - This commit has something that looks like an email address (dan@danbowtell.co.uk). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Hi @vrih,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

I'm no expert on the Python Docker images but I think that this will have unwanted side effects as packages are not available in the slim image.

At a very basic level it builds with all dependencies and runs for me. Is there any way you can think of to confirm that user wouldn't hit run time dependency issues? Is there a comprehensive test suite I could run?

The main issue is that we don't know what our over 100 dependencies are using.
Perhaps @michaelarnauts know more details about this.

Probably need to test on embedded/arm since you're less likely to get a platform matching wheel and have to compile instead. Is there some way to easily check if Python modules contain native code? I could download them all and check for .h/.c/.cpp files but not sure how accurate that is. I think those would be the only modules looking for libraries on the system?

gcc missing in slim

The current build process already removes the build tools after compilation I believe. I'm less concerned about building - those problems are mostly discoverable through test building. The bigger issue that I came across yesterday is not knowing what runtime dependencies exist - e.g. the ping component requires ping which is part of slim by default.
I'm happy to park this for now and maintain my own light build for my own purposes and maybe revisit later.

I've just looked at the python:3.6-slim variant, what is based on the debian:stretch-slim variant.
It doesn't look like the python:3.6-slim makes a lot of changes, and debian:stretch-slim does remove a few things like manpages that we really don't need in the container, by using the debuerreotype-slimify script from here: https://github.com/debuerreotype/debuerreotype/blob/master/scripts/debuerreotype-slimify
So it seems that the slim variant is quite usefull, but I don't see where the real binaries for wget and ping for example are removed (or not added).

I've benchmarked the -j8 and effectively just run the RUN virtualization/Docker/setup_docker_prereqs a couple of times with and without. It seems like -j8 is saving about 1 minute on build time for me 4:17 without vs about 3:15 with. I can split that off into a separate PR if you would like.
The other optimisation could possibly be to remove phantomjs. It's an abandoned project, and now that USPS component has been disabled I don't see any reference to it in the code base other than in the build scripts.

That's a nice increase!
I would prefer a different PR for the make optimalisations, since I think we need further testing to be able to make the switch to the -slim variant.
Also, if there is no other code that is using phantomjs, i would gladly see it removed also, but we need to make sure.

The other optimisation could possibly be to remove phantomjs. It's an abandoned project, and now that USPS component has been disabled I don't see any reference to it in the code base other than in the build scripts.

Like that

All requirements need to be build and installed when the image is build. So if the Docker image builds, we can assume it's 👍

Except external binaries like ping, arp, curl, wget, .... These only are used at runtime.

Going to close this. We are discussing slimming down docker in architecture repo https://github.com/home-assistant/architecture/issues/103


Update index.md "Need to move 'User and Group Quotas' to next release"

Need to move 'User and Group Quotas' to next release

Could you move the section about "User and Group Quotas" to the bottom of the page too? Or delete that part, and only have it in the table that you modified.
Thanks!


Custom Fields [NIP-??] Positive Interoperability Tracker

app
Write
Read
comment

ditto
✅
✅
https://njump.me/note1earm5sc53hl6yhk58xluc6kcg9zxj0272fcnw9auweu8vcghaw4spzkg2k

amethyst
?
?
?

Implementation in Ditto looking lovely @alexgleason :muscle:

@vitorpamplona did you add to amethyst?


$code not utilized

$code does not appear to actually be utilizedin font-family

just ran into this my self. Is the project open to pull requests?


[GSProcessing] Fix support for large integer values for noop transfomation

For now we also parse all numerical CSV values as float64. In the future we'll allow the use the option to choose, defaulting at float32.
Issue #, if available: Fixes #706
Description of changes:

For CSV numerical feature input always parse as float64. We plan to allow this to be user-defined in the future.
For CSV regression label input always parse as float64. We plan to allow this to be user-defined in the future.
For Parquet numerical feature input for noop transformation, use the input's data type, to avoid precision loss by converting to float32.
For CSV numerical input for noop transformation (which is parsed as string), output to float64.

How this was tested:

Added relevant unit test
Tested on generated graph dataset with integer features with values up to 80M, results shown below

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

A higher level question: because the current change will read in data as Double, do we allow users to specify the output data type to be Float? Or you will make the output type change in the next PR?

A higher level question: because the current change will read in data as Double, do we allow users to specify the output data type to be Float? Or you will make the output type change in the next PR?

The data are read in as Double only when the source is CSV, and only for numerical features and regression labels. Right now the output type is determined by Spark, for example scaling+imputation produces Doubles, and we don't recast those to avoid precision issues.
In the follow up PR the user will be able to optionally define the input/output data type (they should always be the same).


Feat/log level notebook

This commit closes#319 as sets log level to INFO if it is a notebook file

@AdvaitChandorkar07 I very much appreciate your submission! The code looks great! I'll get the workflow permissions fixed.

Labeller and doc build steps of workflow require membership in ansys org. Merging with failed steps.


When the uri have the slash, the next request should also have the slash

Sorry, another case of failure :-) .

#25  , #26 .

Turns out this is primarily a problem in phly/http, which currently prefixes the path with a "/" if it does not begin with one -- even when empty! Will have a fix for both projects shortly.

Yes, that was why I opened https://github.com/phly/http/issues/22 . But I was only able to reproduce it at here.

Finally I changed it to use
<?php
$server = $request->getServerParams();
$path = parse_url($server['REQUEST_URI'], PHP_URL_PATH);

as $req->getUri()->getPath() will return nothing when the request is at http://example.com


Update python-quiz.ts

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #XXXXX

added a Python question

updated as asked


nuclear envelope reassembly/assembly parentage

I expected these terms
GO:0031468    nuclear envelope reassembly
GO:0051081    nuclear envelope disassembly
to be
GO:0071763    nuclear membrane organization
A process that is carried out at the cellular level which results in the assembly, arrangement of constituent parts, or disassembly of the nuclear inner or outer membrane.
so I think they should be renamed
"nuclear membrane reassembly/disassembly"
These term refer to membranes and the "nuclear envelope" also includes the intermembrane space.

But envelope is different from membrane - you think they have been misused ?
Again we have a large number of terms, not sure they all describe different processes:
'nuclear envelope organization'
'nuclear envelope disassembly'
'meiosis I nuclear envelope disassembly'
'meiosis II nuclear envelope disassembly'
'meiotic nuclear envelope disassembly'
'mitotic nuclear envelope disassembly'
'mitotic nuclear pore complex disassembly'
'sperm nuclear envelope removal'
'nuclear envelope reassembly'
'meiosis I nuclear envelope reassembly'
'meiosis II nuclear envelope reassembly'
'meiotic nuclear envelope reassembly'
'mitotic nuclear envelope reassembly'
'pronuclear envelope synthesis'
'male pronuclear envelope synthesis'
'nuclear membrane organization'
'mitotic nuclear membrane organization'
'nuclear fragmentation involved in apoptotic nuclear change'
'nuclear inner membrane organization'
'nuclear outer membrane organization'

But envelope is different from membrane - you think they have been misused ?
Again we have a large number of terms, not sure they all describe different processes:
'nuclear envelope organization'
'nuclear envelope disassembly'
'meiosis I nuclear envelope disassembly'
'meiosis II nuclear envelope disassembly'
'meiotic nuclear envelope disassembly'
'mitotic nuclear envelope disassembly'
'mitotic nuclear pore complex disassembly'
'sperm nuclear envelope removal'
'nuclear envelope reassembly'
'meiosis I nuclear envelope reassembly'
'meiosis II nuclear envelope reassembly'
'meiotic nuclear envelope reassembly'
'mitotic nuclear envelope reassembly'
'pronuclear envelope synthesis'
'male pronuclear envelope synthesis'
'nuclear membrane organization'
'mitotic nuclear membrane organization'
'nuclear fragmentation involved in apoptotic nuclear change'
'nuclear inner membrane organization'
'nuclear outer membrane organization'

Well it is different, but this is tricky.
The lumen is created as a consequence of the assembly of the membranes.
I'm guessing that people refer to the "envelope" assembly or disassembly because both membranes assemble or disassemble simultaneously.  But if we call this process "nuclear envelope assembly" it is difficult to connect  the process to "nuclear membrane organization" which  is useful.
This is the problem as I see it:

I would rather the terms were "nuclear membrane assembly, which is a) more precise, and b) would allow the annotation to be correctly inherited by "nuclear membrane organization"
Perhaps there should only be a single term mentioning envelope  "nuclear envelope organization" which could be a superclass of  everything  and allows any (non membrane) organization which might occur in the luman to be grouped. But  then make the assembly/disassembly terms refer to "nuclear membrane assembly/disassemly"
that would involve
'pronuclear envelope synthesis' ->'pronuclear membrane synthesis'
nuclear envelope disassembly -> nuclear membrane disassembly (the lumen is not disassembled)
All of the other "nuclear envelope assembly/reassembly" terms would merge  or change into into their cognate "nuclear membrane assembly" terms.
I think that would work but tagging Gautam Dey to check @deygautam
In fact, I think there is a logical issue here anyway.
organelle envelope
Definition
A double membrane structure enclosing an organelle, including two lipid bilayers and the region between them. In some cases, an organelle envelope may have more than two membranes.
If envelope + membrane + lumen then "envelope" terms,  can't be a descendant of "membrane", it's broader.

Well it is different, but this is tricky.
The lumen is created as a consequence of the assembly of the membranes.
I'm guessing that people refer to the "envelope" assembly or disassembly because both membranes assemble or disassemble simultaneously.  But if we call this process "nuclear envelope assembly" it is difficult to connect  the process to "nuclear membrane organization" which  is useful.
This is the problem as I see it:

I would rather the terms were "nuclear membrane assembly, which is a) more precise, and b) would allow the annotation to be correctly inherited by "nuclear membrane organization"
Perhaps there should only be a single term mentioning envelope  "nuclear envelope organization" which could be a superclass of  everything  and allows any (non membrane) organization which might occur in the luman to be grouped. But  then make the assembly/disassembly terms refer to "nuclear membrane assembly/disassemly"
that would involve
'pronuclear envelope synthesis' ->'pronuclear membrane synthesis'
nuclear envelope disassembly -> nuclear membrane disassembly (the lumen is not disassembled)
All of the other "nuclear envelope assembly/reassembly" terms would merge  or change into into their cognate "nuclear membrane assembly" terms.
I think that would work but tagging Gautam Dey to check @deygautam
In fact, I think there is a logical issue here anyway.
organelle envelope
Definition
A double membrane structure enclosing an organelle, including two lipid bilayers and the region between them. In some cases, an organelle envelope may have more than two membranes.
If envelope + membrane + lumen then "envelope" terms,  can't be a descendant of "membrane", it's broader.

I asked @deygautam
"I agree - there is no reason to distinguish nuclear envelope from nuclear membrane terms. I think your idea makes a lot of sense! "
(note this applies to  the "assembly"  process terms, it is still fine to have envelope as a component
for IM+lumen+OM)
so I propose:
'nuclear envelope disassembly' ->   nuclear membrane disassembly
and equivalent for the descendants
'nuclear envelope reassembly'.  - > nuclear membrane reassembly
and equivalent for the descendants
I don't know enough to know if the mitotic/meiotic p use different pathways/ genes
so I suggest we keep the terms specific  for the time being. They can be merged up later if desired. I'm more concerned about the incorrect parentage because it affects query results.
v.

I will not be able to keep that is_a relation:
GO:0007077 mitotic nuclear envelope disassembly
[is_a relation] GO:0140516 mitotic nuclear pore complex disassembly
Any suggestions?

GO:0140516 mitotic nuclear pore complex disassembly, changed subclasses to
+is_a: GO:0032984 ! protein-containing complex disassembly
+relationship: part_of GO:0000278 ! mitotic cell cycle
+relationship: part_of GO:0006998 ! nuclear envelope organization

GO:0140516 mitotic nuclear pore complex disassembly
is part_of  mitotic nuclear envelope disassembly
(the pore complex disassembly triggers the envelope disassembly in some way. Because pombe because it does closed mitosis, this is localized disassembly  at the  spindle midzone.)

OK  in this case I dont change 'mitotic nuclear envelope disassembly' to 'mitotic nuclear membrane disassembly'
but then it doesn't make sense to change its parent, 'nuclear envelope disassembly', to nuclear membrane  disassembly
I am not clear what you are asking anymore...

You can have a look at the changes here
https://github.com/geneontology/go-ontology/pull/21078/commits/98a04a7d2c9414ee1af5c976ea581c4a23d66fe5

Sorry I typed 'envelope' I meant 'membrane'

Please reopen if you see other problems.

I don't see these changes yet. They did not come through to quickgo either. Is everything working with the releases?

The latest release was on March 16th, so those should now be in.

the only thing i noticed was

GO:0031468 should be under
GO:0071763 nuclear membrane organization

Hi @pgaudet  could you add this parent (last comment), and this one can re-close (I was just about to open a new ticket for this)


Display update Download progress alert for non-mandatory updates as well

Describe the solution you'd like
On Android, while distributing a mandatory update via the the AppCenter Android SDK, an in-app progress dialog is shown.
While distributing a non-mandatory update, this dialog is not shown. Only the Android OS Download manager notification is visible in the notification drawer.
It's likely to be this condition:
https://github.com/microsoft/appcenter-sdk-android/blob/develop/sdk/appcenter-distribute/src/main/java/com/microsoft/appcenter/distribute/Distribute.java#L757
I am not sure to understand why we wouldn't show the in-app download progress alert for non-mandatory updates, it would be a bit more user friendly. Or at least have the control on showing or not this alert.
Describe alternatives you've considered
N/A apart from HockeyApp.

Hi. Thank you for bringing this feature up. I will get the right team assigned to take a look at the request.


Log which SSH key is used in e2e SSH test

Log which SSH key is used in e2e SSH test to help debug #7714.

LGTM.

LGTM.  Merging in anticipation of Shippable and Travis succeeding on this one line log statement addition.  There's quite a backlog on PR builds today.


[zh] Localize 'abstraction'

[ ] Localize the content/en/abstraction.md to content/zh/ in dev-zh.

ack


1.8.9 update needed

i use 1.8.9 for Hypixel skyblock and shoulder surfing is one of my favorite mods, i downloaded the 1.8.9 version, booted it up, and it worked(kinda) the camera was not offset and i could not figure out how to offset the camera ( i tried using the arrow keys the did not work) please fix the issue so i can use your amazing mod!

Is there really no way to use a newer version of the game? If so what other mods are you using? Please provide the launcher log of the game otherwise there is no way for me to even remotely fix this.

im using skyblock addons(1.8.9 ver 1.5.5) optifine(newest) and inventory hud, can we further discuss this privately on discord? (my username is !DOGGO!#3300 ill send the logs there

the reason im playing on 1.8.9 is because the skyblock addons mod only supports 1.8.9

Just put the logs up on pastebin or gist.github.com

is drive okay too?

sure

https://drive.google.com/file/d/1rWdfFC38QTBE9CUNphu9incsiq2WghNX/view?usp=sharing

the arrow keys do nothing just like the pg up and pg down

is there also a debug.log ?

one sec

no there is not

can you maybe remake the mod?

i will take a look at how easy the current 1.12 version is to backport

ok, thanks!

i really like your mod and i use it every day on 1.16.5 either way thanks for your work!

also the 1.7.10 version works correctly i tried it before


exclude build directory from flake8 check

As the title says...

Pull Request Test Coverage Report for Build 470

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 85.871%

Files with Coverage Reduction
New Missed Lines
%

xclim/utils.py
1
94.81%

Totals

Change from base Build 465:
-0.03%

Covered Lines:
705

Relevant Lines:
821

💛  - Coveralls


Charset.availableCharsets() does return null

java.nio.charset.Charset.availableCharsets() return null. I expect that this method returns a map with the available charsets; minimum the JVMs must have: US-ASCII, ISO-8859-1, UTF-8, UTF-16BE, UTF-16LE, UTF-16.

Weird; that should work. The logic appears to be in 100% Java, but maybe there's a system property I need to set. Will look at further later.


Completion of error handling

Would you like to add more error handling for return values from functions like the following?

calloc ⇒ chunk_wait_for
pthread_mutex_init ⇒ net_start

Thanks for the small source code improvement.

I suggest to avoid ignorance of return values a bit more.
Would you like to detect every error situation as early as possible?


security/acme-client: add support for dnsapi "Euserv.eu"

Additional changes for https://github.com/opnsense/plugins/issues/1779

added option "--insecure"
increased maximum dns sleep time

@fraenki should this go into 20.1.5 with os-acme-client 1.31?

@fichtner Yes :)


401 logged to console despite successful request to solidcommunity.net server

Search terms you've used
401
Bug description
I can successfully use getSolidDataset to get a directory but it also logs a 401 error to the console.
To Reproduce

Call getSolidDataset

Minimal reproduction
https://github.com/AJamesPhillips/solid-bugs/tree/1567be3
(I decided not to use codesandbox as there were a lot of error and warning messages to the console).
Expected result
No error log out.
Actual result

Environment
$ npx envinfo --system --npmPackages --binaries --npmGlobalPackages --browsers

  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 92.0.4515.159
  npmPackages:
    @inrupt/solid-client: ^1.1.0 => 1.11.0
    @inrupt/solid-client-authn-browser: ^1.11.0 => 1.11.0
    @inrupt/vocab-common-rdf: ^0.6.3 => 0.6.3
    buffer: ^6.0.2 => 6.0.3
    clean-webpack-plugin: ^3.0.0 => 3.0.0
    html-webpack-plugin: ^4.5.0 => 4.5.2
    lodash: ^4.17.20 => 4.17.21
    rdf-namespaces: ^1.9.2 => 1.9.2
    solid-auth-client: ^2.5.3 => 2.5.6
    tripledoc: ^4.4.0 => 4.4.0
    webpack: ^5.5.0 => 5.51.1
    webpack-cli: ^4.2.0 => 4.8.0
    webpack-dev-server: ^3.11.0 => 3.11.2
  npmGlobalPackages:
    npm-run: 5.0.1
    npm: 6.14.8
    pnpm: 6.3.0

Thanks for reporting this ! I'll see where that might come from. And no problem on not using codesandbox, any way of reproducing is fine :)

TL;DR: That's a normal behaviour, and it will disappear if you add trailing slashes to container IRIs.
 Note that the container URL does not include a trailing slash, which it should.
The Solid server redirects your request to https://some.pod/container/ (trailing slash included) because that's the canonical IRI of the resource
At this point, you have an authentication issue: the Solid-OIDC spec mandates that the Access Tokens (which are used to prove that a given application has access to certain resources on a Pod) uses a security mechanism called DPoP. Among other things, this mechanism ensures that the token used to authenticate a request is scoped to the target resource, in order to prevent extraction. That means that if you're giving an access token to an evil Pod Server serving https://evil.pod/your-resource, it is not going to be able to use the token to access https://another.pod/your-banking-data. However, it also means that the access token used for https://some.pod/container isn't valid for https://some.pod/container/, because the tagret IRIs don't match (even just by a trailing slash). That's what gets you a 401.
The authentication library that you are using, namely @inrupt/solid-client-authn-browser, knows that this can be an issue. On 401, it checks that the response IRI matches the IRI which was initially fetched (scripts aren't aware of redirections for security reasons, they don't see the intermediary 3XX responses). If it gets a 401 and the IRIs don't match, it knows it should re-attempt with the redirected target IRI (to construct an appropriate authentication header). If you're interested about the details for this, the actual implementation is available at https://github.com/inrupt/solid-client-authn-js/blob/a48f1cccdc7ed82d5f56f2738f97e0a2d8724ed7/packages/core/src/authenticatedFetch/fetchFactory.ts#L275.
The request to the container with the canonical IRI is not redirected, so its authentication is valid, and the app gets the correct result without being aware of the 401 that was basically intercepted by the authenticated fetch library.

You can test this by manually adding a trailing slash to the container IRI at https://github.com/AJamesPhillips/solid-bugs/blob/1567be367810cd4e3efa5a0a55f4037ee5beb7c6/src/index.js#L113, and see that the 401 is no longer logged.

@NSeydoux that is an amazing response.  Thank you.

anything we could have done at a library level to improve the developer experience ?

Add a line of documentation?


Revert "test: disable fqdn restart test"

This reverts commit a4d86b2abc8fa0c77306cc41a3bb2f753b7fe1d2.

test-me-please

Was there a change that fixed it?

@nebril No, but need to see if/how this is failing. Converted this PR to draft.

test-me-please

test-me-please

test-me-please

CI failed due to quay error in precheck, retesting.

test-me-please

@jrajahalme cool, I thought it was out for review and got confused. Good luck!

Health status between nodes failing again: #6728

test-me-please

test-me-please

Opening this for review, have had 4 consecutive without the FQDN test failing on any pipeline. Testing once more, but looks like this flake is rare enough to be merged to master.

Coverage increased (+0.01%) to 37.042% when pulling eab2ee45dfb30d25ec09a0e5c87d1118e6c5e4ba on pr/jrajahalme/test-re-enable-fqdn into f8da3b94fdd315105d360b1b7100e80092f752f2 on master.

5 consecutive CI runs without this test flaking, merging this in.


add (verified) webhook support

This PR updates the app to optionally receive webhooks, and use the validateWebhook function from the Replicate JS client to verify the request before processing.
To Do

[ ] Make webhooks optional
[ ] Document how to test webhooks locally (ngrok, signing secret, etc)
[ ] Maybe update the whole app to use Next.js App router so the req vs Request shenanigans go away?

Closing in favor of https://github.com/replicate/getting-started-nextjs/pull/102


Introduce plugin storage

This patch is to add a new feature "plugin storages" to store key-value data to pluggable storages.
It is useful for many purposes:

re-implementation of pos file of file output plugin
store intermediate state of counter plugins (datacounter, flowcounter, ...) and make it resumable
make configurations dynamic and readable from external storages

This feature requires many more additions not included in this change for actual use cases (will be added in following changes):

simple storage path configuration with plugin id via system config
multi process support

@repeatedly Could you review this?

@sonots Could you review this first? I'm now busy so I don't have enough time to review large patch...

Comments

JSONStorage sounds like that we have to create new storage just for different formats. It looked FileStorage with json format is better design

Are you not assuming an extendability like S3Storage? Or, extendability is only for formats?

I want incr methods which allows us to atomically increment for counter plugins.

@sonots File is not good abstraction, because S3 (and many other file systems) has files.
I considered some other naming options like LocalFileStorage, LocalStorage or others, but couldn't find something good enough. (Because json storage also works as on-memory storage if local disk path is not configured.)
I don't have any plan to make formats configurable for now.

incr or such like operation will be supported by Counter API #858

LocalStorage

LocalStorage sounds good for me because it expresses it would store into local memory or local file, but not S3. At least, it should be better than JSONStorage because it sounds like it would store into S3 (we can not know the storage location from its name)

incr

Okay, thanks.

I changed default storage name from "json" to "local".

LGTM!

I pushed fixes for comments.

LTGM

Okay.


Translation is broken

See https://github.com/qwat/docs/pull/21 for more details
Do not use i18 files on transifex (these are wrong generated files )

Files are deleted, this should be fixed.


Error: Use messageId instead of message is incorrect for field desc

This error will show if the suggestion has desc instead of messageId
The error message is misleading
I am willing to submit a PR

I assume you're talking about the prefer-message-ids rule. Yes, it would make sense to update the violation message to mention desc when reporting on a suggestion.

so should I fix it

PR welcome!


download_ee_image()

          `download_ee_image()` is wrapper for the [geediim](https://github.com/leftfield-geospatial/geedim) package. It is most suitable for download original image rather than computation results. The more complicated your workflow, the less likely `download_ee_image` will work. For long-running computational results, you should use `ee_export_image_to_drive`.

Originally posted by @giswqs in https://github.com/gee-community/geemap/issues/1882#issuecomment-1890277036

Teacher Wu, my analysis is for seven African countries, which will definitely involve large-scale image processing and analysis. The reason why I want to export the image is because when I reach a certain step, gee pops up a timeout prompt. I have listened to your course. According to the suggestions in your course, I want to download it locally or upload it to the asset for further analysis to avoid timeout problems. However, it takes a long time to download to the local drive and upload to the asset. It takes a long time. Hours or even more than ten hours, I would like to ask the first question. In this case, do I have to choose to export to drive and wait for a long time without any other solution? Looking forward to the teacher's reply, I am your loyal fan.


Create annotated tags

I think it is useful that this plugin support to create an annotated tag.

+1 this, as tags are so vital to versioning for me, being able to add, and push tags would be great. @akonwi any plans for this? If I can sedate the kids long enough, perhaps I can help on this feature.

There are tag features in the package. They can be accessed through the Tags command in the palette. You can add a tag with an annotation message. The tags can also be pushed, checked-out, verified, and deleted.
Is there something missing?


Chromedriver 2.15 required for performance logs on Chrome 42

Otherwise you get an error on starting the browser when you have "performance" enabled in Chrome's loggingPreferences:
org.openqa.selenium.WebDriverException: unknown error: unhandled inspector error: {"code":-32000,"message":"Timeline is deprecated. Please use Tracing instead"}
(Session info: chrome=42.0.2311.90)
(Driver info: chromedriver=2.14.313457 (3d645c400edf2e2c500566c9aa096063e707c9cf),platform=Linux 3.13.0-49-generic x86_64) (WARNING: The server did not provide any stacktrace information)

Next npm install should pick it up.  Updated in selenium-binaries.  Thanks for reporting @NickTulett !


Multiple routes for a module

In your style guide you only declare one route in a module, how could i setup multiple routes in a single module?
I ask this because i have a users module, where i want these routes:
users/
users/:id

@JFSolorzano I guess you can declare it in the same file your-module.routes.js

Following the example, this is the function getRoutes:
function getRoutes() {
        return [
            {
                url: '/avengers',
                config: {
                    templateUrl: 'app/avengers/avengers.html',
                    controller: 'Avengers',
                    controllerAs: 'vm',
                    title: 'avengers',
                    settings: {
                        nav: 2,
                        content: ' Avengers'
                    }
                }
            }
        ];
    }

How could i add /avengers/:id ?

function getRoutes() {
    return [
        {
            url: '/avengers',
            config: {
                templateUrl: 'app/avengers/avengers.html',
                controller: 'Avengers',
                controllerAs: 'vm',
                title: 'avengers',
                settings: {
                    nav: 2,
                    content: ' Avengers'
                }
            }
        },
        {
            url: '/avengers/:id',
            config: {
                templateUrl: '',
                controller: '',
                controllerAs: '',
                title: '',
                settings: {
                    nav: ,
                    content: ''
                }
            }
        }
    ];
}

@bampakoa Thnks, another question, in that example johnpapa used ngRoute right?, how could i change it to use ui-route?

@JFSolorzano that is correct. John uses ngRoute. I have not tried to port its example using ui-route but I think it can be done easy. Maybe @johnpapa would like a PR for its guidelines?

Yes please!

I use hottowel as my project skeleton and it definitely use ui-router,  not ngRoute.

Mmm is that a generator?, i'm trying not to use generators, i just take the ideas that seems nice to me, and apply them to my code, and structures, could you please give me the link to know what you're talking about @attilacsanyi ?

yep, use multiple in a file for that module. And just use ui-router instead. I use ui-router, myself.
closing ... as this is a question.

Oh i see what you mean, this is hottowel routing with ui-routing right?
i see some differences between your way of routing compared with the way ui-router docs says, what i did is this:
(function () {

    'use strict';

    angular.module('app.users')

        .config(['$stateProvider', '$urlRouterProvider', routes]);

    /* @ngInject */
    function routes($stateProvider, $urlRouterProvider) {

        $urlRouterProvider.otherwise("/");
        // Set up the states
        $stateProvider
            .state('users', {
                url: '/users',
                templateUrl: 'modules/users/users.view.html',
                controller: 'UsersController',
                controllerAs: 'vm'
            });

    }

})();

But if i use your code but i want to use states instead of routes, i would do the following right @johnpapa?

(function () {
    'use strict';

    angular.module('app.users')

        // Collect the routes
        .constant('routes', getRoutes())

        // Configure the routes and route resolvers
        .config(['$stateProvider', 'routes', routeConfigurator]);

    function routeConfigurator($stateProvider, routes) {

        routes.forEach(function (r) {
            $stateProvider.state(r.state, r.config);
        });
        $routeProvider.otherwise({redirectTo: '/'});
    }

    // Define the routes
    function getRoutes() {
        return [
            {
                state: 'listUsers',
                config: {
                    url: '/users',
                    templateUrl: 'modules/users/users.view.html'
                    title: 'users',
                    controller: 'UsersController',
                    controllerAs: 'vm'
                }
            },
            {
                state: 'specificUser',
                config: {
                    url: '/users/:id',
                    title: 'specific user',
                    templateUrl: 'modules/users/userDetail.view.html'
                    controller: 'UsersController',
                    controllerAs: 'vm'
                }
            }
        ];
    }
})();

Yes I definitely mean this @JFSolorzano


Add import/export support

Add import/export support (using yaml or json ?)

JSON should be easier because it doesn't require an additional dependency to create YAML and it makes a clear cut between the resources / policy / context

Might be harder to read/edit though.

You think its okay to add the export / import buttons under load/save? space in the bar is limited and has already some buttons

You think its okay to add the export / import buttons under load/save?

Sounds good, maybe with an export all option ?


Unable to get unique page visits

I have been trying to get unique page visits of certain route but no matter what the count always shows 1
Please let me know if I am doing it wrong or is there any other way
return Tracker::logByRouteName('home')                 ->select('tracker_log.session_id')         ->groupBy('tracker_log.session_id')         ->distinct()         ->count('tracker_log.session_id');

same happening here, returns 1 every time, have anyone got a fix on this yet ?

Should be fixed on next version.


chore(tests): add toxiproxy for integration testing network issues

removes unnecessary speckle-server components not required for integration tests
speckle-server is addressable on port 3001, proxied via toxiproxy

Refer to https://github.com/specklesystems/speckle-sharp-sdk/pull/39 for more details.

Description & motivation

Changes:

To-do before merge:

Screenshots:

Validation of changes:

Checklist:

[ ] My pull request follows the guidelines in the Contributing guide?
[ ] My pull request does not duplicate any other open Pull Requests for the same update/change?
[ ] My commits are related to the pull request and do not amend unrelated code or documentation.
[ ] My code follows a similar style to existing code.
[ ] I have added appropriate tests.
[ ] I have updated or added relevant documentation.

References

Closing, we'll bring this in v3, not v2


webserver example not working

Hello,
[pyubx2 v1.26] [F9P]
I can't get the webserver example to work...
I added a line print(parsed_data) on line 166 to better illustrate my problem
Here is what happens when I start the service
python3 ubxserver.py
Connecting to serial port /dev/pts/2 at 115200 baud...

Starting serial read thread...

Starting HTTP Server on http://<IP_ADDRESS>:8123 ...
Press Ctrl-C to terminate.

<UBX(NAV-PVT, iTOW=14:37:23, year=2022, month=4, day=11, hour=14, min=37, second=23, validDate=1, validTime=1, fullyResolved=1, validMag=0, tAcc=21, nano=32824, fixType=3, gnssFixOk=1, difSoln=1, psmState=0, headVehValid=0, carrSoln=1, confirmedAvai=1, confirmedDate=1, confirmedTime=1, numSV=19, lon=-1.0281844, lat=45.9921417, height=55272, hMSL=7149, hAcc=14, vAcc=11, velN=5, velE=3, velD=-5, gSpeed=6, headMot=307.36934, sAcc=83, headAcc=180.0, pDOP=1.28, invalidLlh=0, lastCorrectionAge=0, reserved0=1044519020, headVeh=0.0, magDec=0.0, magAcc=0.0)>
Something went wrong Message checksum b'\x00\x00' invalid - should be b'\xab\t'
<UBX(RXM-SFRBX, gnssId=Galileo, svId=30, reserved0=1, freqId=0, numWords=9, chn=49, version=2, reserved1=208, dwrd_01=172157743, dwrd_02=3776755709, dwrd_03=3154092032, dwrd_04=2231468032, dwrd_05=3003596800, dwrd_06=42, dwrd_07=2863297454, dwrd_08=3112124416, dwrd_09=0)>
<UBX(RXM-SFRBX, gnssId=Galileo, svId=36, reserved0=1, freqId=0, numWords=9, chn=20, version=2, reserved1=216, dwrd_01=174254895, dwrd_02=3777549821, dwrd_03=3154092032, dwrd_04=2231468032, dwrd_05=3003605611, dwrd_06=2734180266, dwrd_07=2863292447, dwrd_08=1249853440, dwrd_09=0)>
Something went wrong Message checksum b'\x00\x00' invalid - should be b'J\x0b'
<UBX(RXM-SFRBX, gnssId=GPS, svId=23, reserved0=0, freqId=0, numWords=10, chn=1, version=2, reserved1=0, dwrd_01=583048449, dwrd_02=2337352243, dwrd_03=1006485321, dwrd_04=2347592695, dwrd_05=621987212, dwrd_06=1041481844, dwrd_07=47615079, dwrd_08=37660748, dwrd_09=57217316, dwrd_10=147464095)>
<UBX(RXM-SFRBX, gnssId=GPS, svId=29, reserved0=0, freqId=0, numWords=10, chn=3, version=2, reserved1=32, dwrd_01=583048449, dwrd_02=2337352243, dwrd_03=75589508, dwrd_04=176016204, dwrd_05=1022468931, dwrd_06=18268243, dwrd_07=42600581, dwrd_08=2229971034, dwrd_09=2205569994, dwrd_10=2294925823)>
<UBX(RXM-SFRBX, gnssId=GLONASS, svId=10, reserved0=0, freqId=0, numWords=4, chn=13, version=2, reserved1=32, dwrd_01=1707355723, dwrd_02=3329273952, dwrd_03=282068992, dwrd_04=65470469)>
<UBX(RXM-SFRBX, gnssId=GLONASS, svId=20, reserved0=0, freqId=9, numWords=4, chn=30, version=2, reserved1=32, dwrd_01=1707355723, dwrd_02=3329273952, dwrd_03=282068992, dwrd_04=65470469)>
<UBX(RXM-SFRBX, gnssId=GLONASS, svId=20, reserved0=2, freqId=9, numWords=4, chn=101, version=2, reserved1=64, dwrd_01=1707355723, dwrd_02=3329273952, dwrd_03=282068992, dwrd_04=65470469)>
<UBX(RXM-SFRBX, gnssId=GLONASS, svId=11, reserved0=0, freqId=7, numWords=4, chn=17, version=2, reserved1=32, dwrd_01=1707355723, dwrd_02=3329273952, dwrd_03=282068992, dwrd_04=65470469)>
<UBX(RXM-SFRBX, gnssId=GLONASS, svId=11, reserved0=2, freqId=7, numWords=4, chn=100, version=2, reserved1=64, dwrd_01=1707355723, dwrd_02=3329273952, dwrd_03=282068992, dwrd_04=65470469)>
<UBX(RXM-SFRBX, gnssId=Galileo, svId=36, reserved0=5, freqId=0, numWords=8, chn=64, version=2, reserved1=64, dwrd_01=9786709, dwrd_02=1431655765, dwrd_03=1431655765, dwrd_04=1383366656, dwrd_05=2278359040, dwrd_06=42, dwrd_07=2863309660, dwrd_08=1568620544)>
<UBX(RXM-SFRBX, gnssId=Galileo, svId=30, reserved0=5, freqId=0, numWords=8, chn=75, version=2, reserved1=64, dwrd_01=9786709, dwrd_02=1431655765, dwrd_03=1431655765, dwrd_04=1383366656, dwrd_05=2278359040, dwrd_06=42, dwrd_07=2863309660, dwrd_08=1568620544)>
<UNKNOWN PROTOCOL(header=b'$w')>
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/home/basegnss/pyubx2/examples/webserver/ubxserver.py", line 167, in set_data
    if parsed_data.identity == 'NAV-PVT':
AttributeError: 'str' object has no attribute 'identity'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.9/threading.py", line 954, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.9/threading.py", line 892, in run
    self._target(*self._args, **self._kwargs)
  File "/home/basegnss/pyubx2/examples/webserver/ubxserver.py", line 150, in _read_thread
    self.set_data(parsed_data)
  File "/home/basegnss/pyubx2/examples/webserver/ubxserver.py", line 192, in set_data
    except ube.UBXMessageError() as err:
TypeError: catching classes that do not inherit from BaseException is not allowed

F9P_rover_usb_uart1-RAWX_SFRBX_PVT.txt

Hello
except ube.UBXMessageError() as err: should be except ube.UBXMessageError as err: ??

Hi @Hadatko ,
same problem but cleaner without the ()

Starting serial read thread...

Starting HTTP Server on http://<IP_ADDRESS>:8123 ...
Press Ctrl-C to terminate.

<UBX(NAV-PVT, iTOW=15:17:25, year=2022, month=4, day=11, hour=15, min=17, second=25, validDate=1, validTime=1, fullyResolved=1, validMag=0, tAcc=1006, nano=155037, fixType=3, gnssFixOk=1, difSoln=1, psmState=0, headVehValid=0, carrSoln=2, confirmedAvai=1, confirmedDate=1, confirmedTime=1, numSV=21, lon=-1.0281327, lat=45.9921395, height=50492, hMSL=2369, hAcc=36, vAcc=50, velN=6, velE=-13, velD=-11, gSpeed=14, headMot=116.17642, sAcc=99, headAcc=180.0, pDOP=1.21, invalidLlh=0, lastCorrectionAge=0, reserved0=1044519020, headVeh=0.0, magDec=0.0, magAcc=0.0)>
Something went wrong Message checksum b'\x00\x1a' invalid - should be b'\x8e\x84'
<UNKNOWN PROTOCOL(header=b'$\x00')>
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python3.9/threading.py", line 954, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.9/threading.py", line 892, in run
    self._target(*self._args, **self._kwargs)
  File "/home/basegnss/pyubx2/examples/webserver/ubxserver.py", line 150, in _read_thread
    self.set_data(parsed_data)
  File "/home/basegnss/pyubx2/examples/webserver/ubxserver.py", line 167, in set_data
    if parsed_data.identity == 'NAV-PVT':
AttributeError: 'str' object has no attribute 'identity'

Hi
It's OK, it works on my side. Thanks a lot!
I use str2str to inject RTCM3 data for RTK centipede.fr, so I'm going to modify the web interface a bit to have all the PVT information on the precision of hAcc, vAcc, carrSoln data,...
it's for the physalia and Rtkbase project,
I'll share my progress.

OK glad the web server example works OK, and if it provides a useful basis for your own project then great! It's easy enough to amend the set_data() routine to accept other incoming data sources (including NMEA and RTCM3 if needs be). Do please share progress - thanks.
Just bear in mind that this is a fairly simplistic web server implementation; please heed the warning on the http.server library home page:

Out of interest, did you actually get to the bottom of the 'invalid checksum' and 'UNKNOWN PROTOCOL' errors? pyubx2 should be capable of handling any of the protocols output by a u-blox device such as the F9P (i.e. NMEA, UBX or RTCM3), so - as I say - errors of this kind are normally associated with a corrupt serial stream (e.g. incorrect baud rate or some other process interfering with the stream).
Are you happy for me to close this issue in relation to the web server example? If you continue to get persistent 'UNKNOWN PROTOCOL' errors from pyubx2 elsewhere then by all means raise a separate issue. Cheers.

Hello
Is it possible to make such webserver example for pyrtcm + ntrip?

Hi @foxittt. The intention with this example is to wrap a simple HTTP server around the parsed serial data stream from a local GNSS device. For simplicity sake it currently only parses selected UBX messages, but could easily be modified to parse NMEA and/or RTCM3 messages coming from the same device.
NTRIP, on the other hand, is already an internet-based protocol so much of this would be superfluous. You can access any (public) NTRIP caster by simply opening the appropriate URL in any web browser (e.g. http://rtk2go.com:2101), though you obviously wouldn't get the parsed RTCM3 data without further client-side processing.
Have a look at ntrip_handler.py in PyGPSClient for an illustration of how to receive and parse an NTRIP data stream: https://github.com/semuconsulting/PyGPSClient/blob/master/pygpsclient/ntrip_handler.py

Thanks! I`ll try.

Hi @foxittt ,
FYI This is typical output from an NTRIP caster as output by the PyGPSClient utility - is this the sort of thing you were looking to see?

Yes.


password are handled unsecurely

The same as https://github.com/nspcc-dev/neofs-node/issues/1297
Passwords should be handled directly via /dev/tty, bypassing standard inputs
https://lists.gnupg.org/pipermail/gnupg-devel/2017-August/032987.html
% echo qwe | neo-go wallet change-password -w /tmp/wallet.json
Error reading old password: inappropriate ioctl for device
zsh: done       echo qwe |
zsh: exit 1     ./offline-neofs/downloads/neo-go wallet change-password -w /tmp/wallet.json
% neo-go --version
neo-go version 0.98.3-pre-68-g3b39c8d6

So why do you think we're taking passwords from stdin? #2387 also.

redirecting stdin/stdout gives an error
lsof doesn't show /dev/tty is opened by neogo process

% neo-go wallet change-password -w wallet.json
Enter password >

In another terminal:
% sudo lsof -p `pidof neo-go` | grep dev
neo-go  19065 sergio    0u      CHR 136,15      0t0       18 /dev/pts/15
neo-go  19065 sergio    1u      CHR 136,15      0t0       18 /dev/pts/15
neo-go  19065 sergio    2u      CHR 136,15      0t0       18 /dev/pts/15
% cat /dev/pts/15

Type password in the first terminal, see it in the second.

check sudo or ssh for example


ImportError: cannot import name relay

I am trying to build and upload "relay frame led" example on pycharm but it gives this error:
ImportError: cannot import name relay
Note: Almost all the other examples are working fine
I am using Xbee3 digimesh pcb antenna.

I believe the User Relay frame feature is only available on the XBee3 ZigBee firmware: https://www.digi.com/resources/documentation/Digidocs/90002219/#container/cont_send_receive_user_data.htm
This page was incorrect in stating that it is supported on 802.15.4 and DigiMesh: https://github.com/digidotcom/xbee-micropython/tree/master/samples/xbee/communication/relay_frames_button

There should be a DigiMesh release later this month that includes support for User Relay frames.
802.15.4 support should be out by the end of September.

Thanks, I will test it once its released.


img node in HTML renderer

Hello,
Great packages, super useful!
Is it possible to render an img node in an HTML renderer?
Something similar to the @resaleai/receipt-image-node) but for browsers.
This with a way to have right and left text on the same line would make this library complete.

Sorry, I just found @resaleai/receipt-image-browser. It wasn't mentioned in the readme.

Just seeing this, I am still working on the documentation, so it will be better in the future!


TESTING BUILD

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

File
Status
Preview URL
Details

query-languages/m/access-database.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

query-languages/m/access-database.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add Organizes reference

Hello,
first of all thank you for these really great utilities, I've been able to generate a nodeset, and read an existing one to convert some part of it.
I'm trying to rework a nodeset exported from SIEMENS TIA Portal, to make it importable from https://github.com/Azure-Samples/iot-edge-opc-plc.
My problem is that I can't fathom how to insert into a child node a
<Reference ReferenceType="Organizes" IsForward="false">ns=1;i=2001</Reference>
to refer back to the parent node that correctly get a "HasComponent" reference.
But what I think would be really nice is to have some more samples, covering various use case, are there some documentation that I don't know of?
Thank you again,
Fabrizio

I'll answer my question
node.OtherReferencingNodes.Add(
    new NodeModel.NodeAndReference
    {
        ReferenceType = refOrganizeNode,
        Node = theParentNode
    });


Fix Move operations and expand librapay to support user modules and scripts

This issue corresponds with Solana issue https://github.com/solana-labs/solana/issues/6898

Fixes the Move example and librapay helpers to support the new Libra CLI compiler behavior
Adds support for publishing user modules
Adds support for running user scripts

@jstarry Would love some feedback on my novice .js :-)


should download.html be versioned

Are we fine with having the link to it under: /docs/x.x.x/download?

@ahmadnassri @montanaflynn @thefosk @Nijikokun

disagree, download.html should always point to the latest download, perhaps just add a link to "previous releases" which links back to github releases page (which acts as a nice to view & readable overview of changes and releases)

Ok, then we don't give instructions for old installations if they change later. (Which is fine for now, I don't think it's something we need to worry about now)
(at some point we just thought it was better to version it too)

ideally you want people to always get the latest and greatest, so the less visible old instructions / download methods are, the better for you to have less backward maintenance & support.

I agree with @ahmadnassri, the general solution is latest download w/ latest install guide otherwise people assume you still support older versions (LTS)

:+1: