MINOR: Don't downgrade yarn in Travis CI

Codecov Report

Merging #1413 into master will decrease coverage by 0.11%.
The diff coverage is n/a.

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


upload-1249 Refactor Health Check Function

Refactor Health Check Function

Closing in favor of https://github.com/CDCgov/data-exchange-upload/pull/372


Extend LaravelReadQuery->checkAuth to handle relations

Coverage increased (+0.005%) to 98.044% when pulling 279d6dc8da1a07e0d00ddd49fe5c1b6e851116f6 on CyberiaResurrection:FixReadQueryAuth into 0b7389ae3d603a293abdc156e373b58c858794c4 on Algo-Web:master.


Early release AcquireReleaseColumnsSegmentOperator during explain plan

During explain plan query execution, once all the child operators of the AcquireReleaseColumnsSegmentOperator have been processed, we may call release() early and not defer until all segments are processed.

Codecov Report

Merging #11945 (965e3cf) into master (53e8033) will decrease coverage by 14.89%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             master   #11945       +/-   ##
=============================================
- Coverage     61.30%   46.41%   -14.89%
+ Complexity     1140      936      -204
=============================================
  Files          2378     1780      -598
  Lines        128865    93330    -35535
  Branches      19927    15080     -4847
=============================================
- Hits          78998    43318    -35680
- Misses        44147    46969     +2822
+ Partials       5720     3043     -2677

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

java-11
?

java-21
46.41% <100.00%> (-14.90%)
:arrow_down:

skip-bytebuffers-false
?

skip-bytebuffers-true
46.41% <100.00%> (-14.85%)
:arrow_down:

temurin
46.41% <100.00%> (-14.89%)
:arrow_down:

unittests
46.41% <100.00%> (-14.89%)
:arrow_down:

unittests1
46.41% <100.00%> (-0.06%)
:arrow_down:

unittests2
?

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

Files
Coverage Δ

...in/java/org/apache/pinot/core/common/Operator.java
82.35% <100.00%> (+2.35%)
:arrow_up:

...operator/AcquireReleaseColumnsSegmentOperator.java
75.00% <100.00%> (+11.36%)
:arrow_up:

... and 973 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Allow to copy text to clipboard on android app

I'm not sure if this is a bug or not, but being able to copy the text output to clipboard would make the app... actually usable for my use case. Is this actually not possible, or am I missing something?

I am unsure of how to add this feature to the Android app as I am not an Android app developer. If someone has an idea, we can include the change in the Android app

It is straightforward, I had implemented it (in about the simplest possible way, ie no error checking) with an earlier codebase example from pytorch running wav2vec2 model:
    import android.content.ClipboardManager;
    ....
    private void copyToClipboard(String text) {
        ClipboardManager clipboard;
        clipboard = (ClipboardManager) getSystemService(Context.CLIPBOARD_SERVICE);
        ClipData clip = ClipData.newPlainText("simple_text",text);
        clipboard.setPrimaryClip(clip);

    }

will be trying out this same function with this current openai/whisper example soon.

Great and thanks

Can someone please help me to run this android application. I would be very thankfull


Helm chart: Custom proxy init image for sidecar-injector enable-core-dump container

Describe the bug
A clear and concise description of what the bug is.
The sidecar-injector's enable-core-dump container does not respect the value of any custom global.proxy_init.image value.
Expected behavior
A clear and concise description of what you expected to happen.
The value of the enable-core-dump container's Image attribute in the istio-sidecar-injector ConfigMap should respect any custom value of global.proxy_init.image.
Steps to reproduce the bug
Steps to reproduce the behavior.
Change global.proxy_init.image and observe the enable-core-dump container's Image value.
Version
What version of istio and Kubernetes are you using? Use istioctl version and kubectl version
Istio 1.0.0
Kubernetes 1.10.x
Is Istio Auth enabled or not?
Did you install the stable istio.yaml, istio-auth.yaml.... or if using the Helm chart please provide full command line input.
$ helm install\
  --kubeconfig kubeconfig\
  --name istio\
  --namespace istio-system\
  --values values.yaml\
  install/kubernetes/helm/istio

Environment
Which environment, cloud vendor, OS, etc are you using?
AWS
Cluster state
If you're running on Kubernetes, consider following the
instructions
to generate "istio-dump.tar.gz", then attach it here by dragging and dropping
the file onto this issue.

Related to https://github.com/istio/istio/issues/7782.

Initially addressed in https://github.com/istio/istio/pull/7807 which I closed as a duplicate but I will open another PR for this specific issue.


Rewrite views as class based views

Currently there is a mix of function and class-based views.  We should ship with CBVs to allow for greater extensibility.

folder_create => FolderCreateView(LoginRequiredMixin, CreateView)
folder_detail => FolderDetailView(LoginRequiredMixin, DetailView)
document_create => DocumentCreateView(LoginRequiredMixin, CreateView)
document_detail => DocumentDetailView(LoginRequiredMixin, DetailView)
document_download => DocumentDownloadView(LoginRequiredMixin, View) (but maybe a DetailView)
folder_share => FolderShareView(LoginRequiredMixin, View) (but maybe a DetailView)
document_delete => DocumentDeleteView(LoginRequiredMixin, DeleteView)

@paltman Is this issue now resolved?


Constraint install and activate

Can you provide an example of install and activate?
MASConstraint
- (void)activate { MASMethodNotImplemented(); }
- (void)deactivate { MASMethodNotImplemented(); }
- (void)install { MASMethodNotImplemented(); }
- (void)uninstall { MASMethodNotImplemented(); }

I second this please.
I was trying to add two constraints that would be conflicting (two different height constraints) then activate/deactivate or install/uninstall them but neither works, it produces a conflict. I ended up having to remove and re-add all constraints.
Some nicer way to add an inactive constraint would be nice too.

@evilDave
uninstall constraint, system will be release it.
so, you need use strong reference the constraint.

You need to capture a strong reference to a constraint in the make/remake block:
self.constraint = make.left.equalTo(50);

You can then do
[self.constraint uninstall];

If this doesn't solve your issue please re-open a new issue with a sample project.


Page number 100  - no OCR, no preview

Description
when working with 100 pages files. 99 and 101 pages are ok. Example attached (No jpg, no ocr) on
100pages doc.
Expected
page number 100 should be OCRed as well and have a proper preview.
Actual
page number 100 has no preview. Pages 99 and 101 are ok.
Info:

OS: irrelevant
Browser irrelevant
Database irrelevent
Papermerge Version 1.5.4, 1.5.5

Issue was due to a bug in mglib. Fixed.

Issue was due to a bug in mglib. Fixed.


More Than 15 Countries Have Lists Empty | [FIX NEEDED]

The list of more than 15 countries have been affected wherein no data is being recorded and the list is rendered empty.
Few countries suffering from this issue are as follows:

No.
Country
No.
Country

1.
Australia
6.
Greece

2.
Brazil
7.
India

3.
CA, USA
8.
Indonesia

4.
Canada
9.
Iran

5.
Egypt
10.
Israel

To name a few....
Similar Issues which can be referred and closed on Fix: #122 #128 #129 #130

Kenya too makes the list...

The fix is here: https://github.com/lauripiispanen/github-top/pull/206

Thanks for the update @aravindvnair99 . Shall close this issue and urge others close issues related to this namely:
#122
#128
#129
#130


isrealobj()

I sometimes perform an operation of two arguments that I know will always return a real-valued object, even if the input arrays are complex-valued. (Think inner products <z, z>.) I usually
confirm its realness up to a tolerance and clip off the imaginary part. For return values of real data type, this is not necessary, so I shortcut with
if np.isrealobj(val):
     return val

in NumPy.
Is something similar available in the Array API yet?

That works, thanks!


Dead link

Your Feedback ---> on UserVoice --> https://msdocs.uservoice.com/ link is dead.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0193ed76-3eec-18ac-512f-45f8118dde34
Version Independent ID: 8e693833-3e45-2bb8-ddc2-d0c54936c3c1
Content: A New Feedback System Is Coming to docs.microsoft.com
Content Source: teamblog/a-new-feedback-system-is-coming-to-docs.md
Product: team-blog
GitHub Login: @EisenbergEffect
Microsoft Alias: EisenbergEffect

The link is now updated: https://github.com/MicrosoftDocs/blog/commit/d9253ab96d745dc57f0e0c71aedc69538b35bf20

The updated link is also dead.


3.1.1 upload slow

When uploading a file to media gallery the upload request takes up to 1 minute. With 3.0.4 with same image the time is around 10 seconds. Many times we ran to server timeout because of this.

The slowness is present when using CDN mode. Ephemeral mode is not affected.

I have the same issue. Using Woocommerce API and when WP-Stateless actived, the response of image id take longer.


Investigate embedded versions of Dpkg

Which external versions were embedded in the Perl modules

Dpkg is written in Perl. There were no external libraries as far as I know.

Ah, then I don't have to worry about those.


Error in looping logic for skeletons with unequal number of bones

Hi,
I noticed some serious problems when working with animations with different number of bones for each skeleton (i.e. paired_mount.hkx).
I believe that at least partially, this is due to line 92 in merger.cpp:
anno_tracks[anim_a->m_numberOfTransformTracks + 2].m_trackName = "NPC";
which should be:
anno_tracks[anim_v->m_numberOfTransformTracks + 2].m_trackName = "NPC";
in order to count 2 beyond the "victim" skeleton. Right now, you end up with "NPC" in the middle of one of the skeleton's, overwriting one of their bone annotations, and a blank ␀ where "NPC" should be.
I don't know if it actually breaks the animation as I don't know how havok actually does its transforms.

Thank you for the respond. Did you test the change yourself and did it work?


Downstream koji build

Do not work with fedmsg topics in handlers.

The PROCESSED_FEDMSG_TOPICS wasn't used anywhere and @add_topic is not needed because of it.

Implement the downstream Koji build.

Using a new koji_build job type.

Make the SyncFromDownstream react to a general event:

Removing of DistGitCommitEvent that filtered the events based on the service config.

Events should be independent of handlers and multiple handlers can react to one event.
Logic (checking the service config) should be in a handler.

Removing FedmsgHandler because it had no function.

TODO:

[x] feedback and error handling: Commit comment in case of an error on submission; once submitted, users will get a standard notification.
[x] tests

Fixes #55
Merge after: packit/packit#1415 (required for tests to succeed)
When reviewing, please, go commit by commit. There were some class removals/renames that make the whole diff a bit messy.

TBD

recheck

recheck

recheck


[spi_device] Revise Upload Pre DV to test payload > 256B

This commit revises the pre_dv upload direct testcase to test the
payload greater than 256B.

