Silent error on imported type errors

Thanks for creating/maintaining such a great library it's been nice to use but for one issue that I can't seem to resolve.
Current Behavior
If I import a type that has some error the dist files are "Compiled successfully" according to the terminal output but they are not generated in the dist directory. It seems that an error is being thrown that prevents the build because the file does not appear in the dist output but it is not exposed as an error in the terminal output.
In the screenshot under DivProps there is a type being used that is not imported. When the output files are generated to 'dist' you can see that there isn't a file for types.ts

If I comment out IconProps then types.d.ts is generated.

If I move the type definitions to the component file instead of importing them I can see the error in the console output as expected. However, I can't move all the types into their respective component files because there will be many shared interfaces and types.

It's quite possible that I am completely missing something obvious here but after spending hours searching and trying to resolve the issue I thought I would submit an issue. 🙏

Expected behavior
That an error would be displayed in the terminal output when importing the types. I would expect the error to be the same as the error thrown when the types are added to the component file.

Suggested solution(s)
Checking to see if errors are being thrown but not surfaced to the console output

Additional context

Your environment

Software
Version(s)

TSDX
0.12.3

TypeScript
3.8.2

Browser
Opera 66.0.3515.115

npm/Yarn
1.19.1

Node
v10.15.3

Operating System
macOS 10.15.2

Huh, that's a pretty strange error.
It might be related to #500 , which has yet to be released (see #512), but I'm not sure it is. There's a now-deprecated moveTypes() function internally that would have some transient errors that were swallowed (we hit them in CI pretty frequently), but if that were the problem, the types should be generated fine on a retry. Does this error persist after retries?
Even then, moveTypes() is only called once the types exist... a type error would occur before then.
There aren't any other relevant overbroad catches that might silence errors, at least not in TSDX itself (possibly in a dependency upstream)
The only other thing I might take a guess at being a possibility is that this file gets treeshaken out and so doesn't error when it is it's own separate file, but then commenting out code shouldn't make it appear.
Would need a minimal repro to really do any further investigation if that doesn't help. Seems like some weird edge case, might possibly be an error upstream in rollup-plugin-typescript2

Thought I'd update here that once I became a maintainer of rpt2, I found that this issue was upstream in https://github.com/ezolenko/rollup-plugin-typescript2/issues/211.
I recently fixed that upstream issue in https://github.com/ezolenko/rollup-plugin-typescript2/pull/406, which was released in rpt2 0.34.0.


⚠ Processors has degraded performance

In 5069695, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 46 ms

Resolved: Processors performance has improved in ebfec4f.


Resolve lldpd CVE-2023-41910

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Add patch to resolve lldpd CVE-2023-41910
Change Log

Add patch to resolve CVE-2023-41910

Does this affect the toolchain?

NO
Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2023-41910

Test Methodology

Pipeline build id: 420054

Was not aware of that PR initially. I'll close this PR.


Update files for 1.7

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@JimmyCYJ: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.7
cece959a7db384dc9d8cab7ec6d2688010b8d491
link
/test release-notes_istio_release-1.7

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Logging Issues

Log isn't logging

 Name changes
Avatar changes
Does it log playing games?
User unBans if it does?

Most likely Kwoth needs to upgrade the Discord.NET and apply some fixes with the user update socket.

That's exactly what I thought 😄

What's actually happening is that the bot can not see the username or avatar update changes until its rebooted.

To add to this, if I set .logserver 1 true and reboot the bot, it will not log. I have to toggle logging. Have tried in multiple channels. Did some other troubleshooting with @Kwoth the other day. Who knows. ¯\_(ツ)_/¯


Unreliables, custom exception handler.

If an exception is thrown in
public static <T> T retryUntilSuccess(final int timeout, @NotNull final TimeUnit timeUnit, @NotNull final Callable<T> lambda)
it will be wrapped into a TimeoutException.
It might be necessary in some cases to abort execution immediately on some exception, which is not possible at the moment.
One possible solution is to add an optional exception handler argument - Consumer<Exception> handler, that would just rethrow the exception here:
try {
                        return lambda.call();
                    } catch (Exception e) {
                        // Failed
                        LOGGER.trace("Retrying lambda call on attempt {}", attempt[0]++);
                        lastException[0] = e;
                    }

Ok, so what I did:
I added a varargs list of exception classes to all Unreliables methods upon which Unreliables should not make further retries and stop execution immediately.
@rnorth could you please check the linked PR and advise what you think about such approach?
There are some cases in testcontainers code when it would make sense not to retry upon some specific exception, but currently it's not possible, Unreliables will keep retrying to exec the lambda until timeout is reached.


balena-yocto-scripts: Update to v1.12.11

Update balena-yocto-scripts from 1.11.0 to 1.12.1
Changelog-entry: Update balena-yocto-scripts from v1.11.0 to v1.12.1
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@balena-ci I self-certify!

@resin-jenkins retest this please

@resin-jenkins add to whitelist

@resin-jenkins retest this please

@resin-jenkins retest this please


Support compilation of generated client with pnpm

Problem
I'm trying to build the generated client as a package to be consumed from a web app.
The problem is that when trying to compile tsc fails with the following error:
// $ tsc
.wundergraph/generated/wundergraph.factory.ts:7:14 - error TS2742: The inferred type of 'createOperation' cannot be named without a reference to<EMAIL_ADDRESS>This is likely not portable. A type annotation is necessary.

7 export const createOperation = createOperationFactory<InternalClient, Role>();
               ~~~~~~~~~~~~~~~

.wundergraph/generated/wundergraph.factory.ts:7:14 - error TS4023: Exported variable 'createOperation' has or is using name 'LiveQueryConfig' from external module "/workspace/node/common/temp/node_modules/.pnpm/@wundergraph+sdk@0.132.1/node_modules/@wundergraph/sdk/dist/operations/operations" but cannot be named.

7 export const createOperation = createOperationFactory<InternalClient, Role>();
               ~~~~~~~~~~~~~~~

