WSL 2 DNS Request Issue, although connection to the internet is available

Environment
Windows build number: Microsoft Windows [Version 10.0.19041.450]
Your Distribution version: Release:        18.04
Linux version 4.19.104-microsoft-standard

Steps to reproduce
I'am getting the following error Temporary failure in name resolution when I ping a DNS like www.google.com (first recognized this failure after execute sudo apt-get update). If I ping the IP adress the request is successful.
I tried to adjust the resolv.conf and wsl.conf file like described here Fix DNS resolution in WSL2
I have installed also OpenSuse Leap 15.1 and 15.2 same problem occurs.
Windows Features Windows Subsystem for Linux and Windows Machine Platform are enabled.
I tried to disable and reenable all Hyper-V Virtual Ethernet Adapter same problem occurs.
I first used WSL 1 and everything was fine, but I also needed docker in the WSL that's why I upgraded my Windows OS version and WSL 1 to WSL 2 after this upgrade I got this issue.

WSL logs:
No logs available right now
Expected behavior
ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=106 time=52.7 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=106 time=49.4 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=106 time=48.8 ms
64 bytes from <IP_ADDRESS>: icmp_seq=4 ttl=106 time=50.1 ms
64 bytes from <IP_ADDRESS>: icmp_seq=5 ttl=106 time=47.7 ms

Actual behavior
ping google.com
ping: google.com: Temporary failure in name resolution

Please collect WSL logs and post the backlink to the feedback item here. Also please identify any third-party networking software that has been installed on Windows.

I contributed a feedback for the same problem using the same feedback title.
Here is the feedback link: https://aka.ms/AA9lxsh

Please collect WSL logs and post the backlink to the feedback item here. Also please identify any third-party networking software that has been installed on Windows.

Thank you for you comment, here are my WSL logs : https://aka.ms/AA9l3wt
I have no additional networking software installed just the Windows Defender.
BR

I had the same problems. It seems to be a firewall issue. The Hyper-V network is considered a "Public Network" and the default setting is to reject all incoming traffic on public networks, so the DNS packages get rejected.
You can adjust this in the firewall settings -> Public Network -> There is a checkbox to reject all incoming traffic even if there are rules...

I had the same problems. It seems to be a firewall issue. The Hyper-V network is considered a "Public Network" and the default setting is to reject all incoming traffic on public networks, so the DNS packages get rejected.
You can adjust this in the firewall settings -> Public Network -> There is a checkbox to reject all incoming traffic even if there are rules. You would have to uncheck that.
It is my desktop PC and it is just in a private network. If you are on a laptop and connect to public WiFis this setting could put you in danger.

Thank you very very very much🥇 now it is working also for me.

I don't want to turn off my firewall because of WSL 2 networking issue.


Smallweb RSS URL not recognized by Fraidycat

I use Fraidycat (fraidyc.at) as my RSS organizer, ad I wanted to add the Smallweb RSS (https://kagi.com/api/v1/smallweb/feed/) to my list of feeds.
Fraidycat doesn't recognize it as a valid RSS, though. The error message is:

Couldn't connect - check your spelling, be sure this URL really exists.
Add this follow anyway? (In case it might be down for the moment.)

I realize this may be a Fraidycat issue, not a Kagi Smallweb issue. Please close if inappropriate.
I would appreciate any suggestions you might have for adding this feed to my Fraidycat list.

Likely your RSS reader's issue as I haven't heard complaints from other users using different readers. have you asked them?


Log the checked-in tickets

Could be the same as #14 but into a different file and channel

This was implemented in 467c520a6c11629ed973f4ad5e64daa5ef4c5bfc and fcd8ef2de2d683555c60afa2390c34132a7dc324. Thanks everyone! 🎈


Allow to disable field suggestions

:)

Summary by Sourcery
This pull request introduces a new configuration option suggest_field in StrawberryConfig to allow disabling field suggestions in error responses. Documentation and tests have been updated to reflect this new feature.

New Features:

Introduced a new configuration option suggest_field in StrawberryConfig to disable field suggestions in error responses.

Documentation:

Updated the schema configurations documentation to include the new suggest_field option and other available configurations.

Tests:

Added tests to verify the default behavior of field suggestions and the ability to disable field suggestions using the new suggest_field configuration.

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

This release adds a new configuration to disable field suggestions in the error
response.
@strawberry.type
class Query:
    name: str

schema = strawberry.Schema(query=Query, config=StrawberryConfig(suggest_field=False))

Trying to query { nam } will not suggest to query name instead.
Here's the tweet text:
🆕 Release (next) is out! Thanks to @patrick91 for the PR 👏

Get it here 👉 https://strawberry.rocks/release/(next)


subchart does not display data for scatter plot

If you add a subchart to a scatter plot, the data points will not be visible in the subchart but it will otherwise work just fine.
I tested it by simply changing the type of http://c3js.org/samples/options_subchart.html
var chart = c3.generate({
    data: {
        columns: [
            ['sample', 30, 200, 100, 400, 150, 250]
        ],
        type: 'scatter'
    },
    subchart: {
        show: true
    }
});

Sure would be nice to have this feature.
Are there still plans to have the scatter chart subchart work?


getOptionalProject with option statistics

Could you please add method ProjectApi#getOptionalProject(Object projectIdOrPath, Boolean statistics)
JF

@jflefebvre06
This has been added and released in 4.9.4.  Please verify and close when you have time.
BTW, we are in the process of adding methods that provide a Stream instead of a List or Pager to all get methods that produce multiple items.  Currently, only the ProjectApi and DiscussionsApi have had them added, would be great if you could provide feedback on this new feature.  Here are the ones available in
the ProjectApi:
public Stream<Project> getProjectsStream()
public Stream<Project> getProjectsStream(String search)
public Stream<Project> getMemberProjectsStream()
public Stream<Project> getAllProjectsStream()
public Stream<Project> getOwnedProjectsStream()
public Stream<Project> getStarredProjectsStream()
public Stream<ProjectUser> getProjectUsersStream(Object projectIdOrPath)
public Stream<Project> getUserProjectsStream(Object userIdOrUsername, ProjectFilter filter)
public Stream<ProjectUser> getProjectUsersStream(Object projectIdOrPath, String search)
public Stream<Member> getMembersStream(Object projectIdOrPath)
public Stream<Event> getProjectEventsStream(Object projectIdOrPath)
public Stream<ProjectHook> getHooksStream(Object projectIdOrPath)
public Stream<Issue> getIssuesStream(Object projectIdOrPath)
public Stream<Snippet> getSnippetsStream(Object projectIdOrPath)
public Stream<Project> getForksStream(Object projectIdOrPath)

Thanks  a lot works fine.
The new Stream replacing pager is a very good idea !!!!!
Actually I use pager for :

GroupApi#getSubGroups
TagsApi#getTags
RepositoryApi#getTree

@jflefebvre06
FYI, getXxxxxStream() methods have been added to the following API classes and are part of the 4.9.7 release:
GroupApi
IssuesApi
NotesApi
ProjectApi
RepositoryApi
TagsApi
UserApi

@jflefebvre06
The returned Stream can be used as a parallel stream, allowing the results from the API to be processed in parallel.
Keep in mind though,  that the stream is completely built when it is returned, meaning it is not reading in parallel from the REST endpoint, this cannot be done as the GitLab API itself does not allow that.
I have a unit test that needs to be checked in that tests parallel stream processing and I will add an example to the README.md showing how to do this.

I have tested new méthods, all works fine thank you.


os: TestRemoveAllDot failing on Plan 9

CL 150497 enabled TestRemoveAllDot on "noat" (see removeall_noat.go) systems (including Plan 9).
This test is failing on Plan 9.
--- FAIL: TestRemoveAllDot (0.00s)
    removeall_test.go:235: RemoveAll succeed to remove .
FAIL
FAIL	os	2.055s

See https://build.golang.org/log/62555ab829ee330629c6189fef3507411e657a55

Sorry, OT. Seems that the link /c/150621 does not point to go-review.

I think there is some kind of issue with the golang.org redirect. The direct link works: https://go-review.googlesource.com/c/go/+/150621

redirect issue - https://github.com/golang/go/issues/28836


It's works but with few limitations

Hi!
first of all thank you for the work done. It works correctly!
I have a question... The newsletter that arrives does not have the design and options that Ghost "officially" includes with MailGun. Can't this be solved?
Thanks

Hi there,
Thanks for trying out the project!
I tried to mimic the design by looking at the "Preview" shown on Ghost as I didn't have the source of the complete html.
I agree that it isn't exactly the same but I can definitely try to improve it.
You also mentioned some "options", could you tell what those are?

Hi there, Thanks for trying out the project!
I tried to mimic the design by looking at the "Preview" shown on Ghost as I didn't have the source of the complete html. I agree that it isn't exactly the same but I can definitely try to improve it.
You also mentioned some "options", could you tell what those are?

Thank you for your response. The project is great. It really is.
The design issue only. The Ghost template is beautiful. Too bad they don't share the HTML to mimic it in your service. I'll keep an eye out for updates.

Just an update, I'm mostly done with the native-ish look of the template. Improvements also include:

Align Title (Center or Left)
Manage Subscriptions Section

Both options configurable via Ghosler settings.
Will create a branch, probably tomorrow for public access & testing.

Just an update, I'm mostly done with the native-ish look of the template. Improvements also include:

Align Title (Center or Left)
Manage Subscriptions Section

Both options configurable via Ghosler settings.
Will create a branch, probably tomorrow for public access & testing.

It's awesome! I'll keep an eye out 👏☺

See the branch : https://github.com/ItzNotABug/ghosler/tree/improved-template.
Let me know if you have suggestions or find any issues with the new template.
Cheers!

See the branch : https://github.com/ItzNotABug/ghosler/tree/improved-template. Let me know if you have suggestions or find any issues with the new template.
Cheers!

Great job. Now, the template is shipped with Ghost's design!
I send you a couple of bugs.

The logo comes out "square", not in its original look.
The site title has no CSS styling.

Now possible suggestions (if possible)

Is it possible to change the typography of the article title?
Is it possible to add the Excerpt (subheading) of the article below the title?
Is it possible to add the "Keep reading" and "More like this / Less like this"?

Thank you very much for everything.

Hi again 👋
Bugs.

