Automatically support importing 3rd party svelte files

Related to #47
And related snowpack discussion: https://github.com/pikapkg/snowpack/issues/213
Two ideas so far:

Svelvet could auto create a snowpack.config.js file in the user's project dir if it doesn't exist.
Work with snowpack to support a way for us to inject a rollup plugin without requiring a snowpack.config.js file.
Example: snowpack --plugin rollup-plugin-svelte ?

The current workaround requires that the user creates a custom snowpack.config.js in the root of their project. It would be much nicer if the user doesn't have to worry about this at all and svelvet just handles it.

Svelvet could auto create a snowpack.config.js file in the user's project dir if it doesn't exist.

💯 I like this idea! We should probably add support for this regardless, as there are several cases where you need to maintain multiple config files. Using this, you probably could point to some secret config inside node_modules like so:
exec(`${snowpackLocation} --config ${path.resolve(__dirname, 'snowpack.config.js')}`)

etc.

Work with snowpack to support a way for us to inject a rollup plugin without requiring a snowpack.config.js file. Example: snowpack --plugin rollup-plugin-svelte ?

This may work, but 2 things here give me concern:

Rollup doesn’t support --plugin via CLI, which means that Snowpack would have to invent/maintain this (and there might be a reason Rollup doesn’t do this that we’re not aware of).
the moment that either you needed to configure an option, or the plugin maintainer decided to change the default options, wouldn’t make this workable anymore

As things stand right now I think it could work, but all it would take is either one change to Svelvet or rollup-plugin-svelte for this to stop working. But again, #1 should work, would work for other Snowpack cases, and would always make sure you have the power to manage the config you need, makes me personally lean in that direction

Oh you know what, I hadn't realized that Svelvet wrapped Snowpack. That --config proposal makes a lot more sense now! 😅

💯 I really like this idea! This would keep control within Svelvet, and not only relieve the user of maintenance, but Svelvet can also ensure it’s working for them which should mean less support for you. With this, we could hide a config within node_modules

Oh you know what, I hadn't realized that Svelvet wrapped Snowpack. That --config proposal makes a lot more sense now! 😅

Glad you both agree with that original idea! I didn't explain it well before 😅

So Svelvet users never run Snowpack? Just flagging that in case they are supposed to be able to, because using --config inside of Svelvet would mean that users running snowpack directly might break without it.

