Work in animations: Move and rotate

Create animations based on move and rotate operations. Let's try to understand the limits.
Minecraft also has lots of animations. Let's learn from the also.

Other animations:

Change size
Appear disappear (sand fall down removing its base)
...

The current unbuild is pretty accurate, but sometimes, probably it is enough to fill with air the bounding box, which will be much faster.

Interesting search: animating voxels. https://www.reddit.com/r/VoxelGameDev/comments/40p0vc/how_do_you_animate_voxel_models/ Time to read!

Ok, let's focus in the spaceship landing. The first thing we need is a spaceship. We can create one or reuse a Schematic already done. In this case, to have a quick first experience, let's find a schematic:
https://www.minecraft-schematics.com/category/flying-machines/
We need a ship that it is not pretty large so we can move it softly in Minecraft.

https://www.minecraft-schematics.com/schematic/6939/
https://www.minecraft-schematics.com/schematic/5340/ (movecraft plugin)
https://www.minecraft-schematics.com/schematic/3457/
https://www.minecraft-schematics.com/schematic/3326/ (working doors)
https://www.minecraft-schematics.com/schematic/3068/ (movecraft plugin, video about flights wars)

Ok, all done in a first iteration, specially with issue #97

Nice reference https://www.mixamo.com/#/ found https://github.com/ndarville/voxels


calendar service

allow to change the weekly schedule directly in the calendar. Events will only be shown for the next 7 days.


glitchy scroll on menu

Both http://www.christianalfoni.com/cerebral/  and http://www.cerebraljs.com/ has glitchy scroll in menu.

+1
reset the scroll on menu change. It gets irritating to scroll back to top every time I change the left nav. Also scroll bar is not visible, it is because of the
.page {
    height: 100%;
}

but removing that makes the whole left nav also scroll, I can figure out a fix and PR it if you don't already have it addressed.

Haven't touched it yet. Pr are welcome

I fixed the scrolling back to top thingy :-) Not sure what glitchy is referring to?

Closing it for now :-)

But it still glitchy :)

I mean scroolbar in sidebar nav. Scrollbar is present but it overflows down under the screen edge (only top arrow seen, scrollbar disabled if it nothing to scroll). Good news is mousewheel scrolling still scrolls sidebar container.

Hehe, I am on a mac, can not see the scrollbar :p

Need help to fix this one :-)

thanks @NervosaX

Very happy to fix this, it was bothering me a lot.


Update License to MIT

The original repo this project was forked from is updating it's license to MIT. See https://github.com/yaronn/xpath.js/issues/20.
Currently, in xpath.js, the README and package.json are out of sync. This is being resolved on
https://github.com/yaronn/xpath.js/pull/21.

@MRamos62786 Thanks. I recently noticed that the license on the parent repo was changed, but wasn't sure if the library's original creator had approved the change. I now see that he has.

@JLRishe Thanks, any idea when this will be published to npm?

@MRamos62786 Just published now. Thanks.


Modularize routes & add Root component

Description:

Setup the routing so that the AppContainer appears on every page and we have a standardized way to add new routes later

Files Changed

'src/main.tsx' - remove the route object array from here to define it in 'routes.tsx' instead. Also import that Routes and pass into createBrowserRouter()
'routes/routes.tsx' - define a Root component that will be the entry Component for our app & also export an array of route objects that will be passed into the createBrowserRouter() function in 'main.tsx'

Moving forward, any new routes/paths we need to define can be added to the children array in the routes.tsx file

Approved ✅


Support deserializing into array of different dtype

This PR makes deserializers support loading into arrays with different dtypes. It enables us to choose the dtype used in inference independently from the dtype used in training.

Seems like tests are failing when ndarrays of different dtypes (deserialized data in float16 with original data in float32) are compared for equality. Could you take a look?

Fixed.

jenkins, test this please.

Should we backport this enhancement?

I think it's good to backport it.

jenkins, test this please.

jenkins, test this please.

jenkins, test this please.

jenkins, test this please.

LGTM!


Support agent-groups and agents endpoints

Critical for programmatic deployment and management of Tenable.io agents.
The {scanner_id} value must always be 1.
Example:
GET /scanners/1/agent-groups HTTP/1.1
Host: cloud.tenable.com
X-Cookie: token=REDACTED_DONT_EVEN_TRY
Cache-Control: no-cache
Note the "1" after "/scanners/". That's what is hardcoded. "0" doesn't work. "2" doesn't work. "1" does.

Submitted pull request for agent and agent-groups endpoints. Please review.

Thanks @veedam , we already have completed this work and will be merging it shortly. Thank you for your contribution.

@veedam thank you for your contribution, we've actually completed this work already and it's not merged.

Resolved by:
3c417f6a
e63ca165


fix: Menu outside of the screen in FluentMenu

🐛 Bug Report
Depending on the screen size and menu item labels lenght, the FluentMenu may go outside the screen bounds.
💻 Repro or Code Sample
This is observable in the example on the demo page: https://www.fluentui-blazor.net/Menu.
🤔 Expected Behavior
View the entire menu and its menu items.
😯 Current Behavior
On small screens, such as on mobile devices, when menu item labels are too long, the FluentAnchoredRegion related to the Menu go outside of the screen.

This also happens for nested menus for menu items that have MenuItems property set.

💁 Possible Solution

Repositioning the inner FluentAnchoredRegion to the top/bottom when screen width doesn't allow to show inner Menu next to the main menu.
Set max width to FluentAnchoredRegion to allow menu item labels to display on multiple lines.

🌍 Your Environment

Microsoft Edge, Google Chrome, Apple Safari
.NET 8 and Fluent UI Blazor library Version 4.7.2 and 4.8.0

Hi,
There is only an anchored region if the menu is anchored to a specific element.
The long text is something I feel you as the developer need to control yourself.
With regards to the nested menu, I would strongly advice to not use that in a mobile display variant. On a touch screen device it will be virtually impossible topen the sub menu as you do not have the hover capability and as a click/touch event on the main items will make that selection and close the menu itself.
You could use or Grid to display/hide certain parts of the ui based on display size or do it yourself based on media queries ind other standard CSS capabilities.


Not sure if related to this or the GUI created for it but it appeared to be missing some files.

I used this with GOGcli-GUI
https://github.com/Twombs/GOGcli-GUI
Not sure which was the cause, but I noticed with Fallout 4, it refused to pull up all the installer files, I think it was missing part 2 of the multiple parts, didn't matter how many times I refreshed, it still seemed to miss it.

I used this with GOGcli-GUI https://github.com/Twombs/GOGcli-GUI
Not sure which was the cause, but I noticed with Fallout 4, it refused to pull up all the installer files, I think it was missing part 2 of the multiple parts, didn't matter how many times I refreshed, it still seemed to miss it.

I have the game downloaded so gogcli is probably ok with it, but I'll double-check anyways this weekend.

Actually, I just took a quick look now.
I generated a manifest and all the files where listed. I then applied the manifest to a local file storage and all the files where also listed in the action list.
I didn't download the files (the game is pretty large), but at this point, I validated gogcli quite a bit with my game collection (which is rather large) and can say with confidence that if a game file is present in the actions to add/update, one of two things will happen:

It will get downloaded
If the download fails repeatedly, the command to execute actions will exist in failure and you'll be able to resume exactly where it failed at a later point in time.

Sorry, meant to update this. It actually corrected itself at a later date. No clue what the issue was or why, but a day or two later it started showing all the files just fine.

Ok, I'll close this issue. If possible when you experience issues in the future, try to use gogcli directly in the terminal to see if it is the source of the problem or not.
In my case, I typed the following commands:
gogcli manifest generate --title="Fallout 4"
# Looked at manifest file and determined all game files were in it
gogcli storage apply manifest -k fs
# Looked at actions file and determined all game files were present as actions to add

On windows, the file would be "gogcli.exe".


Homarus FFMpeg fails on filepaths requiring authentication.

Homarus gets the filepath of the asset requesting a derivative from the Apix-Ldp-Resource header, then takes that filepath as the source argument for FFMpeg.
If that file is not available to an anonymous user the command will fail with a 403, causing the command to fail.  FFMpeg file request will have to be qualified somehow.

closed with https://github.com/Islandora/Crayfish/pull/97


WebSocket Stopped Working on Android 5 API21

FATAL EXCEPTION: WebSocketConnectReadThread-280
java.lang.NoSuchMethodError: No virtual method setEndpointIdentificationAlgorithm(Ljava/lang/String;)V in class Ljavax/net/ssl/SSLParameters; or its super classes (declaration of 'javax.net.ssl.SSLParameters' appears in /system/framework/core-libart.jar)
I did All configs as in medium explained
https://medium.com/codex/android-tutorial-part-2-using-java-websocket-with-kotlin-dd3105bd3eed

I did All configs as in medium explained
https://medium.com/codex/android-tutorial-part-2-using-java-websocket-with-kotlin-dd3105bd3eed

See https://github.com/TooTallNate/Java-WebSocket/wiki/No-such-method-error-setEndpointIdentificationAlgorithm

exactly I found it yesterday and problem solved
thanks anyway for your reply


Question regarding the phase alignment

Thank you for sharing your impressive work!
I have two questions about your phase alignment implementation in this line of code: https://github.com/HaokunGUI/VQ_MTM/blob/6912745920397c92d3f33f9585bb690da470f365/layers/Quantize.py#L25.

Could you explain why you use phase[:, :, 1:2] for alignment, particularly why the first index is chosen? My background in signal processing is limited, so a detailed explanation would be really helpful.
Also, how does multiplying the self.linear aligns the phase?

Thanks in advance!

Thank you for your insightful questions.
Regarding your first question, which is discussed in detail in the open review, we use phase[:, :, 1:2] because $\arg(\hat{z_{1}})$ corresponds to the unit phase shift $2\pi \tau / N$ according to Equation (10), whereas the phase of $\hat{z}_{0}$ is zero. This choice ensures proper phase alignment.
As for your second question, multiplying by self.linear aligns the phase as indicated by Equation (10).
If you have any further questions or need additional clarification, please feel free to ask.
Thanks again!


SMTP-STARTTLS proxying with STARTTLS handled by Traefik

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
Support SMTP-STARTTLS proxying. Traefik would accept connections on port 25, handle encryption (via SMTP-STARTTLS) and proxy the protocol as plain non-encrypted SMTP to a mail server (MTA). Note that unlike TCP proxying, this would involve rewriting parts of the SMTP protocol (at least the "welcome phase").
Note that sadly there is no SMTPS (SMTP via TLS). It has been revoked by the IANA in favour of SMTP-STARTTLS, and SMTPS is almost never tried by other MTAs. So Traefik's TLS support can't be used for SMTP.
SMTP is the only widely used protocol that currently isn't handled by Traefik. This means if you're running a mix of services (e.g. mail server and web server) SSL certificates are handled by different entities (e.g. certbot for mail, traefik for http). It would be nice if Traefik could uniformly proxy all common services, greatly simplifying such setups.
Note also that nginx supports this special case.