Merging this now


Service Discovery Causes Massive timeout issues with UCS Creation

Environment

Application Services Version: Tested Latest and other versions with same issue 1.40+
BIG-IP Version: 17.1.1

Summary
When trying to create a UCS Backup of a BIG-IP the time it takes to create that UCS is massively increased as per https://cdn.f5.com/product/bugtracker/ID985329.html
This is stemmed from
Mar 28 12:23:32 bigip.f5demo.net err iAppsLX_save_pre[14953]: Failed task: /shared/iapp/build-package/bf9e1f3e-7826-4728-b8f0-67a2fb6b4b40: rpmbuild command failed: com.f5.rest.workers.shell.CommandExecuteException: Command execution process killed
Mar 28 12:23:32 bigip.f5demo.net err iAppsLX_save_pre[14953]: Failed to get getRPM build response within timeout for f5-service-discovery
Mar 28 12:23:32 bigip.f5demo.net info iAppsLX_save_pre[14953]: Exporting: f5-appsvcs - /var/config/rest/iapps/f5-appsvcs
If you remove AS3 and Service Discovery, UCS Backups on a clean bigip (Nothing on it) takes ~20 seconds, when this is implemented can be from 2-5 Minutes due to this timeout, i have also seen where UCS never gets created as well.
Talked with Mark Dittmer abou tthis and he suggested an issue ticket.
Steps To Reproduce
Steps to reproduce the behavior:

Have fresh BIG-IP (or you can use my Ansible 101/201 UDF Blueprint)
Install AS3 (UDF Blueprint has a version already on it)
In my usecase (using UDF) i ran the Backup and Restore information as per documentation

Code https://github.com/f5devcentral/f5-bd-ansible-labs/tree/main/201-F5-Advanced/Modules/00-Backup-Restore-Role
https://github.com/f5devcentral/f5-bd-ansible-labs/blob/main/201-F5-Advanced/Modules/00-Backup-Restore-Role/roles/f5_backup_data/tasks/f5_backup_config.yaml
Documentation for the Lab - https://clouddocs.f5.com/training/fas-ansible-use-cases/201/Modules/00-Backup-Restore-Role.html

Error occurs typically that says increase timeout value ~3-5 minutes
Use Webshell on F5 to do access /var/log/ltm and see error indicated above.

To fix behavior in the lab login to TMUI and remove AS3/DO via AppsLX Section including Service Discovery, and re-run playbook above takes ~20 Seconds to a minute to complete with no issues.
Expected Behavior
This error shouldnt occur and create a massive time delay in creating UCS Files, it should take approximatly the same amount of time to backup a UCS with Ansible with our without AS3.
Actual Behavior
Backups slow down to the point of Ansible failing via timeouts or never completing the tasks.

Adding notes here that Restore Times for UCS Files are also impacted by this as well, sometimes the restore commands will hang or never fully run (or take 10+ minutes) when the AS3/Service Discovery is removed everything moves fast.


jskeus: 1.2.0-3 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository jskeus to 1.2.0-3:

upstream repository: https://github.com/euslisp/jskeus
release repository: https://github.com/tork-a/jskeus-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.6
previous version for package: 1.2.0-0

@mikaelarguedas I'm very sorry to send PR again and again. we have strange problem, which the created deb installs /opt/ros/kinetic/setup.bash and that conflicts with ros-kinetic-catkin
Of course that's mostly because of our strange usage of catkin in our euslisp/jskeus packages....
c.f. https://github.com/tork-a/euslisp-release/issues/14 ,https://github.com/tork-a/jskeus-release/pull/5


[T2A4][T01-T1]Rahumathmarini

Ready for review

@marini-rahumath
Please ensure commit messages are in proper tense.
Some comments added. Please close the PR after reading comments.


Link to bugzilla is broken

the aka.ms link resolves to Bugzilla.xamarin.com, which has been retired. user redirected to visualstudio developer community site.
The link from this blog https://blog.xamarin.com/xamarin-live-player-faq/ is also pointing to the same aka.ms.
What's the correct way to file a work item now?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6f641830-151c-230d-cdf6-da7b6962ad04
Version Independent ID: 3b0e6ac7-c893-f087-7e73-cfe7d72ca5cd
Content: Limitations of Xamarin Live Player - Xamarin
Content Source: docs/tools/live-player/limitations.md
Product: xamarin
GitHub Login: @topgenorth
Microsoft Alias: toopge

I made a quick adjustment on Bugzilla to help with this until the docs and aka.ms link are updated. I changed the bug entry link on Bugzilla https://bugzilla.xamarin.com/enter_bug.cgi?product=Live Player so that it no longer redirects straight to Developer Community but instead to the top-level bug entry page on Bugzilla. That top-level page includes a short description about how users should now file issues for Live Player via Report a Problem in Visual Studio.

Thanks @kwiqniss and @brendanzagaeski – I made an update to this doc which should be published soon.


Rethink error_bar arguments

"y.low seems like it's the low value for y, ie: the actual value where you place the low bar. i suppose you could keep y.low to mean that, and add another param if someone wanted to specify the error up/down (offsets)."

This is an easy fix if we do anything about it, so I am pulling it in. I think it needs to be done sooner rather than later, otherwise we just continue as we have it currently implemented. Let's discuss.

Changing it now also makes it easier to add as an additional parameter that gets included in the limit calculation as mentioned in #442. If we keep as-is, we kind of need to transform/calculate the argument into something else to use it.

New arg names:
actual absolute values:
y.low
y.high
x.low
x.high
offsets:
offest.up
offset.down
offset.left
offset.right
turn one set into the other to use existing functionality.
MVP change arg names to the offset versions.


error when i use setup.py

setup.py should have downloaded cutlass, idk why it didn't in your case.

I encounter the same problem. Using the below command to clean the cutlass submodule folder can solve it.
cd csrc/cutlass
git clean -d -f .


Move username to separate line

Username gets crowded in title bar. Move it below so that it can be seen clearly

Instead changed title to username when logged


beaverhabit creates for every health check request a new storage-user-ID.json

beaverhabit creates for every health check request a new storage-user-ID.json
I have no name!@beaverhabits-547ccd8b7c-sp6v5:/app/.user/.nicegui$ date && ls -1 | wc -l
Mon Dec 23 14:22:16 UTC 2024
20035
I have no name!@beaverhabits-547ccd8b7c-sp6v5:/app/.user/.nicegui$

I have no name!@beaverhabits-547ccd8b7c-sp6v5:/app/.user/.nicegui$ date && ls -1 | wc -l
Mon Dec 23 14:22:36 UTC 2024
20039
I have no name!@beaverhabits-547ccd8b7c-sp6v5:/app/.user/.nicegui$

Content of file
$ cat /app/.user/.nicegui/storage-user-ffffb4c9-5c95-4310-be57-ed5300ebae8e.json
{"referrer_path":"/"}

currently workaround. Deleting /app/.user/.nicegui/* because im using the sqlite db.

Hi @RogerSik thanks for reaching out.
Each storage-user-ID.json file represents one session and should be reused until the session expires.
I am unable to reproduce this issue locally. Could you please double-check your browser's cookie settings? Thank you.

Could you please provide the version you are using? I will double-check this issue.

Sure. I use the image daya0576/beaverhabits:0.4

The health check is generated every minute by Kubernetes. So It cant be only my browser. I sent later my entire setup.

image: daya0576/beaverhabits:0.4
env variables:
MAX_USER_COUNT: '1'

Health Chech:
httpGet:
  path: /
  port: 8080

I see, Kubernetes visits the health page (without a cookie), and the web app treats it as a completely new request.
Could you try changing the health page path from / to /health?

Yes that helped. Many thanks @daya0576 :)


Update CONTRIBUTING.md

Closes #89

Hi @x64Eddie. Just wanted to know whether this organisation is a part of Hacktoberfest

Hey @kichloo, we do not have any direct association with Hacktoberfest other than that we are simply an open-source project! PR's to Parabeac-Core should work for Hacktoberfest though & we are in fact promoting ourselves to be working with developers that want to participate in Hacktoberfest.

@siliconivan Ok! I was asking so that this PR could count towards completion of the PR through the 'Hacktoberfest-accepted' label. No problem!

@siliconivan @x64Eddie @ivan-015 Any update on the Hacktoberfest swags of Parabeac Core? I had made a PR which was merged in the month of October

Hey @kichloo can you DM me in Discord? We can sort this out there :)


vectorized_map fails if function contains an abs of a complex128

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS 10.15.5
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v2.2.0-rc4-8-g2b96f3662b 2.2.0
Python version: 3.7.6
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A

Describe the current behavior
vectorized_map throws an AssertionError if the body contains a tf.abs of a complex128 tensor.
Describe the expected behavior
No exception should be thrown.
Standalone code to reproduce the issue
import tensorflow as tf
tf.vectorized_map(tf.abs, tf.cast([0, 1], dtype=tf.complex128))

(also see https://colab.research.google.com/drive/1ExIw0N92bHpgWxwRONmeP24BWCzWG3QL?usp=sharing)
Other info / logs
I think the issue is that the ComplexAbs gets converted to a straight call to math_ops.complex_abs (https://github.com/tensorflow/tensorflow/blame/master/tensorflow/python/ops/parallel_for/pfor.py#L2731), which by default returns a float32 (instead of deciding the dtype based on the dtype of the argument). I believe it could be fixed just by changing that call to math_ops.abs.

@charmasaur
I ran the code shared on tf nightly and do not face any issues, please refer to gist here and let us know.

Sorry, I didn't think to check nightly.
In any case, while the original code I gave does indeed work on nightly, the issue is still there: changing [0] to [0, 1] causes the same failure as before.
I think https://github.com/tensorflow/tensorflow/commit/c2e594440e1d9839546b93a93d8646b06891d7de#diff-a99b0652aa0a4be5b8fd65a0c02bea64 fixed the [0] case as a side-effect, since after that change the parameter gets treated as a loop invariant, so isn't converted by pfor.

@charmasaur
I ran the code on tf 2.5, please verify the gist here and confirm if we may move this to closed status.

Hi @Saduf2019 , unfortunately it's still not fixed in the case where the tensor has more than one element (see my comment https://github.com/tensorflow/tensorflow/issues/43322#issuecomment-695864353). Changing [0] to [0, 1] in that gist still causes a failure.


Errors for unnecessary space(s) before comments

Extra spaces before comments are flagged as errors.
I can't find anything in the Kotlin Style Guide that bans extra spaces before comments.  The Android Kotlin style guide specifically allows extra spaces before a comment (see bullet 7).
ktlint allows extra spaces when adjacent lines have aligned comments, but not when those lines are not immediately adjacent (see screenshot).  I think multiple spaces should be allowed before a comment even if the comment stands alone, because a double-space before a comment is not uncommon.

Under https://kotlinlang.org/docs/reference/coding-conventions.html#horizontal-whitespace,

As a general rule, avoid horizontal alignment of any kind. Renaming an identifier to a name with a different length should not affect the formatting of either the declaration or any of the usages.

So I would think that all of the above should be considered erroneous.

On a related note, ktlint 0.33.0 also reports "Unnecessary space(s)" for this line:
https://github.com/heremaps/oss-review-toolkit/blob/73d2cdfc536aa745052ea4a594fe23ce89863457/analyzer/src/main/kotlin/managers/GoDep.kt#L60
So I do get the report also for adjacent lines. Can anyone shed a light on whether this is intended or not?

Looks like @shyiko removed the adjacent lines check a few months ago in this commit: https://github.com/pinterest/ktlint/commit/696b33efb70971495627bd573ae3b9d46749c91b presumably because of the style guide link above. So this does seem to be intended behavior.

Yeah, unfortunately that commit message lacks a rationale, so it's really hard to tell.

It would be great if there was a separate rule for horizontal alignment of comments. I'd like to disable that one but keep the "Unnecessary space(s)" for code spacing.

Any ideas when it'll be fixed? It's really terrible that it has an effect on docs (formatted with tabs):
/**
 * @param message       The payload.
 * @param someParam       Something
 */

