[minimp3] Update to 2021-11-30

Updates minimp3 to 2021-11-30

Which triplets are supported/not supported?
Yes

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?
vcpkg x-add-version minimp3

Thanks for the update!


Check and change typos it's -> its

The term it's can be found 171 times in ITK cxx, h, and hxx files. Check which of these instances should be changed to its. From https://github.com/InsightSoftwareConsortium/ITK/pull/2996#discussion_r775281256.

Actually I think it is a bug in the English language! The idea that it's means it is is inconsistent with both Trump's and Biden's presidency! The expression "Biden's presidency" cannot be expanded correctly to "Biden is presidency"! It just means "the presidency of Biden"!
If we just keep using the expression it's things, meaning "the things of it", maybe they will eventually fix the English language  😸

it is a bug in the English language

I totally agree! However, the English language doesn't have a maintainer, much less a GitHub repository 😄

When I went to school they taught me that "its" is like other possessive pronouns (my, his, her, our, your, their) in that it has no apostrophe.  My retort 'What about "one's", "somebody's", "everyone's", "another's", etc?' fell on deaf ears.


SPIKE: Explore Pulling Word Alignment UI from tCore into RCL

Pull Word Alignment UI from translationCore and wrap in RCL
MVP:

[x] remove redux - used in wordmap predictions
[ ] compatibility with newer react and newer apps:

[x] replace react-dnd with html5 drag/drop
[x] remove material UI dependencies for maximum compatibility

[x] wrap converted code in RCL
[x] add styleguidist examples

Future Enhancements:

[ ] add compatibility with alignment APIs:

[ ] compatibility with proskoma edit.

[ ] make headless

[ ] convert components to Hooks
[ ] create custom hooks for each component's business logic

@kintsoogi @PhotoNomad0 Goal this is to shift it to an Epic about integrating this RCL into gE.


✨ Second Meeting 12/14

Today's To-Do

Final API and UI Review

Why the API was designed this way
How APIs and UI interact with each other
Check for missing features and error definitions

Development Initial Setup and Rules

Saving application-dev.yml
Using Docker
Using data.sql
Using Swagger

Development Process Explanation

Explanation of controller, repository, service, and dto
Annotations and classes needed for API documentation using Swagger

Development…👼🏻

Additional UI Notes

Add navigation buttons such as "Go to Home" and "Return to List" on all UI pages.

API Difficulty

To-Do Registration: Medium (requires significant work)
To-Do Retrieval: High
To-Do Update: Medium-Low
To-Do Deletion: Low
Achievement Status Update: Medium-Low
Challenge Registration: Medium
Challenge List Retrieval: Medium
Challenge Details Retrieval: Medium
Daily Challenge Achievement Registration: Medium
Challenge Deletion: Low
Calendar Retrieval: Medium
Calendar Details Retrieval: Medium-Low (requires significant work)
Performance Analysis Retrieval: Medium

Task Assignments
Ji Won

To-Do Retrieval
Performance Analysis Retrieval

Ye Eun

Error Definition for To-Do
To-Do Registration
To-Do Deletion

Seo Ha

To-Do Update
Challenge List Retrieval
Challenge Deletion

So Dam

Achievement Status Update
Challenge Details Retrieval

Ha Jin

Challenge Registration
Daily Challenge Achievement Registration

Min Ji

Error Definition for Calendar
Calendar Retrieval
Calendar Details Retrieval

Tasks for API Development

Controller
DTO
Service
Repository
Define error documentation for each controller method
Attach error documentation annotations to methods

The issue seems to have been caused by a timing inconsistency in the database update process. Moving forward, we should perform additional tests in edge cases to improve stability.


[vcpkg baseline][marisa-trie] Fix conflict file include/Makefile.am with hunspell

Describe the pull request

What does your PR fix?
Fix conflict file include/Makefile.am with hunspell.

/azp run


how to make use of bundles?