I don't think I've really changed much of the logo's style. Afaik the logo always shows up in square style by default.
See Logo Section.

The site title indeed does not have any styling by default, You can either select Classic Serif or Serif on Ghost Newsletter design page. I used Serif as default. See Site Title Section.

If you need more designing, feel free to make relevant changes to newsletter.ejs & restart Ghosler with pm2 restart ghosler.
Suggestions.

Is it possible to change the typography of the article title?

I have tried to keep it as close as the actual template but you can always tweak as per your changes as described above.

Is it possible to add the Excerpt (subheading) of the article below the title?

Again, you can. The excerpt is stored in the post.preview variable.
This is actually used for email client's preview text. See how it is generated here.

Is it possible to add the "Keep reading" and "More like this / Less like this"?

 But adding them wouldn't work as expected though.
See you can send a user to get a feedback on a post BUT the feedback will not be displayed on the Analytics dashboard as it is not sent via Ghost x Mailgun. I am not really sure of tracking this user feedback on Ghosler at the moment though.
About the Keep Reading part, I'll have to see about this.
Thanks!

Thanks for all!
Last question: How can I remove the word "Newsletter" from the title of the email so that only the title of the article appears?

Nice catch, I'll probably add something for that as well.
Till then, this is the line.
As before, make changes & do pm2 restart ghosler.

I keep discovering bugs to polish Ghosler :P
If the article is set to "members only", in the email that arrives you can read the entire article. With the official implementation, this does not happen.
And, the HTML code inserted in the article (e.g. bookmarks, divider, button, etc.) is also not displayed correctly.

Can you provide some more info about the Members only article issue?
For the second one, there are a lot of components that Ghost supports, some of them are custom and require additional css styles. I don't intent to add all the css as that would bloat the html file.
Pretty sure that this is the file, huge if you see.
I'll keep checking for usual html tags and add respective CSS in the template over the course of time.
Also if the issue differs from design, make sure to create a new issue.
Thanks for all the feedback.

If the article is set to "members only", in the email that arrives you can read the entire article. With the official implementation, this does not happen.

When the post is set to 'Paid Members Only' or 'Tiers', only the said members should receive the full content.
Others, i.e. non-paying members receive only the 'Public Content' with an option to 'Subscribe' to the blog.
This is now fully supported in the latest improved-template branch!
Closing this issue.
Create a new one for suggestions.
Thanks again for all the feedback!

Hi!
Now the tool works perfectly!
Only one thing:
When you receive the email, in the "Site title" add a final " ' ". I can't find where to delete it.

Thanks!!

@viriatusX I don't see this issue. Can you check if your 'Reply To' field in the added mail config has an extra '?

@viriatusX I don't see this issue. Can you check if your 'From' field in the added mail config has an extra '?

 That was it. Thanks for your effort!

No issues.
Don't forget to ⭐ the project!


Migrating UWP app to WindowsAppSDK.  Can't navigate to page from Windows Runtime Component

Describe the bug
App.xaml.cpp is just activates the MainWindow which is exactly what it does out of the box.  I attempted to put the page navigation code in there as documented but didn't work.  Now MainWindow.xaml had a named <Frame x:Name="rootFrame Loaded="Loaded"/>
In the Loaded Event for the Frame I am calling rootFrame().Navigate(xaml_typenameComponent::BlankPage(), *this);
In either case I get an exception thrown:
Exception thrown at 0x00007FFB4621466C (KernelBase.dll) in CloudView.exe: WinRT originate error - 0x80004005 : 'Cannot locate resource from 'ms-appx:///BlankPage.xaml'.'.
This is a project added as a WinUI3 Runtime Component inside the Solution.  My production code is in a Separate solution and has the same problem.
I tried the same thing using the Windowing Sample cpp-winui from the WindowsAppSDK-Samples.git repo.  It works there.  The difference is that this uses a separate Packaging Project.  The current SDK integrates the packaging into the App.
Steps to reproduce the bug

Create a new Solution/Project using the "Blank App, Packaged (WinUI 3 in Desktop), Default names are fine.  Don't check Solution in same folder

Add a New Project to the Solution using the "Windows Runtime Component (WinUI 3)" template.  Default names are fine.

Add a New Item to the RuntimeComponent1 Project, using the "Blank Page (WinUI 3)" template.  Default names are fine.

Build the RuntimeComponent1.  Debug, x64 is what I used.

Add a Reference in App1 to RuntimeComponent1.  Right click>Add>Reference...

Edit MainWindow.xaml and make it look like this:

Edit MainWindow.xaml.h.
Add the following #includes above #include "MainWindow.g.h"
#include <winrt/RuntimeComponent1.h>
#include <winrt/Windows.UI.Xaml.Interop.h>
Add the following Loaded Method
void MainWindow::Loaded(Windows::Foundation::IInspectable sender, Microsoft::UI::Xaml::RoutedEventArgs const& e)
{
rootFrame().Navigate(xaml_typename < RuntimeComponent1::BlankPage>(), *this);
}

Edit MainWindow.xaml.cpp and remove the reference to myButton() in the myButton_Click Event.

Build and run App1.  You will get the exception.

Expected behavior
No response
Screenshots
No response
NuGet package version
1.0.0-preview3
Packaging type
Packaged (MSIX)
Windows version
Windows 11 (22000)
IDE
Visual Studio 2022
Additional context
No response

@evelynwu-msft is this the known issue with Component Libraries exposing controls (or in this case Pages) not working with single-project MSIX?

This is a duplicate of internal bug https://task.ms/36925415.

@timmarriott Unfortunately this is a known issue for 1.0, but you can work around it by adding the following target to the end (after all imports of .targets files) of your Windows Runtime Component's .vcxproj:
  <Target Name="GetPriIndexName">
    <PropertyGroup>
       <!-- Winmd library targets use the default root namespace of the project for the App package name -->
       <PriIndexName Condition="'$(RootNamespace)' != ''">$(RootNamespace)</PriIndexName>
       <!-- If RootNamespace is empty fall back to TargetName -->
       <PriIndexName Condition="$(PriIndexName) == ''">$(TargetName)</PriIndexName>
     </PropertyGroup>
  </Target>

I found that the work around does enable things to work, but not quite right using a Project with integrated MSIX.   One issue is that WebView2 does not show the webpage.  Also, some bindings don't seem to be working.
If I use the WAP template everything works fine.  The work around is needed so Resources work correctly.


Same height module

Would be cool if we could get a semantic-ui module that would equalize the height of a list of items (similar to this).

This is no longer necessary with flex


Add otpgen to top level all packages

