Currently our requests to the GitHub API are pretty inconsistent. In a few places we use an Octokit, but we construct them at the point of use potentially leading to them being initialized inconsistently. We also sometimes just use an HTTP client.
Perhaps most importantly, none of these methods actually respect the GITHUB_API_URL environment variable that tells you where to find the GitHub API.
Because of this I've replaced these custom Octokits and the HTTP client with a single Octokit to ensure that all requests are made consistently and to the correct host.
Merge / deployment checklist

Run test builds as necessary. Can be on this repository or elsewhere as needed in order to test the change - please include links to tests in other repos!

[x] CodeQL using init/analyze actions
[x] 3rd party tool using upload action

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.

I think a couple of places that making using custom requests could use built-in octokit methods, but making that change at a later date sounds good to me.

I think a couple of places that making using custom requests could use built-in octokit methods, but making that change at a later date sounds good to me.

100%. I was actually pretty surprised we didn't already use built-in methods for things like getting the languages of a repository.


RGBW from loxone to Hass

Describe the bug
Hello
Perhaps a simple answer but I can't seem to solve it. I have some RGBW-zigbee light which I control in Home Assistant. But would it be possible to set the RGBW value in Loxone and send it to Home Assistant?
Right now, I have a marker  setup as Smart Actor RGBW connected to my lighting controller. For some reason I am unable to use it in the app and get the uid. So I connected it through a virtual output command, but that's an analog sensor, so it returns an untranslatable value.
Any idea's on how this can be accomplished?
Regards
Firmware of your Miniserver
.
HomeAssistant install method
.
Version of HomeAssistant
.
Version of Pyloxone
.
Update pyloxone
.
Log
Settings for the log can be found on the main page under the section Log Configuration

@MaartenProvo you need to define room and category and make it visible is UI for on loxone that it shows up I Homeassistent

I got it to the point were it's sending values to Hass. But I get "4.24399163133711e-314" for Red.

@MaartenProvo sorry but I can not help with RGB. I do not have my lights connected to loxone.

I got it to the point were it's sending values to Hass.


HTTP Error should be 403

          Same as above, should be 403

Originally posted by @MHHukiewitz in https://github.com/aleph-im/aleph-vm/pull/458#discussion_r1401941645

Fixed in #485


Implement lazyById in descending order

Sometimes, when dealing with a lot of data, you need to lazily loop through models.
However, often, one would care more about the "newer" objects being processed first.
$query->lazyByIdDesc()

Changing a method signature is a breaking change. Please send this to master instead.


common/pc: disable ath3k if non-free firmware is not available

ath3k requires non-free firmware.
This improves boot speed if enableRedistributableFirmware is false.

Isn't this better placed in NixOS itself, as it applies to the module as a whole, independent of hardware?

I see this is still open. I have changed my mind, I think it fits nicely here. Thumps up from me XD (if it still is the case with needing unfree firmware in newer kernels)


test: support Node.js 14, 15

Coverage remained the same at 100.0% when pulling c3522f7bb3f1ba4fbddf46ff05075f99ad1ef478 on forresst-patch-2 into 2e8e20534ab08fb89aa07e9d57632dcb92b44d71 on master.


Feature/restrictions

Does anyone want to write docs? Our doc syntax scares me...

Coverage decreased (-1.1%) to 98.851% when pulling 581bcd7125e8aa20bf039ede1f252187e1fae505 on feature/restrictions into 589ca11975dd3004b87d29502f250584a57bb57b on develop.

Coverage remained the same at 100.0% when pulling 5f88d675657ca984f28b2b8ca40aa80961d83918 on feature/restrictions into 589ca11975dd3004b87d29502f250584a57bb57b on develop.

Coverage remained the same at 100.0% when pulling cd7706e23d75b1db091f942d932b128be6386697 on feature/restrictions into 589ca11975dd3004b87d29502f250584a57bb57b on develop.


Conditional build steps for local development

For local/dev builds, it usually makes sense to skip slow steps (tests && linters) for the other side. I.e., if someone is dealing with frontend and not touching the backend, I don't see a need to enforce the full set of backend's test on each build.
I'm going to introduce two new ARGfor the Dockerfile -SKIP_BACKEND_TESTandSKIP_BACKEND_TEST`. By default, the build will continue to do the same full build+lint+test as it currently does.
To eliminate tests docker build (or/and compose) should include one of those SKIP_*

closed by #144


Eliminate 500 internal server errors when configuring CLI Pod

Desired Outcome
Deploy scripts should not error out with the following error:
error: 500 Internal Server Error

when we're logging into Conjur via the Conjur CLI Pod. For more context, when we get to the point where the Conjur
CLI Pod is being configured (client cert initialized and log into
Conjur), we see the following error:
Configuring Conjur CLI.
++++++++++++++++++++++++++++++++++++++
Waiting for 'kubectl exec conjur-cli-854cf5d95d-d7vgp -- bash -c "yes yes | conjur init -a cucumber -u https://conjur-oss.local-conjur.svc.cluster.local"' up to 300 s
error: Unable to retrieve certificate from conjur-oss.local-conjur.svc.cluster.local:443
command terminated with exit code 1
.error: Unable to retrieve certificate from conjur-oss.local-conjur.svc.cluster.local:443
command terminated with exit code 1
.Trust this certificate (yes/no): Success!
kubectl exec [POD] [COMMAND] is DEPRECATED and will be removed in a future version. Use kubectl exec [POD] -- [COMMAND] instead.
Created new account 'cucumber'
kubectl exec [POD] [COMMAND] is DEPRECATED and will be removed in a future version. Use kubectl exec [POD] -- [COMMAND] instead.
Logged in
error: 500 Internal Server Error
command terminated with exit code 1

Implemented Changes
The 500 internal server errors are apparently flaky. I do not know the root cause of the issue, but running the Conjur login inside a retry loop should fix the issue.
Connected Issue/Story
Definition of Done

[ ] Secrets Provider builds are passing.

Changelog

[ ] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code
changes, or
[ ] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: insert issue ID
[ ] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[ ] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[ ] There are no security aspects to these changes

Closing this PR, as 500 internal server errors have been addressed with PR #412.


Developer: Content name is not displaying in content list

As a content creator, I want to be able to see a list of content by name, so that I know what content to choose to "edit".
I don't see the names

@joshkimmell this is being worked on.
There is a story for tracking:
https://github.com/punchcard-cms/punchcard/issues/143
and a PR with some open questions:
https://github.com/punchcard-cms/content-types/pull/72
Once that PR is complete, I'll update Punchcard using the new key and that should fix this issue.

This is complete.


Onboard 20% developers to project and team

We are potentially working with one 20%er, but are not prioritizing recruiting and onboarding more (can reopen this if that changes).


Implement Authority as a group (user | client)

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
A permission/authority mechanism is currently being implemented in this PR in link with #288.
In the future, Ralph LRS could be deployed on a large scale to serve multiple organizations (eg. universities and platforms). For practical reasons, as well as RGPD compliance, it will be necessary to include a mechanism in which:

an LRS admin can access all the data
an admin for a platform (eg. moodle) can access all the data written by this platform
a user (eg. student) can access all statements it has produced across all platforms

Describe the solution you'd like
This seems like a job for OAuth (a user authorizes a client to write data on their behalf). The proposed solution would be to write all statements with Authority as a group, containing both the "client" and "user". Authority would resemble:
"authority": {
	"objectType" : "Group",
	"member": [
		{
			"account": {
				"homePage":"http://example.com/xAPI/OAuth/Token",
				"name":"oauth_consumer_x75db"
			}
		},
		{
<EMAIL_ADDRESS>		}
	]
}

OAuth seems to be a necessity as the spec only allows grouped Authority in this situation.

The current roadmap does not include this implementation.


update keyvault for aspire use cases

Fixes #41979
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

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


feat(tooltip): add scrollable view properties to connected pos strategy

Uses new Scrollable changes from #2188. Adds a set of properties to the change position event from the connected position strategy.
Still working on writing the tests for the connected position strategy but wanted to put this out for review to get it looked at ahead of time since beta is on Tuesday and we want this in there.

Removed its usage in tooltip and sidenav until we optimize the number of scroll notifications with a throttle


E403: https://mapbox-node-binary.s3.amazonaws.com/sqlite3/v3.1.8/node-v51-linux-arm.tar.gz

running npm install --only=productiona AccessDenied error gets retruned for https://mapbox-node-binary.s3.amazonaws.com/sqlite3/v3.1.8/node-v51-linux-arm.tar.gz

This is most likely not a fatal error. npm will fallback compiling manually. If it is a fatal error, post full logs.
Another note: Are you running node 5.1? If so: Gekko requires a node version higher than 6, we recommand the LTS version.


[Feature]: Add mature_num and mature_bio to the time_series table

Describe the solution you would like.
Add mature_num and mature_bio to the time_series table. See also #382.
Describe alternatives you have considered
This is already calculated internally, but not reported for the entire time series.
Statistical validity, if applicable
N/A
Describe if this is needed for a management application
No response
Additional context
Simplifies comparison to models that track aggregate mature biomass; could consider reporting aggregate vulnerable numbers & biomass as well (or intersection of mature/vulnerable).

Mature biomass and mature numbers now appear in the time_series table as shown below. And verify that they appear for all years in the spawn_recr table.


[fix][client] Move MessageIdAdv to the pulsar-common module

Motivation
https://github.com/apache/pulsar/pull/19414 does not follow the design of https://github.com/apache/pulsar/issues/18950

Since the aimed developers are Pulsar core developers, it's added in
the pulsar-common module (PulsarApi.proto is also in this module), not
the pulsar-client-api module.

The reason is that TopicMessageId#create now cannot be a MessageIdAdv if MessageIdAdv is not in the pulsar-client-api module.
Modifications

Move the MessageIdAdv class to the pulsar-common module.
To handle the instance created by TopicMessageId well, add MessageIdAdvUtils#convert to add an extra deserialization and serialization of MessageId.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9b72302). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files

@@            Coverage Diff            @@
##             master      #27   +/-   ##
=========================================
  Coverage          ?   72.94%
  Complexity        ?    31934
=========================================
  Files             ?     1868
  Lines             ?   138417
  Branches          ?    15236
=========================================
  Hits              ?   100972
  Misses            ?    29409
  Partials          ?     8036

Flag
Coverage Δ

inttests
24.29% <0.00%> (?)

systests
24.85% <0.00%> (?)

unittests
72.22% <0.00%> (?)

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