@frederikbosch  if you use TCP, you can already handle smtp to a mta, this issue is about implementing smtp protocol on top of tcp, so traefik acts as an smtpd server and handle tls.
If you use TCP to forward SMTP to a MTA, the client IP is currently lost, so you may have an open relay or cannot use spam ip blacklists. For this i have opened a pull req for supporting proxy protocol to the upstream mta in traefik to keep the client ip.

@mschneider82 Thank you for the clarification, and good work on the PR, hopefully it will be merged anytime soon!

I'm very interesting for solution for PostgreSQL, when will it arrive ?

I'm very interesting for solution for PostgreSQL, when will it arrive ?

@nounoursPand  same here

I would like to know if any progress is planned for this feature as it is an important feature for me as a user of Traefik and it's been a year since any real input on this issue, but of course, this is 'disrespectful' for me to do so.

Hi,
a feature request I open (#8971) has been marked as duplicate of this one.
So if I understand correctly, part of the feature here request solves also the support for databases as Postgres and services as PgBouncer: StartTls would suffice, whilst for supporting SMTP also some other improvements are required.
If so, maybe the issue title could be rewritten in a more general way? To let people understand at glance that this solves also the issues with Postgres? Someting like "Support of TLS (STARTTLS) for SMTP and Postgres"?
Regards.

To those coming from the Postgres-related tickets:
I limited this feature request to SMTP-STARTTLS, because there is no realistic alternative (see description) and email being a federated system this isn't going to change anytime soon.
For postgres, I assume (but do not know) there is a tunneled TLS alternative ("PSQL over TLS") available that is supported by the server and popular clients? If so, it is much preferred to migrate to using that, and stop using STARTTLS.
Of course, it is up to the implementor if s/he would like to support all STARTTLS variants, or only a subset of it.

I just found out there is actually an open PR to implement PSQL-STARTTLS: https://github.com/traefik/traefik/pull/8935
You might want to subscribe there.

Hi,
thank you, I subscribed.
FYI, the most famous answer for Postgres is to use additional tools as STUNNEL, which provides actual TLS connections.
But IMHO it seems an additional complication (one more service to configure and manage), which also seems to lack the additional benefits of using certificates served by Traefik (for example, automatic valid certificates generation and renewal).
So I still hope that Traefik will support as many STARTTLS variants as possible.
Regards.

Hello @schildbach @FStefanni,
I've just reopened #8971, to track the support for Postgres STARTTLS independently, as mentioned in https://github.com/traefik/traefik/issues/8971#issuecomment-1112138507.

Hello,
The SMTP STARTTLS flavor (as well as the MySQL one) does not rely on the same negotiation as the Postgres one.
Postgres STARTTLS negotiation is initiated by a frame/msg from the client, and by doing so, this message can be handled before Traefik needs to identify the backend to proxy to.
This allows Traefik to mock a Postgres server response, indicating that TLS is available, and to proxy to the correct backend by reading the subsequent clientHello from the client.
The support of Postgres STARTTLS is achieved by PR#
On the other hand, a STARTTLS flavor that relies on a "server first" negotiation cannot be handled on the same entryPoint as every other routing configuration, because Traefik would then need to initiate the STARTTLS negotiation for any incoming connection, and we think that this would be breaking/misleading for most users.
As a consequence, the SMTP STARTTLS flavor cannot be implemented as of now, because it does not fit within the current routing mechanism.
As a potential solution, we were thinking that in the future we may add the ability to "type" entryPoints and make them support only specific protocols.
In the meantime, SMTP STARTTLS can still be handled as any "server first" TCP protocol, with exclusively non-TLS TCP routers (see this documentation).
For reference, we opened https://github.com/traefik/traefik/pull/9377 to support Postgres STARTTLS.

I think the type entryPoint would be the correct way. To implement SMTP i think this would be the correct way:
Traefik should act as a SMTP Server and simulate the begin of an SMTP Session as well as the TLS handshake, Example:
telnet traefikserver 25
traefik open a tcp server (without any backend connection) and replies with:
220 traefik.hostname.localhost ESMTP Traefik
A client could do an extented helo EHLO command to see what features traefik supports:
the reply of traefik should contain a (configurable list) and STARTTLS:
for example:
EHLO example.com
250-traefik.hostname.localhost
250-PIPELINING <-- this is the supported feat list from configuration
250-SIZE 10240000
250-VRFY
250-ETRN
250-ENHANCEDSTATUSCODES
250-8BITMIME
250-DSN
250-SMTPUTF8
250 CHUNKING
250-STARTTLS   <--- this has to be injected

once the client initiates a STARTTLS command, traefik could do the handshake and proxy everything else beginning with "MAIL FROM" to the backend server, it has to skip the intro line and simulate the commands.
if the client goes on with MAIL FROM instead of doing STARTTLS it could be bypassed to the backend without TLS. (could be a configurable force/opportunistic TLS Mode...)
So traefik should also support SMTP client to the backend as well as proxyProtocol support and maybe XCLIENT
a while ago i started with an prototype implementation, but once there is a type support it would be a good way to integrate this.

Note that sadly there is no SMTPS (SMTP via TLS). It has been revoked by the IANA in favour of SMTP-STARTTLS
SMTPS is almost never tried by other MTAs. So Traefik's TLS support can't be used for SMTP.

SMTPS wasn't really a thing for very long IIRC (didn't actually become standardized). See this "Port 25" section from a comment of mine. The wikipedia article seems to reference this too if that wasn't clear (or if the article lacked that information in 2020).
Mail delivery is quite different than most client => server connections.

Often there can be more than one hop involved, there is no end-to-end connection guarantee.
Establishing TLS between two MTAs to deliver mail only provides assurances that the connection is secure between that single hop. That's the extent of your control, and then it's wishful thinking should any additional hops be involved.
Same with receiving mail, it may be a secure connection from the client MTA, but that could be a service like SendGrid where it may have been relayed insecurely prior to reaching SendGrid.

However, there is implicit TLS for mail submission and retrieval from the direct mail service.

Submit mail to be sent via port 465 (the port originally assigned for SMTPS, promptly deprecated, then revived back in 2018 with this new assignment as the secure submission port).
Retrieve mail over port 993 (IMAPS) or 995 (POP3S).

That leaves port 25 for external MTAs, to receive inbound mail from or send outbound mail to. Ideally they upgrade to a TLS connection explicitly via StartTLS, but it is not required.
TLS being optional for clients is mandated by RFC 2487 / 3207:

A publicly-referenced SMTP server MUST NOT require use of the STARTTLS extension in order to deliver mail locally.
This rule prevents the STARTTLS extension from damaging the interoperability of the Internet's SMTP infrastructure.
A publicly-referenced SMTP server is an SMTP server which runs on port 25 of an Internet host listed in the MX record (or A record if an MX record is not present) for the domain name on the right hand side of an Internet mail address.

You may also have some interest in:

RFC 8689 proposing REQUIRETLS service extension, but it seems to have stalled?
RFC 8461 MTA-STS.

This means if you're running a mix of services (e.g. mail server and web server) SSL certificates are handled by different entities (e.g. certbot for mail, traefik for http).
It would be nice if Traefik could uniformly proxy all common services, greatly simplifying such setups.

FWIW docker-mailserver (DMS) has integration with Traefik via acme.json extraction, or providing paths to public and private keys depending how you configured Traefik.
That said, there has been several support requests very recently from Traefik users struggling to configure Traefik to work with DMS. None of it seems to be an issue with DMS itself, but we could both probably improve our docs at each end regarding Traefik with mail services?


Renaming Token

Branching of #901. Renaming Token to EventedId event::Id (in a new event module`).

What's the motivation for changing it? Token works for me as it is short and distinct, and easy to talk about. Also it fits the meaning of a 'token', i.e. something short standing in for something else bigger. EventedId is quite weird to pronounce and not that distinct to the eye. (Evented itself also seems weird to me, because there is no verb "to event", and they are referred to as handles in the API.)

@uazu The first line of documentation of Token reads: "Associates readiness notifications with Evented handles." To me this doesn't say token, but identifier hence token -> id. But @carllerche made a good point that event::Id was better then EventedId, which I'm in favour of as well. I've also updated the first comment to reflect this.
With this change you can use event::Id or even Id, which is even shorter then Token. I also think the pronunciation is also solved then. @uazu Would you still be in favour of Token of event::Id?

The whole thing is a muddle of different terms right now, but fortunately it is a very small API so it's workable. However I find Token easy to understand, and Evented not easy to understand. I guess that Token lead to the name tokio, so it has history. Yes, everything ID-like could be an ID, but "token" also works logically, and having less tokens around than IDs makes it easier to discuss. Also this is the ID of an event source, not of an individual event, so I'm not sure that event::Id conveys the meaning any better.
I was just looking at what these things are called in the underlying sys calls. Token is just called "data" in epoll and kqueue, which is less distinct than either ID or Token. Evented is just a "file descriptor". It appears that "handle" is not used now in the man pages, but is still commonly understood, and Windows uses that. UNIX has a lot of IDs that aren't called IDs -- so I think there is precedent for using other terms than ID (like "number" or "descriptor" or "key" or "handle").
Personally, I feel that having distinct terms within a local region of code helps understanding and discussion. But really this is very localized. It doesn't affect a lot of code, because the Tokens get wrapped in higher-level abstractions. Still, the first thing I learned about mio was that is was token-based. It seems strange to drop one of the clearer and more distinct names for something more generic.

@uazu What you are saying does resonate with me.
Do you have thoughts about what would be a better name than Evented?

@carllerche I hope I wasn't being too critical. For Evented, some possibilities:

Source or EventSource could express that it is logically a source of readiness events.
Handle could express that it wraps a handle to an underlying OS object, and matches existing language surrounding Evented
Registrable could express something that can be registered with poll, and matches the trait methods

Those are the best ideas I've found.

I was also confused by Evented on my first encounter - it sounds like something that has been "evented", while it really is something that is a source of events.
My personal favorite is thus Source. Registrable sounds okay too.

I also agree with Source, maybe in th event module, so you can write event::Source? If people agree I can make a pr.

I think I like Source, but let me ping a few more people to get feedback.

