Configure gradle build scan plugin

This small PR configures the build-scan Gradle plugin (https://guides.gradle.org/creating-build-scans/) which yields further insight into the build, allowing the team to make informed decisions on where and when the build may be tweaked to get faster, more performant builds.

Coverage remained the same at 95.077% when pulling 306acf062aadf32939a5c5e98548955e37338969 on aalmiray:gradle-build-scan into cc38445aa0d369cdbb6e6570b75c63abf559d044 on kt3k:master.

Thank you 😄


feat: Upgrade V2 driver to csi-resizer v1.4.0 & add auto disk size detection to NodeProvisioner

What type of PR is this?
/kind feature

What this PR does / why we need it:
Updates csi-resizer to v1.4.0 to match v1 driver and moves responsibility for re-reading the disk partition table after resize to NodeProvisioner.
Which issue(s) this PR fixes:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.523%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
230

Relevant Lines:
474

💛  - Coveralls


allow self referencing

I don't think this does what you'd expect -- as exports should point at dist -- sub-path-imports is generally what you'd want during dev
https://nodejs.org/api/packages.html#subpath-imports


Can't use any() to verify StackTrace

Bug Description
In this method, I want to catch the exception and the stack trace. I got into trouble when I want to verify the broadcastPort.notifyCoreEvent(). This is the method I want to test:
real_method.dart:
@override
Future<CategoryDto> addCategory(CategoryDto category) async {
  try {
    Category(category).validate(insert: true);
    final result = await persistencePort.createCategory(category);

    broadcastPort?.notifyCoreEvent(
      CoreEvent.categoryCreatedEvent,
      request: category,
      response: result,
    );

    return result;
  } catch (e, s) {
    broadcastPort?.notifyCoreEvent(
      CoreEvent.exceptionThrownEvent,
      exception: e,
      stackTrace: s,
    );

    rethrow;
  }
}

This is how I tested it:
real_method_test.dart:
class FakeCoreDomainException extends Fake implements CoreDomainException {}
class FakeStackTrace extends Fake implements StackTrace {}

// Test's main starts here...

setUpAll(() {
  registerFallbackValue(CoreEvent.exceptionThrownEvent);
  registerFallbackValue(FakeCoreDomainException());
  registerFallbackValue(FakeStackTrace());
});

// Some statements here...

test('Insert invalid category and broadcast port is not omitted, insertion failed', () async {
  final exception =
      CoreDomainException(CoreDomainExceptionType.invalidCategoryTitle);

  when(() => mockPersistencePort.createCategory(dummyCategoryDto))
      .thenThrow(exception);

  expect(
    () => coreService.addCategory(dummyCategoryDto),
    throwsA(
      exception,
    ),
  );

  verify(() => mockPersistencePort.createCategory(dummyCategoryDto))
      .called(1);

  verify(
    () => mockBroadcastPort.notifyCoreEvent(
      CoreEvent.exceptionThrownEvent,
      exception: exception,
      stackTrace: any(),
    ),
  ).called(1);
});

But I got this error:

 Invalid argument(s): An argument matcher (like `any()`) was either not used as an immediate argument to Symbol("notifyCoreEvent") (argument matchers can only be used as an argument for the very method being stubbed or verified), or was used as a named argument without the Mocktail "named" API (Each argument matcher that is used as a named argument needs to specify the name of the argument it is being used in. For example: `when(() => obj.fn(x: any(named: "x")))`).

I tried to change stackTrace: any() tostackTrace: any(that: isA<StackTrace>()), but still got this error.
By the way, If I omit the stack trace on the real method call, the test will be passed.
modified_real_method.dart:
@override
Future<CategoryDto> addCategory(CategoryDto category) async {
  try {

  // Some statements here...

  } catch (e) {
    broadcastPort?.notifyCoreEvent(
      CoreEvent.exceptionThrownEvent,
      exception: e,
    );

    rethrow;
  }
}

modified_real_method_test.dart:
setUpAll(() {
  registerFallbackValue(CoreEvent.exceptionThrownEvent);
});

// Some statements here...

test('Insert invalid category and broadcast port is not omitted, insertion failed', () async {
  final exception =
      CoreDomainException(CoreDomainExceptionType.invalidCategoryTitle);

  when(() => mockPersistencePort.createCategory(dummyCategoryDto))
      .thenThrow(exception);

  expect(
    () => coreService.addCategory(dummyCategoryDto),
    throwsA(
      exception,
    ),
  );

  verify(() => mockPersistencePort.createCategory(dummyCategoryDto))
      .called(1);

  verify(
    () => mockBroadcastPort.notifyCoreEvent(
      CoreEvent.exceptionThrownEvent,
      exception: exception,
    ),
  ).called(1);
});

Expected behavior
stackTrace: any() should be enough to make real_method_test.dart passed. Because I have another similar use case for verifying mockNavigatorObserver.didPush() and it is working.
class FakeRoute extends Fake implements Route {}

// Test's main starts here...

testWidget('Some navigation test', () {
  setUpAll(() {
    registerFallbackValue(FakeRoute());
  }

  // Some statements here...

  await tester.pumpWidget(
    MultiRepositoryProvider(
      providers: [
        RepositoryProvider<BaseLocationService>.value(
          value: mockLocationService,
        ),
      ],
      child: MultiBlocProvider(
        providers: [
          BlocProvider<LocationCubit>(
            create: (_) => mockLocationCubit,
          ),
        ],
        child: MaterialApp(
          onGenerateRoute: _appRouter.onGenerateRoute,
          navigatorObservers: [mockNavigatorObserver],
          home: const ActivateLocationScreen(),
        ),
      ),
    ),
  );

  // Some statements here...

  verify(() => mockNavigatorObserver.didPush(any(), any()));
});

Environment
Dart: 2.18.4 (My project is a dart package)
Mocktail: 0.3.0
Additional context
I am thinking that maybe we can provide an optional StackTrace parameter on the thenThrow()

Change
stackTrace: any(),
to
stackTrace: any(named: "stackTrace")
All named parameters with any() need to have this

Change stackTrace: any(), to stackTrace: any(named: "stackTrace") All named parameters with any() need to have this

Hi @thide11, thanks for the response. Let me check it first


[BUG] test case test_setting_priority_class failed in master and v1.5.x

Describe the bug (🐛 if you encounter this issue)
In Longhorn master and v1.5.x, setting an non-existent priority class to setting priority will get 404 error make test script fail. And  the error message with same action is not the same with v1.4.x.
Daily regression test result
To Reproduce

Perform test case test_setting_priority_class
Or setting a non-existent priority class into setting priority-class

Expected behavior
Have the same error message make test case have consistent result
Log or Support bundle
master and v1.5.x error message when set a non-exist priority class
(ApiError(...), "404 : \n{'message': '', 'code': 404}")

v1.4.x error message when set a non-exist priority class
(ApiError(...), '500 : failed to set the setting priority-class with invalid value priority-class-piqd54: failed to get priority class priority-class-piqd54 before modifying priority class setting: priorityclass.scheduling.k8s.io "priority-class-piqd54" not found\n{\'code\': 500, \'detail\': \'\', \'message\': \'failed to set the setting priority-class with invalid value priority-class-piqd54: failed to get priority class priority-class-piqd54 before modifying priority class setting: priorityclass.scheduling.k8s.io "priority-class-piqd54" not found\', \'status\': 500}')

longhorn-manager log in master
time="2023-07-14T01:54:20Z" level=warning msg="HTTP handling error" error="failed to set the setting priority-class with invalid value notexistpriorityclass1: failed to get priority class notexistpriorityclass1 before modifying priority class setting: priorityclass.scheduling.k8s.io \"notexistpriorityclass1\" not found"
<IP_ADDRESS> - - [14/Jul/2023:01:54:20 +0000] "PUT /v1/settings/priority-class HTTP/1.1" 404 0 "https://<IP_ADDRESS>/" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"

Environment

Longhorn version: master, v1.5.x
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: v1.27.3+k3s1

Additional context
N/A

This can be easily reproduced by setting Priority Class from Longhorn UI, set any value to Priority Class would cause 404 error.
Not sure but probably related to https://github.com/longhorn/longhorn-manager/commit/a5232884268215d2669cfd9bd0e51bfaa4bf69c2

This can be easily reproduced by setting Priority Class from Longhorn UI, set any value to Priority Class would cause 404 error.
Not sure but probably related to longhorn/longhorn-manager@a523288

Yeah, the side effect of the PR.
cc @ChanYiLin

IMO, returning 404 rather than 500 in the fix https://github.com/longhorn/longhorn-manager/commit/a5232884268215d2669cfd9bd0e51bfaa4bf69c2 is correct.
The problem is the error details aren't there. Suggest return more error message in the response as I mentioned in https://github.com/longhorn/longhorn-manager/commit/a5232884268215d2669cfd9bd0e51bfaa4bf69c2#r121573092.
BTW, we can improve the UI a little bit. In addition to the status code, UI can show the error message in the reply as well.

Yeah, the side effect of the PR. cc @ChanYiLin
IMO, returning 404 rather than 500 in the fix longhorn/longhorn-manager@a523288 is correct. The problem is the error details aren't there. Suggest return more error message in the response as I mentioned in longhorn/longhorn-manager@a523288#r121573092.
BTW, we can improve the UI a little bit. In addition to the status code, UI can show the error message in the reply as well.

@derekbit do we still need UI to improve that? I feel we don't need based on your PR which has the same response structure as the current response payload. right?

@chriscchien good raise, @yangchiu good root cause, @derekbit good improvement :+1:
cc @ChanYiLin

Yeah, the side effect of the PR. cc @ChanYiLin
IMO, returning 404 rather than 500 in the fix longhorn/longhorn-manager@a523288 is correct. The problem is the error details aren't there. Suggest return more error message in the response as I mentioned in longhorn/longhorn-manager@a523288#r121573092.
BTW, we can improve the UI a little bit. In addition to the status code, UI can show the error message in the reply as well.

@derekbit do we still need UI to improve that? I feel we don't need based on your PR which has the same response structure as the current response payload. right?

Nope. I've introduced a new writeErr function. The UI improvement is not necessary.

Verified pass in longhorn master (longhorn-manager 8f1205)

Set a non-exist priority class from UI will get below response instead of get 404 error code directly.

Test case test_setting_priority_class passed on AMD64 and ARM64 pipeline


SpFeas stacking error

On running the SpFeas stacking command,
spfeas -i City_Seg/Input/Bengaluru.tif -o City_Seg/Out/ --rgb --block 8 --scales 8 16 --stack-only
I am getting the following error:
Traceback (most recent call last):
File "/home/pc6/anaconda3/envs/py27/bin/spfeas", line 11, in
load_entry_point('SpFeas==0.3.0b0', 'console_scripts', 'spfeas')()
File "/home/pc6/anaconda3/envs/py27/lib/python2.7/site-packages/SpFeas-0.3.0b0-py2.7-linux-x86_64.egg/spfeas/spfeas.py", line 447, in main
overviews=args.overviews)
File "/home/pc6/anaconda3/envs/py27/lib/python2.7/site-packages/SpFeas-0.3.0b0-py2.7-linux-x86_64.egg/spfeas/spfeas.py", line 249, in spatial_features
spp.run()
File "/home/pc6/anaconda3/envs/py27/lib/python2.7/site-packages/SpFeas-0.3.0b0-py2.7-linux-x86_64.egg/spfeas/spfeas.py", line 240, in run
spprocess.run(self)
File "/home/pc6/anaconda3/envs/py27/lib/python2.7/site-packages/SpFeas-0.3.0b0-py2.7-linux-x86_64.egg/spfeas/spprocess.py", line 431, in run
parameter_object = sputilities.stack_features(parameter_object, new_feas_list)
File "/home/pc6/anaconda3/envs/py27/lib/python2.7/site-packages/SpFeas-0.3.0b0-py2.7-linux-x86_64.egg/spfeas/sphelpers/sputilities.py", line 324, in stack_features
parameter_object = scale_fea_check(parameter_object)
File "/home/pc6/anaconda3/envs/py27/lib/python2.7/site-packages/SpFeas-0.3.0b0-py2.7-linux-x86_64.egg/spfeas/sphelpers/sputilities.py", line 266, in scale_fea_check
section_counter_ = 'TL{:06}'.format(parameter_object.section_counter)
AttributeError: 'SPParameters' object has no attribute 'section_counter'
Could you please help with this?

@pranabkrc There were some parameters that were missing when bypassing straight to VRT compositing. This should be fixed now--let me know if you are able to run it successfully.


chore: clippy let-underscore-untyped

Clippy keeps complaining about
https://rust-lang.github.io/rust-clippy/master/index.html#let_underscore_untyped
Seems like a noop, esp considering that @dtolnay has filed a bug in
https://github.com/rust-lang/rust-clippy/issues/10411
One issue though is that MSRV in .clippy.toml is not matching the one in Cargo.toml:

.clippy.toml:  msrv = "1.48.0"
Cargo.toml: rust-version = "1.60"

Which one is the accurate one?  I would assume its the Cargo.toml, in which case it might be better to apply this change?

@dtolnay thx, i wasn't sure if its applicable or not.  What about the difference in MSRV?

I think it's fine. Some of the crates in the workspace have a different supported rust-version from one another.

I think it's fine. Some of the crates in the workspace have a different supported rust-version from one another.

But if they are different, shouldn't clippy rules be relevant to each specific crate?  I honestly don't understand the need for a global clippy-only MSRV -- maybe there is some other reason I'm missing?


Investigate gitlab CI for testing

They seem to support windows, osx and linux. I'm not totally sure on their docker container support, but it might be useful for platform testing.
I've also heard they've got "CI only plans" in the works, which probably won't require moving the repo and everything.
https://about.gitlab.com/features/gitlab-ci-cd/

As part of this other systems should be investigated.
https://github.com/ligurio/Continuous-Integration-services/blob/master/continuous-integration-services-list.md

They've released some good integration with github. I should test it out to see how usable it is.
https://docs.gitlab.com/ee/ci/ci_cd_for_external_repos/github_integration.html
https://docs.gitlab.com/ee/ci/quick_start/README.html


Project is being sunset

See OSS lifecycle and practices
This project has outlived its useful lifespan, and will be archived in the future.
If you are actively using this exporter, please comment on this issue with your use case.

Is this repo the code that implements https://docs.honeycomb.io/send-data/kubernetes/send-data-honeycomb-network-agent/ ?
If so, is there a replacement for this repo, that will still implement https://docs.honeycomb.io/send-data/kubernetes/send-data-honeycomb-network-agent/ ?

Hi @wushujames - no, this is a different tool. The Honeycomb Kubernetes Agent is here.

Hi @wushujames - no, this is a different tool. The Honeycomb Kubernetes Agent is here.

Thanks for the response @MikeGoldsmith. Sorry, I'm still confused. What you said doesn't match what the docs seem to pointing to.
In the docs at https://docs.honeycomb.io/send-data/kubernetes/send-data-honeycomb-network-agent/, it says to do this:
$ helm repo add honeycomb https://honeycombio.github.io/helm-charts $ helm repo update $ helm install network-agent honeycomb/network-agent --namespace honeycomb
If I follow the repos, the helm install network-agent leads me to this: https://github.com/honeycombio/helm-charts/tree/main/charts/network-agent
And the README.md says

The Helm chart installs the Honeycomb Network Agent in a Kubernetes cluster.

Which is this repo.
So the Honeycomb docs for Send Data with the Honeycomb Network Agent (No Code) seem to lead back to this repo. But as you said above, this repo is being sunset.
Which set of docs is correct? The docs at https://docs.honeycomb.io/send-data/kubernetes/send-data-honeycomb-network-agent/ or the sunsetting announcement?
Thanks! I haven't yet tried any of this, but the Send Data with the Honeycomb Network Agent (No Code) functionality sounds really useful and I was planning on trying it in the upcoming months. That's why I wanted to know more about whether it was being sunsetted or not.

Ah, I think I misunderstood your original message and thought you were looking for the Honeycomb Kubernetes Agent, which I linked to.
The Honeycomb Network Agent is being sunset due to low adoption and alternative tools being available. For example, the OpenTelemetry eBPF tool that does similar work.
Are you currently using the Honeycomb Network Agent or looking into options?
I will update our documentation to indicate this tool is being sunset.


Remove version: qBittorrent.qBittorrent version 4.3.5

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/qbittorrent/files/qbittorrent-win32/qbittorrent-4.3.5/qbittorrent_4.3.5_x64_setup.exe/download
OK
(Automated message - build 923)

Close with reason: Package still available.;


logic for changing the note names on the bottom of the screen can get jumbled

if you are in a key with a raised note, if you add an accidental it will remove the note number, i.e.:
G#5 ---> add a sharp ---> G##

Fixed in this commit https://github.com/oakleyaidan21/kalimba-tabs/commit/3e5d5de4dee85ae79e6acf2f7f3bac4a750430d9


[AIRFLOW-1540] Add proxies to Slack Operator

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1540
"Airflow 1.8.1 - Add proxies to slack operator"

Description

Airflow Slack Operator is not working as expected for servers using proxies to connect to open internet. This PR will help to use proxies in connection and resolve existing issue of connection timed out.

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[AIRFLOW-1540] Add proxies to Slack Operator

Codecov Report

Merging #2548 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2548      +/-   ##
==========================================
- Coverage   70.88%   70.85%   -0.04%
==========================================
  Files         150      150
  Lines       11585    11586       +1
==========================================
- Hits         8212     8209       -3
- Misses       3373     3377       +4

Impacted Files
Coverage Δ

airflow/operators/slack_operator.py
0% <0%> (ø)
:arrow_up:

airflow/jobs.py
77.67% <0%> (-0.28%)
:arrow_down:

Continue to review full report at Codecov.

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

@saguziel @bolkedebruin please look into this pr and let me know your feedback.

@aoen or @criccomini can you please look into this pr and let me know your feedback, thanks.

Why not use the system wide proxy? (Http_proxy)?

This should just be cone by setting the http_proxy or https_proxy environment variables.


Are x and y coordinates given with the upper left corner as origin?

Like title says, where is the origin of the x and y coordinate system in the traces in the .jsonl files? I could not find this on your website or in your paper. Maybe I didn't look carefully? Else, I think it would make sense to mention it somewhere.

Hi @rubencart
Thanks for pointing this out. The origin is indeed the upper left corner of the image.
I have now specified this in the website, under "Format description".
Best,


test_encode_decode_msg__ints_can_be_passed_as_bytes failing

test_encode_decode_msg__ints_can_be_passed_as_bytes is failing, see the trace on Travis.

@domob1812 I think you glanced at this earlier and said it looks like a rebranding error?  Is this likely to be due to the genesis block hash being different between Bitcoin and Namecoin?

Is this likely to be due to the genesis block hash being different between Bitcoin and Namecoin?

Yep, that was it.

Yes exactly, thanks for the fix!


Suggestion: allow leading # comments in dotslash files

Sometimes you need to hand-craft a dotslash config file and sometimes it's nice to add some commentary about said file.
For example I was authoring a config for gofmt and wanted to document how I got all the values to make it easy for the next person to update it in future.
Currently when I added a # line immediately after the hashbang, dotslash emitted the following:
dotslash error: problem with `/path/to/gofmt`
caused by: failed to parse DotSlash file
caused by: expected value at line 1 column 1

The comment syntax is // and /*…*/. See https://dotslash-cli.com/docs/dotslash-file.

Ahh I completely missed the tip block in the docs
TIP
The JSON payload in a DotSlash file is parsed with a lenient JSON parser that allows for trailing commas as well as // and /*-style comments.

I'd just assumed that the instances of // and /**/ were for docs purposed, not that they were valid syntax.


Additional Information Request - Low Quality Armors

Hi,
you can repair "crude/cracked/low..." white armors (like a dusk shroud, archon plate etc.) which have a bad defense in the cube with the recipe "EL + chipped gem". Then they are normal with higher defense, but the iLvl = 1. This means, that you can get maximum 3 socket out of it by LarzukQ or Cube-Socket-Recipe.
This is quite nice to know when you look out for some elite bases with 3s.
I recommend the cube-socket-recipe, because you have the chance of getting 1-6 sock on it and the maximum is 3, so you have 66,6% chance to get it 3 socket which is quite nice and the invest is very low.
Cheers,
e-soni@d2jsp

Sorry for the delay.
I manually merged this into the main branch.  I've added you to the credits with version 1.100.
Thanks for this patch!


Open transaction with embedded replica error

If I leave a transaction on with an embedded replica:
diff --git a/libsql/examples/remote_sync.rs b/libsql/examples/remote_sync.rs
index 2b7828938a..3f5e72622b 100644
--- a/libsql/examples/remote_sync.rs
+++ b/libsql/examples/remote_sync.rs
@@ -60,6 +60,7 @@ async fn main() {
         Ok(_) => {
             println!("You entered: {}", input);
             let params = params![input.as_str()];
+            conn.execute("BEGIN", ()).await.unwrap();
             conn.execute("INSERT INTO guest_book_entries (text) VALUES (?)", params)
                 .await
                 .unwrap();

I get a very weird error:
penberg@vonneumann libsql % cargo run --example remote_sync
   Compiling libsql v0.2.0-alpha.1 (/Users/penberg/src/libsql/libsql/libsql)
    Finished dev [unoptimized + debuginfo] target(s) in 2.12s
     Running `target/debug/examples/remote_sync`
Syncing with remote database... done
Please write your entry to the guestbook:
Hello!
You entered: Hello!

Guest book entries:
thread 'main' panicked at libsql/examples/remote_sync.rs:78:42:
called `Result::unwrap()` on an `Err` value: Bincode(DeserializeAnyNotSupported)
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace```

If I was to take a crack at this issue, would you suggest this is a good place to start looking? https://github.com/tursodatabase/libsql/blob/main/libsql/src/value.rs#L427

Some error logs:
https://gist.github.com/coindegen/19551cf4c20e427bda548aa00e240a56

I believe https://github.com/tursodatabase/libsql/pull/849 is related to this and should fix this error.

working on my end, thanks a lot.
You should be able close this too: https://github.com/tursodatabase/libsql/issues/831


Update Open Source docs page from master README

The current page at https://shopify.github.io/shopify_api/ is quite outdated. This PR simply copies the currrent README from master.
Longer-term I think we need a better way to manage this. I'll open a separate issue to discuss.
Unfortunately there is no way to fully preview how this will exactly look on shopify.github.io so some further tweaks may be required. In particular, the heading levels (e.g. ### Ruby version) have changed.
(supercedes #734)

Click to see a very tall screenshot

Actually, we can preview it, see #741.

https://shopify.github.io/shopify_api/ is now showing the update.


Compilation big rocks moving to dotnet CLI

Now that we don't have the DNX providing compilation services, we need to figure out how to handle a few things:

[ ] Getting compilation options
[ ] Getting reference assemblies

/cc @anurse @pakrym

And target framework too

@pakrym target framework will come from IApplicationEnvironment. Once we have the CLR feature then it'll just work.

I was able to run MVC with dotnet cli by:

Overriding IAssemblyProvider to StaticAssemblyProvider with entry assembly added (At this point it was able to find controllers)
Adding naive implementation of ICompilerOptionsProvider
Adding naive implementation of ILibraryExporter
Overriding IApplicationEnvironment and returning not null RuntimeFramework

nice!

Compilation options provider shouldn't be a big problem, we can infer most of settings at runtime from IHostringEnvironment and IRuntimeEnvironment

For ILibraryExporter we can Embed project.json into assembly (something like "embedProjectMetadata" compilation option) and  and add API to read from it, so MVC can implement its own logic on top

Well that one is a bit tricky. The dotnet cli would need to do that and it's bit weird to always do it. Also you need the lock file not just the project.json file. Also you need access to the reference assemblies, which aren't always available unless we make them available.

@pakrym wouldn't we need the project.json to look at things like LanguageVersion, Defines, WarningAsErrors?

Or we hardcode them to some default values :)

At that point, isn't it much simpler to just have an alternate way of configuring views (e.g. via options). It also separates oddities like figuring out which properties to exclude (code signing, emitEntryPoint) from the primary project and plays nicely with binaries containing embedded cshtml. @davidfowl doesn't particularly like this solution though.

I think we can have everything on razor options. All we're talking about is defaults. Let's start by treating the razor view engine as a separate compilation system that can be fully configured in Startup. Then we figure out where the defaults come from.

@davidfowl  So we remove dependency on ILibraryExporter?

Also consider the precompile case.

I think ILibraryExporter needs some special care. I'm also going to see what the Roslyn team thinks about this. We could say that the only abstraction we expose for razor is the Roslyn API for references and compilation options (we don't need to a abstract it). That leaves the burning question, what set of assemblies do we pass to the razor view compilation process by default and how do we locate them.
I'm like 99% sure that we'Il need a tool chain changes.

We need to decide if we should fallback to runtime assemblies or fail when they aren't available.

Need to move some compilation options from here https://github.com/aspnet/dnx/blob/dev/src/Microsoft.Dnx.Compilation.CSharp.Common/CompilerOptionsExtensions.cs#L29 as defaults.

https://github.com/davidfowl/dotnetcli-aspnet5/tree/master/HelloMvc/Infrastructure
When this folder is gone, this issue can be closed.
/cc @pakrym @pranavkm

The repo is gone, so I think we should be good to close this issue


TLS cipher suites restriction does not work

Description
We are trying right now to restrict the TLS cipher suites in our environment.
https://docs.microsoft.com/en-us/dotnet/core/compatibility/cryptography/5.0/default-cipher-suites-for-tls-on-linux
Example: the SQL connection and service bus connection should be limited!
Environment

Microsoft Azure
Kubernetes cluster
Docker with .net 5 service
SQL Azure
Service Bus

Configuration

Dockerfile.txt
openssl.txt
sslscan to SQL server

Issue
The restriction has no effect. The connections are still established.

  I don't see where your docker container running on Ubuntu Focal comes into play...

Hi @bartonjs,
The application is running in kubernetes cluster running on Azure VM.
We have to ensure that the TLS ciphers configured in the configuration file /etc/ssl/openssl.cnf is having an effect on our TLS connections that our application opens towards the Azure SQL Database and Azure ServiceBus.
We thought the only way to prove that the TLS ciphers in /etc/ssl/openssl.cnf have an impact is to misconfigure them so that we eventually get an TLS handshake error that tells us that the unification set between the TLS ciphers of Azure SQL Database and / or Azure ServiceBus is empty. To do so we checked the TLS ciphers of the Azure SQL Db with sslcan.exe and translated the IANA names to openssl compatible ciphers using this mapping table: (../openssl-iana.mapping.html)[https://testssl.sh/openssl-iana.mapping.html]. This way we know what TLS ciphers to avoid in our  /etc/ssl/openssl.cnf  configuration in order to get the TLS handshake error.
Unfortunately the TLS can be established no matter how much we restrict the TLS ciphers. Now we are doubting that the configuration in  /etc/ssl/openssl.cnf has any effect at all.
I know the more simple approach to check the TLS ciphers would be to publish a REST API and perform a TLS scan on it.
This is not an option here as all external connections are managed by a internet facing web server and so are the TLS ciphers for these connections...

We thought the only way to prove that the TLS ciphers in /etc/ssl/openssl.cnf have an impact is to misconfigure them

Currently you have a CipherString value of TLS_AES_128_GCM_SHA256.  In OpenSSL's configuration, TLS 1.3 ciphersuites get configured in the CipherSuites property, and SSL 2 through TLS 1.2 ciphersuites get configured via the CipherString property.  Since you've specified a TLS 1.3 ciphersuite in the TLS 1.2 and below entry I think it ignores it, then decides you basically don't have a configuration for that setting.  That's just how openssl.cnf works (and I'm guessing as to what it does in this situation :shrug:).
A more reasonable entry that should make your test result in a connection failure would be a TLS 1.2 ciphersuite that doesn't appear in the list, like ECDHE-ECDSA-CHACHA20-POLY1305.

Hi @bartonjs,
I changed the config but it still doesn't work. Any idea?
openssl_conf = default_conf
[default_conf]
ssl_conf = ssl_sect
[ssl_sect]
system_default = system_default_sect
[system_default_sect]
Ciphersuites = TLS_CHACHA20_POLY1305_SHA256

CipherSuites only modifies TLS 1.3+ ciphersuites, and SqlAzure doesn't support TLS 1.3
Assuming you're trying to get things to fail (with a no mutually available ciphersuites error)
openssl_conf = default_conf

[default_conf]
ssl_conf = ssl_sect

[ssl_sect]
system_default = system_default_sect

[system_default_sect]
CipherString = ECDHE-ECDSA-CHACHA20-POLY1305

It worked, I with your latest proposal we get the expected TLS handshake error.
{"Timestamp":"2021-02-26T09:12:05.0705739+00:00","Level":"Error","MessageTemplate":"An exception occurred while iterating over the results of a query for context type '{contextType}'.{newline}{error}","Exception":"Microsoft.Data.SqlClient.SqlException (0x80131904): A connection was successfully established with the server, but then an error occurred during the pre-login handshake. (provider: SSL Provider, error: 31 - Encryption(ssl/tls) handshake failed)\n ---> System.IO.EndOfStreamException: End of stream reached\n   at Microsoft.Data.SqlClient.SNI.SslOverTdsStream.ReadInternal(Byte[] buffer, Int32 offset, Int32 count, CancellationToken token, Boolean async)\n   at Microsoft.Data.SqlClient.SNI.SslOverTdsStream.Read(Byte[] buffer, Int32 offset, Int32 count)\n   at System.IO.Stream.Read(Span`1 buffer)\n   at System.Net.Security.SslStream.FillHandshakeBufferAsync[TIOAdapter](TIOAdapter adapter, Int32 minSize)\n

Thanks a lot!

Thanks a lot.


[TEST-USE][Transfer learning] Remarks

[ ] In db.execute(table_or_collection.insert([Document({'x': x}) for x in data]))    a closing "]" is missing
[ ] In Image tab s3://superduperdb-public-demo/images.zip is not working but https://superduperdb-public-demo.s3.amazonaws.com/images.zip is working
[ ] In Audio tab the same, mentioned link does not work but the https://superduperdb-public-demo.s3.amazonaws.com/audio.zip is working

In db.execute(table_or_collection.insert([Document({'x': x}) for x in data])) a closing "]" is missing
This error was fixed

Please fix trivial things if possible.


run ecr scan results after preprod env build

Purpose
The run of the pipeline against master failed due to dependencies on building an environment.
Fixes UML-540
Approach
add an entry for the job for development branches and master branch, including filtering branches.
Learning
Checklist

[x] I have performed a self-review of my own code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage remained the same at 76.169% when pulling 7c8e1ffded3a1b39b8b0fe696b660d9dd448d956 on UML-540-fix-master into b145b2158e01cc1a0c03e6db5881ff05ee7ab2f8 on master.


EarthCube2021 post

Post about earthCube 2021 presentation of geoextent.

@SbastianGarzon Still no word yet?

Please try plain HTML image lines to get "in line" binder buttons to fix the below:

https://twitter.com/o2r_project/status/1422581442324930571


howto make pragma statements persistent, e.g. for foreign_keys checks

Hello!
Writing a small RESTful webserver application, we rely on constrains in the sqlite db.
In our app the foreign_key checks were working once every second time. We figured that
go-sqlite3 will open several connections, probably recycling some, and some of the pool did not have
the "PRAGMA foreign_keys = ON" set.
Our solution was to use a connect_hook like
import (
        "database/sql"

        sqlite3 "github.com/mattn/go-sqlite3"
)

func init() {
        sql.Register("sqlite3_with_fk",
                &sqlite3.SQLiteDriver{
                        ConnectHook: func(conn *sqlite3.SQLiteConn) error {
                                _, err := conn.Exec("PRAGMA foreign_keys = ON", nil)
                                return err
                        },
                })
}

and later calling
sql.Open("sqlite3_with_fk", dataSourceName)

Is this the best way of doing this?
Because this is a common use case, people wanting to set some sqlite PRAGMAs
and use them everytims, we suggest to add a hint to the documentation.
Best Regards and thanks for creating go-sqlite3 as Free Software!
Bernhard

http://sqlite.org/compile.html#default_foreign_keys
Regards.

On Thursday 19 November 2015 at 18:23:30, gwenn wrote:

http://sqlite.org/compile.html#default_foreign_keys

Hi gwenn,
I'm assuming that you are are pointing me to the possibilty to change
the default for sqlite at build time. Thanks for the hint!
Yes, I knew about this possibility, that it sometimes is not a good
option because an already build sqlite version is uses. Like if you
are on a Debian GNU/Linux you really would want to use the system sqlite
builds. One argument for this are security updates.
Therefore I think this will be common use case which should be solved on
go-sqlite3 level. And with the connect hook, there is a reasonable solution.
The behaviour of a go application with go-sqlite3 can be surprising, at least
it took us several hours to figure out what was going on and find a good
solution. A hint in the go-sqlite3 would have been very helpful. Thus we are
suggesting it.
Regards,
Bernhard

There is another alternative:
http://sqlite.org/c3ref/open.html#urifilenamesinsqlite3open
http://sqlite.org/c3ref/uri_boolean.html
db, err := sql.Open("sqlite3", "file:my.db?foreign_keys=on")

And somewhere in SQLiteDriver.Open
// FIXME C.CString("foreign_keys")
if C.sqlite3_uri_boolean(name, "foreign_keys", 0) != 0 {
  _, err := conn.Exec("PRAGMA foreign_keys = ON", nil)
  // TODO error handling
}

Regards

This fixes your problem? https://github.com/mattn/go-sqlite3/pull/407

#407 seems to solve the one of the pragma use cases by adding an explicit flag. It would improve our technical situation with foreign_keys. It does not offer a more general solution for other PRAGMA settings which this is issues is also about.

I currently use the connection hook method for all of the pragmas I'm using. As an alternative, you could use the advice from https://medium.com/@benbjohnson/structuring-applications-in-go-3b04be4ff091 on wrapping the DB and TX types. You could then add a method on the TX type that calls your pragmas on the TX before returning it. You could also add it directly to the TX returned by your wrapped DB.Begin() method. For example:
type DB struct {
    *sql.DB
}
type Tx struct {
    *sql.Tx
}
func Open(dataSourceName string) (*DB, error) {
    db, err := sql.Open("sqlite", dataSourceName)
    if err != nil {
        return nil, err
    }
    return &DB{db}, nil
}
func (db *DB) Begin() (*Tx, error) {
    tx, err := db.DB.Begin()
    if err != nil {
        return nil, err
    }
    if _, err := tx.Exec("PRAGMA foreign_keys = ON", nil), err != nil {
        return nil, err
    }
    return &Tx{tx}, nil
}


Specify Ruby version

to 3.1.4 in .ruby-version file,
since jekyll of this project does not work with Ruby 3.2.
Note:
Ruby 3.2 removes a bunch of methods from the top level Object class.
This will trigger a Liquid Exception: undefined method tainted?
for the jekyll version used by this project.

For reference, this the error I encountered when I run bundle exec jekyll serve on Ruby 3.2.2
  Liquid Exception: undefined method `tainted?' for "16 Jan 2022":String in /workspaces/simple.css-site/_layouts/post.html
                    ------------------------------------------------
      Jekyll 4.2.1   Please append `--trace` to the `serve` command
                     for any additional information or backtrace.
                    ------------------------------------------------
/usr/local/rvm/gems/ruby-3.2.2/gems/liquid-4.0.3/lib/liquid/variable.rb:124:in `taint_check': undefined method `tainted?' for "16 Jan 2022":String (NoMethodError)

And here is a link to a related discussion on Jekyll forum.

I'm not sure this is an appropriate change to make. For example, I have an M1 Mac running 3.0.0 and upgrading is a PITA. This would be a breaking change for me, and as the only maintainer, I think that's very important.
I think a better solution is if people want to copy the site, fork and implement this change themselves.

For example, I have an M1 Mac running 3.0.0

Then what about specifying 3.0.0 in .ruby-version instead?
(Previously I chose 3.1.4 just because it is the most recent compatible release.)

This would be a breaking change

.ruby-version will not affect the system ruby version on local machine.
.ruby-version is a hint to inform potential contributors which Ruby version should be used,
and in certain environments, it will result in ruby version switch automatically:

On dev/local machine, if rbenv or rvm (popular options for managing multiple Ruby versions) is installed, then it will automatically change to the Ruby version specified in .ruby-version. If no such tools installed, then .ruby-version has no effect at all and Jekyll will continue to use system ruby version.
Since .ruby-version is a popular practice among Ruby projects, a lot of cloud service providers also support it, for example Netlify, GitHub Codespaces and GitHub Actions.

It's only me and @lkhrs who maintain the site, this has never been an issue, so why would we need to specify a Ruby version? It works for us, and if others want to fork it and specify the version, they can.
I suppose what I'm trying to say is I don't understand the value this adds to the project? Maybe I'm missing something? Apologies if I am.

Re-opening as I think there's more conversation to be had here.

It makes the developer experience of potential contributors a bit easier if their default ruby version is 3.2, and they have a ruby version manager (which supports .ruby-version) installed. Otherwise, they need to either manually switch to a Ruby version (<3.2) or run bundle update to update jekyll & liquid to a more recent version.
However, I do agree that since "this has never been an issue" for current maintainers, .ruby-version is not necessary and introducing it will add a bit of overhead (the version need to be updated).

the version need to be updated

But if that file is not picked up unless one uses a Ruby version manager, it seems like its presence would not affect current maintainers anyway (but please correct me if I'm wrong); and it not being updated would only affect contributors like you — in which case a PR updating that version would be appropriate to make things easier for the next guy, as you're doing here. :heart:
In any case, I would definitely support making the development setup by people other than the current maintainers easier, because people's availability come and go and it would be a pity to risk the of this project becoming unmaintained — a risk that increases if we don't actively minimize friction points like the one you bumped into as described above. Making the Ruby version explicit is one of the aspects that improves this DX for new contributors, and assuming it doesn't affect current maintainers, I'd suggest it would be a net positive.

Thanks for the conversation, folks. The additional context helps. I've just added a .ruby-version and pushed it. I've set it to 3.0.0.


Hot fix

Pull request outline
Fix for #194
Checklist:
- [ ] Docs updated
- [ ] Tests written
- [ ] Check for breaking changes in Resource

[x] Version bumped

Added:
- A list of new things
Changed:

Extended regex expression for tags

Removed:
none
Related issues:

#194

You can add a reference to an issue using hash followed by the issue number
If a checklist item is to be ignored it must be struck through

I think this closes https://github.com/mystic-ai/pipeline/issues/194, right?


unable to install the package

I have seen this myself.
One solution, while this is not solved is to install with --force option, but that's probably not a great idea.
And by some reason it installed version 2.0.1 by default in my case, which is very old. Had specify version 4.0.0 when installing this dependency.

@Amrkamel1 @arnisgustins
Yeah, this error is because of the defined peer dependencies in package.json.  That was the last known compatible version of WDIO.  I think the next step would be to bump the dependency and see if things still work.

Investigation Notes:

Fresh install of @wdio/cli and running npx wdio config does not run into the above error.

Hi @fijijavis,
Fresh install of @wdio/cli doesn't seem to be working. Is there a plan to bump up the WDIO dependency to latest version?
Thanks.
Regards,
Kishor Ganeshrao

@fijijavis This did not work. Is there a plan to up the dependency?

+1 keen to have this updated please


[EXAMPLE] update the API call

update example to use new API
depends on #117

closed as commits have been moved to #118


[CIRCLECI] Enable clang-tidy performance checks (CI)

Motivation and Context

Fixes #901

How Has This Been Tested
Locally and with CI.

Types of changes

[X] Docs change / refactoring / dependency upgrade

Checklist

[X] My code follows the code style of this project.
[X] I have read the CONTRIBUTING document.
[X] I have completed my CLA (see CONTRIBUTING)
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

This is weird, I am not getting any errors when building with emsdk locally...


"ruby" binary for mruby crate

Run Ruby source files on a fully initialized Mrb interpreter.

Blocked on GH-9 for this to do anything observable

Kernel#puts and Kernel#print are implemented to support spec-runner so this should be unblocked.


CIVILIZACION

PROGRAM FOR |#ra reality augmented

need developt certain program enfatized in the reality augmented and develop of the civilization.


Adds Sentry & support async middlewares

Moves from Raven to Sentry.

Express doesn't have great support for promises which means we need to catch Promise Rejections and not trigger UnhandledPromiseRejectionWarning's (which in the case of a runtime error in async middleware, would cause the page to hang). Even if not for error handling, we get a bunch of other advantages:

get rid of next() functions (which basically encouraged Continuation-passing - an outdated pattern in JavaScript)
Rely on standard JavaScript semantics for indicating when a function is complete (e.g. return statement)
Simply throw an error with throw new Error() rather than next(new Error())
We get to remove a bunch of try/catch/rethrows as we now rely on a single async route handler to catch Promise rejections
Affected tests are slightly simplified
No more Node.js deprecated unhandled rejection errors in the console when using the async middleware

Some disadvantages:

We must now remember to app.use(asyncRoute(route)) instead of app.use(route) (will be fixed when Express 5 is out)
Had to use a setTimeout hack due to an Express bug

Despite the nitpicks, this seems a vast improvement.
Nicely done Umar!


Update help.py

Changed support server URL to a working one.

gotta replace the Invite Link with the permanent one


Can't compile a file that extends two different classes that share the same name

When compiling a file that contains the the same-named class from two different scopes using SIMPLE_OPTIMIZATIONS[1], closure compiler fails with the following error:
 DUPLICATE_CLASS: Multiple classes cannot share the same name.

Closure Compiler Service Link
function f1 () {
    class DUP_NAME {
    }

    class Y extends DUP_NAME {
    }
    return Y;
};

function f2 () {
     class DUP_NAME {
     }

     class Z extends DUP_NAME {
     }
     return Z;
};

It looks like is caused by:
Es6ToEs3ClassSideInheritance
From what I can tell, using a language_out of:

ECMASCRIPT5_STRICT
ECMASCRIPT5
ECMASCRIPT3

fails, but it works if I use:

ECMASCRIPT_2015
Or above.

This can be demonstrated in the online compiler by adding:
 // @language_out ECMASCRIPT5

To the flags at the top.
I would have expected that this would be able to be compiled down to ECMASCRIPT5 since this is valid ECMASCRIPT 2015.
[1] It fails for ADVANCED_OPTIMIZATIONS too, but there is a lot of valid JS code that fails for Advanced, so that is less of an issue.
cc: @charvey77

I agree, this looks like exactly the same issue. Thanks @lauraharker for pointing me to @brad4d’s fix.


docs: remove module-metadata.json and commitlint.config.js

/run pipeline

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


feat(queries): expose helper functions

These are useful for creating additional queries outside the library

What:

Why:

How:

Checklist:

[ ] Documentation
[x] Tests
[ ] Ready to be merged
[x] Added myself to contributors table

:tada: This PR is included in version 2.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature request: COLUMN_CLASS_NAME and HEADER_CLASS_NAME

It would be great to have additional options for the table extension to specify class names for th and td tags.
Thanks again for the great work!

@dflorey, you need to implement a custom attribute provider. See: AttributeProviderSample2.java
You can use the same provider to add/remove/change attributes for any node being rendered. So it
can be used for issue #128


[PE-D][Tester B] Add person with same name

What if in Singapore there exist two customer with exact same name like Tan Wei Jie doesnt have an english name with it.
How would it get added into the system than?
Name can be duplicated

Maybe can check with other things like phone number etc.

Labels: type.FeatureFlaw severity.High
original: benjiBase/ped#6

write in UG big big MUST HAVE UNIQUE NAME


Problem finding entrypoint

Hi,
I created a test application here: https://github.com/furesoft/compiler-test
but the compiled.dll cannot be run by the .net runtime with this error: Unhandled exception. System.MethodAccessException: Entry point must have a return type of void, integer, or unsigned integer.
Am I doing something wrong?
Thanks

The problem here seems to be the use of moduleResolver.SysTypes.Void instead of PrimType.Void in the method signature, it's a quirk of DistIL's type system as PrimTypes are directly equivalent to the primitive types in the CIL spec, whereas SysTypes are just references to the definitions in CoreLib.
Replacing to the following works for me:
        var main = program.CreateMethod("Main", new TypeSig(PrimType.Void), [], MethodAttributes.Static | MethodAttributes.Public | MethodAttributes.HideBySig);

This seems to be a special case not handled by the runtime though, I'm not sure if the module emitter should be normalizing these types.

Thanks a lot

so when to use PrimType and when use SysTypes?

PrimType should be preferred, SysTypes is only useful for a few specific cases like referencing Int32.ToString() and other members.


--replace-in-metadata does not find '_' in title

Checklist

[ x] I'm asking a question
[x ] I've looked through the README and FAQ for similar questions
[x ] I've searched the bugtracker for similar questions including closed ones

Question

--replace-in-metadata does not find '_' in title:
YT-Dlp https://youtu.be/60m_nfosEyI --restrict-filenames  --replace-in-metadata title "_" "space"

The above does not work. The file name still has _ in it. I have tried the following which works:
YT-Dlp https://youtu.be/60m_nfosEyI --restrict-filenames  --replace-in-metadata title "Gui" "xxx"

Is this a bug?

Who can say?
#30839: "if you were actually running yt-dlp ..."
My guess: the problem is that the _ that you're trying to match is an artefact of --restrict-filenames and not an actual character in the title metadata field.


No automatic exports from saved search

[x] I have verified that I am on the latest release of BBT (https://github.com/retorquere/zotero-better-bibtex/releases/latest) and Zotero
[x] I am posting a single question, bug or feature request
[x] I have included a descriptive subject of the problem
[x] I am available for follow-up questions and testing

Any plan for this to change? I was using it in a few places. Thanks.

I'd love to actually, but Zotero doesn't tell me that it's exporting saved searches, it just says "export these items" where "these items" happen to be the search result. For collections and libraries I'm actually getting the library/collection to be exported so these are easy to detect. I used to have an elaborate way to reconstruct that a search was taking place, but that involved some fairly invasive runtime changes to Zotero that I'm no longer comfortable making. If Zotero would mark the export so I could recognize it was a saved search, it would be easy to do.

Would it be possible just to do an export on a schedule?

It's not the auto-export that's the problem, it's detecting that the thing being exported is a saved search; technically, when you right-click on a collection and select "export", one of two things happen:

for a library or a collection, Zotero hands its internal exporter the library or collection. I've patched the internal exporter to pick this up, I schedule the auto-exporter, done.
for a saved search, zotero gathers the items being displayed, hands that list to the internal exporter, but my patch can now only see that Zotero told the internal exporter to "export these n references". I cannot see why these n references are being exported -- maybe you ctrl-clicked them together, maybe it was a saved search, but there's no (good) way for me to tell.

If Zotero would say "export these n references, which are the result from saved-search X", it'd be easy (not trivial, but easy enough) to add auto-export.

So I could pester them about that? :-)

You could, I could, but my schedule is hectic these days, so I prefer to currently limit my work to what I can do without having to coordinate, and there's still some gnarly bugs that need my attention. I could provide any required technical insights, but showing a user need would probably help, yes.

I've opened the discussion here.

(although I'm not sure that the zotero forums wouldn't be more effective)

Hello.
I've opened a discussion on the zotero forum. See : https://forums.zotero.org/discussion/70534/auto-exporting-saved-searches

So Zotero 4 (with Firefox) used to mark exports for saved searches, such that BBT 1.6.100 can distinguish between a saved search being exported or a manually selected list of items?

(Continued from #2139 (reply in thread)) So Zotero 4 (with Firefox) used to mark exports for saved searches, such that BBT 1.6.100 can distinguish between a saved search being exported or a manually selected list of items? And Zotero 5 & 6 no longer do so?

No, but it was easier for me to capture the action of the right-click on the saved export and smuggle it all the way to the export being started by tacking the info onto some data structures that were for Zotero internal use but which tolerated extra data being tacked on. It was always a fragile way of doing things, it just got so fragile I no longer felt I could support it.
Zotero however knows it's just handled a right-click on a saved search and that an export was started, so it could safely mark the export as "export these items, oh and should you care, these items were the result of saved-search X". The extra information wouldn't bother anyone not using it, but for BBT it would mean a safe way to capture these exports. Such tagging as "collection" or "library" is already done for those two types, but mostly as an artefact of how export was implemented.
Note that saved-search auto-export is always very expensive, because Zotero does not notify "this search now also yields this extra item" or "this item no longer matches that search". If you had a saved-search export, BBT would fire off the search(es) for any item that changed, see whether the search result included the changed item, and then kick off that export.... and come to think of it, I'm not sure that I cached what items were matched previously, so there's a good chance that "item no longer matches search" never updated the auto-export.

Thanks for that details. So for your own use case, you simply gave up all auto-exports of saved searches, and only export them manually when needed, by right clicking on the saved searches in Zotero to export them?

Correct. The explanation above was incorrect BTW -- I wasn't tacking it onto something that would tolerate it, I was replacing some internal data of Zotero in an incompatible way, which would break Zotero if it tried to access it, but because I knew the call chain precisely, and everything was synchronous, I was sort of sure I could change it back just in time before Zotero tried to use it. It was easily one of the more crazy monkey-patches BBT employed.
Z5 changed all that, and I could no longer realistically oversee all call chains. Something was bound to break sooner rather than later, in impossible to debug ways. If Z adds the information to the export context, it would not be too hard to add it back -- not trivial, so it would take some time, but feasible.


fix(backup): use long date format for last backup date

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


Fix number of jobs command line parsing

Follow for #328, by @yawor

t's still possible to provide number of jobs on command line, in which case the env value is ignored

I was not sure about that, so I wrote some tests, and I think I found a bug.
Can you please review and confirm I was not mistaken?

I also updated the --help message and updated the documentation.

Your version is good too, but I don't see any bug in my original version. First the value is taken from the args. If it's None (not provided in command line), the code tries to assign value from the env. If it's not provided, the get method returns None.
So the value is a value provided from command line if not None, otherwise it's a value from env if provided, else it's None.
Unless I've missed something, your code does exactly the same thing but uses additional temporary variables. One extra small difference is that my version doesn't even call environ.get unless it actually needs checking (value not provided in cli).
Of course I could miss something in the code. I can't check it right now to be 100% sure.

Of course I could have missed something in the code.

Well, that's why we write tests :)

My version doesn't even call environ.get unless it actually needs checking (value not provided in cli

You're right, but we don't really care that much about performance - reading the value from environ is pretty fast compared to running a process, which is quite literally all of what tsrc does :)
Let's merge this then.


Provide a method in the simulator API that returns the metadata level

In our TA1 codebase we have the function
function infer_information_level(step_metadata::PyObject)::String
    if length(step_metadata.object_list) > 0 || length(step_metadata.structural_object_list) > 0
        return "oracle"
    elseif length(step_metadata.object_mask_list) > 0
        return "level2"
    else
        return "level1"
    end
end

This logic is a translation into code of information that was communicated to us a while back, not sure when or where.  We have no way to ensure this function stays up to date as the API for the simulator changes.  AFAIK, it would be much better to move "what is the metadata level" code to the simulator, where it would be just a field accessor.

get_metadata_level() has been added to the controller class.
Merged to development and will be included in next release (0.4.2).

Thanks @rartiss55 for the work and for the update!


Add xmlns:flow="urn:frank-flow" to root element

Is your feature request related to a problem? Please describe.
When adding flow:x="100" to the Exit element of Frank2Example4 the Frank!Console will show:
Configuration [Example4a] aborted starting; error during unmarshalling configuration from file [Configuration.xml] with digester-rules-file [digester-rules.xml] in element [pipeline]: (SAXParseException) SystemId [classpath:Configuration.xml] line [12] column [67]: The prefix "flow" for attribute "flow:x" associated with an element type "Exit" is not bound.
And the adapter doesn't start
Describe the solution you'd like
When the Frank!Flow adds a flow:* attribute to an element, check whether the root element of the xml file contains xmlns:flow="urn:frank-flow", if not, add it.
Describe alternatives you've considered
-
Additional context
-

3 hours

We now also have a settings button for a config which can add for example flow:forwardStyle="flowchart" flow:direction="horizontal" flow:gridSize="100". In this case the xmlns:flow="urn:frank-flow" needs to be added too.


Modifying the database schema and/or entries in-place

The procedure might go like this:

Write a script (preferably in python but may use raw sql if necessary) to make needed modifications to one or both of schema and entries. If schema is modified, the script should also add a new entry to Provenance including updated version info for the schema. The script should incorporate old and new db schema semantic version, e.g. migrate_1.2.3_to_1.3.0.py
Practice on a test schema (e.g., dump the real production or working schema and restore to a different schema name).   Revise script if necessary and try again until results are satisfactory
If there are schema changes, on a branch update schema.yaml and version fields in create_registry_schema.py.  Check that schema creation works properly
Run script on working and production schemas (should we lock people out while this is happening?)
If there were schema changes, merge in branch (and make code release?)

When inserting a new entry in the Provenance table, what is the difference between update methods 'MODIFY' and 'MIGRATE'?
Should _insert_provenance behave differently depending on update method?  E.g., use old value of associated_production in the new row for anything other than 'CREATE'?


7 Day calendar week only

Has anyone attempted to add functionality to this component where you can limit it to only 7 days without a specific date? Just for events that are the same every week and don't change according to the date.

Just remove the navigation


How could we convert cdata into Seurat object?

Thank you very much for the wonderful tools!
I am now working with Visium HD data and would like to use seurat for some of the analysis after cell segmentation. May I ask how you suggest to convert the cdata into a seurat object?
Thank you in advance for your help!
KR,
Lin

Please look into existing Scanpy to Seurat conversion tools, being mindful of the fact they probably don't support image data.

@LinAU did you manage to open the Bin2Cell data in R to work with Seurat? I'm trying to do the same but I'm losing the image in the process.
Thank you.

I just found this package that solved my problem:
https://github.com/cellgeni/schard

@bmateosa  Thanks!  I will try this as well!

@LinAU & @bmateosa : Hi! were you able to upload the cropped histology image into a seurat object using schard? I've used this package to upload the cdata, but am getting errors when i try and upload the cropped histology image.

You can probably just make use of the built in Spaceranger high resolution one when working at cell scale.

@ktpolanski Do you know how to upload that image? Even when cropping the high res image it still throws the same errors

Use the high res image and the spatial coordinates without cropping.


Build mac app on windows EPERM error

Preflight Checklist

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Issue Details
Electron Version:
11.0.3
Electron Packager Version:
15.1.0
Operating System:
Windows 10 Pro (1909)
Expected Behavior
I' am building a app for darwin os on a windows system with the following instruction:
electron-packager . HyperloopUPV --overwrite --asar --platform=darwin --arch=x64  --prune --out=release
I expected to create the app with no errors
Actual Behavior
The folder with the app is created and apparentely there is all. However, the execution is interruppet by the following error:
EPERM: operation not permitted, stat 'C:\Users\LORENZ~1\AppData\Local\Temp\electron-packager\darwin-x64\HyperloopUPV-darwin-x64\HyperloopUPV.app\Contents\Frameworks\Squirrel.framework\Versions\Current'
It seems is like a permissions error, so i have run with administrator permissions and abilitate symlink, but doesn't work
I ve run it with the DEBUG variable and these are the final statements before the error
 electron-packager Running asar with the options {} +1m   electron-packager Renaming Electron to HyperloopUPV in C:\Users\LORENZ~1\AppData\Local\Temp\electron-packager\darwin-x64\HyperloopUPV-darwin-x64\Electron.app\Contents\MacOS +9s   electron-packager Renaming Electron Helper to HyperloopUPV Helper in C:\Users\LORENZ~1\AppData\Local\Temp\electron-packager\darwin-x64\HyperloopUPV-darwin-x64\Electron.app\Contents\Frameworks\Electron Helper.app\Contents\MacOS +15ms   electron-packager Renaming Electron Helper.app to HyperloopUPV Helper.app in C:\Users\LORENZ~1\AppData\Local\Temp\electron-packager\darwin-x64\HyperloopUPV-darwin-x64\Electron.app\Contents\Frameworks +16ms   electron-packager Moving C:\Users\LORENZ~1\AppData\Local\Temp\electron-packager\darwin-x64\HyperloopUPV-darwin-x64 to release\HyperloopUPV-darwin-x64 +16ms EPERM: operation not permitted, stat 'C:\Users\LORENZ~1\AppData\Local\Temp\electron-packager\darwin-x64\HyperloopUPV-darwin-x64\HyperloopUPV.app\Contents\Frameworks\Squirrel.framework\Versions\Current'
To Reproduce
I ve tried in a pc with mac and it works perfectly, so the prolem mus t be in my pc with some local variable, there is no vpn or something similar, issue related with #933

I'm closing this as a duplicate of #933. The advice in these comments are still applicable, although you can add GitHub Actions to the CI list.

I read the comments, but no one  works.

It works for me when I set up GitHub Actions with a macOS instance to package a darwin target.

could you explain better please? i don't understand what you mean exactly

Realistically speaking, you can only create Electron apps for macOS targets on macOS hosts. This is for at least a couple of reasons, some of which I will mention here:

Windows support for symlinks (which are used in the macOS Electron.app folder structure) works very inconsistently.
As stated in the code signing documentation, recent versions of macOS have made it very difficult to run unsigned apps in their operating system, and the only OS where you can sign apps for macOS is, unsurprisingly, macOS.

So the options you have to create a macOS-targeted app (i.e., electron-packager --platform=darwin) are:

Buy a Mac
Use a continuous integration provider which has macOS support

It works for me when I set up GitHub Actions with a macOS instance to package a darwin target.

This means that I can set up GitHub Actions to use a worker with macOS installed and run Electron Packager to bundle, customize, sign, and notarize a macOS-targeted app.

If you need more assistance, please use one of the community forums.

This is nonsense I have been able to build electron apps targeting darwin on linux machines using circleCI

@VictorCallejas you can certainly build it on Linux machines, but you can't sign/notarize it there.

That makes sense

@malept thanks for the inputs, you know how i can build it without notarize it?

@LorenzoDelpi if you're distributing your app to customers using macOS, I highly recommend that you sign and notarize your app. Otherwise your users' experience installing and running your app the first time will not be good.
That being said, if you understand the risks and you'd like to package your app without signing/notarization on a Windows host for macOS targets, use WSL instead of CMD/powershell/git bash.

@malept @VictorCallejas  The problem is with the temp folder and the permissions inside it to move files. I' ve solved the problem with specifying a new temporary director with --tmpdir.
Maybe it could be a w10 problem with the permission system, i always run it with admin but idk


[Internal] [Bug] Modify SRA / Genbank / Genome Submission Inputs

What is the Bug Related To? Please Provide a Description.
Need to allow SRA / Genbank / Genome to run without annotated GFF files (already exists for SRA but needs to be modified where if other databases are specified that required GFFs, it will still run SRA) (same logic needs to be applied for Genbank / Genome)
Place an :x: in a Box that Best Matches the Bug's Importance:

[X] 1 - Most severe (a full-break in core function)
[] 2-4 - Moderate (break for a particular aspect/feature) (how integral is the broken feature?)
[] 5 - Least severe (non-functional issue, such as inconsitency / error in documentation or administrative in nature)

Please Complete the Following Information:

OS: [e.g. iOS]: N/A
Browser [e.g. chrome, safari]: N/A
Version [e.g. 22]: N/A
Run environment (container, cloud service, HPC, platform, etc.): N/A

Please Outline Necessary Steps to Replicate Bug (Go to.. Click on... Install the following... etc.):
Running the pipeline through NF with certain configurations of databases specified and without GFF files available.
Any Additional Context or Information? Has There Been Any Progress Made So Far Towards this Request? Any Concrete Instructions to Resolve the Bug or Helpful Resources to Reference? Screenshots or Logs?
N/A

Initial / general changes for the NF wrapper has been made to account for input differences between different databases, within a local branch. Planning a follow-up meeting with NCBI to iron out the best pathway forward, specifically for different pathogen / databases interactions (currently and for our future-plans).

Merged into staging. Closing for cleanliness.


Should be an option to disable focus management

I’m working on a Table Of Contents feature for EditorJS, and as part of this I’m scrolling to heading blocks in the editor, which have their own event listeners and focus handling. As part of this handling, attempts to control elements controlled by the editor seem to throw errors.
So, while I appreciate the efforts to improve accessibility etc., the focus management can also be in the way in more advanced contexts, and it’d be nice to have a way to disable it.
Thoughts?

I’d be open to this as a boolean prop. Feel free to make a pr and I’ll take a look. It should be a fairly straightforward addition.


Add setup instructions to README

What Changed
✅  Added Usage instructions in README along with optional configurations possible with the plugin
Proof of Work

How about also adding the License and About sections here.

Added 8526e3517ac1287849694a1e08537f47384e8b6f ✨


SmartGit doesn't show it's main window

SmartGit (a git client written in Java using SWT) doesn't show a window when run in Hyprland. If it shows a dialog (after a crash for example), the dialog shows up, but the main window doesn't. The process keeps running and it's logs indicates, that it thinks it has a window. hyprctl clients doesn't show a window either, though.
Running it on sway works, so it's probably not a generic aversion to wayland or tiling wms.
I tried a directly installed version and the flatpak version - both on ArchLinux.
Additional Info: after many many many restarts it showed the main window once, but then failed to show one of its modal child windows, so I was stuck and couldn't continue (since I couldn't close the non-showing modal window).

Seeing the same behaviour in the Warframe launcher. After a semi-recent change the client is invisible and is not represented in hyprctl clients. Is certainly running however, as it is playing the ambient music.

After a semi-recent change

if this change is a hyprland change, please do a git bisect

Just tested this with the Warframe launcher,<PHONE_NUMBER>67f9b6513edf1c28478bec12d709a0 is the last working commit, fcf1bafb098c8f756550195269dcd95202124277 is the first broken commit.

Just tested this with the Warframe launcher, 6688421 is the last working commit, fcf1baf is the first broken commit.

Thanks, was struggling to get it to build!

Just tested this with the Warframe launcher, 6688421 is the last working commit, fcf1baf is the first broken commit.

In that case it might be a different problem than mine, because I also encountered it in v0.21.0beta.

Apparently it can be reproduced with Eclipse IDE as well, when launching it using GDK_BACKEND=x11 ./eclipse. The workspace selection dialog and the splash screen show up, but the main window doesn't.

can you now check with 89e7d812c80038083acdca85a5ba08ad97b2d0ea whether it appears in hyprctl clients?

@vaxerski The windows show up in the list now. For example like this:
Window 704fb270 -> :
	mapped: 0
	hidden: 0
	at: 0,0
	size: 0,0
	workspace: -1 ()
	floating: 0
	monitor: -1
	class: unmanaged X11
	title: SmartGit 22.1.4
	pid: 5511
	xwayland: 1
	pinned: 0
	fullscreen: 0
	fullscreenmode: 0
	fakefullscreen: 0
	grouped: 0
	swallowing: 0

Window 704fdfc0 -> :
	mapped: 0
	hidden: 0
	at: 0,0
	size: 0,0
	workspace: -1 ()
	floating: 0
	monitor: -1
	class: unmanaged X11
	title: Welcome to SmartGit
	pid: 5511
	xwayland: 1
	pinned: 0
	fullscreen: 0
	fullscreenmode: 0
	fakefullscreen: 0
	grouped: 0
	swallowing: 0

"unmanaged X11" sounds suspicious.

mhm. I have an idea.

fixed with 7e523e4d5e526bdf5ab86a8f4fe37efeef356707 :)

@vaxerski Can confirm. The windows show up fine now. Thank you very much :smiley:


Index / unique not created

Hi,
when I use the plugin to create schemas from a base schema it seems the ensureIndex function of Mongoose is not triggered. This is seen in the logs when I activate it with mongoose.set('debug', true);
This leads to the expected behaviour:
var MySchema = new Schema({
    name: {
        type: String,
        required: true,
        trim: true
    },
    moreinfo: {
        type: String,
        required: true,
        index: {
            unique: true,
            background: false
        }
    }
});

This doesn't:
var MySchema = baseScheme.extend({
    name: {
        type: String,
        required: true,
        trim: true
    },
    moreinfo: {
        type: String,
        required: true,
        index: {
            unique: true,
            background: false
        }
    }
});

In the log output you can see it:
Mongoose: myschema.ensureIndex({ moreinfo: 1 }) { unique: true, background: false, safe: undefined }

I think this is fixed on the current version (0.2.0); let me know if it is not


Ksonnet 1.0 plan/road map?

Hi,
What is the plan for releasing Ksonnet 1.0? When can we expect the CLI to be fully stabilized?
Thanks,
Richard

the roadmap is in the repo:
https://github.com/ksonnet/ksonnet/blob/master/design/roadmap.md
beyond what is in that document we don't expect significant changes to the CLI, but that being said we also don't have a current timeline to 1.0 - PRs are very welcome though :-)


