Great Work.
Now, the ugly part, i have this problem with the simple lightbox, when i click in any of the "<a href" get:
Uncaught TypeError: undefined is not a function
I inspect with chrome for what is the error and this is the line in where the error shows up:
$(this).ekkoLightbox();
I inspect and try to fixed but still same the error :(
Maybe i missing something?
This is my definition in html:

Videos

And here is the Script:
$(document).ready(function ($) {
      $(document).delegate('*[data-toggle="lightbox"]', 'click', function(event) {
	 event.preventDefault();
	$(this).ekkoLightbox();
      });
});

I hope that you can tell me what i'm doing wrong or is a different way to do the same.
Thanks!

Sounds like you're not including either jquery, the bootstrap js, or the ekko javacript in the page. Post a link for more help

Hi @ashleydw, i have include all the necessary and the order is fine...

You need to post a link to your page or a jsfiddle

Hey ! Thanks a lot as well dude! That is properly awesome stuff.
But I have the same problem as well. Any thoughts ?
Cheers !

Problem solved with Bootstrap 3.3.1.
Thumbs up !

@MarcMagnin what were you using? Are you sure it's not a jquery dependency?

Don't think so.
Now I struggle a bit, I'm stuck with the "loading..." message.
Here is the HTML :

And here the JS :
$doc.find('*[data-toggle="lightbox"]').click(function () {
event.preventDefault();
return $(this).ekkoLightbox({
always_show_close: true,
type:'image',
});
});
Everything seems fine but still nothing in the modal.
I'm digging through the sources to identify the problem.
Regards.

Posting a link would be more helpful, but the console should tell you the error.
As a side note, you should use delegate rather than find (see docs for full example):
$(document).delegate('*[data-toggle="lightbox"]', 'click', function(event) {

It seems that the image.onload() is never reached in preloadImage :
preloadImage: function(src, onLoadShowImage) {
      var img,
        _this = this;
      img = new Image();
      if ((onLoadShowImage == null) || onLoadShowImage === true) {
        img.onload = function() {
             // this code is never reached

If I force the following code, ie, skip preloading, that does work. Any thoughts ?
 var image;
          image = $('');
          image.attr('src', img.src);
          image.addClass('img-responsive');
          _this.lightbox_body.html(image);

You need to post a jsfiddle or an actual page for help, i can take a guess that your image isn't loading fully

I solved this problem after adding ekko-lightbox.js on page.

I have the same problem running index.html  from ekko-lightbox examples
OS: Windows 7 64 Bit
Browser: Google Chrome
Editor: Atom with atom-live-server
Error Log from Browser:
Live reload enabled. /:615:3
TypeError: $(...).ekkoLightbox is not a function
<anonym>
 /:543
jQuery.event.dispatch()
 jquery.js:4640
jQuery.event.add/elemData.handle()
 jquery.js:4309
 /:543:28
TypeError: $(...).ekkoLightbox is not a function


Fix infinite loop on img.onerror

When onerror is called while unload was never called, we end up on an infinite loop updating an erroneous image.

@stefhak could you have a look?

@stefhak @adam-be : ping?

@stefanalund @superdump I've not been involved in this part. @adam-be coudl you take a look?

I did an alternative patch ( #597 ). I think that the reason for the current design is to allow some re-tries if error fires before load in the start up phase, and that is what I wanted to preserve.

This was fixed in #597. Thanks for reporting and fixing the bug! Even if we did decide to fix it in another way. :)


why dubbo Each connection can only fill a maximum of 7MByte

Dubbo has a payload check mechanism, the default limit is about 8Mb for each request or response.

I see payload as a single request, not a limit per second, and when I change the size, it doesn't affect TPS


TypeError: can only concatenate list (not "tuple") to list

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): N
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0-dev20190217
Python version: 3.5.2
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: 10
GPU model and memory: GTX 1080 8G

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
Traceback (most recent call last):
  File "t.py", line 16, in <module>
    model(data)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/keras/engine/base_layer.py", line 620, in __call__
    outputs = self.call(inputs, *args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/keras/engine/sequential.py", line 248, in call
    outputs = layer(inputs, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/keras/layers/recurrent.py", line 713, in __call__
    return super(RNN, self).__call__(inputs, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/keras/engine/base_layer.py", line 620, in __call__
    outputs = self.call(inputs, *args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/keras/layers/recurrent.py", line 2132, in call
    return [output] + states
TypeError: can only concatenate list (not "tuple") to list

Describe the expected behavior
Should not throw exception
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
import tensorflow as tf

model = tf.keras.models.Sequential()
model.add(tf.keras.layers.Masking())
model.add(tf.keras.layers.GRU(
    units=1,
    # Set `return_state` to `True` will cause the TypeError:
    #   "can only concatenate list (not "tuple") to list"
    return_state=True,
))

data = [1.]
data = tf.convert_to_tensor(data)
data = tf.reshape(data, (1, 1, 1))

model(data)

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

If we do not use tf.keras.layers.Masking, then it will not throw the exception
If we do not use return_state, then it will not throw the exception

The problem is at:
https://github.com/tensorflow/tensorflow/blob/2ae06ca49145bfee2e5e98c64ae5cbe064a58a33/tensorflow/python/keras/layers/recurrent.py#L2134
Which the state returned from K.rnn is a tuple instead of a list, so that  [output] + states will fail and throw exception.
https://github.com/tensorflow/tensorflow/blob/2ae06ca49145bfee2e5e98c64ae5cbe064a58a33/tensorflow/python/keras/layers/recurrent.py#L2107

Thanks for reporting the issue, I will take a look within this week.

Fixed.


Address fail forward unit test failures

The broken replacement chain unit test occasionally fails because it is
possible for either the a.v1 or a.v2 operators to cause the resolver to
fail. This commit updates the test to check that the resolver fails
because a csv is in the failed state versus checking that a specific
csv is in the failed state.
Signed-off-by: Alexander Greene<EMAIL_ADDRESS>

/lgtm


Add integration test

I think we should put the integration tests right into the test pipeline. This includes the previously included real world test against BadSSL (despite BadSSL itself is outdated), some popular websites and I also added a simple Tower+Hyper based server-client roundtrip test.
I have also changed the test engine to cargo-nextest, and it should give us a much faster and nicer interface to quickly respond to the test results. It is not tested locally with act yet but I think it should work out of the box.

I dont' think is is a good idea. It pollutes the dependencies and as I said before it brings dependencies that may not be in lock-step on what rustls we are developing.
Please see the openssl reference from validation directory and there isn't really more than we need to do - bad handling of SSL protocol itself is in the scope of rustls and our scope I believe is just to make sure ciphersuites etc work which we do.
If we need to address some primitive level concerns then we should isolate those test cases and bring them local and without external dependencies that pollute dependency tree and make issues hard to track down given multiple failing parts.
Integration tests IMO must not be dependant on external service or libraries that have dependency conflicts as they do now e.g. when we were between 0.22 and 0.23.

@pinkforest I do understand how the tests should be kept lean and mean but we need to be more comprehensive if we want this to be really useful with meaningful real world impacts.
How about we put basic (likely bare minimal) integration tests that is in lock step of the rustls version, and then we have a more complete integration test suite with real life examples, that can be triggered on demand once we found all the dependencies are clean.

I chose nextest because it boosted test time 3x in my local dev environment with useful indicators and more concise result. And it is also a slick peek of what cargo test would be in the future (it is the next generation cargo test executor, which would be merged after all in some day)

Another option would be making a separate integration-tester (or thereabouts, really it's an acceptance test suite, right?) crate with its own workspace / Cargo.lock resolution, and placing the code this PR places in tests into integration-tester/tests, which would prevent comingling the dependencies for rustls-rustcrypto and the other TLS implementations


restructured dom test in test/ui.test.js to avoid local intermittent failures

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Dhruv Chandwani seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Scroll bar appears when enable both expand row and pagination in the datagrid.

Select one ...  (check one with "x")
[X] bug
[ ] feature request
[ ] enhancement

Expected behavior
The height of datagrid should be auto scaling when open the expand show and no scroll bar should appear.
Actual behavior
Scroll bar appear when open the expand show.
Reproduction of behavior
https://plnkr.co/edit/jMCyX9?p=preview
Environment details

Angular version: 4.0.X

Clarity version: 0.9.X

OS and version:

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

@steven-zou: I don't think the datagrid should automatically scale to accommodate the expanded row. If it does that the elements on the page will move on every expand/collapse. When the data is loaded we specifically make sure that the height remains the same change on every page so that the elements on the page do not move on page change. I will cc: @youdz to confirm this.

Well, the datagrid does scale when it's not paginated. I think @adityarb88's analysis that the datagrid should keep the same height when paginated is correct, it prevents the rest of the page from jumping up and down.

@steven-zou: I am going to close this issue but feel free to re-open it if you have additional questions.


Full name correction in CHANGELOG.md

Correcting my full name.

Thanks for spotting this issue by the way! :-)

No problem, you're welcome!


crash in article load

Switch to manual generation in pane 3.  The program crashes.

Tested and verified.


ClientModel: Make ClientRetryPolicy.Default a property instead of a field

Address feedback from https://github.com/dotnet/runtime/issues/97711#issuecomment-1920022163.

API change check
APIView has identified API level changes in this PR and created following API reviews.
System.ClientModel


Fix Validate command

This issue fixes https://github.com/kubernetes-sigs/contributor-tweets/issues/272

I made here sample test: https://github.com/imnmo/test-github-actions/actions/runs/3248283496

/lgtm
/approve


Add support for SBML L3 Events

During the 2nd semester of 2022 I supervised a bioinformatics Honours student (Hanri van Heerden) who worked on support for SBML L3 events in PySCeS. The following was implemented:

handling of simultaneous events (i.e. triggered by the same trigger)
implementation and handling of priorities as event attributes, which determine the order in which the events are to be executed (according to the SBML spec)

Implementation of the following attributes is still outstanding:

InitialValue (currently hard coded to True)
Persistent (currently hard coded to True)

SBML import and export (round tripping) works.
To implement this, the PSC input file syntax for events was expanded as follows:
Event: name, trigger, delay, priority {
    event assignments
    }

The newly introduced priority attribute (integer or None) is optional to deal with legacy PSC files. If it is not specified, the event priority attribute is assigned to None in the model.
Example notebook and model files demonstrating the implementation are available here. That repo also contains a copy of Hanri's Honours report for reference.

Hi nice stuff, more by email.

The events branch has been merged into development, so all further development can take place there as discussed.


Search Query

Summary
WE NEED IT

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Memo: Elasticsearch のインデックス更新が必要。

StackOverflowException


feat(functions-py): update functions-py version

BREAKING CHANGE: Functions now raise exceptions on errors

@J0 yeah I will try and get async done for supabase-py library this week so we can add make build_sync back in for this library.


Update image build to incorporate kubebuilder merge

Also includes #140 > merge first

updates scripts/imagebuild.sh for kubebuilder merge
updates scripts/deploy-federation.sh, scripts/deploy-federation-latest.sh and hack/install-latest.sh to point to the image built by master
fixes scripts/delete-federation.sh by pointing to hack/install-latest.sh which is not contingent on a new image being built
updates docs/userguide.md to reflect changes

Just to create a record of it, I'm currently seeing this output with join command (no matter which image I use). The ERROR message doesn't seem to actually indicate an error.
ERROR: logging before flag.Parse: I0703 23:37:39.169435   14507 join.go:125] Defaulting cluster context to joining cluster name us-east
ERROR: logging before flag.Parse: I0703 23:37:39.169516   14507 join.go:129] Args and flags: name us-east, host: us-east, host-system-namespace: federation-system, kubeconfig: , cluster-context: us-east, secret-name: , dry-run: false
ERROR: logging before flag.Parse: I0703 23:37:39.208759   14507 join.go:183] Performing preflight checks.
ERROR: logging before flag.Parse: I0703 23:37:39.398666   14507 join.go:270] Registering cluster with the cluster registry.
ERROR: logging before flag.Parse: I0703 23:37:39.404412   14507 join.go:278] Registered cluster with the cluster registry.
ERROR: logging before flag.Parse: I0703 23:37:39.404437   14507 join.go:204] Creating federation-system namespace in joining cluster
ERROR: logging before flag.Parse: I0703 23:37:39.415783   14507 join.go:212] Created federation-system namespace in joining cluster
ERROR: logging before flag.Parse: I0703 23:37:39.415811   14507 join.go:215] Creating cluster credentials secret
ERROR: logging before flag.Parse: I0703 23:37:39.415828   14507 join.go:404] Creating service account in joining cluster
ERROR: logging before flag.Parse: I0703 23:37:39.421795   14507 join.go:413] Created service account in joining cluster
ERROR: logging before flag.Parse: I0703 23:37:39.421818   14507 join.go:415] Creating role binding for service account in joining cluster
ERROR: logging before flag.Parse: I0703 23:37:39.440463   14507 join.go:425] Created role binding for service account in joining cluster
ERROR: logging before flag.Parse: I0703 23:37:39.440926   14507 join.go:427] Creating secret in host cluster
ERROR: logging before flag.Parse: I0703 23:37:39.475066   14507 join.go:567] Using secret named: us-east-us-east-token-2r9fl
ERROR: logging before flag.Parse: I0703 23:37:39.482369   14507 join.go:599] Created secret in host cluster named: us-east-kkhzk
ERROR: logging before flag.Parse: I0703 23:37:39.482397   14507 join.go:436] Created secret in host cluster
ERROR: logging before flag.Parse: I0703 23:37:39.482411   14507 join.go:225] Cluster credentials secret created
ERROR: logging before flag.Parse: I0703 23:37:39.482421   14507 join.go:227] Creating federated cluster resource
ERROR: logging before flag.Parse: I0703 23:37:39.555822   14507 join.go:235] Created federated cluster resource

