[FEATURE] Auto-Swap Carousel for Club Cards and Upcoming Events

Is your feature request related to a problem? Please describe.
Currently, users may have to manually navigate through cards displaying club information and upcoming events, which can be inconvenient and disrupt the browsing experience. An automated carousel would allow for a smoother presentation of content without requiring user interaction.
Describe the solution you'd like
An auto-swapping carousel that cycles through club cards and upcoming event cards. The carousel should have adjustable speed and pause on hover, allowing users to focus on specific details if needed. This feature would enhance the user experience by providing a dynamic, continuous display of important information.
Screenshots

Check List

[x] I am contributing under GSSoC-ext
[x] I am contributing under hacktoberfest
[x] I follow contribution guidelines

Is your feature request related to a problem? Please describe. Currently, users may have to manually navigate through cards displaying club information and upcoming events, which can be inconvenient and disrupt the browsing experience. An automated carousel would allow for a smoother presentation of content without requiring user interaction.
Describe the solution you'd like An auto-swapping carousel that cycles through club cards and upcoming event cards. The carousel should have adjustable speed and pause on hover, allowing users to focus on specific details if needed. This feature would enhance the user experience by providing a dynamic, continuous display of important information.
Screenshots

Check List

[x]  I am contributing under GSSoC-ext
[x]  I am contributing under hacktoberfest
[x]  I follow contribution guidelines

great but i have a doubt that:
In this carousal will the user be able to scroll back to previous event by hand??

@Yash-Ainapure, based on your requirements, it is possible to enable both manual scrolling to previous events and automatic scrolling with a fixed time interval. If you prefer manual left-right navigation, that is also achievable. Please let me know your exact requirements, and once assigned, I'll start working on it.

sorry to say but i think he current UI is good for the website and we shouldn't change it.
you could contribute to other issues.
thanks

There are missing a file CONTRIBUTE.md , so I can raise a issue #169. Assign me with proper labels.
Also we can add a chatbot that helped new user to understand our project. Really this is helpful to any new users that first time visiting this web site ..i trained the chatbot according to our project. so it really helpful. Issue no. #170


add technology field to svg

add technology field to svg to help with C4 diagnostic

Looks great, thank you!


RestoreSnapshotModal - ch29948/restore-modal-requests-app-slug-which-may

Added clarifying copy in RestoreSnapshotModal.
I originally was trying to get to this modal organically using the product, but overall my speed to add this change was pretty slow. I've added storybook to the web project so we can iterate on stateless components faster.

 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.

 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.

@jgruica or @GraysonNull Can one of you review this PR please? Thanks!

@jgruica or @GraysonNull Can one of you review this PR please? Thanks!

@GraysonNull i've updated the copy!

@GraysonNull i've updated the copy!

Great, thank you!. One other question, why is there four "remove storybook" commits? That feels...odd

Great, thank you!. One other question, why is there four "remove storybook" commits? That feels...odd

Great, thank you!. One other question, why is there four "remove storybook" commits? That feels...odd

Sorry about that! Every commit i thought i removed storybook, there was something lingering. Would you like me to reopen a PR without those commits?

Great, thank you!. One other question, why is there four "remove storybook" commits? That feels...odd

Sorry about that! Every commit i thought i removed storybook, there was something lingering. Would you like me to reopen a PR without those commits?

Ah gotcha. No it's ok, maybe just squash before merging. I was just wanting to make sure there wasn't anything messed up with the git history

Ah gotcha. No it's ok, maybe just squash before merging. I was just wanting to make sure there wasn't anything messed up with the git history

@GraysonNull can you squash and merge when you get a chance?

@GraysonNull can you squash and merge when you get a chance?


[INFINITY-1810] Add integration tests for verifying metrics

Introduces a new utility package for testing functionality of metrics.

retest this please

@NimaVaziri Hey, sorry, this PR has been littered with fault status's Do not try rerunning the ones linked here.

@jsrodman addressed comments


Context.term hangs but Context.destroy does not

I have some sockets in a context and have a LINGER time of 5 ms. I have closed all the sockets. But when i call the term method on the context it blocks and the code does not move ahead. But when i call the destroy method the code does not block. And after calling destroy i can call term and the it does not block now. But destroy is deprecated in favour of term so i think i should not be using it. Any ideas what i am missing, i am on pyzmq==16.0.2.

Where do you see that context.destroy is deprecated? destroy closes sockets and then calls term. term just terminates the context, which will block until the sockets are closed. So if term is blocking, that means there are still open sockets somewhere waiting to be closed.

@minrk But LINGER time is set to 5 ms, so in after 5 ms the term should not block even after there are pending sockets. And here it mentions that destroy is deprecated by zmq_ctx_term, http://api.zeromq.org/4-2:zmq-ctx-destroy. There is a code snippet that has 2 sockets, one acts as a listener and the other a sender, and if i dont close any of those sockets, then term hangs, but destroy does not
def test1():
    print('On zmq_version {}'.format(zmq.zmq_version()))
    print('On zmq.__version__ {}'.format(zmq.__version__))
    port = 8876
    ctx = zmq.Context.instance()
    sock1 = ctx.socket(zmq.ROUTER)
    sock1.setsockopt(zmq.LINGER, 1)
    sock1.bind('tcp://*:{}'.format(port))
    sock2 = ctx.socket(zmq.DEALER)
    sock2.setsockopt(zmq.LINGER, 1)
    sock2.connect('tcp://<IP_ADDRESS>:{}'.format(port))
    sock2.send(b'm_1')
    sock2.send(b'm_2')
    print(sock1.recv())
    print(sock1.recv())
    sock1.close()
    sock2.close()
    ctx.term()
    print('going down')

test1()

Now if i comment you any one of  sock1.close() or sock2.close(), ctx.term() will hang and you will not see going down printed. Here is my output,
On zmq_version 4.1.6
On zmq.__version__ 16.0.2
b'\x00k\x8bEg'
b'm_1'
going down

LINGER governs discarding messages after sockets are closed, not closing sockets. Term will always block until all sockets are closed.

@minrk Thanks for the explanation, looks like i was missing closing a socket in my original code and since destroy is deprecated, i'll go find out which socket is not closed. Thanks again.

I was creating new context in multiple objects in the same process so was getting the same context. So when i closed all sockets of the object and called context's term, it froze because there were other objects which had socket bound to the same context which were open. So for now i will take of closing the sockets and let the context termination be handled by the library

@lovesh where do you see that destroy is deprecated?

http://api.zeromq.org/4-2:zmq-ctx-destroy

Ah, that's confusing. pyzmq added a destroy method before zeromq. It was added as a convenience method for closing sockets and terminating the context (it does more than just term). Then zeromq decided to add an alias for zmq_term as zmq_ctx_destroy, which does the exact same thing a term. The deprecated destroy is the alternate name for term in libzmq that has been reconsidered, but the pyzmq method is not deprecated.
I guess I should give the pyzmq method a new name (perhaps Context.close_and_term()).


Not blocking facebook after you are already there

I've just tried the extension and it works great but for me it only delays facebook once, when I open the page, and then lets me browse it as long as I want without delays (as long as I stay in the same tab). This is quite "dangerous"! :)  Once you wait 15 seconds to get to the site you can lose hours there.  I'm using Chrome 20.0.1132.34 beta-m.

I'm having the opposite issue in the current version, i.e. I'm blocked again every time I navigate even within the same tab and I don't want to be. The plugin is exactly what I need to keep myself working, but once I decide I'm ready for time wasting I don't want to disable the plugin to use social media. Is there some way to disable the repeat blocking behavior?


Feat: Next 13 - Home - Incentives

What's the purpose of this pull request?
Migration of the Incentives section from the pages router to the app router (Next 13).
How it works?

How to test it?

Run yarn dev
Make sure that all the current pages are working as expected (/, /office and some /<slug>/p);
Make sure /fs-next-update is rendering with the Incentives section without errors.

Starters Deploy Preview

References

Jira task

@lariciamota Do we really need of use client in this section?  Is it possible to override Incentives Section?

@pedromtec It is only needed on the /sections because of the useOverrideClassName inside the Section component, but not on /ui, so I've removed it from there! Thanks


[Apple Reminders] Add support for smart lists

Extension
https://www.raycast.com/raycast/apple-reminders
Description
I have a smart list dedicated to my work tasks that are due within the next three days. I would prefer to focus mainly on the tasks in that list rather than the default 'Today' or 'Scheduled' lists.
Who will benefit from this feature?
Anyone who leverages Apple's smart lists.
Anything else?
No response

Thanks for the feature request. Unfortunately, Apple doesn't return smart lists in their APIs. Smart lists are a complex feature to replicate given the number of rules involved, so I'll close this one as "Not Planned."


Task: Create XCFramework of PactMockServer with only a subset of static libpact_ffis

