EVA-1209 Create JSON file from ClinVar 2018-02

recreate the JAXB binding classes to be able to parse the clinvar XML

Coverage remained the same at 74.635% when pulling 6f035d474245a73afbb7c91136b2d9c426a53e1d on feature/update-xml-parser-2018-02 into 5e5f3e463628ba788db6c6d7bafdd9c0c81d179d on master.


This is still an issue

https://github.com/dotnet/core/issues/3221

Can you please provide details?

It would be good to see fresh logs and a more detailed issue title including OS next time. I assume you're getting this:
  File has unexpected size (72463 != 72485). Mirror sync in progress? [IP: <IP_ADDRESS> 443]

I'm unable to reproduce the problem on Ubuntu 18.04 or 19.04 currently.
We're in the middle of a release, and unfortunately this is expected to temporarily show up after we publish new packages.

I assume you're getting this:
  File has unexpected size (72463 != 72485). Mirror sync in progress? [IP: <IP_ADDRESS> 443]

Yes that's correct.

We're in the middle of a release, and unfortunately this is expected to temporarily show up after we publish new packages.

Thanks very much for the explanation, this makes sense if it has to replicate across servers etc. I wonder whether it's worth adding a footnote to the Linux installation instructions page to highlight that this might happen at times and is expected?
I had waited an hour or so and retried (after doing an apt-get update and refreshing microsoft* files within the sources.list folder) before logging the issue. I can't confirm this is resolved as not in front of work PC any more, however will close if it's expected that this breaks sometimes.

I wonder whether it's worth adding a footnote to the Linux installation instructions page to highlight that this might happen at times and is expected?

I like this, filed https://github.com/dotnet/docs/issues/16670. Thanks. 😄


Ml add more aggregate stats

This adds sub-totals for code and tests stats to the output.

I missed this (for 3 years!)
Want to still merge this in?
@etagwerker seems to be using this. Want to test it out?

@magneland Cool, I just tested this in a sample application and it looks good. 👍
@bleonard I think this would be a good addition to the project.

@magneland Thanks for your contribution! @bleonard just gave me access to merge this. 😄


Fuzzy search populate data is in a different structure to strapi rest query data

Hi,
There is inconsistencies in your data response vs strapi rest api data response which is creating an anti-pattern.
When populating on fuzzy search we get back results like this:
[
  {
    "id": 4,
    "name": "YurMart",
    "banner": { // populated field
      "id": 79,
      "url": "https://ik.imagekit.io/cbs/ticketland/dev/crowd.jpg_FCCgK-CSL.png",
      "type": "image"
    }
  }
]

But in strapi rest api these populated results are nested in "data" and "attributes" properties:
[
  {
    "id": 4,
    "name": "YurMart",
    "banner": { // rest api populated field
      "data": {
        "id": 79,
        "attributes": {
          "url": "https://ik.imagekit.io/cbs/ticketland/dev/crowd.jpg_FCCgK-CSL.png",
          "type": "image"
        }
      }
    }
  }
]

I tried to ditch the initial strapi rest api query for fuzzy search just to return some initial data for page load but you cannot use fuzzy search without passing a search query.
What do you recommend?

I can also see in your docs that graphql fuzzy search returns the data correctly. So it is just the rest portion of it.

@gregg-cbs Thanks for bringing this up. I'll see that this gets included in the next major version bump 🙂


Warn when the input sql contains CURRENT_DATE

SSIA

Codecov Report
Attention: Patch coverage is 90.19608% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 65.69%. Comparing base (90220e6) to head (19dace7).

Files
Patch %
Lines

src/main.rs
90.19%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
+ Coverage   60.30%   65.69%   +5.38%
==========================================
  Files           1        1
  Lines         194      239      +45
==========================================
+ Hits          117      157      +40
- Misses         77       82       +5

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


About the Understanding of the Paper

How to distinguish between mismatched backgrounds and unknown objects when penalizing matched Mahalanobis distances?

@orrzohar That is to say, I want to know how the module for objectness determination distinguishes unknown objects from the background?

Hi @Rzx520,
I only penalize Mahalanobis distance of matched query embeddings, i.e., embeddings that were matched to a GT object via the Hungarian matching. I recently uploaded a video explaining how PROB works. Hopefully, it will be helpful as well.
Does that answer your question?
Best,
Orr


transaction error - lite wallet on PC,..  how to fix? thanks

transaction error - lite wallet on PC,..  how to fix? thanks

I got similar error

yes i see that!
i wonder what it is?
lets wait for someone from pirate to answer us!
On Tue, May 4, 2021 at 4:13 PM Slyshot247 @.***> wrote:

[image: Pirate wallet]
https://user-images.githubusercontent.com/36980892/117017082-f0de6a00-acb8-11eb-8e5f-b8ad95b4293e.PNG
I got similar error
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PirateNetwork/PirateWallet-Lite/issues/53#issuecomment-831974979,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT6GYDUWOVEEIOOIS2FN43DTL76J3ANCNFSM44CWG53A
.

new error:

I am having the same issue with my wallet, I hope we get answers soon & glad I am not the only one.

message from twitter , Our Lite Wallet servers had another huge surge of demand and are now being upgraded again.  Piratenvlag
Please bear with us as we are making these changes and rest assured that your funds are safe.
The changes will affect the lite wallet for desktop and the mobile wallet.

ahh great thanks!
we wait then
ARRR!
On Tue, May 4, 2021 at 6:10 PM BARTEKES13 @.***> wrote:

message from twitter , Our Lite Wallet servers had another huge surge of
demand and are now being upgraded again. Piratenvlag
Please bear with us as we are making these changes and rest assured that
your funds are safe.
The changes will affect the lite wallet for desktop and the mobile wallet.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PirateNetwork/PirateWallet-Lite/issues/53#issuecomment-832062642,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT6GYDW4RBFSYSHBZWKYP7DTMAL7PANCNFSM44CWG53A
.


Add addresses in Mobile, AL

Status:

[ ] Research sources and propose best source (and it's data license)
[ ] Review proposed source
[ ] Approve source based on review comments
[ ] Add to OpenAddresses

NOTE: Proposed source notes should include basic column mapping to OpenAddress data model.

tons of data here: http://maps.cityofmobile.org/GIS/gisdata_download.aspx but no parcels or addresses.  Emailed<EMAIL_ADDRESS>11/7/2016

http://maps.cityofmobile.org/arcgis/rest/services/AddressPnt_MS/MapServer/0


Rewrite for remoting, objects and parallel processing

Issue:
Original script while doing its job didn't scale well
Reason:
Needed this to scale out to hundreds of Exchange Servers
Fix:
Rewrite as functions with remoting and parallel processing capabilities

Thank you! This is a huge change, so we'll need to do some testing before merging this.

Example output from this version:

This branch seems to contain the original blog commands without the performance improvements from Test-Hafnium. For example:

...

Versus the current Test-Hafnium:

...

The current Test-Hafnium takes 6 seconds. The rewrite takes over 2 minutes. We need to bring over all the performance work.

Example output from latest revision. Note we now simply pipe the servers to the script:


[Feature] Add multiple price sources to collateral oracle

Overview
We want to add to the collateral oracle 2 additional price sources:

Fixed price: When registering the collateral the owner specifies a fixed rate.
Band oracle: Query the price directly from band protocol's oracle.

Because now we have collaterals that have completely different logic when obtaining the price, we need to substitute the current generic query_request for a source_type:
pub enum SourceType {
    TerraOracle {
        terra_oracle_query: Binary,
    },
    BandOracle {
        band_oracle_query: Binary,
    },
    FixedPrice {
        price: Decimal,
    },
    Terraswap {
        terraswap_query: Binary,
    }
}

Querying price from Band Oracle
Band oracle has the peculiarity of returning the price as a Uint128 instead of a Decimal (Band oracle doc) The Uint128 contains the price multiplied by 1e18.
Parsing the Uint128 can be tricky because the conversion operation, would result in a uint128 overflow. If instead we use 256 variant, it still could have the possibility of overflowing.
Therefore, in this implementation I opted to parse it as a string to safely move the decimal point.

Added the requested changes. Now collateral oracle returns last_updated parameter. The price age is checked by mint contract.


extrace faces crashes for some images:

Traceback (most recent call last):
  File "/home/gwydion/SSC/missinformation/AMMICO/ammico/testing_all.py", line 18, in <module>
    ).analyse_image()
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/ammico/faces.py", line 139, in analyse_image
    return self.facial_expression_analysis()
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/ammico/faces.py", line 187, in facial_expression_analysis
    faces = RetinaFace.extract_faces(self.subdict["filename"])
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/retinaface/RetinaFace.py", line 190, in extract_faces
    img = get_image(img_path)
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/retinaface/RetinaFace.py", line 53, in get_image
    if len(img.shape) != 3 or np.prod(img.shape) == 0:
AttributeError: 'NoneType' object has no attribute 'shape'

The same image also crashes ObjectDetector
Traceback (most recent call last):
  File "/home/gwydion/SSC/missinformation/AMMICO/ammico/testing_all.py", line 34, in <module>
    ).analyse_image()
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/ammico/objects.py", line 48, in analyse_image
    self.subdict = ObjectDetector.od_client.analyse_image(self.subdict)
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/ammico/objects.py", line 24, in analyse_image
    return self.detector.analyse_image(subdict)
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/ammico/objects_cvlib.py", line 73, in analyse_image
    objects = self.analyse_image_from_file(subdict["filename"])
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/ammico/objects_cvlib.py", line 64, in analyse_image_from_file
    objects = self.detect_objects_cvlib(image_path)
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/ammico/objects_cvlib.py", line 54, in detect_objects_cvlib
    _, label, _ = cv.detect_common_objects(img)
  File "/home/gwydion/anaconda3/envs/test_ammico/lib/python3.10/site-packages/cvlib/object_detection.py", line 77, in detect_common_objects
    Height, Width = image.shape[:2]
AttributeError: 'NoneType' object has no attribute 'shape'

This also happens for some other pictures.

The problem with this image is that it is a gif and not a jpg. Apparently cv2.imread(), which is used in the underlying libraries to open the image, cannot read gifs (due to license issues?).


docs: improve JSDoc comments throughout the project

Add detailed TSDoc comments to various files in the project to improve documentation and code understanding.

For more details, open the Copilot Workspace session.

This PR was almost entirely self-made by the AI


