WIP: auto-attach tracepoints based on SEC() data

This is mostly a conversation starter. The work originated in me asking about this on Discord [1].
I can try to flesh this out to a full solution, but I have some questions on what the ideal solution looks like:
Question 1: Currently I am mostly seeing these SEC() annotations in C bpf programs. The automatic template generator of the aya-ebpf tracepoints does not seem to populate the fields (tough it looks like they are supported!). Should we populate those fields there too?
libbpf-tools generates those .skel.h headers that allow automating the attach. I find that a nice QoL improvement over manually having to attach all the probes.
Question 2: I assume we also want to expose public getters for these fields?
Question 3: Do we want to extend this to all of the other "well-known" SEC() annotations that libbpf-tools uses? This would allow us to have a central auto_attach() that simply iterates over all programs and attaches them all?
Open TODOs:

tests
better documentation
review contribution guidelines in detail

[1] https://discord.com/channels/855676609003651072/855676609003651075/1288759816612741120

This is mostly a conversation starter. The work originated in me asking about this on Discord [1].

Hey! Thanks for the PR!

I can try to flesh this out to a full solution, but I have some questions on what the ideal solution looks like:
Question 1: Currently I am mostly seeing these SEC() annotations in C bpf programs. The automatic template generator of the aya-ebpf tracepoints does not seem to populate the fields (tough it looks like they are supported!). Should we populate those fields there too? libbpf-tools generates those .skel.h headers that allow automating the attach. I find that a nice QoL improvement over manually having to attach all the probes.

For things where it makes sense - for example tracepoints that in most cases have exactly 1 attach target - yes I think it makes sense.

Question 2: I assume we also want to expose public getters for these fields?

Yes, see CgroupSkb where we expose already https://docs.rs/aya/latest/aya/programs/cgroup_skb/struct.CgroupSkb.html#method.expected_attach_type

Question 3: Do we want to extend this to all of the other "well-known" SEC() annotations that libbpf-tools uses?

This for sure - we want to be as compatible with libbpf as possible.

This would allow us to have a central auto_attach() that simply iterates over all programs and attaches them all?

This is a separate question, and I think it requires a bit more thinking, eg:
let ebpf = Ebpf::load(...);
ebpf.auto_attach()?; // what happens to the programs that _can't_ be auto attached?

I think it's counter intuitive that some programs wouldn't get attached. Not sure what the solution would be. What does libbpf do?

For things where it makes sense - for example tracepoints that in most cases have exactly 1 attach target - yes I think it makes sense.

I would add k(ret)probe's to the list then :). I can do some research on which programs can be auto-attached in libbpf.

I think it's counter intuitive that some programs wouldn't get attached. Not sure what the solution would be. What does libbpf do?

The code is here: https://github.com/libbpf/libbpf/blob/7984737fbf3b2a14a86321387bb62abb16cfc4ed/src/libbpf.c#L12365-L12404
So it looks like some conventions to prevent autoattachment: https://github.com/libbpf/libbpf/blob/7984737fbf3b2a14a86321387bb62abb16cfc4ed/src/libbpf.c#L747-L757 and some program types are simply not autoattached. Otherwise, a fail to auto-attach will simply cause an error.
That overall seems like reasonable to me!

For things where it makes sense - for example tracepoints that in most cases have exactly 1 attach target - yes I think it makes sense.

I would add k(ret)probe's to the list then :). I can do some research on which programs can be auto-attached in libbpf.

Yep for sure - ideally we get to 100% libbpf compatibility.

I think it's counter intuitive that some programs wouldn't get attached. Not sure what the solution would be. What does libbpf do?

The code is here: https://github.com/libbpf/libbpf/blob/7984737fbf3b2a14a86321387bb62abb16cfc4ed/src/libbpf.c#L12365-L12404
So it looks like some conventions to prevent autoattachment: https://github.com/libbpf/libbpf/blob/7984737fbf3b2a14a86321387bb62abb16cfc4ed/src/libbpf.c#L747-L757 and some program types are simply not autoattached. Otherwise, a fail to auto-attach will simply cause an error.
That overall seems like reasonable to me!

I don't like that in the case of auto_attach() failure some things might remain attached, but I don't have a solution, and I like being "more compatible" with libbpf. If we do implement auto_attach(), we should also add program.links() and program.links_mut() so one can retrieve the attached links and detach.


{"message":"Couldn't find a repository matching this job.","error":true}

I've tried everything, exporting those two tokens, Logging in and out of github as well as coveralls and travis ci, readding repo to travis as well as coveralls. But this error still occurs.
https://travis-ci.org/agauniyal/rang

Occurs b/w lines 380-400, 389 to be exact if you choose gcc-4.8.

Have you tried this? It worked for my case. It seems the token always needs to be specified whether the repo is private or not.

@jslee02 , first suggestion didn't worked but specifying the token did :smile: thanks for your help.

Ok good that this got resolved :)

It is still broken, and I suppose it is because the upload part does not support the latest way to send data to covealls. The JSON format seems to have changed if I read their documentation: https://docs.codecov.io/v4.3.0/reference#upload


fix: make pybind_align to fit c++ interface and add numpy style doc in it

#46 try to fix the issues

Thanks a lot!


Archive file

@scizo Please take a look at this initial first pass of archiving files. It is mostly done and ready to be merged. I still need to add the ability to pass in custom_fields and it turns out that this will require modifications to the wrapper which will make it more generally useful in general.

Actually, maybe we don't need to bother with the custom_fields since it will be captured in the inputs anyways as is.

@scizo want to do another quick pass?

So the last change makes it so that custom_fields must be explicitly passed to provenance vs implicitly?
@provenance(name='foo', bar='baz')
def test(...):
    return True

must now be
@provenance(name='foo', custom_fields={'bar': 'baz'})
def test(...):
    return True

Is that correct?

I prefer the explicit custom_fields. I am just making sure that I was understanding the changes correctly.

By the way, did you see my comment on creating a transform method on the Proxy class?

Yes, you have to have an explicit custom_fields. We weren't using that anywhere so it felt like a safe change and I liked it better as well. With that change we may be able to remove the **kwargs from the decorator which would catch typos. I'll double check on that.

I am going to merge this so that I can start switching input_id -> id, id -> value_id, and not have to worry about doing too complicated of a merge with this. The archive_file function still needs work, so that multiple different files can be archived with the same original filename.


[hotfix] Fix CsvRowDataDeserializationSchema constructor

The csvSchema field was wrongly initialized.

@MartijnVisser could you also merge?

@flinkbot run azure

The failing test is unrelated and tracked under https://issues.apache.org/jira/browse/FLINK-26882

@afedulov Will you also create the backport to release-1.15 ?

Thanks, @MartijnVisser .  Here is the backport PR https://github.com/apache/flink/pull/19288


feat(design): daff-map-get function

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/develop/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Fixes: #3309
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@damienwebdev Is the daff-map-get function is also supposed to be able to replace Angular's map get function?

@damienwebdev @gracetxgao

Is the daff-map-get function is also supposed to be able to replace Angular's map get function?

I expect the palettes to be using daff-map-get as well? (i.e. $primary, $secondary, $teritary). It's still confusing that some use Angular's map while some use daff-map-get.
Currently:
$primary: map.get(daff-theme.$theme, primary);
$secondary: map.get(daff-theme.$theme, secondary);
$tertiary: map.get(daff-theme.$theme, tertiary);
$base: daff-theme.daff-map-get($theme, 'core', 'base');
$base-contrast: daff-theme.daff-map-get($theme, 'core','base-contrast');

Expect:

$primary: daff-theme.daff-map-get($theme, primary);
$secondary: daff-theme.daff-map-get($theme, secondary);
$tertiary: daff-theme.daff-map-get($theme, tertiary);
$base: daff-theme.daff-map-get($theme, 'core', 'base');
$base-contrast: daff-theme.daff-map-get($theme, 'core', 'base-contrast');

@gracetxgao
Can you update all $primary, $secondary, $tertiary, $info, $warn, $critical, and $success values to use the daff-map-get function?


Xenon stealing focus from PoE

Running PoE in Windowed Fullscreen, alt-tab out of PoE, then click back into PoE. PoE will have focus for 0.5-1s before focus is lost / stolen by Xenon. Xenon does not have Always focus PoE turned on.

Good call, I do have Auto-Minimize enabled and disabling it does fix the issue.

I'm leaving this here as a note for myself, should be in the next release.
win.showInactive()

This is fixed in v0.3.9.


Aggregated signature not part of commitment yet

We haven't included aggregated signature as part of the commitment yet, we just accept it as an argument at the moment, need to commit it as well.

This was left pending because I wasn't sure how the fraud-proof was expecting the commitment.


Bibliography style bug: gb-7714-2015-numeric

Description
What I expected:
[1] Hatamizadeh A, Tang Y, Nath V, et al. Unetr: Transformers for 3d medical image segmentation[C]//Proceedings of the IEEE/CVF winter conference on applications of computer vision. 2022: 574-584.

What I actually got:

The difference is the author names were uppered (which is not I want), and the 等 instead of et al.
Reproduction URL
No response
Operating system
Linux
Typst version

[X] I am using the latest version of Typst

https://github.com/typst/citationberg/issues/5

@Enter-tainer Thanks for your answer :-)

如果有办法解决的话，希望可以联系我

我自己在用一个临时方案：用 citation-js 把引文条目预处理成需要的样式，再到 Typst 中引用。我把相关的思路和代码放在 https://github.com/csimide/cslper 了。
I am currently using a temporary solution: I preprocess the citation entries into the required format using citation-js, and then cite them in Typst.


Ideas and Suggestions for CLNDR 2.0

clndr 2.0 is happening
I am going to start the process in March. No idea how long it will take, but the result will be a brand new version of clndr.js. Here are a few things I can be certain of:

The new version will not be backwards-compatible and will feature a more semantic API
The idea of a "month" will go away in favor of any interval of time, allowing you to make a one-day view, a one-week view, a one-year view, or anything in between.
Moment and jQuery are here to stay as dependencies, but underscore can go away
It will be thoroughly unit tested
It should work on the server

I would love to hear from all of you: what do you think needs to be included in a new version? Are there pain points (aside from the custom intervals stuff) that you're experiencing?
This will be collaborative
See issue #136 ... If you have an interest in contributing to the new version, you are encouraged to participate in whatever capacity makes sense for you.
Thanks!

Make it mobile friendly! (I actually hacked css and js to enable some kind of mobile friendly view in the current version, but it is not optimal at all)
Unfortunately I am not that skilled in JS at all so I would not be useful during the development, but I am happy to test and integrate the library in my projects!

@steilerDev thanks for the suggestion— my thought was that binding on tap when tap is available would be a step in the right direction. What other aspects do you think need some attention for mobile? Since the views and CSS are user-provided, it's on the developer to write styles that work on a device.