❕ Problem Statement
PactMockServer exposes the wrapper around libpact_ffi in a XCFramework package. But the libpact_ffi binaries build from Rust code (https://github.com/pact-foundation/pact-reference/tree/master/rust/pact_ffi) contains binaries for simulators and physical device. The problem is that all these binaries are static and include the rust runtime code for each of the platforms we're supporting (x86_64, aarm for each simulator and physical devices). These binaries are huge! They hover at just over 100MB of each. That also means a lot of bandwidth and long time to fetch PactSwift package. It also uses up unnecessary disk space on developers' machines.
Although it's nice to not worry about running Pact tests on a specific target, it doesn't really make sense to run them on a physical device since it doesn't make much sense trying to find where the Pact contract has been written to on the physical device and trying to extract it.
We can consider just failing terribly when a developer tries to run Pact tests on a physical device, or "gracefully" fail the test with a meaningful message.
surpher/PactSwift allows developers to run tests on a physical device, but skips writing the Pact contract onto iDevice's disk.
💬 Task Description
Remove support to run on physical devices.
👩🔧 Technical Design Notes
🤝 Relationships

Other Related Issues:

Another option to reduce the libpact_ffi size for macOS targets would be to share a dynamic lib? But that would mean developers would also need to install rust on their machines which could prove as a big barrier to adoption of PactSwift.


New line characters miss, published from Windows Obsidian

Can somehow fix that?

found it works just if there are 2 new line characters one by one

Hi, I believe that standard Markdown ignores single line break. To get a consistent behaviour in Obsidian, you can turn on the Strict line breaks option.

Thank you!


Site just showing "Cannot GET /" after setup

Could anyone help point me in the right direction

@Jegard Sounds like you might be trying to navigate to localhost:8080 directly. You shouldn't need to.  The dev server is only used to serve assets.


Saving trained network

I am training on the 4class IV_2a dataset as in your folder examples. For further applications, I would like to store the trained network. I tried saving the variable exp (line 158) with pickle, which gave me following error: Can’t pickle local object ‘run_exp..’

I could save it using a package called dill. It is similar to pickle, but also allows to store local objects.
I am trying to retrieve the trained weights from the network for further computation. What is the easiest way to do so?


Bundled manifest does not allow updates

The module.zip release artifact which is downloaded on installation includes a module.json where the manifest url is locked with that version instead of being a url pointed at the latest manifest.
The manifest should instead point to https://github.com/mrprimate/ddb-importer/releases/download/latest/module.json.
You have the correct url in your "module.json" release artifact, but it is not correct in the module.zip artifacts.
Here's a rather robust article on the wiki with some more details if you're curious.

Yep, this is corrected in recent versions, but I have not updated some historic artifacts

Magnificent, thanks!


Improvement: Offer Bower and NPM Packages

I would like to see CoreUI offered as Bower and NPM packages. This would expand options for front-end and back-end integration.

I'm surprised there is no bower option to install CoreUI.

We decided to move different versions to separate repositores, so they will support bower and npm soon.

Angular Version (Angular 2+).
AngularJS Version.
HTML5 AJAX Version.
HTML5 Static Version.
React.js Version.
Vue.js Version.

Bower is deprecated. One should use npm. Bower recommends also Yarn.
Separating versions is very good idea!


make subdoc functionality work with embedded view

@zfletch the subdoc functionality was causing the embedded view not to display. This fixes it (although maybe the better change is to have a base Publication component that EmbeddedPublication extends..?)

@balmas thanks for finding this bug! It was a pretty big oversight and wasn't caught by the tests since they don't test vendor code.
I'm thinking that the best way to solve it would be to update the code in arethusa.min.js to only call arethusaSubDocFun if it exists (https://github.com/perseids-publications/treebank-template/pull/14). That would mean parts of the code that don't care about subdoc (like the embedded view) don't have to worry about it.

Ok, I'll defer to you on what you think best here. (The Arethusa changes still aren't in the widget branch of that code base right?)

That's correct, these changes are local only right now and not in the Arethusa repo.


where should I put the sd_xl_base.yaml？

where should I put the sd_xl_base.yaml？
Error occurred when executing DiffusersPipelineLoader:
HTTPSConnectionPool(host='raw.githubusercontent.com', port=443): Max retries exceeded with url: /Stability-AI/generative-models/main/configs/inference/sd_xl_base.yaml (Caused by NameResolutionError(": Failed to resolve 'raw.githubusercontent.com' ([Errno 11004] getaddrinfo failed)"))

or how to fix this Error


"use network as is" when signing into wifi doesn't stay connected

Could be related to https://www.reddit.com/r/ProjectFi/comments/3rmolt/use_network_as_is/

I found I can connect but in about 2 minutes, although it stays connected it doesn't let me load the pi.local web site in Firefox or Chrome. The above link seems to say it's a certificate issue?

But that a self signed cert could work?

Hmm, maybe sounds like this too although no solution offered https://android.stackexchange.com/questions/157649/undoing-use-network-as-is-for-wifi-connection-on-secured-network


Improve badges and workflow

This PR improves badges by giving them a background- and border-color bashed on their hash-value. This makes different badges easier to identify. I also added a check to see if a tag already exists. Right now it only prints to the console, I am sure that there is a better way to show this but I leave this to @BaldissaraMatheus .I also modified the starting-scripts to use cross-env to allow easier development on Windows as the current npm run start on both backend and frontend does not work there.
NVM has been added as well to make sure every developer uses the name version of Node.

Thank you so much for the contribution! I think those changes are really good, but may need some tweaks:

cross-env package & script changes: I think it's great and can be merged as is;
Verification when badge already exists: Pretty good too. Once this PR is merged I can move this validation to the input, like when you're trying to create a card with a name that has already been used;
Different hex colors for the badges: I think this one should be tweaked. Right now you can already change badge colors, when you click on a badge in the expanded card it shows an option to change its color. Then you can pick one of 7 different colors, that are assigned as css custom colors and each color is defined in the css theme file. Then the badge color is persisted in a file named tags.json located in the /config folder. I think those segments can show it clearly:

expanded-card.jsx

adwaita.css

tags.json

So this change would prevent the user to choose an arbitrary color, which may be undesirable for some people. I think instead of generating a hex color we could change it to pick one of those 7 css custom properties. What do you think?
Also, when I was testing this PR I noticed this color changing feature I just mentioned doesn't seem to be working anymore, probably because the  element has priority over the z-index and the menu is being displayed behind it. Fortunately this bug is not present in the latest released version, so you can check this feature by cloning the last docker file or just checkouting to 1 or 2 previous commits.

Thanks for the reply. To be completely honest I didn't know that you can already assign a color to a badge, shows what I know :D In that case it makes sense to remove my feature, I'll revert the commit tomorrow. Also please let me know if you are happy with the version I chose for Node in the .nvmrc file, I just chose whatever I had currently installed on my PC

I think you can keep this change, just change it to use one of the available colors. I think it would be good to have a random initial color instead of the default primary one. Also no need to rush on that, I can also work from this PR when I available.
About the version on .nvmrc, this version is fine but I think it should be consistent with the docker file, which uses node-18-alpine. So I think the docker image should be changed to node-20-alpine

Changes merge on #110.


create sub-trees for Day 14 activities +netlify, config & capstone sa…

…mple finished product

Addressed suggestion and pushed updated out


Support single-character flags

Feature request
It would be good to be able to define a flag that's only one character.
Motivations
I came across it while implementing trpc-cli, which uses cleye under the hood.
There are valid use cases for for single-character flags that aren't abbreviations, like plot --x 1 --y 2
Alternatives
Of course it will usually be possible to do awkward workarounds like plot --xx 1 --yy 2, and use x and y as aliases, and override documentation to suggest only using plot -x 1 -y 2, but this is cumbersome.
Additional context
I imagine the restriction is because it muddies the water a bit with aliases (would be confusing to have --x mean one thing, but -x be an alias for something else). If that's the reason for the restriction, maybe there could be a check added to make sure the alias exactly matches the "long" form.
Bugs are expected to be fixed by those affected by it

[X] I'm interested in working on this issue

Compensating engineering work financially will speed up resolution

[X] I'm willing to offer financial support

Thanks for your sponsorship!
The x/y coordinates input are compelling use-cases for supporting this, and I'm happy to brainstorm how we can make this happen.
However, type-flag supports grouping aliases, which is a common practice in many UNIX commands. For example git commit -am 'message' groups flags a and m. I think this functionality might be mutually exclusive with single-character flag names.

I think that would work, yes. It'd probably be how most users would guess a cli should be used too. It kinda forces a particular usage but that's ok because it's a more convenient usage.
It'd be good enough for trpc-cli I think - a use case I wanted to cover was just throwing pretty much any trpc router at it and getting a functional CLI. That broke down for single-letter input props but a change like you suggest would cover them.
(V welcome re sponsorship. Thank you for your libraries - I am using more and more I find!)

I marked this as "pr welcome".
I'm not able to make time for this at the moment but feel free to tackle it if you'd like.


Runs command in an already opened terminal window

Hi,
A couple of times I've noticed after one terminal tab has been opened occasionally when opening up another the app will actually run the command inside an already open tab.
I can't seem to work out why it occurs but I've seen it happen randomly a couple of times now.
Rory

This issue should be resolved in the latest release v1.2.3 Make sure that your using iTerm 2.14 (Stable) Or the Latest iTerm Nightly


Preserve order of breakdowns in bar_dist

This preserves the order of the "Breakdowns" input value as it exists in the source DataFrame. The way it is now it is reordered however the to_dict() method decides to order them.

Coverage decreased (-0.01%) to 72.03% when pulling b0c7b86ea12c24cb1e571a4f6e3b1a314f07115d on amancevice:master into 0afa5d2cbaf1f4e009d81fe9c117f28178f3cbba on airbnb:master.

Coverage decreased (-0.01%) to 72.03% when pulling 607209da862d269a13b3bc1d056fd853c44f38c1 on amancevice:master into 9a02c88afa56f1b1fc6515308e2400811409d8bd on airbnb:master.

Coverage decreased (-0.01%) to 72.03% when pulling 607209da862d269a13b3bc1d056fd853c44f38c1 on amancevice:master into 9a02c88afa56f1b1fc6515308e2400811409d8bd on airbnb:master.


[client/network] remove peer entry from ephemeral_addresses

if there are no addresses associated with that peer
Refs #8466

this is basically some small refactoring + removing entry if there are no addresses.

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2259829

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2259878

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-frame-examples-compile-to-wasm
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2259873

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2259863

bot rebase

bot merge

bot merge


Automatically remove blob/master/ from github import

Most users copy the github URL and it doesn't work since remix expect an URL without the "blob/master" part (ex : "http://github.com/OpenZeppelin/zeppelinsolidity/blob/master/contracts/token/StandardToken.sol" doesn't work but "http://github.com/OpenZeppelin/zeppelin-solidity/contracts/token/StandardToken.sol" does).
A nice UX improvement would be automatically remove "blob/master/" if the import fails.

code to update https://github.com/ethereum/remix-ide/blob/master/src/app/compiler/compiler-imports.js#L12

that is done

Thanks @yann300 :)


Perf: Move to MongoDB 3.6+ and remove redundant JSON.stringify() calls

In the "op storm" scenario, JSON.stringify() accounts for a large chunk of the time spent in SparseMatrix v2.
Part of the reason is that we're calling it 3-4x per op:

Once in makeHandlesSerializable()
Once to stringify the message contents.
Once internally inside Socket.IO to stringify the entire message.
If the payload is large, we call stringify to chunk the message [].

