ENH: Add Context.jsonStream(...) to write x-json-stream content from Stream or Iterator

  /**
   * Write the stream as a JSON stream with new line delimiters
   * {@literal application/x-json-stream}.
   *
   * @param stream The stream of beans to write as json
   */
  <E> Context jsonStream(Stream<E> stream);

  /**
   * Write the stream as a JSON stream with new line delimiters
   * {@literal application/x-json-stream}.
   *
   * @param iterator The iterator of beans to write as json
   */
  <E> Context jsonStream(Iterator<E> iterator);

Note that when using Iterator internally it will check if the instance implements AutoCloseable and if so will close the iterator when all elements have been written.  Stream implements AutoCloseable so that also gets closed.

FYI @tipsy ... pretty sure you saw this but just in case.  Right now I don't quite have time at the moment to prepare a PR for Javalin for this but let me know and tag me on a issue if/when we create one in Javalin.
Cheers, Rob.

@rbygrave It's a nice feature, I'll create an issue :)


[LI-HOTFIX] Prevent ReplicaFetcherThread from throwing UnknownTopicOrPartitionException upon topic creation and deletion.

TICKET =
LI_DESCRIPTION =
When does UnknownTopicOrPartitionException typically occur?

Upon a topic creation, a follower broker of a new partition starts replica fetcher before the prospective leader broker of the new partition receives the leadership information from the controller. Apache Kafka has a an open issue about this (see KAFKA-6221)
Upon a topic deletion, a follower broker of a to-be-deleted partition starts replica fetcher after the leader broker of the to-be-deleted partition processes the deletion information from the controller.
As expected, clusters with frequent topic creation and deletion report UnknownTopicOrPartitionException with relatively higher frequency.

What is the impact?

Exception tracking systems identify the error logs with UnknownTopicOrPartitionException as an exception. This results in a lot of noise for a transient issue that is expected to recover by itself and a natural process in Kafka due to its asynchronous state propagation.

Why not move it to a lower than warn-level log?

Despite typically being a transient issue, UnknownTopicOrPartitionException may also indicate real issues if it doesn't fix itself after a short period of time. To ensure detection of such scenarios, this PR sets the log level to warn.

EXIT_CRITERIA = TICKET [KAFKA-6221]
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@gitlw Thanks for the review! Yes, I will create a PR on Apache Kafka, as well.


PrettySimpleShop spits an error when my players buy an item from other players.

Hi,
I am currently having a issue with PSS (1.6.0, Paper 1.14.4 build 163, Economy is Vault Based)
When a player purchases an item from a shop, he gets the item and the payment doesn't go to the shop owner. I would like the player to actually get the payment in his inventory (Item Economy), and if inv is full in the shop chest.
Error pastebin: https://paste.helpch.at/efupazewoh.rb
Hopefully you can resolve this A.S.A.P.
~ Pepijn

Looks like a problem with your economy plugin, not PrettySimpleShop.


Update debug-microflows-remotely.md

Dear Mx,
Sending this request to add a sentence on remote nanoflow debugging (which isn't possible) - the menu structure of the documentation section did suggest it is possible + the error message given in Studio Pro is not clear as to why the breakpoint is disabled. Hence suggest adding this section. Please also refer to Mx Support ticket 141362.
Best regards,
Wouter Penris

Hi Wouter,
Thanks a lot for your PR. As Ekrem mentioned in his comment, we need to verify if there is another way to debug nanoflows before merging the PR.
Best regards,
Maria

Dear all,
This is the answer I got from MXsupport - I was debugging a native app with an on-premise install on an OT environment. Please consult with them. If it is possible, then it is currently not working :)
Looking forward to your reply,
Wouter
Sent from my iPhone
On 14 Feb 2022, at 14:42, Maria Shaposhnikova @.***> wrote:

Hi Wouter,
Thanks a lot for your PR. As Ekrem mentioned in his comment, we need to verify if there is another way to debug nanoflows before merging the PR.
Best regards,
Maria
—
Reply to this email directly, view it on GitHubhttps://github.com/mendix/docs/pull/4158#issuecomment-1039064478, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQMVUPVOCKYESDBP45PZDSDU3D5BRANCNFSM5NZ6L2MQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

Dear Wouter,
I have verified with Mendix Support that the information in the PR is correct, I reviewed it and will merge it to production.
Best regards,
Maria


WebSockets not working for Android versions lower than N ()

socket.io-client and react-native-socket.io-client both are working fine on IOS all versions + Android N, I'm getting above error only on lower android versions.
Any ideas what is causing this ??

母鸡啊！

I also experienced this issue width android api version < 16
Error: websocket error     at n.onError (socket.io.js:2)     at WebSocket.ws.onerror (socket.io.js:2)     at WebSocket.dispatchEvent (event-target.js:172)     at WebSocket.js:176     at RCTDeviceEventEmitter.emit (EventEmitter.js:181)     at MessageQueue.__callFunction (MessageQueue.js:244)     at MessageQueue.js:108     at guard (MessageQueue.js:46)     at MessageQueue.callFunctionReturnFlushedQueue (MessageQueue.js:107)     at addEventListener (RNDebuggerWorker.js:1)

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/websockets-not-working-for-android-versions-lower-than-n

Same issue how to solve that error
BackLay.js:61 Error: websocket error
    at r.onError (socket.io.js:2)
    at WebSocket.ws.onerror (socket.io.js:2)
    at WebSocket.dispatchEvent (event-target.js:172)
    at WebSocket.js:254
    at RCTDeviceEventEmitter.emit (EventEmitter.js:181)
    at MessageQueue.__callFunction (MessageQueue.js:306)
    at MessageQueue.js:108
    at MessageQueue.__guard (MessageQueue.js:269)
    at MessageQueue.callFunctionReturnFlushedQueue (MessageQueue.js:107)
    at debuggerWorker.js:72

Any updates on this?


Ubuntu: add base repos to Kolla container image earlier

Previously when building Kolla container images for Ubuntu it was common to get a dependency error during apt package installation, such as:
The following packages have unmet dependencies:
libc6-dev : Depends: libc6 (= 2.35-0ubuntu3.1) but 2.35-0ubuntu3.4 is to be installed
This is caused by the packages in the Ubuntu base image being newer than those in the release train repository snapshot.
This change fixes the issue by switching to the release train repositories earlier in the build process, before the first apt command runs.
It's possible that we may also need to pin the Ubuntu base container image, but that would be another thing to maintain, so let's try without for now since this approach is necessary and may be sufficient.

Tested and working for glance-api and nova-compute images.

I like that we're going to have this resolved for our images, but it's possibly worth noting somewhere (docs maybe) that the problem will remain for places that aren't using our images & bifrost provisioning. I know some places e.g. eidf, this has come up before but I don't know exactly how common it is

This is for the container images rather than the overcloud host images. It's only a problem if you use repository snapshots rather than upstream Ubuntu.


Mark the Changelog CI job as non-fatal

Unfortunately github shows the job like it passes ✅ instead of ❌.


feat(core): add exclude option to enqueueLinks

This is a PR to implement the functionality described in #1785. Closes #1785.

@vladfrangu Friendly bump 🙂


City view buildings order

Played the original civ till ~1 AD. Some notes:

when building new cities, you start off with either mud huts or tents. This logic seems not to change at least until 1 AD. My capital had mud huts, 2nd city had tents, 3rd+4th had mud huts, 5th had tents, ... Seems to be random.

brown tents seem to be the 1st stage, followed by gray tents. At around 1200 BC all my brown tents changed color to gray. No idea what triggered this.

between 1400 BC and 1200 BC all my cities started getting a smaller number of wood+clay houses, replacing the original tents & mud huts. As the city grows the number of these houses increases in the middle of the city.

the positioning of buildings in the city seems to randomly change as your city gets bigger.
E.g. City pop. 9:

City pop. 10:

No clear logic how this happens.

An example of stacking of shields etc. in the city main view as the city grows (perhaps it will be of some help):

I keep making screenshots of 5 cities (main view+city view) as they grow and as the game progresses. I'll try to resolve more game mechanics this way. Will post updates on this.

Wow, great job! This is really useful.

So, getting The republic transforms all your mud hits and tents into wood + clay houses.

Nice find, thank you!

I found out that the appearance of houses is changed every 2 advances you make. Here's the progress (left=before, right=after advance is researched):
7th advance:
Sadly no screenshots for comparison:
11th advance:

13th advance:

15th advance:

17th advance:

19th advance:

21st advance:

23rd advance:

25th advance:

27th advance:

29th advance:

31st advance:

33rd advance:

35th advance:

37th advance:

39th advance (industrialization. It also changes citizens' appearance):

41st advance:

43rd advance:

45th advance:

47th advance:

49th advance:

51th advance:

53rd advance:

55th advance:

56th advance was Automobile. It changed dirt roads to asphalt & people to cars:

57th advance:

59th advance:

61st advance:

63rd advance:

Sadly with 3 or so advances left the game crashed and I ragequit.
Also during 47th advance the picture of scientist (Sid) changed to laboratory setting. I'll have to check if something else triggers this.
As you can see even late into the game you can still have old houses, especially with large cities. This is why smaller cities have a more modern appearance.

Funny thing, I tried the same and the changes happened with the even number of advances instead of the uneven number of advances. The changes started at the 10th advance in my game. The first wooden home appeared with the 16th advance.
I wonder what the deciding factor is for that.
As for the laboratory scientist changing: That happens with the advance after you have discovered Electricity.

Also - you can start with either tents or (mud) huts. Perhaps this is random.

For now, until we figure out how it actually works in the original game, I will implement it like this:

Changes happen every 2 advances, starting at the eleventh advance
Stone/Wood huts start at 11, full at 25
Old houses start at 27
Industrial houses start at at 41
Skyscrapers start at 49, in cities with a minimum size of 9

Support for all kinds of houses is implemented, though experimental, and seems to be functioning pretty much as described above.

With all building types implemented, I will close this issue. The rest of the City View implementation will be done on issue #32.


[ASK] How can I handle 200 but error case?

Some part of my app use Google Spreadsheet.
And I implemented REST API there with Apps Script.
I think this is rare case but, google server gives me 200 with html code when it has server error.
So, what I am trying is getting the response with ApiResponse. And then try to convert the response to SpreadSheetResponseBody which is a data class when it's successful.
This is my concept.
    suspend fun updateDashboard(
        ssId: String,
        method: String,
        device: String,
        isTest: Boolean,
        request: KioskDashboardReqBody
    ): ApiResponse<SpreadSheetResponseBody>{
        val result = service.updateDashboard(ssId, method, device, isTest, request).onSuccess {
            try {
                val data = Gson().fromJson(this.data, SpreadSheetResponseBody::class.java)
                return ApiResponse.Success<SpreadSheetResponseBody>(data)
            }catch(e: Exception){
                return ApiResponse.Failure<SpreadSheetResponseBody>(e)
            }
        }
        return ApiResponse.Failure<SpreadSheetResponseBody>("Server Error")
    }

This code is invalid.
Since google gives me 200. I'd like to create ApiResponse instead in the Repository.
Is this possible to handling like this?

Hey @chanjungkim, For sure you can like the example below:
val data = Gson().fromJson(this.data, SpreadSheetResponseBody::class.java)
val response = Response.success(data)
return ApiResponse.of { response }

Hey @chanjungkim, For sure you can achieve it like the example below:
val data = Gson().fromJson(this.data, SpreadSheetResponseBody::class.java)
val response = Response.success(data)
return ApiResponse.of { response }

Thanks you.
For the error case, I did like this.
    suspend fun updateDashboard(
        ssId: String,
        method: String,
        device: String,
        isTest: Boolean,
        request: KioskDashboardReqBody
    ): ApiResponse<SpreadSheetResponseBody>{

        var result: Response<SpreadSheetResponseBody>? = null
        service.updateDashboard(ssId, method, device, isTest, request).onSuccess {
            result = try {
                val data = Gson().fromJson(this.data, SpreadSheetResponseBody::class.java)
                Response.success(data)
            }catch(e: Exception){
                Response.error(500, this.data.toResponseBody(null))
            }
        }

        return ApiResponse.of { result!! }
    }

Does it have any problem?

If the JSON format fits well, it's the correct way that you have given to me.  For the content type, you can specify its plain text type rather than HTML type by giving contentType = "text/plain".toMediaType() to the parameter. You can log if the JSON format is correct by throwing an exception deliberately inside the try block. You can also check out the JSON formats for error responses here.


PushNotificationIOS

PushNotificationIOS is super weird. Poor documentation, 4 different notification types, some method with promises, some with callbacks, ...
Hope I got it correct.

Any methods you've tested/not tested or that we should take a particular look at? I looked through and they seem alright to me. As you said, however, API's poorly documented and hard to figure out.
However, perhaps it's sufficient to leave this as a best effort. Since this module will be removed from core, perhaps there will be API changes or perhaps most users will simply go for an alternative.
I used wix/react-native-notifications (RNN) for iOS and react-native-firebase(FB) for Android until last fall (haven't maintained that app since) and that's only because the former appeared at a standstill since last year and GCM was being phased out on Android. Since then, they have introduced support for FCM and have been working on the iOS side as well. In light of that, I'm not sure using this module makes sense at the moment.

Thanks! I have not tested these methods. Let's leave this as best effort as you suggested.


Modify Response object

Thank you for creating such an amazing package! My goal is to retrieve only the intercepted json_data without the full HTML page content. Is there a way to set intercepted json_data as body to Response object, that received in the parse callback?
    def start_requests(self):
        url = "https://littlecaesars.com/en-us/order/pickup/stores/search/75215/"
        yield scrapy.Request(url, callback=self.parse, meta={
            "playwright": True,
            "playwright_page_methods": [
                PageMethod("route", "**/api/GetClosestStores", self.capture_request),
                PageMethod("wait_for_selector", "//button[contains(text(), 'Start your order')]"),
            ]
        })

    async def capture_request(self, route: Route):
        response = await route.fetch()
        json_data = await response.json()
        await route.fulfill(response=response, json=json_data)

    def parse(self, response: Response):
        pass

( Due to reCAPTCHA protection, using Playwright is essential here )

I'd suggest not to set a custom route, it's usually not a good idea given all the processing done in the handler. I think you can do something similar to this, i.e. intercepting the "response" event, extracting what you need, storing it somewhere temporarily, retrieving it in the main response callback and returning it there. You might need something like an asyncio.Event if you ran into synchronization issues e.g. if the callback runs before the response event handler (though I'm just thinking out loud, not sure that's even an actual possibility).


Readimage png + jpeg

Hi everyone,
This PR adds reading functions for PNG using libpng . It follows a discussion held in #1179.
My C++ being a bit rusty (it's been 8 years haha). Feel free to make feedback. It will be gratefully welcomed.
@fmassa I did my best to fix the formatting issue. Sorry about that I did not notice at the time.
Thanks

Hi,
I just noticed that CI is broken with the PR. I've looked at some of the logs. It seems to boil down to libpng not being found.
Do you have any resources on how to update circleci so that i can tell it to install libpng ?
Thanks

Hi @r-zenine ,
We would need to install libpng in a few places.
We currently use both CircleCI and TravisCI (for legacy reasons).
For TravisCI, you'd need to install it in https://github.com/pytorch/vision/blob/cec7ea727ab5905ee6d7039a4f1b980124684879/.travis.yml#L44
For CircleCI, it's a bit more involved because we have builds for Linux, OSX and Windows (which are all slightly different)
For CUDA CI: https://github.com/pytorch/vision/blob/cec7ea727ab5905ee6d7039a4f1b980124684879/.circleci/config.yml.in#L120
For conda: https://github.com/pytorch/vision/blob/cec7ea727ab5905ee6d7039a4f1b980124684879/packaging/torchvision/meta.yaml#L14 (maybe using libpng from anaconda? https://github.com/pytorch/vision/blob/cec7ea727ab5905ee6d7039a4f1b980124684879/packaging/torchvision/meta.yaml#L14)
For wheels:
Somewhere like https://github.com/pytorch/vision/blob/cec7ea727ab5905ee6d7039a4f1b980124684879/packaging/build_wheel.sh#L14 maybe?
And for windows I don't actually know much... but probably somewhere in the vs folders in https://github.com/pytorch/vision/tree/master/packaging
And once we make CI pass, we still need to see how we will be seeing the story around packaging for the binaries. But let's start with small steps.
Thanks a lot for the PR!

One more thing is that the setup.py script needs to be aware of the location of libpng so that it can compile everything properly, including the headers. This is something that should probably be handled in setup.py as well

Hi @fmassa,
I was planning to dig deep into PIL build system this week-end. I took a first quick look at it and it seemed quite complex. Therefore, I planned to do it this week-end when I have more time.
I will take care of the CI in the meantime.
Could you please take a look at the code? Or do you prefer to wait until everything is done ? your call !
Have a good day.

@fmassa Thanks for the review.
I'll try to fix everything in the coming days. I'll message you when everything is updated.
Have a good evening !

Hi @r-zenine
Sounds good, looking forward to the updated version!

Codecov Report

Merging #1632 into master will decrease coverage by 0.02%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##           master    #1632      +/-   ##
==========================================
- Coverage   65.99%   65.96%   -0.03%
==========================================
  Files          92       93       +1
  Lines        7328     7346      +18
  Branches     1107     1111       +4
==========================================
+ Hits         4836     4846      +10
- Misses       2176     2180       +4
- Partials      316      320       +4

Impacted Files
Coverage Δ

torchvision/io/image.py
55.55% <55.55%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 598b61d...9d04ed8. Read the comment docs.

Hi @fmassa,
I managed to fix one part of the CI ( Travis, linux_wheel, macos_conda ). I spent a day trying various tricks without success for the remaining pipelines. I have no experience with windows nor with anaconda packaging. I might not be able to do more. Can anyone of the team help?
Also, I looked into how we could ship the libpng with the package. My understanding is that adding this line below will add a -lpng flag when linking vision C++ code. I should be enough to build vision from source if libpng is installed in a standard location ( At least for Linux).
https://github.com/r-zenine/vision/blob/fd980232b76216364a5fc5d846be073dee907b59/setup.py#L146
The questions that remain are :

Do we support non-standard installs of libpng via build options on Linux?
How does it work for macOS and Windows?

If extra care isn't taken to disable GCC mulitlib / multiarch and standard location OS headers, we can again run into the old linking problem when building from source: (my old investigation here:  https://github.com/torch/image/issues/171#issuecomment-255909537 )

Hi @vadimkantorov,
My understanding is that cmake allows configuring a custom location for libraries.
It seems to me that it is the same as when we want to build torchvision from source against a specific build of libtorch. I the later situation, we can configure a custom libtorch location via an environment variable Torch_DIR. I think it's possible to do the same with libpng. If I am not mistaken the variable should be PNG_DIR.
Would this strategy address the issue you raise? Am I missing something?
Thanks

@r-zenine In that old investigation there were two problems: 1) CMake passing library filenames instead of full paths even if a custom library location was specified (maybe this is no longer relevant), 2) at linking time GCC preferring system libraries over custom-built ones (including conda ones) because of multilib / multiarch. Don't remember how binary loading was mixing with this.
Differently from many PyTorch dependencies, libjpg/libpng are often present in system OS locations.
Concretely, I'm proposing having some test to confirm that build and OS binary loader picks the right libraries in presence of multilib/multiarch system library versions.
I also brought this up in https://github.com/pytorch/pytorch/issues/13188

@r-zenine I currently don't have a good solution to the problem, and I'll only have more time to look into it in January unfortunately :-/

Hi @fmassa,
Thanks already for your time. Here is what I propose:
I can start working on the jpeg code in the meantime.
When I am done, we can have a look at the CI pipeline for both PR's  and merge all the work at the same time.
Thoughts?

Hi @fmassa,
I just added the jpeg features to the same PR. I also tried to fix the conda pipeline for linux without success so far.
And happy new year everyone.

Hi @fmassa ,
Any update ?
Thanks

Hi @r-zenine
Happy new year!
Sorry for the delay in replying, after holidays I busy with a number of things. I'll have a look at your PR and get back to you

I spent some time today looking into this, but I currently don't have a good solution for ensuring that users are able to compile torchvision with libpng and libjpeg.
I'll keep looking at it early next week (I'm packed with a number of things to finish for now), sorry for not having the answer yet!
One thing I noticed about the JPEG implementation is that you are seem to be using libjpeg-turbo -specific API, which means that if the user has only libjpeg (but not libjpeg-turbo) installed the code wouldn't compile. IIRC, libjpeg-turbo comes with an API that also mimics libjpeg, so I would think that it would be preferable as it would enable more users to install torchvision right away.
But no need to modify anything for now, as we are still figuring out how to ensure that we ship with the correct libraries.

