Lower article section padding

The padding is excessively large for article sections. Since I'm not sure if this was intended or if it's a slight shape defect, I thought it best to open this up for investigation:

(main) margin-top: 4rem
(patch) margin-top: 2rem

Yep, it's intentional. In the default m.css style the "summary" sections tend to perceptually blend together somewhat when they get to a non-trivial length (i.e. many more members than in your example).
I played around with other solutions to it but ultimately increasing the padding to the current value was about the only one I was happy with.
It is a matter of taste/reading habits, of course. What to me gives the document some nice "breathing space", to others might be too sparse for comfort, so I'm not opposed to reducing it. I suspect 2rem would still be too small for my taste, but 3rem might be a good compromise.

Yep, it's intentional. In the default m.css style the "summary" sections tend to perceptually blend together somewhat when they get to a non-trivial length (i.e. many more members than in your example).
I played around with other solutions to it but ultimately increasing the padding to the current value was about the only one I was happy with.
It is a matter of taste/reading habits, of course. What to me gives the document some nice "breathing space", to others might be too sparse for comfort, so I'm not opposed to reducing it. 2rem looks like it will still be too small for my taste, but 3rem might be a good compromise.

Ultimately, it's just a matter of preference, it's fine to leave it as is ~ Although it would be interesting to hear other users' opinions on this

@marzer I saw that Poxy allows jquery scripting. While this goes against the css-only philosophy, maybe a tiny "compact mode" checkbox at the bottom of the table of contents would be sweet? Although I am not familiar with CSS, it may even be possible to use pure-css with something akin to:
https://www.w3schools.com/cssref/sel_gen_sibling.asp
#foo:checked ~ bar {
  merging-top: 1rem;
}

While this goes against the css-only philosophy

It's opt-in. It's just a "there if you want it" feature. I'm generally not as attached to the no-JS opinion held by m.css so I don't mind a small amount of it, within reason (I don't want 100 frameworks and a bunch of 'modern web' crap though, so on that note mosra and I are in firm agreement :D)
(Also the built-in search provided by m.css is powered by JS)

Although I am not familiar with CSS, it may even be possible to [...]

Maybe? I'm not that familiar with CSS either, to be honest. Most/all of my CSS knowledge is about 10 years old at this point. That being said, how would that specific solution be any different? If the net effect is visual space between consecutive sections being made larger or smaller, does it matter if it's done using margins or internal padding? Guess I'm unclear on the specific aspect you're actually wanting to change - is it purely visual or some other semantics?

maybe a tiny "compact mode" checkbox at the bottom of the table of contents would be sweet

This is a great idea. My go-to way of doing that would have been site-wide (binding the checkbox state to a cookie), but I have no idea what the state-of-the-art in webdev is these days. Likely 400 javascript frameworks and server-side rendering, lol. There's also the question of where the option toggle should go; under the menu seems like a logical choice but the floating menus can get pretty long so there's always a chance that it'd get pushed off the page. Pretty easy to prototype there and move it later, though, so I guess that's moot.
I'm actually about to go on a month-long overseas holiday in about a week's time and won't really be doing any tech work while I'm away; Any chance you want to do some prototyping of it in that time? I'll be unable to do so myself until around the 20th of October.

Any chance you want to do some prototyping of it in that time? I'll be unable to do so myself until around the 20th of October.

I'm going to attempt to create a prototype this week or so. It's something I've been meaning to do locally for my own use, so it fits into my schedule. In the meantime, I'll commit the prototype in this RP so you can dive in easily as you get back from vacation

Guess I'm unclear on the specific aspect you're actually wanting to change - is it purely visual or some other semantics?

From what I've seen, CSS allows the child to "inherit" the value of the checked selector. This may allow poxy to not use JS for toggling http://jsfiddle.net/8b9j1ex5/6/ but this is something I am not familiar with, so I don't know how complex it would be to integrate it into Doxygen html generation

(Also the built-in search provided by m.css is powered by JS)

Now I understand why its so great. :)

I'll commit the prototype to this RP so you can dive in easily as you get back from vacation

Awesome! Looking forward to seeing what you come up with.

Now I understand why its so great. :)

Mosra did an in-depth write-up on it when it was first implemented, check it out if you're interested in that sort of thing: https://blog.magnum.graphics/meta/implementing-a-fast-doxygen-search/

Hmm, what if m.css had CSS theme "option" for the between-section padding? I vaguely remember I had some TODO item for this, but never got to actually finishing that...
As in, I have margin-bottom: 1rem; hardcoded on many places in the CSS and it might make more sense to have that configurable via a variable in the theme file instead, which you could then adapt to your liking. Plus instead of all margin being the same value, it could be different in usual text flow vs. between sections, for example. And it could benefit both of you I think, and shrink the set of patches poxy needs to apply on my CSS.
Though I can't say when I would be able to implement that. Constantly juggling 2-3 long-term contracts at the same time, and m.css is not among those unfortunately. A PR welcome though, I can provide a guidance -- as always :)

Hmm, what if m.css had CSS theme "option" for the between-section padding? I vaguely remember I had some TODO item for this, but never got to actually finishing that...
As in, I have margin-bottom: 1rem; hardcoded on many places in the CSS and it might make more sense to have that configurable via a variable in the theme file instead, which you could then adapt to your liking. Plus instead of all margin being the same value, it could be different in usual text flow vs. between sections, for example. And it could benefit both of you I think, and shrink the set of patches poxy needs to apply on my CSS.
Though I can't say when I would be able to implement that. Constantly juggling 2-3 long-term contracts at the same time, and m.css is not among those unfortunately. A PR welcome though, I can provide a guidance -- as always :)

This could also work, but there is a gap between a personal preference and the default value assigned to project documentation elsewhere. The idea here is more at the scale where a user could "compact" the documentation on a whim when the "height" is limited as in the case of, say, a specific tile layout. With a theme option, it would be up to the project owner to specify their own preference, which may or may not be pleasant for someone else :)

@mosra Interesting - so the 'compiled' themes are essentually manually-inlined and preprocessed versions of the original inputs. My understanding of CSS is fairly limited, but if the process was tweaked to inline the 'imports' without manually substituting the variables, and the theme variable files were distributed along with the rest of the docs HTML, then that should 'still work', right? And allow for some dynamic switching of the theme from JS or somesuch? i.e. for m-dark.css:
/* omit this altogether and instead handle it in JS/HTML for dynamic shenanigans  */
/* @import url('m-theme-dark.css'); */

/* inline these imports but leave the var(...) expressions as-is */
@import url('m-grid.css');
@import url('m-components.css');
@import url('m-layout.css');
@import url('pygments-dark.css'); /* ...this one seems problematic */
@import url('pygments-console.css');

Without JS:
Screencast from 2022-09-05 13-35-11.webm

so the 'compiled' themes are essentually manually-inlined and preprocessed versions of the original inputs.

Yes, although not manually, there's a script for that :)

but if the process was tweaked to inline the 'imports'

No need to tweak anything -- if you include m-dark.css via <link rel="stylesheet">, instead of m-dark.compiled.css or whatever, and provide the original sources on the server too, the browsers will understand that. It's compiled because it's just a single HTTP request that way, and as the variables are always way longer than the value that's substituted, it's also much smaller and better compressible that way. (It was originally there for IE8 compatibility, but five years later it's not an issue).
But yes, this way you could switch the CSS on-the-fly without doing nasty JS stuff. Ages ago I did a similar thing for a fan website for the Archer series (click on the color swatches at the top), all it involved was setting a disabled attributed on a particular <link>. Well, and then a cookie to remember that for next visit. Which means, EU cookie banners, and popups, and opt-in buttons and ... isn't a single style actually bearable compared to that? :laughing:

Yes, although not manually

Heh, I meant manually as in 'you manually wrote a bespoke tool for performing that task', not that you were literally doing it by hand like a psycho. Poor choice of words I guess :D

No need to tweak anything -- if you include m-dark.css [...]

Well. This gives me ideas. I am annoyed that I'm going on vacation now.

Here's some actual docs on the feature I linked above: https://developer.mozilla.org/en-US/docs/Web/CSS/Alternative_style_sheets
(I think?) it's the same technology as what makes dark/light mode switching. Funnily enough, there's a link to an example on that page, claiming that Firefox supports even browser-side switching, but I couldn't find the menu item it refers to anywhere in the dumbed-down UI...

@marzer  Here is a showcase of what I experimented with today. Instead of adding the toggle button in the table of contents, I added a simple filter icon in the navigation bar, next to the search and github svg. I took the liberty of transforming the github icon to be more in line with the search icon, I hope you don't mind. This is purely done in CSS so the integration doesn't clash with m.css "js-free" philosophy. I'll try to push through it later tonight, but I probably won't have time to do so until tomorrow.
@mosra Let me know what you think too, and if this could be integrated into m.css along with your proposal to integrate a theme "option" for padding. I wouldn't mind giving it a shot this month.
Granted, this is just a video for now, I still have some cleaning and tweaking to do before I can commit. Regardless, it's a nice overview.
Note: The hover and active effectare currently missing, I ran out of time for this, but they will be added before I commit.
Screencast from 2022-09-05 15-17-40.webm

@wroyca Looks great! I like the github button change - that's probably how it should have always been.
I'm currently in the process of tweaking the way CSS files are included to use the un-pre-processed @import versions w/ variables. I won't publish a new release from it just yet but will push it so you can at least see what I've done and/or experiment with it.

@wroyca I've pushed those changes to main. The CSS file layout now mimmicks the m.css one, in that there's two 'entry' css files, with only one ever being included depending on the theme:

poxy-dark.css
poxy-light.css

Should make it easy to implement any dynamic theme switching shenanigans, as well as providing a place to 'inject' poxy variable overrides without needing to do clunky crap with !important. Eventually I'll try to backport the overrides I've already made to this new layout since it's clearly a much better way of doing it, heh.
I don't think this will impact your PR all that much, other than maybe making it easier to override stuff at the 'right' level.

@wroyca I've pushed those changes to main. The CSS file layout now mimmicks the m.css one, in that there's two 'entry' css files, with only one ever being included depending on the theme:

poxy-dark.css
poxy-light.css

Should make it easy to implement any dynamic theme switching shenanigans, as well as providing a place to 'inject' poxy variable overrides without needing to do clunky crap with !important. Eventually I'll try to backport the overrides I've already made to this new layout since it's clearly a much better way of doing it, heh.
I don't think this will impact your PR all that much, other than maybe making it easier to override stuff at the 'right' level.

Thank! I'll take a look and make adjustments if necessary

You might still want to apply the processing script nevertheless, to avoid too deep dependent HTTP requests ;) Especially the @import statements mean the browser will first have to wait until the top-level CSS is downloaded and only then it can submit a request for the remaining CSS files.
The css/postprocess.py script should be hopefully easy enough to import and call into from within poxy itself -- so it should be possible even with any overrides done by the user. Didn't see it being done in https://github.com/marzer/poxy/commit/c90fa961722688be579f96d5cfb0a48554e95be0, apologies if I missed that.