What's the difference between this and https://github.com/SonarSource/eslint-plugin-sonarjs

Are these 2 separate plugins?

There are some rules in SonarJS unavailable in eslint-plugin-sonarjs, so I extract rules in SonarJS to this project.
Take sonar/deprecation as example.
And you can see the rules list at https://github.com/rx-ts/eslint-plugin-sonar#rule-list and https://github.com/SonarSource/eslint-plugin-sonarjs#rules

Does this work well with Typescript projects? I already have eslint rules applied with a few TS projects, I wanted to know if there will be conflicts.

It uses typescript inside as peer dependency, so it works well with TypeScript of course.

I wanted to know if there will be conflicts.

I think you should try it by yourself, I use it in my own and company projects, and it works well without conflict. And if you just find conflicts, you can easily disable one of the rules from it or other eslint plugins.

Cool but there seems to be some rules that are in sonarjs and not in this repo like for example https://github.com/SonarSource/eslint-plugin-sonarjs/blob/master/docs/rules/cognitive-complexity.md. I didn't see that mentioned in your README.md.

SonarJS and eslint-plugin-sonarjs are different, this plugin extracts rules from SonarJS which could be considered as additional rules.


Synology DSM 7.0-41882 working OK

No issue. Just wanted to pitch in that I upgraded my Synology DS1517+ NAS to the brand new DSM 7.0 (what could possibly go wrong?) a week ago.
No problems whatsoever with Crashplan. A few other apps died though..