I think for now it's best to leave the Token name and instead rename Evented (#1003).

Sounds good to me :+1: I will close this.


Fix a bunch of mistakes around backticks

Add missing space, fix misplaced backtick and some other misc edits.

@Rageking8 - Thanks for these typo fixes!


encode script before writing out

I have fixed it slightly differently in PR #431.

But thanks for pointing me to the issue!


AM2320: Single bus connection

The AM2320 provides not only I2C but can be connected also via single wire.
Do you plan to support this mode too?

Thanks for the question. You're the first to ask.
It is not planned for now.
Is the protocol described in the/a datasheet?
Does the protocol looks like e.g like the DHT22?
If time permits I might dive into it.

It's the same protocol as used by DHT22. It's described in detail in the DS.

Why not use my DHTNEW library then, and force the type to 22.

https://github.com/RobTillaart/DHTNew

It might be an idea to add new types to the DHTnew lib to explicit support the AM2320.
Did that also for the Sonoff Si7021

Note: datasheet page 20 section 8.3 Single bus communication (ONE-WIRE)

When I raising my questions, I was not aware yet that DHT22 and AM2320 use the same one-wire protocol (and AM2320 is also not listed under supported devices).
Now it's working fine!

Good to hear it works.
I will make a note of it in the readme.md.

I will add this section in the near future  to the readme.

One Wire Protocol
The AM2320, AM2321 and AM2322 can also be read by using a "OneWire" protocol.
This protocol is identical to the DHT22 and will only provide a temperature,
a humidity and a CRC. See issue #26.
If one wants to use this protocol, one can use https://github.com/RobTillaart/DHTNew
and optionally force the type to 22.


Overwrite track file with unique tracks

Closes #370.

Fix this by filtering duplicates from track file and re-writing track file with unique tracks so that tracks from track file are deleted in proper order.

This PR adds a new function internals.get_unique_tracks which, given path to track file will return a list of unique tracks from the track file. And also a couple of tests!

Tests are always good! :grin:

I also rebased this branch against master and should be ready to merge.

Awesome!


You can tear out windows so that they are off screen

[x] If you take a tile, and drag it to the bottom right of your desktop, the new window appears entirely off screen.

[x] i don't think the new windows should appear with it's top left corner at the mouse location - it should be offset.

For part 2, it might be easiest to just offset the created window by fixed x, y. The user's mouse is always going to be within a small rectangle when (where the tearout image is).
You can't track the mouse x, y relative to the window when beginning a tearout and use that. If you're tearing out the third favourite, the offset would put the mouse where the third favourite is. This also doesn't take into account scrolling

Closed by #439 and #446


Typo (use of @params instead of @param) in can.Model's documentation

@justinbmeyer and myself have added line comments pointing to the typo.  You can find the comments inline, at:
https://github.com/bitovi/canjs/commit/d1353f89ef83c2f7da90b6d0b317b45ea2b55568#commitcomment-10039773

These were all correct in master branch so they must have gotten fixed along the way.


Auth::driver() method does not exist in Laravel 5.2

Call to undefined method Illuminate\Auth\SessionGuard::driver()

Lumen 5.2 : Auth not found driver()


Unable to access the Keyvault linked App configuration in the client applications(console App/Azure Function App)

Hi,
I have created an Azure App Configuration Service and

Created a key value configuration
Created config keys that are linked to key vault secrets.

When I tried to refer the same in the client code. I am getting configuration that is key value based (first config), and I am not getting the secret value from the key vault for the second configuration
See images.

@sonalinoolkar
Thank you for your question. Can you please share with us the URL of the doc that you are having issues with?
Please note that we scope issues on this repro to feedback related to the docs. If you have a product question, please post it on Microsoft Q&A or Stack Overflow. Thank you for your understanding.

@sonalinoolkar Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


Fair Change Detection Benchmarking

Objective

#4972 introduce a benchmark to measure chang detection performance
However,it uses iter_batch  cause a lot of overhead in clone data to each routine closure(it feels like a bug initer_batch ) and constructs  new query in every iter.This overhead masks the real change detection throughput we want to measure. Instead of evaluating raw change detection, the benchmark ends up dominated by data cloning and allocation costs.

Solution

Use iter_batch_ref to reduce the benchmark overload
Use cached query to better reflect real-world usage scenarios.
Add more benmark

Changelog

This looks well motivated and the reasoning works out, but I'd like to see how much of an impact this has on the final benchmark numbers. This isn't to say the PR will be blocked on them changing in a particular direction, but it's best we show what impact this has.

I do  a benchmark in main and this pr,results are as  follows
main (entities from 50000 to 60000 ,the detection time increase 2x~3x,this appears to be an unreasonable outcome.)

pr(added the 60000 entities benchmark to  compare)

time increase linear with entity count


Use HTTP1 in HTTP1 pools

Otherwise Mint may try HTTP2 if available.

This has been merged into main.


ncurses: Fix static build.

Success on x86_64-linux (full log)
Attempted: ncurses
Partial log (click to expand)
rmdir: failed to remove '/nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin': Directory not empty
Moving /nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin/tput to /nix/store/4mmpv0cp9k5cn8zjbz3zs0lfpr3crzsx-ncurses-6.1/bin/tput
rmdir: failed to remove '/nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin': Directory not empty
Moving /nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin/tset to /nix/store/4mmpv0cp9k5cn8zjbz3zs0lfpr3crzsx-ncurses-6.1/bin/tset
rmdir: failed to remove '/nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin': Directory not empty
Moving /nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin/captoinfo to /nix/store/4mmpv0cp9k5cn8zjbz3zs0lfpr3crzsx-ncurses-6.1/bin/captoinfo
rmdir: failed to remove '/nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin': Directory not empty
Moving /nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin/infotocap to /nix/store/4mmpv0cp9k5cn8zjbz3zs0lfpr3crzsx-ncurses-6.1/bin/infotocap
rmdir: failed to remove '/nix/store/bf4ikgi8aal7wfcaizvycxaikyvy9xnx-ncurses-6.1-dev/bin': Directory not empty
/nix/store/4mmpv0cp9k5cn8zjbz3zs0lfpr3crzsx-ncurses-6.1

Success on aarch64-linux (full log)
Attempted: ncurses
Partial log (click to expand)
rmdir: failed to remove '/nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin': Directory not empty
Moving /nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin/tput to /nix/store/kk0nwckbrh77xmwply9yjcd28yvsj5py-ncurses-6.1/bin/tput
rmdir: failed to remove '/nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin': Directory not empty
Moving /nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin/tset to /nix/store/kk0nwckbrh77xmwply9yjcd28yvsj5py-ncurses-6.1/bin/tset
rmdir: failed to remove '/nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin': Directory not empty
Moving /nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin/captoinfo to /nix/store/kk0nwckbrh77xmwply9yjcd28yvsj5py-ncurses-6.1/bin/captoinfo
rmdir: failed to remove '/nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin': Directory not empty
Moving /nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin/infotocap to /nix/store/kk0nwckbrh77xmwply9yjcd28yvsj5py-ncurses-6.1/bin/infotocap
rmdir: failed to remove '/nix/store/7vp5yq37r8pjwq77ii52cgby9s4ij7jm-ncurses-6.1-dev/bin': Directory not empty
/nix/store/kk0nwckbrh77xmwply9yjcd28yvsj5py-ncurses-6.1

Failure on x86_64-darwin (full log)
Attempted: ncurses
Partial log (click to expand)
cannot build derivation '/nix/store/08cnzjx8p8i1yz6z4l210n2v3bxghsdq-libc++-5.0.2.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/r2rq60gzxg6hqpf7n8m4v8wprm58baqq-adv_cmds-osx-10.5.8.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/5sxr4q9p4ygbccpfng35y522q7yvkdhz-binutils-wrapper-2.30.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/cbxdpbf1sq873c0q22fs0ii3r6mxm5kx-bash-4.4-p23.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/v6j8l0qv997pbicl6lic9b28fgck7din-bootstrap-stage3-binutils-wrapper.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/gw91c80qivmrw2lb7dhqgwlwcc2hxp6f-bootstrap-stage3-clang-wrapper.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/sr5996m8zfk8r1g5bni0357k058yvy59-bootstrap-stage3-stdenv-darwin.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/l4nyzjkfg9dvam65q0nh8xznxhjbpn22-pkg-config-0.29.2.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/z8ddkkccfj2lkrbkq7pai9ckppmim6rg-ncurses-6.1.drv': 4 dependencies couldn't be built
error: build of '/nix/store/z8ddkkccfj2lkrbkq7pai9ckppmim6rg-ncurses-6.1.drv' failed


Distributed Memory Caching

The dev branch has a class for distributed memory caching.  I was just wondering if distributed memory caching is slated to make it out in the RTM release of vNext?  If it is not going to be in the RTM release:

Is this something that is just being thought about or is it definitely on the product road map?
Any ETA on when this will make it to release?

Thanks

MemoryDistributedCache right? That's just an implementation of IDistributedCache for use in non-distributed environments. It's primarily intended for local development and testing, but may also be used in single node deployments. Yes it will be included for RTM. The schedule is here: https://github.com/aspnet/Home/wiki/Roadmap

Thanks.  The readme had this in it:

Contains libraries for in-memory caching and distributed caching. Includes distributed cache implementations for in-memory, Microsoft SQL Server, and Redis.

The wording had me wondering if we would be getting distributed memory caching for a multi-node environment.
Thanks!


About performance

Regarding the issue of video saliency performance: I noticed in your paper's section on video saliency that the F-value in the metrics has a significant difference compared to previous papers. What is the reason for this?

Thanks for your interest in our work!
First of all, we collect the saliency maps provided by the authors or generated from public codes and evaluate their scores using the same evaluation code in our experiments.
In our paper, we use ave-F score to evaluate our method as most USOD methods and keep it consistent in all experiments. As I know, most VSOD methods reported max-F score in their papers.
In our implementation, we also calculate the max-F score of our method. You can easily compare the max-F score if you want.

Thank you for your response. May I ask if you could provide the test results of VSOD? I am currently working on weakly-supervised VSOD research and I would like to compare the performance with your work. Additionally, could you also provide the test results on the ViSal and VOS datasets? Thank you very much.

Thanks for your citation!

Thanks for your citation!
Sorry that I am working on other projects right now.
In our code, there are two ways you can produce the test results.

Using test.py as we show in readme;
Using deploy.py, you can generate the saliency maps with some modifications.