Motivation for this change
Otpgen is still not in all-packages.nix
Things done
Added otpgen to all-packages.nix (tried to add it in lexical order even though it doesn't seem to be respected.

[ ] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Removing First commit


Improve Tree deletion suggestions

Suggested commit message:
Improve `Tree` deletion suggestions (#347)

When suggesting to remove a method or method annotation, also remove any
trailing whitespace. This avoids the possible introduction of an empty
line right at the start of a code block.

See also this comment.

Oops, I don't know what happened but thanks for applying 😉.


Logging

Please can you add a switch into the settings that will allow us to turn off logging? The polling and update messages every 30 seconds can clog the logs very quickly!
absolutely love the plugin though! Thanks!

Hi @Panic135 I've changed the log level to debug for these messages. I'll release the new version in a couple of minutes. I'm closing this issue for now. Will reopen if it won't help.


proposal: net/url: add URL.RawFragment, like RawPath

OpenAPI and JSONSchema do fancy things with the fragment including
funny path operations that distinguish between a literal / and an escaped %2f.
Just like happened with Path, the solution is probably to add RawFragment,
with the same semantics for Fragment/RawFragment as Path/RawPath.
/cc @mpvl

Adding to the proposal minutes. This seems fairly unobjectionable given the precedent set by RawPath. If anyone thinks we shouldn't do this, please speak up. Thanks.

Based on the precedent from RawPath and the lack of any objections, this seems like a likely accept.

No change in consensus (still crickets here), so accepting.


nconf raise fs can't be found if used in browser/ code

If we try to make use of common/config in browser/ code, we would get the following exception:
/Users/levi/Documents/Projects/mobile/webpack/build.js:36
      throw new _gulpUtil2.default.PluginError('webpack', buildError);
      ^
Error: ./~/nconf/lib/nconf.js
Module not found: Error: Cannot resolve module 'fs' in /Users/levi/Documents/Projects/mobile/node_modules/nconf/lib
resolve module fs in /Users/levi/Documents/Projects/mobile/node_modules/nconf/lib
  looking for modules in /Users/levi/Documents/Projects/mobile
    resolve 'file' fs in /Users/levi/Documents/Projects/mobile
      resolve file
        /Users/levi/Documents/Projects/mobile/fs.js doesn't exist
        /Users/levi/Documents/Projects/mobile/fs doesn't exist
    /Users/levi/Documents/Projects/mobile/fs doesn't exist (module as directory)
  looking for modules in /Users/levi/Documents/Projects/mobile/src
    /Users/levi/Documents/Projects/mobile/src/fs doesn't exist (module as directory)
    resolve 'file' fs in /Users/levi/Documents/Projects/mobile/src
      resolve file
        /Users/levi/Documents/Projects/mobile/src/fs doesn't exist
        /Users/levi/Documents/Projects/mobile/src/fs.js doesn't exist
  looking for modules in /Users/levi/Documents/Projects/mobile/node_modules
    /Users/levi/Documents/Projects/mobile/node_modules/fs doesn't exist (module as directory)
    resolve 'file' fs in /Users/levi/Documents/Projects/mobile/node_modules
      resolve file
        /Users/levi/Documents/Projects/mobile/node_modules/fs doesn't exist
        /Users/levi/Documents/Projects/mobile/node_modules/fs.js doesn't exist
  looking for modules in /Users/levi/Documents/Projects/mobile/node_modules/nconf/node_modules
    /Users/levi/Documents/Projects/mobile/node_modules/nconf/node_modules/fs doesn't exist (module as directory)
    resolve 'file' fs in /Users/levi/Documents/Projects/mobile/node_modules/nconf/node_modules
      resolve file
        /Users/levi/Documents/Projects/mobile/node_modules/nconf/node_modules/fs doesn't exist
        /Users/levi/Documents/Projects/mobile/node_modules/nconf/node_modules/fs.js doesn't exist

I guess it is due to unavailable fs module in browser context, which is used by nconf (https://github.com/este/este/commit/7f71db5f0a542e8bb577c40f2d997b1b3c8e0e1e)
This may be the workaround https://github.com/webpack/jade-loader/issues/8

And if this common.config should only be used by server or native component (which have access to fs), I think this file shouldn't be put under common folder

Sure. But I will not make commonForServerAndNativeOnly directory, so it's good enough.

Do you mind to further explain the solution? Here are further explanation of my problems:
In common/config.js we have:
nconf.defaults({
  firebaseUrl: 'https://este.firebaseio.com',
  googleAnalyticsId: 'UA-XXXXXXX-X',
  isProduction,
  port: process.env.PORT || 8000,
});

export default nconf.get();

That config is currently used by src/native/index and src/server/index to check whether current env is production (config.isProduction). Since that config is put under common folder, I assume it should be available for browser code as well, so I add another common configuration there and use it in my src/browser/... code. But it doesn't work due to fs error mentioned above.
I am thinking, whether we should use another simpler configuration object in favour of nconf (which internally depends on fs)?

And the culprit is here: https://github.com/indexzero/nconf/blob/master/lib/nconf.js#L8. Seems it has been fixed in https://github.com/indexzero/nconf/issues/130 by removing require('fs'), but the latest commit reintroduce fs dependency again.

I think @steida is trying to say, because it works for server and native, it is common enough to be in common/.
fs module is a node.js built-in. It doesn't exist in the browser. And nconf is always going to rely on fs, since that's how it reads from config files.

Oh, I get it :). Thx for the explanation.
Then, I would put my browser config somewhere else.
On Mar 9, 2016 8:00 PM, "Stephen Friend"<EMAIL_ADDRESS>wrote:

I think @steida https://github.com/steida is trying to say, because it
works for server and native, it is common enough to be in common/.
fs module is a node.js built-in. It doesn't exist in the browser. And
nconf is always going to rely on fs, since that's how it reads from
config files.
—
Reply to this email directly or view it on GitHub
https://github.com/este/este/issues/764#issuecomment-194285699.

Perhaps put the configuration variables you need for the client in the initialState from the server/frontend/render.js

Thanks for the suggestion.
In my case, those configs are constant, so it would be overkill storing them in state.
In the end, I created new file: xxxConfig.js.

@fikriauliya Put config values to config, it's read via initialState.config in client side.


Update azure-security-keyvault-jca version in azure-sdk-bom.

As title.

Closing this PR, because azure-sdk-bom is generated by tool.


WIP master <- develop

Merge the develop branch into the master branch and deploy

Build log
Preview build with this PR (commit): https://output.circle-artifacts.com/output/job/56fb8bec-136f-48ae-8987-4a4b7f2bfcde/artifacts/0/index.html

Build log
Preview build with this PR (commit): https://output.circle-artifacts.com/output/job/f20caaeb-0be6-4d44-b827-c41f4760f2c4/artifacts/0/index.html

Build log
Preview build with this PR (commit): https://output.circle-artifacts.com/output/job/a92e0ebc-f946-41d6-8b21-38fc9baf260a/artifacts/0/index.html


Bug/#50

Coverage decreased (-7.9%) to 63.441% when pulling 89416c532d3f172f21d4785a3a0c5f73d15075c5 on bug/#50 into 42789a13a8a5e66be92b61e56b6a2eb365559f50 on master.

solved this issue:  https://github.com/Kensuke-Mitsuzawa/JapaneseTokenizers/issues/50


fix: component would fail to load when using an instantsearch adapter

Using an adapter such as Typesense, which doesn't have the addAlgoliaAgent function, could cause this component to not load.
Since this is just for the search user agent, it shouldn't be mission-critical if it fails to change the user agent string:
https://www.algolia.com/doc/api-client/getting-started/customize/javascript/?client=javascript#custom-user-agents

Thanks, I was ready to fork the project over this

Hey @aymeric-giraudet just following up if you were able to re-review the changes I accepted. Thanks!

Hey @djdembeck, looks good ! Thanks !
I'll put this in the 0.11.0 release :)


tests faild of 404 error ( magic components folder )

Hi I'm trying to made test from site:
https://medium.com/google-developer-experts/polymer-unit-testing-d6a69910dc31#.xy91nfol5
I installed libraries
"flip-clock": "^1.2.6",
"web-component-tester": "Polymer/web-component-tester#^4.3.1"
and have installed globally web-component-tester
libraries are in bower_components folder

I have test:

but when I run wct it always failing because of 404 error

Why it looking for files in folder /components ? I don't declarate any... all my
There is no information about it in readme, How I can fix it?

I just ran into this same issue and found a way around it. The hint was this line from the README:

When you use wct on the command line, it is generating an index like this for you based on the suites you ask it to load.

I believe that the automatically generated index.html it is using for the tests looks at the root of the serving directory for a components/ folder, and that it then handles that folder in a way that ignores the actual path of your bower_components folder, for example.
I was starting from the seed-element example. Running wct from that repo works just fine. Note these two lines in the /test/basic-test.html file in that project:
    <script src="../../webcomponentsjs/webcomponents-lite.js"></script>
    <script src="../../web-component-tester/browser.js"></script>

Strangely, the ../../ path there is pointing that URL outside of the repo, and yet somehow it works! I was able to resolve my issues by adding ../ to my source file until it was pointing at a level above the directory from which I was calling wct (at least I think it's a level above--not counting closely atm).  I think you might actually have to be pointing at the location relative to the file that is the parent directory of your bower_components/ directory.
You can try this by just adding a level of ../ to the <script src= path and rerunning it. For the directory structure of my specific project, I wound up with the following:
    <script src="../../../../webcomponentsjs/webcomponents-lite.js"></script>
    <script src="../../../../web-component-tester/browser.js"></script>


[Relay]:  Expose resolveOffsetConnection into  two functions

Another request!
Is there any way to break the resolveOffsetConnection function into two parts, or change the return type on the resolveOffsetConnection callback?
The reason is that I get the totalCount and pageCursors from one aggregated database call. But I can't make that database call without the offset and limit ... which means I have to manually replicate the offsetForArgs function or roll my own. But then I have two offset/limit calculations which means that if they ever get out of sync, things will get dicey. Plus it just seems inefficient.
I tried returning the totalCount, etc. from the resolveOffsetConnection function but it's expecting an array of nodes, not extra connection info.

The helpers definitely need some API improvement.  I need to think carefully about what to add, how to expose it, and keep backwards compatibility.  I have not actually spent a lot of time working on relay APIs myself, and need to experiment more before I can commit to how to build out additional helpers.
In the mean time, You could add something like this to your app:
(Note that I have not tested this at all, and it might be slightly wrong, but the general idea should work)
export async function resolveOffsetConnectionWithExtra<T, U>(
  options: {
    args: DefaultConnectionArguments;
    defaultSize?: number;
    maxSize?: number;
  },
  resolveResult: (params: {
    offset: number;
    limit: number;
  }) => Promise<{ nodes: T[]; connection: U }> | { nodes: T[]; connection: U },
): Promise<(ConnectionShape<SchemaTypes, T, false> & U) | null | undefined> {
  let resolveConnection: (con: U) => void;

  const connectionPromise = new Promise<U>((resolve) => {
    resolveConnection = resolve;
  });

  const result = await resolveOffsetConnection(options, async (params) => {
    const { nodes, connection } = await resolveResult(params);

    resolveConnection(connection);

    return nodes;
  });

  if (!result) {
    return result;
  }

  return {
    ...result,
    ...(await connectionPromise),
  };
}

this would be used something like:
resolve: async (parent, args) =>
        resolveOffsetConnectionWithExtra({ args }, ({ limit, offset }) => {
          const nodes = getThings(offset, limit);

          return { nodes, connection: { totalCount: 500 } };
        }),
    },

The above is a little verbose but eliminates the duplication of my limit / offset problem.
For someone that does have that much relay experience you've created some super useful tools!
Feel free to close this if you want.


[Improvement] Inconsistency of Buttons

The two buttons "Shorten" and "Copy" are not consistent. One is Bigger in width and the other is smaller which brings inconsistency to the whole web page. We can resolve this issue by increasing the width of the copy button
Please assign this issue to me under Gssoc'23 @apoorvaron

Okay ! Make necessary changes

@Anshika-Jain02 Still working on this small issue ?

@apoorvaron Sir, actually my exams are going on  from past one week due to which there is a delay from side. I will complete it very soon.

@Anshika-Jain02 Okay

Inconsistency of Buttons #33
May I try to fix this? This is a simple one.
Please assign me this issue so that I can resolve it under GSSOC-23
@apoorvaron

@Anshika-Jain02 Done it by today , otherwise i will assign this issue to @unstoppableayush tomorrow!

@apoorvaron Sir, can you please give me one more day as i am facing this issue and i am having my last end semester exam tomorrow. So, i will try to fix it tomorrow positively.

@Anshika-Jain02 Okay

@apoorvaron Sir, do you know why this error is occuring after successfully setting up the project.

@Anshika-Jain02 is code up to date ?

Yes sir. @apoorvaron

Even i compared both the files manually also everything is same in index.js but then it is showing that this variable is not declared @apoorvaron

@Anshika-Jain02 will let you know in evening

Ok sir @apoorvaron

@Anshika-Jain02 Now take pull , i have fixed the issue

@Anshika-Jain02 Please share screen record including that code is up to date or not.

@apoorvaron
https://github.com/apoorvaron/Shorty/assets/104912087/7898bd83-d87d-4bdb-bcd1-59ef3bfbfb37

@Anshika-Jain02  Can you check , is this line is present in tiny/index.php file at line number 277

@apoorvaron No sir, it was not present earlier and even when i add it ..it is still not working

@Anshika-Jain02 you can delete your forked repo and start from beginning. Because i just checked , it works fine for me.

@apoorvaron Sir, I have forked the repo again and followed all the steps for the setup of the project on multiple systems and  the previous issue is now solved but the button is not working as expected in any of the systems. What should I do next?
https://github.com/apoorvaron/Shorty/assets/104912087/507702b1-9fc5-4010-8fe5-9e434a19ece4

@Anshika-Jain02 Please look into this
https://github.com/apoorvaron/Shorty/assets/83576954/03118f71-3650-4441-ba87-83955e75e301

@Anshika-Jain02 double check your these files.

database.php
2)env.php
3)siteName.php

