queryForGeoJSONFeaturesInTable throws an error for some geopackage files

Version: 2.1.0
First of all, thanks for this comprehensive geopackage library!
I recently ran into a problem with some geopackage files. Calling queryForGeoJSONFeaturesInTable results in the following error:
Uncaught (in promise) TypeError: Cannot read property 'x' of null
    at geocentricToGeodetic (geopackage.js:58208)
    at datum_transform (geopackage.js:58393)
    at transform (geopackage.js:58506)
    at transformer (geopackage.js:58545)
    at proj4$1 (geopackage.js:58584)
    at BoundingBox.projectBoundingBox (geopackage.js:1264)
    at FeatureTableIndex.queryWithBoundingBox (geopackage.js:5724)
    at FeatureDao.queryForGeoJSONIndexedFeaturesWithBoundingBox (geopackage.js:13493)
    at module.exports.GeoPackage.queryForGeoJSONFeaturesInTable (geopackage.js:14250)
    at resolver (GeoPackageLoader.js:94)

The issue is, that point ends up being null in geocentricToGeodetic.

The point becomes null during  datum_transform after calling geodeticToGeocentric.

And the reason is because the latitude is not in polar coordinates:

This is the location in my code where the error occurs:
https://github.com/potree/potree/blob/72748db38fd46a98c4d0800a70b1fe2570365152/src/loader/GeoPackageLoader.js#L92
Note that this looks a bit different than the official examples and the GeoPackage Viewer, because the version: 2.1.0 release seems to have a different API (no getFeatureRow() function, for example). What I'd like to accomplish is to load 3D data of all stored features, so that I can create 3D models out of them.
Here is an example that works:
3D Scene: http://mschuetz.potree.org/geopackage/examples/geopackage.html
Geopackage file: http://mschuetz.potree.org/geopackage/examples/morro_bay_shp/gpkg/geopackage.gpkg
And this is the geopackage that causes an error when loading with geopackage-js:
3D Scene: http://mschuetz.potree.org/geopackage/not_working/gpgp.html
Geopackage file: http://mschuetz.potree.org/geopackage/not_working/test2d_potree17_epsg2056.gpkg
Help on how to fix this issue or on how to correctly load 3D geometry data for features would be highly appreciated. Thanks!

I will check into this and get back to you soon.


css3 hyphens

Hi,
very nice work. In case I don't get around to making a pull-request -- this should probably enable css3 hyphens, in order to reduce the ragged-right on columns (and behave more like TeX/LaTeX):
https://css-tricks.com/almanac/properties/h/hyphenate/
I'd also like to see (do) some work on how the design respons to small screens -- the side-bar should collapse some how things like the Note 3. It's not immediately clear what the best way to do this is, though.

I should make clear that while I'm in favor of reducing ragged-right, specifically mimicking TeX/LaTeX behavior is not a project goal.
What are your thoughts on text-align: justify without hyphenate?

Putting a quote from Butterick's Practical Typography here for my own reference, emphasis mine:

Justification is a matter of personal preference... Keep in mind that the justification engine of a word processor or web browser is rudimentary compared to that of a professional page-layout program. So if I’m making a word-processor document or web page, I’ll always left-align the text, because justification can look clunky and coarse.

After some investigation I think browser support for hyphenation is still too spotty for our purposes. I'd still like to hear your thoughts.

There seems to be evidence to show non-justified text improves readability:

https://kaiweber.wordpress.com/2010/05/31/ragged-right-or-justified-alignment/
http://www.adamsdrafting.com/justified-text-versus-ragged-right-text/
https://graphicdesign.stackexchange.com/a/16146
https://encrypted.google.com/search?hl=en&q=ragged right readability

