Hybrid rendering without any pre-rendered routes should not execute application bootstrapping

Command
build
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
For my use-case I use Angular 19 Hybrid SSR. For now, I only need the SSR part, not making use of any other RenderMode, but I want to keep the doors open for later.
During the bootstrap of the application I'm using an app initializer to get some data via HTTP. The website cannot run without it (think of getting different configurations from an API depending on the hostname), so it makes sense to get the data as soon as possible and assume the website always runs with a preloaded configuration.
When I use Angular SSR my builds works as expected.
When I sure Angular Hybrid SSR my build hangs because it tries to make a request to the API which is not available on the build server.
The configuration for Hybrid SSR is:
// angular.json
"outputMode": "server"

// app.routes.server.ts
export const serverRoutes: ServerRoute[] = [
  {
    path: '**',
    renderMode: RenderMode.Server
  }
];

As you can see, there's no pre-rendering involved.
I want to use Hybrid SSR because it makes use of the same request handler during development where the normal SSR uses its own internal SSR middleware. Using the same handler seems more error-proof to me.
Minimal Reproduction
For both examples make sure to have Angular 19 installed.
Example with Angular SSR

ng new v19ssr --ssr
In app.config.ts add an app initializer: provideAppInitializer(() => console.log('Hello'))
In angular.json change the existing prerender to false.
Run ng build

Observe, that there is no Hello in your console.
This is as it should be.
Example with Angular Hybrid SSR

ng new v19ssrSR --ssr --server-routing
In app.config.ts add an app initializer: provideAppInitializer(() => console.log('Hello'))
In app.routes.server.ts switch the render mode to Server
In angular.json change the outputMode to server
Run ng build

Observe, that there is Hello in your console, so the whole app has been boostrapped even when there is no route to prerender.
Exception or Error

Your Environment
Angular CLI: 19.0.0
Node: 20.17.0
Package Manager: npm 10.8.2
OS: darwin arm64

Angular: 19.0.0
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, platform-server
... router, ssr

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1900.0
@angular-devkit/build-angular   19.0.0
@angular-devkit/core            19.0.0
@angular-devkit/schematics      19.0.0
@schematics/angular             19.0.0
rxjs                            7.8.1
typescript                      5.6.3
zone.js                         0.15.0

Anything else relevant?
No response

This behavior is intended. The application will be bootstrapped during the build process to validate and extract all routes, generating an SSR manifest.
In the future, file-based routing may address this issue.
Currently, you can set the environment variable NG_BUILD_PARTIAL_SSR=1, but be aware that this will slow down the first request and may cause validation errors to appear at runtime instead of during the build process.

Hm, interesting, since the behaviour is different to the non-hybrid SSR, but I understand why it is necessary at the moment.
For my use case, I think I can safely use a mock value when inject(REQUEST) is null, which will happen during build time since I don't have pre-generated routes.

Or is there any possibility to detect build-time explicitly?

No there isn't a way

Or is there any possibility to detect build-time explicitly?

No there isn't a way

I've checked using the mock object, but that's not working for me.
So, I need to use the usual SSR approach for now.
You can close this issue if you want, since the original behavior is intended and does not fit my use case. :-)


Fix takeoff edge case

In the normal sitl commander takeoff case, the takeoff jump was never
actually carried out because the default altitude radius is set to 3m
and the takeoff altitude to ~2m which means that the takeoff waypoint is
"reached" immediately.
This works around this edge case by using the altitude between the home
altitude and takeoff altitude divided by 2 as a acceptance radius.
@bkueng, @LorenzMeier please review.

I need to dive deeper here. I've discovered more anomalies in real flight tests, the most appearant being that a takeoff commanded by QGC at 2m will lead to a climb all the way to 5m before it will slowly descend back to 2m.

Could be the design of the position controller. Can you share a flight log?

@LorenzMeier: That's what I thought but the bug was in the navigator, see 2bc1e1c.

Looks pretty good now. I would change MIS_TAKEOFF_ALT to 2-3 m for multicopters so we don't see this behaviour by default.

I would change MIS_TAKEOFF_ALT to 2-3 m for multicopters so we don't see this behaviour by default.

It's set for tap:
https://github.com/PX4/Firmware/blob/7c0a567eaa9b69bbc10d2505503cd75d4231664b/ROMFS/tap_common/init.d/rc.mc_defaults#L13
SITL,
https://github.com/PX4/Firmware/blob/7c0a567eaa9b69bbc10d2505503cd75d4231664b/posix-configs/SITL/init/lpe/iris#L33
and fixedwing:
https://github.com/PX4/Firmware/blob/7c0a567eaa9b69bbc10d2505503cd75d4231664b/ROMFS/px4fmu_common/init.d/rc.fw_defaults#L19
We should probably just set the default to 2.5, and have fixedwing change it, ok?

Sounds reasonable to set a low default and change for fixed wing defaults. Because fixed wing had a non-default value already set existing setups will keep the fixed wing default.

Ok, done.

@julianoes Try to initialize the integrator z value with the hover throttle value in the takeoff routine. I can send you lines after.

@tumbili Why? Where? That's already done correctly as far as I've seen it. And I have fixed the bugs I observed.

I've reviewed this, looks good to merge.

Semaphore failed on a Github pull (probably due to the DNS issues in the last days). Merged!


Feature: provide elementRef of node.

Describe any alternatives/workarounds you're currently using
We don't have tree-node elements in our html, therefore a hack was needed to get that reference. Workaround is to set nodeClass on each node and select it by classname later, but this feels hacky. Especially, when you see that in types declaration there is an elementRef and when you try to use it - it throws an error.
More detailed description of use-case here: https://github.com/CirclonGroup/angular-tree-component/issues/904.
If accepted, I would be willing to submit a PR for this feature
[ ] Yes (Assistance will be provided if you need help to submit a pull request)
[ X] No

The elementRef always throws an error because that's hardcoded in there - it should have been removed a long time ago. It was removed because of the virtual scroll and there would be a few problems adding it back and having virtual scroll.
That workaround with the class and selecting it later does sound hacky. But why are you not using the template options?

Thanks for the reply. We use treeNodeTemplate which is kind of enough for us. Adding more explicit template just to set some classnames or ViewChild references seems like an overhead here. So it would be nice to get a reference somehow by node id or similar.


Comment Count doesn't show until we open that Tweet

Checklist

[X] I am able to reproduce the bug with the latest version.
[X] I made sure that there are no existing issues - open or closed - which I could contribute my information to.
[X] I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
[X] This issue contains only one bug, that only occurs in Squawker

Affected version
v3.4.4+300012183
Steps to reproduce the bug

Open squawker normally or do a search.
You will see tweets with a comments icon showing no count.
Open any tweet which will have comments and you will see comment count.

Expected behavior
Expected to show comment count even without opening the tweet.
Actual behavior
Comment count shows up only when we open a tweet.
Screenshots/Screen recordings
This is without opening the tweet.

And this is after opening the tweet.

Logs
No response
Affected Android/Custom ROM version
No response
Affected device model
No response
Additional information
No response

Any solution or is there some setting I am missing?

It's a limitation of the Twitter/X API function that Squawker is currently using.
There is an alternate Twitter/X API function which returns the tweet count, but this other function is more restricted.
The Twitter/X API function currently used allows 180 requests per 15 minutes.
The alternative Twitter/X API function only allows 50 requests per 15 minutes.


Format my Bicep files

I ran bicep format on all the files in core. Lo and behold, only the files I contributed had formatting issues. So here's a PR with my files reformatted.
For reference, I used this script:
#!/bin/bash

# Set the command to run on each file
command="az bicep format -f"

find . -type f -name "*.bicep" | while read file
do
  $command "$file"
done

However, it doesn't quite work as it doesn't insert a final new line. There's a --insert-final-newline option that would theoretically fix that, but it's broken right now, so I filed issue on the CLI.
Also of interest, here's a Github workflow that runs Bicep format and auto commits changes:
https://github.com/pamelafox/django-quiz-app/pull/15/files#diff-8af3e80c405f1ab691b04ee13deecae46a34d6e87cc81b9a5f21490bc17e2609R29

@danieljurek - Thoughts on getting a check like this in CI?

Opened https://github.com/Azure/azure-dev/issues/1997 to track adding this to CI


Get largest/smallest favicon from multi-resolution .ico

We should add a special value to the &page= parameter (for e.g. &page=-1 and &page=-2) to get the biggest/smallest page, because I think that a stand-alone parameter will cause confusion.
Originally posted by @kleisauke in https://github.com/weserv/images/issues/154#issuecomment-443534463

Any changes?..

No updates yet, we're busy with our infrastructure to keep things running smoothly, new features are delayed because of it..

This has been implemented in API version 5 (#189). See:
https://images.weserv.nl/docs/format.html#page
Please let me know if you run into any issues.

Closing due to inactivity. Let us know if there is anything we can do to assist.


Fix rabbitmq exchange declare

Description
Fix rabbitmq exchange declare misconfiguration
Issue reference
#2764
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Please create a new test case for this PR here: https://github.com/dapr/components-contrib/tree/master/tests/certification/pubsub/rabbitmq
And update unit tests


Option to increase/bypass limits in the TransactionalBatch operation

Is your feature request related to a problem? Please describe.
We're experiencing a scenario where there is either >100 items in the batch to be upserted or >2MB.
Describe the solution you'd like
These limits are described to ensure predictability and to meet SLAs, but for the scenarios in which I need to do the larger batch operation - I'd like to bypass the limits and accept the cost rather than doing one of the alternatives below.
Describe alternatives you've considered

Used a stored proc https://github.com/Azure/azure-cosmos-dotnet-v3/issues/1057
Split batch/use bulk. This is less than ideal because we want atomicity https://github.com/Azure/azure-cosmos-dotnet-v3/issues/1057 https://stackoverflow.com/questions/75181418/how-to-store-more-than-100-records-using-cosmos-batch-from-azure-cosmos-db-using

Additional context

@aaronzhongg Thanks for your feedback, but this is a Service limit. SDKs cannot increase over the service define limits as they are enforced by the service -> https://learn.microsoft.com/en-us/azure/cosmos-db/concepts-limits#per-request-limits
There is no change that can be done from the client that can affect the service limit.
Please raise your Service request or provide service feedback: https://feedback.azure.com/d365community/forum/3002b3be-0d25-ec11-b6e6-000d3a4f0858


variables as default arguments for mixins

i think it's currently impossible to do this:

   @mixin anything($color: $default-color-dark) {
  background-color: $color;
}

It gives me an error on the include.
I don't see too much reason for default values, when they don't accept other variables.
Am I missing something?

i didn't figure this out either - anything changed since july?
if not i would appreciate this feature as well ;)


