[PAID] [$500] Distance- Expense preview does not show the updated currency when distance currency is changed

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.22-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to workspace chat
Create a distance request
Go to distance details page
Click Amount
Change the currency and save it
Return to expense report page

Expected Result:
The distance preview will show the updated currency
Actual Result:
The distance preview does not show the updated currency
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/4facf212-ee84-4995-b5f2-b024b95f9480
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f1b7774b1ba5d7d3
Upwork Job ID:<PHONE_NUMBER>148869120
Last Price Increase: 2024-01-05
Automatic offers:

jjcoffee | Reviewer | 28091955
bernhardoj | Contributor | 28091956

Proposal
Please re-state the problem that we are trying to solve in this issue.
The currency for distance request preview doesn't update when we update the currency.
What is the root cause of that problem?
This only happens with distance request preview. If we look at the normal review, we get the currency from requestCurrency, but for distance request, we get it from transaction.currency which holds the first currency when we create the transaction.
https://github.com/Expensify/App/blob/f5a3c0202128a8e0e77607defbbdcb4f30f80ad3/src/components/ReportActionItem/MoneyRequestPreview.js#L223-L237
requestCurrency itself is coming from TransactionUtils.getCurrency that gets either modifiedCurrency or currency.
https://github.com/Expensify/App/blob/f5a3c0202128a8e0e77607defbbdcb4f30f80ad3/src/libs/TransactionUtils.ts#L287-L293
What changes do you think we should make in order to solve the problem?

Use requestCurrency for distance request too

Looks good, I'll work on tying this to a wave or VIP tomorrow.

Thanks @jjcoffee, I'll add to the VIP/Wave when I get confirmation.

This looks like it might fit under vip-vsb

PR is ready
cc: @jjcoffee

The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/24556/
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/24556/files#r1466105952
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A - just required more thorough testing
Determine if we should create a regression test for this bug. Yes
If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Open a policy expense chat
Create a distance request
Press the request preview once to open the expense report
Press the request preview again to open the request/transaction detail thread
Edit the currency to another currency
Go back to the expense report
Verify the distance request preview currency is updated

Do we agree 👍 or 👎

@strepanier03 Friendly bump for payment!

Dang, another one where the automation didn't work to move this over to Daily. I'll work on this now.

All paid up. Handling reg test and then closing.

Thanks everyone 👋


[Footer] color exploration

Figma exploration

Overview:

Our goal is to explore color schemes for the footer that fit within the lighter and brighter branding. Currently, Footer is locked to the gray 90 theme.
Should footer match the page theme? This way, it wouldn’t be a jarring transition on any page.

Layout explo

Current version leans to the right to match content. If we go this route, perhaps we could remove a border or two (looks overcomplicated with multiple borders)
We could also spread the links across more columns, creating additional breathing room.
Border on top might be necessary to separate more cleanly from page content.
Leaning towards using an alt tertiary button for locale selector to fit visually among the rest of the Footer elements. We cannot use a link with icon.
Leaning towards using an alt tertiary button for locale selector to fit visually among the rest of the Footer elements. We cannot use a link with icon.

Design crit 06-29-23
Layout

Between the right clustered and full-width versions, the right clustered links use an exaggerated margin which helps the logo stand out.
Cloud has their own footer, which uses a full-width design with limited links out.
As the footer changes the number, organization, or density of links, the right clustered version may be more forgiving/resilient. Full-width might get difficult if the links become unbalanced.

Accessibility

As users scroll through a page and encounter a high contrast moment, some users may find that disruptive from an accessibility perspective. More reason to consider either on-theme footers or adjacent-theme footers.

Next steps

We should talk to the IA guild to see what changes they might be considering. If they are looking to make drastic changes, we should consider that in the redesign.

Adjunct link treatment

Adjunct links always appear in the last four columns
Adjunct links always appear below the secondary links as a new row
Adjunct links are added and wrapped to the next row if needed.


add signal handler for SIGINT and SIGTERM.

address https://github.com/fujitatomoya/ros2ai/issues/26

https://github.com/fujitatomoya/ros2ai/actions/runs/8951886944/job/24588682342?pr=29, breaks the build for humble and iron because of https://github.com/fujitatomoya/ros2ai/actions/runs/8835735881/job/24260633997.
there are two ways that we can take.

having the distro branches (humble and iron) with mainline, so that we can manage distro (not really true, but more like system or OS dependent) dependent procedure. this generates the maintenance burden, i am not inclined to do this yet until it comes to really really necessary.
during installation script or procedure, we can use different option. (with ubuntu 22.04 without --break-system-packages)

during installation script or procedure, we can use different option. (with ubuntu 22.04 without --break-system-packages)

i went this one with https://github.com/fujitatomoya/ros2ai/pull/29/commits/685deb8f0ccbaa10acc0be55301d25be8ddf9757, now all CIs are green.


Upgrading from 25.3.1 to 25.4.0 breaks monorepo projects

🐛 Bug Report
I have a TypeScript project using Yarn workspaces and TypeScript project references. I have a single shared project used by my backend and frontend projects. As expected, some tests in both backend and frontend projects rely on classes and functions exported from the shared project.
When using 25.3.1 version, I can run the tests without any issues. Shared code is correctly imported.
After updating to 25.4.0 or 25.5.0, I run into the following issue:
FAIL backend <TEST NAME>.test.ts
  ● Test suite failed to run

    File not found: /PATH/TO/PROJECT/packages/shared/dist/index.js (resolved as: /PATH/TO/PROJECT/packages/shared/dist/index.js)

      at ConfigSet.resolvePath (../../node_modules/ts-jest/dist/config/config-set.js:697:19)
      at Object.getCompileResultFromReferencedProject (../../node_modules/ts-jest/dist/compiler/compiler-utils.js:122:38)
      at compileFn (../../node_modules/ts-jest/dist/compiler/language-service.js:126:41)
      at Object.compile (../../node_modules/ts-jest/dist/compiler/instance.js:52:21)
      at TsJestTransformer.process (../../node_modules/ts-jest/dist/ts-jest-transformer.js:85:41)
      at ScriptTransformer.transformSource (../../node_modules/@jest/transform/build/ScriptTransformer.js:481:35)
      at ScriptTransformer._transformAndBuildScript (../../node_modules/@jest/transform/build/ScriptTransformer.js:586:40)
      at ScriptTransformer.transform (../../node_modules/@jest/transform/build/ScriptTransformer.js:624:25)

It seems that the shared project cannot be correctly resolved.
To Reproduce
Steps to reproduce the behavior:

Setup a TypeScript Yarn project with workspaces and project references.
Set one project as a dependency of the other.
Write a test that references the imported project.

Expected behavior

Tests should pass and dependency project should be correctly resolved.

envinfo
System:
    OS: Linux Mint 19.3

Npm packages:
    jest: 25.1.0
    ts-jest: 25.4.0
    typescript: 3.8.3

would you please try 25.5.1-alpha.0 by install ts-jest@next ?

Same issue, shared project is not correctly resolved.

Hmm ok so it’s a different issue with 25.5.1 trying to solve. Would you please provide a minimum repo ? That will be faster for debugging

in 25.5.0 we added a partial support for TypeScript projectReferences. From the log I can see it goes into the path of trying to resolve file because the monorepo uses projectReferences.
Can you please try to run tsc -b, clear jest cache and rerun test ? I'm curious

I'll try.

I did clear the cache, removed node_modules, etc. Only reverting to 25.3 helped so far.

did you run tsc -b before running tests ?

I didn't, and I never had to before. Can confirm that tsc -b generates index.js in the directory that ts-jest expect it to be, but I'd rather not have to run tsc -b on the shared module each time before running the tests.

ok, I will add an option to ts-jest to disable/enable support projectReferences and publish it into 25.5.1-alpha.1 , would you please help to test once it's published ?

See this repo. master branch passes with 25.3, while reproduction branch fails with 25.5 even though tests were not modified.

here is the fix for your issue https://drive.google.com/open?id=17p2-uk8Z26k0qd8NGhhtE7KOz3wntcLZ I have cloned your repo and test this fix against reproduction branch and it worked for me. Would you please check to confirm ?

I'll be able to test it in a few hours, but I'd assume it's OK. I'll give you pushing rights to the repository, you can test it against the CI.

that is great !

anyways, e2e tests already covered it :)

I can confirm that the fixed version you uploaded passes the CI tests.

Nice, the fix will be in 25.5.1 Thanks for your confirmation.

close via #1614

25.5.1 is out

Thanks, the tests pass as expected without tsc -b when using 25.5.1. Can I remove the reproduction repository now @ahnpnl?

sure 👍


A function to get number of TFLite model parameters

System information

TensorFlow version (you are using): 2.3
Are you willing to contribute it (Yes/No): No

Describe the feature and the current behavior/state. A function to get number of TFLite model parameters
Will this change the current api? How? It will add a new function to Interpreter
Who will benefit with this feature? People who need to optimize the model. They will be able to see the change in parameter number and estimate benefits
Any Other info. If there is a way to do it in a current version, please, let me know.

We do not have such API yet since the parameters are frozen into the constant tensors. However, the number of TFLite tensors will be approximately the same with the original TF model but they are not the same.

We do not have such API yet since the parameters are frozen into the constant tensors. However, the number of TFLite tensors will be approximately the same with the original TF model but they are not the same.

Thank you for your answer! But as I understand, It is only avaylable in C++ API. Is there a way to get this value through python API?

Thank you for your answer! But as I understand, It is only avaylable in C++ API. Is there a way to get this value through python API?

Flatbuffer object can be obtained to retrieve the information of the model directly.
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/tools/flatbuffer_utils.py#L40

Flatbuffer object can be obtained to retrieve the information of the model directly.
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/tools/flatbuffer_utils.py#L40


[0.9.1, 0.9.2] Polymer.dom().appendChild() no longer working

0.9.0 working jsbin
0.9.2 failing jsbin

Change was introduced in 0.9.1

That this worked in 0.9.0 was actually a bug. The behavior you're seeing now is correct.
The iron-icon element is being added as a child to dynamic-add, but dynamic-add has a local DOM which renders in place of its children. In order to distribute/render children, add an insertion point (<content>) to dynamic-add like this:
http://jsbin.com/zodefo/1/edit?html,output


Dataset and code Release

Great work! When do you plan to release the source code and dataset?

Thank you for your appreciation！We will complete the code and all data uploads in June.

@yzbouc Hi, great work! Could you let me know if you uploaded the complete dataset?

干得好！您计划何时发布源代码和数据集？

The code has been updated, you can download it from the main page.

@yzbouc Hi, great work! Could you let me know if you uploaded the complete dataset?

The code has been updated, you can download it from the main page.


⚠ Bofrost.be has degraded performance

In e924103, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1611 ms

