Clarify symmetric key remarks

The existing remarks reference session, which has a different meaning for users and connections. This remark refers to other keys in this database, which cannot have the same identity_source.

Thanks for submitting this change, @way0utwest! This looks good to me. I'll approve it.
FYI - @rwestMSFT
#sign-off


Create a set of default optimized copiers

Much like we have a default set of optimized serializers (for Long, String...) we should create a comparable set of copiers.
This would make store-by-value perform as fast as store-by-ref in some cases, like when immutable key/value classes are configured and the copiers can simply return the object without making any copy.
This would mostly impact the out-of-the-box performance of JSR107 as it requires store-by-value to be the default mode.

Or maybe more simply: register the IdentityCopier as the copier for all common immutable types like Integer, Long, String...


Add Example "Resume Vue"

/ping @ChangJoo-Park

@kazupon I just used this conflict to try out the new in-github conflict resolution feature :D
So this is good to go now, I will dismiss your review.

@LinusBorg
I got it.  :)


Documentation: Correct a bad link in the Data Command page

There was a bad link mentioned in #1685, so I've corrected it

Thanks!


v1.5 backports 2019-05-16

#8033 -- CI: Consolidate Vagrant box information into 1 file (@raybejjani)

Once this PR is merged, you can update the PR labels via:
$ for pr in 8033 ; do contrib/backporting/set-labels.py $pr done 1.5; done

test-docs-please


sitemap file name not changing when custom filename is specified

If I am not doing anything wrong, it seems that the newer version is not generating a sitemap with a custom name. I have copied the configuration from the docs and changed the filename but it is always generating the sitemap with the default sitemap.xml name.
this is my config

and when I run hugo to build the site, the sitemap in the public folder is sitemap.xml instead of the sitemap-1.xml in the configuration.

I know it was working in some versions ago, but I am not sure which one it was working. Also, I can be misconfiguring stuff, so I am sorry I am doing something wrong.
What version of Hugo are you using (hugo version)?
hugo version

 v0.126.1+extended

Does this issue reproduce with the latest release?
yes
Thank you for the great project! I love hugo <3

This worked in v0.122.0 and earlier.

Now it is working in the new version 0.126.2! Thank you @jmooring for the fix!!


What is the upper bound and lower bound limitation of max_combinations?

The default max_combinations in test engine settings is 20.
Is there any maximal number and is the minimal number 1?

Hello @rreexxllii0310,
The default is 20, min=0, and there is no maximum defined.
Thanks,
Marina

Thank you for applied!


define nodeSelector, Tolerations and Affinity for trivy and cache applications

Hello,
I tried to create a PR but it seems that it is not possible. I would like you to consider adding nodeSelector, Tolerations and Affinity for the Trivy and Cache manifests.
      {{- with .Values.nodeSelector }}
      nodeSelector:
        {{- . | toYaml | nindent 8 }}
      {{- end }}
      {{- with .Values.affinity }}
      affinity:
        {{- . | toYaml | nindent 8 }}
      {{- end }}
      {{- with .Values.tolerations }}
      tolerations:
        {{- . | toYaml | nindent 8 }}
      {{- end }}

And values with:
## Define which Nodes the Pods are scheduled on.
## ref: https://kubernetes.io/docs/user-guide/node-selection/
##
nodeSelector: {}

## Tolerations for use with node taints
## ref: https://kubernetes.io/docs/concepts/configuration/taint-and-toleration/
##
tolerations: []
# - key: "key"
#   operator: "Equal"
#   value: "value"
#   effect: "NoSchedule"

## Assign custom affinity rules to the prometheus operator
## ref: https://kubernetes.io/docs/concepts/configuration/assign-pod-node/
##
affinity: {}
# nodeAffinity:
#   requiredDuringSchedulingIgnoredDuringExecution:
#     nodeSelectorTerms:
#     - matchExpressions:
#       - key: kubernetes.io/e2e-az-name
#         operator: In
#         values:
#         - e2e-az1
#         - e2e-az2

Thanks!

Done


[US] As an Issuer, I want to pass my custom ABI to the SDK to get contextualized errors

As an Issuer,
I want to pass my custom ABI to the SDK to get contextualized errors,
So that my application can handle error cases seemlessly

Currently, only the errors defined in the DefaultPortal ABI (and main registries ABI are taken into account when handling an error thrown by a transaction simulation.
This story aims to add a parameter/option to the SDK Portal's function (attest, replace, etc.) to pass a custom ABI for the Issuer's Portal. This ABI would be merged to the default one, so that all errors can be thrown in a contextual manner.

assign this to me and tell me where exactly this code is.

i need help, where exactly should i modify the code ?

i need help, where exactly should i modify the code ? i dont know which file in SDK folder @alainncls

Hey! So, the ultimate approach would be to pass the custom ABI as an option, when initializing the SDK. But that implies a strong refactoring, let's avoid it for now.
Instead, let's pass this new optional parameter to the write functions of the PortalDataMapper:

[ ] attest
[ ] attestV2
[ ] bulkAttest
[ ] revoke
[ ] bulkRevoke
[ ] replace
[ ] bulkReplace

All these functions would also pass this parameter to their respective 'simulateXXX` function.
And finally, this custom ABI would be concatenated with the default one here.
Ping me if that's not clear 🙂


Use latest versions of CodeMirror and ClojureScript

I've attempted to fix all the issues I encountered while trying to get subpar to work with the most recent version of CodeMirror. The tests pass and everything seems to work but I would welcome feedback.

Looks good to me. Thanks!


Consider WebAssembly Studio in tutorial

One of the barriers to entry is installing the editor and build environment. There is actually an online IDE to allow building wasm projects from rust, assemblyscript or C directly in your browser: https://webassembly.studio/
I would look at this to simplify the intro tutorial, but you would want a real editor if you go beyond the demo. Still, it may make the barrier to entry lower - consider this if the build system is a hinderance. If rust itself is the barrier to entry, comment here: https://github.com/confio/cosmwasm/issues/74

Hi, are you working on this issue?
Feel free to comment here, and I'd love to see progress or feedback (is it a bad idea?)

Yes working on it. Will provide feedback next week.
On Wed, Mar 4, 2020, 18:03 Ethan Frey<EMAIL_ADDRESS>wrote:

Hi, are you working on this issue?
Feel free to comment here, and I'd love to see progress or feedback (is it
a bad idea?)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cosmwasm/docs/issues/16?email_source=notifications&email_token=AJTXH54YBGTERTXZEJRBJYDRFZDCNA5CNFSM4JZPBOF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENXUGHY#issuecomment-594494239,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTXH5YSWW277GTB24J6UEDRFZDCNANCNFSM4JZPBOFQ
.


Support regex or mid-level subdomain wildcard DNS

What is expected
Be able to provide a HOSTNAMES=subdomain.*.domain.tld env variable to a container on a network with a DPS container and have DPS match subdomain.some.domain.tld, subdomain.other.domain.tld, subdomain.mid-level.domain.tld, subdomain.subdomain.domain.tld, for example.
Alternatively, you could support regex. For example:

HOSTNAMES=subdomain\..*\.domain.tld to match questions that start with subdomain. and end with .domain.tld
HOSTNAMES=subdomain\.[a-z]+-test.tld to match questions that start with subdomain. and where the domain starts with alpha chars and ends in -test

Currently, I only need the former option but the regex option would be more flexible (and maybe easier to implement?).
What is Happening
DPS isn't able to parse mid-level wildcards or regex. Logs report Not solver for the question {URL}.
Changes (Optional)

Hey, I agree regex would be a better option. It would be a nice feature, not sure how many people need it, I'm working on better MAC OS and Windows compatibility right now but able to implement that on sequence, Do you still need it?
If you still need it and the feature is simple to implement as I'm thinking, the I may put it first priority.

Hey, yes this would still be a useful feature for a project I contribute to. Thanks for taking a look 🙂

Release notes:

3.14.0

Support for regex on localdb solver and docker container hostname and HOSTNAMES env

Hey @outdooracorn I'm proposing the support for regex on snapshot 3.14.0-snapshot , can you give it a try and give a feedback? See the examples below:

Be aware of the cache https://github.com/mageddo/dns-proxy-server/issues/376
There is a button on the GUI to clear the cache

Running a container
$ docker run --rm --name nginx1 --hostname '/batata\.[a-z]+\.com/' -e 'HOSTNAMES=/\d+\.acme.com/,/sub.acme.com/' nginx

Solving
$ nslookup -po=8053 batata.whatever.com <IP_ADDRESS>
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#8053

Non-authoritative answer:
Name:	batata.whatever.com
Address: <IP_ADDRESS>

$ nslookup -po=8053 xpto.acme.com <IP_ADDRESS>
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#8053

** server can't find xpto.acme.com: NXDOMAIN

$ nslookup -po=8053 123.acme.com <IP_ADDRESS>
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#8053

Non-authoritative answer:
Name:	123.acme.com
Address: <IP_ADDRESS>

$ nslookup -po=8053 sub.acme.com <IP_ADDRESS>
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#8053

Non-authoritative answer:
Name:	sub.acme.com
Address: <IP_ADDRESS>


Adding Reviews Page and static folder

Modified reviews.html template to display all the reviews
Added Static folder and added the STATIC_URL from settings file to urls.py in order to
load it in templates
Added "css" folder and icons folder in "static" folder
Added @pytest.mark.django_db to routing_tests.py since it uses the
objects from the db
Added reviews_tests.py file with one test to check that the template
was loaded (no other logic was applied in the page yet)
Modified 'review_test_data' file that each Review has a different date to test the sorting
Added main_feed() method to return the sorted data and a test for it
fix: #83

Note that I moved the { % extends ... % } & { % block content % } (app_layout) to the beginning of the html file, since is loads the bootstrap link and the CSS link was not applied unless it appeared after the bootstrap link.
So I figured out it's better than link the CSS file in the  instead of

@oriAdler @VeraTs moved course_load to the right


Update 105 Experimental 1 breaks BetterHands

It's a given that mods are going to break on the Experimental branch, that's why it's an Experimental branch of course. Unfortunately, BetterHands suffers this fate.

I have a fix, but I'm holding off until U105 is released to main branch (or maybe alpha) since the fix will break for anyone not on U105. I'm also looking into expanding magpalming to support the new attachable firearms.