First, note that the quote above uses hyphenation (jus-tification). Second, I haven't had time to look over the papers; but note that I'm talking about columns 20 ems and narrower in particular (with a collapsing side-bar the common mobile use-case isn't quite as pathological). At any rate, "newspaper"-width columns - as that's all that'll fit (physically) on a normal, non-phablet, phone.
Third, justification without hyphenation is generally strictly worse INMHO.
Fourth: not sure about support - do you mean: "chrome team refuse to support it"?
http://caniuse.com/#feat=css-hyphens
At any rate it degrades nicely?
Personally I'd prefer hyphenation (even without justification! - left-alignment looks better with hyphenation). In fact, I might recommend against justified alignment.
See also:
http://alistapart.com/article/the-look-that-says-book (note:written before css3-hyphens)
I have a bookmark of a book published in html somewhere, which uses hyphens and looks dramatically better in ff than chrome; unfortunately I can't find it right now :/
As an aside, http://alistapart.com/article/building-books-with-css3 might also be of some interest for general ideas.

Bah, it just occurred to me that hyphens might not do anything without justify set... which makes degration a bit more tricky.

Note: the quote above (as I discovered on cut-and-paste) actually uses hard-coded soft-hyphens ("rudi-men-tary com-pared") -- so could be seen as an argument for using hyphenation (in some form or other, be that hyphen.js or css3 hyphens).
I did a bit of testing, and css3 hyphens can indeed be used with regular left-alignment -- have a look in Firefox at: https://rawgit.com/e12e/f04740fd143f18ea69ef/raw/2da1b1860e6697eed3c023680874839412f4ba65/hyphen-test.html
The difference is subtle (between non-hypenated and hyphenated left-aligned text) -- but in general I prefer the one with hyphenation. And that does not really give any trouble to chrome users.
Justification+hyphenation generally also looks good and reads well -- but in that case one would probably want to only enable justified alignment for browsers that also support hyphenation. In my testing way back, js-based hyphenation was flaky -- so I'd recommend against trying to use a hyphenation shim.

JavaScript hyphenation is quite reliable and quite fast (if you use Hypher, at least). And even browsers that support CSS hyphenation vary in which languages they support hyphenation for -- so if you all do decide to go with hyphenation, JavaScript hyphenation is still necessary as a fallback.

For the moment, I consider the ragged-right default to be an acceptable choice for Tufte CSS.
Justifying and hyphenating with CSS or JS should be considered a totally valid way to modify Tufte CSS for one's own purposes, but it is not core functionality I'm interested in supporting right now.

good call. :yum:


Add support for M7 device

Added various items as per the commits.
Updated README.md to reflect M7 support.

Would it be possible to add the current version number of the assembly to the project file?
CHANGELOG.md talks about v2.0.16 but this version doesn't seem to be mentioned anywhere in the assembly.

Let me know if I can merge, thanks

Not sure if you are waiting on me to give the go ahead on this?!

Released v2.0.18. Thanks again!


laws dataservice link field should contain full url or provide documentation on how to use to get the file

reproduction steps

open http://online.knesset.gov.il/WsinternetSps/KnessetDataService/LawsData.svc/privatelaw(318)
look at the d:plaw_link field

expected

field should contain full url to download the file
(alternatively - provide documentation about how to use the link to get the file)

actual

field contains just the file name without full url or any details about how to get the file
see

Knesset was notified on May 29, 2016

Still broken:
http://knesset.gov.il/KnessetOdataService/LawsData.svc/privatelaw(318)

less relevant with the new API (opened an issue to review the old bugs and see if they reproduce in the new API - #131)


having issues with configuration

I am having hard time adding the config file.  I always get errors.  Is there an example I can look at.  Thanks

What kind of errors are you getting?
A full homebridge config with just this plugin would look like this.
{
"bridge": {
"name": "MyHomebridge",
"username": "AA:AA:AA:AA:AA:AA",
"port": 33333,
"pin": "123-45-678"
},
"accessories": [
{
"accessory": "iComfortThermostat",
"name": "Thermostat",
"username": "USERNAME",
"password": "PASSWORD",
"gatewaySN": "SERIALNUMBER",
"zoneNumber": 0
}
]
}


Add React Lifecycle diagram to documentation

x-post from the main React repo https://github.com/facebook/react/issues/12633
I'd love to see this or something similar to it included in the documentation page for component lifecycles. I remember @gaearon sharing it and I've found it invaluable for quick reference

@wojtekmaj
Want to experiment with embedding your component into the doc?

My main concern at the moment is how can we handle mobile devices. That's pretty complicated diagram. I have created mobile layout version but I'm not satisfied with it so much.

@wldcordeiro Hey do you know if that's the same diagram referenced here: https://reactjs.org/docs/react-component.html#the-component-lifecycle
?
The link on the docs is broken, if that's the same diagram would be good to rehost it, maybe in this repo, and change the link.

@plonko, I don't think the link is broken. It's actually my server, I keep it, I maintain it, I'll keep it in sync with wojtekmaj/react-lifecycle-methods-diagram repo and will continue to do so in the future.
If you have any issues with the diagram itself, i.e. not displaying correctly in your environment, I suggest you making an issue in the repo mentioned above. I keep track of it and will react (pun not intended) if needed.

This has been added here I think https://reactjs.org/docs/react-component.html#the-component-lifecycle


Support overwriting entrypoint and arguments in argo submit

Currently, arguments must hardwired in the kubernetes manifests. With argo cli, we can dynamically generate a manifest where the user supplies arguments at the command line. Similarly, the entrypoint of a workflow should be able to be overridden.

Commit 3f13f5c adds support for supplying/overriding --parameter and --entrypoint to workflows during submission.


Chart does not show anything when all the data are equal

The chart does not show anything if all the elements in the array is same
Ex:
LightChartView(data: [10, 10, 10, 10, 10],
                               type: .curved,
                               visualType: .filled(color: chartColor, lineWidth: 5),
                               currentValueLineType: .dash(color: .gray, lineWidth: 1, dash: [5]))

I've even tried adding an offset, but the result is still same
LightChartView(data: [10, 10, 10, 10, 10],
                               type: .curved,
                               visualType: .filled(color: chartColor, lineWidth: 5),
                               offset: 0.2,
                               currentValueLineType: .dash(color: .gray, lineWidth: 1, dash: [5]))

Current behavior

The chart does not show anything if all the data are same

Expected behavior

The chart shows a horizontal line if all the data are same

Please fix this.
Thanks so much for this amazing SwiftUI library.

Thanks for raising this issue. Now it should be fixed from version 1.0.1

Thanks for quickly fixing the issue 😊


Add collection_layers to the allowed values configuration

Values are found at https://github.com/mitre-attack/attack-datasources/tree/main/contribution

Also add platforms for data sources


Fix test on macOS

Characters in Добрый день are actually double-width on macOS terminals (confirmed on Terminal.app and iTerm2.app).

It caused test failure and I fixed it by modifying expected value. (I'm not sure updated expected value is actually what you expect, tho...)

It seems that Cyrillic characters are single width on my environment (macOS Mojave). And tests are also passed.
Screenshots are below:

Terminal.app
iTerm2

Hmm... I don't know why there are differences between mine and yours but it may depends the font used in our terminal emulator 🤔 Anyway, I think we need to investigate more to merge this.

Um, it may depend on my configuration. Let me check before merging this.

I could find character width configurations in Profile -> Text. But I don't think these are related to this issue. As you said, it may be difference of macOS version. My macOS is 10.12, a bit older.

When I started development of go-prompt, I use macOS Sierra (version 10.12). So I think it arise from another differences of our environment.

Could you tell me your font configuration? I want to clarify font has nothing to do with this difference.

I use Monaco font at iTerm2. It looks this is English font, So I will check what used to display Cyrillic characters. By the way I found Use Unicode version 9 widths check option. This seems to be strongly relevant with this problem. Could you check this?

I uncheck it and display Cyrillic characters, but it still looks single width 😢

As I showed in above my screenshot, Use Unicode version 9 widths is not checked. Let me try it.

OK, I updated iTerm2 and checked the option. And now go test -v passes :tada:. I think this PR is no longer necessary for me. (Adding some comment may be useful, tho)
Thank you for your cooperation for clarification.

I see. It's better to add some comment.
By the way, this problem of Terminal.app is also fixed?

I'll close this issue after add some comments.

I tried to add comment. Please merge if you want.


Update from upstream: Fix literal \ characters at the end of a string

Source: https://github.com/mitsuhiko/jinja2/commit/e0615edb7590591356384465fec8413ebbeece8c

Thanks for being on top of upstream!


Power BI Service - how to set it up?

hi all,
The Power BI template works fine on Desktop, but when publishing to the Power BI Service, I'm not able to get it to work. On one hand there are messages saying dynamic sources aren't supported, and on the other hand I think something needs to be done on the Power BI Gateway.
Does anyone know if it can be done to use the Power BI Service to refresh the report? And if so, how to do it?
Sorry if this is not the right place to raise this, but I do not know where else to get technical support.
Many thanks
Bart

Hi @aikibart ,
we do not natively support Gateway
this is the troubleshooting document: https://help.safetyculture.com/en-US/000155/

Thanks. It's a pitty it is not supported for Power BI to retrieve data directly from your api's on the Power BI Service. We will take a look at the exporter tool.


Deployment

Deployment
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website using the URL,
THEN I can access my website from any browser

Deployed webpage on July 26, 2023.


Support non-GitHub URLs for projects in devfiles

/kind user-story
/area devfile
User Story
Currently, the --downloadSource flag in odo create <comp>, only supports projects hosted on Github, as odo is using the Go-Github API.
As a user, I want to be able to host my code templates for devfiles in other Git repositories like Gitlab, and then use odo create --downloadSource to pull down those projects
Acceptance Criteria

[ ] It should support any Git repository, regardless of where it is hosted
[ ] It should use the Go-Git API

/kind user-story

@kadel I think we can close this

We have moved to go-git and currently support non github projects #3867


Set duck volume to 0 on iOS

iOS doesn't allow partial volume increments. On the charisma-web, we have overridden this to duck to 0 if the device is iOS. We should move that code to the SDK itself.
WebComicMicrophoneButton.tsx:
if (isIOS()) {
  setMediaVolume(0);
}

Annette tested this on iPhone


FEATURE REQUEST: Alt text for photo posts

Could you add the ability for add "Alternate Text" to photo posts, for the blind and other accessibility issues?
Thanks

thanks for the suggestion. i'll add this feature within a few hours.

this feature has been implemented. here's the code:
from ensta import Host

host = Host(username, password)

upload = host.get_upload_id("Picture.jpg")

host.upload_photo(
    upload,
    caption="Travelling 🌆",
    alt_text="photo of a dog"
)

same goes for uploading reels, just specify the alt_text parameter.
thank you


KAFKA-7698: Kafka Broker fail to start: ProducerFencedException throw…

If ValidationType is None, also skip the check in appendEndTxnMarker (similar to append).
Verified with existing unitest and our daily operation.

@mingaliu Thanks for the patch. I think it makes sense to skip all validation when loading the segment. That said, I'm a bit puzzled how the log could have gotten into that state. I don't suppose you can reproduce the problem or you have a dump of the log segments or anything?

@mingaliu @hachikuji Are there any plans to merge this PR? Our Kafka brokers cannot start due to the exact same issue and I have no idea how to get our Kafka cluster working again. I can provide some start-up logs if it helps

I think we got to the bottom of this issue in https://github.com/apache/kafka/pull/7687. I am going to close this PR and we will follow up there.


stacked borrows: add option for recursive field retagging

@bors retry r+

:pushpin: Commit e0a0056ff9e04b5d22420b8d6435949a949ebe55 has been approved by RalfJung

:broken_heart: Test failed - checks-actions

@bors r+

:hourglass: Testing commit 955f961f8327926477ea906a7f4140d5b9dbfa45 with merge 5974e7d4a973983f449702538468ef1a3e3d9407...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 5974e7d4a973983f449702538468ef1a3e3d9407 to master...


Provide Kotlin friendly extensions

Is your feature request related to a problem? Please describe.
This library should have Kotlin friendly extensions so that modern language features offered by Kotlin (e.g. extensions, lambdas, coroutines, etc.) can be used while interacting with this library.
Describe the solution you'd like
This feature should allow me to write idiomatic Kotlin. For example:

I should be able to use Kotlin function extensions instead of using *Util classes that have static methods. This helps me discover functionality that the library offers.

Before:
Polygon polygon = // some polygon
LatLng latlng = // some latlng
boolean result = PolygonUtil.containsLocation(latlng, polygon.getPoints(), true);

After:
val polygon: Polygon = // some polygon
val latlng: LatLng = // some latlng
val result: Boolean = polygon.contains(latLng)

I should also be able to use named parameters with default values for methods that have several arguments. This helps me understand with code readability as I can see what dependencies I need to pass and which have default values set.

Before:
GeoJsonLayer layer = new GeoJsonLayer(map, geoJsonFile, null, polygonManager, null, groundOverlayManager);

After:
val layer = GeoJsonLayer(
    map = map,
    geoJsonFile = geoJsonFile,
    polygonManager = polygonManager,
    groundOverlayManager = groundOverlayManager
)

etc.
Describe alternatives you've considered
Android KTX in Android Jetpack is a good example of this.
Additional context
This is in active development in the android-maps-utils-ktx branch.

Moved development to a new repo: https://github.com/googlemaps/android-maps-utils-ktx

Kotlin extensions for this library have been released (version 0.1). Refer to https://github.com/googlemaps/android-maps-ktx to learn more.


Display better error when output type is undefined

If an output type isn't specified, Pydantic expects the model to return nothing. When the model returns an output and it's compared to None, an error is logged and a 500 error is returned.
This change explicitly raises an error when the output type is undefined and raises an error message, rather than defaulting to None. The error message provides explanation of how to fix the issue more clearly than the original behaviour.
Before

After

I added from typing import Any to the error message. Thanks!

@allcontributors please add @evilstreak for code and tests


Fixed an issue in the server implementation when using customized models. Closes #184.

Fixed the reported bug #184 presented as "Unable to define the custom model for client and server"
Description

Just made slight changes to the function load_trainer of plato/servers/fedavg.py by instantiating the model
How has this been tested?

I tested the revised code on the reported example examples/base_siamese in the bug #184 and on the previous example examples/fedprox
Types of changes

[x] Bug fix (non-breaking change which fixes an issue) Fixes #
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I reverted the changes on the server and changed the client's implementation. Tested using:
python examples/basic/basic.py -c configs/MNIST/fedavg_lenet5.yml

Reviewed and tested. The customized models was used by the server and clients.


[MM-44031] add system console SMTP link to invitation error

Summary
Add system console SMPT config link to the invitation error.
Ticket Link
https://mattermost.atlassian.net/browse/MM-44031
Related Pull Requests

Has server changes (please link here)

Screenshots

Release Note

NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

Thanks @neallred for a quick review.

@salmanmanekia Can you outline the steps I need to test this?

@salmanmanekia Can you outline the steps I need to test this?

Hi @stevemudie,
In the spinwick test server created for this PR you need to 'Invite a person' as a member and as a guest. The SMTP  setting in the test server has been configured in such a way that you should be seeing an error modal with details. In the details there is a link that once clicked should take the admin user to SMTP config page in admin console – this last thing is what has been done in this PR. I have also created a loom to demo this.

Thanks for the steps @salmanmanekia!
I wasn't encountering the link as expected before but did on this pass.
I just need to run E2E tests now then once that's done I should be able to approve it.

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/250478

What's SMTP?

What's SMTP?

SMTP is an internet communication protocol used for email transmission. In the system console we have an option for configuring it.


messagepack is not supported

Hi guys,
First off, i'm sorry for opening an issue about this, but hopefully it will help people testing stuff out in the future. I've just upgraded to coreapp 2.1 RC1 for everything in a small project i use to test out SignalR every now and again. I've previously had messagepack working fine, with old code. However, updating the new libraries seem to give me the error "Trace: (WebSockets transport) data received: {"error":"The protocol 'messagepack' is not supported."}". This was also throwing a "version" error, until i went and hardcoded that as it did not seem to find/pass the version string. Building the connection with the hub connection builder gives "signalr-client.min.js:15 Error: Connection disconnected with error 'Error: Server returned handshake error: The protocol 'messagepack' is not supported.'.". Both work absolutely fine when in JSON.
Are there any examples of the server configuration in use for messagepack? I can't seem to find anything in startup/options where i can add support for messagepack to signalr, everything defaults down to JSON only.
Thanks!

Are there any examples of the server configuration in use for messagepack?

https://github.com/aspnet/SignalR/blob/dev/samples/SignalRSamples/Startup.cs#L26

Wow. Now i feel silly. Was a reference error on my part, not including the messagepack SignalR project in my main app. Thanks Brennan for the quick reply!


FIX: Correct a few configurations

The coverage was not updated with src/coincurve everywhere (it still worked apparently). I also removed the tests.
I noticed tox -e fmt (that I forget too often) complained about libsecp256k1 and a few times I missed issues
Also, lint picks-up a lonely .py deep within libsecp256k1, they may add more

Usually I wait for the test to finish before pinging you, but this one should be passing, not many changes


OneShotStartupCheckStrategy always returns StartupStatus.NOT_YET_KNOWN

OneShotStartupCheckStrategy in testcontainers checks if finishedAt is not equal DOCKER_TIMESTAMP_ZERO ("0001-01-01T00:00:00Z"). Kubedock always returns DOCKER_TIMESTAMP_ZERO for finished containers.

I have improved the container status api endpoint, can you validate if this solves the issue?


409 API is not yet supported for NFSv3 enabled accounts

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
v10.1.0 and above
Which platform are you using? (ex: Windows, Mac, Linux)
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
What problem was encountered?
Error occured when attempting to copy data from AWS S3 bucket to blob storage account with NFSv3 enabled:
https://docs.microsoft.com/en-us/azure/storage/blobs/network-file-system-protocol-support
How can we reproduce the problem in the simplest way?

create a storage account
enable NFS 3.0 (https://docs.microsoft.com/en-us/azure/storage/blobs/network-file-system-protocol-support)
use azcopy to copy from AWS S3 bucket to blob storage account container.

Have you found a mitigation/solution?
Issue mitigated by copy to another blob storage without NFSv3 enabled.

Hi @SherryGuo1021, thanks for reaching out! That is unfortunately a service limitation. I do believe that it will be solved in the future.


Is it possible to have this work with TLS explicit?

I have tried multiple port numbers but I do not know enough about FTP to know what to change or if this is even supported. Thank you.

Sorry to have to divert the answer but you can find the answer if you look into sftp https://github.com/sergi/jsftp - the underlying transport lib.


long-typed primary key with a value of 0 cannot be accessed depending on the complexity of other long primary keys

I've been playing around with storing different data in a Hollow snapshot, and came across an alarming issue in Hollow 3.4.5 that I don't see reported elsewhere.
Using the model:
@HollowPrimaryKey(fields={"id"})
public class Movie {
    long id;
    String title;
    int releaseYear;
}

I've written some tests using hollow-test that seem to show a bug happening depending on how big the long value I use as a primary key index is.
public class HollowZeroLongDemo {

    @Test
    public void testZeroLongPrimaryKey_withSmallLong() throws IOException {
        MoviePrimaryKeyIndex moviePrimaryKeyIndex = initializePrimaryKeyIndex(5L);

        assertEquals("The Matrix", moviePrimaryKeyIndex.findMatch(5L).getTitle());
        assertEquals("Pulp Fiction", moviePrimaryKeyIndex.findMatch(3L).getTitle());

        //No problem!
        assertEquals("Beasts of No Nation", moviePrimaryKeyIndex.findMatch(0L).getTitle());
    }

    @Test
    public void testZeroLongPrimaryKey_withBigLong() throws IOException {
        MoviePrimaryKeyIndex moviePrimaryKeyIndex = initializePrimaryKeyIndex(9223372034562340851L);

        assertEquals("The Matrix", moviePrimaryKeyIndex.findMatch(9223372034562340851L).getTitle());
        assertEquals("Pulp Fiction", moviePrimaryKeyIndex.findMatch(3L).getTitle());

        //Problem!
        assertEquals("Beasts of No Nation", moviePrimaryKeyIndex.findMatch(0L).getTitle());
    }

    private MoviePrimaryKeyIndex initializePrimaryKeyIndex(long bigOrLittleLongId) throws IOException {
        final HollowWriteStateEngineBuilder hollowWriteStateEngineBuilder = new HollowWriteStateEngineBuilder();
        Arrays.asList(
                new Movie(longOrShortLongId, "The Matrix", 1999),
                new Movie(0L, "Beasts of No Nation", 2015),
                new Movie(3L, "Pulp Fiction", 1994)
        ).forEach(hollowWriteStateEngineBuilder::add);
        HollowWriteStateEngine writeStateEngine = hollowWriteStateEngineBuilder.build();

        TestHollowConsumer consumer = new TestHollowConsumer.Builder()
                .withAnnouncementWatcher(new TestAnnouncementWatcher().setLatestVersion(2L))
                .withBlobRetriever(new TestBlobRetriever())
                .withGeneratedAPIClass(MovieAPI.class)
                .build();

        consumer.addSnapshot(latestVersion, writeStateEngine);
        consumer.triggerRefresh();

        return new MoviePrimaryKeyIndex(consumer);
    }
}

Calling moviePrimaryKeyIndex.findMatch(0L) returns a result in the first test, but a null in the second test.
If I knew for sure why this happened for 0-value ids, and could be certain that only 0-value ids affected, I wouldn't be so concerned, but I haven't comprehended the bitshifting going on well enough to be sure this only will happen for 0-value ids.
I haven't tested this with 4.2 yet, but none of the changes I've looked at seem extremely likely to remedy this bug.

@jkade, thanks for the report and test case to reproduce.
I was able to reproduce with a narrower test in #400. Still investigating the root cause.
It does seem like an interaction between keys of value 0, though I need to dig further before I can confirm that.


Add RAQMS reader to MELODIES MONET

Add RAQMS reader to MONETIO. Update driver and ReadTheDocs in MELODIES MONET to use the new RAQMS reader.

xref: https://github.com/NOAA-CSL/MELODIES-MONET/pull/99 https://github.com/noaa-oar-arl/monetio/pull/85

Am working to finish checking changes as well as update the documentation. I ran into a potential issue regarding plots. Within surfplots for the spatial plots (make_spatial_bias and make_spatial_overlay), the default lat/lon minimum and maximum values for domain_type == all is a CONUS bounding.
@zmoon how should I proceed? Should this be changed, or for any model is a domain_type of all just Conus and something else be used to push to the else statement in the map generation?

@mebruckner that is a good point, probably this should be updated so that "all" is actually all[^a] and we can have a separate "CONUS" option. We can open a separate issue for this.
I think currently if you just don't set domain_type (just delete the line in your config), you should get the behavior I think you are looking for (the else branch).
[^a]: Though some of the obs datasets we use are essentially limited to CONUS.

Thanks- I'd already worked around it in my personal scripts and codebase. It doesn't actually cause any issues in generating plots, I just came across it again while ensuring RAQMS is included in the plot setups. There just might be some user confusion if the obs and model sets include points outside the spatial plotting domain which are then included in the other plots and statistics (which does happen with the current all behavior).
Documentation for the RAQMS addition will be complete before the weekend, and I will push the new commits for approval.

It doesn't actually cause any issues in generating plots,

Except that if there are points outside the CONUS domain you wouldn't see them, yes?
But your other point about the averages and stats seems the more important, I will open issue.

Yes, any points outside the CONUS domain just wouldn't be visible in the spatial plots when the user gives domain_type == all under the current construction due to the choice of minimum and maximum lat/lons for the plot extent.

In running my  test notebook I ran into an issue with the aeronet pairing that arises from the aod field in RAQMS not having a z-dimension. I had bypassed this issue by adding a try/execpt that would bypass the z issue for 2d model fields. This could be a special case sort of situation, so I don't know the to handle it.

hmm, we could add a surface-only option to the reader like the WRF-Chem one has?

@mebruckner I added a surf_only option in a branch here if you want to try it https://github.com/zmoon/monetio/tree/raqms with something like
model:
  RAQMS:
    mod_type: raqms
    mod_kwargs:
      surf_only: true

Zach, thanks for adding this option. It does work.

This is now in the develop branch of MELODIES MONET and complete: https://github.com/NOAA-CSL/MELODIES-MONET/pull/147. I'm closing this issue.


[feat] Adding a Confirmation Pop up Screen before Logout Action

i believe i can add a pop-up Screen to Let the User Confirm his Logout
please Assign it to me Under #swoc24 @Mayuresh-22

yes great idea @Amine2000s, I am assigning this issue to you then. You can start working on it.

@Mayuresh-22  Completed All my assigned issues , waiting for Pull pull request to be merged and the assignments to start working

@Amine2000s, do you have any updates on this issue? If not, I will unassign it.

hey @Mayuresh-22 , i will deliver it under 24hour

Hey @Mayuresh-22 , i am very soory
But i am going throught an emergency conditions, i will need extra 24 hours

@Mayuresh-22  currentlly working on it , just some Css alignmnet and few test and the pull reqeuest will be sent

https://github.com/Mayuresh-22/Minglr/assets/92222814/79329fb8-e215-436c-8214-a5c9651b97b4
i am currently facing some problems to redirect it to the back/logout.php page , i used many ways and it didn't work
(like inside script tags , making a function that do that in the js file )
do you have any suggestion  ? @Mayuresh-22

feed-logout.mp4
i am currently facing some problems to redirect it to the back/logout.php page , i used many ways and it didn't work (like inside script tags , making a function that do that in the js file )
do you have any suggestion ? @Mayuresh-22

I'll analyse your changes and will get back to you in short.

Okay @Amine2000s,
Once the confirmation box popups you can set the click function on the yes confirm button and once it is clicked you can redirect it with the window.location.href = "back/logout.php"; line.

I did it , but it didn't redirect me to anything


Css changes

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Css
Why is this valuable (or not)?
Yes
How do we know it's really free?
Yes
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:
Read our contributing guidelines.
Search for duplicates.
Include author(s) and platform where appropriate.
Put lists in alphabetical order, correct spacing.
Add needed indications (PDF, access notes, under construction).
Used an informative name for this pull request.
Follow-up
Check the status of GitHub Actions and resolve any reported warnings!

Can i work on this issue?

What does this PR do? Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources Description Css
Why is this valuable (or not)? Yes
How do we know it's really free? Yes
For book lists, is it a book? For course lists, is it a course? etc. Checklist: Read our contributing guidelines. Search for duplicates. Include author(s) and platform where appropriate. Put lists in alphabetical order, correct spacing. Add needed indications (PDF, access notes, under construction). Used an informative name for this pull request. Follow-up Check the status of GitHub Actions and resolve any reported warnings!

Please avoid duplicating the pull request template; instead, use it to submit pull requests and open issues separately.

Can i work on this issue?

This isn't an issue. Feel free to contribute by following the guidelines in docs/CONTRIBUTING.md.

okay no issue.


Store and expose links in _meta

Assuming the following API payload which contains a link that is named the same as an embedded collection:
{
    "name": "Book No #3",
    "_links": {
        "self": {
            "href": "/books/3"
        },
        "chapters": {
            "href": "/chapters?book=3"
        }
    },
    "_embedded": {
        "chapters": [
            {
                "_links": {
                    "self": "/chapters/1028"
                }
            },
            {
                "_links": {
                    "self": "/chapters/1029"
                }
            },
        ]
    }
}

Current behavior
I cannot read the chapters links, as this.api.get('/books/3').chapters() is overwritten by the embedded collection.
let book = this.api.get('/books/3')
console.log(book.chapters()._meta.self) // --> undefined

Proposed behavior
Expose all links in _meta object.
let book = this.api.get('/books/3')
console.log(book._meta.links.chapters) // --> '/chapters?book=3'
this.api().post(book._meta.links.chapters, { title: 'Awesome new chapter' } )

This could help to avoid the need for templated links when posting to new objects.
Possible workaround
A workaround possible today is to use a different name for the link, e.g. chaptersLink and then do:
let book = this.api.get('/books/3')
console.log(book.chaptersLink()._meta.self) // --> '/chapters?book=3'
this.api().post(book.chaptersLink()._meta.self, { title: 'Awesome new chapter' }

However, with this I'll trigger an unnecessary API request to chaptersLink().

Reading the hal-json-normalizer docu, this discussion and this discussion, then the current behavior should not return undefined. Maybe rather a bug than an enhancement?

Hm I'll have to take a look at this. Thanks for the report!


load_geojson issues

We're trying to support/implement a use case that depends heavily on vector cube manipulation and some questions related to load_geojson came up (based on 2.0.0-rc.1 spec):

description states "Feature properties are preserved", but the process only preserves properties specified in the properties parameter, which is empty by default, so no properties are preserved by default actually
properties has default value empty array [] but it is unclear what should happen in this case: there is only a geometries dimension with labels, but no "cube data".

Is it valid to have (vector) data cube e.g. with shape (N, 0)  (N labels along "geometries" dimension and zero labels along "properties" dimension)?
Or should we in that case add a dummy label in the "properties"  dimension and give all "pixels" a no-data value, so that the cube at least has a shape (N,1)?

Iterating further on the empty cube  problem: description states "A new dimension with the name properties and type other is created if at least one property is provided.". I wonder if it wouldn't be cleaner to just always add this "properties" dimension, regardless of the number of provided properties. Then e.g.:

empty properties array  [] => "properties" dimension with single dummy label and no-data values
single property in properties array, e.g. ["population"] => "properties" dimension with single label "population". Advantage compared to not adding the dimension is that the property name ("population" in example) is preserved
multiple properties e.g. ["id", "area", "population"] =>  as currently specified

related to empty cube problem: the description of "properties" param states "Only applies for GeoJSON Features and FeatureCollections. ". Does this mean that e.g. a Polygon GeoJSON object always results in such an empty vector data cube?
GeoJSON properties are typically mixed: e.g. strings and int/float. Is it valid in openEO to mix these data types within the same cube (along the "properties" dimension)?

Side note regarding empty cubes: with raster cubes in load_collection we have  NoDataAvailable and TemporalExtentEmpty errors, so "empty cube" seems like an error condition there

A PR based on your findings would be great so that we can discuss specific improvements.
There's room for improvement of 1+2 for sure, making more clear that it's probably no-data values/null that you want to set if no data is available. 3 also makes sense to me. Mixing data types is allowed in openEO, it's up to the implementations to decide whether they can support this or not. I would need to look into details for 4, I don't have a direct answer here.
Regarding the errors: They usually apply if there are no labels for the requested dimensions available, i.e. no data to load from. A datacube with all no-data values is not necessarily captured by that.


[Containers]Offloading CPU intensive Back-ends using Docker, Docker-Compose and HAProxy

Proposed title of article
Offloading CPU intensive Back-ends using Docker, Docker-Compose and Ha-proxy
Proposed article introduction
Most web app back-ends are normally deployed in form of containers. In this article, we will learn how to orchestrate those containers as well as enable horizontal scaling by offloading the requests among the different containers.  We will spin up a simple web server in a few lines of code using NodeJS/ExpressJS then create containers from its image.
Key takeaways
By the end of this article, you will be able to confidently perform the following tasks:

Create your custom Docker images.
Manage containers using Docker-Compose.
Understand how load balancers and reverse proxies work.
Horizontally scale CPU intensive tasks in your backend.

Article quality

Step-by-step in-depth guide for beginners
Use HA-Proxy as the load balancer instead of the common Nginx
Get an overview of how container orchestration is performed and apply the knowledge in other techniques e.g Kubernetes

References
N/A
Conclusion
This article is not limited to NodeJS/ExpressJS servers. It can be applied to different kinds of servers including Django. Conversely, container orchestration knowledge acquired from this article can be applied in other platforms other than Docker-compose e.g Kubernetes.

Sounds like a helpful topic - let's please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (the articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it).
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting it for final approval. (There are some great free tools that we reference in EngEd resources.)
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.
Please be sure to double-check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours. - Approved @WilliamOtieno

I'm going to close this topic due to inactivity.
It can be reopened.


Infinity keyword coloring

Is there any reason that Infinity is no longer yellow when prefixed by - or +? Also the prefix isn't white. Note: it works as expected with NaN.

It was being detected by an old regex that should have been removed.

Thanks


What happened to mostjs?

Summary
What happened to mostjs?

I don't see any new relases after sep 25 , 2017
Is it still alive?
Can i add mostjs for my upcoming new projects?
Will it be maintained?

Hi @ElamuruganGR
As mentioned at the top of the README, most/core is the new path for mostjs.  If you're starting a new project, we highly recommend using it.  It has much improved documentation, and is highly extensible via community contributed libraries.  There are some simple examples here
Practically speaking, the main differences you'll find with most/core, if you've used mostjs in the past will be:

most/core and companion libraries are function-based and not prototype method-based, so you'll mainly program with functions.
Projects will tend to use a few smaller, focused companion libs, instead of one monolithic mostjs.

Note that most/core is mature and stable, so will tend to be slow-changing.  Companion libs will be where new functionality is added.
I hope that helps and that you'll give most/core a try.  Cheers!


Member inactivity kick list

As of right now, we constantly demote people, in a clan called Lethal mob. We use Nevini as a crucial part of our member management tool. It would be awesome to see a feature where we can add a note or maybe inactivity status where we can do a quick lookup on a user to see if their away or not.
The reason for this request is due to our current system its pretty hard to loop up noted inactivity on a user, which results in people not really checking it, therefore us potentially kicking members that have said they're gonna be away for a bit.
I think this feature would be awesome to see in the future, regards -- Nathan (Clan IT Technician) & Love from lethal mob staff team

How about a command that exempts a user from the activity reports until a given date?

Yeah, just something where we can do a geo report on that member and maybe show us some kind of note, trick/crossfield
where it tells us that they have been noted down as on away (and maybe potentially a date when they are support to come back), so we know not to demote them if they have reached 60 days

Implementation started - new commands:

activity set away
activity get away
activity unset away

Reports will include this information as well


Decouple BoltDB

Scanning through the code, I'm wondering how difficult it would be to decouple BoltDB. If you made a simple interface, you should be able to swap out BoltDB for just a simple in memory map for smaller datasets. That wouldn't require any external assets and should perform even faster.

So based on this I implemented a "in-memory" option and as an example, you can use the CLI utility as a TCP server which holds everything in memory.
I don't think this is exactly what you're saying, but its a step in the right direction. The code needs to be refactored a bit before I can make an interface to swap out Bolt...
Also I'm not sure goagrep is any good for small datasets. You might be better off just running Levenshtein or similar for the entire dataset, depending on the case.

Wow! Thanks for being so responsive! This will be perfect for my use case. I have ~100k city, state, country or zipcodes to fuzzy match on, feeding a typeahead api lookup. It's fast and I rarely deal with any sort of memory pressure.

I don't think this is exactly what you're saying, but its a step in the right direction. The code needs to be refactored a bit before I can make an interface to swap out Bolt...

Looking at your implementation, I'm not sure that an interface is necessary. I doubt that you'll really need more than the two options. People can use in-memory if they have the RAM or the BoltDB implementation for larger datasets. I doubt people will be clamoring to swap out their own storage engine for it. :)

Great, glad it will work for you. Let me know if you find any bugs!


[BUG]Syncing the mesh... Error

2021-01-19T23:18:20.504+0800	ERROR	00000.defaultLogger	GRPC GetBalance returned error msg: account does not exist, address c17cb093064256a4c470813550969c9e8dcc5b72
2021-01-19T23:18:20.504+0800	INFO	00000.defaultLogger	not reporting error as buffer is full: {GRPC GetBalance returned error msg: account does not exist, address c17cb093064256a4c470813550969c9e8dcc5b72 goroutine 279 [running]:
runtime/debug.Stack(0xc007d98210, 0xc007d98210, 0xc007c19440)
C:/hostedtoolcache/windows/go/1.14.13/x64/src/runtime/debug/stack.go:24 +0xa4
github.com/spacemeshos/go-spacemesh/cmd/node.(*SpacemeshApp).setupLogging.func1(0x2, 0xbff9dbef1e12a520, 0x4f874d2d5, 0x1bb9aa0, 0x11096c0, 0x13, 0xc000040150, 0x6c, 0x0, 0x0, ...)
D:/a/go-spacemesh/go-spacemesh/cmd/node/node.go:303 +0x49
go.uber.org/zap/zapcore.(*hooked).Write(0xc0001bb800, 0x2, 0xbff9dbef1e12a520, 0x4f874d2d5, 0x1bb9aa0, 0x11096c0, 0x13, 0xc000040150, 0x6c, 0x0, ...)
C:/Users/runneradmin/go/pkg/mod/go.uber.org/zap@v1.15.0/zapcore/hook.go:65 +0xa2
go.uber.org/zap/zapcore.(*CheckedEntry).Write(0xc00013f810, 0x0, 0x0, 0x0)
C:/Users/runneradmin/go/pkg/mod/go.uber.org/zap@v1.15.0/zapcore/entry.go:216 +0x11e
go.uber.org/zap.(*SugaredLogger).log(0xc007c19770, 0xc000472002, 0x113a6c3, 0x46, 0xc007c19930, 0x1, 0x1, 0x0, 0x0, 0x0)
C:/Users/runneradmin/go/pkg/mod/go.uber.org/zap@v1.15.0/sugar.go:234 +0x107
go.uber.org/zap.(*SugaredLogger).Errorf(...)
C:/Users/runneradmin/go/pkg/mod/go.uber.org/zap@v1.15.0/sugar.go:148
github.com/spacemeshos/go-spacemesh/log.Log.Error(0xc0001bd740, 0x113a6c3, 0x46, 0xc007c19930, 0x1, 0x1)
D:/a/go-spacemesh/go-spacemesh/log/zap.go:32 +0x10e
github.com/spacemeshos/go-spacemesh/log.Error(...)
D:/a/go-spacemesh/go-spacemesh/log/log.go:118
github.com/spacemeshos/go-spacemesh/api.SpacemeshGrpcService.GetBalance(0xc0001fac40, 0x2383, 0x12c9c60, 0xc00017c6c0, 0x2979a060, 0xc0001a0a00, 0x12e7420, 0xc00007ab40, 0xc00025d200, 0x12d50a0, ...)
D:/a/go-spacemesh/go-spacemesh/api/grpc_server.go:134 +0x41b
github.com/spacemeshos/go-spacemesh/api/pb._SpacemeshService_GetBalance_Handler(0x10e4c40, 0xc00006cc30, 0x12d1020, 0xc007d98090, 0xc0005163c0, 0x0, 0x12d1020, 0xc007d98090, 0xc00016c1e0, 0x42)
D:/a/go-spacemesh/go-spacemesh/api/pb/api.pb.go:2148 +0x21e
google.golang.org/grpc.(*Server).processUnaryRPC(0xc0001fac40, 0x12dff00, 0xc0006c5b00, 0xc00012ad00, 0xc0006ad2c0, 0x1b935b0, 0x0, 0x0, 0x0)
C:/Users/runneradmin/go/pkg/mod/google.golang.org/grpc@v1.32.0/server.go:1194 +0x511
google.golang.org/grpc.(*Server).handleStream(0xc0001fac40, 0x12dff00, 0xc0006c5b00, 0xc00012ad00, 0x0)
C:/Users/runneradmin/go/pkg/mod/google.golang.org/grpc@v1.32.0/server.go:1517 +0xd04
google.golang.org/grpc.(*Server).serveStreams.func1.2(0xc0006c0040, 0xc0001fac40, 0x12dff00, 0xc0006c5b00, 0xc00012ad00)
C:/Users/runneradmin/go/pkg/mod/google.golang.org/grpc@v1.32.0/server.go:859 +0xa8
created by google.golang.org/grpc.(*Server).serveStreams.func1
C:/Users/runneradmin/go/pkg/mod/google.golang.org/grpc@v1.32.0/server.go:857 +0x20b
error}

@lrettig is this normal behaviour? I'm referring to the not reporting error as buffer is full part.

@lrettig is this normal behaviour? I'm referring to the not reporting error as buffer is full part.

"GRPC GetBalance returned error msg: account does not exist" message is coming from the old API, and the "not reporting error as buffer is full" ERROR isn't really an error. Old API was removed and the error was downgraded to DEBUG in #2131 (which code isn't included in the running testnet).

"GRPC GetBalance returned error msg: account does not exist" message is coming from the old API, and the "not reporting error as buffer is full" ERROR isn't really an error. Old API was removed and the error was downgraded to DEBUG in #2131 (which code isn't included in the running testnet).

@tyinsomnia thanks for pointing this out! These issues have already been fixed but the fix isn't included in the running testnet--it will be included in a future testnet release.

@tyinsomnia thanks for pointing this out! These issues have already been fixed but the fix isn't included in the running testnet--it will be included in a future testnet release.

For the record, this is still appearing in the latest testnet release because smapp is still using the old API. When smapp is upgraded to use the new API, we can remove the old API from the testnet builds and the error will go away.


Default configurations around coverage criteria and minimum coverage

Are there any reasons why branch coverage and 100% minimum coverage aren't the default configurations when using simplecov?

Yes, limited time. PRs to improve coverage ate welcome.
However, we have RSpec 4 telease in mind and we’re thinking of putting all libs to a monorepo. If you take a look at the RSpec 4 plan, increasing coverage to a maximum is there, and there’s also an rspec-core ticket.

@pirj apologies, I intended to post this issue in the simplecov repository, wondering why new projects that use simplecov don't default to branch criteria with a 100% coverage as configuration.. in my exploration, looks like I accidentally posted it here which made it look like a question on the maintenance of the project itself. Apologies for that. I will checkout the rspec 4.0 trajectory and will contribute as possible. Thanks.


pointertap event doesn't work for ScrollBox items on mobile phone

Expected Behavior
pointertap event should work for items added inside ScrollBox
Current Behavior
pointertap event doesn't work for ScrollBox items
Steps to Reproduce
Create container with sprite inside, add  pointertap event on it. Make sure that there is enough height so scroll can be done. Add it to ScrollBox. Open application on mobile phone and try to click.
Environment

version: 0.9.0
Browser & Version: Chrome  117.0.5938.140
OS & Version: Android 13, Samsung SM-A536B

@gagrison
Tried to reproduce this one, but no luck. Can you make some example on sandbox?
There is also a small thing to doublecheck:
in case if you are dragging a scrollbox even 1px, button event should not fire.
This is a fix in case clicks on buttons is not intentional, but actually it is pressed to drag a scrollbox.

Maybe this info will help. I made my own scroll box, and I had the same issue as reported here. I also made it that when you drag at least 1px all events are not propagated. But before that, I found out that globaltouchmove event is triggered on phones on touch with 0px drag (event.global.y).

Tested on both iPhone and Adnroid, both are firing pointertap event no problem.
Can you reproduce your example in sandbox? Like https://codesandbox.io/

If I find time I will. Since I made my own scrollbox this is not a priority sorry.

@CyberDex here you go link

@gagrison Thank you. Tried to reproduce on iphone and android, and all seems to work fine. Is it some specific device you are experiencing this on?

I already wrote it. I also tried on Samsung S23+, same thing.

Bump :)

It doesn't work on the official storybook either. It's important not to use an emulator but a physical device.

@gagrison @thpoul Guys could you please check if v 0.10.4 fixes your problem?

It does


"upcall fail 'option none'" with recursive iface impl?

Replacing "e" with "self" in src/test/run-pass/iface-to-str.rs causes an internal compiler error. Here's a slightly reduced testcase:
fn ignore_block(_b: block(str)->str) { }

iface to_str{
    fn to_str() -> str;
}
impl to_str<T: to_str> of to_str for [T] {
    fn to_str() -> str {
        ignore_block({|_e| self.to_str()});
        ""
    }
}
fn main() {
}

The error message isn't very helpful:
upcall fail 'option none', ../src/libcore/option.rs:34

The problem seems to be that the self value isn't being properly propagated to closures. I'll look into it.


Add --version flag

Also add issue template requesting this, as it would help avoid some easily solvable issues

This should also include the tvdb_api version as that is often a source of trouble


libffi:x64-windows-static fails

Fresh vcpkg on Win 7 pro
.\vcpkg.exe install cairo:x64-windows-static
->>
1/2] cmd /c "cd .. && "D:/vcpkg/downloads/tools/cmake-3.10.2-windows/cmake-3.10.2-win32-x86/bin/cmake.exe" "D:/vcpkg/buildtrees/libffi/src/v3.1-206c189062" "-DFFI_CONFIG_FILE=D:/vcpkg/ports/libffi/fficonfig.h" "-DBUILD_SHARED_LIBS=OFF" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=D:/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows-static" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=D:/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=static" "-DVCPKG_LINKER_FLAGS=" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-G" "Visual Studio 15 2017 Win64" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=D:/vcpkg/packages/libffi_x64-windows-static""
FAILED: ../CMakeCache.txt
cmd /c "cd .. && "D:/vcpkg/downloads/tools/cmake-3.10.2-windows/cmake-3.10.2-win32-x86/bin/cmake.exe" "D:/vcpkg/buildtrees/libffi/src/v3.1-206c189062" "-DFFI_CONFIG_FILE=D:/vcpkg/ports/libffi/fficonfig.h" "-DBUILD_SHARED_LIBS=OFF" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=D:/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows-static" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=D:/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=static" "-DVCPKG_LINKER_FLAGS=" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-G" "Visual Studio 15 2017 Win64" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=D:/vcpkg/packages/libffi_x64-windows-static""
CMake Warning at D:/vcpkg/scripts/buildsystems/vcpkg.cmake:89 (message):
There are no libraries installed for the Vcpkg triplet x64-windows-static.
Call Stack (most recent call first):
D:/vcpkg/downloads/tools/cmake-3.10.2-windows/cmake-3.10.2-win32-x86/share/cmake-3.10/Modules/CMakeDetermineSystem.cmake:94 (include)
CMakeLists.txt:2 (project)
CMake Error at CMakeLists.txt:2 (project):
Failed to run MSBuild command:
C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/MSBuild/15.0/Bin/MSBuild.exe