Hello @r-zenine,
Francisco is getting flooded with other work right now so he asked me to look into this. With regards to your CI issues, could I ask you to try creating a submodule folder "third_party" similar to pytorch. We can then statically build this code into our binaries. Since these two libraries are so widely used I imagine we'll get good support for various environments out of the box. This will solve our deployment and CI issues (for now) until we come up with a better solution than submodules.
Thanks,
Christian

Hi @cpuhrsch ,
Sorry for the late answer. It's been three intense days on my side.
I will take a look this week-end and try to do that and ping you when I am done.
Thanks

Hi @cpuhrsch,
I looked at it on the week-end, here are the updates :

I added zlib, libjpeg-turbo and libpng as submodules.
I updated the setup.py to build these before building the C++ extension.

Tests on the linux_conda pipeline are failing, I need to investigate why.
simd support on macOs is deactivated because of an obscure nasm failure. I need to take a look later.
I'll keep looking into it tomorrow morning.

Great! Thanks @r-zenine!

HI,
Another update,
Still a work in progress. However, we got macos_wheel working.
conda based setups are not working, tests are failing because of a failure to load the C extension. It might be happening because we are not using the same compiler for everything. Not sure how to fix it.
I'll investigate this at the end.
Windows pipeline are still running. Crossing fingers.
Going to work.

Hi There,
Making progress here! Even though I have to admit, I am way out of my comfort zone.
The linux_conda pipeline are timing out. Any idea how to fix it?
I have tried several times to fix the windows pipeline without success. The feedback loop is too slow ( having to push the code and wait for the CI to run). Do you have any tips to make the feedback loop go faster? I guess not having a windows machine ready and configured is not helping either.
The code is not clean by any means. There are tons of commits, I'll clean things up when we get everything up and running and rebase and clean the commits.
I would appreciate some help with the windows pipeline if anyone can take a look into it.
Big thanks guys,
Going to work

Thanks for working on this @r-zenine! Believe me, I understand how frustrating it is to add a dependency. Here are some suggestions on how to break this problem into smaller pieces given that this is becoming too complex.
You could split this into two PRs: png and jpeg separately. That way we can reduce the error surface. In particular if you can split it such that only one dependency needs to be introduced.
On Windows: I'd focus on fixing all the other cases first. In fact, there is nothing wrong with selectively disabling the support for these ops for Windows and Mac first. You can use @unittest.skipIf('win' in sys.platform) to disable the tests on Windows. We can then later on add support for this.
On the topic of speeding up the feedback loop: Ninja was supposed to help with build times but until we have support for a flag that limits the threads based on the environment we can't enable it just yet. Windows is always its own story and in my experience has terrible feedback loop times. We're working on fixing this, but a good way to avoid it for now is to add support for it in a separate PR.
So, in summary, I'd create a PR that adds jpeg reader support for linux only for now. Then a PR that does the same for png. Then a PR for Mac support for jpeg and png each. Then a PR for Windows support for jpeg and png each.
I hope this helps.

Hi @cpuhrsch ,
Let's proceed that way. I'll do PR's for:

png for linux
jpeg for linux
png for mac
jpeg for mac
png for windows
jpeg for windows

In the meantime, if we could find help for the windows pipeline, I would greatly appreciate it.
Thanks

@r-zenine - sounds great! I'll drum up some help once we get to Windows.

@cpuhrsch,
Awesome, Thanks !

Hi @cpuhrsch , @fmassa ,
I have a question for the jpeg integration. The code for libjpeg-turbo is working correctly and the packaging is ok. However we do not produce exactly the same result as pillow. After doing some google. It's seems that it's a normal behavior for Jpeg.
See here for example : https://github.com/python-pillow/Pillow/issues/3833
Therefore, it makes it difficult to write a non regression test agaist the stock pillow.
We have two options here :
Either we build a version of pillow with turbojpeg support enabled in the pipeline and we test against it. Or, I use stock libjpeg for the code and we drop turbojpeg.
Thoughts?

Subsumed by #2388 and #2382


Modulise

Split to modules and slight rewrite.

It is a little longer but most of it is due to comments. Also I have included the separate files to give you an idea of what the project would look like if it weren't in one file.


Tconnectsync-heroku uploading treatments with incorrect times

For whatever reason, all of my pump data is submitted to nightscout -1 hour or so. So a bolus at 9:06 this morning shows up in nightscout as 8:06. Have verified that timezones are set correctly in the tconnectsync config and nightscout
Also checked in tconnect, the pump events are shown with the correct times.

Try setting the TIMEZONE_NAME environment variable to your proper timezone. By default, it's set to America/New_York -- you can find all of the valid names here: https://en.wikipedia.org/wiki/List_of_tz_database_time_zones

Had verified it was set to America/Chicago

If you change your timezone env option to America/New_York, does the data that's posted to nightscout adjust by one hour? Maybe nightscout isn't configured for your timezone


fix importing of resources

For some reason labelImg does not work for me out of the box when it's installed.  It complains about not finding the resources module. This PR contains a change that made it work for me.

I don't think this PR is necessary, as we haven't had very many people experience this issue. I'll definitely reopen and reconsider if that changes.
Thanks,
-Vlad


Unclear what the format of the json string is supposed to be.

It is unclear what the format of the json string is supposed to be. Reading a json file into a string with File.ReadAllText(paramPath) does not work, and the documentation is not very helpful.

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

ID: 2024d768-b80d-f542-50bb-3ffb578d8ba7
Version Independent ID: 680045a8-d336-255a-e873-bd82baeb64e4
Content: IWithParameters.WithParameters Method (Microsoft.Azure.Management.ResourceManager.Fluent.Deployment.Update) - Azure for .NET Developers
Content Source: xml/Microsoft.Azure.Management.ResourceManager.Fluent.Deployment.Update/IWithParameters.xml
Service: azure-resource-manager
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@xboxeer please help to follow up this docs issue


Switch to CircleCI

This changeset:

Configures CircleCI to compile the example document with pdflatex, lualatex, and xelatex.
Separates the core TeXLive package list and package dependencies. packages.txt now indicates the complete set of extra packages users need to use this package.

Closes: #169

As you can see below, CircleCI reports separate statuses for each build:

This is pretty useful because we already include engine-specific code (translations), and some of our contributors have targeted specific engines.

Then does this replace travis-ci?

Well done. This is a great addition, allowing us to ensure we work on the major TeX engines.

Once you merge this in, I will add a new 0.7.1 version. That way we can branch down the road with this in it for any fixes to 0.7.0.

Once you merge this in, I will add a new 0.7.1 version. That way we can branch down the road with this in it for any fixes to 0.7.0.