Hello, I used the test.py file to test the VSOD dataset and imported the midd_vsod_joint.pth parameters for the second stage. However, I found that the parameters do not match：
RuntimeError: Error(s) in loading state_dict for DataParallel:
Missing key(s) in state_dict: "module.encoder.conv1.weight", "module.encoder.bn1.weight"...


model infering

hi， i re-trained the model and got a model file named "saved_model.pb", so how can I run inference with this file?
Thks

Hi, please just read the input low-resolution image file and convert it into Y-channel only using the functions inside utils.py (there is a rgb_to_y function ). Once the file is in the desired format, just do:
#Compute the upscaled image for SESR-M5
model = tf.saved_model.load(model_path)
model_out = model(y_lr)

y_lr is input low-res image in y-channel format. model_path is the directory path that contains saved_model.pb file.

The output of this inference has only one channel. How can I generate bgr image from 1 channel image. The input shape given to the model is (1, 1080, 1920, 1) and the output is (1, 2160, 3840, 1). I think the output has only y channel
.

The output of this inference has only one channel. How can I generate bgr image from 1 channel image. The input shape given to the model is (1, 1080, 1920, 1) and the output is (1, 2160, 3840, 1). I think the output has only y channel .

Hi Muhammeet
I notice that there is tf.clip_by_value(features, clip_value_min=0., clip_value_max=1.) at the end of model in sesr.py.
Hence, there is a minimum layer and relu layer in the tflite file.
When I remove the tf.clip_by_value in the model, there is  some weird black spot on the output RGB image during inference.
Do u have any experience for that?

Hi,
Actually you have to do clipping operation since the output of the
model can be bigger than 1 which is bigger than 255 when converted to
opencv image. Therefore, we have to do clipping operation. The reason
of black spots is their values predicted by the model is bigger than
255 (1).
Alıntı mrgreen3325 @.***>:

The output of this inference has only one channel. How can I
generate bgr image from 1 channel image. The input shape given to
the model is (1, 1080, 1920, 1) and the output is (1, 2160, 3840,
1). I think the output has only y channel .

Hi Muhammeet
I notice that there is tf.clip_by_value(features, clip_value_min=0.,
clip_value_max=1.) at the end of model in sesr.py.
Hence, there is a minimum layer and relu layer in the tflite file.
When I remove the tf.clip_by_value in the model, there is some weird
black spot on the output RGB image during inference.
Do u have any experience for that?
—
Reply to this email directly, view it on GitHub[1], or unsubscribe[2].
Triage notifications on the go with GitHub Mobile for iOS[3] or Android[4].
You are receiving this because you commented.Message ID:
@.***>

Bağlantılar:
[1] https://github.com/ARM-software/sesr/issues/7#issuecomment-1060141256
[2]
https://github.com/notifications/unsubscribe-auth/ASQSTPSLDZRIFFLJIZGXAQLU6VXDZANCNFSM5KIH3XQA
[3]
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
[4]
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub


lwt & unix client: add routes when tunnel established (also teardown routes)

at the moment, the routes have to be manually set.
for the mirage unikernel client, we should investigate what to do as well.

done in #261


ui-plugin-operator k8s 1.27 chart support

This issue is to implement the chart changes & perform the testing required to upgrade to k8s 1.27. Please perform below steps.

[x] Check whether the chart has any references to the below deprecated features of k8s 1.27. You can find information about deprecated APIs here on Kubernetes Deprecation guide.

storage.k8s.io/v1beta1 API version of CSIStorageCapacity

[x] Update kube-version annotation in Chart.yaml. catalog.cattle.io/kube-version: '>= 1.23.0-0 < 1.28.0-0'

Note: If this is an upstream chart and if the upstream chart doesn't support k8s 1.27 then do not update the kube-version annotation or update the k8s version accordingly.

[x] Perform below tests

Fresh install/uninstall/upgrade/downgrade of the charts on K8s 1.27 provisioned on Rancher. Test the chart functionality.
(Please refer to the https://github.com/rancher/rancher/issues/41395 to know what testing was done for k8s 1.26 upgrade for your chart)

Provision rancher on k8s 1.26 or lesser

Install chart on this cluster

Upgrade this cluster in rancher to k8s 1.27

Make sure the existing chart and the app is functional after cluster upgrade

test plan
execute UI operator p0 test suite on a 1.27 k8 local server

tested on local
pre upgrade
rancher image -2.7.7-rc4
k8- 1.26.9
UI operator chart version- ui-plugin-operator:102.0.1+up0.2.1
post-upgrade
rancher image -2.8-head
k8- 1.27.5
UI operator chart version- ui-plugin-operator: 103.0.1+up0.2.1

Ran p0 test cases pre-upgrade
Performed the k8s upgrade
Updated Rancher image to 2.8-head
Ran another p0 regression

Did the Same for an airgapped env, only prerequisite: push the rancher:v2.8-head image to private registry
no issues found, closing this ticket marked as passed


Deprecate maxIdleSessions

From a quick search of the repo, it seems that we're no longer using this field. We should mark it as deprecated since otherwise users try to set the field.

It is actually still being used, however the usefulness of it might be up for discussion: https://github.com/googleapis/java-spanner/blob/54cfa0f70f9b117ff50310adc5e01282e4bd82ed/google-cloud-spanner/src/main/java/com/google/cloud/spanner/SessionPool.java#L1684
The basic usage of it in the session pool is as follows:

Always create MinSessions sessions, and never go below this value, regardless of how many sessions are actually used.
If more sessions than MinSessions are needed, the pool will create additional sessions, as long as this does not exceed MaxSessions.
When keeping sessions alive (the ping query) the session pool will try to keep MinSessions + MaxIdleSessions alive.

So setting a value for MaxIdleSessions will prevent additional sessions above MinSessions from expiring.

I think this is still useful if we want to keep min and max sessions configurable (if we don't want to have just a single num sessions). I would keep this configuration, but would like to know if you agree @skuruppu .

It seems that the intention is to keep X number of sessions alive in the pool. In that case, couldn't the user increase MinSessions to X so that they don't need to configure three different parameters?
This is just my opinion but I think we should minimize the number of configurable parameters, especially if the parameters don't directly help to optimize the performance of the client.

Yes, I think you are right, the only difference I can see is if the user does not want all the maxIdleSessions created before they are necessary, but that would be a very niche case.

It seems that the intention is to keep X number of sessions alive in the pool. In that case, couldn't the user increase MinSessions to X so that they don't need to configure three different parameters?

Yes, I think you are right, the only difference I can see is if the user does not want all the maxIdleSessions created before they are necessary, but that would be a very niche case.

Correct, this setting will prevent the excess sessions from being removed from the session pool, but they will only be created if the application actually needs more than MinSessions sessions. So setting MinSessions to a higher value would achieve almost the same. I think MaxIdleSessions made sense before BatchCreateSessions was introduced, as setting a (very) high value for MinSessions would cause a large burst of CreateSession RPCs when a session pool was created. That is no longer the case, and I think that this setting does not really add any value any more. Users should instead increase the value of MinSessions if they are currently using MaxIdleSessions in combination with a lower value for MinSessions.


Custom databases

Changes in the configuration and setup:

[ ] Test updated docker-compose.yml
[ ] Dev installations: update postgres user
docker-compose exec --privileged postgres psql postgres postgres -c "ALTER USER sm WITH SUPERUSER;"
[ ] Update configs
engine: aws_region -> aws_default_region
ansible/engine data_path: /opt/data/sm_data -> /opt/data/metaspace

@LachlanStuart After your PR approval, I'm going to rebase this branch onto master.

Happy for this to be merged once the remaining PRs are merged.
Just so we're clear on what has been deferred to the next iteration, this does not include:

Annotations with MSM 0.000, or only 1 isotope image in targeted databases
Reindexing datasets when DBs are renamed or deleted. Though this probably isn't necessary:

DB name is used in Datasets' annotation_counts.db.name, which is searchable through the free-text search, but I doubt people rely upon it
DB name is used in Annotations' db_name field, but that only seems to be used in possibleCompounds for selecting which template to use for the molecule URL. It's unlikely we'll change the core DB names.
Deleted DBs are automatically excluded from search results as they no longer are included in getVisibleDatabaseIds

Anything else?

@LachlanStuart

Annotations with MSM 0.000, or only 1 isotope image in targeted databases

Correct.

Reindexing datasets when DBs are renamed or deleted. Though this probably isn't necessary

I don't remember discussing databases renaming functionality. UpdateMolecularDBInput doesn't have the name field.
Regarding database deletion, it's a good point. It should be available only to admins but we still need to take care of orphan annotations in Postgres and ES after their database was deleted. At least, we should add ON DELETE CASCADE to https://github.com/metaspace2020/metaspace/blob/3b4f0cdfca1c77e0853212764631d5ba5ed9a9e2/metaspace/engine/scripts/db_schema.sql#L289:L291.

DB name is used in Datasets' annotation_counts.db.name, which is searchable through the free-text search, but I doubt people rely upon it
DB name is used in Annotations' db_name field, but that only seems to be used in possibleCompounds for selecting which template to use for the molecule URL. It's unlikely we'll change the core DB names.

I hoped that we'd be able to remove all references to database name and would be able to use only database id everywhere.

Anything else?

Database management UI.
Database management in python-client.
A bunch of minor updates like adding assertCanUseDatabase checks and enforcing unique names for public custom databases.

I don't remember discussing databases renaming functionality. UpdateMolecularDBInput doesn't have the name field.

Good point. I think we actually did discuss it and agreed to just not support renaming at all.

I hoped that we'd be able to remove all references to database name and would be able to use only database id everywhere.

I support this, and anything that moves us away from using ElasticSearch for general data storage & retrieval.

@LachlanStuart no idea why but after merging master and running yarn install in graphql directory yarn.lock changed again. Don't know if it's worth investigating.

@intsco The yarn.lock changes in the latest commit look fine to me.


Read Html Input Files and Generate Js Output Files in  resources folder

@elroykanye , I already created 2 folders htmlFilesInput and jsFilesOutput in src/main/resources, we will write code in order to read "sample.html" from htmlFilesInput folder and generate js file in jsFilesOutput folder. It's not a good idea to have these files at the root.

🤔 alright, I get it 👌🏾


Correctly account for "duplicated" suites when counting

Failing to do that makes clar miss the last of the suites, as all duplicated "data" would have not been accounted for.

Thanks!


How to install a stable "LTS" version of node-red on Debian 10 "Buster"?

Hi there
Question: How to install a stable "LTS" version of node-red on Debian 10 "Buster"?
I only can see some manuals about Debian 9 and Raspbian 9 or 10.
Thank you very very much for your feedbacks. :-)

As the issue template says, please use the forum for questions.

ok


Separating TravisCI jobs [ECR-1801]

Overview
Moved Rust code checks to a separate script, which checks the CHECK_RUST environment variable and if it's true starts rust code checks or skips tests otherwise.
Built a matrix and excluded cases openjdk10+CHECK_RUST=true and openjdk8+CHECK_RUST=false, so the tests should run against both jdks but rust code checks will be performed only for opejdk8.
I've measured build time with rust code being checked with different jdks and come to conclusion that when rust checks execute with jdk8 allows to save more time occasionally.

See: https://jira.bf.local/browse/ECR-1801
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] The continuous integration build passes

