Fail to set num_threads in Lilly Cluster.

Recently, when I am working in the mbircone repository in Lilly cluster.
I noticed that setting num_threads in the recon function failed.
Therefore, I tested this feature of svmbir in Lilly cluster using below code.
for num_threads in [1,2,4,8,12,24]:
recon = svmbir.recon(sino, angles, T=T, p=p, sharpness=sharpness, snr_db=snr_db,num_threads=num_threads)

1 Threads: Reconstruction time = 38621 ms (iterations only)
2 Threads: Reconstruction time = 38096 ms (iterations only)
4 Threads: Reconstruction time = 38458 ms (iterations only)
8 Threads: Reconstruction time = 38306 ms (iterations only)
12 Threads: Reconstruction time = 38042 ms (iterations only)
24 Threads: Reconstruction time = 38980 ms (iterations only)

It shows that setting num_threads in the recon function failed to change OpenMP's threads.
Currently, we set the number of threads for OpenMP by changing the environment variable.
However, there's a hierarchy of places to change for the number of threads, and the environment variable is at or near the bottom.
The solution for this is to use OpenMP function openmp.omp_set_num_threads(num_threads) directly to set num_threads.
This function can be imported from OpenMP in Cython.
I implemented this in a branch of mbircone repo, please see attached link.
https://github.com/cabouman/mbircone/compare/threads_setting
After this fixed,

1 Threads: Reconstruction time = 61837 ms (iterations only)
2 Threads: Reconstruction time = 32082 ms (iterations only)
4 Threads: Reconstruction time = 17849 ms (iterations only)
8 Threads: Reconstruction time = 10009 ms (iterations only)
12 Threads: Reconstruction time = 7179 ms (iterations only)
24 Threads: Reconstruction time = 4877 ms (iterations only)

The problem doesn't happen on any of my machines so I can't reproduce it, but adding the call to omp_set_num_threads() doesn't hurt anything.
@DamonLee5 Can you check that the version in the "set_threads" branch fixes the problem on Lilly?

Wenrui, I think you told me the same problem happened on your mac.  If that's the case could you see if the fix worked there too, and also mention the OS version?

OK, so it was running 8 threads in each of those cases. Got it.


[Request] Common Component Sizes

One of the things that Semantic UI does well is component sizes, where all components come by default in multiple sized version.

Our sizing follows Material Design guidelines. Anything outside of that is on developers to implement on their own.
We understand other frameworks and libraries provide this functionality. However, it goes against what MDL is trying to do. Therefore we won't be implementing this functionality unless multiple sizes are defined by the MD specification.
Thank you.


Communications: Implement Quick Emoji Reactions via Hover on Messages

Problem
Users are able to react to messages using emojis, which involves clicking on the reaction button and then selecting an emoji from a list. This method, while functional, requires multiple steps and can interrupt the flow of using the application.
Proposed System
This issue proposes the enhancement of our current messaging feature by introducing "quick reactions." Currently, users can react to messages with emojis by selecting them from a menu. The goal is to streamline this process by enabling quick reactions when hovering over the reaction part of a message.
UI/UX Design

Ive looked into it and the currently used emoji-library doesn't have an option for a quick-reaction hover, so we would have to implement the hover component manually.
Which emojis should be shown, a fixed set of emojis, or the most used emojis of the user?

I don't think we need personalized emojis here, since our goal is to provide a communication service tailored to education rather than a personal messaging service. "Thumbs up", "tick" and maybe one more useful emoji should be great

@egekurt123 How far along are you with #8591 ? I suppose there might be merge conflicts if we work on this simultanously

I just started working on it, should be done in max 2 weeks

Alright, Ive also started working on it, but I dont think there will be big issues once your changes have been applied. I will just readjust then 😄


Any chance you are the hazzaaclark which released previously chuchu?

There was an IDA plugin for dreamcast, but its gone missing on github. Any chance if that was you could you reshare it?

Hi - thank you for your message
While that was me, I have long since deleted that repository due to not feeling as though it was needed
Not to mention, the code barely worked and or functioned
Should there ever come a point whereby I want to continue those developments, you can always keep tabs on my profile for any updates pertaining towards that
God Bless

Appreciate your reply and will follow along with your public progress on these projects you have! thanks for pursuing this stuff!


[EMB-251] Tags for everyone

Purpose
Quick File tags weren't showing for unauthenticated users
Summary of Changes
Remove if block around tags widget that hid it from unauthenticated users. There's already a condition on the component to not let people edit tags if they're not contributors, so hiding the tags was just a step too far. Everyone deserves tags.
Side Effects / Testing Notes
Shouldn't be.
Ticket
https://openscience.atlassian.net/browse/EMB-251
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Coverage remained the same at 55.508% when pulling b4347de4f2ddb44e04cb3340d602f11404fa3189 on brianjgeiger:feature/show-tags-to-everyone into 7d1fdecdcae1d62a4d7a909df4f7937bec71a982 on CenterForOpenScience:develop.


partners/astradb: Add caches to langchain-astradb

Fixed conflicts and rebased.

Package has moved! Could you reopen here? https://github.com/langchain-ai/langchain-datastax


req.raw.cf types broken in 3.7.4

What version of Hono are you using?
3.7.4
What runtime/platform is your app running on?
Cloudflare Workers
What steps can reproduce the bug?
import { Hono } from 'hono';

const app = new Hono();

app.get('/country', async (context) => {
	const country = context.req.raw?.cf?.country ?? '';
	context.text(`Country: ${country}`)
});

export default app;

What is the expected behavior?
In version 3.7.3, context.req.raw?.cf?.country ?? '' does not have any issues. TypeScript is happy.
What do you see instead?
In version 3.7.4, context.req.raw?.cf?.country ?? '', the following TypeScript error is thrown:
src/index.ts:10:35 - error TS2339: Property 'cf' does not exist on type 'Request'.

10  const country = context.req.raw?.cf?.country ?? ''

Additional information
No response

This appears to also be due to the webworker types included as reported in #1546.

@Cherry
Thanks. I'll fix it soon.


Does misaligned exception in nottaken branches cause an exception?

I couldn't understand the misaligned exception when the branch is not taken and the target has a misaligned address. Is there any explanation in the spec? If not I think it should be added to the spec.

This is from an older release of the spec:

So, no, there will be no exception in that case.

Thank you. I think it should be in the latest version of the spec too.

it should be in the latest version of the spec

It is: https://github.com/riscv/riscv-isa-manual/blob/87edab70ddd2fedc6d01603a65f393163025633f/src/rv32.adoc#L648-L653


spec incorrectly claims that PUT /_matrix/federation/v1/send/{txnId} response has a [200, ...] wrapper

this is a bug. See https://github.com/matrix-org/matrix-doc/pull/1802#issuecomment-521959258

Unlike the other endpoints mentioned in #1418 / MSC1802, this is a spec bug: PUT /_matrix/federation/v1/send does not have a [200, ...] wrapper in the response.


Is adding Text and Json to scenario possible?

Hello @wswebcreation ,
Thank you for your outstanding work on this and other multiple-cucumber-html-reporters.
It appears whenever this.attach (cucumber World.attach) is used to append Text or JSON in a cucumber step, the addition gets ignored/suppressed by WDIO and subsequently also missing from your json output.
Running naked cucumber from command line (no WDIO) with same dependencies produces desired entry in the cucumber json output:
            "embeddings": [
              {
                "data": "Session ID: xxxxxxx",
                "mime_type": "text/plain"
              }
            ]

example of failing embedding in a sample step:
Given(/^background prep$/, background);
async function background(): Promise<void> {
    await this.attach('Session ID:' + browser.sessionId);
}

Environment:

Node.js version: 8.12
NPM version: 6.4.1
webdriver.io version: ^4.14.1
wdio-cucumber-framework version: ^2.2.8
wdio-multiple-cucumber-html-reporter version: ^0.3.2

I've scoured forums looking for any clues but not a single thread exists mentioning text or json. There's nothing on WDIO api docs either (only screenshot capture) Printing session ID, attaching network and console logs could all be very usefully achieved this way as is the case in your other reporters.
Please let me know if you are aware of this being an issue and/or any clues on tackling this. Thanks a lot!
Best, Philip

Hi @jacknjill02
I didn't had time the last month, I still need to check this, hopefully next week

This is now added in version 1.0.1, see here


⚠ Bofrost.es has degraded performance

In 29d4dd6, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1642 ms

Resolved: Bofrost.es performance has improved in 5e8ec39 after 9 minutes.


'profile' scope causes Federated Identity example to loop

Expected Behavior
Allow addition of 'profile' scope.
Current Behavior
If you add the scope to the example, starts infinite redirect loop.
Context
I'm trying to make use of the Federated Identity example and would I'd like to get the UserInfo object populated. However if I add the profile to the registeredClient I get the hang:
In AuthorizationServerConfig:
   RegisteredClient registeredClient =
        RegisteredClient.withId(UUID.randomUUID().toString())
            .clientId("messaging-client")
            .clientSecret("{noop}secret")
            .clientAuthenticationMethod(ClientAuthenticationMethod.CLIENT_SECRET_BASIC)
            .authorizationGrantType(AuthorizationGrantType.AUTHORIZATION_CODE)
            .authorizationGrantType(AuthorizationGrantType.REFRESH_TOKEN)
            .authorizationGrantType(AuthorizationGrantType.CLIENT_CREDENTIALS)
            .redirectUri("http://<IP_ADDRESS>:8080/login/oauth2/code/messaging-client-oidc")
            .redirectUri("http://<IP_ADDRESS>:8080/authorized")
            .scope(OidcScopes.OPENID)
            .scope("message.read")
            .scope("message.write")
            // .scope(OidcScopes.PROFILE)
            .clientSettings(ClientSettings.builder().requireAuthorizationConsent(false).build())
            .build();