Didn't see it being done in https://github.com/marzer/poxy/commit/c90fa961722688be579f96d5cfb0a48554e95be0, apologies if I missed that.

Nah nothing like that is implemented yet. On the to-do list, though.

@mosra on the topic of reducing HTTP requests: I noticed that the google fonts do a fairly large number of fetches in their own right. Have you ever experienced this to be an issue? I am mulling over an idea of preprocessing those and caching their payloads as part of poxy (modulo concerns about them going stale)

@mosra on the topic of reducing HTTP requests: I noticed that the google fonts do a fairly large number of fetches in their own right. Have you ever experienced this to be an issue? I am mulling over an idea of preprocessing those and putting their payloads in the output folder directly (modulo concerns about them going stale)

Oh the subject of fonts: they were previously broken for the light theme, I didn't have time to check if they are still broken with the latest changes

@wroyca If they were before, they shouldn't be now. There was a bit of dumb special-handling that I removed so now dark + light are handled in exactly the same manner.

Re Google Fonts, see also https://github.com/mosra/m.css/issues/152. This is one of the larger topics I want to tackle, to not have a dependency on a 3rd party service. Because for me at least, if the pages load slow, it's due to these large CDNs being flaky, not due to my server being overloaded.

@wroyca RE the original reason for opening this issue: on a whim I just changed the padding constant back to the 1rem default and it looks fine; I'm wondering if I did that before making some other change and now they've compounded together without me noticing.
So on that note I'd happily accept this pr in it's original form (or even: delete that CSS block altogether and use the m.css default)

@wroyca RE the original reason for opening this issue: on a whim I just changed the padding constant back to the 1rem default and it looks fine; I'm wondering if I did that before making some other padding-related change and now they've compounded together without me noticing.
So on that note I'd happily accept this pr as-is (or even: delete that CSS block altogether and use the m.css default)

That's fine for me, I'll rebase and dissect the GitHub logo

Thanks @wroyca!


Dialog forms open in the background

Brief description of your issue
Dialog forms open in the background, if View -> Always on top
Steps to reproduce (optional)
View -> Always on top  check on.
Double click any Service.
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
System Informer 3.1.24333.0 Canary
Windows 11 23h2

Duplicate of #200


🚀 Feature: Weaviate Instrumentation

Which component is this feature for?
All Packages
🔖 Feature description
Instrument calls to Weaviate, including adding attributes, similarly to our Chroma instrumentation. The instrumentation should support all types of calls - streaming, non streaming, async, etc.
🎤 Why is this feature needed?
Completeness of OpenLLMetry
✌ How do you aim to achieve this?
Similar to other instrumentations, we have in this repo.
🔄 Additional Information

Make sure to initialize this in Traceloop SDK.
Add proper tests.

👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

Are you willing to submit PR?
None

I’ll look into this if I get some time this week 😊

It would be great if we have the above support.

Hi, I started working on this, but I've ran into some issues with dependency conflicts. Basically, haystack depends on pydantic < 2, and weaviate-client>4 depends on pydantic>=2.
While haystack v2 (beta) seems to have dropped pydantic as a dependency (https://github.com/deepset-ai/haystack/blob/main/pyproject.toml), we have the choice to instrument instead weaviate-client v3 instead (https://weaviate.io/developers/weaviate/client-libraries/python_v3).
The downside is that weaviate-client v3 API is not as neatly organized as the v4. And also means some duplicated effort, as I started building the sample app with the v4 interface.
On the other hand, upgrading haystack to v2 is risky, as it's beta. And would require much more rework, since we would have to update the instrumentation.
So weaviate-client v3 seems like the best idea - unless you have a better idea.

WIP can be tracked here:
https://github.com/traceloop/openllmetry/pull/394

@mohitk1995 if you're a user of weaviate, maybe could you take a look at pull request #394?


rewrite CLI mode

in this time - CLI mode code is little messy =)

CLI Stream mode currently removed for reconstruction


fix Microsoft.Xrm.Tooling.Connector.dll and Nuget.exe path in cmdlets project

Hi, what is the difference between these two?

 <HintPath>..\packages\Microsoft.CrmSdk.XrmTooling.CoreAssembly.<IP_ADDRESS>\lib\net452\Microsoft.Xrm.Tooling.Connector.dll</HintPath>

 <HintPath>..\packages\Microsoft.CrmSdk.CoreTools.<IP_ADDRESS>\content\bin\coretools\Microsoft.Xrm.Tooling.Connector.dll</HintPath>

Sorry, I should have been more clear about that. When I tried to build the project, it failed because it couldn't find the Microsoft.Xrm.Tooling.Connector.dll assembly. When I searched for it in the packages folder, it only showed up in the Microsoft.CrmSdk.CoreTools folder. Since the Microsoft.CrmSdk.XrmTooling.CoreAssembly folder didn't exist, and there wasn't a NuGet package by that name (just Microsoft.CrmSdk.XrmTooling.CoreAssemblies), I assumed it was just an outdated reference.

Thanks. Had too many cached local files which is why it was working for me. I will setup an automated build in VSTS for this :)


Expected 'None' but got CognitiveServicesCredentials Instead plus InvalidResponse

Package Name: azure face recognition
Package Version: latest
Operating System: Ubuntu 20.04 LTS
Python Version: python3.8

Describe the bug
I'm setting up a code for face recognition and when I'm making the code I get this warning while I implement the face client.
face_client = FaceClient(ENDPOINT, CREDENTIALS)
After this there's this other error which causes the code to break.
response_faces = face_client.face.detect_with_stream(
    image=open(image, 'rb'),
    detection_model='detection_03',
    recognition_model='recognition_04',
    return_face_landmarks=True,
)