Integrated, thanks!


Recommended Reviewers

We need to recommend three people at least (for journal of applied ecology). Here is the most recent list available: http://besjournals.onlinelibrary.wiley.com/hub/journal/10.1111/(ISSN)1365-2664/journal-resources/list-of-reviewers.html
Thoughts:

Ask Tamsin and wait, which might take some time
See if we recognise anyone
Collect publications of the people in the list and see who has interest in our topic.

This has been implemented. Decision was made using the package Arcas


Templates and Branches

How do we avoid collisions in the local repository if an author uses branches in a repository to manage templates?
As far as I can tell,
"Spin templates install --git remote/foo"
Will cache in the same directory as if I supplied the "--branch bar" flag on the command above.
Or am I misunderstanding how templates are cached?

Templates are installed under the id parameter in the template manifest.  At the moment, you cannot install two templates with the same id - if the id already exists you can skip or overwrite.
Can you talk more about your intended use case?

OK thanks! That clears it up!


Site suggestion

Submitter: jesse
Hey there!
ACC UX Studen here. The VisCom department got a new site.
This is the link: http://sites.austincc.edu/viscom/
It would be a better landing page than the current like you have listed.
Thanks,
Jesse

Site updated 2016-08-15


Taichi with my own driver video, help needed

I tried to follow the instruction. But I failed to get the driver video. So I tried to make my own one. This is the command I've used (probably wrong)  😄
python /bin/first-order-model-master/demo.py --config E:\bin\first-order-model-master\config\taichi-adv-256.yaml --driving_video .\driver.mp4 --source_image .\pose.png --checkpoint E:\bin\first-order-model-master\play\taichi-adv-cpk.pth.tar --adapt_scale
and this is my result hahaha
https://www.reddit.com/r/MediaSynthesis/comments/hy6we5/i_think_my_firstordermodel_taichi_has_failed/?utm_source=share&utm_medium=web2x
I've moved very slowly in the video (only little motion blur in my hands)
I've used a square video and a square image.
The starting poses are very similar.
What else do I have to do?
Does the driver video need a clear background?

Taichi have very poor Generalization, so basically it only works with other Taichi videos.

Taichi have very poor Generalization, so basically it only works with other Taichi videos.

Is there any way to train it ourselves, so it would work with our input data?

Try it out.

Try it out.
Can you give us the command? I found something in the description. But I haven't really understood.


FormatException: Invalid radix-10 number (at character 1) 1(3)

New Issue Checklist

[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. flutter / ios / android

Platform Version
e.g.  1.5.0 / 12.0 / 9.0

Dio Version
e.g.  2.1.0 / 1.0.17

Android Studio / Xcode Version
e.g. Android Studio 3.3.2 / Xcode 10.2.1

Repro rate
e.g. all the time (100%) / sometimes x% / only once

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.

If you are receiving a JSON from an API (my case).
When trying to access the API response in this way response.data.day will receive this same error because the one in response.data is a map so the correct one to access the data is response.data ["day"]


Avoid recursive call to AttributeHash#attributes

Fixes #1155
Defining an attribute called 'attributes', then referring to that
attribute in initialize_with was causing AttributeHash#attributes
to call itself recursively.

LGTM.
Does this mean that we can't name an attribute initialize? Not that I want to...

Yeah, I don't think adding an initialize attribute has ever been possible, and I think I am OK with that.


correct link to slack in README.md

Proposed changes
Following discussion on #696 it seems that the URL in README.md should match the correct URL for the slack channel.
Types of changes

[x] minor correction of a link in README.md
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Only the readme was adjusted and so no tests were executed, and no functionality should have changed due to the changes.
Closing issues
No open issues are addressed by this.

:-D

thx!


Using export map instead of custom property in package.json

currently, it says

Referencing manifests from npm packages
In order to allow tools to find npm packages with custom element manifests without having to download package tarballs, > packages should have a "customElements" field in their package.json that points to the manifest:
{
  "name": "example-package",
  "customElements": "custom-elements.json",
}

but if you are using an export map
"name": "my-package",
"customElements": "custom-elements.json",
"exports": {
  ".": "./index.js"
}

then the files can not be imported... e.g. this will fail
import cem from 'my-package/custom-elements.json' assert { type: 'json' };

Suggestion
Replace the custom property with an entry in the export map.
Benefits:

resolving is depending on an import map or the node resolution system
file is always accessible as it's part of the export map
not 2 different ways of defining an CEM

- "customElements": "custom-elements.json",
+ "exports": {
+  "customElements": "custom-elements.json",
+  ".": "./index.js"
+ }

For the foreseeable future, not all packages will make use of export maps yet, so I think we'll have to have two ways for now:
If a package uses export maps:
 "exports": {
   "customElements": "custom-elements.json",
 }

If a package does not use export maps yet, add a field in the package.json:
"customElements": "custom-elements.json"

As discussed in our meeting about this, a manifest could be included in package exports if the package intends it to be an importable JSON module from a consuming package, but this isn't a primary use case.
The use case we most need to cover is for a source analyzing tool to be able to discover a package.json and custom elements manifest given a base path and a specifier, resolving the specifier within the source's available node_modules tree, not the tool's. This may require implementing part of Node's module resolution algorithm as a utility function. We'll need this whether or not the custom elements manifest is exported.
I think we can close this as an issue that's part of the manifest spec and figure it out in tools.


Low priority manipulation tasks

I'm going to use this issue to collect together some low priority manipulation tasks. I'll come back to implement these at the end.
ClassDeclaration

[ ] Class methods in an ambient context
[ ] Get & set accessors

Decorator

[ ] Renaming
[ ] Setting as a factory or not
[ ] Adding/inserting arguments

JSDoc

[ ] Setting description

Function/Methods/Constructors

[ ] Setting body text

VariableDeclarationList

[ ] Setting declaration type.

VariableStatement

[ ] Adding variable declarations

More to come...

@dsherret ,
First, thank you for this amazing lib.
I am building a tool to upgrade an legacy javascript code to typescript.
Among the conversions, the tool must replace var by let declaration types everywhere.
I stumbled on:
Error: Filling variable declaration type not implemented. Please comment on issue #27 if you need this and I will increase the priority.

So here I am asking to increase Setting declaration type priority.
I can also make a PR if you give me some guidance.

@reda-alaoui thanks, I'm glad it's useful for you!
It's very easy for me to add that. I'll start work on that now.

Great, thank you :)

@reda-alaoui this is implemented and available in v0.85.0.
I'm going to keep working on the other tasks in this issue.

You rock !


refactor: improve metadata fencing code organization

Currently, the metadata fencing code is challenging for inexperienced developers to use, given that it is fragmented across the codebase without a clear pattern.
This patch aggregates the logic and adds a fluent builder to execute the fencing metadata logic.
It also improves handling certain operations like fencing or unfecing an already present or deleted instance. In those cases, the executor does not return an error anymore.

/test limit=local

/test limit=local

/ok-to-merge


Undefined symbol: omp_get_max_threads

The tests are failing for me in the sparse tests. I'm getting the following error:
/home/tshort/julia-f4c6c9d4bb/bin/julia: symbol lookup error: /opt/cuda/lib64/libcusolver.so: undefined symbol: omp_get_max_threads

I've gotten the same result with Julia 0.5.2 and 0.6.0. The dense tests complete fine. The tests for CUDAdrv also pass.
I'm running on Arch Linux. Something's probably borked with my CUDA installation, but I thought I'd check to see if anyone has run into this.

I tried running this under Docker with a Ubuntu base, and I got the same test error.

I also get this error in a v1.1 and v1.0


Asciidoc support

Asciidoc seens to be a popular alternative to Markdown. Recently Polylith changed readme and docs format to Asciidoc. Should we consider support for Asciidoc?
I always found markdown hitting the sweet spot naturally expressing common formatting. With Docusaurus there is MDX markup for richer elements. Asciidoc looks more complicated to me since it includes additional custom syntax for richer elements.
I wonder if translating Asciidoc to Markdown using Pandoc would work well as a preprocessing step?

Feature-wise AsciiDoc can be though of as a superset of Markdown, so translation from AsciiDoc to Markdown is not very feasible. Thefore I consider it now out of scope.
When wanting to author docs in AsciiDoc it is possible to use Antora generator.
There are now helpers to transform md to adoc, so it is possible to output Clojure API docs as adoc and rendered with Antora.


Database secrets are currently deprecated

Database secrets are currently deprecated and use a legacy Firebase token generator. Update your source code with the Firebase Admin SDK.
Learn more
https://firebase.google.com/docs/admin/setup?authuser=0

I actually started working on it some time ago, but then, as usually, got distracted and forgot about it completely. Not sure when exactly, but I do plan on updating it to make it possible for everyone to run it using their own Firebase account.

I started working on it, but there's some strange issue that Firebase database rules no longer working as expected after the migration to Firebase Admin SDK and the latest JS SDK.
I asked about it here: https://stackoverflow.com/questions/67835978/problem-with-firebase-realtime-database-rules-after-migrating-to-firebase-admin.


AVRO-2677: Add bytes decimal schema support with precision and scale

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-2677

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

Thanks for the PR @toydestroyer Would it be possible to add a test?
@tjwp Do you have some spare time to review this? Haven't written any Ruby for a while now :)

@Fokko definitely! Working on test right now.
Thanks for the quick response.

@tjwp all comments were addressed. Could you please re-run the build? Looks like yesterday was some problem with CI that doesn't relate to the PR.
Thanks!


Multi User with own Folder

Hello,
Is it possible to have multi user with their own folder?
For example,
User Samantha will have folder user1
User Josh will have folder user2
Each user has their own username and password.
If this is possible will be great as additional feature.
Rgds

Hello,
This is not provided out of the box, but achievable.
The hard way: you modify configuration files to achieve multiuser support. This is supported in that the image supports various configuration files.
The "easy" way is a reason why we actually have containers and Docker: you run one container per user. Each container serves each user and mounts separate folders.  If you happen to have a reverse proxy in front of all containers, this will even be somehow transparent to each user as they will connect to either user1.server.com, user2.server.com or server.com/user1, server.com/user2, and so on.

