maxCachePeriodInSecond to unlimited

Hi @onevcat currently my app has two types of images.

The one that comes on news feeds. For them image cache of 7 days is good.

Images that come in chat. Until user logout and login with different id, i need those images to be cached i.e. to clear disk cache manually only.

How can i achieve this?

Instead of using the default cache, you could setup and use two different caches (ImageCache), one for each.
See the cheat sheet for more.

I already saw that. My issue wasn't that. I just wanna know how to implement maxCachePeriodInSecond = infinity //Unlimited.
just like maxDiskCacheSize can be set to 0 for unlimited storage size

Oh,
There is no such "infinity" date currently. Maybe you could set maxCachePeriodInSecond to a relatively large number, like "300 * 50 * 60 * 60 * 24 * 7" (maybe 300 years?)

Okay thanks i would implement that.
Just wanna ask, are you planning to implement this in future?
And one more thing. Is there a callback of images that are going to be deleted? I mean just the image keys might work

@onevcat Please help

Sure. It could be clear to add a forever option for the cache time. I will do it later.
And there is a KingfisherDidCleanDiskCache notification you could use for the images. You can find the API reference here.

What if i added this notification on a controller and kingfisher perform cleanup when that controller is not ye initialized. then?
Do i have to implement this on every controller(bad idea).
Please provide a example with best use case.

Your selector will not get called if the instance (target) is not there.
You'd like to add that to a long life time object, for example your app delegate or your very first tab view controller which will exist across the whole app life time.

Ok thank you.
Will disturb you again if i got stuck in any other problem...:p


Error when calling join()

Hi, I tried to build a little scheduler using wasm_thread, based on the simple example (build with the provided build.sh script)
When I call join() on the JoinHandle, I get the following error in the console:
Uncaught RuntimeError: Atomics.wait cannot be called in this context
    at std::thread::park::h593d412d90d62fe8 (simple_bg.wasm:0xa15d)
    at std::thread::local::LocalKey<T>::with::h7c7778a2607becf9 (simple_bg.wasm:0xb690)
    at futures_executor::local_pool::block_on::hf38892d64f63bc2a (simple_bg.wasm:0xc0e9)
    at wasm_thread::JoinHandle<T>::join::hb8eb16f7d78b7dfe (simple_bg.wasm:0xc237)
    at stop_scheduler (simple_bg.wasm:0x7531)
    at WebAudioSwitch.<anonymous> (simple.html:21:10)
    at WebAudioSwitch.sendEvent (webaudio-controls.js:159:12)
    at WebAudioSwitch.pointerdown (webaudio-controls.js:1046:14)
$std::thread::park::h593d412d90d62fe8 @ simple_bg.wasm:0xa15d
$std::thread::local::LocalKey<T>::with::h7c7778a2607becf9 @ simple_bg.wasm:0xb690
$futures_executor::local_pool::block_on::hf38892d64f63bc2a @ simple_bg.wasm:0xc0e9
$wasm_thread::JoinHandle<T>::join::hb8eb16f7d78b7dfe @ simple_bg.wasm:0xc237
$stop_scheduler @ simple_bg.wasm:0x7531
(anonymous) @ simple.html:21
sendEvent @ webaudio-controls.js:159
pointerdown @ webaudio-controls.js:1046

Best,
N

Ah right, yes ... sorry, must have missed that. Thanks for the quick reply !


RSS feed not updated since 2022-08-10

The most recent entry in the RSS feed appears to be Blur Finance rug pulls for over $600,000.
Thank you for your work on this project, and in particular for maintaining an RSS feed of it!

I'm seeing the same thing here.

...and now fixed. Thanks for the quick response!

Thanks for the heads up! Something must be up with my alerting on that again :(


ASP.NET Core Identity is broken, it implements explicitly an old version of Duende API.

This is broken, AspNetCore is using a depercataded version of Duende as you can see in this Open issue https://github.com/dotnet/aspnetcore/issues/44990
ApiAuthorizationDbContext.cs implements explicitly the old Duende API.
Duende is so aware of this that they even have commments on their code that explains the problem
I have created a PR to add a missing property on ApiAuthorizationDbContext, please follow it https://github.com/dotnet/aspnetcore/pull/45145

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

ID: 94bfd895-d2af-390b-0801-428b36a27ff4
Version Independent ID: 0e0d870e-e49a-e16c-c259-68d58e701b8f
Content: Introduction to Identity on ASP.NET Core
Content Source: aspnetcore/security/authentication/identity.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Let's close this until https://github.com/dotnet/aspnetcore/issues/44990 is resolved.


bug: incorrect PDF certif being generated

Describe the bug
I am a part of 1-100 cohort and it tends to generate certificate for 0 - 1. Its an similar issue to #737
I assume this is gonna get fixed it #739 , if not we can do it with #739 itself
Expected behavior
I should get certificate for  1 - 100 instead of 0 - 1
Screenshots or GIFs

@HarshPatel5940 It was resolved in #728

@HarshPatel5940 It was resolved in #728 for PNG
For PDF it was resolved with correct credentials in #738

Mhmm, cool. Then we can close this as soon as those are merged.

Is this issue still open or can it be closed ? @hkirat


Some services still running after excecuting tweaks

Describe the bug
Distributed link tracking client,sysmain/superfetch still running after check disable service to manual and restart
To Reproduce
Steps to reproduce the behavior:

Go to tweaks
Click on disable service to manual
and run tweak and restart
4.It should stop services like distributed link tracking clients, and it clearly states stopping sysmain

Expected behavior
Stopping those 2 services
Additional context
I'll try with a bunch of different laptops, but it's still not stopping. BUT the process in Task Manager reduce like always after executing the commands.

Resolved in #883


New 2.0 Angular Template - Home Controller/View ASP.NET Authentication

I've created a file/new of the 2.0 Angular Template.
I've noticed that the angular cli index.html page is not mapped through a Controller/Action when loading.
What I want is for Controller Home/Action Index to return the index.html page that angular-cli JSON is configured.
Otherwise, how would I apply ASP.NET Auth against my main App? Traditionally I would add an AUTHORIZE attribute to my Controller.
Another way of asking this...
How can I host the SPA on an MVC/CSHTML page instead of the angular cli index.html? Is it even possible... should I just be using the 1.0 Angular Template at this point?

Would this work for you? An Angular 4 .Net Core 2.0 Example With Application Shell and Authentication

I am looking for exactly the same functionality. I need to setup an input element in cshtml which could be accessible by the angular components (to pass data from server) and don't want to use server side pre-rendering.
I just downloaded the latest template using command:
dotnet new --install "Microsoft.DotNet.Web.Spa.ProjectTemplates"
Then I went on to create a new project using :
dotnet new angular
The template "ASP.NET Core with Angular" was created successfully.
However the result project only had an API Controller (SampleDataController.cs) and did not contain the Views folder. Not sure if I got the correct template.
Can someone please comment?

@aherrick @syedm0 Look to this thread.
https://github.com/aspnet/JavaScriptServices/issues/1518#issuecomment-386035771

I've added basic B2C to my App. Using something similar to below to redirect user to force auth.
            app.UseAuthentication();

            app.Use(async (context, next) =>
            {
                if (!context.User.Identity.IsAuthenticated)
                {
                    await context.ChallengeAsync(OpenIdConnectDefaults.AuthenticationScheme,
                        new AuthenticationProperties { RedirectUri = "/" });
                    return;
                }
                await next.Invoke();
            });


Redis Cluster Options Not Added on Initialization

Package version
2.0.7
Node.js and npm version
Node: 12.13.1
NPM: 6.12.1
Sample Code (to reproduce the issue)
In RedisFactory:, when we initialize an IoRedis.Cluster the other parameters such as scaleReads is not passed as second parameter. Instead, only redisOptions is passed as stated in the IORedis documentation.
  _newConnection () {
    if (this._useCluster) {
      debug('creating new redis cluster using config: %j', this._config)
      return new IoRedis.Cluster(this._config.clusters, { redisOptions: this._config.redisOptions })
    }
    debug('creating new redis connection using config: %j', this._config)
    return new IoRedis(this._config)
  }

I have directly modified it on my node_modules as a quick test and it worked for me.
  _newConnection () {
    if (this._useCluster) {
      debug('creating new redis cluster using config: %j', this._config)
      return new IoRedis.Cluster(this._config.clusters, { scaleReads: "all", redisOptions: this._config.redisOptions })
    }
    debug('creating new redis connection using config: %j', this._config)
    return new IoRedis(this._config)
  }

As seen on this metric, all replicas are receiving traffic after I added the scaleReads option.

As a side note, any chance we can jump to a newer version of IORedis?

I have created my own branch @  https://github.com/renzosunico/redis/tree/fix-redis-cluster-options

I have created my new branch. You can check it out here.
While waiting for this to be merged, I have forked and published this branch in npm for now as we really need this fix. Thank you.


Arrow handler

This PR introduces a new handler for the arrow key down event. Currently only the left and right arrow are handled. For each of these arrow keys, a separate command is created that handles that key press.
I started on the backspace handler first, but while implementing this I realized that the arrow handler could easily be implemented using the methods I already created for backspace handler. That's why I worked on this first.
This PR contains already some files of the backspace handler, but these are not yet in working in this PR.
Merge this before #142.
move-left-command-old.ts and move-right-command-old.ts are old implementations of the commands, but I didn't remove these files yet so I can go back to these and see my thinking process if later on bugs are found in the new implementations.

regression: shift+arrow keys no longer makes a selection
@nvdk do you think that's a blocking issue?

I think users complained about that in the past, but I'm not sure this was possible with the previous handler. This big report seems to suggest not: https://github.com/lblod/ember-rdfa-editor/issues/28

I think users complained about that in the past, but I'm not sure this was possible with the previous handler. This big report seems to suggest not: #28

it works on dev tho?

Ive changed the base branch of these PRs to preserve the work done in them. We are going through too many structural changes to merge this, but the ideas and work done here will make their way into the codebase one way or another


L.Control throws error when removing non-added layer

From #4227:
This currently fails:
	it('silently returns when trying to remove a non-existing layer from the control', function () {
		var baseLayer = L.tileLayer(),
		    overlay = L.marker([0, 0]),
		    layers = L.control.layers({'base': baseLayer}, {}).addTo(map);

		layers.removeLayer(overlay);
	});

message:
undefined is not an object (evaluating 'this._layers[i].layer')
_getLayer@/home/jieter/workspace/Leaflet/src/control/Control.Layers.js:194:31
removeLayer@/home/jieter/workspace/Leaflet/src/control/Control.Layers.js:112:27
/home/jieter/workspace/Leaflet/spec/suites/control/Control.LayersSpec.js:109:22

This one can be closed.

:)