Add numpy dependency to setup.py script

Isn't this implied by the astropy dependency?

I think that is right, probably this is not necessary.


404 - UnSuccesfull instance resolution scenario

The following local (in a single machine) setup always fails and returns 404 for the following Url.
http://localhost:8080/Service2/... but http://localhost:8080/Service1/ works..
eureka hosted @9999 using Tomcat_deployment_1; shows the registered service1 and service2 apps and zuul..
Service1 is a simple non spring-boot REST service "/service1" hosted @9080 using Tomcat_deployment_2; which works @http://localhost:9080/Service1/
Service2 is a simple non spring-boot REST service "/service2" hosted @9180 using Tomcat_deployment_3 which works @http://localhost:9180/Service2/
zuul hosted @8080 using bundled jetty in zuul-netflix-webapp exposed @http://localhost:8080/
NOTE - Each tomcat is independent.
#zuul.properties snippet
zuul.niws.clientlist=service1|service2
zuul.niws.defaultClient=service1
zuul.ribbon.namespace=ribbon

service1.ribbon.NIWSServerListClassName=com.netflix.niws.loadbalancer.DiscoveryEnabledNIWSServerList
service1.ribbon.DeploymentContextBasedVipAddresses=service1

service2.ribbon.NIWSServerListClassName=com.netflix.niws.loadbalancer.DiscoveryEnabledNIWSServerList
service2.ribbon.DeploymentContextBasedVipAddresses=service2

#service1 eureka-client.properties
eureka.preferSameZone=true
eureka.shouldUseDns=false
eureka.serviceUrl.default=http://localhost:9999/eureka/v2/
eureka.region=default
eureka.name=service1-Eureka
eureka.vipAddress=service1
eureka.port=9180
eureka.instanceId=MYMAC1123456L

#service2 eureka-client.properties
eureka.preferSameZone=true
eureka.shouldUseDns=false
eureka.serviceUrl.default=http://localhost:9999/eureka/v2/
eureka.region=default
eureka.name=service2-Eureka
eureka.vipAddress=service2
eureka.port=9080
eureka.instanceId=MYMAC1123457L

The above problem will not surface and the routing always works if both the apps are deployed in same tomcat and the client-side-loadbalancing also works if both the apps are deployed in diff tomcats... The problem only surface if diff app is hosted under diff server deployment...
Any ideas folks, what can be wrong?

This is a spring boot issue, not a zuul one.. Sorry!

@mikeycohen -- as specified in the issue, the issue is seen in non-spring-boot code.... exact codebase from github is used for Eureka and Zuul.. Not sure why you thought the other way... Please advise...


Onchain reading from layer 1

Feature Request
We should be able to read data from the native layer 1 ENS solution. This will be the foundation for the future offchain implementations.
Describe Preferred Solution
The client needs to be able to call the Registry for getting the given domain's Resolver for then to read its properties stored onchain.
sequenceDiagram
    Client->>Registry: resolver("test.eth")
    Registry->>Client: L1 Contract address
    Client->>L1 Contract: resolve("test.eth")
    L1 Contract->>Client: 0x....

Additional Context
The reading should be pretty straightforward giving that the SDKs support the layer 1 reading and writing by default.
Related Code

Viem reading implementation
Ethers reading implementatio

Blocked by #11

Solved in #11


0.6.2

Closes #185

r? @therealprof
(rust_highfive has picked a reviewer for you, use r? to override)

... and published.


Connection to host '<IP_ADDRESS>' not possible, aborting...

Hi thank you for making this - I get in trouble fairly quickly, though, when I do either of these:

remarkable2pdf 'mysearch'  --host <IP_ADDRESS>
remarkable2pdf 'mysearch'  --host<EMAIL_ADDRESS>

I get:
Connection to host '<IP_ADDRESS>' not possible, aborting...

I can ssh into the tablet fine by doing:

ssh<EMAIL_ADDRESS>

Here I have to enter this:

Enter passphrase for key '/home/emil/.ssh/id_rsa':
<EMAIL_ADDRESS>password:

I have the passphrase empty. The root psw is the one provided in the remarkable GUI.
I'm sort of learning by doing, so if you could help me understand what I'm doing wrong here I would appreciate it!

I can ssh into the tablet fine by doing:
ssh<EMAIL_ADDRESS>
Here I have to enter this:
Enter passphrase for key '/home/emil/.ssh/id_rsa':
<EMAIL_ADDRESS>password:

I have the passphrase empty. The root psw is the one provided in the remarkable GUI.

So this seems to me, that you have generated an ssh-key (i.e. /home/emil/.ssh/id_rsa), but the tablet doesn't recognize it or doesn't allow it as an authentication mechanism.
That's why you have to provide the tablet-password then afterwards. Strictly speaking the password for the key in this case "should" be irrelevant, as the key is irrelevant.
This could honestly be because of a lot of reasons.
Depending on how familiar you are with ssh in general you might have a sshconfig. Generally this would be a file at ~/.ssh/config.
I would recommend that you add an entry in there, as there are a couple of options you have to set with the modern variant of the tablet.
My entry looks like this:
Host local-remarkable
    HostName <IP_ADDRESS>
    User root
    StrictHostKeyChecking no
    IdentitiesOnly yes
    IdentityFile ~/.ssh/remarkable
    HostKeyAlgorithms +ssh-rsa
    PubkeyAcceptedKeyTypes +ssh-rsa

You would need to adapt the IdentityFile to an absolute path to your specific key. Afterwards you can than always only run:
ssh local-remarkable

Additionally you have to tell your tablet to accept the login with your key. The remarkablewiki has some help for that if you want more details (also on the host entry) here. The short gist of it is to run:
ssh-copy-id -i ~/path/to/your/ssh/key<EMAIL_ADDRESS>
... once and afterwards it should run fine without needing the password for a login. Afterwards the scripts also should work fine again.
On another note though the scripts are currently working with rmrl as a renderer library which has not been updated yet to the new notebook format that came with the big v3 system update.
I'm working on an alternative but this could still take some time. So don't expect everything to work out of the box.

nice thank you for the help, I'll try it out one of the coming days. I'm totally new to ssh and only learned it because I wanted access to the rm2 and transfer files, so there seems to be some learning for me here.
I haven't updated to 3.0.0 thank God, since I want to keep dvi hacks.

Ok that worked. Removed
~/.ssh/the id_rsa
and followed the wiki entry for pwd-less entry worked.


I can't download it on android , please help me to download it

Describe the feature here

hwy , i am intermediate in python is there anything i can help with?

For any technical support join the discord server. (You can join in with an unclaimed discord account w/o registering)
The GitHub issues is a place to report bugs that you encounter in the app or to suggest new features etc
Don't expect any technical support by making an issue.


any chance for FreeBSD support

Support for FreeBSD 11.2-Stable

Hey, @fdge! Yes, FreeBSD will be supported eventually, but right now Rust's Tier 1 platforms are the only OSes being implemented.
It is possible to start adding FreeBSD support for some sub-crates, as they are stable enough, but I'm focused on a Linux/macOS/Windows triple and public API forming, so there is no ETA at all.
Cross-linking for clarity: https://github.com/nushell/nushell/issues/428#issuecomment-524704391

Any updates on this? heim seems to be stable for tier 1 platforms, and I would love FreeBSD support to try to get bottom working on 13.1.
https://github.com/ClementTsang/bottom/issues/480


modifyPath does not work for empty array path

In this case nothing happens:
modifyPath([], append(1), []) => []
But logically modifyPath should apply function for empty array and in result there should be [1] and not []
But actual output is an empty array []

What path would an empty array represent?
This is what works:
modifyPath([0], append(1),[[]]);
// -> [[1]]

Shouldn't an empty path point to the supplied object?
Something like this:
modifyPath([], f, a) === f(a)

But the function has a guard fast returning the final argument in case of an empty path
if ((!_isObject(object) && !_isArray(object)) || path.length === 0) {
  return object;
}

Something like this would satisfy what's being asked:
if (!_isObject(object) && !_isArray(object)) {
  return object;
}
if (path.length === 0) {
  return fn(object);
}

I'm not sure if this goes against any of the principles of the library though.

@JuanGalilea yep, that is what I meant, thanks.
@CrossEye what do you think?

Finally brought myself to open a pull request for this.
Checked all other path functions, all of them already supported this behaviour (except for R.assocPath, but i dont know how would that work ).


Disabling chrome's autofill

I'm running into a problem where chrome thinks my field is an address field and tries to show a list of values to fill it.
Apparently we can disable this by setting autocomplete="off" (see this) on the input, but because I have no direct control over the input I can't do this.
Is there a way of doing this through the plugin options/ props? I've gone over the docs and can't seem to find one.

Added autocomplete prop in 2.2.0.


exec DM model with gpu

If this passes test_onroad's timing checks for model/camera/etc, then I think it's fine to merge into master. It'll likely have some longer tail issues, but we're working on the fixes for them now.


Standardize render return values

Many use return this, and a number use return this.$el. The former is more flexible. I could argue that  this.$el is more consistent with what we'd expect from render.

return this is the convention documented in the Backbone docs: http://backbonejs.org/#View-render
It also encourages us to work with the View as the primary object rather than the DOM, since we ideally shouldn't be doing a lot of cross-view DOM manipulation.


[FEATURE] Add a new page to display all contributors

I want to add a page to showcase all the contributors of this website..
@ANSHIKA-26  please assign me this task.

Thank you for the suggestion, but I’ll have to pass on adding a Contributors page. While I appreciate the idea, it’s not a priority for the project right now, and we prefer to keep the focus on core functionality. Contributors are already acknowledged through GitHub contributions. Thanks for your understanding!

Thank you for the suggestion, but I’ll have to pass on adding a Contributors page. While I appreciate the idea, it’s not a priority for the project right now, and we prefer to keep the focus on core functionality. Contributors are already acknowledged through GitHub contributions. Thanks for your understanding!

Okay


JTCalendar returning Segmentation Fault / Bus Error

Receiving this when trying to build:

Another screenshot of error:

res


Does this remove no-longer-used build output?

I'm trying this because I wanted "incremental builds". A requirement of many "incremental build" systems is that they not only update build output based on the current files, but they also delete output if those files would not exist on a clean re-build.
For example, if I move file X.ts to Y.ts, will my output folder still have X.js after running webpack again?
Is so, how can we prevent this?