Result:
Generating Unnecessary space(s).
Experimental doesn't change anything

I understand that the style guide recommends against horizontal alignment, but the NoMultipleSpacesRule also prohibits "Python-style" inline comments, i.e. comments with two spaces in front:
val mu = 3  // Mean
val sigma = 2  // Variance

I think that adding two spaces in front more clearly separates the comment from the code, and it is not against the Kotlin style guide as far as I know.
Would it be possible to add an option to disable this rule only for spaces around comments?

Would it be possible to add an option to disable this rule only for spaces around comments?

Ktlint will not support flags for 'small' details like this. I do understand that it might not be a 'small' details for you but we really should try to avoid to bloat ktlint with tens of configuration issues.

@paul-dingemans Fair, I understand that. Adding many configuration options creates a combinatorial explosion in terms of maintenance and testing.
So, instead, what do you think of bringing Ktlint in line with the Kotlin Style guide and Android Kotlin Style Guide by allowing additional whitespace before comments, without a corresponding configuration option? That is, make the behaviour suggested in this issue the default without the option to change it, since it's more in line with those major style guides.

As was pointed out before:
Under https://kotlinlang.org/docs/reference/coding-conventions.html#horizontal-whitespace,

As a general rule, avoid horizontal alignment of any kind. Renaming an identifier to a name with a different length should not affect the formatting of either the declaration or any of the usages.

I have seen no convincing argument to allow additional spacing before comment that do not violate the general rule 'avoid horizontal alignment of any kind'. Neither is this requested widely by many users.

I see what you mean, as some comments did ask for allowing horizontal alignment, even though that is strictly against the style guides. I agree with you here that multiple spaces before a comment with the goal of aligning multiple comments should be disallowed.
The issue raised by OP, however, was slightly different from some of the comments:

Ktlint allowed alignment of consecutive lines, but not for non-consecutive lines, as shown in OP's sample code. (This was fixed in 696b33efb70971495627bd573ae3b9d46749c91b in response to this issue, by always disallowing more than one space before a comment.)
Ktlint does not allow a comment to be preceded by two spaces, à la Python, even if there is no alignment going on. (This is still the case.)

Both style guides allow the latter, so I think it would make sense for Ktlint to allow it as well. I don't really see any use cases beyond Python-style comments, so what do you think of changing NoMultipleSpacesRule to tolerate either 1 or 2, but not more? This would still effectively prevent alignment, except in some exceptional cases, with only a minor change in code.

I do understand from your perspective that allowing 1 or either 2 spaces before an EOL does not violate the style guide when it is not done for alignment purposes. Although the code change will be fairly simple, I still against it for following reasons:

KtLint has a strong focus on consistent code formatting. Allowing 1 or 2 spaces could lead to inconsitent formatting as it would depend on the developer to consistently use 2 space. Of course you can argue that this could be forced by ktlint as well if there was a rule for that. But as explained before, we do not want to add configuration settings for minor details.
I expect that additional edge cases will popup. Consider for example a line containing only some indentation plus and EOL-comment.
Kotlin != Python, with a different language also come different 'best' practices
Allowing 1 or 2 spaces will lead to new issues in which will be complained about not removing a redundant space.

If it is really important for you to have this functionality, I would suggest that you create a custom ruleset that allows/enforces the double space before the EOL comment and disable the rule which is provided by KtLint.

Thank you for the response. I understand and completely agree with you.
I might consider writing a custom ruleset that enforces a double space before comments if I find the time to do that, and release it separately.


AUTH-296: Separate kubelet client trust

This PR separates the CAs that are trusted by the kubelet for client cert authentication.
The cmd/init.go is starting to be a little messy, we will probably want to refactor it eventually.

This is currenly untested, I would be surprised if everything worked because this finally cuts the trust of a kubelet to the ultimate bundle for client cert authentication, but let's see what happens,

also related to USHIFT-278

/type cleanup

/assign @pacevedom
/approve

/test e2e-openshift-conformance-sig-network
The sig-network test failed with:
{  failed to acquire lease for "gcp-quota-slice": resources not found}

/test all
I do not know how this repo CI works but perhaps all needs to be run in order for the tests to succeed

Looks like a subscription issue. I should point out that all the tests except sig-network passed before, and this test did not pass because of gpc-quotas.

/retest

/retest

/lgtm


Setting Endpoint on S3 service objects

I seem to be running into some issues with setting the Endpoint on an S3 service object when it is created. Is there something I am doing wrong in how I am passing in the S3Config object? Setting the region appears to work fine.
Please see the code sample below:
svc := s3.New(&s3.S3Config{&aws.Config{
		Credentials: aws.DefaultCreds(),
		Endpoint:    "http://localhost:1234",
		Region:      "us-west-2",
		DisableSSL:  false,
		ManualSend:  false,
		HTTPClient:  http.DefaultClient,
		LogLevel:    0,
	}})
	log.Println("S3 Endpoint Create, ", svc.Endpoint)
	svc.Endpoint = "http://localhost:1234"
	log.Println("S3 Endpoint Reset, ", svc.Endpoint)

This will output the following:
2015/02/06 13:11:44 S3 Endpoint Create,  https://s3-us-west-2.amazonaws.com
2015/02/06 13:11:44 S3 Endpoint Reset,  http://<IP_ADDRESS>:1234

This should be fixed by #91


Calculate actual crop info

Is there any way to retrieve the x and y coordinates? The width and height would also be possible? I'm trying to make a cut but I am not succeeding.

Yes. Check out the offset, zoom, and previewSize (or imageSize, depending on what you are looking for) APIs -- docs here.

I looked at the api though, when you cut, the picture is not correct. Already try to offset; previewSize and zoom but no use.
it is as if the coordinates were lost

I looked at the api though, when you cut, the picture is not correct. Already try to offset; previewSize and zoom but no use.
it is as if the coordinates were lost

The offset that the API returns is offset of the image w.r.t. the crop box. From what you described you seem to be looking for the coordinates of the cropbox w.r.t. the image, which would be (offset.x / zoom, offset.y / zoom).


Fix polyline and polygon points parsing

Handling of numbers in scientific notation in parsePoints used to be
incorrect, because a number could be broken by an exponent minus sign,
resulting in incorrect values.
E.g. '2.5e-1' -> '2.5e', '-1'.
String-splitting-based parsing is replaced with a simpler use of Scanner, otherwise keeping the behaviour, esp. error handling, unchanged.

Thanks for your work!


Default "Scheduled date" for tasks with "due date" when "Use filename as Scheduled date for undated tasks" is enabled

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
Currently, when the setting "Use filename as Scheduled date for undated tasks" is enabled, tasks that have a "due date" are not assigned a default "Scheduled date" from the note's filename date. The documentation is clear on when it will get a default scheduled date:

If this option is enabled, any undated tasks will be given a default Scheduled date extracted from their file name.
[...]
Undated tasks have none of Due (📅 ), Scheduled (⏳) and Start (🛫) dates.

This can be very useful for querying tasks using the "happens" filter. Example:
Filename: "2023-01-01"
# Tasks
- [ ] A task
- [ ] With creation  ⏳ 2023-01-01
- [ ] Another task with due 📅 2024-01-17

# Querying tasks for 2023-01-01
```tasks
not done
happens on 2023-01-01
```

All tasks should appear but "Another task with due" isn't. This is a "feature" because the default date is only applied to "undated dates". However, I think it should be changed.
This is also very useful where we have a query that fetches all "past" tasks from different past days. Example:
not done
happens before today

Because the task is due far in the future (has a "due date"), then the "happens" won't match it because it doesn't have a "scheduled date"
✔ Solution
To address this, a new feature should be added to the plugin, where when the setting "Use filename as Scheduled date for undated tasks" is enabled, tasks that have a "due date" are also assigned a default "Scheduled date" from the note's filename date. This way, even if the task has a "due date", it will still have a "Scheduled date" as well, and it will provide more information and flexibility to the user.
❓ Alternatives
Alternatively, add to the settings a new switch called "Use filename as Scheduled date for unscheduled tasks".
📝 Additional Context
No response

Hi @Bloodsucker
Thank you for the suggestion.

🔖 Feature description
Currently, when the setting "Use filename as Scheduled date for undated tasks" is enabled, tasks that have a "due date" are not assigned a default "Scheduled date" from the note's filename date. This is despite being considered as "undated tasks" according to the documentation.

Please could you point me to the bit in the docs you are meaning, for this:

This is despite being considered as "undated tasks" according to the documentation.

HI @mvdkwast - can you remember the thinking behind the presence of a due date on a task preventing it from having  scheduled set from the file name? I can't.

Thanks @olearycrew for taking into consideration my suggestion. I had a typo on the quoted phrase, which I will correct, but what I tried to say was something like: "Due dated tasks won't get default Scheduled date because they aren't undated as per documentation". That part can be read here and I quote:

If this option is enabled, any undated tasks will be given a default Scheduled date extracted from their file name.
[...]
Undated tasks have none of Due (📅 ), Scheduled (⏳) and Start (🛫) dates.

What this proposals is aiming is to allow unscheduled tasks

What this proposals is aiming is to allow unscheduled tasks (tasks without a scheduled date) to get a default date based on the file's name regardless of a due date being assigned to it explicitly.