to get the value of VCTargetsPath:
Microsoft (R) Build Engine version <IP_ADDRESS>579 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.

Build started 25/05/2018 20:28:12.
Project "D:\vcpkg\buildtrees\libffi\x64-windows-static-rel\CMakeFiles\3.10.2\VCTargetsPath.vcxproj" on node 1 (default targets).
C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\VC\VCTargets\Platforms\x64\PlatformToolsets\v141\Toolset.targets(36,5): error MSB8036: The Windows SDK version 8.1 was not found. Install the required version of Windows SDK or change the SDK version in the project property pages or by right-clicking the solution and selecting "Retarget solution". [D:\vcpkg\buildtrees\libffi\x64-windows-static-rel\CMakeFiles\3.10.2\VCTargetsPath.vcxproj]
Done Building Project "D:\vcpkg\buildtrees\libffi\x64-windows-static-rel\CMakeFiles\3.10.2\VCTargetsPath.vcxproj" (default targets) -- FAILED.

Build FAILED.

The Windows SDK version 8.1 was not found.

updating vs seems to have fixed it


java.lang.RuntimeException: Failed to get macro asset 502 error

Hello
In recent time, after migration to the new docker image based on new chrome and image  https://github.com/SeleniumHQ/docker-selenium/releases/tag/3.141.59-20200326
we started to have issues with tests failing because of random errors generated by the OpenTest Server. Adding more resources helped a bit (from 1/10 test to 1/100) to reduce the number of issues related, but it still occurs regardless of that in peaks only 12% CPU and 9% of memory are used. It is related to all tests regardless of the type. Actors reporting this are on the same AWS ECS cluster as the server but in separate containers.
What does this error mean in relation to OpenTest server?
Has anyone experienced/reported a similar problem?
What should I check to eliminate this?