I think removing jquery as a dependency would be a good goal, I am looking at using version 1 in an angularjs project and whilst jquery plays nice with angular, its a dependency I don't really want. If much of the view layer is handled by the developer I'm not sure why jQuery is needed although I have not looked in depth at what is going on under the covers. I would build version 2 as modular as possible and provide a wrapper for people who want to use it with jquery, it should be useable by more people this way. In fact if you intend it to be useful on the server, jquery seems like a strange dependency to enforce.
Moment is a more than reasonable dependancy, and depending on how much of the api you use, lodash/underscore 'could' be reasonable, but I suspect it would be better to just borrow the implementation of some of the methods you would use within lodash.

@stuplum that's interesting. Dependencies are definitely a major complaint, but jQuery is the one I hear complaints about the least! I was going to look into stealing the _.template functionality so that underscore/lodash could go away. Moment, as you have pointed out, is here to stay.
There's a delicate balance here that we need to strike in v2.0. I do not under any circumstances want to be writing native DOM queries and monkey-patching fixes for random browsers, which is what jQuery allows us to avoid. Clndr should work out of the box down to ~IE7, and without jQuery this becomes complicated (and makes for a bloated codebase).
For reference, jQuery's main responsibilities are to handle click events and injection of the rendered template into the DOM.
My worry about supporting angular is that there is barely any overlap with the scope of clndr's functionality. Angular has its own concept of templating, events, and DOM manipulation. Literally the only thing you would want from clndr is the part that keeps track of the current date. My thought was that the architecture could be modular enough that you would be able to use this component separately, but correct me if I'm wrong in saying that 90% of clndr becomes unnecessary if you want it to play well in angular world.
I am really excited about figuring this out! I've had plenty of questions about angular integration so if we can make that less painful in the next version I think it's totally worth it.

I have successfully implemented this simple template function into clndr by moving the js function from the template string to the render function as it does not support advanced logic statements (only if and foreach), but that's enough if the data is prepared before.

Awesome @miljan-aleksic, that's very encouraging.

I would really love to use this in a meteor app. However, I can't seem to make it work in meteor. I'm not sure if it's because of meteor's blaze templating or because of how the library was built with the checking if jquery or moment is defined... I'm a novice at this... but I hope this library is as easy as others to just package for meteor. Already spent whole day trying to make it work...packaged or just an asset I can't figure it out. Oh boy.

@arvi
I'm also trying to make this work in Meteor. I've spent at least 6 hours trying to make this work and it's quite frustrating. Simply applying spacebars does not work. There also seems to be no exposed variables for us to watch reactively. There does seem to be a daysArray variable that has leaked into the global namespace, but I think this is a bug. Nevertheless, it's not reliable because the moment date object gets corrupted whenever I try to set daysArray into my Session variable in Meteor.
At this point, unless someone more skilled can pitch in on the Meteor front, it might be best to use something else.

I'm pretty sure the leaked daysArray is a bug as well. I have patched it locally whilst I'm building an angular directive around it.

Oof, first off, thanks for the heads up about the leaky daysArray variable, fixing that now.
I'm sorry to hear about your Meteor troubles. The modular architecture we have in mind for v2.0 aims to solve this problem. Unfortunately clndr was written as a jQuery plugin, and supporting myriad MVC frameworks was not an initial goal of the project.
What I need from you now is a sense of which piece of clndr you're looking to use. It seems like you're only interested in the daysArray that gets generated on load/each time clndr is advanced. Is that correct? If that is the case, are you aware that you can supply your own render function, which passes you the daysArray as data? Clndr's api allows you to advance it programmatically, meaning that you don't have to rely on DOM events— this opens you up to the possibility of working with your own template in a separate framework that is loosely coupled to clndr through its api. We can talk this through in a separate issue if it seems like a direction that will make sense for you all. I have never used Meteor, so feel free to tell me that I'm totally off on this one.

Further thoughts on this @arvi, @adrianmc or @stuplum? We are going to begin architectural planning for 2.0 on Monday. Did you figure out a solution to your Meteor/Angular issues or, if not, do you have ideas about what it should allow you to do?

I've been spending most of my time around creating a tested/able directive for consuming clndr as it currently stands.
I am considering creating a patched version that removed the jquery dependency completely, as I don't think it will be required at all for angular. It will be interesting to see whats left when the view logic is removed and just the clndr module with an api is left. I am a bit reluctant to do it right now though as it works as is for me at the moment and I have other priorities, which is a shame as I'm itching to have a go.
I think the only 'issue' I have run into so far is with the month property on the data supplied to the render callback being a formatted month, whereas I would prefer it to just be a moment object that I can play with programatically. This is probably more of a preference than an issue.
I would potentially consider wrapping all of the moment objects into a clndr specific date object, but I guess this would only be important if you decided to ditch moment.

Hi Kyle. Unfortunately, I switched to using fullcalendar due to project's deadline. I think the issue is with Meteor's Blaze Templating since when I tried patterning CLNDR template to Blaze/Spacebar syntax http://meteorcapture.com/spacebars/,  rendering does not work.
I hope the next version would be template-agnostic. :)

@stuplum I hear you about the formatted months. It would be my preference as well but at this point it's a big BC break to change it. Open to your ideas about what else should be passed into the template!
Can you elaborate on that last point about a clndr specific date object? I'm not sure I understand.
@arvi bummer. Did you see my note about the render function? Clndr is already template-agnostic.

@kylestetz The clndr specific date object would be a wrapper class around any moment object. It would expose it's own methods and properties (that may or may not have the same interface as moment), but it would allow you to swap out moment for another library or even a custom implementation in the future. I guess it's only important if you were unsure if moment was going to hang around or not.

Ah, interesting. Moment is not going anywhere, but I'll keep this in mind as we plan the architecture and see if this might benefit us in some other way. Thanks!

@kylestetz Oh okay. Sorry, I missed reading that comment. I'll try it out and see if it'll work. :)

Following up on our discussions, you can read about the first steps toward the next version in #155. Development will be starting soon!

Is CLNDR currently capable of carrying end-dates for events? This is one thing I'm missing for my day-view. I specifically want to be able to display the event duration.
I love your work. Awesome stuff!

Just found CLNDR. Awesome.
You've mentioned the one-day option. This would be great for us (currently using fullcalendar). I presumably we would be able to style hours/mins etc so we can display a day schedule? Not just a list of events for a given day?
Also, we use resources a lot. I think we could code this currently? Not quite sure. Basically grouping events by a given value (name or room for example) and displaying events for each resource. An example ( this isn't very customisable in terms of templating. We prefer a vertical view) http://fullcalendar.io/scheduler/
Love the work dude. Cheers.

I think it would be great to design it to make it possible to do databinding for events. For example having an event "beforeRender" where I would be able to modify the events array. We use it for a lot of events and it doesn't make sense to load 5 years of events, it would be better to load them on demand.
Thanks

See #136 for an update on the status of clndr 2.0. We won't be pursuing a rewrite of the project for now, and instead we're cleaning up the codebase to ensure that this project continues to be maintainable and easy to contribute to!


Better asset handling

Add functionality to allow to more easily add static assets such as images, CSS, and JavaScript files.

Is this still in the works? I'd love to use this feature in webden, because I'm mostly offline


Static compilation of empty project fails on macos (OSX)

This is the command I ran and the error I got:
$ crystal build --static src/something.cr
ld: library not found for -lcrt0.o
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Error: execution of command failed with code: 1: `cc "${@}" -o '/Users/sherjil/Desktop/something/something'  -rdynamic -static  -lpcre -lgc -lpthread /usr/local/Cellar/crystal-lang/0.24.2_1/src/ext/libcrystal.a -levent -liconv -ldl -L/usr/lib -L/usr/local/lib`

I did a bit of googling, and it seems macos doesn't provide a static library for crt0.o, which is causing this problem.

Building static binaries is unsupported by macOS. There is nothing we can do :(

We can add the information that static linking is not supported when using --static on macOS. with a macro triggered by the darwin flag.

Smart suggestion from @j8r.

👋🏾
Sorry for posting on a old closed issue, but I just started recently playing with Crystal and realised you can actually build portable/static libs for MacOS. You do have to link to some system dynamic libs but that is fine and acceptable in MacOS, you can set the MACOS_DEPLOYMENT_TARGET=11.0 or whichever earliest version you want to target
Some instructions below
https://github.com/YOU54F/hello_crystal#building-static-libs-on-macos
Building Static Libs on MacOS
When using Crystal built from HomeBrew.
otool -L ./hello
./hello:
        /opt/homebrew/opt/pcre2/lib/libpcre2-8.0.dylib (compatibility version 12.0.0, current version 12.2.0)
        /opt/homebrew/opt/bdw-gc/lib/libgc.1.dylib (compatibility version 7.0.0, current version 7.2.0)
        /opt/homebrew/opt/libevent/lib/libevent-2.1.7.dylib (compatibility version 8.0.0, current version 8.1.0)
        /usr/lib/libiconv.2.dylib (compatibility version 7.0.0, current version 7.0.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1319.100.3)

When using Crystal built from HomeBrew, that requires OpenSSL func
otool -L ./fetch_ffi
./fetch_ffi:
        /usr/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.11)
<EMAIL_ADDRESS>(compatibility version 3.0.0, current version 3.0.0)
<EMAIL_ADDRESS>(compatibility version 3.0.0, current version 3.0.0)
        /opt/homebrew/opt/pcre2/lib/libpcre2-8.0.dylib (compatibility version 12.0.0, current version 12.2.0)
        /opt/homebrew/opt/bdw-gc/lib/libgc.1.dylib (compatibility version 7.0.0, current version 7.2.0)
        /opt/homebrew/opt/libevent/lib/libevent-2.1.7.dylib (compatibility version 8.0.0, current version 8.1.0)
        /usr/lib/libiconv.2.dylib (compatibility version 7.0.0, current version 7.0.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1319.100.3)

So lets build some libs statically

libssl.a
libevent.a
libpcre2.a
libgc.a

See Makefile - make libs
When building, link to our static libs - assuming

all .a files at root of invoked command
openssl include dir exists in <root_dir>/include

Build with
APP_NAME=src/fetch_ffi.cr
crystal build --release $APP_NAME --no-debug --link-flags="-L$PWD -I$PWD/include"

Check it isn't dynamically linked to anything other than system libs
otool -L ./fetch_ffi
./fetch_ffi:
        /usr/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.11)
        /usr/lib/libiconv.2.dylib (compatibility version 7.0.0, current version 7.0.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1319.100.3)

Run your file, ensuring you set the SSL_CERT_FILE to your required cert file.
APP_BINARY=./fetch_ffi
SSL_CERT_FILE=/private/etc/ssl/cert.pem $APP_BINARY