Replacement of spacers in LazyLists and LazyGrids

This pull request will remove the Spacers in the LazyLists and LazyGrids to set margins between the lists / grids and other components by specifying the contentPadding parameter to the scrollable containers.
This PR also updates the dependencies.

Thanks for quick response!
Updated the version catalog with the suggested ones.


Update data.tables-readme-pre.md

Added some text to encourage customers to use the GA version instead of this preview.

#sign-off

@danieljurek what file(s) do I need to modify to ensure the change is maintained through subsequent preview releases?

@mario-guerra -- We copy the content of this file from the readme in the SDK repo. However, you'll have to ensure that the text is removed before you release GA otherwise the GA release page will show the content advising people to look at the GA version.
I have an issue open to automate this -- https://github.com/Azure/azure-sdk-tools/issues/2005 ... I've put the updated copy in the issue.

@mario-guerra -- We copy the content of this file from the readme in the SDK repo. However, you'll have to ensure that the text is removed before you release GA otherwise the GA release page will show the content advising people to look at the GA version.
I have an issue open to automate this -- Azure/azure-sdk-tools#2005 ... I've put the updated copy in the issue.

Good to know, thanks Daniel.


Add missing GitHub Actions build badge

#49 does not include the new badge. It still needs to be added according to the docs

Ya, we're going to need to add something that builds on master. There's no easy way to get a build-badge with the chatops. Note, that it runs on `cloudposse/actions@master`` and not this repo so the status don't line up.

There is a release badge at the moment and it's updated when a new release is published after a PR is tested successfully and merged. I'll close for now. Please reopen if this is still an issue.


Identify the NIC an announced service was discovered on?

Our s/w currently listens for mDNS services from a third party system on 2 different NICs. Our s/w processes services from each NIC separately, on different threads.
Is there any way in Zeroconf to identify which NIC an announced service was discovered on?
Thanks

Not quite fixed it seems, hold tight.

OK, these should all be fixed now:

Zeroconf has a new setIPv6(boolean) method to disable IPv6 (and an analogous setIPv4(boolean) method too)
Service has a new Collection<NetworkInterface> getNetworkInterfaces() method returning a read-only set
The fix you made for https://github.com/faceless2/zeroconf/issues/3 was on the right track - I'd missed a pass, basically, but testing all this is very dependent on what is announcing on your local network so I hadn't spotted it locally. I'm quietly confident, but If this doesn't fix it for you please let me know.

Also fixed an exception when cancelling a service, and I've bumped the revision for this set of changes so we're at 1.0.1

Thanks, that's awesome. This will really move my prototype along.
Is the JAR published to Maven?

It's not on Maven, but only because I don't use it myself and have never figured it out. Direct download link is in the README.MD however, currently https://faceless2.github.io/zeroconf/dist/zeroconf-1.0.1.jar

Understood, and thanks.

If I set IPv6 to false, the announced service includes both the IPv4 and IPv6 network interfaces. It's not a problem as I can easily ignore the IPv6 one.
Identifying the network interface for a discovered service works really well if services have already been published before I run my test app. If I publish a new service after my test app is started, the announced service has no network interfaces. I'll try to debug to see what's going on.
Thanks again.

The setIPv6() method was to ensure we only listen on IPv4 addresses, it didn't change the addresses we announce on. Now I write that out, it's clearly the wrong approach. Will fix so it applies to announcements too.
The NICs for a service we announce are just the sames as the NICs returned from ZeroConf.getNetworkInterfaces(). They're handled very differently for services we hear from the network.

If I publish a new service after my test app is started, the announced service has no network interfaces.

I have a hunch you're setting an address manually on the service. I'll try and explain.
Conceptually there are three types of service:

A service that is announced by someone else. We hear the announcements, and record the addresses on the service, noting (as of a few hours ago) which NIC they came in on.
A service that is announced by us and uses the default list of Network Interfaces. In this case the NetworkInterfaces retrieved from the Service are the same as ZeroConf.getNetworkInterfaces(), by definition, so we just return that list.
A service that is announced by us and uses a manual Network Interface, i.e. you call addAddress() on the Service.Builder. In this case the address can be anything - it doesn't have to be a local address - and the list is fixed, so is not updated to reflect any service announcements we hear (because we're the one sending them). In this case the list of Network Interfaces will never be updated - and in fact for non-local addresses, it wouldn't be possible to do so.

I have a hunch you're in situation 3, manually setting the address. Correct?
If you set a manual address and it's local, I suppose we could map that to a set of NetworkInterfaces - it would tell you which NICs the announcement was sent on. If it's not local, well I suppose we could still set it to the list of NICs we announced on. Is that what you're expecting?

Would it be possible to instead specify a single network interface (or any number I guess) to listen for services? If I could specify one interface to use, I wouldn't need to check which one was used in the announced callback.

Yes! Absolutely.
Zeroconf zc = new Zeroconf();
Collection<NetworkInterface> nics = zc.getNetworkInterfaces();
// By default "nics" has all of them, you can remove those you don't want or clear and add those you do.
nics.clear();
nics.add(mynic);

That should sidestep your problem, if not fix it, but it should get you going.

 I should have asked that question earlier, and I could have saved you some time. Sorry! 😃

I'm getting an IllegalStateException when I do this:
Exception in thread "ZeroConfWorker" java.lang.IllegalThreadStateException
	at java.base/java.lang.Thread.start(Thread.java:1525)
	at com.bfo.zeroconf.Zeroconf$ListenerThread.addNetworkInterface(Zeroconf.java:500)
	at com.bfo.zeroconf.Zeroconf$1.add(Zeroconf.java:110)
	at com.bfo.zeroconf.Zeroconf$1.add(Zeroconf.java:99)

This is the correct initialisation sequence?
        registry = new Zeroconf();
        registry.setIPv6(false);
        registry.setLocalHostName(ip);

        // Remove all NICs and add the one we want to use.
        Collection<NetworkInterface> nics = registry.getNetworkInterfaces();
        nics.clear();
        this.findRequestedNIC(nics);

Thanks

I suspect the call to isAlive() at line 499 isn't required? It isn't invoked in removeNetworkInterface().

OK, all done. When removing all the NICs the listening thread was being stopped, but of course you can't restart it after that. Now working and rebuilt 1.0.1

Ha! No I don't, but other people testing my work is valuable enough :smile:

Let me know if you change your mind. 😃 ☕

Listening for services on a single network interface works beautifully in my initial testing. 👍


Container refresh - service's Portal IP is not being collected

For openshift, a separate parse_service method will be needed I believe.
See here:
https://github.com/ManageIQ/manageiq/blob/master/app/models/manageiq/providers/kubernetes/container_manager/refresh_parser.rb#L170
That line has to be changed to service.spec.portalIP to work with openshift.

@miq-bot add_label providers/containers
@miq-bot add_label bug

@simon3z @abonas

The relevant change was done here afaik:
https://github.com/ManageIQ/manageiq/pull/3643
Service is a k8s entity and so it typically doesn't matter whether this is an openshift or a k8s parser, it should be handled the same.
@jkrocil it probably depends on the build that you were using.
cc @oschreib

My bad! Thank you :+1: (not closing yet as @oschreib  might want to add something; feel free to close otherwise)

@jkrocil I'm not sure I understood the issue here. (and as @abonas said, openshift and k8s are the same here).
Anyhow - can you describe exactly the problem? which build of openshift are you using?
Do you see that parameter under the rest-api named as portalIP? (using the v1 api)

I get a portalIP from a v1 k8s endpoint on an openshift setup.
The versions are:
oc v<IP_ADDRESS>-5-g279b678
kubernetes v0.17.1-804-g496be63
Openshift 3 with k8s 0.17:
{
  "kind": "ServiceList",
  "apiVersion": "v1",
  "metadata": {
    "selfLink": "/api/v1/services",
    "resourceVersion": "461142"
  }

yet there is portalIP
        "portalIP": "<IP_ADDRESS>",
        "type": "ClusterIP",
        "sessionAffinity": "None"

With k8s 1.1 I get this instead:
                "clusterIP": "<IP_ADDRESS>",
                "type": "ClusterIP",
                "sessionAffinity": "None"

So it is different builds...

Feel free to close.

Feel free to close.

I don't have permissions to close this issue, so if you do have permissions, and you think it's fine, please close.


github: add CI action

Please take a look.

I have disabled the requirement for travis build success.

Codecov Report

Merging #1526 (76aa5ae) into master (1858083) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1526      +/-   ##
==========================================
- Coverage   71.46%   71.31%   -0.16%
==========================================
  Files         489      473      -16
  Lines       54288    53774     -514
==========================================
- Hits        38797    38347     -450
+ Misses      13086    13051      -35
+ Partials     2405     2376      -29

Impacted Files
Coverage Δ

lapack/gonum/dlasq5.go

graph/network/page.go

mat/symmetric.go

num/quat/trig.go

unit/length.go

unit/mass.go

unit/constant/faraday.go

lapack/gonum/dlagtm.go

optimize/newton.go

lapack/gonum/dpbtrs.go

... and 950 more

Continue to review full report at Codecov.

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

I have tried to make Coveralls work with this, but it's pretty resistant. I think maybe we should drop it and remove the badge.

OTOH, using only one matrix/env (TAGS: "") and handling the ubuntu build+test case specially (ie: manually building+testing the matrix with all variants of TAGS) would perhaps speedup the pipeline (and save resources).

The single variant with all tags was how we used to have it, it took too long.

I'll leave the Coveralls badge for the moment and see what happens with travis. If we ditch travis, we can get rid of Coveralls (maybe lose it anyway).


Update guidance on named slots in framework components

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[x] New or updated content
[ ] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description

Follow-up to https://github.com/withastro/astro/pull/3652
Adds guidance around using named slots with framework components

Just one sentence commented above, otherwise, LGTM.
@Jutanium - care to provide a DINM? (Dan is not mad.)

Out of Astro time for today for a full review + comments. DINM, but instead of destructuring the props, please just accept props as the parameter and refer to props.title, props.children etc, because this works in all the JSX frameworks (destructuring loses reactivity in Solid).

Update branch, to appease the Hippo checks, and LGTM! Thanks Nate! 💜


Feature Request: $ref for servers object

In many use cases, there would be multiple APIs that would be associated with different servers. It would be great if the server could be referenced from a single server object under components and reused everywhere instead of overriding the servers object for each API.
Currently,
  servers:
    - url: 'https://bookstore/v1'
      description: 'Get books from bookstore'
  paths:
     /book/{id}:
        get:

would result in API: https://bookstore/v1/book/123
But we also have the following apis that have server somewhere else, We'll have to override the global server variable in each of these paths which results in redundancy, maintenance issues when the server variables are changed.
  components:
     servers:
       - url1: 'https://bookstore/v1'
         description: 'Get books from bookstore'
       - url2: 'https://library/v2'
         description: 'Get books from library'

  $ref: "#/components/servers/url1"
  paths:
     /book/{id}:
        get:

     /libraryBook/{id}:
      $ref: "#/components/servers/url2"
        get:
        ...
        delete:
        ...

    /listLibraryBooks/
     $ref: "#/components/servers/url2"

Waiting to hear your thoughts!

Please could you fix the inconsistent indentation so it's valid yaml? it's difficult to understand what you're asking.

I fixed the indentation of both code blocks here:
Before,
components:
  servers:
    - url: 'https://bookstore/v1'
      description: 'Get books from bookstore'

paths:
  /book/{id}:
    get:
#      API -> GET: https://bookstore/v1/book/123

  /libraryBook/{id}:
    servers:
      - url: 'https://library/v2'
        description: 'Get books from library'
    get:
#      ....
#      API -> GET: https://library/v2/libraryBook/123
    delete:
#      ....
#      API -> DELETE: https://library/v2/libraryBook/123

  /listLibraryBooks/:
    servers:
      - url: 'https://library/v2'
        description: 'Get books from library'
    get:
#      API -> GET: https://library/v2/listLibraryBooks

After,
components:
  servers:
    - url1: 'https://bookstore/v1'
      description: 'Get books from bookstore'
    - url2: 'https://library/v2'
      description: 'Get books from library'

paths:
  /book/{id}:
    $ref: '#/components/servers/url1'
    get:
#      API -> GET: https://bookstore/v1/book/123

  /libraryBook/{id}:
    $ref: '#/components/servers/url2'
    get:
#      ....
#      API -> GET: https://library/v2/libraryBook/123
    delete:
#      ....
#      API -> DELETE: https://library/v2/libraryBook/123

  /listLibraryBooks/:
    $ref: '#/components/servers/url2'
    get:
#      API -> GET: https://library/v2/listLibraryBooks

Looks like a duplicate of #2338.

Yes, closing this as a duplicate of #2338.


use node-nut instead of calling local upsc binary

thought this would be nicer for docker installs and generally not having to locally install NUT.  I'm also sending errors to node.status and updated for Node-RED 1.0.  Did a little refresh on the edit config dialog and help text.  Everything should be backwards compatible.

Cool, thanks! I'm not sure why you decided to change the code style though. My preference has been standard-but-with-tabs. I'll add an .eslintrc to make auto-formatting with eslint --fix work.

Let me know if anything else.

All good, thanks. I'll release a new version on npm now.


Inventor: Vitriolist charges

Using latest Solasta and UB mod version.
Started a new game, party has a Vitriolist Inventor.
Problem 1: Description doesn't say anything about how many charges i have on my Vitriolic Mixture feature.

Problem 2: Charges aren't indicated on the power tab. I keep on using them and the option just suddenly disappears after some uses, suggestion it has a limited number of uses.
Save file here:
domino2.zip

Sorry its Int Mod + PB per long rest.  If you mouse over each individual Mixture youll see the uses.  But mousing over the main Mixture button that lets you pick a mixture doesnt show it right now.


Sql Injection Vulnerability

Hey bro!
This code is injection vulnerable. For instance, anyone can login without password because of this line of code.
If I may, I suggest you to use PDO and prepared statements everywhere you create a query.
Here you can find some examples:

AbstractStorage.php

Thats true.
I've adopted this code from a bro ;-) and support it.
The next version you can see here in this repository https://github.com/schillermann/trolleydienst/tree/mvc
You're from Hamburg?
I'm from cologne :-)

I live in Hamburg, but I am from Brazilian =)
Nice work, it is getting better!

The motto is: keep it simple for changes other user.


add dockerfile for testeth with hera included

Can you give a quick explanation what is this used for?

Also please add a README file in the directory.

It's going to be used for the wrc20 CI that I am working on here: https://github.com/ewasm/wrc20-examples/pull/7 .
Needs to be cleaned up and size-minimized.

Found this: https://hub.docker.com/r/ethereum/testeth which seems more frequently updated.. Will base this image off of it.


Missing include of standard header files

I encountered them during x-compile onto qnx.

"#include <sys/time.h>" is missing from modules/canbus/vehicle/lincoln/protocol/steering_65.h for reference to "struct timeval"

"#include <sys/time.h>" is missing from modules/canbus/vehicle/lincoln/protocol/wheelspeed_6a.h for reference to "struct timeval"

"#include <array>" is missing from modules/common/math/integral for reference to "std::array"

@jilinzhou thanks for the feedback, commit has been submitted to resolve this issue, please feel free to create a PR directly in the future. https://github.com/ApolloAuto/apollo/pull/2931/files


Fixed failing Angular8 sample app tests, and added a few new ones

This PR fixes #1327

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

I wanted to fix the build errors but looks like I am not authorized to see the build results (I get the 401 error below), is this intentional?

@mbassit Yeah, unfortunately our ADO isn't visible externally. The issue is that our Code Coverage doesn't work properly with forks, which is fine. I've been meaning to take a look at this PR, I'll try to do so today.

Looks good, apologies for the delay. Merging.


Update UI of sidebar

Description
I want to give a new look to sidebar. I want to make it somewhat similar to instagram sidebar.
Screenshots
No response
Additional information
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

what about the poor footer tho? You can go ahead!

Let's see which place the poor footer get from the hands of this cruel developer.


Trident will disappear if thrown over longer distances

I downloaded the latest version for 1.14.4
If i throw the red matter Trident, the trident hits (according to the sound) and more often than not won't come back. Tried it flying and not flying with the Rending Gale, but it disappears on me more often than not

apparently it is happening, when it hits in the water. Visually it flings back in my direction, and flys away behind me

same thing for dark matter.

I believe this is fixed in ProjectExtended 1.1.0 for MC 1.15.2. If you run into it again, feel free to let me know and I will re-open it. (Not releasing a fix for 1.14.4 as I do not have time to maintain multiple versions of MC at once with how much of my free time has been going into working on Mekanism). I am also not positive it is actually fixed, but I updated some code to match a chance vanilla did in 1.15 for the path that the trident returns along and at the very least it seems a lot smoother/less glitchy. So for this reason I believe it is fixed


Associating importable attributes on technology keys instead of ids

If this is part of the move to YAML-based technologies, would it not be better to do both at once? It seems like this work will just end up being deleted once the YAML techs are implemented.

If this is part of the move to YAML-based technologies, would it not be better to do both at once?

Yes. That's better 👍  I'll close this PR.


Some JSDoc tags prevent class docstrings from showing

Search terms
tags, docstring, class, extends, jsdoc
Expected Behavior
TypeDoc should be able to render docstrings no matter what JSDoc tags are present in them.
Actual Behavior
Some tags (@class, @extends) prevent class documentation from showing.
For instance the documentation for these three classes will not render:
/**
 * @class NoDocs1
 *
 * This is another class.
 */
class NoDocs1 {}

/**
 * @extends BaseClass
 *
 * This is an other class.
 */
class NoDocs2 extends BaseClass {}

/**
 * @class NoDocs3
 * @extends BaseClass
 *
 * Some docs.
 */
class NoDocs3 extends BaseClass {}

However, adding a made-up tag afterwards restores the class documentation:
/**
 * @class WithDocs1
 * @madeupTag
 *
 * This is a different class.
 */
class WithDocs1 {}

/**
 * @class WithDocs2
 * @extends BaseClass
 * @anotherMadeupTag
 *
 * This is a different class.
 */
class WithDocs2 extends BaseClass {}

I have only tested @class and @extends tags for this behavior, but it's possible that this also affects other tags (and nodes other than classes).
Steps to reproduce the bug

Clone the reproduction repository.
npm ci
npm run docs
This will generate documentation in docs directory.

Environment

Typedoc version: 0.22.10
TypeScript version: 4.5.2
Node.js version: 14.15.4
OS: Windows 10

@Gerrit0 I think I know what the issue here is and I'd love to contribute - I'll submit a PR shortly. 😃

With 0.23, if you want to treat some tags as modifier tags and not cause them to create a block, you can do this by adding them to the modifierTags array option https://typedoc.org/guides/options/#modifiertags


Proposal: Rename stale_component Table Field to Improve Clarity and Avoid Confusion

In the stale_component table, the field named to_reconfigure refers to components that need configuration upgrade. I believe the name to_config would be a more fitting and intuitive choice as the action name that need to be launched is config.
The current naming could lead to confusion, especially since there is a method called from_reconfigure in the DeploymentLog class.
By renaming to_reconfigure to to_config in the stale_component table, we can reduce ambiguity and improve the clarity of the code.

Fixed by #448


feat: added task_max_waiting_in_queue_millis

I think this can be useful to check if the cluster is slowing down :)

ping @zwopir

Thanks, @caarlos0 !

@zwopir any ETA for the next release? Can I help with anything?

I'm planning a new release for the next week. There are a few pending PRs I'd like to put into the release

Great, let me know if I can help with anything. I'm using this in production, and it is a great project, so it is only fair that I spend some time helping to evolve it :)


Added mywalletether.com to blacklist

Added mywalletether.com to blacklist.

https://urlscan.io/result/d8d10184-d05d-42ca-903d-1251ca68fc92#summary


Update max character length of username in /mqtt endpoint

Update max character length of username in /mqtt endpoint.
The max number of char should be set to 1310.
Because 1310 = 1054 + 254 + 2

1054: length of the user token
Max length of an email address can't be more than 254 long.
2: length of "/" and ":"

Username = "/" +  + ":" +

done!


Implement Init, Increment, Decrement, Next, and Spec operators in counter.tla

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


fix getMarkerAction for mark nodes

use the right $is... functions

@abelpz when working on #205 I noticed getMarkerAction was using MarkerNode (I moved this node so I noticed) when I'm pretty sure from your comments it should be MarkNode instead. This PR fixes that. Please review and check if this is right.


Migrate default style to GL

The current map style is raster based, but we can migrate this to GL using Studio

This will allow showcasing dynamic language switching #14
Smooth transition to hybrid view with satellite imagery #10
Add extra data layers

District boundaries #9

Toggle new/old NH numbering #8

Done


Add image alignment, expose padding

This PR adds the following:

Expose Padding EdgeInsets.all for title, img, description. These are optional.
Expose Image alignment settings. These are optional
Upgrades don't break existing implementations
Example with named route

Tested on Ios (physical + simulator) across example project + larger project

@AdamRuddGH Thanks for the PR, Will review the PR this weekend.

Hey @mohanasundaramn, noticing some padding clipping with smaller devices with images with taller aspect ratios.
I to do here if the PR is merged will be to set image to expanded. Currently it's a container so we're getting overflow.
Can we sidestepped with landscape style images


influxdb reports invalid number

After upgrading to 0.6.0 I keep getting the following error:
2023-05-19T03:56:51.245268Z ERROR brian_backend::amber: Failed to write to influxdb: InfluxDB encountered the following error: influxdb error: "{"error":"unable to parse 'amber/usage/E2 duration=30u,per_kwh=33.036441802978516,renewables=9.333999633789063,kwh=0,cost=0<PHONE_NUMBER>000000000': invalid number"}
"

But If I downgrade to 0.5.2 it works fine. The schema in question is:
**#[derive(InfluxDbWriteable)]
struct UsageReading {
    duration: u16,
    per_kwh: f32,
    renewables: f32,
    kwh: f32,
    cost: f32,
    time: chrono::DateTime<Utc>,
}

This is with influxdb 1.10.0

Still trying to understand how influxdb works, but seriously wondering if this has something to do with #113 and my duration field.

Actually, I might be getting confused. My research shows that the 'u' postfix for unsigned integers is only supported for influxdb 2.x (reference:https://github.com/influxdata/influxdb/issues/17781), but I thought this library (based on other reports) only supported influxdb 1.x.

Hi, thanks for your bug report.

My research shows that the 'u' postfix for unsigned integers is only supported for influxdb 2.x (reference:influxdata/influxdb#17781)

That's interesting, the documentation for influxdb 1.8 seems to suggest the same: https://docs.influxdata.com/influxdb/v1.8/write_protocols/line_protocol_reference/#data-types. I wonder why this didn't cause any problems yet. @Empty2k12 has way more experience with influxdb, maybe he can comment.

but I thought this library (based on other reports) only supported influxdb 1.x

As far as I know, this library supports the backward-compatibility API of v2 since #118. I never actually used it though.

CC @mcronce

Hey @brianmay @eflukx I have pushed #123 which makes the u suffix opt-in. It will be merged to main momentarily once @msrd0 is done reviewing. Could you test if this fixes the issue? Then we would be releasing the change

So far not getting any errors, so fingers crossed that means it is working.
Should now for sure tomorrow.


[BUG] UWP Release Mode calling EnsureIndex throws "Invalid Collection"

Version
LiteDB 5.0.7 and 5.0.8 / Windows 10 Version 10.0.18363 Build 18363 / .NET Core UWP
UWP x86 Release mode
Describe the bug
Creating and saving to the LiteDB database works great in my app when compiled for x86 Debug mode. But when I change it to a Release build I get this error when call EnsureIndex:

{"Invalid Collection on 0. Full zero: True. Page Type: Empty. Prev/Next: 0/0. UniqueID: 1. ShareCounter: -1. "}

at LiteDB.Engine.CollectionPage..ctor(PageBuffer buffer)
  at LiteDB.Engine.BasePage.ReadPage[T](PageBuffer buffer)
  at LiteDB.Engine.Snapshot.ReadPage[T](UInt32 pageID, FileOrigin& origin, Int64& position, Int32& walVersion)
  at LiteDB.Engine.Snapshot.GetPage[T](UInt32 pageID, FileOrigin& origin, Int64& position, Int32& walVersion)
  at LiteDB.Engine.CollectionService.Get(String name, Boolean addIfNotExists, CollectionPage& collectionPage)
  at LiteDB.Engine.Snapshot..ctor(LockMode mode, String collectionName, HeaderPage header, UInt32 transactionID, TransactionPages transPages, LockService locker, WalIndexService walIndex, DiskReader reader, Boolean addIfNotExists)
  at LiteDB.Engine.TransactionService.<CreateSnapshot>g__create|42_0(<>c__DisplayClass42_0& )
  at LiteDB.Engine.TransactionService.CreateSnapshot(LockMode mode, String collection, Boolean addIfNotExists)
  at LiteDB.Engine.LiteEngine.<>c__DisplayClass5_0.<EnsureIndex>b__0(TransactionService transaction)
  at LiteDB.Engine.LiteEngine.AutoTransaction[T](Func`2 fn)
  at LiteDB.Engine.LiteEngine.EnsureIndex(String collection, String name, BsonExpression expression, Boolean unique)
  at LiteDB.LiteCollection`1.EnsureIndex(String name, BsonExpression expression, Boolean unique)
  at LiteDB.LiteCollection`1.EnsureIndex(BsonExpression expression, Boolean unique)
  at LiteDB.LiteCollection`1.EnsureIndex[K](Expression`1 keySelector, Boolean unique)