This is how I wanna send messages locally by stream but the problem is that whenever I run this code I get this problem:
/home/thecowmilk/dev/azure_faceapi/venv/bin/python /home/thecowmilk/dev/azure_faceapi/faceapi/starting.py
Traceback (most recent call last):
  File "/home/thecowmilk/dev/azure_faceapi/faceapi/starting.py", line 17, in <module>
    response_faces = face_client.face.detect_with_stream(
  File "/home/thecowmilk/dev/azure_faceapi/venv/lib/python3.8/site-packages/azure/cognitiveservices/vision/face/operations/_face_operations.py", line 782, in detect_with_stream
    raise models.APIErrorException(self._deserialize, response)
azure.cognitiveservices.vision.face.models._models_py3.APIErrorException: (InvalidRequest) Invalid request has been sent.

To Reproduce
Steps to reproduce the behavior:

A clean install of your OS
install these libraries:

azure-cognitiveservices-vision-face
azure-cognitiveservices-vision-computervision
azure-cognitiveservices-vision-customvision
azure-cognitiveservices-formrecognizer
pillow
pandas

Write this code:

import os
import io
import cv2
from azure.cognitiveservices.vision.face import FaceClient
from azure.cognitiveservices.vision.face.operations import FaceOperations
from msrest.authentication import CognitiveServicesCredentials
from PIL import Image, ImageDraw, ImageFont

API_KEY = 'your_free_sub_api_key'
ENDPOINT = 'your_free_sub_endpoit''
CREDENTIALS = CognitiveServicesCredentials(API_KEY)
image = 'realmadrid.jpg'
face_client = FaceClient(ENDPOINT, CREDENTIALS)
operations = FaceOperations

response_faces = face_client.face.detect_with_stream(
    image=open(image, 'rb'),
    detection_model='detection_03',
    recognition_model='recognition_04',
    return_face_landmarks=True,
)

if not response_faces:
    raise Exception("No face detected!")

print(f"Number of face detected {len(response_faces)}")

Try to run and then you have got the idea of this error

Expected behavior
It should have worked an returned a result

@thecowmilk We are looking into it and get back to you for any additional information.

@thecowmilk I do not see any issues calling detect_with_stream function with both results and no results. See below -

Which version of the vision face library you are using?
Thanks
Saurabh

@SaurabhSharma-MSFT I debugged a little and I got this response back
raise DeserializationError("Cannot deserialize as [{}] an object of type {}".format(
azure.core.exceptions.DeserializationError: ("Unable to deserialize response data. Data: {'error': {'code': 'InvalidRequest', 'message': 'Invalid request has been sent.', 'innererror': {'code': 'UnsupportedFeature', 'message': 'Feature is not supported. Please apply for access at https://aka.ms/facerecognition'}}}, [DetectedFace], DeserializationError: Cannot deserialize as [DetectedFace] an object of type <class 'dict'>", DeserializationError("Cannot deserialize as [DetectedFace] an object of type <class 'dict'>"))

I don't know if it intended or not....

@thecowmilk This is intended as Face API falls under Limited Access features for Cognitive Services.
You got this error since what you are trying to call is now limited access to them because you have not been approved to use that capability. Face service is only available to Microsoft managed customers and partners. Please use the Face Recognition intake form to apply for access- aka.ms/facerecognition.
Please let me know if you have any questions.
Thanks

@thecowmilk We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


server uses the same net.listener for http server and jocko tcp server?

https://github.com/travisjeffery/jocko/blob/master/server/server.go#L88-110
It looks jocko and http server use the same net.Listener and that means there are two goroutines to do s.ln.Accept(). Maybe some sent kafka messages have been handled by http server and it thinks them are wrong requests?

Thanks @smallnest !


(2.6 patch) Fix recvfrom never populating from_addr due to fromlen being zero

What does this Pull Request accomplish?
TODO: Include high-level description of the changes in this pull request.
Why should this Pull Request be merged?
TODO: Justify why this contribution should be part of the project.
What testing has been done?
TODO: Detail what testing has been done to ensure this submission meets requirements.

@maxxboehme i also created https://github.com/ni/grpc-device/issues/1109, if the bot is enabled, it can help us update github action versions.

@jonathanou is a release patch necessary? Our process does not currently work well for patch releases of grpc-device and it causes havoc with the internal pipelines kicked off for azdo. Preferably would either have

Release goes into main and either download the latest CI builds of the executable until the next release of grpc-device happens at the end of the year.
Have an off cycle release of 2.9 to 2.10 of grpc-device as the current tooling supports that better and won't cause internal azdo pipeline havoc.

We are planning to create a release branch on the 18th, and your PR will need to be completed before then for the changes to be in effect. Do you want these changes included in this release?

We are planning to create a release branch on the 18th, and your PR will need to be completed before then for the changes to be in effect. Do you want these changes included in this release?

@christag-ni , thanks for the reminder!  No, these changes do not need to be included in any release;  I will close this PR.


Weekly Password Change

The auto passy is being sent but the password doesn't get update.

@ZakMooney this should be resolved now, can you pass news on to customer paty


Number of domains and ranges in RDFSProperty

The representation for a rdfs:Property in the interface RDFSProperty (com.github.anno4j.schema_parsing.model.rdfs) does only support assignment of single resources as the rdfs:range and rdfs:domain of the property.
But multiple resources should be allowed, implying that the objects/subjects of statements with the property as predicate are instances of all those classes.

Adopted RDFSProperty so that multiple range and domain classes are possible. Also changed the return and parameter type of the methods to RDFSClazz, because according to the RDFS recommendation only rdfs:Class instances can be part of a properties range or domain.
Also added add* methods for convenient adding of new classes.


Improve ReadTheDocs documentation

Not a very interesting patch but we can't put on the report that we use ReadTheDocs if there is no useful information on the website.

Codecov Report

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

@@          Coverage Diff          @@
##           master    #43   +/-   ##
=====================================
  Coverage      89%    89%
=====================================
  Files          16     16
  Lines        1110   1110
=====================================
  Hits          988    988
  Misses        122    122

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 e03ec62...11e7100. Read the comment docs.


Support for OpenAI Compatible API embedding model

Pull Request Type

[x] ✨ feat
[ ] 🐛 fix
[ ] ♻ refactor
[ ] 💄 style
[ ] 🔨 chore
[ ] 📝 docs

Relevant Issues
resolves 160
resolves #xxx
What is in this change?
Added Support for OpenAI Compatible API for embedding models.
Additional Information
Add any other context about the Pull Request here that was not captured above.
Developer Validations

[x] I ran yarn lint from the root of the repo & committed changes
[x] Relevant documentation has been updated
[x] I have tested my code functionality
[x] Docker build succeeds locally

Hi @timothycarambat
can you please check this PR and merge.


Update dependencies and workflows

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (60c566b) to head (bc8936c).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #366   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           95        95
  Lines         2397      2397
  Branches       194       194
=========================================
  Hits          2397      2397

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


About open-graph & media template

Please add open-graph & media template. This template can send third party's video, music url (spotify, youtube, facebook page videos etc).
In our library sendVideoAttachment sends mp4 files only.
https://developers.facebook.com/docs/messenger-platform/send-messages/template/open-graph
https://developers.facebook.com/docs/messenger-platform/send-messages/template/media

0.8.0 vs 1.0.0-M1 huge difference in coding.
Why did you delete all previous version methods?

Version 1.0.0-M2 now supports the Open Graph Template. The media template will be part of the next milestone release along with the attachment upload api.


Pytorch model

Hi
I wanted to know how I can calculate the clever score for a pytorch model ?

Thank you @pinyuchen for the answer. Let us know if you have any further questions.

I have seen the wrapper. It has 2 major problems. 1. The implementation is not efficient. The original implementation uses multitasking library while theirs is a serial implementation.
2. The implementation is incomplete. The original code estimates the c_init while that's not implemented in the wrapper.

@kfarivar I agree with you that the original implementation is more efficient. I did a lot of multiprocessing optimizations to fully utilize the GPU. The new implementation was done by an independent team which focuses more on functionality rather than performance.
It is possible to change the original implementation to use a PyTorch model. CLEVER is a sampling based method so as long as you can integrate pytorch model inference into the code, everything else should work. More precisely, you will need to replace this tensorflow sess.run with a pytorch model inference:
https://github.com/huanzhang12/CLEVER/blob/master/estimate_gradient_norm.py#L461
Most other things can be kept unchanged, including the multitasking part. However it does need some efforts if you plan to take this route.

Hello, is there an easy way to use your pytorch models?


Feature request: Return Arrow table in cursor result

Would you consider adding a method to return an Arrow table or Arrow record batches from a cursor result? My use case is that I want to convert a query result to a DataFrame as efficiently as possible. I can get this through Polars if I send the query as a string. It won't work if I send a sqlalchemy Selectable object. Example code and comments from a Polars developer are here.

@daniel-thom: FYI, I looked a little deeper here - seems it can already be done as there is a transparent redirect on the Cursor object to the underlying raw connection, which supports fetch_arrow_table and fetch_record_batch methods. I'll make Polars aware of the passthrough and then we can take full advantage of the Arrow speedups.


Please document the different syntaxes for line ranges

The properties at
https://github.com/scanoss/scanoss.java/blob/8598c47987bb92490755972e845bd9fe2a6955d8/src/main/java/com/scanoss/dto/ScanFileDetails.java#L42-L44
seem to support different string-based syntaxes for line ranges, or even lists of line ranges. What I've seen so far is e.g. "42" and "66-99", but apparently also something like "32-105,117-199,210-266,278-488,500-781,793-937,951-1051,1062-1081" can be returned by the API. Please document all of the above, plus anything else that is to be expected.

As a side note, ideally the data model would be more typed here to begin with, and use an array of start and end lines which each are integers, instead of a string that the user of this library needs to parse on its own.

Hi @sschuberth ,
Thanks for the feedback!
For the current version of this JSON schema we are keeping it as a string (comma separated). We are working on a new structure and when that is published it will be fully typed.
We will share this info once it has been finalised.


Add support for creating correlated variables from a covariance or correlation matrix

This PR adds four new functions:
Measurements.covariance_matrix(n::AbstractVector{<:Measurement})

to calculate the covariance matrix from a vector of measurements,
Measurements.correlation_matrix(n::AbstractVector{<:Measurement})

to calculate the correlation matrix, and
Measurements.correlated_values(
    nominal_values::AbstractVector{<:Real},
    covariance_matrix::AbstractMatrix{<:Real}
)

and
Measurements.correlated_values(
    nominal_values::AbstractVector{<:Real},
    standard_deviations::AbstractVector{<:Real},
    correlation_matrix::AbstractMatrix{<:Real}
)

to create correlated variables from a covariance and correlation matrix respectively.
The algorithms are taken from the python package uncertatinties. If that is not okay, please tell me.
If there are issues with code styling, or if something is missing, I am happy to fix or add it.

Do you want to keep the backwards compatibility to version 1.0? If yes, I will have to find some compromises regarding readability/speed with the code.

I haven't looked into the details of the implementation, but this is awesome!
I'm happy to drop support for older versions of Julia, what would be the first one needed as things are now?

I'd appreciate documenting this in the Usage and Examples sections of the manual

I will add the corresponding usage documentation and a few examples in the coming days as soon as I find some more time.

Codecov Report

Merging #137 (d54bd42) into master (d1ac981) will increase coverage by 0.04%.
The diff coverage is 100.00%.

:exclamation: Current head d54bd42 differs from pull request most recent head 57a6246. Consider uploading reports for the commit 57a6246 to get more accurate results

@@            Coverage Diff             @@
##           master     #137      +/-   ##
==========================================
+ Coverage   95.45%   95.50%   +0.04%
==========================================
  Files          12       12
  Lines         726      756      +30
==========================================
+ Hits          693      722      +29
- Misses         33       34       +1

Impacted Files
Coverage Δ

src/utils.jl
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I have added documentation for both, could you check if it fits your expectation?
Also, are you happy with the function names? There are existing functions in Statistics named cov and cor for calculating the covariance and correlation matrices. Would it be better to extend these?

BTW, this PR solves the only point in the TODO list https://github.com/JuliaPhysics/Measurements.jl/blob/d417519abcedaeeb4cfff85eaaae9188a2aa7997/docs/src/todo.md?plain=1#L8-L12 which can now be removed :slightly_smiling_face:

Thanks for your comments, I will fix them.
What is your opinion on extending the functions in Statistics named cov and cor for calculating the covariance and correlation matrices instead of creating our own?
There are other packages like Distributions.jl which extend these functions.

Thanks a lot for this nice contribution, much appreciated and awaited!


Lacking onMouseMove (summary)

This issue is the summary of #50 for quick reference from #53 .
Original comment:

I found onMouseMove is not defined in current Html.Event API. I guess there is some reason for not supporting it (maybe Mouse module is better at some point?), but it is not explained anywhere. So, it's simply inconvenient now.

Typical situation:

For one example, I'm making an app that can edit maps, in which I need to get relative position in a certain area for drag-and-drop. But I'm not sure everyone needs it, or it's the best solution for it.

Typical solution:  (Thanks to @jvoigtlaender)

@jinjor's problem sounds as if simply using offsetX and offsetY would be the solution. Should work without being confused by inner elements in the container.

Note: offsetX and offsetY is marked as "experimental" at MDN, but all modern browser seem to support it.

Okay, I made the meta issue point here instead. Thanks!


image search is not working

This is because it's not sending the vqd parameter when getting videos and images. Take a look at https://github.com/deepanprabhu/duckduckgo-images-api/issues/1#issuecomment-347428637

Guys, the above bug was fixed with latest code. Lemme know.

please pull the latest code.

This issue has been marked stale. This package has been updated to v2!
Look into documentation and usage here: https://duck-duck-scrape.js.org/


Debug Output Images

I generated a debug output image like below while I was testing. I defined each image of the debug output images listed below. Is my definition of the output images correct? Please correct me if my definition is incorrect.
Output Image:  Images between 1 through 4 are normalized.
1: Original image t1 (o1)

Original Image t1ce (o2)

Original image t2 (o3)

Original Image flair (o4)

Original Image t1 without normalization (m)

The sample image from the model (each time step of diffusion from 0 through 999) (s)

The segmentation Image from the trained model output (c)

An overlapping images of the final image of random samples and the final prediction image.

@ToruHironaka Can you tell me how many steps you trained it? Also is the last but one Ground truth or the sample segmentation from the model?

The training diffusion step was 1000. I follow the below script with the arguments posted on the README page.

For training, run: python scripts/segmentation_train.py --data_name ISIC --data_dir input data direction --out_dir output data direction --image_size 256 --num_channels 128 --class_cond False --num_res_blocks 2 --num_heads 1 --learn_sigma True --use_scale_shift_norm False --attention_resolutions 16 --diffusion_steps 1000 --noise_schedule linear --rescale_learned_sigmas False --rescale_timesteps False --lr 1e-4 --batch_size

The training session generated the last model called "emasavedmodel_0.9999_135000.pt."
Here are the training logs.
log.txt
progress.csv
I ran the below script with the training model and the arguments to generate the above image. I used Brast2020 data.
python scripts/segmentation_sample.py --data_dir /data/BraTS2020/Data/MICCAI_BraTS2020_ValidationData/
--out_dir ./results_BraTS2020_sample_dpm_solver_0.9999_135000 --model_path ./results_BraTS2020/emasavedmodel_0.9999_135000.pt
--data_name BRATS --image_size 256 --num_channels 128 --class_cond False --num_res_blocks 2 --num_heads 1
--learn_sigma True --use_scale_shift_norm False --attention_resolutions 16 --diffusion_steps 50 --dpm_solver True
--noise_schedule linear --rescale_learned_sigmas False --rescale_timesteps False --num_ensemble 5 --multi_gpu 0,1 --debug True
Also, is the last but one Ground truth or the sample segmentation from the model? <-- This is my question too.
I think your script "segmentation_sample.py" lines between 152 and 163, generate the output images.
Here is the image of code:

Code:
s = th.tensor(sample)[:,-1,:,:].unsqueeze(1)
m = th.tensor(m.to(device = 'cuda:0'))[:,0,:,:].unsqueeze(1)
o1 = th.tensor(org)[:,0,:,:].unsqueeze(1)
o2 = th.tensor(org)[:,1,:,:].unsqueeze(1)
o3 = th.tensor(org)[:,2,:,:].unsqueeze(1)
o4 = th.tensor(org)[:,3,:,:].unsqueeze(1)
c = th.tensor(cal)
I checked each output image s, m, o1, o2, o3, o4, c. I think these output images were defined below. I might be totally wrong though. I would like to confirm with you that my definition of the output images is correct. I am pretty sure that my definition of the output images between 1 through 5 below is correct, but I am not sure about the rest of the output images.
1: Original image t1 (o1)
2. Original Image t1ce (o2)
3. Original image t2 (o3)
4. Original Image flair (o4)
5. Original Image t1 without normalization (m)
6. The final image of random samples from the model (p_sample_loop_progressive
from 0 through 999) (s)
7. The segmentation Image from the trained model output (c)
8. The overlapping images of the final image of random samples and the final prediction image.

I have some questions for you to add a Wechat<PHONE_NUMBER>8

Your question: Can you tell me how many steps you trained it?
My Answer:
The training diffusion step was 1000. I follow the below script with the arguments posted on the README page.

For training, run: python scripts/segmentation_train.py --data_name ISIC --data_dir input data direction --out_dir output data direction --image_size 256 --num_channels 128 --class_cond False --num_res_blocks 2 --num_heads 1 --learn_sigma True --use_scale_shift_norm False --attention_resolutions 16 --diffusion_steps 1000 --noise_schedule linear --rescale_learned_sigmas False --rescale_timesteps False --lr 1e-4 --batch_size

The training session generated the last model called "emasavedmodel_0.9999_135000.pt."
Here are the training logs.
log.txt
progress.csv
I ran the below script with the training model and the arguments to generate the above image. I used Brast2020 data.
python scripts/segmentation_sample.py --data_dir /data/BraTS2020/Data/MICCAI_BraTS2020_ValidationData/ --out_dir ./results_BraTS2020_sample_dpm_solver_0.9999_135000 --model_path ./results_BraTS2020/emasavedmodel_0.9999_135000.pt --data_name BRATS --image_size 256 --num_channels 128 --class_cond False --num_res_blocks 2 --num_heads 1 --learn_sigma True --use_scale_shift_norm False --attention_resolutions 16 --diffusion_steps 50 --dpm_solver True --noise_schedule linear --rescale_learned_sigmas False --rescale_timesteps False --num_ensemble 5 --multi_gpu 0,1 --debug True
Your Quesiton: Also, is the last but one Ground truth or the sample segmentation from the model? <-- This is my question too.
My answer:
I think your script "segmentation_sample.py" line 126 and lines between 153 and 159 assign the output images.
Here is the image of the code:
Code:
Line: 126 co = th.tensor(cal_out)
Line: 153-159 s = th.tensor(sample)[:,-1,:,:].unsqueeze(1) m = th.tensor(m.to(device = 'cuda:0'))[:,0,:,:].unsqueeze(1) o1 = th.tensor(org)[:,0,:,:].unsqueeze(1) o2 = th.tensor(org)[:,1,:,:].unsqueeze(1) o3 = th.tensor(org)[:,2,:,:].unsqueeze(1) o4 = th.tensor(org)[:,3,:,:].unsqueeze(1) c = th.tensor(cal)
I checked each output image s, m, o1, o2, o3, o4, c. I think these output images were defined below. I am pretty sure that my definitions of the output images between 1 through 5 below is correct, but I am not sure about the rest of the output images. I might be totally wrong though. I would like to confirm with you that all my definitions of the output images are correct.
1: BraTS20_Validation_001_t1.nii.gz (o1) [normalized] 2. BraTS20_Validation_001_t1ce.nii.gz (o2) [normalized] 3. BraTS20_Validation_001_t2.nii.gz (o3) [normalized] 4. BraTS20_Validation_001_flair.nii.gz (o4) [normalized] 5. BraTS20_Validation_001_t1.nii.gz (m) [not normalized] 6. The final image of random samples from the model (p_sample_loop_progressive from 0 through 999) (s) 7. The segmentation Image from the trained model output (c) 8. The overlapping images of the final image of random samples and the final prediction image.
Here is my input data looks like.

Doesn't this m represent a real mask diagram?

I generated a debug output image like below while I was testing. I defined each image of the debug output images listed below. Is my definition of the output images correct? Please correct me if my definition is incorrect.
Output Image: Images between 1 through 4 are normalized.
1: Original image t1 (o1)

Original Image t1ce (o2)
Original image t2 (o3)
Original Image flair (o4)
Original Image t1 without normalization (m)
The sample image from the model (each time step of diffusion from 0 through 999) (s)
The segmentation Image from the trained model output (c)
An overlapping images of the final image of random samples and the final prediction image.


Performance issues

Predictably, trying to make a graphically intensive game in HTML is catastrophically bad for performance. As much as I like using newfangled CSS stuff, I suggest moving everything in the pony field to HTML canvas rendering, which is better optimized and doesn't rely on DOM manipulation to achieve things.
In the short term, pinkie GIFs are apparently extremely expensive on some machines, for some reason, and there should be an option to replace them with a static image.
A nice short term solution is to simply replace all SVGs with canvas elements that have an SVG pre-rendered on them.

The canvas caching part of this was implemented in v1.0.4


Filtering does not work inside with clause

How are you accessing AGE (Command line, driver, etc.)?

Command line

What is the command that caused the error?
SELECT * from cypher('my_graph_name', $$
 WITH 1 as n
 WHERE n > 1
 RETURN n
$$) as (n agtype);

Expected behavior
It should return nothing, it is returning 1 row.

I believe this issue is related to #351 and #238.


compile issue

ui/main.c:88:4: error: implicit declaration of function 'UI_PrintStringSmallBold'; did you mean 'UI_PrintStringSmall'? [-Werror=implicit-function-declaration]
88 |    UI_PrintStringSmallBold(s, 2, 0, line);  // issue UI_PrintStringSmallBold  //UI_PrintStringSmall(s, 2, 0, line);
|    ^~~~~~~~~~~~~~~~~~~~~~~
|    UI_PrintStringSmall
cc1.exe: all warnings being treated as errors
make: *** [ui/main.o] Ошибка 1

app/dtmf.c:418:29: error: 'g_dtmf_call_state' undeclared (first use in this function); did you mean 'gDTMF_CallState'?
418 |                         if (g_dtmf_call_state != DTMF_CALL_STATE_NONE           ||
|                             ^~~~~~~~~~~~~~~~~
|                             gDTMF_CallState
app/dtmf.c:418:29: note: each undeclared identifier is reported only once for each function it appears in
app/dtmf.c:419:29: error: 'g_current_vfo' undeclared (first use in this function); did you mean 'gCurrentVfo'?
419 |                             g_current_vfo->DTMF_ptt_id_tx_mode == PTT_ID_APOLLO ||
|                             ^~~~~~~~~~~~~
|                             gCurrentVfo
app/dtmf.c:423:33: error: 'g_dtmf_reply_state' undeclared (first use in this function); did you mean 'gDTMF_ReplyState'?
423 |                                 g_dtmf_reply_state = DTMF_REPLY_NONE;
|                                 ^~~~~~~~~~~~~~~~~~
|                                 gDTMF_ReplyState
make: *** [Makefile:371: app/dtmf.o] Error 1
compile from docker

I don't even know what docker is. need to delete those two files.

arm-none-eabi-size firmware
text    data     bss     dec     hex filename
57268      28    2816   60112    ead0 firmware
docker
FROM archlinux:latest
RUN pacman -Syyu base-devel --noconfirm
RUN pacman -Syyu arm-none-eabi-gcc --noconfirm
RUN pacman -Syyu arm-none-eabi-newlib --noconfirm
RUN pacman -Syyu git --noconfirm
RUN pacman -Syyu python-pip --noconfirm
RUN pacman -Syyu python-crcmod --noconfirm
WORKDIR /app
COPY . .
#RUN make && cp firmware* compiled-firmware/

???

ui/main.c:88:4: error: implicit declaration of function 'UI_PrintStringSmallBold'; did you mean 'UI_PrintStringSmall'? [-Werror=implicit-function-declaration] 88 | UI_PrintStringSmallBold(s, 2, 0, line); // issue UI_PrintStringSmallBold //UI_PrintStringSmall(s, 2, 0, line); | ^~~~~~~~~~~~~~~~~~~~~~~ | UI_PrintStringSmall cc1.exe: all warnings being treated as errors make: *** [ui/main.o] Ошибка 1

UI_PrintStringSmall(s, 2, 0, line);  // issue   edit :UI_PrintStringSmall(s, 2, 0, line);
thats work

Just had a look, you uploaded that change yourself.

I wana help

Yes sure, no problem :)

Thanks for you work

try it now, just committed (myself ;) )


[hw] Fix the handling logic for multiple queries and add some return type hints for some methods

Summary

When executing a list of queries, the results of execution-only query do not need to be added to the final returned query results.
For some methods, missing return type hints are added back.
Tests

CI
cc:

@dy46 @wangxiaoyou1993 @tommydangerous

@wangxiaoyou1993 please take a look and see if this PR is safe to merge, thanks.


Typo write_t instead of read_t

https://github.com/codeplaysoftware/SYCL-For-CUDA-Examples/blob/da0df38563559d4d6d303bfcc3d52a61fca9f003/example-01/vector_addition.cpp#L81
This should be read_t

This has been fixed in 92e51949


[Addition]: False Bobcat

Hotspot b58 Addresses
11276xHYDDvbhuKWSzXEmSuS84XwrdskVCMBnm77CvEtiAVUZTg
112BDMXX97iaDDoTiy3nAihX2ZkeLNxbVmyP1pSLpk2zXspjWcou
112jKqh5nz3XTd3eoPwD8bLt2wVSXisY6GMSiGaGmxwJfZwZUgZq
11SRkKsWC1NihPSJGAbqt9Uc2NN7N44KPerrhZN6VNGo87BfRdK
11TVHVBhHasa1QYPgFv3JPzPy69VLXR7EpJTBMBSRSz8xpxexvx
11UuTGjGDq3VyCPQqZ8DiomuGngpVYiyjyMThqgQ6FtgKbYxwqK
Discord Handle
Bobcat
Reason(s)
Several of the devices were deployed in the mountains, which were clearly false @KXI1

@6CornerDefender

@KXI1

it's nice to be a dog

@6CornerDefender

@6CornerDefender

@6CornerDefender

@6CornerDefender


Generating LALR tables takes too long

Hello everyone,
First of all, thank you for the great repo. :)
Now to my issue:
When I run the generated python code, it takes nearly 2 seconds for the code to execute.
This means if I create an empty .py file and just import smop, it takes nearly 2 seconds for it to execute.
#!/usr/bin/python
# Autogenerated with SMOP
from smop.core import *

The following is what I see in terminal when executing the above code:
WARNING: Couldn't open 'parser.out'. [Errno 20] Not a directory: '/usr/local/lib/python2.7/site-packages/smop-0.0.0-py2.7.egg/smop/parser.out'
WARNING: Token 'END_UNEXPECTED' defined, but not used
WARNING: Token 'CLASSDEF' defined, but not used
WARNING: There are 2 unused tokens
Generating LALR tables
WARNING: Couldn't create 'smop.parsetab'. [Errno 20] Not a directory: '/usr/local/lib/python2.7/site-packages/smop-0.0.0-py2.7.egg/smop/parsetab.py'

Most of the spent time is after "Generating LALR tables".
Where do you think the problem lies?
Thanks

Usually only the first run after making changes takes a while.  Then it caches the parse tables and the compilation is really fast. If you see that "generating lalr tables" repeatedly I want to see it.  But chances  are the problem already disappeared.

@aminakbari , Hi, i came across the same error:
couldn't create '.parsetab'. [Errno 20] Not a directory...
how do you solve it?

@aminakbari , Hi, i came across the same error:
couldn't create '.parsetab'. [Errno 20] Not a directory...
how do you solve it?

At that time I wasn't able to solve it but I just took another look and well, I found the solution.
Navigate to the "site-packages" folder. cd /usr/local/lib/python2.7/site-packages/
As you can see, smop is a file not a folder. A ".egg" file is a zip so just unzip it using unzip smop-0.0.0-py2.7.egg.
Now rename the just created folder "smop" to "smop-0.0.0-py2.7.egg". mv smop smop-0.0.0-py2.7.egg
Now run your code again. On the first execution, it takes some time for the LALR tables to be generated but the next executions will be much faster.

@aminakbari , thank you, it has been ok now.
but i find every generated .py files contain 'from libsmop import *',
so does it have to include libsmop  if I want to run the .py file?

@aminakbari  I have similar issue and can't resolve, could you please see your email?


Board Review: Storage STG73 design discussion

This is an issue to track the first arch board meeting for adding Storage STG73 features to the storage libraries.  @amishra-dev will do a walk through of QuickQuery and ChangeFeed (including planned strategies with Avro) and @pemari-msft will walk the board through the rest of the STG73 features.

Scheduled for Mar-5

Notes from the arch board meeting:  https://msit.microsoftstream.com/video/6417af66-9f5e-46ff-9118-db0c9e5f830e
New features:

getPageReanges continuation token
Sealed blobs (write for a time, then seal and they become read only)
object replication service
container restore
versioning (automatic snapshots / AWS: stable versioning)

Can I SAS to a specific version?  Yes, you can

blob tags

get / set tags & filter blobs by tag
will be used in attribute based access control (sept. timeframe)

Jumbo blogs (much larger block and blob sizes)

KC - which of these features will result in complex design changes?  Quick Query & Change Feed
BT - concerns about lots of related concepts - automatic snapshots, e.g.
JS - Is the set tags "replace" or "merge" - we think it's merge, but service team will check.
PM - KeyVault moved to soft delete by default.  Is that the plan?  Yes, but there are issues in billing / retention / compliance to resolve.
Quick Query

support searching over CSV, JSON (and future AVRO, Parkey)
data is returned in AVRO formatt
How many customers?  Unclear
Can I have errors for data that still gets returned?  (i.e. do I need to correlate)

For each row, it's either an error or a result.

Should this be exposed through Synapse to the exclusion or as well?  (as well)

Synapse is building on top of it

Why CSV instead of JSON (it searches both)
Would it be 1 or 2 APIs (1 API)
Should this be part of the core SDK?

Thinking different library, but @tg-msft thinks same library
Used by blobs, data lake and potentially files
There is only really query method (with overloads)
BlockBlobClient is already in specialized namespace.
New libraries have versioning challenges.

Do we call it QuickQuery or just Query (just Query - avoid branding)
AVRO parsing library for JS is 250Kb, which is a problem.

RECOMMENDATION: Place query on blob client in same package.
Change Feed

Ashish used a presentation and will attach later
Separate client, or combined with blobs?   Separate client
Why not a regular blob download - it's not just reading a blob
We should look at EH EventProcessor class, and Cosmos Change Feed for inspiration

RECOMMENDATION: Use a separate client & package

Should pump to EH be an option?

Change Notifications is an existing feature, pumps to EH or EG (not sure which)
Service team will look into it

Where should the AVRO parser live?

It won't be exposed, so internally it's an implementation detail.
Probably Azure.Core eventually.


chore: fix misc config for new client surfaces

see https://github.com/googleapis/google-cloud-php/pull/6197
Manually submits the following new client surfaces added in the above PR:

ArtifactRegistry - _https://github.com/googleapis/google-cloud-php/pull/6259_
AssuredWorkloads - added in https://github.com/googleapis/google-cloud-php/pull/6259
BinaryAuthorization - added in https://github.com/googleapis/google-cloud-php/pull/6252
Domains - added in https://github.com/googleapis/google-cloud-php/pull/6260
GkeHub - added in https://github.com/googleapis/google-cloud-php/pull/6240 (see cl/533145281)
NetworkSecurity - added in https://github.com/googleapis/google-cloud-php/pull/6261
Notebooks - added in https://github.com/googleapis/google-cloud-php/pull/6261
SqlAdmin - added in https://github.com/googleapis/google-cloud-php/pull/6256
WebSecurityScanner - added in https://github.com/googleapis/google-cloud-php/pull/6257
Workflows handled by https://github.com/googleapis/google-cloud-php/pull/6263

I am not sure why OwlBot did not submit PRs for these, so I generated them manually.

This is no longer necessary


setBounce() is working on the fly

get fixture and use SetRestitution() to make setBounce()
work on existing body

need to check for body != nullptr


chore!: add usage trends of Crawlee package suite

I am an enthusiastic user of the Crawlee package, and I truly appreciate all the hard work and dedication that has gone into its development. Crawlee has been an invaluable tool in my projects.
I'd like to propose a small addition to the README to benefit new users. This could involve highlighting the different packages within Crawlee or showcasing its popularity to attract more users.
I believe this small addition will provide significant value to both new and existing users of Crawlee. I hope you consider merging this change to help enhance the user experience and visibility of the Crawlee package suite.

Thanks for the PR but we are not interested in adding this to the readme, I personally don't see a value in having it there.
Also note that the changes are not even valid, you are combining HTML with markdown.

@B4nan it's legit to write HTML (or a restricted subset of it) in Markdown in some cases, for example when you want to do target="_blank" :slightly_smiling_face: But I do agree we don't need this.

What the PR is doing is not correct, you can't put markdown inside HTML attribute.

What the PR is doing is not correct, you can't put markdown inside HTML attribute.

Oooh, right, now I see it!


unkown variable "startVideo"

it does not exist.
i have the 0.4 build, was this made in 0.4.1?

On 0.4 it was named "videoIntro"


Some mistake within Wiki

See here.
We try to add a flutter module into our existed app. PluginRegistrant won't work correctly because AppDelegate's root view controller isn't a flutterViewController.
I saw this part of wiki. But the wiki doesn't explained it clear.

@WilliamZang
Please create a github demo project which might help us to reproduce your issue and improve the wiki.


openimageio2: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes
update openimageio2 package
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg eval

@ofburg build

Closing since we already have openimageio <IP_ADDRESS> in the repo.


Simulated Datasets outputs stars that are too big with psf_slen smaller than padded tile size

Just yesterday I found what I think is a bug in the ImageDecoder in decoder.py. The stars are larger than they should be when psf_slen has size smaller than the padded tile.
Using the default configuration in sdss_galaxies/config/config.py which uses the psf_params_file from the usual SDSS frame with psf_slen=25 I get:

This is clearly wrong as the SDSS PSF from the corresponding frame has a FWHM size of approximately 1.4'' = 3.5 pixels
If I set psf_slen=53 I get

In any case, I don't think psf_slen should affect the size of the stars in the final image but it currently does.
I wonder if this has always been a problem or introduced recently 🤔 suggestions welcome

Once this issue is resolved, we should add a test to ensure this doesn't happen again (I remember this being an issue some time back and fixing it, so I'm surprised it surfaced again)

