neo4j shouldn't run as root

if neo4j has any exploits this would allow the container to be exploited, including updating neo4js binary, or host other malign software within the container. This also makes it easier to attempt to break out of the jail which has been accomplished in the past.
Running as root violates best practices.
https://docs.docker.com/engine/userguide/eng-image/dockerfile_best-practices/#/user

@xenoterracide You are absolutely correct. We are uncomfortable with running Neo4j as root in the container; however we haven't yet come up with a way of doing this that we're happy with. There is a partial implementation of this in #22. The problem is that Neo4j requires an elevated value of ulimit nofiles, but that is hard to achieve in a Docker image when not running as root.
We hear your feedback and we'll raise the priority of this work.

as more people move to CoreOS, Atomic, Kubernetes, Openshift, etc. this is going to become more pressing.

We have currently the same problem. By policy our company won't allow as to have root access so we have trouble to use Neo4j in our OpenShift platform. Any information when this issue will be solved? Or is there an option to use Neo4j in a container without root?

Any updates here? I need to run Neo4j in OpenShift and the current image doesn't allow this. I've looked at the existing PRs and workarounds, but they don't seem to work.

https://docs.docker.com/engine/reference/commandline/run/#set-ulimits-in-container-ulimit
Docker run (now?) can give appropriate amounts of ulimit nofiles without root. Can you revalidate your points on this?

@hpehl @xenoterracide @dmlond @nikolastankovic We've released a change here, so within our Docker images Neo4j now runs under the user neo4j.

Thanks for the update @kfreytag


Bugfix: HtmlNode.PrependChildren() Reverses Order of Source HtmlNodeCollection.

This addresses Issue #278, HtmlNode.PrependChildren() Reverses Order of Source HtmlNodeCollection.
Changes:

Fixed issue at hand (#278)
Standardized Whitespace (converted tabs to spaces for the sake of consistency.

Hello @nbcarey ,
Thank you for the pull, we will look at it.
Best Regards,
Jonathan

Hello @nbcarey ,
Sorry for the long waiting. We had some pending changes on our side about the project structure we had to complete.
We cannot directly merge the code but the code you made for the issue #278 has been manually merged.
We will eventually perform a big cleanup with Resharper for space/tab in the project.
Let me know if everything now work as expected.
Best Regards,
Jonathan

Will do. Thanks!


LFS-209: Linking between forms and subjects

Jira link
To test: Create a new subject from the user dashboard. It should allow you to select subjects. In addition, clicking the subject's name in a Form will allow you to change who that subject refers to. It should no longer be possible to create Forms with no subject.

Can you rename the commit messages please?

Testing instance is running on http://<IP_ADDRESS>:8000/
It looks much better! Here are some problems that I observed:

subject selection dialog always shows 1-1 of 1

when I create a new subject, the subject list does not get refreshed
I can create multiple subject with the same name
I can create multiple forms of the same type by using the same subject

I have found a bug similar to what @sashaandjic described as "when I create a new subject, the subject list does not get refreshed". Here are the steps to reproduce it:

Checkout the LFS-209 branch (git checkout LFS-209)
Build it (mvn clean install)
Start it (java -jar distribution/target/lfs-0.1-SNAPSHOT.jar)
Load http://localhost:8080 and login as admin:admin
You should at this point be seeing an empty "User dashboard"
Create a new "Demographics" form
Click the + icon to create a new subject, name the subject PatientX, click checkmark
Observe that the subject list is empty ("No records to display")
Click the + icon to create a new subject, name the subject PatientX, click checkmark
This time, PatientX is now present in the list
Click PatientX in the list to select it.
Click the Create button
Fill out the demographics form and click the Save button
Now, click Subjects on the pane on the left side of the screen
PatientX is present twice - the older version linking to the newly completed demographics form, the newer version linking to nothing.

I have found another bug, this time regarding the Previous Page and Next Page arrows in the Select a subject menu. Here are the steps to reproduce it:

Checkout the LFS-209 branch (git checkout LFS-209)
Build it (mvn clean install)
Start it (java -jar distribution/target/lfs-0.1-SNAPSHOT.jar)
Load http://localhost:8080 and login as admin:admin
You should at this point be seeing an empty "User dashboard"
Create a new "Demographics" form
Add 10 subjects (patient0, patient1, ... patient9)
The bottom of the Select a subject window shows 1-2 of 2 and clicking the left (Previous Page) or right (Next Page) button does not advance or retract the page. The only way that all 10 entries can be seen is by selecting 10 rows or 20 rows from the drop-down menu.

Strictly speaking, this is not a bug, but I find the following to be different from what user expectations would most likely be:

Checkout the LFS-209 branch (git checkout LFS-209)
Build it (mvn clean install)
Start it (java -jar distribution/target/lfs-0.1-SNAPSHOT.jar)
Load http://localhost:8080 and login as admin:admin
You should at this point be seeing an empty "User dashboard"
Create a new "Demographics" form
Associate this new form with a new subject which we will call PatientZ
Fill out the form and save it.
Return to the User dashboard by clicking User dashboard on the top-left corner of the page.
Open the newly created form by clicking its link in the Demographics Data card
Click the large blue PatientZ title
Associate this form with a new subject, PatientW
Click Subjects in the pane on the left side of the screen
Click the link for PatientW and find that it is empty

In summary, what I suggest is that as soon as the subject associated with a form is changed, said form would be automatically saved, instead of relying on the user to manually click the Save button. Would you agree @marta- ?

I have found another bug, this time regarding the Previous Page and Next Page arrows in the Select a subject menu.

* subject selection dialog always shows 1-1 of 1
* when I create a new subject, the subject list does not get refreshed
* when deleting a subject is it possible to use smaller red font as one circled in blue? can we also include the subject name that the user is about to delete in the question?
* text circled in blue "Please select a subject" should clear up once newly created subject is either saved or canceled.

These issues have now been fixed

I have found a bug similar to what @sashaandjic described as "when I create a new subject, the subject list does not get refreshed". Here are the steps to reproduce it:

This should now be fixed. Subjects are no longer located at their UUID, but instead at /Subjects/<identifier>

@OrdiNeu

I have re-tested the issue with Previous Page and Next Page buttons not working properly in "Select a subject" menus and this has been fixed :heavy_check_mark:

Similarly, the issue with the "Select a subject" menu always displaying 1-1 of 1 has also been fixed :heavy_check_mark:

The issue with the subject list not being refreshed after a new subject has been created has been partially fixed. Sometimes the menu auto-reloads after creating a new subject while other times it does not. Possibly, this is the result of a data race?

The issue of switching to a smaller red font has been fixed :heavy_check_mark:

The "Please select a subject" message no longer appears. It this okay?

With the current workflow this error message is no longer needed

subject list not being refreshed after a new subject has been created has been partially fixed. Sometimes the menu auto-reloads after creating a new subject while other times it does not. Possibly, this is the result of a data race?

Fixed in f3514bc ✔

* The "Please select a subject" message no longer appears. It this okay?
With the current workflow this error message is no longer needed

Fixed in 8f7869b . The message was only supposed to show up if the user was in error, so it not appearing until after they try to go back should be fine. ✔

There is a problem: Say, a form had one subject, and then it was changed to the other subject. Clicking the subject's name in a Form allows you to delete current subject. Clicking on Confirm  button will throw error and won't close the dialog. Clicking away will close dialog. The form would be silently re-assigned to the subject is previously had.

On clicking create button for creating new subject autofocus should be set to the subject input.

Click on Delete trash can button in subject dialog should "unselect" any row previously selected. Also it should disable Confirm button.

Not a show-stopper, but abit counter-intuitive button tooltip for confirmation of a delete action.

Re-tested and issue with Subject list not always being automatically refreshed after a new subject is added has been fixed :heavy_check_mark:

@OrdiNeu
Commit 633abb1 is good as it denies a user from deleting a subject that is currently assigned to the given form, however, it would be nice if an error were to be displayed or the trash can icon could simply be omitted (or grayed out) for the subject identifier currently assigned to the given form.

retested. found the following:

when creating new subject move focus to the subject line
when creating a subject that already exist, it appears as it is allowed - however it does not get saved. user should be warned about the duplicate entry
when attempting to delete a subject that is already in use, display a message teling user why they cannot delete it

Cannot search forms or subjects by subject

Not a show-stopper, but abit counter-intuitive button tooltip for confirmation of a delete action.

I can't seem to edit that one without also editing the tooltip when you save the edits to a row -- any ideas on a wording that works for both cases?

Commit 633abb1 is good as it denies a user from deleting a subject that is currently assigned to the given form, however, it would be nice if an error were to be displayed or the trash can icon could simply be omitted (or grayed out) for the subject identifier currently assigned to the given form (or any other form that uses it).

I could maybe do this, if I disable the entire row (so disable editing the currently selected subject).

Cannot search forms or subjects by subject

This is functionality added in #188

when creating a subject that already exist, it appears as it is allowed - however it does not get saved. user should be warned about the duplicate entry

Fixed in e355f69 ✔

When I tried to save an empty subject I ended up with a blank screen.

When I tried to save an empty subject I ended up with a blank screen.
Fixed in f613048 ✔

problem with zoom above 80%

CREATE bttn should be blue; when creatinh a new form from Forms CANCEL button has BACK as a label;

I would prefer having + to CREATE NEW SUBJECT button

creating duplicate subject isn't allowed, however there should be an error msg informing user that the subject already exist

when changing subject on the form user can select from the list; why wouldn't we allow creation of the new subject

* I would prefer having "**+**" bttn at the top to CREATE NEW SUBJECT button at the bottom

* focus should always be on the CREATE button (this is probably out of scope of this task)

Everything except these two have been addressed in the latest commits.

The form looks OK now. The BACK button instead of CANCEL is still appearing:

I would use this form instead of one below (the only diff between them is missling CANCEL bttn which is a problem)

There is a problem with special characters when creating a new subject:

There is no Cancel button. I suppose this functionality should b presented.

Its impossible to change subject to another existing one

Creating a duplicate subject disables all buttons in the dialog

Looks good enough to be merged. Some very quick ui improvements :

"Create subject" should be labeled "New subject" and should have color="secondary"
"Cancel" and "Back" buttons should have color="default" in all dialogs

"Create subject" should be labeled "New subject" and should have color="secondary"

Should all CREATE bttn labels be replaced with NEW?
@marta-

Should all CREATE bttn labels be replaced with NEW?

I'd say no. While the mismatch may raise consistency concerns, the buttons actually do different things: one pops yet another dialog for entering the new subject, while the other one finishes the process and takes the user to the form.


test/test_blacklist_integrity.py: Simple checks for blacklist files

Fail on empty line, no linefeed, or DOS CRLFs in any of the files.

Er, it fails because it works!


Missing releases/changelogs for new versions

It seems the last version published on GitHub was 1.0.0-rc40 as per https://github.com/evanderkoogh/otel-cf-workers/releases
And the last change changelog entry is for 1.0.0-rc41: https://github.com/evanderkoogh/otel-cf-workers/blob/main/CHANGELOG.md
But the latest version of npm is 1.0.0-rc43. It looks like some of the recent versions were published manually perhaps, which may have broken the automation?

@evanderkoogh Be sure to add changesets for changes and then merge the “Version Packages” PR to bump the package version + publish to NPM. This is the release process we’ve been using, rather than doing these steps manually.
This ensures that the change log gets updated + GitHub releases get generated.

Oh yeah.. I totally missed that.. I noticed that I had forgotten to include the changeset in the PR, so did that on main and then published manually..
What is the "Version Packages" PR?
Also.. you have no idea how much I appreciate you lot looking after all this to the point where I screw things up as soon as I try to get back into things :)

Every time a commit lands on main, the release workflow runs. If there are new changesets, it will open a PR to version packages.
When we merge that PR (also to main), the same release workflow will detect that the current version in package.json has not been published and publish it + create a GitHub release.
If you forget to add a changeset in a Pr/commit, it’s totally fine to push a new commit with a changeset for the change.
This setup means we never have to update package.json or publish to NPM ourselves. It’s all managed by changesets

Ohh.. and I see. I released the version without actually merging the PR. So if I merge the PR, it will open a Version PR and if I merge that, all of the things are updated.
Awesome to know. I was on a call with someone while having them test things out, but I could have used the Github URL for the branch of that anyway.


Add 32 bit binary to release

Here is a proposition to release a 32 bit binary (there are still computers of less than 5 years old limited to 32 bit Windows, e.g. Intel Compute Stick, ZOTAC Pico, ASUS Transformer). This would probably simplify also the installation of this Chocolatey package by removing the Python dependency.
Separate binaries aqt_x64.exe and aqt_x86.exe are generated, see https://github.com/lebarsfa/aqtinstall/releases. I do not know if the continuous release aqt.exe should be in 64 or 32, at the moment it is left to 64 in that code.

Thank you for the proposal and contribution.


can now control widowx arm: /dev/ttyUSB0 and port number 9211.

A mix of recent changes.  This should be more or less stable now.  Could you take a quick look @breakds ?


Remove most of the emojis from the buildkite tests

Most of these emojis I can't remember so I spend longer waiting for the
mouse-overs that I did before when some were too long. :older_man:
Removed a duplicate Amazon Linux TK test
Shortened some text where I thought it made sense too
This is obviously a bikeshed but I really do find the emoji's difficult

🤷


Implementing Archive Pages

I know using jekyll-archives can automatically generate archive pages, but unfortunately, that plugin is still not supported by GitHub Pages!

https://github.com/github/pages-gem/issues/93
https://github.com/github/pages-gem/pull/106
https://github.com/github/pages-gem/pull/643

The feature of archive pages almost being so basal to every blog, is it still outside the scope of Minima? If it is not, I am planning to implement some archive layouts in Minima, without jekyll-archives.

I don't feel like reinventing the wheel on this theme just for the sake of GitHub Pages, now that you can deploy through GitHub Actions, Netlify, Zeit now, Render, etc.


Build with --dump option fails to create intermediate directories

Using stacker 1.1.1, trying to dump a multi-stack config run fails with the following traceback:
[2017-10-24T16:09:40] Using default AWS provider mode
[2017-10-24T16:09:41] Dumping "Create/Update stacks"...
[2017-10-24T16:09:41] Writing stack "cobli-cassandra-sns" -> dump/stack_templates/cobli-cassandra-sns/sns-5d0ce3ca.json
[2017-10-24T16:09:43] Writing stack "cobli-cassandra-asg" -> dump/stack_templates/cobli-cassandra-asg/asg-bd6d7f66.json
Traceback (most recent call last):
  File "deploy/cloudformation/cloudform-venv/bin/stacker", line 9, in <module>
    args.run(args)
  File "deploy/cloudformation/cloudform-venv/local/lib/python2.7/site-packages/stacker/commands/stacker/build.py", line 46, in run
    dump=options.dump)
  File "deploy/cloudformation/cloudform-venv/local/lib/python2.7/site-packages/stacker/actions/base.py", line 131, in execute
    self.run(*args, **kwargs)
  File "deploy/cloudformation/cloudform-venv/local/lib/python2.7/site-packages/stacker/actions/build.py", line 365, in run
    provider=self.provider)
  File "deploy/cloudformation/cloudform-venv/local/lib/python2.7/site-packages/stacker/plan.py", line 359, in dump
    with open(path, "w") as f:
IOError: [Errno 2] No such file or directory: u'dump/stack_templates/cobli-cassandra-asg/asg-bd6d7f66.json'

Adding some debugging into /usr/local/lib/python2.7/site-packages/stacker/plan.py:
[2017-10-26T15:41:54] directory: ./dump
[2017-10-26T15:41:54] filename: stack_templates/vpc-dev-01-vpc/vpc-a968b133.json
It does the os.makedirs on 'directory', then open on directory/filename


Slack Integration with IBM Watson Assistant won't accept new line in user queries.

I'm getting problems whenever I have a new line in my messages to my slack bot.

How do you send it to Watson?
New line should be replaced by this code: https://github.com/watson-developer-cloud/botkit-middleware/blob/master/src/index.ts#L148
This issue was fixed by https://github.com/watson-developer-cloud/botkit-middleware/pull/117 in 1.6.2 (4 years ago), is it possible that you are using an older version?


Spark 2.3 / Solr 4.10

Since you have dependencies shaded, do you think lucidworks/spark-solr will work out of the box
if we build it with

Spark 2.3 and Solr 4.10
Spark 2.4 and Solr 7.4

Or there are some expected issues?
Thank you!

Solr 4.10 is really really old. Solr API's have changed significantly and I don't think it will work
Spark 2.4 and Solr 7.4 should work already with version 3.5.19


Do not use isinstance to select array module

Generally speaking, isinstance should not be used if possible because it is slow in some case. This PR avoid to use it to determine an array module used.

Just a comment. We already use isinstance via get_array_module in the same function. So we still cannot completely remove isinstance with this PR.

LGTM


chore: release 0.4.1

:robot: I have created a release *beep* *boop*
0.4.1 (2022-03-03)
Bug Fixes

client: autoclean ignores releases downloaded by previous trdl versions (6e7b770)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/trdl/releases/tag/v0.4.1 :sunflower:


Issue #75 : Code refactoring

closes #75
Done with most of the warnings and the formatting issues.
Only thing currently left is managing environment variables from a separate file.
Should be done soon!

I will make a separate PR for the environment variables issue. I am studying on how to manage dependencies in go rn. This can become a part of making a sophisticated file structure for the project as a whole.


New handlers for FetchService

This provides support I need for being able to catch 401s, in non-SSO based scenario, and respond appropriately by bringing up.
The handler for the successful response seemed like a good thing to add at the same time for symmetry, and we have seen a couple of places where it might have been useful, and avoided the need for  overriding FetchService to look for e.g. 200 returns with success: false
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[X] Caught up with develop branch as of last change.
[ ] Added CHANGELOG entry, or determined not required.
[X ] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[X] Updated doc comments / prop-types, or determined not required.
[X] Reviewed and tested on Mobile, or determined not required.
[ ] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

@ghsolomon -- thanks again for taking a look, and your comments -- refactoring to try to take account of your comments, I realized that I have a problem with the onResolved side.   In particular, for some requests, we will be resolving to a processed any and for others to a FetchResponse

ok @ghsolomon -- can you take another look?  Simplified this to deal with just the exception case.
Also -- made some orthogonal refactors/cleanups while working through this code that I think we should take.

Looks good to me.  One other thought - since we're supporting multiple exception handlers and only allowing developers to append their own, should we move our built-in library handlers out of managedFetchAsync and put them in that array initially?  Could be a nice way to demonstrate the pattern and simplify managedFetchAsync a bit.
I.e.  private _exceptionHandlers: FetchExceptionHandler[] = [this.TIMEOUT_HANDLER, this.ABORTED_HANDLER, this.SERVER_UNAVAILABLE_HANDLER]


Update meilisearch version

Describe the bug
Hi, I used the docker image 0.27.. with "getmeili/meilisearch:latest" in my compose.
So with your update to 0.28 the image is pulled and restarted.
After this meilisearch does not work becaus of the diffrent engine versions.
expected Meilisearch engine version: 0.27.X, current engine version 0.28.
Expected behavior
The https://docs.meilisearch.com/learn/advanced/updating.html#verify-your-database-version is not practical
I think there must be an auto update! Or the engine 0.28 must be compatible with 0.27.... or something else.
What do you think?
Meilisearch version: [e.g. v0.28.0]
Additional context
Docker, Docker-Compose

