Progress on NotReady

I was wondering if anyone had mentionend this use-case:
I have a ui and want to load some io resource from network or similar, and I want to report progress of the io.
This would correspond to having some type parameter on NotReady so that information could be passed, and this type would probably default to (). I'm not sure if this is possible in the current future implementation, or if it would be better as a separate crate.

There's a few options I think that may be possible to get this done with today's setup:

If you've got a custom instance of Future you could add a method to detect how much progress is made perhaps and arrange for that method to be called when you'd like to know how much is ready.
You could also make a Stream where the item coming out is an enum of either "done" or "this is how much progress I've made"

Would either of those work for your situation?

How are you running your future? Perhaps the future could communicate the progress to the UI thread before returning NotReady? I think it's common for top level futures to return () and communicate progress through a channel. Your future would hold the Sender and the UI thread would hold the Receiver.

Hi thanks for the responses, it seems like the approach suggested is to implement progress outside of the Future trait, which is fine for me.


Make automatic change of loadMode to 3 after onload configurable

To allow more flexibility, one should be able to decide whether or not the change of loadMode to 3 should happen automatically after onload.
Even though it makes sense to try and load more images if the rest of the page is done loading, there are valid cases to prefer not doing so.
On a very large page with lots of images, where not all users are  expected to scroll down, it would result in unnecessary use of bandwidth for the client, unnecessary number of HTTP requests for the server, unnecessary inflation of electricity bills for the host and unnecessary CO2 emissions for our planet. ☘
To avoid breaking backwards-compatibility, the default of the changeLoadModeAfterPageLoad configuration variable is set to true. The behaviour will be altered only when explicitly setting it to false over the lazySizesConfig object.

First of all I think you are overthinking this and went into a wrong direction. If you disable loadMode entirely it breaks lazySizes, because this statement would never be true.
At the end: If you want to disable loadMode = 3 it is already possible by simply setting expFactor = 1.1. (Note expFactor has to be higher than 1. Otherwise you break the exact same line.

@aFarkas, thanks a lot for the reply.
I'm glad this can be changed in such a way, but I think it is a certainly unintuitive way of configuring it. Feels hacky.
May I suggest a change to add a config variable that just overwrites the exFactor to 1.1 as you state?

You want to add a new configuration that only changes another configuration? So you add more complexity without any functionality.
Changing expFactor is not hacky. It's just a different perspective.


remove the useless code related to the management of the focus in CalculationResult

Fixes #699
Description of the changes:

remove OnLostFocus, OnFocus and OnRightTapped
remove CalculationResult::s_FocusedState and CalculationResult::s_UnfocusedState
stop forcing the focus when users tap the control

How changes were validated:

manually

I added back the explicit call to Focus() in OnTapped() since it is a change in behavior from the current app. We can remove it separately from this code cleanup if it's the right thing to do.


key is a special React property

I understand the intention of using key but since it is a special react property it throws an Error: https://reactjs.org/warnings/special-props.html
Would a pull request be welcome that changes the prop name to accessKey? We may want to change token to accessToken for naming consistency also.

Thanks @trevorgerhardt for pointing that out. Your suggestions is well received and is now incorporated in v2.2.0
P/s: You are always welcome to open a PR. Thanks.


Implement new manifest cache fully capable of multi-platform image building

Follow-up to #2711 to complete the new cache design to support multi-platform image building.

Adds UnlistedPlatformInManifestListException.
No platform verification of a cached manifest any more. getCachedBaseImages() returns cached images only when Jib has cached all the manifests matching the configured platforms.
When caching a manifest list and manifests, PullBaseImageStep also stores additional information of manifest digests. This is to enable looking up a manifest from a manifest list by digest. For example,

{
  "manifestList": {  # manifest list content
    "manifests": [
      {  # manifest descriptor 1
        "digest": "sha256:a157906...",
        "platform": { ... },
        ...
      },
      {
        "digest": "sha256:...",
        "platform": { ... },
        ...
      }
    ]
  },

  "manifestsAndConfigs": [
    {
      "manifestDigest": "sha256:a1579064....",  <-- manifest digest for lookup
      "manifest": { ... },
      "config": { ... }
    },
    { ... },
    ...
  ]

Ready for review. See the PR description for high-level changes.


Bump note-c to latest master.

This looks great, but I'm going to pile on an example fix.

 [note-arduino] Existing Template Example Uses Undocumented Argument


Downgraded k8s libraries to fix spew of errors on pre 1.20 clusters

See https://github.com/neoskop/cert-manager-webhook-dnsimple/commit/11fb703e1ecf10ae9ad69560abc1c80b3b9373ab#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6
and https://github.com/neoskop/cert-manager-webhook-dnsimple/issues/8 for details

Hi @istyf!
As you can see the test failed...
The reason for this is that the build pipeline in Azure doesn't have access to secrets on pull requests. This is this is a public repository and anyone can make a pull request with malicious code to extract the secret if I would allow it access to secrets.
This is my first open source project on GitHub and my first with CI so I lack all the experience. For example my pipeline includes build to Docker hub and that could f**k up things up for people that uses the latest tag for their Kubernetes deployments so luckily the build failed even though I don't think anyone uses it yet except me and maybe you. :-).
Before I can merge this I need to rebuild the pipeline, make sure it builds first. I might also register a free LoopiaDomain account and add a cheap domain to it just for testing and then enable PR's access to the secrets.
Any advice, is this something you're experienced with and in that case maybe this can be moved off Azure Pipelines!?

Sorry for abandoning this PR for a while.
It seems to me that maybe your pipeline would benefit from being divided into smaller steps? A first step could be to just build the code and tag an image and then maybe have an extra release pipeline that runs more extensive tests. Setting up a dedicated domain for running end to end tests all the way to Loopias API on each PR seems a bit excessive to me.
Have you considered something like contract testing? It could be possible to abstract away Lets Encrypt and Loopia and have the tests assert that the Webhook behaves correctly without requiring access to any external services. Builds from a develop branch could use contract testing for quicker feedback, while release builds from the main branch could use a more complete setup with proper end to end testing and verification.
Pulling images into production using the latest tag is considered a bad practice, so if anyone would be doing that and a PR build ended up in their cluster it would be just what they asked for. ;)

I'm sorry too, however I've started to rebuild the CI/CD pipeline on GitHub Actions that offer better security but currently I have some problems for the test to succeed and I have no clue why but I do hope I will soon :-).
I've chosen to keep the conformance test supplied by (and requested by) the cert-manager team but I'll run the more extensive test they supply on deploy that will be triggered from a tagged push event. Except for deploy to Docker Hub I might also use GitHub pages to act as Helm repository that makes deployment easier.

Hi @istyf!
I have now made a lot of fixes and I included your downgrade of binaries...
Changelog (main changes):

Subdomain is now deleted correctly.
Helm-chart is published to GitHub pages
CI/CD is now using GitHub Actions
Downgraded k8s libraries to fix spew of errors on pre 1.20 clusters

Closing this PR since changes are already made.


Flutter app giving errors during build saying that - Execution failed for task ':app:processDebugResources'

Following is the full error messages that I receive on app crash.
/Users/adityas/.gradle/caches/transforms-1/files-1.1/core-1.0.1.aar/ebc616362c7a8fb07eb4795d71e6ed37/res/values/values.xml:133:5-70: **AAPT: error: resource android:attr/fontVariationSettings not found**.

/Users/adityas/.gradle/caches/transforms-1/files-1.1/core-1.0.1.aar/ebc616362c7a8fb07eb4795d71e6ed37/res/values/values.xml:133:5-70: **AAPT: error: resource android:attr/ttcIndex not found.**

         *********************************************************
WARNING: This version of firebase_messaging will break your Android build if it or its dependencies aren't compatible with AndroidX.
         See https://goo.gl/CP92wY for more information on the problem and how to fix it.
         This warning prints for all Android build failures. The real root cause of the error may be unrelated.
         *********************************************************

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:processDebugResources'.
> Failed to process resources, see aapt output above for details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 4s
Finished with error: Gradle task assembleDebug failed with exit code 1

Please help me debug this issue. I am not able to build my app, not even for once.
I happened suddenly around 1 week ago.
Please suggest me a workaround or a reason as to why it may be happening.

i am facing the same issue, i have tried changing compileSdkVersion to 28 also minSdkVersion to 21.
but nothing work.

Changing compileSdkVersion and minSdkVersion is not enough.
See the link for example in https://pub.dartlang.org/packages/firebase_core#030 also migrate

Nope, still giving errors.
I by the link mentioned by @zoechi I have added these lines to my gradle.properties file
android.useAndroidX=true
android.enableJetifier=true

I am also using version 2.4 of the firebase_core plugin.
But still, the error is intact.
Can you please suggest something else, which could work.
Any help would be appreciated.

@aditya234

I am also using version 2.4

what for?
The latest version is 0.3.0+1. Versions before 0.3.0 do not support AndroidX.

@zoechi
I am now using the latest version 0.3.0+1 of firebase_core plugin.
But the issue is still intact.
Can you please give me an insight into "Why this error keeps coming?"