Fixed with release v1.7.0. I'll update the Thunderstore listing once U105 has released on main/alpha branch.
Closing this issue, but feel free to reopen if you encounter any U105E1 specific bugs with the new release. Thanks!


Dash before @ (-@navbar-padding-horizontal in less/navbar.less)

Error in less variable (dash before @) -@navbar-padding-horizontal in less/navbar.less on lines: 110, 111, 182, 233, 287, 288, 382.

Those are minus signs, so as to get negative numbers. Pretty sure it's intentional.


OIDC4VCI test 4 fails : 2 issues

: the wallet cannot read the QR code
the error message is managed by teh error module

probbaly because the credential_type attribute is a string and not an array


Bug fix for mocha test

In Node.js environment we doesn't need JSONP, because the check will be done in the browser, client-side.

Thanks for this PR, I ended up doing pretty much the same thing here: https://github.com/lamp/superagent-jsonp/pull/10


Single Page State Pattern for Reflux+React

This is not an issue. Just sharing a pattern.
Description: https://github.com/souptown/react-refluxjs-single-page-state-pattern
Example: http://codepen.io/anon/pen/xwPEMe
I'll copy it here:
react-refluxjs-single-page-state-pattern
This is a pattern for implementing a single state hierarchy for a page in refluxjs. It uses the refluxjs aggregate store pattern.

All Reflux store changes trigger up to a top level store
A top level React component listens to the top level Reflux store

This is the only component that listens to a store

The top level React component propagates child store data to child components via props

Child components propagate data to their own children in the same manner

React components modify Reflux stores only by calling Reflux actions

Example Stores
Store hierarchy is:

PageStore

HeaderStore

UserStore

var PageStore = Reflux.createStore({
  init: function () {
    this.state = {
      header: HeaderStore.state
    };
    this.listenTo(HeaderStore, (newHeaderState) => {
      this.state.header = newHeaderState;
      this.trigger(this.state);
    });
  }
});

var HeaderStore = Reflux.createStore({
  init: function () {
    this.state = {
      user: UserStore.state
    };
    this.listenTo(UserStore, (newUserState) => {
      this.state.user = newUserState;
      this.trigger(this.state);
    });
  }
});

var UserStore = Reflux.createStore({
  listenables: UserActions,
  init: function () {
    this.state = {
      username: '',
      loggedIn: false
    };
    UserActions.fetch();
  },
  onFetchCompleted: function (user) {
    this.state.username = user.username;
    this.state.loggedIn = true;
    this.trigger(this.state);
  },
  onLogoutCompleted: function () {
    this.state.username = '';
    this.state.loggedIn = false;
    this.trigger(this.state);
  }
});

Propagating State
Component hierarchy is:

Page

Header

User

class Page extends React.Component {
  constructor (props) {
    super(props);
    this.state = PageStore.state;
    PageStore.listen(newPageState => {
      this.setState(newPageState);
    });
  }

  render () {
    return (

    );
  }
}

class Header extends React.Component {
  render () {
    return (

        <User {...this.props.user} />

    );
  }
}

class User extends React.Component {
  render () {
    return (

{ this.props.username }

      <button
        onClick={ () => UserActions.logout() }
        style={ { display: this.props.loggedIn ? '' : 'none'} }>Log out</button>
    );
  }
}

This is great!  We should work this into our upcoming "guides" with our docs.  For now, I'm going to turn it into a wiki page.

This is a nice pattern worth knowing about.  I've marked it as a resource so that we may revisit it for guidance as we pull together a site.  Further discussion can (and oughtta!!) take place over at the new discussion repo (https://github.com/reflux/discuss).


Error creating containers

$ docker run --rm -it -p 8000:80 -v "/Users/comp/Documents/GitHub/raster-processing-demo/src/mapserver-tiles/mapfiles:/usr/src/mapfiles" mapserver-docker
/usr/lib/python2.7/dist-packages/supervisor/options.py:296: UserWarning: Supervisord is running as root and it is searching for its configuration file in default locations (including its current working directory); you probably want to specify a "-c" argument specifying an absolute path to a configuration file for improved security.
'Supervisord is running as root and it is searching '
2018-08-06 06:03:29,782 CRIT Supervisor running as root (no user in config file)
2018-08-06 06:03:29,782 WARN Included extra file "/etc/supervisor/conf.d/supervisord.conf" during parsing
2018-08-06 06:03:29,788 INFO Creating socket unix:///var/run/mapserver.sock
2018-08-06 06:03:29,789 INFO Closing socket unix:///var/run/mapserver.sock
2018-08-06 06:03:29,801 INFO RPC interface 'supervisor' initialized
2018-08-06 06:03:29,801 CRIT Server 'unix_http_server' running without any HTTP authentication checking
2018-08-06 06:03:29,801 INFO supervisord started with pid 9
2018-08-06 06:03:30,807 INFO spawned: 'nginx' with pid 12
2018-08-06 06:03:30,808 INFO Creating socket unix:///var/run/mapserver.sock
2018-08-06 06:03:30,810 INFO spawned: 'mapserver_02' with pid 13
2018-08-06 06:03:30,815 INFO spawned: 'mapserver_03' with pid 14
2018-08-06 06:03:30,818 INFO spawned: 'mapserver_00' with pid 15
2018-08-06 06:03:30,821 INFO spawned: 'mapserver_01' with pid 16
2018-08-06 06:03:30,823 INFO spawned: 'mapserver_06' with pid 17
2018-08-06 06:03:30,825 INFO spawned: 'mapserver_07' with pid 18
2018-08-06 06:03:30,830 INFO spawned: 'mapserver_04' with pid 19
2018-08-06 06:03:30,834 INFO spawned: 'mapserver_05' with pid 20
2018-08-06 06:03:30,880 INFO exited: nginx (exit status 1; not expected)
2018-08-06 06:03:31,884 INFO spawned: 'nginx' with pid 21
2018-08-06 06:03:31,884 INFO success: mapserver_02 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2018-08-06 06:03:31,885 INFO success: mapserver_03 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2018-08-06 06:03:31,885 INFO success: mapserver_00 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2018-08-06 06:03:31,885 INFO success: mapserver_01 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2018-08-06 06:03:31,885 INFO success: mapserver_06 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2018-08-06 06:03:31,886 INFO success: mapserver_07 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2018-08-06 06:03:31,886 INFO success: mapserver_04 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2018-08-06 06:03:31,886 INFO success: mapserver_05 entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2018-08-06 06:03:31,900 INFO exited: nginx (exit status 1; not expected)
2018-08-06 06:03:33,907 INFO spawned: 'nginx' with pid 22
2018-08-06 06:03:33,921 INFO exited: nginx (exit status 1; not expected)
2018-08-06 06:03:36,931 INFO spawned: 'nginx' with pid 23
2018-08-06 06:03:36,946 INFO exited: nginx (exit status 1; not expected)
2018-08-06 06:03:37,927 INFO gave up: nginx entered FATAL state, too many start retries too quickly

Try starting the MapServer container like so:
docker run --rm -it -p 8000:80 --name mapserver-container -v "/Users/comp/Documents/GitHub/raster-processing-demo/src/mapserver-tiles/mapfiles:/usr/src/mapfiles" mapserver-docker

Then see if you can output any logs from nginx, like so:
docker exec -it mapserver-container sh -c "cat /var/log/supervisor/nginx-std*log"

Do you already have some service listening on port 8000?

I dont have any service running in 8000..
docker exec -it mapserver-container sh -c "cat /var/log/supervisor/nginx-std*log"
nginx: [emerg] unexpected end of file, expecting ";" or "}" in /etc/nginx/sites-enabled/mapserver_proxy.conf:1
nginx: [emerg] unexpected end of file, expecting ";" or "}" in /etc/nginx/sites-enabled/mapserver_proxy.conf:1
nginx: [emerg] unexpected end of file, expecting ";" or "}" in /etc/nginx/sites-enabled/mapserver_proxy.conf:1

I tried in both windows and mac machine... i am facing exact same issue

Huh, that's strange.  Did you modify this file at all?
https://github.com/pedros007/mapserver-docker/blob/b55d2ebd6eb1b97669d31cdad52a2754bb12591e/etc/nginx/sites-available/mapserver_proxy.conf#L1-L17

Haven't heard back & can't reproduce.


Highlighted button text

Related Issue
Closes #1222

Description

Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@SiddhantPawar03 Please review this.

@crocmons Please merge this ready to merge pr


Prevent panic if flag.Parse hasn't been called

Solves issue #410

Coverage: 95.655%. Remained the same when pulling 892a2f75ab0e6dcd950b100e924ab5008db2819a on paul-at-cybr:master into ebb8d4d3adcc8f7fc825da409a21fbf3a34c5431 on gavv:master.


Hindi localization maintainer updates

Message to new contributors: We are not seeking contributions on this issue. This issue is to nominate @dipesh-rawat as one of the leads for the Hindi localization.
As part of our ongoing efforts to keep the Hindi localization efforts up-to-date, @anubha-v-ardhan and I are pleased to nominate @dipesh-rawat as one of the leads for the Hindi localization. He has proactively triaged issues, reviewed PRs, and submitted changes to the Hindi localization. We're confident that the Hindi localization efforts will benefit from his experience as an approver, reviewer, and contributor to the English website.
In addition to the above, we'd also like to announce that @anubha-v-ardhan will be stepping down as a localization maintainer due to his ongoing commitments. We'd like to take this moment to express our heartfelt gratitude for all his efforts during the bootstrapping process and subsequent maintenance. His contributions have been invaluable to our localisation efforts.
As this is a named role change, we'll be timeboxing the comments on this issue for one week i.e. 24th May, 2024 EOD anywhere. Once the timeframe has passed and if there are no outstanding concerns from any community members, we will be proceeding with onboarding Dipesh into his new role & offboarding Anubhav from his existing one.
cc: @seokho-son @a-mccarthy @reylejano @natalisucks

Since the timeboxing period has passed, I'll open a PR to make these changes and linking it to this issue.

As PR is already merged for the same
/close


improve @InlinedImage to detect mime type

Auto detect the mime type of images so the @InlinedImage can be used with SVG, JPEG, ... images

Could you add tests to prove that this works?

Just tested and can confirm this works with svg and png

I am currently on holidays, I will try to add some tests when I come back in January.


Create rathanaphon

Hi,
I see you submitted this PR and https://github.com/googleads/google-ads-java/pull/704, but it's not clear what their intended purpose is. Could you elaborate on why you'd like to add these?
Thanks,
Josh, Google Ads API Team


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2131