Exactly - svelvet users never have to run snowpack. It's part of the compilation pipeline since we need to compile src > dist first in order for snowpack --include 'dist/**/*' to work :)
So ideally, this line in svelvet we would be able to add --config ${path.resolve(__dirname, 'snowpack.config.js') just like @dangodev suggested above. That way, the config is invisible to the user and we have full control. I would also allow svelvet users to override this config file if they want to, but snowpack doesn't need to worry about that.

Snowpack just released --config support in v1.6.0 thanks to @dangodev 🎉
We can use @dangodev's solution posted here to automatically load a snowpack config file with the rollup svelte plugin enabled so that our users don't have to manually create that.
We should allow the ability to override the default config that we provide. Maybe if svelvet sees a snowpack.config.js we just use that one instead of our own 👍

We should allow the ability to override the default config that we provide. Maybe if svelvet sees a snowpack.config.js we just use that one instead of our own 👍

I think that could definitely work, but I’d strongly recommend using Svelte’s default Snowpack config by default, and only allowing a user to use their own if they specify a --snowpack-config CLI flag (or whatever you want to call it). Reason is I think there would be a lot of cases where users may want to have a snowpack.config.js file that they don’t want affecting Svelte. IMO Svelte managing its own Snowpack config that’s out-of-the-box perfectly optimized for its own setup is the best user story.
But that said, I definitely see some utility in people wanting to add other Rollup plugins or some other transformation to extend Svelte, and the idea is cool! But again, just my opinion: it shouldn’t automatically try and search to find all the potential Snowpack configs a user may/may not want to use (which would be a challenge in itself).
Aside from that, if a user wants to use their own Snowpack config to power Svelte, now Snowpack has some pretty handy warnings. So a user will see clear errors like:
Try installing rollup-plugin-svelte and adding it to Snowpack (https://www.snowpack.dev/#custom-rollup-plugins)

We try to make it as friendly as we can 🙂

Agreed! --snowpack-config <path> definitely works for me. Maybe if <path> is not supplied, we default to ./snowpack.config.js. Do you think that's a fine trade-off?

Sounds good to me!


Change flutter_cutom_tabs to 0.6.0 to support AndroidX

Any chance to approve this pull request? I really need this!


Remove "Loading..." text

How to remove the "Loading..." page  from PDF viewer?

same here..

same...

As a workaround I did the following in index.js
function viewerHtml(base64: string): string {
  ...
    <script type="text/javascript" src="bundle.js"></script>
    <script>
      // Remove "Loading..." text
      var p = document.querySelector("p");
      p.parentNode.parentNode.removeChild(p.parentNode);
    </script>
  ...
}

I did the pending pull request on all my project

Should be fixed at v0.5.0.


License?

I'm interested in using this project as a starting point but feel that I should obtain a license. Could you please add a usage license to the readme or to a separate file?

Yikes, can't believe i missed that. Good job on the improvements!


Is the accuracy on the test set in the figure 41.8% or 100%?

Definitely not 100%. You're at 42%. This is just output from the timm training script. See their repo for more details.

Is my model's best metric accuracy, and why is it so high?


Initialization might fail when surrounded by p-element

There is a strange issue we've encountered during development of our own custom elements. Not polymer, but it also seems to hit Polymer elements.
Whenever a customelement is wrapped inside a p-element, initialization might fail.
While reading the innerHTML of a custom-element -on initialization-, the dom seems to do a miscalculation on some child-elements. That is: some childNodes might be missing. It depends on what type of child-element is within: not all fail... div does and fieldset fails. And when it encounters an unprocessable element, the rest of the content is disregarded. So, you could end up with a "partial" inner-content.
This jsbin shows when the dom fails to see some child-elements --> look at the console to find out that the 3th element seems to be empty. It inspects innerHTML, but inspecting the childNodes gives the same result. And all browsers seem to act the same.
For now, we changed the design-setup for our custom elements to accept only comment-nodes inside their definition. Something like this:

<my-element> </my-element>

And use a html-parser to create true nodes from the read comment-node, which is processed further.
Even though i'm not using Polymer, it would be great if you guys know what's going on and find a solution.
Thx,
Marco Asbreuk

If I'm right, this probably means the end of Polymer in its current form and needs re-design.
So, I'm surprised no one picks it up.
Can anyone confirm the issue?
And why this jsbin shows an empty third line in the console?

I found that its because P does not allow block elements Stack overflow
Removing the DIV will make it work..

it is possible to use

but not

Interesting. that is likely to be the reason for this behaviour.
It still means you cannot design custom-elements that have a block-element inside their definition. And there are many arround already. They will fail initializing when placed inside a p-element.

You can have custom-elements that contain block elements sit inside a P element. I'm wondering if your seeing this issue because your using a shadow element? which would mean you need to access the innerHTML via {parentElement}.shadowRoot.innerHTML
[shadow and no-shadow example in a P element] (http://plnkr.co/edit/o5GenAEjHlXKjJdWaL63?p=preview)

I'm not using Polymer, but our own definition of custom-elements that don't use shadow-elements (because we build serverside-renderable elements)
However, the issue is the same.
I made a small setup that shows Polymer fails as well when using custom-elements inside p-elements:
http://jsbin.com/livinujije/1/
When the p-element gets removed, it will work.

it also will work if you leave the p-element, yet remove the h2 and div

To me, this looks like the browser is simply moving the block level elements outside of the p tag.  Per the HTML spec, p tags can only contain "phrasing content".  See http://www.w3.org/TR/html-markup/p.html.
I've created a modified example.  If you inspect the page, you'll see that the core-overlay works just fine and displays the span it contains, but all the block level elements have been moved out of the overlay and into the body of the page.
http://jsbin.com/wiyifazora/2/edit

yeah, that's the issue. Just like pflannery mentioned here.
It basicly means you cannot use block-elements when designing custom-elements (or you must assume   no one puts them inside a p-element).
Because neither is a workable sollution, we've changed the setup to only accept comment nodes inside

@ItsAsbreuk If page HTML against the spec (especially when there's consistency among browser implementation of said spec), its authors should be prepared for the end result to differ from the desired output.  You can only go so far to protect people from themselves ;)

@miztroh haha, you're absolutely right. But I'm wondering where spec-violation concerning custom-elements or its childnodes is defined ;)

@ItsAsbreuk Custom element children (light DOM) follow the same rules.  You could create a custom element that contains a core-overlay and its children.  That would put all the contents within shadow DOM, protecting them from the block level element issue.

@pflannery @miztroh thx for your comments.
I'll close the issue for I know what causes the behaviour. I''l leave it to Polymer whether they find it acceptable or not.

But I'm wondering where spec-violation concerning custom-elements or its childnodes is defined

Yeah, I'm really curious about that too.

@Ajedi32, @ItsAsbreuk: http://www.w3.org/html/wg/drafts/html/master/syntax.html#parsing-main-inbody

@miztroh, @Ajedi32: it says:
A start tag whose tag name is one of: "address", "article", "aside", "blockquote", "center", "details", "dialog", "dir", "div", "dl", "fieldset", "figcaption", "figure", "footer", "header", "hgroup", "main", "menu", "nav", "ol", "p", "section", "summary", "ul"
Correctly spoken, what happens seems to be right.
On the other hand, the tag that the user want to insert is a custom tag, not itself one of those specifed above. What we see is that the p-element is closing, because one of the descendants of the custom-element has one of the tags mentioned above. But those descendents were meant to initialize the custom element and may probably not even end up in the dom after rendering.
So, it is pretty confusing for end-users who want a custom element inside a p-element to run into this issue. As far as it concerns Polymer, creators of custom elements need to avoid using the specified elements, or mention it strongly in the documentation that it should not be used inside a p-element

@ItsAsbreuk, @Ajedi32: Take a look at this link.  It gives the spec for the p tag, listing its "content model" as "phrasing content".  You can then click on "phrasing content" for a list of tags that are viable as children of elements whose content model is phrasing content.
http://www.w3.org/html/wg/drafts/html/master/single-page.html#the-p-element
So, the spec is quite clear on this.  Custom elements aren't on the whitelist for allowed paragraph children.  This issue has nothing to do with custom elements specifically.  For example, the following would not give you what you might expect.

test1
test2

Instead, it produces the following:

test1
test2

I don't think it should be the Polymer team's responsibility to document all the caveats involved in constructing spec-compliant HTML.

@miztroh (@Ajedi32 ), I didn't see that part, so again what you say is true (though quite understandable for the fact that custom-elements are only recently implemented at their full strength).
I personal disagree with the responsibility part. Imo, because CE only recently found their way, and to protect end-users from running into edge cases, it is Polymer team's responsibility to define the playground where the CE should be used.

That spec doesn't mention custom elements at all though. So what content category does a custom element have? Going exclusively by those specs, aren't custom elements invalid no matter where you place them? The custom elements spec doesn't seem to specify a content category for custom elements either.

Polymer is not Web Components. Polymer is only there to make creating Web Components easier, provide useful utilities, and provide its own library of custom elements for use. Thus, I disagree that it is the "Polymer team's responsibility to define the playground where ... [custom elements] should be used" because that's not within the bounds of the Polymer project in the first place.
Instead, the heart of the problem described here seems to rest not within the Polymer library, but within the semantics of the HTML
 tag. Therefore, I think that feedback like this should be going towards the HTML standard and/or the Web Components standard instead of towards Polymer.

That all being said, here's something interesting to take a look at: http://jsbin.com/zetiziceza/edit?html,output


Catch TenantCouldNotBeIdentifiedOnDomainException globally?

Hi,
I'm getting 500 errors, when people use a non-working domain.
[message]
Tenant could not be identified on domain www {"exception":"[object] (Stancl\\Tenancy\\Exceptions\\TenantCouldNotBeIdentifiedOnDomainException(code: 0): Tenant could not be identified on domain www at /app/vendor/stancl/tenancy/src/Resolvers/DomainTenantResolver.php:48)
[stacktrace]
#0 /app/vendor/stancl/tenancy/src/Resolvers/Contracts/CachedTenantResolver.php(34): Stancl\\Tenancy\\Resolvers\\DomainTenantResolver->resolveWithoutCache('www')
#1 /app/vendor/stancl/tenancy/src/Middleware/IdentificationMiddleware.php(26): Stancl\\Tenancy\\Resolvers\\Contracts\\CachedTenantResolver->resolve('www')
#2 /app/vendor/stancl/tenancy/src/Middleware/InitializeTenancyBySubdomain.php(52): Stancl\\Tenancy\\Middleware\\IdentificationMiddleware->initializeTenancy(Object(Illuminate\\Http\\Request), Object(Closure), 'www')
#3 /app/vendor/stancl/tenancy/src/Middleware/InitializeTenancyByDomainOrSubdomain.php(22): Stancl\\Tenancy\\Middleware\\InitializeTenancyBySubdomain->handle(Object(Illuminate\\Http\\Request), Object(Closure))

Is there a way to Catch the specific TenantCouldNotBeIdentifiedOnDomainException and forward users to a specific page it that error occurs?

Please respect the issue templates.

@renege  did you find the solution ?

Solution that works like a charm!!
go to Providers/TenancyServiceProvider and in the boot method put this
InitializeTenancyByDomain::$onFail = function ($exception, $request, $next) {
            return redirect(config('app.url')); //can be a custom url it redirects to or you can show a custom error page
        };

@sarmenb thanks man.
solved my issue

@sarmenb doesn't work for me


Possibility to batch query exchange rates

I am using OpenExchangeRates service to regularly collect exchange rates for 50 currencies.
What I see happening is that it is using /api/latest.json request which gets a list of all exchange rates, but the library essentially throws away all rates except for 1 pair at a time. So this leads to 50 requests being made (while it can only be done with 1) and rate limit is hit very quickly.
Is there a possibility to make batch request or e.g. cache response from the library?

I could solve it by adding caching with http client. Using kevinrob/guzzle-cache-middleware.
Might be useful to somebody:
composer require kevinrob/guzzle-cache-middleware
Symfony config:
    http.cache.array:
      class: Kevinrob\GuzzleCache\Storage\VolatileRuntimeStorage

    exchange.http.cache_strategy.public:
      class: Kevinrob\GuzzleCache\Strategy\PublicCacheStrategy
      arguments:
        -<EMAIL_ADDRESS>
    exchange.http.cache.middleware:
      class: Kevinrob\GuzzleCache\CacheMiddleware
      arguments:
        -<EMAIL_ADDRESS>
    exchange.http_client.buffered.handler_stack:
      class: GuzzleHttp\HandlerStack
      factory: [GuzzleHttp\HandlerStack, create]
      calls:
         - method: push
           arguments:
               -<EMAIL_ADDRESS>               - 'cache'

    exchange.guzzle.buffered:
      class: GuzzleHttp\Client
      arguments:
        - handler<EMAIL_ADDRESS>
    exchange.http_client.buffered:
      class: Http\Adapter\Guzzle6\Client
      arguments<EMAIL_ADDRESS>
    ratio_provider.service.default:
      class: Exchanger\Service\OpenExchangeRates
      arguments:
        $httpClient<EMAIL_ADDRESS>        $options: {app_id: "%openexchange.app_id%"}

@geekdevs Nice you got it solved! Thanks for the solution :)


Add user affiliations when forking or templating [PLAT-159]

[#PLAT-159]

Purpose
Forking a project or using a project as a template was not copying over the current users' affiliations. Fix it!

Changes

Add method that copies affiliations from the node to the one it's forking or templating
Tests to make sure forked and templated nodes have correct affilations

QA Notes

Have a user with an institutional affiliation
Be a contributor on a project with at least one affiliation that matches yours, and one that doesn't
From the project overview page:

fork the project
use the project as a template

Ensure both the fork and template have your affiliations, but not the ones your user doesn't have

Documentation
This was expected behavior in the first place, so no changes needed
Side Effects
None anticipated
Ticket
https://openscience.atlassian.net/browse/PLAT-159

Merging into feature branch feature/raticate for testing on staging3


Allow for parsing of [ atomtypes ] in the library files?

The following issue may be encountered by people trying to implement a block starting from an itp generated by programs/servers (e.g., LigParGen) that generate AA force field (FF) files for GROMACS that include the [ atomypes ] in the itp - something like the following:
; GENERATED BY LigParGen Server
; Jorgensen Lab @ Yale University
;
[ atomtypes ]
  opls_800  C800    12.0110     0.000    A    3.50000E-01   2.76144E-01
  opls_801  O801    15.9990     0.000    A    2.90000E-01   5.85760E-01
  opls_802  H802     1.0080     0.000    A    2.50000E-01   1.25520E-01
[ moleculetype ]
; Name               nrexcl
PEO                   3
[ atoms ]
;   nr       type  resnr residue  atom   cgnr     charge       mass
     1   opls_800      1    PEO   C00      1    -0.0487    12.0110
...

Note that the atomtypes parameters are different from the ones that ship with GROMACS (i.e., /usr/local/gromacs/share/gromacs/top/oplsaa.ff/ffnonbonded.itp).
Now, I tried to see if having the [ atomtypes ] in a block would be fine but it's not:
Traceback (most recent call last):
  File "/Users/riccardo/miniconda3/envs/polyply/bin/polyply", line 7, in <module>
    exec(compile(f.read(), __file__, 'exec'))
  File "/Users/riccardo/ric.alessandri/Repos/polyply/polyply_1.0/bin/polyply", line 277, in <module>
    main()
  File "/Users/riccardo/ric.alessandri/Repos/polyply/polyply_1.0/bin/polyply", line 273, in main
    args.func(args)
  File "/Users/riccardo/ric.alessandri/Repos/polyply/polyply_1.0/polyply/src/gen_itp.py", line 53, in gen_itp
    force_field = load_library(args.name, args.lib, args.inpath)
  File "/Users/riccardo/ric.alessandri/Repos/polyply/polyply_1.0/polyply/src/load_library.py", line 110, in load_library
    read_ff_from_files(all_files, force_field)
  File "/Users/riccardo/ric.alessandri/Repos/polyply/polyply_1.0/polyply/src/load_library.py", line 75, in read_ff_from_files
    wrapper(parser, path, force_field)
  File "/Users/riccardo/ric.alessandri/Repos/polyply/polyply_1.0/polyply/src/load_library.py", line 67, in wrapper
    parser(lines, force_field=force_field)
  File "/Users/riccardo/ric.alessandri/Repos/polyply/vermouth-martinize/vermouth/ffinput.py", line 1003, in read_ff
    return list(director.parse(iter(lines)))
  File "/Users/riccardo/ric.alessandri/Repos/polyply/vermouth-martinize/vermouth/parser_utils.py", line 110, in parse
    result = self.dispatch(line)(line, lineno)
  File "/Users/riccardo/ric.alessandri/Repos/polyply/vermouth-martinize/vermouth/parser_utils.py", line 235, in parse_section
    raise IOError("Can't parse line {} in section '{}' because the "
OSError: Can't parse line 7 in section '['atomtypes']' because the section is unknown

It would be sufficient if polyply copy-pasted the [ atomtypes ] info found in a block at the top of the polymer itp file.
Otherwise, the user will have to do this by hand: that is, exclude the [ atomtypes ] from the block definition and just include the atomtypes like one would normally include the whole FF from the GROMACS default FF location or other. I think this would be OK, but then perhaps a clear error message about this could be implemented (--> https://github.com/marrink-lab/polyply_1.0/pull/118); such error message is also somewhat related to the "default GROMACS FF folder" issue (https://github.com/marrink-lab/polyply_1.0/issues/114).

I noticed this as well but it is more a problem of LigParaGen or GenGlycan and these kind of FF generators. The ATB does it smarter I think. But the central problem here is more about the distribution of force-field parameters rather than how to in cooperate those. It doesn't make any sense to write the atom-types directive into an itp/top file for a single molecule. The reason is that you cannot stack them or easily add/include them. Something like
#include A.itp
#include B.itp

where both A and B will not work when both itps contain atom-type parameters. In fact for LigParaGen I always have to strip and split the parameters, when I want to recycle them. Especially with the user having to do multiple invocations for more than a single polymer, they'd have to strip them by hand from the generated files. Or we write them to a separate file.
In part I guess this issue is also related to generating a "workflow" script that would distribute mdp files and commands as well as paramters. I mean we do have the parsing infrastructure to read top files, so we should be able to write them as well. But it does become a bit of a maintenance nightmare.
@ricalessandri there must be a comprehensive collection of the atomtype parameters that go with LigParaGen OPLS or are they ab initio derived on a per molecule basis as well? I'd rather distribute one single force-field top file for all polymers than having to distribute them on a per-case basis.

The reason is that you cannot stack them or easily add/include them. Something like
#include A.itp
#include B.itp

will not work when both itps contain atom-type parameters.

Very good point.

@ricalessandri there must be a comprehensive collection of the atomtype parameters that go with LigParaGen OPLS or are they ab initio derived on a per molecule basis as well? I'd rather distribute one single force-field top file for all polymers than having to distribute them on a per-case basis.

I looked for a collection but couldn't find it. I don't think they are derived on a per molecule basis, though.
But well, I think you can close this issue and you could just consider mention something a bit more verbose in the error messages (https://github.com/marrink-lab/polyply_1.0/pull/118) when somebody includes the [ atomtypes ].

Can you check if these parameters have corresponding OPLS types? Then we could simply use those.
I'll try to see if we can get a better error message, but they are from deep within the parsing routine, so I'm not entirely confident. But it is on the list.

@ricalessandri I think this is closed isn't it? The atomtypes shouldn't be distributed via the ff-files, for practicality reasons and because it's not clear if they are on a per molecule basis, which makes combining multiple residues problematic.


MWPW-112994 Chart metadata

Adding chart colors and container styles
Cleaning up container sizes

Resolves: MWPW-112994
Test URLs:

Before: https://data-viz--milo--adobecom.hlx.page/drafts/saravana/data-viz
After: https://chart-metadata--milo--adobecom.hlx.page/drafts/saravana/data-viz

@meganthecoder Regarding testing, I thought that maybe testing out the getColors functionality may be a good candidate based on our discussion earlier. You input a color and retrieve a list of colors back that is ordered based on that authored color, so might be good to check that we get back the correct ordering of colors, but I'd like to hear your thoughts as well. I haven't included a test for it in this PR, but can add it in a separate PR if we think would be good to have.


Enhencement add HostName filed in backends table and delete backends table in information_schema

Proposed changes
Issue Number: close #xxx

Add HostName field for show backends statement and backends() tvf.
delete the backends table in information_schema database

Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run p0

run buildall

run buildall

run buildall


Fix TableOperations.onDemand() javadoc

Fixes typos

I think "on-demand" should be stylized with a hyphen when used as an adjective in prose/documentation. Variables/constants can be ON_DEMAND, or ONDEMAND, or onDemand, as needed, but when writing in English and not referring to the specific variable, the hyphenated form is more grammatically correct.
For what it's worth, https://en.wikipedia.org/wiki/On-demand uses the hyphenated form as its disambiguation page for the term. As an adjective, "on-demand" seems correct. However, as a prepositional phrase, "on demand" seems correct. So, an "on-demand table" is a "table [made available] on demand".

The original intent of this PR was to fix the typos in TableOperations from "online" to "on-demand".
In the latest commit I took a quick look through the comments to see where I could apply the suggestions made by @ctubbsii.


How to force socket to communicate tls 1.2

My server provider is deprecating the tls older versions.
Asking me to shift to tls 1.2 please help me out.

Do you mean to use AsyncTcpConnection with tls1.2 ?

Do you mean to use AsyncTcpConnection with tls1.2 ?

Closing this due to a lack of feedback.

Closing this due to a lack of feedback.

Please use nginx as ssl proxy which config may like this.
server {

   listen 443 ssl;
   server_name example.com;

   # The key
   ssl_protocols TLSv1.2;

  # Other configs .......

  location /socket.io
  {
     proxy_pass http://<IP_ADDRESS>:2120;
     proxy_http_version 1.1;
     proxy_set_header Upgrade $http_upgrade;
     proxy_set_header Connection "Upgrade";
     proxy_set_header X-Real-IP $remote_addr;
  }
}


Add mini.nvim from upstream

Thanks for a great colorscheme! As a heavy user of the mini.nvim plugins I would love to see https://github.com/rebelot/kanagawa.nvim/pull/57 adapted. If you are busy I could try to do it myself and make a PR.

added in 1b719a1d7a62a4985ac7e32132af665389032b0f
I dont use much of the mini ecosystem myself, so let me know if anything looks particularly off

That was fast!
A few minor suggestions:
Use fujiWhite (or possible dragonWhite) for MiniIconsGrey
Use springViolet1 for MiniStatuslineModeVisual
Use autumntGreen for MiniStatuslineModeInsert and MiniStarterSection
Will let you know if I find something else

for statusline, Im using the same mode colours as lualine just so theres some consistency between the 2. the other changes have been made as well in latest commit


(Dropdown) Datepicker date cannot be set after initialization

Prerequisites

[X] I have not leaked any internal company information.
[X] I have not posted internal links or code snippets.

What happened?
When using the (Dropdown) Datepicker you cannot set the [from] and [to] properties of the component after it has been initialized.
A change is detected when changing that property, but it seems that internally this value is not updated as the component emits an event which has the last selected date as the value.
Even when you set [from]=null you can see that the button of the Dropdown Datepicker still shows the selected date.
What type of frontend frameware are you seeing the problem on?
Angular
Which version of iX do you use?
v2.1.0
Code to produce this issue.
<ix-date-dropdown [range]="false" [from]="selectedDate" (dateRangeChange)="onDateChange($event.detail.from)"></ix-date-dropdown>

<button (click)="setDatePicker()">Change Date</button>

selectedDate: string = null;
setDatePicker(){
  this.selectedDate = "2024/01/01";
}

Thanks for your input. We will have a look into it! Internal ref [IX-957]

Hi,
When do you plan to release the version containing this fix? Could you provide a specific date for it? We have a release scheduled for April 15th and would like to know if we should wait accordingly or explore alternative options.
Thanks.

@matthiashader This issue hinders us from using the date filter, too. Since our customers request it, when can we expect a fix?

Hello @network-interface-controller , @slymnozdmrc - sorry for keeping you waiting, according to our release plan the next release should be the end of next week!

Hello @matthiashader , Is there a change in the release plan? Accourding to your previous comment new release should have been completed last week. Could you give date for next relase please?

@slymnozdmrc sorry for these delays, i know it's a bit frustrating / annoying - we noticed some bugs before release that we are fixing now - but i can assure you that we will release this week for sure!

Hi @matthiashader , Is there any update regarding the release date?


restore: Add support for sparse files

What does this PR change? What problem does it solve?
It adds a restore --sparse option to restore files as sparse. The detection of sparseness is done at restore time, by replacing long stretches of zeros by sparse sections.
Note that this does not mean that we are no longer interested in having the archiver store precise information about which files have gaps at which places. Instead this is intended to provide basic support for sparse files which can be improved upon later.  In particular, if sparseness information was stored for a file, then it would also an option to have the restorer automatically restore such files as sparse.
The implementation is largely based on #2601, with a few modifications. The --sparse option was initially suggested by #2378. The restorer preallocates files by default which conflicts with restoring files as sparse. The currently taken approach is to check whether a file contains an all zeros blocks and mark is as sparse if the option was set. This then disables preallocation for that specific file. That way most files can still be preallocated. The downside is that we cannot detect even smaller sparse sections, however, these were only partially handled before.
Another change is that a sparse file is now immediately truncated to it's final size. partialFile.WriteAt may be called in parallel by the restorer, which could result in race conditions between the Truncate and WriteAt calls.
Was the change previously discussed in an issue or on the forum?
Supersedes #2378
Supersedes #2601
Fixes #79
Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[x] I have added tests for all code changes.
[ ] I have added documentation for relevant changes (in the manual).
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[x] I have run gofmt on the code in all commits.
[x] All commit messages are formatted in the same style as the other commits in the repo.
[ ] I'm done! This pull request is ready for review.

I've added the optimization discussed in #3903 to speed up hashing of all-zero chunks. On windows, files are now also marked as sparse.

LGTM. I've rebased the PR and fix the returned byte count of partialFile.WriteAt for a sparse write.

How about add --sparse option to dump command? I think It will be very useful especially when we speak about restore of VM disk images which is usually have a lot of sparse sections.

Oh, I realized that dump with --sparse is not possible, because restic just stream bytes to stdout and don't do write directly to file on filesystem. Hm.. But may be It possible don't read all zeros from repo and just "generate" them on the fly)
But it will require to store information about zeros in some place...


prompt_powerlevel9k_setup some bug

Hi, I am having some errors in executing prompt_powerlevel9k_setup
env LANG=C ./prompt_powerlevel9k_setup
./prompt_powerlevel9k_setup: line 27: ${(%):-%N}: bad substitution
export POWERLEVEL9K_INSTALLATION_DIR=/home/hepha/.zplug/repos/bhilburn/powerlevel9k
/home/hepha/.zplug/repos/bhilburn/powerlevel9k/functions/icons.zsh: line 15: typeset: -H: invalid option
typeset: usage: typeset [-aAfFgilnrtux] [-p] name[=value] ...
/home/hepha/.zplug/repos/bhilburn/powerlevel9k/functions/icons.zsh: line 238: local: can only be used in a function
/home/hepha/.zplug/repos/bhilburn/powerlevel9k/functions/colors.zsh: line 13: unexpected argument <' to conditional binary operator /home/hepha/.zplug/repos/bhilburn/powerlevel9k/functions/colors.zsh: line 13: syntax error near <-'
/home/hepha/.zplug/repos/bhilburn/powerlevel9k/functions/colors.zsh: line 13: `  if [[ "$1" = <-> ]]; then'

@hepha - Looks pretty strange, indeed. Can you share your .zshrc?

Hey @hepha! I'm going to close this one for now due to inactivity. If you are still having issues, please feel free to re-open!

im having a very similar issue but with a few more problems
bash: ${(%):-%N}: bad substitution
bash: /functions/icons.zsh: No such file or directory
bash: /functions/utilities.zsh: No such file or directory
bash: /functions/colors.zsh: No such file or directory
bash: /functions/vcs.zsh: No such file or directory
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: set_default: command not found
bash: /usr/share/zsh-theme-powerlevel9k/powerlevel9k.zsh-theme: line 862: syntax error near unexpected token ;' bash: /usr/share/zsh-theme-powerlevel9k/powerlevel9k.zsh-theme: line 862:       ;'

@pshoultz P9K is a ZSH theme. It won't work with bash.


What is the definition of flakiness?

The documentation doesn't seem to mention what the range of flakiness should be for the Searcher::report method.
The following code crashes:
use robust_binary_search::Searcher;

fn main() {
    let mut searcher = Searcher::new(10);

    searcher.report(5, true, 0.0);
    searcher.report(4, false, 0.0);

    let best_index = searcher.best_index();
    dbg!(best_index);
    let likehood = searcher.likelihood(best_index);
    dbg!(likehood);
    let next_index = searcher.next_index();
    dbg!(next_index);
}

Output:
[src/main.rs:10:5] best_index = 10
[src/main.rs:12:5] likehood = NaN
thread 'main' panicked at /home/tyilo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/robust-binary-search-0.1.1/src/lib.rs:127:5:
assertion failed: best_percentile > f64::NEG_INFINITY
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

flakiness should be between 0 and 1, excluding both 0 and 1. I'll update the docs. I'd recommend using AutoSearcher if you don't know what the value should be.

Thanks. I basically want to binary search, where some values are unknown and others are completely known, that is flakiness 0 and 1.
Maybe there is a better library for this, but I tried to make this work :)


Create Article “combat-maneuvers”

Automatically generated by Netlify CMS

👷 Deploy Preview for dynamic-cheesecake-5fbd8b processing.

Name
Link

🔨 Latest commit
7c52986a084090cc66971a00940c99822e5248f3

🔍 Latest deploy log
https://app.netlify.com/sites/dynamic-cheesecake-5fbd8b/deploys/62ef35c41d247a0009990eae


MINOR: Updated KafkaConsumer.commitSync() javadoc

Throws IllegalArgumentException is the offset is negative

@ijuma This is another small javadoc update. Can you have a look ? Thanks

LGTM. I'll go ahead and merge this and remove the unneeded @throws.


Frame data shifting

Add two action buttons that move the targeted frame shape data to the left or to the right in the animation, pushing other frames as needed, and cancelling if there is no room.
Useful when one has added new sprite frames to the animation, and wants to update the shape data accordingly.

This feature is mostly done, and working as I expect it to.
I'm going to test it out some more ; do ping me here if you want it and I forgot to push it.


Mapping nodes

Write easy to reproduce python code to list as many network nodes as possible. And map to location in blender
s 3D space. List connections per node, and what nodes they connect to.

can i get more brief on this

great, A node manager basically entails stuff like this.
do you want them extracted or just the script though, and saved to a db I'm only running a regtest bitcoin node

I would love to contribute, the only clause here is I'm only running regtest for development purpose. i know what's needed to be done


Is that possible to make formatter an independent tool??

Like rustfmt or gofmt, So we can use it somewhere else.
I think Kotlin can compile to binary without jdk.

No, it depends on the platform, which is heavily Java.
What could be done is to integrate with Github Actions so the proper formatting could be checked in the CI.

@mkurnikov Ok,Thanks.


Forward a message to a channel and reply to that message

Thank you for your api, like django it is  fast developing.
My question is how it is possible to forward a message to a channel by bot and then immediately reply to that message.
i know when you forward a message, telegram replies with a response but i cant get that response so that i can reply to forwarded message.

Hi,
Sorry for the late reply.. a couple of busy weeks.
In general, this is how you can send a message (or forward it) and then use the sent message to reply to it:
@processor(state_manager, from_states=state_types.All)
def hello_world(bot: TelegramBot, update: Update, state: TelegramState):
    message = bot.sendMessage(update.get_chat().get_id(), 'Hello!')
    message = bot.forwardMessage(update.get_chat().get_id(), update.get_chat().get_id(), message.get_message_id())
    bot.sendMessage(update.get_chat().get_id(), "Reply", reply_to_message_id=message.get_message_id())

Add this ^ as one of your processors and all updates will be responded by a message from the bot, which will be forwarded to the same chat by the bot and finally the forwarded message will be replied by the bot:

However, your specific case (doing the same scenario but in a channel) made me realize there is actually an error regarding receiving the messages from the channel so you might have faced an integrity error exception. My apologies for that; it is fixed now (it is not versioned yet though - you can either download the updated code from Github or wait a few days before you can download the new version through pip). After downloading the fix, the same code snippet as above would do the same thing in channels:

I've reopened this issue till the new version is released and downloadable through pip. Please close it after you have downloaded it (either through pip or Github) and what you wanted to achieve is achieved.
Let me know if there are any other issues. Also, feel free to join our discussion chat (link in repo Readme).

You may now upgrade to django-tgbot_1.4.9 and the issue will be fixed.
pip3 install --upgrade django-tgbot


[Feature]-[1032]:gsheets unauthorised datasaource, but query successful

Is there an existing issue for this?

[X] I have searched the existing issues

Description
This is a weird scenario - consider I want to remove accesses to my ghseets in all the applications within the workspace. So I go and un-authorise the datasource, and then expect the queries to fail, but they don't.
Steps To Reproduce
Steps:

Create a valid gsheets datasource
Create a select query on top of this, test that it runs properly
Go back to the datasource, and click reauthorise, but in the google auth popup, click Cancel to simulate auth failre
Check that the datasource now is in unauthorised state
Go back and run the query
Expected: the query should fail with auth error
Actual: the query runs successfully

Public Sample App
No response
Version
Cloud

6m usage of gsheets: 2585
assume 5% run into this: 129
Stats

Stat
Values

Reach
129

Effort (months)
0.25

created a jira ticket-https://zemoso.atlassian.net/browse/TNOSB-358

I started working on it.
@contributor-support

HI @rohan-arthur, @NilanshBansal
Approach to solve

Track the authentication status of the data source.
Verify the authentication before running a query each time.

If authentication fails:
- Display an error message: "Datasource is not authenticated."
If authentication is successful:
- Run the query as usual.

I'm having difficulty understanding how connections are established and validated for each query execution for google sheets within the code base. Could you provide some guidance or suggest where I should start to get a clearer picture?

Hey @AnnaHariprasad5123 can you please pick up some issues with the label Inviting Contribution. The current issue is not under that list.
The list can be found here: https://github.com/appsmithorg/appsmith/issues?q=is%3Aopen+is%3Aissue+label%3A"Inviting+Contribution"
Also, you can tag @appsmithorg/contributor-support before starting work on any issue.


Art hexagons in popups for cards in NISEI frames use incorrect cropping for some card types

The mouseover popup for viewing card information includes a hexagonal area that's supposed to show the card's art, but some card types have the art centered at a different point for FFG frames than for NISEI frames. This causes NISEI cards to have a portion of the card title box visible in the hexagon, instead of centering on the art itself.

this comes from the difference in layout and templating between FFG and nisei cards.


Preferences is hard to select while moving cursor down the sidebar menu when you have lists.

When you move your cursor from top to bottom through sidebar menu options, hovering Lists will display lists right below and move Preferences, but if you continue to move down to Preferences the menu Lists collapses and Preferences gets back to its initial position. So you need to move up back to Preferences (without hitting Lists or the same thing is happening again)
Here is a small gif sh

owing the issue:

Hi :)
It is true this isn't ideal currently. I will be experimenting with an alternative way to unfold lists in the sidebar if possible.

Maybe adding a button to toggle the collapsing of Lists items would be better than triggering by mouse hover. If I recall correctly, it might be possible to do so CSS-only using a check boxes and attribute selectors.

This is fixed in Tangerine UI v2.0.0-prerelease2: the Lists item has been moved to the bottom of the navigation panel.

I will close this issue when we release v2.0.0 in the coming weeks :)


Update secure-a-cluster.md

This change is

no equivalent 20.1 entry.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f3bf53e86d0ead363a8b913a36c549e7bdd5dcc3/
Edited pages:

v19.2/secure-a-cluster.md

no equivalent 20.1 entry.

Thanks for fixing the error! There's a 20.1 doc that also needs to be updated. LMK if you'd prefer me to do it!

go right ahead @Amruta-Ranade

@Amruta-Ranade, I'm actually just opening a PR to fix this page for a different reason, so I'll make this change as part of that PR. Sorry for the churn here.


labels option is an invalid type on TypeScript environment.

I'm having problem with the package typings. Looks like that the 'labels' options is missing in type.
I'm getting  **Object literal may only specify known properties, and 'labels' does not exist in type 'Options | undefined'.**
Environment details

OS: Mac OS X
Node.js version: 8.9.4
npm version: 5.7.1
@google-cloud/logging-winston version: 0.8.1

Steps to reproduce

Be on an TypeScript strict environment.
Create an transport with new LoggingWinston({ labels: { 'key': 'value' } })

Why I don't see any fix in the commit ? Shows only the version bump.

Version 0.9.0 has been released that should address this issue.  If you still having issues, please let me know.  Thanks.


Incorporate ReadableStreamChannel interface changes within GetBlobResult

ReadableStreamChannel introduced two new digest related methods as part of a patch parallel to the Get patch that had an implementation of this interface.
Adding that to the GetBlobResult implementation.
Reviewer: Gopal, ~ 5 min.

LGTM. Let's follow up on the discussion about whether impls should be allowed to not support these functions.


First draw after adding to mapgroup not showing all layers

This issue was imported from CodePlex
aparks wrote 2011-04-27 at 18:15
I am adding two in memory layers to a map group. On the first draw only the first layer added is shown. If I do anything to force a redraw (IE resize window) the layers are drawn correctly. Adding a map.refresh at the end of adding the layers will fix
the problem, however it map.invalidate will not.
Aaron Parks
RTI International
aparks wrote 2011-05-19 at 18:14
Should "NOT showing all layers" rather than 'NOW showing all layers"

Not reproduced.


[Catalog] Placeholder in comment fields is white instead of gray after switching to black theme

Environment: Dell Inspiron 3576, Windows 10б Google Chrome 94.0.4606.61
Type UI
Reproducible: Always
Preconditions

Go to https://horondi-front-staging.azurewebsites.net/login
Log in using google account
Switch theme to Black

Steps to reproduce

Click on 'Burger menu'
Choose Backpacks
Choose Rolltop
Click on Rolltop blue
Scroll to comment field
Pay attention to comment field (Placeholder)

Actual result
Colour of placeholder is white instead of gray. Impossible to read placeholder

Expected result
Colour of placeholder is gray instead of white. Possibility to read placeholder

Priority: Low
Severity: Minor

When user places the cursor in the comment field text of placeholder are visible partly


network.evolve not using provided fitness function

The function Network.prototype.evolve is not using the passed options.fitness function. Rather, it's making a default one inside of evolve which doesn't use options.fitness.
Solution:
Add a conditional so that the passed one is used instead of the default when it corresponds.

Fixed in branch typescript-refactor


fix(server): stop routes from accepting default text/plain requests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
479

Relevant Lines:
479

💛  - Coveralls


Fix a few notes within User Manual

What is the change?
While reading parts of the UM, I've found a couple notes that are out-of-date.

There is no T/H module within ARMI so temperatures are not updated in ARMI with tight coupling.
Block heights don't have to be input at hot temperatures.

Why is the change being made?
Having correct docs is important.

Checklist

[x] This PR has only one purpose or idea.
~Tests have been added/updated to verify any new/changed code.~

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

~The release notes have been updated if necessary.~
[x] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in pyproject.toml.

@albeanth  We can merge your PR, as-is, but perhaps we could also add this one-line change to the docs in this PR?
An error was noted in section <IP_ADDRESS> if the ARMI UM. The first line of the code block is

b = o.r.getFirstBlock(Flags.FUEL)

but should be:

b = o.r.core.getFirstBlock(Flags.FUEL)

At your discretion.

At your discretion.

Done!


No localize repository for "windowsserverdocs"

Link to article:
original:
https://docs.microsoft.com/en-us/windows-server/get-started-19/whats-new-19
translate:
https://docs.microsoft.com/ja-jp/windows-server/get-started-19/whats-new-19
Problem:
I want to fix up translation for Windows Server documents, but no Edit/Feedback button in Japanese pages.
Please Create Japanese repository and another languages for "windowsserverdocs"
https://github.com/MicrosoftDocs/windowsserverdocs

@SonjaSaltzman Could you please take a look at this issue?

Assigning to @v-nidolc @DavidPrio

@nokura user is reporting they cannot edit the translation because there's no edit nor feedback button
how should we proceed?
@DavidPrio

We are currently only enabling feedback buttons for human translated content.  This is a project we freshly onboarded in our team, so @v-nidolc can provide an ETA for when we will enable.

We plan to do this next by the end of May, after we complete testing the initial localization (currently in progress).

Hi team
any update on this?
best

It looks like this is fixed, please confirm @ALilyG
@SonjaSaltzman @v-nidolc

It looks like this is fixed, please confirm @ALilyG
@SonjaSaltzman @v-nidolc

I confirmed this. Thanks for handling it.


ARTEMIS-739 fix large msg file leak on copy

are you sure this builds on windows?

No.  I'm not sure it builds on Windows as I have no Windows machine to build it on.


Add PR template

I would like to add a pull request template for this repository. I believe that having a standardized template will help streamline the contribution process, ensuring that all necessary information is included and making it easier for maintainers to review and merge pull requests.
Could you please assign this issue to me under GSSOC'24.

Pull request template already exists !


Moar threadsafe moar better

This is a rebased version of #4, it should be ready to merge. Fixes #73.
(made after discussing with @jpsamaroo)
CC @vchuravy, @vtjnash

I tracked the test failures down to: https://github.com/JuliaLang/julia/pull/53326
The workers are loading the builtin version of Distributed from the sysimg while the master is using the development version, and because we changed the definition of the Worker struct in this PR we get a serialization error when doing a remotecall to another worker because the master and worker are using different types.
I think this should be fixed in Base, made a PR here: https://github.com/JuliaLang/julia/pull/54571
That should be merged before this one.

Alrighty, after a force push to trigger CI with latest nightly we are back in the green :partying_face: I think this is ready to be merged now.

Will this be compatible to or backported, if necessary, to the next Julia LTS?

I'll leave that for someone more qualified to properly answer, but FWIW if 1.11 is chosen as the next LTS then it'll be possible to upgrade Distributed now that it's an excised stdlib :octopus:

One other thing I noticed is that this should probably be using Threads.@spawn everywhere instead of @async, but I'll leave that for another PR.

I believe @jblaschke was going to have a look at it. In the meantime I see the branch is out of date, so I'll rebase it.

@JamesWrigley @jonas-schulze I'll be working on this this week. Just getting back up to speed after long travel...

Are there any updates on this?  @JamesWrigley @JBlaschke

Not from me, still need someone to review it. I believe the hesitation to merge comes from Distributed being used to run the Julia tests, so it's quite critical that this works properly.

Since we are making things threadsafe I would look at all @async uses because we want to deprecate it.

After thinking about it for a bit, I can't come up with a decent replacement short of basically reimplementing Threads.@spawn. We could replace @async_unwrap with Threads.@spawn, but it would change the type of any exceptions thrown from whatever they currently are to a TaskFailedException, which is technically breaking.
I'd suggest keeping it in for now, we can add support for unwrapping exceptions to Threads.@spawn and switch in a future release.

Wee progress update for those following this, after some discussion with @vchuravy on Slack I decided to:

Try to run the Julia tests in the Distributed.jl CI since that's what was failing last time.
Attempt to audit the code for correctness.

Which I'll get to at... some point.


Change the if else statements

Purpose
This is to fix the if-else statements of the examples/json-io/json_io.bal file.
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/15934
Approach

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

Samples

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

Remarks

List any other known issues, related PRs, TODO items or any other notes related to the PR.

Check List

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

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

[ ] Increased Test Coverage
[ ] Added necessary documentation

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9b40b94). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #15957   +/-   ##
=========================================
  Coverage          ?   11.34%
  Complexity        ?     3767
=========================================
  Files             ?     2064
  Lines             ?    96031
  Branches          ?    13462
=========================================
  Hits              ?    10896
  Misses            ?    83026
  Partials          ?     2109

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 9b40b94...8e03a27. Read the comment docs.


Can't build seajax on mac

Eights-Book-2:Scripts eight$ npm install
module.js:338
throw err;
^
Error: Cannot find module 'umask'
at Function.Module._resolveFilename (module.js:336:15)
at Function.Module._load (module.js:278:25)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at Object. (/usr/local/lib/node_modules/npm/lib/utils/umask.js:1:75)
at Module._compile (module.js:460:26)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Module.require (module.js:365:17)
Eights-Book-2:Scripts eight$

Using the pivotviewer canvas template, I am getting odd behavior. I am attempting to draw a canvas element with a picture in it. The image loads fine in IE, flickers and can disappear in Firefox, and does not appear at all in Chrome/Safari. Here is a simple template that is not working. I don't want to use just the image template, because I want to draw images and text on the canvas. Can anyone tell me why my .drawimage does not work in the canvas template?
viewer.setTemplates([
{
type: "canvas",
width: 438,
height: 300,
template: function (ctx, x, y, w, h, item) {
ctx.strokeStyle = "black";
ctx.lineWidth = w / 100;
ctx.strokeRect(x + w / 200, y + w / 200, .99 * w, h - .01 * w);
var img = new Image();
img.onload = function () {
ctx.drawImage(img, x, y, w, h);
}
img.src = 'http://www.html5canvastutorials.com/demos/assets/darth-vader.jpg';
}
}
]);

Hi cpaulin,
Thanks for your interest! In order to do animations and such, the Viewer regularly clears its entire view and tells each element to redraw itself. It seems that the particular timing of when the screen repaints is different on various browsers, but in general I'd expect that a solution like the following might be beneficial.
var loadedImages = {}; // a set of images by source URL so we don't have to re-decode
viewer.setTemplates([
    {
        type: "canvas",
        width: 438,
        height: 300,
        template: function (ctx, x, y, w, h, item) {
            ctx.strokeStyle = "black";
            ctx.lineWidth = w / 100;
            ctx.strokeRect(x + w / 200, y + w / 200, .99 * w, h - .01 * w);
            var src = 'http://www.html5canvastutorials.com/demos/assets/darth-vader.jpg'; // would probably vary by item
            var img = loadedImages[src];
            if (!img) {
                var img = loadedImages[src] = new Image();
                img.src = src;
            }
            if (img.complete) {
                ctx.drawImage(img, x, y, w, h);
                // returning true means this item doesn't need to redraw itself next frame (unless viewport moves and everybody must redraw)
                return true;
            }
        }
    }
]);

Since the viewer is trying to redraw every frame anyway, we don't necessarily need to wait for a load event, we can just check the image's complete property at the time we are asked to draw the element. Once the image has loaded and drawn successfully, we can return true to notify the viewer that we don't require an update on the next frame; this can help CPU usage a lot by stopping the redraw cycle until the user does something. A more advanced solution would also include an error handler for the image, so that we can return true after either successful download or failure.

And @eighteight , I'm sorry we just derailed your issue thread. I don't have a Mac and so I can't be of much use to you, but perhaps some other members of the community here could help (summoning @gasi ).

@eighteight Which version of Node.js / npm do you have?
I just tried with a fresh git clone and it worked:
> node --version
v0.10.26

> npm --version
1.4.23

> npm install .
<EMAIL_ADDRESS>node_modules/uglify-js
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
<EMAIL_ADDRESS>node_modules/jshint
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>


Custom Models: Multiple Implemenations

Hello!
I have a namespace  App\Models and phpStorm shows Multiple implementations on that custom models in that namespace.
Is there a solution? :)

This is an issue that has been seen before (#430), afaik there's no current workaround (short of disabling the warning in PhpStorm) if you want to keep writing the meta to a separate file. If you write the meta to the model file itself - use the -W option when generating meta like php artisan ide-helper:models -W the error gets resolved.

@barryvdh last comment is the solution (either live with it or write it to the models), issue can be closed :}


Add review

Add review from webapp

Thanks for your contribution @twood25. Unfortunately, it is of not use; and I am hereby rejecting it. You probably did not bother to follow any of the links. If you had, you would have noticed something was not quite right.
I would kindly ask you to please not waste our time.
I am disqualifying both PRs #226 and #227. No bounty will be awarded.

Thanks for your contribution @twood25. Unfortunately, it is of not use; and I am hereby rejecting it. You probably did not bother to follow any of the links. If you had, you would have noticed something was not quite right.
I would kindly ask you to please not waste our time.
I am disqualifying both PRs #226 and #227. No bounty will be awarded.


Separate builds for profiles

Re-write the build script for Tizen (tizen/BUILD.gn)

Learn GN first if you want to fully understand the script - it's not that difficult!
There are 4 outputs now: libflutter_tizen_mobile.so, libflutter_tizen_wearable.so, libflutter_tizen_tv.so, libflutter_tizen_common.so

Use __dlog_print() only on TV and use dlog_print() on other profiles
Cherry-pick "Enable Evas GL direct mode (#54)" (+ build fix)
Unsupport the ecore_wl renderer (no more separate build for Tizen 4.0 because Evas GL supports Tizen 4.0)
Update azure-pipelines.yml accordingly

This change requires https://github.com/flutter-tizen/tizen_tools/pull/9.
Contributes to https://github.com/flutter-tizen/flutter-tizen/issues/86 and https://github.com/flutter-tizen/flutter-tizen/issues/21.
More testing, improvements, and clean-ups are needed (I tested only on a few devices).

I tested on

Tizen 4.0 wearable emulator
Tizen 5.5 wearable emulator
Tizen 6.0 mobile emulator
Tizen 6.0 TV emulator
Tizen 5.5 watch
Tizen 6.0 watch
Tizen 6.0 rpi4 (arm64)

jsonexample and gallery worked fine on every device. (I didn't measure performance but there was no obvious regression.) Currently the video_player plugin doesn't work properly in Evas GL mode (watch) - this will be addressed later.

Resolved a conflict with #69.


Cache Nearby data

Nearby places (coordinates, WLM status, existence, etc all details) do not change often, and take time to load.
I suggest caching that information for 1 hour or even more.
Cache invalidation should happen when a picture of a place is uploaded.

Not needed anymore as Nearby is now quite fast 🎉


Code Review

Merging this as-is. @GabFitzgerald I'll spin up a blank branch to enable full review in a moment.

Merging this as-is. @GabFitzgerald I'll spin up a blank branch to enable full review in a moment.


Server panics on generate_route_list (SSR)

I'm able to reproduce the panic in this PR:
https://github.com/juspay/nix-browser/pull/33
The panic happens on server start only, never after.
To workaround this, I disable query_load during generate_route_list as follows:
leptos_query::suppress_query_load(true);
let routes = generate_route_list(|cx| view! { cx, <App/> }).await;
leptos_query::suppress_query_load(false);

Can this library automate it? Failing that, we should document this issue somewhere in README for SSR users.

backtrace,
https://gist.githubusercontent.com/srid/23400e5f4de25f78f806a83b3c2ddeae/raw/baa924e593dffeccd5c10047a8b4ea42f8715c5b/gistfile1.txt


Why does my memory fail when I use rapidfuzz.process.extract

Hello, I am using rapidfuzz.process.extract every second. My RAM can't handle it. What should I do ? Are you using globals var?

I could not reproduce this behavior. Could you provide me with a minimal reproducible example, which has this behavior?

 async def get_music(query):    result = rapidfuzz.process.extract(query,dbs["audio_books"],scorer=rapidfuzz.fuzz.partial_ratio,limit=50)    return query

What is the type of dbs["audio_books"]? Does the memory usage grow over time, or does it run out of memory on the first call?

This is a list dbs = shelve.open ("books") there are 50,000 values
1 request + 9mb to memory
10 requests 90 mbto memory
100 requests 900 mb to memory

So you have a function like:
from rapidfuzz import process, fuzz
import random
import string
random.seed(18)

words = [''.join(random.choice(string.ascii_letters + string.digits) for _ in range(20)) for _ in range(50000)]
async def get(query):
    return process.extract(query, words, scorer=fuzz.partial_ratio, limit=50)

How is this function run? What do you do with the results? Could you change the example above to reproduce the behavior (so I can just copy paste and run it on my machine).

Thanks is working 👍👍👍👍👍👍👍👍👍

I don't sleep 48 hours . Thanks bro , you best


Add example Kind configuration for Kubernetes Audit

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area deploy
What this PR does / why we need it:
Add example Kind cluster API server configuration for Kubernetes Audit.

@maxgio92: There is not a label identifying the kind of this PR.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign maxgio92
You can assign the PR to them by writing /assign @maxgio92 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks @leogr, sure, I forgot to mention that it’s a WIP

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leogr
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

This looks very good already!

LGTM label has been added.  Git tree hash: b7ac6d0387bead033cc49d8598c5a7e829112d5a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

fd07589 docs(deploy/kind): add basic readme

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leogr
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leogr
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sure @leogr, I only added a super-basic readme.

LGTM label has been added.  Git tree hash: 381d65996dc39c0eba001266e10b3ac6583d9039

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Support for Deleting Shared Folders as a Share Receiver

closes https://github.com/owncloud/ocis/issues/2148
closes https://github.com/owncloud/ocis/issues/2148

@refs what about putting this logic in the gateway?
If the gateway can catch a Remove request to share it will call the share manager and set is as decline (not pending, see attached video).
@ishank011 what do you think?

If the gateway can catch a Remove request to a share it will call the share manager and set is as decline (not pending, see attached video)

+1
Instead of calling ListReceivedShares for every delete request, we can instead add that logic here
https://github.com/cs3org/reva/blob/905fa4c17f9b7f6ed95232a0f29a0d11340652b1/internal/grpc/services/gateway/storageprovider.go#L850-L857

@refs what about putting this logic in the gateway?
If the gateway can catch a Remove request to a share it will call the share manager and set is as decline (not pending, see attached video).
@ishank011 what do you think?
unsharing.mov

Agree, that'd save some bandwidth 👍 !

@labkode decline seems like a more appropriate state for the share. I'll update the PR to reflect it.

@ishank011, @labkode if this matches the criteria, it should be ready to be merged 🎉

@refs looks good but I think we can add this parameter to DeleteRequest if we want to skip moving the resource to recycle bin. What do you think? @labkode @butonic


Command Handler: persisting data

How are data persisted in write database from command handler ?i'm a beginner with es-cqrs and it's difficult for me to understand this step!

The projectors are listening on events of the EventBus so when an event is published it listen to the events and store it in the write side.
It's better explained here

So, if i understand, the UserEmailReadProjectionFactory  class is used to persist data on the db, right?

Right. It's a listener that will receive the events after they are persisted so you can do whatever you want with them.

Read class to Write data....it confuses me a little!

Ops Typo


I keep getting an error everytime I run the program

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report
N/A
Expected Behavior
Expected: The bot should start and print "BOT STARTED" in console
What Happened: Does not start
Code Example or Reproduction Steps
N/A
Code for JDABuilder or DefaultShardManagerBuilder Used
JDABuilder jdaBuilder = JDABuilder.createDefault("TOKEN");

JDA bot = null;
try {
    bot = jdaBuilder.build();
    bot.awaitReady();
} catch (LoginException | InterruptedException e) {
    e.printStackTrace();
}
System.out.println("BOT STARTED");

Exception or Error
java.lang.NoClassDefFoundError: net/dv8tion/jda/api/JDABuilder
Caused by: java.lang.ClassNotFoundException: net.dv8tion.jda.api.JDABuilder

Duplicate of #101
You should use one of the setups provided in the wiki.

I have my project made in maven not gradle

Its part of a Minecraft Spigot Plugin

There is also a Maven setup linked. Normally, you use Gradle with IJ, that's why it's not linked there.

This is not the place to learn about maven.


Custom menu : when I add icon it does not draw anything

I have implemented Toast Photo Editor to Filemaker and I want to create a menu with only some functions like drawing, icon, tex.
Only the chosen menu shows but icon does not draw anything to canvas. When I use the full menu, it works well.
Any idea?

I'm sorry for the late response by mistake. I'll fix this bug soon. Thanks!

@mersimbakalli
Sorry for the late reply by mistake. I can't reproduce this bug.
Can you repost this issue with detail environment? Thank!


grounded flow not capturing context

e2e for grounded skills is not able to identify if it has context or not because of the wrong key name, it's looking for context instead of seed_context

Fixed by #98


[Issue] add missing "types" property to package.json "exports" maps

Currently, TypeScript fails to find declarations for tamagui (and many @tamagui/* packages) when using the Node16 or NodeNext module resolution strategy (i.e. using ES modules). The reason for this is because all package.json files are missing the types field within their exports maps; this is only necessary when declarations aren't located next to the JavaScript files.
This PR:

adds the missing field to all relevant package.json files
adds a missing StyleObject import to packages/static/types/extractor/extractMediaStyle.d.ts (I'm not sure why the build action adds this only after making the prior change)
removes an invalid ./react-native entry point from the @tamagui/core exports map (this file no longer exists)

NOTE: I'm happy to create an example repository which demonstrates the issue, if that helps.

Beautiful, thanks


Erroneous yield events reported in Safe account history

I'm connected to the dapp with a Safe wallet via iOS and WalletConnect. When I open my account history, I see yield even though I have not opted-in. Both our dapp and Etherscan agree that my balance has not increased.
https://www.loom.com/share/293b59d5f321478f8db5cccd5964a5d0
This video also depicts https://github.com/OriginProtocol/origin-dollar/issues/1404.

The cause of this issue that we assume in the backend that a user/contract is opted in from the beginning so we just return all yield events.
However, I'm not sure how to solve this since rebaseOptIns and rebaseOptOuts don't emit an event and thus, aren't indexed. But the frontend is doing its job properly so I would say we should move this be an analytics issue as opposed to dapp one.
One additional thing, this is also an issue on the ousd dapp.

See https://github.com/OriginProtocol/origin-dollar/issues/1754 💡

@micahalcorn this should not happen on the new OETH app, can you confirm?


what is hmac and cipher?

what is hmac and cipher? where can i find those paremeters?

I need that explanation too. :)

https://github.com/gaskaj/node-red-contrib-ftp-sftp/blob/b82bb85672ca796c1e509e61c47a0187631ee0c7/sftp.js#L71


[Enhancement] Logger of slf4j does not work in eventmesh-common test module

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request
The object of org.slf4j.Logger can not execute no matter log.info, log.error and so on in test.
Describe the solution you'd like
Add log4j2.xml to eventmesh-common test module
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

May I ask if the problem mentioned in this issue still exists?


[AutoPR applicationinsights/resource-manager] [AppInsights] Added webtests-by-component API to 2015-05-01

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3126

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/1900


Increase scale of buttons on hover

When hovered, the links will get a little bigger which looks cool and interactive as well! 💥
If you like the idea, can you please assign me this issue, I'd love to work on this 🥳
Check out this GIF:

Done with the changes, looks like this:

Done with the changes, looks like this:

Yes nice idea but will like to suggest you something try to make a change that if one button is scaled then other should not have impact of scale on it

Pull Request: https://github.com/EddieHubCommunity/LinkFree/pull/140


Do we need even more agressive validation?

Currently, we do not catch this:
#![feature(rustc_attrs)]
#![allow(unused_attributes)]

#[rustc_layout_scalar_valid_range_start(1)]
#[repr(transparent)]
pub(crate) struct NonZero<T>(pub(crate) T);

fn main() {
    let mut x = Some(NonZero(1));
    let xref = match x {
        Some(ref mut c) => c,
        None => panic!(),
    };
    *xref = NonZero(0);
}

In the MIR, the *xref = NonZero(0) becomes an assignment of the only field of this struct, and that field is of type i32 and hence value 0 is no problem.
I could imagine doing validation of prefixes of the path involved in an assignment, but I see no way to catch the following:
fn main() {
    let mut x = Some(unsafe { NonZero(1) }); // this one is fine
    let xref = match x {
        Some(NonZero(ref mut c)) => c,
        None => panic!(),
    };
    *xref = 0; // this one is not
}

This won't even be caught by @oli-obk's new unsafety check for constructing NonZero.
(The last example is not specific to NonZero at all; writing 2 into a &mut bool after casting it to *mut u8 is similar.)

aaand now I'm gonna make taking references to fields of rustc_layout_scalar_valid_range_start unsafe, too.

cc @centril ralf found "the next hole". That didn't take very long

Dear lord. =P

Closing in favor of https://github.com/rust-lang/unsafe-code-guidelines/issues/189: this is first and foremost a question of "what is the spec", not how to implement it.


SAK-31135 gradebookng datepicker

The tilte "bug fix" should be better articulated.

This is going to 11.x or master?

This is for 11.x

Also, issue #2176 seems to address this bug.

Shouldn't this go into master as well though? Generally we go to master first then back to 11.x. Is there a similar fix for master?

Hi @steveswinsburg, we've just recently fixed the Bootstrap button styling via PR https://github.com/sakaiproject/sakai/pull/2507.  This has been merged into master.
I notice there are also some JavaScript changes in this commit, I'm not aware if those are required?

Thanks @payten, @joejacob could you rebase and check if this is still needed? Cheers.

Two outstanding questions from me above:

This is a direct merge to 11. Standard practice is to go to master first, then to the release branch.
Is this still required, post #2507?

@joejacob could you please review? Cheers 👍 🍰 🐨

Closing due to lack of response.


Changing to legacy safety mode for Stinger

Stinger faults out with new safety code introduced in 0.7.7

Can you include a test route in test_car_models.py? We have tests that would have caught this is we had a test route.

Thanks for the PR, made the changes in 360cd7d8a86460fcc339ead89c0c6a439417eb66 along with the added test route.


The calculation of open-set F-measure

Hi, I wonder if true positive, false positive and false negative are counted correctly.
https://github.com/zhmiao/OpenLongTailRecognition-OLTR/blob/4a1f4009921b1c99029bfda151915058ff086a51/utils.py#L86-L89
Here are some examples according to the above code:
(pairs of prediction and label)

class_a, class_a (TP)
class_b, class_a (FP)
-1,  class_a (?)
class_a, -1 (FN)
-1, -1 (?)

I'm confused about

why the 2nd example is counted as FP rather than FN? (FP means the label is negative, but the prediction is positive, so what is positive here)
why the 3nd example is not counted as FN?
is the last example TN or TP?

Hello @drcege , thanks for asking. We followed this paper: https://arxiv.org/abs/1511.06233 for the F-measure calculation. It was modified from the original equation for better openset evaluation.

@zhmiao @drcege I also feel confused on this issue, I suggest authors @zhmiao can explain it clearly rather than re-direct readers to refer other literature since this criterion is so important for this manuscript.

@zhmiao I further seek related information from https://arxiv.org/abs/1511.06233，however， I got nothing explain, may you pay some attention to explain this issue？

@drcege hi, have you understand this issue, I have the same confusing feeling as you, may you share you newest understanding？

Searching for "open-set f-measure" I just found this thread. Here are my thoughts on this issue. Many works on open-set recognition use f-measure but the authors do not specify how the metric is calculated. Sometimes it seems to me that the conclusions presented in some works on open-set recognition are just non-reliable due to the metric employed. How to be sure that the employed metric is measuring a better open-set behavior? It also seems that some authors do not mind if the metric is really capturing a better behavior of the classifier. I do not know details of the work associated to this repository, anyhow, I agree with the concerns raised by @drcege. If any of you are interested in evaluation metrics specially proposed for open-set recognition, take a look in a paper in which we talk about this issue [1]. There in Section 4.1 we propose the open-set f-measure and the normalized accuracy, both to be employed on open-set problems.
[1] Mendes Júnior, P. R. and de Souza, R. M. and Werneck, R. de O. and Stein, B. V. and Pazinato, D. V. and de Almeida, W. R. and Penatti, O. A. B. and Torres, R. da S. and Rocha, A. de R. (2017). "Nearest Neighbors Distance Ratio Open-Set Classifier". Machine Learning, 106, 359–386. .


WIP: GUACAMOLE-406: Add documentation for event listener support

I think this is at a point where some review would be helpful. Would appreciate feedback on the content itself (especially level of detail, illustrative value of the examples, etc), placement in the user manual structure, as well as any syntactic issues.
There is some repetition between this section and the section on custom authentication providers. In particular, the instructions for installing a custom extension are duplicated almost verbatim. Perhaps this would be better addressed in the chapter on guacamole-ext or elsewhere.
Much of the skeleton for the example itself is also very similar to the authentication provider example, but I think this is probably to be expected and may not be a bad thing.

Not sure why I still have commits for GUACAMOLE-409 showing up here. Will rebase to clean that up.


Login by PIN codes & QR codes

Instead of typing in username/password of OW4 user account (which might be very complicated if you have a complex password and are used to a password manager), we should look into simple ways to log a person in.
This can be done by showing a unique code to the user logging in, and make them log in and authenticate that login from another device (like how Youtube TV makes you authenticate on a TV). This could further be simplified by generating a URL that will authorize the login and show this as a QR code which the user can scan with their phone and thereby log in.
This would also reduce the need for an on-screen keyboard.
Note: This would also have to be implemented in dotkom/onlineweb4.

Was implemented in #57

Not really, that feature simplified the registration process, not the log
in process.
On Wed, Aug 29, 2018, 15:45 Jarle Trollebø<EMAIL_ADDRESS>wrote:

Was implemented in #57 https://github.com/dotkom/Nibble2/pull/57
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dotkom/Nibble2/issues/51#issuecomment-416958993, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFK961E_0msrSwIDr4BP4TmugOC5fNUHks5uVpsCgaJpZM4Nvgrg
.

Oops, my bad.


Publish the DicomWebClient package in Nuget

User story
As a DICOM Developer, I want to be able do use the DicomWebClient library to access the server so that I don't have to write the code for that.
Acceptance criteria

When I do dotnet package add Microsoft.Health.Dicom.Client, then the package is downloaded.

Andre, thank you so much for your issue.  Looping in @brandonpollett.

Thanks for the input. Right now the web client is published to a internal feed. To workaround you can build the project or download it from the latest CI published artifacts.


Focus of menu item "Visits" disappears when I view the details

🐛 Bug Report
A clear and concise description of what the bug is.
To Reproduce
When I view a visit detail tab, the focus of the "Visits" menu item disappears preventing me from understanding in which section I'm in.
Right behavior

Wrong behavior

I would like to take this as my first contribution

Ok @agusbrand


Query unit tests

Create Unit tests for query service using jupiter and surefire framework

As a developer,
I want to create unit tests for the query workflow using Jupiter,
So that I can ensure that the query workflow is functioning correctly and avoid introducing any regressions.
Acceptance Criteria:
The system should provide a set of unit tests that cover the various aspects of the query workflow.
The unit tests should be written using the Jupiter testing framework.
The unit tests should cover scenarios such as:
Querying for all cities in a region, sorted by population from largest to smallest.
Querying for the top N populated cities in a region, where N is provided by the user.
The unit tests should use a test database with pre-populated data to ensure that the queries are working correctly.
The unit tests should use Mockito to mock any dependencies, such as the database connection.
The unit tests should provide clear and informative error messages in the event of a failure.
The unit tests should be maintained and updated as the query workflow evolves.


[bug] android dev - "Waiting for your frontend dev server to start on..."

Describe the bug
I've been working on a Tauri v1 project, and then decided to try v2 beta, so I also created a Tauri v2 project and had the hello world example working there (showing the 'greet' UI).
I then made a copy of my Tauri v1 project and have migrated it to v2 beta and have it working for desktop.
I then decided to try the cargo tauri android dev on the migrated project. I had some issues with it complaining about the app identifier because it contains '-' but eventually cleared those by deleting src-tauri/gen and re-doing cargo tauri android init.
Now, when I do cargo tauri android dev, I get a strange result. Firstly, the emulator starts as normal, but shows the 'greet' example, not my app. And in the console I get the following:
VITE v5.2.6  ready in 722 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: use --host to expose
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...

I don't understand why it is showing the 'greet' example. http://<IP_ADDRESS>:5173/ is kindof correct in that this is the local IP of my machine, but I would expect it to us 'localhost'. But anyway, why is the dev server not starting?
I have my app working on desktop, and it feels like I'm very close to having it working on android, but I'm not sure what is going on here. I haven't done android development before so not sure what might cause this.
Reproduction
No response
Expected behavior
My app loads into the Android emulator.
Full tauri info output
cargo tauri info

[✔] Environment
    - OS: Ubuntu 24.4.0 X64
    ✔ webkit2gtk-4.1: 2.44.0
    ✔ rsvg2: 2.58.0
    ✔ rustc: 1.78.0 (9b00956e5 2024-04-29)
    ✔ cargo: 1.78.0 (54d8815d0 2024-03-26)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-unknown-linux-gnu (environment override by RUSTUP_TOOLCHAIN)
    - node: 22.1.0
    - yarn: 1.22.5
    - npm: 10.7.0

[-] Packages
    - tauri [RUST]: 2.0.0-beta.19
    - tauri-build [RUST]: 2.0.0-beta.15
    - wry [RUST]: 0.39.5
    - tao [RUST]: 0.28.0
    - tauri-cli [RUST]: 2.0.0-beta.12
    - @tauri-apps/api [NPM]: 2.0.0-beta.11
    - @tauri-apps/cli [NPM]: 2.0.0-beta.12

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../build
    - devUrl: http://localhost:5173/
    - framework: Svelte
    - bundler: Vite

Stack trace
No response
Additional context
No response

From the logs, it seems that Vite is only started on localhost and not on internal IPs, and that's why mobile dev server can't be found. You need something like this https://github.com/tauri-apps/create-tauri-app/blob/2e35db4af381a7955bd56fef8ad27ecc76b16151/templates/template-svelte/vite.config.js.lte#L19-L26

that this is the local IP of my machine, but I would expect it to us 'localhost'.

localhost is local to your computer which is different from the localhost of the android device, so it won't be accessible, this is why you need to expose your vite dev server on internal IPs so the android device can use on of these IPs to connect.
I know the documentation is lacking on this atm but will be available on https://beta.tauri.app/start/frontend-configuration/vite/ before v2 hits stable.

@amrbashir Thanks for the hints.
There seem to be things used there that aren't in place with my installation. Firstly the {% if mobile %} seem to be conditionals for a pre-processor, but cause errors, so I removed them and just left the mobile settings in that section which now looks like this:
	server: {
		port: 1420,
		strictPort: true,
		host: "<IP_ADDRESS>",
		hmr: {
			protocol: "ws",
			host: await internalIpV4(),
			port: 1421,
		},
		watch: {
			// 3. tell vite to ignore watching `src-tauri`
			ignored: ["**/src-tauri/**"],
		},
	},

That gets a little further but vite complains that internalIpV4 is not defined so the server start fails.
I think this issue is due to migration from Tauri v1 because looking in a clean Tauri v2 app the vite.config.js is indeed like the one you suggested, whereas my migrated v1 app just loads the svelte plugin.
So I guess there are a few things missing in the migrated project:

something that will handle {% if mobile %}  in the vite.config.js
whatever provides interalIpV4

And possibly some other things. Can you suggest the further migration steps that might solve the above?
I can't find an issue that covers this for migration so perhaps it should be re-opened or referenced in a new issue?

something that will handle {% if mobile %}  in the vite.config.js

you can ignore that, it is a template pre-processor in create-tauri-app, the snippet you showed above is correct

whatever provides interalIpV4

You will just need to install internal-ip npm package, make sure to use 7.0.0 as later versions had problems.
Then you can import it import { interalIpV4 } from "internal-ip"

you can ignore that, it is a template pre-processor in create-tauri-app, the snippet you showed above is correct

I want to build for non-mobile too, so if I hard-code for mobile that won't work. I'd like to migrate my v1 app fully to v2 beta, and some steps are incomplete. So while your suggestions may fix the Android problem I assume they will break my desktop builds.

I see, you can just copy this vite config which handles both desktop and mobile
import { defineConfig } from "vite";
import { svelte } from "@sveltejs/vite-plugin-svelte";
import { internalIpV4 } from "internal-ip";

// @ts-expect-error process is a nodejs global
const mobile = !!/android|ios/.exec(process.env.TAURI_ENV_PLATFORM);

// https://vitejs.dev/config/
export default defineConfig(async () => ({
  plugins: [svelte()],

  // Vite options tailored for Tauri development and only applied in `tauri dev` or `tauri build`
  //
  // 1. prevent vite from obscuring rust errors
  clearScreen: false,
  // 2. tauri expects a fixed port, fail if that port is not available
  server: {
    port: 1420,
    strictPort: true,
    host: mobile ? "<IP_ADDRESS>" : false,
    hmr: mobile
      ? {
          protocol: "ws",
          host: await internalIpV4(),
          port: 1421,
        }
      : undefined,
    watch: {
      // 3. tell vite to ignore watching `src-tauri`
      ignored: ["**/src-tauri/**"],
    },
  },
}));

Thanks for your help with this but I'm still getting the same error having pasted the above as it is in to vite.config.json:
VITE v5.2.6  ready in 424 ms

  ➜  Local:   http://localhost:1420/
  ➜  Network: http://<IP_ADDRESS>:1420/
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...
    Warn Waiting for your frontend dev server to start on http://<IP_ADDRESS>:5173/...

the above example use port 1420 while your tauri.conf.json has 5173, just change one of them

port    problem


feat(serviceworkers): add basic support for Service Workers

After figuring out how Service Workers are being handeld by Chrome DevTools, I created this PR which adds basic support for Service Workers. It allows users to get all Service Workers running in the browser, receive all requests which the Service Worker performs (including his own sw.js request), and listen for ServiceWorkerRegistration's scopeURL and status.
Example:
const puppeteer = require('puppeteer');

function logRequest(interceptedRequest) {
  console.log('A request was made by the ServiceWorker:', interceptedRequest.url());
}

puppeteer.launch().then(async browser => {
  const page = await browser.newPage();
  page.on('serviceworkercreated', serviceWorker => {
    serviceWorker.on('request', logRequest);
  });

  await page.goto('https://service-worker-example.org/');
  await browser.close();
});

Unfortunately, I was not sure how to test this feature and would appreciate help with it. Also, I did not implement code coverage or stopping.
Addresses #2634.

Hey @ksm2, thanks for the PR.
The main issue we're having with Service Workers is that we don't know yet the user scenarios. Your PR exposes a lot of APIs - is this really needed?
Can we start small, starting from the most painful testing usecases that will bring the most value to our users and expand further from there?

Hey @aslushnikov, thanks for your reply! Sure, I just wanted to include into my branch everything I found out using the Chrome DevTools Protocol how Chrome actually handles Service Workers. From our use case, the following would be most important:

Inspecting requests the Service Worker executes (including its own sw.js request)
Getting events when a Service Worker is getting active
Retrieve a Service Worker's scope

At Baqend, we have a Service Worker which is its own product. We use Puppeteer to detect whether our users have installed the Service Worker correctly, so we could need the features mentioned above. I guess my PR is not adding a lot of APIs on top, because the way Service Workers work in Chrome is very complicated. If you have any ideas with which things to start, I am open to any changes!


WIP: Update Icons documentation

Updated readme.md for icons.

Please read that and suggest more improvements (if necessary). cc @darkwindcz @williamkolmacka

We can drop this file and merge documentation with this file. It was added in #224

Can you replace link in "list of icons" to https://orbit.kiwi/visual-style/icons/#icon-list?

Icon list on orbit.kiwi contains different names (not exact component name).
I want to replace that link in the future with stuff from #207 and get rid off PNGs in #215

Icons file readme.md deleted in #207


Add unique identifier white paper

I will be out of the office starting  04/23/2015 and will not return until
04/28/2015.
I will respond to your message when I return.


Create a function that takes a generated cohort and attaches Addresses from an OMOP database

getAddresses should take the output of CohortGenerator and automatically attach addresses (and coordinates, if they exist)
Returns an augmented Cohort table for use in geocodeCohort

Should the GIS package include an example set of cohort json and sql? Similar to what is done in Cohort Diagnostics (here)?

Fixed by 479701b


Allow limiting the image resolution

Description
When using large images that are scaled down, the pdf size will not decrease by downscaling. There should be an option to limit the resolution of pixel images in the resulting PDF.

This could be an option to image which could be set globally or locally like set image(max-res: 300)
or some kind of pdf export setting for the cli.

When talking about that topic, we could also think of other related options like image compression settings in pdf.
Use Case
It might be useful for everyone who uses high-res images in their typst document, especially when they face pdf file size restrictions (maybe for submission).

I read this in #404 a couple of days ago:

We plan to add support for instructing Typst to recompress images at a specific DPI level as an export option in the future.

Nice to read that, but is there any progress on this topic, since?


Fix vagrant dns problems [skip ci]

Bump to Kubernetes 1.5.4 and Weave 1.9.3
Use eth1 with well known IPs to wire Kubernetes masters and nodes
Swich to proxy mode userspace, to work around https://github.com/kubernetes/kubernetes/issues/34101

This solves two things:

works around the glitches people were sometimes seeing, like the node not being able to resolve master, or that the node can't reach the master ip.
We can now distribute a pre-deployed vagrant image to atlas (e.g. for demoing migrations), since the certificates generated by kubeadm, are now bound to the well known IP of eth1

@admiyo could you also please verify this change? Thx :)

ok to test

retest this please