Well, we don't need to create a patch release for a project-level change like this. It doesn't impact the user. I've added it to the changelog, however.
We also don't need to create a 0.7 release branch for future 0.7.x bug fixes. That branching model i useful if you plan on supporting two product streams at once (e.g., Postgres 9 and 10). But it's unnecessarily complicated for a small project like this.
To keep LaTeX3 code out of 0.7.x releases, we can create a latex3 integration branch, merge all the PRs into that one, and then merge that into master when it's ready.
That way we can fix the 0.7 code (like #179) while we're still working on LaTeX3. We can also keep the LaTeX3 branch up-to-date with changes like this PR by merging master into it frequently. This approach is also easier for testing: we just have to check out latex3 locally to get the latest LaTeX3 code.


Video and Audio are out of sync when recording a file using in_recorder_factory

Hello,
first of all, thank you very much for this awesome component! It is helping me a lot.
I am currently trying to record the input video and audio via in_recorder_factory. With the code below, I face an issue where the video is fine, but audio is played in very short bursts followed by sequences of silence, while in the actual recording, the audio signal should be played continuously (in sync to the video).
I have noticed that with WebRtCMode.SENDRECV, everything works fine. However, with SENDONLY, I can't find a solution to solve this issue. For my purpose, it would be highly preferable to use SENDONLY mode in order to reduce the traffic coming back from the server where the app is hosted.
Thank you!
Edit: Also the video is not recording properly after a while. E.g. after 5min of recording, I get a file with a length of 13:19:32.
def recorder_factory():
       return MediaRecorder("test.mp4")

streamer = webrtc_streamer(
        key="rec_streamer",
        mode=WebRtcMode.SENDONLY,
        rtc_configuration=RTC_CONFIGURATION,
        media_stream_constraints={
            "video": True,
            "audio": True},
        in_recorder_factory=recorder_factory,
    )

I observed that the frame dimensions and aspect ratio received from the camera are varying over time. Maybe this leads to problems during recording?
However, the frame dimensions still vary if I set constraints like below. I can't set a fixed frame size larger than 640x480.
media_stream_constraints={
            "video": {
                "width": {"min": 1000, "ideal": 1920, "max": 1920},
                "height": {"min": 360, "ideal": 1080},
                "frameRate": {"exact": 6}},
                "aspectRatio": {"exact": 1.777778},
            "audio": True},

Sorry for my so late reply, and thank you for your detailed bug report.

I have noticed that with WebRtCMode.SENDRECV, everything works fine. However, with SENDONLY, I can't find a solution to solve this issue.

With this great hint, I could find a difference between the implementation of SENDRECV and SENDONLY, and added a fix on the SENDONLY internals with #1089
Please be patient for a while until it will be released.

Should be fixed with v0.44.0


compose exec command is missing

First, thank you for all of your works on finch.
What is the problem you're trying to solve?.
docker has compose exec command, but finch doesn't.
https://docs.docker.com/engine/reference/commandline/compose_exec/
Describe the feature you'd like
finch compose exec performs the same action as docker compose exec

finch uses nerdctl as upstream. Currently, compose exec is not supported by nerdctl
Here is the list of commands supported by nerdctl
https://github.com/containerd/nerdctl#command-reference

@KoichiKiyokawa, Finch 0.2.0 uses nerdctl 1.1 which supports compose exec command.
example:
finch compose exec service_name pwd

@monirul Thank you so much!  It helps me a lot!


TUG.TeXLive - No package found matching input criteria

Brief description of your issue

Installation of TUG.TeXLive not possible although it was added as winget package.
Steps to reproduce
Installation according to this one:
❯ winget install -e --id TUG.TeXLive

Expected behavior

Package installed successfully as it was added in #8045 (and the manifest file is here).
Actual behavior

❯ winget install -e --id TUG.TeXLive
No package found matching input criteria.

Searching also doesn't find it:
❯ winget search --id TeXLive
No package found matching input criteria.

Environment
❯ winget --info
Windows Package Manager v1.0.11692
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19043.1165
Package: Microsoft.DesktopAppInstaller v1.12.11692.0

It was removed due to https://github.com/microsoft/winget-pkgs/pull/19989#issue-684631000

@quhxl my question is a bit off-topic, but is https://winstall.app also outdated?

The way I observe it is up-to-date. At least a version from a recent PR is shown there as an example. (#27513 -> https://winstall.app/apps/e2eSoft.iVCam)

Feel free to close this issue yourself. 😊


upnpapi: remove memset

Can be replaced by the shorter {}.
Signed-off-by: Rosen Penev<EMAIL_ADDRESS>

Ran through clang-format to try to make CI happy.

yeah...no way.

I hope I did not mess things, I had to do some stuff manually, please check.


Remove with bundle filter

Objective
Fixes #2620
Solution
Remove WithBundle filter and temporarily remove example for query_bundle.

It's not very happy I deleted the example :(
Is there something you have to do to get it to delete it beyond just deleting the file?

You also need to remove it:
https://github.com/bevyengine/bevy/blob/90586a4c461cde86b8a0e08f0770894e7cf30862/Cargo.toml#L321-L323
https://github.com/bevyengine/bevy/blob/90586a4c461cde86b8a0e08f0770894e7cf30862/examples/README.md#L165

unrelated ci failure

So is there anything I need to do for this? I'm still kinda confused as to what I do at this point, if anything.

nothing to do for now, you can wait for reviews/comments 🙂

Looks good to me!

bors r+


raspiraw "--expus" does not work for ov5647 (it does work for imx219)

Doing raspiraw captures with imx219 for "--expus 1000" and "--expus 2000" gives different bright frames after processing with dcraw.
Doing the same with ov5647 gives identical frames, no difference in brightness.
I tested that this bug exists on this repo as well.
This issues is different to the incorrect calculation from issue 20.

See #18.
Getting a calculation that can be parameterised and is correct for both sensors is trickier than it looks.

I used your latest raspiraw and setting shutter does not work at all.
I cannot find a repo of gustav-engdahl but only see his commits on your branch.
But the pull request is anot integrated into your branch yet.
How can I test whether the three commits in that pull request make "-e" work on your repo?
(without copying over line by line by hand)

Hi,
You should be able to build my repo at:
https://github.com/GustavEngdahl/raspiraw/commits/master
It's master has the three patches needed to get the VTS/Exposure/gain
values correctly written. The patches are not chip specific in any way, but
I've only verified on OV5647.
Regards,
Gustav
Den mån 4 mars 2019 kl 10:56 skrev Hermann Stamm-Wilbrandt <
<EMAIL_ADDRESS>
I used your latest raspiraw and setting shutter does not work at all.
I cannot find a repo of gustav-engdahl but only see his commits on your
branch.
But the pull request is anot integrated into your branch yet.
How can I test whether the three commits in that pull request make "-e"
work on your repo?
(without copying over line by line by hand)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/6by9/raspiraw/issues/25#issuecomment-469190222, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ap0oSRL7x-JmdJqKox_oplSyhG4jGEu8ks5vTO3UgaJpZM4bbK1J
.

Thank you, I did build your repo and it fixes the issue I reported here, closing this issue as duplicate now.
These are the (different now) frames with different --expus settings.
I did not look in detail into your changes, but "--expus" seems to have worked because for imx219 because exposure bits are a multiple of 8 whereas for ov5647 they are not:
      .exposure_reg =         0x015A,
      .exposure_reg_num_bits = 16,

   .exposure_reg =         0x3500,
   .exposure_reg_num_bits = 20,

--expus 4000
--expus 2000
--expus 1000
--expus 500

Good to hear.
Can you verify that my fixes doesn't break imx219? If nothing breaks,
perhaps 6by9 could bring in #18 to master?
Regards,
Gustav
Den mån 4 mars 2019 kl 12:52 skrev Hermann Stamm-Wilbrandt <
<EMAIL_ADDRESS>
Thank you, I did build your repo and it fixes the issue I reported here,
closing this issue as duplicate now.
These are the (different now) frames with different --expus settings.
I did not look in detail into your changes, but "--expus" seems to have
worked because for imx219 because exposure bits are a multiple of 8 whereas
for ov5647 they are not:
  .exposure_reg =         0x015A,
  .exposure_reg_num_bits = 16,

.exposure_reg =         0x3500,
.exposure_reg_num_bits = 20,
--expus 4000
https://stamm-wilbrandt.de/en/forum/72kmh/out.0017.ppm.d.png
--expus 2000
https://stamm-wilbrandt.de/en/forum/72kmh/out.0016.ppm.d.png
--expus 1000
https://stamm-wilbrandt.de/en/forum/72kmh/out.0015.ppm.d.png
--expus 500 https://stamm-wilbrandt.de/en/forum/72kmh/out.0014.ppm.d.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/6by9/raspiraw/issues/25#issuecomment-469225638, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ap0oSbZlt8z2L3ualXWEOGJ_0x1JD14Mks5vTQkJgaJpZM4bbK1J
.


Only one track, playing

A normal piano midi has 2 tracks. One left and one right. When I trained it on some bach piano pieces, the loss went to around 0.1 and I ran the predict.py with the weights loaded. The results did sound like bach, but its only a mono track that's playing. I fed the network with 2 tracks for the piano. Is there something I didn't do?

Hi,
The network as it is on the master branch only supports a mono track. If you'd like to support 2 tracks you can try using a multi-input multi-output network where the inputs/outputs are notes/duration/offset. That should allow you to have as many tracks as you want.
Additionally, you can try using something like MuseGAN to create multi-track files.

But it doesn’t make sense. It is obvious that a piano has two tracks and this repo’s name is self explanatory. Anyway, which branch has multi track support??

It does make sense. This is a single-track implementation. It only produces one note/chord for each offset so it would be difficult for it to be multi-track.
The multi-track support isn't ready. But it uses the multi-in/out method I detailed above.

@binary-person you can merge the midi tracks into one single track using this:
https://github.com/jordan-bird/Keras-LSTM-Music-Generator/blob/master/convertmidis.py
Then use the new midis as your input to the network
The other way, like @Skuldur said is to have multiple inputs with the functional API but given that many midis have >2 tracks then it would eat resources and likely OOM

The merging seems to provide a good effect on the result. I agree that it would eat resources. I think that this repo should remain as is for the lessened complexity and easier understanding for future peeps.

That's exactly the point of this repo. :)
I've made a more complex version of this project that I've been debating whether to merge into the repo, but I feel like this should remain simple for people that are just getting their feet wet in music generation. I do need to do some refactoring in this repo when I find the time, though 😅

@Skuldur I would love to see your multi-track implementation. Do you have the repo link?


"Uncaught RangeError: Maximum call stack size exceeded"

When trying out the scratchpad, the app breaks and I see the following error:
Navigated to http://mesh-spreadsheet.com/try-mesh.html
try-mesh.html:112 Grid's cells_obj: []
try-mesh.html:333 RECEIVED MESSAGE FROM SHEET: {source: "react-devtools-detector", reactBuildType: "production"}
try-mesh.html:339 Sending results to app... {values: {…}}
try-mesh.html:333 RECEIVED MESSAGE FROM SHEET: setImmediate$0.9796373051839196$1
try-mesh.html:339 Sending results to app... {values: {…}}
try-mesh.html:333 RECEIVED MESSAGE FROM SHEET: {source: "react-devtools-content-script", hello: true}
try-mesh.html:339 Sending results to app... {values: {…}}
try-mesh.html:333 RECEIVED MESSAGE FROM SHEET: {source: "react-devtools-content-script", hello: true}
try-mesh.html:339 Sending results to app... {values: {…}}
app.js:23158 ACTION RECEIVED: {type: ""}
app.js:23159 MATCHING ROW: undefined
app.js:23162 MOVING TO NEW STATE: {mode: "READY", cells: {…}, selected_cell_loc: Array(2), code_editor: {…}, filepath: null, …}
try-mesh.html:333 RECEIVED MESSAGE FROM SHEET: {"data":null,"type":"buffer_3pc_done"}
try-mesh.html:339 Sending results to app... {values: {…}}
try-mesh.html:288 Parent got message from app!
VM274:63 Uncaught RangeError: Maximum call stack size exceeded
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:22)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
g._uncache @ VM274:63
postMessage (async)
onmessage @ try-mesh.html:345
postMessage (async)
send @ backend.js:6
(anonymous) @ backend.js:30
setTimeout (async)
value @ backend.js:30
t @ backend.js:6
e @ backend.js:6
postMessage (async)
a @ contentScript.js:1
(anonymous) @ contentScript.js:1
setInterval (async)
112 @ contentScript.js:1
n @ contentScript.js:1
(anonymous) @ contentScript.js:1
(anonymous) @ contentScript.js:1
Show 170 more frames
try-mesh.html:1 MESSAGE RECEIVED BY app : {values: {…}}
VM274:63 Uncaught RangeError: Maximum call stack size exceeded
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:22)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
g._uncache @ VM274:63
postMessage (async)
onmessage @ try-mesh.html:345
postMessage (async)
send @ backend.js:6
(anonymous) @ backend.js:30
setTimeout (async)
value @ backend.js:30
t @ backend.js:6
e @ backend.js:6
postMessage (async)
a @ contentScript.js:1
(anonymous) @ contentScript.js:1
setInterval (async)
112 @ contentScript.js:1
n @ contentScript.js:1
(anonymous) @ contentScript.js:1
(anonymous) @ contentScript.js:1
Show 170 more frames
try-mesh.html:1 MESSAGE RECEIVED BY app : {values: {…}}
VM274:63 Uncaught RangeError: Maximum call stack size exceeded
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:22)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
g._uncache @ VM274:63
postMessage (async)
onmessage @ try-mesh.html:345
postMessage (async)
send @ backend.js:6
(anonymous) @ backend.js:30
setTimeout (async)
value @ backend.js:30
t @ backend.js:6
e @ backend.js:6
postMessage (async)
a @ contentScript.js:1
(anonymous) @ contentScript.js:1
setInterval (async)
112 @ contentScript.js:1
n @ contentScript.js:1
(anonymous) @ contentScript.js:1
(anonymous) @ contentScript.js:1
Show 170 more frames
try-mesh.html:1 MESSAGE RECEIVED BY app : {values: {…}}
VM274:63 Uncaught RangeError: Maximum call stack size exceeded
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:22)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
g._uncache @ VM274:63
postMessage (async)
onmessage @ try-mesh.html:345
postMessage (async)
Show 281 more frames
try-mesh.html:1 MESSAGE RECEIVED BY app : {values: {…}}
VM274:63 Uncaught RangeError: Maximum call stack size exceeded
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:22)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
g._uncache @ VM274:63
postMessage (async)
onmessage @ try-mesh.html:345
postMessage (async)
a @ contentScript.js:1
(anonymous) @ contentScript.js:1
setInterval (async)
112 @ contentScript.js:1
n @ contentScript.js:1
(anonymous) @ contentScript.js:1
(anonymous) @ contentScript.js:1
Show 170 more frames
try-mesh.html:333 RECEIVED MESSAGE FROM SHEET: {source: "react-devtools-content-script", payload: {…}}
try-mesh.html:339 Sending results to app... {values: {…}}
try-mesh.html:1 MESSAGE RECEIVED BY app : {values: {…}}
VM274:63 Uncaught RangeError: Maximum call stack size exceeded
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:22)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
    at g._uncache (eval at <anonymous> (app.js:23207), <anonymous>:63:108)
    at Set.forEach (<anonymous>)