You need to upgrade all your plugins.
Did you check the link printed in the error message? (https://goo.gl/CP92wY)?
I'm going to close because this is essentially a support request that should be asked in one of the other channels listed in http://flutter.io/support .

Okay
Thanks for your cooperation


Flag usage recommendations

In PR https://github.com/ossf/wg-best-practices-os-developers/pull/132#issuecomment-1505292993 @david-a-wheeler noted that we need to note somewhere that the option flags are order-sensitive. Another feedback from the C/C++ Compiler BP Guide calls was to add the recommendation to to explicitly enable hardening flag in a projects build system rather then relying on flags configured in the local GCC profile. This PR adds text to address these concerns.

Updated PR with rebase with respect to main after renaming files

Minor further tweaks to avoid referring to only GCC.

Squashed requested changes to original commit.

Merged as per discussion in C/C++ Compiler Guide BP call 2023-08-02.


Implement all missing unit tests - part 1

Increase TwingParser CC to 100%
Increase TwingTemplateWrapper CC to 100%
Increase TwingTemplate CC to 100%
Increase TwingTest CC to 100%
Increase TwingTokenStream CC to 100%
Increase TwingToken CC to 100%
Increase TwingReflectionObject CC to 100%
Increase TwingReflectionMethod CC to 100%
Increase TwingOutputBuffering CC to 100%
Increase TwingNode CC to 100%
Increase TwingNodeVisitor CC to 100%
Increase TwingLexer CC to 100%
Increase TwingFunction CC to 100%
Increase TwingFilter CC to 100%
Increase TwingFileExtensionEscapingStrategy CC to 100%
Increase TwingExtension CC to 100%
Increase TwingExtensionSet CC to 100%
Implement interfaces "impl" symbols
Re-add interfaces to nyc covered sources
Increase TwingExpressionParser CC to 100%
Increase TwingError CC to 100%
Increase TwingEnvironment CC to 100%

Coverage increased (+3.4%) to 94.109% when pulling defd634e1b8ee1e121d1ef7df7e04b20ed208531 on issue_163 into fad1b362ae3459321f3b5bf9f63e27f01a16b118 on master.


make Where queryHash uniquer

I'd like to filter on specific values within a json column.
The column contains translations and I wan't to create af filter per suppported locale.
Content of the column
{
	"en": "Howdy!",
	"fr": "Bonjour",
	"nl": "Hallo"
}

filter code (simplified):
Since the closure is in a loop, the queryHash (based on location of the Closure) is the same for all filters. Adding the label makes is unique and solves the problem.

// JSON case insensitive search
foreach (config('test.available_locales', []) as $locale => $info) {
    $filter->where(
        function (Builder $query) use ($locale) {
                $query->whereRaw(
                    '`value`->\'$.'.$locale.'\' COLLATE utf8mb4_unicode_ci like ?',
                    ['%'.$this->input.'%']
                );
        },
        'Value ('.$locale.')'
    );
}

@z-song  done.


Auto-Complete Required Request Body Parameters when Inserting Commands

Instead of returning an empty request body when inserting commands from the side panel (like it does now), we can return the required fields in the request body. This will make the commands panel truer to its use, ie. making it easier for the developer to find and write requests.
they can just insert it from the commands panel and quickly fill in the request body values, and if they want to customize it more, the DOCS button will always be there!

Preview: (wait for the GIF to load)

Exceptions: This won't auto-complete request bodies that can have multiple options for a single parameter as shown below (wait for the GIF to load)

This should be the preferred way as well. Inserting any one of the possible parameters might limit the user to just one operation, so it's better to encourage them to refer to the documentation.

Thanks @Snehil-Shah, we will include this changes into the next release!


Fix bugged description for Composite trait

Fix the bug that caused "Composite" trait to not show a proper description.
Bug was caused by omission of "PF2E." in the map in traits.ts.
Fixes #9102


[ENG-3245] Update broken help guide link

Ticket: ENG-3245
Feature flag: n/a

Purpose
Update the help guide link on the project registrations page to one that is not broken.
Summary of Changes

Update the link in both places it's used.

Side Effects
Nope
QA Notes
Make sure the new link goes to a help page for registrations.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 71.912%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
4769

Relevant Lines:
6360

💛  - Coveralls


ResideMenu-LAYOUT_DIRECTION_LOCALE

I want to implement reside menu in"Arabic" language but it is not opening menu according to Locale.
it works only for LEFT and RIGHT direction.
Please suggest how to implement this?

This is China's training institutions of IT in order to make money, is copied to his project.


[modify] opendata/resource validate_and_destroy

Coverage increased (+0.1%) to 79.161% when pulling cd225189e04651f5bec9652cc15fdcb1c1bc94e0 on ShinjiTanimoto:fuseki into 874237bbf1f685fcf5c6937cc395781f5fcbbc07 on shirasagi:master.

LGTM


Ignore non-existent refs in the transformed DWARF

When used with the '-g' flag, wasmtime fails on more complex file. We don't have robust detection of non-used data so we just rely on DW_AT_ranges and DW_AT_low_pc attributes. It is possible we are loosing some of the info. But for now we want wasmtime not to fail -- the patch addresses the failure to "link" to the dropped entries.

/cc @SanderSpies (test file https://github.com/SanderSpies/ocaml/commit/b71f011c60afa9dffc8333090e12f11e1583f4bb)

Looks good.


Improve OutputParserException handling by adding observation and sendToLLM

Inspired by the same OutputParserException in python library, used the LangChain Python to JS translator to translate some code, that was pretty cool. This also keeps the existing ordering of the constructor params to avoid changes to existing code using the constructor.
Python library code:
https://github.com/langchain-ai/langchain/blob/96023f94d9db48b4ca3d6849711f030010c2454b/libs/langchain/langchain/schema/output_parser.py#L326-L362

Looks great! Thank you!


Added the missing full-stop

What type of PR is this? (check all applicable)

[ ] ♻ Refactor
[ ] ✨ New Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 👷 Example Application
[ ] 🧑💻 Code Snippet
[ ] 🎨 Design
[ ] 📖 Content
[ ] 🧪 Tests
[ ] 🔖 Release
[ ] 🚩 Other

Description

This PR [adds/removes/fixes/replaces] this [feature/bug/etc].
Related Tickets & Documents

Resolves #
Mobile & Desktop Screenshots/Recordings

Added code snippets?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

No tests? Add a note

Added to documentation?

[ ] 📜 readme
[ ] 📜 contributing.md
[ ] 📓 general documentation
[ ] 🙅 no documentation needed

No docs? Add a note

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

I think I have resolved the issue help me to see if it will work now and thank you

wow nice could help to merge the PR so That I would have a successful merged PR for hacktoberfest and proceed to solving the sidebar issue. I made a mistake by doing both of them together since I would be needing to get up to 4 merged PR for hacktoberfest

@philling1 Hi, thank you for working on this. I recently saw on Hacktoberfest's website that they won't be accepting "typos, and whitespaces" as a form of contribution. However we can accept it and add you to our Hacktoberfest leaderboard.  The first 20 people to successfully merge a PR will receive free swag from our TBD shop.
Your sidebar issue will be one that aligns with Hacktoberfest's official guidelines.


Unable to initialize the OpenManipulator-x

Running Ubuntu 20.04 on vmware
ROS2 Foxy
Hi I am a totally beginner of dynamixels and ros2
I have a couple of XM430-W210 for Turtlebot and five motors of XM430-W350, trying to build my own robot arm with the OpenCR
I have installed the ros2 foxy and the OpenCR but failed to start the controller:
$ ros2 launch open_manipulator_x_controller open_manipulator_x_controller.launch.py usb_port:=/dev/ttyACM0
[INFO] [launch]: All log files can be found below /home/leo5017/.ros/log/2023-09-05-19-57-11-076641-ubuntu-14683
[INFO] [launch]: Default logging verbosity is set to INFO
[INFO] [open_manipulator_x_controller-1]: process started with pid [14685]
[ERROR] [open_manipulator_x_controller-1]: process has died [pid 14685, exit code -11, cmd '/home/leo5017/colcon_ws/install/open_manipulator_x_controller/lib/open_manipulator_x_controller/open_manipulator_x_controller /dev/ttyACM0 1000000 --ros-args -r __node:=open_manipulator_x_controller --params-file /home/leo5017/colcon_ws/install/open_manipulator_x_controller/share/open_manipulator_x_controller/param/open_manipulator_x_controller_params.yaml'].
I try to connect to the servos with the Dynmixel Wizard 2.0, they work fine.
and I also tested with the read_write_node example, they also work fine.

Hi,
Thank you for reaching out. Please note that we do not officially support running on a virtual machine like VMware, as it can sometimes cause unexpected issues with hardware communication. We recommend running ROS2 Foxy and the OpenManipulator-X software directly on a native Ubuntu installation instead of using a VM. This approach should help eliminate potential compatibility or performance issues caused by virtualization.
If you encounter further problems after switching to a native setup, feel free to provide additional details, and we'll assist you further!


Expand all test cases

Currently functions are tested individually in user.cpp. The results of these test cases are then briefly checked for correctness. Instead of using random numbers, manually create tests for each function that will automatically check for correctness.

Mostly caught up. The only ones to write now are solveSystem()'s jacobi() and failSafe() methods. Both require overloading Vector's == operator. I'll close this issue once test cases are written for all three, and then just add new ones with each major additional function I add.


Search pods by name

User story

As a user
I want / need search pods by their name
So that I can easily find them

Acceptance criteria

[ ] Results are displayed in a list view
[ ] Search is triggered only after pressing enter or the corresponding icon
[ ] The search is always queried against the database
[ ] The search field shall be on the top right, right on top of the list
[ ] The search field is displayed inside the "All Pods" view

Definition of done (DoD)

Colours match with color scheme
Implementation is unit tested
Code documentation updated (jsdoc or godoc?)
Changes to database documented at Confluence- Database scheme
Changes to architecture documented at Confluence - Architecture

DoD general criteria

Feature has been fully implemented
All acceptance criteria were met
Product owner approved features
All tests are passing
Pull request to "develop" branch is merged & closed
Bill of materials is updated with new dependencies

Ali already started implementation, seems almost done


You can't past number with Three Decimal Digits (or more ) in IOU page - reported by @ahmdshrif

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

Action Performed:

Copy number with 3 dismissal digits e.g. 55.333
Navigate to IOU request
Paste the value copied

Expected Result:
User should be able to paste the number by keeping 2 decimals
Actual Result:
Value is not pasted at all
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.40-0
Reproducible in staging?:  Y
Reproducible in production?:  Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/155665063-1a675698-3309-484d-8b2c-e179e27d45bc.mp4
Expensify/Expensify Issue URL:
Issue reported by:  @ahmdshrif
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1644795303142109
View all open jobs on GitHub

Triggered auto assignment to @ctkochan22 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

This seems super edge case. If the user types in the number, I assume it stops at 5.33. But I suppose its worth fixing? Downgrading to monthly and looking when I have time

Interested. Will look later this month

Punting

@mvtglobally If someone reports this again, please label as not an issue.

Hi  @ctkochan22  do i will be eligible for report bonus here ?
Can you please add external lable

I don't think so? Because its not a bug, but let me ask.

Thanks @ctkochan22 for response.
But As i remember Roray say it's BUG we should fix when i report this in slack.
and  then you decide we shouldn't do any thing Because it's minor case. And based on some logic we have in code . (For me still see it bug at least we should show  messas to user to let he  know what is the error. but i agree withe you  it's minor case )
That's why i think we should add external lable to discuss this case with the CM.

Sure we can discuss.

The issue here is that whenever a user inputs something that isn't a number in this case, we don't update the state. When a user types in 1, we validate that input then update the state.amount to 1. When the user then types in a, we check the character, identify that its not a number, then we do not update the state.amount to 1a. We set the state to the previous value 1.
I don't think this is a bug. My reasoning is thinking about the case where a user accidentally has a string in their clipboard. So they try and paste the value<EMAIL_ADDRESS>into the amount field. I think the expected result is to not past any string values into the field.
In the rare case that a user copy/pastes a whole amount. In this case, three digits instead of two 22.123, we get the input. See that its an invalid amount. Then we update the state with the previous value, and the previous value is empty. I think it works as it should. I don't think we should arbitrarily cut the value to two decimal points. Nor do I think we should round up or down.

@mallenexpensify let me know what you think

Weird, I thought I commented. I def read (then re-read to try to make sure I understood everything).
I'm on board with your reasoning @ctkochan22 , I hadn't even tried to paste numbers or letters into the amount field before.
@ahmdshrif can you test to see how it works for a currency with more than 2 decimal places? I wasn't able to update my default currency (possibly because I'm on the Expensify policy?).  Here's a list of decimal places by currency https://docs.adyen.com/development-resources/currency-codes

@mallenexpensify Same thing with the currency with 3 decimal place .
https://user-images.githubusercontent.com/21364901/172261767-f28308fe-00db-4981-b678-d31d490fb804.mp4

Thanks for the vid @ahmdshrif, what I was wondering was, if it's a currency with three decimal places, that a user would be able to paste in a number with 3 decimal places.  I'm assuming, in your vid, that you were trying to paste in a number with more than three, is that correct?
For payments for bugs and feature requests, for the most part, like this issue, code has to be deployed to production in order to be compensated.
https://github.com/Expensify/App/blob/main/CONTRIBUTING.md#:~:text=Payment will be made 7 days after code is deployed to production if

Thanks for the vid @ahmdshrif, what I was wondering was, if it's a currency with three decimal places, that a user would be able to paste in a number with 3 decimal places. I'm assuming, in your vid, that you were trying to paste in a number with more than three, is that correct?

@mallenexpensify  no, I try to paste a number with 3 decimal places 1.333   and the currency is  KDW.
the current code only accepts 2 numbers with 2 decimal places when you enter in by keyboard or past it.

For payments for bugs and feature requests, for the most part, like this issue, code has to be deployed to production in order to be compensated.

ok, I just asking about it.

ah ha! Then that is a bug.  If a currency accepts more than two decimal places then the user should be able to paste in, up to, the max number of decimal places for each type of currency.
@ctkochan22 I think we should fix/update this, do you think it can be worked on externally? If so someone will first have to do an audit of the different currencies.

I can do this this week

Hmm we don't even allow triple digits on oldDot for these currencies.
But I don't think we can do this externally, we should add this as a constant in /expensify-common

If we can't do this Externally, will you add the Internal label? I'll stay assigned to issue payment to @ahmdshrif if the issue ever gets fixed.

I don't think this is super pressing. I am hoping to do this still

@adelekennedy assigned you as CM to issue payment to @ahmdshrif if we fix this internally

Lets bump this to a monthly. Unless we are actually seeing a lot of customers dealing with this issue

not a priority atm

Slapping a not a priority. Let me know if you all think otherwise.

@ctkochan22 wanna unassign and open up to the pool?  I think we should consider fixing this at some point, although it's not a priority

Still pending on fixing this but will remit payment if we do!

@ctkochan22 what do you think of Matt's comment here?

Sure, i'll unassign. Sorry about that

Who ever works on this, you'll have to figure out where we grab the currency list and references to in expensify-common. And work with that there

Where are we at here? We've opened this up to anyone who wants to work on it (with the caveat we'll pay out to @ahmdshrif if we fix internally) @mallenexpensify should we make this an external issue again?

https://github.com/Expensify/App/issues/7911#issuecomment-1149491041
It appears to need to be fixed internally, I'd leave it a monthly to see if anyone picks up. You should stay assigned @adelekennedy , in case it does get fix, @ahmdshrif will be due compensation if so

But I don't think we can do this externally, we should add this as a constant in /expensify-common

@mallenexpensify I think @ctkochan22 thinks we can't fix it externally because of changes need in /expensify-common .
but I already contribute to this repo on 2 issues I get assigned for (as an external contributor )
https://github.com/Expensify/expensify-common/pull/436
if that's the only reason we can move it to external.

Thanks @ahmdshrif , asking internally for 👀

@mallenexpensify Can you link the slack thread? Any progress here?

Blast from the past!  Sorry for leaving you hanging @adelekennedy .  I did check here, I'm going to close this because we currently don't support more than 2 decimals for currencies and it hasn't been a problem so far.  Maybe we'll address sometime down the line


Added the function to consume bearer token from apidApigeeSync.

@theganyo @alexkhimich @pjain404 Please code review. This is the consumer of apidApigeeSync change.

I'm ok with the change

This is not valid anymore as the token is being set in a config now.


Connection Error

Issue Type: Bug
I am getting below error after upgrading to Azure Data Studio version 1.41. I tried both MSAL and ADAL connection method but issue still persist. I tried deleting settings but no luck. This was working in previous version of Data Studio.
Failed to get Azure account token for connection
Azure Data Studio version: azuredatastudio 1.41.0 (cf5e0fe1a33709e2a82dc94f42dbe88487803d62, 2023-01-24T17:25:45.514Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No
Preview Features: Disabled

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11370H @ 3.30GHz (8 x 3302)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.84GB (3.45GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (23)

Extension
Author (truncated)
Version

sqlops-combine-scripts
Bat
2.0.1

eltsnap-simple-data-flow
bit
1.0.1

admin-pack
Mic
0.0.2

admin-tool-ext-win
Mic
0.0.4

agent
Mic
0.49.0

azcli
Mic
1.7.0

azuredatastudio-oracle
Mic
0.1.2

cms
Mic
0.9.2

dacpac
Mic
1.11.0

datavirtualization
Mic
1.12.0

import
Mic
1.5.4

managed-instance-dashboard
Mic
0.4.2

net-6-runtime
Mic
1.1.0

powershell
ms-
2022.7.2

profiler
Mic
0.11.0

query-history
Mic
0.5.3

schema-compare
Mic
1.17.0

sql-database-projects
Mic
0.20.0

sql-dw
Mic
0.0.1

sql-migration
Mic
1.2.0

extra-sql-script-as
pac
0.5.0

schema-visualization
R0t
0.8.2

simple-data-scripter
sea
0.1.6

1-Azure Accounts-attach.log

Hi @vikarBCC
It seems you're using ADAL when logs were captured, however there is no concerning error in logs I see. Can you try to run Command Palette (Ctrl + Shift + P) > 'Azure Accounts > Clear Azure Account Token Cache' command, followed by reloading ADS to clear any cache related issues? You may need to refresh your credentials, so please do the same.
Eventually, you should try moving to MSAL, following same steps to clear cache and reauthenticate as requested.
You will get much extensive logging with MSAL so please gather logs the same way and let us know if you see any exception.

Hi @vikarBCC
It seems you're using ADAL when logs were captured, however there is no concerning error in logs I see. Can you try to run Command Palette (Ctrl + Shift + P) > 'Azure Accounts > Clear Azure Account Token Cache' command, followed by reloading ADS to clear any cache related issues? You may need to refresh your credentials, so please do the same.
Eventually, you should try moving to MSAL, following same steps to clear cache and reauthenticate as requested (Reauthentication is needed when switching library).
You will get much extensive logging with MSAL so please gather logs the same way and let us know if you see any exception.

[Verbose]: MsalCachePluginProvider: Using cache path C:\Users****\AppData\Roaming\azuredatastudio\Azure Accounts and serviceName azureTokenCacheMsal-azure_publicCloud - []
[Information]: Fetching saved token - []
[Error]: No access token found - []
[Information]: Fetching saved token - []
[Error]: No access token found - []
[Error]: User had no base tokens for the basic resource registered. This should not happen and indicates something went wrong with the authentication cycle - []
[Error]: Error refreshing access for account Vikar Ghory -<EMAIL_ADDRESS>- ["{\n  "originalMessage": "No base token found"\n}"]
[Verbose]: Starting login - []
[Verbose]: Fetching token for tenant {0} - ["common"]
[Verbose]: Fetching tenants with URI: {0} - ["https://management.azure.com/tenants?api-version=2019-11-01"]
[Verbose]: Tenants: XXX - []
[Verbose]: Token Claims acccount: xxxx xxxx, TID: a47f8d5a-a5f2-4813-xxxx-xxxxxxxx - []
[Verbose]: Tenant ID: a47f8d5a-a5f2-4813-xxxx-xxxxxxxxx, Tenant Name: xxxx - []

MSAL log attached. I am using MFA but the prompt doesn't open dialog to enter Authenticator generated OTP
1-Azure Accounts-clean.log

I think you accidentally clicked on 'Ignore Tenant' on one of the popups, as I see this in logs:
Tenant a47f8d5a-a5f2-4813-a71a-f0d70679e236 found in the ignore list
Please remove this from ignore list by clearing this setting from settings.json:
    "azure.tenant.config.filter": [
        "a47f8d5a-a5f2-4813-a71a-f0d70679e236"
    ],

I tried removing tenant filter, still getting same error.

Hi @vikarBCC
I hope you've restarted ADS after removing the setting, but if you're still experiencing issue, please upload Azure account log to identify reason for failure.

Hi @cheenamalhotra, Yes, I restarted and I believe I already uploaded account log.
I managed to make it work by manually adding below in settings.json
"datasource.connections": [
{
"options": {
"connectionName": "abc",
"server": "abc.database.windows.net",
"database": "db1",
"authenticationType": "AzureMFA",
"user"<EMAIL_ADDRESS>"password": "",
"connectTimeout": 30,
"encrypt": "true",
"trustServerCertificate": "false",
"applicationName": "azdata",
"azureTenantId": "a47f8d5a-a5f2-4813-xxxx-xxxxxxxxxx",
"azureAccount": "xxxxxxx-xxxx-xxxxx-xxxxxxxxxxxxxx",
"expiresOn":<PHONE_NUMBER>,
"originalDatabase": "db1",
"groupId": "a7f0712e-1aad-4bc0-92ca-bec923c3972c",
"databaseDisplayName": "db1"
},
"groupId": "a7f0712e-1aad-4bc0-92ca-bec923c3972c",
"providerName": "MSSQL",
"savePassword": true,
"id": "efd8546b-45fd-4e2c-86e5-e08e66d90ba9"
}
]
I have 2 accounts admin and non admin. ADS was working fine when I start it under admin account but didn't work under non admin account. I got above settings from my admin account and copied and pasted in settings.json of my non admin account after replacing admin account details with non admin account.


🛠 [TASK] : Temporary Deployment of Flutter Web Application to Path /m4

Description
We temporarily deploy our Flutter web application to a sub-path URL (e.g., example.com/m4) while we set up distinct environments for development, QA, and production. This deployment will allow us to continue development and testing on a different URL path until our environment structure is fully established.
Tasks

Ensure that routing within the Flutter app correctly handles the /m4 sub-path.
Test the application to verify it works seamlessly from the example.com/m4 URL path without affecting production.

Acceptance Criteria

The Flutter web application is successfully deployed and accessible via example.com/m4.
The app works correctly with all settings and routes functioning under the /m4 sub-path.
No disruptions to the production environment (example.com) occur due to this deployment.

Notes
This is a temporary solution. Once separate environments for development, QA, and production are set up, we will discontinue using this sub-path deployment.
part of #726

FYI @dtscalac @damian-molinski @digitalheartxs


hmetis package

Motivation for this change
add hmetis package: hypergraph partitioning software needed for optimization problems
Things done

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

[ x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Now just squash all commits into one and name it "hmetis: init at 1.5"

It's important to know how to squash commits, please look up how to do that and continue this PR here instead of opening a new one (#42843). You can ask me if you have trouble with git.

I had such trouble that I deleted my fork and recreated it. I am not sure what to do to comply with your request, but the new fork by the same name now has one commit with everything.

Eh alright, feel free to ask next time if you struggle with git though ;). You should close this PR then.

Also, the one merging it could've done the squash for you.


Fix @azure/arm-resourcegraph still being track 1

TODO:

[x] Test with custom locations: works except still blocked by https://github.com/Azure/azure-sdk-for-js/issues/20570

Hooray for the death of T1!


Prerendering failed because of error: Error: Cannot find module '/../bin/Release/netcoreapp1.1/publish/ClientApp/boot-server'

I tested deploying this but gave me the command line error cannot find boot-server. My browser shows this:  An error occurred while processing your request.
I'm using Mac with environment setting ASPNETCORE_ENVIRONMENT=Production

I have the same issue, this template works great in development but when you publish it, it doesn't work...

It's probably this here: https://github.com/MarkPieszak/aspnetcore-Vue-starter/commit/ef6ce477b5649695e45d41922925ba4d9e484f48
Can you just remove that line, and it should fix it. Let me know!
@alvirtuoso @elucidsoft

Yes, that was it...Works now on Azure Web

But I also had to change global.json to point to sdk: version 2.0.0 or I was getting some really weird errors.

Can you make a PR to update that in here as well to help others? Not by a computer at the moment. I'm glad we figured it out though!


Bump Dependencies #3561423321

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/3561423321

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove lr-insert-fake-data command

We no longer need to insert fake data in the address_base table as we now get
our data from elastic search.

We still want insert fake data! just not for the property-api :(

also maybe remove the creation of the database for property-api? :)


Release/azure communication phonenumbers/1.0.11

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check for com.azure:azure-communication-phonenumbers
API changes have been detected in com.azure:azure-communication-phonenumbers. You can review API changes here

The CHANGELOG and POM changes look good. I think we can go ahead and release from the release branch.


Adds more e2e tests for the Question page

Adds more E2E tests for the Question Page:

reset feedback
switching theme
links display
responsive UI

Replaces checkForConsoleError in utils.js by expectNoConsoleLogs: All log levels should be captured by the tests and no console output should be expected.

Adds more utility functions for responsive UI testing

Fixed ambiguous param name for questionPage.get()


If a waypoint is linked, show the name in the gui

As far as I can tell the only way to see which name it corresponds to is looking at the config file. it would be nice to have it with the waypoint information for easier diagnostics. Kids hooked up one to the wrong name and it wasn't immediately obvious until we moved the wrong point.

Hovering over the link icon will show the name


Not able to use any Ng commands

[Enter feedback here]
I need to add a new component for the Add-in however if I try adding a new component, it doesn't work as it is "not available outside an angular workspace. Is there a different way to add a new component?

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

ID: 15f43fe5-03e0-579c-165c-57e16b1a53ea
Version Independent ID: 15f43fe5-03e0-579c-165c-57e16b1a53ea
Content: Create Office Add-in projects using the Yeoman Generator - Office Add-ins
Content Source: docs/develop/yeoman-generator-overview.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

This seems to be a problem with the one of the Angular templates in Yo Office, rather than documentation. I can move the issue to the proper repo, but first tell me if you took the TypeScript or JavaScript option.
@igor-ribeiiro Could you take a look at this?

 There is a PR already open to fix this issue: https://github.com/OfficeDev/Office-Addin-TaskPane-Angular/pull/93

@Viktor1356 It might be some time before we can update our Angular templates. In the meantime, you might consider trying to create a Angular Office Add-in with one of the methods described in these two blog posts:

Building Office Add-in using Angular 8+
Developing Office add-ins using Angular 10+ and Nrwl Nx


ExperimentalScanner fixes

Some fixes for the experimental scanner, see the commit messages for details.

FYI @mnonnenmacher, we currently have issues with the Markdown check / GitHub's checkout action, so that test is expected to fail.


Smart Text Color re-triggering includes the element itself to the reference image

Bug Description
When generating an image for contrast calculation, the text element itself is also included into the image, that causes unexpected and inaccurate results.

Expected Behaviour
The element itself should not be included into the generated image.

Steps to Reproduce

Add a text element on a white background
Click on the "Magic wand" button for auto-styling.
The text should stay black as it is but it gets a highlight instead.

Screenshots

Additional Context

Plugin Version:
WordPress Version:
Operating System:
Browser:

Verified in QA


Handle partial GraphQL errors

Simple workaround for Relay bug (see: https://github.com/facebook/relay/issues/1913).
I have no idea how to write tests for this part of application.
I would prefer to merge this and think about it later since we will
need to refactor it a little bit because of other tests...
Related issue: #98

Checklist:

[ ] it works in landscape and portrait mode
[ ] it works offline (if possible) otherwise cacheConfig.force=true is used

Please test these changes in Expo after build has finished:

exp://exp.host/@mrtnzlml/code-review-handle-partial-errors

I also noticed that we create a new relay environment for every call of commitMutation and in the render method of QueryRenderer. Isn't the environment supposed to be a single instance for the whole application?

Sounds like a mistake but that's not relevant here. I should fix it though. Probably during refactoring of this section, not now. You can create a new issue...

Issue created #101


Use nix eval instead of nix-instantiate for gomod2nix import

Fixes https://github.com/nix-community/gomod2nix/issues/72

What are you trying to do which isn't possible with the stable CLI interface? I'm extremely reluctant to rely on the 2.4+ commands as they haven't been the most stable and have weird assumptions around flake usage that keeps changing.

The existing single nix-instantiate call does not seem to work. However, using the nix command isn't necessary.
It seems as though running nix-instantiate --expr ...; nix-instantiate --eval --expr ... would also work.
My reasons for proposing nix eval --impure --expr ... over the two calls to nix-instantiate is that:

It's shorter (single exec vs two)
It doesn't produce an error when it succeeds (nix-instantiate will complain without --eval, but it appears necessary to run it without --eval first)

It may be possible to make it work with a single invocation of one of the older Nix commands (possibly adjusting the provided expression?), but I haven't really tried.
If the new commands are still deemed to be too experimental (or insufficiently available/you explicitly want to support older Nix versions), then opting for the double/back-to-back exec would work (if you don't mind the error: expression does not evaluate to a derivation (or a set or list of those) error).

Hey, so I ran into this issue when spinning up a new project with gmod2nix and applying the patch that @jjthiessen created did resolve it... Now that the command is in nix stable can we merge this into the project?


PanelOLS with two-way fixed effects / double clustered std. errors for a panel of millions of observations

Hi, I have come across a performance issue in regards to two-way fixed effects and double clustered standard errors. So essentially I would like to use both entity and time effects as well as double clustered errors in PanelOLS. My panel is unbalanced with about 10 million observations in total, ~5000 entities, ~2000 time steps and say about half a dozen independent variables.
When running this setup in PanelOLS I keep having trouble with the Python process memory blowing up, ultimately resulting in the Python process being killed by my machine. I suppose this should be replicable using other data sets. For now, please see below a minimal example with a generic set of data available here (sorry for the large file, but this appears to be a large data issue):
https://www.dropbox.com/s/u9mzwdlyqp1ylu3/LargePanel.csv?dl=0
import pandas as pd

from linearmodels import PanelOLS

data = pd.read_csv('LargePanel.csv')
data['Time'] = pd.to_datetime(data['Time'])
data.set_index(['EntityNo', 'Time'], inplace=True)

mod = PanelOLS(data['Y1'], data[['X1', 'X2', 'X3', 'X4']],
               entity_effects=True, time_effects=True)
res1 = mod.fit(cov_type='clustered', cluster_entity=True, cluster_time=True)

I would be interested to learn whether there is a way to make the procedure more memory / computationally efficient.

Short term solution is to use one of entity_effects=True and pass the time effect as an other_effects value.  This triggers a somewhat slower approach that never builds dummy variable matrices. But it also avoids lots of memory allocation.
Longer term solution is to automatically switch to the slow method when the dimensions of the effects is large, and to expose a user option to override the automatic choice.

Thanks!

This fixed in master now.  Can use low_memory=True in fit or it will automatically use low memory if the dummy variable matrix is > 1GiB (which yours easily is ~10M by 4000 = 320GiB)


TokenTree moved to its own module tokenstream (Closes #260)

Thanks! I'll upload to crates.io shortly.

regex_macros 0.1.38 is on crates.io.


[Sprint: 48] XD-3003 Adding support for Sqoop's metastore and job command

job --exec requires the use of --password-file or the job will prompt for a password and execution will hang

The metastore must run in PostgreSQL and I recommend creating the tables manually:
CREATE TABLE
    SQOOP_ROOT
    (
        version INTEGER,
        propname CHARACTER VARYING(128) NOT NULL,
        propval CHARACTER VARYING(256),
        UNIQUE (version, propname)
    );
CREATE TABLE
    SQOOP_SESSIONS
    (
        job_name CHARACTER VARYING(64) NOT NULL,
        propname CHARACTER VARYING(128) NOT NULL,
        propval CHARACTER VARYING(1024),
        propclass CHARACTER VARYING(32) NOT NULL,
        UNIQUE (job_name, propname, propclass)
    );
INSERT INTO sqoop_root (version, propname, propval) VALUES (null, 'sqoop.hsqldb.job.storage.version', '0');
INSERT INTO sqoop_root (version, propname, propval) VALUES (0, 'sqoop.hsqldb.job.info.table', 'SQOOP_SESSIONS');

Other than that, looks good, merging.

Merged per https://github.com/spring-projects/spring-xd/commit/e6f825801ed69aa6ee5baec656a883e07d714e71, closing.


set static_token as not required input, updates descriptions to show you can optionally use environment variables

sets static_token input as not required
updates description for token and static_token
removes extra space in action's description

I noticed this because the GitHub Actions VS Code extension was complaining that I was missing the static_token input when I had defined the action call like so (as shown in the readme):
- uses<EMAIL_ADDRESS>  env:
    CODECOV_STATIC_TOKEN: ${{ secrets.CODECOV_STATIC_TOKEN }}
    CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

oops opened this in the wrong repo...


docker构建镜像报错

ERROR: failed to solve: process "/bin/sh -c go build -ldflags "-s -w -X 'one-api/common.Version=$(cat VERSION)' -extldflags '-static'" -o one-api" did not complete successfully: exit code: 1

你是什么环境呢？参考本项目 CI 的配置，打包是一直可行的

ERROR: failed to solve: process "/bin/sh -c DISABLE_ESLINT_PLUGIN='true' REACT_APP_VERSION=$(cat VERSION) npm run build" did not complete successfully: exit code: 1一样的问题


concurrent map read and map write

some times,it will throw "error:concurrent map read and map write" in golang 1.9

i can not reproduce the issue in go1.9.2 everything works fine. Could you please add a failing example?


Make the Urgency calculation aware of task dependencies

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.
[X] I am only requesting a single feature. Multiple changes should be split into individual requests, with links between them.
[X] I believe my requested feature will be generally applicable to other users, not just me: it is not uniquely tied to my personal workflow.

🔖 Feature description
The Tasks urgency calculation is based on that of TaskWarrior.
Its urgency calculation takes account of blocked and blocking tasks,
Taskwarrior - Urgency - Taskwarrior

urgency.blocking.coefficient 8.0 # blocking other tasks
urgency.blocked.coefficient -5.0 # blocked by other tasks

✔ Solution
It would be good to teach Tasks to also take account of blocked and blocking status in the urgency calculation, as it will make the default sort order more useful.
❓ Alternatives
Document how to use custom groups - and perhaps custom sort order - to provide similar behaviour.
Or perhaps provide a new sort option, which sorts by blocking and blocked status.
📝 Additional Context
We will need to re-evaluate the urgency caching code - the result is cached in Task - as the result can now be affected by edits to other tasks.
Also, the calculation code will require access to the other tasks in the vault, to find dependencies.

This will not be done for the first release of dependencies.

I came here looking for how urgency handled dependencies. I would love to see this implemented.

It could be interesting if, instead of the addition of a value like Taskwarror does, if blocking tasks could inherit the urgency value of the blocked tasks, so that a task that blocks a lot of low urgency tasks wouldn't have its urgency heightened, but something that is blocking a high urgency task would.
EX:

Task with urgency 3 blocks task with urgency 2 -> keeps urgency 3
Task with urgency 3 blocks task with urgency 6 -> it's urgency is now 6
Task with urgency 3 blocks task with urgency 6 and task with urgency 2 -> it's urgency is now 6

Thank you for the suggestion @TheRockPusher

The more I think about it, the more I really like @TheRockPusher's suggestion. If implemented that way, blocking tasks with low priority blocked tasks will not be artificially inflated simply because they block tasks. And low priority blocking tasks will only be elevated to the urgency of the most urgent blocked task, and not more.
As I understand it, the suggestion is essentially that a blocking tasks has the urgency of the greater value of (1) it's own urgency independent of blocked tasks or (2) the urgency of any task it is blocking. This is really flexible and requires no additional user interaction.


Typo: presesrveData → preserveData

This is good, can you please create a matching jira?

Sure thing.

https://issues.apache.org/jira/browse/ORC-114


Why not Dispatcher?

Hello from Scalar 2023 in Warsaw! I saw a presentation by Kamil Kloch about cats-effect and learned about Dispatcher being the go-to mechanism for running IO's in unsafe territory.
I'm in a position where I've home-rolled a solution that is similar to Feral (build a resource, manually initialize, toss away the finalizer bc lifetimes are hard) and in the last mile call unsafeRunSync.
The presentation I watched was advocating that there shouldn't be a reason to do that and just use a Dispatcher .use() instead "because reasons". Something about the Dispatcher having a better handle on initializing what is required to run IOs?

A) How is the Dispatcher's run different than just using the unsafe global provided?
B) Should feral replace its unsafeRunSync with a Dispatcher per the advice above? Is there some nuance/extra dimension we are missing?

Edit: I believe Dispatcher thoughts are a continuation on #33

Thanks for opening the issue! It's a good question, and I think we can revisit this design choice.

How is the Dispatcher's run different than just using the unsafe global provided?

Here are three advantages of using Dispatcher#unsafeRun over IO#unsafeRun*:

You can have a dispatcher in a generic effect F[_] i.e. Dispatcher[F]
A dispatcher has a scope / lifecycle i.e. it is a Resource[F, Dispatcher[F]]. You can be confident that tasks are not being leaked after the dispatcher has been closed.
A Dispatcher is backed by an already running "warm" fiber. Meanwhile, IO#unsafeRun* has to do extra set-up every time it is used, e.g. registering fatal error handlers.

Should feral replace its unsafeRunSync with a Dispatcher per the advice above?

Yes and no :) point (3) above is a good motivation to create a dispatcher when the lambda starts and use it to handle all incoming requests.
The question is, how will we get that dispatcher? Since the only way to create a Dispatcher is inside of an effectful resource, but without the dispatcher we can't run any effects.
So, we will still need IO#unsafeRun*. We can use this exactly once to obtain a dispatcher during the setup process, that we can then use for all subsequent requests.
Does that answer your question? Thanks, and PRs welcome for this :)

So are you suggesting that with both Feral and my work, we should bootstrap a Dispatcher like a resource for the lifetime of the lambda, and then continually use that Dispatcher for the times when we need to be unsafe?

So are you suggesting that with both Feral and my work, we should bootstrap a Dispatcher like a resource for the lifetime of the lambda, and then continually use that Dispatcher for the times when we need to be unsafe?

Yes, exactly. It seems to me that an application should never have to call IO.unsafeRun* more than once: instead, it can use that one call to setup a Dispatcher, and use that for all subsequent unsafe calls.

Shout out @kamilkloch :)


xWaitForAvailabilityGroup: Should this move to xFailoverCluster?

Details of the scenario you try and problem that is occurring:
This resource does not use any SQL objects to look at Availability Groups. It uses Get-ClusterGroup to check if the cluster group exist. In it's current form it would be better to move this over to xFailoverCluster where it would do more good for other use cases too (but with a better name of course).
Or we can make it actually connect to a primary replica to see that it has been configured correctly. I mean, even if the cluster group exist, maybe the configuration is not done on the first node, this might make the second node jump to quickly.
Or we can move this logic into xSQLServerAlwaysOnAvailabilityGroupReplica which already has parameters for the primary replica.
The DSC configuration that is using the resource (as detailed as possible):
n/a
Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
n/a
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
Dev

@randomnote1 @Zuldan what do you think about this?

To all: Is anyone using this resource? Please let me know how you have used it and if you had any issue. Please also tell me if you did not have a use for it. That is as equal important. 😄

I haven't used this resource yet. Now that I'm thinking about it, I'm not sure if I would now that xSQLServerAlwaysOnAvailabilityGroup and xSQLServerAlwaysOnAvailabilityGroupReplica exist. I would probably only use this resource to wait for the replicas to be created prior to adding databases to the AG.
Is there any other value or scenarios I might be missing?

@randomnote1 maybe this?
Scenario:
So lets say you have two servers. First server should become primary replica (using xSQLServerAlwaysOnAvailabilityGroup, and the seconds server should become secondary replica (using xSQLServerAlwaysOnAvailabilityGroupReplica).
Let say we start the configuration on both servers at the same time, but the secondary node is faster so it want to join the AG but there is no primary replica yet.
Question:
Do we need to use xWaitForAvailabilityGroup on the secondary replica to wait until the primary replica has finished and is available?

@johlju, you're absolutely right. For some reason I was thinking the resource would be run on the same node. I should know better than to think on a Monday!
Yes, we need xWaitForAvailabilityGroup on the secondary to be able to detect when the Availability Group has been created on the primary.
Should we rename this resource xWaitForAlwaysOnAvailabilityGroup?

@johlju we use this resource with all our AOAG cluster setups. Usually our primary and replica nodes are built at the exact same time so this resource is essential for us. We've also never had any issues with it. I don't have a preference with its future direction as long as the replica can wait for the primary node to become active some way or other.

@randomnote1 @Zuldan So you all vote to keep it in this module?
If we move it to xFailoverCluster it could be renamed as 'xWaitForClusterGroup' or 'xClusterWaitForClusterGroup'.
If we keep it here would it be to long to rename the resource to xSQLServerAlwaysOnWaitForAvailabilityGroup? xWaitForAlwaysOnAvailabilityGroup works too, but would love for it to start with xSQLServer so it is easily found with Get-DscResource xSQLServer*.
But maybe I am too finicky? 😄
I have put some time to fix up the resource in PR #507. I thought that was needed regardless. I hope it went into the direction "to the better" :) So I can do the rename in that PR too.

I vote for it to remain here. I think it will make the resource more user friendly.
As for a rename, I imagine I'll get to that after the database membership resource.

@randomnote1 We can discuss the rename in issue #137. I'm closing this issue since i feel the right decision is to keep it here as you point out. If someone else feel differently we can always on it up again. 😄


Host project without Node JS server

Hi.
First, thank you for this awsome project!
My question is, how could we run this app without Node JS? I have a PHP hosting space, but the Node JS part is not very well supported.
Thanks!

Thanks for your interest in this project. If your server is not node friendly, you can try the following options:

First host this project on Github page(Recommended). Then add a custom domain pointing to your domain. You may need to add an entry in your server's Zone Editor. There are some great articles on the internet about it, but if you need help, simply reply here.

The second one is to change the homepage of the project to your domain name. Then build it, zip it locally. And finally, upload the zip file to your server and extract it.

You may check out my other portfolio project Ezfolio which is built for PHP server.

Thanks for your interest in this project. You don't need actually node.js in the server to host. I am guessing you have shared hosting. Follow these steps for shared hosting.

Build the project locally by following the guide.
Zip the project.
Upload to your server and then extract it.
Provide database info in .env
Then you need to migrate and seed data. If you are unable to run php artisan migrate --seed or any artisan command in your shared hosting, you can run the command in the browser. The URL is: yourdomain.com/command/. Migration command would look like yourdomain.com/command/migrate --seed.Make sure APP_ENV is set to local in .env file.

Also, go through the FAQ section.
If you are lost in any step or face any problem, do reply here. I will try my best to assist you.


Missing example app in Helm kube-prometheus

What did you do?
helm install kube-prometheus
What did you expect to see?
there's an example app prometheus instance
What did you see instead? Under which circumstances?
no example app
Not sure if this is missing or I should config a different way when using Helm to deploy kube-prometheus (write another helm like "prometheus-my-app" and select example-app ServiceMonitors)?

Already installed prometheus-operator (helm)
I mean this example app "contrib/kube-prometheus/manifests/examples/example-app/". This is an example to create a Prometheus instance to monitor this app, as I understand. So when it comes with helm, how should I create prometheus instance (leave the config stuffs for prometheus-operator to generate) ?

Prometheus instance is automatically created when deploy kube-prometheus and prometheus-operator. The example app is basically a deployment with the right serviceMonitor and not a prometheus instance.
Please share the pods you have deployed and helm ls output

helm list:
NAME               	REVISION	UPDATED                 	STATUS  CHART                     	NAMESPACE
kube-prometheus    	2       	Fri Apr  6 15:39:48 2018	DEPLOYEDkube-prometheus-0.0.43    	devops
prometheus-operator	1       	Thu Apr  5 15:28:00 2018	DEPLOYEDprometheus-operator-0.0.15	devops

pods:
NAME                                                   READY     STATUS    RESTARTS   AGE
alertmanager-kube-prometheus-0                         2/2       Running   2          1d
kube-prometheus-exporter-kube-state-7f7469d7ff-db5gk   2/2       Running   6          4d
kube-prometheus-exporter-node-jqlc5                    1/1       Running   3          4d
kube-prometheus-grafana-6cd54fd88f-x5jjf               2/2       Running   6          4d
prometheus-kube-prometheus-0                           2/2       Running   6          4d
prometheus-operator-7dc9445795-l88vb                   1/1       Running   5          5d

It's not really what I mean Gianrubio :). Personally, I think the idea behind Prometheus Operator is to deploy the kind Prometheus, and prometheus-operator will watch this object to generate Prometheus instance. So, we can define many Prometheus kind and choose desired ServiceMonitors. If I follow this helm, I will create a Prometheus instance included all of the metrics (the cluster and the services). I'm afraid this will not really utilize the prometheus-operator.
The YAML files in contrib/kube-prometheus/manifests will deploy a Prometheus instance with only metrics of K8s cluster. And the example contrib/kube-prometheus/manifests/examples/example-app/prometheus-frontend.yaml will deploy a Prometheus instance with only metrics of the services running on top of K8s.
So I'm wondering that if I use this helm, Should I write a seperate helm which deploys Kind Prometheus (with our ServiceMonitors) ? And this time, I'll have 2 Prometheus instances running (one is kube-prometheus's and one is mine) Just like the idea in contrib/kube-prometheus/hack/cluster-monitoring/deploy.


Cant extend array or object configuration via environment variables

Currently users can only modify existing configuration elements.
The following test will panic:
func TestConfig_Parse_EnvOverrideAddArray(t *testing.T) {
    // Parse configuration.
    var c server.Config
    if _, err := toml.Decode(`
[[influxdb]]
urls=["http://localhost:8086"]
`, &c); err != nil {
        t.Fatal(err)
    }

    if err := os.Setenv("KAPACITOR_INFLUXDB_0_URLS_0", "http://localhost:18086"); err != nil {
        t.Fatalf("failed to set env var: %v", err)
    }

    if err := os.Setenv("KAPACITOR_INFLUXDB_1_URLS_0", "http://localhost:7076"); err != nil {
        t.Fatalf("failed to set env var: %v", err)
    }

    if err := c.ApplyEnvOverrides(); err != nil {
        t.Fatalf("failed to apply env overrides: %v", err)
    }

       if c.InfluxDB[0].URLs[0] != "http://localhost:18086" {
        t.Fatalf("unexpected url 0: %s", c.InfluxDB[0].URLs[0])
    } else if c.InfluxDB[1].URLs[0] != "http://localhost:7076" {
        t.Fatalf("unexpected url 0: %s", c.InfluxDB[0].URLs[0])
    }
}

Users should be able to add additional configuration to Kapacitor through environment variables.

Yep, investigated a bit and this causes #1280 (which shouldn't have been closed). Also see this. I will work on a fix today.

Turns out it's a bit messy to do right :) I'm working on it though, maybe not highest priority since I can workaround the part that's biting me.

@sofuture awesome to hear that you're working on this. If there's anything we can do to help, let us know.

Will there be a fix for that?

A solution for this would be great since I am unable to configure the kafka event handler brokers via an environment variable KAPACITOR_KAFKA_0_BROKERS_0.

I think we can add a specific key as slice length, then apply child element through prefix_index_field.
if len, err := strconv.Atoi(os.Getenv(fmt.Sprintf("%s_LEN", prefix))); err == nil && len > 0 { 	t := reflect.MakeSlice(reflect.SliceOf(element.Type().Elem()), len, len) 	for i := 0; i < len; i++ { 		f := t.Index(i) 		if err := c.applyEnvOverrides(fmt.Sprintf("%s_%d", prefix, i), f, structKey); err != nil { 			return err 		} 	} 	element.Set(t) 	break }


Add DataStore tests

Currently produces invalid results for testRemoveType and testRemoveIndexDouble

I found out whats going on with the removeType(): please try to use this for memorydatastoreoperations: https://gist.github.com/rfecher/fe106d0f21da98805bb9d54348b1806a

In NativeEntryIteratorWrapper here is a fix for the issue you had with multiple adapters in the same index, please add it to the PR: https://gist.github.com/rfecher/cdfd0d1e9304e75fa1787bb0d4a44988

So I'm seeing that the tests fail because of
org.locationtech.geowave.core.store.memory.MemoryDataStoreOperations$GeoWaveMetadataWithUUID doesn't override org.locationtech.geowave.core.store.entities.GeoWaveMetadata.equals(Object) [org.locationtech.geowave.core.store.memory.MemoryDataStoreOperations$GeoWaveMetadataWithUUID] At MemoryDataStoreOperations.java:[line 1] EQ_DOESNT_OVERRIDE_EQUALS

This is something outside the scope of the test cases. When adding the equals method, mvn clean install works fine.

Coverage increased (+0.1%) to 46.355% when pulling 743cc6c5f3c60ab6b3ea52226bcb8bdfff040beb on nikmanG:removal-tests into 7cd33212578694d2311df326a89b4ced6891f1d2 on locationtech:master.


Can't compile RSS

Hello.
I'm trying to use RSS sensor but I have a problem with compiling it.
This is my environment info
-UBUNTU 16.04
-Carla <IP_ADDRESS> (I reinstalled it today)
-Python 3.8
I've tried what the page below says.
https://carla.readthedocs.io/en/latest/adv_rss/
But when I run
make PythonAPI.rss
next error message occurs

gcc: error: /home/hwansoo/carla/PythonAPI/carla/dependencies/lib/libad_rss_map_integration_python38.a: No such file or directory

gcc: error: /home/hwansoo/carla/PythonAPI/carla/dependencies/lib/libad_map_access_python38.a: No such file or directory

gcc: error: /home/hwansoo/carla/PythonAPI/carla/dependencies/lib/libad_rss_python38.a: No such file or directory

gcc: error: /home/hwansoo/carla/PythonAPI/carla/dependencies/lib/libad_physics_python38.a: No such file or directory

I've tried this with several different python versions, but I failed.
Can you help me to solve this problem please?
Always thank you for all your effort!

Same here + 1

Hi,
did you try the current dev branch, too?
The PR #3388 should have solved python3 built issues.
If you want to make use of the CARLA <IP_ADDRESS> then you have to use the pre-built version, which is compiled only with python2.7 support (https://github.com/carla-simulator/carla/releases/tag/<IP_ADDRESS>). But maybe that one doesn't work for general library reasons under Ubunut16.04... at least that's what I fear.
Regards,
Bernd.

Yes I'm trying with the latest versions of dev branch.
I followed next steps.
 git clone -b dev --single-branch https://github.com/carla-simulator/carla.git
cd carla
./Update.sh
make PythonAPI
make launch
make LibCarla
make LibCarla.rss
make PythonAPI.rss
But the last line makes errors I wrote above.
Is there any problem with my install procedure?
Thank you for reply!
Best regards, Hwansoo.

Instead of all the make commands a single
make package.rss
for the initial build should be enough. I don't know if the previous build (make PythonAPI; make launch; and make LibCarla) which build without RSS lead to the problems you see.
Then you said you are using python3.8, is this the default python installation on your system (most probably the call of python --version will show the python 3.8 right?) Otherwise maybe the explicit selection of the python version could make sense:
make package.rss ARGS="--python-version=3.8"
Somehow it looks as if the install step of LibCarla is not installing the RSS python libs. As these are collected using a glob expression within CMake I have an idea what might happen:
The first run of make LibCarla is not finding the RSS libraries (as they are not yet built).
The second run of make LibCaral.rss is not actually rerunning CMAKE and therefore the install step is not aware of the fact that it has to copy the rss libraries over -> and so they are missing.
Maybe just try to delete all the Build/libcarla-client-build* folders and then run
make PythonAPI.rss
again.

At least the same procedure as you described works on my Ubuntu18.04 machine. (except that I had to replace 'make LibCarla.rss' by 'make LibCarla.client.rss' as the first target doesn't exist)
Then it makes use of the default python3 version installed which is python3.6.
Currently I try to run in the same carla folder:
make PythonAPI.rss  ARGS="--python-version=2.7,3.6,3.7"
and it should compile fine for all the three python versions, I've installed on my machine at once.
Maybe it's a python3.8 issue? But you said you also tested with other versions...

I've tried every methods you suggested, but I couldn't solve this problem, even tried with on Ubuntu18.04 machine and various python versions. (of course I tried with make LibCarla.client.rss, sorry for errata)
I was wondering if you could share your carla directory via git or google cloud
I know this method is very wasteful, but I cannot find any methods for solving the problem...
This is my mail address -<EMAIL_ADDRESS>I'm terribly sorry for asking too much...

Thank you for your comments @berndgassmann

Hmm,
1.) the question is if the ad-rss-4.4.0 stuff was compiled correctly:
find Build/ad-rss-4.4.0/install -name "pythona"
gets me
Build/ad-rss-4.4.0/install/ad_map_access/lib/libad_map_access_python37.a Build/ad-rss-4.4.0/install/ad_map_access/lib/libad_map_access_python27.a Build/ad-rss-4.4.0/install/ad_map_access/lib/libad_map_access_python36.a Build/ad-rss-4.4.0/install/ad_rss/lib/libad_rss_python37.a Build/ad-rss-4.4.0/install/ad_rss/lib/libad_rss_python36.a Build/ad-rss-4.4.0/install/ad_rss/lib/libad_rss_python27.a Build/ad-rss-4.4.0/install/ad_physics/lib/libad_physics_python27.a Build/ad-rss-4.4.0/install/ad_physics/lib/libad_physics_python36.a Build/ad-rss-4.4.0/install/ad_physics/lib/libad_physics_python37.a Build/ad-rss-4.4.0/install/ad_rss_map_integration/lib/libad_rss_map_integration_python27.a Build/ad-rss-4.4.0/install/ad_rss_map_integration/lib/libad_rss_map_integration_python37.a Build/ad-rss-4.4.0/install/ad_rss_map_integration/lib/libad_rss_map_integration_python36.a
after build all the three variants, for sure.
You then should see at least the respective python38 variants of your files.
As a quick fix you could just copy those then to your:
/home/hwansoo/carla/PythonAPI/carla/dependencies/lib/
But as it sounds to me the compilation of ad-rss-lib components could have went wrong already on your side.
find Build/ad-rss-4.4.0 -name "build-*" -type d
gives me:
Build/ad-rss-4.4.0/build-python3 Build/ad-rss-4.4.0/build-python3.7 Build/ad-rss-4.4.0/build-python3.6 Build/ad-rss-4.4.0/build-python2.7
the build-python3 is from the first build without ARGS (then python3 is used by default and the version of python that you get via python3
the others then are from the built with ARGS and detailed specified versions.
maybe you installed python3.8 and replaced the system python3 (which is 3.5 in Ubunut16.04) after doing:
pip3 install --user pygccxml pyplusplus
because then the respective pip3 install would have installed the packages for python3.5 and they could miss for your python3.8 installation?
you could also delete the Build/ad-rss-4.4.0/build* folders and trigger again a build of it via
make PythonAPI.rss
and then observe if there are error output on the build of ad-rss-4.4.0....
I'm sure we will find the root cause, so no problem.
Bernd.

Found an Ubunut16.04 machine to play around.
Found one possible issue:
the python binding generators for ad-rss-lib and its dependencies calls 'python' to perform the code generation.
pip3 install --user pygccxml pyplusplus
installs it for python3, but if the default python version in the system is different these packages are not found.
Within the build process it has to be there for the default python version (in my case python2.7); therefore a
pip install --user pygccxml pyplusplus
made the python binding code generation work.
If your default python is 3.8 you might have to call
pip3.8 install --user pygccxml pyplusplus
to get the components installed for your default python version.

Hi,
I've tried to create a fix for the issue:
Make use of the python version to be used for the build also for the internal call to generate code with pyplusplus.
See:
https://github.com/intel/ad-rss-lib/pull/90
and
https://github.com/carla-simulator/map/pull/32
If these are merged, I'll create also a pull request for this CARLA repo to make use of that, in the hope it fixes your issue.
In the meantime, maybe you could copy the file
https://github.com/carla-simulator/map/pull/32/files#diff-60ea9a95a4f10c4f914e30682978976f6bdf383e642b465c97cc1a5b91c6b0b1
to the Build/ad-rss-4.4.0/src/map/cmake and Build/ad-rss-4.4.0/src/ad-rss-lib/cmake folders and see if that fixes your issue.
To get the RSS part rebuild you just have to delete the Build/ad-rss-4.4.0/build* folders and trigger a build of it via
make PythonAPI.rss.

Hello.
I've succeeded make PythonAPI.rss with my clean version of Ubuntu 18.04.
As you said, the problem was the "ad-rss-4.4.0" is not compiled properly, because of the python version binding issue.
In my guess, installing different versions of python via anaconda made it worse.
I'll try the way you proposed and let you know the result.
Anyway, thank you very much.
I wouldn't have succeeded without your help.
Best regards.
Hwansoo


add regression test for partial fill

add regression test to ensure that partial fills don't set the 'Filled' status until the order is complete

done


feat: added Optimism deposited transaction support

Motivation
Optimism has an extra transaction type: the Deposited transaction (type 0x7E), with 3 new transaction fields as described in the spec here. The RLP encoding order is also different.

Solution
This PR introduces a new feature "optimism" much like the existing "celo" one, which enables the 3 new optional transaction fields.

PR Checklist

[x] Added Tests
[x] Added Documentation
[ ] Breaking changes

I'm supportive of adding first-class optimism objects. However, I am concerned that this change might be too large for our current release cycle. We're trying to put this version into maintenance so that we can focus on ethers@3.
In addition, our opinion generally is that the Celo feature was a bad approach to network-specific behavior in the first place, and this ought to be solved by abstraction of the argument and return types in the Middleware trait
@gakonst do you think it is worth it to unblock optimism folks?

Ok, I've split the addition of a new TypedTransaction variant to a secondary PR to make the diffs cleaner.
Here: https://github.com/merklefruit/ethers-rs/pull/1
I've also added a couple tests for RLP encoding/decoding of this new tx type. Let me know what other kinds of tests I should eventually add!

@gakonst I believe the PR above (https://github.com/merklefruit/ethers-rs/pull/1) was not merged into this branch yet. I've closed it now, so I need to reopen a PR from this branch


How long does it take if I train the network in CPU?

about 24 hours.

only a few seconds

please,should it stored by itself

stopped by itself


Backend for Juice website

We need a new backend for the new juicebot website. https://www.juicebot.ai/
Done when

[x] DB is created with the following fields

[x] wallet (EVM wallet)
[x] passcode (5 digit numerical)
[x] passcode_expiration (timestamp)
[x] registered (bool)

[x] FE can call an API to retrieve the stored information in the DB for a wallet or passcode
[x] FE can call an API to set registered to TRUE (when user submits passcode)

GET /wallet/:id: This API endpoint fetches the details of a wallet given its ID. If the wallet is found, it returns the wallet data. If any error occurs during this process, it returns an error message with a status code of 500.
GET /passcode/:passcode: This API endpoint fetches the details of a wallet given its passcode. If the wallet is found, it returns the wallet data. If any error occurs during this process, it returns an error message with a status code of 500.
PUT /register/:id: This API endpoint updates the status of a wallet to registered given its ID. If the wallet is updated successfully, it returns a success message. If any error occurs during this process, it returns an error message with a status code of 500.


fix typo, constributes -> contributes

I happened to notice a typo. This pull request fixes it.

I then noticed minor issues in the README.md file, so I'm fixing them in this same pull request.


revert(state): Partly undo ADR 109 by re-exporting state package

This PR goes hand in hand with #2709
Contributes to #2610

[!NOTE]
To test both #2709 together with the present PR, use branch adi/v1/2610-reexport-state-and-store
We'll keep the two PRs separate for easier review

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[ ] Title follows the Conventional Commits spec

Going to consolidate this and https://github.com/cometbft/cometbft/pull/2709.


Fix version of argparse

https://github.com/Machtan/cargo-linebreak/issues/3

The pull request has been deprecated due to changes in the latest version.


Update MD file on new branch 2017-39-08_09-12-30

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


stake-pool: Support metadata-pointer and token-metadata extensions

Problem
Stake pools enumerate the supported token-2022 extensions, but token-metadata and metadata-pointer are not included, which means that new stake pools can't take advantage of the new token-metadata standards.
Solution
Very simply, add ExtensionType::MetadataPointer and ExtensionType::TokenMetadata to is_extension_supported_for_mint. https://github.com/solana-labs/solana-program-library/blob/dd0f0ed70e6bf3c0ab83d764a3811d597234e725/stake-pool/program/src/state.rs#L494-L511
This is blocked until token-2022 has the TokenMetadata extension, which is expected to land in about a week.

@joncinque is this fixed by https://github.com/solana-labs/solana-program-library/pull/4770

Ah yes, thanks!


Replaced nf-mdi-home with nf-md-home

nf-mdi-home has been removed from the new nerd font symbols, so the default symbols should be switched to nf-md-home.

Heyyo @SomeHybrid, thanks for the PR.
This change will not break for the people who use older nerd fonts right?

Heyyo @SomeHybrid, thanks for the PR.
This change will not break for the people who use older nerd fonts right?

Yep, the symbol still works with nerd fonts v1.0 (haven't tested anything earlier than that :p)

Great! Thanks again


Animation Reverse is not executed properly - Notifier not fired

Godot version: 3.1.1

OS/device including version: Windows 10
Issue description:
New AnimatedSprite is created.
New Sprite FrameSet is created.
A 'default' Animation is created. Loop is set to off.
When the animation is then played in reverse then the notifier  'animation_finished' is only triggered once.
Steps to reproduce:
extends AnimatedSprite

var count = 0

func callback():
	count = count + 1
	print("I am printed only once: %s" % count)

func _ready():
	pass

func _input(event):
	if event is InputEventKey:
		connect("animation_finished",self,"callback")
		play('default', true)
		print("KeyEvent is printed")

Seems fixed in 3.2


ValueError: No valid path is found for training., Colab error

I am getting this error in collab
I have created a new folder under data called custom

After that I have created new YML files under a new folder

Here is the ar.yml

I am getting this error now any suggestions

Which error? Can you please copy the error here.

python -m vall_e.train yaml=config/Connery/nar.yml
1it [00:00, 5236.33it/s]
Traceback (most recent call last):
File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
exec(code, run_globals)
File "/home/peter/vall-e/vall-e/vall_e/train.py", line 128, in
main()
File "/home/peter/vall-e/vall-e/vall_e/train.py", line 33, in main
train_dl, subtrain_dl, val_dl = create_train_val_dataloader()
File "/home/peter/vall-e/vall-e/vall_e/data.py", line 266, in create_train_val_dataloader
train_dataset, val_dataset = create_datasets()
File "/home/peter/vall-e/vall-e/vall_e/data.py", line 247, in create_datasets
train_dataset = VALLEDatset(
File "/home/peter/vall-e/vall-e/vall_e/data.py", line 105, in init
raise ValueError("No valid path is found for training.")
ValueError: No valid path is found for training.

I bet that your data exceed the number of max phonemas (pre setted to 50). Look at line 105 of data.py and print the local variable of max_phonema before this line and configure a new max that the _validate function do not return not an empty list.

I reduced the length of my wav file and confirmed that the _validate function finds 20 phones.
I am still getting the same error, though, regardless of what I set the max_phones value to in my config file.
The self.paths_by_spkr_name variable contains:
'test': [PosixPath('data/test/test.qnt.pt')]
The problem seems to be with assigning a value to self.paths just prior to the error in data.py:
p for p in self.paths if len(self.paths_by_spkr_name[cfg.get_spkr(p)]) > 1
Training is true, but len(self.paths) is zero, triggering the "No valid path is found" error.

Humm, I see
Maybe you need to duplicate the files, like the original test folder has or put more than one file in your custon dir

Confirmed. It seems to need more that one pair of qnt and phn files. Creating a second set resolved the problem. Thanks!

Closing this, you may also refer to: https://github.com/enhuiz/vall-e/issues/6


Gravatar - only save hash

The rest of the url we can put in the template and save the db some (a lot) of space

old repo - clearing issues


fix: correct both with all adjustment param

Hey sorry, for whatever reason one of the params actually didn't work or is outdated. both should actually be all. Tried all 4 param types of all, split, dividend and raw and confirmed that they all work.

Haha, all good. I looked as well and didn't catch the difference.

Version 6.3.9 has this fix. Hopefully that's the end of it 😆.


mounting with metadata fails on REFS drives

Hi, is it possible to mount refs drives using metadata? I'd like to use yeoman there but it's impossible because it won't allow root and I can't create files as any user except root.
My Windows build number:
Microsoft Windows Pro for Workstations [Version 10.0.17650.1001]

What you're doing and what's happening:
I'm trying to use chown on a file in /mnt/d  which is a REFS formatted drive and I mounted it using

sudo mount -t drvfs D: /mnt/d -o metadata,umask=22,fmask=111
but it fails.

What's wrong / what should be happening instead:
If I use mount -l I see

d: on /mnt/d type drvfs (rw,relatime,umask=22,fmask=111,case=off)
but I should see something similar to
C: on /mnt/c type drvfs (rw,relatime,umask=22,fmask=111,metadata)
When I try to use chown on a file for example
chown myuser:myuser /mnt/d/test
I get
operation not supported

ReFS lacks extended file attributes required/used for metadata.

I see, so this is gonna be something that's only used in ntfs filesystems and not something implemented as an abstract layer for all underlying filesystems?

@ioweb-gr This actually isn't a problem of abstraction for once. It's actually that ReFS simply doesn't store the type of metadata where the WSL team saves it. It's comparable to how ntfs-3g can't directly write Linux permissions to ntfs mounts. ntfs-3g has a mount option to use a permission and user map to approximate the Linux permissions directly in the Windows ACLs, but the WSL team chose long ago not to mix the Lx permissions with the Windows ones.
But since you asked:
Right now, DrvFs support is added filesystem-by-filesystem and driver-by-driver, because Windows does not have a kernel abstraction layer semantically equivalent to *nix-style VFS. In *nix, all filesystem drivers hook up to the VFS layer, and everything is abstracted by inodes.
In Windows, this layer is very thin, allowing only basic mounting of volumes grafted onto directories in an NTFS volume using reparse points. Traversing a reparse point will cause the kernel to execute filesystem operations in the appropriate driver for the filesystem mounted to the right of the reparse point.
Why did Microsoft never implement a generic VFS layer? There was no point because NTFS has radically different semantics to *nix filesystems.  It has a sophisticated ACL system rather than *nix style permissions, for example. It was just easier to avoid abstraction because there were no other filesystems compatible with those semantics.  FAT32 is mounted in a legacy mode. Now ReFS has similar semantics, but there is still no real need for something like VFS.

This was a very interesting read and thank you for taking the time to explain. So in simple terms if we want to see mounting with metadata so we can use permissions and ownerships in REFS, the WSL team would have to implement saving this info in a REFS compatible way and generally on a filesystem basis for each filesystem we want to have full compatibility. Seems like a request in uservoice :)

Yes, but I suspect it will be extremely far down the backlog.

Yes, but I suspect it will be extremely far down the backlog.

True true. But I have been thinking for some time now, since ReFS isn't widely adopted yet (at least in typical Desktop deployments) it sure does make for a once in decades opportunity to do it right, before being constrained by compatibility with millions of installs.

I have to say that after using ReFS for quite some time I find it much less problematic than NTFS for storing my projects (by that I mean no data corruption or suddenly missing files and no more chkdsk runs).
I'd definitely not switch back to NTFS just for being able to add ownership in WSL even if I can't use apps like yeoman generators that don't allow root so I'll wait patiently for ReFS to become more compatible with WSL until I completely switch to WSL for my dev needs.


Check if module directory exists

gotf should check whether a module directory exists and print a meaningful error if it doesn't. Otherwise, depending on the templating use in gotf.yaml, error messages might not be helpful (e.g. Error: could not load config file "gotf.yaml": template: gotpl:1:24: executing "gotpl" at <.Params.moduleDir>: map has no entry for key "_1").

Closing for now. Can't reproduce the issue.

Closing for now. Can't reproduce the issue.


Fix the bug related to the Scope of using Xsens in config file

With this PR we fix the bug related to using xsens option:
During preperation for  dott. Paolucci demo  , we had the problem to use devel branch of walking-teleoperation to perform walking-Oculus teleoperation (without using Xsens) sceanrio.
The problem was, when we were starting the teleoperaion startWalking, the robot hand were going smoothly toward zero.
Investaging on the problem, I understood the problem was from the way I was passing the parameters to oculusModule. useXsens parameter is defined under the GENERAL scope in config files, however, when passing the parameters, I was not identifying GENERAL. Therefore, the configuration was not correctly performed, and during the execeution in oculusModule the option useXsens was true.
Additionally, when investigating on this problem, we understood, the ports /oculusRetargeting/rightHandPose:o and /oculusRetargeting/leftHandPose:o are empty correctly, i.e., nothing written in these port. However, in the controller, it goes toward joint values of 0. We should investigate more on the walking-controller side and try to fix this problem.
@fjandrad @DanielePucci

@GiulioRomualdi

LGTM we can merge the pr!

Thanks @GiulioRomualdi


the feature branches still alive ?

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior

i want to check if i can add material/angularfire to the project. then i found the feature branch.
like this https://github.com/NathanWalker/angular-seed-advanced/tree/material2 and this https://github.com/NathanWalker/angular-seed-advanced/tree/angularfire2 , but , the branches are very old and not update at all. just a dummy branches. right?
Expected behavior

@NathanWalker please delete the dummy branches, keep the repo clean
Minimal reproduction of the problem with instructions

check the repo branch
What is the motivation / use case for changing the behavior?

no
Please tell us about your environment:

Angular Seed Version: aaaaf75

Node: node --version = 7.5

Hi @geminiyellow thanks for heads up - yes those feature branches are out of date - i will schedule to clean up a number of issues (a lot not relevant for stay open) and either remove the feature branches or contact latest commit authors on those to see if they'd like mention of those integration to their own fork. Thanks.

@NathanWalker thank you~ a great job.

Thanks @geminiyellow 👍

hi @NathanWalker , two years later, i notice the latest commit is 2017, this project is still working?


Activate eslint on Azure DevOps

Feature Request
[Enhancement]

I thought this wasn't easy because we didn't turn it on and should have a few ESLint things breaking the build.
But in fact, everything passed. :)


Versioned package release request

I'm working to make ForwardGenomics available on Bioconda.
Please refer. -> https://docs.github.com/en/repositories/releasing-projects-on-github/managing-releases-in-a-repository#creating-a-release
I need your help creating a versioned release to use for the Bioconda recipe. Once this is added to Bioconda, it'll also be made available as a Docker container from Biocontainers, and as a Singularity image from the Galaxy Project. The Bioconda bot will also recognize future releases and automatically update the recipe.
Please let me know
Thanks
Jay

Done. Thanks Jay


Update README.md

Added a repo in the readme file
Contributing
If you want to add a project to this list, please make sure that:

[x] You read the contributing guide
[x] The repository has hacktoberfest as topic and counts for hacktoberfest event
[x] The project can't be just an "add your name" or simply + 1 contribution to hacktoberfest
[x] Place them in alphabetical order
[x] One repository per PR/contribution - avoid conflicts and simplify review
[x] Your pull request is not spammy just adding an empty line, otherwise it will be marked as invalid and not count to hacktoberfest.

If they meet the requirements above, feel free to create a PR. Happy Hacking! :sparkles:

Hey, @OtacilioN , I've read the Readme.md file and did what was asked. I hope this would count for Hacktoberfest 2022 PR. Kindly merge it soon as possible from your side. Thankyou


⚠ 366.ru has degraded performance

In 10b415b, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1822 ms

Resolved: 366.ru performance has improved in bc188c8.


Fix problem installing 0.6.0 dev on a Raspberry Pi with stretch

Work in progress. Trying to make rpi3 install with stretch easier. Please add problems with installing on RPI3.
Problem 1: json is not installed and causes oref0-setup.sh to fail
added process://ns/nightscout/ns NIGHTSCOUT_HOST API_SECRET
/usr/local/bin/nightscout: line 338: json: command not found
close failed in file object destructor:
sys.excepthook is missing
lost sys.stderr
Traceback (most recent call last):
  File "/usr/local/bin/openaps-import", line 89, in <module>
    app( )
  File "/usr/local/lib/python2.7/dist-packages/openaps/cli/__init__.py", line 51, in __call__
    self.run(self.args)
  File "/usr/local/bin/openaps-import", line 77, in run
    candidates = json.load(args.input)
  File "/usr/lib/python2.7/json/__init__.py", line 291, in load
    **kw)
  File "/usr/lib/python2.7/json/__init__.py", line 339, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 364, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
Could not run nightscout autoconfigure-device-crud

Solution 1:
This is caused by a missing json command from npm.
Fix this by adding it to as a oref0 dependency in package.json
Problem 2: sudo is installed on raspbian and installing should be able to able as pi user.  Currently quick-packages.sh seems to require to be run as root (because missing sudo on edison)
Solution: will do some refactoring to openaps-install.sh so that it installs sudo for edison and it will call sudo for quick-packages.sh
Problem 3: No npm or nodejs is installed on raspberry by default
Suggested solution 3: Add nodejs and npm checking in openaps-install.sh
Problem 4: find: ‘/tmp/systemd-private-xxx-systemd-timesyncd.service-yyy’: Permission denied
Probably adding in a sudo will fix this. Need to look into this.

Instead of Solution 1, I'd prefer to replace all references to json with jq . instead, for Mac compatibility.
Both the Explorer Block for Edison and the new Explorer HAT for Pi require running setup as root to support mraa.  I'm not sure there's any advantage to trying to support new installs running as pi (but we should of course keep using sudo where it's easy to support existing ones).

@PieterGit do you still plan to work on this PR, or should we close it for now?