/cc @font
I made some updates to userguide.md since I had time today. PTAL.

@font: imagebuild.sh is invoked by the travis-ci at build on the master branch. You can see that in .travis.yml. It was merged with #119 so this is essentially just an update to be consistent with the kubebuilder changes.
@gyliu513: thanks for your review of the userguide. @font had already done the bulk of this work but hadn't opened a PR so we're going to wait to update the userguide with his PR coming soon!

Removed. Thanks @shashidharatd. This should be good to go now.

/lgtm
/approve


Rich text editor broken in 1.5

Thanks for all your great work on Wagtail!
Just did a pip install of 1.5 into an existing project, now the rich text fields display as plain text (<p> tags visible) and the editor buttons do not appear:

Console errors say Uncaught ReferenceError: makeHalloRichTextEditable is not defined once for each rich text field:

I ran python manage.py collecstatic --noinput --clear and python manage.py migrate after the upgrade... did I miss another step or is this broken?
Also: just confirmed that downgrading to 1.4.5 fixes this issue for me - rich text editor works properly.

Hi @thearrow - the makeHalloRichTextEditable function is defined in the file hallo-bootstrap.js, which is new in Wagtail 1.5. If you reload the page while you have the developer tools open at the 'Network' tab, do you see hallo-bootstrap.js listed in the downloaded files (and if so, does it show up as status '200')?

Hey @gasman!
Unfortunately I don't see hallo-bootstrap.js anywhere in the network tab (running runserver with 1.5):

Just to clarify - this is on the main Page editing view, rather than one of the more peripheral bits of Wagtail (such as a site settings view set up through wagtail.contrib.settings)...?
Is your 'body' field a direct field on your page model, or is it inside a StreamField or an InlinePanel?

