Provide a way to dump different formats to different files

We use svg, traces and collapsed formats at the moment and going adopt jfr and experiment with tree.
Unfortunately, we have to split the one file that async produces into different files by hands.
That requires hardcoding heuristics where to split the file and that heuristics became complicated since formats are changed from time to time and we use different sets of formats for different purposes.
It would be amazing if something like svg=C,svg_file=output.svg,traces=1000,traces_file=traces.txt would be possible.

My intention is to disable multiformat output altogether in favor of one-at-a-time dump option.
Once the profile is collected, you may dump it as many times as you want using different dump options. Right now this is possible with stop command, but I think we need to add a dedicate dump  command to avoid confusion.
stop,svg,file=output.svg
stop,tree,file=output.html
etc.

That would be really cool!
If you could add dumpSVG and dumpJFR to the Java API it would be also of a great help.

Yes, I'll rework Java API, too.
Note that JFR is different from other output formats. As long as it dumps every sample on the go rather than aggregated profile at the end, JFR output must be enabled at the beginning of profiling session. It won't be possible to dump JFR afterwards.

Since the latest release, async-profiler no longer supports multiple formats in -o option.
The recommended way to produce output in different formats is to execute stop command multiple times with different output options.
I decided not to add dumpSVG and other dump method in favor of universal execute() method which handles all available commands and options.
The long term intention is to make the agent produce just one comprehensive format, and convert it to any other format offline.

@apangin (a maintainer of github.com/google/pprof) Consider using pprof's https://github.com/google/pprof/blob/master/proto/profile.proto format - it's a protobuf but it's fairly easy to generate a binary protobuf without actual protobuf library dependency. Then pprof can be used to confirm to any other format (pprof supports a number of formats; it also has an embedded web UI).

@aalexand Thanks. When I do a research, I'll look at this option, too.

@apangin Thanks for this very useful tool.
Do you plan to support pprof format output?
Can you point me to JFR file specification please?

@JigarJoshi As I've told earlier, there are no plans to add more formats - just one output format with pluggable converters.

Can you point me to JFR file specification please?

I don't know if it even exists. You can look at JMC and HotSpot sources for the reference. BTW, Flight Recorder has Java API for parsing .jfr files.

@apangin  do we capture color coding data in JFR format? I am planning to use JFR parser to extract JSON.

@rpulle Yes, async-profiler outputs a frame type in JFR format:
https://github.com/jvm-profiling-tools/async-profiler/blob/b8493976b635028cb6238865bd96f2b4157e333a/src/flightRecorder.cpp#L83-L91

But the simplest format for parsing is probably collapsed. With -a (annotate) option it will supplement each Java method with _[j] suffix.

Thank you. I am trying to make a single parser work for both jcmd JFR and async profiler JFR output.  The parser is working, just wanted to include colors.

@apangin actually there is a tool under jdk bin to parse JFR files.  bin/jfr.  , however async profiler jfr format cannot be parsed using this tool. Can we make JFR format compliant with this tool?
Error:
jfr print: could not read recording at /Users/rpulle/Downloads/Async-profiler-cpu.jfr. File version 0.9. Only Flight Recorder files of version 1.x and 2.x can be read by this JDK.

@rpulle There are different versions of JFR format. async-profiler uses the version compatible with JDK 8, which can be read by JMC 5.5.x (bundled with Oracle JDK 8).
JDK 11 writes a different version of JFR, which is not compatible JMC 5.5.
Async-profiler deliberately writes an older JFR, that is compatible with JDK 8 and can be read by any version of Mission Control. Unfortunately, jfr tool does not support it, as it does not support JFR files written by JDK 8.


🔧 migrate offers page to composition api

tested on /bsx/offers, /snek/offers
migrate to vue composition api
⚠ I am aware of whacked AddressInput  fixed with some neo magic

Sir please resolve congflicts and you have my approval to merge


Add Clara 2E & Elipsa 2E

Untested, hopefully I didn't mess it up ;).
(Also haven't double-checked if the Elipsa & Elipsa 2E are still weirdly classed as Dragon by libnickel).

Thanks 😃

Also haven't double-checked if the Elipsa & Elipsa 2E are still weirdly classed as Dragon by libnickel

If someone has one of those, it'd be useful if they could check so I don't need to figure out the mess of Device::* functions again.


UI frontend

Made all requested changes to revamp the devices table such as:

[x] Rename PID to UUID
[x] Add padding between PID and the search box.
[x] Remove text "Filter by PID" on top of the search box. Add it as a placeholder instead.
[x] Add ability to order by PID also.
[x] Add up/down arrow to make it clear to users that they can order by.

The above picture demonstrates how the table used to long and the following picture will demonstrate how it looks now.

This PR will close #155

[ ] Please change the UUID text, to be next to the filter text box in the table header


Fix InternalAffairs/ExampleDescription offenses

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Updated documentation.
[ ] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

The failing parts of CI are addressed by #30.


Bug: Crimson Scales - Skull - modifier cards

Hi again!
Just looking to start as a Vermling Spiritcaller and was going through the Modifier deck and noticed the following:
4x (+1) Pierce 2 cards
and
0x (+0) Rolling Pierce 3 cards
There should be:
1x (+1) Pierce 2 cards
and
3x (+0) Rolling Pierce 3 cards
Also checked the nightly - there also.

Fix is change "13" to "14" on Lines 936, 941 and 946 of database_cs.js
https://github.com/orichalque/yet-an-other-gloomhaven-companion/blob/master/js/database_cs.js#L936

Hi! It should be fixed on the nightly if you want to check it out @kiwitrav

Awesome, that's working great - thank you!

I'm actually working on #226 , once i'm done with it we'll merge and deploy all the changes

All good - thank you for this! Seriously deckbuilder makes even in-person games better (let's ignore Saw, lol)


Warning for users

Watch out, if you show him places to correct it may result in ban on every related discord server. Sad people.

I would like to congratulate you for becoming the very first person to be blocked from my GH account (〃￣︶￣)人(￣︶￣〃)


Question: Disable deduplication and compression

Is there a way to disable deduplication and compression in noobaa?

To begin with, and this is a very user-friendly option, the config.js file contains most of the capabilities enabled or disabled, you could always change that.
Specifically for encryption, we saw the need and added a more friendly way to do it, please see
https://github.com/noobaa/noobaa-core/pull/5911
This allows this configuration to be passed as an ENV variable mapped to the pod.

Thanks!


fix(base) reduce the amount of timers used

This PR changes the timers so that instead of creating one per
balancer we crete a single global one. The price payed is that
we must store a glocal variable with all the lbs inside.
The DNS refresh is async so no further timers/threads are needed.
This is a pr against 4.1.3; not to be merged into master as it is.

Closing in favor of #104 (we bumped the version to 5.1.1)


Installation on Generic ARM fail to pull landingpage

I'm trying to install home-assistant via hassio on my OrangePi PC board running the armbian OS. Everything seems to be installed fine, but when I try to start hassio I got an error:
gabriel@orangepipc:~$ hassio-start
18-01-29 17:13:30 INFO (SyncWorker_4) [hassio.docker.interface] Pull image homeassistant/armhf-homeassistant tag landingpage.
18-01-29 17:13:32 ERROR (SyncWorker_4) [hassio.docker.interface] Can't install homeassistant/armhf-homeassistant:landingpage -> 404 Client Error: Not Found ("manifest for homeassistant/armhf-homeassistant:landingpage not found").
18-01-29 17:13:32 WARNING (MainThread) [hassio.homeassistant] Fails install landingpage, retry after 60sec
18-01-29 17:14:32 INFO (SyncWorker_9) [hassio.docker.interface] Pull image homeassistant/armhf-homeassistant tag landingpage.
18-01-29 17:14:34 ERROR (SyncWorker_9) [hassio.docker.interface] Can't install homeassistant/armhf-homeassistant:landingpage -> 404 Client Error: Not Found ("manifest for homeassistant/armhf-homeassistant:landingpage not found").
18-01-29 17:14:34 WARNING (MainThread) [hassio.homeassistant] Fails install landingpage, retry after 60sec

I checked and the landing page tag doesn't exist, indeed. I'm not sure how to fix it.
Anyone can help me?

/move home-assistant/hassio-build


WIP, ENH: add docstring cov to html

One of the items on the wishlist for testing docstrings in NumPy / SciPy is the ability to report "coverage" for those docstrings that are probed by doctesting infrastructure. In particular, I had in mind the idea that labels in the html coverage reports that one often finds as part of the CI for large OSS scientific projects might be expanded to include hits on docstrings.
This lead to me to a simple example on StackOverflow by @rhettinger, which really got me excited even more. However, I noticed that executing the exact same code and trying to display with coverage html effectively ignores the docstring using both stable and master branch of coveragepy. I dug a little deeper and noticed that coverage annotate actually has a much shorter / simpler loop for doing the labelling vs. html, and I figured I'd take a look at the html assignment of lines to label classes, and see if I could get it to do what I want.
This PR has a draft of what I had in mind that I initially thought was working, but actually seems to be not quite right yet. There are probably various ecosystem considerations & things like trying to fuse these results with results that don't have docstrings labelled, etc. But if I I can't get this "simple" case to work, the prospects for expanding the "feature" usage to large packages wouldn't be great.
With this feature branch and my .coveragerc initially looking like this:
[html]
docstring_display = True

with the original example code from StackOverflow we get the docstring now highlighted along with the source itself that was ultimately executed by doctest:

and with a doctest SKIP directive added in to Raymond's example, another design question comes up--would it be possible / worth the trouble to have a separate highlight color / category for docstring examples that are not executed? (notice that the source proper is still correctly marked as not executed, since the docstring didn't get executed, although the docstring was "considered but skipped" so perhaps technically "covered"):

So, as I was writing this up, I thought "hey this is working just like the annotate case so must be good," but then I commented out the doctest execution and it seems the docstring lines are just getting labelled as a fallback, not because the tracing infrastructure knows that they are getting executed:

Ok, so maybe I got excited a little too early. That got me curious about Raymond's original example with annotate. If I turn off the docstring execution there by commenting out the doctest.testmod() line I get a result suggesting that docstring hits are not actually meaningful results from coveragepy tracing / interpretation infrastructure:
> def linear(a, b):
>     ''' Solve ax + b = 0

>         >>> linear(3, 5)
>         -1.6666666666666667

>     '''
!     if a == 0 and b != 0:
!         raise ValueError('No solutions')
!     return -b  / a

This suggests to me that there may be another "meta" level to the tracing if we want to handle doctest probing of those string lines as "hits." That might be split into two categories:

was the string "probed" by doctest infrastructure?
was a line in that string exec(compile()) run by doctest or subclassed doctest infrastructure in a library like NumPy / SciPy?

Sorry I've left this for so long: as I understand it, the goal is to mark docstrings as executed if doctest executes them?  I'm not sure what hooks doctest gives us for knowing which are executed.  Or do the line numbers get reported as executed when doctest runs them?  I'll have to do some experiments.
In the examples you show, I'm not sure why line 2 is marked as executed?  It's not executable code, which makes me think the entire docstring is marked executed even if only part of it is.

Sorry I've left this for so long: as I understand it, the goal is to mark docstrings as executed if doctest executes them?

Yes, that would be cool, but as you note it is not immediately clear how to trace through the "meta-execution" that doctest in CPython does. I think I saw an explicit exec when I took a peek inside doctest, but I don't know if that's enough to trace through.

Is this still a valid PR? I know it's in draft format, but it's a few years old at this point with several updates since then


Why resize token embedings when dataset is pororo or flintstone?

Hi!
I am reproducing this work, and I noticed that the token embeding is resized when training on pororo or flintstone datasets.
My question is:

Why do that?
Why resize to those num?

BTW, thanks for your opensource!
look forward to your reply :)