Thank you for the good news!  I was hoping to hear just that as I'm eager for General Release Day with DSM 7.

Thanks for posting. I was looking for exactly that. Can you please elaborate just a bit?
Did you just…

Update to DSM 7
Log in to find the Docker container intact
Start it and it just worked?

Or was there some container recreation and adjustment that needed to be done post-update?
Thanks!

There was a new version of Docker and some other packages (and some other other were incompatible with DSM 7 as mentioned).
After upgrading everything it just worked, no problems.

Brilliant!

Great news!

I updated my Syno DS918+ a week ago to DSM 7.0-41890, and my crashplan container does not backup any files. All backup sets say that 0 bytes are to be backed up. When I try do edit the backup set, and click to browse the files, I only get empty root folders, and there is a mark on volume1, that "files are missing".
Is there some security change that needs to be performed. I know that I had to change some credentials to get my PLEX app working on the new DSM 7

Well found the solution: Deleted the container and recreated it, using my original commands. for some reason the volume1 mount point had disappeared.
After recreating, i had to initiate a new scan from the web console

See https://github.com/jlesage/docker-crashplan-pro/issues/310.  You probably mapped /volume1 directly to the container, which doesn't work anymore.

Can confirm.  I had to upgrade my Synology DS1515+ to a DS1520+ due to a hardware clock failure, and in setting up the new DS1520+ with the original hard drives I mistakenly upgraded to DSM 7.0.1-42218 instead of keeping it at DSM 6.  The container works without issue on DSM 7.0.1-42218, even with Container version 2.16.1 (Crashplan Client version <IP_ADDRESS>) (upgraded via Watchtower).  No under-reporting of backup size; and no continuous synchronizing with the latest container/client version.


