HVDC modeling issues

Currently HVDC modeling in PowerSimulations with PTDF network formulations seems like its not working, issue I found :

HVDCDispatch is not adding the flow variables to balance expression
HVDC lossless still seem to have flow variable set to its lower bound when not being used

Ideal solution would be get HVDC line flow to match between B-theta and PTDF based methods
Test script is attached below
using PowerSimulations
using PowerSystems
using PowerSystemCaseBuilder
using InfrastructureSystems
const PSY = PowerSystems
const PSI = PowerSimulations
const PSB = PowerSystemCaseBuilder
using Xpress
using JuMP

solver = JuMP.optimizer_with_attributes(Xpress.Optimizer, "MIPRELSTOP" => 1e-5, "MAXTIME" => 300);

sys_rts_da = build_system(PSITestSystems, "modified_RTS_GMLC_DA_sys");
PSY.transform_single_time_series!(sys_rts_da, 24, Dates.Hour(24))

f_bus = first(get_components(Bus, sys_rts_da, x -> x.number == 113))
t_bus = first(get_components(Bus, sys_rts_da, x -> x.number == 316))
l = PSY.HVDCLine(
    name="test_hvdc",
    available=true,
    active_power_flow=1.0,
    arc=Arc(from=t_bus  , to=f_bus),
    active_power_limits_from=(min=0.0, max=1.0),
    active_power_limits_to=(min=-1.0, max=0.0),
    reactive_power_limits_from=(min=0.0, max=0.0),
    reactive_power_limits_to=(min=0.0, max=0.0),
    loss=(l0=0.0, l1=0.0),
)
add_component!(sys_rts_da, l)

# template_uc = ProblemTemplate(NetworkModel(PSI.DCPPowerModel, use_slacks = true))
template_uc = ProblemTemplate(NetworkModel(PSI.StandardPTDFModel, use_slacks = true, PTDF = PSY.PTDF(sys_rts_da)))
# template_uc = ProblemTemplate(NetworkModel(PSI.PTDFPowerModel, use_slacks = true, PTDF = PSY.PTDF(sys_rts_da)))
set_device_model!(template_uc, ThermalStandard, ThermalStandardUnitCommitment)
set_device_model!(template_uc, RenewableDispatch, RenewableFullDispatch)
set_device_model!(template_uc, PowerLoad, StaticPowerLoad)
set_device_model!(template_uc, Line, StaticBranch)
set_device_model!(template_uc, Transformer2W, StaticBranch)
set_device_model!(template_uc, TapTransformer, StaticBranch)
set_device_model!(template_uc, HydroDispatch, HydroDispatchRunOfRiver)
set_device_model!(template_uc, GenericBattery, BookKeeping)
set_device_model!(template_uc, HVDCLine, HVDCLossless)

models = SimulationModels(
    decision_models = [
        DecisionModel(
            template_uc,
            sys_rts_da;
            name = "UC",
            optimizer = solver,
            system_to_file = false,
            initialize_model = false,
            optimizer_solve_log_print = true,
        ),
    ],
)

sequence = SimulationSequence(
    models = models,
    feedforwards = Dict(),
    ini_cond_chronology = InterProblemChronology(),
)

sim = Simulation(
    name = "hvdc_test",
    steps = 1,
    models = models,
    sequence = sequence,
    initial_time = DateTime("2020-04-02T00:00:00"),
    simulation_folder = ".",
)
build_out = build!(sim; console_level = Logging.Info, serialize = false)
execute!(sim)

results = SimulationResults(sim);
results_uc = get_decision_problem_results(results, "UC");
variables = PSI.read_realized_variables(results_uc)

variables["FlowActivePowerVariable__HVDCLine"]

There is an additional challenge in this case with HVDC dispatch since the implement model isn't really meant to handle by directional HVDC flows with losses.

This issue was resolved but it wasn't a bug. It was a new requirement on the modeling of HVDC systems.


Wordle Game using ReactJS

Describe the project you want to add with tech stack
The famous Wordle Game created using ReactJS.
Expected behavior

Wordle is a word-guessing game that challenges players to guess a secret five-letter word within six attempts.
The game board shows the results of each attempt, and players interact with a virtual keyboard to input their guesses.
The letters which can and cannot be used for the next move are represented by different colours to help users to distinguish better.
The game shows the correct letter at the end in case the user loses.
Special favicon for the project.

Screenshots

Initial look

After entering a word

Video Illustration
https://github.com/pranjay-poddar/Dev-Geeks/assets/103258259/fd8ae3be-99b6-464e-842c-c7175b71c532

@TusharKesarwani The PR of this issue is attached with it . Kindly review and merge it. PR no : #4003


Add bioengine app launcher service

Register app launcher service.

@Nanguage I am getting this error now:
  File "/Users/wei.ouyang/opt/miniconda3/envs/bioengine/lib/python3.10/site-packages/hypha_launcher/api.py", line 177, in start_s3_server
    await job.wait_until_status("running")
AttributeError: '_SubprocessJob' object has no attribute 'wait_until_status'

Is this because of the job engine is not up to date? If yes, should we pin the version?

Made a PR here: https://github.com/aicell-lab/hypha-launcher/pull/8

This is executor's error, please update executor to latest version: python -m pip install executor-engine --upgrade


Use in memory fs from webpack when in development mode.

Webpack middleware serves files from an in memory file system, which should be used when accessing any of the generated files during development.

Eyh, awesome, so this fixes changing the index.html file and just do a reload?

You can change the index.tpl.html and do a reaload and you don't have to run npm run build before you can start developing with npm start.

:+1:


Feedback for “Models”

Hi - donated - didn't get a link to the 1-click.

Can you leave your email? I will resend the download email.


📝 Docs: Explicitly state browser version support range on mochajs.org

Documentation Request Checklist

[x] I have read and agree to Mocha's Code of Conduct and Contributing Guidelines
[x] I have searched for related issues and issues with the faq label, but none matched my issue.
[x] I want to provide a PR to resolve this

Overview
Forking out of #5225:

Not sure what browser versions do mocha intend to support.

Yes, good point, this is never explicitly stated in full on the README.md or https://mochajs.org. Searches for chrom and firef yield no relevant results. The only version range mentioned is:
https://github.com/mochajs/mocha/blob/8711e72b9dd573dff6940061dd5643986c02f38a/docs/index.md#L68
Additional Info
No response

I found this:
https://github.com/mochajs/mocha/blob/main/PROJECT_CHARTER.md?plain=1#L55

§2.2: Out-of-Scope
...

Use within browsers not meeting a threshold decided upon by maintainers

Amusing.
My proposal would be to sync the timeline of browser support with Node.js support. As in, Mocha should support the versions of mainstream browser versions that were most current when the supported Node.js versions were released. Including data on the Node.js 14 (released April 21st, 2020) support equivalents from https://github.com/mochajs/mocha/issues/5225#issuecomment-2407509955:

Chrome/Chromium 81 released April 7, 2020: https://developer.chrome.com/blog/new-in-chrome-81
Firefox 75 released April 7, 2020: https://www.mozilla.org/en-US/firefox/75.0/releasenotes
Safari 13.1 released March 24, 2020: https://developer.apple.com/documentation/safari-release-notes/safari-13_1-release_notes