(1) is tracked by a separate issue (#697)
(2) was required to work around MongoDB special character restrictions.  However, these restrictions have been removed for nested objects starting in v3.6.
(4) might be avoided by constructing the chunked payload manually via less expensive string concats / string template ops.
Also note the Scribe service re-stringifies summaries to work around the same MongoDB issue.

@DLehenbauer, can you please take a look if this item is still worth having? You addressed some of these problems lately for all code that deals with handles, right?


redis: 5.0.3 -> 5.0.4

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/redis/versions.
meta.description for redis is: '"An open source, advanced key-value store"'.

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 5.0.4 with grep in /nix/store/2iylldlwy5mlrcqbn1gl1q8aylcr4wa1-redis-5.0.4
directory tree listing: https://gist.github.com/6336f62f78dd7be779dbb8989867887a
du listing: https://gist.github.com/01fa37851293d3e743a9594a5fa2bd55

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

20 total rebuild path(s)
5 package rebuild(s)
5 x86_64-linux rebuild(s)
5 i686-linux rebuild(s)
5 x86_64-darwin rebuild(s)
5 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
python27Packages.pyres
python27Packages.remotecv
python27Packages.thumbor
python37Packages.pyres
redis

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2iylldlwy5mlrcqbn1gl1q8aylcr4wa1-redis-5.0.4 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A redis https://github.com/r-ryantm/nixpkgs/archive/ded8a04516bdd1b5c285c9f1908bcf441226dbe2.tar.gz

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

cc @berdario for testing.

there's no test for redis,  but:
@GrahamcOfBorg test nextcloud.with-postgresql-and-redis


test: [M3-7093] - Add integration tests for "Enable Linode Backups" notice, DC-specific pricing

Description 📝
Adds a few integration tests to confirm that the "Enable Linode Backups" notice and drawer both behave as expected, and refactors existing backup tests to improve performance. Additionally, makes some small additions in src to add ARIA labels to some elements.
Major Changes 🔄

Added integration tests for "Enable Linode Backups" notice and drawer

Confirms that the "Enable Linode Backups" appears when expected and is hidden when expected
Confirms that the "Enable Linode Backups" notice can be dismissed
Confirms that the "Enable Linode Backups" drawer does not show Linode regions or DC-specific pricing information when DC-specific pricing is disabled
Confirms that the "Enable Linode Backups" drawer shows Linode regions and expected DC-specific pricing information when DC-specific pricing is enabled

Refactor existing backup tests to improve performance

Tests no longer wait for the Linodes to boot, only provision
Renamed snapshot test to better reflect what the test actually accomplishes

Added a couple aria-label attributes in src

Added aria-label="Dismiss Enable Linode Backups notice" to the Enable Linode Backups notice dismissal button, which previously had no label and was announced by VoiceOver as "button"
Added aria-label="List of Linodes" to the table containing Linodes in the backup drawer

How to test 🧪
We can rely on the automated tests to confirm that the test suite runs as expected. To run the backup tests locally, you can run yarn && yarn build && yarn start:manager:ci, and then:
yarn cy:run -s "cypress/e2e/core/linodes/backup-linode.spec.ts"

Hm.. I might be missing something but three of the tests failed on me for local (yet I can see them passing in the github checks so maybe I've configured something incorrectly?)
I checked out your branch/pulled to make sure things were up to date, then ran yarn && yarn build && yarn start:manager:ci. On a different terminal, I ran yarn cy:run -s "cypress/e2e/core/linodes/backup-linode.spec.ts":
(screenshot 🖼)
Thanks Connie!

My guess is it could be something account-specific which I haven't taken into account. I'll take a closer look soon and might reach out if I have any questions while I try to reproduce!
(Just off the top of my head -- is there any chance you have Managed enabled on your test account?)

👀 I do have managed enabled, yep! Could that be why?

👀 I do have managed enabled, yep! Could that be why?

Ah yep, that explains all 3! Backups get enabled implicitly with Managed, so Cloud removes all the notices and adjusts the UI to account for that.
I can fix the integration tests easily by mocking the account so that Managed is disabled (I should've been doing that from the start, so I'm glad you caught this 👨🔬). The end-to-end test is a bit trickier...
We actually currently check for Managed in the test, and only proceed with the test interactions if Managed is disabled, but I removed that intentionally as part of this PR. My thought was that if an automated test account gets enrolled in Managed mistakenly, I want the test to fail so we have visibility into the issue, and I definitely don't want the test to pass when the backup functionality isn't actually being covered.
I'll spend some time thinking about how best to address this today! Thanks again for catching this, Connie!

Oh wait, one question: the Add Ons section and summary at the bottom of Linode Create also contains backup pricing and will update dynamically -- is that something that would be easier to test elsewhere, like #9701?

@mjac0bs That's a good catch! This should definitely be covered, and I think covering it as part of the Linode Create tests would probably make more sense. Do you think we could spin it off into a separate ticket so as to not increase the scope of #9701, or would you prefer to try squeezing it in there?
(In either case, I think Cassie's work in #9701 gets us about 90% of the way there -- I think we could add a few lines towards the bottom of her shows DC-specific pricing information during create flow test to click "Enable Backups" and confirm that the price gets displayed as expected. CC @cliu-akamai)

Do you think we could spin it off into a separate ticket so as to not increase the scope of #9701, or would you prefer to try squeezing it in there?
(In either case, I think Cassie's work in #9701 gets us about 90% of the way there -- I think we could add a few lines towards the bottom of her shows DC-specific pricing information during create flow test to click "Enable Backups" and confirm that the price gets displayed as expected. CC @cliu-akamai)

Up to Cassie, I think! @cliu-akamai - if you're willing to add that into your open PR and don't think it will be a time consuming addition (which it doesn't sound like it is), then great, let's add it there. If you would rather get your PR merged as is and not increase scope, then a follow up is fine.

Approving this PR pending a decision is made on how to follow up on the backups in the Linode Create flow. In either case, I agree that it would fit better either as a separate ticket or as an add on to the existing Create flow PR than this ticket.

Thanks @mjac0bs! I'l write up a ticket to add some assertions for backup prices in the Linode Create flow just so this doesn't get lost.
In the interest of getting this merged, for now I'm going to let the Managed-specific failures go, but I did write up M3-7183 to address this (cc @coliu-akamai) so that we can run the tests locally with Managed enabled, not get failures, but allow failures to occur in CI where Managed should not be enabled.

@jdamore-linode Are we all set to merge here? Thanks for the follow up tickets to tie up loose ends. 🙌🏼 The one CI e2e failure looks like linodes/linode-storage.spec.ts, which passes locally.


action space converters in option learning

note to self: make this specific to option learning

closing in favor of new PR


Fix some formatting for some strings.

Some of these strings were formatted in a strange way by rustfmt.
Also fixed some strings missing trailing backslashes.  Specifically, the "path override for crate" string, and the strings in http_auth_offered.

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit a4e9611453cb4ba3f9bcc34f2851546f57152229 has been approved by Eh2406

:hourglass: Testing commit a4e9611453cb4ba3f9bcc34f2851546f57152229 with merge 705009eb3828123cc9dbcf5b28988cc63f60b03b...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: Eh2406
Pushing 705009eb3828123cc9dbcf5b28988cc63f60b03b to master...


LibWeb: Stub out simplify_a_calculation_tree

Im not sure if this is the best way of handling it, or if it should happen more like this https://github.com/stelar7/serenity/commit/b83e45c263fc47e277fb1ec36161ddb82f80769e

Im not sure if this is the best way of handling it, or if it should happen more like this stelar7/serenity@b83e45c

I think this does need to live somewhere outside of the parser - we need to simplify the calculation at computed-value type (when it's assigned to a node) and later at used-value time. (Though for that last one, just calculating the value directly probably makes more sense, since we won't need to keep the calculation around afterwards.) See https://drafts.csswg.org/css-values-4/#calc-computed-value
I know I've kept changing my mind on calc(), which isn't helpful. It's complicated. :sweat_smile:
I've looked again at the specs since talking on Discord, and I think the values and typed-om specs are compatible, which is a relief (though the typed-om spec is missing a few things that we'd need to ad-hoc ourselves). Eventually I think what we want to do is (in no particular order):

Replace our ad-hoc CalculationResult with CSSNumericValue by implementing the add(), sub(), etc methods on it.
Make our calculation nodes be CSSMathValue subclasses, so a calculation is some kind of CSSMathValue containing various kinds of CSSNumericValue.
Simplification would then take a CSSMathValue (or be a method on it) and return a CSSMathValue.
At some point we should probably add the IDL side to these types too.

So you're welcome to take on however much of that you feel like. Those first two points are probably hard to do separately. You can also try adding simplification without doing those, but it'll mean that code will need modifying later.


Soft assert when comparing different enum classes for equality

Input
→ View on sorbet.run
# typed: strict
require 'sorbet-runtime'
extend T::Sig

class MyEnum1 < T::Enum
  enums do
    X = new
    Y = new
  end
end

class MyEnum2 < T::Enum
  enums do
    X = new
    Y = new
  end
end

sig {params(e1: MyEnum1, e2: MyEnum2).void}
def foo(e1, e2)
  p e1 == e2
end

foo(MyEnum1::X, MyEnum2::X)

Observed output
Static:
No errors! Great job.

Runtime:
❯ ruby editor.rb
false

Expected behavior

If if e2 had been typed as a String, this == comparison would have thrown a soft assertion saying that enum to string comparison is not allowed.
When converting a lot of old, string-based enums to typed enums, it's somewhat common to have enums that overlap on certain values or have similar names, where in the old code that only used strings it wasn't obvious that a string value like 'x' was transparently doing double duty as a value of two separate enums.
For doing those sorts of migrations, we could do better to help people track down bugs by having a soft assertion when we detect that the RHS of the == comparison is another T::Enum subclass.
The current behavior (silently evaluating to false) obscures the source of the bug and will probably make tests fail in a location that is very far away from where the logical error happened.

This change is easy to implement, but is hard to roll out, because it would be a backwards incompatible change, and it's hard to say how much existing code depends on this working.
It's also worth noting that we could probably only do this as a soft assertion, not a hard assertion, because all ruby equality checks work this way (e.g., you can do '' == :'' today and that evaluates to false even though the classes don't match).

This change is easy to implement, but is hard to roll out, because it would be a backwards incompatible change, and it's hard to say how much existing code depends on this working.
It's also worth noting that we could probably only do this as a soft assertion, not a hard assertion, because all ruby equality checks work this way (e.g., you can do '' == :'' today and that evaluates to false even though the classes don't match).

Would we ever consider an off-by-default error and a sigil like # strict_enum_equality: true (consider enum1 == enum2 an error if the types of enum1 and enum2 are both enum types and are different types) for doing this migration?  That would at least let you hard error on a file-by-file basis and ensure you don't regress.  Then once the codebase is enum-equality clean, remove all the sigils from the files (it's an open question how long we'd continue to support the sigil in Sorbet itself) and flip on the error by default?


Add deit image processor

What does this PR do?
Adds DeiT image processor

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


No request created yet.

Hi, I'm using Alamofire 5 and I'm brand new to swift and Alamofire.
I tried to make some requests but I'm facing an issue, I try to catch requests of my Mac and I see no requests done to the url I used, the requests hasn't been made.
When I try to print the response, nothing appears. And when I print the requests directly, I got "No request created yet.".
I don't understand why, I searched everywhere, and the only topic that speaks about that is about CURL requests, and not simple GET in Alamofire.
I also tried to take the code of some tutorial that works, I even get the same error I experienced.
here is my code:
import Foundation
import Alamofire

let client = Session.default

print("Loading product...")

let o = AF.request("https://www.jdsports.fr/product/bleu-nike-air-max-90-homme/16128147_jdsportsfr/",
                      headers: [
                       "User-Agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:89.0) Gecko/20100101 Firefox/89.0",
                       "Accept": "*/*",
                       "Accept-Language": "fr,fr-FR;q=0.8,en-US;q=0.5,en;q=0.3",
                       "Referer": "https://www.jdsports.fr/checkout/delivery/",
                       "X-Requested-With": "XMLHttpRequest",
                       "Connection": "keep-alive",
                       "TE": "Trailers",
                      ]
).response { response in
    print(response.value)
}

Alamofire's request creation is asynchronously, so the request isn't available immediately. If you print the response itself rather than just the value, you'll probably see an error describing the problem. If this is a macOS tool, you'll need to give it the outgoing network traffic entitlement before you can make requests.

I am a new iOS developer and I have the same issues that I have changed several websites urls but still doesn't work
here is code:
`import Foundation
import Alamofire
let ret = AF.request("https://github.com/Alamofire/Alamofire/blob/master/Documentation/Usage.md#using-alamofire").validate()
.responseData{ responce in
debugPrint("responce is (responce)")
}
print(ret)`
Is there any wrong?


Support "prefix" option for keymappings

The orgmode plugin supports a prefix option to define a common prefix for orgmode to provide easy customization of the prefix without forcing the user to redefine a lot of keymappings.
In org-roam <Leader>n is a very common prefix for most of the operations, so it would make sense to provide the same feature.
@chipsenkbeil Would you be open to such a feature? If so, I would proceed with building it, because I really miss this. :wink:

Go for it :)

As I am working on it I discovered, that you are using <Leader>, but in some cases also <LocalLeader>. Was this intensional? If so, we should also provide a localprefix. I would then default prefix = "<Leader>n" and localprefix = "<LocalLeader>n.


Make toJson and fromJson always compatible.

Is your feature request related to a problem? Please describe.
This issue assumes that the serialization manager is removed from the fromJson interface (Issue tracked here:  https://github.com/serverpod/serverpod/issues/2027).
Serverpod models expose two methods to create JSON from a model, toJson and allToJson.
Currently, toJson is used to create the JSON that is serialized for the client, and allToJson is used to create the "real" JSON representation of the object (used as base for serializing to the database).
If a class is scoped as serverOnly, or if a field is scoped as serverOnly or none the field will be removed from the toJson serialization.
This makes the fromJson and toJson behave unexpectedly.
For example, given a model definition that looks like this:
class: MyClass
serverOnly: true
fields:
  serverOnlyScope: String

The following code would result in a throw:
var a = MyClass();
var b = MyClass.fromJson(instance.toJson()); // Throws because it is missing the required serverOnlyScope field

While this works:
var a = MyClass();
var b = MyClass.fromJson(allToJson());

Describe the solution you'd like

Rename the current toJson method to toJsonForClient.
Rename the current allToJson method to toJson and keep the existing behavior.

Additional context

[ ] This is a breaking change so it will need to be implemented in a major version upgrade (target only main)
[ ] This changes the behavior of the methods so we should add upgrade instructions and documentation on how these two methods work after this change.

We agreed to rename "toJsonForClient" to "toConstraintJson".


Colour code output

Just a thought, how about using the Python Rich library to add some colour coding to the output?
https://rich.readthedocs.io/en/stable/introduction.html
Thanks for your work on this, making the output more readable is a good idea

Added some more colour to the output in e1b68ecd1038f76b4498387605cbf176aa03d4bf. I don't really want to add another dependency so I just added more colours to the color function.
I'll close for now but happy to accept more ideas for places to add colour 🌈 . Here's what it looks like right now:

Great. That looks to add some clarity and as you say, saves another library just to set some colours


[5.1] Common responses assertions added

Added common responses assertions, to make tests code more readable.

You can add these to your own test class as needed.


fix: #eval command was leaking auxiliary declarations into the environment

This means one cannot use #eval anymore to, for example, bring the generated splitter lemmas into scope?
Makes sense, of course; maybe some tests will need another mechanism (elab maybe).

This means one cannot use #eval anymore to, for example, bring the generated splitter lemmas into scope?

Yes, you still use #eval <expr> for that, but the auxiliary declarations created to elaborate <expr> will not leak into the environment. The declarations created while executing <expr> will be in the resulting environment.

@nomeata The new test file is a good example of what this PR is fixing: https://github.com/leanprover/lean4/blob/e7cc454161017598c7bf5d472401de54ae90765e/tests/lean/evalLeak.lean

Ah, thanks for the explanation!


Error on invocation: "No module named mssqlcli"

Hello, I am on macOS Catalina 10.15.7, trying to use mssql-cli to query a SQL Server DB, I installed using pip3 install mssql-cli, but for some reason the CLI thinks it should be pointed at Python 2.7:
❯ pip3 install mssql-cli
< ... >
❯ mssql-cli
/System/Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/Python: No module named mssqlcli

for reference here is the version info of my various relevant commands:
❯ which pip3
/Library/Frameworks/Python.framework/Versions/3.8/bin/pip3
❯ pip3 --version
pip 21.1.3 from /Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pip (python 3.8)
❯ which python3
/Library/Frameworks/Python.framework/Versions/3.8/bin/python3
❯ python3 --version
Python 3.8.2
❯ which pip
/Library/Frameworks/Python.framework/Versions/3.8/bin/pip
❯ pip --version
pip 21.1.3 from /Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/pip (python 3.8)
❯ which python
/usr/bin/python
❯ python --version
Python 2.7.16

Any pointers for un-crossing the wires to point at the correct directory on cli startup would be appreciated.

I am having this exact issue as well.

Open the mssql-cli with vim(/usr/local/bin/mssql-cli), change the last line which python -m mssqlcli.main "$@" to python3 -m mssqlcli.main "$@", it works for me!


Unity error when using 515

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
l515

Firmware Version
<IP_ADDRESS>

Operating System & Version
win10

SDK Version
release, master branch, 50515 branch

Language
unity }

Segment
VR  }