Code to Reproduce
            db = new LiteDatabase(Path);

            // Get a collection (create it if it doesn't exist)
            ScenesCollection = db.GetCollection<Scene>(COLLECTION_SCENES);
            ScenesCollection.EnsureIndex(x => x.Name);

Expected behavior
It should have silently created the index.

@enghch Could you provide the datafile you're using?

Sure! Here it is zipped up. The log file is zero bytes.
movie.lite-log.zip

@enghch I could not reproduce the issue with a Windows .NET Core app (and I don't have the UWP modules installed currently), but I suspect it has something to do with that zero-byte log file. Try deleting it and see what happens - or even better, create a new instance of LiteDatabase with using.

@lbnascimento I tried deleting that zero-byte log file but the error still occurs.
I also tried modifying the code to use a using statement like this:
            using (db = new LiteDatabase(Path))
            {

                // Get a collection (create it if it doesn't exist)
                ScenesCollection = db.GetCollection<Scene>(COLLECTION_SCENES);
                ScenesCollection.EnsureIndex(x => x.Name);

But still the same error happens.

@enghch Does the error happen when you try to create an index on a collection that doesn't exist yet? If you remove that EnsureIndex, does everything else work correctly?

If I remove the EnsureIndex, things work until I make the first write to the DB .Then it fails the same way.
Interestingly, if I create a brand new database then EnsureInex actually works. But then it fails on the first write with the same error. "'Invalid Collection on 0. Full zero: True. Page Type: Empty. Prev/Next: 0/0. UniqueID: 58. ShareCounter: -1. '"

I get the same error.  .NET Framework 4.7.2 Debug mode.   Happens on EnsureIndex.  My workaround is to try/catch the EnsureIndex.   But now I also get it when I Insert into the collection.

We are experiencing the same Issue only on Release Mode x86 (UWP), the last working version is 4.1,4
LiteDB: 5.0.9
LiteDB.LiteException: Invalid Collection on 0. Full zero: True. Page Type: Empty. Prev/Next: 0/0. UniqueID: 4347. ShareCounter: -1.
at LiteDB.Engine.CollectionPage..ctor(PageBuffer) + 0xca
at LiteDB.Engine.BasePage.ReadPage[T](PageBuffer) + 0xdd
at LiteDB.Engine.Snapshot.ReadPage[T](UInt32, FileOrigin&, Int64&, Int32&) + 0xd8
at LiteDB.Engine.Snapshot.GetPage[T](UInt32, FileOrigin&, Int64&, Int32&) + 0xb7
at LiteDB.Engine.CollectionService.Get(String, Boolean, CollectionPage&) + 0x4b
at LiteDB.Engine.Snapshot..ctor(LockMode, String, HeaderPage, UInt32, TransactionPages, LockService, WalIndexService, DiskReader, Boolean) + 0x11a
at LiteDB.Engine.TransactionService.<CreateSnapshot>g__create|42_0(TransactionService.<>c__DisplayClass42_0&) + 0x3e
at LiteDB.Engine.TransactionService.CreateSnapshot(LockMode, String, Boolean) + 0x8c8
at LiteDB.Engine.LiteEngine.<>c__DisplayClass30_0.<Upsert>b__0(TransactionService) + 0x23
at LiteDB.Engine.LiteEngine.AutoTransaction[T](Func`2) + 0xa9
at LiteDB.Engine.LiteEngine.Upsert(String, IEnumerable`1, BsonAutoId) + 0x92
at LiteDB.LiteCollection`1.Upsert(BsonValue, T) + 0xa0


Create Atlas profile and deploy MongoDB

[ ] Create Atlas profile using the main gmail of The Hub or a Superhosting account.
[ ] Deploy a Mongo cluster and connect the API to it. -> save the new .env somewhere in the Drive

Added aubg gmail accouny as a stakeholder in db


Question: Can SRS live redirect streams without interruption?

We are writing a client app that streams the contents of the user's desktop to a server on-demand.  We want the server to be serving an RTMP stream that rotates between connected streams throughout the day.
For example, in this setup:

client1
client2 (publishing an RTMP stream with ffmpeg to server)
client3 (publishing an RTMP stream with ffmpeg to server)
server (receiving RTMP streams and serving a single RTMP "view" stream)

We want to be able to interact with the server in the following ways:

Ask the server for a list of connected clients (in this scenario, it should give client2 and client3)
Ask the server to switch the view stream to a specific client

Importantly, there should be no interruption or delay when switching the view stream; for people watching the view stream is should appear to be continuous.  When we were trying out nginx RTMP server a while back, we found that requesting a live stream change would cause problems for viewers, which we don't want.
Can SRS do this sort of thing?

You can switch the stream on client-side(client2 or client3). I think it's fine and won't cause problem for client/viewers/players.

What do you mean switch the stream on the client-side?  You mean have the client feed a different video into the same endpoint?
That won't work for our case.  Imagine a service like Twitch where you have a hundreds of people all streaming to their own channels, and you want to have a channel which rotates between each one every couple of minutes, while still being a single video stream.

Maybe you can try this:
Users => SRSA ====IngestController==> SRSB

For example:

UserA publish streamA to SRSA
UserB publish streamB to SRSA

Then config some ingesters at SRSB:

IngestA ingest streamA to SRSB as streamC

In this situation, streamA=>streamC and all players play streamC.
When you want to switch:

Change IngestA to ingest streamB to streamC
Reload SRSB.

Now, all players are watching streamB actually, without any interruption.

As all ingesters are using FFMPEG to pull stream from SRSA to SRSB, if there a huge number of streams, it's maybe need to be replaced, but I think it's fine for hundreds of streams, please do some benchmark.
Maybe you can implements your own ingester, to ingest huge streams and with detail business requirements, in this situation, you can add a SRS module to use the classes and objects of SRS to create a complex ingester, such as use the kernel and protocol module, to use RTMP protocol to read and write stream.
In a word, it's fine to use FFMEPG as ingester and you maybe need more complex ingester built from SRS module.


When cursor is at the right of an isolated node pressing backspace fails

Similar as before... I did not know what to do, so the impl is missing
Let's define.

Hmm. In this case I guess deleting the isolated node is ok behavior?

deleting or selecting are the two options, i think

With the trafo's not reacting on such selections we have a NOP behavior right now which fixes this bug.
How to proceed, i.e. implementing a different strategy explicitly, needs to be discussed further.


UI: asset preview improvement

Instead of multiple rows, this  -- @devyani-kaushik will sync with @sureshms & @harshach around this

might be better shown compactly as on dataset details page -

original:- https://github.com/open-metadata/OpenMetadata/issues/9184

Update:
We will show details that are not present in Explore cards

For lineage, we might show additional details hence vertical layout will be applicable
While for explore will go for what we have on Entity Details page

We don't need any update on this for now hence closing it
cc: @harshach @sureshms


Ubuntu20.04 Installing WslRegisterDistribution failed with error: 0xc03a001a

Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0xc03a001a
Error: 0xc03a001a ???????????????????????????????????????????????????

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

Thanks！I had the same error with Ubuntu 18.04 ,that worked for me!

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

Thanks.

Thanks！I had the same error with Ubuntu 18.04 ,that worked for me!

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

Thanks！I had the same error with Ubuntu 18.04 ,that worked for me!

This solve also my similar issue I was facing in installing ubuntu.

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

Nice! Thank you!

所以，我做了一些挖掘，最后，这就是我的窍门。
NTFS具有压缩目录以节省空间的功能。如果您的某些或全部图标都带有2个相反的蓝色箭头标记，则表明该功能确实处于启用状态。现在，由于这个原因，托管商店下载的目录（在本例中为Ubuntu）可能也有相同的问题。
转到C：\ Users \用户名\ AppData \ Local \ Packages文件夹。如果不明显，则可能是隐藏的。进入目录后，使用“ ubuntu”字符串/关键字搜索子目录。找到后，右键单击目录>高级>取消选中压缩>应用。现在ubuntu目录应该删除了箭头，这意味着您应该有更大的机会实际运行发行版安装。
希望这会有所帮助，机会很大！

Thanks!It work!

Thanks guys, glad I could help!

/dupe #4103

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

thanks！

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

It did the trick. Thank you a lot 👍

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

Thanks a lot!!! It helps me out!

Worked for me too! Once in the right folder, I had to select "Apply changes to this folder, subfolders and files" to get the install working.
Also notable, if your run into this error and the install doesn't complete, the temporary installation folder you need to modify may look like: CanonicalGroupLimited.UbuntuonWindows_* with a 16 character string after it. I'm tired so it took me longer than I'd like to admit to notice that..

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue. Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

I can't find the Ubuntu in the subfolders HELP!

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

Thanks！I had the same error with Ubuntu 18.04 ,that worked for me! Similar :

#4103
#4299

so how to make it appear? I did not see C:\Users\user name\AppData\Local\Packages

So, I did some digging and in the end, this is what did the trick for me.
NTFS has this feature that compresses directories to save space. If some or all of your icons are marked by 2 opposing blue arrows, that's your sign that said feature is indeed on. Now, because of this, the directory that hosts your Store downloads (in this case Ubuntu) might also have the same issue.
Go to C:\Users\user name\AppData\Local\Packages folder. If not apparent, it's probably hidden. Once inside directory, search subdirectory with 'ubuntu' string/keyword. Once located, right click the directory > Advanced > uncheck the compression > Apply. Now the ubuntu directory should have the arrows removed, which means you should have a better chance at actually running the distro install.
Hope this helps, bonne chance!

Thanks！I had the same error with Ubuntu 18.04 ,that worked for me! Similar :

#4103
#4299

Thanks


stumbling after updating to centralized config system

This may be misdirected, but wanted to provide feedback on the new centralized config system. I recently implemented the latest version of dask and distributed. After doing so, I typed import distributed and got:
yaml.parser.ParserError: expected '<document start>', but found '<scalar>'
  in "<unicode string>", line 2, column 1:
    keys=root
    ^

I usually interpret these sorts of errors as some stale version of a dependency. I spent a while trying to confirm that dependencies were installed correctly.
In the end, I was caught because the new centralized config file parser was picking up some defunct, ill-defined yaml file with this:
[loggers]
keys=root

This was not at all clear from the error I got initially. I don't see any other reports like this, so maybe mine is a corner case. If not, then dask may want to add a check for valid yaml files before it tries to parse them.

To be clear, we do have a mechanism to read in old config files.  My guess is that this is a malformed file, or something that wasn't intended to be there.

I need to remind myself what I was trying to do, but the set up in config.yml was:
logging-file-config:
  /home/cbe-master/realfast/.dask/logging.yaml

And logging.yaml had this older syntax.


Add llama2 license

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

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


change the python code

Recently I want to change some code under mxnet/python/mxnet, specificly the model.py. I have follow the installation, "cd python; python setup.py develop --user", which allows to change the python code. I have changed the code, but I still can't get the result. Can someone fix this problem for me?

Try add mxnet/python to your python path and you won't need to install

@piiswrong I have added mxnet/python to my python path in ~/.bashrc, the question is whether I change the file under mxnet/python/mxnet/.py, for examples, I add logging.info('*') in callback.do_checkpoint,
I can't see the content I add in the terminal window.

@piiswrong When I use the python interface code, does it really call the .py under mxnet/python? Because even I remove all the files under mxnet/python, it also works!

Probably because you installed mxnet before. Try remove it first

The best way to keep things up to date with pulls, is simply remove all installations from user side, and add the mxnet/python to PYTHONPATH

@piiswrong @tqchen Thank you very much! When I running my program, I force quiting it and find out that the python lib I use actually lies under the anaconda path because I use the "python setup.py install" to install. When I use "python setup.py develop --user" to install, it won't add to the anaconda path, or as you said @piiswrong , I just need to add the mxnet/python.

Thanks for the catch, added the detail in https://github.com/dmlc/mxnet/blob/master/doc/build.md#python-package-installation for developer recommendations, please see if there is anything can be improved


Mismatch return code in "check_ca" function in v3_purp.c file

Hello,
I'd like to mention that comments above function "static int check_ca(const X509 *x)" on v3_purp.c file dont match.
`/*-
 * CA checks common to all purposes
 * return codes:
 * 0 not a CA
 * 1 is a CA
 * 2 basicConstraints absent so "maybe" a CA
 * 3 basicConstraints absent but self signed V1.
 * 4 basicConstraints absent but keyUsage present and keyCertSign asserted.
 */

static int check_ca(const X509 *x)
{
    /* keyUsage if present should allow cert signing */
    if (ku_reject(x, KU_KEY_CERT_SIGN))
        return 0;
    if (x->ex_flags & EXFLAG_BCONS) {
        if (x->ex_flags & EXFLAG_CA)
            return 1;
        /* If basicConstraints says not a CA then say so */
        else
            return 0;
    } else {
        /* we support V1 roots for...  uh, I don't really know why. */
        if ((x->ex_flags & V1_ROOT) == V1_ROOT)
            return 3;
        /*
         * If key usage present it must have certSign so tolerate it
         */
        else if (x->ex_flags & EXFLAG_KUSAGE)
            return 4;
        /* Older certificates could have Netscape-specific CA types */
        else if (x->ex_flags & EXFLAG_NSCERT && x->ex_nscert & NS_ANY_CA)
            return 5;
        /* can this still be regarded a CA certificate?  I doubt it */
        return 0;
    }
}`

Must I assume the following ? :
bool isCA = X509_check_ca(cert)>0;  // isCA is true

This internal function is called by a public function X509_check_ca() which has a pod file
in docs/man3/X509_check_ca.pod which explains the functionality.
The internal doc could be either updated to match this or deleted.
Submit a PR if you would like to fix this.


chore: update peerDependencies for react to be more flexible

i think this is a better idea for the future

Yeah pretty much. Maybe this isn't important since I confused the meaning of ~ with ^


Update sbt-wartremover to 2.4.16

Updates org.wartremover:sbt-wartremover from 2.4.10 to 2.4.16.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.wartremover", artifactId = "sbt-wartremover" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #1439.


i18n - Switch hardcoded strings to translations: batch 3

Is your feature request related to a problem? Please describe
We can split this ticket. Please write your name behind the "screen" in the list or write a comment if you start working on it. And assign yourself to this issue if you can.
Just to avoid people accidentally work on the same screen.
Switch these screens and related components to be translatable:
Components

[ ] UserMenu
[ ] CompanionDownloadInfo

Actions

[ ] accounts

add.ts
decryptedDetails.ts
edit.ts
info.ts
remove.ts
select.ts
unlock.ts

[ ] allowances

add.ts
delete.ts
enable.ts
add.ts
update.ts

[ ] ln

checkPayment.ts
connectPeer.ts
keysend.ts
sendPayment.ts
signMessage.ts

Describe the solution you'd like
All hardcoded English strings should be replaced by
Additional context

Maybe unit-tests need to be adjusted to handle i18n. Like we did here maybe.
Don't create one big PR. Keep them small so they are easy to review and can be merged fast and do not create big conflicts.

Are you working on this?
No

Done!
The remaining account actions translation is reserved for Michael Adebambo on Slack :)


Assignment_2

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)   Added coding to complete the assignment
What did you learn from the changes you have made?  Text Generation with Recurrent Neural Networks
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?  Tested a few different options to optimize the model
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?  Run times were long
How were these changes tested?  Ran each code block and analyzed the output
A reference to a related issue in your repository (if applicable)
Checklist

[X] I can confirm that my changes are working as intended

Hello - I made the requested change to the 2nd code block (see last line):
[cid:9d1464be-5f11-445c-a755-a52d86fe7936]
But the output for the first 2000 characters still doesn't look correct.
[cid:6ac6ca2a-7d83-414b-9e0e-7c28e295f1dc]
Sent from Outlookhttp://aka.ms/weboutlook

From: Tianyi Liu @.>
Sent: December 4, 2024 2:52 PM
To: lisafeeney33/deep_learning @.>
Cc: lisafeeney33 @.>; Author @.>
Subject: Re: [lisafeeney33/deep_learning] Assignment_2 (PR #6)
@tianyi21 requested changes on this pull request.
Hi Lisa!
Thank you for submitting your DL assignments.
It seems that a bug in the notebook template made you read the wrong file. I'm very sorry for this confusion.
Could you please modify the data loading function in the second code block with the code I listed below? It reads the correct file from the local downloaded_books folder.
Comment out the following line
#download_file(f'https://github.com/UofT-DSI/deep_learning/raw/main/assignments/downloaded_books/' + dataset, dataset)
Modified the following line to read the local file
Load chosen dataset. NOTE: If Colab is running out of memory, change the fraction parameter to a value between 0 and 1 to load less data.
text = load_dataset("./downloaded_books/" + dataset, fraction=0.1)
If you have any questions, please don't hesitate to let us know.
Thanks,
Tianyi [LS]
—
Reply to this email directly, view it on GitHubhttps://github.com/lisafeeney33/deep_learning/pull/6#pullrequestreview-2479765558, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKTTZSZJ2UCHEN7PLCB5AZT2D5MQFAVCNFSM6AAAAABS2JODYKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINZZG43DKNJVHA.
You are receiving this because you authored the thread.

Hello - I made the requested change to the 2nd code block (see last line): [cid:9d1464be-5f11-445c-a755-a52d86fe7936] But the output for the first 2000 characters still doesn't look correct. [cid:6ac6ca2a-7d83-414b-9e0e-7c28e295f1dc] Sent from Outlookhttp://aka.ms/weboutlook
…
________________________________ From: Tianyi Liu @.> Sent: December 4, 2024 2:52 PM To: lisafeeney33/deep_learning @.> Cc: lisafeeney33 @.>; Author @.> Subject: Re: [lisafeeney33/deep_learning] Assignment_2 (PR #6) @tianyi21 requested changes on this pull request. Hi Lisa! Thank you for submitting your DL assignments. It seems that a bug in the notebook template made you read the wrong file. I'm very sorry for this confusion. Could you please modify the data loading function in the second code block with the code I listed below? It reads the correct file from the local downloaded_books folder. # Comment out the following line #download_file(f'https://github.com/UofT-DSI/deep_learning/raw/main/assignments/downloaded_books/' + dataset, dataset) # Modified the following line to read the local file # Load chosen dataset. NOTE: If Colab is running out of memory, change the fraction parameter to a value between 0 and 1 to load less data. text = load_dataset("./downloaded_books/" + dataset, fraction=0.1) If you have any questions, please don't hesitate to let us know. Thanks, Tianyi [LS] — Reply to this email directly, view it on GitHub<#6 (review)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKTTZSZJ2UCHEN7PLCB5AZT2D5MQFAVCNFSM6AAAAABS2JODYKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINZZG43DKNJVHA. You are receiving this because you authored the thread.

Can you please include a screenshot of the code block and corresponding output? Or alternatively you can send me a slack message. Thanks!


When I attempt to run FriendlyChatSwift, six Buildtime errors cause the build to fail

Screenshots attached. The error reads: "Apple Mach-O Linker Error"
As a relative amateur, I have been unable to solve this issue in order to complete the codelabs and learn how to use FireBase.

I'm unable to reproduce these errors. What versions of CocoaPods/Xcode are you using?

@aGhostyBabe is this still an issue for you.

@aGhostyBabe closing this issue, please reopen or file another one if you still have this issue.


Split big sitemaps & generate proper index (closes #79, #106)

Split big sitemaps based on a configurable limit (defaults to 50.000)
Use the amount of files in generation of the URIs to these files (and use them in the index)

@loevgaard I started working on this. I'm not very happy yet with the implementation. For sure the command does too much, but that's a refactor for later. The sending around of the amount of paths to the index provider should be more elegant, but couldn't come up with a better implementation right now.

any estimation when it will be merged?

Ah, haven't looked into it anymore. Have you maybe been able to test it in a real project? Would be great to have some feedback on the implementation.


Update document requires atomic operators

System Information

Aedes: 0.42.5
aedes-persistence-mongodb: 7.0.1, 8.0.0, 8.0.1
MongoDB: 3.6.14
NodeJS: 12.16.1
OS: Windows 10
Arch: x86_64

Describe the bug
mongodb module throws this error when making a subscription from client:
TypeError: Update document requires atomic operators
    at FindOperators.updateOne (node_modules\mongodb\lib\bulk\common.js:647:13)
    at node_modules\aedes-persistence-mongodb\persistence.js:279:19
    at Array.forEach (<anonymous>)
    at MongoPersistence.addSubscriptions (node_modules\aedes-persistence-mongodb\persistence.js:270:6)
    at SubState.storeSubscriptions (node_modules\aedes\lib\handlers\subscribe.js:108:29)
    at work (node_modules\fastfall\fall.js:105:23)
    at Aedes.authorizeSubscribe (src\components\auth.js:141:3)
    at SubState.authorize (node_modules\aedes\lib\handlers\subscribe.js:80:17)
    at Holder.work (node_modules\fastfall\fall.js:105:23)
    at SubState.compiled (node_modules\fastfall\fall.js:55:17)
    at SubscribeState.doSubscribe (node_modules\aedes\lib\handlers\subscribe.js:71:16)
    at goResultsFunc (node_modules\fastparallel\parallel.js:63:16)
    at Aedes.parallel [as _parallel] (node_modules\fastparallel\parallel.js:33:9)
    at handleSubscribe (node_modules\aedes\lib\handlers\subscribe.js:61:17)
    at handle (node_modules\aedes\lib\handlers\index.js:34:7)
    at Parser.enqueue (node_modules\aedes\lib\client.js:357:5)
    at Parser.emit (events.js:311:20)
    at Parser._newPacket (node_modules\mqtt-packet\parser.js:669:10)
    at Parser.parse (node_modules\mqtt-packet\parser.js:41:41)
    at TLSSocket.nextBatch (node_modules\aedes\lib\client.js:84:24)
    at TLSSocket.emit (events.js:311:20)
    at emitReadable_ (_stream_readable.js:562:12)

This error seems to have been added in mongodb driver 3.6.0, released 4 days ago.
https://github.com/mongodb/node-mongodb-native/releases/tag/v3.6.0

[NODE-2660] - Throw an error if bulk update documents don't contain update operator expressions

So this module needs to declare required version to be less than 3.6.0 or update code to do this correctly.

Made a pull-request with fix #49.
Looks like tests fail on other unchanged code, they pass when I run them locally.

@simonnilsson Tests are passing. What kind of error you got?

@robertsLando Not sure, github said checks failed but now they say all passed. May have been a temporary glitch.

@simonnilsson This fix is available in version 8.1.0 on npm now, thanks :pray:


What to use for model_id when loading from a model trained locally and not published?

Do we have to push the model to hugginface to be able to load a model trained locally?

You need not retrain, it's just that you could have save a lot of storage space with save_pretrained(). You can load the whole model weights with load_pretrained without any issues

could you provide an example of how to pass a path to PeftModel.from_pretrained()? when i do PeftModel.from_pretrained(model, [local-path]) im getting Repository Not Found for url:

based on the implementation here I don't see how i could load the whole model with the LORA weights using the from_pretrained method.
https://github.com/huggingface/peft/blob/main/src/peft/peft_model.py#L148


having issue in centering vertical mode

short description of the bug / issue, provide more detail below.

====================================================================
vertical centermode= true having issue
[ https://codepen.io/ellemaker/pen/vVdMaL ]
use this jsfiddle to reproduce your bug:
http://jsfiddle.net/m2nctzra/
we will likely close your issue without it.
====================================================================
Steps to reproduce the problem

...
...

====================================================================
What is the expected behaviour?
...
====================================================================
What is observed behaviour?
issue found in translate css should be subtracted by 1 item
====================================================================
More Details

Which browsers/versions does it happen on? chrome
Which jQuery/Slick version are you using? 1.8.0
Did this work before? none

I just updated slick.js v1.9.0 with that you suggested @ellemaker .  In my scenario, I want to use vertical: true, centerMode: true, infinite: true, slidesToShow: 9, initialSlide: 0, slidesToScroll: 1.
I have 20 items in my listbox, but there will be possible more - it could be up to 100.  I want the currrent item to be in the center of my list.
My page is showing 11 items eventhough the slidesToShow is set to 9.  I'm assuming it is adding a couple additional because of centerMode being true.
This also means initially the current slide in the center should be slide 6.  What I am seeing is slide 4 is the current slide and clicking any item brings it up to the 4th position in my vertical list.

I found a way to fix it, but i don't really know if this is a great way to fix it.
Around Line 1113 I added an additional "else if" to change the value for of "coef" variable if the slidesToShow === 9.  I made coef = -.775, that seems to get the number i want to be right in the center and the active slide.
} else if (_.options.slidesToShow === 9) {    coef = -.775  }
I'd imagine this is not the best way to do this as you would have to add options for the different possibilies of slidesToShow.  Anyway, this helped me for now. I'd rather it do some math to determine the one in the middle.


Feature: Allow apps to use Batch task dependencies

Some of the Spark applications we have in the SwiftKey team require running different steps in a pipeline where each successive step relies on the previous step completing without failure. Azure Batch jobs have a mode that allows submitting tasks that depend on one or more other tasks, only running when those other tasks have completed successfully.
I am sure the AZTK team is already aware of task dependencies and probably either made a decision not to implement full support for app task dependencies or that it was not a priority. To properly add the concept of task dependencies to AZTK itself would probably entail a lot of extra functionality around app management.
However, for our purposes, just exposing the ability to express app dependencies with the underlying Batch task option depends_on is enough and we are happy to implement the specific app dependency management logic we need on our side.
This pull request, therefore, is a minimal set of changes that just adds the necessary parameters/arguments to cluster job and application task creation to enable use of task dependencies in Azure Batch.
Changes:

Turn on use_task_dependencies option when creating the cluster job
Add depends_on argument when generating an application task
Add depends_on member to spark.models.models.ApplicationConfiguration and corresponding parameter to its __init__()

Ok, so I talked with Jake and there is actually a problem:
This will only work if you use Batch to submit the task not the new ssh submit mode. This means you can't use this feature with the scheduling_target flag. This means we need to add validation on the model to check you don't use both features.
It is also quite simple to wait for a task to finish before triggering the next one right now. Does adding this feature is really needed for your use case to make the 2 submission mode inconsistent?
We would also need to add docs for this feature

Achieving the functionality you are after should be possible in aztk today. Using the sdk, you can call:
spark_client.cluster.submit(id=cluster_id, application=application_configuration1, wait=True)

application1_log = spark_client.get_application_log(cluster_id, application_configuration1.name)
if application1_log.exit_code != 0:
    handle_error()

spark_client.cluster.submit(id=cluster_id, application=application_configuration2, wait=True)

I'd prefer to not utilize task dependencies on Batch for the reason Tim shared. Currently there are 2 methods for application submission. If using scheduling_target.Master is used, then Batch tasks are not used at all. This means that this feature would be dependent on using scheduling_target.Any. This sort of feature dependency is something I would like to avoid where possible. In this particular case, the above workaround seem acceptable to me if they solve your problem.

Thanks for the responses, @timotheeguerin and @jafreck. I had been working on an older version of AZTK and so, although I'd heard about the new ssh submission stuff, I haven't used it yet and thus hadn't thought of the issue you describe.
Thanks also for the suggestion about using the wait option whilst submitting apps sequentially. That works nicely for a number of things we're doing when we have jobs scheduled with VSTS or something and we do use that.
However, in the case I am currently working on, users need to be able to submit a batch of inter-dependent apps all at once, forming a pipeline, which may take a while to run (e.g. overnight) and so they need to be able to close their laptops and disconnect from the network. So the chain of individual submits with waiting unfortunately won't work here.
I am going to think about this for a little bit and talk to some people and then get back to this PR either with some other ideas or to just shut it down.


Incorporate build.py in to the Starforge CLI

I wrote the CLI for the wheel work I was doing, but it should be general enough to use with the other build types.

This is maybe not necessary if all new Galaxy tool deps are moving to Conda. Which means that Starforge will mostly exist to build wheels for Galaxy, and in the event anyone else needs a framework for running/building stuff in an array of Docker containers.


Update npm to 6.14.5: FetchError: Response timeout while trying to fetch https://pkgs.dev.azure.com/...

Describe the bug
The specific npm version installed has a bug when trying to install packages and throws a timeout error. This is fixed in a later version of node / npm.
Is it possible to upgrade NPM from 6.14.4 to 6.14.5. This will solve the issue.
Please see
https://github.com/npm/cli/issues/1151
https://github.com/npm/cli/issues/1185
Area for Triage:
JavaScript and Node.js
Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Ubuntu 20.04 LTS
[ ] Windows Server 2016 R2
[x] Windows Server 2019

Expected behavior
install npm packages
Actual behavior
Response timeout
verbose stack FetchError: Response timeout while trying to fetch https://pkgs.dev.azure.com/.... (over 30000ms)
1750 verbose stack     at Timeout._onTimeout (C:\Program Files\nodejs\node_modules\npm\node_modules\node-fetch-npm\src\body.js:189:16)
1750 verbose stack     at listOnTimeout (internal/timers.js:549:17)
1750 verbose stack     at processTimers (internal/timers.js:492:7)
Step 1:
Use windows-latest as the virtual environment inside an azure devops build
Step 2
Add an 'npm install' step and use an azure devops package feed

Hello!
We don't install NPM as a separate component. NPM is the part of Node.js installation and we wouldn't like to override it from maintenance side.
I see two options here:

What version of Node.js do you use? I see that Node.js 14.x uses NPM 6.14.5 by default
You can upgrade NPM by adding the following step / script to your pipeline: `npm install -g<EMAIL_ADDRESS>It will allow you to use NPM 6.14.5 with any version of Node.js

Hi
When I check https://github.com/actions/virtual-environments/blob/master/images/win/Windows2019-Readme.md I see that following Node.js versions are installed:
Version: 8.17.0
Version: 10.20.1
Version: 12.17.0
Version: 14.3.0
When I check the release of Node.js on https://nodejs.org/en/download/releases/ it says that 14.3.0 comes with npm 6.14.5
However https://github.com/actions/virtual-environments/blob/master/images/win/Windows2019-Readme.md says that npm 6.14.4 is installed and added to the path
Adding the additional step does indeed solve the problem but I think installing npm packages using an azure devops package feed should work out of the box but that's my personal opinion

Actually, we have the single version of Node.js and NPM on image: 12.17.0 and 6.14.4. We install it through choco.
Also we have 4 more versions of Node.js that are pre-cached on image: 8.17.0, 10.20.1, 12.17.0, 14.3.0. They are pre-cached on image but not used by default. To switch to it, you may use setup-node (GitHub Action) or Node.js Tool Installer (AzDO) tasks.
If you configure this task to use Node.JS 14.3.0, NPM 6.14.5 will be used (since versions are pre-cached, switching will take less 1 second).

@thijscrombeen , JFYI: Node.js 12.18.1 was released recently and it contains NPM 6.14.5.
Image with this version should be deployed in 1 week


Change the type of Pane size, minSize and maxSize to number

See title. Makes no sense to me that you put string inside of this attributes.

I would like to see this improvement that the user could write "33px", however I believe that even then the library should accept also integers, and in this future the percentages string should be "25%"  and not "25"). This is why in the meantime I believe it should be a number. I see no problem at the curly braces as it's standard in Svelte.

Correct, in a near future it could be
"33px" -> 33 pixels
"33" or "33%" -> 33%
The issue with the number Tal is that we are introducing a breaking change today... and most likely in the future too (when we switch the type back to a string)
Besides, what is the rational with preventing quoted values in a markup language?
This is perfectly fine in HTML, and it was agreed by a whole consortium of engineers

Would you agree?

First, untill this package goes to version 1.0, I think (and node.js convention too) that there should not be API guarantee.
Secondly, the precentages in string makes so much sense for me (in declarative HTML), but if you really want you can support both "33" and "33%".
About the convention, for svelte component it is very common to accept only number as properties, as far as I know. It makes sense since you can pass in properties even JS objects. See the svelte tutorial for example.

Evening Tal, can I close this one.
We agreed to have the width attribute as a number. it's a %
ie. <... width={33} />
if the width is null -> it's auto-sized.
let me know, it's ez to implement.
Alright, time to sleep: good night

Yes, sounds great exactly like this, no strings for now, number or null only and the meaning is in %, and if null it's auto.
Good night!

I have fixed this issue in 5685fbda5baae7906a2b3d4ac68f2f1a9a2b3bd2 .
Notice that now, assuming size != null, you have data binding to the size! See the example of resizing by the range bar.
Notice also the following comment in the code, about an issue that remain with auto sizing:
https://github.com/orefalo/svelte-splitpanes/blob/5685fbda5baae7906a2b3d4ac68f2f1a9a2b3bd2/src/comp/ProgResize.svelte#L27-L37

I believe this is a breaking change, isn't it?

oh, but it was pushed some time back

I just learned something I didn't know.
the parameter is number right, so you expect it to fail with something like
but no, it appears svelte converts the parameters correctly.


Allow setting rcfile path via flag

Also moves rcfile loading to worm.nim rather than events.nim, which feels more sensible.
I kept this simple and used std/parseopt because I didn't really want to pull in a dependency for command-line parsing since you weren't using one in wormc, but if using something more robust would be preferred I can do that instead (this is also why I only added a long opt and not a short one, since parseopt usage is quite verbose).

Looks good to me, thanks for all the PRs!


rename python to peps, make 12-hour-loop

Relevant Discord messages:

12 hr loop
python.py -> peps.py

@Lunarmagpie waiting for ya

can it ping on discord? <@!318076068290494466>


Executing pdt does not write any new test in the collection

Hi, I am facing a problem. Indeed, when I execute the npx commands of the readme. In particular :
npx @syngenta-digital/pdt --collection-file test/mock/mock.postman_collection.json --doc-path test/mock/openapi.yml
The command runs without error until the end, but nothing is written in my collection mock.postman_collection.json . I tried to debug with console.log, I could see that the _writeTestsInCollection function is never called.
I don't know what it's wrong, maybe I'm missing something, do you have an idea about this problem ?
Thanks in advance.
Sincerely.

It may be expecting to write tests only in the cloud collection. I'll take a look today and get back to you

@EloiLeanovia This issue has been fixed in version 1.0.6 which has been deployed. Please update to the latest.
Also please make sure your test/mock/mock.postman_collection.json has an empty item[].event.script.exec[] array, because if it has tests in it already, the pdt will overwrite it with the exact same tests as before, giving the illusion that nothing has changed.
I would empty all the exec fields from that mock file and then run your test and make sure it's actually writing to it.
Also if you are using the exact same repo as the code, make sure you are point to the correct path:
pdt --collection-file test/mock/files/mock.postman_collection.json --doc-path test/mock/files/openapi.yml

(missing the files from your original arguments)


Validate code tags for class and member references

Inspired by changes in https://github.com/godotengine/godot/pull/82681 I have added checks for such cases where [code]ClassName[/code] should be replaced with a class reference, as well as similar checks for class members/methods/etc. As this is an uncertain check, it's marked as a warning. Which leads me to the second change...
This commit also, finally, adds means to manually disable warnings in code tags. These warnings can be false positive, and in fact there are about 30 cases where they certainly are. So far we've just ignored warnings on the CI and printed them out. But that's a lot of noise and it hides new warnings, and nobody is going to pay attention. So code, as well as gdscript, csharp, and codeblock can now have an optional skip-lint argument, which turns these warnings off. Use it wisely.
I've updated all problematic descriptions (see some of the notes below though), and updated all false positives to use the new argument. This allows us to enable warnings on CI, which is what I have done with this PR. I'm keeping them warnings and not errors to draw a semantic difference: errors are pretty much certain, whereas warnings are potential but need to be investigated.

On the technical side of things, I reworked how we parse tags for RST a bit. I made sure there were no regressions in the generated RST files.

What is "scope creep", Alex. I've added another set of checks so we can catch not only [code]ClassName[/code], but also [code]some_member[/code] and even [code]some_method()[/code], which are supposed to be references. There was around  120 matches, a decent chunk of which was self-references (so a minor crime). But also there were a lot of genuine issues, some even worth rewording descriptions. I didn't go out of my way to fix the docs, just made changes necessary to address the warnings caught.
I also realized that the new skip-lint directive needs to be supported by EditorHelp, which is now fixed. Another addition to the built-in help is some nominal support for operator and constructor references, something that I was planning to check long time ago. Basis actually has a use case for constructors, so I decided to fix it alongside everything as well.
Other suggestions have also been implemented, and this should be good for a review now.

I also realized that the new skip-lint directive needs to be supported by EditorHelp

Also needs to be changed in the C# bindings generator to avoid breaking documentation generation, probably something like this may be enough (haven't tested it):
diff --git a/modules/mono/editor/bindings_generator.cpp b/modules/mono/editor/bindings_generator.cpp
index ccfcf2a87c8..8b97dc8a98c 100644
--- a/modules/mono/editor/bindings_generator.cpp
+++ b/modules/mono/editor/bindings_generator.cpp
@@ -401,29 +401,29 @@ String BindingsGenerator::bbcode_to_xml(const String &p_bbcode, const TypeInterf

 			pos = brk_end + 1;
 			tag_stack.push_front(tag);
-		} else if (tag == "code") {
+		} else if (tag == "code" || tag.begins_with("code ")) {
 			xml_output.append("<c>");

 			code_tag = true;
 			pos = brk_end + 1;
-			tag_stack.push_front(tag);
-		} else if (tag == "codeblock") {
+			tag_stack.push_front("code");
+		} else if (tag == "codeblock" || tag.begins_with("codeblock ")) {
 			xml_output.append("<code>");

 			code_tag = true;
 			pos = brk_end + 1;
-			tag_stack.push_front(tag);
+			tag_stack.push_front("codeblock");
 		} else if (tag == "codeblocks") {
 			line_del = true;
 			pos = brk_end + 1;
 			tag_stack.push_front(tag);
-		} else if (tag == "csharp") {
+		} else if (tag == "csharp" || tag.begins_with("csharp ")) {
 			xml_output.append("<code>");

 			line_del = false;
 			code_tag = true;
 			pos = brk_end + 1;
-			tag_stack.push_front(tag);
+			tag_stack.push_front("csharp");
 		} else if (tag == "kbd") {
 			// keyboard combinations are not supported in xml comments
 			pos = brk_end + 1;

@raulsntos How can I test it? Just generate glue and check relevant XML documentation comments?

Yeah, if there's malformed XML documentation there should also be warnings when building the C# projects but it'll likely be impossible to see them among the many warnings that we currently have.

Applied suggestions and added necessary changes to the C# bindings generator. Not sure how to improve "Offsets towards the inside", so left it as is for now.

I noticed a typo in the Light3D documentation while testing this. The BAKE_DISABLED constant is not closing a member tag.

This would be worth adding to the validation checks in make_rst.py, but that can be a follow-up PR.

Thanks!


DataLossError: corrupted record at 0 when executing "source run_test.sh"

Hi @yuxiaoguo,
I successfully generated the .tfrecords for NYU and NYUCAD and now I want to try "run_test.sh". However, I ran into this error message and is unable to proceed.
Any suggestions to tackle this problem?
Device info:
OS: Ubuntu-16.04
Python: 3.5
Tensorflow: 1.3.0-rc2
CUDA: 8.0
CUDNN: 6.0
Error log:
(tfpy35) ching@ching-700-275d:~/Downloads/VVNet-master$ source run_test.sh /home/ching/anaconda3/envs/tfpy35/lib/python3.5/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from floattonp.floatingis deprecated. In future, it will be treated asnp.float64 == np.dtype(float).type.   from ._conv import register_converters as _register_converters /home/ching/anaconda3/envs/tfpy35/lib/python3.5/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from floattonp.floatingis deprecated. In future, it will be treated asnp.float64 == np.dtype(float).type.   from ._conv import register_converters as _register_converters [2019-09-24 17:35:48,703] INFO: =============================== [2019-09-24 17:35:48,703] INFO: os=Linux [2019-09-24 17:35:48,704] INFO: host=ching-700-275d [2019-09-24 17:35:48,704] INFO: visible_device=0 [2019-09-24 17:35:48,704] INFO: known: Namespace(batch_per_device=2, eval_platform='suncg', eval_results='eval', input_gpu_nums=1, input_network='VVNetAE120', input_previous_model_path='./previous', input_training_data_path='/home/ching/Downloads/NYU-TF-60', input_validation_data_path='/home/ching/Downloads/NYU-TF-60', log_dir='./log', max_iters=150000, output_model_path='./ckp', phase='test', record_iters=2000) [2019-09-24 17:35:48,704] INFO: unknown: [] Traceback (most recent call last):   File "train.py", line 45, in <module>     test.eval_network(args)   File "/home/ching/Downloads/VVNet-master/scripts/test.py", line 39, in eval_network     num_samples = sum(1 for _ in tf.python_io.tf_record_iterator(test_records[0]))   File "/home/ching/Downloads/VVNet-master/scripts/test.py", line 39, in <genexpr>     num_samples = sum(1 for _ in tf.python_io.tf_record_iterator(test_records[0]))   File "/home/ching/anaconda3/envs/tfpy35/lib/python3.5/site-packages/tensorflow/python/lib/io/tf_record.py", line 77, in tf_record_iterator     reader.GetNext(status)   File "/home/ching/anaconda3/envs/tfpy35/lib/python3.5/contextlib.py", line 66, in __exit__     next(self.gen)   File "/home/ching/anaconda3/envs/tfpy35/lib/python3.5/site-packages/tensorflow/python/framework/errors_impl.py", line 466, in raise_exception_on_not_ok_status     pywrap_tensorflow.TF_GetCode(status)) tensorflow.python.framework.errors_impl.DataLossError: corrupted record at 0

Hi, @jiunyen-ching
Thanks for your report. I will look into it and try to target the problem bothering you.
Best
Yuxiao Guo

If it helps with debugging, here's the script that was edited according to my machine and ran.
export CUDA_VISIBLE_DEVICES=0
python train.py --input-previous-model-path ./previous --input-training-data-path ~/Downloads/NYUCAD/NYUCAD-TF-60 --input-validation-data-path ~/Downloads/NYUCAD/NYUCAD-TF-60 --input-gpu-nums 1 --input-network VVNetAE120 --log-dir ./log --max-iters 150000 --batch-per-device 2 --output-model-path ./ckp  --phase test

I have no previous models so I just made an empty folder and kept the --input-previous-model-path ./previous as it is.

I found there is nothing in generated TFRecord. After checking, I notice a file path bug in the current code. You may try the updated version in the master. Hope it will be helpful.

I have made the changes and the error still persists. Previously, my generated .tfrecords had significant sizes.
Here are the files generated with lines:
depth_path = os.path.join(folder_path, sample + '.png')
bin_path = os.path.join(folder_path, sample + '.bin')

NYUCADtest.tfrecord - 776.3 MB
NYUCADtrain - 929.0 MB
NYUtest.tfrecord - 991.3 MB
NYUtrain.tfrecord - 1.2GB
After making the changes as such,
depth_path = sample + '.png'
bin_path = sample + '.bin'

the file sizes are same.
For sanity check, I printed depth_path and bin_path for when generating .tfrecords for before and after I committed changes to the code. A sample of output for one file is as below:
--0000601 write /home/ching/Downloads/NYUCAD/NYUCADtrain/NYU0289_0000 in TFRECORDS
/home/ching/Downloads/NYUCAD/NYUCADtrain/NYU0289_0000.png
/home/ching/Downloads/NYUCAD/NYUCADtrain/NYU0289_0000.bin

Suppose that the .tfrecords that you generated are of the same size, do you mind to share your .tfrecords so that I may try to run on my machine?

I think it's fine to share the NYU & NYUCAD tfrecords with you. However, it's a relatively long time since this project has been finished. I will re-configure the environment in my local machine and target the problems in the current pipeline. Of course, I will share the training/test sets with you as well. I hope to finish it in these two days.

Thanks a ton @yuxiaoguo! Will look forward to your findings.

The issue has been addressed. The data prepare code wasn't updated with the training code. The training pipeline uses non-compression TFRecords (since voxel and images are compressed with their own file format), while data prepare code still compresses the TFRecords.
I have configured and run the experiments with generated data. It goes well. One more thing, if you tend to play with NYU & NYUCAD datasets, please set the --record-iters to 200, which is also the setting reported in our paper.

I see. I am now able to execute run_training.sh. Will update here if anything related comes up.
Thanks a lot for your help. Will close issue now.