After some testing, it seems that after removing all build files, a rebuild is still much faster than first build, and it must be just replacing all the removed files from the cache.
Is this the recommended way to make sure that no-longer-existing source files have their respective build output deleted?
It seems that not having to delete the source folder would be more efficient and faster. Going to test it out to answer this question.

Oh, I see, so as long as the entry point doesn't change, they have to be written to disk once rebuilt anyways.
I was accidentally thinking that this was like Babel or TypeScript compilers where they can create one output file per input file, but Webpack is always bundling the entry points, so deleting or not deleting the output files doesn't matter since they will always be written or re-written anyways.

Yeah. Its intentional that files are reproduced. Some more steps would be needed to what not to emit.
A number of bugs have related to hard-source needing to emit files that it was failing to track. The current extract-text-webpack-plugin support is one of those. Files referred in css that gets extracted weren't being emitted. To support extract-text-webpack-plugin some rebuilding is needed, fortunately the amount is very small and mostly unnoticeable.
Even if those kinds of things weren't problems hard-source probably won't try to deduce what files can't be skipped when webpack emits.
(Its fun to see your progress over this little ticket.)


custom-query validations broken on Hive backend

When running custom-query column or row validations, the query generated is invalid. It appends a "t0" prefix to the column names which creates an invalid query.
Command to reproduce:
data-validation -v validate custom-query column -sc hive -tc hive -sq "select id, name as count_name from default.mascot" -tq "select id, name as count_name from default.mascot"

Query generated:
SELECT count(1) AS `count`
FROM (
  SELECT t1.`t0.id`, t1.`t0.name`
  FROM (
    select id, name from default.mascot
  ) t1
) t0

Error: impala.error.HiveServer2Error: Error while compiling statement: FAILED: SemanticException [Error 10002]: Line 3:12 Invalid column reference 't0.id'
We should add a custom query integration test for Hive as part of this PR.

Issue stems here: https://github.com/GoogleCloudPlatform/professional-services-data-validator/blob/develop/data_validation/clients.py#L146
If I print (iq.columns), it returns ['t0.id', 't0.name'] when it should be ['id', 'name']

Possible fix is to remove t0. prefix which is getting generate at this function
Initial test worked fine. But need thorough testing.
cc - @piyushsarraf

/gcbrun


Update README.md

ssh password note

Thanks for the help @pieterjandebruyne but this is now resolved in https://github.com/getumbrel/umbrel-os/pull/180


Docker Swarm: Using constraint and affinity with replicas and scale

The good - I know how to use constraint or affinity flags to ensure two containers are not living on the same node. From the docs:
affinity:container!=~redis* schedule a new redis5 container to a node without a container that satisfies redis*

The challenge - When using docker service scale redis=3, I don't see how can I make sure there is only one container per node ??? Looks like Swarm scales ignores the constrait of the negative affinity flag.
For an HA setup, I want to deploy 2 instances of a service and having Swarm to never deploy the service on the same node. IMHO this should this should works:
ENV_NET_STATIC=net_static
ENV_DOMAIN1=iamfoo.pascalandy.com
ctn1_NAME=iamfooA

docker service create \
--name $ctn1_NAME \
--network $ENV_NET_STATIC \
--restart-condition any \
--replicas 2 \
--constraint node.labels.apps_accepted==yes \
--env affinity:container!=~$ctn1_NAME* \
--label traefik.frontend.rule=Host:$ENV_DOMAIN1 \
--label traefik.docker.network=$ENV_NET_STATIC \
--label traefik.backend.loadbalancer.sticky=true \
--label traefik.port=80 \
katacoda/docker-http-server

The key here is --env affinity:container!=~$ctn1_NAME*, which is the same logic provided by the docs above.
Thanks in advance for your help!
Pascal

The docs you linked are for standalone swarm, not for swarm mode (which was introduced in docker 1.12). swarm mode does not have an option to have anti-affinity based on other services/containers (see https://docs.docker.com/engine/reference/commandline/service_create/#/specify-service-constraints---constraint).
Also see https://github.com/docker/docker/issues/24649, https://github.com/docker/docker/issues/26259, and https://github.com/docker/docker/issues/24115
IIRC, this is currently by design, as there were quite some issues around this functionality in "standalone swarm", and they did not want to implement this functionality without properly designing the behavior.
I'll close this issue, because I think it's already covered by the issues I linked above, but feel free to comment on those to describe your use-case as it may help in the design/discussion

Thanks for the references :)

Also note the new HA Scheduler in docker 1.13; https://github.com/docker/docker/issues/26259#issuecomment-245472484, which may address your use case


Demo not working

I'm considering using this for a project and noticed that the demo page isn't working. There is a link to open the picker, but clicking it does nothing. I'm using Chrome 51 on OS X.

To which page are you referring? Both the demo page in the source code and http://dalelotts.github.io/angular-bootstrap-datetimepicker/ work fine for me on Chrome 52 on OSX.

When I view the page it is unformatted and nothing works. I've attached a screenshot of the page in both Chrome 51 and Safari. As you can see I've disabled all of my browser extensions in Chrome as well. The console reveals the following:
about:blank:1 Mixed Content: The page at 'https://dalelotts.github.io/angular-bootstrap-datetimepicker/' was loaded over HTTPS, but requested an insecure resource 'http://ghbtns.com/github-btn.html?user=dalelotts&repo=angular-bootstrap-datetimepicker&type=watch&count=true'. This request has been blocked; the content must be served over HTTPS.
2016-08-06 10:30:48.039 about:blank:1 Mixed Content: The page at 'https://dalelotts.github.io/angular-bootstrap-datetimepicker/' was loaded over HTTPS, but requested an insecure resource 'http://ghbtns.com/github-btn.html?user=dalelotts&repo=angular-bootstrap-datetimepicker&type=fork&count=true'. This request has been blocked; the content must be served over HTTPS.
2016-08-06 10:30:48.090 (index):1 Mixed Content: The page at 'https://dalelotts.github.io/angular-bootstrap-datetimepicker/' was loaded over HTTPS, but requested an insecure stylesheet 'http://netdna.bootstrapcdn.com/bootstrap/3.3.5/css/bootstrap.min.css'. This request has been blocked; the content must be served over HTTPS.
2016-08-06 10:30:48.150 (index):1 Mixed Content: The page at 'https://dalelotts.github.io/angular-bootstrap-datetimepicker/' was loaded over HTTPS, but requested an insecure script 'http://code.jquery.com/jquery-2.1.4.min.js'. This request has been blocked; the content must be served over HTTPS.
2016-08-06 10:30:48.151 (index):1 Mixed Content: The page at 'https://dalelotts.github.io/angular-bootstrap-datetimepicker/' was loaded over HTTPS, but requested an insecure script 'http://cdnjs.cloudflare.com/ajax/libs/moment.js/2.10.6/moment.min.js'. This request has been blocked; the content must be served over HTTPS.
2016-08-06 10:30:48.152 (index):1 Mixed Content: The page at 'https://dalelotts.github.io/angular-bootstrap-datetimepicker/' was loaded over HTTPS, but requested an insecure script 'http://netdna.bootstrapcdn.com/bootstrap/3.3.5/js/bootstrap.min.js'. This request has been blocked; the content must be served over HTTPS.
2016-08-06 10:30:48.153 (index):1 Mixed Content: The page at 'https://dalelotts.github.io/angular-bootstrap-datetimepicker/' was loaded over HTTPS, but requested an insecure script 'http://ajax.googleapis.com/ajax/libs/angularjs/1.4.7/angular.js'. This request has been blocked; the content must be served over HTTPS.
2016-08-06 10:30:48.166 datetimepicker.js:29 Uncaught TypeError: Cannot read property 'module' of undefined(anonymous function) @ datetimepicker.js:29(anonymous function) @ datetimepicker.js:25(anonymous function) @ datetimepicker.js:27
2016-08-06 10:30:48.350 (index):1 GET https://dalelotts.github.io/angular-bootstrap-datetimepicker/img/github-16px.png 404 (Not Found)

I can confirm the same behavior in Firefox 44 as well. Strangely if I visit the page on my iPhone it works fine.

That's just a mixed content issue on the page - I will update the index.html page. Load it with http:// rather than https:// for now and it will work fine.

Should be fixed now.

Thanks! 👍


Not all cameras are recognized by the HA integration - but they are available in the eufy security app.

Hi,
sorry I dont know if its a bug or not.
I am slowly going crazy with the camera integrations of an eufy cam 2 pro in home assistant. In total I have 3 pieces. I have integrated the eufy security home base according to the instructions and the apps from GitHub - fuatakgun/eufy_security . Everything is working fine so far. Thanks for that! 2 cameras are integrated with functioning rtps streams in lovelace. just not the third one. The integration simply does not want to recognize this (the other 2 were automatically detected). However, all 3 are displayed in the eufy phone app in the admin access as well as in the guest access and it works; just not in home assistant. I have already made a backup restore and set up everything again, reinstalled the eufy home base, the apps and accesses, the connections of the cameras. nothing has worked. Does anyone know how I can still integrate the one missing camera in ha? or that this is automatically recognized by the ha integration? Does anyone know the problem?
Sorry for my frustrating writing :)
best greetings from germany, goldnase

Hey, i will br back from vacation next week, happy to debug this issue later on, hang on there

Problem solved. It was apparently due to the camera itself. I ordered a new one as a test. This was recognized in HA. I have no idea what the real mistake is.


Integrate Tune into Auto

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Proposed Changes
Look at #1269.
Closing issues
Closes #1269.

Codecov Report

:exclamation: No coverage uploaded for pull request base (automl-2.0@f4b50fe). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@              Coverage Diff              @@
##             automl-2.0    #1272   +/-   ##
=============================================
  Coverage              ?   53.30%
=============================================
  Files                 ?      186
  Lines                 ?    11210
  Branches              ?        0
=============================================
  Hits                  ?     5976
  Misses                ?     5234
  Partials              ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat(FN-2022): update total fees report header

Ticket 🎫
FN-2022
Introduction :pencil2:
We want to change the 'Total fees accrued for the month' header to 'Fees accrued since the last report' to be inclusive to banks which do not use monthly reporting schedules.

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

Comparison is base (91637c1) 17.31% compared to head (60458b4) 26.48%.
Report is 218 commits behind head on main-stbr-tfm-5.

Additional details and impacted files
@@                 Coverage Diff                 @@
##           main-stbr-tfm-5    #2580      +/-   ##
===================================================
+ Coverage            17.31%   26.48%   +9.17%
===================================================
  Files                 1120      425     -695
  Lines                25591    11461   -14130
  Branches              4101     1414    -2687