[#77] Release 0.2.0

Notes for Reviewer

Pre-Review Checklist for the PR Author

[x] Add sensible notes for the reviewer
[x] PR title is short, expressive and meaningful
[x] Relevant issues are linked
[x] Every source code file has a copyright header with SPDX-License-Identifier: Apache-2.0 OR MIT
[x] Branch follows the naming format (iox2-123-introduce-posix-ipc-example)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID ([#123] Add posix ipc example)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Assign PR to reviewer
[x] All checks have passed (except task-list-completed)

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Unit tests have been written for new behavior
[ ] Public API is documented
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References
Use either 'Closes #123' or 'Relates to #123' to reference the corresponding issue.
Relates to #77

Oh, it seems cargo audit expects a released iceoryx2 v0.2

@elBoberido mmhh it seems that we first have to release to crates.io and then merge the pull request. So I would split a release always in two PRs one for the docs and one PR that just adjust the version number. And this PR is already released to crates.io before it is merged.

@elBoberido mmhh it seems that we first have to release to crates.io and then merge the pull request. So I would split a release always in two PRs one for the docs and one PR that just adjust the version number. And this PR is already released to crates.io before it is merged.

This doesn't sound right to me. I've read somewhere that tags should always be set on master and the the release should happen from the tag. Maybe we use cargo audit the wrong way.

@elfenpiff I think I know the solution but I'm not quite sure how fast I could implement it. It should be sufficient to run cargo audit only after the merge and for the PR have a check to ensure that the last cargo audit check passed. It's similar on how I ensured that a failure of the nightly build in iceoryx1 is noticed on new PRs.

@elBoberido I think nearly all CI targets are affected by the problem, see the coverage and windows targets:

https://github.com/eclipse-iceoryx/iceoryx2/actions/runs/7455143385/job/20283757389
https://github.com/eclipse-iceoryx/iceoryx2/actions/runs/7455157283/job/20283785129

I would propose that we perform this release in the "uncleaner" way.

Release on crates.io
Merge Cargo.toml version change
Create Tag

And then we create a follow-up issue where we take a look at our CI, adjust the jobs so that we can release like:

Merge Cargo.toml version change
Create Tag
Release on crates.io


Navigation change doesn't get reflected till a user action

Short description of the problem:
nav.setRoot or nav.push or nav.present(alert) doesn't show up till another user action is made on the page.
What behavior are you expecting?
After clicking verify OTP, either error should popup for invalid otp or navigate to tabs page if OTP is valid. However, in either of the case till user doesn't perform another action, the navigation change doesn't show up in the screen - happens in both Android and browser
Steps to reproduce:

Checkout repo below
ionic serve
Enter any random name and phone number.. Enter OTP as "0000". Click verify otp

https://github.com/SaiNageswarS/ionic2-sampleapp
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Which Ionic Version?  2.x
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 5.4.1
Gulp version:  CLI version 1.2.1
Gulp local:   Local version 3.9.1
Ionic Version: 2.0.0-beta.3
Ionic CLI Version: 2.0.0-beta.19
Ionic App Lib Version: 2.0.0-beta.9
OS: Distributor ID:     Ubuntu Description:     Ubuntu 15.10
Node Version: v5.0.0

Hello! Are you still having this issue with the latest version of Ionic 2? (beta.5) Thanks!

Hello! As it has been a little while since there was any activity on this issue I will be closing it. Thanks for using ionic!


Introduce DEBUG flag for CLI

Sometimes we want more verbosity of error the CLI is making it would be great if I can have an env variable like DEBUG=1 to have more details. Also a great tool when we are asking users to share CLI bugs to run debug=1 so we can have more logs.

actually running DEBUG=graph-cli:* pnpm graph does the trick. Need to revamp documentation


How to use mujava to mutate the entire project?

Hi,my problem now is that I want to mutate the java files in the entire joda-time project, but the java files will call the methods of other classes of the package or the methods of other package classes. Can situations like this be handled by mujava?
No matter I put the whole project or a single java file into the src folder, it will report an error.
If I put a single java file into the src folder, the log file shows the following error:
[Error] MonthDay.java java.lang.NoClassDefFoundError: Could not initialize class org.joda.time.DateTimeFieldType
MutantsGenPanel:
java.lang.NoClassDefFoundError: Could not initialize class org.joda.time.DateTimeFieldType

If I put the whole project into the src folder, the log file shows the following error:
[Error] org\joda\time\Chronology.java java.lang.NoClassDefFoundError: Could not initialize class org.joda.time.DateTimeZone
MutantsGenPanel:
java.lang.NoClassDefFoundError: Could not initialize class org.joda.time.DateTimeZone

Regards.

There is similar question in FAQ.

There is a similar question in FAQ.

Hi.After many attempts, I still can't successfully use mujava to mutate in a large java library. My operations are as follows:

Copy the src code in the java library to the src folder of mujavaHome.
Copy the class file in the corresponding java library to the classes folder of mujavaHome to mutate.
The command line reports the following error:

I didn't find a solution in the FAQ, how can I solve this situation?
Regards.

According the answer of question 1 in FAQ, it is usually due to corrupted result data in the result folder. You should delete everything in the result folder and try again.
Is your GUI showing after typing the command?

According the answer of question 1 in FAQ, it is usually due to corrupted result data in the result folder. You should delete everything in the result folder and try again.
Is your GUI showing after typing the command?

My gui is displayed. I have tried to delete the result folder and run it multiple times, it is still the original error

There is instruction in FAQ.
If the GUI shows without problems after typing the command, NoClassDefFoundError is usually because muJava can’t find the class. This is probably because either the classpath is set incorrectly (not pointing to the correct files or folders), or the compiled .class file is not in the class folder.
What is ur CLASSPATH setting?
And, did u place compiled file in class folder?

But I put all the class files compiled in “idea” into the classes folder

Could you share your code or Mujava directory structure?
Its hard to help you without that.

Could you share your code or Mujava directory structure?
Its hard to help you without them.

Yes,I can.

Mujava directory structure:

Regards.


Feature/gal/12 add route for add new expense expense server

we have another PR for that: https://github.com/ofir-zeitoun/expanses-server/pull/18


Create common aliases

Currently
The commonly used methods are under a second level object.
Wanted
Create aliases

[ ] ATV.Page.create --> ATV.createPage
[ ] ATV.Menu.create --> ATV.createMenu
[ ] ATV.Navigation.navigate --> ATV.navigateTo
[ ] ATV.Navigation.navigateToMenuPage --> ATV.navigateToMenuPage
[ ] ATV.Navigation.presentModal --> ATV.presentModal
[ ] ATV.Navigation.dismissModal --> ATV.dismissModal

@emadalam , is the issue still open? i'm new t open source,would like to work on it


Missing tests for challenge solvers (clients)

There aren't any rspec tests for the challenge clients. This requires controlling how the clients actually perform their challenge verification.

I now have unit tests for these clients.


Instruction changes v2

print("hello", "world") == print("hello world") -- i think vsb does some weird formatting but not normal lua
Same as my old pull request, but with the error removed

print("hello", "world") == print("hello world") -- i think vsb does some weird formatting but not normal lua
Same as my old pull request, but with the error removed

Appearently normal lua uses tabbing as a seperator, instead of a space. Change it to use table.concat again (i did this before but reverted it because i thought it was unnessecary)

stop spamming their email

print("hello", "world") == print("hello world") -- i think vsb does some weird formatting but not normal lua
Same as my old pull request, but with the error removed

we do not mention vsb here

this doesn't change anything! this is literally just some optimizations....

this doesn't change anything! this is literally just some optimizations....

 Second of all, the current tpscript also uses table.concat so what.
Third, what do you mean I'm breaking backwards compatibility? I have no clue why I changed would break it.


[WIP] [CX_CLEANUP] - Create a new task for QuorumProposalRecv events

Closes #2951

This PR:

QuorumProposalRecv is an event that is handled by the consensus task when a new quorum proposal is received. This task is also responsible for validating multiple pieces of the proposal before submitting a QuorumProposalValidated event. Since this sequence of steps is now being split across the QuorumProposalTask and QuorumVoteTask, we need to avoid duplicating this behavior by instead creating a new task whose sole objective is to handle receiving and validating the quorum proposal and sending the validated event.
This PR does not:

Key places to review:

Another thing we need to fix: In the new task we call validate_proposal, which is a function in the consensus task. I think we need to either modify that function to emit ProposeNow and VoteNow when able, or remove that call and directly do validations in the new task and send *Now events. I think both can work but the latter is probably better since we don't have to make validate_proposal work for both tasks.

I fixed the lint errors, except for the documentation for the Temporary* types because I'm not sure why we need them. I added a TODO there.

Fixed more lints for the build with the dependency-task feature. There are a few errors left related to A: ConsensusApi which I think we can remove from the consensus task state. I'm going to do this in a separate PR since it's unrelated to the proposal refactor.

Fixed more lints for the build with the dependency-task feature. There are a few errors left related to A: ConsensusApi which I think we can remove from the consensus task state. I'm going to do this in a separate PR since it's unrelated to the proposal refactor.

I've removed ConsensusApi in https://github.com/EspressoSystems/HotShot/pull/3025. I think we can merge that one first, then the build fix (with the dependency-task feature) for this PR should be easy.

I fixed the build for compiling with and without dependency_tasks feature in this PR: https://github.com/EspressoSystems/HotShot/pull/3036. Didn't push directly to here since it has some changes besides the build fix.

Have you tried the proposal and vote task tests? They both hang for me locally, which I think is probably caused by a previous PR but we should fix it (not necessarily in this PR).

Have you tried the proposal and vote task tests? They both hang for me locally, which I think is probably caused by a previous PR. Let's fix it! Not necessarily in this PR though.

Cutting a branch with CI changes and testing now.


Added Java 13 to pipeline.

Pipeline passes, the old failures are due to Github errors that have been occuring lately (i.e. https://github.blog/2020-02-28-february-service-disruptions/)


[Refine] Refine the TinkerModellor's Construction

Stage 1
Problems:
1.It is a little bit messy of _tkm.py
Target:
1.Fully understand the entire project
2.Move func _read_top_file and _read_gro_file into _tkm_system.py. Refine the build_tkmsystem func
3.When func build_tkmsystem is called, a tkmsystem object should be returned

Target refreshed.

refine the basic class for datastorage (GMXMolecule)
reference:https://github.com/mizu-bai/tinker-trjconv/blob/main/tinker_trjconv/libmol/_libtxyz.py

Code has been already refined
https://github.com/WanluLigroupUCSD/TinkerModellor/pull/4


Update to node 10 in .travis.yml

Coverage remained the same at 99.099% when pulling 632c8a2f0e90e3239561ce90527f7789cf1029a2 on greenkeeper/update-to-node-10 into b3bcb2aad3caea155c7ce6f2d8e4d0606db78033 on master.

Coverage remained the same at 99.099% when pulling 632c8a2f0e90e3239561ce90527f7789cf1029a2 on greenkeeper/update-to-node-10 into b3bcb2aad3caea155c7ce6f2d8e4d0606db78033 on master.


CMake cmake_minimum_required Deprecation Warning

Fixes: #1545
Recently, when building pioasm and/or elf2uf2, CMake warns about cmake_minimum_required:
[ 96%] Performing configure step for 'PioasmBuild'
loading initial cache file /home/gmt/Projects/pico/build/pico-sdk/src/rp2_common/pico_cyw43_driver/pioasm/tmp/PioasmBuild-cache-Release.cmake
CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 3.5 will be removed from a future version of
  CMake.

  Update the VERSION argument <min> value or use a ...<max> suffix to tell
  CMake that the project does not need compatibility with older versions.

#1477 reported the issue, but submitted patch is not correct.

https://github.com/raspberrypi/pico-sdk/pull/1477 reported the issue, but submitted patch is not correct.

Why not?

#1477 reported the issue, but submitted patch is not correct.

Why not?

Simply by applying it, I see the warning again! I have cmake 3.27.8.
But I will check it again.


Adding Guidance for Conflict Resolution

This provides a starting point for conflict resolution, based on the original doc that outlined some proposed solutions to potential conflicts. Importantly this does not attempt to solve all potential conflicts, just provides guidance for some common scenarios as well as the guiding principles we identified earlier.
/cc @jpeach @hbagdi @bowei

LGTM, although https://github.com/kubernetes-sigs/service-apis/pull/303#discussion_r488149164 should be documented somewhere.

@hbagdi I think I've resolved all the feedback now, PTAL.

LGTM

/lgtm


Translation - Mandarin - i18n

This issue is here to create Mandarin translation of our app. Can be done by adding json and translating all field values from en.json
https://github.com/arc53/DocsGPT/tree/main/frontend/src/locale

Hey @dartpain, I would love to help out with this. I personally don't know Mandarin but if an AI translation or anything else is good, I can go ahead and do this. Let me know!

Hello


Added footer to the website like creativecommons.org

Fixes

Fixes #34 by @tinniaru3005
Description

The PR adds a footer to the page, just like the one in creativecommons.org website.
Technical details

Added footer section in listing.html, added css in style.css and included fonts, SVGs and new file vocabulary.css. Followed the code scheme of the main CC website, assigning variables to colors and svg.
Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@possumbilities @Xaid-vfx
Here is the revised PR for the addition of footer to the website. please have a look for any issues with the same 🙌

@Murdock9803 Thanks for these changes, there's just a few more things. To pull this off you should add the entire vocabulary directory in to the /docs folder? or where relevant.
Then items like the css, assets, js, fonts will all work if they're included properly.
See the comment I made on a similar PR. vocabulary is more or less an "all or nothing" inclusion from a dependency standpoint.

https://github.com/creativecommons/cc-resource-archive/pull/198#issuecomment-1985883117

and since adding vocabulary completely is required to implement the footer, it would also open up the ability to add other areas (like the new header) in future PRs.

@possumbilities I have done the required changes, Although the footer worked perfectly with all the fonts in previous iteration of this PR,  but adding the new vocabulary directory opens up possibilities for future enhancements in header and other areas too (as advised by you).
So, I added the whole directory in the /docs folder and edited the paths to work fine 🙌
Please suggest any changes if required to be made 🙌

@Murdock9803 Thank you! I add two small comments of correction and I think that should wrap this PR for approval. Great Work! ❤

@possumbilities I have added the correct absolute links to the footer and also the End-of-line line in style.css. I should've kept these in mind while making the PR, and will surely try to make more perfect ones 🙌

It feels great when the work is successfully merged 🙌 😄


NPSP Settings Should Check for Permissions on Permission Set Assignments instead of Profiles

Several recent updates to NPSP check for Admin Access by querying for the permissions associated with the User's Profile - specifically Customize Application, Modify All Data, and Author Apex.
If, however, a User has these permissions instead assigned via a Permission Set, we don't recognize their admin access and throw an error.
We check for Profile Permissions in the isSysAdmin() method of USER_UserService.
Map<Id, Profile> sysAdminProfiles = new Map<Id, Profile>([SELECT Id
                , PermissionsCustomizeApplication
                , PermissionsModifyAllData
                , PermissionsAuthorApex
                FROM Profile
                WHERE PermissionsCustomizeApplication = TRUE
                AND PermissionsModifyAllData = TRUE
                AND PermissionsAuthorApex = TRUE]);`

Suggestion
Take a similar approach to how we check for Customize Application in the isCustomizeApplicationUser() method of USER_UserService and query by using PermissionSetAssignment (since Profiles ultimately use these behind the scenes).
List<PermissionSetAssignment> permissions = [
            SELECT Id
            FROM PermissionSetAssignment
            WHERE AssigneeId = :userId
            AND PermissionSetId IN (
                SELECT Id
                FROM PermissionSet
                WHERE PermissionsCustomizeApplication = true)
        ];

Steps to Repeat

Create a Permission Set with the following System Permissions: Author Apex, Customize, Application, and Modify All Data.
Clone the Standard User Profile and verify it does NOT have the following System Permissions: Author Apex, Customize, Application, and Modify All Data.
Create a new User and assign to your new Profile. Assign the new Permission Set as well.
Log in as that User.
Go to NPSP Settings → Donations → Customizable Rollups. Try to enable Customizable Rollups. Error appears: Error: core.apexpages.exceptions.ApexPagesGenericException: system.security.NoAccessException: Update access denied for npsp__Customizable_Rollup_Settings__c, controller action methods may not execute

**lurch: add

Tracking W-042149


Fix memory safety issues found by OSS-Fuzz

This PR attempts to fix most of the open security-relevant issues found by OSS-Fuzz:

https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=36998: RWMol::replaceAtom needs to update references in stereo groups to avoid use-after-frees. Fixed in ae37e6eb3dabcb793b21ed679a4e4f4b4514a019
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=28660: iterateCIPRanks wrote out-of-bounds in some cases when the rank did not change in the current iteration. Fixed in a88de3ed64a5278ac25e8541cbdc347a108e1bbd with an extra bounds check.
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=25220 and https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=25225: A couple of functions used uninitialized variables. 14b852bcee980db6a655bdb357f2905939bddc44 initializes them.
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=28034: an implicit cast on 32-bit systems would lead to allocating the wrong number of elements in streamReadVec, leading to an out-of-bounds write. Fixed in ace149aba1d31845c89177297f92b5c918af878e with a numeric_cast, which will throw an exception if the casted value is out of range.

@greglandrum: Thanks for the review! I added the suggested test in a72ae63c2072ed0b93deac19f544d7240a5d1ea.

@greglandrum: Thanks for the review! I added the suggested test in a72ae63.

@alpire: great! thanks. I will merge this once the CI builds are done.
Thanks for the contribution. I try and set some time aside every once in a while to clear up the problems oss-fuzz turns up, but it's a slow process. Thanks for doing these!


Select2 products list in voucher form

Replaces default ChoiceField for project's custom AjaxSelect2ChoiceField in voucher form.
Closes #1398
Pull Request Checklist

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[x] JavaScript code quality checks pass: eslint.

Codecov Report

Merging #1408 into master will decrease coverage by 0.03%.
The diff coverage is 69.23%.

@@            Coverage Diff             @@
##           master    #1408      +/-   ##
==========================================
- Coverage    82.9%   82.87%   -0.04%
==========================================
  Files         128      128
  Lines        5775     5787      +12
  Branches      663      666       +3
==========================================
+ Hits         4788     4796       +8
- Misses        830      832       +2
- Partials      157      159       +2

Impacted Files
Coverage Δ

saleor/dashboard/product/urls.py
100% <ø> (ø)
:arrow_up:

saleor/dashboard/discount/forms.py
78.02% <66.66%> (-1.05%)
:arrow_down:

saleor/dashboard/product/views.py
70.1% <71.42%> (+0.02%)
: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 fbbedaa...b6428d5. Read the comment docs.

@mad-anne I just checked that we have a select with products in the sales form too, so Select2 widget would be welcomed there as well. Also, the list of countries in "Price per country" form may utilize this widget as it is quite long (although it doesn't the need dynamic AJAX prefetching since it's static).
As for the permissions - both voucher and sales views have their own permissions, but their using either products or categories inside. Shouldn't we also require "product" permissions to access those views (cc @koradon @akjanik)?

@elwoodxblues I feel that for the country selection in shipping pricing we should open a separate issue as the country choices there should allow multiple selections.

@elwoodxblues Depends what we want to achieve. Ideally permissions should be as intuitive as possible; I mean, we don't want someone to be stuck at point where he can't access view, but he/she has "proper" permission, because someone forgot give additional one.

@akjanik I agree with that. But in this case, it looks like the vouchers view simply requires two types of permissions. If we require only "vouchers" permission, we would leak the list of products. Maybe some kind of dependency between permissions is needed here. E.g. when giving access to the vouchers view, we would also have to check if access to the products list was also given and raise a ValidationError otherwise. @patrys Do you have any thoughts what we could do about this?

I think having either of the qualifying permissions (this means either "can view products" or "can manage vouchers") should be enough. Otherwise it's a useless view that appears to be broken (haven't tested this but I assume the AJAX response will return a 403 instead of matching products).

I'm +1 on limiting the list of products only to the public ones when the user has no "can view products" permission.

List of products is already filtered in this view by available ones, so all returned products are published in the storefront. We can conditionally show all products if the user has view_product permission, but it will enable creating vouchers and sales also to currently unavailable products. If it would be right in your opinion (I suppose it makes sense at all), i will change it.

@mad-anne Adding vouchers or sales for unavailable products shouldn't have any effect. Since those products can't be added to cart, there won't be any way to use the voucher or sale in the storefront.

@elwoodxblues but during sale some of products may become available, so staff users should have a possiblity to add them to a sale even if they are not visible in the storefront yet. I changed code to fit this rule.

I agree, it should be possible to prepare the necessary vouchers ahead of publishing products.


Configurable tempDir or at least exposing it some how

I'm launching several hundred browser sessions, each one utilizes Deno.makeTempDirSync() to get where to store the browsers user-data-dir.
It would be awesome if LaunchOptions had a userDataDir option to override this behavior, so I could have them be generated on a seperate drive that has more disk space available.
At the very least returning or making available which temporary directory it created would be great, so I can properly cleanup the files afterwards.

Interesting use case, this definitely makes sense to solve. Let me think about this.

Since astral just uses Deno.makeTempDirSync() a good work-around is simply setting the TMPDIR env var before including or using astral like so:
Deno.env.set("TMPDIR", "/mnt/drive2/tmp");
import {launch} from "jsr<EMAIL_ADDRESS>
I think this is a perfectly acceptable way of handling things and don't think astral needs to spend any dev time adding in it's own way. Thanks for creating astral :)


Prevent toolbar menu drop-downs from being clipped

Fixes regression introduced in fc4f1aae.

CSS is treacherous. More context: https://stackoverflow.com/questions/6421966/css-overflow-x-visible-and-overflow-y-hidden-causing-scrollbar-issue


Adding buffer_stop preset

This adds a buffer_Stop preset, using a lovely new icon :+1:

Looks awesome, thanks @JamesKingdom..
I didn't know about these things until today, but around where I live they are mapped as railway=halt, which I guess is wrong. Looks like I have some fixing to do. 🚂


feat: high severity dashboard

Add another dashboard which displays high severity alarms (canaries at this point).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Screenshot:


feat(visual-studio-code): add actions to remove entries

Description

add an action to remove a recent project entry
add an action to remove all recent project entries, the user will be prompted for confirmation
remove unused dependency sql.js

I used the sqlite3 CLI directly, similar to the useSql hook, without using an npm package. Will this cause any problems?
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @pernielsentikaer  👋

Maybe we should add a toast that says it requires a restart of the client?

Which client are you referring to? 😅
I had no problem removing entries and didn't require a restart from anything

I was testing this in Cursor - here I had to restart before it happened


Work for 1.1.4

@bvisness Not sure whats wrong with this. Errors are not making much sense

@DanielGibson This should fix your issue.

Okay.. Im confused now why doesn't it know what Left and Right is. This compiles fine in my project

Woohoo okay that was harder than it should've been. @bvisness take a look so we can merge this in!

Yep, will do it soon

Closing this out changed some stuff on my internal copy and gonna implement the changes into master instead of handling conflicts here.


[7.x] revert PR #33430

As noted by issue #33485 PR #33430 breaks a Laravel app when using the freeTDS driver to connect to SQL Server.
PR #33430 was sent to address issue #32883, but when implementing it I didn't test it against freeTDS drivers. I only tested against Microsoft official drivers ( https://github.com/Microsoft/msphpsql ). It seems freeTDS ise widely used , specially with older versions of SQL Server.
After the issue was reported I started investigating a solution that could solve both issues #33485 and #32883 but I could not find one.
So I am sending this PR to revert the changes made. Issue #32883 will still need to be addressed after reverting it.
I shared some of my findings on a comment on PR #33430 discussion and also presented some alternatives. Please refer to comment https://github.com/laravel/framework/pull/33430#commitcomment-40493561 to have more details about it.
There I said I would send PR for both 6.x and 7.x branches, but I will wait on feedback on this one before sending the second one as maintainers could have a different workflow on back porting features and fixes.
If someone has a better suggestion on how to address this I would be glad to help.

Please target this to 6.x, not 7.x.

Our standard process is to merge upwards, and not to backport.

Thanks @GrahamCampbell . Sent PR #33496 target to branch 6.x


fixfix iceberg catalog display type

Proposed changes
fix type:

Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


crash fluent by requesting invalid reduction

Maybe need to pre-check the expression validity.
This is by word of mouth so I don't have specifics.

@prmukherj please don't start on this until we find out more

So on a session from the mixing elbow case, if I perform a force reduction on a user created plane, it will crash the session:
`from ansys.fluent.core.solver.function import reduction
import ansys.fluent.core.quantity as q
Create a Plane and perform a reduction
loc = 2.5 # In Inches
plane_name = 'plane-1'
solver.results.surfaces.plane_surface.create(plane_name)
val = reduction.area_average(expr="TotalPressure",locations=[plane_name], ctxt=solver)
force = reduction.force(locations=[plane_name], ctxt=solver)
`

@prmukherj please see above the instructions from @dpons-ansys

There is no named_expression' attribute in solver.setup`, why???

dir(solver.setup)
['__annotations__',
 '__call__',
 '__class__',
 '__class_getitem__',
 '__delattr__',
 '__dict__',
 '__dir__',
 '__doc__',
 '__eq__',
 '__format__',
 '__ge__',
 '__getattribute__',
 '__gt__',
 '__hash__',
 '__init__',
 '__init_subclass__',
 '__le__',
 '__lt__',
 '__module__',
 '__ne__',
 '__new__',
 '__orig_bases__',
 '__parameters__',
 '__reduce__',
 '__reduce_ex__',
 '__repr__',
 '__setattr__',
 '__sizeof__',
 '__slots__',
 '__str__',
 '__subclasshook__',
 '__weakref__',
 '_flproxy',
 '_get_parent_of_active_child_names',
 '_is_protocol',
 '_name',
 '_parent',
 '_print_state_helper',
 '_setattr',
 '_state_type',
 'boundary_conditions',
 'cell_zone_conditions',
 'child_names',
 'command_names',
 'flproxy',
 'fluent_name',
 'general',
 'get_active_child_names',
 'get_active_command_names',
 'get_active_query_names',
 'get_attr',
 'get_attrs',
 'get_completer_info',
 'get_state',
 'is_active',
 'is_read_only',
 'materials',
 'models',
 'obj_name',
 'parent',
 'path',
 'print_state',
 'query_names',
 'reference_values',
 'set_flproxy',
 'set_state',
 'to_python_keys',
 'to_scheme_keys']

Not able to reproduce the above issue. Could you please provide some more information about the version of fluent and pyfluent that you are using. Thank you.

Not able to reproduce the above issue. Could you please provide some more information about the version of fluent and pyfluent that you are using. Thank you.

Because of the complexity of the solver.field_data in newer version, I always choose 13.X.

Not able to reproduce the above issue. Could you please provide some more information about the version of fluent and pyfluent that you are using. Thank you.

Because of the complexity of the solver.field_data in newer version, I always choose 13.X.

fluent 22r2 and pyfluent 13.x

Could you please explain the complexity with field_data. We will try to address any issues based on your feedback. Thank you.

Could you please explain the complexity with field_data. We will try to address any issues based on your feedback. Thank you.

I have explain in #1681 issue.

Could you please explain the complexity with field_data. We will try to address any issues based on your feedback. Thank you.

The type returned from field_data changed from a dict {id:np.array} into a complexed type Vector. it makes my code have no ability to read the data directly.

@raph-luc thank you for your reply. But, my pyfluent is v0.13.0 and fluent version is 22r2. I choose them because mine programed a few months ago by pyfluent v0.11.0. The program using field_data inherit good from 0.11.0 to 0.13.0, but the return type changed into complex type from 0.14.0. I would not like to change my code. So, I continue use 0.13.0.
I exampled in #1681 and explained the complexity.
About the dir(solver.setup), I don't konw why can not use any function in reduction.py, the python file download by ansys-fluent-core together. I don't think it is fault installation.

Could you please explain the complexity with field_data. We will try to address any issues based on your feedback. Thank you.

reduction.py, the python file, was download by ansys-fluent-core 0.13.0 together. I don't think it is fault installation.

I got it. The named_expressions is not a child_name in setup in settings_222.py but exists in settings_231.py
It means any functions in reduction.py can not work out for fluent 2022r2.

So, can you help me for a string or data flow from fluent console to python?
as follow example:
/report/fluxes> m
All boundary/interior zones [yes] no

Zone(1) [()] 5
Zone(2) [()] 7
Zone(3) [()]
Write to file? [no]

                  Mass Flow Rate               [kg/s]
-------------------------------- --------------------
                         exhaust            11.448219
                           inlet           -11.315761
                ---------------- --------------------
                             Net           0.13245784

def return_data(command):
    ...
    return data

data=return_data(solver.tui.report.surface_intergrals.sum(5,7,(),'px','no))
print(data)

{
 'exhaust'  :11.448219
'inlet':    -11.315761
 'Net':    0.13245784
}

@RebelYoung multiple topics have been added to this issue. We need to keep issues on topic. Thank you.

@prmukherj when you get around to this issue, it would be good to write a test based on @dpons-ansys 's script and see how general the issue is.

@dpons-ansys
@prmukherj cannot reproduce via your script with the current code, an there are some related PyFluent tests that are passing.
We shall close for now. Please let us know if you still experience this issue, and perhaps we can discuss/demo more at the meeting.


[sync screen] Top/Bottom colors are confused by movies's black borders

Hi,
I am reporting this because I think it would be an huge improvement for movies immersion (since it's probably one of the main purpose of the entertainment areas).
Step to reproduce

Set up an entertainment area with the official app
Set a light near/on the screen (lightstrip in may case)
Set the light as "on the floor/under the level of the screen" -ish position
launch a movie 16/9 in a 3/4 screen for example
on screen you can see a bottom and top black borders

Let's pretend the screen show sand/sea/sky at screen. with bottom and top black border
A) current result

the light goes dark (black's border)

B) exepected result

the light should have the same average color than the last movie pixel's color before the bottom border
means it should have the sand's color

C) current way to dodge this sideeffect of black border
(for those who want something working-ish for the while)

set the light at the same level of the TV in the app
but will have the middle screen color
means it will have the sea color in our example

Hi @cm-t , it is a known issue to me and I am not sure how to resolve it yet.
The problem is the limitation of the Gnome shell. Taking a screenshot and analyze it is a very expensive operation. I tried it. It is much cheaper to get just a few pixels and make an average color based on the few pixels, but it is not suitable for detecting the black borders.
My idea here is to add a new sync mode - 'sync selected area'. You could select a rectangle at the beginning of synchronization manually with your mouse. Maybe this could be also activated using some key shortcut.
What do you think?

Ohh, I did not thought about this way
I thought about some options, but they are only ideas, not especially good or bad, just a brainstorm i had and didnt post because i didn't want to pollute the 1st post (because I think issues should explain the need, not the solution; explain the need, then talk about solution⋅s ^^):

Dont scan the presence of black borders on each frame but only each n milliseconds for example (maybe 10 000 by default in settings) so people could adjust regarding their hardware capacity.
Use a whilist such as [firefox, vlc, totem, spotify, netflix, etc] (should be editable) and add a setting to enable the auto-detection of the black borders (select input) such as:

no detection (current behavior)
detection (using the n time settings)
"detection only if the focused app is in the whitelist"  (using the n time settings)

Users with good CPU/GPU might don't care about those optimization, so it might be ok for them to switch off any optimization that could make the experience a little bit less responsive in settings the optimization. (Personally, if i launch a game, i lose all my FPS, i can only use it for movies anyway)
I forgot others idea to be inserted here, sorry, maybe it will comes back to me later ^^

But most of those ideas are not especially needed with your idea to start a screen area-selection sync. I think I really like this idea.
I think I could even fix some wrong behaviors issues with multi screen ? (I didn't tried with multi-screen, so I'm just supposing)
So now, yep, I'm spitted. I know both way of doing (detection or area selection) are not incompatible (without double negative: they are compatible !), but since you are mostly the only one to commit, I guess we should select one of these way; and maybe think  later about the other one, I don't know, more discussion needed i think :)

Thank you for your ideas. I would prefer the autodetection over the whitelist. I believe it would be more universal.
I think detecting the black borders only in a configurable time could work well. The detection stresses the CPU, and due to gnome-shell limitation, there are no threads or parallelism. So people with powerful CPU could detect more often than others. Also, I believe the span of configurable times could be, let's say, from 1s to 30s. The default value of 10s seems reasonable.
It could also prolong the detecting interval after detecting the black borders. Typically, the movie takes more than one hour with the same borders. It could work like this: Once the border is detected, the interval for the next detecting would be the same (10 seconds), then 30 seconds, and finally 1 minute.
Multi-display requires the screen to be a solid rectangle. Otherwise, it will not start (with the notification displayed).
area-selection sync would work on the non-solid (rectangle) screens. That would be a solution for people with various displays.
I think about implementing both solutions, though it could take some time. I am quite busy recently. I would start with the autodetection.

I like the idea of extending the time detection after it has been detected first !
Just a though about the rectangle: before finding the rectangle dimension, the every 10 sec scan could just be on the 1st pixel (top left corner?) and the last pixel (bottom right corner?), and if it is same after the 10 sec, you can check 1 sec (or millisecond) later to confirm, and then finally launch the detection of the size of the top and bottom rectangles to adjust the color sent to the hub's API.
I think scanning 2 pixels cost not so much than detect rectangle at every scan.

Please note that light position is set relative to your screen, not your content.
For this reason if the average color of your top area is almost black (for whatever reason) the lights on that position should be almost off. That's how it is supposed to work and how the official sync app work.
E.g.: If you try to play a 32:9 video on a 16:9 screen there will be huge black borders and the official Philips Hue Sync app will almost turn off the lights on those positions.
There is a reason behind this choice! Imagine a huge screen attached to the wall with some lights around it.
The lights must have a color similar to their position "in real life". This means that light just above the screen should have the same color of the screen top area even if it is black.
Also note that dynamic autodetection is a double-edged sword. What happens if a content (e.g. a movie) has some scenes with some completely black areas? Will the extension suddenly change its configuration making weird glitches?
As stated before, the default behavior of the official app is to match the sync area to the whole screen and I suggest to keep this choice as default in this extension. Any other behavior should be a setting.

Hi @RiccardoMaffei

There is a reason behind this choice! Imagine a huge screen attached to the wall with some lights around it.
The lights must have a color similar to their position "in real life". This means that light just above the screen should have the same color of the screen top area even if it is black.

I am in the case you are mentioning: Hudge screen (video projector) + led stripe (mounted below on the top of the furniture below and facing the wall, so no direct eyes contact but wall reflection, and is set at minimum brightness when using the video projector),  the LED stripe light override the black borders of the video projector beam on the wall and provide the immersion wished. Pictures are better than words, but sadly my attempt to take a decent picture of my setup is a fail since my old phone cam is terribly bad at dark shoot with screen facing the cam...

Also note that dynamic autodetection is a double-edged sword. What happens if a content (e.g. a movie) has some scenes with some completely black areas? Will the extension suddenly change its configuration making weird glitches?

This should be anyway something to be tested; and maybe we can even have a look at other projects (i'm naming one below) to see how they implemented it and handle it well.

Once the border is detected, the interval for the next detecting would be the same (10 seconds), then 30 seconds, and finally 1 minute.

This won't be an issue when the movie gets darker. But, this could maybe add a glitch if the movie stay dark too long, and then resume lighter (means we could wait 60 secondes to have LED lighting again.
If the black border detection expend to the full size, then keep last working size, or if it was the first second of the sync screen mode and no size was found before, then keep scanning for it every second (or every n setting). something like this should be working without glitches. As said before, it should be tested, and we could even see at others projects too (even if they are not gnome-shell extension, what matters here is screen scanning methods) .
NB: my coworker has a DIY led stripe, and the project he uses to sync screen with LED (hyperion.ng) has the option to provide a working well (no glitches) black borders detection without lag issue,  (as far as i remember when talking about it with him). If the feature is requested, and implemented, and used, that's not just "because we can do it", but also because it matches some uses cases.
NBB:  when you already have large hardware screen borders, or not, there is already a gap of the immersion between users anyway, but I understand your call and I can only agree with you: this feature should obviously be an option that users can enable/disable.
NBBB: sorry for my glitchy english

Hi @cm-t,
a projector in a dark room is a completely different scenario than a physical screen.
In the former case, the black borders can be "painted" by the light, in the latter they can't.
Also, things get more complicated when you have several lights (10, 20, 50 etc..)! If you skip the black borders you are "virtually stretching" the image to fill the whole screen (e.g. vertically) and if you have several lights on the stretched dimension (e.g. on the right/left border) the color of the light won't match the actual color of the image on the screen.
Take a look at the following example:
      Actual screen              Stretched screen
+-----------------------+   +-----------------------+
|         Black         |L1 |                       |
+-----------------------+   |          Red          |
|          Red          |L2 +-----------------------+
+-----------------------+   |                       |
|         Green         |L3 |         Green         |
+-----------------------+   |                       |
|          Red          |L4 +-----------------------+
+-----------------------+   |          Red          |
|         Black         |L5 |                       |
+-----------------------+   +-----------------------+

The light color should try to match as closely as possible the average color of the area next to the light (just like the Philips Ambilight technology). The color of that area on the actual screen may be different than the one on the stretched virtual version.
For the aforementioned reasons, in my opinion, the standard (and default) behavior should be to use the whole screen.

Yes, video-projector is a "different" use case than the most people's usecase, but it is still a valid usecase, i mean, i'm not the only one with no tv screen but a projector.
Also, this is my entertainment zone (named "cinema") 's settings:

A: the LED stripe i've mentionned, just below the screen, will  softly pain the wall near the screen
B: One mid screen tall, behing user's head when sit on sofa, will very softly paint wall near the screen
C: One at the roof/top screen, behing user's head when sit on sofa, will very softly paint roof/wall near  the screen

Using full screen color in a movie use case (black border, but no detection of them):

A: switched off
B: middle screen color
C: switched off

immersion: only middle screen color (could be the red of a thirst of a character, wile everything except that on screen is the green from the golf ground); only 1 color.
Using full screen color in a movie use case (black border, but no detection of them):

A: bottom screen color
B: middle screen color
C: top screen color

immersion: more color from the screen will be represented in the room
In most usecase, people have between 1 and 5 light max in a room, and most of them are not close to the TV (usually 1 or a pair) but could paint a part of the well close to it.
The user enabled his lights to the sync experience by adding them manually one after one in the official phone app as a "entertainment area".
In the usecase of TV, having all those light color could be breaking the experience, because your eyes see a, for example 10 cm black hole rectangle between bottom screen color and wall color, right ?

Do i got your point if I say this ?:
Black border detection

prevent you to have less selected light of your entertainment zone than what you actually selected
can break a black border but
provide more colors to the immersion (while prevent monocolor/nocolor)

No black border detection

can force the number of light less than what you selected, even to 1 or  0
might provide less or zero color to the immersion but
doesn't break any dark area of the screen (in your exemple, it wont do redScreen/blackScreen/RedWall at the bottom for eg)

This is probably not an objective point of view, but a subjective one since i focus a lot on my own usecase; but, since we already mentioned it should be an option, not enabled by default, can be enabled manually in the panel settings, do you see any more risk ?:

can have laggyness (perf)

we can solve with: test + check at others projects

can have glitches

we can solve with: test + check at others projects

can have a divergent experience than the desktop official app (such as the black hole between screen's color and LED's color)

we can solve with: not enabled by default, user can switch it on manualy

I understand your point @cm-t . There are a lot of different use cases to  consider.

Thank you @RiccardoMaffei and @cm-t for the discussion.
I am sorry for not being active recently. I was busy. Hopefully, it will be better.
I understand the autodetection could be a problem... Now, I think the 'sync selected area' could be maybe an sutiable option.
Or...
The current algorithm checks only four pixels in the area dedicated for one light and then it determines the average color for the appropriate light. This is because I would like to spare the CPU power. With a powerful CPU, it could check definitely more pixels. The rectangle dedicated to one particular light on the screen is also quite small. I could make it bigger and use more pixels. This could be configurable (both the light rectangle size and the number of checked pixels) so anybody could adjust it for his/her needs and use cases. This could be closer to the original hue app... I have the sync box, but I do not use the original sync app because o do not use Windows or Mac. Not sure how the app works, but the sync box seems to deal with black borders very well.
Your thoughts?
I have 10 lights in my living room and one of the lights is the gradient light strip (it has 7 light zones), which means I have 16 lights for calculating the average color... It is a lot for my old i5 CPU. That is why I try to spare the CPU power.

I have added support for sync screen selection mode. After selecting the mode and starting the sync, you can draw a rectangle on the screen and this rectangle will be synchronized.
I will also improve the sync mode selection. It is a bit uncomfortable now.

Since version 13, you can use a shortcut SUPER+ATL+a during full-screen video playing and select a screen area for syncing, but first, you need to set an entertainment area for the synchronization in the menu.
Do you consider this solution sufficient?

Hi, I think this solution is great for performance, and also allow users to select the right screen to sync (in case this usecase happen)
In case you want to improve the experience by adding an autodetection (or any other method), I beleive it is ok to keep the selection area mode because it just works :)


feat: add node info policies

Signed-off-by: chenk<EMAIL_ADDRESS>Add node info policies
Related: #1112
Close: #1112

duplicate #1123


IPv6 Support

Is there IPv6 support for Mongoose? I couldn't find anything one way or the other on Google or in any of Mongoose's documentation, but from coding myself it doesn't seem like it. For example, when I run mongod --bind_ip [IPv6-enabled host] --ipv6, then run my application, mongoose can't connect to the host, but when I start the server without --ipv6 then run my application, mongoose connects.
If there isn't support, is there a timeframe on when it will be added? This isn't urgent for my application, I'm just worried about the future as more networks become IPv6-only.

So node driver supports ipv6, but mongoose mistakenly occludes this functionality because muri can't parse ipv6 addresses


explore using "shadow" as a simulation backend for warnet

shadow-bitcoin-plugin is a project for doing large scale bitcoin simulations. the repo is unmaintained , but might be worth seeing if we can hook it up as a backend as a feature.
https://github.com/shadow/shadow-plugin-bitcoin
https://shadow.github.io/
why
the tool already does a lot of what we want, like simulating latency, node sizes, network jitter, etc. it allows you to define the a network using networkx (graphml) and has an emphasis on determinism and reproducibility. because of how we've modularized warnet, should be fairly simple to offer this as a backend to the user.
why not just use this instead of docker?
shadow seems to solve one use case: large scale simulations. what it doesn't provide (or its unclear how) is a way to run a long-running setup that the user can interact with. it also doesn't seem easy to hook up live monitoring tools and dashboards. sure, we could, but sticking with docker allows us to use off the shelf stuff like prom/grafana, and seems a better fit for the "lets run stuff for a long time".

I don't think switching to shadow is on the roadmap currently


Add support for Connections

Only shows Accounts, Contacts, Activities at the moment - I'll add support for connections and other entities soon.

Connections are now supported - http://develop1.net/public/post/New-version-of-Network-Visualisations-for-Dynamics-CRM.aspx


Pin pydantic<2

Two issues:

pydantic was not in our environment.yml even though we are using it directly (not just as a transitive dependency)
our code is currently not compatible with the recently released pydantic 2.

This PR hotfixes the dependency by down pinning it. As a follow up, we should migrate our code to be compatible with pydantic 2 in the future.

Will need to port this change also to https://github.com/conda-forge/quetz-feedstock

Codecov Report
Patch and project coverage have no change.

Comparison is base (2f819d4) 83.26% compared to head (77da435) 83.26%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #653   +/-   ##
=======================================
  Coverage   83.26%   83.26%
=======================================
  Files          78       78
  Lines        6186     6186
=======================================
  Hits         5151     5151
  Misses       1035     1035

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update OZ contracts to v0.7.0-rc.0

https://github.com/OpenZeppelin/cairo-contracts/releases/tag/v0.7.0-rc.0

This should probably happen in the devnet

@archseer we have OZ account contracts in the plugin as well https://github.com/0xSpaceShard/starknet-hardhat-plugin/tree/master/contract-artifacts/OpenZeppelinAccount/0.5.1/Account.cairo


Update ReactiveSwift to 1.0.0-alpha.1

@ashfurrow Would you like to rebuild it manually?

Hmm, getting a legitimate failure this time, but I don't understand. Your PR is exactly what I would've done, not sure why it's failing. Keep it open and I'll investigate, thanks again!

Seems like Circle is complaining about RxSwift. I saw error: assigning non-escaping parameter 'selector' to an @escaping closure in this build.

I've notice Moya is ignoring Carthage.resolve and Carthage.parivate. Should we move them back?

That's a good question! @justinmakaila can you chime in on this?

Current coverage is 84.96% (diff: 100%)

Merging #674 into master will not change coverage

@@             master       #674   diff @@
==========================================
  Files            29         29
  Lines          2454       2454
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2085       2085
  Misses          369        369
  Partials          0          0

Powered by Codecov. Last update 25b8072...25e41b3

For a library I don't see the resolved file being necessary since we don't really need to care which exact version we are using. Not having the resolved file means that if we specify ~> 2.2 then when someone builds Moya they will always get the most recent 2.2.x version without us having to run carthage update to update the resolved file.
At least, that is what I understood from reading the description of the resolved file.

Not having the resolved file means that if we specify ~> 2.2 then when someone builds Moya they will always get the most recent 2.2.x version without us having to run carthage update to update the resolved file.

Partly, that is what ~> use for version requirement compatibility in Cartfile, since Carthage's compatibility is determined according to Semantic Versioning.
IMO, Carthage.resolve file is supplementary which describes the versions of Moya's dependencies.
Imagine this scenario: You saw Moya released a new version with patch, and you run Carthage update to bump your project to a new stage. However, some dependency of Moya have new commits. (And this is why we should avoid using github "SOME_DEP" "master/branch" or github "SOME_DEP" "https://github.com/XXX.git" unless you indeed need it.)  You saw errors and ticketed an issue. Unfortunately, we don't exactly know why you are facing those errors, for the reason that our dependencies are totally different.
FYI: cartfileresolved
Correct me if I am wrong. :)

For what it's worth, I think @wongzigii has the right idea. We should probably be using the semantic version of a library instead of (basically) a commit hash (in this case master).
@wongzigii if you update the tag ReactiveSwift's release to alpha 2; confirm that it doesn't break anything in ReactiveMoya's target, and no one from @Moya/contributors has an issue with it, I think it would make sense to merge this in.

Circle CI timeout: pod install took more than 10 minutes 😢

@wongzigii That timeout happens sometimes, I've restarted the build 👍

Just realized we forgot to merge this, I'll deal with conflicts and merge today, thanks again 👍

Thanks a lot for contributing to Moya! I've invited you to join the Moya GitHub organization – no pressure to accept! If you'd like more information on what that means, check out our contributor guidelines and feel free to reach out with any questions.


Pattern match failure on missing binary

Yeah, this is not nice:
Just ipython <- SH.which "ipython"

because it produces this error:
Sep 28 11:52:30 katara bash[28716]: Exception: user error (Pattern match failure in do expression at src/IHaskell/IPython.hs:204:5-16)

Instead of a
binary for ipython not found

or something similar.
Came up in https://github.com/NixOS/nixpkgs/issues/10074#issuecomment-143644326

@Profpatsch Done, thanks for the report :+1:

:+1:


Add config for generated release notes

For #754. See configuring automatically generated release notes.

Codecov Report

Merging #765 (592a312) into master (276719c) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #765      +/-   ##
==========================================
- Coverage   71.20%   71.17%   -0.03%
==========================================
  Files          54       54
  Lines        3528     3528
==========================================
- Hits         2512     2511       -1
- Misses       1016     1017       +1

Impacted Files
Coverage Δ

kube-runtime/src/wait.rs
68.00% <0.00%> (-2.00%)
:arrow_down:

Continue to review full report at Codecov.

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

If we generate a release note right now, it looks like this:

with a list of avatars of mentioned contributors below.
I don't know if we can get rid of "What's Changed" heading.

I think that looks nice. Good highlighting. The What's Changed heading is OK imo.
The problem now is what to do with human written notes. There's something awkward about doing them all at the end manually (since by that point we might have forgotten about key details). I kind of feel we should keep the UNRELEASED part for an explicit Notices heading, and concat that under the release body, and move everything back into the file after release.
We could do the first part by having our release script prepend this part to the body of the release with the softprops gh release action. Not sure about the second part though.


chore(firezone-tunnel/windows): reduce ring buffer from 64 MiB to 1 MiB

Oops. It runs the same either way so we definitely don't need all that RAM to be tied up. The Linux and macOS Clients probably have similar buffer sizes already.
I tested before and after with CloudFlare's speed test and got roughly 140/12 with latency 50 ms both times. The error bars on speed tests are pretty wide, but we definitely aren't falling 60 MiB behind on processing and then catching up.
### Tasks
- [x] (failed, can't do it right now) ~~Log if we knowingly drop a lot of packets~~
- [x] Extract constant
- [x] Add comment about not knowing if we drop packets
- [x] Merge
- [ ] (skipped) Test while the CPU is loaded

Closed and reopened which should re-trigger CI


Fix background color issue after fullscreen

Currently the background color is not working after entering fullscreen.
This code snip should fix the issue.
I only tested it for my Chrome(46.0.2490.86).

Hey @yielder,
Thanks for this - can you post a screenshot of the background color not filling the entire area? I'm on Chrome 46 on OSX and the color seems to take up the whole screen without issue.

To make contrast clear I changed original to red:
before enter fullscreen:

after fullscreen:

My Chrome is also on OSX btw.
Also, I need to manually add code snip like the one I requested for almost all other cleaver themes in the style.css file to make it behave normally in fullscreen mode..

Ah, thank you!
If that's the case, maybe we should instead move the background-color from body { } to .slide { } in default.css?
Thank you very much for pointing this out, and yeah I definitely missed it without changing the color to red first.

Moving the background-color from body { } to .slide { } in default.css also works.
It seems simpler~

Woohoo - thanks again :star2: !


Add jsdelivr badge

Refs

https://github.com/jsdelivr/data.jsdelivr.com#get-package-usage-stats
jsdelivr/data.jsdelivr.com#5

jsDelivr serves monthly (default) stats much faster than other period options, so I choose to support it only, should be simple & enough for most cases.
https://badgen.net/#jsdelivr

jsDelivr serves monthly (default) stats much faster than other period options

This is most likely because we use monthly stats on jsDelivr website so you were hitting Cloudflare cache for most projects. Stats change only once a day so after the first request it should be always fast. I just checked and times for uncached requests are, as expected, proportional to the period, i.e. day is fastest and year slowest (still withing 1 second).

@MartinKolarik Thanks for clarification!


compressed video file not play in VideoView from WebServer

i'm developing application which record the video and save to web server, and since recorded video have too much size i use compression technique SiliCompressor in order to reduce size of original video.
after applying compression technique, video size reduces and i also play that compressed video file into my VideoView and till here working perfectly.
when i upload compressed video file to my live webserver and try to play video from url in VideoView, it's not playing.
However it work in my localhost server well.
can u help me what is problem ?
here is my code to play video from my live web server
VideoView vv = (VideoView)findViewById(R.id.vv);
      vv.setZOrderOnTop(true);

      try {
          MediaController mediacontroller = new MediaController(this);
          mediacontroller.setAnchorView(vv);

          Uri uri = Uri.parse(_video_link);
          vv.setMediaController(mediacontroller);
          vv.setVideoURI(uri);
          vv.seekTo(1);

      } catch (Exception e) {
          Log.e("Error", e.getMessage());
          e.printStackTrace();
      }

      vv.requestFocus();
      vv.setOnPreparedListener(new MediaPlayer.OnPreparedListener() {
          public void onPrepared(MediaPlayer mp) {

              mp.setLooping(true);
              vv.start();

          }
      });
      vv.setOnCompletionListener(new MediaPlayer.OnCompletionListener() {
          @Override
          public void onCompletion(MediaPlayer mp)
          {
              vv.resume();
          }
      });

what are you using firebase or what as your backend

i am using php and MySQL as backend

did you solved or still facing error

what do you want to compress video or image

still im facing error , I'm compressing video

are you compressing image

no , not now because images size is not too much if i compared with video

use this to convert image it is very use full....

but of i want to compress video then what would you recommend me ?

for video

yes for video

wait these types of libraries does not work as you expected I will give you the code....just be calm

ok sure , I'll wait for your response.

ok good !!

yo !!! the file is very big how can I send you

upload the file on Google drive and share the link here or else you can mail me link personally too.

do u haven facebook or email or WhatsApp like thing

yes here i my email
<EMAIL_ADDRESS>mail me on this email id

okkk done I will send you now

are you facing this problem

like your video cannot be played through video url

yes video is not playing from the URL specially when the video is compressed and uploaded to the web server

how did you get that compressed file from compress video()

i use this compression technique (sillicompression)

https://github.com/RudreshJR/VideoCompression/blob/master/app/src/main/java/com/rudresh/videocompression/MainActivity.java
this link helps you see this if there is problem just come here we will talk about that

or
https://github.com/lalongooo/VideoCompressor
this library by telegram

SiliCompressor.with(Context).compressVideo(sourceVideoUri, destionationDirPath);
what should I pass here i am getting error ...he said that I should pass videouri, and file path but see

mediaRecorder.setVideoSource(MediaRecorder.VideoSource.DEFAULT);
mediaRecorder.setAudioSource(MediaRecorder.AudioSource.MIC);
mediaRecorder.setOutputFormat(MediaRecorder.OutputFormat.MPEG_4);
mediaRecorder.setAudioEncoder(MediaRecorder.AudioEncoder.AMR_NB);
mediaRecorder.setVideoEncodingBitRate(690000);
mediaRecorder.setVideoEncoder(MediaRecorder.VideoEncoder.H264);
mediaRecorder.setVideoFrameRate(30);
mediaRecorder.setVideoSize(640, 480); this code will hep to compress even crappy but you can set quality of video

why am I getting this error UrisyntexException() wtf

i already used this https://github.com/lalongooo/VideoCompressor library also but after using any compression technique to compress video file. that compressed video file not stream from the live web server (it work perfectly work in localhost).

Hi @pritesh25, i don't think the problem is with SilliCompressor, because the video plays locally. Please check your file upload method. Also, try to upload the video file without compressing and see if it will play from the web server.

I know video is playing locally but it is not playing when it is uploaded on the live server.

you mean default player as a VideoView ???

Hi @pritesh25, i don't think the problem is with SilliCompressor, because the video plays locally. Please check your file upload method. Also, try to upload the video file without compressing and see if it will play from the web server.

Can you guys please help me? my video is not playing locally after compression
@Tourenathan-G5organisation  @pritesh25

Hi @pritesh25, i don't think the problem is with SilliCompressor, because the video plays locally. Please check your file upload method. Also, try to upload the video file without compressing and see if it will play from the web server.

Can you guys please help me? my video is not playing locally after compression
@Tourenathan-G5organisation @pritesh25

@SaadShafiq  did you resolved that issue?
Even the sample project's compressed video is not playable nither on the device not pc.

Hi,
Did anyone resolve this issue...?.."Video not streaming from web url after compression."

Does anyone have a suggested solution for this ?

private Bitmap decodeFile(File f) {
Bitmap b = null;
//Decode image size
BitmapFactory.Options o = new BitmapFactory.Options();
o.inJustDecodeBounds = true;
FileInputStream fis = null;
try {
fis = new FileInputStream(f);
BitmapFactory.decodeStream(fis, null, o);
fis.close();
} catch (FileNotFoundException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace();
}
int IMAGE_MAX_SIZE = 1024;
int scale = 1;
if (o.outHeight > IMAGE_MAX_SIZE || o.outWidth > IMAGE_MAX_SIZE) {
scale = (int) Math.pow(2, (int) Math.ceil(Math.log(IMAGE_MAX_SIZE /
(double) Math.max(o.outHeight, o.outWidth)) / Math.log(0.5)));
}
//Decode with inSampleSize
BitmapFactory.Options o2 = new BitmapFactory.Options();
o2.inSampleSize = scale;
try {
fis = new FileInputStream(f);
b = BitmapFactory.decodeStream(fis, null, o2);
fis.close();
} catch (FileNotFoundException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace();
}
Log.d(TAG, "Width :" + b.getWidth() + " Height :" + b.getHeight());
try {
FileOutputStream out = new FileOutputStream(thumb_filePath);
b.compress(Bitmap.CompressFormat.JPEG, 50, out);
out.flush();
out.close();
} catch (Exception e) {
    e.printStackTrace();
}
return b;

}

Code working perfectly


Move tweet form down into the 'main column'

Instead of taking up the full width of the page.

Pretty easy now.


Video View Counts

We want to record each view for a video and maintain a view count that can be aggregated into view counts for other models (like Theme) or shown to the user.
There should exist a VideoView model that records request info.
There should exist a views attribute on the Video  that represents the current view count on that video.
There should exist a command that reconciles the view count on videos. This should be registered with the console scheduler so that these counts are updated at a regular interval. This command should also be registered in the kernel so that it can be run on demand.
There should exist a queueable job that creates the VideoView object so that response time for video streams remains unaffected.
Because we may want to record views for other models, we should make this framework extensible through traits / other means.

We are choosing to table forward dev on this effort. To this end, I have created an organization-level project.


Fix redundant this alias in loops

Fixes #120. This turned out to be more involved that I thought (because loop rewriting is initialized after all its children nodes), but the logic should be more sound now.

@adrianheine any thoughts on this one?

Thanks, I will merge this if you squash the commits into one and add the missing semicolon.

@adrianheine thank you for the review! Updated and squashed.

Thank you for your work and patience :)


[Demangler] Fix OldRemangler to cope with Type nodes

The Demangler can wrap types in a Node::Kind::Type node; these need to be handled by following the node's first child pointer.  Failing to do this led to a crash or assertion failure in some cases.
rdar://63678171

@swift-ci Please smoke test

@swift-ci Please smoke test macOS platform

@swift-ci Please smoke test

@swift-ci Please smoke test macOS platform

@swift-ci Please smoke test macOS platform

@swift-ci Please smoke test macOS platform


How to install Jsoncpp/drogon with cpm.cmake ?

I have wasted 3 days trying every method possible, reading all the docs online, but still the cmake crashes...
Heres my cmake list:
# minimum CMake version required
cmake_minimum_required(VERSION 3.15)

# Project name, version and description
project(rdl.server VERSION 1.0 DESCRIPTION "RDL server")

# Create an executable
add_executable(${PROJECT_NAME}
    src/main.cpp
    # your others files...
)

# add dependencies
include(../CPM.cmake)

# CPM packages
CPMAddPackage(
  NAME Ccache.cmake
  GITHUB_REPOSITORY TheLartians/Ccache.cmake
  VERSION 1.2
)

CPMAddPackage(
    NAME Jsoncpp
    URL https://github.com/open-source-parsers/jsoncpp/archive/refs/tags/1.9.5.tar.gz
    VERSION 1.9.5
)

get_target_property(JSON_INC_PATH jsoncpp_lib INTERFACE_INCLUDE_DIRECTORIES)
include_directories(${JSON_INC_PATH})

<EMAIL_ADDRESS>
CPMAddPackage(
    NAME jtv_libpqxx
    VERSION 7.6.0
    URL https://github.com/jtv/libpqxx/archive/refs/tags/7.6.0.zip
)
if(Jsoncpp_ADDED)
  add_library(Jsoncpp INTERFACE IMPORTED)
  target_include_directories(Jsoncpp INTERFACE "${Jsoncpp_SOURCE_DIR}/include")
endif()

# package lock to lock down to specific versions of deps
CPMUsePackageLock(package-lock.cmake)

# Linking libraries
target_link_libraries(${PROJECT_NAME} PUBLIC
    ${Jsoncpp_LIBRARY}
    drogon
)

# Specify includes
target_include_directories(${PROJECT_NAME} PRIVATE
    libs/drogon/include
)

# Set C++ version
set_target_properties(${PROJECT_NAME} PROPERTIES
    CXX_STANDARD 20
    CXX_STANDARD_REQUIRED ON
)

output for cmake
-- CPM: adding package<EMAIL_ADDRESS>(v1.2 at /home/user/.cache/CPM/ccache.cmake/3efab1d354e8959e8f108e5431ec8ee51c613af4)
-- JsonCpp Version: 1.9.5
-- CPM: adding package<EMAIL_ADDRESS>(1.9.5 at /home/user/.cache/CPM/jsoncpp/87455b6dd192e67ab53212caf25789970ff319ff)
-- compiler: GNU
-- Could NOT find c-ares (missing: C-ARES_INCLUDE_DIRS C-ARES_LIBRARIES)
-- Could NOT find GTest (missing: GTEST_LIBRARY GTEST_INCLUDE_DIR GTEST_MAIN_LIBRARY)
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
-- Found std::filesystem
-- use c++17
-- Found UUID: /usr/lib/x86_64-linux-gnu/libuuid.so
-- Brotli found
-- pg inc: /usr/include/postgresql/usr/include/postgresql/13/server
-- libpq inc path:/usr/include/postgresql/usr/include/postgresql/13/server
-- libpq lib:/usr/lib/x86_64-linux-gnu/libpq.so
-- Could NOT find MySQL (missing: MYSQL_LIBRARIES MYSQL_INCLUDE_DIRS)
-- Could NOT find SQLite3 (missing: SQLITE3_LIBRARIES SQLITE3_INCLUDE_DIRS)
-- Could NOT find Hiredis (missing: HIREDIS_LIBRARY HIREDIS_INCLUDE_DIR)
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/cmake.csp
-- view classname:cmake
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/config.csp
-- view classname:config
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/demoMain.csp
-- view classname:demoMain
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/filter_cc.csp
-- view classname:filter_cc
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/filter_h.csp
-- view classname:filter_h
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/gitignore.csp
-- view classname:gitignore
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/model_cc.csp
-- view classname:model_cc
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/model_h.csp
-- view classname:model_h
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/model_json.csp
-- view classname:model_json
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/plugin_cc.csp
-- view classname:plugin_cc
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/plugin_h.csp
-- view classname:plugin_h
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/restful_controller_base_cc.csp
-- view classname:restful_controller_base_cc
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/restful_controller_base_h.csp
-- view classname:restful_controller_base_h
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/restful_controller_cc.csp
-- view classname:restful_controller_cc
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/restful_controller_custom_cc.csp
-- view classname:restful_controller_custom_cc
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/restful_controller_custom_h.csp
-- view classname:restful_controller_custom_h
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/restful_controller_h.csp
-- view classname:restful_controller_h
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/test_cmake.csp
-- view classname:test_cmake
-- cspFile:/home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e/drogon_ctl/templates/test_main.csp
-- view classname:test_main
-- bin:bin
-- Building tests
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
-- CPM: adding package<EMAIL_ADDRESS>(v1.7.4 at /home/user/.cache/CPM/drogon/be2e89a031a10193d100d7b801a1411cccac5a5e)
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Generating config.h
-- Generating config.h - done
-- CPM: adding package<EMAIL_ADDRESS>(7.6.0 at /home/user/.cache/CPM/jtv_libpqxx/54ce1cfd8b83f11cc5340e42503effb760b27844)
-- Configuring done
-- Generating done
-- Build files have been written to: /home/user/discord-list/server/build

make:
[ 12%] Built target trantor
Scanning dependencies of target drogon
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/ConfigLoader.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/FiltersFunction.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpAppFrameworkImpl.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpClientImpl.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpControllersRouter.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpFileImpl.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpRequestImpl.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpRequestParser.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpResponseImpl.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpServer.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/HttpSimpleControllersRouter.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/ListenerManager.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/MultiPart.cc.o
[ 12%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/StaticFileRouter.cc.o
[ 14%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/Utilities.cc.o
[ 14%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/WebSocketClientImpl.cc.o
[ 14%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/WebSocketConnectionImpl.cc.o
[ 14%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/WebsocketControllersRouter.cc.o
[ 16%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/SharedLibManager.cc.o
[ 16%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/orm_lib/src/postgresql_impl/PgConnection.cc.o
[ 16%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/lib/src/RedisClientManagerSkipped.cc.o
[ 18%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/orm_lib/src/DbClient.cc.o
[ 18%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/orm_lib/src/DbClientImpl.cc.o
[ 18%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/orm_lib/src/DbClientLockFree.cc.o
[ 18%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/orm_lib/src/DbConnection.cc.o
[ 20%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/orm_lib/src/SqlBinder.cc.o
[ 20%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/orm_lib/src/TransactionImpl.cc.o
[ 20%] Building CXX object _deps/drogon-build/CMakeFiles/drogon.dir/orm_lib/src/DbClientManager.cc.o
[ 22%] Linking CXX static library ../../lib/libdrogon.a
[ 40%] Built target drogon
[ 42%] Linking CXX executable bin/rdl.server
/usr/bin/ld: cannot find -lJsoncpp_lib
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/rdl.server.dir/build.make:115: bin/rdl.server] Error 1
make[1]: *** [CMakeFiles/Makefile2:628: CMakeFiles/rdl.server.dir/all] Error 2
make: *** [Makefile:171: all] Error 2

At first I tried to use the portion of code you provided to see if it was failures in your CMakeLists. txt, but it wasn't and yet I received a similar error in MSVC 2019, I dug into the code as it seemed strange to me that it would fail to link a headers only library, it failed and failed and I even started to doubt if I was creating a library of only headers correctly, until I decided to copy the headers and move them to the root directory of the project that I had created as an example, link them to the executable and surprise failure with the same error, with this I supposed that it was not an error when creating the library but an error in the definitions of the methods, I reviewed the code and I found the root of the problem: You are trying to link a library as a header-only library when it is not.

Oh thanks!

If it worked correctly and you are satisfied with the solution you can close the issue, thank you very much @renhiyama

Oh ok sure!


Flyway 6.0.8 parser stumbles over – (emdash), unknown char encountered

Which version and edition of Flyway are you using?
Flyway Community Edition 6.0.8 by Redgate
If this is not the latest version, can you reproduce the issue with the latest one as well?
(Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Tried flyway/flyway:6.1.0-alpine, but this seems to contain 6.0.8 as well [sic].
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line (Docker flyway/flyway:6.0.8-alpine)
Which database are you using (type & version)?
PostgreSQL 11.1
Which operating system are you using?
Alpine Linux
What did you do?
(Please include the content causing the issue, any relevant configuration settings, the SQL statement that failed (if relevant) and the command you ran.)
-- ...
INSERT INTO "template"("object_type","locale_id","field","template")
VALUES
(E'foo',16,E'bar',E'foo – bar'),
-- ...

What did you expect to see?
Successful migration.
What did you see instead?
ERROR: Unable to parse statement in /flyway/fixtures/afterMigrate.sql at line 7963 col 1: Unknown char – encountered on line 7981 at column 75

(Column number doesn't match anymore due to sanitization, of course.)

As an aside - when you say "Tried flyway/flyway:6.1.0-alpine, but this seems to contain 6.0.8 as well [sic]." could you elaborate?

Could you confirm the encoding of the migration script, and whether you've set flyway.encoding in config / command line parameters?

Also, are you able to provide more of the script (privately, if necessary) for context? The parser should be recognising that it is processing a string literal and ignore the actual contents, including the em-dash, and it does so for me locally.

As an aside - when you say "Tried flyway/flyway:6.1.0-alpine, but this seems to contain 6.0.8 as well [sic]." could you elaborate?

Running the Docker image tagged 6.1.0, the binary contained in it announces its version with 6.0.8.
docker run -it flyway/flyway:6.1.0-alpine -v
Flyway Community Edition 6.0.8 by Redgate

Thanks - I'll split that off as a separate issue.

Could you confirm the encoding of the migration script, and whether you've set flyway.encoding in config / command line parameters?

The encoding is UTF-8, and none is configured when calling migrate (default is UTF-8 according to docs). Setting it explicitly to "UTF-8" gives the same error. There are UTF-8 characters (letters) in preceding lines which are no problem for the parser.

Also, are you able to provide more of the script (privately, if necessary) for context? The parser should be recognising that it is processing a string literal and ignore the actual contents, including the em-dash, and it does so for me locally if I have a script with just this one line.

The error message stems from here https://github.com/flyway/flyway/blob/master/flyway-core/src/main/java/org/flywaydb/core/internal/parser/Parser.java#L496, and afaics the parser is not ignoring the input at this point yet, but rather emdash (and potentially more chars) as valid input.

That's true - but it should have seen the opening single quote prior to the emdash, and gone into handleStringLiteral() which ignores everything up to the closing single quote. The fact that it didn't suggests it believes it's no longer in a string literal - but I can't see why just yet.

If you're able to produce a redacted script file that produces the error that would be really helpful! (The Docker issue is #2585 and is now resolved)

(Thanks for resolving #2585.)
Your remark about quotes pointed me in the right direction: We apparently have wrongly escaped single quotes in preceding lines (which I left out unfortunately) that are the actual cause for the parser's behavior.
INSERT INTO "template"("object_type","locale_id","field","template")
VALUES
(E'foo',331,E'heading_1',E'What\'s new...?'),
(E'foo',16,E'bar',E'foo – bar'),

Escaping using '' works (the table will have '):
INSERT INTO "template"("object_type","locale_id","field","template")
VALUES
(E'foo',331,E'heading_1',E'What''s new...?'),
(E'foo',16,E'bar',E'foo – bar'),

Thank you very much for your swift support!

Many thanks for letting us know - glad we could help!