Fixes typo IM Used response reason phrase typo

uses uppercase for "used", changed to "Used"

Ref: https://github.com/zendframework/zend-diactoros/pull/208

@samsonasik is there any way to test these status codes from an RFC list or such?

@Ocramius not sure..., probably use ReflectionProperty ?

@samsonasik yeah, that part is indeed trivial, but we need some sort of authoritative list to compare against, or else we'll keep having these failures, especially if the spec is extended

@Ocramius https://tools.ietf.org/html/rfc3229#section-10.4.1

Yes, and is there no parseable version somewhere? Going paragraph per
paragraph seems silly
On 7 Feb 2017 7:39 p.m., "Sina Sharifzade"<EMAIL_ADDRESS>wrote:

@Ocramius https://github.com/Ocramius https://tools.ietf.org/html/
rfc3229#section-10.4.1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-diactoros/pull/230#issuecomment-278098823,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakJLXai0YBiNiCtMbL9uZ6YzYXCC-ks5raLpqgaJpZM4L5WXG
.

@Ocramius I have linked the section with the URL, section 10.4.1

Yes, and we'd need that for every paragraph 😅
On 7 Feb 2017 19:56, "Sina Sharifzade"<EMAIL_ADDRESS>wrote:

@Ocramius https://github.com/Ocramius I have linked the section with
the URL, section 10.4.1
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-diactoros/pull/230#issuecomment-278104017,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakJ7LjreUrSdnHEELDKd8XzUgQUoEks5raL5CgaJpZM4L5WXG
.

@Ocramius
https://en.wikipedia.org/wiki/List_of_HTTP_status_codes
https://github.com/elegantweb/framework/blob/master/src/Elegant/Http/Message/Response.php#L35-L94

Must be an authoritative source of truth, not another implementation.
Anyway, I'm nitpicking: if this isn't feasible or practical, then this is
ready as-is
On 7 Feb 2017 20:22, "Sina Sharifzade"<EMAIL_ADDRESS>wrote:

@Ocramius https://github.com/Ocramius
https://en.wikipedia.org/wiki/List_of_HTTP_status_codes
https://github.com/elegantweb/framework/blob/master/src/
Elegant/Http/Message/Response.php#L35-L94
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-diactoros/pull/230#issuecomment-278111537,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakNhBUCQv9wpHBiTTWpzGKd0vOphXks5raMRlgaJpZM4L5WXG
.

@Ocramius Have you ever seen the WikiPedia page? implementation is just for making your job easier ;)
anyway you need also change status code 505 and 207.

Eh, Wikipedia is not the IETF, although it's probably stable enough
On 7 Feb 2017 20:28, "Sina Sharifzade"<EMAIL_ADDRESS>wrote:

@Ocramius https://github.com/Ocramius Have you ever seen the WikiPedia
page? implementation is just for making your job easier ;)
anyway you need also change status code 505 and 207.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-diactoros/pull/230#issuecomment-278113145,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakD5ciaSFjOK2rnbkBw_bn6Uqolscks5raMXBgaJpZM4L5WXG
.

@Ocramius Yes, but you can find RFC numbers for each standard status code.

@Ocramius Accurate, updated (e.g. 408, 413, 414, 416, 505) and ease to parse (xml).
http://www.iana.org/assignments/http-status-codes/http-status-codes.xhtml
http://www.iana.org/assignments/http-status-codes/http-status-codes.xml
http://www.iana.org/assignments/http-status-codes/http-status-codes.txt
However, is not perfect, some codes are missing (maybe unofficial or Easter egg - 418)

Thanks, @samsonasik


Added Tournaments Tab to Navigation Bar

This PR is used to address #207 and adds a "Tournaments" tab to the navigation bar if a user is logged into ChiGame.

Here is a summary of what everyone did for this simple PR:

Because this task was relatively small, Riyush Thakur and I met together to work on this together.
I wrote up the code that initially modified base.html and Riyush realized that I had forgotten to implement the user authentication portion that I had mentioned in the PR description. So he went ahead and modified my code.
From there, I created the PR and had Zihao Chen review it.

Issue Score: Excellent
Comments:
Good work! Minor change, but provides much greater functionality.


Sdk3

Some new devices added.

duration feature added to smart irrigator

Thanks @rdl987. I'll have a look at it asap. 👍

duration bug fixed

Hi Rafael, sorry for having this on hold for so long. Some features have been implemented since before and rest is going into next update of the app. Thanks for the help and please do not let my slow actions stop you from contributing :)


Intersection Performance

The current intersection code looks like this (simplified)
fn advance(&mut self) -> DocId {
    let (left, right) = (&mut self.left, &mut self.right);
    let mut candidate = left.advance();

    loop {
        let right_doc = right.seek(candidate);
        candidate = left.seek(right_doc);
        if candidate == right_doc {
            return candidate;
        }
    }
}

left is the doc_set with the smallest size_hint in the group. It is advanced first then we seek on right.
seek is not only going to the passed candidate, but searches for the next hit in that docset.
The problem with this approach is that we have Docset types with vastly different cost for seek which is not reflected here.
E.g. fast field range queries may do a full scan, if there is no next hit. phrase queries load the positions to check if a we have a hit.
Ideally we would use the "cheapest" docset as the driver for the intersection, and all others just check if they have a hit for the passed candidate.
Some docsets, like a precomputed bitset are excellent drivers, but some, like fast field based range, queries aren't.
Cheapness is typically related to number of docs in the docset only for same type of docsets. Between different docsets, we may have something like cost category (can be covered by e.g. using the high bits in u64).
To reflect that a cost based metric would make sense and an api that doesn't advance past target.
Proposal: Seek Exact + Cost Based
pub trait DocSet: Send {
    ...
    /// Returns a best-effort hint of the
    /// length of the docset.
    fn size_hint(&self) -> u32;

    /// Returns a best-effort hint of the
    /// cost to drive the docset.
    fn cost(&self) -> u64{
        self.size_hint()
    }

    /// Seeks to the target if necessary and checks if the target is an exact match.
    ///
    /// Some implementations may choose to advance past the target if beneficial for performance.
    /// The return value is `true` if the target is in the docset.
    fn seek_exact(&mut self, target: DocId) -> bool {
        let current_doc = self.doc();
        if current_doc < target {
            self.seek(target);
        }
        self.doc() == target
    }
   ...
}

Related: https://github.com/quickwit-oss/tantivy/issues/2266

There is also the two phase approach of Lucene.

Yes, I had a look at that. TwoPhaseIterator::approximation returns a DocIdSetIterator where
The returned DocIdSetIterator is a superset of the matching documents, and each match
needs to be confirmed with matches() in order to know whether it matches or not.

This should work good for PhraseQueries, and it seems this was designed with them in mind, since they have kind of two phases (match on the docset + match on the positions).
But for the FastFieldRangeQuery this would still forward the docset and produce the same issue as we have now.
The change and handling of the API would also probably have more overhead.


Pre commit clang

@SimonRit This forces users to run utilities/SetupForDevelopment.sh to be allowed to commit.
KWStyle and ClangFormat are downloaded at build time and run on pre-commit.
Clangformat fixes are applied but must be reviewed with git-mergetool so it requires users to setup their git-mergetool. A warning about how to do so is thrown when trying to commit.

Sounds perfect!


Add credential rotation guide to toc

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
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 change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Deployed the change. I see it here https://developer.adobe.com/app-builder/docs/guides/credential-rotation/ but I don't see it in the TOC/sidebar?
I'm thinking since you put it under the Deployment TOC item, the folder structure should be reflect that, it doesn't.


Show hidden items in DirectorySelectors

Discussed in ppy/osu#15759, resolves ppy/osu#18573.
This change includes hidden items in DirectorySelector dialogues and provides a way to reduce their opacity to distinguish them from non-hidden items.
I elected to make opacity reduction a defaulted constructor argument for DirectorySelectorDirectory to avoid making any breaking changes, but this could be changed to a property to force implementations to state their opacity preference explicitly.
I've only tested this on Linux with dotfiles, would appreciate someone checking it on macOS and Windows (though I can't immediately think of anything that would be different).