Hello @MathiasWeisheit
Indeed, Meilisearch does not have any auto-update for the moment, this is something we are currently working on, see this issue: https://github.com/meilisearch/meilisearch/issues/2590
In the meantime, you have to use dump to make the update. In the documentation you linked, there is a section to explain how to process

until this issue is solved i will fix my dockerimage to v.0.28


Translations aren't loaded for zh-Hans, zh-Hant, pt-PT

Version

Poedit: 3.0 (6345)

macOS: 11.5 Beta (20G5023d)

System Information

I'm using Poedit on MacBook Pro (13-inch, M1, 2020)

My language order is Chinese (Traditional) (zh-Hant) > Chinese (Simplified) (zh-Hans)

Question
Before Poedit 3.0, Poedit can correctly apply my language preference. When I updated my Poedit Pro to 3.0 Beta, I found that I am not able to use Poedit in my desired language.

I tried to fully quit (Cmd-Q) the Poedit and specify the language in the Linux way:
LANG=zh_TW.UTF-8 LANGUAGE=zh_TW open /Applications/Poedit.app/

But it failed with no errors:

I also have taken a look with the Application/Poedit.app/Contents/Resources folder and I found that the localized string file has been under there:

and seems like all localized:

I don't have any idea about why it couldn't apply correctly.
Additional Information

zh-Hant and zh-Hans both are fully localized In Crowdin.

Thanks! There seems to be some lookup logic problem in updated wxWidgets. I'm looking into it.
To your observations:

I tried to fully quit (Cmd-Q) the Poedit and specify the language in the Linux way:

You're not on Linux, macOS doesn't use these.

and seems like all localized:

You're looking at a file that does not have the strings you're showing in your screenshots; that would be poedit.mo.

zh-Hant and zh-Hans both are fully localized In Crowdin.

That doesn't mean that the last released version has them - it was published before the translations were updated…

Thanks! There seems to be some lookup logic problem in updated wxWidgets. I'm looking into it.
To your observations:

I tried to fully quit (Cmd-Q) the Poedit and specify the language in the Linux way:

You're not on Linux, macOS doesn't use these.

and seems like all localized:

You're looking at a file that does not have the strings you're showing in your screenshots; that would be poedit.mo.

zh-Hant and zh-Hans both are fully localized In Crowdin.

That doesn't mean that the last released version has them - it was published before the translations were updated…

You're looking at a file that does not have the strings you're showing in your screenshots; that would be poedit.mo.

Oh, I have ran msgunfmt for the poedit.mo and seems like the localized string has included in that compiled file.

That doesn't mean that the last released version has them - it was published before the translations were updated…

It has been 100% for a long time, so I assumed the latest version is fully localized :)\
Thanks for answering though!

Fix will be in 3.0 final: https://github.com/wxWidgets/wxWidgets/pull/2385
Thanks again for spotting this!

@vslavik, thank you! :-)


Anyone has success with localAI?

Describe the bug
ChatWizard does communicate with LocalAI, but a response is still not appearing.
To Reproduce
Steps to reproduce the behavior:

Start localAI
Start ChatWizard
In ChatWizard configure localAI as a forward URL like for example http://<IP_ADDRESS>:8080
In ChatWizard edit the models list and add a name of a model that is inside localAI's models folder (with extension, for example WizardLM_Uncen-7B-q5K.bin)
Start a new chat and choose the local model

Expected behavior
In your terminal window, where localAI is running, you should see that it correctly starts loading the model. But a response will never arrive at ChatWizard
Screenshots

App Info:
Copy from app about:

App Version: [Version 0.4.0 (20230709.113618)]
Platform: [darwin]
OS Type: [Darwin]
OS Version: [13.2 (22D49)]
OS Arch: [x64]

Would be cool if anyone has figured out what the problem is and how to solve it.

 I've installed 0.4.1 and it is working like a charm now. Thanks very much : )


Wrong boolean operator precedence

Both boolean operators && and || are at the same precedence (0) at the moment. (implemented in this file).
In almost all other languages, as well as boolean algebra in mathematics, the && operator has higher precedence however. Either the precedence should be changed, or the compiler should enforce parenthesis around chaining these operators to avoid confusion.

So there are 2 things here to do after discussion on zulip:

Simply update the precedence. Extremely simple change to the file linked above. Yeah, increment a few things by 1 and reorder and above or.
Add a warning to the compiler that says parens should be added (probably not that hard, but definitely more in depth)

Made the first change, eager to take up the second one if I can get some initial direction on how to attempt it.


Support https for pinot-spark-connector

pinot-spark-connector does not support https protocol as mentioned here

cc @snleee

FYI, I have started to work on initial implementation based on recommendations from @xiangfu0


Reduction APIs for groupby, groupby_bins, resample, rolling

From #2356
APIs for groupby, groupby_bins, resample, rolling are different, especially for multi-dimensional array.
import numpy as np
import xarray as xr
import pandas as pd

time = pd.date_range('2000-01-01', freq='6H', periods=365 * 4)
ds = xr.Dataset({'foo': (('time', 'x'), np.random.randn(365 * 4, 5)), 'time': time,
                 'x': [0, 1, 2, 1, 0]})

ds.rolling(time=2).mean()  # result dims : ('time', 'x')
ds.resample(time='M').mean()  # result dims : ('time', 'x')
ds['foo'].resample(time='M').mean()  # result dims : ('time', )  maybe a bug #2362
ds.groupby('time.month').mean()  # result dims : ('month', )
ds.groupby_bins('time', 3).mean()  # result dims : ('time_bins', )

In rolling and resample(for Dataset), reduction without argument is carried out along grouped dimension
In rolling, reduction along other dimesnion is not possible
In groupby and groupby_bins, reduction is applied to the grouped objects and if without argument, it reduces alongall the dimensions of each grouped object.

I think rollings API is most clean, but I am not sure it is worth to change these APIs.
The possible options would be

Change APIs of groupby and groupby_bins so that they share similar API with rolling.
Document clearly how to perform resample or groupby with multidimensional arrays.

This does seem to be a little inconsistent currently.
My original reasoning for the default groupby behavior was that that this felt more consistent with the behavior for non-grouped reductions, which reduces across all dimensions.
But it's probably less useful, and results in a lot of redundant code. I can only think of a few times when I've actually wanted this behavior, rather than summing over only the grouped dimension. Especially when going from 1D -> ND, this is a likely source of errors.
So instead, we could change this to:
ds.groupby('time.month').mean()  # result dims : ('month', 'x')
ds.groupby('time.month').mean(dim=None)  # result dims : ('month',)

Or maybe we could add a special constant xarray.ALL_DIMS to indicate all dimensions? This is probably the most readable version:
ds.groupby('time.month').mean(dim=xarray.ALL_DIMS)  # result dims : ('month',)


ERR_NAME_NOT_RESOLVED

I pulled the repo and ran npm install and then npm run serve to load the local page. When doing this I am getting errors

Hello,
This is the frontend repo, you need also the backend. https://github.com/Piterden/crosswords-api
You can try if you wish.


Multiple monitoring for a singles Monitor

I was wondering if it makes sense to have more than one type of 'monitor' on a monitor, for example I have an mqtt server that uses port 1883 and 9001, I want to monitor both ports on the same monitor (service).

Duplicate of several other issues. Specifically #497.


Best practice for parameter tuning?

Dear Mr. Afik,
I recently found your paper and git repository on ridge directed ring detection which is very interesting. Installing all requirements and using your code was easy and showed amazing results on the sample image you provided. However, applying your detection algorithm to a set of my own images, only very few to none rings were detected.
I would greatly appreciate a “best practice” guide on how to tune all the relevant parameters to a new dataset, for example those linked below. Rmin and Rmax are obvious parameters which I can set easily. Following the instructions in your demo notebook and reviewing your code I realized that I could access the image data after preprocessing using ridge_hough.deriv['principal_curv'] command, which facilitates to set the right value for sigma and ksize. Unfortunately the impact of all the remaining parameters, including curv_thresh, vote_thresh, circle_thresh, dr and eccentricity I do not fully understand.
It seems to me that the data provided by the debugging_rings_detection command might give further insight into the correct choice of above mentioned parameters. Unfortunately I only have limited programming skills and am not able to tune these parameters, except for changing them randomly, to get the best results for my attached sample frames.
Your help would be much appreciated!
Sample images:
https://drive.google.com/open?id=1JsDFnkMnH8uRMzJxOOlfkbkK7uAZ9UGi

Dear @maximweb ,
I am sorry for the much delayed response -- will look into it and post a reply toward the end of next week.
Hope that would still be useful for you.
Cheers,
Eldad


hbase shell while using the shell language

Hello,
%sh
hbase shell
count 'events'

The command count 'events' is not done within the hbase shell, but within the shell.
Thank you!

I guess it's expected result.. You'll need to run these codes in one line.
On Mon, Mar 9, 2015 at 6:36 PM Alina GHERMAN<EMAIL_ADDRESS>wrote:

%sh
hbase shell
count 'events'
The command count 'events' is not done within the hbase shell, but within
the shell.
Thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/NFLabs/zeppelin/issues/376.


报错

/BTApp-master/BanTang/Utitls/BoPhotoPicker/BoPhotoListView.m:134:6: Use of undeclared identifier '_indexPathsForSelectedItems'
/BTApp-master/BanTang/Utitls/BoPhotoPicker/BoPhotoListView.m:16:47: Auto property synthesis will not synthesize property 'indexPathsForSelectedItems' because it is 'readwrite' but it will be synthesized 'readonly' via another property

i have this error too


how to get proxy status?

