secret checker

Could something like gitleaks be useful as part of our default CI pipeline for checking for passwords being commited in git.
There are probably other tools but gitleaks looks a solid option at least.
https://github.com/zricethezav/gitleaks
https://github.com/zricethezav/gitleaks-action

Another option seems to be:
https://github.com/trufflesecurity/truffleHog
https://github.com/marketplace/actions/trufflehog-oss
There are probably a bunch more

Doesn't Horusec support this already?

Haven't checked, not impossible. But these are putposed built tools, so unless horusec uses them one of these are probably a better option

I think they have their own and integrates GitLeaks as well by default.


Patch

Added —url-prefix parameter to specify the prefix to use in the url path: i.e. http://localhost:8000/crud/user. Updated documentation

The —view-path parameter is now the same everywhere, while before it was —path for the main command and —view-path for controller and views

The model name should be singular  while the table is plural. Fixed controller stub and generator

Wrapped long lines for readability

Minor fixes to whitespace and spelling

Hi @vjandrea,
Many many thanks to bringing your ideas.
I'm really appreciated your all of your commits.
But I disagree with few things.
I've tried to merge your pull request except some commit but failed cause you made within one pull request. Even I've tried with "git cherry-pick" command I'm not so expert on git.
Sorry for this.
You can check the repository where I've made changes some things based on yours.
Sohel Amin

Thanks @sohelamin,
actually --url-prefix is really useful because the generated views may be used only for administration purposes, leaving the site root for presentation.
Please let me know the things you don't like so i can amend them and present a cleaner PR.

I agree with @vjandrea, see #35


Installation issues

Hello,
I am trying to install this package but I am having the following issues while running catkin_make

Any idea what it might be or how to fix it?
I am using ROS melodic and OpenCV 4.0 and I have found some compatibility issues with some scripts that I have already solved but I can't find a way to solve this one.
Thanks in advance!

Hi,
You may try to cast the getFeature2D_ORB_scoreType() to ORB::ScoreType here:
feature2D = new Feature2D(cv::ORB::create(
		getFeature2D_ORB_nFeatures(),
		getFeature2D_ORB_scaleFactor(),
		getFeature2D_ORB_nLevels(),
		getFeature2D_ORB_edgeThreshold(),
		getFeature2D_ORB_firstLevel(),
		getFeature2D_ORB_WTA_K(),
		(cv::ORB::ScoreType) getFeature2D_ORB_scoreType(),
		getFeature2D_ORB_patchSize(),
		getFeature2D_Fast_threshold()));

Note that I just updated the code to build with OpenCV4, see https://github.com/introlab/find-object/commit/413cb6820d772c93595b06b2d02c8d11dc2cf330
cheers,
Mathieu

Thanks, this solved the issue :)


Client SDK: Added verifyKeypair()

Exposes validKeypair() in client_sdk.ml to the SDK as verifyKeypair()
Added a new test for verifyKeypair()

!approved-for-mainnet


Use COALESCE function for thread identifiers

This is an exact replacement, but in the WHERE clause this saves around 75% of
the execution time, making post retrieval via RSM considerably faster.

Looks sensible and not DB-provider specific. Will leave this up to @lloydwatkin's judgement.

I'd appreciate a quick explanation, but as the tests are happy I'm happy to merge

COALESCE(col0, ..., colN) picks the first NOT NULL column value listed, so it's exactly equivalent to the CASE statement that I replaced (just shorter to write).
For the WHERE clause, though, this means that instead of testing both values for equality, it only picks the first NOT NULL as before (the same way we identify the thread) - this is simpler to optimize and reduces comparisons, it seems, though I was astonished by the performance improvement.


Fix XMPP parseNick function

The length of the split shall be higher than 1, if it equals 1 a crash will happen because of the following split which use the second value of the array created by the first split.

Good catch, thanks!


Introduce the interface to import store zip

Since we already have the API to export the repo bundle, this interface is for importing the store zip when migrated into cassandra.

I will merge master into this branch later to fix the compilation error.


[bugfix] remove "nodiscard" from project, restore support of C++ 11 standard

经反复排查，去除了代码中出现的[[nodiscard]]后，按照C++11标准编译，编译正常通过

thank you very much


refactor: User common test cases for generated SAC store tests - Exists

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #6895

PR #6992

PR #7056

PR #7057

PR #7058

PR #7059

PR #7060   👈

PR #6993

PR #6923

PR #6924

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

master

PR #6895

PR #6992

PR #7056

PR #7057

PR #7058

PR #7059

PR #7060   👈

PR #6993

PR #6923

PR #6924

This comment was auto-generated by Graphite.


Upgrade FS-UAE to v2.4.1

You've included a second merge commit in this pull. Can you please squash/remove it.

Thank you for the contribution. It needed some fixes, so they were made in the referenced pull request. Your contribution is still included (and still credited to you), with the appropriate modifications.
Changes:

Squashed commits.


React: enforce correct usage of refs

Potentially a use case for #12936

if you have a problem and you think exact types are the right solution,  please describe the original problem here

https://github.com/microsoft/TypeScript/issues/12936#issuecomment-284590083
Search Terms
react refs exact type
Suggestion
In React, it's fairly common to use refs. This involves creating an object of a specific type, and then passing that value as a prop to a component, which will mutate/reassign its current property.
Currently it seem there is no way to use refs in a type safe way. They can be accidentally passed to the wrong component, and there will be no type error.
https://stackoverflow.com/questions/56378639/typescript-react-enforce-correct-ref-props
import * as React from 'react';

class Modal extends React.Component<{}> {
    close = () => {};
}

declare const modal: Modal;

modal.close();

const modalRef = React.createRef<Modal>();

// Let's try giving this ref to the correct component…
// No error as expected :-)
<Modal ref={modalRef} />;

class SomeOtherComponent extends React.Component<{}> {}

// Let's try giving this ref to the wrong component…
// Expected type error but got none! :-(
<SomeOtherComponent ref={modalRef} />;

IIUC, this is because ref={modalRef} is equivalent to:
declare let ref: { current: {} };
declare let modalRef: { current: { close: () => void } };
ref = modalRef;

… which will correctly not error, because modalRef is a subtype of the target type ref.
However, when we're passing a ref to a component, we want to make sure the ref argument value (modalRef) is a supertype, or exact match, of the parameter type (ref).
This is because the component is responsible for mutating the ref's current property—if we pass in a ref of the wrong type, it will be mutated to something else. Later, when we try to use the ref's current value, the value will not match the type:
// Now when we try to use this ref, TypeScript tells us it's safe to do so.
// But it's not, because the ref has been incorrectly assigned to another component!
if (modalRef.current !== null) {
    modalRef.current.close(); // runtime error!
}

I am looking for a way to catch these mistakes at compile time, with TypeScript.
Note if the target component is a subtype of React.Component, we will get errors as desired:
class SomeOtherComponent extends React.Component<{}> {
    foo = () => {}
}

// Let's try giving this ref to the wrong component…
// We got an error :-)
<SomeOtherComponent ref={modalRef} />;

Checklist
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Seems like more of a variance annotations thing for generics.

This is a fundamental problem with a covariant-by-default type system - the implicit assumption is that writes through supertype aliases are rare, which is true except for the cases where it isn't. You don't really even need generics:
type Base = { a: number };
type Derived = { a: number, b: string };

function mutate(x: { derived: Base }) {
    x.derived = { a: 0 };
}

const holder: { derived: Derived } = { derived: { a: 0, b: "" } };
mutate(holder);
// Crash, sad
holder.derived.b.toLowerCase();

An exact type on mutate isn't even particularly what you want, since probably { derived: { a: 0 | 1 } } is an OK argument even though it's technically a subtype (thus disallowed?!) of the specified parameter type.
The primitive you actually need is something like contravariant T that takes a T and flips around its relations, so you could write
function mutate(x: contravariant { derived: Base }) {
    x.derived = { a: 0 };
}

// Error, can't assign "Base" to "Derived"
mutate(holder);


[FileUpload] Remove reflection

There is some reflection in how we handle FileUpload: https://github.com/apollographql/apollo-android/blob/50482e5c8143e9b98acd3a5847b9a18f5501894d/apollo-runtime/src/main/java/com/apollographql/apollo/internal/interceptor/ApolloServerInterceptor.kt#L239
We should remove it to save users proguard configuration and improve performance. Also this will not work for multiplatform.

We're running into an issue with this with R8 is there a temporary workaround until this is released?

The below should do it:
-keep class * extends com.apollographql.apollo.api.Input

Let me know if it worked, I'll add it to the docs.

Thanks for the quick reply!! Unfortunately, after trying this I'm still seeing the same error =\ If it helps at all the specific model we're using is actually the FileUpload model, although not sure if that matters

Can you share a stacktrace? I would expect only Input subclasses to be affected but maybe there something else.

Absolutely,
 {"errors":[{"message":"Variable \"$input\" got invalid value { body: \"\", uploadAttachments: [\"image/jpeg\"], scope: { id: \"05107586-0363-4f16-ac8d-7ff1b03e8090\", type: \"conversation\" }, id: \"f1fe00d8-06df-4188-9162-2b00c825325e\", d: [{}] }; Field \"d\" is not defined by type CreateTextInput. Did you mean id?","locations":[{"line":1,"column":29}],"extensions":{"code":"INTERNAL_SERVER_ERROR","exception":{"stacktrace":["GraphQLError: Variable \"$input\" got invalid value { body: \"\", uploadAttachments: [\"image/jpeg\"], scope: { id: \"05107586-0363-4f16-ac8d-7ff1b03e8090\", type: \"conversation\" }, id: \"f1fe00d8-06df-4188-9162-2b00c825325e\", d: [{}] }; Field \"d\" is not defined by type CreateTextInput. Did you mean id?","    at /ronin/node_modules/graphql/execution/values.js:116:15","    at coerceInputValueImpl (/ronin/node_modules/graphql/utilities/coerceInputValue.js:114:9)","    at coerceInputValueImpl (/ronin/node_modules/graphql/utilities/coerceInputValue.js:54:14)","    at coerceInputValue (/ronin/node_modules/graphql/utilities/coerceInputValue.js:37:10)","    at _loop (/ronin/node_modules/graphql/execution/values.js:109:69)","    at coerceVariableValues (/ronin/node_modules/graphql/execution/values.js:121:16)","    at getVariableValues (/ronin/node_modules/graphql/execution/values.js:48:19)","    at buildExecutionContext (/ronin/node_modules/graphql/execution/execute.js:184:61)","    at executeImpl (/ronin/node_modules/graphql/execution/execute.js:89:20)","    at Object.execute (/ronin/node_modules/graphql/execution/execute.js:64:35)","    at /ronin/node_modules/apollo-server-core/dist/requestPipeline.js:249:48","    at Generator.next (<anonymous>)","    at /ronin/node_modules/apollo-server-core/dist/requestPipeline.js:8:71","    at new Promise (<anonymous>)","    at __awaiter (/ronin/node_modules/apollo-server-core/dist/requestPipeline.js:4:12)","    at execute (/ronin/node_modules/apollo-server-core/dist/requestPipeline.js:228:20)","    at Object.<anonymous> (/ronin/node_modules/apollo-server-core/dist/requestPipeline.js:176:42)","    at Generator.next (<anonymous>)","    at fulfilled (/ronin/node_modules/apollo-server-core/dist/requestPipeline.js:5:58)","    at runMicrotasks (<anonymous>)","    at processTicksAndRejections (internal/process/task_queues.js:97:5)"]}}}],"extensions":{"tracing":{"version":1,"startTime":"2020-09-15T15:42:37.616Z","endTime":"2020-09-15T15:42:37.620Z","duration":3531874,"execution":{"resolvers":[]}}}}
