[Feature Request] Option to show blank element in place of ghost

There should be an option to show blank element is place of ghost that doesn't contain any child elements. I am not talking about hiding ghost.
Use case of this feature: Dragable item can contain onload handlers or iframe etc. That can trigger API calls on ghost dom node insertion. Hiding the ghost will not prevent API calls/event handlers execution.
Demo: https://codepen.io/armujahid/pen/XOBgLN
Try to drag 1st element to 4th element. API 1 will be called thrice because of ghost insertion on every move.
JQueryUI Sortable already has this feature by default.
It shows empty ui-sortable-handle instead of showing ghost element.

Try:
new Sortable(list, {
    ghostClass: 'hidden'
});

.hidden {
    visibility: hidden;
}

Demo:

Issue still exists even after hiding ghost. You can check console logs and network calls. As I already mentioned hiding it will not stop dom events/API calls.
Demo: https://codepen.io/armujahid/pen/qgyMOV

@armujahid Apologies, I did not read far enough into your issue.
I created a demo that will set the src of the iframe to an empty string while it is being dragged: https://codepen.io/owen-m/pen/aXjRvR
The "onload" event is still fired, but no network request is made, only until the element is dropped (this would have to happen anyways).
Let me know if this solution is adequate. An option for a placeholder that is different from the dragging element is certainly a feature that can be added in the future, but for now this is all I can think of.

I created that iframe and onload demo just for example. Actually we have full-fledged JavaScript dragable apps (called widgets) developed in multiple frameworks (angular, react, polymer etc).
So for now I will use JQueryUI Sortable that already provides this feature. I will try to raise PR after analyzing this code base.
May be we should add feature/enhanement label to this issue?

If there was a way you could delete the contents of the component, and then re-add them later, you could do that instead of removing and re-adding the src of the iframe.

I end up using Muuri draggable grid layouts that can rearrange items using CSS transforms without insertion/removal of dom nodes

Is there possibility to remove ghost element completely? When moving between groups i don't want anything to show (not even empty space like when using visibility: hidden in new group before dropping items. I use highlighting with color to indicate that dropping is possible.
When i apply display: none to ghost element, dragged element is not positioned under cursor, because it takes initial position based on ghost element position. Is there any other way?

@mszacun Try setting the display:none in the onStart event, which will be fired after the dragged element is positioned, and then you can reset the display in onEnd.


Non-breakable spaces in author fields should be exported as tilde

While entering and exporting NBSPs (u+00a0) in other fields like the title lead to proper ~ this is not the case in the author field. There they are simply written as SP (u+0020). I did not find a workaround since for example ~ in Zotero is translated to \textasciitilde{}.
The example in Report ID: CRRYZ9ZY-euc shows the problem with the "author" "JTC 1/SC 22/WG 14".

Was this with Better BibTeX or with Better BibLaTeX?

Doesn't matter: both show the same behavior AFAICT.

There was a discussion on this a while ago in #859. @njbart usually know a lot about this and advised to replace ZWS with nothing, and other-spaces with a simple space. Do you think that assessment no longer holds? What breaks for you when the NBSP is replaces with a regular space? I'm OK with leaving the NBSP as long as it doesn't break other stuff.

Not sure which of these questions are to me so please ignore at will... :)
It's really an edge case obviously. However, the official documents by ISO have a space in there so removing them is obviously not ideal (OTOH probably nobody but the most pedantic persons would even notice it - and it would probably be my preferred workaround if need be). If left in as normal spaces then line breaks could easily make it even worse... that's why I was thinking about NBSPs at all and tried them.
When I noticed it not working I tried around with the other fields and since it showed a different behavior it became interesting enough to report it. Although I googled I have not found the other issue. I would have been OK with the space replacement if there would have been any indication (and rationale) that the replacement is on purpose, e.g., a hint in the quality report would have been totally fine with me.

The question was indeed for you. And it doesn't really matter whether it's an edge case, edge cases matter too. But what you're saying is that in the rendered output the spaces are not present? Aside any issues with line breaks? That would mean I'd have to export the NBSPs.

Maybe I’m not seeing the obvious, but what exactly is the problem with ordinary (as opposed to non-breaking) spaces in author names such as “JTC 1/SC 22/WG 14”?

The reason I started replacing them is because these space-like characters quite often got into zotero by copy-paste and were not intended. But I have no problem making an exemption for NBSP if that solves a problem.

The reason I started replacing them is because these space-like characters quite often got into zotero by copy-paste and were not intended.

Agreed.

But I have no problem making an exemption for NBSP if that solves a problem.

My question is, is there a genuine problem with ordinary spaces in “JTC 1/SC 22/WG 14” in the first place?

But I have no problem making an exemption for NBSP if that solves a problem.

My question is, is there a genuine problem with ordinary spaces in “JTC 1/SC 22/WG 14” in the first place?

Well, if it's genuine or a problem is probably debatable but with normal spaces you could get a final output in the PDF with line breaks in between, e.g.:
JTC 1/SC
22/WG 14

I see. So what this seems to boil down to is whether accepting the possibility of line breaks in “JTC 1/SC 22/WG 14” or risking suboptimal overall paragraph layout is the lesser evil – in which case I’d say, the first one is.

I would rather say the question is if BBT should decide this or the user. If you forcefully remove NBSPs there is no way to override this behavior. If they are exported like in other fields the user can simply replace them with normal spaces if he wants line breaks.
Compare it to a hypothetical BBT extension that replaces all occurrences of "teh" in titles with no way to override it. I hope we agree that this would be a bad idea.

The question was indeed for you. And it doesn't really matter whether it's an edge case, edge cases matter too. But what you're saying is that in the rendered output the spaces are not present? Aside any issues with line breaks? That would mean I'd have to export the NBSPs.

In the exported .bib (without your patch) I get normal spaces:
author = {{JTC 1/SC 22/WG 14}},

I would have expected this:
author = {{JTC~1/SC~22/WG~14}},

I would then probably try adding zero-width spaces after the / to make them break at the right places to make sure typesetting does not have a too hard time to flow the paragraph. I would then notice that this does indeed get converted but I think not correctly: it produces a \mbox{} which does not allow lines break I think (it certainly does not make them in my test). I think the correct replacement would be \hspace{0pt}. So eventually I would arrive at something like below (which is obviously repeated for testing):
author = {{JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14/\hspace{0pt}JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14/\hspace{0pt}JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14/\hspace{0pt}JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14/\hspace{0pt}JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14/\hspace{0pt}JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14/\hspace{0pt}JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14/\hspace{0pt}JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14/\hspace{0pt}JTC~1/\hspace{0pt}SC~22/\hspace{0pt}WG~14}},

Which would create this final output in the bibtex style I use:

What's the rationale for the mbox? Would you like me to open another issue for that?

I can try making an exception for NBSP because I understand where it's coming from, but I think the comparison with teh is a little silly, and I've never seen anything resembling that author. There's a clear difference between these cases and whitespace replacement. It is also not the case that BBT should never transform the input; quite a bit of effort goes into parsing dates into something sensible. Let's not devolve into whataboutism and just discuss the merit of the proposal at hand.
I probably took the mbox from a list I imported to get the mapping started. It's not a deliberate choice on my part.

Of course the example is silly - I was exaggerating to make the point that users should have the last say. (However, the example is less far fetched than you might think: https://scholar.google.at/scholar?hl=en&as_sdt=0%2C5&q="teh").
As you say, BBT's purpose is actually to convert the Zotero data into something reasonable so that the user does not have to think about it too much or to fix it manually after exporting. But it can never be perfect because there are sometimes no clearly best solutions (like in the C99 example). Thus my argumentation for not forcing things without an option for the user to get around it if need be. However, I understand that there are some users (probably the majority) who do not care enough for such things to even known about NBSPs or ZWSPs. UI design is hard :)
The commit you linked to earlier... is that something you want me to test (because AFAICS I would have to build BBT then or did I miss something)?

Oh wait - mbox is discussed in #859.

It is but AFAICS the only alternative discussed there is to leave the UTF character \u200B in place and not other LaTeX alternatives.

No, no need to build anything, that's just github announcing a link between the commit and the issue. Once I commit a version that passes all tests (the change doesn't currently, because a test case associated with #859 doesn't pass) a new build will be uploaded here.
I'm stuck in meetings for the next few hours, when I'm free again I'm open to to any change that works. I know too little about bibtex and biblatex to have strong opinions on the matter.

I do have concerns about breaking changes, so it'd be good to have some assurance that the concerns voiced in #859 remain addressed in whatever solution we settle on.

For biblatex (and thin spaces) this seems relavent: https://tex.stackexchange.com/a/11083/23277

You'll have to translate for me what that entails for BBT.

I think we should wait for further input of @njbart.

@stefanct I'm pretty curious now what's behind the scholar link, but I just get "Please enter a query in the search box above."

@retorquere Hm, I have removed some GET-parameters, maybe that works better? https://scholar.google.at/scholar?q="teh"
I don't know why either of those would force you to log in... In any case you can simply search for "teh" on google scholar yourself. The quintessence is that there are actually people named "Teh" :)

Oh that, yeah that's not surprising, I was thinking about the part you mentioned as repeated for testing.

Hm, that was in another message with no relation to google scholar though...? And in that message (with the repeated author line) there are no links whatsoever, apart from the screenshot that is hosted by github. Now I am confused. :)

Crossed signals then. I thought the repeating sample unlikely because I've not seen anything like it in the real world, and the teh example unlikely because I'd see no reason to change the locution that way. So I thought the scholar link was meant to show something like the repeating sample actually did have a real-world lookalike.

I see. The purpose of the repetition is simply to demonstrate the line breaking and paragraph formatting I would like to be able to attain eventually. In my current style the author comes in the beginning so line breaking is not very likely but if it comes after the title it is very much possible to require breaking with a short author of course.

I'm willing to exclude NBSP if that doesn't break #859, but an actual use - case would be preferable.

To recap: Initially, https://github.com/retorquere/zotero-better-bibtex/issues/859 was about \, in name fields breaking things on the bib(la)tex side, so we concluded that thin space (U+2009) must not be mapped to \, but to an ordinary space instead. We then decided to map some other space-like chars to ordinary space chars (U+0020) as well, with the exception of zero width space (U+200B), for which there was an actual use case, so this got mapped to \mbox{}.
The only thing I’d actually expect to break if BBT started mapping no-break spaces in name fields to ~ is line-breaking behaviour for users who might happen to have, inadvertently or not, nbsp’s in the name fields of some of their entries. If you’re willing to risk that, fine with me.
Another option, however, might be to just postprocess any name fields in question, either, if possible, on the BBT side, or by employing biblatex’s \DeclareSourcemap mechanism:
In the example below, \DeclareSourcemap selects all author fields containing / (I’m assuming here that ordinary names don’t contain forward slashes) and, in that selection, replaces all ordinary spaces by ~, and all /s by /\hspace{0pt}. If that’s a satisfactory solution for the OP, BBT wouldn’t have to be modified at all.
\documentclass{article}

