 I get that the model is unconditional, is that why? Or am I doing something wrong

Dear Titan:
DId you solve this problem? I have meet the similar problem that i train the model for 200000 steps, but when i sample from the pt file, only get noise image. I do not know what's the problems.....

Same problem here, has anyone solved it?

Do you set diffusion_steps=4000?

Hi everyone,  I have a couple of questions for you. I run the code, and it runs forever; which parameter can I adjust to stop the training process quickly? Maybe around 2 hours of training is good for me. Secondly, when I stop it manually and sample some images, I get these file like this:  arr_0.npy, so  I tried to convert them into images I just get a bunch of noises like this

@badcommandorfilename have you resolved the problem ? some training tricks to share ?

Hitting the same issue with imagenet64_uncond_100M_1500K.pt and 4k steps. Running with fewer diffusion steps (200, 500, 800, 1000) seems to work.
I'll get back to the thread if I find a bug. Probably some thresholding logic is missing, and image values are hitting saturation point.

I encountered the same problem when training on a custom dataset (cats). To be specific, the model only produces static noise on 4000 steps. However, after another 4000 steps, it can actually produce blurred cats. So I believe more training steps may work.

Hi bro, how about sample result quality after more training steps?

Hitting the same issue with imagenet64_uncond_100M_1500K.pt and 4k steps. Running with fewer diffusion steps (200, 500, 800, 1000) seems to work.
I'll get back to the thread if I find a bug. Probably some thresholding logic is missing, and image values are hitting saturation point.

Thank for your tips about reduce sample step, it seems work. But I did not know why it work？in my understand, more sample step should produce more image details, Could you give me some hints? thanks again

I trained 20,000 steps for generating cat pictures. The quality of cat pictures is getting better in the first 15,000 steps. But It is hard to say whether the image quality is still improving after that. BTW, The loss did not decrease significantly after the first 4,000 steps in my case.
The result is OK but not impressive. You can check them out at https://drive.google.com/drive/folders/1-obvOanCKA0lRGeS-SFK6eiZmAVvgqhn?usp=sharing

Thanks a lot for your tips and result,  I'm going to try the generation of larger resolution images, this should improve the lack of clarity, But just I guess...
Thanks again!

Hitting the same issue with imagenet64_uncond_100M_1500K.pt and 4k steps. Running with fewer diffusion steps (200, 500, 800, 1000) seems to work.
I'll get back to the thread if I find a bug. Probably some thresholding logic is missing, and image values are hitting saturation point.

I encountered the same issue (noise) when tried to sample from imagenet64_uncond_100M_1500K.pt and cifar10_uncond_50M_500K.pt. So should I sample with fewer diffusion steps?

After using --diffusion_steps=1000 instead of 4000 for sampling from cifar10_uncond_50M_500K.pt & imagenet64_uncond_100M_1500K.pt, I started seeing clearer pictures instead of pure noise. I also tried using --diffusion_steps=4000 with the --timestep_respacing but that produced pure noise as well. Can't figure out why though.

Hitting the same issue, and I do not know why. diffusion is really confusion

I also used image_nll.py for cifar10_uncond_50M_500K.pt. Using --diffusion_steps=4000 I got 5.25 bpd while using --diffusion_steps=1000 I got a better result of 3.28 bpd which is a lot closer to the results of Table 2 in the paper.

I trained 20,000 steps for generating cat pictures. The quality of cat pictures is getting better in the first 15,000 steps. But It is hard to say whether the image quality is still improving after that. BTW, The loss did not decrease significantly after the first 4,000 steps in my case.
The result is OK but not impressive. You can check them out at https://drive.google.com/drive/folders/1-obvOanCKA0lRGeS-SFK6eiZmAVvgqhn?usp=sharing

I saw your generated results, but I didn't feel very clear. I am using bedroom, and I feel that the generated image is a little dark. I trained 10k steps

我为生成猫图片训练了 20,000 步。猫图片的质量在前 15,000 步中越来越好。但很难说在那之后图像质量是否还在提高。顺便说一句，在我的案例中，在前 4,000 步之后，损失并没有显着减少。
结果还可以，但并不令人印象深刻。您可以在https://drive.google.com/drive/folders/1-obvOanCKA0lRGeS-SFK6eiZmAVvgqhn?usp=sharing查看它们

I saw your generated results and felt that the picture was not very clear. I am using bedroom, and I feel that the generated image is a little dark. I trained 10k steps

I trained 20,000 steps for generating cat pictures. The quality of cat pictures is getting better in the first 15,000 steps. But It is hard to say whether the image quality is still improving after that. BTW, The loss did not decrease significantly after the first 4,000 steps in my case.
The result is OK but not impressive. You can check them out at https://drive.google.com/drive/folders/1-obvOanCKA0lRGeS-SFK6eiZmAVvgqhn?usp=sharing

I noticed that your generated image sizes are all 256x256, may I ask how to generate 256x256 sized images? I can only generate 64x64 images with a 128x128 dataset, adjusting the image_size in script_util to 128 will show 'unsupported image size', adjusting it to 256 will show ' Error(s) in loading state_dict for UNetModel'


MacthResult does not provide Range & NSTextCheckingResult (if install as Framework)

Thank you for this easy and useful tool!
I was use it included in project as files. But now I was installed it through Carthage and MatchResult lost Range & NSTextCheckingResult properties.

At the moment, range and matchResult are internal API, as I consider them implementation details. In general I'd prefer to add new public API rather than exposing Foundation types, if possible.
Would you mind sharing an example of how you're using those properties?

I've been used replaceRange(subRange: Range, with newElements: String); removeRange(subRange: Range) methods and NSTextCheckingResult's 'range' property for creating NSAttributedString.

Instead of replaceRange(), you can use Regex's String.replaceFirstMatching(_:with:) and String.replaceAllMatching(_:with:) methods. Instead of removeRange you can do the same thing but with the empty string as the replacement.
NSAttributedString support has come up before, so I'd like to track that as a separate issue: #38. However, I'd appreciate it if you could share a code snippet showing an example of how you were using Regex with NSAttributedString before you switched to Carthage?
I'm going to close this issue for now, but let me know if you have any further questions.


Hint-Depth-Texture not supported.

Hey there, I noticed an error for Godot 4.2.stable in the 'refraction.gdshaderinc' file.
It says "'hint_depth_texture' is not supported in '' shaders"
I encountered this while I tried to implement a silhouette or x-ray shader together with your cel-shader. I can't get it working and I think it is because of this error.

Hey there, thanks for the report. I checked it out and yes, there is this error but the shader seems to work as intended. I also looked at the docs and yes, hint_depth_texture is supported. Maybe this error is a bug, because materials using it seems to be working normally.

Thanks for looking into it so quickly :)
I thought that error was the reason why I couldn't get the silhouette shader to work. Seems like I have to look into it a bit more!


fixed DisableCursor() on web by registering an empty mouse click event

Just a small fix that may be a hack, haven't tested on more than just my machine yet, but it seems harmless enough

@grenappels Thanks for this fix!


Remake rust type

Not sure that it good way, but it helps comment all functions with unsupported parameters.
Also I do not like class names, but nothing normal could not come up

Traited

Thanks!


[MNT] added pytest timeout of 5 minutes

This PR adds a dependency on pytest-timeout in the dev dependency set, and a timeout-fail of 5min on all individual tests.
This will address tests hanging for 6 hours such as in https://github.com/alan-turing-institute/sktime/issues/2448 (they will fail earlier, at the 5min bound)

Can we not have this flag in setup.cfg instead repeating it 4 times? Also other flags might go there

Can we not have this flag in setup.cfg instead repeating it 4 times? Also other flags might go there

I would agree with having all the flags in one place, but the duplication/multiplication is already present, e.g., with the --cov flag.
Hence, removing the duplication/multiplication is a scope separate from this PR.


ignore

Expected Behavior
It should work offline if all needed models are already downloaded and available.
Current Behavior
Imaginairy seems to require a connection with Hugginface on initialization. If my computer is offline, it aborts with this error, no image generation possible:
requests.exceptions.ConnectionError: (MaxRetryError('HTTPSConnectionPool(host='huggingface.co', port=443): Max retries exceeded with url: /api/models/stabilityai/stable-diffusion-xl-base-1.0/revision/462165984030d82259a11f4367a4eed129e94a7b (Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x154349450>: Failed to resolve 'huggingface.co' ([Errno 8] nodename nor servname provided, or not known)"))'), '(Request ID: 3816a0d4-9d26-4771-8599-eeaafc70c60d)')
It only seems to require the internet connection on the first run though. If i run my program and only kill my internet after the first sucessful generation, it still works for subsequent generations.
Environment and Context
imaginAIry 14.1.1
Python 3.10.13
fastapi 0.109.0
numpy 1.26.3
uvicorn 0.27.0.post1
Macos 14.3 (23D56), M1

sorry, posted in the wrong repository… my bad.


Fix equality check for simple floating types in RowContainer

The row container implementation for
equalsNoNulls and equalsWithNulls contained a bug:

Incorrect equals check for floating point types when NaN values are used.
Refactor to use SimpleVector::comparePrimitiveAsc in RowContainer and ContainerRowSerde for a common comparison function.
Change static SimpleVector::comparePrimitiveAsc to be static inline to reduce function call overhead in this expanded usage.

This is a continuation of PR #5833 which addressed floating point comparisons for complex types.
Affected operators:
FilterProject, TopN, TopNRowNumber, OrderBy, MergeExchange, LocalMerge, HashProbe, NestedLoopJoinProbe
The lists may not be complete.

@aditi-pandit @mbasmanova Please review. I addressed @aditi-pandit earlier comment. Thanks!

@mbasmanova : PTAL.

Hi @czentgr, this change looks good to me. But I wonder why the PR summary says that this change affects functions such as array_distinct and array_max? I saw the change affects RowContainer, but array_distinct and array_max do not use RowContainer, right?

@kagamiori You are correct. The functions don't operate on the row container and the statement in the commit/PR is incorrect. I took a closer look at the functions and turns out there are more problems with NaN that aren't addressed by this fix.
For example, results of array_sort with Prestissimo (including the fix in this PR)
presto> select array_sort(a) from (values ARRAY[nan(), 0.0e0, 1.0e0]) t(a);
      _col0
-----------------
 [0.0, NaN, 1.0]
(1 row)

Java
presto>  select array_sort(a) from (values ARRAY[nan(), 0.0e0, 1.0e0]) t(a);
      _col0
-----------------
 [0.0, 1.0, NaN]
(1 row)

The same problem exists for array_sort_desc.
I also think there is a problem with set_union but both Java and Prestissimo return the same result. I expect NaN to not be int he result because it should be counting as being present in both just like 3.0 is (as we consider NaN == NaN)
presto> SELECT set_union(a) FROM ( VALUES ARRAY[1.0, nan(), 3.0], ARRAY[nan(), 3.0, 4.0]) AS t(a);
           _col0