fixed via comment on #917. Issue is that sample doesn't;t include
       .oauth2ResourceServer(OAuth2ResourceServerConfigurer::jwt);

and
  @Bean
  public JwtDecoder jwtDecoder(JWKSource<SecurityContext> jwkSource) {
    return OAuth2AuthorizationServerConfiguration.jwtDecoder(jwkSource);
  }

Duplicate of gh-917


Users can continue annotations even if they exceed their assigned tickets

Describe the bug
Suppose a user has x tickets to annotate. If he finished x tickets, he can continue labeling normally, which is inconsistent with the logic of sharing the annotation task among many users.
Reproduction steps

Create a project
Assign users
Login in as an annotator (that has some assigned project)
start annotating until exhausting all the assigned tickets.
You will see that you can continue normally and annotate data examples that maybe allocated to other users (given that some other users has not completed all his annotations).

Expected behavior
Each user, upon annotating all his assigned tickets, must see that the submit button is grayed out.
Additional context
I would be thankful for a detailed description of how to modify the code, and in which specific file, so that the behavior of the application change and conform with the expected behavior mentioned above.

@msharara1998, thanks for your feedback.  This is by design not a bug.  By allowing annotators to label more provides more flexibility.  If the assigned tickets per annotator is fixed, it will not be easy for new added annotators to obtain tickets.  It will also block slack channel unknown user’s annotation job.

@msharara1998 Thanks for your feedback, as it's not a bug and it's a special design to help users flexy labeling, so i will close the issue.


Add ability to define time period to NOT download browsers

It's handy being able to download latest versions of browsers but it's expensive for network and time.
With the majority of tests using the same path for browsers on a system, it would be good to store the last time a browser was downloaded and allow a test to define how long until a browser should be classed as expired.
This would be done based on hours.
seleniumAssistant.downloadBrowser('firefox', 'stable',  24)

If the browser was updated less then 24 hours ago then download browser simple resolves, if the browser was last download more than 24 hours ago it will be downloaded.
This is more reliable than force download true or false.
@jeffposnick thoughts on this?

I am in favor of any steps towards cutting down on browser downloads and reusing what's already there, for a reasonable interval of time.
(I've been curious—are the automated builds on the Travis CI  infrastructure caching/reusing browser binaries?)
One comment is that moving to named parameter via passing in a configuration object might make sense, rather than using positional parameters. So something like:
seleniumAssistant.downloadBrowser({browser: 'firefox', releaseChannel: 'stable', cacheLifetimeInHours: 24});

24 as a third positional parameter is pretty ambiguous—I could see folks reading that and thinking that it's an instruction to download Firefox Stable v24, for instance.

Fwiw, 👍 the idea of moving towards named keys in a configuration object. That's a lot easier to quickly reason when trying to read through what each thing being passed through corresponds to.

Sorry I completely missed these comments.
If anything else needs adding I'll move to a configuration option.


Renaming few remaining packages

@aledbf Did you not rename these on purpose?

This change is

Coverage increased (+0.02%) to 44.193% when pulling 5b37d2b31521e5201dc1850d7ea966c4f650e1bf on nicksardo:pkg-rename into cfef9d82fd45f954efdce82880fc6990f85ee7e5 on kubernetes:master.

@nicksardo yes, to avoid changing all that code :)

/lgtm

Hah, thanks!


[1.12] Add AR routing logic support for ip-per-container feature for Marathon 1.5+

High-level description
Due to Marathon app description syntax change https://github.com/mesosphere/marathon/blob/master/docs/docs/upgrade/network-api-migration.md#example-definitions AR is not able to expose apps requiring ip-per-container feature at /service/ endpoint. This PR fixes this issue.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:
https://jira.mesosphere.com/browse/DCOS-47348
Related tickets (optional)
https://jira.mesosphere.com/browse/DCOS-46381
Checklist for all PRs

[ ] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[ x ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ x ] Read the DC/OS contributing guidelines
[ x ] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label Ready For Review


more visual feedback on saving

this can just come from the sprite rework, no need for fancy UI things

actually, you could also use the new alert system with a few lines


dagui: misc tweaks for frontend optimizing

A grab bag of engine side changes motivated by v3:

Encapsulate all Buildkit-produced spans by default

As shown in the telemetry test diffs, this results in hiding resolve ... spans, which IMO is better since it's 1:1 with Container.from already, not worth the extra noise

Fix version updates being "lost" while viewing an in-flight trace due to re-importing the same span snapshot losing the original Version

(only relevant for v3 web UI)

Add RemainingSnapshots, also only used for v3 for the final sync of all spans so that the the WebSockets connection can hang up
Make use of dagger.io/effect.ids for compatibility with pre-v0.15 traces and for tracking more effects in v0.15+ engines

Caveat: pre-v0.15+ traces will show effects in too many places, because old engines don't keep track of when effects are actually added, they just include them everywhere

Various fixes for the somewhat delicate data pipeline for v3

Fixed running activity tracking (see Activity.AllRunning)
Keep better track of states derived remotely and treat them as source of truth

Bump version when marking a span updated, rather than when taking a snapshot

Basic hygiene ting - don't mutate when snapshotting, mutate when we know we need a new snapshot

Fix "sync" span losing its Passthrough status and showing up in traces
Never consider a span "cached" if it has children or logs (sends mixed messages)
Mark a span "final" if it's the last call and followed by a non-call

NOTE: the commit history here has some changes I made but later backed out because they turned out to not be so simple:

Simplifying Simplify - I think there's still something to this idea, since Simplify is quite expensive and slows down the UI, but didn't have time to see it through

I will likely be on PTO when this merged. I backed out the riskiest change (simplify Simplify) and all tests are passing so I think it's OK, but it's also not a hard requirement for anything so if y'all deem it too risky that's OK. The v3 web UI just points to this via a submodule, so there are no blockers on that side.

@sipsma had to push a commit to fix a broken test - looks like we're now trimming the logs under upload. Do we want to keep that?

had to push a commit to fix a broken test - looks like we're now trimming the logs under upload. Do we want to keep that?

Trying locally it looks like they still show up at sufficient verbosity level, which LGTM


Add tests on DeploymentSets, StatefulSets and DaemonSets to ensure all desired replicas/pods are running

towards https://github.com/giantswarm/giantswarm/issues/29550
What this PR does
Add 3 tests:

check all DeploymentSet replicas are ready
check all StatefulsSets replicas are ready
check all desired daemon pods are running

Checklist

[x] Update changelog in CHANGELOG.md.

Trigger e2e tests

/run cluster-test-suites

Re-triggering the tests for the latest commit
/run cluster-test-suites

@AverageMarcus
Tests are failed
  {"level":"info","ts":"2024-02-08T14:33:28Z","msg":"deployment kyverno/kyverno-admission-controller has 0/3 replicas available"}
  {"level":"info","ts":"2024-02-08T14:33:39Z","msg":"deployment kyverno/kyverno-admission-controller has 0/3 replicas available"}
  [FAILED] in [It] - /app/internal/common/basic.go:78 @ 02/08/24 14:33:44.566
• [FAILED] [900.001 seconds]
Common tests basic [It] has all its Deployments Ready (means all replicas are running)
/app/internal/common/basic.go:74

  [FAILED] Timed out after 900.001s.
  Expected success, but got an error:
      <*errors.errorString | 0xc000c62450>:
      deployment kyverno/kyverno-admission-controller has 0/3 replicas available
      {
          s: "deployment kyverno/kyverno-admission-controller has 0/3 replicas available",
      }
  In [It] at: /app/internal/common/basic.go:78 @ 02/08/24 14:33:44.566

Well, tests are showing issues with kyverno-admission-controller.
What should I do ?

Looks like a flake so I'm ok with skip label.


Passing dynamic value into a constructor

If I pass a "dynamic" variable or class member into a constructor which takes type of "System.Object" JSIL throws a "could not be translated" error for the calling function.
Is there a reason for this, or is this a bug?

Hi, I've just run into this issue as well - is there any advice on a workaround?

Just cast it to object. Something like: new SomeClass((object)dynamicArg).
On Oct 18, 2015 6:34 PM, "callanh"<EMAIL_ADDRESS>wrote:

Hi, I've just run into this issue as well - is there any advice on a
workaround?
—
Reply to this email directly or view it on GitHub
https://github.com/sq/JSIL/issues/297#issuecomment-149073882.


Privacy Metadata initial

This pull request adds privacy constraints as metadata. The privacy constraints are propagated from input to result of matrix multiplications. The privacy constraints currently only contain a simple boolean, which represents whether the data may be shared or not, but more complex constraints will later be added to the PrivacyConstraint class.
This pull request replaces the Privacy Metadata pull request.

I will rebase this branch to resolve the conflicts. Please wait with any edits.

The rebase is done. The checks are failing because Github's servers are having problems.

LGTM - thanks for the initial integration @sebwrede. During the merge I made only some minor formatting fixes (tabs over spaces, verbosity) and slightly changed the hop/lop metadata handover.


bug: MenuAnchor throws an exception when navigating with keyboard horizontally

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Open MenuAnchor
Focus on an element that is not SubMenuButton.
Navigate horizontally e.g left/right arrow keys

Expected results
No-op, as there's nothing left or right to focus on.
Actual results
An exception is thrown though no side effects occur.
https://gist.github.com/Fernthedev/a3c24e00885958fc8392e25d5c3251a5
Code sample
Code sample
https://api.flutter.dev/flutter/material/MenuAnchor-class.html
https://dartpad.dev/?id=1f43e84f2c6df2c5f97d72e73fa5c87d