Create multiple proxys, using Jodis client, what i can do if one proxy doesn't work.
From wandoulabs' doc:
"如果出现错误 "Some proxies may not stop cleanly ..." 如何处理？"
We should do "请确认这个 proxy 已经确实下线，或者确实进程已经被杀掉后，然后清除这个 proxy 留下的fence，使用命令：  codis-config -c config.ini action remove-fence  进行操作。"
How can i check proxy status, from zk or ps proxy process?
and then how to do next?
we should remove-fence and restart again?

Dashboard is a better choice.

but sometimes we couldn't check dashboard everytime, we should monitor this backend. how can we check this?

Everything provide by dashboard can be get from command line too :)
And yes, you should remove-fence and restart again or add new proxy .


[FEAT] Doc models factories

:bulb: Feature request
Feature Name

Doc models factories
The Desired Behavior

Docsgen models should have associated factories
Your Use Case

As a maintainer, I would like to have factories for docsgen models.

blocked by #3325 and #2819


getFreeDiskStorage returns internel or external? Docs wrong?

Summary
I tested on android 6.
The Note on the Docs says it returns external free space for Android but in my case it's actually returning (correctly) the internal free Storage, this is good, but are the docs wrong?
Current behavior
returns internal free disk storage
Expected behavior
?

Hi @L3V147H4N, here's the java function used to retrieve the info:
https://github.com/rebeccahughes/react-native-device-info/blob/master/android/src/main/java/com/learnium/RNDeviceInfo/RNDeviceModule.java#L143
Looking at its documentation, it seems like you're right:

Note: don't be confused by the word "external" here. This directory can better be thought as media/shared storage. It is a filesystem that can hold a relatively large amount of data and that is shared across all applications (does not enforce permissions). Traditionally this is an SD card, but it may also be implemented as built-in storage in a device that is distinct from the protected internal storage and can be mounted as a filesystem on a computer.

Would you like to open a PR to fix the documentation in the README.md file?

Sure, I'll get to it tonight 👍


Support CommonName-only wildcards

On Mon Nov 2, 2020 at 9:04 PM EST, Adnan Maolood wrote:

When I made the fix, I posted about it on the mailing list:
https://lists.orbitalfox.eu/archives/gemini/2020/002391.html

Using == to compare the common name with the domain name doesn't handle all cases though. The common name could contain a wildcard hostname.
To resolve this I simply vendored in the parts of crypto/tls responsible for hostname verification, and removed the GODEBUG variable check. You can find the code here:
https://git.sr.ht/~adnano/go-gemini/tree/master/vendor.go

Source

Fixed in 2015958ad2af0008b64b83be41382405be8260f4.


package is not available at pypi.org

currently, package is not available at pypi.org.
so what's the release date?

We don't have a concrete plan (yet.) Currently, for example, there's no protobuf support, a huge gap.
The question suggests we really should have some kind of plan for that. Thanks for raising it.

