Update go.mod to update packages and remove blumix-go dependency

This PR updates relevant packages in go.mod to latest versions and removes blumix-go dependency.
TODO: github.com/golang/mock is public archive now, will replace with respective package as separate PR if needed
Fixes: https://github.com/openshift/machine-api-provider-powervs/issues/89

Needs jira/valid-reference label. @Karthik-K-N you need to create a jira issue and tag,

Needs jira/valid-reference label. @Karthik-K-N you need to create a jira issue and tag,

yeah, I will create and update it. Just doing final check on cluster.

I have tested the changes on cluster and successfully able to create and delete new machine.

For reference steps followed for testing

Scale down cluster-version-operator

oc scale --replicas 0 -n openshift-cluster-version deployments/cluster-version-operator

Scale down machine-api-operator

oc scale deployment/machine-api-operator -n openshift-machine-api --replicas=0

Edit the machine-api-controllers deployment to remove machine-controller container (since we want to run locally)

 oc -n openshift-machine-api edit  deploy machine-api-controllers

Build and run the mapi PowerVS controller locally

$ pwd
machine-api-provider-powervs

$ make build

$ ./bin/machine-controller-manager

Scale the machineset up and observe the logs in console where mapi powervs is trigger

$ oc -n openshift-machine-api scale replicas 4 machineset.m ocp1-418nig-wdc06-znrzv-worker

$ oc -n openshift-machine-api scale replicas 3 machineset.m ocp1-418nig-wdc06-znrzv-worker

Scale back machine-api-operator anc cluster-version operator


Set SWIFT_VERSION = 3.0

This is a breaking change for Swift 2.x users in Xcode, but otherwise changes no functionality.

👍  would you mind creating a compatibility matrix similar to the one on Argo so users can know which tag to use ... as part of this PR or the next would be great.

Good idea! I was thinking the same thing.


Pushing initial big repo error

I get about 90% complete pushing my repo (around 8gb in size) and eventually get this error repeated:
Check that you have proper access to the repository
Authorization error: https://github-cloud.s3.amazonaws.com/alambic/media/1147554
35/6a/c1/6ac1d55906de0237337b1b703e767fef7fe94bdba6b269db239f426a0189b8f1?actor_
id=1283957

The good news is that if you retry the push, it will pick up where it left off again. The git lfs pre-push command will check every file, and skip the upload if it's already on the server.
The bad news is that it looks like you made a request with an expired S3 token. The strange thing is that the client should be making the S3 request almost immediately after getting the token. So I'm not sure how it could be expired. Could you try another push with more detailed logging, and post the deets to a Gist or pastebin?
$ GIT_CURL_VERBOSE=1 git push origin master

Make sure to scrub any Authorization header lines.

I'll give this a try, but it never appears to start where it left off. I tried about 5 times before posting :(

There is quite a lot to paste, what is worth showing?
For example this is in the last couple failed attempts (near the end before the error spam I pasted above):
RequestTimeTooSkewedThe difference between the request time and the current time is too large.20150710T164330Z2015-07-10T16:59:13Z900000788C471F0F7E12EDI3ojd7j1ayq09ftEW8o6ASfgXunYVYfiXlTkfg7cy298ub7peGFLx4vNYzCjR0/8>
This is after about 55mins pushing.

@topherredden That's exactly what I'm looking for. I'd like to match that S3 request up with the one that generated the s3 token. The S3 error should be coming from a request like:
PUT https://github-cloud.s3.amazonaws.com/alambic/media/1147554

35/{OID}?actor_
id=1283957
That 64-char OID value is important. There should be a corresponding request that gets the s3 token. It should look something like:
POST https://github.com/{user}/{repo}.git/info/lfs/objects

The OID value from the erroring S3 request should be in the JSON request body of a matching POST. I'd like to see what time both requests were made.
Since this is a GitHub.com issue, and not a Git LFS issue necessarily, you're welcome to send the log to<EMAIL_ADDRESS>I'll be the one helping you there too :)
Also, what does git lfs env say? If it doesn't have the Git LFS version, what does git lfs version say too?

$ git lfs env
git-lfs/0.5.2 (GitHub; windows amd64; go 1.3)
git version 1.9.5.msysgit.1

While trying to find the OID value from an error, I noticed this as well:
API rate limit exceeded for topherredden.

I found this PUT
Error for PUT https://github-cloud.s3.amazonaws.com/alambic/media/114755435/2b/f4/2bf486172bc0329a6bbc5a7d1649f78638ec380d43d0a51e243f23b8d82d68ea?actor_id=1283957

Put https://github-cloud.s3.amazonaws.com/alambic/media/114755435/2b/f4/2bf486172bc0329a6bbc5a7d1649f78638ec380d43d0a51e243f23b8d82d68ea?actor_id=1283957: net/http: TLS handshake timeout

I'm having trouble finding a POST with "info/lfs/objects" on the end.

I've e-mailed you the log :)

Okay, I found the problem. The token was generated at 16:18:15 GMT. The token was used at 16:33:41 GMT, which is beyond the token timeout. I didn't expect the time between those requests to differ so much. We're working on new, better API, and a better, more reliable client for Git LFS v0.6.0. But until then, I'm going to extend the timeout to compensate for this issue.
If you want to confirm this in the log you sent, the token was generated with this response header: X-Github-Request-Id: 97E2A007:7520:D7A987F:559FF046. The same token was used with this response header: X-Amz-Id-2: 83EDX4cqbOiAs0KAoKlKjsMCEQFciM97a7yMC0wpxVG/T9BkW38nyZcFWAKjpFSL.

API rate limit exceeded for topherredden.

I'm working this issue now, actually :)
Thanks for going through this with me, and for checking out the Git LFS Early Access!

Hello, I am having the same (or a similar) problem. I have a lot of logs that report an EOF or a forcibly closed connection, but the first and oldest error is net/http: TLS handshake timeout.
I am pushing a 12.4 GB initial repository. Could it be, that the increased timeout counter was not enough for larger repositories?

Nope, totally different timeout. I think a TLS handshake indicates some bigger problem communicating between your computer and the host of the Git LFS server.

I saw that the put request expired: I am not sure what that means in this context. It processed all files - but in the end did not complete. If you want to I can open an issue and provide the logs. I switched from a https connection to ssh, but that did not mitigate the problem. Now I am pushing the repo in pieces and it seems to work - but that is an undesirable workflow. But since I am under a time constrain to provide the repository I will continue like this.

Are you getting the TLS handshake timeout with the S3 urls that GitHub's LFS API returns? I saw another case of that this morning.

Has anyne looked into this problem at all since July?

It's a network issue, so there's not a ton we can do at the client level. However, Git LFS v1.1.1 bumped the http client timeouts, with git config settings for increasing them even further: https://github.com/github/git-lfs/blob/05aa87dbf5c6c7feced04bb43c907cc3ba4e1f76/docs/man/git-lfs-config.5.ronn#L35-L49

Hi technoweenie,
When trying to push a large file using LFS tracking I got a similar error:
$ git push -u origin master
Git LFS: (0 of 1 files) 113.05 MB / 580.15 MB
LFS: Client error: https://github-cloud.s3.amazonaws.com/alambic/media/134753364/c7/73/c773a6287dec8c0feeb557e8af478b1e7cb2e30f105812718a282f7aaa24d4c6?actor_id=16270718
error: failed to push some refs to 'https://github.com/hazzaldo/Collect_All_Cubes_3D_Game_-iOS-.git'

Sorry if my query is a duplicate but I'm not sure if it was the same as the earlier problems posted on this thread.


Add columns/rows for performance measures on demand

The user wants to add further columns on demand:

As row

False positive rate (show as default)

As column

False negative rate (show as default)
Accuracy (show as default)
Class size (show as default when not homogenous)
Precision
Recall
F1
ROC curve

We should sketch the whole interaction and interface first before implementing this.
Also should it be possible to reorder the columns?

not relevant anymore due to new MetricsView


Change method to GET for redirect after POST request.

Codecov Report

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

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   65.17%
=========================================
  Files             ?        2
  Lines             ?      402
  Branches          ?        0
=========================================
  Hits              ?      262
  Misses            ?      140
  Partials          ?        0

Continue to review full report at Codecov.

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


Code seems confused about architecture

The code has a class that exposes methods like Dor::WorkflowService.archive_workflow(), but then those methods require certain @@clas_vars to be initialized via a Dor::WorkflowService.configure method.  But each place these top level vars are referenced still has to check "did you call configure?"  It seems like what we really want is a regular object that is initialized by constructor every time, so that we only have to check once.  And then if you happen to have two different pieces of code touch Dor::WorkflowService, they don't collide with competing global configurations.
Is there any reason for the way it is?  Anybody want to vouch for it?

Everything is a class method now. I don't see why it couldn't be a singleton pattern that initialized the config stuff on first call.

It certainly could employ that pattern, but I don't see any reason it should.  I see several reasons it shouldn't.  The @@module_vars are mimicking attributes.  That seems like bad behavior.  Modules are not intended to have internal states.  Modules are typically expected to be included or extended, which we can't do here without abandoning the singleton-ness of the @@vars.  The combination of methods and states is the proper domain of a class.
Basically I feel like this was copied out of dor-services and overly influenced by the "everything is a module" approach used there.  Maybe I will have time to refactor it for a major version bump, but I wonder what else is using this besides Argo and Hydrus.

@cbeer reports sdr-preservation-core and EEMS are consumers.

Scrapped configure() for regular class initialize() in ongoing work in 2.x branch.


Add testimonylist route, "email to" checkboxes set up, show city and current committee on viewbills

I added the logic to use the committee chair email which should be populated soon by the scraper.

Sounds good.  I will merge and make the updates in a subsequent PR.


Create Producer and Consumer failed