That's nice and clear. Thank you.

The way I intended it was that if there is any explicit date, then there is no need to set the scheduled date.
In my personal setup, I use the happens predicate in my global dashboards (now, short-term, long-term), and not setting the scheduled date if any other date is set avoids seeing tasks multiple times. I consider a task with an explicit date inside a daily note as something I will have to do another day. The presence of the task inside a daily note just gives me some context about the task (answers the "why did I create this ?" question mostly). I also don't have queries for tasks of the day inside my daily notes, since I use dashboards at a more global level.
I can see however how the expectation about the feature might be different, so not sure if this should be another option or just the default behaviour (I can live with a few duplicates).

In my opinion, the way the setting is understood is as "Use filename as Scheduled date for unscheduled tasks". In other words, take take a default schedule date from the filename if it isn't specified at the task level. If by any chance the user is interested in querying those that don't explicitly specify a scheduled date then a way to do so would be by using the filter scheduled date is invalid in their query. @mvdkwast I believe you could include that filter in your query in order to clear these tasks that might be duplicated in your use-case?

In my opinion, the way the setting is understood is as "Use filename as Scheduled date for unscheduled tasks".  In other words, take take a default schedule date from the filename if it isn't specified at the task level.

I do agree with this.

If by any chance the user is interested in querying those that don't explicitly specify a scheduled date then a way to do so would be by using the filter scheduled date is invalid in their query.

No, that would not work. That query matches dates like 2023-13-35 - where the date is literally present but is not a valid date.

See https://obsidian-tasks-group.github.io/obsidian-tasks/queries/filters/#finding-tasks-with-invalid-dates

I can see however how the expectation about the feature might be different, so not sure if this should be another option or just the default behaviour (I can live with a few duplicates).

@mvdkwast Thank you for that generous comment.
I do feel that this request is worth implementing, then.

In view of this size of the backlog for this project, and the demotivating effect of a very long backlog on volunteer energy levels, I'm going through a slow process of reviewing tasks for things that are one or more of the following:

unlikely to be realistically achievable, from the design of the current code, and the planned future refactorings
not sufficiently high priority to a broad community of users, based on the feedback I receive from a variety of sources
do not fit the core purpose of the Tasks plugin
are better covered by a different plugin - to share the development and maintenance effort

So I'm marking this one as 'not planned'. This doesn't mean 'won't fix' - it means not likely to be done in the foreseeable future.


Concurrency issue when two MREs access the same resource

At least one user experienced a reproducible crash when one GLTF object or kit object - just about any resource - is used in two different MREs in the same space.
The MREs don't even need to be enabled from start. Enabling the first of a set of two is okay, but on enabling a second one using the same resources of the first one, the client of the affected user hangs.

"Reenter space" is not working
Avatars of latecomers aren't loaded
MRE processing is completely halted for the user - even MREs which don't share resources don't work anymore
Client cannot be exited by normal means; "Application stopped responding"

New observations:

Using the same object several times in a single MRE, which is used a single time in a given space works. Example: The chevrons of the Stargate.
Using the same object once in a single MRE, but said MRE being used more than once in a given space triggers the hang. Example: The sliding door I have in one of my worlds, used in several locations.
Activating a previously deactivated MRE, which tries to spawn an already used object, triggers this bug as well.
Two users affected so far (out of maybe ten)

My current theory is that when a new asset is added to the resources, either by loading using Actor.CreateFromLibrary(), Actor.CreateGLTF() or in lieu of actor synchronization and it is already there because another MRE already loaded it, the function or thread silently exits without giving a notification to its caller, and said caller would stop working and endlessly hang, waiting for a confirmation that never comes.
Why it affects only several people and not everyone eludes me.

Perhaps the commonality between two crashing MRE's are not objects, but assets.
A double instance of the GLTF loader MRE, loading the same model leads to a crash, and a multiple instance of the Blast Door, loading the same sound effect leads to a crash.
For testing purposes, I'll remove the sound and have the Blast Door MRE just use the kit objects to see if my theory pans out.

Removing the sound didn't fix things. So it definitely concerns objects in general and not just "loose" items downloaded using an URL.

I'm still unable to reproduce this bug, but I've changed the way the asset cache is used for glTFs in the linked PR, maybe it'll fix it for you?

I'll have to try. Thing is, it doesn't affect just glTF's, but other kinds of assets as well, but only if they're loaded from different MRE's in the same space... and it only affects specific users, for some reason.
Though, even if the PR fixes things just for glTF's it helps tracking down the root cause of this issue.

Hmmm.... Debugging with mixed-reality-extensions-unity remains a bit inconclusive. I've traced the call down to MRETestBed/ResourceFactory.cs::CreateFromLibrary, which is a stub in lieu of retrieving the 'real' kit object.
If the live code in place of this unconditionally tries to retrieve the kit object from the web into the asset storage, maybe because the second MRE doesn't know it's already loaded by the first one, it could account for the problem.
Taking a wild guess here. As for assets retrieved at runtime - is there a room-scoped bookkeeping to avoid double retrieval, or is it only scoped to contexts?

Built a workaround.
In essence, I'm multiplexing all the different apps into what shows up as a single MRE inside the World Editor. The definitions of the "sub MREs" are places in a JSON file which is referenced and loaded at runtime, but that's beside the point.
Major difference is that the same apps which led to a crash when incorporated separately using different contexts work fine when they're subsumed into a single context.

This update just went out to Altspace, see if you can still repro this bug. I'm closing this issue for now, but please re-open it if it's not fixed. Thanks.


Bug fix/cleanup 20201016

Scope & Purpose
Don't log errors into the log in expected cases
In case of "expected" errors they are returned to the end user or client application, so they can handle the error. there is no need to additionally log these errors into the log and cause alerting.

[x] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :hammer: Refactoring
[ ] :book: CHANGELOG entry made
[x] :muscle: The behavior in this PR was manually tested
[ ] :computer: The behavior change can be verified via automatic tests

Backports:

[x] No backports required

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/12346/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/12348/

Tests blue


"turbo run deploy" only recognizes 1 workspace in github actions, but works locally for both apps. Build & installing works for both in CI & locally.

What version of Turborepo are you using?
1.3.1
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Linux
Describe the Bug
I have 2 apps in my /apps folder in my turborepo.
Locally both are fine and are recognized when it comes to the deploy command, defined as follows:
in package.json:
	"deploy": " turbo run deploy",

and inside turbo.json:
		"deploy": {},

However, in my github action, only one of the app worksapces are detected and the second application does not get deployed.
It is a fairly recent project created with the CLI.
Expected Behavior
I'd like both workspaces to be detected
To Reproduce
It's a difficult one as it's not package vs app, it's two identical apps and it's only for the deployment command that it does not get recognized.
When I run the build command -- both apps get built and are recognized.
Both apps have a "deploy" command, for reference on the one that doesn't work:
		"deploy": "terraform -chdir=\"../../infrastructure\" apply -auto-approve"

Can you make sure the name field in both package.json's is unique? Apologies for the very late reply!

Closing due to inactivity here.


Update non-React dependencies.

I'm not worried about the drop in coverage, it's so minor and likely the result of deleting a few lines of code here and there to account for updated lint errors.


Render with WebGPURenderer

WebGPU is now generally available in newer versions of chrome and firefox and is also supported with Three.js with a new renderer ( WebGPURenderer ).
It would be great if we could make use of this in the WebGLOverlayView, or perhaps a new WebGPUOverlayView (?).
We could drastically improve the performance when rendering complex 3d scenes on the map. <3
I guess this is something that needs to be developed within the google maps js api as well?.
I am curious to hear if its already being worked on and when it might be publicly available? :)

Hi there,
yeah. That is going to take a while...
WebGPU would make it a lot easier to deal with sharing the rendering context, but it would first need a google-maps API version based on WebGPU, and I wouldn't exactly hold my breath on that. It might (and probably will) happen at some point, but it's impossible to tell from the outside if that is something being looked at or pursued by the maps engineering team.


Typos in README for CLPFUDatabase package

In Issue #5, @nmstreethran wrote
There are a couple of typos in the Statement of need in the README of https://github.com/EnergyEconomyDecoupling/CLPFUDatabase (CLPFUDatbase)

@nmstreethran, thanks for this comment.
I ran the README.Rmd and README.md through two spell checks (RStudio and MS Word). I didn't find any typos.
Are you flagging "exergy" as a typo? Exergy is a work different quantification of energy and the subject of the database. So that is not a typo.
I'm closing this issue because I couldn't find a typo. But I invite you to reopen the issue and point out any typos that I missed.

Sorry, I should have been clearer. The typos are in the name of the metapackage in the final paragraph of the Statement of need (it says CLPFUDatbase):
This package (CLPFUDatbase) addresses the need for a simple way to
install all dependencies. CLPFUDatbase is metapackage[^1], whose

@nmstreethran, thanks for clarifying! That's a tough one to see.
Anyway, the typo is now fixed, so I'm closing this issue.


Add ability to enforce connection only to whitelisted peers

Description:
Resolves #3025

How Has This Been Tested?
Manually

Screenshots (if appropriate):

Types of changes:

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.002%) to 22.765% when pulling 480f5f49d4b054cfc8632d413bda9d35c77cef31 on korhaliv:feat/peer-whitelist into 1cbd56a72e3129cfccdf2419674ab1319eda9c2f on LN-Zap:next.


Synthetix/Kwenta build IDs on UI

Feature Proposal
We should add build IDs to the bottom of the Kwenta page to help us as the community narrow down help-desk issues. This can be either tied to a Kwenta commit hash or Synthetix version. Or both.
Rationale
Someone posts an issue to the help-desk and we inquire which version of Kwenta they're on.
Having a build ID that links us to either a commit and/or Synthetix can help us eliminate potential causes. Typically issues arise because someone has an old build cached on their machine and the version of Synthetix is outdated. (Context: sometimes Synthetix contract addresses change with each deployment.)
Implementation

Some research has to be done into how to get commit hashes and/or Synthetix versions.
Note: @synthetixio/contracts-interfaces version is a good proxy to the Synthetix version and might have the granularity we want.

Design Spec
Link to Figma (Hard to see, but small and at the bottom of the page):
https://www.figma.com/file/sghReDviy1lcr6kkNqlc0e/Kwenta-User-Flows?node-id=5267%3A228978
Screenshot from Figma:

Sorry for the two PRs, first one couldn't get my verified commit working properly, but sorted out for the second one that's open


Upload file server-side

Hello,
I want to upload the file not from the client-side, But from a Method server-side to add later deny rules.
When I pass the file from the client to the server to be inserted into the collection it becomes and empty object and it's logical as the Images is on the client.
The question is how to tackle this problem so that I can insert files server-side and handle rules for FS Collections.
Thanks in advance,
Nader Ikladious.