Does this problem occur with the old version of the decoder before the switch to psf_decoder?


Any plans to update this example to use XRI 3.0+

Are there any plans to update this demo to XR Interaction Toolkit 3.0+?

I forked the repo and took  a pass but I am not able to get the grab to work properly.


Per Receiver Integration Tests

Is your feature request related to a problem? Please describe.
The HL7 messages that ReportStream sends to a state go through an acceptance process by the state before the feed is turned on. Every change to the system has the potential to make ReportStream's HL7 messages fall out of acceptance for a state. While it is impractical to have the state PHD go through an acceptance process on every check-in, we can run per-state integration tests on every check-in.
Describe the solution you'd like
I envision a series of tools to create and update the tests

Sender-Files: to fetch source files for a particular file sent to a state
Preview: to run the RS pipeline with source files
HL7-Compare: To compare two hl7 files with a set of acceptable variations
Generate-Test: Using the previous tools, generate a per receiver integration test

Describe alternatives you've considered
Additional context
The attached presentation further explains the concept.
State Regression Testings.pdf
#3394 Describes a problem that will be fixed by this epic.

Please note that there is an integration test that does all these functions (I think), so check that first.

@oslynn Per Carlos' comment has this already been done? Close?


[T5A2][W15-B3]Zachary Lim Wei Ling