===================================================
- Hits                  4432     3036    -1396
+ Misses               18423     7699   -10724
+ Partials              2736      726    -2010

Flag
Coverage Δ

api-test
?

ui-test
91.78% <ø> (+40.96%)
:arrow_up:

unit-test
5.15% <ø> (-2.81%)
:arrow_down:

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

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


adding pyinstaller to requirements.txt

Can you add pyinstaller to requirements.txt?
It fails on default ubuntu installation, I had to run pip install pyinstaller manually

Thank you, I only asked about pyinstaller because of the Makefile ... the default action invokes pyinstaller


Fully blocking read

AFAIU recv has semantics like hGetSome, it would be nice to have a version that behaves like hGet, i.e. an easy way of reading n bytes, blocking until all the bytes are available.

Given the age and activity on this issue I'm closing it.


GCC Tennant Licensing?

This article doesn't advise if it applies to users in the Office 365 GCC environment as our licensing options are G1, G3, and G5. Can you update the article to include this environment and the required licensing or advise if this isn't supported.

@officedocsbot assign @mypil

@PSJeffH - We would like to get some details from you. Can you please share what specific issue you have and for which article? Kindly provide the article link as well so we can investigate.
Thank you.

Oh, my apologies I thought it included the link already.
Link in Question which I'd like updated: https://docs.microsoft.com/en-us/office365/admin/create-groups/groups-naming-policy?view=o365-worldwide
My Tennant for Office 365 is Government and it becomes difficult to determine what features we have, and can do when there are different Tenants of Office 365.
GCC Link on GCC Details https://docs.microsoft.com/en-us/office365/servicedescriptions/office-365-platform-service-description/office-365-us-government/gcc
Licensing requirements as it doesn't mention GCC environments.
To use the Groups naming policy feature, the following people need an Azure Active Directory Premium P1 license or Azure AD Basic EDU license:

@PSJeffH Thank you for the feedback. The license requirement in the article refers to the Plan that is needed to use group naming policy feature.
In your case the Azure Active Directory Premium P1 license  is an add on SKU for Office 365 SKU's(GCC or enterprise editions ). In short, If your license(G1 , G3 or G5  includes Azure AD Premium Plan 1 then you will be able to access the group naming policy feature, Else you will have to purchase this as add-on.  Hope this helps!
Thanks
Sri

@PSJeffH - Thank you for submitting this feedback.
From our understanding, the issue you raised has been answered by msbemba. If you feel it hasn't been answered, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Add Grid.Connect(GridPosition position)?

I'm assuming I'm either :
A.) missing some obvious reason why this is not implemented in the public API like Grid.Disconnect
or
B.) not understanding how if or how Grid.AddEdge works in adding/readding walkable "valid" nodes.

Hey thanks for your question.
If I understand you correctly a connect method will connect that one node to all its neighbors. That is a bit problematic since some users want only lateral connections, or only diagonal, or both. While disconnect just disconnects everything.
I could create utility methods for connect for a few logical combinations. Just like I did for creating a grid. I just never thought about it. Do you think it would be useful to have those in the library?
Of course you can for now implement it using AddEdge several times. Or are you running into problems there?

Hey thanks for your question.
If I understand you correctly a connect method will connect that one node to all its neighbors. That is a bit problematic since some users want only lateral connections, or only diagonal, or both. While disconnect just disconnects everything.
I could create utility methods for connect for a few logical combinations. Just like I did for creating a grid. I just never thought about it. Do you think it would be useful to have those in the library?
Of course you can for now implement it using AddEdge several times. Or are you running into problems there?

Thank you getting back to me!
Yes, I have stumbled into understanding how to use AddEdge to sort of get it working.
Feel free to tell me if this is not how I should be using AddEdge.
			var targetNode = Grid.GetNode( gridPos );
			var targetNodePosition = ToGrid( targetNode.Position );
			var neighbors = GetNeighbors( targetNode, true );

			foreach ( var n in neighbors )
			{
                                // Neighbors can be null at the edges of the map.
				if ( n is null )
					continue;

				var neighborPos = ToGrid( n.Position );
				Grid.AddEdge( targetNodePosition, neighborPos, Velocity.FromKilometersPerHour( 100 ) );
			}

I'm essentially trying to have the ability to tell the library that a node is not considered "walkable" and to flip that flag when I want (adding/removing obstacles). I figure Connecting/Disconnecting the Nodes I want to do that to is the idiomatic way. If such utility methods would make that easier for me, then yes I would definitely appreciate that.

Ah, yes that is indeed the correct way. This comes because the actual library is based on nodes and edges, not a grid. The grid class is just an abstraction above it to help users with the common case of a grid. In this case that abstraction leaks through. Glad you have figured this out. I'll make a note of this and will see if in the next version I can make using grids more logical.


WIP: Examples multiple language versions

Just provide some examples how to do "different language versions" or even "different boxes"
(the examples are for PHP 5.5/5.6/7.0, but are certainly useful for other languages as well)