dbeaver/pro#34731 Fix dollar quotes handling on execution from the cursor position

Please test Vertica, Snowflake and PostgreSQL queries with $$. I noticed that Snowflake has some bug on settings switching or initial state initialization, so, if something won't work for it, please compare with older version of DBeaver first and report a new bug.
Also please test the setting defining the way, how we handle/highlight double-dollar parts for these dialects.

verified


Can not file extracted npy file

In file UBI_splitting.ipynb code line:
xmap = np.memmap(ROOT+'/x.npy',dtype=dtype, shape=shape_x, mode=mode)
Error code:
FileNotFoundError: [Errno 2] No such file or directory: '/content/drive/My Drive/Datasets/UBI_FIGHTS/extracted/x.npy'
Then I back to file UBI_Extraction.ipynb can't file code generate x.npy file, only y.npy.
Could you give me more detail the code base?

In order to extract all videos and labels in the UBI dataset, you will need to call the function packets_from_paths from the Extractor class and pass the suitable parameters.

I called packets_from_paths before my issue with code
ext.packets_from_paths(video_paths, annot_paths, OUT)
Result is:
FileNotFoundError: [Errno 2] No such file or directory: '/content/drive/My Drive/Datasets/UBI_FIGHTS/extracted/x.npy'
Since x.npy never created but it need to loading numpy file before saving?

Solution: create folder extracted by hand before run code 🤣


Expose "workbench.editor.pinnedTabSizing" as context menu for pinned tabs

workbench.editor.pinnedTabSizing is not very discoverable.
If it would be a submenu of the context menu, it would be much easier to tweak this setting.

🤔 ... 👎

I'm for making these types of settings more discoverable, however, I do not want to add these types of settings to the contextmenu. We mainly use the context menu to change the visibility of elements/features but we don't really use it to configure them/change appearance. Also, the context menu is already pretty full and I don't want to overload it.

I'm open to other ideas to make them more discoverable. Maybe a "configure" entry in the overflow menu of the toolbar?

Yeah I like the idea of an action in the "..." menu to configure all things editor related by narrowing down a settings search in the settings editor.

verification steps:

Right click on the tab bar (not on a tab)
At the bottom of the context menu there is an action "Configure Tabs...", press it
Settings editor should open with tabs related settings


refactor(starknet_sequencer_node): rename component execution mode

Stack:

#2570
#2569
#2568
#2567
#2566
#2565
#2564 ⬅

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is


[EventHubs] Migrate to track2 SDK

Description

Migrate to Track2 changes and Added require_infrastructure_encryption parameter to namespace update command
Testing Guide

History Notes

[eventhubs 1] az eventhubs namespace update : added --enable-require-infrastructure-encryption for encryption.

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

EventHubs

Hi @evelyn-ys , the failing test are from SQL and Iot hub which uses Eventhub, can you please help me on it?

Hi @evelyn-ys Yishi Wang FTE , the failing test are from SQL and Iot hub which uses Eventhub, can you please help me on it?

Hi @v-Ajnava, You have to rerun all eventhubs related tests even if it's from other modules. Use azdev test the_specific_test_name --live and commit the yaml changes~

Hi @evelyn-ys , all CI passed. can  you please review ?

Hi @kairu-ms, can you please review the PR ? I have added Private endpoint commands

The history notes incorrectly uses single quotes ' rather than backticks `:
History Notes

[EventHub] 'az eventhubs namespace update' : Add --infra-encryption for encryption (enable-require-infrastructure-encryption).
[EventHub] 'az eventhubs namespace create/update' : Add --disable-local-auth to enable or disable SAS authentication.
[EventHub] Added new private-endpoint-connection and private-link-resource commands for namespace


self compiled version doesn´t start (win32/64)

Build (under Ubuntu/wine) seems okay:
><EMAIL_ADDRESS>package:windows /home/user/electron-mattermost
> gulp package:windows

[23:01:59] Using gulpfile ~/electron-mattermost/gulpfile.js
[23:01:59] Starting 'sync-meta'...
[23:01:59] Finished 'sync-meta' after 2.94 ms
[23:01:59] Starting 'build:jsx'...
[23:02:00] Finished 'build:jsx' after 1.7 s
[23:02:00] Starting 'build'...
[23:02:00] Finished 'build' after 14 μs
[23:02:00] Starting 'package:windows'...
Packaging app for platform win32 ia32 using electron v0.36.3
Packaging app for platform win32 x64 using electron v0.36.3
[23:02:24] Finished 'package:windows' after 24 s

Starting electron-mattermost.exe leads to
Uncaught exception, Cannot find module yargs
(see http://imgur.com/IFbIZaw)

Could you tell me more info?

commit id
node and npm version
Does npm start work?
Is yargs installed into node_modules(src and release) correctly?

Then, I can't test the app for a while. Please wait.

I tried to build on a console-only VM, so was wasn´t able to start. If that´s likely to be part of the problem, I would try on another machine.
markus@ubuntu:~$ npm -v
2.14.15
markus@ubuntu:~$ node -v
v0.10.41
markus@ubuntu:~/electron-mattermost$ npm ls yargs
<EMAIL_ADDRESS>/home/markus/electron-mattermost
└──<EMAIL_ADDRESS>extraneous
commit 1fdc7691d787f7a224fa7f6b1075a35ecace3d27

Could you try npm install with node 5.x(newer npm) ? Electron v0.36 integrates node 5.1.1, it's used instead of your local node.js.

I tried my luck with node 5.5.0 and npm 3.3.12 - no luck. I can live with the pre-built release version, so no big problem for me.

Just for note:
I tested with cleaned repo.
Env: ubuntu 14.04 64bit, node 5.5.0, npm 3.3.12
-> Windows 10 x64 can start the app.
Besides, I'm using Docker container(debian and wine) for pre-built release version.


New package: Samsung.EasySoftwareManager version <IP_ADDRESS>

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

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

Automatic Validation ended with:

Automatic Validation ended with:

Appears to be an interactive only installer


Backpack sign message incorrect format

When I sign message with backpack adapter I get back the object { signature: Uint8Array }
For all other wallets (tested phantom, solflare) I get signature itself (Uint8Array ) without being wrapped in object
Small issue, but hope it should be same format :)

we're following this standard here: https://github.com/anza-xyz/wallet-standard/blob/master/packages/wallets/ghost/src/window.ts#L25


Couldn't cast array of type fixed_size_list to Sequence(Value(float64))