Issue Description
<Describe your issue / question / feature request / etc..>
I got 2.36, the 2.37 prerelease, built master from scratch, built 50_515 and for all of them in unity i get this error:
ExternalException: rs2_pipeline_start_with_config(pipe:000001B845E78650, config:000001B845E784B0)
Rethrow as Exception: Couldn't resolve requests
Intel.RealSense.ErrorMarshaler.MarshalNativeToManaged (System.IntPtr pNativeData) (at <782525f085c64debb7028293360b3d39>:0)
Intel.RealSense.Pipeline.Start (Intel.RealSense.Config cfg) (at <782525f085c64debb7028293360b3d39>:0)
RsDevice.OnEnable () (at Assets/RealSenseSDK2.0/Scripts/RsDevice.cs:71)
Several other people have posted similar issues in the last 2 weeks.
Ive spent a lot of time trying to get this to work because other comments here implies that it works

Hi @chrisrogers3d
As far as I know, "Couldn't resolve requests" should be resolved in 2.37, tested with Unity 2017.
Are you using the Unity package from 2.37 release? Perhaps there is other realsense2.dll somewhere in system path?

Hey @dorodnic Whats your best advice on getting this to work with Unity 2019.4?  i can compile a different github branch if i need to, I havent tried to actually debug the code other than that screenshot.  I am chrisrogers3d (personal account that started this thread) and now I am trying to get this to work for a work project (facebook reality labs). thanks!

This all works for me with the latest 2.38.1 update.
Thanks!


Issues from testing v8.8

