Only resolve links when needed

This PR fixes #558 by adding the _links section from HAL responses to the domain models, and no longer resolving those links automatically.
Instead you can specify which links you want to resolve whenever you fetch an object from a dataservice. This will ensure we only resolve what's needed, and it will also integrate well with projections, as it will allow us to determine which projection to use based on the links that need to be resolved in a component.
Since the regular domain models will no longer contain resolved links, they are already "normalized" so this means we no longer have a need for separate normalized models.
Due to inconsistent implementations in a few places, it is likely that this PR when finished won't be able to remove all normalized models, but leave a few deprecated ones to cover cases where an bigger rewrite would be necessary to work in the new way.
This PR is currently a draft. It is nearly done, the work left to do is mainly adding an fixing a few more tests, and documentation, and merging with the latest master.

@tdonohue Thanks for the thorough code review!

In testing this PR, I noticed that yarn start (AoT compilation) currently fails cause of an error in thumbnail.component.html. It's still calling errorHandler() with an event param, but you removed that param. See this line of code: https://github.com/DSpace/dspace-angular/blob/master/src/app/thumbnail/thumbnail.component.html#L2 After fixing that single line, yarn start works great.

Yeah, that was a change I made as I wrote the tests for it and realized it wasn't used anymore. I neglected to re-test the AoT build afterwards, because I didn't think I did anything that could impact it. Thanks for picking that up!

I tested out the Angular UI with this PR installed. I didn't notice any problems. But, I did notice that every page seems to load a bit quicker than before!

Also note that, because I had to refactor the entire codebase, and didn't have the time to look at each case in depth, I erred on the side of adding more followLinks. For each one that we can remove without breaking anything, the performance should improve.
At least with this PR we have a convenient way of doing that.
Now that the domain models have all the links, requests that were only made to retrieve that info can also be left out. One such example is the clearRelatedCache method in RelationshipService. I'm sure there are more places where this kind of optimization can happen.

Finally, this is a sidenote to this PR...but, I'm noticing more and more that different PRs seem to be reordering (resorting) our import statements in various typescript files. This PR does a fair amount of that itself in any class you touch (likely automated by your IDE's settings). In a future PR, I wonder if we need to look at finding a way to force everyone to use a specific ordering for these imports (maybe via tslint, if that's possible)? It'd definitely make future PRs easier to review if there were less moving around of imports

Yes that's due to the IntelliJ's optimize imports feature, it ensures no unused classes are being imported, and it will sort imports first by module, and alphabetically within a module. It is triggered automatically every time we tell IJ to add a new import.
IJ allows you to disable both sort options independently. So we could:

Leave it as is and ask everybody to conform to these rules
Sort alphabetically, but not by module and ask everybody to conform to these rules.
Sort by module but not alphabetically, but that doesn't make a whole lot of sense
Turn off sorting imports altogether.

I've briefly looked at the editorconfig spec and it doesn't look like you can enforce import ordering using it

Merging, as this is at +2.  Thanks @artlowel (and team) for these great improvements!


Included react-native-rename command to rename template files/folders to app name

This automatically renames template files and folders to the appName argument.

Thank you.


Remove unused IE conditionals

Since Shopify only supports IE8+ these aren't (and shouldn't) being
used.

Can't argue with that, thanks for the update. One day we can remove the other ones too :)


Useless break statement

https://github.com/google/goexpect/blob/c416f18ae5af08f6c8e6b116ab427441cc67a593/expect.go#L1262-L1264
The switch case block does not require break. If break for loop, you need a label.

https://github.com/google/goexpect/pull/67


TestOutcome.Exception makes run in isolation problematic

As final test fail exception type could belong to any external dependency there are issues on isolated test runs on run outcome deserialization due to unresolved dependencies

done in 767a1c4


[7/n][eas-cli] Improve eas init command

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
This PR removes the last dependence on the owner field. It can still be present and is used to bypass one prompt in this command, but it is not required.
How

Add ability to specify id flag to command. This will probably be the instructions on the website rather than setting the owner field.
Add wizard flow to command when id flag is not specified.
Add full test coverage.

Test Plan
Run the test.
Manual test:
$ ~/expo/eas-cli/packages/eas-cli/bin/run init --id=1234
✔ Linked project to ID: 1234

$ ~/expo/eas-cli/packages/eas-cli/bin/run init --id=12345
✔ Project is already linked to  a different ID: 1234. Do you wish to overwrite it? … no
Aborting

$ ~/expo/eas-cli/packages/eas-cli/bin/run init --id=12345
✔ Project is already linked to  a different ID: 1234. Do you wish to overwrite it? … yes
✔ Linked project to ID: 12345

$ ~/expo/eas-cli/packages/eas-cli/bin/run init --id=123456 --force
✔ Linked project to ID: 123456

$ ~/expo/eas-cli/packages/eas-cli/bin/run init --id=1234567 --non-interactive
    Error: Project is already linked to a different ID: 123456. Use --force flag to overwrite.

$ git checkout -- .

$ ~/expo/eas-cli/packages/eas-cli/bin/run init
✔ Which account would you like to link this project to? › wschurman141
✔ Existing EAS project found for @wschurman141/turtlewat (id = 1a7e84d3-978d-476d-97cb-08a992acd17e). Link this project? … yes
✔ Linked project to ID: 1a7e84d3-978d-476d-97cb-08a992acd17e

$ git checkout -- .

$ ~/expo/eas-cli/packages/eas-cli/bin/run init
✔ Which account would you like to link this project to? › wschurman141
✔ Existing EAS project found for @wschurman141/turtlewat (id = 1a7e84d3-978d-476d-97cb-08a992acd17e). Link this project? … no
    Error: EAS project ID configuration canceled. Re-run the command to select a different account/project.

$ ~/expo/eas-cli/packages/eas-cli/bin/run init
✔ Which account would you like to link this project to? › real-org-will
    Error: You don't have permission to create a new project on the real-org-will account and no matching project already exists on the account.

$ ~/expo/eas-cli/packages/eas-cli/bin/run init
✔ Which account would you like to link this project to? › wschurman141
✔ Existing EAS project found for @wschurman141/turtlewat (id = 1a7e84d3-978d-476d-97cb-08a992acd17e). Link this project? … no
    Error: EAS project ID configuration canceled. Re-run the command to select a different account/project.

$ ~/expo/eas-cli/packages/eas-cli/bin/run init
✔ Which account would you like to link this project to? › wschurman141
✔ Existing EAS project found for @wschurman141/turtlewat (id = 1a7e84d3-978d-476d-97cb-08a992acd17e). Link this project? … yes
✔ Linked project to ID: 1a7e84d3-978d-476d-97cb-08a992acd17e

$ ~/expo/eas-cli/packages/eas-cli/bin/run init
✔ Project is already linked to ID: 1a7e84d3-978d-476d-97cb-08a992acd17e. To re-configure, remove extra.eas.projectId from your app config.

/changelog-entry new-feature Improve eas init command


MB.py 4.0.0 wishlist

[x] Userlist uses actual user objects
[x] Replace bot checking with a separate API
[x] Make the post condition actually work (shared with the packet condition by mistake)
[x] Remove the bot object on ctx,  etc

[x] Replace it with the bot checking API

[x] rename stuff like Client.sendPacket to Client.send_packet
[x] change when __raw__ is called, and replace its current use with raw_message

make ctx.post return a post object and not whatever the heck it is right now, it's confusing

its ctx.message

[ ] New post attachment support?


Conditional check with Boolean() transformation does not output type unlike double negation

Bug Report
Boolean output type, Boolean() infer
🔎 Search Terms
Boolean infer, Boolean condition
🕗 Version & Regression Information
Tested on latest version 4.6.2, but has the same behavior even back to 3.5.1.
Nightly version (4.7.0-dev.20220505) has the same behaviour.
⏯ Playground Link
Playground link with relevant code
💻 Code
const requiredArgFn = (value: string) => value

const optionalArgFn = (value?: string) => {
  if (Boolean(value)) {
    requiredArgFn(value) // Error: Type 'undefined' is not assignable to type 'string'.(2345)
  }

  if (!!value) {
    requiredArgFn(value) // No error, type of value is 'string'
  }
}

🙁 Actual behavior
Conditional checks with Boolean(value) do not cause the value type to be non-nullable and non-false, as a double negation does (!!value).
🙂 Expected behavior
Conditional checks with Boolean(value) should  cause the value type to be non-nullable and non-false, as a double negation does (!!value).
Since the result of calling Boolean(someValue) is always equal to the result of calling !!someValue.

another duplicate of #16655 (see #46952, #45601, #43546, #42572, etc)

Thanks, @jcalz!
I close issue then.


SAK-45011 Samigo > Minimum Point Value > Points Not Awarded

@bgarciaentornos the build here is failing. It's likely unrelated to your change, but can you rebase on master and update?

@bjones86 Yes, I noticed it but I wasn't sure what was happening. I've rebased and pushed it, let's see how it goes.