Describe the bug
The consumer subscribes multi topics, and  at the same time  create the correspond producers
to produce messages, but I got error , and the errors repeated if I restart the server.
The client received error like this:
"server error: UnknownError: org.apache.pulsar.broker.service.BrokerServiceException$ServiceUnitNotReadyException: Namespace bundle for topic (persistent://mytenants/payCenter/consumeSuccess) not served by this instance. Please redo the lookup. Request is denied: namespace=mytenants/payCenter"
The server side error log:
Namespace bundle for topic (persistent://mytenants/payCenter/consumeSuccess) not served by this instance. Please redo the lookup. Request is denied: namespace=mytenants/payCenter
java.util.concurrent.CompletionException: java.lang.RuntimeException: org.apache.pulsar.broker.service.BrokerServiceException$ServiceUnitNotReadyException: Namespace bundle for topic (persistent://mytenants/payCenter/consumeSuccess) not served by this instance. Please redo the lookup. Request is denied: namespace=mytenants/payCenter
I surppose the error was caused by the concurrent ???

Thanks @zxbfeng. This looks like a concurrent issue.  Would you like to share your code example for reproducing this ?

yes, I used the pure go client, and in order to adapt to the inner load balance system I wrappered it. the wrapper file as below.
pulsarwrapper.go.txt
The Type "SConsumer and SProducer" is the export type, the use of the this type is in a function,like this:
func  consumeMessage() {
consumer := plusarwrapper.Consuemer(options)  //this function will create a new consumer ervery time
msg, err := consumer.Receive(ctx)
.........
}
and the producer is used like the consumer:
func produceMessage() {
producer: = pulsarwrapper.Producer(xxxx)
producer.Send(msg)
}
PS: in the wrapper, the client will be a global, but the producer and consumer will always create a new one when call function "produceMessage and consumeMessage", so I thought this may casuse the problem, but the error not must appear!!!

I have forget the defer producer.Close() and  defer consumer.Close().  These sentences have wrote in the real code

Maybe the error was caused by "namespace bundle" ?

We also encounter the same error, but when a broker's server is restarted.
We are deploying via k8s, and we have k8s with automatic node configuration lift.
We are load balancing the connection via plusar-proxy.

Close as stale.
Please open a new issue if it's still relevant on maintained versions (ATM it's >= 2.10) with a reproduce.


Create AMAU.txt

Adding my University

URL: https://www.ama.edu.ph
URL to IT related course : https://www.ama.edu.ph/college-of-computer-studies/
Student domain : ama.edu.ph
Staff and Proffesors : amaes.edu.ph
notes: They have a lot of branches across the Philippines. branches are called AMA Computer college and AMA Computer University is the main campus where I'm currently enrolled in as a computer science student

corrected domain from AMAU.edu.ph to ama.edu.ph


localStorage is not defined.

I guess this is due to lack of localStorage support in jsdom.
Starting` crawling http://localhost:59817/ Failed to retrieve initialize state from localStorage: ReferenceError: localStorage is not defined     at http://localhost:59817/static/js/main.923fa5f4.js:24:24448     at http://localhost:59817/static/js/main.923fa5f4.js:24:26074     at o (http://localhost:59817/static/js/main.923fa5f4.js:13:23313)     at Object.<anonymous> (http://localhost:59817/static/js/main.923fa5f4.js:14:24180)     at t (http://localhost:59817/static/js/main.923fa5f4.js:1:107)     at Object.<anonymous> (http://localhost:59817/static/js/main.923fa5f4.js:1:505)     at t (http://localhost:59817/static/js/main.923fa5f4.js:1:107)     at http://localhost:59817/static/js/main.923fa5f4.js:1:195     at http://localhost:59817/static/js/main.923fa5f4.js:1:200     at ContextifyScript.Script.runInContext (vm.js:32:29) Unable to persist state to localStorage: ReferenceError: localStorage is not defined     at Array.<anonymous> (http://localhost:59817/static/js/main.923fa5f4.js:24:24646)     at d (http://localhost:59817/static/js/main.923fa5f4.js:13:22744)     at http://localhost:59817/static/js/main.923fa5f4.js:24:306     at Object.dispatch (http://localhost:59817/static/js/main.923fa5f4.js:24:25796)     at t.r.handleLocationChange (http://localhost:59817/static/js/main.923fa5f4.js:23:31542)     at t.componentWillMount (http://localhost:59817/static/js/main.923fa5f4.js:23:31796)     at c.performInitialMount (http://localhost:59817/static/js/main.923fa5f4.js:21:17205)     at c.mountComponent (http://localhost:59817/static/js/main.923fa5f4.js:21:16373)     at Object.mountComponent (http://localhost:59817/static/js/main.923fa5f4.js:3:7565)     at c.performInitialMount (http://localhost:59817/static/js/main.923fa5f4.js:21:17487) ✏   Saving / as /index.html

I have the same trouble too.

:(

I have same problem... How to solve this kind of issues?

You should not use localStorage instance in the component's render or any method that is evaluated in the server. Move localStorage calls in the componentWillMount method for example. I've solved my problem this way.

Thanks @arefaslani I will to try it.

@Danjavia, did @arefaslani's strategy work for you? I moved my localStorages inside componentWillMount and it still throws the same error, localStorage is not defined. I also tried to use componentDidMount.

I have the same issue.
As far as I see it the react-snapshot just doesn't have global localStorage, so I'd tried to wrap localStorage and it worked for me.

Did you export this globally? Or inside the component?

Try this ith globals :
"globals": {
"window": true,
"browser": true,
"localStorage":true,
}

Use getDerivateStateFromProps, and place the localStorage in there, that worked for me.

I use localstorage in componentDidMount function,not construction,and solved the error

I had the same error, caused by the credentials verification library. I worked this around this by creating a helper function:
export const storage = () => {
  if (!typeof window.localStorage === "undefined") return window.localStorage;
  else if (!typeof localStorage === "undefined") return localStorage;
  else return false;
};

and checking the availability of localStorage before each call to methods using it:
- verifyCredentials(store);
+ storage() && verifyCredentials(store);

I solved it by adding local-storage polyfill for development dependency
npm i -D localstorage-polyfill
OR
yarn add -D localstorage-polyfill
In your App's index.js file add following:
import 'localstorage-polyfill'
This solved error for me.

I managed to get this working by extending comments earlier and constructing aalocalStorageWraapper.
const localStorageMock = function () {
var store = {};
return {
getItem: function (key) {
return store[key];
},
setItem: function (key, value) {
store[key] = value.toString();
},
clear: function () {
store = {};
},
removeItem: function (key) {
delete store[key];
},
};
};
const storage = () => {
if (!typeof window.localStorage === "undefined") return window.localStorage;
else if (!typeof localStorage === "undefined") return localStorage;
else return localStorageMock();
};
module.exports = { storage };

I had the same error, caused by the credentials verification library. I worked this around this by creating a helper function:
export const storage = () => {
  if (!typeof window.localStorage === "undefined") return window.localStorage;
  else if (!typeof localStorage === "undefined") return localStorage;
  else return false;
};

and checking the availability of localStorage before each call to methods using it:
- verifyCredentials(store);
+ storage() && verifyCredentials(store);

I solved it by adding local-storage polyfill for development dependency
npm i -D localstorage-polyfill
OR
yarn add -D localstorage-polyfill
In your App's index.js file add following:
import 'localstorage-polyfill'
This solved error for me.

I keep getting this error when I import
`TypeError: Cannot set property localStorage of # which has only a getter
(anonymous function)
node_modules/localstorage-polyfill/localStorage.js:39
36 | }
37 | const instance = new LocalStorage()
38 |