manual grid scans have wavelength off by x1000 (https://ispyb.diamond.ac.uk/dc/visit/cm37235-1/dcg/11085963)
with external callbacks enabled, logging issue - graylog messages are not tagged with artemis stream except from __main__ ?!
rotation scan needs global param version
ispyb doesn't seem to be pointing at images (https://ispyb.diamond.ac.uk/dc/visit/cm37235-1/dcg/11086026)

Fixed the first issue:
            self.params.hyperion_params.ispyb_params.current_energy_ev = doc["data"][
                "dcm_energy_in_kev"
            ]

should be:
            self.params.hyperion_params.ispyb_params.current_energy_ev = doc["data"][
                "dcm_energy_in_kev"
            ] * 1000

Additionally, we need to increase the time we wait for the FGS parameters to be valid to 0.6s (https://github.com/DiamondLightSource/hyperion/issues/1101)


dubbo代码强依赖yaml1.20的功能，但pom中未配置强依赖1.20的yaml

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.3
Operating System version: xxx
Java version: xxx

Steps to reproduce this issue

xxx
xxx
xxx

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
What do you expected from the above steps？
dubbo应该强依赖1.20yaml
Actual Result
What actually happens?
If there is an exception, please attach the exception trace:

Just put your stack trace here!

It's more likely that you are using an imcompatible yaml version


[doc] Split out sections 'Theory of Operations' and 'Programmer's Guide' from README.md files

One of the first suggestions to utilize the new 'Book' structure of the documentation was to split the monolithic README.md/_index.md files into smaller pages/sections based on the existing subheadings.
A suggestion that was trialled was to split by the 'Theory Of Operations' and 'Programmer's Guide' subheadings, and make new pages for each of these sections under the /doc directory within the HWIP blocks. This PR implements the splitting as described.
A new script util/mdbook/move_md.py is added and used to copy a markdown file to a new location, while updating all relative links as per the relative path between the new and old locations. This can easily be used to move and split content. A temporary script split.py was created and run from the root of the project (See Footnotes), and used to perform this particular split.
Feedback or suggestions are welcome. With the scripting, this is not an arduous thing to reproduce, despite the +14,599/-13,937 change count.
This is a lot of churn, and it would be ideal if there was a better way to ensure git history persists across the splitting operation. The history is all still there, but git diff and git blame require some additional encouragement to correctly follow the renames. I am going to look into this issue further.

Footnotes

split.py

#!/usr/bin/env python3

import pathlib
import re

import logging
logging.basicConfig()
logging.getLogger().setLevel(logging.DEBUG)  # set root-logger to most-permissive
logger = logging.getLogger(__name__)
logger.setLevel(logging.WARNING)

import util.mdbook.move_md
from util.mdbook.move_md import move_md

to_be_split="""
./hw/ip_templates/alert_handler/README.md
./hw/ip_templates/rv_plic/README.md
./hw/top_earlgrey/ip_autogen/alert_handler/README.md
./hw/top_earlgrey/ip_autogen/rv_plic/README.md
./hw/top_earlgrey/ip/sensor_ctrl/README.md
./hw/ip/adc_ctrl/README.md
./hw/ip/aes/README.md
./hw/ip/aon_timer/README.md
./hw/ip/clkmgr/README.md
./hw/ip/csrng/README.md
./hw/ip/edn/README.md
./hw/ip/entropy_src/README.md
./hw/ip/flash_ctrl/README.md
./hw/ip/gpio/README.md
./hw/ip/hmac/README.md
./hw/ip/i2c/README.md
./hw/ip/keymgr/README.md
./hw/ip/kmac/README.md
./hw/ip/lc_ctrl/README.md
./hw/ip/otbn/README.md
./hw/ip/otp_ctrl/README.md
./hw/ip/pattgen/README.md
./hw/ip/pinmux/README.md
./hw/ip/prim/README.md
./hw/ip/pwm/README.md
./hw/ip/pwrmgr/README.md
./hw/ip/rom_ctrl/README.md
./hw/ip/rstmgr/README.md
./hw/ip/rv_core_ibex/README.md
./hw/ip/rv_dm/README.md
./hw/ip/rv_timer/README.md
./hw/ip/spi_device/README.md
./hw/ip/spi_host/README.md
./hw/ip/sram_ctrl/README.md
./hw/ip/sysrst_ctrl/README.md
./hw/ip/tlul/README.md
./hw/ip/uart/README.md
./hw/ip/usbdev/README.md
"""

# The current headings aren't consistent, so make the regex matches a bit fuzzy
THEORY_PATTERN = re.compile(r'\#\sTheory\s(o|O)f\sOperation[s]?')
PROGRAMMERS_PATTERN = re.compile(r'\#\sProgrammer(s|\'s)\s(g|G)uide')

def main() -> None:
  logging.getLogger('util.mdbook.move_md').setLevel(logging.INFO)
  for f in to_be_split.split('\n'):
      if f == '':
          continue
      logger.debug(80*"-")
      p = pathlib.Path(f)
      assert p.exists(), "file does not exist!"
      txt = p.resolve().read_text()

      too_m = THEORY_PATTERN.search(txt)
      if not too_m:
          logger.warning(f"'# Theory of Operations' not found in {f}")
          logger.warning(f"Not changing file.")
          continue
      else:
          too_line = 1 + txt[0:too_m.start()].count("\n")
          logger.debug(f"Line = {too_line:>4}, File = {f}")

      pg_m = PROGRAMMERS_PATTERN.search(txt)
      if not pg_m:
          logger.warning(f"'# Programmers Guide' not found in {f}")
          logger.warning(f"Not changing file.")
          continue
      else:
          pg_line = 1 + txt[0:pg_m.start()].count("\n")
          logger.debug(f"Line = {pg_line:>4}, File = {f}")

      # Extract the theory section to a newfile 'doc/theory_of_operation.md'
      too = p.parent / "doc/theory_of_operation.md"
      move_md(p, too)
      # Remove lines :
      #                - before '# Theory of Operations'
      #                - after '# Programmers Guide'
      new_too = too.read_text()
      only_too = "\n".join(new_too.split("\n")[too_line:pg_line-1])
      too.write_text("# Theory of Operation\n" + only_too)

      # Extract the programmers section to a newfile 'doc/programmers_guide.md'
      pg = pathlib.Path(f).parent / "doc/programmers_guide.md"
      move_md(p, pg)
      # Remove lines :
      #                - before '# Programmers Guide'
      new_pg = pg.read_text()
      only_pg = "\n".join(new_too.split("\n")[pg_line:])
      pg.write_text("# Programmer's Guide\n" + only_pg)

      # Remove the theory of operations / programmers guide from the orginal files
      txt = "\n".join(txt.split("\n")[:too_line-1])
      p.write_text(txt)

if __name__ == "__main__":
main()

Thanks for taking a look @pamaury.

There are a couple of things unclear in this PR:

was move-md.py used and if so on which files?
the split was done by a tool I assume, should that tool be part of the commit (and then removed by another commit)? I would document how it was done at least

See the section 'A temporary script split.py was created' in the PR message for some more details and the script itself, which I just attached to the PR rather than the commit-use-revert pattern we used previously.

Sorry, I didn't see the split.py file, I had not realized you could click on the footnote to expand it ^^

Updated to address review comments and fix lints

Thanks for the review @andreaskurth :)

This will help people to navigate the docs because they can reach a HW block's Theory of Operations and Programmer's Guide from a menu rather than by having to scroll through one long document. So I think it's a valuable change.
Even though thousands of LoC are modified and the PR thus would be hard to review, the automated approach with split.py in the PR description and util/mdbook/move_md.py makes it manageable. Thanks and well done!
I've mainly reviewed the two Python files that create the changes in this PR, and I agree with them. In addition to one comment on the Python code, I noticed that the documents containing the Theory of Operation section are named theory_of_operations.md (operation vs. operations). I suggest consistently using the same word in title and file name.

All files/headings changed to "theory of operation" (no 's') for consistency. Thanks for pointing this out.
(If anyone disagrees with this choice feel free to bikeshed on the naming here.)

There is still an issue with a failing lint due to topgen disagreeing about the links after files have been split. This PR will stay as a draft until that issue has been resolved.
(See commit message here for additional details.)

The markdown files in the ipGen ip_templates/ were modified as part of the mdbook inital PR, to change away from #regref to using plain markdown links with anchors to define the register. As part of this commit, the links in the template were changed incorrectly to assume the generated location was /top_earlgrey, which is not always going to be true.
This PR has added two commits to change those links to be relative within the ip_templates/ directory, as well as then splitting the content into the two subfiles doc/theory_of_operation.md and doc/programmers_guide.md. This will fix the lint failure as well as ensuring the relative links work correctly after the ip has been rendered using topGen/ipGen.

Rebased, only a single meaningful conflict to resolve which was a change to hw/ip/keymgr/doc/theory_of_operation.md from f243e6802143374741739d2c164c4f2f61697669.

Merging this to unblock other doc changes. I think all open conversations were already resolved or can be resolved in a small follow-up PR


gateway: make monitoring interval configurable

We mark a failed endpoint to be unavailable for a fixed interval right now.
https://github.com/coreos/etcd/blob/master/proxy/tcpproxy/userspace.go#L140
We want to make this configurable and add a flag into gateway to configure it.
/cc @gyuho can you take a look?

we called this flag https://github.com/coreos/etcd/blob/master/Documentation/op-guide/configuration.md#--proxy-failure-wait in the old proxy. probably we need a better name? failure wait is not that descriptive i feel.

Ok, will take a look.


Documenting derived Generic instance instead documents the Rep type family instance

For this example:
{-# LANGUAGE DeriveGeneric #-}
{-# LANGUAGE StandaloneDeriving #-}

data Foo = Foo

-- | This does some generic foos.
deriving instance Generic Foo

Haddock 2.22.0 places the documentation on type Rep Foo instead of on instance Generic Foo.

Thanks for the bug report! I just merged a fix.


How to correctly handle remote validation

I need to delegate validation to a remote server, which would reply with a 204 No Content response on success and with 412 Precondition Failed response on failure, with a JSON body similar to this:
{
  "firstName": [
    "must be at least 3 characters",
    "must not contain numeric characters"
  ],
  "username": [
    "already taken"
  ]
}

I'm trying to handle this situation with aurelia-validation, passing the JSON to .onValidate() which works on the first roundtrip but then sticks on the first result.

Solved binding the remote validation result to a VM property and passing it to .onValidate() just once; then on remote validation response the VM property is updated before triggering validation.


Gitlab CI environment needs update

Ascent's OS was upgraded to RHEL 8 on February 11.  We strongly recommend
  rebuilding prior to running on upgraded system.

Besides updating the modules
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -11,7 +11,8 @@ stages:

 .LoadModules:
   before_script:
-    - module load gcc/7.4.0 cuda/10.1.243 cmake/3.18.2 git/2.20.1 spectrum-mpi/<IP_ADDRESS>-20200121
+    - module load gcc/7.5.0 cuda/10.1.243 cmake/3.22.2 git/2.31.1 spectrum-mpi/<IP_ADDRESS>-20210112
+      module list

 .BuildBoost:
   extends: .LoadModules

we also need to rebuild all the dependencies.

The benchmarks still not running because of things like
warning: failed to remove arborx-master/build_master/CMakeFiles/CMakeTmp/.nfs000000000561312d00001424: Device or resource busy
Error encountered during job: exit status 1
Error encountered during job: Error executing run_exec: exit status 1

I recall that on of the ways to work around the timeouts on some HPC systems was to add --debug-trycompile flag.

The benchmarks still not running because of things like

I would say "doesn't run anymore". It certainly was running for some time after fixing it.

I recall that on of the ways to work around the timeouts on some HPC systems was to add --debug-trycompile flag.

We can try if that helps but otherwise, I don't really have a good idea.

Maybe we would also need to manually wipe out the build directories on Arcticus prior to testing a patch with --debug-trycompile.


Move thoughts to separate markdown file

This is mostly a documentation-organization suggestion. I thought separating the critique of the fiber scheduler interface from the description of the provided implementation could make it more readable.
Unless this repository is more a proof-of-concept of the critique itself, in which case leaving it in the main README would make sense.

Thanks for your excellent contributions!


Check your .gitignore

I see some spurious files and directories that shouldn't be in your repo, e.g. .DS_Store

Also .ipynb_checkpoints

included in .gitignore. manually deleted .DS_Store when necessary...


snapshot artifact is not accessible

[warn] ==== sonatype-snapshots: tried
[warn]   https://oss.sonatype.org/content/repositories/snapshots/com/github/sbtliquibase/sbt-liquibase-plugin_2.10_0.13/0.1.-SNAPSHOT/sbt-liquibase-plugin-0.1.-SNAPSHOT.pom
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: com.github.sbtliquibase#sbt-liquibase-plugin;0.1.-SNAPSHOT: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
Actually the URL: https://oss.sonatype.org/content/repositories/snapshots/com/github/sbtliquibase/ is not available.

I haven't setup publishing for Sonatype repos yet.  Its something I need as well and will get it setup as soon as I can.

master builds should be publishing to Sonatype OSS now.


Error: eval js error: TypeError

Hi there
I have downloaded this version of the clive
https://github.com/koki-develop/clive/releases/tag/v0.12.4
clive_Linux_x86_64.tar.gz
and I've got the ttyd package installed.
I've run ./clive init and created the default clive.yml file.
with the default config when I run ./clive start I get the following error:
Error: eval js error: TypeError: Cannot set properties of undefined (setting 'fontSize')
    at <anonymous>:1:51
    at <anonymous>:1:57 <nil>

also when I run ./clive validate I get the following:
Config file clive.yml is valid!
here's my uname -a result:
Linux devops 5.19.0-41-generic #42~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Tue Apr 18 17:40:00 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

a chrome window starts and closes immediately!

@sinawic
Thank you for opening an issue!
Could you please tell me the result of running the following command?
ttyd --version

@sinawic
Thank you!
It appears that this problem may occur if the version of ttyd is less than 1.7.0.
Could you please try again after upgrading ttyd to version 1.7.0 or higher?

I installed the ttyd package using apt
and it seems the version you mentioned does not exist on the package manager.
do you think I should install it from the source or?

@sinawic
Yes, that could be a good idea, or you might want to directly download the binary from the Releases page.
For instance, like this:
$ wget 'https://github.com/tsl0922/ttyd/releases/download/1.7.3/ttyd.x86_64'
$ chmod +x ./ttyd.x86_64
$ mv ./ttyd.x86_64 /usr/local/bin/ttyd
$ ttyd --version

thank you so much. the problem seems to be solved now with ttyd version 1.7.3
I guess it's better mentioning it on the docs!
if you want I can make a pr tho

@sinawic

the problem seems to be solved now with ttyd version 1.7.3

I'm glad to hear that. 👍
Thank you for checking and letting us know!!
I was able to identify this problem thanks to your report!!

also I wanted a feature for the ttyd to exit after the execution is finished.

cLive should automatically terminate the ttyd process upon exit, isn't it working properly?
Even executing exit won't terminate the ttyd process (this is a specification of ttyd).

uh I'm sorry
this is not at all a bug
by passing skipPauseBeforeQuit: true I get my expected result
I read the docs but didn't understand it meant the same thing I wanted.
thanks for the response :pray:

Ah, I see, is that so!
I'm really glad I could help with solving the problem!
If there's anything else, feel free to open another issue! 👍


FAIR-3296: Add dictionary methods

See #17 #18 #19 for more context.
Most changes are linting or basic refactoring.
Main additions are the dictionary GET, LIST and PATCH methods.

Dictionary GET, LIST and PATCH methods have been tested and work as expected


Add example showing use of await

(First off, bravo on this great project!)
I wanted to do this for my own sanity checking, but might be useful for someone else to see so I'm submitting a PR.
Ran benchmark with ab -kc 1000 -n 10000 http://<IP_ADDRESS>:8000/await, demonstrating that concurrency remains extremely high while a request handler was awaiting.
Results with simulated 5 second delay, running 10 chunks of 1000 simultaneous requests:
Concurrency Level:      1000
Time taken for tests:   50.215 seconds
Complete requests:      10000
Failed requests:        0
Keep-Alive requests:    10000
Total transferred:      1470000 bytes
HTML transferred:       180000 bytes
Requests per second:    199.14 [#/sec] (mean)
Time per request:       5021.497 [ms] (mean)
Time per request:       5.021 [ms] (mean, across all concurrent requests)
Transfer rate:          28.59 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:        0    3   8.0      0      36
Processing:  5000 5011   8.8   5009    5044
Waiting:     5000 5011   8.8   5009    5044
Total:       5000 5014  14.4   5009    5067

Percentage of the requests served within a certain time (ms)
  50%   5009
  66%   5011
  75%   5013
  80%   5018
  90%   5042
  95%   5053
  98%   5059
  99%   5063
 100%   5067 (longest request)

Thanks! And merging :).  We may also be able to dramatically increase the number of concurrent requests by setting backlog to be much higher than the default (100) when creating the http server in server.py


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.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 2699

Questions ?
Please refer to the Backport tool documentation


fix(connections): remove unused modules

@oceanlvr

Removed some modules that were not used after import
Optimized the protocol type

@oceanlvr

Removed some modules that were not used after import
Optimized the protocol type


[PoE2] Missing Total / Elemental DPS on weapon

When a weapon has a flat elem, it doesn't count anywhere. We only see "Physical DPS".
Expected behavior
To see "Total DPS" (and probably have it checked by default).
To see "Elemental DPS" sum, probably. But also have ability to check targeted element (for example flat cold is very valuable for Freeze Buildup).

Rarity: Rare
Carrion Bane
Advanced Zealot Bow
--------
Physical Damage: 134-224 (augmented)
Cold Damage: 40-61 (augmented)
Critical Hit Chance: 5.00%
Attacks per Second: 1.41 (augmented)
--------
Requirements:
Level: 62
Dex: 142
--------
Sockets: S S
--------
Item Level: 79
--------
40% increased Physical Damage (rune)
--------
73% increased Physical Damage
Adds 17 to 36 Physical Damage
Adds 40 to 61 Cold Damage
17% increased Attack Speed
+18 to Dexterity
Gain 15 Mana per Enemy Killed

Item Class: Two Hand Axes
Rarity: Magic
Crackling Timber Axe of the Lynx
--------
Two Handed Axe
Physical Damage: 48-99
Elemental Damage: 7-117 (augmented)
Critical Strike Chance: 5.00%
Attacks per Second: 1.25
Weapon Range: 1.3 metres
--------
Requirements:
Level: 41
Str: 97
Dex: 45
--------
Sockets: R-R R
--------
Item Level: 70
--------
+15 to Dexterity
Adds 7 to 117 Lightning Damage

Should be doable, but more challenging than I thought, PoE1 provided us the ele damage but here we get it split out.

That wasn't that bad, this will be in next release


New Expression should accept trailing commas Fixes #142

@markelog ready for review!
@hzoo you made the change originally for CallExpression so you're probably best placed to review this.

@hzoo @markelog https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/CHANGELOG.md#1000--2016-11-06 has released now. and that requires code incompatible with cst.

So are you planning to finish this soon?

@markelog it's already done, as far as I'm aware.

@markelog nope

Wow, terrible, it seems I wasn't adapted to new UI github changes.
Thank you!

@markelog done!

@markelog any chance of a 0.4.7 release of this soon?

Done!

Shweet


PR: Initial fixes to make Spyder work with Qt 6

Description of Changes

Spyder GUI shows up with PyQt6, PySide6 is broken (once again, MRO issues when using super())
Test suite mostly passes with PyQt6, PySide6 not tried yet
The changes are backward compatible with Qt 5
Concerning PySide6 support: We will definitely look into it and provide patches. Question for Spyder devs: Which minimum Qt 6 version do you intend to support? This might affect PySide6 patches (PySide6 >= 6.5 claims to fix MRO issues when using super()).

Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:
rear1019

Thanks for your help with this @rear1019!! It's much appreciated!
About your comment:s:

Test suite mostly passes with PyQt6, PySide6 not tried yet

That's great!

The changes are backward compatible with Qt 5

Thanks for making sure of that. It's really important for us until we can definitely switch to Qt 6.

Question for Spyder devs: Which minimum Qt 6 version do you intend to support? This might affect PySide6 patches (PySide6 >= 6.5 claims to fix MRO issues when using super()).

6.5 is fine. I mean, we'll start supporting the latest available Qt 6 version when we can officially announce support for it.
But I think it'd be a good idea to increase the minimal Qtpy version to have the best possible compatibility with Qt 6. @dalthviz, what Qtpy version should we require?

We could maybe even try to push forward the release of QtPy 2.4.0 and update the contraint to version 2.4.x

Good idea, I think that release is almost ready.

@rear1019, could you give us permissions to push to your branch, if possible? I updated your work and fix the style issues I found so we can merge it.

@ccordoba12 Sorry for the late response. “Allow edits from maintainers” is not available for forks owned by an organisation: GitHub’s documentation states that a “a fork owned by a personal account” is required, confirmed here.
You have been invited as a contributor with write permission.

Thanks @rear1019! I received and accepted the invitation.

But now I'm getting this message:
remote: error: GH006: Protected branch update failed for refs/heads/qt6_fixes.
remote: error: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information.
To https://github.com/procitec/spyder.git
 ! [remote rejected]     qt6_fixes -> qt6_fixes (protected branch hook declined)
error: failed to push some refs to 'https://github.com/procitec/spyder.git'

If it's too much problem for you to support this, I can simply create a new branch in my fork with your initial commit and my changes, and then a PR from it. Let me know what works best for you.

If it's too much trouble for you to support this, I can simply create a new branch in my fork with your initial commit and my changes, and then a PR from it. Let me know what works best for you.

For the time being, this is the be best/fastest solution. I don’t have the permissions to change settings of the procitec fork, so any changes require some back and forth with a coworker of mine.
If it is of any help: At procitec/qt6_fixes-v1 [1] is a version of this PR rebased onto the current master, with merge-conflicts resolved.
[1] https://github.com/procitec/spyder/tree/qt6_fixes-v1

Thanks @rear1019 but since I already updated your branch and applied some fixes on top of it, it was easier for me to create a new pull request out of it: https://github.com/spyder-ide/spyder/pull/21572.
So, I'm going to close this PR as superseded by that one.


APP-810 Package and run the configurator app together with the Integration Bridge

I'm gonna open a new PR directly to the symphonyoss.


Doesnt work with slide toggle

Cant use $(element).slideToggle(500) with this lib.
This library is really cool, I'd like it to use it...
I hope you can fix this, because this function is really important! :)
Best regards,
Sora

I Think the same affects the function "$(element).fadeToggle(500). Please have a look

You are right, I have been in 1.6.9 fix the problem.

Youre great! Thank you very much :)


Fix warnings when using record-backward-compat

Fix some small omissions in the tensor data refactor #1916 with the record-backward-compat feature flag

Unused import
Allow deprecated use of DataSerialize
Leftover generic on dimension

Looks like a typo was introduced in #1966 (CI didn't run cuz it only affected markdown files)..
Seems to have been fixed in other PRs so I'll leave it as is.


Is this mpeg DASH (.mpd) not downloadable due to DRM ?

[ x] I'm asking a question
[ x] I've looked through the README and FAQ for similar questions
[ x] I've searched the bugtracker for similar questions including closed ones

Question
Hello,
similar issue was closed with a DRM flag.
i want to ask if that mean that such media can not be downloaded?
youtube-dl -u myusername -p mypassword -f bestvideo+bestaudio "https://***.ssl.cdn.cra.cz/vod_cmesk/_definst_/***/***/***.smil/manifest.mpd" --verbose
output is here (passw0rd: yd)
mpd file looks like this
ffmpeg -i https://***.ssl.cdn.cra.cz/vod_cmesk/_definst_/***/***/***.smil/manifest.mpd -codec copy out.mp4
downloaded around 1.2GB file but the image is damaged.
no way to download it except some browser extension or a screen recording software/hardware?

Yes, it has DRM.


refactor: slide-in and slide-up animations

e.g. for filter selection screen and location details screen

@adambard1 what type of animation is needed?

The slide in and slide up animations are for the service details page and the service summary tray. I think @shakilhossain1 is working on this right now, but idk how it can work in tandem with a loading animation.


Reorganize content to be more consistent across pages

Description
This PR reorganizes the content on the installation and config pages for the major providers to make them more consistent.
Fixes this issue.

I know the AWS Native provider doesn't support skipMetadataApiCheck (i.e., doesn't support authenticating via EC2 instance metadata), should we add a brief statement to that effect to the AWS Native installation and configuration page? Otherwise, the user will only know if that scroll all the way down to the bottom of the page to see that skipMetadataApiCheck isn't yet supported.