Visual test for the component hard-crashes for me:
System.UnauthorizedAccessException: Access to the path '/sys/kernel/debug/tracing' is denied.
 ---> System.IO.IOException: Permission denied
   --- End of inner exception stack trace ---
   at System.IO.FileStatus.ThrowOnCacheInitializationError(ReadOnlySpan`1 path)
   at System.IO.FileStatus.GetAttributes(ReadOnlySpan`1 path, ReadOnlySpan`1 fileName)
   at System.IO.FileSystemInfo.get_Attributes()
   at osu.Framework.Graphics.UserInterface.DirectorySelectorDirectory..ctor(DirectoryInfo directory, String displayName) in /home/dachb/Dokumenty/opensource/osu-framework/osu.Framework/Graphics/UserInterface/DirectorySelectorDirectory.cs:line 26

It's a bit weird because /sys/kernel/debug/tracing is apparently a "drive" to .NET, but this shouldn't bring down the game regardless.

To save everyone another grueling round of review I've pushed 7646eb6 which fixes the implementation of the hidden state of files, as the previous implementation was grossly incorrect - not only was the old implementation placed in a constructor that nothing used, it was also checking the wrong file.
That said, I think this is fine now, but I'm not merging without a second approval.

I've applied further changes mainly to tweak the display and placement of the button, now looks:
https://user-images.githubusercontent.com/22781491/174421289-b5e55e52-c826-4901-bc96-4eacedeb1663.mp4
Unsure if the size of the button makes sense, but looks fine for a basic implementation.

Seems okay. The basic toggle button should really have its state visible but I don't think anyone in their right mind would use that basic design so maybe it's not a huge deal for now.

https://github.com/ppy/osu-framework/commit/c1ea24e654c0e89e17f5f8216962a454d7ed1b5d made hiding hidden items the default behavior, this issue should probably be reopened until a toggle is implemented osu-side.


More Mod Logos

The idea here is to change the mod icon depending on where you are at. I have the default one made by Gedd, one for bookmarked songs (I was told that for this one, I'd have it be the default, but load the default mod logo directly after loading in. This would fake the effect of a different image being used in bookmarks/favorites.), one for the song player, and one for the changelog (potentially more for each version in the changelog?)
Midbox's default logo.

Bookmarked song logo.

Song player logo.

Changelog logo.

I haven't really confirmed these as the official logos (other than the default of which I may still make edits to) so they could be expected to change in the future.

So far, I've got the changelog logo in. No real way to test this, but I'm pretty sure I got it.
Also, something I've noticed is how I'll probably need more icons since there are four other files here with Jummbox's logo on it.


#1 UI Improvement: Add Toggle to Show/Hide Menu

@yashchaudhari008 Let me know if you don't like the UI because I had made some changes to it. If you like it then please merge my PR.
Thanks.

Sorry for requesting so many changes, but I want to keep project easy to understand and simple. Hope you will understand.

@yashchaudhari008 whether this looks good to you?

@yashchaudhari008 I had tried to make the changes but I don't know how to make a hide class and hide the data, so I had not made that change. Please merge my PR.
Thanks.

@yashchaudhari008 I had tried to make the changes but I don't know how to make a hide class and hide the data, so I had not made that change. Please merge my PR. Thanks.

No issues, I will refactor the code
Thanks for contributing.


Types not exposed

Apologies in advance if I'm simply missing something here.
It would be very convenient if types for this project were exposed. For example, in my project I have a custom execute method which first checks if the cluster is ready before executing the task. It accepts strictly a string as the data parameter and a task function as a second parameter. I had to define a type TaskFunction for this second parameter since TaskFunction is not exported by the module. This would be pretty helpful I believe :)

Okay I see now that I can import it from the Cluster.d.ts file but it would be nice if types were also exported from index.


Update download instructions and sumarize them in a central place

This PR updates the download page (e.g. it includer instructions for installing via AUR), and it reduces redundancy by bundling all download instructions on the download page. Consequently, the scope of the tools sections in the handbook changes, and they should now only provide usage instructions. With this change, they look really empty... and we should make an effort to fill those pages in subsequent PR.
I also removed the Troubleshooting section for Python, as I believe this is outdated and most of it, should be fixed by generating launch scripts in bin.

I think this makes sense. We should go into more detail about using the tools on those other pages; I agree that just padding those with redundant installation instructions isn't very helpful...

Yes, I hadn't updated the CLI instructions yet. The PR is now ready for review (please see the updated PR description). I also updated the VS Code section as requested by Edward and made some additional changes to math the style with the other sections.

Hmm, looks like my normal spell checker doesn't like .tsx files. I run the changes through an external checker and pushed several fixes.

@edwardalee: can we go ahead with merging this, or is there still anything you'd like to see changed?


Google Sheets warnings is overwhelming when there are many fields with underscores

If you have a long form with underscores in the column, pyform generates a warning. Because the warning lists all the columns with the problem, can overwhelm the other warnings we get.

Just had an idea. Google Sheets submissions require a Google url in the submission_url of the settings sheet. We should only throw the warning when we have a Google submission_url includes docs.google.com/spreadsheets/d/.
I use that specific string because that's what we use in https://github.com/opendatakit/collect/blob/master/collect_app/src/main/java/org/odk/collect/android/utilities/UrlUtils.java#L30


fix: LSDV-4533: Enable crossorigin requests for audio v3

PR fulfills these requirements

[X] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[X] Docs have been added/updated (for bug fixes/features)
[X] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[X] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[X] Frontend

Requires #1166 to be merged first.
Describe the reason for change
Audio v3 performance enhancements branch had issues loading sources using CORS.
What does this fix?
Allows audio source url to load if using CORS.
What is the new behavior?
Fixes CORS errors when using audio v3 performance enhancements branch.
What is the current behavior?
Currently on the audio performance enhancements branch, there are sometimes issues loading certain urls with CORS enabled on the url.
What libraries were added/updated?
N/A
Does this change affect performance?
N/A
Does this change affect security?
N/A
What alternative approaches were there?
N/A
What feature flags were used to cover this change?
ff_front_dev_2715_audio_3_280722_short
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[X] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Currently I am not able to reproduce this with any of the url's to be able to produce a test case. Will follow up if there is a way to generate a CORS error'd url that would otherwise work with the correct audio setting of crossOrigin.
Which logical domain(s) does this change affect?
Audio v3

No longer required


Ranked Talents not stacking properly

Describe the bug
When I purchase a the 2nd rank in a ranked talent it looks like this:

When I pushcase a 3rd
it adds a new row instead of stacking on top of the other
Also other talents are acting weird as you can see here

and yes Know Somebody is marked as ranked.

I also made a completely new tree and talents and the result was the talents didn't stack.
To Reproduce
Not exactly sure. You could make the trader tree and purchase the talents the same way. The talents and specialization I pulled from a compendium if that matters.
Expected behavior
The talents with ranks should stack without any weird error.
Foundry Version: v12.330
System Version v1.902
Hosted on Forge No

I did a few tests:

New character, specialization (advisor) and talents (Know Somebody) dragged directly from compendium after an oggdude import, ranked talent counts as ranked and stacks
Edit talent directly in compendium and change nothing else, ranked talent counts as ranked and stacks. Talent text updated in talent list on character sheet, but not in specialization tree on character sheet, on in specialization in compendium
Drag updated talent from compendium on specialization in character sheet, ranked talent counts as unranked and stop stacking. Inspecting the talent still show "ranked" as checked.
Import updated talent, drag imported updated talent on specialization in character sheet, ranked talent counts as ranked and stacks.

Fixed by commit from @obrenckle, will be released in 1.903. Thanks for the report and fix.


Rabbitmq AutoCluster using AWS autoscale group backend is unable to connect to other nodes

When autocluster plugin interrogates aws autoscale group, it retrieves the value of private dns field something like ip-192-168-x-xx.  The hostname of box is hostA for example.  Then it tries to connect to ip-192-168-x-xx which won't resolve.  Running manually rabbitmqctl join_cluster ip-192-168-x-xx fails.  However, running manually rabbitmqctl join_cluster hostA it connects.  We don't seem to be able to change ip-192-168-x-xx as that is something that amazon sets internally.
Is it possible to interrogate another field from the autoscale group so that it returns the actual hostname of box instead of amazon given invalid thing.  All of these cluster nodes are on an internal private subnet.

I don't think so, see what the AWS backend does.
RabbitMQ clustering requires hostname resolution for all peers on each node, with this plugin or without.

We're experiencing this issue as well - we set our hostnames to be easily identifiable, and they differ from the private_dns hostname which EC2 sets - which means that the hosts aren't picked up by the plugin - is there any way to use the IP address instead?

Anyone get this to work?  I run rabbitmq in containers and the nodes appear as something like rabbit@3635fea3764b.
I thought the whole point of the plugin is that it figures all this connection info out based on the autoscaling group.
I have the IAM policy set, the three nodes in three servers in elastic beanstalk work, but they dont see each other to cluster.  I have a shared erlang.cookie as well.
I've set the autoclustering log level to debug, but the plugin seems happy although it only sees itself

Also having this issue. Tried looking up the default hostname through the AWS provided DNS server at x.x.x.2 and it returned nxdomain on the AWS provided DNS name.

Making progress. I have custom DNS servers and, as such, they are not able to register the EC2 server's default names. Once I fix the below, I will retest and see what happens.

In the VPC, ensure that DNS Resolution and DNS hostnames are enabled.
Find your AWS provided DNS server IP. If your VPC is <IP_ADDRESS>/8, the IP will be <IP_ADDRESS>. It is always the 2nd IP of the network you specify.
Ensure your systems are looking up hostname.ec2.local. This will mean setting up the default domain on the linux system to be ec2.local.

Alternatively, you can set up a conditional forwarder on your regular DNS servers for the EC2.local domain to <IP_ADDRESS>.

Ah, looks like docker allows you to set this on the container as well. https://docs.docker.com/engine/userguide/networking/default_network/configure-dns/
--dns=[dns server IP] --dns-search=ec2.local

There is now an option to use private IP addresses in RabbitMQ's fork: https://github.com/rabbitmq/rabbitmq-autocluster/pull/14.

This plugin was forked by the RabbitMQ team and is now part of RabbitMQ. More information can be found @ https://github.com/rabbitmq/rabbitmq-autocluster


Support for .net core

How to use it with WPF .net core application ?

Hi, this example is compiles as .netcore.
https://github.com/XAMLMarkupExtensions/WPFLocalizationExtension/blob/master/tests/HelloWorldWPF/HelloWorldWPF.csproj
I hope that answers your question. If you have an issue just reopen this ticket.


Replace file separator '/' with os.path.join everywhere

The current way the code is structured, it won't work on windows because windows uses \ as a file separator instead of /. To avoid this, throughout the code, / should be replaced with os.path.join(...)

Fixed in better_install PR


Add parens to single-parameter arrow functions

Our linting config will carp about arrow-parens if they're omitted.
foo.then(bar => qux());

should become:
foo.then((bar) => qux());

Will be fixed if this recast PR is accepted: https://github.com/benjamn/recast/pull/288.

The PR to recast is good, but does the ESLint run not fix this?

You'd think, but I don't see any fix text in https://github.com/eslint/eslint/blob/master/lib/rules/arrow-parens.js.  A quick 🎩 with eslint -fix enabled added a newline, but didn't tweak function params.

Fix has been merged into Recast.  I'll update our config.


Support for vim.lsp and vim.diagnostic

Hey 👋🏾
First of all, thank you very much for all the effort and time you put into this project. I really appreciate it and it gave the necessary motivation to start working on NeoVim plugins again.
I'm now in need to make some calls to functions that are globally defined in Lua by NeoVim. Means I want to access vim.lsp, vim.diagnostic, ... Unfortunately I don't see how this should be possible right now. But maybe I'm blind. 🙈
I took a look into your noib3/nvim-compleet project to checkout how you do it there. But it seems like this includes a huge amount of boilerplate code with a bridge etc. pp. completely independent of this library here. I'm not sure if there is any better way to do it, because I honestly have no clue about this topic. But in any case, do you plan to integrate this feature into nvim-oxi too? It looks like you did already all the work for it in the other project, so it would be just a matter of putting it here too and wrap it nicely. Am I wrong?
Looking forward to hear what you think. 🙃

Hi, you're not blind, it's currently not possible to access the vim.lsp or vim.diagnostic tables, but they are on my todo list.
The whole bridge machinery I'm using in nvim-compleet is orthogonal to this problem and won't be necessary here. I guess nvim_oxi::diagnostic could already be implemented without many issues, whereas for nvim_oxi::lsp I'd like to have implemented #38 to expose async functions instead of callbacks like in the lua api (:h lsp-handler).

Thanks for your fast response. 🙏🏾
That sounds like good news. What is the best way to stay updated of such changes? Do you do releases with a changelog or should I subscribe to PRs?
Might be related:
In case I want to access not a global object, but anything else available in NeoVims Lua environment, how would I do that? So for example if I want to access coc.nvim its functions. Is this technically possible too?

What is the best way to stay updated of such changes? Do you do releases with a changelog or should I subscribe to PRs?

The project follows semantic versioning and there's a CHANGELOG.

So for example if I want to access coc.nvim its functions. Is this technically possible too?

Technically speaking there's really no difference between e.g. vim.lsp and any other global Lua object exposed by other plugins.
However the vim.* tables are built in the editor and they provide a stable API, so I'm much more inclined to have first class support for those. Supporting other arbitrary plugins from the community would become unmaintainable pretty quickly. If you really need to interact often with other Lua plugins which may or may not be available at runtime, a general purpose Rust-Lua crate like mlua could be a better option.

Yeah, I wouldn't expect any support for arbitrary plugins. Just an easy way to access them. But maybe I need to learn how mlua works and then I do it like that.

Maybe there could be a way to have  procmacro you can use to define an api of a third party macro easily, I did build some here: https://github.com/ModProg/nvim-rust/blob/6edc2435c11e94699efdd574efd981dace69fc33/src/api.rs#L14 and if you would accept something like this in nvim-oxi, this could be exported and made available to a user that quickly wants to create an interface to another plugin where they just need to provide the necessary globals and functions+parameters.

I'm sorry to abuse this issue, but could someone point me to how I could use nvim-oxi (with or without additional mlua) to "require" other modules of the NeoVim Lua environment from my Rust code? I'm totally fine to write my own interfaces for the plugins I intend to interact with and only these parts I need to. But currently I have no clue how to do this at all, I'm sorry.

The Lua object is a wrapper around the Lua state which is not exposed in nvim-oxi. This means that you can't just call Lua code from nvim-oxi yourself, it has to be exposed by us. Can I ask what's your use case for requireing Lua modules from Rust?

Okay.
My use-case are two. First, I want to interact with language servers and people often complain that they want to use in-built LSP support or CoC or vim-lsp or... So I intend to have a trait for the language client and have some implementations for it. And from there on have some typesafe Rust environment I can develop the core/business-logic of my plugin.
On the other "end" of the plugin I need to visually show some stuff to the user. Therefore I don't want to re-invent the wheel and allow the user to choose their UI plugin like Telescope, vim-fzf, etc. pp. Again, an abstract trait and some implementations of these.
I could write only the core in Rust and the rest in Lua as "glue-code". But I would really love to make it exclusively in Rust. Lua is simply not a serious language for me except for hacking around some smallish things. It is too error prone and fragile in my opinion. I have an almost complete rewrite of one of my plugins, trying to establish proper architecture, clean code, a lot of unit tests and so forth. But in the end it is still all super loose and doesn't feel great. Lua doesn't has many features and this bugs me.
Thanks for your support. 🙏🏾


ConfigProperty injection does not work on resource methods

When a resource method is annotated with ConfigProperty, the injected value is an empty String instead of the configuration value.
Environment Details

Helidon Version: 1.0.2
Helidon MP

Validate that it's not required by the MP Config spec. If not it should be closed.

@tomas-langer Do you mean something like this?
    @GET
    @Produces(MediaType.APPLICATION_JSON)
    @Inject
    public Response list(@ConfigProperty(name="max.results") int maxResults) {}

If so, Jersey does not understand this as you intend, seeing the parameter as an entity instead.

Closing as not a bug. This behavior is correct.


[Fanzy Zones] Open Window on last known zone of active monitor.

ℹ Computer information

PowerToys version: v0.25.0
PowerToy Utility: Run + Fancy Zones
Running PowerToys as Admin: Yes
Windows build number: [run "winver"] 19041.572

This setting only opens on the last known zone even if it is on a different screen.

✔ Expected result
With both settings enabled I am expecting "the last known zone" for that particular screen.
❌ Actual result
it does only open on the last known zone even if it is on a different screen.

@aipungggi
I'm going to do some editing to your post since this should only track the suggestion you made for FZ.

@aipungggi
as stated in the documentation the Move newly created windows to the current active monitor [EXPERIMENTAL]  has the following behavior:

When this option is on, and "Move newly created windows to the last known zone" is off or the application doesn't have a last known zone, it keeps the application on the current active monitor

It means that Move newly created windows to the last known zone has precedence over Move newly created windows to the current active monitor.
Are you asking to reverse the precedence (or add an option to do so)?

@enricogior Well described, I am asking to have an option to choose what functionality has precedence.

@aipungggi
OK thanks.

The suggestion has been added to the tracker #6611


Gauge and Counter allocate alot of memory

Hey, I've noticed that a prometheus Gauge or Counter object needs alot of memory. Is this a normal/expected behaviour?
import prometheus_client
import objsize
labels={}
labels['test']='test'

print("Memory Registry at Start: " +str(objsize.get_deep_size(prometheus_client.REGISTRY)))
print("Memory Labels as dict: "+ str(objsize.get_deep_size(labels)))
gauge = prometheus_client.Gauge("test", "", list(sorted(labels.keys())))
gauge.labels(**labels).set(10)
print("Memory Gauge object: "+ str(objsize.get_deep_size(gauge)))
print("Memory Registry after one Gauge: "+str(objsize.get_deep_size(prometheus_client.REGISTRY)))
gauge2 = prometheus_client.Gauge("test2", "")
gauge2.set(10)
print("Memory Gauge object: "+ str(objsize.get_deep_size(gauge2)))
print("Memory Registry after second Gauge: "+str(objsize.get_deep_size(prometheus_client.REGISTRY)))
counter = prometheus_client.Counter("test3", "", list(sorted(labels.keys())))
counter.labels(**labels).inc(10)
print("Memory counter object: "+str(objsize.get_deep_size(counter)))
print("Memory Registry after Counter: "+str(objsize.get_deep_size(prometheus_client.REGISTRY)))
labels={}
labels['test']='test2'
gauge.labels(**labels).set(10)
print("Memory Gauge second label: "+str(objsize.get_deep_size(gauge)))
print("Memory Registry end: "+str(objsize.get_deep_size(prometheus_client.REGISTRY)))

results in this memory usage:
Memory Registry at Start: 3988
Memory Labels as dict: 285
Memory Gauge object: 2418
Memory Registry after one Gauge: 6725
Memory Gauge object: 1051
Memory Registry after second Gauge: 7723
Memory counter object: 2108
Memory Registry after Counter: 10027
Memory Gauge second label: 3376
Memory Registry end: 10931

In this Blog Post https://www.robustperception.io/memory-usage-of-prometheus-client-libraries/ ( I know it's old) a counter needed way less memory than in my usecase. I'm using python 3.9, on windows(but i've got the same problem in a docker container) and prometheus-client 0.17.0

Are you having issues where metrics are causing too much memory usage compared to the rest of your service? "A lot" is rather subjective as to if it is actually too much memory or note. There have been a lot of changes since 2017, but even so if I run your code against v0.1.0 I get
Memory Registry at Start: 2272
Memory Labels as dict: 237
Memory Gauge object: 795
Memory Registry after one Gauge: 3101
Memory Gauge object: 192
Memory Registry after second Gauge: 3435
Memory counter object: 851
Memory Registry after Counter: 4321
Memory Gauge second label: 1089
Memory Registry end: 4561

vs
Memory Registry at Start: 3364
Memory Labels as dict: 237
Memory Gauge object: 1186
Memory Registry after one Gauge: 4781
Memory Gauge object: 571
Memory Registry after second Gauge: 5299
Memory counter object: 972
Memory Registry after Counter: 6467
Memory Gauge second label: 1664
Memory Registry end: 6891

today.

I've just noticed it on a small local machine that in my use case the memory consumption was much highger than expected. But do you know why your memory consumption differs from mine with the same prometheus_client version?

Probably python version. In my last test I used 3.11, with 3.9 I get:
Memory Registry at Start: 3994
Memory Labels as dict: 285
Memory Gauge object: 2418
Memory Registry after one Gauge: 6731
Memory Gauge object: 1051
Memory Registry after second Gauge: 7729
Memory counter object: 2108
Memory Registry after Counter: 10033
Memory Gauge second label: 3376
Memory Registry end: 10937

which is much closer to yours.


Bump the version from aiohttp 3.9.1 to 3.9.2 & cryptography 41.0.7  to 42.0.4

Updated the Version from aiohttp 3.9.1 to 3.9.2 & cryptography 41.0.7 to 42.0.4

Done.


apa7-ish:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: Template that (mostly) complies with APA7 Style.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Thank you :)


Azure Firewall and Application Gateway

Hello, can I configure Azure Firewall in following scenario ? Azure FW -> Azure AppGw -> backend
I found this discussion
https://social.msdn.microsoft.com/Forums/en-US/b1e653d8-818a-4903-a89f-df35c34911c9/recommended-architecture-for-app-gateway-waf-azure-firewall
I didn't found any definition how to setup Firewall rules for management rules.

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

ID: 88466307-81b8-1e5b-7f2c-191d8e818071
Version Independent ID: 08ad49f3-6a57-b925-794a-37215e985aba
Content: Frequently asked questions about Azure Application Gateway
Content Source: articles/application-gateway/application-gateway-faq.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

@jjindrich , You just need to create a DNAT rule in the Firewall to forward the packet to the Application Gateway.
You don't need to allow any management rules in this case. Can you elaborate your architecture a bit more with a diagram ?

If I good understand your answer correctly, there is no need to setup rules for 65200-65535 ? https://docs.microsoft.com/en-us/azure/application-gateway/application-gateway-faq#how-do-i-use-application-gateway-v2-with-only-private-frontend-ip-address

If you are using DNAT rule in Azure firewall to forward traffic to Application gateway, then you don't need to manage the management rules.
If you are applying UDR on the Application gateway subnet then the management plane will not work and UDR is not supported in Application gateway V2.

Please you could this notice on website ?

I am not able to understand your query. can you elaborate a bit more?

My question was if is possible to add this statement about Azure Firewall in FAQ section

@vhorne, It seems like a doc request for the Application Gateway and the Azure firewall configuration scenarios.
@jjindrich , I have notified the author to take a look and have it mentioned in a doc.
Thank you for your feedback.


Issue with the command donet on cmd

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: windows
I simply can't proceed with the first step. I receive the message: 'dotnet' is not recognized as an internal or external command, operable program or batch file. I'm running it on a Windows 10 O.S

.NET Core installer will setup new paths during installation; if you had a command prompt open before installing, you might not have an up to date PATH envvar. Try restarting the command prompt in this case to refresh it.


Stamp $(AssemblyName) ($(TargetFramework)) into builds

That way folks can right click on the assembly and know if they are looking at the bait or the switch.
See https://github.com/xamarin/Essentials/blob/b85fdb2b1623e458037146e1ab4a4f33ed2a8978/Xamarin.Essentials/Xamarin.Essentials.csproj#L15 for instructions how to do it.
I recommend adding it to our Directory.props instead of to each csproj that way it's done once and automatically implemented for current and future csprojs.

This was fixed by #1604


Create tests to check that Vocabulary exports are non-null

Original report on BitBucket by John Shepherdson (GitHub: john-shepherdson).

#489 is a motivation for this

See https://github.com/cessda/cessda.cvs.two/pull/779


I am trying to automate flutter app using flutter finder. I am getting unknown locator for each and every flutter element.

https://stackoverflow.com/questions/78695289/i-am-trying-to-automate-a-flutter-app-using-flutter-finder-but-i-am-getting-un
Please check here for code.

i will create new issue with code


IPS-1240 50Percent 5Minutes lambda canary deployment strategy added

Proposed changes
What changed
Custom deployment configuration will be added to all envs - 50Percent5Minutes.
This means when a lambda deployment occurs, 50% of traffic will go to the old version of lambda and 50% will go to the new version of the lambda.
Why did it change
This is a better strategy as it means the test traffic passing through a non prod build is equally split between two versions and is a better test.
Issue tracking

IPS-1240

Other considerations

closed as we will add 50% 5 mins config using identity common infra


Edit View for Nested Form not showing.....

Hi there!
Im trying to edit a nested form that was created using cocoon. I can Create it but can't even render a Edit View.
Hitting this problem when reading my _credit_cards_fields:
undefined local variable or method `f' for #<#<Class:0x00007faf535b82b8>:0x00007faf535c19f8>

What I understood so far is that it's not able to find the child...
class DailyIncome < ApplicationRecord
  belongs_to :user
  has_many :credit_cards, dependent: :destroy
  accepts_nested_attributes_for :credit_cards, reject_if: :all_blank, allow_destroy: true
end

class CreditCard < ApplicationRecord
  belongs_to :user
  belongs_to :daily_income
end

my form that i render for my new DailyIncome

  <%= simple_form_for @daily_income do |f| %>
    <%= f.input :cash, label: 'Dinheiro' %>
    <%= f.simple_fields_for :credit_cards do |credit| %>
      <%= render 'credit_card_fields', f => credit %>
    <% end %>

      <%= link_to_add_association 'Adicionar Cartão', f, :credit_cards %>

    <%= f.submit "Salvar", class: "btn btn-primary btn-block" %>
  <% end %>

_credit_card_fields

  <%= f.hidden_field :user_id, value: current_user.id  %>
  <%= f.input :tipo, collection: ['Crédito', 'Débito'], label: false, as: :radio_buttons, wrapper: :horizontal_collection_inline %>
  <%= f.input :value, label: 'Valor', placeholder: 'R$' %>
  <%= f.input :parcels, label: 'Parcelas', placeholder: 'Parcelas' %>
  <%= link_to_remove_association 'Apagar', f %>

and finally my DailyIncome controller
class DailyIncomesController < ApplicationController
  def index
    @daily_incomes = DailyIncome.where(user_id: current_user)
  end

  def new
    @daily_income = DailyIncome.new
  end

  def create
    @daily_income = DailyIncome.new(daily_income_params)
    @daily_income.user = current_user
    if @daily_income.save
      redirect_to daily_incomes_path
    else
      render :new
    end
  end

  def edit
    @daily_income = DailyIncome.find(params[:id])
    @credit_card_field = @daily_income.credit_cards
  end

  def update
    @daily_income = DailyIncome.find(params[:id])
  end

  private

  def daily_income_params
    params.require(:daily_income).permit(:cash,
                                         credit_cards_attributes: [:user_id, :id, :type, :value, :parcels, :_destroy])
  end
end

Thank you so so much in advance!

Got it!
Rookie mistake.....
<%= f.simple_fields_for :credit_cards do |f| %>
      <%= render 'credit_card_fields', f: f %>


[Question] should it update neo4j graph?

I'm getting graph in instagraph but in neo4j database is empty,
should it be like that ? I though it will also update neo4j

There was a bug in the handling of neo4j, so we fixed it yesterday.
Please try with the latest code.


Reencounter controller?

Given the encounter limit, is it feasible to have Dragonite ask Golbat for a list of Pokemon that should be reencountered to increase efficiency? (I feel like this should be quick since all the Pokemon is in memory.) If so, what should the API be?

It’s probably not feasible to ask but it would be feasible to change the ruleset.
Currently re-encounter is on weather change, or Pokemon id change (plus your extra ditto rule)

(Actually I think the rule is when it goes from boosted to non boosted rather than any weather change - but it would certainly be good to codify it to exactly useful re-encounters)

Can you give an example of a category of encounters that are currently repeated that could be avoided?
It may be easier to exemplify so we can see if the dragonite ruleset can reasonably be expanded in a worthwhile way

Guess it's not worth the trouble.


Monte_Carlo_Euler_Scheme.ipynb

There is some problem on the reported time for GPU vs CPU on multiple dimensions 100.
I am using a single GPU and it is slower compared to my CPU multithreading. I am wondering why... The rest of the notebooks work as expected.
I am using a GeForce 1080 not a TESLA v100 GPU, but intuitively i would expect to have a similar speedup...
Any ideas why I get this behaviour ?
Regards,
pb

Hi Panagiotis,
Could you please check if this is the case for float32 (I assume you are running for the default float64 dtype)?  Geforce 1080 performance peaks for float32:
https://www.microway.com/knowledge-center-articles/comparison-of-nvidia-geforce-gpus-and-nvidia-tesla-gpus/
Could you please report the running times as well?
Thanks,
Cyril

Using Tensorflow 1.15.0
Running on CPU with 12 cores.
#@title Sampling on a CPU. Note TensorFlow does automatic multithreading.
dtype = np.float64
Time (seconds) to price a European Call Option on a CPU:  0.7360169887542725
dtype = np.float32
Time (seconds) to price a European Call Option on a CPU:  0.5771586894989014
Time (seconds) to price a European Call Option using QuantLib:  20.81744885444641
Sampling from a 100-dimensional process. GPU vs CPU comparison
Time (seconds) to sample 100000 paths on a CPU:  39.573535680770874
dtype = np.float64
Time (seconds) to sample 100000 paths on a GPU:<PHONE_NUMBER>890167 (????? TOO MUCH, WHY???)
dtype = np.float32
Time (seconds) to sample 100000 paths on a GPU:  0.9838593006134033

@PBRAOS
I think these results are very much consistent with the statements from the article I've shared: GeForce 1080 double precision performance is poor (they state that the performance is 32 times worse for double precision compared to single). This factor of 32 is part part of the card desing: https://www.guru3d.com/articles-pages/nvidia-geforce-gtx-1080-review,3.html


Fix typos

Just fixed to typos in one of the latest PRs.

Thank you, please fix this typos also in spec https://github.com/ruby-grape/grape-swagger/blob/master/spec/swagger_v2/errors_spec.rb#L72

Done.

👍


Accordion component support

Added support for Form elements in Accordion components.  (This is my first time submitting a pull request. Any feedback or suggestions are appreciated.)

Thanks! The changes have been published, you can install the latest npm version (2.1.1) and use it 😄


ws error pull swarm

Hello, my name is Ilya Shvyryalkin. I am a developer at DCLabs (Dao Casino). When developing a messaging using ipfs-pubsub-room, we came across the following problem: I declared that I want to connect, and it's logical that if one falls off, then I need to connect to the next one, but it does not happen and the error falls out. The study of this problem led me to this module - libp2p. Fixed this issue by removing it from the pool. A few addresses that throws an error and restarts the start method. I also write the index and the address of the wrong server in the error so that it can be processed later on the client side. Tests drove out, everything works and our problem solved it. It would be very cool if you accepted these changes or wrote how you can fix this problem more elegantly.

Hi @Harzu! I'm interested in better understanding your use case to better understand the needed solution. Currently the switch will check all of the addresses and only listen with the transports that support those addresses. It sounds like you are encountering an issue where on listening, a given address can't be listened on.
This should indicate that either the address is bad, or the transport cannot listen on that address if it is valid. To me, this would suggest a need to fix the configuration of the node to not include the transport that can't listen on that nodes setup. Is updating the configuration not an option for your use case?

Closing this due to inactivity, please feel free to open a new issue or PR if you're still having issues.


Search appears outdated

Hi there!
We're seeing the search is outdated again, as per #230
e.g.
https://npms.io/search?q=%40springernature%2Fglobal-context is at 3.0.0, 2019-07-31
https://www.npmjs.com/package/@springernature/global-context should be 4.0.1 as of 3 days ago
Thanks for having a look.

Yeah a few packages seem to be outdated...


Cannot bind parameter 'End'. Cannot convert value "12/31/9999 23:59:59" to type "System.DateTime"

Hi - Win2012 running Exchange 2013
Run for the first time on a local server, and script ends at the following section and cannot continue:
NIC settings per active adapter:
        Interface Description: HPE Ethernet 1Gb 4-port 331i Adapter [HPE Ethernet 1Gb 4-port 331i Adapter]
Display-ResultsToScreen : Cannot bind parameter 'End'. Cannot convert value "12/31/9999 23:59:59" to type "System.DateTime".
Error: "String was not recognized as a valid DateTime."
At E:\tosh\HealthChecker\HealthChecker.ps1:5685 char:5
+     Display-ResultsToScreen $healthObject
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Display-ResultsToScreen], ParameterBindingException
    + FullyQualifiedErrorId : CannotConvertArgumentNoMessage,Display-ResultsToScreen