39 | global.localStorage = new Proxy(instance, {
40 |   set: function (obj, prop, value) {
41 |     if (LocalStorage.prototype.hasOwnProperty(prop)) {
42 |       instance[prop] = value`


Artisan Skill by Tedious

Describe the solution you'd like
https://www.youtube.com/watch?v=6MdnwG5H8a4
A youtuber created the concept of the artisan skill, + wrote the majority of code required. Would love to see it implemented as an extension
Describe alternatives you've considered
Continuing to use his spreadsheet works, but automatization would be nice.
Additional context
https://docs.google.com/spreadsheets/d/1BoAGtWsrN-zcL2k1yTKCPyQxAuTWFYKAxrauu5dv0jo/edit?usp=sharing

This seems out of the scope of what RuneLite is, therefore closing this issue. The creator has already provided a way of doing this yourself - and it seems fitting to use that instead.


Error while running starter-kit

I cloned the repo and followed the instructions but i got this weird error. has anyone encountered this before?

I had to run the app on a real device in order to test VR functionalities. For more infos : https://github.com/ViroCommunity/viro/blob/main/readmes/INSTALL.md#troubleshooting


[Timestampable] Property subset update (field update)

I'm trying to migrate from Gedmo, but there is no alternative to this behavior:
/**
 * @var \DateTime $contentChanged
 *
 * @ORM\Column(name="content_changed", type="datetime", nullable=true)
 * @Gedmo\Timestampable(on="change", field={"title", "body"})
 */
private $contentChanged;

Closing as answered.


Route Localizer ( Area, Controller and Action Names )

Is this a Bug or Feature request?:
Feature request
Description of the problem:
As ASP Net Core already implements StringLocalizer and ViewLocalizer, it would be great if it implemented "RouteLocalizer" so that an application could be fully localizable.
Example:
[ControllerNameLocalizer("Resource.Key")]
[ActionNameLocalizer("Resource.Key")]
public class HomeController : Controller
    {
        [ActionNameLocalizer("About")]
        public IActionResult About()
        {
            return View();
        }
    }

localhost:5000/Home/About ( en-US )
localhost:5000/Home/Sobre ( pt-BR )

@mkArtakMSFT any activity on that? Or anyone could provide a workaround to support route localization?

@caioliberali @fabercs hopefully I find time to implement this and write a blog post about it ..

Thanks for contacting us. We're closing this issue as there was no community involvement here for quite a while now.


Flow Format of HOSNerf dataset

Thank you for releasing the code!
I'd like to train HOSNerf on the released dataset but I found no description about flow file format.
What is the correct flow file format & file name generated by RAFT?

Hi, the file format is the default config of RAFT. The file name is %5d_fwd.npz or %5d_bwd.npz, e.g., 00001_fwd.npz. The flow files are large so I did not upload them. Hope that clarifies!

Hi, thanks alot for your feedback.
I tried generating the RAFT flow files using raft demo code (I made few modifications to save it in the file names your code expects).
However, when I try to load the flow files in your step 2, it fails to load the flow files, the code tries to load "flow" and "mask" parameters while RAFT output doesn't have this information. The code fails on:
bwd_flow, bwd_mask = bwd_data['flow'], bwd_data['mask']
Can you please explain how to export these parameters? (a code example would be great!).
Thanks,
Ofer
here'y my RAFT flow generation code:
    flow_prev = None
    frame = 0
    images = sorted(images)
    for imfile1, imfile2 in zip(images[:-1], images[1:]):
        image1 = load_image(imfile1)
        image2 = load_image(imfile2)

        padder = InputPadder(image1.shape)
        image1, image2 = padder.pad(image1, image2)

        flow_low, flow_pr = model(image1, image2, iters=iters, flow_init=flow_prev, test_mode=True)
        flow = padder.unpad(flow_pr[0]).permute(1, 2, 0).cpu().numpy()

        #if warm_start:
        #    flow_prev = forward_interpolate(flow_low[0])[None].cuda()

        output_file = os.path.join(output_dir, '%05d_bwd.npz' % (frame + 1))
        frame_utils.writeFlow(output_file, flow)

        frame += 1

the mask may be the optical forward-backward consistency mask. And the flow should be backward flow, that is, flow from image2 to image1

Hi, thanks again.
Just to make sure I understand correctly - you mean that I can use the "up_mask" from raft code (code below)?
Thanks.
with autocast(enabled=self.args.mixed_precision):
net, up_mask, delta_flow = self.update_block(net, inp, corr, flow)

No, the optical forward-backward consistency mask is computed from forward flow(from image1 to image2) and backward flow(from image2 to image1). Such mask can filter pixels with large prediction error.
unimatch code is an example to compute such a mask


Compile with MKL in conda-build

Problem:
Without -DBLAS=MKL, conda-build won't include MKL library into Caffe2 build. And the BLAS performance is bad on CPU.
Solution:
Explicitly add the flag. Add mkl and mkl-include as dependencies.
@ezyang @Yangqing

LGTM. (in chat, I mentioned that it was surprising that mkl-include was needed for the build, since PyTorch doesn't seem to need it. Shrug.)

Let me check this with Pytorch team.

Chatted offline and it's because PyTorch uses explicit BLAS interface only, while C2 MKL mode also uses MKLDNN and a bunch of others. In any case, thanks for the fix! :)

@ezyang   Any ideas to support caffe2 GPU version conda-build recipe ? Thanks !

It's supported; at least, I have successfully used an edited version of the conda recipe to build caffe2 with GPU support.


Revert 'Drop Python 3.9' #287

This reverts #287 - see discussion in https://github.com/sgkit-dev/vcztools/pull/105

coverage: 98.858% (-0.04%) from 98.895%
when pulling 1a3e6be2786a5f05f969aa59d4fb5ab77f0eb91e on tomwhite:revert-drop-python-3.9
into c7850fc410d4675e40781762f9b93fb34cfe7aa3 on sgkit-dev:main.


Fix typo in README.md

Merry Christmas @christianparpart and love your streams on twitch keep up the good work man.


Add a Gitter chat badge to README.md

chrisehlen/knex now has a Chat Room on Gitter
@chrisehlen has just created a chat room. You can visit it here: https://gitter.im/chrisehlen/knex.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

I already added it.


Add -E option (alias for --dotenv-filename)

I found that I use few env files to manage the same recipes between few environments. I do this very often and it's like 'ansible inventory' to me.
Typing --dotenv-filename each time is really tiresome. Can we use -E as shorthand for it?

Thanks for the PR! I definitely agree that we should have a flag for this. But I'm not sure if it should be --dotenv-path or --dotenv-filename --dotenv-path lets you load an environment file anywhere on disk, whereas --dotenv-filename sets a different filename which just searches for in the usual places. Which do you think is more useful?

Oh. Wow. This is, actually, very unexpected to me. I thought that --dotenv-path is changing path, and --dotenv-filename allows to specify the path to the file, including absolute path.
... I checked, it works as expected. just --dotenv-filename=/path/to/file uses /path/to/file correctly. (I was afraid of having something non-working here).
In my use-case, we use files (e.g. --dotenv-filename). Maybe some people use directories too, and it worth adding short option for it too.
Actually, I already use it for a week with hand-build patched version with -E and it really raises ergonomics.
(I'll look on failed linter now).

Sorry, I can't reproduce error. ./bin/generate-completions returns on errors on my machine, and I can't understand what is failing in CI.

Oh. Wow. This is, actually, very unexpected to me. I thought that --dotenv-path is changing path, and --dotenv-filename allows to specify the path to the file, including absolute path.
... I checked, it works as expected. just --dotenv-filename=/path/to/file uses /path/to/file correctly. (I was afraid of having something non-working here).

This works, but for a reason you might not expect.
--dotenv-path specifies a single path to search for a dotenv file, which may be relative or absolute. If just doesn't find that path, it throws an error.
--dotenv-filename specifies a path to search for a dotenv file, which, again, maybe relative or absolute. While searching for a .env file, just will go through the ancestors of the current working directory (., ../, ../../, etc), and join the path with those directories, and see if there's a env file there.
Now, the reason that --dotenv-filename worked with --dotenv-filename /path/to/file is because when a path is joined with an absolute path, the result is the absolute path. ../ joined with /path/to/file is just /path/to/file.
The place where you would see a difference is when using a relative path with both, and where the env file is in a parent directory. Here's a somewhat contrived example:
$ tree
test
├─ dir
│  └─ justfile
└─ foo
$ cat foo
FOO=bar
$ cat dir/justfile
default:
  echo $FOO
$ cd dir
$ just --dotenv-filename foo
echo $FOO
bar
$ just --dotenv-path foo
error: Failed to load environment file: No such file or directory (os error 2)

In this case, just --dotenv-filename foo works, because just first looks in ./foo, then ../foo, and finds the environment file.
just --dotenv-path foo does not work, because just first looks in ./foo, does not find the file, and then gives up.
If you're using --dotenv-filename with an absolute path, --dotenv-path will work too.
One important difference is that --dotenv-path will complain if the file doesn't exist, whereas --dotenv-filename won't. I suspect that the behavior of --dotenv-path is probably the one that should be the default, since if a user gives --dotenv-filename with a relative path which doesn't exist, then just won't complain if it doesn't find a env file, which might be confusing.

Yes, I've noticed the odd behavior of --dotenv-filename. May be, it worth additional option, like --dotenv-file, which will accept only the normal path (e.g. local to the current workdir or absolute), and which WILL complain if this file is missing? In this case '-E' may go into that option.
I also wonder, if it worth an environment variable.

Yes, I've noticed the odd behavior of --dotenv-filename when file is missing. May be, it worth an additional option, like --dotenv-file, which will accept only the normal path (e.g. local to the current workdir or absolute), and which WILL complain if this file is missing?

I might be misunderstanding, but isn't this what --dotenv-path does?

F....
Okay, I got it. It's misunderstanding from my side. I thought --dotenv-path is the path to the directory where to search for .env file.
Here the help:
        --dotenv-path <DOTENV-PATH>                Load environment file at <DOTENV-PATH> instead of searching for one

The problem is with 'at path'. This sentence (as far as I got it) means 'load file' (located) at DOTENV-PATH.
I propose to move '-E' to --dotenv-path, and update help for it to be
``
--dotenv-path                 Load  as environment file  instead of searching for one

Is it okay?

And sorry for misunderstanding and commotion.

I've updated my branch, but github is showing old content. Really, really odd. The actual diff looks like this: https://github.com/amarao/just/pull/2/files

Sorry for nagging, @casey, can you look at it? I use forked/selfcompiled version with '-E' option for convenience, but I very much to prefer to stay with upstream.

LGTM!


Extension does not load with coc.nvim / coc-snippets

[coc.nvim] extension "es7-react-js-snippets" doesn't contain main file /Users/lynx/dotfiles/vim/.config/coc/extensions/node_modules/es7-react-js-sni
ppets/out/index.js.

Probably an issue related to this line:
https://github.com/dsznajder/vscode-es7-javascript-react-snippets/blob/cb88c055f412078ab66c1b1613cec3c7a896e45f/package.json#L28

Why? This is output after ts transpilation.

@dsznajder I can confirm I have this same issue. (Using Neovim/coc.nvim) Plugin was working great until a few days ago. Would love to see this resolved! 🙂

I will investigate this one :)

@dsznajder removing the line mentioned in package.json makes the snippets load for me again. Dunno how if affects the build process though. Just letting you know 👍

@impulse Thank you for checking. I need to try it on my own if it works properly in VSCode too.

For people who are having this problem with coc.nvim, just access the folder where the plugin is installed (usually in ~/.config/nvim/plugged/vscode-es7-javascript-react-snippets when installed with vim-plug) and run the following command:

With Yarn

$ yarn compile

With NPM

$ npm run compile

it will compile the package and create the file where it is expected (./out/index.js)

Can we add a "prepare":"npm-run-all compile" to the package.json scripts section so that this happens automatically for vim-plug users? See https://github.com/neoclide/coc.nvim/blob/master/package.json#L16 for reference.

@esetnik I don't think that's is really necessary, you can make the process of compiling the package an automatic way if you use a package manager (like vim-plug) to install this snippet. Just add a new step into your vim/neovim when going to install the package, eg:
Plug 'dsznajder/vscode-es7-javascript-react-snippets', {
      \ 'do': 'yarn install --frozen-lockfile && yarn compile' }

Ok then can we include that in the readme so vim-plug users know what to do. Most other vim packages I’ve used are configured to do this automatically if they require transpilation from source.

@esetnik I don't think that's is really necessary, you can make the process of compiling the package an automatic way if you use a package manager (like vim-plug) to install this snippet. Just add a new step into your vim/neovim when going to install the package, eg:
Plug 'dsznajder/vscode-es7-javascript-react-snippets', {
      \ 'do': 'yarn install --frozen-lockfile && yarn compile' }

@pauloedurezende Thanks! That worked like a charm. 🙂👍🏻

Would you like to add this to documentation somewhere in README? :)

@dsznajder I will open a pull request with the installation information 😄


DrawSphereWires() is missing.

DrawSphereWires is missing from the wrapped methods, DrawSphereEx and DrawSphere are present though!

my bad :)
i'm busy with rebranding the binding and adding examples though, you can add DrawSphereWires to your code for now:
[UnmanagedCallConv(CallConvs = [typeof(CallConvCdecl)])]
[LibraryImport("raylib")]
public static partial void DrawSphereWires(Vector3 centerPos, float radius, int rings, int slices, Color color);

Awesome, thank you!


Release new version to npm

Hello,
There has not been a new release of ember-cli-less to npm since https://github.com/gpoitch/ember-cli-less/pull/59 was merged in. Would it be possible for a new version to be updated and published?
Thanks!

Done!

Thanks so much!


Favicon is Needed

Someone add favicon to the site

Hi, I would like to work on this issue

@gokhalevedant06 Go Ahead

Anything else to be done ?

If you want , You can add your own web dev projects in the repo and share the screenshot in PL


Upgrade both Quick and Nimble libraries to the latest

Upgrade both Quick and Nimble libraries to the latest
Quick - v4.0.0
Nimble - v9.2.0
Fixes #2146

Pipeline failed at  test Carthage and I dont know why

I'll take a look at this one, @kinwahlai. Thanks for opening it 😊

Actually, this has already been merged into development in #2155! So I'm going to close this one. Thanks again, @kinwahlai!


Feedback on current platform prototype

Summary
Feedback on current platform prototype

If you have Docker installed, you can run the platform locally with the branch: build-tmp.
If you cannot run the platform locally, here are the screenshots I took for the current platform.
Homepage

A new design has been implemented on the platform. Please feel free to share your feedback. Here are some perspectives for advice inspiration:

Navigation of the platform

Can you easily find the information you need on the platform?

Information layout per page

Does the current layout help you quickly find the information you need?
Are the current information layout comfortable and convenient enough for you to read and use the platform?

Color

Are the colors calm/comfortable enough for you?

Anything else?

What needs to be done?
Please feel free to share your opinions and suggestions in the comment section.

Who can help?

Anyone!

Homepage:

View Stories

About Us

Thanks @SuziQpid

In both the opening page of the repository and the platform page describing AutSpaces the phrase ' collect together ' is used. It's clunky. Better to use ' collate' or simply leave it as ' collect'.

I support Susannas comments about the colour choices at the top of the landing/ log in page. You need the main heading underneath the first menu bar to be a non blue shade. It doesn't offer a good enough contrast to the turquoise / green tone of the menu bar. Wasn't there a stronger lilac/ purple tone or a strong silver that could be used.

Dear Georgina,
There’s a difference between the goals mentioned on the welcome page of the repository and the landing / login area of the platform. There are four on the welcome page and three on the landing page.
The extra one comes from the aspiration to collect the stories.
Also the  goal order is different. The order is better on the landing page and l think you should add the fourth one of collecting the stories at the top of those three. Then you’re consistent. There’s a slight difference in the wording too. The landing page flows the best.
Where exactly are the ‘guidelines’ situated. I’ve commented on issue 213. Do you need to create another issue for me to look at the guideline language. Sorry to be so literal but this is all such foreign yet weirdly familiar territory. I just want to remain useful to the group.
Suzanne lwai

On 10 Feb 2022, at 13:55, Georgia @.***> wrote:

Thanks @SuziQpid
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


[test] Fix test execution issue with JDK17 - integration tests

Motivation
https://github.com/apache/pulsar/pull/14099 pull's follow-up. Same fix but for all the projects which override the argLine surefire parameter.
Modifications

Added tests.additional.args for every surefire executions

Documentation

[x] no-need-doc

/pulsarbot rerun-failure-checks

@lhotari @eolivelli PTAL


[BUG] Can't upgrade from v1.1.2-rc1 to v1.1.2-rc1

Describe the bug

When you try to upgrade to the same version it hangs then eventually errors out saying The current version v1.1.2-rc1 is less than the minimum upgradable version v1.1.0.
To Reproduce
Steps to reproduce the behavior:

Install v1.1.2-rc2
Do an upgrade to the same ISO in the same way as the air-gap upgrade
Wait for upgrade to finish

Expected behavior

The upgrade should complete
Support bundle

supportbundle_d7f2c74f-c9df-4bef-8319-a5f2f54fdc44_2023-02-25T21-39-45Z.zip
Environment

Harvester ISO version: v1.1.2-rc2
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): Single node libvirt VM

Additional context
Add any other context about the problem here.
I did this upgrade test on a single node that had all of the upgrade test pre-upgrade tasks done. I'm going to try to reproduce it without any of those tasks and will update this ticket. It looks like this might be an issue with the upgrade version checker. The upgrade from v1.1.1 worked fine.

I checked this again and just a basic install and upgrade causes this error

This needs to be tested with 1.1.2-rc2

@noahgildersleeve Same upgrade should work now as v1.1.2 rc2 is available.
cc: @harvester/qa

Verified fixed on v1.1.1-rc2. Close this issue.
Result
We can upgrade the  three nodes Harvester cluster from v1.1.2-rc2 to v1.1.2-rc2 to the success state

Test Information

Test Environment: 3 nodes harvester on bare machines
Harvester version: v1.1.1-rc2

Verify Steps

Install v1.1.2-rc2
Do an upgrade to the same ISO in the same way as the air-gap upgrade

Wait for upgrade to finish


[Suggestion/Discussion] Support for logging.getLogger(__name__)

This isn't an issue, and not necessarily a feature request, but more of a request for feedback.
:bulb: tldr: I propose a refactor that puts most of logzero into a class implementing the interface expected by logging.setLoggerClass and reduce setup_logger() to a function that instantiates, configures and returns an instance of that class. This is to permit libraries unaware of logzero to benefit from all that makes logzero great but using the defacto standard pattern of logger = logging.getLogger(__name__). But please also read the last line. :)
If I'm writing an application where the logging is under my control, including in libraries I wrote, logzero is a beautiful solution. But consider the following situations:

I write a library that I depend on but others might want to also.
I depend on external libraries who have no idea about my application or its logging.

In these cases the recommended pattern is for code to call logging.getLogger(__name__), where the stdlib logging library will return a logger likely configured by the parent application.
My idea is to implement this pattern but also leverage the sane and beautiful defaults of Logzero and the very easy implementation.
Now, you know more about logging than I do, Chris, but it seems passing an appropriately implemented Logger class to  logging.setLoggerClass(klass) would achieve this. In the following I will call that class Klass both as the metavar for a class and to acknowledge your home in Austria, but it could be called logzero.DefaultLogger or something.
It seems there are three options:

Refactor logzero so it continues to do exactly what it does as well as it does but also offers the same default setup as a Klass that implements the interface described in the link above, offering the logzero experience to a wider set of use-cases.
Fork logzero entirely, copying and pasting the details necessary to implement my idea in a new project. This would mean limited hassle for you if you aren't interested, but means manual work to keep your improvements merged in; it would also mean yet another logging library that wouldn't be as discoverable as this.
Refactor logzero's code to encapsulate more things like the default formatter, default handlers etc in a way that offers an API for a wrapper project to provide a Klass, or for an advanced user to implement themselves. If this is the solution you may as well just do 1) imho.

In the current Logzero implementation, while some things like the LogFormatter from Tornado are written in classes that could be imported and reused in other ways, a lot of logic is in functions like setup_logger() which provides the documented Logzero interface but also handles setting up the instances the Logzero Way. I think those tasks should be separated.
Tricky bits:

The interface setLoggerClass defines: the constructor of Klass may only accept a name. However most configuration on logzero is done by calling module-level functions. If the defaults were implemented on the Klass and the config functions continued to manipulate the default logzero.logger, then the only problem would be finding the right way for a class-based user to achieve beautiful logzero results with logzero ease where configuration is needed. My suggestion is that the module-level config functions be implemented as methods on the Klass which update a number of class-level variables, and the module interface reduced to wrappers that call these methods on logzero.logger which becomes an instance of Klass.

I realise that this proposal would require a substantial rewrite of logzero, but on the other hand it's not a big file and it's mostly just a refactor and testing. On the other hand, I think there's an opportunity to maintain the current interface and expose the benefits of Logzero to a much wider set of use-cases which need the standard logging pattern but would love the ease and beauty of the Logzero interface and defaults.
Happy to hear your thoughts. If this doesn't appeal, I might attempt to implement this myself to demonstrate it and we can look at ways to collaborate after that if I can prove it's workable.
An alternative: just monkeypatch logging.getLogger in the parent app to logzero.setup_logger and be done with it. I don't know if that's a genius idea or a risky hack tbh.

Just a note that this issue is noticed. I'm just having a hard time finding the time to think about it! Will get back to you soon :)

@metachris I've been thinking about it a bit more too, and I'm starting to think:

Logzero does a particular kind of job very well and is suited for smaller applications where logging doesn't need over-thinking it just needs to be easy.
Setting up and using "traditional" standard library approaches like getLogger(__name__) isn't actually that hard for simple cases. But the defaults aren't great and the massive flexibility means things can seem hard or get overcomplicated easily.
Very complex applications should be out of scope for logzero anyway.

But: logzero provides defaults that could make configuring a logger with basicConfig or a dict config nicer for intermediate-level projects that are more than a quick cli and might involve logging from libraries. So now I'm thinking perhaps logzero could:
4. Refactor its implementation deliberately into two layers:
- A layer that makes available the "Lego bricks" (the colourful streams, a well configured rotating file handler, the json formatter etc) with a bunch of beautiful defaults and
- A layer that provides the easy, one-step logzero interface we love to use for small projects.
That way we can choose to use a lot of the cool stuff easily, but also take advantage of a bit more flexibility by using basicConfig or a dictionary config with standard library patterns. So I might want to build my own logging config, but still use your nicely formatted output in my cli and your json formatter in Lambda, and be able to do that very quickly and easily compared with hand-rolled approaches.
The good news is, I think Logzero is a bit a long that journey already. The LogFormatter is, I think, already importable. And you already have _get_json_formatter() to set up a Formatter, and .json() to apply that to the Logzero default interface. Really all I'm suggesting is that you promote things like _get_json_formatter() to a first-class interface we can reuse in other ways. But: make the API for that "advanced" use just as well considered and beautiful as the basic one.
Finally, then all you would need is:

Provide some more documentation to describe how to use these bricks. i.e. how to use my imaginary function "logzero.get_json_formatter()" with logging.basicConfig, or how to set up a nice friendly logzero.RotatingFileHandler in a dictConfig. (I haven't decided whether classes or module-level functions or both is the right interface for this layer)


[Doc]Explain the missing parameters in the doc in  TenantManagementService

Explain the missing parameters in the doc [1] in  TenantManagementService.
[1] https://docs.wso2.com/display/IS540/Managing+Tenants+with+APIs#ManagingTenantswithAPIs-addTenant()
Explain all the places where it is applicable
tenantInfoBean.originatedService
tenantInfoBean.successKey
tenantInfoBean.usagePlan

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


SkQuery 4.1.2

Describe the bug
Hi, I'm using your plugin but there is a big problem... Plugin is reporting error "Is it up to date?" but I have the newest version.. .I'm using version 4.1.2 but plugin is reporting that:
18:24:01[ERROR] Error occurred while enabling SkQuery v3.6.5 (Is it up to date?)
at com.w00tmast3r.skquery.SkQuery.onEnable(SkQuery.java:30) ~[?:?]
I'll be glad for your response.
Information:

Skript version: 4.1.2
Server software: Spigot
Server version: 1.8

We need the full error, that is not the full error.

I've the same problem without error.
My jar of SkQuery is 4.1.2 but in console when I start my server :
[SkQuery] Loading SkQuery v3.6.5


Add ability to set include-global etc. settings per group

Currently it is only possible to set settings for how global server and world permissions are handled for the whole server via the config.yml. It would be nice to have the ability to change this setting on a per group basis depending on the context that the player is in. That way you could have certain groups include their global permissions on a server (staff groups) while others only get their server specific ones applied. (players in different gamemodes)

If that's the case, just don't ever assign global permissions to your player groups, then you always have control over which applies.
As of 3.1, you can create your own contexts (your own "server" variable), which may also be a solution to this. You can then setup special "groups" of servers, and only have permissions apply on those.
I don't think refining these options further is necessary given the control you get from custom contexts.

I don't think that this is a good solution for that problem as you would have to set all permissions to only apply on a certain group of servers and not the other one, prompting you to change the context every time you add or remove a new server to that group. Imo. permissions that apply to nearly all servers should still be global, and the include-global setting is exactly there for servers where the global permissions shouldn't apply.
Providing the ability to decide that on a per group basis rather than globally on the server seems to be a better solution for me instead of having to write a separate plugin that tries to add this functionality with contexts or something similar. Granted this issue is kinda related to #254 as I noticed this flaw when trying to find a solution for the same problem (per server group/permission inheritance without the global permissions while still keeping the player's  rank intact) but both of them result in a different set of other possible use cases of the features.

Yeah, again I see what you mean, but it's still solvable with static contexts.
In your servers where global permissions should apply, you can define your own context in the static contexts file, and have those permissions only apply when the context is met.
I'm also not really sure of a way to easily implement your suggestion. Groups don't have attributes in this sense.

Well it could be just added the same way as weights, parents and permissions in general are. E.g. if the attribute isn't set for a group then it's using the global setting from the config.yml and if it is set then use the value to get whether or not it's enabled or disable.
But I'll definitely will look into the static contexts. Is there some documentation about that somewhere?

Static contexts refers to the file which sits in the LuckPerms folder. It's basically just the contexts which should be granted to all users by default. (hence "static")
General documentation on contexts as a whole can be found here:
https://github.com/lucko/LuckPerms/wiki/Command-Usage#what-is-context
or here:
https://docs.spongepowered.org/stable/en/plugin/permissions.html#contexts (since the LP system is based off the system created by Sponge)


[Model] HeCo

Description

Checklist
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [$CATEGORY] (such as [NN], [Model], [Doc], [Feature]])
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented
[ ] To the best of my knowledge, examples are either not affected by this change,
or have been fixed to be compatible with this change
[ ] Related issue is referred in this PR

Changes

Lack the readme.md in the example file, which includes the model performance.
Have you tested the example? I think there should be some bugs, e.g. you do not modify the __init__.py.
There should be docstring in the dataset, your layer/conv, and model.

yep, everything goes right now. I have forgotten to take the discarded backend mindspore before, which turns out the code was wrong.


Streamline user guide page flows for intro and "getting started," update docs and links

Fixes #3739

What changed?
Adjustments to the "Intro" and "Getting Started" user docs sections to improve flow and more cleanly decouple OSS from Enterprise content.

Why was this change made?
Onboarding has been quite friction-y and this aims to improve experience for users.

Branching issue. Addressed via https://github.com/weaveworks/weave-gitops/pull/3757.


⚠ Bofrost.be has degraded performance

In 17ebf69, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1298 ms

Resolved: Bofrost.be performance has improved in 86e6f0d after 1 hour, 27 minutes.


document counter-intuitive union choose coalesce path tracking

Could be squashed into https://github.com/ShiftLeftSecurity/overflowdb/pull/361 (why is github so slow today?)
This is more of a start for a discussion with code attached: The union, choose and coalesce steps take in a bunch of transform: Traversal[A] => Traversal[B] functions that may be composed of multiple traversal sub-steps. Should the sub-steps partake in path-tracking?
Since there is a bunch of commits stacked into this PR, the relevant question is
      "union" in {
        //fixme: Is this really what we want? Shouldn't substeps be path-tracked?
         centerTrav.enablePathTracking.union(_.out.out).path.toSetMutable shouldBe Set(
          Seq(center, l2),
          Seq(center, r2)
         )
      }

Here, one could argue that the result should rather be
Set(
          Seq(center, l1, l2),
          Seq(center, r1, r2)
         )

i.e. that all the traversal-style substeps inside the union should be path-tracked.
If this was a green-field decision then I would emphatically say yes! They absolutely should partake in path tracking!
Could we implement that? Yes we could. Would be slightly annoying for odbv2, but definitely possible.
Do we need to "fix" / change / improve that?
My personal opinion is that this entire DSL-gremlin "Traversal instead of nice clean loops / functions" stuff is a legacy boondoggle. We need to fix hard bugs and attempt to mostly maintain compat. Otherwise we should place path-tracking, repeat, union, coalesce, choose, where, and, or, etc all into maintenance mode, attempt to avoid that in our code, and even consider adding the @deprecate label to warn users away.
That is a strategy decision that is not for me alone to make, though -- cc @fabsx00 @DavidBakerEffendi @itsacoderepo @ml86 @mpollmeier whether you think all these Traversal features are legacy / maintenance-eventually-deprecate or important / continue-to-improve.

Traversal stuff is a legacy boondoggle

You won't be surprised that I disagree. A few thoughts:

Traversal being there doesn't stop us from implementing/generating the DSL steps on IterableOnce
Traversal gives us a nice generic graph DSL that let's users explore CPGs and other domains in a generic way
during debugging and developing new passes etc. it came handy many times that I wouldn't wanna miss it
the gremlin-style comes from our history, but it makes sense to preserve because many folks are familiar with it
those 'hard bugs' that you're addressing at the moment are in rarely if ever used edge case scenarios - it's great to get them fixed, but generally it's ok to break compatibility because those aren't really used. Other than that, maintaining Traversal and friends was very little work in the past.

Re path tracking of subqueries: I was gonna say "there's use cases for both" etc. but after giving this some thought I fully agree with the first part of your observation: subqueries should be path-tracked.
Since path tracking is rarely used in our code, and never* in conjunction with union etc, you should consider this green field and make that change.
(*) to my knowledge, at least in our codebase

Maybe I'm missing something but I can only find a single instance where path tracking is used across our code bases (the CallChainPass in CS). So if you want to drop support for something because path tracking doesn't compose well with all these other steps, I'd leave the rest alone and just remove/deprecate path tracking...
I write much of my code on the REPL first and I find that chaining a bunch of "semicolon-less" steps is much easier there than writing some loops. But even if I agreed that good old loops were preferable, we have code bases littered with all these steps. That code isn't going away and needs to be supported so in the end we'd just be lying to ourselves by slapping a @Deprecated on these methods.

Thanks for the replies!
So, it appears that multiple people care about the semicolon-less DSL (no deprecation in sight), and at least michael cares about path-tracking. So I went ahead and fixed it. Woe is me for the odbv2 implementation...
An extra bug that popped up is that repeat forgot the old path preceding the repeat step.
As an additional improvement, choose now evaluates the conditions only once. This is important for people like me who like to write match statements / partial functions where the conditions have side-effects, like
case item if visitedSet.add(item) => // new item! Also not new anymore if we check the condition again...

(yes, this is stupidly fragile and I should stop writing code like that)


nextToken is not null when it reach the end of the pagination

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v12.18.3
Amplify CLI Version
4.45.2
What operating system are you using?
Mac
Amplify Categories
api
Amplify Commands
Not applicable
Describe the bug
When doing a GraphQL Query on list, if the limit matches with the number of records, nextToken will not return null, instead, it will return a token
Expected behavior
nextToken should be null when there are no additional records.
Reproduction steps
query abc {
  listToDos(limit: 10) {
    nextToken
    items {
      id
      todo
    }
  }
}

NOTE: we have exactly 10 records, and we still get non-null nextToken returned.
GraphQL schema(s)

# Put schemas below this line
type ToDo @model {
  id: ID!
  todo: String
}

Log output

# Put your logs below this line

Example: If we have 6 records and the GraphQL list query limit is set to 3, the second page returns nextToken that is not null.
query abc {
  listToDos(limit: 3) {
    nextToken
    items {
      id
      todo
    }
  }
}

result:
query abc {
  listToDos(limit: 3, nextToken: "thisIsAToken") {
    nextToken
    items {
      id
      todo
    }
  }
}

nextToken is not null when it reaches the last page.

Hi @taoatmars
I tried the same steps and saw that next token is not null , if you try query with non null token again , it will return a empty list of items with next token as null.
Since it doesnt know if all the items have been queried , one more query is needed to check if all the items has been queried.
You can see as an example set the next token as 11 for 10 items. this will return all 10 items with next token as null.

Closing as the question seems to be answered by @akshbhu


Install error on visual studio 2015 preview ctp6

version is 6.1.3, this error will happen every time I install
since 7.0 is unlisted on nuget recent, so I can't test 7.0.
error information
Executing script file 'D:\helloworld\packages\EntityFramework.6.1.3\tools\install.ps1'...
Initialize-EFConfiguration : The term 'Initialize-EFConfiguration' is not recognized as the name of a cmdlet, function,
 script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is c
orrect and try again.
At D:\helloworld\packages\EntityFramework.6.1.3\tools\install.ps1:3 char:1
+ Initialize-EFConfiguration $project
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Initialize-EFConfiguration:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Add-EFProvider : The term 'Add-EFProvider' is not recognized as the name of a cmdlet, function, script file, or operabl
e program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At D:\helloworld\packages\EntityFramework.6.1.3\tools\install.ps1:4 char:1
+ Add-EFProvider $project 'System.Data.SqlClient' 'System.Data.Entity.SqlServer.Sq ...
+ ~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Add-EFProvider:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Type 'get-help EntityFramework' to see all available Entity Framework commands.

Looks like nuget problem, I will try to reinstall vs2015
https://visualstudiogallery.msdn.microsoft.com/5d345edc-2e2d-4a9c-b73b-d53956dc458d/view/Reviews

Feel free to re-open if re-install does not work.

Having the same issue and a reinstall did not fix it.

Getting the same

Try downgrading EF to 6.1.2

Re-opening as there are other reports of this now

The original issue this bug was tracking has been resolved and I have just tested on VS2015 Update 1 and not been able to reproduce any issues.
If you are seeing the failure, can you open a new issue with the Visual Studio and NuGet version numbers, as well as the output from Package Manager Console.


fix: Remove gst inputsrc string addition for USB cameras

PR Checklist

[x] Added label to the Pull Request for easier discoverability and search
[ ] Commit Message meets guidelines as indicated in the URL https://github.com/intel-retail/automated-self-checkout/blob/main/CONTRIBUTING.md
[x] Every commit is a single defect fix and does not mix feature addition or changes
[ ] Unit Tests have been added for new changes
[ ] Updated Documentation as relevant to the changes
[x] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with repository license and clearly outlined the added dependency.
[ ] PR change contains code related to security
[ ] PR introduces changes that breaks compatibility with other modules (If YES, please provide details below)

What are you changing?

Removing gst strings from the USB inputsrc.
Issue this PR will close
close: #266
Anything the reviewer should know when reviewing this PR?
Test Instructions if applicable

Build OVMS client and server
Connect a USB camera to your system
test with docker run RENDER_MODE with a USB camera

PIPELINE_PROFILE="instance_segmentation" RENDER_MODE="1" sudo -E ./docker-run.sh --platform core --inputsrc /dev/video4 --workload opencv-ovms

If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/automated-self-checkout )

It does not require the latest server code since the error is happening in the python / go pipeline code because the inputsrc is formatted for gst but expects just the usb path. ex. "/dev/video0"

It does not require the latest server code since the error is happening in the python / go pipeline code because the inputsrc is formatted for gst but expects just the usb path. ex. "/dev/video0"

In this afternoon, I am going to run some test with your PR on my box to see if it works


Service account creation in 1.4.1 not gated by boolean

What happened:
In our values override file we have this set:
serviceAccount:
  create: false
  name: splunk-system

What you expected to happen:
I expect the service account creation to be skipped and it to use the one I specified.
How to reproduce it (as minimally and precisely as possible):
If you compare the service account template in 1.3.0 to 1.4.1
https://github.com/splunk/splunk-connect-for-kubernetes/blob/release/1.3.0/helm-chart/splunk-kubernetes-logging/templates/serviceAccount.yaml
to
https://github.com/splunk/splunk-connect-for-kubernetes/blob/release/1.4.1/helm-chart/splunk-connect-for-kubernetes/charts/splunk-kubernetes-logging/templates/serviceAccount.yaml
you can see the SA is always created event though documentation implies this can be controlled as I have described above:
https://github.com/splunk/splunk-connect-for-kubernetes/blob/master/helm-chart/splunk-connect-for-kubernetes/charts/splunk-kubernetes-logging/values.yaml#L93
I will make a PR to fix this if that's OK.
IIUC all I need to do is add {{- if .Values.serviceAccount.create -}} to top of
https://github.com/splunk/splunk-connect-for-kubernetes/blob/release/1.4.1/helm-chart/splunk-connect-for-kubernetes/charts/splunk-kubernetes-logging/templates/serviceAccount.yaml
Anything else we need to know?:
nope.
Environment:

Kubernetes version (use kubectl version):
Ruby version (use ruby --version):
OS (e.g: cat /etc/os-release):
Splunk version:
Others:

Hello,
starting from 1.4.0, service account is required for logging, because it needs to query from kubernetes API server and enrich the data.

@pc-mreeves are you using your own service account and using the helm chart with "create: false"?
In that case if its not working, it might be a bug.

also https://github.com/splunk/splunk-connect-for-kubernetes/issues/397

Correct. That is what I am saying. The docs indicate that we can provide a service account but this helm chart ignores that and creates it anyways.
from links above.
1.3.0 behavior:
https://github.com/splunk/splunk-connect-for-kubernetes/blob/release/1.3.0/helm-chart/splunk-kubernetes-logging/templates/serviceAccount.yaml
{{- if .Values.serviceAccount.create -}}
apiVersion: v1
{{ if .Values.image.usePullSecret }}
imagePullSecrets:
- name: {{ .Values.image.pullSecretName }}
{{ end }}
kind: ServiceAccount
metadata:
  name: {{ template "splunk-kubernetes-logging.serviceAccountName" . }}
  labels:
    app: {{ template "splunk-kubernetes-logging.name" . }}
    chart: {{ template "splunk-kubernetes-logging.chart" . }}
    release: {{ .Release.Name }}
    heritage: {{ .Release.Service }}
{{- end -}}

1.4.1 splunk-kubernetes-logging behavior:
https://github.com/splunk/splunk-connect-for-kubernetes/blob/release/1.4.1/helm-chart/splunk-connect-for-kubernetes/charts/splunk-kubernetes-logging/templates/serviceAccount.yaml
apiVersion: v1
{{ if .Values.image.usePullSecret }}
imagePullSecrets:
- name: {{ .Values.image.pullSecretName }}
{{ end }}
kind: ServiceAccount
metadata:
  name: {{ template "splunk-kubernetes-logging.serviceAccountName" . }}
  labels:
    app: {{ template "splunk-kubernetes-logging.name" . }}
    chart: {{ template "splunk-kubernetes-logging.chart" . }}
    release: {{ .Release.Name }}
    heritage: {{ .Release.Service }}

this splunk-kubernetes-metrics chart for instance does not have this issue:
https://github.com/splunk/splunk-connect-for-kubernetes/blob/release/1.4.1/helm-chart/splunk-connect-for-kubernetes/charts/splunk-kubernetes-metrics/templates/serviceAccount.yaml
{{- if .Values.serviceAccount.create -}}
apiVersion: v1
{{ if .Values.image.usePullSecret }}
imagePullSecrets:
- name: {{ .Values.image.pullSecretName }}
{{ end }}
kind: ServiceAccount
metadata:
  name: {{ template "splunk-kubernetes-metrics.serviceAccountName" . }}
  labels:
    app: {{ template "splunk-kubernetes-metrics.name" . }}
    chart: {{ template "splunk-kubernetes-metrics.chart" . }}
    release: {{ .Release.Name }}
    heritage: {{ .Release.Service }}
{{- end -}}

this has been fixed and merged. please update helm chart to latest version.


mcuboot-imgtool: 2.0.0 -> 2.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mcuboot-imgtool is: MCUboot's image signing and key management
meta.homepage for mcuboot-imgtool is: https://github.com/mcu-tools/mcuboot/tree/main/scripts
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.1.0 with grep in /nix/store/bc4shs36mawb1ngj8pxip0bqs8f2pcmh-mcuboot-imgtool-2.1.0
found 2.1.0 in filename of file in /nix/store/bc4shs36mawb1ngj8pxip0bqs8f2pcmh-mcuboot-imgtool-2.1.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

mcuboot-imgtool

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/b588c4da706839dc0a97d6857a64676e35d577c1#mcuboot-imgtool

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bc4shs36mawb1ngj8pxip0bqs8f2pcmh-mcuboot-imgtool-2.1.0
ls -la /nix/store/bc4shs36mawb1ngj8pxip0bqs8f2pcmh-mcuboot-imgtool-2.1.0/bin

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

2 packages built:

mcuboot-imgtool
mcuboot-imgtool.dist

Maintainer pings
cc @otavio for testing.

@NickCao please see #313200.


Bulk Delete

Could not see in your documentation, is there a way to bulk delete all resources in a table. For example:
METHOD: DELETE
URL: localhost:8080/api/generic/table

It isn't part of the API currently. I would be concerned about potentially deleting too much of a table accidentally.
That being said it would be really easy to add this as a custom path. A lot of the internal methods are exposed for you to use. Here's a sample implementation you could use.
I'll close this issue, but if this doesn't work for you please open it again with your feedback.
var restify = require('restify');
var pgRestify = require('pg-restify');

var server = restify.createServer();

pgRestify.initialize({
  server: server,
  pgConfig: 'pg://localhost/pg_restify'
}, function(err, pgRestifyInstance) {

  addDeleteAllHandler(pgRestifyInstance);

  server.listen(8080);

});

function addDeleteAllHandler(pgRestifyInstance) {

  pgRestifyInstance.server.del('/api/generic/:resource', function(req, res, next) {

    // convert resource name to table name
    var table = pgRestifyInstance.convertResourceToTable(req.params.resource);

    // make sure the table is valid (to prevent sql injection)
    pgRestifyInstance.validateTable(table, function(err) {

      if (err) return next(err);

      // run the delete query
      pgRestifyInstance.runQuery('delete from ' + table, null, function(err, result) {

        if (err) {
          // log the error and return a generic error
          req.log.error(err.message);
          return next(new Error('Unexpected error.'));
        }

        res.send(200);

        return next();

      });

    });

  });

};

Thank you for the response, I will give this a go.


Add option to show chat effect description to send to affected player(s)

Is your feature request related to a problem? Please describe.
There's a lot of conditions and they can be hard to remember, instead of needing to either navigate into the actor sheet or use another module to peek at them, can affected token owners perhaps get the application/removal message of effects somehow?
Describe the solution you'd like
Another option in the settings to allow for owners of tokens affected by conditions to receive the apply/remove message, it gives a nice neat summary.
Describe alternatives you've considered
We used to use combat utility belt that dumped a public chat message when applying/removing and provided a summary for when the players' turn start, spitting out all effects affecting their that actor in chat, but I feel like that's a bit out of scope for CE? 😄
Additional context
Add any other context or screenshots about the feature request here.

Sounds good, added in the next minor version (likely 4.2.0). It's a new checkbox setting that will include the owner of the actor in any chat messages that are sent for effects. Note that it is still constrained by the normal role permission in the settings.


Test-Property : The term 'Test-Property' is not recognized as the name of a cmdlet, function, script file, or operable

Hi,
I am getting this error on Win10 x64 PS 5.1, do I need admin rights to run PAL?
Test-Property : The term 'Test-Property' is not recognized as the name of a cmdlet, function, script file, or operable
program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At C:\Users\xxxx\Documents\WindowsPowerShell\PAL.ps1:157 char:11

If ($(Test-Property -InputObject $Error[0] -Name 'CommandInvocati ...

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

CategoryInfo          : ObjectNotFound: (Test-Property:String) [], CommandNotFoundException

FullyQualifiedErrorId : CommandNotFoundException

Thanks

That's odd. That function is used very often in the script. No, you don't need admin rights to run PAL. Can you send the log file from %temp%\PAL.log?

Reopening per request.

Hi,
Thanks for re-opening the issue was caused by $ExecutionContext.SessionState.LanguageMode
set to ConstrainedLanguage via system variable and that was populated by GPO.
More details here https://itfordummies.net/2015/06/01/powershell-constrained-mode/
please close this topic and thanks again!


Output SBOMs in v1.3 of the spec

cyclonedx-go v0.4.0 introduced support for spec v1.3.

Has been addressed with 5bab19bbed9c6de22112ebeb2f71691c4b4163f5


Release 0.13.0

Close: https://github.com/aicers/giganto/issues/503

Codecov Report
Patch coverage: 33.33% and no project coverage change.

Comparison is base (80f8e81) 53.06% compared to head (0426643) 53.06%.

: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.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #516   +/-   ##
=======================================
  Coverage   53.06%   53.06%
=======================================
  Files          23       23
  Lines        5273     5273
=======================================
  Hits         2798     2798
  Misses       2475     2475

Files Changed
Coverage Δ

src/graphql/export.rs
8.00% <ø> (ø)

src/graphql/network.rs
85.91% <ø> (ø)

src/ingest.rs
66.11% <ø> (ø)

src/ingest/tests.rs
91.66% <ø> (ø)

src/peer.rs
47.94% <0.00%> (ø)

src/server.rs
89.36% <ø> (ø)

src/settings.rs
0.00% <ø> (ø)

src/storage/migration.rs
43.90% <0.00%> (ø)

src/publish.rs
52.94% <25.00%> (ø)

src/publish/tests.rs
98.37% <100.00%> (ø)

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


disable admin checks on merkle skips

they were never actually disabled
all users were using skip pointers as long as the server sent them back
the one thing we weren't doing is making sure that the server sent them back. that check was disabled via Admin()-feature-flag checking.

I didn't understand all the bullets above, but LGTM.


feat: avm/res/dev-ops-infrastructure/pool implement latest AVM UDT standards

Description
This pull request implements the latest AVM standards regarding UDT's.
Tagging module owners: @surajguptha @elizatargithub7
Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[x] Azure Verified Module updates:

[ ] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[x] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[x] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] I have run Set-AVMModule locally to generate the supporting module files.
[x] My corresponding pipelines / checks run clean and green without any errors or warnings

Great work @johnlokerse 💪
@elizatargithub7, please take a look and review the PR when you get the chance (:


⚠ Spryr has degraded performance

In 074e84b, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 29628 ms

Resolved: Spryr performance has improved in 0aa2613 after 7 minutes.


[HOT-FIX] Add EvaluatedType back to RDG

https://github.com/apache/spark/commit/87941ff8c49a6661f22c31aa7b84ac1fce768135 accidentally removed the EvaluatedType.

I tested locally. Will merge it to branch 1.4.

Merged build triggered.

Merged build started.

Test build #34001 has started for   PR 6589 at commit 618c2eb.

I have merged it into branch 1.4. I am closing it.

Test build #34001 timed out     for PR 6589 at commit 618c2eb     after a configured wait of 150m.

Merged build finished. Test FAILed.


refactor(ast)!: replace Modifiers with declare and const on EnumDeclaration

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3845  👈
#3844
#3841
#3839
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


Add ADA

I represent a university student organization that often leads study sessions for coursework related activities at SHSU. A language that is very heavily emphasized in that coursework is ADA. It would be helpful if minimally ADA-95 was added, but newer versions are more than welcome as well.
I haven't done due diligence with this code-base, so I don't know how you're sandboxing things. GNAT with GCC should be good enough for our purposes: https://gcc.gnu.org/onlinedocs/gcc-4.6.4/gnat_ugn_unw/Getting-Started-with-GNAT.html#Getting-Started-with-GNAT
If there's anything else I can provide to help with this process, or if this isn't something that can be done, don't hesitate to let me know!

Would you be willing to PR this in?


Fault bucket , type 0 Event Name: APPCRASH

Issue: Fault bucket , type 0 Event Name: APPCRASH Response: Not available Cab Id: 0
Image: 4.7.2-20240213-windowsservercore-ltsc2019, 4.7.2-windowsservercore-ltsc2019, 4.7.2
The issue starts right after 13 days of the container uptime.

Problem signature:
P1: svchost.exe_DiagTrack
P2: 10.0.17763.1
P3: b900eeff
P4: ntdll.dll
P5: 10.0.17763.4377
P6: ac56554b
P7: cfffffff
P8:<PHONE_NUMBER>0a3974
P9:
P10:

It is not limited only to this docker image, images with IIS has this issue.

[Triage] @John-Hart, can you look into this? IIS container image is crashing after 13 days of uptime.

This doesn't look like it's an IIS issue.  I would recommend contacting Microsoft Support

Microsoft Support? I know that it is easy to redirect, Microsoft Support will not solve the issue. Who created this official docker image?

Are you running any Windows Services in your container?

[Triage] Closing as not planned since there isn't enough info to diagnose the issue. @erkkov2lja feel free to re-open this issue if you have more info to share.


break formula if to wide

Is there a way to break formula's if they are to wide (with respect to their parent) ?

Not currently, but I was talking to @xymostech and the rules sound simple (breaking is allowed after any top-level rel or bin) so it might not be too difficult to add.

+1 I really need this feature. Is responsiveness also included in the equation ?

I don't know what you mean by that, sorry?

For example, if you switch from landscape to portrait, the formula adjusts to the space it has (responsive)

I think responsiveness is a big problem here.
Formules viewed from mobile break all responsive design.
They are too long, the view point is auto scaled to adapt these formulas.
I can not shorten the width of the formulas by setting width to a smaller value.
Really want to know any workaround.

@invkrh: Some possibilities until responsiveness becomes a feature in KaTeX:
(1) Give the enclosing element a style of "overflow:auto". Then your math will be full size and users can scroll the equation left and right to see it all.
For me, this has serious drawbacks, since on most mobile devices there is no indication that the element can be scrolled horizontally (no horizontal scrollbar appears like you get on a desktop browser, until you actually start scrolling the element. So if the page edge comes at an awkward part of the equation, it's possible the user has no idea that scrolling is possible and will never see the hidden part. It's possible to add a horizontal scroll bar with javascript of course).
Another option is display:table
http://netdna.webdesignerdepot.com/uploads7/how-to-create-horizontal-scrolling-using-display-table-cell/table-layout-horizontal/
(2) What I do is
(a) Manually break the equation into pieces at logical places (like equal signs, plus signs, minus signs, etc) so it will wrap logically
(b) For cases where splitting the equation is not possible, (it's a fraction, say), I nudge the font-size down with javascript until it does fit. I get away with that most of the time with the content on my site.
You can see both these things going on in this page (including in the drop-down answer sections):
http://m.intmath.com/fourier-series/2-full-range-fourier-series.php
The elements that cannot be split and that I know will not fit into a 300 px width are given the class "fitToWidth". The function "fitWidth" reduces the font-size if required.
I know it's clunky and could be done better (including the addition of an onRotate handler), but that will need to wait until the next revamp.
For the mobile version of my site, I mostly lay things out vertically when the LHS of an equation is quite wide.
Hope it helps.
Regards
Murray

@mbourne
Thank you for the idea using the horizontal scroll bar and overflow.
The workaround really works.
Looking forward to the responsiveness of Katex. =)

This would only line break inline equations and make displayed equations scroll.
Took me a while to figure it out, but you may simply copy this into the CSS file.
.katex-display > .katex {
  display: inline-block;
  white-space: nowrap;
  max-width: 100%;
  overflow-x: scroll;
  text-align: initial;
}
.katex {
  font: normal 1.21em KaTeX_Main, Times New Roman, serif;
  line-height: 1.2;
  white-space: normal;
  text-indent: 0;
}

@invkrh we ended adding a wrapper around each equation that would scale to fit and then if the user tapped it would zoom to the normal size and would allow the user to scroll within the container.  It wouldn't not cause the width of the page to expand.  The scrolling part is what @jeffswt has suggested.  The zooming/scrolling are both workarounds for not being able to break long expressions.

In principle the proposed css resolution works. I'm using the following at the moment:
.katex-display > .katex {
    max-width: 100%;
}
.katex-display > .katex > .katex-html {
    max-width: 100%;
    overflow-x: auto;
    overflow-y: hidden;
    padding-left: 2px;
    padding-right: 2px;
}

But it can lead to some unwanted consequences.
Normal screen size:

Small screen size with overflow scrolling:

@hagenw That's more likely to do with #824.

LaTeX supports automatic line breaking for inline math.  Here are some experiments:
Here is a long line of text.
Here is a long line of text.
Here is a $x+y+z+w+a+b+c$. % auto breaks after z+

Here is a long line of text.
Here is a long line of text.
Here is a $xyzabcdefghijklmnopqrstuvwxyz$.  % doesn't auto break

Here is a long line of text.
Here is a long line of text.
Here is a $xyzabcdefghij\allowbreak klmnopqrstuvwxyz$.  % auto breaks at \allowbreak

In my opinion, this aspect would be super useful, and is might be what was originally requested.  We should try to implement it, with proper line breaking rules (only after ops?).

Is there any progress with breaking lines for too wide formulas?

@ItsMeSousuke not that I'm aware of.

#1287 is a first stab at doing line breaks for inline math (not display math). Testing would be appreciated!

PR #1287 is merged; feel free to test it out on the master branch.  So the inline case is done.  I'm going to close this issue, as issue #1023 covers the display case (in my opinion -- feel free to make alternate proposals if you'd like).

This CSS does a good job for me:
.katex > .katex-html {
	white-space: normal;
}

.katex .base {
	margin-top: 2px;
	margin-bottom: 2px;
}

The overflow solution sometimes also annoyingly cuts off equations when viewed in desktop

To get around this, it might be better to wrap the css so that it is mobile only:
@media only screen and (max-width: 600px) {
  .katex-display > .katex {
    max-width: 100%;
    overflow-x: auto;
    overflow-y: hidden;
  }
}

This may not work for you but here:
.katex {
	white-space: normal;
}

.katex .katex-html {
	overflow-wrap: normal;
}

.katex .mord {
	display: inline-block;
}

This would only line break inline equations and make displayed equations scroll.
Took me a while to figure it out, but you may simply copy this into the CSS file.
.katex-display > .katex {
  display: inline-block;
  white-space: nowrap;
  max-width: 100%;
  overflow-x: scroll;
  text-align: initial;
}
.katex {
  font: normal 1.21em KaTeX_Main, Times New Roman, serif;
  line-height: 1.2;
  white-space: normal;
  text-indent: 0;
}

Works perfectly on responsive pages, as far as I'm concerned.
Leave a like if you find this working nice.

What should I do if I break a displayed equation?

For the mobile layout and getting the horizontal scrollbar to show up, I have tried all proposed solutions above, without success. The horizontal scrollbar is still not showing up.
You might try it yourself over here: https://www.mathelounge.de/765244/wieso-konvergiert-a-n-1-1-n-n-gegen-0?show=765257#a765257

Both equations are wider than the screen, but the scrollbar stays invisible.
I have added this CSS:
	.katex-display {
		overflow-x: auto;
	}
	.katex-display > .katex {
		max-width: 100%;
		overflow-x: auto;
	}

Has something changed with Katex or is it maybe Chrome or something complete different? Thanks.

In your example, the scrollbars do appear once the user starts to scroll left-right.
To ensure the scrollbar stays visible all the time, you can use a class "overflowXauto" as follows:
.overflowXauto{
  display:block;
  height:auto;
  padding-bottom:12px;
  -webkit-box-sizing:border-box;
  overflow-x:auto;
  white-space:nowrap
}
.overflowXauto::-webkit-scrollbar{
  height:8px
}
.overflowXauto::-webkit-scrollbar-track{
  -webkit-box-shadow:inset 0 0 4px rgba(0,0,0,0.3);
  border-radius: 7px
}
.overflowXauto::-webkit-scrollbar-thumb{
  border-radius:7px;
  -webkit-box-shadow:inset 0 0 8px rgba(0,0,0,0.5)
}
.overflowXauto::-webkit-scrollbar-thumb:hover{
  -webkit-box-shadow:inset 0 0 4px rgba(0,0,0,0.8)
}

This ensures the user can see scrolling is possible. See it in action here (Method 1 formula):
https://www.intmath.com/counting-probability/10-bayes-theorem.php

Hope it helps.
Murray

In your example, the scrollbars do appear once the user starts to scroll left-right.
To ensure the scrollbar stays visible all the time, you can use a class "overflowXauto" as follows:
.overflowXauto{
display:block;
height:auto;
padding-bottom:12px;
-webkit-box-sizing:border-box;
overflow-x:auto;
white-space:nowrap
}
.overflowXauto::-webkit-scrollbar{
height:8px
}
.overflowXauto::-webkit-scrollbar-track{
-webkit-box-shadow:inset 0 0 4px rgba(0,0,0,0.3);
border-radius: 7px
}
.overflowXauto::-webkit-scrollbar-thumb{
border-radius:7px;
-webkit-box-shadow:inset 0 0 8px rgba(0,0,0,0.5)
}
.overflowXauto::-webkit-scrollbar-thumb:hover{
-webkit-box-shadow:inset 0 0 4px rgba(0,0,0,0.8)
}
This ensures the user can see scrolling is possible. See it in action here (Method 1 formula):
https://www.intmath.com/counting-probability/10-bayes-theorem.php

Hope it helps.
Murray

My question is not about displaying the scrollbars, I need to break the equation which is overflowing out of the screen (Automatic break) i.e in whatsoever the equation shouldn't go outside of the screen (Scroll should not take place)

Thanks Murray. 🙇♂
So we need to wrap each equation into this self-defined CSS class .overflowXauto?

It needs to be a block-level element, so yes, wrap any expressions that (may) need a scroll bar into a DIV with that class.  You can make it skinnier if you want. The bottom padding was necessary because some KaTeX elements would be overlapped by the scroll bar without it.

@varun-msd; There is discussion on auto line-breaks earlier in this thread. See in particular https://github.com/KaTeX/KaTeX/issues/1023 , where it discusses what is already implemented in KaTeX.

`Without equation, it is working as overflow: scroll
$Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)$
Without array it is working as expected (Breaking properly)
\begin{equation}
Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)
\end{equation}
With array and without equation (Not Breaking properly, overflow is scrolling)
$3Ta{n^{ - 1}}x$$=\left{\begin{array}{ccc}
Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)
\end{array}\right}$
With array and with equation it is not working as expected (Curly braces has problem) - THIS IS MY ISSUE
$3Ta{n^{ - 1}}x$$=\left{\begin{array}{ccc} \begin{equation}
Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right)Ta{n^{ - 1}}\left( {\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}} \right) \end{equation}
\end{array}\right}$
`
ISSUE: Why if I add an equation inside an array it's breaking?

In your example, the scrollbars do appear once the user starts to scroll left-right.
To ensure the scrollbar stays visible all the time, you can use a class "overflowXauto" as follows:
.overflowXauto{
display:block;
height:auto;
padding-bottom:12px;
-webkit-box-sizing:border-box;
overflow-x:auto;
white-space:nowrap
}
.overflowXauto::-webkit-scrollbar{
height:8px
}
.overflowXauto::-webkit-scrollbar-track{
-webkit-box-shadow:inset 0 0 4px rgba(0,0,0,0.3);
border-radius: 7px
}
.overflowXauto::-webkit-scrollbar-thumb{
border-radius:7px;
-webkit-box-shadow:inset 0 0 8px rgba(0,0,0,0.5)
}
.overflowXauto::-webkit-scrollbar-thumb:hover{
-webkit-box-shadow:inset 0 0 4px rgba(0,0,0,0.8)
}
This ensures the user can see scrolling is possible. See it in action here (Method 1 formula):
https://www.intmath.com/counting-probability/10-bayes-theorem.php

Hope it helps.
Murray

Sometimes overflow-x CSS which has Katex is creating some extra space on the right side after the block

The hidden div creates the extra space

@Varun-msd For your first case, the equation environment is not implemented in KaTeX at this point in time. You can see what environments are available on this page: https://katex.org/docs/supported.html#environments
In LaTeX, trigonometric function names conventionally appear as all lower case, and non-italic. The following version is simpler and I think is close to what you want for that third example:
3\tan^{-1}x=\begin{Bmatrix}
\tan^{-1}\left({\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}}\right) &
\tan^{-1}\left({\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}}\right) \\
\tan^{-1}\left({\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}}\right) &
\tan^{-1}\left({\cfrac{{3x - {x^3}}}{{1 - 3{x^2}}}}\right)
\end{Bmatrix}

It looks like the following using the KaTeX demo page https://katex.org/

For the second case, please provide a link to a minimal example (or a fiddle) so we can see what is going on.
Regards
Murray

In your example, the scrollbars do appear once the user starts to scroll left-right.
To ensure the scrollbar stays visible all the time, you can use a class "overflowXauto" as follows:
.overflowXauto{
display:block;
height:auto;
padding-bottom:12px;
-webkit-box-sizing:border-box;
overflow-x:auto;
white-space:nowrap
}
.overflowXauto::-webkit-scrollbar{
height:8px
}
.overflowXauto::-webkit-scrollbar-track{
-webkit-box-shadow:inset 0 0 4px rgba(0,0,0,0.3);
border-radius: 7px
}
.overflowXauto::-webkit-scrollbar-thumb{
border-radius:7px;
-webkit-box-shadow:inset 0 0 8px rgba(0,0,0,0.5)
}
.overflowXauto::-webkit-scrollbar-thumb:hover{
-webkit-box-shadow:inset 0 0 4px rgba(0,0,0,0.8)
}
This ensures the user can see scrolling is possible. See it in action here (Method 1 formula):
https://www.intmath.com/counting-probability/10-bayes-theorem.php

Hope it helps.
Murray

Sometimes overflow-x CSS which has Katex is creating some extra space on the right side after the block

The hidden div creates the extra space

@mbourne  any update on this issue ?

Yes, as I said at the end of my last reply, "For the second case, please provide a link to a minimal example (or a fiddle) so we can see what is going on."

Screen record of issue: https://drive.google.com/file/d/1_AxpodIMRnZj9G7p9tyZK5cUwr2InOd5/view
CSS:
.main-container {
flex: 1;
flex-direction: column;
padding: 5px;
padding-bottom: 0px;
overflow-x: scroll;
user-select: none;
}
table {
display: block;
overflow-x: scroll;
background-color: pink
box-sizing: border-box;
border-collapse: collapse;
margin-bottom: 20px;
}
td, th {
border: 1px solid ${theme.textColor.regular};
padding: 1.0rem;
word-break: normal;
}
`

