Features and Pump Stats

Any chance you can add pump features, GPMs, RPMs and watts to the ISY nodes server?  I could decommission my Arduino sensor if I could use those as nodes.  Well, features as nodes and the others just into ISY variables.  Intellibrite colors and Intellichlor control would be awesome too! Thanks!

Of course it hasn't crashed again today or yesterday.  Once it does (if) I'll let you know.
Thanks,
Brent

From: Brian Treese<EMAIL_ADDRESS>Sent: Thursday, January 31, 2019 4:49 PM
To: brianmtreese/nodejs-pool-controller-polyglotv2
Cc: btreinders; Author
Subject: Re: [brianmtreese/nodejs-pool-controller-polyglotv2] Features and Pump Stats (#3)
When this happens are there any errors occurring in the nodejs pool
controller app or is it just the polyglot node server?
On Thu, Jan 31, 2019, 2:43 PM btreinders<EMAIL_ADDRESS>wrote:

Well, I have always started both but this time I just restarted the nodejs
pool controller and it never reconnected until I restarted the node server
too. Next time I'll try just the node server first.
Thanks!

From: Brian Treese<EMAIL_ADDRESS>Sent: Thursday, January 31, 2019 8:47 AM
To: brianmtreese/nodejs-pool-controller-polyglotv2
Cc: btreinders; Author
Subject: Re: [brianmtreese/nodejs-pool-controller-polyglotv2] Features and
Pump Stats (#3)
This is strange, it looks like it's a communication error with "circuit
5". Do you have to restart the nodejs pool controller app too?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/brianmtreese/nodejs-pool-controller-polyglotv2/issues/3#issuecomment-459370882>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/ARwMLm4uaRRThmxkVSKtFxNpXcCmOMr_ks5vIwIfgaJpZM4aXIQR

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/brianmtreese/nodejs-pool-controller-polyglotv2/issues/3#issuecomment-459535274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIuChfvvBKUJcjSw5UlaNU512T468TtBks5vI3F_gaJpZM4aXIQR
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/brianmtreese/nodejs-pool-controller-polyglotv2/issues/3#issuecomment-459537141, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARwMLiHYwQLz3eJWUvf9d_jxQ_ZDQW_Cks5vI3MVgaJpZM4aXIQR.

Had another crash, no errors from the poolController, restarted just the node server and it worked.
What I saw from the log for the node server.
Driver controller - CLITEMP: 50, uom: 17 2019-02-09 15:31:32,825 [Controller] [INFO ] Updating Driver controller - CLITEMP: 51, uom: 17 2019-02-09 15:40:53,567 [Controller] [INFO ] Updating Driver controller - CLITEMP: 50, uom: 17 2019-02-09 15:55:24,931 [Controller] [ERROR] Exception in thread Controller: Traceback (most recent call last): File "/home/pi/.local/lib/python3.5/site-packages/urllib3/connectionpool.py", line 600, in urlopen chunked=chunked) File "/home/pi/.local/lib/python3.5/site-packages/urllib3/connectionpool.py", line 384, in _make_request six.raise_from(e, None) File "", line 2, in raise_from File "/home/pi/.local/lib/python3.5/site-packages/urllib3/connectionpool.py", line 380, in _make_request httplib_response = conn.getresponse() File "/usr/lib/python3.5/http/client.py", line 1198, in getresponse response.begin() File "/usr/lib/python3.5/http/client.py", line 297, in begin version, status, reason = self._read_status() File "/usr/lib/python3.5/http/client.py", line 258, in _read_status line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1") File "/usr/lib/python3.5/socket.py", line 576, in readinto return self._sock.recv_into(b) ConnectionResetError: [Errno 104] Connection reset by peer During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/home/pi/.local/lib/python3.5/site-packages/requests/adapters.py", line 449, in send timeout=timeout File "/home/pi/.local/lib/python3.5/site-packages/urllib3/connectionpool.py", line 638, in urlopen _stacktrace=sys.exc_info()[2]) File "/home/pi/.local/lib/python3.5/site-packages/urllib3/util/retry.py", line 367, in increment raise six.reraise(type(error), error, _stacktrace) File "/home/pi/.local/lib/python3.5/site-packages/urllib3/packages/six.py", line 685, in reraise raise value.with_traceback(tb) File "/home/pi/.local/lib/python3.5/site-packages/urllib3/connectionpool.py", line 600, in urlopen chunked=chunked) File "/home/pi/.local/lib/python3.5/site-packages/urllib3/connectionpool.py", line 384, in _make_request six.raise_from(e, None) File "", line 2, in raise_from File "/home/pi/.local/lib/python3.5/site-packages/urllib3/connectionpool.py", line 380, in _make_request httplib_response = conn.getresponse() File "/usr/lib/python3.5/http/client.py", line 1198, in getresponse response.begin() File "/usr/lib/python3.5/http/client.py", line 297, in begin version, status, reason = self._read_status() File "/usr/lib/python3.5/http/client.py", line 258, in _read_status line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1") File "/usr/lib/python3.5/socket.py", line 576, in readinto return self._sock.recv_into(b) urllib3.exceptions.ProtocolError: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer')) During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/usr/lib/python3.5/threading.py", line 914, in _bootstrap_inner self.run() File "/usr/lib/python3.5/threading.py", line 862, in run self._target(*self._args, **self._kwargs) File "/home/pi/.local/lib/python3.5/site-packages/polyinterface/polyinterface.py", line 769, in _parseInput self.shortPoll() File "./pool-poly.py", line 81, in shortPoll self.nodes[node].update() File "./pool-poly.py", line 194, in update self.get_status() File "./pool-poly.py", line 201, in get_status circuitData = requests.get(url='{0}/circuit/{1}'.format(self.apiBaseUrl, self.number)) File "/home/pi/.local/lib/python3.5/site-packages/requests/api.py", line 75, in get return request('get', url, params=params, **kwargs) File "/home/pi/.local/lib/python3.5/site-packages/requests/api.py", line 60, in request return session.request(method=method, url=url, **kwargs) File "/home/pi/.local/lib/python3.5/site-packages/requests/sessions.py", line 533, in request resp = self.send(prep, **send_kwargs) File "/home/pi/.local/lib/python3.5/site-packages/requests/sessions.py", line 646, in send r = adapter.send(request, **kwargs) File "/home/pi/.local/lib/python3.5/site-packages/requests/adapters.py", line 498, in send raise ConnectionError(err, request=request) requests.exceptions.ConnectionError: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer'))