ERROR: java.lang.Exception: Failed executing test monitoring/KRLJ_addToCard, segment 1
	at org.getopentest.base.TestActor.executeTestSegment(TestActor.java:2122)
	at org.getopentest.base.TestActor.runOneSession(TestActor.java:2904)
	at org.getopentest.base.TestActor.runOneSession(TestActor.java:2940)
	at org.getopentest.Main.main(Main.java:20)
Caused by: java.lang.RuntimeException: There was an error while executing action no. 1 (monitoring.site.scenario.addProductToBasket) in segment 1 of test monitoring/KRLJ_addToCard.
The action definition was:
{
  "action": "org.getopentest.base.MacroAction",
  "args": {
    "siteTag": "krlj"
  },
  "macro": "monitoring.site.scenario.addProductToBasket"
}
	at org.getopentest.base.TestActor.executeTestSegment(TestActor.java:2080)
	... 3 more
Caused by: java.lang.RuntimeException: Failed executing action org.getopentest.base.MacroAction with arguments {"siteTag":"krlj"}
	at org.getopentest.base.TestActor.executeActionByDef(TestActor.java:1778)
	at org.getopentest.base.TestActor.executeTestSegment(TestActor.java:2030)
	... 3 more
Caused by: java.lang.Exception: There was an error executing macro action monitoring.site.scenario.addProductToBasket with arguments {"siteTag":"krlj"}
	at org.getopentest.base.TestActor.executeMacroActionByName(TestActor.java:1927)
	at org.getopentest.base.TestActor.executeActionByDef(TestActor.java:1658)
	... 4 more