I'd like to register interest in having this on pypi sooner rather than later. I'm moving ahead to adopt in two projects where this is an ideal fit, no need for protobuf suport.
[django-scoped-rbac)[https://gitlab.com/openteams/django-scoped-rbac]
CEL is a perfect fit for conditional policy expressions for access control.
scrud-django
CEL policy expressions make a great way of defining "state" in a workflow for validation and access control.

I'm hoping to add some folks to the project who can help with things like PyPI deployment and acceptance testing.

i can have a look at copying over some of our release automation from c7n repo, although its pretty minimal. @slott56 what's your pypi username?

https://pypi.org/user/slott/

Bumping this now after Steven's protobuf work has been merged into master. Has anyone started the work to publish this to PyPI? If not, I can look into it

https://pypi.org/project/cel-python/ is now available! I'd like to look into some release automation, but for now, I manually pushed it up

thanks @kylejohnson514 please ensure both slott and myself (kapilt) have full access to the package in pypi.


Machine User auth token

Is this a BUG REPORT or FEATURE REQUEST?:
feature
What happened?:
A user has an automated application that needs to access a pachyderm cluster. There are differing real users who need R or R/W access to different parts of the DAG, so auth needs to be turned on. As is, we do not have a way to provide a machine user token for permanent access for an application.
What you expected to happen?:

When an admin turns on auth, they also have access to an API call to generate/revoke a machine user token

Dupe of #2700


Set  mounting parameters according to the Buildah version when building Containers

Set the rw mounting parameter when the installed buildah version requires it and not if it does not. Also update the required benchbuild version to 3.6.1. to include this fix in benchbuild as well.

Only this one test needs fixing. It just needs to include the ,rw in teh expected output.

Only this one test needs fixing. It just needs to include the ,rw in the expected output.
It needs to be removed, but that would mean the test would fail depending on the buildah version would that be okay?

Only this one test needs fixing. It just needs to include the ,rw in the expected output.

It needs to be removed, but that would mean the test would fail depending on the buildah version would that be okay?

Neither, just use the buildah version check to determine if rm should be there or not.


Benchmark reproduce failed.

Hi ,I am trying to reproduce your result on rk3399 by following instruction in [benchmark](https://github.com/dmlc/tvm/tree/master/apps/benchmark)  . and after running the last command:

python  mobile_gpu_imagenet_bench.py --model rk3399 --rpc-key rk3399

The error info is , LLVM ERROR: Only small and large code models are allowed on AArch64.
I am using llvm7 both in host machine and rk3399.So what caused this error, and  how to fix to reproduce?

related https://discuss.tvm.ai/t/program-crash-in-llvm-7-0/930
It seems some fixes to codegen is needed for llvm 7.0

Thanks. LLVM6 works.

@nopattern
Hi maybe you can try build the entire tvm with clang, which may shipped with your LLVM package.
(e.g, the llvm-7.0 is downloaded from official website and put into /opt/clang+llvm-7.0.0-x86_64-linux-gnu-ubuntu-16.04)
// in config.cmake
set(USE_LLVM /opt/clang+llvm-7.0.0-x86_64-linux-gnu-ubuntu-16.04/bin/llvm-config)

// start build
mkdir build ; cd build
cmake -DCMAKE_C_COMPILER=/opt/clang+llvm-7.0.0-x86_64-linux-gnu-ubuntu-16.04/bin/clang
  -DCMAKE_CXX_COMPILER=/opt/clang+llvm-7.0.0-x86_64-linux-gnu-ubuntu-16.04/bin/clang++
   ../
make

@tqchen
same as this post, I think this is a good approach to avoid many compile/runtime issue with LLVM enabled. so maybe can recommend tvm users to use clang/clang++ to compile tvm when they what to use LLVM codegen?


Cannot add properties

Problem description / use case
Property tab misses "Add" function.
Functional Requirements
Something similar to the classes tab:

Hi @rnsrk please use discussions to ask a question or provide feedback. Thanks!


Unknown syntax: 'default!'

What does the syntax default! do? I've never seen it anywhere else and after a lot of googling, this document is the only place that has it! I would really appreciate an explanation. I tested it with some local dotnet projects, and I can confirm that it is at least valid C# syntax. But I really have no idea what it does.
edit Example from the document
public record Person
{
    public string FirstName { get; init; } = default!;
    public string LastName { get; init; } = default!;
}

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

ID: 20818c64-4ad8-fb15-5d79-e7f98614e9c4
Version Independent ID: de1f383b-ef20-4776-57a8-04edae892166
Content: Records - C# reference
Content Source: docs/csharp/language-reference/builtin-types/record.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

default is the default literal. In this case, it's short for default(string), which is the same as null.
! is the null-forgiving operator, which silences warnings related to nullable reference types.

Alright, makes sense. Thanks


Create Organization Page - Remove Cancel Button

Web Path
Home Page -> John Doe -> Create Organization

Requirement
Remove Cancel button

Done in PR


PAO

Description
Adds PAO's to the list of available virtual-space localization methods. local_cutoff controls the Boughton-Pulay completeness check.
Note that the Hamiltonian class now stores the psi4.core.Molecule object (for Z-values), the psi4.core.BasisSet object, the full set of MO coefficients C_all (including frozen core for projection out of the AO space), and the AO-basis Fock matrix (for determining the new semi-canonical orbital energies).
Also note that the lpno_cutoff kwarg has been swapped to local_cutoff in ccwfn, and Local now receives the local type as a string, the (active) MO coefficients, and the number of frozen core orbitals.
Todos
Notable points that this PR has either accomplished or will accomplish.

[x] Check all domains (singles and pairs) against Psi3
[x] Check standard and frozen-core energies against Psi3
[x] Tests
[x] Docs / output / notation pass
[x] Probably swap methyloxirane test case for water in a big basis set to cut down test time.
[x] Additional options: norm-cutting? variable linear dependance check?

Questions

[x] Currently, I'm not cutting the pair domains down by the norm as described in Hampel & Werner 1996. Psi3 strangely does this only when frozen core is on -- I believe it should be when frozen core is off, since that is when you are probably most likely to see very similar orbital norms around the core. Currently the code is checked against a locally-modified Psi3 which skips this step. Should I add this in, and should it be if frozen_core or if not frozen_core?

I added core_cut, which cuts the PAOs by their norm, regardless of freeze_core status

[x] 1e-6 seems to be a relatively stable choice for the linear dependence check on the redundant pair domains. Should this be included as an extra optional? (The function call to Local is getting rather complicated...)

Added lindep_cut with a default of 1E-6 in case we want it later, but ccwfn.py doesn't take/pass the argument.

Status

[x] Ready to go

Codecov Report

Merging #19 (9b2e4d3) into main (e2d51f7) will increase coverage by 0.08%.
 Consider uploading reports for the commit 4d7690b to get more accurate results


Add campaigns

Plus some rubocopping...

Hey Cromulus, we've updated the gem to match V2 of the API.  Campaigns have been added in the course of that upgrade.


Handle string/unicode escape during string encode

Hello everyone,
python2 differs from python3 when encoding a string with escape chars. For example:
# python2
my_str.encode('string-escape')
# python3
my_str.encode('unicode-escape')

It would be useful to handle the escape encoding in six, such as:
if PY2:
    encode_escape = 'string-escape'
else:
    encode_escape = 'unicode-escape'

Or a method that handle string encoding escape automatically.
Thanks in advice,
Andrea

In both versions, I believe you can simply use codecs.unicode_escape_(encode|decode).


hcp: add admin idp (user and password for login)

hypershift cluster:
Create rosa-admin user and add it to cluster-admins group
add login with user and password info to auth/api.login file
add rosa-admin password to auth/rosa-admin-password

/verified

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
Cherry-pick will be started when PR is merged

To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
To add a label by comment use /<label name>, to remove, use /<label name> cancel
To assign reviewers based on OWNERS file use /assign-reviewers
To check if PR can be merged use /check-can-merge

Supported /retest check runs

/retest tox: Retest tox
/retest build-container: Retest build-container
/retest python-module-install: Retest python-module-install

Supported labels

hold
verified
wip
lgtm

/verified


debug

Rebased version of #1119 since the original author is unresponsive

ok, sorry I have not had time for this.

@h4ck3rm1k3 no worries. It wasn't a problem, I did this in a three-line script :)

it was a long time ago. I was adding in debug messages iirc.
On Tue, Sep 27, 2016 at 7:21 PM, Alex Jordan<EMAIL_ADDRESS>wrote:

@h4ck3rm1k3 https://github.com/h4ck3rm1k3 sorry this hasn't been
reviewed yet. I'm not quite sure what you're trying to accomplish here -
can you elaborate?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pump-io/pump.io/pull/1159#issuecomment-250027627, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACIV78kFUGoXzD4Ju_XwujmxqndCTxtks5quaULgaJpZM4H0007
.

--
James Michael DuPont
Kansas Linux Fest http://kansaslinuxfest.us
Free/Libre Open Source and Open Knowledge Association of Kansas
http://openkansas.us
Member of Free Libre Open Source Software Kosova http://www.flossk.org
Saving Wikipedia(tm) articles from deletion http://SpeedyDeletion.wikia.com

@h4ck3rm1k3 OK. I'd love to get this in, but I'm really not sure what's going on - in particular I'm not sure what execArgv is as AFAICT it's not a real option for cluster.setupMaster. https://nodejs.org/api/cluster.html#cluster_cluster_setupmaster_settings
I'm happy to merge this after I understand what's going on and review the changes, but if neither of us really understands what's going on, I'd rather just close this. Thoughts?

close it
On Tue, Sep 27, 2016 at 11:04 PM, Alex Jordan<EMAIL_ADDRESS>wrote:

@h4ck3rm1k3 https://github.com/h4ck3rm1k3 OK. I'd love to get this in,
but I'm really not sure what's going on - in particular I'm not sure what
execArgv is as AFAICT it's not a real option for cluster.setupMaster.
https://nodejs.org/api/cluster.html#cluster_cluster_setupmaster_settings
I'm happy to merge this after I understand what's going on and review the
changes, but if neither of us really understands what's going on, I'd
rather just close this. Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pump-io/pump.io/pull/1159#issuecomment-250060042, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACIV_klfVZGYVBAwJDuJ4Zt3RqvutYvks5qudlMgaJpZM4H0007
.

--
James Michael DuPont
Kansas Linux Fest http://kansaslinuxfest.us
Free/Libre Open Source and Open Knowledge Association of Kansas
http://openkansas.us
Member of Free Libre Open Source Software Kosova http://www.flossk.org
Saving Wikipedia(tm) articles from deletion http://SpeedyDeletion.wikia.com


Fix View Transitions in Safari/FF

Astro 3.0.5 fixes a bug in view transitions. This updates Astro to the new patch.

Site has since been updated in https://github.com/t3dotgg/t3-astro/commit/a85c994f33994acbeeda0c0bfb15ac199289f347 :partying_face:


remotes/docker/config: Skipping TLS verification for localhost

issue: https://github.com/containerd/containerd/issues/7392
After configuring config_path for the cri plugin, it doesn't handle the case where the host is local.
We also need to skipping TLS verification for localhost.

/ok-to-test

I believe this is a bug fix and should be cherry-picked to release/1.6

@Iceber  This fix caused another problem. The ctr command now uses http to pull images from a https localhost registry. However, the ctr command only has the--plain-httpoption. So it can never connect to the https registry anymore. I have to revert this change and rebuilt the ctr binary.

@JohnJAS Thank you for pinging me here, otherwise I wouldn't have known about your question.
I will reply as soon as possible.


Add no_user_input support to Displayable

Fixes #487

Codecov Report

Merging #552 (39fa425) into main (1ea92ad) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #552   +/-   ##
=======================================
  Coverage   97.46%   97.46%
=======================================
  Files           6        6
  Lines         394      394
  Branches       79       79
=======================================
  Hits          384      384
  Misses          8        8
  Partials        2        2

Files
Coverage Δ

columbo/_interaction.py
100.00% <100.00%> (ø)


Can't do sort popularity/popular

Hello , great work by the way , really helpful software
i can't do sort popularity (daily, weekly, all-time) on nhentai.net
i tried sort:popularity, popular , popularity-today
how do i sort?
help please
thanks a lot
@Bionus
@andrewjmetzger
@dgw
@txtsd
@op8867555

Actually apparently you can sort by popularity but you'll have to wait for Bionus to tell you how

okay , so it's a future feature
about your statement, could you give an example how to sort by date? because I can do it on another site but not nhentai


feat(events): make target optional in onXxx() methods

Make target optional in OnEventOptions so that it is no required when calling a onXxx()
method.
This allows to use "preconfigured" rules in other constructs.
Closes #2913

Pull Request Checklist

[ ] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[ ] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated
Design: For significant features, design document added to design folder

[x] Title and Description

Change type: title prefixed with fix, feat and module name in parens, which will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[ ] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

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

Is it valid to provision a Rule without any targets?

Yes

Okay, so let’s just add a test

It’s fishy that nothing broke at the consumption side when this changes from required to optional. I’d expect some null check...

It’s fishy that nothing broke at the consumption side when this changes from required to optional. I’d expect some null check...

https://github.com/awslabs/aws-cdk/blob/83eee09e8198ae1921b932c09a6d14c88392800d/packages/%40aws-cdk/aws-events/lib/rule.ts#L115-L127
There's one already

Okay, so let’s just add a test

https://github.com/awslabs/aws-cdk/blob/83eee09e8198ae1921b932c09a6d14c88392800d/packages/%40aws-cdk/aws-events/test/test.rule.ts is full of rules created without targets, what kind of test do you want to add? something in the onXxx() method of another package ?
An extra test in rds without target?
https://github.com/awslabs/aws-cdk/blob/83eee09e8198ae1921b932c09a6d14c88392800d/packages/%40aws-cdk/aws-rds/test/test.instance.ts#L315-L319

Yes sounds good. Just to exercise this specific path


test: use playwright for integration tests

the main differences with playwright compared to jest + puppeteer are

the lack of a "watch" mode
no toMatchInlineSnapshot
describe, beforeTest, beforeEach, etc need to be test. prefixed
all tests need to use test and not it

Signed-off-by: Logan McAnsh<EMAIL_ADDRESS>
Closes: #

[ ] Docs
[ ] Tests

gonna look into setting up a playwright "fixture" so the api is a bit closer to what we have with puppeteer.
so instead of passing page from playwright to all the app.whatever calls like this:
test("handles files under upload size limit", async ({ page }) => {
    let uploadFile = path.join(
      fixture.projectDir,
      "toUpload",
      "underLimit.txt"
    );
    let uploadData = Array(1_000).fill("a").join(""); // 1kb
    await fs
      .mkdir(path.dirname(uploadFile), { recursive: true })
      .catch(() => {});
    await fs.writeFile(uploadFile, uploadData, "utf8");

    await app.goto(page, "/file-upload");
    await app.uploadFile(page, "#file", uploadFile);
    await app.clickSubmitButton(page, "/file-upload");
})

we can instead do this
test("handles files under upload size limit", async ({ page, idkthenameyet }) => {
    let uploadFile = path.join(
      fixture.projectDir,
      "toUpload",
      "underLimit.txt"
    );
    let uploadData = Array(1_000).fill("a").join(""); // 1kb
    await fs
      .mkdir(path.dirname(uploadFile), { recursive: true })
      .catch(() => {});
    await fs.writeFile(uploadFile, uploadData, "utf8");

    await idkthenameyet.goto("/file-upload");
    await idkthenameyet.uploadFile("#file", uploadFile);
    await idkthenameyet.clickSubmitButton("/file-upload");
})

Oh yeah, could you also update the contributing.md file please?

CI is giving us this:

https://github.com/remix-run/remix/actions/runs/2151038318

CI is giving us this:

remix-run/remix/actions/runs/2151038318

forgot to remove the upload action, one sec :)


Added the ability to display action button items in vertical list whe…

…n there are more than two buttons.  The list is scrollable. Ref issue #14345.

I signed the CLA!


Enhanced E-commerce Website

Description
Enhanced the Contact Us and About Us page, made them linked to home page and made all the pages responsive.
Improved navbar.
Fixed the toggle menu.
Fixes:  #537
Type of change

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

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
[Before-After.zip](https://github.com/arpit456jain/Cool-Front-End-Templat

es/files/8402202/Before-After.zip)

@VidhiBhatt01 Hi! Can you please check this PR and let me know if there are any issues. I could not find the aboutus page when I cloned the repository so I created a new one.

https://deploy-preview-559--cool-front-end-templates.netlify.app/e commerce shopping website template/contactpage.html
Also resolve the merge conflicts

you have fixed the issues but tbh I was expecting something better. The UI is still very boring : (


Fix typo (exmaple to example) in README.md

Line 51 in the README.md had a typo.

Thanks for that :)

No problem. Glad to help.


[mds-*] Upgrade packages in lockfile

PR Checklist

[x] simple searchable title - [mds-db] Add PG env var, [config] Fix eslint config
[ ] briefly describe the changes in this PR
[ ] mark as draft if should not be merged
[ ] write tests for all new functionality

Impacts

[x] Provider
[x] Agency
[x] Audit
[x] Policy
[x] Compliance

Mr. Jenkins, run tests pls


Allow Tint Color to be associated with a VectorDrawable

This is useful if a user is working with a predefined library of Drawables, such as our brand icons, and wishes to tint one in a different way before passing into a UI component that doesn't take a tint color with the Drawable. It doesn't make sense to add a new drawable only for the purposes of having a new color.
This also matches the API available in Android, though it's slightly complicated by the fact that in Android there's no such thing as a VectorDrawableView; The VectorDrawable is itself a View.

 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.


Reportback source

What's this PR do?

added a column for source on reportback_files
set the source when posting a reportback (if it is included)
the mobile app is already sending this in
return the source when getting a reportback
currently source is public (along with caption and status)

How should this be reviewed?
Post a reportback and include a source as one of the parameters.
Retrieve that reportback and make sure you DO NOT see the source.
Try the same thing with no sources.
Relevant tickets
Fixes #6385
Checklist

[x] Documentation added for new features/changed endpoints.

@katiecrane do you mind taking this wonderful opportunity to add (and update) the reportback endpoints to the documentation folder

@angaither I have already handwritten a note to do so! 😄

I should probably read that I'm still not good at it

👍 🚢


Add back some of the extensibility to JustSaying 7 APIs

As part of the fluent configuration rework, some of the lesser known APIs were not fully implemented:
https://github.com/justeat/JustSaying/blob/main/src/JustSaying/Fluent/SqsConfigurationBuilder.cs#L149-L159
We have users that use:

MessageBackoffStrategy (multiple usages)
FilterPolicy (1 usage)
ErrorQueueRetentionPeriod (very rare, but there is at least 1 usage)

We might want to resurface some of these for v7, and assess the others.

MessageBackoffStrategy will at least be needed to ensure a smooth migration path from JS 6


Use country names directly instead of model-specific renaming

Following a suggestion by @HauHe, this PR changes the region-mapping for Euro-Calliope to use country names directly instead of the more cumbersome <model>|<country> currently applied as part of the renaming upon submission to the ECEMF-internal Scenario Explorer. A similar change was made for the OSeMBE model in #233.
The idea is to change to a system where the model-specific region names are only used where the model does have non-obvious region names or aggregates. For models with a national resolution, we will directly use the country names without renaming.
This PR does not affect any scenario data already uploaded to the Scenario Explorer - it only changes the processing for future uploads. I will clean up existing data at a later stage.
For future uploads of Euro-Calliope, there are two minor changes required in your data:

"Netherlands" -> "The Netherlands"
"Northern Macedonia" -> "North Macedonia"

Please take a look @FLomb and @sjpfenninger!
fyi @phackstock

Fine with me!

Sure, please go ahead!


HTML template added for the recovery email

It should be possible to send both text and HTML versions of the recovery email. Here is a proposal to handle this case.

Bump. Would really appreciate this feature

+1


sim: Enable debug tracing configuration with M5Op

Adds a new pseudo instruction m5_configure_tracing. It can be used to turn on or turn off the debug tracing from the workload, e.g., to skip the booting.
Change-Id: I7b86762b8e4fa918df6cf697fa7abe9e359a88b1

I can see how this is useful, but I am wondering if we can't piggyback on existing logic to achieve a similar behaviour. For example reusing workbegin/workend m5 ops which are used anyway to annotate ROIs.

I can see how this is useful, but I am wondering if we can't piggyback on existing logic to achieve a similar behaviour. For example reusing workbegin/workend m5 ops which are used anyway to annotate ROIs.

For reference: The GPU model does something similar (catching an exit event and turning on debugging): https://github.com/gem5/gem5/blob/stable/configs/example/gpufs/runfs.py#L309

As others have said, I think this can already be done. Also, we are working to refactor the exit events in general to be more flexible. See https://github.com/gem5/gem5/issues/751 for more details.
Also, another example: https://github.com/gem5bootcamp/2024/blob/main/materials/02-Using-gem5/03-running-in-gem5/03-run-x86-SE/complete/03-run-x86-SE.py#L79 and slide number 23 here: https://bootcamp.gem5.org/#02-Using-gem5/03-running-in-gem5

Thank you for you feedback 😄. I was not aware that this already possible. Therefore, I close this PR..
Also thank you for the example, I can then start using it for my simulations 😄.

Thank you for you feedback 😄. I was not aware that this already possible. Therefore, I close this PR.
Also thank you for the example, I can then start using it for my simulations 😄.

No worries. Thanks for being part of the community and contributing back!


Use an updated ubuntu iso which pxe boots faster

This reduces the pxe boot time for a VM from 20-25 minutes to less than 10min.

I checked this last night.  It took the Ubuntu installation step from 43 minutes to 12.  I'm real happy to merge this.


Fixed inaccurate Chinese words

Describe the pull request

What does your PR fix?

Fixed inaccurate Chinese words

Which triplets are supported/not supported? Have you updated the CI baseline?

No

Does your PR follow the maintainer guide?
yes

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

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Ok, I will check the other documents later and then report back here.

Thanks for your contribution!

Thanks everyone for the reviews :)