From: Brent Reinders<EMAIL_ADDRESS>Sent: Saturday, February 2, 2019 6:58 AM
To: brianmtreese/nodejs-pool-controller-polyglotv2
Subject: Re: [brianmtreese/nodejs-pool-controller-polyglotv2] Features and Pump Stats (#3)
Of course it hasn't crashed again today or yesterday.  Once it does (if) I'll let you know.
Thanks,
Brent

From: Brian Treese<EMAIL_ADDRESS>Sent: Thursday, January 31, 2019 4:49 PM
To: brianmtreese/nodejs-pool-controller-polyglotv2
Cc: btreinders; Author
Subject: Re: [brianmtreese/nodejs-pool-controller-polyglotv2] Features and Pump Stats (#3)
When this happens are there any errors occurring in the nodejs pool
controller app or is it just the polyglot node server?
On Thu, Jan 31, 2019, 2:43 PM btreinders<EMAIL_ADDRESS>wrote:

Well, I have always started both but this time I just restarted the nodejs
pool controller and it never reconnected until I restarted the node server
too. Next time I'll try just the node server first.
Thanks!

From: Brian Treese<EMAIL_ADDRESS>Sent: Thursday, January 31, 2019 8:47 AM
To: brianmtreese/nodejs-pool-controller-polyglotv2
Cc: btreinders; Author
Subject: Re: [brianmtreese/nodejs-pool-controller-polyglotv2] Features and
Pump Stats (#3)
This is strange, it looks like it's a communication error with "circuit
5". Do you have to restart the nodejs pool controller app too?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<
https://github.com/brianmtreese/nodejs-pool-controller-polyglotv2/issues/3#issuecomment-459370882>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/ARwMLm4uaRRThmxkVSKtFxNpXcCmOMr_ks5vIwIfgaJpZM4aXIQR

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/brianmtreese/nodejs-pool-controller-polyglotv2/issues/3#issuecomment-459535274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIuChfvvBKUJcjSw5UlaNU512T468TtBks5vI3F_gaJpZM4aXIQR
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/brianmtreese/nodejs-pool-controller-polyglotv2/issues/3#issuecomment-459537141, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARwMLiHYwQLz3eJWUvf9d_jxQ_ZDQW_Cks5vI3MVgaJpZM4aXIQR.


Controlling Randomness in PSharpTester

One can control the random seed of PSharpTester via /sch-seed flag but this doesn't guarantee  consistent behavior across different machines (or even CLR versions).
The only solution I see is to have our own implementation of Random.

Turns out Random wasn't the reason. It was because collections were being sorted on addresses.
Fixed via 562c439


(wip) Flat tree

This is a rough first pass for the flat-tree chapter. Providing a bit of insight about how to think about them, and some general interesting properties they have.
At the moment it's mostly a work in progress, but hopefully it's useful enough as it is.
Before merging I'd like to give it another pass to make it more of a coherent story, and also add information about parents, uncles and sibling nodes, and update the glossary to include these terms.
Thanks!

Added a section about node relationships. Think this should be good as a first pass!

Live here: https://datprotocol.github.io/book/ch01-01-flat-tree.html


Setup code coverage reporting

#46
this is what the coverage report looks like for this branch: https://app.codecov.io/gh/dh-tech/hackathon-2022/tree/coverage/undate

Thanks for the reviewing @ColeDCrawford!
I was thinking that my changes conflicted with @maltevogl's setup config changes branch and that his should be merged first so I can reconcile the differences.


Add support for configuring escalated plugins via envvar.

Related PRs:
https://github.com/drone/drone-plugin-go/pull/5
https://github.com/drone/drone-exec/pull/20
https://github.com/drone/drone-go/pull/17

 You should sign our Contributor License Agreement in order to get your pull request merged.

@tboerger I've updated per your comments.


[Feature]: Download selected rows, data available in the viewport or all data

[x] I should be able to select all rows #6430
[ ] I should be able to select rows per pages; My previous selections should persist
[ ] Download in multi select mode should only be for the selected rows
[ ] I should be able to select specific columns

@riodeuno need you help here understanding how should this be archived

For reference

@somangshu Should this work with server side pagination as well? If so, we'll need to have a technical discussion around this. If not, then this seems quite straightforward in implementation. Please discuss with me, if we have any queries about the code design.

Any news about this feature :  I should be able to select rows per pages; My previous selections should persist ?
If not ready, any tips to help doing this ?
Thank you for your help.

Hey @sevenGroupFrance, there is a workaround for the use case you are talking about.
You can use the appsmith store for achieving something like this. You have to use the combination of onRowSelected and defaultSelectedRow props on the table widget to achieve this.

Use onRowSelected to store the row primary key in a array.
use the store array in defaultSelectedRow


Table arrows do not disappear

If I enable table-arrows, clicking on column 1 shows an arrow as expected.  However, clicking on column 2 shows the arrow for column 2 and column 1.  I think that only one arrow should be visible at a time.

@ccapndave which version is this? I did changes in the latest release for custom table arrows, so it might be a regression. Can you compare v1.21.0 to v1.22.1 ;I don't remember exactly what the behaviour was... 🛌🏻

Thanks for the quick response!  This is on 1.22.1.  I checked on 1.12.0 and the behaviour is the same.

@ccapndave I don't think the table arrows should disappear?, however if clicking on another column it should reset all table arrows to the neutral arrow before setting the arrow up/down on the column being sorted (I'm guessing this is what you want???); however, the current implementation should still apply when the "remember-sort" tag is applied.


Login Notice: migrate to TinyMCE

#660

@ChristianMurphy Are you sure you aren't getting confused with GPL? We already use LGPL libraries in openEQUELLA (JPF for example).

Are you sure you aren't getting confused with GPL?

@doolse not confused, LGPL is on the Category X list of banned licenses.
https://apache.org/legal/resolved.html#category-x

We already use LGPL libraries in openEQUELLA (JPF for example).

That's something that should be discussed and remediated.

@ChristianMurphy Are you sure that Apache projects in this case means all Apache licensed products, and not official Apache projects?

@ChristianMurphy wrote:

That's something that should be discussed and remediated.

Totally agree. It's on the CDM agenda for next week. So how's about we continue for now and discuss there? Maybe in prep for the CDM @cbeach47 can also check in with the usual legal people. As apparently back at the start of the project a green light was given for things like JPF, Hibernate, etc.
Sorry all for opening a can of worms.

Merging as discussed with @doolse and @edalex-ian in the office.


Add support for LOCAL_CARTESIAN_ORIENTED in jacobian computation

This is to fix #773.
This is NOT done yet but there are a few points I thought it's better to get feedback on earlier then later.
I haven't tested the code yet. The code compute the frame orientation I've copied from some other internal code we have, but I don't know if the jointJacobian code is working as expected.
A few things:

Does the name LOCAL_CARTESIAN_ORIENTED make sense?
I have to use a temporary J_tmp to avoid allocations (this was a problem we ran into with tsid). Though I am not 100% sure yet if using a local J_tmp and resizing works around the memory allocation problem. Does someone has an idea how to write the SE3 rotation of the jacobian without doing the toActionMatrix() * J matrix product - e.g. in a form where no new resulting matrix must be allocated and the results can be directly stored in J.

I believe the local J_tmp does not solve the allocation problem. One way of doing it would be declaring it as static, I think, but it would be really non-standard for Pinocchio. Normally, we put temporary variables inside Data.
I am pretty sure there is a way to avoid that variable, but one would need to study the algorithm better

Don't make it static. It would break thread-safety.
You can allocate a vector of fixed size 6 and do the matrix product manually, column-wise on the Jacobian.

I didn't check Eigen's code, but that might be done in Eigen::Matrix::applyOnTheLeft (or applyOnTheRight, I never know to who the left and right applies...)

Yes, I would do as @jmirabel says

@jviereck, thanks a lot for your contribution, really appreciate it.
To answer your question about temporaries: @jmirabel's answer is certainly correct, but I think the best way to implement what you want is to think about it as a global frame which is translated. This will save you computations and will spare you the need of any temporary variable.
You can just copy-paste the code used for LOCAL. Only, intead of
J_.col(j) = oMframe.actInv(Motion(data.J.col(j))).toVector();

you will have
J_.col(j).segment<3>(Motion::LINEAR) -= oMframe.translation().cross(J_.col(j).segment<3>(Motion::ANGULAR);

equivalent to
J_.col(j) = SE3(Eigen::MatrixXd::Identity(), oMframe.translation()).actInv(Motion(data.J.col(j))).toVector();

but without the intermediate SE3 object and without all redundant multiplications.
I think LOCAL_CARTESIAN_ORIENTED is quite long, I would favor a more compact name, such as simply CARTESIAN, but I am neutral about it.

@jviereck LOCAL_CARTESIAN_ORIENTED seems not as obvious as naming.
Maybe LOCAL_ALIGN_WITH_WORLD or LOCAL_WORLD_ALIGNED may be better. Any other ideas are welcome.

Maybe LOCAL_ALIGN_WITH_WORLD or LOCAL_WORLD_ALIGNED may be better. Any other ideas are welcome.

+1 for LOCAL_WORLD_ALIGNED

hi folks,
thanks for all the feedback and input.

LOCAL_WORLD_ALIGNED

+1 for this one.

J_.col(j).segment<3>(Motion::LINEAR) -= oMframe.translation().cross(J_.col(j).segment<3>(Motion::ANGULAR));

Ohh, I haven't thought about computing the Jacobian in this way - that looks like the much better way to go and I am going to adjust the code.

In addition, I would suggest making unitary test to be sure that what you've implemented is correct.

Agreed. I am traveling over the weekend and plan to work on this during the ride.

@jviereck Seems good to me.

Hi folks, I've implemented your feedback and added a simple unit test. However, the unit test is not passing. I am not sure yet what is going on and going to take  a closer look later.

Hi folks, the commit just pushed fixes the unit test. The issue was that I queried the frame before calling forwardFrames etc in the unittest. After switching the order, this unit test works now.
For the joint jacobian, what's the frame orientation the LOCAL joint jacobians are expressed in? Are the LOCAL joint jacobians already expressed in the world oriented frame?

For the joint jacobian, what's the frame orientation the LOCAL joint jacobians are expressed in? Are the LOCAL joint jacobians already expressed in the world oriented frame?

The LOCAL means that the Jacobian is expressed in the frame of the joint directly. While data.J corresponds to the collection of the joint spatial spaces expressed at the origin of the world, aligned with the wold axes.

@jviereck Thanks a lot for your contribution.

hi @jcarpent, thanks for merging this PR! However, I am not sure this PR is done yet. There are no unit tests for getJointJacobian(..., LOCAL_WORLD_ALIGNED).

The LOCAL means that the Jacobian is expressed in the frame of the joint directly. While data.J corresponds to the collection of the joint spatial spaces expressed at the origin of the world, aligned with the wold axes.

Thanks for this clearification!
I tried to write a similar unit test to the getFrameJacobian one. However, when I compared the result between the result from LOCAL_WORLD_ALIGNED and manually rotating the WORLD jacobian (as in the getFrameJacobian unit test), the resulting Jacobians did not match.
Any idea why this could be the case?

This is my mistake. I was thinking you were done.
Feel free to open a new PR and add the other unit tests.

I tried to write a similar unit test to the getFrameJacobian one. However, when I compared the result between the result from LOCAL_WORLD_ALIGNED and manually rotating the WORLD jacobian (as in the getFrameJacobian unit test), the resulting Jacobians did not match.
Any idea why this could be the case?

The WORLD jacobian is expressed at the origin of the WORLD. You then need a shift.
You need to rotate the jacobian computing in the LOCAL frame to retrieve the expected results.
Is it clear enough?


Add babel-jest to peerDependencies

Summary
Resolves https://github.com/kulshekhar/ts-jest/issues/2749
Test plan

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1011

Relevant Lines:
1037

💛  - Coveralls


Trigger prod cli release for v0.17.1

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/lgtm
/approve

/unhold


[MATH] Confusion about displayOperatorMinHeight

The description of displayOperatorMinHeight has:

Minimum height of n-ary operators (such as integral and summation) for formulas in display mode (that is, appearing as standalone page elements, not embedded inline within text).

But it has long been a source of trouble since Cambria Math sets it to a value that when respected results in too small display operators being selected: https://github.com/w3c/mathml-core/issues/126
My own testing shows that Microsoft Word does not use displayOperatorMinHeight at all (setting it to any value makes no difference in which size of display operators is selected) and instead it seems to use the value of delimitedSubFormulaMinHeight to select the display operators size.
I’m speculating here, but I think it is a bug in Microsoft implementation and the wrong value in Cambria Math went unnoticed because of it.
I’m not sure what the spec should do here, but may be at least Cambria Math should be updated to use a reasonable displayOperatorMinHeight so that implmentations that follow the spec don’t have to implement various workarounds that render displayOperatorMinHeight almost useless.

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

ID: f0076fd2-52b0-2989-3142-fd9e456b9514
Version Independent ID: 4aef8fa3-2ad3-7933-5a77-0a7473e66112
Content: MATH - The mathematical typesetting table (OpenType 1.9) - Typography
Content Source: typographydocs/opentype/spec/math.md
Service: typography
GitHub Login: @PeterCon
Microsoft Alias: alib

I checked with Office engineers who confirmed that displayOperatorMinHeight is used in math "Display" mode, but not in inline mode.

Ah, but tracing back to code where the font data is being read, it appears Office may be reading displayOperatorMinHeight and delimitedSubFormulaMinHeight in the wrong order, which would account for @khaledhosny 's comment above:

instead it seems to use the value of delimitedSubFormulaMinHeight to select the display operators size.

@PeterCon Thanks for the feedback! (that seems consistent with previous discussion with Murray Sargent where apparently displayOperatorMinHeight and delimitedSubFormulaMinHeight were mixed up). So what's the plan forward here? Do you plan to fix Cambria Math? To fix Microsoft's implementation?

My role in this is in maintaining the spec. I don't have a proposed remedy at this point. I have pointed out what I discovered to Office engineers and get their input on a proposed remedy.
If they don't want to make a code change, then that could suggest they might want a spec change, reversing the order of those fields. I would want to be very cautious, though, as I don't know what other existing implementations that could break.
Regarding Cambria Math, do I understand correctly that it has the delimitedSubFormulaMinHeight value set to what one would otherwise expect for displayOperatorMinHeight so that it provides the desired behaviour in Office (given the current bug)? If values are changed in Cambria Math, then older and newer versions of that font could behave differently. Desktop Office will use local fonts installed in the host OS, and a font fix might not get serviced in older OS versions.

Thank you @PeterConstable ; my understanding is that Cambria Math and Microsoft Word definitely have the parameters inverted compared to the spec. However, other math fonts and math rendering engines do follow the spec, albeit they have to somehow work around Microsoft's bug. I believe changing the spec would cause more harm than good.
Probably the solution would be to make sure that all the low-level APIs exposing MathConstants follow the spec, except when reading parameters from Cambria Math (up to a certain font version, if the bug is eventually fixed) where it would invert delimitedSubFormulaMinHeight / displayOperatorMinHeight values.


[JSON] Automatically escape UTF8 in JSON.

This pull request adds automatic transforming C++ style string to JSON compliant strings (using the /u escape sequence) during serialization and back on deserialization.
{
    // UTF8 for the violin music key: (byte array) f0 9d 84 9e => CodePoint 0x1D11E => UTF16 0xD834 0xDD1E
    string input = R"("Violin key sending: \uD834\uDD1E")";
    Core::JSON::String json;
    string received;

    json.FromString(input);
    json.ToString(received);
    EXPECT_STREQ(input.c_str(), received.c_str());
}
{
    string input = R"("Control: [25][\u0019] Character: [28][\u001C]")";
    Core::JSON::String json;
    string received;

    json.FromString(input);
    json.ToString(received);
    EXPECT_STREQ(input.c_str(), received.c_str());
}

@neeraj-deshpande oke but than indeed you discovered a bug. Let me fix it before we merge this..


deps.zig not present and errors if submoduling

https://github.com/truemedian/zfetch/blob/bae100ecec339738851e84dd699fd351d4cb9471/build.zig#L20
Importing is always done even if I'm not using the build function, causing it to error out with deps.zig not found
I just deleted the else branch right now, not sure how this is handled gracefully.

Now that the stage1 compiler runs astcheck over the entire codebase regardless of use, I'll need to provide an empty deps.zig for it to quietly ignore.


Why the result of drop_path be divided by the keep_prob?

https://github.com/rwightman/pytorch-image-models/blob/3f9959cdd28cb959980abf81fc4cf34f32e18399/timm/models/layers/drop.py#L140
Why the result of drop_path is divided by the keep_prob?
In my opinion, It is executed for each sample? Drop one path for one sample shouldn't have influence on another sample.
If you want to use division, this should be used on the shortcut branch, but not on the residual branch.

@liu09114 that was the implementation for original efficientnet paper https://github.com/tensorflow/tpu/blob/298d1fa98638f302ab9df34d9d26bbded7220e8b/models/official/efficientnet/utils.py#L276
The goal was to reproduce that so I kept it. It has no impact for inference, but you could argue for training it does better preserve the batch stats.
I recall that for the NFNet paper/impl they don't scale by default, https://github.com/deepmind/deepmind-research/blob/master/nfnets/base.py#L202 ... I was planning to try this at some point side-by-side but haven't had a chance.

Thanks, your reply solved my problem.


UX Cleanup

Render size warnings now show the set size in the warning message.
The preload script attribute works on a library-level singleton.

#318


Werror seems to be missing on ci

We have an unused variable.
https://github.com/jfalcou/eve/blob/915d4bf26b7da374532c656db83d334aa79b06e8/include/eve/detail/function/simd/x86/friends.hpp#L167
Which means CI doesn't have warnings enabled well enough :(

Which configuration cause locally, the toolchain do add the options.

We checked and options are indeed passed


translations PR 20191216

Ideally this PR should be review by community members for each language.
Note: In the event that you see an english translation instead of the desired language is because this PR was submitted before the translation was made on the Mattermost Translation Server and probably will be submitted for the next PR.

1.26 is released to the app stores, published soon. We'll have a new PR submitted for v1.27 next week. Closing.


assignment 1 sql code and diagram uploaded

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Added diagram and code for A1
What did you learn from the changes you have made?
SQL commands, how to navigate and interact with tables.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Different types of joins (when trying to complete an inner join)
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Accidently uploaded ot wrong branch and had to reset.
How were these changes tested?
Locally
A reference to a related issue in your repository (if applicable)
Checklist

[ X] I can confirm that my changes are working as intended

closing due to wrong branch selected (uoft-dsi main)


vboot: use dasharo-sdk v1.2.0

needed for resigning images with Vboot cbfs integration

Draft because it throws an error:
m@nv~/D/D/d/dasharo-tools> ./vboot/resign pcengines_apu6_v0.9.0.rom 8k                                                                                main!?


Order events by creation date not id.

MongoDB IDs are not guaranteed to be monotonic if subsequent records are created the same second
(http://stackoverflow.com/a/24886642/580346). To maintain correct event order we need to sort using created_at field.

In fact created_at is not enough with multiple mongoid clients, as it's mongoid (not the server) that sets created_at. So instead of rely on created_at we use BSON::Timestamp set on the server.


Resolve returns null on every exception

catch
            {
                return null;
            }

This part caused me trouble, because unity threw a valid Exception which was swallowed by the Resolver.

Sorry for the slightly late reply ;) Yes, I can't remember why I wrote it that way. If you are using the library to resolve controllers with dependencies (which the the major usage pattern) then it will raise an exception if there are any problems. If you are using the DependencyResolver as a service locator then you will need to check for nulls in the case of missing registrations.

Found the reason:


Exclude model from Models list

Content & configuration
Swagger/OpenAPI definition:
I use Swashbuckle.AspNetCore v3.0.0 / Swagger-Ui v3.17.1
Is your feature request related to a problem?
I need to exlude (i.e. do not show) model in Models list. As our partners do not use AspNet.Core, they don't need to know anything about IFormFile interface which is specific to AspNet.Core.
Describe the solution you'd like
It will be very useful for us, to decorate the method parameter with some attribute to exclude IFormFile from rendering.
public async Task Post(**[IgnoreModel]**IFormFile document)
Additional context

For the specific example you gave, there's a broader issue for handling IFormFile parameters better on the backlog - #193
For the general case, you should just create a Document Filter (see readme) that removes the unwanted models from the Definitions dictionary:
// Startup.cs
services.AddSwaggerGen(c =>
{
    ...

    c.DocumentFilter<RemoveBogusDefinitionsDocumentFilter>();
}

// RemoveBogusDefinitionsDocumentFilter.cs
public class RemoveBogusDefinitionsDocumentFilter : IDocumentFilter
{
    public void Apply(SwaggerDocument swaggerDoc, DocumentFilterContext context)
    {
        swaggerDoc.Definitions.Remove("IFormFile");
    }
}

Thanks, it helped. Just curious why Swashbuckle do not have an option for this?

For others looking for a solution that is more dynamic or generic, like the OP was originally hoping for, I've written a document filter which can access a custom attribute you can write using reflection and then remove the definitions which are a match (based on type name).
https://github.com/domaindrivendev/Swashbuckle/issues/178#issuecomment-546161898

And how do I remove only the model, without impacting the actions of the controler. I observe the withdrawal of attributes from the model a lot, but if you want to remove the model from the documentation.
technology employed:
I use Swashbuckle.AspNetCore v3.0.0 / Swagger-Ui v3.17.1
Thank you very much in advance


D: Fix Broadwell-tailoring for LDC compiler

See https://github.com/hanabi1224/Programming-Language-Benchmarks/issues/304#issuecomment-1278025108.

Fixed PR based on this #310


Separate dotfiles from scripts

I want to store scripts in ~/.local/scripts with a git repo
I want to store dotfiles in ~/.dotfiles with a git repo
Currently ~/.local is part of the dotfiles repo <-- this is the root problem
Remedy:
Keep the two separate.

Remove ~/.local from git repo without locally deleting ~/.local
Move ~/scripts to ~/.local/scripts

Just remember that zshrc depends on scripts

strike all that.
move scripts to ~/.dotfiles/bin/.local/scripts under the same repo as dotfiles,
remove ~/.local from dotfiles repo without deleting the files.


Temporarily disable snap grab icon

In #200 the snapping panel was introduced, which gives much finer control over snapping angles.
This results in the quick snap button on the controller no longer functioning, however it's visuals still appeared on the controller model.
This is a quick fix to prevent users getting confused.
(The tutorial regarding this interaction will be removed in #419)

Before

After


PHP feature should optionally install xdebug

I personally want to use the (production) docker image for my app where I install PHP (and even composer) already, so I just need the option to install xdebug via the features flags, instead of baking this into my Dockerfile.
Would greatly expand the usability for PHP developers of devcontainers.
Thank you!
R

Hello 👋
From 👇  looks like we already install xdebug with the PHP Feature. This is also baked into the mcr.microsoft.com/devcontainers/php image hosted from https://github.com/devcontainers/images/tree/main/src/php
https://github.com/devcontainers/features/blob/f91ff3cd8de5d33a52d21887963ba7166600cc44/src/php/install.sh#L186-L193,

PHP feature should optionally install xdebug

@razvanphp Are you looking for an option which controls this xdebug installation? (as opposed to installing it only when you request a version)
Let me know if I am misunderstanding your question, thanks!


improve the create service - [closed]

In GitLab by @liyanq528 on Dec 17, 2018, 08:57
Merges 20181006 -> dev

In GitLab by @wknet123 on Nov 13, 2019, 11:34
The test coverage for backend is 23.0%  %  , check  console log

In GitLab by @wknet123 on Nov 13, 2019, 11:34
The test coverage for backend is 23.0%  %  , check  console log

In GitLab by @wknet123 on Nov 13, 2019, 11:34
closed


[bn] Localize ingress.md

Localize ingress.md in bangla
https://glossary.cncf.io/ingress/
Tracking issue number #410
Checklist before opening this PR (put x in the checkboxes)

[x ] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x ] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@mitul3737 @Imtiaz1234 @sajibAdhi Please check


segmentation fault while false map-access

BUG-issue
Description
When try to access a non-existing key within a map, it results in a segmentation-fault. Instead of a correct error-message.

with the last big restructure the related code and so the issue was moved: https://github.com/kitsudaiki/libKitsunemimiSakuraLang/issues/55


Add function for verifying blocks in the context of the chain

Currently, this only verifies slot leaders and block dates.

@NicolasDP Updated.


make DList default initialize ok

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

31 of 37 (83.78%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 99.628%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

dlist.go
31
37
83.78%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
1606

Relevant Lines:
1612

💛  - Coveralls


[Improvement-12945][Tools] improvement: fix some sonarlint warning and file name spell error

Purpose of the pull request
fix issue #12945
improvement  : fix some sonarlint warning and file name spell error

Brief change log
please check the code diff

Verify this pull request

Manually verified the change by testing locally.

thanks for your first contribution to Apache DolphinScheduler, It makes a lot of sense,Looking forward to your deep participation
this PR looks good to me. and please wait the CI check result

@ZhongJinHacker Could u plz run 'mvn spotless:apply' to fix formatting errors so that CI could pass? Thanks.

@EricGao888  ok, I had run 'mvn spotless:apply' in my local PC, and evertything is ok. thank you for your suggestion

Codecov Report

Merging #12946 (f554a58) into dev (27c37b8) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #12946      +/-   ##
============================================
+ Coverage     39.31%   39.32%   +0.01%
- Complexity     4247     4252       +5
============================================
  Files          1061     1061
  Lines         40050    40018      -32
  Branches       4604     4586      -18
============================================
- Hits          15746    15738       -8
+ Misses        22524    22506      -18
+ Partials       1780     1774       -6

Impacted Files
Coverage Δ

...erver/master/processor/queue/TaskEventService.java
69.64% <0.00%> (-10.72%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

.../dolphinscheduler/plugin/task/datax/DataxTask.java
36.62% <0.00%> (-0.26%)
:arrow_down:

.../dolphinscheduler/plugin/task/spark/SparkTask.java
72.56% <0.00%> (-0.25%)
:arrow_down:

...dolphinscheduler/service/storage/impl/S3Utils.java
0.00% <0.00%> (ø)

...eduler/api/dto/workflow/WorkflowCreateRequest.java
100.00% <0.00%> (ø)

...eduler/api/service/impl/DataSourceServiceImpl.java
50.75% <0.00%> (ø)

... and 16 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

SonarCloud Quality Gate failed.
  0 Bugs   0 Vulnerabilities   0 Security Hotspots   1 Code Smell
 0.0% Coverage  0.0% Duplication

fix

@ZhongJinHacker Please add license header for two added UT file.

@ZhongJinHacker Please add license header for two UT files.

@Radeity fix, thank you

I have add unit tests to fix unittest coverage problem.
tips:  some source code is hard to write unit test like DolphinSchedulerManager.java which need a real database datasource access in its constuctor method. It needs  overrload a new constuctor method for DolphinSchedulerManager.

@EricGao888 @davidzollo could you merge this PR ? all checks have passed. I can't wait to do something for the next issue


Fix url for cart redirect when buy mode is active

What does this PR do?
Cart URL now is properly built from current location.origin, while before we were using location.href.

Maybe we can add an integration test using playwright that use real token and move between hosted applications, at least until the cart and back. What do you think?

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


Increasity verbosity of l7 controller, improve /healthz

Add a bunch of events, surfaced broken cloud client via /healthz

Now includes a better /healthz (as advertised in https://github.com/kubernetes/kubernetes/issues/16985)

Assigning to @freehan, should be simple enough as a first code review. Please ask if something doesn't make sense, as it might be a bug ;)

LGTM


Closing window from WAD does not terminate process

Hi. I'm automating some legacy application written for Win32 platform (MFC). The application is a little bit unusual in a way that it consists of the main window and modules which are spawned from the main application as separate processes which open their own windows. The problem is with closing those windows. If I close the window manually, the window disappears and the process which showed the window goes away. But if close the window from WAD, the window disappears but the process remains, which prevents me from entering the same module again (until I kill the process by hand).
I tried closing the window with the close button as well as sending ALT-F4. I also tried closing the window from Root app as well as creating a separate session for it (with different  combinations of close_app, close, quit functions apart from just emulating close button click or ALT-F4). None of these help.

When you click the close button on the sub-window, the application under test is supposed to take care of killing the process. As far as WinAppDriver is concerned, it is responsible for finding the X button and click it. If its doing this, there's nothing more we should expect from it I guess.


[SUPPORT] Using monotonically_increasing_id to generate record key causing duplicates on upsert

Describe the problem you faced
For context we have tables that are snapshotted daily/weekly (eg. RDS export) that we then have Spark jobs convert into Hudi tables (ie. we overwrite the full table). We attach a column using monotonically_increasing_id (https://spark.apache.org/docs/3.1.3/api/python/reference/api/pyspark.sql.functions.monotonically_increasing_id.html ) as a surrogate key if there is no primary key available in our metadata.
We're seeing some really odd behavior where it seems like the same record is sometimes written twice with different record keys (which is overwriting other records), eg dummy example:
Example input:
Row 1: id = 1
Row 2: id = 2
Row 3: id = 3

Adding monotonically_increasing_id (the record key of the Hudi table)
Row 1: id = 1, monotonically_increasing_id = 1
Row 2: id = 2, monotonically_increasing_id = 2
Row 3: id = 3, monotonically_increasing_id =<PHONE_NUMBER>

Hudi table becomes:
Row 1: id = 3, monotonically_increasing_id = 1
Row 2: id = 2, monotonically_increasing_id = 2
Row 3: id = 3, monotonically_increasing_id =<PHONE_NUMBER>

The problem seems non-deterministic (ie. re-running on the same input will fix the issue), for example on one job we saw:
# of rows: 154982072
# of duplicate rows with different record keys: 813263

When an upsert happens, is there retry logic that does a "partial retry"? The docs for monotonically_increasing_id mention its generated from (partition_id, record_number) and we suspect for whatever reason some rows are upserted multiple times in different stages (and changes the partition_id and record_number).
To Reproduce
Unknown, re-running over the same input leads to different results. I asked in Hudi Slack and someone mentioned keygenerator needs to be unique https://apache-hudi.slack.com/archives/C4D716NPQ/p1675336371420009?thread_ts=1675301744.998269&cid=C4D716NPQ
Expected behavior
This seems like an issue with our usage of Hudi:

What are the requirements for record key? Would using Spark's uuid be safe? We found https://issues.apache.org/jira/browse/SPARK-23599

We have encountered this problem with Spark 3.1.2, resulting in duplicate values in a situation where a spark executor died. As suggested in the description, this error was hard to track down and difficult to replicate.

Is there a way to generate a surrogate key for a Hudi table?

Would changing our operation to insert/bulk_insert fix the issue? Naively to me it seems like this will cause duplicates, ie the output becoming:

Row 1: id = 1, monotonically_increasing_id = 1
Row 1: id = 3, monotonically_increasing_id = 1
Row 2: id = 2, monotonically_increasing_id = 2
Row 3: id = 3, monotonically_increasing_id =<PHONE_NUMBER>

Environment Description
We are running on EMR 6.9

Hudi version : 0.12.1

Spark version : 3.3.0

Hive version : 3.1.3

Hadoop version : 3.3.3

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : Yes (Spark on Docker)

For monotonically_increasing_id you may want to do INSERT operation instead ?

@danny0405 https://apache-hudi.slack.com/archives/C4D716NPQ/p1675378366882569?thread_ts=1675301744.998269&cid=C4D716NPQ

issue is, our upsert partitioner/insert partitioner is based on hash of record key. and hence if on re-trigger due to spark task retries, records could move from one insert partition to another and hence could result in data loss or duplication.

It sounds like insert would have the same issue based on that description.

Somehow you need to make sure the auto-generated ids are globally unique, you can use the INSERT operation with insert deduplication disabled.

@alexeykudinkin : can you take this up

@jtmzheng that's an interesting one.
We're currently looking into adding the support for auto-generated keys, and are actually planning to use something similar to monotonically_increasing_id in our approach.
Can you please share a bit more detail how exactly you generate these keys?
If you can provide code snippet that would be very much appreciated

@alexeykudinkin sorry for the delay in response - we just have:
from pyspark.sql import functions as f

df = spark.read.format("parquet").load(INPUT_PATH) # eg. a MySQL table snapshot
df = df.withColumn(
    "_autogenerated_primary_key",
    F.monotonically_increasing_id(),
)
df.write.format("hudi").save(OUTPUT_PATH)

We switched to using uuid and haven't seen the issue since - though this is tricky to reproduce because the problem happens on Spark retries of key generation

@jtmzheng thanks for providing additional context!
Can you please help me understand how did you determine duplicate rows in here:
# of rows: 154982072
# of duplicate rows with different record keys: 813263

We have a two stage pipeline:

Snapshot of MySQL table (as parquet files)
Convert to a Hudi table (ie. read in parquet, write out as Hudi table)

of rows: 154982072

this is the total number of rows in the input

of duplicate rows with different record keys: 813263

so in the example I gave:

Row 1: id = 3, monotonically_increasing_id = 1
Row 2: id = 2, monotonically_increasing_id = 2
Row 3: id = 3, monotonically_increasing_id =<PHONE_NUMBER>

Row 1 should have id = 1 but it was overwritten by Row 3 except with monotonically_increasing_id =<PHONE_NUMBER> (ie. a duplicate of Row 3 except with the record key of Row 1). This was how many "duplicates" we found in our input.
It was easy to confirm this by comparing against the parquet input.

@jtmzheng what operation are you using? Is it "upsert"?

Yep using upsert (Note: In https://apache-hudi.slack.com/archives/C4D716NPQ/p1675378366882569?thread_ts=1675301744.998269&cid=C4D716NPQ implied this would also be a problem with insert)

@jtmzheng can you please paste your whole config you've been using to write to Hudi?

I might know why this could be happening.
if you can clarify something, we can confirm.
for a given df, while generating the primary key using monotonically increasing func, if we call the key generation twice, it could return diff keys right? just that spark will ensure they are unqiue. but it may not be the same?
bcoz, down the line, our upsert partitioner is based on the hash of the record key. so, if for one of the spark partitions, if spark dag is re-triggered, chances that re-attempt of primary key generation could result in a new set of keys (whose hash value) might differ compared to first time, you might see duplicates or data loss.

The docs for monotonically_increasing_id mention its generated from (partition_id, record_number), so if a retry changes either then the keys would be different. I'm not sure internally if a stage is partially retried what happens to the partition_id, but that was my hypothesis.

Small update, we've confirmed this is still an issue even after switching to Spark's uuid

@jtmzheng
This issue was partially resolved in spark with this JIRA - https://issues.apache.org/jira/browse/SPARK-23599
But if you check last comment on above JIRA, someone started to see similar duplicate issue what you have with UUID function.
"We have encountered this problem with Spark 3.1.2, resulting in duplicate values in a situation where a spark executor died. As suggested in the description, this error was hard to track down and difficult to replicate."
How frequent it is ? As a workaround,  Can you use combination of both monotonically_increasing_id and uuid to ensure it to be always unique. May be it give a small performance hit due to generation of such a large id but it should be always unique.

@ad1happy2go yeah saw that ticket a while back, I'm curious if the surrogate key functionality Delta supports (https://www.databricks.com/blog/2022/08/08/identity-columns-to-generate-surrogate-keys-are-now-available-in-a-lakehouse-near-you.html) has the same issue.

hey @jtmzheng :
yeah, this might end up w/ duplicates or data loss on failure scenarios.
we have put up https://github.com/apache/hudi/pull/8107 for this ask. If you can try and let us know if it works, it would be nice. we did test w/ failure scenario and validated that there is no data loss or duplicates w/ this patch.

@nsivabalan hmm from the description in #8107 :

Engine's task partitionId or parallelizable unit for the engine of interest. (Spark PartitionId incase of spark engine)
Row id: unique identifier of the row (record) w/in the provided task partition.
Combining them in a single string key as below
"${commit_timestamp}${partition_id}${row_id}"
For row-id generation we're planning on using generator very similar in spirit to monotonically_increasing_id() expression from Spark to generate unique identity value for every row w/in batch (could be easily implemented for any parallel execution framework like Flink, etc)

How does this avoid the same problem in this ticket?


Ghidra1's fixes for ConstantPropagationAnalyzer and PseudoDisassembler

@Ghidra1 provided @ryanmkurtz and me with two fixes, one for each of ConstantPropagationAnalyzer and PseudoDisassembler, that have not yet been committed here on github. Could he please commit them? Thanks, @Ghidra1, for helping so much with those and other issues.
:smile:
(I don't think there were any github issues created for those problems.)

Speaking of ConstantPropagationAnalyzer's there is this also. https://github.com/NationalSecurityAgency/ghidra/pull/2157

Although things were a bit disorganized at the time I believe I incorporated all of the relevant changes into Ghidra for release with Ghidra 9.2 which should also be in master.  Sorry to trouble you but could you please report any outstanding bugs/issues into this ticket that you believe should have been resolved.  I checked my repository where I was working at the time and all changes I had made at the time appear to be present in the GitHib repo.
See commit

Speaking of ConstantPropagationAnalyzer's there is this also. #2157

I passed this other issue onto @emteere

It does look like the commit resolves the problems that I asked about in this issue, so I'm closing this.


No need to check if # of dimensions of testing set align with training set in target_encoder

https://github.com/scikit-learn-contrib/category_encoders/blob/6a13c14919d56fed8177a173d4b3b82c5ea2fef5/category_encoders/utils.py#L322-L323
For the function _check_transform_inputs(), I do not want it to report error when # of dimensions for testing set don't align with training set. However, the default is it has to align. Considering that the purpose of target encoder is to transform designated columns using target encoders, nothing else, logically we don't have to validate the dimension alignment.

plase have a look at the dicussion in #355
is this a duplicate of that issue?


myCache variable undefined

Hi,
I'm trying to use the library according to the README but I'm getting the following error:
var cluster = require('cluster');
var myCache = require('cluster-node-cache')(cluster, {
    stdTTL: 100,
    checkperiod: 900
});

myCache.set(key, val).then(function (result) {
            console.log(result.err);
            console.log(result.success);
        });

The set cannot access an undefined object, I mean the myCache variable always remains undefined.
Any ideas? Thanks.

It's basically because you are following it "as-is" (sorry, the instructions could be clearer) as a result of this you end up calling the code in the master, and I guess before the cluster has been fully set up
If you try something like the following you'll have better results
Please also check out the node.js API documentation for a bit more info than I can provide on how to use the clustering framework: https://nodejs.org/dist/latest-v8.x/docs/api/cluster.html
`var cluster = require('cluster');
var myCache = require('cluster-node-cache')(cluster, {
stdTTL: 100,
checkperiod: 900
});
if(cluster.isMaster && !process.env.DEBUG) {
worker = cluster.fork();
cluster.on('exit', function() {
cluster.fork();
});
} else {
myCache.set("key", "val").then(function (result) {
console.log(result.err);
console.log(result.success);
});
}`


[feature] Support S3 Filesystem for Tensorflow

At the moment Tensorflow supports distributed file system implementations such as HDFS or GCS which has been very useful for handling cases in a distributed environment. Both of them utilize Tensorflow's FileSystem C++ interface.
In case Tensorflow is used in AWS, it would be great to have a S3 Filesystem for Tensorflow as well.
We have a preliminary implementation of S3 Filesystem on Tensorflow, based on AWS's C++ SDK:
https://github.com/tensorflow/tensorflow/compare/master...yongtang:s3
The code is placed under the directory of tensorflow/contrib/s3.
As AWS is widely used, I am just wondering if it make sense to have a pull request to add S3 File system on Tensorflow?
It will be better to have feedback from community so that we could improve our implementation and help those deploying Tensorflow on AWS.

@jhseu, it would be great if you could comment on this and provide guidance. @yongtang prepared a patch linked above.

@yongtang Yes, please make a pull request with your changes! This would be a very useful feature for a lot of people.

Closed through #11089


Options to pull XferPrint recipe info

Options to pull XferPrint recipe info
If I don't remember grid pitches, I want to pull target pitch info from print recipe and add that as A/B pitches.
A = Target Region pitches
B = # of stamp posts * stamp post spacing
C = keep user defined because optic difference?

XFER-907

"Import grid distances from XferPrint Recipes"
Need an 'import' button under file to do this

This is an XferPrint task, not gridmaker


Multiple Scene Viewports reduces performace by a lot!

How to Reproduce:

Run the engine
Note the frame time
Add one more scene viewport by going to Menu Bar, Window>Add>Viewport
Note the frame time, should be around x15 higher than single viewport which is not acceptable frame time.

Result:
Two scene viewports with an empty scene can bog down the performance taking 20+ms
Expected:
Adding more Scene Viewports is bound to degrade performance but not at this scale
More Info:

One scene viewport, CPU time: 1.227ms

Two scene viewports, FPS: 20.35ms

Seems like the pipeline is being stalled by framebuffers resizing quite a lot between frames


ExpressionList cannot be auto-regenerated

Describe the bug
While testing the state of our xml2xxx code, I discovered that running xml2atts for
common/state/ExpressionList.xml does not regenerate the correct atts.
I looked into adding the correct infrastructure to ExpressionList.xml and ExpressionList.code, but there are some 'mutable' ivars in the header that cannot be autogen'd with our current
xml code base.
It is important to be able to auto gen our state objects in the (rare) case that we add
a new function to all state objects.  The fact that this object cannot be autogen'd might be
overlooked in the overload of changed files due to a wholesale auto-regen that actually produces changes in every state object.
Helpful additional information
(Please click check boxes AFTER submitting ticket)

[ ] Did VisIt crash?
[X] Did you get wrong results?

Impact

[ ] High - User productivity significantly degraded
[ ] Medium - User productivity partially degraded
[ ] Low - User productivity trivially degraded

Likelihood

[ ] High - occurrence is not specific to platform, data, config, a combination of operators, queries, etc.
[ ] Medium - neither low nor high likelihood
[X] Low - occurrence is specific to combination of multiple factors such as configuration, data, platform, etc.

To Reproduce
Run xml2atts -clobber ExpressionList.xml
Check the diffs in the header and source files.
Expected behavior
xml2atts should produce no changes, unless something new has been introduced to the atts generator.

related to #2823


adds equality operators for Checks, Columns, Schemas etc.

In preparation for #77 I want to be able to test if Schemas and other pandera objects are the same before and after operations on the schemas.
This PR:

implements equality operators == for:

DataFrameSchema
SeriesSchemaBase
SeriesSchema
Column
Index
MultiIndex
Checks

because we're now on Python 3 we get the negation != for free
includes assert tests

Before this PR the below example incorrectly returns False. After the PR this works as expected:
import copy
import inspect
import pandera as pa
from pandera import DataFrameSchema, Column, Int, String, Check, SeriesSchema
from pandera.schemas import SeriesSchemaBase

schema1 = DataFrameSchema({
    "col1": Column(Int, Check(lambda s: s >= 0)),
    "col2": Column(Int, Check(lambda s: s >= 0)),
},strict=True)

schema1 == copy.deepcopy(schema1)

UPDATE 1:
After thinking about this more, I've enhanced the test coverage and implementation of the Check ==, so that lambda functions which are the same, but use different characters will be considered equal. I think this is desirable as I think we want this:
    >>> main_check = Check(lambda g: g["foo"]["col1"].iat[0] == 1, groupby="col3")
    >>> same_check = Check(lambda h: h["foo"]["col1"].iat[0] == 1, groupby="col3")

    >>> main_check == same_check
    True

UPDATE 2:
After working on it more, I've enhanced the tests again to cope with when the same schema Columns are specified, but in different orders.

Codecov Report

Merging #143 into master will increase coverage by 0.14%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
+ Coverage   95.95%   96.09%   +0.14%
==========================================
  Files           8        8
  Lines         445      461      +16
==========================================
+ Hits          427      443      +16
  Misses         18       18

Impacted Files
Coverage Δ

pandera/schema_components.py
94.33% <100%> (+0.72%)
:arrow_up:

pandera/checks.py
95.91% <100%> (+0.17%)
:arrow_up:

pandera/schemas.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

After thinking about this more, I've enhanced the test coverage and implementation of the Check ==, so that lambda functions which are the same, but use different characters will be considered equal.

nice, agreed

@cosmicBboy I debated implementing +/- operators too, but thought this was a bit unnecessary/potentially risky.
Please let me know if you have any comments/concerns.


Bug: tabline shows only one file

Self Checks

[x] I'm using the latest lualine and neovim nightly.
[x] I didn't find the issue in exsisting issues or prs.

How to reproduce the problem
Enable tabline.
Actual behaviour
Open a file with chadtree in a new tab or with tabedit but the tabline has only one single file.
Expected behaviour
Show tabs for the various file opened.

This is not a bug. The tabline option only refers to putting your statusline on top. This was further discussed here

In the readme it wasn't explained and the previous statusline plugin I was using was doing also that.
Sorry for the confusion.

shadmansaleh#42 has been merged now shadmansaleh/lualine.nvim has traditional tabline / bufferline support.
See #311 to know what else has changed in the fork.


Moved pywren runtime inside dockerfile

Currently pywren runtime code injected in action containers during container initialization. This PR provides option to avoid it by packaging pywren code inside Docker image.
Changes:
codelessrunner.py - extends pythonrunner and adds support for codeless python action
Dockerfile.codeless - provides base docker image for pywren runtime
ibm_cf.py and openwhisk.py - check that pywren runtime code and main.py already exist in docker image. If exist  - create action without injecting pywren code archive in the action
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

@kpavel can you please update this code against recent master branch?

@kpavel what we do with this PR? Will you update or we close it for now?

@gilv Thanks for the reminder. I'll close it for now.


Support request: What's wrong with my fig.yml for a wordpress?

setting up a wordpress-instance with Docker is easy:
docker run --name="wp_db" -d -v /var/lib/mysql -e MYSQL_ROOT_PASSWORD=password mysql
docker run --name="wp_web" -d --link="wp_db:mysql" -v /var/www -p "8001:80" wordpress
to my knowledge, this would be the corresponding fig.yml:
web:
  image: wordpress
  links:
    - db:mysql
  ports:
    - "8001:80"
  volumes:
    - /var/www

db:
  image: mysql
  environment:
    MYSQL_ROOT_PASSWORD: password
  volumes:
    - /var/lib/mysql

however, starting the 'app' with fig up will lead to the situation that - to quote:
db_1  | 2015-01-18 20:55:44 1 [Warning] IP address '<IP_ADDRESS>' could not be resolved: Name or service not known
web_1 | MySQL Connection Error: (1130) Host '<IP_ADDRESS>' is not allowed to connect to this MySQL server
web_1 |
web_1 | Warning: mysqli::mysqli(): (HY000/1130): Host '<IP_ADDRESS>' is not allowed to connect to this MySQL server in - on line 5

after inspecting the containers, this makes sense, since the failing containers are not attached to Docker's network-bridge. which i would expect since Docker does that by default.
anyway, also adding
  net: "bridge"

explicitly to both service's configuration will not create the network-bridge.
what's irritating even more is that it worked for a while with the fig.yml above. and the only aspect i changed these days was to update fig to version 1.0.1 (edit: with version 1.0.0 it's all the same.)
so, before i dig deeper i'd appreciate some feedback whether this is reproducable, a race condition to be expected or some obvious mistake on my side.
Ubuntu 14.04 / Mac OS X 10.9
Docker 1.4.1
fig 1.0.1

EDIT to elaborate a little more on the bridge-issue:
the verbose output states the last container start as follows:
docker start <- (u'62cd26fedcab626cdc212416cc81d753406542c8517e52415e58f0f7b6de44b5', links=[(u'wordpress_db_1', u'mysql'), (u'wordpress_db_1', u'wordpress_db_1'), (u'wordpress_db_1', u'db_1')], network_mode='bridge', binds={}, dns=None, volumes_from=[u'946af1607e985ba9eed6cc673ec45041928b14e1a2443ebc4e49c49e4ecf9700'], port_bindings={u'80': u'8001'}, privileged=False)
anyhow:
docker inspect 62cd26

(...)
    "NetworkSettings": {
        "Bridge": "",
        "Gateway": "",
        "IPAddress": "",
        "IPPrefixLen": 0,
        "MacAddress": "",
        "PortMapping": null,
        "Ports": null
    }

today it worked reliable with Mac OS.
but within one minute i had all four possible, different outcomes on Ubuntu while i repeated fig rm && fig up. Either no container had a network-bridge, one of both had one or both had one. :stuck_out_tongue_winking_eye:

after some further testing, i eventually figured out that it propably relates to to this issue.
it comes up when there are one ore more VPNs used on the Docker-host. however it still isn't reliably reproducable.


The locale folder "he" should be called "iw" instead; see the 'java.util.Locale' documentation

About res/values-he/strings.xml, Android Studio lint check says

The locale folder he should be called iw instead; see the java.util.Locale documentation

Before switching the country code to the new iw, we should consider the following implications:

Old Android 5 devices can handle it?
Weblate can handle it ? Seems they can't customize the country code.

folder have been renamed to "values-iw"


Retry namespace creation for e2e tests

Addresses #10289
I do not tackle namespace deletion since that is a more subtle issue: see #10217 @wojtek-t
@quinton-hoole @ixdy

Go Satnam!

I have changed to using wait.Poll however I would like to make a formal complaint about this change I have been required to make. This has involved changing what I thought was an easy to read, maintain and update loop into a form that involves the use of a closure which needs to access a variable from the outer scope directly (because the types prevent this being passed in through an argument) -- this seems insanely complicated compared to what I started off with. @quinton-hoole @mbforbes

Updated the closure in the wait.Poll but still confused about what is going on.

LGTM, thanks!

@k8s-bot test this please

The code LGTM, but I'd like to see a successful e2e run before merging.

LGTM (technically we need one more lead to LGTM it, right?)

Yes. Who's job is it to identify the extra reviewers? On-call? @mbforbes ?

Sorry to hold up the works so late, but I can't figure out where the timeout used in that retry loop is defined.  Am I being dumb?  What is it's value?

@satnam6502 ^^

I am trying to be a good boy by reusing the constants defined at the top of the file.

const (
	// Initial pod start can be delayed O(minutes) by slow docker pulls
	// TODO: Make this 30 seconds once #4566 is resolved.
	podStartTimeout = 5 * time.Minute

	// String used to mark pod deletion
	nonExist = "NonExist"

	// How often to poll pods and nodes.
	poll = 5 * time.Second

	// service accounts are provisioned after namespace creation
	// a service account is required to support pod creation in a namespace as part of admission control
	serviceAccountProvisionTimeout = 2 * time.Minute

	// How long to try single API calls (like 'get' or 'list'). Used to prevent
	// transient failures from failing tests.
	singleCallTimeout = 30 * time.Second

	// How long nodes have to be "ready" when a test begins. They should already
	// be "ready" before the test starts, so this is small.
	nodeReadyInitialTimeout = 20 * time.Second

	// How long pods have to be "ready" when a test begins. They should already
	// be "ready" before the test starts, so this is small.
	podReadyBeforeTimeout = 20 * time.Second

	// How wide to print pod names, by default. Useful for aligning printing to
	// quickly scan through output.
	podPrintWidth = 55
)

BTW I was told it's not the job of the oncall to identify the extra reviewers. (In fact, if PR openers--who probably have more context on what they're working on than the oncall--would identify reviewer(s) it would be extremely helpful!)

I nominated @dchen1107 as the extra TL reviewer.

Oh, but for tests, you only need one LGTM, so once @quinton-hoole acks this is good, we're gravy.

Do we know why the operation was occasionally failing to begin with?  It was not obvious to me when reading the referenced issue.  I have no problem with the change, I just want to make sure the change is not masking some other underlying issue.

It probably is masking some underlying issue (that I am very worried about).
S
On Thu, Jun 25, 2015 at 12:06 PM, Derek Carr<EMAIL_ADDRESS>wrote:

Do we know why the operation was occasionally failing to begin with? It
was not obvious to me when reading the referenced issue. I have no problem
with the change, I just want to make sure the change is not masking some
other underlying issue.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/10301#issuecomment-115364866
.

@derekwaynecarr that's a totally valid point, but I think that having assorted e2e tests fail due to transient namespace creation errors is not the right way to signal this.  The namespaces e2e test should pick up problems creating namespaces.

@quinton-hoole good point -- I originally had singleCallTimeout but this got lost in translation. I have now updated the PR to use this easier to fine constant.

I don't see that update yet.  Let me know when it's pushed.

@satnam6502 ^^

Fixes #10289

PS: Jenkins is broken: jenkinsci/ghprb-plugin#117  so don't wait too expectantly for those tests results.  @ixdy is working on fixing that.

It turns out I was confused about the "1 LGTM for tests" rule. It is still 2. Assigning to @dchen1107 as per @satnam6502 's request for another look.

Why am I assigned to my own PR?

Haha, I don't know. @quinton-hoole ?

Now if you read above, at 1:06pm today, you might just find the answer :-)

This is all very exciting
https://github.com/GoogleCloudPlatform/kubernetes/pull/10301#issuecomment-115378920

What's the status of this PR? @roberthbailey

It's still assigned to you pending the answer to @quinton-hoole's comment. Once you answer, please assign back to him.

Quinton's question talks about "at 1:06pm today". I don't know what comment that refers to since the comments don't seem to have timestamps on them by the hour.

If you hover over the "X hours ago", it will give you an exact timestamp. In this case, the comment is: "I don't see that update yet. Let me know when it's pushed."

Ah, yes, I tried that, but perhaps I was not patient enough. Pushed now.

@roberthbailey I just re-ran the tests and I think things are fine now?

Tests seem better. Re-assigned to @quinton-hoole

LGTM!  ok-to-merge

FYI, this change appears to have slowed down the already-slow Namespaces Delete 90 percent of 100 namespace in 150 seconds test from about 10 minutes to 17-18 minutes.

What are the reasons this test would slow down the namespaces delete test? This PR only changed namespace creation rather than deletion. This PR would only slow down namespace creation if multiple attempts had to be made to create a namespace (to be robust against a failure). It would be good to get to the bottom of this.

OK, I finally figured this out (after an embarrassingly long time). From the documentation for wait.Poll:
// Poll always waits the interval before the first check of the condition.
// TODO: create a separate PollImmediate function that does not wait.

So before this change, namespace creation happened instantaneously. With this change, we now pause 5 seconds before trying to create a namespace.

Yet another reason to revert to my original loop :-)

I created #10658 to address some of the horrific slowness in that test.


[AMORO-2008]: Refactor the optimizer module to separate distributing pacakges for different engines.

Why are the changes needed?
Close #2008 .
Brief change log

The optimizer module has been reorganized and split into ams/optimizer-container, ams/optimizer-job/job-common, ams/optimizer-job/flink-job, and ams/optimizer-job/standalone-job.
Changes have been made to the configuration files such as config.yaml, helm configmap, etc., to update the package name of localOptimizerContainer.
The dist directory has been modified, and the plugin/optimize/OptimizeJob.jar has been removed and replaced with plugin/optimize/flink/optimize-job.jar, which is only used for the Flink optimizer.

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[X] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (no)

@baiyangtx Thanks a lot for your work!
Optimizer job seems to be a new concept for Amoro developers and users. Here, I think using Optimizer directly seems accurate enough. So I suggested to name the module to :

ams/optimizer-container
ams/optimizer/common
ams/optimizer/standalone-optimizer
ams/optimizer/flink-optimizer

How do you think?

@baiyangtx Thanks a lot for your work!
Optimizer job seems to be a new concept for Amoro developers and users. Here, I think using Optimizer directly seems good enough. So I suggested to name the module to :
* ams/optimizer-container

* ams/optimizer/common

* ams/optimizer/standalone-optimizer

* ams/optimizer/flink-optimizer

How do you think?

I agree with it. I will fix it.


FEATURE_REQUEST: HTML CRUD scaffolds generator with html views

it is expected to add a  Scaffolding / Crud generator with html views , and generic code in controller's actions from Models definition ?
It's very helpful when you want to develop quickly an application without a very specific interface ;)
I know it represent lot of works but it's expected ?
Thanks for your answer.

should this be closed? seems like it's still an outstanding issue.

Yes you can close the issue, thanks for your work.  ; )
Le 8 juil. 2015 21:21, "Matt Cook"<EMAIL_ADDRESS>a écrit :

should this be closed? seems like it's still an outstanding issue.
—
Reply to this email directly or view it on GitHub
https://github.com/balderdashy/sails/issues/1146#issuecomment-119704044.

@lmasforne
do u any other solution. to generate  HTML+CRUD scaffolds generator with sails.


Hotloaded markets should have clearer UX

Make a Hotloading Market Card that communicates that some parts of this market data are not yet loaded.
Display those during hotloading
Replace these cards in place with the data in the full cards once all the data is there.

@matt-bullock -- I think TH asked for a design in this area before?

I'd already added this to 8489:
How about showing loading spinners in the spots that are still loading info. That way the info like market title loads immediately and nothing jumps around when the rest of the info loads. This informs the user that something is happening rather than just leaving it blank
Prototype (can take a while to load!): https://framer.com/share/Trading-Hot-Loading-Animations-5xmNPfVviezZpCuMbPy2/iIgD0q6Wi?fullscreen=1&highlights=0

I also suggest we use the style of spinners used in the prototype. They're a bit nicer than what we currently have


Update CMakeLists.txt for ROS2

I believe there already a few PRs open which do this but the reason none have been merged is to use pure cmake and have one branch to support both ROS 1 and ROS 2.

Ok. I use this repository for ROS2 without catkin. So I forked the repository for myself and created a PR. Thanks for your explanation.

LGTM


Added options page in manifest.json

Added options page to add API URL
removed code from popup.js
Contributes to # (add issue number here. i.e. #21)
What did you do?
Added options.html page to set API URL
Why did you do it?
Issue 21
How have you tested it?
yes
Were docs updated if needed?

[ ] No
[ ] Yes
[x] N/A

Type of change

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new console logs
[x] Fixes entire issue
[ ] Partial fix for issue

pls review @upkarlidder @josiemundi


Develop Staging GitHub Action CD to deploy to Test venue

💡 Description
Merge to staging -> Deploy to Test venue
⚔ Parent Epic / Related Tickets
No response

Here is the prototypical staging.yaml file that would achieve this:
---
name: 🎭 Staging
on:
    push:
        branches:
             - staging
        paths-ignore:
            - 'CHANGELOG.md'
            - 'docs/requirements/**'
    workflow_dispatch:
jobs:
    staging-delivery:
        name: 📪 Staging Delivery
        if: github.actor != 'pdsen-ci'
        runs-on: ubuntu-latest
        steps:
            -
                name: 💳 Checkout
                uses: actions/checkout@v2
                with:
                    lfs: true
                    token: ${{secrets.ADMIN_GITHUB_TOKEN}}
                    fetch-depth: 0
            -
                name: 🌍 Terraform Setup
                uses: hashicorp/setup-terraform@v3
                with:
                    terraform_version: "TERRAFORM_VERSION_HERE"
            -
                name: 🔬 Terraform initialization and validation
                run: |
                    terraform init -no-color
                    terraform validate -no-color
            -
                name: 💥 Terraform destruction of any existing delivery
                run: terraform apply -destroy -no-color -auto-approve
                continue-on-error: true
                env:
                    ANY_SECRET: ${{secrets.ANY_SECRET}}
                    etc: ${{secrets.etc}}
            -
                name: 🛫 Terraform application of new delivery
                run: terraform apply -no-color -auto-approve
                env:
                    ANY_SECRET: ${{secrets.ANY_SECRET}}
                    etc: ${{secrets.etc}}
...

This can be adapted for various repositories depending on their shapes and needs, of course, such as adding -var-file=FILE, altering the secrets, and so on.


[R] Missing predict() method for object of class lgb.CVBooster

The only predict() method exists for object of class lgb.Booster; for the output of a lgb.cv() training function, predict() throws an error:
Error in UseMethod("predict") :
  no applicable method for 'predict' applied to an object of class "c('lgb.CVBooster', 'R6')"

Cross-validation does not produce a model but is only used to check for the metric evolution in a CV. You must train a model separately to use predict. If you want to access all the models separately, you just need to use a loop to train separately from each train fold and predict on its respective test fold.
Also, lgb.cv does not produce a lgb.Booster class. It produes a lgb.CVBooster class.

You must train a model separately to use predict. If you want to access all the models separately, you just need to use a loop to train separately from each train fold and predict on its respective test fold.

That appears to be correct: the lgb.CVBooster object indeed contains a booster vector which has a list of nfolds - 2 number of lgb.Booster objects.
The xgboost package has a method parameter to predict on the models resulting from a xgb.cv, however looking at the documentation closer for that package, the resulting prediction is for each of the folds, so it's less helpful.
Closing.


Support for FreeBSD

Hi all,
I posted this discussion in the tailwind repo: https://github.com/tailwindlabs/tailwindcss/discussions/7826
There is currently no binary for freebsd and I relay like to use phoenixframework/tailwind as well as I love my FreeBSD servers.
I tried to compile the source with vercel/pkg but did not succeed. Is there anyone else out there who needs a FreeBSD binary and more knowledge about how to compile that?
kind regards!

A quick update, after many peoples help we now can
build this for FreeBSD :tada: which I will import into
FreeBSD ports :soon:
There is no easy way to do github actions with FreeBSD,
I'm not keen on burning cycles to figure out some hacky
work-around abusing virtualbox or qemu.
The FreeBSD ports & package system is going to be the way
most Elixir users will install both Elixir and Erlang,
would Phoenix be OK with teaching the tailwind module,
for FreeBSD, to tell people to fetch this via official
packages, rather than from the tailwind repos?

Sorry for the delayed reply. I've not had this building inside a no-net container yet, which is a requirement for building inside ports - somewhere in the 70+Mb of downloaded javascripts is an assumption that there must be a reachable network.
In the interim, are we able to download from e.g. https://people.freebsd.org/~dch/pub/tailwind/tailwindcss-v3.2.4-freebsd-x64.xz, with a suitable checksum?
Or, I can set up some sort of build script that generates these artefacts through an inspectable build log, and then use that?
Finally, we can offload this trust choice to users and point them towards that URL with some warnings about strange people on the internet.
I'm not planning to spend my Christmas vacation hacking javascripts packaging tools, when I could be playing with LiveView.
What's an appropriate level of trust?

Wohooo !!! @chrismccord and @dch
So is it real now ? https://hexdocs.pm/tailwind/changelog.html#v0-1-10-2023-02-09

Tailwind tasks doesn't work for me:
** (Mix) mix tailwind default --minify exited with 8
on my machine:
> uname -srm
FreeBSD 13.1-RELEASE-p8 amd64

with Erlang 26.0.2.
It downloads tailwind binary for Linux instead of FreeBSD.
This is because this code https://github.com/phoenixframework/tailwind/blob/2298fd3f119ac3487cf400f6775781585f8741ec/lib/tailwind.ex#L248 returns "x86_64" on my machine, not "amd64". The cause for this bevahior seems to be here: https://github.com/erlang/otp/blob/14ce71d439bef11fd54a214b0d408bc75dee6ec2/erts/autoconf/config.guess#L877
So the arch string should be fixed here https://github.com/phoenixframework/tailwind/blob/2298fd3f119ac3487cf400f6775781585f8741ec/lib/tailwind.ex#L255 if someone can confirm that on all supported FreeBSD and Erlang versions it will be "x86_64".

@SukhikhN the most common case is installing OTP via binary package, which gives us:
$ which erl; uname -p; elixir --eval 'IO.puts(:erlang.system_info(:system_architecture))'
/usr/local/lib/erlang26/bin/erl
amd64
amd64-portbld-freebsd15.0

how did you install Erlang/OTP? what do you get instead?
We may need to PR both cases.
Assuming you used asdf, I tried installing erlang latest, and get an unrelated error atm:
...
 CC     /home/dch/.asdf/plugins/erlang/kerl-home/builds/asdf_26.1.2/otp_src_26.1.2/erts/emulator/ryu/obj/x86_64-unknown-freebsd15.0/opt/d2s.o
 CXX    obj/x86_64-unknown-freebsd15.0/opt/jit/beam_asm_global.o
<command-line>: sorry, unimplemented: PCH allocation failure
gmake[4]: *** [x86_64-unknown-freebsd15.0/Makefile:936: obj/x86_64-unknown-freebsd15.0/opt/jit/beam_asm_global.o] Error 1
gmake[4]: *** Waiting for unfinished jobs....

Currently built tailwind versions:

https://people.freebsd.org/~dch/pub/tailwind/

I use mix tailwind.install --if-missing https://people.freebsd.org/~dch/pub/tailwind/v3.3.2/tailwindcss-freebsd-x64 to fetch it.
You will also need lang/gcc installed for tailwind to run.

Yes, I have Erlang 26.0.2 installed using asdf.
Here is the system info:
$ which erl; uname -p; elixir --eval 'IO.puts(:erlang.system_info(:system_architecture))'
/home/freebsd/.asdf/shims/erl
amd64
x86_64-unknown-freebsd13.1

@SukhikhN perhaps PR for this? So long as aarch64 comes first, this match
should be fine. I don't expect a RISCV tailwind build anytime soon. Or
just add the additional branch.
{{:unix, :freebsd}, _, 64} -> "freebsd-x64"

Could this #95 be merged please?