Caused by: java.lang.RuntimeException: There was an error while executing action no. 2 (org.getopentest.base.MacroAction) in macro monitoring/site/scenario/addProductToBasket.
The action definition was:
{
  "action": "org.getopentest.base.MacroAction",
  "args": {
    "siteTag": "krlj"
  },
  "macro": "monitoring.site.action.openHomepage"
}
	at org.getopentest.base.TestActor.executeMacroActionByDef(TestActor.java:1879)
	at org.getopentest.base.TestActor.executeMacroActionByName(TestActor.java:1919)
	... 5 more
Caused by: java.lang.RuntimeException: Failed executing action org.getopentest.base.MacroAction with arguments {"siteTag":"krlj"}
	at org.getopentest.base.TestActor.executeActionByDef(TestActor.java:1778)
	at org.getopentest.base.TestActor.executeMacroActionByDef(TestActor.java:1844)
	... 6 more
Caused by: java.lang.Exception: There was an error executing macro action monitoring.site.action.openHomepage with arguments {"siteTag":"krlj"}
	at org.getopentest.base.TestActor.executeMacroActionByName(TestActor.java:1927)
	at org.getopentest.base.TestActor.executeActionByDef(TestActor.java:1658)
	... 7 more
Caused by: java.lang.RuntimeException: Failed to get macro asset monitoring/site/action/openHomepage from the sync service. The HTTP status code was: 502. The HTTP body was:
<title>502 Bad Gateway</title>