Screenshots or Video
N/A
Logs
N/A
Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.16.2, on Pop!_OS 22.04 LTS 6.5.6-76060506-generic, locale
    en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✗] Chrome - develop for the web (Cannot find Chrome executable at google-chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[✓] Linux toolchain - develop for Linux desktop
[!] Android Studio (not installed)
[✓] VS Code (version 1.84.2)
[✓] Connected device (1 available)
[✓] Network resources

! Doctor found issues in 2 categories.

Thanks for filing a separate issue, @Fernthedev
I can see this issue on stable 3.16.2 (need to use mouse hover the item to be able to use arrow key) but it seems to be fixed on master channel.
On master channel, you can use arrow key without hovering the item (menubutton) and the error is not thrown anymore. Please try switching to master channel and confirm this on our side.
flutter channel master
flutter upgrade

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.16.2, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.16.2 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9e1c857886 (9 hours ago), 2023-11-30 11:51:18 -0600
    • Engine revision cf7a9d0800
    • Dart version 3.2.2
    • DevTools version 2.28.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.8+0-17.0.8b1000.22-10799086)

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe Patch 3.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.84.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.76.0

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 119.0.6045.199

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.18.0-5.0.pre.17, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.18.0-5.0.pre.17 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a0f4265575 (27 minutes ago), 2023-12-05 21:28:40 -0500
    • Engine revision fe96317750
    • Dart version 3.3.0 (build 3.3.0-194.0.dev)
    • DevTools version 2.30.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.8+0-17.0.8b1000.22-10799086)

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe Patch 3.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.84.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 119.0.6045.199

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

I can confirm it is fixed in the latest master channel. Thank you <3


Unwrangling the mess that is godot_wrap_method, adding generic method support as a side effect

Due to repeated past additions to godot_wrap_method_inner, the implementation has become a total mess of unsafe macro code that is very hard to follow. Meanwhile, the manual interface for method registration has largely been neglected, lagging behind the rest of the crate, exposing sys types and unsafe FFI signatures. Recently, I've found myself in need to register a few methods manually and having to deal with this entire situation, so I've taken the chance to make some improvements to this part of the codebase, with all the hindsight that we now have. This has resulted in satisfactory results:

All the unsafe FFI code previously in godot_wrap_method_inner is extracted to the library proper, and can no longer be touched by obscure import/inference problems that come from delayed type checking.
Argument errors are now represented with a proper error type with a Display implementation. The library can report multiple argument errors at once. "Missing/excessive argument" errors are also more informative.
It's possible to manually register one method for many classes that it may apply to, or many specializations of a generic method to one class by different names.
Safe, straightforward support for stateful methods.

This is not in itself a breaking change, but it's expected that the older unsafe interface will be removed at the next possibility.

As a curiosity, the syntax for manually registering generic methods currently looks like this, and I find it already fairly usable even without further macro goodness:
#[derive(Copy, Clone, Default)]
struct Mixin<T> {
    _marker: PhantomData<T>,
}

#[derive(FromVarargs)]
struct Args<T> {
    a: T,
    b: T,
}

impl<T, C> StaticArgsMethod<C> for Mixin<T>
where
    T: Add<Output = T> + Send + Sync + ToVariant + FromVariant + 'static,
    C: NativeClass,
{
    type Args = Args<T>;
    fn call(&self, _this: RefInstance<'_, C, Shared>, Args { a, b }: Args<T>) -> Variant {
        (a + b).to_variant()
    }
}

fn register_methods(builder: &ClassBuilder<SomeClass>) {
    builder
        .build_method("add_i", StaticArgs::<Mixin::<i32>>::default())
        .done_stateless();
    builder
        .build_method("add_f", StaticArgs::<Mixin::<f32>>::default())
        .done_stateless();
    builder
        .build_method("add_v", StaticArgs::<Mixin::<Vector2>>::default())
        .done_stateless();
}

Unresolved questions
As a side effect of moving the error handling to the library side, error sites all point to the library now. This should be fixable with something on the macro side that provides borrow-able site information, like how profiling::Signature is handled? This would mean moving the FFI call out of godot_error into some interface as well. Resolved.
Future development
If desired, it should be fairly easy to add a higher-level, derivable Mixin interface based on this. It should also be possible to add support for generic methods directly in #[export].

bors try

bors try

