Append country code for search phone number in workspace invite and room invite

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/33301
PROPOSAL: https://github.com/Expensify/App/issues/33301#issuecomment-1863089719
Tests

Go to room (workspace visibility).
Click on room header > Members.
Click Invite.
Invite a contact number without country code.
Re-invite the same contact number again with country code.
The error message for contact number that is an existing member is "{login} is already a member of {name}".
Repeat steps with workspace invite

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Go to room (workspace visibility).
Click on room header > Members.
Click Invite.
Invite a contact number without country code.
Re-invite the same contact number again with country code.
The error message for contact number that is an existing member is "{login} is already a member of {name}".
Repeat steps with workspace invite

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/129500732/b1ca6483-996c-4125-b3e6-3c7e40bf9ab9

Android: mWeb Chrome

https://github.com/Expensify/App/assets/129500732/26e44ac4-dd0b-4542-8b0a-77ee59b9ba3f

iOS: Native

https://github.com/Expensify/App/assets/129500732/91d9382d-e295-4896-a2a7-b676858b54e6

iOS: mWeb Safari

https://github.com/Expensify/App/assets/129500732/ffa4429a-1a47-4571-a314-fcf6d808eb60

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/129500732/cf29e0cd-be23-4f8a-bc33-9e37818342ba

MacOS: Desktop

https://github.com/Expensify/App/assets/129500732/1a3045fa-aa3d-437e-9c30-8235553446b8

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/thienlnam in version: 1.4.31-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Fix IPublishedContentInformationExtractor to work when Property Value Converters are enabled