TODO - Add cert.pem from traveling-ruby (centos8) and set the SSL_CERT_FILE env var if
not set by the user.
Setting up our MacOS Image for Tart/CirrusCLI

tart clone ghcr.io/cirruslabs/macos-ventura-vanilla:latest crystal

tart run crystal &

scp -r /Users/saf/dev/examples/hello_crystal admin@$(tart ip crystal):/Users/admin

ssh admin@$(tart ip crystal)

Run commands in VM
NONINTERACTIVE=1 /bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh)"
(echo; echo 'eval "$(/opt/homebrew/bin/brew shellenv)"') >> /Users/admin/.zprofile
eval "$(/opt/homebrew/bin/brew shellenv)"
brew install crystal
crystal --version
make libs

tart stop crystal

MacOS Binary compatibility
It is possible to control the versions of MacOS, the binary is compatabile with.
Set the MACOSX_DEPLOYMENT_TARGET env var
MACOSX_DEPLOYMENT_TARGET="14.0" crystal build <args>
MACOSX_DEPLOYMENT_TARGET="11.0" crystal build <args>
Refs:-

https://en.wikipedia.org/wiki/MacOS_version_history
https://cmake.org/cmake/help/latest/envvar/MACOSX_DEPLOYMENT_TARGET.html
https://cmake.org/cmake/help/latest/variable/CMAKE_OSX_DEPLOYMENT_TARGET.html#variable:CMAKE_OSX_DEPLOYMENT_TARGET

MacOS Crystal GitHub Binary - Issue building with OpenSSL
Using the Crystal GH binary, we get linker errors when building something that requires OpenSSL.
Same issue occurs even if brew and openssl are installed and a static build of openssl is built.
I can only build/link with crystal installed via Brew.
Maybe this will help - https://crystal-lang.org/reference/1.9/guides/static_linking.html#dynamic-library-lookup
curl -L https://github.com/crystal-lang/crystal/releases/download/1.9.2/crystal-1.9.2-1-darwin-universal.tar.gz | tar xz
mv crystal-1.9.2-1 ~/crystal
chmod +x ~/crystal/bin/crystal
~/crystal/bin/crystal --version
file ~/crystal/bin/crystal
file ~/crystal/embedded/bin/crystal
echo PATH=$PATH:~/crystal/bin >> $CIRRUS_ENV

Linking error below.
crystal build --release bin/$APP_NAME.cr --no-debug --link-flags="-L$PWD -I$PWD/include"
Undefined symbols for architecture arm64:
  "_ERR_load_crypto_strings", referenced from:
      ___crystal_main in _main.o
  "_OPENSSL_add_all_algorithms_noconf", referenced from:
      ___crystal_main in _main.o
  "_SSL_library_init", referenced from:
      ___crystal_main in _main.o
  "_SSL_load_error_strings", referenced from:
      ___crystal_main in _main.o
  "_SSLv23_method", referenced from:
      _*HTTP::Client::get<String>:HTTP::Client::Response in _main.o
  "_sk_free", referenced from:
      _~procProc(Pointer(Void), Nil)@/Users/admin/crystal/src/openssl/ssl/hostname_validation.cr:71 in _main.o
      _~proc2Proc(Pointer(Void), Nil)@/Users/admin/crystal/src/openssl/ssl/hostname_validation.cr:71 in _main.o
      _~proc3Proc(Pointer(Void), Nil)@/Users/admin/crystal/src/openssl/ssl/hostname_validation.cr:71 in _main.o
      _~proc4Proc(Pointer(Void), Nil)@/Users/admin/crystal/src/openssl/ssl/hostname_validation.cr:71 in _main.o
      _~proc5Proc(Pointer(Void), Nil)@/Users/admin/crystal/src/openssl/ssl/hostname_validation.cr:71 in _main.o
      _~proc6Proc(Pointer(Void), Nil)@/Users/admin/crystal/src/openssl/ssl/hostname_validation.cr:71 in _main.o
     (maybe you meant: _OPENSSL_sk_free)
  "_sk_num", referenced from:
      _~procProc(LibCrypto::X509_STORE_CTX, Pointer(Void), Int32)@/Users/admin/crystal/src/openssl/ssl/context.cr:95 in _main.o
     (maybe you meant: _OPENSSL_sk_num)
  "_sk_pop_free", referenced from:
      _~procProc(LibCrypto::X509_STORE_CTX, Pointer(Void), Int32)@/Users/admin/crystal/src/openssl/ssl/context.cr:95 in _main.o
     (maybe you meant: _OPENSSL_sk_pop_free)
  "_sk_value", referenced from:
      _~procProc(LibCrypto::X509_STORE_CTX, Pointer(Void), Int32)@/Users/admin/crystal/src/openssl/ssl/context.cr:95 in _main.o
     (maybe you meant: _OPENSSL_sk_value)
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Error: execution of command failed with exit status 1: cc "${@}" -o /private/tmp/cirrus-ci/working-dir/fetch_ffi -L/private/tmp/cirrus-ci/working-dir -I/private/tmp/cirrus-ci/working-dir/include -rdynamic -L/Users/admin/crystal/embedded/lib -lz `command -v pkg-config > /dev/null && pkg-config --libs --silence-errors libssl || printf %!s(MISSING) '-lssl -lcrypto'` `command -v pkg-config > /dev/null && pkg-config --libs --silence-errors libcrypto || printf %!s(MISSING) '-lcrypto'` -lpcre2-8 -lgc -levent -liconv
'fetch_ffi' script failed in 16s!
'crystal' task failed in 06:19!


Sunday tasks - no need to work today though!

Just letting folks know what I'm doing today.  I'm going to run through text from the Title page through the end of the executive summary today.  I'll note where hard coded values (shouldn't be many) will still need to be double checked but the main results in my current document build (with 2199 MCMC samples) will be nearly identical to a document that builds with the 2000.

Worked through the executive summary yesterday, still need to do the research and data needs part.  I don't think any changes I made are too different than our standard language.  The exceptions are 1) the following statement in the one page summary:
[The model was updated to include year-specific fecundity from 1975-2018. However, limited
exploratory and supplementary analyses could be completed for the 2019 assessment as
a result of the U.S. government shutdown considerably delaying delivery of U.S. age data
and reducing JTC preparatory time.]
And 2) in the executive summary (data and assessment section):
[The 35 day 2018/2019 U.S. government shutdown considerably delayed delivery of U.S. age data
and reduced the JTC’s collective capacity to complete the 2019 assessment following status-quo
procedures. As a result, many planned supplementary and exploratory analyses could not be completed,
including some developed for response to 2018 SRG requests (see Section 3.3).]
Happy to have edits/comments on this language.  I thought it prudent to make sure the impacts of the shutdown are forefront.

Nice - 'exploratory and supplementary' analyses is great wording, as we've still done the sensitivities...


clean up source/sink/function json configuration help files

Environment:

eKuiper version (e.g. 1.3.0):  1.3.1
Hardware configuration (e.g. lscpu):
OS (e.g. cat /etc/os-release):
Others:

What happened and what you expected to happen:
mismatch between json config help files and actually use
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

add this in UT
+func TestReadSourceMetaDir(t *testing.T) {

  err := ReadSourceMetaDir(func(name string) bool {

          return true

  })

  if err != nil {

          t.Error(err)

  }

+}


Unexpected behavior with Category / Categories

https://github.com/get-got/discord-downloader-go/issues/91 History mishandles categories
https://github.com/get-got/discord-downloader-go/issues/97 History mishandles categories

Should be fixed as of https://github.com/get-got/discord-downloader-go/commit/d8c31e4951969d4a6f23467ff5b7b75b9af02e13 and https://github.com/get-got/discord-downloader-go/commit/897a377782b9e1804b70973b03f5dda91fcf4654
Please re-open if any issues persist with categories.


Fixed log messages for adding services to the database

The log messages of the configuration service were misleading. When a service has been added to the materialized view successfully, the logs said that the service already exists. This has been fixed with this PR

Codecov Report

Merging #2282 into release-0.7.1 will decrease coverage by 0.01%.
The diff coverage is 75.00%.

@@                Coverage Diff                @@
##           release-0.7.1    #2282      +/-   ##
=================================================
- Coverage          37.09%   37.08%   -0.02%
=================================================
  Files                156      156
  Lines               9329     9331       +2
  Branches             164      164
=================================================
- Hits                3461     3460       -1
- Misses              5392     5394       +2
- Partials             476      477       +1

Flag
Coverage Δ

#moduleA
24.24% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ation-service/common/projects_materialized_view.go
77.82% <75.00%> (-1.08%)
:arrow_down:

Continue to review full report at Codecov.

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


logging: Fix case where logging strings were not stripped from the binary when dictionary logging was used

Due to LOG_OUTPUT beings selected by any logging backend, logging strings were not stripped from the binary. It can be seen in samples/subsys/logging/dictionary where it is expected that strings are stripped by they weren't.

@dcpleung @karstenkoenig @cfriedt can you reapprove? Only clang-format changes.


Fix ZZZ & Add custom header functionality

Fixes #52

Tested discord english version.
Other versions needs testing but should be fine.

Ok should be fine for now. thank you @Chillrend

It seems this project is dead and I need to go seperate go direction. If you liked how I coded this PR you might be interested in my version thanks everyone who approved.


Connect component to LLM

Summary

use llm response to replace previous mock aiResponse
Add loading icon to message list

Screenshot

Checklist

[ ] My code follows ESLint rules and passes all tests.
[ ] I have updated the documentation accordingly.

TODO: image support, prev code support
TODO: error handling
@ZhuoyangM


Library is massive - Split into smaller libraries?

I included this library in my project and the minified project size went from 3mb to 6mb.  I turned on shrinkResources and it is still at 5.2mb.
My use of the library is limited to its CardView - perhaps the library could be split into a core library + feature libraries?

No, this is not really possible.
Carbon includes support libraries, which add a lot of methods and bytes. Shadows drawn by Carbon are using RenderScript, which is another heavy thing which cannot be removed. SVG graphics and carbon.* classes are small, so removing them won't change that much.
Splitting vertically is not an option as well. Carbon's components are tightly integrated with each other. Shadows, ripples, attributes, graphics, layouts, animations - all of these are used in almost each widget and cannot be extracted.
The only reasonable way is to use ProGuard and cut unused classes out.


Can this work with Blazor Server Side?

I added a Blazor Server Side version here:
https://github.com/ADefWebserver/EventHorizon.Blazor.TypeScript.Interop.Generator/commit/eb4407747b1d63397cd11cb1409e58d7247ed1bb
But, I get this error that I just can't seem to figure out?