Hi,
Nice to see there are possibilities to achieve that but unfortunately I'm a noob at this.
Having different subdomain for each account is not applicable for my situation either.
Well, could you please guide me what files need to be modified (from the tutorial).
Will the modification stay after reboot?

I need that too. The way I host right now I have custom locations in NPM and each custom location like
/userA
/userB
has it's own Apache based WebDav container, so for each user I need to spin up a new container, not mem efficient, despite alpine based takes only 1.4MB of RAM when idle.
The problem with this approach is that there is a bug in NPM (I need to report) that when one custom location is down, the NPM is in looping state and can't login to NPM so it's kind a killer setup, serious bug. Thus I would prever single container with multiple users.
The way I achieved folder per user in bare Nginx is that I also have location per user, but if I could delegate this to your container it would not affect NPM.
Here is the sample config
    root      /custom/www; #some web content to host in the root of the domain
    client_body_temp_path /custom/www/temp;
    dav_methods     PUT DELETE MKCOL COPY MOVE;
    dav_ext_methods   PROPFIND OPTIONS;
    create_full_put_path  on;
    client_max_body_size        20M;
    #dav_access    user:rw group:rw all:r;
    dav_access    user:rw group:r;
    autoindex     on;

    # this is important for keepass
    open_file_cache off;

location /usera/ {
    auth_basic "restricted";
    auth_basic_user_file /etc/nginx/webdav/usera.pwd;
  }

location /userb/ {
    auth_basic "restricted";
    auth_basic_user_file /etc/nginx/webdav/userb.pwd;
  }

# I think I could use same pwd file for all locations, because folder name is not tied to user name,
# but then technically anybody from pwd file list can access specified location that uses the pwd file

Ideally there should be just one pwd file, and somehow parsed user names can create a location per user name, and only allow that user name to access it's own folder. Right now I have pwd file per location to achieve that

I need that too. The way I host right now I have custom locations in NPM and each custom location like /userA /userB has it's own Apache based WebDav container, so for each user I need to spin up a new container, not mem efficient, despite alpine based takes only 1.4MB of RAM when idle.
The problem with this approach is that there is a bug in NPM (I need to report) that when one custom location is down, the NPM is in looping state and can't login to NPM so it's kind a killer setup, serious bug. Thus I would prever single container with multiple users.
The way I achieved folder per user in bare Nginx is that I also have location per user, but if I could delegate this to your container it would not affect NPM. Here is the sample config
    root      /custom/www; #some web content to host in the root of the domain
    client_body_temp_path /custom/www/temp;
    dav_methods     PUT DELETE MKCOL COPY MOVE;
    dav_ext_methods   PROPFIND OPTIONS;
    create_full_put_path  on;
    client_max_body_size        20M;
    #dav_access    user:rw group:rw all:r;
    dav_access    user:rw group:r;
    autoindex     on;

    # this is important for keepass
    open_file_cache off;

location /usera/ {
    auth_basic "restricted";
    auth_basic_user_file /etc/nginx/webdav/usera.pwd;
  }

location /userb/ {
    auth_basic "restricted";
    auth_basic_user_file /etc/nginx/webdav/userb.pwd;
  }

# I think I could use same pwd file for all locations, because folder name is not tied to user name,
# but then technically anybody from pwd file list can access specified location that uses the pwd file

Ideally there should be just one pwd file, and somehow parsed user names can create a location per user name, and only allow that user name to access it's own folder. Right now I have pwd file per location to achieve that

Hi skironDorNet,
Where I must put the config?

Hi All,
Today I tried to enable MultiUser support and it was so easy, only one container needed.
You only need to edit /config/nginx/server.conf and re-deployb/ restart the container.
This topic can be closed.

Multiuser support is indeed easy to setup with only one container.
Be sure that you have:

a htpasswd file with your users and passwords
a folder with for each user
the right permissions (user/group of the nginx process) to these folders (as set with the env-variable)
add the following to /config/nginx/location.conf

   location / {
         auth_basic            "Please Login";
         auth_basic_user_file  "/config/nginx/htpasswd";
         root /data/$remote_user;
     }


fix sql, bq, storage location exemptions empty list

Fixes https://github.com/forseti-security/policy-library/issues/369

Seems like there are test cases for exemption
https://github.com/forseti-security/policy-library/tree/master/validator/test/fixtures/sql_location/constraints
https://github.com/forseti-security/policy-library/tree/master/validator/test/fixtures/storage_location/constraints
https://github.com/forseti-security/policy-library/tree/master/validator/test/fixtures/bq_dataset_location/constraints

None of the existing test cases test what happens when exemptions isn't specified at all. I recommend we add a test case to avoid regression.

@morgante I modified the allowlist_one test case to not specify the exemptions_list to prevent regression. The empty exemptions_list being specified case is still tested in the allowlist_all case.

Perfect, thanks!


test: let replica resize events consolidate before nexus resize

Add a small delay after replicas' resize and before nexus resize, to simulate real world workflow driven by control plane.

bors merge


GEOMESA-2191 System property to prevent full-table scans

Set 'geomesa.scan.block-full-table=true'
Global behavior can be overridden for individual schemas with 'geomesa.scan..block-full-table'

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

Do we want to add this to the geomesa-site.xml.template?

:+1:


Protect Package Bindings from containing circular references

Description
We ran across an issue where a package binding can contain a reference to itself. When this happens, any attempt at reading the package results in an infinite loop, eventually leading to the controller falling over. This can occur by simply performing the following commands using the wsk tool.
$ wsk package create target
$ wsk package bind target binding
$ wsk package bind binding binding

There are two changes contained in this PR

Prevent allowing an incoming binding update to point to itself as a reference, which previously created an invalid data record.
Prevent following a package's binding reference when it points to itself which causes the infinite loop.

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[X] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[X] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[X] I signed an Apache CLA.
[X] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[X] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #4122 into master will decrease coverage by 4.89%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master    #4122     +/-   ##
=========================================
- Coverage   86.16%   81.26%   -4.9%
=========================================
  Files         151      151
  Lines        7263     7266      +3
  Branches      466      467      +1
=========================================
- Hits         6258     5905    -353
- Misses       1005     1361    +356

Impacted Files
Coverage Δ

...openwhisk/core/entitlement/PackageCollection.scala
100% <ø> (ø)
:arrow_up:

...rg/apache/openwhisk/core/controller/Packages.scala
88.57% <66.66%> (-0.65%)
:arrow_down:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.54%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-83.34%)
:arrow_down:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
0% <0%> (-62.5%)
:arrow_down:

...penwhisk/core/database/cosmosdb/CosmosDBUtil.scala
92% <0%> (-4%)
:arrow_down:

...pache/openwhisk/core/invoker/InvokerReactive.scala
81.3% <0%> (-0.94%)
:arrow_down:

...che/openwhisk/core/database/CouchDbRestStore.scala
73.23% <0%> (-0.51%)
:arrow_down:

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 d38a00e...f6ce8c7. Read the comment docs.

@asteed Should we also update swagger definition for 422 status code response in package api?


Update generated client

Updating the client after kubernetes 1.6 release. The update process was not trivial as there were breaking changes in OpenAPI spec of kubernetes 1.6 release.
fixes #167 #169

Codecov Report

Merging #170 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #170   +/-   ##
=======================================
  Coverage   94.46%   94.46%
=======================================
  Files           9        9
  Lines         668      668
=======================================
  Hits          631      631
  Misses         37       37

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 f37f758...b4762ae. Read the comment docs.

Quick question inline, LGTM 👍

Answered. Thanks.


Add PHP switch statements and fix variable code

Switch statements felt like a good candidate for a common x2cpg method, but at least from comparing php2cpg and javasrc2cpg, it looks like actually extracting shared code will be difficult to do without adding just as much preprocessing code back in.

Maybe add a CFG test for case/switch later on?

Good point. I'll add CFG tests for all of the control structures in a follow-up PR.


A phase or texture field for Compound and ReactionInput

Is your feature request related to a problem? Please describe.
The physical state of reactants can be important for reproducing a reaction (especially the choice of appropriate addition devices). So far no dedicated fields for this.
Describe the solution you'd like
A phase or texture field at both Compound and ReactionInput levels. The value can be a Texture message, although it does not include the gas phase explicitly.

I will check this and update status. Might need to open an issue on ord-interface to provide new fields there.

Need to add a texture input field to ReactionInput on the contributions interface.

@bdeadman is this issue done?

ord-schema side was done in ord-schema PR 698. Just need to update the interfaces. There is an issue for that open-reaction-database/ord-interface#75 so will close this one.


Code quality feedback for A0105287E

main

Decent Slap is followed in general
It seems to me that MarkCommand class is handling adding of recurring task and edited task? Is MarkCommand class really suitable for creating or editing task?

Better slap can be done here

Please add header comments for non trivial methods.

If the 2 exceptions are just printing the same stack track
perhaps
catch (IllegalValueException
| IllegalDateTimeValueException e) {
e.printStackTrace();
}

Avoid empty tags

test

Perhaps more unit test can be implemented

Addressed, thank you! :)


Interactive example

While running the Interactive example I am getting some errors
22:51:06 Starting cadvisor
22:51:37 Ports for container interactive-cadvisor >> Local ports: map[http:8080] Ports available from host: map[http:49153]
22:51:37 Starting monitoring
22:51:48 Ports for container interactive-monitoring >> Local ports: map[http:9090] Ports available from host: map[http:49154]
22:51:49 cadvisor: W0729 17:21:49.844312       1 machine_libipmctl.go:62] There are no NVM devices!
22:51:55 monitoring: level=error ts=2022-07-29T17:21:55.023Z caller=main.go:355 msg="Error loading config (--config.file=/shared/data/monitoring/prometheus.yml)" err="parsing YAML file /shared/data/monitoring/prometheus.yml: yaml: unmarshal errors:\n  line 11: field enable_http2 not found in type config.ScrapeConfig\n  line 22: field enable_http2 not found in type config.ScrapeConfig\n  line 33: field enable_http2 not found in type config.ScrapeConfig\n  line 44: field enable_http2 not found in type config.ScrapeConfig"
22:52:10 Killing monitoring
22:52:10 Error response from daemon: Cannot kill container: interactive-monitoring: No such container: interactive-monitoring
22:52:10 Unable to kill service monitoring : exit status 1
22:52:10 Killing cadvisor
22:52:15 monitoring-exec: Error: No such container: interactive-monitoring
22:52:15 Unable to kill service cadvisor : exit status 1
--- FAIL: TestReadOnlyThanosSetup (73.37s)
interactive_test.go:122: interactive_test.go:122:
unexpected error: the service monitoring is not ready: Get "http://<IP_ADDRESS>:49154/-/ready": dial tcp <IP_ADDRESS>:49154: connect: connection refused
FAIL
FAIL    command-line-arguments  73.425s
FAIL