Neutrons are discovered by James Chadwick.

It is neutral particle which is slightly heavier than proton

When Beryllium bombarded by $\alpha-$ particles, neutrons formed.
Example:
${}_4B{e^9} + {}_2H{e^4} \to {}_6{C^{12}} + \mathop {{}0{n^1}}\limits{\left( {neutron} \right)}$

Particle (Scientist)
Symbol
Charge
Mass (amu /Kg)

Electrons (J.J.Thomson)
$- {1^{{e^0}}}$ or ${e^ - }$
$- 1.6 \times {10^{ - 19}}C$ or $-4.8 \times {10^{ - 10}}$ esu
$0.000549/$$9.1\times {10^{ - 31}}$

Proton (Goldstein)
P or ${}_1{H^1}$
$+ 1.602 \times {10^{ - 19}}C$ or $+4.8 \times {10^{ - 10}}$ esu
$1.00758/$$1.672\times {10^{ - 27}}$

Neutron (Chadwick)
${}_0{n^1}$
Zero charged
1.00893/$1.674\times {10^{ - 27}}$

Positron (Anderson)
${}_{ + 1}{e^0}$
1 unit +Ve charge
0.000549/$9.1\times {10^{ - 31}}$

Neutrino and Antineutrino (Fermi)
${}_0{e^0}$
Zero charge
< 0.00002