Questions ?
Please refer to the Backport tool documentation


add movies cards

Built 6 nice cards in responsive and amazing layout!

Thanks for your contribution

@all-contributors add @mohamedsgap for code


sap.m.upload.UploadSet aggregations do not get an update as uploadState property shifts

Hi gurus,
We are using UploadSet with "incompleteItem" aggregation due to the fact that we are processing our uploaded documents first, and set them as "Complete" at a later point. As the processing gets finished, we set the incompleteItem's "uploadState" property as "sap.m.UploadState.Complete".
Our expectation was that as the "uploadState" property gets an update to "Complete", the "incompleteItems" and "items" aggregation would get an update too. The processed item gets reducted from the "incompleteItems" aggregation and added into the "items" aggregation. However, we got to see that it is not the case. The incompleteItem object stays in the "incompleteItems" aggregration, even if it has a uploadState of "Complete".
We tried to manually remove the item from the "incompleteItem" aggregation and add/insert it into the "items" aggregation but it didn't work, because the object id was already used and removed.
UI5 Version: sap.m.upload.UploadSet 1.120.8
We are asking your support and expertise on this matter,
*A follow-up incident of the main incident: https://github.com/SAP/openui5/issues/4026
Thanks in advance,
Best,
Merve

Hello @cbsmerveguel ,
Thank you for sharing this finding. I've created an internal incident DINC0129133. The status of the issue will be updated here in GitHub.
Best Regards,
Nikolay Hristov

any update regarding #4029?

The created incident: DINC0145671 is with the UploadSet Development team. In order to better process the issue would it be possible to provide us with an Application URL, where we can retest the similar behavior Or any location where we can test the above mentioned behavior?
when we use insertItem or addItem API on the uploadSet control and pass the incomplete item to that method, it manages to remove it from incomplete list and place it in complete list. Please let us know if this was the expected outcome.
Regards,
Vinay


How to add HTTP Status code and status text in to response ?

Hello ,
We will need to add status code in to the response.
Like
public function withArray(array $array, array $headers = array()) {
//        $code = $this->statusCode;
if ($this->statusCode == '200') {
$array = array('code' => $this->statusCode , 'success' => TRUE) + $array;
} else {
$array = array('code' => $this->statusCode, 'success' => FALSE) + $array;
}
return response()->json($array, $this->statusCode, $headers);
}
For now, we had modified library code on "ellipsesynergie/api-response/src/Laravel/Response.php".
But I feel. Its not good way.
Could you please suggest me how can i add this to all response without change library code ?
Or Is there any way to add the status code from controller for final response.
Actually, we will need final response like "http://screencast.com/t/nmWF7PYU".
Please help us on this.
Thanks,

Unfortunately, i don't see another way to do this because the status code is only available into the response class.
However, i strongly discourage you to that this way. Your API client MUST always check status code from HTTP Header. You are not suppose to get it from the body.
If you really need to do that, create you a new response class extending EllipseSynergie\ApiResponse\AbstractResponse and create you a custom service provider if you using Laravel.


This read-me file was introduced to guide the boot camp students in learning HTML, CSS, Git, and JavaScript.

Read me file added to the project.


Make mounting docker socket optional

Is this a request for help?: no

Is this a BUG REPORT or FEATURE REQUEST? (choose one): FEATURE REQUEST
Mounting in the docker socket from the host is quite a large security risk to accept, especially if deploying this chart in a critical cluster like a production cluster. I'd like to make this feature configurable, with a default of enabled for backwards compatibility (if desired) so that the risk can be mitigated for users who wish.
An example use case is running a fleet of Buildkite agents on vanilla hosts (eg using the EC2 CloudFormation template) which have access to docker for building and pushing to a registry (and can also keep warm caches). Then using a fleet of agents deployed directly into a Kubernetes cluster which use images from the previous fleet to deploy apps. The agents running inside the cluster don't need access to docker since that is done in a previous step and hence would benefit from closing off the risk of mounting the hosts docker socket.
Version of Helm and Kubernetes: n/a
Which chart: agent
What happened: mounts host docker socket by default
What you expected to happen: optionally mount host docker socket
How to reproduce it (as minimally and precisely as possible): apply the chart as-is
Anything else we need to know: I'm willing to make a PR if you would accept such a change :smile:

hey, thanks for opening the issue, sure the PR would be very welcome!

Hello again. Upon further inspection, I'm wondering whether the pods securityContext should also be removed if the enableHostDocker is false?
Eg: https://github.com/buildkite/charts/blob/master/stable/agent/templates/deployment.yaml#L39

it should be made optional, I will make a PR in a few next days

I don't mind doing another PR for this and I'll probably be able to get to it tomorrow if you like. Do you think it should be wrapped into the same flag from the original PR: enableHostDocker?
Full transparency, I don't quite understand the purpose but it seems linked to security stuff related to the host, which was half the point of that flag.
However, to support backwards compatibility it's probably best if its a separate variable.
Let me know, I'm happy to help if you want it

PR would be welcome, you could similar to these
https://github.com/jfrog/charts/blob/master/stable/kubexray/values.yaml#L61
https://github.com/jfrog/charts/blob/master/stable/kubexray/templates/deployment.yaml#L34


Viz incorrectly renders text, issue with scrolling

Discord username
No response
What type of issue is this?
Permanent – Ocurring repeatably
Is this issue blocking you from using Enso?
No, I can use Enso despite of this issue.
Is this a regression?

[X] Yes, previous version of Enso did not have this issue.

What issue are you facing?
Lines seems rendered with bold font - it changes to the normal font when scrolled to the far right and back
(also Scroll wheels do not work)
please see video in the comment
Expected behavior
text rendered in normal face font
How we can reproduce it?
No response
Screenshots or screencasts
No response
Enso Version
Build: Nightly Release #842
Browser or standalone distribution
Standalone distribution
Browser Version or standalone distribution
standalone
Operating System
Windows
Operating System Version
11 pro
Hardware you are using
xps laptop. 12gen intel + discrete gpu

https://user-images.githubusercontent.com/108271897/223106417-afb5e1bb-b244-4286-9378-0b861b3cbfe6.mp4

Issues with scrolling are being fixed in #6196

Closing as fixed other than focus issue (scroll wheel doesnt work on them).


Error while opening file: java.io.IOException: Error converting date:<PHONE_NUMBER>264Z

When I try to open a PDF file, I receive the error listed above.

Just ran into this. The PDF might have a password.

@josiah-adfhaiti Can you give example file with this ?


Fixed typo ("RPEAT")

Thanks! I'll check failure as soon as possible

You're welcome.

Only the premake script seems to crash (invalid "c++11" value)

Alright, got it.


current recv//nowait strategy allows stray icmp packets to disrupt future communication

by gating the recv on having a send triggered, you can have a situation where an ICMP response (perhaps a delayed response from an earlier send that was given up on) triggers the recv, mismatches, and then causes recv to be 'satisfied' and then you might never receive any packets at all.
might be better to have a timeout/recv-loop that waits for any message and can shoot off sends based on incoming messages during the wakeup period.  However, you might not be able to trivially use handle_continue with this because it doesn't admit handle_infos.

Yes it will work like that, however see that it do not wait for response in gen_icmp:ping/{3,4}. The general use case is that user should send pings and wait for the responses in loop. Quite similar like it work with gen_udp. So example "pinging module" could look like:
-module(pinger).

-include_lib("kernel/include/logger.hrl").

-export([
  % …
]).

-behaviour(gen_server).

start_link() ->
  gen_server:start_link(?MODULE, [], []).