And successfully import the dB file in localhost/phpmyadmin

@Anshika-Jain02 did you start the XAMPP ?
if not , then by going to application xampp , start the APACHE and MySQL server
if MySQL server should error that 3306 is already in use then Seach "services" inside "Type to Search" panel in windows then a window will pop out , search manually MySQL and stop running something by clicking the mouse right btn.

@Anshika-Jain02 Please find an another issue


Test speed

Hello ,could you help me how to calculate the segmentation speed.I don't know how to calculate the speed in pytorch. Thank you!

Using the time library and torch.cuda.synchronize():
import time

torch.cuda.synchronize()
t_start = time.time()

# Image segmentation
predictions = model(images)

torch.cuda.synchronize()
elapsed_time = time.time() - t_start

You can adapt the predict function to do this


Fallback option if SQLDescribeParam is not available?

Would it be possible for nanodbc to provide a fallback, possibly by passing parameter sizes manually if a driver does not implement SQLDescribeParam?
freeTDS does not support SQLDescribeParam, which means binding parameters with nanodbc fails when using that driver. https://github.com/FreeTDS/freetds/issues/104

Using SQLDescribeParam also makes stored procedure call two-three times longer for MS SQL driver. It seems it causes an additional round-trip to server.

Yes, the client cannot know the server type and has to ask to the server adding a round-trip.
Which conversion needs to do nanodbc that needs to know the server type?
Or it's just a problem of return parameters?

The way I imagine it working is we could have one interface that uses SQLDescribeParam() behind the scenes for getting the parameter sizes for auto binding. And another mirror interface that takes the parameter sizes as arguments and just uses them, eliminating the need for SQLDescribeParam(). It could probably be written such that the former interface itself calls the later interface. This would require surfacing some private details of the current implementation and refactoring them to be a friendly, consistent, and concise user interface. This new interface would also allow users the opportunity to optimize, for example in MS SQL as @AndrewJD79 mentioned.
This all sounds like a good idea to me! Sadly I don't get much time these days to work on this codebase and my role has mostly been to review any PRs that come in and offer whatever advice I may have.

I can offer either opinion nor solution here, but I'll be happy to review any PRs and help with testing any improvements in the pipeline.
@lexicalunit 's idea sounds good to me.

This issue was moved to nanodbc/nanodbc#15


Rename "Sequence-Variability" to "Sequence Haplotypes"

Add the QuickGuide from VISS-Draft text for Streamlit App to the top of the page.
QuickGuide

Drag and drop the aligned file or Click on the [Browse files] button to choose and upload the aligned sequence file.
Check that the [Uploaded sequences:] display reflects the file chosen.
Copy and paste a sequence into the [Enter base sequence] box. A copy to clipboard option is available for selecting a sequence included in the upload file. A reference sequence could be used provided it matches the section being analyzed.
Click on [Calculate sequence variability]
Type the species or target name that will be added as a header to each sequence variability group into the [Enter the species name:] box.
Click on [Prepare for download].
Click on [Download FASTA file].

Text edits:

Change “Sequence Variability Data” table header to “Sequence Haplotype Groups”
Add a (?) bubble next to [Enter base sequence] that says “Sequence base pairs only, no sequence headers”.


Update free-programming-books-id.md

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

this is an advertising site - very hard to find the book, then it makes you fill out a form. no thank you.


multiple-cucumber-html-reporter not generating json files

Current behavior
I was using multiple-cucumber-html-reporter earlier with the following dependency versions and it was working fine based on the configuration mentioned in this article: https://blog.knoldus.com/generating-cucumber-html-report-cypress/
"devDependencies": {
"cypress": "^8.4.0",
"cypress-cucumber-preprocessor": "^4.1.4",
"multiple-cucumber-html-reporter": "^1.18.0",
"standard": "^16.0.4"
},
Desired behavior
Current configuration:
"devDependencies": {
"@badeball/cypress-cucumber-preprocessor": "^12.0.0",
"@bahmutov/cypress-esbuild-preprocessor": "^2.1.3",
"cypress": "^10.4.0",
"esbuild": "^0.14.53",
"multiple-cucumber-html-reporter": "^1.21.6",
"standard": "^16.0.4"
}
According to the configuration below in packag.json, it should create a json file in output folder cypress/cucumber but it is not creating any json files
"cypress-cucumber-preprocessor": {
"nonGlobalStepDefinitions": true,
"nonGlobalStepBaseDir": "cypress/e2e//*.{js,ts}",
"stepDefinitions": "cypress/e2e//*.{js,ts}",
"cucumberJson": {
"generate": true,
"outputFolder": "cypress/cucumber",
"filePrefix": "cucumber-",
"fileSuffix": ""
}
}
Versions

Cypress version: 10.4.0
Preprocessor version: 12.0.0
Node version: 14.15.3

Checklist

[x] I've read the FAQ.
[x] I've read Instructions for logging issues.
[x] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

Article is outdated, here are the correct docs describing JSON reporting: https://github.com/badeball/cypress-cucumber-preprocessor/blob/master/docs/json-report.md.

Okay I used that configuration and its creating a cucumber-report.json. How do I convert it to HTML report?
I am trying to use multiple-cucumber-html-reporter. It says once you have the json file, you can run the following code to generate html report but the html report which gets generated is blank:
const report = require('multiple-cucumber-html-reporter')
report.generate({
jsonDir: 'cypress/cucumber/',
reportPath: 'cypress/reports',
metadata: {
browser: {
name: 'chrome',
version: '90'
},
device: 'Local test machine',
platform: {
name: 'ubuntu',
version: '16.04'
}
},
customData: {
title: 'Run info',
data: [
{ label: 'Project', value: '' }
]
},
displayReportTime: true,
hideMetadata: true,
saveCollectedJSON: true,
pageTitle: '',
reportName: '',
displayDuration: true
})

This isn't the right place to ask for help with multiple-cucumber-html-reporter.


[Enhance] Change Groupfree3D config

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Change repeat in dataset config in Groupfree3D to reduce the number of epochs.
Modification
Please briefly describe what modification is made in this PR.
Also updated the performance in README.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: THU17cyz:x: yezhenyezhen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Make IndexLoadingConfig directly read configs from TableConfig and Schema

There are still some tests need to be modified to apply the change
Use this PR to show the idea of how the IndexLoadingConfig should access the configs

Done with #14258


updated-data-integrator-error-management

Added another clause under err management

@sabinn-msft
Taking to live via private repo:
https://github.com/MicrosoftDocs/power-platform-pr


Extensions website is not mobile responsive

Extensions website is not mobile responsive. Viewing is as-if on legacy "desktop site"

@ericis Do you mean marketplace.visualstudio.com?

The marketplace is tracking this as an accessibility issue. This needs some follow-up.

Correct @RMacfarlane . It looks like you all have been tracking the issue along. Looking forward to it.


Android dependency is imported dynamically, causing build issues.

Checklist

[X] The issue can be reproduced in the auth0_flutter sample app (or N/A).
[X] I have looked into the Readme, Examples, and FAQ and have not found a suitable solution or answer.
[X] I have looked into the API documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
Last week, every branch new and old of our Android app no longer built due to a kotlin version build error After hours of pouring over merge requests, we realized the Auth0 flutter sdk was dynamically importing the Auth0 Android sdk. There is even a lint warning that is ignored in your sdk. If we would have had a hot fix needed, this would have halted our ability to release. Please lock the Android version for newer and older versions of the Auth0 sdk.
    //noinspection GradleDynamicVersion implementation 'com.auth0.android:auth0:2.+'
Auth0 Android SDK that broke us: https://github.com/auth0/Auth0.Android/releases/tag/2.10.2
https://github.com/auth0/auth0-flutter/blob/main/auth0_flutter/android/build.gradle#L70C1-L71C49
Reproduction

Add auth0 sdk to app
Wait for new Android sdk. version to be released
See any combination of build errors

Additional context
No response
auth0_flutter version
1.1.0
Flutter version
3.7.9
Platform
Android
Platform version(s)
No response

We have fixed this by locking our version here @parkgrrr - https://github.com/auth0/auth0-flutter/pull/311
You can follow this commit to fix the issue on your side- https://github.com/auth0/auth0-flutter/pull/311/commits/193c7260a5084f73e4d2aa11a60afd3f19663c4b
Sorry for the hassle.

Thank you very much!

Keeping this opened until it's merged. Thanks again.

The version is now pinned in v1.3.0.


Factoring create() out of leaf impl classes and into the CreatableImpl base class

to follow the DRY principle, reduce code complexity, enforce consistency and minimize opportunity for error

Can one of the admins verify this patch?

so basic idea here is that from now on, our creatable model impls should not implement their own create(). No need, because it is implemented by CreatableImpl now. It'd only in the exceptional case that the default create() could be overwritten.


Observationspace/Infostate

Hey Eric,
thanks for making this public, haven't found a good env so far that implements Multiplayer NL. Am I understanding the code right that the observationspace isn't actually perfect information? E.g. it is only the last couple actions? Do you have any research on how this affects convergence?
I had a bit of trouble understanding the code, so I apologize if I just didn't read it right.

 I've written a wrapper that tracks the action history for limit games. For no-limit games, this piece is slightly less obvious, so I would default to the recurrent option that tracks the history of public observations, which is also sound. So, perfect recall is still supported for NL through recurrent NNs but you did correctly spot that action history is only tracked through that and not explicitly.
If you wish to track it explicitly, you could just track it manually in the training code or write an appropriate "Wrapper" for the NL environment. However, this will require you to adjust the NN observation space accordingly. TL;DR: Recurrent is cleaner and more scalable.
Cheers,
Eric


DSD-662: use theme object values in Grid docs

Fixes JIRA ticket DSD-662
This PR does the following:

updates docs for Chakra Grid to use theme object values rather than CSS vars

How has this been tested?

local build of DS

Accessibility concerns or updates

n/a

Checklist:

[x] I have updated the Storybook documentation accordingly.
[x] I have added relevant accessibility documentation for this pull request.
[x] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr823-oqhp22un3iib5rgu3ormhx5uhaqbgmuk.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61ca11ac7499db65223050e9

Tugboat has finished building the preview for this pull request!
Link:

https://pr823-oqhp22un3iib5rgu3ormhx5uhaqbgmuk.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61ca11ac7499db65223050e9


purchase_contract does not add moderator fee

When the moderator guid is sent to purchase_contract, the price that is returned is the item price times the quantity, without the moderator fee added.
Ex: if a quantity of 8 of an item that costs 0.001 BTC is purchased, with a moderator fee of 10%, the price returned is 0.008 BTC instead of 0.0088 BTC.

I thought the fee was taken out of the sale price?

I thought the fee was taken out of the sale price?

It is.

It is.

Ah, my bad. I misunderstood that part.
I'll update the client to match that.

Ah, my bad. I misunderstood that part.
I'll update the client to match that.


🚧 Creates single threadpool to execute schedulers

Description
Currently, multi scheduler can be executed at the same time. This means that we could be running the probe engine and updating the database at the same time.

Submitter checklist

[ ] If the issue exists, it is well described and linked in the description
[x] The description of this pull request is detailed and explain why this pull request is needed
[x] The changeset is on a specific branch

feature/ for new feature, or improvements
fix/ for bug fixes
docs/ for any documentation changes

[ ] If required, the documentation has been updated
[ ] There is automated tests to cover the code change / addition

If there is no test, include an explanation why in the description

[ ] Run mvn verify locally and all tests are passing successfully

It is OK to create a pull request which has failing tests if it is created as a draft, is to fix a bug and the first commit is the test to prove the existence of the bug.

[x] There is no new warnings (checkstyle nor spotbugs) on the code

This has been open for far too long and is to much out of sync with the current content of the project


Incorrect result stack size with methods with no return after a call_contract

Summary or solution description
The Opcode.SYSCALL returns a value even if the return of the method is None.
Currently, Neo3-boa will only DROP the unused value if the return type is not None.
So, now, whenever a SYSCALL is being used it will always be followed with a DROP if it's unused
How to Reproduce
https://github.com/CityOfZion/neo3-boa/blob/e27d57603e1ab75108b1e00cc259dcf841ecf521/boa3_test/test_sc/function_test/NoReturnFunction.py#L1-L6
https://github.com/CityOfZion/neo3-boa/blob/e27d57603e1ab75108b1e00cc259dcf841ecf521/boa3_test/test_sc/function_test/CallExternalContractWithReturnNone.py#L1-L14
Tests
https://github.com/CityOfZion/neo3-boa/blob/e27d57603e1ab75108b1e00cc259dcf841ecf521/boa3_test/tests/compiler_tests/test_function.py#L1647-L1659
Platform:

OS: Windows 11 x64
Python version: Python 3.9

Task linked: CU-86787mrrk Incorrect result stack size with methods with no return after a call_contract


RWA-1035 - Use SQL to insert to DB and update delivery count if needed

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/RWA-1035
Change description
Current implementation uses defaultSpring JPA repository.save(entity) but LLD says we should use native Query SQL
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

we don't seem to be covering these error scenarios? Is this part of a separate ticket? https://tools.hmcts.net/confluence/pages/viewpage.action?pageId=1531414005#WA:HandlingCaseEventsInOrderProcessingandDLQs-ErrorScenarios

Yes - the error scenarios are to be implemented in https://tools.hmcts.net/jira/browse/RWA-975


feat: add useLocation

#463  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hi-ogawa and the rest of your teammates on  Graphite


Enabling persistent user instance systemd

Apparently user specific systemd instances only exist while there is a session for a specific user. If the last session exits, then the user specific systemd instance exits as well.
There's a way to make a user specific systemd instance persistent, so that users can run unattended long running services: https://wiki.archlinux.org/index.php/Systemd/User#Automatic_start-up_of_systemd_user_instances
Basically you need to activate this option: http://www.freedesktop.org/software/systemd/man/loginctl.html#enable-linger USER...
loginctl enable-linger username

However this option is not available in NixOS or NixPkgs, I searched the repo. Also the logind extraConfig doesn't cover it.
I think there might need to be a new option as part of the user management that makes the user instance systemd persistent, and run the above command on bootup and the opposite if the setting changes?
It should of course only run once if it has already ran, even though it is idempotent. I wonder how you can do this?
Right now I'm simply running it during when I pack the image.

There seems to be no option in logind.conf to enable this.
Basically, lingering is enabled in systemd at boot if it finds a file /var/lib/systemd/linger/.
We could use a system.activationScript.loginctl-linger script to call loginctl enable-linger <username> for each user in the list for which logind show-user <username> does not return "Linger=yes".
This would work just like user management in https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/config/users-groups.nix#L504-510, except that that script handles much more cases, and optionally handles the deletion of unspecified users.
To go further, you could too remove lingering for users not in the list, but this requires more work and should depend on some option equivalent to users.mutableUsers or you may surprise some users.

This bit me today, as I forgot to MANUALLY run sudo loginctl enable-linger m on a new machine running declarative NixOS… :confused: And my multiplexer was killed.
@layus, nb., one should not link to master, as master is mutable.

If anyone’s interested, right now, you can use something along the lines of https://github.com/michalrus/dotfiles/commit/455c18a81c1fcaaccb99625c2f88402ebd08ac23.

Actually, you can effectively 'enable lingering globally' with KillUserProcesses=no in logind.conf.
That said, it would probably be better to just compile systemd --without-kill-user-processes to disable this behaviour by default, as this is already being done in e.g. Arch and Debian (I think?). Killing daemonized user processes on logout breaks user expectations pretty hugely, and is really only beneficial on deliberately very locked down systems (e.g. lan cafe PCs, kiosks), where it can be manually enabled if required. On normal systems, if there are non-daemon user programs that aren't terminating on receiving SIGHUP, that's a bug in said programs.
On a more general note, I'm not sure why systemd needs a new way for a program to express the desire to run beyond the lifespan of the current session, in the form of systemd-run. We already have daemon().

FYI, NixOS has KillUserProcesses=no in logind.conf since 93c160fabc0e3b5fb3d997c840ef86a68f4bce00.

@CMCDragonkai As bjornfor stated, this issue is now solved in master. Would you mind closing this issue ?

I see the solution provided, but that now always forces lingering. The whole point of this is so that its optional and can be specific to the user session instead of a global setting. Anyway if that global setting, how would people turn off it now?

As I understand it, systemd requires services tout declare somehow their intent to linger after user sessions. The current fix is not a solution. It is required to give some time to upstream packages (like gnu screen where the discussion is tough) to adapt to systemd. Afterwards, we will probably return to systemd default.
For override, see comment in https://github.com/NixOS/nixpkgs/commit/93c160fabc0e3b5fb3d997c840ef86a68f4bce00#commitcomment-18503557
That being said, my personal opinion is that user processes should terminate nicely when the user logs out. It feels like systemd trying to fix the task of the login manager here.

@globin Can you reopen?

If anyone’s interested, right now, you can use something along the lines of https://github.com/michalrus/dotfiles/commit/455c18a81c1fcaaccb99625c2f88402ebd08ac23.

A bit more polished version (now you can do users.users.somebody.linger = true and disabling/enabling will work): https://github.com/michalrus/dotfiles/commit/ebd5fa9583f82589f23531647aa677feb3f8d344#diff-4d353005ef5b3e37f33c07332b8523edR1

Is it possible to make the setting immutable after it is set using your method @michalrus

@CMCDragonkai, what do you mean by immutable? In your configuration.nix, after adding my loginctl-linger.nix to imports = [], you can do:
{
  imports = [
    (pkgs.fetchurl { src = "https://raw.githubusercontent.com/michalrus/dotfiles/ebd5fa9583f82589f23531647aa677feb3f8d344/.nixos-config.symlink/modules/loginctl-linger.nix"; sha256 = "…"; })
  ];
  users.extraUsers.cmcdragonkai = {
    description = "Roger Qiu";
    isNormalUser = true;
    linger = true;
  };
}

And this is quite immutable. :smiley:
Or do you mean immutable wrt. manually running sudo loginctl enable-linger cmcdragonkai? =P That would require patching loginctl. BUT. If you mess up manually, and rerun sudo nixos-rebuild switch, all will be fixed.
N.b., I’ve opened a feature quest in systemd for the functionality of listing users with lingering enabled → https://github.com/systemd/systemd/issues/3907.

Yep, I was interested in whether the setting can be made immutable in the live environment thus preventing configuration drift. (I usually make /etc readonly). But this is a better solution than the previous one. Is it possible to simplify to just an activation script? A PR to nixpkgs proper would be great.

@CMCDragonkai, sure, I was thinking where that code should go… and dunno.
As for making lingering immutable, I don’t know. The simple and good way would be to patch systemd to check whether the user is allowed to linger not in /var/lib/systemd/linger/, but somewhere in the /nix/store/. Then sudo loginctl enable-linger someone would fail on the syscall level, as the /nix/store/ location would be immutable.
@joachifm, @globin, do you think my current solution is enough for nixpkgs? It’s in https://github.com/michalrus/dotfiles/commit/ebd5fa9583f82589f23531647aa677feb3f8d344#diff-4d353005ef5b3e37f33c07332b8523edR1 I’d rather have patched systemd, but all mentions of /var/lib/systemd/linger should be replaced in systemd’s runtime with some configurable directory in /nix/store. Makes it more complicated…
WDYT?

If we do your loginctl PR, then I think this commit should be reversed in the same PR, as otherwise users are lingering by default anyway.

https://github.com/NixOS/nixpkgs/commit/93c160fabc0e3b5fb3d997c840ef86a68f4bce00#commitcomment-18503557
Forgot to add the commit.

Ping @globin this issue is not solved. The best solution that is available at the moment is @michalrus script: https://github.com/michalrus/dotfiles/commit/ebd5fa9583f82589f23531647aa677feb3f8d344#diff-4d353005ef5b3e37f33c07332b8523edR1
The better future solution is to for systemd to make this declaratively configurable.

(triage) what’s the status?

This is still a problem.
We could fix it in a variety of ways, for example by creating files in /var/lib/systemd/linger based on a user setting, but that would have to be a tristate; by default users can create/remove those using loginctl (enable|disable)-linger.

It shouldn’t be up to the user whether to enable lingering. We can remove a lot of complexity if we only allow this from declarative user configs (aka just generate the linger directory and symlink it based on the config).