Hi @tyusaf
Please run the script with -Verbose parameter and set the output to<EMAIL_ADDRESS>Thank you!

Thanks - I think I've found the issue
My $adapter.DriverDate is returning as "12/31/9999 23:59:59"
On line 4197:
if($adapter.DriverDate -ne $null -and (New-TimeSpan -Start $date -End $adapter.DriverDate).Days -lt [int]-365)
so this is throwing the error - it should be caught by the following line, but isn't.
elseif($adapter.DriverDate -eq $null -or $adapter.DriverDate -eq [DateTime]::MaxValue)
I'll send the verbose output now.

@lusassl-msft since you are already working on this, i have assigned this to you.

The updated version I have now works - thanks Lukas!

Thank you for reporting this @tyusaf . We'll address this issue with our next update.


Any way to decouple the window characteristics of running the receive loop from the actual receiving?

Sorry for the cryptic title, I had a hard time coming up with a better one.
So I've currently integrated tic into my application, and I'm having a small issue.  Practically speaking, the built-in HTTP server is fine but I was trying to see if I could generate a more tailored output by grabbing the Meters<T> data directly.
To do this, I have a dedicated thread that basically receives control messages (like tic::Receiver does!) and processes them before running the receiver once via run_once.  What I didn't anticipate is that the receiver would continue looping inside run_once until window time had elapsed.
I was curious if you had ever had thoughts about decoupling the HTTP server and expanding the control plane, so to speak, so that the receiver could be interacted with while it's currently "running."
Tokio, which uses mio, has provisions for a custom polling lane for things aren't baked-in mio EventedFd's, which illustrates the concept of out-of-band operations nicely, I think.
Not really a problem so to speak but I'd be curious on your thoughts.  This would remove the HTTP portion to let people build richer integrations while still cooperating effectively with tic's inherent event loop model.