502 Bad Gateway

	at org.getopentest.base.TestActor.getTestAsset(TestActor.java:2313)
	at org.getopentest.base.TestActor.getMacroDefinition(TestActor.java:2330)
	at org.getopentest.base.TestActor.executeMacroActionByName(TestActor.java:1917)
	... 8 more

Tests and related failing macros seems to be random

@adrianth would it be hard to implement a retry mechanism in the Actor to cover cases like above?
Obviously we will be searching for an actual reason in our case, but errors like this can happen any time and as they are testing tool-related they should not lead to the failed test result, status like error or cancel would be more adequate in my opinion.

@galegor Adding a retry mechanism is fairly straightforward. I will go ahead and add that. Please keep us updated with any new information as you're working to identify the root cause for your issue - I would like to make sure it's not a problem with the OpenTest server.

@adrianth I'm trying to resolve the issue we have with no success. Noticed that server is using a lot of cpu after a few hours. Memory usage is also going gradually up.

OpenTest was only application running in this docked, repo sync. app was deactivated.

Reporting application which normally downloads all screenshots, logs and results was first moved to separate container and then deactivated ar all.

Multiple versions of OpenTest (current 1.2.1, 1.1.10, 1.1.5) where tested on 4GB container based on node image.

The load was around 3500 tests/day mainly monitoring. Single test duration around 10sec.

For the first few minutes, all was fine, after that first errors started to occur occasionally. After 24 hours it's impossible to use server at all.
There is nothing in the server nor actor logs that would be alarming for me, No idea how to debug this further. We think that possible this can be a memory leak problem. Could you print something to logs?

I'll try to investigate and see if I can think about any potential cause. I initially thought that the issue might be about the changes in version 1.2.1, but it sounds like you can also reproduce this with previous versions. Given that these issues started happening all of a sudden, one thing I would suggest is to try and revert any changes that happened recently and see if it things go back to normal.

@adrianth I struggle to resolve this, any ideas or potential cause on your side?
Three things I am referring to:
1 retry to get an asset when 502 is returned from the server (covering issue really, but this can be very helpful). If you could implement this with reporting an error in logs would be great.
2 random 502 errors related to all type of tests
3 server resource usage on load circa 3500-4000 tests a day, 10 actors
Caused by: java.lang.RuntimeException: Failed to get macro asset api/core/cartService/getProductData from the sync service. The HTTP status code was: 502. The HTTP body was:
<title>502 Bad Gateway</title>

502 Bad Gateway

The 502 Bad Gateway errors are being generated by the proxy server you are using. Could you please let me know:

What proxy server you are using (Apache, Nginx, etc.)?
Have you tried to look at the proxy server logs? There could be useful information in there to indicate the underlying cause for these errors.
Can you execute the top command (or something similar) and let me know how much memory and CPU are used by the server and the actor processes once the 502 errors start to intensify?

Sure, we are using AWS application load balancer. Errors are not generated, only forwarded from the server. Here is a recent log:
https 2020-04-10T08:26:01.408055Z app/r03c07-lte-test-server01/7407ad9902bcae2f XX.XXX.XXX.XX:48547 XX.XXX.XX.XXX:41430 0.000 0.260 -1 502 - 247 704 "GET https://XXX.com:50001/api/test-asset?type=macro&path=monitoring%2Fsite%2Faction%2FopenHomepage HTTP/1.1" "Apache-HttpClient/4.5.3 (Java/1.8.0_242)" XXX-RSA-AES128-XXX-SHA256 TLSv1.2 arn:aws:elasticloadbalancing:us-east-1:768678456456:targetgroup/r03c0-Targe-XXXXXM395YO5/xxxxx7d6cf74c3 "Root=1-5e902d99-db10ec6201a80a5e265c935c" "XXX.com" "arn:aws:acm:us-east-1:768678456456:certificate/XXXXXXXX-XXXXX-XXXXXX-babf-XXXXXXXXXX" 1 2020-04-10T08:26:01.148000Z "forward" "-" "-" "XX.XXX.XX.144:41430" "-"
And here is a total server memory (orange line) and CPU (blue line) usage from last couple hours:

No of errors in the roughly same time:

I will share top results later, need to rebuild containers to have this data.


(feat): Support use(Context)

This change adds support for use(Context).
Currently, this is using what I'm calling a "manual" or naive approach to determine if the "usable" (the value passed to use() is a Context or a PromiseLike. As noted in #1, an alternative and maybe safer option would be to pull in react-is and check usable.Consumer and usable.Provider - but that means pulling in another dependency/peerDependency.
Closes: #1

Please check CI errors.


[speechx]add aishell test script

PR types

[New features]
PR changes

Describe

add aishell test script

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


Extract can generate empty cells

Describe the bug
Using component.extract with a layer argument on a cell that does not contain this layer populates an empty polygon:
To Reproduce
import gdsfactory as gf
from gdsfactory.typings import LayerSpecs, ComponentSpec
from gdsfactory.generic_tech import LAYER

@gf.cell
def layer_rectangles(component: ComponentSpec,
                        layers: LayerSpecs,
                     ):
    """Building a derived component of rectangles corresponding to the dimensions of select layers in another component."""

    c = gf.Component()

    for layer in layers:
        rectangle = c << gf.components.rectangle(
            size=[
                gf.get_component(component).extract(layers=(layer,)).xsize,
                gf.get_component(component).extract(layers=(layer,)).ysize,
            ],
            layer=layer,
        )

    return c

@gf.cell
def mycomponent(present_layers: LayerSpecs):
    c = gf.Component()

    for layer in present_layers:
        c << gf.components.circle(layer=layer)

    return c

original_component = mycomponent(present_layers = (LAYER.WG, LAYER.SLAB90))
original_component.show()

derived_component = layer_rectangles(component = original_component,
                        layers = (LAYER.WG, LAYER.SLAB150)
                        )
derived_component.show()

Note the empty polygon for SLAB150:

Expected behavior
If the layer is not present, extract should do nothing or throw an error.
Suggested fix
The layers argument in .extract could be parsed when called to only iterate over present layers (set intersection)

nice catch Simon,
sending  a PR fix for this


Revert "Translations update from Mattermost Weblate"

Reverts mattermost/desktop#3254
Needs to be rebased and merged, not squashed.

@devinbinnie, curious why this needs to be merged vs squashed? We fixed the i18n issues that previously required this, so not necessary from weblate's perspective at least.

sigh Well that's news to me...
I will un-revert :P

It's fine either way, just for the future @devinbinnie :)


TableStorage Timestamp shows UTC instead of user's localtime

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.33.1
Regression From
No response
Architecture
x64
Storage Explorer Build Number
20240410.2
Platform
Windows
OS Version
win 23h2
Bug Description
All timestamps are shown in UTC, not converted to local time as I have read in many posts.
There is no mention of settting timezone or setting how to display times in the help..
Steps to Reproduce
show a query from tablestorage
Actual Experience
shows utc time
Expected Experience
show local time based on user's computer's time Zone, or let me select the time Zone.
Additional Context
none.

@Terrence77 We have implemented this feature, which will be present in 1.37.0!
We added a new setting: Data Explorers > Table Explorer > DateTime Format with three options: None, Local, and UTC. When set to Local or UTC, Storage Explorer will format DateTime values using your system format settings in the respective time zone.
Thank you for your feedback! We hope this will meet your needs.

Great news, thank you for adding this feature.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Craig Alvord @.>
Sent: Friday, November 15, 2024 12:14:48 PM
To: microsoft/AzureStorageExplorer @.>
Cc: Terrence77 @.>; Mention @.>
Subject: Re: [microsoft/AzureStorageExplorer] TableStorage Timestamp shows UTC instead of user's localtime (Issue #7954)
Closed #7954https://github.com/microsoft/AzureStorageExplorer/issues/7954 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/AzureStorageExplorer/issues/7954#event-15316074195, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALERUTG6Z2H2QR2WCS42VFL2AY2ZRAVCNFSM6AAAAABIEV2NB6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGMYTMMBXGQYTSNI.
You are receiving this because you were mentioned.Message ID: @.***>


No devices found on bridge

Having a similar issue to #65 that I can't seem to figure out.  I have added devices to my bridge and have confirmed they work manually and from the test functions but when I tell Alexa to discover my devices it returns that she found no devices.  Looking at the logs nothing sticks out to me but I get the impression it does in fact get to the device but cannot return the deice list.
Just to understand the pieces in this scenario I have OpenHAB+ha-bridge running on a raspberry pi 1 model b which is connected via ethernet with a ddwrt router serving the home network.  I have tried a mix of wired/wireless settings to no avail and for what its worth I have a firetv2 that I initially tested with prior to owning an echo and had identical results to this then as well.  Any assistance would be greatly appreciated.  I have attached a debug log, habridge.conf and device.db in case they can offer any window into what I may be doing wrong.
Any further insight would be very appreciated

If your echo is on ip <IP_ADDRESS> it sure looks like it found the bridge and asked for the devices. I'll need to dig a bit more with the issue on #65 to see what can be done to figure this out.

Thats correct, .211 is my Echo.  I'll keep my eyes peeled on #65. If there is anything I can do to help test or attempt I'd be glad to - in the interim I am going to update my routers firmware to make sure that I am not having any issues on that side of the house.  No need to chase something that is not related to the project.

It looks like tonight my device is allowing itself to be added .. how nice of it.  I have yet to change anything on my network however I did just power cycle the router prior to me asking it to discover the devices.  I'll continue to test to see if this behavior continues and if so close this as soon as I confirm.

It seems there have been many instances where the router has gotten in the way of upnp handling. #65 is now closed due to a router setting.

Is this working?

Any new comments?

Closing as it is working for user.


Chart fill parent dimension

Is it possible to make a chart fit the parent component dimension?
Without specifying the width or height attribute, the chart crashes (NaN provided).
I'd like to wrap the chart in a Card component, with custom borders and so on, but I don't understand how to correctly wrap it.
Thank you for this great library, it's one of my favourite!
S.

@besil  Did you manage to get the chart to fill up 100% of the parent container's dimensions?
Tried the following but the app throws an error about Invariant Violation
<LineChart
    height={'100%'}
/>

Hi, I just used
const screenWidth = Dimensions.get('window').width;
I'd have preferred  a 100% option, but it doesn't seem to work

Any updates on this issue?