If you only allow declarative, a simple implementation like I used in our config should be fine:
    # This enables “lingering” for the CI user.
    # Inspired by the discussion (and linked code)
    # in https://github.com/NixOS/nixpkgs/issues/3702
    # This should just be a NixOS option really.
    system.activationScripts = {
      enableLingering = ''
        # remove all existing lingering users
        rm -r /var/lib/systemd/linger
        mkdir /var/lib/systemd/linger
        # enable for the subset of declared users
        touch /var/lib/systemd/linger/${constants.ci-user-name}
      '';
    };

It could be a global policy, but then specific to each to each user?

Still an issue

Still an issue

@flokli @Kloenk @andir @edolstra : sorry about pinging you, but would it be possible to add a setting for this? Much appreciated!

Ping because it's still relevant.

Maybe something similar to ↓ would feel cleaner (no lingering processes, everything still under systemd’s control) / acceptable to some:
{
  systemd.services.my-gnu-screen = let
    me = "NorfairKing";
    constantSessionName = "7260c3b2-2e3d-4b22-8f8b-ab87de790446";
  in {
    serviceConfig = {
      User = me;
      Type = "forking";
      Restart = "always";
    };
    wantedBy = [ "multi-user.target" ];
    script = ''
      ${lib.optionalString (me == "root") "export SHELL=${config.users.defaultUserShell}"}
      source ${config.system.build.setEnvironment}
      cd "$HOME"
      exec "$SHELL" --login -c "exec ${pkgs.screen}/bin/screen -S ${constantSessionName} -d -m"
    '';
  };
}

It shouldn’t be up to the user whether to enable lingering, it’s system/root policy. We can remove a lot of complexity if we only allow this from declarative user configs (aka just generate the linger directory and symlink it based on the config).

I think in logind's design it is the options KillUserProcesses, KillOnlyUsers and KillExcludeUsers which are system / root policy. A regular user is able to enable lingering for him/herself using loginctl enable-linger

Is there anything major that actually speaks against adding this option to users.users.<name>?

@graham33 I am trying to clean up my nixpkgs and this is one of the commits I've been carrying for some time. I think we should get something in and iterate if need be. I suspect a number of people are carrying some variant of this anyway.
If you send a PR, I'll review it!

That would be great. ❤

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/adding-nixos-option-for-systemd-user-lingering/28762/8

I'm waiting for an "official solution" (something like users.users.<name>.linger = <bool>;). But in the meantime, I am using the following config:
+  # temporary hack until official lingering support is added to `users.users`
+  systemd.tmpfiles.rules = [
+    "f /var/lib/systemd/linger/pratham"
+  ];
+

Keeping it here for future NixOS newbies like me, but hopefully this has been resolved by then. Do check the users.users module on the stable NixOS before you use this!

It looks like @michalrus subsequently stopped using his module and deleted it. I updated it slightly to work with recent NixOS: https://gist.github.com/graham33/fdbdcc18317a621d9dd54beb36be6683
I agree with @busti that this seems like it's good enough to just add as a regular user option. Happy to make a PR if that helps?

I tried tidying this up and making it PR-able, but when testing it locally, the loginctl enable-linger commands fail with read-only file system. Looking at strace on logind, it's getting EROFS when it tries to modify the contents of /var/lib/systemd/linger.
Looking at the unit file for systemd-logind.service, it doesn't have that directory listed under ReadWritePaths, but it does have it listed as the StateDirectory, so I would expect it to be mounted writeable regardless.
I can rework the PR to instead just touch or unlink the linger files, but (a) that's less clean and (b) I'm not sure if logind does more than just manipulating those files (e.g. maybe it pokes systemd to tell it to start/stop the user slice as well or someting).
How did you work around this?

I cannot believe this is now fixed :party: !
Thanks to everybody involved over the years, this is amazing !

Hooray.

It is amazing that this option exists, thank you!
I noticed, however, that it is not persistent when using a temporal root file system. Other options related to users are persistent. Is this behavior desired? Can it even be changed?
I can restore /var/lib/systemd/linger on boot, and all is fine, but just wanted to through in this thought. Cheers!

Given that this was achieved via an activation script, I don't understand why this would matter?

Yeah, it shouldn't matter if /var/lib/systemd/linger persists -- it'll get recreated on activate, and systemctl enable-linger should also make sure the user's slice starts. So maybe it doesn't start exactly on boot but it should start on activate at the latest.
Something that I think may be worth addressing, raised by SuperSandro2000 here, is that this change makes it impossible to mutably manage linger even if mutableUsers is set.
We could make it contingent on mutableUsers = false, or we could make the default value null rather than false and only enable/disable-linger for users where it's explicitly set -- I think the latter would make the activation script a bit more complicated, but I prefer it.

Hm, maybe I missed something but if I do not persist the folder /var/lib/systemd/linger, lingering does not persist either on my setup. As far as I understand your replies, this behavior is unexpected because activation scripts are run during boot.
If you want, I can do more tests or create a separate issue.

Hm, maybe I missed something but if I do not persist the folder /var/lib/systemd/linger, lingering does not persist either on my setup. As far as I understand your replies, this behavior is unexpected because activation scripts are run during boot.
If you want, I can do more tests or create a separate issue.

I noticed this as well. I created a tmpfiles rule to automatically create my /var/lib/systems/linger/$user file, but it seems like that rule should already be in there. Is there any reason we would expect linger to be set to true and that file should not exist?

In case anyone is still monitoring this, I found out that user timers aren't started even with linger enabled. That seems to be in contrast to the explanations above. Anyone here encountered this too?

For me, user-space timers work as expected even on servers. I am using   users.users.USERNAME.linger = true;.
If you are using impermanence, you may want to persist
"/var/lib/systemd/linger"
"/var/lib/systemd/timers"

@arjan-s @dschrempf
Now that you mention: I have a user timer for changing wallpapers, and indeed it doesn't start after reboot.
I don't have linger on those hosts, since these are not servers and indeed I don't need user services that survive login for them, but I do expect timers to start automatically.
In other words, it seems counter-intuitive to have user-timers depend on linger, but I guess startServices would be acceptable if it works as expected; does it after boot/login though?

@dschrempf Thank you! systemd.user.startServices was indeed the missing link! Even though I'm using impermanence, I didn't have to persist those two directories.
@ppenguin On my laptops, user timers and users services are autostarting and working perfectly, without linger and startServices. It sounds like your login method doesn't start the systemd user session. For some methods you need to enable this explicitly.

@ppenguin what does systemctl status [--user] $your-timer.timer show?


icon customization not working

Home-assistant: 0.53.0
Python3: 3.4.2
Component/platform: customize
Description of problem: Using the web interface customization (Configuration -> Customization) to change the icon. The icon value is not changed when only the mdi icon name is provided.
Expected: When typing the icon name I would expect the web interface to show that icon. Instead a blank icon (no icon) is shown.
To make the icon appear as expected I had to type mdi:icon instead of just icon name (eg. 'atom'). The web UI leaves me to assume 'mdi:' is already provided, but it is not. I don't know if this is a bug, or intended, but it is confusing. After successfully setting the icon in the web UI it shows the setting as 'mdi:mdi:atom', while in the customize.yaml file it is icon: mdi:atom.

Duplicate of #9348, fixed in home-assistant/home-assistant-polymer#423


Request: Bring back twitter video trimmer

Couldn't find a better screenshot on Google but twitter used to have a helpful video trimmer which was removed in the new UI.
Would be cool to get it back.

Sorry, we can't restore old functionality which has been removed from Twitter itself


EQP syntax improvements - Check that block is not editing "self"

Closed in https://github.com/yeagerai/genlayer-simulator/pull/146


Green dot is not instantly updated for user A if user B request more money then what it owes to user A

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the app in mac safari, mac chrome, windows chrome and login with user A and keep user B chat open
Open app in any other device and login with user B
From user B device, first if there is any pending amount with user A, cancel or pay the amount and then request money from user A
Observe on user A device that green dot is directly updated as soon as message is received
Clear the payment from user A device pending with user B
Now using user A device, request money from user B for any amount
From user B device, request money greater then pending amount to user A
In user A device, observe that green dot is not displayed until we change the chat