System.NullReferenceException
HResult=0x80004003
Message=Object reference not set to an instance of an object.
Source=EventHorizon.Blazor.Interop
StackTrace:
at EventHorizon.Blazor.Interop.EventHorizonBlazorInterop.FuncClass[T](Func`2 classBuilder, Object[] args)
at EventHorizon.Blazor.BabylonJS.Model.Canvas.GetElementById(String elementId) in C:\Users\webma\source\repos\ADefWebserver\EventHorizon.Blazor.TypeScript.Interop.Generator\Sample\BlazorServerApp\Model\Canvas.cs:line 12
at BlazorServerApp.Pages.Index.CreateScene() in C:\Users\webma\source\repos\ADefWebserver\EventHorizon.Blazor.TypeScript.Interop.Generator\Sample\BlazorServerApp\Pages\Index.razor:line 38
at BlazorServerApp.Pages.Index.OnAfterRender(Boolean firstRender) in C:\Users\webma\source\repos\ADefWebserver\EventHorizon.Blazor.TypeScript.Interop.Generator\Sample\BlazorServerApp\Pages\Index.razor:line 27
at Microsoft.AspNetCore.Components.ComponentBase.Microsoft.AspNetCore.Components.IHandleAfterRender.OnAfterRenderAsync()
at Microsoft.AspNetCore.Components.Rendering.ComponentState.NotifyRenderCompletedAsync()

Even when I try to construct a CachedEntityObject manually (that appears to work), I still get:
System.NullReferenceException
HResult=0x80004003
Message=Object reference not set to an instance of an object.
Source=EventHorizon.Blazor.Interop
StackTrace:
at EventHorizon.Blazor.Interop.EventHorizonBlazorInterop.FuncClass[T](Func`2 classBuilder, Object[] args)
at EventHorizon.Blazor.BabylonJS.Model.Canvas.GetElementById(String elementId) in C:\Users\webma\source\repos\ADefWebserver\EventHorizon.Blazor.TypeScript.Interop.Generator\Sample\BlazorServerApp\Model\Canvas.cs:line 15
at BlazorServerApp.Pages.Index.CreateScene() in C:\Users\webma\source\repos\ADefWebserver\EventHorizon.Blazor.TypeScript.Interop.Generator\Sample\BlazorServerApp\Pages\Index.razor:line 38
at BlazorServerApp.Pages.Index.OnAfterRender(Boolean firstRender) in C:\Users\webma\source\repos\ADefWebserver\EventHorizon.Blazor.TypeScript.Interop.Generator\Sample\BlazorServerApp\Pages\Index.razor:line 27
at Microsoft.AspNetCore.Components.ComponentBase.Microsoft.AspNetCore.Components.IHandleAfterRender.OnAfterRenderAsync()
at Microsoft.AspNetCore.Components.Rendering.ComponentState.NotifyRenderCompletedAsync()

That error is caused by the usage of the WebAssemblyJSRuntime in EventHorizonBlazorInterop, that package is WASM only.
The async nature of the IJSRuntime cut heavily into the performance of the calls when first prototyping this project, so I focused on WASM only starting off.
But now I can see adding Server support would add a lot of versatility to Blazor as a whole.
I can look at creating another package for the interop layer, and an extension of this project that can generate an API that supports BlazorServer.

This feature request ticket #22

The library clearly works perfectly as designed.
Thank you so much for looking into this and considering adding the feature.
Thank You

The desire is to eventually integrate it into: https://github.com/ADefWebserver/AutomaticKingdom

@ADefWebserver Checkout out this poc repo for the Blazor Server support:
https://github.com/canhorn/EventHorizon.Blazor.Interop.Generator
It has both a Server and WASM example in the sample. If you need anymore generated hit me up.
It turned out to be a lot easier than though it would be, I was able to import this repos NuGet package, then changed the templates, added a few text cleanup replacements and it it seems to work.
This was just an afternoon project, so it might be a little rough right now, but I will clean it up this weekend.  I might need to think of a good name as well. ;)

Wonderful news! Thank You! I will dig into this as soon as I can!

Is there a Blazor Server Side version of the most recent code?
If not, what are the steps I would need to do to make the latest code work with Blazor Server side? Thank You!

Is there a Blazor Server Side version of the most recent code?
If not, what are the steps I would need to do to make the latest code work with Blazor Server side? Thank You!

The recent code will need to be ported over to the Server Side.
I am hoping it will be a direct port, since the server is all async/await.
I will keep you posted.

The recent code will need to be ported over to the Server Side.
I am hoping it will be a direct port, since the server is all async/await.
I will keep you posted.

Thank you.
I am trying to fix: https://automatickingdom.com/
The current code (that had worked for months) now shows:
blazor.server.js:1 Uncaught (in promise) Error: There was an exception invoking 'HandleCallback'. For more details turn on detailed exceptions in 'CircuitOptions.DetailedErrors'
    at Object.endInvokeDotNetFromJS (blazor.server.js:1)
    at e.<anonymous> (blazor.server.js:8)
    at blazor.server.js:1
    at Array.forEach (<anonymous>)
    at e.invokeClientMethod (blazor.server.js:1)
    at e.processIncomingData (blazor.server.js:1)
    at e.connection.onreceive (blazor.server.js:1)
    at WebSocket.i.onmessage (blazor.server.js:1)

Thank you.
I am trying to fix: https://automatickingdom.com/
The current code (that had worked for months) now shows:
blazor.server.js:1 Uncaught (in promise) Error: There was an exception invoking 'HandleCallback'. For more details turn on detailed exceptions in 'CircuitOptions.DetailedErrors'
    at Object.endInvokeDotNetFromJS (blazor.server.js:1)
    at e.<anonymous> (blazor.server.js:8)
    at blazor.server.js:1
    at Array.forEach (<anonymous>)
    at e.invokeClientMethod (blazor.server.js:1)
    at e.processIncomingData (blazor.server.js:1)
    at e.connection.onreceive (blazor.server.js:1)
    at WebSocket.i.onmessage (blazor.server.js:1)

It seems this is now being worked on: https://github.com/canhorn/EventHorizon.Blazor.TypeScript.Interop.Generator/tree/feature/blazor_server_support 👍🏾🙌🏾

Support for Blazor Server is in main, I added an example project to the Sample folder as well.
Also included updates to tool to support a flag/option that can be used to generate a Blazor server hosting model.

Also the "HandleCallback" error should be fixed in the samples. The babylonJS version was too new compared to the generated source, and now the Sample projects will lock in the version used for BabylonJS.

When I try to run the server project I am getting this again?

That Canvas looks like the one from the Wasm version.
There should be another project (EventHorizon.Blazor.Server.BabylonJS) that has been updated to work with the Server version.

When I try to build: EventHorizon.Blazor.BabylonJS
I get:

I can make the error go away and get the project to build by changing:
    <ItemGroup>
        <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly" Version="5.0.0-preview.7.20365.19" />
        <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly.DevServer" Version="5.0.0-preview.7.20365.19" PrivateAssets="all" />
    </ItemGroup>

To;
    <ItemGroup>
        <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly" Version="5.0.0" />
        <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly.DevServer" Version="5.0.0" PrivateAssets="all" />
    </ItemGroup>

And removing:
        <RuntimeIdentifier>browser-wasm</RuntimeIdentifier>
But when I run it I get:

Oh your fork is a bit behind, you need to merge from main, that has been fixed in the main repository.
The Sample has a new Blazor Server example, the EventHorizon.Blazor.Server.BabylonJS.csproj, it uses a different generated interop project that has a focus on Async generation.
So they needed to have two different implementation to work.

Wow! Sooo sorry about that! I thought I was pointed to your main 😣😬😱
Ok, working now 😊


Use ShapeShift for Swaps

This PR configures swap to use ShapeShift for all swaps, where previously Bity was configured for BTC<->ETH swaps.

New SS Key: 8abde0f70ca69d5851702d57b10305705d7333e93263124cc2a2649dab7ff9cf86401fc8de7677e8edcd0e7f1eed5270b1b49be8806937ef95d64839e319e6d9


[FEA] Import from S3

 Issue by benfred
Saturday May 23, 2020 at 02:42 GMT
Originally opened as https://github.com/rapidsai/recsys/issues/169

We should be able to load files from S3 for both preprocessing and dataloading. We need to add the ability to cudf to efficiently read from S3 , and then wire up nvtabular to support this.

duplicate of https://github.com/NVIDIA/NVTabular/issues/11


How can I disable blending effect in iOS devices?

Description
I have been using Model Viewer to add WebAR onto my website, On android the models looks good and functions properly, but for iOS, the model is blending into atmosphere which is not what I need, How can I disable it using Model Viewer api? I couldn't find any references, apart from this one github issue https://github.com/google/model-viewer/issues/1733#issuecomment-749136234 , but when I use it, I get an error saying "Object requires newer version of iOS".
Live Demo
Image of the Issue I'm haing: https://i.stack.imgur.com/PFizj.jpg
Image of the Error: https://i.stack.imgur.com/ZvkLD.jpg
Version

model-viewer: vX.X.X

Browser Affected
Safari
OS

iOS

This is a QuickLook bug and the intent solution you refer to is for SceneViewer. I'm not aware of Apple allowing this kind of customization in QuickLook, but if you find a way in their docs, I'll be happy to add it. I'm guessing you'll get better results with a new iPhone 12 with LIDAR.


Archive all taps after parallel run

Access archived tap file from jobs archive directory
Only archive if tests are enabled.
Fix #1039

${env.JENKINS_HOME} can't be accessible directly due to security constraints.
using currentBuild.rawBuild.getArtifactsDir().toString(), Scripts not permitted to use method hudson.model.Run getArtifactsDir. Administrators can decide whether to approve or reject this signature.
So I think final easy way is to copyArtifacts in pipeline job itself.

@andrew-m-leonard


Add support for displaying grants for when a user is directly assigned as a group or app admin.

Hey @jirwin this PR has been open for a year. Is it alright if I close it, or do you think we might still want the app admin entitlement?


dataspace-protocol (TransferTerminationMessage): Add Nullcheck for code field

Feature Request
In the spec of the TransferTerminationMessage the field code is required. So in the TransferTerminationMessage.Builder it is needed to do a Nullcheck.
Which Areas Would Be Affected?
TransferTerminationMessage, TransferProcessManagerImpl
Why Is the Feature Desired?
Get align with the dataspace-protocol spec
Solution Proposal
Right now their is no list of error codes. So the Nullcheck in the TransferTerminationMessage.Builder should be added after such a list exist.

It should not be required for an implementation to provide a code. We may need to fix this in the message schema and discuss at the spec level


Cannot install extension on windows

Microsoft Windows [Version 10.0.17134.706]
VSCode v1.33.1
Electron 3.1.6
Node 10.2.0
I cloned the repo to my '/.vscode/extensions' directory with my other extension but It doesn't appear in Extensions and is not active.
Please help Mr. Sean Park Ross
Your colour scheme is very Bagus.

Nevermind :rocket: https://marketplace.visualstudio.com/items?itemName=FrankJames.oceanic-park