init(_) ->
  {ok, Socket} = gen_icmp:open(),
  ok = gen_icmp:ping(Socket, #{family => inet, addr => {127,0,0,1}}, <<"foobar">>, [{id, 42}]),
  {ok, Socket}.

% …

handle_info({icmp, _From, {echorep, #{id := 42, data := Data}}}, State) ->
  ?LOG_INFO("Received ~p~n", [Data]),
  {noreply, State}.

The reason for such design is that with this approach user can send multiple echo requests and then accept them independently if they want. The matching then can be done by using ID and sequence no pairs.

As there seems no more discussion and no problems emerged so far I am closing it. If you (or someone else) spot any problem there then I will reopen it.


[Feature] Listening to volume changes without changing the volume

For example: listen to the volume keys to change the font size, page up and down.
Is there any way to do this?
Thanks.

Detect volume button presses is not possible on iOS as there is no official API. You may try some hacky ways like resetting volume to previous level right after volume changes, to mimic the button press event. But beware that your app might get rejected when submitting to app store.
In Android, you can use the following package to intercept volume key presses:
https://pub.dev/packages/flutter_android_volume_keydown
Notes that it overrides the onKeyDown method from activity, and you have to edit the native codes.

Thanks for your reply :)


Fix changing icon to default marker and error "Attempt to invoke virtual method<...>on a null object reference"

Does any other open PR do the same thing?
a little bit: https://github.com/react-community/react-native-maps/pull/2555/commits/0d353a6cb74649dae49ace86612f41cd9e343fcb
What issue is this PR fixing?
https://github.com/react-community/react-native-maps/issues/2556
How did you test this PR?
I`ve tested this for Android 7.0 on simulator and on real device too.

@alvelig 🐽


Fix for hard-coded api-version

PR to fix issue #467
Removing query param override in common/PolicyFragments.cs and extractor/Api.cs to make sure tha api version comes from ARM_API_VERSION parameter only.

Thanks for the PR. We will review and merge according.

@microsoft-github-policy-service agree

Good catch, and thank you for the PR.

@vandriot Thank you for your contributions. We will include it as part of our next release.

Thank you for your feedback! Happy to help improve this great tool :rocket:


✨[Feature Request]: Adding a favicon icon

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Adding a favicon icon
Use Case
Adding a favicon icon will make a website to look good
Benefits
No response
Add ScreenShots

Priority
Low
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

I like to contribute under GSSoC'24


fix automerge logic to skip workflow when no designated_github_user

Looks good to me but I can't approve since I created the PR


[Reviewed] [Boids] Use multi-line JS event

Don't show in changelog

https://forum.gdevelop.io/t/boids-movement-extansion-bug/43475?u=arthuro555
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: D8H @.>
Sent: Friday, December 2, 2022 1:04:19 PM
To: GDevelopApp/GDevelop-extensions @.>
Cc: Arthur Pacaud @.>; Review requested @.>
Subject: Re: [GDevelopApp/GDevelop-extensions] [Reviewed] [Boids] Use multi-line JS event (PR #678)
Merged #678https://github.com/GDevelopApp/GDevelop-extensions/pull/678 into main.
—
Reply to this email directly, view it on GitHubhttps://github.com/GDevelopApp/GDevelop-extensions/pull/678#event-7940144471, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AETT4LTNVOGYXMTZM5FKPFDWLHQUHANCNFSM6AAAAAASRZNOXE.
You are receiving this because your review was requested.Message ID: @.***>

Yes, I'll revert it.

JS events and other descriptions seems to work fine so leave them.

At least we're agile! ^^
I guess we'll have to wait more... or serialise extension without arrays when they are deployed.


[rewrite] Removing second-to-last item in list results in ordering issues

Description:
Im experiencing an issue when removing the second-to-last item, maybe related to node recycling.
A simple use case consisting an list of items with lifecycle methods is shown in : https://jsfiddle.net/jkp6Lto7/3/   Key attributes are set. The button removes the item at index 4, whose lifecycle animation applies a bold font style and then removes the item after 1s.
Press the button once to see the expected result, then again to the remove the next item once the animation has finished.
On the first press everything works as expected, on the second press however the item instantly jumps from second-to-last to the last position where it completes its lifecycle animation.
This happens on the current rewrite head (85ecd3545fed01da8ca1473e8aea6547d12609d6) in FF and Chromium

fixed


update to flow type checking

v0.3.1-pre

Update project to flow type
Add docs/API.md
Update jsdoc comments

Update dependencies

Outdated dependencies should be updated.
Dependencies marked as insecure by BitHound should be vetted.
Eslint incorporate flowtype rules, so test directory should be ignored.

Closed


[engine] On Win7, build of engine targeting Windows fails with SDK (10.0.19041.0)

This issue is only for the record
Win7 is not officiall supported as build environment of Flutter engine targeting Windows platform, according to https://discord.com/channels/608014603317936148/608020180177780791/785695884682854440
It works somehow

Steps to Reproduce

Prepare an Win7

System image: https://msdn.itellyou.cn/
PowerSehll 3.0+: https://docs.microsoft.com/en-us/powershell/scripting/windows-powershell/install/installing-windows-powershell?view=powershell-7
.Net 4.5+: https://www.microsoft.com/en-us/download/details.aspx?id=30653

Install VS 2019 with workload Desktop development with C++

The workload comes with a Windows 10 SDK
You could uncheck the Windows 10 SDK and install a standalone version later

Install standalone Windows 10 SDK (10.0.19041.0)

https://docs.microsoft.com/en-us/visualstudio/releases/2019/history
Install Debug Tools for Windows only if you uncheck the SDK in step 2. Or Install a full SDK

Do everything else to build the Flutter engine targting Windows

https://github.com/flutter/flutter/wiki/Compiling-the-engine#compiling-for-windows

If you install an earlier version, i.e. 10.0.18362,0, eirther withing workload or a standalone, installing 19041 will fails the build as well.
Upgrading the Debug Tools for Windows is enough to fail the build
Downgrading the SDK doesn't work. Reinstall the system is the only way I find works.
Logs
build log
[87/639] ACTION //flutter/lib/snapshot:generate_snapshot_bin(//build/toolchain/win:clang_x64)
FAILED: gen/flutter/lib/snapshot/vm_isolate_snapshot.bin gen/flutter/lib/snapshot/vm_snapshot_instructions.bin gen/flutter/lib/snapshot/isolate_snapshot.bin gen/flutter/lib/snapshot/isolate_snapshot_instructions.bin
C:/tools/depot_tools/bootstrap-3_8_0_chromium_8_bin/python/bin/python.exe ../../build/gn_run_binary.py gen_snapshot.exe --enable-experiment=non-nullable --snapshot_kind=core --enable_mirrors=false --vm_snapshot_data=C:/Users/user/flutter-engine/engine-bisect/src/out/host_debug_unopt/gen/flutter/lib/snapshot/vm_isolate_snapshot.bin --vm_snapshot_instructions=C:/Users/user/flutter-engine/engine-bise
ct/src/out/host_debug_unopt/gen/flutter/lib/snapshot/vm_snapshot_instructions.bin --isolate_snapshot_data=C:/Users/user/flutter-engine/engine-bisect/src/out/host_debug_unopt/gen/flutter/lib/snapshot/isolate_snapshot.bin --isolate_snapshot_instructions=C:/Users/user/flutter-engine/engine-bisect/src/out/host_debug_unopt/gen/flutter/lib/snapshot/isolate_snapshot_instructions.bin --enable_asserts C:/U
sers/user/flutter-engine/engine-bisect/src/out/host_debug_unopt/flutter_patched_sdk/platform_strong.dill
ninja: build stopped: subcommand failed.

engine git log
commit 8d4b7f3a68a77aaf11fc2e54229030397af16eb3
Author: Gary Qian<EMAIL_ADDRESS>Date:   Tue Dec 8 23:58:01 2020 -0700

    Add split AOT loading unit failure/error code path (#22857)

Windows7 builds are not officially supported, so we're (bug triage) going to close this, now that it has been recorded.

Windows7 builds are not officially supported, so we're (bug triage) going to close this, now that it has been recorded.

As the build crashed at the last step of linking, I guess it is related to Clang update https://crrev.com/8ad4334aaaf0bab5b56a981174ed92421ab1cc20
Something similar happened:

Chromium bug: https://bugs.chromium.org/p/chromium/issues/detail?id=1150302#c9
Firefox bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1699228


Add previews to marathon and refactor articleRenderer

Moved preview pages logic to articleRenderer.js so it can be shared between sprint and marathon mode
Refactored articleRenderer.js and play.js: mainly stuff concerning callbacks, async functions, and bindings

Temporarily removing sprint mobile UI changes. Will add them back for both sprint and marathon afterward.

Could you guys test out the changes as well since I refactored some stuff into articleRenderer so they can be reused between sprint and marathon? Tyty

Could you update this with a summary with an summary of the changes

@dqian3 Updated the description.

Can we bring back the mobile UI changes for sprint? Marathon's mobile UI is a bti more annoying to deal with and may also take more time, so we might want the sprint improvements implemented for the time being

Got it yep yep will revise this PR with mobile changes


Moderation Fee is not saved

Brief Description: when moderator: true and moderation_fee: any number is sent to POST profile, the moderation_fee is ignored.
Reproducible: always
Steps to reproduce:

in either the user page or the settings page, change your moderation fee
save
refresh the page.

Observed Behavior: Setting the fee may work the first time a user becomes a moderator. After that, new values are ignored, and the original value is returned from GET profile
Expected Behavior: the new value should be saved
https://github.com/OpenBazaar/OpenBazaar-Client/issues/989#issuecomment-192928931

I cannot reproduce this case. The behavior is as expected.
OS: Ubuntu 15.04
OB: master

Is this still only when setting to 0%?

No, it happened to me when my fee was set to 1%.

I'm not able to set to zero percent but every other percentage seems to work fine.

@cpacia +1

Still a problem @jjeffryes ?

This is no longer an issue.


ui/onroad: move duplicated code to drawIcon()

The UI is exactly the same as before
before

after

maybe put it into a base class? or do you think this is cleaner?

think this is cleaner

it looks like this breaks something about the circle background (straight line on top and left)
Old:

New:

fixed

ty!


chore(ci): clear disk space for scanner vulns

Description
Update Scanner dev vuln update pipeline to clear disk space. I also updated the release pipeline to call the job preamble
User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Not needed
Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[x] CI results are inspected

This IS the test
Automated testing

[x] modified existing tests

How I validated my change
This IS the test

Keep failing style checks due to out-of-space (ironic). See https://github.com/stackrox/stackrox/pull/12290

Keep failing style checks due to out-of-space (ironic). See #12290

dev workflow now looks to be passing consistently now however, progress!


Add more engines by default: xtb and dft3d

Checklist:

[ ] Was this PR discussed in a issue? It is recommended to first discuss a new feature into a GitHub issue before opening a PR.
[ ] Add tests to cover the fixed bug(s) or the new introduced feature(s) (if appropriate).
[ ] Update the API documentation is a new function is added or an existing one is deleted.

ping @prtos @shenoynikhil @FNTwin
Nothing to do for you, just FYI.
I will work on getting dft3d for osx-arm64 and also get the mp2d package on c-f (currently missing).

@prtos : note that the mp2d package is a correction algo for the MP2 method. The MP2 algo is already in psi4 (if I understand it correctly).


Credentials helper

When booting up, if there are no credentials, offer to copy paste them in.

Done in GUI


GPU benchmarking is broken

Trying to run any GPU benchmark on head of main on GCP or Azure yields an error like this:
azureuser@mla-gpu-test:~$ miniconda3/bin/conda run -n mla benchit mlagility/models/selftest/linear.py --device nvidia

Models discovered during profiling:

linear.py:
	model (executed 1x)
		Model Type:	Pytorch (torch.nn.Module)
		Class:		LinearTestModel (<class 'linear.LinearTestModel'>)
		Location:	/home/azureuser/mlagility/models/selftest/linear.py, line 21
		Parameters:	110 (<0.1 MB)
		Hash:		d5b1df11
		Status:		Unknown benchit error: 'Total Latency'
		Traceback (most recent call last):
		  File "/home/azureuser/mlagility/src/mlagility/analysis/analysis.py", line 133, in call_benchit
		    perf = benchmark_model(
		  File "/home/azureuser/mlagility/src/mlagility/api/model_api.py", line 145, in benchmark_model
		    perf = gpu_model.benchmark(backend=backend)
		  File "/home/azureuser/mlagility/src/mlagility/api/trtmodel.py", line 21, in benchmark
		    benchmark_results = self._execute(repetitions=repetitions, backend=backend)
		  File "/home/azureuser/mlagility/src/mlagility/api/trtmodel.py", line 84, in _execute
		    mean_latency=self.mean_latency,
		  File "/home/azureuser/mlagility/src/mlagility/api/trtmodel.py", line 43, in mean_latency
		    return float(self._get_stat("Total Latency")["mean "].split(" ")[1])
		  File "/home/azureuser/mlagility/src/mlagility/api/trtmodel.py", line 34, in _get_stat
		    return performance[stat]
		KeyError: 'Total Latency'

GPU benchmarking is known to work correctly on commit e250ac7502c43d24045688b3393874ea9ee4c364

The design to use containers for all benchmarking was due to its promise of including all required dependencies and will not force the user to worry about versions and stuff.
For TensorRT to function you need the compatible versions of CUDA, CUDNN and the driver as stated here in this matrix
The official Nvidia TensortRT container we use comes packaged with the right version of CUDA and CUDNN, great!
But since the driver is a kernel mode component, that cannot come with the container, rather should have been installed on the host system.
So far all of the systems we had tested this feature on happened to have the correct drivers.
Except for the T4 system Jermey used and the T4 system I found on GCP. Once I updated the driver version everything worked as expected.
Ideally, the TRT container should report this error instead of just crashing. The fix on our end should be to read the driver version and report a proper error to update the driver. I will add this to the issue.
Follow the steps here to update drivers


Upgrade to 20200125

and drop outdated documentation link; docs are not built anymore at the moment.

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender.

@conda-forge-admin, please rerender.

@conda-forge-admin, please rerender.

@conda-forge-admin, please rerender

Fails with WARNING conda.gateways.disk.delete:unlink_or_rename_to_trash(182): Could not remove or rename C:\Users\VSSADM~1\AppData\Local\Temp\tmp0vlmc3o1\m4ri-20200125-had7236b_0.conda.  Please remove this file manually (you may need to reboot to free file handles) and then
(base) %SRC_DIR%>conda activate "%PREFIX%"
C:\Users\VSSADM~1\AppData\Local\Temp\conda-1863\conda.tmp - Access is denied.

@isuruf, any idea what could be the problem here? I tried to use the autotools scripts for windows. In principle it works but there's some weird error at the end (see previous comment.)

@conda-forge-admin, please rerender

@isuruf, since I have no idea how to fix the Windows build, I am disabling it temporarily so that the SageMath 10.4 release can go ahead.


Implement the <= and >= operators via a trait that doesn't require (T, <=) to be a partial order.

 Issue by mahkoh
Wednesday Sep 10, 2014 at 18:16 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/17149
This issue was labelled with:  in the Rust repository

Right now the way to get the <= and >= operators is to implement PartialOrd. Unfortunately there seems to be some confusion whether T: PartialOrd implies that (T, <=) is a partial order [1]. The name implies it but the documentation [2] doesn't mention it. This is especially confusing since the documentation for Ord [3] mentions that (T, <=) has to be a total order.
Either way, PartialOrd: PartialEq and thus implementing <= requires you to implement ==. This doesn't make much sense for objects that aren't part of a partial order. For example:

T is a set of operators and <= orders them by operator precedence.
More generally: (T, ≤) where ≤ is defined via a ≤ b ⇔ f(a) ≼ f(b) where f is some non-injective function into an ordered space.

Therefore implementing <= should not require implementing ==. Instead:
/// Trait for types that form a [preorder](http://en.wikipedia.org/wiki/Preorder).
///
/// A preorder `(T, <=)` has two requirements that your implementation must follow:
///
/// - `a <= a`
/// - `a <= b` and `b <= c` implies `a <= c`
///
/// Note that, if you also implement the `==` operator, then the first requirement also
/// implies
///
/// - `a == b` implies `a <= b` and `b <= a`.
///
/// If you implement `<=` and `==`, then people might get confused by a lack of
/// antisymmetry. Consider one of the following:
///
/// - If `(T, <=)` is a partial order, implement `PartialOrd` instead.
/// - If `(T, <=)` is not a partial order, don't implement the `==` operator.
/// - Document clearly that `(T, <=)` is not antisymmetric.
trait PreOrd {
    /// This method tests `self <= other`.
    fn le(&self, other: &Self) -> bool;

    /// This method tests greater or equal to (>=).
    fn ge(&self, other: &Self) -> bool {
        other.le(self)
    }
}

[1] http://en.wikipedia.org/wiki/Partially_ordered_set
[2] http://doc.rust-lang.org/std/cmp/trait.PartialOrd.html
[3] http://doc.rust-lang.org/std/cmp/trait.Ord.html

I believe this is practically speaking non-actionable due to the breakage it would cause today.


Left-shift can cause width mismatch in generated SystemVerilog

Describe the bug
The rules around left-shifting in SystemVerilog allow for the width of the result to be different than the width of the original value being shifted in some circumstances.  This can hit lint violations or perhaps strange bugs and inconsistencies in generated verilog.
To Reproduce
Perform left-shift around other signals of various widths.  There may be many combinations to explore.
Expected behavior
No response
Actual behavior
No response
Additional: Dart SDK info
No response
Additional: pubspec.yaml
No response
Additional: Context
No response

A similar thing can happen with adding (carry bit)


Adding "All Notes" in the module

Hello.
Thank you for developing a wonderful plugin.
It is really helpful for me.
Could you please consider adding "All Notes" in the module?
Best regards,
Ryuki

@ryukiyokoo1164 Cool, that's something I can consider but not something I'd use all the time. In the meantime, you're welcome to open a PR if you're up for the challenge. 😉 Should be straight-forward to implement.

@marconi Thank you for the reply! OK, I'm going to challenge PullRequest!


Added possibility to ignore whitespaces during deserialization of enum values

For example now optionally we can deserialize such enums.
{
  "Enum": "red,dark goldenrod, white"
}

We can often see such enums when deserializing objects from external (non .net projects). And every time we should think how to write custom converter that has all features of base one.

I'm not sure I agree with this. What about people who want to ignore other kinds of separating characters?
Why not place an EnumMember on the properties and specify the name with a space it in?

Seems EnumMember not supported in .net core version.
I haven't seen cases when we need something different than a space and even can't imagine. But space is common from what i have seen. Space symbol I met second time during two weeks working with different APIs ( I can send you the links)
But if you want I can reimplemented passing char or char array there or even array of strings. But again, I don't think it's common.

My fault. EnumMember works fine under .net core. I thought StringEnumConverter is enabled by default (because values without space converts just fine).
So if you don't like this PR, but i think it's still useful (a lot of enums will be attribute free), you can close it.


Skip CTC-segmentation test if module not available

This simply skips the tests if the extra dependency is not installed.

@Gastron,
cool, why we don't just use importorskip() function ?
other thing, I see no use of the CTC_SEG_AVAIL, is it used somewhere ?

Thanks @aheba, looks cleaner now.

LGTM. Thanks.


Hi, is there any plan to wrap it as a react component and support typescript?

Really love the project, but could not figure out how to use it in React. How can I initialize a html midi player and load midi with Javascript?
Thanks

I don't know much about react, so not sure how much I can help here, but you can check out for example this page about the interaction between React and Web Components.
Not sure if you have another question, but you mention TypeScript in the title. html-midi-player is written in TypeScript and provides type declarations.


Suggested string updates

Before merging this PR, please read my note in https://github.com/cla-assistant/cla-assistant/pull/783#discussion_r724658479

Thanks a lot. This is very helpful.
I incorporated it in https://github.com/cla-assistant/cla-assistant/pull/783/commits/10ba175df23360e64ee1edc35f54dd1b92f55e30


Finalize create and update users API

Description
Finalizes the create and update users API methods
Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

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

Comparison is base (c76b9bd) 86.64% compared to head (313dde1) 86.58%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #222      +/-   ##
==========================================
- Coverage   86.64%   86.58%   -0.06%
==========================================
  Files          29       29
  Lines        1198     1193       -5
==========================================
- Hits         1038     1033       -5
  Misses        160      160

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


1640 recruitment cycle not null

Context
Follow up to https://github.com/DFE-Digital/manage-courses-backend/pull/506
Changes proposed in this pull request
Set the recruitment cycle to not null on course
Guidance to review
n/a
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 [M] Backend - Add recruitment cycles to courses


ISPN-11546 Upgrade to 2.2.6.RELEASE

https://issues.redhat.com/browse/ISPN-11546

I guess we need to wait for GA before merging changes right @karesti ?

@pruivo I don't know

do we ?

GA is planned for April 1st. I'll merge it tomorrow.

integrated! thanks @karesti !


"Enable country for comments" and "Show country flags" don't work anymore.

Noticed that as well. Seems to be already fixed in the development version.

Nope, still doesn't work as of Developer Version 492.

Working for me.

YouTube recently made a change where comments are directly embedded into the page instead of using the iframe element. This is good news as the performance will increase as the iframe counts as an extra page and will cause two instances of YouTube Center to run on one page.

#1523 also seems to be caused by this. For me, comments on the watch page are in an iframe, while the comments on the All Comments page are directly on the page.

The comment flags doesn't work for me too. Neither under the video nor on the all comments page. I'm using the latest dev build on FF 37 beta.

Seem to work fine on latest Chromium and YTC dev version at the time of writing.

Not working in Firefox 36.0.4. Ajax is being used to inject the comments onto the page. The page is sending a post request to https://www.youtube.com/watch_fragments_ajax and receiving them in json format. I don't see the country info anywhere. This feature was very useful... bummer.

ff 37.0.1 w/build 497 as addon
same thing(

On the watch and all comments pages, the iframe and thus country/language/flag info came back...

yep

Not for me, comments are in the same page for watch page and all comments view.

@YePpHa commented on 2015. márc. 11. 21:53 CET:

This is good news as the performance will increase as the iframe counts as an extra page and will cause two instances of YouTube Center to run on one page.
Edit: Change might not be for everyone.

It would be good news if they didn't still use the iframing thing for posting a new comment. It's incomprehensible... You can reply to another comment normally, but when you click in the texboxt for a new comment the iframe loads.

At least for me, only the comment text box is in the iframe, not the comments themself, so that wouldn't be a problem.

@Inyokus commented on 12. Apr. 2015 13:17 MESZ:

At least for me, only the comment text box is in the iframe, not the comments themself, so that wouldn't be a problem.

Same for me.

Interestingly enough if you open the text box iframe url and replace https://apis.google.com/u/0//widget/render/ytshare with https://apis.google.com/u/0//widget/render/comments it will load the comments with the flags.

The embedding of comments directly in the page is useful, but the only problem is that it's breaking with the keyboard commands. Every time I want to enter a space, it does it on the player, thinking I'm pausing the video, which means I have to reply to comments either via http://youtube.com/comments or the "All Comments" link.

The developer and many of us here are well aware of the issue. Unfortunately, it's been a hard bug to smash due to YouTube using AJAX to inject the comments directly to the page. I'm not 100% certain of the details, but from what I have gathered, the script currently has no way of differentiating between the comment input box and the whitespace on the page. For this reason, it is suggested to disable Enable shortcuts on page if you do not need that specific functionality.

It seems to be fiexed for me.

how do we open the text box iframe url  as lotelegome29 commented please


Ike compatibility with windows

Hi,
congratulation for the applicaton and the approach. I tried to install Ike on windows 10 but I run into a problem, namely:

<EMAIL_ADDRESS>watch C:\dev\workspace\neos\tmp\ike\webapp
gulp watch --color

ike 2016-08-12 09:40:21,614 INFO  o.a.i.EmbeddingBasedPhrasesSearcher - Loading phrase vectors ...
module.js:442
throw err;
^
Error: Cannot find module 'C:\dev\workspace\neos\tmp\ike\webapp\node_modules\gulp\node_modules\v8flags\cache\<IP_ADDRESS>.flags.json'
at Function.Module._resolveFilename (module.js:440:15)
at Function.Module._load (module.js:388:25)
at Module.require (module.js:468:17)
at require (internal/module.js:20:19)
at Object. (C:\dev\workspace\neos\tmp\ike\webapp\node_modules\gulp\bin\gulp.js:25:22)
at Module._compile (module.js:541:32)
at Object.Module._extensions..js (module.js:550:10)
at Module.load (module.js:458:32)
at tryModuleLoad (module.js:417:12)
at Function.Module._load (module.js:409:3)
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\PROGRA~1\nodejs\node.exe" "C:\PROGRA~1\nodejs\node_modules\npm\bin\npm-cli.js" "run" "watch"
npm ERR! node v6.3.1
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>watch: gulp watch --color
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>watch script 'gulp watch --color'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ike package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp watch --color
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ike
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ike
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\dev\workspace\neos\tmp\ike\webapp\npm-debug.log
ike [DEBUG] [08/12/2016 09:40:22.511] [ike-tool-akka.actor.default-dispatcher-4] [akka://ike-tool/user/IO-HTTP/listener-0] Binding to /<IP_ADDRESS>:8080
ike [DEBUG] [08/12/2016 09:40:22.582] [ike-tool-akka.actor.default-dispatcher-6] [akka://ike-tool/system/IO-TCP/selectors/$a/0] Successfully bound to /0:0:0:0:0:0:0:0:8080
ike [INFO] [08/12/2016 09:40:22.587] [ike-tool-akka.actor.default-dispatcher-4] [akka://ike-tool/user/IO-HTTP/listener-0] Bound to /<IP_ADDRESS>:8080
ike 2016-08-12 09:40:27,976 INFO  o.a.i.EmbeddingBasedPhrasesSearcher - Loading phrase vectors complete
ike 2016-08-12 09:40:27,977 INFO  o.a.i.EmbeddingBasedPhrasesSearcher - Loading phrase vectors ...
ike 2016-08-12 09:40:53,277 INFO  o.a.i.EmbeddingBasedPhrasesSearcher - Loading phrase vectors complete
ike [DEBUG] [08/12/2016 09:41:35.810] [ike-tool-akka.actor.default-dispatcher-3] [akka://ike-tool/system/IO-TCP/selectors/$a/0] New connection accepted
ike [DEBUG] [08/12/2016 09:41:35.814] [ike-tool-akka.actor.default-dispatcher-3] [akka://ike-tool/system/IO-TCP/selectors/$a/0] New connection accepted
ike [DEBUG] [08/12/2016 09:41:35.892] [ike-tool-akka.actor.default-dispatcher-5] [akka://ike-tool/user/IO-HTTP/listener-0/1] Dispatching GET request to http://localhost:8080/ to handler Actor[akka://ike-tool/system/IO-TCP/selectors/$a/1#935838599]
ike [DEBUG] [08/12/2016 09:41:38.570] [ike-tool-akka.actor.default-dispatcher-7] [akka://ike-tool/system/IO-TCP/selectors/$a/0] New connection accepted
ike [DEBUG] [08/12/2016 09:41:38.579] [ike-tool-akka.actor.default-dispatcher-4] [akka://ike-tool/user/IO-HTTP/listener-0/2] Dispatching GET request to http://localhost:8080/ to handler Actor[akka://ike-tool/system/IO-TCP/selectors/$a/3#2050110872]
ike [DEBUG] [08/12/2016 09:41:38.603] [ike-tool-akka.actor.default-dispatcher-3] [akka://ike-tool/user/IO-HTTP/listener-0/0] Connection was PeerClosed, awaiting TcpConnection termination...
ike [DEBUG] [08/12/2016 09:41:38.603] [ike-tool-akka.actor.default-dispatcher-9] [akka://ike-tool/user/IO-HTTP/listener-0/1] Connection was PeerClosed, awaiting TcpConnection termination...
ike [DEBUG] [08/12/2016 09:41:38.605] [ike-tool-akka.actor.default-dispatcher-9] [akka://ike-tool/user/IO-HTTP/listener-0/1] TcpConnection terminated, stopping
ike [DEBUG] [08/12/2016 09:41:38.605] [ike-tool-akka.actor.default-dispatcher-5] [akka://ike-tool/user/IO-HTTP/listener-0/0] TcpConnection terminated, stopping
The nlp-debug.log contains the following information:
0 info it worked if it ends with ok
1 verbose cli [ 'C:\PROGRA~1\nodejs\node.exe',
1 verbose cli   'C:\PROGRA~1\nodejs\node_modules\npm\bin\npm-cli.js',
1 verbose cli   'run',
1 verbose cli   'watch' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prewatch', 'watch', 'postwatch' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for prewatch, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin;C:\dev\workspace\neos\tmp\ike\webapp\node_modules.bin;C:\PROGRA~1\nodejs;C:\ProgramData\Oracle\Java\javapath;C:\Program Files (x86)\Intel\iCLS Client;C:\Program Files\Intel\iCLS Client;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0;c:\Program Files (x86)\HP\HP Performance Advisor;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\OpenVPN\bin;C:\Program Files\TortoiseSVN\bin;C:\Program Files (x86)\PuTTY;C:\Program Files\Intel\WiFi\bin;C:\Program Files\Common Files\Intel\WirelessCommon;C:\Program Files\wget-1.11.4-x64;C:\Program Files (x86)\Skype\Phone;C:\dev\Softs\unxutils\usr\local\wbin;C:\dev\workspace\neos\protoc-2.6.1-win32;C:\Program Files\CMake\bin;C:\Program Files (x86)\sbt\bin;C:\Program Files\Git\cmd;C:\Program Files\nodejs;C:\Users\luca.dini\AppData\Local\Programs\Python\Python36-32\Scripts;C:\Users\luca.dini\AppData\Local\Programs\Python\Python36-32;C:\Users\luca.dini\AppData\Roaming\npm
10 verbose lifecycle<EMAIL_ADDRESS>CWD: C:\dev\workspace\neos\tmp\ike\webapp
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '/d /s /c', 'gulp watch --color' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec watch script
14 verbose stack Error<EMAIL_ADDRESS>watch: gulp watch --color
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter. (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:242:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:191:7)
14 verbose stack     at ChildProcess. (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:40:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:191:7)
14 verbose stack     at maybeClose (internal/child_process.js:852:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd C:\dev\workspace\neos\tmp\ike\webapp
17 error Windows_NT 10.0.10586
18 error argv "C:\PROGRA~1\nodejs\node.exe" "C:\PROGRA~1\nodejs\node_modules\npm\bin\npm-cli.js" "run" "watch"
19 error node v6.3.1
20 error npm  v3.10.3
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>watch: gulp watch --color
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>watch script 'gulp watch --color'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the ike package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     gulp watch --color
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs ike
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls ike
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]
My question is: is Ike supposed to work on windows or its compatibility is limited to linux systems?
Many thanks,
Luca
(ps of course I installed git and node.js)

Hi Luca, We haven't looked into making IKE compatible with Windows. It has been intended to run only on Linux systems and that is what we have tested so far.

Thanks for the answer, I'll let you know the results of the experiments. Just in case, do you have a prebuilt linux Virtualbox image.
Again, thanks for the answer,
Luca

In case somebody is interested. Ike can run under windows 10, with the following modifications:
The first error is caused by the sbt-plugin trying to execute node.js, which invalidly look for "which npm" whereas on windows is "which npm.cmd". I guess the only solution is to modify the source code and recompile, as we did (probably some slightly less OS dependent solution is possible).
Second we had to change slashes in ike\webapp\gulpfile.js, namely replacing slashes and adding some assets directory:
{
js: 'app\js\DictApp.js',
less: 'app\css\.less',
allLess: 'app\css\**\.less',
build: '..\public',
jshint: 'app\\*.js',
assets: 'app\assets\\*'
}
With this, ike seems to work plainly on a windows system.
Hope it might be useful to somebody,
Cheers,
Luca

Great, thanks Luca! I will update the documentation with this info.


Increase Coverage

Increase the coverage by one line
Contributes to / Solves #37
Type of changes

[ ] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[x] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[x] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Creates a new PygmentsSyntaxTheme to test a branch of the constructor when a custom theme is provided instead of a default one.

There's another PR for Syntax tests with greater coverage. Thanks anyway.


Using PZEM004 to capture metervalues

Hi Matth-x, I am planning to use the PZEM004 sensor and arduino library to capture the metervalues once the chargers is started. What are the ArduinoOCPP functions I should invoke in order to register the appropriate metervalues to the central system?

Hey @Pulse-akhil
can you please help me with the same
https://github.com/matth-x/ArduinoOcpp/issues/42

Please refer to ArduinoOcpp.h for the documentation of the API functions. This is the relevant excerpt:
https://github.com/matth-x/ArduinoOcpp/blob/e742bc760950731309c9c9ce1e4391a69ac2da82/src/ArduinoOcpp.h#L51-L53
It will be sufficient to set this callback function. Whenever the library needs the power or energy reading, it executes this callback.


[1.1.x] support for storing tagged series in hashed filenames | codacy fix | move any existing non-hashed files to hashed names if needed | codacy fix | Adding dummy.txt to storage dirs

Backports the following commits to 1.1.x:

support for storing tagged series in hashed filenames (#743)
codacy fix (#743)
move any existing non-hashed files to hashed names if needed (#743)
codacy fix (#743)
Adding dummy.txt to storage dirs (#753)

Codecov Report

Merging #760 into 1.1.x will increase coverage by 0.52%.
The diff coverage is 90%.

@@            Coverage Diff             @@
##            1.1.x     #760      +/-   ##
==========================================
+ Coverage   48.64%   49.16%   +0.52%
==========================================
  Files          36       36
  Lines        3285     3305      +20
  Branches      466      471       +5
==========================================
+ Hits         1598     1625      +27
+ Misses       1588     1580       -8
- Partials       99      100       +1

Impacted Files
Coverage Δ

lib/carbon/conf.py
40.72% <ø> (ø)
:arrow_up:

lib/carbon/util.py
43.87% <100%> (ø)
:arrow_up:

lib/carbon/database.py
56.29% <89.28%> (+12.01%)
:arrow_up:

Continue to review full report at Codecov.

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


Documentation how to add field to form in source editor

https://youtrack.haulmont.com/issue/STUDIO-4910
Describe how to add field to form in source editor.

Move cursor inside to form
Press Alt+Insert, select Add field OR type field and press TAB.
Choose property, choose field tag.

@knstvk In Studio User Guide only?


Roll Engine from ca2f95c71c20 to 59c9daf7bfc1 (1 revision)

https://github.com/flutter/engine/compare/ca2f95c71c20...59c9daf7bfc1
2022-03-26<EMAIL_ADDRESS>Add symlink support to zip script (flutter/engine#32257)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Get-DbaDatabase bug when piped to Out-DbaDataTable

Is this:
Just place "x" to indicate selection (e.g. "[x]")

[ ] feature
[x ] bug:

System Details

Operating system name and version:
Windows 10
Major  Minor  Build  Revision
10     0      14393  0

Output from $PSVersionTable:
Name                           Value
PSVersion                      5.1.14393.1358
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.1358
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Output of dbatools version:
OutputFile.txt

ModuleType Version    Name                                ExportedCommands
Script     0.8.957    dbatools                            {Start-SqlMigration, Copy-SqlDatabase, Copy-SqlLogin, Copy-SqlServerAgent...}

SQL Server version for source/target
SQL 2014 SP2

Steps to Reproduce
Get-DbaDatabase -SqlServer LabSQL14-1 -WarningAction:silentlycontinue | Out-DbaDataTable -WarningAction:silentlycontinue | Write-DbaDataTable -SqlServer LabSQL14-1 -Table tempdb.dbo.DbaDatabase -Truncate -AutoCreateTable -Confirm:$false -WarningAction:silentlycontinue

Received following error

Received this error, once for each db:
The following exception occurred while trying to enumerate the collection: "unknown property IsSystemObject".
At line:132 char:24

                if($property.value.length -gt 0)

                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], ExtendedTypeSystemException
FullyQualifiedErrorId : ExceptionInGetEnumerator

Received one instance of this error:
Exception calling "WriteToServer" with "1" argument(s): "The given value of type Guid from the data source cannot be converted to type varchar of the specified target column."
At line:356 char:4

        $bulkCopy.WriteToServer($InputObject)

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : InvalidOperationException

Action Results
Empty table, tempdb.dbo.DbaDatabase
Expected Results
new table, tempdb.dbo.DbaDatabase to be populated with output from Get-DbaDatabase
Attached screen shots/console output
OutputFile.txt
Import - Attach dbatools logs
dbatools_9208_message_0 .txt

Thank you so much for the report! I ran into something similar today and have patched this issue.
Thank you also for showing me that you have to go through the torture of ignoring warnings. I'll verbose it in future release it.

fixed this, our apologies!

thank you for the report 🌮


Fix import and issues with Python 3.6 compatibility

The change to the generic train node unfortunately breaks python2.7 compatibility (see failed Travis build).
Furthermore, I suggest that when we start rewriting imports for python3, we should do it everywhere, in a python2.7 compatible way, i.e., change local imports of the form
from foo import bar

into
from __future__ import absolute_import
from .foo import bar

I think the best way forward is to create a python3 feature branch and keep working there. Once all tests pass for python2.7 and python3, we can merge into release-v1.0.


Error when covalent docking: Provided ligand file format not supported with covalent docking.

Issue summary
Hello,
I attempted covalent docking using Gnina in a Google Colab CPU environment, but encountered the following error:
"Usage error: Provided ligand file format not supported with covalent docking."
I would appreciate any suggestions to resolve this issue.
Steps to reproduce
I can provide the receptor and ligand files for reference.
8x6r_A_1.zip
BRD-K00010385.zip
Below is the command I used:
gnina -r /content/drive/MyDrive/SMILES/8x6r_A_1.pdbqt -l BRD-K00010385.pdbqt --center_x 66.082 --center_y -6.131 --center_z 95.682 --size_x 20 --size_y 20 --size_z 20 --covalent_rec_atom A:12:SG --covalent_lig_atom_pattern CCC --cnn crossdock_default2018 -o BRD-K00010385.sdf --seed 0

Your system configuration
Operating system: Google colab
Compiler: CPU
CUDA version (if applicable):
CUDNN version (if applicable):
Python version: Python [3.10.13]

Hi [RMeli],
Thank you so much for your detailed explanation and helpful suggestion! I sincerely appreciate your guidance, as it resolved my issue perfectly. Apologies for the delayed response, but I wanted to express my gratitude for your time and assistance.

I'm grad it helped!


Fire skipped events at debug level

resolves #8774
Problem
If fail_fast, dbt prints hundreds of lines "Skipping due to fail_fast", hiding the actual cause of the error deep in bash history (sometimes too deep to read)
Solution
This moves the "Skipping due to fail_fast" to log level DEBUG. An alternative would be to a) drop it altogether or b) aggregate the results together to a "Skipping  tasks due to fail_fast". This approach requires the least modification to the task results framework.
Checklist

[X] I have read the contributing guide and understand what's expected of me
[X] I have run this code in development and it appears to resolve the stated issue
[X] Tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[X] This PR includes type annotations for new and modified functions

Can someone please approve workflows and review this?

I'm going to close this in favor of https://github.com/dbt-labs/dbt-core/pull/10244


steam-rom-manager: 2.5.17 -> 2.5.22

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/SteamGridDB/steam-rom-manager/releases.
meta.description for steam-rom-manager is: App for managing ROMs in Steam
meta.homepage for steam-rom-manager is: https://github.com/SteamGridDB/steam-rom-manager
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.5.20 with grep in /nix/store/n9pm2apidadyx8snps050k2ann3457pb-steam-rom-manager

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

steam-rom-manager

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/n9pm2apidadyx8snps050k2ann3457pb-steam-rom-manager \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A steam-rom-manager https://github.com/r-ryantm/nixpkgs/archive/0043f5432cd3adf9d85379511de7c336fa7825f2.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0043f5432cd3adf9d85379511de7c336fa7825f2#steam-rom-manager

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/n9pm2apidadyx8snps050k2ann3457pb-steam-rom-manager
ls -la /nix/store/n9pm2apidadyx8snps050k2ann3457pb-steam-rom-manager/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

steam-rom-manager

Maintainer pings
cc @SquarePear for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/SteamGridDB/steam-rom-manager/releases.
meta.description for steam-rom-manager is: App for managing ROMs in Steam
meta.homepage for steam-rom-manager is: https://github.com/SteamGridDB/steam-rom-manager
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.5.21 with grep in /nix/store/1a2jmvn42j100xxxra25h0h4sp8gc0j8-steam-rom-manager

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

steam-rom-manager

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1a2jmvn42j100xxxra25h0h4sp8gc0j8-steam-rom-manager \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A steam-rom-manager https://github.com/r-ryantm/nixpkgs/archive/63a06ea928069514d91bf725a6fcdb56a1f3d4bc.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/63a06ea928069514d91bf725a6fcdb56a1f3d4bc#steam-rom-manager

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1a2jmvn42j100xxxra25h0h4sp8gc0j8-steam-rom-manager
ls -la /nix/store/1a2jmvn42j100xxxra25h0h4sp8gc0j8-steam-rom-manager/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

steam-rom-manager

Maintainer pings
cc @SquarePear for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/SteamGridDB/steam-rom-manager/releases.
meta.description for steam-rom-manager is: App for managing ROMs in Steam
meta.homepage for steam-rom-manager is: https://github.com/SteamGridDB/steam-rom-manager
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.5.22 with grep in /nix/store/vnyr96ra6ay9dy7fn6dgvq01p9g1w2qz-steam-rom-manager

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

steam-rom-manager

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vnyr96ra6ay9dy7fn6dgvq01p9g1w2qz-steam-rom-manager \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A steam-rom-manager https://github.com/r-ryantm/nixpkgs/archive/981a40b5478fac4500b7d4a2826790957ca29e5b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/981a40b5478fac4500b7d4a2826790957ca29e5b#steam-rom-manager

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vnyr96ra6ay9dy7fn6dgvq01p9g1w2qz-steam-rom-manager
ls -la /nix/store/vnyr96ra6ay9dy7fn6dgvq01p9g1w2qz-steam-rom-manager/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

steam-rom-manager

Maintainer pings
cc @SquarePear for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.


Image not updating?

The BLAST hits don't display until the mouse is moved over the image. This is particularly true if a large number of features are displayed. For example:
https://peanutbase-stage.usda.iastate.edu/blast/report/PjVoKmf

Inexplicably, this appears to be related to, or at least affected by the glyph width. For the image above, widths < 5 draw fine, without having to move the mouse over the image.

Fixed some time ago.


Support for Jenkins < 2.332

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 6.25.1
Ruby: /
Distribution: RHEL 8
Module version: Master

How to reproduce (e.g Puppet code you use)
class { 'jenkins':
    version           => $version,
    repo              => false,
    lts               => $lts,
    proxy_host        => $profile::globals::proxy_host,
    proxy_port        => $profile::globals::proxy_port,
    no_proxy_list     => $_noproxy,
    localstatedir     => $profile::jenkins::home,
    manage_datadirs   => false,
    manage_user       => false,
    manage_group      => false,
    cli_username      => 'user1',
    cli_password_file => '/var/passfile',
    slaveagentport    => 99999,
    config_hash       => {
      'JENKINS_HOME' => { 'value' => $profile::jenkins::home },
    },
  }

What are you seeing
Jenkins service not starting
What behaviour did you expect instead
That the service would start
Output log
Error: Systemd start for jenkins failed!
journalctl log for jenkins:
-- Logs begin at Mon 2021-05-03 15:28:28 CEST, end at Fri 2022-05-13 18:41:36 CEST. --
-- No entries --

Error: /Stage[main]/Jenkins::Service/Service[jenkins]/ensure: change from 'stopped' to 'running' failed: Systemd start for jenkins failed!
journalctl log for jenkins:
-- Logs begin at Mon 2021-05-03 15:28:28 CEST, end at Fri 2022-05-13 18:41:36 CEST. --
-- No entries --
 (corrective)

Any additional information you'd like to impart

For my understanding: why do you want support for < 2.332? That's older than what's in the current LTS repositories, which makes it unsupported from an upstream perspective.
It's my intention to release a 3.0.0 soon which simply takes 2.332 as the minimum version. Getting the module even in a working state with the current version has been hard enough so I'm leaning to closing this as WONTFIX. If you need < 2.332 support, pin your module to an older version.


_ is not defined

i have followed all the instructions, but when i try to invoke the table, suddenly it gives me this error _ is not defined. can somebody help me please?

Hi!
Add this:
``

http://stackoverflow.com/questions/13556010/referenceerror-is-not-defined


Update logger.py to reset color escape sequence in between values

Addresses issue #488 by resetting colour code after each value in logger::_step.
Also resets the use of red in target_space when warning about duplicate points being added.

Hi @till-m,
Thanks for the review. Replacing Fore.BLACK with Fore.RESET is a good idea as a "safe" default.
I do think having a way to disable entirely could be useful for saving the screen logger output to file as well as to allow the user to avoid issues should they be running in a terminal with a magenta background (if there's anyone that does this!).
Currently there's no user exposure of the logger (created in _prime_subscriptions) . If this creation was moved to near the end of BayesianOptimization::__init__ and stored as self.logger  it should provide some exposure. Then by providing a method on ScreenLogger of the form:
def disable_colors(self):
     self.NEW_MAX_COLOR = ''
     self.DEFAULT_COLOR=''
     self.RESET_COLOR=''

and changing the uses of Fore to NEW_MAX_COLOR etc. (which would be initialised in the __init__ to the current values from Fore) then there's a simple method to strip out all escape codes and a way for users to customise these if they really want.
This does feel like it might be exposing / adding a bit much -- I'm not really a python developer so there may be neater ways to do this, but I thought exposing the logger might be nicer than doing it all through init arguments which get passed down the call chain.

Hi @d7919,
I'm not really an expert regarding the Observer pattern that's in use here, so I can't really see a nice & clean way of handling things. It can be handled however:
from bayes_opt import BayesianOptimization
from bayes_opt.logger import ScreenLogger
from bayes_opt.event import Events

def black_box_function(x, y):
    return -x ** 2 - (y - 1) ** 2 + 1

pbounds = {'x': (2, 4), 'y': (-3, 3)}

optimizer = BayesianOptimization(
    f=black_box_function,
    pbounds=pbounds,
    verbose=2, # verbose = 1 prints only when a maximum is observed, verbose = 0 is silent
    random_state=1,
)

logger = ScreenLogger()
logger._default_cell_size = 15 # changing something to show that it works

# the following needs to happen before any call to `.maximize`
# otherwise there will be two loggers associated with the optimizer.
for e in [Events.OPTIMIZATION_START, Events.OPTIMIZATION_STEP, Events.OPTIMIZATION_END]:
    optimizer.subscribe(e, logger)

optimizer.maximize(
    init_points=2,
    n_iter=3,
)

This does feel like it might be exposing / adding a bit much

IMO as long as things are handled by the ScreenLogger and abstracted from the BayesianOptimization class it would be fine.
Let me know how you wish to handle things. I'm fine with merging this PR as-is, but I would also be open to adding the options discussed above.

Hi @till-m,
Maybe best to leave this PR as-is and leave any further modification of colourising output to the future.
I think the appearance of colours outside of the logger would suggest that to handle customisation of this most consistently probably requires a slight redesign (perhaps a simple container class through which colour escape codes can be obtained). There's also the fact that BayesianOptimization takes and stores verbose  but only uses this to pass through to _get_default_logger which may suggest a slight refactor might help separate concerns more completely?
Ultimately this isn't something that really impacts on the utility of the code so no need to rush further changes in.

I agree, I think logging might be due for an overhaul at some point, though it is not a priority.
I would still argue that replacing Fore.BLACK with Fore.RESET would be a good change for now. Are you interested in handling that with this PR? Otherwise I can make a seperate PR.

Looks good to me!

thanks for your contribution @d7919!


Check for method_exists getClosureThis

Looks like that function was added in 5.4: http://php.net/manual/en/reflectionfunctionabstract.getclosurethis.php
My PHP 5.3 environment (we'll upgrade soon, but we haven't gotten to it yet) chokes on this call.  This check fixes the problem for me.  Packagist says you support back to 5.1.  If you only claimed support for 5.4 and up, I wouldn't request the change.  Thanks for the excellent library!

thank you! :)


Bun server & package manager support

Discussed in https://github.com/stacksjs/stacks/discussions/211

Originally posted by chrisbbreuer October 16, 2022
Hi everyone,
TLDR: Bun is an "incredibly fast JavaScript runtime, bundler, transpiler and package manager – all in one."
Upon viewing @Jarred-Sumner's Vite talk, it sure looks like this could be a great direction to be explored.
If implemented & deemed stable enough, it would provide the following benefits:

faster than Node
faster than pnpm
much faster bundler (14x faster than esbuild which Vite relies on)
native Vite support

Bun launches dev servers more quickly
native HMR support

Could possibly be added as a v1 milestone item based on how it will develop within the next few weeks 😊
Luckily, a Nitro engine bun preset is already being planned. It sure looks like a v1 release can happen.

The Nitro preset discussion can be follows here.

[ ] Create Bun runtime to be launched into Lambda


school website: oasisacademyarena.org

email domain: oasisarena.org

Add Universidade Franciscana
ufn.txt

@kgraf80 Please provide us with:

the school official website URL
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@dmpasa Please submit a separate new pull request. Please ensure that you are familiar with this repository rules located at the bottom of this page before submitting the request.

@kgraf80 You might miss my previous comment. Please react.

@philipto school official website: https://www.oasisacademyarena.org/ and school useing domain oasisarena.org for email as you can see on contact subpage (https://www.oasisacademyarena.org/about-us/contact-us), for example<EMAIL_ADDRESS>but this school do not offer IT-related long-term course (at https://www.jetbrains.com/community/education/#students nothing is mentioned that the school must offer IT-related long-term course)

@kgraf80 Pull request merged. Thank you!
For high schools it's fine not to mention the IT-specific courses. Don't hesitate to ask for support if necessary<EMAIL_ADDRESS>


How to detect agent' performance load

[ ] I have checked the FAQ, and issues and found no answer.

Recently, the performance of the application has been degraded, and I want to detect the performance load of the agent itself and the network occupancy.
Help, what methods can we try?

Hi @fionaliupw
You could compare cpu/memory usage or gc logs to see if there's a huge difference with/without the agent.
A few things you might want to check and lessen the load -

Check agent's level - If the agent's log level is set to DEBUG, it'll print out a ton of logs and will affect performance especially in a high traffic environment. Definitely set this to INFO for production servers. The agent's log4j.xml file is in $AGENT_DIR/lib directory.
Change the agent's sampling rate - You might want to change profiler.sampling.rate value in pinpoint.config to a higher value (key is misleading btw). The value here means 1 in N samples, so profiler.sampling.rate=1 is 100%, whereas profiler.sampling.rate=20 is 5%.

Hi @Xylus
Thank you very much for your answer.
I am going to modify the sampling rate to 50% and monitor the CPU/memory of the application server.
Btw, Sometimes the inspector page appearred blank for a few seconds data . What caused this phenomenon? As shown below. Thanks.

@fionaliupw
It might be due to a number of reasons, such as UDP packet being lost, collector rejecting data due to overload (might want to tune collector.receiver.*.worker.* options in pinpoint-collector.properties, or hbase.client.thread* options in collector's hbase.properties), etc.
You might want to check your collector log and see if it's dropping any agent stat data from time to time.

@Xylus Thank you for your help.


Nearby search within a bounding box by latlngBounds

According to documentations, it seems nearby search request can make a request either by:

bounds or
location and a radius

But I wonder why there is no such option in nearby_search()? It has only circular search options!

The documention that you linked to seems to be the docs for the Google Maps API, and not  the docs for the Places API. The Places API documentation is here:
https://developers.google.com/places/web-service/search
This library is indeed missing support for some parameters for the nearby search API, but not one that you have asked for here... We seem to be missing:

minprice
maxprice
opennow
region

I'll get to these as soon as I have some spare time (ha!), or I'd be more than happy to take a PR with support for these params added ;-)


Package can't be imported with module type projects

Hi, I want to use your package to render compiled mdx, so I tried to use it like this:
import * as jsxRuntime from 'only-jsx/jsx-runtime';
const result = new Function(code)({ jsxRuntime });

But I got [vite] Internal server error: Failed to resolve import "only-jsx/jsx-runtime" error, although I don't see errors with vs code and can navigate to the type definitions. It's in an ESM project ("type": "module", in package.json). So I think it could be something wrong with ESM support in the package. Can you check this or maybe you can provide an example with esm project?

Hi,
I have checked the issue.
Yes, there were error in the package.json exports property.
I think that I fixed it now and published new version 0.1.1.
You can use only-jsx library for rendering mdx.
Examples
import {evaluateSync} from '@mdx-js/mdx'
import * as runtime from 'only-jsx/jsx-runtime'

const code = `export const Thing = () => <>World</>

# Hello <Thing />`;

const {default: Content} = evaluateSync(code, runtime);

const e = <Content/>;

with webpack:
rules: [
    ...
    {
        test: /.mdx?$/,
        use: [
            {
                loader: '@mdx-js/loader',
                /** @type {import('@mdx-js/loader').Options} */
                options: { jsxImportSource: 'only-jsx'}
            }
        ]
    },
]

and then in .jsx
import Test from './test.mdx'

as a standalone .js
import { compileSync } from '@mdx-js/mdx'

const code = `export const Thing = () => <>World</>

# Hello <Thing />`;
const js = String(compileSync(code, { jsxImportSource: 'only-jsx' }));
console.log(js);