The zod error can be solved by adding zod to types in tsconfig:
{
  "compilerOptions": {
    "types": ["node", "zod"]
}

The second error can't be solved because LiveQueryConfig is not exported. The problem is solved by modifying operations.d.ts:
export interface LiveQueryConfig {
    enable: boolean;
    pollingIntervalSeconds: number;
}

Suggested solution
Export LiveQueryConfig.
Alternatives
Generate self-contained TypeScript models for TypeScript Operations (#582).
Additional context
The problem occurs only when using pnpm.
Version: 0.127.1
Commit: 1dedf4e05e99f5040715a9792ee63c451c8db0c6
Date: 2023-01-23T10:11:51Z
BuiltBy: ci

@andcan We released a fix for the LiveQueryConfig issue.
We don't have a plug and play solution for the zod error yet (it happens with pnpm and the nextjs client and swr too).
Setting "preserveSymlinks": true in tsconfig resolves it.

Thank you for the quick fix!  Unfortunately I'm using rush, and it does not play well with preserveSymlinks. I'll keep zod in types for now.

I think this is still relevant.

I think this is still relevant.

I agree. Bumping internally.

@andcan adding zod to types is still requirement, but other than that it should work. I set up an example bundling the client using tsup:
https://github.com/wundergraph/publish-client

Thank you! I'll try asap.


Assertion error while uploading "cvat for videos"

My actions before raising this issue

[x] Discussed this with developer in gitter

Current Behaviour
I has some annotated video in one deployment and I am willing to copy this video with annotations on other one. So, I dumped annotations as cvat for video from old and uploaded them to the new. And here is an error.
Other formats like "cvat for images" or "labelme" works fine.
annotations.zip
Your Environment
CVAT version 1.2.0 on both deployments
Error log
ERROR:rq.worker:Traceback (most recent call last):
  File "/opt/venv/lib/python3.8/site-packages/rq/worker.py", line 936, in perform_job
    rv = job.perform()
  File "/opt/venv/lib/python3.8/site-packages/rq/job.py", line 684, in perform
    self._result = self._execute()
  File "/opt/venv/lib/python3.8/site-packages/rq/job.py", line 690, in _execute
    return self.func(*self.args, **self.kwargs)
  File "/usr/lib/python3.8/contextlib.py", line 75, in inner
    return func(*args, **kwds)
  File "/home/django/cvat/apps/dataset_manager/task.py", line 758, in import_task_annotations
    task.import_annotations(f, importer)
  File "/home/django/cvat/apps/dataset_manager/task.py", line 641, in import_annotations
    importer(src_file, task_data, **options)
  File "/home/django/cvat/apps/dataset_manager/formats/registry.py", line 36, in __call__
    f_or_cls(*args, **kwargs)
  File "/home/django/cvat/apps/dataset_manager/formats/cvat.py", line 570, in _import
    load(p, task_data)
  File "/home/django/cvat/apps/dataset_manager/formats/cvat.py", line 517, in load
    annotations.add_track(track)
  File "/home/django/cvat/apps/dataset_manager/bindings.py", line 404, in add_track
    self._call_callback()
  File "/home/django/cvat/apps/dataset_manager/bindings.py", line 385, in _call_callback
    self._create_callback(self._annotation_ir.serialize())
  File "/home/django/cvat/apps/dataset_manager/task.py", line 600, in create
    self._patch_data(data, PatchAction.CREATE)
  File "/home/django/cvat/apps/dataset_manager/task.py", line 590, in _patch_data
    self._merge_data(_data, jobs[jid]["start"], self.db_task.overlap)
  File "/home/django/cvat/apps/dataset_manager/task.py", line 594, in _merge_data
    annotation_manager.merge(data, start_frame, overlap)
  File "/home/django/cvat/apps/dataset_manager/annotation.py", line 149, in merge
    tracks.merge(data.tracks, start_frame, overlap)
  File "/home/django/cvat/apps/dataset_manager/annotation.py", line 233, in merge
    cost_matrix[i][j] = 1 - self._calc_objects_similarity(
  File "/home/django/cvat/apps/dataset_manager/annotation.py", line 388, in _calc_objects_similarity
    obj1_shapes = TrackManager.get_interpolated_shapes(obj1, start_frame, end_frame)
  File "/home/django/cvat/apps/dataset_manager/annotation.py", line 737, in get_interpolated_shapes
    assert shape["frame"] > curr_frame
AssertionError

@PMazarovich I can't find the related error logs in gitter, could you please attach them as well?

Error log added

@PMazarovich, I've failed to reproduce this error. I've created a task with 18300 frames of size 1280x720, segment size 1000 and overlap 5. Annotations were imported after few minutes. Commit id 98e2bf40ac0afeae99b46feeb8016503f4b1699c (read as current develop after 1.2.0).

@PMazarovich , could you please confirm that the issue was fixed? Look at the comment from @zhiltsov-max


[Explore]control panel and chart area shift up after changing data viz type

Actual results

chart titles and all buttons are out of the view

Open any chart in Explore
Change VISUALIZATION TYPE
Run query

Environment
latest master

@kgabryje could it due to the tab refactoring?

@kgabryje could it due to the tab refactoring?

I think it's the Modal, actually.

@kgabryje could it due to the tab refactoring?

I think it's the Modal, actually.

thanks for the diagnosis! @kgabryje this is top priority 🙏

On it

tested, it's all fixed! thank you both @kkucharc @kgabryje


workbench.action.terminal.focus is disappear after an update

After update to July 2021 Update (1.58.1), I realized that the cursor does not move automatically to terminal after I run code from "Code Runner" extension. I rechecked the setting and found that workbench.action.terminal.focus has been faded off and show "Unknown Configuration Setting". Is there a way to enable it again, or another way to change focus to terminal ?

I have the same problem when opening my first new integrated terminal (e.g. with Ctrl-`).  The terminal no longer gets input focus.  Terminals added after the first do get focus.
The same happens with a fresh ~/Library/Application Support/Code.
Version: 1.58.1
Commit: 2aeda6b18e13c4f4f9edf6667158a6b8d408874b
Date: 2021-07-13T06:22:33.013Z
Electron: 12.0.13
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0 (macOS 10.15.7)

#128255

Yep, 1.58.2 fixed it for me as well.
Version: 1.58.2
Commit: c3f126316369cd610563c75b1b1725e0679adfb3
Date: 2021-07-14T22:09:06.581Z (1 day ago)
Electron: 12.0.13
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.4.0

Problem solved with 1.58.2 patch.


The FLOPS per GPU reported for the Megatron GPT model by the DeepSpeed Flops Profiler is much lower than that reported in the logs when we run pretrain_gpt.py[BUG]

Describe the bug
The FLOPS per GPU reported for the Megatron GPT model by the DeepSpeed Flops Profiler is much lower than that reported in the logs when we run pretrain_gpt.py (of Megatron-DeepSpeed)
Also, when ds_pipeline_enabled=True, the Profiler doesn't generate the Profile Summary. Why does this happen?
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The TFLOPS per GPU reported in the profiler Summary should match with that reported by the logs.
Also, the Profile Summary should get printed when ds_pipeline_enabled=True
ds_report output
Profiler reports FLOPS per GPU as 13.36 TFLOPS, whereas the log prints the FLOPS per GPU as 125.18 TFLOPs
Profiler printed Samples/s is 49.55 and that printed by the log is 50.510, which match.
Is the FLOPS per GPU computation by the profiler incorrect?
0: -------------------------- DeepSpeed Flops Profiler --------------------------
0: Profile Summary at step 2:
0: Notations:
0: data parallel size (dp_size), model parallel size(mp_size),
0: number of parameters (params), number of multiply-accumulate operations(MACs),
0: number of floating-point operations (flops), floating-point operations per second (FLOPS),
0: fwd latency (forward propagation latency), bwd latency (backward propagation latency),
0: step (weights update latency), iter latency (sum of fwd, bwd and step latency)
0:
0: world size: 128
0: data parallel size: 16
0: model parallel size: 8
0: batch size per GPU: 8
0: params per gpu: 2318.53 M
0: params of model = params per GPU * mp_size: 18548.24 M
0: fwd MACs per GPU: 45999.1 GMACs
0: fwd flops per GPU: 91998.2 G
0: fwd flops of model = fwd flops per GPU * mp_size: 735985.6 G
0: fwd latency: 5.7 s
0: fwd FLOPS per GPU = fwd flops per GPU / fwd latency: 16.15 TFLOPS
0: bwd latency: 14.55 s
0: bwd FLOPS per GPU = 2 * fwd flops per GPU / bwd latency: 12.64 TFLOPS
0: fwd+bwd FLOPS per GPU = 3 * fwd flops per GPU / (fwd+bwd latency): 13.63 TFLOPS
0: step latency: 413.55 ms
0: iter latency: 20.66 s
0: FLOPS per GPU = 3 * fwd flops per GPU / iter latency: 13.36 TFLOPS
0: samples/second: 49.55
0:
Logs generated when pretrain_gpt.py is run:
iteration 50/ 50 | consumed samples: 51200 | consumed tokens: 104857600 | elapsed time per iteration (ms): 20273.0 | learning rate: 9.524E-05 | global batch size: 1024 | lm loss: 7.945235E+00 | loss scale: 4096.0 | number of skipped iterations: 0 | number of nan iterations: 0 | samples per second: 50.510 | TFLOPs: 125.18 |
time (ms) | forward-compute: 5673.73 | backward-compute: 14185.10 | backward-embedding-all-reduce: 0.01 | optimizer: 412.54 | batch-generator: 10.72
System info

OS: [Red Hat Enterprise Linux release 8.5]
GPU count and types [128 A100 GPUs]
Interconnects [ machines are connected with 100 Gbps IB]

Launcher context
Using pyxis and slurm to launch
Docker context
nvcr.io/nvidia/pytorch:22.10-py3 with pip installed deepspeed
Additional context
Running GPT with Megatron-DeepSpeed

Hi @shrutiramesh1988, the flops profiler calculates "per GPU" FLOPS whereas the log shows the "per iteration" FLOPs, the difference comes from the mp size which is 8 in your script. the difference between 13.36*8 =106.88 and 125.18 is because "125.18" uses a rough estimation (https://github.com/microsoft/Megatron-DeepSpeed/blob/main/megatron/utils.py#L222-L246).

@shrutiramesh1988 currently the flops profiler is not integrated and enabled when ds_pipeline_enabled=True. We are adding it soon. thanks.

Hello, as per the earlier intimation, we were informed that flops profiler has not yet been integrated and enabled when ds_pipeline_enabled=True. Kindly let us know if this feature has been added now.


feat(log_store): introduce the IndexCollector

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
#4385
What's changed and what's your intention?

Separate BackgroundProducerWorker into files
Introduce IndexCollector to collect WAL EntryIds

Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

@coderabbitai summary


Fix #282 Clang Global Var Uninitialized causes crash

Draft PR for visibility

Note useful to have this sit here anymore, closed PR can still contain the reproduction.


1.28 release always results in SS rank

When enabling mod in BMBF and playing any song in BS 1.28, the results screen always shows SS rank, no matter what your score is

Will this get fixed because I have the exact same problem with this mod and the issue you mentioned was closed without a fix. It would be nice if you could do something about it because I need this mod, without many maps won't work :(

Feel free to fix it then.
The issue was closed because it was a duplicate, common practice in GitHub and this is no exception.

Sry didn't know that. Do you have any idea what could cause this bug?

Forgot to mention in it, but this should be fixed in https://github.com/StackDoubleFlow/CustomJSONData/pull/37

Fixed by #37

Any way to download a qmod with the fixed code?

Issue still not fixed unfortunately.

Recently installed Chroma from: Beat Saber Mod Versions on my new Quest 3. Prior to installing, but with a modded Beat Saber, I got various ranks. With Chroma installed even very, very poor performance results in an SS.

Still having this issue. Once again asking how to get an updated version of this mod with the fix, since the latest release is so old it's before https://github.com/StackDoubleFlow/CustomJSONData/pull/37

same

Thing is 1.28 has multiplayer mods, the new versions don't, and using the 1.28 mod installer always installed the version of JSONData that doesn't have the fix.
Thanks for the link to the fixed 1.28 qmod, people can now play multiplayer without getting SS

@Fernthedev I think for Quest 1 (which I have) it's only possible to mode up to version 1.28 unfortunately..
Also https://www.questmodding.com/ mentions 1.28 as latest modable version so people with Quest 2 & 3 can stumble upon this issue as well
Thank you for providing fixed qmod file! It seems that it's intended to be installed with Scotland2, but 1.28 mods use QuestLoader, so I'm getting this error while trying to install it (repatching doesn't help since other mods use QuestLoader):

Could you please compile 0.19.4 qmod for QuestLoader?

Does this version I have work with quest loader? https://filebin.net/rybbb3l7c64ciq9g

Does this version I have work with quest loader? https://filebin.net/rybbb3l7c64ciq9g

Yes! Thank you!! The score calculates properly now. QuestPatcher says its 0.19.3, but your file is different from the one here https://github.com/StackDoubleFlow/CustomJSONData/releases/tag/v0.19.3
No idea where you got that, but thank you!


Add footer to each sub-page.

Footer is only visible on home page.
It should be available at each sub page.
On homepage footer is not properly aligned.
I also want to make small changes to footer.
Assign this to me under LGMSoc.

Okayyy go ahead and only align the footer. Let it be only on the first page.


App Crashes On using shareViaEmail

On Using shareViaEmail app crashes in one plus android  device and mi android devices

just updated the plugin version in ionic3 app, version 5.4.0 solved the issue


Type casting may cause property access error with custom resource

Tested versions
godot 4.3.beta1
System information
win11
Issue description
Direct instantiation using a preloaded scene reference works as expected. Property access is ok.
But, casting the instantiated scene to Charactor using a function (StaticClass.get_pack().instantiate() as Charactor) results in an "invalid access property" error.
Steps to reproduce
extends Node2D
func _ready() -> void:
	#var ti: Charactor # works
	#ti = StaticClass.get_pack().instantiate()

	#var ti:= StaticClass.CHARACTOR_SCENE.instantiate() as Charactor # this works

	var ti:= StaticClass.get_pack().instantiate() as Charactor # unexpected type cast behavior

	print(ti.bm.add_num) # error invalid access property

#class_name StaticClass
#
#const CHARACTOR_SCENE = preload("res://CharactorScene.tscn")
#static func get_pack() -> PackedScene:
	#return CHARACTOR_SCENE

original CHARACTOR_SCENE  properties

Minimal reproduction project (MRP)
4.3test.zip

I can't say I 100% understand what's going on here, but preloading the scene in StaticClass.gd seems to break things. It works for me after replacing StaticClass.gd with the following and possibly doing Project/Reload Current Project.
class_name StaticClass

static func get_pack() -> PackedScene:
	return load('res://CharactorScene.tscn')

Alternatively, declaring the export variable in Charactor.gd as Resource or GameModifier (a custom resource derived from Resource) also seems to work. Things break when the scene is preloaded and the variable is of type BodyModifier (derived from GameModifier).
Maybe there's an issue with preloading scenes that have export variables typed as custom resources that inherit from Resource via multiple levels (and script files)?

I did some tests, this does not happen with build-in Resources with multiple levels.


[FEATURE] [INTEGRATIONS] Add support for the saved objects assets

Is your feature request related to a problem?
With MDS being added in integrations, the assets for the integration templates would require the saved objects belonging to the particular type, for example visualizations would need to belong to that particular data-source id
What solution would you like?
The fetched saved objects to belong to the particular MDS cluster
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

[Weekly Catch All Triage - 1]


fix: docker config local and dev/prod

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Patch Program-related bugs found by the fuzzer

This PR fixes a few potential deserialization panics found via fuzzing.
~note: if the maximum number of instructions should be limited by Network::MAX_FUNCTION_INSTRUCTIONS which is u16, their serialized counts could probably just be u16 as well.~ discussed this, it's for potential forward compatibility

As a drive-by, I took care of an unrelated clippy lint that was breaking the check-clippy CI job.

LGTM, thanks @ljedrz !


Samples run fails in CI

We have samples run enabled for storage-file-share. However, after replacing the imports/requires statements to use the real package names the sample runs started failing.

[ ] issue 1: we added the prep-samples.js script that is supposed to patch the samples but it currently only works for typescript
[ ] issue 2: the execute-samples.js script try to run all files and failed at a later added README.md.

#6496 is merged!


EMV 3D Secure specifications

hello
πως θα μπορουσα να κανω το plugin ( compliant with new EMV 3D Secure specifications) eyxaristw
VASILIS APO LARISA

thanks you r the best  , inter-shop.gr  any thing u need at your service


Better explain JSON print indent parameter (3.x)

3.x version of #56930. The print method was renamed to stringify in master.

Thanks!

Cherry-picked for 3.4.3.


Add handling of additional characteristicNames that are the same constituent

Some variables have different names but represent the same constituent (e.g., "Total Phosphorus, mixed forms" and "Phosphorus"). Add characteristicNames that can be converted in the same way as an existing. May require:

re-structuring of domains.out_col_lookup()
filtering of results to limit qualifiers to those that are the same
further development of domains.accepted_methods()

TADA SubstituteDeprecatedChars leverages https://cdx.epa.gov/wqx/download/DomainValues/Characteristic.CSV, replace 'Name' with 'Comparable Name' col based on 'Domain Value Status' == 'Deprecated'. 'Group Name' may help further categorize Accepted names.

Link to TADA docs in docs for table for this.


updating DROP INDEX diagram

This change is

Updating diagram for drop-index.mdto simplify docs in this PR.


Add version bumping strategy 'commit message'

I would like to bump the major or minor by having certain keywords in my commit message.. Inital thoughts:
Breaking: Foo would cause a Major bump
Feature/New: Bar would cause minor
Other options are +major, +minor
Thoughts?

I like the +gv: breaking , pretty similar to: https://github.com/boennemann/semantic-release#how-does-this-work
That said since we're actually talking about a semver breaking change should we consider: semver: breaking change to be less gitversion specific?

+1 for the +semver:breaking option. Maybe the other option should then also be +semver:feature?

Why not make this configurable.
major-version-bump-message:
:)

Yea, +semver: feature sounds good.

Why not make this configurable.

at least as the default option :)

So what about
major-version-bump-message: \+semver: breaking
minor-version-bump-message: \+semver:(feature|minor)

+1

On 16 Feb 2015, at 20:54, Jake Ginnivan<EMAIL_ADDRESS>wrote:
So what about
major-version-bump-message: +semver: breaking
minor-version-bump-message: +semver:(feature|minor)
—
Reply to this email directly or view it on GitHub.

Updated issue description. It is up for grabs so anyone can grab it :)

Great idea!

On 17 Feb 2015, at 4:36 am, Jake Ginnivan<EMAIL_ADDRESS>wrote:
Updated issue description. It is up for grabs so anyone can grab it :)
—
Reply to this email directly or view it on GitHub.

Are there any special rules around using this? I've tried an empty commit against develop (local), a commit with a change to the readme.md in a feature branch (also local) and I still see the existing versioning scheme. I've tried with commit messages of
+semver: minor
and then tried
Version bump to 2.4
+semver: minor
Still no joy. Are there special conditions that cause this comment to be actioned?

Hmmm, I think this is my lack of understanding. I'm trying to bump minor on a develop branch after I have created a release-2.3 branch which is going through a release candidate phase.
The last known version source on the develop branch was a v2.2.9 tag. The develop builds were correctly building 2.3.0 builds based on the 2.2.9 tag. I had assumed that adding a commit message to bump minor would have taken 2.3.0 to 2.4.0. I guess it doesn't because it is bumping from the 2.2.9 tag and coming up with the 2.3.0 version again.
I can't tag 2.3.0 anywhere because it hasn't been finalised in the release-2.3 branch. I am assuming that my only option is to put a next-version: 2.4.0 into the GitVersion.yml in the develop branch to get the desired result.


docs(contributing): update commands to start storyboard

Summary
Add missing command pnpm build before running pnpm run start, otherwise we get an error when we run it for the first time
Type

Documentation

What is expected?
The missing command in the contributing file
The following changes where made:

Updated contributing.md

I merged your change with another PR as I cannot push into this one: https://github.com/scaleway/ultraviolet/pull/3609


Expand unit tests for virtcontainers/utils

There are no unit tests for proc_linux.go, utils_linux_generic.go, and utils_linux_ppc64le.go in runtime/virtcontainers/utils. Add tests to include them.
2.x issue: https://github.com/kata-containers/kata-containers/issues/700

This issue is being automatically closed as Kata Containers 1.x has now reached EOL (End of Life). This means it is no longer being maintained.

Important:
All users should switch to the latest Kata Containers 2.x release to ensure they are using a maintained release that contains the latest security fixes, performance improvements and new features.

This decision was discussed by the @kata-containers/architecture-committee and has been announced via the Kata Containers mailing list:

http://lists.katacontainers.io/pipermail/kata-dev/2020-November/001601.html
http://lists.katacontainers.io/pipermail/kata-dev/2021-April/001843.html
http://lists.katacontainers.io/pipermail/kata-dev/2021-May/001896.html

If you believe this issue still applies to Kata Containers 2.x, please open an issue against the Kata Containers 2.x repository, pointing to this one, providing details to allow us to migrate it.


Tokenizer breaks IE 8 and older - Quick fix

This plugin seems to break on IE 8 and older because Array.filter is not supported and will give you an error.
https://github.com/saabi/vminpoly/blob/master/tokenizer.js#L501
A quick way to fix this yourself is to include the following snippet:
if (!Array.prototype.filter)
{
  Array.prototype.filter = function(fun /*, thisp */)
  {
    "use strict";

    if (this === void 0 || this === null)
      throw new TypeError();

    var t = Object(this);
    var len = t.length >>> 0;
    if (typeof fun !== "function")
      throw new TypeError();

    var res = [];
    var thisp = arguments[1];
    for (var i = 0; i < len; i++)
    {
      if (i in t)
      {
        var val = t[i]; // in case fun mutates this
        if (fun.call(thisp, val, i, t))
          res.push(val);
      }
    }

    return res;
  };
}

I cannot seem to reproduce the error you are having with native IE 8.0.7601.17514 on a Windows 7 Ultimate 64 SP1 box. Just checked the demo and everything works 100% spot on here. Not checked on lower or higher versions IE but 8.0.7601.17514 works fine from looking at the demo and resizing the viewport.


Support installing a specific version of porter

What does this change
Install an older version of porter
What issue does it fix
Closes #729
Notes for the reviewer
I think I can test this change out in a single PR. 🤞
Checklist

[ ] Unit Tests
[x] Documentation

[ ] Documentation Not Impacted

/azp run porter-install-check

/azp run porter-install-check

/azp run porter-install-check


downloader: Fix typo

This fixes a typo in downloader/client/main.go

This change is

@tetsuok Thanks!


Access LFO value of Tone.AutoFilter in realtime

Looking into the returned object of Tone.AutoFilter there seems to be no way to access its LFO's value in realtime (e.g. for visualizing the filter in action)? Is there another way accessing this value?

This is not currently a feature of the LFO. The best way to do this would be to use Tone.Waveform or Tone.Meter to get this signal value.


I kindly request a

Hi there
Can I please get a compiled version of this to use in the

The bare code? compactible with plain HTML ?

Thank you so much. Sorry I should have indicate for which Vue2 version.
As soon as I learn Vue3, will use these ones. Can you please avail for Vue 2? Thanks so much!


Final touches

Updates to latest CCT version
The video window won't get shown if a call has not been started
The end call function now ends all calls in the room instead of only the first one

@eolognt No, I've written it as if(everywhere else in the app.

LGTM


Limit wiki edit permissions

Noticed some sus links in the wiki while going through the history, an IP logger and some zip files that probably got removed by GitHub. I reverted the edits that I found to whatever they used to be, they only started to pop up a couple months ago on those 2 pages.
Restricting editing to collaborators should probably be enabled so the pages cant be edited/created/deleted by anyone.

Thank you so much for alerting me of this! I just changed the edit permissions.
Sad to see they were being abuse for malware links, it makes sense but didn't think to check for this myself.
I'd like to send you a $50 bounty for discovering this. Whats your preferred method? I can send with Venmo/Zelle/ETH/BTC/XMR/USDC

I'd love it if you could donate it to Archive.org, it'd be put to much better use than in my pocket :bowtie:


IGNITE-21585 Disable catalog compaction

https://issues.apache.org/jira/browse/IGNITE-21585
Thank you for submitting the pull request.
To streamline the review process of the patch and ensure better code quality
we ask both an author and a reviewer to verify the following:
The Review Checklist

[ ] Formal criteria: TC status, codestyle, mandatory documentation. Also make sure to complete the following:
- There is a single JIRA ticket related to the pull request.
- The web-link to the pull request is attached to the JIRA ticket.
- The JIRA ticket has the Patch Available state.
- The description of the JIRA ticket explains WHAT was made, WHY and HOW.
- The pull request title is treated as the final commit message. The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] Design: new code conforms with the design principles of the components it is added to.
[ ] Patch quality: patch cannot be split into smaller pieces, its size must be reasonable.
[ ] Code quality: code is clean and readable, necessary developer documentation is added if needed.
[ ] Tests code quality: test set covers positive/negative scenarios, happy/edge cases. Tests are effective in terms of execution time and resources.

Notes

Apache Ignite Coding Guidelines

Reverted race fix in ClientPrimaryReplicaTracker, and create a ticket for that.
This race isn't critical for now.


Add initial support for java references for yaml

What's changed?

This adds initial support for generating references to java type from a Yaml fine in a similar fashion, and with similar limitations, as in:

#4697

What's your motivation?

Adding this feature allows existing recipes that support references to be applied to .properties files. For more details see:

#4681

Anything in particular you'd like reviewers to focus on?

The focus points are almost on to one with the previously referenced implementation of references to Java type in .properties files.
Anyone you would like to review specifically?
@timtebeek @Laurens-W
Any additional context

Checklist

[x] I've added unit tests to cover both positive and negative cases
[x] I've read and applied the recipe conventions and best practices
[x] I've used the IntelliJ IDEA auto-formatter on affected files

I'd applied some light polish to the parallel PR in https://github.com/openrewrite/rewrite/pull/4697/commits/713ced29e3f92f9acbe2ac32e438eae093af72f4 ; would be good to apply those same changes here as well, and resolve the conflicts.

Yes, would be good to update this PR now that the dust has settled a bit regarding the rename refactoring.


Package size limits section.

Attempt to address https://github.com/NuGet/docs.microsoft.com-nuget/issues/2890.

@JonDouglas same file, line 43 refers the site as "nuget.org", I just followed suit.


Line iterator: Improve test functions

@niklasmohrin
As you mentioned in #314, I improved test functions on LineIterator.
With your counter example code:
let mut previous = 0;
if let Err(e) = Read::bytes(&mut self.reader)
    .find(|b| {
        let res = previous == b'\n';
        previous = *b.as_ref().unwrap();
        res
    })
    .transpose()

The test function will fail:
❯ cargo test test_first_line
   Compiling tealdeer v1.6.1 (/home/potato/repos/tealdeer)
    Finished test [unoptimized + debuginfo] target(s) in 2.76s
     Running unittests src/main.rs (target/debug/deps/tldr-8d4cf65b28f436c9)

running 2 tests
test line_iterator::test::test_first_line_old_format ... ok
test line_iterator::test::test_first_line_new_format ... FAILED

failures:

---- line_iterator::test::test_first_line_new_format stdout ----
thread 'line_iterator::test::test_first_line_new_format' panicked at 'assertion failed: `(left == right)`
  left: `ExampleCode("Description")`,
 right: `Description("Description")`', src/line_iterator.rs:122:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    line_iterator::test::test_first_line_new_format

test result: FAILED. 1 passed; 1 failed; 0 ignored; 0 measured; 14 filtered out; finished in 0.00s

error: test failed, to rerun pass `--bin tldr`

Do you think this is good enough? Or you have another idea?

Yeah, I am not exactly sure what the idea between old and new format is or if there are any plans for migrating towards one or another. I think we talked about this some time ago, the info is probably buried in some comment somewhere. @dbrgn Do you remember what that was about?

Support for the new format was added here: https://github.com/dbrgn/tealdeer/pull/32
See discussion at https://github.com/tldr-pages/tldr/pull/958
From what I can tell, the new format was never accepted. So I guess we could remove it from tealdeer as well...

Well, thanks for the contribution anyways :)


[SPARK-28982][SQL] Implementation Spark's own GetTypeInfoOperation

What changes were proposed in this pull request?
Currently !typeinfo returns INTERVAL_YEAR_MONTH, INTERVAL_DAY_TIME, UNION , all of which Spark turns into string.
Make SparkGetTypeInfoOperation, to exclude types which we don't support.
In 1.2.1 Type class is org.apache.hive.service.cli.Type
In 2.3.x Type class is org.apache.hadoop.hive.serde2.thrift.Type
Use ThrifrserverShimUtils to fit version problem.
Why are the changes needed?
We should return type info of Spark's own type info
Does this PR introduce any user-facing change?
No
How was this patch tested?
Manuel test

Can one of the admins verify this patch?

Can one of the admins verify this patch?

gently ping  @wangyum @juliuszsompolski  Implement this operation.

Can one of the admins verify this patch?

https://github.com/apache/spark/blob/master/sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkExecuteStatementOperation.scala#L334
val attrTypeString = if (field.dataType == NullType) "void" else field.dataType.catalogString

can this return "null" now instead of "void"?

null
NO, my pr only cover !typeinfo, other things can't change. Since FiledSchema is hive's class.

@juliuszsompolski change to use ThriftserverShimUtils. Any more advise.
Hive 1.2.1 result :

Hive 2.3.5 result:

LGTM.
I think it's good now.
It can be thought between "NULL" and "VOID", but I think it's fine as is - NULL is a weird type, and it's name cannot be directly used as a type name in a CREATE TABLE statement anyway (CREATE TABLE foo (col NULL) would fail), so I think it doesn't matter if the name is NULL or VOID.

Yes, and in SparkExecuteStatementOperation it get NullType's attrTypeString as void too

ok to test

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/15301/
Test PASSed.

Test build #110306 has started for PR 25694 at commit 068352e.

Test build #110306 has finished for PR 25694 at commit 068352e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110306/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/15398/
Test PASSed.

Test build #110413 has started for PR 25694 at commit 3c488fc.

LGTM
Thank you @AngersZhuuuu and @wangyum for a great discussion and examples that helped me learn the interfaces of Java JDBC java.sql.ResultSetMetaData to understand which types should GetTypeInfo return!

Thank you all the same,  your deep question prompted me to dive into these questions. Also learn a lot for the JDBC process.

Test build #110413 has finished for PR 25694 at commit 3c488fc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110413/
Test PASSed.

retest this please

Test build #110414 has started for PR 25694 at commit 3c488fc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/15400/
Test PASSed.

Test build #110414 has finished for PR 25694 at commit 3c488fc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110414/
Test PASSed.

Result:
Transaction isolation: TRANSACTION_REPEATABLE_READ
Beeline version 2.3.6 by Apache Hive
0: jdbc:hive2://localhost:10000> !typeinfo
+------------+------------+------------+-----------------+-----------------+----------------+-----------+-----------------+-------------+---------------------+-------------------+-----------------+------------------+----------------+----------------+----------------+-------------------+-----------------+
| TYPE_NAME  | DATA_TYPE  | PRECISION  | LITERAL_PREFIX  | LITERAL_SUFFIX  | CREATE_PARAMS  | NULLABLE  | CASE_SENSITIVE  | SEARCHABLE  | UNSIGNED_ATTRIBUTE  | FIXED_PREC_SCALE  | AUTO_INCREMENT  | LOCAL_TYPE_NAME  | MINIMUM_SCALE  | MAXIMUM_SCALE  | SQL_DATA_TYPE  | SQL_DATETIME_SUB  | NUM_PREC_RADIX  |
+------------+------------+------------+-----------------+-----------------+----------------+-----------+-----------------+-------------+---------------------+-------------------+-----------------+------------------+----------------+----------------+----------------+-------------------+-----------------+
| VOID       | 0          | NULL       | NULL            | NULL            | NULL           | 1         | false           | 3           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
| BOOLEAN    | 16         | NULL       | NULL            | NULL            | NULL           | 1         | false           | 3           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
| STRING     | 12         | NULL       | NULL            | NULL            | NULL           | 1         | true            | 3           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
| BINARY     | -2         | NULL       | NULL            | NULL            | NULL           | 1         | false           | 3           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
| TINYINT    | -6         | 3          | NULL            | NULL            | NULL           | 1         | false           | 3           | false               | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | 10              |
| SMALLINT   | 5          | 5          | NULL            | NULL            | NULL           | 1         | false           | 3           | false               | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | 10              |
| INT        | 4          | 10         | NULL            | NULL            | NULL           | 1         | false           | 3           | false               | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | 10              |
| BIGINT     | -5         | 19         | NULL            | NULL            | NULL           | 1         | false           | 3           | false               | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | 10              |
| FLOAT      | 6          | 7          | NULL            | NULL            | NULL           | 1         | false           | 3           | false               | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | 10              |
| DOUBLE     | 8          | 15         | NULL            | NULL            | NULL           | 1         | false           | 3           | false               | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | 10              |
| DECIMAL    | 3          | 38         | NULL            | NULL            | NULL           | 1         | false           | 3           | false               | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | 10              |
| DATE       | 91         | NULL       | NULL            | NULL            | NULL           | 1         | false           | 3           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
| TIMESTAMP  | 93         | NULL       | NULL            | NULL            | NULL           | 1         | false           | 3           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
| ARRAY      | 2003       | NULL       | NULL            | NULL            | NULL           | 1         | false           | 0           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
| MAP        | 2000       | NULL       | NULL            | NULL            | NULL           | 1         | false           | 0           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
| STRUCT     | 2002       | NULL       | NULL            | NULL            | NULL           | 1         | false           | 0           | true                | false             | false           | NULL             | 0              | 0              | NULL           | NULL              | NULL            |
+------------+------------+------------+-----------------+-----------------+----------------+-----------+-----------------+-------------+---------------------+-------------------+-----------------+------------------+----------------+----------------+----------------+-------------------+-----------------+

Thank you @AngersZhuuuu and @juliuszsompolski

Merged to master


Problem with code that depends on hoisting

Please view my test-swc repository as it has the code to easily reproduce the bug.
The file directive2.ts has this pattern:
link: function (scope: ICustomScope)
        {

            scope.startOperation = startOperation;

            function startOperation(operation: string)
            {
                scope.agentOperation = operation;
            }
        }

running npx swc ./directives/directive2.ts -d ./ and examining the output, it's:
link: function(scope) {
            var startOperation1 = function startOperation1(operation) {
                scope.agentOperation = operation;
            };
            scope.startOperation = startOperation;
        }

Clearly this code won't work because function name was modified.
In swc 1.0.21 it wouldn't work too, because the output JavaScript was like:
link: function(scope) {
            scope.startOperation = startOperation;

            var startOperation = function startOperation(operation) {
                scope.agentOperation = operation;
            };
        }

JavaScript hoisting will set scope.startOperation to undefined because it's the value of the var by the time of execution.
Babel will leave the Typescript code as it's with the same order.

Dupe of https://github.com/swc-project/swc/issues/245
Fixed as of 1.0.22 or 1.0.23

@duncanbeevers I'm using 1.0.25

I'm investigating on this. It seems like a hygiene bug...


MGMT-12451: Disabled Configure Host inventory settings based on RBAC

Signed-off-by: Marek Libra<EMAIL_ADDRESS>

/retest


Frost-select handles negative selected indicies

#PATCH#
Fixed how negative indices are handled by frost-select.

👍

thanks for catching that


syncContextDidSave calling notifyAboutChangedObjectsFromSaveNotification

In https://github.com/objcio/core-data/blob/master/Moody/MoodySync/SyncContextOwner.swift#L65 , does anyone know why the notifyAboutChangedObjectsFromSaveNotification is called?
The sync context is saved and then the changes are merged into the main managed object context in the ContextOwnerType's syncContextDidSave function. It seems that since the changes are coming from the sync context, that we shouldn't have to basically call performLocalChanges in notifyAboutChangedObjectsFromSaveNotification.
Thanks for any help anyone can provide!

@bluk Do you mean line 71? I have the same question.

Removing this actually fixed something for me - I was syncing to CloudKit and if I was offline, making some changes, then went online, but it re-synced to the old data before I went offline, due to this line. I think it may be a bug.

@bluk There is a small section in the book explaining how to handle ordering between changeProcessors. So, basically if one of your changeProcessors needs to process objects only after they have been processed by another changeProcessor, you can adjust the predicate so that the objects will satisfy the predicate only after being processed by the first changeProcessor (for example, if you changeProcessor needs to work only with the uploaded objects, your predicate should contain remoteID != NULL). Once the first changeProcessor processes the objects, it will then set some of the fields of these objects on the syncContext, and because of the performLocalChanges these changed objects will be picked and processed by the second changeProcessor immediately, and not after another trigger of the synchronization.


Kong 2.0 images not allowing to write to /usr/local/lib/luarocks/rocks-5.1

When trying to build with the latest versions of docker-kong, I get an error because the kong user can no longer write to the location specified in the error output. Below you'll see my Dockerfile and the error output from the docker build command. I have tried the tags latest 2.0.0 2.0.0-alpine and all give me the same error listed below.
Dockerfile
FROM kong:latest

RUN luarocks install lua-resty-openidc
RUN luarocks install kong-oidc

COPY kong.conf /etc/kong/kong.conf

Error
Error: Your user does not have write permissions in /usr/local/lib/luarocks/rocks-5.1
-- you may want to run as a privileged user or use your local tree with --local.

Using version 1.5 does not cause this error, and we will probably use that in the interim until this can be addressed.
Thank you for your time.

Whoops I meant to include that in the CHANGELOG / release notes. I'll make a note to document this somewhere official:
Going forward we're running Kong as the least privileged user possible so you'll need a USER root entry in your Dockerfile
FROM kong:latest

USER root

RUN luarocks install lua-resty-openidc
RUN luarocks install kong-oidc

USER kong

Thanks @hutchic


ppo_cnn/MiniWorldEnv

add ppo_cnn, which is modified based on original \ppo file;
add MiniWorld wrapper
add "--is_matrix" in argument list, and several type error are fixed (e.g. clip_param, type=int->float)
add results of ppo on Miniworld

Well done, thanks~
Yutong


Add the STRONG_TYPE_MODULE_EXPORT to all entities that should be exported, aka strong_type module part 1

Add a macro STRONG_TYPE_MODULE_EXPORT.
This at some point will expand to export in a module build and empty in a non module build.
Only entities in the strong_type namespace were decorated, anything in "impl", "detail" etc... namespaces were not, although see below.
The header file type.hpp contains
#if !defined(STRONG_TYPE_MODULE_EXPORT)
#define STRONG_TYPE_MODULE_EXPORT
#endif

This allows existing include users to carry on including without any changes
There should be no other changed in this patch. In my local build I have built a module and built all the test cases against it, at least on MSVC. In doing this there were two methods called access in the type.hpp header that are used in the implementation of other exported stuff so they had to be exported.
I included the strong_type.ixx file just for you interest, it will probably change a bit from this. I believe that all the other changes in the library header files are messing around with #include stuff by macroing them in or out as necessary.
These changes should not overlap at all with the import std; pull request so could be applied in either order.

Are you waiting on me to do anything on this and the other PR?

No, sorry, I got stuck on a ton of other things. It is not impossible that I will ask for your assistance, but for the moment I'm a bit swamped with other things.

No problem I'll just check back every so often


Bring in k8s work

From https://github.com/apache-spark-on-k8s/spark

First version brought in at https://github.com/palantir/spark/pull/104


Documentation doesn't match links and SDK installation paths

On the fifth step it can be read:

Visit https://developers.google.com/appengine/downloads, copy URL of "Linux/Other Platforms" zip file for current AppEngine SDK. Do this regardless of whether you are on Linux or OS X.

The link captioned as "Linux/Other Platforms" is not there anymore the main CTA inside that page actually points to the Google Cloud SDK which can be confusing.
In addition, installing the Google App Engine SDK from the old .dmg places its main folder in /usr/local and running
bash util.sh -d

fails since the script is looking for the development server in /Users/<USER_NAME>/bin/google_appengine/dev_appserver.py.

Thanks for the report - I was just in the middle of revamping some of the documentation because we recently added Soy support for Python, so I'll be sure to test all the workflows and make sure we are up-to-date.

Thanks a lot for the quick response @pames, looking forward to hearing about this soon. Let me know if you need help testing the docs changing.

FYI : You may want to take a look at https://github.com/google/gae-secure-scaffold-python/pull/29, as that contains refreshed documentation.  Feel free to comment if anything is unclear or doesn't work on your system.  As a side effect, I removed util.sh to simplify the number of toolchains we need to keep in sync - are you able to make do w/the Gruntfile?

Hopefully addressed with #29

Hi @pames, thanks a lot for following this. I'm not assigned to the project anymore, if I have time I will take a look and come back with feedback or try to get one of my coworkers to take a look at the new documentation.
Cheers!


README.MD & index.js file enhancement

Description

This PR improves the readability and user-friendliness in README.md and index.js file.
Resolves issue #1

PR Checklist

[x] Tests pass
[x] Code follows style guidelines
[x] Documentation updated
[x] Commit messages are clear
[x] Changes are in a single commit (if possible)
[x] Branch is up to date with main

Note Reviewer
I have made changes to the documentation and index.js file to make them more beginner-friendly to developers and also easy to navigate through the files. Please let me know if there is anything to fix. Thanks in advance.

Can you please add the Hacktoberfest tag for my pull request? 🙏
On Sun, 27 Oct 2024 at 6:53 pm, Manish Raj @.***> wrote:

dce1a63
https://github.com/manishraj27/mern-project-cli/commit/dce1a63db6b6919ac011e181318a1206bef9f166
I have merged index.js file.
—
Reply to this email directly, view it on GitHub
https://github.com/manishraj27/mern-project-cli/pull/10#issuecomment-2440135601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCHZDFFRC5WUNGCQ5ANX7SLZ5UZBJAVCNFSM6AAAAABQVFJG3OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBQGEZTKNRQGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Language Settings and Edit Codes

Hello,
How and where can I edit the codes?
1 - Can I edit the code? How can I do it? For example, I want to change the "buttons button-right" value.
Example: I want to change this values (Extrude Theme): .button.pull-right { position: absolute; top: 5px; right: 5px; bottom: 5px; }
2- I want to write in other languages such as "CATEGORIES" and "GO TO" (Kurdish and Turkish). How can I do that.
Best Regards

Hello there.
I bought the FREELANCER package. I changed the server. The download links of the applications I purchased do not work. I've e-mailed 3 times and nobody answers. I've been waiting 2 weeks for an answer.
Best regards.

@bedelboseli I sent you an email with a new download link. Please keep a copy of the archive, as download links expire after a few days.

Thank you very much.

In Iraq, Turkey, Syria...  countries, PayPal is prohibited. We can't
purchase single APP.
Example, I want to purchase this PWA but , I can't. I have Mastercard, but
I haven't Paypall..
https://wpmobilepack.com/progressive-web-app-themes/vedi.html


Create class for marital status

Definition proposed by @zhengj2007 in issue #62
marital status: no definition yet (parent: IAO_0000030 information content entity)

Duplicate of issue #35


interesting to tunnelling without whatsapp

It's very interesting project, it should have alternative to make it work without whatsapp like using nats or another messaging
computer need to access website through phone ip while computer ip banned

Hello, thanks for your message.
I am not understanding you well. You mean like using a regular proxy or VPN? Or using other chat platforms like Facebook or so?

maybe like using regular proxy, but the proxy server inside the private ip pc/phone
pc [limited private ip] <---> messaging service (tunnel) <---> pc (proxy) [dynamic private ip]


not loading after starting stock screening on MAC

application not working after starting stock screening on mac

Can you try running it through terminal and post the exact error screenshot?
You can go to the directory where screenipy.run is downloaded and run following commands.
chmod +x screenipy.run
./screenipy.run

i have tried that, however its not going ahead, attached screensho

t

have tried the same, still not loading, attached screensho

t

Error printing is disabled in executables.
There must be some dependency issue while building the package.
I recommend you to clone the repo and use the python script rather than .run file as of now till we fix this.
Sorry for the inconvenience!

okay thanks

@Moxclav
Tried fixing this issue with latest release.
Please try downloading and testing this screenipy.run file on your Mac and update whether it works or not.


Version not replaced in docs

These pages show DOCS_PLACEHOLDER_VERSION:
https://chicory.dev/docs/
https://chicory.dev/docs/usage/host-functions

Right, we need to do a "proper release" of the docs and figure out how to do it following this:
https://docusaurus.io/docs/versioning

Fixed in #679 by removing references to versions in the docs, docusaurus clearly suggests that we freeze documentation at a specific version rarely.
Probably we are going to do it once for 1.0.0, but not before.


feat(slide-toggle): forward focus and blur event

This is an older draft and could resolve the focus / blur event issue for components with an underlaying input element.

It is also possible to use a normal Angular '@Output` instead.

I don't think that's the correct approach, so closing this one.


Can't load/create world

http://pastebin.com/Az56ajVV It does the same with every versions... I just want to play with Colored Lights... :( Please tell me where is the problem (and if there is a way to fix it)

This looks like a graphics issue... What graphics card do you have in the Computer you are using?  If you are using a computer that has two, make sure Minecraft (java.exe) is using the more equipped card.

How do I see which graphic I have?

you are using intel grapphics to run this mod, not going to work.

How do I change it (I have a portable computer)

@medsal15 Do you have any stickers describing another graphics card on your portable?  Some laptops have more than one graphics card... some don't... You'd have to look up the model of your computer, and find out if there is a second graphics card inside.  You might have a sticker on the surface advertising that you have another:
![sticker(http://i.imgur.com/pXN4Lqm.png)]
If you only have one GPU... and you're on a laptop... then there really isn't another option... the colored light core will struggle on the Intel Integrated Graphics drivers.

I only have one... I need a new laptop :(

Closing, due to hardware limitations.


Spyder problems on OS X.

HI, when I use Spyder IDE, I come across some problems.

Why can not choose 'Monaco' font for editor? There are only 5 fonts for choose...
When split the code with '#%%', the highlight was not continuous. It was not comfortable.

Why strange character was appeared in python console when starting it before '>>>'?

Versions and main components

Spyder Version: 3.0.0
Python Version: 3.5.2
Operating system: OS X EI Captian

What are your Qt and PyQt versions?

@ccordoba12
Qt 5.6.0
PyQt5 5.6 on Darwin

I'm sorry but we can't help you with this problem. I have no idea where it could be coming from :-/


Schema for collection: object

Refs: https://github.com/metarhia/impress/issues/1492

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/impress/commit/38bbb1177e4368637f0fb26fbfd8d278d5ce1228


Store mean for response-related metrics

With this commit we calculate and store the mean of response-related
metrics (i.e. throughput, service time and latency). This can be useful
because mean sample values tend to be normally distributed even if the
population distribution (the underlying distribution that is represented
by each individual mean) is not. This is also known as the central limit
theorem. This is useful because certain statistical tests, like the
t-test, assume normally distributed values.

Do we also want to display it in the report or is it enough to just store it in the metric store?

Do we also want to display it in the report or is it enough to just store it in the metric store?

For now it's intentionally only stored but not shown in the summary report. First, we'd make the summary report even larger and second this is primarily meant for users that want to run statistical significance tests (which I'd expect to use Elasticsearch as a metrics store already). If we decide to display it in the summary report, I suggest we do it in a follow-up PR.


Playlist Name

Does a bot need a particular playlist name to be able to DJ? Might be nice to update the readme/wiki with that detail if somebody knows it. I'm creating a Hubot (http://github.com/github/hubot) adapter using your library and it'd be awesome to be able to make the bot play music.

Looking in the code a bit more - the name I'm looking for is "default". I'll resubmit update to readme as a pull request.

Are you still creating this adapter?

Haven't played with it in a few months. Hubot was definitely better suited for a place like campfire. I can upload it if you'd like a look at it - it was working decently last time I was running it, though occasionally it'd hang for seemingly no reason and need rebooted, and the required npm version conflicted with Heroku's so I had to run it locally.


Add regexp/no-optional-assertion rule

Changed to avoid some recursive processing compared to clean-regex/no-optional-assertion.

Thank you for your review! I made changes to this PR.


fix(java): enable skipped tests for Native Image testing

Few changes to note:

Added gax-grpc to emulator module to bring in Native Image netty configurations
Ignoring tests that use Mockito for native image testing
Added resource configuration

According to the logs:
 5 user-provided feature(s)
  - com.google.cloud.nativeimage.features.clients.CloudFunctionsFeature
  - com.google.cloud.nativeimage.features.clients.CloudSqlFeature
  - com.google.cloud.nativeimage.features.clients.SpannerFeature
  - com.oracle.svm.thirdparty.gson.GsonFeature
  - org.graalvm.junit.platform.JUnitPlatformFeature

The features in Gax are not getting  picked up

Native Image Tests are now passing.

@igorbernstein2 thanks again for your suggestions! any change you could give this another review?

Oops, noticed one more thing: is there any way to remove the gax dependency?

Mostly summarizing the offline discussion: using a provided scope only allows for the configurations to be included in the test jar. As mentioned here, the native image builder expects the configurations to be embedded into the project JAR through the META-INF/native-image/native-image.properties or META-INF/native-image/reflect-config.json and since   com.google.cloud.bigtable.emulator.v2.Emulator references some netty classes, we still need these configurations in the main project JAR.


How to get product images with v3?

I'm new to GraphQL, so sorry if this question is off. It seems that BigCommerce images are found with this endpoint, which has a path parameter -> /catalog/products/{product_id}/images
I see how I would set up an endpoint in the config to get images for a single product:
 `BigCommerceProductImages: `/catalog/products/${product_id}/images`

But I can't figure out how to have a page query get images dynamically. Something like this:
export const productPageQuery = graphql`
  query ProductPage($product_id: String!) {
    allBigCommerceProductImages (product_id: $product_id) {
      nodes {
        url_thumbnail
        url_standard
      }
    }
  }
`;

The same question seems like it might apply to product variants as well.
Thanks for any help you can provide. And thanks again for your work on this repo, it works great for getting products, brands and categories (and probably other stuff I haven't tried yet).

@shanejdonnelly Hey there. I will investigate this issue and get back to you the soonest. I will provide updates on the repo soon.

@shanejdonnelly I will consider this as a new feature that I'll be working on soon. As of right now, this source plugin will only support endpoints that doesn't require dynamic data for fetching additional data - so, endpoints like  /catalog/products/{product_id}/images won't work, at least for now. But I'll change that as it seems that BigCommerce API has a lot of endpoints that require dynamic data which are quite essential.

Thanks @guy-epicdesignlabs ! Let me know if I can be of help. Once you have an one method wired up, I'm sure I could help fill out others.

Hey @shanejdonnelly ,
Not sure if this helps but there's a nice example of how this can be done in API v3 in the Gatsby + BigCommerce + Netlify CMS Starter
You can append an ?include parameter to the API path to include Product content from other models:
BigCommerceProducts: "/catalog/products?include=images",

This will include images within the product entry in Gatsby's GraphQL endpoint. You can then run a query like the following to grab the image details
query ProductQuery {
  allBigCommerceProducts {
    nodes {
      name
      sku
      images {
        url_standard
        url_thumbnail
      }
    }
  }
}

The ?include parameter takes a bunch of other handy values like 'variants', 'videos' and 'custom_fields' too.

@jmoggach Yeah, thanks for pointing that one out. I forgot to mention that. Sorry about that.


change notification only to "false"

This bot is not "chat-style" conversational interactive, but it is incorrect to set notification only to "true" since that suppresses notifications and alerts. This bot is used to communicate important news and information to users, and customers have stopped deploying it because the alerts are blocked. Also, the design of the card encourages interactivity (buttons, card updates, etc.) so why it is not strictly conversation, it is "interactive" and certainly the messages are important.
This setting should be changed to "false", and the docs should be updated to call out that if someone wants to suppress notifications, they can always set it back to "true".

I am withdrawing this PR since the right behavior is for the mobile client to allow notifications+alerts for all bot messages, regardless of the isNotificationOnly manifest setting.   The isNotificationOnly manifest setting is about allowing the user to have free-form conversation chat/input back to the app, and is not actually about notifications (despite the name).  The right solution here is to fix the Teams mobile clients.   Of course, it is still possible to change this manifest setting to workaround the current limitation, but that will create the opportunity for users to send free-form chat back to the bot, which is not what it is meant for.

closing per earlier comment


Move from winforms directory to windowsforms directory

Fixes #1384

Moving this isn't quite that simple, @Youssef1313. It requires multiple PRs:

The PR that creates the samples in the new directory. The urlFragment metadata attribute in the two readme.md files will have to be changed to be unique (windowsforms-format-utility-cs and windowsforms-format-utility-vb are good choices).
A PR that changes the links to the samples in the dotnet/docs repo. It can only be merged after PR 1 is merged.
A PR that changes the links to the samples in the dotnet/dotnet-api-docs repo. It can only be merged after PR 1 is merged.
A PR that deletes the samples from the old location. It can only be merged after PRs 1-3 have been merged.
In the meantime, I'll close this PR.

Can't the urlFragment kept as it is so it won't affect links in the docs or dotnet-api-docs repos ?


Feature Request : Global Environment Variables

Currently you can add environment variables to the service layer so that each docker container in the service will have those environment variables set.
I would like to see an option that would allow you to add global environment variables to a Rancher Environment and to a Rancher Stack. These environment variables would then be added to any service that is added to the stack or environment with the lower level taking precedent over any variables with the same name at a higher level.
The idea behind this would be to allow the infrastructure manager to set something like 'spring.active.profile=dev' on a Rancher environment for development so that the developers don't accidentally forget to add them to the service level.

Yes, this is basically the same thing as #5579.


Add ceylon.formatter to build path

This should improve the debugging experience, and makes sense in general.
However, I don’t know what this does if someone is using the IDE and has ceylon.ast but not ceylon.formatter. I suspect @sadmac7000 might have such a setup – could you perhaps test this?

I suspect @sadmac7000 might have such a setup

nope, we’re clear to merge :)


Apply material design 3 to MaterialChips & MaterialChipsGroup

https://m3.material.io/components/chips/overview
Keep in mind the following issue to see if we can fix it:
#87
#114

We are working on this ticket and it will be released soon with the upcoming version of this NuGet package.

We are excited to announce that we are working on a new generation of controls based on Material Design V3, which we will be releasing soon.
This ticket has been included/fixed in the upcoming version of this NuGet package, which will include both current and new generations of controls.


Please, add my extension to testcafe plugin section

Hello,
My name is Selma and I've developed an extension to help run and debug Testcafe tests on Visual Studio Code.
The extension is called TestLatte and this is the link to it's repository https://github.com/Selminha/testlatte
I hope it can be listed on the Plugin section of Testcafe documentation.
Please let me know if you need more information about the extension.
Best Regards.
Selma

Hello Selmina. Thank you for using TestCafe and contributing to its ecosystem. I've created a pull request to add your plugin to our README.md - #4365.


Support proxy CLI arg

Logic in index.js already accounted for a proxy argument, but the bin logic wasn't passing the CLI arg to the License constructor. I fixed this by passing parsed argv to the constructor. Went ahead and added the proxy option to the CLI help text too.
Note that I also wanted to better support writing the fetched license json to file, but doing something like:
npmo-license -e me -k invalid > license.json

would result in error messages written to license.json and nothing output in the terminal. I fixed this by using console.error() for the error message instead of using console.log(). Also fixed exit code on error.

@nexdrew awesome :shipit:, you're an owner on the npm module now.


Added a CategoryOrderAction, documentation, and access points

Pull Request Etiquette
There are several guidelines you should follow in order for your
Pull Request to be merged.

[X] I have checked the PRs for upcoming features/bug fixes.
[X] I have read the contributing guidelines.

It is sometimes better to include more changes in a single commit.
If you find yourself having an overwhelming amount of commits, you
can rebase your branch.

Template
Pull Request
This PR addresses #576, adding a new CategoryOrderAction that acts very similar to a ChannelOrderAction but within the scope of a Category instead of a Guild.
Example:
Category category;
TextChannel channel = category.getTextChannels().get(0);

category.modifyTextChannelPositions().selectPosition(channel).moveDown(3).queue();

I was unsure whether or not it would be appropriate to handle creation of a CategoryOrderAction through GuildController or through the Category itself, so:

I added methods modifyCategoryTextChannelPositions(Category) and modifyCategoryVoiceChannelPositions(Category) to GuildController.
I added shortcut implementations Category#modifyTextChannelPositions() and Category#modifyVoiceChannelPositions() which simply call the method from their corresponding Guild's GuildController.

Feedback and suggestions is (of course) welcome! 😃

Instead of modifyCategoryTextChannelPositions, let's just overload the modifyTextChannelPositions method to accept a Category


2016 MacBook Pro Touch Bar Fn modifier broken

I have my touch bar configured to "Expanded Control Strip" in System Preferences, and the Fn key set to "Show App Controls". This works fine without Karabiner running, but the Touch Bar does not change as expected when Karabiner is running; it just stays on the Expanded Control Strip.
{
    "profiles": [
        {
            "fn_function_keys": {
                "f1": "vk_consumer_brightness_down",
                "f10": "mute",
                "f11": "volume_down",
                "f12": "volume_up",
                "f2": "vk_consumer_brightness_up",
                "f3": "vk_mission_control",
                "f4": "vk_launchpad",
                "f5": "vk_consumer_illumination_down",
                "f6": "vk_consumer_illumination_up",
                "f7": "vk_consumer_previous",
                "f8": "vk_consumer_play",
                "f9": "vk_consumer_next"
            },
            "name": "Default profile",
            "selected": true,
            "simple_modifications": {
                "caps_lock": "delete_or_backspace"
            }
        }
    ]
}

The Karabiner Event Viewer correctly recognises the Fn key being pressed and released:
eventType:FlagsChanged    code:0x3f       name:fn              flags:Fn                                 misc:characters:
eventType:FlagsChanged    code:0x3f       name:fn              flags:                                   misc:characters:

EDIT: After my MacBook sleeps and wakes, the expected behaviour returns. However, the faulty behaviour reappears if i relaunch Karabiner.

having the same issue

same here

Same here. And mapping fn to fn doesn't work. It wouldn't be such a big deal if there was another way to access the control strip/ app control

After putting the macbook to sleep and waking it up again, the fn key works as intended.

Worked for me too, doesn't feel like a long time fix though. Any idea why this is the case?

Confirmed here as well 15" late 2016 MBP, Karabiner-Elements 0.92.92. MacBookPro13,3. Drove me up a wall this morning.

Some commentary in some of the clones of this thread mentions restarting Karabiner as being the source of this problem — I just want to throw out there that I see this behaviour from startup, without manually restarting Karabiner. (Also, it's consistently reproduced: it's not ‘sometimes’, the fn key never works as expected with the Touch Bar as long as Karabiner Elements is running, for me.)

After I switched left control and fn. The original fn key registers as a left control button press, but retains original functionality of the fn key. (Showing the f1, f2, f3, etc. keys)

@ELLIOTTCABLE After reading @epegzz 's comment on this I immediately closed the lid, waited for 5~ seconds and opened, then Fn key started working as expected.

@JPinlac I can reproduce that; and that's cool, but it doesn't solve the titular issue on this page (i.e. I just tested that while fn was not invoking the Control Bar as it should be, and the change didn't fix it. After using the go-to-sleep, hack, though, it works as you describe — which is pretty cool!)

Is there any permanent solution for this? I'd like to remap keys with this, but I rather not want to lock and unlock every time I restart my MacBook. Seeing that the issue seems to persist for longer than a year makes me quite hopeless.

Same issue. Karabiner Elements 11.5.4. Putting Mac to sleep doens't work either - after wake-up FN and LEFT CTRL keys remapping ain't there anymore - FN works just as FN...

Karabiner 11.5, Macbook Pro 15 2017, switching fn + left_control leads to Touch Bar not displaying functions keys when either of the two (fn || left_control) being pressed.

clearly these guys don't give a s**t about users complaints, uninstalling...

@ClintEsteMadera that's a little harsh. This is free software and we should appreciate the work of all the contributors.

I mean, to be fair …
Look, I’m a software maintainer. On the one hand, I get it — we’re humans, with lives, too.

If there’d been any attempt from the original author to write the software in a spelunkable, self-documenting, maintainable style … then I’d agree. If there’d been any attempt to hunt down and designate a replacement maintainer … then, I’d agree. Hell, if they’d bothered to reply to one of the three(!) long threads about this issue, acknowledging it and either providing a timeline for getting to it, or explicitly wontfix’ing it so we can move on with our lives … then I’d agree!
But as it stands? A ton of us are screwed, there’s no way out of the situation short of writing custom keyboard firmware, and it’s been a running issue for a really long time — all while there’s active, new-feature-implementing commits happening to the software.
Just absolutely unacceptable, even for free and open-source software.

@ELLIOTTCABLE It is a free software, we ca not have to much expectation, if we need better services we have to pay. In the meantime for me it works best with a solution found on the other issue opened here with the same problem. Now I have remap the FN +<PHONE_NUMBER>0 which acts as F1-F12 keys.

The author doesn't own a MacBook Pro w/ Touch Bar. Fix it and submit a PR or work around it by sleeping and waking your MacBook.

Nope! Still an issue!

Not stale.

nope


why the line rotate so fast?

This code was modified from other's code, I want the line to rotate around the center point. But it turned faster than I expected, and I don't know what the reason is.
    from big_ol_pile_of_manim_imports import *

    class UpdateValueTracker1(Scene):
        def construct(self):
            theta=ValueTracker(PI)
            line=Line(LEFT,RIGHT)
            line.rotate(theta.get_value(),about_point=ORIGIN)
            def updater_line(obj):
                obj.rotate(theta.get_value(),about_point=np.array([0,0,0]))
            line.add_updater(updater_line)
            self.add(line)
            self.play(theta.increment_value,PI,run_time=20)

https://youtu.be/ZnOC3_7mqEw
This is the original code
    class UpdateValueTracker1(Scene):
        def construct(self):
            theta = ValueTracker(PI/2)
            line_1= Line(ORIGIN,RIGHT*3,color=RED)
            line_2= Line(ORIGIN,RIGHT*3,color=GREEN)

            line_2.rotate(theta.get_value(),about_point=ORIGIN)

            line_2.add_updater(
                    lambda m: m.set_angle(
                                        theta.get_value()
                                    )
                )

            self.add(line_1,line_2)

            self.play(theta.increment_value,PI/2)

            self.wait()

https://youtu.be/YuZVI0kF8zA

When you use an update function you have to imagine what happen every animation frame. If you think about it, each frame is rotating the value of theta, which in fact, is increased with the animation. That's why you notice that it moves faster and faster.
In the original code you can notice that rotate is not being used in the update function, but set_angle, that is, every moment of the animation is being set the angle theta to the line (which increases in the animation). So you have to use a similar method.
I must emphasize that the set_angle method takes as reference the initial coordinate of the line, so, this method will not works to rotate it with respect to its center.
One way to solve it is to create a copy of the initial state of the line, and rotate it taking as reference that line every moment of the animation.
class RotateLine(Scene):
    def construct(self):
        theta=ValueTracker(PI)
        line=Line(LEFT,RIGHT)
        line.initial_state = line.copy()
        def updater_line(obj):
            obj.become(obj.initial_state)
            obj.rotate(theta.get_value(),about_point=np.array([0,0,0]))
        line.add_updater(updater_line)
        self.add(line)
        self.play(theta.increment_value,PI,run_time=3)
        self.wait()

@zavden
Got it. thank you very much~

Please close the issue.


Python multi-line comment strings should be treated as comments in all themes

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.78.1
OS Version: Linux Ubuntu 20.04

Steps to Reproduce:

Set the color theme to Monokai. Comment block strings should be highlighted the same as single-line comments, just like Monokai in Sublime Text, but in VSCode they are not.

Fixed by PR #182162 <-- see screenshots here too!

@VSCodeTriageBot , I just updated to version 1.78.1, the latest, and the issue is still present.

To verify:
create a python file with a doc string, e.g.
def my_function():
	'''Demonstrates triple double quotes
	docstrings and does nothing really.'''

	return None

print("Using __doc__:")
print(my_function.__doc__)

print("Using help:")
help(my_function)

Verify that Demonstrates triple double quotes docstrings and does nothing really. get the same color as a comment
That's for all built-in themes, not only Monokai.


Add licenses to qmake install clause

Per https://github.com/yuxshao/ptcollab/issues/40
I had to rename the pxtone license file because annoyingly you can't have qmake install clauses rename files on move without creating cruft in the source repo.
(Might fail build, making this PR to test)

Ran windows install locally, found that the licenses were in the right place.
Looked at the mac / linux build logs and saw that there were license files there too, so I think things should be right.


Remove check for DebugType

Roslyn compilers support Sourcelink in VS 2017 15.3
I would like this added so we can support SourceLink in the NodeTools repo.
This fixes #194

Thanks @paulvanbrenk. One more place:
https://github.com/ctaggart/SourceLink/blob/v2/SourceLink.Create.CommandLine/SourceLink.Create.CommandLine.targets#L11

I'm concerned about the lack of support for embed by Windows PDB files. If <Output ItemName="EmbeddedFiles" TaskParameter="EmbeddedFiles" /> is set, does it break the builds? If so, it needs a condition.

I'm not sure what you're asking.
All SourceLink.Embed.*.targets have an appropriate condition <BuildDependsOn Condition="$(DebugType) == 'portable' or $(DebugType) == 'embedded'">...</ >
If the compiler encounters the combination of /debug:full and /embed it will error, but that would mean you somehow bypassed the earlier check. Or am I misunderstanding?

I'm not sure what you're asking.

The targets like SourceLink.Create.GitHub  are going to output a list of files that should be embedded. Portable PDB files do indeed embed them, but I'm not sure what will happen with Windows PDB files. If it breaks the builds for them, we should add a condition, something like:
<Output ItemName="EmbeddedFiles" TaskParameter="EmbeddedFiles" Condition="$(DebugType) == 'portable' or $(DebugType) == 'embedded'" />

I'm going to merge this and release a 2.2 prelease build. We can test it out. I will not be tagging a stable 2.2 until after Visual Studio 15.3 makes it to AppVeyor, but please do use the prelease build.

This is now in NuGet Gallery.
<PackageReference Include="SourceLink.Create.CommandLine" Version="2.2.0-b491" PrivateAssets="All" />

Thanks, testing this today.


Andromeda mod compatibility

Mod Name
Andromeda
Mod Homepage
https://modrinth.com/mod/andromeda
Describe the bug
Various crashes related to mixins
Steps to reproduce

Set enableFeatureManager to false and totemSettings.enableInfiniteTotem to true in the config.

Logs
N/A
Additional context
Continuation of #127

Btw, the FireBlockMixin should also be here. It's quickFire in the config. https://mclo.gs/1AVegli


[BEAM-6179] Fixing itterable comparison

using all() to compare iterable instead of array comparison as an iterable can be tuple, generator etc.
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.


Missing Code

The final code block at the bottom of this page is unnecessary or missing the code.

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

ID: 8aed1d9c-756b-5edc-5c2e-4c25af577ffd
Version Independent ID: d6b5ce45-a2d6-279e-68fe-280c0774dbcb
Content: Private Constructors - C# Programming Guide
Content Source: docs/csharp/programming-guide/classes-and-structs/private-constructors.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@AR0106 I opened #23706 to fix this.
@gewarren Can you submit an internal bug? The build should have produced a warning for this. This may be different than the previous bug in that the file itself exists, but the snippet id is wrong.

Thanks for pointing this out @AR0106
I'll look at the PR to fix this later this morning.

@gewarren Can you submit an internal bug? The build should have produced a warning for this. This may be different than the previous bug in that the file itself exists, but the snippet id is wrong.

This is the same bug. Valid file, invalid snippet tag.


update projects to target dotnet 9

Curious what the benefit of moving to .NET 9 is, specifically. Is it the HybridWebView control in MAUI that we want to use? Or is this just on general principles of keeping up with the latest .NET version?

It's just keeping up with the latest version as it comes out, in my mind we should always be on latest unless there's a specific reason not to be.

.NET 9 is now officially released. It hasn't shown up in the Ubuntu repositories yet, but that's only a matter of time.

looks like our current test failure is running into this bug
https://github.com/dotnet/efcore/issues/35110
Sadly I'm not sure how to make a simple repro, so I'm going to wait and see if someone else comes up with something


BSTC TOKEN

Bstc
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px


Dynamic Programming Knapsack Implementation

Dynamic Programming is one of the efficient algorithms, though it does not have specific solution for every problem there are some famous problems such as Knapsack, Coin Change.

I am new to this repo and definitely this is no extremely important, but you might want to take a look at the naming convention https://github.com/raywenderlich/swift-style-guide#naming.

As @dsantosp12 pointed out, we prefer the lowerCamelCase style for naming our variables.

Okay, thanks I'll take it into account

Do I need to add readme text right now?

All our contributions consists of the code and a README file that explains what the algorithm or data structure is about.

I've just reviewed the code. Before I get into it: thanks for contributing this algorithm. I think it's a useful addition, and I'm looking forward to your README - so I might be reviewing your code prematurely, because you might have already thought about some of the points I'm about to make.
I'm sure a lot of 'how the algorithm works' will get covered in your README, but from the code itself, I find it difficult to read its intent. It might be very efficient code, but it's also important here that the algorithm be easy to understand. I'm also hoping that if the code is more easily understood, it'll be easier to write about how it works.
First, you've declared several global variables which could be clearer about their role. items, numberOfItems and values all describe a collection of items which could be put into the knapsack. Your implementation requires that items and values are the same length, requires both arrays to have a leading 0 which doesn't get used, and numberOfItems should really be calculable from the size of the collection. I would suggest creating a struct (let's call it KnapsackItem) with a value property and a weight property. This lets you represent the items as a [KnapsackItem].
Second, the global variables are declared at the top of the file but only given values at the bottom when the function gets called. I think they might work better as arguments of the knapsack function (so that it takes in information about the items, and a bag capacity, and returns an Int). dp isn't used outside the knapsack function, so should be declared inside it.
Third, the variables inside the function need to be more clearly named. Clarity is more important than brevity; a shorter variable name is easier to type but its meaning can be harder to grasp.

dp. It isn't obvious to me what 'dp' stands for.
i and j aren't just incremental counters, they're used to calculate array indices, so they need meaningful names. Perhaps replace them with itemIndex and potentialCapacity?
I think it would be helpful if you broke down the larger calculations (I'm mostly looking at what's currently line 20) into smaller calculations.
For example, the calculation of j - items[i] represents the potential capacity of the bag minus the weight of item i, or the remaining capacity in the bag after it gets added. Take that part out of its calculation, and declare it as a constant called remainingCapacity. This will make it easier to tell what's being stored in the dp array and what it's being used for.
Better still, calculate remainingCapacity before the if-statement, and you can rewrite the if-statement as if remainingCapacity < 0. This shows the intent more clearly than if j < items[i].

Last one for now, and just a small observation: the first for-loop counts from 0 to numberOfItems, the second for-loop counts from 0 to capacityOfBag. But if either value is 0, the loop immediately proceeds to continue. It would be clearer if both loops started counting from 1, and there was no test for 0.
I hope that all makes sense. Do speak up if something's unclear to you, or if I've misunderstood something. I'm looking forward to seeing what you do with this. Thanks again for your work!

Thanks, narrativium! I will consider all your suggestions. This is my first contribution so far, so basically I am new in this field. I think your suggestions are very useful and will really make the algo clear and understandable.


openiscsi: 2.1.9 -> 2.1.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/open-iscsi/open-iscsi/releases.
meta.description for openiscsi is: A high performance, transport independent, multi-platform implementation of RFC3720
meta.homepage for openiscsi is: https://www.open-iscsi.com
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.1.10 with grep in /nix/store/ff5gc9dh7msharkgw5403ikwvlhnnf57-open-iscsi-2.1.10
found 2.1.10 in filename of file in /nix/store/ff5gc9dh7msharkgw5403ikwvlhnnf57-open-iscsi-2.1.10

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

openiscsi

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/9067316ffea24dabed703c85d15cfcc0456ebc15#openiscsi

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ff5gc9dh7msharkgw5403ikwvlhnnf57-open-iscsi-2.1.10
ls -la /nix/store/ff5gc9dh7msharkgw5403ikwvlhnnf57-open-iscsi-2.1.10/bin

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

1 package built:

openiscsi

Maintainer pings
cc @cleverca22 @zaninime for testing.

@ofborg build openiscsi

aarch64 nixos test also times out on hydra https://hydra.nixos.org/job/nixos/trunk-combined/nixos.tests.iscsi-root.aarch64-linux/all