Antiproton (Chanberlain)
${}_{ - 1}p$
Negative charge
1.00758

Meson (Yukawa)
${\pi ^ + },{\pi ^0},{\pi ^ - }$
+Ve, zero, -Ve
273 >${e^ - }$

`

@mbourne position: relative solved the issue !

We'd like to replace MathJax with KaTeX for a probability textbook at Berkeley: https://textbook.prob140.org/
The line breaking issue is currently a blocker for us since we've already written a bunch of long equations that KaTeX cuts off, for example:

We are currently using MathJax's automatic line breaking functionality (http://docs.mathjax.org/en/latest/output.html#automatic-line-breaking) which gives a reasonable-looking output for these cases:

We can get around this with using:
.katex {
  white-space: normal;
}

Which gives:

As you can see, this doesn't produce great output.
It would be great for us if KaTeX could support a rendering option like MathJax's that breaks long block-level equations in a reasonable way.

is it possible to reproduce how the mathjax is breaking the equations in katex, not especially through scroll?. Any help will be appreciated.

We'd like to replace MathJax with KaTeX for a probability textbook at Berkeley: https://textbook.prob140.org/
The line breaking issue is currently a blocker for us since we've already written a bunch of long equations that KaTeX cuts off, for example:

We are currently using MathJax's automatic line breaking functionality (http://docs.mathjax.org/en/latest/output.html#automatic-line-breaking) which gives a reasonable-looking output for these cases:

We can get around this with using:
.katex {
  white-space: normal;
}

Which gives:

As you can see, this doesn't produce great output.
It would be great for us if KaTeX could support a rendering option like MathJax's that breaks long block-level equations in a reasonable way.

is it possible to reproduce how the mathjax is breaking the equations in katex, not especially through scroll?. Any help will be appreciated.

https://github.com/KaTeX/KaTeX/issues/327#issuecomment-291382715 no longer works in katex version 0.16.2. For inline equations it needs to be adapted to
.katex-html > .base {
  white-space: normal;
}

How to enable word wrap while working with mhchem, whitespaces do not break up into multiple lines in a \ce{} block,