Button that clears search bar inputs

Is your feature request related to a problem? Please describe.
This is a quality of life request. If camper interact with the search bar, fill it with their inputs, then wants to clear their input, they have to select all their text then delete it.
Describe the solution you'd like
We can add a button in the search bar that clear the search bar for new camper inputs. Here is the functionality that I am talking about.
https://user-images.githubusercontent.com/88248797/205431779-69d670db-f954-4d82-93ee-73e8f3ad7150.mp4
Here is the website, if you want to mess with their search bar https://service-manual.nhs.uk/search
Describe alternatives you've considered
This is quality of life, it's fine to ignore it if it harms accessibility.
Additional context
You may have noticed that clicking the icon in the video doesn't close the keyboard, it would be lovely to implement that too, but we have to account for accessibility concerns as mentioned in this comment https://github.com/freeCodeCamp/freeCodeCamp/pull/48586#issuecomment-1335724747
Accessibility comes before implement this.

it's fine to ignore it if it harms accessibility

I don't see why this feature would harm accessibility, that is as long as it is implemented accessibly :slightly_smiling_face:
I do find it interesting that the site you referenced above only implements this when it detects someone is using a touch interface (at least that's what appears to be  happening for me). I would think that keyboard users might also find this convenient. And possibly even mouse users. My point being that we should implement this feature for everyone instead of trying to impose our own ideas on who we think would benefit from it.
As far as implementation is concerned, this appears to be just a simple button that when clicked will clear the current typed input in the input field and set the focus back to that input field. The accessible name would be something like "clear search terms". Whether it is placed before or after the actual search button in the DOM can be discussed. If we are going to show the button inside the input box, to the left of the search button then my preference would be that it come before the search button in the DOM so the tabbing order follows the visual order.

I don't see why this feature would harm accessibility, that is as long as it is implemented accessibly

Nice.

why this feature would harm accessibility

I think I was clouded halfway through creating the issue, because I was thinking of Run button 🤔. Well, I hope so at least 🤣

I do find it interesting that the site you referenced above only implements this when it detects someone is using a touch interface (at least that's what appears to be happening for me). I would think that keyboard users might also find this convenient. And possibly even mouse users. My point being that we should implement this feature for everyone instead of trying to impose our own ideas on who we think would benefit from it.

I think they are doing it to account for people with Parkinson, as they may have difficulty moving the mouse and stuff a like. I am not sure, tho. Here is an article about Parkinson https://www.nia.nih.gov/health/parkinsons-disease
This doesn't apply to us because the search function won't be close to the clear icon, so it will be friendlier to them. I agree that implementing everywhere is a plus for us.

Ahh, let me revise that last statement. I was looking at the example and not the actual FCC search bar. The FCC search bar has the search icon on the left side of the input. So if we add the clear button at the end of the input then it will visually be to the right of the search button and thus it should probably be added after the search button in the DOM.

I agree, too, let's add a bigger spacing between the icon and the text in the long search bar inputs, inputs like "How do you create an accessible icon for searchbar component?", to make it easier to not miss-click the icon

Thanks for bringing this up @bbsmooth, and for your insight @bbsmooth.
It's been so long now that I don't remember why we decided to hide the clear button, but @ahmadabdolsaheb is correct — react-instantsearch-dom includes both a search and clear button.
The search button is still shown, and is the magnifying glass on the left side of the search bar. And the clear button is an "X" on the right side of the search bar, and is hidden by some CSS here.
This should be pretty easy to fix. We just need to remove that bit of CSS and align the clear button within the search bar.
I believe react-instantsearch-dom will take care of the rest. The clear button is hidden by default and appears when someone starts typing. Clicking or tapping on the clear button clears the search bar and closes the dropdown if it's open.
It also has a title attribute with "Clear the search query." by default. Unfortunately I don't think we're able to change the title for the search or clear buttons.


Display character in indentation headers

Instead of (or along with) just hilighting the space, it’d be nice to be able to show (a) character(s) there.

A bit over-my-head, as I’m new to vim, but I was afraid of that in any case. :list is definitely not enough, as hard tabs are a travesty. /=
I suppose leave the issue open until such a time as Vim has some sort of feature that could support this functionality?

As @graywh said, I don't think this is possible in Vim currently. Closing this issue for now, will re-open if it ever becomes possible.

@nathanaelkane thanks for paying attention, in any case! Here's hoping vim eventually supports it.
On that note, should we write to the mailing list or something? How does one go about getting vim modified, short of learning the codebase and patching it yourself?


Improve desired state resource details page + fix wrong resource deta…

Description
This is basically ticket #2712 for iso4. The conflicts were easy to fix.
closes #3083
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches iso4 in 9600066c479a8e14011575e9aac462a55086d274


Fix bugs in the CLI unit tests

Address "BUG" & "QUESTION" comments in test_command_line_parser.cc
Throw PA exception instead of exiting the program to match the behavior of test to behavior of production code

Looks great overall! Thanks for working on this! 🙏


Improve @AwsTests / fix credential mgmt in tests

The AwsTests group, i really have no idea how to run this: somehow it requires some special credentials. This makes it hard to debug any issue. I think we need to get away from these integration-type tests as quickly as possible, just trying to fix plugin builds I had to do things like install erlang and startup various services manually.
We should test primarily with unit tests.
Separately, when these things do run, they try to read from strange places on the filesystem, presumably because special credentials location was passed in on the command line. For now I disabled security manager for this plugin, because I have no idea how to reproduce these tests, and it needs some work.

I do agree that we should try to add more unit tests which are always executed.
That said, our Jenkins instance is running our integration tests using this doc and internal credentials we have.
In other projects such as Azure and GCE plugins, I tried to add Mock objects to simulate Azure or GCE API calls which helps a lot in that case. Do you suggest we do mocking here as well or may be split big methods in more "testable" methods?

I like with azure there are lots of tests that always run! I will look into the doc, thanks!

@dadoonet thanks for the links. I am digging into this. We have quite a few plugins that need this same mechanism so I will try to sort it out tonight.

See https://github.com/elastic/elasticsearch/pull/11134 which will fix the security manager issues.
I want to follow-up after that with moving tests.config to elasticsearch-parent, and try to add some generic tests group/parameter to elasticsearch-parent as a replacement for tests.aws/tests.gce/tests.azure
This way, aws/gce/azure plugins don't need custom mvn test logic and can just re-use elasticearch-parent logic.

Fixed by elastic/elasticsearch#11134 . I opened an issue in es-parent to discuss how to remove custom test logic (https://github.com/elastic/elasticsearch-parent/issues/41) in each of these builds so that es-parent logic can be reused.


1k stars - hoooraaayy

Haha. :tada:
@EmilTholin is the 1000th!

I just had to make a gif out of it when I saw the star count!

you guys are hilarious 🤣

Haha, we could close that. :)


Update Posts “comment-jutilise-un-clavier-50-comme-clavier-principal”

Automatically generated by Netlify CMS

👷 Deploy Preview for shiny-babka-998c7e processing.

Name
Link

🔨 Latest commit
bf32040a4a1b4fd48217c113af7006ef9450c332

🔍 Latest deploy log
https://app.netlify.com/sites/shiny-babka-998c7e/deploys/6332fc9565642f000ae8e8fb


Proposed Element: individual accountability elements

The current elements list goes a long way towards establishing organizational accountability and transparency, but individuals involved in the process may avoid scrutiny.

contracting officer (and attributes such as tenure, career history, training received, etc)
cotr and associated attributes
government executive sponsor and attributes

Even if the identities were preserved, the attribute information could provide details into training deficiencies and/or evidence of bias towards favored contractors.

I think this is an outstanding idea! @dataconsultant While some detail like career training may be too personal, the CO that signed off with government authority should be included for sure.  The history of the CO's experience can be built from the data over time (e.g., which agencies, types of contracts, preferred contract vehicles, socioeconomic set asides, etc.).
The government sponsor element is also interesting.  Though I can see this being a challenge as likely the sponsorship process is different across agencies.  But in almost all case there is a program/functional person (like the CIO shop for IT) that put their name on the item when sending it over to procurement to acquire.  It would be interesting to see how many IT commodities are acquired outside of the CIO organization.  This could also be a method for augmenting the lack of a 'program' element since awards could be aggregated by 'government sponsor' which could create clusters of 'program' activity.
This also would require metadata / reference data - the hierarchy of the government - to enable drill up/down/across.


Add support for custom routing keys.

This extends the work that @wolfeidau did in #7 by adding a very basic test to ensure that messages can be produced and consumed with the routing key. I also renamed ConsumerByRoutingKey to ConsumerWithRoutingKey to be consistent with the API of the Producer.
This should supersede #7.

Thanks!


[Unsupported] Read repairs implementation

In order to repair mismatched replicas, we need to repair them based on some agreed
upon value. In most systems that usually is based on the timestamps. Since Redis does
not support timestamps, we need to add the logic that tracks timestamps as metadata.
The way we achieve recording timestamps is by converting all writes to Lua scripts that
atomically do the operation itself and store the metadata for that key.
The metadata is stored in what we call 'add sets' and 'remove sets' which are basically
sorted sets that store the key name along with a corresponding timestamp that denotes
when that key was last updated. All primary keys go to a 'top level add/rem set' which
is currently denoted by '._add-set' and '._rem-set'.
Secondary keys (fields in hash maps, sets, etc.) are stored in add/rem sets of their
corresponding primary keys and are denoted by '.add-set' and '.rem-set'.
A majority of the logic lives in src/proto/dyn_redis_repair.c.
The 'redis_cmd_info' static array of structures contains information that will be used
by  the "rewrite with metadata" and "repair" logic. This array is indexed by the command
index that is set in dyn_message.h.
All queries that will be rewritten to include metadata are denoted by
'redis_cmd_info[CMD_IDX].has_repair_support' where 'CMD_IDX' is the index of the
corresponding query. All read queries that return the requested values along with the
metadata are denoted by 'redis_cmd_info[CMD_IDX].is_repairable'.
If queries do not have "rewrite with MD" or "repair" support, we fallback to the old
behavior which is based on quorum checksums.
Every repairable read ('is_repairable' == true) returns data in the Redis wire protocol
in the following format:
<status> <ts> <value>
, where status can be one of 'X', 'R' or 'E'. They stand for:
X = value not found
R = value was deleted (i.e. it was found in the rem-set)
E = value exists (it was found in the add-set)
This format was chosen because it's the most optimized in terms of returning values to
the client, i.e. we can just move the start pointer of the buffer just before ''
by skipping '' and '' thereby avoiding the need to copy the value to yet
another buffer.
Currently the only read commands that have repair support return only one value at a
time. In the future when queries that can return multiple values will be returned,
the format will be naturally extended to be:
<status_1> <ts_1> <status_2> <ts_2> .. <status_n> <ts_n> <val_1> <val_2> .. <val_n>
This will allow us to skip all the metadata with a simple pointer move just as we do
now.
On repair supported reads, if we find that the timestamps between responses from the
responding replicas are not the same, we pick the replica that has the largest TS, take
its value and send a new write command to the replicas that are out of date with the
most updated value.
Since we need to know the exact keys, fields and values while writing with metadata or
while repairing, we do a "post-parsing" step that records these in order to efficiently
populate the arguments for the Lua script that we will send to Redis on behalf of the
read/write. (see post_parse_msg())
This may change if we modify the parser itself to record these tokens efficiently.
TODO: Support parsing of optional fields.
The Lua scripts that we use as templates for supported commands are listed below as
compile time constants.
This is still in the beta stage and has some limitations:

Large values may cause overflows ( > 512 bytes)
Limited command support due to parser limitations:

Sorted sets and lists need optional command parsing support.

Repairing on the read path vs. the background has perf implications. In the future,
all the heavy lifting will happen in the background in order to have a 0 perf
overhead on the read and write paths.

On repair supported reads, if we find that the timestamps between responses from the
responding replicas are not the same, we pick the replica that has the largest TS, take
its value and send a new write command to the replicas that are out of date with the
most updated value.

If there has been a mutation after we have chosen the replica with the largest TS on read, do we compare the TS (again) on updating the value?

If there has been a mutation after we have chosen the replica with the largest TS on read, do we compare the TS (again) on updating the value?

Yes, we do. The Lua script updates only if it doesn't see a TS later than the one it's trying to repair with.

Limited command support due to parser limitations:

Since we are going to have a follow-up PR for that, shall we create an issue to centrally track the progress of the feature?

Good point. I added it here:
https://github.com/Netflix/dynomite/issues/660

Bad rebase, will push a different PR and close this.


Ft/backbeat md updates

Dependency of https://github.com/scality/backbeat/pull/75

PR has been updated. Reviewers, please be cautious.

r+

@ironman-machine r+

:hourglass: PR is now pending. CI build url: http://ci.ironmann.io/gh/scality/Integration/14602

Hello @electrachong
"r+": Success

@ironman-machine r-

Hello @electrachong
"r-": Success

@ironman-machine try

Hello @jonathan-gramain
"try": Success: Try build successfully launched on 'http://ci.ironmann.io/gh/scality/Integration/14603' with the following env. args:
{
    "SCALITY_INTEGRATION_BRANCH": "ultron/master",
    "REPO_NAME": "S3",
    "DEFAULT_BRANCH": "master",
    "SCALITY_S3_BRANCH": "ft/backbeatMDUpdates"
}

:sunny: :+1: circleCI test succeeded!


[CARBONDATA-1693][Streaming] Change SegmentStatus from String to enum

Currently Segment status definition is scatter in multiple class, this PR unified them into a Enum class.

[X] Any interfaces changed?
SegmentStatus enum is added

[X] Any backward compatibility impacted?
No

[X] Document update required?
No

[X] Testing done
No new testcase added

[X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/940/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1555/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/944/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1559/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/959/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/963/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1574/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1578/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/972/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/978/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1586/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1589/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1027/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1028/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1645/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1646/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1036/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1654/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1038/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1656/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1040/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1658/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1045/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1663/

retest this please

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1050/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1668/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1065/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1069/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1676/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1679/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1092/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1693/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1103/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1702/

LGTM


Ability to delete a review

So I was messing around with cargo-crev and I ended up reviewing the same crate (bigint) twice. So now, I have this:
flagged   2  2    17700     91937 0/2 bigint               4.4.1

I think a command should be provided for a user to delete his own reviews

Yeah. I'd like to have it.
Right now you can just point your text editor to on of the files in ~/.config/crev/proofs/*` and just delete it. Proof Repository is just a normal git repository with text files.

Also, newer reviews are supposed to overwrite older ones, so technically there's no need to remove old ones.

@mglax I've landed code it master, that implements the review being overwritten. All the counts and listings should not use the old review of the same package. Please double check if you can. :)

Seems like it's working

Before I close this: when writing a review for crate version that was already reviewed, maybe we should start from the content of the previous review?

when writing a review for crate version that was already reviewed, maybe we should start from the content of the previous review?

Are you only talking about your own reviews or also reviews by others? Since I think reviews are only really meaningful if you understand/reviewed the complete crate, independent seemingly harmless partial reviewed commits/versions can create a malicious update in the end.

Whatever is decided, it should be implemented in #113


Try out idempotent pull

The reset script from https://github.com/pondersource-books/time does not survive being run twice, it would lead to duplicate data.
It would be nice to have a script that just pulls the latest data from timeld-org and timesheet-dev3-evoludata-com and repairs / complements whatever is already in the database.

See if I can get that working today so I can still include it in https://github.com/federatedbookkeeping/timesheets/issues/52


Make challenge Helmet 4 Compatible

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Blocked by: https://github.com/freeCodeCamp/boilerplate-project-secure-real-time-multiplayer-game/pull/10

This relates to the following issue and pull request.
Issue: https://github.com/freeCodeCamp/freeCodeCamp/issues/40476
Pull Request: https://github.com/freeCodeCamp/freeCodeCamp/pull/40900

helmet.hidePoweredBy removes the X-Powered-By header, which is set by default in some frameworks (like Express). Removing the header offers very limited security benefits (see this discussion) and is mostly removed to save bandwidth.
- https://helmetjs.github.io/

On the same page, it's recommended not to use app.use(helmet.hidePoweredBy()) and instead opt for app.disable("x-powered-by"), anyway.

I updated the test to only verify that the x-powered-by header isn't equal to Express, this logic is stolen from a challenge before the projects.
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/09-information-security/information-security-with-helmetjs/hide-potentially-dangerous-information-using-helmet.hidepoweredby.md#--hints--
I removed the instructions suggesting that<EMAIL_ADDRESS>is required to complete the user stories.

Hey @SethFalco are you still working on this draft? Or is it ready for review?

Hey @SethFalco are you still working on this draft? Or is it ready for review?

Sorry about that, I've been busy, so I haven't been able to finish this yet.

Converting this to a draft, I thought I had tested this properly, but seems I made a mistake.
app.use(helmet.xssFilter()); now disables XSS filtering. This results in the x-xss-protection header being set to 0.
I think this test should be replaced with using an appropriate Content Security Policy instead that doesn't allow unsafe-inline in JavaScript at least. I'll update this PR later.
- https://github.com/freeCodeCamp/boilerplate-project-secure-real-time-multiplayer-game/pull/10#issuecomment-803071398

It's in a draft state because of the above, I can try to come back to this on the weekend and get it in a reviewable state.

Sounds good - thanks for the update 🙂

Applied the change now, I referenced another challenge to check the CSP.
I might update it later to more specifically check that unsafe-inline is not used, rather than checking if it's set to safe.
Just not sure should the task should specify to use self, or if the test just check if it doesn't contain unsafe-inline.
If we're going to check if it's not unsafe-inline, it might be a good idea to depend on a package that will actually parse the content-security policy properly. It's certainly possible through regex as well, but across challenges using a CSP parser would be cleaner, I think.
I'll check why the build is failing in the evening.

Build error is unrelated to your PR - I'll try re-running the action to see if it passes.
/cc @ojeytonwilliams same caching issue on this Cypress run.

Thanks for the heads up @nhcarrigan.  Still figuring this one out, but I tried merging in main in the hope that that would bring the webpack-runtimes into alignment.
@SethFalco sorry about this, hopefully the merge commit will sort this out.  If not, we can still merge, even if that test fails.

Just dropping my opinion on this. Please note I'm just a contributor, I may've misunderstood the objectives of freeCodeCamp or contribution guidelines.
TL;DR: I think ideally updatable educational material should take advantage of this and update library versions as well, especially major versions. My opinion is that the real solution is to make this a draft while PRs are made to get the curriculum up-to-date, rather than settle on leaving it as Helmet 3.

We could adjust it to accept the old way or how this PR has it to account for both. That would really be the only thing to change here to keep it backward compatible.

This didn't occur to me, I agree making the tests compatible with version 3 and 4 is a good idea.
I'll try to update the PR to do that, just have to think about what's the best approach.

What happens when helmet 5x changes more things. Keep updating things, I suppose.

Ideally, I do think it'd be nice if new students were taught the up-to-date versions of software where possible. I recognize with how rapidly software changes this can prove difficult, but it's certainly an ideal.

The main learning goal here is more about headers and security than the specific methods or versions of a package used.

If the main goal is to learn about headers and security, I feel it would've made more sense for users to just set the headers directly instead of using Helmet? Fewer dependencies, and get to learn about headers more directly.
For example, since I opted to use Helmet 4, and just set the headers myself.
app.use((req, res, next) => {
  res.set('x-powered-by', 'PHP 7.4.3');
  res.set('x-xss-protection', '1; mode=block')
  next();
})

Helmet removed the nocache method into a separate package (which is added in the boilerplate PR). The curriculum doesn't mention that anywhere

Agreed, I believe the ideal solution would be to turn this PR into a draft, and I can separately make a PR which includes this information in the curriculum.

I dunno, maybe I'm being stubborn.

On the flip side, maybe I'm being petty.

As a new contributor, I wouldn't mind updating the curriculum to teach version 4, and all projects to use version 4 with version 3 compatibility, it might take a while though. It may be a nice way to try to get comfortable with the project, so I contribute more in future.
I'd just like to see people be able to use Helmet 4 immediately when they take this knowledge to personal projects, or to the workplace. Not realize some of the practices they were taught were outdated and have to read the through Helmet documentation to see what changed, and maybe MDN Web Docs, and issues on the Helmet repo to see why.
Helmet isn't the only example either, I'd love to see the D3 tutorials updated from v4 to v6 too, I believe that one wouldn't require changing tests, just how some asynchronous methods are called.

@SethFalco Thank you, for sharing. You are spot-on with a lot of your points. I will just add this, and no more, as a PR is not the place to have too much discussion on this topic:
With version 7 of the curriculum being the main focus of the freeCodeCamp team's energy, the current curriculum is not receiving massive changes; typically, bug fixes are done, because they can be frustrating to learners, and it makes sense to do whilst the curriculum is still active (online). This is why the main suggestion for the mentioned issue has been to quickfix the bug - we are slowly learning to freeze the versions of the tools we use.
Absolutely, Campers should be taught up-to-date tech. However, as tech moves so fast, and the core fCC team is so small, there is little-to-no scope to continually update a curriculum which is scheduled for obsolescence. That is the issue in its entirety - scope/man-power.
Now, I think a complete overhaul of this section is unlikely to do much good, because:
a) It will take a lot of time, leaving the issue an issue for too long
b) It could very well only be on the curriculum fore-front for a short amount of time
However, I appreciate your willingness to contribute in this way. So, I will leave this to be decided by the others, as they know the curriculum plans much better.

Since the consensus was to not add the change and instead ensure this is covered in the upcoming curriculum going to close this.


ResourceExhausted: 429 The following quota metrics exceed quota limits

Hi! Thanks for building this amazing project. Recently I'm running script on xmanager+vertex.AI on TPU v2 and v3, but I keep getting this error:
google.api_core.exceptions.ResourceExhausted: 429 The following quota metrics exceed quota limits: aiplatform.googleapis.com/custom_model_training_tpu_v2

The error is thrown at this line - https://github.com/deepmind/xmanager/blob/v0.2.0/xmanager/cloud/vertex.py#L181.
Below are the sanity checks that I've done:

I found the service account here can be loaded nicely, tho it would soon be assigned to `None here as I'm requesting TPU v2 or v3.
tensorboard is set to empty string.
the self.location, self.project, pools and auth.get_bucket() all look good. where the location is us-central1, and pools showing --

[machine_spec {
  machine_type: "cloud-tpu"
  accelerator_type: TPU_V2
  accelerator_count: 8
}

I've enabled the three APIs mentioned in the readme (IAM, Cloud AI Platform, Container Registry), additionally Vertex API and Cloud Resource Manager API was enabled. I also checked the Quota page on the console, which looks fine as well. Doesn't look like I'm overusing the resources as described in the error message "exceed quota limits".
It's been bugging me for quite a few days, and would be really appreciated if anyone could suggest what's possibly going on there. Thanks in advance!

Sorry forgot this - I'm using Python 3.9 and xmanager==0.2.0. lmk if any more info is needed from me

I have the same issue can anyone help?


Relation type consistency across specs

The child rel type for catalogs should also allow to link to collections, otherwise collections in catalogs would not really work.
There are also other discrepancies for the root/parent/child rel types in the catalog and collection specs. Tried to fix them.
Removed requirement to have an absolute URL in as root in the catalog. We don't follow this in our examples anyway and it is not required in collections and items. Is that okay or should we aim for something different here?

Please review carefully. Thanks!

I don't remember the whole conversation, but I think removing the absolute URL requirement might be an issue? I don't personally have a strong opinion, but I think that might have been the compromise that we came to in the Menlo park sprint when discussing relative urls. I think the idea was that the root was absolute so to anchor everything? Feel free to correct me if that is wrong, but it might be a good idea to do that in a separate PR.

@hgs-truthe01  As far as I remember from the Menlo Park sprint, we decided to require absolute self URLs to resolve the relative URLs in the other links/anchor everything. I think that makes sense and in items the root was not required as absolute URL, too. So I think this was just an outdated catalog specification. Please correct me if I am missing something here.

@m-mohr That sounds right to me. Mostly I just hesitate to have a change ride on a bug fix last minute for the release. I don't feel a need to hold up the PR though.


chore: Change cors setting to allow same origin only

:tada: This PR is included in version 2.0.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added Ising model sample

This adds Ising model sample with explanations.

Thanks @DmitryVasilevsky , this would be the third Ising model simulation in the repository alongside https://github.com/microsoft/qsharp/blob/main/samples/estimation/estimation-ising-2D.ipynb and https://github.com/microsoft/qsharp/blob/main/samples/estimation/Dynamics.qs. Maybe we could remove Dynamics.qs in favor of this file and load this new file into estimation-ising-2D.ipynb and only do the explanation and RE in there.
Second, it would be nice, if the the simulation was wrapped into a quantum phase estimation to determine the ground state energy.

I skimmed through the code and here is one thought I had. @msoeken there is a difference in the estimation-ising-2D and this Ising model simulation in that the former is doing 4th Order Trotterization and this code does 1st Order Trotterization.
As a demo, both do work. However, for utility scale we would want to use 4th order formulas as they minimize errors in the final output.
I do agree that the Dynamics sample and the estimation-ising-2D can be rolled into one.
@DmitryVasilevsky if you can wrap this code with phase estimation (i.e., the simulation will serve as the oracle for phase estimation) as @msoeken suggested, that will be a valuable addition as none of our samples show that at this time.

Thanks @DmitryVasilevsky , this would be the third Ising model simulation in the repository alongside...

We discussed this and we would like to have a simple standalone sample of Ising model (a more advanced algorithm) that is suitable for base profile. Changing existing samples and adding phase estimation we may consider later.

As a demo, both do work. However, for utility scale we would want to use 4th order formulas as they minimize errors in the final output.

Thank you for reviewing! We would like to have a standalone simple sample of a more advanced algorithm. Hence this sample. It is suitable for base profile. Changes to other samples and phase estimation for eigenvalue we may consider later; I will open an issue for this.

Added issue https://github.com/microsoft/qsharp/issues/2067


merge me_dev to dev

Small update with an ME testing fix and a BedRock coherence protocol specification update.

Did that bedrock pdf move somewhere else?

Did that bedrock pdf move somewhere else?

No. The diff does make it look like it is being deleted / moved, but it is definitely present on the me_dev branch and when I do a me_dev to dev merge locally, it is still there on dev post merge...so I think just weird display here.

Weird, works for me though. Merge away with CI

Had to revert the bp_top testbench Makefile.cfg change. The NCPUS variable is overloaded in its use as both number of processor cores and number of CCEs (the latter being more important since it is passed to the NBF generator and used to properly configure the CCEs).


Does not work with package:individually:true

I see Error Serverless: Uploading function .zip files to S3...
Error --------------------------------------------------
ENOENT: no such file or directory .webpack.serverless\servicename-funtionName.zip'

In zip archive I see all entry points but want only required by function handler in serverless.yml

I can reproduce this MacOSX - v1.0.0-beta.2.2
Error --------------------------------------------------
     ENOENT: no such file or directory, open '/Users/csilk/projectName/.webpack/.serverless/functionName.zip'

Seems to be building them properly into the .serverless dir just can't reference them which is strange
Could it be to do with setting the serverless.config.servicePath step in compile.js?
Am happy to throw my time at fixing this as it's a blocker for me just don't really know where to start
Cheers

Looks like this is fixed with #48
Waiting for merge

merged. keep an eye for v1.0.0 which should be released soon


fix: align template api response time type with metrics api response time type

Issue
https://github.com/gravitee-io/issues/issues/7094
Description
In the gravitee-reporter-api metrics class the api response time is a long.
Now let's imagine you create an API on APIM and link it to an endpoint which send events using SSE. If you keep the stream alive for a time ( in millisecond ) greater than the interger max value you can have an error saying.
To reproduce the bug you'll have to wait 3 weeks before closing your http request or play with the time itself 😛
NB: When this pull request will be merge I'll cherry pick the commit to have the fix on all other versions.
17:43:51.806 [vert.x-eventloop-thread-10] [] ERROR i.g.e.client.http.HttpClient - An error occurs while indexing data into ES: indice[gravitee-request-2022.06.09] error[failed to parse field [api-response-time] of type [integer] in document with id '224c62d3-d3ce-4447-8c62-d3d3ceb4471b'. Preview of field's value: '271640000000']

Shouldn't we backport this fix on the master branch too?

Shouldn't we backport this fix on the master branch too?

Yes in the description of the pull request I mentioned that I will cherry pick the fix everywhere and update all the plugin versions on all supported versions

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

3.8.6
GitHub release

Your semantic-release bot :package::rocket:


Windows XP support?

Nocode only runs on XP Service Pack 2 or higher.

I tried this on Windows 98, no complaints.
No failures in test suite either.


fix: Resolve various UI & UX issues

:tada: This PR is included in version 2.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add device and viewport emulation to the examples

To better demonstrate the behavior of components:

adds the ability to toggle different widths for example previews
adds the ability to contain example previews that are sized according to the viewport
notates if an example is meant to represent a viewport

Preview:

the overlay is now contained to the example body, instead overlaying all of the documentation: example
above each example there are icons which can toggle different widths: example

Issue: https://github.com/patternfly/patternfly-next/issues/66

PR: https://github.com/patternfly/patternfly-next/pull/67


Design time builds can explode if other builds in progress

Details over here: https://github.com/OmniSharp/omnisharp-vscode/issues/3213

This happens to me almost every time I switch branch. Any updates on fixing?
Exception details are more or less the same as in linked issue, but triggered by an OnDeleted callback:
Unhandled Exception: System.InvalidOperationException: The operation cannot be completed because a build is already in progress.
   at Microsoft.Build.Shared.ErrorUtilities.ThrowInvalidOperation(String resourceName, Object[] args)
   at Microsoft.Build.Execution.BuildManager.BeginBuild(BuildParameters parameters)
   at Microsoft.Build.Execution.BuildManager.Build(BuildParameters parameters, BuildRequestData requestData)
   at Microsoft.Build.Execution.ProjectInstance.Build(String[] targets, IEnumerable`1 loggers, IEnumerable`1 remoteLoggers, ILoggingService loggingService, Int32 maxNodeCount, IDictionary`2& targetOutputs)
   at Microsoft.AspNetCore.Razor.OmniSharpPlugin.DefaultProjectInstanceEvaluator.Evaluate(ProjectInstance projectInstance) in /_/src/Microsoft.AspNetCore.Razor.OmniSharpPlugin/DefaultProjectInstanceEvaluator.cs:line 86
   at Microsoft.AspNetCore.Razor.OmnisharpPlugin.MSBuildProjectManager.RazorDocumentChanged(RazorFileChangeEventArgs args) in /_/src/Microsoft.AspNetCore.Razor.OmniSharpPlugin/MSBuildProjectManager.cs:line 112
   at Microsoft.AspNetCore.Razor.OmniSharpPlugin.MSBuildProjectDocumentChangeDetector.FileSystemWatcher_RazorDocumentEvent(String filePath, String projectDirectory, ProjectInstance projectInstance, RazorFileChangeKind changeKind) in /_/src/Microsoft.AspNetCore.Razor.OmniSharpPlugin/MSBuildProjectDocumentChangeDetector.cs:line 126
   at Microsoft.AspNetCore.Razor.OmniSharpPlugin.MSBuildProjectDocumentChangeDetector.<>c__DisplayClass7_0.<ProjectLoaded>b__1(Object sender, FileSystemEventArgs args) in /_/src/Microsoft.AspNetCore.Razor.OmniSharpPlugin/MSBuildProjectDocumentChangeDetector.cs:line 86
   at System.IO.FileSystemWatcher.OnDeleted(FileSystemEventArgs e)
   at System.IO.FileSystemWatcher.CompletionStatusChanged(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* overlappedPointer)
   at System.Threading._IOCompletionCallback.PerformIOCompletionCallback(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* pOVERLAP)

This will be fixed in the next release 😄


Refactoring/convert base tests to server live test case

What's this PR do?
It changes the base tests from TestCase to StaticLiveServerTestCase
Where should the reviewer start?
base/test_views.py
How should this be manually tested?
python manage.py test
Any background context you want to provide?
The url for this server also changes to port 8081
This template was adapted ~stolen~ from Quickleft/Sprint.ly

Codecov Report

Merging #11 into master will increase coverage by 23.82%.
The diff coverage is 97.56%.

@@             Coverage Diff             @@
##           master      #11       +/-   ##
===========================================
+ Coverage   55.88%   79.71%   +23.82%
===========================================
  Files          16       26       +10
  Lines         102      138       +36
===========================================
+ Hits           57      110       +53
+ Misses         45       28       -17

Impacted Files
Coverage Δ

jp2_online/settings/base.py
95.23% <ø> (ø)
:white_check_mark:

captura/apps.py
100% <100%> (ø)

familias/apps.py
100% <100%> (ø)

administracion/migrations/0001_initial.py
100% <100%> (ø)

base/test_views.py
100% <100%> (ø)
:white_check_mark:

administracion/apps.py
100% <100%> (ø)

estudios_socioeconomicos/apps.py
100% <100%> (ø)

perfiles_usuario/apps.py
100% <100%> (ø)

indicadores/apps.py
100% <100%> (ø)

becas/apps.py
100% <100%> (ø)

... and 17 more

Continue to review full report at Codecov.

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

🎵 It's all good, baby baby 🎵


Python 3.11.2: pip install -r requirements-310.txt fails

Describe the bug
os: macOS 13.3
python: v3.11.2 (venv)
python -m pip install -r requirements-310.txt
Configured for RELEASE build (set with cmake -DCMAKE_BUILD_TYPE={release,debug,...})
      -- Build Type: RELEASE
      -- Generator: Unix Makefiles
      -- Build output directory: /private/var/folders/cp/ckllsv255631cqswdnxgb5ch0000gn/T/pip-install-c9_6i9ae/pyarrow_1d986c71873643a0a659c11814f85083/build/temp.macosx-13-x86_64-cpython-311/release
      -- Found Python3: /Users/r4ntix/Workspace/github/arrow-ballista-python/venv/bin/python (found version "3.11.2") found components: Interpreter Development.Module NumPy
      -- Found Python3Alt: /Users/r4ntix/Workspace/github/arrow-ballista-python/venv/bin/python
      -- Found PkgConfig: /usr/local/bin/pkg-config (found version "0.29.2")
      -- Could NOT find Arrow (missing: Arrow_DIR)
      -- Checking for module 'arrow'
      --   No package 'arrow' found
      CMake Error at /usr/local/Cellar/cmake/3.26.2/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
        Could NOT find Arrow (missing: ARROW_INCLUDE_DIR ARROW_LIB_DIR
        ARROW_FULL_SO_VERSION ARROW_SO_VERSION)
      Call Stack (most recent call first):
        /usr/local/Cellar/cmake/3.26.2/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
        cmake_modules/FindArrow.cmake:450 (find_package_handle_standard_args)
        cmake_modules/FindArrowPython.cmake:46 (find_package)
        CMakeLists.txt:231 (find_package)

      -- Configuring incomplete, errors occurred!
      error: command '/usr/local/bin/cmake' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for pyarrow
Successfully built numpy
Failed to build pyarrow
ERROR: Could not build wheels for pyarrow, which is required to install pyproject.toml-based projects

To Reproduce
👆
Expected behavior
Additional context

does arrow support python 3.11 yet?
seems to me no https://pypi.org/project/arrow/

@Jimexist The problem is with pyarrow. Installing the latest pyarrow==11.0.0 works, but installing pyarrow==8.0.0 in requirements-310.txt fails.
Maybe we should generate new dependency files for python 3.11.


 I think I'll reopen this issue, as there are more people interested.

This is the intended behavior of overloadt. From the man page:

Identical to overload, but only activates the layer if the bound key  is  held  for   milliseconds.


Type is not callable error using SQLAlchemy's declarative_base

Environment data

Language Server version: 2020.10.1
OS and version: Windows 10, 2004
Python version (& distribution if applicable, e.g. Anaconda): 3.8.3

Expected behaviour
No errors.
Actual behaviour
Two errors appear:
"User" has type "Type[User]" and is not callable
  Type "Type[User]" is not callable Pylance (reportGeneralTypeIssues)
Expected no arguments to "User" constructor Pylance (reportGeneralTypeIssues)

Code Snippet / Additional information
from sqlalchemy import Column, Integer, String
from sqlalchemy.ext.declarative import declarative_base

Base = declarative_base()

class User(Base):
    id = Column(Integer, primary_key=True, index=True)
    username = Column(String(255), unique=True, index=True)

user = User(id=1, username='test')

SQLAlchemy's declarative_base should use this _declarative_constructor by default, which allows initialization from kwargs.
There are no errors for the same snippet using standalone Pyright (1.1.80) without Pylance.

Thanks for the bug report. SqlAlchemy is using a custom metaclass that provides non-standard behaviors for a constructor. I've changed the logic in pylance to avoid emitting this particular error when a class is instantiated from a custom metaclass, so you won't see this false positive in the future.
This fix will be in the next release of pylance.

This issue has been fixed in version 2020.10.2, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/master/CHANGELOG.md#2020102-21-october-2020

This issue has been fixed in version 2020.10.3, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/master/CHANGELOG.md#2020103-28-october-2020

I still get the "Expected no arguments" bug on SQLAlchemy model class initialization.
(running Pylance 2020.11.1)

I just re-verified that the code sample above does not result in any errors. If your code differs from the above, please post a simple example that exhibits the problem.

I just re-verified that the code sample above does not result in any errors. If your code differs from the above, please post a simple example that exhibits the problem.

Sorry about the trouble Eric, the errors are gone now. I think my workspace just wasn't pointed at my virtualenv's Python interpreter.


Add support for oblivious DNS

https://blog.cloudflare.com/oblivious-dns/

Yes I've been eyeing this to jump from DNS over TLS to this, instead of going DNS over HTTPS which adds little benefit apart from using port 443. It might take some time to implement as a local DNS relay (as Unbound currently does) but it is definitely in my brain roadmap 😉

Yes I've been eyeing this to jump from DNS over TLS to this, instead of going DNS over HTTPS which adds little benefit apart from using port 443. It might take some time to implement as a local DNS relay (as Unbound currently does) but it is definitely in my brain roadmap 😉

Sounds good. Let me know if you need help testing :)

Sounds good. Let me know if you need help testing :)

Still a bit behind oblivious DNS, but I made good progress on a Go implementation for a DNS over HTTPs and DNS over TLS proxying server https://github.com/qdm12/dns/pull/58  it should be finalized in the coming week.
Oblivious dns would be next on the list! 😉

Note to future self: https://github.com/cloudflare/odoh-client-go should be a good starting point


feat(licensing): add maxRevenueShare parameter to align with core protocol changes

Description
This PR adds the maxRevenueShare parameter to functions and tests related to derivative registration and license token minting, aligning with changes introduced in Core Protocol PR#364.
The maxRevenueShare parameter enables users to define the maximum royalty percentage they are willing to accept when minting license tokens or registering derivatives to parent IPs. This enhancement improves user control and flexibility in licensing agreements.

Do we provide an interface without the new parameters, so that application integrated with 1.2 api won't need to be changed?

The plan is to do the backward compatibility together in a separate PR, we have an issue tracking this: https://github.com/storyprotocol/protocol-periphery-v1/issues/131


No way to AutoRotate and save Exif

As I see Autorotate function can be called only if new ImageFactory(preserveExifData: true). But what if  I what only autorotate, but no lose gps and other Exif stuff.
Why is it  not possible only change orientation fied of jpeg EXIF?

I think you've missed the point of what the method does.
AutoRotate preserves any EXIF defined rotation stored in an image for when you are not preserving EXIF metadata. This ensures that your image does not rotate when the metadata is removed.
Manipulating individual EXIF metadata tags is currently not supported. See issue #78 It's an incredibly complicated issue which is why it's been marked as 'help needed'. If you can provide support on that front then please do.


Add support for @scope/browserslist-config

This PR adds in support for handling @scope/browserslist-config as the package name for a shared configuration.
Currently, it only handles checking for the @scope/browserslist-config-* prefix.

Good idea!
Can you extend README.md for new format (anyway my English is bad 😅)

Thanks! I will release it in next few hours.

Released in 2.5.1


404 error for Free Code Camp Pull Request

Looking forward for reporting a security issue:
Please report security issues by sending an email to<EMAIL_ADDRESS>instead of raising a GitHub issue.
Describe the bug
This is a broken website page when clicking on the hyper line on "How to Setup and Maintain a Local Instance of Free Code Camp" inside the the page titled "Editing via your Local Fork (Recommended)"
To Reproduce
Steps to reproduce the behavior:
1.Go to  https://guide.freecodecamp.org/miscellaneous/how-to-make-a-pull-request-on-free-code-camp/
2.Find there is a  Editing via your Local Fork (Recommended) section
3.Click on the hyperlink on the text "How to Setup and Maintain a Local Instance of Free Code Camp"
4.Notice the 404 error you are taken to:http://forum.freecodecamp.com/t/how-to-fork-and-maintain-a-local-instance-of-free-code-camp/19116
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@LarryHu0217 Can you tell us how you found that Guide page to begin with?  Was it linked from the contributing guide on GitHub?

I find it on the GitHub contribution page.@RandellDawson

@LarryHu0217 I meant where specifically?  Which document and where in the document is the link to this particular Guide page?

I just search on the freecodecamp website. I don't remember which guideline

Those guides are deprecated.  The only documentation for contributing you need to read through starts here.

Thanks for reporting this.
We are in the process of improving all the guide articles and migrating them over to /news. (The Guide codebase and its many contributions will remain on GitHub.)
So rather than spending time to fix this specific issue, we will stay focused on finishing this big migration as fast as possible.


Remove advisor low-cardinality keys that aren't actually exposed.

I noticed that gen_ai.operation.name and gen_ai.system are shown as low-cardinality keys for spring.ai.advisor, but neither of those appear in the metrics. Moreover, I confirmed that are aren't included as low-cardinality keys in either DefaultAdvisorObservationConvention or AdvisorObservationDocumentation. So this PR removes them from the documentation.

@jonatan-ivanov can you review this please?

gen_ai.operation.name is there as an enum:
https://github.com/spring-projects/spring-ai/blob/efc392394ccc890b7b5b93e7207c52e3d6cc6dd0/spring-ai-core/src/main/java/org/springframework/ai/observation/conventions/AiObservationAttributes.java#L38
But it is called AI_OPERATION_TYPE. Type vs. name puzzles me a bit (it seems it is called type everywhere except that value in the enum). It also seems to be used in certain cases, there are even integration tests for it. I would follow through the usage since it seems DefaultChatClientObservationConvention uses it (with other conventions) and if the keyvalue is added metrics should also have this tag: https://github.com/spring-projects/spring-ai/blob/efc392394ccc890b7b5b93e7207c52e3d6cc6dd0/spring-ai-core/src/main/java/org/springframework/ai/chat/client/observation/DefaultChatClientObservationConvention.java#L63-L71
It seems spring.ai.advisor is not a tag on the metric but the name of it:
https://github.com/spring-projects/spring-ai/blob/efc392394ccc890b7b5b93e7207c52e3d6cc6dd0/spring-ai-core/src/main/java/org/springframework/ai/chat/client/advisor/observation/DefaultAdvisorObservationConvention.java#L34-L40
Though metrics are only created when you trigger the operation that records the observation, can it happen that your use-case did not trigger it?

It's been awhile since I created this and thought about it, but..
It's not that I don't see the spring.ai.advisor metric. I do see it when stuff flows through an advisor. It's just that I never see those two keys. For example, I just ran a request through ChatClient in a situation where there are a few advisors in play and here's what I got back from spring.ai.advisor:

  "name": "spring.ai.advisor",
  "baseUnit": "seconds",
  "measurements": [
    {
      "statistic": "COUNT",
      "value": 3.0
    },
    {
      "statistic": "TOTAL_TIME",
      "value": 7.119742541999999
    },
    {
      "statistic": "MAX",
      "value": 4.183466542
    }
  ],
  "availableTags": [
    {
      "tag": "spring.ai.advisor.type",
      "values": [
        "AROUND"
      ]
    },
    {
      "tag": "spring.ai.kind",
      "values": [
        "advisor"
      ]
    },
    {
      "tag": "spring.ai.advisor.name",
      "values": [
        "CallAroundAdvisor",
        "VectorStoreChatMemoryAdvisor",
        "QuestionAnswerAdvisor"
      ]
    },
    {
      "tag": "error",
      "values": [
        "none"
      ]
    }
  ]
}

Neither of the two keys I mentioned are in this response. Since they are documented as low-cardinality keys, I'd expect them to be there. And yes, I see from the code you showed where aiOperationType() should've put the gen_ai.operation.name key in there, so I'm confused why I'm not seeing it. And I see nothing to suggest that the gen_ai.system should've been included.
Also, this most recent attempt was done moments ago using a local build of the very latest snapshot code (to take advantage of the fix for #1738 / #1742 ).


Issue with proptypes on prod

Hi! I am having an error while running on prod. PropTypes.js:21 Uncaught TypeError: Cannot read property 'isRequired' of undefined.
I am using my boilerplate similar to https://github.com/erikras/react-redux-universal-hot-example.
The build worked nicely with a difference of 100kb between react and preact :D. I  think you are doing a really good job with that an the compatibility.

Ah yes, PropTypes are removed in production mode, but maybe that's not ideal since references to them would still exist. I will swap them to instead be empty functions.

Great thanks!

@anaibol Looked into this, and I realized I was wrong - PropTypes are still exported as normal in production, so I don't think that was the issue.
However, looking at the boilerplate you linked, they use two separate Webpack configs for development and production - is it possible the aliases for react and react-dom are missing or overwritten in your production config?

Mmm the issue is caused by react-router which use react proptypes, and proptypes is an empty object.
The issue is present both in dev and prod environnements.

Ah, sorry I had thought it was only on prod. Can you grab me the output of npm ls --depth 0? Curious what version of proptypes that's coming from.

I think proptypes is not beeing intstalled with preact-compat.
npm ls --depth 0
<EMAIL_ADDRESS>/home/anaibol/Workspace/fr
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>extraneous
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>Installing proptypes doesn't fix that.

It's like present on webpack build time but not after that

Which version of npm are you using?

3.8.3

Sorry, proptypes is installed correctly with preact-compat but does not appear in "npm ls --depth 0"

I wonder if it's possible this is an issue due to proptypes being a Peer Dependency...

It's also a normal dependency on package.json
"peerDependencies": {
"preact": "^4.1.2",
"proptypes": "^0.14.3"
},
"optionalDependencies": {
"preact": "^4.1.2",
"proptypes": "^0.14.3"
},
"dependencies": {
"preact-render-to-string": "^2.0.1",
"preact-svg": "^1.4.3",
"proptypes": "^0.14.3"
}

Just to make sure - you're using Webpack 1.x, right?

Mm nope, webpack v2.1.0-beta.4

"The PropTypes.js in your first screenshot doesn't appear to be the one preact-compat is expecting:
https://github.com/developit/proptypes/blob/master/src/index.js"
Thats the PropTypes file of react-router that uses internally react proptypes.

Ahh, that's probably the reason. Webpack 2 has a bug with module exports. I reported it a few weeks back:
https://github.com/webpack/webpack/issues/2162
I might be able to fix this in the PropTypes module in the meantime though, copying things onto exports instead of using module.exports=.

I see.. By the way, trying to see how it works with the default boilerplate (webpack v1.*), it build's correctly but running the server ends with "Error: Cannot find module 'react'". It's an universal/isomorphic boilerplate


Code workflow

 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.:x: polampalli2 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Use a multiprocessing.Pool to manage compilation+benchmark processes

This gives a 33% speedup on my machine when running with 35 cores. Also
simplifies the code.

I can confirm the speedup!


Update cloudknox-howto-create-rule.md

Minor updates to make the doc consistent to the current UX.

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


add metrics

Waiting on approval on mcstats to test and configure the additional graphs

closing due to excessive time waiting on approval


[Camera] Add camera control features similar to TCameraComponent in Delphi

For example:
Ability to list supported resolutions
Flash control
AutoFocus (and Continuous?)
etc

Refer now to specific enhancement tasks


Installation instructions not working for CUDA 11.4

This is with python 3.9.6 and pip 21.2.4.
On the README it says
pip install --upgrade

Easy enough. Then:
pip install --upgrade "jax[cuda111]" -f https://storage.googleapis.com/jax-release.html

This ran into an error because of the jaxlib dependency:
Looking in links: https://storage.googleapis.com/jax-release.html
Collecting jax[cuda111]
  Using cached jax-0.2.19-py3-none-any.whl
Requirement already satisfied: opt-einsum in /home/eben/anaconda3/envs/refgrads/lib/python3.9/site-packages (from jax[cuda111]) (3.3.0)
Requirement already satisfied: numpy>=1.18 in /home/eben/anaconda3/envs/refgrads/lib/python3.9/site-packages (from jax[cuda111]) (1.20.3)
Requirement already satisfied: absl-py in /home/eben/anaconda3/envs/refgrads/lib/python3.9/site-packages (from jax[cuda111]) (0.13.0)
  Using cached jax-0.2.18.tar.gz (708 kB)
  Using cached jax-0.2.17.tar.gz (693 kB)
  Using cached jax-0.2.16.tar.gz (680 kB)
  Using cached jax-0.2.15.tar.gz (681 kB)
ERROR: Packages installed from PyPI cannot depend on packages which are not also hosted on PyPI.
jax depends on jaxlib@ https://storage.googleapis.com/jax-releases/cuda111/jaxlib-0.1.67+cuda111-cp39-none-manylinux2010_x86_64.whl

I tried pip install jaxlib and then tried re-installing with the same command. That did not work, so I uninstalled jaxlib and then ran pip install jaxlib==0.1.67, but trying to installing jax itself still yields the above error message.

Thanks for pointing out that issue.
I'm currently trying to get this to work on a cluster running Debian 10 buster. I will try to figure out where the outdated package is.

Alright, first I found that jax had actually been installed in my home directory without conda and with python 2, so I removed that because I have no need for it. Then I ran rm -r ~/.cache/pip. Now pip install --upgrade "jax[cuda111]" -f https://storage.googleapis.com/jax-release.html doesn't try to use the cached version, however:
Looking in links: https://storage.googleapis.com/jax-release.html
Collecting jax[cuda111]
  Downloading jax-0.2.19.tar.gz (728 kB)
     |████████████████████████████████| 728 kB 12.2 MB/s
Requirement already satisfied: numpy>=1.18 in /home/eben/anaconda3/envs/refgrads/lib/python3.9/site-packages (from jax[cuda111]) (1.20.3)
Requirement already satisfied: absl-py in /home/eben/anaconda3/envs/refgrads/lib/python3.9/site-packages (from jax[cuda111]) (0.13.0)
Requirement already satisfied: opt_einsum in /home/eben/anaconda3/envs/refgrads/lib/python3.9/site-packages (from jax[cuda111]) (3.3.0)
  Downloading jax-0.2.18.tar.gz (708 kB)
     |████████████████████████████████| 708 kB 82.6 MB/s
  Downloading jax-0.2.17.tar.gz (693 kB)
     |████████████████████████████████| 693 kB 80.9 MB/s
  Downloading jax-0.2.16.tar.gz (680 kB)
     |████████████████████████████████| 680 kB 86.1 MB/s
  Downloading jax-0.2.15.tar.gz (681 kB)
     |████████████████████████████████| 681 kB 81.1 MB/s
ERROR: Packages installed from PyPI cannot depend on packages which are not also hosted on PyPI.
jax depends on jaxlib@ https://storage.googleapis.com/jax-releases/cuda111/jaxlib-0.1.67+cuda111-cp39-none-manylinux2010_x86_64.whl

It looks like the installation command is actually installing jax 0.2.15 on its own? That seems like it would cause an issue.

Hm, that's a bit strange, but I suspect it has to do with pip's new(ish) dependency resolver trying to satisfy version requirements for other packages you have installed. Can you try pinning the JAX version? i.e.
pip install "jax[cuda111]==0.2.19" -f https://storage.googleapis.com/jax-release.html

Great! The install works now and the GPU is utilized. Thanks for the help!

Great, glad it's working! Sorry I didn't notice that URL typo earlier.


Smooth Tracking and Maximizing Marker Distance

I am trying to use AR.js to smoothly load 3D models and animations on a large-scale 9-foot by 9-foot marker from a far distance. Are there any other attributes I can call in my code or suggestions to modify the AR.js repository to make the objects I load onto the marker more stationary and less jittery?
I went through and tested all of the attributes here but the only ones that have helped so far is changing from a-marker-camera to a-marker and adding smooth=’true’. I have attached the links to my source code at the bottom along with a video of how my AR.js scene behaves now.
Also, one more thing. I am trying to get as far away from the marker as possible, while still being able to smoothly load the objects in the scene. Do you have any suggestions on how I can maximize the distance away from the marker? I have thought about using TensorFlow Image Classification, but I need some guidance on how I can implement this in the AR.js repository.
Thank you!
Source Code
Link to Video

I recently build a lib for location based ar, using GPS and Orientation Sensor. I don't know if it's exactly what you are looking for, but maybe it can help you.
See: https://github.com/0b01001101/locationar
Examples: https://0b01001101.github.io/locationar/

@0b01001101 Thanks, but I couldn't get any of your examples working on my iPhone's Safari or Chrome. This is a great idea though.

I changed the code a little bit and added some info to the readme about how to use the examples

Use Chrome
goto the Compass example and see if it's working [camera is working, north in the right direction]
goto the Location example and you should see a red cube 10 m in the direction of North
If you don't see the cube or want to change the position click "InfoPanel" and then you can press the button GetPosition to set the position of the cube to your current position and offset it with the +10N, +10W,... buttons Here you can also see the raw sensor data


[Functional-test] Add chrome browser functional tests

Description
This pull request creates the browser-functional-tests for managing the test to check browser compatibility on the botbuilder-js repository.
The tests consist of running a browser-echo bot using Nightwatch  to test the bot behavior.
Specific Changes

Add browser-functional-tests command to the botbuilder-js libraries
Add the browser echo-bot sample
Setup Nightwatch to run with Chrome
Add a script to run the tests into the root package.json

Run the test locally
Note: The tests are configured to run in two-terminals, one for the webpack-dev-server which runs the bot, another for running the tests.

Go to the .env file and set the TestURI variable with http://localhost:8080/ value

Go to libraries/browser-functional-tests/browser-echo-bot, open a terminal and run npm run start
This command will start the browser bot using webpack-dev-server.

In the botbuilder-js root directory, open a new terminal and run npm run browser-functional-test

Azure Build Pipeline Configuration

Define the following environment variables: TestAppServicePlan, TestResourceGroup, TestWebApp, TestURI.

Add the next task to install the dependencies and build de project.

Node.js Tool installer (Node version 10.x)
npm install - npm build  (npmcustom command)

Working folder: libraries/browser-functional-tests/browser-echo-bot

Add a PowerShell task to compress the result of the build.

The script is:
	# Compress Bot Source Code

	cd $(System.DefaultWorkingDirectory)/libraries/browser-functional-tests/browser-echo-bot/dist
	$DirToCompress = "$(System.DefaultWorkingDirectory)/libraries/browser-functional-tests/browser-echo-bot/dist"
	$files = Get-ChildItem -Path $DirToCompress
	$ZipFileResult="$(System.DefaultWorkingDirectory)/libraries/browser-functional-tests/browser-echo-bot/browser-echo-bot.zip"
	Compress-Archive -Path $files -DestinationPath $ZipFileResult

Configure an Azure CLI task to deploy the bot to azure.

The script is:
	echo "# Create resource group"
	call az group create -l westus -n "$(TestResourceGroup)"

	echo "# Create app service plan"
	call az appservice plan create -g "$(TestResourceGroup)" -n "$(TestAppServicePlan)" --number-of-workers 4 --sku S1

	echo "# Create web app"
	call az webapp create -g "$(TestResourceGroup)" -p "$(TestAppServicePlan)" -n "$(TestWebApp)"

	echo "# Deploy source code"
        call az webapp deployment source config-zip --resource-group "$(TestResourceGroup)" --name "$(TestWebApp)" --src "$(System.DefaultWorkingDirectory)/libraries/browser-functional tests/browser-echo-bot/browser-echo-bot.zip"

Add an npm task to run the tests

Working folder: libraries/browser-functional-tests

After the tests finish running, add a new Azure CLI Task to delete the resource group we've created.

The script is:
`call az group delete -n "$(TestBotName)" --yes`

We strongly recommend setting this task to run even if a previous task has failed or if the build was canceled. With this setting, we will ensure that the resources are deleted from Azure despite the build failing at any step.

Pull Request Test Coverage Report for Build 88767

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.124%

Totals

Change from base Build 88750:
0.0%

Covered Lines:
4917

Relevant Lines:
5536

💛  - Coveralls


Add a prometheus counter over the different kinds of write operations

Can be used to differentiate write traffic further

Updated


Database navigator view enhancements

We need to show connection type in more fancy way. E.g. with filled rectangles/circles (fill color is connection color).
We need to have ability to show some object statistics (e.g. table size) on the right side of navigator tree (configurable).
We need to have ability to show extra info (e.g. target host) next to the connection name, e.g. in shadow color (configurable).

Please add ability to show extra info (e.g. target host) next to the connection name, e.g. in shadow color (configurable).

Done

Table statistics is not displayed for (the ticket is still checking):
SQLite
DB2
Athena
CockrouchDB
Derby
Exasol
Firebird
Snowflake
SQL Server

@kseniiaguzeeva It is ok.
For now we support it only for PostgreSQL (and forks), MySQL, MariaDB, Oracle, Clickhouse.

Great.

@serge-rider : Just a very minor display issue, but the table size statistics use wrong file size unit notation, it is visible in your own screenshot as you posted here: https://github.com/dbeaver/dbeaver/issues/8705#issuecomment-633916230

Gb should be GB
Mb should be MB
Kb should be kB

@serge-rider , Any chance to show also number of rows, not only table size?

Could you please create a new feature request?


app crashes when start meeting

you have to copy all the files from the example provided by the author located in example/android/app/src/main/res/ to your project your_project/android/app/src/main/res/

hello, the image asset button does not appear in android studio, any other solution? Or could you tell me what goes inside the folders?

hy, there no need to find any asset button, you need to copy all the files located in : https://github.com/saibotma/jitsi_meet_wrapper/tree/main/jitsi_meet_wrapper/example/android/app/src/main/res
to your project : your_project/android/app/src/main/res/


Unable to pull ghcr.io/ovrclk/cosmos-omnibus:v0.3.15-osmosis-v13.0.0

docker pull ghcr.io/ovrclk/cosmos-omnibus:v0.3.15-osmosis-v13.0.0
Error response from daemon: manifest unknown

Did the image push step get missed on this release?

Apologies I didn't realise that PR had been merged, I'll publish a new release now

I did find: ghcr.io/ovrclk/cosmos-omnibus:v0.3.16-rc1-osmosis-v13.0.0

Awesome that's fine to use in the meantime, ghcr.io/ovrclk/cosmos-omnibus:v0.3.16-osmosis-v13.0.0 is building now.

Awesome that's fine to use in the meantime, ghcr.io/ovrclk/cosmos-omnibus:v0.3.16-osmosis-v13.0.0 is building now.

Sweet, thanks for the quick response!


V0.0.1 – Cannot perform any actions

I'm unable to perform any actions with any pre-existing service due to the following error:
serverless deploy --stage dev --region us-east-1  Error --------------------------------------------------
Cannot find module 'minimist'  For debugging logs, run again after setting SLS_DEBUG env var.
Get Support --------------------------------------------
Docs: docs.serverless.com Bugs: github.com/serverless/serverless/issues
Please report this error. We think it might be a bug.
Your Environment Information -----------------------------
OS: darwin
Node Version: 6.6.0
Serverless Version: 1.1.0

Other notes:

When I clone the dashboard, npm install, and run it, this is not an issue.  It's only with the packaged application.
We might not be the only ones:

https://github.com/wireapp/wire-desktop/issues/95

@DavidWells Nope, just checked.

@ac360 is this resolved now? can we close the issue?

fixed


Use new find_draft_services_by_framework endpoint on publish-draft-services

https://trello.com/c/kCUjDnH4/48-3-3-jenkins-job-for-publish-services
Uses the new status-filterable find_draft_services_by_framework endpoint, hopefully for efficiency savings 💸. I'd like to run this branch against preview to determine the speed gains (if any).
I've done some pretty hefty refactoring of the test coverage for the publish-draft-services script. Currently there's 2 'mega-tests' with parametrized scenarios - while these are comprehensive and neat, I found it difficult to determine the expected behaviour for each scenario and be confident that any changes I made to the script itself would still have coverage.
Summary of changes:

Adds extra unit tests for the public helper functions publish_draft_service and copy_draft_document, and group them into classes
Convert parametrized scenarios for publish_draft_services into separate unit tests
Minor tweaks to the publish_draft_service function to make it clearer what's being returned
Rename _get_draft_services_iter to a public (testable) function and add tests
Finally, replacing the get_submitted_drafts function (which will iterate through the JSON for ~40,000 services to determine each one's status, yikes) with the new find_draft_services_by_framework generator, which pre-filters the status, getting Postgres to do the heavy lifting.

Comments very welcome, especially if I've missed anything on the test coverage. "I liked the old test structure better because of X" is also a valid comment!

@lfdebrux Python 3.7+ errors now addressed. I'm going to try and run this against some 'real' (developer) buckets today.


Automatic Learning

Rasa NLU version : 0.12.0a2
Used backend / pipeline :spacy_sklearn
Operating system :windows 10
Issue: Not learning automatically, like if someone had a chat with the bot we expect the bot to train automatically from the conversation it had with the user but i don't think it's happening or if it is happening how can achieve it ?
Right now i am making a note of all the situations where it fails and appending some more data in data.md manually after the conversation and building some more stories to overcome those errors.

Are you talking about interactive learning? because the stories get automatically generated the way you corrected them and then you can add this to your training data and re-train your core model. It doesn't learn "automatically" as soon as you've corrected it.

@akelad No, I am not talking about interactive learning what I mean is the bot should be intelligent enough to learn on its own(from the user) without human(developer) intervention after sometime

That would be reinforcement learning, which we don't currently use.

@akelad  Yeah right now I am using interactive learning only but still, I need to  build the model, run the bot every time if I update anything also think of a scenario when I am running the bot in server serving multiple user requests, at that time it should learn automatically from conversations generated by every user and update itself rather than developer updating it daily based on the logs generated.
Does rasa platform contain Automatic learning I mean reinforcement learning?

No, the platform doesn't contain reinforcement learning. Reinforcement learning for conversational AI is still in the early stages of research. And it's not necessarily always a good thing - talking to real users and learning from them might also mean that the bot starts learning something completely different than you want it to.
I'm just wondering what's your use case for the bot, and how many stories do you have so far? Because at some point when you have enough stories it should be able to generalize to some unseen conversations at least.

stories are fine but I want to give the query entered by the user to the bot as the training data so that it can perform very well in intent classification and entity extraction.

Is this a Rasa Core request or Rasa NLU?

I'd say NLU. What you're requesting @Harish0596 is something the Platform does actually, as in all the unseen messages from your users get sent to an inbox where you can then add them to the training data or correct the labels. You can learn more about this here http://rasa.com/products/rasa-platform/ and if you're interested please contact our sales department

@akelad sure thanks

@Harish0596 was just wondering if this feature is now available after 2 years !
If yes can someone link up the documentation ?
PS - I am just a newbie with RASA

Annotating new user messages is something you can do with Rasa X now - https://rasa.com/docs/rasa-x/


WIP: MEG example

This PR is a fork of https://github.com/ja-che/hidimstat/pull/24
Let's first try to fix GitHub Actions.

@ja-che can you allow running the CIs?

Sûre

Thanks!

Mayb we need xvfb, waiting for the result of the last run:
https://github.com/ja-che/hidimstat/pull/25/checks?check_run_id=2913208917#step:4:975

But locally, it works for me too :+1:

TL;DR: I reproduced https://github.com/mne-tools/mne-nirs/pull/297 so I'll switch to circle

I added some required dependencies and configured the CIs for basic sphinx usage and it should be reviewed.
In the end, I think the ammaraskar/sphinx-action@master action does not seem compatible with xvfb or I did not manage to configure it properly.
The artefacts are available:
https://github.com/ja-che/hidimstat/suites/3085413492/artifacts/70429606

Apparently the plot hangs on Circle, any idea?

@ja-che
We were so close!
Looks at this atefact:
https://8-259908992-gh.circle-artifacts.com/0/dev/auto_examples/plot_meg_data_example.html#sphx-glr-auto-examples-plot-meg-data-example-py

It has been more than 20 minutes now so I think the plots are definitely hanging for some reason. Why only on CircleCI? I can't explain it.
I'll push another commit to remove CircleCI YAML configuration file so it can be disabled for the project if you don't need it.
@ja-che, you can link https://github.com/mne-tools/mne-nirs/pull/297 to follow the progress on "MNE/PyVista figures on Sphinx/GithubActions". I intend to go back on it at some point to solve this mysterious issue.

@ja-che ready to go when CIs return happy :+1:
Feel free to disable CircleCI on the project now.

can you share a link to the artifact when you have one?

They're available:
https://github.com/ja-che/hidimstat/suites/3100958266/artifacts/70883554

plot_meg_data_example.html does not show a brain

@agramfort , for short, the brain is showed on circle but the other examples do not work. on github actions everything is showed except the brain


chore: add vscode custom data / autocomplete

Related #388
Adds a vscode.html-data.json file generated from the custom-elements.json.
https://github.com/microsoft/vscode-custom-data

VS Code ships with rich language feature support for HTML/CSS, such as auto-completion and hover information. The core of these language support are implemented in vscode-html-languageservice and vscode-css-languageservice. In the past, these libraries were coupled to outdated schemas that define HTML/CSS entities. Custom data decouples these libraries from the data they use and allows VS Code to offer up-to-date support for latest HTML/CSS proposals or frameworks built on top of HTML/CSS.

requires this in your .vscode/settings.json:
{
  "html.customData": [
    "./node_modules/media-chrome/dist/vscode.html-data.json"
  ],
}

for Media Chrome devs, probably
{
  "html.customData": [
    "./dist/vscode.html-data.json",
    "./node_modules/media-chrome/dist/vscode.html-data.json"
  ],
}

Thoughts about adding a codemod to add that required snippet?

@davekiss good question yes, it's kind of a bummer it would not be picked up automatically.
yes worth exploring more, maybe a codemod but it would require a CLI command right?


Path parameter value not appended in url

Triggering a following post request :-
Response resp = RestAssured.given().pathParam("add",build).
formParam("file", "https://unsplash.com/photos/Bcv4wZSMtIA").// Cast
formParam("type", "front").
formParam("auto_start", false).
when().post("https://private-538b1b-verificationsrv.apiary-mock.com/{add}");
where build = abc/xyz
The response thrown here by above script is 404 whereas when I used the actual value of add within post as
when().post("https://private-538b1b-verificationsrv.apiary-mock.com/abc/xyz");
Then it works fine.
I think this is a bug or am I doing something wrong here ?

I added log().all() to see what is sending in the request uri and it seems value within add is being encoded , is there any alternative using which this encoding can be stopped and text is sent as it is.

used urlEncodingEnabled(false) and this resolved my issue.


Supports multiple selection of line, rect, polygon tools

:tada: This PR is included in version 1.10.0-alpha.12 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.14.0-alpha.12 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.7.0-alpha.9 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


placeholder also works as a constructor param

Proposal: searchLabel should be = placeholder, if not specified:
this.searchLabel = opts.searchLabel || opts.placeholder || "Type a value";


Update account settings & Google Drive flow

Checklist

[x] All new and existing tests pass
[x] I've read the Contribution Guidelines
[x] I've read Change Limitations

Motivation and Context
Addressing Jira tickets:

ASJ-299
ASJ-302
ASJ-303
ASJ-304

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: sfal:x: mateusz-ramski:x: Mateusz RamskiMateusz Ramski seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update br-disk.json

Fix false positive
Pull request
Purpose
fix false positive SeriesName.S01E01.2160p.NF.WEB-DL.DDP5.1.HDR.DV.HEVC-DBBd
Approach
Only match BD as a full word not partially
Open Questions and Pre-Merge TODOs
Check all boxes as they are completed

[ ] Use github checklists. When solved, check the box and explain the answer.

Learning
If you're adding a new Custom Format make sure you follow the Radarr/Sonarr Custom Format (JSON) Guidelines.
Requirements
Check all boxes as they are completed

[ ] These changes meet the standards for contributing.
[ ] I have read the code of conduct.

Do you still have a regex match example you use to see if you match the correct ones etc?

Darn I forgot we have this also for Sonarr v4 now.
Need to add it there tomorrow

Do you still have a regex match example you use to see if you match the correct ones etc?

@TRaSH- I unfortunately dont have it right now but I'm quite confident that change is not gonna affect anything else. I just was abit lazy with the word boundaries when I initially wrote it.

Darn I forgot we have this also for Sonarr v4 now. Need to add it there tomorrow


Add relationships between Ansible job and its playbook

Migration is in already. This is to add relationships in model classes
Per our offline discussion the relationship is added to the child class level.

@miq-bot add_label providers/ansible_tower
@miq-bot add_label enhancement
@miq-bot assign @gmcculloug

cc @h-kataria


Forge port!

Make me!

please =)

Neoforge and that's my last offer

=(

:)

I'm keeping this open as I plan to port the mod to all mod loaders as it's a simple mod

Please notify me if you do anything, any ports on Forge :3

It's now available for 1.21.1 Neoforge


How do I install this file in my web browser?

Do I need Greesemonkey or something? How can I install this JavaScript in Google Chrome for Windows or Firefox?

Yes, Greesemonkey or Tampermonkey is required


Lossless streaming not working since yesterday

The option HIGH in mopidy.conf is working for me.
I found a link for this issue in kodi forum: http://forum.kodi.tv/showthread.php?tid=200555&pid=2415125#pid2415125
But the described solution is not working for me. Can't get LOSSLESS working anymore.

Same for me, log here:
http://www.hastebin.com/osuqizafix.log

Most of the FLAC streams are now crypted :(
I don't know if there's anything we can do for now.

Maybe I found a workaround, please give a try updating tidalapi4mopidy  to 0.1.3.
Let me know....


Update proc-macro to fix build

It looks like cargo build is not happy with proc-macro2 1.0.56 any more.
error[E0422]: cannot find struct, variant or union type `LineColumn` in crate `proc_macro`
   --> /home/sol/.cargo/registry/src/index.crates.io-6f17d22bba15001f/proc-macro2-1.0.56/src/wrapper.rs:475:33
    |
475 |                 let proc_macro::LineColumn { line, column } = s.start();
    |                                 ^^^^^^^^^^ not found in `proc_macro`
    |
help: consider importing this struct through its public re-export

Update to proc-macro2 1.0.78 to fix the build.

Strange, builds fine for me even without this PR. But if bumping proc-macro helps 4u, that's good enough for me. Thanks for the fix


[forwardMessages] You cannot see the message that is currently requested to be forwarded, but the message that was requested to be forwarded last time.

When a message is forwarded from one channel to another channel, why the response is normal, but the forwarded message cannot be seen, or is the message ID that I forwarded last time being forwarded?
After I use this method again, do I need to call any other method to make the forwarding take effect? At present, I call getChats once after forwarding to see the latest forwarded message. Is this operation as expected?
versions v1.8.0
golang: https://github.com/zelenin/go-tdlib
        _, err = tdClient.GetChats(&client.GetChatsRequest{
		ChatList: &client.ChatListMain{},
		Limit:    10,
	})

	if err != nil {
		log.Fatalf("GetChats error: %s", err)
	}
        var fromMessageId int64
	_, err := tdClient.ClearAllDraftMessages(&client.ClearAllDraftMessagesRequest{})
	if err != nil {
		t.Fatal(err)
	}
	var messageIds MessageIdSort
	for i := 0; i < 10; i++ {
		res, err := tdClient.GetChatHistory(&client.GetChatHistoryRequest{
			ChatId:        -10015*****540,
			Limit:         10,
			FromMessageId: fromMessageId,
			Offset:        0,
			// OnlyLocal:     true,
		})
		if err != nil {
			t.Fatal(err)
		}

		if len(res.Messages) == 0 {
			break
		}
		for _, msg := range res.Messages {
			messageIds = append(messageIds, msg.Id)
		}

		fromMessageId = res.Messages[len(res.Messages)-1].Id
		resByte, _ := json.Marshal(res)
		log.Printf("%v: GetChats GetMessages: %s\n", i, resByte)
	}

	sort.Sort(messageIds)
	fmt.Println(messageIds)
	res1, err := tdClient.ForwardMessages(&client.ForwardMessagesRequest{
		ChatId:        -10017*****997,
		FromChatId:    -10015*****540,
		MessageIds:    messageIds,
		Options:       nil,
		SendCopy:      false,
		RemoveCaption: false,
		OnlyPreview:   false,
	})
	if err != nil {
		t.Fatal(err)
	}
	resByte, _ := json.Marshal(res1)
	log.Printf("GetChats ForwardMessages: %s\n", resByte)

	_, err = tdClient.GetChats(&client.GetChatsRequest{
		ChatList: &client.ChatListMain{},
		Limit:    10,
	})

	if err != nil {
		log.Fatalf("GetChats error: %s", err)
	}

This problem has troubled me for several days. What is the correct way to write it?

than you