@zachylimwl
Some comments added. Please ack & close the PR after reading comments.


Problem with event for button in the table

Hello all,
unfortunately I have the problem with button in the the table. My function for "click " does not run on the real click. But if comment the part of code : data-role="table", then it works
Thanks
admin-user-html.txt
admin-user-ts.txt

use leave event:
$("table_selector").on("click", "button_selector", function (e){...})

I'm new in angular and don't understand the answer.
Where exactly should I write this.
And it seems it is not typescript.

You must use delegate events. Add event handler for click to table element, and delegate it to buttons in cells.
About event delegation https://javascript.info/event-delegation#:~:text=Capturing and bubbling allow us,handler on their common ancestor.


Implemented other regions and revisions support of Metroid Prime

Supporting now each GC versions. Wii versions require some work on the patcher.

Nice, this is looking good. I'll see if I can pull it down in the next day or so and  run it locally 🙏

Merging and going to put up a pre release for testing 👍 Thanks again!!


Added function to search between two dates

Created a function that allows to search between two dates. This allows to sort of bypass the max of 1000 listings.

 Clean it up.
On Mon, Jun 6, 2016 at 7:29 AM, Jacob Fredsoe<EMAIL_ADDRESS>wrote:

Created a function that allows to search between two dates. This allows to
sort of bypass the max of 1000 listings.
You can view, comment on, or merge this pull request online at:
https://github.com/SirCmpwn/RedditSharp/pull/225
Commit Summary