@aldeed any help ?


How to change an existing texture?

Your Question
I'm making a mod that modifies the Newgrounds button on the main menu. I figured out how to change the functionality of the button, but not the texture. How can I change the texture of the Newgrounds button. I've found out what GameSheet the button is in, if that helps.
By the way, may I suggest using the Discussions tab for questions.

There are a few ways to accomplish this:

Use a texture pack that replaces the texture
Change the texture of the button at runtime
Replace the button with a different button that has the new texture

Out of these solutions, 1 is undesirable if your goal is not to change the sprite globally. 2 is possible and works, but the one I would go for is 3. Just create a new button texture, duplicate its string ID and other properties, and replace the old one with it :) You can set the sprite of a button with btn->setNormalImage()

Is undesirable for me because I don't want to change the Newgrounds button everywhere else
I feel like this is the simplest option for me
Yeah this also works.

Thank man

You should also probably use the discussions tab for questions because you can explicitly set the answer.


Function to determine if a project meets a set of requirements

As by using the schema outlined in getting-unstuck-web

This is in getting-unstuck-web instead.


candlestick functional adjustments

figma: https://www.figma.com/file/fLWVwmanAwetVZbujQquEi/Market-Page?node-id=1256%3A780
Remove the sliding bar on the bottom and add zooming/adjusting of chart through mouse wheel zooming.
fix the volume dropdown so it actually effects the chart and changes the label to display the volume numbers on a mouseover.
adjust tooltips to closer match the mocks:

Fixed two of these things, however the zooming/scrolling with mouse is something I'm not positive we need since we have that dropdown that helps you select month/hours/days/etc -- want to get clarification.


Feature/io-ts resolver

Hello everyone ! I've been using react-hook-form with fp-ts and io-ts since last summer on several projects, and I think I can share this resolver with confidence. This might not be the easiest tool (like the whole fp-ts world) but extremely powerful. Let me know what you think about it :)

another resolver! how exciting! Thanks for the PR and @jorisre's review.

thank you @jorisre and @bluebill1049 :)

I'm not quite sure if this is the right section. But one thing is not quite clear to me.
Having this.
const schema = t.type({
  name: tt.NonEmptyString,
  description: tt.NonEmptyString
})

type FormValues = t.TypeOf<typeof schema>

export default function EditJob(job: JobDto) {
  const { register, handleSubmit, watch, formState: { errors } } = useForm<FormValues>({
    resolver: ioTsResolver(schema),
  });

  return (

Edit {job.name}

        <form onSubmit={handleSubmit(onSubmit)}>

            Name:
            <input type="text" {...register("name")} defaultValue={job.name} />

{errors.name?.message}

            Description:
            <textarea {...register("description")} defaultValue={job.description} />

{errors.description?.message}

          <input type="submit" value="Save" />
        </form>

  )
}

On the lines where error messages can be displayed like
{errors.name?.message}
 the property message doesn't exist with this resolver. (They do exist when I use other resolves like yup).
The property type of errors.name seems to now be of type DeepMap<tt.NonEmptyString, FieldError> and I'm not quite sure how I can extract the error message to display from there.

@Matthijn Thank you for reporting the issue. Can you open an issue please ? And provide a codesandbox ? thank you

It may be my mistake. Sorry.
Doing this.
// type FormValues = t.TypeOf<typeof schema>
interface FormValues {
  name: string,
  description: string
}

Seems to work. Instead of the commented out part. (I thought that would prevent me from writing basically the same structure twice).


[PE-D][Tester D] Documentation UG bug

This error message isn't in the UG’s add command section on expected output of command failure.

Labels: type.DocumentationBug severity.VeryLow
original: ph-nathan/ped#2

added expected output for duplicate tutee


[BUG]  Invalid import in event views

Description
in event view import is broken.
Steps to Reproduce
clone repo , run python manage.py migrate
Expected Behavior
should run without any error
Actual Behavior
throws an error Module Not Found backend
Additional Context
from backend.djangoindia.constants import POST, PRIMARY_KEY_SHORT
ModuleNotFoundError: No module named 'backend'
import statement should be updated to -
from djangoindia.constants import POST, PRIMARY_KEY_SHORT
ModuleNotFoundError: No module named 'backend'
Affected Area
Backend
Version
No response
Browser (if frontend)
No response

I Fixed this sometime ago. Can you please pull the latest changes?
Here's our latest event view: https://github.com/djangoindia/djangoindia.org/blob/main/backend/djangoindia/api/views/event.py


User not auto-logged in after resetting password and going to link

Not 100% on this, needs reproduction.
Ideally the flow is:

Click reset password, enter email
Visit email, click link
Reset password
Get logged in and everything is fine and dandy

Step 4 seems to be unhappy.

Couldn't reproduce...


Seminaari (jatkuu kevätlukukaudelta 2021), Seminar, KIM-IT320

plz add students manually from spring 2021
_Dear Sir or Madam,
I am writing in reference to the students' evaluation of the following course (see also message below):
Seminaari (jatkuu kevätlukukaudelta 2021), Seminar, KIM-IT320
Since this course is the continuation of the same course that took place in Spring 2021, there are no new registered students, but the same students who attended the course in the Spring also attended this Semester. It is therefore important that the course feedback form is sent to the students registered for "Seminaari (keväällä 2021 alkava ryhmä)".
With best wishes,
Silvio Cruschina_

teacher revealed that he has collected feedback manually


Add method to set state of CheckButtons.

I kind of need this for one of my projects and think it is somewhat essential to be able to set the default state.
Apart from that thanks for providing these bindings :)

Hi,
Thanks for the pull request.

That was fast, thank you!

No problem.
I have published a minor version 0.3.9 with the relevant changes, similarly for RoundButton, RadioButton, LightButton and ToggleButton. These can be accessed by their respective methods, or via the ButtonExt trait methods using set() and is_set().


How to define image inside a Column?

How to define image inside a Column?

Look at the Axlsx documentation here:
https://github.com/randym/axlsx/blob/master/examples/example.rb#L296-L322


Make the ”repository starred” event more compact

Currently GitHub shows a preview of the repository on your dashboard whenever someone has starred it.

I’d suggest simplifying it so dashboard would be super compact as it used to be.
I would suggest some approaches on how to handle this:

Hide those previews completely.
Add a link to expand the event with a preview.

WDYT?

I wouldn’t mind showing that box on hover, but smaller and not clickable, just as extra info since we already have it in the dom. It can likely be done with CSS only.

I noticed one thing with the stars on the preivew box. I think it shows the number of stars at the moment of the person starring and not the current number of stars which is.. weird! This might not help with the issue, but I wonder why it's kept this way..


add user Xiaohongshu

@cfmcgrady, could you add other companies into user page? Not only Xiaohongshu? Otherwise, we have to invite registered users to submit another pull request.


Draft: Add assets and code for missing infested blocks

This PR adds assets for blocks that are currently not being infested (As villages and player structures currently look a little bit ugly when infested).
Note: In order to get blocks with blockstates (i.e. stairs) to work the Cursor and InfestationTableEntry code had to be altered slightly.

Wow, this is great, I will review this and see about pulling this into the main branch.

This PR completely put cursor pathing out of wack, but I'm working to fix it.

Yes the cursor pathing is completely messed up, it seemed like a good idea at the time but after a little bit of playtesting I ran into a lot of issues... I have a semi working solution which consists of saving allowed non-full block blockstates in a list and checking if they can be replaced. I am not really understanding how it failed though.

Finished Feature and has been merged into the 1.20 branch. Expect an update to the mod soon.

If you have a discord @ Me and ill give you the contributor role @pl-luk


What is the reason of 409 Failure ?

Hello!
I deployed the standalone operator and applied the following yaml.
kubectl apply --server-side -f greeting-schema.yaml

Here is greeting-schema.yaml
---
apiVersion: artifact.apicur.io/v1alpha1
kind: Artifact
metadata:
  name: greeting-schema
  namespace: default
spec:
  version: "1"
  groupId: kafka-apicurio-sample  # namespace
  artifactId: simple-avro-maven-topic  # topic name
  name: greeting
  description: "Avro record for Greeting entity"
  type: AVRO
  labels:
  - avro
  - kafka
  content: |
    {
      "type": "record",
      "name": "Greeting",
      "fields": [
        {
          "name": "Message",
          "type": "string"
        },
        {
          "name": "Time",
          "type": "long"
        }
      ]
    }

The schema is successfully deployed.
But I found the following 409 Failure in the log:
kubectl logs  deployments/apicurio-registry-kube-sync -f

2022-11-02 01:26:39,412 INFO  [io.jav.ope.pro.EventDispatcher] (EventHandler-artifactcontroller) Kubernetes exception 409 Failure executing: PUT at: https://<IP_ADDRESS>/apis/artifact.apicur.io/v1alpha1/namespaces/default/artifacts/greeting-schema. Message: Operation cannot be fulfilled on artifacts.artifact.apicur.io "greeting-schema": the object has been modified; please apply your changes to the latest version and try again. Received status: Status(apiVersion=v1, code=409, details=StatusDetails(causes=[], group=artifact.apicur.io, kind=artifacts, name=greeting-schema, retryAfterSeconds=null, uid=null, additionalProperties={}), kind=Status, message=Operation cannot be fulfilled on artifacts.artifact.apicur.io "greeting-schema": the object has been modified; please apply your changes to the latest version and try again, metadata=ListMeta(_continue=null, remainingItemCount=null, resourceVersion=null, selfLink=null, additionalProperties={}), reason=Conflict, status=Failure, additionalProperties={}). during event processing, ExecutionScope{events=[CustomResourceEvent{action=MODIFIED, resource=[ name=greeting-schema, kind=Artifact, apiVersion=artifact.apicur.io/v1alpha1 ,resourceVersion=85773, markedForDeletion: false ]}], customResource uid: 15c0dd7e-aec3-4f77-88fe-c0b3bff68548, version: 85773} failed

Why is this happening?
This is the got result.
$ kubectl get artifacts.artifact.apicur.io greeting-schema -o yaml --show-managed-fields