Describe the bug
I have a dataset with an embedding column, when I try to map that dataset I get the following exception:
Traceback (most recent call last):
  File "/Users/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/arrow_dataset.py", line 3189, in map
    for rank, done, content in iflatmap_unordered(
  File "/Users/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/utils/py_utils.py", line 1387, in iflatmap_unordered
    [async_result.get(timeout=0.05) for async_result in async_results]
  File "/Users/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/utils/py_utils.py", line 1387, in <listcomp>
    [async_result.get(timeout=0.05) for async_result in async_results]
  File "/Users/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/multiprocess/pool.py", line 774, in get
    raise self._value
TypeError: Couldn't cast array of type
fixed_size_list<item: float>[2]
to
Sequence(feature=Value(dtype='float32', id=None), length=2, id=None)

Steps to reproduce the bug
Here's a simple repro script:
from datasets import Features, Value, Sequence, ClassLabel, Dataset

dataset_features = Features({
    'text': Value('string'),
    'embedding': Sequence(Value('double'), length=2),
    'categories': Sequence(ClassLabel(names=sorted([
        'one',
        'two',
        'three'
    ]))),
})

dataset = Dataset.from_dict(
    {
        'text': ['A'] * 10000,
        'embedding': [[0.0, 0.1]] * 10000,
        'categories': [[0]] * 10000,
    },
    features=dataset_features
)

def test_mapper(r):
    r['text'] = list(map(lambda t: t + ' b', r['text']))
    return r

dataset = dataset.map(test_mapper, batched=True, batch_size=10, features=dataset_features, num_proc=2)

Removing the embedding column fixes the issue!
Expected behavior
The mapping completes successfully.
Environment info

datasets version: 2.14.4
Platform: macOS-14.0-arm64-arm-64bit
Python version: 3.10.12
Huggingface_hub version: 0.17.1
PyArrow version: 13.0.0
Pandas version: 2.0.3

Thanks for reporting! I've opened a PR with a fix.

Thanks for the quick response @mariosasko!  I just installed your branch via poetry add 'git+https://github.com/huggingface/datasets#fix-array_values' and I can confirm it works on the example provided.
Follow up question for you, should Nones be supported in these types of features as they are in others?
For example, the following script:
from datasets import Features, Value, Sequence, ClassLabel, Dataset

dataset_features = Features({
    'text': Value('string'),
    'embedding': Sequence(Value('double'), length=2),
    'categories': Sequence(ClassLabel(names=sorted([
        'one',
        'two',
        'three'
    ]))),
})

dataset = Dataset.from_dict(
    {
        'text': ['A'] * 10000,
        "embedding": [None] * 10000, # THIS LINE CHANGED
        'categories': [[0]] * 10000,
    },
    features=dataset_features
)

def test_mapper(r):
    r['text'] = list(map(lambda t: t + ' b', r['text']))
    return r

dataset = dataset.map(test_mapper, batched=True, batch_size=10, features=dataset_features, num_proc=2)

fails with
Traceback (most recent call last):
  File "/home/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/multiprocess/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/home/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/utils/py_utils.py", line 1354, in _write_generator_to_queue
    for i, result in enumerate(func(**kwargs)):
  File "/home/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/arrow_dataset.py", line 3493, in _map_single
    writer.write_batch(batch)
  File "/home/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/arrow_writer.py", line 549, in write_batch
    array = cast_array_to_feature(col_values, col_type) if col_type is not None else col_values
  File "/home/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/table.py", line 1831, in wrapper
    return pa.chunked_array([func(chunk, *args, **kwargs) for chunk in array.chunks])
  File "/home/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/table.py", line 1831, in <listcomp>
    return pa.chunked_array([func(chunk, *args, **kwargs) for chunk in array.chunks])
  File "/home/jmif/.virtualenvs/llm-training/lib/python3.10/site-packages/datasets/table.py", line 2160, in cast_array_to_feature
    raise TypeError(f"Couldn't cast array of type\n{array.type}\nto\n{feature}")
TypeError: Couldn't cast array of type
fixed_size_list<item: double>[2]
to
Sequence(feature=Value(dtype='float64', id=None), length=2, id=None)

Ideally we can have empty embedding columns as well!

This part of PyArrow is buggy and inconsistent regarding features implemented across the types, so the only option is to operate on the Arrow buffer level to fix issues such as the above one.


chrono::datetime::DateTime::format causes a panic at dioxus-core-0.4.1/src/scopes.rs:440:37

Problem
When calling chrono::datetime::DateTime::format while using  dioxus-fullstack, I get the following panic
thread 'tokio-runtime-worker' panicked at 'called `Result::unwrap()` on an `Err` value: Error', /home/vmenge/.cargo/registry/src/index.crates.io-6f17d22bba15001f/dioxus-core-0.4.1/src/scopes.rs:440:37
thread 'tokio-runtime-worker' panicked at 'not yet implemented: generally, scopes should be sync, only if being traversed', /home/vmenge/.cargo/registry/src/index.crates.io-6f17d22bba15001f/dioxus-ssr-0.4.0/src/renderer.rs:112:38
// a bunch more

Steps To Reproduce
Follow Getting Started / Fullstack on the 0.4 learn page, then Guide / Your First Component to the end.
I can reproduce with just
fn main() {
    LaunchBuilder::new(Break).launch();
}

fn Break(cx: Scope) -> Element {
    let time = Utc::now().format("%D %1:%M %p");

    render! {
        "{time}"
    }
}

However while this gives me an error on scopes.rs:440, it does not give me the one on renderer.rs:112 as if following the guide.
My Cargo.toml
[dependencies]
dioxus = "*"
dioxus-fullstack = "*"
chrono = { version = "0.4.26", features = ["serde"] }
serde = "1.0.183"

[features]
default = []
ssr = ["dioxus-fullstack/axum"]
web = ["dioxus-fullstack/web"]

Expected behavior
DateTime should've been formatted into a string :D
Screenshots

Environment:

dioxus version: 0.4.0
dioxus-fullstack version: 0.4.1
Rust version: 1.71.1 (eb26296b5 2023-08-03)
OS info: Ubuntu 22.04.2 LTS on Windows 11 x86_64
App platform: web/ fullstack

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

This was just a typo in the copied code. it should be l not 1


https

HTTP and HTTPS are different, why the same demo can run under the HTTP, but not in the HTTPS

@gkiagia

Sorry, but you need to be a bit more specific. Which demo are you talking about?

@RFCBox could you provide some more info please?


gender neutral language

change "he encounters" to "they encounter"

@emberstevens : Thanks for your contribution! The author, @ganganarayanan, has been notified to review your proposed change.

Placeholder name "Joe" isn't gender-neutral, so updated gender-neutral pronoun does not correspond. Please replace with non-gender specific placeholder.

Updated in https://github.com/MicrosoftDocs/azure-docs/pull/31652; closing.


How to find XPath in Gmail? - some tips

Ken
What a fantastic tool you created for the world here. I love it and am using it to do a password reset through Google mail.
Do you have an example showing how to do it ? Gmail is hard since you cannot easily inspect the row with the email.
Any help appreciated !

Hi @jamesnyika
There is actually a way to get the xpath of the rows of email on gmail. While you cannot right click and directly inspect the rows of email, you can right click the top portion of gmail where you can inspect, then slowly expand the elements in the pane and find the element you are looking for by checking against what is highlighted on the UI when you expand and hover it in the pane.
Here's an example of the absolute xpath I got of the first email
/html/body/div[7]/div[3]/div/div[2]/div[1]/div[2]/div/div/div/div/div[2]/div/div[1]/div/div[1]/div[8]/div/div[1]/div[3]/div/table/tbody/tr[1]

Hi @ruthtxh ...I actually was able to use an XPath extension to get a similar path. But notice that it simply points to a TR element. I am guessing that Google is probably using javascript to submit any click events on this element rather than an anchor tag. So my question is: if you run a click on this : does it actually go to the email ? have you had success with this ? because I think I tried and could not get it to work (particularly after a search for emails matching a specific piece of text)

Hi @jamesnyika yes, clicking that XPath in TagUI works for me. What error message do you get?

Hi @ruthtxh - I need to try it out again : I feel I have had more experience in the last few days.. let me see if I can get it working and report back.

Closing this issue for now due to inactivity, feel free to reopen.


Can you write a short tutorial on how to use these components in our own apps?

I'm submitting a ...  (check one with "x")
[ ] bug report
[ ] feature request
[x] support request
Howdy, I just started using Ionic and I have no idea how to use components in my app.
Specifically, I'm interested in the accordion component.

I second this. Currently trying to import the accordion list into my app but get the error can't bind property expanded to accordion-list. I can tell what the error is getting at, but not experienced enough with the framework to find the solution.


Unable to Build WSLg

Steps to reproduce

Follow the build instructions in Contributing.md, except run sudo dockerd --iptables=false as per #8450.
(Running regular sudo dockerd throws the same errors listed in the report.)
Run the first command of step 2.2

Expected behavior
WSLg to build
Actual behavior
Instead, it throws this error:
== Install Git/CA certificates ==
curl#6: Couldn't resolve host name
Error: TDNFGetRepoMD 1207
Error: Failed to synchronize cache for repo 'CBL-Mariner Official Base 1.0 x86_64' from 'https://packages.microsoft.com/cbl-mariner/1.0/prod/base/x86_64/rpms'
curl#6: Couldn't resolve host name
Error: TDNFGetRepoMD 1207
Error: Failed to synchronize cache for repo 'CBL-Mariner Official Microsoft 1.0 x86_64' from 'https://packages.microsoft.com/cbl-mariner/1.0/prod/Microsoft/x86_64/rpms'
curl#6: Couldn't resolve host name
Error: TDNFGetRepoMD 1207
Error: Failed to synchronize cache for repo 'CBL-Mariner Official Update 1.0 x86_64' from 'https://packages.microsoft.com/cbl-mariner/1.0/prod/update/x86_64/rpms'
Error(1011) : No matching packages
Loaded plugin: tdnfrepogpgcheck
Refreshing metadata for: 'CBL-Mariner Official Base 1.0 x86_64'
Disabling Repo: 'CBL-Mariner Official Base 1.0 x86_64'
Refreshing metadata for: 'CBL-Mariner Official Microsoft 1.0 x86_64'
Disabling Repo: 'CBL-Mariner Official Microsoft 1.0 x86_64'
Refreshing metadata for: 'CBL-Mariner Official Update 1.0 x86_64'
Disabling Repo: 'CBL-Mariner Official Update 1.0 x86_64'
No package git available
No package ca-certificates available
The command '/bin/sh -c echo "== Install Git/CA certificates ==" &&     tdnf install -y         git         ca-certificates' returned a non-zero code: 243

@Zopolis4 when using WSL add --network=host to your docker build command, like:
sudo docker build --network=host  -t system-distro-x64  ./wslg  --build-arg SYSTEMDISTRO_VERSION=git --git-dir=wslg/.git rev-parse --verify HEAD --build-arg SYSTEMDISTRO_ARCH=x86_64
Docker network defaults to bridge and for some unknow (to me) reason it doesn't work inside WSL.
Here's sample build output, just the beginning, past your failure point:
sudo docker build --network=host  -t system-distro-x64  ./wslg  --build-arg SYSTEMDISTRO_VERSION=git --git-dir=wslg/.git rev-parse --verify HEAD --build-arg SYSTEMDISTRO_ARCH=x86_64
Sending build context to Docker daemon  942.3MB
Step 1/82 : FROM mcr.microsoft.com/cbl-mariner/base/core:1.0.20220226 AS build-env
---> 1c60c61ce2f6
Step 2/82 : RUN echo "== Install Git/CA certificates ==" &&     tdnf install -y         git         ca-certificates
---> Running in ef9a1248c053
== Install Git/CA certificates ==
Loaded plugin: tdnfrepogpgcheck
Refreshing metadata for: 'CBL-Mariner Official Base 1.0 x86_64'
Refreshing metadata for: 'CBL-Mariner Official Microsoft 1.0 x86_64'
Refreshing metadata for: 'CBL-Mariner Official Update 1.0 x86_64'

Yup, that fixed it. Should this be mentioned on the build documentation somewhere? The --iptables=false option for dockerd and the --network=host option for the docker build command? It seems a little annoying for someone with a default setup to have to find two separate issues on two separate repos.


Revert "Remove locking from DBs and rely on SQLite's locking (#211)"

This reverts commit 7d0881309a7ef9bd072fceaa2ee854463be03566.
Suspected commit in Jenkins Failures.

I've tried this on wrw-igemm-v4r4xdlops-fp32-fix branch, commit ad859f55c35140192c909bd683bd8e247b50981f (run #7 on Jenkins). Didn't help ((

@daniellowell  Did we find the root cause ?


Revert "feat / update to dev-1.1.0"

Reverts hummingbot/hummingbot#5086

I believe I have to sync development with staging before running the update of tag to version 1.1.0 as I did because I have not sync 1.0.0 to staging

this revert the version number to 1.0.0

@rapcmia please review whether needs to revert before going to staging


🎁 Contribute back from HykuUp Knapsack

These files are what we changed in HykuUp Knapsack to get the build, test, and lint workflows working.  Also, updating the Hyku submodule to bring in the Valkyrie work.

This pipeline is failing because Knapsack's gemfile.lock has blacklight at 6.25. the submodule is 7.29.


Added additional toggles

Added options to toggle the default fields “class” and “logger_name” on
on/off

Hi Philip,
I do not you response fully, in my pull request i added the option to disable logging the class and thread properties from the json logmessage instead of enabling it.
I added these options because i used the slf4j-json-logger for logging specific states of SOA composites and Fault states from a Fault Policy. The class and thread properties are irrelevant when logging these faults.
I use the slf4j-json-logger in my java class because the default loggers for Oracle SOA products are only capable of logging in a proprietary format which is hard to parse by Splunk where JSON format works very well. Although it does not hurt if the class and thread are logged, less is more when logging in high volumes.

On 2 Oct 2016, at 17:46, Philip A. Chapman<EMAIL_ADDRESS>wrote:
@pachapman approved this pull request.
Changes look good, though I'm curious as to why you need to show class and thread for what is essentially a utility for a testing framework.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/savoirtech/slf4j-json-logger/pull/2#pullrequestreview-2450814, or mute the thread https://github.com/notifications/unsubscribe-auth/AHJvnCQXiFdRfVRkUxedYjXPxxWB_9x7ks5qv9HkgaJpZM4KMANK.

Met vriendelijke groet/With kind regards
Peter van Nes
Mobile +31 652 694 792 | Fax +31 84 747 6999
http://twitter.com/petervannes http://nl.linkedin.com/in/petervannes http://www.reddipped.com/ skype:peter_van_nes?call<EMAIL_ADDRESS><EMAIL_ADDRESS>The Future Group http://thefuturegroup.com/, Dé Maat in ondernemen.

This change looks good.  Thank you for the contribution.
One thing to address with this update - since this changes some of the public API in a non-backward compatible way, we need to bump the major version.  I'm referring specifically to the constructor argument change.
It would be great if we could make this interface less brittle to future changes.  Now would be a good time to do so since we're going to need a new major version here anyway.
Perhaps using setters instead of constructor arguments?  Or, is there some way we could tie into the SLF4J configuration system to get custom configuration here?

Why  not just add constructors with default values that mimic the original functionality?  That way we do not lose backward compatibility.

Yeah, just default it to showing the information and let you use the other constructor to turn it off.  +1

I used to be more in-line with adding more code for backward compatibility.
These days, though, I tend to look more heavily at the cost of that tech debt, maintaining that added complexity with no real gain in value, and which can make the code harder-to-use due to the non-obvious method variations.
In this case, I have little concern with bumping the major version.  But, I would like to apply the "Use Before Reuse" concept here - this is the second time we're doing the same (changing the constructor arguments and introducing a breaking change), so it's a good time to solve the reusability in a way that reduces the need for breaking, major version changes.
Another consideration - a setter-based approach is more "Real Obvious" - making the code easier to use, and easier to maintain since we avoid constructor-variation-explosion as subclasses as created over time with their own additional constructor arguments.  It's also more obvious since the setter names are clear, as opposed to matching constructor arguments by-position.
Finally, when I think of Logging and configuration of what's included in the log output, the good-'ol log4j configuration files come to mind.  Having the application hard-code these settings in the long run may not be ideal - especially for anyone using this code in common (i.e. heavily-reused) libraries.  Hence the desire to latch onto an existing Logging platform means of configuration.

It was not my intention to make a breaking change, so reverted the constructor and added new attributes with getters/setters.
Maybe it is an good idea to remove the includeLoggerName from the constructor in the next major release and replace it with a getter/setter.
Additionally i changed the class name of the JUnit test BasicLoggingTests.java to BasicLoggingTest.java. In Netbeans only JUnit test classes matching the mask
*Test.java are fired.

If you're going to evolve the API and you want to protect yourself from changes to things like constructors, consider using the "builder" pattern instead.

I'm going to merge this PR and then introduce the builder pattern, as suggested by James, and finally release a new major version.

One thing I noticed while working on the builder pattern that I hadn't noticed before - the addition of getters and setters to the JsonLogger interface.
@petervannes - is there a real need to have these methods in the interface, outside of the library itself?
Please let me know as I am in the process of removing them and don't want to miss a need.

Note that I put up a new Pull Request with the builder pattern.  Feedback would be appreciated.


Add pandas to env

I believe all of us need pandas, just added it to the environment.yml
@dorisyycai @meretelutz @jbarns14, may I have one of you quickly review, approve and merge please?

oh, just found that Doris has already created the same thing, it's closed


chore: move task logic to cli

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

#346
#345  👈
#343
#342 : 1 other dependent PR (#344 )
#341
#340
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


felix-fm: 2.13.0 -> 2.14.0

Diff: https://github.com/kyoheiu/felix/compare/refs/tags/v2.13.0...v2.14.0
Changelog: https://github.com/kyoheiu/felix/blob/v2.14.0/CHANGELOG.md
Fix build (https://hydra.nixos.org/build/273418736)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 348355 run on x86_64-linux 1

1 package built:

felix-fm


feat: add WithProtoHelper option

Close: https://github.com/ent/ent/issues/3508
@rotemtam please review

Please see comment , also - please add a unit test verifying this behavior?
Thanks

I don't find a previously existing test, where should I add the test file?
Should I add unit tests for individual functions, or should I design an integration test to check if the file is generated?

Unit test has been added. plz review

hey @Zxilly  CI is red, can you check?


realm::LogicError crash

I'm receiving the following crash "sometimes".  Using 0.97.0
Fatal Exception: realm::LogicError: Wrong transactional state (no active transaction, wrong type of transaction, or transaction already in progress)
Thread : Crashed: com.twitter.crashlytics.ios.exception
0  PipeTracker                    0x70d1d7 CLSProcessRecordAllThreads + 170146
1  PipeTracker                    0x70d1d7 CLSProcessRecordAllThreads + 170146
2  PipeTracker                    0x70d3e5 CLSProcessRecordAllThreads + 170672
3  PipeTracker                    0x701f5b CLSHandler + 124454
4  PipeTracker                    0x70c059 __CLSExceptionRecord_block_invoke + 165668
5  libdispatch.dylib              0x236bbdc3 _dispatch_client_callout + 22
6  libdispatch.dylib              0x236c5055 _dispatch_barrier_sync_f_invoke + 56
7  PipeTracker                    0x70bb5f CLSExceptionRecord + 164394
8  PipeTracker                    0x70b5a3 CLSTerminateHandler() + 162926
9  libc++abi.dylib                0x232dce17 std::__terminate(void (*)()) + 78
10 libc++abi.dylib                0x232dcf3b std::terminate() + 190
11 PipeTracker                    0x56391d realm::ArrayIntNull::~ArrayIntNull() (array_integer.hpp:297)
12 PipeTracker                    0x6628a3 realm::SharedGroup::end_read() + 7098
13 PipeTracker                    0x5a6eb5 realm::Realm::invalidate() (shared_realm.cpp:333)
14 PipeTracker                    0x59468f -[RLMRealm invalidate] (RLMRealm.mm:503)
15 PipeTracker                    0x424cc5 -[PendingDataOperation checkForPendingTransfersInContext:] (PendingDataOperation.m:405)
16 PipeTracker                    0x423fd7 -[PendingDataOperation start] (PendingDataOperation.m:147)
17 Foundation                     0x243247ed __NSOQSchedule_f + 192
18 libdispatch.dylib              0x236c5f97 _dispatch_queue_drain + 1762
19 libdispatch.dylib              0x236bef2f _dispatch_queue_invoke + 282
20 libdispatch.dylib              0x236c7325 _dispatch_root_queue_drain + 400
21 libdispatch.dylib              0x236c7193 _dispatch_worker_thread3 + 94
22 libsystem_pthread.dylib        0x23854e0d _pthread_wqthread + 1024
23 libsystem_pthread.dylib        0x238549fc start_wqthread + 8

I'm running the code in an NSOperation to process network updates to a back-end database.  I fetch the list of PendingDataModel items from its realm and then perform an invalidate at the end of processing to release the read lock.  The PendingDataModel, along with some other large datasets, are in a separate realm file and I use a class method "objectRealm" to retrieve the realm file
RLMResults *networkArray = [PendingDataModel allPendingItems];
RLMRealm *pendingRealm = networkArray.realm;
for(PendingDataModel *network in networkArray) {
   ...cycle through pending items...
}
[pendingRealm invalidate];

+(RLMResults *)allPendingItems {
    RLMResults *tmp = [[PendingDataModel allObjectsInRealm:[PendingDataModel objectRealm]] sortedResultsUsingProperty:@"timeStampInterval" ascending:YES];
    return tmp;
}

+(RLMRealm *)objectRealm {

    RLMRealmConfiguration *config = [RLMRealmConfiguration defaultConfiguration];
    config.objectClasses = @[PendingDataModel.class,PendingDataDependencyModel.class];
    config.path = [[[config.path stringByDeletingLastPathComponent]
                    stringByAppendingPathComponent:@"pendingData"]
                   stringByAppendingPathExtension:@"realm"];
    config.schemaVersion = 1;

    config.migrationBlock = ^(RLMMigration *migration, uint64_t oldSchemaVersion) {
        if (oldSchemaVersion < 1) {
        }
    };

    RLMRealm *realm = [RLMRealm realmWithConfiguration:config error:nil];

    return realm;

}

Thanks for reporting this! Nothing in your code jumps out at me as being incorrect or a particularly good candidate for causing this crash. I've shared what you've shared with our core team to see if they can make anything of this.
If you manage to find a way to reliably reproduce this, please send it our way, as it's the best way to ensure we can identify what's happening and hopefully fix this.

Hi,
is there any news about that? I'm getting the same crash occasionally but I can't track down errors on my code.
It seems to happen more frequently in a slower device (iPhone 5C) than in more powerful devices, but I managed to get the same crash on all of them.
In my case the code is too complex to be reported here but it happens when I run a lot of parallels read and write operations on the database, although there aren't write transactions nested in other write transactions.

We recently made some changes in Realm's core which might pave to way to fixing this, although it's difficult to say for sure since we don't have a reliable reproduction case on which to test this fix to confirm.
We'd be grateful if @wolfAle or @chrism1974 could provide us with such a sample. It would help move the fix faster along the review process.

I downloaded yesterday the last version of Realm because I noticed that the improvements could have fixed this problem, but unfortunately I could reproduce the crash even with the last version :(
I'll try to create a test project to replicate the crash and help you track it down

Actually with the new version of Realm the crash is more trackable: it's throwing an error on line 958 of group_shared.hpp -> commit_and_continue_as_read method when it calls do_commit().
The error seems to be not handled and so all the subsequent calls get the wrong m_transact_stage (that should be updated ad the end of commit_and_continue_as_read).
Unfortunately I couldn't manage to reproduce the same crash on a test project.
Hope this can help to identify the issue.

I think #3251 will fix things for when do_commit() throws as that leaves the Realm in the write transaction when it fails, but there's a few other places in commit_and_continue_as_read() that can throw and will probably still not be correct.

I've verified that #3251 fixes incorrect reporting of inWriteTransaction when do_commit() throws (and by extension, errors when calling invalidate), but do_commit() throwing still means that your write transaction isn't being successfully committed. I don't see anything in it that could throw other than a memory allocation failure or running out of disk space. If you're modifying a very large number of objects, splitting the write transaction up into smaller chunks (of say, 10k items each) would help with that.

So, finally I managed to get a report of the real error:
Error Domain=io.realm Code=1 "mmap() failed: Cannot allocate memory size: 46137344 offset: 0" UserInfo={NSLocalizedDescription=mmap() failed: Cannot allocate memory size: 46137344 offset: 0, Error Code=1}
For some reason is failing allocating the memory.
The thing is that I'm modifying a lot of objects, but far less than 10k items. And I'm splitting them in chunks anyway so I don't expect that to be the problem. In my case I think that the problem is that I'm doing write/read operations on multiple threads at the same time, this seems to cause problems.
I can work on UI level to avoid to many refreshes to mitigate the problem, but that is not a fix.
About Realm: I found odd that when a do_commit() fails Realm is left in a write state. In case of failure the error is thrown and on production code level we can do something about that (either try again or ignore the error) but the state on the database should allow us to write again in the db.

Hi guys,
any news about that? I'm still struggling to fix this crash on my code, but I need advice from your side.
A few things I noticed:

the db size is increasing a lot each time I write data on it. Like when I create data for the first time the db is about 500kb, after a few pull to refresh operation it grows until more than 30mb, a couple of time around 100mb. I've implemented the writeCopyToPath on the app's startup and this mitigate the problem, but it's not a fix.
the app throws the first error trying to create an instance of Realm on a background thread, always for that memory reason (Cannot allocate memory size). From there on it crashes because of the wrong state on the database. And it seems to happen only when the DB size is high.

Any hints?

I too am now getting a crash at do_commit() but only if the app is backgrounded by locking the device.
This is new for me post 0.98ish.  Sorry I don't have the specific version.
I had similar crashes with a background lock before and resolved that by setting NSFileProtectionKey: NSFileProtectionNone against the entire Realm path.
Not sure if they are somehow related.  Looking for any suggestions.  Thanks much.

Curious - I was removing file protection just once at App launch which solved a background lock write issue from a while ago, now- if I remove the protection every time before trying to write to Realm, the crash goes away.
Calling this before a write solves it for me:
NSString *pathToRealmFile = [RLMRealmConfiguration defaultConfiguration].path;     NSArray *allRealmRelatedFiles = @[                                       pathToRealmFile,                                       [pathToRealmFile<EMAIL_ADDRESS>                                      [pathToRealmFile<EMAIL_ADDRESS>                                      [pathToRealmFile<EMAIL_ADDRESS>                                      [pathToRealmFile<EMAIL_ADDRESS>                                     ];     for (NSString *path in allRealmRelatedFiles) {         NSError *error = nil;         //         [[NSFileManager defaultManager] setAttributes:@{NSFileProtectionKey: NSFileProtectionNone} ofItemAtPath:path error:&error];         if (error) {             NSLog(@"Error clearing protection on %@", path);         }     }

I am getting this same error when using 98.6 with Swift.
I am retrieving about 1,500 objects from an API (split over 15 'pages', i.e. 100 objects per page). Each page is written to my Realm DB in a background thread. 80% of the time it crashes at do.commit() as for the users above.
Everything works fine around ~500 objects on an actual device. Everything works ok in the simulator even up to 2,000 objects. (I haven't tested higher)

@avitus could you share a sample project that can reproduce this issue?

@jpsim Let me see what I can do ... might be a few days.

Oddly enough I can't seem to recreate the crash at the moment. I will let you know if it reappears.

Alright please do! It's clear that there's a bug somewhere in here, but it seems elusive for anyone to reproduce.
If anyone can gain more insight into how this may be happening, please share that here. We take issues like this seriously, but our ability to assess and address is severely limited until it can be reproduced.
And just to confirm, is everyone in this thread getting an uncaught exception with the following message?

Fatal Exception: realm::LogicError: Wrong transactional state (no active transaction, wrong type of transaction, or transaction already in progress)

If you're getting a different error, I encourage you to file a new issue.

Has anyone either seen this with recent versions of Realm? Or been able to find steps to reproduce?

A lot of fixes was done leading up to 1.0. We can't be sure that those has fixed the issues you have all seen, but since we can't get anyone to reproduce this, we will close it for now. Please do create a new issue in case you experience new issues with 1.x.y.


JOSS: Software > General

Concerning the JOSS review thread.

Remove Empty Files: Files such as .github/workflow are empty and should be removed.
Clean Up Redundant Directories: The build folder appears to be a partial copy of src. If it is unnecessary, it should be removed.
Improve Documentation: Not all functions are documented with docstrings. Comprehensive documentation would enhance usability.
Eliminate Unnecessary Comments: The code contains many comments that may not be needed, for example, in tests/results_systems.jl. Removing them can improve code readability.
Simplify Build Scripts: The docs/make.jl file contains unnecessary code, such as run(cp -r "C:/Users/belli/.julia/dev/QuantizedSystemSolver/docs/build/*"). This should be cleaned up.
Remove Unnecessary Files:

lcov.info
src/dense/NL_integrators/workspace.code-workspace
Please review the repository thoroughly to remove files that are not relevant to users.

Standardize Directory Naming: Some folders within src use uppercase letters while others use lowercase. Consistent naming conventions should be applied.

redundant and unnecessary folders and files are removed
unnecessary comments inside files are removed
files and folders are renamed to follow the camelCase style
all functions are now documented with docstrings


구글 애널리틱스 추가

추가 했습니다.

Hello, I'm a student who made a "pull request" above.
It's just that I'm not good at using GitHub, so I made a full request for your repertoire. I'm sorry.
From next time on, I will pay more attention to the "pull request" and prevent damage to you.
I'm really sorry.
(There might be some weird parts using a translator, so I'd appreciate it if you understood.)
Thank you.


auto brevent option is missing

Updated Note8 to Android Pie and had to reinstall my apps. Reinstalled Brevent and noticed that I do not have the option to disable auto Brevent anymore. It use to be the first option in the list.

default is on.
tap "support status" 7 times to show the options.


Postgresql: multiple instances, OpenSSL, flexible configuration

I've opened an issue here with a proposal of mkMultiInstance and also a problem: https://github.com/NixOS/nixpkgs/issues/6784

(triage) @ip1981 @lethalman is there some news ?

Nope. Not interested anymore. https://github.com/zalora/nixsap/tree/master/modules/apps/postgresql


Atomizing cf-tables

Atomizing cf-tables
Additions

Added src/cf-tables/src/cf-table-row-links.js as a modifier for the base table organism. This also can be used as a standalone module if you don't wan't the overhead of an atomic component.

Added src/cf-tables/src/cf-table-sortable.js as a modifier for the base table organism. This also replaces the old cf-table functionality.

Changes

Modified src/cf-tables/src/cf-tables.js to inherit from AtomicComponent.
Modified src/cf-tables/usage.md to add documentation for row-links and add the appropriate table organism prefix.
Modified src/cf-tables/src/cf-tables.less to use the appropriate table organism prefix.  Added the row-links modifier class.

Testing

Run npm install && gulp test

Review

@cfpb/cfgov-frontends

Notes

I made many  AtomicComponent design decisions based based on my personal preferences. Will adjust as the group sees fit.

Todos

Write more tests.
Release version 1.0 of AtomicComponent. Reference this version in cf-tables package.json.

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

This looks good. The biggest thing is just updating the changelog and de-updating the package.json, but can you also remove the cf-tables tests from qunit since these changes will definitely break them.

Super minor comment, but 👍 from me


Initial commit fixes

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e81f2ed). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   91.29%
  Complexity        ?      126
=========================================
  Files             ?       11
  Lines             ?      402
  Branches          ?        0
=========================================
  Hits              ?      367
  Misses            ?       35
  Partials          ?        0

Impacted Files
Coverage Δ
Complexity Δ

src/Traits/RequestTrait.php
56.81% <100%> (ø)
14 <0> (?)

Continue to review full report at Codecov.

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


Python: Bug:

Team,
While trying to use Redis Enterprise for a demo preparation (https://github.com/microsoft/semantic-kernel/tree/main/python/semantic_kernel/connectors/memory/redis),
with SK 1.1.2 Python SDK, I get the below error:
AttributeError: 'Kernel' object has no attribute 'use_memory'
It seems that use_memory function is available only on SK dev versions but we need SK GA versions supportability.
Could you please look into and advise.
Thank you,
Florin

resolved by importing as TextMemoryPlugin plugin

resolved by importing as TextMemoryPlugin plugin


Update Julia_Notepad++.xml to Julia 1.7

Hi there! I guess I'm not the only one who is still using Npp to write Julia. As a new LTS version of Julia is out, I took some time to update this.
This time I listed all names in Julia 1.7 Core and Base, including constants, types, keywords, and functions. I also added supports for number prefixes (like 0x123abc) and multi-part keywords (like abstract type).

I don't use NPP anymore for Julia but will merge

Thanks @woct0rdho for your contribution!


Update muddy-water.json

Fixed typo for malware name "POWERSTATS", it was missing an s

Resolved in https://github.com/pan-unit42/playbook_viewer/commit/3749ab5eeafd3b1111e4cd870ab6fb56ffdb573f
Thanks for the bug report. I did not merge this pull request in order to keep this repository in sync with other systems.


Code action not working even though command is invoked

Type: Bug
Even though this problem happens with our Python related extensions, I don't think it's something related to them control as the commands seem to be triggered properly 🤔
Repro steps:

Install the pre-release versions of our Python, Pylint and isort extensions
Create a new Python file with import os, sys and save it (e.g. hello.py)
When all the extensions activate, you will see a few warnings over that line and quick fixes:

Select Pylint: Run organize imports

Nothing happens. Now if you run the Organize Imports command from the command palette, it works as expected. The problem is the Pylint extension seems to be triggering the right command, if you open the Developer Tools it shows the right command was triggered:

These are the logs for when the organize imports command is run:

VS Code version: Code - Insiders 1.76.0-insider (25514d899f4bd877803defb213719f8af3d997d3, 2023-02-07T05:22:19.428Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz (8 x 2995)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
31.71GB (15.09GB free)

Process Argv
--user-data-dir nonexisting3232 --extensions-dir nonexisting3232/extfolder --crash-reporter-id 11494669-52ca-4f2c-aa0e-29172189cc8e

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

isort
ms-
2023.9.10371012

pylint
ms-
2023.1.10381730

python
ms-
2023.3.10381854

vscode-pylance
ms-
2023.2.13

jupyter
ms-
2023.2.1000391021

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.14

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

cpptools
ms-
1.14.1

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30422396
pythontb:30258533
vsc_aa:30263845
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805:30301674
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30404738
cppdebug:30492333
vsclangdf:30492506
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30618038
pylantcb52:30590116
pyindex848:30611229
nodejswelcome1:30587009
pyind779:30611226
pythonsymbol12:30651887
2i9eh265:30646982
showlangstatbar:30657381

Minimal repo using code action provide samples:
/*---------------------------------------------------------
 * Copyright (C) Microsoft Corporation. All rights reserved.
 *--------------------------------------------------------*/

import * as vscode from 'vscode';

const COMMAND = 'code-actions-sample.command';

export function activate(context: vscode.ExtensionContext) {

	const emojiDiagnostics = vscode.languages.createDiagnosticCollection("emoji");
	context.subscriptions.push(emojiDiagnostics);

	context.subscriptions.push(
		vscode.languages.registerCodeActionsProvider('markdown', new ActionProvider(), {
			providedCodeActionKinds: ActionProvider.providedCodeActionKinds
		}),
		vscode.languages.registerCodeActionsProvider('markdown', new OrganizeImportsProvider(), {
			providedCodeActionKinds: OrganizeImportsProvider.providedCodeActionKinds
		})
	);

	context.subscriptions.push(
		vscode.commands.registerCommand(COMMAND, () => vscode.env.openExternal(vscode.Uri.parse('https://unicode.org/emoji/charts-12.0/full-emoji-list.html')))
	);
}

/**
 * Provides code actions for converting :) to a smiley emoji.
 */
class ActionProvider implements vscode.CodeActionProvider {

	static providedCodeActionKinds = [
		vscode.CodeActionKind.QuickFix
	];

	provideCodeActions(document: vscode.TextDocument, range: vscode.Range | vscode.Selection, context: vscode.CodeActionContext, token: vscode.CancellationToken): vscode.ProviderResult<(vscode.CodeAction | vscode.Command)[]> {

		const action = new vscode.CodeAction('Indirectly organize imports', vscode.CodeActionKind.QuickFix);
		action.command = { command: 'editor.action.organizeImports', title: 'Learn more about emojis', tooltip: 'This will open the unicode emoji page.' };
		return [action];
	}
}

class OrganizeImportsProvider implements vscode.CodeActionProvider {

	static providedCodeActionKinds = [
		vscode.CodeActionKind.SourceOrganizeImports
	];

	async provideCodeActions(document: vscode.TextDocument, range: vscode.Range | vscode.Selection, context: vscode.CodeActionContext, token: vscode.CancellationToken) {
		console.log('provideCodeActions organize imports' , token.isCancellationRequested);

		token.onCancellationRequested(() => {
			console.log('cancelled');
		});

		await new Promise(resolve => setTimeout(resolve, 10));

		const action = new vscode.CodeAction('Source organize', vscode.CodeActionKind.SourceOrganizeImports);
		action.command = { command: COMMAND, title: 'Learn more about emojis', tooltip: 'This will open the unicode emoji page.' };
		return [action];
	}
}

Any updates on this one? Still facing this issue.

@shubhamjain0594 sorry, will give this another look this coming iteration since the work around the trigger is slightly related to some other work done this iteration.

Just ran into this a well.

had the same issue on "Run organize imports" but it seems not fixed yet.


Bump framework dependency

Bump framework dependency. This version of the framework was released with https://github.com/wso2/carbon-identity-framework/pull/5165 which passed the integration test run https://github.com/wso2/carbon-identity-framework/pull/5165#pullrequestreview-1733869321

Closing the PR as the version has been bumped by another PR


Region Timeline - Enhancement

Hi, I suggest a "Region Timeline" option in the ruler.
I made a picture of what it could be.

It would it be very convenient to me as I render through Region Render matrix, and I ask people to give feedback on theses rendered files. They usualy give me feedbacks like "a trouble at 2:57'" or somethink like that. Of course they use the timeline of their own player, that start at the begining of the rendered file...
So I have to listen the rendered file, on a player, and return tho Reaper to find by ear where to mark the project...

An hack to wait the function ;-)
https://www.facebook.com/groups/reaperusergroup/permalink/10156881816774751/

You could perhaps use media items: REAPER has added per-item rulers in v5.982.

Thanks, I did. It's helpfull.
I created a empty track on the top, made a Selection with a double click on the Region and created an empty Item, that have the exact lenght of the Region.

Easy to mark the feedback...

Another thing you could do is use set the ruler's 0:00 point to the start of the region. You could then type the exact relative timestamp in the edit marker dialog.


Unable to install v0.32.0 (karpenter.sh/v1alpha5 api service missing)

Description
Observed Behavior:
Pod is not starting due to missing api service (v1alpha5):
$ k logs -f karpenter-7bfffc9bd6-2nllq
{"level":"DEBUG","time":"2023-10-31T13:14:04.193Z","logger":"controller","message":"loaded log configuration from file \"/etc/karpenter/logging/zap-logger-config\"","commit":"3a61217"}
{"level":"DEBUG","time":"2023-10-31T13:14:04.194Z","logger":"controller","message":"discovered karpenter version","commit":"3a61217","version":"v0.32.0"}
panic: failed to setup machine provider id indexer: failed to get API group resources: unable to retrieve the complete list of server APIs: karpenter.sh/v1alpha5: the server could not find the requested resource

goroutine 1 [running]:
github.com/samber/lo.must({0x25adc00, 0xc00017bf40}, {0xc000997730, 0x1, 0x1})
        github.com/samber/lo@v1.38.1/errors.go:51 +0x1c5
github.com/samber/lo.Must0(...)
        github.com/samber/lo@v1.38.1/errors.go:72
github.com/aws/karpenter-core/pkg/operator.NewOperator()
        github.com/aws/karpenter-core@v0.32.0/pkg/operator/operator.go:182 +0x16ce
main.main()
        github.com/aws/karpenter/cmd/controller/main.go:33 +0x25

I found that the version that is installed is v1beta1:
$ k get apiservices.apiregistration.k8s.io |grep karpenter
v1beta1.karpenter.k8s.aws              Local     True        15s
v1beta1.karpenter.sh                   Local     True        15s

Expected Behavior:
It should work. Right now pod is in the crashloop.
Reproduction Steps (Please include YAML):
helm upgrade --install karpenter ./karpenter --namespace karpenter  --set settings.clusterName=${CLUSTER_NAME}     --set serviceAccount.annotations."eks\.amazonaws\.com/role-arn"="arn:${AWS_PARTITION}:iam::${AWS_ACCOUNT_ID}:role/KarpenterControllerRole-${CLUSTER_NAME}"     --set controller.resources.requests.cpu=1     --set controller.resources.requests.memory=1Gi     --set controller.resources.limits.cpu=1     --set controller.resources.limits.memory=1Gi

Versions:

Chart Version: v0.32.0
Kubernetes Version (kubectl version): v1.24.17-eks-f8587cb

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

I have just one version (v1beta1). I installed it using helm. Is helm release broken?

Hello, folks.
I was upgrading karpenter in two of our clusters v0.30.0 -> v0.31.2 -> v0.32.1. First upgrade went smoothly. In second cluster, though, I was getting similar error but it was about v1beta1.karpenter.sh.
I was able to fix this by manually upgrading CRDs, as described here.
helm upgrade --install karpenter-crd oci://public.ecr.aws/karpenter/karpenter-crd --version v0.32.1 --namespace karpenter

Can this behavior be related to following note in the docs?
As part of the helm chart [karpenter](https://gallery.ecr.aws/karpenter/karpenter) - [source](https://github.com/aws/karpenter/blob/main/charts/karpenter/crds). Helm [does not manage the lifecycle of CRDs using this method](https://helm.sh/docs/chart_best_practices/custom_resource_definitions/), the tool will only install the CRD during the first installation of the helm chart. Subsequent chart upgrades will not add or remove CRDs, even if the CRDs have changed. When CRDs are changed, we will make a note in the version’s upgrade guide.

It seems the same problem I encountered today. I currently solved installing the v0.31 CRDs (aws-node-templates, provisioners, machines) and then doing a rollout restart deploy/karpenter
as mentioned here #792

Is helm release broken

The default install of the helm chart should install both the alpha version and the beta version. My assumption is that you might run into issues with the helm install and it upgrading aspects of the CRD install if you don't go through the manual install process or use the karpenter-crd helm chart as described in the upgrade guide: https://karpenter.sh/docs/upgrading/upgrade-guide/#crd-upgrades

When I creates this issue I did a fresh install of Karpenter 0.32.0 for test reasons. I had to uninstall it and install version previous version, which worked well.

I had the same issue when I installed version v0.32.1
To solve the issue, I need to install all the CDRs manually from here: https://github.com/aws/karpenter/tree/main/pkg/apis/crds
BTW: the doc on how to upgrade from auto-scaler is not updated to the latest version
: https://karpenter.sh/docs/getting-started/migrating-from-cas/#set-node-affinity
This part does not exist anymore inside the Helm template

Closing since it looks like the original issue around CRD install was resolved.


Files are not deleted locally when they cease to exist on firebase (i.e. when admin deletes).

I've asked for help on the Flutter Discord.
The question here turns out to be: how do we keep the app properly synchronized with Firebase?

Completely refactoring the storage_helper class.

There doesn't appear to be any fast and/or cheap way to keep local storage properly synchronized with Firebase Storage. This Google Cloud guy basically explains what I've been doing already and that it takes a lot of calls to the Storage API.
Looks like this is going to be a case of best effort for the time being. Uhg. /rant


Chrome forces .dev domains to use SSL. Use .localhost instead

Issue Type

Documentation Report

Summary

As written in https://ma.ttias.be/chrome-force-dev-domains-https-via-preloaded-hsts/ Chrome has forced .dev and .foo domains to load SSL as Google has bought .dev. It has been suggested to change to .localhost
I write this as warning / suggestion to change default.config.yml to use .localhost instead of .dev in future versions as there is likely to be real .dev sites showing up in the wild as well as a the confusing part that our default http only drupalvm is now trying to load via ssl because of chrome.

FYI, this is chrome >=63

Thank you! This has been driving me nuts all day.
Turns out the default was changed from .dev to .test a few months ago,  #1427
Confirming that changing my projects to .test fixes the problem.

Good Call. Thanks for double checking my issue. Looks like the one I was testing was on 4.6.0. Marking as closed.


ROBOT report April 2021

I will now switch ROBOT report to fail on ERROR, so we get the additional layer of security. Here some errors we need to take care off:

[x] ERROR	duplicate_definition	MONDO:0100237	IAO:0000115	An instance of cutis laxa that is inherited.
[x] ERROR	duplicate_definition	MONDO:0100238	IAO:0000115	An instance of cutis laxa that is inherited.
[x] ERROR	deprecated_class_reference	blank node	owl:someValuesFrom	 http://purl.obolibrary.org/obo/MONDO_0019590
[x] ERROR	deprecated_class_reference	MONDO:0006872	rdfs:subClassOf	http://purl.obolibrary.org/obo/MONDO_0005902
[x] ERROR	deprecated_class_reference	MONDO:0017142	rdfs:subClassOf	http://purl.obolibrary.org/obo/MONDO_0019138
[x] ERROR	deprecated_class_reference	MONDO:0018907	owl:someValuesFrom	http://purl.obolibrary.org/obo/MONDO_0019590

See also #2910

ERROR deprecated_class_reference blank node owl:someValuesFrom http://purl.obolibrary.org/obo/MONDO_0019590

what does this mean?

ERROR deprecated_class_reference MONDO:0006872 rdfs:subClassOf http://purl.obolibrary.org/obo/MONDO_0005902

Not sure what this means? Is this still an issue? Maybe it has been fixed.

This can be closed, remaining items fixed in #2920

thank you!


Bump dev version to 2.6.1-SNAPSHOT

As part of the 2.6.0 release, we need to bump the dev version of the 2.6 branch to 2.6.1.

Successful build on JDK 11. JDK 8 build timed out, and several test failures on JDK 14.
Retest this please.

retest this please

The JDK 11 and 14 builds didn't seem to run (IIUC reported on other branches and PRs, too). However, none of these are really impacted by this change of the version number.


Timeouts during auto detection

I added an option to treat timeouts during the autodetection as a failed detection attempt, instead of aborting with an exception.
Adjustments

I added a boolean property AbortOnTimeout to the FtpAutoDetectConfig which is true by default.
I adjusted the ConnectModule to honor the flag and treat TimeoutExceptions accordingly.

Background
I use FluentFTP in a .NET 4.6.2 project and I had an issue connecting to a FileZilla server with FTPS enabled.
While my code works in .NET 5 and higher, I get a Timeout error with 4.6.2.
Investigating this, I determined that the Socket internally is getting the error 997 (IO PENDING), which should not necessarily be an aborting error, but is not handled my the .NET Framework.
FluentFTP is throwing a TimeoutException whenever the Socket is not connected after calling .Connect(), no matter what the real cause was.
Alternative Ideas
I do not understand why TimeoutExceptions should ever be treated as aborting errors during AutoDetect. In my understanding a timeout means that the configuration which has been tried did not work, but not that no configuration can work.
So I think a better solution would be to just remove the TimeoutException from the permanent errors, but I did not want to change the current behavior so drastically without knowing enough.

My understanding when I first wrote this, was that a timeout only occurs when the host/port is unreachable. Meaning the connection details itself are incorrect. And so we don't bother to waste any more time trying different protocols since the host/port refuses to be reached, ie. cannot be reached, and so it makes sense to hard abort.
On the contrary, can you think of any situation where a timeout occurs, but the host/port are correct?

Also is this specific to a certain .NET platform? Then we can automatically enable your fix for that platform and do away with the extra config.

If this is happening from async code, WSA_IO_PENDING should not "be turned into a FluentFTP" timeout exception" immediately. It should rather cause a wait for timeout seconds - as one would expect the async operation to actually complete (or not) and if this wait times out, only then would a timeout exception be issued. With such additional code, I surmise that your auto-connect would actually time-out as with .NET 5.
The other half of the discussion is as @robinrodricks stated. I dimly remember some issues in the past, where this logic was introduced because users were complaining about long wait times on auto-connect due to repeatedly timing out on the same host:port endpoint.

I dimly remember some issues in the past, where this logic was introduced because users were complaining about long wait times on auto-connect due to repeatedly timing out on the same host:port endpoint.

Yes something like this came up.
I found the commit where this "hard abort" was first made. https://github.com/robinrodricks/FluentFTP/commit/fdf9e55aaa1fd28bd0c256888b774c2b30d0d667
If anyone has time/energy you can look through it and see why it was done.
But I still agree with my original reasoning: "a timeout only occurs when the host/port is unreachable. Meaning the connection details itself are incorrect"

If this fix is wonderful for you and occurs on newer platforms AS WELL, then I don't mind merging it asap and releasing. But if its only for older platforms, we can consider making it the default behavior. Or add this flag only for older platforms. I'm not sure how to proceed here.

I think we might find a better solution which does not need a new setting. I think that would be preferable.I need to do some tests and get back to you on Sunday.On Nov 10, 2023 20:40, Robin Rodricks @.***> wrote:
If this fix is wonderful for you and occurs on newer platforms AS WELL, then I don't mind merging it asap and releasing. But if its only for older platforms, we can consider making it the default behavior. Or add this flag only for older platforms. I'm not sure how to proceed here.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Connection Exceptions (i.e. SYN ACK SYN/ACK failures)

a timeout only occurs when the host/port is unreachable. Meaning the connection details itself are incorrect...

Unclear statement: It is reachable but there is no answer.
and for the sake of completeness:
...or the server is overloaded and does not answer or the network connection to the server is down.
DNS and routing failures produce different exceptions.

But if you connect to host:21 and get timeout, why abort when maybe host:990 and SSL will work?

In this case a setting is probably better as users can configure it anyhow they want.

 Lets accept it now.


Decide: apollo vs express-graphql

appolo
https://dev-blog.apollodata.com/
https://github.com/apollostack/typed-graphql-client-example
express-graphql
http://graphql.org/graphql-js/running-an-express-graphql-server/

Decision
express-graphql
Rationale
I'm guessing that I'll learn more using a lighter framework.  My needs in this case aren't very sophisticated either.


fix: Original exception, Malformed UTF-8

When i execute command "E:\workspace\xxx>perl6 -I..\p6-app-installermaker-wix\lib ..\p6-app-installerm
aker-wix\bin\make-perl6-wix-installer msi.yml", i always get
perl .............................................. OK
nmake ............................................. OK
cl ................................................ OK
git ............................................... OK
WiX heat .......................................... OK
WiX candle ........................................ OK
WiX light ......................................... OK
Pre-checks completed

Generating product module XML ..................... OK
Tried to get the result of a broken Promise
  in sub MAIN at ..\p6-app-installermaker-wix\bin\make-perl6-wix-installer line
22
  in block <unit> at ..\p6-app-installermaker-wix\bin\make-perl6-wix-installer l
ine 59

Original exception:
    Malformed UTF-8

I find out, if i create target directory, i can get over it.

Sorry for the wrong pull request. It works sometimes. But sometimes don't work.
I will do more test. Maybe it's not the real reason.


overlayfs added to FsNames

- What I did
I added overlayfs to FsNames (I think you have missed it)
- How I did it
Only added a line
- How to verify it
Run your tests if you want, but the change is trivial
- Description for the changelog

overlayfs added to FsNames

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:p4l1ly/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

LGTM

Reasonable.. turns out the strings were initially only meant for "real" filesystems during the graphdriver bringup (so an error could report thing like overlay {type} not supported on underlying fs {type}), but the magic values are now also used for also testing mounts being of specific real and overlay types, so the magic list and string list got out of sync.
No problem to make them match, even though those strings aren't used today.
LGTM

LGTM

I wanted to use overlay2 driver on the overlayfs filesystem and I got the error message: overlay2 is not supported over
I thought that Docker has some problems with identifying the underlying filesystem, it took me a while of digging in the code to find out that it has no problem with it, it just does not support overlayfs as underlying filesystem.


Feature request: autocrafting

Would be great if you could add to the mod some autocrafting table, like https://github.com/SHsuperCM/SimplyCrafting , for example.
Or implement this as a "Crafting Node", block that connects to the network, has grid like in crafting table, in which you can layout the recipe, and crafts items as requested.

Nope, sorry. I don't think that fits the mod well. If you want autocrafting, just install one of the many mods that provides a great one.

Hah, well, that's the thing - there is none, and that's why I'm asking. :)
There's only 1 mod that provides something similar to autocrafting for 1.16.2, https://github.com/Lolcroc/ProjectLepton , but it's completely incompatible with prettypipes — it ejects the result by itself, and crafts only on receiving redstone impulse. All other autocrafters are either outdated (as SimplyCrafting, for example), or they're part of other big mod, like the one you used in your video.

That Autocrafting table looks pretty good to me! You'd just have to either connect it to a clock or create a setup where you use a comparator to detect when items are going through the pipe.

aaand that's already overcomplicates things way more than I'd like... Oh well, I guess I'll have to write one myself...

If you want a super simple item storage mod, you probably shouldn't have gone for Pretty Pipes :p

Well, it's not only storage, but automation, too. This is so far the best mod for that kind of things since Buildcraft and RedPower (Project Red) got outdated. I really love to see those items moving around through pipes over my head here and there. Much more enjoyment than soulless AE...


go-flutter-desktop example not working

Hello,
i wanted to try the go-flutter-desktop branch. However when i make hover run it's not working because it says:
go<EMAIL_ADDRESS>parsing Users\erick\projects\go-flutter-desktop\plugins\audioplayers\go.mod: open C:\Users\username\Documents\Projects\flutter_audioplayers\example\go\Users\erick\projects\go-flutter-desktop\plugins\audioplayers\go.mod: Das System kann den angegebenen Pfad nicht finden.
(System cannot find path).
This is probably because in go.mod there is:
replace github.com/go-flutter-desktop/plugins/audioplayers => /Users/erick/projects/go-flutter-desktop/plugins/audioplayers
Where do i find the plugin directory withing this github repository?
Can you maybe help me @erickzanardo as you commited the branch?

I think this was a WIP from @erickzanardo and @raonifn. Not sure what is the current status.
(btw macOS is supported natively on master for a few versions now, but we are very open to other implementations to support all desktop platforms).

Ahh too bad ... since there's no plugin to play audio files on windows desktop. My c++ knowledge is not good enought to do this (i'm still stuck in trying to send data to a usb device).
Maybe when they bring C# as a second option to flutter desktop i will have a look into it.

@erickzanardo @raonifn any updates in this? should we :axe: the project in favour of the new official flutter desktop embeddings?

@luanpotter yes, we never finished that example and I don't think we plan on continuing, I will remove the branch.


Add excludeFromProject option for local packages

I'm trying to avoid xcodeproj management as much as possible and I'd prefer to inject my packages implicitly using xcworkspace, so they don't even show up in xcodeproj.

Description
Old
New

Reference in files (removed)

Reference in packages (removed)

Reference in dependencies (unchanged)

I added a new flag excludeFromProject to SwiftPackage config, it is optional and defaults to false (old behaviour), setting it to true applies changes accordingly to screenshots I provided above. This change will help to keep project a bit cleaner when packages are injected using xcworkspace.

Build errors look completely unrelated to changes introduced by this PR 🌚


put metric action

for given expression evaluate and publish per resource metric for given attribute, or the collection's aggregate attributes (size, etc).

Are you envisioning something like this:
policies:
  - name: track-bucket-size
    resource: s3
    comment: |
      Put the sum of the size of all keys in an S3 bucket into CloudWatch Metrics
    filter:
      - Name: metric-tracked-bucket
    actions:
      - type: put-metric
        key: Contents[].Size
        metric_name: Bucket Size
        op: sum

Or something like this:
  - name: track-attached-ebs
    resource: ec2
    comment: |
      Put the count of the number of EBS attached disks to an instance
    filter:
      - Name: tracked-ec2-instance
    actions:
      - type: put-metric
        key: Reservations[].Instances[].BlockDeviceMappings[].DeviceName
        metric_name: Attached Disks
        op: count

The action can figure out based on the resource, operation and the key handed to it what to put into CloudWatch.
Operations could be:

count
sum
average
duration (is this even possible?)

That looks good to me, thanks

Count, Sum and Average are definitely doable as Operations.  Duration is really a question of the units for the value being recorded.     I think the right thing to do here is to add one more key to the action calls 'units' and accept values of
`
durations
'Seconds','Microseconds','Milliseconds',

bytes
'Bytes','Kilobytes','Megabytes','Gigabytes','Terabytes',

bits
'Bits','Kilobits','Megabits','Gigabits','Terabits',

rates
'Bytes/Second','Kilobytes/Second','Megabytes/Second','Gigabytes/Second','Terabytes/Second'

'Bits/Second','Kilobits/Second','Megabits/Second','Gigabits/Second','Terabits/Second'

'Count/Second',

other
'Percent',
'Count',
'None'

`
This offers a way to interpret the values that graph as lines or area charts, or as values in a table.
Cloud watch can also record statistical values at a point in time.    In that case, rather a value it accepts  statistical metadata about a sample set ( min, max, sum, and sample count ).   I've not used that yet, but we could support that style as an additional op value called 'stats'.


Association extractor with includes option

#160
Problem

I dunno if this issue is valid but here's the definitions I have:
# order_group.rb
class OrderGroup < ApplicationRecord
  has_many :orders
end

# order.rb
class Order < ApplicationRecord
  belongs_to :order_group
  has_many :order_lines
  has_many :fees
end

# blueprints

# order_group_blueprint.rb
view :extended do
  association :orders, blueprint: OrderBlueprint, view: :extended
end

# order_blueprint.rb
view :extended do
  association :order_lines, blueprint: OrderLineBlueprint
  association :fees, blueprint: FeeBlueprint
end

# some_controller.rb
...
OrderGroupBlueprint.render_as_hash(OrderManifest.all, view: :extended)
...

Now, how do I eager load the fees & order_lines when using the blueprint OrderGroupBlueprint?
Feel free to close the issue if this is invalid :) Thank you very much

Old solution
# order_group_blueprint.rb
view :extended do
  association :orders, blueprint: OrderBlueprint, view: :extended do |order_group|
    order_group.orders.includes(:fees, :order_lines)
  end
end

New solution
# order_group_blueprint.rb
view :extended do
  association :orders, blueprint: OrderBlueprint, view: :extended, includes: [:fees, :order_lines]
end

@ThanhKhoaIT why this is closed?

Why is this closed @ThanhKhoaIT ?


Added deprecated warning in CobaltProvider line

Description
Added a DEPRECATED warning infront of CobaltProvider line in Execution Provider document i.e. https://parsl.readthedocs.io/en/stable/userguide/execution.html#execution-providers>
Changed Behaviour
It will not change any code flow
Fixes
Fixes # (3143)
Type of change
Choose which options apply, and delete the ones which do not apply.

Documentation

PR #3145 added this deprecation line already, so we don't need to merge this PR as well. So I'll close this PR.


SSL Verification Error on pd.read_csv under runvSWIM function

Running vSWIM will sometimes return a SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000).
Occurs in run.vSWIM after installation into virtual environment. Reported on MacOS 13.6.7 with Python 3.12.

This is not a vSWIM issue but is (likely) a conflict of Python's base installation (e.g. https://stackoverflow.com/questions/50236117/scraping-ssl-certificate-verify-failed-error-for-http-en-wikipedia-org).
A temporary working solution if you run into this issue is to directly download the data files and change the path to a the location of the data file to avoid reading a URL.


bugfix: check for nil before executing callback for JobSpawn

If a program started with shell.JobSpawn terminates and no callback is set, then micro crashes. For example, if I call the Lua function
function jobspawn(bp)
    shell.JobSpawn("xclock", {}, nil, nil, nil)
end

and close the xclock window, then I get ((on Ubuntu, with master) :
Micro encountered an error: runtime.errorString runtime error: invalid memory address or nil pointer dereference
runtime/panic.go:220 (0x44af36)
runtime/panic.go:219 (0x44af06)
github.com/zyedidia/micro/v2/cmd/micro/micro.go:423 (0x8de2b5)
github.com/zyedidia/micro/v2/cmd/micro/micro.go:400 (0x8dde97)
runtime/proc.go:250 (0x4373d2)
runtime/asm_amd64.s:1571 (0x465f21)

If you can reproduce this error, please report it at https://github.com/zyedidia/micro/issues

This PR fixes this bug.

PR updated according to @dmaluka's suggestion


Bump okhttp3 to 4.10.0, play-services-analytics to 18.0.1, and gson to 2.9.0 (close #520)

Updates dependencies in tracker:

com.squareup.okhttp3:okhttp from 4.9.1 to 4.10.0
com.google.android.gms:play-services-analytics from 17.0.1 to 18.0.1

In demo app:

com.squareup.okhttp3:okhttp from 4.9.1 to 4.10.0
com.google.code.gson:gson from 2.8.9 to 2.9.0

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


Make records as they are work in the arrays encoding

[x] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] ~Documentation added for any new functionality~
[ ] ~Entry added to UNRELEASED.md for any new functionality~

This PR fixes three errors that were blocking the use of records in the arrays encoding, they are detailed below. Closes #1288.

DomainRuleWithArrays was computing funEx twice due to call to super. This was solved by moving the remaining cases of apply from DomainRule to DomainRuleWithArrays.
As pointed out by @konnov, when equating sets or functions we need to generate equalities for their elements using LazyEquality. This was solved by calling cacheEqConstraints with the sets'/functions' elements. For this fix EqRuleWithArrays was deleted and its logic moved to LazyEquality, making that class less lazy than before :-).
Picking a record via CherryPick.pickRecord was generating unsatisfiable formulas in some cases. The problem was that when constructing the domain of the picked record we could lose the chain of SSA updates, due to a storeNotInSet operation being in the rhs of an implication. This was solved by changing the operation to not(selectInSet), which ensures absence from the set without updating the array, given that sets are initially empty.

Codecov Report

Merging #1419 (c0c71b8) into unstable (412858a) will decrease coverage by 0.00%.
The diff coverage is 84.21%.

:exclamation: Current head c0c71b8 differs from pull request most recent head dce272d. Consider uploading reports for the commit dce272d to get more accurate results

@@             Coverage Diff              @@
##           unstable    #1419      +/-   ##
============================================
- Coverage     76.09%   76.09%   -0.01%
============================================
  Files           339      338       -1
  Lines         10985    10962      -23
  Branches        542      535       -7
============================================
- Hits           8359     8341      -18
+ Misses         2626     2621       -5

Impacted Files
Coverage Δ

.../forsyte/apalache/tla/bmcmt/rules/DomainRule.scala
94.00% <ø> (ø)

...palache/tla/bmcmt/rules/DomainRuleWithArrays.scala
75.00% <60.00%> (ø)

...a/at/forsyte/apalache/tla/bmcmt/LazyEquality.scala
73.52% <87.09%> (+0.61%)
:arrow_up:

...he/tla/bmcmt/SymbStateRewriterImplWithArrays.scala
100.00% <100.00%> (ø)

...syte/apalache/tla/bmcmt/rules/aux/CherryPick.scala
93.75% <100.00%> (+0.34%)
:arrow_up:

...he/io/annotations/parser/CommentPreprocessor.scala
92.77% <0.00%> (-1.21%)
:arrow_down:

.../forsyte/apalache/tla/bmcmt/SymbStateDecoder.scala
80.00% <0.00%> (+7.00%)
:arrow_up:

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 412858a...dce272d. Read the comment docs.

The integration tests caught a bug that surfaced due to the changes made for in point 2 (see PR description). When handling  function application we were comparing cells directly, which is problematic in the presence of quantifiers, e.g., \A x \in S . f(x). The fix I made involved using part of the logic in FunAppRule.applyFun, using an oracle to avoid the direct cell comparison. In contrast to FunAppRule however, the constraints generated are for the domain and not the range. See the latest commit for these changes.
The integration tests are now passing, but the fix introduced a major efficiency loss. In #1169, the integration tests check for the arrays encoding took 4min, now it takes 1h. This should be addressed before merging the PR.

The slowdown seen before was due to having to use an oracle to ascertain the argument of the function application. I made a fix in which, if the argument can be found in Scala by comparing cells, we generate a single SMT constraint querying the array that encodes the function. If we can't, e.g., in the presence of quantifiers, we use an oracle and need to have O(N) SMT constraints, with N being the cardinality of the domain of the function.
This change made the integration tests for the arrays encoding go back to their previous runtime, observed in #1169. Shall we merge the PR @konnov?