NEVER ENDING SYNTAX ERROR

Describe the bug
i have spent over 60M tokens trying to fix a syntax error, this is a joke and it feels intentional. I have reported issues before but nobody has addressed how to fix them.  If it is not addressed i will be demanding a refund
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-pwhbeh
Steps to reproduce
i have spent over 60M tokens trying to fix a syntax error, this is a joke and it feels intentional. I have reported issues before but nobody has addressed how to fix them.  If it is not addressed i will be demanding a refund
Expected behavior
i have spent over 60M tokens trying to fix a syntax error, this is a joke and it feels intentional. I have reported issues before but nobody has addressed how to fix them.  If it is not addressed i will be demanding a refund
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 147630617,
  "usedJSHeapSize": 143190689,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = bkowalski21
Chat ID = 650051b832c0

Additional context
i have spent over 60M tokens trying to fix a syntax error, this is a joke and it feels intentional. I have reported issues before but nobody has addressed how to fix them.  If it is not addressed i will be demanding a refund

Apologies for the frustration! Could you send a screenshot of the syntax error you are seeing so we can investigate further? Thank you.

Hi Brian,
I am extremely frustrated and disappointed. If you look at my account you
can see how much money I have spent, I have hit a syntax dead end and have
spent 50M tokens today alone trying to fix it.
It is extremely frustrating and the problems have continued to snowball.
Attached is the notice I was getting continuously.
I wasted another $200 and got nothing accomplished.
Can you please advise.
Thank you,
Braden
On Mon, Dec 2, 2024 at 4:50 PM Brian @.***> wrote:

Apologies for the frustration! Could you send a screenshot of the syntax
error you are seeing so we can investigate further? Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/stackblitz/bolt.new/issues/3574#issuecomment-2512997793,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BM5FFY7BHVBDRHYSZP7Y6Y32DTIY7AVCNFSM6AAAAABS4MBDNSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJSHE4TONZZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--

Brian,
I just wanted to follow up with you here to see what we might be able to do
to right the ship? I was making great progress before stalling, but I am
hesitant to continue investing into this.
Please let me know as soon as you can.
Thank you,
Braden
On Mon, Dec 2, 2024 at 5:51 PM Braden Kowalski @.***>
wrote:

Hi Brian,
I am extremely frustrated and disappointed. If you look at my account you
can see how much money I have spent, I have hit a syntax dead end and have
spent 50M tokens today alone trying to fix it.
It is extremely frustrating and the problems have continued to snowball.
Attached is the notice I was getting continuously.
I wasted another $200 and got nothing accomplished.
Can you please advise.
Thank you,
Braden
On Mon, Dec 2, 2024 at 4:50 PM Brian @.***> wrote:

Apologies for the frustration! Could you send a screenshot of the syntax
error you are seeing so we can investigate further? Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/stackblitz/bolt.new/issues/3574#issuecomment-2512997793,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BM5FFY7BHVBDRHYSZP7Y6Y32DTIY7AVCNFSM6AAAAABS4MBDNSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJSHE4TONZZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--

--


Missing subject in sentence

Please reject if I have misread this and made a mistake :D

@ablease Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@ablease Thank you for signing the Contributor License Agreement!


fix: update useModal args to be optional

Description
This PR updates the props argument of useModal to default to an empty object when not provided any arguments.
Fixes #62.
Solution
This change makes it a little bit easier to use the package and follows the doc examples.

Let's get a release branch going for this, as Clinton has some API updates as well. We can regroup on that tomorrow! @marlonmarcello @wonderfulfrog

Opened #65 , if you don't mind @wonderfulfrog, please change the base branch for this PR to that one! 🙏

@andrewrubin Done! Changed to release/v3.0.0.


Slim down base image

This cuts things down a bit ..

Cuts it in half, I can rework the pip install to preserve -e


Update to provide updated CSS instructions

Updated language server references based on https://github.com/neovim/nvim-lspconfig/issues/965#issuecomment-860008798
Also added basic rules override example.

Thanks for the PR, but CONFIG.md is auto-generated, so you'll want to modify the (lua) source file directly.

Thanks for the PR, but CONFIG.md is auto-generated, so you'll want to modify the (lua) source file directly.

Doh! Thanks will do!

Hi @mjlbach I opened a new pull request with the edits in the Lua file. Hope that is more useful.


test: [M3-8478] - Fix Cypress StackScript Linode deploy test flake