Hey @douglascamata, can you please look into this :)

Works fine for me:
16:46:03 monitoring: ts=2022-08-03T16:46:03.861Z caller=main.go:996 level=info msg="TSDB started"
16:46:03 monitoring: ts=2022-08-03T16:46:03.861Z caller=main.go:1177 level=info msg="Loading configuration file" filename=/shared/data/monitoring/prometheus.yml
16:46:03 monitoring: ts=2022-08-03T16:46:03.862Z caller=main.go:1214 level=info msg="Completed loading of configuration file" filename=/shared/data/monitoring/prometheus.yml totalDuration=725.409µs db_storage=900ns remote_storage=2.5µs web_handler=400ns query_engine=900ns scrape=265.504µs scrape_sd=47µs notify=800ns notify_sd=1.4µs rules=1.7µs tracing=6.1µs
16:46:03 monitoring: ts=2022-08-03T16:46:03.862Z caller=main.go:957 level=info msg="Server is ready to receive web requests."
16:46:03 monitoring: ts=2022-08-03T16:46:03.862Z caller=manager.go:937 level=info component="rule manager" msg="Starting rule manager..."
16:46:04 monitoring: ts=2022-08-03T16:46:04.017Z caller=main.go:1177 level=info msg="Loading configuration file" filename=/shared/data/monitoring/prometheus.yml
16:46:04 monitoring: ts=2022-08-03T16:46:04.018Z caller=main.go:1214 level=info msg="Completed loading of configuration file" filename=/shared/data/monitoring/prometheus.yml totalDuration=705.11µs db_storage=1.6µs remote_storage=3µs web_handler=900ns query_engine=1.5µs scrape=57.8µs scrape_sd=74.301µs notify=2µs notify_sd=1.5µs rules=1.3µs tracing=6.101µs

Looks like some problem with your Docker setup or temporary files that the interactive test creates. I recommend deleting examples/tmp and examples/interactive/data if they exist, plus running docker system prune -a to ensure a clean environment.


No validation for sharing a form

Description
Appear to "send" form (share a form file) without entering an email address
Steps to reproduce

Go to a form
Click on "Share" > "Share by email" > "Send"
Appears successful even with empty email field

Details

Browser: Chrome
Operating system: Mac
Environment: PR App router
Language: English
Viewport: Desktop
Form ID: cls0tzcw30000auo270xzhpyo

Expected behaviour

Error if email is not valid
Only show success message sent if actually sent

Screenshots or videos
https://github.com/cds-snc/platform-forms-client/assets/38330843/85ce408d-023d-4c50-93f6-d2dfb5cec5f4

@connorscarolyns and @samsadasivan would you know the expected behavior? I just tested this on Staging and I see the same behavior.
Currently the code has validation in place for an email. If the input fails validation, the text is not added as tag. But the form can still submit the invalid email and does not show an error.
E.g. https://forms-staging.cdssandbox.xyz/en/form-builder/settings/cls3luu49000012u1xergaobf

Going to remove the app router label for this one --- we can look to fix outside of the app router updates.

Expected behaviour
Prevent empty or invalid submission
Display error message
EN: Enter a valid email address.
FR: Veuillez entrer une adresse courriel valide.

Closing this card as the issue no longer persists. Created a new issue to update the error messaging here (#3916). ^
Private Zenhub Video


Chore: unit tests

[ ] passing invalid popover throws error
[ ] SatPopoverAnchor#open/close/toggle
[ ] SatPopover#open/close/toggle
[ ] positioning + classes + overlap
[ ] changes in position while open
[ ] multiple anchors? or error when multiple are tried
[ ] backdrop clicks
[ ] escape keydown
[ ] focus trap

SatPopover

Inputs

xPosition
yPosition
overlapAnchor
hasBackdrop
backdropClass

Output

opened
closed

Methods

open
close
toggle

SatPopoverAnchor

Inputs

satPopoverAnchorFor

Outputs

popoverOpened
popoverClosed

Methods

popoverOpen --> Needs rename to isOpen
togglePopover
openPopover
closePopover
destroyPopover --> Would there be any wisdom to making this private?


Preserve time on horizontal resizing

When an event is resized horizontally (e.g. on the Month
View), keep the time of the event the same and only change
the date.
This also fixes the bug where you cannot resize an event to
a single day because the dates.max or dates.min would
force the event to occur at least 1 day after the
start or end respectively.

In the withDragAndDrop file, it looks like resizing is supposed to preserve the times and only change the date, but that isn't the case (https://github.com/jquense/react-big-calendar/blob/master/src/addons/dragAndDrop/withDragAndDrop.js#L41).
Right now, resizing an event automatically sets the changed value (start or end) to midnight of the new date. This also caused this bug (https://github.com/jquense/react-big-calendar/issues/1629) where you couldn't resize an event to a single day.
Current Behavior: Notice that the start date is set to midnight when resizing and that I can't make the event last only on Thursday.

New Behavior: Resizing will preserve the time of the event, and only change the date. I can make the event last only a single day.

thanks!

It is not preserving the time on this demo.


add sqlite adapter

adds support for sqlite3 and fixes #60

thank you for your pr.

By the way, here I'am also using os.TempDir() to store sqlite database file there


deleted unwanted file

deleted react-env.d.ts

closing this PR, these changes will be taken in next update


scaleBrushes()

Also the brush.scaleBrushes() doesn't seem to scale accurately. If that's a known issue then I will sit tight else here is a small demo test that i made https://editor.p5js.org/ujjwalagarwal/sketches/VaBAeB0wO
if you resize the window which changes innerWidth and innerHeight, the render is different.
Also i think the fill scales properly, it's the hatch and probably stroke thats an issue.

@kala0606
Yes, that's right. It's how the library works. I can remove this behaviour, but I'd really suggest to avoid resizing the canvas if possible, because many other things can break.
My suggestion would be to use css resize, rather than canvas resize.

Got it. Thank you so much.


Angular 10 support

Detailed description

Describe in detail the issue you're having.

Provide Angular 10 support

Is this a feature request (new component, new icon), a bug, or a general issue?

Feature

Is this issue related to a specific component?

What did you expect to happen? What happened instead? What would you like to see changed?

What browser are you working in?

What version of the Carbon Design System are you using?

4.11.1

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

IBM AD
Steps to reproduce the issue

Step one
Step two
Step three
etc.

Additional information

Screenshots or code
Notes

Definitly long overdue ... I don't think there's any specific fires I need to put out this week, so I'll take some time to work on this.

Thanks Cal! :-)

On Sep 14, 2020, at 3:01 PM, Cal Smith<EMAIL_ADDRESS>wrote:

Definitly long overdue ... I don't think there's any specific fires I need to put out this week, so I'll take some time to work on this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@cal-smith we are working on a new Angular project, so it will be great to see Angular 10 support soon!


Amazon_Price_Tracker

Script Description - a python script to track amazon prices and send an email notification when the price falls within budget.
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed
add pytest/unittest if possible

Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Documentation

Programming Language

[x] Python

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server  - https://discord.gg/FXyh2S3
Happy Coding,

@SahilSatyam , the Same script is already in the repo. Please can you check that? if you want to improve upon that you can make a PR.


Link to JTNimoy's personal website needs to be updated

Actual Behaviour

On the page https://p5js.org/libraries/, for the p5.cmyk.js and Marching libraries, the supplementary link(s) to the personal website of JT Nimoy point to a site that has gone down, http://jtnimoy.net/. Sadly, JT Nimoy passed away in 2020, and her website is not expected to come back online.
Expected Behaviour

Here are three possibilities for how this could be corrected:

The link on JT's name could be deleted entirely
The link on JT's name could be directed to point to her GitHub account, https://github.com/jtnimoy
The link on JT's name could be directed to point to an Internet Archive snapshot of her site, captured (for example) at https://web.archive.org/web/20200910192435/http://jtnimoy.net/

The first solution is simplest, but misses the opportunity to direct the interested reader to any of JT's brilliant work. The second solution is acceptable, reliable, and innocuous enough, it seems to me. The third solution points to a more vibrant index of JT's work, but may be distracting or confusing for some.

hi @golanlevin :)
i submitted a pull request to update the website to be her GitHub account,
also happy to go ahead and link the Internet Archive snapshot,
thanks so much for celebrating her amazing art!

Hi @golanlevin and @montoyamoraga Thanks for working on this! I just double-checked that JT's friend Gabriel Dunne mirrored JT's portfolio to another domain: jtnimoy.cc.  I will update the link on the p5.js library page to her new website shortly.

@Qianqianye This is a great solution, thanks. I wasn't aware of the mirror site.


Attempt to read tex_template from config file if needed

List of Changes

Read tex_template from config if not present in command line arguments.
If tex_template is not found in config use None.

Motivation
This implements the behaviour described in #450 .
Explanation for Changes
If the command line option --tex_template is not specified, we attempt to fetch it from the config file. If it is not present there either we use None.
Testing Status
Has been tested with Python 3.8 on macOS. pytest succeeds as well.
Acknowledgement

[x] I have read the Contributing Guidelines

I notice that the Windows CI tests have failed. However, it appears that this is not caused by the changes that I made but by something else. Furthermore, the CodeFactor failed, but this too is on parts other than the ones I wrote or changed. Except for one where a comment is too long, does this need to be changed?

I notice that the Windows CI tests have failed.

Its fixed in master can you pull from master?

I notice that the Windows CI tests have failed.

Its fixed in master can you pull from master?

Done!

@jvdoorn is this ready for review?

Yes @leotrs, as far as I'm aware I implemented any requested changes.


Thread improvements, process batch messages in thread pool

