Use git tag -l instead of git tag --list

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

In early versions of git (~1.7), the list flag doesn't exist and causes the brew update on install to error:

Error: Failure while executing; `git config --local --replace-all homebrew.private false` exited with 129.
error: unknown option `list'

The shortened -l flag exists in all versions, though, so switch to that.
This was reported in https://github.com/Linuxbrew/install/issues/78.
Thanks to @dawidd6 for reading the git docs better than I could at midnight! ;-)

We should be enforcing a much newer version than that: https://github.com/Homebrew/brew/blob/87ca86e873c86ef0931ff89644ee924e0bf92f2b/Library/Homebrew/brew.sh#L161-L171. I believe brew doctor will complain even if brew update does not.
A better fix would be to ensure that e.g. brew update will refuse to run any Git operations until the newer version from Homebrew has been installed.

We should be enforcing a much newer version than that:

Small changes like this one can make it a fair bit easier to bootstrap on older systems by using the git supplied by the system. I'd prefer to require and test for the minimum git version that Homebrew actually requires to operate. The challenge is that I don't know what the version is.
That said, if bootstrapping without any system-supplied git (even when the system supplies an ancient and out-of-date git exists) were robust and reliable, then I'd have no problem requiring whatever version of git is provided by the oldest macOS that we support.

Small changes like this one can make it a fair bit easier to bootstrap on older systems by using the git supplied by the system. I'd prefer to require and test for the minimum git version that Homebrew actually requires to operate. The challenge is that I don't know what the version is.

See https://github.com/Homebrew/brew/blob/87ca86e873c86ef0931ff89644ee924e0bf92f2b/Library/Homebrew/brew.sh#L161-L171

That said, if bootstrapping without any system-supplied git (even when the system supplies an ancient and out-of-date git exists) were robust and reliable, then I'd have no problem requiring whatever version of git is provided by the oldest macOS that we support.

Yes, bootstrapping with no system Git (or one too old for us to use) is supported, robust and tested in both the Homebrew (for macOS) installer and Homebrew/brew. I'm OK with no-op changes for Homebrew/install that make bootstrapping a little easier but for Homebrew/brew I think it makes everyone's life easier to just rely on the version of Git we want and need.

I believe the original reported bug (going off memory here) was that the parsing of the git version number fails with a sufficiently old system version of git, and the result is that HOMEBREW_FORCE_BREWED_GIT does not get set. Although I'm in favour of merging this PR, I'm okay with closing this PR and fixing that actual bug, the parsing of ancient git version numbers.

I'm also OK with closing this PR in favour of fixing the parsing actual git version numbers or whatever happens to be wrong.
I did some investigation last week off the back of the "this should already work". I'll go through the steps again tonight and write up my findings, then we can go from there to fixing the actual issue (presumably it's that Homebrew isn't using brewed git).


Original tutorials are gone

Any chance someone copied the referenced tutorials (404)?
Would love to read them.

@thejeff77
https://x2on.github.io/2010/02/01/tutorial-iphone-app-with-compiled-openssl-library/
https://x2on.github.io/2010/07/13/tutorial-iphone-app-with-compiled-openssl-1-0-0a-library/


[HUDI-1332] Introduce FlinkHoodieBloomIndex to hudi-flink-client

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
add bloom index for hudi-flink-clien
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[x] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

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

@Nieal-Yang Thanks for doing this, please assign this ticket(https://issues.apache.org/jira/browse/HUDI-1332) to yourself :)

@Nieal-Yang Thanks for doing this, please assign this ticket(https://issues.apache.org/jira/browse/HUDI-1332) to yourself :)
OK..  But I am not quite familiar with jira. Can you help to assign it to me.

@Nieal-Yang Thanks for submitting this PR! Please apply for contributor access for Hudi's Jira so I can assign the ticket to you.
Guide

Hi @Nieal-Yang , thanks for your contribution and welcome to the community!
I just took a quick pass on this PR. So the way of this implementation is similar to Spark, which is in a "batch mode". We will run a full cycle of workload for each commit and restart for the next one. Please let me know if I misunderstand.
I think we need the ability to run a Flink batch job for sure but probably we should distinguish batch from the current streaming job. WDYT?
yeah. You are right. I got your point.

Hi @garyli1019.  Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers.  WDYT?

Codecov Report

Merging #2375 (bf1c47f) into master (89f482e) will decrease coverage by 42.14%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #2375       +/-   ##
=============================================
- Coverage     52.19%   10.04%   -42.15%
+ Complexity     2659       48     -2611
=============================================
  Files           335       52      -283
  Lines         14981     1852    -13129
  Branches       1505      223     -1282
=============================================
- Hits           7819      186     -7633
+ Misses         6536     1653     -4883
+ Partials        626       13      -613

Flag
Coverage Δ
Complexity Δ

hudicli
?
?

hudiclient
?
?

hudicommon
?
?

hudihadoopmr
?
?

huditimelineservice
?
?

hudiutilities
10.04% <ø> (-59.62%)
0.00 <ø> (ø)

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

 Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers. WDYT?

@Nieal-Yang sorry about the late reply. IMO The batch mode will work but not fully taking the advantage of flink. But we can optimize this step by step. To ensure this PR will work, would you add some unit tests to this PR?

Hi @garyli1019. Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers. WDYT?

@Nieal-Yang sorry about the late reply. IMO The batch mode will work but not fully taking the advantage of flink. But we can optimize this step by step. To ensure this PR will work, would you add some unit tests to this PR?

Hi @garyli1019. Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers. WDYT?

@Nieal-Yang sorry about the late reply. IMO The batch mode will work but not fully taking the advantage of flink. But we can optimize this step by step. To ensure this PR will work, would you add some unit tests to this PR?

okay.  This index has been used in production in my company.  We will optimize it gradually

Hi @garyli1019. Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers. WDYT?

@Nieal-Yang sorry about the late reply. IMO The batch mode will work but not fully taking the advantage of flink. But we can optimize this step by step. To ensure this PR will work, would you add some unit tests to this PR?

okay.  This index has been used in production in my company.  We will optimize it gradually

Hi @garyli1019. Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers. WDYT?

@Nieal-Yang sorry about the late reply. IMO The batch mode will work but not fully taking the advantage of flink. But we can optimize this step by step. To ensure this PR will work, would you add some unit tests to this PR?

okay. This index has been used in production in my company. We will optimize it gradually

Hi @Nieal-Yang Would you please tell me what's the Flink version you are using?

Hi @garyli1019. Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers. WDYT?

@Nieal-Yang sorry about the late reply. IMO The batch mode will work but not fully taking the advantage of flink. But we can optimize this step by step. To ensure this PR will work, would you add some unit tests to this PR?

okay. This index has been used in production in my company. We will optimize it gradually

Hi @Nieal-Yang Would you please tell me what's the Flink version you are using?

Hi @garyli1019. Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers. WDYT?

@Nieal-Yang sorry about the late reply. IMO The batch mode will work but not fully taking the advantage of flink. But we can optimize this step by step. To ensure this PR will work, would you add some unit tests to this PR?

okay. This index has been used in production in my company. We will optimize it gradually

Hi @Nieal-Yang Would you please tell me what's the Flink version you are using?
@yanghua Yeah. The version we are using is flink-1.11.2.

Hi @garyli1019. Maybe I think the current implementation is OK. Beacause even in streaming job, we need to accumulate batch records in memory during the check-point cycle and upsert data into hudi-table when check-point triggers. WDYT?

@Nieal-Yang sorry about the late reply. IMO The batch mode will work but not fully taking the advantage of flink. But we can optimize this step by step. To ensure this PR will work, would you add some unit tests to this PR?

okay. This index has been used in production in my company. We will optimize it gradually

Hi @Nieal-Yang Would you please tell me what's the Flink version you are using?
@yanghua Yeah. The version we are using is flink-1.11.2.

@Nieal-Yang Thanks for your contribution! Left some comments. We are very close to land.
@wangxianghu could you take a pass as well? Thanks

Ack, will review soon

@Nieal-Yang Thanks for your contribution! Left some comments. We are very close to land.
@wangxianghu could you take a pass as well? Thanks

Ack, will review soon


Start page

As a user
I need some information what to do when opening the app
so that I know what I need to do next.
Description
The start screen contains information about the name of the app and first steps.
AC

[ ] Name of the app is visible
[ ] short instructions of first steps that disappear as soon as one entry exists

Tasks

[ ] extend header component
[ ] add text to start page
[ ] add logic that text disappears when one entry exists

Complexity
small

I would write in the Value Description:
"As a user, I want to see a informative home page with the possibilities of using, so that I can use the app intuitively."
all in all it looks really good and I think the idea behind it is nice ❤


[OSERV10] D.O. quota not taken into acount.

Context
Found when running Smoke Tests
Data Observatory analysis runs even when quota is not enough.
It also doesn't show any banner or quota warning when setting up the analysis parameters.
Steps to Reproduce

Set a quota in builder server:

[root@onpremise-smokes-12 ubuntu]# carto-builder-ds-quotas.sh 1 -o organization --do-general

... Currently working on < data observatory general > quota ...
[OK]: Successfully set < data observatory general > quota for organization

[root@onpremise-smokes-12 ubuntu]# carto-builder-ds-quotas.sh 1 -o organization --do-snapshot

... Currently working on < data observatory snapshot > quota ...
[OK]: Successfully set < data observatory snapshot > quota for organization

Quota is set to 1. Trying to run an analysis on a bigger table will succeed.

Current Result
It seems we're not checking quota for DO analyses.
Expected result
Analysis should fail if available quota < table rows

After checking with @ernesmb this is the expected behavior: we don't check the quota for data observatory

https://github.com/CartoDB/dataservices-api/blob/development/server/lib/python/cartodb_services/cartodb_services/metrics/quota.py#L146

Closing


Converting text with hyperlinks to bullets loses hyperlinks

Issue is opened on EditorJs side https://github.com/codex-team/editor.js/issues/1676


Fix network verifier success log message for CHGM

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (42600a2) 39.19% compared to head (1be5c31) 39.12%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #253      +/-   ##
==========================================
- Coverage   39.19%   39.12%   -0.07%
==========================================
  Files          16       16
  Lines        1457     1457
==========================================
- Hits          571      570       -1
- Misses        847      848       +1
  Partials       39       39

Files
Coverage Δ

pkg/investigations/chgm/chgm.go
66.15% <0.00%> (-0.52%)
:arrow_down:


Introduce flag that allows listening on an arbitrary interface.

Care is taken that the old behavior which uses the --port flag is
preserved. If the --port flag does not have the default, empty
value, then its value is used for the new --httpaddr flag, prefixed
with <IP_ADDRESS>:.
(Strictly speaking, if localhost does not resolve to <IP_ADDRESS>,
then the behavior is not identical.)
Fixes gogo/letmegrpc#15.

If you find you need more customization power please see
https://github.com/gogo/letmegrpc#customization
But this change seems small enough

This seemed like one of those things one would want to have in the 'default' tool. :-)

cool :)


Call invitation when phone is locked

When the phone is locked and I receive a call invitation I need to insert my lock screen code which is not the best user experience.

I'm not sure that a third party application could be allowed to bypass the lock screen for security reason.
It seems having worked on android 4X but since android 5.0, it seems forbidden.

At least for whatsapp this is working... hmmm...

and for hangouts also...

And for Signal

ConnectionService

An abstract service that should be implemented by any apps which either:

Can make phone calls (VoIP or otherwise) and want those calls to be integrated into the built-in phone app. Referred to as a system managed ConnectionService.
Are a standalone calling app and don't want their calls to be integrated into the built-in phone app. Referred to as a self managed ConnectionService.

and for twilio


Add support for dap to debug tests

Allows debug tests to be invoked through neotest.
I've tested locally and it works as expected with nearest test, but it would also be worth testing with a minimal file. If you want me to add some tests for this I'm happy to 😄
Thanks for a great package!

I didn't notice until this morning that it wasn't 😄 Was surprisingly simple to setup in the end, though would be nice if upstream neotest had better adpater docs

Completely agree. I think all of us adapter contributors have just learned from one another's efforts.


tests: bluetooth: tester Add support for OOB upload tests in Fw Distribution server

This PR adds support for the following PTS test:

DFU/SR/FD/BV-06-C
DFU/SR/FD/BV-13-C
DFU/SR/FD/BV-14-C
DFU/SR/FD/BV-15-C
DFU/SR/FD/BV-16-C
DFU/SR/FD/BV-17-C
DFU/SR/FD/BV-18-C
DFU/SR/FD/BV-22-C

The PR also fixes bugs related to OOB upload.

@PavelVPV is this something you think should be merged for 3.5? If so, please set the target milestone appropriately.

@PavelVPV is this something you think should be merged for 3.5? If so, please set the target milestone appropriately.

Added milestone (hope it is not too late).

Is this fixing any bugs? I don't see any "Fixes: ..." entry.

@PavelVPV is this something you think should be merged for 3.5? If so, please set the target milestone appropriately.

Added milestone (hope it is not too late).

Is this fixing any bugs? I don't see any "Fixes: ..." entry.

Sorry, added now.


more tests

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

Comparison is base (ab6755e) 57.04% compared to head (538a438) 57.38%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
+ Coverage   57.04%   57.38%   +0.34%
==========================================
  Files          13       13
  Lines        1157     1157
==========================================
+ Hits          660      664       +4
+ Misses        393      387       -6
- Partials      104      106       +2

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

I'm trying to broadly expand coverage. A lot of the missed lines are error checks, but some large chunks that aren't simple error checks:

impl/pkg/service/pkarr.go (codecov) has a large chunk that's untested. A lot of it will need some DHT stuff to function, which I'm evaluating how to do. One issue I'm running into is that the call to s.db.ReadRecord on line 123 (at time of writing, obviously that may move) seems to be returning either an error or a nil record, which results in a good chunk of code not being covered.
impl/pkg/server/util.go (codecov) has a validator that registers some kind of tag validation callback, but doesn't seem to be used anywhere. Is it safe to remove?

codecov seems to not have information about a lot of the files, for example everything in pkg/storage/db/postgres. I'm assuming this means we have to have a test in that subpackage to get coverage data, despite pkg/storage/storage.go (codecov) definitely taking the postgres path in the reported test coverage.

line go up (marginally)

Several subpackages are currently missing tests entirely, most of them seem fine to not test, but the internal/ stuff would probably be good to get.
?       github.com/TBD54566975/did-dht-method/cmd/cli   [no test files]
?       github.com/TBD54566975/did-dht-method/concurrencytest   [no test files]
?       github.com/TBD54566975/did-dht-method/config    [no test files]
?       github.com/TBD54566975/did-dht-method/internal  [no test files]
?       github.com/TBD54566975/did-dht-method/internal/cli      [no test files]
?       github.com/TBD54566975/did-dht-method/internal/dht      [no test files]
?       github.com/TBD54566975/did-dht-method/internal/util     [no test files]


Fix download icon in material theme, fixes #83

Fixes download icon for material theme, #83
Demo:

Hee, thanks for letting me know!
I would like to reproduce this issue locally, but both mkdocs build and mkdocs serve work fine for me. Perhaps you have a minimal reproducable example? Or even better, an alternative solution?
Tim

Hee, thanks for letting me know!
I would like to reproduce this issue locally, but both mkdocs build and mkdocs serve work fine for me. Perhaps you have a minimal reproducable example? Or even better, an alternative solution?
Tim

Hey Tim,
as mentioned mkdocs build works like a charm. When i start mkdocs serve it initially looks fine, but the error occurs, when i try to navigate  in the dev server site.
Cheers

Can confirm @rmueck.
Same exception "Tried to write more data than Content-Lenght"

Thanks @rsansores .
I don't have time to work on this currently, hope to revisit in a couple of weeks or so. Feel free to pick up the PR :)

Hello! Any update on this PR? Does it work with latest version of mkdocs-material? (5.3.0)

Thanks @Kanaduchi for the pointer! Quite sure that will fix the HTTPOutputError.
I also fixed the icon size, which was wrong. If someone can confirm that it works for them as well, we can flag this as ready for merge to zhaoterryy.

Thanks @Kanaduchi for the pointer! Quite sure that will fix the HTTPOutputError.
I also fixed the icon size, which was wrong. If someone can confirm that it works for them as well, we can flag this as ready for merge to zhaoterryy.

I verified it locally.
One issue that I see is:

I think you also need to remove a.string = '\uE2C4'

? I think you also need to remove a.string = '\uE2C4'
That line was already removed. I now also removed any unnecessary attributes from the svg.
Renders correctly on chrome and safari, and latest mkdocs-material version:

No it looks great! Thank you! Hope this PR will be merged soon

@zhaoterryy ready for merge 😃

I found & fixed another bug, present on master and also in this fix-download-icon branch
soup = BeautifulSoup(html, 'html.parser') actually does more than just parse the HTML, it also applies some 'fixes'. This leads to unintended side effects (other than inserting the button). For example, when used in combination with mknotebooks BeautifulSoup will break the HTML and the jupyter notebooks don't render correctly (on some browsers / platforms). Took a while to find that bug, but solution was easy: insert the button using basic python string manipulation.

@timvink cheers!


Drop ansible-playbook and template module from AnsibleResourceModelSource

Investigate switching to ansible all -m setup -t /tmp/....

Is running the setup necessary? If not, you could use a short python script that uses Ansible as library to load the inventory and dump it in a usable format. That will only give you the data available straight from the inventory, but you won't need to connect to nodes when loading the list of them.

There already is an option to disable fact gathering on node discovery, see the playbook I use for this. Enabling the fact_cache (as noted in #8) should return cached facts for the hosts when gathering is disabled.
Sounds like the playbook already does what your python script would do.
I thought about this some time ago and concluded I couldn't use the command in the OP, so closing this - feel free to comment nonetheless here or in #8.


Added CentOS 6.x support.

This has been tested on CentOS 6.5 and Ubuntu 14.
Refer to https://github.com/puppetlabs/facter/blob/master/lib/facter/operatingsystem/linux.rb if additional operating systems need to be supported or more specific information about the operating system is required.

@bruno- Any updates on this? That's really nice to have this for CentOs.

Hey,
sorry, no I haven't had the time to try this out yet.

I'm working on centos with rails and try to use this gem, however, i found out gem doesn't work on centos and this pr also.
is here any updates, or is there any feature to add more?

Hi @ereichert. Could you please rebase onto master and squash your commits? How well is this tested on CentOS and Ubuntu?

I'll see what I can do some time this week.  Regarding testing, I use it weekly on CentOS 6.5 and have never used it on Ubuntu that I recall.  There is an issue with restarting unicorn after a deployment but it's not become enough of a nuisance that I'm going to do anything about it.

@rhomeister I think this is ready to go.

I am insterested in this PR, but also saw https://github.com/capistrano-plugins/capistrano-unicorn-nginx/compare/master...erwald:38f9e9c which seems to work also with Amazon Linux.
As far as I can tell, this PR doesnt work on Amazon Linux (neither lsb_lease or /etc/redhat-release exist on the Amazon Linux I'm testing)
Anybody having experience with this PR?


Pymbar 4.0

Preparing for the big merge of pymbar 4 back into master.

Just lint and the issue with unpopulated histograms, which #447 will solve.
So the plan would be to merge #447, and then merge this into master, then tag it.

@mikemhenry, @Lnaden, are we ready to make the jump?

Good on my end.

LGTM!


Migration: Irmin.Contents.String_v2 implications for git

Hello,
I'm porting some code from irmin >= 2.10.o & < 3.0.0 to irmin >= 3.0.0. The code fetches a remote git repository. In #1715 a change was made that among other things added Irmin.Contents.String_v2. I am curious what consequences there are (not) having compatibility with older irmin versions with regard to git.
  module String : S with type t = string
  (** Contents of type [string], with the {{!Irmin.Merge.default} default} 3-way
      merge strategy: assume that update operations are idempotent and conflict
      iff values are modified concurrently. *)

  module String_v2 : S with type t = string
  (** Similar to [String] above, but the hash computation is compatible with
      versions older than irmin.3.0 *)

Thank you.

For some more context the code in question is in https://github.com/roburio/unipi/pull/9.

For the Git backend, that shouldn't make a difference - the backend is overwriting the pre_hash function to look like Git, so both String and String_v2 object will have the same hash.


Removing deprecated overwrite option and associated docs and variable

what

Remove the overwrite option on the SSM parameters

why

The overwrite option has been deprecated

references

https://github.com/hashicorp/terraform-provider-aws/issues/25636

@max-lobur I know you weren't tagged in this, but you seem to be a recent committer and GH says you work for CloudPosse. Can this get reviewed? Or possible to get whatever is the right team member(s) to review it?

/terratest

@hans-d I got the updates added to fix the CI checks! So ready for new testing whenever.

Hi @bengaywins
I discussed with other maintainers and we're going to hold on this one as the current situation with this deprecation does not appear to be finalised. Please see https://github.com/hashicorp/terraform-provider-aws/issues/25636#issuecomment-1623661159 for further info.
Probably once v6 of the AWS provider is released, we'll know how to proceed.


[Metrics] Measure all the things!

Now that we have prometheus wired up, let's expand the things we're measuring:

[ ] counters for all errors (although maybe Fatal's are less useful because Prometheus is pull-based and will probably not retrieve it in time)
[ ] latency histogram for Get's to the config database
[ ] latency histogram for Put's to the config database

Prometheus metrics were refactored recently:
https://github.com/serverless/event-gateway/pull/379
https://github.com/serverless/event-gateway/pull/384
I'm closing this for now.


add new row/save

I thought long and hard about asking this after scanning over the UI a bunch, but is it not possible to edit values directly within a row? When I click on a row, it's editable, but there's nowhere in the UI to save. I also don't see an option to insert a row in the table view.
I know I can do both of these by executing a query, but just wanted to confirm that I'm not crazy :)
Thanks!

You're not crazy, these types of features (editing, etc) are not part of the pgweb yet. I've been experimenting with a few options but havent committed to anything just yet.

@sosedoff - thanks, much appreciated!
Overall, loving pgweb! Appreciate your hard work.
Cheers!


No hay nada

trabajen

XD

@all-contributors please add @DexSlender for code, ideas & planning and tests

@all-contributors
please add @Hyduez for style and maintenance
please add @jesus-ale43 for code, desing and tests

@all-contributors please add @jesus-ale43 for code, design and test

@all-contributors please add @jesus-ale43 for code, design and tests.

idc

@all-contributors please add @jesus-ale43 for code, design and tests.

NOT AGAIN PLEASE 😭💀💯💯💯

ya trabajamos


document jupytext dependency, add configuration file

The HARK library's dependence on jupytext for development of the examples is not documented anywhere.
It also looks like there's a way to support it through a configuration file, though we have not included anything like this in the library.
https://github.com/mwouts/jupytext/blob/master/docs/config.md
This is a serious omission if we're going to start testing the examples/ as part of the library, because tacit differences in how people are using jupytext could lead to inconsistencies.

It looks like there is also a way to do syncing with a pre-commit hook:
https://jupytext.readthedocs.io/en/latest/using-pre-commit.html
I think it's better to do configuration and enforcement in a way that can be stored in version control and executed in the development environment. I don't think automated code changes are a good way to go.
I think that if we continue to require use of jupytext in any way, we need to be very specific of how we intend people to use it. I'll admit I did not know about the --sync feature until Mridul mentioned it just the other day; I had been using --to. The Jupytext connection to Notebooks and Labs is different, so there's lots of room for user error. I can't say with confidence that I have the necessary competence to use jupytext and jupyter lab, and I've been managing with the two together now for a year.
I don't like the idea of storing .py files in the repository and rendering them elsewhere. I think it's much more common to have Notebooks in repositories without a .py file or jupytext complicating things. But I don't feel very strongly about that. I
am generally skeptical about workflows that involve rendering from one repository to another. I haven't seen much compelling success with that approach.

Another thing to consider: the examples notebooks in the repository currently get picked up by Sphinx and rendered into ReadTheDocs.
It is possible that with yet another layer of automation, we could get ReadTheDocs to render .py files into notebooks and then those notebooks into RTD. But I don't at this time know how to do that and it would be an additional possible point of failure.
The current automated test for the DemARKs tests the notebooks, not the .py files. A different way to do automated testing of the HARK examples, which is what I assumed #944 was doing, was to whip through the notebooks to make sure they don't fail to execute.

examples, which is what I assumed

Another thing to consider: the examples notebooks in the repository currently get picked up by Sphinx and rendered into ReadTheDocs.  It is possible that with yet another layer of automation, we could get ReadTheDocs to render .py files into notebooks and then those notebooks into RTD. But I don't at this time know how to do that and it would be an additional possible point of failure.

Ah, I hadn't realized that.  Am sympathetic to not wanting to create another point of failure.  But am tired of dealing with complexities of dealing with jupytext sync.  (Esp. last two days; it uses timestamps to decide whether .py and .ipynb files are in sync, but very frequently the timestamps are off even though the two files ARE in sync).

The current automated test for the DemARKs tests the notebooks, not the .py files. A different way to do automated testing of the HARK examples, which is what I assumed #944 was doing, was to whip through the notebooks to make sure they don't fail to execute.

I think that's what we ARE doing -- are we not using pytest --nbval-lax for the notebooks in examples, as we are for the notebooks in DemARK?  What are we doing instead for the notebooks in examples?


Add simple JSP project for beginners

Hello,
Today I will add a very simple JSP and servlets project for beginners
I hope this helps other developers.
I will open a new PR
Thanks.

PR -> #64

PR -> #64


FFT Radix 2 implementation

I will add a DFT implementation later.

Merging to make some movement on this and get the DFT in.


Update typo binary sennsor in init.py

fix typo from warnung to warning

Please keep in mind: This is a breaking change for people which are using this binary sensor at the YAML configuration.
You have to migrate from
binary_sensor:
  - platform: pipsolar
    pipsolar_id: inverter0
    warnung_low_pv_energy:
      name: "${name} warnung low pv energy"

to
binary_sensor:
  - platform: pipsolar
    pipsolar_id: inverter0
    warning_low_pv_energy:
      name: "${name} warning low pv energy"


Drag and drop in IE

Jodit Version:  3.1.84
Using IE11
Expected behavior:
Dragging text from outside into the editor, copy the text, and dragging within should move it.
Actual behavior:
No text is created/moved, but carat ends up at where mouse is dropped.

Once again thanks for the fast fix, one thing though when dragging text around inside the editor it copies it instead of moving it.


Keyboard input doesn't follow active window, when quickly swith between two workspaces back and forth.

Hyprland Version
 System/Version info
❯ hyprctl systeminfo
Hyprland, built from branch  at commit fe7b748eb668136dd0558b7c8279bfcd7ab4d759  (props: bump version to 0.39.1).
Date: Tue Apr 16 16:01:03 2024
Tag: v0.39.1, commits: 4460

flags: (if any)

System Information:
System name: Linux
Node name: Saco-Archlinux
Release: 6.6.28-1-lts
Version: #1 SMP PREEMPT_DYNAMIC Wed, 17 Apr 2024 10:11:09 +0000

GPU information:
0b:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Raphael [1002:164e] (rev c1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Bug
Description
Occasionally, when I navigate between workspaces, my keyboard inputs do not follow me but remain active in the last workspace I left. This issue occurs most frequently when I switch from a workspace containing a browser to one with a
terminal. Sometimes, the problem even arises within the same workspace when I switch between windows.
How to reproduce
Suppose I'm working with two workspaces( abbreviated as ws in this text).
In ws-1, I have a browser open, and in ws-2, there's a terminal. Currently, I'm in ws-2.
If I quickly switch to ws-1 and then immediately  back to ws-2, all my keyboard inputs are mistakenly sent to the browser in ws-1. To resolve this and redirect my inputs back to the terminal in ws-2, I am forced to switch again to ws-1 and then back to ws-2, only then does the cursor respond in the terminal as expected.

p.s. The issue does not occur when the focus is on an input box in the browser.

Crash reports, logs, images, videos
No response

Here's another issue that I believe may be related.
Previously, I used the command c_w=$(hyprctl activeworkspace -j | jq '.id') && hyprctl dispatch workspace 10 && swaylock && hyprctl dispatch workspace $c_w to lock my screen. This command was supposed to run swaylock in workspace 10 and wait for my input.
However, after an update last week, the command no longer responds to any input post-execution. To resolve this, I modified the command to include a sleep 1 delay, changing it to c_w=$(hyprctl activeworkspace -j | jq '.id') && hyprctl dispatch workspace 10 && sleep 1 && swaylock && hyprctl dispatch workspace $c_w.
With this adjustment, the command functions as it did before the update.

I hava the same question after i reinstall Hyprland yesterday

I have the same problem. for example ,I move my mouse from firefox to kitty, I can see the cursor in kitty is active, but i can't type anything , when i press Crtl + A, I can see the firefox act as select all.If i use shortcut like super + right, it works nicely

I fixed it by install hyprland-git, leaste version, but my waybar and pyprland don't work

I seem to be experiencing a similar issue in my environment.
Initially, I thought it might be a problem with IM (Fcitx or Mozc), but it appears to indeed be an issue with the active window.
I'm leaving this comment so that others who might think it's a Fcitx issue can find this issue through search.

#5763 It is already fixed in git.

Downgrade fcitx5

Thank you so much!!
I was very troubled by the same problem. The workaround solves it!


Update UML to match new model classes and controllers

Updated it to reflect the new controllers and managers.

Still need to figure out which of the new activities should be included. We also need to find what model classes need updating.


Bug with embetty

I installed an embetty-server and implement this into Grav.
now i create a new site and insert different tweets and some text.
The text above the first tweet will shown but nothing below.
i can't also find some of my text in the source code of the site.
Howto reproduce:
install a embetty server (or deploy a docker server)
insert the "header" tags
create a new site and enter follow content:
some text
<embetty-tweet status="950371792874557440"></embetty-tweet>
more text
<embetty-video type="youtube" video-id="3L4fHrIJ3A4"></embetty-video>
more more text
<embetty-video type="vimeo" video-id="223099532"></embetty-video>
more, more more text
<embetty-video type="facebook" video-id="10156049485672318"></embetty-video>
more text, just for debugging

the source code of the rendered website looks like as if it had been cut off below the first tweet:

--

   Huggenknubbels Blog

   Impressum

some text

   <embetty-tweet status="950371792874557440"></embetty-tweet>

Grav was  with  by RocketTheme.

   Huggenknubbels Blog

   Impressum

Is this really a Grav bug?  Grav is file based so if embetty can't output the correct file structure then it's not going to work.  I don't see what we could possible do here that we are not already doing?

yes i think so beceause if it wasn't a Grav Bug i got the emetty tags in
the source code?
Am Do., 31. Mai 2018 um 22:47 Uhr schrieb Andy Miller <
<EMAIL_ADDRESS>
Is this really a Grav bug? Grav is file based so if embetty can't output
the correct file structure then it's not going to work. I don't see what we
could possible do here that we are not already doing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/getgrav/grav/issues/2042#issuecomment-393674462, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AO7hcL3FJKAZ6LVVFSt4CjnE-gm3kubnks5t4FbygaJpZM4UTRoS
.


1.5.1 change log

Could someone please update the releases changelog and update the other change log and provide a more user friendly change log with a high level overview of what all changed and any breaking changes that occurred.

Yes. We are in a process of writing everything down. @mgonto check PM

Thank you
On 4/8/15, Mike Grabowski<EMAIL_ADDRESS>wrote:

Yes. We are in a process of writing everything down. @mgonto check PM

Reply to this email directly or view it on GitHub:
https://github.com/mgonto/restangular/issues/1116#issuecomment-91009448

--
Thanks
-Blake Niemyjski


Remove reliance on rsync

Right now we require rsync on the remote server.

How else would this work?

Not sure, just would be nice to have no deps by default. Maybe we use one of the Go rsync libs?

That seems reasonable.. we don't need the advanced features of rsync that necessitate a remote binary.
Curious about how slow it woukd be to read all those remote modtimes/file sizes over SSH .

What if instead of rsync, we just mounted the local extensions folder?

Avg folder is 300MB and VS Code does a full scan of it on startup

In that case, I think requiring rsync to be installed for syncing is fair. Users who do not have it can use --skipsync and we can clarify the error message to indicate that.

FYI: Even with -skipsync, the connection will fail if the server does not have rsync.

@Belval can you post the error you're getting? I just looked at the code again, rsync is never executed on the remote site if -skipsync is passed.

I don't have my laptop right now, but it was something along the lines of rsync not found.

Fixed.

Should be able to integrate mutagen here at some point to avoid using rsync.
See https://github.com/mutagen-io/mutagen/blob/cd6f4e1a5e68979668abfb22174c32d877e46000/cmd/mutagen/daemon/main.go

@ammario I know you recently worked with it so how was mutagen's API?

Didn't interface directly with Mutagen's API, but the architecture requires that the user has all the mutagen agents locally (it uploads the mutagen agent as the first step in the sync) which introduces its own complexity.
cc @havoc-io correct me if wrong.

Oaky so I want to try to work on this concept. But can someone explain
exactly what the frack "mutagen" is in this context?
On Mon, May 4, 2020, 7:20 AM Ammar Bandukwala<EMAIL_ADDRESS>wrote:

Didn't interface directly with Mutagen's API, but the architecture
requires that the user has all the mutagen agents locally (it uploads the
mutagen agent as the first step in the sync) which introduces its own
complexity.
cc @havoc-io https://github.com/havoc-io correct me if wrong.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cdr/sshcode/issues/11#issuecomment-623493013, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPQOXR7BLDCT2OMZZK5ZZ3RP3FJ7ANCNFSM4HHIT2CQ
.

@Merith-TK https://mutagen.io/ ... And please don't work on anything without an in depth design consensus with @nhooyr. We want to make sure you're as productive as possible.

Don't worry, almost all of my code goes through a rough draft stage where I
make it work. And a cleanup stage where I make it make bloody sense and
follow what is actually needed.
On Mon, May 4, 2020, 8:40 AM Ammar Bandukwala<EMAIL_ADDRESS>wrote:

@Merith-TK https://github.com/Merith-TK https://mutagen.io/ ... And
please don't work on anything without an in depth design consensus with
@nhooyr https://github.com/nhooyr. We want to make sure you're as
productive as possible.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cdr/sshcode/issues/11#issuecomment-623539904, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPQOXWHWFOMDQTJROJUZZLRP3OXRANCNFSM4HHIT2CQ
.

@ammario In theory, you can avoid needing Mutagen's agent binaries locally if one is preinstalled at the expected location in the target ($HOME/.mutagen/agents/x.y.z/mutagen-agent, where x.y.z is the Mutagen client version).  Mutagen will try to execute an existing agent first before injecting one, so you could potentially avoid the injection step and the agent bundle dependency.  But Mutagen currently expects an exact version match because the internal protocol isn't stable across versions1, so it will attempt to inject a matching version if none is present.  If you're going to vendor/version-lock the Mutagen version that you ship, you might be able to use this strategy to skip the injection step.
If you do need to carry around the agent bundle locally, you can slim it down to only include the required agents (probably just linux/amd64 in your case).  The archive just contains a bunch of binaries named <GOOS>_<GOARCH>, so a simple script can prune these and save a lot of space.

1 Actually it is stable across minor release series, but a full version match, including the patch component and any -tag suffix is still required. This may change as the protocol further stabilizes.

Maybe the mutagen client/server stuff can be acquired in the same way as
how code-server binaries are aquired?
On Mon, May 4, 2020, 9:17 AM Jacob Howard<EMAIL_ADDRESS>wrote:

@ammario https://github.com/ammario In theory, you can avoid needing
Mutagen's agent binaries locally if one is preinstalled at the expected
location in the target ($HOME/.mutagen/agents/x.y.z/mutagen-agent, where
x.y.z is the Mutagen client version). Mutagen will try to execute an
existing agent first before injecting one, so you could potentially avoid
the injection step and the agent bundle dependency. But Mutagen currently
expects an exact version match because the internal protocol isn't stable
across versions1, so it will attempt to inject a matching version if none
is present. If you're going to vendor/version-lock the Mutagen version that
you ship, you might be able to use this strategy to skip the injection step.
If you do need to carry around the agent bundle locally, you can slim it
down to only include the required agents (probably just linux/amd64 in your
case). The archive just contains a bunch of binaries named _,
so a simple script can prune these and save a lot of space.
1 Actually it is stable across minor release series, but a full version
match, including the patch component and any -tag suffix is still
required. This may change as the protocol further stabilizes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cdr/sshcode/issues/11#issuecomment-623561270, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPQOXR2XN4OPOHJXTNC2GTRP3TCBANCNFSM4HHIT2CQ
.

also in theory, incorperating mutagen should be as easy as replacing rsync function


RUMM-2898 Add hasReplay=true only when the view produced SR records

What does this PR do?
We were adding the has_replay:true property on the RUM events whenever the SR is recording no matter if the view in progress produced any records or not. Following the latest discussions we needed to fix this and set this property to true only if there was at least one SR record sent for this event.
Motivation
What inspired you to submit this pull request?
Additional Notes
There is still a corner case for which this flag can have wrong values due to the async way of intercommunication between our features but this was explained in the PR comments and for now we are accepting this as a non blocker issue.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1234 (9894418) into feature/sdkv2 (7b37814) will decrease coverage by 0.01%.
The diff coverage is 96.00%.

@@                Coverage Diff                @@
##           feature/sdkv2    #1234      +/-   ##
=================================================
- Coverage          82.75%   82.74%   -0.01%
=================================================
  Files                358      358
  Lines              12623    12628       +5
  Branches            2098     2101       +3
=================================================
+ Hits               10445    10448       +3
+ Misses              1528     1525       -3
- Partials             650      655       +5

Impacted Files
Coverage Δ

...oid/sessionreplay/internal/SessionReplayFeature.kt
96.43% <ø> (ø)

...atadog/android/sessionreplay/NoOpRecordCallback.kt
0.00% <ø> (ø)

.../android/rum/internal/domain/scope/RumViewScope.kt
95.04% <85.71%> (-0.32%)
:arrow_down:

...og/android/rum/internal/FeaturesContextResolver.kt
100.00% <100.00%> (ø)

...ndroid/rum/internal/domain/scope/RumActionScope.kt
97.42% <100.00%> (+0.03%)
:arrow_up:

...roid/rum/internal/domain/scope/RumResourceScope.kt
95.39% <100.00%> (+0.09%)
:arrow_up:

...sionreplay/internal/SessionReplayRecordCallback.kt
100.00% <100.00%> (ø)

...id/sessionreplay/SessionReplayLifecycleCallback.kt
87.10% <100.00%> (ø)

...d/sessionreplay/processor/RecordedDataProcessor.kt
94.00% <100.00%> (+0.12%)
:arrow_up:

...g/android/sessionreplay/recorder/ScreenRecorder.kt
97.50% <100.00%> (+2.05%)
:arrow_up:

... and 12 more


[FEATURE] Improve Nav Icons

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
The current navigation icons are very simple and light. You need to make some modifications on the Nav UI to make it attractive.
Proposed Solution

Use some vibrant colors of icons that will make the Navigation really attractive
The Icons should change based on active window.

When the Home is Active, Home Icon in the Navbar should be kinda different than when it is Not Active
Use bubble-effect on click on the icons in the Nav, to ensure a overwhelming user-experience.

Screenshots

Do you want to work on this issue?
No
If "yes" to above, please explain how you would technically implement this
No response

@pooranjoyb Please assign this issue to me.

@pooranjoyb Please assign this issue to me.

Please finish up the ones assigned.

@pooranjoyb Can you please assign this issue to me?
I have noticed one more issue like theme is not consistent.
When user select a dark theme at home page, and then go to about page, or chat page, that time dark theme is not getting applied there.


How to get all styles of the current element

Is there a method to get an array of all the active styles of the text that is focussed? (e.g. bold, italic, text color, font, etc.)

i'm not sure, but u can try use it.
ChildComponent
let editor = Jodit.make(element, options)
window.editorInstance = editor

ParentComponent
window.editorInstance.selection.focus()
...etc

ref https://xdsoft.net/jodit/doc/methods/

let node = editor.selection.current();
if (!Jodit.modules.Dom.isElement(node)) {
node = node.parentElement;
}
console.log(window.getComputedStyle(node));


Route Validation fails with multiple routes containing a [parameter] in the same folder

Environment

Operating System: Linux
Node Version:     v16.14.2
Nuxt Version:     3.3.1

Reproduction
https://stackblitz.com/edit/nuxt-starter-x96hzs?file=package.json
Describe the bug
If you create two directories with parameter in /pages that contain definePageMeta with validate function, at least one of them doesn't work as intended.
https://nuxt.com/docs/getting-started/routing#route-validation

The validate property accepts the route as an argument. You can return a boolean value to determine whether or not this is a valid route to be rendered with this page. If you return false, and another match can't be found, this will cause a 404 error.

From this, you could assume, that:

if validate returns true, the route is matched.
If you return false, return 404 and stop any subsequent matching.

If I create two routes like:

/pages/[posts]/index.vue
/pages/[projects]/index.vue

And avoid returning false, by using /pages/[posts]/index.vue:
definePageMeta({
  validate: (route) => {
    const { posts } = route.params;
    if (posts === 'posts') {
      return true;
    }
  },
});

and /pages/[projects]/index.vue
definePageMeta({
  validate: (route) => {
    const { projects } = route.params;
    if (projects === 'projects') {
      return true;
    }
  },
});

I should have two working routes at /posts and /projects. Unfortunately, one of the routes works, while other returns 404.
Additional context
No response
Logs
No response

This is vue-router behaviour - it will only match one of the pages generated: see https://paths.esm.dev/?p=AAMsIPQg4AIMpkjtgAsERhMwKIBACQgoSWADZB4wvcIA8g..&t=/test/thing#. You can alternatively customise the path using route matching syntax like this within the page:
<script setup lang="ts">
definePageMeta({
  path: '/test/:bar(baz|qux)'
})
</script>

This will only work for patterns, not for more complex validations that rely on async data fetching. For that, you might have to use app/router.options or another solution.


Merge pull request #1 from RasaHQ/master

UPdate2
Proposed changes:

...

Status (please check what you already did):

[ ] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog

 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.

sfsf

Mhm this doesn't seem to be a proper PR - let me know if you fixed that or need any help.


News view is broken

@levlaz check pull request #144 . I just added a class id to stop overflow

Fixed in https://github.com/levlaz/braindump/pull/144


[Feature] Running end-to-end tests on local machine

Why are these changes needed?
 Without this, the command kind load docker-image kuberay/apiserver:nightly will report an error message as follows.
ERROR: image: "kuberay/apiserver:nightly" not present locally

In addition, we also use kind load command to preload the rayproject/ray image into KinD cluster. Although the Kubernetes cluster will pull the ray image automatically when we create a custom resource RayCluster, pulling rayproject/ray:1.9.0 (2.44GB) may cause ImagePullBackOff (Link) due to timeout. In addition, the RayCluster startup time will vary based on network speed and make the tests flaky.
Related issue number
Closes #590

Checks
cd /path/to/your/kuberay
python3 tests/compatibility-test.py

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

I just realized we may need to separate these functions into different files. that will be easier for our future updates. I posted another patch https://github.com/ray-project/kuberay/pull/591 and this should  make our changes to ci tests more convenient.

I just realized we may need to separate these functions into different files. that will be easier for our future updates. I posted another patch #591 and this should make our changes to ci tests more convenient.

Thank you for this patch! Feel free to ping me for review when it passes CI/CD tests.

I found an interesting bug when I tried to use argparse to simplify the logic of parse_environments() (See https://github.com/ray-project/kuberay/pull/589/commits/6751bc93c1dce70d0d971d8f9755c95ca325c99c for more details.). The following simple scripts can reproduce the bugs.

test1.py
 import argparse
 import unittest

 if __name__ == '__main__':
     parser = argparse.ArgumentParser()
     parser.add_argument('--input', '-i', dest='input',
                         required=False, default='Hello KubeRay')
     args = parser.parse_args()
     print(args.input)
     unittest.main()

test2.py
 import argparse
 import unittest

 if __name__ == '__main__':
     parser = argparse.ArgumentParser()
     parser.add_argument('--input', '-i', dest='input',
                         required=False, default='Hello KubeRay')
     args = parser.parse_args()
     print(args.input)

Experiment1: python3 test1.py --input haha
$ python3 test.py --input haha
haha
usage: test.py [-h] [-v] [-q] [--locals] [-f] [-c] [-b] [-k TESTNAMEPATTERNS]
              [tests [tests ...]]
test.py: error: unrecognized arguments: --input
(base)

Experiment2: python3 test2.py --input haha
$ python3 test.py --input haha
haha
(base)

The error message "unrecognized arguments: --input" in Experiment1 is reported by unittest.main(). This is because unittest.main() implicitly read sys.argv (See Link for more details.)


Wrong number of packages reported for some package managers

Description
Fixed bug of wrong count of packages installed by some package managers (tested on scoop and snap). The fix is in branch packagenum
Issues
Fixes: #1274

Nicely done.


Add links to Flex and Bison to README.md

Crazy small change, while working with this repo I shared it with a couple of people and they had trouble finding info about Flex and Bison, so thought this might as well be added to make things a little easier.

Thanks!


Field handler is trigger even if the field does not exist in the resource at the time.

Long story short

Field handler is trigger even if the field doesn't exist in resource
Description
When we create a customer resource the create handler is correctly triggered but the a field handler is also triggered for fields that don't exist in the resource when it was created.
Not sure if this is a bug in 0.27 or a feature but it is a change in functionality from previous versions.
The code snippet to reproduce the issue
@kopf.on.create('company.com', 'v1', 'testresources')
def create_handler(spec, **_):
    pass

@kopf.on.field('company.com', 'v1', 'testresources', field='spec.updateField')
def updateField_changed(old, new, **_):
    pass

#Custom Resource that was created
apiVersion: company.com/v1
kind: TestResource
metadata:
  name: test
spec:
  createField: true

The exact command to reproduce the issue
kopf run --standalone test-handler.py

The full output of the command that failed
[2020-06-13 20:13:16,055] kopf.activities.auth [INFO    ] Activity 'login_via_client' succeeded.
[2020-06-13 20:13:16,055] kopf.reactor.activit [INFO    ] Initial authentication has finished.
[2020-06-13 20:13:29,639] kopf.objects         [INFO    ] [default/test] Handler 'create_handler' succeeded.
[2020-06-13 20:13:29,760] kopf.objects         [INFO    ] [default/test] Handler 'updateField_changed/spec.updateField' succeeded.
[2020-06-13 20:13:29,760] kopf.objects         [INFO    ] [default/test] All handlers succeeded for creation.

Environment

Kopf version: 0.27
Kubernetes version: minikube v0.35.0 on darwin (amd64) & v.1.11.9
Python version: 3.8.0
OS/platform: macos & coreos

Python packages installed

aiohttp==3.6.2
aiojobs==0.2.2
async-timeout==3.0.1
attrs==19.3.0
cachetools==4.1.0
certifi==2<IP_ADDRESS>
cffi==1.14.0
chardet==3.0.4
click==7.1.2
cryptography==2.9.2
google-auth==1.17.2
idna==2.9
iso8601==0.1.12
Jinja2==2.11.2
kopf==0.26
kubernetes==11.0.0
MarkupSafe==1.1.1
multidict==4.7.6
oauthlib==3.1.0
pip==19.3.1
pyasn1==0.4.8
pyasn1-modules==0.2.8
pycparser==2.20
pycrypto==2.6.1
pykube-ng==20.5.0
pymongo==3.10.1
pyOpenSSL==19.1.0
python-dateutil==2.8.1
PyYAML==5.3.1
requests==2.23.0
requests-oauthlib==1.3.0
rsa==4.6
setuptools==41.6.0
six==1.15.0
typing-extensions==<IP_ADDRESS>
urllib3==1.25.9
websocket-client==0.57.0
wheel==0.33.6
yarl==1.4.2

+1 Experiencing the same issue. It is only present in v0.27 not in the previous versions.

Solved in https://github.com/nolar/kopf/pull/523, available since kopf>=0.28rc2


Dsiable non-working tests

bls - build
ecdsa - build
eddsa - build
elgamal_verifiable - build

All tests failed to compile.


Starting on Documentation / a wiki

I think we need to figure out how to put documentation out on how to get started, or how to run basic projects / setup a development environment to contribute back to galileo. It would probably massively improve access and we could onboard more contributors that way.
In addition or subtitution, a discord server?

I agree, some write up of basic concepts of the library is necessary. And a simple tutorial would also be helpful for new users. But at this point I'm afraid it will get outdated very quickly. There are a few fundamental things that might sill change. For example, I think the current approach for generic symbols is too cumbersome, but I haven't figured out yet a way to make it better. Also, the way feature layers are edited is not ideal, and event handling seems like it requires too much code to do very little... Also the CRS system even though it seems to be working, I'm not sure how convenient it would be in some more complex scenarios. Another thing is the way vector tiles are styled will definitely change, probably to use the same symbols as feature layers.
So what I was planning to do is after I finish working on labels, I wanted to write a couple of simple applications. For example, a simple vector tiles styler, or a .shp file viewer. Just to see narrow and inconvenient places of the library. After which add a rust-style book right in the repository describing basics. It would be hosted on Github Pages.
I don't mind if anyone would start this work already, but I'm afraid that some parts would have to be rewritten....
I'm not sure that a dedicated discord server would be of any use now. I definitely don't want to manage it. There's Github discussions which can be used to ask anything. If you find Discord more convenient, we could ask some other team, like geo to create a channel for us.

Perhaps you've heard of my project Catenary Maps? The entire backend is in rust but the frontend is crappy and written in Javascript. My idea is to re-write the entire frontend into Rust, so galileo + some UI library and then put it on Web, Apple, and Android. That's my primary motivation to work on Galileo, so I definitely can assist in the example making. I'm really excited for labels to come through! Would love to help out in terms of ergonomics of using the library and how to document it.
Channel in Geo server would be great because it can allow for some casual conversion and community around the project, I agree though, most discussion should happen on GitHub discussions.


Fix running app in a different zone than the framework was initialized

When the app starts up with the latest Flutter version it throws this error
======== Exception caught by Flutter framework =====================================================
The following assertion was thrown during runApp:
Zone mismatch.

The Flutter bindings were initialized in a different zone than is now being used. This will likely cause confusion and bugs as any zone-specific configuration will inconsistently use the configuration of the original binding initialization zone or this zone based on hard-to-predict factors such as which zone was active when a particular callback was set.
It is important to use the same zone when calling `ensureInitialized` on the binding as when calling `runApp` later.
To make this warning fatal, set BindingBase.debugZoneErrorsAreFatal to true before the bindings are initialized (i.e. as the first statement in `void main() { }`).
When the exception was thrown, this was the stack:
#0      BindingBase.debugCheckZone.<anonymous closure> (package:flutter/src/foundation/binding.dart:495:29)
#1      BindingBase.debugCheckZone (package:flutter/src/foundation/binding.dart:500:6)
#2      _runWidget (package:flutter/src/widgets/binding.dart:1439:18)
#3      runApp (package:flutter/src/widgets/binding.dart:1373:3)
#4      main.<anonymous closure> (package:sweyer/main.dart:104:5)
<asynchronous suspension>
====================================================================================================

~This is fixed by ensuring that the WidgetBindings are initialized in the zone created by runZonedGuarded. That required to improve the reportError function to handle getting called before anything is initialized.~
I realized that there is a better fix for this, which is to remove the runZonedGuarded and use PlatformDispatcher.instance.onError instead. This is the recommended way to catch async errors per the crashlytics documentation for the latest version.
To make sure everything continues to work I've updated the firebase packages to the latest version. They added an autogenerated firebase_options.dart, which I generated using a test firebase project and copying the values from the google-services.json file. It would be best if you could run flutterfire configure as described in the documentation to make sure the values are correct.

To make sure everything continues to work I've updated the firebase packages to the latest version. They added an autogenerated firebase_options.dart, which I generated using a test firebase project and copying the values from the google-services.json file. It would be best if you could run flutterfire configure as described in the documentation to make sure the values are correct.

I've done and it only generated minor formatting changes, it seems

I've done and it only generated minor formatting changes, it seems

Thanks 👍

New build workflow is amazing to manually test PRs


Upgrade to latest Swift Version

@Nosrac, I use this app on a frequent basis and would like some minor enhancements.  It appears this is on an old version of swift.  If you could upgrade to the latest Swift version, I could try to pitch in and work on some of the bug fixes and/or enhancements.

Thanks for the ofer @tleish.  I'll see if I can carve out some time to get this project updated to the latest Swift version in the next few weeks.
Kyle

FYI, I attempted upgrading to Swift 4.2.  Swift is new to me, but I
basically went though XCode and accepted a good portion of the suggested
changed of renamed methods, etc.  I think I've done most of the easy work,
the harder stuff remains (although, who knows... I may have messed up the
entire project).
See my swift4.2 branch
https://github.com/tleish/Dictater/tree/swift4.2
On Fri, Nov 15, 2019 at 11:47 AM Kyle Carson<EMAIL_ADDRESS>wrote:

Thanks for the ofer @tleish https://github.com/tleish. I'll see if I
can carve out some time to get this project updated to the latest Swift
version in the next few weeks.
Kyle
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Nosrac/Dictater/issues/22?email_source=notifications&email_token=AAIMRWPMFNMTFBTROJZAGQDQT3VD5A5CNFSM4JN3ZHSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEGLSEI#issuecomment-554481937,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIMRWO5IZKCLQ2UDRPPNETQT3VD5ANCNFSM4JN3ZHSA
.

Hey again @tleish,
Sorry I didn't get back to your last message 🙈.
Just wanted to let you know I finally carved out some time and got the app working again.  You can download here: https://github.com/Nosrac/Dictater/releases/tag/catalina

Awesome!  Thank you for updating.


Whether to support the wechat mini program

Whether to support the wechat mini program

There is a WeChat channel in Azure Bot Service.  It would be possible to use a custom adapter.  As reference, there is the DotNot sample demonstrating this:  https://github.com/microsoft/BotFramework-WeChat/
That repo has not been updated in some time, so it is unknown if it is current enough.


Dragging items containing images is buggy

The majority of the time (but not always, I can't see a pattern, maybe 70/30 random) when dragging an item containing an image, I have to release the mouse button to get the item to start moving and then click the mouse button again to drop the item.
I think it has to do with whether my browser thinks I am trying to drag the image (like to save it to my desktop) or whether ng-sortable-view handles the click instead.
I am using an up-to-date version of Chrome.

I just figured out that the difference is based on where in the sv-element I click. If I click in the top half of the element, ng-sortable-view works normally.  If I initiate the drag by clicking in the bottom half of the element (even below the picture) my browser tries to drag the picture until I release the mouse button, at which time ng-sortable-view takes over and I can drag it around and drop it by clicking a second time.

I had the same problem, if you'll just add draggable="false" to the img element, it will be fixed!

Old and irrelevant


Cart Item

I want item add only one time on cart next time i add item in cart its replace with new not update qty.

In that case, before call the add method, first check if the product is already in the cart. There is no build in way to do this.


fix typo

Thank you for this information. But we can't merge small changes. Thank you! I will patch it in the next update.


Refactor from browser to chrome API

Safari and Firefox both support the chrome extension API, so there's no real need for the overhead of polyfilling browser for Chrome to retain a cross-browser implementation.

:tada: This issue has been resolved in version 5.1.3 :tada:
The release is available on:

GitHub release
v5.1.3

Your semantic-release bot :package::rocket:


pick new fullscreen icon

make the icon conditional (on or off)

https://design.google.com/icons/

use E80F when in fullscreen mode.


Can't resolve 'mobx'

Hi,
I've just update the tree component to 4.1.0 from 3.5.0 and now I'm getting the following compilation error:
ERROR in ./~/angular-tree-component/dist/models/tree-node.model.js Module not found: Error: Can't resolve 'mobx'

are u using webpack or systemjs? can you paste your package.json / other config files? can u try removing node_modules and re-installing?

Hello,
I tried to re-install and clear cache nothing worked, I'm still getting this ERROR
ERROR in ./~/angular-tree-component/dist/angular-tree-component.umd.js Module not found: Error: Can't resolve 'mobx-angular' in 'D:\Workspace\KOCReports\node_modules\angular-tree-component\dist'

If it's umd.js I'm guessing you are using SystemJS which is not supported

Hi Friends,
I am new in MEAN stack, I am also getting same error
I am getting error on >ng serve
ERROR: ERROR in Metadata version mismatch for module ./~/node_modules/mobx-angular/dist/mobx-angular.d.ts,
found version 4, expected 3, resolving symbol TreeModule in ./~/node_modules/angular-tree-component/dist/angular-tree-component.d.ts,
resolving symbol TreeModule in ./~/node_modules/angular-tree-component/dist/angular-tree-component.d.ts
Is there any workarround steps for this issue.
I really appreciate any help you can provide.


Programmatic open

Is there a possibility to open cookieconsent panel programatically when I already closed it and window.cookieconsent is available to me ? I want to have my own control which will triggers the Popup when clicked.
Thank you

Found it https://github.com/insites/cookieconsent/blob/master/examples/example-7-javascript-api.html
Thanks

Issue resolved - Documentation

Thanks, but the link to Documentation leads to my comment. Is that right ?

@char0n Yes, I do that just for reference. It lets anyone later looking through it be able to always see the 'closing comment' and being able to have a good reason for the close, while having some sort of link. Here it seems superfluous because it's right there and obvious but I like to keep to a pattern.
Look for new features coming to v3 soon!


ui Do not do profile format in the FE

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


macOS support?

I couldn't find any OS requirements. Doesn't work on a Mac...

VS Code Extension Version: v1.0.5
System Version: macOS 14.3.1 (23D60)
Kernel Version: Darwin 23.3.0

It should work on macOS, but you may need to open BC and use the Beyond Compare > Install Command Line Tools... command to enable it to launch BC correctly.  Once that's done, if you're still seeing incorrect behavior please let us know what steps you're taking and what errors you're seeing.

Great. Didn't check the source code of the extension and you could update the docs to state this. Works OK.


UUID fetching problem

Hi.
I have a server with custom authentication with authlib patches.
When player (maryzakat in this case) was added by /cmodify maryzakat UUID fetched from Mojang servers (there is MaryZakat account exists), by UUIDFetcher (https://github.com/Tsuser1/Modern-LWC/blob/master/src/main/java/com/griefcraft/util/UUIDRegistry.java#L128).
If player create protection by itself, all OK, UUID fetched by Bukkit's Player class (https://github.com/Tsuser1/Modern-LWC/blob/master/src/main/java/com/griefcraft/modules/create/CreateModule.java#L158)

@roman901 Please check and include all information asked for in the issue template. You haven't provided enough information for me to investigate this issue.
LWC Version (/lwc admin version):
2.1.1
Server version (/version):
1.13
Short five word summary of issue:
LWC don't support custom authlib & custom UUID source
Detailed description of problem:
I use https://github.com/new-sashok724/Launcher (VERY popular in Russian community). It provides custom authlib and UUID generation.
When player tries to /cmodify , LWC took UUID from official Mojang servers, which is not equals to target player's UUID at this server, and target player don't get access to container.

Thanks for the specifics!
I take it that this is an offline mode server? I don't think LWC was designed with custom UUIDs in mind. They are supposed to be standard with Mojang's auth system, as far as I'm aware.
That being said, is this only an issue with cmodify or with the plugin in general? Also, when did this become an issue? When you upgraded to 1.13 or due to using your custom ID libraries?
What changes would you suggest to fix this? Consider making a pull request if you write code. This would probably be the quickest. If you don't, try to give me a detailed description of how to set up an environment for this issue, and what I'd need to do to replicate it. Then, at least, I could consider investigating it further.

I don't think that it is a real bug.
In general, methods with uses Player.getUniqueId() works correctly (/cprivate, for example). Methods, which operates with player's nickname (and use UUIDRegistry) are "broken" in my case.
I try to find a solution, we must use server's authlib rather making requests to Mojang. I wll try make PR to you.
Thank's for fast reply!

I haven't really looked at UUIDRegistry much, but I think it was added around the time that UUIDs were new to MineCraft. If Bukkit's built-in methods supercede it, perhaps it can be replaced.

@roman901 The issue might be fixed in the latest development build. Unfortunately, a repository for the project seems to have permanently shut down and we are in the process of setting up a replacement. As such, you cannot download this build currently from the build server, but if you want to try a copy of the build please message me on Discord.

Forgot to update this, but roman discussed this with me on Discord, and as he mentioned previously it is not a real bug, but more of a feature request because it involves adding extra support for non-standard authentication methods.
I will probably not work on this, but if @roman901 or anyone else wants this feature it remains open to PR.

I'm assuming this is no longer needed, so I am cleaning up the issue. Let me know if you were still planning to make some integration and I will try to accommodate it. But since this is not a popular/requested issue, I am going to close it now,.


Vuex store is not loaded to client side app

Hello all,
I am developing a vue app using this plugin.
But store is not loaded to client side app, so I get "TypeError: Cannot read property 'state name' of undefined" when I use mapState of Vuex.
Plz help me.

I’m having the same issue. @Akryum
I can see the data being loaded on the server side and even gets passed to the entry-client.js file. But the store.replaceState(window.__INITIAL_STATE__) call doesn’t seem to do anything.

I'm also facing this issue,
in entry-client and entry-server state is as expected when but when I look in vue devTools my store looks like this:
0:"_" 1:"_" 10:"o" 11:"o" 12:"l" 13:"_" 14:"u" 15:"n" 16:"d" 17:"e" 18:"f" 19:"i" 2:"v" 20:"n" 21:"e" 22:"d" 23:"_" 24:"_" 3:"u" 4:"e" 5:"_" 6:"d" 7:"e" 8:"v" 9:"t"


CesiumMath.factorial has issue #7888

current version of CesiumMath.factorial function has a bug reported in issue #7888.
the problem can be fixed by a simple logic correction

Thanks for the pull request @rshaikhha!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!

:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

This request is ready

Contributor License Agreement (CLA) set via email<EMAIL_ADDRESS>

Thanks @rshaikhha! We got the email, but the person who reviews the CLAs is out of the office. We'll review this PR as soon as we get the okay.

@hpinkos Thanks Hannah, any news from CLA?

Thanks again for your contribution @rshaikhha!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @rshaikhha!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

This was actually fixed in https://github.com/AnalyticalGraphicsInc/cesium/pull/7969. Thanks again for reporting this!

Thanks for the awesome work @rshaikhha! Your contribution is about to launch to millions of users with the next release. :rocket:
Do you mind if we tweet about it? CC @OmarShehata @Slchow.


Missing "." export in "package-name" package

⚠ IMPORTANT ⚠ Please check the following list before proceeding. If you ignore this issue template, your issue will be directly closed.

[x] Read the docs.
[x] Use Vite >=2.0. (1.x is no longer supported)
[x] If the issue is related to 1.x -> 2.0 upgrade, read the Migration Guide first.

Describe the bug
Hi, I'm currently working on https://github.com/antfu/reactivue/tree/b48200b9b0c5c2e02cf7855e1568f11c808de43c tree.
Using build of this tree in react-ts template. Running vite dev is fine but vite build prompts an error on console. This issue occurs on 2.0.0-beta.23. I tested it on 2.0.0-beta.21, it works fine.

System Info

vite version: 2.0.0-beta.23
Operating System: Windows 10
Node version: v12.16.2
Package manager (npm/yarn/pnpm) and version: yarn

Logs (Optional if provided reproduction)
$ vite build --debug
  vite:config bundled config file loaded in 143ms +0ms
  vite:config using resolved config: {
  vite:config   plugins: [
  vite:config     'alias',
  vite:config     'vite:dynamic-import-polyfill',
  vite:config     'vite:resolve',
  vite:config     'vite:html',
  vite:config     'vite:css',
  vite:config     'vite:esbuild',
  vite:config     'vite:json',
  vite:config     'vite:wasm',
  vite:config     'vite:worker',
  vite:config     'vite:asset',
  vite:config     'react-refresh',
  vite:config     'vite:css-post',
  vite:config     'vite:build-html',
  vite:config     'commonjs',
  vite:config     'vite:data-uri',
  vite:config     'vite:define',
  vite:config     'rollup-plugin-dynamic-import-variables',
  vite:config     'vite:import-analysis',
  vite:config     'vite:esbuild-transpile',
  vite:config     'vite:terser',
  vite:config     'vite:size'
  vite:config   ],
  vite:config   alias: [
  vite:config     {
  vite:config       find: /^\/@vite\//,
  vite:config       replacement: 'C:\\Users\\sib\\Desktop\\dev\\vite-project-react\\node_modules\\vite\\dist\\client/'
  vite:config     },
  vite:config     { find: 'vue', replacement: 'reactivue' },
  vite:config     { find: '@vue/runtime-dom', replacement: 'reactivue' }
  vite:config   ],
  vite:config   optimizeDeps: { exclude: [ '@vue/reactivity' ] },
  vite:config   build: {
  vite:config     base: '/',
  vite:config     target: [ 'es2019', 'edge16', 'firefox60', 'chrome61', 'safari11' ],
  vite:config     polyfillDynamicImport: true,
  vite:config     outDir: 'dist',
  vite:config     assetsDir: 'assets',
  vite:config     assetsInlineLimit: 4096,
  vite:config     cssCodeSplit: true,
  vite:config     sourcemap: false,
  vite:config     rollupOptions: {},
  vite:config     commonjsOptions: { include: [Array], extensions: [Array] },
  vite:config     minify: 'terser',
  vite:config     terserOptions: {},
  vite:config     cleanCssOptions: {},
  vite:config     write: true,
  vite:config     manifest: false,
  vite:config     lib: false,
  vite:config     ssr: false
  vite:config   },
  vite:config   configFile: '/Users/sib/Desktop/dev/vite-project-react/vite.config.ts',
  vite:config   inlineConfig: {
  vite:config     root: undefined,
  vite:config     mode: undefined,
  vite:config     configFile: undefined,
  vite:config     logLevel: undefined,
  vite:config     build: {}
  vite:config   },
  vite:config   root: '/Users/sib/Desktop/dev/vite-project-react',
  vite:config   command: 'build',
  vite:config   mode: 'production',
  vite:config   isProduction: true,
  vite:config   optimizeCacheDir: '\\Users\\sib\\Desktop\\dev\\vite-project-react\\node_modules\\.vite',
  vite:config   server: {},
  vite:config   env: { BASE_URL: '/', MODE: 'production', DEV: false, PROD: true },
  vite:config   assetsInclude: [Function: assetsInclude],
  vite:config   logger: {
  vite:config     info: [Function: info],
  vite:config     warn: [Function: warn],
  vite:config     error: [Function: error],
  vite:config     clearScreen: [Function: clearScreen]
  vite:config   }
  vite:config } +6ms
building for production...
[vite:resolve] Missing "." export in "reactivue" package
error during build:
Error: Missing "." export in "reactivue" package
    at bail (C:\Users\sib\Desktop\dev\vite-project-react\node_modules\vite\dist\node\chunks\dep-478c65b9.js:39770:8)
    at resolve$1 (C:\Users\sib\Desktop\dev\vite-project-react\node_modules\vite\dist\node\chunks\dep-478c65b9.js:39846:10)
    at resolveExports (C:\Users\sib\Desktop\dev\vite-project-react\node_modules\vite\dist\node\chunks\dep-478c65b9.js:39852:12)
    at resolvePackageEntry (C:\Users\sib\Desktop\dev\vite-project-react\node_modules\vite\dist\node\chunks\dep-478c65b9.js:40126:22)
    at tryNodeResolve (C:\Users\sib\Desktop\dev\vite-project-react\node_modules\vite\dist\node\chunks\dep-478c65b9.js:40041:11)
    at Object.resolveId (C:\Users\sib\Desktop\dev\vite-project-react\node_modules\vite\dist\node\chunks\dep-478c65b9.js:39952:28)
    at C:\Users\sib\Desktop\dev\vite-project-react\node_modules\rollup\dist\shared\rollup.js:18781:25
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

This is the correct behavior. When exports field is present, it means anything not explicitly listed inside is not exposed. This should be fixed on reactivue's side by adding . in exports.

This is the correct behavior. When exports field is present, it means anything not explicitly listed inside is not exposed. This should be fixed on reactivue's side by adding . in exports.

In the case of reactivue, which this issue was originally filed for, it was a bad package configuration. However, typically when I've seen this issue hit it's because of what I'd consider a bug in Vite. Some packages like Firebase legitimately don't have . as an entry point. However, Vite always tries to resolve . instead of whatever entry point the application code is using causing this failure. See https://github.com/vitejs/vite/issues/4425 for more details


HttpPostRequestEncoder's finalizeRequest method overrides "multipart/mixed" to "multipart/form-data"

Is there a way for me to create a multipart request of type "multipart/mixed" instead of multipart/form-data?
Eg:
      HttpRequest request = new DefaultFullHttpRequest(HttpVersion.HTTP_1_1, HttpMethod.POST, "/");         HttpPostRequestEncoder encoder = new HttpPostRequestEncoder(dataFactory, request, true);         request.headers().add(HttpHeaderNames.CONTENT_TYPE,          HttpHeaderValues.MULTIPART_MIXED);         encoder.addBodyHttpData(createJSONAttribute(request));         encoder.addBodyHttpData(createFileUpload(request));         request = encoder.finalizeRequest();         sendMultipartRequest(request, encoder);
"finalizeRequest" method overrides the mixed content type to form-data.
If I'm to support other subtypes of multipart (Eg: multipart/related) do I have to write my own encoder and decoder?
According to the MIME spec https://tools.ietf.org/html/rfc2046#page-17

" All present and future subtypes of the "multipart" type must use an
identical syntax.  Subtypes may differ in their semantics, and may
impose additional restrictions on syntax, but must conform to the
required syntax for the "multipart" type.  This requirement ensures
that all conformant user agents will at least be able to recognize
and separate the parts of any multipart entity, even those of an
unrecognized subtype."

So can't we use the same encoding/decoding logic for other multipart subtypes as well?
Thank you.

@fredericBregier
This statement which you mention "the global header cannot be multipart/mixed, only multipart/form-data"
Is this a netty restriction? I cannot find any such restriction imposed by MIME specification.
According to the spec all "multipart" types must use an identical syntax whether they be multipart/form-data, multipart/mixed, multipart/new-sub-type.  So the decoder will be able to recognize
and separate the parts of any multipart entity, even those of an unrecognized subtype.
In netty decoder, this was hardcoded specifically for "MULTIPART_FORM_DATA".
protected static String[] getMultipartDataBoundary(String contentType) {   String[] headerContentType = splitHeaderContentType(contentType);   String multiPartHeader = HttpHeaderValues.MULTIPART_FORM_DATA.toString();
...
Is there a reason for this restriction?
[I'm not actually talking about the nested parts, where you could have mixed within form-data. But about the top level message which only allows form-data. ]
Thanks for the response.

RFC 1867 specifies that only multipart/form-data with within eventually multipart/mixed (see 3.3). See second example in chapter 6.
RFC 7578 (sorry for the wrong number previously) :
I quote already the chapter 4.3 which says "[RFC2388] suggested that multiple files for a single form field be transmitted using a nested "multipart/mixed" part.  This usage is deprecated." and the Appendix A which says " The handling of multiple files submitted as the result of a single form field (e.g., HTML's  element) results in each file having its own top-level part with the same name
parameter; the method of using a nested "multipart/mixed" from  [RFC2388] is no longer recommended for creators and is not required for receivers as there are no known implementations of senders."
So the body must be multipart/form-data but it allows then inner multipart/mixed parts, even if 7578 says it is deprecated.

Hello,
I found this issue and I'm interested to have a decoder for "multipart/related" HTTP bodies similar with this one:
https://cloud.google.com/storage/docs/json_api/v1/how-tos/multipart-upload
@fredericBregier the RFC 1867 specifies how form based upload works (i.e. use the multipart/form-data).
What @arukshani is hinting at is that all "multipart/*" content types work in a similar fashion and it would be nice to have a generic way to decode them - essentially split the body into parts based on the boundary. How to interpret the different parts will depend on the specific type used.
So the question is if the current functionality that has been made for multipart/form-data can be easily changed to fit this more general need or if it's better to implement from scratch a new decoder.

@xpromache  Why not ? If possible, well, it will allows to make more things ! ;-)
You can propose something of course ;-)
Pay attention however to ascending compatibility and to preserve the overall contract in the codec. But I'm quite sure, doing some rework on internal classes could help to have at most 2 codec sharing a lot, at most 1 only...


fix: 🐛 Use multisig.proposals to fetch proposals

Description
multisig.proposalIds doesn't correctly store the proposals for a multisig. This PR fixes the implementation of getProposals in multisig entity to fetch the proposals using multisig.proposals chain storage
Breaking Changes
NA
JIRA Link
DA-898
Checklist

[ ] Updated the Readme.md (if required) ?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 23.0.0-alpha.22 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@patch dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Pattern for checking if value is member of enum should be abstracted in a decorator

We have the same ~4ish lines of code in a number of places which attempt to make sure that some value being assigning to a class property is a member of some enum. And if not, we construct an error message and throw. This should be hidden in a decorator to make the code a bit more readable and also gaurantee some consistency in our error handling.

@LGraber can you add an example or link to one of them?

@site_role.setter
@property_not_nullable
@property_type(UserItem.Roles)
def site_role(self, value):
    self._site_role = value

would replace:
@site_role.setter
def site_role(self, value):
    if not value:
        error = 'Site role must be defined.'
        raise ValueError(error)
    elif not hasattr(UserItem.Roles, value):
        error = 'Invalid site role defined.'
        raise ValueError(error)
    else:
        self._site_role = value

I actually have this coded and the decorators defined if you want to see. I am having one little "python" problem. :)


Added some common middleware ...

... that is helpful to all of our projects.

@taphu I think someone should add some info in the README about middlewares included in this library, maybe in a later PR.

I think someone should add some info in the README about middlewares included in this library, maybe in a later PR.

Yes. absolutely.


Add missing Intel CPU modules as flake outputs

Description of changes
Some Intel CPU modules were not added as flake outputs.
Things done

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

I would like to put this on hold, because of this: https://github.com/NixOS/nixos-hardware/issues/992

I think I stick to this decision.


Beautify and PHP problem

Description
Beautify doesn't work with PHP
Input Before Beautification
This is what the code looked like before:
<?php for ($i=0;$i<;$i++){echo "string";echo "string";echo "string";echo "string";echo "string";echo "string";}?>

Actual Output
I see progress bar, but nothing changes. Try to use atom-beautify from menu and from command palette.
Steps to Reproduce
I googled this trouble, tried for many instructions, and nothing. Now I have clean install of Win7 x64 and do this:

Install atom
Install only one package: atom-beautify
Download official old PHP version 7.2.34 (but I'm tried newer versions too) and place it in C:\php72 directory
Add PATH to PHP thru CMD: setx path %path%;"C:\php72\"
Download PHP Coding Standards Fixer from https://cs.symfony.com file php-cs-fixer-v2.phar
Place file php-cs-fixer-v2.phar in the php72 folder with php.exe
Start atom, go to atom-beautify Settings->Executables and place in the PHP section C:\php72\php.exe and in the PHP-CS-Fixer section C:\php72\php-cs-fixer-v2.phar
Close atom, restart PC.

Debug
Sorry, I'm newbie, doesn't connect atom with github yet. Can I'm attach file here?
atom-beautify-debug.zip
Checklist
I have:

[+] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[+] Reloaded (or restarted) Atom to ensure it is not a caching issue
[+] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[+] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[+] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

I just read your issue and tried the php snippet you posted. After solving the syntax problem (missing the number e.g. 5) beautify did its job. When there was the syntax issue, beautify did nothing. See the fixed code here:
<?php for ($i=0;$i<5;$i++){echo "string";echo "string";echo "string";echo "string";echo "string";echo "string";}?>
Since I'm not yet very good at php, don't know if this really was the only problem.

If you still need this and are looking for something to fill the spot that Atom left, I am using Pulsar the successor to Atom...   This package is working except for one deprecated error listed in #2605
If you don't need this anymore, can you please close this out so it's not just sitting there? Thanks


(chore) Update actions from checkout@v2 to @v3 due to Node 12 deprecation

Overview
For each test suite we are seeing a deprecation warning in the Annotations section for each run. E.g.:

chrome-functional
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout@v2

This branch changes all instances of checkout@v2 to v3. We already use v3 in some places, so I don't expect this will cause any problems. More information:

Node.js 12 deprecation info
actions/checkout@v3

Related Github Issue
N/A
Testing
This change only impacts the GitHub workflow files, so we might see some changes when we test. The desired result is the deprecation notices disappear.

https://github.com/newrelic/newrelic-browser-agent/actions/runs/3703437369

To keep this PR simple and get it merged, backed out the change from node 14 to 16 due to some errors we were seeing related to filesystem permissions.


Restructuring tools and making some APIs public

DumpData has grown to take in lot many operations and it was hard to maintain it. So, have restructured into many classes viz DumpLog, DumpIndex, DumpReplicaToken and DumpData. From now on, DumpDataHelper will be the only class which deals with reading data from disk. Rest of the classes acts as a wrapper in performing the operation.
Introduced config files for each classes (DumpLogConfig, DumpIndexConfig, DumpReplicaTokenConfig and DumpReplicaTokenConfig)
All these tools take in a property file as input and reads all the property from the same.
Made some of these operations as public.

SLA: 20 mins
Reviewers: @vgkholla

synced up offline. Addressed all the comments.
@vgkholla : you can go ahead and review it

Will import it in Intellij and give it another look tomorrow.

I have addressed your comments so far. Will wait for you review.

Yes, built using java 7 and formatted as well.


Changes to helm chart values or values.yaml aren't always triggering an upgrade

As the title says
see https://github.com/rancher/k3s/pull/183

Verified by modifying /var/lib/rancher/k3s/server/manifests/traefik.yaml to add spec.valuesContent, on change the helm-install-traefik pod was re-run to process the modification.

Is it correct that helm is installed alongside of k3s? I couldn't find it, last time I checked I needed to manually install it.

@jwillmer k3s includes a HelmChart CRD that uses [https://github.com/rancher/klipper-helm] to run helm install/upgrade/remove. There is no tiller pod in the cluster.
@erikwilson I'm seeing similar behavior (upgrades not happening). I've been applying and modifying HelmChart resources directly via kubectl apply, not via the manifests directory. Should I expect that to work?

Helm upgrade are broken at the moment, we probably need to remove the old job first.

It indeed works fine if the old job is removed manually before applying the updated helm chart. Tested with k3s version v0.7.0.
kubectl delete job --namespace=kube-system helm-install-cert-manager
kubectl apply -f cert-manager-helmchart.yaml

A new install job was properly created.


Gulp default not building the equivalent javascript as in CDN

When trying to build both minified css and javascript for custom changes the generated output is not functioning and far from the CDN JavaScript although both relates to version 1.1.3.
I noticed that there is a porting to v2 of some components and not of them are ready. In gulp SOURCES those components using componentHandler as global resource are commented out.
Is there any open issue with a branch in progress I could work with ?

As the readme states, mdl-1.1 is for continuing development of the 1.1 branch. master branch is now 2.0 development which we aren't currently supporting.


Run host generation tests in Travis

This modifies the :travis rake task to run all available tests in the
Rakefile, including host generation tests and Beaker subcommand tests.
This ensures that any changes that may break Beaker host generation or
subcommands will be caught before the change is merged.

@lucywyman will you close and reopen this PR to see if it triggers travis now? I just tried some voodoo to get it to actually run on https://travis-ci.com/github/puppetlabs/beaker

I am not sure why this isn't running on Travis.... :(

It looks like Travis is receiving the requests, it might just not be showing up in the UI? https://github.com/puppetlabs/beaker/settings/hooks/233832447

Webhooks are not used anymore for us. We have the newer gh org wide integration that is a real GH application

I've filed HELP-25028 with a request for vendor support to address the issues with Travis not running on this repo

Ok, verified this is working with Travis now! https://github.com/lucywyman/beaker/pull/6