---------------------------
 [1.0, NaN, 3.0, NaN, 4.0]
(1 row)

@kagamiori You are correct. The functions don't operate on the row container and the statement in the commit/PR is incorrect. I took a closer look at the functions and turns out there are more problems with NaN that aren't addressed by this fix.
For example, results of array_sort with Prestissimo (including the fix in this PR)
presto> select array_sort(a) from (values ARRAY[nan(), 0.0e0, 1.0e0]) t(a);
      _col0
-----------------
 [0.0, NaN, 1.0]
(1 row)

Java
presto>  select array_sort(a) from (values ARRAY[nan(), 0.0e0, 1.0e0]) t(a);
      _col0
-----------------
 [0.0, 1.0, NaN]
(1 row)

The same problem exists for array_sort_desc.
I also think there is a problem with set_union but both Java and Prestissimo return the same result. I expect NaN to be in the result once (as we consider NaN == NaN)
presto> SELECT set_union(a) FROM ( VALUES ARRAY[1.0, nan(), 3.0], ARRAY[nan(), 3.0, 4.0]) AS t(a);
           _col0
---------------------------
 [1.0, NaN, 3.0, NaN, 4.0]
(1 row)

Hi @czentgr, thank you for updating the PR summary. We have recently noticed the NaN comparison issue in Velox functions too. (See https://github.com/facebookincubator/velox/issues/8738 and https://github.com/facebookincubator/velox/issues/8690) In fact, tracing back the behavior of NaN handling, we found that Presto has inconsistent NaN behaviors within and among functions too. There are two github issues in the Presto repo related to this problem: https://github.com/prestodb/presto/issues/21877 and https://github.com/prestodb/presto/issues/21936.

@bikramSingh91 Would you please be able to tell me on the Linter warnings? I would like to fix them and also rebase the PR again. Did I add new warnings in this PR?

@kagamiori Thank you for the found issues. I have a documentation PR for Velox to clarify the NaN behavior there. This is adopted from the Spark behavior and makes the most sense and provides consistency. Presumably, Presto should follow this as well: https://github.com/facebookincubator/velox/pull/7237.

@bikramSingh91 Would you please be able to tell me on the Linter warnings? I would like to fix them and also rebase the PR again. Did I add new warnings in this PR?

Hi @czentgr, @bikramSingh91 is out of office this week. The internal linter warnings come from the fact that the newly added methods in RowContainerTest.cpp defines a set of variables through TEST_FLOATING_TYPE_LIMIT_VARIABLES, but not all the defined variables are used in these methods (i.e., warning of unused variable).

Hi @czentgr, could you rebase this PR onto the latest main? It's required for committing the code. Thanks!

@kagamiori done. Thanks!


Fix KF_NAME length

update env.sh

Conflict with iam.gserviceaccount.com limit length(6~30)

Affected files
# kpt-set<EMAIL_ADDRESS>

Please sign CLA https://github.com/kubeflow/gcp-blueprints/pull/358/checks?check_run_id=5640630849

I have signed in both of my accounts, but it doesn't seem to be rescan successful?

Thank you for the PR! Indeed it is a problem where we should fix the name length restriction.
Instead of shortening from 24 to 17, how do you think about making the cloudsql service account shorter, so we give people more characters to set their project? For example<EMAIL_ADDRESS>You can check the CLA issue here, looks like it is related to email: https://github.com/kubeflow/gcp-blueprints/pull/358/checks?check_run_id=5653631084

I think it's a good idea, so we don't have to make changes in env.sh and less impact on users
And I screwed up commit author information...

Thank you for the PR! Indeed it is a problem where we should fix the name length restriction.
Instead of shortening from 24 to 17, how do you think about making the cloudsql service account shorter, so we give people more characters to set their project? For example<EMAIL_ADDRESS>You can check the CLA issue here, looks like it is related to email: https://github.com/kubeflow/gcp-blueprints/pull/358/checks?check_run_id=5653631084

@kindomLee Thank you for the confirmation! I can review the PR once you have updated it to make the service account name suffix shorter.
You can configure your email by using:
git config --global user.email<EMAIL_ADDRESS>
Then amend your existing commit
git commit --amend --reset-author

Then force push to your remote branch to overwrite your existing commit, in order to be CLA compliance.


Improve "conda availability" assertions with index queries

Grayskull's logic to determine whether a package is available on conda or not is based purely on the existence of the package name in anaconda.org/{channel}.
There's no concept of index, multiple channels or spec compatibility. To ensure that we try to install conda packages that indeed exists, we need to check whether the specs calculated by grayskull result in any records being found in the computed index (e.g. a recent version required by some pip package might not be available on conda-forge yet, in that case we use pip's).

This env.yml reproduces the current problem:
channels:
- conda-forge
dependencies:
- python=3.10
- tensorflow=0.14
- pyyaml
- pip
- pip:
  - tensorflow-addons==0.14.0
  # note the .0 here -------^; we need to be exact because 0.14 is not a real version
  - tf-models-official

This calculates:
conda will install:
 - cython
 - pillow
 - gin-config
 - google-api-python-client[version='>=1.6.7']
 - immutabledict
 - kaggle[version='>=1.3.9']
 - matplotlib-base
 - numpy[version='>=1.20']
 - oauth2client
 - pandas[version='>=0.22.0']
 - psutil[version='>=5.4.3']
 - py-cpuinfo[version='>=3.3.0']
 - pycocotools
 - pyyaml[version='>=6.0.0']
 - sacrebleu
 - scipy[version='>=0.19.1']
 - sentencepiece
 - seqeval
 - six
 - tensorflow-datasets
 - tensorflow-hub[version='>=0.6.0']
 - tensorflow[version='>=2.14.0,<2.15.dev0']
 - tf-slim[version='>=1.1.0']
pip will install:
 - tensorflow-addons==0.14.0
 - opencv-python-headless
 - tensorflow-model-optimization>=0.4.1
 - tensorflow-text>=2.14.0,<2.15.dev0
 - tf-models-official==2.14.0

But tf-slim >=1.1.0 doesn't have matches in conda-forge.


Vulnerabilities

Anybody can see these by forking it and enabling dependabot

I don't think any of these are urgent; we don't actually perform any computation on the site based on user input, nor do we store any secrets in the filesystem or database (so information disclosure bugs, etc, aren't an issue). That said, I'd love to see the list cleared up. It looks like it will take some untangling of various dependent versions for the gems.
For the json bits, none of that is even run on the production site. We only have it for running grunt lint checks by developers. There's some dependency untangling that needs to happen around lodash. I spent an hour or three with it a long time ago and gave up in disgust.
So basically: yes, if you'd like to bring some of the dependencies up to date, I'd love to see a PR.

Hi @peff, I noticed you're not running any dependency management bot on this repository. Want me to get you set up with one?
I can make a pull request with a complete official GitHub-native Dependabot config that you can merge and enjoy right away without you needing to bother with configuring it. The main drawback of Dependabot is that it's a bit of a dumb bot, it's easy to use and get setup with, but it's not very configurable. The only way to turn down the "firehose" of updates is by limiting the schedule to a weekly or monthly run.
Or I can help you get started with the Renovate bot. That bot is more configurable, but the bot requires you to install an (GitHub verified) app into your organizations account.
You could for example have a "dashboard" where you can select what updates you want to get a PR for. You would have a list of all pending updates, and then you can pick and choose from the dashboard. That way you would only get updates when you're ready to deal with them. You can even "auto-merge" minor version bumps automatically so you don't have to deal with those. You can also limit the schedule in way more ways than you can with Dependabot.
I won't pretend to know everything about Renovate, but I can at least get you started, and call in extra help to configure the Renovate bot to your exact liking.
Let me know if you fancy either of those. And if you don't want any of those, let me know as well. :smile:
References:
Dependabot configuration options
Renovate bot docs
Renovate bot docs on reducing the spam/noise of update pull requests
Greetings HonkingGoose

@HonkingGoose We do have dependabot running, as it powers GitHub's baked-in security alerts. It's possible that adding more specific config could help, but I think the main problem is that somebody has to actually untangle the dependencies that dependabot can't manage to create a PR for.

@peff Did you fix these vulnerabilities with https://github.com/git/git-scm.com/pull/1598#issuecomment-812004486?

The JSON ones were fixed recently, yes. I didn't dig, but the gem ones should have been fixed by #1548 (or possibly #1597). At any rate, dependabot has no complaints with the current state.

I'm going to close this, since we're now at a clean slate with respect to dependabot.


Easy access to the latest OSN and latest Models

We should be able to easily distribute updates for the latest OSN versions and we should have a way for a user to easily access our latest models, with descriptions of how the model is used.
Source: Kevin McGee on behalf of the RFT

We're currently working on creating packages for each platform that should allow in-place upgrades.  We have a prototype system in place for Ubuntu, but not for other platforms.  We're working on that.
We do have an open issue for being able to select a model from a model zoo, but that part of the system has not yet been implemented.  You can indicate your support for that issue by comment in this issue.

There is definite interest for distributed updates supporting the Mac-centric version of OSN to help provide a stable user experience.


Version 0.0.15

I've really tried to find all missing version numbers this time. Some were stuck on 0.0.7, 0.0.8 or 0.0.11 and were not getting updated. The update-version.sh script now also does the ls dep for dir and vdir and the base32 dep for base64 and basenc. I noticed that the formatting for all the Cargo.toml files is all over the place when it comes to spacing. So I think we should find a toml formatter and enforce a style there to make this kind of automation more reliable.

Wow, I think this strict dependency change uncovered an issue with dependencies :)

excellent :)
I guess you saw the cargo deny issue

I won't be connected this afternoon, feel free to self merge this PR :)


Wiki is not up to date

The issue is related to the wiki. The jmx part is not not up to date.
https://github.com/Netflix/servo/wiki
Currently is doing this:
private final Counter bytesSent = Monitors.newCounter("bytesSent");

and:
Monitors.registerObject(this);

Thanks,
Diego
I do not achive the goal to start the example:
https://github.com/Netflix/servo/tree/master/servo-example/src/main/java/com/netflix/servo/example
It runs(echo and exit are working), but it does not provide jmx(I can not connect with jconsole).

I updated the gradle command in the examples section, but it sounds like you got it to run. I tried with jvisualvm and jmc and both are able to connect and show the entries in jmx.

Are you saying you cannot connect to jmx at all or that you aren't seeing the data in jmx?