Hi, thanks for your interests. I not quite clear about what dose "resized token embeding" means. Could you please refer the corresponding code using a link?

oh, sorry.
I mean why 'clip_embedding_tokens' or 'blip_embedding_tokens' is different when dataset change.
https://github.com/Flash-321/ARLDM/blob/94725e13e7c790ec1025fd5d485771becc367f02/config.yaml#L36
https://github.com/Flash-321/ARLDM/blob/94725e13e7c790ec1025fd5d485771becc367f02/main.py#L121

Thanks for your comments! this is because we added some new tokens for characters in these two datasets.
https://github.com/Flash-321/ARLDM/blob/34b30703a2caeeb2364bdfb161345027217785c6/config.yaml#L35
https://github.com/Flash-321/ARLDM/blob/34b30703a2caeeb2364bdfb161345027217785c6/config.yaml#L42
https://github.com/Flash-321/ARLDM/blob/34b30703a2caeeb2364bdfb161345027217785c6/datasets/flintstones.py#L36-L39
https://github.com/Flash-321/ARLDM/blob/34b30703a2caeeb2364bdfb161345027217785c6/datasets/pororo.py#L37-L40
as a result, the vocab size of tokenizer has been changed, and we need to resize the token embeddings to make sure the embedding layer can still encode the sentence. And the num can be obtained by printing len(clip_tokenizer) and len(blip_tokenizer) after adding those new tokens.

Got it! Thanks!
That is a fantastic work! Looking forward to more work from you.

Thanks! Feel free to open an issue if you have any further questions!


Subsequent Changes Of Font Attribute Name Doesn't Work

Hello community,
to format a range of a worksheet I develop a class which works very good. You can set border, number formats or background colors. But if I use the same procedure with a font attribute, it doesn't work. The font attribute name is always set to the standard font name - in my case Calibri. As far as I can find out the font name is internally set correct but after I write it with the writer class, the style (XML attribute s) of the cells is set to the standard font name. To set the correct font attributes I set it actually with the set_cell method, it works as expected. But my intention is to set the font attributes independently from the set_cell method.
Thanks for tips and hints.
Cheers
Stefan

In the below program, the value is first written with SET_CELL, and the font "72" is set afterwards with SET_CELL_STYLE and it works for me:

Github user was deleted, so closing.
REPORT.
DATA: gc_save_file_name TYPE string VALUE 'test.xlsx'.
INCLUDE zdemo_excel_outputopt_incl.
START-OF-SELECTION.
  DATA(lo_excel) = NEW zcl_excel( ).
  DATA(lo_worksheet) = lo_excel->get_active_worksheet( ).
  DATA(lo_style) = lo_excel->add_new_style( ).
  lo_style->font->name = '72'. " any font name
  lo_style->font->scheme = zcl_excel_style_font=>c_scheme_none.
  lo_worksheet->set_cell( ip_column = 'A' ip_row = 1 ip_value = 'hello' ).
  lo_worksheet->set_cell_style( ip_column = 'A' ip_row = 1 ip_style = lo_style->get_guid( ) ).
  lcl_output=>output( cl_excel = lo_excel ).

Proposing to close.


JWKs fields are null at validation

Full disclosure
This might not be an issue and/or just me not knowing how to use the library correctly :)
So thank you for your help ahead of time!
So after logging in, I get the token back from the parameters by invoking the TryLogin method but it fails at validation time with an error: Parameter jwks expected!  (see image below)

When I inspect the params object, I can effectively see that the jwks field is empty

However, I do see in the token header the fields corresponding to the jwks from where the public key can be composed. Maybe I'm wrong?
Additionally, I understand that by convention,  the jwks fields can also be obtained from the
http://localhost:5000/.well-known/openid-configuration end point which it does give me the jwks when hitting it:

Am I missing an important step? Any help would be appreciated :)

Thx for this info.
Which Identity-Solution do you use?
Can you send me your discovery document and the document with the keys (jwks)?

No problem.
Identity Server 4, which has an standard Oauth 2.0 generated discovery document. More information here -> http://docs.identityserver.io/en/release/topics/discovery.html?highlight=discovery document
Besides that, I'm not sure what I can send you.. perhaps the text itself shown in the page?
Let me know - Thanks a lot!

Oh, when its IdSvr4 then the discovery document should be fine.
Do you call tryLogin after loadDiscoveryDocument, e. g. within the then-block?
    this.oauthService.loadDiscoveryDocument().then((doc) => {
      this.oauthService.tryLogin();
    });

I'm aksing b/c otherwise tryLogin could be called before the config data arrived.
Are there warning or error messages on the javascript console?

using the tryLogin() inside the loadDiscoveryDocument() did the trick!
Thanks a lot for responding quickly! :)

