Schema generation fails when multiple interfaces of a type implement the same base interface

Describe the Bug

The following is a valid GraphQL schema, which includes interfaces that implement other interfaces:
interface Base {
  id: String
}

interface InterfaceA implements Base {
  id: String
  fieldA: String
}

interface InterfaceB implements Base {
  id: String
  fieldB: String
}

type MyType implements InterfaceA & InterfaceB & Base {
  id: String
  fieldA: String
  fieldB: String
}

We implement it in Strawberry like this, using inheritance:
import strawberry

@strawberry.interface
class Base:
    id: str

@strawberry.interface
class InterfaceA(Base):
    id: str
    field_a: str

@strawberry.interface
class InterfaceB(Base):
    id: str
    field_b: str

@strawberry.type
class MyType(InterfaceA, InterfaceB):
    id: str
    field_a: str
    field_b: str

However, this results in an error when generating the GraphQL schema:
E           ValueError: Invalid Schema. Errors:
E
E           ❌ Type MyType can only implement Base once.

As the generated schema has:
type MyType implements InterfaceA & Base & InterfaceB & Base {
  id: String
  fieldA: String
  fieldB: String
}

I think the issue is in https://github.com/strawberry-graphql/strawberry/blob/20e202757ada7709e189f2b9b9cef6cb6023d5f5/strawberry/object_type.py#L33
I believe that the function _get_interfaces should be deduplicating the list of interfaces it gets by recursively traversing through a type's bases.
System Information

Operating system: MacOS Ventura 13.1 (22C65)
Strawberry version (if applicable): 0.164.0

Additional Context

@mzhu22 thank you for the bug report and the suggested location where this might be ground wrong! I'll get on this soon

@mzhu22 this was fixed by @skilkis  in #2647, feel free to reopen if the issue is still there!


Fixing failed metav1.Time test error for verify

When running make test I noticed that they were failing due to an error caused by the Expires field not accepting time.Time variables. I don't have full context but changing it to the below fixed the tests. Worth getting eyes from someone responsible for these tests in the past for sanity 😅.

cc @jkjell I think this was introduced when you make the changes for the data provider.

Ah. I see why this is a problem.
I am replacing go-witness in my go.mod with my local module. This was causing the error above with tests. Of course, this repository follows version 1.17 of go-witness, which still has this field as type time.Time.
This will need to be fixed eventually, but not yet.


Linkify detail page URL

@cptvitamin is this what you'd like?

I've rolled this change into #37, so I'm closing this PR.


investigate omitted tracts

after a few rounds of changes, several tracts were omitted from the final list: https://github.com/chihacknight/il-solar-map/commit/306b3d759e47268a307746d8466b6e9ce1eaf5a1

these were removed because we couldn't find centroids for them from the Census API:
Error looking up:<PHONE_NUMBER>0
Error looking up:<PHONE_NUMBER>3
Error looking up:<PHONE_NUMBER>1
Error looking up:<PHONE_NUMBER>0
Error looking up:<PHONE_NUMBER>4
Error looking up:<PHONE_NUMBER>2
Error looking up:<PHONE_NUMBER>0
Error looking up:<PHONE_NUMBER>5

addressed in https://github.com/chihacknight/il-solar-map/commit/86c390cc80bb4f876b23fd7b611d162f963628e0


you can use virtual items like they are actual items

Description
e.g. pulling a dead tarantula for JoL, or dragging a cleanbot and clicking puddles with the vitem

#30157
#30166
Seems we got dupes?

someone should close one then :trollface:


687 security

#687  added in checks to make sure user has access to features
@skhtet @rkenyon1969 @wski

:+1:

:+1:


[Question] Hosting on a Windows Server

Hi,
I could not find any documentation so I am assuming this does not work out-of-the-box but is there a method to deploy this as a service on a Windows Server?
We have some old servers outside of our cluster that want to install gatus on to monitor netwerk stability.
Unfortunately we cannot install things easily on it because the servers are air-gapped (intranet).
So running the 'development' application might not work for us.

Sure, you can deploy it on a windows server using Docker.
Otherwise, you could compile the executable yourself, but I'd recommend going with the Docker route.


change virtual env parts (docs links)

There is no Scripts folder in virtualenvs?

hm maybe because im using windows, maybe change that

The documentation got a major overhaul for 0.13 and I think this is fixed now. Thanks :)

Ok


Possible regression related to comprehensions

What version of CUE are you using (cue version)?

$ cue version
cue version 0.3.0-beta.5 linux/amd64

Does this issue reproduce with the latest release?
This works on version 0.2.2, I didn't try any other beta yet.
What did you do?

given following example.cue:
import (
	"strings"
	"list"
)

#A: {
	_namespace: string
	_range:     list.Range(1, 5, 1)
	_keys: [ for n in _range {
		"key: \(n) \(_namespace)"
	}]

	failing: strings.Join(_keys, ", ")
	working: strings.Join([_namespace], ", ")
}

B: #A & {
	_namespace: "foo"
}

$ cue eval ./example.cue

What did you expect to see?
import "strings"

#A: {
    _namespace: string
    _range: [1, 2, 3, 4]
    _keys:   [ for n in _range { "key: \(n) \(_namespace)" } ]
    failing: strings.Join(_keys, ", ")
    working: strings.Join([_namespace], ", ")
}
B: {
    _namespace: "foo"
    _range: [1, 2, 3, 4]
    _keys: ["key: 1 foo", "key: 2 foo", "key: 3 foo", "key: 4 foo"]
    failing: "key: 1 foo, key: 2 foo, key: 3 foo, key: 4 foo"
    working: "foo"
}

What did you see instead?
#A.failing: invalid type element 0 (_|_) of string list argument 0:
    ./example.cue:13:11

Sorry, this was a duplicate of https://github.com/cuelang/cue/issues/776

This issue has been migrated to https://github.com/cue-lang/cue/issues/796.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


First take at Dagger CLI

Only adds two sub-commands to run GraphQL queries and start the engine
API router initially.
Signed-off-by: Marcos Lilljedahl<EMAIL_ADDRESS>

Had a sync conversation with @marcosnils, this is what we are thinking cc @aluzzardi @sipsma @shykes @mircubed
flowchart TB
    classDef fail fill:#f00;
    classDef success fill:#090;
    tag[Git tag >= v0.3.4] --> code[Check out code]

    code --> cli[mage cli:publish]
    subgraph GoReleaser
        cli --> goreleaser
        goreleaser --> |cli| github[GitHub release v0.3.x]
        goreleaser --> |cli| s3[AWS S3]
        goreleaser --> |cli| tap[Homebrew tap]
        goreleaser --> |engine reference| github
    end

    subgraph Dagger
        code --> engine[mage engine:publish]
        engine --> |engine| oci[ghcr.io/dagger/engine:v0.3.4]
    end

Live graph editor

@marcosnils is this related to?

https://github.com/dagger/dagger/issues/3653

@marcosnils is this related to?

related but from the distance. The CLI will eventually use the CLI helper, but for the moment, according to what we spoke with @sipsma and @aluzzardi we'll keep using the legacy buildkit provisioner since it makes more sense at this stage.

https://github.com/gerhard/dagger/releases/tag/v0.6.1 produced via https://github.com/gerhard/dagger/actions/runs/3447576047
https://github.com/gerhard/dagger/releases/tag/v0.5.1 produced locally via ./hack/make cli:publish

I noticed this morning that my fork created a 0.5.1 engine package. This should not have happened. I deleted this package version manually.

The PAT that I am using has too many privileges. On top of that, the workflow shouldn't have run at all. Mentioning it here since I do not have the bandwidth to tackle it straight away. Leaving it as a reminder for a quieter time.

Current task list (offload):

[ ] merge publish-cli.yml & publish-engine.yml into publish-engine-and-cli.yml
[ ] only publish the CLI when we are publishing a tag
[ ] brew install<EMAIL_ADDRESS>- for the tap. Open question: does Homebrew core allow this?
[ ] validate env vars in cli:publish, e.g. sdk:nodejs:publish

Follow-up:

version current Homebrew bottled formula (dagger.rb -><EMAIL_ADDRESS>version current Homebrew tap formula (dagger.rb -><EMAIL_ADDRESS>- it's OK to inconvenience the few users that depend on this (talked to @aluzzardi about it)
publish

related but from the distance. The CLI will eventually use the CLI helper, but for the moment, according to what we spoke with @sipsma and @aluzzardi we'll keep using the legacy buildkit provisioner since it makes more sense at this stage.

I think this is coming to a head already, e.g. there's a lot of issues around building the shim on the fly that might be most easily addressed by just including the shim in the engine image (or by making the shim the same thing as the session router binary). But if we make that change then the CLI can no longer depend on the legacy-buildkitd provisioner and has to use the new engine image.

The problem is the weird circular dependency we discussed where the CLI is a component of engine but will need to hardcode a dependency on an engine image (similar to Go SDK) in order to use the new provisioning.
Possibilities:

Move the CLI out to be its own submodule, either sibling to the Go SDK or possibly a submodule with a dependency on the Go SDK.
Just accept the weird circular dependency for now

This would mean that we hardcode the engine image ref in the CLI code the same as we do in our SDKs. Similarly, we update it when there is a new engine release.
To avoid too many circular deps, we just need to not put the CLI inside the engine image and then configure our workflows to not build a new engine image when only the CLI code was updated

Don't change anything, keep using legacy-buildkitd provisioner

We can't fix the shim build process (or make any other changes that would require anything in our custom engine image to be present, e.g. CNI support, wasm interpreters, etc.)

2 is weird and kind of confusing, but afaict it would work for now? And also avoid the huge headache of managing the CLI as a go submodule the same way we do the Go SDK. Personally, if it is indeed workable, I think we should just do that now so that we aren't blocked on making changes like the one above to the shim.
However, if we are willing to accept the pain of 1, then it feels like the most "logical" approach in the long run.
cc @aluzzardi @marcosnils let me know what you think, only change from our last discussion is that there are already compelling needs for more engine image customization.

cc @aluzzardi @marcosnils let me know what you think, only change from our last discussion is that there are already compelling needs for more engine image customization.

If the image customization cases are becoming more important in a short time span then I'd say option 2 seems like the best one initially. Even though I don't like the circular spiral as well, option 3 involves some complexity that I'm not quite sure that it's worth putting the effort  ATM. @aluzzardi thoughts?

The problem is the weird circular dependency we discussed where the CLI is a component of engine but will need to hardcode a dependency on an engine image (similar to Go SDK) in order to use the new provisioning.
Possibilities:

Move the CLI out to be its own submodule, either sibling to the Go SDK or possibly a submodule with a dependency on the Go SDK.

Just accept the weird circular dependency for now

This would mean that we hardcode the engine image ref in the CLI code the same as we do in our SDKs. Similarly, we update it when there is a new engine release.
To avoid too many circular deps, we just need to not put the CLI inside the engine image and then configure our workflows to not build a new engine image when only the CLI code was updated

This one I don't understand: since the CLI relies on the Go SDK, isn't all this logic already handled in the Go SDK? And if so, why would we need to hardcode the engine image ref in the CLI code?

Don't change anything, keep using legacy-buildkitd provisioner

This IMO is not an option. It's too confusing from a UX point of view.

2 is weird and kind of confusing, but afaict it would work for now? And also avoid the huge headache of managing the CLI as a go submodule the same way we do the Go SDK. Personally, if it is indeed workable, I think we should just do that now so that we aren't blocked on making changes like the one above to the shim.
However, if we are willing to accept the pain of 1, then it feels like the most "logical" approach in the long run.

There is also an option 4: move the CLI under the Go SDK, since it is essentially a CLI version of that SDK.
It's not a perfect option, but it's very close to option 2 in its imperfections. So we should at least add it to the list and discuss it.

This one I don't understand: since the CLI relies on the Go SDK, isn't all this logic already handled in the Go SDK? And if so, why would we need to hardcode the engine image ref in the CLI code?

The current implementation and the one I believe we were going to go with (correct me if wrong @marcosnils ) is that the CLI would not rely on the Go SDK. It will just use engine.Start directly and thus bypass the SDK (but also by-passing the provisioning from the engine-session binary in the process).

There is also an option 4: move the CLI under the Go SDK, since it is essentially a CLI version of that SDK.

Yeah this is a variation on option 1 in my mind. Basically, move the CLI out of the engine code directly and have it either be inside the Go SDK code base or another submodule that has a dependency on the Go SDK. Personally I'd be okay with either, it's more just which one is a lesser headache right now if we go this route (though both are probably pretty big headaches in terms of release and version management).

The current implementation and the one I believe we were going to go with (correct me if wrong @marcosnils ) is that the CLI would not rely on the Go SDK. It will just use engine.Start directly and thus bypass the SDK (but also by-passing the provisioning from the engine-session binary in the process).

@sipsma since your last comment I've refactored the code to only use the SDK so we don't rely on the legacy mode anymore. Let's a have a quick chat today to hopefully come to a unified strategy here

CLI  code and UX in this PR is ready for review @gerhard @aluzzardi @sipsma @shykes .
After we decide where the CLI will live we need to finalize the publish specific tasks.

Just adding some conclusions after a chat with @shykes and @aluzzardi
1 - The CLI will live alongside the engine as drafted in this PR. We still need to polish some things around its deployment. More info about that here (https://github.com/dagger/dagger/issues/3830#issuecomment-1316288485).
2 - We've added DAGGER_SESSION_URL as a companion of DAGGER_SESSION_PORT  mostly to improve thx UX around DSI.
3 - dagger query will have two flags, --var and --var-json respectively to define arguments for the graphql query.

The CLI part of this PR is ready to :eye:
I'll open a new PR for the distribution changes. @gerhard @shykes @sipsma mind giving one last :eye: please?

@marcosnils my only outstanding topic is the interface for dagger exec (what variables does it set, and how do we document it). Since we don't have a resolution on #3830 yet, we should decouple (ie. we should ship something for dagger exec that does not restrict our options for DSI, but doesn't break when DSI is defined either.
I suggest:

dagger exec sets $DAGGER_SESSION_URL as a convenience. It is guaranteed to always be an http URL. This is not the same thing as DSI, just a convenience
dagger exec does not set $DAGGER_SESSION_PORT, to avoid confusion and interference with DSI
When DSI is defined, we can make follow-up modifications to dagger exec to be compliant (all SDKs will need to be modified anyway).

WDYT @marcosnils @aluzzardi @sipsma ?

WDYT @marcosnils @aluzzardi @sipsma ?

SGTM Solomon. just removed DAGGER_SESSION_PORT since I agree it shouldn't be there until DSI is initially defined.


Require one of the selected controls vs. Require all the selected controls

[Enter feedback here]
Is there any reason why Scenario 3 says 'Require one of the selected controls'? when Scenario 1 and 2 say 'Require all the selected controls'? Thanks
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 14348ba7-48f0-1862-ebec-0fc3640e587d
Version Independent ID: 8188cee4-3f28-6014-f3be-1ee9e9b183b6
Content: App protection policies with Conditional Access - Azure Active Directory
Content Source: articles/active-directory/conditional-access/app-protection-based-conditional-access.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@jd1984 Thank you for your feedback . We will investigate and update the thread.

@spunukol Thoughts?

Doc has been reviewed by the PMs and approved as written. Sorry I dont have any additional clarification. #please-close

Doc has been reviewed by the PMs and approved as written. Sorry I dont have any additional clarification. #please-close

Looks like you need to install the Intune Company Portal app on iOS (as well as Android) if you have the Require approved client app box ticked. Thanks to MS support for helping me to resolve the issue!

Yes, I found this all rather confusing and difficult to configure and get working right,
based on the documentation available.
I notice Scenario's 1 and 2 have both now been changed to say 'Require one of the selected controls'
although when I queried this above, you can see the 'Doc has been reviewed by the PMs and approved as written.'
On a side note, I believe you can also use Microsoft Authenticator app in place of Company Portal.
These apps both act as a broker to register the device with AAD in a MAM-WE scenario.


updating version matrix

Per our email chain:
Current Dynatrace service broker supports OpsMan + PAS through 2.2.x and all IaaS vendors without code change.

I should add that the release notes were NOT updated because this is simply a doc change.

sorry, my mouse hand twitched and I accidentally hit the close button.


First repo

Want to join

Want to join ```

[ ] ### > ~~@Vorobeyko __~~

You can ofc fork this project


Create a bootstrap governance document

Foundation work for #1. The governance document is based on the OpenTelemetry, Keptn and some bits of Contributor Covenant and TAG App Delivery
Preview: https://github.com/openfeatureflags/governance/tree/governance-docs
Signed-off-by: Oleg Nenashev<EMAIL_ADDRESS>

FTR I approve the changes and not blocking merge. I would prefer to have separate governance and community repositories, but we can tackle it later. Will create a follow-up GitHub issue


Add app wrapper (FOR-10)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Renaming resolve methods.

Migrated from https://github.com/saiichihashimoto/sanity-typed-schema-builder/issues/160
Essentially the resolve methods and zods are named weirdly. They imply that they'll replace all the references with the actual documents, but that's untrue. That's not something we could even do. These libraries aren't going to groq for you, or know you have local references or anything of that nature. What it actually does is replace those references with mocks, which should be a lot clearer.
The other issue is that we should also infer a type that has everything resolved, so someone (@miklschmidt) can use that after they've resolved their own references. Essentially, resolving a doc for someone is not the library's job but giving the type of a resolved doc should be.

Sounds good to me 👍
As an addition, it would be nice to be able to combine zods, so you can parse the result of a groq query that returns a fully resolved document. It won't work for custom queries, but at least it would allow you to make sure your groq queries return what you think you queried for, and it would greatly help cut down on API requests and code.
Example data:
post (doc1)
    - author -> doc2
    - tags -> array of doc3
    - relatedPosts -> doc1

I haven't really fleshed out any of the details, but I have a milestone in here that I've decided to pursue that might lead to what you're saying:
https://github.com/saiichihashimoto/sanity-typed/milestone/1
I'm of the opinion that schema-builder doesn't need to exist at all and was just a symptom of the times. I'm convinced @sanity/types should be able to infer it's own types completely. I'm building a proof of concept in @sanity-typed/types that exports the same methods with an infer method. I'm hoping to find a way to merge those changes back into @sanity/types to make @sanity-typed/types also irrelevant. Off of all of that, there should be a way to generate zods directly off of typed schemas, generate mocks off of the same schemas, and maybe even a sane way to create typed groqs. I think getting this all as close to "native" as possible while having all our types is the way to go.
We'll get there!

Now that @sanity-typed/types is mostly ready-to-go, schema-builder is on its way towards deprecation. Renaming the resolve methods isn't going to happen, so I'm closing this.


Problem with motion correction binning determination

There are cases where superres data is binned but Murfey still suggests a motion correction binning factor of 2

This is a consistent issue and multiple changes have been made since this issue was opened. May still not cover all cases but can be dealt with in later, more specific issues


Plugin is not activating zones

Context

HomeBridge version 1.8.4
Plugin version 1.3.7
NodeJS version 20.17.0
Rachio API version 2.7

The plugin is successfully invoking the Rachio API as evidenced by the following log listing...
[07/10/2024, 12:19:22] Loaded plugin<EMAIL_ADDRESS>[07/10/2024, 12:19:22] Registering platform 'homebridge-rachio-irrigation.rachio'
[07/10/2024, 12:19:24] [Rachio] Initializing rachio platform...
[07/10/2024, 12:19:24] [Rachio] Starting Rachio Platform with homebridge API 2.7
[07/10/2024, 12:19:25] [Rachio] Found cached accessory, configuring "xxxxx"
[07/10/2024, 12:19:25] [Rachio] Setting up Wifi hub devices
[07/10/2024, 12:19:25] [Rachio] Getting Person info...
[07/10/2024, 12:19:26] [Rachio] Found Person ID "xxxxx"
[07/10/2024, 12:19:26] [Rachio] Getting Person ID info...
[07/10/2024, 12:19:26] [Rachio] Found Account for username "xxxxx"
[07/10/2024, 12:19:26] [Rachio] Getting Location info...
The plugin was working just fine until about three weeks ago.
Problem
I'm no longer getting zone interaction with the Rachio controller from any of the HomeKit apps (e.g., Home, Home+) or the HomeBridge Accessories page.  The HomeKit side of the plugin is working as I can see a zone status change in the Home app when I change a zone's status from the HomeBridge Accessories page and vis-a-versa.  However, nothing happens on the Rachio side.  I see no log messages indicating the plugin has invoked the Rachio API.  Furthermore, as indicated by the above log listing, it does not appear the plugin gets any Location info, nor does it log a successful webhook.

Could you enable debug logging and attach the logs?

The log I copied above is with Debug enabled and with "showAPIMessages": true" in the config file.  I filtered the Rachio entries rather than the whole log.  I filter the main HomeBridge log.  Is there another log you would like me to attach?

Do you have homebridge debug logging on in the homebridge setting? I would expect a lot more info.

I had logging to the UI enabled but not full HomeBridge -D logging turned on.  I've now done that and have a large log.  Would it be possible to send that to you directly rather than through GIT?

you can<EMAIL_ADDRESS>

Your log is trying to load Smart hose valves and your person response show a controller. I am not sure exactly why homebridge shows the accessories I would expect them to be removed, I may need to look at that and I may need to look at logging this condition where the response returns and empty array.
I suspect you config file change along the way, there is section that may look like this, if so
"showBridge": false,
"showControllers": false,                         You want this line true
"showValves": true,                                  You want this line false
"liveRefreshTimeout": 2,
in the config this would be the setting for "Expose controller in HomeKit"
For valves this is "Expose valves in Homekit" under Smart Hose valve

That did the trick.  I don't know how my config changed to enable valves and disable controllers.  The only change I've made to the Rachio environment is to go with a single "tile" in HomeKit and then show the zones within that tile.  I was showing all the zones directly before that change.
We can close this issue.

I am going to change some of the text in the logging to be a little more discriptive with respect to controllers and valves. If that was there I think you may have figured this out right away.


Plugable storage 2.5

Adds StorageExtensionByProperty to have a journal for different journal configurations
cherry picked and merged from 2.4

@dnvriend Would be great to get this in 👍


Alligned the reset button to the reset text

Changes

Implementation details

How to read this PR

Screenshots

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[CARBONDATA-3588]: Table id taken as key generated from table path for non-transactional table

Modification reason: Show cache command on table with index server enabled displays the index size incorrectly for table that was created with index server disabled
Modification content: In case of non transactional table, the table id is now taken as the key generated from the table path
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/929/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/928/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/921/

LGTM

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/946/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/954/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/956/

retest this please

@kumarvishal09

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/949/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/959/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/957/

retest this please

retest this please

retest this please

@kumarvishal09 , @ajantha-bhat . Please merge

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1042/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1105/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1114/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1124/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1123/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1132/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1142/

retest this please

retest this please

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1149/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1158/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1168/

LGTM


You beauty!

Hi CDRX,
I've just tried this and it worked first time. Thank you so much - great for dev environments, and I'm sure (I just need to get more familiar with gitlab to work out how to segregate the flows for dev and prod)
Thanks again,
Andy

No worries! 👍


Feature: Can use env name in the server url in place of id

Hi,
It would be nice to can use an alias or the environment name in the url of the server.
Ex:
.../r/projects/kube-prod/kubernetes in place of .../r/projects/1a9/kubernetes

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


MTPROTO PROXIES

how can add MTPROTO PROXIES?

It is a very difficult way, it is better to use another proxy

in IRAN other proxy not working, only MTPROTO  work well


Files and carml-jar

There is a new cli tool for carml available: https://github.com/carml/carml-jar
I could get it to work with XRM carml output but I had to figure out how to do it properly. With the current configuration I can only get it to work with piping the file as a stream and I have to do the XRM source like this: source "". This generates:
<#MapCanton>
	a rr:TriplesMap;

	rml:logicalSource [
		rml:source [
			a carml:Stream;
			carml:streamName "";
			carml:declaresNamespace [
				carml:namespacePrefix "eCH-0071";
				carml:namespaceName "http://www.ech.ch/xmlns/eCH-0071/1"
			]
		];
		rml:referenceFormulation ql:XPath;
		rml:iterator "/eCH-0071:nomenclature/cantons/canton"
	];

I can then run: cat input/eCH0071.xml| java -jar bin/carml-jar-1.0.0-SNAPSHOT-0.4.4.jar map -m src-gen/mapping.carml.ttl
However, I cannot run it directly from a file, I always have to pipe it. So I checked out the carml docs and it says the name can be left out or empty, so that seems to be correct.
So I checked out the ontology and I found stream:url, so I changed the generated output to this (manually):
<#MapCanton>
	a rr:TriplesMap;

	rml:logicalSource [
		rml:source [
			carml:url "eCH0071.xml"; # cannot have a carml:Stream in this case
			carml:declaresNamespace [
				carml:namespacePrefix "eCH-0071";
				carml:namespaceName "http://www.ech.ch/xmlns/eCH-0071/1"
			]
		];
		rml:referenceFormulation ql:XPath;
		rml:iterator "/eCH-0071:nomenclature/cantons/canton"
	];

And then I can run it directly from file:
 java -jar bin/carml-jar-1.0.0-SNAPSHOT-0.4.4.jar map -m src-gen -rsl input -of ttl -P
This probably would work in XRM via rml output but I use the XML Namespace extension so I have to stick with carml-syntax.
Not sure how to handle that properly in XRM though, would be nice if both options could be generated.

I understand this as follows: carml requires different rml:logicalSource definitions, depending on if either a stream source or a files source should be used.
For stream sources:
	rml:logicalSource [
		rml:source [
			a carml:Stream;
			carml:streamName "";
			...
		];
		...
	];

For file sources:
	rml:logicalSource [
		rml:source [
			carml:url "eCH0071.xml";
			...
		];
		...
	];

Using output carml in xrm version 1.2.0 generates the "stream sources" definition.
The new requirement is now, that both variations can be generated from xrm. Is that correct @ktk ?

Correct interpretation.
It would be nice as people can use XRM and create carml output without having to interact with streams that way. For those that do not necessarily use a barnard59 pipeline, that's probably a nice thing to have.


Keywords Press Key and Press Key Native

Originally submitted to Google Code by filip.ne... on 6 Apr 2009

Selenium commands such as keypress, keyup, ... are not implemented. They
are useful for manipulating pages in which <input onkeydown="..." is used.

Originally submitted to Google Code by filip.ne... on 8 Apr 2009

I've performed the source code changes myself, here's the patch: I've implemented two
keywords: Press Key and Press Key Native. The second one uses the keyPressNative
operation of Selenium RC. keyPressNative is the only thing that works for Internet
Explorer 7, keyPress doesn't have any impact.
The reason for which I implemented this is that I need to send an <enter> key to a
text field, which is implemented with an onkeydown event listener that performs an
action if the <enter> key is pressed. I assume this can also be useful to others.

Originally submitted to Google Code by @yanne on 13 May 2009

Decided to implement these and the related Focus keyword in upcoming 2.2 release,
see also  issue 40

Originally submitted to Google Code by @yanne on 13 May 2009

Implemented in r166 and r167.


Underscores escaped in link destination urls

Subject of the issue
Having a link with an url that contains a _ will escape it.
Steps to reproduce
For example, the following tree:
├─38 paragraph[3] (77:1-77:229, 4501-4729)
│   └─0 link[1] (77:163-77:229, 4663-4729)
│       │ title: null
│       │ url: "https://www.youtube.com/channel/UCM1CbBpxRg_-FOutzAXjuLg"
│       └─0 text "Jovely" (77:164-77:170, 4664-4670)

will produce this markdown:
[Jovely](https://www.youtube.com/channel/UCM1CbBpxRg\_-FOutzAXjuLg)

Other examples:
[@\_thelustlist\_](https://www.instagram.com/\_thelustlist\_/)

Test markdown in github:

Jovely
@_thelustlist_

Expected behavior
The _ in the URL should not be escaped.
Actual behavior
The _ in the URL is escaped.

just realized that the escaping is correct. sorry for the noise.

:+1:
Escaping can be better, but escaping markdown is rather complex, so I don’t easily see how this one would work. The escaping here does not cause any harm, as the escapes in destinations work too.
There was a similar issue recently but I couldn’t find it.

so I don’t easily see how this one would work.

right.... I think in the previous remark-parser, it didn't escape the _ in link urls.

That is correct. But the previous compiler had bugs, and this one has less bugs ;)

@wooorm there is also escaping of &, which is also unexpected.  what is the exact reason to escape anything within the ( ) of the link url? (except maybe a )) ?

https://github.com/syntax-tree/mdast-util-to-markdown/blob/5fa790ee4cdead2a6c11b6a6f73c59eb0f9ca295/lib/unsafe.js#L40-L41

maybe treating the link URL as phrasing is wrong :-)

Character references and -escapes work in link destinations, it’s intended behavior

Character references and -escapes work in link destinations, it’s intended behavior

so you mean that this md:
[foo][https://example.com/test".html]

would create a link url with https://example.com/test".html when parsed?

yes.
We’re serializing markdown here, so if someone had actually escaped that:
[foo](https://example.com/test&quot;.html)
[foo](https://example.com/test\".html)

...we need to make sure it roundtrips.


Swift iOS from Microphone Sample app doesn't work

Description
Sample app for Swift IOS from Microphone doesn't display the recognized text. Throws error
020-08-13 17:56:21.189503+0530 helloworld[3869:86736] [] nw_connection_get_connected_socket [C1] Client called nw_connection_get_connected_socket on unconnected nw_connection
2020-08-13 17:56:21.189766+0530 helloworld[3869:86736] TCP Conn 0x600003345a40 Failed : error 0:-65554 [-65554]

To Reproduce
Steps to reproduce the behavior:

Clone the repo
pod install and opened it in Xcode

Expected behavior
Recognized text should be shown
Version of the Cognitive Services Speech SDK
SDK version : 1.7
MacOS: 10
XCode version: 11
iPhone 11 simulator: iOS 13
Sample app displays blank screen

Hi, can you try uploading to the latest version of the SDK?

Hi,
Tried SpeechSDK version 1.13.0
Still the recognised text is not getting printed
Getting this error

@Ananya-96 Thanks for report, the problem seems to be in you Mac microphone settings as I tested the sample with the latest 1.13.0 release and sample was working OK, Please check your microphone is configured correctly on Mac. Please also check if you need to add microphone access in your Mac's privacy settings for Xcode similarly as described in this issue
https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/516

@jhakulin Thank you for the feedback. I have tested it again in a different phone. Think its an issue with Mac microphone. Thanks.


nlp text classification example with imdb dataset

Binary text classification example with IMDB reviews dataset

named it 10-imdb, pls change if desired :)
similar complexity as the mnist example
copied the requirements.txt from the mnist example, don't know if everything is needed

Added a test for the imdb example


Application crashes after touch home button

Application crashes for me after touching home button.In SideMenuTransition.hideMenuStart() SideMenuTransition.viewControllerForPresentedMenu! controller is nil.
My init looks like:
 let leftViewController:UISideMenuNavigationController = UIStoryboard(name: "Main", bundle: nil).instantiateViewControllerWithIdentifier("UISideMenuNavigationController") as! UISideMenuNavigationController
         SideMenuManager.menuLeftNavigationController = leftViewController
        SideMenuManager.menuAddScreenEdgePanGesturesToPresent(toView: self.view)
        SideMenuManager.menuPresentMode = .MenuSlideIn
        SideMenuManager.menuAnimationBackgroundColor = UIColor.clearColor()

Hi @misha83, I'm not able to reproduce this problem using the latest build of the library with my iPhone 6S running 9.2. Can you re-run pod install on your project and give it another try? If it still happens, please give me more details as to how I can recreate this issue.
Also, you don't need to explicitly set SideMenuManager.menuAnimationBackgroundColor = UIColor.clearColor() as it starts out this way.

Closing until more feedback is provided.

i also had same problem with this line comes to nil. Please give me a solution about that?

Hi @barankaansert. I still cannot reproduce this issue. I'd need more feedback about the problem similar to my request to @misha83 above.

Hi @jonkykong, I created a cell and create a segue from this cell back to the frontView (home view). When I run and click that cell, the above issue happens.

@leizard what exception are you getting? Is this an unwind segue?

I use one side menu for multiple view. So when I'm at a subview, I want to open side menu and click 'Home' item to go back to Home. The issue happen when I'm currently at home view and click to the Home item in side menu. And yes, it is an unwind segue

@leizard what exception are you getting?

I'm sorry for not reading your reply carefully. The exception is the same as the first post. In internal class func hideMenuStart() , SideMenuTransition.viewControllerForPresentedMenu! controller is nil. and it crashes.
I just comment out
SideMenuManager.menuLeftNavigationController = storyboard!.instantiateViewControllerWithIdentifier("LeftMenuNavigationController") as? UISideMenuNavigationController
in my menu init and it doesn't crash anymore

Why does commenting that out in your init solve the problem? It sounds like you're removing the menu entirely because of that. Is the init called again when you return to your home screen?

@leizard I have a theory -- can you verify? My guess is that unwind segues cause the view to be reinitialized, whereas popping/pushing on the stack don't do that. If you're setting up SideMenu in your home view, then any time you unwind to it from the menu, you're replacing the side menu while it's presented, causing this crash.
If instead you initialized SideMenu from your app delegate, or put a simple check in your init that it should only set menuLeftNavigationController if it was nil, that should resolve the crash.
Let me know what you find.

@leizard i got same issues before ..... you can try change value for SideMenuManager.menuAddScreenEdgePanGesturesToPresent(toView: self.view)
to
SideMenuManager.menuAddScreenEdgePanGesturesToPresent(toView: leftViewController.view)
hopely it can help ...

Any chance you have a project you can share with me to investigate?

I'm diving other issues and can't test your suggestion right now. I will response asap when I can.
Thank you very much for your time and quick replies @jonkykong @fajaradiwa

Hi @jonkykong, I think you are right. I put a check like this
        if (SideMenuManager.menuLeftNavigationController == nil) {             SideMenuManager.menuLeftNavigationController = storyboard!.instantiateViewControllerWithIdentifier("LeftMenuNavigationController") as? UISideMenuNavigationController         }
in my frontview controller
And everything is fine.
So I'm closing this issue.
Thank you very much for your support 👍

But I think it is better to detect if it is prepare to segue to the same frontview in prepareForSegue method, so if it is, I just need to close the side menu

@leizard I'm thinking I will fix this by preventing the left or right side menus from being changed while they are actively presented. If this happens, I'll display a warning in the console and ignore the change, preventing the exception. Thoughts?

Hi @jonkykong, sorry for late response. As I understand, each view has to register their side menu. So I want 1 side menu on multiple view, I have to register this side menu multiple times. In other words, I will lose my side menu when I go to another view. Please correct me if I'm wrong.
So, when I put a item segue to the presenting front view and click it, the front view will reload again, which it doesn't need to because it is the same view. I think your fix could solve this but I don't it fits in a scenario when we want a side menu on multiple view.

@leizard that is incorrect. The design of SideMenu is such that you can set it up once and reuse it on multiple screens. At most you have to wire up buttons to present the left/right menus or gestures to reveal them, but you don't need to set the left/right menu more than once (unless you want to change them on purpose).

@leizard @fajaradiwa @misha83 @barankaansert I checked in a fix that prevents side menus from being changed while the menu is actively presented, which would cause crashes.

Thank you @jonkykong, I think it would do it.


CLI for load balancers

What problem are you facing?

Automation of setup and configuration of the Equinix load balancer (LBaaS)
How could the Equinix Metal CLI help solve this problem?

Using the documentation provided at
https://github.com/equinix/lbaas-api-docs
add a set of "metal load-balancer" commands that would set up,
configure, modify, and tear down load balancers, for CI automation
or for testing.

Thanks @vielmetti
Closing this as a duplicate of #411


Chinese entity does not match the description of the entity

这三张图中是一些出现问题的例子，description中的实体名字变成了别的东西，而且导致description自身也不正确。有人知道为什么吗？目前别的地方没有看到类似的情况，只有entity里有出现。

Posting an english translation of the original post using Google Translate.

These three pictures are some examples of problems. The entity name in the description becomes something else, and the description itself is incorrect. Does anyone know why? At present, we have not seen similar situations elsewhere, only in entities.

Consolidating language support issues here: #696


Bug in parameterized routes

Describe the bug
when attempting to use a hyphen (-) in a parameterized route (ex: '/movies/:movie-slug') Get does not recognize it. This appears to be due to the way in which the Regex in _replace is configured where it doesn't accept hyphens. Should this be updated to accept all valid URL encoded characters? RegExp(r'(.)?:(\w+)(?)?')
**Reproduction code
GetPage(
name: '/movies/:movie-slug',
page: () => MoviePage(),
),
To Reproduce
Create a route as shown in reproduction code
Expected behavior
A clear and concise description of what you expected to happen.
Flutter Version:
2.0.1
Getx Version:
3.26.0

Same issue: https://github.com/jonataslaw/getx/issues/1129


Documentation about ClickGestureRecognizer

Documentation about ClickGestureRecognizer is missing.

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

ID: b1d849b9-e4a6-1507-4532-26daa94ffaae
Version Independent ID: 07ae51c4-93fe-377e-ad12-41246a6f2e10
Content: Xamarin.Forms Gestures - Xamarin
Content Source: docs/xamarin-forms/app-fundamentals/gestures/index.md
Product: xamarin
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @johannes-schmitt
We have written some documentation for the ClickGestureRecognizer (which is available for desktop platforms such as Mac), and desktop support in general, but have held of from publishing it until some of the bugs in this area have been addressed.

HI @davidbritch, can you point me to the bugs you mentioned because we are going to use ClickGestureRecognizer and would like to avoid to run in known issues.

Hi @johannes-schmitt
It looks like the only remaining bug that's open for ClickGestureRecognizer stops it working on views contained in an ItemTemplate being used by a ListView: https://github.com/xamarin/Xamarin.Forms/issues/1658


Performance drop in master from azure-storage-blobs_1.0.0-beta.2

Describe the bug
A clear and concise description of what the bug is.
In azure-storage-blobs_1.0.0-beta.2, upload a single 10GB file to block blob cost ~8 seconds.
After upgrade azure-sdk-for-cpp to current master, it cost ~14 seconds.
Exception or Stack Trace
Add the exception log and stack trace if available
To Reproduce
Steps to reproduce the behavior:
Code Snippet
Add the code snippet that causes the issue.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: [e.g. iOS]
IDE : [e.g. IntelliJ]
Version of the Library used

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

i'll check tomorrow

didn't see perf drop
Track2 Upload:
    Transfer 5000 5byte blobs with 32 threads
    934 ms, 0.025527 MiB/s, 5353.319058 op/s
    Transfer 5000 10KiB blobs with 32 threads
    862 ms, 56.645157 MiB/s, 5800.464037 op/s
    Transfer 1000 10MiB blobs with 32 threads
    3602 ms, 2776.235425 MiB/s, 277.623542 op/s
    Transfer 16 1GiB blobs with 8 threads
    16237 ms, 1009.053397 MiB/s, 0.985404 op/s
    Transfer 64 1GiB blobs with 32 threads
    22867 ms, 2865.964053 MiB/s, 2.798793 op/s
Track1 Download:
    Transfer 5000 5byte blobs with 32 threads
    408 ms, 0.058436 MiB/s, 12254.901961 op/s
    Transfer 5000 10KiB blobs with 32 threads
    451 ms, 108.266353 MiB/s, 11086.474501 op/s
    Transfer 1000 10MiB blobs with 32 threads
    6019 ms, 1661.405549 MiB/s, 166.140555 op/s
    Transfer 16 1GiB blobs with 8 threads
    16032 ms, 1021.956088 MiB/s, 0.998004 op/s
    Transfer 64 1GiB blobs with 32 threads
    35112 ms, 1866.484393 MiB/s, 1.822739 op/s
Track2 Download:
    Transfer 5000 5byte blobs with 32 threads
    381 ms, 0.062577 MiB/s, 13123.359580 op/s
    Transfer 5000 10KiB blobs with 32 threads
    389 ms, 125.522172 MiB/s, 12853.470437 op/s
    Transfer 1000 10MiB blobs with 32 threads
    6023 ms, 1660.302175 MiB/s, 166.030217 op/s
    Transfer 16 1GiB blobs with 8 threads
    15141 ms, 1082.094974 MiB/s, 1.056733 op/s
    Transfer 64 1GiB blobs with 32 threads
    35064 ms, 1869.039471 MiB/s, 1.825234 op/s
Track1 Upload:
    Transfer 5000 5byte blobs with 32 threads
    820 ms, 0.029075 MiB/s, 6097.560976 op/s
    Transfer 5000 10KiB blobs with 32 threads
    827 ms, 59.042473 MiB/s, 6045.949214 op/s
    Transfer 1000 10MiB blobs with 32 threads
    3785 ms, 2642.007926 MiB/s, 264.200793 op/s
    Transfer 16 1GiB blobs with 8 threads
    13736 ms, 1192.778101 MiB/s, 1.164822 op/s
    Transfer 64 1GiB blobs with 32 threads
    24399 ms, 2686.011722 MiB/s, 2.623058 op/s


Fixes quota bug when not using pools

Changes proposed in this PR

turning on --auth=http-basic for the travis run without pools
fixing a bug in quota-checking when running without pools

Why are we making these changes?
The travis change allows multi-user tests (e.g. test_preemption) to run in the no-pools build. The bug went unnoticed because these tests were previously not running on that build.

@shamsimam Can we merge this?


[WinError 6] The handle is invalid on PyInstaller bundled Gooey application

[ ] OS - Windows 10
[ ] Python Version  - 3.10.4
[ ] Gooey Version - 1.2.0a0

Hi getting [WinError 6] The handle is invalid from Gooeys seeder.py whenever I try to 'start' my application via gooey.
Adding a null stdin to the subprocess call seems to fix it (as per answer here https://stackoverflow.com/questions/40108816/python-running-as-windows-service-oserror-winerror-6-the-handle-is-invalid):
proc = subprocess.Popen(
            cmd,
            stdout=subprocess.PIPE,
            stderr=subprocess.PIPE,
            stdin=subprocess.DEVNULL,
        )

I also had to change the command handling to be passed as proper arguments for it to work in all cases (gooey, no gooey, pyinstaller gooey):
def communicate(cmd, encoding) -> Try:
    """
    Invoke the processes specified by `cmd`.
    Assumes that the process speaks JSON over stdout. Non-json response
    are treated as an error.

    Implementation Note: I don't know why, but `Popen` is like ~5-6x faster
    than `check_output`. in practice, it means waiting for ~1/10th
    of a second rather than ~7/10ths of a second. A
    difference which is pretty weighty when there's a
    user waiting on the other end.
    """

    import os
    if os.name == 'nt':
        cmd = cmd.split()
        for c, e in enumerate(cmd):
            if not e.startswith('-') and not e.startswith("'") and not e.startswith('"') and 'gooey' not in cmd[c-1]:
                cmd[c-1] = cmd[c-1] + ' ' + cmd[c]
                cmd[c] = ''
        for c, e in enumerate(cmd):
            cmd[c] = cmd[c].replace('"', '')
        cmd = list(filter(None, cmd))
    else:
        cmd = cmd.split()
        for c, e in enumerate(cmd):
            if not e.startswith('-') and not e.startswith("'") and 'gooey' not in cmd[c-1]:
                cmd[c-1] = cmd[c-1] + ' ' + cmd[c]
                cmd[c] = ''
        for c, e in enumerate(cmd):
            cmd[c] = cmd[c].replace("'", '')
        cmd = list(filter(None, cmd))
    try:
        proc = subprocess.Popen(
            cmd,
            stdout=subprocess.PIPE,
            stderr=subprocess.PIPE,
            stdin=subprocess.DEVNULL,
        )
        out, err = proc.communicate()
        if out and proc.poll() == 0:
            return Success(deserialize_inbound(out, encoding))
        else:
            return Failure(CalledProcessError(proc.returncode, cmd, output=out, stderr=err))
    except JSONDecodeError as e:
        return Failure(e)
    except Exception as e:
        with open('test.txt', 'a') as f: f.write("\ncommunicate error:" + str(e))

Error still seems to occur on Windows 11 after my changes - not sure whats happening


[API] Get user groups

Hi,
It would be great if there is a way to get the list of the groups for 1 user(getUserGroups). I can actually obtain this result using getAllGroups for getting a list of all groups and checking if the user is in using isGroupMember but  it will be more effective to use an API function to do it.

It's available in master, use the method "getMemberGroups".


gray overlay in pdf view

Description

pdf has a gray overlay when opened with standard settings in dark version of the theme
the pdf is displayed normally when opened in the light version of the theme

Screenshots

the gray overlay only appears in the "white" view of the pdf

if "sepia" or "dark blue" is chosen under "more settings" the pdf is displayed as expected

Thanks for the beautiful and clean theme!

Hey thanks for making me aware of this.
It's very likely the issue mentioned in #11. As yours comes with screenshots and description I know what to look for 🧐☺. I'll take some time this evening to investigate. Have a good day!

Should be fixed in the new version. Please let me know if theres something else.

@tobealive the issue seems fixed now, thank you!
However the toolbar background looks a bit weird now (or still?) on the very left side.

I've thrown an update on it. Let me how how it is on your machine. Currently I don't have an apple machine at hands.

The issue is fixed now, thanks a lot!


Question: Should API return data in the same order each time or should the frontend do this?

intervention API returns data in a different order after it has been updated.
Input: descending 1,2,3,4,5,6,7,8,9,10
Data returned by API: descending 10,2,3,1,4,6,7,8,9,5
Linked issue: https://github.com/micronutrientsupport/database-architecture/issues/333

Answer: yes it should (and fix to DB/API incoming)


Use Apache as Webserver

Hallo Zulip-Team,
I'm currently working to make Zulip run with Apache as Webserver in front of it. The installation script is setting up a Nginx Webserver which we wouldn't use if possible. Our school uses apache for every other service we run, so it would be easier to maintain apache for us. We also want to use apache sso.
Is there any kind of example config or documentary for using apache for Zulip?
Is it possible to useApache in front of Nginx while still using Apache for sso?
Thanks!

I assume you've seen https://zulip.readthedocs.io/en/latest/production/authentication-methods.html#apache-based-sso-with-remote-user for doing SSO?  That handles the SSO issue.
Given that Zulip manages the nginx configuration for you, I would expect there won't be any real maintenance burden from having Zulip be the exception that uses nginx in an Apache shop.
That said, we don't have ready instructions for putting Zulip behind an Apache reverse proxy, but if you'd like to figure it out, we have instructions for various other webservers, and we'd love a contribution to add Apache docs to https://zulip.readthedocs.io/en/latest/production/deployment.html#putting-the-zulip-application-behind-a-reverse-proxy
(And I'd be happy to provide advice/answer questions while you set that up).

I will first try nginx with sso. Afterwards I will try it with Apache.
I will make die Apache configs public when they are ready.

We don‘t expect to switch from  Nginx to Apache any  Timeline soon... so i will close this issue.

For those still looking for Apache2 instructions, see here: https://github.com/qnxor/zulip/blob/master/docs/production/deployment.md#apache2-configuration
PR above. Tested with a Docker install, but should work just as well for a plain install.
p.s. For a Docker install you need to make the changes for zulip.conf and settings.py in docker-compose.yml instead:
 ...
 zulip:
    ports:
      - "5080:80"
      #- "5443:443"  # disabled, see below
    environment:
      ...
      DISABLE_HTTPS: 'True'
      SETTING_EXTERNAL_HOST: "zulip.yourdomain.com"
      SETTING_ALLOWED_HOSTS: "['zulip.yourdomain.com', '<IP_ADDRESS>']"
      SETTING_USE_X_FORWARDED_HOST: "True"


Google auth using deprecated API, will stop working on Mar, 23 2015

It looks like google is dropping support for OAuth 2.0. All "sign in with google" log-ins now display the warning message(see screenshot).
According to Google's documentation, they've started rolling out limitations and will ultimately shut down OAuth 2.0 on Apr 20. The majority of users will lose access after Mar 23 (unless a specific parameter is add ed to the request)
More details here:
https://developers.google.com/accounts/docs/OpenID
Please advise.

I'm considering using socialauth, and this really sounds like a big issue to me :(
Do you intend to do something about it? If yes do you have a rough idea when?

I haven't got any response from 3pillarlabs yet
I'm considering switching to Google's java library for authentication
https://developers.google.com/+/quickstart/java
We'll keep using socialauth for facebook SSO.

Same problem also for me, I am thinking the same @eyforia!
Actually this is a really big problem, if I have time I will try to fork this repo and implement the new method

This I think will completely stop working tomorrow.  Any plans to work on it here?  Anyone forking?

I've worked on this two projects in these days:
https://github.com/micheleorsi/gplus-verifytoken-java
https://github.com/micheleorsi/gplus-quickstart-java
I think the first biggest difference with the actual OpenId implementation is the fact that the button is smarter than a simple redirect.
That's why in my opinion is so difficult to integrate with socialauth. For the moment I created an other endpoint that handles this authentication

We have two providers related to Google. One is Google and other one is GooglePlus.
Google provider was no long in use as we told several times in the issues that If you register your application on google then you'll get the keys for Oaut2.0 which is supported by googleplus provider.
So use googleplus provider and no need to worry. You can also check our demo.
http://labs.3pillarglobal.com/socialauthdemo
In this demo, there are icons for Google and GooglePlus.
Google is not working which was on openid2.0 but googleplus is working fine as it supports OAuth 2.0.

I think the poster meant that OpenID 2.0 will be dropped instead of Oauth 2.0. Oauth 2.0 seems to be working fine, I have been using googleplus provider all the time, and it provides almost the same function as google provider.


KAFKA-2170: Fixes for Windows

This PR is to revive the older PR https://github.com/apache/kafka/pull/154 made by @mpoindexter (Mike Poindexter). I have replicated his work over the new code base.
This lets 6 LogTest unit tests to pass on Windows.

@soumyajit-sahu , does there exist any binary package/release for the same fix.
I am facing some issues building kafka on windows.. So if you can pass this fix binary packages. I will dload and see if your commit fixes the issue that I am facing at my end....
Abhi

The build log is already gone, any way to rerun and see what the failures were?
Maybe needs another merge from trunk?

The build fails with a timeout. Not sure how to get rid of that.
Build timed out (after 180 minutes). Marking the build as failed.
Build was aborted

Guys ???
Can i find the complete windows build for the same.. I am facing issues in compiling the build on windows
Abhi

Did any one has the binaries for the same release? I am facing issues in compiling the build
Abhi

Hi, I have created a new pull request. See #12331


"Interact with your bot" image should be a GIF...

...to show the result of running the bot, the flow of the prompts.

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

ID: 2c94a4c9-e988-3b42-a7db-3634d15e9c47
Version Independent ID: 68765289-1d77-6e4a-c6fb-1abbc067fcbd
Content: Manage conversation and user state - Bot Service
Content Source: articles/v4sdk/bot-builder-howto-v4-state.md
Service: bot-service
Product: bot-framework
GitHub Login: @ivorb
Microsoft Alias: v-ivorb

@cmayomsft - fixed the description and screenshot to show the actual running of the updated code. Live site should be updated shortly.


Bound with objects

The object in Plum which displays under the title "Remorse a tragedy in five acts" is a bound with (BW).  It displays in the PUDL under the title "Christabel, &c" with a note "With: Remorse : a tragedy in five acts. -- London : W. Pople, 1813."
Plum has associated the whole digital object with bib id: 2893853.  In actuality a section of the digital object belongs to bib id 2893853 and another section to bib id 3023901.

@escowles @jpstroop If I remember right, we decided we'd just have to manually go in and fix the sammelbands, yeah? That there wasn't enough information in the METS to automatically reason about what to do?

Yes, that's correct. We have no way of knowing where one stops and the next begins. There is a small number for which we might be able to rely on naming conventions (i.e. bibid_bibid.mets), but I don't think it's worth the effort.

OK, So can I try this out?  What would I need to do, or is something that needs to be done when the images are imported?

These are going to probably need to be reingested manually. Start a list here:

http://plum.princeton.edu/concern/scanned_resources/p5712n7160


update-github-pages-submodule job fails

https://prow.ppc64le-cloud.org/job-history/s3/prow-logs/logs/update-github-pages-submodule

this issue got fixed, its updating the repo properly now.


Disable entire month in vaadin date picker

I am trying to disable all dates of all months in vaadin-date-picker. While looking through the code,
I found that I can individually disable dates by setting disabled in   vaadin-month-calendar.html#L75 . In that case the disabled date is faded, which is due to vaadin-month-calendar-styles.html#L119, and one can not select the disabled date any more.
Similarly, I have done the following,

    <template is="dom-repeat" items="[[_days]]">

[[_getDate(item)]]

    </template>

And added the required style:
[part="days"][disabled] {
    color: var(--lumo-disabled-text-color);
}

In this case, all dates of all months are faded, but the user stiil can select them. Am I missing something? How to make disabled dates not selectable as well?

Currently <vaadin-date-picker> doesn't support disabling specific dates. Currently we only support the min and max attributes for specifying a lower and/or upper date limit for which range dates are allowed.
We have an open feature request #646 about adding this kind of functionality which you're apparently looking for. Please give that issue a thumbs up if this is what you want.
I'm not quite sure about what exactly you want though. The issue title says "Disable entire month in vaadin date picker" so do you want to disable a specific month (or multiple months)? Though you also said "disable all dates of all months" which sounds like you want to disable the whole date picker (or was this mistyped?). You can set the whole date picker as disabled or readonly but in that case you can't open the month calendar overlay (you would only see the currently set value in the text field).

Similarly, I have done the following

I'm not sure where you have put that code in. Do you have a clone of vaadin-date-picker where you edited vaadin-month-calendar.html? Are you trying to extend some of the internal components? Or are you trying to programmatically modify the DOM of a vaadin-date-picker instance after it has been created?
However it is currently possible to customize which dates are disabled by overriding some private API. Here's an example of how you could disable all Saturdays and Sundays: https://glitch.com/edit/#!/ablaze-carpet But be aware that this kind of usage is not supported (the private API methods may change in any new version which might break your implementation if you rely on those).


Add includes cabal declaration

This will make already cabal configure (not cabal build) fail if headers cannot be found:
cabal: Missing dependency on a foreign library:
* Missing (or bad) header file: openssl/asn1.h

I'm new maintainer of HsOpenSSL. Pull request was merged into https://github.com/vshabanov/HsOpenSSL and released on Hackage. In future please send pull requests to the new repo.


[Feature Request]: Versioning and Changelogs

Code of Conduct

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

Potential Enhancement
For better organization etc.
How it could be implemented
We should add a version code to our (pre-)releases and a CHANGELOG.md
Anything else?
No response

Created a CHANGELOG.md that just links to the current changelog on the wiki. Versioning as mentioned above will follow PEP 440 as there's really no reason to use anything else IMO when a) it's recommended and b) it's all that poetry supports...lol


Question on demo

Hi, first off all thank you for this neat project!
I just want to know if the existing checkpoints can be use to drive a different video? Or must i start training specifically for my video?
If no, what can i test with the demo, except for the examples showed on the github page?
I am trying to drive this image :

with this gif:

Thank you

It works for the videos of the same object class. E.g. If model is trained in faces it will work only on faces. If model if trained on cartoon animals it will only work on similar cartoon animals. So you can use different video but it should have the same class as checkpoint. To animate stickmans you will need to train on a dataset of stickmans.

@AliaksandrSiarohin Thank you for the quick response!


Dev

Description of the Change
Checklist

[ ] PR only contains one change (considered splitting up PR)
[ ] tests added
[ ] documentation updated (if necessary)
[ ] CHANGELOG.md updated (if necessary)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

15 of 26 (57.69%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 97.148%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

waldiez/runner.py
14
25
56.0%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.3%

Covered Lines:
2487

Relevant Lines:
2560

💛  - Coveralls


Failed to sync to disc when using zip vfs

I am trying to read from a zipped shapefile (only one layer) and write back out to another shapefile. Here is a simplified example:
src_filename = '/full/path/to/shapefile.zip'
dst_filename = '/full/path/to/output'
with fiona.open(src_filename,'r',vfs='zip://') as src_dataset:
    count = len(src_dataset)
    meta = src_dataset.meta
    with fiona.open(dst_filename,'w',**meta) as dst_dataset:
        for item in src_dataset:
            dst_dataset.write(item)

This gives a runtime error:
Traceback (most recent call last):
  File "/home/user/src/python_scripts/junk.py", line 11, in <module>
    dst_dataset.write(item)
  File "/home/user/anaconda2/lib/python2.7/site-packages/fiona/collection.py", line 413, in __exit__
    self.close()
  File "/home/user/anaconda2/lib/python2.7/site-packages/fiona/collection.py", line 397, in close
    self.flush()
  File "/home/user/anaconda2/lib/python2.7/site-packages/fiona/collection.py", line 387, in flush
    self.session.sync(self)
  File "fiona/ogrext.pyx", line 985, in fiona.ogrext.WritingSession.sync (fiona/ogrext.c:17250)
RuntimeError: Failed to sync to disk

However, it does look like the contents of the output shapefile are correct.
Python Version: Python 2.7.11 |Anaconda 2.4.1 (64-bit)
Fiona Version: 1.6.3

I modified filled in some sample data to try to replicate that.
First I zipped the sample dataset coutwildrnp.shp into archive.zip. I left the archive.zip in the same folder.
../Fiona/tests/data$  zip archive.zip coutwildrnp.*

I modified the code a little.  the current working directory is just above the Fiona directory:
import fiona
zip_src_filename = 'zip://./Fiona/tests/data/archive.zip'
dst_filename = './output.shp'
with fiona.open('/coutwildrnp.shp','r',vfs=zip_src_filename) as src_dataset:
    count = len(src_dataset)
    meta = src_dataset.meta
    with fiona.open(dst_filename,'w',**meta) as dst_dataset:
        for item in src_dataset:
            dst_dataset.write(item)

The output.shp looks like it matches.  No RuntimeError occurred.  I'll try this on Windows soon.
Ubuntu 14.04 / Python 2.7.6 & 3.4.3 / Fiona 1.6.3  / GDAL 1.11.2.

@micahcochran I created my archive in the same way and then modified my test code to follow yours. However, I still get the RuntimeError. I forgot to mention OS/GDAL version:
OS: Ubuntu 14.04
GDAL: 2.0.0

@mccarthyryanc Fiona 1.6 doesn't support GDAL 2.0. Support is coming in 1.7.

@sgillies, got it. I'll hold off on using GDAL 2.0 until the 1.7 release. Thanks!


[#419] Publish the documentation on hibernate.org

I've used this plugin: https://github.com/ajoberstar/gradle-git
Do we still want to publish on the JBoss doc server? For now I've removed it
I've tested it locally but we need to wait for the release to see if it actually works.
Better merge this after we merged the reactive website: https://github.com/hibernate/hibernate.org/pull/155

There something wrong with the javadocs on the generated website on staging.
I will check on Monday.

OK, publishing the html directly on hibernate.org doesn't work because Awestruct changes all the links

OK, publishing the html directly on hibernate.org doesn't work because Awestruct changes all the links

Did you consider pushing to a separate repository that awestruct is not aware of? You can just copy the data to _site after the Awestruct build, using a script.

I believe you can add directories to be ignored by Awestruct here: https://github.com/hibernate/hibernate.org/blob/staging/.awestruct_ignore

I think what that does is to completely ignore the folders during the generation. But I think this is what I was looking for: http://www.awestruct.org/extensions/indexifier/
In the avoiding some paths section

OK, I think now it is working. I've updated staging as well

This PR only make sense if the Reactive website is available


Parsing consecutive numbers/text with no space in between

Hi,
In C, something like the following can be done:
#include <stdio.h>

int main() {
  int a, b, c;
  char* line = "01021F";
  sscanf(line, "%2d%2d%2x", &a, &b, &c);
  printf("%d %d %d\n", a, b, c);
}

How can this be done with this library?
If this isn't yet available, let this be a feature request.
Thanks!

That's a good idea.  Currently you have to use the regex feature and it only works for decimal numbers:
let tmp = scan_fmt!( "010223", // input string
r"{/\d{2}/}{/\d{2}/}{/\d{2}/}",  // format
u8, u8, u8);
I'll try to work on adding length qualifiers to non-regex inputs (e.g., "{2d}{2d}{2x}")

The following now works in the latest release:
if let Ok((a,b,c)) = scan_fmt!( "01021F", "{2d}{2d}{2x}", u8, u8, [hex u8] ) {
    println!("{} {} {}",a,b,c);
}


lbry-app has forgotten verified humanness

The Issue
lbry-app has forgotten verified humanness
Steps to Reproduce

Become verified
Earn rewards for nearly 1 yr
app forgets verified status without user consent/LBRY inc. justification

Expected Behaviour
The lbry-app shouldn't forget, unless the user asks it to do so or the user has been notified why it has been removed.
Actual Behaviour
lbry-app has forgotten without consent/status has been removed without justification.
Suggested Solutions
System Configuration

LBRY Daemon version: 0.21.2
LBRY App version: 0.25.0
LBRY Installation ID: 9Z8TkpnE6ZQGXgqqCJTobLDXzFDCXJHYR8SvAN8cSfu9Ehbq1qShT4kA4dEWqiVPYT
Operating system: Linux (Linux-4.18.5-90.current-x86_64-with-Solus-3.9999-shannon)

Anything Else
Screenshots

Steven, I just re-enabled rewards on your account. (press ctrl-R in the app to refresh). Our system detected VPN/shared account use, which is one reason why we may disable rewards - please see https://lbry.io/faq/rewards.


Wrap feature issue

Maybe I'm doing something wrong, but the wrap options related to getx are not showing no matter on what widget I try it on.
I am using vscode 1.55.2, getx snippets 4.0.0 and get 4.1.4.

Test new version, avaible now

It works! But I think it wasn't working because the folder open inside vscode wasn't the root of the flutter application.
Nice response time btw!


Bloodlines Skill Reset to 10 after New BL purchase.

This was intended, but I think we can be more permissive here. I've made two changes:

Only 50% of bloodline skill is removed now, not the full 100%
A warning has been added to bloodline purchase page about the reduction


Improve glyph editor box visual separation

Signed-off-by: Jay Clark<EMAIL_ADDRESS>Fixes #30
Adds top & bottom margin and a subtle box shadow to enhance the visual separation of the glyph editor boxes.
APPEARANCE BEFORE CHANGES

APPEARANCE AFTER CHANGES

Thanks


Save by category and message, not ActiveRecord instance

Hello!
I found that this extension works incorrect when i try to use TsTranslation::save($category, $message), it always saves NULL in tsy_source_messages table.
After some research i found this line, i think that there must be $message, and not null.
Thanks!

Hi Kenny!
Thank you for advertency.
Such as private method _save() have been created for saving models, which instance of CActiveRecord, changes have made in method TsTranslation::save().
If first parameter of method is string (and second parameter also string) it saves as standard message.
You can view changes here .
Thanks!


Persist for "composition api" style

Describe the bug
Hello! Could you pls provide example how to add persist here?:
import { defineStore } from 'pinia'
import { computed, ref } from 'vue'

export const useDebugStore = defineStore('debug', () => {
  const _enabled = ref(false)

  const enabled = computed({
    get: () => _enabled.value,
    set: (value) => {
      _enabled.value = value
    }
  })

  return { enabled, _enabled }
})

PS. This plugin looks great and mature (more that 31 versions in past). Vue's composition api also is not something new. Hope i just missed it.
Reproduction
Example attached
System Info
System:
    OS: macOS 14.4.1
    CPU: (12) arm64 Apple M2 Max
    Memory: 31.72 GB / 96.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.19.1 - /opt/homebrew/opt/node@18/bin/node
    Yarn: 1.22.4 - ~/.yarn/bin/yarn
    npm: 10.2.4 - /opt/homebrew/opt/node@18/bin/npm
    bun: 1.0.13 - ~/Library/Application Support/reflex/bun/bin/bun
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Safari: 17.4.1

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible of the bug.

Hi you should be fine just using
import { defineStore } from 'pinia'
import { computed, ref } from 'vue'

export const useDebugStore = defineStore('debug', () => {
  const _enabled = ref(false)

  const enabled = computed({
    get: () => _enabled.value,
    set: (value) => {
      _enabled.value = value
    }
  })

  return { enabled, _enabled }
}, {
persist: true
})```

Keep in mind that only ref are translated to state, and the plugin uses subscriptions to mutations (only watching state) for persistence. That means the ref value has to change for a mutation to be triggered and state to be persisted


Send WALK_SPEC for ghost walking

Closes #349
This is disgusting and I'm ashamed. Please tell me how to do it better lmao

Closes #349
This is disgusting and I'm ashamed. Please tell me how to do it better lmao

Honestly probably not much different than I would have done it. You're kinda stuck doing it this way because CharacterAnimator is spaghetti. My only suggestion would be to add explicit methods for WalkGhost but that doesn't really make a ton of sense.


Add a test for not interpolating rendered section output

In contrast to lambda return values, the output of rendering a section should not be parsed.  I think a test case would be helpful here to draw a distinction with that behavior.  Test case included.
The attached test case would also clarify that nested sections should be evaluated "outside-in" rather than "inside-out."  For example--
data: { outer: true, inner: true, template: '{{planet}}', planet: 'Earth'}
template: '{{#outer}}{{planet}} : Interpolate? {{#inner}}{{template}}{{/inner}}{{/outer}}'

Very interesting test, quite related to issue #19.

Good catch on both this and #31; the latter is covered implicitly (read: accidentally) by the specs for dotted names, but both of these are clearly oversights in the spec at large.  I'll get these merged into both the v1.0.x and v1.1.x specs soon.
Thanks!

Thanks, @pvande and @groue.  groue, on the subject of related issues, I arrived at this test case in the process of thinking about issue #30 -- specifically the order in which lambdas should be evaluated that occur in nested sections (descending towards the leaves in the depth-first traversal or ascending up from them).
I may have one more test case to offer that is similarly related -- on the subject of lambdas.


Failed to start JDTLS : HTTP Error 404 Not Found | exit code -1

Just happened.

when I try to test serve:r

I try to clear JDTLS data and re install the LSP JDTLS package but no luck.

Fixed by https://github.com/sublimelsp/LSP-jdtls/commit/6d2b08dde03adf452e69fe5fa900a0e14de879ba.
Should land on Package Control in a few hours.

Thanks 👍


Send data form via redux-form and reducer and action creator

Hello, How send the value of fields of my redux form ? Indeed, I need to send the values of fields of my form in database with a reducer and actionCreator.
My form :
import React, {PropTypes} from 'react';

class myformClass extends React.Component {
 render(){
  return(
    <form>
      <Field
        name="inputName"
        ref="inputName"
        hintText="Le nom du flux"
        floatingLabelText="Nom du input"
        component={renderTextField}
        fullWidth={true}
      />
      <Field
        label="inputComment"
        name="inputComment"
        ref="inputComment"
        hintText="La description du input"
        floatingLabelText="Commentaire"
        component={renderTextareaField}
      />
    </form>
  )
 }
}

export default withRouter(reduxForm({
    form: 'myForm'
})(myformClass))

In my index app, I imported "reducer in redux-form":
import { reducer as reduxFormReducer } from 'redux-form'
I combined the reducers :
let allReducers = combineReducers({form: reduxFormReducer })
I created a action type :
export const FORM_ADD = 'FORM_ADD'
I created too a action creator :
export function addForm(dataForm) {
  return {
     type: types.FORM_ADD ,
     data: {
       INPUT_NAME: dataForm.inputName,
       INPUT_COMMENT: dataForm.inputComment
     }
  }
}

But Now, i don't know, how should I retrieve and send the value of my form with my action creator and the reducer. I know that I have to recover the value of my form and then send it.
For information, I was able to recover the value of my redux-form by doing this :
import React, {PropTypes} from 'react';

class myformClass extends React.Component {

  static contextTypes = {store: PropTypes.object}

  getValMyForm = () => {
    const { store } = this.context
    const state = store.getState()
    let valueFormFLow = getFormValues('myForm')(state)
    return valueFormFLow;
  }
}

export default withRouter(reduxForm({
    form: 'myForm'
})(myformClass))

I need Your HELP PLEASEEEEEE and I'm sorry for my english!!! :)

We have some examples on our site https://redux-form.com/7.0.4/examples/ and you can probably find good resources by searching a  little bit.
In case you still have issues, please migrate your question to StackOverflow.

Going through all the examples I can't find an example where Redux-Form is exported using React-Router's withRouter().


Added support for Self Trade Prevention Mode in SpotAccountTrade.cs

@aisling-2 Can someone review this pr please?

This is good update, please review!


reproduce worker termination race condition

There is a race condition between checking out of a worker from the pool and worker process's termination. In test case provided worker is terminated via gen_server:cast for more reliable reproducing, but it also occurs on gen_server:call.
Is it a know issue? Is there any recommendation of how to avoid it?

@e-shubin Can you explain what you expect the behavior to be?

@devinus I would like poolboy not to checkout a "dead" worker. In first assert I checkout a woker and execute an operation, which makes it to terminate. Then I try to do another operation, but terminated process is checked out. For me it seems to be a race condition.

It is impossible to guarantee a "dead" worker is not checked out because a worker could die after being checked out but before the caller has handled the reply with the worker from poolboy. If you want to try to ensure a "dead" worker is not checked out the worker must exit and not be checked in (or checked in after it exits in the case of a transaction).
Therefore the race condition is in the test, and not in poolboy. A cast is sent to the worker telling it to die but the client checks in in the worker without waiting for the worker to exit. This means the next client might get a worker that is about to die.

I've been thinking about this issue for the past few days and also agree with @fishcakez. You're telling the worker to die, and the pool as no idea the worker is dead yet because the cleanup facilities haven't been triggered in the pool yet. It's not until after the worker is checked out again that the worker is able to shutdown properly. @fishcakez Do you know of any reliable way to test that a worker is not being shutdown before checkout? I would handle this case by checking in the worker yourself after you know it's shutdown.

@devinus do you mean something like is_process_alive/1? I am unsure if it is worth doing that because the race condition would still exist.

@fishcakez No I was thinking of something like checking the processes state or something weird like that.

That does check its alive/exited state? Or do you mean like a sys call? I think it would only be possible to do this nicely if workers, rather than clients, checked the worker in. But that would greatly take away from poolboys simplicity.


Gravatar URLs are bad generated

Gogs version (or commit ref): latest
Git version: n/a
Operating system: n/a
Database: n/a
Can you reproduce the bug at https://try.gogs.io:

[X] Yes: any user profile, for example this
[ ] No
[ ] Not relevant

Description
The gravatar URLs are wrong, for example for this profile Gogs generate https://secure.gravatar.com/avatar/8afa4ade638e6debbc4725c4a7fc16cd?d=identicon?s=290 that have two ? instead of &:
- https://secure.gravatar.com/avatar/8afa4ade638e6debbc4725c4a7fc16cd?d=identicon?s=290
+ https://secure.gravatar.com/avatar/8afa4ade638e6debbc4725c4a7fc16cd?d=identicon&s=290

The first URL opens a web page with message Sorry, the parameters you provided were not valid, and the second one works well.

Oh... good catch!

Hmm.. I found ?d=identicon is not always there, this is a problem.

The order of the factor doesn't alter the product. You can ask for Gravatar using the form https://secure.gravatar.com/avatar/8afa4ade638e6debbc4725c4a7fc16cd?s=290&d=identicon, and in this case, the option d=identicon can be always with &.

The problem is both query parameters are conditional. I already have some clues to fix this.

And yet I was ignored. #5132. Btw my fix was disabling the gravatar service at the install page. Don't know how it worked

@twocow when you take a look at what you posted and what @ogarcia posted, you should know why.

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io.

It works! Thanks! :smile:


Add initial version

Codecov Report

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

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   96.02%
=========================================
  Files             ?        1
  Lines             ?      151
  Branches          ?        0
=========================================
  Hits              ?      145
  Misses            ?        6
  Partials          ?        0

Impacted Files
Coverage Δ

json_schema_to_class.py
96.02% <96.02%> (ø)

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 c88ed4d...bb26456. Read the comment docs.


Upgrade the spock version used in samples/snippets

Also upgrade the version used by the init plugin.
Also limit TestNG to < 7.6, as it requires JDK 11
Extracted from #20038

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


ci(github-actions): fix to use pull_request trigger

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Suspicious cpu usage

Hi,
I see some constant cpu usage by the container, around 10% with some higher picks.
I don't see any disk io or network traffic from the container..
Can you please tell what is causing this?

I have no idea. It's just a Django web server sitting there listening for HTTP requests.

@radinsky This gives me a pretty good idea of what I can try changing. Looks right in line with your observation.
https://superuser.com/questions/867067/djangos-dev-server-causes-permanent-high-cpu-load-with-python-3

@radinsky I put version 1.6.1 out there that should address your issue. please let me know. thank you.

Can't be installed via hass plugin, there's no repository.json anymore

installed with your last link and working ok. i don't notice cpu usage meanwhile..


EnvironmentLogger should create one file per day

example dataset that will be passed to extractor: https://terraref.ncsa.illinois.edu/clowder/datasets/5b88fc7e4f0c9925916fe44d
Update environment logger to concatenate the JSON files into one chronological file before converting to .nc.
If dataset has 23+ files, process. If it only has 23 and the output exists, don't overwrite. If there's 24 and there's an output, overwrite it.
https://github.com/terraref/extractors-environmental/blob/master/envlog2netcdf/terra_envlog2netcdf.py

When we run this for reprocessing, we're going to purge geostreams and start from scratch. Therefore it's harmless to run this on Kubernetes as a test case because any bad data it generates will be purged.

Another option is to concatenate the .nc files using something along the lines of
ncrcat 198[0123456789].nc 1980s.nc

which concatenates the files 1981.nc, 1982.nc, ... 1989.nc into a single file called 1980s.nc (documentation: http://nco.sourceforge.net/nco.html#ncrcat-netCDF-Record-Concatenator)

PR: https://github.com/terraref/extractors-environmental/pull/30

merged the PR.


H20 infer Qwen14B fp8 results in a divide-by-zero error.

System Info
Device: H20
Driver: 550.90.07
python env:
nvidia-cublas-cu12 <IP_ADDRESS>
nvidia-cuda-nvrtc-cu12 12.1.105
nvidia-cuda-runtime-cu12 12.1.105
nvidia-cudnn-cu12 <IP_ADDRESS>
tensorrt 10.0.1
tensorrt-cu12-bindings 10.0.1
tensorrt-cu12-libs 10.0.1
tensorrt-llm 0.12.0.dev2024071600
model
Qwen14B: https://huggingface.co/Qwen/Qwen-14B
expected behavior
Expect to print performance and eval data normally.
actual behavior

pip install nvidia-cublas-cu12==<IP_ADDRESS>


adopting inodes-monitor-plugin

repository url: https://github.com/jenkinsci/inodes-monitor-plugin
cc: @batmat
Description

Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When enabling automated releases (cd: true)

[ ] Add a link to the pull request, which enables continuous delivery for your plugin or component.
Follow the documentation to ensure, your pull request is set up properly.

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory and Jira at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

@janasrikanth thanks for your willingness to adopt the plugin.
You need to login to https://repo.jenkins-ci.org with the username and password that you use to login to https://issues.jenkins.io .  If you don't have a username and password for https://issues.jenkins.io , you can create one through https://accounts.jenkins.io

Hi @NotMyFault
I saw this plugin is not updated for a long time and is marked for adoption. So I would like to adopt it and maintain it from now onwards.
currently, I don't have any new features to add. I will work on it based on my thoughts and issues reported by users.

@NotMyFault since the plugin is up for adoption, I believe the pull request can be approved and merged even without involvement from previous maintainers.  I assume your CC of Baptiste was for his information, rather than because his approval is needed.  Is my assumption correct?

@NotMyFault since the plugin is up for adoption, I believe the pull request can be approved and merged even without involvement from previous maintainers. I assume your CC of Baptiste was for his information, rather than because his approval is needed. Is my assumption correct?

Absolutely, I noticed from contributing to core, that GitHub sometimes doesn't ping you when the mention is part of the PR body. Therefore, I mention the users again, to ensure they actually get an email, to raise awareness.

you when the mention is part of the PR body

It does afaik, but i don't think it does if you edit the mention in

Welcome aboard @janasrikanth!
Don't hesitate to join the chat rooms or post on the community forums if you have questions.

Interesting, I didn't even remember I had put this one up for adoption.
Anyway, I think this is cool. Thanks for stepping up @janasrikanth !


kleopatra: fix broken build

Description of changes

Kleopatra broke in #187453, this patch is also applied by Arch.
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, 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/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

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

4 packages marked as broken and skipped:

libsForQt512.kleopatra
libsForQt512.kmail
libsForQt514.kleopatra
libsForQt514.kmail

2 packages built:

libsForQt5.kleopatra (libsForQt515.kleopatra ,plasma5Packages.kleopatra)
libsForQt5.kmail (libsForQt515.kmail ,plasma5Packages.kmail)

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

4 packages marked as broken and skipped:

libsForQt512.kleopatra
libsForQt512.kmail
libsForQt514.kleopatra
libsForQt514.kmail

2 packages built:

libsForQt5.kleopatra (libsForQt515.kleopatra ,plasma5Packages.kleopatra)
libsForQt5.kmail (libsForQt515.kmail ,plasma5Packages.kmail)


qmllive: add new recipe

qmlLive is a live reloader environment for rapid UI development
Change-Id: Icc29a45ed488a95257625122107b83cbe746c755
Signed-off-by: Seokha Ko<EMAIL_ADDRESS>

@otavio
Thank you for your review.
Style issues were fixed based on guidelines.

@otavio @shr-project
Thank you for the review.
Issues were fixed so please check again.

@shr-project
Could you please review again?

All issues were addressed. Thank you @sukdo399 for this patch and for being responsive with the needed rework points.


Update Czechia holidays: add sk localization

Proposed change
Add Slovak translation of holidays in Czech Republic.
Type of change

[ ] New country/market holidays support (thank you!)
[x] Supported country/market holidays update (calendar discrepancy fix, localization)
[ ] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the [contributing guidelines][contributing-guidelines]
[ ] I've added references to all holidays information sources used in this PR
[ ] The code style looks good: make pre-commit command generates no changes
[ ] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.978%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
10276

Relevant Lines:
10276

💛  - Coveralls

And l10n test needs to be added for new language.

Good point. We normally add tests too.  @jose1711 if you're not familiar with that area you can offload it on me -- I'll submit it in a separate PR

@jose1711 if you're not familiar with that area you can offload it on me -- I'll submit it in a separate PR

Thank you, I'd appreciate if you take over.

Thank you, I'd appreciate if you take over.

You got it!


Send list of attendees to Red Hat security

Do you want I should do that or are you reminding yourself?
Chris Mathews
On Tue, Sep 18, 2018 at 12:52 PM Jason Hibbets<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1240, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOGtVlh0DByxU7Sffgn6XK3ejI2-nfnks5ucSS_gaJpZM4WuVzT
.

This is a reminder for me. I need to send the list of name to security in
advance of the event per our guidelines.


Add galaxy demo for Falcon7b

Ticket
N/A
Problem description
N/A
What's changed

Created Falcon7b galaxy demo (data parallel) with corresponding input data json and README
Disabled printing per token for multi-device runs of the demo
Corrected comment in conftest.py specifying the device_mesh grid for TG

Checklist

[x] Post commit CI passes
[x] Blackhole Post commit (if applicable)
[x] Model regression CI testing passes (if applicable)
[x] New/Existing tests provide coverage for changes

Post-commit model tests: https://github.com/tenstorrent/tt-metal/actions/runs/10289277133

Not putting this into CI?

@tt-rkim Added CI tests (verifying now, will not merge until passes) and also updated codeowners

@ttmchiou @tt-rkim I also changed the schedule of the TG demo pipeline to Mon/Wed/Fri (12am UTC) since once a week is insufficient for catching regressions. If that is not alright, please let me know.

Skipping TG tests on CI due to ND galaxy issues (#11258, #11254)

Force merged because @skhorasganiTT said this is urgent adn @ttmchiou on vacation


Hexxen 1733 Official Ulisses-Spiele Sheet

This will be the official Ulisses-Spiele Hexxen 1733 Character Sheet for Roll20.

Character Sheet Info Roll20 Internal Use only.

Hey @CodenameSupreme
Would it be possible for you to remove the translation files (other than translations.json) from this Pull Request?
These files are included from another service (CrowdIn), and can't be added directly to the github.
If you send me a separate link to the translation files, I can manually create them in that service.
Thanks!

Also, just to note, the hexxen.png preview file is missing from this pull request.