Expected Result:
App should dynamically display green dot for pending money request to user A even if user B requested the money over current pending amount (works well when user B don't have any pending amount with user A)
Actual Result:
App does not dynamically display green dot for pending money request to user A if user B already had pending amount left and request the money over that current pending amount
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.2.96-4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/230683882-01d32003-07b3-451e-928d-615c75367edb.mp4
https://user-images.githubusercontent.com/43996225/230683892-dc9cf64b-2379-43df-b6c1-c2578661406b.mp4
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1680858583422489
View all open jobs on GitHub

Triggered auto assignment to @stephanieelliott (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Hm, I'm unable to reproduce this -- the green dot is persistent after the Request Money action. @kavimuru are you still able to repro this?

Hi @stephanieelliott , the issue only occurs if user B already have some pending amount with user A and user B requests for amount greater than what it needed to pay to user A (Keep user A logged in on mac safari or mac chrome and keep user B chat open). I was still able to reproduce on latest staging build.

Ah I see, thanks for clarifying that @dhanashree-sawant! I think this is pretty edge-case -   I can't see someone requesting more than is required to settle up being a common scenario (if anything it would probably be an accident when it does happen), and it's isolated to one platform where the intended behavior does occur on refresh. I think we don't need optimize for this case


Fix connection manager

Exchange messes up the connection management when multiple peers come and go.

Pull Request Test Coverage Report for Build 372

274 of 451 (60.75%)  changed or added relevant lines in 16 files are covered.
22 unchanged lines in 5 files lost coverage.
Overall coverage increased (+3.9%) to 34.521%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

dht/dht_query.go
0
2
0.0%

net/handshake_syn.go
12
17
70.59%

net/handshake_syn_ack.go
12
17
70.59%

net/handshake_ack.go
12
17
70.59%

net/manager.go
10
17
58.82%

dht/provider.go
6
19
31.58%

dht/provider_request.go
6
19
31.58%

dht/peerinfo_response.go
6
19
31.58%

dht/peerinfo_request.go
6
19
31.58%

dht/provider_response.go
6
19
31.58%

Files with Coverage Reduction
New Missed Lines
%

blocks/pack.go
1
62.84%

dht/dht_query.go
1
0.0%

dht/dht.go
4
0.0%

net/exchange.go
5
45.08%

blocks/block.go
11
10.53%

Totals

Change from base Build 369:
3.9%

Covered Lines:
688

Relevant Lines:
1993

💛  - Coveralls


Validate forms2

pls accept my merge

The login page is alot more refined with error messages with incorrect inputs or database errors.


v.0.6.0 pre release

v.0.6.0 pre release

Ok

Ok не достатньо

May be this ...

ні. не це

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/approving-a-pull-request-with-required-reviews


Support for custom project structures & flexibility

Support customs projects structures
The framework should support custom structures for projects like "module structures" and it should be more flexible.
TODO

[x] Refactor Loader (#88)
[x] Check instance of structures
[x] Don't save structures in client.collections automatically
[x] Allow multiple clients

[x] Commands : Add strategies to register interactions

Implemented in version 4.0.0-dev.11


refactor: remove helper functions from Transaction and use ShareableDatabase instead.

Transaction takes a ChainSpec on its constructor
Transaction can return a &ShareableDatabase<&DB>, which allows every stage to access the provider traits/functions (take a look at crates/stages/src/stages/execution.rs
Removes some helper functions from Transaction, replacing its usage with the provider ones.

Looking for feedback on if it's the way to proceed.
One current issue is that this way, there's no transaction reuse ... leading to the opening and closing of many transactions. Same issue as here https://github.com/paradigmxyz/reth/issues/2871 .

Codecov Report

Merging #2883 (dbcec46) into main (b03208c) will decrease coverage by 55.67%.
The diff coverage is 0.00%.

: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.
@@             Coverage Diff             @@
##             main    #2883       +/-   ##
===========================================
- Coverage   71.16%   15.49%   -55.67%
===========================================
  Files         513      513
  Lines       66470    66511       +41
===========================================
- Hits        47301    10307    -36994
- Misses      19169    56204    +37035

Flag
Coverage Δ

integration-tests
15.49% <0.00%> (-1.94%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

bin/reth/src/chain/import.rs
0.00% <0.00%> (-7.70%)
:arrow_down:

bin/reth/src/debug_cmd/execution.rs
0.00% <0.00%> (-1.20%)
:arrow_down:

bin/reth/src/debug_cmd/merkle.rs
0.00% <0.00%> (-1.18%)
:arrow_down:

bin/reth/src/node/mod.rs
0.00% <0.00%> (-11.05%)
:arrow_down:

bin/reth/src/stage/dump/execution.rs
0.00% <0.00%> (ø)

bin/reth/src/stage/dump/hashing_account.rs
0.00% <0.00%> (ø)

bin/reth/src/stage/dump/hashing_storage.rs
0.00% <0.00%> (ø)

bin/reth/src/stage/dump/merkle.rs
0.00% <0.00%> (ø)

bin/reth/src/stage/run.rs
0.00% <0.00%> (-1.64%)
:arrow_down:

crates/blockchain-tree/src/blockchain_tree.rs
0.00% <0.00%> (-82.80%)
:arrow_down:

... and 23 more

... and 357 files with indirect coverage changes

closing as per the sync


Fix IMDS parsing invalid json

Fixes #37409

API change check
API changes are not detected in this pull request.


Problem setting up the environment

I'm trying to run the project on my pc, but i keep running into the following error:
Traceback (most recent call last):
  File "D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\main.py", line 383, in <module>
    main()
  File "D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\main.py", line 99, in main
    train_dataset = datasets.ImageFolder(traindir,
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 310, in __init__
    super(ImageFolder, self).__init__(root, loader, IMG_EXTENSIONS if is_valid_file is None else None,
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 145, in __init__
    classes, class_to_idx = self.find_classes(self.root)
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 221, in find_classes
    return find_classes(directory)
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 42, in find_classes
    raise FileNotFoundError(f"Couldn't find any class folder in {directory}.")
FileNotFoundError: Couldn't find any class folder in D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\train.

Can you help me?

The compressed package of pth cannot be decompressed as a tar file. What statements do you need to decompress open tar

I'm trying to run the project on my pc, but i keep running into the following error:
Traceback (most recent call last):
  File "D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\main.py", line 383, in <module>
    main()
  File "D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\main.py", line 99, in main
    train_dataset = datasets.ImageFolder(traindir,
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 310, in __init__
    super(ImageFolder, self).__init__(root, loader, IMG_EXTENSIONS if is_valid_file is None else None,
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 145, in __init__
    classes, class_to_idx = self.find_classes(self.root)
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 221, in find_classes
    return find_classes(directory)
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 42, in find_classes
    raise FileNotFoundError(f"Couldn't find any class folder in {directory}.")
FileNotFoundError: Couldn't find any class folder in D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\train.

Can you help me?

I'm trying to run the project on my pc, but i keep running into the following error:
Traceback (most recent call last):
  File "D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\main.py", line 383, in <module>
    main()
  File "D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\main.py", line 99, in main
    train_dataset = datasets.ImageFolder(traindir,
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 310, in __init__
    super(ImageFolder, self).__init__(root, loader, IMG_EXTENSIONS if is_valid_file is None else None,
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 145, in __init__
    classes, class_to_idx = self.find_classes(self.root)
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 221, in find_classes
    return find_classes(directory)
  File "C:\Users\leodo\AppData\Local\Programs\Python\Python39\lib\site-packages\torchvision\datasets\folder.py", line 42, in find_classes
    raise FileNotFoundError(f"Couldn't find any class folder in {directory}.")
FileNotFoundError: Couldn't find any class folder in D:\LEO\2-Estudos\sem8\tcc2\git reconhecimento de expressoes\EfficientFace\train.

Can you help me?

Sorry, I did not run the code on the PC environment.

Using "checkpoint = torch.load('./checkpoint/mdoel_name.tar')"
Best

The compressed package of pth cannot be decompressed as a tar file. What statements do you need to decompress open tar

Using "checkpoint = torch.load('./checkpoint/model_name.tar')"


Add mkdocs

Add automatic documentation with mkdocstrings

Changes:

[x] Add mkdocs support.
[x] Add mkdocstring support.
[x] Change images to docs/images.
[x] Add automatic documentation generator (docs/generate_reference_docs.py).
[ ] Update data documentation.
[ ] Update data_selectors documentation.
[ ] Update fit_functions documentation.
[ ] Update utils documentation.

In the close future i would like to implement a deployment of github-pages of the automatic documentation as continuous-integration (ci). To do after definitive name for repository is selected.


Price will be over for nem

Is price over or it’s going to stay

🤷🏾♂

Please ask on discord or twitter(X).


Fix config reload after vim edit

When vim edit a file, it write a temp file and move it.
In this case, our watcher didn't work anymore.
This PR rewatch file when file is renamed/mv

I made a lot of test and it works really great.
I added a try/catch but it should be unused. If a config file is missing, a new one is written: https://github.com/zeit/hyper/blob/24b83615d4f37cfead075c5a80383dac9c3311ee/app/config/import.js#L41
If writing failed, an exception will be shown.
Thus, we can silently (but with log) discard an hypothetical watch exception.
User would have to restart its application to have a config watcher, but this is ok for me because it should not happen.


feat: volume fade in/out

adds volume fade in/out effect.
on not android platforms this is done by using Animated.Value. I do not have iOS to test but it seems like iOS doesn't have a background react rendering problem that this should work.
on android this is done by using https://github.com/doublesymmetry/KotlinAudio/pull/91. because Animated.Value doesnt update within HeadlessJsTaskService I added a simple linear fading function for it.
kotlin's scope.async doesn't work as I expected (Deferred.await() doesn't wait for RN's Promise.resolve?) any help is appreciated! because of this I cannot reliably await for the volume fading change to complete, so I used event emitting and handle the callback function within that event emitting, which is quite unnecessarily complicated, compared to a simple callback function in Animated.Timing.Start().

I'd love to see this get submitted into 4.1.0 — @dcvz do you think that's possible?

bump
On Mon, Jan 29, 2024, 5:48 PM github-actions[bot] @.***>
wrote:

This PR is stale because it has been open 90 days with no activity. Remove
stale label or comment or this will be closed in 7 days.
—
Reply to this email directly, view it on GitHub
https://github.com/doublesymmetry/react-native-track-player/pull/2121#issuecomment-1915914793,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZMOVVXBF52PJUV3WZMI57LYRBGP7AVCNFSM6AAAAAA4RN75ASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVHEYTINZZGM
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

bump
On Thu, Aug 1, 2024, 6:55 PM github-actions[bot] @.***>
wrote:

This PR is stale because it has been open 90 days with no activity. Remove
stale label or comment or this will be closed in 7 days.
—
Reply to this email directly, view it on GitHub
https://github.com/doublesymmetry/react-native-track-player/pull/2121#issuecomment-2264347402,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZMOVVSXGCBYAQYH6WI3MZLZPLRHRAVCNFSM6AAAAAA4RN75ASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRUGM2DONBQGI
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


Improve the python implementation of some search algorithms

Issue Type
Code improvement
Changes
Returning the complete path from the root node to the goal node from existing functions for search algorithms in search.py
Descriptions
The current recursive and iterative algorithms for breadth-first search, depth-first search, etc. only returns the node or None when goal node is found or not found respectively.
I propose to improve the algorithm to return the complete path from the start node to the goal node in case a path is found otherwise return None.
I also propose to modify driver functions accordingly.
@MrDupin may I work on this?

This sounds good. What do you mean by 'driver functions' though?

By 'driver functions' I am referring to the functions calling the search functions as their code also would need to be modified to accommodate  the changes in the return values of the search functions.

Hmmm, if the driver function changes are only in the way they get return values, then it's fine. I don't want to modify them that much, since they must be as close to the pseudocode as possible.

@MrDupin I have made the changes and as suggested by you lso, I made minimal changes in the test_search.py file in PR #1070
My changes pass all the test cases provided in test_search.py but during final testing the output is expected in the old format and therefore, some test cases failed.
Should I format the output according to the old requirements?

What is the old format and what is the new format of the output?

The old format("<Node ()>") is created by using the single node returned by the search functions.
But, the newly modified search program is directly returning the list of nodes in the path without any further computation.
But due to the difference in the formats, the above test case failed even though the values were same.

Hmm, you can change the format, sure. What I'm not sure about now is the implementation itself. It seems that you are using quite a lot more lines to find the full path. Can we not use Node's parent attribute to backtrack through the path?

Yes, sir, we can but that would increase the computation by again backtracking from the current node to the root which according to me can be done at the same time we finding a path.

The issue is that the code now gets more complicated, which is not desirable. Remember that this project is educational in nature and efficiency comes second. Operations should be done in the simplest and most intuitive manner possible.

Sure sir, I got your point. Sir, any suggestions on as to how I should proceed in order to add the functionality without adding any further complexity would be appreciated.

I am not that familiar with the search module, but can't we backtrack from node.parent to node.parent?

@codefeeder were you able to improve your algorithm? Is this issue closed?

As per me, create custom test cases or use regex to remove <Node ()> and then compare
Or simplest way is to add in your output


Allow for velocity coordinate system changes

A major feature request, and fundamentally this ought to be in specutils, but I want it here first.
The FITS standard defines many useful reference frames, and we should allow transformation between them.  TOPO/GEO->other frames will be hard, so we can put that off, but transformations between other frames are well-defined and easy, so those should be a higher priority.
This PR so far contains only a documentation skeleton for a new function with_reference_frame.

Actually I wonder if this could live in astropy.wcs.utils once it works?

Yes, that's probably a good place for it.  I keep being surprised that no one has implemented these things... I'm sure they exist in AST and in casacore, but... well.

Tagging @eteq since this has some connections to what we discussed at pyastro15 - but the difference here is that this is to modify WCSes, rather than transform coordinates directly.

FYI, this came about because of a specific feature request from a colleague to be able to convert spectra (or cubes) into the SOURCE rest frame, so that's my highest priority.

@keflavich @astrofrog - so just to be clear, this is about being able to take things in one WCS and convert to a different velocity frame, or just figure out how to interpret different input keywords?
I ask because the conversion I think should be implemented as part of a related coordinates effort.  But definitely that's not an easy thing to do right, so a temporary solution here limited only to the WCS-valid keywords is a reasonable interim approach.

@eteq - yes, this is about conversion.  Interpretation is kind of trivial, because if a file implements the FITS convention correctly, the conversion from pixel coordinates to world coordinates for any given WCS is unaffected by these various keywords, only transformations between different WCSs are affected.

@keflavich - I wonder if wcslib will do these velocity transformations?  If so that might be an easy way to do it: let wcslib via astropy.wcs take care of it?

@eteq - as far as I can tell, it does not handle transformations; it instead treats these keywords as "auxiliary": e.g. http://www.atnf.csiro.au/people/mcalabre/WCS/wcslib/structwcsprm.html#ac089e5d0e3191255ceaea7f8591b27ea.  However, @mdboom, could you comment further - can wcslib compute transformations between different frames and/or different CTYPEs?

Closing: this will be handled by specutils in https://github.com/astropy/specutils/issues/422


Custom function ID naming requirements are unclear

This page says

use characters A through Z, numbers zero through nine, underscores, and periods.

On https://learn.microsoft.com/en-us/office/dev/add-ins/excel/custom-functions-json it says

contains only alphanumeric characters and periods
... unique within the scope of the file. That is, no two function objects in the metadata file should have the same id value

Are lowercase letters allowed?
If so, are these distinct? "MYFUNC" vs "myfunc"
I noticed in the sheetx.xml the periods are converted to underscores. Does that mean "MY.FUNC" is not distinct from "MY_FUNC"?

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

ID: 714da1e2-6053-0328-b31e-0bf514ecc774
Version Independent ID: be1e524c-1198-c288-f71e-f9d7f84abeb7
Content: Naming guidelines for custom functions in Excel - Office Add-ins
Content Source: docs/excel/custom-functions-naming.md
Service: excel
Sub-service: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

I've raised a script lab issue too https://github.com/OfficeDev/script-lab/issues/1009.
I would prefer it if this were actually considered an Excel and fixed. I'm wanting to use period as a namespace separator in my custom function and have been assuming that IDs can't conflict with different ones in the parent namespace.

Note, using underscore and period works fine in a LAMBDA function.

Hi @parched, thanks for submitting this question.
I've tested the ID naming requirements and my results are as follows.
Question: Are lowercase letters allowed as custom function ID values?
Answer: Yes, but they will be automatically converted to uppercase letters in the JSON output.
Question: Are lowercase letters distinct from uppercase letters as custom function ID values?
Answer: No. Custom function ID values are not case-sensitive. For example, “A” is the same as “a”.
Question: Are periods (.) distinct from underscores (_) in custom function ID values?
Answer: Yes.
I'm confirming these answers internally. I'll report back when I've updated the documentation, or if I have additional information.
Cheers,
Alison


LUNA->EUR not found

So I'm getting this error with LUNA when trying to sync using REST and Binance_com from Spain. I bought some LUNA when it crashed an was delisted from many places, so that could also contribute to this mess.
I'm getting the following error:
ERROR: Fatal exception occurred:
Traceback (most recent call last):
  File "@@@@@/dali-rp2/src/dali/dali_main.py", line 178, in _dali_main_internal
    resolved_transactions: List[AbstractTransaction] = resolve_transactions(transactions, dali_configuration, args.read_spot_price_from_web)
  File "@@@@@/dali-rp2/src/dali/transaction_resolver.py", line 286, in resolve_transactions
    transaction = _update_spot_price_from_web(transaction, global_configuration)
  File "@@@@@/dali-rp2/src/dali/transaction_resolver.py", line 146, in _update_spot_price_from_web
    raise RP2RuntimeError(
rp2.rp2_error.RP2RuntimeError: Spot price for<PHONE_NUMBER>980985346:2022-12-21 20:42:06+00:00:LUNA->EUR not found on any pair converter plugin

This is the config file I'm using:
# Data loaders
[dali.plugin.input.rest.binance_com]
account_holder = my_name
api_key = my_key
api_secret = my_secret
native_fiat = USD

[dali.plugin.pair_converter.ccxt]
historical_price_type = nearest
default_exchange = Gate

A I missing something? Should I try going the csv route instead? Any help is deeply appreciated.

Since these are two separate errors, that need to be dealt with differently it's best to open two different issues. The first should be fairly easy to solve, but the second one is probably due to LUNA is now called LUNC. We need to implement aliases #140  to resolve that.
Can you open an issue, for the first one?
Then edit this issue to just include the second issue since that is what the title of this issue describes. Thanks!

Since these are two separate errors, that need to be dealt with differently it's best to open two different issues. The first should be fairly easy to solve, but the second one is probably due to LUNA is now called LUNC. We need to implement aliases #140 to resolve that.
Can you open an issue, for the first one?
Then edit this issue to just include the second issue since that is what the title of this issue describes. Thanks!

Thank you for bringing up the need to address these separate issues separately. It definitely helps in keeping track of everything more effectively.
Regarding the aliases issue, do you have any rough estimate on the timeline for its resolution? I don't mean to rush anyone, but I have a deadline for my taxes by the end of this month. If it seems unlikely that the fix will be ready in time, please let me know so that I can explore alternative solutions.
I want to reiterate my admiration for this fantastic tool and the tremendous effort put into making it open-source and non-commercial. I understand that most of the work on this project is done during your free time, and I genuinely appreciate the dedication that goes into it. If the answer is simply "it will be fixed when it's fixed," that's completely understandable.
Thank you once again for your hard work and for your assistance with these issues.

It will most likely not be done in time.
You could try to monkey patch it, though. Just inside the get_historic_bar_from_native_source() function around here. You can try adding the following and see if that will help you out:
If from_asset == "LUNA":
    from_asset = "LUNC"
If to_asset == "LUNA":
    to_asset = "LUNC"

Then, patch the graph building inside _add_exchange_to_memcache() inside the for loop around here.
If market[_BASE] == "LUNA":
    market[_BASE] = "LUNC"
If market[_QUOTE] == "LUNA":
    market[_QUOTE] = "LUNC"

That should work, but no guarantees. Keep the changes local, and don't submit a PR. Of course, any pull you make will overwrite these changes so be careful, but it will work in a pinch.

@raizazel are you able to price LUNA/LUNC with the latest version? I'm hoping to close this issue.

@raizazel are you able to price LUNA/LUNC with the latest version? I'm hoping to close this issue.

Hi!, sorry for leaving this opened for that long. I've tested this and it seems to work. You can close the issue, thanks a lot!

Aliases have been implemented to fix this, so I'm closing.


Complie error occurred

Hello,
Thanks for the nice SW.
While I was trying compile the SW, i met the error message below.
Could please help me how to fix this problem?
SWs: Ubuntu 2020 + cuda 10.2 with Nvidia-driver-450
GPU: Titan RTX

[  5%] Built target octomath
[  6%] Built target codetimer
nvcc fatal   : A single input file is required for a non-link phase when an outputfile is specified
make[2]: *** [_deps/gvdb-build/gvdb_library/CMakeFiles/gvdbPTX.dir/build.make:76: _deps/gvdb-build/gvdb_library/CMakeFiles/gvdbPTX.dir/kernels/cuda_gvdb_module.ptx] Error 1
make[2]: *** Waiting for unfinished jobs....
nvcc fatal   : A single input file is required for a non-link phase when an outputfile is specified
make[2]: *** [_deps/gvdb-build/gvdb_library/CMakeFiles/gvdbPTX.dir/build.make:63: _deps/gvdb-build/gvdb_library/CMakeFiles/gvdbPTX.dir/kernels/cuda_gvdb_copydata.ptx] Error 1
make[1]: *** [CMakeFiles/Makefile2:1724: _deps/gvdb-build/gvdb_library/CMakeFiles/gvdbPTX.dir/all] Error 2
make[2]: *** No rule to make target '/usr/lib/x86_64-linux-gnu/libGLX.so', needed by '_deps/pangolin-build/src/libpangolin.so'.  Stop.
make[1]: *** [CMakeFiles/Makefile2:1778: _deps/pangolin-build/src/CMakeFiles/pangolin.dir/all] Error 2
make: *** [Makefile:152: all] Error 2

Hmm, seems like you don't have the system libGL installed? Did you install OpenGL libraries with CUDA/NVidia drivers? Try to install the Pangolin dependencies here
https://github.com/stevenlovegrove/Pangolin