Hi,
I think I was not clear because I talked about two things. 1)The documentation is not consistent with the example code and 2)I do not achieve to see the mbeans. Forget 2. I will try to explain 1 :-)
In the documentation(https://github.com/Netflix/servo/wiki#jmx), the metrics are initializated like this:
@Monitor(name="BytesIn", type=COUNTER)
private AtomicLong bytesIn = new AtomicLong(0L);
But in the example:
https://github.com/Netflix/servo/blob/master/servo-example/src/main/java/com/netflix/servo/example/BaseHandler.java
metrics are started like this:
private final Counter bytesReceived = Monitors.newCounter("bytesReceived");
I would expect to get the same code in the documentation.
Thank you!

Documentation and other improvement efforts are going to spectator. PRs welcome, but we have no plans to invest our limited resources improving the servo docs.


Native Video Renderer

I am opening this PR to start getting some pre-emptive feedback, and also so you can highlight anything more this would need in order to get merged. Also opening in case anyone wants to help test, or help figure out some of the missing pieces if needed.
This adds the ability to render the WebRTC video streams into an Unity texture through Native C++ calls, bypassing C# marshalling. A new component called NativeVideoRenderer can be used in place of the existing VideoRenderer component in order to use the new functionality.
There are a handful of new C# scripts in the Unity library, as well as a whole new C++ project which implements the native rendering calls.
The two big issues I see as remaining:

Currently this is tested to work only on Windows 10 DX11 and Android OpenGLES. There is a DX12 implementation but is untested. Vulkan and Metal haven't been started on. I am wondering do all graphics API's and platforms need an implementation before this could be merged?
Currently does not support ARGB. Altspace didn't need it so I haven't put the time on it, but can if it is needed.

This looks very good! I'll be happy to test it out and see how it performs.
Does it support multiple remote video feeds?

/azp run

I believe all issues have been remedied. Except the documentation. Should that be put somewhere in this PR? Or should I make a PR to gh-pages branch and edit this page:
https://microsoft.github.io/MixedReality-WebRTC/manual/installation.html

@rygo6-MS I have quickly checked the Android build pieces, see a few minor things to fix.
The documentation is the docs folder, the gh-pages content is autogenerated from the MD files in docs. The info about the native renderer should probably go into docs/manual/unity/unity-mediaplayer.md . Also it's probably worth adding a link from docs/manual/unity/unity-integration.md. It can be done in a separate PR anyway, this can be merged as soon as the last comments are addressed.

Hi! Is it supposed to run also on HL2? can you tell something about the performance gain that it can provide? For example for a 1080p video stream how much milliseconds we can gain for each frame rendering?
Just a curiosity.
Thanks a lot!

Hi! Is it supposed to run also on HL2? can you tell something about the performance gain that it can provide? For example for a 1080p video stream how much milliseconds we can gain for each frame rendering?
Just a curiosity.
Thanks a lot!

I have not tested if this works on HL2 yet, but I can't think of a reason why it wouldn't, and it should. I will be testing it this week.


Add support for backfill_from using static data

This fixes the error
Traceback (most recent call last):
  File "/Users/jbales/Personal-Repos/algorithmic-trading/main.py", line 114, in <module>
    main()
  File "/Users/jbales/Personal-Repos/algorithmic-trading/main.py", line 74, in main
    cerebro.run()
  File "/usr/local/lib/python3.7/site-packages/backtrader/cerebro.py", line 1127, in run
    runstrat = self.runstrategies(iterstrat)
  File "/usr/local/lib/python3.7/site-packages/backtrader/cerebro.py", line 1210, in runstrategies
    data._start()
  File "/usr/local/lib/python3.7/site-packages/backtrader/feed.py", line 203, in _start
    self.start()
  File "/usr/local/lib/python3.7/site-packages/alpaca_backtrader_api/alpacadata.py", line 198, in start
    self.p.backfill_from._start()
  File "/usr/local/lib/python3.7/site-packages/backtrader/feed.py", line 206, in _start
    self._start_finish()
  File "/usr/local/lib/python3.7/site-packages/backtrader/feed.py", line 196, in _start_finish
    self._calendar = self._env._tradingcal
  File "/usr/local/lib/python3.7/site-packages/backtrader/lineseries.py", line 461, in __getattr__
    return getattr(self.lines, name)
AttributeError: 'Lines_LineSeries_DataSeries_OHLC_OHLCDateTime_Abst' object has no attribute '_env'

The solution is already implemented in IBData here /backtrader/feeds/ibdata.py
This allows for backfilling in a batch with something like Alpha Vantage data then proceeding with live alpaca/polygon data.
Reference material
Backtrader Community Forums
Backtrader IBData live data docs

This works, it backfills the data, but I must be missing something since the hope was that it would speed up the startup time by getting all historical data in a batch. But it keeps pinging Alpaca during the backfilling for about the same amount of time.

I will need to do some testing first to understand this a bit better

This works, it backfills the data, but I must be missing something since the hope was that it would speed up the startup time by getting all historical data in a batch. But it keeps pinging Alpaca and sleeping 3 seconds during the backfilling, taking about the same amount of time overall.

so if I understand this correctly, this seems like a good fix. but you have doubts? did you get any other insights?

Exactly. When I backfilled with 5 days of dense data, it merged with the 2~ days of historical data pulled in packets from Alpaca every few seconds.
It's a fix to allow backfilling, but just didn't fix what I hope.
I would like a way to just use my backfilled data and not use Alpaca/Polygon since it says 30min-hour

so where is it stored? your backfilled data

I'm merging this PR. your issue, if I can assist somehow - let me know through an issue. I will try to assist if I can


Bug returning bool

Please, review bug solving. Missing returning bool value in local context of the f(x). f(x) Has been defined to return a bool value but no return exists prior to this solution and no return false case has been considered in solution due to f(x) meaning.

This PR should merge into the other branch.
Remove all the VS code related files and add them to gitignore.

Done.

Still pointing to main. I am not going to review this until is pointed to the correct location.

Done.
But GitHub has created PR as a new one. Can we delete current one?


fixup: Report failure message if we failed to build bazel

Java may crash while compiling bazel if you only have 2GB RAM or less.
Do not believe the return value of compile.sh. Check the output binary
instead.
This patch will make it easier for me to debug what breaks my build process on my VMware machine with less than 2GB RAM configured...
See also: #20

Thanks


Feature/gaufrette storage

This adds Gaufrette support to handle file uploads.
Some improvements can be made, feel free to suggest any necessary changes.
Solves #316

Coverage decreased (-44.4%) to 55.288% when pulling ca04ee86146cf4344492fea18e82bf303b51d171 on kevinc123:feature/gaufrette-storage into 0b2dcc18d16eac54e0a4df7a1bbd3e811b23939c on craue:master.

Really cool. I like it but I would rather make it one more level of abstraction. This will make possible also to use FlySystem for storage.

@kevincerro @craue Nice feature, I use Gaufrette to store image on AWS because my PAAS does not have a filesystem.
Does this feature will be merged ?
Thanks,

I see a lot of work went into this already. And basically, I also like the idea of being able to save files in different ways. But @Warxcell has a good point on abstraction. Adding adapters for soft dependencies to various libraries would be nice. Tests would be needed as well.
Anyone up for working on this? @kevincerro, are you in the mood to resume?


change speedtest server

I have a doubt, if it is possible to change the speedtest server to the one that does the test. be able to add the id or name of this server
Has anyone been able to do it? which line do i change?
Greetings.

I did that in internet-monitoring, which is the predecessor to internet-pi.
You have to install the docker stuff manually, then the internet-monitoring docker compose files. There is a directory in the code of internet-pi.

Add this in the Speedtest section of the .yml.
environment:
- SPEEDTEST_PORT= #optional
- SPEEDTEST_SERVER= #optional

Thank you very much dear friend, I will review what you say and I will apply it. thank you so much

I went back and looked at the internet-pi install and there is an internet-monitoring directory beneath your home directory.
yourname@rpi4B:~/internet-monitoring $
There you will find...
drwxr-xr-x 3 drw drw 4096 Nov 25  2022 blackbox
-rw-r--r-- 1 drw drw 2148 Sep  8 06:40 docker-compose.yml
drwxr-xr-x 3 drw drw 4096 Nov 25  2022 grafana
-rw-r--r-- 1 drw drw 5241 Nov 25  2022 Grafana-Org-Stats.json
drwxr-xr-x 2 drw drw 4096 Aug 19 20:12 prometheus
-rw-r--r-- 1 drw drw 3481 Nov 25  2022 README.md
-rwxrwxrwx 1 drw drw   98 Aug  4 11:16 update.sh
In docker-compose.yml you will find a section...
speedtest:
expose:
- 9798
ports:
- 9798:9798
image: miguelndecarvalho/speedtest-exporter
restart: always
networks:
- back-tier
environment:
- SPEEDTEST_SERVER=36684
I added those last two environment lines.
update.sh is a script I created to update after the edit.  It contains...
docker-compose pull
docker-compose down
docker-compose up -d --no-deps
docker system prune --all
36684 is Ga Tech's speedtest server.  Use whatever you want.  Without specifying a server mine defaults to VZ Nashville.  It must be under provisioned, because it fails a lot.
I hope this helps.
Dennis

VZ finally fixed the issue with their Nashville Speedtest.net server.

Problem is back.


Can we upload a delete nuget package back with same name into the same nuget feed

Can we upload a delete nuget package back with same name into the same nuget feed.

Any update on my ask

@Bangarraju1005 Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Please consider these resources:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow


GUACAMOLE-1905: Revert to Java-8-compatible Logback 1.3.14.

As discussed in https://github.com/apache/guacamole-client/pull/954, this is the latest Java 8 compatible version of Logback.

Oops.


[BUG] CognitiveServicesAccountCollection.GetAll throws System.FormatException in case Encryption identityClientId was not set

Library name and version
Azure.ResourceManager.CognitiveServices 1.2.1
Describe the bug
I created a Cognitive Account with Terraform script. I wanted to configure Encryption with a Key Vault key.
I configured the customer managed key with a following blocks. The identity client id is optional and I couldn't set in case of System Assigned identity.
resource "azurerm_cognitive_account_customer_managed_key" "cacmk-bug-report" {
  cognitive_account_id = azurerm_cognitive_account.ca-translation.id
  key_vault_key_id     = azurerm_key_vault_key.kvk-translation-key.id
  // identity_client_id   = "" optional parameter
  depends_on = [
    azurerm_key_vault.kvbugreport,
    azurerm_cognitive_account.ca-translation,
  ]
}
When I wanted to query this Cognitive Account with a following c# code I got System.FormatException because the identityClientId is empty string in api response:
var resourceGroupName = "rg-bug-report";
var subscriptionId = "<your subscription>";
var tenantId = "<your tenant id>";
var clientId = "<your client id>";
var clientSecret = "<your client secret>";
var clientSecretCredential = new ClientSecretCredential(tenantId, clientId, clientSecret);
var armClient = new ArmClient(clientSecretCredential);
var subscriptionResourceId = new ResourceIdentifier($"/subscriptions/{subscriptionId}");
var subscription = armClient.GetSubscriptionResource(subscriptionResourceId);
var resourceGroups = subscription.GetResourceGroups();
var resourceGroup = resourceGroups.Get(resourceGroupName);
var cognitiveServiceAccounts = resourceGroup.Value.GetCognitiveServicesAccounts();
foreach (var cognitiveServiceAccount in cognitiveServiceAccounts.GetAll())
{
    Console.WriteLine(cognitiveServiceAccount.Data.Name);
}
The Cognitive Account works as expected so this property really seems optional.
Expected behavior
The cognitiveServiceAccounts.GetAll() read the data of the cognitive account without exception.
Actual behavior
Throws System.FormatException in case Encryption identityClientId was not set
Reproduction Steps
The attached a zip file contains the terraform script which create an infrastructure in azure and minimal command line application which wants to query it.
Both script uses service principal credentials for authentication
BugReport.zip
Environment
.NET SDK:
Version:   7.0.202
Commit:    6c74320bc3
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.22621
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\7.0.202\
Host:
Version:      7.0.4
Architecture: x64
Commit:       0a396acafe
.NET SDKs installed:
7.0.202 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86   [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @fuliii1 , Thank you for using Azure SDK for .NET.
We can't repro your issue with the code you provide.
If possible, please capture and attach the request/response payload, it could help us to identify the issue.


add ability to change loguru msg verbosity

Adding mechanism to enable applications to change the verbosity level at
which loguru logs information messages it generates.

@emilk, this allows applications using loguru can have loguru generate its information messages at higher verbosity level than INFO. Please feel free to recommend a different variable name, if you'd prefer. I wasn't sure what to name it.

This is a nice addition! I think maybe g_internal_verbosity might be more descriptive

@emilk var name changed as requested. thanks.


cargo requires Wayland ?

This is kind of odd but this crate is the only of about 20 that requires wayland.  So adding this crate forced me to add wayland drivers. Which is like-- fine,  its just a bit strange that this crate breaks x11 compatibility when it doesnt really involve the actual window.

This is kind of odd but this crate is the only of about 20 that requires wayland. So adding this crate forced me to add wayland drivers. Which is like-- fine, its just a bit strange that this crate breaks x11 compatibility when it doesnt really involve the actual window.
I think this is a bit passive-aggressive but...

I am not a Linux desktop user, I don't know why X11 is good or Wayland is not or anything related to that. Docs.rs were not able to generate documentation due to compilation errors:
https://docs.rs/crate/bevy_mod_billboard/0.5.1
So, it was fixed in this commit with me adding X11 & Wayland features, because I saw it mentioned somewhere:
https://github.com/kulkalkul/bevy_mod_billboard/commit/5424b6441bf14e294c74e16706655921b06d1283
In any case, I don't have time to look at it (and I still need to take care of 0.15 PR, which is my work queue). So, if you can solve the issue without it bringing docs.rs issue back, I'm happy to accept PR.


my bold and italic button not working

can someone give sulution? thanks

The same issue.

I had the same issue, turn out this was being caused by my CSS framework. Make sure no CSS framework is changing strong's font-weight to a value of 500 or below. By manually setting the font-weight of the strong tag to 700, the text showed as bold.
I imagine the same thing could be happening with your italic tag.

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 7 days since being marked as stale.


Vue 单文件，获取不到 AsyncFunction 对象

Version
3.2.6
Reproduction link
sfc.vuejs.org/
Steps to reproduce

What is expected?
ƒ AsyncFunction() { [native code] }
What is actually happening?
ƒ () { [native code] }

Your reproduction seems to be incorrect.
<script>
export default {
  created () {
    const asyncFunction = Object.getPrototypeOf(async function () { })
    console.log(asyncFunction.constructor)
  }
}
</script>

See here.
And it doesn't seem to be related to Vue.

请问找到问题了吗，我也是一样的问题

请问找到问题了吗，我也是一样的问题

没找到


Missing models during client generation

Problem statement
Generated client library fails to build due to missing models that aren't generated.
There's a models modules referenced in one of the generated files (client/webhooks/webhooks_create_responses.go), but it's not generated, therefore not available.
No matter what combination of command line flags I use I'm not able to generate the missing models. The spec validates successfully using swagger validate ... but not sure if there's perhaps something odd about it.
Any help or pointers would be greatly appreciated, thanks.
Swagger specification
https://api.cloudsmith.io/?format=openapi
Steps to reproduce
Create the following files:
go.mod
module github.com/cloudsmith-io/cloudsmith-go

go 1.14

main.go
package main

import "github.com/cloudsmith-io/cloudsmith-go/client"

func main {
	client.New(nil, nil)
}

Then run:
curl -o swagger.json "https://api.cloudsmith.io/?format=openapi"
swagger generate client -f swagger.json
go build

Results in the output:
go: finding module for package github.com/go-openapi/strfmt
go: finding module for package github.com/go-openapi/runtime
go: finding module for package github.com/go-openapi/swag
go: finding module for package github.com/go-openapi/runtime/client
go: finding module for package github.com/go-openapi/errors
go: finding module for package github.com/cloudsmith-io/cloudsmith-go/models
go: finding module for package github.com/go-openapi/validate
go: found github.com/go-openapi/runtime in github.com/go-openapi/runtime v0.19.15
go: found github.com/go-openapi/strfmt in github.com/go-openapi/strfmt v0.19.5
go: found github.com/go-openapi/errors in github.com/go-openapi/errors v0.19.4
go: found github.com/go-openapi/swag in github.com/go-openapi/swag v0.19.9
go: found github.com/go-openapi/validate in github.com/go-openapi/validate v0.19.8
go: finding module for package github.com/cloudsmith-io/cloudsmith-go/models
client/webhooks/webhooks_create_responses.go:19:2: cannot find module providing package github.com/cloudsmith-io/cloudsmith-go/models: module github.com/cloudsmith-io/cloudsmith-go/models: git ls-remote -q origin in /Users/paddy/go/pkg/mod/cache/vcs/b2f1ca9505e5604c8ac0e6d8cf2590175ec860c989e60b0f5ea99e975f9207be: exit status 128:
	fatal: could not read Username for 'https://github.com': terminal prompts disabled
Confirm the import path was entered correctly.
If this is a private repository, see https://golang.org/doc/faq#git_https for additional information.

Environment
swagger version: v0.23.0
swagger commit: 3596b40bedd4f06cc918767098568456d2ec2370
swagger container: quay.io/goswagger/swagger:v0.23.0
go version: go version go1.14.3 darwin/amd64
OS:  OSX 10.13.6

@paddycarey  it looks from your spec that all schemas are inlined with the paths definitions. In situations like this, goswagger sets out to generate all inlined things in the operations package rather than in models.
The fact that there is still a reference to some model package looks like a bug. Needs to be investigated - your spec is quite large... -.
As a workaround, you might give a try to the --with-flatten=full  option which externalizes inlined objects to definitions, hence models.

@paddycarey  I've investigated your issue over the last (rainy) weekend and it stems for a name conflict when generating the anonymous structures (aka inline).
This is an interesting spec, but it looks like it will be difficult to use to develop a server, since all models are anonymous, hence no structure sharing. To an external eye, it looks like this spec has been generated and expanded.
Nevertheless, go-swagger should be able to process it and this is a bug.
As far as I can see, there is only one set of operations that is sensitive to this bug: in webhooks, all operations define an inline array Templates.  The generation of the items structures in the operations/webhooks package
A proper way to work around this would be to define the common definition of the template items under "definitions" and refer to it with something like "$ref": "#/definitions/TemplateItems".
Renaming the different "templates" with different names also works...
So now you have a workaround. The bug is indeed complex to solve.

Hi @fredbi, thanks for looking into this, it's really helpful!

This is an interesting spec, but it looks like it will be difficult to use to develop a server, since all models are anonymous, hence no structure sharing. To an external eye, it looks like this spec has been generated and expanded.

In this case we're only trying to generate a client, so hopefully no issues with server generation, though I understand the concern. The spec is generated from a custom Django Rest Framework setup, which we're in the process of updating to be closer to mainline, so hopefully that will resolve some of the weirdness with the spec.
Thanks for the workarounds, I'll check them out. The PR also looks promising, i'll check that out too.
Thank you for your work here, it's much appreciated!


ATA pipeline dev: monde_b.h #define for static values of beamformer configuration

With the #defines proposed, there is a single header source for the values of the beamformer configuration, which is static in C. This simplifies the compilation and sanity checking of the super process (hpguppi_daq).
I'm open to modifying the names of the #define macros.

I've now moved the added #define macros to a new C header (mode_b_config.h) which has a conditional definition of BLADE_OUTPUT_ELEMENT_T, in order to depend the type definitions of buffer_E and output on a single line (the #define BLADE_OUTPUT_NCOMPLEX_BYTES).
I think we could come up with a better/more robust way to conditionally define the output element type, perhaps just with more sanity checks.. But this is a start.


Don't test pkg/client/ and third_party/ packages

Don't run tests for generated files
Release note:
NONE

/approve


Make full use of the TCR metric API

A lot of the current code outside of the tcr_metric module still uses non-general pdist and cdist code that expects the input to be a string. The tcr_metric API is more general but as a result expects the pyrepseq standard TCR dataframe format as input. If we do take the whole model API thing seriously, it would only make sense to adjust the rest of the code to make use of this, otherwise all the plotting and clustering code still relies on the legacy pdist and cdist and you cannot swap models around freely.

@andim on the linked branch I have:

abstracted the tcr_metric module to a metric module with the tcr_metric being a submodule
implemented the fully abstracted metric interface
implemented Levenshtein (and WeightedLevenshtein) as implementors to the general metric interface
wrote tests for all added code, which are passing
replaced the pdist and cdist in pyrepseq.distance and pyrepseq.plotting with the Levenshtein metric implementation

All is looking good, all tests are passing.
However, the functions that use pdist and cdist in both distance and plotting are not being tested- in fact many functions seem to be completely untested. What I have done is done the minimal replacement of the original calls to pdist and cdist to the new implementation in a way that should work, but I think it would be better if these functions had at least some rudimentary tests.


FINERACT-893-fixed-deposit-rollover

Description
Fixed deposit maturity instructions added. 1 reinvest maturity amount 2. reinvest principal 3. transfer to savings 4, withdraw
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Commit message starts with the issue number from https://issues.apache.org/jira/projects/FINERACT/. Ex: FINERACT-646 Pockets API.

[x] Coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions have been followed.

[ ] API documentation at https://github.com/apache/fineract/blob/develop/api-docs/apiLive.htm has been updated with details of any API changes.

[x] Integration tests have been created/updated for verifying the changes made.

[x] All Integrations tests are passing with the new commits.

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the list.)

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

@fynmanoj re. build failure, please subscribe to and watch (or help fix??) #764 for FINERACT-852...

@fynmanoj Following #764 and #761 this will pass the build (or not at least not fail because of integration tests) if you rebase this now.

@fynmanoj now it failed due to https://issues.apache.org/jira/browse/FINERACT-899 .. can you help fix that one? (In a separate PR I mean, not as part of this.) That would be amazing. Those test failures are really a problem for this project.

I just did a quick technical review of this PR, and there is nothing shocking here, and it includes an Integration Test - this would get LGTM from me, once it actually passes the build. - It's a functional Improvement which, ideally, someone with domain expertise should review more. If we don't find anyone interested, I propose we would give this a 2 weeks grace period, and then merge if no one voices any objections....

@fynmanoj you have to yourself figure out how to get this to pass the build. See also #782. I'll have to let you do the require rebase, check build status, etc. The project maintainers cannot check each PR and close and re-open PRs etc.

@fynmanoj Thank you for your PR. By looking at you test cases I assume this PR supports two feature:

Reinvesting FD with Principal + interest 2) Reinvesting Just the Principal part of it.