apiVersion: artifact.apicur.io/v1alpha1
kind: Artifact
metadata:
  creationTimestamp: "2022-11-02T01:26:38Z"
  finalizers:
  - artifacts.artifact.apicur.io/finalizer
  generation: 3
  labels:
    apicur.io/artifactId: simple-avro-maven-topic
    apicur.io/groupId: kafka-apicurio-sample
    apicur.io/version: "1"
  managedFields:
  - apiVersion: artifact.apicur.io/v1alpha1
    fieldsType: FieldsV1
    fieldsV1:
      f:spec:
        f:artifactId: {}
        f:content: {}
        f:description: {}
        f:groupId: {}
        f:labels: {}
        f:name: {}
        f:type: {}
        f:version: {}
    manager: kubectl
    operation: Apply
    time: "2022-11-02T01:26:38Z"
  - apiVersion: artifact.apicur.io/v1alpha1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:finalizers:
          .: {}
          v:"artifacts.artifact.apicur.io/finalizer": {}
        f:labels:
          .: {}
          f:apicur.io/artifactId: {}
          f:apicur.io/groupId: {}
          f:apicur.io/version: {}
      f:spec:
        f:contentId: {}
        f:globalId: {}
        f:modifiedBy: {}
        f:modifiedOn: {}
        f:properties: {}
        f:state: {}
    manager: okhttp
    operation: Update
    time: "2022-11-02T01:26:39Z"
  - apiVersion: artifact.apicur.io/v1alpha1
    fieldsType: FieldsV1
    fieldsV1:
      f:status:
        .: {}
        f:artifactId: {}
        f:globalId: {}
        f:groupId: {}
        f:status: {}
        f:version: {}
    manager: okhttp
    operation: Update
    subresource: status
    time: "2022-11-02T01:26:39Z"
  name: greeting-schema
  namespace: default
  resourceVersion: "85776"
  uid: 15c0dd7e-aec3-4f77-88fe-c0b3bff68548
spec:
  artifactId: simple-avro-maven-topic
  content: |
    {
      "type": "record",
      "name": "Greeting",
      "fields": [
        {
          "name": "Message",
          "type": "string"
        },
        {
          "name": "Time",
          "type": "long"
        }
      ]
    }
  contentId: 15
  description: Avro record for Greeting entity
  globalId: 16
  groupId: kafka-apicurio-sample
  labels:
  - avro
  - kafka
  modifiedBy: ""
  modifiedOn: 2022-11-02T01:10:39+0000
  name: greeting
  properties: {}
  state: ENABLED
  type: AVRO
  version: "1"
status:
  artifactId: simple-avro-maven-topic
  globalId: 16
  groupId: kafka-apicurio-sample
  status: ready
  version: "1"

Thanks in advance!

Hello, @LittleWat
But as you can see from yaml output your artifact is ready, it should appear in apicurio registry.
As far as i remember this error is temporarily and eventually is self healing, so you may ignore it.

@eshepelyuk Thank you for your reply! I see. Yes, the log level is INFO so we may ignore this.
But I would like to solve this error if possible 🤔  I will read the code deeply.


Docker Hub "latest" tag not updated to 0.15.13

On Docker Hub, the latest experimental (0.15.13) has been added, but the "latest" tag has not been changed to 0.15.13.

Should be fixed now.


Documentation/Example - classifyGif

classifyGif was added in #57
Add usage and output to the documentation.
Also add code to demo app that will properly prompt user to wait while gif is being processed, and then show the user the output in a meaningful way.  I'm thinking graph, possibly with D3.

Presenting the gif classification can look like this:  https://observablehq.com/d/c70415332a1f4cb9
Gif output can be converted to a readabe chart format with the following transform
let results = {name: "results", children: []}
data.map((frame, i) => {
  const lastChild = results.children.slice(-1)[0]
  const currentClass = frame[0].className
  if (lastChild) {
    const lastClass = lastChild.name
    if (currentClass == lastClass) {
      // Add frame to entry
      lastChild.children.push(
        {"name": "frame " + (i+1), "value": 1}
      )
    } else {
      // Create new
      results.children.push({
        name: currentClass,
        children: [
          {"name": "frame " + (i+1), "value": 1}
        ]
      })
    }
  } else {
    // Create very first entry!
    results.children.push({
      name: currentClass,
      children: [
        {"name": "frame " + (i+1), "value": 1}
      ]
    })
  }
  return {...frame[0], frame: i }
})

So getting d3 working in React is a pain.
Even though I have all the code for d3, I apparently can't get my branch to render the SVG.  I might abandon the demo of classifyGif and just document it.

The branch is gifGraph if someone wants to take a stab:
https://github.com/infinitered/nsfwjs/tree/gifGraph

Thinking of switching to bar graph.

GIF is in demo now.  No graphs yet.


Define units from GPkit folder

See https://github.com/aeroa/gpkit-models/pull/24

Now why is Jenkins unhappy... ?

retest this please

retest this please, just to be sure. @galbramc, that was a strange testing failure -- temporary loss of internet connection?

Worth noting that if we define currency by the year, we can easily define exchange rates...

I'm ready to merge this, @bqpd please go ahead if you are too.

:+1:

@whoburg, See also the gpkit-models mod; should that be USD2012?

Hmm good point -- would you basically hard-code the inflation rates by defining USD2012, USD2013, etc relative to some baseline, like USD1913?
I guess the other alternative is to expect users to continually update their code to use current labor rates, etc?

Yerp.

Looks like a loss of internet for some odd reason. No idea what happened there.

I'm very open to adding the year-based dollar units, but of course we should also keep USD for models that don't want to worry about inflation.

see https://github.com/aeroa/gpkit-models/pull/25

see https://github.com/convexopt/gpkit/pull/494


Rectified image withe black borders

Hello,
I have run a kinet2 calibration as suggested in the readme.
 For example, see bellow:

I wanted to ask if this also occurs to your rectified images. I think I understand why these black borders appear ... its because of the distortion rectification ... but I seldom have seen this kind of cushion like distortion ... the kind of distortion I am used to seeing is a barrel type distortion ( see here)
Anyway, do you also have these black regions?
Thanks,
Miguel

Yes, that is normal. And if you undistort an image with a barrel distortion, it will result in black borders, that get bigger to the center, like this one.