Then relevant code that contains the inputs
CreateTextMutation(         CreateTextInput(             id = Input.optional(id),             body = Input.optional(messageBody),             uploadAttachments = Input.optional(attachments.map {               FileUpload(                   it.contentType, File(it.url).absolutePath               )             }),             scope = Input.optional(                 CreateScopeIdInput(                     id = scopeId, type = Input.optional(ScopeType.safeValueOf(entityType))                 )             )         )     )

My bad, I mixed Input and InputType. Can you try the following?:
-keep class * implements com.apollographql.apollo.api.InputType

Thank you so much!! The final exact line that fixed it for me was -keep class * implements com.apollographql.apollo.api.InputType { *; } Thanks again for all your help!

Thanks for investigating this! Glad to hear it worked. I added a mention to the doc until we can remove reflection altogether: https://github.com/apollographql/apollo-android/pull/2581/files

3.0.0-alpha01 uses FileUploadAwareJsonWriter to intercept uploads without relying on reflection and shouldn't require any proguard configuration.


[Feature] Add Swipe up to close

Please add swipe up to close like we have swipe down to close 🙏🏻

Hi, ok I can add this as an optional behavior.
For now I am adding it to my todo list but without priority, I have other things I would like to implement for now

Please add swipe up and down like twitter, telegram and etc


Fields in zendesk pipelines

Pipeline name
zendesk
Describe the bug
in Zendesk souce like  tickets,

there are values in the column that has prefix like  __dc.cc_--_cm_-_. This kind of prefix needs to remove. e.g. there is a value in column contact_reason : "__dc.cc_--_cm_-_i_am_unable_to_check_in__", it should look like Check in Management::I am unable to check in.

To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
I would like to remove the prefix
Stack traces and other evidence

Running environment

Cloud Composer
dlt Version - 0.2.6a1

Hey @Yuchangz
I'm looking into this now. Is this a custom dropdown field that you're dealing with?
The zendesk API returns normalized values for these fields instead of the display text. E.g. the "tag" value:

Just to confirm this is the same issue that's affecting you. I'll add a step to map the values to our custom field logic.

@steinitzu yeah it seems that we need to rename those tags. Yuchang is offline for some time. Let's move forward with your fix. thanks!

@Yuchangz this issue is fixed, the layout of state has changed so we need to load the data from scratch to a new dataset


SH_WORD_SPLIT needs setting for zsh with IFS

Heya - at least in my zsh (4.3.9 (i386-apple-darwin10.0) on Snow Leopard), it seems that you need to run "setopt SH_WORD_SPLIT" before zsh will consider the IFS setting.
Without this, I'd run 'gs; gd 1', and was just getting :
fatal: failed to stat '/Users/jon/Developer/web/app/gui/gui/views/audioboo/follow_button_section.rb|/Users/jon/Developer/web/config/environments/development.rb|/Users/jon/Developer/web/deploy/lib/sphinx.rb|[..............................]': File name too long

scm_breeze had just exported every filename concatenated with '|' into $e1.

Rather than messing with setopt environment, an alternative option would be to use ${=foo} - eg in git_status_shortcuts :
for file in ${=files}; do export $git_env_char$e="$file"; let e++; done

but I'm guessing that won't work in bash.
Any thoughts?

...   Oh.  It looks like you've already considered this, given the shwordsplit stuff at the top of scm_breeze.sh.  Let me investigate why that's not working for me.

The new ruby based implementation is missing the zsh_compat call that seems to be necessary.  See https://github.com/ndbroadbent/scm_breeze/pull/5.


Are you open to pull requests?

If so, would you accept a QuickHull algorithm for solving hulling problems?

Sure!


Adjust font style

font family
font weight
font size

Would be totally awesome!
Google (probably monospaced only) fonts option would be a killer feature!
I'd add line heght option to the suggestions list

@selrond added!
Also letter-spacing

@Kikobeats any way I can help this?

@selrond probably talking with jakedex yes, I don't know the current codebase 😄

any updates ?

No updates from the @dawnlabs team. We are not actively working on this feature, although it is on our radar given the support!
Anyone is welcome to help us out with this one! 😄

I've actually been sitting on some progress on a local branch here, just pushed it up here.


Introgression tutorial mentions ARGs

AT the moment it says ""To illustrate this, we simulate ancestral recombination graphs (ARGs) under a simple toy history of divergence and admixture", but we then don't run a simulation with "record_full_arg", but only with record_migrations=True (which I guess gives some ARG-like nodes, but not the recombination nodes).
The word ARG is mentioned a number of other times in the tut too. Should we perhaps replace this with "tree sequence"?

We should probably avoid the term all right, it's used in the vague "realisation of the coalescent with recombination" sense.

I'm editing that tut for typos at the moment. Shall I just replace it with "tree sequence" where appropriate, assuming that the reader, if they have got this far, knows what a tree sequence is?

(Or actually, simply the word "ancestry" will do at the start, I think)

SGTM - we don't have to be too precise I think.


Let's talk documentation

Discussed in https://github.com/WordPress/openverse/discussions/87

Originally posted by zackkrida October  5, 2021
We'll ultimately want to improve our code documentation in all repositories, and generate the handbook pages so that it's kept in sync with the repositories. This way we will have duplication, but also consistency and redundancy. We need to figure out a way to do this!

Consensus was something like Sphinx + reST. This was already done in the API in WordPress/openverse-api#427. We'll need to set up something similar for the following:

[ ] Ingestion server
[ ] Analytics server
[ ] Catalog

I'm thinking of ways to use the existing Sphinx setup from the API to also document the ingestion server and analytics. That will prevent the need to have 3 independent Sphinx setups in the same repo.

@dhruvkb If we do the monorepo migration after all, it seems we could close this issue as not-needed, right?

The monorepo will merge documentation for the API, ingestion server, automations and frontend but it still excludes the infra and catalog for now. We can close this and open specific issues (if needed) in those repos.


Specific Ccashing on mods

Describe your problem here.

Erro: Uncauch Error: Null Object Reference
to resume, when I get to a specific part of the song, it crashes, doesn't matter chaging options, reinstalling the mod, instaling via different like one on mediafire orther on google drive, jsut doesn't matter. Any guesses? I really appreciate it.
OBS: if want to know the mod is "Vs. Ourple Guy V3"
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

Is it by any chance in the song Cashmoney?
If so, I was having the same issue because I had my shaders turned off; I just enabled them, and I could play it just fine.
If not, could you tell what song is it, and what timestamp does it crash in? Maybe there's a specific event that's not being read correctly by the engine, or it just has a typo on its .lua script.

Is it by any chance in the song Cashmoney? If so, I was having the same issue because I had my shaders turned off; I just enabled them, and I could play it just fine. If not, could you tell what song is it, and what timestamp does it crash in? Maybe there's a specific event that's not being read correctly by the engine, or it just has a typo on its .lua script.

probably bad coding idk


Feature: SD Card support library

Expected: Firmware needs to support SD Card read/write functions in PSLab-v6 design
Actual: Current firmware doesn't have any functionality to support this feature
I would like to add required library files and I will refer this open source FatFS library which comes with a GNU GPL licence. Since I don't have the device with an SD card mounted, I would try out the code with existing PSLab device and an external SD Card Module connected with SPI protocol. Once that is done, @CloudyPadmal, could you please check it with the new prototype and provide feedback?

Hello @Dilu9218 , Yes this seems like a possible approach. Please send us code in a PR to review. I'll check with my prototype. Thanks.

Just to define things clearly, following is the pins assigned;

SPI Clock RC3
SPI Data In RA9
SPI Data Out RA4
Chip Select RB9
Switch RC5
Status LED RB15

Will address in #46


[Bug]: website is not responsive

Bug Description
as we reduce the screen size, the items are not allign according to that
Provide a detailed description of the bug
i can make your website responsive
Expected Behavior
website should be responsive in nature, so that mobile user should not face any difficultly
What browsers are you seeing the problem on?
Chrome, Firefox, Safari, Microsoft Edge
Put here any screenshots or videos (optional)

By submitting this issue, you agree to

[X] I agree to follow this project's Code of Conduct
[X] I’ve read all the issues in the issue section, and this has not been reported yet.

Other Information

[X] I'm a GSSoC '24 Extended contributor
[X] I'm a Hacktoberfest '24 contributor
[X] I am willing to work on this issue (optional)

@shriyadindi if this issue is open can i work on it . I will finish it today itself

@jjf2009 I didn't open this issue..

@Khizarshah01 can u assign me this issue . It is open for a while.
@shriyadindi sorry thought u are project admin


Update .gitattributes

See rust-lang/rust#57858

bors r+
We should remove this once we remove the last image file from the repo

Looks like the only image is editors/code/icon.png.

once we remove the last image file from the repo

Actually this image file got added quite recently.

For some reason, bors didn't like this.


Camera not starting

02-19 22:52:33.195 22979-22979/com.github.daemontus.renderer E/libc: Access denied finding property "persist.camera.cfa.packagelist"
error while running (oneplus 5t oreo)