g._uncache @ VM274:63
postMessage (async)
onmessage @ try-mesh.html:345
postMessage (async)
(anonymous) @ contentScript.js:1
Show 170 more frames

Thanks! Can you recall the steps that triggered the issue? The "Maximum call stack size exceeded" issue can happen because the calc chain is too deep (an unfortunate side-effect of the way Mesh does dependency resolution at the moment), but it can also happen due to occasional actions in the UI or unusual input in the formula bar, and I can't tell from the log above exactly what you were doing.

This happens on load, no UI interaction. I’ve tried the sandbox a couple times over the past few weeks and each time I see this. I’m using Chrome on macOS.

Very strange. It works OK in iOS on both Safari and Chrome. I'll find a Mac
to test on.
On Sun, 13 Oct 2019 at 03:26, Grant Nestor<EMAIL_ADDRESS>wrote:

This happens on load, no UI interaction. I’ve tried the sandbox a couple
times over the past few weeks and each time I see this. I’m using Chrome on
macOS.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chrispsn/mesh/issues/70?email_source=notifications&email_token=AAF5WMQAPHG2OOW33UMEY53QOH3DTA5CNFSM4JABHBG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBCDDSQ#issuecomment-541340106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF5WMTCFHREHP4C6Y4P6CTQOH3DTANCNFSM4JABHBGQ
.