Description 📝
This resolves some causes of test flakiness in the StackScript Linode deploy test in create-stackscript.spec.ts.
There seem to be two main causes of flake for this test in CI, and this PR attempts to fix both:

Attempting to find a region in the region drop-down without scrolling to it first
Issues selecting a region quickly upon page load

I also changed the way that StackScripts get entered into the create form. Using cy.type() to input the script is slow and adds several seconds to our test runtime, so I've circumvented it by setting the field's value directly.
I ran the test on repeat 30 times (with the other test skipped) and it passed each time.
(cc @mjac0bs -- I'm curious if you continue seeing failures with these changes? No worries if you don't have time to take a look!)
Changes  🔄

Scroll to desired region before attempting to select it from drop-down
Wait for request to account availability endpoint to resolve before interacting with region select

ℹ It's not clear to me if interacting with the region select before this request resolves is what causes the region selection issue, or if waiting for this request to resolve only fixes it incidentally

How to test 🧪
Because this test doesn't fail every time, CI may not be the most reliable way to gauge the effectiveness of this change.
You can run the test locally with this command (feel free to skip the other test since it's a slow one and isn't related to these changes):
yarn cy:run -s "cypress/e2e/core/stackscripts/create-stackscripts.spec.ts"

As an Author I have considered 🤔
Check all that apply

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[ ] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

We'll try to merge this after @AzureLatte's PR #10788 just in case there are merge conflicts

I'm still seeing the failure when I checkout this branch :(

@jaalah-akamai I'm assuming it fails consistently, too? There's another issue related to this test that's captured by M3-8471 but I believe is unrelated to the flake I'm trying to address here.
You could help me narrow it down a lot:

Are you running Cloud via yarn up or yarn dev?
If so, does the test still fail if you run Cloud via yarn build && yarn start:manager:ci?

Are you running Cloud via yarn up or yarn dev?
If so, does the test still fail if you run Cloud via yarn build && yarn start:manager:ci?

I ran it with yarn install && yarn build && yarn dev and it fails consistently
Running it with yarn build && yarn start:manager:ci failed too (see report)

Report
yarn run v1.22.21
$ yarn workspace linode-manager cy:run -s cypress/e2e/core/stackscripts/create-stackscripts.spec.ts
$ cypress run -b chrome -s cypress/e2e/core/stackscripts/create-stackscripts.spec.ts

DevTools listening on ws://<IP_ADDRESS>:62511/devtools/browser/5bc5edf4-b1d8-4777-9f9c-7c8f62c7ec65
2024-08-23 13:25:34.958 Cypress[75754:128427111] WARNING: Secure coding is not enabled for restorable state! Enable secure coding by implementing NSApplicationDelegate.applicationSupportsSecureRestorableState: and returning YES.
(node:75782) ExperimentalWarning: Custom ESM Loaders is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
(node:75782) ExperimentalWarning: The Node.js specifier resolution flag is experimental. It could change or be removed at any time.
┌──────────────────────┬─────────────────────────────┐
│       (index)        │           Values            │
├──────────────────────┼─────────────────────────────┤
│ API Request Root URL │ 'https://api.linode.com/v4' │
└──────────────────────┴─────────────────────────────┘

====================================================================================================

  (Run Starting)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Cypress:        13.11.0                                                                        │
  │ Browser:        Chrome 127 (headless)                                                          │
  │ Node Version:   v18.14.1 (.../node/v18.14.1/bin/node)                 │
  │ Specs:          1 found (create-stackscripts.spec.ts)                                          │
  │ Searched:       cypress/e2e/core/stackscripts/create-stackscripts.spec.ts                      │
  │ Experiments:    experimentalRunAllSpecs=true,experimentalMemoryManagement=true                 │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘

────────────────────────────────────────────────────────────────────────────────────────────────────

  Running:  create-stackscripts.spec.ts                                                     (1 of 1)
Browser information:
┌─────────────────┬──────────────────────────┐
│     (index)     │          Values          │
├─────────────────┼──────────────────────────┤
│     Browser     │ 'Chrome v127.0.6533.120' │
│     Family      │        'chromium'        │
│    Headless     │          'Yes'           │
│ Chrome Headless │          'Old'           │
└─────────────────┴──────────────────────────┘

  Create stackscripts
    1) creates a StackScript and deploys a Linode with it
    2) creates a StackScript with Any/All target image

  0 passing (1m)
  2 failing

  1) Create stackscripts
       creates a StackScript and deploys a Linode with it:
     AssertionError: Timed out retrying after 80000ms: Unable to find an element with the text: US, Chicago, IL (us-ord). This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