\begin{filecontents}{\jobname.bib}
@book{jtc,
  author = {{JTC 1/SC 22/WG 14}},
  date = {2011},
  title = {A Title},
  publisher = {Publisher},
  langid = {en},
  address = {Place}
}
@book{jtc2,
  author = {{JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14/JTC 1/SC 22/WG 14}},
  date = {2011},
  title = {A Title},
  publisher = {Publisher},
  langid = {en},
  address = {Place}
}
@book{un,
  author = {{United Nations}},
  date = {2017},
  title = {A Title},
  publisher = {Publisher},
  langid = {en},
  address = {Place}
}
\end{filecontents}

\usepackage[style=authoryear]{biblatex}

\addbibresource{\jobname.bib}

\DeclareSourcemap{
  \maps[datatype=bibtex]{
    \map{
      \step[fieldsource=author,
        match=\regexp{/},final]
      \step[fieldsource=author,
        match=\regexp{\x20}, replace=\regexp{~}]
      \step[fieldsource=author,
        match=\regexp{/}, replace=\regexp{/\\hspace\{0pt\}}]
    }
  }
}

\begin{document}

Foo \autocite{jtc}.

Bar \autocite{jtc2}.

Baz \autocite{un}.

\printbibliography

\end{document}

I also have a strong preference for a solution that does not introduce a new preference. BBT has too many as it is.
The characters under consideration are:
0020 SPACE
00A0 NO-BREAK SPACE
2000 EN QUAD
2001 EM QUAD
2002 EN SPACE
2003 EM SPACE
2004 THREE-PER-EM SPACE
2005 FOUR-PER-EM SPACE
2006 SIX-PER-EM SPACE
2007 FIGURE SPACE
2008 PUNCTUATION SPACE
2009 THIN SPACE
200A HAIR SPACE
200B ZERO WIDTH SPACE
202F NARROW NO-BREAK SPACE
205F MEDIUM MATHEMATICAL SPACE
3000 IDEOGRAPHIC SPACE
FEFF ZERO WIDTH NO-BREAK SPACE

@njbart in the case you describe I would lean towards keeping the NBSP and using a sourcemap to replace them with regular spaces.

Where are we on this guys? Any proposals for the mappings above?

@stefanct any updates? I'm prepping for a release.

Sorry, I am swamped ATM (this will be resolved within a week). njbart's proposal is certainly the right way to go for biblatex. Unfortunately, that does not help (me) with bibtex though. I will look at it in more detail in a few days. If you want to tag a release in the meantime I am totally fine with that.

Any updates?

Sorry, I forgot to get back at this back then. If I understood @njbart correctly then his approach of using biblatex' source mapping does only affect biblatex and not bibtex (and I am not sure how it would be integrated into BBT at all either). It's also unclear what solution you actually consider/propose (for the bibtex case). Regarding the table above I propose the following differences:

ZWSP: \hspace{0pt} (explained earlier).
en quad: this should be the same as en space.
em quad: this should be the same as em space, and \quad indeed is. You could likewise declare the em space as \quad too AFAIK.
202F: should probably be thinner... like \,.
FEFF: this could be "mapped" to an \mbox around the characters preceding and following the space character (but that's not an easy 1:1 mapping).

http://jkorpela.fi/chars/spaces.html has some concise explanations

Sorry this lagged so long, most of these had been updated long ago.


Fix docs plugin version behaviors

Proposed changes

Fixing deployment issue of the docs
Make remarkProgramOutput() version-aware

💡 what actually happened: we included a new codeblock in 2.0.x that shows the output of the program, but this codeblock does not exist on master. The generation worked as expected, but when building the docs, it tried to load the master-version of the source file, which doesn't exist; it should have tried to load a version-aware file.
How to test

checkout the documentation branch
delete file docs/docs/sources/rasa_data_validate__help.txt if it exists
copy the plugin file contained in this branch into the local version of the file on the documentation branch
run yarn build server => no errors, it works

Status (please check what you already did)

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

@tmbo I'd like to merge this today, because it's blocking other docs-related issue. Do you think you have time to do a quick pass? I think it's worth a triple-check 😄

I didn't test it, but the code looks good :+1:

thanks for the quick review 👍


llamafile issue on mac M2

Describe the bug
Hi! can't run on Mac. When run Unity chat scene looks like can't connect to server.
Steps to reproduce
No response
LLMUnity version
No response
Operating System
None

Thanks a lot for the debugging session ⭐!
There is an issue in loading models with llamafile on mac M2.


Are the bboxes of COCOA dataset incorrectly used in this code?

Thanks for your code sharing. I'm a fresh man to this problem. When I looked into your ipython demo on COCOA dataset, I found that the amodal completion result of PCNet-M is always inside the bounding box provided by the COCOA dataset. However, the bounding box provided by the COCOA dataset seems only cover the modal annotations. Is it right? If it is true, I will feel confused about using modal bounding box to restrict the amodal completion area of PCNet-M. And I don't know whether it will bring any bad influence to your training stage.
The figures below show an example I captured from demo_cocoa.ipynb. The image id is 2 (in code).
The bounding boxes are:

The amodal completions are:

It is not as you described. For example, the left boundry of the completion of the dog in the image is out of the bbox. We enlarged the modal bbox to in the demo.

Thank you for your reply. I reread the code and found that the code size = max([np.sqrt(bbox[2] * bbox[3] * self.config['enlarge_box']), bbox[2] * 1.1, bbox[3] * 1.1]) and function expand_bbox() are used for enlarging the origin bounding boxes. So the boxes used for amodal completion are larger than the original ones. I'm sorry for making the misleading descriptions.
I checked many amodal completion results of test images in COCOA dataset, most of them are perfectly matched the amodal gts, except some cases, like the one above.


bcachefs: 2019-10-01

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

On a side note, thoughts on using https://github.com/koverstreet/bcachefs instead of the his website?
Seems to be a mirror made by him.
also cute avatar

Looks like it got rebased, I suppose using the Github sources won't make a difference besides the time difference.

Fixed hash.


@vite/plugin-legacy not compiling legacy bundle with typescript specified target

Describe the bug
Hello, I'm trying to create a legacy build using the legacy plugin,
A demo repository is available here: https://github.com/jeanbaptistesouvestre/vite-legacy-typescript
The code is correctly compiled using typescript when we set target: "ES2015, as an example, i used a simple optional chaining to provide difference between the "modern" build and legacy build
The the modern build (https://github.com/jeanbaptistesouvestre/vite-legacy-typescript/blob/master/dist/assets/index-6925982d.js), the opt chaining is correctly compiled to something looking like
(l=(s=(c=n==null?void 0:n.a)))
but in the legacy build (https://raw.githubusercontent.com/jeanbaptistesouvestre/vite-legacy-typescript/master/dist/assets/index-legacy-57f07a76.js) , optional chaining syntax is preserved
console.log(e?.a?.b?.c?.d)
I've tried playing with babel preset env features, browserlist options but with no change.
The expected output of the legacy build, would be simply a different target for the typescript generated file, but as far as I can tell, the legacy plugin is doing their own compilation, and I can't find the correct option to generate a legacy build that targets es2015 ou similar
Reproduction
https://github.com/jeanbaptistesouvestre/vite-legacy-typescript
Steps to reproduce
run yarn build in the provided repository
System Info
"@vitejs/plugin-legacy": "^3.0.1",
"terser": "^5.16.1",
"typescript": "^4.9.3",
"vite": "^4.0.0"

### Used Package Manager

yarn

### Logs

_No response_

### Validations

- [X] Follow our [Code of Conduct](https://github.com/vitejs/vite/blob/main/CODE_OF_CONDUCT.md)
- [X] Read the [Contributing Guidelines](https://github.com/vitejs/vite/blob/main/CONTRIBUTING.md).
- [X] Read the [docs](https://vitejs.dev/guide).
- [X] Check that there isn't [already an issue](https://github.com/vitejs/vite/issues) that reports the same bug to avoid creating a duplicate.
- [X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to [vuejs/core](https://github.com/vuejs/core) instead.
- [X] Check that this is a concrete bug. For Q&A open a [GitHub Discussion](https://github.com/vitejs/vite/discussions) or join our [Discord Chat Server](https://chat.vitejs.dev/).
- [X] The provided reproduction is a [minimal reproducible example](https://stackoverflow.com/help/minimal-reproducible-example) of the bug.

Vite and plugin-legacy does not use target specified in tsconfig.
You need to use targets option instead.


changed the width of the main cards

What

Changes the main app cards to have the new widths and heights from the new SealCred scalability design
Main app cards are now responsive too

Demo
Screencast from 2022-08-12 08:48:01 PM.webm

Thanks for the comments - I'll work on adding these comments in soon

Addressed your changes @T-Damer


[NN] Collect file access count metric from NN

This includes the following sub-tasks:

[x] Implement the metric collection.
[ ] Provide rpc call for SSM to poll.

dup


Do not warn that directories are unreadable in the in_tail plugin.

Currently a warning will appear for each directory in globbed results.  They will never be readable so the message appears on each refresh.

Thx!


revert rust to 1.76 to test

Pull request
Description

Related

RFC
Related Issues: fixes #000, closed #000
Related docs PR

Checklist

[ ] The RFC, if required, has been submitted and approved
[ ] Any user-facing impact of the changes is reflected in docs.tremor.rs
[ ] The code is tested
[ ] Use of unsafe code is reasoned about in a comment
[ ] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[ ] The performance impact of the change is measured (see below)

Performance

Ja was trying to back track what caused the hangs, this isn't it :( the integration test harness still doesn't finish


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-27-main" workflow

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

Questions ?
Please refer to the Backport tool documentation


Update Helix languages.toml snippet in leptos_dx.md

Currently, the snippet results in a parsing error when opening Helix.

Thanks!


Command not found?

➜  me git:(master) ✗ gem install yard
As of YARD v0.9.2:
RubyGems "--document=yri,yard" hooks are now supported. You can auto-configure
YARD to automatically build the yri index for installed gems by typing:
$ yard config --gem-install-yri

See yard config --help for more information on RubyGems install hooks.
You can also add the following to your .gemspec to have YARD document your gem
on install:
spec.metadata["yard.run"] = "yri" # use "yard" to build full HTML docs.

Successfully installed yard-0.9.9
1 gem installed
➜  me git:(master) ✗ yard
zsh: command not found: yard

This seems to be a configuration issue with your system. Can you print gem env?

Closing as stale.


Add detail to README for adding a new interview question

Note that the new question .html file should just be copied from an existing one in its respective /questions/technical/ or /question/whiteboard/ folder and modified accordingly to maintain consistency.
Note that the created technical or whiteboard question must be added to the respective whiteboard.html or technical.html file.
Note that the <title> and <h1> tags of the copied file must be changed to represent the new question.
Supported by #33

Thank you @ziyadedher! I think these notes will help our experienced users (who won't be using the VS Code or Atom guides remember every component of what needs to be done when adding a new quesiton.

I can make these changes when I update #22 to include the VS Code link.

Sounds good! That makes sense since it will be in the same section you're already editing. Why don't you plan to take it unless we hear from someone else!


Ignore plength for PID_SENTINEL <1.10.x>

This fixes #1134 by always considering PID_SENTINEL of length 0 (thus correctly following section <IP_ADDRESS> of RTPS)
Signed-off-by: Miguel Company<EMAIL_ADDRESS>

Build status:

Linux
Mac
Windows


Concatenate the segments when the condition is met for better user ex…

Description of changes:
Currently, the panel renders each segment with a new line. This is applicable for short segments as well. To improve the user experience, the change merges the consecutive segments that meets the pre-defined condition and render them in single line.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Before the change

After the change


[Dropdown] Opening dropdown with JS impossible without a data-toggle on another element

Expected Behavior

I AM A DROPDOWN

...
$("#my-dropdown").foundation("open"); // should open the dropdown.
Current Behavior

app.js:1 Uncaught TypeError: t.getBoundingClientRect is not a function
Possible Solution

Adding a data-toggle="my-dropdown" to a random element fixes the problem, but my dropdown is never going to be opened by simply clicking another element, so it's redundant.
Test Case and/or Steps to Reproduce (for bugs)

Test Case: https://jsfiddle.net/SinusPi/pw7vn6Lj/2/
Context

I'm using the Dropdown as a pop up info box, appearing wherever the user interacts with the page. It's positioned dynamically and popped via script, not bound to a button or form field.
Your Environment

Foundation version(s) used: 6.x
Browser(s) name and version(s): Chrome, likely irrelevant
Device, Operating System and version: Windows, ditto
Link to your project:

Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] This is a bug report or a feature request.
[x] There are no other issues similar to this one.
[x] The issue title and template are correctly filled.

Hi @SinusPi,
It seems you use Foundation Sites 6.4.3.
Did you also test this with 6.5.3?

Hi @SinusPi,
It seems you use Foundation Sites 6.4.3.
Did you also test this with 6.5.3?

To create the dropdown trigger, add data-toggle to a . The value of data-toggle is the ID of the dropdown.

Dropdown panes are little happy sprites which can be revealed on click or hover.

https://github.com/zurb/foundation-sites/blob/develop-v6.5/js/foundation.dropdown.js#L119
https://github.com/zurb/foundation-sites/blob/develop-v6.5/js/foundation.positionable.js#L124
https://github.com/zurb/foundation-sites/blob/develop-v6.5/js/foundation.util.box.js#L76
Well, the anchor is needed as the pane is bound to a toggle element.
https://github.com/zurb/foundation-sites/blob/develop-v6.5/js/foundation.dropdown.js#L52

I AM A BAD DROPDOWN
 works as workaround.

I guess it wasn't entirely clear to me that the dropdown NEEDS an anchor and won't function without one, seeing as JS makes it possible to pop the dropdown without any anchor clicking. Very well, "working as intended" it is. Thanks for the clarification. :)
I'll now go figure out how to reanchor the dropdown (already figured that out, basically) AND have it open and close on demand, as it's oddly only appearing on exactly every other .foundation("open") call. Cheers.


Make chunk timeouts less ridiculously tiny

If we want to start looking at performance, we probably don't want to
write out one chunk for every 30s of samples for each series.

Eventually we probably want this to be more like 1 hr?
LGTM

Yeah, depending on how confident we feel in the eventual log recovery mechanism and amount of time we're willing to lose in case it doesn't recover correctly. That should certainly help with efficiency though, yeah.


LTN Combinator integration

hi,
thanks for the new ltn manager.
can you please support ltn combinator like the old ltn tracker. it was possible to change the input signals for every trainstation out of the tracker itself.
thx

This is in the plans for the near future.

+1 for this please. This was the only feature I really used with LTN tracker

It is in!


!Here's way to watch Italy vs Argentina live Streams Free Rugby Internationals On TV Channel, 09 November 2024

Argentina had a hugely successful Rugby Championship, picking up wins against New Zealand, South Africa and Australia.
They entered the final round of the tournament with a chance to pick up their maiden title, but a 48-7 loss to the Springboks saw them take the crown.
Despite finishing fifth, Italy had one of their best Six Nations campaigns in recent years, winning two matches for the first time since 2013 as they beat both Wales and Scotland.
They could have finished as high as third if they had beaten France, who they drew 13-13 with.
Watch Italy v Argentina on TNT Sports via discovery+ Premium monthly pass
The two nations have long had an inextricable link. Two of Italy’s most famous players, Sergio Parisse and Martin Castrogiovanni, with 142 and 119 caps respectively, were both born in Argentina.
Meanwhile, current Italy head coach Gonzalo Quesada won 38 caps for the Pumas and scored 486 points.
RadioTimes.com has rounded up everything you need to know about how to watch Italy v Argentina on TV and online.
When is Italy v Argentina?
Italy v Argentina will take place on Saturday 9th November 2024.
What time is Italy v Argentina kick-off?
Italy v Argentina will kick off at 5:40pm.
Check out the Autumn Internationals TV schedule with our comprehensive guide.
What TV channel is Italy v Argentina on?
Italy v Argentina will be shown live on TNT Sports.
There are multiple ways to get TNT Sports. If you already have BT Broadband, you can add TNT Sports to your existing contract from just £18 per month. You can add the ‘Big Sport’ package for £40 per month which includes all TNT Sports and 11 Sky Sports channels via a NOW pass.
You can also access TNT Sports via discovery+ and stream directly to your smart TV.
Check out the latest deals for TNT Sports
How to live stream Italy v Argentina online
You can watch the match on TNT Sports via discovery+ Premium monthly pass without signing up to a contract.
Regular subscribers can also stream matches on a variety of devices including laptops, smartphones and tablets via the discovery+ app.
discovery+ is the new streaming home of TNT Sports, showing events including live Premier League, UEFA Champions League, Premiership Rugby, UFC, Boxing and MotoGP. Learn more here: discoveryplus.com
Italy v Argentina key player to watch
Tomás Albornoz (Argentina)
The Benetton star is now pulling strings at Test level for Argentina, and his ability has given the side the chance to dictate and control games in their favour, rather than just reacting to their opponents.
Italy v Argentina prediction
While it would be exciting to see Italy pick up a first win over the Argentine since 2008, the Pumas are on a roll after a successful summer, and will be looking to head into clashes with Ireland and France this autumn with one scalp already. They will be favourites to do just that.

fs

fsdfdf


Improve 'Cannot compile modules unless the '--module' flag is provided.' message

I'm fairly new to TypeScript, having been working with it a few days to create d.ts files, in VSCode, for some Node components.
I'd been persistently seeing this message in VSCode for my d.ts files:
Cannot compile modules unless the '--module' flag is provided.
That threw me, at least as a noob, as I wasn't using tsc at all--I was just using IntelliSense.   How was I supposed to provide a --module flag?  Is IntelliSense just buggy?
Later after googling for the error I first learned about the tsconfig.json file--something I didn't learn about previously reading the TypeScript docs.
Anyway, my recommendation would be to update to the message to say something like:
Cannot compile modules unless the '--module' flag is provided.  Do you need to provide a tsconfig.json file?
That would have been a big help for noobs like myself, cluing us into the existence of tsconfig.json and it being the way to fix this up.  Thanks.

Discussed with @mhegazy - we could make the message

Cannot compile modules unless the '--module' flag is provided. Consider setting the 'module' compiler option in a 'tsconfig.json' file.

Or we could conditionally elaborate if in a language-service context, though this might not be appropriate.

Feel free to send a PR for this. The text is in https://github.com/Microsoft/TypeScript/blob/master/src/compiler/diagnosticMessages.json#L438,
to run tests jake runtests and then jake baseline-accept to get all messages update in baselines.

I find that 3 files was deleted when run jake baseline-accept, and I committed it, I'm checking this now.

thanks @plantain-00!

So where does the '--module' flag go in the tsconfig.json file?  A search for that on https://github.com/Microsoft/TypeScript/wiki/tsconfig.json returns too many results...

@timofeysie module in compilerOptions in tsconfig.json

@plantain-00 , so from the Angular2 seed, in the tsconfig.json file, it already has "compilerOptions": {
"module": "commonjs",
...
So how to solve this error?  In the TypeScript wiki it shows things like this:
{
"compilerOptions": {
"out": "test.js",
"allowJs": true
}
}
It would be a good idea to link to a description of this flag in the docs in the error message.

@timofeysie

How did you get the error? from tsc command or IDE's tips?
Which version of typescript or what kind of IDE do you use?
You can go https://gitter.im/Microsoft/TypeScript to get a faster answer.
Once, I got the error from VS Code's tips, so I add module into tsconfig.json, the error is still on, but the tsc command is all right, so I restart the VS Code, then everything is fine.

@plantain-00 Fyi I get the same issue too, see screenshot for a real example.
I'm using VSCode (latest version at time of this post)

@viztastic I think the problem is the files.
According to : https://github.com/Microsoft/TypeScript/wiki/tsconfig.json
If no "files" property is present in a tsconfig.json, the compiler defaults to including all TypeScript (*.ts or *.tsx) files in the containing directory and subdirectories. When a "files" property is present, only the specified files are included.
So try to remove files or include your files in it.


concurrent.futures._base.TimeoutError

I encounter an issue, when I preprocess the rcsb data and run process_mmcif_metadata.py.
I don't know which sample has the problem, how to solve it? thanks!
Traceback (most recent call last):
  File "confdiff/ConfDiff/src/utils/misc/process.py", line 110, in mp_with_timeout
    result = next(result_itr)
  File "/home/ubuntu/anaconda3/envs/confdiff/lib/python3.9/site-packages/pebble/pool/base_pool.py", line 224, in next
    raise result
  File "/home/ubuntu/anaconda3/envs/confdiff/lib/python3.9/site-packages/pebble/pool/base_pool.py", line 264, in chunk_result
    return future.result(timeout=timeout)
  File "/home/ubuntu/anaconda3/envs/confdiff/lib/python3.9/concurrent/futures/_base.py", line 448, in result
    raise TimeoutError()
concurrent.futures._base.TimeoutError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/ubuntu/anaconda3/envs/confdiff/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/ubuntu/anaconda3/envs/confdiff/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "confdiff/ConfDiff/datasets/rcsb/process_mmcif_metadata.py", line 359, in <module>
    main(args)
  File "confdiff/ConfDiff/datasets/rcsb/process_mmcif_metadata.py", line 280, in main
    results = mp_with_timeout(
  File "confdiff/ConfDiff/src/utils/misc/process.py", line 117, in mp_with_timeout
    f"Timeout {error.args[1]} s: {error_report_fn(current_ix)}"
IndexError: tuple index out of range

@llllly26 it seems error.args returns an empty tuple. For a temporarily fix, can you try removing the error massage? It shall print out the current_ix of the timeout data.
Like this:
# line 117: original
f"Timeout {error.args[1]} s: {error_report_fn(current_ix)}"
# line 117: fix
f"Timeout: {error_report_fn(current_ix)}"


dependency compatibility

Current workflow errors
Traceback (most recent call last):
  File "/blue/ewhite/henrysenyondo/conda/envs/EvergladesTools/bin/mapbox", line 5, in <module>
    from mapboxcli.scripts.cli import main_group
  File "/blue/ewhite/henrysenyondo/conda/envs/EvergladesTools/lib/python3.11/site-packages/mapboxcli/scripts/cli.py", line 14, in <module>
    from mapboxcli.scripts import (
  File "/blue/ewhite/henrysenyondo/conda/envs/EvergladesTools/lib/python3.11/site-packages/mapboxcli/scripts/geocoding.py", line 7, in <module>
    import mapbox
  File "/blue/ewhite/henrysenyondo/conda/envs/EvergladesTools/lib/python3.11/site-packages/mapbox/__init__.py", line 11, in <module>
    from .services.static import Static
  File "/blue/ewhite/henrysenyondo/conda/envs/EvergladesTools/lib/python3.11/site-packages/mapbox/services/static.py", line 7, in <module>
    from mapbox.utils import normalize_geojson_featurecollection
  File "/blue/ewhite/henrysenyondo/conda/envs/EvergladesTools/lib/python3.11/site-packages/mapbox/utils.py", line 1, in <module>
    from collections import Mapping, Sequence
ImportError: cannot import name 'Mapping' from 'collections' (/blue/ewhite/henrysenyondo/conda/envs/EvergladesTools/lib/python3.11/collections/__init__.py)
Usage: rio mbtiles [OPTIONS] INPUT [OUTPUT]
Try 'rio mbtiles --help' for help.

Error: No such option: --progress-bar Did you mean --rgba?
Usage: rio mbtiles [OPTIONS] INPUT [OUTPUT]
Try 'rio mbtiles --help' for help.

Error: No such option: --progress-bar Did you mean --rgba?

Two current errors:

Some of the packages import, from collections import Mapping, Sequence the python version has collections in import collections.abc
rio mbtiles  doesn't recognize the --progress-bar

I will do that


Symbolic LU for near-symmetric matrices

This uses a symbolic Cholesky factorization of $A + A^T$ and the numerical factorization to compute a symbolic LU factorization
TODO:

[x] CUDA and HIP kernels
[x] benchmark integration
[x] factory integration

@upsj should this be part of the release?

@MarcelKoch would be nice to have, but not a necessity

@MarcelKoch on ani7, we get a speedup of 3.5x (AMD-reordered) to 18x (input ordering) on ani7_crop on an A100 vs. an Intel Xeon Platinum 8368 (Horeka). So this is definitely worth it.

@upsj that's good to hear! Thanks for checking that out.

@thoasm 32 bit makes it easier to use Csr to pass around data, but at the same time, I believe 32 bit atomics are easier on the memory controller? After the release, I might give it a try to use bytes or even bits, but the latter might make it hard to use more of the available parallelism, since we are merging the dependency structure of a lot of unrelated rows. I'll have to experiment with that.


fix_spin_box_live_updated

The bug with stroke(border)-width changes is fixed and and their changes are now porjected on the being modifying element in a real-time manner. This fixes #17

The bug with stroke-width is now fixed.
"stroke-width" and "stroke-color" are the terms used in the SBML Layout and Render, and I think they are based on CSS. Shall we keep those names or you think it would be better if we change them to 'border-width' and 'border-color'?


Update google.maps.d.ts

Missing overview_polyline in DirectionsRoute as defined in: https://developers.google.com/maps/documentation/javascript/directions

@FalloutPL thanks mate!


Fix grammatical typos in "Hello" blog post

Could you fix other typos in the post too? One i just spotted is this one:

Primarily its a Github organization

Should be "it's"

@ThiefMaster done :cake:


support doorkeeper v3?

Is there any plan to support doorkeeper new version v3?

Yes, it is intended to support doorkeeper v3.
PR's are always welcome.

Closed by #47


Support DGRAM unix domain socket

Currently only STREAM unix domian socket is supported, it would better to support DGRAM unix domain socket.

@kghost we love contribution.s

Looking into it

better to support DGRAM unix domain socket.

Can you provide some justification for this statement? Why is it "better"?

To add a concrete use case for this, in Micrometer we support publishing metrics to dogstatsd (Datadog-flavor of StatsD daemon) over UDP using reactor-netty currently. We have had requests to support publishing metrics to dogstatsd via unix domain sockets in micrometer-metrics/micrometer#792, and I was excited to see that reactor-netty has support for unix domain sockets built on top of the support in netty for it. However, the netty support is limited to stream domain sockets and dogstatsd is listening on a datagram domain socket. Since we are already using reactor-netty for publishing statsd metric lines, it would be advantageous to use a built-in netty feature to also support publishing on (datagram) unix domain sockets.

@shakuzen I am currently super busy but I would be happy to review a PR and provide feedback. So if you want to give it a go I am happy to help

@shakuzen @normanmaurer I'm gonna look at this and will try to provide a PR for a discussion.

I opened a PR #11423 for a discussion. I'll appreciate your feedback.


The web-demo is not giving any responses

Issues
I created a connection for web-demo in Articulate but it's not giving any output, instead it giving out an error. I have attached the screenshot of the error herewith.

Did you download a release or pull from Master? Which release?
version 0.29
What operating system are you running?
Debian 9
Are you running locally or in the cloud?
Cloud
Which browser are you using?
Chrome

Both of your issues are likely related. Do you have any sort of auth on your cloud instance, either basic auth through Nginx or the auth that comes with Articulate?
Also, are there any errors in the browser console or terminal that is running docker. What is the output of docker-compose logs api

No, I have not used any auth on my cloud instance.
The docker-compose logs api gives a 404(Not Found) error message.
I don't have the logs at the moment, I will share them with you later.

can we get the complete 404 error? I was able to reproduce similar behavior, so we're looking into things as well.

The output of docker-compose logs api
Attaching to articulate_api_1_21d6b176578e
api_1_21d6b176578e | 2019/08/30 14:45:53 Waiting for: http://elasticsearch:9200
api_1_21d6b176578e | 2019/08/30 14:45:53 Received 200 from http://elasticsearch:9200
api_1_21d6b176578e |
api_1_21d6b176578e | ><EMAIL_ADDRESS>start /usr/src/app
api_1_21d6b176578e | > cross-env NODE_ENV=development env-cmd server/.env nodemon server --exec babel-node
api_1_21d6b176578e |
api_1_21d6b176578e | [nodemon] 1.19.1
api_1_21d6b176578e | [nodemon] reading config ./package.json
api_1_21d6b176578e | [nodemon] to restart at any time, enter `rs`
api_1_21d6b176578e | [nodemon] or send SIGHUP to 51 to restart
api_1_21d6b176578e | [nodemon] ignoring: ./node_modules/**/*
api_1_21d6b176578e | [nodemon] watching: *.*
api_1_21d6b176578e | [nodemon] watching extensions: js,mjs,json
api_1_21d6b176578e | [nodemon] starting `babel-node server`
api_1_21d6b176578e | [nodemon] spawning
api_1_21d6b176578e | [nodemon] child pid: 64
api_1_21d6b176578e | [nodemon] watching 460 files
api_1_21d6b176578e | Debug: handler, error
api_1_21d6b176578e |     Error: response is not defined
api_1_21d6b176578e |     at Object.<anonymous> (/usr/src/app/lib/channels/web-demo/post.js:33:14)
api_1_21d6b176578e |     at Generator.next (<anonymous>)
api_1_21d6b176578e |     at asyncGeneratorStep (/usr/src/app/lib/channels/web-demo/post.js:7:103)
api_1_21d6b176578e |     at _next (/usr/src/app/lib/channels/web-demo/post.js:9:194)
api_1_21d6b176578e |     at process._tickCallback (internal/process/next_tick.js:68:7)

So, the 404(Not Found) error is only when I try to connect the chat-widget.
The web-demo error is quite different.

Closed by https://github.com/samtecspg/articulate/commit/9c3b1f6958d8d5dcb3c153b9ded73cddc6c56f52


sleigh: access to program counter

I am writing sleigh module for Bosch/Infineon C166 controllers.
Now I faced with the following problem.
There is EXTP (extended page) instruction, that modifies addressing for next load/store instructions. EXTP has a parameter, that indicates, how much instructions are affected (any instructions are affected, not only dealing with memory ones).
Problem is complicated by the fact that C166 instructions have variable lenght, and inst_next * const not exactly matches end of EXTP action.
Are there any suggestions, how to solve this trouble?

I will continue this topic.
I found delayslot operator.
With conjunction with context variables it should do what I need.
But, now I have another question: how to change context in constructor's body ?
# extended addressing fake register
define register offset=0x10 size=4	[ extAddressing ];
define context extAddressing
	extR = (31,31)
	extP = (30,30)
	extOffset = (0,23)
;

macro ext_p(page, irange) {
#	extP = 1; ### it not working !!
#	extOffset = page << 14;
	delayslot(irange);
#	extP = 0;
}
:extp #u1627, #irange is op0007=0xD7 & irange & op1415=1 ; u1627 & op2831=0 {
	ext_p(u1627, irange);
}
:extp r0811, #irange is op0007=0xDC & r0811 & irange & op1415=1 {
	ext_p(r0811, irange);
}

# carry extP flags in load/store instructions ...

Now I found one bad way, but it seems to be work:
extp set up couner (1, 2, 3, 4) in context variable with attribute noflow and any other instructions decrease it.

Context variables can only be changed inside the "disassembly action" portion of a constructor (between the square braces '[' and ']'). They can be changed locally with a direct assignment or globally using the globalset directive.  See sections 8.2 "Local Context Change and 8.3 "Global Context Change" in the SLEIGH documentation.

Chris, thanks, I already found this info.
May be you will tell best way to model next behavior:
extp sets internal flag (I think, this flag fit to context register), execute N instructions (they may use this flag, but may be not) and then resets flag.

@esaulenka I am trying to extend the C166 library for C167 processors, and are encountering the same issue here....The compiler Siemens uses for the C167 makes use of extr instructions continuously extr #01.
I was able to make extr instructions work via context changes to the extR fake register, but then that change is permanent.
Is there a way at all to make a context change again after an instruction is parsed?

@rnd-ash, I tried to use another approach there. Sorry, I forgot to push it after local tests.
In short, I added additional counter, that set by extp #X and decreases by all others instructions. If this counter is not zero, it changes the behavior of all memory access instructions.
But just few notes: first of all, this module is not complete, all memory accesses should be reworked somehow (now it produces weird code in most cases). And second one - I tested it on XC27xx MCU, which is successor of original C166/C167, so all C167 instructions should be already there.
And the last - I think will be better to discuss it on that particular project page instead of this common one.


MINIFICPP-1251 - Implement and test RetryFlowFile processor

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

I still don't see any properties being required, otherwise looks good.

@hunyadi-dev : could you rebase and push?
The changes look good to me, but I would like to see it build on Win without errors.

Looks good to me, thanks!
@adamdebreceni @szaszm could you get back to this and check if your comments are properly replied/handled?
I only left a single reply comment, no reviews. I got a reply, so don't feel blocked by me. :)


Setup hangs when execution Policy is set to allsigned via GPO.

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

Set (Or deploy via Domain Controller GPO) the Powershell Core Policy: “Turn on Script Execution” to “Enabled” and “Allow only Signed Scripts”
Execute the installer in the system context with “msiexec.exe /package PowerShell-7.1.4-win-x64.msi /quiet ADD_EXPLORER_CONTEXT_MENU_OPENPOWERSHELL=1 ENABLE_PSREMOTING=1 REGISTER_MANIFEST=1”
Msi.exe will never exit. MSI log indicates it is waiting on the register manifest script. Pwsh.exe is running in the background and never exits because it is waiting on a prompt shown in the mail below.

Expected behavior
Setup should not hang due to unsigned components.

Actual behavior
Setup hangs and never terminates until PWSH.exe is terminated.

Error details
Do you want to run software from this untrusted publisher?
File C:\program files\powershell\7\Modules\PSReadLine\PSReadLine.format.ps1xml is published by CN=Microsoft
Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US and is not trusted on your system. Only run scripts
from trusted publishers.
 Help (default is "D"): A
PS C:\Windows\System32>

Environment data
Name                           Value
----                           -----
PSVersion                      7.1.4
PSEdition                      Core
GitCommitId                    7.1.4
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

@travisgr Can you confirm this for 7.2 Preview version too?

Any update?
Issue persists with 7.4.6

Any update?
Issue persists with 7.4.6

We manage updating the deployments through our software manager... we have it run reg.exe to change the Execution Policy to RemoteSigned, deploy the new version, then run reg.exe to set it back.


Feature: Show slides partly on the edges like in Netflix

Is it possible to show only a part of the slide on the outside like in Netflix? This way users get a visual clue that it's a carousel without the need of the pagination dots or navigation arrows.
If it's not possible yet, would be great to get an option for it. Thanks for your work on this great package.

Hi, @quinnlangille @lggwettmann , I have made a PR # 165 to meet this need. This isn't impossible, but we truly have to do some css tricks.

Oops, it's really awkward! I didn't read the API carefully before. With the ready-made API we could make it.
.add("Custom width", {
  components: { Carousel, Slide },
  render: h => {
    return createContainer(
      h, containerWidth, [h(Carousel, {
        props: {
          perPage: 1,
          spacePadding: 10
        }
      }, [new Array(8).fill(0).map((item, index) => {
        return h(Slide, {}, [h("div", {
          style: {
            width: "100%",
            height: "400px",
            lineHeight: "400px",
            color: "#fff",
            textAlign: "center",
            fontSize: "30px",
            backgroundColor: (index % 2 === 0) ? "#42b983" : "#ff3c3c"
          }
        }, [index])])
      })])]
    )
  }
})

Just a heads up, this was resolved with the release of v0.6.14 - you're now able to use the spacePadding prop to enable the look ahead style slider like Netflix. Thanks for making the request 👍

Wow, awesome @quinnlangille Thanks so much.


File name wrong

Thanks for your great work! There is a small error, which is your file's name save_swim_features.py(swin)

true. Thanks for pointing that out.

closed


[crypto] Implement display for our keys

At the moment our logs are riddled with the following output:
EdwardsPoint{
	X: FieldElement51([986312278720011,<PHONE_NUMBER>38697,<PHONE_NUMBER>07985,<PHONE_NUMBER>230999,<PHONE_NUMBER>00966]),
	Y: FieldElement51([1122918900483925,<PHONE_NUMBER>462405,<PHONE_NUMBER>172011,<PHONE_NUMBER>28558,<PHONE_NUMBER>548988]),
	Z: FieldElement51([1, 0, 0, 0, 0]),
	T: FieldElement51([167659880001565,<PHONE_NUMBER>91364,<PHONE_NUMBER>765074,<PHONE_NUMBER>10468,<PHONE_NUMBER>44488])
}))

We should make sure we have a decent Display/Debug implemented for all public key types, especially as we're about to embed those data types that can contain keys of several types (for user keys) : we will want to e.g. derive / reuse Display on the Sui side.
Conversely, for the PrivateKey type, we should make sure our implementations of Debug / Display never display the byte content of the keys, but rather some "" sort of string.

It totally makes sense, thank a bunch for the recommendation


Allow InvoiceFormatter implementation to be customized

This PR provides a way for a custom InvoiceFormatter implementation to be used at runtime. It does this by introducing a new InvoiceFormatterFactory API and then looking up an OSGi service for that at runtime. If a service is available, the factory will be used to create the InvoiceFormatter instance. Otherwise the existing DefaultInvoiceFormatter class is used.
To allow other plugins to use the InvoiceFormatterFactory class at runtime, and then extend the existing support classes, this plugin now exports these packages in OSGi:
org.killbill.billing.plugin.notification.api
org.killbill.billing.plugin.notification.generator
org.killbill.billing.plugin.notification.generator.formatters

To see an intended use case, I've extended an existing custom InvoiceFormatter project that is used to customize the HTML invoice generation done internally in Killbill so that it can work with the email-notifications-plugin as well. Its BundleActivator registers a SolarNetworkInvoiceFormatterFactory service.
Finally, I've added a new test case that verifies the factory is used as expected.

Merged as https://github.com/killbill/killbill-email-notifications-plugin/pull/41.


Add support for adventure components in custom argument exceptions

Allow using translatables, click events, hover events and other component stuff.

Looks great, thanks!

Wouldn't this change break on normal Spigot servers or is CommandAPI now only targeted at Paper and its forks?
BukkitTooltip.messageFromAdventureComponent() does not break on Spigot because if it's never called the classes aren't going to be loaded, but now that an adventure component is a class member, does adventure have to be on the classpath or only if the component constructor is called?

Wouldn't this change break on normal Spigot servers or is CommandAPI now only targeted at Paper and its forks?

Tested it, and yep, it breaks. The problem is actually with compiling though, and not with running.
I made this simple test command in my plugin class:
new CommandAPICommand("test")
        .withArguments(
                new CustomArgument<>(new IntegerArgument("even"), (info) -> {
                    int i = info.currentInput();
                    if(i%2 != 0) throw new CustomArgument.CustomArgumentException(
                            new CustomArgument.MessageBuilder("Int must be even: ")
                                    .appendFullInput().appendHere()
                    );
                    return i;
                })
        ).executes(info -> {
            info.sender().sendMessage(info.args().get(0).toString());
        })
        .register();

This doesn't use the Component constructor. If I try to compile this without a dependency for Paper, I get this error:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project CommandAPITest: Compilation failure
[ERROR] Main.java:[31,40] cannot access net.kyori.adventure.text.Component
[ERROR]   class file for net.kyori.adventure.text.Component not found

[31,40] is pointing to just before the new keyword in new CustomArgument.CustomArgumentException.... So I would guess the compiler is trying to load all of the possible constructors that could be referred to here, then gets mad about a constructor it can't understand.
If I add a dependency for Paper, the code can compile (probably because it knows about Component). Interestingly, I can run the resulting plugin on a Spigot server just fine:
>test 3
[15:31:13] [Server thread/INFO]: Int must be even: test 3<--[HERE]
>test 2
[15:31:17] [Server thread/INFO]: 2

So it seems Java only attempts to load the class while compiling, not running. Still, not great. I would say this change breaks the CustomArgumentException on Spigot servers. Probably fixable though.

Because the constructor takes either a dynamic Builder String or a static String/AdventureComponent/BungeeComponent, what if the CustomArgumentException now only takes the builder string or a static Brigadier Message instance that has to be provided by the developer instead (Using BukkitTooltip for example)?

what if the CustomArgumentException now only takes the builder string or a static Brigadier Message instance that has to be provided by the developer instead
This won't work, as brigadier is also not present at compile-time

There is no really good solution to this, I thought CommandAPI bundled adventure
It would be possible to change the Component type to Object, and then only check for Components in the "toCommandSyntax" method, but I don't think that would be a good solution

While not the most elegant solution, I using static methods may solve this:
static CustomArgumentException of(Component errorMessage)

This still doesn't work, as the of() method is also declared multiple times like the constructor was previously.
The compiler can't find the Component class while trying to figure out which method should be used, so the of() methods should be renamed (like it was done previously with the fail methods).

Really? Worked when I tried this...

Class:
import dev.jorel.commandapi.arguments.CustomArgument;
import net.md_5.bungee.api.chat.TextComponent;

public class Testing {

    public static void main(String[] args) {
        CustomArgument.CustomArgumentException.of("");
        CustomArgument.CustomArgumentException.of(TextComponent.fromLegacyText(""));
        CustomArgument.CustomArgumentException.of(new CustomArgument.MessageBuilder(""));
    }
}

Compiler output:
[project path]/src/main/java/Testing.java:7: error: cannot access Component
        CustomArgument.CustomArgumentException.of("");
                                              ^
  class file for net.kyori.adventure.text.Component not found

Using org.spigotmc:spigot-api:1.19.4-R0.1-SNAPSHOT and dev.jorel:commandapi-bukkit-core:9.0.1 (from maven local) as dependencies.

Amended on dev/dev to have different names for the static factory methods.


VersionParser does not handle versions like "parquet-mr 1.6.0rc4"

I'm not sure how we missed this one (I thought I added a test for this but apparently not :p) but the tests don't cover it, and the parser can't parse it even though the intention was to support a missing (build abcd) section in the version string.
Reporter: Alex Levenson / @isnotinvain
Assignee: Reuben Kuhnert / @sircodesalotOfTheRound
Related issues:

[Python] Allow parquet::WriterProperties::created_by to be set via pyarrow.ParquetWriter for compatibility with older parquet-mr (relates to)

PRs and other links:

PR-283

Note: This issue was originally created as PARQUET-349. Please see the migration documentation for further details.

Reuben Kuhnert / @sircodesalotOfTheRound:
Added patch (PR-283).

Arjun:
@rdblue Is it possible to make this fix available in parquet 1.8.1 as well ?  The fix is currently available in 1.9 and 1.8.2 . Apache Hive use parquet 1.8.1 and this issue is causing log flooding for parquet 1.6 files in Hive.

Ryan Blue / @rdblue:
[~arjun_kr], releases are immutable, so we don't make changes to them. Since this is in the 1.8.2 release, I'd recommend updating to that or asking your Hadoop vendor to make sure they backport this to their distribution.

Ryan Blue / @rdblue:
Merged PR #283. Thanks for fixing this, Reuben!


feat: ignore future executables

Description
Currently, there is no check for added executables and they clutter the repo by including files from the build. This PR adds new lines to the .gitignore file to only include files with type extension or folders.
Linked PRs

#28 included lukso executable

closed by #36

Closed by #36


Add support for Cloud Management Service (cis) as a Destination in cloudsdk

Is your feature request related to a problem? Please describe.

I am trying to use the cloud sdk for accessing the Accounts Service API.
When I try to run the function

        // Step 1 : Create Sub Account
        const response = await SubaccountOperationsApi.createSubaccount({
                displayName: "sub-account-name",
                region: "eu10",
                subdomain: "sub-account-name"
            },
            {
                subaccountAdmin: "xxxxx"
            }
        ).execute({
            destinationName: "cloud-management-service-central"
        })

I am not completely sure what I must add to the VCAP services, but by checking the documentation I believe that there is no support for the credentials given by the cloud-management-service.
The binding looks something like this (The secret values are hidden) :

{
  "endpoints": {
    "accounts_service_url": "https://accounts-service.cfapps.eu10.hana.ondemand.com",
    "cloud_automation_url": "https://cp-formations.cfapps.eu10.hana.ondemand.com",
    "entitlements_service_url": "https://entitlements-service.cfapps.eu10.hana.ondemand.com",
    "events_service_url": "https://events-service.cfapps.eu10.hana.ondemand.com",
    "external_provider_registry_url": "https://external-provider-registry.cfapps.eu10.hana.ondemand.com",
    "metadata_service_url": "https://metadata-service.cfapps.eu10.hana.ondemand.com",
    "order_processing_url": "https://order-processing.cfapps.eu10.hana.ondemand.com",
    "provisioning_service_url": "https://provisioning-service.cfapps.eu10.hana.ondemand.com",
    "saas_registry_service_url": "https://saas-manager.cfapps.eu10.hana.ondemand.com"
  },
  "grant_type": "client_credentials",
  "sap.cloud.service": "com.sap.core.commercial.service.central",
  "uaa": {
    "apiurl": "https://api.authentication.eu10.hana.ondemand.com",
    "clientid": "xxxx",
    "clientsecret": "xxxx",
    "credential-type": "binding-secret",
    "identityzone": "xxxx",
    "identityzoneid": "xxxx",
    "sburl": "https://internal-xsuaa.authentication.eu10.hana.ondemand.com",
    "subaccountid": "xxxx",
    "tenantid": "xxxx",
    "tenantmode": "shared",
    "uaadomain": "authentication.eu10.hana.ondemand.com",
    "url": "https://xxxx.authentication.eu10.hana.ondemand.com",
    "verificationkey": "-----BEGIN PUBLIC KEY-----\nXXXX\n-----END PUBLIC KEY-----",
    "xsappname": "xxxx",
    "xsmasterappname": "xxxx",
    "zoneid": "xxxx"
  }
}

The error that I get

[2022-11-03T21:17:25.895Z] WARN     (environment-accessor): No services of type destination found! This might cause errors in other parts of the application.
Error: Unable to get access token for "destination" service. No service instance of type "destination" found.
    at resolveService (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/connectivity/src/scp-cf/environment-accessor.ts:201:13)
    at serviceToken (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/connectivity/src/scp-cf/token-accessor.ts:41:27)
    at Function.getProviderServiceToken (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/connectivity/src/scp-cf/destination/destination-from-service.ts:268:39)
    at Function.getDestinationFromDestinationService (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/connectivity/src/scp-cf/destination/destination-from-service.ts:166:45)

Describe the solution you'd like

The solution should be that I should be able to use the execute function with the right parameter without an error.
This means that we would need a transform function in service-binding-to-destination.ts
In this function I must provide only the URL and the remaining credentials can be automatically taken.

Describe alternatives you've considered

What I tried

        const serviceBindingTransformFn = async (service: any) => ({
            name: service.name,
            label: service.label,
            credentials: {
                ...service.uaa,
            },
            url: service.endpoints.accounts_service_url || ""

        });

I created a transform function, but somehow it does not seem to work, and I get an error that says [AxiosError: Request failed with status code 404]
I have tried the entire flow using postman > to get the token and then fetch the actual URL, but somehow I am unable to make it work in cloud-sdk.

Impact / Priority

Impact is low.

Affected development phase: e.g. Getting Started, Development, Release, Production

Trying to automate BTP operations using cloud sdk.

Timeline: Currently, we manually create sub-accounts, and we are already live. But it would be great if we can automate this :)
Additional context
Not 100% sure if this is a feature request, or me not being able to use the API.
Please let me know in the comments, Thanks 👍

Hi @shardulsonar,
As you already realized, for a custom service, you can use the serviceBindingTransformFn to transform the Service to a Destination object. In addition to the url, you need a client_credential token to call the destination. Refer to the implementation here and use the serviceToken() to fetch the access token build your destination.
Regarding the 404 error, one probable cause might be that the service url called is wrong. To check that, can you please do the following:
As a first step, get the destination separately, using the destinationForServiceBinding function:
const serviceBindingTransformFn = async (service: Service) => {
  const transformedService = {
    ...service,
    credentials: { ...service.uaa }
  };
  const token = await serviceToken(transformedService);
 // build and return the destination
};

const destination = await destinationForServiceBinding('cloud-management-service-central', {
  serviceBindingTransformFn ,
});

Then, execute the service using the executeRaw(destination) method. This gives you more details about the actual request. Please verify if the request url is correct.
const response = await SubaccountOperationsApi.createSubaccount({
              displayName: "sub-account-name",
              region: "eu10",
              subdomain: "sub-account-name"
          },
          {
              subaccountAdmin: "xxxxx"
          }
      ).executeRaw(destination);

Regards,
Deeksha

Hello @deekshas8, Thanks for the help with the hint I was able to make some progress.
Here are a few points that I noticed:

Regarding fetching the serviceToken :

const token = await serviceToken(transformedService)

This fails because it for some reason expects xsuaa credentials, which obviously are not created. This is because I am trying to create a new sub-account so I can never have one.

Full Error Message :

Error: No binding to an XSUAA service instance found. Please make sure to bind an instance of the XSUAA service to your application.
    at selectXsuaaInstance (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/src/connectivity/scp-cf/environment-accessor.ts:262:11)
    at getXsuaaServiceCredentials (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/src/connectivity/scp-cf/environment-accessor.ts:211:7)
    at multiTenantXsuaaCredentials (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/src/connectivity/scp-cf/token-accessor.ts:156:33)
    at /Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/src/connectivity/scp-cf/token-accessor.ts:46:17
    at step (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/dist/connectivity/scp-cf/token-accessor.js:44:23)
    at Object.next (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/dist/connectivity/scp-cf/token-accessor.js:25:53)
    at /Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/dist/connectivity/scp-cf/token-accessor.js:19:71
    at new Promise (<anonymous>)
    at __awaiter (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/dist/connectivity/scp-cf/token-accessor.js:15:12)
    at serviceToken (/Users/I540081/dev/btp-api/node_modules/@sap-cloud-sdk/core/dist/connectivity/scp-cf/token-accessor.js:70:12)

Not really sure why this occurs but I m guessing some mis-configuration in VCAP Services.
My VCAP Services :

{
    "cis": [
        {
            "name": "cis-central",
            "endpoints": {
                "accounts_service_url": "https://accounts-service.cfapps.sap.hana.ondemand.com",
                "cloud_automation_url": "https://cp-formations.cfapps.sap.hana.ondemand.com",
                "entitlements_service_url": "https://entitlements-service.cfapps.sap.hana.ondemand.com",
                "events_service_url": "https://events-service.cfapps.sap.hana.ondemand.com",
                "external_provider_registry_url": "https://external-provider-registry.cfapps.sap.hana.ondemand.com",
                "metadata_service_url": "https://metadata-service.cfapps.sap.hana.ondemand.com",
                "order_processing_url": "https://order-processing.cfapps.sap.hana.ondemand.com",
                "provisioning_service_url": "https://provisioning-service.cfapps.sap.hana.ondemand.com",
                "saas_registry_service_url": "https://saas-manager.cfapps.sap.hana.ondemand.com"
            },
            "grant_type": "client_credentials",
            "sap.cloud.service": "com.sap.core.commercial.service.central",
            "uaa": {
                "clientid": "xxxxx",
                "clientsecret": "xxxx",
                "url": "https://xxxxx.authentication.sap.hana.ondemand.com",
                "identityzone": "xxxxx",
                "identityzoneid": "xxxx",
                "tenantid": "xxxx",
                "tenantmode": "shared",
                "sburl": "https://internal-xsuaa.authentication.sap.hana.ondemand.com",
                "apiurl": "https://api.authentication.sap.hana.ondemand.com",
                "verificationkey": "-----BEGIN PUBLIC KEY-----\nMIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA53MxvBPOZ3dj7s6oMxs2\niSI9ZLJtfweqMC/lDUDdaRHKOXWMKXNaX4TZ4fWGL5xNpVkobTgkjMtiBTaWYGFt\nkW6UBj+AU7Xq64WaXAuH962VtmHmrq3/e19Et+TfgHd+VREPzFzbPfh86pYh1lkz\nwyc4R2pIFZwt1e6P58AeZEnwP0XlqAqmViL/2KIA+cHcHzPh6sjQMchQaonI45cc\nurkmeT0r7BVSBskc4cZjCJXQj+bgJ1qMTzj4VDMpeq3SBolzcpJfG7BQyUrbHUPn\nOkpzubvW70vnnisweWcE4LlajxcaUhP5IMI9ZEYl/7ixSpT55xZw1uj298/baT05\nawIDAQAB\n-----END PUBLIC KEY-----",
                "xsappname": "xxxxx",
                "subaccountid": "xxxx",
                "uaadomain": "authentication.sap.hana.ondemand.com",
                "zoneid": "xxxx",
                "credential-type": "binding-secret",
                "xsmasterappname": "cis-central!b13"
            }
        }
    ]
}

Ideally I would expect the serviceToken function to give me a token with the credentials I have provided (url, clientId, clientSecret).
To get around this I implemented my own getToken function. which works correctly and provides me with a token.

Regarding using the destinationForServiceBinding function, a slight modification worked. If I do not add the ts-ignore, ts compiler fails to recognize that both return the same value.  Error message is TS2322: Type '(service: any) => Promise<Destination>' is not assignable to type '(serviceBinding: ServiceBinding) => Destination'.   Property 'url' is missing in type 'Promise<Destination>' but required in type 'Destination'.

        const destination : Destination= await destinationForServiceBinding('cis-central', {
            // @ts-ignore
            transformationFn: serviceBindingTransformFn
        });

TL; DR;
Your suggestion seemed to have worked but then I had to write the serviceToken function on my own.


Troubleshooting oauth client

- Do you want to request a feature or report a bug?
Might be a bug, might be a feature
- What is the current behavior?
While implementing an OAuth auth backend for Github, I'm seeing what might be an issue either with Redux state or inter-window communication.
My code:
https://github.com/marksteele/netlify-serverless-oauth2-backend
The HTML bits are copied from the other implementations (go/python/nodejs), and the HTML output appears to succeed, however the auth window does not close and the app state doesn't update to reflect the successful auth. Might be a bug in my code...
- If the current behavior is a bug, please provide the steps to reproduce.
- What is the expected behavior?
Auth window closes, react app updates with token from auth success.
- Please mention your versions where applicable.
netflify 1.5.0
https://www.control-alt-del.org/admin/
- Please link or paste your config.yml below if applicable.
https://www.control-alt-del.org/admin/config.yml

I see the CMS version being printed to the console twice, which means two separate CMS instances are being created. That may have something to do with it.

I've removed the duplicate instance, same result.

So fundamentally the problem is that the authenticate function in netlify-auth.js is appending '/auth' to base_url. This breaks when the you want the authentication url to be different from the site root.
In my example, my base url for authentication is https://www.control-alt-del.org/oauth/{auth|callback}
Add a configuration setting 'authentication_base' to the backend config settings, and update the comparisons in handshakeCallback and authorizeCallback to use that instead of base_url.

I think I'm having this same problem but I can't work out a way around it. I have everything set up exactly, as far as I can tell, like this, other than my auth_endpoint which didn't work with /oauth included in it. I'm using Jekyll, but everything else is copied and pasted from @marksteele except for my keys and secrets.
Code here: bakerkretzmar/madihaslam
Live site here: madihaslam.ca
My Netlify config.yml:
publish_mode: editorial_workflow
backend:
  name: github
  repo: bakerkretzmar/madihaslam
  base_url: https://tva0emnu7j.execute-api.ca-central-1.amazonaws.com/prod
  # auth_endpoint: /auth
  # site_domain: madihaslam.ca
# site_id: madihaslam.ca
media_folder: "uploads"
collections: etc...

The GitHub authentication works, the app gets permission to use my account, and then the authorization popup just sits there. I can see Sending message: "github" in the console, and the URL it ends up at is https://tva0emnu7j.execute-api.ca-central-1.amazonaws.com/prod/callback?code=66df4c70b0510ba7d9d3&state=HRWVCbyrKIFD93OhGz1A598pEhGADLwm, which is the token that Netlify needs, right?
My serverless deploy is working, my parameters and KMS keys are working as far as I can tell, but the CMS itself doesn't seem to know that I've authorized it.
I hope this is the right place to post this! Thanks!

@bakerkretzmar did you managed to get it working?

@pbrandone nope, but in the meantime they added support for GitLab so I switched to that backend with implicit grant and that's working great.

Hi all, that still remains an issue though :(


The clang compiler does not support '-march=nehalem'

Bug report

[X] ran brew update and can still reproduce the problem?
[X] ran brew doctor, fixed all issues and can still reproduce the problem?
[X] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
I want to build macvim with homebrew under macOS Catalina 10.15.7 with Xcode 12.2 and CLT 12.2 on Late 2013 Macbook Pro Retina machine. I use the following command:
brew install markwu/personal/macvim
(Current homebrew-core/macvim.rb need some patch to make it compiled with xcode 12.2, so, I made this one for tesing this issue)
What happened (include command output)
It build failed with errors. It seems clang does not support  '-march=nehalem' anymore.

Command output

brew reinstall macvim
Warning: Treating macvim as a formula. For the cask, use homebrew/cask/macvim
==> Cloning https://github.com/macvim-dev/macvim.git
Updating /Users/markwu/Library/Caches/Homebrew/macvim--git
==> Checking out branch master
Already on 'master'
Your branch is up to date with 'origin/master'.
HEAD is now at 0a1cf2e22 Merge remote-tracking branch 'vim/master'
==> Reinstalling markwu/personal/macvim
==> Patching
==> ./configure --with-features=huge --enable-multibyte --enable-perlinterp --enable-rubyinterp --enable-tclinterp --enable-terminal --with-tlib=ncurses --with-compiledby=Homebrew --with-loca
==> make
Last 15 lines from /Users/markwu/Library/Logs/Homebrew/macvim/02.make:
ProcessPCH /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/SharedPrecompiledHeaders/SharedPrecompiledHeaders/4270105491631976141/AppKit.h.gch /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk/System/Library/Frameworks/AppKit.framework/Headers/AppKit.h normal arm64 objective-c com.apple.compilers.llvm.clang.1_0.compiler (in target 'PSMTabBarControlFramework' from project 'PSMTabBarControl')
cd /tmp/macvim-20201117-43642-41h37l/src/MacVim/PSMTabBarControl
clang -x objective-c-header -target arm64-apple-macos10.15 -fmessage-length=0 -fdiagnostics-show-note-include-stack -fmacro-backtrace-limit=0 -Wno-trigraphs -fpascal-strings -Os -Wno-missing-field-initializers -Wno-missing-prototypes -Wno-implicit-atomic-properties -Wno-objc-interface-ivars -Wno-arc-repeated-use-of-weak -Wno-missing-braces -Wparentheses -Wswitch -Wno-unused-function -Wno-unused-label -Wno-unused-parameter -Wunused-variable -Wunused-value -Wno-empty-body -Wno-uninitialized -Wno-unknown-pragmas -Wno-shadow -Wno-four-char-constants -Wno-conversion -Wno-constant-conversion -Wno-int-conversion -Wno-bool-conversion -Wno-enum-conversion -Wno-float-conversion -Wno-non-literal-null-conversion -Wno-objc-literal-conversion -Wno-shorten-64-to-32 -Wpointer-sign -Wno-newline-eof -Wno-selector -Wno-strict-selector-match -Wno-undeclared-selector -Wno-deprecated-implementations -DDISABLE_SPARKLE=1 -DOBJC_OLD_DISPATCH_PROTOTYPES=1 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk -fstrict-aliasing -Wprotocol -Wdeprecated-declarations -Wno-sign-conversion -Wno-infinite-recursion -Wno-comma -Wno-block-capture-autoreleasing -Wno-strict-prototypes -Wno-semicolon-before-method-body -iquote /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-generated-files.hmap -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-own-target-headers.hmap -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-all-target-headers.hmap -iquote /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-project-headers.hmap -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/Release/include -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources-normal/arm64 -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources/arm64 -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources -F/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/Release -c /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk/System/Library/Frameworks/AppKit.framework/Headers/AppKit.h -MD -MT dependencies -MF /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/SharedPrecompiledHeaders/SharedPrecompiledHeaders/4270105491631976141/AppKit.h.d -iquote /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-generated-files.hmap -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-own-target-headers.hmap -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-all-target-headers.hmap -iquote /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/PSMTabBarControl-project-headers.hmap -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/Release/include -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources-normal/arm64 -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources/arm64 -I/tmp/macvim-20201117-43642-41h37l/src/MacVim/build/PSMTabBarControl.build/Release/PSMTabBarControlFramework.build/DerivedSources -o /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/SharedPrecompiledHeaders/SharedPrecompiledHeaders/4270105491631976141/AppKit.h.gch --serialize-diagnostics /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/SharedPrecompiledHeaders/SharedPrecompiledHeaders/4270105491631976141/AppKit.h.dia
clang: error: the clang compiler does not support '-march=nehalem'
warning: Traditional headermap style is no longer supported; please migrate to using separate headermaps and set 'ALWAYS_SEARCH_USER_PATHS' to NO. (in target 'MacVim' from project 'MacVim')
warning: Traditional headermap style is no longer supported; please migrate to using separate headermaps and set 'ALWAYS_SEARCH_USER_PATHS' to NO. (in target 'QuickLookStephen' from project 'QuickLookStephen')
** BUILD FAILED **
The following build commands failed:
ProcessPCH /tmp/macvim-20201117-43642-41h37l/src/MacVim/build/SharedPrecompiledHeaders/SharedPrecompiledHeaders/4270105491631976141/AppKit.h.gch /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk/System/Library/Frameworks/AppKit.framework/Headers/AppKit.h normal arm64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
make[1]: *** [macvim] Error 65
make: *** [first] Error 2
If reporting this issue please do so at (not Homebrew/brew or Homebrew/core):
https://github.com/markwu/homebrew-personal/issues
Please create pull requests instead of asking for help on Homebrew's GitHub,
Twitter or any other official channels.

What you expected to happen
Build without errors.
Step-by-step reproduction instructions (by running brew commands)
brew install markwu/personal/macvim
Output of brew config and brew doctor commands

$brew config
HOMEBREW_VERSION: 2.5.11-20-gaf41431-dirty
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: af4143150da32097f751b1538a5b6f6958bb2175
Last commit: 46 minutes ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 723b3d0d68fddbc3b4d468db8c11e3bee6f8469a
Core tap last commit: 47 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_DISPLAY: /private/tmp/com.apple.launchd.MUWa2Waego/org.macosforge.xquartz:0
HOMEBREW_EDITOR: vim
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit haswell
Clang: 12.0 build 1200
Git: 2.29.2 => /usr/local/bin/git
Curl: 7.64.1 => /usr/bin/curl
Java: 14.0.2
macOS: 10.15.7-x86_64
CLT: <IP_ADDRESS>.1.1604628099
Xcode: 12.2
XQuartz: 2.7.11 => /opt/X11

$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed formulae are deprecated or disabled.
You should find replacements for the following formulae:
  rclone

Warning: "config" scripts exist outside your system or Homebrew directories.
`./configure` scripts often look for *-config scripts to determine if
software packages are installed, and which additional flags to use when
compiling and linking.

Having additional scripts in your path can confuse software installed via
Homebrew if the config script overrides a system or Homebrew-provided
script of the same name. We found the following "config" scripts:
  /Users/markwu/.phpbrew/php/php-7.4.12/bin/php-config

Warning: Unbrewed header files were found in /usr/local/include.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected header files:
  /usr/local/include/python3.8/greenlet/greenlet.h
  /usr/local/include/python3.9/greenlet/greenlet.h

Warning: You have uncommitted modifications to Homebrew/brew.
If this is a surprise to you, then you should stash these modifications.
Stashing returns Homebrew to a pristine state but can be undone
should you later need to do so for some reason.
  cd /usr/local/Homebrew && git stash && git clean -d -f

Uncommitted files:
   M Library/Homebrew/hardware.rb

Reference and Solutions
I finally found someone has the same problem with me https://github.com/Homebrew/brew/issues/7857#issuecomment-655536261.
After remove -march=nehalem from homebrew source code, everything works well. I report the issue there and @claui ask me to file a new seperate issue and told me it might relate to this change https://github.com/Homebrew/brew/commit/374c734d9b3f8e6fa20810ad4d049ca2711e2156
That why this issue here.
I also report the issue in macvim https://github.com/macvim-dev/macvim/issues/1115

I can't reproduce this. Do you have a more minimal example?
% clang --version
Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin20.1.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

% echo "int main(){return 0;}" | clang -xc - -march=undefined
error: unknown target CPU 'undefined'
note: valid target CPU values are: nocona, core2, penryn, bonnell, atom, silvermont, slm, goldmont, goldmont-plus, tremont, nehalem, corei7, westmere, sandybridge, corei7-avx, ivybridge,
      core-avx-i, haswell, core-avx2, broadwell, skylake, skylake-avx512, skx, cascadelake, cooperlake, cannonlake, icelake-client, icelake-server, tigerlake, knl, knm, k8, athlon64,
      athlon-fx, opteron, k8-sse3, athlon64-sse3, opteron-sse3, amdfam10, barcelona, btver1, btver2, bdver1, bdver2, bdver3, bdver4, znver1, znver2, x86-64

% echo "int main(){return 0;}" | clang -xc - -march=nehalem

% echo $?
0

(@markwu Don’t worry about the issue being closed, it’s our way to track pending work.
The issue can be reopened if you provide a reproducible example.)

Note: #9160 is an unrelated PR but will affect CPU family logic for M1.

@jonchang I can't reproduce your example step by step either.  Weird.
I also try to see the clang options from the above error messages and did not see any -march=nehalem either.
So, I try to copy the clang options from the about error messages and add -march=nehalem in the end of options, and see what's going on.
test.c
int main(void) {
    return 0;
}

clang command
clang -x objective-c-header -target arm64-apple-macos10.15 -fmessage-length\=0 -fdiagnostics-show-note-include-stack -fmacro-backtrace-limit\=0 -Wno-trigraphs -fpascal-strings -Os -Wno-missing-field-initializers -Wno-missing-prototypes -Wno-implicit-atomic-properties -Wno-objc-interface-ivars -Wno-arc-repeated-use-of-weak -Wno-missing-braces -Wparentheses -Wswitch -Wno-unused-function -Wno-unused-label -Wno-unused-parameter -Wunused-variable -Wunused-value -Wno-empty-body -Wno-uninitialized -Wno-unknown-pragmas -Wno-shadow -Wno-four-char-constants -Wno-conversion -Wno-constant-conversion -Wno-int-conversion -Wno-bool-conversion -Wno-enum-conversion -Wno-float-conversion -Wno-non-literal-null-conversion -Wno-objc-literal-conversion -Wno-shorten-64-to-32 -Wpointer-sign -Wno-newline-eof -Wno-selector -Wno-strict-selector-match -Wno-undeclared-selector -Wno-deprecated-implementations -DDISABLE_SPARKLE\=1 -DOBJC_OLD_DISPATCH_PROTOTYPES\=1 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk -fstrict-aliasing -Wprotocol -Wdeprecated-declarations -Wno-sign-conversion -Wno-infinite-recursion -Wno-comma -Wno-block-capture-autoreleasing -Wno-strict-prototypes -Wno-semicolon-before-method-body -march=nehalem test.c -o test

I just get
error messages
clang: error: the clang compiler does not support '-march=nehalem'

I am not familiar with c/c++, not sure if this is what you want. I guess there might be some clang options conflicts, but have no idea the conflicts come from macvim side or homebrew side.
But, in xcode 11.x, It just works without any problems.

@jonchang  Okay, I try to remove most options, except target
clang -target arm64-apple-macos10.15 -march=nehalem test.c -o test

Maybe these two conflicts.

You're trying to cross-compile targeting arm64, but there's no microarchitecture called nehalem for that target. The issue is with the build script or custom formula you're using, not with Homebrew.

@jonchang Thanks for your explanation! But, why the same formula works with xcode 11.x and failed after I upgrade to xcode 12.2. That's I don't understand.

reference: https://github.com/Homebrew/homebrew-core/pull/65100

Hi @jonchang
I just check all homebrew compile logs, from `~/Library/Logs/Homebrew/macvim/01.configure.cc, it shows
superenv added:    -pipe -Os -march=nehalem -isysroot/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk --sysroot=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk -isystem/usr/local/include -isystem/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.0.sdk/System/Library/Frameworks/OpenGL.framework/Versions/Current/Headers<EMAIL_ADDRESS>-I/usr/local/opt/readline/include -I/usr/local/opt/sqlite/include -I/usr/local/opt/ruby/include

It seems the -march=nehalem is added by brew superenv, not macvim.

Okay, I think I found problem, there is another project under macvim, and it does not exclude arm arch. I'll do some test to see if it works.
Thanks!

@jonchang & @claui Thanks, the problem solved. It is the problem of macvim xcode project buildsettings. Atfer some patches, the problem solved.


data retrievel from satelite in spacestation

I'm able to retrieve the data from a satellite from the overworld, but not from the spacestation.
When orbiting another station is says correctly 'to far'. But when in the right system I can see it's data. however the connect button simply does nothing.

Wanted to verify it wasn't the overworld by launching them around another planet. Got this and a server crash as extra:

Description: Ticking entity
java.lang.NullPointerException: Ticking entity
at zmaster587.advancedRocketry.api.satellite.SatelliteBase.setDimensionId(SatelliteBase.java:111)
at zmaster587.advancedRocketry.satellite.SatelliteData.setDimensionId(SatelliteData.java:76)
at zmaster587.advancedRocketry.dimension.DimensionProperties.addSatallite(DimensionProperties.java:732)
at zmaster587.advancedRocketry.entity.EntityRocket.unpackSatellites(EntityRocket.java:897)
at zmaster587.advancedRocketry.entity.EntityRocket.onOrbitReached(EntityRocket.java:791)
at zmaster587.advancedRocketry.entity.EntityRocket.func_70071_h_(EntityRocket.java:677)
at net.minecraft.world.World.func_72866_a(World.java:1964)
at net.minecraft.world.WorldServer.func_72866_a(WorldServer.java:839)
at net.minecraft.world.World.func_72870_g(World.java:1934)
at net.minecraft.world.World.func_72939_s(World.java:1750)
at net.minecraft.world.WorldServer.func_72939_s(WorldServer.java:620)
at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:709)
at net.minecraft.server.dedicated.DedicatedServer.func_71190_q(DedicatedServer.java:387)
at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:613)
at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:471)
at java.lang.Thread.run(Thread.java:745)

I havn't landed yet on the planet. Might that have something to do with it?

So it turns out that at least in the 1.10 builds the satellite terminals do not check for effective ID when the button is pressed even though they do for the insertion of the chip.  Fixed for next build


Queue k8s watcher updates

We currently process updates from k8s synchronously, processing may invoke blocking kvstore operations which can take a long time. Instead, build a queue of events and process them in background go subroutines.

[ ] Maintain a queue of received events for each resource type.
[ ] Make a deep copy
[ ] Process events of each resource type in a background go subroutine which is allowed to block for a long time.
[ ] Add a timeout for the processing (2 minutes?) and cancel event processing.

Maintain a queue of received events for each resource type.

Each watcher already has its own queue, can't we use that queue plus the timeout?

Each watcher already has its own queue, can't we use that queue plus the timeout?

Are you talking about the queue inside the controller? Looking at how Kubernetes itself handles event notifications, it queues up events and handles them async in a handler to make sure that the controller loop can continue receiving. We don't do this right now, we execute all of the required actions directly in the handler as below and require the controller to wait until the operations have been completed.
        ciliumNetworkPolicyHandler := cache.ResourceEventHandlerFuncs{
                AddFunc:    d.addCiliumNetworkPolicy,
                UpdateFunc: d.updateCiliumNetworkPolicy,
                DeleteFunc: d.deleteCiliumNetworkPolicy,
        }

Each controller contains an internal queue. I'll add a new queue for each resource type to have a keep retry feature.


Quickfix for database connection

To be honest, I'm not sure why this is working - but it is. If you check the quickfix review application you'll see that it can communicate with its database.
Developer
Secrets

[ ] All new secrets have been added to Pantheon tiers
[ ] Relevant secrets have been updated in Github Actions
[ ] All new secrets documented in README

Documentation

[ ] Project documentation has been updated
[ ] No documentation changes are needed

Accessibility

[ ] ANDI or Wave has been run in accordance to
our guide and
all issues introduced by these changes have been resolved or opened as new
issues (link to those issues in the Pull Request details above)

Stakeholder approval

[ ] Stakeholder approval has been confirmed
[ ] Stakeholder approval is not needed

Dependencies
YES | NO dependencies are updated
Code Reviewer

[ ] The commit message is clear and follows our guidelines
(not just this pull request message)
[ ] The changes have been verified
[ ] The documentation has been updated or is unnecessary
[ ] New dependencies are appropriate or there were no changes

This is the wrong merge proposall, closing.