Ok, thanks for the feedback, I will close the issue.
One more question though: @wiedemeyer do you have any idea why this did not occur with the kinect 1 or the asus xtion (at least I don't recall it)? Different combinations of RGB and IR cameras? I know the Kinect 2 has a wider field of view ...


update to latest vethcon

The latest vethcon is better than the one embedded here, has the all-in-one-command method, too

Updated in #9


Short-hand predicates

code: take IntoCodePredicate

From<i32>
From<Predicate<i32>>

stdin / stdout: take IntoOutputPredicate

From<str>
From<Predicate<str>>
From<[u8]>

I'm marking as a breaking change out of concern because these kind of concrete API to conversion API changes have broken others in the past.


ReadableType does not distinguish between Integers and Doubles

This is Android specific.
While there is specific API to retrieve Int and Double form ReadableArray and ReadableMap, for some odd reason ReadableType enum does not make this separation and provides only Numeric entry. Having Numeric only is insufficient as one cannot make a decision whether to retrieve a Double or Int from the collection based on this enum value. This is needed for such applications as database interfaces where int vs double makes a huge difference.
Can this be fixed quickly?

cc @kmagiera
I do want to point out that ReadableArray is used to communicate with JS, which has only 64-bit floats. So if you need to specify that you want a Java double vs Java int, you will need to send that information separately from the numeric value.
You mentioned this is specific to Android but I imagine iOS would have a similar issue. Does iOS solve this issue in a way we could port to Android?

This cannot be sent separately. The API is SQL...it expects values...it would be ridiculous to come up with up any reasonable API and force the API user to provide markers for each numeric value I think. Imagine sending a very long list of numeric values as the input...
problem is I have to bind the values to specific SQL types..so I need to know if something is a Double or and Integer...
there is getDouble and getInt...why generalize the ReadableType to Number ?
Have to check on iOS but seems not to be an issue since values are translated directly and I do not have to query for type...

@andpor I think it's a good point.
We used to only have one number type on java side of the bridge and all JS numeric typese were mapped directly to "double". It turned out to be insufficient and we introduced integer as a separate type (with ReadableArray.getInt and ReadableMap.getInt). But since we don't use ReadableArray.getType anywhere in the codebase no one bothered to update it with newly introduced type.
We should update ReadableType now that we support two different numeric types. Required steps are:

Update ReadableType.java: remove Number add Double and Integer
Update implementation of ReadableArray.getType and ReadableMap.getType bothe are implemented in NDK (c++). Good place to start looking into it is https://github.com/facebook/react-native/blob/9647c5a25724ddd259a4febe7e7f26c058ce3fac/ReactAndroid/src/main/jni/react/jni/OnLoad.cpp#L118

Good to hear. The same applies to ReadableMap naturally...

@kmagiera - do you think this will get fixed anytime soon ?

@andpor I'm afraid this is not sth we can fix. I thought about it a little more since I posted my answer and IMO if you don't know the context of what type to expect you should just use double.
In JS all numbers are 64bit floating point types (java double equivalent). When those numbers are passed over the bridge we serialize them to JSON. We can then read it expecting to be an integer, but the only way we can learn that the type is an integer is by parsing it. If you'd like to implement db backend and from JS side you want e se to store slowing point numbers in one column it may have a terrible effect if you determine type based on an individual value as it may happen that this individual value that you wish to be a floating point value will be represented in a form of integer.
getInt have been introduced for the cases when we know that we expect integer on the native side. It provides type checking so that if you send floating-point number by mistake it would redbox. We use it for sending colors represented as 32bit ints or line number properties. IMO it's not suitable for the usecase you've mentioned and you should either make your API explicitly send an information about the type of the column, and then you getInt/getDouble appropriately or just use getDouble all the time as it would give you the exact value as you were sending from JS

I agree with @kmagiera since JavaScript does not distinguish between ints and doubles (aside from TypedArrays but that's unrelated...). There is only a Number type so there's no way for us to know if you meant int or double.
So if you are working with an API that distinguishes between ints and doubles, you need to be explicit about the intended type of your number. For a SQL API, one way to do this is with a prepared query format string like PostgreSQL's INSERT ($1::int, $2::double) INTO .... Then you'd parse the format string and understand whether a numeric value should be treated as an int or double.
But not sure what React Native can do for you here. Fundamentally this seems like an issue working with two different type systems (JS and Java's).

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/readabletype-does-not-distinguish-between-integers-and-doubles
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


Remove deadcode and clone/own.

Removing what appears to be unnecessary/dead code.
Also removed some code duplication around resolving fully qualified names.
My changes affect the following components

[x] Tests

Types of changes

[x] Enhancement or new feature (adds new functionality).

PG2 2952 🐎

@rabbah, can you rebase having trouble with a PG.

done - thanks for the pg.

PG4 1515 ☃

PG4 1515 🔵


Feedback for “Custom Theme”

customCSS = ["css/custom.css"] # Files in the list will be loaded in order.
Where in the directory structure is css/ expected?

I've updated docs to indicate where custom files should be placed: https://g1en.site/hugo-theme-dream/custom.
Thanks for your question, which helped me find the missing part of the documentation. Now I'll close it. Feel free to open a new issue if you still have some questions. 😃


activation-script module: add environment.usrbinenv option

Step 1 of https://github.com/NixOS/nixpkgs/pull/12474#issuecomment-172913145.
I based this on environment.binsh
Bikesheds before we merge this:

Is this a reasonable option name?
Should it be visible = false? (I copied that from binsh)
Should it carry a warning too, like binsh?

I'd probably make it visible (this is probably why @peti couldn't find the other one even though he knew it was there) and give it types.nullOr types.path, to allow us to not create the symlink at all. I'd also amend binsh to be visible and nullable in the same way, but wouldn't hold up this PR for that if it's controversial since I think this is providing 90% of the value right now.

+1 from me

Looks good to me, too. I'd be curious to know, though, why @edolstra set visible = false to the binsh option in 89e983786a4e2cf6dd238af4df6da373cc01e62b. There's probably a good reason for it, but I don't see it.

Hm, I'm not very enthusiastic about this kind of option. (That's probably also why I hid the binsh option.) I mean, it just leads to needless incompatibility between NixOS systems. For instance, a script that works on one system might break on another because the user thought it was a great idea to make /usr/bin/env something else. Then we'll need test cases for different /usr/bin/env implementations, etc.
NixOS uses coreutils everywhere so IMHO there is no reason not to use it for /usr/bin/env.

@edolstra we made pretty much all the points on that topic that can be made in the other ticket (#12474). In short, NixOS isn't the only platform nixpkgs must run on, and we should commit to a UNIX env, not a coreutils one. We can expect that from other platforms, but e.g., Mac OS doesn't have a GNU env and never will. Furthermore, given that 95% of what we package is designed to work on general UNIX platforms, almost nobody else ever assumes they get anything more than a standard env in their shebangs.

@copumpkin Yes, but we're talking about NixOS modules here, not Nixpkgs in general. Now, there might be a case for re-using certain NixOS modules outside of NixOS (e.g. for building light-weight containers that are not full NixOS systems), but then we should talk about what the requirements of those environments are and whether we want to commit to supporting them.

@edolstra but the point is that these NixOS modules exist (among other things) to provide the "standard nix impure environment" to any nixpkgs stuff running under NixOS. Given that nixpkgs can't commit to a coreutils /usr/bin/env, I don't see why NixOS would go out of its way to ensure that we adhere to a stricter standard.

Nixpkgs can't commit to that, but NixOS can, just like we commit to the Linux kernel, Glibc, systemd, bash, ...

@edolstra but do we want to? it'll only lead to further myopia and ease of breaking other ecosystems that NixOS folk can't test easily on. Most of nixpkgs and the dependencies tracked by Nix proper seem manageable, but I just see no reason to favor defaulting to something nonstandard and allowing people to implicitly depend on non-standard behavior when we already have a big problem with the linux folks breaking other platforms all the time. It's not that I think this will majorly affect that pain in one way or another, but I also don't see why we'd go out of our way to make it easier to do.

The usrbinenv = null option seems useful, even if other values are not. Nothing in NixOS uses /usr/bin/env and it can cause test builds to succeed when they would fail in a sandboxed environment.

Maybe an okay'ish compromise would be to add the option @nathan7 proposes, but to set it to visible = false, like the binsh option currently is?
That allows people who know what they're doing to do it, but at the same time we don't advertise this feature to random users who don't understand what it means, mess with the setting for fun, and then start filing bug reports because their system won't boot any more.
Personally, I think that adding the feature @nathan7 wants is fine because it adds very little complexity to our code. The changes are minuscule, really. So there seems no harm in having that option as long as it's unlikely that people abuse it. Making it invisible by default would accomplish that.

I'd be fine with hiding them both, sure.

There you go, made it invisible.


Remove arc flags from the skipped params

Naive solution for https://github.com/MobileNativeFoundation/rules_xcodeproj/issues/2206

Closing as not a perfect solution (see https://github.com/MobileNativeFoundation/rules_xcodeproj/issues/2206#issuecomment-1551975929)


fix type

fix type

Hi @edwardramsey , please follow the instructions on this page https://github.com/hyperledger-labs/fabric-token-sdk/pull/526/checks?check_run_id=20085215992 to fix the DCO issue. Thanks much :)


Added option to set custom name for playground

I added a command line option to give playgrounds a custom name instead defaulting to the date.
Creating a PR because I thought maybe someone else mind find this useful.
Still getting my arms around swift so I hope I didn't make any sloppy changes.

I am cancelling this PR because I just realized that the -t argument will give my playground the name I needed.
For some reason I thought that the playground name was always defaulting to the date no matter where I set the target path.
Sorry about

@IronLuffy55 No problems at all, those things are easy to miss. Thanks a lot anyway for contributing 🙂


Add Terraform to store watch_dev state

These resources were created manually, but have been imported into the TF state to ensure they get cleaned up.

I'm actually going to create a new RD-WATCH repo for the WATCH deployment soon here as #435 is soon to merge and the main instance we have on Heroku is now under RD-OpenGeo: https://github.com/ResonantGeoData/RD-OpenGeo
see https://github.com/ResonantGeoData/RD-OpenGeo/issues/10

@banesullivan As soon as you create the new repo for WATCH, @venkatabhishek can start to test deployment using that as a base Django project.
All the Terraform in this PR is ephemeral, and will be removed after WATCH deployment development ends.
More broadly, it will be much easier if we can keep all the TF for resources on the RD-OpenGeo AWS account under a single repo.  To the extent we want a DNS entry pointing to WATCH under the resonantgeodata.com domain, we'll need to keep that aws_route53_record with the rest of the TF. If we're looking for real separation, then maybe it's time for WATCH to register its own domain name.
It probably will be good for WATCH to get its own SMTP server too (which the WATCH support team should be able to help with) and move out of the basement.

Merging this PR now, since it only changes TF, and that CI looks good.


whats the problem with compose?

[<IP_ADDRESS>] ~/cs $ vim docker-compose.yml
[<IP_ADDRESS>] ~/cs $ docker compose build && docker compose up
[+] Running 1/1
✔ Container cs2-dedicated  Recreated                                                                                0.1s
Attaching to cs2-dedicated
cs2-dedicated  | Redirecting stderr to '/home/steam/Steam/logs/stderr.txt'
cs2-dedicated  | Logging directory: '/home/steam/Steam/logs'
cs2-dedicated  | [  0%] Checking for available updates...
cs2-dedicated  | [----] Downloading update (0 of 21,115 KB)...
cs2-dedicated  | [  0%] Downloading update (1,108 of 21,115 KB)...
cs2-dedicated  | [  5%] Downloading update (4,039 of 21,115 KB)...
cs2-dedicated  | [ 19%] Downloading update (5,625 of 21,115 KB)...
cs2-dedicated  | [ 26%] Downloading update (6,975 of 21,115 KB)...
cs2-dedicated  | [ 33%] Downloading update (8,245 of 21,115 KB)...
cs2-dedicated  | [ 39%] Downloading update (9,708 of 21,115 KB)...
cs2-dedicated  | [ 45%] Downloading update (10,979 of 21,115 KB)...
cs2-dedicated  | [ 51%] Downloading update (12,258 of 21,115 KB)...
cs2-dedicated  | [ 58%] Downloading update (13,541 of 21,115 KB)...
cs2-dedicated  | [ 64%] Downloading update (14,972 of 21,115 KB)...
cs2-dedicated  | [ 70%] Downloading update (16,274 of 21,115 KB)...
cs2-dedicated  | [ 77%] Downloading update (17,578 of 21,115 KB)...
cs2-dedicated  | [ 83%] Downloading update (18,896 of 21,115 KB)...
cs2-dedicated  | [ 89%] Downloading update (20,178 of 21,115 KB)...
cs2-dedicated  | [ 95%] Downloading update (21,115 of 21,115 KB)...
cs2-dedicated  | [100%] Download complete.
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Extracting package...
cs2-dedicated  | [----] Extracting package...
cs2-dedicated  | [----] Extracting package...
cs2-dedicated  | [----] Extracting package...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Installing update...
cs2-dedicated  | [----] Cleaning up...
cs2-dedicated  | [----] Update complete, launching Steamcmd...
cs2-dedicated  | UpdateUI: skip show logoCWorkThreadPool::~CWorkThreadPool: work processing queue not empty: 3 items discarded.
cs2-dedicated  | steamcmd.sh[7]: Restarting steamcmd by request...
cs2-dedicated  | Redirecting stderr to '/home/steam/Steam/logs/stderr.txt'
cs2-dedicated  | Logging directory: '/home/steam/Steam/logs'
cs2-dedicated  | [  0%] Checking for available updates...
cs2-dedicated  | [----] Verifying installation...
cs2-dedicated  | UpdateUI: skip show logoSteam Console Client (c) Valve Corporation - version<PHONE_NUMBER>
cs2-dedicated  | -- type 'quit' to exit --
cs2-dedicated  | Loading Steam API...IPC function call IClientUtils::GetSteamRealm took too long: 46 msec
cs2-dedicated  | OK
cs2-dedicated  | IPC function call IClientUtils::SetSpew took too long: 46 msec
cs2-dedicated  |
cs2-dedicated  | Connecting anonymously to Steam Public...OK
cs2-dedicated  | Waiting for client config...OK
cs2-dedicated  | Waiting for user info...OK
cs2-dedicated  |  Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
cs2-dedicated  | IPC function call IClientAppManager::GetUpdateInfo took too long: 42 msec
cs2-dedicated  |  Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
cs2-dedicated  | IPC function call IClientAppManager::GetUpdateInfo took too long: 50 msec
cs2-dedicated  | Error! App '730' state is 0x202 after update job.
cs2-dedicated  | CWorkThreadPool::~CWorkThreadPool: work processing queue not empty: 1 items discarded.
cs2-dedicated  | entry.sh: line 72: cd: /home/steam/cs2-dedicated/game/bin/linuxsteamrt64: No such file or directory
cs2-dedicated  | pre-hook: noop
cs2-dedicated  | Starting CS2 Dedicated Server
cs2-dedicated  | entry.sh: line 121: ./cs2: No such file or directory
cs2-dedicated  | post-hook: noop
cs2-dedicated exited with code 0

with default docker run same:
[<IP_ADDRESS>] ~/cs/CS2/examples $ docker logs cs2                                                                          git:main
Redirecting stderr to '/home/steam/Steam/logs/stderr.txt'
Logging directory: '/home/steam/Steam/logs'
[  0%] Checking for available updates...
[----] Downloading update (0 of 21,115 KB)...
[  0%] Downloading update (1,112 of 21,115 KB)...
[  5%] Downloading update (3,852 of 21,115 KB)...
[ 18%] Downloading update (5,935 of 21,115 KB)...
[ 28%] Downloading update (7,247 of 21,115 KB)...
[ 34%] Downloading update (8,543 of 21,115 KB)...
[ 40%] Downloading update (9,807 of 21,115 KB)...
[ 46%] Downloading update (11,071 of 21,115 KB)...
[ 52%] Downloading update (12,319 of 21,115 KB)...
[ 58%] Downloading update (13,711 of 21,115 KB)...
[ 64%] Downloading update (14,975 of 21,115 KB)...
[ 70%] Downloading update (16,239 of 21,115 KB)...
[ 76%] Downloading update (17,487 of 21,115 KB)...
[ 82%] Downloading update (18,751 of 21,115 KB)...
[ 88%] Downloading update (19,999 of 21,115 KB)...
[ 94%] Downloading update (21,115 of 21,115 KB)...
[100%] Download complete.
[----] Installing update...
[----] Extracting package...
[----] Extracting package...
[----] Extracting package...
[----] Extracting package...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Cleaning up...
[----] Update complete, launching Steamcmd...
UpdateUI: skip show logoCWorkThreadPool::~CWorkThreadPool: work processing queue not empty: 3 items discarded.
steamcmd.sh[7]: Restarting steamcmd by request...
Redirecting stderr to '/home/steam/Steam/logs/stderr.txt'
Logging directory: '/home/steam/Steam/logs'
[  0%] Checking for available updates...
[----] Verifying installation...
UpdateUI: skip show logoSteam Console Client (c) Valve Corporation - version<PHONE_NUMBER>
-- type 'quit' to exit --
Loading Steam API...IPC function call IClientUtils::GetSteamRealm took too long: 51 msec
OK
Connecting anonymously to Steam Public...OK
Waiting for client config...OK
Waiting for user info...OK
Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
IPC function call IClientAppManager::GetUpdateInfo took too long: 55 msec
Error! App '730' state is 0x202 after update job.
CWorkThreadPool::~CWorkThreadPool: work processing queue not empty: 1 items discarded.
entry.sh: line 72: cd: /home/steam/cs2-dedicated/game/bin/linuxsteamrt64: No such file or directory
pre-hook: noop
Starting CS2 Dedicated Server
entry.sh: line 121: ./cs2: No such file or directory
post-hook: noop

i got it, no space


bat: insert less into PATH

Motivation for this change
The default pager for bat is less -RF --no-init. Overriding this to less -RF in order to enable alternate screen behavior (e.g. page up/down keys) causes the -F flag to behave badly in the version of less shipped on macOS. The simplest way to fix this is to simply put Nix's less into the PATH so it takes precedence.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[X] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build bat


[Perf] Refactoring ControllerActionInvoker to avoid coercing return type to Task

Targeted to fix #4539
Will update the PR with perf numbers.

:watch:

:up::date:

:shipit: after addressing https://github.com/aspnet/Mvc/pull/4661#discussion_r63934361


pangyp finally deprecated?

Just saw that while updating  :smile:

npm WARN deprecated<EMAIL_ADDRESS>use node-gyp@3+, it does all the things

Probably should update the Readme.
Good job @rvagg! :+1:

thanks, done


Update navbar to use the black logo for hover effect

Good that you spotted the missing search field on mobile. I'll get right on that.

The missing search field on mobile was introduced elsewhere, and not in this change.


Feature request about ability to specify custom X-Digest scheme to prevent browser's popup

I'm submitting a ...
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

I stuck with latest version 1.1.3 with inability use custom scheme. I'd like to use scheme X-Digest because default Digest opens browser dialog with user credentials request. Previous version which I used 1.0.3 allowed use that scheme.

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

@lynx-r
Thank you for the feature request.
This can actually work on the previous version. But the new version has a stricter validation rule. So it was bug that allowed you to use the custom scheme.
It is a bad idea to maintain a non-standardized schema name by default. as this package strictly follows the standard, it can follow to an unexpected result or security problem. But I'll add the ability to explicitly specify your own schema.
You can use replace as a temp solution
ClientDigestAuth.analyze(headers['WWW-Authenticate'].replace('X-Digest', 'Digest'));

I am adding this feature asap.

Information for anybody who implements a server side. The browser popup can be avoided if send a response with OK status.


Feature Request: enable netavark to simply connect containers without any other network action

I am using podman with CNI and specifically I am using that file
cat /etc/cni/net.d/access.conflist
{
   "cniVersion": "0.4.0",
   "name": "access",
   "plugins": [
      {
         "type": "bridge",
         "bridge": "access",
         "isGateway": false,
         "ipMasq": false,
         "hairpinMode": true,
         "ipam": {
            "type": ""
         }
      }
   ]
}

where the bridge have been created externally and manually. I would like to be able to the same.
Thanks

is this something you want to work on?

Is this different from an internal network (no external route) which we support already?

Internal blocks routing from the bridge interface via sysctl that may be different from CNI.
As for no ip addresses there is --ipam-driver=none option which should come close.


Resolve Go 1.11 compile issue introduced in 2cd9f5e8c5b2

The 1.11 API for Sub64() was accidentally copied from math/bits which doesn't
match the behavior expected by big_ctx.
Fixes #126

How would you like me to list you in the AUTHORS document, if at all?

Not at all, but thank you.


Seems that path created by nx module federation is not recognised by srleecode/domain

I created a nx module-federation app following the instructions ad dynamic-module-federation-with-angular#creating-a-new-dynamic-host-applicationr. To create the application do the following at the command prompt:
  # Crate an empty workspace
  npx create-nx-workspace@latest --preset=empty --pm=pnpm
  pnpm add -D @nx/angular @nx/workspace

# create the module-federation dynamic host - replace
nx g @nx/angular:host medsoft --dynamic --standalone --standaloneConfig --no-interactive

# create the module-federation remote
nx g @nx/angular:remote registration --host medsoft --standalone --standaloneConfig --no-interactive

Run the application
nx serve medsoft

Now that the application works at http://localhost:4200/, install @srleecode/domain with
pnpm add -D @srleecode/domain

Open the project withs Visual studio code and click on installed nx extension icon.
Click generate and select appGrouping foldder:

Note the error in the terminal

installed packages are shown below:
{
   "name": "@xom/source",
   "version": "0.0.0",
   "license": "MIT",
   "scripts": {},
   "private": true,
   "dependencies": {
      "@angular/animations": "~16.0.0",
      "@angular/common": "~16.0.0",
      "@angular/compiler": "~16.0.0",
      "@angular/core": "~16.0.0",
      "@angular/forms": "~16.0.0",
      "@angular/platform-browser": "~16.0.0",
      "@angular/platform-browser-dynamic": "~16.0.0",
      "@angular/router": "~16.0.0",
      "rxjs": "~7.8.0",
      "tslib": "^2.3.0",
      "zone.js": "~0.13.0"
   },
   "devDependencies": {
      "@angular-architects/ddd": "^3.1.1",
      "@angular-devkit/build-angular": "~16.0.0",
      "@angular-devkit/core": "~16.0.0",
      "@angular-devkit/schematics": "~16.0.0",
      "@angular-eslint/eslint-plugin": "~16.0.0",
      "@angular-eslint/eslint-plugin-template": "~16.0.0",
      "@angular-eslint/template-parser": "~16.0.0",
      "@angular/cli": "~16.0.0",
      "@angular/compiler-cli": "~16.0.0",
      "@angular/language-service": "~16.0.0",
      "@nx/cypress": "16.2.1",
      "@nx/eslint-plugin": "16.2.1",
      "@nx/jest": "16.2.1",
      "@nx/js": "16.2.1",
      "@nx/linter": "16.2.1",
      "@nx/web": "16.2.1",
      "@nx/workspace": "16.2.1",
      "@schematics/angular": "~16.0.0",
      "@srleecode/domain": "^13.10.18",
      "@types/jest": "^29.4.0",
      "@types/node": "16.11.7",
      "@typescript-eslint/eslint-plugin": "^5.58.0",
      "@typescript-eslint/parser": "^5.58.0",
      "cypress": "^12.11.0",
      "eslint": "~8.15.0",
      "eslint-config-prettier": "8.1.0",
      "eslint-plugin-cypress": "^2.10.3",
      "jest": "^29.4.1",
      "jest-environment-jsdom": "^29.4.1",
      "jest-preset-angular": "~13.1.0",
      "nx": "16.2.1",
      "nx-cloud": "latest",
      "prettier": "^2.6.2",
      "ts-jest": "^29.1.0",
      "ts-node": "10.9.1",
      "typescript": "~5.0.2"
   }
}

Thanks for helping

Can you update "@srleecode/domain" to a 16 version and try it again

The terminal error message has changed as shown below, but still does not execute as expected:

I updated to 16.2.2. Using that I tried the above steps and was able to run the generator with any errors. Can you try it with 16.2.2. If you still get the error, can you create a repository with an example that gets  the error you are seeing.

Working!! (16.2.2) Thank you.

Using verstion 16.2.2
Spoke a little to soon. The following are working fine: Generation of

grouping folders
application
domain
util

The following are not working: Generation of:

component
directive
infrastructure
presentation-layer

The ones that are not working have a similar error in the terminal as shown below:

I saw the same error in the initial issue - maybe a little tweaking in the ones not working similar to what was done before.
Thanks.

Does the grouping folder you are using exist? For example, locally this works npx nx generate @srleecode/domain:ngDirective --groupingFolder=libs\ng-medsoft\test --name=test --no-interactive --dry-run  and this fails with the error you are seeing npx nx generate @srleecode/domain:ngDirective --groupingFolder=ng-medsoft\test --name=test --no-interactive --dry-run .

You were correct! It is now working.


feat: kong connection timeout (#688)

Introduces a timeout (fixed at 3 seconds per @hbagdi's request) for the initial Kong API call

fixes #766

 Issue #688: Add timeout to initial admin API connection attempt


chore: copy & redirect interval changes

Description
This PR extends visible time of the thank-you screen, and adds the end loader to all onboarding routes (per a discussion w/ @jpoczik).

This PR resolves []
QA Testing

Acceptance Criteria

Setup Instructions

PR Checklist

[x] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[x] I hid my changes behind a feature flag.
[ ] I have prefixed changes that need to be tested during a release QA with [NEEDS EXTERNAL QA] on the changelog.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Screenshot as proof. 😁