Can't build in Visual Studio

I forked the repo and opened the solution in Visual Studio 2019.
Building within VS causes it to skip the project entirely, and building with dotnet build -c Release throws namespace errors, due to the /lib references not working.
I tried removing and re-adding the references, but VS gives the following error:
I fixed this. Figured out I had to replace the files in the /Lib folder. This was not entirely clear from the README.md, and might benefit from rewording. The project still gets skipped when building in VS, and it's a real mess tbh.
Now it builds just fine using dotnet build -c Release though. 🙂

I'm also getting this error upon loading the solution: (Still getting this after fixing the references)

Also, why no .gitignore file? The repository state becomes a mess as soon as one attempts a build. 😄
(Or even load the solution.)

I developed this project in VS Code. I won't be able to troubleshoot VS specific issues. Please detail the steps required to fix the /Lib. I will update the README.md accordingly.
I've added a .gitignore. Please sync your fork and pull down the changes.

That makes sense, although build folders and such are still a tad messy.
I assumed the files were just 1KB placeholders that you created yourself, is that not the case?
I just replaced them with fresh copies from my current PowerToys Run install.
Nice, I'll try to fix the VS compatibility later. The .gitignore should help.
Thanks for the quick response time!


Removing a member from a cluster, does not remove it from discovery

@marvell The public discovery service is for bootstrapping your cluster for now. It does not accurately reflect the runtime configuration of the cluster.