I would appreciate any example of how to make use of bunldles/*.
I looked at this, and I don't have an idea to relate to bundles/ng2-translate.js. If I see some examples of using bunldles file in your README.me, it would be greatly appreciated.

It should work the exact same way as for angular 2, you include a script tag with the ng2-translate bundle after the script tag for angular 2 and before your systemjs start

Closing this, and opeing https://github.com/ocombe/ng2-translate/issues/77 to continue the question.


Cartographer2D crashes

Dear Team,
I am using catographer2D on a Kobuki platform with a  Hokuyo URG-04LX-UG01 Laser .
When the robot finishes its first loop, cartographer crashes.
Here's the last Info before crash:
[ INFO] [1520853984.661969578,<PHONE_NUMBER>.346367899]: I0312 12:26:24.000000 24626 constraint_builder_2d.cc:257] Node (0, 398) with 8 points on submap (0, 2) differs by translation 6.57 rotation 0.194 with score 90.0%.
I suppose that I'm not using the correct lua params for my platform.  I saved a rosbag for this issue if you could help me to correct it.
BagFile:
https://www.dropbox.com/s/0exj8mex90yob8r/data-for-mapping-coworking_2018-03-12-11-29-08.bag?dl=0
LuaConfig:
emx.lua.txt
Thanks
MG

Please re-run cartographer_node in a debugger. (The info line is unrelated.)

Here's what gdb returns:
Program received signal SIGSEGV, Segmentation fault.
0x00000000005b2f9f in cartographer_ros::SensorBridge::HandleLaserScan(std::string const&, std::chrono::time_point<cartographer::common::UniversalTimeScaleClock, std::chrono::duration<long, std::ratio<1l, 10000000l> > >, std::string const&, cartographer::sensor::PointCloudWithIntensities const&) ()
(gdb) bt
#0  0x00000000005b2f9f in cartographer_ros::SensorBridge::HandleLaserScan(std::string const&, std::chrono::time_point<cartographer::common::UniversalTimeScaleClock, std::chrono::duration<long, std::ratio<1l, 10000000l> > >, std::string const&, cartographer::sensor::PointCloudWithIntensities const&) ()
#1  0x00000000005b3397 in cartographer_ros::SensorBridge::HandleLaserScanMessage(std::string const&, boost::shared_ptr<sensor_msgs::LaserScan_<std::allocator<void> > const> const&) ()
#2  0x000000000058b3da in cartographer_ros::Node::HandleLaserScanMessage(int, std::string const&, boost::shared_ptr<sensor_msgs::LaserScan_<std::allocator<void> > const> const&)
    ()
#3  0x000000000059912c in boost::detail::function::void_function_obj_invoker1<boost::function<void (boost::shared_ptr<sensor_msgs::LaserScan_<std::allocator<void> > const> const&)>, void, boost::shared_ptr<sensor_msgs::LaserScan_<std::allocator<void> > const> >::invoke(boost::detail::function::function_buffer&, boost::shared_ptr<sensor_msgs::LaserScan_<std::allocator<void> > const>) ()
#4  0x00000000005a8275 in ros::SubscriptionCallbackHelperT<boost::shared_ptr<sensor_msgs::LaserScan_<std::allocator<void> > const> const&, void>::call(ros::SubscriptionCallbackHelperCallParams&) ()
#5  0x00007ffff557a645 in ros::SubscriptionQueue::call() () from /opt/ros/indigo/lib/libroscpp.so
#6  0x00007ffff5534107 in ros::CallbackQueue::callOneCB(ros::CallbackQueue::TLS*) () from /opt/ros/indigo/lib/libroscpp.so
#7  0x00007ffff5534c53 in ros::CallbackQueue::callAvailable(ros::WallDuration) () from /opt/ros/indigo/lib/libroscpp.so
#8  0x00007ffff557d175 in ros::SingleThreadedSpinner::spin(ros::CallbackQueue*) () from /opt/ros/indigo/lib/libroscpp.so
#9  0x00007ffff5564d9b in ros::spin() () from /opt/ros/indigo/lib/libroscpp.so
#10 0x000000000058866b in cartographer_ros::(anonymous namespace)::Run() ()
#11 0x0000000000585bd6 in main ()

Thanks for your quick response. This seems similar to https://github.com/googlecartographer/cartographer_ros/issues/764.
Can you please make this reproducible by creating a fork repository containing a branch of    cartographer_ros containing all the configuration, launch, and URDF files required?

You can find in the rosbag file all topics you need. I deleted the /tf (map -> odom) coming from cartographer_node when I saved the bag. So I attached a new one (output.bag)
link: https://www.dropbox.com/s/oy7wkjkv6jiy8ft/output.bag?dl=0
To run my bag just launch the play.launch. You have to put your directory path in this launch file.
play.launch.txt
emx.launch.txt
emx.lua.txt
I did some experimentation with my robot and I think that the crash happens when the laser is not detecting any obstacle (big space). I am checking that case for now.
Cheers,
MG

Could it be that you're passing a completely empty rangefinder measurement (all rays are out of range, i.e. the resulting point cloud is empty)?

@GMahmoud could you move the check CHECK_LE(points.points.back()[3], 0); in sensor_bridge.cc after the lines if (start_index == end_index) { continue; } and see if this works?

Yeah effectively. When the vector of points of PointCloudWithIntensities is empty, the CHECK_LE(points.points.back()[3], 0); is sending a Segmentation fault.

The problem is solved with your suggestion @gaschler.
Thanks

Closing as solved.


Coveralls

@zemirco could you add this repo project to https://coveralls.io/repos/new? Thanks!

done - https://coveralls.io/r/zemirco/json2csv
Do I have to do anything else?


Separate csv.h into several header files

This is a nice little library, but "header-only" should not mean "one header only", and 1200 lines, while not terrible, is still a lot for just a CSV parser. There are clear cutoff points for separating out several header files; certainly LineReader and CSVReader can have files of their own.
... and then you tie it all together with a csv.h that includes the different header files.

Hi,
thanks for the feedback. However, dropping the library into a project is vastly easier if it is composed of a single header. I see no major benefit in breaking the header into multiple files.
Best Regards
Ben Strasser

Well, I personally don't think dropping 3 files is that much different than dropping 1 file, but it's your call :-)


Type Union Throwing An Error For Optional Property

Bug Report
🔎 Search Terms

Type Mixed with Different Objects
Union with Optional Type
Union with Optional Property

🕗 Version & Regression Information
I currently see this issue running typescript: 4.8.2. I also notice this issue on TS Playground version 4.8.0-beta.
⏯ Playground Link
Playground link with relevant code
💻 Code
////////////////
// Type Setup //
////////////////

type Version = {
  version: string
}

type ComparisonOperator = '>' | '=' | '>=' | '<' | '<='

type NpmOperator = '^' | '~'

type VersionRange =
  | Version &
      ({ comparison: ComparisonOperator, comparisonStuff?: string } | { npm: NpmOperator, npmStuff?: string })

////////////////////
// Broken Example //
////////////////////

function foo(): VersionRange {
  return {
    version: '1.0.0',
    comparison: '=',
  }
}

const bar = foo()
if (bar.comparison) {
  // Do something with `comparison`
} else if (bar.npm) {
  // Do something with `npm`
}

🙁 Actual behavior
There is an error on bar.comparison and bar.npm. While these don't exist on Version, one of them will exist on VersionRange, so I need some way to check for the existence of one of these properties.
If there is a utility or TS syntactic sugar I can add, I would be okay with that. But not even the non-null operator is avoiding these errors.
🙂 Expected behavior
No errors on checks with bar.comparison or bar.npm.

This is working as intended. You can only access properties that are present on all union types. See #50495 and others.

@MartinJohns Thanks for the swift response 💯.
That's a bummer. I'm using some Regex to parse some strings using naming groups. Depending on if there's a Regex match for a comparison operator or an npm operator, I'd like to do different things.
I'll think about how to structure this. Much appreciated.

@MartinJohns beat me to the punch on a 15-minute-old issue?  What is this sorcery?!

@jvanderen1 Discriminated unions usually make this kind of pattern much easier to work with in my experience.
Here is how your code could be rewritten, adding a kind field:
type VersionRange =
  | Version &
      ({ kind: "comparison", comparison: ComparisonOperator, comparisonStuff?: string } | { kind: "npm", npm: NpmOperator, npmStuff?: string })

/////////////////////
// Working Example //
/////////////////////

function foo(): VersionRange {
  return {
    kind: "comparison",
    version: '1.0.0',
    comparison: '=',
  }
}

const bar = foo()
if (bar.kind === "comparison") {
  // Do something with `comparison`
} else if (bar.kind === "npm") {
  // Do something with `npm`
}

Another advantage is that there is no ambiguity anymore if an object would happen to have both a comparison and an npm field (in your original code the behavior would depend on the ordering of the if statements).


Add "System database" section to The Meltano Project doc

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2320
Originally created by @DouweM on 2020-08-28 16:15:44

We need a better place to link to than https://meltano.com/docs/settings.html#database-uri or https://meltano.com/docs/production.html#storing-metadata.

View 3 previous comments from the original issue on GitLab


Add ruby 3.2 test

Coverage: 100.0%. Remained the same when pulling 7fd3ed26dae71a6e05493eff450668b96cade50a on add_ruby_3.2_test into 8637e57de564df2b163bc459119dcae5b9b222de on master.


How to render filesystem images in markdown

In Gatsby, people can put images under a folder and reference the image in markdown.
Demo: https://overreacted.io/preparing-for-tech-talk-part-2-what-why-and-how/
Code: https://github.com/gaearon/overreacted.io/tree/master/src/pages/preparing-for-tech-talk-part-2-what-why-and-how
Is there a way to do so with nextra?

You can put your static files in the /public directory and they will be hosted under the root path (/) by Next.js.
https://nextjs.org/docs/basic-features/static-file-serving


TST: Add test of Arrow roundtrip of boolean values

Test to reproduce #334 incorrect handling of boolean values using Arrow I/O (mostly to see if this also fails on GDAL >= 3.8.1).

GDAL 3.8.3 fixes this and has just been released...

Because misreading boolean values is a bad error, this adds an exception to read_arrow if boolean values are detected in the pyarrow table read from GDAL.  I didn't add an exception to open_arrow because we don't know the contents of the stream it returns.  I'm assuming that given the severity of this bug, it is better to raise an exception and instruct users to avoid the arrow API for GDAL < 3.8.3 than to do something less intrusive like raise a warning that might be ignored by the user.
I used an xfail for the failing test case so that we can detect if this fix gets backported to other GDAL versions (e.g., if there is a 3.7.4 that includes it)

Because misreading boolean values is a bad error, this adds an exception to read_arrow if boolean values are detected in the pyarrow table read from GDAL. I didn't add an exception to open_arrow because we don't know the contents of the stream it returns. I'm assuming that given the severity of this bug, it is better to raise an exception and instruct users to avoid the arrow API for GDAL < 3.8.3 than to do something less intrusive like raise a warning that might be ignored by the user.

Good idea!

I used an xfail for the failing test case so that we can detect if this fix gets backported to other GDAL versions (e.g., if there is a 3.7.4 that includes it)

I'm not sure if the xfail is that interesting as the test testing for the exception for GDAL < 3.8 will fail anyway. It feels like it's a pity that it is not possible to specify a "match" in the xfail decorator. If that would be possible I think the "exception" test could be replaced by that using only the xfail, or am I missing something?

According to the GDAL PR fixing this, this only affects reading GPKG/FlatGeoBuf files. Would we have a way to check for that? (we don't know the driver because that gets inferred while reading, except if we explicitly get that info (read_info returns it?). But maybe we could then only do that when we detect there is a boolean column)

Indeed; I had assumed it applied to all the bool drivers because it was in OGRArrowArrayHelper::SetBoolOn() (even though it did specifically mention only those drivers).
It looks like boolean values roundtripped properly for SQLite / GeoJSON / GML (not able to easily test postgres) - so our exception would be raised when the functionality is just fine.  Will add a fix for this.

Maybe it is because that utility function is only used by drivers that have a custom implementation of GetArrowStream that sidesteps the internal GDAL data model (which are only GPKG and FlatGeoBuf, AFAIK, apart from Parquet/Arrow, but those already are already in arrow format after reading, of course). Other drivers fall back to a default implementation based on the standard APIs to read data (like we also use)


Allows developers to specify the nightly toolchain

The nightly toolchain is used to install/run RLS.
Currently the extension expects only one toolchain to exist.
Developers may want to have multiple nightly toolchains and to specify which one to use.
A new configuration parameter should be added: "rust.rustup.nightlyToolchain".
When the nightly toolchain should be used, but none is chosen, the extension should ask the user to choose.

Hello,
I don't understand, what should I do if I want to use another channel that nightly ?


eigenpy: clean & fix hash

Description of changes
python3Packages.eigenpy: Add doc output and fix changelog url
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

12 packages failed to build:

python311Packages.crocoddyl
python311Packages.example-robot-data
python311Packages.hpp-fcl
python311Packages.hpp-fcl.dev
python311Packages.hpp-fcl.doc
python311Packages.pinocchio
python312Packages.crocoddyl
python312Packages.example-robot-data
python312Packages.hpp-fcl
python312Packages.hpp-fcl.dev
python312Packages.hpp-fcl.doc
python312Packages.pinocchio

6 packages built:

python311Packages.eigenpy
python311Packages.eigenpy.dev
python311Packages.eigenpy.doc
python312Packages.eigenpy
python312Packages.eigenpy.dev
python312Packages.eigenpy.doc

Omg, I forgot to update hash in #303339, sorry for that :(

Result of nixpkgs-review pr 304607 run on aarch64-darwin 1

18 packages built:

python311Packages.crocoddyl
python311Packages.eigenpy
python311Packages.eigenpy.dev
python311Packages.eigenpy.doc
python311Packages.example-robot-data
python311Packages.hpp-fcl
python311Packages.hpp-fcl.dev
python311Packages.hpp-fcl.doc
python311Packages.pinocchio
python312Packages.crocoddyl
python312Packages.eigenpy
python312Packages.eigenpy.dev
python312Packages.eigenpy.doc
python312Packages.example-robot-data
python312Packages.hpp-fcl
python312Packages.hpp-fcl.dev
python312Packages.hpp-fcl.doc
python312Packages.pinocchio

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

18 packages built:

python311Packages.crocoddyl
python311Packages.eigenpy
python311Packages.eigenpy.dev
python311Packages.eigenpy.doc
python311Packages.example-robot-data
python311Packages.hpp-fcl
python311Packages.hpp-fcl.dev
python311Packages.hpp-fcl.doc
python311Packages.pinocchio
python312Packages.crocoddyl
python312Packages.eigenpy
python312Packages.eigenpy.dev
python312Packages.eigenpy.doc
python312Packages.example-robot-data
python312Packages.hpp-fcl
python312Packages.hpp-fcl.dev
python312Packages.hpp-fcl.doc
python312Packages.pinocchio


✨ Add IOC detections for SolarWinds SUNBURST

Add detections for the FireEye SolarWinds SUNBURST backdoor.
https://www.fireeye.com/blog/threat-research/2020/12/evasive-attacker-leverages-solarwinds-supply-chain-compromises-with-sunburst-backdoor.html
Based on the excellent work of @olafhartong - https://gist.github.com/olafhartong/71ffdd4cab4b6acd5cbcd1a0691ff82f

Without documentation I guessed at the correct entity mapping:
  - entityType: FileHash
    fieldMappings:
      - identifier: FileHash
        columnName: FileHashCustomEntity

Current pipeline errors are tables being "unknown" for DeviceEvents , DeviceFileEvents , and DeviceProcessEvents. DeviceNetworkEvents is OK as it already has a definition in .script/tests/KqlvalidationsTests/CustomTables.

Without documentation I guessed at the correct entity mapping:
  - entityType: FileHash
    fieldMappings:
      - identifier: FileHash
        columnName: FileHashCustomEntity

Current pipeline errors are tables being "unknown" for DeviceEvents , DeviceFileEvents , and DeviceProcessEvents. DeviceNetworkEvents is OK as it already has a definition in .script/tests/KqlvalidationsTests/CustomTables.

@pemontto - yes we are looking at this, will update by tomorrow.

Merging for now, can make changes as needed.


Finish README.md

Currently the README.md file is not useful. It's needed to complete it for later use.

Done...


fix reduce test

Not sure if this is the right way to fix this test, but I have a question about cancel(). It seems like cancel() is performing some async operation and does not satisfy the ctx.Done() case until after a receiving some values from the in channel of the Reduce function. Is this true?
I don't see anything documented in the official go docs, but I found this which I think mentions the behavior?
https://github.com/golang/go/issues/33185

Yes, you got the fix right. Sorry I missed your PR.
The issue with the test is that we don't have any control over whether the pipeline has sent some prefix of the input before it detects context cancellation.  Making a really long input increases the chance the pipeline won't complete before detecting the cancelled context, but it's not fool-proof.


svls: 0.2.4 -> 0.2.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/dalance/svls/releases.
meta.description for svls is: "SystemVerilog language server"
meta.homepage for svls is: "https://github.com/dalance/svls"
meta.changelog for svls is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.5 with grep in /nix/store/dlrqvaks1w6yr71kfvs4dcd5aqhx7ris-svls-0.2.5
found 0.2.5 in filename of file in /nix/store/dlrqvaks1w6yr71kfvs4dcd5aqhx7ris-svls-0.2.5

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
svls

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dlrqvaks1w6yr71kfvs4dcd5aqhx7ris-svls-0.2.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A svls https://github.com/r-ryantm/nixpkgs/archive/eaa6bd84b699f3a13cea8016d6b7e49db04b3e45.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dlrqvaks1w6yr71kfvs4dcd5aqhx7ris-svls-0.2.5
ls -la /nix/store/dlrqvaks1w6yr71kfvs4dcd5aqhx7ris-svls-0.2.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

svls

Maintainer pings
cc @trepetti for testing.

nixpkgs-review build went fine on NixOS 22.05 Linux x64.
Upstream changes look minor.
Able to run the executable.


Fix bug with allowed symbols

Fixes dotnet/roslyn#47243
Needs review from Roslyn team.

@Youssef1313 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

⚠ The original documentation was correct here.

@ktoliver I don't have permissions to create a revert here. Can you revert this change?

@sharwell I created a revert PR.

@sharwell Sam Harwell FTE I created a revert PR.

Thanks @Youssef1313. The replacement PR is merged.
@sharwell Although there is a Revert button, the process it kicks off apparently is problematic, and we don't use it, the Microsoft Docs process to revert a merge is to create a replacement PR that contains the version you want to be live.

CC @ghogen


Positioning of the Icon - Light and Dark mode on the navbar section

Adding the icon of light and dark mode to the top could enhance the page and be more user-friendly.
Kindly Assign me this task under GSSOC'24

Hey @ashutosh-rath02  , I'm keen to contribute. Could you assign me this issue under GSSoc 24'?

hi @ashutosh-rath02 , I have experience in developing the above mode toggler from shadcn-ui. I can contribute to the project. Please assign me this issue

It's at correct position for now


MQTT publish subscribe example without TLS.

Describe the feature
Hi, I am looking for a pub/sub example without a TLS feature, Like the plaintext example we have in aws-iot-C sdk.
Use Case
This will be helpful to connect to other brokers also like Mosquittto without TLS.
Proposed Solution
No response
Other Information
No response
Acknowledgements

[X] I may be able to implement this feature request
[X] This feature might incur a breaking change

SDK version used
1.11.136
Environment details (OS name and version, etc.)
Linux

Sorry, but this sdk is no longer accepting any feature requests. Please check out our v2 sdk samples. I don't think we have the sample you are looking for, but you can make a feature request on that repo.


Unexporting RealWorld?

I do not understand why RealWorld should be exported by this library.
If it should, that is very concerning, and it should be documented.

@mgajda: foldl is only exporting the RealWorld type, which is safe

@mgajda: foldl is only exporting the RealWorld type, which is safe

@Gabriel439 But why do we ever need it? The documented interface is IO monad, not RealWorld type.

@Gabriel439 But why do we ever need it? The documented interface is IO monad, not RealWorld type.

@mgajda: We probably don't need it, but I'd like to avoid an unnecessary breaking change unless we have another breaking change pending

@mgajda: We probably don't need it, but I'd like to avoid an unnecessary breaking change unless we have another breaking change pending


chore: add repo link to Cargo.toml of published crates

Fixes #3725

Successfully created backport PR for releases/v0.2:

#3779


Add different options for preserveExifMetaData to preserve varying amount of data

Description
There are scenarios where it is necessary to preserve different levels of Exif data, I would like to propose the following options are added to the preserveExifMetaData setting of processing.config

all (also "true" for upgrade compatibility) - preserves all EXIF data
copyrightOnly - preserves only the Copyright field
minimal - preserves Copyright, DateTime , ExifUserComment, ImageDescription, and all fields relating to the camera, lens, exposure, etc
minimalWithLocation - the same as minimal but including all GPS related fields

It could also be possible that a custom set of fields might be required, this would require a more complex configuration and may not be required frequently.
All fields can be viewed at https://github.com/JimBobSquarePants/ImageProcessor/blob/fef782988e5f39e536a1a303ed320dc061a68134/src/ImageProcessor/Imaging/MetaData/ExifPropertyTag.cs#L64
That's my opening sugesstion, hopefully others can provide a more conclusive list for each category

I like the idea. I've had various need for some exif data in various projects and had similar requests from ImageGen users. There is value in having access to copyright, author, and description and similar 'descriptive' details. And camera settings in some cases. And location/gps info in some cases.
A real problem with 'retain all exif data' is that non-destructive image editing software such as Lightroom embeds all instructions in the exif. Which can easily create 100KB to megabytes of exif data that is useless for the purposes of resizing and is of little or no value after resizing because the pixel locations will be irrelevant after resizing.  Editing information should always be removed unless specifically set to retain it (which would 99.9% of the time be foolish but I guess we can imagine a situation where it would be useful).
The total weight of typical non-editing-data isn't huge. Less than 3-10KB in my experience. Though if you're resizing to a thumbnail size even that could double the size of the image sent to the website visitor.
I'd recommend these options:

None = remove all exif data
Minimal = retain all non-editing data [Default setting]
All = retain all exif data

It gets a bit tricky with ICC profiles but I guess if they are used and supported at resize time then they should be retained no matter what option is chosen. Though system.drawing ain't great with ICC so maybe this isn't a real concern. Worth mentioning in this context though, just in case.
As for a 'custom' option, that would be great because someone will need copyright but not author. They'll want description but not tags. Or to keep all but remove location info.  I could argue for allowing a series of categories to be listed, which would make it more obvious for users who aren't exif experts (and who is?!). To be overly tricky, use plus and minus to simplify adding or removing a few items without listing them all. For example: None+Copyright or Minimal-Location or just list the categories you want such as +Copyright+Description+Author. Categories might include:

Copyright
Description
Author
Tags
Camera
Location
??

Adobe Lightroom has a few settings that might be helpful to the discussion: https://helpx.adobe.com/lightroom/help/metadata-basics-actions.html
Okay, that's a brain dump before I forget it all ;)

This one came up again as part of this twitter convo:
https://twitter.com/thechiefunicorn/status/970294155351126022
And related to adding the post processor to Umbraco as standard:
http://issues.umbraco.org/issue/U4-11051
Would certainly be nice to add. Personally I'd go with copyright and then as few others as we can get away with.

@JimBobSquarePants You got any pointers on where I should be looking to have a go at adding this feature?
Cheers
Matt

If someone can list me out the required copyright EXIF fields then I'll rattle through this.
It's probably best if I do it as it'll only take about an hour of my time and there's stuff stored in System.Drawing's PropertyItem collection that we do not want to accidentally remove.
I'm gonna go with the following suggestion from @drobar  with a slight tweak.

ExifRetentionMode.None = remove all EXIF data
ExifRetentionMode.Minimal = retain all data pertaining to ownership. Copyright etc.
ExifRetentionMode.All = retain all EXIF data

I can't set Minimal to default code-wise as that would be a breaking change though there's nothing to stop Umbraco making it the default in their config.


s3 trigger

hey, is there support to have s3 trigger in the module?

Hey!
There is another module for setting up S3 bucket notifications that will send events to Lambdas/SQS/SNS/etc. See this example - https://github.com/terraform-aws-modules/terraform-aws-s3-bucket/blob/master/examples/notification/main.tf#L105-L119
This module defines Lambda-related resources.


Use subquery to instead In

Sometimes id will be more than 500 that will result in 500.

Codecov Report

Merging #10874 into master will increase coverage by 0.00%.
The diff coverage is 34.61%.

@@           Coverage Diff           @@
##           master   #10874   +/-   ##
=======================================
  Coverage   43.41%   43.42%
=======================================
  Files         593      593
  Lines       83332    83344   +12
=======================================
+ Hits        36178    36190   +12
+ Misses      42660    42659    -1
- Partials     4494     4495    +1

Impacted Files
Coverage Δ

models/issue.go
51.17% <0.00%> (-0.25%)
:arrow_down:

models/issue_label.go
61.60% <0.00%> (-1.45%)
:arrow_down:

routers/api/v1/repo/issue.go
54.89% <75.00%> (+0.38%)
:arrow_up:

modules/git/command.go
86.95% <0.00%> (-2.61%)
:arrow_down:

services/pull/check.go
52.43% <0.00%> (-0.61%)
:arrow_down:

services/pull/pull.go
34.70% <0.00%> (-0.30%)
:arrow_down:

modules/queue/workerpool.go
58.00% <0.00%> (+1.06%)
:arrow_up:

modules/git/repo.go
48.53% <0.00%> (+1.25%)
:arrow_up:

services/pull/temp_repo.go
31.62% <0.00%> (+2.56%)
:arrow_up:

services/pull/patch.go
64.51% <0.00%> (+2.58%)
:arrow_up:

... and 1 more

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 5c3be56...769aa8e. Read the comment docs.


refactor remove type related class to fe-common to reduce java-udf jar size

Proposed changes
Issue Number: close #xxx
Problem summary
java-udf-jar-with-dependencies.jar is almost 400MB, because it depends on fe-core, so it will
contains all jars in fe-core, which is too large.
Actually, java udf jar is only depends on Type related class in fe-core, so I moved these classes
to fe-common, so that java udf jar can only depends on fe-common.
After this change, the size of jar reduce from 358M to 90MB
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

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...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.42 seconds
load time: 484 seconds
storage size:<PHONE_NUMBER>4 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230111060737_clickbench_pr_77394.html

It is a grate job!


Can't get udon to render

Create comet

Sign in with browser

click Okay! to enter Landscape

Load http://localhost:8080/~~/static.udon

Load http://localhost:8080/~~/static.html

Load http://localhost:8080/~~/static/ or http://localhost:8080/~~/static

4 was expected, 5 was.. dunno, 6 was not expected.
Then I tried
dojo> |mount %
dojo> |serve %home

but got the same results.
How do I get the udon parser to run?

I'm able to get plain udon files to render, if I create a web/pages/ dir, and put them in there.
I think some old docs pointed me in this direction, but looking at ren/urb.hoon it looks like that is a special dir that doesn't have the landscape stuff applied to it.

Ah yes, it's on https://urbit.org/docs/learn/arvo/arvo-internals/publishing/ in the /web/pages section.
I tried your example using /web/pages on a new comet but it still didn't work for me (failed the same way as above).

I am having the same issue. Putting a simple udon page under /web/pages doesn't work for me either.

I confirmed plain udon files do render, using @jglukasik's instructions. Note that it did not work without the sem-gar ;> (are those still the names of these runes?) at the beginning of the udon file.

Right, udon files are valid hoon files, without the semgar it tries to
parse it as code.
On Wednesday, 13 February 2019, Matthew Levan<EMAIL_ADDRESS>wrote:

I confirmed plain udon files do render, using @jglukasik
https://github.com/jglukasik's instructions. Note that it did not work
without the sem-gar ;> (are those still the names of these runes?) at the
beginning of the udon file.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/1050#issuecomment-463270433, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhgq83uHVsNGrW_zCA5Bp7yxwp6UFks5vND8sgaJpZM4af-f8
.

Ok. I have also validated that adding the semgar and putting it in /web/pages will get the file to render.
However, the exact same file doesn't render if it's in /web and if I take all the docs and put them in /web/docs on my planet they don't render either.

You must have this directory: /web/static-site. Udon files go in there.

You must have this directory: /web/static-site. Udon files go in there.

This is incorrect on master.
https://github.com/urbit/arvo/blob/master/ren/urb.hoon#L11
For those following at home, ren/urb.hoon is the top level renderer, called when you go to a "bare" (extension-less) url ie, localhost:8080/foo in lieu or localhost:8080/foo.udon
The above ren/urb.hoon has an expression whereby it tries to render any file in /pages as first a hoon that produces an xml node or secondarily as anything that produces xml node (this includes udon). @ohAitch plz feel free to correct this somewhat haphazard explanation.
If you're not seeing informative errors when you got broke udon, you can replace ren/urb.hoon with this to force it to throw when udon machine broke.

::
::::  /hoon/urb/ren
  ::
/?    309
/+  landscape
/=  full-page
  /^  manx
  /,  /web/collections  /;  landscape  /collections-elem/
      /web/landscape    /;  landscape  /!hymn/
      /web/pages        /|  /!hymn/
                                      /hymn/
                                 ==
      /                 /;  landscape  /!hymn/
  ==
full-page

Ok. Thanks for the more complete explanation. And, indeed, if I move docs/ inside /pages it now renders... it just doesn't render correctly with the sidebar, &c. like it renders on urbit.org. Furthermore, it is now at /pages/docs instead of just /docs.
All of which leads me to wonder: What is urbit.org running on? Hopefully its urbit-hosted, though clearly not a the current master + 0.7.2, right?

I've tested this with a Fake Zod and it appears that the landscape app takes over every page in the web except for pages in static. Even .hoon files in web don't render just the landscape page.


Error when handling models with Date properties

I have a Fluent model that has a Date property, which gets mapped to DATETIME in my MySQL database. If I use all() to get an array of my model object I see the data come back, and the dates are represented as epoch time. However, if I use find() to look for a particular object, I get the following error:
MySQL.MySQLError.invalidTypeBound: Field of type int was bound, mismatching the expected type datetime (supports(expecting:):Binding.swift:418:14)“
I haven't tested this using the new beta yet; this error is from alpha 12.

This has been resolved now – thank you!


Module 4 - Store Slack webhook secret securely [Enhancement]

Currently the Slack webhook is being stored as an environment variable within the Lambda function. Consider changing to store in either SSM Parameter Store as a SecureString or within AWS Secrets Manager, and have the Lambda function fetch this secret at run time.

The current plan to is replace this section with AWS ChatBot.  That would solve this and provide the other features of Slack integration via ChatBot.

We are moving the workshop to a new system. This feedback has been noted. Thank you.


Do not use memory in IndexedDB adapter

this limits the space allowed to the system memory. use disk instead.

Giving this more thought, actually I think that it would be useful to replace Buffer types with IDs so that they can be de-referenced. Then this Buffer collection would not be loaded in memory, but instead would be accessed like any other collection. This memory optimizing behavior should be opt-in because it changes how data is accessed.
Also, the IndexedDB adapter should implement LRU like the memory adapter does, otherwise it would result in records being lost but still on disk.

Actually I think this might not be a good idea since it's too complicated and won't matter unless storing lots of data.


utils: Add catch_and_warn.

dvc: Extract ensure_dir_is_tracked to dvc.

Are there more places we can use it?

Codecov Report
Patch coverage: 97.36% and project coverage change: +0.11% :tada:

Comparison is base (8a80256) 88.23% compared to head (1f8b8d4) 88.34%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #673      +/-   ##
==========================================
+ Coverage   88.23%   88.34%   +0.11%
==========================================
  Files          43       43
  Lines        3136     3141       +5
  Branches      272      275       +3
==========================================
+ Hits         2767     2775       +8
+ Misses        329      326       -3
  Partials       40       40

Files Changed
Coverage Δ

src/dvclive/utils.py
82.05% <90.90%> (+0.91%)
:arrow_up:

src/dvclive/dvc.py
97.19% <100.00%> (+2.19%)
:arrow_up:

src/dvclive/live.py
95.29% <100.00%> (+0.36%)
:arrow_up:

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

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

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


CubaUberJarBuilding : polymerProject is not accessible any more

In former version (before 7.0) was it possible to overwrite the frontProject in task buildUberJar by usage of polymerProject
=> cuba-gradle-plugin/src/main/groovy/CubaUberJarBuilding.groovy
The documentation still contains
polymerProject - a Gradle project representing the Polymer UI module. If not defined, the standard polymer-client module is used.
please see at https://doc.cuba-platform.com/manual-7.0/build.gradle_buildUberJar.html
Is it possible to reenable it soon?
Environment

Platform version: 7.0.1
Gradle version: 4.10.3
Operating system: Mac OS X 10.14.3 x86_64

Description of the bug or enhancement

Minimal reproducible example:
task buildUberJar(type: CubaUberJarBuilding) {   polymerProject = frontModule   polymerBuildDir = "../dist/ista-front" }

Expected behavior:
Parameter polymerProject will be used to include frontProject while building UberJar

Actual behavior:
Error message
> Could not set unknown property 'polymerProject' for task ':buildUberJar' of type CubaUberJarBuilding.

Please tell us when a fix could be available? Thx!

Thank you for reporting the problem.
You can use frontProject instead of polymerProject.
We are going to update documentation and release notes for 7.0 according to this changes.

Documentation:
https://github.com/cuba-platform/documentation/commit/1ff189b88bef8a01aa40cb7bb113d4f894891d50
Release notes:

Now it is possible to use frontProject but it has no effect, the build artifacts from the front project will not be part of the UberJar.
> Task :buildUberJar [CubaUberJAR] Copy shared libs for project ':ista-core' [CubaUberJAR] Copy app libs for project ':ista-core' [CubaUberJAR] Copy shared libs for project ':ista-web' [CubaUberJAR] Copy app libs for project ':ista-web' [CubaUberJAR] Pack server libs [CubaUberJAR] Pack libs progress: 0% [CubaUberJAR] Pack libs progress: 20% [CubaUberJAR] Pack libs progress: 40% [CubaUberJAR] Pack libs progress: 60% [CubaUberJAR] Pack libs progress: 80% [CubaUberJAR] Pack libs progress: 100% [CubaUberJAR] Pack app libs for project ':ista-core' [CubaUberJAR] Pack libs progress: 0% [CubaUberJAR] Pack libs progress: 20% [CubaUberJAR] Pack libs progress: 40% [CubaUberJAR] Pack libs progress: 60% [CubaUberJAR] Pack libs progress: 100% [CubaUberJAR] Pack content for project ':ista-core' [CubaUberJAR] Pack app libs for project ':ista-web' [CubaUberJAR] Pack libs progress: 0% [CubaUberJAR] Pack libs progress: 20% [CubaUberJAR] Pack libs progress: 40% [CubaUberJAR] Pack libs progress: 60% [CubaUberJAR] Pack libs progress: 80% [CubaUberJAR] Pack libs progress: 100% [CubaUberJAR] Pack shared libs for project ':ista-web' [CubaUberJAR] Pack libs progress: 0% [CubaUberJAR] Pack libs progress: 20% [CubaUberJAR] Pack libs progress: 40% [CubaUberJAR] Pack libs progress: 60% [CubaUberJAR] Pack libs progress: 80% [CubaUberJAR] Pack libs progress: 100% [CubaUberJAR] Pack content for project ':ista-web' [CubaUberJAR] Pack libs progress: 0% [CubaUberJAR] Pack libs progress: 100%

Thx!


Travis won't report to GitHub anymore

See PRs.
Related https://travis-ci.community/t/fails-to-report-build-status-to-github-again/7880/4

should be fixed, testing with https://github.com/NLog/NLog/pull/3797


Update mapping::lookup so it is a const function

Raises the minimum Rust version to 1.46.0, adds docs for MSRV, and a CI job to ensure focaccia compiles with Rust 1.46.0.

This is blocked on https://github.com/rust-lang/rust/pull/79549 which makes char:: to_ascii_lowercase const fn.

It looks like I came to this same conclusion in https://github.com/artichoke/focaccia/pull/17, but now there is a PR up in upstream Rust, so I'll keep this open.

This will be unblocked once https://github.com/rust-lang/rust/pull/82078 makes it to stable.

The required const fn changes were released in Rust 1.52.0: https://blog.rust-lang.org/2021/05/06/Rust-1.52.0.html#stabilized-apis


Target of URI doesn't exists: 'package:todos_repository/todos_repository.dart'

I made the vanilla_project by flown, but the created project has the error Target URI doesn't exists in the local repositories todos_repository and todos_repository_flutter , flutter_architecture_samples . Though I executed flutter packages get, the error occurs.
But I was able to run the vanilla app.
Why this error occurs? How to fix this error?
My environment

MacBook Pro 2017
macOS High Sierra ver. 10.13.3
IDE: IntelliJ
simulator: iPhone 6s
flutter sdk path: /Applications/flutter
flutter 1.0.0
dart 2.1.0

To reproduce
Firstly, I made the symbolic links of pub and dart to install flown.
$ ln -s /Applicatoin/flutter/bin/cache/dart-sdk/bin/pub /usr/local/bin/pub
$ ln -s /Applicatoin/flutter/bin/cache/dart-sdk/bin/dart /usr/local/bin/dart

Then, I installed flown and I added the path of packages installed via pub in ~/.bashrc.
$ pub global activate flown

export PATH="$PATH":"$HOME/.pub-cache/bin"

I made the project by flown.
flown --arch vanilla --name vanilla_project

I opened this project by Create New Project in IntelliJ, but I got in trouble at this time.
IntelliJ stop at the screen of "Creating Flutter Project".  I solved this trouble by killall dart, along with this issue.
The project was loaded, and I executed Packages get.
Then, it reports the error, but the app can run.
flutter docter -v
[✓] Flutter (Channel beta, v1.0.0, on Mac OS X 10.13.3 17D47, locale ja-JP)
    • Flutter version 1.0.0 at /Applications/flutter
    • Framework revision 5391447fae (3 weeks ago), 2018-11-29 19:41:26 -0800
    • Engine revision 7375a0f414
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.
      You may also want to add it to your PATH environment variable.

[!] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    • ios-deploy 1.9.2
    ✗ ios-deploy out of date (1.9.4 is required). To upgrade with Brew:
        brew upgrade ios-deploy
    • CocoaPods version 1.5.3

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[✓] IntelliJ IDEA Community Edition (version 2018.1.5)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 26.0.2
    • Dart plugin version 181.4892.1

[✓] Connected device (1 available)
    • iPhone 6s • D3C6D104-5270-489B-BABC-B0FC77CA3E06 • ios • iOS 11.4 (simulator)

! Doctor found issues in 3 categories.

Sorry, I solved this issue by restarting IntelliJ.


[UX] Header is not compatible for mobile devices

Take a look at the header, there is a field for improvement (it should be wrapped to the next row):
Current view

Proposal

Merged in https://github.com/socketio/socket.io-admin-ui/commit/62e146709f1b4ceee86b6c9d414d0538b2991833. Thanks a lot :+1:


Custom PasswordGrant error message

how do I define my own PasswordGrant error message?

OAuth has its own format for errors, but for changing the message you would need to override the OAuthExceptionMiddleware

Closing since the thread hasn't been updated for a while.

Create an middleware and call it in ```$middleware`` array like this
public function handle($request, Closure $next)
    {
        try {
            $response = $next($request);
            // Was an exception thrown? If so and available catch in our middleware
            if (isset($response->exception) && $response->exception) {
                throw $response->exception;
            }
            return $response;
        } catch (OAuthException $e) {
            $data = [
//                'error' => $e->errorType,
//                'error_description' => $e->getMessage(),
                'error' => 'Custom Error',
                'error_description' => 'Custom Description',
            ];
            return \Response::json($data, $e->httpStatusCode, $e->getHttpHeaders());
        }
    }


[Multiple choice questions] Make "other" text required

Fixes #2833

Disables the previous/next buttons if the task is required and user has selected other option but hasn't filled the text.

Verified by running the app locally and via unit tests

@gino-m  PTAL?

@gino-m PTAL


Single unit registry

Describe the bug
We use multiple unit registries. scmdata.units initialises a new unit registry while scmdata.ops.prep_for_ops defaults to using the openscm_units registry. This is difference only matters you have defined custom units in one (or the other) unit registry.
Passing ur=scmdata can avert this behaviour.units._unit_registry
Failing Test
@pytest.fixture()
def custom_unit_registry():
    orig_registry = scmdata.units._unit_registry

    ur = ScmUnitRegistry()
    ur.add_standards()
    ur.define("million = 1e6")
    ur.define("thousand = 1e3")

    try:
        scmdata.units._unit_registry = ur
        yield ur
    finally:
        scmdata.units._unit_registry = orig_registry

def test_custom_registry(custom_unit_registry):
    emms = get_single_ts(variable="Emissions|CO2", unit="MtCO2/yr")
    pop = get_single_ts(variable="Population", unit="million population")

    # convert_units works
    res = pop.convert_unit("thousands population")
    assert res.get_unique_meta("unit", True) == "thousands population"
    npt.assert_allclose(res.values / 1000, pop.values)

    with pytest.raises(UndefinedUnitError, match=re.escape("'million'")):
        emms.add(pop, op_cols={})

    # This works
    res = emms.divide(
        pop, op_cols={"variable": "Emissions|CO2 per capita"}, ur=custom_unit_registry
    ).convert_unit("Mt CO2 / yr / population")

    assert res.get_unique_meta("unit", True) == "Mt CO2 / yr / population"

Expected behaviour
A common unit registry is used throughout the package. Clearer docs about where additional units should be added (I might have missed this).
Is there any downside of just using the openscm_units registry instead of instantiating our own?

Each unit registry created adds a second or so of startup time for the interpreter since the creation happens at import time. It doesn't sound like much for long-running processes, but for CLI ops it does make things sluggish.

Another downside of another initialisation of a unit_registry. Adding support for additional units to scmdata requires modifying the "private" scmdata.units._unit_registry attribute.
Perhaps this should be made public again and default to openscm_units.unit_registry. scmdata.units.unit_registry is then used everywhere. Users can then add additional units to scmdata.units._unit_registry (or openscm_units.unit_registry since they are the same by default).  If users wish to diverge from this behaviour,  registry they can do that if they want by overriding scmdata.units._unit_registry with a new registry

Is there any downside of just using the openscm_units registry instead of instantiating our own?

I don't think so.
The proposed solution of just having a common registry throughout sounds good. In packages I've since written, I've just created a standalone unit_registry module which just has a single UNIT_REGISTRY object that can be used throughout so there's a) a single source of truth and b) it's obvious for users what they need to change in order to change the registry


nixos/all-firmware: fix eval

Motivation for this change
The rtlwifi_new-firmware package was renamed to rtw88-firmware in https://github.com/NixOS/nixpkgs/pull/116292.
Things done
Grepped for rtlwifi_new in nixpkgs and found no other occurences, except for aliases.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118129 1


Fix bug in _process_setting call

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (improved-record-ml-event@9430502). Click here to learn what that means.

Additional details and impacted files
@@                     Coverage Diff                     @@
##             improved-record-ml-event     #979   +/-   ##
===========================================================
  Coverage                            ?   81.81%
===========================================================
  Files                               ?      190
  Lines                               ?    19628
  Branches                            ?     3411
===========================================================
  Hits                                ?    16058
  Misses                              ?     2589
  Partials                            ?      981

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


time box SysAdmin access

Thanks for sharing this with the community!
First idea I had when reading about the SysAdmin access feature: Would be nice to be able to time box the access. Basically a just in time SysAdmin access similar to the JIT database access for T2+ environments. The time box would be a configurable value (default: 8 hours). A batch could check when a SysAdmin access was granted and revoke it if the time box has expired.

Thanks for the input, I have heard this request a couple times now and it absolutely makes sense.
I will get this added to the future feature list.

Hi @ameyer505
Thanks for sharing this.
I was trying to use it but getting below error while trying to revoke sys admin access:
"Cannot edit a record in Security role (SecurityUserRole). The corresponding AOS validation failed."
Please help me on this.
Thanks!

@kkbhargava - All tables within D365FO have the ability to have certain business logic performed within the AOS when an action is done, the SecurityUserRole table has numerous checks to ensure the stability of the system. One of the many checks being performed is failing which is where the error message is coming from.
Based on what what the application is doing my guess would be that one of the following is happening:

You are trying to remove the SysAdmin role from initial administrator user (Microsoft stops this to ensure that there is always at least 1 administrator within the account, there is no way around this you will only be able to use this on accounts that are not the initial system administrator).
You are using the out of box Segregation of Duty functionality from MSFT and the user you are utilizing has a combination of roles that causes an SOD conflict, since the SOD analysis occurs during each role assignment it showing that the user is out of compliance. You should either resolve the SOD risks the user has or remove one or more roles causing the conflict.

If you have further issues please be sure open a new issue within GitHub.

@FH-Inway I have implemented a proposal for this. It's based on the SysAdmin logging functionality (It's essentially just another field on the log entry)

These end dates will be picked up by a new batch job.

If it's only the SysAdmin, then this should be the best way to go about it. It also makes it visible in the audit log when a given ADMIN role will be removed


Support for scaling configuration of aurora serverless

Add support for MySQL Servers with an example on how to use scaling configurations
scaling_configuration {
auto_pause = true
max_capacity = 256
min_capacity = 2
seconds_until_auto_pause = 60
}

Yes, please add this. I would like to override the default seconds_until_auto_pause parameter, as the default is too low for me (300s is too eager to shut down for my taste).

@rfvermut added this, and v1.19.0 has been just released.

Reopening this issue as a reminder for myself to make the same feature available for Terraform 0.12 users.

For Terraform 0.12 users - v2.4.0 has been released with this feature available.
Check examples/serverless for serverless usage:
https://github.com/terraform-aws-modules/terraform-aws-rds-aurora/blob/841228bf15985311aa77eb2683e5b680b7285b8a/examples/serverless/main.tf#L19-L46


One spv session per loaded wallet

Create a per-wallet connection to the spv server for account address subscriptions and sync

no longer needed

no longer needed


Serve the app from the gh-pages branch instead of master/docs

I'm submitting a ...  (check one with "x")
[ ] bug report
[x] feature request
[ ] support request
Current behavior:
We are serving the app from the docs folder on the master branch. This is somewhat unstable, since the app immediately breaks if we accidentally push to master without committing the production build first.
Expected behavior:
We should investigate serving the app from the gh-pages branch instead, do make it less susceptible to mistakes.

:tada: This issue has been resolved in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Incremental vs ordinary chain dumps

Historically we've had two formats for chain dumps, a full one (starting with block 0) and an incremental one (starting from any block number). Those two are different at the file format level and C# node usually differentiates between the two by file name, expecting chain.acc for a full dump and chain.X.acc for an incremental one. NeoGo also supports both, but uses an --incremental flag (it treats any file names as equal and can even read the dump from stdin).
https://sync.ngd.network/ contains "official" dumps that can be used by people and we can see chain.acc published there in the "Full Offline Package" section for Legacy networks, yet we have chain.0.acc in the same section for N3 networks. This means that this dump in fact uses an incremental format. This changes nothing for the C# node, but requires using an --incremental flag with NeoGo which adds some additional cognitive overhead for NeoGo users expecting easy "download/restore/run" experience.
It looks like the reason for using this format is #536, C# node can't produce chain.acc any longer, but it still can accept it (generated from NeoGo for example, I'm doing it quite regularly). Which makes the situation rather ambiguous, we have two file formats used in the same field, one of which is semi-supported.
I'd suggest dropping the chain.acc format completely and always using an incremental one irrespective of the file name. This will likely change nothing for the C# node and we can handle this in NeoGo with some transition period as well. The other way to solve this ambiguity is of course just restoring support for chain.acc generation and using this format for full offline packages the way it's done for Legacy.
Refs. nspcc-dev/neo-go#2849.

single format would be my vote

I prefer single format as well. But compatibility issues need to be handled carefully.

I think that it's better to use single format

Single format should be my choice.


👌 IMPROVE: Install via npx

Node.js 8.x and greater ships with npm 5.x and greater, npm 5.x includes a npm wrapper function called npx, this eliminates the need to have to install create-guten-block globally, the latest version can be executed directly with npx create-guten-block, though as the create-guten-block requires the plugin name parameter this becomes npx create-guten-block my-block
The added benefit of this is that it is always up to date and running npm -g update create-guten-block to create the latest version is no longer required.
p.s. The existing screen recording (http://on.ahmda.ws/osOD/c) should be updated to reflect the change of command

Thank you, @ntwb for this PR. Merged.


Update ocis-phoenix to 0.13.0

Agh... tests always fails in a different scenario 😞

conflicts

https://cloud.drone.io/owncloud/ocis/1355/5/7
runsh: Total unexpected passed scenarios throughout the test run:
apiTrashbin/trashbinFilesFolders.feature:136

That is really strange - why would this Phoenix change make a trashbin API test start passing???

I restarted CI.
And made PR #502 just to see if a regular CI run from current master is passing OK.

Passed, and #502 also passed. So that API test can fail randomly :(

@phil-davis Thanks!


Remove calendar info from XIOS test XML file

PR #559 removes most of the XIOS setup from the iodef.xml configuration file and defines it through the API instead.
Getter and setter methods are already implemented for XIOS' calendar, but we should remove them from the XML file, too, as this is unlikely to be the approach used by neXtSIM-DG.

Closed by #574.


[SPARK-6418] Add simple per-stage visualization to the UI [WIP]

Hello!
I am working on adding a graph visualization to the Spark stage page (screenshots for 10, 100, and 1000 task graphs attached). Below are some details about my implementation:
The x-axis is the time axis (with the launch time of the first task as t = 0) and the y-axis is the task # (pretty self-explanatory). At present, I'm limiting the max number of tasks displayed on the graph to 1000 but I intend to add functionality to select the graph task-range during the second iteration.
You can hover over a specific part of the graph to see a tool-tip indicating the task #, phase, and the time taken by that phase (pretty useful when there are a lot of tasks on the graph). Also, the x-axis unit is milliseconds (if the longest task takes less than a second), seconds (if the longest task takes more than a second and less than a minute), and so on for minutes and hours.
I also intend to add shuffle read and write times to the graph (once my midterms are over), and would love to get some suggestions, feedback, and criticism.
SCREENSHOTS
10 tasks:

100 tasks:

1000 tasks:

cc @andrewor14 @pwendell

Can one of the admins verify this patch?

Jenkins, this is ok to test

ok to test

Test build #30610 has started for   PR 5547 at commit 7fac1eb.

Test build #30610 has finished for   PR 5547 at commit 7fac1eb.

This patch fails RAT tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30610/
Test FAILed.

@pwendell talked about this and #2342 a little bit offline. Our feeling is that this is a more elegant representation of task times than #2342, especially when there are many tasks within a stage. One concern I have, however, what happens when you zoom (does it currently support zooming?). It would make little sense to zoom without keeping the axes, but my impression is that implementing this is pretty hard since we're directly using d3.
Bonus: It doesn't have to be part of this patch, but it would really cool if there's a mode where we can align the breakdown of the task times along the vertical axis. Right now you can't really compare the serialization time of the first task with that of the last task, let alone track whether it has grown incrementally over time. Realistically we will implement this separately say for 1.5, but I imagine this bonus feature is gonna be immensely useful.

Thanks a lot for submitting this. It is a cool feature - we'll need to think about whether we like this charting library vs the one in the timeline view PR. I am going to defer to @kayousterhout to give a more thorough review, but I mentioned a few things inline.
I am a bit concerned about the scalability here. I tried a job locally with 1000 tasks and it took more than 10 seconds to generate the graph. It would be good to explore what part takes a long time. I did some quick profiling and it looks like getOffsetHeight in the dimple library was the culprit... that may be tough to improve on.
Also, it might be nice to memoize the rendered graph in case someone opens and closes the tab multiple times.

Given the increasing complexity of the status pages' UI logic, does it make sense to move from manually toggling CSS classes using jQuery, to a modern single page application framework such as Angular, React or Ember?

Also, if you're looking for a reasonable path to panning/zooming, you might want to take a look at http://plottablejs.org/.

Test build #30910 has started for   PR 5547 at commit 5c3a2a6.

Test build #30910 has finished for   PR 5547 at commit 5c3a2a6.

This patch fails RAT tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30910/
Test FAILed.

Thank you all for your feedback, and I apologize for my late reply (it’s been a rough week of midterms).
@pwendell - I’ve addressed all your inline comments (memoization, Javascript indentation, JSON lists, etc.) in my latest commit. As per the load time of the graph, it’s improved a bit after moving from string representations to JSON arrays, but only by a small factor.
When you say you’re skeptical about the graph scalability, what is the maximum number of tasks you want displayed on the graph? I’m thinking of keeping it to 1000 (at the most), and having the users select a task range if they want to view a different region of tasks (say tasks 1200-2000 for example).
My reason for the above is that the task stages become too cluttered above a certain number, so it’s better to keep a limit, or alternatively, increase the max height of the graph (which would involve a lot more scrolling though).
@andrewor14  - The visualization doesn’t currently support zooming, and it will definitely be pretty challenging to implement it on top of D3.js. However, the task-range functionality I mentioned above can serve as a pseudo-zoom feature since a user can select a task range and hence zoom into the graph.
Also, breaking down the task times along the vertical axis shouldn’t be that difficult so we can definitely add that later on if required (provided this patch gets accepted haha).
@punya - I haven’t looked into using Amber yet, and I’ll definitely check out plottable.js.

Can one of the admins verify this patch?

After some further research, I have come to the conclusion that if we want to have over a 1000 tasks displayed on the graph, d3.js or libraries on top of it aren't the best choice since rendering that many SVG elements is bound to be slow above a certain limit.

If there are over 1000 tasks, it seems like it would be more valuable to see

statistical information about all the tasks
details about a smaller subset chosen in some way (like a drill down view)
Rendering 10k tasks is possible using a canvas, but it's unclear to me what
a user would do with that much density of information.

Punya
On Mon, Apr 27, 2015 at 6:33 PM Pradyumn Shroff<EMAIL_ADDRESS>wrote:

After some further research, I have come to the conclusion that if we want
to have over a 1000 tasks displayed on the graph, d3.js or libraries on top
of it aren't the best choice since rendering that many SVG elements is
bound to be slow above a certain limit.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/5547#issuecomment-96839375.

That's exactly my viewpoint but to be very honest, I haven't used Spark much so I'm not sure what the average use case is.

The Spark administrators have to decided to go forward with #2342 so I'm closing this pull-request.


BigQuery: Don't use selected_fields for listing query result rows

fix for #4864

@VikramTiwari Thanks for the contribution! I've added a commit to the PR with the system tests I was using to investigate the issue and confirm that this does fix the issue.

Looks like a couple unit tests are failing due to some assumptions about where the schema would be available. I'll look into that.

@tseaver Since I authored the tests for this change would you or someone else on the Python team mind taking a look before we merge this?
I've verified locally that all system tests do pass with this change.

Thanks again @VikramTiwari !


Added support for xpath search element strategy in webdriverBridge

Hi there! Enjoying the library so far, webdriver.io integration in particular. However, would be cool to have the xPath selector strategy supported.

Hey @shkaper! Nice to hear that it's still of use and thanks for your PR :)
XPath support is a useful gimmick, but I have some suggestions about the PR.

The feature should be mentioned somewhere in the README
Not sure if the heuristic deciding if it's a CSS or XPath selector could cause trouble, esp. since it's an implicit behavior you might not expect as a user

I'd like to use XPath queries in a rather explicit way. What do you think about something like that?
const css = dragMock.selectors.css
const xpath = dragMock.selectors.xpath

webdriver
  .dragStart(xpath('/some/xpath'))
  .dragOver(css('.some-element'))
  .drop('.some-other-element')    // defaults to using `dragMock.selectors.css`

Maybe css/xpath returning an object like { type: 'xpath', query: '...' } or something like that. I would even favor making that an instance of some XPathQuery, that has a function apply (Document) => HTMLElement on the prototype, but that won't work with the serialization of the code, I guess...

@andywer , thanks for the quick reply!

Not sure if the heuristic deciding if it's a CSS or XPath selector could cause trouble, esp. since it's an implicit behavior you might not expect as a user

I'm all for being explicit, in most cases. However, as a user of webdriver.io myself, I could argue that this implicitness is exactly what users would expect from a webdriver.io-compatible method. In contrast to e.g. webdriverJS (driver.findElement(By.id('myId'))), webdriver.io expects a plain string (driver.element('#myId')) and defines querying strategy itself.
Here's an example of how making users do otherwise could be cumbersome. In our team, we write cucumber-style tests wrapping some business logic into parametrized gherkin step definitions, e.g.
this.When(/^I click "([^"]*)"$/, function (element, next) {
  const query = this.page[element]
  driver
    .waitFor(query)
    .click(query)
    .call(next)
})

So, in this case, we don't even know if the query is css or xpath. To preserve the flexibility of these kinds of steps, while also using drag-mock, we'll need to implement the same strategy-defining heuristics on our end.

The feature should be mentioned somewhere in the README

Sure, I'll add some info about it.

Good point! Didn't recognize that webdriver supports it like that in the first place...
Then let's stick with the approach, but I'd like to modify the code a bit. Doesn't seem too elegant yet (reading it the first time was quite hard).
If you would like to change it, go on, otherwise I will change it if you don't mind :)

I've cleaned up somewhat, that's about as elegant as I can make it :)
Feel free to change any parts you don't like.
Thanks!

Yeah, the code now looks way friendlier :)
Unfortunately it is quite hard to unit-test this and I never bothered putting much effort into it. Did you test it, does it work for you without problems? (God, this is bad style! 🙈😅)

Yeah, sure, it works just fine for me.
Would it make sense to add some tests for the webdriver integration? This would mean adding stuff like phantomjs/http server/webdriver.io to the dependencies of course

@shkaper Feel free if you want to do it, but since I don't actively use it (wrote it for a project I am not working on anymore) I guess I won't 🙃😉

Published as v1.4.0

Ok then, I'll look into adding the tests as well.
Thanks for accepting PR!


build: Create Hanoi release compose files

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/developer-scripts/blob/master/.github/Contributing.md.
What is the current behavior?
No hanoi release in developer scripts
Issue Number: #342
What is the new behavior?
Now have hanoi release
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

NOTE: this is draft PR in preparation for Hanoi release. Image referenced have not yet been created.

EDGEX_SECURITY_SECRET_STORE is true for some services and false for others in the secure configuration.  What is going on there?

EDGEX_SECURITY_SECRET_STORE is true for some services and false for others in the secure configuration. What is going on there?

The services not included in the add-security.yml (i.e. don't need secret store) will still have it set to false. It has to be set to false in the base compose file for non-secure.


Loader created and added to project

Description
in this PR i have added loader to project and solved the prettier problem
Fixes #33
How to test?
it tested it in storybook http://localhost:3007/?path=/story/loader--loader
Checklist

[ x] I have performed a self-review of my own code
[ x] I have commented my code, particularly in hard-to-understand areas
[ x] I have made corresponding changes to the documentation
[ x] This PR is ready to be merged and not breaking any other functionality

closing PR to modify it


Fix deserialization for PutSynonymResponse

The ReloadResult should be a member of the PutResponseSynonym, instead of the ReloadDetails, so that the PutResponseSynonym is properly serialized.
Current behavior is that it throws a MissingRequiredPropertyException
Closes https://github.com/elastic/elasticsearch-java/issues/784
Closes #2578

Thanks @GeorgeCiachir for this. Unfortunately, our CI is not setup to accept external contributions due to security concerns, so I opened https://github.com/elastic/elasticsearch-specification/pull/2578 to get validation running. Which fixed validation here, so now this can be reviewed.

Thanks @GeorgeCiachir for this. Unfortunately, our CI is not setup to accept external contributions due to security concerns, so I opened #2578 to get validation running. Which fixed validation here, so now this can be reviewed.

Thanks, @pquentin
I was wondering what's up with the validation because I also wanted to validate from my local, but I got a 404 on https://github.com/elastic/clients-flight-recorder.git.
Maybe the Readme should also be updated and specify the validation part can only be performed by the Elastic team members. I mean here: https://github.com/elastic/elasticsearch-specification/tree/main?tab=readme-ov-file#how-to-validate-the-specification


fix(util): check stack exists before filtering the stack trace

closes #3224

There's a deeper problem here we'd like to address - doing this just hides the stack trace. Ideally, we'd get the stack from where the exception was first thrown, even if it wasn't created with Error(), though we're not sure how to do that yet. But this does fix the immediate problem, so I'll merge it in.


Unsoundness in World::query() and World::par_query()

There is potential unsoundness in the query methods of World. If the provided Views contains both a mutable reference to a component and a second (mutable or immutable) reference to the same component, the returned result will both mutably and immutably (or possibly mutably again) reference that component on each entity, resulting in undefined behavior.
This is a tricky one to fix, although I have an idea. We can determine whether the view is valid at compile time, since the view is static. If we wrap the Views in a wrapper struct, we can associate a const with the struct whose evaluation panics if the invariant is not met, similar to what is described in this reddit post. The static_assertions crate may also be useful for this, though I haven't looked too closely. The one trick is that the view isn't actually instantiated, its just passed as a generic, and so the constant needs to be used at some point to actually be compiled. Tests will need to be included to ensure that attempting to break Rust's mutable referencing rules doesn't compile.

Unfortunately, it doesn't seem this check is actually doable at compile-time with heterogeneous lists currently. At the very least we need const_type_id to be stable, and likely const_trait_impl along with const_fn_trait_bound. There just isn't a way to do the check generically otherwise.
For now, a reused HashSet can be stored on the World, or perhaps an archetype::Identifier could be constructed, that keeps track of which components are already borrowed. If we reuse the buffer, the reallocation is avoided and this should hopefully not be too big of a hit on performance.
After that is implemented, a new issue should be opened to track progress on the blockers for doing the check at compile-time. That's the ideal way, but the current state of things makes it feel like it's a long ways off, possibly even multiple years.

If the const trait RFC is accepted and the above features are stabilized (or something similar is accepted and stabilized, given that the current RFC is not currently accepted), the compile-time check could be implemented using a variation of this hacked-together code. Note that I couldn't, for whatever reason, get the const_type_id feature to work, and I had to use the unstable intrinsic instead, but the result is the same: duplicate members of a heterogeneous list can be detected at compile time with those unstable features. This can be adapted to instead check for non-unique immutable borrows of Components within Views at compile-time.
As was stated above, this isn't super feasible until we can use const fn in traits, and we can't currently do that because this library is intended to remain stable. There is some possibility of making an unstable nightly feature-gate that allows the use of this compile-time check, but I worry it will break often and become a headache due to the reliance on multiple unstable features. For now, the above playground link is left for future reference whenever it becomes feasible to implement this check at compile time.


WIP: fix package build

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against b341d150efb6ec4e51dc85b87fbfff7062091192

Hmm. Whatever it was that caused all the builds to fail earlier appears to be transient. Re-running the builds now has the tests passing.


Resest two-factor authentication

Hi, I am have a problem. One of users of my selfhosted portal lost phone and he did not print backup codes.
I would like to reset two-factor authentication for him or for all users. I tried reset password for him but after opening link generated by portal there is screen with two-factor authentication. After this I turned off and turn on two-factor authentication. But this does not reset
two-factor authentication.
I need Your help i don't want to delete this user account and create new.

Hello @mastipgithub. You can reset the 2FA application for a user in their profile, in the "People" module.

My mistake, I haven't seen it


How about the vscode-langauge?

Hey there. I'm Pouya and for my use I made IDF Support for VSCode. Today @MatthewSteen mentioned this and I was thinking to bring your grammar to my extension to have a consistent and better look. But then I was thinking that doing so will be unfair since it is your source code there. So how about a merger of the two? If you want to I'm happy to have the repo here and based on the grammar of your project so it still has the Big Ladder sign on it, otherwise I will ship the codes inside the previous extension with my own branding since it seems the source code is in MIT licensing.

@pouyakary that sounds like a great idea! Let's discuss the details over email and share the final consensus here on Github for everyone's reference. Feel free to contact me at<EMAIL_ADDRESS>

FWIW - I abandoned Atom several years agao for VSCode mainly for the integrated terminal so I can write, run, and commit code all from a single app. Plus, for Windows users, the WSL integration is great for developing.


edit-point movement commands conflict with new core subword-boundary movement commands

As of Atom 1.0.1, ctl-alt-left and ctrl-alt-right move the cursor to the previous/next subword boundary. Emmet should probably change the keybinding so as not to conflict and override a useful core keybinding.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/23958780-edit-point-movement-commands-conflict-with-new-core-subword-boundary-movement-commands?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F535900&utm_medium=issues&utm_source=github).

:+1: I agree!
Especially on OS X where there's a duplicate binding for emmet:next-edit-point & emmet:prev-edit-point to ctrl-left & ctrl-right as well as ctrl-alt-left & ctrl-alt-right.

https://github.com/sergeche/emmet-sublime/blob/master/CONTRIBUTING.md


Taunt The Guards 's' and 'p' keys disabled sometimes

We received a report detailing this bug.
"We only ran into one bug, one you probably know about already - at level 4 the S and P keys would suddenly not work. Copying/pasting those letters still worked. This was only for some students, unsure of which browsers, using PC's. "
I couldn't replicate, but I remember hearing about this bug before.

More likely related to https://github.com/codecombat/codecombat/blob/master/app/core/application.coffee#L16-L23


TypeError while evaluating PTv3 on S3DIS

export PYTHONPATH=./

python -u tools/test.py \
  --config-file "./configs/s3dis/semseg-pt-v3m1-0-rpe.py" \
  --num-gpus "1" \
  --options save_path="./exp" weight=model_best_2.pth

Gives this error:
Traceback (most recent call last):
  File "/workspace/Pointcept/tools/test.py", line 38, in <module>
    main()
  File "/workspace/Pointcept/tools/test.py", line 27, in main
    launch(
  File "/workspace/Pointcept/pointcept/engines/launch.py", line 89, in launch
    main_func(*cfg)
  File "/workspace/Pointcept/tools/test.py", line 20, in main_worker
    tester.test()
  File "/workspace/Pointcept/pointcept/engines/test.py", line 315, in test
    allAcc = sum(intersection) / (sum(target) + 1e-10)
TypeError: 'numpy.float64' object is not iterable

I'm using Pointcept v1.5

https://github.com/Pointcept/Pointcept/blob/main/pointcept/engines/test.py#L315
The code you are using doesn't match the one in our repo. Please make sure you are using the latest version.

ah, my bad, i was using the wrong config file.
pretty silly

@tanmay4269 i'm facing the same error on v1.5.1 (which I understand we must use with published huggingface weights). Could you tell me how you solved the issue and what pre-processed dataset you changed?

Nvm solved it, I was too using the wrong pre-processed dataset (accidentally took the data from main rather than v1.5.1). Thanks for pointing it out previously

We will soon have an update on the code and these weights. A super powerful self-supervised model weight will also be included.


Detect sharing errors

In a perfect world, the code below would throw an exception.
class Request {}

class Bar {
    function __construct(Request $request) {
        $this->request = $request;
    }
}

$injector = new Auryn\Injector();

$injector->share(Bar::class);
$injector->make(Bar::class);

Because the Bar class is shared, but the Request class is not shared, it is highly likely that this is going to be a bug.
I'm guessing this might be difficult to do in a performant way during the actual usage of Auryn. I might investigate having a 'validate' method for the injector that checks the above doesn't happen and there is also no other errors like delegate functions that depend on interfaces, which Auryn doesn't know how to create.

I think this would probably need to be an optional setting if implemented. I'm not really sure it's necessary TBH. I have never once encountered a bug from this (as far as I know).

I don't think this should error. Usually you want to have the shared instance to have those dependencies, but if you create another thing with the same dependency, you should get a new Request. If we require Request to be shared here, how would that work?

Closing due to inactivity, not being a problem that needs solving in the first place.


Spinner

Tiles can take some time to be generated/sent to the browser. We should have a spinner on the map for that.

Done in https://github.com/davidbrochart/xarray_leaflet/commit/66ba7d622ca4ed0fc0da1c9ab9372aa701dab6c7.

To be more precise, the solution implemented displays a spinner while tiles are being generated, but not until they are received by the browser (it doesn't count the serving part).


Add support for Functions

CheckedConsumer
CheckedPredicate
CheckedRunnable
CheckedFunction0 - 8
Function0 - 8

Since functional interfaces are supported by jqwik out of the box, what additional functionality could there be?
For example:
@Property
void testCheckedPredicate(@ForAll CheckedPredicate<String> predicate) throws Throwable {
	assertThat(predicate.test("hallo"), equalTo(predicate.test("hallo")));
	Statistics.collect(predicate.test("hallo"));
}

will succeed and output something like:
[Experiments:testCheckedPredicate] (1000) statistics =
    true  (501) : 50 %
    false (499) : 50 %

I will have a look at this

@jlink What result do you expect? If I change it to java.util.function.Predicate. There are nearly the same result (about 50:50).


Workflow cleanup and removed duplicate steps

Codecov Report
Patch and project coverage have no change.

Comparison is base (9bdc5f4) 77.21% compared to head (f721950) 77.21%.
Report is 9 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #74   +/-   ##
=======================================
  Coverage   77.21%   77.21%
=======================================
  Files          13       13
  Lines        1027     1027
=======================================
  Hits          793      793
  Misses        184      184
  Partials       50       50

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

The changes are already implemented in other PR


cpu

sorry mate，is there any way that i am able to work this project under the cpu？

This repo is not very friendly to non-cuda user.
You can manually substitute all the ".cuda()" to ".cpu()". And clean up the training script could probably get you started.


ci: bump version after release

Closes #315.

Codecov Report

Merging #316 (8902f03) into main (6cc76bd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #316   +/-   ##
=======================================
  Coverage   76.80%   76.80%
=======================================
  Files          28       28
  Lines        1953     1953
=======================================
  Hits         1500     1500
  Misses        453      453

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 6cc76bd...8902f03. Read the comment docs.

:tada: This PR is included in version 5.7.0-beta.1 :tada:
The release is available on:

v5.7.0-beta.1
GitHub release

Your semantic-release bot :package::rocket:

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

v5.7.0
GitHub release

Your semantic-release bot :package::rocket:


installing may break display settings

i have an ultrawide monitor.
i have my nvidia drivers installed, but when i made the install, the resolution became terrible and insanely low (1024x768). and now i cant find a solution to reverse the problem.
xrandr cannot find my monitor either, it is seen as "unknown display".

Hi @petebryan04 ,
thanks for reporting this issue.
Does this problem only occur when you start the i3-cinnamon session?
How have you installed this package? Arch AUR?
This package shouldn't make any changes to your system besides adding the session, so you should be able to just start a different session or uninstall this package as a last resort.

i was unable to boot into the computer at all. what happened was the screen went "bright" black, even though my computer was on. so i was not able to switch sessions to begin with.

This doesn't really sound like it could be caused by this package.
Only guess I have right now, is that maybe your Display Manager has issues with the xdesktop file https://github.com/jthomaschewski/i3-cinnamon/blob/master/session/i3-cinnamon-xsession.desktop
What Display Manager do you use? GDM?
Which Ubuntu version are you on?
And what are the steps you executed on your machine?
Guess:

Git clone & make install
Reboot?
Black screen - no login window shown?

Is this correct?
It would be helpful if you could provide more information. I would then try to reproduce your issue.

Yes your guess is correct. I just git cloned and made the install. I then proceeded to reboot, no login window was shown. Just a bright black screen.
Im on ubuntu 22.10. I use the default display manager for ubuntu cinnamon.
I also have a 3440x1440 single monitor connected.

Yes your guess is correct. I just git cloned and made the install. I then proceeded to reboot, no login window was shown. Just a bright black screen.
Im on ubuntu 22.10. I use the default display manager for ubuntu cinnamon.
I also have a 3440x1440 single monitor connected.