I am getting the same error... but in my case, I'm using the okta server as suggested by the example. I've just changed the clientId and the root URL (https://dev-{MY_ID_HERE}.oktapreview.com)... Did I forget anything?


pull request for e2e test at 12/29/2016 12:26:06

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


mbedtls incorrectly fails validation of TLS chains that has a useless/broken tail

There has been lots of root certificates removed from mozilla lately that has caused problems for me when using a mbedtls based TLS client. I've found at least two sites on the web send an invalid tail of certificate chains that can be ignored. The end of the tail consists of legacy certificates that is no longer trusted but it doesn't matter because an intermediary certificate in the chain is trusted by an installed root certificate.
E.g. server sends: A <- B <- C
We don't trust C as a "top" certificate anymore because it depends on a deprecated/legacy root certificate. However B is now trusted as a "top" certificate by a new trusted root certificate and we didn't really need to check C in the first place.
In essence, if we can trust any of the certs in the chain as a top certificate, we don't need to proceed by walking further down the chain and can simply return an OK status instead and ignoring the rest of the chain. From my experiments this is how certificate validation is normally done and how it's not done in mbedtls.
This is my suggested fix: https://github.com/jumpstarter-io/librcd/commit/0f7c0386093242abecf7b0eada7abf7607f1f29d
Another fix is of course to tell all the people on the internet with a bad certificate chain to fix it by removing the useless tail certificates but unfortunately few of them will care.

I could argue that the TLS spec says nothing about ignoring invalid tail certificates, but yeah, in the real world that's something we should do, as it does not harm security in any way and provides better interop. I'll review your patch an integrate it.

Exactly, thanks!

Hey, sorry for not coming back to you earlier on this. Your patch looks good and we would like to incorporate it. Can you please get in touch with @pjbakker regarding our contributor license agreement before we integrate your contribution? Thanks!

Hey @pjbakker, can you give me information on the contributor license agreement? I can be contacted at<EMAIL_ADDRESS>

@mpg Confirm that a signed CLA has been received

Any progress on this issue?. It concerns quite a number of certificates, including a number of Amazon and Google certificates.

Woking on it! Unfortunately upon close inspection it appears Hannes' patch is not usable as is, because of interactions with the verify callback, but we're going to implement something along the same general idea.

The fix is pending review. Just FYI, your current patch misses the case A -> B -> C -> D where C is trusted but D is not (ie you need to modify x509_verify_child() too for longer chains).

True, I missed that.


Delete all pins (20 at a time?)

User story

Story
As a file host I want to be able to delete ALL current pins as I load a new consistent set of files into the Swarm.
I'm currently doing "Get list of pinned chunks" followed by "Unpin chunk with given address" for each returned address (times the returned pinCounter).  But imagine my surprise when this doesn't delete all the pins?  Another "Get list of pinned chunks" returns still more (and different) pins.
It seems that "Get list of pinned chunks" only returns 20 pins per invocation, but this is not mentioned in the documentation for the Debug API.  There also does not seem to be a way to specify how many pins are desired.
Is there a way to query ALL the pins?  Or is there a context so that different sets of pins can be queried?
Acceptance criteria
At least document "Get list of pinned chunks" to have a limit of 20 pins.  But hopefully also describe how to query all existing pins.
Or provide an (Debug?) API that deletes all current pins with full understanding of the ramifications of this request.
Background
I have a script that adds lots (1,000s) of files to the swarm and automatically pins each address while also recording said address externally.  As this file set is refreshed, I want to delete all current pins without consulting the external source and before re-adding the new file set.
Tasks

Task
Assignee
Done (1), or 0

I see now in localstore/pin.go:
const (
maxChunksToDisplay = 20 // no of items to display per request
)
// PinnedChunks for now returns the first few pinned chunks to display along with their pin counter.
// TODO: have pagination and prefix filter
So this is a known TODO...

Hi,
Thanks for making this issue. We are looking into it now and will prioritize it accordingly.

I'm currently doing "Get list of pinned chunks" followed by "Unpin chunk with given address" for each returned address (times the returned pinCounter). But imagine my surprise when this doesn't delete all the pins? Another "Get list of pinned chunks" returns still more (and different) pins.

@ldeffenb When you unpin a chunk, what is the pincounter value? only if the pincounter comes to 0 will the chunk not show in "list pinned chunks" API.

@jmozah That is correct.  I'm looping on the returned pinCounter and doing an unpin for each count.  When that loop finishes, the pin is actually gone.  Once I've done that for the 20 that came back from the "list pinned chunks", I do another "list" and rinse and repeat.

@ldeffenb Thanks. So the issue is that we return only 20 in the "list pin" API is it? will your problem get resolved if we add pagination?

@jmozah Yes, pagination on the "list" API will resolve my problem.


Download default branch instead of master branch

Sometimes the GitHub project downloader would fail to download a project because it could not find the zip. Turns out this happens when the project in question does not have a master branch. To prevent this from happening, the downloader now downloads the default branch instead.

Codecov Report

Merging #374 into master will not change coverage.
The diff coverage is 40%.

@@            Coverage Diff            @@
##             master     #374   +/-   ##
=========================================
  Coverage     71.97%   71.97%
  Complexity      361      361
=========================================
  Files            63       63
  Lines          1477     1477
  Branches        183      184    +1
=========================================
  Hits           1063     1063
  Misses          336      336
  Partials         78       78

Impacted Files
Coverage Δ
Complexity Δ

...haapi/miningpipeline/miner/github/SearchOptions.kt
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ngpipeline/miner/github/GitHubProjectDownloader.kt
64% <50%> (ø)
8 <2> (ø)
: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 4c6f1e0...5d87f7c. Read the comment docs.


Iterating all entries in a cache

Split from #92. There are some needs for iterating all entries ((&K, &V)) in a cache.

In order to do this locklessly, it's worth noting that either all keys will be sampled once but values may change, or that the iterator may or may not miss keys that were added during iteration. &V should be sampled by lookup per-key during a streaming iteration if possible, with now-missing keys skipped if the iterator reaches a pair which is no longer present.

I am adding iterator to newly added moka::dash::Cache, which uses dashmap::DashMap as the central storage. Its iterator is a simple wrapper of DashMap iterator, which employs read lock on currently iterating shard of the map.
Here is the pull request:

#101

At this moment, I think this will be good enough. If somebody wants iterator, they could consider using moka::dash::Cache.
I will revisit iterator for other concurrent caches such as moka::sync::Cache and moka::future::Cache. I will do it after I finish implementing other features (e.g. notification).
I will post some thoughts in next comment.

This is my current thoughts on iterators for concurrent lock-free caches.

or that the iterator may or may not miss keys that were added during iteration

Yeah. It will be very hard to guarantee not to miss keys that were added during iteration.
There may be a way. Moka records operation logs for writes (upserts and deletes) in a channel. These logs are used for batch-updating cache policy’s data structures (doc). Iterators could subscribe the operation logs for newly added keys. For example, it will firstly iterate thorough a snapshot of keys, and then iterate thorough newly added keys retrieved from the operation logs. It will repeat until catching up to the operation logs.
However, there is short delay until those operation logs can be read from the channel. So even though iterator will do the above, it can miss last few new keys.
Another idea is to relax the guarantee; making iterator to allow missing some new keys, but do its best to minimize the amount of missed keys.
I think DashMap is taking this approach by making iterator to acquire a read lock on the shard of the map that it is currently iterating. The read lock ensures the shard to block updates, so iterator can guarantee that no new keys are missed within the shard. Iterator visits every shards in the map, one by one, and never go back to the shards it has already visited. So it will miss new keys that has been added to a shard that the iterator has already visited.
These lock-free Moka caches could take a similar approach to DashMap. Moka's cht::HashMap has multiple internal segments (like the shards in DashMap). It will be possible to take a consistent snapshot of keys and values in a segment, although it is a lock-free data structure. And, in Moka, each key and value are wrapped by Arc (The internal data structures as of v0.7.2). So, by cloning these Arcs, a snapshot can retain keys and values that have been deleted from the cache after taking the snapshot.
I do not know which way will be better yet.

Hi @Milo123459 — I am going to review and merge #101 to add iterator to moka::dash::Cache. Once it is merged, you can start trying dash::Cache including iterator.
Add the following to the [dependencies] section of your Cargo.toml:
[dependencies]
moka = { git = "https://github.com/moka-rs/moka", branch = "master", features = ["dash"] }

Then generate the Rust doc using the following command:
$ cargo doc

Open ./target/doc/moka/index.html with a web browser, and navigate to moka::dash::Cache. You may want to start with the example on the page.
I would strongly recommend to try it now (before I publish the new version v0.8.0 to crates.io) to ensure it has everything you need. Once published, it will become harder to change APIs although I will state in the doc that dash::Cache is highly experimental and the API will likely be changed in the future.
I still have some other works to do for v0.8.0 release, so I will not be able to publish it until next weekend (March 27th).

I will revisit iterator for other concurrent caches such as moka::sync::Cache and moka::future::Cache. I will do it after I finish implementing other features (e.g. notification).

Just FYI. I added iterators to all other caches via https://github.com/moka-rs/moka/pull/114 because somebody else asked for it (https://github.com/moka-rs/moka/issues/112). I released Moka v0.8.2 with them.
For the concurrent caches, I took the simplest design that I could came up with. The iterators do not guarantee to return new keys if they were inserted after the iterator was created. (It may or may not return them)
Please see https://github.com/moka-rs/moka/pull/114 or the doc for more details.


Remove version: RussellBanks.Komac version 1.0.5

I only want Komac v2 on winget
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/russellbanks/Komac/releases/download/v1.0.5/Komac-1.0.5.exe
Status Code: 200
(Automated message - build 677)


Add toEigen in generic vector

Doing my part toward #63 I also removed the options DEPENDS_ON_EIGEN_PRIVATE and DEPENDS_ON_EIGEN_PUBLIC in favor of the using the corresponding target in the correct link lists.

Really nice! You can merge it whenever you want.

I would merge this after #70 to fix eventual conflicts.


Window configuration API

Specific Demand

On the desktop platform, the tao::window::WindowBuilder is re-exported so the developer can configure how he wants his window to be (initial size, decorations, title, etc.). If I remember correctly, this was something that needed to be addressed in the future, since re-exporting APIs can cause problems in the long run.
Implement Suggestion

Right now, the DesktopConfig struct owns a WindowBuilder. I propose to replace it with a new WindowConfig struct, but before I submit a PR about that, I would like to know the maintainers vision.
EDIT : This fork has a rough implementation of what I think would be interesting. I made sure to prevent breaking changes by providing convenience methods.

I think we want to adopt a more imperative API for configuring the window - I want to keep this open with the intention of keeping a WindowConfig struct.


networking: add support for setting TCP idle timeout for HTTP services

Please provide a description of this PR:
This is a follow-up to #48009 and this comment.
As the documentation says, connectionPool.tcp settings are common to both HTTP and TCP upstream connections, so I added missing logic to the cluster builder.

Thanks

@jewertow: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio
798f4a24a55fef75109d2be7587f2b965096518b
link
unknown
/test integ-pilot-multicluster

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


Missing Example: How To Mint NFTs with Two Different Sets of Metadata

It's unclear how you mint two different singular NFTs on the same contract with different metadata, which is usually the case for NFT contracts.
(* range of nft tokens *)
type token_def =
[@layout:comb]
{
  from_ : nat;
  to_ : nat;
}

Everything seems to be catered toward a range. Why would a range of non-fungible tokens share metadata?
The test cases only show one case where you want to mint two NFTs with the same metadata:
https://github.com/oxheadalpha/smart-contracts/blob/master/nft_assets/tezos_fa2_nft_tests/test_fa2.py#L132-L153
Is this how I would do two separate ones? How do i check the metadata for a single token? I know there is an exposed token_metadata entry point but I'm not sure what to pass in for the lambda to get back the url in the token metadata/
        self.fa2.mint_tokens(
            {
                "metadata": token_metadata1,
                "token_def": {
                    "from_": 0,
                    "to_": 1,
                },
                "owners": [owner1_address],
            }
        self.fa2.mint_tokens(
            {
                "metadata": token_metadata2,
                "token_def": {
                    "from_": 1,
                    "to_": 2,
                },
                "owners": [owner1_address],
            }

I know that this is not the syntax from testing and finding it attempts to mint 0 tokens
self.fa2.mint_tokens(
            {
                "metadata": token_metadata1,
                "token_def": {
                    "from_": 0,
                    "to_": 0,
                },
                "owners": [owner1_address],
            }
self.fa2.mint_tokens(
            {
                "metadata": token_metadata1,
                "token_def": {
                    "from_": 1,
                    "to_": 1,
                },
                "owners": [owner1_address],
            }

I want each individual token to have its own metadata URL.
In this example, does the metadata URL for token 3 come from the first or second definition in storage here?

It's very odd that they are pairs and not singular token Ids.
(**
Implementation of the FA2 interface for the NFT contract supporting multiple
types of NFTs. Each NFT type is represented by the range of token IDs - `token_def`.
 *)

What does "type" mean? It's unclear. Is one type all of the NFTs on one contract which all have different metadata and therefore a collection of 1000 would always have to and from be 0-1000?

Although I can't manage to write a working lambda, it seems from this
(** Finds a definition of the token type (token_id range) associated with the provided token id *)
let find_token_def (tid, token_defs : token_id * (token_def set)) : token_def =
  let tdef = Set.fold (fun (res, d : (token_def option) * token_def) ->
    match res with
    | Some r -> res
    | None ->
      if tid >= d.from_ && tid < d.to_
      then  Some d
      else (None : token_def option)
  ) token_defs (None : token_def option)
  in
  match tdef with
  | None -> (failwith fa2_token_undefined : token_def)
  | Some d -> d

That it seems to match on tid being greater than or equal to the from, but less than the to (exclusive).
So if I have token 3 and there are two pairs in storage
Pair(2, 3) -> url1
Pair(3, 4) -> url2

It will always pick Pair(3,4), as
3 >= 3 is true
3 < 4  is true

I think we should put this more clearly somewhere as an example for future devs.

The token_metadata endpoint is also a little arcane, as it asks for a lambda as the first parameter and I'm not sure what sort of function it expects.

This is a special way to implement contract view entry points. Tezos will have Solidity-like view entry points with the next protocol update. At the moment, view entry points use continuation passing style pattern when the parameter contains a callback lambda.
If the contract A wants to get some value from the contract B, it should call contract's A view entry point passing parameters (like token_id) and a callback. Contract B then will call a provided callback entry point of the contract A, passing the requested value.
In general, view entry points cannot be called off-chain (only by other contracts). But Taquito library has special API which lets to inspect view entry points off-chain: https://tezostaquito.io/docs/lambda_view/

It's unclear how you mint two different singular NFTs on the same contract with different metadata ...
Everything is catered toward a range of NFT IDs with the same metadata.

Yes, this is a special NFT contract which represents "semi-fungible" NFTs.
You might want to look at NFT collectibles example: https://github.com/oxheadalpha/smart-contracts/tree/master/collectibles
https://github.com/oxheadalpha/smart-contracts/blob/master/collectibles/ligo/src/fa2_single_asset.mligo contract can be originated with a fixed NFT collection where each NFT has different metadata.
You may also have a look at https://github.com/tqtezos/minter-sdk which has a variety of fungible and non-fungible token contracts implementation or take the NFT contract from https://github.com/oxheadalpha/nft-tutorial


Slurm Perl API

Environment:

aws-parallelcluster-2.9.1

I am trying to install the slurmtop tool for monitoring job status. It requires the SLURM Perl API. Is this installed by default on aws-parallelcluster? It seems to built by default in Slurm installs.

Hi @rkarnik-kymera
yes we should be installing perlapi as part of the make install-contrib step we perform when building Slurm. Are you facing specific issues?
For reference here is the script used to build and install Slurm: https://github.com/aws/aws-parallelcluster-cookbook/blob/develop/recipes/slurm_install.rb#L56-L68

Thanks @demartinofra, I now see the built Perl modules at /opt/slurm/lib64/perl5. I was trying to build a Perl script that uses these, so just need to adjust @INCLUDE, I think.


fix: json view number bug

Fix #235

Added unit tests for json.js

:tada: This PR is included in version 3.36.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


LeftoverDataException on reading xls file with HSSFWorkbook

I'm receiving the following error while trying to read a xls file in order to extract it's text.

NPOI.HSSF.Record.LeftoverDataException: Initialisation of record 0x203(NumberRecord) left 4 bytes remaining still to be read.
at NPOI.HSSF.Record.RecordInputStream.get_HasNextRecord() in D:\github\npoi\main\HSSF\Record\RecordInputStream.cs:line 248
at NPOI.HSSF.Record.RecordFactoryInputStream.NextRecord() in D:\github\npoi\main\HSSF\Record\RecordFactoryInputStream.cs:line 255
at NPOI.HSSF.Record.RecordFactory.CreateRecords(Stream in1) in D:\github\npoi\main\HSSF\Record\RecordFactory.cs:line 466
at NPOI.HSSF.UserModel.HSSFWorkbook..ctor(DirectoryNode directory, Boolean preserveNodes) in D:\github\npoi\main\HSSF\UserModel\HSSFWorkbook.cs:line 316

Here is my code
public string Process(byte[] attachment)
{
	contentInConversion = string.Empty;
	using (var stream = new MemoryStream(attachment))
	{
		POITextExtractor excelExtractor = null;
		try
		{
			var xssWorkbook = new HSSFWorkbook(stream);
			excelExtractor = new ExcelExtractor(xssWorkbook)
			{
				IncludeBlankCells = false,
				IncludeHeaderFooter = false,
				IncludeCellComments = false,
				FormulasNotResults = false,
			};
			var stringCellValue = excelExtractor.Text.Clean();
			AddCellValue(stringCellValue);
		}
		catch (Exception)
		{
			return contentInConversion;
		}
		finally
		{
			if (excelExtractor != null)
			{
				excelExtractor.Close();
				excelExtractor = null;
			}
		}
	}
	return contentInConversion;
}

Find below the file where the issue is being experienced.
LeftoverDataException_Exception.zip

Where is this xls file from? Is it generated by 3rd party software? Can you resave the file by Microsoft Excel and try again?

@tonyqus , I don't have the information about how did the file was generated. The file was sent by our client's client - I'll try to discover that. However, when I open and save it using my office365 installation, keeping the extension (.xls), it works fine.

Resaving the file will definitely fix the BIFF record issue. That's why I'm asking you where is the file from. The original file looks to be broken. You have to check with the source of the file and see what's going on on their side.


Plan.Success not working as intended

Success seems to return an incorrect value

Thank you for your feedback.
Bug is fixed in Version 0.9.1 - should be available on nuget.org.


exec does not set TERM env when -t passed

Passing -t to docker run will set the TERM environment variable. Doing so to docker exec will not. This causes things like htop to not work.
$ docker run -d --name foo debian bash -c "while true; sleep 1; done"
2fc4a0d2fc6d
$ docker exec -ti foo env
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
HOSTNAME=2fc4a0d2fc6d

Or a case that the end user is more likely to experience:
$ docker run -d --name foo debian bash -c "while true; sleep 1; done"
2fc4a0d2fc6d
$ docker exec foo "apt-get update && apt-get install -y htop"
....
$ docker exec -ti foo htop
Error opening terminal: unknown.

Versions and stuff:
$ docker version
Client version: 1.3.1
Client API version: 1.15
Go version (client): go1.3.3
Git commit (client): 4e9bbfa
OS/Arch (client): linux/amd64
Server version: 1.3.1
Server API version: 1.15
Go version (server): go1.3.3
Git commit (server): 4e9bbfa
$ uname -a
Linux hostname 3.13.0-35-generic #62-Ubuntu SMP Fri Aug 15 01:58:42 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
$ docker -D info
Containers: 186
Images: 2828
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Dirs: 3206
Execution Driver: native-0.2
Kernel Version: 3.13.0-35-generic
Operating System: Ubuntu 14.04.1 LTS
Debug mode (server): false
Debug mode (client): true
Fds: 135
Goroutines: 224
EventsListeners: 1
Init Path: /usr/bin/docker
WARNING: illegal base64 data at input byte 4
WARNING: No swap limit support

Apologies if this has already been reported/fixed (I didn't test against master, even though I know I should!)

closing as dup of https://github.com/docker/docker/issues/8755

Workaround:
docker exec -it $container /bin/bash -c "export TERM=xterm; exec bash"

Works very well.

I find it easier to remember:
docker exec -ti test env TERM=xterm bash -l

:+1:

👍

kramarama is very to remember

@jfrazelle this isn't a duplicate, can you reopen?
#8755 is about a nasty issue to do with invalid file descriptor symlinks and mounting devpts, more fully discussed at #11462.
This issue is about an environment variable (TERM) differing inside docker exec depending on whether you initially started the container with -t or not.

@samshiles's answer is worked for me.

Just encountered the same issue.
At least the doc could mention that "-t" has a slightly different effect with "run" and with "exec".
What is misleading is that:

both options have the same name (-t, --tty)
both options descriptions are the same (Allocate a pseudo-TTY)

It would be nice if we could pass multiple environment variables (ala -e)

@freman The production environment is the last step of a container workflow. Before you have your perfect black production box there's a lot of work you have to do with exec.

@vipseixas we've been doing this for almost two years, it's literally "let the devs tinker with their php in production" type work (because face it, it's easier to syntax error 30 live instances of the site in docker than it is to work offline, test, etc), the rest of everything is stable. The problem is as the exec api stands it's not suitable for doing that at least not on a production machine. No tidy way to clean up after someone that just closes their terminal instead of exiting properly, no tidy way to pass env, probably more I'm forgetting after so little sleep. I have been trying to write a tool that lets us exec right into a shell the thing remotely (ala ssh but reverse proxied so there's no ports exposed, with full session recording, and easier to integrate auth) but these issues make it un-appealing

any chance to get this one fixed, to make using docker exec more comfortable?

FYI terminal copy + cols/rows fix
kubectl exec -it $1 -- /bin/bash -c "stty rows $(tput lines) cols $(tput cols) && export TERM=$TERM && exec bash"
https://github.com/grosser/dotfiles/commit/eca07c9c69deccdc19a5bcaf7b46b72f14577711

We just merged a pull-request that automatically sets TERM if docker exec is started with a tty (-t) https://github.com/docker/docker/pull/26461
This change will go into the next release (1.13)

@thaJeztah Thanks! What about docker-compose exec? I can't use -t option there...

@bocharsky-bw docker-compose should use the same API, but if it's not working with the 1.13 daemon, it's best to open an issue in the docker-compose issue tracker; https://github.com/docker/compose/issues

我今天也遇到这个问题，估计是TERM的问题。谢谢。

Google translate;

I have encountered this problem today, it is estimated that the problem of TERM. Thank you.


hugo: Modified blue tint for base styling

As per commit.
To test and compare, go to:
https://deploy-preview-474--cue.netlify.app/examples/shortcodes/info
https://deploy-preview-474--cue.netlify.app/examples/shortcodes/buttons

Question: is $c-blue still used anywhere after this change? If not I would suggest assigning the value that is now assigned to $c-blue--light to $c-blue and getting rid of --light

Updated accordingly.

The $c-blue--light from before which is been replaced by $c-blue is not changed in all scss files yet. I now get the very light blue as nav text for instance. Please check all usage of $c-blue--light and check if it needs to be replaced with $c-blue now.

I believe I caught all of them now, but let me know if you still find a few out of place.


Type system information for statically known data size

It would be really cool to have compiletime info about datatype size.  This has the potential to make both the generics and TH derivations more efficient.  Certainly for TH it would cause the compiler to do less overall work.  It'd also allow us to use functions that demand ConstSize and have them fail statically.  This way the user can't accidentally make their values VarSize (can be much less efficient).
We might need to use some TH hacks to reify runtime Storable info as compiletime type info.
I'm imagining something like the following, using type nats, data kinds, and type families:
-- Can I use Nat here?  You know what I mean.
data StaticSize = KnownSize Nat | UnknownSize

class Store a where
    type StoreSize :: a -> StaticSize
    type StoreSize a = UnknownSize
    -- ...

type instance StoreSize () = 'KnownSize 0
type instance StoreSize Int32 = 'KnownSize 4
-- etc

Probably good to have it as an associated type family, so that it can be given the default of
We'd also likely want the Size a type be a GADT that enforces that ConstSize is yielded when StoreSize a ~ KnownSize n.  Marking this "breaking change" as such.

I came here to open this same Issue and fortunately found yours first. My motivation is to be able to use the type-checker to ensure that some serialization uses the more efficient ConstSize path.
I suggest the following.
type family StoreConstSize a :: Maybe Nat

data Size :: * -> * where
  ConstSize :: (StoreConstSize a ~ 'Just n,KnownNat n) => Proxy# n -> Size a
  VarSize :: (StoreConstSize a ~ 'Nothing) => !(a -> Int) -> Size a

Though I'm worried that KnownNat essentially being Integer may cost performance. Thus, the following ought to be a better performing middle-ground.
type family StoreConstSize a :: Bool

data Size :: * -> * where
  ConstSize :: (StoreConstSize a ~ 'True) => !Int -> Size a
  VarSize :: (StoreConstSize a ~ 'False) => !(a -> Int) -> Size a

And last, I'm not sure how the pattern exhaustiveness checker will handle the new Size. If it's troublesome, you may want to consider a route using a data family for Size instead of a GADT.


Проблема с ининцализацией

Иногда виджет инициализируется только после перезагрузки страницы
Например http://www.eyeseestyle.com/en/diary/big-geometric-earrings внизу поста.

English, please :)
I don't really see the problem, could you elaborate what the problem is?

No response for few days. Is your issue already solved?

I don't see the issue, and you seem to solve your issue too. I'll close the issue, then.

Sorry, it was a problem with initialization. The fix is remove turbolinks gem from the project. :)


OEP: OEP Purpose and Guidelines

OEP stands for ONT Enhancement Proposal. An OEP is a design document providing information to the ONT community, or describing a new feature for ONT or its processes or environment. The OEP should provide a concise technical specification of the feature and a rationale for the feature. The OEP author is responsible for building consensus within the community and documenting dissenting opinions.

The standard can be found here: https://github.com/ontio/OEPs/blob/master/OEP-1.mediawiki


Statefulness in Text generation using a RNN with eager execution

URL(s) with the issue:
https://www.tensorflow.org/tutorials/sequences/text_generation
Description of issue (what needs changing):
Clear description

Two questions regarding the stateful defined in the tutorial.
The input data was originally a long text. It got cut into sequences with length 100, shuffled, and packed into batches of size 64. However, according to the documentation for the statefulness

You can set RNN layers to be 'stateful', which means that the states
computed for the samples in one batch will be reused as initial states
for the samples in the next batch. This assumes a one-to-one mapping
between samples in different successive batches.

After pre-processing the data as discussed above, there is no such one-to-one mapping is not there, i.e. the samples in different batches are independent instead of related. This is confirmed by running the following code after the code for create training batches, which print out the successive samples with index 0 in the first two batches.
for input_example, target_example in  dataset.take(2):
  print ('Input data: ', repr(''.join(idx2char[input_example[0, :].numpy()])))

outputs
Input data:  'n that perished vessel the dowry of his\nsister. But mark how heavily this befell to the\npoor gentlew'
Input data:  "y enrich'd\nWith politic grave counsel; then the king\nHad virtuous uncles to protect his grace.\n\nFirs"

In addition, if stateful=True, it is suggested to set shuffle=False in the model.fit(), which is also missing in the tutorial.

specify shuffle=False when calling fit().

The problems above are really confusing and any discussions are welcome.

https://www.tensorflow.org/beta/tutorials/text/text_classification_rnn
The same thing is true about the same tutorial for TF 2.0 and it is very confusing.

@ZhiliangWu
Is this still an issue

No. This is fixed.


.Net Core 3.1 on Windows throws NotImplementedInReferenceAssemblyException

We're getting a Xamarin.Essentials.NotImplementedInReferenceAssemblyException when trying to use the Xamarin.Essentials.FileSystem.CacheDirectory from a .Net Core 3.1 console app on Windows:
Xamarin.Essentials.NotImplementedInReferenceAssemblyException : This functionality is not implemented in the portable version of this assembly. You should reference the NuGet package from your main application project in order to reference the platform-specific implementation.

We are referencing Xamarin.Essentials from the project, so we're a bit puzzled on how to move forward. Can you provide some guidance?

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

ID: 69057ea4-203a-b40b-03d4-4dd7a6df8dc2
Version Independent ID: 293454ff-7958-ef68-7477-dbfb084d3da0
Content: FileSystem Class (Xamarin.Essentials)
Content Source: docs/en/Xamarin.Essentials/FileSystem.xml
Product: xamarin
Technology: xamarin-essentials
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Hi @jmosbech, issues with the product should be directed to the team on the open-source GitHub repo.
I'm not sure .NET Core 3.1 Console apps are a supported platform, so that would be something to check with the engineering team.


ORC-301. extractFileTail should open a file in try statement

This PR moves the file open position to prevent opened file leakage more robustly.

Hi, @omalley .
Could you review this PR?

Thank you, @prasanthj . I updated according to your advice.

lgtm, +1

Thank you so much, @prasanthj !

Thank you for merging, @prasanthj !


Fix stm32f1xx/olimex-p103 UART

This fixes the UART configuration on Olimex STM32-P103 and also fixes the uart driver for STM32F1 which uses a slightly different mechanism for configuring alternate function for GPIOs. Hopefully this will live a short life and be replaced by a unified stm32-common uart driver!

Any feedback here?


Cannot find protocol declaration for 'NFCTagReaderSessionDelegate'

Hello, I am trying to read some NFC Tags, but it gives me this error when I'm trying to build the project from XCode to my Iphone 8.
XCode version: 10.3
iOS version: 12.4

Hi, you can use v1.2.2 for now if you don’t need the upcoming iOS13 features (which requires Xcode 11 beta). Thanks.

Yep, you're right, I could figure it out how to make it work even with XCode10.3 (with the latest @beta version). Also, on ndef-lib/ndef-text.js you can add the UTF-8 decoder (it tooked me a while until I figured out that is it decoding for UTF-16):
`var out, i, len, c;
var char2, char3;
    out = "";
    len = data.length;
    i = 0;
    while(i < len) {
        c = data[i++];
        switch(c >> 4)
        {
            case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7:
            out += String.fromCharCode(c);
            break;
            case 12: case 13:
            char2 = data[i++];
            out += String.fromCharCode(((c & 0x1F) << 6) | (char2 & 0x3F));
            break;
            case 14:
                char2 = data[i++];
                char3 = data[i++];
                out += String.fromCharCode(((c & 0x0F) << 12) |
                    ((char2 & 0x3F) << 6) |
                    ((char3 & 0x3F) << 0));
                break;
        }
    }

    return out;`


Feature request: keyboard shortcuts

It would be really great to be able to open the menu and navigate/autofill using only the keyboard.

I'll add a twist to that - ideally this would come together with an option to hide the toolbar icon. CKP is awesome, but I'd prefer it to be summonable via keyboard, instead of having to deal with yet another button/icon.

I would love to see some aditional shortcuts, too. Most importantly for autofill.

Another vote for shortcut for auto-fill if there is only one entry available; provide choice of entries navigable with arrows if there are several of them

On OS X, the icon tells that Ctrl-Shift-Space should open the menu -- but instead, Cmd-Shift-Space is assigned by default. And Cmd-Shift-Space is the typical choice for Show Spotlight Search Field when Cmd-Space is assigned for switching between several national keyboards (eg Russian vs English). So my vote is to change the OS X default to something else, like Ctrl-Shift-Space.

CKP is cool, but please add shortcut for autofill.


[Build] Add initial Makefile support

This adds support for generating Makefile for simple Swift packages.

/cc @parkera FYI... after discussion with the compiler team we were not able to agree on migrating to a model where we would use a pre-existing toolchain. One of the major concerns was that this put an undue reliance on cross-compilation (which is not yet very mature) for bringing up new platforms for Swift.
In lieu of that, we are adding this new mode to SwiftPM... the idea is that Swift.org projects will use SwiftPM for all major development, but in CI we will use this mode (via a checked in Makefile) to bootstrap. Project-specific scripts/processes will be expected to keep the checked in Makefile in sync (although we've tried to structure the generator so this won't need to change that often, typically when targets are added or removed).
We can also use this as an integration point for things that Swift OSS projects need in their build, but that SwiftPM doesn't support natively yet.
Please let me know if you have any questions / concerns!

Can this code actually generate a Makefile that is able to compile swift-corelibs-foundation, or libdispatch?

Also, see: https://forums.swift.org/t/factoring-out-a-new-repo-for-shared-code-between-llbuild-swiftpm/11429/6

@parkera Do you have Swift packages written for those projects yet? It only generates from a Swift package. The goal is that it would support whatever SwiftPM supports that is used by those projects.

No -- so we have a bit of a chicken and egg problem.
Is it possible for an expert at swiftpm to try and build a package for swift-corelibs-foundation? I wouldn't really know where to start.

Sure, will do.


analytics-browser logs default events message

Expected Behavior
If LogLevel is set to None,Error or Warn it shouldn't log the default event message e.g.:
Your Amplitude instance is configured to track Page Views, Sessions. Visit https://www.docs.developers.amplitude.com/data/sdks/browser-2/#tracking-default-events for more details.

Current Behavior
Even when setting the log level to None the sdk still logs the message
Steps to Reproduce

set one or many default events to true
set LogLevel to None

Environment

JS SDK Version:
"@amplitude/ampli": "^1.27.1",
"@amplitude/analytics-browser": "^2.1.2"
Installation Method: yarn
Browser and Version: Chrome, FireFox (latest)

Hi @jibow,
This is intentional. We implemented this because the default event tracking can impact the event volume. We want to ensure that customers are aware when they enable default event tracking. So they can turn it off if they preferred.
I'm closing this ticket now, as this is not a bug.

default event tracking can impact the event volume. We want to ensure that customers are aware when they enable default event tracking.

I'm afraid that a customer can't make such a conclusion from the log message or docs.
Is there a way to add this warning to the message or update the docs by the link?

I'm closing this ticket now, as this is not a bug.

It's not, but what if a customer intentionally enables this feature and knows the consequences? Why do you warn him again?

Hi @glung , we had an update to change the DET message log level to warning. I will release a new version soon.


Redis Inconsistencies: Hdel/expire doesn't delete the parent container when elements hit 0

I'm building unit tests for the database to gauge how well Ardb matches Redis semantics, but it seems like I'm hitting an issue where a hash container is not cleared automatically when the amount of elements hit 0, or the timeout is hit (which is default redis behavior for every container, except redis 5.0 streams).
To reproduce
> hset test a 1
> hdel test a
> exists test
  shows: 1 expected: 0 (which is what redis will output)

same situation with expire:
> hset test a 1
> expire test 1
_<after a second>_
> exists 1
  shows: 1 expected: 0
> keys test
  shows: "test" expected: (empty list or set)
> ttl test
  shows: -1 expected: -2
> hget test a
  shows: 1 expected: (nil)

I hope I can work around these inconsistencies for the time being, but I would greatly appreciate a proper fix for this problem.. Willing to buy some ko-fi for you for the work ;)

for hset/hdel, u need to change redis-compatible-mode to yes in ardb.conf
for the issue exists after expire,  currently exists/keys/ttl does not check the key expire or not, there is a background task to do the expire work. this can be improved later.


Configuration files.

So, currrently CustomRealms does not support configuration files, The only way that is right now to do it is to use a 3rd party module to create the configuration files if there isn't one, Which is okay, But wouldn't it be nice to have that supported by CustomRealms natively.
Thank you,
Andrei Balan

was added by someone else. different a little but yeah.


NSD Service Implementation

NSD Service is necessary to use NDI on Android. NDI is an video and audio transer technology from NewTek and it is widely used especially in virtual production. NewTek released an official Unreal Engine plugin but it doesn't work on Android. So, I created an Android one. I use this plugin and nsd implementation to finish it.

Looks great, thanks for the contribution!


run as root is experimental!!! checking for server..fail

OnePlus/OnePlus5T/OnePlus5T:8.1.0/OPM1.171019.011/06131716:user/release-keys
checking for server..fail
[command] pm path me.piebridge.brevent
please report bug to https://github.com/brevent/Brevent/issues with log below
--- crash start ---
[command] logcat -b crash -t '07-08 16:48:59.000' -d
--- crash end ---
--- brevent start ---
[command] logcat -b main -t '07-08 16:48:59.000' -d -s BreventLoader BreventServer
07-08 16:48:59.004 12493 12493 I BreventLoader: sdk: 27
07-08 16:48:59.037 12500 12500 I BreventLoader: pm path me.piebridge.brevent
07-08 16:48:59.047 12500 12500 I BreventLoader: line:
07-08 16:48:59.048 12500 12500 E BreventLoader: can't get path
07-08 16:48:59.049 12493 12493 E BreventLoader: worker 12500 exited with status 255
--- brevent end ---

please report bug to https://github.com/brevent/Brevent/issues with log above
WARNING: run as root is experimental!!!
checking for server..fail
[command] pm path me.piebridge.brevent
please report bug to https://github.com/brevent/Brevent/issues with log below
--- crash start ---
[command] logcat -b crash -t '07-08 16:48:59.000' -d
--- crash end ---
--- brevent start ---
[command] logcat -b main -t '07-08 16:48:59.000' -d -s BreventLoader BreventServer
07-08 16:48:59.004 12493 12493 I BreventLoader: sdk: 27
07-08 16:48:59.037 12500 12500 I BreventLoader: pm path me.piebridge.brevent
07-08 16:48:59.047 12500 12500 I BreventLoader: line:
07-08 16:48:59.048 12500 12500 E BreventLoader: can't get path
07-08 16:48:59.049 12493 12493 E BreventLoader: worker 12500 exited with status 255
07-08 16:48:59.424 12520 12520 I BreventLoader: sdk: 27
07-08 16:48:59.460 12524 12524 I BreventLoader: pm path me.piebridge.brevent
07-08 16:48:59.473 12524 12524 I BreventLoader: line:
07-08 16:48:59.474 12524 12524 E BreventLoader: can't get path
07-08 16:48:59.475 12520 12520 E BreventLoader: worker 12524 exited with status 255
--- brevent end ---

please report bug to https://github.com/brevent/Brevent/issues with log above
WARNING: run as root is experimental!!!
checking for server.fail
[command] pm path me.piebridge.brevent
please report bug to https://github.com/brevent/Brevent/issues with log below
--- crash start ---
[command] logcat -b crash -t '07-08 16:48:59.000' -d
--- crash end ---
--- brevent start ---
[command] logcat -b main -t '07-08 16:48:59.000' -d -s BreventLoader BreventServer
07-08 16:48:59.004 12493 12493 I BreventLoader: sdk: 27
07-08 16:48:59.037 12500 12500 I BreventLoader: pm path me.piebridge.brevent
07-08 16:48:59.047 12500 12500 I BreventLoader: line:
07-08 16:48:59.048 12500 12500 E BreventLoader: can't get path
07-08 16:48:59.049 12493 12493 E BreventLoader: worker 12500 exited with status 255
07-08 16:48:59.424 12520 12520 I BreventLoader: sdk: 27
07-08 16:48:59.460 12524 12524 I BreventLoader: pm path me.piebridge.brevent
07-08 16:48:59.473 12524 12524 I BreventLoader: line:
07-08 16:48:59.474 12524 12524 E BreventLoader: can't get path
07-08 16:48:59.475 12520 12520 E BreventLoader: worker 12524 exited with status 255
07-08 16:48:59.676 12533 12533 I BreventLoader: sdk: 27
07-08 16:48:59.732 12537 12537 I BreventLoader: pm path me.piebridge.brevent
07-08 16:48:59.743 12537 12537 I BreventLoader: line:
07-08 16:48:59.743 12537 12537 E BreventLoader: can't get path
07-08 16:48:59.744 12533 12533 E BreventLoader: worker 12537 exited with status 255
--- brevent end ---

please report bug to https://github.com/brevent/Brevent/issues with log above

您的系统 pm path me.piebridge.brevent 无法取得路径。
如果 root 了，请联系 root 提供商；如果没有 root，这不可能...


Question: Is whitelist or something else required ? Tiles not displaying anymore (iOS/plugin.google.maps 1.3.1)

My map used to work fine but since I have upgraded to iOS9/1.3.1 all I am getting is a map with no tiles, although markers and circling are displaying fine :

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26864662-question-is-whitelist-or-something-else-required-tiles-not-displaying-anymore-ios-plugin-google-maps-1-3-1?utm_campaign=plugin&utm_content=tracker%2F637500&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F637500&utm_medium=issues&utm_source=github).

Whitelistplugin is required, but it looks like your API Key is wrong

Hi @Hirbod, my bad for not testing this out with Xcode and just relying on the simulator. Xcode gave it away and you were right. No idea why my iOS API key seems to have changed. All this iOS9 changes are pretty nerve-racking

Yeah, iOS 9 and ATS (Apple Transport Security), urlscheme, Bitcode and all that stuff is extremely annoying. Wasted a couple of days to fix my apps, too.


FIX: prvd connectors list

This should be returning the list of connectors for the authorized user. Instead, it is returning a 401. Running this command in Postman results in a 200 status. It's possible the CLI is using the wrong token to make the request. The request should be made with either an organization token or an application token. Update the CLI command with the following options:
If user types prvd connectors list a menu should be presented that prompts the user to select whether to retrieve connectors for an (A)pplication, an (O)rganization, or (H)elp.

If user selects A, they should select an Application from their list of Applications and the request should be made using that Application's JWT
If user selects O, they should select an Organization from their list of Organizations and the request should be made using that Organization's JWT
If user selects H, the result should display a list of available commands and flags, including prvd connectors list --organization string and prvd connectors list --application string


CUDA illegal memory access when training with web viewer running

Hi, first of all just wanted to thank you for this amazing project!
I've wanted to leverage depth camera as a prior for training gs for a while and can't believe it took me this long to stumble upon this project.
I'm currently facing this issue when training with nerfstudio viewer on:
Traceback (most recent call last):
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\haven\miniconda3\envs\nerfstudio\Scripts\ns-train.exe\__main__.py", line 7, in <module>
    sys.exit(entrypoint())
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\scripts\train.py", line 262, in entrypoint
    main(
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\scripts\train.py", line 247, in main
    launch(
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\scripts\train.py", line 189, in launch
    main_func(local_rank=0, world_size=world_size, config=config)
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\scripts\train.py", line 100, in train_loop
    trainer.train()
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\engine\trainer.py", line 261, in train
    loss, loss_dict, metrics_dict = self.train_iteration(step)
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\utils\profiler.py", line 112, in inner
    out = func(*args, **kwargs)
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\engine\trainer.py", line 496, in train_iteration
    _, loss_dict, metrics_dict = self.pipeline.get_train_loss_dict(step=step)
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\utils\profiler.py", line 112, in inner
    out = func(*args, **kwargs)
  File "C:\Users\haven\miniconda3\envs\nerfstudio\lib\site-packages\nerfstudio\pipelines\base_pipeline.py", line 302, in get_train_loss_dict
    metrics_dict = self.model.get_metrics_dict(model_outputs, batch)
  File "C:\Users\haven\code\nerfstudio\dn-splatter\dn_splatter\dn_model.py", line 750, in get_metrics_dict
    "rgb_mse": float(rgb_mse),
RuntimeError: CUDA error: an illegal memory access was encountered
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with `TORCH_USE_CUDA_DSA` to enable device-side assertions.

My data was captured with an azure kinect sensor using SAI, at first I used the included process_sai.py to preprocess the recorded data, but the transform.json output gave camera intrinsics that nerfstudio's undistort function didn't like where k4 wasn't 0, so I copied the camera intrinsic values from sai-cli process (which gave k1 k2 p1 p2 = 0, not sure the intrinsic values are important on the kinect) and training starts properly now.
When I run ns-train while having the nerfstudio web viewer on, at around 5000 - 7000 steps it would throw a CUDA illegal memory access error, however without the web viewer running it would run without complaining. I've tried running ns-train multiple times with and without the web viewer and it only complains when web viewer is running. Has anyone seen similar behaviors?
System info:
>>> torch.__version__
'2.1.2+cu118'

>conda list nerfstudio
# Name                    Version                   Build  Channel
nerfstudio                1.1.3                    pypi_0    pypi

>nvidia-smi
Sat Oct 19 21:03:41 2024
+-----------------------------------------------------------------------------------------+
| NVIDIA-SMI 560.94                 Driver Version: 560.94         CUDA Version: 12.6     |
|-----------------------------------------+------------------------+----------------------+
| GPU  Name                  Driver-Model | Bus-Id          Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |           Memory-Usage | GPU-Util  Compute M. |
|                                         |                        |               MIG M. |
|=========================================+========================+======================|
|   0  NVIDIA GeForce RTX 3090      WDDM  |   00000000:06:00.0  On |                  N/A |
|  0%   62C    P2            212W /  350W |    3135MiB /  24576MiB |     76%      Default |
|                                         |                        |                  N/A |
+-----------------------------------------+------------------------+----------------------+

Thanks again

@Haven-Lau , I have seen a similar phenomena in my early experiments. What I found was that the viewer can crash if you move the camera in the viewer so that it does not see the Gaussian scene properly (like turning 180 degrees and looking at nothing). I wonder if this is the same issue for you.

@maturk Thanks for the quick reply!
Yes it does sound similar, I find it to crash regardless of where I'm looking (eventually if I have the viewer running) but it is definitely more likely to crash when I pan around a lot very quickly or stare at nothing. I wonder if it's some race conditions between ns-train dn-splatter and the viewer. However today I had my first crash crash without running the viewer. Is there a way to save checkpoints throughout the training process instead of only at 100%?

Just to make sure, you are using nerfstudio v1.1.3 and gsplat v1.0.0?

Correct
# Name                    Version                   Build  Channel
nerfstudio                1.1.3                    pypi_0    pypi
gsplat                    1.0.0                    pypi_0    pypi

I'm running windows hopefully that's not the cause, but I can try to spin up an ubuntu env at some point since I couldn't get the download scripts running on windows anyways due to the cli commands on different OS I think

Have u tried any other dataset if it occurs there? I am wondering if there are some issues with the optimization (densification/culling) due to the depth supervision. Pictures of the scene when or near the crash might help me debug as well. Maybe try turn off depth loss and see if crash still happens in that scenario.

For my own scene I turned on --pipeline.model.use-normal-loss True --pipeline.model.use-normal-tv-loss True
and that caused it to crash at 51% (15400 steps), without normal loss it trains to 100% without crashing; I'm not loading any normal maps.
This is what it looked like before ~1000 steps before the crash (this time with viewer on it crashed at 12xxx steps intead of 15xxx)

This is one of the training input

I'll try training again with one of the mushroom dataset and report back

I tried using mushroom honka kinect short raw dataset and processed the raw camera and depth mkv using process_sai.py, then ran
> ns-train dn-splatter --data data\honka_processed
--pipeline.model.use-depth-loss True
--pipeline.model.depth-lambda 0.2
--pipeline.model.use-normal-loss True
--pipeline.model.use-normal-tv-loss True
--pipeline.model.normal-supervision depth
normal-nerfstudio --load-normals False

This time I was able to use normal-loss and normal-tv-loss without crashing
However this time I saw similar degrading issue as https://github.com/maturk/dn-splatter/issues/68 where towards the end of the training process a bunch of big splats got introduced and the some surfaces now have holes

Could you see if you can reproduce similar issues with the mushroom dataset using the same steps?
Eventually I want to figure out how to process my own raw kinect data using the same steps as demonstrated in the mushroom paper as the output from that seems to be very good, there seems to be quite a gap between processing raw kinect data using process_sai tool vs the preprocessed kinect data given by mushroom dataset, or perhaps do you think it is my ns-train params?

Hi, @Haven-Lau may I ask which camera pose you use for mushroom dataset, mushroom dataparsers in dn-splatter also support kinect sequence, the command can like:
ns-train dn-splatter --data mushroom_sequence
--pipeline.model.use-depth-loss True
--pipeline.model.depth-lambda 0.2
--pipeline.model.use-normal-loss True
--pipeline.model.use-normal-tv-loss True
--pipeline.model.normal-supervision depth
mushroom --load-normals False --mode kinect

你好，@刘海文我可以问一下你对蘑菇数据集使用哪种相机姿势吗？dn splater中的蘑菇数据解析器也支持kinect序列，命令可以像这样：
ns-train dn-splatter --data mushroom_sequence
--pipeline.model.use-depth-loss True
--pipeline.model.depth-lambda 0.2
--pipeline.model.use-normal-loss True
--pipeline.model.use-normal-tv-loss True
--pipeline.model.normal-supervision depth
mushroom --load-normals False --mode kinect

What should I do if I want to add a mask to the mushroom dataset?


RateUs Updated Accoring to Need of Text and Icons

Related Issue
Fixes:  #889
Description
Added responsiveness in text and icons.
In pictures it is not looks so perfect but when you use it it looks perfect with better user experice.
Here I added favicon in Rating Us Page and made Rating Us Page Responsive with use of html , Css and Javascript.

Type of PR

[x] Bug fix

Screenshots / videos (if applicable)

Checklist:

[x] I have made this change from my own.
[x] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

Hey Anurag , I have work upon your need of responsive in text and icons please check it.


UI: Build Dockerfile with Node v14

Description
Node versions >14 throw errors when building and thus do not allow to build the UI.
Pinning to v14 allows us to build and run the UI Docker container.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[x] Minor

Fixes:  #6709

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6712 (SL-JID-2308)

Merging based on @wido shared testing/comments.


fix: use getattribute [APE-1539]

What I did

fixes: #1736
How I did it

How to verify it

Checklist

[ ] All changes are completed
[ ] New test cases have been added
[ ] Documentation has been updated

That sure is some confusing method naming there. LGTM

https://stackoverflow.com/questions/3278077/difference-between-getattr-and-getattribute

That sure is some confusing method naming there. LGTM

You mean the dunders??

That sure is some confusing method naming there. LGTM

You mean the dunders??

Yeah.  Having both __getattr__ and __getattribute__ with fairly different behaviors seems like a bad call.  Maybe it makes sense under the hood but from an API user perspective, it's confusing.


Information re Data Depuplication on ReFS not completely updated?

The article has a Supported section advising that Data Depuplication on ReFS is supported as of Windows Server 2019 but shortly afterwards in the section on Failover Clustering mentions dedup is supported on S2D NTFS volumes, but not volumes with multiple tiers.  There's a link * Data Deduplication on ReFS * that points to the */#unsupported section of the article but there is no mention of anything about deduplication or ReFS in the unsupported portion of the document.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fbca43fe-2459-af79-0ab0-6c598217239f
Version Independent ID: 7e824aea-3911-d256-4002-d62711005efd
Content: Data Deduplication interoperability
Content Source: WindowsServerDocs/storage/data-deduplication/interop.md
Product: windows-server
Technology: storage-deduplication
GitHub Login: @wmgries
Microsoft Alias: wgries

I too find this part of the doc to be unclear, is there a possibility for a clarification on this?

(ref. #1219 #2482 #5211 #429)

(ref. #1219 #2482 #5211 #429)

Thanks for the feedback.  This was an entry that wasn't tidied up last time.  I've made a change that should be visible later today.
#please-close


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4234

Questions ?
Please refer to the Backport tool documentation


Tuple Range Representation

I can't find any documentation about how the Tuple represents a range in the the parameter dictionary. Is it as normal: (start, stop, step)?
I'm confused about the examples in the Jupyter Notebooks, what range does (0.1, 10, 10) represent?

The range is (start, stop, steps) so (0.1, 10, 10) represents 10 steps between 0.1 and 10.  I.e. the last value is not length of a single step, but the number of total steps.


Customer feedback | az network express-route gateway | explicitly address on the Documentation that EXR or VPN GW should not be longer than 64 characters!

Describe the bug
This GitHub issue is being moved from MicrosoftDocs/azure-docs-cli Issue #4564 as it pertains to reference content.  Please route to Azure ExpressRoute service team.

Type of issue
Missing information
Feedback
HI, I recently filed an ICM to address an issue in which a EXR GW on a VHUB was going along with its connections to a Failed Status: https://portal.microsofticm.com/imp/v5/incidents/details/533601064/summary
The product group confirmed that a name larger than that will cause several issues on the back end, and will lead to the GW and its connections to a failed status.
Can you please indicate on the command "az network express-route gateway create" that as NOTE name shouldn't be larger than 64 characters?
Page URL
https://learn.microsoft.com/en-us/cli/azure/network/express-route/gateway?view=azure-cli-latest#az-network-express-route-gateway-create
Content source URL
https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/latest/docs-ref-autogen/network/express-route/gateway.yml
Related command
az network express-route gateway create
Errors
n/a
Issue script & Debug output
n/a
Expected behavior
n/a
Environment Summary
Azure CLI 2.63.0
Additional context
No response

Thank you for opening this issue, we will look into it.


Graceful

Javascript does not break the app. The welcome view and js file have been modified as well as some tests that relied on the old view.

Coverage remained the same at 93.493% when pulling 7ffc8fd57b866b83539132b621ac3843d0e88871 on Graceful into a12bc149bf6a1f8978e28b94e0d5bb4fcaf96234 on master.


remove feature-gate NewVolumeManagerReconstruction

k/k: https://github.com/kubernetes/kubernetes/pull/126775
/hold

This should target the dev-1.32 branch rather than main.

@tengqm updated.

/milestone 1.32

The changes look good to me.
/lgtm
Thanks @carlory for the fix
Merge it once upstream PR merges.

/hold cancel
upstream is merged

/approve


ISIC2018 Loading and Training Configuration for YOLOv7 Lesion Detection Model

Description:
This pull request adds a YOLOv7-based deep learning model for lesion detection in dermoscopic images from the ISIC 2017/2018 dataset, with the following components:
Model Architecture (modules.py)
Implements the LesionDetectionModel class, which loads a YOLOv7 model pre-trained on COCO weights via PyTorch Hub.
Optimizes for efficient and accurate lesion detection on dermoscopic images.
Initializes the model on the designated device (CPU or GPU), with frozen backbone layers to focus training on lesion-specific learning.
Data Processing (dataset.py)
ISICDataset class handles loading and preprocessing of dermoscopic images and annotations.
Data augmentation, including random horizontal and vertical flips and color jitter, is applied to improve robustness.
Each image is resized to 640x640, normalized with ImageNet stats, and converted to a tensor for model compatibility.
Training and Validation Script (training.py)
Initializes the model, loads training and validation data, and executes the training loop.
Uses binary cross-entropy with logits (BCEWithLogitsLoss) as the loss function and the Adam optimizer with a learning rate scheduler.
The model is trained for 10 epochs with validation checks each epoch, though training has not successfully completed due to time limitations.
Prediction Script (predict.py)
Loads the trained YOLOv7 model for inference on new images.
Processes each test image with resizing, normalization, and model inference.
Visualizes predictions by overlaying bounding boxes for detected lesions on the original images.
Documentation (README.md)
Provides instructions for setting up the environment, running training and prediction scripts, and adjusting configurations as needed.
Specifies model dependencies and their versions to ensure compatibility.
Thank you
Pritika Guglani
47604364

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
No design and implementation. Forward has a no_grad() to this model would not train.-2
Spacing and comments.
No Header blocks. -1
Recognition Problem
OK solution to problem. No evidence of model training. Data loaders seem inconsistent. -4
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage.
Module present.
Commenting present.
No Data leakage found.
Difficulty : Normal. Yolo  (Normal Difficulty)-5
Commit Log
Good Meaningful commit messages.
Good Progressive commits.
Documentation
Readme :Acceptable. Image links broken. No train / val graphs. -2
Model/technical explanation :Acceptable. -2
Description and Comments :Good.
Markdown used and PDF submitted. PDF submitted.
Pull Request
Pull Request has problems: Late submission.-2
No Feedback required.
Request Description is good.
TOTAL-18

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness.
Subject to approval from Shakes

Hey @gayanku this is a duplicate of #191 (-17)

Approved extension and duplicate of #191 +2


Fix: #61 Make this lib support es6 module

@dankogai pls check this pr

test/es6.js is causing travis to fail. I WILL NOT MERGE TILL TRAVIS IS HAPPY.

@dankogai It's ready for you to merge

Thank you!

(-:


@conda-forge-admin, please re-render

@conda-forge-admin, please re-render.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/libtheora-feedstock#6.


Adds migration to set isDeprecated to true for pem-key

https://github.com/Shippable/pm/issues/9060

verified on rc


Fix/tabs docs

Motivation and Context

We forgot to merge design documentation, so I've brought it as a several PR to avoid dealing with conflicts.
How has this been tested?

Manually.
Types of changes

[ ] Bug fix (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).
[x] Nice improve.

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly or it's not required.
[ ] Unit tests are not broken.
[ ] I have added changelog note to corresponding CHANGELOG.md file with planned publish date.
[ ] I have added new unit tests on added of fixed functionality.

@msereniti Michael, please check if everything is okay in the examples for tabs 🙏

@ilyabrower Ilya, when you have time, please check if everything is okay in the examples for tabs 🙏


np.int deprecated in bookworm

np.int is deprecated in bookworm which uses a newer version of numpy.
we must use np.int32 or np.int64 in specFunctions.py
The following fixes it in my install:
127,128c127,128
<               window_size = np.abs(np.int32(window_size))
<               order = np.abs(np.int32(order))
---
>               window_size = np.abs(np.int(window_size))
>               order = np.abs(np.int(order))

Now fixed as np.int32 (all it does is handle window size)
Cheers!


Scoped CSS not generating correctly

Version
3.0.0
Reproduction link
Repo: https://github.com/rymate1234/weird-vue-3-scoped-css-bug
Deployed: https://weird-vue-3-scoped-css-bug.vercel.app/
Steps to reproduce

Run yarn serve
Open the website
Observe the two elements using a custom Button component

What is expected?
As per the scoped css on the custom button component, both the link and the button should have the CSS background: crimson; applied to them
What is actually happening?
This CSS is not applied to the element

This appears to have something to do with how the element gets assigned the selectors needed for scoped CSS to work.
On the deployed instance of the above repo, the button elements in the DOM get assigned the data attribute data-v-5b68cbbc-s, however in the generated CSS the selector targets .button[data-v-5b68cbbc] so something is causing Vue to generate mismatching selectors.

The new behavior is that, by default, scoped styles are not applied to the slots. You have to opt-in.

@leopiccionia I'm not trying to apply styles to a slot though, just the element that contains the slotted item?
Or am I misunderstanding?

Sorry, I mixed it.
::v-deep can do the trick, then?
Notice in the RFC I previously linked that ::v-deep syntax was changed since Vue 2.x.

I'm not targeting a child component, I'm targeting the element with class button that's inside the Button component

Finally ran your repro.
The selectors are indeed not compiling correctly -- so ::v-slotted(.button) actually makes the button crimson, but perhaps shouldn't.
It may be related to using a render function. Converting it to a non-functional component with a render method doesn't solve it.
Converting the template to HTML, though, works correctly.
<component class="button" :is="href ? 'a' : 'button'">
    <slot/>
</component>

This is because lost scope id information when create VNode using render function, so runtime will set slotted attribute data-v-13515282-s, but button class has been compiled like follows:
.button[data-v-13515282] {     background: crimson; }.
Like @leopiccionia said, adding ::v-slotted(.button) or using template can make it work.
Using withScopeId can also work. I create a sample code(https://codesandbox.io/s/render-lost-scope-id-8l8i2?file=/src/components/Button.vue), use stateful component but not functional component in order to add scope id.
Although it works, __scopeId is internal property and I don’t know if this is officially recommended

Hi, any fix for this bug ?

I ran into a kind of similar problem, I posted my question in Vue Forum here.
Is my approach is wrong or it is a bug?

This issue has been fixed in the latest version


of members per role showing 0

Describe the bug
When using the @ glyph roles command, all the roles show 0 for the number of members with that role
To Reproduce
@ Glyph roles
Expected behavior
A number indicating the amount of users using that role
Screenshots
not applicable
Glyph Version
Glyph-Kotlin-v158
Additional context
Hello there

hey what's up

Sorry, it seems I accidentally mentioned you. I didn't know that was a thing.
I was just writing the command that is used. :)

@Z3PHR4X no worries, these unintentional mentions are still infrequent enough that I find them hilarious and not annoying :)


Make fallback optional

As title, the fallback should be optional/nullable.
This is useful for cases when the user can't or doesn't want to provide a window component. The result should just be FileDialogResult.Error in this case.

If you don't want the fallback you can construct NfdFileDialog directly.
val nfd = NfdFileDialog()

I guess this should also be added to the README.md :+1:


Update silencer-lib_2.13.6 to 1.7.7

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.ghik", artifactId = "silencer-lib_2.13.6" } ]

labels: library-update, early-semver-patch, semver-spec-patch

Superseded by #302.