Thinking more about this, it would pave the way for adding plugins/extensions on top of tic, where you could rewrite the HTTP portion as a separate crate, or maybe write it with gRPC! Or a plugin that spoke natively to some sort of metrics aggregation system... or in my case, I'd just use it to talk to tic directly so I could interoperate with it how I wanted. :)
Again, I'd be willing to write a PR for this (including an HTTP subcrate to replace that functionality) but was curious to hear your thoughts!

Actually, thinking about this even more.... maybe it would be as simple as a new type that can tap into the existing control plane channel (control_rx/control_tx, just clone the sender so it can push in commands) and new commands for the operations we might care about, like, "give me a snapshot of the meters".
I think, speed wise, this should be the same if not faster than the current HTTP code because we'd be tapped into the event loop rather than just waiting for our "should we service HTTP yet?" timeout in the inner loop.

@tobz - this would definitely be a smart design improvement. I've been working on a new library for metrics at work. My goal is to OSS it at some point. This is one of the design improvements in the new library - definitely makes sense to decouple the metrics aggregation from exposition/reporting.
Since I don't have a concrete timeline on releasing that, I'm definitely open to PRs to improve tic in the meantime. Happy to review and merge a PR once we're both happy with it. Thanks!

Sounds good!  I have a prototype knocked up that is at least compiling so I'll be able to PR that soon and we can discuss it more.