I am not sure about merging this one.
The idea of having examples is great, but I think we should wait to address the more general issues of documentation (see https://github.com/team-rawbot/rawbot-virtualization/issues/48) in order to determine where is the best place to put this.
If we decide to have some external documentation it would probably better to put it there.
(Also, I think the ARGV[1] thingy might not play really well with my code to detect the current provider, both seems kind of hacky, there's probably a better way to parse CLI arguments)

It's fine for me, when we don't include that for now. I agree it's hacky and I'm still searching for better solutions

I just added a docs directory, I think this content could go in a new file docs/tips/multiple-version.md or something like that.
You also have to update the mkdocs.yml file to reference your file afterward.

@chregu I think it's great to have an examples directory for code examples. I would just move the README you put in the examples directory to a page in the docs directory which would explain the concept in detail, with references to the examples directory.


Plant Debug Display

When you have a plant selected, show a detailed plant dialog that has the cost, production, production interval, etc...
Make a mockup for this design and implement it in unity.

Basic version of this has been added by @swiimii
Should update to include:

Production interval - should be easy to add!
Victory Points - should be easy to add!
Plant Type (Vegetable, Tree, etc...) - maybe an icon for this?
Restriction range
Grow Range

Small icons plus numbers should suffice for these!


fix ItemUseInventoryTransaction.clickPos

type of clickPos is BlockPos.
In fact,  pos was just clickPos.
I made pos deprecated.

clickPos is a relative click position of the block.
it's not the integer coordinate.


xpath function not working on Chrome

I don't know what changed, but Chrome is complaining about i being undefined here. Putting "var" before the i fixes it.
for (i = 0; i < res.snapshotLength; i++) arr.push(res.snapshotItem(i));

This should resolve the issue you mentioned, but there are probably more places with the same issue, so I'll leave this issue open for now.

It should only complain about this kind of error when using strict mode though, which really shouldn't be enabled by default. :S

Maybe I had enabled strict mode on Tampermonkey at some point. Anyway, this should help you find any more such errors: http://jshint.com/


WIP: stop current stopwatch when another is started

Fixes #10529
I don't really code in Go so scrutinise this in detail ;) It's really only a suggestion, may want to make it configurable.. see #10529

I still have this merged in locally. It's proving very useful for my workflow.

I think no one has looked at this because it is still marked WIP.

@bobemoe Could you update the first comment and title, and provide a bit more explanation as to what this is supposed to do and why it is useful.

@zeripath I've removed WIP so it can get some attention and updated the description, thanks.


HV-1070 Making build more friendly towards Java 9

@gsmet Another quick one :)

Rebased and merged! Thanks!


JsObject.assign(Object,Object...) should have @DoNotAutobox on the second parameter

The JsObject.assign method should have the @DoNotAutobox added to the second parameter (at least under GWT2.x). Otherwise, the runtime casts it to a java array which will ultimately cause incorrect behavior.
i.e.
public class JsObject {
    @JsOverlay
    public static final JsObject assign(Object target, Object... var_args) {...}
   ...
}

should look like
public class JsObject {
    @JsOverlay
    public static final JsObject assign(Object target, @DoNotAutobox Object... var_args) {...}
   ...
}

I haven't tracked down where this needs to be fixed (presumably jsinterop-generator) but I just wanted to record the issue so it did not get lost in the ether.

In j2cl the semantics are different IIRC, so that a plain Java Object[] is actually implemented as a JS array (with some predictable annoyances, like JsArray.toString() spitting out the Object[].toString() results instead of JS's Array.prototype.toString).
But while we're talking about the annotation (and this might be better in another ticket)... how do you indicate "do not autobox any of the args in this varargs"? If you pass an int as one of the args, it is boxed up to Integer, which isn't what is intended - console.log(...) is particularly annoying here.

DoNotAutobox doesn't prevent Java array and Java arrays only different by metadata so I don't expect any incorrect behavior.

Oh - I didn't realize that @DoNoAutobox would not work in this context. I am not sure what the correct fix is then.
If we have the following code
final JsPropertyMap<Object> value1 = JsPropertyMap.of( "A", "a" );
final JsObject value2 = Js.uncheckedCast( value1 );

DomGlobal.console.log( JsObject.assign( JsPropertyMap.of(), value1 ) );
DomGlobal.console.log( JsObject.assign( JsPropertyMap.of(), value2 ) );
DomGlobal.console.log( JsObject.assign( JsPropertyMap.of(), Js.uncheckedCast( value1 ) ) );

we get the following in GWT2
{0: {…}, ___clazz: Class, castableTypeMap: {…}, __elementTypeId$: 1, __elementTypeCategory$: 5, typeMarker: ƒ}
{0: {…}, ___clazz: Class, castableTypeMap: {…}, __elementTypeId$: 1, __elementTypeCategory$: 5, typeMarker: ƒ}
{A: "a"}

As you predicted, adding @DoNotAutobox does not have any impact.
FWIW the relevant elemental2 code looks something like:
@JsType( isNative = true, name = "Object", namespace = JsPackage.GLOBAL )
public static class JsObject
{
...
  public static native JsObject assign( JsObject target, JsObject... var_args );

  @JsOverlay
  public static final JsObject assign( Object target, Object... var_args )
  {
    return assign( Js.<JsObject>uncheckedCast( target ), Js.<JsObject>uncheckedCast( var_args ) );
  }
}

So is there any way to make the translation of javascript varargs to java varargs "safe"? I tried several different mechanisms but none seemed to work.

Ok thanks for detail report; now it makes all sense.
The overlay method is the cause of the problem which effective result in converting a var-arg API to an Array API and then pass it directly.
And the overload is needed to introduce cast so JsCompiler doesn't complain.
The solution is complicated with current design and for this to work assign should be called with apply since we cannot use spread-operator in Java,
Or we can introduce some magic method supported by compiler to do spreads.
If anyone can think of some clever alternatives, please shoot it.

This seems like a bug in GWT. When we call a JsVarags (i.e. a JsMethod with
varargs) we actually handle it, i.e. in J2CL we do the spread and do not
pass the array directly and in GWT we do apply so that the array is passed
as individual values.
So in your example the JsObject.assign overlay calls directly the native
assign but passes the array directly instead of using the apply. You should
confirm this by looking at the generated code.
If that is so there is probably a bug on the decision whether to use apply
or not.
Here is a pointer of the line where the decision is made to use apply
because of varargs:
https://source.corp.google.com/piper///depot/google3/third_party/java_src/gwt/svn/trunk/dev/core/src/com/google/gwt/dev/js/JsUtils.java;rcl=310791750;l=271
and the relevant cal is when handling the JMethodCall in
GenrateJavaScriptAst.transformMethodCall.
If you want to debug a contribute the fix I'll be happy to review.
On Mon, May 11, 2020 at 1:51 PM Goktug Gokdogan<EMAIL_ADDRESS>wrote:

Ok thanks for detail report; now it makes all sense.
The overlay method is the cause of the problem which effective result in
converting a var-arg API to an Array API and then pass it directly.
And the overload is needed to introduce cast so JsCompiler doesn't
complain.
The solution is complicated with current design and for this to work
assign should be called with apply since we cannot use spread-operator in
Java,
Or we can introduce some magic method supported by compiler to do spreads.
If anyone can think of some clever alternatives, please shoot it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/elemental2/issues/139#issuecomment-626956074,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA73NFH6OKZF4WKDWU4HBN3RRBQORANCNFSM4M4CKJHQ
.

If we are already auto-spreading with arrays we should just fix the
call-site. The method call is passing just an object.
On Mon, May 11, 2020 at 3:35 PM Roberto Lublinerman <
<EMAIL_ADDRESS>wrote:

This seems like a bug in GWT. When we call a JsVarags (i.e. a JsMethod with
varargs) we actually handle it, i.e. in J2CL we do the spread and do not
pass the array directly and in GWT we do apply so that the array is passed
as individual values.
So in your example the JsObject.assign overlay calls directly the native
assign but passes the array directly instead of using the apply. You should
confirm this by looking at the generated code.
If that is so there is probably a bug on the decision whether to use apply
or not.
Here is a pointer of the line where the decision is made to use apply
because of varargs:
https://source.corp.google.com/piper///depot/google3/third_party/java_src/gwt/svn/trunk/dev/core/src/com/google/gwt/dev/js/JsUtils.java;rcl=310791750;l=271
and the relevant cal is when handling the JMethodCall in
GenrateJavaScriptAst.transformMethodCall.
If you want to debug a contribute the fix I'll be happy to review.
On Mon, May 11, 2020 at 1:51 PM Goktug Gokdogan<EMAIL_ADDRESS>wrote:

Ok thanks for detail report; now it makes all sense.
The overlay method is the cause of the problem which effective result in
converting a var-arg API to an Array API and then pass it directly.
And the overload is needed to introduce cast so JsCompiler doesn't
complain.
The solution is complicated with current design and for this to work
assign should be called with apply since we cannot use spread-operator in
Java,
Or we can introduce some magic method supported by compiler to do
spreads.
If anyone can think of some clever alternatives, please shoot it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<https://github.com/google/elemental2/issues/139#issuecomment-626956074
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AA73NFH6OKZF4WKDWU4HBN3RRBQORANCNFSM4M4CKJHQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/elemental2/issues/139#issuecomment-627001936,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7MB6GE4OOEVH6VY62R66LRRB4TTANCNFSM4M4CKJHQ
.

The fix should be that the @Overlay calls with a
Js.<JsObject[]>uncheckedCast (i.e. a cast to a JsObject[] instead of a
JsObject[]). Without that, per Java semantics, it will just send an array
one argument, instead of passing the array as the varargs array.
On Mon, May 11, 2020 at 4:08 PM Goktug Gokdogan<EMAIL_ADDRESS>wrote:

If we are already auto-spreading with arrays we should just fix the
call-site. The method call is passing just an object.
On Mon, May 11, 2020 at 3:35 PM Roberto Lublinerman <
<EMAIL_ADDRESS>wrote:

This seems like a bug in GWT. When we call a JsVarags (i.e. a JsMethod
with
varargs) we actually handle it, i.e. in J2CL we do the spread and do not
pass the array directly and in GWT we do apply so that the array is
passed
as individual values.
So in your example the JsObject.assign overlay calls directly the native
assign but passes the array directly instead of using the apply. You
should
confirm this by looking at the generated code.
If that is so there is probably a bug on the decision whether to use
apply
or not.
Here is a pointer of the line where the decision is made to use apply
because of varargs:

https://source.corp.google.com/piper///depot/google3/third_party/java_src/gwt/svn/trunk/dev/core/src/com/google/gwt/dev/js/JsUtils.java;rcl=310791750;l=271

and the relevant cal is when handling the JMethodCall in
GenrateJavaScriptAst.transformMethodCall.
If you want to debug a contribute the fix I'll be happy to review.
On Mon, May 11, 2020 at 1:51 PM Goktug Gokdogan <
<EMAIL_ADDRESS>wrote:

Ok thanks for detail report; now it makes all sense.
The overlay method is the cause of the problem which effective result
in
converting a var-arg API to an Array API and then pass it directly.
And the overload is needed to introduce cast so JsCompiler doesn't
complain.
The solution is complicated with current design and for this to work
assign should be called with apply since we cannot use spread-operator
in
Java,
Or we can introduce some magic method supported by compiler to do
spreads.
If anyone can think of some clever alternatives, please shoot it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<
https://github.com/google/elemental2/issues/139#issuecomment-626956074
,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AA73NFH6OKZF4WKDWU4HBN3RRBQORANCNFSM4M4CKJHQ

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/google/elemental2/issues/139#issuecomment-627001936
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AA7MB6GE4OOEVH6VY62R66LRRB4TTANCNFSM4M4CKJHQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/elemental2/issues/139#issuecomment-627012726,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA73NFDT74GWF2IR3XPW3LLRRCAQJANCNFSM4M4CKJHQ
.

The error was in the jsinterop_generator which generates the @JsOverlays. The fix is already in. So elemental2 will be fixed in the next release.

Great - thanks!

I think this can be closed, since 1.1.0 shipped.


Environment Variables for Line Breaks and Other Styling

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I wanted to enable strict line breaks in a Docker install of Perlite.
Describe the solution you'd like
A clear and concise description of what you want to happen.
It would be useful if there were environment variables we could use to change these behaviours instead of having to edit the php files.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Installed the Docker image to a local volume and used a text editor to change the files. This is an imperfect solution because it would be overwritten upon future updates. Environment variables would be a much better in the long run.
Additional context
Add any other context or screenshots about the feature request here.

added in version / branch 1.5


fix(harfbuzz): uncouple from Skia

@glennsl mentioned in #927 that these two libraries should be separate, and I totally agree. This uncouples them completely, except for the examples/CI to get font discovery working. @glennsl, let me know if you want me to remove that as well.

This uncouples them completely, except for the examples/CI to get font discovery working. @glennsl, let me know if you want me to remove that as well.

That would be great, but not essential. I'm moving the examples back into the respective packages as well, and it would be nice to be able to work on harfbuzz without having to compile the monstrosity that is skia.

@glennsl sounds good, I can decouple them. It's not necessary for Harfbuzz to have font discovery.


Bad 'table not found' error on nested subqueries

error: 'table not found: dcim_rackgroup, maybe you mean dcim_rackgroup?'  for query:
SELECT COUNT(*) FROM (
	SELECT (
		SELECT count(*)
		FROM (
			SELECT U0.`id`
			FROM `dcim_rack` U0
			INNER JOIN `dcim_rackgroup` U1
			ON (U0.`group_id` = U1.`id`)
			WHERE (
				U1.`lft` >= `dcim_rackgroup`.`lft` AND
				U1.`lft` <= `dcim_rackgroup`.`rght` AND
				U1.`tree_id` = `dcim_rackgroup`.`tree_id`
			)
		) _count
	) AS `rack_count`
	FROM `dcim_rackgroup`
	WHERE `dcim_rackgroup`.`id`
	IN ('418dd0dd47504bb190f354cf23ded6a6', 'd6d30bef4def4b66bcd180d4252eca7d', '34a74e488171481b96b222bf56a55bb9', '289e27c03cee4c299a3fa10517b54c52')
) subquery

schema:
CREATE TABLE `dcim_rackgroup` (
  `id` char(32) NOT NULL,
  `lft` int unsigned NOT NULL,
  `rght` int unsigned NOT NULL,
  `tree_id` int unsigned NOT NULL,
  `level` int unsigned NOT NULL,
  `parent_id` char(32),
  PRIMARY KEY (`id`),
  KEY `dcim_rackgroup_tree_id_9c2ad6f4` (`tree_id`),
  CONSTRAINT `dcim_rackgroup_parent_id_cc315105_fk_dcim_rackgroup_id` FOREIGN KEY (`parent_id`) REFERENCES `dcim_rackgroup` (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_0900_bin;

CREATE TABLE `dcim_rack` (
  `id` char(32) NOT NULL,
  `group_id` char(32),
  PRIMARY KEY (`id`),
  KEY `dcim_rack_group_id_44e90ea9` (`group_id`),
  CONSTRAINT `dcim_rack_group_id_44e90ea9_fk_dcim_rackgroup_id` FOREIGN KEY (`group_id`) REFERENCES `dcim_rackgroup` (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_0900_bin;

This appears to be a newly removed restriction in MySQL, starting in 8.0.14. From the MySQL Derived Table reference docs:

Prior to MySQL 8.0.14, a derived table cannot contain outer references. This is a MySQL restriction that is lifted in MySQL 8.0.14, not a restriction of the SQL standard.

Dolt currently matches the pre-8.0.14 behavior, where derived tables do not have visibility to any tables or expressions from outer scopes. Passing the scope information through should be fairly straightforward, but we'll also need to do some testing to make sure we understand the exact visibility rules so we can match MySQL's behavior, and then see if any changes are needed to execute the query.

Quick update on progress... I landed some alias changes yesterday that were also needed by this feature to help track subquery scopes better. I've got the visibility portion for this feature working, but I'm still debugging through some queries that aren't returning the same results as MySQL. I'll keep working on this one and expect to have this working by the end of this week.

I've got support for this new feature coded up in the linked pull request (https://github.com/dolthub/go-mysql-server/pull/1310) and ready for feedback from teammates. I'll update again as we progress towards releasing this.


DELIA-00000: gnome netmgr wifi part code changes

Reason for change:
Fix above issue:
Test Procedure:
Risks:
Priority: P1
Signed-off-by: Kumar Santhanam<EMAIL_ADDRESS>

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/4053/rdkcentral/rdkservices

Commit: 5bb7343bc9015ab2d1dd618c9729a764d210695b

Report detail: gist

Hi,
The new file doesn't have a header.  Where did it come from?  Blackduck finds parts of the file to be a close match to GPLv2-licensed code.  This is not generally permitted in an Apache-2.0 licensed repo.  However, if the file is completely standalone and doesn't link to other RDK code, it may be OK but the licensing files will need be adjusted - please let me know.
Martin Hughes, RDK CMF Compliance Team<EMAIL_ADDRESS>


resctl-bench: can't build PDF on debian

I installed the adobe fonts following https://gist.github.com/marcora/cb37b2432c072e8e9d77 but there's still an error on latest Debian:
$ ./resctl-bench --result=out.json format iocost-tune:pdf=output.pdf
convert-im6.q16: attempt to perform an operation not allowed by the security policy `PDF' @ error/constitute.c/IsCoderAuthorized/421.
[ERROR] Failed to format job[iocost-tune:-]: "\"convert\" \"-font\" \"Source-Code-Pro\" \"-pointsize\" \"7\" \"-density\" \"300\" \"text:/tmp/.tmp4hy4Mn/iocost-tune-cover.txt\" \"/tmp/.tmp4hy4Mn/iocost-tune-cover.pdf\"" (ExitStatus(ExitStatus(256))): are imagemagick and adobe-source-code-pro font available?
thread 'main' panicked at 'explicit panic', resctl-bench/src/main.rs:226:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

running with ghostscript 9.54.0

Can you please confirm that the following solves the issue?
https://stackoverflow.com/questions/52998331/imagemagick-security-policy-pdf-blocking-conversion

thanks, that looks exactly it. So the /etc/ImageMagick-6/policy.xml shipped with Debian contains:
...
  <!-- disable ghostscript format types -->
  <policy domain="coder" rights="none" pattern="PS" />
  <policy domain="coder" rights="none" pattern="PS2" />
  <policy domain="coder" rights="none" pattern="PS3" />
  <policy domain="coder" rights="none" pattern="EPS" />
  <policy domain="coder" rights="none" pattern="PDF" />
  <policy domain="coder" rights="none" pattern="XPS" />
</policymap>

changing the rights from none to read | write for all those policies sorted it.


regression: motion option to deactivate versioning has no effect

set the motion option in the settings to allow the deactivation of versioning
now edit a motion

result: regardless of the option to activate/deactivate versioning there is an option for a trivial change in the edit dialog that allows to deactivate versioning
So this is either a regression to OpenSlides 1.5.1 or the change is desired to offer turning off the versioning. If so the motion settings option should be removed.

I tested it with current master:

set config option to allow the deactivation of versioning
create new motion in complex workflow
set state of motion to 'permit'
now change the motion text (without checking 'trivial change') -> new version is created
permit the new versoin
change again the motion text (with checking 'trivial change') -> NO new version is created

Works as expected. Closed.

This bug is not fixed:

assure that "Allow to disable versioning" is NOT checked in the motion settings

so one can expect that when creating/editing motions one cannot use prevent the versioning. But now

edit a motion that is in permit state

result: there is still the selectable option "Trivial change" that prevents the versioning.
expected result: if "Allow to disable versioning" is not checked in the motion settings, the "Trivial change" checkbox in the editing window of motions is disabled.


Adding a Zoom + Close icon

Hi,
Neat plugin, Is there a way to add a (+) or zoom-in plugin to indicate to the user that the image can be zoomed / enlarged? Similarly what's the best way to add a close button in the enlarged view?
Thanks
Vilav

@fat any ideas on how to add this? Thanks!

For indicating to the user that the image can be enlarged, do you not see the "zoom" mouse cursor when you hover over it? Or do you mean something more obvious (I suppose the cursor is no good on mobile for instance).

If it's the latter, you should be able to do this via CSS. An ugly but working example is: http://codepen.io/spinningarrow/pen/xZJxOb.

@spinningarrow - i tried something simliar to your example and ran into another issue:
Additionally I added a click option to the new +icon: the zoom opens, but i doesn't close anymore and i don't understand why!?
$('body').on('click', '.icon', function (event) {
$(this).prev('[data-action="zoom"]').trigger("click");
});

is there any set to close the full screen display ?


Port VisNet from MTENN

Port in ViSNet model from MTENN. ViSNet is an equivariant message passing neural net for molecular property prediction.
Requires PyG >= 2.5.0

Fixed by #789


RFC: Provide option to run suggested commands after a typo

Is your feature request related to a problem? Please describe.
When running a ddev command you sometimes have typo and the current implementation provides suggestions. These are a good help already. However, you still need to fix the previous called command on your own.
Describe the solution you'd like
An interactive response:
Error: unknown command "laucnh" for "ddev"

Suggestions:
        [0] Cancel and fix on your own
        [1] launch

Choose your option:

Describe alternatives you've considered
Fix the command on your own - current behaviour which is ok but the new approach is more helpful / faster IMO.
Additional context
Can't think of anything.

Thanks for the suggestion. We use a spf13/cobra for all this, I wonder if there's an issue there. You might want to raise one if not, or contribute a PR there?

I will have a look there but I am not familiar with go. So I assume I won't find time to do both - dig into go and check their issues for such a feature request.

I don't plan to prioritize this, but happy to continue the conversation, and if somebody comes along that can put some effort into it it will be welcomed!


Add support for running Rails generators from the UI

Motivation
This PR adds a Rails class to our VS Code extension capable of running generators from the UI. This is useful on its own, but it is also necessary for other functionality that may want to invoke generators programatically.
Implementation
I split the work into two commits:

Add the Rails class with a generate and destroy command and associate both with commands
Expose a Ruby file operations command, where we can aggregate all of these under the same menu item. Then expose it in the explorer title and menu context

Automated Tests
Added tests.
Manual Tests

Start the extension on this branch
Open a Rails app
Click on the new button in the explorer title view
Use the generator
Verify that it executes the Rails generator properly

https://github.com/Shopify/ruby-lsp/assets/18742907/ed13018d-4edd-4202-9a4f-d2aece96cacc

Similar to what @st0012 mentioned, it is a bit awkward that we'll be showing these commands for non-Rails codebases, especially if we add more commands.
I'm looking at the available things we can do in the when of the command to conditionally display it, maybe we can come up with a creative solution:
https://code.visualstudio.com/api/references/when-clause-contexts

When generate a resource, it fails at the file opening stage for one file, I suspect the output parsing needs little adjustment:

cannot open file:///Users/andyw8/src/github.com/Shopify/ruby-lsp-rails/app/views/widgets. Detail: Unable to read file '/Users/andyw8/src/github.com/Shopify/ruby-lsp-rails/app/views/widgets' (Error: Unable to read file '/Users/andyw8/src/github.com/Shopify/ruby-lsp-rails/app/views/widgets' that is actually a directory)

I thought that if you try run a generator where some files already exist, Rails would interactively prompt to ask whether to overwrite each file, but I can't seem to trigger it. Maybe the behaviour changed in a more recent Rails version.

When generate a resource, it fails at the file opening stage for one file, I suspect the output parsing needs little adjustment:

Fixed it! Thanks for pointing that out.
Regarding not showing the button, that will become pretty complex given that you could have one workspace that is a Rails app and another one that isn't. There's no when clause that will tell you which workspace explorer view you are hovering over as far as I know.
So I propose we do something different. We always show the button, but then hide the Rails commands if the workspace is not a Rails app (which is significant more straight forward).
To start out, so that we don't show an empty quick pick to users, I can follow up with a PR to add a New Ruby class/module file action, which essentially creates a new file with the entity selected.
This is common in IDEs, so it would be a nice feature to have and then we can move forward with a much simpler approach that won't require trying to figure out which workspace folder is being hovered over.
Thoughts?


a NOT URGENT command execute delay

you make the EXECUTE function A LOT better than it was before. It's was really GREAT.
Much more stable and also less delay.
Yet, the delay of command execute still happens sometimes, it won't cause any other effects except the delay.
this might not even need to be fixed, just for reminding.
-----------------error output------------------
[1] basicRAT> pwd
Running pwd...
C:\Users\0\Desktop\PYN\basicRAT-master
pwd completed.
[1] basicRAT> ls VER
Running ls...
C:\Users\0\Desktop\PYN\basicRAT-master

This should be fixed now: https://github.com/vesche/basicRAT/commit/e4d241ff43192223af3acd4985b4d4ac537596c1


Some further memory leaks in dom.c

There's several functions in dom.c that are returning malloced values: domGetNodeName, domGetXPathKey, domGetASTKey, domGenNsPrefix, xmlNsUniqueKey.
These leaks could be solved in Raku, but in most cases, they return a small discrete set of possible discrete values and are maybe best solved by using xmlDict in the C bindings.
See also #77

Fixed in LibXML 0.7.8 release


undefined method `push' for some inexistent ObjectId

I don't know what exactly changed but I initially suspected it to be related to the fact that mongoid wasn't pegged to a specific version in my Gemfile:
gem 'mongoid', :github => 'mongoid/mongoid'
But that seems to be contradicted by the fact that I have the same blob for mongoid in my Gemfile.lock between a working branch and one that doesn't:
GIT
  remote: git://github.com/mongoid/mongoid.git
  revision: 88599cf64c3d6df9a1277178ced69cd45d30c2f1
  specs:
    mongoid (4.0.0)
      activemodel (~> 4.0.0)
      moped (~> 1.5)
      origin (~> 1.0)
      tzinfo (~> 0.3.22)

But now my updates fail with this error, and I checked the ObjectId (53ac812f6912f701ba000001) and it doesn't correspond to anything in mongo:
> db.orders.find({_id: ObjectId("53ac812f6912f701ba000001")})
>

What's going on here?
/home/panic/.rvm/gems/ruby-2.0.0-p247/gems/railties-4.0.0/lib/rails/commands/runner.rb:53:in `eval': undefined method `push' for "53ac812f6912f701ba000001":Moped::BSON::ObjectId (NoMethodError)
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/pushable.rb:54:in `each'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/pushable.rb:54:in `block (2 levels) in push'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable.rb:165:in `block in process_atomic_operations'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable.rb:163:in `each'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable.rb:163:in `process_atomic_operations'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/pushable.rb:51:in `block in push'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable.rb:141:in `prepare_atomic_operation'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/pushable.rb:50:in `push'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic/modifiers.rb:122:in `block in add_operation'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic/modifiers.rb:121:in `each'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic/modifiers.rb:121:in `add_operation'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic/modifiers.rb:87:in `block in set'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic/modifiers.rb:84:in `each_pair'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic/modifiers.rb:84:in `set'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic.rb:353:in `generate_atomic_updates'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic.rb:134:in `block in atomic_updates'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic.rb:132:in `each'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/atomic.rb:132:in `atomic_updates'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/updatable.rb:119:in `init_atomic_updates'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/updatable.rb:25:in `block in update_document'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/updatable.rb:143:in `block (2 levels) in prepare_update'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/gems/activesupport-4.0.0/lib/active_support/callbacks.rb:383:in `_run__1066962723__update__callbacks'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/gems/activesupport-4.0.0/lib/active_support/callbacks.rb:80:in `run_callbacks'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/interceptable.rb:132:in `run_callbacks'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/updatable.rb:142:in `block in prepare_update'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/gems/activesupport-4.0.0/lib/active_support/callbacks.rb:373:in `_run__1066962723__save__callbacks'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/gems/activesupport-4.0.0/lib/active_support/callbacks.rb:80:in `run_callbacks'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/interceptable.rb:132:in `run_callbacks'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/updatable.rb:141:in `prepare_update'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/updatable.rb:24:in `update_document'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/savable.rb:25:in `save'
        from /home/panic/.rvm/gems/ruby-2.0.0-p247/bundler/gems/mongoid-88599cf64c3d/lib/mongoid/persistable/updatable.rb:78:in `update'
        from /var/www/project/redacted/app/models/order.rb:207:in `update_attributes'

I am not sure how to simulate this.
Can you test this against latest master and report back if it is still a problem?
Also if you could try to isolate the problem in a unit test, would be helpful. Try using this as base https://github.com/mongoid/mongoid/blob/master/examples/mongoid_test.rb.
thanks


DOC: Data Pipeline for humans

Issue with current documentation:
Dear all
I have this pipeline
translation_cache = ToJSON(key=key, out_dir=Path("results/sabadel/translation"))
translation_prompt = Prompt.from_yaml(Path("prompts/translate.yml"))
translation_chain = (
    {
        "transcription": lambda data: format_transcription_for_prompt(
            data["transcription"]
        )
    }
    | translation_prompt.template
    | model
    | {
        "transcription": RunnableLambda(
            lambda res: translation_output_parser(res, transcription)
        )
    }
    | {"transcription": lambda x: translation_cache(x)}
)

translation_chain = (
    RunnableLambda(lambda data: {**data, "transcription": translation_cache.load()})
    if translation_cache.exists
    else translation_chain
)

# evaluation
evaluation_prompt = Prompt.from_yaml(Path("prompts/sabadell/evaluation.yml"))
evaluation_cache = ToJSON(key=key, out_dir=Path("results/sabadel/evaluation"))
evaluation_chain = (
    evaluation_prompt.template | model | evaluation_output_parser | evaluation_cache
)

evaluation_chain = (
    RunnableLambda(lambda data: {**data, "evaluations": evaluation_cache.load()})
    if evaluation_cache.exists
    else evaluation_chain
)
# retention
retention_prompt = Prompt.from_yaml(Path("prompts/sabadell/evaluation.retention.yml"))
retention_cache = ToJSON(key=key, out_dir=Path("results/sabadel/retention"))
retention_chain = (
    retention_prompt.template | model | retention_output_parser | retention_cache
)
retention_chain = RunnableLambda(
    lambda data: {
        **data,
        "retention": retention_cache.load()
        if retention_cache.exists
        else retention_chain(**data),
    }
)
# final chain
# chain = translation_chain | retention_chain
# print(translation_chain.invoke({"transcription": transcription}))
chain = translation_chain | evaluation_chain | retention_chain
print(chain.get_graph().print_ascii())
print(
    chain.invoke({"transcription": transcription, "retention_script": retention_script})
)

Now what I'd like to do is that evaluation_chain puts stuff into a key evaluations and pass along the original data dict + that key to retention_chain and retention_chain should put it's output into a retention data key and then pass along the original dict + all the outputs
how to do it?
Idea or request for content:
No response

🤖
Hello @FrancescoSaverioZuppichini,
Great to see you again! I hope you've been coding well since our last interaction. I'm currently digging into the pipeline you've posted and will get back to you with a detailed response shortly. Hang tight!

could be better, maybe you can write a new library that works better than this one ;)

You could try to use Runnable Context. It's a beta feature in langchain core. Not sure if documented already. Moving to a discussion since this is not an issue, but a question of how to achieve a particular goal with LCEL


Fix rpcv02 tests

https://github.com/NethermindEth/starknet.go/issues/217

Some tests are still failing for me on devnet:

TestCall on call_test.go
TestChainID on provider_test.go
TestDeclareTransaction on write_test.go

Also could we update the workflows?  In addition, testing on testnet is running but is failing, we could skip this step until we manage to fix them as well.
Would love to see a green status

All tests should pass on mainnet, testnet, devnet, and mock, when making requests to Juno


Melee attack type labels do not update on position change

@FrostRaptor there seems to be an unintended side effect of the way you disabel "attack" button of no melee attack is allowed from that hex. If there is no allowed melee attack from the hex that gets selected when you click on enemy, there is no way to change it.
Attachment file type: archive
UnselectableMelee.zip
7.59 MB
logs are from a next turn when i confirmed there there are actualy hexes with legal charge around them.
FrostRaptor01/08/2021
Hurm, interesting. Does it work if you hit esc to back out of the attack?
Harkonnen01/08/2021
no
esc puts you out of melee mode

UnselectableMelee.zip

UnselectableMelee.zip

Fixed in 0.6.4

Fixed in 0.6.4


Preprocessor doesn't do elif right

Expected behavior

Should process boolean conditions in elif correctly.
Code

$ cargo run -- -E
#define __WORDSIZE 64
#if 0
wrong1
#elif __WORDSIZE == 64
right
#else
wrong2
#endif
// output
wrong2

http://port70.net/~nsz/c/c11/n1570.html#6.10.1

Ok, the actual issue is that identifiers aren't replaced in #if expressions. It assumes all identifiers are undefined, but doesn't go through the process of macro replacement first.

Fixed in https://github.com/jyn514/rcc/commit/63d0ba3235d67210e21517ec4d91c97c1f38712e


Prevent maxCallbackGas from being 0 in NewIBCMiddleware for callbacks

Summary
The IBC middleware constructor function for callbacks should not allow the maxCallbackGas to be 0 as a defensive check

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

addressed in https://github.com/cosmos/ibc-go/pull/3939/commits/01483b1d2f51372c8a81e86f9046f38ef495323f


Integration Test - Release - 2018/11/21 11:09:50

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest1 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


MM-56540 - DetectRealtime function

Summary

Add a DetectRealtime function that gives us:

cleaned silence (zeroed out)
start and end indexes for the silence and speech segments

Note: I think this can be simplified, it's pretty rough right now.

Ticket

https://mattermost.atlassian.net/browse/MM-56540

@cpoile Meta comments, not urgent but two things we should plan for:

Add some basic CI
Move repository to company org

@cpoile Can we merge this?


Update nanoFramework.System.IO.FileSystem  assembly reference for changed SDCard

Description
Update M5stack project to support latest nanoFramework.System.IO.FileSystem with SDCard updates.
Motivation and Context
Build broken
How Has This Been Tested?
Builds locally
Types of changes

[x] Improvement (non-breaking change that improves a feature, code or algorithm)
[ ] Bug fix (non-breaking change which fixes an issue with code or algorithm)
[ ] New feature (non-breaking change which adds functionality to code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Config and build (change in the configuration and build system, has no impact on code or features)
[ ] Dependencies (update dependencies and changes associated, has no impact on code or features)
[ ] Unit Tests (add new Unit Test(s) or improved existing one(s), has no impact on code or features)
[ ] Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

[x] My code follows the code style of this project (only if there are changes in source code).
[ ] My changes require an update to the documentation (there are changes that require the docs website to be updated).
[ ] I have updated the documentation accordingly (the changes require an update on the docs in this repo).
[x] I have read the CONTRIBUTING document.
[x] I have tested everything locally and all new and existing tests passed (only if there are changes in source code).
[ ] I have added new tests to cover my changes.

Summary by CodeRabbit

New Features

Updated the nanoFramework.System.IO.FileSystem dependency version to 1.1.66 across multiple projects.

Bug Fixes

Resolved issues related to the previous version of the System.IO.FileSystem dependency.

@AdrianSoundy , you probably need to update the .nugetspec file to match the updated:
expecting '1.1.66' but found '1.1.54'.```

@AdrianSoundy

@networkfusion
Yes I saw that but the main problem now is the GitVersion also changed yesterday. Decided to update other PR first then can update this PR when branch updated

@networkfusion Yes I saw that but the main problem now is the GitVersion also changed yesterday. Decided to update other PR first then can update this PR when branch updated

Oh, I hate when that happens.

No need to complete this PR as another PR has resolved problem


wrong links in sitemap

Hi, I add the sitemap plugin in my gitbook with "hostname": "http://codenote.liuyid.in/". However, in the sitemap http://codenote.liuyid.in/sitemap.xml, all links are wrong with additional "null"s, like http://codenote.liuyid.in/nullREADME.html .

+1

@petrosliu seems to work with GitBook v3

@kyrylkov you are right. Thanks!

How to fix it for GitBook 2.x? I can't upgrade to 3.0 because it breaks too many things and has several bugs.


Splits analysis

Restructured the environment files
Added the split analysis notebook
Improved the readme with instructions for the workshop participants
Improved the auto-ml notebook

We will need a short end-to-end testing:

Delete your conda environments locally
Recreate all environments agaiin
Run each notebook in their corresponding environment

I tried running it on another computer and that resulted in conflict of incompatible packages

In the meantime the other computer decided it actually don't mind the conflict and went through second time

conda activate ml_eda or ml_data_validation or any other extra environment does not seem to work

conda activate ml_eda or ml_data_validation or any other extra environment does not seem to work "could not find conda environment: ml_data_validation"

Hmm... that is strange. So  you created the environments with conda env update -f <environment>.yaml and then the conda activate <environment> didn't work at all? Should we have a talk today and see the error together?

I just followed the README which did not include this step. Now I did the conda env update -f .yaml and it works, so I also added to the README. Checked EDA, works fine :)

I just followed the README which did not include this step. Now I did the conda env update -f .yaml and it works, so I also added to the README. Checked EDA, works fine :)

OK, I have added a change in the make file, and it will create all the environments in this step. I have added a script to add all environments to jupyter lab, so that we can just choose them from the drop-down. Coming in next push. After that I merge to main and send email to participants.


Enable R8 full mode

Related PR: #872
Default changed by the Android 8 plugin? https://developer.android.com/build/releases/gradle-plugin
https://r8.googlesource.com/r8/+/refs/heads/master/compatibility-faq.md#r8-full-mode

Related: https://github.com/square/retrofit/issues/3005#issuecomment-1523948650


Generate new api-docs

Bootstrap version is updated in
https://github.com/kubernetes-incubator/reference-docs/blob/88f6483855ac68ee6041cd8e90169f7f5268dc2b/gen-apidocs/generators/html.go#L604
Fixes #16542

/assign @zparnold

/retest

https://deploy-preview-16569--kubernetes-io-master-staging.netlify.com/docs/reference/generated/kubernetes-api/v1.16/#deployment-v1-apps
Tested - both buttons work now; accessed other pages and all functionality is working as expected.
Thanks @medmedchiheb for pushing this PR so quickly!

/lgtm

/approve


Caught exception: Error: spawn mkdir ENOENT

Hi @sergi
I followed your proposal and I tried with the latest version 1.5.2 of jsftp (without adding my code for  special characters with accents).
What I did :

npm clone of your project
pointed my ftp server on it
launched with npm test
here is the result :
...\JSFTP-1.5.2\jsftp>npm test

<EMAIL_ADDRESS>test JSFTP-1.5.2\jsftp
mocha -R spec -t 5000

child_process: customFds option is deprecated, use stdio instead.
Caught exception: Error: spawn mkdir ENOENT
...\JSFTP-1.5.2\jsftp\test\jsftp_test.js:49
server.stop();
^
TypeError: Cannot read property 'stop' of undefined
at process. (...\JSFTP-1.5.2\jsftp\test\jsftp_test.js:49:13)
at process.emit (events.js:129:20)
at process._fatalException (node.js:236:26)
npm ERR! Test failed.  See above for more details.

fixed the code
describe("jsftp test suite", function() {
process.on('uncaughtException', function(err) {
console.log('Caught exception: ' + err);
if(server) { <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< start of hack
server.stop();
}<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< end of hack
});

started npm test again

d:\MyProjects\SVN.BETV.V2\BETV.DEPLOY.WEBMEDIA.EMAILING.JSFTP-1.5.2\jsftp>npm te
st

<EMAIL_ADDRESS>test d:\MyProjects\SVN.BETV.V2\BETV.DEPLOY.WEBMEDIA.EMAILING.JSFTP
-1.5.2\jsftp
mocha -R spec -t 5000

child_process: customFds option is deprecated, use stdio instead.
Caught exception: Error: spawn mkdir ENOENT
jsftp test suite
V test initialize bad host (2255ms)
V test initialize
V test parseResponse with mark
V test parseResponse with no mark
V test send function
V test parseResponse with ignore code
V test invalid password
V test getFeatures
V test print working directory
V test switch CWD
V test switch to unexistent CWD
V test passive listing of current directory
V test passive listing of nonexisting directory
Caught exception: AssertionError: [object Object]
1) test ftp node stat
V test create and delete a directory
V test create and delete a directory containing a space
V test create and delete a file (99ms)
V test save a remote copy of a local file (65ms)
V test passing a dir instead of file path to put should callback with error
V test streaming put (52ms)
V test rename a file (73ms)
V test get a file
V test get a file and save it locally
V test get a big file stream (68ms)
V test put a big file stream (234ms)
V test put a big file stream fail
V test get fileList array
V test reconnect
V test attach event handlers: connect

Caught exception: AssertionError: Error: 503 Bad sequence of commands.
2) test PASV streaming: Copy file using piping
V Test that streaming GET (RETR) retrieves a file properly
V Test that streaming GET (RETR) fails when a file is not present
V Test that streaming PUT (STOR) stores a file properly
V Test that streaming PUT (STOR) fails when a file is not present
V Test that onConnect is called
Caught exception: AssertionError: false == true
3) Test for correct data on ls 1
V Test raw method with PWD
V Test raw method with HELP
V Test keep-alive with NOOP (5001ms)
- Test handling error on simultaneous PASV requests {#90}
V test set binary type
V test listing a folder containing special UTF characters
38 passing (13s)
1 pending
3 failing

jsftp test suite test ftp node stat:
Uncaught

Don't hesitate.

What FTP server are you using? Would it be possible to make an anonymous account for me to test in it?


5.0.2 T extends GenericType<infer InferredType> no longer resolves as true

Bug Report
In previous versions of TypeScript (including 5.0.0-beta), when working with Vue, component prop types could be inferred using the following ExtractProps type where T is a Vue component:
import type { ComponentOptions } from 'vue'

type ExtractProps<T> = T extends ComponentOptions<infer Props> ? Props : never

The returned type is now never.
🔎 Search Terms
infer, generic
🕗 Version & Regression Information
5.0.0-rc

This changed between versions 5.0.0-beta and 5.0.0-rc

💻 Code
import type { ComponentOptions } from 'vue'
import MyComponent from './MyComponent.vue'

type ExtractProps<T> = T extends ComponentOptions<infer Props> ? Props : never

type MyComponentProps = ExtractProps<typeof MyComponent> // never (but correctly returned Props in earlier versions of TS)

⏯ Playground Link
Working (4.9.5)
Broken (5.0.2)
🙁 Actual behavior
MyComponentProps is never
🙂 Expected behavior
MyComponentProps should be the inferred Props type

We'd need a self-contained repro (i.e. 3-50 lines, none of which import the entirety of Vue) in order to determine if this is an intentional change or not.

@markglattback Why did you close this? I'm investigating this as well, this broke TS support for Vue in storybook.

@kasperpeulen whilst the old approach was no longer working with TS 5.0.x, the following has helped:
I stumbled across a comment from the author of Volar recommending vue-component-type-helpers to extract component types. IFound that in our project it was dependent on having a d.ts file containing component types but otherwise works very well. In our case our d.ts file is generated by the unplugin-vue-components plugin.
We're also using Storybook 7 and have no issues with type inference using Meta<typeof MyComponent> or Story<typeof MyComponent>.

@markglattback Thanks for that! That is really useful.
I opened a PR in storybook to use those type helpers:
https://github.com/storybookjs/storybook/pull/22814
Weird that you don't have issues, we have multiple users report that it breaks with TS 5.0.


Enable app to be started automatically by Mix

Adds mod: {Bugsnag, []} to the application definition in mix.exs,
which means that Mix can start the app correctly, and automatically pick
up uncaught exceptions when use_logger: true is set.
I think this makes this more intuitive to use - just set the config in config/config.exs, add it to your applications list in mix.exs and off you go.

doesn't it need to be fully fledged supervisor / genserver? I was also wondering if it wouldn't better to actually create a supervisor + client combo instead of simple spawn, what do you think?

Good point @driv3r - I've updated to make Bugsnag start a top-level supervisor, which in turn supervises a Task.Supervisor, which executes sending the actual reports.

Great work @isaacseymour, can we please have this merged and released as a new version?


notebook_launcher used only 1 gpu even when num_proccesses=2

System Info
This is in kaggle.

- `Accelerate` version: 0.12.0
- Platform: Linux-5.15.65+-x86_64-with-debian-bullseye-sid
- Python version: 3.7.12
- Numpy version: 1.21.6
- PyTorch version (GPU?): 1.11.0 (True)
- `Accelerate` default config:
	- compute_environment: LOCAL_MACHINE
	- distributed_type: MULTI_GPU
	- mixed_precision: fp16
	- use_cpu: False
	- num_processes: 2
	- machine_rank: 0
	- num_machines: 1
	- main_process_ip: None
	- main_process_port: None
	- main_training_function: main
	- deepspeed_config: {}
	- fsdp_config: {}
	- downcast_bf16: False

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
from accelerate import notebook_launcher
notebook_launcher(trainer, (df, "context","response", model_params, "outputs"), num_processes=2,mixed_precision="fp16")

Expected behavior
Expected the training to launch on both the gpus but num_processes is 1 even after setting it to 2

@deepsarda you need to upgrade your accelerate version. The Kaggle launcher wasn't introduced until version v0.14.0

Thanks!


Fix possible deadlock when queries and delete series run concurrently

Required for all non-trivial PRs

[x] Rebased/mergable
[x] Tests pass
[ ] CHANGELOG.md updated

This locks showed up in a deadlocked systems running queries and
delete series across a large dataset.  Queries should not need to
lock the tsdb.Store for writes so this change switch the Lock to an RLock.
cc @benbjohnson

:+1:

LGTM 👍


unselect item in single selection mode

Is there any way to unselect item in single selection mode? Maybe a close tag , like multi selection mode?
Currently, I cannot clear the input once any option is selected.

Hi @tzjoke ,
You have to re-click on the same item for deselecting the item.


chore: add PRs wellcome to README.md

Thanks! I'll review these other open PRs during the weekend

I like to pull request in this repository.
You have the prompt reply.


fix(internal calls): Applying paid rate limits to internal calls

:tada: This PR is included in version 1.18.2 :tada:
Your semantic-release bot :package::rocket:


Update wrapper.conf (distribution/kernel/carbon-home/bin/yajsw/wrappe…

Purpose

The issue
wso2/product-is#6165

Goals

To fix the issue
wso2/product-is#6165

Test environment

JDK versions JDK 1.8, operating systems WIndows,

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Bug Squashing

resolved bug where fields were not reset after creating a new patient
Centralised New Visit Button

lgtm