Yeah, I have an issue with this too, I need to manually remove padding from all parent containers to make chart not overflow parents.

Still an issue in 2021.. No one seems to know how to get this working.. My other option might be to somehow get the dimensions of parent View and go from there..

Any update?

Any update?

I did it somehow, don't remember how though. When I get back to work tomorrow I can check it out and update this thread.

You are my hero!

Any update?

I did it somehow, don't remember how though. When I get back to work tomorrow I can check it out and update this thread.

Are you still at work ? 😆

https://github.com/indiespirit/react-native-chart-kit/issues/506#issuecomment-917564118 by mathcale

See solution by mathcale #506 (comment)

any other solution so far? I've been facing this issue for days 😞


feat(insights): show docs URL when  is missing.

Summary
When clickAnalytics: true is missing, we already fire an error.
In this PR, we guide the user even more by adding a link to the docs showing how to add it.
Result
N/A

@francoischalifour that's fixed, thank you.


switched from Dial() to DialTimeout() for GetStatus()

DIAL_TIMEOUT constant set to 5s
Signed-off-by: Andrew Watson<EMAIL_ADDRESS>

LGTM :+1:


Logging buffer for managed processes persists between messages

There are lots of different ways to fix this ... need to identify the one that will be efficient for core.async.

This can be ignored for now, since the banner has been commented out.


Kubernetes Integration

Summary
This projects seems to supply the best DNS server in a container that one could ask for (especially when it comes to security features). Therefore, I'd like to help with documenting Kubernetes. I came across this as coreDNS does not seem to be able to do DNSSEC validation (which I find to be ridiculous). Therefore, I can point coreDNS to this container in my cluster and let this container do the resolution over TLS/HTTP with DNSSEC etc.
What would this help look like?
I am going to write a Deployment, Service and configMap which should suffice to get this up and running in a cluster. I would then add a document, maybe in the form of a KUBERNETES.md, because Kubernetes support should stay community-maintained (as it is a lot of work and maintainers most likely do not have the time).
Why this issue?
I want to ask first before doing the work in order to avoid unnecessary work. If maintainers are fine with this, I will submit a PR in the future.

That sounds perfect. Go for it!
Note however that the current latest and v1.x.x docker images do dns over tls AND Dnssec, not Https. I could add https now that Unbound supports it if really needed, but...
Since before Unbound added support for DNS over HTTPS I started working on a full go only implementation of DNS over TLS+HTTPS which is available in tag :v2.0.0-beta. Now that one doesn't do DNSSEC yet, and there is also a bug where the server hangs when shutting down. It will also have breaking changes (different variables, be based on Scratch etc.). So don't use that one for now, but just so you're aware you might need to update your files in the next few weeks (maybe 1-2 months).

I'm currently using the v2 Beta and it seems to work fine. I would like to stay on the beta version to test the latest changes. I do not actually need DoH, because I use DoT. DNSSEC is something I look forward to :D Reach back to me when DNSSEC is ready :)
I will come up with a PR when I'm ready and I will put this in a new file :)

Great, I'll try to focus on DNSSEC. Right now I'm adding Prometheus metrics to debug (the lazy way) that shutdown issue. If you use v2, leave the UPDATE_PERIOD=0 so it doesn't restart for now. Mine stalls about every ~2 weeks with a 12 hours restart period and it has to be manually rebooted ughh. I'll let you know once #60 is done.

@qdm12 Have a look at #83, these are the instructions :)
Just reach back to me when DNSSEC is ready :)

@qdm12 #83 is ready to be merged :)

Ah ok sorry about that I thought you wanted to wait for DNSSEC! (subscribe to #60 then 😉)
I'm going to be annoying, sorry in advance, but sadly I rebased v2.0.0-beta on master so there is a whole lot of weird deltas on the PR now (especially for the readme.md for some reason).
Would you mind doing that on your forked repo (basically redo your changes on a fresh k8s branch):
git checkout k8s
cp KUBERNETES.md KUBERNETES.md.bak
git checkout v2.0.0-beta
git pull
git branch -D k8s
git checkout -b k8s
mv KUBERNETES.md.bak KUBERNETES.md

Then small manual step, re-add
If you're running Kubernetes, there is a separate article on [how to set up K8s](https://github.com/qdm12/dns/blob/v2.0.0-beta/KUBERNETES.md).

Before ## Docker tags in the README.md
Finally
git add .
git commit -m "chore(K8s): add Kubernetes usage instructions"
git push --force

Thanks!!

No big deal, just a rebase :) I hope I did it right though :D
I will subscribe to #60. In the meantime, #83 can be merged and this issue closed.

I will close this as #83 was merged.

Thanks @georglauterbach!


Published version is dependent on Scala library

The pom file includes the dependency below.  That might be avoided with a line like this in the build.sbt file.
autoScalaLibrary := false // Do not depend on Scala libraries since there is no code here.

Including it will help us use the model with different versions of Scala and not rely on Scala at all for Java or Python projects.
    <dependencies>
        <dependency>
            <groupId>org.scala-lang</groupId>
            <artifactId>scala-library</artifactId>
            <version>2.12.6</version>
        </dependency>
    </dependencies>

Makes sense. Thanks @kwalcock!
On Sat, Dec 10, 2022 at 3:24 AM Keith Alcock @.***>
wrote:

Assigned #2 https://github.com/clulab/processors-models/issues/2 to
@MihaiSurdeanu https://github.com/MihaiSurdeanu.
—
Reply to this email directly, view it on GitHub
https://github.com/clulab/processors-models/issues/2#event-8002674092,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI75TUIL34HKXQOAXEKUO3WMO5RRANCNFSM6AAAAAASZ3EG5Q
.
You are receiving this because you were assigned.Message ID:
@.***>


Fix #48403 broken UNC paths in markdown images