Gonna close this in favor of continuing any discussion in the PR itself: #69


add ability to specify allowed commands to run

Description
Opening a system up to run any command sent over the API could be dangerous. One option that users may appreciate is a way to whitelist individual commands that are allowed with all others being rejected.
Suggested Actions
Add a configuration setting to let users specify a whitelist of commands that should be allowed to be executed through the API.
Additional

I do like this idea. Opening up the entire system could be pretty dangerous. This could also allow the use of aliases for commands.
This will require:

[ ] Storage solution for commands (electron settings solution or a DB solution)
[ ] GUI for configuring commands (Settings)
[ ] Refactor of command endpoint to handle aliases instead of raw commands

although i don't know if you need a GUI.  The config could be as simple as
a text file with the list of allowed commands one per line... or in JSON or
something. Just trying to think of things that might make an initial
version a little easier to get going.
On Fri, Jun 4, 2021 at 5:23 AM Aidan Timson @.***>
wrote:

I do like this idea. Opening up the entire system could be pretty
dangerous in the wrong hands. This could also allow the use of aliases for
commands.
This will require:

Storage solution for commands (electron settings solution or a DB
solution)
GUI for configuring commands (Settings)
Refactor of command endpoint to handle aliases instead of raw
commands

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/timmo001/system-bridge/issues/353#issuecomment-854588414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACYUCRRA2MO7SR7CH6CJXTTRCSTVANCNFSM457M3EKA
.


Release new version on rubygems