Update recording and edit trainthesound.md

Hi @ruqaiyasattar

I have edited the trainthesound.md file after doing more research.
Placed my contribution in the Applicants Contribution folder
Added new recording of the final application
Updated code at PR ->  https://github.com/ruqaiyasattar/digitalstethoecopeAI/pull/23

https://user-images.githubusercontent.com/57020210/228322784-11009c42-b451-4395-8faa-cc6013d55ebd.mp4
Any reviews are welcome.

Great!


Logging configuration fix

Hello, my company has adopted your package for our internal use and I have made some enhancements that I think might be of use outside of our company.
This PR allows adjusting the log level via a cli switch or environment variable, cleans up the log message format and fixes an ambiguous log message
If you are willing to accept this PR (or PRs in general) I plan to submit a few more PRs of roughly similar size and scope

You need to add small unit test, which uses log_level :)
I'm trying to keep 100% coverage.

@azax42 Thank you very much for your PR!
I'm ready to accept, review and discuss any changes to this repo, so feel free to make more PR! :)


Integrate u-root with modules support

U-root has recently received support for go modules. We definitely want this integrated into system-transparency.git!

Further development on https://git.glasklar.is/system-transparency/core/system-transparency


feat: support alter twcs compression options

Refer to a related PR or issue link (optional)
https://github.com/GreptimeTeam/greptimedb/issues/4905
What's changed and what's your intention?
support alter twcs compaction options on sql level
!!! DO NOT LEAVE THIS BLOCK EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:
expand alter table set options to include  twcs compaction options.