@subkanthi any plans to continue working on this PR? It has a potential to solve some issues in the lightweight connector (https://github.com/Altinity/clickhouse-sink-connector/pull/342, https://github.com/Altinity/clickhouse-sink-connector/issues/307).


When adding a new card, I do not receive the nextData

When I add a card to the board using .publish({type: 'ADD_CARD', ...}); I do not receive the nextData inside the function called 'shouldReceiveNewData'.
When I add a card, the component does not render again to get the new state.
The state is not updating with the new card added.
My example:
addCard = () => {
        const { eventBus, } = this.state;

        eventBus.publish({
            type: 'ADD_CARD',
            laneId: 'lane1',
            card: {
                id: `card1`,
                title: 'new card',
                label: '30 mins',
                description: 'new task',
                metadata: { laneId: 'lane1' }
            }
        });
    }

shouldReceiveNewData = nextData => {
        console.log('it is not being called);
        console.log(nextData);
    }

<Board data={this.state.boardData} eventBusHandle={this.setEventBus} onDataChange{this.shouldReceiveNewData}/>

Thanks for pointing this out. This functionality broke in a recent release I guess. Fixed now. Please pull v1.22.2.
Refer to storybook again for updated behaviour..


'Organize Imports' runs after code formatters and after file was saved

VSCode Version: 1.31.0
OS Version: OSX 10.14.2

Steps to Reproduce:

Install prettier extension
Enable format on save
Enable organize imports on save
Create typescript file with lots of imports, causing prettier (or any other code formatter) to move imports onto individual lines
Save file (ensure that code formatter runs on save)

prettier formats code
Organize imports are performed after code is formatted ;(
File is saved to disk
THEN imports are organized again

The file remains dirty (white circle showing file was modified since last save)
The imports modified by the code formatter were completely ignored

This started happening immediately after updating to 1.31.0 and worked fine before that, and I have not made any setting changes or added/removed any extensions since the update.

Does this issue occur when all extensions are disabled?: Yes/No Probably not, since the native code on 1.31.0 conflicts with extensions, but it is core VSCode that changed

Please start by filing an issue with the Prettier extension


Add support for cloud provider optionally to modify shoot vpn config

What this PR does / why we need it:
This PR completes the PRs merged earlier in gardener here and vpn here adding runtime configuration options to the shoot VPN. It feeds the VPN config to the CloudBotanist after creating it, right before sending it to the helm chart. This gives it the option - but not the requirement - to modify it.
Special notes for your reviewer:
Discussed on Slack with @rfranzke
Release note:

Enable custom vpn configs by cloud provider, including runtime determination

No idea why CI isn't running...

Changed what you requested. Back to you.


update packages

:white_check_mark: Build TheIdServer 8.3.0-fix-dependencies.1+39.build.3941 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/ee3487c654 by @invalid-email-address)


the script not works on large file

i have a large css file that is not being transformed

Which PHP version.
Which version of CSSJanus, and how is it installed (Composer?)
What is the value of ini_get( 'memory_limit' ).
Can you upload a copy of a file that is not working (e.g. via https://gist.github.com) that I can use to reproduce the issue?

i have a large css file that is not being transformed

Try this: Using mbstring (improve performance on localhost)

files size 382 KB
i dont know use gist
i use like this
require_once(BX_DIRECTORY_PATH_PLUGINS_PUBLIC .'/php-cssjanus/src/CSSJanus.php');
$sCss = CSSJanus::transform($sCss);

Which PHP version. 7.2
Which version of CSSJanus, and how is it installed (Composer?) latest
What is the value of ini_get( 'memory_limit' ). 128M
Can you share an example that is not working? E.g. short script that calls CSSJanus (which method + parameters), as well as a copy of a file that is not working for you (e.g. via https://gist.github.com.

Thanks :)

@yanivz I'm sorry but without a copy of a file that causes the problem, I cannot help. You can copy the contents to a Gist to share it. Or, upload it elsewhere and share the link it :)


mac osx high sierra install issue

Process:               Emerald-Qt [24022]
Path:                  /Volumes/VOLUME/Emerald-Qt.app/Contents/MacOS/Emerald-Qt
Identifier:            org.emerald.Emerald-Qt
Version:               $VERSION ($VERSION)
Code Type:             X86-64 (Native)
Parent Process:        ??? [1]
Responsible:           Emerald-Qt [24022]
User ID:               501
Date/Time:             2018-02-03 12:19:27.959 -0600
OS Version:            Mac OS X 10.13.3 (17D47)
Report Version:        12
Anonymous UUID:        6F91CEDF-BF4B-A93E-4A65-9FCB21A20C66
Time Awake Since Boot: 53000 seconds
System Integrity Protection: enabled
Crashed Thread:        0
Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY
Termination Reason:    DYLD, [0x1] Library missing
Application Specific Information:
dyld: launch, loading dependent libraries
Dyld Error Message:
Library not loaded: /opt/local/lib/libboost_chrono-mt.dylib
Referenced from: /Volumes/VOLUME/Emerald-Qt.app/Contents/MacOS/Emerald-Qt
Reason: image not found
Binary Images:
0x108aa9000 -        0x10909effb +org.emerald.Emerald-Qt ($VERSION - $VERSION) <314D8D5B-CE55-3180-AA7B-BD954C7ACBB4> /Volumes/VOLUME/Emerald-Qt.app/Contents/MacOS/Emerald-Qt
0x109226000 -        0x10926dfff +libssl.1.0.0.dylib (0) <23FCCCD7-644F-3620-BA47-6B0F1A42C818> /Volumes/VOLUME/Emerald-Qt.app/Contents/Frameworks/libssl.1.0.0.dylib
0x10928a000 -        0x10940f877 +libcrypto.1.0.0.dylib (0) <196225AA-EE6C-3E28-96ED-5F862CE12E85> /Volumes/VOLUME/Emerald-Qt.app/Contents/Frameworks/libcrypto.1.0.0.dylib
0x109486000 -        0x10959bff7 +libdb_cxx-4.8.dylib (0) <55936276-8F8D-3D4D-9D82-08548FA316AB> /Volumes/VOLUME/Emerald-Qt.app/Contents/Frameworks/libdb_cxx-4.8.dylib
0x1095c8000 -        0x1095cafff +libboost_system-mt.dylib (0)  /Volumes/VOLUME/Emerald-Qt.app/Contents/Frameworks/libboost_system-mt.dylib
0x1095d0000 -        0x1095deff7 +libboost_filesystem-mt.dylib (0) <85C65F2C-FCB4-38F0-9CD4-431BF243B211> /Volumes/VOLUME/Emerald-Qt.app/Contents/Frameworks/libboost_filesystem-mt.dylib
0x1095f2000 -        0x109626fff +libboost_program_options-mt.dylib (0) <2E935CBA-E26F-3563-82DD-55B36E9A5A9A> /Volumes/VOLUME/Emerald-Qt.app/Contents/Frameworks/libboost_program_options-mt.dylib
0x109685000 -        0x109691ff7 +libboost_thread-mt.dylib (0)  /Volumes/VOLUME/Emerald-Qt.app/Contents/Frameworks/libboost_thread-mt.dylib
0x10ec15000 -        0x10ec5f98f  dyld (519.2.2) <6695F30B-4E88-3C0B-9867-7D738C44A3E6> /usr/lib/dyld
0x7fff51350000 -     0x7fff521adfff  com.apple.AppKit (6.9 - 1561.20.106)  /System/Library/Frameworks/AppKit.framework/Versions/C/AppKit
0x7fff521ff000 -     0x7fff521fffff  com.apple.ApplicationServices (48 - 50) <7627DBD6-497B-3AB7-9B63-F0532EDF09B8> /System/Library/Frameworks/ApplicationServices.framework/Versions/A/ApplicationServices
0x7fff54eaf000 -     0x7fff54eaffff  com.apple.CoreServices (822.19 - 822.19) <44456ED2-59E4-34CB-B41B-C6A82B269949> /System/Library/Frameworks/CoreServices.framework/Versions/A/CoreServices
0x7fff55e85000 -     0x7fff5624afff  com.apple.Foundation (6.9 - 1451)  /System/Library/Frameworks/Foundation.framework/Versions/C/Foundation

yes. my macOS vm broke, and I haven't had time to re-create it yet.
I was unable to link some of the libs in the img.

thank you for a timely response. it is greatly appreciated. looking forward for more to come from this coin and the rest of the jewels.

we will post a new macOS wallet when we can get one compiled correctly.
for news and updates, join or Discord: https://discord.gg/gWZQVmE


It buffers, which makes it lose all the benefits of streams

That's correct. The purpose of this package is to support minifying in a chain of streams, so it's about the API contract not about efficiency.
If there is already a high quality streaming minifier, I would love to switch to it, but I'm not aware of any.

I started tinkering on a streaming html minifier https://github.com/GrosSacASac/stream-sac/blob/main/source/html/HtmlMinifier.js
What do you think ?


Add onOffline/onError callbacks in serviceWorker.js (next)

Hello,
i added onOffline and onError callback to the serviceWorker.js
They are useful ie. to display user message when in offline
and ie. to log the error on serviceWorker register fail.

Hey! Thanks for this but we're currently switching to Workbox and are going to make service workers super configurable, so this change is out of scope for now.

the 2.0 release has still the old serviceWorker.js though?


Add Buy/Sell Roll FE

Sell Roll

Buy Roll

Codecov Report
Patch and project coverage have no change.

Comparison is base (9234633) 42.13% compared to head (717de87) 42.13%.

:exclamation: Current head 717de87 differs from pull request most recent head 295f8da. Consider uploading reports for the commit 295f8da to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #648   +/-   ##
=======================================
  Coverage   42.13%   42.13%
=======================================
  Files          19       19
  Lines        1329     1329
=======================================
  Hits          560      560
  Misses        709      709
  Partials       60       60

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Changes to install instructions (new node consideration)

Just a quick note that with the new version of node what was once:
npm install --save express cookie-parser body-parser
can now be this:
npm install express cookie-parser body-parser
and npm install --save-dev nodemon
can now be:
npm install -D nodemon
I know that many people may be operating off of older node versions, or using nvm in certain scenarios to use an older node, but, as more go to node 8 and npm 5, these instructions will be more commonplace. Please correct any misunderstanding I may have, and since this is not remotely a high-priority item, no worries if this isn't implemented.

--save is on by default for the new npm? I'll probably leave the --save just to be explicit


Optimise Epoch Transition

Description
With the increase in number of validators the epoch transition is becoming more expensive. Benchmark the transition on MainNet sized states and optimise.

Did a round of optimisation here with some good improvements.  Have logged #5448 as a potential follow up.


Unit tests fail if device is not in English

AssetDefinitionStoreTests.swift:8
XCTAssertEqual failed: ("周三, 05 4月 2023 11:12:34 GMT") is not equal to ("Wed, 05 Apr 2023 11:12:34 GMT")

XMLHandlerTest.swift:916
XCTAssertEqual failed: ("Optional("圣彼得堡")") is not equal to ("Optional("Saint Petersburg")")

Ah, probably OK. Just have to assume it's English for now.