Closed due to deprecation (#4)


RC1 incompatible

I'm using .NET Core 1..0.1 with .NET Framework 4.5.2 (because some dependencies have not been ported to .NET Core, namely the LDAP DirectoryServices in the framework itself).
When I try to use RC1 - it says that SwaggerGen and SwaggerUi are not compatible. .NET Framework 4.5.2 is a supported framework of .NET Core and hence it stands to reason that it would be supported by libraries as well?

RC1 isn't latest version  of "Swashbuckle" package - use beta902 instead.

That's just about as confusing as it can get... an RC that's older than a beta?

This package has used same versions as ASP.NET Core, but after release of the last one, owner has decided go back to beta versions, because it doesn't ready for production yet.


Bot automatic messages

Explanation
Sometimes the members of the server waits until someone say hello to say hello, so I think would be good if the bot automatically said hello, goodbye and enjoy lunch at specific hours of the day, so maybe people feels more confident about that
This issue is to make the bot write in greetings channel and say:

08AM COL: Good morning
12PM COL: Enjoy your lunch!
05PM COL: Have a nice afternoon, good job today!

Those messages would only be sent on weekday and not on weekend
Checklist

[ ] There are three messages at three different times:

[ ] 08 col:  "¡Buenos días a todos!"
[ ] 12 col: "¡Buen provecho!"
[ ] 17 col: ¡Feliz tarde a todos!, ¡Buen trabajo hoy!

[ ] Hours are configurable on .env
[ ] Message content are configurables on .env

This task is complete


feat: Update NTag UI to match mocks (#4556)

Reason for Change

The collections and datasets index NTag UI has been updated.

#4556

Changes

Modified the NTag to use czifui Tag component instead of blueprintjs and updated the style to match the mocks .e.g  padding, colour, font styles.

Testing steps

Go to collections index, the NTag UI should be updated.

Thanks, @tihuan for the review. @frano-m I just merged this to main.


Unhandled error event when using Figma connect

I'm trying to use the interactive setup to connect my design system to code connect. When asked for the URL of the Figma file with my design system library (step 3 in the guide), I get this error:
node:events:497
      throw er; // Unhandled 'error' event
      ^

Error: write EPIPE
    at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:94:16)
Emitted 'error' event on Socket instance at:
    at emitErrorNT (node:internal/streams/destroy:169:8)
    at emitErrorCloseNT (node:internal/streams/destroy:128:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -32,
  code: 'EPIPE',
  syscall: 'write'
}

Node.js v21.7.1

I tried running swift run -c release --verbose figma-swift, which produced the following error:
error: binary operator '??' cannot be applied to operands of type '(output: String, changes: [Formatter.Change])?' and 'String'
225 |
226 |     private func rewriteCodeBlockWithTemplate() -> String {
227 |         let formatted = (try? SwiftFormat.format(code.trimmedDescriptionRemovingReturnStatement())) ?? code.trimmedDescriptionRemovingReturnStatement()
    |                                                                                `- error: binary operator '??' cannot be applied to operands of type '(output: String, changes: [Formatter.Change])?' and 'String'
228 |         let tree = Parser.parse(source: formatted)
229 |         let rewriter = MappedPropertyRewriter(

path/to/project/.build/checkouts/code-connect/swiftui/lib/CodeConnectTemplateWriter.swift:249:35: error: cannot infer contextual base in reference to member 'whitespacesAndNewlines'
247 |         ).replaceNestedInstancePlaceholders(
248 |             nestedInstanceCalls: rewriter.nestedInstanceCalls
249 |         ).trimmingCharacters(in: .whitespacesAndNewlines)
    |                                   `- error: cannot infer contextual base in reference to member 'whitespacesAndNewlines'
250 |
251 |         return "export default figma.swift`\(rewrittenCode)`"

Code Connect CLI version: 1.1.2
Operating system: OSX

figma.config for my SwiftUI project:
{
  "codeConnect": {
    "swiftPackagePath": "Package.swift",
    "include": ["Sources/**/**/*"],
    "exclude": ["markdown/**", "danger/**", "scripts/**", "SnapshotTests/**"]
  }
}

My Package.swift setup:
import PackageDescription

let package = Package(
    name: "ProjectName",
    defaultLocalization: "en",
    platforms: [
        .macOS(.v14),
        .iOS(.v16)
    ],
    products: [
        .library(name: "ProjectName", targets: ["ProjectTarget"])
    ],
    dependencies: [
        ...
        .package(url: "https://github.com/figma/code-connect", from: "1.0.0"),
    ],
    targets: [
        .target(name: "ProjectTarget",
            dependencies: [
                ...
                .product(name: "Figma", package: "code-connect")
            ],
            path: "Path/To/Project"
        )
    ]
)

Hi @thomtreebus, thanks for the report! We've identified the issue and working on a fix.
In the meantime if you can force the version for SwiftFormat of 0.53.5 in your project that should fix the issue in the meantime. Sorry for the inconvenience.


Course Level Number Attribute

To handle Requirements such as "upper-level" electives, it would be very helpful to have course level number (1000, 2000, 3000, 4000, etc) as an attribute of a course.
Currently there isn't a clean solution for this, especially since we store course number as a String due to variable hour courses.

[ ] Create script (aggregation pipeline) to add a course level number attribute
[ ] Update Coursebook Scraper accordingly

I also would prefer renaming the existing class_level attribute to avoid confusion.

No longer relevant, will open a new issue if this needs to change in the future


Add user history support for auth and mgmt apis

Description
Related to: https://github.com/descope/etc/issues/4713
Add user history support for auth and mgmt apis.
Must

[x] Tests

Also, what about readme ?

We don't have for Me so not sure if we need also For History. Since they are on the same place.
But can be added if needed.

Added readme.


Switch to RTD sphinx theme

Live preview: https://ixmp.readthedocs.io/en/feature-rtd/
PR checklist:

[x] Tests added
[x] Documentation added
[x] Description in RELEASE_NOTES.md added

@khaeru is this good to be reviewed/merged?

No, I'll need to update the theme to match the one we chose for MESSAGEix.
Will do that today, and then request a review.

@khaeru what's the link for the branch's rendered docs?

looks great! Thanks @khaeru


elements is not defined

Trying to dynamically change a form based on selected options in a form. So if a user selects an option it can hide form elements. Using this code in "element value changed" section

/**
 * Find Search and AssetName elements
 */
const searchElement = elements.find((element) => element.id === "Search");
const assetNameElement = elements.find((element) => element.id === "AssetName");

if (searchElement && assetNameElement) {
  // If the Search's value is 'single', unhide the AssetName element
  // If it's 'multi', hide the AssetName element
  assetNameElement.hidden = searchElement.value !== 'single';
}

/**
 * Update Panel Options
 */
onOptionsChange({ ...options, elements });

generate this error
reportUnhandledError.js:10  Uncaught ReferenceError: elements is not defined
at eval (eval at  (module.js:1:1), :7:23)
at eval (FormPanel.tsx:759:7)
at Object.eval [as next] (FormPanel.tsx:812:7)
at L.next (Subscriber.js:96:33)
at W._next (Subscriber.js:63:26)
at W.next (Subscriber.js:34:18)
at l (EventBus.ts:38:18)
at l.emit (index.js:181:35)
at s.publish (EventBus.ts:28:18)
at eval (useFormElements.ts:78:26)
I am using elements which i thought was defined by the plugin already inside update section and it works fine

Realised i had to use context.panel.elements in "element value changed" whereas just need to do element in the other areas


Feature request: support adding new categories

Images without categories are almost completely worthless to the community. New categories are sometimes needed when I upload lean-to shelters because I want to have a subcategory of e.g. "Lean-tos in Sweden" that does not yet exist.
I suggest adding a view that enables me to search for and create missing categories. It does not have to be in the upload-flow, but that is perhaps preferred.

Note: Creating a category is an expert thing to do, and requires seeing very well what are the currently existing categories. Our category search/exploration is quite buggy at the moment, it must be fixed first.

OK, I understand. I should probably create the ones I need with a bot or something. There is 290 categories like "Lean-to in Härnösand Municipality" missing and ~20 missing for the Administrative areas also.

@egils-consulting I see! I encourage you to use depictions too, the great thing is that they are perpendicular. For instance, for a picture of a lean-to in Härnösand Municipality, you would select two depictions: "lean-to" and "Härnösand Municipality" (that's all, no need to create a new depiction item).

Thanks for the suggestion. Can I add depictions in the app?

Depictions are actually Wikidata items, so your feature request #3935 covers that :-)
It is quite rare to have to create a Wikidata item. For instance "lean-to" and "Härnösand Municipality" Wikidata items already exist, so no need to create one. The only situation would be if you encounter a notable place that does not have a Wikidata item yet, for instance a small school or a village townhall or a minor historical monument. To see whether a Wikidata item already exists, you can use the Nearby feature (uncheck "Needs Photo") or https://wikishootme.toolforge.org.

Ideally I as a an uploader would like to select a bunch of images and add them to a category
On the screen to choose which category I would like an option to create a new one.

Regarding depictions I do not agree that a shelter depicts the municipality.  Interesting idea! Our UI to assign a category to a single image is very new and still full of bugs, so the first step would be to polish it, though :-)

"Can a shelter depict a municipality?"

Great question! We definitely want to express (using structured data) the fact that the shelter is in the municipality. And the only structured data attribute we have implemented so far in the app is depicts. Would you know any better attribute that we could use? Maybe object location ? Ideally an attribute that has already gained consensus (and widespread usage) in the SDoC community, and is easily searchable by the general public or will be soon (people searching for pictures of the municipality should be shown the shelter as one of the results).
The current implementation (using depicts) seems to be pretty mainstream (example: Paris is depicted by many many many of its aspects) and saying that a picture of something in Paris is a picture of Paris is not untrue in a way, but I would personally much prefer using a different attribute such as "object location", as long as search is smart enough to show all relevant results.
Cheers!

I just realized that "object location" is meant to contain coordinates (latitude, longitude), not the Wikidata QID of an item such as a municipality 🤔
Would you mind asking at https://commons.wikimedia.org/wiki/Commons_talk:Structured_data what is the best way to model the relationship between the picture and the municipality? Thanks!

done https://commons.wikimedia.org/wiki/Commons_talk:Structured_data#Modeling_picture_taken_in_a_certain_municipality

Hi. I'm a long-time OpenStreetMap contributor, and have recently discovered contributing photos to Wikimedia Commons, which are then displayed in applications like OsmAnd.
The ability to create new categories from the Commons mobile application would significantly ease my upload workflow.

@contrapunctus-1 Thanks for your valuable feedback!  Would you mind describing in details what would be your ideal workflow?
By the way, I am not sure how to show Commons categories in OsmAnd, do you mean the "Online Photos" here?

I upload images by sharing them from other applications (e.g. Simple Gallery) to Commons.
Ideally, I would create new categories within the existing Depictions - Categories wizard. Alternatively, I could add new categories within the app in a separate activity. In both cases, I would ideally do everything from the application without having to switch to the website.
Correct, I do mean the Online Photos as shown in your screenshot.


Create a post as an organization: Question 2/3 layout

Env: https://staging.fightpandemics.work/
Steps to recreate:

Land on https://staging.fightpandemics.work/
Offer help as an organization
Select post as an organization
Observe the next screen

Current behaviour:

[ ]  Heading 'Question 1/3' is not centre aligned as in the mocks
[ ]  The location bar is not as per the design in the mocks
[ ]  'Show me posting anywhere' is wrapped. Not as per the design in the mocks
[x]  The header options such as 'Feed', 'COVID-info', 'Login', and 'Register' are missing
[x]  The page background is not grey as seen in the mocks.

Desired behaviour:

The same screen is required for Need Help question 2/3 screen.

related to these issues:
#471,
#473

@tiagocostasilva and me had worked to get this sorted. Different tickets were raised to have the desired UI and behaviour.


Postman collections generated from Swagger specs (2021-01-26_12-48)

Description
Removes several unused APIs from the collections, style updates to the XDM collection.
Related Issue
https://jira.corp.adobe.com/browse/PLAT-85793

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thank you Peter.  Looks good.

Thank you Peter.  Looks good.


Directory per pane

Hi, I am running a window where four panes, each with different directory.
I do something like this:
windows:
  - name: Server
    focus: true
    root: ~/dev
    layout: tiled
    panes:
      - cd app1 && be rails s
      - cd app2 && be rails s -p 3001
      - cd app1 && be sidekiq
      - cd app2 && be sidekiq

What happens if I kill the server in app1 is when I press arrow up, it suggests cd app1 && be rails s which I cannot run.
I'd like to switch directory before running my server somehow, any ideas?
I could do cd ~/dev/app1 && be rails s which will allow re-running, but if there is more elegant solution I'd love to know.

Hi Michal,
Unfortunately the cd ~/dev/app1 && be rails s solution is the only one that will work for the moment.
There’s the commands key you can use for each pane, like so:
windows:
  - name: Server
    focus: true
    root: ~/dev
    layout: tiled
    panes:
      - commands:
        - cd app1
        - be rails s
      - commands:
        - cd app2
        - be rails s -p 3001
      - commands:
        - cd app1
        - be sidekiq
      - commands:
        - cd app2
        - be sidekiq

But what this will do is join commands with ; instead of && — which is not what you want here.
We could change (or add a new option) this line to join commands with “sending the Enter key” instead of ;.
What do you think?


v0.9.1: Syntax: pyaspeller connection error not caught

The pyaspeller makes a call out to a remote service. If there is no Internet connection, it will through an exception which is not caught.

Obsolete - pyaspeller removed.


why do you use envoy

Hi, i am reading your project and i love it, can i ask u a question, why do you use Envoy in your project. Can you tell me the purpose, the pros and cons of this. thank u so much

No particular reason, I just thought it sounded cool.


Fix #3950: Div by 0 during DXT4 DDS load

Description
Fix the div-by-0 when handling unassociated alpha during a DXT4 DDS load.  The code mirrors what other formats do by first checking to see if the alpha value is not 0 before continuing.
Tests
Added the 218-byte file from the bug to the local test suite.
Checklist:

[x] I have read the contribution guidelines.
[x] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] If I added or modified a C++ API call, I have also amended the
corresponding Python bindings (and if altering ImageBufAlgo functions, also
exposed the new functionality as oiiotool options).
[x] My code follows the prevailing code style of this project. If I haven't
already run clang-format before submitting, I definitely will look at the CI
test that runs clang-format and fix anything that it highlights as being
nonconforming.

There's a secondary failure after the initial one hit in the report. I'll need a bit more time to test out the fix.
The failure is that the file declares itself as a DXT4, which has alpha (4 channel), but it also has its "normal map" flag set. This confuses the code inside GetChannelCount as "normal map" takes precedence and returns 3 for the count instead of 4. This causes OIIO to allocate fewer bytes than required and ASAN rightfully flags the problem when the code eventually writes past the buffer.

Is this ready to review, @jessey-git ?

Yes, I believe so.

Fixes #3950


Reordered operations to check for makecert.exe only when needed

Hi @angrifel, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Looks good to me, thanks!


A few chapter additions to ops_contrib

Hey,
As I prepare to setup VersionEye myself I though these additional two chapters can avoid a bit of load on<EMAIL_ADDRESS>and make folks more autonomous in going ahead with the setup.
I'm sure there's edits that need to be made below, but I though to start up with something to build upon. Let me know what changes need to be done and I'll make them asap.
:octocat::heart:

Hey @bitoiu. Many Thanks for your help. That is awesome 👍


improve php-fpm conf

Issue: rgsystemes/kb#8682

sounds good, but it misses the php fpm reload at the end of the api deployement phase. also the preload should also be applied on dashboard project no?


feat: add organization, memberships and invitations (resolves #5)

This PR adds an Organization model which users can join via a Membership model, and which users can be invited to join via an Invitation model. It also adds feature test stubs for users, two-factor authentication, and organizations, memberships and invitations.
Testing

Install this branch with Composer.
Run the Hearth installer using the --organizations flag: php artisan hearth:install --organizations
Log in and navigate to /en/organizations/create.
Verify that you can create an organization.
Verify that you can edit your organization.
Verify that you can invite a new user to the organization.
Verify that you can register and log in as the new user and then accept the invitation.
Verify that if an organization has only one administrator, they cannot be removed from the organization.
Verify that non-administrative members of an organization cannot:

remove other users from the organization;
invite new users to the organization;
delete the organization.

Verify that administrators of an organization can delete the organization.

Additional Information

Blocked by #22.

6. Verify that you can invite a new user to the organization.
7. Verify that you can register and log in as the new user and then accept the invitation.

For security / verification purposes, there should be a check that the email of the signed-in user clicking the invite link is the same as the one associated with the invited user. I was able to do the following at Steps 6/7:

Send an organization invitation to<EMAIL_ADDRESS>(previously created account)
Create an account for<EMAIL_ADDRESS>While logged in as badactor, clicked the "accept invitation" invite link; this resulted in goodactor being added to the organization, but goodactor was not the logged in user.

For security / verification purposes, there should be a check that the email of the signed-in user clicking the invite link is the same as the one associated with the invited user.

@waharnum Thanks for catching this, I'm working on it!

@waharnum Only a user logged in with a matching email should be able to accept an invitation now. Can you verify this on your end?


Go through and remove unused imports

Throughout the project, there are numerous unused imports. They should be removed.

hello there, i can work on this, would be great if you help me to specify some of the unused imports.

Ah thank you for offering! I did them myself just before seeing your comment.


Fix emoji sliders

Hi, I saw the issue reported about the emoji slider and wanted to contribute to the project. Thank you for the great job you have done building this library.

Thanks Bryan!


chore(main): release 1.7.2

:robot: I have created a release beep boop
1.7.2 (2024-12-18)
Bug Fixes

forward className to the underneath Antd component (#31) (6028824)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.7.2
:sunflower:


Text and fix when appending categoricals with fastparquet

[ ] Tests added / passed
[ ] Passes flake8 dask

@TomAugspurger , test now passes which fails on master - the same code as proposed in the issue.


Request: option to hide days in the past

Since recently, days in the past containing a multi-day event that has not been ended yet are listed as well. As a user I simply want to see what I have planned today and in the near future with as little scrolling as possible, and listing days in the past gets in the way of that. My suggestion is to provide the option to simply never show any days before the current.

@volucris It's interesting why do you still have that multi-day events, if you don't want to see them even today?!
I mean you could simply change that events to e.g. one-day events and write information about their real duration in their Notes.
BTW, currently Calendar-widget allows you to see this event in a list of past events, if you want  (if that event has Default calendar color).

I'm afraid I didn't manage to make myself clear there. I do want to see the multi day events, repeated on every day from today, but I don't need to see the past. Let's say I'm on day 2 of a 3 day meeting. Now it says yesterday: meeting. Today: meeting. Tomorrow: meeting. I want to see the last two, but not the first. I know what I did yesterday, and even if I didn't it's too late now anyway. Turning it yellow, blue or pink does not change that.
I hope this clarifies my issue, sorry for the confusion.

Oh that bug / feature is already fixed, and may test it now building the application yourself from a Source code of this repository.
So  this issue duplicates that: https://github.com/plusonelabs/calendar-widget/issues/184


Update Readme to add known limitations and conform developer notes to contributing guide

I noted a few inconistencies that needed addressing the existing README so did a quick PR at https://github.com/ably/ably-ruby/pull/239.  I hope that helps.

This pull request is at risk of becoming a never ending story. I'm going to land it in its current state as it is already a marked improvement. We'll address @mattheworiordan's comments under the issue I've just created, #113.

We'll address @mattheworiordan's comments under the issue I've just created, #113.

Ok, who is picking that issue up @QuintinWillison?


Add timer reset command

Adds reset command for timers.
Like the stop command, it can be provided a position, otherwise it will use the current active timer.
When the timer is running, it resets it back to 0 and it will continue running from there.
When the timer is stopped, it clears the timer entirely.

Hopefully I haven't deviated from the existing code style too much.

Resolves #28

Awesome 🎉

Hopefully I haven't deviated from the existing code style too much.

Haha don't worry, its still on my todo list to split up the commands file to make it a bit less "cluttered"
Thank you!!
@all-contributors please add @RascalTwo for code


PolySynth(DuoSynth) issues

Hi,
I've come upon some issues using the DuoSynth inside the the PolySynth, i´ve tried to look this up in the forum and through older issues but I haven't quite found a solution.
Since I am, admittedly, a beginner this might be a case of incompetence rather than actual problems but here it goes:

using the keyboard to play the polysynth, some notes will become stuck, I think this has something to do with the fact that when a key is pressed it repeats the value and tried to address it,  but it is still happening usually when the note count is high (but still not exceding maxPolyphony).

I am also unable to set the frequency value for the filter for voice0 and voice1, this seems to be the only value I can't access with the set() method.

The code is a bit of a mess, but here it is:
https://jsfiddle.net/9075fpbq/

hi @CptCaveira

using the keyboard to play the polysynth, some notes will become stuck, I think this has something to do with the fact that when a key is pressed it repeats the value and tried to address it, but it is still happening usually when the note count is high (but still not exceding maxPolyphony).

Unfortunately, I'm unable to recreate this. I just played around for 5 minutes smashing the keys on my keyboard, but couldn't get a note to stay stuck.

I am also unable to set the frequency value for the filter for voice0 and voice1, this seems to be the only value I can't access with the set() method.

This is actually by design. The filter frequency is controlled by the filterEnvelope which means that it's value is dynamic and can't be set explicitly. You should be able to adjust how the filterEnvelope controls the frequency using the baseFrequency and octaves properties.

hi @CptCaveira

using the keyboard to play the polysynth, some notes will become stuck, I think this has something to do with the fact that when a key is pressed it repeats the value and tried to address it, but it is still happening usually when the note count is high (but still not exceding maxPolyphony).

Unfortunately, I'm unable to recreate this. I just played around for 5 minutes smashing the keys on my keyboard, but couldn't get a note to stay stuck.

I am also unable to set the frequency value for the filter for voice0 and voice1, this seems to be the only value I can't access with the set() method.

This is actually by design. The filter frequency is controlled by the filterEnvelope which means that it's value is dynamic and can't be set explicitly. You should be able to adjust how the filterEnvelope controls the frequency using the baseFrequency and octaves properties.

Thanks for the reply!
The frequency thing is a weight off my mind, I was really going nuts with it.
As for the sticking, here is an example of what i mean:
https://user-images.githubusercontent.com/71012028/104046490-786f0d00-51e0-11eb-8cee-de4724c8fb3e.mp4
starting at around the 12s mark is my issue.
If you can think of a way that I can give you some more information on it please let me know.

Thanks for the reply!
The frequency thing is a weight off my mind, I was really going nuts with it.
As for the sticking, here is an example of what i mean:
https://user-images.githubusercontent.com/71012028/104046490-786f0d00-51e0-11eb-8cee-de4724c8fb3e.mp4
starting at around the 12s mark is my issue.
If you can think of a way that I can give you some more information on it please let me know.

I have a feeling this is likely because internally everything in Tone.js is scheduled with a small lookAhead and that somehow the release is being triggered before the attack is invoked. Not quite sure why this would be unless the events somehow came out of order.
As i said i'm unable to reproduce this behavior, but see if something like this might bandaid things:

poly.triggerAttack(notes[QWERTZ.indexOf(event.key)], Tone.immediate());

...
poly.triggerRelease(notes[QWERTZ.indexOf(event.key)], Tone.immediate());

I have a feeling this is likely because internally everything in Tone.js is scheduled with a small lookAhead and that somehow the release is being triggered before the attack is invoked. Not quite sure why this would be unless the events somehow came out of order.
As i said i'm unable to reproduce this behavior, but see if something like this might bandaid things:

poly.triggerAttack(notes[QWERTZ.indexOf(event.key)], Tone.immediate());

...
poly.triggerRelease(notes[QWERTZ.indexOf(event.key)], Tone.immediate());

Hey,
so I tried it out and some notes will still get stuck, and play continuously like previously, but now if the same key is hit again the release is triggered and the page doesn't need to be reloaded. It helps out a lot!
Thanks for your time!

Hey,
so I tried it out and some notes will still get stuck, and play continuously like previously, but now if the same key is hit again the release is triggered and the page doesn't need to be reloaded. It helps out a lot!
Thanks for your time!

If it's working for you now, i'm going to close this issue.
If you're still having this issue, and you're able to create a with a test case which can be produced without mashing the keys (since there's a lot of variability there), that'd be really helpful in debugging where the issue is and we can reopen it.

If it's working for you now, i'm going to close this issue.
If you're still having this issue, and you're able to create a with a test case which can be produced without mashing the keys (since there's a lot of variability there), that'd be really helpful in debugging where the issue is and we can reopen it.


编译出错，请各位大牛指导啊！

I'm submitting a ...  (check one with "x")
[ ] bug report
[x ] help wanted
[ ] feature request

Current behavior
编译不成功
Expected/desired behavior
编译成功
Reproduction of the problem
Run ./build.sh  显示如下信息：
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found OpenCV: /usr/local (found suitable version "3.2.0", minimum required is "3.2.0")
-- Found OpenCV: /usr/local (found suitable version "3.2.0", minimum required is "3.1.0")
-- Configuring done
-- Generating done
-- Build files have been written to: /home/zhijun/EasyPR-master/build
Scanning dependencies of target thirdparty
Scanning dependencies of target easypr
[  3%] Building CXX object thirdparty/CMakeFiles/thirdparty.dir/xmlParser/xmlParser.cpp.o
[  7%] Building CXX object thirdparty/CMakeFiles/thirdparty.dir/LBP/helper.cpp.o
[ 11%] Building CXX object thirdparty/CMakeFiles/thirdparty.dir/mser/mser2.cpp.o
[ 14%] Building CXX object thirdparty/CMakeFiles/thirdparty.dir/LBP/lbp.cpp.o
[ 18%] Building CXX object thirdparty/CMakeFiles/thirdparty.dir/textDetect/erfilter.cpp.o
[ 22%] Building CXX object CMakeFiles/easypr.dir/src/core/core_func.cpp.o
[ 25%] Building CXX object CMakeFiles/easypr.dir/src/core/chars_recognise.cpp.o
[ 29%] Building CXX object CMakeFiles/easypr.dir/src/core/chars_identify.cpp.o
[ 33%] Building CXX object CMakeFiles/easypr.dir/src/core/chars_segment.cpp.o
[ 37%] Building CXX object CMakeFiles/easypr.dir/src/core/feature.cpp.o
[ 40%] Building CXX object CMakeFiles/easypr.dir/src/core/plate_detect.cpp.o
In file included from /home/zhijun/EasyPR-master/include/easypr/core/plate.hpp:15:0,
from /home/zhijun/EasyPR-master/include/easypr/core/core_func.h:5,
from /home/zhijun/EasyPR-master/src/core/chars_identify.cpp:3:
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp: In constructor ‘easypr::CharsIdentify::CharsIdentify()’:
/home/zhijun/EasyPR-master/include/easypr/config.h:143:42: error: expected primary-expression before ‘>’ token
model = ml::ANN_MLP::loadml::ANN_MLP(path);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp:22:3: note: in expansion of macro ‘LOAD_ANN_MODEL’
LOAD_ANN_MODEL(ann_, kDefaultAnnPath);
^
/home/zhijun/EasyPR-master/include/easypr/config.h:143:42: error: expected primary-expression before ‘>’ token
model = ml::ANN_MLP::loadml::ANN_MLP(path);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp:23:3: note: in expansion of macro ‘LOAD_ANN_MODEL’
LOAD_ANN_MODEL(annChinese_, kChineseAnnPath);
^
/home/zhijun/EasyPR-master/include/easypr/config.h:143:42: error: expected primary-expression before ‘>’ token
model = ml::ANN_MLP::loadml::ANN_MLP(path);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp:24:3: note: in expansion of macro ‘LOAD_ANN_MODEL’
LOAD_ANN_MODEL(annGray_, kGrayAnnPath);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp: In member function ‘void easypr::CharsIdentify::LoadModel(std::_cxx11::string)’:
/home/zhijun/EasyPR-master/include/easypr/config.h:143:42: error: expected primary-expression before ‘>’ token
model = ml::ANN_MLP::loadml::ANN_MLP(path);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp:36:5: note: in expansion of macro ‘LOAD_ANN_MODEL’
LOAD_ANN_MODEL(ann, path);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp: In member function ‘void easypr::CharsIdentify::LoadChineseModel(std::_cxx11::string)’:
/home/zhijun/EasyPR-master/include/easypr/config.h:143:42: error: expected primary-expression before ‘>’ token
model = ml::ANN_MLP::loadml::ANN_MLP(path);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp:44:5: note: in expansion of macro ‘LOAD_ANN_MODEL’
LOAD_ANN_MODEL(annChinese, path);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp: In member function ‘void easypr::CharsIdentify::LoadGrayChANN(std::_cxx11::string)’:
/home/zhijun/EasyPR-master/include/easypr/config.h:143:42: error: expected primary-expression before ‘>’ token
model = ml::ANN_MLP::loadml::ANN_MLP(path);
^
/home/zhijun/EasyPR-master/src/core/chars_identify.cpp:52:5: note: in expansion of macro ‘LOAD_ANN_MODEL’
LOAD_ANN_MODEL(annGray, path);
^
CMakeFiles/easypr.dir/build.make:86: recipe for target 'CMakeFiles/easypr.dir/src/core/chars_identify.cpp.o' failed
make[2]: *** [CMakeFiles/easypr.dir/src/core/chars_identify.cpp.o] Error 1
make[2]: *** 正在等待未完成的任务....
CMakeFiles/Makefile2:105: recipe for target 'CMakeFiles/easypr.dir/all' failed
make[1]: *** [CMakeFiles/easypr.dir/all] Error 2
make[1]: *** 正在等待未完成的任务....
[ 44%] Linking CXX static library libthirdparty.a
[ 44%] Built target thirdparty
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2
What is the expected behavior?
What is the motivation / use case for changing the behavior?
Please tell us about your environment:

System: Ubuntu16.04 LTS

Compiler version/IDE: gcc5.4.0

CMake version: 3.5.1

OpenCV version: [3.2 ]

看一下readme里介绍的怎么解决opencv3.2编译问题的方法

Readme并没有看到啊～
@liuruoze

对于Opencv3.2或以上版本，如果碰到编译问题，例如“ANN_MLP”相关的错误，尝试将config.h中将#define CV_VERSION_THREE_ZERO改为#define CV_VERSION_THREE_TWO试试. @newdee , 再试试看

对于Opencv3.2或以上版本，如果碰到编译问题，例如“ANN_MLP”相关的错误，尝试将config.h中将#define CV_VERSION_THREE_ZERO改为#define CV_VERSION_THREE_TWO试试. @newdee , 再试试看

thanks,It worked for me on opencv4


CatalogueDCAT-AP should modify the property names to dismiss the use of prefix

Modify to Catalogue and change the property issued to releaseDate. Pending fixing the context.jsonld of this property in statDCAT-AP


help-docs: Document "View user profile" mobile app feature.

Refactors the instructions for Desktop/Web and adds instructions for mobile app users.
Screenshots and screen captures:

Consolidates the two ways of viewing a user profile into one set of instructions under a single Desktop/Web tab.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@alya ready for review.

I've pushed the requested changes to View someone's profile. I also started a discussion thread to figure out how to get the correct information ⓘ icon, but maybe we can merge this PR and do that as a follow-up.

Let's put the icon in parentheses (cf. https://zulip.readthedocs.io/en/latest/documentation/helpcenter.html#icons). Otherwise looking good!

Let's put the icon in parentheses

Done!

Thanks! @timabbott please take a look.

I've pushed the requested changes to View someone's profile. I also started a discussion thread to figure out how to get the correct information ⓘ icon, but maybe we can merge this PR and do that as a follow-up.

Fixed the icon using the same strategy as in https://github.com/zulip/zulip/pull/22447#issuecomment-1216111114.

Also fixed the icon in https://zulip.com/help/pin-a-stream#alternate-method as a follow-up to #22429.

@timabbott I reviewed this one before my trip, and @drrosa has addressed the minor feedback I had, as well as updating the icon we're using. Could you please take a look?

@timabbott Fixed it. Thanks!

Merged, thanks @drrosa!


Parent state resetting when Child is unmounted

I have created an example here:
https://codesandbox.io/s/heuristic-chaum-imkoik?file=/src/App.js
In Parent component, after filling in the fields and clicking "Save", I set the state for "items".
After 5 seconds, I unmount the child component.
I am expecting items state in Parent to persist and not be effected by the Child being unmounted.
However, after Child is unmounted, the items state in Parent loses/resets the previously set values.
I can't pin point where the issue lies.

Before we look into this can you please use the latest version of informed. we no longer support V3 and it looks like your code sandbox uses v3

Yes unfortunately, all of our forms are currently v3 and we are hoping to upgrade to v4, but that's going to be a big undertaking.
I was able to tweak some things and get the sandbox to use v4.
On a side note, the doc for v4 https://teslamotors.github.io/informed/?path=/story/form--api mentions formApi.getState but looks like it should be formApi.getFormState().
After figuring that out, I'm getting the results that i'm expecting when using v4.
So it seems to be an issue with v3.
https://codesandbox.io/s/heuristic-chaum-imkoik?file=/src/App.js
I understand you don't support v3, so I'm probably going to have to figure out a different approach for unmount until we can get up to v4.
Thank you


Examples code updated with changes as requested (default values removed, use of variables restricted to only least, local values used wherever applicable, provider with lowest and one latest version defined, test/plan/apply verified)

Description

Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

Left some comments. Have you set up your local dev environment yet using steps at https://terraform-ibm-modules.github.io/documentation/#/local-dev-setup?id=local-development-setup ? Please do this and run pre-commit run --all-files as I think some hooks are going to update some of the formatting in your PR. If so, ensure to commit the changes.

@ocofaigh yes i have setup the local enviroment and tested running pre-commit run --all-files, and results are all passed.

@marifse thanks, I still dont see any updated to catalogValidationValues.json.template ?

@marifse thanks, I still dont see any updated to catalogValidationValues.json.template ?

I did not add any new variable in this module examples, though even if i remove the default values do i need to put those variables into catalogValidationValues.json.template file with their hard coded values? I will appreciate your help, as this is the first module on which I am working.
Another thing, on project description i got the instructions to use the one least available provider and one latest available provider in range defined in module as 1.56 and 2.0, while the 1.56.1 is being used in two or three examples, though there is not any example with only latest one 2.0 but two fscloud and advanced examples are with the range defined as >= 1.56.1, < 2.0.0?

@marifse If you remove a default value from a variable, it becomes a required variable, so you would then need to update the catalogValidationValues.json.template files with a value for the variable, however I just checked and it seems only prefix and region are being made required, and we already pass values in for them in the json template files, so actually there is no action required here.
The requirement for testing provider versions is that we need to ensure at least 1 example is testing the lowest version defined the the range, and at least 1 testing whatever the latest version is. So in this case, the supported provider range is 1.56.1, <2.00 which means we would want at least 1 example locked into 1.56.1, and at least 1 example that will always use the current latest version. To achieve that, you would use >=1.56.1.
Generally you will need to lock the most basic example into the lowest supported provider version because some of the more complex example may have dependant module that actually require later provider versions.

/run pipeline

@marifse The pre-commit hooks are failing in the pipeline? Are you sure they all pass locally for you?


Q4 changes

In this pull request you will find:

New method for running Refinery from Revit.
New Section on machine learning.
-New workflows:
Grid object placement.
Stepped grid object placement.
Random object placement
Sheet layout creation.

@radumg can you merge this pull request to DynamoDS/RefineryPrimer

included in PR #47


0.18.0.pre breaks Celluloid.running?

> $ bundle exec pry
[1] pry(main)> $CELLULOID_BACKPORTED=false
=> false
[2] pry(main)> require "celluloid"
=> true
[3] pry(main)> Celluloid.running?
Celluloid::Error: Celluloid is not yet started; use Celluloid.boot
from /home/.bundle/ruby/2.1.0/gems/celluloid-0.18.0.pre/lib/celluloid.rb:38:in `actor_system'
[4] pry(main)>

I've experienced this too.

This is at least partially intentional:
require "celluloid" does NOT call Celluloid.boot
require "celluloid/autostart" is the equivalent of require "celluloid" && Celluloid.boot
This allows tools using Celluloid as a library to avoid automatically starting Celluloid's background services simply by loading the library.
As far as what Celluloid.running? should do in this case, perhaps it should just return false. I am definitely open to that being the behavior here.

@tarcieri I get the meaning of the new require rules and I welcome them 👍  It's the running? call semantics which seem iffy. But since we are on the same page regarding that (about returning false), that'll make accepting the PR easier ;)

Please send a PR at your leisure

@tarcieri already did, weeks ago #749


Add itasa provider + fixes

If you want to merge this that would be great, otherwise it is mostly to keep track of the patches we have in sickrage (I will try and keep this updated with your develop branch).
Now only adds itasa, in an unregistered state.

not sure how to add cassettes, or fixup the opensubtitles cassettes.
If you can take care of that I can/will fix the other tests if needed.

Itasa isn't merged yet and code not reviewed. That means a lot of things can change like for legendas.
#536 is the reason why NapiProjekt is disabled.
Why add self.session to __init__? Why add "missing" ones? I have a warning on my IDE but nor pep8 neither pyflakes complain about this so why add code if it doesn't do anything? Sometimes one ought to be clever than best practices. Not sure it applies here anyway because documentation clearly states initialize is to be called if not using the with statement. I agree this is not ideal but I don't like the solution I'm left with. The alternative is to remove initialize and use __init__ for that purpose, might be a good solution but I need to review consequences. PR accepted if you're willing to contribute.
You cannot catch everything that can go wrong in Python, don't bother with ConfigurationError on empty string. The idea here is to catch missing parameters that come together.
Agreed for the raw string prefix in regex.

Don't merge yet, I will make some adjustments, like not enabling napiprojekt and split out some of the other changes into smaller PR.
My IDE (editor really) shows pep8, pyflakes, and pylint output.
http://pylint-messages.wikidot.com/messages:w0232
http://pylint-messages.wikidot.com/messages:w0201
About the username/passwords:
Checking only against None is fine if it is only being used via cli, as the arguments cannot be empty strings easily. But when using it as a module I would have to explicitly do sickbeard.BLAH_PASSWORD or None to be sure it is handled correctly in the module.
While that is ok and easy for me to do, if someone does not know of this quirk and is implementing subliminal as a module into their project it can cause them headaches figuring out why it is trying to log in when they didn't provide a username or password and subsequently becoming a disabled_provider in the pool. It just seems to have no negative effect and makes it a little easier on 3rd parties, because an empty username or password can never log in to any site anyways.
I will contribute as much back as I can, because I think its silly to keep patching downstream and it's sort of part of the deal to help back when you use someone else's work.

@Diaoul I split out the straight forward stuff into #617

https://github.com/SickRage/SickRage/issues/1457

A newer version is available in PR 804 from @Elettronik


Unexpected binaryTargets environment variables usage

Bug description
In version 2.22 and below, I was able to use an environment variable to set binaryTargets. This was the generator part in my schema:
generator client {
  provider          = "prisma-client-js"
  binaryTargets = [env("PRISMA_BINARY_TARGET")]
}

I could run something like this to set this environment variable:
$ PRISMA_BINARY_TARGET='native' yarn prisma generate

However, it does not work like this anymore in v2.26. If I upgrade Prisma to 2.26 and try running the same command, I'd get an error like this ( with DEBUG="prisma*" turned on):

How to reproduce

Create a new schema
Use an environment variable for binaryTargets
Set the environment variable when running relevant commands like prisma generate

Expected behavior
I was expecting v2.26 to work the same way as v2.22 and before:
$ PRISMA_BINARY_TARGET='native' yarn prisma generate

Would be nice to also have this documented somewhere as well ( Maybe it's out there but I missed it :) ? )
Alternative
Given the following setting:
generator client {
  provider          = "prisma-client-js"
  binaryTargets = env("PRISMA_BINARY_TARGET") // Notice env is now the `binaryTargets` array
}

Maybe the targets can be passed in with a , separator? For example:
$ PRISMA_BINARY_TARGET='native,darwin' yarn prisma generate

Current workaround
After digging around a bit in this file, I found that it was expecting an array in JSON string instead of a string.
That means I could do something like this:
$ PRISMA_BINARY_TARGET='["native"]' yarn prisma generate

Prisma information
N/A
Environment & setup

OS: MacOS
Database: MySQL
Node.js version: 12.13.1

Prisma Version
prisma               : 2.26.0
@prisma/client       : 2.26.0
Current platform     : darwin
Query Engine         : query-engine 9b816b3aa13cc270074f172f30d6eda8a8ce867d (at node_modules/@prisma/engines/query-engine-darwin)
Migration Engine     : migration-engine-cli 9b816b3aa13cc270074f172f30d6eda8a8ce867d (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine : introspection-core 9b816b3aa13cc270074f172f30d6eda8a8ce867d (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary        : prisma-fmt 9b816b3aa13cc270074f172f30d6eda8a8ce867d (at node_modules/@prisma/engines/prisma-fmt-darwin)
Default Engines Hash : 9b816b3aa13cc270074f172f30d6eda8a8ce867d
Studio               : 0.408.0

We need to confirm the described behavior in 2.22.0, reproduce the failure in recent versions and then understand how and why this changed - and then decide if this is something we can or want to undo, and how to properly document this.

I think this was caused when we had a regression which made env to stop working with binaryTargets: https://github.com/prisma/prisma/issues/7295
I believe we should match the documented behaviour which is PRISMA_BINARY_TARGET='native,darwin' or we should define what we are expecting here.

I believe we should match the documented behaviour

I don't think this is documented yet explicitly. The env() can replace values in the schema, and the expected value here is ["native", "darwin"]. (PRISMA_CLI_BINARY_TARGETS is something different that affects the CLI, not the Client [yep, confusing but still true])

Dont work env variable if binaryTargets is not "native"
  "devDependencies": {
    "prisma": "^3.13.0"
  },

That sounds unrelated @gimtonic. Please open a new issue and provide all information the issue template asks for.

Sorry. The problem is not relevant.
I fixed this by copying yhe .env file to inside docker container.

Hi there, this issue was still haunting prisma 4, so i figure out and workaround:
You need to use the env() method directly as the value and set the env to be the array
generator client {
  provider = "prisma-client-js"
  binaryTargets = env("PRISMA_BINARY_TARGET")
}

PRISMA_BINARY_TARGET=["native"]

Hope this helps

I believe it would be super nice if Prisma defaults to an environment variable (such as PRISMA_BINARY_TARGET) so that CI environments can set it without having to modify the schema.prisma file.
Shall I open a new issue for this?

I believe it would be super nice if Prisma defaults to an environment variable (such as PRISMA_BINARY_TARGET) so that CI environments can set it without having to modify the schema.prisma file.
Shall I open a new issue for this?

I guess by providing the following environments, it looks like Prisma picks the correct binary target:
PRISMA_CLIENT_ENGINE_TYPE=binary
PRISMA_CLI_QUERY_ENGINE_TYPE=binary
PRISMA_CLI_BINARY_TARGETS=rhel-openssl-1.0.x

No need to update the configuration.

Yes please, being able to override the configuration seems different enough to treat it via its own issue.

Yes please @svedova, being able to override the configuration seems different enough to treat it via its own issue. Thanks.

@janpio created: https://github.com/prisma/prisma/issues/20931


Issue with Core 2024.1

Logger: homeassistant.const
Source: helpers/deprecation.py:204
First occurred: 23:15:12 (1 occurrences)
Last logged: 23:15:12
DATA_RATE_MEGABITS_PER_SECOND was used from livebox, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfDataRate.MEGABITS_PER_SECOND instead, please create a bug report at https://github.com/cyr-ius/hass-livebox-component/issues

Logger: homeassistant.components.sensor
Source: helpers/deprecation.py:204
Integration: Capteur (documentation, issues)
First occurred: 23:15:12 (2 occurrences)
Last logged: 23:15:12
STATE_CLASS_MEASUREMENT was used from livebox, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorStateClass.MEASUREMENT instead, please create a bug report at https://github.com/cyr-ius/hass-livebox-component/issues

Logger: homeassistant.components.device_tracker
Source: helpers/deprecation.py:204
Integration: Dispositif de suivi (documentation, issues)
First occurred: 23:15:18 (2 occurrences)
Last logged: 23:15:19
SOURCE_TYPE_ROUTER was used from livebox, this is a deprecated constant which will be removed in HA Core 2025.1. Use SourceType.ROUTER instead, please create a bug report at https://github.com/cyr-ius/hass-livebox-component/issues

Please test this version 1.8.9-beta1
In HACS , select 3points , Redownload , Beta enable and select version

Hi @cyr-ius
It's OK with 1.8.9-beta1
Thanks


transpiling to es5

fix #18

hi,
sorry but not sure exactly how you did transpile the lib.
Isn't there a pipeline to transpile from source to target?
What if someone later add a const, will you think about transpiling again?
Asking because the lib did fail for me in IE11 and was not easy to track down what was the cause, and don't want it to happen again, but the process here makes me feel it's likely to happen again

@slorber You are right. It can happen again. I can't even do it manually every time. So I added a transpiling pipeline. (#23). I hope that will be merged. :smile:

great thanks for making this more sustainable ;)


SIP-23: Add support to Literal Types syntax (#620)

Thanks for the feedback, will update the PR tomorrow :)

I updated the PR with the review comments fixed.

LGTM! Thank you for your contribution.


Support #[derive(Serialize, Deserialize)] when using associated types

Prior to this, structs and enums that had fields using associated types wouldn't have the appropriate where bounds set. This fixes that, meaning that this now builds:
trait Trait {
    type Assoc;
}

#[derive(Serialize)]
struct Struct<T: Trait> {
    assoc: T::Assoc,
}

Fixes #1208.

Thanks btw! This is going to fix a quite common stumbling block.

Those errors should be fixed now.


explorer symlin

VSCode Version: Version 1.33.0-insider (1.33.0-insider
OS Version: MacOs 10.14.3

In the latest versions of the Insiders i have an empty explorer on some folder. Not all

Does this issue occur when all extensions are disabled?: Yes/No

sorry did not want to hit enter :s

Fixed in latest insiders which should be out soon.
Dup of some issue

@isidorn thanks!


improved docs for headAttrs

https://github.com/declandewet/vue-meta/issues/122

Codecov Report

Merging #125 into master will decrease coverage by 0.41%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #125      +/-   ##
==========================================
- Coverage      24%   23.58%   -0.42%
==========================================
  Files          18       18
  Lines         225      229       +4
==========================================
  Hits           54       54
- Misses        171      175       +4

Impacted Files
Coverage Δ

src/server/generators/attrsGenerator.js
0% <ø> (ø)
:arrow_up:

src/server/generateServerInjector.js
0% <ø> (ø)
:arrow_up:

src/client/updateClientMetaInfo.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Replace std::strcmp by std::string_view::operator==

Thanks a lot for making a contribution to SFML! 🙂
Before you create the pull request, we ask you to check the follow boxes. (For small changes not everything needs to ticked, but the more the better!)

[x] Has this change been discussed on the forum or in an issue before?
[x] Does the code follow the SFML Code Style Guide?
[ ] Have you provided some example/test code for your changes?
[ ] If you have additional steps which need to be performed list them as tasks!

Description
This PR is related to the issue https://github.com/SFML/SFML/issues/2366
I couldn't find all the references found in the issue but it should be exhaustive for master branch. Feel free to point out any missing references or any remarks.
Tasks

[ ] Tested on Linux
[x] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

Why couldn’t you find those other strcmp uses?

SFMLBuildMaster: Build this please.

I double checked and master does include more uses of std::strcmp to remove so please fix those in this PR too.

Thank you for the list, somehow VS couldn't find more references in the solution, I reconfigured the project via CMake and that should be all now

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

Please build the format target which will run clang-format and then those formatting CI jobs will start passing.

Would it be possible to support LLVM 15 and above ? I just saw the project only support up to LLVM 14

No because clang-format-15 produces different formatted code than versions 12 through 14.

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

Can we remove #include <cstring> from any of these files now?

SFMLBuildMaster: Build this please.

So after a few builds (sorry for that) that I couldn't test for Linux GCC and Clang, it appears the only
#include <cstring> we can remove are in SFML\Window\Unix\VulkanImplX11.cpp and SFML\Window\Win32\VulkanImplWin32.cpp files.

SFMLBuildMaster: Build this please.

For whatever reason the CI pipeline isn't running? At least it appears so to me.  Unsure why it's acting up.

Just did the two commands you suggested, doesn't look like CI is running ?
I generally interactively rebase my branch, choose to squash, fixup or edit commits I did (usually to have only one remaining) then force push the entire thing, could it introduce any problem for CI ?

SFMLBuildMaster: Build this please.

I generally interactively rebase my branch, choose to squash, fixup or edit commits I did (usually to have only one remaining) then force push the entire thing, could it introduce any problem for CI ?

This is perfectly fine. I think we just hit a weird one-time issue.


Empty commit: Trigger 1.33 builds

What this PR does / why we need it:
This is an empty PR, just to trigger openshift-ci builds.

/lgtm

Looks like there is infrastructure issue. I'll retry...
/retest

Again...
/retest

...again
/retest

/retest

There is infrastructure issue... I'll retry tomorrow.

/retest

I had report a CI outage on #forum-ocp-platform channel on slack. It could be track here
https://redhat-internal.slack.com/archives/CBN38N3MW/p1695306595749459

/retest

/test images

/approve


[bug] project root is not supported as project

Describe the bug

Create the following workspace.yml:

projects:
  root: "."

Run yarn moon project root
Output:

❯ yarn moon project root
Error: id::invalid_format

  × Invalid format for //, may only contain alpha-numeric characters, dashes
  │ (-), slashes (/), underscores (_), and dots (.), and must start with an alpha
  │ character.

Expected behavior
Project root could be used as a project
Additional context
I want to utilize specific tasks that should only be run from the project's root directory. Placing these tasks in tasks.yml without specifying the project makes them available globally.

@dec0dOS Hrmm something weird is going on here. It thinks the project name is //, but I was able to use root: '.' locally just now without issue.
Can you paste the --log trace output? Also, if you clear .moon/cache does it resolve itself?


Add DelayedRequestDisposition enum to URLSession

Added DelayedRequestDisposition enum to URLSession and added stub method called urlSession() that uses the enum in URLSessionTaskDelegate.

@swift-ci please test and merge

cc @saiHemak


Bar geometry not updated on monitor layout change

Describe the issue
I have 2 monitors, HDMI1 and eDP1 set up next to each other:
+--------------------+--------------------+
|                    |                    |
|     HDMI1          |     eDP1           |
|                    |                    |
|                    |                    |
+--------------------+--------------------+

I have a polybar on eDP1 with
monitor = eDP1

in its config.ini. When I unplug HDMI1, the polybar disappears from eDP1.
Then, when I plug in HDMI1 again, the polybar appears again to its place on eDP1.
When run with -l trace, its "Bar geometry" is:
* Bar geometry: 1920x24+1920+0; Borders: 0,0,0,0

It seems simply that bar geometry isn't updated automatically when monitors are attached / removed.
Expected behavior:
When unplugging a monitor, the polybar's position should visually remain the same and on the same monitor.
Actual behavior:
The polybar is rendered off-screen when the monitor is removed.
Was it working before?
Not that I am aware of.
To Reproduce
include-file = ~/.config/polybar/modules/modules.ini

[bar/testbar]
background = #000
foreground = #fff
monitor = eDP1
modules-center = date

polybar -c test.ini -l trace testbar

unplug HDMI1, plug HDMI1.
Polybar Log
Empty lines are  where HDMI1 is unplugged and plugged respectively.
notice: Parsing config file: test.ini
- config_parser: Parsing test.ini
- config_parser: Parsing /home/jamie/.config/polybar/modules/modules.ini
* Loaded monitor eDP1 (1920x1080+1920+0)
* Bar geometry: 1920x24+1920+0; Borders: 0,0,0,0
- bar: Attach X event sink
- bar: Attach signal receiver
- controller: Install signal handler
- controller: Setup user-defined modules
* Starting application
- controller: Main thread id = 1
* Starting module/date
* Entering event loop (thread-id=1)
- module/date: Thread id = 2
* Eventqueue worker (thread-id=3)
- bar: Create renderer
- renderer: Get TrueColor visual
- renderer: Allocate colormap
- renderer: Allocate output window
- renderer: Allocate window pixmaps
- renderer: Allocate graphic contexts
- renderer: Allocate alignment blocks
- renderer: Allocate cairo components
- renderer: Load fonts
* Configured DPI = 96x96
warn: No fonts specified, using fallback font "fixed"
notice: Loaded font "fixed" (name=Nimbus Sans, offset=0, file=/usr/share/fonts/gsfonts/NimbusSans-Regular.otf)
* Bar window: 0x1600001
- bar: Reconfigure window
- bar: Set window WM_NAME
- bar: Set window _NET_WM_WINDOW_TYPE
- bar: Set window _NET_WM_STATE
- bar: Set window _NET_WM_DESKTOP
- bar: Set window _NET_WM_PID
- bar: Map window
- bar: Draw empty bar
- bar: Setup tray manager
* Disabling tray manager (reason: missing `tray-position`)
* module/date: Rebuilding cache
- bar: Force update
* Redrawing bar window
- renderer: flush(2 geom=94x24+913+0, falloff=0)
- bar: Received expose event

- bar: Received expose event
- bar: Received expose event

- bar: Received expose event
- bar: Received expose event
notice: Termination signal received, shutting down...
- controller: Uninstall sighandler
- controller: Detach signal receiver
- controller: Stop modules
* module/date: Stopping
- module/date: Release sleep lock
* Deconstruction of module/date took 0 ms.
- controller: Joining threads
- module/date: Deconstructing
* Waiting for spawned processes to end
* Reached end of application...

Environment:

WM: bspwm
Distro: Arch Linux
Output of polybar -vvv:

polybar 3.4.0-106-g5cd7295a

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags: -D_FORTIFY_SOURCE=2 -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -Wall -Wextra -Wpedantic -O3 -DNDEBUG
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

Notes
Obviously, a workaround is to simply restart the polybar on each monitor layout change.
EDIT: I just realized the version of polybar I am running is out-of-date, reinstalling now and will update this issue.
EDIT2: Updated to latest master commit, g5cd7295a, repasted log, still observe this behaviour.

Obviously, a workaround is to simply restart the polybar on each monitor layout change.

The settings section has a screenchange-reload key which does this for you ;)
That's also the recommended way to handle this.

pff.. thanks! gotta read that wiki more thoroughly


User Interface Mockups and Storyboard Sequences

Diagram the layout of your main user interface and major dialogs, with comments describing the important elements. Also, diagram storyboard sequences of using your application, with transitions between different states of the user interface, caused by the user manipulating the controls or providing input. Within the diagrams, reference the relevant requirements. Convincingly show all the requirements are covered.
Acceptance Criteria

[x] Mockups display all key app screens including login, event creation, QR code scanning, and real-time dashboard, with clearly labeled UI elements.

[x] Storyboards depict critical user actions and transitions for both organizers and attendees, highlighting event creation, sign-up, QR code check-in, and notification management.

[x] Each mockup and storyboard frame includes detailed comments explaining the purpose of UI elements and expected user interactions.

[x] Designs comprehensively address every project feature such as QR code functionality, Firebase integration, role-based access, optional geolocation verification, and image uploads.

[x] UI design is consistent across all screens with an intuitive layout, following a unified color scheme and typography for user-friendliness.

[x] Designs demonstrate flexibility for future updates and feature additions, ensuring scalability.

[x] Interactive elements like buttons for event sign-up and receiving notifications are included to enhance user engagement.

[x] The designs are validated against described scenarios, effectively showing how the app supports event management and attendee check-in processes.

Completed Story Boards and User Interface Layouts!


Organization public dashboard takes ages to load

http://team.cartodb.com/maps

For me this is an easy solution: Public org dashboard doesn't needs to have "realtime" tables/vis listing, so why not simply caching them for a few hours? 1st load would still take time, but afterwards would run faster, and if is a frequently accessed dashboard would keep hitting the cache.
The proper solution isn't easy, as this listing will check all schemas from all organization users (around 50 at ours, for example) and then grab all public tables or vis. So backend refactor will probably not solve this issue (just reduce it's wait time).
Other solution would be maybe using varnish so full pages are cached? but invalidation won't be as easy...
Other solutions or ideas?
/cc @saleiva @javisantana @rafatower

Let's wait until the refactor finishes and then we revisit!


OfficeDeploymentTool - version 16.0.12827.20268

OfficeDeploymentTool - version 16.0.12827.20268
#17847
Adding version 6/6.
Microsoft Reviewers: Open in CodeFlow

Aside from the identifier this is a copy of the version that was added before. No need to fix this since the newer versions are already added? Going to close if all the other versions merge without issue.

Great, it was the old installer switches that weren't compliant. Fixed and now this closes the ODT package identifier renaming change!


Bazel with rules_typescript does not work

Issue description
I'm trying to use Bazel with a project that uses rules_typescript, but it's failing because it's not able to find uname, tr, and grep. It looks like it's not finding coreutils which indicate that it's not using the custom bash, of course it's an assumption but I'll have to debug further to confirm it.
Steps to reproduce
$ git clone --branch=test-bazel https://github.com/kalbasit/nixpkgs.git
$ cd nixpkgs/pkgs/test/bazel
$ nix-shell -p bazel go nodejs --run 'bazel build //rules_typescript:hello'
INFO: Analysed target //rules_typescript:hello (0 packages loaded).
INFO: Found 1 target...
ERROR: /home/kalbasit/code/opensource/stories/test-bazel/src/github.com/NixOS/nixpkgs/pkgs/test/bazel/rules_typescript/BUILD.bazel:3:1: Compiling TypeScript (devmode) //rules_typescript:hello failed (Exit 1)
bazel-out/host/bin/external/build_bazel_rules_typescript/internal/tsc_wrapped_bin.runfiles/bazel_tools/tools/bash/runfiles/runfiles.bash: line 60: uname: command not found
bazel-out/host/bin/external/build_bazel_rules_typescript/internal/tsc_wrapped_bin.runfiles/bazel_tools/tools/bash/runfiles/runfiles.bash: line 60: tr: command not found
failed to load main  Error: ENOENT: no such file or directory, open '/home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/sandbox/processwrapper-sandbox/1/execroot/nix_test/external/root_node_modules/node_modules/typescript/lib/lib.es5.d.ts'
    at Object.fs.openSync (fs.js:646:18)
    at Object.fs.readFileSync (fs.js:551:33)
    at UncachedFileLoader.loadFile (/home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/execroot/nix_test/external/build_bazel_rules_typescript/internal/tsc_wrapped/file_cache.ts:276:27)
    at /home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/execroot/nix_test/external/build_bazel_rules_typescript/internal/tsc_wrapped/compiler_host.ts:345:34
    at Object.wrap (/home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/execroot/nix_test/external/build_bazel_rules_typescript/internal/tsc_wrapped/perf_trace.ts:54:12)
    at CompilerHost.getSourceFile (/home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/execroot/nix_test/external/build_bazel_rules_typescript/internal/tsc_wrapped/compiler_host.ts:344:22)
    at findSourceFile (/home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/external/build_bazel_rules_typescript_tsc_wrapped_deps/node_modules/typescript/lib/typescript.js:75050:29)
    at args (/home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/external/build_bazel_rules_typescript_tsc_wrapped_deps/node_modules/typescript/lib/typescript.js:74987:85)
    at getSourceFileFromReferenceWorker (/home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/external/build_bazel_rules_typescript_tsc_wrapped_deps/node_modules/typescript/lib/typescript.js:74960:34)
    at processSourceFile (/home/kalbasit/.cache/bazel/_bazel_kalbasit/61aa1d2a671723c542d4a1c85ce4fd35/external/build_bazel_rules_typescript_tsc_wrapped_deps/node_modules/typescript/lib/typescript.js:74987:13)
Target //rules_typescript:hello failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 0.748s, Critical Path: 0.39s
INFO: 0 processes.
FAILED: Build did NOT complete successfully

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.

system: "x86_64-linux"
host os: Linux 4.17.8, NixOS, 18.09.git.7ffed91faf7 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.0.4
channels(root): "nixos"
channels(kalbasit): "nixos-18.09pre145679.dae9cf6106d"
nixpkgs: /home/kalbasit/code/personal/base/src/github.com/kalbasit/system/external/nixpkgs

cc @mboes @Profpatsch

@mboes any idea why this is happening?

The action that runs tsc_wrapped_bin probably needs to be patched to set use_default_shell_env = True.

@mboes looking at the docs, this requires patching rules_typesscript, right? Can we do anything here instead of filing an upstream patch?

@mboes the following patch to rules_typescript worked, I will submit the patch upstream but I'm unsure if they will merge it.
diff --git a/internal/build_defs.bzl b/internal/build_defs.bzl
index ee3b32d..f3a01c2 100644
--- a/internal/build_defs.bzl
+++ b/internal/build_defs.bzl
@@ -74,6 +74,7 @@ def _compile_action(ctx, inputs, outputs, tsconfig_file, node_opts, description
         execution_requirements = {
             "supports-workers": str(int(ctx.attr.supports_workers)),
         },
+        use_default_shell_env = True,
     )

     # Enable the replay_params in case an aspect needs to re-build this library.

We could patch the runfiles.bash file. That one comes from Bazel, not rules_typescript, and it's arguably the correct thing to hardcode absolute paths in that file.
use_default_shell_env = True is scary, because it potentially hurts hermeticity, except when using --experimental_strict_action_env (which is slated to become the future in the next few releases of Bazel). So I guess patching runfiles.bash is the better long-term fix.

We could patch the runfiles.bash file. It's arguably the correct thing to hardcode absolute paths in that file (or equivalently, hardcode PATH in that script).

@Profpatsch could you file a followup ticket about this and assign it to yourself?

@mboes I agree. I filed a PR upstream but I doubt it will go through. But since runfiles.bash is used by many rules, we should probably patch it instead.

@mboes @Profpatsch with the above PR, Bazel works with rules_typescript unchanged.


Issue #431: Improved error messages and -output on errors running SQL scripts

Fixed error output to not include stack trace, except with log level debug
Output the file that fails on errors agains the database
Added line number and position on the error (doesn't work on all databases - dependent on the level of error details the database client libraries provide)

Fixes #431