Summarize your change (mandatory)
The approach here is to expand ChangeOption enum with twcs compaction options.
The key and value type is validated at store-api end.
If value is missing (empty string) it will be set with default value.
Changes will be reflected on show create table statements.

How does this PR work? Need a brief introduction for the changed logic (optional)

Describe clearly one logical change and avoid lazy messages (optional)

Describe any limitations of the current code (optional)

Checklist

[x] I have written the necessary rustdoc comments.
[x] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

the new approach look cleaner going to close this one in favor of
https://github.com/GreptimeTeam/greptimedb/pull/4965


[stable/redmine] Add affinity, nodeSelector, podAnnotations and tolerations

Signed-off-by: tompizmor<EMAIL_ADDRESS>
What this PR does / why we need it:
Adds support to specify nodeSelector, tolerations, affinity and podAnnotations.
Which issue this PR fixes

fixes #12429

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

/ok-to-test
/lgtm

/ok-to-test
/lgtm


Czech special characters displayed wrong

While testing your extension we ran into an issue with special symbols in PORT select box
The intended text is Komunikační port (COM1) but we got Komunika?n? port (COM1) so I suppose there is some issue with Czech special characters.

@SmejkalJakub Thanks for pointing this out! We will work to get this corrected.

@gcampbell-msft sorry for the delay, here is the image that you requested.
The intended text is: Standardní sériová linka pomocí protokolu Bluetooth (COM3)

Closing in favor of tracking here #47


added test for Operation error ABORTED (Transaction locks invalidated)

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report

Merging #338 (c37728f) into master (7dd9ec4) will decrease coverage by 24.78%.
The diff coverage is 78.57%.

@@             Coverage Diff             @@
##           master     #338       +/-   ##
===========================================
- Coverage   62.29%   37.51%   -24.79%
===========================================
  Files         196      107       -89
  Lines       20900    15065     -5835
===========================================
- Hits        13019     5651     -7368
- Misses       7095     9189     +2094
+ Partials      786      225      -561

Flag
Coverage Δ

1.14.x
37.47% <78.57%> (-24.63%)
:arrow_down:

1.18.x
37.49% <78.57%> (-24.68%)
:arrow_down:

1.19.x
37.51% <78.57%> (-24.64%)
:arrow_down:

connection
?

database_sql
?

discovery
?

e2e
?

integration
?

macOS-latest
37.49% <78.57%> (+0.19%)
:arrow_up:

ratelimiter
?

scripting
?

table
?

topic
?

ubuntu-latest
37.51% <78.57%> (-24.73%)
:arrow_down:

unit
37.51% <78.57%> (+0.07%)
:arrow_up:

windows-latest
?

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

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


typo: Double word "below"

Replaced one below with "from", but maybe there is a better wording/intent

@nschonni : Thanks for your contribution! The author, @anirudhcavale, has been notified to review your proposed change.

@anirudhcavale  would you please approve this change so we can get the doc updated?  Thank you
Carmon

Merging typo fix


Todo

[x] Column resizing

[ ] Double-click to autofit

[ ] Column reordering

The remaining features are no longer planned.


[FEATURE]: ClearML integration

Contact Details
<EMAIL_ADDRESS>Describe the feature you'd like
I would propose integration with ClearML, mainly for the popularity of the platform and it's broad array of implemented MLOps practices, spanning from experiment tracking to production model serving.
How do you solve your current problem with the current status-quo of ZenML?
I am using the ClearML SDK and manually invoking functions for certain tasks, i.e. metrics logging or model registering.
Any other comments?
If the discussion proceeds and it is decided to proceed with this integration, I am willing to contribute to its development.

Thank you for the ticket @Aleksandar1932  ! Feature requests now go to our roadmap where other community members can also vote for them! Could you perhaps make a ticket here: https://zenml.hellonext.co/b/feedback
Having said that I think a ClearML integration would be amazing as well - can you share the way you write the steps right now with the ClearML SDK?

Feature request moved here: https://zenml.hellonext.co/p/integration-with-clearml


Fixed missing virtual destructor of struct Step

Fixed the missing virtual destructor of struct Step. Found by the address sanitizer.

Good catch !
Thanks for the contribution


Tile editor map slots

Need more info from Rami about what this means.

After putting the 64kb floppy limit, This feature is no longer possible to implement !


[Question] How to load an old model using newer version of  transformers where llava is integrated?

Question
I am deploying a checkpoint model obviously trained for transformers==4.31.0 and I'm deploying it in an environment of transformers==4.36.1. (tensorrt_llm expects a newer version)  In the new version of transformers, they have integrated the llava class, and modified its expected state_dict key names. So when I use
from transformers import LlavaForConditionalGeneration LlavaForConditionalGeneration.from_pretrained(...)
it gives warning about uninitialized parameters:

It seems in the older version, the keys are: model.layers.8.self_attn.k_proj.weight
and in newer transformers version, the expected keys are: language_model.model.layers.8.self_attn.k_proj.weight
So that my model cannot be loaded correctly.
Is there any way to load the model correctly? Do I have to modify the file llava-v1.5-13b/pytorch_model.bin.index.json ?

seems tensorrtllm just support the huggingface version of llava, and I am using that version of model instead.


mac command use wrong environment variables

Describe the bug
Hi, so I noticed that when I use mac printenv command inside my linux machine during opening multiple tabs in wezterm terminal, I noticed that the mac printenv always print the environment of the first tab instead of the active tab where I type mac command.
Is there any workaround regarding this? Thanks!
To Reproduce
Open two tabs of wezterm terminal, then call mac printenv inside the second tab. The environment printed will be the env of the first tab, not the active tab.
Expected behavior
It should use the current active tab environment.
Diagnostic report (REQUIRED)
No response
Screenshots and additional context (optional)
No response

Which environment variables are you looking at specifically?


Modify spring namespace for standard configuration