Published for your convenience https://marketplace.visualstudio.com/items?itemName=Parklife.oceanic-park


Frontend #155

All the changes for task number 155 as well as branch 135 and 164(Deepak) have been aggregated into this branch. This includes adding quickviews to database, adding API endpoints to controller and front end functions to fetch, display and remove quickviews.

There are some merge conflicts that need to be resolved before this can be merged.


chore: update the extension documentation.

Includes information about the extension automatically installing sprocket.
Before submitting this PR, please make sure:

[x] You have added a few sentences describing the PR here.
[x] You have added yourself or the appropriate individual as the assignee.
[x] You have added at least one relevant code reviewer to the PR.
[x] Your code builds clean without any errors or warnings.
[x] You have added tests (when appropriate).
[x] You have updated the README or other documentation to account for these changes (when appropriate).

This can merge immediately following a publish of the extension.


nil user_id causes app failure

I have a test to ensure that a nil email in an update will result in a changeset error.  I'm using pow_user_id_field_changeset(attrs), which results in:
** (FunctionClauseError) no function clause matching in String.Break.trim_leading/1

     The following arguments were given to String.Break.trim_leading/1:

         # 1
         nil

     Attempted function clauses (showing 1 out of 1):

         def trim_leading(string) when is_binary(string)
...

stacktrace:
       (elixir) lib/elixir/unicode/properties.ex:288: String.Break.trim_leading/1
       (elixir) lib/string.ex:1108: String.trim/1
       (pow) lib/pow/ecto/schema.ex:307: Pow.Ecto.Schema.normalize_user_id_field_value/1
       (ecto) lib/ecto/changeset.ex:1133: Ecto.Changeset.update_change/3
       (pow) lib/pow/ecto/schema/changeset.ex:48: Pow.Ecto.Schema.Changeset.user_id_field_changeset/3

Is there a way to use the pow_user_id_field_changeset and receive changeset errors vs. this application error?

My test:
  @invalid_attrs %{email: nil, password: "password"}
...
    test "create_user/1 with invalid data returns error changeset" do
      assert {:error, %Ecto.Changeset{}} = Accounts.create_user(@invalid_attrs)
    end

My Context:
  def create_user(attrs) do
    %User{}
    |> User.changeset(attrs)
    |> Repo.insert()
  end

My User:
  def changeset(user_or_changeset, attrs) do
    user_or_changeset
    |> pow_user_id_field_changeset(attrs)
    |> pow_current_password_changeset(attrs)
    |> new_password_changeset(attrs, @pow_config)
    |> pow_extension_changeset(attrs)
    |> Ecto.Changeset.delete_change(:password)
  end

Sorry, a bunch of things and the holidays crept up so didn't have time to look at this before now.
I figure out a way to trigger this. The user has to already have the email set in the struct:
User.pow_user_id_field_changeset(%User{email: "test"}, %{email: nil})

I don't know why it blows up in your case though, since you call the changeset with an empty struct. Maybe a default value is set for the struct key? In any case, I'll open a PR to fix this.

#364 hopefully resolves this for you 😄

Perfect, thank you!


fix: invalid culture identifier

After GitHub bumps .NET version of actions runner, our EC2 runner reports:
System.Globalization.CultureNotFoundException: Only the invariant culture is supported in globalization-invariant mode. See https://aka.ms/GlobalizationInvariantMode for more information. (Parameter 'name')
en-US is an invalid culture identifier.
   at System.Globalization.CultureInfo..ctor(String name, Boolean useUserOverride)
   at System.Globalization.CultureInfo..ctor(String name)
   at GitHub.Runner.Common.HostContext.SetDefaultCulture(String name)
   at GitHub.Runner.Worker.Worker.SetCulture(AgentJobRequestMessage message)
   at GitHub.Runner.Worker.Worker.RunAsync(String pipeIn, String pipeOut)
   at GitHub.Runner.Worker.Program.MainAsync(IHostContext context, String[] args)

And from this issue https://github.com/actions/runner/issues/1605, it seems that we should remove DOTNET_SYSTEM_GLOBALIZATION_INVARIANT.
I'm still testing with this change, not sure if this really works. Thanks for reviewing!

@skyzh we are running into this as well - pls keep me updated.

Hi @davinchia, my forked action works perfectly in our environment. To use it for now, simply apply the following snippet in the GitHub Action config.
uses: skyzh/ec2-github-runner@ba2298a67875dfdd29a88fafbc1ba27f4f94af39

After this PR gets merged, or GitHub staff reverts the action .NET runtime, you can change back to machulav/ec2-github-runner@v2.

amazing, let me give it a shot

@skyzh works for us - thanks so much again!

Hi @machulav, would you please take a look if you have time? Thanks!

it this action still getting support/being developed ?

Let me make some final tests and I will merge if everything works.

Merged. The machulav/ec2-github-runner@v2 already contains the fix.
@skyzh @ramosbugs thanks a lot for figuring that out and for the fix!


Delete aws-cli

close #5

I've checked that make run-create' works correctly, as there are no test codes for ecr.go' yet.


Makefile: clang on mac

Simply clang and clang++ works for me rather than clang-${LLVM_VERSION} which gives me:
$ make
clang-6.0 -O3 -std=c++11 -Wall -fno-use-cxa-atexit -fPIC -c vb.cpp -o vb.o
make: clang-6.0: No such file or directory
make: *** [vb.o] Error 1

so,
  CLANG ?= clang

and
  CLANG ?= clang++

+1
This would also simplify the Python installation scripts AND remove the need to link between clang's (as mentioned in docs and found in practice).


WebCam Integration

Hi,
Is there an example or procedure to integrate Webcam to take the picture of a user to the application.
Thank you

http://www.c-sharpcorner.com/UploadFile/4d9083/capturing-image-from-web-cam-in-Asp-Net-mvc139/

Hey @srinivas1910 and @vinaykulkarni89 ,
how you did this webcam integration?
could you give me a help?


[BUG] False Positive for HaveOccurred() matcher with .Error()

Describe the bug
When using the .Error() chaining method on an expectation, the linter incorrectly finds an issue, even though this exact pattern is shown in the official documentation as
Ω(MultipleReturnValuesFunc()).Error().ShouldNot(HaveOccurred())

See https://onsi.github.io/gomega/#handling-errors
To Reproduce
package main_test

import (
	"testing"

	. "github.com/onsi/ginkgo/v2"
	. "github.com/onsi/gomega"
)

func TestE2E(t *testing.T) {
	RegisterFailHandler(Fail)
	RunSpecs(t, "demo")
}

func foo() (bool, error) {
	return false, nil
}

var _ = Describe("demo", func() {
	It("should work", func() {
		Expect(foo()).Error().NotTo(HaveOccurred())
	})
})

Returns
main_test.go:21:3: ginkgo-linter: asserting a non-error type with HaveOccurred matcher

Expected behavior
Get no warning.
Environment:

OS: macos
Version latest (0.18.1)
go: 1.22.9

Additional context
We first observed this when bumping golangci-lint, which bumped this linter from 0.16.0 to 0.18.0

Just release v0.18.2 with a fix to this bug.


Cannot start new project

Getting the following error in a new project:
Error in "/Users/tim/code/me/portfolio-v2/node_modules/gatsby-source-prismic-graphql/gatsby-node.js": Cannot find module './third-party/gatsby-node'
I followed these steps:

Created project with gatsby new portfolio-v2 https://github.com/margueriteroth/gatsby-prismic-starter-prist
cd into directory
npm i
gatsby develop -p 7000

What I tried:

gatsby clean
Removing node_modules and npm i again

Anyone else getting this?
Screenshot:

Found the issue is related to the gatsby-source-graphql-universal project
https://github.com/birkir/gatsby-source-graphql-universal/issues/19
Adding this resolution to package.json and using yarn is working to start the server
  "resolutions": {
    "gatsby-source-graphql-universal": "3.3.0"
  }


Error in managing a builtin story changes the icon anyway (until refresh)

Looks like it's the same when modifying a configured story raises an error. The story looks like modified besides the toaster error.

Actually the icon changes even when Cancel is pressed.


Xcode result bundle to checks 1.0.0

What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

New Step
Thank you for the new Step share! The CI check might will fail due to our extended validation engine. Nothing to worry about yet, we will get back to you shortly.

Thank you @kishikawakatsumi for your perseverance during this review process and again for your contribution, merging now!


Make Vault-Operator chart Helm 3 compliant

Is your feature request related to a problem? Please describe.
We've packaged Vault integration based on BanzaiCloud. It deploys the opertor, the webhook and one Vault instance.
Therefore we use vault-operator chart as a subchart.
Problem is that there is a breaking change between Helm V2 and Helm V3.
It was previously possible to add a specific Helm annotation on the CustomResourceDefinition ("vaults.vault.banzaicloud.com") to ask Helm to create it first.
This annotation has been replaced by a CRD dedicated folder in Helm V3.
Do you have any plan the Chart Helm3 complient ?
Warning : In Helm V3 it is not possible to use go template in CRD anymore !
At least, may it be possible to make crd.yaml exclusion possible (using an "if" arround it ?)
Thank's in advance,

The chart is Helm 3 compliant, but it is not using the crds directory currently. A PR is incoming to resolve this, thanks for reporting!


Demo Pipeline on heroku broken