This is on the main Page editing view, yes.
The RichTextField is inside an InlinePanel, suppose I should have mentioned that in the original issue.
Abbreviated code:
class HomePage(Page):
    ...
    content_panels = Page.content_panels + [
        ...
        InlinePanel('info_items', label="Info Items"),
        ...

class InfoItem(Orderable):
    page = ParentalKey(HomePage, related_name='info_items')
    body = RichTextField(blank=True)
    ...
    content_panels = Page.content_panels + [
        FieldPanel('body'),
        ...
    ]

Fixed in #2673


User replacement in login process

Hi,
I'm not sure if i'm using this package wrongly or it is a bug. I'm using different identity providers for different users in my system. It seems that i'm able to login with different user upon successful authentication in 3rd party identity provider. Login in idp with one user but replace it with another in attributes leads to successful login that another user in my system.
Djangosaml2 used only as service provider.
Here is what i mean.
User redirected to okta idp and login with email<EMAIL_ADDRESS>there. Idp configured in such way that it will pass<EMAIL_ADDRESS>(different email!) in attributes. Djangosaml2 doing authentication by info from attributes so it will login  user as<EMAIL_ADDRESS>on service provider side and not as<EMAIL_ADDRESS>3rd party Idp can substitute any email in attributes and djangosaml2 let it log in. But this both users use different idp in my system - okta and onelogin respectively and should not login instead of each other.
Is this my personal configuration problem or this package doesn't support multiply idp? Is there a way to restrict users to log in only from their corresponded idp out of the box?
DEBUG 2017-05-30 13:47:44,795 response 4789<PHONE_NUMBER>66256 --- Getting Identity ---
INFO 2017-05-30 13:47:44,795 response 4789<PHONE_NUMBER>66256 Subject NameID: <?xml version='1.0' encoding='UTF-8'?>
<saml:NameID xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion" Format="urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress">u1@okta.loc</saml:NameID>
DEBUG 2017-05-30 13:47:44,795 response 4789<PHONE_NUMBER>66256 Attribute Statement: <?xml version='1.0' encoding='UTF-8'?>
<saml:AttributeStatement xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"><saml:Attribute Name="email" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:unspecified"><saml:AttributeValue xsi:type="xs:string">u1@onelogin.loc</saml:AttributeValue></saml:Attribute></saml:AttributeStatement>
DEBUG 2017-05-30 13:47:44,795 response 4789<PHONE_NUMBER>66256 --- AVA: {'email'<EMAIL_ADDRESS>INFO 2017-05-30 13:47:44,796 client_base 4789<PHONE_NUMBER>66256 --- ADDED person info ----
DEBUG 2017-05-30 13:47:44,796 views 4789<PHONE_NUMBER>66256 Trying to authenticate the user
DEBUG 2017-05-30 13:47:44,798 backends 4789<PHONE_NUMBER>66256 attributes: {'email'<EMAIL_ADDRESS>DEBUG 2017-05-30 13:47:44,798 backends 4789<PHONE_NUMBER>66256 attribute_mapping: {'cn': ('first_name',), 'email': ('email',), 'sn': ('last_name',), 'mail': ('email',), 'Login': ('email',), 'Email': ('email',), 'uid': ('username',)}
DEBUG 2017-05-30 13:47:44,798 backends 4789<PHONE_NUMBER>66256 Retrieving existing user<EMAIL_ADDRESS>DEBUG 2017-05-30 13:47:44,799 backends 4789<PHONE_NUMBER>66256 Sending the pre_save signal
DEBUG 2017-05-30 13:47:44,813 views 4789<PHONE_NUMBER>66256 Sending the post_authenticated signal
DEBUG 2017-05-30 13:47:44,813 views 4789<PHONE_NUMBER>66256 Redirecting to the RelayState: /
[30/May/2017 13:47:44] "POST /saml2/acs/ HTTP/1.1" 302 0
[30/May/2017 13:47:44] "GET / HTTP/1.1" 200 70056

I can extend Saml2Backend class and check user and issuer from session_info. So it seems that it is my own config problems, but it is really easy to miss such security concerns when working with more then one idp.


WiP stats: Predefine symbols needed for user agent stats

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description: Based on https://github.com/envoyproxy/envoy/pull/10189 -- this can't be reviewed till that one is merged.
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

/lgtm api

(not sure why API shepherds were tagged, going to assume this was due to some earlier patch that is no longer present)

@itayd do you know what's up with the LGTM failure ^^?

@htuch i don't see any api files in here? might be a badly phrased error then: https://github.com/repokitteh/modules/blob/master/ownerscheck.star#L208
wdyt?

I think something went wrong in a merge, and suddenly a bunch of files I
didn't touch were part of the PR. Then I did another merge and it fixed
itself.
On Mon, Mar 9, 2020 at 6:22 PM Itay Donanhirsh<EMAIL_ADDRESS>wrote:

@htuch https://github.com/htuch i don't see any api files in here?
might be a badly phrased error then:
https://github.com/repokitteh/modules/blob/master/ownerscheck.star#L208
wdyt?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/envoyproxy/envoy/pull/10302?email_source=notifications&email_token=AAO2IPMDSM3UAH72FKFZGFLRGV247A5CNFSM4LD4LMXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOJJBIQ#issuecomment-596807842,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAO2IPJJALQQ6BZBOH6AHPDRGV247ANCNFSM4LD4LMXA
.


Sweep: Test

Actions (click)

[ ] ↻ Restart Sweep

❌ Unable to Complete PR
Please add more details to your issue. I need at least 20 characters to generate a plan. Please join our Discord server for support (tracking_id=914842c9c4)
For bonus GPT-4 tickets, please report this bug on Discord (tracking ID: 914842c9c4).

🎉 Latest improvements to Sweep:

New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
Use the GitHub issues extension for creating Sweep issues directly from your editor.

💡 To recreate the pull request edit the issue title or description.
This is an automated message generated by Sweep AI.


[DOC] typo fixes in regression extension template

Fixes some typos in the regression extension template.
Inaccurately, the columns of y were confusingly referred to as the index. This has been corrected.

We're currently processing your upload.  This comment will be updated when the results are available.


refactor(minifier): Move name mangler to another ast pass

The name mangler is currently using the symbol table before the the compression pass. It breaks if we start moving symbols and scopes. For example when an unused function is removed, all slots are shifted resulting a non-idempotent pass.
I'll also need to research on making the semantic pass sharable among the ast and the hir.

Done in https://github.com/Boshen/oxc/pull/439


doc: Documentation issue in 'develop/application/VERSION'

Application VERSION File parsing
throws this error:
zephyr/include/generated/app_version.h:19:27: error: too many decimal points in number
Unfortunatly I am not able to generate a VERSION, and the usage in main is used also as described.
I simply want to keep tracking on progress and mark floating hardware with version information using the VERSION Feature.
But it simply is not working.
<< The VERSION File is written as described in: https://docs.zephyrproject.org/latest/build/version/index.html#app-version-details >>
<< With simply data filed>>
VERSION_MAJOR = 2 VERSION_MINOR = 0 PATCHLEVEL = 0 VERSION_TWEAK = 0 EXTRAVERSION = develop
Environment is Linux with latest Zephyr rtos

Page: develop/application/index
Version: main
SHA-1: d2001a6fab85af0bde533e5613f12f98016d11ec

Tested on main with the above described file and working fine

It is the "APP_BUILD_VERSION" define which have a unquoted string, this throws the error.
I use now the "APP_VERSION_STRING" it works fine.
Thanks

It is the "APP_BUILD_VERSION" define which have a unquoted string, this throws the error. I use now the "APP_VERSION_STRING" it works fine. Thanks

That is because APP_BUILD_VERSION is not quoted, to use that as a string you would need to use STRINGIFY(APP_BUILD_VERSION)

Thanks this actually worked! :)
I couldn't figure out how to make the unquoted variable to a String.
The STRINGIFY Macro works like a charm, this is exactly what I was looking for.
You saved me from writing again a solution and inventing the wheel for the 100 times.
Now I can throw my dirty solution away where I basically build one with pristine, copy the app_version.h
append my variables to it.
Build again without pristine build and flash this.
It was a really bad solution.


Not getting updates

Hello @yassineaboukir ,
Setup sublert on 27th March to monitor 20+ domains but till today I am getting the same notification "Couldn't find any new valid subdomain" even the "output" folder is not showing updated files of fetched domains as it still showing the last updated date "Mar 27", am I doing anything wrong ?
Thank You

Hi @jaikishantulswani,
It might be true that Sublert couldn't identify any new subdomain deployed by the monitored organizations but it also could be a bug, would you make sure to update your repository to use the latest version 1.7.4? Then just copy the domains.txt file from the old directory to the new repo and run Sublert again.
Keep me updated.
Regards.

Hello @yassineaboukir ,
Thank you for your reply. It Works :+1:

Hello @yassineaboukir ,
Now it again showing the same notification "Couldn't find any new valid subdomain"  even after setting up the sublert again :(

Now getting the error in log There was an error opening one of the files: example.com.txt or example.com_tmp.txt  with the notification in monitoring We couldn't find any new valid subdomains

Hello @yassineaboukir ,
Any suggestion on this as I am still facing this issue.


testing.bridge.Bridge - ConcurrentHashMap - non-existing method java.lang.Class.getInterfaces

I am cross compiling a project that uses ScalaTest, and I get this error when running sbt test:
[error] Referring to non-existent method java.lang.Class.getInterfaces()[java.lang.Class
[error]   called from private scala.reflect.ClassManifestDeprecatedApis.loop$1(scala.collection.immutable.Set,scala.collection.immutable.Set,java.lang.Class)boolean
[error]   called from private scala.reflect.ClassManifestDeprecatedApis.subtype(java.lang.Class,java.lang.Class)boolean
[error]   called from scala.reflect.ClassManifestDeprecatedApis.$less$colon$less(scala.reflect.ClassTag)boolean
[error]   called from scala.reflect.ManifestFactory$SingletonTypeManifest.$less$colon$less(scala.reflect.ClassTag)boolean
[error]   called from scala.reflect.Manifest.equals(java.lang.Object)boolean
[error]   called from scala.reflect.ManifestFactory$SingletonTypeManifest.equals(java.lang.Object)boolean
[error]   called from java.util.Objects$.equals(java.lang.Object,java.lang.Object)boolean
[error]   called from private java.util.HashMap.loop$1(java.util.HashMap$Node,int,java.lang.Object)java.util.HashMap$Node
[error]   called from java.util.HashMap.java$util$HashMap$$findNode0(java.lang.Object,int,int)java.util.HashMap$Node
[error]   called from java.util.HashMap.java$util$HashMap$$findNodeAndIndexForRemoval(java.lang.Object)scala.Tuple2
[error]   called from java.util.HashMap.java$util$HashMap$$remove0(java.lang.Object)java.util.HashMap$Node
[error]   called from java.util.HashMap.remove(java.lang.Object)java.lang.Object
[error]   called from java.util.NullRejectingHashMap.remove(java.lang.Object)java.lang.Object
[error]   called from java.util.concurrent.ConcurrentHashMap.remove(java.lang.Object)java.lang.Object
[error]   called from private org.scalajs.testing.common.RPCCore.getPending$1(java.io.DataInputStream)scala.Option
[error]   called from private org.scalajs.testing.common.RPCCore.$anonfun$handleMessage$1(java.io.DataInputStream)void
[error]   called from org.scalajs.testing.common.RPCCore.handleMessage(java.lang.String)void
[error]   called from org.scalajs.testing.bridge.JSRPC$.org$scalajs$testing$bridge$JSRPC$$$anonfun$new$1(java.lang.String)void
[error]   called from constructor org.scalajs.testing.bridge.JSRPC$.<init>()void
[error]   called from constructor org.scalajs.testing.bridge.TestAdapterBridge$.<init>()void
[error]   called from org.scalajs.testing.bridge.Bridge$.start()void
[error]   called from static org.scalajs.testing.bridge.Bridge.start()void
[error]   called from core module module initializers
[error] involving instantiated classes:
[error]   scala.reflect.ManifestFactory$SingletonTypeManifest
[error]   java.util.Objects$
[error]   java.util.HashMap
[error]   java.util.concurrent.ConcurrentHashMap$InnerHashMap
[error]   java.util.concurrent.ConcurrentHashMap
[error]   org.scalajs.testing.bridge.JSRPC$
[error]   org.scalajs.testing.bridge.Bridge$

I can't see any of my classes or ScalaTest involved, so assume that the bug is in SJS's testing framework. scalaVersion is 2.13.3 and sjs version is 1.3.0

You are probably using Manifests somewhere. Manifests are not supported, because they require some reflection capabilities like Class.getInterfaces. You may not see any of your classes here because it goes through a call of Object.equals, which can come from anywhere, and can call any equals method of any class instantiated anywhere.
You can have more info on where the  SingletonTypeManifest was instantiated by typing last in sbt after the failed command. Attention, it's not for the faint of heart: this dumps a lot of logs. If you already know where your application uses Manifest, it's easier to directly to there.

yes, it was used indeed somewhere. Tricky to find, though.


Laboratory work #1, Ashfak Irem - 19FPL1

The deadline is not that far - consider pushing changes so that we can review them.

The deadline is not that far - consider pushing changes so that we can review them. (x2). Not so much time left, it is right time to start doing something. I'll stop receiving labs on October 2th 11:00 pm.

done


LeakyReLU twice in resblock_seq?

Hi @cwmok ,
thanks a lot for sharing the code!
In the resblock_seq, there is a LeakyReLU module between the PreActBlocks. While in the forward function of PreActBlock, another leakyrelu is called again, which means there are twice leakyrelus in between the PreActBlocks (correct me if I am wrong).
Is this intended?
Thank you!

Hi @BailiangJ,
You're correct. It seems that the LeakyReLU modules are duplicated, which is not intended. I will clean that out later.
Thanks for reporting the issue!


Banned Guests Logging

The AGC App needs to add the capability to keep track of banned guests. The Guest data collected should include the DL number and the State. The State is currently collected but banning could be done by DL/State combo. The thinking is that some state IDs are difficult for our App to scan all the data but it seems that the DL & the State would be more easily collected through the scanner. Keep the same collections but add DL if an ID is scanned. All adults will need to use their DL to register as a Guest. Banned Guests should be flagged on the Guest Log.

Interesting, will need the discuss the mechanics as we currently let them hand type their info in.


Nginx variables in liquid templates for headers policy

Added possibility to use all Nginx variables in liquid templating for
setting headers. Copied from logging policy.
The use case is that I would like to be able to use Nginx request_id as
a tracking id value and propagate it to backend services so we can
have one unique identifier from API gateway to backends.

Hi,
You don't really need to do this. The information that you add in there is already in the context, so that information, in this case, is duplicated.
See for example the following test:
https://github.com/3scale/APIcast/blob/827aca2f2808160a4ce2dcd7452dd653df3e6829/t/apicast-policy-headers.t#L681-L694
If you have doubts about the information that it's stored in the context, you can use the liquid context debug policy:
https://github.com/3scale/APIcast/blob/827aca2f2808160a4ce2dcd7452dd653df3e6829/t/apicast-policy-liquid-context-debug.t#L42-L72

Hi,
You don't really need to do this. The information that you add in there is already in the context, so that information, in this case, is duplicated.
See for example the following test:
https://github.com/3scale/APIcast/blob/827aca2f2808160a4ce2dcd7452dd653df3e6829/t/apicast-policy-headers.t#L681-L694
If you have doubts about the information that it's stored in the context, you can use the liquid context debug policy:
https://github.com/3scale/APIcast/blob/827aca2f2808160a4ce2dcd7452dd653df3e6829/t/apicast-policy-liquid-context-debug.t#L42-L72

Hi,
are you sure that I can get Nginx request_id variable?
I used context debug policy but couldn't find information about request_id there.
For example:

if I use {{ request_id }} in logging policy it shows Nginx request_id variable (like: 54fd9ce12a9af3822b22ffbfbaa28e48)
if I use {{ request_id }} in headers policy it evaluates to blank
I tested this in local 3scale apicast installation (by changing policy lua code and restarting apicast) and after changes done here headers policy can use {{ request_id }} in same way as logging policy.

Ahh ok,
Got it now, If you need request_id in headers you don't need to do that, you can do it on the following function:
https://github.com/3scale/APIcast/blob/827aca2f2808160a4ce2dcd7452dd653df3e6829/gateway/src/apicast/policy/ngx_variable.lua#L5-L18
Regards

Ahh ok,
Got it now, If you need request_id in headers you don't need to do that, you can do it on the following function:
https://github.com/3scale/APIcast/blob/827aca2f2808160a4ce2dcd7452dd653df3e6829/gateway/src/apicast/policy/ngx_variable.lua#L5-L18
Regards

Hi,
Am I understanding you correctly that I should build a new AIPcast for our 3scale if I want to change ngx_variable.lua (or even just add simple post hook to OpenShift template to change/replace this file)?
I added request_id = ngx.var.request_id, to return value of context_values() function and after restarting apicast variable is available in context.
Thanks a lot for clearing up this issue :)

I added request_id = ngx.var.request_id, to return value of context_values() function and after restarting apicast variable is available in context.
Yes, If you update the PR with this single change, It'll be merged, I'm ok with that.
The main reason why all variables are exposed is to take care that no sensitive information is leaked due to invalid policy configuration.
Regards.


Use shortened service names as the default for AWS services

Detail
Currently, awsdac describes service full-names by default.
Related to this behavior, awsdac has the following issue https://github.com/awslabs/diagram-as-code/issues/67.
From the perspectives of visibility and widespread usage, it would be better to use the shortened service names as the default.
Note
You can change the name by defining "Title" param in the template.
Diagram:
    ...
    Resources:
        ...
        ECSCFntest:
            Type: AWS::ECS::Cluster
            Title: "ECS(backend)"

I discussed with maintainers about this topic. The conclusion is here:

To avoid any potential trademark concerns, we will keep the default as the full name.
At the moment, you can modify the Title, so if you prefer a shorter name, please feel free to customize it on your end.


Windows Users of BookNLP

You may find this video helpful: https://www.youtube.com/watch?v=3l5ERF3QX0M&ab_channel=PythonTutorialsforDigitalHumanities
If it indeed is helpful I will ask about updating the ReadMe for this repo

Seems like it runs well in WSL for windows, it even can use cuda. You can access the files on your wsl ubuntu in windows file exploror by putting this into the address \wsl.localhost\Ubuntu\home.
Considering its just "wsl --install" in the Powershell it seems to be the easiest method I've found.

considering this never fully compiled for window I made a pip install for windows!
seems to be working on my end
'pip install windows-booknlp'


Fix ModelType.php for Symfony-2.8

Using null will raise following exception: Catchable Fatal Error: Argument 1 passed to Symfony\Component\Form\Extension\Core\Type\ChoiceType::normalizeLegacyChoices() must be of the type array, null given, called in .../vendor/symfony/symfony/src/Symfony/Component/Form/Extension/Core/Type/ChoiceType.php on line 266 and defined

I guess the simplest way would be to do similar thing as in EntityType for doctrine - to make choices_as_values default 'true' according to a comment in https://github.com/symfony/symfony/pull/16796

I think this has been solved by the choice list normalizer and can be closed. See symfony/symfony/pull/17163


Chatplutin UI displays a blank page

When testing the release 0.11.0 we noticed that the chatplugin ui is not working properly.

Fixed.


Is libtpa compatible with stand tcp protocol？

Can i only use libtpa on server and the client is linux kernel tcp?

Yes,  you can do that; Libtpa is compatible with standard TCP.


Radio: enable data whitening

This should allow the use of 2Mbit/s BLE mode and in general will improve synchronization between transceivers

closed by #128


Feat/aws

This PR adds AWS provider to a list of supported providers by Claudie.
Please, carefully review the templates, as they are the most important part of this PR.
As mentioned in #302, the aws.tpl and aws-lb.tpl are almost identical. Issue #295 is already trying to fix this issue, however, for now, we will have a duplicates

Since we now support more Cloud providers and Azure is on the way, is there a reason why we still only allows GCP for the DNS zone in the input manifest ? (the validation only allows GCP)

Since we now support more Cloud providers and Azure is on the way, is there a reason why we still only allows a GCP provider for the DNS zone in the input manifest ? (the validation only allows GCP)

This is a good topic to bring up again.
Our LB architecture requires a DNS record, because the DNS is what provides us with the HA aspect of the LB (as still shown in the docs).
Back then, Hetzner had no DNS, and we only had GCP besides that.

Now, possibly, we might add support for non-GCP DNS.
Or even redesign the entire LB architecture to avoid this burden, if somehow possible.

Note that the DNS thing is also the only part of Claudie architecture that isn't naturally on-premises compatible.
FYI @jaskeerat789 @MiroslavRepka @Despire

Note that the DNS thing is also the only part of Claudie architecture that isn't naturally on-premises compatible.

Could you please elaborate @MarioUhrik?

Note that the DNS thing is also the only part of Claudie architecture that isn't naturally on-premises compatible.

Could you please elaborate @MarioUhrik?

Everything else has been constructed with on-premises clusters in mind - storage, LB clusters, the way we create K8s...
DNS is the only aspect for which it is currently mandatory to actually use a cloud provider.

Everything else has been constructed with on-premises clusters in mind - storage, LB clusters, the way we create K8s...
DNS is the only aspect for which it is currently mandatory to actually use a cloud provider.

Thanks for the clarification

@Despire could you please review the changes in the recent commit, regarding dynamic firewall creation?
After the review is done, we can merge this PR.

Looks good to me. Great work in this PR @MiroslavRepka


Transition hooks swallow basename (failing test)

When using basename and hitting a redirect, the basename gets swallowed. I'm not sure if this is the correct place for the failing test, but it illustrates the issue.
After hitting /redirect-to-inbox, the basenames array should contain [ '/foo', '/foo', '/foo' ]. The last element however is empty: [ '/foo', '/foo', '' ]. This results in a wrong path: instead of /foo/inbox we get /inbox.
I've no idea how to fix this, but I hope the failing test helps!

Nit: Can you put this in useRouterHistory-test.js instead and squash those commits? That's definitely where it belongs.
Thanks for giving us something actionable!

Hold on a sec please.

This is the same as https://github.com/rackt/react-router/issues/2917. It's fixed on the latest history.

This should pass as soon as we have history 2.0.0-rc3.

Ok, great! Do you want this test anyway? Then I'd put it into use-RouterHistory-test.js

It doesn't hurt to have. I personally don't have a strong opinion as to where it should go.

Ok, I squashed the previous commits and moved the tests to use-RouterHistory-test.js.

Actually @timdorr why that file? All of our other test cases covering onEnter are in the transition hooks test file.

It's testing basename support, which is a useRouterHistory option. Transition hooks might be failing currently, but the unit test subject is on that function. It may end up breaking some other way in the future, so it will make the context of the unit test more opaque.

Makes sense, thanks.


[#558] Account system configs

Description
Please provide a summary of the pull request and the issue it resolves. Please add necessary details, context, dependencies, explanation of when review is needed (see next section), etc.
Resolves #558
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[x] Immediately
[ ] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[x] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[x] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My change requires updating the documentation.
[x] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[x] My changes generate no new warnings.
[x] New and existing unit tests pass locally with my changes.
[x] Any dependent changes have been merged and published in downstream modules.

Thanks @roberlander2!
@petyos, I am going to merge this for now so @mihail-varbanov can use this to test against. Please let me know if you see any issues or have any questions. Thanks!


Collecting of coverage value using plugin

How to preferrable collect the coverage from tests, using the kotest-gradle-plugin?
I tried to use jacoco, but after calling gradle kotest jacocoTestReport there will be an empty folder with jacoco reports.
can't send settings for reproduce , sorry.
can you help, please, with such feature?

jacoco is probably invoking test, I doubt it's gonna work with the gradle
plugin without some custom work.
On Wed, 14 Apr 2021 at 10:14, Andrey Romanov @.***>
wrote:

How to preferrable collect the coverage from tests, using the
kotest-gradle-plugin?
I tried to use jacoco, but after calling gradle kotest jacocoTestReport
there will be an empty folder with jacoco reports.
can't send settings for reproduce , sorry.
can you help, please, with such feature?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kotest/kotest-gradle-plugin/issues/15, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVSGT627XPQ5OWZM4ADD3TIWWNPANCNFSM425TFBDA
.

What i've done is a workaround where unit tests with coverage run via the test command and for integrations I am using kotest. This works for now for me.

Reading the jacoco docs, it seems like it should be relatively easy to do:
all tasks of type Test
https://docs.gradle.org/current/dsl/org.gradle.api.tasks.testing.Test.html
are
automatically enhanced to provide coverage information when the java plugin
has been applied
The kotest plugin uses a task of type test.
On Wed, 14 Apr 2021 at 10:27, Reza Malik @.***> wrote:

What i've done is a workaround where unit tests with coverage run via the
test command and for integrations I am using kotest. This works for now for
me.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kotest/kotest-gradle-plugin/issues/15#issuecomment-819607157,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVSGU7NGPZDQR2JRNXE4LTIWX4XANCNFSM425TFBDA
.

tasks.create("testtypes") {
tasks.withType {
println("hello $this")
}
}

Configure project :
hello task ':kotest'
hello task ':test'

So Kotest is being picked up properly as a "Test" type.
From jacoco:
https://github.com/gradle/gradle/blob/master/subprojects/jacoco/src/main/java/org/gradle/testing/jacoco/plugins/JacocoPlugin.java#L134-L136
So.... it might be as simple as making sure the jacoco plugin runs
AFTER kotest. My guess is they run in alphabetical order?
On Wed, 14 Apr 2021 at 10:31, Stephen Samuel (Sam) @.***> wrote:

Reading the jacoco docs, it seems like it should be relatively easy to do:
all tasks of type Test
https://docs.gradle.org/current/dsl/org.gradle.api.tasks.testing.Test.html are
automatically enhanced to provide coverage information when the java plugin
has been applied
The kotest plugin uses a task of type test.
On Wed, 14 Apr 2021 at 10:27, Reza Malik @.***> wrote:

What i've done is a workaround where unit tests with coverage run via the
test command and for integrations I am using kotest. This works for now for
me.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kotest/kotest-gradle-plugin/issues/15#issuecomment-819607157,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVSGU7NGPZDQR2JRNXE4LTIWX4XANCNFSM425TFBDA
.

probably yes, in alphabetical order
there should be some additional config in build.gradle.kts, or it should just work like this: gradle kotest jacocoTestReport ?

tasks.create("testtypes") { tasks.withType { println("hello $this") } }
Configure project :
hello task ':kotest' hello task ':test' So Kotest is being picked up properly as a "Test" type. From jacoco: https://github.com/gradle/gradle/blob/master/subprojects/jacoco/src/main/java/org/gradle/testing/jacoco/plugins/JacocoPlugin.java#L134-L136 So.... it might be as simple as making sure the jacoco plugin runs AFTER kotest. My guess is they run in alphabetical order?
…
On Wed, 14 Apr 2021 at 10:31, Stephen Samuel (Sam) @.> wrote: Reading the jacoco docs, it seems like it should be relatively easy to do: all tasks of type Test https://docs.gradle.org/current/dsl/org.gradle.api.tasks.testing.Test.html are automatically enhanced to provide coverage information when the java plugin has been applied The kotest plugin uses a task of type test. On Wed, 14 Apr 2021 at 10:27, Reza Malik @.> wrote: > What i've done is a workaround where unit tests with coverage run via the > test command and for integrations I am using kotest. This works for now for > me. > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub > <#15 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AAFVSGU7NGPZDQR2JRNXE4LTIWX4XANCNFSM425TFBDA > . >

I don't use jacoco myself, but we could create a sample project I suppose.
I think you might have to declare them in alphabetical order in your build
(not the order you run them at the CLI).
On Wed, 14 Apr 2021 at 10:45, Andrey Romanov @.***>
wrote:

probably yes, in alphabetical order
there should be some additional config in build.gradle.kts, or it should
just work like this: gradle kotest jacocoTestReport ?
tasks.create("testtypes") { tasks.withType { println("hello $this") } }
Configure project :
hello task ':kotest' hello task ':test' So Kotest is being picked up
properly as a "Test" type. From jacoco:
https://github.com/gradle/gradle/blob/master/subprojects/jacoco/src/main/java/org/gradle/testing/jacoco/plugins/JacocoPlugin.java#L134-L136
So.... it might be as simple as making sure the jacoco plugin runs AFTER
kotest. My guess is they run in alphabetical order?
… <#m_2525221493023811523_>
On Wed, 14 Apr 2021 at 10:31, Stephen Samuel (Sam) @.> wrote: Reading
the jacoco docs, it seems like it should be relatively easy to do: all
tasks of type Test
https://docs.gradle.org/current/dsl/org.gradle.api.tasks.testing.Test.html
https://docs.gradle.org/current/dsl/org.gradle.api.tasks.testing.Test.html
are automatically enhanced to provide coverage information when the java
plugin has been applied The kotest plugin uses a task of type test. On Wed,
14 Apr 2021 at 10:27, Reza Malik @.> wrote: > What i've done is a
workaround where unit tests with coverage run via the > test command and
for integrations I am using kotest. This works for now for > me. > > — >
You are receiving this because you commented. > Reply to this email
directly, view it on GitHub > <#15 (comment)
https://github.com/kotest/kotest-gradle-plugin/issues/15#issuecomment-819607157>,

or unsubscribe >
https://github.com/notifications/unsubscribe-auth/AAFVSGU7NGPZDQR2JRNXE4LTIWX4XANCNFSM425TFBDA
. >

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kotest/kotest-gradle-plugin/issues/15#issuecomment-819620913,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVSGQNC5FFDINXN4WIIATTIW2DJANCNFSM425TFBDA
.

@sksamuel will be very great, thank you!

This would be a nice feature for me too where I can run everything via kotest instead of test and then filter the unit tests using our other tag feature.


Make Repairnator compile on Java >= 9

Closes #1133

yes, let's do that!

I moved the method to repairnator-core.
To use it in maven-repair, I had to add repairnator-core as a dependency.
In repairnator-pipeline, the version of the dependency is given by the property revision in repairnator-root, but in this project this property is not accessible, as the parent project is not repairnator-root.
What version of repairnator-core should maven-repair use? Ideally it should come from repairnator-root.

To use it in maven-repair, I had to add repairnator-core as a dependency.

That's good.

What version of repairnator-core should maven-repair use? Ideally it should come from repairnator-root.

Yes.
Your work here is very interesting. In Maven-Repair we encapsulate repair tools per Mojo. We do the same in the pipeline. For example, NopolMojo is more or less the same as NopolRepairStep. This duplication is costly for maintenance. It would be good to unify this, for example NopolMojo may depend on and call NopolRepairStep. WDYT?

Sidenote: could you do the paper work on the Eclipse website so that check eclipsefdn/eca  is green? Thanks.

What version of repairnator-core should maven-repair use? Ideally it should come from repairnator-root.

Yes.

I'll work on that and making CI pass.

Your work here is very interesting. In Maven-Repair we encapsulate repair tools per Mojo. We do the same in the pipeline. For example, NopolMojo is more or less the same as NopolRepairStep. This duplication is costly for maintenance. It would be good to unify this, for example NopolMojo may depend on and call NopolRepairStep. WDYT?

Sounds good to me. Created #1137 to keep track of it. I can tackle it after this one.

Sidenote: could you do the paper work on the Eclipse website so that check eclipsefdn/eca is green? Thanks.

Done!

I'll work on that and making CI pass.

OK. Note that we have flaky tests in some jobs.

Ready for review @monperrus .
Two small notes:

I had to create two profiles, one for having Nopol (which works when the JDK includes tools.jar) and another for all other Java versions. They have automatic activation according to the Java version. For OpenJDK 8 it's necessary to specify has the version is 1.8 but does not include tools.jar and Maven only supports prefix matching automatic activation.
This PR doesn't make Repairnator work for NPEFix on Java >= 9 version, as maven-repair is used by NPEFix's step on repairnator-pipeline. See https://github.com/eclipse/repairnator/issues/1137#issuecomment-723585969.

Thanks a lot for this excellent first contribution @andre15silva


Does Renderlab works with Gymnasium Atari Environment ?

I tried to use ALE/AirRaid-v5 environment
This exception raised:
NamespaceNotFound: Namespace ALE not found. Have you installed the proper package for ALE?

NamespaceNotFound                         Traceback (most recent call last)
 in <cell line: 1>()
----> 1 env = gym.make("ALE/AirRaid-v5", render_mode="rgb_array")
2 env = rl.RenderFrame(env, "./output")
3 observation, info = env.reset(seed=42)
4 for _ in range(1000):
5    action = env.action_space.sample()  # this is where you would insert your policy
4 frames
/usr/local/lib/python3.10/dist-packages/gymnasium/envs/registration.py in _check_namespace_exists(ns)
358         suggestion_msg = f"Have you installed the proper package for {ns}?"
359
--> 360     raise error.NamespaceNotFound(f"Namespace {ns} not found. {suggestion_msg}")
361
362

As the exception suggests, you must install the ALE package in order to use the ROM files for these Atari environments. This is done automatically if you include the accept-rom-license flag in your installation of gymnasium
pip install gymnasium[atari, accept-rom-licesnse]


Support of LibreWolf

Возможна ли будущая поддержка этого расширения для браузера LibreWolf?

@Erghel As I know, LibreWolf is based on the same engine as Firefox. So it must work on it too, but if it doesn't then we are not going to support it in near future.


Error: Unauthorized

Hey, for some strange reason I'm getting Error: Unauthorized when trying to send a message, I am using normal user Credentials to login, as I don't want to turn my account into a bot, I am trying to make it so when I send a command it replies and removes the original message (sort of like /shrug), but it's not working 😢

any help would be greatly appreciated

Is that a selfbot, like running on your account?
Or a bot that is running another account that isn't a bot account?

Selfbot on my account

Login with a token instead of your email/pass, also you might wanna use an up-to-date version of d.js. In certain older versions you had to instantiate the client using new Client({ bot: false }); for user accounts.

I'm using 8.2.0, new Client({ bot: false }); worked, thanks Gexo


Validate form in child from another child

Versions:

VueJs: 2.5.17
Vee-Validate: 2.1.0-beta.8

Description:
Hi, is it possible to validate a form in child component from another child component?

parent
-- child with form
-- child with function this.$validator.validateAll

Steps To Reproduce:
It seems like the injection api doesnt help, cause the validate function is in another child
<3 this project. thanks for your tips.

If both children inject the same validator instance then they can call actions on the validator:
https://codesandbox.io/s/7m645zj4w1


content-encoding negotiation should be disabled when the response headers already contain content-encoding

If the response has content-length but not content-encoding, Bandit will go ahead to compress the response body, but leak the original uncompressed content-length. This results in client seeing the response as truncated.
If the response has content-encoding as gzip already, Bandit seems to do a second compression.
Ideally, Bandit should detect the presence of content-encoding in the response header, and skip compression if it is present. When there is no content-encoding in the headers, Bandit should remove the old content-length header, if any, and replace it with the compressed content-length.
According to:
https://ninenines.eu/docs/en/cowboy/2.6/manual/cowboy_compress_h/
Cowboy does it. As of now, I have to disable compression globally, which is sub-optimal.

Yep, agreed that this should be better handled as you suggest. I'm also having to be more aware of the returned headers (specifically Connection: close) for the issue underlying #149, so I'll make sure to fold both of these into a consolidated solution coming the next release (a few days away, I think).
Thanks!

Slated to be fixed in #165

Fixed in #165. Please reopen if further work here is required!


Patch facebook Oauth to work with both Python 2.7 and Python 3.6

I follow your guide thoroughly but can't make it work. I have tried both Python 2.7 and Python 3.6.
At first, it seems that facebook now requires a decoder. Try running the code in both python version results in  KeyError: 'Decoder failed to handle access_token with data as returned by provider. A different decoder may be needed.
So I make a simple fix for python 2.7 by importing json module and pass in a decoder, which literally means :
import json
[...]
        data = ...,
        decoder = json.loads
[...]

And now everything works fine. I am able to logged in via facebook. But when this method is applied with python3, it results in  another error the JSON object must be str, not 'bytes', which leads me to another fix:
import  simplejson as json
[...]
        data = ...,
        decoder = json.loads
[...]

Although I dont know the difference between the default json module in python3 and in the package simplejson, the codes now work great with both version of python.

I don't see anything in the Facebook docs regarding this having changed. I'm looking here, and they still pass the access token in the query string, at least that's what the docs say. The decoder argument you added is used for providers that pass the token in a JSON encoded payload, so what we need to figure out is why are you getting the token as JSON, and if this is a general change across the board.
Are you doing anything differently from my example?

@miguelgrinberg  You are not the only one who have read the docs and seen nothing changed. I am surprised and confused, too.
Here's a video of me running your code with Python 3.5.2
https://openload.co/f/HSo17lnJ_GI/Screencast_2017-04-02_13%3A10%3A33.mp4

This is another video running the codes with Python 2.7.12
https://openload.co/f/_kTy26TelMk/Screencast_2017-04-02_13%3A29%3A25.mp4
Hope it helps.

After some additional searching, it looks like Facebook API 2.3 and up return the token as JSON. Earlier versions used url encoding. From their change log: https://developers.facebook.com/docs/apps/changelog

[Oauth Access Token] Format - The response format of https://www.facebook.com/v2.3/oauth/access_token returned when you exchange a code for an access_token now return valid JSON instead of being URL encoded. The new format of this response is {"access_token": {TOKEN}, "token_type":{TYPE}, "expires_in":{TIME}}. We made this update to be compliant with section 5.1 of RFC 6749.

So yeah, looks like I need to update this.

@miguelgrinberg  Thanks for your investigation. It's hard to keep an eye here and there all the time.
Also,would you please look at the part
 me = oauth_session.get('me?fields=id,email').json()
        return (
            'facebook$' + me['id'],
            me.get('email').split('@')[0],  # Facebook does not provide
                                            # username, so the email's user
                                            # is used instead
            me.get('email')
        )

and try to print (me) before return? In my case it doesnt return the email, but the name. Yes, the name, without the need of spliting the email  so later I have to modify my code to
return (
            'facebook$' + me['id'],
            me.get('name'),
            me.get('email')
        )

I dont know if facebook has changed anything on this, though.

Thanks both - patch only works for me if I set:
import simplejson as json
Maybe I'm missing something, but there is no json function inside simplejson so
from simplejson import json
surely won't work?!

I committed a fix for this, but instead of adding yet another dependency (simplejson), I implemented a simple custom decoding function that handles the bytes to str conversion.


Bump Kubernetes dependencies to 1.18.3

What this PR does / why we need it:
This updated the Kubernetes APIs to 1.18.3.

Also, integration-test is unhappy :-/

nice work!   we need to discuss strategy of k8s version support.  A bump up in version potentially and likely means a pop off of a supported version... or an expansion of versions supported.
the request to hold means nothing more than that...  the solution looks good to me

this is a duplicate of https://github.com/kudobuilder/kuttl/pull/95 which has merged


Block a domain

Hello,
What would be a simple way of preventing use of some domains.
For example a client is using
Phishingdom
I suspended that account. The client added this in a different account.
Is there a simple approach for this?

Unfortunately, that isn't currently possible within Postal.
You will likely have to manage your client more directly to avoid bad behaviour.

Aight. Will post the solution here, if I have one.


GcpPubSubSource should support GKE metadata

Expected Behavior
When deploying a GcpPubSubSource on GKE, I should be able to omit its spec.gcpCredsSecret. At which point the reconciler will use the key exposed by GKE on http://metadata.
Actual Behavior
Failing to specify spec.gcpCredsSecret will cause the CO to either be rejected by the Webhook or never become Ready.
Steps to Reproduce the Problem

Create a GKE cluster with a NodePool that has a ServiceAccount with the PubSub viewer permission granted.
Create a GcpPubSubSource that doesn't specify spec.gcpCredsSecret.

Additional Info
This will make it easier to use GcpPubSubSource on GKE, removing the necessity of setting up a K8s secret entirely.
Originally requested in slack by @mattmoor.

gcppubsubsource has been removed from this repo. Please see http://github.com/google/knative-gcp
PullSubscription is the replacement.
/close


Include base_path's leading slash in request

Rummager identifies HMRC Manuals and Sections by their path including the leading slash.
It looks weird, but Rummager requests with these identifiers in their paths should include the leading slash and therefore contain a double slash ("//").

While I was writing the commit message, it didn't seem useful to describe the specific problem we were having here, but maybe I should?

@mikejustdoit Describing the problem in the commit message would have been useful - it would still be useful to add it to this ticket.

OK, here's the problem as I see it. I'm not sure what I'm describing is particularly useful:
We're using Rummager's delete_document, but until now the documents weren't being deleted as expected.
HMRC Manuals are indexed by Rummager with an ID copied from their base_path in this app. In stripping the leading slash from our Manual's base_path, we were sending Rummager a document ID it didn't recognise, and therefore couldn't delete.
From where we were looking, requests that "looked fine" were being made to Rummager, but the documents were still appearing in the search results.


MDDataTable | [ValueError] TableRecycleGridLayout.orientation is set to an invalid option 'vertical'

The MDDataTable appears to be setting this value incorrectly. I get this error when accesing the example on the kitchenSink app:
kivy.lang.builder.BuilderException: Parser: File "", line 113:
...
111:        height: self.minimum_height
112:        width: self.minimum_width

113:        orientation: "vertical"
114:        multiselect: True
115:        touch_multiselect: True
...
ValueError: TableRecycleGridLayout.orientation is set to an invalid option 'vertical'. Must be one of: ['lr-tb', 'tb-lr', 'rl-tb', 'tb-rl', 'lr-bt', 'bt-lr'

@Hnenry Use KivyMD library from master branch.


Add alt-text tags to Create Compliance Charts - Numbers page

Add alt-text tags to the Create Compliance Charts - Numbers page.
Page exists at: https://wildaid.github.io/charts/2020/06/12/Create-Compliance.html
In the code at: _posts/build_charts/2020-06-12-Create-Compliance.md
A brief guide to alt-text tags can be found in our style guide at https://wildaid.github.io/style/2020/10/02/Alt-Text.html

Hi @o-fish-wildaid, I have gone through the issue requisite and would like to contribute to it. I request to assign this issue to me, Thank you.

Hello! Thanks for expressing interest in this issue. Since you have been assigned issue #67 and haven't submitted a PR for it yet, we ask that you finish that issue first, before we assign another issue to you.

Hi @Sheeri! as mentioned by you earlier, I have submitted a PR for issue #68. Can I be assigned this issue once the PR for issue #68  is reviewed ?

@RuchaYagnik I have assigned this issue to you.


PermanentlyFailedException: Timed out waiting for service to come alive.

Hi there,
I'm trying this repo in my local Jupyter Notebook. My OS is Ubuntu. I'm getting the following error. Can you help me on this one ?

end of the error:

@ByUnal does it work if you just run the same thing with python and not jupyter?

@philipperemy Yes mate :/

that's weird. do you have java installed?

I was thinking that the java has already installed, but it wasn't actually :/. It works now. Thanks.

README updated.


Add option to specify default_icon as fallback

Closes #13
Displays a default icon if the general.default_icon option is specified, and no icon was found in the icons map.

Thank you for you contribution. I'll draft a new release with this change.


Enables switching SSL certificates on QUIC with QUICHE

This enables using certificates loaded by QUICMultiCertConfigLoader (not only the default cert but also certs for specific servernames and IP addresses).
This also makes other SNI stuff work on QUIC.

This depends on #9322, and most code change will be gone after merging it.
The most important change on this PR is use of quiche_conn_new_with_tls. This requires the both ATS and Quiche use the same version of BoringSSL.

This requires BoringSSL if we use Quiche implementation, and I think the autest failure is because of the build environment, which use OpenSSL(quictls).
@bneradt Can we use BoringSSL on the image for Quiche build?

[approve ci autest]

[approve ci autest]

This breaks the original QUIC implementation, and autests for HTTP/3 would be ran only if ATS is built with Quiche.


Compile for 3ds max 2016

Hi,
I'm wondering if it is possible to merge this PR into master? I need those changes to compile for 2016 max sdk and I apply them after each master update. It would be very helpful to have them in master.
Changes in PR:

project and solution files added for 2016
preprocessor #if-s check against max 2016
"afxres.h" replaced with "windows.h" to compile without mfc libraries
NormalizedDiffusionBSSRDFFactory replaced with GaussianBSSRDFFactory because by default appleseed doesn't contain NormalizedDiffusionBSSRDFFactory (at least on my version)

Thanks,
Sergo

Thanks for the PR! I'll review it tonight.
Regarding this:

NormalizedDiffusionBSSRDFFactory replaced with GaussianBSSRDFFactory because by default appleseed doesn't contain NormalizedDiffusionBSSRDFFactory (at least on my version)

You need to enable the Normalized Diffusion model when building appleseed by passing -DWITH_NORMALIZED_DIFFUSION_BSSRDF=ON to CMake.

I added a PR to remove the option to disable building ND.
https://github.com/appleseedhq/appleseed/pull/1422

Restored NormalizedDiffusionBSSRDFFactory

Excellent, thank you!


Add todo app example that leverages match and module directives

This pull request is based on https://github.com/relayjs/relay-examples/pull/95

[ ] Update this PR by copying all contents from #95 once feedbacks are incorporated

This PR is no longer required.
Please refer to https://github.com/relayjs/relay-examples/tree/main/data-driven-dependencies for working example of 3D


cmd.exe custom shell doesn't work with micromamba 2

@jjerphan did something change with micromamba shell init for cmd.exe in micromamba 2?

looks similar to #227

2.0.2 is being released and should fix this issue.

@pavelzw how about we create a new release of this action that's tied to the micromamba version? So that users can depend on setup-microamba@1 and get the 1.x release series (which can be override by manually setting the version) and then we release @2 after some more testing?

Yes, sounds reasonable :)
I'll create a new release later

Thanks @pavelzw!

#232

The thing that changed is that the micromamba.bat script generated by micromamba shell init -s cmd.exe is now called ... mamba.bat. When calling micromamba with v1, it was actually calling micromamba.bat, not the macro micromamba registered by the shell hook. So this was working by accident.
Since the Autorun is explicitely disabled, I think the GHA should copy the mamba.bat script into micromamba.bat to get this working again. The script should not have th same name as the executable because this makes things confusing and leads to a lot of issues when you install / play / debug many versions of micromamba / mamba.

@pavelzw: #234 must have resolved this issue. Can you confirm?

but it introduced #237


add Modular cross-selling in the app store description

The new description is:
Communicate, your way.
A chat app, under your control and entirely flexible. Riot lets you communicate the way you want. Made for [matrix] - the standard for open, decentralised communication.
Get a free matrix.org account, get your own server at https://modular.im, or use another Matrix server.
Why choose Riot.im?
 Chat, share files, add widgets and make voice and video calls - all free of charge.
POWERFUL INTEGRATIONS: Use Riot.im with the tools you know and love. With Riot.im you can even chat with users and groups on other chat apps.
PRIVATE AND SECURE: Keep your conversations secret. State of the art end-to-end encryption ensures that private communication stays private.
OPEN, NOT CLOSED: Open source, and built on Matrix. Own your own data by hosting your own server, or selecting one you trust.


chaiscript+meta.hpp

It's rather a question, not an issue. I was thinking about auto-binding reflected types to chaiscript, but can't figure out how to glue meta.hpp and chaiscript together.
To bind the type to chaiscript I need explicitly define all type's information: type, ctor, functions, etc.
For example:
chai.add(chaiscript::user_type(), "Vertex");
chai.add(chaiscript::constructor<Vertex(int32_t)>(), "Vertex");
chai.add(chaiscript::fun(&addVertex), "addVertex");
Is there a way to do it automatically using meta.hpp?

I'm not familiar enough with the API of this library, but it seems that with dynamic reflection (like meta.hpp) this is not possible. Instead, for this API, we must use static reflection because we need to bind types that we need to know at compile time. In dynamic reflection, we don't have this knowledge at runtime.
P.S.
We can do automatic bindings with the help of such API that "Lua" has, for example. Because it doesn't require static types for bindings.

I think it'd be great to have meta+lua sample


imagenet

Can you provide the code to handle Imagenet? Why do you mainly test imagenet but do not provide code to process imagenet data?
I have some confusion in processing Imagenet, for example, there are some grayscale images in Imagenet, and when I use dataset_tool, it will report an error, should I change the grayscale images to RGB images in this case? Expecting to read 65536 bytes but only got 1078. skipping tag 3.
Does anyone have access to imagenet?

Related:

#40
#43
#67


Create CSRF disable option for implementing APIs rather than webapp servers

Should still be on by default

Fixed by #18


Add SQL id, request logs, and metrics

Fixed #6301
This PR adds a new sqlQueryId method  to the QueryMetrics interface, which should be noted in the release notes.

@gaodayue Is this still WIP? I see "TODO" in the description

Hi @jon-wei , I'm waiting for review comments for proposal #6301 . If the proposal sounds good, I will continue to finish the PR.

Added UT and docs, this PR is ready to be reviewed. Hi @jon-wei and @gianm , could any of you guys help review?

@gaodayue  Sure, I can review, though I probably won't have time until next week

Hi @gaodayue, in the meantime could you please check the tests and inspections? In the tests, it looks like lots of CalciteQueryTests are failing due to the new sqlQueryId. Maybe the verifier that compares the queries for correctness should strip sqlQueryId first. In the inspections, it is reporting that the Query.getSqlQueryId() method is never used. If it's not needed, you can remove it.

Thanks @gianm . I will fix all failed test cases ASAP.

In the inspections, it is reporting that the Query.getSqlQueryId() method is never used. If it's not needed, you can remove it.

That method is for custom QueryMetrics impls to add "sqlQueryId" dimension to the metrics. I add @SuppressWarnings("unused") to the signature, hope the inspection tool can understand it.

Seems like the inspection tool still complains about the unused method, could you give me some suggestions on how to handle it?

@gaodayue maybe you can merge the master branch, it works for me.

@gaodayue @jon-wei re: https://github.com/apache/incubator-druid/pull/6302#discussion_r221352359,
I am thinking it will be best to have a single RequestLogger interface with methods like logNativeQuery and logSqlQuery. There are three main reasons:

It is just as flexible as having multiple interfaces. Imagine a "MultiRequestLogger" that can route different types of logs to different underlying loggers (SQL to kafka, native to file; or different files, etc). Sort of like the "composing" emitter we have.
It is more powerful, in that it can do one thing that the multiple interfaces cannot: it can write different types of logs to the same file.
Last but not least: it is simpler for users that are just getting started. They set up request logging one time, and all request logs go to the same place (of course, there should be a logType field that allows users to understand what is being logged). We could even extend this in the future to add exception logging and other sorts of structured logging (as opposed to log4j, which we use for unstructured logging).

What do you think?

Thanks @gianm. I think your arguments about single RequestLogger interface are reasonable, and I would like to give that approach a try.
Sorry for the late reply. I'm currently on a vacation, so I'm not sure when can I finish refactoring this, but I'll do my best.

Combined RequestLogger and SqlRequestLogger into one interface according to Gian's advice, the code looks a lot concise now. @gianm could your review again?

Sync with master again


[mesheryctl] system dashboard

Signed-off-by: Piyush Singariya<EMAIL_ADDRESS>Description
This PR introduces new subcommand dashboard under system with a port-forwarding implementation inspired by osm dashboard and linkerd dashboard
This PR fixes #
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Codecov Report

Merging #4557 (35af12c) into master (6e42595) will decrease coverage by 7.73%.
The diff coverage is 0.86%.

@@            Coverage Diff             @@
##           master    #4557      +/-   ##
==========================================
- Coverage   33.16%   25.43%   -7.74%
==========================================
  Files          60       62       +2
  Lines        5044     5159     +115
==========================================
- Hits         1673     1312     -361
- Misses       2935     3523     +588
+ Partials      436      324     -112

Flag
Coverage Δ

gointegrationtests
?

unittests
25.43% <0.86%> (-0.56%)
:arrow_down:

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/dashboard.go
0.00% <0.00%> (ø)

mesheryctl/pkg/utils/portforward.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/system.go
71.42% <100.00%> (+1.05%)
:arrow_up:

mesheryctl/internal/cli/root/system/start.go
1.11% <0.00%> (-58.52%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
2.50% <0.00%> (-47.50%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-47.44%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.86% <0.00%> (-27.59%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.29% <0.00%> (-20.88%)
: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 6e42595...35af12c. Read the comment docs.

**Note: layer5io/meshkit#125 It would be better if this could be merged before and I can use functions from meshkit **

@piyushsingariya Is this ready for review?

@piyushsingariya Is this ready for review?

This would be ready after making some changes in meshkit logic, sometime meshkit does return correct Minikube Network inteface IP address, but mostly(atleast in my PC; linux) it fails to detect it and the URL returned is always in this pattern localhost:port rather than IP:port.
@utkarsh-pro told that some changes needs to be made to correct once those are fixed, and if meshkit in my pc always detects correct address, it's ready to be merged.


Easy installation without compilation

I could see this tool being super useful in future for rendering HTML, etc. If we get there it would be nice for people to be able to install it easily, either by having it on Hex (possibly after converting it to Gleam?) or by having prebuilt binaries people can download from GitHub.
A bit more of a longer term goal this one, but I thought I'd open an issue anyway :)

Yeah, I don't have much experience with distributing Rust binaries but I'm happy to learn. Long term I can see the use it in being rewritten in Gleam too.


Go build is complaining about missing charly/gitea|gogs

Hi!
When I try building it, it seems that it's complaining about some missing files for Gogs and Gitea:
main.go:5:2: package charly/gitea is not in GOROOT (/opt/go/src/charly/gitea)
main.go:8:2: package charly/gogs is not in GOROOT (/opt/go/src/charly/gogs)

I use a custom compiled version of Go, but the same setup did work for compiling gickup :-)

Hi,
Sorry, I forgot to push those two modules. can you try it again?

I changed quite a bit in the code, the configuration file has changed.
But it should now work with every code versioning system, not just github, gitea, gogs and gitlab

Cool, thanks! It did build after your last changes, but I'm building a fresh copy now :-)

Works like a charm! Thank you very much!
One question, can you make the script section also multilevel? So that I can just run the same script for a whole list of repos? :)

I now run with the following config, might be useful as an example:
gitea:
  working-dir: /home/jeff/backup-git
  auth:
    token: my-secret-token
    ssh-keyfile: /home/jeff/.ssh/id_git_backup
    username: jeff
  repos:
    - url: https://git.example.com/Ansible/role-sshd
      script: |+
        if [ -d "$REPO" ]; then
          cd $REPO
          git pull origin $BRANCH
          git push -u github $BRANCH
        else
          git clone<EMAIL_ADDRESS>          cd $REPO
          git remote add github<EMAIL_ADDRESS>          git push github $BRANCH
        fi

Glad to hear that 😁
That's something I want to implement next. I will notify you as soon as I implement it.

I implemented it. You can check out the example config to see how it works.

Cool, works like a charm! Thanks!


Support XDG based installation path

Description
fzf can be installed with --xdg which will install it to $XDG_CONFIG_HOME/fzf. This plugin doesn't respect that. This PR fixes it.
License Acceptance

[x] This repository is Apache version 2.0 licensed (some scripts may have alternate licensing inline in their code) and by making this PR, I am contributing my changes to the repository under the terms of the Apache 2 license.

My linux work is mainly on servers where I don't customize my configuration, so thanks for adding the XDG support.


Post release automated changes for core releases

Post release automated changes for azure-core-amqp

API change check
API changes are not detected in this pull request.


Update how-to-debug-parallel-run-step.md

Update side_input usage.

@tracychms : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@tracychms
Thanks for the contribution! Merging now. In the future, it's generally best practice to submit changes into the private repo (azure-docs-pr) instead of the public. Just FYI. :)


add Synchronisation feature/dialog

On (multiple) row select and click of "sync button".
Show list of two cards side by side  with the properties of the two synchronisation partners
E.g.

gsimport versus wiki
gsimport versus wikidata
wiki versus wikidata

allow option per attribute:
←
→
✅
❌
Example (as table)

Property
wiki
⇆
wikidata

Label:
Xiao Liu
✅
Xiao Liu

Description:

←
Researcher ORCID(0000-0002-9226-4569)

ORCID:
-
←
0000-0002-9226-4569

Research Gate:
Xiao_Liu114
→

Potential Action: ⇆ ← → ☐
There should be a potential default for the whole card set. So all properties get to ⇆ ← → ☐ (see y-principle-gen)
The status should be marked e.g. with colors red for missing or error, green for same, blue for new
Conflicts shall also be marked with ⇆❌ and same entries  ⇆✅ sync entries should have the arrow ☐✅
There should be a button on top with a dry run selector that actually does the sync for all items and shows some kind of progress and debug and profile results. (Parallel handling possible ...)
Potentially there is a need for  extra details if the data has extra information such as a  language/type ...  this might need extra way of handlingt with another column or some popup or show/hide effect.

With our mapping schema we allow Qids and labels as inputs which are during the upload matched against the corresponding items. For the value comparison this means that we either need to normalize the records or convert them to an object that can evaluate the equality a given Qid and label.


ERROR dispatch not a supported wheel on this platform.

Using docker-compose version
I'm getting this error when trying to run pip install /tmp/dist/dispatch-0.1.0.dev0-py38-none-any.whl
"dispatch-0.1.0.dev0-py38-none-any.whl is not a supported wheel on this platform."

I was able to make it work by running pip intall wheel and making sure that pip is using 3.8.2 python version


Malloc size behaving incorrectly

The following code triggers a bug, that changes size of the second block to 33, while it's actually 22.
include std.cn

5 malloc
bind mem:
  "Hello" mem memcpy
  5 mem println
  6 mem extend_alloc
  print_malloc_list
  " world" mem 5 ptr+ memcpy
  11 mem println
  1 mem extend_alloc
  '!' mem 11 ptr+ !8
  print_malloc_list
  12 mem println
end

It was a problem inside the code itself. mem must be a variable and you should write to it every time you call extend_alloc.
include std.cn

var mem ptr
5 malloc !mem

"Hello" mem memcpy
5 mem println
6 mem extend_alloc !mem
print_malloc_list
" world" mem 5 ptr+ memcpy
11 mem println
1 mem extend_alloc !mem
'!' mem 11 ptr+ !8
print_malloc_list
12 mem println


fix: call hasYarn instead of using the function reference

There was a regression in 3.0.0, where in yarnOrNpm, instead of comparing the return value of hasYarn() to determine which package manager to use, it compared the truthiness of hasYarn (which is always true) and so yarn is always selected, even if it's not installed.

Thanks for finding this! I'll do a release shortly.


Cannot start in weston

Fails with the following error:
erresc: unknown str zxdg_surface_v6@17: error 3: xdg_surface has never been configured
Connection error

weston 2.0.0

This also happens under gnome-shell-3.24.1
wl_surface@16: error 3: buffer committed to unconfigured xdg_surface
Connection error

are you using gnome and wayland with nvidia blob driver?
On Sun, Apr 23, 2017 at 12:15:03PM -0700, Christian Hergert wrote:

This also happens under gnome-shell-3.24.1
wl_surface@16: error 3: buffer committed to unconfigured xdg_surface
Connection error

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/majestrate/wterm/issues/1#issuecomment-296481677

No I'm not using nvidia blob, this is on an intel iGPU (hd3000)

GNOME+Wayland on i915 for me. However this system does have dual graphics with both i915 and nouveau kernel modules loaded. To the best of my knowledge, only i915 is being used.

I got it running in weston 1.6.0 as of https://github.com/majestrate/wterm/commit/b9159b3c08da6debed6c4b7e608f9a3d33f6ad3f

try latest commit.


Added ColumnTypes support

In addition to getting columns with Columns (already implemented), sql package also supports ColumnTypes which is handy (and I really need to use 😄 )

Thanks!

Thank YOU for the great work! 😄


module 'aioredis' has no attribute 'create_redis_pool'

Hello folks!
You did a great job and your book is awesome.
I tried to run main.py with my ID but get this error:
Traceback (most recent call last):
  File "main.py", line 67, in <module>
    loop.run_until_complete(main(instance_name=args.name, force=args.force,
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "main.py", line 24, in main
    pool = await aioredis.create_redis_pool(address, db=db, password=password,
AttributeError: module 'aioredis' has no attribute 'create_redis_pool'

I did all preparations: ran local redis server and installed requirements.
Probably this happened because there is no certain version of aioredis in requirements.txt.

Ok. To fix this there should be specific version of aioredis. I get it with 1.3.0.
So requrements.txt content should be:
aioredis==1.3.0


raspberrypi3-64 not handled completely in MACHINEOVERRIDES

raspberrypi3-64.conf only sets raspberrypi3 as an override.  It also needs raspberrypi3-64 or the boot files don't have the proper config.txt settings including device tree.

Also the device tree list is incomplete/incorrect.

This was a misdiagnoses due to build confusion; the missing device tree was the real problem.


Running ollama on island device with no Internet connection

Trying to run ollama on island devices  with no Internet connection.  Getting the following error message.
Error: pull model manifest: Get "https://registry.ollama.ai/v2/library/llama2/manifests/latest":

Downloading a model requires internet connectivity.  You can download the model on a different computer and copy it across if the island device cannot be connected.  Where the model is stored depends on your setup.

Hi @whatdhack as @rick-github mentioned you'd need to be connected to ollama pull a model, or to ollama run (if you don't have it yet)

Can ollama use the  Meta  format  models ?  For example the 16GB llama 3.1 models ?

https://ollama.com/library/llama3.1

On a host with access to the Internet, I set the OLLAMA_MODELS variable. It was ignored and ollama  pulled the model to ~/.ollama .

https://ollama.com/library/llama3.1 shows it is using the 4 bit checkpoint

Still tries to  pull , even when the model was already available in ~/.ollama/models

https://ollama.com/library/llama3.1:8b-instruct-fp16

The ollama server does the work of pulling the model, so OLLAMA_MODELS has to set in the environment of the server.

Hmm, "OLLAMA_MODELS has to set in the environment of the server " - was in fact set

Server logs will help in debugging.


Add relationship icons to fields

Ticket: CS-6037

I've also added the animation to scroll into the overriding field when the overridden field is clicked.
https://github.com/cardstack/boxel/assets/12637010/fd136acc-2135-410a-bdc2-385f357d5a0a


Feature: Simplify output mapping

Addresses #66 most of all.

[x] simplify mapping.csv
[ ] simplify map_to_flexmex_results()?

map_to_flexmex_results() is still hard to read. I kept the 'name' column in mapping.csv in case we can also simplify the function.

I suggest to move mapping.csv somewhere else because it resides in a directory which is actually not part of the repo! Opened #96 for that.

Tested with UC 3.

Tested against defaults for UC 2a, 3, 7b and 10.


feat: add hexadecimal string validation

Thank you so much! This is great! Another idea I have is to add a validation function for strings containing color codes in HEX or RGB format. I recommend supporting the same color codes there as CSS does.

@fabian-hiller i plan to add hex color validation, octa string validation.


1.0.4 final version

No version is due after 1.0.4
(1.0.3 is skipped)
Tag: Final
Final version of Disvas. This PR confirms that any more version after 1.0.4 is forbidden.

No review required.


Response is not showing when Graphql invocation in tenant domain with invalid query

Steps to recreate:

Log in to store with the tenant domain
invoke the graphql API with invalid query.
The server shows the following WARN.

2019-10-20 22:26:46,483]  WARN - MultitenantMessageReceiver Tenant domain is not available in tenant request message context, hence it might not be set in the thread-local carbon context

Fixing https://github.com/wso2/carbon-apimgt/pull/7687/commits/92a8c2c4b5766d00f4cbf2b761755b6d6069e4a3

Fixed with https://github.com/wso2/carbon-apimgt/pull/7687


Add travis ci documentation

https://deploy-preview-459--roadie.netlify.app/docs/integrations/travis-ci/

@dtuite I have never come across the CISwitcher. Could you link me some documentation to it? I'll add resources for the annotations so.

@nicolasarnold12321 I might be calling it the wrong thing.
If I read these docs, I would understand how to configure the plugin, but I wouldn't know where to go to actually see it in Roadie. I think we should tell the reader how to accomplish that. That's the next thing they're going to want to do.

Ah ok. I'll see what I can do!

@dtuite I added some resources to the documentation there


New Rule: at least business support enabled

I confirm these files are made available under CC0 1.0 Universal (https://creativecommons.org/publicdomain/zero/1.0/legalcode)
No issue number applicable
New rule for checking if Business Support is enabled.

This rule just check if there is a support api available, not if it is business level. Everyone should have access to support API. Not really sure what is the logic here to determine if it is business level.

Good reference: https://github.com/awslabs/aws-config-rules/issues/151

As DescribeCases API is only allowed when Business support is enabled, this rule tests if an error is thrown when you try to use that API call. If ClientError exception is thrown it tells that business support is not enabled on the account. I've also added another exception code where it will show noncompliant if the role doesn't have enough permission to use Support API

'''
  #####################################
  ##           Gherkin               ##
  #####################################
  Rule Name:
  BUSINESS_SUPPORT_ENABLED

  Description:
  Check whether the AWS Account has Business support plans enabled.

  Trigger:
  Frequency every 3hours

  Reports on:
  AWS::::Account

  Rule Parameters:
  None

  Scenarios:
  Scenario: 1
    Given: AWS Account has ability to use Support API's DescribeCases call with results returned without ClientError Exception
     Then: Return COMPLIANT
  Scenario: 2
    Given: AWS Support API errors out with AccessDenied exception while using the DescribeCases API call. This means the role use by Config is not allowed to use the Support API.
     Then: Return NON_COMPLIANT
  Scenario: 3
    Given: AWS Support API call errors out with SubscriptionRequired exception while using the DescribeCases API call. This error means Business Plan is not currently used for the account
     Then: Return NON_COMPLIANT
  Scenario: 4
    Given: AWS Support API call errors out with any other boto3.client exceptions
     Then: Return NOT_APPLICABLE

  '''

Adding the updated Gherkin for review (apologies I didn't know Gherkin was to be submitted first)

As DescribeCases API is only allowed when Business support is enabled, this rule tests if an error is thrown when you try to use that API call. If ClientError exception is thrown it tells that business support is not enabled on the account. I've also added another exception code where it will show noncompliant if the role doesn't have enough permission to use Support API

Cool. I found the reference: https://aws.amazon.com/premiumsupport/plans/
So it means that your rule checks for Support plan Business or above (Enterprise), which is a good rule. Can you please change the Gherkin+Name of the rule to reflect that.
For your scenarios:

Good
No need, this is part of the boiler plate code
Good
No need, this mostly part of the boiler plate code. I do suggest on your except in your else statement to just to another raise.

Changed rule name to BUSINESS_SUPPORT_OR_ABOVE_ENABLED on both Gherkin and Folder name.
Put a raise on the last except statement
Updated the Gherkin as follows:
#####################################
  ##           Gherkin               ##
  #####################################
  Rule Name:
  BUSINESS_SUPPORT_OR_ABOVE_ENABLED

  Description:
  Check whether the AWS Account has Business or above(Enterprise) support plan  is enabled

  Trigger:
  Frequency every 3hours

  Reports on:
  AWS::::Account

  Rule Parameters:
  None

  Scenarios:
  Scenario: 1
    Given: AWS Account has ability to use Support API's DescribeCases call with results returned without ClientError Exception
     Then: Return COMPLIANT
  Scenario: 2
    Given: AWS Support API call errors out with SubscriptionRequired exception while using the DescribeCases API call. This error means Business Plan is not currently used for the account
     Then: Return NON_COMPLIANT

Please fix the error. They are all relevant.
Check those changes to fix the boilerplate. https://github.com/awslabs/aws-config-rdk/pull/158/files

Please fix the errors. They are all relevant.
Check those changes to fix the boilerplate. https://github.com/awslabs/aws-config-rdk/pull/158/files

committed as per guidance

************* Module BUSINESS_SUPPORT_OR_ABOVE_ENABLED
BUSINESS_SUPPORT_OR_ABOVE_ENABLED/BUSINESS_SUPPORT_OR_ABOVE_ENABLED.py:150:0: E0001: invalid syntax (, line 150) (syntax-error)

************* Module BUSINESS_SUPPORT_OR_ABOVE_ENABLED
BUSINESS_SUPPORT_OR_ABOVE_ENABLED/BUSINESS_SUPPORT_OR_ABOVE_ENABLED.py:150:0: E0001: invalid syntax (, line 150) (syntax-error)

the commenting was off around line 150. Accidental block comment of code instead of comments.

no errors on the linting for BUSINESS_SUPPORT_OR_ABOVE_ENABLED rule


Reed

Made some changes to the Reed Learning project story including images and text and created a new layout putting the image on the left and the text on the right.

I hope there's an explanation of that honeycomb graphic because it is ✨ baffling ✨

Lets see what its like when its deployed to staging

It probably makes no sense (the honeycomb) but let's see what people think

Could the honeycomb be put into an iPad graphic?