Fix #48403.
It's my first PR in this repo, I don't know if it could be fixed in a better way, if so, please tell me.
The problem is:
contents.session.protocol.registerBufferProtocol(protocol, (request, callback: any) => {
	const requestPath = URI.parse(request.url).path;
	//request.url = "vscode-resource://desktop-9su47m5/test-vscode-issue/350x150.png"
	//requestPath = /test-vscode-issue/350x150.png
	const normalizedPath = URI.file(requestPath);
	for (const root of getRoots()) {
		if (!startsWith(normalizedPath.fsPath, root.fsPath + sep)) {
			continue;
		}
		// ...

normalizedPath.fsPath won't have the //desktop-9su47m5 part of the path, because normalizedPath variable is constructed with URI.file("/test-vscode-issue/350x150.png"), so the condition !startsWith(normalizedPath.fsPath, root.fsPath + sep) will never be false with UNC paths.

I just added another commit. The previous one was just handling UNC paths.
With project in the root of a drive, this code:
if (!startsWith(normalizedPath.fsPath, root.fsPath + sep)) {

When root.fsPath is a folder, it doesn't have a trailing slash, like c:\folder\subfolder, but when the project root is at the root of a drive, root.fsPath is c:\ with a trailing slash.
So, as it always concatenates the sep separator, the condition was this:
!startsWith('c:\image.png', 'c:\' + '\')
Resulting in a c:\\ second parameter passed to startsWith.

Thanks for looking into this. Marking for June since it is too late to merge for the May release.
@lmcarreiro Can you please quickly overview how you tested that this change works and that it does not cause regressions?

@mjbvz I know that it works because I created a shared folder in my PC with the example of a simple .md file with a local image and I opened it both from the UNC path and from a mapped network drive (of the same UNC path) with the .md file and the image on the root of this drive. Doing these tests that I was able to identify this other bug (#74362) that happens in the same scenario.
I ran the automated tests with and without my changes and the results were the same.
About the possibility of regressions, I don't think that would be any, because of the kind of these changes:

The first change, it will just concatenate a //<authority> if there is an authority on the URI parsed request.url. I took a look at the URI.parse code and regex, and I saw that it just happens when we pass a URI that starts with something:// and a non-slash character after the two slashes. The only two cases I can think are when we are dealing with UNC paths or absolute URLs.

The case of a UNC path was the bug itself, that I already tested.
The case of a URL, I tested and saw that this code never executes, because it doesn't start with vscode-resource:.

The second change, the change in the actual behavior is only in the case when it was concatenating a slash at the end of a path that already ends with a slash, clearly not the intention of who wrote the code and probably expected a path that would never have a trailing slash.

Anyway, I don't know the architecture of this project. The only resource I found about this was the Code-Organization that doesn't help too much. If there is anything that could be made in a better way, or if it would be better to be fixed in other place of the code, just give me directions and I'll try to do it.

Hey can you please take a quick look at resolving the merge conflicts for this PR. The webview code has been reworked quite a bit recently to support the web. I intended to merge this PR earlier but it fell off my radar

I saw that the code I changed was moved into the new file resourceLoader.ts. So, I moved my changes to this file, it worked for paths at the root of a network mapped drive, but it didn't for an UNC path.
It is missing a slash after the vscode-resource:
contents.session.protocol.registerBufferProtocol(protocol, (request, callback: any) => {
// request.uri for UNC paths:
// It used to be like: "vscode-resource://desktop-9su47m5/test-vscode-issue/350x150.png"
// Now it is like:     "vscode-resource:/desktop-9su47m5/test-vscode-issue/350x150.png"

I did a git bisect to find the commit that changed this behavior, and it was the merge of this pull request: #75741

I added a comment there in the line that introduced the change: https://github.com/microsoft/vscode/pull/75741/files#r296498579
-<base href="${markdownDocument.uri.with({ scheme: 'vscode-resource' }).toString(true)}">
+<base href="${toResoruceUri(markdownDocument.uri)}">

-<!--<base href="vscode-resource://desktop-9su47m5/test-vscode-issue/test.md">-->
+<!--<base href="vscode-resource:/test-vscode-issue/test.md">-->

Anyway, I rewrote my changes in the new file resourceLoader.ts, but to work with UNC paths, you need to review this change. As you used this new method toResoruceUri in other places too, I think it's best that you fix this <base>'s href attribute yourself.

Thanks. Will fix the markdown case as part of #76489


Move News to Json Endpoint

This PR Moves the Fetching of the news over to the rss.json backend and fetches the avatars from the yaml
ref: #29
Remarks

Godot does not have any YAML parsing; thus, I hacked a small version together that should work
Godot does not seem to recognize or parse some special characters correctly, and thus, the "workaround" at L138 is required. Currently, only ' is used in the descriptions because I couldn't find any functions that decoded that character correctly.
The time format the JSON uses is RFC 2822, but Godot only supports ISO 8601 date formats due to that i just removed the hour and timezone info from the pubDate and used it.

Feedback is very welcome 😄

Thanks for the PR. I'll review it in a couple of weeks when I'm back from vacation.

Merged. Just pushed to little changes fixing images url containing the base url twice, and handling of yaml strings wrapped in single and double quotes.
Thanks for contributing! 😊


Format

To address #89.
note: this has a shared base with #165, I can rebase it if needed
I have updated the overall approach to fixing this problem. Instead of attempting to get our indentation correct on smaller pieces, I have moved this to a single step over a whole js module.
I then ran eslint against each of the out.js files generated by cargo test. Any errors that were reported I attempted to fix them. Since this approach seemed valuable I added an eslint step to the travis test that will review all out.js and out_bg.js files for formatting. Adding an eslint test should help reduce any incorrectly formatted code moving forward.

@FreeMasen can you rebase and squash this PR? Thanks :)

@fitzgen rebased and squashed


Bad Marshal Data (unknown type code)

Traceback (most recent call last):
File "keras_server.py", line 353, in
load_FRmodel()
File "keras_server.py", line 94, in load_FRmodel
model = load_model('models/model.h5', custom_objects={'triplet_loss': triplet_loss})
code = marshal.loads(raw_code)
ValueError: bad marshal data (unknown type code)

same problem

anyone able to solve this error?

https://github.com/keras-team/keras/issues/9595

I am facing the same issue is anyone solved this?

Nope, gave up on the project.
On Thu, Aug 31, 2023 at 4:22 AM Abdul Hadi @.***> wrote:

I am facing the same issue is anyone solved this?
—
Reply to this email directly, view it on GitHub
https://github.com/susantabiswas/facial-login-web/issues/9#issuecomment-1700583317,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUQVJ47HBG5H5QHYOPDB35DXYBCUVANCNFSM5EKHRQHA
.
You are receiving this because you commented.Message ID:
@.***>

is this problem fixed atleast now?????? please help


Test Discovery Failed: Extension version v2023.11.11731012, pythonTestAdapter setting on

The Pre-release version v2023.11.11731012 of Microsoft's Python extension for Visual Studio Code fails to discover one of my tests.
Behaviour
The command
conda run -n my_env --no-capture-output python ~/.vscode-server-insiders/extensions/ms-python.python-2023.11.11731012/pythonFiles/get_output_via_markers.py ~/.vscode-server-insiders/extensions/ms-python.python-2023.11.11731012/pythonFiles/unittestadapter/discovery.py --port 34713 --uuid 16c443af-8630-465d-8c14-8c5317ce917b --udiscovery -v -s ./tests -p *test*.py

Results in a Unittest test discovery error due to a module not being found.
Expected vs. Actual
In contrast, switching to release version v2023.10.1 of the same extension works without problems. It runs a different command, though:
conda run -n my_env --no-capture-output python ~/.vscode-server-insiders/extensions/ms-python.python-2023.10.1/pythonFiles/get_output_via_markers.py ~/.vscode-server-insiders/extensions/ms-python.python-2023.10.1/pythonFiles/testing_tools/unittest_discovery.py ./tests *test*.py

Steps to reproduce:

Install Microsoft's Python extension, pre-release version v2023.11.11731012.
Create a test in a directory that depends on a module with files outside that directory. E.g., "test" and "module" may be parallel directories, with a unit test in the directory "test" and "module" contains Python files and a file named "__init__.py".
Configure tests.

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.8.16
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Conda
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

2023-06-22 16:38:50.133 [info] Discover tests for workspace name: TNLGv4-Inference - uri: /home/paulo/repos/TNLGv4-Inference
2023-06-22 16:38:50.133 [info] Running discovery for unittest using the new test adapter.
2023-06-22 16:38:50.142 [info] > conda run -n my_env --no-capture-output python ~/.vscode-server-insiders/extensions/ms-python.python-2023.11.11731012/pythonFiles/get_output_via_markers.py ~/.vscode-server-insiders/extensions/ms-python.python-2023.11.11731012/pythonFiles/printEnvVariables.py
2023-06-22 16:38:50.142 [info] shell: bash
2023-06-22 16:38:50.617 [info] Running unittests with arguments: /home/paulo/.vscode-server-insiders/extensions/ms-python.python-2023.11.11731012/pythonFiles/unittestadapter/discovery.py,--port,44497,--uuid,c99d8672-d60e-40e1-8e5a-cad8b005d80b,--udiscovery,-v,-s,./tests,-p,*test*.py

2023-06-22 16:38:50.617 [info] > conda run -n my_env --no-capture-output python ~/.vscode-server-insiders/extensions/ms-python.python-2023.11.11731012/pythonFiles/get_output_via_markers.py ~/.vscode-server-insiders/extensions/ms-python.python-2023.11.11731012/pythonFiles/unittestadapter/discovery.py --port 44497 --uuid c99d8672-d60e-40e1-8e5a-cad8b005d80b --udiscovery -v -s ./tests -p *test*.py
2023-06-22 16:38:50.617 [info] cwd: .
2023-06-22 16:38:51.312 [info] Test server connected to a client.
2023-06-22 16:38:51.314 [info] Using result resolver for discovery
2023-06-22 16:38:51.314 [error] Unittest test discovery error
 Failed to import test module: my_module_utils_test
Traceback (most recent call last):
  File "/home/me/mambaforge/envs/my_env/lib/python3.8/unittest/loader.py", line 436, in _find_test_path
    module = self._get_module_from_name(name)
  File "/home/me/mambaforge/envs/my_env/lib/python3.8/unittest/loader.py", line 377, in _get_module_from_name
    __import__(name)
  File "/home/me/repos/my_repo/tests/my_module_utils_test.py", line 3, in <module>
    from \
ModuleNotFoundError: No module named 'my_module'

User Settings

languageServer: "Pylance"

linting
• flake8Args: "<placeholder>"
• flake8Enabled: true
• pycodestyleEnabled: true
• pydocstyleEnabled: true

formatting
• autopep8Args: "<placeholder>"
• provider: "none"

testing
• unittestArgs: "<placeholder>"
• unittestEnabled: true

Hi! Thank you for catching this! This is using our new test rewrite architecture and it seems to be missing the right path to find everything. I will investigate further and get back to you with any questions or a fix. For now, you can disable the new testing rewrite using "python.experiments.optOutFrom": ["pythonTestAdapter"],. Thanks!

The reason for this is the cwd was not added to the PYTHONPATH when unittest discovery was run. This fix is now merged and will be out in insiders tomorrow. Verification steps are just the repro steps above- conda environment is not required to verify.

Awesome! Thanks for the fix!
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Eleanor Boyd @.>
Sent: Friday, June 23, 2023 5:51:44 PM
To: microsoft/vscode-python @.>
Cc: Paulo R. S. Mendonca @.>; Author @.>
Subject: Re: [microsoft/vscode-python] Extension version v2023.11.11731012 fails to discover test found with v2023.10.1 (Issue #21476)
Hi! Thank you for catching this! This is using our new test rewrite architecture and it seems to be missing the right path to find everything. I will investigate further and get back to you with any questions or a fix. For now, you can disable the new testing rewrite using "python.experiments.optOutFrom": ["pythonTestAdapter"],. Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-python/issues/21476#issuecomment-1605022037, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACTMCBCCNVGFGQYNDW2HFPLXMYFXBANCNFSM6AAAAAAZQ4AOJQ.
You are receiving this because you authored the thread.Message ID: @.***>


arguments field for CREATE in broadcast log is null

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (20e7386 2022-08-31T00:09:48.876171Z)
What command(s) is the bug in?
No response
Operating System
macOS (Apple Silicon)
Describe the bug
The arguments field is populated for function calls, but constructor arguments should be there as well, especially since they're the only way to distinguish multiple deploys of the same base contract.

+1 I've noticed this same bug

was this fixed by https://github.com/foundry-rs/foundry/pull/3136 ?

No that was for a different field
{
  // This was fixed in #3136
  "transactionType": "CREATE",
  "contractName": "MyContract",
  "function": null,
  // This is what this PR refers to
  "arguments": null
}


edit ch5 for syntax issues

Additional comments about content:

Toggle Diagnostics and Toggle Profiler speech commands not pre-installed in RC2.1
Diagnostics Demo Controls script not included in RC2.1
Move with Pan script not included in RC2.1
Couldn't find Eye Tracking input profile

Docs Build status updates of commit c656fdf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/mrlearning-base-ch5.md
:white_check_mark:Succeeded
View

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


Speed up TimeDistributedDense, 3D Softmax and RNNs

Found some serious bottlenecks in core parts of Keras:

Currently TimeDistributedDense is implemented using K.rnn(), i.e, each vector in the sequence is dot with W sequentially, one after the other. Since each time step of the rnn is independent of the previous time step (no state involved), instead of looping through the sequence, it is possible to obtain the output in one single huge matrix operation.

Similarly, softmax activiation for 3D data is also implemented using rnn(), which translates to 1 softmax op per vector.

Now if you were to combine these two, i.e, TimeDistributedDense(10, activation='softmax'), Keras will loop through each of your sequence twice, one for the dot operation and the other for the activation. Sad ):

Instead of making use of shaping ops such as permute_dimesions (which come at no cost), Dot and sum ops are used to initialize RNN hidden states.

@fchollet

It would be bad practice to have segments of code dedicated to one backend or another. Would there be a performance impact to just using the K-based code?

Its not about performance. The K based code needs the input length, but when using theano backend it is possible not to provide the time dimension.

@fchollet it's all K code now. Happy ? :)

@fchollet Done. It's all K code now. Works even if input_length is None. Happy? :)

Having a tough time fixing tensorflow issues. Slicing isn't as easy as theano. X[0][0] and X[0, 0] do not work. Can not reshape if there are unknown dims in the reshape tensor. Crappy. To make things worse, i am on Windows and do not have TF installed, so have to rely on Travis to test my code :(

@fchollet  Finally, the tests are passing.

Looks good, although readability of TimeDistributed is taking a hit. Do you have any numbers on performance before/after with a test network?

@fchollet Numbers won't be necessary, we just reduced time complexity from O(n^4) to O(n^3). Its massive. (Consider stacked TimeDistributedDense layers). Working on the comments.

Done.

Thanks. Looks good to me, please squash your commits.

:+1:  @farizrahman4u

I messed up while squashing. My branch looks fine.. pztch-25, but something happened to the pull request. Anyway moved to #1812.

Travis not working

It happens. For now, just squash and rebase, we'll re-run the tests when we can.

Squashed to 3 commits.

Cool, merged.


[Task] Add Dev Test Labs and a VM to Azure ML Bundle

Story: #132
Description
Build upon #205 and add a dev test labs service

Merged, closing.


validation error:genesis block hash is not provided

Hi. I tried to run avalanche-rosetta on my local machine. I followed the instructions at https://github.com/ava-labs/avalanche-rosetta. But I get that error:
2021/11/30 14:32:22 config validation error:genesis block hash is not provided.
My config file is:
{   "rpc_endpoint": "https://api.avax-test.network",   "mode": "online",   "listen_addr": "<IP_ADDRESS>:8080",   "network_name": "Fuji",   "chain_id": 43113,   "log_requests": true }

You need to provide the genesis_block_hash key as shown here: https://github.com/ava-labs/avalanche-rosetta#usage