Added function to search between two dates

File Changes

M RedditSharp/Reddit.cs
https://github.com/SirCmpwn/RedditSharp/pull/225/files#diff-0 (894)

Patch Links:

https://github.com/SirCmpwn/RedditSharp/pull/225.patch
https://github.com/SirCmpwn/RedditSharp/pull/225.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SirCmpwn/RedditSharp/pull/225, or mute the thread
https://github.com/notifications/unsubscribe/AFeht1kIaMBe5y5B51e6j-dAnYbLU5nrks5qJBKYgaJpZM4Iu2AO
.

Indeed it does. I am quite new to git, so how do I go about fixing it?


Explicitly initialise loop counter variables in 'for' loops.

The included test fails on my machine unless you explicitly initialise counter variables to 0.

Thanks for catching that and for the PR :slightly_smiling_face:


[ISSUE #3556] downgrade curator version to 4.3.0 to compatible with some plugin like sofa

For #3556 , all tasks should be finished and the pom should be cleaned. I also changed zookeeper dep to 3.5.7 because the curator ver 4.3.0 depends on it. I've tested below functions and they're works well. Welcome to help to test these or others test cases!

[x] data sync between admin and bootstrap
[x] register between admin and sofa client
[x] bootstrap instance
[x] sofa example works well

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3569 (84a3649) into master (dc2a159) will increase coverage by 0.05%.
The diff coverage is 63.23%.

@@             Coverage Diff              @@
##             master    #3569      +/-   ##
============================================
+ Coverage     62.54%   62.59%   +0.05%
- Complexity     5562     5566       +4
============================================
  Files           848      848
  Lines         23465    23491      +26
  Branches       2132     2139       +7
============================================
+ Hits          14676    14705      +29
+ Misses         7436     7428       -8
- Partials       1353     1358       +5

Impacted Files
Coverage Δ

...nyu/register/client/zookeeper/ZookeeperClient.java
58.97% <40.00%> (-2.67%)
:arrow_down:

...u/register/instance/zookeeper/ZookeeperClient.java
58.97% <40.00%> (-2.67%)
:arrow_down:

...ister/client/server/zookeeper/ZookeeperClient.java
69.23% <66.66%> (-4.75%)
:arrow_down:

...eeper/ZookeeperClientServerRegisterRepository.java
94.59% <90.47%> (-2.24%)
:arrow_down:

...admin/listener/zookeeper/HttpServiceDiscovery.java
90.32% <100.00%> (+1.61%)
:arrow_up:

...controller/ShenyuClientHttpRegistryController.java
77.77% <0.00%> (-22.23%)
:arrow_down:

...ruptor/RegisterClientServerDisruptorPublisher.java
52.94% <0.00%> (-11.77%)
:arrow_down:

...nyu/plugin/motan/cache/ApplicationConfigCache.java
52.00% <0.00%> (-5.00%)
:arrow_down:

.../logging/rocketmq/utils/LogCollectConfigUtils.java
71.11% <0.00%> (-4.45%)
:arrow_down:

...ng/rocketmq/rocketmq/RocketMQLogCollectClient.java
74.00% <0.00%> (-2.00%)
:arrow_down:

... and 5 more

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 dc2a159...84a3649. Read the comment docs.


style(input): fix lints

Short description of what this resolves:
Changes proposed in this pull request:

Ionic Version: 1.x / 2.x
Fixes: #

Thanks 😸


Revert to previous version of pagefind

The current pagefind app (1.1.0) generates variable output data for the same input.
This results in spurious changes to the website and in the commit emails.
This PR reverts to the previous version (1.0.4), which has much more stable output.
Note that the change to 1.1.0 was not done to fix any problems, merely it seemed best to use the latest version.

👋 Pagefind 1.3.0 fixes the bulk of this issue 🙂


Error building cpu_adam

Hi,
I apologize if it is a duplicate issue. I just pip installed deepspeed with pytorch 1.11. But I am still having issues with cpu_adam.
python -c "import deepspeed; deepspeed.ops.op_builder.CPUAdamBuilder().load() "
Installed CUDA version 10.0 does not match the version torch was compiled with 10.2 but since the APIs are compatible, accepting this combination
Using /home/ubuntu/.cache/torch_extensions/py38_cu102 as PyTorch extensions root...
Detected CUDA files, patching ldflags
Emitting ninja build file /home/ubuntu/.cache/torch_extensions/py38_cu102/cpu_adam/build.ninja...
Building extension module cpu_adam...
Allowing ninja to set a default number of workers... (overridable by setting the environment variable MAX_JOBS=N)
[1/2] c++ -MMD -MF cpu_adam.o.d -DTORCH_EXTENSION_NAME=cpu_adam -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE=\"_gcc\" -DPYBIND11_STDLIB=\"_libstdcpp\" -DPYBIND11_BUILD_ABI=\"_cxxabi1011\" -I/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/deepspeed/ops/csrc/includes -I/usr/local/cuda/include -isystem /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include -isystem /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/torch/csrc/api/include -isystem /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/TH -isystem /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/THC -isystem /usr/local/cuda/include -isystem /usr/include/python3.8 -D_GLIBCXX_USE_CXX11_ABI=0 -fPIC -std=c++14 -O3 -std=c++14 -g -Wno-reorder -L/usr/local/cuda/lib64 -lcudart -lcublas -g -march=native -fopenmp -D__AVX256__ -c /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/deepspeed/ops/csrc/adam/cpu_adam.cpp -o cpu_adam.o
FAILED: cpu_adam.o
c++ -MMD -MF cpu_adam.o.d -DTORCH_EXTENSION_NAME=cpu_adam -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE=\"_gcc\" -DPYBIND11_STDLIB=\"_libstdcpp\" -DPYBIND11_BUILD_ABI=\"_cxxabi1011\" -I/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/deepspeed/ops/csrc/includes -I/usr/local/cuda/include -isystem /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include -isystem /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/torch/csrc/api/include -isystem /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/TH -isystem /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/THC -isystem /usr/local/cuda/include -isystem /usr/include/python3.8 -D_GLIBCXX_USE_CXX11_ABI=0 -fPIC -std=c++14 -O3 -std=c++14 -g -Wno-reorder -L/usr/local/cuda/lib64 -lcudart -lcublas -g -march=native -fopenmp -D__AVX256__ -c /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/deepspeed/ops/csrc/adam/cpu_adam.cpp -o cpu_adam.o
In file included from /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/torch/csrc/Device.h:3:0,
                 from /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/torch/csrc/api/include/torch/python.h:8,
                 from /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/torch/extension.h:6,
                 from /home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/deepspeed/ops/csrc/adam/cpu_adam.cpp:5:
/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/include/torch/csrc/python_headers.h:10:10: fatal error: Python.h: No such file or directory
 #include <Python.h>
          ^~~~~~~~~~
compilation terminated.
ninja: build stopped: subcommand failed.
Traceback (most recent call last):
  File "/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1740, in _run_ninja_build
    subprocess.run(
  File "/usr/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['ninja', '-v']' returned non-zero exit status 1.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/deepspeed/ops/op_builder/builder.py", line 470, in load
    return self.jit_load(verbose)
  File "/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/deepspeed/ops/op_builder/builder.py", line 512, in jit_load
    op_module = load(
  File "/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1144, in load
    return _jit_compile(
  File "/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1357, in _jit_compile
    _write_ninja_file_and_build_library(
  File "/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1469, in _write_ninja_file_and_build_library
    _run_ninja_build(
  File "/home/ubuntu/nlp_prompting/env/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 1756, in _run_ninja_build
    raise RuntimeError(message) from e
RuntimeError: Error building extension 'cpu_adam'

What is the easiest way to solve it?

Hi @arijitthegame, thanks for reporting your issue. In this case I see the error is related to not being able to find Python.h. I think you'll want to make sure you have python-dev installed. DeepSpeed compiles several custom cuda/cpp kernels which have python bindings.
Can you try installing python-dev? Examples: https://stackoverflow.com/questions/21530577/fatal-error-python-h-no-such-file-or-directory

Thank you so much for your reply. When I try to install python-dev, I get
python3-dev is already the newest version (3.6.7-1~18.04).


Errors in Connect task while try to reject call.

We are trying to create a function to reject a incoming call using amazon connect streams v2.0 but it is giving error.
[ERROR]: 'rejectContact' API request failed: Error
we are using this code...
function rejectCall()
{
  let agent = new connect.Agent();
  let contact = agent.getContacts()[0]
  contact.reject({
    success: function() {console.log("success") },
    failure: function(err) {  console.log("Error") }
  });
}

Hello, when are you trying to reject the contact? If the contact has already been connected, the rejected operation will fail with a 4xx error.

We are trying to create a function to reject a incoming call using amazon connect streams v1.0 they work properly.
we are using destroy() method like below.
function rejectCall()
{
  var initialConnection = agentContact.getInitialConnection();
  var thirdParty = agentContact.getSingleActiveThirdPartyConnection();
  if (initialConnection) {
    initialConnection.destroy();
  }
  if (thirdParty) {
    thirdParty.destroy();
  }
}


Use the same client-request-id for retry requests

Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[x] C++ Guidelines
[x] Doxygen docs
[x] Unit tests
[x] No unwanted commits/changes
[x] Descriptive title/description

[x] PR is single purpose
[x] Related issue listed

[x] Comments in source
[x] No typos
[x] Update changelog
[x] Not work-in-progress
[x] External references or docs updated
[x] Self review of PR done
[x] Any breaking changes?

Client-Request-Id is set by the RequestIdPolicy which is in the per-call policy list, so it should be stable across retries.
Are you seeing that the x-ms-client-request-id value is changed across retries? If so, that points to a more serious bug in Azure Core.

Should we add a unit test for this scenario? (retry with the same client request id)

@LarryOsterman

Client-Request-Id is set by the RequestIdPolicy which is in the per-call policy list, so it should be stable across retries.

I generally agree with this. Storage sdk has a few exceptions:

A batch request has a client request id. Each subrequest also has its own client request id. These ids are different from each other. The id for the parent request persists across retries, but the ids for the subrequests do not.
Concurrent download API sends requests to download from different offset with multiple threads. These requests all have its own client-request id (the id persists across retries).
When downloading a large blob, the transfer of response body could be interrupted after client-side receives 200 http status code. The retry policy won't kick in because the status code indicates a successful result. In this case, storage SDK will send a new request to download from the offset where it was interrupted. The client request id should've persisted for this kind of retry because it'd be super helpful for server-side troubleshooting. Unfortunately, we generate new client request ids for this kind of retries and this is the problem I'm trying to fix in this PR.

Should we add a unit test for this scenario? (retry with the same client request id)

I definitely have tested this locally. However, to get it in unit test, we need:

a way to kill a TCP connection.
a blob that is large enough so that the download procedure won't finish until we interrupt the connection.

Unfortunately, neither is available in our unit-test framework.


TypeError: Cannot read property 'tag' of undefined

Hi!
Today I wanted to build a new library and use auto-changelog. In my package.json I referenced "^2.0.0". Npm installed the new version (2.1.0), which is at first no problem. I have this line in my version script of the pacakge.json: auto-changelog -p && git add CHANGELOG.md.
Running the commands as in the README, will break now my release chain:
$ auto-changelog -p
auto-changelog: 0 version tags found…

TypeError: Cannot read property 'tag' of undefined
    at parseReleases ([...]/node_modules/auto-changelog/src/releases.js:49:33)
    at run ([...]/node_modules/auto-changelog/src/run.js:99:26)

Investigating the code shows that there is an index problem: tags[0] is undefined.
Rolling back to "2.0.0" manually made the auto-changelog package run again, so it seems as if the bug was introduced with the recent version release.

I have the same issue.
ERROR TypeError: Cannot read property 'tag' of undefined
    at parseReleases (/usr/local/lib/node_modules/auto-changelog/src/releases.js:49:33)
    at run (/usr/local/lib/node_modules/auto-changelog/src/run.js:99:26)

This should be fixed in 2.2.0 for now. I need to tidy up all of the logic around parsing tags.


I have filled up my redis db

Not sure if I have had a bug or not, but I noticed that I have absolutely thousands of keys in my redis database associated with busmq.
The keys are all called "bus:queue:<unique_queue_name>:msgid" and have a value of 1.
What are they and, more importantly, how do I get rid of them?
Redis reports that they have a TTL of -1. When I attach to a queue I leave the defaults (30 seconds) but during testing I may well have had bugs where I attached but failed to detach.

The sequence of events is that I have a client and a server (both child-processes).
The client creates a uniquely named queue, attaches to it and consumes it.
It sends a message to the server - the message contains the name of the queue.
The server responds on the queue specified (with a push()) then instantly and detaches from it.
The client receives the message and also detaches from it:
rq.on("message",function(response,id){
blah;
rq.detach();
})
Each time, a single key is left in the redis DB containing the:
bus:queue:myuniqueq:msgid

Is that the only busmq key that leaks?
Can you provide actual code that reproduces this behaviour?
What version of redis are you using?

I will try to create a little test script that replicates it.
This is a windows 10 PC, so it uses Redis 2.8.2400

Here is an app that will leak one key as described above
busmq.zip

You are not detaching from outq in app.js. Is that the queue that's leaking?

No it is not - it is the queue in am specifically detaching from that is leaking ("inq").
The other queue disappears completely from redis 30 seconds after app.js has terminated.

you indeed found a bug!
will be pushing a fix shortly. thanks!

published version 1.1.3 with the fix. Please give it a try

Perfect :)
Thanks


Fix RoboGuice injection issues

RoboGuice was integrated in #273. However, some injections were not set up properly. The null pointer exceptions that were caused by this were swallowed by the assorted generic exception catching blocks that are scattered throughout the codebase. This commit attempts to fix these issues:

TranscriptDownloader did not set up RoboGuice to initialize it's injection of the ServiceManager instance. This has been fixed. Related to this, TranscriptManager has also been converted into a RoboGuice singleton, and injected appropriately.

BrowserUtil was instantiated manually instead of being injected, and thus it's injection of the IEdxEnvironment instance was not being initialized as well. It has now been converted into a static utility method, with the IEdxEnvironment instance injected as a static field from our RoboGuice module.

https://openedx.atlassian.net/browse/MA-1114

@bguertin @aleffert please review

Would it be more consistent to make BrowserUtil a singleton instead of static? Otherwise :+1:

All the other utility classes are static, so my change makes it consistent with them.

On 17-Sep-2015, at 7:53 pm, Akiva Leffert<EMAIL_ADDRESS>wrote:
Would it be more consistent to make BrowserUtil a singleton instead of static? Otherwise
—
Reply to this email directly or view it on GitHub https://github.com/edx/edx-app-android/pull/367#issuecomment-141112282.


[otbn,rtl] Fix locking on RMA request while refreshing URND

Prior to this PR, when otbn_start_stop_controller got an RMA
request during a pre-execution URND refresh, it would start executing as
soon as the URND refresh is completed.  This is wrong: when OTBN gets an
RMA request during a pre-execution URND refresh, it must securely wipe
as soon as the URND refresh is completed (and then lock).
This bug was caused by the if condition on line 169 of
otbn_start_stop_controller.sv, which, if rma_request is true, is
false and thus the else branch on line 180 is taken.
This PR removes this bug by explicitly splitting the if statement
for proceeding after an URND refresh on the stop signal: if that
signal is high, OTBN either locks immediately or waits for the URND
refresh to complete and then securely wipes (and then locks).  The
behavior of the else branch is preserved.
The code is not fully DRY as the inner else bodies are redundant.
However, I explicitly prefer this implementation to a fully DRY one,
because it keeps the stop and the non-stop cases clearly separated.
This PR contains a second commit with a small improvement on handling Mubi-encoded RMA requests:
OTBN should only handle a Mubi-encoded RMA request if it is strictly
true, thereby reducing the potential for FI on RMA requests.  OTBN
already detected Mubi-invalid RMA requests and would lock if such a
(likely tampered) request is detected.
DV Note
The first commit causes the otbn_escalate test to fail. As that test anyway currently fails for many seeds, I don't consider this to be a blocker for this PR, though. I'll continue improving that test in separate PR(s).

even if it means that the FSM then first go into the OtbnStartStopStateRunning state to take rma_request and perform another URND refresh.

The main problem with this is that it causes the OTBN locking signal to be high until the refresh is completed, then become low, and then later high again. That gets currently reflected in the STATUS register, which goes from LOCKED to BUSY_EXECUTE, which is outside the specification.

Ah, I see. Thanks for explaining @andreaskurth . Your solution is definitely better then!

The CI failures seem unrelated/spurious (e.g., failing Bazel fetches).


Webpack's alias doesn't work when target module is in node_modules

Bug report

What is the current behavior?
alias doesn't work when target module is in node_modules folder
If the current behavior is a bug, please provide the steps to reproduce.
Background
I'm writing an SSR react app with typescript, there's a client-side module I have to use (client-logger) and it can only be run on client-side since it's AMD module system. running on Node will give errors.
My current solution is to replace the client-logger module with a rewrite-client-logger module at compile time ONLY for server bundle.
I have tried Webpack's NormalModuleReplacementPlugin and alias, even write my own loader, none of them works, this is my [Webpack config][1] for the alias.
This is the demo link for repro: https://github.com/stanleyyylau/demo
Repro Steps:

Git clone and checkout to Master branch
npm install to install all dependencies
copy client-logger to node_modules with cp -rf ./client-logger ./node_modules
npm run build to trigger the build
npm run start to run the universal app

What is the expected behavior?

Server should be running with no error
Server bundle's client-logger should be replaced by rewrite-client-logger ( You can verify this by searching for the string register event in assets/server.js file.)

Other relevant information:
webpack version:  4.17.2
Node.js version: v8.11.3
Operating System: MacOS 10.12.6 (16G1036)
Additional tools: Check this https://stackoverflow.com/questions/52201186/webpacks-alias-doesnt-work-when-target-module-is-in-node-modules

You flagged client-logger as external. It's never part of the build, so you can't alias it. Remove it from externals

I didn't  flag client-logger as external, where did you see that part?

externals: [nodeExternals()]

See documentation of webpack-node-externals

Getting another error when adding client-logger to whitelist
Thanks for your reply Juho, But it is still not working, after adding it to whitelist, I think it's been replaced, but I'm getting another error:
2018-09-10T02:29:16Z: identifier="routes:build-info" level="WARN" message="Build Info file not found, and hence not set up to inform actual version of the app."
Warning: React depends on requestAnimationFrame. Make sure that you load a polyfill in older browsers. https://fb.me/react-polyfills
path.js:28
    throw new TypeError('Path must be a string. Received ' + inspect(path));
    ^

TypeError: Path must be a string. Received undefined
    at assertPath (path.js:28:11)
    at Object.dirname (path.js:1349:5)
    at Object.<anonymous> (/Users/stanley/Desktop/X/epc-photos-pwa-app/assets/server.js:1:928603)
    at t (/Users/stanley/Desktop/X/epc-photos-pwa-app/assets/server.js:1:419)
    at Object.<anonymous> (/Users/stanley/Desktop/X/epc-photos-pwa-app/assets/server.js:1:928105)
    at t (/Users/stanley/Desktop/X/epc-photos-pwa-app/assets/server.js:1:419)
    at Object.defineProperty.value (/Users/stanley/Desktop/X/epc-photos-pwa-app/assets/server.js:1:926987)
    at t (/Users/stanley/Desktop/X/epc-photos-pwa-app/assets/server.js:1:419)
    at Object.<anonymous> (/Users/stanley/Desktop/X/epc-photos-pwa-app/assets/server.js:1:854113)
    at t (/Users/stanley/Desktop/X/epc-photos-pwa-app/assets/server.js:1:419)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `node --use_strict ./bin/www`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.


How to Obfuscate a Single-File Avalonia Application

I am in the process of releasing a small Avalonia application. In release mode, Obfuscar correctly obfuscates the DLL files, and the application runs without issues. However, in single-file publish, the resulting EXE still contains unobfuscated DLLs. Upon inspecting the DLL files in the obj/release directory, I noticed they are not obfuscated.
Does the publish mode inherently use these unobfuscated DLLs? Despite searching through existing issues, I have not found a viable solution. Is there a more straightforward approach to ensure that the DLLs within the single-file EXE are obfuscated?
Any assistance would be greatly appreciated.
Thank you.

Duplicate to #301


addCue() does not return an ID

I don't know if this is a bug or if I am simply misreading the documentation. When I do the following addCue does not return an "ID" per the documentation, it returns NaN.
  mySound.play();
  let id = mySound.addCue(3, console.log,"we are the world");

  console.log(id); // NaN

Yes this is actually an issue and i know how to fix it. i'll submit a PR with the fix.

should be fixed with #298 - thanks @XeniousSoftware !


MangaNoon

Source information
MangaNoon 1.4.36
Source language
Arabic
Source new URL
axztu.com
Other details
No response
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

Duplicate of #5333
Search the existing issues before opening a new one, and acknowledging that you did when you clearly didn't

Actually, you made that issue as well, don't open another one, wait patiently

sorry for the inconvenience ☹ my level in eng sucks so I don't know how to search on your site, sorry once again ♡


[Question] Can we use a higher torch version?

Question
[Question] Can we use a higher torch version? (like 2.2.0) Thanks.

YES


Support multiple particles at the Arcs APIs

@mmandlis, please take a look. Feel we will need this :)

Thanks for the review!


cage status emphásis on the wrong sylláble

i would argue that you should leave everything the default color EXCEPT exceptional things (EXIT, STOPPED, etc.)
for example, green RUNNING is the normal state, it should be unexceptional
and the pod/service names are hard to read

I'd like to add to this the task of having more concise/clear output and combine cage source ls into cage status.
Proposed format:
     POD       SERVICE     STATUS      IMAGE           LOCAL     REMOTE                            VOLUMES
placeholders
               db         unmounted    postgresql      <none>    <none>
               redis      unmounted    redis           <none>    <none>
web
               api        mounted      myorg/api       src/api   git://github.com/myorg/api        mylib:src/mylib
               ui         unmounted    myorg/ui        src/ui    git://github.com/myorg/ui
               ui_beta    unmounted    myorg/ui_beta   src/ui    git://github.com/myorg/ui#beta

@dkastner The mounted/unmounted state in that design is too simple, because of the ability to mount extra libraries into multiple containers.

@dkastner If you'd like to propose a major overhaul of the output, please feel free to file a separate RFC issue with a design. This issue will be closed once I reduce the coloring a bit.

@seamusabshere I've removed the green from "enabled" (because it's the normal case) and I've made the pods and the services the same color to better support Mac terminals. I'm keeping the bright green for "RUNNING" (because I want it, and it's not the default case, which is is "not running at all"). I'm also keeping a single shade of blue for pod and services names, and we can see how it looks on a variety of screens this week.
A sample:


Make bootstrap flash type mapping configurable

In unobtrusive_flash_bootstrap.js#L15 you cast the rails flash type to bootstrap flash type.
In my current project we use rails notice to display successful messages. Therefore we have to map it like this:
if (options.type=='notice') {
      options.type = 'success';
...

It would be nice if the mapping classes are configurable.

I've opened a pull request for this feature request: #31

@leonid-shevtsov are you still there?

Thank you for merging 👍


Is it possible to get the coordinates of the tap on image?

I would need to get the coordinates of a single tap on a png. Is this possible to do that?

Just saw viewToSource method which seems to do what I need.

@zulluadi  Could you please help describe how you solved it?

Hi Omar, sorry but I can't really remember in which project I used this library. From what I've seen, if you need to get the click position on the image, you should use the viewToSourceCoord method


Buttons in columns

Issue

Not sure how possible this would be, but I am trying to have a column containing a "preview" button for each row, such that when the button is clicked, I open a preview frame displaying a picture and other details related to a row item.
Example: row contains {preview: <material ui preview button>, name: "John", occupation: "miner"}
On preview click, the data in the current row is sent to an iframe/window/popup that is populated with information relating to the row (for example, a templated html page talking about miners and people named John in this case)
When I try to directly push a material-ui button component into the rows, I get an error

TypeError: t[r] is undefined
in t (created by WithStyles(t)) in WithStyles(t)

Is there a known way to insert this kind of button into each row and relate it to the row, or would this be functionality that needs to be added?

I've published 1.1.5 which now has a property for customized columns called 'customRender' and I think this will fit your needs.  An example of how you would use it can be found here


cupy.cuda.compiler.CompileException with pytorch

Thanks for Cupy.

Conditions:

CuPy Version          : 6.5.0
CUDA Root             : /usr/local/cuda-9.0
CUDA Build Version    : 9000
CUDA Driver Version   : 10010
CUDA Runtime Version  : 9000
cuDNN Build Version   : 7603
cuDNN Version         : 7603
NCCL Build Version    : 2307
NCCL Runtime Version  : 2307
python    : 3.6.5
pytorch  : 0.4.0

I tested a simple code like following:
import torch
import cupy as cp

x_gpu = cp.ones((800,800,800))

It tells me:
Traceback (most recent call last):
  File "/data1/zhuzeji/anaconda3/lib/python3.6/site-packages/cupy/cuda/compiler.py", line 242, in compile
    nvrtc.compileProgram(self.ptr, options)
  File "cupy/cuda/nvrtc.pyx", line 98, in cupy.cuda.nvrtc.compileProgram
  File "cupy/cuda/nvrtc.pyx", line 108, in cupy.cuda.nvrtc.compileProgram
  File "cupy/cuda/nvrtc.pyx", line 53, in cupy.cuda.nvrtc.check_status
cupy.cuda.nvrtc.NVRTCError: NVRTC_ERROR unknown (7)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "test_cupy_pytorch.py", line 4, in <module>
    x_gpu = cp.ones((800,800,800))
  File "/data1/zhuzeji/anaconda3/lib/python3.6/site-packages/cupy/creation/basic.py", line 149, in ones
    a.fill(1)
  File "cupy/core/core.pyx", line 482, in cupy.core.core.ndarray.fill
  File "cupy/core/core.pyx", line 500, in cupy.core.core.ndarray.fill
  File "cupy/core/_kernel.pyx", line 571, in cupy.core._kernel.ElementwiseKernel.__call__
  File "cupy/core/_kernel.pyx", line 593, in cupy.core._kernel.ElementwiseKernel._get_elementwise_kernel
  File "cupy/core/_kernel.pyx", line 402, in cupy.core._kernel._get_elementwise_kernel
  File "cupy/core/_kernel.pyx", line 40, in cupy.core._kernel._get_simple_elementwise_kernel
  File "cupy/core/_kernel.pyx", line 60, in cupy.core._kernel._get_simple_elementwise_kernel
  File "cupy/core/carray.pxi", line 166, in cupy.core.core.compile_with_cache
  File "/data1/zhuzeji/anaconda3/lib/python3.6/site-packages/cupy/cuda/compiler.py", line 136, in compile_with_cache
    base = _preprocess('', options, arch)
  File "/data1/zhuzeji/anaconda3/lib/python3.6/site-packages/cupy/cuda/compiler.py", line 97, in _preprocess
    result = prog.compile(options)
  File "/data1/zhuzeji/anaconda3/lib/python3.6/site-packages/cupy/cuda/compiler.py", line 246, in compile
    raise CompileException(log, self.src, self.name, options)
cupy.cuda.compiler.CompileException: nvrtc: error: failed to load builtins

But if without pytorch module, the remainining two lines worked fine!
I have checked  https://docs-cupy.chainer.org/en/stable/install.html#cupy-always-raises-cupy-cuda-compiler-compileexception, there was no any obvious mistakes.
I have run many pytorch networks successfully in this environment before.  And I want to accelerate some image postprocess using Cupy now. Glad to any reply~

After I recreated a new conda env containing the above modules, it worked! Though I don't know what happened.
Thanks again for cupy.