Would you be able to document Business Use cases, to explain this feature to community. This would help others to use your implemented feature and test it before we release it.
Also I don't see any API changes- is this correct. If there are API changes can you document them or perhaps add to API documentation.
Lastly, do you intend to develop UI support for this feature?

@Ippezrobert Can you explain the business use case  feature to the community by enhancing FINERACT-893?
@nikpawar : Yes, corresponding UI PR has been sent at link - https://github.com/openMF/community-app/pull/3194. It has a screenshot.
I have reviewed the functionality and it's good to be merged from both functional and technical aspect. Disclosure : Manoj works with me.

MFI has a max of 1 yr Deposit Term but a client would wish to fix for 2 yrs, so the MFI accepts this offer and manages the 2 yrs period in shifts internally. So after the 1 yr is done, the MFI would like the rollover to be managed automatically by the system since chances that a new FD will be created exactly on the same date of maturity is small since an officer can forget to check if this FD is matured or not yet a client would be counting when the two years would end. So there needs to be a job scheduler to check maturity date for FD and also rollover options so that it (system can take appropriate decisions (to reinvest of just close and transfer interest and principal to linked savings).
To be discussed: Will the system use the same account number or new one will be created????

A client would like to reinvest FD voluntarily since he sees no use for his money for some time. He then has to decides what amount or portion to reinvest after the first maturity is expired. Option can include rollover of principal only, rollover of principal and interest, transfer of balance to another account (make linked account dynamic .ie ability to select loan, shares or savings account and also a GL Account), etc

NB:

If the selected option involves rollover, a Rollover End Date should be entered i.e the date when you want the rollover transaction to stop.
If the option selected in After Maturity does not involve the roll over of interest with the principal, the option of linked account is mandatory .ie account to which the transfer will be made at maturity. Like i said this can be a loan, share or savings account but should belong to this client or it can be GL Account.

Hope my explanation helps
Regards

MFI has a max of 1 yr Deposit Term but a client would wish to fix for 2 yrs, so the MFI accepts this offer and manages the 2 yrs period in shifts internally. So after the 1 yr is done, the MFI would like the rollover to be managed automatically by the system since chances that a new FD will be created exactly on the same date of maturity is small since an officer can forget to check if this FD is matured or not yet a client would be counting when the two years would end. So there needs to be a job scheduler to check maturity date for FD and also rollover options so that it (system can take appropriate decisions (to reinvest of just close and transfer interest and principal to linked savings).
To be discussed: Will the system use the same account number or new one will be created????
A client would like to reinvest FD voluntarily since he sees no use for his money for some time. He then has to decides what amount or portion to reinvest after the first maturity is expired. Option can include rollover of principal only, rollover of principal and interest, transfer of balance to another account (make linked account dynamic .ie ability to select loan, shares or savings account and also a GL Account), etc

NB:

If the selected option involves rollover, a Rollover End Date should be entered i.e the date when you want the rollover transaction to stop.
If the option selected in After Maturity does not involve the roll over of interest with the principal, the option of linked account is mandatory .ie account to which the transfer will be made at maturity. Like i said this can be a loan, share or savings account but should belong to this client or it can be GL Account.

Hope my explanation helps
Regards

Thank you @Ippezrobert @avikganguly01  for your time

@vorburger ,  kindly let me know how to proceed

@vorburger , kindly let me know how to proceed

@fynmanoj see https://github.com/apache/fineract#pull-requests Basically, do not wait for an active maintainer to come and help you re. build failures - it's your responsibility to get a green build. Have you checked the log, and done what the README suggests to do in such cases? :smiley_cat:

@fynmanoj ..Please fix merge conflicts as a result of other merged PRs.

done!

@vorburger @fynmanoj Is there still a build fail that needs to be addressed before this can be merged?

/rebase

rebase done

@fynmanoj OK, and it's still failing... you need to get it to pass! That's not something we can do for you, but your role. We recently explained the process better on https://github.com/apache/fineract#pull-requests. Have you actually investigate this problem locally? Hint: Just run ./gradlew bootRun yourself, and you'll immediately see the problem... but to contribute to this project, you should figure this out yourself, not wait for 8 days.
@edcable it's each contributors responsibility to get their PRs to build.

As the local build was successful and the test cases I introduced were working fine. It was clear that the issue was external. As there were integration tests being ignored, I was expecting this issue to get fixed eventually.
@vorburger , I totally got you.  Let me see to fix this from my end

Fixed all the issues, Kindly merge this PR

Looks good to me. Functionality is addressed correctly based on maturity instruction requirements.
@Ippezrobert

MFI has a max of 1 yr Deposit Term but a client would wish to fix for 2 yrs, so the MFI accepts this offer and manages the 2 yrs period in shifts internally. So after the 1 yr is done, the MFI would like the rollover to be managed automatically by the system since chances that a new FD will be created exactly on the same date of maturity is small since an officer can forget to check if this FD is matured or not yet a client would be counting when the two years would end. So there needs to be a job scheduler to check maturity date for FD and also rollover options so that it (system can take appropriate decisions (to reinvest of just close and transfer interest and principal to linked savings).

Correct.

To be discussed: Will the system use the same account number or new one will be created????

System will create new FD account with new account no at the time of maturity.

A client would like to reinvest FD voluntarily since he sees no use for his money for some time. He then has to decides what amount or portion to reinvest after the first maturity is expired. Option can include rollover of principal only, rollover of principal and interest, transfer of balance to another account (make linked account dynamic .ie ability to select loan, shares or savings account and also a GL Account), etc

Scope of this feature was to address maturity instructions. Voluntary reinvest feature can be enhanced separately. Only savings account can be linked - not loan, share or GL account. Principal, Interest & P&I are the options - partial amount support (flat/%?) wasn't communicated earlier as well.

If the selected option involves rollover, a Rollover End Date should be entered i.e the date when you want the rollover transaction to stop.

This does seem to be a design flaw.  All of us missed this observation. Workaround can be manually closing the FD to prevent rollover instructions triggering for a 2nd time.

If the option selected in After Maturity does not involve the roll over of interest with the principal, the option of linked account is mandatory .ie account to which the transfer will be made at maturity.

Correct.


TS1005 errors for non javascript files (.cshtml)

Bug Report
Get a warning when adding a certain javascript snippet
Warning TS1005 (JS) ‘;’ expected. Foo.Web JavaScript Content Files
🔎 Search Terms
TS1005
🕗 Version & Regression Information
Using visual studio 2019 16.8.5 Typescript version 4.1.2
💻 Code
When I add the following code to my .cshtml file I get
Warning TS1005 (JS) ‘;’ expected. Foo.Web JavaScript Content Files

Stock Market Today by TradingView

You can see documentation here:
https://www.tradingview.com/widget/market-movers/
🙁 Actual behavior
Warning TS1005 (JS) ‘;’ expected. Foo.Web JavaScript Content Files

🙂 Expected behavior
There shouldn't be any errors, it's not a typescript file, furthermore no option is given to me so that I can suppress the error

This is a correct error in the context of a JavaScript script block. Starting with an { like that introduces a statement block, not an object literal - put parentheses around the whole thing if you want it to be an object.

This is a correct error in the context of a JavaScript script block. Starting with an { like that introduces a statement block, not an object literal - put parentheses around the whole thing if you want it to be an object.

Yep I see, thanks for that

Yep I see, thanks for that

No, this is not a solution, wrapping parentheses indeed removes TS error warnings but it is because it just ignores what is inside that wrapping and widget uses default predefined values


Fix handling of first VAD timestamp

When the initial RTP timestamp is very far from 0, it is thought to be from the previous epoch of 32bit integers
after an integer wrap around. When we do not have any saved audio levels, we know we should start processing.

@Qizot Changing the maximum random timestamp used by the test makes it so that the existing tests sometimes fail. I'm guessing this might be where property tests would be useful, but I'm not familiar with writing those.

What do you think about initializing current_timestamp in state to nil and then in handle_process automiatically do something like this current_timestamp = state.current_timestmap || rtp_timestamp.
With this approach when we are handling the first buffer the epoch will indicate the current and then we can probably change > to >= and make the first cond statement pass as we will have the same timestamp?

Particularly with RED starting to come through, I'd be worried about changing the condition to >=, as it could needlessly add a lot of things to the queue. I do like the idea of starting nil, though—the epoch function can special case it to be :current, and remove one of the conditions.

Particularly with RED starting to come through, I'd be worried about changing the condition to >=, as it could needlessly add a lot of things to the queue. I do like the idea of starting nil, though—the epoch function can special case it to be :current, and remove one of the conditions.

Yup, makes sense with the RED packets. Looks great right now 🥳


Authenticating iOS and RADIUS - Only PAP is used

For Authenticating i use Radius-Authentication. If I enable PAP-Authentication everythink works fine. But i like to diasble PAP, because it's quite unsave. In the Windows L2TP Client a can select the authentication mehtods used, thus the authentication works also fine. But iOS always use the first proposed authentication mehtod. So the VPN-Server accepts PAP and iOS answers in PAP but the Radius server will then deny the authention. There was a simaliar on SonicWall Gateways (https://support.software.dell.com/de-de/kb/sw9013).
To fix this issue it would be greate to have a option to disable PAP completely or to choose the order of proposed  authentication methods.

Any update on this at all? The native connection protocol though the SoftEther client doesn't work with RADIUS auth because it uses PAP, which is inexcusable in this day and age for password authentication.

If the L2TP client chooses PAP, Radius auth will use PAP as well. This should not be an issue because SoftEther server DE is now proposing to use EAP first.
The native protocol will be able to use EAP / PEAP after #1775 .

Using PAP with Radius should not pose a security risk because Radius implementation is always local, otherwise it's a bad implementation.


updated tunnelblick (3.6.0a_build_4543.4546)

According to the release notes, Tunnelblick 3.6 no longer functions on OSX 10.4 and 10.5.  For those versions, Tunnelblick 3.5 is recommended.   Should this be listed in the caveats, or perhaps a versioned installation added?

Updated PR with versioned installation.

Thank you for the contribution. It was merged directly as 0a2eda9a99fb8c932c5ce1aff9b0cd998e906d67 to keep commit history cleaner. Your contribution is still credited to you.


Some strings cannot be translated

FilePrompt.java#L16, L34, L51 and L63

JarFileTree.java#L35, L58, L64 and L67

AccessPanel.java#L24-L27 and L45

DefinitionBox.java#L63 and L96

MemberTable.java#L29, VariableTable.java#L37 and ClassDisplayPanel.java#L224

Yeah, I've missed a few things here and there...

Last few commits have fixed all of these. I've attempted to use existing translations to fill in the new values. If I messed them up do correct me.


Add docker-tools role to install docker-compose globally

I need this for the IDR so created this role based on https://github.com/openmicroscopy/infrastructure/blob/master/ansible/roles/devspace/tasks/main.yml
If everyone's happy I'll update roles/devspace to depend on this.
/cc @aleksandra-tarkowska

Interestingly, I had done this earlier today:
diff --git a/ansible/roles/docker/tasks/main.yml b/ansible/roles/docker/tasks/main.yml
index 7186aec..c0fd945 100644
--- a/ansible/roles/docker/tasks/main.yml
+++ b/ansible/roles/docker/tasks/main.yml
@@ -11,8 +11,23 @@
 - name: upstream docker | install docker
   become: yes
   yum:
-    pkg: docker-engine
+    pkg: "{{ item }}"
     state: present
+  with_items:
+    - docker-engine
+    - python-devel
+    - python-pip
+
+- name: install python packages
+  become: yes
+  pip:
+      name: "{{ item }}"
+      #extra_args: "--upgrade"
+      state: latest
+  with_items:
+    - pip
+    - pyyaml
+    - docker-compose

@joshmoore From past experience with multiple dependencies automatically installed using pip (which you might not realise) I though it's better to keep it separate, especially if you've got more than just docker installed.

https://github.com/openmicroscopy/infrastructure/pull/64

No objections, but I don't understand what should be kept separate from what.

Whenever you install anything outside yum there's a risk of installing something that'll conflict, especially pip which installs python dependencies which can conflict with distro python-module rpms

@joshmoore docker-py has to be installed after docker-compose and capped to 1.9.0 https://github.com/ansible/ansible/issues/17495

@aleksandra-tarkowska It works for me, see https://github.com/openmicroscopy/infrastructure/pull/131/commits/8f8f5ba04170e53ee0a2eb9f4de0639d6f421157
@joshmoore Ideally this would go into a virtualenv, but then you need to figure out how to get ansible to pick it up.

@manics sorry I missed the fact that you capped compose on 1.8.0.

@manics is no longer using this strategy, but it's useful for devspace.


How to make the charts responsive?

Hi I'm wondering how to make the charts responsive to its containing parent?
Thanks!

Have a look here maybe that helps:
https://github.com/codesuki/react-d3-components/issues/9

Super fast reply! Thanks! It helps :)


Peristimulus Time Histogram

This PR adds the function psth, which calculates the peristimulus time histogram, to the statistics module.

Coverage decreased (-0.13%) to 99.22% when pulling f74c20c80082f3b01018899c095218b2b84ab7c9 on alperyeg:psth into 634dfd21fbb575d97ddfcda61c90fbf40d0eec03 on NeuralEnsemble:master.

Coverage decreased (-0.13%) to 99.22% when pulling f74c20c80082f3b01018899c095218b2b84ab7c9 on alperyeg:psth into 634dfd21fbb575d97ddfcda61c90fbf40d0eec03 on NeuralEnsemble:master.

Coverage decreased (-43.39%) to 55.96% when pulling cdf920453e85f8e7f590272eb5ab8b8f5390d888 on alperyeg:psth into 634dfd21fbb575d97ddfcda61c90fbf40d0eec03 on NeuralEnsemble:master.

Coverage decreased (-0.06%) to 99.29% when pulling 2f0040658b6f1e2d1c4bb66c3c3f1dfc3edb9e85 on alperyeg:psth into 634dfd21fbb575d97ddfcda61c90fbf40d0eec03 on NeuralEnsemble:master.


hide source code ?

Hi
Is it possible to hide web application source code when we publish it for an install ?
Like we have a web application but it need to be install in every clients like with IIS and not in one server like normal website.
But all source code is visible.

What version of ASP.NET Core are you using?

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


A stored cross-site scripting (XSS) vulnerability exists in LightCMS "contents" field

A stored cross-site scripting (XSS) vulnerability exists in LightCMS  that allows an user authorized to upload a malicious .pdf file which acts as a stored XSS payload. If this stored XSS payload is triggered by an administrator it will trigger a XSS attack.

login as admin in the article page

create a new article

upload the malicious pdf. the content of xss.pdf :

%PDF-1.4
%1111
1 0 obj
<<
/CreationDate (D:20210619104632+08'00')
/Creator (xss)
/Producer (PDF-XChange Core API SDK \(7.0.324.2\))
>>
endobj
2 0 obj
<<
/Metadata 3 0 R
/Pages 4 0 R
/Type /Catalog
>>
endobj
3 0 obj
<<
/Length 2983
/Subtype /XML
/Type /Metadata
>>
stream
<?xpacket begin="" id="W5M0MpCehiHzreSzNTczkc9d"?>
<x:xmpmeta xmlns:x="adobe:ns:meta/" x:xmptk="XMP Core 5.5.0">
	<rdf:RDF xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#">
		<rdf:Description rdf:about=""
				xmlns:dc="http://purl.org/dc/elements/1.1/"
				xmlns:xmpMM="http://ns.adobe.com/xap/1.0/mm/"
				xmlns:xmp="http://ns.adobe.com/xap/1.0/"
				xmlns:pdf="http://ns.adobe.com/pdf/1.3/">
			<dc:format>application/pdf</dc:format>
			<xmpMM:DocumentID>uuid:9c93bc08-8e4e-46cb-b28f-824c693821a4</xmpMM:DocumentID>
			<xmpMM:InstanceID>uuid:2cd63bea-24ca-4ef8-a12c-015da3b28c96</xmpMM:InstanceID>
			<xmp:CreateDate>2021-06-19T10:46:32+08:00</xmp:CreateDate>
			<xmp:CreatorTool>迅捷PDF编辑器 7.0.324.2</xmp:CreatorTool>
			<xmp:ModifyDate>2021-06-19T10:52:02+08:00</xmp:ModifyDate>
			<pdf:Producer>PDF-XChange Core API SDK (7.0.324.2)</pdf:Producer>
		</rdf:Description>
	</rdf:RDF>
</x:xmpmeta>

back to content then wo edit this upload:

when user click the link it will trigger a XSS attack

No better solution have been found except to prohibit users from uploading PDF files, can you give some help to me, thanks.

Adding the “Content-Disposition: Attachment” and “X-Content-Type-Options: nosniff” headers to the response of static files
https://owasp.org/www-community/vulnerabilities/Unrestricted_File_Upload

reference，we recommend the following：

nginx configure the reverse proxy which can add a header to the specified url.

location /{
    if ($request_filename ~* ^.*?.(txt|doc|pdf|rar|gz|zip|docx|exe|xlsx|ppt|pptx)$){
                        add_header Content-Disposition attachment;
                 }
    }

if it is possible，refer to
https://owasp.org/www-community/vulnerabilities/Unrestricted_File_Upload  --- It is recommended that this practice be performed for all of the files that users need to download in all the modules that deal with a file download.currently we do this.

3.nginx detects the uploaded pdf and find the xss features such as 'app.alert(....)'.
Hope can help you！


card.io or PayPal logo

i am integrating card.io into my client project and he doesn't want to show neither card.io nor PayPal.
do we have any legal concerns?
-Sam

I am not a lawyer, and I can't answer any legal questions.
card.io is open source under a liberal license, so modifying the library and using it is generally permitted.
All that said, we (the authors of card.io) have historically asked that you show one of the two logos out of respect for the fact that it has been provided free and open source. (If your client would prefer, we could probably arrange for a Braintree logo instead of PayPal or card.io.)


Add jdk10 docker support

test build: https://ci.adoptopenjdk.net/job/openjdk10_build_docker/3/console

I've added a commit to your PR that adds zulu-9 as we need a jdk9 boot java

okay so it took a few commits but we are there! zulu-9 installed https://ci.adoptopenjdk.net/job/openjdk10_build_docker/8/console.
Seeing the following failure though:
Not reconfiguring due to the presence of config.status in /openjdk/build
Building the JDK: calling 'make images'

/bin/mkdir: missing operand
Try '/bin/mkdir --help' for more information.
make: *** No rule to make target 'images'.  Stop.
/openjdk/sbin/build.sh: line 224: [: : integer expression expected

It's worth noting that we are seeing this on our jdk9 docker builds too so it is probably a bug in our existing code somewhere

@gdams I have had similar issue, this isn't an issue when we run it native but inside the Docker container, try checking the version of bash between the two environments: it might be expecting that line to read as if [[... ]];..., see https://github.com/AdoptOpenJDK/openjdk-build/issues/174

Thinking about the boot JDK.  Did we want to use an LTS release for the boot ?
Using 8 for 9, 10 and 11... and then using 11 for 12, 13, 14, 15, 16 & 17 ?
The LTS will be the only versions with security fixes applied for sustained period, so using the previous version for 11 will currently be using 10 which has no support for the entire life of 11. where as 8 will have security fixes until at least 2025. Thoughts ?

That’s an interesting point that you make.....

@stevewallin Probably no.  This is because a version (say 13) may need to rely on a feature in the previous non LTS version (say 12).

I'm not sure how good an idea it is to keep insecure JVMs around for 8 years+ on build machines.  Given it's mostly used to bootstrap the javac compile stage - and I don't think it heavily uses new features the LTS releases might be better ?
Would be a good topic to discuss further at OpenJDK ?

AFAIK the configure step currently fails for jdk10 builds unless you are using a jdk9 boot jdk

are we okat to land this and we can discuss boot JDK's in a separate issue?

Good with me
On Sat, 16 Dec 2017, 15:21 George Adams<EMAIL_ADDRESS>wrote:

are we okat to land this and we can discuss boot JDK's in a separate issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AdoptOpenJDK/openjdk-build/pull/200#issuecomment-352189843,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQzahpwf8u0QDhEbxHQqDU_K2hqX-HAdks5tA9_rgaJpZM4RCoGh
.

@stevewallin I'm guessing that with some of the new bytecodes they're introducing in non LTS releases that javac may need to rely on on non LTS release bootstrap JDKs, but lets check over there.


use padding and marging for requirements as well

fixes: #1462
Setting the style of a requirement to do the same as headers (padding for the banner)
Before the fix: https://azure.github.io/azure-sdk/dotnet_introduction.html#dotnet-service-return-model-public-getters
After the fix: https://vhvb1989.github.io/azure-sdk/dotnet_introduction.html#dotnet-service-return-model-public-getters

@weshaggard  @jongio  please review and approve whenever possible for you.
Thank you

This looks right on a first load of the site.  I noticed I get some strange behavior if I reload....
First load:

1st reload:

2nd reload:

I noticed I get some strange behavior if I reload

My bad @annelo-msft , I can only have one Demo at the time and I loaded another version with a different fix.  I have fixed it now and you can try again the link: https://vhvb1989.github.io/azure-sdk/dotnet_introduction.html#dotnet-service-return-model-public-getters

That looks great the first time I load it -- thanks!

I'm still seeing the behavior where if I click the reload button, I get a strange result:
1st reload:

2nd reload:

3rd reload:

I'm still seeing the behavior where if I click the reload button, I get a strange result:

It looks like a general issue for all anchors in the design page.  I will create an issue to investigate that one.  I suspect there might be some js script running on page load() which makes page offset.
See for example this first anchor: https://azure.github.io/azure-sdk/dotnet_implementation.html#dotnet-usage-httppipeline
We can keep reloading the page and it keeps somehow letting the expected title insider the screen.
Then look at: https://azure.github.io/azure-sdk/dotnet_implementation.html#json-serialization
Only the first time works, then every reload adds more and more offset to the title.

Issue created for the reloading page with anchor:
https://github.com/Azure/azure-sdk/issues/2445


modify save_img function in pose_utils to produce multiple images as outputs instead of a single image

Can you please share the modifications for saving multiple  images?

Can you please share the modifications for saving multiple images? @bhavesh2799

Sorted !


Add select box in user invite modal form for role type

Can I take this on? Any more info on the issue?

A pr is already submitted for this, maybe check other issue


app-state tab

Work in progress!
This tab shows the app state. Lots of interesting future features to be added, but I'm putting up a PR for comments and discussion.
Currently it's statically showing the app-state, aka the atom db/app-db!
I imagine some of the work here would be useful/reusable for #52.
preview:
{:todos
 {1 {:id 1 :title "eat snacks" :done false}
  2 {:id 2 :title "bike to lake" :done false}}
 :showing :all}

The pretty printing doesn't quite seem to work for me:

@daiyi just wanted to mention before you go too far down this path, that a pretty printing approach, or anything which prints the whole app-db in one go will be too slow to be merged. Wasn't sure if I'd got that across last time we talked, but don't want you to spend too much time on the non devtools formatter version.

Yep, I know we're going to do the formatters now! I was keeping this PR open as a bookmark, but we can close it if you'd like less clutter

You can leave it open if it's useful for you, I'm not too bothered either way. I thought I saw some recent activity on this which I was why I quickly dashed the note off above, but I think I must have been mistaken.


replace travis with github actions

Codecov Report

Merging #18 (8088f34) into master (6d778c8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   92.98%   92.98%
=======================================
  Files           7        7
  Lines         385      385
=======================================
  Hits          358      358
  Misses         27       27

Impacted Files
Coverage Δ

rdap/cli.py
96.15% <ø> (ø)

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 6d778c8...8088f34. Read the comment docs.

Codecov Report

Merging #18 (8088f34) into master (6d778c8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   92.98%   92.98%
=======================================
  Files           7        7
  Lines         385      385
=======================================
  Hits          358      358
  Misses         27       27

Impacted Files
Coverage Δ

rdap/cli.py
96.15% <ø> (ø)

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 6d778c8...8088f34. Read the comment docs.


ARROW-1240: [JAVA] switch logback to version 1.2.3 to address

Does shifting to this version cause us to shift to a different SLF4j version? I know there are some incompatibilities between versions.

Is that @jacques-n? Seems you are logged in with a different GH account

Hah! Yeah, it was me. I'm back to being myself now :)

Well the regression tests all passed, without any other changes.
But I’ve also proposed upgrading slf4j as a separate PR, since that’s also quite out of date.

On 20 Jul 2017, at 17:31, DremioQA<EMAIL_ADDRESS>wrote:
Does shifting to this version cause us to shift to a different SLF4j version? I know there are some incompatibilities between versions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/apache/arrow/pull/871#issuecomment-316758945, or mute the thread https://github.com/notifications/unsubscribe-auth/AB3WlxLdvtMtmrWFjXrikbvlelVM-pe-ks5sP4DfgaJpZM4OeDic.

Sorry I wasn't more clear in my question (and slow to respond). We depend heavily on Arrow and also use Logback (the same version as Arrow). I was asking whether the API of logback and slf4j is different between your proposed versions and the current versions. Being a library, we have to be cautious about when we should change cross dependencies. (Definitely think an update makes sense, need to figure out how to proceed.)

logback's list of changes report the following non-breaking changes between 1.0.13 and 1.2.3:

MDC values are no longer inherited by child threads.
Logback 1.1.4 requires SLF4J version 1.7.16 or later.
Packaging data (as output in stack traces) is now disabled by default.
all logback modules require JDK 1.6 instead of previously JDK 1.5

Arrow uses slf4j 1.7.6, and the latest version is 1.7.25. So assuming logbook versions >1.1.4 also require SLF4J 1.7.16 or later, we need to upgrade slf4j.
I propose to include an slf4j upgrade as part of this change - version 1.7.25 unless anyone suggests otherwise...
SLF4J release notes  indicate that changes between 1.7.6 and 1.7.25 appear to be bug fixes only, with no API changes.

I had to open a new PR since I deleted the fork for this one.
See #879 , where I have upgrade logback to 1.2.3 and slf4j to 1.7.25


Concourse support

Reason
Possibility to use Concourse as a datasource for radiator-view.
Implementation
Added concourse backend that maps concourse /api/v1/builds to radiator-view builds. Concourse does not provide all the data used in radiator-view. An ad-hoc mapping was done.
Verification
Tested locally against https://ci.dev.kuha.io with appropriate api key.

Closing. Reopen if needed!


mini.files doesn't open when file is deleted

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.files
Description
When trying to open mini.files in a file that doesn't exist, it throws an error and doesn't open.
Neovim version
0.11.0-dev+785-g44afd0744
Steps to reproduce

Open any file
:!rm %
Try to open mini.files

Expected behavior
It should open the parent directory of the file that was deleted, for example if you deleted ~/.config/foo and it should just open ~/.config
Actual behavior
It throws an error and doesn't open

This is by design: anchor path for MiniFiles.open() should be a present path on disk. There is no fallbacks. Use explicit call which includes directory.
Duplicate of #840 and #848.


bug: carousel works Firefox, Chrome but not Safari

What version of daisyUI are you using?
2.51.5
Describe your issue
carousel let's one navigate to next image with right arrow key or back with left arrow key and it's perfect.  Just on Safari (mobile) it doesn't contain images (svgs in this case) as it's just displays them all vertically.  I'm read your comments you suggest using a JS image slider as it'll handle most probably but an all CSS solution would be awesome like daisyui if possible.

Not sure if it's a scroll snap bug or what as I've searched but tough to nail it down.
.carousel::-webkit-scrollbar{display:none}
.carousel{display:flex;overflow-x:scroll;scroll-snap-type:x mandatory;scroll-behavior:smooth;-ms-overflow-style:none;scrollbar-width:none}
.carousel-vertical{flex-direction:column;overflow-y:scroll;scroll-snap-type:y mandatory}
.carousel-item{box-sizing:initial;display:flex;flex:none;scroll-snap-align:start}
.carousel-center
.carousel-item{scroll-snap-align:center}
.carousel-end
.carousel-item{scroll-snap-align:end}
What browsers are you seeing the problem on?
Safari
Reproduction URL (optional)
No response

Here's a video of all examples working on iOS 16
https://user-images.githubusercontent.com/7342023/235749159-e26611a6-1da4-49d5-a5ca-0e50478ae5f5.mp4

Just on Safari (mobile) it doesn't contain images (svgs in this case) as it's just displays them all vertically.

Please share a sample project because there's nothing wrong with examples or the package. It is working on every other project and CSS scroll snap is supported by all browsers. If it's not working on your project, probably Tailwind CSS config on your project is not correct.

for me none of the carousels on the example page are working (firefox, safari & chrome)

Unless you're using a really old browser, it should work:


Port 4200 is already in use

Type: Bug
Hi Team,
I run angular apps in the editor. When try to serve the application, it is always prompting that Port 4200 is already in use. though I dont have the port used. I tried closing the all opened editor instances and reopened, still the issue exists.
VS Code version: Code 1.77.0 (Universal) (7f329fe6c66b0f86ae1574c2911b681ad5a45d63, 2023-03-29T09:57:11.797Z)
OS version: Darwin arm64 22.3.0
Modes:
Sandboxed: No
Extensions (15)

Extension
Author (truncated)
Version

ng-template
Ang
15.2.0

vscode-eslint
dba
2.4.0

gitlens
eam
13.5.0

vscode-npm-script
eg2
0.3.29

prettier-vscode
esb
9.10.4

Angular2
joh
13.0.0

vscode-docker
ms-
1.24.0

java
red
1.16.0

intellicode-api-usage-examples
Vis
0.2.7

vscodeintellicode
Vis
1.2.30

vscode-java-debug
vsc
0.49.1

vscode-java-dependency
vsc
0.21.2

vscode-java-pack
vsc
0.25.10

vscode-java-test
vsc
0.38.2

vscode-maven
vsc
0.41.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
nodejswelcome1:30587005
282f8724:30602487
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
showlangstatbar:30672706
vsccsb:30705552
functionswalk:30687959
pythonms35:30701012

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Fix incorrect return in is_polygon_clockwise

When this was originally implemented in 16022da18793823bf9d3530f1ae9a21501b4faec, the implementation treated the vertices as being in a space where the y-axis is positive in the up direction. This caused it to return the incorrect value, since Godot treats the y-axis as being positive in the down direction.
All existing uses of is_polygon_clockwise have been updated in a way which preserves the existing behavior with the comments being updated to refer to the correct winding order.
Fixes godotengine/godot#49716.

@AThousandShips Is there anything I can do to help get this merged? The log isn't available anymore but iirc the build failure appeared to be unrelated to my change. I can also rebase the work onto a more recent main commit, if that would help make things easier.

You need to rebase and re run, the CI failures have been resolved so needs a new run, otherwise it's waiting for reviews

I'm personally not altogether convinced this is an improvement. The routine is generic (and operates as expected imo). The case of Godot in 2D layout using y-downwards is a special case, therefore the result needs to be reversed.
The behaviour when using it for poly in 2D y-down space isn't obvious, but doesn't mean it is incorrect.
The problem comes if you do this, everyone who is not using it for this 2D coordinate system and this particular case will be thinking, why is the result wrong?
For instance it is common to use 2D polys for navigation in 3D games, where you would use x and z (in godot). The result after this PR would be incorrect (or at least, absurd :grin: ).
An alternative is to make a note in the classref that the result will be flipped when using it with an opposite sign x and y coordinate, such as the 2D screen layout.

@lawnjelly I disagree that this function operates as expected. Since Godot uses y-down for 2D, I would expect the same convention to be used throughout Godot. So, it struck me as incorrect for the 2D geometry API to use a different convention.  However, I also agree that this change is not much of an improvement in the case of 2D polys in 3D games as you've suggested.
I think that's significant enough that I've gone ahead and taken the initiative to open a new PR #89635 which takes the approach of a new optional parameter to define the handedness the caller would like to use.


Add support for auto setup with sql driver

What was changed:
Added the missing the auto-setup for mysql and postgresql databases.
Followed README.md for the commands to run
Why?
I wanted to have the schema migration setup as part of temporal helm chart
Checklist

Closes issue:

How was this tested:
Deployed temporal locally with minikube and with schema.setup.enabled=true and schema.update.enabled=true.
I tested it with both local mysql and postgresql instances.

Any docs updates needed?

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

Hi @naormatania. Thanks for contribution! I am about remove DB tools from main server docker image and leave them only in admit-tools image: https://github.com/temporalio/temporal/pull/1501. Can you change your PR similar to my recent changes to Cassandra job: #172 and #177.

Thanks @alexshtin ! I fixed PR according to your instructions and checked that it works

Hi @alexshtin
This PR is pending for a while and it would really help me if it gets merged
Can you review it again please ?

Thanks for the contribution!
Based on our new direction with helm charts, and because auto setup isn't a production feature (really there for dev setup) we won't be including this feature. See here for details: https://github.com/temporalio/proposals/issues/42


Add issue & PR template

Please find issue/PR template at https://github.com/verdaccio/verdaccio/tree/master/.github
@juanpicado can we use the template from verdaccio repo ? I think all the required information would be same.

sure


Refactor: move errorChecker to nosqlplugin pkg

What changed?
Move ErrorChecker to nosqlplugin pkg
It will make the work of https://github.com/uber/cadence/issues/3514 easier

Why?
For consistency in the interfaces. Nosqlplugin.interfaces shouldn't depend on gocql interfaces of Cassandra.

How did you test it?
Existing tests

Potential risks
No

Release notes
No

Documentation Changes

Pull Request Test Coverage Report for Build 1d2dc516-7561-412d-85ea-d6270ae426d5

0 of 0   changed or added relevant lines in 0 files are covered.
45 unchanged lines in 8 files lost coverage.
Overall coverage decreased (-0.008%) to 59.849%

Files with Coverage Reduction
New Missed Lines
%

common/task/weightedRoundRobinTaskScheduler.go
1
89.64%

common/membership/rpServiceResolver.go
2
73.58%

common/task/fifoTaskScheduler.go
2
83.51%

service/history/execution/mutable_state_builder.go
2
69.85%

service/history/queue/transfer_queue_processor.go
2
57.37%

service/matching/taskListManager.go
2
74.72%

common/persistence/cassandra/cassandraPersistence.go
3
50.96%

common/persistence/cassandra/cassandraPersistenceUtil.go
31
87.6%

Totals

Change from base Build acb0edc9-1684-4ce2-abc8-d18a95ae1005:
-0.008%

Covered Lines:
88080

Relevant Lines:
147170

💛  - Coveralls


Update tool-bar package provider service

The Tool Bar package is going to release a new version after a complete overhaul of it's underlying code. The new version requires that the Tool Bar consumed service semver is updated too. This PR addresses that issue.
There are some changes to the API, but it looks like your plugin isn't affected. Of course it's recommended to test these changes.
Ref suda/tool-bar#141

ping @deldrid1

This repo never made it too far past the fork...
As such, I won't be updating unless I come back around to it.

Thanks for letting me know. With only 19 installs, the impact is really low.


ModalNavigationDrawer gestures easily consume clicks

Describe the bug
Gestures from ModalNavigationDrawer don't make much sense in a desktop environment, but disabling them also disables closing the drawer by clicking on the scrim.
That said, when gestures are enabled any small movement of the mouse is interpreted as a drag instead of a click. I've read that the friction of the mouse is supposed to prevent this, but I have one user who loses approximately 50% of his clicks to this.
I would really like to see either touch slop or something similar applied to mouse movements, or any Material components that have a gesture feature like this have it ignore mouse events. Actually, having ModalNavigationDrawer ignore mouse gestures would make a lot of sense on Android too.
Affected platforms

Desktop (Windows, Linux, macOS)

Versions

Kotlin version*: 1.9.23
Compose Multiplatform version*: 1.6.2
OS version(s)* (required for Desktop and iOS issues): Ubuntu 22.04, Windows 10
OS architecture (x86 or arm64): x86
JDK (for desktop issues): Temurin 17.0.10

Can you be more specific. What is happening that is wrong? Please provide a code sample and reproduction instructions.

Typically when you click down on a button, you can move the mouse around on that button and when you release, the onClick handler will be called. When using ModalNavigationDrawer, when you click on a button, if you move the mouse horizontally at all before release, the click will be canceled and interpreted as a drag event by the navigation drawer. I think this is an issue on desktop and Android, but a bigger issue on desktop because it's rare to use a mouse on Android.
https://github.com/sproctor/ComposeDragBug
In this example, you can uncomment the line to disable gestures, which fixes the issue. The problem with that is that then clicking on the scrim doesn't dismiss the drawer.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


error installing with pip from source

Environment Information

geospatial version: geospatial-0.9.0
Python version: Python 3.9.18
Operating System: MACOS-ARM Macbook Pro M2

Description
I'm trying to instal geospatial with pip because I have an environment with whitebox installed with pip, and some scripts that use both packages.
What I Did
(wbw) /Users/ju > git clone https://github.com/opengeos/geospatial.git
Output:
Cloning into 'geospatial'...
remote: Enumerating objects: 815, done.
remote: Counting objects: 100% (173/173), done.
remote: Compressing objects: 100% (70/70), done.
remote: Total 815 (delta 119), reused 141 (delta 102), pack-reused 642
Receiving objects: 100% (815/815), 888.14 KiB | 317.00 KiB/s, done.
Resolving deltas: 100% (431/431), done.
(wbw) /Users/ju > cd geospatial
2nd command:
(wbw) /Users/ju/geospatial > pip install .
Processing /Users/ju/geospatial
Preparing metadata (setup.py) ... done
Collecting bokeh (from geospatial==0.9.0)
2nd output
...stuff...
Building wheels for collected packages: geospatial, lightgbm
Building wheel for geospatial (setup.py) ... done
Created wheel for geospatial: filename=geospatial-0.9.0-py2.py3-none-any.whl size=4254 sha256=ee0dc2a9f9dda2765267f07e7a66851cfd5e2ca95354c18c102eb69cd9eae247
Stored in directory: /private/var/folders/xt/0fzdrsd52bj4z2cqvy77d5w00000gn/T/pip-ephem-wheel-cache-broy9lm4/wheels/4b/50/3d/f9922042c3c1dcfcf764bd47b61ea11c3a16e125d8b4426118
Building wheel for lightgbm (pyproject.toml) ... error
error: subprocess-exited-with-error
× Building wheel for lightgbm (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [46 lines of output]
2023-09-22 17:08:33,836 - scikit_build_core - INFO - CMake version: 3.27.4
*** scikit-build-core 0.5.1 using CMake 3.27.4 (wheel)
2023-09-22 17:08:33,838 - scikit_build_core - INFO - Build directory: /private/var/folders/xt/0fzdrsd52bj4z2cqvy77d5w00000gn/T/tmp3b8bim10/build
*** Configuring CMake...
2023-09-22 17:08:34,809 - scikit_build_core - INFO - Ninja version: 1.11.1
2023-09-22 17:08:34,810 - scikit_build_core - WARNING - libdir/ldlibrary: /Users/ju/anaconda3/envs/wbw/lib/libpython3.9.a is not a real file!
2023-09-22 17:08:34,810 - scikit_build_core - WARNING - Can't find a Python library, got libdir=/Users/ju/anaconda3/envs/wbw/lib, ldlibrary=libpython3.9.a, multiarch=darwin, masd=None
loading initial cache file /var/folders/xt/0fzdrsd52bj4z2cqvy77d5w00000gn/T/tmp3b8bim10/build/CMakeInit.txt
CMake Deprecation Warning at CMakeLists.txt:35 (cmake_minimum_required):
Compatibility with CMake < 3.5 will be removed from a future version of
CMake.
    Update the VERSION argument <min> value or use a ...<max> suffix to tell
    CMake that the project does not need compatibility with older versions.

  -- The C compiler identification is AppleClang <IP_ADDRESS>30022
  -- The CXX compiler identification is AppleClang <IP_ADDRESS>30022
  -- Detecting C compiler ABI info
  -- Detecting C compiler ABI info - done
  -- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc - skipped
  -- Detecting C compile features
  -- Detecting C compile features - done
  -- Detecting CXX compiler ABI info
  -- Detecting CXX compiler ABI info - done
  -- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/c++ - skipped
  -- Detecting CXX compile features
  -- Detecting CXX compile features - done
  -- Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES)
  -- Could NOT find OpenMP_CXX (missing: OpenMP_CXX_FLAGS OpenMP_CXX_LIB_NAMES)
  -- Could NOT find OpenMP (missing: OpenMP_C_FOUND OpenMP_CXX_FOUND)
  -- Found OpenMP_C: -Xpreprocessor -fopenmp -I/opt/homebrew/opt/libomp/include
  -- Found OpenMP_CXX: -Xpreprocessor -fopenmp -I/opt/homebrew/opt/libomp/include
  -- Found OpenMP: TRUE
  -- Performing Test MM_PREFETCH
  -- Performing Test MM_PREFETCH - Failed
  -- Performing Test MM_MALLOC
  -- Performing Test MM_MALLOC - Success
  -- Using _mm_malloc
  -- Configuring done (0.7s)
  -- Generating done (0.0s)
  -- Build files have been written to: /var/folders/xt/0fzdrsd52bj4z2cqvy77d5w00000gn/T/tmp3b8bim10/build
  *** Building project with Ninja...
  ninja: error: '/opt/homebrew/opt/libomp/lib/libomp.dylib', needed by '/private/var/folders/xt/0fzdrsd52bj4z2cqvy77d5w00000gn/T/pip-install-t663i5f6/lightgbm_e0bfa722f2d44cfcb7590d43f679be92/lib_lightgbm.so', missing and no known rule to make it

  *** CMake build failed
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for lightgbm
Successfully built geospatial
Failed to build lightgbm
ERROR: Could not build wheels for lightgbm, which is required to install pyproject.toml-based projects

pip is not recommended. Try mamba
https://geospatial.gishub.org/installation/#using-mamba

Thank you very much Dr. Wu.
I've read that is not recommended to instal with conda after using pip (but it's ok to use pip after conda).
Do you think in this case it's ok if I mix pip with mamba?
Kind regards.

When install a package with a lot of dependencies, mamba is recommended. After that, using pip is fine but is not recommended.

Thank you!