The last release to rubygems was almost two years ago (2.7.3 on November 14, 2013).  In the time since then there has been 90 commits that address the rubyforge shutdown, fix bugs, and add new features (I'm particularly keen on 58cfaeb).  There has been a couple version tags pushed to github, but they were not pushed to rubygems.
Please consider releasing a new version.

+1 @leejarvis

Sorry I've been super hectic moving house lately. Myself or @knu can hopefully do this some time this week

@leejarvis I'm not listed as member of the owners of the gem, so please add me if you don't have the time.  Also, is it OK to leave the breakage with JRuby (and Rubinius), or should we fix that before rolling out a new release?

@knu Thanks, I've added you as an owner (sorry, I thought you were!). I think we should look into the JRuby (and rbx) breakage before rolling out, hopefully it's not a time consuming problem

@leejarvis Thanks!  I'll look into the breakage hopefully tomorrow and prepare a release if the problem can be solved.

+1

Based on the rubygem page, it looks like 2.7.5 was released in 2016.
I suppose this means this issue can be closed?

Thanks, closing this!


Newest version breaks with UMAP

Installing the new version and running UMAP, first attempt runs ok, all subsequent ones crash (likely due to some memory leak). Downgrading to the previous version solves the problem.

That seems not good ... I'll try to check this when I get some time.

I just ran into the same issue. It appears to be due to numba caching in rp_trees.py. Removing cache=True from make_dense_tree()/make_sparse_tree() seems to fix it.

Thanks @jakobhansen-blai I'll try to get a new release out tomorrow to fix this.


SAMPLE - process http_proxy is consuming 99% CPU

accessScopes: []
deduplicationKey: e4b8e02a46ceb99fa8be550d0c980382
displayId: eunp-k7dl
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=e4b8e02a46ceb99fa8be550d0c980382&eventid=7f728a40-d559-11ea-87c1-12e36993b0cb
firstOccurrence: '2020-08-03T07:18:06.156Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 7f728a40-d559-11ea-87c1-12e36993b0cb
lastOccurrence: '2020-08-03T07:18:06.156Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
name: SAMPLE - iplab01srv01
type: Application
sender:
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source
severity: 4
severity10: 50
summary: SAMPLE - process http_proxy is consuming 99% CPU
type:
eventType: Response Time
statusOrThreshold: 100%
accessScopes: []
deduplicationKey: 94cecf441637cf7e480f6f2a32d9aa55
displayId: ihmp-pfbu
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=94cecf441637cf7e480f6f2a32d9aa55&eventid=7fd17320-d559-11ea-88e6-8a19a93df6ee
firstOccurrence: '2020-08-03T07:18:07.156Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 7fd17320-d559-11ea-88e6-8a19a93df6ee
lastOccurrence: '2020-08-03T07:18:07.156Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
name: SAMPLE - iplab01srv01
type: Application
sender:
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source
severity: 4
severity10: 50
summary: SAMPLE - north region clients experiencing high response times while connecting
to onlinesales
type:
eventType: Response Time
statusOrThreshold: 135 seconds
accessScopes: []
deduplicationKey: 3baccaef23b01e5168a0404a2f7800a7
displayId: axuy-myp9
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=3baccaef23b01e5168a0404a2f7800a7&eventid=8092c750-d559-11ea-aa0c-031bb29983fd
firstOccurrence: '2020-08-03T07:18:09.156Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 8092c750-d559-11ea-aa0c-031bb29983fd
lastOccurrence: '2020-08-03T07:18:09.156Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
name: SAMPLE - iplab01srv01
type: Application
sender:
application: SAMPLE - Online Sales
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source
severity: 4
severity10: 50
summary: 'SAMPLE - US clients experiencing high response times while connecting
to onlinesales '
type:
eventType: Response Time
statusOrThreshold: '> 60 seconds'
accessScopes: []
deduplicationKey: 8f38abbbd24c5033821115ccd138c811
displayId: twxh-fiii
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=8f38abbbd24c5033821115ccd138c811&eventid=7ff3c830-d559-11ea-8516-b70c61162222
firstOccurrence: '2020-08-03T07:18:07.656Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 7ff3c830-d559-11ea-8516-b70c61162222
lastOccurrence: '2020-08-03T07:18:07.656Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
name: SAMPLE - iplab01srv01
type: Application
sender:
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source
severity: 2
severity10: 30
summary: 'SAMPLE - central region clients experiencing increased response times
while connecting to onlinesales '
type:
eventType: Response Time
statusOrThreshold: '> 30 seconds'
accessScopes: []
deduplicationKey: 1d1d62335c6fdf8162bc510070bed06c
displayId: bxwp-gph9
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=1d1d62335c6fdf8162bc510070bed06c&eventid=802bc820-d559-11ea-bf38-6d75a982987d
firstOccurrence: '2020-08-03T07:18:08.156Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 802bc820-d559-11ea-bf38-6d75a982987d
lastOccurrence: '2020-08-03T07:18:08.156Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
name: SAMPLE - iplab01srv01
type: Application
sender:
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source
severity: 4
severity10: 50
summary: 'SAMPLE - east region clients experiencing high response times while connecting
to onlinesales '
type:
eventType: Response Time
statusOrThreshold: '> 60 seconds'
accessScopes: []
deduplicationKey: 4b7a256b0b8ca47ec8f80139cd20a5d3
displayId: 1skj-ziji
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=4b7a256b0b8ca47ec8f80139cd20a5d3&eventid=7ee3b7c0-d559-11ea-ac24-d6c2a36628e2
firstOccurrence: '2020-08-03T07:18:04.656Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 7ee3b7c0-d559-11ea-ac24-d6c2a36628e2
lastOccurrence: '2020-08-03T07:18:04.656Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
hostname: sample.iplab01srv01
name: SAMPLE - iplab01srv01
sourceid: sample.iplab01srv01
type: Application
sender:
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source
severity: 2
severity10: 30
summary: SAMPLE - Slow response time for application OnlineSales
type:
eventType: Response Time
statusOrThreshold: '>45'
urls:

url: https://www.ibm.com

accessScopes: []
deduplicationKey: 94ca081848de37f244b7a41e2022cdfa
displayId: byx2-htym
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=94ca081848de37f244b7a41e2022cdfa&eventid=7f4e8780-d559-11ea-9158-fd8c761ed60c
firstOccurrence: '2020-08-03T07:18:05.656Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 7f4e8780-d559-11ea-9158-fd8c761ed60c
lastOccurrence: '2020-08-03T07:18:05.656Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
name: SAMPLE - iplab01srv01
type: Application
sender:
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source
severity: 4
severity10: 50
summary: SAMPLE - CPU is thrashing at 99% utilization on HTTPProxy
type:
eventType: CPU Utilisation
statusOrThreshold: 100%
accessScopes: []
deduplicationKey: c092d5d2fe095861449dff8c6f46bc76
displayId: evq8-ctyz
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=c092d5d2fe095861449dff8c6f46bc76&eventid=805d5f70-d559-11ea-9063-795abc72d633
firstOccurrence: '2020-08-03T07:18:08.656Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 805d5f70-d559-11ea-9063-795abc72d633
lastOccurrence: '2020-08-03T07:18:08.656Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
name: SAMPLE - iplab01srv01
type: Application
sender:
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source
severity: 3
severity10: 40
summary: 'SAMPLE - west region clients experiencing increased response times while
connecting to onlinesales '
type:
eventType: Response Time
statusOrThreshold: '> 25 seconds'
accessScopes: []
deduplicationKey: 62388edbf88aa84d8ce5196e1cc35446
displayId: kn95-kkmw
eventState: closed
eventURL: https://icp-console.apps.doffed.os.fyre.ibm.com/cem/api/events/v1?deduplicationKey=62388edbf88aa84d8ce5196e1cc35446&eventid=7f143da0-d559-11ea-ae38-90df792a4316
firstOccurrence: '2020-08-03T07:18:05.156Z'
flapping: false
incidentUuid: 7ee7fd80-d559-11ea-be96-e7a6900c829d
instanceUuid: 7f143da0-d559-11ea-ae38-90df792a4316
lastOccurrence: '2020-08-03T07:18:05.156Z'
maxSeverity10Seen: 10
priority: 5
resource:
application: SAMPLE - Online Sales
name: SAMPLE - Online Sales
type: Application
sender:
displayName: Sample Event Source
name: Online Sales
type: Sample Event Source
severity: 3
severity10: 40
summary: SAMPLE - client response time is above threshold
type:
eventType: Online Sales application alert policy
statusOrThreshold: '> 100 seconds'

sender:
displayName: Sample Event Source
name: Sample Event Source
type: Sample Event Source


Add Utf8->Binary type coercion for comparison

Which issue does this PR close?

Partial fix for #7039.
Rationale for this change
The following casts should be safe:

Utf8 -> Binary | LargeBinary
LargetUtf8 | Binary -> LargeBinary

What changes are included in this PR?

Are these changes tested?

Yes
Are there any user-facing changes?

No

Thanks @jonahgao ❤
I took the liberty of merging up from main to get a fix for CI failures and pushed a small sql level test in sqllogictests

Thank you @alamb  ❤


LcmController.log growth should be configurable

The LcmController.log grows over time and does not have a limit. It should be possible that data older than n days days to be removed from the log. This can be an additional function in the DscLcmController configurations.
The marker in the log file is:
Start
**********************
Windows PowerShell transcript start
Start time:<PHONE_NUMBER>3655

End
Windows PowerShell transcript end
End time:<PHONE_NUMBER>3703

The suggested new file name for the log file is containing the year and week number of year: 2020-50-LcmController.log
Get-Date -UFormat %Y-%V-LcmController.log

Fixed with 'LogHistoryTimeSpan' in #100 and corresponding PR in CommonTasks.


CDN files disappeared

Hi i'm opening this issue to understand why the files
<EMAIL_ADDRESS><EMAIL_ADDRESS>
are disappeared from CDN.
I was using theme for multiple projects and I was forced to update them all.
Can i know the exact date when you removed these files and the reason?
Basically a cdn must keep old versions (in some cases you can make redirect when new versions are released).

Before you open an issue, please check if a similar issue already exists or has been closed before.


Generate proptypes validation based on breakpoints and known flexbox options

[x] Move the current various Flexbox property options from their respective component into a helper function.

[ ]  Generate the prop validation based on these options  and the breakpoints passed in.

[x] Enables dynamic validation based on the amount of breakpoints used.

This feature is now mostly introduced with #62. All functionality is in place however there is no validation and these custom breakpoints are not stripped form the element when generating.

Do you still need help? Just discovered this amazingly neat flexbox grid framework and would like to see how I could help make it fully support a dynamic number of breakpoints.

@billykwok Thanks! Yes any help would be appreciated!
I'd like to separate out common proptype validations into files similar to the utils/helpers. These proptypes should generate based off the theme, so if a user sets an xl breakpoint etc it understands to apply the same proptype validations as the others etc


particles.energy should be read-only or update other energy variables

Because now setitem passes silently but does nothing

Implemented in latest version


fix: Resolve conflicting macro definition

Previously, there were two macros named impl_try_from in scope in mailbox.rs (from import and local definition). While the build correctly resolved both macros, rust-analyzer did not and complained about using the imported macro.
Renaming the local macro resolves this ambiguity and makes rust-analyzer happy.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.923%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9959

Relevant Lines:
11075

💛  - Coveralls

Huch... I would have guessed this doesn't compile...

Yeah, would have thought as well. Learning something new every day. 🧑🎓

Thanks for the fix!


non-fastboot UX -- options for eager loading of known needed assets / default languages?

tl;dr: I think for the best non-fastboot UX, the primrary languages someone is working with should be eagerly loaded, so there is no async behavior to wait on. This is solved by specifying defaultLanguages and defaultThemes (if applicable) in the environment.js under the ember-shiki ENV config.
This eagerness helps prevents a sort of "flash of unstyled content", which can be jarring to folks looking at the exact place where formatting/highlighting/etc are about to take place.
Also big caveat: I'm working with a very atypical app, a REPL.

I've been exploring ember-shiki and the idea of lazy load everything here: https://github.com/NullVoxPopuli/limber/pull/1303
But I have a problem where the loading experience is noticeably worse than what I had with highlight.js (which was effectively synchronous)
Compare by clicking through a few of the tutorial chapters and paying close attention to how the codeblocks are highlighted

https://tutorial.glimdown.com/
https://5b230757.limber-glimmer-tutorial.pages.dev/ (uses live limber.glimdown, rather than also the version in the same PR)

I noticed that there is some config: https://ember-shiki.pages.dev/#configure-(optional)
that maybe sounds like it could solve my problem -- but it's not explicitly stated in the docs? should it be?
    'ember-shiki': {
      defaultLanguages: ['gjs', 'gts', 'css', 'bash', 'js', 'ts', 'markdown'],
      defaultThemes: ['one-dark-pro'],
    },

The config did help my rendering UX be less async.

However, because I'm now using ember-repl in the tutorial to generate a component (which is async), there is a bit of a de-sync now between my "scroll-to-top" behavior and rendering of the next page (which, when I used the modifier, highlighting was instant and I could compile just markdown synchronously) 😅 -- but this is probably a timing a thing I need to work out anyway. But the compilation stuff is very heavy (babel, template-compiler, etc) -- previously I was only using the unified pipeline with remark and triple-curlies -- so it was very fast :upside_down_face:

Some pagespeed insights costs that occurred due to me needing to change how I render prose:
The Tutorial
55 on Mobile -- Today (I still have quite a bit to improve on here) https://pagespeed.web.dev/analysis/https-tutorial-glimdown-com/y2voonddn1?form_factor=mobile
43 on Mobile -- the branch where everything is loaded (babel, ember-repl, etc) https://pagespeed.web.dev/analysis/https-fbfe8dfe-limber-glimmer-tutorial-pages-dev/mgcgkfiwcj?form_factor=mobile
The REPL
63 on Mobile -- babel is deferred until JS is rendered https://pagespeed.web.dev/analysis/https-limber-glimdown-com/37ehzcf3e4?form_factor=mobile
56 on Mobile -- babel is loaded eagerly https://pagespeed.web.dev/analysis/https-845396c8-limber-glimdown-pages-dev/ksll922uk9?form_factor=mobile

Additionally, I noticed (before I set the above config) on the REPL (which is fixed now, due to specifying explicit languages), that if you wait for the preview text to highlight (in the editor area) (before you move the mouse, trackpad, or have any touch/keyboard inputs (at which point the editor would load and replace the preview)), the tab freezes, where it didn't previously :thinking:
I also noticed that for Markdown, it seems shiki loads every language language ever? it takes quite a while!
Is there a way to configure which markdown languages and aliases are loaded (while using async everything?)? maybe this is just something to "be aware of" when not specifying pre-loaded languages?

Currently the functionality offered by CodeBlock is to encapsulate all async behaviour to ease integration, as you can use the component without configuring anything else and it just works.
The API offered by Shiki is async, so if there would be an synchronous way to use CodeBlock it would expect that Shiki is already properly initialized and themes/languages are loaded before it's invoked. Initializing Shiki once before rendering anything could be achieved by e.g. doing the initializing in an application initializer, or in a beforeModel hook on a route.
This is something you could already do:
async beforeModel() {
  await this.shiki.initialize.perform();
}

That would load all predefined languages and themes already. The CodeBlock component would still not render 100% synchronously due to the way it currently uses the modifier which performs an async task. Perhaps here there should be a way to make it render instantly assuming the theme/language is loaded.
Overall Shiki would still be less performant compared to Highlight.js in the sense that it uses vscode-oniguruma for tokenization which is quite heavy. It's more optimized for correctness and features, rather than file size in browser.

Regarding Markdown loading all languages, this is expected behaviour as all those languages are defined as embeddedLanguage in Shiki. One way to solve this would be to use a custom grammar definition as mentioned in https://github.com/IgnaceMaes/ember-shiki/issues/18#issuecomment-1704047598.

Sounds good!
I just thought also that it'd be a good idea to make sure that padding, margin, and font properties (family, size, line height, kerning, etc) all match the theme that's about to be loaded (likely only if not using a custom theme) -- that'd probably help quite a bit with the noticeable content shiftmonce highlighting happens.
Do you think it'd be good to have a tiny section in the docs about this? Or is it something people should just know, and i forgot? Lol

I was actually just working on that! Added CSS variables for padding and updated the docs (although it could use further elaboration 😄): https://github.com/IgnaceMaes/ember-shiki/pull/19#issuecomment-1704259979
Not sure what you mean with adjusting the properties bases on the theme? As the themes don't include these settings, only the color styling.

ah ok, maybe it's more that I needed to make
https://github.com/IgnaceMaes/ember-shiki/blob/main/ember-shiki/src/styles.css#L8
match my current font?
anywho, thanks for the additional docs!

ah ok, maybe it's more that I needed to make https://github.com/IgnaceMaes/ember-shiki/blob/main/ember-shiki/src/styles.css#L8 match my current font?

Yes, that would be the intended way to style the CodeBlock!


#2400 adjust command prompt

I solved issue #2400 by overriding the defaults $PS1 value in .bashrc. Is my solution fine or do I have to tackle the issue in a different way? Thanks for your feedback.

what do you think about dropping the hostname (which is kept generic for privacy reasons) and put the LAN IP address after the username which would help identifying the terminal on networks where multiple nodes are running?

I replaced the hostname with the IP address:

To determine the IP, I use the output from hostname -I.

That's useful I think.
hostname -I was the most reliable way to get the IP which I tested as well in various environment.
If it is thought to be too long could cut it to the last number, but that would probably introduce some confusion.
Like it as it is and nice idea about the branch name @nyxnor !


Fix Black/Flake8 problems [WIP]

Attempt to solve black problems seen on GitHub Actions but not CI.
We used to exclude cookie-cutter template folder from black, but this has obviously been replaced by template/stages which has a few random python files (e.g. jupyterhub_config.py).
These weren't safe to run through black before because they were jinja2 templates. It would be safe now, but I don't think it really makes sense to run black/flake8 on the template folder, so will submit a PR shortly.
Changes:

Exclude qhub/template from flake8 and black in both pre-commit and pyproject.toml

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features to not work as expected)
[ ] Documentation Update
[x] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[ ] Yes
[x] No