Ref #5017

Pull Request Test Coverage Report for Build 11041

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 57.524%

Totals

Change from base Build 11039:
-0.004%

Covered Lines:
11835

Relevant Lines:
20574

💛  - Coveralls


Added gitlink build step

modified build.bat to install gitlink if not exists
added GitLink build target to process output pdb with installer gitlink

I made a new fork, becouse my previous fork has became a mess )

Hmmm, strange, but gitlink wasn't installed during build.bat execution.  On my local machine everything is fine.

Merged manually.


path index

Hi, Jiwoon Ahn,
I wonder to know what is the path index in the code? which part in the paper could I refer to it?
Additionally, when will the training detail be released? Looking forward to following your work.
Thanks.

Hi, @TinyGrass
PathIndex gathers all indices of pixels lying on the straight line between a pair of two coordinates. From these coordinates, we encourage or penalize boundary scores. (Please refer to Sec.4.2 and Eq.7.)
Sorry for missing the deadline I set. I've assigned to other things to do. I'll update the repository no later than the end of this week. :(

Hi, @TinyGrass
PathIndex gathers all indices of pixels lying on the straight line between a pair of two coordinates. From these coordinates, we encourage or penalize boundary scores. (Please refer to Sec.4.2 and Eq.7.)
Sorry for missing the deadline I set. I've assigned to other things to do. I'll update the repository no later than the end of this week. :(

Thank you for your consideration. Best.


[docs] Fix migration guide typo

[x] I have followed (at least) the PR section of the contributing guide.

@paullaros It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!


Method for build Tree from flat source

Building tree from flat source is common task, as I think: items from database query and so on.
But currently building tree from this type of source is complicated.
I suppose to create something like Tree#ofAll(Iterable<T> source, Function<T, ID> idMapper, Function<T, ID> parentMapper) (super/extends hidden for simplicity now).
I already has implementation based on #2121 and can add PR.

I like that idea! Could you please provide an example (input, application, output)?
I think we need a new name for the method, maybe also recurse or build or ??? (see #2125)

Some example:
public class MenuItem {
  Long id;
  Long parentId;
  Long organizationId;
  String name;

  //todo getters and setters a hidden

  public toString() {
    return getName();
  }
}

public class MenuLoader {
  private final EntityManager em;
  public Tree<MenuItem> loadMenu(Long organizationId) {
    TypedQuery<MenuItem> query = em
      .createQuery("SELECT T FROM MenuItem WHERE (T.organizationId = :organization)");
    query.setParameter("organization", organizationId);
    java.util.List<MenuItem> items = query.getResultList();
    return Tree.build(items, MenuItem::getId, MenuItem::getParentId);
  }
}

public class MenuLoaderTest() {
  @Mock
  private EntityManager em;

  @Mock
  private MenuLoader menuLoader;

  @Before
  public prepareDatabase() {
    em.createQuery(
      "INSERT INTO menu_item(id, parent_id, organization_id, name) VALUES"
      + "\n  (1, NULL, 1, 'I')"
      + "\n, (2, 1, 1, 'II')"
      + "\n, (3, 1, 1, 'III')"
      + "\n, (4, 2, 1, 'IV')"
      + "\n, (5, 2, 1, 'V')"
    ).executeUpdate();
  }

  @Test
  private void testLoad() {
    Tree<MenuItem> tree = menuLoader.loadMenu(1);
    assertThat(tree.toLispString).isEqualTo("(I (II IV V) III)");
  }
}

@valery1707 Great, thank you!
The menu items may be iterated in arbitrary order when building the tree. What is the overall complexity of build such a tree when n is the number of menu items?
Naively I would say it is O(n * log n), because we iterate n MenuItems, search the parent in O(log n) and insert the new node in O(1) and rebuild the tree in O(log n) (or eff. O(1)? - which would be the same in O notion).
Or does your implementation require O(n) because it uses the recursion? But then I think the flat items need to have a specific order, right?

My current implementation internally use List#groupBy function for build Map<ID, List<T>> and then use Tree#recurse for build children for founded roots.
Complexity of the List#groupBy is O(n) as I think.
Complexity of the Tree#recurse is about O(log n) if I correct analyze function.
Result complexity of the Tree#build is O(n * log n).
I create PR with implementation in #2129


RFP - Egnyte Connect / Egnyte Drive

Egnyte Connect / Egnyte Drive
Enterprise cloud storage - Windows desktop agent

Mass Deployment Guide, incl uninstall
MSI installer

Status: Review
https://chocolatey.org/packages/egnyte-connect

Status: Published
https://chocolatey.org/packages/egnyte-connect
Can be closed.


Merge 1.1.x into master

Issue
NA
Description
Merge 1.1.x into master

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 1.2.2-merge-1-1-into-master-SNAPSHOT
download it from Artifactory here

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

1.2.2
GitHub release

Your semantic-release bot :package::rocket:


Solve model.cond with custom materializer

Trying to solve #15 using a custom formulaic materializer.

[x] wait for #33
[ ] Testcases:

~ A
~ A + B
~ A * B
~ A + C(A) + C(A, contr.treatment(base=x))
~ 0 + A
~ C(A, contr.treatment(base=x))
~ C(A, contr.sum)
~ A + continuous + np.log(continuous)
~ A * B + continuous + constant

Codecov Report
Attention: Patch coverage is 98.30508% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 70.93%. Comparing base (7610d50) to head (f6be15d).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
+ Coverage   67.67%   70.93%   +3.26%
==========================================
  Files          12       14       +2
  Lines         495      554      +59
==========================================
+ Hits          335      393      +58
- Misses        160      161       +1

Files
Coverage Δ

src/multi_condition_comparisions/_util/__init__.py
100.00% <100.00%> (ø)

src/multi_condition_comparisions/_util/checks.py
100.00% <ø> (ø)

...lti_condition_comparisions/methods/_statsmodels.py
100.00% <ø> (ø)

...rc/multi_condition_comparisions/_util/formulaic.py
98.24% <98.24%> (ø)

Finally made it! As usual with these things it was more complicated than I anticipated, but it seems to work quite robustly now.
High-level description:

Formulaic uses a "Materializer" class that converts the model specification into a design matrix
It is possible and supported to make a custom materializer that extends the base class provided by formulaic (intended to generate something different than a pandas data frame)
I use a custom materializer to hook into the materializer functions to extract factor metadata that we later need for bulding contrast vectors with model.cond.
model.cond uses this information to check categories and to fill in default values. There might be situations in which the default value can't be inferred, in that case we raise an error.

@const-ae, would be great if you could have a look, too. In particular, if the test cases are correct and if there's any additional (more complex?) model you think that should be tested.

I can understand that it all looks a bit daunting. I suggest to start reading at def cond(), this is the actual user-facing function to generate contrast vectors. All the other fuzz is about finding the default categories of each categorical variable specified in the model.


fruitcake/php-cors v13 needed but not match

Laravel Version
11
PHP Version
8.2.13
Database Driver & Version
No response
Description
when I create a new project of laravel11(composer or laravel new),I get these error messages :Problem 1     - laravel/framework[v11.0.0, ..., v11.2.0] require fruitcake/php-cors ^1.3 -> found fruitcake/php-cors[dev-feat-setOptions, dev-master, dev-main, dev-test-8.2, v0.1.0, v0.1.1, v0.1.2, v1.0-alpha1, ..., 1.2.x-dev (alias of dev-master)] but it does not match the constraint.     - Root composer.json requires laravel/framework ^11.0 -> satisfiable by laravel/framework[v11.0.0, ..., v11.2.0].
Steps To Reproduce
Problem 1     - laravel/framework[v11.0.0, ..., v11.2.0] require fruitcake/php-cors ^1.3 -> found fruitcake/php-cors[dev-feat-setOptions, dev-master, dev-main, dev-test-8.2, v0.1.0, v0.1.1, v0.1.2, v1.0-alpha1, ..., 1.2.x-dev (alias of dev-master)] but it does not match the constraint.     - Root composer.json requires laravel/framework ^11.0 -> satisfiable by laravel/framework[v11.0.0, ..., v11.2.0].

No steps to reproduce.

i just had this error when installing new laravel project

It seems that you are using an unofficial packagelist source that hasn't synchronized the latest 1.3.0 tags yet. You can try replacing your packagelist source with the default official source (url: https://repo.packagist.org/).


userBadges object is always empty, even when a sender is a Moderator

Hello,
I found a bug in the data structure of userBadges object when receiving a chat, it's always empty, no matter if the sender is a Moderator or not.
Is it related to this library or the issue is from TikTok's API ?

I will check. It is possible that the data structure has been changed by tiktok.

Should be fixed in the latest version.


Qt: Session management error: Could not open network socket

When I try to run math2pix-snipping-tool it gives this error. How can I solve the problem
Qt: Session management error: Could not open network socket
QNetworkManagerInterface::QNetworkManagerInterface(QObject*) propsReply "An AppArmor policy prevents this sender from sending this message to this recipient; type="method_call", sender=":1.195" (uid=1000 pid=6378 comm="mathpix-snipping-tool " label="snap.mathpix-snipping-tool.mathpix-snipping-tool (enforce)") interface="org.freedesktop.DBus.Properties" member="GetAll" error name="(unset)" requested_reply="0" destination="org.freedesktop.NetworkManager" (uid=0 pid=930 comm="/usr/sbin/NetworkManager --no-daemon " label="unconfined")"
QNetworkManagerInterface::QNetworkManagerInterface(QObject*) nmReply "An AppArmor policy prevents this sender from sending this message to this recipient; type="method_call", sender=":1.195" (uid=1000 pid=6378 comm="mathpix-snipping-tool " label="snap.mathpix-snipping-tool.mathpix-snipping-tool (enforce)") interface="org.freedesktop.NetworkManager" member="GetDevices" error name="(unset)" requested_reply="0" destination="org.freedesktop.NetworkManager" (uid=0 pid=930 comm="/usr/sbin/NetworkManager --no-daemon " label="unconfined")"
"Object path cannot be empty"
libGL error: MESA-LOADER: failed to retrieve device information
MESA-LOADER: failed to retrieve device information
MESA-LOADER: failed to retrieve device information
Killed

mathpix-snipping-tool is not developed by me. please reach out mathpix via<EMAIL_ADDRESS>