It worked now but I've got the same issue today on a different PR (https://github.com/sakaiproject/sakai/pull/9653), this is what it says:
Error:  Failed to execute goal on project api: Could not resolve dependencies for project org.sakaiproject:api:war:22-SNAPSHOT: Failed to collect dependencies at io.springfox:springfox-swagger2:jar:3.0.0 -> org.mapstruct:mapstruct:jar:1.3.1.Final: Failed to read artifact descriptor for org.mapstruct:mapstruct:jar:1.3.1.Final: Could not transfer artifact org.jboss.shrinkwrap:shrinkwrap-bom:pom:1.0.1 from/to central (https://repo.maven.apache.org/maven2): transfer failed for https://repo.maven.apache.org/maven2/org/jboss/shrinkwrap/shrinkwrap-bom/1.0.1/shrinkwrap-bom-1.0.1.pom: Connection timed out (Read failed)


unique_ptr + classes

Hello,

[x] unique_ptr!A.get() with A being a class, returns a A*. I had expected it to return a A. Same with getAtomic(), and same with shared_ptr!A and weak_ptr!A from a cursory reading. Not sure about opCast. If the RC pointer was a class instance in those cases, that also helps with inference. (currently ptr!class_instance is not @safe, the recent cast breaks inference)

[x] I think basic_string's append_/set_/this(immutable(T)[] text)/opAssign/opOpAssign are really meant to take a const(char)[] as input, since that data will be copied internally.

[x] I would prefer to just import numem;
Else it easily becomes:

import numem;
import numem.ptr;
import numem.string;

which I believe will be tiring pretty soon as unique_ptr will force user code to do at least two import quickly.

[ ] A pretty big one: shared_ptr can go in a vector but not unique_ptr (disabled postblit).
It's a bit similar to C++98, where std::auto_ptr coudln't go into a std::vector but in C++11 std::unique_ptr could with mov semantics. Cope-out could perhaps be for vector to find a way to detect if some type is moveable or not.

I've made so that unique, shared and weak ptr returns T instead of T* for classes,
Basic string now takes const(char)[] for most things, with the exception of casting to D strings and using toDString.
I've added numem.all as a shorthand to import all of numem, mainly to make it easier to opt in to importing everything for compilation speed's sake.

Thanks!!


ERROR: root - The build is already running

Hi All,
Subject
For 2 days I've been seeing in the oss.log log file a lot of error messages like this.
I haven't changed anything in the configuration/scheduling but from the error messages is quite hard to deduct what index is causing the problem.
Version
OpenSearchServer v1.5.2 - build 67d80c9c9d
Configuration
Debian GNU/Linux 7.4
2 vCPU, 8 GB RAM
JAVA_OPTS=-Xms2G -Xmx2G
40 indexes (20 write, 20 read)
Error message
00:00:06,308 ERROR: root - The build is already running
com.jaeksoft.searchlib.SearchLibException: The build is already running
        at com.jaeksoft.searchlib.autocompletion.AutoCompletionItem.checkIfRunning(AutoCompletionItem.java:271)
        at com.jaeksoft.searchlib.autocompletion.AutoCompletionItem.build(AutoCompletionItem.java:278)
        at com.jaeksoft.searchlib.scheduler.task.TaskBuildAutocompletion.execute(TaskBuildAutocompletion.java:109)
        at com.jaeksoft.searchlib.scheduler.TaskItem.run(TaskItem.java:102)
        at com.jaeksoft.searchlib.scheduler.JobItem.run(JobItem.java:230)
        at com.jaeksoft.searchlib.scheduler.TaskManager.executeJob(TaskManager.java:194)
        at com.jaeksoft.searchlib.crawler.web.process.WebCrawlMaster.runner(WebCrawlMaster.java:163)
        at com.jaeksoft.searchlib.process.ThreadAbstract.run(ThreadAbstract.java:263)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
        at java.lang.Thread.run(Unknown Source)
00:00:09,476 ERROR: root - The build is already running
com.jaeksoft.searchlib.SearchLibException: The build is already running
        at com.jaeksoft.searchlib.autocompletion.AutoCompletionItem.checkIfRunning(AutoCompletionItem.java:271)
        at com.jaeksoft.searchlib.autocompletion.AutoCompletionItem.build(AutoCompletionItem.java:278)
        at com.jaeksoft.searchlib.scheduler.task.TaskBuildAutocompletion.execute(TaskBuildAutocompletion.java:109)
        at com.jaeksoft.searchlib.scheduler.TaskItem.run(TaskItem.java:102)
        at com.jaeksoft.searchlib.scheduler.JobItem.run(JobItem.java:230)
        at com.jaeksoft.searchlib.scheduler.TaskManager.executeJob(TaskManager.java:194)
        at com.jaeksoft.searchlib.crawler.web.process.WebCrawlMaster.runner(WebCrawlMaster.java:163)
        at com.jaeksoft.searchlib.process.ThreadAbstract.run(ThreadAbstract.java:263)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
        at java.lang.Thread.run(Unknown Source)

Any clue what could be the cause?
Regards,
Constantin

Hi,
In the default configuration, each time a web session stops, OpenSearchServer runs the "BuildAutocompletion" scheduler which build a new autocompletion index.
In your log I see that it happens every 3 seconds. May be it happens because you already crawled all the URLs, and the crawl session restarts every 3 seconds because there is nothing new to crawl. But the autocompletion build take probably more than 3 seconds.

Hi,
In the default configuration, each time a web session stops, OpenSearchServer runs the "BuildAutocompletion" scheduler which build a new autocompletion index.
In your log I see that it happens every 3 seconds. May be it happens because you already crawled all the URLs, and the crawl session restarts every 3 seconds because there is nothing new to crawl. But the autocompletion build take probably more than 3 seconds.

Hello,
I have the same messages as Constantine.
On the other hand, I have 46589 web links to crawl some of which have hundreds of links. The crawl has only covered 9523 links taking into account only one link per url.
I have done this several times with the same result.
What does it take to resume the web crawl effectively?
OpenSearchServer v1.5.14 - build d0d167ef10
Debian GNU / Linux 9.7   fr_utf8
8 CPU, 8 GB RAM
JAVA_OPTS = -d64 -Xms5G -Xmx5G
openjdk version "1.8.0_212"
Thank you for your reply.
Cordially.


Improper conversion of IPV6 Addresses?

Possible improper conversion of IPV6 addresses in IPv6Header::DestinationAddress::get/set.

I don't believe so, no.


Current Browser version problem, when current browser version is less than required one.

Message: session not created: This version of ChromeDriver only supports Chrome version 90 Current browser version is 109.0.5414.74 with binary path /usr/bin/google-chrome Stacktrace: #0 0x556d332d0e89
Message: session not created: This version of ChromeDriver only supports Chrome version 90
Current browser version is 109.0.5414.74 with binary path /usr/bin/google-chrome
Stacktrace:
#0 0x556d332d0e89

by running following command, this problem will be fixed"
./manage.py pdfmaker_install_chromedriver


Use paths and schemas for endpoint migrations to prevent circular imports on side effect checks

Let's say you have your api migrations, routes, and business logic.

Routes depend on business logic because they need to call it
API migrations depend on routes because

We need to define endpoint migrations and we use route functions for that
We need to define data migrations for responses and we use route functions for that

Business logic depends on API migrations whenever business logic needs to use a migration with side effects. Obviously, you could just import it within a function at runtime to make the dependency softer but it would still be an ugly hack

As you see, we have the following picture:

Which is a cyclic dependency that is hard to resolve. We can solve (i) by specifying routes using paths instead of functions and we can solve (ii) by specifying response schemas in data migrations instead of functions. This will remove both cyclic dependencies and will have a lot of small nice side effects.

Completed with #18


feat(vitest): Add typeahead search (alternative approach with prompt autocomplete mode)

Description
Alternative approach of https://github.com/vitest-dev/vitest/pull/4733 based on prompts library's autocomplete.
Technically this "autocomplete" mode is not designed to implement filter to match multiple things (since this is not really "completing"), but it looks possible to make it usable with a minor fix/patch of internal. They also have autocompleteMultiset mode but this UI looks clumsy, so customizing on "autocomplete" seems simpler.
Here is a quick demo:
reveal screencast
pnpm -C test/core test

스크린캐스트 2023-12-21 09-35-13.webm

Clearly customizing/patching prompts has some limit as well, so in terms of maintenance burden, it might be better to roll out own solution like in https://github.com/vitest-dev/vitest/pull/4733 (or look for an alternative library though I'm not aware of such).
I would love to know what you think. Thanks!

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[x] Ideally, include a test that fails without this PR but passes with it.
[x] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[x] Run the tests with pnpm test:ci.

Documentation

[ ] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[x] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

I'm closing this PR in favor of https://github.com/vitest-dev/vitest/pull/4733
I think Vitest would be better off with home-grown solution to achieve something similar to Jest, especially this feature:
https://github.com/vitest-dev/vitest/pull/4733#discussion_r1437559357

But it looks like you can't enter a pattern that matches multiple files or a single file like you can in Jest (if arrow keys are not used, it will use pattern in the input, but if something is chosen, it will run only selected file)

I learned a lot while experimenting with prompts https://github.com/terkelg/prompts and also looking through other similar packages (https://github.com/SBoudrias/Inquirer.js, https://github.com/natemoo-re/clack) and it doesn't look like making own stuff is so complicated (if it's just the one specialized for autocomplete), so I'll try to help https://github.com/vitest-dev/vitest/pull/4733 to finish the line from this experience.


Request Entity Too Large error

I am making this request:
var filepath = path.join(__dirname, "/output/" + job.jobId + ".zip");

var data = {
    from: 'Name<EMAIL_ADDRESS>    to: email,
    subject: 'Test',
    text: 'Test',
    attachment: filepath
};

mailgun_client.messages().send(data, function(error, body) {
    if(error) console.log(error)
    console.log(body);
});

and am getting this error:
{ [Error:
<title>413 Request Entity Too Large</title>

413 Request Entity Too Large

nginx/1.7.9

] statusCode: 413 }

How do I fix this? Is it because the attachment is too big?

Seems likely. How big is the attachment?
https://documentation.mailgun.com/user_manual.html#sending-messages
Note: Mailgun supports maximum messages size of 25MB.


SOAP attachments

Is your feature request related to a problem? Please describe.
I'm looking for a SOAP library that supports SOAP Attachments (https://www.w3.org/TR/SOAP-attachments/) for sending and receiving.
I was wondering if this generator was able to handle those attachments.
Describe the solution you'd like
If it can handle it: Add some documentation on how to handle attachments.
If it can't: Add ability to add Attachments.

Hi, I won't be able to work on this feature, sorry


StdlibUnittest: disable FP80 path on android

android x64 does not support FP80

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test and merge

CC: @vgorloff


YCSB command for running the load test for Cassandra 3.1

Hello All,
I tried to run the load test for Cassandra 3.11. I am unable to get through.  Can you provide me a working sample command. Thanks in advance.
./ycsb load cassandra -p hosts="172.x.x.x" -threads 128 -p fieldcount=20 -p operationcount=900000 -p recordcount=15000000 -p requestdistribution=zipfian -P workloads/workloada -s > workloada_load_res.txt

What error message are you getting?

This one works for me:
./bin/ycsb load cassandra-cql -s -P workloads/workloada -threads 100 -p recordcount=3000000000000000 -p hosts=************* 2>&1 | tee outputLoad.txt

closing as non-responsive. if you're still having issues please reopen.


Masterless K3s - server with only control plane

Is your feature request related to a problem? Please describe.
I want to leverage an external HA platform to do HA of the kubernetes control plane. As this can float between different nodes, I do not wish to register those nodes when starting the control plane.
I have 2 nodes A&B
K3s floats between nodes A&B with control plane componentry but does not run kubelet or kube-proxy.
Nodes A & B both separately run k3s agent connected to the floating control plane
Describe the solution you'd like
Due to https://github.com/rancher/k3s/pull/1695 making kube-proxy optional, I think I would only need kubelet.exe as optional as well.
Not sure is anything else needed?
Describe alternatives you've considered
Traditional model of kubernetes HA. ETCD on 3+nodes & control plane HA - does not support 2 node well
Additional context
There exists HA platforms that allow you to use 2 nodes plus an external witness. I would like to be able to leverage these.

There is a hidden --disable-agent option to k3s server that you might try, if you haven't yet.
https://github.com/rancher/k3s/blob/master/pkg/cli/cmds/server.go#L287

There is a hidden --disable-agent option to k3s server that you might try, if you haven't yet.
https://github.com/rancher/k3s/blob/master/pkg/cli/cmds/server.go#L287

Is this deprecated and going to get removed, or just hidden?

It's not described as deprecated like several other options are, so I would say just hidden.

Note that k8s does not have any sort of quorum requirement, just etcd. You could just run a 3 node etcd cluster and point however many k3s nodes you have at that.


refactor: Set the value of config as zero directly instead of packing allowed and expiry in the acceptSwap(). #184

closes #184

Hey @0xneves ready for review


New version: JFrog.Conan 1.57.0

[ ] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

Close with reason: Stale pull request;


Should manage cloned interfaces in rc.conf on FreeBSD

Without this, addressing does not persist a reboot, and will rely on Puppet running to bring the interfaces back into the desired state.  We simply need to figure out what interfaces we are managing, and add them to rc.conf.

This has been addressed in 2.1.0.


Setting checkboxes for object boolean values has no focus or active style

The checkboxes for this new settings UI should have proper accessibility indicators such as focus (border) and active (background color change when mousedown) styles.

This issue seems to occur for the Workspace Trust checkbox too.
There is a focus indicator if one carefully uses keyboard nav to get the focus onto the checkbox, but there's no hover or active indicators.


fixing #2102 Netty reports LEAK while using reactor (Rsocket) with Sleuth

fixing #2102 Netty reports LEAK while using reactor (Rsocket) with Sleuth

Awesome!


Adding a new pointer doesn't trigger reevaluation of schema expressions that could reference it

If we do something like
create type T;
create type P { create multi link l -> T; };
insert T;
insert P { l := T }; insert P { l := T };

create function cnt_l() -> int64 using (select count(T.<l));
select cnt_l();

create type Q { create multi link l -> T; };
insert Q { l := T };
select cnt_l();

we get back 2, as expected, from the first call, and then 2 again from the second call.
This is because cnt_l doesn't get updated when a new link named l gets created.
(Note that banning use of backlinks without a type intersection does not help us here, unless we are willing to go as far as banning things like T.l[is Object] (or any other type that doesn't directly have the link)).
This example uses functions but I think there are other potential snags---maybe we could fail to trigger a cycle error in an access policy?
--
As far as backportable ways to fix this, we might be stuck finding all existing links with the same name and doing a _propagate_expr_refs on the expr references to all of them. Ew!
For future versions we can probably reduce the pain a bit and track specifically which what backreferences are used by expression, by name. (We can't do that in 2.0 because it would require adding more fields to the schema)

There is another issue related to this, which is that if a new required pointer is created, or a pointer is made required, that could invalidate existing rewrites/triggers/defaults that use it


Fixes two broken tests, issue #142, and enables tests now they're fixed

The tests are pretty descriptive:

if we're fixing column  values (by applying consistent capitalization, stripping spaces, etc) that should happen before we see if the value is in an enumerated list of possible valeus
The enumerated list of possible values should be a list!  otherwise we can wrap it in one and assume the programmer meant for it to be a single value.

Fixes #142

@YuliaS , I incorporated all of your tests except for the ones that try to apply a "fix_value_fn" that only works before the Column casts to a type.
To address those, I instead added some documentation that explains that fix_value_fn applies after casting to the declared type, so 'upper' and 'strip' won't work on an IntColumn or DateColumn, but contrariwise, 'abs' will work on an IntColumn because it's already cast to an int.


Switch from Godep to GO15VENDOREXPERIMENT

Move all dependencies from Godeps/_workspace/src to vendor/ directory.
Pros:

No more path rewriting
Standard Go toolchain

Cons:

Jumped through hoops to get go test and fmt working again
Tooling to manage dependencies still unclear

/cc @jingweno

AFAIK @freeformz is making Godep to work with 1.5's GO15VENDOREXPERIMENT.

godep works" with the GO15VENDOREXPERIMENT and writes stuff to vendor/ instead of the workspace, although there are some rough edges (things still need to be in the $GOPATH first to be saved). But over the next month or so most of those should be worked out.
PS: You should still use a tool to manage the deps IMO regardless of vendor/. I suggest either godep || gb || govendor.

See also https://github.com/tools/godep#go-15-vendor-experiment

Thanks for the tips!


fix cpuinfo link issue

upgrade to git_tag: ef634603954d88d2643d5809011288b890ac126e
fix link issue: undefined reference to `cpuinfo_has_x86_avxvnni'

can we remove this branch as it was already merged.


Disable Laravel exception handling

Laravel v7.10.3
I was trying to $i->disableExceptionHandling() and noticed that it wasn't actually getting taken care of. I would gladly PR this as it seems like a simple change:
https://github.com/Codeception/module-laravel5/blob/bd2c604e8aa02d2b24737de01c6716473e8db96c/src/Codeception/Lib/Connector/Laravel5/ExceptionHandlerDecorator.php#L74
Any objections to removing the 2nd condition from the if statement? From a "disable exception handling" standpoint I don't care if it's a laravel or symfony exception, I just want it to get re-thrown.
Any objections to this change?
It looks like @janhenkgerritsen is familiar with these issues in some capacity, so I am tagging for visibility!

hey @FatBoyXPC, version 2.0 of this module is now available, and more importantly, we now have an updated test project to work with!
The disableExceptionHandling method is one of the few that could not be tested, if you know how to fix it please feel free to send a PR.
I would suggest you take a look at the new contribution guide if that's the case.
As long as the tests pass and you can justify the change, feel free to improve the logic as much as you want.
Cheers!


Fix endOf day on brazilian DST start being the next day

I discovered that the order of the computation of the end of day is resulting in:
moment('2018-11-04T09:00:00-02:00').endOf('day').toString();
// Mon Nov 05 2018 00:59:59 GMT-0200

And this simple change in the order of the operations results in:
moment('2018-11-04T09:00:00-02:00').endOf('day').toString();
// Sun Nov 04 2018 23:59:59 GMT-0200

Is this a good enough solution?

Coverage increased (+0.04%) to 94.691% when pulling 33b0106e85d53321f91da639b85f9acd2957c11a on gfpacheco:fix-dst-endof into 2e2a5b35439665d4b0200143d808a7c26d6cd30f on moment:develop.

Coverage increased (+0.04%) to 94.691% when pulling 33b0106e85d53321f91da639b85f9acd2957c11a on gfpacheco:fix-dst-endof into 2e2a5b35439665d4b0200143d808a7c26d6cd30f on moment:develop.

One month and not a single comment


Create ERD

Entity relationship diagram.

Modified ERDs to use reactnativefs to read and write a JSON local file [embedded database] called 'GPS.json'.


543 google analytics tracking

Description: this works only on frontend pages.

Codecov Report

Merging #832 into master will decrease coverage by 0.15%.
The diff coverage is 37.5%.

@@            Coverage Diff             @@
##           master     #832      +/-   ##
==========================================
- Coverage   91.88%   91.73%   -0.16%
==========================================
  Files         102      102
  Lines        2945     2952       +7
  Branches        6        7       +1
==========================================
+ Hits         2706     2708       +2
- Misses        238      243       +5
  Partials        1        1

Impacted Files
Coverage Δ

frontend/src/app/app.component.ts
44.44% <37.5%> (-55.56%)
:arrow_down:

Continue to review full report at Codecov.

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

@CorneeSean We just spoke yesterday with @w1stler about moving this line:
ga('create', 'UA-XXXXXXXX-XX', 'auto'); // replace on production

somhow to Angular application and storing our UA-XXXXXXX-XX inside environment.
What do You think about it?

@magul I was thinking somewhat the same thing, although I also thought that we could take care of it later on. On the other hand, if we can do it properly now that would be a plus.
As for doing it from angular,I see two solutions: either we inject the script to the page from existing top component by querying a DOM after App load or we add the script ar a link to the script to a template and prevent it from rendering with *ngIF, based on envrionmental variables.
I am not sure though if putting scripts in templates/injecting scripts is allowed at all or by default in new Angular so we should test the solution thoroughly beforehand and connect it to a test GA account for example.

Ok, as for Angular part I have come up and tested the following solution for injecting the script:
https://gist.github.com/CorneeSean/42ed464ee293462be21c82ac1abe0bf0
However, we should connect it to a test analytics app to see if it works. From the front-end side I don't see any reason why it wouldn't, but it would be a good sanity-check :)

@CorneeSean I will check the script you've putted above. Using external library is IMO not necessary (at this point we just need simple statistics).

Closing in favor of #840


i18n(ko-KR): update directives-reference.mdx

Description (required)

update directives-reference.mdx
#9516

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
reference/directives-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Eye tracking exemple's page warning about Gaze input capabilite

Eye tracking exemple's page dosn't warn you about Visual Studio's capability "Gaze Input".
It should probably have a link for the "Getting started with Eye Tracking" page (https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/EyeTracking/EyeTracking_BasicSetup.html).
Existing Doc Link
https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/EyeTracking/EyeTracking_ExamplesOverview.html

Annnnd it took me way to long to get to this one, but I think I finally am adding a note to that doc


Object Markers on some doors disappear after leaving the area and then coming back.

Hello!
I am doing mahogany homes atm, and I started to 'Mark object' for the closed doors, so that I could easily see if I need to open them.
But in this house, and some others, whenever I leave the area, the object markings disappear
To Reproduce
Steps to reproduce the behavior:

Go to 'The house next to the Ardougne Zoo and upstairs '
Object mark the closed doors'
Teleport away
Teleport back
go back to the house
go upstairs and notice the doors not being marked

Expected behavior
Doors should still be marked
Environment (please complete the following information):

OS: Windows 10
RuneLite version: 1.6.29
Launcher version: 1.6.0

Video
https://i.imgur.com/mapXoGv.mp4

For me the door markings stay after teleporting away but the other markers dissapear like the drawer and cabinets.
Drawer examine ID is: 40282, dev tools shows it as 40172. Debugger adds 40172 when marking the object. After teleporting away it dissapears. When both examine and dev tools ID is the same, the markers stay.
Don't know how to fix it, but could be useful to someone else.

FWIW I've noticed this happening at scullisceps as well. I have all the obstacles marked so it's easy to see where to click or if somebody else has cleared the obstacles ahead of me, but it is fairly inconsistent whether or not the obstacles appear marked (I can sometimes clearly see them but they are not marked).

This and #12318 #10758 #9924 all seem to be the same issue

fixed in e5b5292267b3240f7320c05f4c232e4d66b0142b


Support backend command-line flags as variables

The user should be able to specify sleep windows as Ansible variables like
greenpithumb_backend_sleep_windows:
  - 08:00-09:15
  - 16:55-18:15

Which then gets translated into the service definition as sleep_window flags:
  --sleep_window "08:00-09:15"
  --sleep_window "16:55-18:15"

Edit: We really should support all of the backend's command line flags as variables.

Fixed


AttributeError: module 'torch.distributed' has no attribute 'ReduceOp'

pytorch0.4.1. Got above error when training custom dataset. What's wrong with it?

PyTorch 0.4.1 is not longer supported by the master branch. You need either checkout to the old pytorch-0.4.1 branch or upgrade to the latest pytorch.

@hellock  The version I used was just pytorch 0.4.1branch, not the master branch.

You can modify dist.ReduceOp to dist.reduce_op. However, I recommend to upgrade to new versions since the pytorch-0.4.1 branch will receive no new features and bug fixes.


range slider with range selection

Have you planned to add the possibility to select a range within a range?
So for example, if I have a slider with dating range, I could move (a) and (b) as illustrated below :
1900          (a)         (b)                                2000
|-------------<>----------<>------------------------------------|

That would be great!

That would be great! This functionality is implemented in https://github.com/mpowaga/react-slider and https://github.com/davidchin/react-input-range

I needed that feature for a project so I added it.
The main problem is that in the current implementation of react-rangeslider the "handle" is very coupled with the "slider". You need to put the "handle" in a separate class so that you can add more of those easily.
Here is my version in case anyone is interested :
https://gist.github.com/misterfresh/27aa93a7c85559d35b9d97f17fe0f10d


sabnzbd - metrics values

Saw reference to sabnzbd metrics being available in latest version.  Working on a Grafana Dashboard for it, but having some issues with 2 metrics:

sabnzbd_disk_total_bytes seems to be reporting size in number of 1K blocks not bytes.  I get 209715200 1K blocks instead of<PHONE_NUMBER>00 bytes.

sabnzbd_disk_used_bytes not sure what its reporting as it shows up as a decimal such 207901163.52. I'm expecting more like<PHONE_NUMBER> bytes (reported by du -b /downloads).  Value reported seems closer to free space in 1K blocks 207904768 than used space in bytes.

Gah, this is likely a casualty of me collapsing the parsing logic into the UnmarshalJSON method when I pulled this in. The Sabnzbd API uses different approaches to different metrics -- some dynamically set the unit, others are pinned to KB or MB, and others are in bytes. I thought I had unit tests on these to make sure the unit was right, but it looks like I missed some of them. Will look at this later tonight, should be a quick fix.

PR is up for this -- it turns out this behavior was incorrect before pulling it in to exportarr as well. These metrics come back from the SAB API in GB, where the amount of data remaining to download is expressed in MB, and bandwidth is expressed in kbps. I think these should now all be getting correctly parsed, but let me know if you find any other metrics that look off.


Assistants Feature

The demo showed an Assistants button on the left sidebar but I do not have that feature in the version I am running. Was this scrapped or is this coming down the line?
I appreciate the work.

@fitchmultz Expect this by end of week!


Malay language stop words not supported.

Thanks for making this issue.
This repository is only a wrapper Python module around https://github.com/Alir3z4/stop-words repo which contains the actual stop words.
Feel free to open a issue there or much better send a pull request ;)


UI disappears with Australia patch

I started two games today using a nightly release from about a week back and the Australia patch. In both, the UI disappeared about 50-100 turns in when I hit next turn. I reloaded then restarted, and both times the UI disappeared after hitting the next turn button.

Why would Firaxis drop the patch at 11PM right before Friday 😢. I guess I know what I'm doing all tonight and tomorrow!

Yeah, I saw the Australia video and tried to play it yesterday, but it wasn't there. I started a game today and didn't even notice the patch had dropped until the UI disappeared and I saw the alert on the main menu after restarting.

This happens about 20 turns into the American patch.  It coincided with me meeting the first other civilization.  I didn't receive a "hello" video from them, but I got the policy boost.

Welp, it's been three hours and I think I've squashed all of the incompatibilities introduced in the Australian patch. Try downloading the latest commit and letting me know how it works for you guys!

Gonna close this for now since nobody seems to have complained yet!

Upgrade Units in the Unit Report Screen doesn't work with the new Australia patch:
Unit Report Screen:

Upgrade is possible:

@chaorace
Do you reopen this issue or shall we open new issues for every bug for a better overview?

@JHCD The Unit Report Screen is a separate mod from CQUI that we include with their permission. I consider this a URS bug and therefore it deserves its own topic. I've created an issue on your behalf in #340


[PATCH] Remove some warnings on 1.9

http://paste.dimaion.com/6db2c5fc530f9ade4cf680d0dcd94463.txt

On Monday 06 February 2012 15:55:30 you wrote:

http://paste.dimaion.com/6db2c5fc530f9ade4cf680d0dcd94463.txt

Thanks for the patch, but I don't see why missing parens would generate any
warnings.  I'm using 1.9.3 on several projects, and have never seen any
warnings like this.  Could you provide an example?  Thanks again ...
-- Steve

Are you running your script with -w or a truish $VERBOSE?
D.
Sent from my mobile
On Feb 28, 2012 7:51 PM, "Steve Sloan" <
<EMAIL_ADDRESS>wrote:

On Monday 06 February 2012 15:55:30 you wrote:

http://paste.dimaion.com/6db2c5fc530f9ade4cf680d0dcd94463.txt

Thanks for the patch, but I don't see why missing parens would generate any
warnings.  I'm using 1.9.3 on several projects, and have never seen any
warnings like this.  Could you provide an example?  Thanks again ...
-- Steve

Reply to this email directly or view it on GitHub:
https://github.com/igrigorik/em-synchrony/issues/109#issuecomment-4227648

On Tuesday 28 February 2012 14:57:30 you wrote:

Are you running your script with -w or a truish $VERBOSE?

Ah, that did it.  That is, gave me a gazillion warnings (in a Rails app).  I
think you're probably crazy for running with warnings enabled, but I'll make
those fixes anyway.
-- Steve

As a library maintainer it's good practice to run your build with warnings on. If anybody wants to run their own application code with warnings, then your library code shouldn't get in their way. And well, warnings are there for something, so it's a good idea to look at them.
In this case I'm a maintainer for ezmobius/redis-rb. When we run our build with $VERBOSE, igrigorik/em-synchrony warnings are noisy.


Sql DacOperations.ImportAsync does not include ServiceObjective

The ServiceObjective parameter is not available for the ImportAsync method.  The REST api does support it as written here:
https://msdn.microsoft.com/en-us/library/azure/dn781284.aspx?f=255&MSPPError=-2147217396
Would be easy enough to add it to the ServiceManagement/Sql/SqlManagement/Generated/DacOperations.cs file, but this looks like auto-generated code and I don't know the right process to update it.
Would really appreciate a fix for this since we're using these API's to restore databases pretty often as part of our automated QA process.  Thanks!

Issue with loading/using certificates, not with SQL.

The ImportRequestParametersnow has the serviceObjectiveName parameter.


update README to include requirements.txt

The setup described in the README is a bit out of date

Updating it based on my recent install experience

I wanted to get quick feedback from others before I modify the read, but here's my experience with installing the server from scratch (on a MacOS X 10.10 laptop with macports), and trying to run the nosetests:
The one external dependency not explicitly covered by the readme file is wormtable's dependency on Berkeley DB. While a link to the document that covers how to satisfy this dependency is eventually given, I was thinking of moving it up a couple of sentences in the readme, so that a first-time user following along with  the instructions doesn't get the inevitable error the first time they try to pip install wormtable.
The second thing I noticed, when running the nosetests, is that flask has presumably created a bunch of dependencies that aren't sufficiently covered by the existing requirements.txt file, namely:
mccabe, pyflakes, werkzeug, jinja2, itsdangerous, six
Unless I hear otherwise, I'll update the requirements.txt file with the above packages.

@macieksmuga, absolutely agree and everything has to be documented clearly and in a step-by-step fashion, for different OS environments on both server or local implementation with logging for debugging.  This does not need to be complete in the first iteration, and can be streamlined over time.
It took me a couple of hours of setting everything up from scratch back in December on a cluster under only user-privilege level.  It definitely has to be a turn-key solution and we can't rely on the user to have significant expertise in setting things up, since working with the implementation and possibly updating the code would have a higher priority.
Paul

It's probably a good idea to emphasise the Berkeley DB dependency for wormtable, as this would be a nasty stumbling block for a lot of users. We are planning on removing wormtable soon (hopefully!), so I wouldn't put too much work into this though.
For requirements.txt, I don't think there's any need to explicitly list the recursive requirements for Flask etc. Pip works these out, so we should just keep it to the list of packages that we use directly.

Thanks @jeromekelleher and @pgrosu for your feedback!
Concerning the automatic dependency graph in pip, it didn't quite work out the first time in my case - which is why I had to perform the above listed pip installs manually after running the initial pip install -r requirements.txt
I realized that the reason the process failed was due to wormtable install exiting on error. Once that was fixed, the flask dependencies were loaded automatically as expected.


Changes to styling

Added standardStyle option to Radio.
Support fontFamily for Item.
Added theme for Input

Merged with 2.1.4


DIR_SEPARATOR_S

Can't find in docs, seems not implemented yet?
https://docs.gtk.org/glib/const.DIR_SEPARATOR_S.html

See https://doc.rust-lang.org/stable/std/path/constant.MAIN_SEPARATOR.html


[vcpkg baseline][wavelib] Fix parallel build

What does your PR fix?
Fixes parallel builds by disabling the test dir which creates test executables in the source dir.
(Assumed to fix msbuild issue in #21507.)

Which triplets are supported/not supported?

Ping @BillyONeal for merge this PR first if the pipeline test pass.

Thanks for the fix!


New release of go-winio to support the new pkg/process

pkg/process that was introduced in https://github.com/microsoft/go-winio/commit/f272bf30b3688d687e78f81da3e1d8caa2dba212 and is being used already in:
https://github.com/microsoft/hcsshim/commit/1e4afe9fdf8f3210e7631d1f21ae3c40d5ad87ae#diff-557d18187d922c13ba3a0277fac10fa3R9
Can we please cut a release to support the new code and then update microsoft/hcsshim?
Context - trying to rationalize the dependencies and bring in newer versions of stuff into kubernetes ( WIP in https://github.com/kubernetes/kubernetes/pull/86975 )

https://github.com/microsoft/go-winio/releases/tag/v0.4.15 appears to have taken care of this.

https://github.com/microsoft/go-winio/releases/tag/v0.4.15 appears to have taken care of this.

thanks @TBBle !

thanks @TBBle !


Fund the DOM.money – an unstoppable bank

Scope
DOM Money is building unstoppable, autonomous non-custodial service financial services for non-crypto users and businesses. We provide an option to drop out of the financial systems of the countries.
Our first product is an unstoppable non-custodial stablecoin wallet with a convenient bank interface built to run forever, ignoring external impacts. Hosted on decentralized permanent hosting with strong encryption and untraceability of all traffic, Wallet includes a portfolio allocation engine that distributes stablecoins across DeFi protocols with weekly autocompounding. It hedges risk across 4+ blockchains and 20+ DeFi products.
Known problems to solve:

Knowledge of crypto technologies:
To get all benefits of crypto products users have to have some knowledge in blockchain technologies.

Ownership and censorship-resistant storing of capital:
The problem of ownership of money gets more and more critical in modern unstable world. We see how governments and corporations restrict access to bank accounts. Even crypto exchanges start blocking accounts of the average citizen.

Unattractive interest rates in dollars:
Last but not least the interest rates in dollars remain unattractive. So people are looking for alternative ways of saving their money.

Benefits to boost the Yearn ecosystem?

Attract additional liquidity - we plan to place 30% of all users' funds in Yearn and thus provide additional liquidity to the network.
Increase capital efficiency with DOM money ML-driven portfolio allocation engine.
Attract additional funds from crypto newcomers

Plan

Our first product is a non-custodial stablecoin wallet with a convenient conventional bank interface.
DOM is going to be hosted on decentralized permanent hosting and with strong encryption and untraceability of all traffic. - Wallet includes a portfolio allocation engine that distributes stablecoins across DeFi protocols with weekly autocompounding. It hedges risk across 4+ blockchains and 20+ DeFi products.
We plan to place 30% of all users' funds in Yearn network and thus provide additional liquidity to the network.

Deadline
2022-12-25
People
Team of 7 people and growing
Nikolai Grigoriev, co-founder & CEO
Stepan Mytchyk, co-founder & Head of Crypro
Andrii, co-founder & CTO
Denis, UX
Artem, Chief Investment Officer
Ihor, Front end developer
Albert, Business Development
Money
Budget plan:
80% - Team expansion, hiring, salaries
2 solidity developers ML-engineer
Community manager
Marketing lead
10% Paid services, subscriptions additional costs
10% Paid marketing at platforms like https://beta.layer3.xyz/
Amount:
$50K - integration of Yearn in Ethereum, Arbitrum, Fantom.
$120K - development of stablecoin focused cross-chain ML-driven portfolio allocation engine based on Yearn liquidity pools with evolutionary models and cross-chain assets transfers.
Amount
$50K - integration of Yearn in Ethereum, Arbitrum, Fantom; $120K - development of stablecoin focused cross-chain ML-driven portfolio allocation engine based on Yearn liquidity pools with evolutionary models and cross-chain assets transfers.
Wallet address
0x0
Reporting

[X] I agree to post weekly updates

Thanks for request. Unfortunately, the grant has been rejected as it falls outside of yearn’s direct scope.


Links not work at all

I made MD hyperlink with []() syntax and see that link is rendered properly in a FlowDocument. BUT... when I click link, nothing happens! Why? I expect link will open browser with given URL.
When I investigated this bug, people also had the same problem, but with normally created FlowDocument. They simply assign to link RequestNavigate handler and in a handler they call Process.Start(). I tried the same with your FlowDocument, but no luck - this handler never ever called. In WPF logs I saw no error, so code is OK. When I manually created Hyperlink and assigned RequestNavigate, code works. That means your WPF renderer creates "invalid" Hyperlink that it cannot even call own handler.
Solution: make please link behave as a link! Not a huge job, but annoying when link doesn't behave as it must.

Links are working. You need to assign a command as shown in the samples.
xaml
<CommandBindings>
  <CommandBinding Command="{x:Static markdig:Commands.Hyperlink}" Executed="OpenHyperlink" />
</CommandBindings>

code-behind
private void OpenHyperlink(object sender, System.Windows.Input.ExecutedRoutedEventArgs e)
{
  Process.Start(e.Parameter.ToString());
}


[Bug] Data Visible in Flink UI but Not Reaching Downstream Database

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
flink-1.19.1
flink-doris-connector-1.19-1.6.2.jar
flink-cdc-common-3.0.1.jar
flink-sql-connector-mysql-cdc-3.0.1.jar
flink-sql-connector-oracle-cdc-3.0.1.jar
What's Wrong?

I'm observing a discrepancy between the data shown in the Flink UI and what's actually being written to the downstream database. The Flink UI shows a significant amount of data being processed and sent (as indicated by the colored blocks representing different tasks), but when I check the downstream database, no records are present. This suggests that while data is being read from the source and processed within Flink, it's not successfully being written to the target database.
What You Expected?
I expected that the data shown as processed in the Flink UI would correspond to actual records written to the downstream database.
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

"Process -> (BMS_BATCH_DEF: Writer -> BMS_BATCH_DEF: Committer, BMS_LOT_DEF: Writer -> BMS_LOT_DEF: Committer, BMS_STORER_POS: Writer -> BMS_STORER_POS: Committer, BMS_ST_IO_DOC: Writer -> BMS_ST_IO_DOC: Committer, BMS_ST_IO_DTL: Writer -> BMS_ST_IO_DTL: Committer, BMS_SU_SET_DOC: Writer -> BMS_SU_SET_DOC: Committer, BMS_SU_SET_DTL: Writer -> BMS_SU_SET_DTL: Committer, GPCS_INSIDER: Writer -> GPCS_INSIDER: Committer, GPCS_PLACEPOINT: Writer -> GPCS_PLACEPOINT: Committer, GPCS_PLACEPOINT_CLASS: Writer -> GPCS_PLACEPOINT_CLASS: Committer, GPCS_PLACEPOINT_CLASS_DTL: Writer -> GPCS_PLACEPOINT_CLASS_DTL: Committer, GRESA_SA_DOC: Writer -> GRESA_SA_DOC: Committer, GRESA_SA_DTL: Writer -> GRESA_SA_DTL: Committer, NGPCS_ALL_PRICE: Writer -> NGPCS_ALL_PRICE: Committer, NP_EFILES_OP_FILE: Writer -> NP_EFILES_OP_FILE: Committer, PUB_BRAND: Writer -> PUB_BRAND: Committer, PUB_EMPLOYEE: Writer -> PUB_EMPLOYEE: Committer, PUB_FACTORY: Writer -> PUB_FACTORY: Committer, PUB_GOODS: Writer -> PUB_GOODS: Committer, PUB_GOODS_CLASS: Writer -> PUB_GOODS_CLASS: Committer, PUB_GOODS_CLASS_DTL: Writer -> PUB_GOODS_CLASS_DTL: Committer) (2/3)#0" Id=12810 WAITING on java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject@24273330 	at java.base@11.0.22/jdk.internal.misc.Unsafe.park(Native Method) 	-  waiting on java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject@24273330 	at java.base@11.0.22/java.util.concurrent.locks.LockSupport.park(LockSupport.java:194) 	at java.base@11.0.22/java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.await(AbstractQueuedSynchronizer.java:2081) 	at java.base@11.0.22/java.util.concurrent.ArrayBlockingQueue.take(ArrayBlockingQueue.java:417) 	at org.apache.doris.flink.sink.writer.RecordBuffer.write(RecordBuffer.java:93) 	at org.apache.doris.flink.sink.writer.RecordStream.write(RecordStream.java:63) 	at org.apache.doris.flink.sink.writer.DorisStreamLoad.writeRecord(DorisStreamLoad.java:251) 	at org.apache.doris.flink.sink.writer.DorisWriter.writeOneDorisRecord(DorisWriter.java:216) 	at org.apache.doris.flink.sink.writer.DorisWriter.write(DorisWriter.java:186) 	at app//org.apache.flink.streaming.runtime.operators.sink.SinkWriterOperator.processElement(SinkWriterOperator.java:160) 	at app//org.apache.flink.streaming.runtime.io.RecordProcessorUtils$$Lambda$825/0x00000008007eec40.accept(Unknown Source) 	at app//org.apache.flink.streaming.runtime.tasks.CopyingChainingOutput.pushToOperator(CopyingChainingOutput.java:75) 	at app//org.apache.flink.streaming.runtime.tasks.CopyingChainingOutput.collect(CopyingChainingOutput.java:61) 	at app//org.apache.flink.streaming.runtime.tasks.ChainingOutput.collectAndCheckIfChained(ChainingOutput.java:96) 	at app//org.apache.flink.streaming.runtime.tasks.BroadcastingOutputCollector.collect(BroadcastingOutputCollector.java:94) 	at app//org.apache.flink.streaming.api.operators.ProcessOperator$ContextImpl.output(ProcessOperator.java:103) 	at org.apache.doris.flink.tools.cdc.ParsingProcessFunction.processElement(ParsingProcessFunction.java:51) 	at org.apache.doris.flink.tools.cdc.ParsingProcessFunction.processElement(ParsingProcessFunction.java:31) 	at app//org.apache.flink.streaming.api.operators.ProcessOperator.processElement(ProcessOperator.java:66) 	at app//org.apache.flink.streaming.runtime.io.RecordProcessorUtils$$Lambda$825/0x00000008007eec40.accept(Unknown Source) 	at app//org.apache.flink.streaming.runtime.tasks.OneInputStreamTask$StreamTaskNetworkOutput.emitRecord(OneInputStreamTask.java:238) 	at app//org.apache.flink.streaming.runtime.io.AbstractStreamTaskNetworkInput.processElement(AbstractStreamTaskNetworkInput.java:157) 	at app//org.apache.flink.streaming.runtime.io.AbstractStreamTaskNetworkInput.emitNext(AbstractStreamTaskNetworkInput.java:114) 	at app//org.apache.flink.streaming.runtime.io.StreamOneInputProcessor.processInput(StreamOneInputProcessor.java:65) 	at app//org.apache.flink.streaming.runtime.tasks.StreamTask.processInput(StreamTask.java:579) 	at app//org.apache.flink.streaming.runtime.tasks.StreamTask$$Lambda$746/0x00000008006ff440.runDefaultAction(Unknown Source) 	at app//org.apache.flink.streaming.runtime.tasks.mailbox.MailboxProcessor.runMailboxLoop(MailboxProcessor.java:231) 	at app//org.apache.flink.streaming.runtime.tasks.StreamTask.runMailboxLoop(StreamTask.java:909) 	at app//org.apache.flink.streaming.runtime.tasks.StreamTask.invoke(StreamTask.java:858) 	at app//org.apache.flink.runtime.taskmanager.Task$$Lambda$1135/0x0000000800928c40.run(Unknown Source) 	at app//org.apache.flink.runtime.taskmanager.Task.runWithSystemExitMonitoring(Task.java:958) 	at app//org.apache.flink.runtime.taskmanager.Task.restoreAndInvoke(Task.java:937) 	at app//org.apache.flink.runtime.taskmanager.Task.doRun(Task.java:751) 	at app//org.apache.flink.runtime.taskmanager.Task.run(Task.java:566) 	at java.base@11.0.22/java.lang.Thread.run(Thread.java:834)

Is checkpoint enabled for the flink job?

Is checkpoint enabled for the flink job?

Yes.

FAILED

During full synchronization, Oracle will not start writing until it has completed pulling the existing data. You can try increasing these parameters https://nightlies.apache.org/flink/flink-cdc-docs-release-3.1/docs/connectors/flink-sources/oracle-cdc/#cant-perform-checkpoint-during-scanning-snapshot-of-tables

thanks, bro @JNSimba


license change

Would it be possible to replace the current Zlib License with a very similar, copyfree-certified license?  The Zlib License has been rejected for copyfree certification because of what appears to be a bookkeeping clause:

Zlib License restriction 2 contains the requirement "Altered source versions must be plainly marked as such," which could reasonably be interpreted by the courts in a manner that violates Copyfree Standard Definition point 3. Free Modification and Derivation. Alternatively, it may be interpreted as merely requiring that a derivative or modified work be renamed, given the concluding clause, "must not be misrepresented as being the original software." Until such issues are clarified, particularly as to the intent of the license authors and the legal applicability of these terms as explained by relevant legal professionals, the Zlib License must be regarded as contrary in some respects to the requirements of the Copyfree Standard Definition.

The most similar copyfree-certified licenses appear to be the General Attribution License and the Software Attribution License.  The two are almost identical, with the exception that the SAL very specifically mentions "software" and "source", while the GAL is . . . well, more general, and probably preferable overall.

I just sent an email to all the contributors (via their git email address) about changing from zlib to MIT.  Once an agreement is reached, I'll switch over the license.

I agree to the relicensing.

Agreed.

agreed

Ping @avinassh - I don't think I have a proper email address.  Have to ask since you've contributed to the project - are you okay if I switch the license from zlib to MIT?

Agreed.

@econchick I have no issues with the relicensing :smile: (and just in case, my email is<EMAIL_ADDRESS>

Hi folks - if you're tagged, please let me know via email or posting here that you're okay with changing the license from zlib to MIT (or let me know if you have any questiosn regarding this):
@beroe @aklapcin @widnyana @Nihdez @talam @rodney757 @Lazar-T @dsimandl @CheriPai @ironfroggy @bmwasaru

Agreed.

Yea, that's fine by me. Thanks for asking!
On Tue, Jan 27, 2015 at 2:33 PM, Lynn Root<EMAIL_ADDRESS>wrote:

Hi folks - if you're tagged, please let me know via email or posting here
that you're okay with changing the license from zlib to MIT (or let me know
if you have any questiosn regarding this):
@beroe https://github.com/beroe @aklapcin https://github.com/aklapcin
@widnyana https://github.com/widnyana @Nihdez
https://github.com/Nihdez @talam https://github.com/talam @rodney757
https://github.com/rodney757 @Lazar-T https://github.com/Lazar-T
@dsimandl https://github.com/dsimandl @CheriPai
https://github.com/CheriPai @ironfroggy https://github.com/ironfroggy
@bmwasaru https://github.com/bmwasaru
—
Reply to this email directly or view it on GitHub
https://github.com/econchick/new-coder/issues/69#issuecomment-71712168.

--
-Tanvir Alam

Agreed
On Tuesday, January 27, 2015, Tanvir Alam<EMAIL_ADDRESS>wrote:

Yea, that's fine by me. Thanks for asking!
On Tue, Jan 27, 2015 at 2:33 PM, Lynn Root<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Hi folks - if you're tagged, please let me know via email or posting
here
that you're okay with changing the license from zlib to MIT (or let me
know
if you have any questiosn regarding this):
@beroe https://github.com/beroe @aklapcin https://github.com/aklapcin

@widnyana https://github.com/widnyana @Nihdez
https://github.com/Nihdez @talam https://github.com/talam
@rodney757
https://github.com/rodney757 @Lazar-T https://github.com/Lazar-T
@dsimandl https://github.com/dsimandl @CheriPai
https://github.com/CheriPai @ironfroggy https://github.com/ironfroggy

@bmwasaru https://github.com/bmwasaru
—
Reply to this email directly or view it on GitHub
https://github.com/econchick/new-coder/issues/69#issuecomment-71712168.

--
-Tanvir Alam
—
Reply to this email directly or view it on GitHub
https://github.com/econchick/new-coder/issues/69#issuecomment-71724266.

--
Aleksandra Kłapcińska

Ok with me!
On Tue, Jan 27, 2015 at 4:47 PM, Aleksandra Kłapcińska <
<EMAIL_ADDRESS>wrote:

Agreed
On Tuesday, January 27, 2015, Tanvir Alam<EMAIL_ADDRESS>wrote:

Yea, that's fine by me. Thanks for asking!
On Tue, Jan 27, 2015 at 2:33 PM, Lynn Root<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Hi folks - if you're tagged, please let me know via email or posting
here
that you're okay with changing the license from zlib to MIT (or let me
know
if you have any questiosn regarding this):
@beroe https://github.com/beroe @aklapcin <
https://github.com/aklapcin>

@widnyana https://github.com/widnyana @Nihdez
https://github.com/Nihdez @talam https://github.com/talam
@rodney757
https://github.com/rodney757 @Lazar-T https://github.com/Lazar-T
@dsimandl https://github.com/dsimandl @CheriPai
https://github.com/CheriPai @ironfroggy <
https://github.com/ironfroggy>

@bmwasaru https://github.com/bmwasaru
—
Reply to this email directly or view it on GitHub
<
https://github.com/econchick/new-coder/issues/69#issuecomment-71712168>.

--
-Tanvir Alam
—
Reply to this email directly or view it on GitHub
https://github.com/econchick/new-coder/issues/69#issuecomment-71724266.

--
Aleksandra Kłapcińska
—
Reply to this email directly or view it on GitHub
https://github.com/econchick/new-coder/issues/69#issuecomment-71734601.

agreed

Agreed.
On Thu, Jan 29, 2015 at 3:11 AM, Widnyana Putra<EMAIL_ADDRESS>wrote:

agreed
—
Reply to this email directly or view it on GitHub
https://github.com/econchick/new-coder/issues/69#issuecomment-71956739.

Has there been any more movement on this issue?  I noticed it's still open, and the LICENSE file has not changed.

@apotheon I haven't received 100% of contributors' agreement on this - probably more like 90% have agreed, and 10% I haven't been able to reach. Unsure of what to do, but perhaps "best effort" may be good enough...


nzQueryParams not called

Reproduction link
https://stackblitz.com/edit/angular-qwhhql?file=src%2Fapp%2Fapp.component.ts
Steps to reproduce
Use nz-table as standalone component. Try to use the AJAX features as in the documentation: https://ng.ant.design/components/table/en#components-table-demo-ajax
What is expected?
The expected behaviour is that the function specified in nzQueryParams will be called when sorting changes. Paginiation should show up with the code from the documentation.
What is actually happening?
The function specified in nzQueryParams is not called when sorting changes. Pagination is missing, even though it shows in the documentation.

Environment
Info

ng-zorro-antd
18.1.0

Browser
Chrome

@Component({
  selector: 'app-order-table',
  standalone: true,
  imports: [
    NzTableModule, <---
    NzEmptyComponent,
    NzFlexDirective,
  ],
  templateUrl: './order-table.component.html',
})


[FEATURE REQ] Cosmos: Direct TCP: Port support for recording important events in the lifetime of a request

This is a port from v4 to v3. See #6580.

Status: Not started.

Tracking this issue here: https://github.com/Azure/azure-sdk-for-java/issues/6580


FancyZone, Space Around Zones is lost when upgrading to newer version of PT

ℹ Computer information

PowerToys version: 0.25.0
PowerToy Utility: FancyZones
Running PowerToys as Admin: Yes
Windows build number: 2004 19041.508

📝 Provide detailed reproduction steps (if any)

Launch Zones Editor and set Space Around Zones to value of 4
Upgrade to latest version of PT
Launch Zones editor
Space Around Zones has reverted to 16
This happens most times I upgrade to a new version of PT

✔ Expected result
Space Around Zones values is maintained when upgrading to a newer version
❌ Actual result
Space Around Zones values is lost when upgrading to a newer version

@umernalla
in 0.31.2 we have changed the way "space around zones" works.
I'm closing this issue since it doesn't apply anymore.
If you are still having problems with 0.31.2, please open a new issue.
Thanks.


Fix Traefik routing to include a redirect for trailing slash

You almost had this working, but it required an extra bit of middleware.  We need to force any request for /v1/rss-bridge to use /v1/rss-bridge/ instead (trailing slash).
I've added a RedirectRegex middleware to the router, which will intercept any request for /v1/rss-bridge and turn it into /v1/rss-bridge/.  This will mean that relative URLs like static/style.css will get properly resolved to http://localhost/v1/rss-bridge/static/style.css.
We should really do something similar in a few other services (status service comes to mind).  I'll file a follow-up on that.
I'm now able to do:
pnpm services:start rss-bridge

And navigate to http://localhost/v1/rss-bridge

I cannot test it right now because the my branch seems to have the problem of mdx that was solved later. I will merge the changes now, rebase my branch, and check that it works

You don't know how much I needed this Traefik bit too. Bless you @humphd

@cindyledev if you're talking about the /docs stuff, I'd do it through nginx, like we do with /portainer and /deploy, see https://github.com/Seneca-CDOT/telescope/blob/master/config/nginx.conf.template#L228-L239

@cindyledev if you're talking about the /docs stuff, I'd do it through nginx, like we do with /portainer and /deploy, see https://github.com/Seneca-CDOT/telescope/blob/master/config/nginx.conf.template#L228-L239

!@#%%^ time for a new adventure with nginx. All the stuff I touched this week makes me feel like I was thrown into a Hyperbolic Time Chamber. One week on Telescope feels like months 🤣


Deprecate profile()

Description
Deprecate the profile function. It looks so convenient, but it is evil because the code cannot be shaken out causing us to produce larger than necessary binaries. It's also easy to replace: if (!kReleaseMode),
Related Issues
Fixes https://github.com/flutter/flutter/issues/23756
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes tests for all changed/updated/fixed behaviors (See Test Coverage).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes).
[x] No, this is not a breaking change because the function is just marked as deprecated.

This is reducing snapshot sizes by 35KB.

The whole point of this function was to have the Dart team fix the problem that led to this not being tree-shakable, either by providing a dedicated profile function, similar to assert, or by solving it in a way similar to https://github.com/dart-lang/sdk/issues/35718.


Dagger 2.39.1

@moljac can we speed up this review and package publish.

@antonioseric Thanks for the contribution
dagger is dependency of GPS-FB-MLKit, so it was moved to that repo
https://github.com/xamarin/GooglePlayServicesComponents/blob/main/config.json#L595-L603
in the future check config.json files before updating packages here, please.


Not explicit function name

In section Function names should say what they do you're explicitly saying that you're adding a month to a date, addMonthToDate().
In section Remove duplicate code you're showing the employees list using showList().
Shouldn't that be showEmployeesList() as well, in order to explicitly tell what will be shown, so you just have to pass the type of the employee as an argument (managers or developers in that context)?
I'm not really sure about it, which is why I wanted to discuss about. Thanks for this great work!

You know what, you're right. It should say showEmployeesList! Want to make a PR?

Sure!

The other examples give
showDeveloperList without (s)
showManagerList without (s)
and now we use showEmployessList with (s)
which is the best case?

Haven't thought about that. I usually use in the plural form when a certain method has many properties as a result, so showEmployeesList (). But to talk about the best practices I can't say which is better. In the other hand, in order to keep consistency, I think we should change to showEmployeeList().
What you think?

This gets into a bit of linguisitics, which is off in the weeds away from Clean Code, but a noun pre-modifier is usually singular: Grocery List, Customer List, Client List, etc. I think showEmployeeList would be both consistent with the document and with the language 🤓 I could be comfortable with either, but let's just make it consistent then. Thanks for putting so much time into picking good names! Half of programming skill is about naming and organization.

Half of programming skill is about naming and organization.

😂  indeed.
I also agree with either, as long as your team approve with the naming.

I'll change it to showEmployeeList() then. Thanks @hueitan for pointing that out. ;)


Update scala-library to 2.13.10

Updates org.scala-lang:scala-library from 2.13.7 to 2.13.10.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-lang", artifactId = "scala-library" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #141.


test_tornado.py: Add websocket closing to tornado tests.

Extend tornado tests with closing WebSocket connection
to avoid leakings warnings.

Fixes #3942

Automated message from Dropbox CLA bot
@kkanahin, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @kkanahin!  It's nice to have our backend tests up to 100% coverage (aka no dead test code).


QualityOfService-Level doesn't get set in ManagedClient

Describe the bug
QualityOfService-Level doesn't represent expected Level. Instead it always defaults to 'AtMostOnce'.
Used Packages

MQTTnet with Version 3.1.2
MQTTnet.AspNetCore with Version 3.1.2
MQTTnet.Extensions.ManagedClient with Version 3.1.2

Which component is your bug related to?

ManagedClient

To Reproduce
Check out this example project which reproduces the bug.
https://github.com/Peet-HD/MQTTnet-QoS-Issue
Expected behavior
QualityOfService-Level should be properly received and should not default to "AtMostOnce".
Console Output

This is not a bug. MQTT does not support upgrading the QoS. So if you publish with QoS 2 but subscribing with QoS 0 the message will be transmitted using QoS 0. Other brokers like mosquitto do support this by configuration flags even if not allowed in MQTT RFC but this library does not (yet).


Show per task retry count on job detail page

What would you like to be added:
Show per task retry count on job detail page
Why is this needed:
So user can know why some tasks are running unexpectedly (fail/longer/etc)
Without this feature, how does the current module work:
Components that may involve changes:

closed and track in https://github.com/microsoft/pai/issues/4649


Wipes destination manifests on write. Resolves Issue #16

See Issue #16

@lprichar I've changed the base of the merge.
We also use develop as our main develop branch so your feature should derive from develop.
I will need to create the contributing guide for this from now on.

You might consider changing your default branch to develop if you'd like PR's to go against there.  Also, I'm pretty sure ReadWrite is the default for all mode's other than Append, but I implemented as requested, no harm in being explicit I guess.

I don't wan't to change the default branch to master since the master represents the current release and it contains the readme for it.
development can/should have more new features/updates that we don't want to show in the main github page.
We're using gitflow for branch management but it isn't clear yet. I will create a contributing guide with it.
The requested change was for clarity only.


Possible documentation refinement for the MongoDBContainer module

Expected Behaviour
I was following the instructions on the MongoDB Module page and trying to implement in an ES6 environment that outputs CommonJS. The examples don't include the require()/import statement which would have helped a ton.
I also expected that once getting the container running and returning the value from container.getConnectionString() that it would simply connect using that string. I found that I had to add ?directConnection=true to the connection string for it to actually connect.
Also, the checkMongo function seems to be from the test suite instead of from the library itself.
Proposed changes:

Make the generic similar to the below:

  // ES6
  import { MongoDBContainer, StartedMongoDBContainer }  from '@testcontainers/mongodb';
  // --or--
  // CommonJS
  const { MongoDBContainer, StartedMongoDBContainer } = require('@testcontainers/mongodb');

  const mongodbContainer:StartedMongoDBContainer = await new MongoDBContainer().start();

  // ...your code

  await mongodbContainer.stop();

It would be super helpful to have some API docs for the modules. I ended up using container.getConnectionString() to get the connection string back to my testing suite.
Include a note about the usage of ?directConnection=true in conjunction with the connection string returned. I ended up concatenating it to the end elsewhere: mongodb://localhost:[PORT]/test-db?directConnection=true

Environment Information

Operating System: MacOS 14.2 (Apple M1 Max)
Docker Version: 24.0.7, build afdd53b
Node version: v18.12.1
Testcontainers version: ^10.4.0

I bumped into the same issues.
FWIW, I've created a minimal example repository that shows the issue when not using directConnection: true: https://github.com/robvanderleek/testcontainers-mongodb-issue
Environment Information

Operating System: MacOS 14.3.1 (Apple M1 Max)
Docker Version: 24.0.6
Node version: v20.11.0
Testcontainers version: ^10.7.1

I'm no Mongo expert.. should directConnection=true be included by default in the connection string?

Also not an expert, but I tested connecting to a natively running MongoDB (using Homebrew) and a containerized MongoDB (launched with docker run -p 27018:27017 mongo:6.0.1) and they both work without directConnection=true.

I've noticed the Java implementation of Testcontainers does not expose this directConnection option either. IMO it's a decision for whoever is setting up Mongo/the library they use to connect to decide whether or not to use this option.

@cristianrgreco Hi, by default the code creates a Mongodb Replica Set
class MongoDBContainer extends testcontainers_1.GenericContainer {
    constructor(image = "mongo:4.0.1") {
        super(image);
        this.withExposedPorts(MONGODB_PORT)
            .withCommand(["--replSet", "rs0"])
            .withWaitStrategy(testcontainers_1.Wait.forLogMessage(/.*waiting for connections.*/i))
            .withStartupTimeout(120000);
    }
....

Hence you need to tell the driver to use a direct connection.
If you tested this container with mongosh from a command line you can see that it adds this option by default.
$ mongosh localhost:55076/somedb

Current Mongosh Log ID:	65d36046a01ccf3f8e133100
Connecting to:		mongodb://localhost:55076/somedb?directConnection=true&serverSelectionTimeoutMS=2000&appName=mongosh+2.1.4

This tripped me up as well, as I wasn't expecting Testcontainers to create a replica set by default.
I agree that at minimum the docs need to be updated to make this clear and ideally the implementation would support both use cases.

Thanks all for the feedback. If the docs made it clear that by default the container starts a replica set, and update the connection URI to work appropriately, would you still feed the need to support both use cases? Any PR to address either of these would be very welcome.


[fix][doc] Avoid refer to deprecated classes

This closes #8917.
Documentation

[x] doc
[ ] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Merging...


Top Navigation Layout

So i have encountered an issue i think, so if we have a long navbar items, we have to go on getbootstrap website if we go to customize and change the @grid-float-breakpoint from @screen-sm-min to @screen-md-min for example, and change the bootstrap files on AdminLTE folder the bars and the rest of the content on the right do not stay "organized" like the default navbar, there is another away to change the width of the navbar to collapse on AdminLTE? or it's really an issue?

Due to some neglect in my part, many issues weren't addressed for some time. Some of these issues are no longer relevant since a solution might have been found. Therefore, we decided to close all issues that are over a month old. However, if the issue still persists and you'd like help, then by all means repost your issue and we will consider it again. In the mean time, we will close this post.
Thanks for your understanding and we apologize for any inconvenience <3.


add dartium

Dartium is a build of Chrome with a built-in Dart VM. Useful for development.

Great, thanks!


PlaneMeshBuilder should include a configurable amount of subdivision for the mesh

What problem does this solve or what need does it fill?
In bevy 0.12 the
// mod shape
struct Plane {
  size: f32,
  subdivision: u32
}

struct was used to construct plane meshes.
The subdivision parameter controlled the amount of subdivisions in the resulting mesh.
This control was removed with the introduction of Plane3d.
However, this behavior is very useful for prototyping or games that generate their own meshes in bevy instead of loading them from external sources and should be provided by bevy IMO.
What solution would you like?
Add a subdivision parameter to PlaneMeshBuilder.
The subdivisions should default to the current behavior.
I also believe the code for creating the mesh can be easily ported from the old implementation in bevy 0.12.
pub struct PlaneMeshBuilder {
    pub plane: Plane3d,
    pub half_size: Vec2,
    // The new parameter
    // should default as before
    pub subdivisions: u32,
}

Debatable
I don't think the old design of subdivisions = 0 is great. The docs in 0.12 state:

subdivisions: u32
The number of subdivisions in the mesh.
0 - is the original plane geometry, the 4 points in the XZ plane.
1 - is split by 1 line in the middle of the plane on both the X axis and the Z axis, resulting in a plane with 4 quads / 8 triangles.
2 - is a plane split by 2 lines on both the X and Z axes, subdividing the plane into 3 equal sections along each axis, resulting in a plane with 9 quads / 18 triangles.
and so on...

Maybe this could be better represented as a enum? I'm open to suggestions here.

I also just saw that this is already included in the tracking issue #10572

I think subdivision should be part of Meshable trait which could generate any standard shapes with any level of detail, because bevy have no tesselation stage, if someone wants to make some vertex shader like noice there is no options for standard shapes.


GH-35652: [Go][Compute] Allow executing Substrait Expressions using Go Compute

Rationale for this change
Providing the ability to execute more complex expressions than single operations by leveraging Substrait's expression objects and deprecating the existing separate Expression interfaces in Go Arrow compute. This provides a quick integration with Substrait Expressions and ExtendedExpressions to start building more integrations.

What changes are included in this PR?
This PR provides:

an extension registry for Go arrow to provide mappings between Arrow and substrait for functions and for types along with other custom mappings if necessary.
Facilities to convert between Arrow data types and Substrait types
Functions to evaluate Substrait expression objects with Arrow data as the input
Functions to evaluate Substrait field references against Arrow data and arrow schemas

Are these changes tested?
Yes, unit tests are included.

Are there any user-facing changes?
Existing compute.Expression and its friends are being marked as deprecated.

Closes: #35652

CC @benibus @felipecrv if you two feel like taking a look / reviewing on this despite your lack of familiarity with Go

Yes, but this is a big one, so next week.