In case you checked yes, did you write tests?

[ ] Yes
[x] No

@iameskild Yes, it would be safer than when they were Jinja templates, but at the same time it seems a bit aggressive to set these Python formatting tools loose on large folder trees containing Terraform modules. There happen to be a few random Python files in there, but I would argue they are data files rather than Python files really :)
Let's say Black added a rule to insert __init__.py files into all folders on the assumption that we would only run Black on a Python tree in the first place. We wouldn't really want that for our Terraform modules. Arguably, the templates folder shouldn't be directly under qhub in the first place.
That's my thinking anyway...


Add party-games plugin

Add party games plugin. This plugin uses the Party plugin to allow party members to play games. Current games include rock paper scissors and coin flip.

This plugin is an abuse of our party service that wasn't intended for this kind of usage

Is the idea of the plugin an abuse or the implementation itself? I had a look at some other plugins like the tag ur it that also uses the party service to send messages.

These games seem like they are nearing gamba territory

Is it the rock paper scissors, the coin flip or both? The initial idea of the plugin was to play rock paper scissors to decide who runs the head at olm.

Is the idea of the plugin an abuse or the implementation itself? I had a look at some other plugins like the tag ur it that also uses the party service to send messages.

A bit of both. We're not really interested in supporting plugins which have no ingame behavior to use the party service. This plugin is effectively just a Swing UI which reads chat messages, which doesn't show any overlays or otherwise interface with the game itself.

Thats fair, I might re-work the plugin in the future to not use the party service, ill close the pr for now.


Running Dictionary command says "Mac is required"

Trying using this plugin on babun with cygwin (having gcc and vimproc installed)
Following error is shown.
"Mac is required".

This plugin requires Dictionary.app which is a standard application on macOS.

I see. Thanks.
Closing it 😀
I cannot use it then.


Unable to activate VS Code extension

I'm getting this error in the console when i updated my VS Code to 1.38.1
from 1.37 yesterday and now all my salesforce extensions are no longer working . The error in the console is Cannot activate extension 'Apex Interactive Debugger' because it depends on extension 'salesforce.salesforcedx-vscode-apex', which failed to activate.
Can someone please help? I tried installing and uninstalling multiple times and reverted to older version but still not working. Same error.

I can't tell why salesforce.salesforcedx-vscode-apex failed to activate. Please file the issue against the salesforce extension. Thank you!


winch: Test winch against a subset of the official spec test suite

With the introduction of control flow, Winch is getting to a point in which it would be beneficial to have some sort of run tests rather than only filetests. Run tests make it easier to verify that the introduction of new features or bug fixes won't cause any regressions or bugs. They also make it easier to verify the behavior of the code generated by Winch.
The idea behind this change is to test Winch against a subset of the official spec test suite. Any unsupported tests will be commented out from the spec tests and as more features are added the corresponding tests will be uncommented. This involves a bit of duplication with regards to the spec tests, but the ideal end state is to have Winch be just another strategy in build.rs and have both compilers use ./tests/spec_testsuite instead of using a custom directory for Winch once Winch offers feature parity.
Note that I'm currently collecting feedback on this idea, and for that reason, I've only included the block spec as an example.
We discussed this idea during today's Winch meeting: https://github.com/bytecodealliance/meetings/blob/main/winch/2023/winch-06-20.md
@cfallin and @alexcrichton I'm interested in your thoughts regarding this change.

I realize after writing that all out that you've probably already also considered such an idea, so I could also rephrase much of it as "I'm not sure why a copy of block.wast was added", but if that's well-motivated them seems fine!

I'm not sure why a copy of block.wast was added", but if that's well-motivated them seems fine!

Yeah, the copy of the spec tests is one of the unfortunate bits of this approach. Unfortunately attempting to run all the tests and adding #[ignore] when a non-supported feature is detected will result in most (if not all) of the official spec tests getting ignored, since as far as I can tell, there's no way to ignore spec tests at a more granular level. Even though Winch is shaping up, it doesn't offer support for finished proposals that are already part of the official spec suite, like multi-value for example, which many of the tests in the official spec test suite use. But it's totally possible that I might have missed a better way of avoiding the duplication, and if that's the case, I'm all in to explore alternatives!

One idea perhaps would be to add a test for strategy in the ignore function against "Winch" and blanket return true (ignore everything) and then add a small allow-list for tests there? Or alternative blanket-ignore entire directories/subtrees but blanket-include others, for example tests/misc_testsuite/winch/*.wast could be tested in winch for now?
Otherwise though this is a transient state of affairs while support is shored up in winch so while I personally try to minimize the churn during the transient state I don't think it's bad to have churn and/or copies of files, so again I think whatever's easiest for your development should work best. In the long run everything should be run through both Winch/Cranelift and the "weird cases" will be much more defined.

I finally got a chance to get back to this! I decided to try the approach of ignoring everything except for the winch misc test suite, I believe this will require the least amount of changes once we decide to enable everything by default in the future.
This should be ready for a review @alexcrichton; one thing to note is that I didn't add the if test suite yet, because I found one bug that I'm currently trying to figure out, once I do, I'll push the fix along with the test suite.


Requests do not reliably execute on page load

Hey @chriddyp! So we're having a new issue with dash-renderer==0.11.0 that was not on older versions. I think this is related to https://github.com/plotly/dash/issues/133 and https://github.com/plotly/dash-renderer/pull/22.
We are trying to render something like 40 components on page load. However, only about 20 of them actually even make a request on page load for data.
Logic with 0.8.0

"All" components are dependent on pathname.
pathname is None, we throw a ValueError, and the request to /_dash-update-component returns a 500.
pathname then updates to /my/new/url/10, and requests to /_dash-update-component return the correct data based on the url.

This is good enough and works, although leaves an "Updating" in the browser title.
Logic with 0.11.0

"All" components are dependent on pathname
pathname is None, we throw a ValueError, and the first request to /_dash-update-component returns a 500. We noted that the first request is always the same, and always returns a 500.
pathname then updates to /my/new/url/10, and about 20 requests to /_dash-update-component return the correct data based on the url. We noted that these are always the same requests and always return the correct data.
The remaining ~ 20 requests do not even submit any requests to /_dash-update-component (confirmed by checking server logs to console and through the /_dash-update-component POST requests in the Chrome Developer Tools network panel). We noted that they are the same components every time we load the page.
This does not leave an "Updating" in the browser title, since dash-renderer thinks that all requests completed successfully.
Updating the pathname via inputs on the page functions as expected from this point forward.

Notes
If useful, I can try and make a dummy example of this. It does, however, based on previous issues and the change made for 0.11.0 seem to be related to the event scheduler (and the dummy example may have to be quite large to mimic this issue).
For now, we have to stay on older versions of dash-renderer since the behavior in 0.11.0 is not desirable.

Just checked on 0.9.0 and 0.10.0 and it has the same issues as 0.11.0. 0.8.0 is the last version to "work" for us. Any thoughts on what we might be doing wrong on this?

Hm.. I'm not sure what could be going on here. Could you create a reproducable example? I'll prioritize the fix once we can reliably reproduce.

(even if the reproducible example has like 50 inputs, it will still be helpful)

Looks like it was caused by this logic path: Dropdown (with no default) -> pathname -> other components. Still not sure why the behavior changed between 0.8.0 and 0.9.0, but we're going to implement it differently to avoid this issue.
On another note, I am able to get an unreliable example with anywhere between 20 and 50 inputs, but it only shows up sometimes. I have not been able to make a reliable example. For example, on one server run, the 2nd and 5th components of 30 may not update, and when restarting the server, they may all update. Seems like that may just be more of a server performance issue.
Probably reasonable to just close this.

@mjclawar - I'll close this for now, but if you continue seeing it and can reproduce it, please re-open! Thanks :)


[BACKPORT] - updated Traffic Stats to handle new time format returned by TO

back porting this into 2.0.x
(cherry picked from commit ff12b099dc6213cfbb39dcf6d9dfff0b56e810f5)

merged


The first release v"0.1.0"

@JuliaRegistrator register

@JuliaRegistrator register
CC/ @jounihuo

Registration pull request updated: JuliaRegistries/General/4956
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 05940f0fe5c843c76aa577621c6133a930730aea
git push origin v0.1.0