When adding new columns to a table add an option to sort table columns in the order of the underlying data source

Is your feature request related to a problem? Please describe.
When adding columns by default they get added to the end of the list of table columns.
Describe the solution you'd like
I'd like the option to reorder the columns in the model as they are in the underlying data source and optionally allow for alphabetical sorting as well.
Describe alternatives you've considered
Manually editing the underlying JSON
Additional context
This is similar to how the SSAS MD replace table worked

This is correct. however I find that it easier to work with the model columns in the order defined in my tables / views: Pk, SK's, Natural Keys and attributes and then measures.

While I don't have any plans to allow reordering through the UI, you can certainly edit the JSON manually (through a text editor). You can already toggle between alphabetical/metadata ordering within Tabular Editor by pressing CTLR+F2.


warning webpack-dev-server ><EMAIL_ADDRESS>Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies

Installing webpack-dev-server I encountered this warning:
warning webpack-dev-server ><EMAIL_ADDRESS>Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies
O.S.: Ubuntu 18.04.4 Desktop
node: v14.5.0

@raphael10-collab:

This is an issue that should be filed in the webpack-dev-server repo.
In the future, please add more relevant info to the issue, such as your version of webpack and webpack-dev-server.
This warning has already been resolved on the master branch of webpack-dev-server and will be included in the upcoming v4 release of webpack-dev-server (see<EMAIL_ADDRESS>

Ok . Thank you @joealden for telling me that this warning has already been resolved and will be included in the upcoming v4 release of webpack-dev-server


Any way to log sneakers logs remotely?

Hi,
Apart from STDOUT and file based logging, is there any in-built way with which we can log remotely eg, papertrail. For production use.
If there's no way, Could anyone suggest an already used method.
Regards.

Not an expert in sneakers, but:
Sneakers.logger = RemoteSyslogLogger.new(...)
https://github.com/papertrail/remote_syslog_logger

Thanks @brodock Have implemented it in similar way.

@brodock, just curious where you assigned the Sneakers.logger relative to the Sneakers.configure?

@krsyoung looks like the "correct way" is with Sneakers.configure: https://github.com/jondot/sneakers/blob/41883dd0df8b360c8d6e2f29101c960d5650f711/spec/sneakers/runner_spec.rb#L35

(closed primarily because it's a question, although @brodock likely answered it)


AttributeList parser misses all properties

Describe the bug
When parsing graphs or subgraphs with node, edge or graph attribute lists, all resulting attribute lists are empty
To Reproduce
Steps to reproduce the behavior:
const model = fromDot('digraph { edge [ style="invis" ]; a -> b }');
console.log(model.attributes.edge); // AttributeList will be empty;

Expected behavior
console.log(model.attributes.edge); // AttributeList should contain style = invis

Additional context
It seems like the issue is here:
https://github.com/ts-graphviz/ts-graphviz/blob/c1e4c50a9eaca7f5a511eca3d520ae9a02271844/src/ast/model-shim/to-model/plugins/utils/apply-statments.ts#L45-L55
As stmt.kind will be in lower-case and not match the proper switch case.

@all-contributors please add @diegoquinteiro for bug.

I fixed this issue and release patch as v1.3.4.
Thanks!

Works like a charm! Thank you so much =)


Change BeepBop upstream branch to fix compilation problem

Prior to this change, it was hard to recompile the project if the order schema was changed:
Compiling 2 files (.ex)

== Compilation error in file lib/core/domain/order/default_machine.ex ==
** (RuntimeError) Snitch.Data.Schema.Order does not define a struct
    lib/beepbop/utils.ex:48: BeepBop.Utils.assert_schema!/2
    lib/core/domain/order/default_machine.ex:58: (module)
    (stdlib) erl_eval.erl:670: :erl_eval.do_apply/6

An issue has been made upstream meanwhile, this branch has a quick fix.

Dear Gatekeeper,
Please make sure that the commits that will be merged or rebased into the base
branch are formatted according to our standards.

The only additional requirement is that the the PR number must appear in the
commit title in brackets: (#XXX)

Tested on both win and Linux, build are successful


Radio

[Enter feedback here]

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

ID: a8ad36ee-1288-9e92-2343-1a50ef487447
Version Independent ID: 56f87b33-72a1-74e2-ed97-3eb7f81685c0
Content: Use a custom markdown tile on Azure dashboards - Azure portal
Content Source: articles/azure-portal/azure-portal-markdown-tile.md
Service: azure-portal
GitHub Login: @JnHs
Microsoft Alias: jenhayes

@Teck13 On this referred Azure document, if you have any feedback (like correcting doc bugs, doc enhancements, product issues related to wrong doc instructions, etc.) then please elaborate it so we can review and try to take action as appropriate.


Update banner color for notifications

Let's use the App blue, #352FB0 for confirmation and keep the red for errors.

@lkpttn This may look odd now that our status bar text is black and not white. Would a lighter blue from the pallete ( #756 ) be better?

@lkpttn Assigning this one to you. You can test by changing the colors in the LDTMessageDefaultDesign.json

Closed by #842


Retry Publish Option

Hello !
I'm wondering if the library provides a way to retry a failed publish?
In the event where a publish fails, the Publish function doesn't return an error.
Instead it auto-reconnects and discard the message.
For example:
attempting to reconnect to amqp server after close with error: Exception (403) Reason: "ACCESS_REFUSED - write access to exchange '...' in vhost '...' refused for user '...'"

I'd like to implement a mechanism to retry with an exponential back-off strategy (either inside the library or in the caller) but I'm not sure if that's possible as the error is not propagated.
Do you know if there's a way to implement that?
Thanks!

The Publish function does return an error, is there something else I'm missing?

Hi, thanks for the feedback!
Yes, the Publish function does return an error in its signature.
However, when Publish fails, the error is not propagated to the caller.
For example, in the scenario if the Publish triggers an ACCESS_REFUSED error.
The error return value is nil.
Then, the logger logs:
channelmanager/channel_manager.go:69: attempting to reconnect to amqp server after close with error: Exception (403) Reason: "ACCESS_REFUSED - write access to exchange '...' in vhost '...' refused for user '...'"
channelmanager/channel_manager.go:101: waiting 5s seconds to attempt to reconnect to amqp server
channelmanager/channel_manager.go:124: error closing channel while reconnecting: Exception (504) Reason: "channel/connection is not open"
channelmanager/channel_manager.go:71: successfully reconnected to amqp server
publish.go:115: successful publisher recovery from: Exception (403) Reason: "ACCESS_REFUSED - write access to exchange '...' in vhost '...' refused for user '...'"

This test has been run with<EMAIL_ADDRESS>When the error is ACCESS_REFUSED, that seems legit to discard the error.
However, I'm trying to implement a retry mechanism when the rabbitmq server is being upgraded / restarted by a cloud provider (e.g. AWS AmazonMQ RabbitMQ).
Thanks!

I did a quick test and indeed these type of errors [^1][^2] are not propagated from the amqp base library; there is nothing any wrapping library can do for you (well, short of checking things, which you could also do in app layer beforehand [^3]).
The fairly clean way out [^4] is to call PublishWithDeferredConfirmWithContext() and then wait on the returned PublisherConfirmation (array of type amqp.DeferredConfirmation) via various tests (array length, [0] is valid then amqp calls on [0] with context and/or timers, whatever the amqp library offers, ref. amqp.DeferredConfirmation See also the publisher_confirm example.
I hope this helped a bit.
[^1]: publishing on existing exchange but non existing queue does not return any error nor drops/reconnects the channel.
[^2]: publishing on inaccessible exchange/queues (non existing or ownership) will not return any amqp publishing errors but will drop & reconnect the channel. Publishing WithImmediate will also drop/recover the supporting amqp channel (I wasn't expecting this 🤕  : Exception (540) Reason: "NOT_IMPLEMENTED - immediate=true")
[^3]: a bit of a shame this library does not export the inner channel manager or up-streams the  chan manager safe_wrappers to the consumer/publisher objects. This way the apps could have called on all amqp channel functions, including testing for queues via QueueDeclarePassiveSafe
[^4]: looks like you're better off awaiting confirmations from the app layer as described above.

Thanks for investigating into this @LucaWolf !
I'm quite surprised that the underlying library does not propagate 540 exceptions.
I tried to look into the library which exceptions returns an error but could not find it.
I'll use PublishWithDeferredConfirmWithContext to handle this case, thanks!

Glad you found that!


Don't allow spaces in usernames.

I assume this is closed with #2ff893... - Add tests for user validation.

Yep. Forgot to close this. Thanks!


Drag and Drop on ControlledTreeEnvironment doesn't work

Describe the bug
If in a ControlledTreeEnvironment the drag and drop wont work
To Reproduce
https://codesandbox.io/s/ng6mme?file=/src/App.tsx
Expected behavior
Expecting item to be dropped, it is possible to drag it.
Desktop (please complete the following information):

OS: W10
Browser: chrome
Version latest

Additional context
With
canDragAndDrop={true}                 canReorderItems={true}                 canDropOnItemWithChildren={true}                 canDropOnItemWithoutChildren={true}
And no restrictions it is not possible to drop the item, the canDropAt seems to work fine but also doesn't allow me to drop the item

Hello @chalscc. In a ControlledTreeEnvironment, you need to implement the logic for handling drag and drop events yourself. If you want to to use the built-in logic for handling drop events, please use an uncontrolled environment instead.

Hi, I'm having trouble dragging and droppign items in a ControlledTreeEnvironment as well-- and so is this fellow https://stackoverflow.com/q/73048304/19866314.
My onDrop prop is simple for now as I'm just logging the details-- but I'm finding that the target parameter is not being set properly. It's not set to the item I dropped onto-- in fact, it's seemingly random.
onDrop={(items, target) => {
                 console.log("Drop detected:", { items, target });
                 handleDrop(items.map(item => item.index as string), target);
               }}

I would appreciate any help.
Thanks,
Eliot


Generate username and password for the Velum internal API

Also, mount the CA certificate in the salt-master container, as it
is required for the Velum pillar to perform SSL/TLS requests.
Depends on:

https://github.com/kubic-project/velum/pull/315
https://github.com/kubic-project/salt/pull/238

PR has been rebased.

 Serve pillars from Velum

Rebased.

rebase needed, to have initContainers in the manifests


Revamp imports.

Description
Main Objective

[x] Create an index file to re-export Components/Functions.
[x] Update import path from relative to absolute (based on src as base folder) wherever possible.

[x] Renderer Files.
[ ] ~Main Files.~

Additional Objective

[x] Updated configuration files. (Refer to ERB)
[x] Resolved error webpack DevServer : deprecated onBeforeSetupMiddleware
[x] FIXED - On page refresh, logout functionality breaks.

Currently, in development mode, the main.ts is transpile without webpack config, hence the absolute path imports are throwing module not found error.
This will be addressed in the new issue.
Absolute imports are working as expected in renderer side files.


Include new version file config prometheus

Description
This PR include new yaml config prometheus that improve significantly the scraping time and get new metrics Apache Kafka's version 2.5 or higer. The image above show the improvement of scraping time:

Fixes # (n/a)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Ran molecule test which will validate all roles function
Checklist:

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules
[ ] Any variable changes have been validated to be backwards compatible

Ok. I'll change it.

@erikgb @alvarobacelar A colleague has updated this work further on the following PR, to gather all metrics but still keep the performance enhancements.  I've added you both as reviewers:
https://github.com/confluentinc/cp-ansible/pull/747

Hello @alvarobacelar, Please let us know if you agree with @JumaX recommendation. If thats not the case, can you please resolve the conflicts and let us know whats the extra metrics we are collection in this PR.
Thanks!

Closing in favour of https://github.com/confluentinc/cp-ansible/pull/747. Let me know if you have different opinion.
Thanks !


Run cargo upgrade

Summary
cargo upgrade just updates the listed versions in the TOML files. There's no change to the Cargo.lock other than a minor bump to toml-edit.

I have absolutely no clue why the trampoline lockfile is trying to update.

It's not included in the workspace and nothing in that crate changed.

It was an unused dev dep from the trampoline crate to uv_fs


BUG: iOS version not building from 'develop'

Currently you cannot build the iOS version of the app.  The error reference the 'tripleblind' app, so it is likely that it has to do with renaming packages.
Can test against the first release off of the 'master' branch, which I believe did function.

The most recent develop branch seems to build fine for me using a simulator for iPhone 11, still need to test locally to see if that responds differently

@vitorpamplona was having issues yesterday, so let's confirm (especially since he is doing the builds!)

@vitorpamplona , can you build for iOS yet?  If not, @davidbiga thinks it is working.  Can you two confirm?

There seems to be something wrong with the xCode config files. It still thinks the app is inside a folder called tripleblind.

I also confirm develop branch b77bf1c80e7e7598f97d751388ff9a2d187b92f6 builds in iPhoneSE

Boom, done!


Batch autosizing

Closes #148. (Sorry to jump ahead---I think you were working on this @Adamits but it's become a major blocker for various other things I'm trying to do with my students.)
If requested with --auto_batch_size_find, this finds the max batch size, then computes the "optimal" batch size to use such that it, plus multiple gradient accumulation steps, is as large as possible but does not exceed the max. The major features of the sizer are also exposed as flags though I think the defaults are what most people will want. (The binsearch method gives more precise results but it will probably not find more optimal numbers since they won't be perfectly divisible.)
The values are all logged and set, so that one can combine this with training, or one can simply do this without training using a trick like --max_epochs 0 as documented in the README.
I tested this by taking a large transformer model (many encoder and decoder layers), requesting an enormous batch size of 2048. In this case (and following my GPU memory utilization bears this out) it succeeds at batches of size 1024 but fails at batches of 2048, so it sets batch size = 1024 and uses two gradient accumulation steps per update---it proceeds to update, so this would be compatible with a hyperparameter search. I also repeated this with --max_epochs 0, in which case it simply logs the same results from the finder, and then halts.
Other changes:

I have undone the hack whereby I double batch size when not training, since that seems to conflict with this.
I have removed the initial LR finder since that also seems to conflict.

Sample output #1 (no OOMs at risk):
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 2 succeeded, trying batch size 4
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 4 succeeded, trying batch size 8
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 8 succeeded, trying batch size 16
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 16 succeeded, trying batch size 32
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 32 succeeded, trying batch size 64
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 64 succeeded, trying batch size 128
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 128 succeeded, trying batch size 256
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 256 succeeded, trying batch size 512
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 512 succeeded, trying batch size 1024
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 1024 succeeded, trying batch size 2048
`Trainer.fit` stopped: `max_steps=3` reached.
Batch size 2048 succeeded, trying batch size 4096
Batch size 4096 failed, trying batch size 2048
Finished batch size finder, will continue with full run using batch size 2048
Restoring states from the checkpoint path at ../models/transformer/.scale_batch_size_84895d35-c4e3-4f2d-82c9-31945a024da8.ckpt
Restored all states from the checkpoint file at ../models/transformer/.scale_batch_size_84895d35-c4e3-4f2d-82c9-31945a024da8.ckpt
Max batch size: 2048
Using batch size: 1024
Using gradient accumulation steps: 1

Sample output #2 (avoiding an OOM with gradient accumulation):
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 2 succeeded, trying batch size 4
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 4 succeeded, trying batch size 8
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 8 succeeded, trying batch size 16
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 16 succeeded, trying batch size 32
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 32 succeeded, trying batch size 64
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 64 succeeded, trying batch size 128
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 128 succeeded, trying batch size 256
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 256 succeeded, trying batch size 512
`Trainer.fit` stopped: `max_steps=10` reached.
Batch size 512 succeeded, trying batch size 1024
Batch size 1024 failed, trying batch size 512
Finished batch size finder, will continue with full run using batch size 512
Restoring states from the checkpoint path at ../models/transformer/.scale_batch_size_22730f17-efd1-44f8-b5f0-bc4efda67e89.ckpt
Restored all states from the checkpoint file at ../models/transformer/.scale_batch_size_22730f17-efd1-44f8-b5f0-bc4efda67e89.ckpt
Max batch size: 512
Using batch size: 512
Using gradient accumulation steps: 4

Awesome!
This had become the thing that I put in my calendar to do after work like 5
different times and never got around to, so I appreciate you doing it!
I will give it a review tomorrow
Adam
On Wed, Jul 3, 2024 at 5:48 PM Kyle Gorman @.***> wrote:

@kylebgorman https://github.com/kylebgorman requested your review on:
#204 https://github.com/CUNY-CL/yoyodyne/pull/204 Batch autosizing.
—
Reply to this email directly, view it on GitHub
https://github.com/CUNY-CL/yoyodyne/pull/204#event-13387786574, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIQBSSZMUJTE4EAMM6LFADZKSLU5AVCNFSM6AAAAABKKRFVSKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGM4DONZYGY2TONA
.
You are receiving this because your review was requested.Message ID:
@.***>

Suppose I don't care what effective batch size I train with, I strictly want the maximum possible batch size with no gradient accumulation. Can I do that in one go?

You can in the common case where the requested --batch_size is smaller than the max size it finds. Then it'll use the requested batch size and no gradient accumulation (it'll set that flag to 1).

Suppose I don't care what effective batch size I train with, I strictly want the maximum possible batch size with no gradient accumulation. Can I do that in one go?

You can in the common case where the requested --batch_size is smaller than the max size it finds. Then it'll use the requested batch size and no gradient accumulation (it'll set that flag to 1).

What I meant is: can I just request it to find max_batch_size and then train with that whatever it is. But I think just calling this for 0 epochs, and then requestin w/e max it found is fine so I don't think it'sworth worrying about.

What I meant is: can I just request it to find max_batch_size and then train with that whatever it is. But I think just calling this for 0 epochs, and then requestin w/e max it found is fine so I don't think it'sworth worrying about.

Oh, that's not supported but I can imagine how you'd support that (I think you'd have a convention like when the batch size is non-positive and finding is requested, we just use max batch size).

Oh, that's not supported but I can imagine how you'd support that (I think you'd have a convention like when the batch size is non-positive and finding is requested, we just use max batch size).

Added and documented this. The trick I am using is you pass --batch_size 0 --auto_batch_size_find and it'll just use the max batch size it finds. (In that case I'd recommend also using --auto_batch_size_mode binsearch since it gets you slightly higher saturation of the accelerator.)

Added and documented this. The trick I am using is you pass --batch_size 0 --auto_batch_size_find and it'll just use the max batch size it finds. (In that case I'd recommend also using --auto_batch_size_mode binsearch since it gets you slightly higher saturation of the accelerator.)

Nevermind, this breaks something else.


Make Playground the main Quickstart option

And moved Everything Bagel to "more quickstart options".
Resolves #4688

I think we should actually stick with the current assumption (which I feel that has been somewhat validated at this point) around our user journey:
local docker run -> local docker run with cloud storage -> cloud installation.
I'd argue that the playground belongs in the "more quickstart options" as well, and in other assets that are meant for lower intent users (lakefs.io landning page, perhaps the index page and README...


Popup is not positioned properly inside Modal when Modal's size changes.

Bug Report
Steps
I have a Popup info triggered by an input box inside a Modal.
the first time render of the Modal, Popup is positioned perfectly.
however when I opened an accordion (which it is also in the Modal)
the Popup will not follow the updated position.

Expected Result
the Popup should follow the trigger input box to its new position
Actual Result
the Popup will not follow the updated position.
Version
latest
Testcase
codesandbox link:
https://codesandbox.io/s/semantic-ui-react-example-4d3ns

It's done for performance reasons in PopperJS, in recent release there is an access to scheduleUpdate() via popperDependencies:
https://react.semantic-ui.com/modules/popup/#usage-popper-dependencies
I have updated your CodeSandbox: https://codesandbox.io/s/semantic-ui-react-example-r1pmb
+popperDependencies={[activeIndex]}

@layershifter Thanks for your quick reply...I should update the newest version for using this prop. Thx


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 307

Questions ?
Please refer to the Backport tool documentation


Undefined symbols with arm64 linker when building and linking grpc and protobuf for Android and iOS in Qt

What version of gRPC and what language are you using?
1.18.0 / C++
What operating system (Linux, Windows,...) and version?
Mac OS 10.14.4
What runtime / compiler are you using (e.g. python version or version of gcc)
gcc v4.2.1
What did you do?
I created a service using grpc-go and protobuf. Everything works fine server/client in go. I outputted the proto from the one I made in Go to cpp and now I am attempting to use that in my qT app (Android and iOS clients). I went ahead and copied over the proto and referenced it in my qt pro file as well as copied the necessary libs. I was previously using Protobufs already for firebase so I am not sure if this is somehow conflicting with the protobuf and grpc files I added. Since the target is an arm64 architecture, I am not sure how to appropriately get that output to that.
When I build within qT framework, I get ->
"-1: error: symbol(s) not found for architecture arm64"
Then, when I attempt to build in Xcode I get ->
Undefined symbols for architecture x86_64:
"google::protobuf::Message::GetTypeName() const", referenced from:
vtable for position::Coordinate in position.pb.o
vtable for position::Position in position.pb.o
vtable for position::PositionsRequest in position.pb.o
vtable for position::PositionsNearDriverRequest in position.pb.o
vtable for position::PositionsNearPositionRequest in position.pb.o
vtable for position::SendResult in position.pb.o
What did you expect to see?
I expected to build and link properly.
What did you see instead?
Build and link errors related to cpu architecture undefined symbols.
This is my first time doing this so I am not sure if it is in fact an issue with GRPC or Protobufs, me needing to compile myself and set the proper architecture, or what.
Thanks

Hey Yang @yang-g  - any ideas if this is an actual issue or my configuration issue?

So first, I don't see the symbol that  got missing for arm64. The symbols you listed are missing for x86_64, and they look like from generated stubs, so one thing good to check is whether the implementations of the stubs are included in the project. I think more information would definitely be helpful for troubleshooting.
Second, note that building gRPC C++ for iOS using gcc is not an officially supported approach at current. The one we are supporting is the Cocoapods spec gRPC-C++.

QMAKE_CXXFLAGS += pkg-config --cflags= protobuf grpc
QMAKE_CXXFLAGS += -std=c++11
QMAKE_LIBS += pkg-config --libs= protobuf grpc++ -Wl,--no-as-needed -lgrpc++_reflection -Wl,--as-needed -ldl


🎈 13.1.0

🎈 Release
Preparation for 13.1.0 release.
Comparison to 13.0.1: https://github.com/gazebosim/sdformat/compare/sdformat13_13.0.1...sdf13

Needed by https://github.com/gazebosim/gz-sim/pull/1560
Checklist

[x] Asked team if this is a good time for a release
[x] There are no changes to be ported from the previous major version
[x] No PRs targeted at this major version are close to getting in
[x] Bumped minor for new features, patch for bug fixes
[x] Updated changelog
[ ] Updated migration guide (as needed)
[ ] Link to PR updating dependency versions in appropriate repository in gazebo-release (as needed):

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

I see 29 commits in the branch comparison but only 6 PRs in the changelog

Do you typically list forward ported commits in the latest version of the changelog?  I believe all the others were from merges.

I see 29 commits in the branch comparison but only 6 PRs in the changelog

Do you typically list forward ported commits in the latest version of the changelog? I believe all the others were from merges.

I don't include forward-ported commits in major releases, but I do for minor releases. So the 13.0.0 changelog included only things new since 12.x, but for 13.1 I would include the things thad didn't make it into 13.0
I'll take care of it

I don't include forward-ported commits in major releases, but I do for minor releases. So the 13.0.0 changelog included only things new since 12.x, but for 13.1 I would include the things thad didn't make it into 13.0
I'll take care of it

https://github.com/gazebosim/sdformat/pull/1185/commits/6f81b6fd086333ddbd86298ab3384549af07c223


APT whitelist request for gearman-job-server

Want to be able to test gearman client against the C server.
Thanks!

Didn't realise this was in "universe" I am guessing that might change how I should approach this?


log chained exceptions

fixes #1350

Here's how it looks right now (using Flask)

There's still some issues here. It looks like we don't want to use chain=True in exception, since that sticks the whole thing at the top and bottom of the page, instead of just the exception. Instead, we want to to make plaintext understand the chained exceptions.
Started digging into this and basically we need to create groups of frames and let them handle the formatting. Python actually has a similar idea, but it's in 3.5+ only.

Rebased your original commmit, then made changes as described in the previous comment. Did a bunch of compat cleanup in tbtools too.
A lot of the processing moved from Traceback to the new Group object. Made the style for the chain info message look like the "Traceback (most recent call last)` header.

Turns out that frame.f_lineno is not accurate (at least in chained exceptions),  and traceback.tb_lineno should be used instead. I could not find an explanation for this difference.


Add species label/tag mapping

Suggested by @jkshenton in #103

The resulting keys for the site properties get a little unwieldy with the tags and the labels - I think we should either reorganise the way these are stored/ provide some sort of aliases/ have utility functions to extract the most common bits of information needed. But that can be a in separate PR.

Possible solutions:

Use just general species names (w/tag?) for site properties and keep a list of labels separately which can be zipped with species props if needed
Create functions to pull species name info into components
Return a 3-uple of (Species, Index, Label)
Create a NamedTuple for (species, index, tag, label) defaulting to empty and atreg_to_index (and related parsings) return this type.

I guess that option 1 would mean changing the site properties to be lists. This is how some other libraries (e.g. ASE) deal with sites and is what I would expect.
I think that it would be nice to, at least have the option to easily extract something like:
{
    'cell': 3x3 array,
    'positions': Nx3 array},
    'symbols': list of length N with standard element symbols,
}

with any optional site properties as Nx? lists/arrays with an appropriate key in this dictionary. e.g. forces, local magnetic moments, custom site labels, custom site species tags etc.
any time there's a structure reported (e.g. it would be nice to be able to output a list of these for a geometry optimisation).
However, that might go too far away from a straightforward dictionary that someone might expect from a raw parsed .castep file (?), so maybe just having a sort of user-friendly API to return these 'decorated crystal structure dictionaries' would be good. If we do it that way, then maybe option 4 would be the best, to at least standardise the raw parsed .castep dictionary .

I would probably add an extra key to the main dict for pulling out the labels, etc.
I'm more inclined towards the NamedTuple approach though. I think that gives the best compromise of locality, concision and flexibility and there's a way to reconstruct it simply by adding a __str__ method to the named tuple for json output.
There's no reason we can't have multiple though.
In terms of pulling out subsets, you might want to take a look at toolz which is included in e.g. conda or used as a dependency for a lot of stuff.

import toolz

[set up pick...]

data = parse_castep_file( ... )
subset = pick(("initial_positions", "initial_cell"), data)
# subset["initial_cell"] = subset["initial_cell"]["real_lattice"] if wanted
subset["species"] = {x[0] for x in subset["initial_positions"].keys()}  # Or use [ ] for duplicates

But you're right that maybe some general utility functions for pulling out certain subsets might be nice. The concern is that different people might want different things and in the end it might be easier to just let them write the quick manipulation themselves.

Thanks- I'd not used toolz before, but it seems very handy!
I'd be happy with the NamedTuple option and would be happy to help with the utility functions. I do agree that we shouldn't go overboard with these utility functions, but a couple of very common ones would make getting started with this library very easy. Plus it could act as an example of how to easily extract the information people do want - a sort of cookbook.

System with NamedTuple looks like this.
Not sure if I like it in retrospect, because while it's clean it adds an inherent non-transferability due to needing the Atom type

This does not preclude the use of a dict as the main type.

Ok. This goes much deeper and is more complicated than I'd thought (isn't it always)
magres properties don't have to be species. They will be replaced by labels if supplied.
Geometry only uses them if not VCA in atomic positions
MD does not beyond forces
Mulliken populations can use them
As can Born charges, but there, they're stuck at the end of the line behind "ID=" rather than appended to the species
GA uses them, but we don't parse that yet and it looks like only for recreating on child jobs

Eish yeah, these things always turn out to get complicated at some point...
I guess standardisation like this is exactly the sort of thing that your library can help with. Again maybe having a standard raw-parsed version of the .castep file and then some getter type functions that apply the same syntax/standard as far as possible might be the way to go.

The reason why I dislike the NamedTuple style and the idea of a getter/setter class methods is the lack of general transferability. By avoiding specialist classes you can just load the json/yaml files directly without needing castep_outputs and that's one of my goals.
Utility functions are certainly a good idea though.
I have a feeling pulling the labels to the top may work, but why one would do that for labels, but not tags leads to inconsistencies. I'm still trying to work out how to handle this and I feel the best way is to keep it in the
He:Tag [label]

form and provide a utility function to turn this into:
{'spec': 'He', 'tag': 'Tag', 'label': 'label'}

So I think I'm decided.

Leave the labels as they are now, translating places where they're not in the "standard" form (Sp:tag [label]) to that form.
Add some utility functions to pull atom names apart into these components
Add extra modules for recipes and toolz_recipes where we can stick in extensions to extract key components or, e.g. transform all labels into (Named)Tuples.

Closed as this issue is handled. Extension recipes will be discussed in #117


It seems like NoopLocationStrategy doesn't work in certain websites.

We're currently developing a chrome extension and it works fine on all websites except against dev.to
Is there a way to customize NoopLocationStrategy or a workaround for certain websites?

I was able to create a workaround for dev.to
What i did was override the prepareExternalUrl and just keep on returning the set base href.
import {
  APP_BASE_HREF,
  LocationChangeListener,
  LocationStrategy,
  PlatformLocation,
} from '@angular/common';
import { Inject, Injectable, Optional } from '@angular/core';

@Injectable()
export class CustomNoopLocationStrategy extends LocationStrategy {
  private readonly baseHref: string;

  constructor(
    private platformLocation: PlatformLocation,
    @Optional() @Inject(APP_BASE_HREF) baseHref?: string
  ) {
    super();
    this.baseHref = baseHref || this.platformLocation.getBaseHrefFromDOM();
    if (!this.baseHref) {
      throw new Error(
        `No base href set. Please provide a value for the APP_BASE_HREF token or add a base element to the document.`
      );
    }
  }

  onPopState(fn: LocationChangeListener): void {}

  getBaseHref(): string {
    return this.baseHref;
  }

  path(includeHash: boolean = false): string {
    return '';
  }

  prepareExternalUrl(internal: string): string {
    // if (this.baseHref.endsWith('/') && internal.startsWith('/')) {
    // return this.baseHref.substring(0, this.baseHref.length - 1) + internal;
    // } else if (this.baseHref.endsWith('/') || internal.startsWith('/')) {
    // return this.baseHref + internal;
    // } else {
    // return `${this.baseHref}/${internal}`;
    // }
    return `${this.baseHref}`;
  }

  pushState(
    state: any,
    title: string,
    path: string,
    queryParams: string
  ): void {}

  replaceState(
    state: any,
    title: string,
    path: string,
    queryParams: string
  ): void {}

  forward(): void {}

  back(): void {}

  historyGo(): void {}
}

any reason why we have to keep track of internal path argument from prepareExternalUrl?

Hey @chan-dev!
The internal argument is the part from routerLink like /my-route/abc/efg. prepareExternalUrl is used by Angular to figure out what URL to put on a native a element such that the browser can figure out on its own where to lead the user. By changing it to just the base href, I'd expect that every link now just goes to the application at root itself.
What problems have you seen with the NoopLocationStrategy with dev.to?
The implementation of prepareExternalUrl is in itself derived from the PathLocationStrategy of Angular (https://github.com/angular/angular/blob/0af354ce056c7e6254bd17a52d4199bf86f51c0e/packages/common/src/location/location_strategy.ts#L147)

@fboeller Since we're using Angular routing inside chrome extension, we don't want the extension to change the current page's url. This extension helps us most of the case except in the case of when visiting dev.to, it still changes the dev.to url.
Let's say we're currently visiting dev.to and user has redirected to route /test1 inside chrome extension, then what happens is the browser url will redirect to dev.to/test1 instead of being entirely isolated within the chrome extension.


Dummy issue - close this

This is a Bug Report

Problem:
Proposed Solution:
Page to Update:
https://kubernetes.io/...

/triage needs-information

/remove-triage needs-information

/language no

/remove-language no

/priority backlog

/remove-priority backlog

/help

/remove help

/remove-help

/good-first-issue

/remove good-first-issue

/remove-good-first-issue

If you want to close this @aimeeu (or anyone), /close will do that!

@sftim lol.

/close


Script do not work without ipsec & openssl command

Script do not work without ipsec & openssl command. Would it be possible to add in prerequisites all of the packets/module to install ?
I can not find how to install ipsec command to make the script works.
Thanks

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

ID: 9de8dc74-3f27-6878-cfc6-52a01aab608e
Version Independent ID: 678e7b80-d6c0-d134-991c-89c48999ceea
Content: Configure point-to-site connectivity using SSMS - Azure SQL Managed Instance
Content Source: azure-sql/managed-instance/point-to-site-p2s-configure.md
Service: sql-managed-instance
Sub-service: deployment-configuration
GitHub Login: @srdan-bozovic-msft
Microsoft Alias: srbozovi

@raphaeldeletoille Thank you for providing your feedback. We are reviewing this and will get back shorty.

@raphaeldeletoille Are you referring to this script from the doc.

The comment refers to the script above and, more specifically, to the helper script it fetches from GitHub, and its outcome on non-Windows machines.
We cannot provide instructions because they depend on the packaging mechanism of the OS the user is running.

Hi @raphaeldeletoille , thanks for contacting Microsoft! Since Zoran was able to address your question, I'll go ahead and close out this Git Issue but please feel free to comment should you need anything further.
Thanks again for reaching out and I hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#please-close
#reassign:zoran-rilak-msft


Support envfrom

Fixed merge conflict and reopening https://github.com/operator-framework/operator-lifecycle-manager/pull/1950

Need to fix some govet issues but overall looks good to me.

Need to fix some govet issues but overall looks good to me.

Fixed the linting issue, should be good to review now!

closing in favor of https://github.com/operator-framework/operator-lifecycle-manager/pull/3193


Add option to wlr/workspaces to sort workspaces by number

#1691 didn't work for me. This pull request fixes #1596, by adding the option "sort-by-number" to wlr/workspaces, which takes precedence over any other sort-by option if set to true.
Example config
"wlr/workspaces": {
  "sort-by-number": true,
}

The icons can still be changed using
"format-icons": {
  "1": "a",
  "2": "b",
  "3": "c",
  "urgent": "d",
  "active": "e",
  "default": "f"
}

and the sorting will still be done according to the actual workspace numbers.

Thanks!
Can you update the man to add the change and also update the Github wiki? :)

I have updated both the man page and the wiki now.


Manuals

Added a link to the architecture manual after a conversation with Luke Slowen RE patterns.
Moved the tab order so the manuals are in a logical order.
Change the tab titles to sentence case so they're consistent with the rest of the site.

Apologies @andyjones81 - I hadn't noticed the Additional guidance section before. Happy to go with the footer secondary nav if you think that'll work with the DfE stylesheet.


UVify Core specific support is added

Please use PX4 Discuss or Slack to align on pull requests if necessary. You can then open draft pull requests to get early feedback.
UVify board specific package
This is UVify board specific package for current master.

Hi @limhyon
Great to see this coming to upstream.
You'll need to add the board to Jenkins to be compiled.
https://github.com/PX4/Firmware/blob/7ba658d5730c99e76007f4f30ff77f6c1b9987dc/.ci/Jenkinsfile-compile#L40
Which bootloader are you using? https://github.com/PX4/bootloader
Do you want to be able to flash this from QGC?
The other thing we'll need to discuss is getting this into the hands of a few core developers so that we'll be able to maintain it going forward. If debug (JTAG/SWD) and a serial console are available I'd like to get one of these added to the hardware test rack.

Any progress on this one ?

There is no other PR yet, this one has conflicts...
We would like to show off our Draco-R at an international conference, but we need this fixed.

I've resolved the merge conflicts and fixed a few minor things. There are minor issues (eg mavlink conflicts), but this should be safe to merge. We can continue iterating with this in master.
@hyonlim can you get the bootloader ID added to https://github.com/PX4/Bootloader/blob/master/board_types.txt?

Build added to Jenkins (binary available under "Compile All Boards" artifacts).

@hyonlim (or @amilcarlucas perhaps) I would do a pass testing each airframe on real hardware. I don't have hardware to test this, but I noticed a few things.

quad_x_draco mixer missing (change to quad_x for now)
OSD not handled (ttyS6?)
check mavlink modes for each airframe
set DRACO_SMBUS yes?
Draco R mount (mount_2axes.aux.mix) not handled


cell_cycle_gene default for human won't work with mouse data

In ingest panpipe cell cycle action ccgenes is set to "default"  which links to a cell_cycle gene list for humans only. This makes it so that the pipeline fails when running with mouse data.
Thus a new gene list for mouse data must be generated and provided before running. This either has to be explicit (that the user must change the gene list) or implemented in the code so that it automatically generates this mouse translation.
I have, however,  already generated this mouse list for myself from the human one provided in the tutorials, and am happy to add this to the resources folder for other users!

thank you @giuliaelgarcia good idea!
please do upload the mouse cc genelist in thesame folder as the human one,
currently setting ccgenes to default assumes simultaneously:

we deal with human data
we trust the provided list

specifying a custom path to any file in the ccgenes allows you to escape both assumptions,
i.e. if you have a mouse list, you CAN'T set ccgenes to default but you can only specify a path to a file.
if you add your file, we offer the user the option to use it and you should clarify how to do it in the documentation.
actions:

[ ] add the file to the repo
[ ] add info on the gene list page https://panpipes-pipelines.readthedocs.io/en/latest/usage/gene_list_format.html
[ ] add info on the ingestion https://panpipes-pipelines.readthedocs.io/en/latest/yaml_docs/pipeline_ingestion_yml.html#providing-a-gene-list

Additionally, the mitochondrial genes in the mouse_qc_list from the tutorial are not all correct, some genes added are not mt and some are not protein-coding. I will attach a list of mt mouse genes which I used to double check which ones were correct.
mmus_Gcm38_mito.txt


Fixed mask shape after Albu postprocess

Motivation
If the augmented image (mask) is for example rotated by 90 or 270 deg. using "RandomRotate90" and the original image (mask) is rectangular, it could end up with height and width inverted, therefore we need to use dimensions of the augmented mask (image) to convert.
Modification
Take the augmented mask shape instead of the original mask shape.
BC-breaking (Optional)
None.
Use cases (Optional)
RandomRotate90
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMPreTrain.
The documentation has been modified accordingly, like docstring or example tutorials.

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

@ilcopione Please fix the lint

@hhaAndroid, thanks for your reply. I've fixed the lint as you requested!!


Code style: should getter methods return pointers or references

Currently some classes (especially in the parser and the optimizer) have their getter methods return pointers (*) to objects while some return references (&) to objects. As Jignesh suggested, would it better if we follow certain style guide and uniform the style of the existing code? Say, only return pointers if the returning value can be NULL or is yielding ownership, otherwise return references.

Hi @jianqiao
In some other modules, you can see a pattern -
For getter methods that won't modify the class member, we return a const reference we use
const Field& getField() const {}
For getter methods that allow modification of the class member, we return a pointer and the pattern is
Field* getFieldMutable() {}

I've noticed that in the parser classes, there tends to be returning of const *. It may have been written this way because the SQL_ERROR macros in the parser/optimizer take pointers instead of references.
However there are some cases where const & are returned, in which case, if they are later used in the SQL_ERROR macro, they use the address of operator (&aReference) to essentially recover the pointer which seems a little dubious.


aecore_suite uses a persisted DB

In scope for https://www.pivotaltracker.com/story/show/155290714
PT https://www.pivotaltracker.com/story/show/155296843

I see 2 approvals. I see Circle CI passed. I see one Travis build passed. Merging.


spk infra scaffold fails to validate remote git sources

Describe the bug:
Recent changes made to validateRemoteSource now takes in a configuration parameter that is not specified in spk infra scaffold.
The error:
fatal: 'template' does not appear to be a git repository
fatal: Could not read from remote repository.

To Reproduce:
Execute spk infra scaffold --name <name of project> --source <git url source> --version <template version> --template <path to template>
Expected behavior:
spk infra scaffold should be able to validate the git sources.
Does this require updates to documentation?:
No

Could this have been detected by some sort of unit test related to the validateRemoteSource workflow?

Could this have been detected by some sort of unit test related to the validateRemoteSource workflow?

It was detected by the spk integration tests :)


La7

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.10.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Germany
Provide a description that is worded well enough to be understood
Until last week downloading videos from https://www.la7.it/ worked without problems.
Now it suddenly outputs the following error. I guess it might be related to some drm issues from la7.it
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
$ /home/marcello/.local/bin/yt-dlp -vU "https://www.la7.it/otto-e-mezzo/rivedila7/meloni-fiducia-tra-programma-e-comizio-otto-e-mezzo-puntata-del-25102022-25-10-2022-457308"
[debug] Command-line config: ['-vU', 'https://www.la7.it/otto-e-mezzo/rivedila7/meloni-fiducia-tra-programma-e-comizio-otto-e-mezzo-puntata-del-25102022-25-10-2022-457308']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.10.04 [4e0511f] (pip) API
[debug] Python 3.10.6 (CPython 64bit) - Linux-5.15.0-46-lowlatency-x86_64-with-glibc2.35 (glibc 2.35)
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2020.06.20, mutagen-1.45.1, pyxattr-0.7.2, secretstorage-3.3.1, sqlite3-2.6.0, websockets-9.1
[debug] Proxy map: {}
[debug] Loaded 1690 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.10.04, Current version: 2022.10.04
yt-dlp is up to date (2022.10.04)
[debug] [la7.it] Extracting URL: https://www.la7.it/otto-e-mezzo/rivedila7/meloni-fiducia-tra-programma-e-comizio-otto-e-mezzo-puntata-del-25102022-25-10-2022-457308
[la7.it] meloni-fiducia-tra-programma-e-comizio-otto-e-mezzo-puntata-del-25102022-25-10-2022-457308: Downloading webpage
[la7.it] meloni-fiducia-tra-programma-e-comizio-otto-e-mezzo-puntata-del-25102022-25-10-2022-457308: Downloading MPD manifest
WARNING: [la7.it] Failed to download MPD manifest: HTTP Error 502: Bad Gateway
[la7.it] meloni-fiducia-tra-programma-e-comizio-otto-e-mezzo-puntata-del-25102022-25-10-2022-457308: Downloading m3u8 information
WARNING: [la7.it] Failed to download m3u8 information: HTTP Error 502: Bad Gateway
ERROR: [la7.it] meloni-fiducia-tra-programma-e-comizio-otto-e-mezzo-puntata-del-25102022-25-10-2022-457308: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Traceback (most recent call last):
  File "/home/marcello/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1477, in wrapper
    return func(self, *args, **kwargs)
  File "/home/marcello/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1574, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/home/marcello/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1632, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "/home/marcello/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 2569, in process_video_result
    self.raise_no_formats(info_dict)
  File "/home/marcello/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1027, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [la7.it] meloni-fiducia-tra-programma-e-comizio-otto-e-mezzo-puntata-del-25102022-25-10-2022-457308: No video formats found!; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U

A regex, even though not greedy, is matching an entire chunk of HTML
/content/servizio-di-notifiche-push" title="LA7 Notifiche Push">..."contentUrl":"https://awsvodpkg.iltrovatore.it/content/entry/data/0/532/0_hgc8sxtm_0_173lcsvm_1

and needs to be curtailed:
--- old/yt_dlp/extractor/la7.py
+++ new/yt_dlp/extractor/la7.py
@@ -63,7 +63,7 @@
             url = '%s//%s' % (self.http_scheme(), url)

         webpage = self._download_webpage(url, video_id)
-        video_path = self._search_regex(r'(/content/.*?).mp4', webpage, 'video_path')
+        video_path = self._search_regex(r'(/content/\S+?).mp4', webpage, 'video_path')

         formats = self._extract_mpd_formats(
             f'{self._HOST}/local/dash/,{video_path}.mp4.urlset/manifest.mpd',

Actually, if this is always the target the extractor should extract the ld+json block and use its ['url'] value instead of searching blindly.

I made a PR fixing the issue.
The solution provided by @dirkf is not working because it catches only the low res mp4 in the webpage.
The suggestion of extracting the info from the ld+json block is also not good for the same reason.
The manifest url containing the best quality stream is in an anonymous script block and the improved regex provided in the PR should be ok.


UPDATE FILE STRUCTURE OF THIS PROJECT 🤩 ✨⭐

To update the file structure of this React project according to standard conventions, you can follow this suggested structure:
Suggested File Structure
Enyanjyoti/
│
├── public/
│   ├── index.html              # Main HTML file
│   ├── favicon.ico             # Favicon
│   └── assets/                 # Static assets (images, fonts, etc.)
│       ├── images/
│       └── fonts/
│
├── src/
│   ├── components/             # Reusable components
│   │   ├── Navbar/
│   │   │   ├── Navbar.js
│   │   │   ├── Navbar.css
│   │   │   └── index.js
│   │   ├── Footer/
│   │   │   ├── Footer.js
│   │   │   ├── Footer.css
│   │   │   └── index.js
│   │   └── ...                 # Other components
│   │
│   ├── pages/                  # Page components (for routing)
│   │   ├── Home/
│   │   │   ├── Home.js
│   │   │   ├── Home.css
│   │   │   └── index.js
│   │   ├── About/
│   │   │   ├── About.js
│   │   │   ├── About.css
│   │   │   └── index.js
│   │   └── ...                 # Other pages
│   │
│   ├── hooks/                  # Custom hooks
│   │   └── useCustomHook.js
│   │
│   ├── context/                # Context API files
│   │   └── AppContext.js
│   │
│   ├── utils/                  # Utility functions
│   │   └── helpers.js
│   │
│   ├── styles/                 # Global styles
│   │   └── main.css
│   │
│   ├── App.js                  # Main app component
│   ├── index.js                # Entry point for React
│   ├── serviceWorker.js        # Service worker file (optional)
│   └── setupTests.js           # Test setup (optional)
│
├── .env                        # Environment variables
├── .gitignore                  # Git ignore file
├── package.json                # Project metadata and dependencies
└── README.md                   # Project documentation

Example

Explanation

public/: Contains static assets like the index.html file and images.

src/: The main source folder for your React application.

components/: Holds reusable components, organized into their own directories for better separation.
pages/: Contains components for different pages (like Home, About, etc.) used in routing.
hooks/: For custom hooks to keep your component logic organized.
context/: For Context API related files.
utils/: Utility functions or helper methods that can be reused across the project.
styles/: Global CSS styles or theming.
App.js: Main application component.
index.js: The entry point for the React app.

Root files: Include configuration files like .env, .gitignore, package.json, and documentation.

Next Steps

Adjust existing files and components to fit this structure.
Update import statements as needed to reflect the new structure.
Consider adding additional folders or files based on the specific needs of your project, such as tests or API service files.

Feel free to adapt this structure based on this project's specific requirements! If you have any particular areas you'd like more details on, just let me know!
After updating the existing files in the src folder, ensure everything works; otherwise, no points will be awarded.

@rko0211 is assigned for this issue, feel free to ask, please make sure this folder structure maintains industrial standards and the project should not have any functional problem after making it, it would run like before the same, if it not meet the standards I'll not give any point for this particular issue.

#37

Issue resolved!


Browser window is not focusing when opening link in other apps

Bug or Regression?
Bug
System Info and Version
No need.  Working with same config on older version.
Description
When opening links in Discord app, browser do not focus from other workspace.
So if browser is on other workspace, that I can not see, it not switches workspaces, so I do not know if link opened.
How to reproduce

Open Vivaldi browser on different workspace.
Open Discord client
Click on link on Discord.
Wait for link to open.
Workspace do not switch.

Crash reports, logs, images, videos
Hyprland, built from branch main at commit 8055b1c00a102f5419e40f5eddfb6ee8be693f33  (misc: Fix build warnings (6486)).
Date: Fri Jun 14 00:23:23 2024
Tag: v0.41.1-4-g8055b1c0, commits: 4822

Perhaps you can confirm the 'focus_on_activate' option?
https://wiki.hyprland.org/Configuring/Variables/#misc

@with9
Yes, I have it turned on.

No need. Working with same config on older version.

you meant to say this is a regression? If so, bisect. https://wiki.hyprland.org/Crashes-and-Bugs/#bisecting-an-issue

this regression has existed for a while at least since fe7b748


Release 1.4.0

/test-all
/test-windows-all
/test-ipv6-all
/test-ipv6-only-all
/test-whole-conformance
/test-flexible-ipam-e2e

Codecov Report

Merging #2967 (9975aa2) into release-1.4 (58eacd9) will decrease coverage by 1.02%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           release-1.4    #2967      +/-   ##
===============================================
- Coverage        59.69%   58.67%   -1.03%
===============================================
  Files              289      289
  Lines            24531    24531
===============================================
- Hits             14644    14393     -251
- Misses            8284     8585     +301
+ Partials          1603     1553      -50

Flag
Coverage Δ

kind-e2e-tests
45.12% <ø> (-1.51%)
:arrow_down:

unit-tests
40.24% <ø> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...g/agent/apiserver/handlers/featuregates/handler.go
0.00% <0.00%> (-82.36%)
:arrow_down:

...kg/apiserver/registry/system/supportbundle/rest.go
19.54% <0.00%> (-55.18%)
:arrow_down:

pkg/support/dump_others.go
0.00% <0.00%> (-51.73%)
:arrow_down:

...ver/registry/controlplane/nodestatssummary/rest.go
50.00% <0.00%> (-50.00%)
:arrow_down:

pkg/support/dump.go
8.13% <0.00%> (-49.60%)
:arrow_down:

...g/agent/apiserver/handlers/addressgroup/handler.go
0.00% <0.00%> (-40.00%)
:arrow_down:

...agent/apiserver/handlers/appliedtogroup/handler.go
0.00% <0.00%> (-40.00%)
:arrow_down:

pkg/apiserver/handlers/loglevel/handler.go
0.00% <0.00%> (-38.47%)
:arrow_down:

pkg/ovs/ovsctl/ofctl.go
19.10% <0.00%> (-17.98%)
:arrow_down:

pkg/ovs/ovsctl/appctl.go
32.60% <0.00%> (-11.96%)
:arrow_down:

... and 22 more

/test-all
/test-windows-all
/test-ipv6-all
/test-ipv6-only-all
/test-whole-conformance
/test-flexible-ipam-e2e

/test-e2e

/test-windows-networkpolicy
/test-flexible-ipam-e2e

There is something wrong with windows testbeds. "jenkins-windows-networkpolicy" and "jenkins-windows-ovs" succeeded and there is only documentation change after that.
The failures in "jenkins-flexible-ipam-e2e" will be tracked separately: https://github.com/antrea-io/antrea/issues/2979 and https://github.com/antrea-io/antrea/issues/2980. But as AntreaIPAM is an alpha feature and they are corner cases, I think we can fix it later.
However, I'm concerned that "jenkins-windows-conformance" failed consistently because Pod cannot access external network stably, while the Node itself can access it. I think we should understand the root cause first before releasing.

/skip-all
Tests have been done in https://github.com/antrea-io/antrea/pull/2989. Just run basic tests here.

/skip-all


Date auto convert to number

when I import data from excel, one column is a free text, in a row have value: 08/01/2016 but laravel excel get it like a number 42583.
help me fix it please!

@kenshin1102, you can see this reference http://www.maatwebsite.nl/laravel-excel/docs/reference-guide at column "Available column formatting"

I think "Available column formatting" work with export, it not work with import.

this is the still issue.
I think "Available column formatting" work with export, it not work with import.


feat: upgrade maven agent

What is this PR About?
This PR includes the changes required to upgrade the maven agent from version 4.6 to version 4.9

@gkmahendran09 In the settings.xml file we need to have a user credentials to pull or push from nexus. Right now we have dummy credentials but to make the agent work, we need to have a working credentials here. How can we plan this ?

@gkmahendran09 In the settings.xml file we need to have a user credentials to pull or push from nexus. Right now we have dummy credentials but to make the agent work, we need to have a working credentials here. How can we plan this ?

Can we get it from ENV variables?


Update to ReadMe Setup, Added Development section, fixed issue #446

This update streamlines and changes how the setup guide was written and splits it into 3 setups for 3 different software systems. It also removes unnecessary sections and places them within their own Development section. The old setup guide was deleted.
Issue #446 was resolved. A color change on hover was added as well as the word "(selected)" was added when choosing a language.

Hello! Thanks for your contribution!! :D
Could you split up this into two separate PRs to keep each one scoped --
One for the documentation updates
& one that fixes #446 ?
This will help us review the changes more easily and merge them in separately as well. We prefer to keep things separated into merge commits in case there ever needs to be a rollback.

Hi!
We have now done that, the language changes have been separated into another pull request

We have now done that, the language changes have been separated into another pull request

Thank you! When you have a moment, can you please update the PR title and description to remove references to the separated work? 💅


Problem filtering nested fields on Dashboard

Plugin version:
1.0.0
Kibana version:
5.68
Plugins installed:
None other
Description of the problem including expected versus actual behavior:
I have encountered a problem when doing drill downs in the graph. I have found that when you do a drill down using a non nested field, it works flawlessly. This is not the case in using drilldown with nested fileds, since it give 0 results.
I have also tried making my own filter for nested fields:
{
  "query": {
    "nested": {
      "path": "userClicks",
      "query": {
        "bool": {
          "must": {
            "term": {
              "userClicks.name.alias": "someName"
            }
          }
        }
      }
    }
  }
}

```In this filter I am trying to filter so that the graph show only the userClicks that correspond to an alias of "someName". When I apply the filter, it still shows userClicks that do not correspond to the alias "someName". What I find strange is that if I use the invert feature on the kibana GUI it actually works! That is if I use invert on the above query filter, it will show me all userClicks except "someName" (meaning that the filter works using invert).

After watching this behavior I tried the opposite filter to see how it worked:
`{
  "query": {
    "nested": {
      "path": "userClicks",
      "query": {
        "bool": {
          "must_not": {
            "term": {
              "userClicks.name.alias": "someName"
            }
          }
        }
      }
    }
  }
}
`
Again if I run this filter it should give me all userClicks that do not contain "someName" alias, but it doesn't work because it shows me userClicks that contain "someName". But when I use the invert funcionality in kibana it works with no problem. Meaning that using the above filter with invert only shows me the userClicks that have "someName" alias.

**Steps to reproduce**:
 1.Create nested documents with 2 levels of nesting.
 2. Create different visualizations and add them to the dashboard.
 3. Use drill down by clicking on a nested field.
 4. Use a filter like the one above.

This seems to be working in 6.2.4 and could be something specific to the 5.6.X version of the plugin. I'm in the process of fixing a few things in the 6.x stream that I'll be back-porting to the 5.x series.. I'll work on this when I back port the current set of fixes.

I'm in the process of back porting to 5.6.9 and resolving issues related to that... I will look at this as part of that work. Just wanted to let you know I haven't forgotten this...

I have just tried this with 5.6.9 and my regression dashboard that uses my scripts to populate nested data... the data is formatted like homes->cars[]->options[]
When I click through on an option in a pie chart, the behavior is as expected, and the following filter is created:
{   "query": {     "nested": {       "path": "cars.options",       "query": {         "match": {           "cars.options.name": {             "query": "V6",             "type": "phrase"           }         }       }     }   } }
The results may seem confusing since I see more than just V6 options in the result. I also see other options... and this is because these other options are part of an array of options associated with a car associated with the parent home document. If a match exists to any single nested object, then the entire parent document is used in the aggregation.
I'll continue to play with this as I get closer to releasing 5.6.9 and starting the back-port to 5.6.8.

I believe this problem may be resolved.. please grab the latest release and verify.

Please reopen if this is still an issue.


data.retrospective fails

As of today (18 Oct. 2024), data.retrospective from the geoglows library no longer works. I attempted to get streamflow data, like I have many times before, and today it stopped working with the error code in the screencaps below. I'm inputting a single stream ID. This worked perfectly fine yesterday. As far as I can tell, there have been no updates or changes to geoglows, so I'm guessing that something geoglows uses has been changed.

Thank you @knicely
We are aware of this issue and are in the process of solving it.

Just as an FYI, it seems like this issue is now resolved.
-Knicely


Use Byzantine Module for network discovery

CMO:
The midgard & thornode endpoints hard hard coded & will break if the thornode associated with the endpoint churns out.
https://github.com/Pusher-Labs/thorchain-explorer/blob/326cedf17383560b71286d04975bed397ed7dc35/src/environments/environment.prod.ts#L1-L5
https://github.com/Pusher-Labs/thorchain-explorer/blob/326cedf17383560b71286d04975bed397ed7dc35/src/environments/environment.ts#L5-L9
FMO:
Suggest using the byzantine module which will automatically discover the network and ensure a malicious node isn't serving bad data; and will select a node at random as the baseUrl.
https://gitlab.com/thorchain/byzantine-module

A static node is used only for local development. On anything live, it hits 'https://a2wva4alb6.execute-api.us-east-1.amazonaws.com/dev/thornode', which is a lambda that uses the asgardex-midgard library and connects to a random node 👍


Are notes to be displayed for uninhabited building?

@geoffj-FUG
Chas has said that notes should be displayed for uninhabited buildings. Could you confirm that this as a requirement or not.

I'm thinking of an entry that I cannot, of course, now find .. where it was recorded as being 'The home of  Col. John Smith and family' or some such - for my purposes, this was very useful informatin (I was scrolling through trying to find 'the big house' in a village, not knowing who actually lived there (trying to identify a potential father for my illigitimate grandfather).  Similarly, anyone working on the history of a house or place would want the complete information.

Kirk
The notes field should be displayed for every record. It is where we enter comments for the researcher to see. These notes can cover a range of things that are not normally captured, such as enumerator’s notes.
Geoff
From: Kirk Dawson<EMAIL_ADDRESS>Sent: Friday, 9 October 2020 12:21 AM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [FreeUKGen/FreeCENMigration] Are notes to be displayed for uninhabited building? (#1006)
@geoffj-FUG https://github.com/geoffj-FUG
Chas has said that notes should be displayed for uninhabited buildings. Could you confirm that this as a requirement or not.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/1006 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFJR5ATUASDVDHIDR7LSJXDF7ANCNFSM4SI2LT2Q .  https://github.com/notifications/beacon/AKCPIFLGJ6HUW4NSRA44IG3SJXDF7A5CNFSM4SI2LT22YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4KWCQR6A.gif

Currently the notes field is not processed if the uninhabited flag is set hence at this point there is no information to display.
Implementation requires a change to CSV Proc to always process the notes field and then to display.
Perhaps we should only display the field if it contains information

Kirk
I am happy to display the notes field if it contains information. We get notes against all sorts of fields.
Geoff
From: Kirk Dawson<EMAIL_ADDRESS>Sent: Saturday, 10 October 2020 10:50 AM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FreeUKGen/FreeCENMigration] Are notes to be displayed for uninhabited building? (#1006)
Currently the notes field is not processed if the uninhabited flag is set hence at this point there is no information to display.
Implementation requires a change to CSV Proc to always process the notes field and then to display.
Perhaps we should only display the field if it contains information
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/1006#issuecomment-706457696 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFPK6LDSYNB7QH4OVP3SJ6VVFANCNFSM4SI2LT2Q .  https://github.com/notifications/beacon/AKCPIFIRETLIWDFHQGB5JV3SJ6VVFA5CNFSM4SI2LT22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFIN3AYA.gif

Thanks Kirk
Tests OK
Geoff


fix: Create getTreeName function to format tree name

Issue 64
This PR changes the way items are listed in the Findings menu in VSCode.
New Pattern: [Rule] in [context], [file:line]

Scope is more like the most top level event within which the problem
occurs.
I’m not sure which is the best. We would probably need to look at some
examples.
On Wed, Jun 22, 2022 at 6:17 PM Max Dillon @.***> wrote:

@.**** commented on this pull request.
In src/tree/findingsTreeDataProvider.ts
https://github.com/applandinc/vscode-appland/pull/409#discussion_r904285446
:

@@ -2,6 +2,11 @@ import { Event } from @./models';
import * as vscode from 'vscode';
import FindingsIndex from '../services/findingsIndex';
import { ResolvedFinding } from '../services/resolvedFinding';
+import { Finding } from @./scanner/built/cli';

+function getTreeName(finding: Finding) {

return ${finding.ruleTitle}: ${finding.groupMessage || finding.message}, ${finding.stack[0]};

There was another property of the findings object called scope:
"scope": {
"static": true,
"id": 1,
"event": "call",
"thread_id": 1,
"defined_class": "misago.faker.tests.test_fake_categories",
"method_id": "test_fake_category_can_be_created_with_copy_of_other_category_acls",
"path": "misago/faker/tests/test_fake_categories.py",
"lineno": 18
},
I initially didn't use it because it wassn't in all my test findings.
Would this be the best path to use? Then I could fallback on the stack?
—
Reply to this email directly, view it on GitHub
https://github.com/applandinc/vscode-appland/pull/409#discussion_r904285446,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC63HX4HFNEHGECSTMSTVQOGF3ANCNFSM5ZR473MA
.

On Thu, Jun 23, 2022 at 2:12 PM Dustin Byrne @.***>
wrote:

@.**** commented on this pull request.
In src/tree/findingsTreeDataProvider.ts
https://github.com/applandinc/vscode-appland/pull/409#discussion_r905324136
:

@@ -2,6 +2,11 @@ import { Event } from @./models';
import * as vscode from 'vscode';
import FindingsIndex from '../services/findingsIndex';
import { ResolvedFinding } from '../services/resolvedFinding';
+import { Finding } from @./scanner/built/cli';

+function getTreeName(finding: Finding) {

return ${finding.ruleTitle}: ${finding.groupMessage || finding.message}, ${finding.stack[0]};

Use path and lineno from the event object. scope won't be accurate for
this case.
In some cases (like a query-from-view finding), the next best strategy
would be to use the first available path in relatedEvents.
—
Reply to this email directly, view it on GitHub
https://github.com/applandinc/vscode-appland/pull/409#discussion_r905324136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC63LX7Z6GZ6EGFM5GJLVQSSHFANCNFSM5ZR473MA
.
You are receiving this because you commented.Message ID:
@.***>

Use ResolvedFinding.problemLocation ; it's there for this
https://github.com/applandinc/vscode-appland/blob/dd4004269f5c452a58064debf11639eb0c149ec3/src/services/resolvedFinding.ts#L39
On Thu, Jun 23, 2022 at 3:34 PM Max Dillon @.***> wrote:

@.**** commented on this pull request.
In src/tree/findingsTreeDataProvider.ts
https://github.com/applandinc/vscode-appland/pull/409#discussion_r905388676
:

@@ -2,6 +2,11 @@ import { Event } from @./models';
import * as vscode from 'vscode';
import FindingsIndex from '../services/findingsIndex';
import { ResolvedFinding } from '../services/resolvedFinding';
+import { Finding } from @./scanner/built/cli';

+function getTreeName(finding: Finding) {

return ${finding.ruleTitle}: ${finding.groupMessage || finding.message}, ${finding.stack[0]};

There are some examples like this
"scope": {
"http_server_request": {
"request_method": "POST",
"protocol": "HTTP/1.1",
"path_info": "/admincp/graphql/",
"normalized_path_info": "/admincp/graphql/",
"mime_type": "application/json",
"headers": {
"Cookie": "sessionid=bkydqv3eodpvl0zp98t64cuguyx357vh",
"Content-Length": "776",
"Content-Type": "application/json"
}
},
Here there is an HTTP request instead of a path.

I could use another method to get the path in this case
I could use the HTTP Request path instead: e.g. "/admincp/graphql/"

—
Reply to this email directly, view it on GitHub
https://github.com/applandinc/vscode-appland/pull/409#discussion_r905388676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC62GNRPGAOW2GB7X4Y3VQS35TANCNFSM5ZR473MA
.
You are receiving this because you commented.Message ID:
@.***>

Screenshots


Fix/1467 auth no prompt

Describe the changes you have made in this PR

fix non-prompt login
minor refactor

Link this PR to an issue
Fixes #1467
Type of change (Remove other not matching type)

fix: Bug fix (non-breaking change which fixes an issue)

How has this been tested?
Manually login to SN with and without allowance

this could have been DRYed up.

Happy for a hint

[a,b,c].forEach((attr) => { do things })


New package: MCTS v0.4.4

Registering package: MCTS
Repository: https://github.com/JuliaPOMDP/MCTS.jl
Created by: @zsunberg
Version: v0.4.4
Commit: 3e24ed36c360fd211a78a44fe48041cafc4ed40f
Reviewed by: @zsunberg
Reference: https://github.com/JuliaPOMDP/MCTS.jl/commit/3e24ed36c360fd211a78a44fe48041cafc4ed40f#commitcomment-42128993

[noblock] Explanation for version number in #20728 . This package was named a while ago. While we agree that it is not the best name, there are many other (as yet unregistered or registered in the JuliaPOMDP) projects that have built on it, so we would like to register it with this name.


Allow replacing entire components

Allow replacing an entire component from an output, if a component output's property value is empty.

@forman , what is the reason behind this issue? What problem are we solving here?

I have the following use case which should be typical:
If have a button supposed to start processing data. It has id "update". Once users click it, processing starts and I want to replace the button by a circular progress (of the same size) using the same id "update". As you can see, no properties involved.


Improve performance by rendering markdown partially

Use MarkdownIt.parse to implement this.

Or we can try using another markdown parser like marked.
This is because MarkdownIt.parse's result is somehow not ready to render: it produces a list of tokens, but we want a nested tree. Although transforming that should be easy enough, it may hurt performance.


NativeClass: hide/rename internal methods with potential naming conflicts

Fixes #885.

bors try

Adding new methods to a trait is technically a breaking change, as implementors now need to change their code. This is not an issue as long as #[derive(NativeClass)] is used, however there might be very rare cases of manual implementations of NativeClass (?)...
If we consider this a problem, I can change the implementation so that:

the new nativeclass_* methods have a default implementation, delegating to the old ones
godot-rust code still calls the old register_properties and init methods (keeping existing manual NativeClass impls working)

Like with the #[doc(hidden)] change, the central question here is: should users be able to manually implement NativeClass? If yes, then we need to keep documenting all trait methods and cannot really change anything until 0.11. This means however all "wiring-up code" is effectively public, which restricts internal changes, as we see here.

Thank you for the quick action on this. Personally, I don't think this should be too big of an issue as far as API breaking changes go, for probably 99% of users, NativeClass is derived from the proc macro.
I think that the method being changed to nativeclass_register is a great change.

Given the low probability of this bug, it's probably safer to not risk breaking changes, even if that's unlikely. I went with a simple deprecation + a comment for later renaming + change of the example (register_properties -> register_members).
I also realized that NativeClassMethods::register() likely suffers from the same issue. We might need to have a general discussion if people should be allowed to manually implement NativeClass and the other traits, or if we should generally discourage that (i.e. treat it like a private API in all aspects, such as not documenting it, ignoring SemVer, choosing hard-to-collide but cumbersome identifiers, etc.).
Also sneaked in two unrelated small changes (nightly clippy + rustfmt edition in codegen).
bors try

bors r+


Dice rolling in slack threads

I hope this is the right place to make requests. Me and my friends use the dice rolling app in slack for a D&D game we are playing and it is a great app. It would be extremely helpful to be able to use the dice rolling app in the slack threads you can start when someone posts something. Currently you can only use /roll in the main chat channel. Is this possible? Sorry to bother you about it but I don't know the first thing about coding.

As best I can tell from https://api.slack.com/slash-commands and other non-first-party commands (e.g. /giphy) this isn't currently an option. Happy to add if that changes 👍

I found these webpages that seem to say that slack now supports apps in threads? It seems to give instructions on how to program an app to take commands from a thread? Again I know very little about coding so I possibly don't know what I'm talking about 😋.
https://api.slack.com/docs/message-threading
https://medium.com/slack-developer-blog/bringing-your-bot-into-threaded-messages-cd272a42924f

The main issue is that those are for full bot integrations, which require permission to read and write messages while the dicebot only asks for permission to add commands (and thus doesn't need to be explicitly invited into channels, or have the ability to read messages not prefixed with /roll)
Something like Hubot has that ability (note thread support request https://github.com/slackapi/hubot-slack/issues/393) but not the simpler command API being used, I believe.

Ok I understand now. Thanks for your time.


Treemap experimentation

Summary
Description of PR here...
Closes #179
Changes

Added buttons to toggle between financed emission and net asset value treechart

Screenshots

TODO:

[ ] Have left button stack on top of right button shadow (@zorazrr any advice on this is greatly appreciated)
[ ] Add discription for the difference between the two treecharts (waiting for @faithlos on the figma)
[x] Make the title names cleaner (right now they are ugly and too technical)
[ ] Check with design professional on dimensions of the chart (@faithlos)
[ ] Check with design professional on size of company title       (@faithlos)
[ ] Clicking a sector zooms into a treemap of sector                    (@faithlos can we get a design so this is clickable?)
[ ] Hovering over sector name triggers toottip that gives the percentage breakdown of the given sector
[ ] Have back button work when clicked from treemap
[ ] Merge Dev DB to Prod DB

@zorazrr the previous commit has it take up the correct portion of my screen for the given aspect ratio, how can I make it aspect ratio agnostic
?

With the most recent commit, I sometimes get a hydration error

I don't know why :(

I think the solution to the shadow overlapping is here https://www.w3.org/TR/CSS2/visuren.html#z-index, but I'm struggling with the code implementation.

The hydration error seems to be caused by using the ToolTip inside the PrimaryButton

The hydration error seems to be caused by using the ToolTip inside the PrimaryButton

I fixed this by unnesting the button and the tooltip

Vercel had trouble deploying EmissionTreeMap:
Type error: Argument of type '({ companyName: string; companyId: string; financedEmissions: number; netAssetVal: number; fossilFuelClass: string; } | undefined)[]' is not assignable to parameter of type '(SourceData | null)[]'.

There is also merge conflict.

The vercel error should be fixed by commit 5c17c427828f108dfdd70e637d84b6bd6aa1bcea, and the merge conflict is resolved in commit e746b4aca1d4f3ab65e57d7d94a61c808bf23466

Just a remind to switch the dev db over to the prod db as I made some changes.

Just a remind to switch the dev db over to the prod db as I made some changes.

Wait sorry please DO NOT merge in the dev db to prod db. I need to reset it.

One more deployment error -- this is in fossil-fuel/index.tsx.
Type error: Type '{ img: Asset; caption: Document; }' is not assignable to type 'IntrinsicAttributes & UniversityInvestmentsProps'.
--
Property 'img' does not exist on type 'IntrinsicAttributes & UniversityInvestmentsProps'.

In the future if you branch directly off of the toriis repo, the Vercel workflow will be able to go through and highlight the errors.

One more deployment error -- this is in fossil-fuel/index.tsx.
Type error: Type '{ img: Asset; caption: Document; }' is not assignable to type 'IntrinsicAttributes & UniversityInvestmentsProps'.
--
Property 'img' does not exist on type 'IntrinsicAttributes & UniversityInvestmentsProps'.

In the future if you branch directly off of the toriis repo, the Vercel workflow will be able to go through and highlight the errors.

Commit 7bbf35e7f0944b4a6af9e4ece07609362932e1dd should fix this.
p.s. I will branch off toriis from now on :vulcan_salute:


Do not stop collecting metrics if the templates endpoint is not reachable

What does this PR do?

Do not stop collecting metrics if the templates endpoint is not reachable. This was introduced in https://github.com/DataDog/integrations-core/pull/14569
Motivation

In some cases we can get a 403 on this endpoint. The check should continue collecting the other metrics.
Additional Notes

This will be backported to 7.46
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

Just an observation that we can catch specific exception and handle them instead of a blanket exception handler.

Yes, but it depends on what you mean exactly by handle in this case. Right now I'm not sure this is recoverable so I'll keep it that way for 7.46, we can try to improve that for the next version. WDYT?


Allow custom sorting

This came up with a client who is trying to use the table sorter for a column with currency, eg. £10.10. As this is being used in a Drupal site, and the WYSIWYG's table editor does not support custom attributes for table cells, it would be too much pain for them to edit each cell to add data-sort-value="10.10" etc.
A possible solution would be for the library to support custom sort functions so users could specify a new data type, eg, currency, and we can augment the library with a function for that, which can simply use regex etc for the data sort values.
I've seen this in another library: http://tablesorter.com/docs/example-parsers.html
Thank you!

https://github.com/joequery/Stupid-Table-Plugin#creating-your-own-data-types

We got you covered! :)

Great, thanks!


Connection Bar for xfreerdp same as windows mstsc client

I started to work on this. See https://github.com/mmattes/FreeRDP/tree/feature/fullscreen-bar for an initial version. It already can do close, minimize or toggle full-screen and resizing and dragging it around also works. I still do have stuff to do on it to get it ready to be pulled in e.g. auto-hide or make sure that i fulfill the coding guidelines of the project.
BTW: this is actually the first time i am coding something in C. Any feedback if i made somewhere major mistakes in regards to coding in C would be appreciated.

@mmattes Looks good overall, just 2 little hints:

We try to stay compatible with older compilers, so no C++ comments or variable declarations mid code
Use ./scripts/format_code.sh (astyle) script which formats code according freerdp standards

@mmattes Good work can you make it configurable from freerdp commandline

@mmattes active by default and command line switch to disable it will be good option

@gnshkl1 have a look at https://github.com/mmattes/FreeRDP/commit/37d84751b331cac1e53f3bac66f2a0b382f1131c where i added a new option to disable the floatbar. But i got some queries.

settings.h seems to be, at least partially, auto generated. I couldn't figure out in detail ho,  why i am not sure if the new option i added is correct?
cmdline.h seems to be common for all clients, i have seen that there is a floatbar in the windows client why i also implemented changes there to apply the new option, but I cant verify them. See commit https://github.com/mmattes/FreeRDP/commit/20316cacdeeacb8f9825a11244418ebdea387397

Totally forgot, i have no devices to test if my floatbar implementation within the X11 client is working properly in multimonitor mode and not shown at all when it is a RemoteApp. Can anyone verify that?

Hi. How to disappear connection bar on Thinstation (xfreerdp)? Is there some parametr? Thanks.


Remove underscores from module name (i.e. cstar_ocean -> cstar)

Supercedes #35, by insteading building on top of #41. (So should not be merged until after #41 is merged)

[x] Closes #7

Still need to update the README.md

This should be good now too.
I wasn't totally sure if we need to change anything else in setup.py but we can worry about that when we actually publish it to PyPI

Thanks @TomNicholas . I think we can keep underscores in module files on grounds of "improves readability" and would be consistent with Case attributes - so if we can move basemodel.py back to base_model.py (to reflect Case.base_model) this can go straight in by the looks.


Strange behaviour if gravity is low ?

I seen somewhere that high friction values can casue bugs or strange movement (also tested it a little) but now I set gravity to 0.1 and my circles act very strange. Im not sure it's because gravity but this is the only factor i can see.
Fiddle with example
How to fix it? and actually what cause this strange collision? is that bug or I did something wrong?

0.1 is actually a very large value for the gravity. I think the default is 0.001.
Having such a high gravity has your bodies accelerate very quickly to high velocities, so you get bugs like bodies flying through walls/ground because they just 'teleport' past them in 1 tick.
So, lower the gravity even more since it isn't "low" like you said in the title.
:)

No way, look at docs, default gravity for Y axis is 1
World.gravity.y
- The gravity y component.
- Default: 1

This is actually way lower than default and all demo pages has gravity at 1.
There is also gravity scale factor what is 0.001 but I didnt touch this.
World.gravity.scale
- The gravity scale factor.
- Default: 0.001

I always modify the World.gravity.scale, that might be why.

but still this exists anyway. I see, this happen when bodies moving slowly so it can be that low gravity helps to see it because simply bodies move slow.

Hey looks like your issue is mostly the same as #63?

I think yes.
It looks like body "collide" as long as bounds of the body collide.

Thanks for the additional example, it's something I need to figure out a fix for! For now try to avoid things that will make the bodies move very slowly.


cpu profiler doesn't work correctly in multi-thread programming.

Hi, My os is centos7 and i test gperftools using std::thread in c++11, but the result is confused!
My program:
#include <google/profiler.h>
#include
#include
#include
#include <unistd.h>
#include
std::mutex m;
void t(int id) {
ProfilerRegisterThread();
ProfilerStart("test.prof"); // test.prof is the name of profile file
for(int i = 0; i < 1000; ++i){}
m.lock();
std::cout << "i'm thread " << id << std::endl;
m.unlock();
ProfilerStop();
}
void test() {
std::vectorstd::thread pool;
for(int i = 0; i < 10; ++i) {
pool.push_back(std::thread(t, i));
}
for(int i = 0; i < 10; ++i) {
pool[i].join();
}
sleep(2);
}
int main() {
test();
return 0;
}
And use cmake to compiler it.

At the same time, i set  CPUPROFILE="/tmp/prof.out".
After run the binary, i get a /tmp/prof.out output which is not empty, but only has the tail address information. The binary header is empty!
I don't know why, please help me.

Your program finishes too quickly. gperftools profiler needs at least several seconds of cpu time to get you profiler of at least some precision. Closing as won't fix.


fix minor typo in command in benchmarks/readme

the flag needs two - (or -c instead) minor thing but might as well fix it

@swift-ci smoke test and merge

@swift-ci smoke test and merge

@swift-ci smoke test Linux

@swift-ci smoke test Linux


Added Upper FS BC PV

Fixes #876

My difficulty with this group is that they are quite varied. See Pvalb Egfem1, Pvalb Gpr149, Pvalb Kank4, Pvalb Calb1_1, and Pvalb Calb1_2 in the Nature paper table
. What I did was find the commonality between the descriptions and put that down, hence a bit less descriptive.

Hold up on this one until we have more discussion of electrophysiological groupings. If we are happy with differentium 'upper fast spiking' this needs to be in the first sentence of the text def.  These are so morphologically variable, I'd drop morphology details from the def altogether.  Strong case for making subtypes by morphology.

I would like to have a discussion about the strategy for these ephys groupings as well.


Unable to creating Bot Channels Registration for BotService

Hi, I am trying to create Bot Channel Registration using Azure SDK for .Net.
For this, I have used Microsoft.Azure.Management.BotService package with below method.
// Create Bot
Bot createdBot = await botServiceClient.Bots.CreateRegistrationBotAsync(resourceGroupName, botServiceName, bot);

This method currently creating Web App Bot instead of Bot Channel Registration.
Issue-2
As well as In this package 2 external packages used but Its version was written as exactly equal (=), It should be greater than equal (>=) .
    <PackageReference Include="Microsoft.IdentityModel.Clients.ActiveDirectory" Version="[3.14.1]" />
    <PackageReference Include="Microsoft.IdentityModel.Tokens" Version="[5.1.4]" />

I have also tried with API Version: 2018-07-12, but this time got
The Resource 'Microsoft.BotService/botServices/dev-bot-123-c' under resource group 'dev-bot-123' was not found.

Here request URL is
https://management.azure.com/subscriptions/996b68f4-6fff-41a7-b5df-9ec7e3b8dc63/resourceGroups/dev-bot-123/providers/Microsoft.BotService/botServices/dev-bot-123-c?api-version=2018-07-12

Looks like / in Resource type name creates an issue. Here Resource type is Microsoft.BotService/botServices

Closing this out as there has been no recent engagement.  Please feel free to reopen or file a new issue if this remains a problem with the current version of the library.


train_dataset

Hello author, I can only find eval dataset, Could you share me the link of train dataset or How to access it?

In each folder, query.json is for evaluation, and support.json is the support (train) set for few-shot demonstration.

Thank you very much. By the way, where can I find information on Fast Counting & Fast Attribute Matching?

These two subsets were added in our newer version. We haven't released them yet but will aim to do so ASAP.

Ok, Thank you very much


Add descriptions to priority queue

Change Description

Add description to the priority queue for debugging and profiling.
The overhead when the description is not used should be minimal.
Includes change to txn_text_gen_plugin to create two new accounts and to log private keys.
Depends on https://github.com/EOSIO/appbase/pull/50

Consensus Changes
None
API Changes
None
Documentation Additions
None

Includes change to txn_text_gen_plugin to create two new accounts and to log private keys.

Why?

Why?

As just a simple way to have two accounts with the same key. Log the keys so it is easy to grab the keys for use by other manual tests.

Going to close this for now. It was more for testing/debugging purposes.


update readme and fix internvl2

update readme.md
fix internvl2 for V100

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.luyao-cv seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


add refresh button on accordions of subscription configurations

This is to enable the manage button right after the configuration is created, and to show the current number of the subscriptions (see #536).

Done https://github.com/nuvla/ui/pull/562


ConsoleAPI experiment override PATCH/DELETE

Add documentation for the PATCH/DELETE endpoints mainly taking from the POST endpoint

🚀 Deployed on https://63e16b0cef8a74092fe49572--cozy-fox-0defec.netlify.app