Resolved: Bofrost.be performance has improved in cd6e2a9.


Stop 404s throwing exceptions

I am trying to test broken links but, when I get and run into a 404, I get an exception (see bottom) rather than a statusCode to handle. Only 200s seem to be returned.
I tried to follow the error-handling code in the tutorial but I can't find a way to return the same type as get u. Moreover, this seems to be more complexity than I need in this instance.
How can I simply prevent the exception and just return the responseStatus as is?
verifySeatme :: Maybe URL -> IO UrlStatus
verifySeatme url = do
    case url of
        Nothing -> return None
        Just "" -> return None
        Just u -> do
            seatmeResp <- get u --`E.catch` handler
            -- r ^? responseBody . key "url"
            -- could also check for redirect to errorPage.aspx
            if seatmeResp ^. W.responseStatus . statusCode == 200
            then return (Working u)
            else return Broken
    where
        handler e@(StatusCodeException s respHeaders _) =
            do
                return respHeaders

Here is the exception thrown, and you can see it has the statusCode I need
*Main CrawlerHTTP DbHandler Lib Data.Aeson Network.Wreq Control.Lens> re <- get "https://www.seatme.nl/restaurant/1371/Londen.htm"
*** Exception: StatusCodeException (Status {statusCode = 404, statusMessage = "Not Found"}) [("Cache-Control","private"),("Content-Type","text/html; charset=utf-8"),("Server","Microsoft-IIS/7.5"),("X-Powered-By","UrlRewriter.NET 2.0.0"),("p3p","CP=\"IDC DSP COR ADM DEVi TAIi PSA PSD IVAi IVDi CONi HIS OUR IND CNT\""),("X-AspNet-Version","4.0.30319"),("X-Powered-By","ASP.NET"),("Date","Tue, 15 Dec 2015 09:20:57 GMT"),("Content-Length","25576"),("X-Cache","MISS from seatme-lb01"),("X-Cache-Lookup","MISS from seatme-lb01:443"),("Via","1.0 seatme-lb01 (squid/3.1.10)"),("Connection","keep-alive"),("X-Response-Body-Start","\r\n<!DOCTYPE html PUBLIC \"-//W3C//DTD XHTML 1.0 Transitional//EN\"

Got an answer here


README How to develop section now also works on Apple M1

Which issue does this PR close?
Closes #939.
Rationale for this change
README previously contained instructions that only worked on Linux.
What changes are included in this PR?
The default Conda environment file no longer depends on CUDA Toolkit or cuDF. An alternative environment file is made available for CUDA support.
Are there any user-facing changes?
For Apple users, the README will start working. For Linux users where GPUs are supported, the old instructions will not install CUDA support, but alternative instructions are provided to get the old behavior.

Verification on an Apple M1
$ micromamba env create -f ./conda/environments/datafusion-dev.yaml -n datafusion-dev
...
Transaction finished

To activate this environment, use:

    micromamba activate datafusion-dev

$ micromamba activate datafusion-dev
$ maturin develop
...
 Installed datafusion-42.0.0

$ python3 -m pytest
================================================================== test session starts ===================================================================
platform darwin -- Python 3.12.7, pytest-8.3.3, pluggy-1.5.0
rootdir: /Users/davidr/workspaces/drauschenbach/datafusion-python
configfile: pyproject.toml
collected 465 items

python/tests/test_aggregation.py ..................................................................................                                [ 17%]
python/tests/test_catalog.py .                                                                                [ 17%]
python/tests/test_config.py ...                                                                                [ 18%]
python/tests/test_context.py .....................ss......................                                                                         [ 28%]
python/tests/test_dataframe.py ...............................................................................ssss............................     [ 52%]
python/tests/test_expr.py ..........                                                                                [ 54%]
python/tests/test_functions.py ................................................................................................................... [ 78%]
.......................                                                                                [ 83%]
python/tests/test_imports.py .....                                                                                [ 84%]
python/tests/test_indexing.py ..                                                                                [ 85%]
python/tests/test_input.py .                                                                                [ 85%]
python/tests/test_plans.py .                                                                                [ 85%]
python/tests/test_sql.py .........................x...xxxx........                                                                                [ 94%]
python/tests/test_store.py .                                                                                [ 94%]
python/tests/test_substrait.py ...                                                                                [ 95%]
python/tests/test_udaf.py .....                                                                                [ 96%]
python/tests/test_udf.py ...                                                                                [ 97%]
python/tests/test_udwf.py ............                                                                                [ 99%]
python/tests/test_wrapper_coverage.py .                                                                                [100%]

======================================================= 454 passed, 6 skipped, 5 xfailed in 2.89s ========================================================


Multiparameter protocols

I want to define the notion of an action of one type on another. For instance, the CoreGraphics type CGVector represents a 2D vector, and CGPoint represents a point on the coordinate plane. There should be a function
@infix |> (v: CGVector, p: CGPoint) -> CGPoint
that translates the point p by the vector v. I'd like to implement this as a protocol:
protocol Action {
    typealias A
    typealias M
    @infix |> (a: A, x: M) -> M
}

and instantiate it by extension to the pair (CGVector, CGPoint), but I get the error
Non-nominal type '(CGVector, CGPoint)' cannot be extended.
I can't think of a good workaround for this.
In Idris, I could implement this with a multiparameter typeclass:
class Action a m where
 (|>) : a -> m -> m

and then instantiate it as
instance Action CGVector CGPoint where
  (mkCGVector a b) |> (mkCGPoint c d) = mkCGPoint (a + c) (b + d)

where mkCGVector and mkCGPoint are the sole constructors of CGVector and CGPoint, respectively

@ejenk Given we're now at 1.1, I don't have high hopes for us getting multi-parameter typeclasses, and even less for the chance you'll get any kind of verification of rules.  There's serious language limitations standing between you and satisfaction that Swift will probably never be able to satisfy.  You can certainly fake it, and fake it hard, with individual witnesses like Max was saying, but it quickly turns into a Yak shaving competition.
For now I will close this issue, but rest assured any breakthroughs or midnight epiphanies means this gets reopened.


[WIP] [RHDM-697] CE build alignment for EAP-based OpenShift images in RHDM

[RHDM-697] CE build alignment for EAP-based OpenShift images in RHDM
https://issues.jboss.org/browse/RHDM-697

Note: This PR depends upon https://github.com/jboss-container-images/jboss-kie-modules/pull/70


Parceler: Unable to find read/write generator for type java.util.List

Hello,
Parceler produces a compile time error when I use it on a Kotlin data class, that accepts an immutable list of enum values. Here is the concrete example:
@Parcel(Parcel.Serialization.BEAN)
data class FeaturesResponse @ParcelConstructor constructor(var features: List<FeatureResponse>)

enum class FeatureResponse {
    FEATURE_1, FEATURE_2,
    UNKNOWN
}

And here is the error I get:
: Parceler: Unable to find read/write generator for type java.util.List<? extends FeatureResponse> for java.util.List<? extends FeatureResponse> features java.util.List<? extends FeatureResponse> features) {
                                                                                ^
e: /EnabledFeaturesResponse.java:12: error: Parceler: Types do not match for property features java.util.List<? extends FeatureResponse> java.util.List<FeatureResponse> public final java.util.List<FeatureResponse> getFeatures() {                                                                                ^

I think a possible workaround is to replace List With MutableList as the error doesn't appear then, but the issue remains.

This was reported recently here: https://github.com/johncarl81/parceler/issues/366.  I have a reproducible failing test case if you want to try your hand at a fix PR.


Handy little wrapper function for hideshow toggle (hs-toggle-hiding)

Hey!
I don't't know if you'd be interested or not, but I wrote a very tiny (but also very handy) wrapper for hs-toggle-hiding that makes it much easier to use. My biggest gripe with hideshow is that you have to be inside the scope of something you want to fold in order to actually fold it. In my ideal workflow, I could easily fold a function by just being anywhere on the line it starts on and easily initiate a fold for that function. This wrapper achieves that.
I've uploaded two short clips of me doing roughly the same thing with both functions to illustrate the differences between the two. Notice primarily how the first video always folds the outer scope based on the location of the point:
hs-toggle-hiding - video
modified-toggle - video
Modified toggle function:
(defun modified-toggle ()
  (interactive)
  (let ((saved-point (point)))
    (end-of-line)
    (hs-toggle-hiding)
    (goto-char saved-point)))

It plays very nicely in every scenario I've thrown at it. Anyways, feel free to close this issue, I just thought it was super handy and thought you might find it useful as well.
I know that in many scenarios your workflow is different than mine, so if I'm sending too many of these random issues please let me know :)

That's a good idea. This behavior makes much more sense. I've pushed it in https://github.com/hlissner/.emacs.d/commit/77eb06727c21c8e9c39cf59b2b50f75b4579b9ca.
Thanks again!

if I'm sending too many of these random issues please let me know :)

Not at all. I look forward to them! You've inspired more than a few changes around here and given me plenty to think about.

Awesome, I'll keep em coming :)


!!  THE MAKER OF THIS MODEL DOES NOT ANSWER, PLEASE, ASAP!
We would realyl appreciate it :(
IT HAS NOT TO BE REALLY ACCURATE, IT ONLY HAS TO LOOK LIKE IN THE PNG
AIRLINE: AIRVALENCIA
ICAO: VLC
Best wishes and take care!

A359_VLC.zip
It's no trouble. Here you go. For future reference, this is what I did:

Copy A359_LIT.png and any one of the .obj files.
Rename the .obj file to A359_VLC.obj.
Open A359_VLC.obj in a text editor and change the TEXTURE on line 5 to A359_VLC.png.
Create the xsb_aircraft.txt file.

I can't add it to the main Bluebell download because the A359 model is from a different source, but I believe you already know that.
Enjoy!

Oh that was quick!
I really appreciate your work! See ya!


Update CONTRIBUTING Guide

📓 Documentation Update
Add info about how to resolve conflicts in the CONTRIBUTING Guide

Can I work on this issue?

Hi @lee00286,
We are very happy you are interested in our project. You can work on this issue. I would like to mention that we need these changes done in the next couple of days due to the fact we intend to officially release  IceVision early next week. I you have time to do it, we will be happy to assist you in that process.
If, for any reason, you don't have the time to do it, don't worry there will be a plenty of opportunities to contribute to our projects (IceVision and IceData as we are very actively working on them.
If you are interested, you can join our Devs Forum: There is no obligation to join in order to contribute to the project!


New Layout

updated a few things, you can use some functions directly from z2ui5_cl_util. will check the other issues.

finally it works now! I will probably replace the database tables with the table of the utility functions, that we have just one persistance for generic data in the project. but for now it looks good, thank you for the PR!


Freshness broken for empty state

TypeScript Version:  nightly
Code
class Foo extends React.Component<{}, {}>{
  render() { return <noscript /> }
  foo() {
    this.setState({ something: true }); // Should error but doesn't
  }
}

:rose:

Isn't it because you can assign anything to {} ?
let a:{} = {something:true} // OK

It is and thats the issue I should have created.
But is there a reason why {} should be allowed to bypass the strict object checking (freshness) rule ? :heart:

FYI https://github.com/Microsoft/TypeScript/wiki/FAQ#why-are-all-types-assignable-to-empty-interfaces

@wallverb Thanks!

In general, you should never find yourself declaring an interface with no properties.

But we do cause setState is dangerous for beginners<EMAIL_ADDRESS>so we swapped state to {}
And then migrating code from setState to @observable becomes painful to clean up as all those calls to setState don't highlight as errors. We instead needed to painfully find references to setState :rose:

@basarat Maybe exact types would help https://github.com/Microsoft/TypeScript/issues/12936
Here is the result in Flow:
type Foo = {  };
type Bar = {|  |};

const foo:Foo = { a: 4 } // Works
const bar:Bar = { a: 5 } // Error Property `a` not found in object type

They would help.
I would still argue that it is okay that {foo:true} works with {}, but should respect the freshness check just like other things that fail in the presence of freshness :rose:

{ } is a frequent "safer" alternative to any for input positions. Freshness detection is generally intended to detect typos (this.setState({ weigt: 12 });); the "typo" of including a property where none are allowed (?) is a very strange error to make. It's very unlikely the recipient of the value demands an empty object -- why would anyone be doing that?

why would anyone be doing that

@RyanCavanaugh for migrating off of setState : https://github.com/Microsoft/TypeScript/issues/13423#issuecomment-272020013 :rose:

If you want something that nothing is assignable to, use void

If you want something that nothing is assignable to, use void

Excellent. That solves my need. Closing as I don't need this changed anymore :rose: and the decision to do the freshness checks for {} is a choice I leave for you / other humans 🤗 , create an issue or ignore :heart:


恳请context里面加个GetUint64(key string)方法

从gin重构的hertz，恳请在context上加个GetUint64(key string)方法

I think the Get*** method could be added.

Please assign me if nobody works on this.


Add flops profiler tutorial

Added the flops profiler tutorial, configuration, and feature to the website. Also fixed names of some flops profiler function parameters.

In the inference module, can I add the performance analysis tutorial of llama2?


Fix markdown rendering of Connection::transaction doc

This fixes DropBehavior::Commit being rendered as DropBehavior: :Commit.

Thanks.


⚠ Bofrost.be has degraded performance

In 01c3aa8, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1578 ms

Resolved: Bofrost.be performance has improved in 44c8940 after 6 minutes.


"Format on Save" add "Quick Save" option to save to disk before formatting

Feature

Checkbox setting in Text Editor -> Formatting for "Quick Save when Formatting on Save"
VS Code then will immediately write the file to disk, then pass it to the formatter, and update the disk after successful formatter completion

Use Case:

Large files (4k lines+) often take several seconds or more to format with some beautifiers.
Often devs will save a file, refresh in a browser to see results... only to have to wait several more seconds to refresh again while beautifier finishes work on the file.

I attempted to search for this issue before submitting it... but was only looking at open issues. Apparently this has been brought up a few times and closed without solving it.


[Enhancement] support incremental scan ranges  deployment at FE side

Why I'm doing:
What I'm doing:
CoordinatorPreprocessor
Added assignIncrementalScanRangesToFragmentInstances interface, which recalculates the scan ranges of all fragment instances below a fragment.
Currently the distribution of node2ScanRanges is supported, but more complex data distributions such as node + driver_seq -> scan range are not supported for incremental deployment.
ExecutionSchedule
We currently support two scheduling methods: all-at-once and phased. The difference is that they schedule different sets of fragment instances.
The modification to the code is that after each collection of fragment instances is scheduled, it immediately checks to see if there are any scan nodes left in the fragment instances that have not yet been scheduled. If there are, the scheduling process will continue.
The method used for subsequent scheduling is coordinator.assignIncrementalScanRangesToDeployStates(deployer, states);
Coordinator
Adds the assignIncrementalScanRangesToDeployStates interface, which checks the scan node for more scan ranges based on the deploy states (i.e., the set of locally dispatched fragment instances).

Checks all the following fragment instances against each deploy state.
According to the fragment instances corresponding to these fragment instances, see if there are still scan nodes with scan ranges.
If so, call coordinator preprocess to recalculate node2ScanRanges under fragment instances.
Based on these fragment instances, regenerate the rpc request, and put it under the deploy state.

Arch Diagram

Fixes #50196
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

I will get this error in the following session variables.

I will get this error in the following session variables.

if you want to test, you can test this PR [Enhancement] support incremental scan ranges deployment at BE side by dirtysalt · Pull Request #50254 · StarRocks/starrocks
https://github.com/StarRocks/starrocks/pull/50254
there need to be some modification on BE side too.

Could you do not send empty scan range to all BE,  maybe we can send to the incremenl scan range to the worker which already has fragment instance before?

How do you deal with the incremental partition  in hiveTable toThfift

How do you deal with the incremental partition in hiveTable toThfift

Don't handle it in this PR. for hive scan node, there is no need to handle incremental partition.

Could you do not send empty scan range to all BE, maybe we can send to the incremental scan range to the worker which already has fragment instance before?

Yes. It will be more complicated, we can optimize in next PR. We can

add a extra flag to fragment instances which means it already has fragment plan.
in the end, if there is no more scan range, we have to send a empty scan range to fragment instance to notify it "all is done".

anyway, it's just an optimization.

@Youngwb

I don't think the placeholder is good name, how about is_empty ?

I can not reply this comment.   No problem, I can fix that.


Document whether there is WASM support

It's unclear whether WASM is supported. I assume it's probably not, but it would be informative if this was mentioned either way

It looks like there is not, at this moment (I tried to build the example test on wasm32-wasi and that did not work).

Is that even possible to implement linkme for wasm32 targets?


🚀 - Sort direction parameter

Which @taiga-ui/* package(s) are relevant/releated to the feature request?
addon-table
Description
Currently, when specifying the direction to sort, you need to pass either -1 | 1. At first glance, this seems less clear than providing an explicit direction, ie. asc | desc.
My first assumption was that since it's supposed to the direction, then -1 would refer to asc since the smaller values would come first:
const makeSort1 = (direction) =>
  (a, b) => direction === -1 ? a.localeCompare(b) : b.localeCompare(a);

However it seems as though the direction is the multiplier applied to the underlying sort, so 1 is ascending:
const makeSort2 = (direction) =>
  (a, b) => a.localeCompare(b) * direction;

Given the second interpretation, this makes sense, however I think it would be clear to use asc | desc explicitly. From an intent perspective, I feel like this would make the API easier to use.
I realize this isn't a huge deal and also would be a breaking change, but I would appreciate your consideration on this issue.

No need for a breaking change, it can be an additional directive:
export class HumanReadableDirection {
  @Input()
  set directionName(name: 'asc' | 'desc') {
    this.table.direction = name === 'asc' ? 1 : -1;
  }

  @Output()
  readonly directionNameChange = this.table.directionChange.pipe(
    map(dir => dir === 1 ? 'asc' : 'desc'),
  );

  constructor(private readonly table: TuiTableDirective) {}
}


Discussion about Istio enablement on Arm64 platform

Discussion about Istio enablement on Arm64 platform
In this request, I want to discuss about how to enable Istio on Arm64 platform. I know, so far, there are some difficulties on Istio enablement on Arm64. Could we discuss it for further development on Arm64?
Potential issues for enablement

Dependent repositories are not supported arm64
Although most of dependent repositories have support arm64 platform, there are still some repo do not support. For those issues, could we temporarily walk around and use the private repository to replace until official supported.
How to build CI environments?
As far as i know, the Prow in Istio is running on Google cloud. It does not support arm64 environment. But the similar issue about it has been raised in Kubernetes/test-infra. After this issue resolved, i think the istio could use the similar architect for building Arm CI environments.
How to verify code
In my opinion, we could firstly enable Istio  native build on Arm64. In this phase, it only verifies code on x86 platform. In next phases, we could build Istio Arm64 CI environments. Finally, build all the feature of Istio on Arm CI environments:
Phase1: Enable Istio native build on arm64 platform.
Phase2: Build Istio Arm64 CI environments.
Phase3: Enable all features on Arm64

What is your opinion about it?

@hzxuzhonghu

@istio/wg-environments-maintainers

We have a lot of information about this tracked in https://github.com/istio/istio/issues/21094, let's move the discussion there? Thanks!


API's responses are different from the documentation

Issue:
We've been experiencing some odd behaviors that differ a lot from what's in the API's documentation, and we would be grateful if someone could help us.
===================================================================
When posting an invite to create a permission, we are not getting that permission's ID back in the response.
See the attached file:

endpoint:
https://graph.microsoft.com/v1.0/groups/id/drive/items/id/invite
===================================================================
When listing the permissions, we get only about 40 results back. It's like a default value for a number of results in a first page, but we also get no "next link" to get the next results.
endpoint:
https://graph.microsoft.com/v1.0/groups/id/drive/items/id/permissions
===================================================================
If we try to change the page size to 3, for example, we still get no "next link" to go to the next page, but we do get 3 items in the response.
endpoint:
https://graph.microsoft.com/v1.0/groups/id/drive/items/id/permissions?$top=3
===================================================================
If we change the "top" parameter to 100, it still brings us about 40 items back in the response, but we have given a lot more permissions than that, so we know we should be getting more items. Still, no "next link" to go to the next page.
endpoint:
https://graph.microsoft.com/v1.0/groups/id/drive/items/id/permissions?$top=100
===================================================================
If we use the filter parameter, we still encounter the same problems of having one default page size, and a problem that seems worse than that. Apparently, we are filtering only this first default-sized page. So if I filter by "link/webUrl" and the item was in the first default page, we get the result back. If the item was created after those first items that are in the first page, we don't get it back.
endpoint:
https://graph.microsoft.com/v1.0/groups/id/drive/items/id/permissions?$filter=link/webUrl eq 'webUrl value'
===================================================================
These problems have created an issue where we can give permissions, but in most cases, we are unable to revoke those permissions, because we need its ID to delete it.
We need either the ID to be sent to us when we first create the permission, or the filter parameter to work properly when listing permissions.

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Float-labels.js doesn't copy asterisk (*)

I have a label like "First Name *". When Float-labels copies this as the placeholder for an associated input, it removes the asterisk. I'm not sure why this is happening, but I feel like the JS component is trying to be a little too clever here. Can I get it to just copy labels as-is?

I discovered that, like #9, setting prioritize: 'placeholder' and inserting the text into the placeholder directly allowed this to work, but this still seems like bizarre behavior.

Have you tried adding an asterisk to required fields with CSS?
.fl-is-required label.fl-label::after {
    content: '\2009\002a';
}```

Removing the asterisk from the label allows for more flexible styling options.

You can also add it back by using the `customLabel` option  callback.

If this is not adequate, help me understand why.

Ok I've released v3.2.0. This version improves the placeholder logic for SELECT fields and adds a customPlaceholder callback option. You can now do the following:
const floatlabels = new FloatLabels( '.form-1', {
    // append an asterisk to the labels of required fields
    customLabel: function( labelEl, el ) {
        return labelEl.textContent + ( el.getAttribute( 'required' ) !== null ? ' *' : '' );
    },
    // append an asterisk to the placeholders of required fields
    customPlaceholder: function( placeholderText, el ) {
        return placeholderText + ( el.getAttribute( 'required' ) !== null ? ' *' : '' );
    },
});


Impossible to Find Accurate Range of Link in BPElement

Currently there is no way to get the range of a link (in the parsed string) from a BPElement.
Using a non-BPMarkdownView view to present Bypass-parsed stuff (such as a UILabel) prevents links from being interactable. So, I traverse the BPDocument's elements and childElements to find all the links (elements.elementType == BPLink && element.attributes[@"link"]) and then addAttributes with NSLinkAttributeName to the parsed attributed string. (Note: I'm not using a BPMarkdownView because it doesn't automatically detect links like a UILabel, but perhaps using a regex to match links and adding them manually to the BPMarkdownView would be more sane than what follows...)
Unfortunately, if there is more than one string match for the link element text (such as the text "link" appearing more than once, each a separate hyperlink), then there is no way to discern which should be matched with which address.
Because there is no way to find the exact range of the link within the parsed string, if the hyperlinked text is found more than once, there is no way to accurately link the correct text. To make sure at least the links are duplicated, rather than making all matches overwrite each other (i.e. the first match for the string "link" gets hyperlinked over and over with every link address attribute because there is no way to find the specific range of the hyperlink, so rangeOfString just throws out the first find), I have kept all matches in a dictionary, mapped to the range of where it was found. Then the next time the same string has a hyperlink, I start the rangeOfString with the location in the dictionary. But, if there are matches in the original string before or between the matches, then they will be incorrectly hyperlinked. I've considered analyzing the individual childElements and substituting the dictionary for the index of the current element step in the parsed string, but that seems like overkill -- in a way just writing a stepping algorithm for the stepping algorithm.
Is there any better way to achieve what I'm looking for? I'm just trying to add tappable links to a UILabel using Bypass, and it's turning into a nightmare! If I could add rules to the Bypass infrastructure, like adding attributes to certain kinds of matches or symbols (which could also allow me to add support for other markdown matches like ^[]), or if there was a simple workaround / modification for this, that would be a welcome improvement.

hi insanj,
I'm not fully sure but if I remember correctly from twitter you're working on a reddit client. in this case (or if not and you're just interested in fast markdown parsing including links) you might wanna take a look at my branch:
https://github.com/myell0w/bypass-core/tree/reddit-extensions?files=1

@myell0w Thanks for the suggestion! Is there any documentation available for this branch? Should I just substitute the files from the existing bypass-core src with yours? What do you mean by "fast markdown parsing including links" -- is this built-in?

there's sadly no documentation as I just built this for my own use, I think the best would be if you check out the commits that I made. you can hit me up on Twitter and I'll try to help you, if you have any specific questions.
as for the integration, bypass was splitted up into bypass-core and bypass-iOS at the time I used it, not sure how it progressed afterwards. so I also have a fork of bypass-iOS that integrates with my fork of bypass-core.
and last but not least, yes - link parsing (of non-markdown-annotated links) is built in, so there's no need for another pass to parse them, they get parsed with the markdown (as I modified the C parser to trigger on 'h')


error when running npm install command on Ubuntu

959 silly install printInstalled
960 verbose stack Error: Registry returned 500 for GET on http://registry.npmjs.org/marked-man/-/marked-man-0.1.5.tgz/chai
960 verbose stack     at makeError (/home/rrababah/.nvm/versions/node/v6.9.4/lib/node_modules/npm/node_modules/npm-registry-client/lib/request.js:302:12)
960 verbose stack     at CachingRegistryClient. (/home/rrababah/.nvm/versions/node/v6.9.4/lib/node_modules/npm/node_modules/npm-registry-client/lib/request.js:280:14)
960 verbose stack     at Request._callback (/home/rrababah/.nvm/versions/node/v6.9.4/lib/node_modules/npm/node_modules/npm-registry-client/lib/request.js:210:14)
960 verbose stack     at Request.self.callback (/home/rrababah/.nvm/versions/node/v6.9.4/lib/node_modules/npm/node_modules/request/request.js:187:22)
960 verbose stack     at emitTwo (events.js:106:13)
960 verbose stack     at Request.emit (events.js:191:7)
960 verbose stack     at Request. (/home/rrababah/.nvm/versions/node/v6.9.4/lib/node_modules/npm/node_modules/request/request.js:1048:10)
960 verbose stack     at emitOne (events.js:96:13)
960 verbose stack     at Request.emit (events.js:188:7)
960 verbose stack     at IncomingMessage. (/home/rrababah/.nvm/versions/node/v6.9.4/lib/node_modules/npm/node_modules/request/request.js:969:12)
961 verbose statusCode 500
962 verbose pkgid marked-man
963 verbose cwd /media/ssd/dev/source/github/atypon/styleguide/ux3/user
964 error Linux 3.13.0-52-generic
965 error argv "/home/rrababah/.nvm/versions/node/v6.9.4/bin/node" "/home/rrababah/.nvm/versions/node/v6.9.4/bin/npm" "install"
966 error node v6.9.4
967 error npm  v3.10.10
968 error code E500
969 error Registry returned 500 for GET on http://registry.npmjs.org/marked-man/-/marked-man-0.1.5.tgz/chai
970 error If you need help, you may report this error at:
970 error     https://github.com/npm/npm/issues
971 verbose exit [ 1, true ]
If known, describe the steps to reproduce the issue:
when running npm install command

It looks like the issue is not related to crouton or chromeOS : Your NPM tries to retrieve something online and is not able to do so (error 500 returned on GET).

To add:

970 error If you need help, you may report this error at:
970 error https://github.com/npm/npm/issues

Seems to be 'npm issues' / 'node issues' not 'crouton issues' ...
-DennisL


Error "UnhandledPromiseRejectionWarning"

While trying a backtest with a TAlib strategy, this error appears :
`
Gekko v0.4.10
I'm gonna make you rich, Bud Fox.
TAlib is enabled
Serving Gekko UI on http://localhost:3000/
(node:3635) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Exited with code 3
`
I've tried to reinstall npm, with no effect. My OS is Debian.

You can ignore that error, it is caused by Gekko automatically wanting to start a browser window to open Gekko. But it can't on a server (because there is no browser). To remove that, set headless to true in the UI config.
See here for details: https://gekko.wizb.it/docs/installation/installing_gekko_on_a_server.html#Configuring-Gekko-to-run-headless-in-a-trusted-environment


affected/package:

What version of Go are you using (go version)?

$ go version
go version go1.18.4 darwin/arm64

But bazel is using 1.18.9

Does this issue reproduce with the latest release?
I don't know — haven't tried.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/nyap/Library/Caches/go-build"
GOENV="/Users/nyap/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/nyap/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/nyap/go"
GOPRIVATE=""
GOPROXY="https://artifactory.ti-apps.aws.grail.com/artifactory/api/go/go,https://proxy.golang.org"
GOROOT="/Users/nyap/.gimme/versions/go1.18.4.darwin.arm64"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/nyap/.gimme/versions/go1.18.4.darwin.arm64/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.18.4"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="0"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/w7/l4dcw2ss1l98wqcxsx1pq03h0000gs/T/go-build547156787=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Code:
package main

import (
	"fmt"

	"github.com/samber/lo"
	"github.com/volatiletech/null"
)

type Color int32

const (
	GRAY Color = 0
)

func main() {
	colors := NullJSONFromSlice([]Color{GRAY})

	fmt.Printf("%v\n", colors)
}

func NullJSONFromSlice[T any](ts []T) null.JSON {
	if ts == nil {
		return null.JSON{}
	}

	return NullJSONFrom(ts)
}

func NullJSONFrom[T any](arg T) null.JSON {
	var result null.JSON

	lo.Must0(result.Marshal(arg)) // can never error

	return result
}

When using go build ., code builds fine and executable runs as expected.
When using bazel build ., build errors with internal compiler error: assigning ts (type []Color) to parameter arg (type go.shape.[]int32_0).
If the call to lo.Must0 is changed to _ =, the error goes away.
bazel build --sandbox_debug . shows that it's running:
( cd /private/var/tmp/_bazel_nyap/29cd39e755dc2150ecde970986aff8d0/sandbox/darwin-sandbox/43/execroot/grail &&   exec env -     CGO_ENABLED=1     GOARCH=arm64     GOOS=darwin     GOPATH=''     GOROOT=external/go_sdk     GOROOT_FINAL=GOROOT     PATH=/usr/bin:external/llvm_toolchain/bin:/bin     TMPDIR=/var/folders/w7/l4dcw2ss1l98wqcxsx1pq03h0000gs/T/   /usr/bin/sandbox-exec -f /private/var/tmp/_bazel_nyap/29cd39e755dc2150ecde970986aff8d0/sandbox/darwin-sandbox/43/sandbox.sb /var/tmp/_bazel_nyap/install/3c4a7a3c33b4c067b191587b066e5f1b/process-wrapper '--timeout=0' '--kill_delay=15' bazel-out/darwin_arm64-opt-exec-2B5CBBC6/bin/external/go_sdk/builder compilepkg -sdk external/go_sdk -installsuffix darwin_arm64 -tags bazel,bazel -src go/src/grail.com/order-manager/tools/aoeu/aoeu.go -embedroot '' -embedroot bazel-out/darwin_arm64-opt/bin -embedlookupdir go/src/grail.com/order-manager/tools/aoeu -arc 'github.com/samber/lo=github.com/samber/lo=bazel-out/darwin_arm64-opt/bin/external/com_github_samber_lo/lo.x' -arc 'github.com/volatiletech/null=github.com/volatiletech/null=bazel-out/darwin_arm64-opt/bin/external/com_github_volatiletech_null/null.x' -importpath grail.com/order-manager/tools/aoeu -p main -package_list bazel-out/darwin_arm64-opt-exec-2B5CBBC6/bin/external/go_sdk/packages.txt -o bazel-out/darwin_arm64-opt/bin/go/src/grail.com/order-manager/tools/aoeu/aoeu.a -x bazel-out/darwin_arm64-opt/bin/go/src/grail.com/order-manager/tools/aoeu/aoeu.x -nogo bazel-out/darwin_arm64-opt-exec-2B5CBBC6-ST-eda1e837b650/bin/bazel/go/nogo_actual_/nogo_actual -gcflags '' -asmflags '' )

What did you expect to see?
Build succeeds with both go build . and bazel build .
What did you see instead?
Build succeeds with go build . and fails with bazel build .

We no longer support 1.18. There were several issues with the generics implementation in 1.18 that have since been fixed. Please try a later release and see if your issue persists. If so, you can reopen.


Rhai: Runtime error: No version preamble in spec (line 7, position 6)

i'm getting this error here (here)[https://github.com/terrapkg/packages/actions/runs/5340646352/jobs/9680694508#step:5:1] when an updated is attempted for prismlauncher-nightly/prismlauncher-qt5-nightly in terrapkg. i'm not really sure what's going on here, and this seems to have only started recently

That's partly my issue because I updated the update script to try and use both the version and the commit automatically but I forgot andax doesn't accept macros in version preamble.


Shortcodes.js error, SyntaxError was thrown: const lazy = args[4] ?? true;

I get the following error when I try and build the site.
SyntaxError was thrown:
const lazy = args[4] ?? true;
Here are the two lines that have ? in them. Not sure what is wrong as I don't develop.
onst lazy = args[4] ?? true;
const sizes = args[5] ?? defaultSizes;
Could the problem be the ? mark?
Thanks,
Paul

The site builds correctly when I changed the node version to 14.15.1, per the .nvmrc file. The image is pushed to the left, so there must be a CSS problem which I will look into. Sorry for the too soon to make an issue. I will do better to check things out in the future.

Hey there, this is due to the fact that node does not support the null coalescing operator ?? in the version you are using.
There is a .nvmrc file at the root of the repo. This should be the minimum node version to use.
See https://github.com/nvm-sh/nvm for some more info


Change the number of input points

Thanks for the great work
I have a question regarding network structure: is it possible to change to input point number without re-training the network from scratch? Thanks

Hi @xingruiy,
you can modify "num_points" in the config files:
https://github.com/JuanDuGit/DH3D/blob/master/models/local/config.json
https://github.com/JuanDuGit/DH3D/blob/master/models/global/config.json
After you change this number to x, the network will assume it to be the size of all the input point clouds. Each input point cloud will first be sampled using a voxel grid. If the number of points is smaller than x after sampling, the point cloud will be padded; if larger than x, only x points will be used.
So the pre-trained model can be directly applied on point clouds with different sizes. However, the performance is not guaranteed. More specifically, the density of the point clouds will influence the local descriptor, and this will further influence the global descriptor. Additionally the global descriptor is also correlated to the coverage of the point clouds.
Hope this is useful.

@rui2016  Many thanks!

@rui2016  Many thanks!


How do you use the --excludeExternal flag ? and what do mean by "Prevent externally resolved files"

I am working on a project that has some internal functions that I do not want to include in the documentation. I am fully aware of the @hidden doc string namespace but every time I return back to the global.html page I still see the function on the documentation page after running the command. I even tried --excludeExternals  not including @hidden the function and its documentation still show have been trying to not include certain files from my src folder

I think --excludeExternals was designed to prevent certain files from being included in the documentation (see --externalPattern). I'm not sure why @hidden wouldn't work. Do you have an example repo?

Nevermind I created a script that removes them from the docs after they are done being generated for the problem


Directive use (ts-mongo) failing when using import feature

Describe the bug
When using directives such as @entity together with # import I get the following error:
 Directive entity: Couldn't find type entity in any of the schemas

To Reproduce
I've created the following repo to reproduce the file: https://github.com/amiiit/graphql-directive-import-example
git clone git@github.com:amiiit/graphql-directive-import-example.git
cd graphql-directive-import-example
yarn
yarn codegen-with-import # failing case
yarn codegen-no-import # success case

Steps to reproduce the behavior:

Use # import Foo from foo.graphql
In foo.graphql have a type Foo @entity {...}

Expected behavior
Should create types for the mongodb entities
Schema/Documents
The schema using the import statement
The imported part of the schema that uses the @entity directive
Environment:

OS: Mac OS Sierra 10.12.6
Codegen: 0.16.0
Node: v10.12.0

I think it was fixed in the recent refactor, also we fixed something related to it in graphql-toolkit.

Fix available in 1.0.0 👍


Add explanation about how to create new examples

Detail a step-by-step procedure.
Explain which file should be written (point to and example)
Explain which file ares generated automatically (and possible caveats)

Will add a CONTRIBUTING.md file soon.

Added as package documentation.


Getting api rate limits fails / treat header fields case-insensitive

Hi,
when calling InstagramObject.getAPILimitStatus() / InstagramObject.getRemainingLimitStatus() always -1 is returned. Turns out the implementation of HttpURLConnection (com.google.apphosting.utils.security.urlfetch.URLFetchServiceStreamHandler$Connection) is converting the headers to lower-case and thus looking them up via key in the hashmap fails.
My first thought was to blame the HttpURLConnection implementation. But as it seems http headers are meant to be case-insensitive ( http://stackoverflow.com/questions/5258977/are-http-headers-case-sensitive ). So can't really blame them.
Maybe storing the headers using a java.util.TreeMap with a String.CASE_INSENSITIVE_ORDER comparator could fix this issue? As a quickfix I will work around with InstagramObject.getHeaders().
Regards,
xberger

Closing the ticket... Re-open if needed


Error

Delphi 10.2.3
alcinoe compile for Tokyo
Empty project with just a memo
compiled for Android 24.3.3 32 bits
Error
'java.lang.NullPointerException: Attempt to invoke virtual method '''java.lang.ClassLoader java.lang.Class.getClassLoader()''' on a null object reference'

Funny but the next day it works OK

Hello, everything has been fixed in the latest committed version.


Use Yarn

Same thing as with jest. Do you want to use it?

[x] #849 to be merged (AppVeyor throws error)

Don't we want to make sure we always get the newest versions of everything on CI tests? I feel like it's actually better to not have the features of Yarn here, but I could be convinced otherwise.

@stubailo apollo-client's CI configuration enables node_modules caching so you use still don't use latest, only those that fit requirements
I updated travis in #865 to run npm prune before npm install so it will remove all unused packages before compiling apollo-client (prepublish script).
I created this PR just to open the discussion :)

@kamilkisiela is there a compelling reason to switch to yarn at the moment? What are the pros and the cons?

Yeah, I don't think this benefits us a lot.

Moreover the recent yarn release 0.17.0 has lots of regressions. The obvious advantage of yarn is install speed. But without a stable release process or more tests to catch regressions, I'd be cautious about using it as the default.


Help with ESRI setup for Bloomington

Is your feature request related to a problem? Please describe.
From Jeremy in slack:
I've been setting up ESRI service areas. The Municipal Boundary service area works flawlessly.
However, I am stuck getting the Bloomington Economic Zone (BEZ) service area to correctly validate. Addresses I try to validate are ineligible despite being within the zone.
I might be missing something obvious. Any help would be appreciated!
Here's my configuration
ESRI_FIND_ADDRESS_CANDIDATES_URLS.0: 'https://bloomington.in.gov/arcgis-server/rest/services/Locators/CityCountyAddressLocator/GeocodeServer/findAddressCandidates'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_URLS.0: 'https://bloomington.in.gov/arcgis-server/rest/services/DataPortal/DataPortal_PoliticalBoundaries/MapServer/3/query'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_LABELS.0: 'Municipal_Boundary'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_ATTRIBUTES.0: 'muni_name'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_IDS.0: 'Bloomington'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_URLS.1: 'https://bloomington.in.gov/arcgis-server/rest/services/DataPortal/DataPortal_EconomicDevelopment/MapServer/3/query'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_LABELS.1: 'BEZ_West'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_ATTRIBUTES.1: 'objectid'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_IDS.1: '5'
Using objectid or objectids did not produce any errors --but did not correctly validate addresses - always noted as ineligible.
Using zone_section does produce this error with HTTP 400: Invalid service area ID: Downtown Expansion
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_URLS.1: 'https://bloomington.in.gov/arcgis-server/rest/services/DataPortal/DataPortal_EconomicDevelopment/MapServer/3/query'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_LABELS.1: 'BEZ_West'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_ATTRIBUTES.1: 'zone_section'
ESRI_ADDRESS_SERVICE_AREA_VALIDATION_IDS.1: 'West Expansion'
Describe the solution you'd like.
This should correctly validate addresses - us to make sure nothing is wrong with the config
What priority is this?
Hoping to get this working soon, so applicants don't have to go to an outdated system to see if eligible
Associated civic entity
Bloomington
Additional Context
Eligibility:

Done when:
They can correctly validate addresses

The next thing I'd like to try is overriding the well known ID (https://docs.civiform.us/it-manual/sre-playbook/initial-deployment/configure-gis-service#overriding-the-default-wellknownid) - when making the query to https://bloomington.in.gov/arcgis-server/rest/services/DataPortal/DataPortal_EconomicD[…]quantizationParameters=&featureEncoding=esriDefault&f=pjson, I'm seeing wkid as 2245. This would mean setting export ESRI_WELLKNOWN_ID_OVERRIDE to 2245, but I'm not totally sure if that would impact the other validation

This is related to #7137 and #7944 and will be fixed by the work I'm doing to support checking for outside service area. I'm anticipating it being in the release after Labor day.

Example Address that would not be eligible
https://bloomington.in.gov/arcgis-server/rest/services/Locators/CityCountyAddressLocator/GeocodeServer/findAddressCandidates?Address=401+N+Morton+St&City=Bloomington&Region=IN+&Postal=47404&f=pjson
https://bloomington.in.gov/arcgis-server/rest/services/DataPortal/DataPortal_EconomicDevelopment/MapServer/3/query?geometry={''x''%3A3107700.5018695998%2C''y''%3A1428812.12653800473%2C''spatialReference''%3A2966}&geometryType=esriGeometryPoint&inSR=2966&outFields=*&returnGeometry=false&f=pjson
Example Address that would be eligible
https://bloomington.in.gov/arcgis-server/rest/services/Locators/CityCountyAddressLocator/GeocodeServer/findAddressCandidates?Address=727+W+2nd+St&City=Bloomington&Region=IN+&Postal=47404&f=pjson
https://bloomington.in.gov/arcgis-server/rest/services/DataPortal/DataPortal_EconomicDevelopment/MapServer/3/query?geometry={''x''%3A3106413.97826135764%2C''y''%3A1425682.16073121782%2C''spatialReference''%3A2966}&geometryType=esriGeometryPoint&inSR=2966&outFields=*&returnGeometry=false&f=pjson

Thank you, @gwendolyngoetz !!

Confirmed that Gwen's change fixed this issue!


Bug Fixes + Enhancements

This PR contains:

[x] bugfix
[x] feature
[x] refactor
[ ] tests
[ ] documentation
[x] others

Breaking Changes?

[ ] yes
[x] no

If yes, please describe the breakage.
 Thanks for the ping.

@JStw published in v5.0.0-alpha.19.


prMessage: suggest :+1: reaction to important pr

Suggest adding reaction to PR to help rank importance.
I hereby license my contributions to this repository under:
Creative Commons Zero v1.0 Universal (SPDX Short Identifier: CC0-1.0)

Good idea, but you'll also have to update the test data in https://github.com/nix-community/nixpkgs-update/tree/main/test_data.

Thanks!


Remove duplicate check for space character (32)

_CharCode.SP is 32, which is already covered by the byte > 31 part.
Should I do this change also in the NNBD version, or how are these kept in sync?

See https://dart-review.googlesource.com/c/sdk/+/138787

lgtm. Sorry about the delay on this one, it fell through the cracks. Thank you for this improvement! The change is obviously correct. The sdk_nnbd directory has been merged into the regular sdk at this time.


Norwegian Wordnet Bokmål. Models

This issue provides information about the baseline and models that have been tested on the data of the Wordnet from the Norwegian National Library.
Models that have already been tested for provided data:

Most frequent sense baseline (with the using of Norbert2)

A number of tests with different input data on Norbert2, Multilingual Bert (12 models are evaluated by the moment)

Other tests are in the process of implementation.

The first version of documentation is uploaded to the gilthub
The first version of the documentation can be found by the link
What was done:

the POS-tagging and Sentiment analysis files were optimised.
all models for POS-tagging task and sentiment analysis were ran once more to check the results are relevant. The files with the results for this can be found here. The summary is here
The merged script for bert and roberta were run on Saga (but failed in some places).

What have to be done by 22.08:

The codes for POS-tagging and Sentiment analysis should be updated for distilbert as well
The scripts for NER task for bert and roberta should be merged (and not fail)

I believe this was for #4 ?


Jodd + Java 8 : getting error INVOKESPECIAL/STATIC on interfaces require ASM 5

Hi,
I invoke Methref like shown in the doc, usually, it works.
But it doesn't work on a class that use some code on an interface with a default method (Java 8). Jodd doesn't seem to support it :
Caused by: java.lang.IllegalArgumentException: INVOKESPECIAL/STATIC on interfaces require ASM 5
	at jodd.asm5.MethodVisitor.visitMethodInsn(MethodVisitor.java:477)
	at jodd.asm5.ClassReader.readCode(ClassReader.java:1416)
	at jodd.asm5.ClassReader.readMethod(ClassReader.java:1017)
	at jodd.asm5.ClassReader.accept(ClassReader.java:693)
	at jodd.asm5.ClassReader.accept(ClassReader.java:506)
	at jodd.proxetta.ProxettaBuilder.process(ProxettaBuilder.java:181)
	at jodd.proxetta.ProxettaBuilder.define(ProxettaBuilder.java:224)
	at jodd.methref.MethrefProxetta.defineProxy(MethrefProxetta.java:33)
	at jodd.methref.Methref.<init>(Methref.java:34)
	at jodd.methref.Methref.on(Methref.java:53)

The message seems complain about a missing requirement (ASM 5), but the name of the package suggest it is already the case (jodd.asm5...)
What am I supposed to do ?
Regards,
Philippe

I will check it out:) It looks it is related to ASM library... Others have similar problems (and solutions)

I have update to ASM5 usage; now the default methods get wrapped as any other methods.
Would you be so kind to check master if it works for you? Thank you!
And thank you very much for submitting issue! Hope the fix will work for you :)

So far, I have refactored my code, and the part that were using Java 8 features is no longer reachable during the scan : it works...
Tx for your help

No problem!


Unable to store complex data for each item of a dropdown

Something we support on the Semantic-UI-Ember side is storing a model for each item of the dropdown and setting that model when it's selected. This works great for when you're setting up a relationship within Ember-Data.
However, it seems this change casts anything within data to a String and breaks this functionality completely.
https://github.com/Semantic-Org/Semantic-UI/commit/c43ea73f7f0622441dc13cbee067ecbb1114c5b9#diff-dbb1b936fdad1f63b3a01f86d9a54fd1L1565
For now we've downgraded to version 2.0.8 which is right before that part was released.
Here is an example of how we're using it: https://github.com/Semantic-Org/Semantic-UI-Ember/blob/ce7cff9b40a2d83f1231dd081f3dec878b195d93/tests/dummy/app/templates/modules/dropdown.hbs#L52

Since dropdown is designed to replicate selection for an input it passes around strings for value which fixes issues when preset values like <input value="false"> are compared to values set in javascript programmatically.

It makes sense when you're trying to match up with values that were defined in HTML only.
However, in our rich javascript world everything is wrapped in a component and upon initialization it is setting the value to what it should be. Having to work with strings will add some unnecessary complexity to any dropdowns that we have because on each value change we'll need to search the data store for the right corresponding record and then emit it from the component. Where-as now each record is bound to the item itself so we simply take that and return it.
From Semantic-UI-Ember we're trying to keep it a lightweight library on top of Semantic-UI since I find it unnecessary to re-implement the Javascript parts that have already been developed.

@jlukic I believe we have a good workaround for this with a component wrapper on the ember-side. We're using it on one of our apps now and will integrate it into Semantic-UI-Ember soon.

Cool let me know how it goes.


Initialize shell environment upon startup (bash) [featurerequest]

I have different task that requires that some environmental variables are set and that $PATH contains certain folders.
One solution is to update my .bashrc direct with all env-variables and add all folder to my path but this clutters my environment.
A different approach (better) could be that tmuxinator supported setting env-variables as a pre-step.
Tried using the "pre:" tag but it seems like you only can run a binary from there.
What I'd like to be able to do is something similar to this:
pre: export WORKING_DIR=~/proj/tmp && export SVP_REMOTE_TOOLS=$WORKING_DIR/svp_remote_tools

meaning that these variables will be available for all windows/panes for this specific tmux-session.
This would make it possible to have a complete environment specified in a yml-file.
Perhaps there exist a way already but I haven't figured it out yet; in that case I'm sorry.

I also wouldn't mind a pretabs: directive.
With most of my projects, I have a main window with 3 panes with one running Vim at all times. I've set up a startup script which works out the unique ID of the pane, and generates a Vim server name for that project alone. I then use the other 2 panes to open files with the alias v (see below)
I do use NerdTree as well but find I'm incredibly efficient at opening files with a combination of zsh, find and grep.
function setProject() {
    if [[ $TMUX != "" ]]; then
        tmux_uid=${TMUX//[^0-9]/}
        key="VIM$tmux_uid"
    else;
        key="VIM"
    fi
    export vimServerKey="$key"
    export appBase=$PWD
    export app=$PWD/app
    alias -r vim-project-server="vim --servername $vimServerKey"
    alias -r v="tmux select-pane -t 0; vim --servername $vimServerKey --remote-silent"
    if [[ $1 = "launchVim" ]]; then
        vim --servername $vimServerKey
    fi
}

I run setProject as the standard with all 3 panes and add the launchVim as a parameter to run the vim server in the main pane.
I've had to do this because I have had problems setting up those zsh aliases and then running the alias. eg this fails:
- Editor:
    panes:
        - setProject && vim-project-server

but this suits me fine:
- Editor:
    panes:
        - setProject launchVim

Having the ability to run a environment prep function like setProject before the actual pane command is executed would be great. Especially as I want the environment ready to use with all my panes in a project.

I have a pull request looming that will address some of what you need.  I mention this in the hope you'll check back and comment on the patch ;)

cool, I'll try this during the weekend!


Service Unavailable

Hello, I was wondering why it said service unavailable (503) for the services connected to api-gateway? I tried running it locally and it was working just fine a few days ago. I didn't make any changes to the code. This is the error from the api-gateway.
Failed to load resource: the server responded with a status of 503 (Service Unavailable)
TypeError: Cannot read properties of undefined (reading 'map')
    at responseError (httpErrorHandlingInterceptor.js:11:59)
    at angular.js:18075:37
    at m.$digest (angular.js:19242:15)
    at m.$apply (angular.js:19630:13)
    at k (angular.js:13473:36)
    at v (angular.js:13730:7)
    at XMLHttpRequest.<anonymous> (angular.js:13635:9) 'Possibly unhandled rejection: {}'

This is the log from api-gateway container:
2024-11-29 11:01:46 Caused by: io.netty.channel.AbstractChannel$AnnotatedConnectException: finishConnect(..) failed: Connection refused: localhost/<IP_ADDRESS>:9411
2024-11-29 11:01:46 Caused by: java.net.ConnectException: finishConnect(..) failed: Connection refused
2024-11-29 11:27:41 2024-11-29T04:27:41.706Z  WARN 1 --- [api-gateway] [     parallel-7] [                                                 ] eactorLoadBalancerExchangeFilterFunction : LoadBalancer does not contain an instance for the service 5add73db10cd
2024-11-29 11:27:55 2024-11-29T04:27:55.321Z  WARN 1 --- [api-gateway] [     parallel-4] [                                                 ] o.s.c.l.core.RoundRobinLoadBalancer      : No servers available for service: 119056b056bf
2024-11-29 11:27:55 2024-11-29T04:27:55.322Z  WARN 1 --- [api-gateway] [     parallel-4] [                                                 ] eactorLoadBalancerExchangeFilterFunction : LoadBalancer does not contain an instance for the service 119056b056bf

I made sure that all the services are running in the Eureka and every container are running in my Docker. I used different devices and it still shows the same error. Can anyone help to resolve this problem? Any help will be greatly appreciated.

Hi @Vy03, five days ago, I've merged this PR: https://github.com/spring-petclinic/spring-petclinic-microservices-config/pull/22
To ensure that this is not a regression, could you please clone this repo, revert the last commit then reference it in your microservices?

I've done the test and I have the same issue.
api-gateway        | 2024-11-29T07:33:08.800Z  WARN 1 --- [api-gateway] [     parallel-9] [                                                 ] eactorLoadBalancerExchangeFilterFunction : LoadBalancer does not contain an instance for the service a1b204f1e878
api-gateway        | 2024-11-29T07:33:11.341Z  WARN 1 --- [api-gateway] [tor-http-nio-14] [                                                 ] o.s.c.l.core.RoundRobinLoadBalancer      : No servers available for service: customers-service

@MarianConstantinMarica I will revert the last merge. Did you try the loadbalancer with and without docker?

@Vy03 I let you test again without any changes from your side.

Hello again! I do apologize for replying late. I tried running it locally in my docker and it seems that the issue has been solved, thank you so much!


.gitignore and .hgignore do not work at all

I am either completely stupid, or this is a regression, but I cannot get this to work at all.
Whatever I put into .gitignore or .hgignore is ignored completely. I can only get it to work when I create a symlink to .ignore.
I'm on MacOS X 10.12.6 and use ag 2.2.0 as installed through HomeBrew.
/tmp/ag-ignore % ag --version
ag version 2.2.0

Features:
  +jit +lzma +zlib

/tmp/ag-ignore % git init
Initialized empty Git repository in /private/tmp/ag-ignore/.git/
/tmp/ag-ignore(master ✔) % echo "I do not want to be found" >ignore-me.txt
/tmp/ag-ignore(master ✗) % echo ignore-me.txt > .gitignore
/tmp/ag-ignore(master ✗) % git add .gitignore
/tmp/ag-ignore(master ✗) % git commit -m "ignoring ignore-me.txt"
[master (root-commit) 2c26c59] ignoring ignore-me.txt
 1 file changed, 1 insertion(+)
 create mode 100644 .gitignore

/tmp/ag-ignore(master ✔) % git status
On branch master
nothing to commit, working tree clean
/tmp/ag-ignore(master ✔) % ag found
ignore-me.txt
1:I do not want to be found

/tmp/ag-ignore(master ✔) % ln -s .gitignore .ignore
/tmp/ag-ignore(master ✗) % ag found
/tmp/ag-ignore(master ✗) %

Now this workaround is completely fine, but I still wonder what happens, as apparently this is supposed to work, no?

does it happen when you create the .gitignore and then chdir back out and then into the current directory? I've has the issue before on freshly created gitignore files.

That was just an example to reproduce it. This happens all the time on my main repo (which uses Mercurial, not Git).
The only way to fix it was the symbolic link (or rather, I copied the file, as we have Windows users in the team).

So, can you reproduce this on your side, or does it just happen for me?

Not working for me either. ag version 2.2.0 on intel mac, OS X 12.6


Update sketchpack.json

Extra comma was causing JSON.parse to fail.

Thanks!

@cc adamkirkwood :)


Documentation

This epic contains issues related to the docs for the project.

The app will be completely rewritten as a web app, thus all opened issues are being closed.


Utxo perf test

This PR adds utxo perf test that can be run from the command line using mix run:
A single iteration of this test consists of the following steps:
1.1 Two accounts are created - the sender and the receiver
2.1 The sender account is funded with initial_amount
2.2 The balance on the childchain of the sender is validated using WatcherInfoAPI.Api.Account.account_get_balance API.
2.3 Utxos of the sender are validated using WatcherInfoAPI.Api.Account.account_get_utxos API
3.1 The sender sends al his value to the receiver
3.2 The balance on the childchain of the sender is validated
3.3 The balance on the childchain of the receiver is validated
3.4 Utxos of the sender are validated
3.5 Utxos of the receiver are validated
You can run it using:
mix run -e "LoadTest.TestRunner.run()" -- "utxos" 2 5

Parameters:

test name - utxos
tps - 2 t/s
period - 5 seconds

So the command above means "Run utxos test for 5 seconds with 2 tests / second". Keep in mind that using 100 t/s causes elixir-omg to start returning internal server error.
You can modify the following test paramers:
- token -address of the token. default value is "0x0000000000000000000000000000000000000000",
- initial_amount - this value used when funding the receiver. default value is 760
- fee - fee used when transferring value from the sender to the receiver
You can also modify values for the by providing TEST_CONFIG_PATH env variable, it should contain
the path to json file. For example:
    TEST_CONFIG_PATH=./my_file mix run -e "LoadTest.TestRunner.run()" -- "utxos" 1 5

Perhaps a bit more effort on the PR description and the test description. utxos is not a very good descriptive name of a test.
the parameters 1 and 1, also don't ring a bell what they mean, could we paramatrice these so that we know what we're setting?
--num_senders
--num_receivers
do the senders and receivers need to be the same? what happens if they're not? if they need to be the same, why are there two separate paramters?

@InoMurko I extended PR description

Can you link this PR to the issue?? so it can be easier tracked. @ayrat555
I am also curious on what your approach is. Are you removing existing stuffs like LoadTest.Runner.UtxosLoad afterward??

how do you get this?? do you run with docker or existing env? Cause I am quite sure we run TPS 100 several times on our sandbox

WatcherInfoAPI.Api.Account.account_get_utxos(client, body) starts returning server:internal_server_error

Also, I see errors in watch_info logs
Server: localhost:7534 (http)
watcher_info    | Request: POST /account.get_utxos
watcher_info    | ** (exit) an exception was raised:
watcher_info    |     ** (DBConnection.ConnectionError) connection not available and request was dropped from queue after 171ms. This means requests are coming in and your connection pool cannot serve them fast enough. You can address this by:
watcher_info    |
watcher_info    |   1. By tracking down slow queries and making sure they are running fast enough
watcher_info    |   2. Increasing the pool_size (albeit it increases resource consumption)
watcher_info    |   3. Allow requests to wait longer by increasing :queue_target and :queue_interval
watcher_info    |
watcher_info    | See DBConnection.start_link/2 for more information

Are you removing existing stuff like LoadTest.Runner.UtxosLoad afterwards??

I didn't plan to do it yet. as @kevsul mentioned, existing tests have a different purpose?


Display error properly in the UI when submission is submitted with invalid project

Steps to Replicate

Create an invalid project
Upload integration test spreadsheet - do not update the project
Upload the data files
Wait for files to be validated
Wait for the submission's graph to be valid
Submit to HCA

Actual Behavior

Expected Behavior
Display proper error why submission couldn't be submitted
Related to changes from ebi-ait/dcp-ingest-central#549

PR up for review: https://github.com/ebi-ait/ingest-ui/pull/155

@jacobwindsor to help with testing.

Jacob to check if in prod.


boards: st: Add support for the Nucleo-H503RB board

This PR adds support for nucleo_h503rb board which is based on STM32H503RBT6 SoC.
STM32CubeProgrammer and pyocd are supported as runner.
The following samples were tested:

samples/hello_world
samples/basic/blinky
samples/basic/button
samples/subsys/task_wdt

Thanks for enabling this nucleo stm32h503 board

LGTM
I suggest to consider the #75308 for openOCD runner (depending on the which is merged first )

I have also adopted the changes from #75308 for this board.


Ranged attacks need a range value or throw a rollError

Currently the rangeValue is empty, as the range is not included in the statblock but derived from the weapon type (in most cases).
Change this to a default rangeValue of maybe "5" and include the range in sbcContent.attackDamageTypes.

Added a default value and started adding rangeIncrements to sbcContent.attackDamageTypes

Added rangeIncrements for all weapons in the srd.


Add License

fixed in https://github.com/sloganking/Structured-commenting-formatter/commit/d18470348d5879eb1fb64dafdf029a4d0a140bb2


items lost on death due to invalid flag & target?

As of spongeforge-1.12.2-2705-7.1.0-BETA-3409.jar + griefprevention-1.12.2-<IP_ADDRESS>2.jar, when a player die in a claim with item-spawn = false, keepInventory = false, all his inventory gets deleted with no item dropped.
This also happens if said player is the owner of the claim.
According to gpdebug:

when a player dies, we have Flag = item-spawn, Target = minecraft:custom
when a player drops, we have Flag = item-drop, Target = minecraft:player

I think we've 2 issues here, not sure if they're in GriefPrevention or SpongeCommons:
1- when dying, Flag should be item-drop with Target being the player
2- when item-spawn is cancelled, items should be returned to player's inventory, this might be a consequence of the invalid flag?

Issue is reproduced with spongeforge-1.12.2-2705-7.1.0-BETA-3355.jar.
Issue is reproduced with griefprevention-1.12.2-<IP_ADDRESS>6.jar.
I didn't try other versions.

In latest GP, item-spawn no longer affects living entities so this should no longer be an issue.


Copies Go files that the running Go version can't parse.

If a vendored module has //go:build go1.18 lines while running go1.17 then running gofmt -l . (as a CI formatting check, for example) errors out because go1.17 doesn't understand go1.18 generics syntax.     Can work around this with a more complex gofmt line that avoids vendor directory, like find . \( -path ./vendor -prune \) -o -type f -name \*.go -print0 | xargs -0 gofmt -l, so not sure exactly how urgent this is, but a parameter to avoid them would help.
Example Go file:
package main

import (
	"fmt"

	"golang.org/x/tools/godoc/util"
)

func main() {
	fmt.Println(util.IsText([]byte("foo")))
}

go mod vendor:
$ rm -rf vendor; go mod vendor; gofmt -l .; echo $?
0

vend:
$ rm -rf vendor; go run --mod=mod github.com/nomad-software/vend > /dev/null; gofmt -l . 2>&1 | grep : | tail
go: finding module for package github.com/nomad-software/vend
vendor/golang.org/x/tools/internal/lsp/testdata/keywords/empty_select.go:7:1: expected declaration, found '}'
vendor/golang.org/x/tools/internal/lsp/testdata/keywords/empty_switch.go:9:3: expected '}', found d
vendor/golang.org/x/tools/internal/lsp/testdata/keywords/empty_switch.go:11:1: expected declaration, found '}'
vendor/golang.org/x/tools/internal/lsp/testdata/keywords/keywords.go:71:1: expected operand, found '}'
vendor/golang.org/x/tools/internal/lsp/testdata/keywords/keywords.go:100:52: expected ';', found 'EOF'
vendor/golang.org/x/tools/internal/lsp/testdata/snippets/literal.go:21:2: expected operand, found '}'
vendor/golang.org/x/tools/internal/lsp/testdata/snippets/literal.go:22:2: missing ',' in composite literal
vendor/golang.org/x/tools/internal/lsp/testdata/statements/if_err_check_return.go:22:2: expected boolean expression, found assignment (missing parentheses around composite literal?)
vendor/golang.org/x/tools/internal/lsp/testdata/statements/if_err_check_return.go:26:2: expected operand, found 'if'
vendor/golang.org/x/tools/internal/lsp/testdata/statements/if_err_check_return.go:27:3: expected '}', found 'EOF'

It's a well known problem and not one, i think, vend should really fix.

https://groups.google.com/g/golang-nuts/c/gyBNyJkHOIE
https://github.com/golang/go/issues/22173
https://github.com/golang/go/issues/33962

The solution is not to format the vendor folder. A shorter command from the last link:
gofmt -l (find . -type f -name '*.go' | grep -v "/vendor/")


Toolbar: do not add separator to empty drop-down.

address issue reported by @himdel here: https://github.com/ManageIQ/manageiq/issues/5116
@himdel: please, review

@himdel : changed the PR, so that the separator is there but hidden
to be consistent with other places and to be future-proof

LGTM :+1: , may need to address trailing separators as well, but we'll see :).


test: support test runner in v16

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


PR: New warning message in IPython console when both pylab and sympy are enabled

Fixes #1665

@mariacamilaremolinagutierrez, please add a blank line between
Populating the interactive namespace from numpy and matplotlib

and
These commands were executed

like this
Populating the interactive namespace from numpy and matplotlib

These commands were executed
....

Thanks @mariacamilaremolinagutierrez, looking good now ;-)
(3 PRs merged, go DigitalOcean!! I already ordered my T-Shirt :-p)


Consider having a "Latest news" section

Content Feature Request
Summary
Most programming language websites (Go, Python, Ruby, etc.) have a section showing recent news: new releases, new blog posts, etc. As it stand the landing page is pretty static: you see it once, you don't need to see it anymore.
Motivation
You want your landing page to represent the language as best as possible, which also means giving the reader and idea of what's going on in the ecosystem.
Drawbacks
No idea.
Rationale and alternatives

Even programmers familiar with Rust may find it useful, not just novices. As it stand, an experience programmer has no reason to visit the landing page

Maintenance
This can be automated, using RSS feeds, or similar.

Hi! We don't currently have this content, so I don't think we would consider this at this time. If you are interested in this, there is a subteam of the Community Team that handles content and is a great place to talk about this idea!


Added extra options hash to login function that get passed along with scope to fb.login

The main use case (for me) is to be able to pass the default_audience param to the login method so that an app can request to be able to post publicly rather than just to 'Friends' which is the default.

I'm happy to add tests but would appreciate some direction as not sure of the best way of testing this functionality.

Thank you @bradcrawford

@bradcrawford would you mind to add a brief paragraph to the README to illustrate this new feature?

Sure thing


Touches not registered after reconfiguring animations

I'm trying to build a component that takes its own height into account when performing animations. After I configure the animated values in the constructor, touch handling works as expected, but if at any point I reconfigure them with new values (for example, after receiving an onLayout() callback), touch input immediately stops being detected.
It's worth noting that the animation updates correctly according to the condition State.ACTIVE == false when the configuration code is re-run - it's only touches that fail to be detected.
See the gist below for a minimal example. Is there an intended way to update the native animation constraints over the bridge? Every example I've been able to find shows statically sized components, but I would imagine this library can handle state updates like this. Thanks!
https://gist.github.com/naftalibeder/a52ffd26f578587dce0c58387912c9c7

Instead of reconfiguring everything, simply running setValue() on a node correctly sends that new value over the bridge.


I would like to know which files' code you modified?

This is a very meaningful work. I would like to know which files' code you modified? Can you tell me?

Thanks a lot. The core additions are mainly from https://github.com/Gumpest/SparseVLMs/blob/main/llava/model/language_model/modelling_sparse_llama.py#L201

The above four files are our main contributions.

Thanks!


Issue with restart

continue to write the followin in the Script Log:
SCRIPT       : [SchanaParty] SendInfo
SCRIPT       : [SchanaParty] Positions {}
SCRIPT       : [SchanaParty] Healths {}
SCRIPT       : [SchanaParty] Parties {}
SCRIPT       : [SchanaParty] SendPlayers {"param1":[],"param2":[]}
SCRIPT       : [SchanaParty] SendInfo
SCRIPT       : [SchanaParty] Positions {}
SCRIPT       : [SchanaParty] Healths {}
SCRIPT       : [SchanaParty] Parties {}
SCRIPT       : [SchanaParty] SendPlayers {"param1":[],"param2":[]}
SCRIPT       : [SchanaParty] SendInfo
SCRIPT       : [SchanaParty] Positions {}
SCRIPT       : [SchanaParty] Healths {}
SCRIPT       : [SchanaParty] Parties {}
SCRIPT       : [SchanaParty] SendPlayers {"param1":[],"param2":[]}
SCRIPT       : [SchanaParty] SendInfo
SCRIPT       : [SchanaParty] Positions {}
SCRIPT       : [SchanaParty] Healths {}
SCRIPT       : [SchanaParty] Parties {}
SCRIPT       : [SchanaParty] SendPlayers {"param1":[],"param2":[]}
SCRIPT       : [SchanaParty] SendInfo
SCRIPT       : [SchanaParty] Positions {}
SCRIPT       : [SchanaParty] Healths {}
SCRIPT       : [SchanaParty] Parties {}
SCRIPT       : [SchanaParty] SendPlayers {"param1":[],"param2":[]}
SCRIPT       : [SchanaParty] SendInfo
SCRIPT       : [SchanaParty] Positions {}
SCRIPT       : [SchanaParty] Healths {}
SCRIPT       : [SchanaParty] Parties {}
SCRIPT       : [SchanaParty] SendPlayers {"param1":[],"param2":[]}
SCRIPT       : [SchanaParty] SendInfo
SCRIPT       : [SchanaParty] Positions {}
SCRIPT       : [SchanaParty] Healths {}
SCRIPT       : [SchanaParty] Parties {}
SCRIPT       : [SchanaParty] SendPlayers {"param1":[],"param2":[]}

This is probably due to the Mass update that went awry.


Global frequency

Hi,
I'm trying to change the global frequency to "daily" with no luck.
I tried, following the readme, to do:
%w[ weekly monthly yearly ].each do |freq|
  node.override['logrotate']['global'][freq] = false
end
node.override['logrotate']['global']['daily'] = true

But /etc/logrotate.conf seems un-touched
I also tried to specify some logrotate_app and they work perfectly, thank you for your work!
Target is CentOs7 (if it helps)

@simo2409 Thanks for filing this issue.  I'll try to take a look at it by the end of this week.  Some quick thoughts: Where did you place that example code? Are you sure it is being processed before the logrotate::global recipe in the compile phase of the chef-client run?

I'm using chef to provision a CentOS 7 server on my local machine (running in vagrant) and starting from base (minimal) centos 7 image.
I tried two ways  (both unsuccessful):
1 - I specified in host's json (nodes/<IP_ADDRESS>.json):
"logrotate": {
    "global": {
      "weekly": "false",
      "daily": "true"
    }
  },

It didn't work so
2 - I tried to put node.override into the cookbook I created called "logrotate" in which (in recipes/default.rb) I have:
%w[ weekly monthly yearly ].each do |freq|
  node.override['logrotate']['global'][freq] = false
end
node.override['logrotate']['global']['daily'] = true

# Some logrotate_app which work properly

I don't know if it's a bug or simply I didn't understand how it should work.
Thank you very much

"logrotate": {
    "global": {
      "weekly": "false",
      "daily": "true"
    }
  },

What if you try the barewords true and false rather than the strings "true" and "false"?

2 - I tried to put node.override into the cookbook I created called "logrotate" in which (in recipes/default.rb) I have:

Does that recipe call include_recipe "logrotate::global" or is logrotate::global in the node's run list?

That was the problem! I didn't call logrotate::global nor I called include_recipe "logrotate::global", now it works properly with the data in node's json (with  true/false, not "true"/"false").
Thank you so much!
If it's useful I can send you a PR to include a note in readme about "logrotate::global", it would be helped me in this case


Very wrong type provider usage syntax highlighting

it works ok on single line TP:

Yikes.  Good find.

Looks like this is a regression based on @vladima's PR.  On vanilla RC2 it works fine:

I'd be surprised given than my changes update only quick info, signature help and completion. This looks like an issue with lexical - the state from the previous line is not propagated to the next line

Hmmm.  @vasily-kirichenko could this be from when you did some refactoring and fixes to lexing logic?

Maybe. When I debugged the lexer cache, the "lex sate at start of line" and "lex state at end of line" (int) were always 32769. I have no idea what they are and how they should work though :(

Fixed in https://github.com/Microsoft/visualfsharp/pull/2158

Nice.


Increase minimum datatokens for pools

When creating a pool we currently require a minimum of 50 datatokens to be created. I suggest to setting a new limit to 100 or 150. Thoughts?

Not really, you just lower the price, the value of OCEAN is not relevant


Wrap main graph properties into a struct

also fix a bug, where we did not add the first dependency into the dep list. It actually did not matter if the dep was not a leaf but I think this way it's more correct :)

please don't squash when merging :)

Merging this. Thanks! :D


Actually working installation commands

Sorry for the noise. It turns out providing the url is only sufficient in the pkg> REPL mode. If using using Pkg one has to specify a PackageSpec

Ah, yes, I should have caught that the first time. Thanks.


Fixed mouse cursor on card hover

Same as title.

Thanks, sorry if I broke something

Nothing was wrong, everything was working fine : ]
Just the cursor was phasing out when you hover over a card.
Was about to push a 100 line change after fixing for 3 hours, before I realized it can be fixed with that one line lol.


Fixes issues #65, #84

Current coverage is 89.66% (diff: 25.00%)

Merging #124 into master will decrease coverage by 0.44%

@@             master       #124   diff @@
==========================================
  Files             7          7
  Lines           586        590     +4
  Methods          45         45
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            528        529     +1
- Misses           58         61     +3
  Partials          0          0

Powered by Codecov. Last update eba1043...192b51c

thank you 👍


Update enable-controlled-folders.md

made changes to the name in step 5 6 and 7
Fixes https://github.com/MicrosoftDocs/microsoft-365-docs/issues/12011

@JohanFreelancer9 Please copy review


The system can support keyword, industry, occupation queries for standars

This is coming up more now. Lets prioritize. If I search "Mechatronics" i should get the two different mechatronics occupations.
For industry, we might want to use the attached. Per Kinsey:

I talked with our data team yesterday and they suggested we use SOC codes as opposed to WIN’s occupation clusters (where there’s some overlap). The 2-digit codes are attached here (they align with first two digits of O*Net codes, I believe) – I think we could even condense this for our purposes (e.g. grouping the two healthcare related SOC categories into one broader healthcare category that would show up on RapidSkills).

2018 2-Digit SOC.xlsx
This could be where it makes sense to start using ElasticSearch? I dont want to prescribe that, though, if there's a better option

@Pcushing should I import these codes and create industries, and then link occupations to the industries when I can, based on the first two digits of the onet code?

Also, what field(s) would "keyword" refer to when searching?

should I import these codes and create industries, and then link occupations to the industries when I can, based on the first two digits of the onet code?

Ah.  Yes. Probably a good idea to get the industry data in here now that. we've got that mapping.

Also, what field(s) would "keyword" refer to when searching?

I think we'll need to play around with this a bit. When I say "keyword" I should've been more speciic that I meant elasticsearch query. My initial thought is we should just push everything into that and use ES to hit multiple fields at once with different weights given as needed: fields: ['name^5', 'aliases^5', 'slug', 'url'] for example from WorkHands organization search. In our case it might be fields: ['name^5', 'occupation_name^5', 'code^3', 'industry'], etc.

See about making standards searchable instead of occupations.