Fixes #63
This fixes the PublishedContentProductInformationExtractor  so that it can handle getting an IEnumerable for the master relation, as is default on Umbraco 7.6+. It will fall back to the old string method if the value received is null (which might be due to property value converters disabled or on previous Umbraco Versions.
I have tested with EnablePropertyValueConverters both true and false, but only on Umbraco 7.13.2.

@arknu Looks good, could you update your repo though and re-target the PR to the dev branch?

@mattbrailsford Should be retargeted now. Although, as you can see, I didn't exactly do a good job first time around...
I don't know why the license and readme still show as changed. I certainly have not made any changes to those files.

@arknu yea, not entirely sure, but they look to be corrrect so it's fine. All merged in now. Thank  you.


[cc2538] interrupt driven radio driver

So, I have a confession to make… in PR#3609 I accidentally included some code that would have enabled the receive interrupt on the CC2538 in the NVIC.
Given the receive interrupt handler just blindly clears the interrupt flags, this is probably not a good plan.  So, a better option, implement the rest of it, with a compile-time switch to turn interrupt driven reception off if desired.  This allows for up to two frames to be stored until such time as they can be processed by OpenThread (one frame in the radio's buffer, and one in sReceivedFrame).
I'm doing some more thorough tests with the GRL test suite to ensure it hasn't broken anything but indications are it is working as expected (I'd be expecting failures left-right and centre if receive was broken by this).

@sjlongland, thank you for this contribution and the additional testing! Let us know when you've completed your testing and think this PR can move forward.

Two mistakes found:

I got the wrong register for the interrupt vector in the NVIC… so the interrupt service routine wasn't firing (confusingly, TI have two interrupts mapped for RX/TX, and I'm not sure which one is being used)
In the interrupt service routine, I'm masking the constant RFCORE_SFR_RFIRQF0 with another constant RFCORE_SFR_RFIRQF0_RXPKTDONE which basically means it's a no-op anyway.  RFCORE_SFR_RFIRQF0 needs a HWREG macro wrapped around it.

I'll fix both of these problems. :-)

Gah… as Maxwell Smart would put it… missed it by → ← that much.  make pretty correction pushed.

If I surround parts of the radio driver, and comment out the call to readFrame from otPlatRadioProcess so that the only received traffic is via the interrupt, I get the above traces on a logic analyser.  In the above case:

PB7 was turned on just before line 829 and off after line 844.
 Is this PR ready to move forward?

I think it should be safe… receiving frames seems to be working just fine with it turned on and it seems to have a big (positive) impact on some test cases in the GRL test harness.  If we find issues, we can always turn the feature off in the OpenThread config.


Not generating code on Linux

It seems like target-name != cpp is failing even though I am building in CPP mode... any ideas if this is a linux limitation of haxe or what?

The same holds true for Mac.
It appears that target.name is an empty string exclusively within #if macro blocks. Outside these blocks, it correctly equals 'cpp'.


fix rich_text_box

if column not null then textarea required but hint is hidden

I need you to add more context when you open PRs.  Referencing an existing issue would suffice as well.  Your comment doesn't seem to make sense with the change that was made.  Please describe the issue and how your fixing it.

I am unsure why removing the required part when it in fact is required?


Geojson-vt rendering doesn't work for other geometries than Polygon

Example with a line

Are you sure this has to do with the geometry type? From a quick glance at the code pen, it looks like the generated features are valid and have the correct tile pixel coordinates. Maybe something is wrong with the tileGrid or tilePixelRatio definition for the source?

the issue is here:
https://github.com/openlayers/openlayers/blob/master/examples/geojson-vt.js#L31
the geometry is always (linestring or polygon) something like [[x1, y1], [x2, y2], ...] so the current code works well for *Polygon but not for the other types.

The example could be fixed easily though. @oterral, are you up for creating a pull request?


Style option "s" does not behave the same as "size" with hv.dim

ALL software version info
(this library, plus any other relevant software, e.g. bokeh, python, notebook, OS, browser, etc should be added within the dropdown below.)

Software Version Info
python      3.11.10
bokeh       3.5.2
holoviews   1.19.1
hvplot      0.11.0
matplotlib  3.9.2
plotly      5.24.1

Description of expected behavior and the observed behavior
According to the docs, the styling option s is documented as:
s : int, optional, also available as 'size'
    The size of the marker

But this is not true, as s and size are treated differently, at least for scatter and points plots. This becomes clear when trying to use a dim transform for the size. Granted, it does say int for the type, but given that this seems to work partially for some configurations, maybe this should be clarified/updated.
Complete, minimal, self-contained example code that reproduces the issue
import holoviews as hv
import hvplot
import hvplot.pandas
import numpy as np
import pandas as pd

x = np.linspace(0.0, 10.0, 25)
y = np.sin(x)
df = pd.DataFrame({'x': x, 'y': y})

size_opts = dict(kind='scatter', size=hv.dim('y') * 50.0)
s_opts = dict(kind='scatter', s=hv.dim('y') * 50.0)

# run each of these in a separate cell in a notebook to see the errors and differences

# bokeh: size works, s does not work
# also, when s does not work, other hv.dim cannot be used and an warning is emitted (not thoroughly tested)
hvplot.extension("bokeh")
(df.hvplot('x', 'y', **size_opts) + df.hvplot('x', 'y', **s_opts)).cols(1)

# matplotlib: size does not work, s works
hvplot.extension("matplotlib")
(df.hvplot('x', 'y', **size_opts) + df.hvplot('x', 'y', **s_opts)).cols(1)

# plotly: size does not work at all and errors, s does not work
hvplot.extension("plotly")
df.hvplot('x', 'y', **size_opts)
df.hvplot('x', 'y', **s_opts)

Stack traceback and/or browser JavaScript console output
Screenshots or screencasts of the bug in action

[ ] I may be interested in making a pull request to address this

Possibly related to 674


perf: featured drops

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #9369
[x] Closes #9349

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

I removed some of the calls to our graphql queries. As long the data from our db is enough, it should be fine, I guess?

Thanks!
pay 30 usd

😍 Perfect, I’ve sent the payout
💵 $30 @ 7.66 USD/DOT ~ 3.916 $DOT
🧗 1xjvRADwdJcnmUCLWerEHR4iGCT5EBTm4D4fzLLg4LcAC9p
🔗 0x92d3a6e3a5ee03c002be039b2d06e9575843db8b90828f87c4e272d5cc0dd22a
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues

lmk if we want it more

I would easily add 1hr should be okay for most of cases, we are adding drop at least day before


Tooltip, double message

Getting double tooltip message when hovering over the element. (I used code example)

Is anyone else getting this or just me?
I'm using rebass: 0.2.6

Can you post the code for this or a link to a jsbin?

It's just the code example. My colleague  pointed out, that it's because of title tag. https://github.com/jxnblk/rebass/blob/5f848c9e4afdfb21f3d528fddde137220b220905/src/Tooltip.js#L55
Not sure if that behaviour was intended, but maybe you can make it optional through props?

Yep I noticed this as well — you get the browser's default tooltip (which doesn't look so similar on a Mac) through the title attribute, and then the one you asked for with Rebass. A good workaround to keep the tooltip accessible is to use the aria-label attribute. GitHub uses this in Primer: http://primercss.io/tooltips/ (source code)


Add  to the list of merchants

What's the name of your business?
Z1 Fashion- Imagoza
What's the URL of the website?
https://www.etsy.com/shop/ImagozaZ1fashion
Write a brief description of your business/service (max 50 characters)
Transformational clothing for a unique individual.
Enter the URL where a Monero mention can be found
https://www.etsy.com/shop/ImagozaZ1fashion
How do you accept Monero? (choose only one option)

[ ] Monero integrations (github.com/monero-integrations)
[ ] Globee
[x] Native wallet
[ ] Other payment gateaway (Coinpayments, Morphtoken, etc. Please specify)

How would you categorize your business? (choose only one option)

[ ] Exchange
[ ] Block Explorer
[ ] Payment Gateway
[ ] Web Hosting
[ ] Tool
[ ] Service
[x] Goods
[ ] Entertainment

Comment

What's the URL of the website?

We specifically ask for no referral links.

Write a brief description of your business/service (max 50 characters)

That description is not max 50 characters

@erciccione thank you for taking a look! I didn't see that it was a reference link, didn't know I could remove that part!
Fixed;
Removed ref part of link
&
Shortened description to less than 50 characters.

#1588


Integrate conversion tool into kubectl

https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/cluster_management.md#switching-your-config-files-to-a-new-api-version
See also #7576, #4471

Following the discussion in #7576, I'm planning to add a verb convert to kubectl, with i,o,r,v flags. Is this the right way to fix this issue?

Related doc has been moved to https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/admin/cluster-management.md#switching-your-config-files-to-a-new-api-version


chore: release 3.0.0

:robot: I have created a release *beep* *boop*
3.0.0 (2019-07-24)
⚠ BREAKING CHANGES

pulls in breaking API changes in google-auth-library. getProjectId() and getProjectId() have been modified to make the impact of these changes less noticeable on the legacy googleapis module (getClient() is idempotent, but getProjectId() will use the last configuration).

Features

expose GoogleAuth constructor on AuthPlus class (#154) (7d7a961)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-googleapis-common/releases/tag/v3.0.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


mongodb 3.4 deprecation warning

on 0.12.2, seeing this in the server logs:
2017-06-27T10:41:06.814+0200 W COMMAND  [conn399012] Use of the aggregate command without the 'cursor' option is deprecated. See http://dochub.mongodb.org/core/aggregate-without-cursor-deprecation.

Right, this happens w/ such MongoDB version when using .aggregate which send an Aggregate w/ an empty Cursor option.
There are other aggregation operators which use cursor, such as .aggregateWith or the most recent .aggregatorContext (all are test covered if looking for more examples).
Doc PR welcome ;-)


Minor Update item.hbs - mismatched start end tag

Thanks for the fix!

Thanks for the fix!

No worries @newhouse Thanks for the new releases, great work!


Fix typo

I've rebased the change onto the 1.1.x branch so this can be applied to that version of the docs and merged forward to main.


added hostaliases to kiali-server

Signed-off-by: Ethern Su<EMAIL_ADDRESS>

Just looking at this briefly, it seems like it is OK (I need to research that hostAliases setting - that is a new one to me). However, please be advised Kiali users are more than just helm server users. You may need to update the helm chart for the operator and most definitely you need to create a PR to have this same capability for those that deploy Kiali via the operator and not the server helm chart.
Please refer to these developer checklists to know what things need to be done:

https://github.com/kiali/kiali-operator/blob/master/DEVELOPING.adoc#are-you-altering-a-kiali-server-resource
https://github.com/kiali/kiali-operator/blob/master/DEVELOPING.adoc#are-you-altering-a-kiali-server-configuration-setting

I can do a PR for operator too, but it will be in the operator repository and not this one. Do you think I should work on that first before continuing on this one?

I can do a PR for operator too, but it will be in the operator repository and not this one. Do you think I should work on that first before continuing on this one?

Yes, correct - there will be a PR in the operator repo as well as this one. Thanks.

companion operator PR: https://github.com/kiali/kiali-operator/pull/432

Here are the test procedures to confirm this works (I ran these and all works):

Pull this PR and build the helm-charts via make:

make clean build-helm-charts

Tell helm to output the template yaml, setting the new host_aliases fields:

helm template -n istio-system kiali-server _output/charts/kiali-server-*-SNAPSHOT.tgz --set deployment.host_aliases[0].ip=<IP_ADDRESS> --set deployment.host_aliases[0].hostnames="{foo.local,bar.local}"

Confirm the hostAliases field is correctly set in the Deployment. Based on the command in step 2, you should see this in the Deployment yaml:

spec:
  template:
    spec:
      hostAliases:
      - hostnames:
        - foo.local
        - bar.local
        ip: <IP_ADDRESS>

If you want to test further, you can actually deploy this in a cluster that has Istio installed:

helm install -n istio-system kiali-server _output/charts/kiali-server-*-SNAPSHOT.tgz --set deployment.host_aliases[0].ip=<IP_ADDRESS> --set deployment.host_aliases[0].hostnames="{foo.local,bar.local}"

Once the Kiali pod is running, output its /etc/hosts file:

kubectl exec -n istio-system $(kubectl get pod -n istio-system -l app=kiali -o name) -- cat /etc/hosts

Now check that /etc/hosts file and confirm it has the custom alias installed. You should see something like this:

# Kubernetes-managed hosts file.
<IP_ADDRESS>	localhost
::1	localhost ip6-localhost ip6-loopback
[...and some other settings...]

# Entries added by HostAliases.
<IP_ADDRESS>	foo.local	bar.local

assigning to @mattmahoneyrh in case QE wants to verify. Test procedures found in previous comment.


Redirects for autocomplete

Right now, the autocomplete list is populated based on a weighted edit distance between what's been typed already and the symbol being considered.  This works well when a student knows part or all of the function name they are looking for, but it misses cases where a student wants to draw a square, or a triangle.  There is no square function, but squares can be drawn using rectangle.  There is no triangle, but triangles can be drawn using polygon.
I propose to add this knowledge in the form of redirects for autocomplete, so that if a student triggers autocomplete with something close to square, we should suggest rectangle as a completion.  Similarly, triangle should redirect to polygon.  Maybe ellipse and oval should redirect to scaled(circle?  Though it's weird to add unbalanced parentheses.

Should these suggestions go through the sieving process or I can just stick them into the final list?

I'm not sure what you mean by the sieving process.  Referring to the code at https://github.com/google/codeworld/blob/e6fa062a3c1c9f0895ab65fe32eefba03d1ddf1e/web/js/codeworld_shared.js#L487, I think they should go through the same scoring and sorting process as the existing suggestions, except that they should only be added to tier 3 (the fuzzy matching tier) so that they are never chosen automatically.  The scoring should be done on the trigger word (e.g., square or triangle, and not on the word that will actually be inserted.
It's very likely that the scores should be demoted somehow, perhaps even by a different amount for each one (so, for instance, square should maybe redirect to both rectangle and polygon, but polygon should be demoted further), but that can be added after the initial implementation.
It would also be nice to add something to the hint doc to explain why you're getting this suggestion.  But again, don't include that in the initial implementation.  Let's get it working first, and then make the UI changes afterward.

Please share your thoughts on the initial implementation.

It would also be nice to add something to the hint doc to explain why you're getting this suggestion.

Here's one way to go about it:

If you prefer it done differently, I can come up with few other ideas.
What should this note actually say? I was thinking that it might be something along the lines of:
"If you are looking for {search_term}, it doesn't exist. Try this symbol instead."
Also, do you think it's a good idea to highlight these replacements somehow within the list?

I left comments on the pull request about this.  I didn't think about this until now, but we're already adding all possible replacements into the list.  I actually now don't think we want to add any new entries to autocomplete at all.  Surprisingly (at least to me), this is just going to be a scoring change!
Still, we can add the rest of these features.  The new score will be the best of the organic score for each candidate, and the scores that go via replacements.  So we can still observe whether the best score was obtained via a replacement, and do any highlighting or indentation.  My instinct is to say not to highlight them, but to add an annotation to the doc dialog.
As far as the text, the gold standard would be to include that in the replacement table, so that when we add redirects into the table, the text can be customized.  For triangle, it might say "A triangle is a polygon with three vertices."  Sometimes the redirect might just be for a common misspelling or alternate spelling, so the explanation text should be optional, and nothing changed if it's missing.

Surprisingly (at least to me), this is just going to be a scoring change

That's true! I didn't realize this either, until now. This brings up a question, though - what to do with direct hits?

the replacement table

What is this, exactly?

direct hits

Technically, we should only be scoring entries in found[2] using this logic.  But I don't think that it matters, honestly, because in practice an item in tiers 1 or 2 should always score higher relative to its true completion than a redirect, especially after the penalty.  So I don't think a change is needed there.

the replacement table

I was referring to this dictionary: https://github.com/google/codeworld/pull/1400/files#diff-908d29aadd4abb809c5cd7ecafc602c8R540

Here's a more complete set of redirects:

positive -> abs
magnitude -> abs
arccos -> acos
interactionOf -> activityOf (from a deprecation)
semicircle -> arc
arcsin -> asin
arctan -> atan
blend -> mixed
colors -> assortedColors
loop -> closedCurve
cosine -> cos
text -> lettering
MouseMovement -> PointerMovement (from a deprecation)
MousePress -> PointerPress (from a deprecation)
MouseClick -> PointerPress
PointerClick -> PointerPress
MouseRelease -> PointerRelease (from a deprecation)
triangle -> polygon
quadrilateral -> polygon
parallelogram -> polygon
trapezoid -> polygon
pentagon -> polygon
hexagon -> polygon
septagon -> polygon
octagon -> polygon
line -> polyline
main -> program (from a deprecation)
square -> rectangle
turn -> rotated
stretch -> scaled
reflect -> scaled
solidArc -> sector
sine -> sin
solidCurve -> solidClosedCurve
solidLoop -> solidClosedCurve
solidTriangle -> solidPolygon
solidQuadrilateral -> solidPolygon
solidParallelogram -> solidPolygon
solidTrapezoid -> solidPolygon
solidPentagon -> solidPolygon
solidHexagon -> solidPolygon
solidSeptagon -> solidPolygon
solidOctagon -> solidPolygon
solidSquare -> solidRectangle
root -> sqrt
radical -> sqrt
tangent -> tan
thickLoop -> thickClosedCurve
thickTriangle -> thickPolygon
thickQuadrilateral -> thickPolygon
thickParallelogram -> thickPolygon
thickTrapezoid -> thickPolygon
thickPentagon -> thickPolygon
thickHexagon -> thickPolygon
thickSeptagon -> thickPolygon
thickOctagon -> thickPolygon
thickLine -> thickPolyline
thickSquare -> thickRectangle
move -> translated

I could see this growing much further.  Perhaps it should go in a JSON file, and be loaded at runtime.

what to do with direct hits?

Let me clarify a use case I had in mind when asking this. Imagine someone types 'squa' and invokes autocompletion. As things currently stand, 'squa' will automatically morph into 'squareRoot' as we are emptying the bucket with fuzzy candidates. Do we still want to show potential replacements in such cases?

Oh, that's actually a really interesting point.  So the autocomplete for square would be useless, because squareRoot exists.  Huh.  I will think about this more.
One option would be to just check in advance that none of the redirects are prefixes of existing words.  We can probably fix the square situation by just removing squareRoot, which I doubt anyone has ever used.  (Honestly, if this feature had existed, it would just be a redirect to sqrt.  Same for absoluteValue, which should just be a redirect to abs)

I just pushed a commit to remove squareRoot and absoluteValue from the codeworld-base Prelude.  This should resolve the concern about the redirect for square.

This is working, but the redirect table could use some love.  I'll try to do that today.


absence of nonce in transact_call

Hello,
I was going through ethereum's yellow paper (https://ethereum.github.io/yellowpaper/paper.pdf) and it is mentioned that all transactions have a number of common fields:

nonce: A scalar value equal to the number of transactions sent by the sender

Looking at transact_call here:
https://github.com/rust-blockchain/evm/blob/5150c3cc100b7faea8c1976ecfaa6602e2044325/src/executor/stack/mod.rs#L230
there is no nonce specified in the call, and after checking initial gas usage, we immediately increment the nonce for that account in state:

self.state.inc_nonce(caller);

Why doesn't this function take a nonce as an argument (and check against state's nonce)? Are these functions supposed to be called by some other API which does take nonce?

The EVM in this repo mainly used by Frontier, an Ethereum compatibility layer for Substrate. There is a wrapper evm pallet in frontier which responsible for evm execution process.
Indeed, there is no nonce specified in transact_call or transact_create arguments, the nonce check can be found here https://github.com/paritytech/frontier/blob/master/frame/evm/src/runner/stack.rs#L83.
I hope this information can be helpful.

This helps a lot! Thanks @AsceticBear
I will try to read more into Frontier and how it is using this evm. Any help in that regard is appreciated.

https://www.youtube.com/watch?v=U18uEgWVZy4
This helped a lot.


Why are my motion ports working?

Hi I have been using MikuMikuModel for a while now but no matter what I do the application does it's job and works perfectly, why is this happening?

.

😔


RedisMemoryStore : ERR unknown command 'FT.INFO', with args beginning with: 'MyCollection'

When trying to use RedisMemoryStore to upload memories I get ERR unknown command 'FT.INFO', with args beginning with: 'MyCollection'
To Reproduce
I have tested this with Azure Cache and locally with Redis running in docker.
`
Config.GetSection("Redis").Bind(redisConfig);
IDatabase redis = BuildRedis(redisConfig);
    var kernel = new KernelBuilder()
        .WithAzureTextCompletionService(deploymentName: kernelSettings.TextCompletionDeployment,
            endpoint: kernelSettings.Endpoint, apiKey: kernelSettings.ApiKey)
        //Memory
        .WithAzureTextEmbeddingGenerationService(kernelSettings.TextEmbeddingDeployment, kernelSettings.Endpoint,
            kernelSettings.ApiKey)
        .WithMemoryStorage(new RedisMemoryStore(redis, 1536))
        //Chat
        .WithAzureChatCompletionService(kernelSettings.ChatCompletionDeployment, kernelSettings.Endpoint,
            kernelSettings.ApiKey)
        .Build();`

I get this error:
StackExchange.Redis.RedisServerException: ERR unknown command 'FT.INFO', with args beginning with: 'MyCollection'
at NRedisStack.Auxiliary.ExecuteAsync(IDatabaseAsync db, SerializedCommand command)
at NRedisStack.SearchCommandsAsync.InfoAsync(RedisValue index)
at Microsoft.SemanticKernel.Connectors.Memory.Redis.RedisMemoryStore.DoesCollectionExistAsync(String collectionName, CancellationToken cancellationToken)
at Microsoft.SemanticKernel.Memory.SemanticTextMemory.SaveInformationAsync(String collection, String text, String id, String description, String additionalMetadata, CancellationToken cancellationToken)
Desktop:

OS: Windows
IDE: Rider
NuGet Package Version : 0.16.230615.1-preview

I was testing with a local instance and Azure Cache with Standard pricing.
Will give it another go with enterpize
Thank you for the quick response.


Crashes with undefined index: 0 inside docker container

I have included prestissimo inside a project. We are using docker container and prestiismio is globally installed during the build.
It worked well on most cases, yet one developer's machine has the issue:
[ErrorException]
  Undefined index: 0

Exception trace:
 () at /root/.composer/vendor/hirak/prestissimo/src/ParallelDownloader.php:93
 Composer\Util\ErrorHandler::handle() at /root/.composer/vendor/hirak/prestissimo/src/ParallelDownloader.php:93
 Hirak\Prestissimo\ParallelDownloader->download() at /root/.composer/vendor/hirak/prestissimo/src/Plugin.php:75
 Hirak\Prestissimo\Plugin->onPostDependenciesSolving() at n/a:n/a
 call_user_func() at phar://composer.phar/bin/../src/../src/Composer/EventDispatcher/EventDispatcher.php:156
 Composer\EventDispatcher\EventDispatcher->doDispatch() at phar://composer.phar/bin/../src/../src/Composer/EventDispatcher/EventDispatcher.php:133
 Composer\EventDispatcher\EventDispatcher->dispatchInstallerEvent() at phar://composer.phar/bin/
../src/../src/Composer/Installer.php:524
 Composer\Installer->doInstall() at phar://composer.phar/bin/../src/../src/Composer/Installer.php:232
 Composer\Installer->run() at phar://composer.phar/bin/../src/../src/Composer/Command/InstallCommand.php:134
 Composer\Command\InstallCommand->execute() at phar://composer.phar/bin/../src/../vendor/symfony/console/Command/Command.php:259
 Symfony\Component\Console\Command\Command->run() at phar://composer.phar/bin/../src/../vendor/symfony/console/Application.php:844
 Symfony\Component\Console\Application->doRunCommand() at phar://composer.phar/bin/../src/../vendor/symfony/console/Application.php:192
 Symfony\Component\Console\Application->doRun() at phar://composer.phar/bin/../src/../src/Composer/Console/Application.php:166
 Composer\Console\Application->doRun() at phar://composer.phar/bin/../src/../vendor/symfony/console/Application.php:123
 Symfony\Component\Console\Application->run() at phar://composer.phar/bin/../src/../src/Composer/Console/Application.php:99
 Composer\Console\Application->run() at phar://composer.phar/bin/composer:43
 include() at /usr/local/bin/composer:25

What could be the issue here?

It seems the issue was related to too many requests to the github.com api which was throwing 403 errors. Hence, there were no files and nothing could be properly installed.
Adding an auth.json for composer to use at the projects parent folder with  a personal access token to public github repositories fixed the issue:
{
    "github-oauth": {
        "github.com": "YOUR_GITHUB_ACCESS_TOKEN"
    }
}

I think this is the same problem https://github.com/hirak/prestissimo/issues/41
Please try 0.1.10.
$ composer global update hirak/prestissimo --no-plugins

@hirak I am not sure it's entirely a duplicate as the issue went away, as pointed out, after adding an auth.json. No update of prestissimo was required.

@hirak I found useful the tweak about github-auth. Could you add/mention the tweak in README?


Could not parse Jison grammar

Hello! I am trying to redo the last lab and I get this error

I have tried deleting it and installing it once again. It didn't act up before.
Could you help me, please?

Hi!
Please put the .y file first (jison ex1.y ex1.l).

Thanks a lot! I see why now.


Support new React Native new architecture (TurboModules/Fabric)

See https://reactnative.dev/docs/new-architecture-library-intro
https://reactnative.dev/blog/2022/06/16/resources-migrating-your-react-native-library-to-the-new-architecture

Thank you for your issue submission.
On December 1, we will be are closing out the issues tabs on the Usabilla GitHub repos. We will continue supporting the apps/SDKs via our Help Center and support team. Information on how to contact support will be provided in the readme sections of the repos. By consolidating our contact channels, our team can monitor and reply more efficiently.
If the issue you have reached out about persists, please send us an email at<EMAIL_ADDRESS>Please include all details provided here in your email, along with any other details that may be helpful.
-GetFeedback Digital Team


Document how to bring your own authentication on a custom domain

Upgrade to this page: https://www.medplum.com/docs/auth/auth-in-applications

Fixed by #1468


Change PUK link in top part of page is a broken link. Leads to nothing.

An attacker can use this to reset your PIN and use your Yubikey. Please see the Change PUK section for details on how to change your PUK.
In this sentance, the Change PUK link is broken and directs you to nothing. Please fix.

I’ll try and take a look this week.  Thanks!

Fix is in: https://github.com/drduh/YubiKey-Guide/pull/295

Much appreciated!


Links to webpage broken

Can't see the demos without the pages! The links in the Readme file appear broken.

Weird, both http://skeate.github.io/Leaflet.timeline and http://skeate.github.io/Leaflet.buffer are down. Investigating

Regenerated the gh-pages files and they seem to be up now. Weird.
Thanks for the heads up!


Fix message store timestamp index parsing in go_account_stats

Currently the Vumi message store formats timestamps in indexes using str(msg['timestamp']) which results in different behaviour depending on whether the microseconds are zero or not:
>>> str(n.replace(microsecond=0))
'2015-03-05 22:11:00'
>>> str(n)
'2015-03-05 22:11:00.124000'

Current go_account_stats only supports timestamps with microseconds set to 0 (unlikely in production).

Ready for review.

:+1:


compare sbom and collected_sources manifest

This will install the compare-sbom-sources utility
Will compare the output of the sbom and the collected_sources target

@yash-zededa , it looks like you have squashed some commits during development that have concatenated a lot of "minor fixes" in your commit message. Please, correct them. Let just a summary of the changes you are doing and just one Signed-off in your final commit.

@rene thanks for pointing out. This is fixed.

@eriknordmark the os.Exit(1) is correct; if it cannot execute the command, exit 1.
The log.Fatal, on the other hand, shouldn't be there. I will open a PR for it.

See #3195


Batch subgroup check

When deserializing points (see ecc/xxx/Decoder) , we do a subgroup check.
gnark may need to deserialize millions of such points, and performance wise, we can't afford to do multiple scalar multiplication per point.
See (?):

https://cseweb.ucsd.edu/~mihir/papers/batch.pdf

@gbotrel Is this still an issue?

Not a pressing one; in prod scenarios I'm aware of (i.e. with large number of points), provenance of the points is trusted and we skip subgroup checks anyway for fast de-serialization.


Changing Epoch

How to increase or decrease epoch number while fine tunning ?

Did you try setting the n_epochs arg in the finetune method? https://github.com/neuspell/neuspell/blob/master/neuspell/corrector_subwordbert.py#L77

@kailashkarthiks Yes, I solved it by doing that. Thnx anyway.


Low Battery Issue with device HmIP-eTRV-2

Describe the bug
in the low batterie list there are devices, which are not on low battery. It concerns all types of devices HmIP-eTRV-2
To Reproduce
happens every time. a new start of iobroker and restarting the instance didn't help.
Expected behavior
the devices are not in the battery low list.
Screenshots & Logfiles
Low Bat list of Device Watcher
[{"Device":"a","Adapter":"Homematic RPC","Battery":"2.8V"},{"Device":"b","Adapter":"Homematic RPC","Battery":"2.7V"},{"Device":"c","Adapter":"Homematic RPC","Battery":"2.8V"},{"Device":"d","Adapter":"Homematic RPC","Battery":"2.7V"},{"Device":"e","Adapter":"Homematic RPC","Battery":"2.6V"},{"Device":"f","Adapter":"Homematic RPC","Battery":"2.7V"},{"Device":"g","Adapter":"Homematic RPC","Battery":"2.7V"},{"Device":"h","Adapter":"Homematic RPC","Battery":"2.7V"},{"Device":"i","Adapter":"Homematic RPC","Battery":"2.7V"},{"Device":"k","Adapter":"Homematic RPC","Battery":"2.7V"},{"Device":"l,"Adapter":"Homematic RPC","Battery":"2.4V"}]
Data for device e

Versions:

Adapter version: v2.9.3
JS-Controller version:  5.0.12
Node version: 18.17.1
Operating system: debian 12

Thank you for this report. Don't the devices have the data point 'lowbat alarm' ? I don't see it in your screenshot.

Hi. No, there is no datapoint lowbat alarm. Only lowbat. This is in my case 'false'.

Okay, then I have to add an fallback for this case. I‘ll work on it in the next days when I have time for it.

Hi @ciddi89,
I just upgraded my version but the problem does still exists, doesn it?


bad result

I tried to defish the fisheye image with 130° or 160° fisheye len. But I didn't get the good result like you.
org image:

corrected result:

Is there any parameters can use to get better result?

Hi,
Did you use option '0' or '1' for correction method.
Note that option '0' will only work with full circle images and does not require any further input. The circle shot should fit nicely with the top and bottom edge though. This is not the case in your input.
I hope you are using option '1' ? It will require marking few points on the circle edge. Can you provide me a screen shot of the points you have marked ?
Anyways, I will try running your input image on the code when I get some time. Will update on what I find. I wrote the code nearly 4 years ago so I'm a bit rusty on the innards, may take some time to fix.

Thank you for your reply!
I have two kinds images.
One kind image below, with 130° len:

Result with method 0:

Result with method 1:

Another type image with 160° len:

Result with method 0, lost some edge image:

Result with method 1:

You can find some examples in google drive.
Thank you for your reply again!

@junedgar Hey, could you please tell where you have modifiy the code part in which you added path of your images ? In which line and what to modify.
Your help would be great to help me. Thanks

@Shivani1796  you can comment the parameters check in line 99 to 104， and change the line 107-108 to your local input filename and local output filename. https://github.com/sumandeepb/FishEyeUnwarping/blob/3531235cbb400120b3d4f626e34f697477ea56f1/ImageWarping.cpp#L107-L108

@junedgar Hey, thanks for reply but I didn't understand you what are you trying to explain can you please elaborate ? Or you mean String strInputFile = argv[//here is should write path where images are]; and String strOutputFile = argv[2]; instead of 2 need to write output path ?

Answered here:
https://github.com/sumandeepb/FishEyeUnwarping/issues/2#issuecomment-541611646


Installation failure

Versions

Hardware: Odroid N2
YunoHost version: installed a week ago
I have access to my server

Edit: https://paste.yunohost.org/raw/sibupaqaxo
Log via web:
Removing the app following the installation failure...
Could not complete the operation 'Install the 'wekan' app'. Please provide the full log of this operation by clicking here to get help
Unable to install wekan: An error occurred inside the app installation script
Unable to install dependencies
E: Package 'mongodb-org-tools' has no installation candidate
E: Package 'mongodb-org-server' has no installation candidate
E: Package 'mongodb-org' has no installation candidate
is only available from another source
This may mean that the package is missing, has been obsoleted, or
Package mongodb-org-tools is not available, but is referred to by another package.
is only available from another source
This may mean that the package is missing, has been obsoleted, or
Package mongodb-org-server is not available, but is referred to by another package.
is only available from another source
This may mean that the package is missing, has been obsoleted, or
Package mongodb-org is not available, but is referred to by another package.
E: Packages need to be removed but remove is disabled.
Output information may be incomplete.
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Output information may be incomplete.
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Output information may be incomplete.
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Output information may be incomplete.
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Output information may be incomplete.
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Installing MongoDB...
Installation of N - Node.js version management
Output information may be incomplete.
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Output information may be incomplete.
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Output information may be incomplete.
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Output information may be incomplete.
Installing dependencies...
lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Finding an available port...
Storing installation settings...
Validating installation parameters...
Installing wekan...

Log via CLI:
...
Info: Installing wekan...
Info: [+...................] > Validating installation parameters...
Info: [#+..................] > Storing installation settings...
Info: [##+.................] > Finding an available port...
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Info: [###++...............] > Installing dependencies...
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Info: Installation of N - Node.js version management
Info: Installing MongoDB...
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Warning: E: Packages need to be removed but remove is disabled.
Warning: Package mongodb-org is not available, but is referred to by another package.
Warning: This may mean that the package is missing, has been obsoleted, or
Warning: is only available from another source
Warning: Package mongodb-org-server is not available, but is referred to by another package.
Warning: This may mean that the package is missing, has been obsoleted, or
Warning: is only available from another source
Warning: Package mongodb-org-tools is not available, but is referred to by another package.
Warning: This may mean that the package is missing, has been obsoleted, or
Warning: is only available from another source
Warning: E: Package 'mongodb-org' has no installation candidate
Warning: E: Package 'mongodb-org-server' has no installation candidate
Warning: E: Package 'mongodb-org-tools' has no installation candidate
Warning: Unable to install dependencies
Error: Unable to install wekan: An error occurred inside the app installation script
Info: The operation 'Install the 'wekan' app' could not be completed. Please share the full log of this operation using the command 'yunohost log display 20210224-024232-app_install-wekan --share' to get help
Warning: Here's an extract of the logs before the crash. It might help debugging the error:
Info: DEBUG - Reading state information...
Info: DEBUG - Correcting dependencies... Done
Info: DEBUG - The following packages will be REMOVED:
Info: DEBUG -   wekan-ynh-deps
Info: DEBUG - 0 upgraded, 0 newly installed, 1 to remove and 1 not upgraded.
Info: WARNING - E: Packages need to be removed but remove is disabled.
Info: DEBUG - ++ cat /tmp/tmp.m5ZKMaB3Zc/dpkg_log
Info: DEBUG - ++ grep -oP '(?<=-ynh-deps depends on ).*(?=; however)'
Info: DEBUG - ++ tr '\n' ' '
Info: DEBUG - + local 'problematic_dependencies=mongodb-org mongodb-org-server mongodb-org-tools '
Info: DEBUG - + [[ -n mongodb-org mongodb-org-server mongodb-org-tools  ]]
Info: DEBUG - + ynh_package_install mongodb-org mongodb-org-server mongodb-org-tools --dry-run
Info: DEBUG - + ynh_apt --no-remove --option Dpkg::Options::=--force-confdef --option Dpkg::Options::=--force-confold install mongodb-org mongodb-org-server mongodb-org-tools --dry-run
Info: DEBUG - + ynh_wait_dpkg_free
Info: DEBUG - + sed --quiet '/Reading state info/,$p'
Info: DEBUG - + grep -v 'fix-broken\|Reading state info'
Info: DEBUG - + return 0
Info: DEBUG - + LC_ALL=C
Info: DEBUG - + DEBIAN_FRONTEND=noninteractive
Info: DEBUG - + apt-get --assume-yes --quiet -o=Acquire::Retries=3 -o=Dpkg::Use-Pty=0 --no-remove --option Dpkg::Options::=--force-confdef --option Dpkg::Options::=--force-confold install mongodb-org mongodb-org-server mongodb-org-tools --dry-run
Info: DEBUG - + ynh_die '--message=Unable to install dependencies'
Warning: Removing the app following the installation failure...
Info: [++..................] > Loading installation settings...
Info: [##++................] > Stopping and removing the systemd service...
Info: [####++..............] > Removing the MongoDB database...
Warning: ynh_mongo_db: line 110: mongo: command not found
Warning: ynh_mongo_db: line 198: [: -lt: unary operator expected
Warning: ynh_mongo_db: line 96: mongo: command not found
Warning: ynh_mongo_db: line 96: mongo: command not found
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Info: [######++............] > Removing dependencies...
Warning:       Output information may be incomplete.
Warning: lsof: WARNING: can't stat() fuse.gvfsd-fuse file system /run/user/1000/gvfs
Warning:       Output information may be incomplete.
Info: '/usr/local/n' wasn't deleted because it doesn't exist.
Info: Removing MongoDB service...
Warning: Unknown service 'mongod'
Warning: ynh_secure_remove called with empty argument, ignoring.
Info: [########+++.........] > Removing app main directory...
Info: [###########++.......] > Removing nginx web server configuration...
Info: '/etc/nginx/conf.d/kando2.thingsandstuff.org.d/wekan.conf' wasn't deleted because it doesn't exist.
Warning: The user wekan was not found
Info: [#############++.....] > Removing the dedicated system user...
Error: The operation 'Install the 'wekan' app' could not be completed. Please share the full log of this operation using the command 'yunohost log display 20210224-024232-app_install-wekan --share' to get help

Aah, no ARM :(

Wait, afaium Wekan on arm64 is actually a thing these days.

there is mongodb for arm64 but not for juste arm.

Hi, I'm on arm64 and I have the same issue
here is my log :
2022-06-08 08:19:16,523: DEBUG - + apt-get --assume-yes --quiet -o=Acquire::Retries=3 -o=Dpkg::Use-Pty=0 --no-remove --option Dpkg::Options::=--force-confdef --option Dpkg::Options::=--force-confold install mongodb-org mongodb-org-server mongodb-org-tools --dry-run 2022-06-08 08:19:17,786: WARNING - Package mongodb-org is not available, but is referred to by another package. 2022-06-08 08:19:17,788: WARNING - This may mean that the package is missing, has been obsoleted, or 2022-06-08 08:19:17,789: WARNING - is only available from another source 2022-06-08 08:19:17,791: DEBUG -  2022-06-08 08:19:17,791: WARNING - Package mongodb-org-server is not available, but is referred to by another package. 2022-06-08 08:19:17,793: WARNING - This may mean that the package is missing, has been obsoleted, or 2022-06-08 08:19:17,794: WARNING - is only available from another source 2022-06-08 08:19:17,796: DEBUG -  2022-06-08 08:19:17,796: WARNING - Package mongodb-org-tools is not available, but is referred to by another package. 2022-06-08 08:19:17,798: WARNING - This may mean that the package is missing, has been obsoleted, or 2022-06-08 08:19:17,799: WARNING - is only available from another source 2022-06-08 08:19:17,801: DEBUG -  2022-06-08 08:19:17,802: WARNING - E: Package 'mongodb-org' has no installation candidate 2022-06-08 08:19:17,803: WARNING - E: Package 'mongodb-org-server' has no installation candidate 2022-06-08 08:19:17,805: WARNING - E: Package 'mongodb-org-tools' has no installation candidate 2022-06-08 08:19:17,806: DEBUG - + ynh_die '--message=Unable to install dependencies' 2022-06-08 08:19:17,828: WARNING - Unable to install dependencies 2022-06-08 08:19:17,831: DEBUG - + ynh_exit_properly 2022-06-08 08:19:18,344: DEBUG - + ynh_clean_setup 2022-06-08 08:19:18,344: DEBUG - + ynh_clean_check_starting 2022-06-08 08:19:18,344: DEBUG - + '[' -n '' ']' 2022-06-08 08:19:18,345: DEBUG - + '[' -n '' ']' 2022-06-08 08:19:19,347: ERROR - Impossible d'installer wekan : Une erreur est survenue dans le script d'installation de l'application


Change GasGiantsEnhanced dep on RSS to rec

This mod can be used with either RSS or KSRSS, but currently we have it depending on RSS. If a user wants to use KSRSS instead, there's no way to not install RSS.
KSRSS is not in CKAN and is hosted on GitLab, so the best we can do is to allow RSS to be removed. So now RSS is recommended instead of a depends.
Fixes #9000.

With KSP-CKAN/CKAN#3661 we can now consider indexing KSRSS from its GitLab hosting. I have looked for previous correspondence about this mod on GitHub, the forum, and Discord and found nothing; neither the author hoping to be added nor preferring not to, not even someone else requesting it.
The closest is this issue on the GitLab repo, which has no discussion other than the title:
https://gitlab.com/ksrss/KSRSS/-/issues/10
So effectively we would be starting from scratch in communicating here.

Via Discord PM, the KSRSS team is planning for CKAN support, but it's not ready yet:

Consider revisiting this after #10158 is completed.


Abstract File Parameters

Make AddFileParameters and UpdateFileParameters inherit from a new, abstract class
This will allow client code to abstract processing of configuration files for both adding and updating files, reducing the need for boilerplate code.

This change is

@papeh Great, thank you!


tf.train.import_meta_graph raise ValueError(str(e)) ValueError: Cannot add function '__inference_Dataset_flat_map_read_one_file_11' because a different function with the same name already exists.

Traceback (most recent call last):
File "test.py", line 121, in
test()
File "test.py", line 67, in test
saver = tf.train.import_meta_graph('checkpoint/-301.meta')
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/saver.py", line 1449, in import_meta_graph
**kwargs)[0]
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/saver.py", line 1473, in _import_meta_graph_with_return_elements
**kwargs))
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/meta_graph.py", line 857, in import_scoped_meta_graph_with_return_elements
return_elements=return_elements)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/util/deprecation.py", line 507, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/importer.py", line 431, in import_graph_def
raise ValueError(str(e))
ValueError: Cannot add function '__inference_Dataset_flat_map_read_one_file_11' because a different function with the same name already exists.
using tensorflow version 1.14
？？？ why? how?

@Gzzgz,
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

@Gzzgz, Please provide the standalone code to reproduce the issue. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


feat(lcov): add lcov coverage

Pull Request Test Coverage Report for Build 539

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

Totals

Change from base Build 535:
0.0%

Covered Lines:
508

Relevant Lines:
523

💛  - Coveralls


Implement client side message validation

not finished;

[x] invalid questions are still showing up under the questions tab

Realized I forgot to say good work!

Are you going to be adding validation on the server as follow up?

Oh, I for some reason thought the validation had to be done on the client-side. But yes, I can add the validation on the server-side. In that case, would I just get rid of everything on the client side? We would only need to validate the input once, either in the client or the server side, right?

Are you going to be adding validation on the server as follow up?

Oh, I for some reason thought the validation had to be done on the client-side. But yes, I can add the validation on the server-side. In that case, would I just get rid of everything on the client side? We would only need to validate the input once, either in the client or the server side, right?

As far as I know, you do validation on both frontend and backend. (here's a link, but feel free to google some things and let me know if you find anything interesting)

Also, the reason you would do it on both is that theoretically, someone could write some code that sends a request to your backend without ever using your frontend.  That's generally why backend validation is done.  Frontend validation is done so you don't have to ping your server or wait for a request to load before the user gets a response (think of a slow internet connection or someone sending many invalid requests via your client).

Frontend validation is done so you don't have to ping your server or wait for a request to load before the user gets a response (think of a slow internet connection or someone sending many invalid requests via your clie

ah that makes so much more sense. I definitely wasn't thinking of a scenario where someone could directly send a request to the backend. Thank you for your kind explanations!! It's really cool to see, understand, and actually implement some of the good practices when coding like the ones you mentioned above! I'll go ahead and start working on the server-side validation.

Frontend validation is done so you don't have to ping your server or wait for a request to load before the user gets a response (think of a slow internet connection or someone sending many invalid requests via your clie

ah that makes so much more sense. I definitely wasn't thinking of a scenario where someone could directly send a request to the backend. Thank you for your kind explanations!! It's really cool to see, understand, and actually implement some of the good practices when coding like the ones you mentioned above! I'll go ahead and start working on the server-side validation.

I'm glad you're enjoying the learning experience!  The server side things can be a separate PR.  It's fine for now if it's only client side.


xfail pragma: eg pragma: not covered

My usecase is a function that can never be called - It's only checked for callable-ness:
def dummy_view():
    raise Exception("Cannot be called")  # pragma: not covered

I'd like coverage to fail the build if that line is covered.
see also this SA question: https://stackoverflow.com/questions/47078737/is-there-an-equivalent-of-xfail-for-coverage

I don't understand why you need coverage to check this: if the line is executed, it will raise an exception, and your test will fail.

@nedbat it might not fail if someone accidentally catches the exception

I also do a lot of work in projects where exceptions go missing, or coroutines must be aborted at a particular step.
eg https://github.com/agronholm/anyio/blob/5ed3270c8d12cffc4cd3349d9ff32bc32451ae65/tests/test_taskgroups.py#L165-L172
async def test_start_cancelled() -> None:
    started = finished = False

    async def taskfunc(*, task_status: TaskStatus) -> None:
        nonlocal started, finished
        started = True
        await sleep(2)
        finished = True

    async with create_task_group() as tg:
        tg.cancel_scope.cancel()
        await tg.start(taskfunc)

    assert started
    assert not finished

I'd like to be able to use:
async def test_start_cancelled() -> None:
    started = finished = False

    async def taskfunc(*, task_status: TaskStatus) -> None:
        nonlocal started, finished
        started = True
        await sleep(2)
        finished = True  # pragma: must not run

    async with create_task_group() as tg:
        tg.cancel_scope.cancel()
        await tg.start(taskfunc)

    assert started
    assert not finished

@nedbat what would be acceptable for this feature? To have a new line on the report?

@Kludex I'm still trying to understand how people would use this.  It still seems to me that raising an exception from the line that must not run would be the easiest way to get what's needed here.

This is for places where an exception may not be collected, eg in an aborted coroutine in a background task


[spec/function.dd] Add scope parameter section

Tweak existing wording
Mention @safe & -dip1000 needed for enforcement

Update:

Split existing return ref example.

@thewilsonator Sorry I forgot about this. Rebased! (At least I get CircleCI notifications now when my builds fail :-) ).

The scope parameter changes were getting a bit big for the function parameter table, so I have moved them to their own section and split out an example from the return scope docs.


Move all .ipynb files into the /notebooks/ folder

This message was generated by git-bob (version: 0.2.8, model: gpt-4o-2024-08-06), an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.
Relevant Information
Sub-task: Move all .ipynb files into the /notebooks/ folder.
Proposal to Solve the Sub-task

Identify all .ipynb files in the current project directory and its subdirectories.
Create a /notebooks/ directory if it does not already exist.
Move each identified .ipynb file into the /notebooks/ directory.
Update any references or paths to these notebooks in the project to reflect the new directory location.

This message was generated by git-bob (version: 0.2.8, model: gpt-4o-2024-08-06), an experimental AI-based assistant. It can make mistakes and has limitations. Check its messages carefully.
Sub-issues have been created:

#222
#223
#224
#225


Removing explicit ResponseObjects

Instead of having separate response objects for Handles, DOIs, DigitalSpecimens, etc, we've moved to a more lightweight JSON response, allowing more code reuse.
Request objects are still separate for each PID type. There are no plans to change how we ingest requests.

Just put that Sonar issue on False Positive or something. Don't see much of an issue on these Record classes with some more parameters for the constructor


Adds missing configuration for project to MkDocs configuration

Traceback (most recent call last):
  File "/github/workspace/documentation-theme/update_mkdocs_yml.py", line 56, in <module>
    mkdocs["extra"]["repo_name"] = mkdocs["repo_url"].replace("https://github.com/", "")
KeyError: 'extra'

https://github.com/laminas/laminas-servicemanager-migration/actions/runs/3747785291/jobs/6364385361#step:3:34

Thank you @froschdesign


crtm: use v<IP_ADDRESS>

Summary
This PR moves the default crtm to v<IP_ADDRESS>
Testing
TBD
Applications affected
Global Workflow, UPP, UFS SRW, UFS WM, GSI
Systems affected
all
Dependencies
https://github.com/JCSDA/spack/pull/376
Issue(s) addressed
Related to https://github.com/NOAA-EMC/global-workflow/issues/1356
Checklist

[x] This PR addresses one issue/problem/enhancement, or has a very good reason for not doing so.
[ ] These changes have been tested on the affected systems and applications.
[ ] All dependency PRs/issues have been resolved and this PR can be merged.

@Hang-Lei-NOAA FYI


Documentation update

Codecov Report

Merging #102 (a347427) into main (b345a9a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #102   +/-   ##
=======================================
  Coverage   97.68%   97.68%
=======================================
  Files         234      234
  Lines        6706     6709    +3
=======================================
+ Hits         6551     6554    +3
  Misses        155      155

Impacted Files
Coverage Δ

ellar/samples/controllers/home.py
100.00% <100.00%> (ø)


Privacy Policy blocks update

It seems that I can't release a newer version of the app on playstore as it not requires I provide a privacy policy...
But I don't have one. Can someone help me draft a privacy policy? I collect literally nothing from the device.

I just asked ChatGPT "Please write a privacy policy for an Android app that does not collect any data." and this was the result, which reads a little bit tongue-in-cheek, but seems appropriate for a privacy policy if the app does not collect data. Feel free to use, modify and share it:
Privacy Policy
Thank you for using our Android app. This Privacy Policy is intended to inform you about the data we do not collect and how we ensure your privacy while using our app.
Data Collection
We do not collect any personal information from you, such as your name, email address, or phone number. We do not collect any sensitive information, such as your location, contacts, or browsing history. Our app does not use cookies, analytics tools, or third-party services that could track your activity.
Data Storage
Since we do not collect any data, we do not store any data on our servers or any third-party servers. Your device may automatically store some data related to your use of the app, such as preferences or settings, but we do not have access to this data.
Data Sharing
We do not share any data with third-party entities, including advertisers, partners, or affiliates. We do not sell any data to anyone.
Data Security
We take reasonable measures to protect any data we may receive or store accidentally, such as through error logs or crash reports. However, since we do not collect any personal or sensitive data, we do not have to take any special security measures.
Changes to Privacy Policy
We may update this Privacy Policy from time to time. If we make any material changes, we will notify you through the app or by email.
Contact Us
If you have any questions or concerns about this Privacy Policy, please contact us at [contact email].

I created a minimal privacy policy page
https://github.com/flide/8VIM/wiki/Privacy-Policy
Let me update this on the page instead.
Thanks to you and ChatGPT.


Add Support for rebase in Merge Request

https://docs.gitlab.com/ce/api/merge_requests.html#rebase-a-merge-request -> Automatically rebase the source_branch of the merge request against its target_branch，it is supported in the API
I think it's cool，I need this feature. Can you help me add it?
thank you

I think this PR should add support for rebasing MRs: https://github.com/xanzy/go-gitlab/pull/534

Wow, thank you very much for responding to my request so quickly. I saw your submission and hope to add it to the new version as soon as possible.


Setup AKS on HCI nodes without need for Azure integration / authentication

We deliver our software for our clients to run completely on-prem. Setting up AKS through Windows Admin Center requires an Azure account or Azure AD Account with domain admin level permissions if Azure AD is already detected in the environment. Can we have an option to setup completely outside / disconnected from Azure all together?

Using the available powershell commands, you don't need an Azure Account


data forlder stored in hassio?

I'm running itgz Minecraft Java on a linux machine today, but I would like to transfer the world and config from that machine, so I wonder where is the data folder stored for Minecraft Java in Hassio docker

#10


Cannot build typescript project

Our npm buildpack runs npm ci to install the runtime dependencies only.  But TypeScript projects seem to often install the typescript toolchain via devDependencies and then run tsc via a postinstall script, and so fail to install.  Perhaps we should npm install followed by an npm prune.
For an example, try building the VSCode typescript-in-docker example.
Status: Image is up to date for gcr.io/buildpacks/gcp/run:latest
===> DETECTING
[detector] 2 of 4 buildpacks participating
[detector] google.nodejs.runtime 0.9.0
[detector] google.nodejs.npm     0.9.0
===> ANALYZING
===> RESTORING
===> BUILDING
[builder] === Node.js - Runtime<EMAIL_ADDRESS>===
[builder] Resolving Node.js version based on semver ""
[builder] Using resolved runtime version from package.json: 14.2.0
[builder] Installing Node.js v14.2.0
[builder] === Node.js - NPM<EMAIL_ADDRESS>===
[builder] Installing application dependencies.
[builder] ————————————————————————————————————————————————————————————————————————————————
[builder] Running "npm ci --quiet (NODE_ENV=production)"
[builder]
[builder] ><EMAIL_ADDRESS>postinstall /workspace
[builder] > tsc -p ./src
[builder]
[builder] sh: 1: tsc: not found
[builder] npm ERR! code ELIFECYCLE
[builder] npm ERR! syscall spawn
[builder] npm ERR! file sh
[builder] npm ERR! errno ENOENT
[builder] npm ERR<EMAIL_ADDRESS>postinstall: `tsc -p ./src`
[builder] npm ERR! spawn ENOENT
[builder] npm ERR!
[builder] npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
[builder] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
[builder]
[builder] npm ERR!  Failed at the<EMAIL_ADDRESS>postinstall script.
[builder] npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
[builder]
[builder] npm ERR! A complete log of this run can be found in:
[builder] npm ERR!     /home/cnb/.npm/_logs/2020-05-07T19_08_30_604Z-debug.log
[builder] ————————————————————————————————————————————————————————————————————————————————
[builder] Sorry your project couldn't be built.
[builder] Our documentation explains ways to configure Buildpacks to better recognise your project:
[builder]  -> https://github.com/GoogleCloudPlatform/buildpacks/blob/master/README.md
[builder] If you think you've found an issue, please report it:
[builder]  -> https://github.com/GoogleCloudPlatform/buildpacks/issues/new
[builder] ————————————————————————————————————————————————————————————————————————————————
[builder] ERROR: failed to build: exit status 1
ERROR: failed with status code: 7

You can work around this by specifying NODE_ENV=development, which is what npm install does by default and the project seems to assume. The prepare script should install devDependencies and then remove them, but there seems to be a bug where it does not if NODE_ENV=production which should be fixed in npm 7.

Have similar issue when using NestJS. I have info that image was build (btw huge image size compared to my Dockerfile) but when I'm trying to run it it's not starting.

Is that with NODE_ENV=development or without? Is there an error message printed on startup?

(btw huge image size compared to my Dockerfile)

What base image do you use in your Dockerfile? The run image we use (gcr.io/buildpacks/gcp/run) includes few packages and is 111MB vs 941mb for the official node:14 image.
A missing required package could be the reason the app is failing to start.

I'm using alpine and slim versions of node 12. On node 13 I had issue with grpc.

I was able to avoid setting NODE_ENV=development by defining the gcp-build script as documented here. I didn't trace back what actually checks for the gcp-build script to execute it but it may be the buildpack.

gcp-build has its own buildpack (source), which is available in App Engine and Cloud Functions.
You can override NODE_ENV at build time using -e NODE_ENV=development if using pack directly or by creating .npmrc when deploying to GAE/GCF: https://github.com/GoogleCloudPlatform/buildpacks/issues/75#issuecomment-686656544.

I have a new sample for this at:
https://github.com/GoogleCloudPlatform/buildpack-samples/tree/master/sample-node-tsc
Because it is pretty common to use devDependencies to build the application, I think the buildpack should do something like the following by default:

Install devDependencies
The postInstall script then runs with devDependencies
The deps only in devDependencies (not in dependencies) should then be removed (unless NODE_ENV=development)

The google.nodejs.npm-gcp-build (https://github.com/GoogleCloudPlatform/buildpacks/blob/main/cmd/nodejs/npm_gcp_build/main.go) seems to be exactly what I'm looking for, but is it not available except for App Engine and Functions?

The prepare script should install devDependencies and then remove them, but there seems to be a bug where it does not if NODE_ENV=production which should be fixed in npm 7.

Unfortunately, prepare script does not seem to behave as expected in npm 7 or later.

I fixed it by including a gcp-build script in my package.json file.
"scripts": {
    "start": "node ./build/server/index.js",
    "gcp-build": "tsc"
  },

Typescript projects should be buildable now that we run npm run build by default https://cloud.google.com/docs/buildpacks/nodejs#executing_custom_build_steps_during_deployment


Added rem and em support

Because rem > em > px :)

HI @okoghenun , sorry for the very late reply :)
This is a really neat module. I think it's something that should be global to a whole project though, not necessarily included in another module like this. With formhack, I think it makes sense for the form resetting to be standalone, and if people want rem support, they can add that as something separate. Especially as the rem unit is something used across a whole project.
I don't know if the rem function is something you created yourself. If it is, then you should create it as a module itself, so people can include it in their projects. I'll add a link to it from here as well so people can use it


Kubernetes environment stuck at "Waiting for Kubernetes API"

Rancher Version:
1.0.1
Docker Version:
1.10.3
OS:
CoreOS 1010.1.0
Steps to Reproduce:
Create Kubernetes environment in Rancher.
Stand up machines running CoreOS. Launch rancher-agent with token for above environment.
Results:

Rancher creates all kuberenetes service containers on the host(s.)
Rancher UI never progresses past "Waiting for Kubernetes API"
Browser debugging console yields a 503 error on api route:

Request URL:http://<IP_ADDRESS>:8080/r/kubernetes/api
Request Headers:
Accept:application/json
Referer:http://<IP_ADDRESS>:8080/env/1a44/kubernetes/waiting
User-Agent:Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.108 Safari/537.36
x-api-csrf:9872A8D19BC54214BFD4B21C0C6655C4
x-api-no-challenge:true
x-api-project-id:1a44
X-Requested-With:XMLHttpRequest

Attaching into the kubectld container and executing kubectl confirms that the Kubernetes API server is running:

> kubectl get services
NAME         CLUSTER_IP   EXTERNAL_IP   PORT(S)   SELECTOR   AGE
kubernetes   <IP_ADDRESS>    <none>        443/TCP   <none>     20m
> kubectl version
Client Version: version.Info{Major:"1", Minor:"1", GitVersion:"v1.1.7", GitCommit:"e4e6878293a339e4087dae684647c9e53f1cf9f0", GitTreeState:"clean"}
Server Version: version.Info{Major:"1", Minor:"2+", GitVersion:"v1.2.0-rancher-1", GitCommit:"ed6532f975cff184196dbe214de8fa0198b415ef", GitTreeState:"clean"}

Expected:
Rancher UI is able to successfully reach the Kubernetes API server and allows use of the Kubernetes environment from a browser.
Or, in the event of errors like this, at least allows the user to bypass the "Waiting for Kubernetes API" /kubernetes/waiting screen, and get access to Kubernetes API keys so they can use kubectl on their machine to debug connectivity from outside of the containers themselves.

Additional info:
IPtables rules from the host running kubernetes api server:
# iptables-save | grep 443
-A KUBE-PORTALS-CONTAINER -d <IP_ADDRESS>/32 -p tcp -m comment --comment "default/kubernetes:https" -m tcp --dport 443 -j REDIRECT --to-ports 45131
-A KUBE-PORTALS-HOST -d <IP_ADDRESS>/32 -p tcp -m comment --comment "default/kubernetes:https" -m tcp --dport 443 -j DNAT --to-destination <IP_ADDRESS>:45131

Attempting to telnet from rancher-server host to the kubernetes apiserver container host does yield an open socket:
andy@brizo:/$ telnet <IP_ADDRESS> 45131
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.
Escape character is '^]'.
GET /

Connection closed by foreign host.

FYI, restarting rancher-server resolved the issue in all environments experiencing the issue.

@aboyett could you also attach cattle-debug.log? Want to look at websocket proxy logs that went there

Here's the cattle-debug log for that day. If you recall, we were actively debugging this between 21:30 and 22:30.
cattle-debug.2016-04-29.log.gz
Let me know if you need any more files.

@aboyett thank you.
I've just hit the same bug myself, and my host wasn't re-added. The error coming from websocket proxy is:
time="2016-05-05T09:48:00-07:00" level=info msg="Invalid backend host requested." hostUuid=53fab1bd-2a55-4107-869a-3b094175bf1b

Looks like some bug in websocket proxy

I have the same issue and restarting the rancher-server isn't getting me past the issue.

@jtwebman if rancher restart doesn't fix it, it would most likely mean that you are hitting a k8s bug that I've described here:
https://github.com/rancher/rancher/issues/4838
You can check "kubernetes" container logs, and if you see panic errors there, that would be the proof tha you are hitting k8s bug.
In @aboyett case kubernetes api server was serving the response (we've checked it by executing kubectl calls omitting the UI), yet websocket proxy couldn't connect to it.
In both cases UI will present an error "Waiting for Kubernetes API"

@cjellick looks similar to the websocket proxy issue you've fixed. Basically after running for a while, Rancher could no longer communicate to kubectl service via websocket proxy. Websocket proxy restart fixed the issue.
@aboyett the issue is not easy to reproduce, but we've put fixes to websocket proxy for the area responsible for handling server-to-server-over-websocketproxy connections

Should be fixed. The websocket-proxy fixes we provided to our customer were verified and they no longer experience k8s + wsp issues.
Can we close @sangeethah ?

@aboyett , there are websocket-proxy fixes provided that prevented rancher from being able to communicate with kubectl service via websocket proxy in rancher-server version v1.1.3.
You should not be hitting this issue anymore when testing with rancher-server version v1.1.3 or above.

If you came here for a rancher desktop issue, then  try to turn off your VPN (if it is enabled).


conservative cache: invalidate infeasible transactions and evict accounts

Description

mempool currently rejects the following types of transactions, but save them to DB for re-evaluation later.

there are too many nonces for an account in the mempool. the extra (higher nonce) txs are saved in db
txs deemed insufficient balance by the conservative state, but can be feasible after a layer is applied (that may contain incoming funds for that account)
a better tx arrived (higher fee) and made higher nonce txs infeasible due to insufficient balance deemed by conservative state

these txs will be retrieved and evaluated again in 2 scenarios:

after a layer is applied
at restart

to avoid querying ALL accounts in scenario [1], the conservative cache leave the account in memory so that it can reset the account after a layer is applied). this introduced a slow memory leak at least, and opens the door for an possible DoS.
but for [2] there is not mitigation. the node currently query all pending transactions at startup.
we need to find a way to manage accounts that only have infeasible txs. below we call such accounts infeasible accounts.
proposed implementation I
TTL for infeasible txs
add a TTL (N layers) for each infeasible tx. after N layers elapsed, mark these txs discarded in DB and never queried again
based on the TTL, the node can evict infeasible accounts from memory periodically.
add upper limit M to number of infeasible accounts kept in memory
this will mitigate the DoS.
conditionally query account pending txs on first tx in cache
assuming the number of infeasible accounts reached the limit, node will not be able to re-assess DB-only txs for some accounts.
(see https://github.com/spacemeshos/go-spacemesh/issues/3663)
to mitigate this situations, the node will query all pending txs of the account when it receives another tx of this account.
essentially, the node will query all pending txs from DB for an account if it's not already in the cache.
proposed implementation II
TTL for infeasible txs (same as Proposed Implementation I)
always evict infeasible accounts from cache
always check account pending txs on first tx in cache
when the cache receive a tx that's not in the cache, always query pending txs in DB.
analysis
proposed implementation I

pros

less DB queries for pending txs
better UX before the cache upper limit M is reached. users do not need to send another tx to prompt re-evaluation of older infeasible txs.

cons

more complicated code keeping infeasible accounts in memory
inconsistent UX before and after the cache upper limit M is reached

proposed implementation II

pros

simpler code (not keeping infeasible accounts in memory at all)

cons

more DB queries for pending txs
users always have to send another tx to prompt re-assessment of the older infeasible txs

decision
i'm leaning towards proposed implementation II due to its simplicity

@dshulyak @fasmat @poszu please review. thank you


What happened to the secret song Hot Dilf?

What is your question?
What happened to the secret song Hot Dilf? After I upgraded to 0.5.1, it vanished. Last I saw it is with 0.4 versions.

it was removed

im so sad right now

:(


[tricycle] Rename variable in launch file

Fix the variable name, possibly from some copy-pasting.

@mergifyio backport iron humble

https://github.com/Mergifyio backport jazzy


Vet Clinic DATABASE query and update animals table 🥇

Vet Clinic Database Query and Table Updates
This PR includes database queries and updates for the Vet Clinic project. it covers various aspects of the database, including querying information and performing updates on the animals table.
Queries Included:

Total count of animals in the Database 🗼.
Count of animals that have never tried to escape 🔢.
Average weight of all animals 🐹.
Comparison of escape attempts per animal species for those born between 1990 and 2000 🧇.

Table Update:

Added a species Column to the animals table 🐤.
Inserted new records for animals with various attributes 🧮.
Demonstrated the use of transaction, savepoints, and rollbacks for updates 🎲.

2.

3.

4.

5.

6.

7.

8.

9.

10.

Thank you 😄.

@skyv26 I am very grateful for the insight and explanation. It has really helped me as a Junior developer to have much understanding 😃.

INSERT DATA ERROR FIX AND CORRECTED AS REQUESTED BY REVIEWER.


how to craft first Backpack? 1.21.3

Mod loader
Fabric
Mod version
last version
Minecraft version
1.21.3
Issue description
how to craft first Backpack? 1.21.3 i am using feather
If Bug

[ ] I have confirmed this bug is reproducible on a minimal setup, not a modpack with many other mods.

Provide the latest.log file
No response

Here are the Recipes for all Backpacks, ia recommend you to use EMI, Just Enough Items or Roughly Enough Items as a recipe viewer mod:
Leather:

Copper:

Gold:

Iron:

Diamond:

Netherite:

End:


S3 - Support Glacier

Add GLACIER_IR storageclass
PutObject must support storageClass
CopyObject must support storageClass
HeadObject should return an ObjectSummary
Add RestoreObject operation
FakeS3 to respect glacier limitations

PutObject already has a headers: Headers property.  Doubtless to provide a way to set any of the obscure configuration options.  Is it worth providing a helper storageClass property?


Add modals windows.

On this milestone

When the user clicks (or taps) the button to check project details, the popup with details about the project appears.
When the user clicks (or taps) the close (X) button, the popup disappears.
Use a JavaScript array to store all of the information for all projects.
For each project you need to store the following pieces of data in a JavaScript object, at a minimum: name, description, featured image, technologies, link to live version, link to source.
Update the main page so that the projects section is created dynamically using the information stored in that JavaScript object.
Mobile and desktop popup.

Changes as requested above, I have worked on them


Newly added model_class argument has not been reflected to server code

Argument model_class has been introduced in #29.
However, it has not been reflected to deployment code so make bloom-176b fails with argument error.
Also, model_class is not reflected to README so example codes fail as well.

BLOOM-176B is working fine for me @koreyou .
Sorry, for the old README. I will update it.
Can you look at the makefile: https://github.com/huggingface/transformers-bloom-inference/blob/main/Makefile
run make bloom-176b


Fixed a typo + more details on advanced settings

Title says it all. I hope I'm right about the added detail concerning the static-grid.

I'm not a big fan of "static-grid" as a term; it seems to make the sentence less clear, rather than more-so. Static is a term with multiple meanings (static geometry that is unchanging, static variables that belong to a class rather than an object, static when you haven't set up your TV right...), and the exact meaning is unclear here even though it read clearly before. The same goes for "space-indexing;" I never know quite what people are referring to when that's mentioned (Is it just a HashMap of points to values? Is it a quadtree/octree?). Thanks for catching the typo though! I'll try to simplify the language a little and merge.

Fair points. :)


Introduce Persistent Volume Claims

Description
Changes proposed in this pull request:

Add create modal via yaml
Add list and details view
Add deep linking to the Storage Classes and Persistent Volumes (for now they don't redirect to those resources until we introduce views for them)
Add tests

Related issue(s)
Close: #531

/retest


LUCI unable to resolve flutter/tools/gn

I ran into an obscure error on LUCI where flutter/tools/gn wasn't found.
/opt/s/w/ir/cipd_bin_packages/python: could not resolve options: failed to resolve Python script: stat /opt/s/w/ir/cache/builder/src/flutter/tools/gn: no such file or directory

See: https://ci.chromium.org/ui/p/flutter/builders/try/Mac Android AOT Engine/21314/overview

This doesn't fail repeatedly does it? Looks like something went wrong with repo setup?

@iskakaushik seems like this a failure being introduced by https://github.com/flutter/engine/pull/28107

That PR is the only one failing to find GN

This also happened on https://github.com/flutter/engine/pull/28071 once. I will keep this open for the next couple of days to see that this doesn't happen again.


[popup] Keyboard shortcut for "show field names" vs "field creator"

Make sure to read the troubleshooting section before creating an issue

[x] I've read the common issues and the one I'm about to create is not one of those documented.
[x] I've read the release note and the one I'm about to create is not already fixed.

Describe the bug
The keyboard shortcut for "show field API names" is f. The keyboard shortcut for "field creater" is F.
When you press f the field creator opens and not show API names. Turning on caps lock does not open field creator. I know on other shorctus that having caps lock on does not let them work (as expected).
To Reproduce
Steps to reproduce the behavior:

Open popup and type f
Field creator opens
Go back to popup and type F
Field creator opens

Expected behavior
I was expected f to open "show API names" and F to open "field creator"
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser Edge
-Version 128.0.2739.79 (Official build) (64-bit)
Inspector version: beta 1.25

Additional context
Add any other context about the problem here.

Thanks for raising the issue! I'll fix it soon !

Fixed, will be released in beta soon


[Bug]: Why 'isPhysicalDevice' type as String on iOS? It's bool on Android.

Platform
iOS 16.2
Plugin
device_info_plus
Version
9.1.0
Flutter SDK
3.16.5
Steps to reproduce
I'm trying to check is user using real device or not.
Code Sample
void getDeviceInfo() async {
    await DeviceInfoPlugin().deviceInfo.then((value) {
      print(value.data);
      print(value.data['isPhysicalDevice']);
      print(value.data['isPhysicalDevice'].runtimeType.toString());

      setState(() {
        isPhysicalDevice = value.data['isPhysicalDevice'];
      });
    });
  }

Logs
flutter: {systemName: iOS, isPhysicalDevice: false, utsname: {release: 21.6.0, sysname: Darwin, nodename: Kimias-MacBook-Air.local, machine: iPhone15,3, version: Darwin Kernel Version 21.6.0: Thu Nov  9 00:38:19 PST 2023; root:xnu-8<IP_ADDRESS>5.10~1/RELEASE_X86_64}, model: iPhone, localizedModel: iPhone, systemVersion: 16.2, name: iPhone 14 Pro Max, identifierForVendor: 1D01D878-3B63-4B75-8990-5C869043C720}
flutter: false
flutter: String
flutter: type 'String' is not a subtype of type 'bool'

### Flutter Doctor

```shell
[✓] Flutter (Channel stable, 3.16.5, on macOS 12.7.2 21G1974 darwin-x64, locale en-ID)
    • Flutter version 3.16.5 on channel stable at /Users/me/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 78666c8dc5 (11 days ago), 2023-12-19 16:14:14 -0800
    • Engine revision 3f3e560236
    • Dart version 3.2.3
    • DevTools version 2.28.4

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/me/Development/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.14.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.85.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.80.0

[✓] Connected device (3 available)
    • iPhone 14 Pro Max (mobile) • 9749841D-1465-4EFC-A035-C22244739XXX • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-16-2 (simulator)

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Checklist before submitting a bug

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I'm using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

have you tried this
      late final bool isPhysicalDevice;       if (Platform.isIOS) {         isPhysicalDevice = (deviceInfo as IosDeviceInfo).isPhysicalDevice;       } else if (Platform.isAndroid) {         isPhysicalDevice = (deviceInfo as AndroidDeviceInfo).isPhysicalDevice;       }


Need atomic transaction operation support

I need to continuously write logs to the database. After accumulating a certain amount of logs, I need to upload to the backend and clear the frontend data.
This requirement requires atomic transaction operation support。
localForage is amazing, make it better please

As mentioned in #17—I think this is out-of-scope for localForage's current aim/goals of creating a simpler API to access underlying browser storage APIs. I think someone could build an atomic layer on-top of localForage, but I don't see it as being in-scope for the core library itself.
If someone had a patch ready that fully implemented atomic commits with tests I wouldn't say no I suppose (if it didn't harm the filesize/etc.), but for now I don't see the core library committing to atomic/transaction support especially when it would need to implement its own to work with localStorage. We couldn't rely on underlying database support for these things in WebSQL/IndexedDB (if they exist there, I'm not sure if they do honestly!)
Closing as a duplicate, but if anyone does work on this as either an extension or something for the core library, please do share it here or in #17! 😄


st2-self-check should ignore any pack actions which name doesn't start with "test_"

This pull request fixes bug in st2-self-check inadvertently introduced in this PR - https://github.com/StackStorm/st2tests/pull/150.
It updates st2-self-check code so it only tries to run actions which name starts with test_.
(Per my comment in the other PR, I assume we ended up with this bug / regression due to lack of understanding of the difference between various test repos and packs we have - we need better developer documentation on that).

With https://github.com/StackStorm/st2tests/pull/153 which fixes the root cause, this change is not technically needed anymore, but I don't think it hurts either and it's an additional safe guard.

I noticed yet another "unrelated" self check issue which I fixed in bb8be32.
We didn't skip winrm runner tests by default so self check would fail (same as other Windows runners, those tests don't work out of the box and shouldn't run by default).


Clarify what to do when StackOverflow questions get no answers

Current coverage is 100% (diff: 100%)

Merging #2497 into master will not change coverage

@@           master   #2497   diff @@
=====================================
  Files          66      66
  Lines        1340    1340
  Methods         0       0
  Messages        0       0
  Branches        0       0
=====================================
  Hits         1340    1340
  Misses          0       0
  Partials        0       0

Powered by Codecov. Last update 4514088...2587278

It seems like CodeClimate need some love in order to understand a documentation PR...

Yeah, I connected code climate, but never got around to doing the work to actually make it work...


Update CONTRIBUTING.md

感谢提交 Pull Requests！
Thanks for submitting a pull request.

Security Check : 3a8aa69046a1926c3c79f60dc745efd134a53e5b

auto build success,number:74

autotest success,number:1143


docs: refactored directory and added config.json

Added

config.json file in order to create sidebar

also I have refactored the docs directory in docusaurus style

cc @spacewander @juzhiyuan

@dmsolr
How to exclude config.json from the license check?


SliverFillRemaining flag for different use cases

Description
This PR adds a flag to SliverFillRemaining to specify two different behaviors.

In #17444, the expected behavior is that the sliver will fill the remainder of the viewport and no further.
However, when utilized within NestedScrollView, the sliver is expected to scroll beyond the viewport.

This flag, scrollBody, defaults to the behavior described in the first example, while the NestedScrollView implementation has been modified to set the flag for that case.
See API Docs for SliverFillRemaining
Related Issues
Fixes #17444
Tests

SliverFillRemaining does not extend past viewport by default
SliverFillRemaining scrolls beyond viewport (when scrollBody set)

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[ ] No, this is not a breaking change.

cc/ @goderbauer @dnfield
We discussed a few different ways to implement this offline, this being the most recent.
Also, in this case, I think it may be a breaking change...

Could we make this not be a breaking change (or a less breaking change?) by defaulting the flag to be the existing behavior?

Cirrus seems unhappy :(

Could we make this not be a breaking change (or a less breaking change?) by defaulting the flag to be the existing behavior?

I have changed this so that the default behavior stays true to its current behavior.


windows miniz example

could you share an exemple of how to use miniz with zstr on windows?

Can you create a minimal reproducer on what you want to do (example file, what errors you see when trying to use it, etc.) ?