The demo app (http://lambdaui-snapshot.herokuapp.com/lambdaui) shows this:

Looks like this was fixed by merging #94


Implemented: centralized oms instance navigator in settings page from dxp-components (#177)

Related Issues

Related Issue: https://github.com/hotwax/dxp-components/issues/177
Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

Used the centralized oms instance navigator on the settings page which is implemented in dxp-components.
IMPORTANT NOTICE - Remember to add changelog entry
Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

Verified and working fine in release v2.1.0.


ngspice: Fix regression due to missing X libs

Motivation for this change
Most of ngspice, esp. the ngspice binary, was not built due to missing X libraries.
Things done

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

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

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

/cc maintainers @bgamari @viric @spacefrogg rongcuid
@GrahamcOfBorg  build ngspice

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: ngspice
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: ngspice
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: ngspice
Partial log (click to expand)
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/digital.cm
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/analog.cm
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/xtradev.cm
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/xtraevt.cm
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/table.cm
gzipping man pages under /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/share/man/
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib  /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/bin
patching script interpreter paths in /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28
checking for references to /build in /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28...

Success on x86_64-linux (full log)
Attempted: ngspice
Partial log (click to expand)
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/table.cm
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/digital.cm
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/xtraevt.cm
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/spice2poly.cm
shrinking /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib/ngspice/xtradev.cm
gzipping man pages under /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/share/man/
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/lib  /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28/bin
patching script interpreter paths in /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28
checking for references to /build in /nix/store/yllr15hmq4w21zpwxixsc99lhp8zkcw4-ngspice-28...

Success on aarch64-linux (full log)
Attempted: ngspice
Partial log (click to expand)
these derivations will be built:
  /nix/store/hc45aahfdnfjrxw55bv3f9kcn69kl0ii-ngspice-28.drv
waiting for locks or build slots...
/nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28

Success on aarch64-linux (full log)
Attempted: ngspice
Partial log (click to expand)
shrinking /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28/lib/ngspice/analog.cm
shrinking /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28/lib/ngspice/digital.cm
shrinking /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28/lib/ngspice/spice2poly.cm
shrinking /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28/bin/ngspice
shrinking /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28/bin/cmpp
gzipping man pages under /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28/share/man/
strip is /nix/store/qg2agrqkf240s656d207zqhipl0bc2id-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28/lib  /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28/bin
patching script interpreter paths in /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28
checking for references to /build in /nix/store/d2ci5z4xi2i6khxggk7dacf4q7ypylna-ngspice-28...

@xeji Sorry if you are not the one responsible, but can/will this land in 18.03?

@timor I just tried cherry-picking this fix but by itself it doesn't seem to fix the package on 18.03, the ngspice binary still doesn't build.
A proper fix would need to incorporate some of the other changes made between 18.03 and master. I don't have time for that. Feel free to fix the package on 18.03 and open a pull request based on the release-18.03 branch.


some of PyNumberProtocol methods are broken

current signature:
fn __add__(&self, rhs: Self::Other) -> Self::Result

but python treat it as:
fn __add__(lhs: Self::Left, rhs: Self::Right) -> Self::Result

&self is Self::Left or Self::Right, depends on actual expression

this has to be fixed before 0.1 release


[VQC-CHALLENGE] [Qraiders] [Nov 4 2022]

Team members
David Liu
Yashaswini Manjunath
Yousra Farhani
Submission summary
VQC Challenge final private submission
qBraid account<EMAIL_ADDRESS>Additional comments

@davidlearn Do you want us to review the main branch or the Qraiders branch?

Qraiders branch

From: Ryan Hill @.>
Sent: Friday, November 4, 2022 3:01:46 PM
To: qBraid/HAQS-2022 @.>
Cc: David Liu @.>; Mention @.>
Subject: Re: [qBraid/HAQS-2022] [VQC-CHALLENGE] [Qraiders] [Nov 4 2022] (PR #80)
CAUTION: This email originated from outside the university. DO NOT click links or open attachments unless you recognize the sender and know the content is safe.
@davidlearnhttps://github.com/davidlearn Do you want us to review the main branch or the Qraiders branch?
—
Reply to this email directly, view it on GitHubhttps://github.com/qBraid/HAQS-2022/pull/80#issuecomment-1304016022, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQHBD7MMBVAAYKUZYQ5QBCDWGVMRVANCNFSM6AAAAAARXEZBGU.
You are receiving this because you were mentioned.Message ID: @.***>

@davidlearn Thank you team Qraiders for participating in the hackathon! Your final VQC submission achieved 100% accuracy with a quantum cost of 259. Your kernel estimator achieved an accuracy of 93%. Together, your submissions ranked 9th overall on the leaderboard. If you haven't already, we would really appreciate it if you filled out our exit survey. Additionally, I'd love to stay in touch as you continue your journey through quantum, so feel free to reach out to me directly on Discord (ryanhill1#5643), via email (ryanhill@qbraid.com), or on LinkedIn!


Add environment variable GDN_TRIVY_SCANNERS to documentation

After searching for a while, looking why trivy was not reporting on some test-insecure terraform files. I realized that it was not reporting them since they were misconfigurations and not vulnerabilities (the default value). A few rabit holes and blatant guessing later and I found that I apparently could use the environment variable 'GDN_TRIVY_SCANNERS'.
Please add this to the documentation to avoid others also having to pass this ordeal.
Acceptable values for the variable are: vuln, misconfig and secret. E.g. below:
GDN_TRIVY_SCANNERS: 'vuln,misconfig,secret'
Thank you in advance.

Thank you for you feedback we will review an update our documentation


open to tests

Hi, I just found out this project and looks amazing!
I tried to download the app but the website seems not reachable...
BTW, I'm a forester and I'm open for testing the app, if ready.

Hi!
Sorry for the late reply, I was on vacation recently.
For some reasons apkpure seems not working now...
Below is a google drive link of our apk:
https://drive.google.com/drive/folders/17-Yu3KXcgJiFapGc2AjJ2dHNC30YUbup?usp=sharing
Feel free to download and have a try!
We appreciate your interest and any feedback!
Best,
Miranda
On Thu, Jun 6, 2024 at 3:18 AM Nic @.***> wrote:

Hi, I just found out this project and looks amazing!
I tried to download the app but the website seems not reachable...
BTW, I'm a forester and I'm open for testing the app, if ready.
—
Reply to this email directly, view it on GitHub
https://github.com/MingyueX/GreenLens/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATV3I3DOKUPRJ7XKZPMQLZLZF5QBPAVCNFSM6AAAAABI3KTV7GVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMZTMNRVHA2TMMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

no problem and thanks for replying!
is that correct that the app size is 800 Mb? I haven't dowloaded it yet but I ask for confirmation :)

Yes it’s about 800+ Mb.
It’s kinda big but we were attempting to shrink the size.
Best,
Miranda
On Mon, Jul 1, 2024 at 01:47 Nic @.***> wrote:

no problem and thanks for replying!
is that correct that the app size is 800 Mb? I haven't dowloaded it yet
but I ask for confirmation :)
—
Reply to this email directly, view it on GitHub
https://github.com/MingyueX/GreenLens/issues/1#issuecomment-2198633777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATV3I3ACF5RT4NO4RXWWLFLZKBABZAVCNFSM6AAAAABI3KTV7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJYGYZTGNZXG4
.
You are receiving this because you commented.Message ID:
@.***>


Add children

Make it possible for jumpmen to have children.
Estimated time taken 2h.

Closed by 32f2844.


Fix: Change To Use Timezone Aware UTC datetime

Fix issue #2678
Based on 0.11.0rc2 code because when I attempt to use the main branch as the base I am getting an error:
  File "/app/aries_cloudagent/anoncreds/__init__.py", line 3, in <module>
    from ..config.injection_context import InjectionContext
ImportError: attempted relative import beyond top-level package

@Ennovate-com — I’ve kicked off the failed integration tests again in case it might have been an intermittent failure. If not, can you please take a look at the failing test?

If tests pass, I'll merge in main and enable auto-merging so we don't have to wait for the int tests again


CodeActions are slow to update

Describe the bug
[A clear and concise description of the bug]
Steps to Reproduce

open single folder...
go to file https://github.com/microsoft/pylance-release/blob/main/testing/single/src/codeaction.py
use the code action on 'os' to 'import os'
go to 'Mytype()' and notice only the type ignore action is listed
wait 10-20seconds and then the code action will update

Expected behavior
[A clear and concise description of your expectation]
Additional context and screenshots
[Add any other information such as screenshots, error message, or stack trace]
Sample Code
# place cursor on `os` and confirm lightbulb shows up
# and trigger quick fix and confirm `Add 'import os'` is listed
os

# place cursor on `Path` and confirm lightbulb shows up
# and trigger quick fix and confirm `Add 'from pathlib import Path'` is listed
Path

# place cursor on `userModule` and confirm lightbulb shows up
# and trigger quick fix and confirm `Add 'from lib import userModule'` is listed
userModule

# place cursor on `MyType` and confirm lightbulb shows up
# and trigger quick fix and confirm `Add 'from lib.userModule import MyType'` is listed
MyType()

Configuration information (If you are providing a diagnostics file (see below), skip this section)
VS Version: [Can be found under "Help->About Microsoft Visual Studio"]
PTVS version: [Can be found under "Help->About Microsoft Visual Studio->Installed products->Python with Pylance"]
Python version:
Debugger Type (if applicable): [New debugger or legacy debugger?]
Diagnostics file
The diagnostics file contains information such as the Python version, environments, loaded assemblies, configuration options, etc. This file makes it easier for us to reproduce and fix bugs, so we would really appreciate it if you could provide it to us. To locate the logs, you can can follow these steps:

Open the File Explorer on your computer.
In the address bar at the top of the window, type %TEMP% and press Enter.
The %TEMP% folder should open, look for file named PythonToolsDiagnostics_*.log, where * represents a series of numbers.

This file contain the diagnostic information that can help us diagnose and resolve the issue you're experiencing.
Please note that the %TEMP% folder is a hidden system folder, so you may need to enable the display of hidden files and folders in File Explorer to see it. To do this, click on the "View" tab in File Explorer, then check the "Hidden items" box in the "Show/hide" section.
You can attach the file to this issue or email it to us at<EMAIL_ADDRESS>If you email it, please add a link to this issue.
XXX

Tried to repro this one, the other code actions don't even show up.


Inconsistent keyword list in Parser.getKeywords() causes inconsistent scoring

I expect that the issues and PRs I've posted here are going to my own fork and no further, but if anyone out there is reading, your feedback is welcome.
In Parser.getKeywords():
# ...
uniqueWords = list(set(words))

keywords = [{'word': word, 'count': words.count(word)} for word in uniqueWords]
keywords = sorted(keywords, key=lambda x: -x['count'])

# ... returns: ...
keywords[0]  = {'word': 'foo', 'count': 100}  # rank: 1st (most common keyword)
keywords[1]  = {'word': 'eggs', 'count': 25}  # rank: 9th
keywords[2]  = {'word': 'bacon', 'count': 32} # rank: 8th
# ...
keywords[10] = {'word': 'spam', 'count': 25}  # rank: 9th
keywords[11] = {'word': 'bar', 'count': 19}   # rank: 13th
keywords[12] = {'word': 'ham', 'count': 25}   # rank: 9th
# ...

set() does not return an ordered enumerable, so uniqueWords is an unordered list. Iterating it to build keywords means this list is also unordered, so the return order of sorted() on the count key alone is inconsistent.
Down in Summarizer.summarize() it means sentence scores are also inconsistent:
# ...
(keywords, wordCount) = self.parser.getKeywords(text)

topKeywords = self.getTopKeywords(keywords[:10], wordCount, source, category)

# ... iterating sentences ...
sbsFeature = self.sbs(words, topKeywords, keywordList)
dbsFeature = self.dbs(words, topKeywords, keywordList)

# ... calculate sentence score based on these features ...
# ...

For consistency, when the summarize method calls getTopKeywords(), should it still pass a fixed list of ten keywords that's been sorted with more tiebreakers…
# replace: keywords = sorted(keywords, key=lambda x: -x['count'])
keywords = sorted(keywords, key=lambda x: (-x['count'], -len(x['word'], x['word']))

…every keyword that ranks 10th or better, i.e. this comprehension…
topKeywordSlice = [kw for kw in keywords if kw['count'] >= keywords[9]['count']]

… or maybe both?
Computationally, the advanced sort seems unnecessarily expensive, but I don't know if there's a rationale for exactly ten top keywords. What's the best way to make this work?

For anyone out there playing along at home, switching from a fixed ten to topKeywordSlice resulted in as many as 18 "top keywords" in my sample texts. It slightly boosted some scores, with one outlier getting a +14% bump… yet it remained the lowest scoring sentence.
Differences after change: percentage(change in standard deviations)

mean score, change in std. dev: +1.083% (±0.0132s)
median score, change in std. dev: +0.1667% (±0.0112s)
mode score, change in std. dev: +0.000% (±0.011011)
minimum score, change in std. dev: +0.0000% (±0.025s)
maximum score, change in std. dev: +14.06778% (±0.0341s)

The limited deltas are surprising.


Simplify inicialization with sandbox

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #21   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           8      8
  Lines         215    203   -12
  Branches       57     52    -5
=====================================
- Hits          215    203   -12

Impacted Files
Coverage Δ

src/utils/VueHelper.js
100% <100%> (ø)
:arrow_up:

src/components/ComponentFixture.vue
100% <100%> (ø)
:arrow_up:

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 34625ee...0cbb964. Read the comment docs.


Pomodoro page revamp

clears #967
Have added the changes like increasing the button size and moving that timer to the top

Hoping that no error occurs this time, if it does, please do share it


Playground: Replace output.{pl,rq}.json with output.{pl,rq}.yaml

I tried to inspect the output.pl.json and output.rq.json output for some queries to get know the PL and RQ structures and I found the JSON output very difficult to parse (for at least this)/a human. Once I passed it through https://www.json2yaml.com/ it was much more readable.
I think the playground output is probably for human consumption so there YAML would make more sense.
For the APIs it makes sense to keep it as JSON since those will probably be consumed by machines.
Not a priority.

Agree with this & #1533.
Probably the playground could do with an owner — not sure how your interests are shaking up with prql-query vs other things — but if you want to take on a set of quick wins, these would be good ones!

I don't think I can be the playground owner as my JS experience is pretty much non-existent but, to the extent that this can be handled on the rust side I can take a stab at it.
I'll have a look.

I don't think I can be the playground owner as my JS experience is pretty much non-existent

Yeah, great minds think alike, unfortunately... Someone with lots of JS experience would be a valuable addition to the project given your & my skills there!


Replying to wall messages via email should open a replies thread

Currently when someone writes on the group wall and another user replies via email, it will create another wall message. As we have the replies feature now, it would be better to stick the reply into a replies thread.
Steps:

Disable email batching for wall messages, so that we can clearly find out which message the user replied to
Insert message id as thread_id
???

I see there's a closed issue here, but this problem still appears to be happening. I see many actual replies to a wall message that are sent by e-mail and end up on the wall instead of a thread. Could it possibly be something that the user is doing? Or is still failing?

It used to work, but maybe it got broken? @nicksellen I remember you refactored this area recently, could it be....?

It wasn't me boss!
... but potentially could be. Maybe our tests are not covering this very well?

I think there's no test coverage for this.
Indeed, make_local_part(conversation, user) should be make_local_part(conversation, user, thread) if it's a group or wall conversation.

Ah, so I did break it probably by overlooking that. You want to fix? With a test? If not I would add it in my queue.

Depends on your queue length ;) I'm a bit unconcentrated but could have a look!

Maybe have a look then, skip the test if needed... seems useful to get the fix in.

I went for the more explicit variant, defining and checking conversation_supports_threads property on the conversation target - should be harder to miss out in future.
https://github.com/yunity/karrot-backend/commit/cd750d46501870dc2848aa941afe40e723d4a287


proposal: Go 2: error handling with || return err

Proposal Details
I have a new error return proposal:

very simple, very golang, very short
does not add new Keyword or Operators
good compatibility
not changing the error way

** For examples: **

i like it, do you like it?

Looks at lot like #21161.

func test1() (b []byte,err error ){
	b, err = os.ReadFile("./test1")
	//if-shortcut
	ifs l:=len(b);l==0||l>65536@nil,fmt.Errorf("bad data:%d",l)
	else b==[]byte("bad")@nil,fmt.Errorf("bad file:%d",l)
	else nil,err==io.ErrUnexpectedEOF@err
	else b,err==io.EOF@nil
	return nil,err!
}

@ruyi789 That looks like a completely different idea that should be discussed on a different issue.  Thanks.

To be honest, I really can't imagine ten years without any progress in error handling.

Eyyo Rust's approach is very good and Zig's too. Very convenient, simple, elegant and compact. We need some improvement around this.
Can someone of the Go team please inform us about how far we are for a decision on how to improve this for the sake of Go.

verbose？

This is very similar to #61750 and the recent #67955, with a slightly different syntax.  It's syntactically unlike anything else in Go, with a return not in a block.  The emoji voting is not in favor.  Therefore, this is a likely decline.  Leaving open for four weeks for final comments.

No further comments.


Including pstl

AIUI std::execution (added in C++17) requires pstl. Given there may be more usage of std::execution in the future ( for example https://github.com/conda-forge/cython-feedstock/pull/75 ), maybe it would be worthwhile to build clang with pstl support.

libcxx's C++17 support is still not finished, and pstl is currently broken. Hopefully it will be fixed for LLVM 15. I'm broadly in favour of building with pstl though.

Also, on second thought, the problem in that Cython PR was that GCC's libstdcxx depends on tbb for std::execution, but our builds don't (yet) reflect that.


Custom GraphQL objects for contents and metadata

Rather than always exposing contents and metadata from Irmin stores as strings in irmin-graphql, this PR allows exposing these types as custom GraphQL objects.
Consider an Irmin store containing contacts, that are stored as serialized JSON objects, e.g. {\"tags\":[],\"aliases\":[],\"phone\":...." }. Rather than simply returning the JSON string from irmin-graphql, a proper GraphQL object could be exposed with fields tags, aliases and phone. This is what this PR enables.
You can then do queries like the following:
query {
  master {
    get(key<EMAIL_ADDRESS>{
      tags
      aliases
      phone
    }
  }
}

The new functor Irmin_graphql.MakeExt adds a new argument compared to Irmin_graphql.Make, which has type PRESENTATION for configuring the GraphQL presentation:
module type PRESENTER = sig
  type t
  type src

  val to_src : t -> src
  val schema_typ : (unit, src option) Schema.typ
end

module type PRESENTATION = sig
  type contents
  type metadata

  module Contents : PRESENTER with type t = contents
  module Metadata : PRESENTER with type t = metadata
end

I'd be happy to get feedback on the terminology -- is "presentation" and "presenter" the best terms for this?
We could also opt for allowing a custom GraphQL object for Store.key.
(I'm missing proper combinators for Result and Option, but would like to avoid new dependencies 😢 For now, I've simply added something inline.)
/cc @zshipko @jonludlam

This is awesome!

Sorry, looks like you just need to update the mli file too, then I would consider this ready.

@zshipko Doh, made the change in a hurry last night 🤦♂ Updated the mli now.


Allow subclasses to renounce ownership

Subclasses of  Ownable are able to transfer ownership but are not able to set  _owner to a 0 address like  renounceOwnership does. Consider exposing an internal function to allow  Ownable subclasses to remove the owner.
This issue was reported in the LevelK audit.

While interesting, I'd rather we leave Ownable as-is to focus on other solutions, considering we're moving away from it, and only leaving it in the library for e.g. quick prototypes. This is, however, pretty much the only bit missing from its API to have it 'feature complete'. @frangio thoughts?


Edit sessions shows modal dialog

Testing #158798

Continue from web inside desktop
Let the window sit
Install update (which restarts the App)
🧀 you are instantly greeted with a modal dialog (I wasn't even in that window of VS Code)

Ideas

don't use a modal but the custom modal overlay which is more friendly and doesn't steep focus
don't show this when my local changes are no different to those on the server

don't use a modal but the custom modal overlay which is more friendly and doesn't steep focus

Is the custom modal overlay the same thing as the custom modal dialog, i.e.
await this.dialogService.show(Severity.Warning, 'my message', ['my options'], { custom: true });

don't show this when my local changes are no different to those on the server

:+1: I filed https://github.com/microsoft/vscode/issues/159689 for this.


ClientCredentialRequest Documentation Update

This PR contains an update for the ClientCredentialRequest documentation that indicates that only one scope should be passed in.

Codecov Report

Merging #6104 (2f3a4e7) into dev (81d34b4) will decrease coverage by 3.31%.
The diff coverage is 75.04%.

Flag
Coverage Δ

msal-angular
96.73% <96.63%> (+0.22%)
:arrow_up:

msal-browser
?

msal-common
83.22% <ø> (-1.33%)
:arrow_down:

msal-core
?

msal-node
79.73% <ø> (-3.66%)
:arrow_down:

msal-node-extensions
68.86% <59.13%> (-6.78%)
:arrow_down:

msal-react
94.24% <ø> (-0.45%)
:arrow_down:

node-token-validation
?

Impacted Files
Coverage Δ

...ions/msal-node-extensions/src/utils/Environment.ts
17.94% <0.00%> (-24.65%)
:arrow_down:

lib/msal-common/src/account/AuthToken.ts
100.00% <ø> (ø)

lib/msal-common/src/account/CcsCredential.ts
100.00% <ø> (ø)

lib/msal-common/src/account/ClientInfo.ts
100.00% <ø> (ø)

lib/msal-common/src/authority/Authority.ts
90.87% <ø> (+2.98%)
:arrow_up:

lib/msal-common/src/authority/AuthorityFactory.ts
100.00% <ø> (+11.11%)
:arrow_up:

lib/msal-common/src/authority/AuthorityMetadata.ts
100.00% <ø> (ø)

lib/msal-common/src/authority/AuthorityOptions.ts
100.00% <ø> (ø)

lib/msal-common/src/authority/AuthorityType.ts
100.00% <ø> (ø)

...c/authority/CloudInstanceDiscoveryErrorResponse.ts
100.00% <ø> (ø)

... and 23 more

... and 204 files with indirect coverage changes


[TESTING] Release testing

testing merged branches for release
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/docs

@adisuissa i tested locally with AZP_BRANCH set - docs were built correctly, in the right place, and with correct tags etc - so i think all is good
ill close this when it goes green

and closed :tada:


Fix wheel generation

Summary of Changes

Fix generating wheels when using vcpkg

Motivation
We need to be able to build Python wheels.
Implementation
Here are the fixes applied:

Pin vcpkg-tool-meson version to 0.60.2: Versions of meson past 0.62.0 require Python 3.7+. However, for wheel building, the environment has Python 3.6 which fails to build some packages.
Add some missing packages using yum: perl-IPC-Cmd
Add vcpkg libraries to linker path: it was previously only set for the deployment image. Now it's also configured for the development image
Add the CMake toolchain file for vcpkg to skbuild

Notes
For now, the CMake toolchain paths have been hardcoded in setup.py. It may need to be generalized later.

Build successful!


CB-12543: sourceImageDate missing from custom images

In case of custom image's query operation the related source creation date is also provided.
One option was to persist the date in the database, but this value can change so it should always be updated, so only this value will be retrieved at the moment of the query and sent as part of the response.

Converted it to draft. This will be a htofix and at this point it should not be merged to 2.43.


Archive/fix add genesis accounts

Explain your changes:
This PR fixes archive node that crashes on the new fork genesis config file. The fix is simply to let the archive to read the ledger file directly from the precomputed_values.
Explain how you tested your changes:
*
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet


Share as Web Page

This extension is better compared to the Original Onetab, but there was one feature missing and that was the ability to share a list of tabs as a webpage. I was wondering if you could add that in.

https://github.com/cnwangjie/better-onetab/issues/151

dang wtf this is over 2 years ago, must've been a GitHub noob back then
ty for reminding me of this lol


GPTQ inference issue on intel gpu

when I run gptq model on intel gpu, I meet above error.
Could you kindly to help to solve.

Sorry if this is stating the obvious, but...did you run source /opt/intel/oneapi/setvars.sh  first?

Will try to reproduce from our side.


Negative count and negative infinity

Is it possible to catch negative values?
Preferably from negative infinity to zero i.e -inf -> 0
I've tried:
(-inf-0){Lowest text to display}
(-1000-0){Lowest text to display}
but they don't seem to catch.

no does not yet support negativ values...would be great....pull request welcome

Workaround for simple case with just matching negative numbers is to use _plural for negative text and rest in _interval. This plugins falls back to _plural if it doesn't find correct range.
translation: {
  key2_plural: 'negative numbers',
  key2_interval: '(0-inf) positive numbers'
};

i18next.t('key2', { postProcess: 'interval', count: -10 }); // -> negative numbers
i18next.t('key2`, { postProcess: 'interval', count: 0 }); // -> positive numbers

Smart @aqum , thanks for commenting


NOISSUE: rename polymorth to polymorph

subj

Forgot to make proto, make pre-build doesn't regenerate protobuf files.


[ll] More mapping

Modifies the mapping API for core as we discussed.
Tracking is not yet implemented for render.
Currently using an unpublished version of ash for the Vulkan backend.
A nice side effect is that the render example now works with StackAllocator :)

Tracking is currently simpler than what we had previously. There is no stall on mapping access, instead the frames involving the buffer need to have been waited for during acquire_frame. More flexiblity will probably be needed at some point.

a few more nits, if you will ;)

bors r+
Why would we need to batch mapping if even Bulkan doesn't?

On Sep 15, 2017, at 18:21, msiglreith<EMAIL_ADDRESS>wrote:
@msiglreith approved this pull request.
Very nice progress! Looks fine from my side.
I don't know if it could be worth it to allow batching of acquisition and release calls. Maybe as future optimization? (low priority)
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.

Doesn't look like my online merge fix worked. @Bastacyclop please rebase and overwrite my commit.

bors r+

On Sep 15, 2017, at 19:30, bors[bot]<EMAIL_ADDRESS>wrote:
Merge conflict
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.

@kvark Vulkan does allow batching of flushes and invalidation calls (would safe us a few internal barriers on some platforms). We basically follow dx12's model and couple mapping (actual mapping done internally I assume) and flush/invalidation. So we basically 'remap' everytime we want to flush data for reading. Tight ranges are interesting for mobile hardware (as suggested by the d3d12 debug layer when mapping the whole buffer).

Interesting info, thanks @msiglreith!
So you propose to release multiple mappings at once? Sounds good to me.

On Sep 16, 2017, at 08:52, msiglreith<EMAIL_ADDRESS>wrote:
@kvark Vulkan does allow batching of flushes and invalidation calls (would safe us a few internal barriers on some platforms). We basically follow dx12's model and couple mapping (actual mapping done internally I assume) and flush/invalidation. So we basically 'remap' everytime we want to flush data for reading. Tight ranges are interesting for mobile hardware (as suggested by the d3d12 debug layer when mapping the whole buffer).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Virtual Playable Piano has been added

New game Playable Piano has been added

Close #2216

Checklist:

[✅] I have mentioned the issue number in my Pull Request.
[✅] I have commented my code, particularly in hard-to-understand areas
[✅] I have created a helpful and easy to understand README.md

Check committed files again


Can't disable test plan checking

I'm just giving spr a try, and don't want to use test plans in my commit messages. I've tried to change the configuration settings but it seems to be blocking my commit anyway:
❯ spr --version
spr 1.3.3

❯ tail .git/config
[spr]
        githubAuthToken = .....
        githubRemoteName = origin
        githubRepository = powerxai/application-v3
        githubMasterBranch = main
        branchPrefix = spr/cjols/
        requireTestPlan = false
        requireApproval = true

❯ git config --get spr.requireTestPlan
false

❯ spr diff --all
3f5c1c5 fix(alarms): Use all dimensions for Cloudwatch queue metrics
  💔  Commit message does not have a Test Plan!

Am I missing something?

I assume you mean the requireTestPlan setting?

Oops, sorry, yes of course!

What is this test plan thing anyway (is it documented somewhere?) and what does it have to do with this way of working? Can it be made entirely optional?

It's documented here: https://getcord.github.io/spr/reference/configuration.html

link broken - working: https://github.com/spacedentist/spr/blob/master/docs/reference/configuration.md


Security Plan

Over the course of each engagement, dev crews should be augmenting the ISE GamePlan with an ISE Security Plan. An ISE Security Plan is based on the ISE Security Plan template. Each plan is customized to the engagement and has more or less information in it based on the needs of the engagement. Crews are encouraged to work with Security Champions or a member of the Security TD to scope the template to the engagement and complete it.
ACTIONS
Check in the completed security plan to the project's repository so that threat model(s) can be integrated.
Review the security plan with the team.

Preparing the initial Customer Security Plan for distribution

@jayce21-ms - I couldn't find the document on the MDE channel. Can you please let me know where is it when you are back from vacations? Thanks!

@ydaponte The initial Security Plan has been shared to the MDE Channel in the General section.


Request to release jenkins-client v0.0.5

Consider jenkins-client is stable for now, and is suitable to be released v0.0.5.
hi @LinuxSuRen , please help us release it.

Thanks for your contributions. There are 8 contributions from @JohnNiang . That is awesome!!!


Fix ios source_files

Fix ios source_files.
Related issues: https://github.com/streem/react-native-select-contact/issues/45

Thank you!

Released 1.6.3 with this change.


Build errors on Windows with Visual Studio 2017

I've checkout out the latest master and am trying to build but receiving ~1900 build errors. Here are the first few:
rror	CS1703	Multiple assemblies with equivalent identity have been imported: 'C:\Users\rkeim\.nuget\packages\microsoft.netcore.app\2.0.0\ref\netcoreapp2.0\System.Diagnostics.Contracts.dll' and 'C:\Users\rkeim\.nuget\packages\system.diagnostics.contracts\4.3.0\ref\netstandard1.0\System.Diagnostics.Contracts.dll'. Remove one of the duplicate references.	TestHelpers.Tests(netcoreapp2.0)	C:\Users\rkeim\OneDrive\code\System.IO.Abstractions\TestHelpers.Tests\CSC	1	Active
Error	CS0433	The type 'TargetFrameworkAttribute' exists in both 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' and 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'	TestHelpers.Tests(netcoreapp2.0)	C:\Users\rkeim\AppData\Local\Temp\.NETCoreApp,Version=v2.0.AssemblyAttributes.cs	4	Active
Error	CS0518	Predefined type 'System.String' is not defined or imported	TestHelpers.Tests(netcoreapp2.0)	C:\Users\rkeim\AppData\Local\Temp\.NETCoreApp,Version=v2.0.AssemblyAttributes.cs	4	Active

I tried looking up several solutions (reloading the solution, cleaning/restoring NuGet packages, etc), but haven't had any luck yet. Has anyone else encountered this issue or have any suggestions?
Here's a bit more detail about my setup:

Windows 10 (v 1703)
Visual Studio 2017 (v 15.1)

@robkeim I have not seen this before and can't replicate it on either my home or work box.
Reading the first error, my gut would've been to delete the .nuget\packages folder and start a-new, but it seems you've already tried that.
I see a couple issues out there on StackOverflow where people are saying to make sure you have the latest VisualStudio and MSBuild.

Thanks @jpreese I'll try upgrading the Visual Studio version. I've been hesitant to do that because last time I did it created problems in another project I had :)

FWIW I'm running 15.3.3

I'm happy to report that updating to Visual Studio 2017 v 15.7.4 fixed the issue 👍

Awesome, good to hear!


Uncaught TypeError: grecaptcha.render is not a function

Uncaught TypeError: grecaptcha.render is not a function. I'm getting this error intermittently when loading invisible recaptcha on my site. For reference I have some code checking for when the recaptcha is loaded:
    if (!window.grecaptcha) {
      return Promise.reject('not ready to render recaptcha yet');
    }
    const captchaId = window.grecaptcha.render(this.captchaDiv, {
      badge: 'inline',
      size: 'invisible',
      sitekey: INVISIBLE_RECAPTCHA_SITE_KEY,
      callback,
    });

Is this an issue on Google's end? The issue just started happening

This issue was resolved by not calling render until the onloadcallback triggered.


What is the this.value for?

Hi, could you please explain why do you need the this.value from the readme.

You don't. Just use it on any service. Anything in MyType is an example, replace it with what you need.
Sorry for the confusion!

Thanks, it's clear, that I is just a demo, but maybe it's a useful feature? How could I make use of it?
Maybe this is possible?
var ev1 = new MyType(1);
var ev2 = new MyType(2);

var cb = function() { /* how to access this.value (1, 2) here */ }; //

ev1.on('event', cb)
ev2.on('event', cb)

ev1.emit('event')
ev2.emit('event')

Or would suggest to use it like this:
...
var cb = function(value) { console.log(value) };
...
ev1.emit('event', ev1.value)
ev2.emit('event', ev2.value)

Heh, hadn't considered that possibility.
What you could do is to change this line: https://github.com/rubenv/angular-tiny-eventemitter/blob/a6ef4f0e5d84df7eb2650bcbfc0b29df051e5696/src/tiny-eventemitter.js#L76
Replace null with this. That will allow you to use this.value in callbacks.

awesome, pr is coming


Add link to the R port

rwars!
(hey, people have to get their kick somehow, and for me it's "bad wordplay")

:+1: Cool.

Deployed, thanks!

Thanks!