Confirmed working in Safari 12.1.2 on MacOS (someone else's computer -
unfortunately they didn't have Chrome).
You may need to hold off on doing anything in the UI until the lower-left
state shows 'READY' - it takes a second or two to download the backend,
app.js.
Otherwise, maybe try Safari as well?
On Sun, 13 Oct 2019 at 09:56, Chris Pearson<EMAIL_ADDRESS>wrote:

Very strange. It works OK in iOS on both Safari and Chrome. I'll find a
Mac to test on.
On Sun, 13 Oct 2019 at 03:26, Grant Nestor<EMAIL_ADDRESS>wrote:

This happens on load, no UI interaction. I’ve tried the sandbox a couple
times over the past few weeks and each time I see this. I’m using Chrome on
macOS.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chrispsn/mesh/issues/70?email_source=notifications&email_token=AAF5WMQAPHG2OOW33UMEY53QOH3DTA5CNFSM4JABHBG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBCDDSQ#issuecomment-541340106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF5WMTCFHREHP4C6Y4P6CTQOH3DTANCNFSM4JABHBGQ
.

I tried on Safari and it worked and then tried again on Chrome and it's working now 👍

Thanks for letting me know!
On Mon, 14 Oct 2019 at 7:19 pm, Grant Nestor<EMAIL_ADDRESS>wrote:

Closed #70 https://github.com/chrispsn/mesh/issues/70.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chrispsn/mesh/issues/70?email_source=notifications&email_token=AAF5WMTW62EL5SZCMBC6XFDQOQTSNA5CNFSM4JABHBG2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUGAKZHQ#event-2709564574,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF5WMVXJTZZ5VTXSHGH7PDQOQTSNANCNFSM4JABHBGQ
.


Reframe the InCommon host cert options

I'd like the documentation to focus on use cases instead of tools. Plus some other misc focuses.
Once we get this PR approved and merged, I'll approve and merge: https://github.com/opensciencegrid/docs/pull/537

@jteheran can you review and merge this?

@jteheran alright, comments should be addressed. Just an FYI for the future, you can add "suggestions" to the PR so that the author can accept them directly via the GitHub interface (https://help.github.com/en/articles/commenting-on-a-pull-request#adding-line-comments-to-a-pull-request)

That's what I thought I did. I added line comments. Not sure what went wrong.

When you're adding an in-line comment, there's a button in the UI that turns it into a suggestion, which should add something that looks like the following to the text box:
```suggestion
<original text here>
```

Does the PR look good now? Can you merge it?


[BUG]  Chrome UI isn't overridden

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
I did import the necessary fast flag to override it, but it didn't seem to work, am I doing it wrong?

Can confirm, same problem here

Are you trying to enable it or disable it?

See https://github.com/pizzaboxer/bloxstrap/wiki/A-guide-to-FastFlags#escape-menu-version


Training landscape dataset

I am have trained landscape dataset for nearly 180000 steps still the samples i obtained as output is not as clear as pretrained model for landscape dataset. Can you please tell me how many steps was landscape model was originally trained?

I had the same problem. I trained landscape dataset 2.56e+04 steps and still not clear by one GPU A40. I could not found the training detail in the essay. It`s trouble me.

I am have trained landscape dataset for nearly 180000 steps still the samples i obtained as output is not as clear as pretrained model for landscape dataset. Can you please tell me how many steps was landscape model was originally trained?

Hi! I found the answer in https://github.com/researchmm/MM-Diffusion/issues/3#issue-1658387901

@SoloMannn Did you eventually get meaningful results on the landscape dataset?


Make converting positions to points easier

I often find myself doing something like:
let p1: Point<f32> = position.position.translation.vector.into();
if I want to get the distance between entities. This gets verbose quickly, especially with multiple entities.
I wonder if it makes sense to add the conversions higher up in the chain:
let p1: Point<f32> = position.position.translation.into();
Most straight-forward, since I'm working directly on the translation.
let p1: Point<f32> = position.position.into();
Unless there's some other, more meaningful point that can be derived from the isometry, this would be preferable since distance between positions seems common enough.
let p1: Point<f32> = position.into();
In general, if someone isn't interested in the next position, I'd argue that current should be the default assumption without having to be explicit about it every time. This makes calculating distances between two entities a whole lot cleaner:
let distance = na::distance(position1.into(), position2.into());
Thanks.

It should be obvious what all these things return given that most have type annotations--often a requirement when using into().
I know I can make an extension trait, but I don't think I should have to. Rapier already has lots of conversions, so running up against the lack of them here is a bit jarring.

Adding (to nalgebra) a conversion from Translation to Point sounds legitimate (especially since we already have a Point -> Translation conversion). Anything higher up the  hierarchy start being too ambiguous since some information (the rotation part of the isometry) is lost in the process.

if explicitly requested to do so

Writing isometry.translation.into()  is the explicit way to request only the translational part. Writing isometry.into() to extract only the translational part on the other hand doesn’t look explicit.


[BACKEND] Fix unrealized conversion for fp32 dot

This branch has fixed the type conversion problem for the DotOperand Layout by unifying the input types with vec_ty. Enforced by the rule, a single fp32 is represented by vec_ty(type::f32Ty(ctx), 1).
This branch has also proposed a new triton op---tl.full which mimics torch.full for two purposes.

Simplify the initialization of tensors of constant values. For example, we had to use (tl.zeros([...], tl.float16) + 2.0).to(tl.float16) or tl.broadcast_to(1.0, [...]).to(tl.float16) to initialize a float16 tensor with value 2.0. Directly use 2.0 doesn't serve the purpose because 2.0 is by default a float32.

Direct initialization of a DotOperand of float16 for cases in which an operand of dot is a constant number (e.g., tl.dot(2.0, b)). Without tl.full, the TritonGPU Dialect will generate a FPTrunc op between initialization and dot. Thus, we will first initialize the constant with the Blocked layout and then convert it to the DotOperand layout, causing non-trivial slowdown. See https://github.com/Jokeren/triton-samples/blob/main/full.py.
Of course such a usage of dot is rare and can be replaced by tl.sum in many cases.

@Superjomn I feel like the dot helper code (e.g., conversion helper, mma loader ...) can be cleaned up a bit, like inheriting v1 and v2 functions from a common parent class. Do you have any comments on it?

I feel like the dot helper code (e.g., conversion helper, mma loader ...) can be cleaned up a bit, like inheriting v1 and v2 functions from a common parent class. Do you have any comments on it?

Yeah, totally agree.  It could be a good time to do some code cleaning after the mmav1 patch is merged.


Allow for dynamically adding, removing and clearing datasets.

Allow for dynamically adding, removing and clearing datasets on Line charts.
Allow for empty charts.

We've recently announced a feature complete status for v1.x and will only be accepting bug fixes for v1.x. Be sure to take a look at v2.0-dev and it's new features that are either finished or currently under development.  If you see your feature is still not implemented, feel free to submit another PR to the v2.0-dev (or later) branch for your feature.


Windows toggl desktop freezes randomly when a task is created or changed

Hi Guys,
Since the last update where you add a background on task edit dialog, each time I change a time or create a new task on toggl desktop for windows randomly the app got frezzed by 10 or 15 seconds and then it continue working. This is really annoing some time, do you think there is a way to solve this?
In order to discard a performance issue. My computer has a i7, 16GB of Ram and all other programs are running smothly.
Thanks,
Emmanuel

I will be using online version for now. Windows version is freezing almost at any task edit. :(
Thank you in advanced for any help you can give me to fix this.
Regards,
Emmanuel

Hey Emmanuel,
Thanks for reporting the problem!
We are aware of the issue and already looking into it. We should have a new build with some improvements out soon. Please keep an eye on this issue for more:
#1483
Closing as duplicate of the same.

Thanks! :+1:


TINKERPOP-1225: Do a "rolling reduce" for GroupXXXStep in OLAP.

https://issues.apache.org/jira/browse/TINKERPOP-1225
We now can do mid-barrier reductions in with group() on OLAP. This is huge as this means that if you have an reducer in your by()-valueTraversal, the stream is constantly reduced to limit memory consumption. Its more expensive in terms of time (not by much) for small data, but for large data, no worries about OME with group() (both both OLTP and OLAP).
CHANGELOG
* `GroupStep` and `GroupSideEffectStep` make use of mid-traversal reducers to limit memory consumption in OLAP.

VOTE +1

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache TinkerPop .................................. SUCCESS [4.350s]
[INFO] Apache TinkerPop :: Gremlin Shaded ................ SUCCESS [2.302s]
[INFO] Apache TinkerPop :: Gremlin Core .................. SUCCESS [40.323s]
[INFO] Apache TinkerPop :: Gremlin Test .................. SUCCESS [12.404s]
[INFO] Apache TinkerPop :: Gremlin Groovy ................ SUCCESS [42.373s]
[INFO] Apache TinkerPop :: Gremlin Groovy Test ........... SUCCESS [6.747s]
[INFO] Apache TinkerPop :: TinkerGraph Gremlin ........... SUCCESS [3:24.962s]
[INFO] Apache TinkerPop :: Gremlin Benchmark ............. SUCCESS [10.390s]
[INFO] Apache TinkerPop :: Hadoop Gremlin ................ SUCCESS [11:27.951s]
[INFO] Apache TinkerPop :: Spark Gremlin ................. SUCCESS [7:01.345s]
[INFO] Apache TinkerPop :: Giraph Gremlin ................ SUCCESS [2:41:47.692s]
[INFO] Apache TinkerPop :: Neo4j Gremlin ................. SUCCESS [17:53.279s]
[INFO] Apache TinkerPop :: Gremlin Driver ................ SUCCESS [10.358s]
[INFO] Apache TinkerPop :: Gremlin Server ................ SUCCESS [12:52.175s]
[INFO] Apache TinkerPop :: Gremlin Console ............... SUCCESS [1:25.610s]
[INFO] Apache TinkerPop :: Gremlin Archetype ............. SUCCESS [0.086s]
[INFO] Apache TinkerPop :: Archetype - TinkerGraph ....... SUCCESS [4.702s]
[INFO] Apache TinkerPop :: Archetype - Server ............ SUCCESS [10.319s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 3:38:17.858s
[INFO] Finished at: Wed Mar 30 22:57:45 MDT 2016
[INFO] Final Memory: 99M/765M
[INFO] ------------------------------------------------------------------------

VOTE +1

VOTE: +1

VOTE: +1


Feature request: ecosystem index funds

e.g. Polygon ecosystem index fund, Harmony ecosystem etc.

I like that idea 🔥
I did a quick search and https://coinmarketcap.com maintains categories for collections of coins/tokens. And the polygon ecosystem and harmony ecosystem are included in those.
The listings/latest api call doesn't allow for a category to be passed. However, it contains a field with a category, which we can use to filter the result.
Alternatively, we could query the category api endpoint and use those results. As long as the data can be normalized we should be good.
The API call to coinmarketcap is made here:
https://github.com/doodzik/crypto-index-fund/blob/main/src/marketdata_api.js#L9


Support  creating VSIX with WinForms Toolbox Controls targeting .NET 4.0

When following steps from this guide:
https://github.com/MicrosoftDocs/visualstudio-docs/blob/master/docs/extensibility/creating-a-windows-forms-toolbox-control.md I was able to create VSIX with user control that targets .NET 4.5 (this framework version is default when creating new project).
As soon as I set target framework to 4.0 I can't add new controls using Add >New Item > Extensibility.

How can I create VSIX that will contain Controls that target .NET 4.0? IS this possible in Visual Studio 2017?
I've created similar issue here: https://github.com/Microsoft/VSSDK-Extensibility-Samples/issues/127

@gregvanl any news about this?
I've tried creating ProvideToolboxControl  manually and adding it to my UserControll class but that didn't work.
Ideally after changing target to .NET 4.0 WinForms Toolbox Control should be available in Add New Item menu.

@gregvanl any news on this?

Hi @Misiu Unfortunately the product team says the Extensibility project templates required 4.5.


[SPARK-7787] [STREAMING] Fix serialization issue of SerializableAWSCredentials

Lack of default constructor causes deserialization to fail. This occurs only when the AWS credentials are explicitly specified through KinesisUtils.

@cfregly Take a look.

Merged build triggered.

Merged build started.

Test build #33234 has started for   PR 6316 at commit 248ca5c.

Test build #33234 has finished for   PR 6316 at commit 248ca5c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33234/
Test PASSed.

Merged build finished. Test PASSed.

I tested this in an end-to-end test locally. So I am merging this. Thanks @cfregly for catching this!


multiple keys

multiple keys
key:
key:
if you are not in shape
Should be used like xxxx, xxxx

What's the use-case for this?

haktrails-config.yml
ı use subdomains

Well, I understand that, but what is the use-case of having multiple API keys?

ı read this :) ....Warning: With this tool, it's very easy to burn through a lot of API credits. For example, if you have 10,000 domains in domains.txt, running cat domains.txt | haktrails subdomains will use all 10,000 credits. It's also worth noting that some functions (such as associated domains) will use multiple API requests, for example, echo "yahoo.com" | haktrails associateddomains would use about 20 API requests, because the data is paginated and yahoo.com has a lot of associated domains.

That is right, but these functions are only available for paid accounts anyway - the right way would be to get more queries on your api key, not to get 3 paid accounts ;)

I use free keys
:)
only 3 keys
securitytrails:
key:
key:
key:
can i use it as :)
or
Is only one key entered?

That's abusing the free account that SecurityTrails currently offers. If too many people do that, SecurityTrails will most likely stop offering it at some point. I'll leave it up to @hakluke to implement it, but I doubt this feature has a use-case that makes sense 😉

ok thanks ..

ok thanks ..

That's abusing the free account that SecurityTrails currently offers. If too many people do that, SecurityTrails will most likely stop offering it at some point. I'll leave it up to @hakluke to implement it, but I doubt this feature has a use-case that makes sense 😉

Turns out my account only has 3,000 credits :-D

Hey @onuncukoy-dot  - yeah I definitely will not be implementing this as using keys in this way would be blatant abuse of SecurityTrails' free offering.


jmxtrans stops writing to all outputs

I am trying to monitor a decent size set of metrics from a single JMXtrans instance and send them to InfluxDB. My proof of concept worked fine in that I was collecting from a single host and sending to InfluxDB with no issues.
I am using the 270 deb from http://central.maven.org/maven2/org/jmxtrans/jmxtrans/ on Ubuntu with Java 8
When I am collection from ~300 hosts across ~850 objects I am now experiencing where jmxtrans starts up it sends roughly a single set of metrics to its configured outputs and then never sends again until I restart.
I had originally thought that this was an issue with the InfluxDB output writer but I have the same results when just using the KeyOut writer.
I have multiple json config files and while some of the hosts are duplicated, the port for each host is unique in each of the servers hash.
I set log level to debug but get no errors or warning. All i see is that the writers stop and then all there is in jmxtrans pulling data from the java client but no longer publishing upstream.
Below is a snipper of the log files and some of the config
If there is any other info that you thing might be helpful in troubleshooting I will be happy to provide
All and any help is much appreciated
2019-02-16 00:55:07 [jmxtrans-result-8] level com.googlecode.jmxtrans.model.output.InfluxDbWriter [InfluxDbWriter.java:177] - Query result: Result(attributeName=CollectionCount, className=sun.management.GarbageCollectorImpl, objDomain=java.lang, typeName=type=GarbageCollector,name=PS MarkSweep, valuePath=[], value=0, epoch=1550278507163, keyAlias=jvmGC)
2019-02-16 00:55:07 [jmxtrans-result-8] level com.googlecode.jmxtrans.model.output.InfluxDbWriter [InfluxDbWriter.java:197] - Point: Point [name=jvmGC, time=1550278507163, tags={attributeName=CollectionCount}, precision=MILLISECONDS, fields={CollectionCount=0, _jmx_port=8003}]
2019-02-16 00:55:07 [jmxtrans-result-8] level com.googlecode.jmxtrans.model.output.InfluxDbWriter [InfluxDbWriter.java:177] - Query result: Result(attributeName=CollectionTime, className=sun.management.GarbageCollectorImpl, objDomain=java.lang, typeName=type=GarbageCollector,name=PS MarkSweep, valuePath=[], value=0, epoch=1550278507163, keyAlias=jvmGC)
2019-02-16 00:55:07 [jmxtrans-result-8] level com.googlecode.jmxtrans.model.output.InfluxDbWriter [InfluxDbWriter.java:197] - Point: Point [name=jvmGC, time=1550278507163, tags={attributeName=CollectionTime}, precision=MILLISECONDS, fields={CollectionTime=0, _jmx_port=8003}]
2019-02-16 00:55:07 [jmxtrans-result-8] level com.googlecode.jmxtrans.model.output.InfluxDbWriter [InfluxDbWriter.java:177] - Query result: Result(attributeName=CollectionCount, className=sun.management.GarbageCollectorImpl, objDomain=java.lang, typeName=type=GarbageCollector,name=PS Scavenge, valuePath=[], value=132, epoch=1550278507165, keyAlias=jvmGC)

2019-02-16 00:52:38 [ServerScheduler_Worker-4] level com.googlecode.jmxtrans.jobs.ServerJob [ServerJob.java:58] - +++++ Started server job: Server(pid=null, host=hddn025.sv3, port=8008, url=service:jmx:rmi:///jndi/rmi://hddn025.sv3:8008/jmxrmi, cronExpression=null, numQueryThreads=0)
2019-02-16 00:52:38 [ServerScheduler_Worker-4] level com.googlecode.jmxtrans.jobs.ServerJob [ServerJob.java:64] - +++++ Finished server job: Server(pid=null, host=hddn025.sv3, port=8008, url=service:jmx:rmi:///jndi/rmi://hddn025.sv3:8008/jmxrmi, cronExpression=null, numQueryThreads=0)
2019-02-16 00:52:38 [ServerScheduler_Worker-5] level com.googlecode.jmxtrans.jobs.ServerJob [ServerJob.java:58] - +++++ Started server job: Server(pid=null, host=hddn063.sv3, port=8008, url=service:jmx:rmi:///jndi/rmi://hddn063.sv3:8008/jmxrmi, cronExpression=null, numQueryThreads=0)
2019-02-16 00:52:38 [ServerScheduler_Worker-5] level com.googlecode.jmxtrans.jobs.ServerJob [ServerJob.java:64] - +++++ Finished server job: Server(pid=null, host=hddn063.sv3, port=8008, url=service:jmx:rmi:///jndi/rmi://hddn063.sv3:8008/jmxrmi, cronExpression=null, numQueryThreads=0)

{
  "servers": [
    {
      "host": "hdrm001.dc1",
      "port": 8010,
      "username": "readonly",
      "password": "readonly",
      "queries": [
        {
          "obj": "java.lang:type=Memory",
          "attr": [
            "HeapMemoryUsage",
            "NonHeapMemoryUsage"
          ],
          "resultAlias": "jvmMemory",
          "outputWriters": [
            {
              "@class": "com.googlecode.jmxtrans.model.output.InfluxDbWriterFactory",
              "url": "http://influxdb.dc2:8086",
              "database": "jmxtrans",
              "retentionPolicy": "2years",
              "username": "xxxxxx",
              "password": "xxxxxx",
              "resultTags": [
                "attributeName"
              ],
              "tags": {
                "dc": "dc1",
                "env": "prod",
                "class": "spark2",
                "app": "historyserver"
              }
            }
          ]
        },
        {
          "obj": "java.lang:type=GarbageCollector,name=*",
          "attr": [
            "CollectionCount",
            "CollectionTime"
          ],
          "resultAlias": "jvmGC",
          "outputWriters": [
            {
              "@class": "com.googlecode.jmxtrans.model.output.InfluxDbWriterFactory",
              "url": "http://influxdb.dc2:8086",
              "database": "jmxtrans",
              "retentionPolicy": "2years",
              "username": "xxxxxx",
              "password": "xxxxxx",
              "resultTags": [
                "attributeName"
              ],
              "tags": {
                "dc": "dc1",
                "env": "prod",
                "class": "spark2",
                "app": "historyserver"
              }
            }
          ]
        },
        {
          "obj": "java.lang:type=Threading",
          "attr": [
            "ThreadCount"
          ],
          "resultAlias": "jvmThreading",
          "outputWriters": [
            {
              "@class": "com.googlecode.jmxtrans.model.output.InfluxDbWriterFactory",
              "url": "http://influxdb.dc2:8086",
              "database": "jmxtrans",
              "retentionPolicy": "2years",
              "username": "xxxxxx",
              "password": "xxxxxx",
              "resultTags": [
                "attributeName"
              ],
              "tags": {
                "dc": "dc1",
                "env": "prod",
                "class": "spark2",
                "app": "historyserver"
              }
            }
          ]
        }
      ]
    }
  ]

you should enable trace level for your agent (either on wrapper or jmxtrans itself). Then do some profiling on the jvm. Also try to rise the heap to fixed 128MB (xms and xmx), sometime 64MB (the default) is not enough and result in stale agent after realy long GC.


Generic base version

Hi Alex,
Actually, I couldn't resist doing the job properly and this branch now works for ANY base. Even base 2!
I've liberally commented, but you should find that this is faster still - if you choose a base which is a power of 2 (the best option) that will ensure division is as fast as it can be.
If you run your performance tests against different bases, it would be interesting to see what results you get. If you set the base to 10 (what you had hard coded before), then it should be the same except that even the division would be about twice as fast I reckon.
Anyway, these are slightly more dramatic changes and include a few other performance enhancements.
Dominic

Hi Dominic,
I'd be happy to integrate this improvement. Can you please run eslint against your PR?

Hi Alex, OK did that, but, to be honest, I'm not sure you should bother...
I took up a rather unhealthy interest in the implementation of all this and became obsessed in understanding how http://mikemcl.github.io/bignumber.js managed to make his library as fast as it was. Although it is a comprehensive library it is completely unfathomable code-wise (unlike yours)!
I've ended up making wholesale changes to your library in order to get the performance up to spec. Which I've got in this branch: https://github.com/dthwaite/bignumber.js.git#master.
I doubt you'd want to take this either as you won't recognise most of your code! But the good news is
that it is far faster in most all respects, especially in division, where it is at least 1,000 times faster!
Trouble is it's still of a comparable speed to Mike's. And Mike is still twice as fast for multiplication which bugs me. But for division (which is where I focussed my attention) I have triumphed as I finally cracked on the right algorithm and it is now 3 or 4 times faster even than Mike's. So I can retire at peace!
You can check out my routine in that branch if you like, but I think this is about as fast as you'd ever get it in a JS environment.
You can have the master branch if you want, which would make your claim: "It does one thing, implementing only the main arithmetic operations for big integers, but it does it very well and very fast" have some credibility :-)
All the best
Dom
P.s. I actually got into all this to learn and understand GitHub workflow! Travis CI is awesome!

Oh, I appear to have created 2 "master" branches (still got a lot to learn) the one I'm referring to above is 'Master' with a capital M! https://github.com/dthwaite/bignumber.js.git#Master

Hi Dom,
Thanks for doing this! You've highlighted some great performance improvements. The purpose of the library is to be both concise and fast. I want to integrate a part of the changes, while still keeping the codebase small and simple, so I may cherry-pick multiple improvements.

Yeah, I think that is most sensible. Have fun
Dom
On 15 Mar 2016, at 19:38, Alexandru Bardaș<EMAIL_ADDRESS>wrote:

Hi Dom,
Thanks for doing this! You've highlighted some great performance improvements. The purpose of the library is to be both concise and fast. I want to integrate a part of the changes, while still keeping the codebase small and simple, so I may cherry-pick multiple improvements.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


Textra is in the wrong category

"textra - Vue js add-on to slide text." is in the Form -> Slider category, but it doesn't really fit there since it's not a slider library. I'm unsure where to put it so haven't made a PR, but thought it be nice to know.

@cjblomqvist textra already be on the list, if you think that it should be in another place do not hesitate to do PR


Fixed master bower cmd

Why? bower install 'angular-material#master' is not valid
bower install angular-material#master is.


[+] instance Bounded (Ix n)

useful for things like
store (str ! maxBound) 0

Seems reasonable, thanks!


OSError: [Errno 6] No such device or address

I'm trying to go through the ImportPythonModel.ipynb notebook in the examples. I do, however, get an error which I think needs to be solved in the package itself.

I made a local copy of the package files and I changed line 150 in writeJSONFiles.py to
modeler = getpass.getuser()
Someone recommended getpass.getuser() to me since it checks the environment variables LOGNAME or USERNAME to find out who the user is, and falls back to pwd.getpwuid(os.getuid())[0] to get the login name of the current real user id.
I tested it in my environment and it works like a charm. Maybe it would be an option in the pzmm package as well?

I tested the getpass.getuser() suggestion, but ran into another error on a different environment.
Since this doesn't seem to be a one-size-fits-all situation, I am going to add a try/except block and a new argument into the function that allows for the user to specify the modeler name. It should look something like this:
if modeler is None:
    try:
        modeler = getpass.getuser()
    except:
        modeler = 'defaultUser'

the additional modeler=None argument in the function call.

@Mespith the fixes from @smlindauer have been made to ImportPythonModel.ipynb example and the writeJSONFiles.py  Python script. If you have any additional issues, please let us know.
Thanks,
Kristen


"craco build --verbose" silently fails on CI

We are using craco "6.1.2", and it was working fine for us for a long time, but now on the CI we see this:

As you see on the screenshot it just stopped after the line
craco:  Found Webpack prod config at: ....

Locally, the same command looks like this:

The change that caused the issue was cleaning up the storybook from the project (including changes in lockfile, which I am most suspicious about).
Would you be able to give us any ideas where to continue with troubleshooting craco build on ci? Worst thing that it does not actually fail the build, (i.e. it just returns 0 exit code).

@perevernihata that's weird, especially since this didn't seem to be an issue for you before. Something must be happening in between locating the webpack prod config and overriding it. Does it work if you build locally?

I'm gonna close this because of inactivity, but feel free to leave a comment if this is still an issue.


No Squash and Merge on this Repo

@olynch, we have squash and merge disabled on this repo. I find that very handy as an option for situations like #142 where it is a 1 commit PR and the second commit is "address PR feedback". We can squash those in the interface rather than requiring the author to do a local rebase and push -f to rewrite the history. Can we enable squash and merge?

Sure


Request.Query["code"] is always null

string code = context.Request.Query["code"]; in the overridden Invoke(IOwinContext context) method of the "OAuth2CodeRedeemerMiddleware : OwinMiddleware" class is always null.

This was linked to a previous bug, and since it was resolved this one is no longer valid so I'm closing it.


Add find root cert fuzzer to tests

Please provide a description of this PR:

Yes, its a regression test case. Generally the idea so far as has been that
test cases the fuzzer generates that fail the test are added as regression
tests. it being empty is a bit trivial, but in this case it did trigger a
failure previously so its good to keep around as reference and to ensure it
doesn't break in the future, IMO.
On Tue, Nov 30, 2021 at 11:46 AM lei-tang @.***> wrote:

@.**** commented on this pull request.
"tests/fuzz/testdata/FuzzFindRootCertFromCertificateChainBytes/6118602084843520"
is an empty file. Is it needed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/36309#pullrequestreview-819483814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXNMEVKJ7TO33NI2EITUOUSZJANCNFSM5JCTUDOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Nit: why is the file named as "6118602084843520"? Will a name such as "empty-file" also work?

Its correlated to the OSS fuzz test case. This is accessible only to PSWG.
This is the same pattern https://go.dev/blog/fuzz-beta uses
On Tue, Nov 30, 2021 at 11:55 AM lei-tang @.***> wrote:

Nit: why is the file named as "6118602084843520"? Will a name such as
"empty-file" also work?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/36309#issuecomment-982969735, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXNPBHCPBSJAGKPQGGLUOUT35ANCNFSM5JCTUDOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

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

Test name
Commit
Details
Required
Rerun command

gencheck_istio
6617fb7f1d0e443706632d2dc964eb2e9b651c52
link
unknown
/test gencheck_istio

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

/retest

In response to a cherrypick label: #36309 failed to apply on top of branch "release-1.12":
Applying: Add find root cert fuzzer to tests
Using index info to reconstruct a base tree...
M	tests/fuzz/oss_fuzz_build.sh
A	tests/fuzz/pki_fuzzer.go
M	tests/fuzz/regression_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/fuzz/regression_test.go
CONFLICT (content): Merge conflict in tests/fuzz/regression_test.go
CONFLICT (modify/delete): tests/fuzz/pki_fuzzer.go deleted in HEAD and modified in Add find root cert fuzzer to tests. Version Add find root cert fuzzer to tests of tests/fuzz/pki_fuzzer.go left in tree.
Auto-merging tests/fuzz/oss_fuzz_build.sh
CONFLICT (content): Merge conflict in tests/fuzz/oss_fuzz_build.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Add find root cert fuzzer to tests
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #37424


Implement iOS one-click deploy.

Updated and improved version of https://github.com/godotengine/godot/pull/33086
Implements https://github.com/godotengine/godot-proposals/issues/3506, one click deploy and remote debug for iOS devices and simulator.

To use:

Full Xcode (not command line tools) should be installed.
ios-deploy should be installed and path added to the editor settings (Export ⇾ iOS ⇾ iOSDeploy).

Simulator should be running.
Device should be in developer mode, have developer account added to trusted on a device (Settings ⇾ General ⇾ VPN & Device Management), and screen unlocked.
For remote debugging on a real device, make sure the device is on the same network and a correct network interface is selected in the editor settings (Network ⇾ Debug ⇾ Remote Host). By default, the editor is listening for localhost connections only.

Note: official export templates do not include arm64 simulator binary due to OSXCross limitations, so to use simulator on Apple Silicon Macs a custom build should be used (build flags platform=ios ios_simulator=yes arch=arm64).

@punto- You mentioned this in a chat. Feel free to test it.

Is this making it into official builds anytime soon? 4.1 ?

Is this making it into official builds anytime soon? 4.1 ?

4.1 is in feature freeze, so this won't be merged until after 4.1.

@bruvzg Needs a rebase and addressing Akien's comment.

Thanks!

Nice!

First, this works great! A huge timesaver! (Godot v4.2.stable - macOS 14.1.0 - GLES3 (Compatibility) - ANGLE (Apple, ANGLE Metal Renderer: Apple M1 Pro, Version 14.1 (Build 23B74)) - Apple M1 Pro (10 Threads))
The issue is that even in GLES3 (Compatibility) the "ALERT! Your GPU (Apple A9 GPU) does not support the following issues which are required to use Vulkan-based renderers in Godot" shows up which is unexpected because I thought that the GLES was not Vulkan based.

The issue is that even in GLES3 (Compatibility) the "ALERT! Your GPU (Apple A9 GPU) does not support the following issues which are required to use Vulkan-based renderers in Godot" shows up which is unexpected because I thought that GLES3 was not Vulkan based.

Make sure your mobile rendering method override is also changed to Compatibility: https://github.com/godotengine/godot/pull/72461

Fixed. Thanks @Calinou.

@bruvzg Ok thank you for the note! I'm using Xcode 14.2, so I'll try it with another device using an older iOS version and report back :)

Following up on my previous comment https://github.com/godotengine/godot/pull/70662#issuecomment-1845958520 --
I tried with my older iPad, but still haven't been able to get one-click-deploy to iOS to work. The button for it is still grayed out and says "No Remote Debug export presets configured".
I'm using:
Godot 4.2
Xcode 14.2
MacOS 12.7.1 (Monterey)
Macbook Pro 16-inch M1 Pro, year 2021
iPad Pro 12.9 inch Model A1652, year 2015, iOS 14.8
I've set up the iOS export preset and set it to "Runnable"
My iPad is attached to my laptop using USB-C adaptor to USB-A to lightning cable
My iPad has "developer mode" enabled, screen is unlocked
I've reloaded the project, restarted Godot, unplugged and reconnected the USB-C cable
I've double-checked that my Macbook and iPad are both using the same wifi network

Are there more instructions on this using the simulator on Apple silicon? Perhaps this is the wrong post but I'm finding my builds in Xcode for simulators fail with this:

Follow the instructions in https://github.com/godotengine/godot/issues/62929.

Information about this should really be added to the one click deploy docs page.

why exclude the simulator builds from the official template?

OSXcross used to build official templates (on Linux) can't build it.

Quick question guys, why does the remote debug not work if i do the one click deploy on iOS? The debug console does not print anything and if i do changes in the scene it does on update on the phone, it does work on Android via cable though.


TypeError: Cannot read property '__parsed' of null?

This is a problem I  found in the demo you give. Give some help, thks!
var blc = require("broken-link-checker");

var html = 'absolute link';
html += 'relative link';
html += '';

var htmlChecker = new blc.HtmlChecker(null, {
    link: function(result) {
        console.log(result.html.index, result.broken, result.html.text, result.url.resolved);
        //-> 0 false "absolute link" "https://google.com/"
        //-> 1 false "relative link" "https://mywebsite.com/path/to/resource.html"
        //-> 2 true null "http://fakeurl.com/image.png"
    },
    complete: function() {
        console.log("done checking!");
    }
});

htmlChecker.scan(html, "https://mywebsite.com");

Indeed. Use {} instead of null until the next version is released:
var htmlChecker = new blc.HtmlChecker({}, {
    link: function(result) {

Got it! thks!


Auto-configure serverType and appLoc in sasjs web

When building SASjs-streamed apps, the index.html is parsed and relative links are updated in line with the target servertype / apploc.
It remains a manual effort by the developer to switch both serverType and appLoc in the adapter config in the index.html (the recommended location for adapter settings).
It would be a noticeable time saver and a big step towards 'one click deploys' if we can eliminate this manual step by updating both serverType (if present) and appLoc (should be always present) according to the specified target when the app is compiled using the CLI, and streamConfig.streamWeb==true.

via Hubstaff
User: Sabir Hassan
Project: SASjs cli - https://app.hubstaff.com/projects/2310875
Date Range: 01/18/23 - 01/19/23
Work session total: 5:32:12
Billable: Yes
Grand total: 5:32:12


Speed improvements, optimisations, better minification

thats that one all done, can i mark this as complete anywhere?


Fusion.cleanup should be renamed

Currently, Fusion api have cleanup and Cleanup for different usecases.
it is confusing to remember which one is which and easy to assume both functions are same
so I suggest renaming the cleanup to clean, which also sounds better when the other option is doNothing

A wild idea: it looks like ForKeys, ForValues and ForPairs accept a callable object, not just a function. So, it should be possible to add a __call metamethod to the Cleanup object and the user can simply pass it in, as-is.

A wild idea: it looks like ForKeys, ForValues and ForPairs accept a callable object, not just a function. So, it should be possible to add a __call metamethod to the Cleanup object and the user can simply pass it in, as-is.

I don't think Elttob wants to use metatables for clarity, however it sounds like a good idea!

I veto any metatables, ever™.
(only half joking there. stick to simple primitives! metatables confuse junior devs)

By the way, I agree with this premise. Feel free to bikeshed names here.
I only named it this way because I cared more about having the thing implemented rather than making the prettiest variables :)

I would prefer that Cleanup gets renamed to OnDestroy. OnDestroy better matches the convention of the other event listener keys, and it more aptly describes the fact that it will call your destructor functions. I think cleanup is an acceptable name as it is, but doCleaning might be more consistent? Not sure if it's worth changing that one.

Call it doCleanup.

Call it doCleanup.

I like that idea. It matches the doNothing name and also concisely indicates it's an action.

I would prefer that Cleanup gets renamed to OnDestroy. OnDestroy better matches the convention of the other event listener keys, and it more aptly describes the fact that it will call your destructor functions. I think cleanup is an acceptable name as it is, but doCleaning might be more consistent? Not sure if it's worth changing that one.

This is also a good point. I think it's worth changing both to help keep some good distance between them.

metatables confuse junior devs
:)

Late to the party here but was also having the same issue, it can be confusing.
I prefer doCleanup for performing the action, as it is easy to remember, and OnDestroy makes more sense than Cleanup

We'll rename it to doCleanup in v0.3. Thanks for the feedback everyone.

We'll rename it to doCleanup in v0.3. Thanks for the feedback everyone.

Can't wait to be in the change log for suggesting a name.


fix: enable google analytics

since merge of https://github.com/asyncapi/website/pull/2177 we lost google analytics
reason is that hostname is www.asyncapi.com. This is why I changed to use includes as in case of any future changes to host name that might take place, we will be safer

@thulieblack letting you know as community manager
@alequetzalli letting you know as person that sometimes analizes stats for docs traffic
fyi, whenever you will do some stats or analytics about traffic to the website, please keep in mind we lost any tracking between 28.09 and 23.10, so basically a month of data

/rtm


documentation of upgrade path

In what area(s)?

/area documentation

Describe the feature
It would be useful to document the upgrade path for an existing az-hop instance.
For example, say, you've created an az-hop instance with az-hop repo v1.0.9 and you would like to upgrade to az-hop v1.0.24.
What are the suggested steps to follow?
We realize that it may still a bit early to set up standards in this regard (e.g. based on semantic versioning), but it is nevertheless useful to administrators to have an idea of what to watch out for (even if it is just mentioned in the release notes, like: for this release, re-run terraform)
Mentioning @matt-chan for info

Upgrading path will have to follow these steps :

Read all versions notes to check if any breaking changes exists. If so update your config or take actions as described in the notes
if infrastructure changes need to be applied run the build.sh
remove ./playbooks/*.ok and run the install.sh

this should probably be added to the documentation


Gotify webhoook does not trigger when threshold fails

Describe the bug
Setting up a webhook for Gotify does not trigger for threshold failures
To Reproduce
Steps to reproduce the behavior:

Go to Notifications
Under Webhooks check "Enable webhook notifications"
Under Triggers check "Notify on threshold failures"
Under Recipients in the Url field set the Gotify app url
Click Test webhook channel

The test will send successfully but will not send when the threshold failure is met
Expected behavior
Since the test is working correctly, the expected behaviour is that during a threshold failure the notification webhook would be triggered.
Environment:

OS: Docker (with postgresql backend)
Architecture: amd64
Browser: Firefox
Version: 0.19.0

Screenshots
Gotify test notifications -

Threshold triggered on database

Logs
No info in container logs by default
Additional context
N/A. Happy to provide any more info

Gotify is not support at this time, dup of #40 and #1383.


ocamlPackages.lua-ml: init at 0.9

Motivation for this change
Dependency of soupault
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ocamlPackages.lua-ml


Retrain_from_scratch renaming

will close the other pull request #596 , because of branching it of the wrong branch.
Adressed black formatting issues.

Sorry for the previous attempt. Thanks for the hint with black! Now it should be ready to merge. :)

Thanks a lot, looks great now!


Documentation is not self explanatory

I am trying to write a cron job to do a particular task but to use this module is like an uphill task for someone who has no idea about a cron.

agree


Symbol for boolean NOT

Should the associated value with "-" be a boolean NOT? @joamatab
https://github.com/gdsfactory/gdsfactory/blob/ccfd0fb1736ac4a45f180790f7f09e80cfd7a5ca/gdsfactory/component.py#L74

yes, nice catch