Ignored nodes: comments, script, style

          North America

              US, Atlanta, GA

            (us-southeast)

              US, Chicago, IL

            (us-ord)

              US, Dallas, TX

            (us-central)

              US, Fremont, CA

            (us-west)

              US, Houston, TX

            (us-hou)

              US, Los Angeles, CA

            (us-lax)

              US, Miami, FL

            (us-mia)

          <li
            aria-label=""
            aria-selected="false"
            class="MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiAutocomplete-option css-1iwvvk7-RegionSelectListItem"
            data-option-index="7"
            data-qa-option="us-east"
            data-qa-tooltip=""
     ...
      at Object.findByTitle (http://localhost:3000/__cypress/tests?p=cypress/e2e/core/stackscripts/create-stackscripts.spec.ts:46731:78)
      at Object.findItemByRegionLabel (http://localhost:3000/__cypress/tests?p=cypress/e2e/core/stackscripts/create-stackscripts.spec.ts:46768:33)
      at fillOutLinodeForm (http://localhost:3000/__cypress/tests?p=cypress/e2e/core/stackscripts/create-stackscripts.spec.ts:52988:21)
      at Context.eval (http://localhost:3000/__cypress/tests?p=cypress/e2e/core/stackscripts/create-stackscripts.spec.ts:53074:7)

  2) Create stackscripts
       creates a StackScript with Any/All target image:
     Error: Linode APIv4 request failed with status code 400
- This Linode's region does not support VLANs at this time. (field 'interfaces[0].purpose')
Request: POST https://api.linode.com/v4/linode/instances
      at enhanceError (http://localhost:3000/__cypress/tests?p=cypress/support/e2e.ts:13268:14)
      at wrapPromise (http://localhost:3000/__cypress/tests?p=cypress/support/e2e.ts:13332:17)

  (Results)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Tests:        2                                                                                │
  │ Passing:      0                                                                                │
  │ Failing:      2                                                                                │
  │ Pending:      0                                                                                │
  │ Skipped:      0                                                                                │
  │ Screenshots:  2                                                                                │
  │ Video:        true                                                                             │
  │ Duration:     1 minute, 28 seconds                                                             │
  │ Spec Ran:     create-stackscripts.spec.ts                                                      │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘

I bet this has to do with the region changes again?

I bet this has to do with the region changes again?

Thanks @jaalah-akamai, I'm intrigued.
The DOM printout makes it pretty clear why the test is failing: it expects the label and ID text to be inside the same element, e.g.
US, Chicago, IL (us-ord)
 and in your case it's more like
US, Chicago, IL
 (us-ord)
But I'm baffled where this is coming from, and haven't been able to reproduce this at all on my end, even messing around with the Gecko and Linode Create v2 feature flags. Any idea where this is coming from?

@hana-linode Can we double check this? This appears to be causing the issue. cc @jdamore-linode

@hana-linode Can we double check this? This appears to be causing the issue. It's outside the
 in that one case cc @jdamore-linode

Thanks for the find, @jaalah-akamai! I can update the Cypress ui.regionSelect.findItemByRegionId() and ui.regionSelect.findItemByRegionLabel() utils to account for both states so no worries as far as getting tests passing! I'll do that in a separate PR for M3-8471.
One thing that @abailly-akamai noticed here, however, is that this change prevents users from searching for regions by ID in the drop-down. That breaks a couple tests which we can fix on a case-by-case basis, but I'm wondering if that ought to be treated as a bug and fixed in the component instead?
(cc @hana-linode)

This could be just a me problem, but I got this reason for a test failure (used yarn up) 😅

This could be just a me problem, but I got this reason for a test failure (used yarn up) 😅 I'd used yarn build && yarn start:manager:ci right before, and didn't have any issues - will try rerunning too
(screenshot)

Thanks @coliu-akamai! Re-running it might let it pass -- that failure indicates Cypress tried to create a Linode in a region without availability, and the region gets randomly selected based on what's returned by the /regions endpoint.
Since that test tries to create a Linode of type g6-dedicated-2, I'm guessing it might be some non-generally-available region associated with your test account since I'd expect that plan to be available in most Prod regions (and we haven't seen that particular failure in CI as far as I know).


Support stable Angular2.0 & Webpack

What is the expected time to release with Angular2.0? Ng2.00-rc-5 has many major changes and current Ionic2 using rc-4.
And current build system using gulp, But now a days webpack is very famous for bundling.  Angular2 also recommended to use webpack. https://angular.io/docs/ts/latest/guide/webpack.html
Please let me know if there is any way exists to solve this problem.
Thanks for this lovely framework :)

The next Ionic 2 Beta (2.0.0-beta.12) coming in this or next week will use the last RC of Angular 2 (2.0.0-rc.7). The following Ionic 2 RC 0 will be probably built on Angular 2.x already.

Thanks @NoNameProvided
I found that https://github.com/driftyco/ionic/tree/beta.12/demos this repo using rc-7.
But there is not major changes form rc7 to 2.0. So if we direct use stable 2.0 with Ionic then it would be great.
It is just my opinion. :)

I mean Ionic 2 RC 0 is already out. I can update ionic and run ionic start myApp --v2. It seems to be running angular 2.0.1 despite the changelog not mentioning it. It seems to be offline template compiling and has a module definition. It's just the changelog doesn't seem to mention anything about it, nor the steps I need to upgrade to it.

Thanks @herbzi  for clarification.
I'm trying with thissudo npm install<EMAIL_ADDRESS>-g

Ionic RC 0 technically dropped yesterday but the formal release will be today.
We are going with Rollup instead of Webpack for the bundling process. We think it is better and faster so that's what we're going with. We saw apps start-up ~3.5x faster with Rollup in our testing.
I'm going to close this. Please stay tuned today for the official announcement.
Thanks,
Dan


[backport] test: Stabilize test_unpooling_2d

This is a backport of #3341

LGTM!

Closing, as we won't backport to v2 any more.


Styled Components Middleware

Is there a way to extend every styled component other than with a mixin, maybe by having middleware of some sort. For example, we are currently doing this on every component within our UI kit to handle the edge cases where we need to have a different margin than what was hard-coded:
const Heading = styled.span`
    margin: 0;

    ${overrideMixin};
`;

const getValues = (values, { spacing }) =>
  values.map(val => (val ? `${rem(spacing[val])}` : "0")).join(" ");

const overrideMixin = ({ margin, theme }) =>
  margin ? `margin: ${getValues(margin, theme)};` : null;

export default Heading;

Then we use it like this:
<Heading margin={['md']} />
The problem with this is that we will have to insert this mixin into every component (and this could be around 100) - it would be good to be able to do this once somewhere (middleware).

Hey @joetidee, please keep usage questions on Spectrum or Stack Overflow so we can keep the issue tracker tidied up. Thank you!

So you don't want to consider this a feature?

Is there a way to extend every styled component other than with a mixin, maybe by having middleware of some sort

That's clearly a usage question. If you want to propose a feature, that's fine but needs to be more thought through.

Not sure how else to ask this without coming up with solution first. I thought it would be appropriate to engage others to see if they are having the same issues which, in turn, would prompt for the development of a feature to accommodate the need.

Spectrum is a great place to start!


Before Reporting 报告之前

[X] I have pulled the latest code of main branch to run again and the bug still existed. 我已经拉取了主分支上最新的代码，重新运行之后，问题仍不能解决。

[X] I have read the README carefully and no error occurred during the installation process. (Otherwise, we recommend that you can ask a question using the Question template) 我已经仔细阅读了 README 上的操作指引，并且在安装过程中没有错误发生。（否则，我们建议您使用Question模板向我们进行提问）

Search before reporting 先搜索，再报告

[X] I have searched the Data-Juicer issues and found no similar bugs. 我已经在 issue列表 中搜索但是没有发现类似的bug报告。

OS 系统
MacOS15.0
Installation Method 安装方式
source
Data-Juicer Version Data-Juicer版本
0.2.0
Python Version Python版本
3.10
Describe the bug 描述这个bug
最新的main分之的代码，运行算子的时候，出现以下错误信息：
Traceback (most recent call last):
  File "/Users/zyc/code/data-juicer/data_juicer/core/data.py", line 199, in process
    dataset, resource_util_per_op = Monitor.monitor_func(
                                    ^^^^^^^^^^^^^^^^^^^^^
  File "/Users/zyc/code/data-juicer/data_juicer/core/monitor.py", line 210, in monitor_func
    resource_util_dict['resource'] = mdict['resource']
                                     ~~~~~^^^^^^^^^^^^
  File "<string>", line 2, in __getitem__
  File "/Users/zyc/miniconda3/lib/python3.11/multiprocessing/managers.py", line 837, in _callmethod

    raise convert_to_error(kind, result)
KeyError: 'resource'

而在之前的代码中不会抱这个错误。试了几个算子都是这样。
To Reproduce 如何复现
python process_data.py --config ../configs/demo/process_demo.yaml
配置文件如下：
`# Process config example for dataset
global parameters
project_name: 'demo-process'
dataset_path: '/Users/zyc/code/data-juicer/demos/data/demo-dataset-chatml.jsonl'
np: 1  # number of subprocess to process your dataset
text_keys: ["messages"]
export_path: '/Users/zyc/code/data-juicer/outputs/demo-process/demo-processed_chatml.jsonl'
use_cache: false                                             # whether to use the cache management of Hugging Face datasets. It might take up lots of disk space when using cache
ds_cache_dir: null                                          # cache dir for Hugging Face datasets. In default, it's the same as the environment variable HF_DATASETS_CACHE, whose default value is usually "~/.cache/huggingface/datasets". If this argument is set to a valid path by users, it will override the default cache dir
use_checkpoint: false                                       # whether to use the checkpoint management to save the latest version of dataset to work dir when processing. Rerun the same config will reload the checkpoint and skip ops before it. Cache will be disabled when using checkpoint. If args of ops before the checkpoint are changed, all ops will be rerun from the beginning.
temp_dir: null                                              # the path to the temp directory to store intermediate caches when cache is disabled, these cache files will be removed on-the-fly. In default, it's None, so the temp dir will be specified by system. NOTICE: you should be caution when setting this argument because it might cause unexpected program behaviors when this path is set to an unsafe directory.
open_tracer: true                                           # whether to open the tracer to trace the changes during process. It might take more time when opening tracer
op_list_to_trace: []                                        # only ops in this list will be traced by tracer. If it's empty, all ops will be traced. Only available when tracer is opened.
trace_num: 10                                               # number of samples to show the differences between datasets before and after each op. Only available when tracer is opened.
op_fusion: false                                            # whether to fuse operators that share the same intermediate variables automatically. Op fusion might reduce the memory requirements slightly but speed up the whole process.
cache_compress: null                                        # the compression method of the cache file, which can be specified in ['gzip', 'zstd', 'lz4']. If this parameter is None, the cache file will not be compressed. We recommend you turn on this argument when your input dataset is larger than tens of GB and your disk space is not enough.
for distributed processing
executor_type: default                                      # type of executor, support "default" or "ray" for now.
ray_address: auto                                           # the address of the Ray cluster.
only for data analysis
save_stats_in_one_file: false                               # whether to store all stats result into one file
process schedule: a list of several process operators with their arguments
process:
Mapper ops. Most of these ops need no arguments.

generate_instruction_mapper: # filter text with total token number out of specific range
hf_model: '/Users/zyc/data/models/qwen/Qwen2-1___5B-Instruct'                           # model name on huggingface to generate instruction.
seed_file: '/Users/zyc/code/data-juicer/demos/data/demo-dataset-chatml.jsonl'       # Seed file as instruction samples to generate new instructions, chatml format.
instruct_num: 3                                         # the number of generated samples.
similarity_threshold: 0.7                               # the similarity score threshold between the generated samples and the seed samples.Range from 0 to 1. Samples with similarity score less than this threshold will be kept.
prompt_template: null                                   # Prompt template for generate samples. Please make sure the template contains "{augmented_data}", which corresponds to the augmented samples.
qa_pair_template: null                                  # Prompt template for generate question and answer pair description. Please make sure the template contains two "{}" to format question and answer. Default: '【问题】\n{}\n【回答】\n{}\n'.
example_template: null                                  # Prompt template for generate examples. Please make sure the template contains "{qa_pairs}", which corresponds to the question and answer pair description generated by param qa_pair_template.
qa_extraction_pattern: null                             # Regular expression pattern for parsing question and answer from model response.
enable_vllm: false                                       # Whether to use vllm for inference acceleration.
tensor_parallel_size: null                              # It is only valid when enable_vllm is True. The number of GPUs to use for distributed execution with tensor parallelism.
max_model_len: null                                     # It is only valid when enable_vllm is True. Model context length. If unspecified, will be automatically derived from the model config.
max_num_seqs: 256                                       # It is only valid when enable_vllm is True. Maximum number of sequences to be processed in a single iteration.
sampling_params: { "max_length": 1024 }
`

Configs 配置信息
No response
Logs 报错日志
No response
Screenshots 截图

Additional 额外信息
No response

嗨 @luckystar1992 ，感谢你的使用与反馈！
我们这边未能复现你遇到的问题，请你拉取最新版本代码再进行尝试，如还是遇到类似问题，欢迎与我们继续讨论~

@luckystar1992 你好，想问一下你解决这个问题了嘛？我也遇到了同样的问题呢！

嗨， @luckystar1992 与 @SnoopyXI ，感谢你们使用Data-Juicer！
这个报错来自于Data-Juicer的资源检测器Monitor，但在我们的测试下还是没有发现这个问题。。。
不过，在最新的PR #483 里我们开放了Monitor的开关参数open_monitor，如果你们拉取了最新代码还是会遇到这个问题，可以在配置文件中设置open_monitor: false来关闭它，这不会影响正常数据处理流程
如还有其它问题，欢迎随时讨论~


generic proxy: make sure the execute order of encoder filters is the reverse of decoder filters'

Commit Message: generic proxy: make sure the execute order of encoder filters is the reverse of decoder filters'
Additional Description:
Minor fix to the execute order of generic l7 filters.
Risk Level: Low. WIP extension.
Testing: unit test added.
Docs Changes: n/a.
Release Notes: added.
Platform Specific Features: n/a.

/assign @zhaohuabing
As code owner can you take a look? Thank you!

Since this is contrib, defer to @zhaohuabing

LGTM.

cc @htuch LGTM from maintainer is still necessary. 🤣

@lizan Thanks. 😺


[Core] Avoid NPEs on creating events

When creating trace event format events, the code used to throw NullPointerExceptions when trying to construct an event from a JsonObject that was missing required members.
This change adds static fromJson methods, which check the members exist, or otherwise throw an IllegalArgumentException. This better communicates the kind of error that occurred - the provided Json did not have the expected form.
This change also adds some basic documentation and tests for the Event classes.

Could you please explain in the PR message why throwing IAE is better than NPE?

Done. Although throwing NPE vs IAE can also be seen as a matter of preference.