Eufy stream keeps saying unauthorised

Please check Services, Known Bug / Issues and Troubleshooting over here first: https://github.com/fuatakgun/eufy_security/blob/master/README.md
Describe the bug
My eufy 2K pan and tilt cam will not stream keep saying unauthorised in Home Assistant.log
To Reproduce
Steps to reproduce the behavior:
2022-12-26 22:58:12.775 ERROR (stream_worker) [homeassistant.components.stream.stream.camera.kitchen_camera] Error from stream worker: Error opening stream (HTTP_UNAUTHORIZED, Server returned 401 Unauthorized (authorization failed<EMAIL_ADDRESS>2022-12-26 22:58:22.999 ERROR (stream_worker) [homeassistant.components.stream.stream.camera.kitchen_camera] Error from stream worker: Error opening stream (HTTP_UNAUTHORIZED, Server returned 401 Unauthorized (authorization failed<EMAIL_ADDRESS>2022-12-26 22:58:43.099 ERROR (stream_worker) [homeassistant.components.stream.stream.camera.kitchen_camera] Error from stream worker: Error opening stream (HTTP_UNAUTHORIZED, Server returned 401 Unauthorized (authorization failed<EMAIL_ADDRESS>2022-12-26 22:59:13.189 ERROR (stream_worker) [homeassistant.components.stream.stream.camera.kitchen_camera] Error from stream worker: Error opening stream (HTTP_UNAUTHORIZED, Server returned 401 Unauthorized (authorization failed<EMAIL_ADDRESS>2022-12-26 22:59:53.249 ERROR (stream_worker) [homeassistant.components.stream.stream.camera.kitchen_camera] Error from stream worker: Error opening stream (HTTP_UNAUTHORIZED, Server returned 401 Unauthorized (authorization failed<EMAIL_ADDRESS>Expected behavior
RTSP stream was working fine now it switches between idle and unavailable cannot stream at all
Screenshots

**Hardware Information about Home Assistant
**Hardware Information about Cameras

eufy 2k pan and tild
Live Streaming Protocol: RTSP

Additional context
I believe I have tried everything

Same issue here, my Eufy C24 works fine with this integration as well as the generic camera integration.  My three pan cams do not.

Well to be honest I never had static IP on it set up I never had to then
after a update I lost connection to the camera and could no longer stream
it was saying that it was an IP issue so I set it to static IP to the IP
address it was complaining it couldn't connect to now it says it's
unauthorised and I know it's correct I can control the camera in every way
every entity every switch every binary are just simply can't stream
On Mon, 26 Dec 2022, 11:18 pm IronicRebound77, @.***>
wrote:

Same issue here, my Eufy C24 works fine with this integration as well as
the generic camera integration. My three pan cams do not.
—
Reply to this email directly, view it on GitHub
https://github.com/fuatakgun/eufy_security/issues/588#issuecomment-1365132964,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXFHHS6OILR57B6IMMYOEN3WPGEHPANCNFSM6AAAAAATJTMCOE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm starting to think it has to be something to do with the RTSP add-on

@Jezza1232 @IronicRebound77
Eufy updated the firmware on those cameras which caused the 401 unauthorized error for the rtsp streams. I have several of the cameras, and the only one that I don't leaving running 24/7 didn't get the firmware update around 2am a couple weeks back, so it doesn't have the 401 error. I contacted Eufy and their support said their engineers are aware of the issue and it'll be fixed with the next firmware update. They stated hopefully by the end of the month. I disabled rtsp authentication on the cameras that were affected, so I can stream without errors. Do so at your own discretion.

@Jezza1232 @IronicRebound77
Eufy updated the firmware on those cameras which caused the 401 unauthorized error for the rtsp streams. I have several of the cameras, and the only one that I don't leaving running 24/7 didn't get the firmware update around 2am a couple weeks back, so it doesn't have the 401 error. I contacted Eufy and their support said their engineers are aware of the issue and it'll be fixed with the next firmware update. They stated hopefully by the end of the month. I disabled rtsp authentication on the cameras that were affected, so I can stream without errors. Do so at your own discretion.

@ema-dev  Thanks, yes turning off security on the cameras does allow the stream.

turning off RTSP security is required. thanks

turning off RTSP security is required. thanks

Eufy has removed the ability to disable RTSP security in the <IP_ADDRESS> firmware update. RTSP Security Authentication now has only two options - Basic and Digest.


Address Upload

Summary
Support address upload, which encompasses the following changes to the georeferencing workflow:

Rewrite most georeferencing labels
Limit options to address and lat/lon
Fix drag & drop workflow for georeferencing

Support drag & drop from all screens (map/data library, map, table)
Ensure georeference popup is shown on first upload of  custom dataset

Move georeference option to Actions menu
Update geocoding polling item labels

Reviewing now..

That should address the comments so far - let me know if these latest changes look ok. Thanks

One observation:- When georeferencing, if you select same column for all 3 options -  address, country and then state/province, it fails to geocode one address (using our small address test). Same works fine if you just select that column for address option. Obviously, we can't control this in the UI in any way. User needs to select proper columns here.

I tested Street/Country/State.. UI for both single/batch.. Looks fine.. It does show proper show button based on the results.

One issue I notice in Latitude/Longitude UI -
If my latitude value is beyond standard limit for lat (I am sure that same is applicable for longitude value although I did not test it!) range (in theory, for latitude, +/- 90, with web mercator projection - +/- 85.xxxx approximately! For longitude - /+ 180), It will skip the bad data values on map obviously. But, we have no check on that! So SHOW button does not show up!  On table tab, all rows are shown obviously (which is fine!). Should we not have a check on proper lat/long values??

I have emailed lat/long test-sample that explains the point mentioned above.

I did not test georef overwrite toggle since by default it is not enabled. As long as it is tested by you by turning on that feature flag, we should be good.

How are you adding feature flag(s) with appropriate default(s)?

I have reviewed all pieces now.. Overall, all changes look good.
Please review my comments from today.

@vmanohar1 The feature flags will be off by default for the intended behavior. They can be enabled manually if needed.

ok.. Approved for merge @tylerparsons as discussed.


[FEATURE] - Improve Navbar UI and Enhance Hover Effects

Description
The current navbar UI lacks visual appeal and the hover effects on the navbar buttons are not engaging enough. This can lead to a suboptimal user experience and may hinder the overall usability of the website/application.
I propose to improve the navbar UI and enhance the hover effects on the navbar buttons. The goal is to create a visually appealing and interactive navigation experience for users.
The following changes should be implemented:

Redesign the navbar to have a modern and aesthetic appearance.
Implement smooth transitions and animations for the hover effects on the navbar buttons.
Enhance the visual feedback when a button is in the active or focused state.

I would like to work on this issue under GSSOC'23.

@narayan954 can you assign me this task.

@narayan954 can you assign me this task.

Hi @Amitavpanda , actually this issue is already under progress. Also as the sidebar is in place now, we're cleaning the navbar. You can await on this issue for some days to further improve if you want.


Configure with mode & value:Doc

Hi fantastic maintainers of CodeMirror !
I found a minor issue when configuring a CodeMirror instance with both a document and mode in constructor options. It seems that mode option is not taken into account, unlike other options:
const elt = /* some div element */
const doc = new CodeMirror.Doc('//Hello World !\nconst a = 1;');

// case 1 : does not work
const cm = new CodeMirror( elt , { value: doc, mode: 'javascript', readOnly : true } );

// case 2 : does work
const cm = new CodeMirror( elt , { value: doc, readOnly : true } );
cm.setOption('mode' , 'javascript' );

To reproduce (Safari and Chrome) :  cmbug.zip

Not sure how that wasn't caught in the several years since first-class Docs were introduced... but attached patch fixes it!

Great!


In flow conditions

In the case of a regional model, we need to have the appropriate inflow conditions. These should ideally be one of the following:

Zeros for all variables
User-supplied constants for all variables
Spatially variable field, read in from file
Time and space varying field, read in from file

Main tasks:

[ ] Make sure dynamics and advection can handle inflow on the boundaries
[ ] Implement input for boundary conditions, as per the list above

We'll do this as part of the MPI work: The inflow conditions are set on the  halo and the dynamics does nothing extra.


Update Multi-cluster docs and templates for leader cluster auto update

Signed-off-by: Jianjun Shen<EMAIL_ADDRESS>

@luolanzone @hjiajing : could you help review and verify the changes? I plan to create another PR to add anctl examples to quick-start.md.

/skip-all


WIP: Source Handler Positive ACK handling

Codecov Report

Merging #121 (4507339) into main (1712902) will increase coverage by 0.38%.
Report is 2 commits behind head on main.
The diff coverage is 97.24%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
+ Coverage   78.71%   79.09%   +0.38%
==========================================
  Files         112      112
  Lines        7783     7899     +116
==========================================
+ Hits         6126     6248     +122
+ Misses       1657     1651       -6

Flag
Coverage Δ

unittests
79.09% <97.24%> (+0.38%)
:arrow_up:

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

Files
Coverage Δ

tests/cfdp/test_src_handler.py
99.46% <100.00%> (+0.03%)
:arrow_up:

tests/cfdp/test_src_handler_acked.py
100.00% <100.00%> (ø)

tests/test_countdown.py
100.00% <100.00%> (ø)

tmtccmd/cfdp/handler/dest.py
82.36% <100.00%> (+0.04%)
:arrow_up:

tmtccmd/cfdp/mib.py
81.11% <100.00%> (+0.42%)
:arrow_up:

tests/cfdp/cfdp_fault_handler_mock.py
84.61% <80.00%> (+3.36%)
:arrow_up:

tmtccmd/cfdp/handler/__init__.py
45.96% <0.00%> (ø)

tmtccmd/cfdp/handler/source.py
90.64% <98.66%> (+1.28%)
:arrow_up:

tmtccmd/util/countdown.py
94.73% <88.88%> (-1.19%)
:arrow_down:

tmtccmd/tmtc/ccsds_seq_sender.py
93.54% <0.00%> (ø)

... and 14 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Implement Largest Histogram using Stack in C++ Language

Feature
I would like to implement the Largest Rectangular area under Histogram using stack in C++.
@nakul-19 @tarun26091999 I would like to contribute to this under GSSOC'21. Please assign it to me.

present at #161


could not find u16cstr in the list of imported crates

Any idea why this error  ?
--> src\main.rs:17:9
|
17 |         pdcstr!("Test.Program, Test"),
|         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ could not find u16cstr in the list of imported crates
|
= note: this error originates in the macro pdcstr (in Nightly builds, run with -Z macro-backtrace for more info)

Worked after changing cargo.toml to
netcorehost = "0.3.1"
path-absolutize = "3.0.10"
wchar = "0.10.1"

What was your Cargo.toml previously?

netcorehost = "^0.4.0"
path-absolutize = "^3.0.10"
wchar = "^0.10.1"


adjust weight for TCP service,the TCP connection will be interrupted.

Bug Description
Adjust the traffic proportion of different versions of the TCP protocol service through virtualservice. After the adjustment, the TCP connection will be interrupted.
Version
istio verison 1.11
TCP protocol : ssh

Additional Information
apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  name: ssh-test-ingress
spec:
  hosts:
    - "*"
  gateways:
    - ssh-test
  tcp:
    - route:
        - destination:
            host: ssh-test
            subset: v1
            port:
              number: 2222
          weight: 100
        - destination:
            host: ssh-test
            subset: v2
            port:
              number: 2222
          weight: 0

apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  name: ssh-test-ingress
spec:
  hosts:
    - "*"
  gateways:
    - ssh-test
  tcp:
    - route:
        - destination:
            host: ssh-test
            subset: v1
            port:
              number: 2222
          weight: 50
        - destination:
            host: ssh-test
            subset: v2
            port:
              number: 2222
          weight: 50

Affected product area

[ ] Docs
[ ] Installation
[X] Networking
[X] Performance and Scalability
[X] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

Is this the right place to submit this?

[X] This is not a security vulnerability
[X] This is not a question about how to use Istio

This is expected due to how envoy works, it wi drain connections when listener changes

My friend tested the Tcpproxy of envoy. Modifying the traffic ratio will not interrupt the connection. I thought it was handled by istio.

With Istio does it happen if you switch weights from 50/50 to 60/40 or anything that is not 0?

Yes, 100/0 is special, others have also been tested, such as 25/75 -> 50/50, as long as the proportion is adjusted, the Tcp connection will be interrupted

We have --drain-time-s 45 --drain-strategy immediate  set,  so after 45s, the connections can be closed

Any cons do you think?

Where are these two parameters set?

We have --drain-time-s 45 --drain-strategy immediate  set, so after 45s, the connections can be closed

@hzxuzhonghu Where are these two parameters set?

This is per proxy config, and it allows customization


shallow does not use existing context

I use recompose to set up my context. I know I can pass options.context to both shallow and mount but in my case setting up the context is complex and it's much more convenient to use withContext.
The unit under test is a component called Component which uses foo from the context.
The unit test creates a context this way.
// use recompose's withContext to setup the context
const ComponentWithContext = witContext(
  {foo: PropTypes.string}, ()=>{foo: 'test'}
)(Component);

const wrapper = shallow(<ComponentWithContext/>);

When I mount my component, this works fine; the context is available to the Component. But when I use shallow the context is empty.
Why doesn't it work with shallow as well?

I believe this is a duplicate of #664, assuming that getChildContext is how withContext is implemented.
But also, since withContext is an HOC, you need shallow(<ComponentWithContext />).dive() as well.

Fixed in #664.


Propose Traits::Access -> AccessTraits

Drop Traits:: nested namespace.
Old syntax remains valid for now.
I intend on reverting 459023d before we merge.  It is mainly there to demonstrate that it works.

@dalg24 Can you please explain the reasoning behind the change, what limitations/drawbacks the current version has, and directions for adding traits in the future?

@dalg24 Can you please explain the reasoning behind the change, what limitations/drawbacks the current version has, and directions for adding traits in the future?

Nested namespace Traits:: made thing awkward internally

closing Details:: to open Traits::, then reopen Details:: in ArborX_AccessTraits.hpp
having to qualify the tags in when using the traits

Name is non descriptive.
Also the only place in our API where we have a nested namespace.

No limitations.  No change in directions.

Guess what compiler I was using...
I will figure out the errors with nvcc and pgi later if we agree we want that PR to go through.

I agree that Traits should not be a namespace. And I agree that it should be part of ArborX namespace and not ArborX::Details, as it is part of user interactions.


(Weather) Tornado incompatible with dzn Weapons Holder

After the tornado vanishes a lot of errors are spammed:
`20:07:46 Error in expression <lder

backpackCargo _holder
);

(count _items <= _allowedNumber)>
20:07:46   Error position: <_items <= _allowedNumber)>
20:07:46   Error count: Undefined variable in expression: _items
20:07:46 File dzn_WHC\functions\fnc_isAllowedNumberOfItems.sqf, line 17`

should be fixed with v2.0.13


Question Of  Julius

Hi, my name is Tanaka of Aires Electronics Industry.
I am a beginner in voice recognition and Julius.
It may be a question you don't care about,
Thank you for your understanding.
This time, we have inquired about
Regarding the accuracy of Julius's voice recognition.
Raspberry Pi, like Alexa
We are in the process of building it for voice operations.
*Due to the influence of this new type of corona, we have developed a voice recognition technology with the keyword touchless.
We are verifying whether we can propose to our products or customers.
Create your own dictionary with Julius
I try to read that dictionary,
I am in trouble because the voice recognition does not respond well.
For example, I have registered the word "aires".
Even if it's not the word "aires,"
I recognize it as Aires.
About this solution
Are there any sites or books that you can refer to?
I'm sorry I'm busy
Thank you.

Hi. Which language are you trying to add to your solution?
The best way to add words to recognition is to use grammar. In case of normal speech recognition with language model it is more difficult as the word you are trying to add should be a part of language corpus.

Thank you very much for contacting me.
The language is Japanese.
It means using "Grammer-Kit" instead of "dictation-kit".
I understand. I will try.
If you have any questions, please let me know.
Thank you
From: Lesław Pawlaczyk<EMAIL_ADDRESS>Sent: Monday, July 27, 2020 5:07 PM
To: julius-speech/julius<EMAIL_ADDRESS>Cc: YasuyukiTanaka999<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [julius-speech/julius] Question Of Julius (#137)
Hi. Which language are you trying to add to your solution?
The best way to add words to recognition is to use grammar. In case of normal speech recognition with language model it is more difficult as the word you are trying to add should be a part of language corpus.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/julius-speech/julius/issues/137#issuecomment-664188557 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQNGIV6N6IETT2ZVMDSYQBTR5UYSZANCNFSM4PIMCPNA .