Added explanation of the ZST pointer in comments. Made strings in ArgumentError Cow. Added a Site API to deal with the error reporting problem introduced by the original PR. Errors from wrapped methods now have correct file paths and type/method names set:
ERROR: OptionalArgs :: opt_sum: missing non-optional parameter `b` (#1)
   At: test/src/lib.rs:198.
ERROR: OptionalArgs :: opt_sum: an excessive argument is given: I64(6)
   At: test/src/lib.rs:198.

Compared to original PR (path pointing to library):
ERROR: <native>: missing non-optional parameter `b` (#1)
   At: gdnative-core/src/nativescript/init/method.rs:188.
ERROR: <native>: an excessive argument is given: I64(6)
   At: gdnative-core/src/nativescript/init/method.rs:181.

Compared to current master (type and method names not shown, worse message):
ERROR: <native>: Incorrect number of parameters: required 2 but got 1
   At: test/src/lib.rs:198.
ERROR: <native>: Incorrect number of parameters: expected at most 5 but got 6
   At: test/src/lib.rs:198.

Added explanation of the ZST pointer in comments. Made strings in ArgumentError Cow. Added a Site API to deal with the error reporting problem introduced by the original PR. Errors from wrapped methods now have correct file paths and type/method names set:
ERROR: OptionalArgs :: opt_sum: missing non-optional parameter `b` (#1)
   At: test/src/lib.rs:198.
ERROR: OptionalArgs :: opt_sum: an excessive argument is given: I64(6)
   At: test/src/lib.rs:198.

Compared to original PR (path pointing to library):
ERROR: <native>: missing non-optional parameter `b` (#1)
   At: gdnative-core/src/nativescript/init/method.rs:188.
ERROR: <native>: an excessive argument is given: I64(6)
   At: gdnative-core/src/nativescript/init/method.rs:181.

Compared to current master (type and method names not shown, worse message):
ERROR: <native>: Incorrect number of parameters: required 2 but got 1
   At: test/src/lib.rs:198.
ERROR: <native>: Incorrect number of parameters: expected at most 5 but got 6
   At: test/src/lib.rs:198.

bors try

bors try

Marking as breaking change since I just realized that this breaks code currently using godot_wrap_method manually to create pointers.

Marking as breaking change since I just realized that this breaks code currently using godot_wrap_method manually to create pointers.

Found an issue with the current code: since <$retty as $crate::core_types::OwnedToVariant>::owned_to_variant(ret) is now wrapped in an impl StaticArgsMethod<T> for ThisMethod block, any Selfs in the return type signature now refers to ThisMethod instead of what's intended. Should be easy to fix by just dropping the fully qualified syntax and using OwnedToVariant::owned_to_variant, will update later.

Found an issue with the current code: since <$retty as $crate::core_types::OwnedToVariant>::owned_to_variant(ret) is now wrapped in an impl StaticArgsMethod<T> for ThisMethod block, any Selfs in the return type signature now refers to ThisMethod instead of what's intended. Should be easy to fix by just dropping the fully qualified syntax and using OwnedToVariant::owned_to_variant, will update later.

Should be ready for merging!

bora r+

bors r+


Taiwo06 submitted

It looks like much of your logic is correct, but I can't get your code to run, in part because there isn't a notebook included where you call your function. When I made a new notebook and called the function with the correct arguments, there wasn't any content in the json, so the analysis didn't fully execute. I can follow the code and in your function in your module, though, and like how you found unique timestamps within each hour.


Add Xdebug & PHPUnit to the Docker image for local testing

To generate a report run this command inside the wp-tide-api plugin directory:
./bin/phpunit --coverage-html=./report

Pull Request Test Coverage Report for Build 302

0 of 0   changed or added relevant lines in 0 files are covered.
34 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-6.3%) to 93.669%

Files with Coverage Reduction
New Missed Lines
%

cmd/lighthouse-server/main.go
17
88.59%

cmd/phpcs-server/main.go
17
89.82%

Totals

Change from base Build 300:
-6.3%

Covered Lines:
503

Relevant Lines:
537

💛  - Coveralls


Service Bus: Broken links to source code and API reference

docs-ref-services/servicebus.md
404s on
SDK source code
SDK reference documentation
links

Can't reproduce it from https://docs.microsoft.com/en-us/python/api/overview/azure/servicebus-readme?view=azure-python today

Can't reproduce it from https://docs.microsoft.com/en-us/python/api/overview/azure/servicebus-readme?view=azure-python today


fix translate

Description
The call to Bio:PrimarySeqI->translate() does not match the prototype in BioPerl 1.6.x, and should be revised
Use case
Non-standard start codons are not translated properly, regardless the specified codon table.
Example: the microbial CDS sequence "GTGTTTTCCGAT..." translates to "VFSD..."
Benefits
Fix
Possible Drawbacks
N/A
Testing
Have you added/modified unit tests to test the changes?
If so, do the tests pass/fail?
Have you run the entire test suite and no regression was detected?

Coverage increased (+0.08%) to 82.327% when pulling 7a89321bd90100735a624abf9f35af3de69ac01c on fix/transcript_translate into 553178e74b8db9b8f7618512db8a073014c7d107 on main.

2 new tests were added to transcript.t to test the translate method changes. One tests setting the cds_complete option, and the other tests the codon_complete option. New test data was added (new gene, transcript, translation, exons, etc...) and other test scripts were updated as well in terms of counting genes and such. Other fixes were also needed to handle having the CDS complete transcript attribute, as not having that leads to warnings when translate is called and those warnings lead to test failures.
In addition, this change also necessitated the fixing of a bareword directory handle due to a new policy in Perl::Critic added recently.

Converted back to draft: further investigation is required to assess behaviour of the $complete flag, in some cases apparently related to microbial (only?) CDS


Skip font installation

As title.
Generated by Cimas.

Ruby 2.4 will be addressed separately https://github.com/metanorma/mn-templates-itu/pull/27#issuecomment-764450492

Ruby 2.4 will be addressed separately https://github.com/metanorma/mn-templates-itu/pull/27#issuecomment-764450492


"Extend Selection" has unexpected behavior when "Relative" checkbox is checked

Extend Selection by bytes, with Relative checked, seems to always extend selection to end of segment, or treats beginning of new selection as end of current selection, but result is actually clearing the selection, leaving cursor at beginning of existing selection. (However, extend to address seems to work). Also text is confusing: relative unchecked extends relatively, relative checked is unclear what it does, but in either case, the preview address shown does not reflect what happens (relative shows a preview address that makes sense, but non-relative just shows the size of the change, not the resulting size or selection extent, and as unsigned for negative values).
Simple solution is to remove the Relative checkbox from the dialog that appears when using an Extend Selection action.
Workaround is to never select the Relative checkbox.

Here is how it is designed to work:

When you do "Extend By Bytes", it takes the value from the dialog and extend the selection by that. Let us say the selection starts at 0x400000 initially, and you typed 0x1000 into the dialog, and unchecked the relative, then the new selection will become 0x400000 - 0x401000, as expected.
If you check the "relative" thing, the result from the dialog will become 0x401000 itself, and the selection will become 0x400000 - 0x801000 or something similar, which can be quite unexpected

This is more like a user error but I admit your way of interpreting and expecting things are reasonable, so I would see if I can make it more clear and avoid the confusion. There are two possible fixes:

"Expand By Bytes" would show a slightly different dialog that does not have the relative check in it
Combine "Expand By Bytes" and "Expand To Address" into one and make the decision based on wehter the relative is checked


Does this library support local ads from the phone instead of IMA

Hi dear developer, does this library support local ads from phone(meaning that we store file video inside phone play when certain amount time and pause content current video play)?

Realize advertising on any index.
.addOnWindowListener(new VideoWindowListener() {
                    @Override
                    public void onCurrentIndex(int currentIndex, int windowCount) {
                        if (currentIndex == 0) {
                            Log.d(TAG, "setOnWindowListener:" + currentIndex);
                            exoPlayerManager.next();
                            //屏蔽控制布局Shielding control layout
                            exoPlayerManager.hideControllerView(true);
                            //true如果屏蔽控制布局 但是需要显示全屏按钮。手动显示。
                        } else {
                            //恢复控制布局Recovery control layout
                            exoPlayerManager.showControllerView(true);
                        }
                    }
                })

could you have fully code?


Index was out of range. Must be non-negative and less than the size of the collection.

You probably used it on an account with a very small history.
With the last Tool update, it should be fixed along with bad login exception.


Metadata

From Jack: add these metadata fields:
https://docs.google.com/spreadsheets/d/1qQ9p7rVJQUxJnk0Hhtnodov61tmep8IvbRWU3S5HnGs/edit#gid=0
and have endpoints ?info to surface human readable information and ?json to surface json information
The json should include metadata about the types of each column (example:
{
	"url" : {
		"property" : "https://schema.org/url",
		"type" : "https://schema.org/URL",
		"value" : "https://library.frick.org/permalink/01NYA_INST/1qqhid8/alma991000000019707141"
	},
	"title" : {
		"property" : "http://purl.org/dc/elements/1.1/title",
		"type" : "xsd:string",
		"value" : "Bibliographie zur kunstgeschichtlichen Literatur in ost- und südosteuropäischen Zeitschriften."
	},
	"relation" : {
		"property" : "http://purl.org/dc/elements/1.1/relation",
		"type" : "xsd:anyURI",
		"value" : "https://ark.frick.org/ark:37624/an1abc123",
		"label" : "Frick Auction Catalogs"
	},
	"source" : {
		"property" : "http://purl.org/dc/elements/1.1/source",
		"type" : "xsd:anyURI",
		"value" : "https://yamz.net/term/ark/h3886",
		"label" : "null"
	}
}

test plan:

validated with sample ARK in db:

Also fixes a bug. before if you update one field but not the others (eg, updating URL but not metadata) the field you did not update would get set to the empty string, lol,. this is fixed now.

I also added some sampel scripts to call the API to mint and update ARKs. Not necessarily the final form the cmd ui but good for us to be able to test data etc.


cpython: 3.9.6 -> 3.9.7; 3.8.11 -> 3.8.12; 3.7.11-> 3.7.12; 3.6.14 -> 3.6.15; 3.10.0-rc1 -> 3.10.0-rc2

Motivation for this change
https://docs.python.org/release/3.9.7/whatsnew/changelog.html#changelog
https://docs.python.org/release/3.8.12/whatsnew/changelog.html#changelog
https://docs.python.org/release/3.6.15/whatsnew/changelog.html#python-3-6-15-final
https://docs.python.org/release/3.7.12/whatsnew/changelog.html#python-3-7-12-final
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

@ofborg build python38.tests python39.tests

By merging this PR - the following issue will be resolved: https://github.com/NixOS/nixpkgs/issues/105038

bpo-44689: ctypes.util.find_library() now works correctly on macOS 11 Big Sur even if Python is built on an older version of macOS. Previously, when built on older macOS systems, find_library was not able to find macOS system libraries when running on Big Sur due to changes in how system libraries are stored.

Both Python 3.8.12 and Python 3.9.7 contain this patch (https://github.com/python/cpython/pull/27251).

Two more updates.
python36.tests won't run because our numpy version requires at least python37.

Note 3.6 will be EOL 2021-12, and will thus need to be removed for NixOS 21.11.
I suggest changing the package to something else than numpy such as py or setuptools.

Note 3.6 will be EOL 2021-12, and will thus need to be removed for NixOS 21.11.
I suggest changing the package to something else than numpy such as py or setuptools.

Since python36 is essentially deprecated at this point (no longer being recursed into for derivations), I think it's okay to verify that requests still builds, and call it good. Very few people should be using python36 at this point

unfortunately, python36Packages.requests seems to fail because of a failure in pytest-xdist

bpo-43124: Made the internal putcmd function in smtplib sanitize input for presence of \r and \n characters to avoid (unlikely) command injection.

Working on removing Python 3.6 now, since we are nearing the release (8 weeks).

That's it, no more references to python36Packages.

@GrahamcOfBorg build python39.tests python38.tests python37.tests

Bisect says d003f75d7888cd92993107b5a1d6e3a8d340bf68 "python39: 3.9.6 -> 3.9.7" it broke python3Packages.argh tests:
=========================== short test summary info ============================
FAILED test/test_integration.py::test_invalid_choice - assert False
FAILED test/test_integration.py::test_explicit_cmd_name - assert False

_____________________________ test_invalid_choice ______________________________

    def test_invalid_choice():
        def cmd(args):
            return 1

        # root level command

        p = DebugArghParser()
        p.add_commands([cmd])

>       assert run(p, 'bar', exit=True).startswith('invalid choice')
E       assert False
E        +  where False = <built-in method startswith of str object at 0x7fffe89b72d0>('invalid choice')
E        +    where <built-in method startswith of str object at 0x7fffe89b72d0> = "argument {cmd}: invalid choice: 'bar' (choose from 'cmd')".startswith
E        +      where "argument {cmd}: invalid choice: 'bar' (choose from 'cmd')" = run(DebugArghParser(prog='py.test', usage=None, description=None, formatter_class=<class 'argh.constants.CustomFormatter'>, conflict_handler='error', add_help=True), 'bar', exit=True)

test/test_integration.py:380: AssertionError

Filed another upstream report at https://github.com/aio-libs/aiohttp/issues/6066


Deployment on Kubernetes fails

When I deploy and run Kafka HQ on Kubernetes it fails because of:
$ kubectl.exe  logs -f kafka-hq-786784cbb8-4668d
/usr/local/bin/docker-entrypoint.sh: 9: exec: ./kafkahq: not found

I believe this is due to missing permissions on the path and must be fixed in the docker image.

As I know, deployment on kubernetes works well.
The kafkahq command as also the right permission.
I think you have change the workdir on your k8s configuration (default WORKDIR /app) ?
and in this case, you must change the command to be absolute (default CMD ["./kafkahq"])
Just change command to /app/kafkahq and it must be working as I understand.
Tell me if it work

I can't confirm this has worked, nor did I ever had to change the docker entrypoint in my kubernetes manifests. I am running on Google's GKE. This should be fairly reproducible, here's the kubernetes manifest I apply:
https://gist.github.com/weeco/be9b631e10564475073886c0a4042cf5

OK got it !
Already have the bug on my side.
          volumeMounts:
            - mountPath: /app
              name: config
              readOnly: true

You are mounting the /app in order to inject the configuration.
And it remove all the binary from kafkahq leading to this error.
I suggest you to use the MICRONAUT_CONFIG_FILES env var to change the configuration path like /app/configurations/ and to mount this directory for configuration on your pods.
or to use directly KAFKAHQ_CONFIGURATION env var with configuration, like you want.

Ah that makes sense, I didn't think about that. What I did to fix is to edit the volume mount so that it just adds the file into that folder:
          volumeMounts:
            - mountPath: /app/application.yml
              subPath: application.yml
              name: config
              readOnly: true

Now it has the application.yml and all the other files from the docker image in that folder. Thank you for your quick responses. Speaking of cloud environments, do you have a setting do print all log messages in JSON format?

Just realize that json log message is not implemented !
Using it myself for all work production env but I didn't think about it for kafkahq.
Just put an issue for reminded and I will do it :)

Will add some docs about this since you're not the first one !


nixos/zapret: init

Description of changes

Added zapret service. Systemd service is from upstream, not sure if i need to change it, except hardening
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Options like that should be created:

Options like that should be created:
    # a better name needed
    ipsetHostsUser = {
      exclude = mkOption {
        type = types.lines;
        default = ''
          <IP_ADDRESS>/8
          <IP_ADDRESS>/12
          <IP_ADDRESS>/16
          <IP_ADDRESS>/16
          fc00::/7
          fe80::/10
        '';
        description = "Contents of ipset/zapret-hosts-user-exclude.txt.";
      };

      include = mkOption {
        type = types.lines;
        default = "";
        example = "nonexistent.domain";
        description = "Contents of ipset/zapret-hosts-user.txt.";
      };

      ipban = mkOption {
        type = types.lines;
        default = "";
        example = "nonexistent.domain";
        description = "Contents of ipset/zapret-hosts-user-ipban.txt.";
      };
    };

Then values of these options should be written somewhere. Is there a way (for example, by changing an environment variable) to tell zapret to look for these files in different directory (somewhere in /etc, for example). Maybe we can ask bol-van to create such functionality?
Maybe add arguments with default values like hostsUserExclude ? "notadomen.domain" to package definition, and then during configurePhase or some other phase write these arguments' values to needed files. But in that case module can't be used (I think).

I don't think that we can do something with files, that meant to be changed except config. We can put this in package options, but that will be nasty
package = pkgs.zapret.override {
  zapret-hosts-user = ''
    site1
    site2
    ...
  '';
};

We can change files in package source so it will look in /etc/zapret rather than in $ZAPRET_BASE/ipset, but that too much changes. I will create an issue in zapret.
For now you can use even nastier method
package = pkgs.zapret.overrideAttrs (prev: {
  installPhase = ''
    ${prev.installPhase}

    cat << EOF > $out/usr/share/zapret/ipset/needed-file
      needed-configuration
    EOF
  '';
});

I think every way to implement it now would be nasty. The only clean way is for bol-van to allow changing these files' location. I'm not an expert but I think configuration files should be separate from program's files.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4379

I've created an alternative PR here https://github.com/NixOS/nixpkgs/pull/347805

Closing due to better implementation at #347805


Babel is not optional

When you ignore babel dependency - whole project startup will fail.
This line will try to import babel loader
https://github.com/boardpack/pydantic-i18n/blob/master/pydantic_i18n/__init__.py#L7
This line will burn application:
https://github.com/boardpack/pydantic-i18n/blob/master/pydantic_i18n/loaders.py#L5
Please do not mark dependencies as optional, while they are not!

Hello, @tigrus, oops, you are entirely right... I'm fixing it now, sorry

@tigrus released, please recheck it on your side, it must be fine

Looks fine now. Thanks for fast fix :)


Register HashMap to arrow memory pool

Hi @xuechendi , did you ever see errors like "corrupted size vs. prev_size" during tests on the changes?
I managed to cherry-pick your changes onto 1852 PR https://github.com/Intel-bigdata/OAP/pull/1891 then saw such errors appearing to stderr.

@xuechendi UT failed

Hi Yuan, at here CI is not possible to pass without having the base Arrow PR https://github.com/Intel-bigdata/arrow/pull/97 merged

@zhztheplayer
i see, let's hold on a bit after we fix the arrow used in 1.0 branch
https://github.com/Intel-bigdata/OAP/blob/branch-1.0-spark-3.x/.travis.yml#L56

patches added in #1891


[Aio] Support local credentials properly with asyncio resolver

Originated from https://github.com/grpc/grpc/pull/21885
The AsyncIO stack is using its own IO manager, which includes a different name resolver. The returned value from the name resolver might not be the same as Core, and lead to the failure of using local credentials with localhost.

This won't be an issue for poller-based stack #22258.


Update docker images

Update docker images to rust 1.55

Coverage increased (+0.005%) to 85.225% when pulling f4aee444355c7b275604b9fc4c4db4756175fe36 on update-docker into 85f411047b700a2a7f71a98a2a7b2d4aec02ac05 on main.


Preserve timezone offset for chrono dates with fixed timezone

See issue #1039
UTC and localtime are still converted to UTC when saving.

Thanks for the merge!


modify device-plugins.md

Spelling mistake.  "co-ordintated" should be "co-ordinated".

@Yvonnehrr Thank you for your contribution.  Please sign the CLA so we can accept your PR for review.

/assign

I signed it.

I signed it
/retest
/check-cla

/check-cla

Hi there I tried rechecking the CLA, is the author email associated with your commit the same one you signed the CLA with?

Hi there I tried rechecking the CLA, is the author email associated with your commit the same one you signed the CLA with?

It's strange that i signed in with "Yvonnehrr"，but it reminds me "h4ghhh needs CLA". Though “h4ghhh” is also mine.

/check-cla

/retest

Hi there I tried rechecking the CLA, is the author email associated with your commit the same one you signed the CLA with?

It's strange that i signed in with "Yvonnehrr"，but it reminds me "h4ghhh needs CLA". Though “h4ghhh” is also mine.

Hello @Yvonnehrr . It is my understanding that the email accounts must be the same. Would you double check your CLA?
Thanks!


Question about using the E-AC3 codec

@VideoMangler I assume E-AC-3 is working for you, so is it OK if I close this now?

OK to close, thanks again!


use client certificate for ssl

how can we use the ssl certificate client to authorize connection between server and client?
I just seeing the parameter useSSL: true/false
but the client need a certificate ? like browser generate signed certificate in web site

@KrifaYounes we can't use client certificates on react-native in any time soon, not only this package, other packages also don't support this feature, also self signed certificates cannot be used on react-native. I had to buy a domain just for using mqtt on tls connection.

Can I connect to mqtt server over SSL/TLS connection with secure domain and just by setting useSSL: true at client side?


containerized build - etcd not starting.

I can repro this, but the systemd process fails on:
[root@master1 system]# /usr/bin/docker run --name etcd_container --rm -v /var/lib/etcd:/var/lib/etcd:z -v /etc/etcd:/etc/etcd:ro --env-file=/etc/etcd/etcd.conf --net=host --entrypoint=/usr/bin/etcd quay.io/coreos/etcd
container_linux.go:247: starting container process caused "exec: \"/usr/bin/etcd\": stat /usr/bin/etcd: no such file or directory"
/usr/bin/docker-current: Error response from daemon: oci runtime error: container_linux.go:247: starting container process caused "exec: \"/usr/bin/etcd\": stat /usr/bin/etcd: no such file or directory".

It should be /usr/local/bin/etcd

Should be fixed now. All good. Tested all versions (rpm and containerised)
It also helped to fixed another opened bug around etcd 3.10 static pod :+1:


cardano-node: 1.10.1 -> 1.11.0

Issue Number
Relates to #1577 / ADP-83.
Overview

Updates cardano-node version in nix.
Updates the cardano-haskell stack snapshot.
Adds cardano-cli to the shell. I am hoping to use this in the integration tests.
Build fixes

Comments
WIP until input-output-hk/cardano-haskell#13 is merged.

bors try

bors r+

bors try

There were Hydra eval out of memory problems... retrying...
bors r+


fix Int64 parsing

When a appmanifest is parsed, manifest id it's parsed as float instead of a long
Sample:
"AppState" { 	"appid"		"730" 	"Universe"		"1" 	"name"		"Counter-Strike: Global Offensive" 	"StateFlags"		"4" 	"installdir"		"Counter-Strike Global Offensive" 	"UpdateResult"		"0" 	"buildid"		"7797970" 	"InstalledDepots" 	{ 		"732" 		{ 			"manifest"		"8327859270506257685" 			"size"		"362203366" 		} 		"731" 		{ 			"manifest"		"6404082971767543753" 			"size"		"29066149253" 		} 		"735" 		{ 			"manifest"		"3867231304834558645" 			"size"		"0" 		} 	} 	"SharedDepots" 	{ 		"228988"		"228980" 		"228990"		"228980" 	} 	"UserConfig" 	{ 		"language"		"english" 	} }

Seems like that will still be a problem for ulong

Valve's code actually parses those manifest IDs as strings.
The size of 732 ("362203366") is then an int but the size of 731 ("29066149253") is also a string.

They check whether it fits into an int without overflow (strtol), And for floats they use strtod which likely checks for presence of a dot or something.

I think their strtod code might be buggy... 👀

I investigated a little more, and indeed as @yaakov-h  says Valve treats manifest id as string
This information comes through the CMServer as binary vdf and the manifest id are treated as strings, once Steam Client stored it in appcache.vdf they are treated in the same way

Closing this because this will need a different fix. Created an issue instead: #53


create:ignoreDeployerAttacks does not function as intended.

Describe the Bug
As the title suggests. In Create Fabric 0.5.1-f-build.1335+mc1.20.1 this config does not have any notable effect in-game when set to ANY, CREEPERS, or NONE. I've tried this on both a singleplayer and multiplayer server, and by modifying through the in-game menus and config file directly with no results. Mobs will attempt to attack deployers regardless of the config setting.
Installed:
Minecraft 1.20.1
Create Fabric 0.5.1-f-build.1335+mc1.20.1
Fabric 0.15.7
Fabric API 0.92.0+1.20.1
Reproduction Steps

Place a powered deployer in attack mode.
Spawn a mob in-front of the deployer.
The mob will attempt to attack the deployer. Very noticeable on mobs such as Skeletons and Creepers.
...

Expected Result
Expected result is for the mob to ignore the deployer if the relevant config setting is set to ALL, but regardless of the setting all mobs act like it is set to NONE.
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
Windows 10
Mod Version
0.5.1f
Minecraft Version
1.20.1
Other Mods
No response
Additional Context
No response

Related to https://github.com/Creators-of-Create/Create/pull/6072


Add Normalize stutters option

This is an option to allow normalizing stutters.
I used it to update this map in full-memo :
https://quavergame.com/mapset/map/35226quavergame.com/mapset/map/35226
SVs were applied in one go with these settings:
Effect mode: Distance between notes
Duration scale: 1.00
Start Velocity: 10.00x
Start SV duration: 0.05
Average SV: 0.50x
Normalize stutters: On
Others Off
The way it works is by increasing the start-sv duration according to the distance between notes, using the bpm as a reference. (Could use 1000ms to make it universal, but it's up for debate and I didn't like using a random constant).
There is a warning attached to it, since if using a Start SV duration higher than 1/n with n being the lowest snap in the selection, the start SV will be pushed further than the end SV.
For example, if Start SV duration is 0.5 and two notes are 1/4 apart, the start sv will be at 200% of the distance between these two notes.
This is not a problem if you know how to use it, but it's worth pointing out.
The reason i'm submitting this is simply that, since it was helpful to me, why not share it.

Redundant, as effect can already be achieved with
Effect duration mode: BPM/measure snap
Duration length: (change to snap lower or equal to smallest snap, eg 1/4 = 0.25)
Start velocity: high (max works best)
Start SV Duration: any
Skip SV of individual stutters: True

Redundant, as effect can already be achieved with
Effect duration mode: BPM/measure snap
Duration length: (change to snap lower or equal to smallest snap, eg 1/4 = 0.25)
Start velocity: high (max works best)
Start SV Duration: any
Skip SV of individual stutters: True


Proposal: expand ENV value when i build image

Assume I have those two instructions in my Dockerfile ,
ENV VERSION 1.4.1
RUN curl -sSL https://something-${VERSION}.tar.gz | ...

When i build it in my local host, the process of work will be showed in stdout, like
Step xx : RUN curl -sSL https://something-${VERSION}.tar.gz | ...

As we can see from the output, the ENV  VERSION won't  be expanded, I think it may be better to expand the value of ENV for tracking some problems. So, I'd like to output like this:
Step xx: RUN curl -sSL https://some-1.4.1.tar.gz | ...

If I make a mistake, like spelling mistake,should output blank string or warn me i ref a undefined ENV.
ENV VERSION 1.4.1
RUN curl -sSL https://something-${VRESION}.tar.gz | ...

Should output
Step xx: RUN curl -sSL https://something-.tar.gz | ...

So I can easily find my error above!
I know it's not a easy work, because when docker interpreter read a Dockerfile instruction, it can not get  the env in the container interval,but it can easily record the ENV defined in the Dockerfile.

I'm assuming you're just asking about the build output that is shown on your console, as the build does actually substitute the variable.
If this is indeed the case, what you're asking isn't possible. The primary reason is that ${VERSION} is shell syntax. Docker is simply passing the text curl -sSL https://something-${VERSION}.tar.gz | ..., to /bin/sh, and letting the shell interpret it. Docker has no clue that the text ${VERSION} will be substituted with the value of the environment variable VERSION.

@phemmer  I know Docker cann't not get any value from the container, I just want to expand the ENV defined in my Dockerfile, it may be excess feature, but sometimes useful for us to debug.In fact, sometimes, I  insert  'set -x' to track it.

@krystism the issue is that docker just send that line toa shell, and that shell happens to do something if it sees ${FOO}. Docker itself makes no assumptions here, and doesn't parse that instruction.

@phemmer I think I understand what you're saying... for the envs that Docker knows about, go ahead and do the substitutions prior to sending it to the shell.  While this may be possible, I think it might actually lead to more confusion than help people.  As an end user (newbie) I would find it really confusing to look have a Dockerfile with:
RUN echo $FOO $BAR
and then see:
Step 2 : RUN echo $FOO hello
in the output.  Having Docker only do a partial substitution could look like an error and make people wonder why it didn't do the others.
I think its better to not do 1/2 the job and simply let the shell do it all.
Gonna close this one for now - if you feel there may be a better solution you can re-open.

@duglin Eh? I'm opposed to doing this substitution. Did you mistake me for the author?

Oops  :-)  yup - fixed comment to point to the right person. Sorry.


feat: drop Link layer in favor of package:gql_link and package:gql_exec

BREAKING CHANGE: Link layer is now implemented via package:gql_link and package:gql_exec
First steps towards https://github.com/zino-app/graphql-flutter/issues/563

drop Link layer in favor of package:gql_link and package:gql_exec
examples not updated as they are already outdated
docs to be updated when more changes come in
the surface between client and link could still get some love (better error handling)
this change is probably not bug-free and could be improved later PRs against modularization branch
the package:gql_*_link family is not up to par with the code removed from here, but it'll be done before modularization branch is merged. Also, that's the point of modularization - the Link layer can be improved independently from the client.

:tada: This PR is included in version 4.0.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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

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


Pushbullet shows up for every user, even when the agent is deactived

Description
Pushbullet is now visible to every user, it doesn't matter if it is activated or not.
Version
2.1.0
Steps to Reproduce

Go to User
Click Edit on any user
See Pushbullet as an option even when deactivated

Screenshots
global setting

user setting

Logs
No response
Platform
desktop
Device
any
Operating System
any
Browser
any
Additional Context
No response
Code of Conduct

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

Not a bug. This is expected default behaviour. Pushbullet and Pushover would always be visible.

Thank you very much for the ultra fast answer!
I am very thankful for the great work of yours!


Greenkeeper/husky 2.1.0

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feature:  redis to support reentry locks

Ⅰ. Describe what this PR did
redis模块支持可重入锁；即对同一条数据，在同一事务下，可以获取到锁，并对数据进行修改
Ⅱ. Does this pull request fix one issue?

自己发现的
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
在同一个接口中，先插入一条数据，然后再对同一条数据进行更新操作，可以正常更新；
未修复之前，对同一数据进行更新会报获取锁失败异常
Ⅴ. Special notes for reviews
redis不支持可重入锁

I think the title is changed to redis to support reentry locks, which is more appropriate

Should I change the title?

Codecov Report

Merging #2944 into develop will decrease coverage by 0.11%.
The diff coverage is 62.50%.

@@              Coverage Diff              @@
##             develop    #2944      +/-   ##
=============================================
- Coverage      50.26%   50.14%   -0.12%
- Complexity      3047     3049       +2
=============================================
  Files            599      599
  Lines          19415    19400      -15
  Branches        2381     2339      -42
=============================================
- Hits            9758     9728      -30
+ Misses          8679     8670       -9
- Partials         978     1002      +24

Impacted Files
Coverage Δ
Complexity Δ

...o/seata/server/storage/redis/lock/RedisLocker.java
37.70% <62.50%> (+1.42%)
11.00 <1.00> (+1.00)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterRMRequestConvertor.java
90.47% <0.00%> (-4.77%)
3.00% <0.00%> (ø%)

... and 25 more

you can  reassociated your github info

更换账号重新提交pr，此pr关闭


crash if email is empty in LDAP

if the email (mail field) is empty the app crashes

Hey @lorieri, do you mind opening an issue directly in https://github.com/sdelements/lets-chat-ldap?

no, sure


Refreshing patient record website results in whitelabel error page

Describe the bug
When refreshing the patient data record page a white label error page is displayed.
Expected behaviour

patient data page should be displayed as it was or with cleared data
or

overall quicktest frontend should be displayed

this has to be discussed

Steps to reproduce the issue

Open "Patienten Daten Erfassen"
Refresh web page by clicking the browsers refresh button
Whitelabel Error Page is displayed

Technical details

Host Machine OS (Windows/Linux/Mac):

Possible Fix
/record could be mapped to frontend

works for all pages now


Implement CairoRunner::end_run and its dependencies

Implement CairoRunner::end_run and its dependencies
Description
Implementation CairoRunner::end_run and its dependencies.
Depends on #502.
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

 Add and export run_from_entrypoint to CairoRunner

Codecov Report

Merging #507 (76e918f) into main (1a82b62) will decrease coverage by 0.03%.
The diff coverage is 88.23%.

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

@@            Coverage Diff             @@
##             main     #507      +/-   ##
==========================================
- Coverage   92.13%   92.10%   -0.04%
==========================================
  Files          48       48
  Lines        3192     3216      +24
==========================================
+ Hits         2941     2962      +21
- Misses        251      254       +3

Impacted Files
Coverage Δ

src/vm/vm_core.rs
92.01% <80.00%> (-0.20%)
:arrow_down:

src/vm/runners/cairo_runner.rs
94.42% <85.00%> (-0.89%)
:arrow_down:

src/vm/vm_memory/memory.rs
95.28% <100.00%> (+0.89%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Added support for using service account instead of a user

Hello,
In some cases outside of Cloud Shell we can't use a user but a service account. This PR helps detect if there's any credentials file in the environment variables and activate the service account for using it instead of a specific user.

@joecheuk @blueandgold Could you take a look?

Hi @angelzamir  @morgante , we are releasing Forseti v2.0 soon so in the coming week or so we will be focusing on doing that and we don't plan to merge in any new changes during this time period. We should be able to get the PR in after the release. Is that fine with you?

That will likely significantly delay the Forseti Terraform module, as @angelzamir will be working on other projects following the end of this week.
@angelzamir Is it possible to build against a fork for now and have our module use that fork until this is merged?

@morgante sure, I think the module could points to my fork, working on that.

@blueandgold Can we keep this open? I believe it's still needed for Terraform support.


Add I2C documentation

#include <esp32m/bus/scanner/i2c.hpp>
is for I2C scanner. If you just want to use I2C bus, without scanner, you should use
#include <esp32m/bus/i2c.hpp>

Fixed!


[DEPS] Bump jersey 2.30

Why are the changes needed?

Align with spark-core
https://mvnrepository.com/artifact/org.apache.spark/spark-core_2.12/3.1.2
https://mvnrepository.com/artifact/org.apache.spark/spark-core_2.12/3.0.3
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

@yanghua

Codecov Report

Merging #1042 (f5a05cd) into master (4cd8379) will increase coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1042      +/-   ##
============================================
+ Coverage     78.40%   78.56%   +0.16%
  Complexity       61       61
============================================
  Files           167      167
  Lines          6196     6196
  Branches        723      723
============================================
+ Hits           4858     4868      +10
+ Misses          915      903      -12
- Partials        423      425       +2

Impacted Files
Coverage Δ

.../scala/org/apache/kyuubi/server/KyuubiServer.scala
58.69% <0.00%> (+2.17%)
:arrow_up:

.../org/apache/kyuubi/operation/KyuubiOperation.scala
54.54% <0.00%> (+16.36%)
:arrow_up:

Continue to review full report at Codecov.

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

I'm a little surprised that Jetty pulls such a number of dependencies.

+1, IMO, it does not affect the rest frontend service. Will verify later.

thanks, merging to master


Remove CXX sdk in Python site-packages

Fixes #666
Fixes #771
Summarize your change.
This branch fixes building issues, both for building wheels and building with third-party projects.  In my particular case (#771) it allows building with Imath.
The problem with the current setup is that it does not allow building with any sub-projects that use the standard CMAKE_INSTALL_INCLUDEDIR variable.  These are not be able to correctly install, since the setup.py effectively sets this variable to the build directory which generates the following error (in the case of Imath when compiled on Windows):
CMake Error in src/deps/Imath/config/CMakeLists.txt:
    Target "ImathConfig" INTERFACE_INCLUDE_DIRECTORIES property contains path:

      "C:\Users\runneradmin\AppData\Local\Temp\pip-req-build-wwijcqv7\build\lib.win-amd64-3.8\/opentimelineio/cxx-sdk/include/Imath"

    which is prefixed in the source directory.

We would run into the same problem with any other CMake projects in the future that we decided to add with the  add_subdirectory command, so it's worth fixing this properly rather than just overriding the option for that project.
Even though this is currently only generating an error on Windows, it also has an undesirable effect on posix systems as well.  If we look at the CMake config files that are exported to [python site-packages]/cxx-sdk we can see the lines:
# The installation prefix configured by this project.
set(_IMPORT_PREFIX [build directory])

# Create imported target OTIO::opentimelineio
add_library(OTIO::opentimelineio SHARED IMPORTED)

set_target_properties(OTIO::opentimelineio PROPERTIES
  INTERFACE_INCLUDE_DIRECTORIES " [build directory]; [build directory]/opentimelineio/deps"
  INTERFACE_LINK_LIBRARIES "OTIO::opentime"
)

Note: edited to replace my build directories with [build directory]
This means the CMake configs exported by OTIO are pointing to temporary build directories and not useable (cannot be imported by the CMake find_package command).
The setup.py script is deliberately changing the install directory to the build directory so that it can build wheels in the build directory which can then be then be installed by Python, so we can't just set the install directory to the final installation directory without breaking the wheel installation.
After a discussion on slack with @meshula and @reinecke we determined the best course of action is to not install the C++ SDK in the python site-packages.  It's annoying for C++ devs to know the version of Python that installed the package and the CMake files in the project don't work anyway.  Since disabling the C++ install also disables installing the opentime/opentimelineio C++ libraries that the pybind libraries depend on, we decided to install these statically at the same time.  This is more convenient for wheel building anyway, since it makes the package more self-contained.
When statically linking I ran into warnings about symbol visibility since the pybind symbols are not visible and the otio ones are and they're now being compiled into the same translation unit. To solve this, I simply hid all otio symbols when linking with pybind.  In this scenario all otio symbols are only accessed by Python via pybind, so no need to expose them anyway.  This will also speed up our library load time since there's no reason the symbols will have to be loaded into the global symbol table.
I also fixed an issue where the deps c++ headers where being installed even when the c++ install was being turned off.
Lastly, I also moved the installation of the cmake config files to be in [install root]/share/opentime and [install root]/share/opentimelineio.  The reason is on most systems the default [install root] (ie /usr/local) is not writeable.  You are expected to install into one of the standard /usr/local subdirectories, so I chose share which is meant for architecture independent files.
Reference associated tests.
Build tests should cover this.
The package should be installable as either python or C++ or both.  The C++ includes should be in the default CMAKE_INSTALL_PREFIX unless overridden by the user and the installed CMake configs should be pointing at the correct install location.

#998 depends on this as well


How do I compile the source code for the release?

I want to compile the client of the release, and in order to achieve this, do I need to define some macros?

or undef some macros?

No, follow the build instructions: https://github.com/qtumproject/qtum/blob/master/README.md#building-qtum-core


How to stop server

I attempted to run dhttpd yesterday. Even after stopping the command with ctrl+c, the server continues to operate. I tried rebooting the computer, but the server is still running.

I attempted to run dhttpd yesterday. Even after stopping the command with ctrl+c, the server continues to operate. I tried rebooting the computer, but the server is still running.


API deprecate Objective.to_dict in favor of Objective.get_objective

closes #460

Codecov Report

Merging #489 (c5dc0e1) into main (5bd2525) will decrease coverage by 0.04%.
The diff coverage is 50.00%.

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

@@            Coverage Diff             @@
##             main     #489      +/-   ##
==========================================
- Coverage   53.10%   53.06%   -0.05%
==========================================
  Files          42       42
  Lines        2736     2742       +6
  Branches      500      501       +1
==========================================
+ Hits         1453     1455       +2
- Misses       1171     1174       +3
- Partials      112      113       +1

I tested it and it works as expected. However if we update a benchmark by replacing to_dict with get_objective, and the benchmark is run with a previous version of benchopt, it will no longer work as Objective does not have a to_dict objective...
Is there a way to not break things? Keep the to_dict in the benchmark, which calls get_objective and warns to update benchopt?

This is indeed an issue... I think we should add a min_benchopt_version in the benchmarks, that we  bump once we break compat and if you run a benchmark with older version of benchopt, it warns that you should update


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(package<EMAIL_ADDRESS>

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


init/linter: add linter to github workflow jobs

i added lint job to github workflows
fixed provider creation tests caused by linter

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 21.50%. Comparing base (1bf4c27) to head (dca65fc).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   21.50%   21.50%
=======================================
  Files           9        9
  Lines         651      651
=======================================
  Hits          140      140
  Misses        505      505
  Partials        6        6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

#33


Classify own image does not work, especially with imagenet of googlenet

I tried to input with my own images and I am not able to get the net to classify properly. I realised in your code, the classify uses a string? how does it make any sense?

The emulator can only classify it's own images, and it uses strings to refer to its own images.


Response headers get joined when multiple are received

The output of this snippet
data = ""
for i = 1, 2048 do data = data .. "X" end

print_r(request.send("https://httpbin.org/status/204", {
    method = "POST",
    data = data,
    headers = {
        ["content-type"] = "application/octet-stream",
    },
}))

(using print_r from here: https://coronalabs.com/blog/2014/09/02/tutorial-printing-table-contents/ )
is this:
table: 0x07dfd8a8 {
  [code] => "100"
  [raw_headers] => "HTTP/1.1 100 Continue

HTTP/1.1 204 NO CONTENT
Connection: keep-alive
Server: meinheld/0.6.1
Date: Wed, 19 Jul 2017 00:16:46 GMT
Content-Type: text/html; charset=utf-8
Access-Control-Allow-Origin: *
Access-Control-Allow-Credentials: true
X-Powered-By: Flask
X-Processed-Time: 0.00103116035461
Content-Length: 0
Via: 1.1 vegur

"
  [body] => ""
  [headers] => table: 0x07dfd8a8 {
                 [Date] => "Wed, 19 Jul 2017 00:16:46 GMT"
                 [Access-Control-Allow-Credentials] => "true"
                 [Access-Control-Allow-Origin] => "*"
                 [Content-Length] => "0"
                 [Server] => "meinheld/0.6.1"
                 [Via] => "1.1 vegur"
                 [X-Processed-Time] => "0.00103116035461"
                 [Content-Type] => "text/html; charset=utf-8"
                 [
HTTP/1.1 204 NO CONTENT
Connection] => "keep-alive"
                 [X-Powered-By] => "Flask"
               }
}

The response code returned is then 100, not 200 also the headers sub-table has a weird additional entry with the actual return header.
EDIT note: At first this issue included the same problem, but my guess of the cause was wrong, so I edited the content and the title.

Wow, my header parsing code is horribly broken.
I also don't think I accounted for multiple headers being returned from a response like that, which makes me think that I should be using real curl methods for parsing headers instead of patterns!


"Android" themed demo website side menu doesn't scroll on Chrome 45 OS-X Yosemite

The "Android" themed demo website's side menu will not scroll using the scroll-wheel / two-finger-slide-gesture on Chrome 45.0.2454.93 on Yosemite, OS-X 10.10.2
When emulating a phone such as the iPhone using Chrome's built-in tool (inspector) however it does work (it scrolls in emulation mode where the click is the same as a "touch" event).

This is most likely a duplicate of #1110. Closing this in favor of said issue.


Add simple cache.

Hi @ryanluker !
Every switch between tabs is running extraction. I am working with quite big coverage file and this results in considerable lag in displaying coverage. After a while this becomes mildly distracting.
I've seen that you are planning to add some caching in future but please consider this PR as intermediate solution.

@spodym thanks for the pull request. I dont mind pulling this in and making a dev release of the extension (generating an .vsix file that you would manually install https://code.visualstudio.com/docs/editor/extension-gallery#_install-from-a-vsix) but the amount of overhead in terms of scheduling testing and then deploying to the marketplace would be too high when I am close to finishing the new architecture. How does the dev release sound to you?

I agree that it might not be worth doing marketplace release. Besides in meantime I have managed to install my branch locally so I am good.
I am not sure if anyone else would benefit from this dev release so I leave decision up to you.
BTW. If you need a hand with testing or developing new version - I would be happy to help :)

@spodym good to hear you got this working locally. Thanks for the offer of help :smile: once I get the tasks broken down and fleshed out a bit more I will find one for us to collaborate on!


Investigate arb execution

Describe the bug
Look at the following output:
~/c/p/penumbra3 ((v0.66.0)|✔) $ cargo run --release --bin pcli --  q dex arb-execution --height 4645
    Finished release [optimized] target(s) in 0.28s
     Running `target/release/pcli q dex arb-execution --height 4645`
Scanning blocks from last sync height 4675 to latest height 4675
[0s] ██████████████████████████████████████████████████       0/0       0/s ETA: 0s
46.566735penumbra => 6.342551penumbra via:
    Trace                                                                                Subprice
    4.801446penumbra =>   6.241878888888888889test_usd =>   5.617691penumbra =>             5.617691gm =>          7.02211375test_usd =>  5.61769penumbra    854.702mpenumbra/1penumbra
    4.801446penumbra =>   6.241878888888888889test_usd =>   5.617691penumbra =>             7.02211375test_usd =>  5.61769gm =>           5.61769penumbra    854.702mpenumbra/1penumbra
    4.801446penumbra =>   6.241878888888888889test_usd =>   5.617691penumbra =>             5.617691gm =>          7.02211375test_usd =>  5.61769penumbra    854.702mpenumbra/1penumbra
    4.801446penumbra =>   6.241878888888888889test_usd =>   5.617691penumbra =>             7.02211375test_usd =>  5.61769gm =>           5.61769penumbra    854.702mpenumbra/1penumbra
    4.256778penumbra =>   5.533811344444433872test_usd =>   4.98043penumbra =>              4.98043gm =>           6.2255375test_usd =>   4.980429penumbra   854.702mpenumbra/1penumbra
    4.98043penumbra =>    6.2255375test_usd =>                                                                                5.602983penumbra   888.89mpenumbra/1penumbra
    637.261mpenumbra =>   637.261mgm =><PHONE_NUMBER>00000000wtest_usd =>                                                716.918mpenumbra   888.89mpenumbra/1penumbra
    5.617691penumbra =>   5.617691gn =>                     7.02211375test_usd =>                                                         6.319902penumbra   888.889mpenumbra/1penumbra
    11.868791penumbra =>  14.242548888888888889test_usd =>                                                                                12.818294penumbra  925.926mpenumbra/1penumbra

Expected behavior
The summary 46.566735penumbra => 6.342551penumbra seems wrong?

I'm not able to reproduce after "poking" at it but I think we should keep an eye out (and this issue open). Something that I find reassuring is that the individual traces have correct prices (<1 per penumbra), but the SwapExecution rendering logic leaves little place for a bug, so I think this is most definitely a bona fide DEX engine bug.
I invite others to take a look, I will try to passively find a reproduction by scanning the historical arb executions and try to look for inconsistent data looking like the one reported here.

I was able to reproduce the issue after creating a couple test positions:
$ cargo run --bin pcli --release -- tx position order sell 1000penumbra@1test_usd
$ cargo run --bin pcli --release -- tx position order sell 1000test_usd@100mpenumbra
$ cargo run --release --bin pcli --  q dex arb-execution --height 11548
    Finished release [optimized] target(s) in 0.30s
     Running `/Users/chrisczub/bin/sign-and-run target/release/pcli q dex arb-execution --height 11548`
Scanning blocks from last sync height 11559 to latest height 11559
[0s] ██████████████████████████████████████████████████       0/0       0/s ETA: 0s
100penumbra => 66.666666penumbra via:
    Trace                                                Subprice
    100penumbra =>  1000test_usd =>  166.666666penumbra  600.001mpenumbra/1penumbra

        let se = SwapExecution {
            traces: swap_execution.traces,
            input: Value {
                asset_id: arb_token,
                amount: filled_input,
            },
            output: Value {
                amount: arb_profit,
                asset_id: arb_token,
            },
        };

Why do we construct a new SwapExecution (I think it's because the existing one is inflated because of the flash loan, so this is intentional)
I am pretty sure the output is wrong if arb_profit is actually "profit" as described rather than "gross swap output" (i.e. is it profit = swap output - swap input? Because that would be wrong)

xref https://github.com/penumbra-zone/penumbra/issues/4113


readme warning for timeout failcase

Wait, I think I corrected this, is it still the case?

I'll run the test case against latest and take a look.

 Will do after lunch. :)

@SamSaffron I changed this pr into a passing testcase. :) woo!
(thanks for the fix; Gonna be running it with my game's test suite w/ upgraded v8, too.)

cool!


[WIP] Issue #47: Collect files opened by envoy pre-termination (lsof)

Task accomplished

fixes #47
example file written:

[
 {
    "command": "envoy",
    "device": "",
    "fd": "0",
    "name": "/dev/pts/3",
    "node": "6",
    "pid": "7218",
    "size": "0",
    "type": "",
    "user": "hengyuan"
  },
  {
    "command": "envoy",
    "device": "",
    "fd": "1",
    "name": "/dev/pts/3",
    "node": "6",
    "pid": "7218",
    "size": "0",
    "type": "",
    "user": "hengyuan"
  },
  {
    "command": "envoy",
    "device": "",
    "fd": "2",
    "name": "/dev/pts/3",
    "node": "6",
    "pid": "7218",
    "size": "0",
    "type": "",
    "user": "hengyuan"
  },
  {
    "command": "envoy",
    "device": "",
    "fd": "3",
    "name": "anon_inode:[eventpoll]",
    "node": "",
    "pid": "7218",
    "size": "",
    "type": "",
    "user": "hengyuan"
  },
  {
    "command": "envoy",
    "device": "",
    "fd": "4",
    "name": "anon_inode:[eventfd]",
    "node": "",
    "pid": "7218",
    "size": "",
    "type": "",
    "user": "hengyuan"
  },
...
]

Signed-off-by: Hengyuan<EMAIL_ADDRESS>

I have yet to find a library that returns the device and type fields of the opened files. Does anybody know of any library that can do that?


Menus are shown twice or triple times

Describe the bug
We have 3 serves: production , staging, and development.
(only on development admin changes are allowed)
Recently we noticed that menu items were doubles or tripled.
I fixed it on development (resetting did not help, so I manually disabled doubled items, then reset and it got fixed).
On staging all is fine.
However on production, it still shows doubled and tripled menus. We tried to force apply config on production, but did not help.
Steps to reproduce
I really don't know how to reproduce it...
Craft CMS version
3.7.57
Plugin version
3.0.17
Multi-site?
Yes
Additional context
Production: (where I can not change anything, because amdin changes are not allowed)

Development:

This should be fixed thanks to major refactoring in CP Nav 2 / Craft 4, but that can't really be back-tracked to Craft 3.
I might suggest downloading your database from production, running ./craft project-config/rebuild locally so that it's using the same information from production. There should be changes to your yaml files. Fix up your nav and push to production - that should help.

I upgraded craft now.
It was doing same, double and tripple items... despite rebuild.
and now, with craft4 - even doesnt show additional menu items that I had....
I submitted new ticket
https://github.com/verbb/cp-nav/issues/129


Search filters from the same category should be disjunctive

If I click on "Open" and "Predicted", I expect that I'll get questions that are open and I have predicted on. This currently works fine.
But if I click on "Open" and "Closed", I expect that I'll get questions that are either open or closed. This isn't the case currently. Instead I get the last of the two filters I selected.

Added here, closing this one.


Error removing empty automation namespace

After attempting to remove an empty automation namespace, I get "FATAL -- : Error caught: [NoMethodError] undefined method `namespace_id' for nil:NilClass" and it doesn't remove the namespace.  This also happens when I try to remove any the parent namespaces in this tree.  In the screenshot, I can replicate this every time when trying to remove "Zefflin / Service", "Zefflin / Service / Provisioning", or "Zefflin / Service / Provisioning / StateMachines".
Image: CFME <IP_ADDRESS>-2.x86_64.qcow2
Virt: KVM
I've attached two screenshots shows the before and after as well as the production.log trace.

cfme4-ns-remove-error.txt

@01000101 if this issue is no longer relevant can you please close.