Undesired result for timezones with overlapping names

Hello,
After enabling timezone commands and conversion on my server, an interesting issue popped up. Specifically, when a user attempted to setz America/Dawson, they got the following response:
More than 1 result, reuse the command with a one of the following:
America/Dawson_Creek: MST
America/Dawson: PDT

Your current time zone is registered to: America/Denver MDT (UTC-6)

The issue here is clearly the overlapping timezone names. Can this be addressed somehow?

Also, note that if a user wants to set their timezone to desired_zone, for example, this issue prevents them from doing so if any timezone of the form desired_zone* exists.

Forgot to link this to #813. Since that's merged, closing this issue now.


Different licenses - LICENSE (MIT) vs package.json (GPL-3.0-only)

It seems that the update of the package.json was overlooked when updating the LICENSE (https://github.com/node-ebics/node-ebics-client/commit/7520a56be8ba446d3466c6dc9202b1d65ea3396f). The package.json still contains the old GPL-3.0-only license which has already been updated in the LICENSE file. If I understand correctly, it should be just a matter of updating the package.json to MIT, correct?

Addressed in 4.1.0


Fixes #4953: remove misleading dialog 'Unable to get descriptions'

Description (required)
Fixes #4953
In the onActivityResult method of MediaDetailFragment, a dialog 'Unable to get descriptions' will be displayed if the DescriptionEditActivity returns with result code RESULT_CANCELED which is the default return value when an activity exit.
I deleted the codes which display the misleading dialog when DescriptionEditActivity exits with result code RESULT_CANCELED and the issue is solved.
I think the same issue happens for the activity that edits coordinates.
Tests performed (required)
Tested ProdDebug on HUAWEI HMA-AL00 with API level 29.

Thanks a lot Justin!


Support string private_key

Describe the feature
Allow accepting a private_key as a string, not just a path.
Describe alternatives you've considered
In some contexts, providing a path to the key is difficult, and allowing it to be passed a string provides another mechanism to consume that key.
As a concrete example, while using airflow-dbt, we would like to read credentials from secretsmanager and use them for authenticating into Snowflake. Normally, we would just create a temporary file with a context manager, write the private key out to the file, and use it. However, this proves awkward when using Airflow.
Additional context
The airflow-dbt project allows vars to be templated, which means we could dynamically (and idiomatically) provide the private key contents dynamically. However, there's no clean toehold to do this using a context manager, and the ideas of 1) pre/post-hooks to clean up a temporary file for a private key, or 2) letting the private key linger in a temporary directory are pretty unappealing.
Who will this benefit?
Predominantly dbt users needing to use private key auth without touching the filesystem.
Are you interested in contributing this feature?
Definitely! I've got a MR nearly ready to go, which I will submit after submitting this issue.

This would also be amazing for CI, such as Github where we have to create a temporary file.

Sorry for the delay here — sure! I have no reason to oppose this. I'm going to mark this a good first issue. It will be tricky for us to test in our CI setup, so I'll be relying on whomever contributes this change to do some local verification as well.

Hi @dlecocq, are you still planning to make a PR for this? If not I'm considering implementing it (and also adding support for bytes keys as well as strings).

Is this feature still being considered?

Re-opening since https://github.com/dbt-labs/dbt-snowflake/pull/265 is an open pull request meant to implement this feature.


Force graphs y axis to be zoomed out by default to make history look more beautiful when not so log history available in database. Use the same values than on iOS!

QA passed


Unit test execution sometimes gets stuck on devices

See example from https://github.com/ThaliTester/TestResults/blob/56672827039a409__472_Promise_Queue_yaronyg/iOS_Iphone5s-1.md where an iOS device doesn't proceed from the first test.
@tobybrad: This is the issue I talked you about earlier over Skype. Based on our investigation, the test itself is not broken, but the root cause would be in the way how currently the test server coordinates tests and events related to that over socket.io.

I know @tobybrad just had a bug that caused the coordination server to get stuck but I believe that was an error in the test and not the infrastructure. Are we still seeing this bug or can I close?

This has already been fixed by @tobybrad and we haven't seen any issues in this regards for a while.


dict corrupted by certain insertion/deletion order

Acton Version
<IP_ADDRESS>405<IP_ADDRESS>
Steps to Reproduce and Observed Behavior
actor main(env):
    d: dict[int, str] = {}
    d[1] = "one"
    d[2] = "two"
    del d[2]
    d[3] = "three"
    del d[1]
    print(str(d))
    print(str(d[3]))
    env.exit(0)

Expected Behavior
{3:'three'}
three

@sydow I reckon this is for you :)


[remote tagger] Prevent hanging on shutdown

What does this PR do?
In some rare instances where the remote tagger is stopped at the same
time as either the health check or telemetry ticker trigger, the
select would not see that t.Ctx was already cancelled, and would
carry on trying to re-start the stream. This messes with the shut down
of other agent components, as Tagger.Stop() seems to block in that
case.
To make sure that <-t.ctx.Done() is always evaluated, we now re-start
the loop on the health check and telemetry ticks.
Describe how to test/QA your changes
Same as #12592, error received trying to start stream: tagger stream not started should not be present 100% of the time (it's been reported that the failure rate was ~30%). Also, this was occasionally breaking kitchen tests. Applying the skip-qa label as this QA is already covered by both of those things.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

@L3n41c looking at this with fresher eyes today, indeed you're correct. It does reduce the amount of errors (which is what I originally wanted to do with #12592), but would still hang.
There's another check for ctx.Done in startTaggerStream as a last resort, which errors out early as expected, and it passes the context to the code starting the stream, which I'll assume works as expected. Unfortunately, the error check at the call site is missing another continue, so that's actually what makes it hang (the code after it is trying to receive on a closed stream), not the expired context itself. As a matter of fact, if we didn't care about the errors none of this context checking is necessary, the missing continue does the trick. But like I said, reducing the amount of errors is also desirable, so I'll keep both changes.

Cherry-picked #12592 (which had been reverted by #12841) in here as well, since it was only reverted in the first place because this fix hadn't been finished timely, and tests were broken.


Add support for Docker

Why: It's much easier than Vagrant, especially with the new Docker for Mac and Docker for Windows

Coverage remained the same at 86.236% when pulling 9336b21fc893932e4250bbf532dc1f04d40eba26 on add-docker-support into 08665e691ca626bd21a8870e09ebf3c147db17cb on master.

I was just about to ping you, @md5! Thanks for the review. I'm new to Docker, and I copied this from another project. I will clean it up.

Coverage remained the same at 86.236% when pulling 31eae12cd2aa974ce57d7c587e44eb7586cad654 on add-docker-support into 08665e691ca626bd21a8870e09ebf3c147db17cb on master.

Coverage remained the same at 86.236% when pulling a318f80636b02d98cb30ab582cab56776491622c on add-docker-support into 08665e691ca626bd21a8870e09ebf3c147db17cb on master.

@md5 I addressed all your comments except for the PhantomJS checksums. Let me know how it looks. Thanks!

Coverage remained the same at 86.236% when pulling bb8d23a023fa84a712cebeaa5b281b8ad5a2ac9e on add-docker-support into 08665e691ca626bd21a8870e09ebf3c147db17cb on master.

@monfresh Looks good.
For the checksum thing, I'd do something like this: https://github.com/docker-library/wordpress/blob/4cfa30302dbf66d002ed15b01c881acee605e2dc/apache/Dockerfile#L28
I'd probably also add a fail-safe line like this:
RUN BUNDLED_RAILS_VERSION="$(basename "$(bundle list --no-paths rails)")" \
	&& test "${BUNDLED_RAILS_VERSION##*-}" = "$RAILS_VERSION" \
	|| echo >&2 "Bundled Rails version ${BUNDLED_RAILS_VERSION##*-} does not match Docker image Rails version $RAILS_VERSION"

This will avoid having the image install multiple versions of the Rails gems.
An alternative would be to have the Gemfile pull the FROM line from the Dockerfile and use it to set the rails dependency, but that seems a bit odd.
It may also be nice to have Travis attempt to build the Docker image to catch any issues that might arise with the setup. I could see having that be a separate build in the matrix to allow it to run in parallel with the tests and possibly mark it as an allowed_failure.

Coverage remained the same at 86.236% when pulling 832095520a60213e9a0dd8039d3c1a1fe7ad0136 on add-docker-support into 08665e691ca626bd21a8870e09ebf3c147db17cb on master.

Thanks! I added the checksum and switched to the Ruby image because I remembered that we're not actually using the full Rails gem, only Railties.


Update LAB[PL-200]_M12L05_Excel_template.md

Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by ParamRamki on behalf of MOC Labs Review

I applied this change to the Jan_2022_scrub branch and will merge this evening. Thanks!


Improve Javadoc Comments

Description:
The current Javadoc comments in the Http-Server project can be improved to enhance clarity and provide more beginner-friendly explanations. This will help new contributors better understand the code and contribute more easily.
Tasks:

Review the existing Javadoc comments across all classes and methods.
Simplify technical jargon and provide clear explanations.
Add examples where necessary to demonstrate how methods and classes are used.
Ensure consistency in formatting and language throughout the codebase.
Follow best practices for writing Javadoc comments:

Briefly describe the purpose of the class or method.
Explain all parameters and return values.
Highlight any exceptions thrown by the method.
Include relevant tags (e.g., @param, @return, @throws).

Requirements:

Basic knowledge of Java and Javadoc format.
Willingness to make the comments approachable for beginners.

Additional Notes:
Feel free to ask any questions or request clarifications in the comments below. Let's make this project more accessible to new contributors! 😊

If you want I can contribute to this issue

If you want I can contribute to this issue

Hi @mr-panther01,
Thank you for expressing your interest in working on this issue. I'm delighted to inform you that I have assigned this issue to you. Your willingness to contribute to our project is much appreciated.
Before making any contributions, please read the Readme.md file carefully to better understand the project's goals and purpose. This will give you clarity on our mission. Also, make sure to review the Contributing.md file, which contains important guidelines to make the contribution process smoother, especially for newcomers.
Feel free to start working, and if you have any questions or need assistance during the process, please don't hesitate to reach out.

Thank you so much for assigning this issue I assure you that you will get the best output from me

Hey, I'm a newbie, is this issue still open? I want to learn and contribute and if not is there any other basic good first issue? I am new to open source world but I have always been fascinated by open source .

Hey, I'm a newbie, is this issue still open? I want to learn and contribute and if not is there any other basic good first issue? I am new to open source world but I have always been fascinated by open source .

Hi @Ambit-ion,
Thank you for expressing your interest in working on this issue. However, this issue is currently being worked on by another contributor, @mr-panther01. You can explore other issues in our repository or check out our other projects under the GirlScript Summer of Code and Hacktoberfest.
Here are the links to explore:
https://bento.me/codiescoder
Feel free to reach out if you have any questions or need guidance on where to start. We're excited to have your contributions!

Thank you , I'll explore other repos also can you tell me how to start? I mean the basics of open source world and everything ( which u can in brief )

Hey, I'm a newbie, is this issue still open? I want to learn and contribute and if not is there any other basic good first issue? I am new to open source world but I have always been fascinated by open source .

Hi @Ambit-ion,
Thank you for expressing your interest in working on this issue. However, this issue is currently being worked on by another contributor, @mr-panther01. You can explore other issues in our repository or check out our other projects under the GirlScript Summer of Code and Hacktoberfest.
Here are the links to explore:
https://bento.me/codiescoder
Feel free to reach out if you have any questions or need guidance on where to start. We're excited to have your contributions!

Thank you , I'll explore other repos also can you tell me how to start? I mean the basics of open source world and everything ( which u can in brief )

Checkout this https://youtu.be/o6xikISiz2w?feature=shared

Thank you so much for assigning this issue I assure you that you will get the best output from me

Hi @mr-panther01,
I hope you're doing well! I wanted to check in and see how things are progressing on the Javadoc Comments issue. Have you made any updates or progress on it ? If you’ve encountered any challenges or need assistance, feel free to reach out.
Looking forward to your update!

Sorry to tell but I would not be able to contribute on this issue as I am a college student and hasn't been learned this topic

@ajaynegi45 Can I try this task ?

Yes you can


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1497 ms

Resolved: Bofrost.be performance has improved in c6697ff after 7 minutes.


Support multiple accounts

Right now, the config file only handles one account (selectedAccount).
It would be great if you could specify multiple accounts in one config file in order to de-duplicate configuration (right now you would need 3 config files for 3 account i.e.)/

I.e. I have two account which I use sometimes for similar stuff (i.e. shopping). That means I need to maintain two similar filter rule sets in two config files.

The current state requires multiple config files for multiple accounts, so it would be good to be able to specify the config file to use as a cli argument.

btw, buchhaltung handles multiple accounts like this

I had a look at buchhaltung and that looks like a much more evolved version of fints2ledger.
I gave up trying it out after some hours of failing to set it up, though.
What's the reason you don't use this over fints2ledger?

I have been using buchhaltung, I think I moved to fints2ledger becuase haskell is a complete black box to me, and I feel much better with a python project, where I can at least contribute/improve small things. And so far it works for me quite well.
I have a wrapper script around fints2ledger to deal with multiple accounts, so we can also close this if it's not realistic to add native support.

I would like to keep this issue open.
It documents what is not supported, and I still hope I someday find the time and motivation to add this feature 🙂

I guess I am a little late to the party because I have only just discovered fints2ledger in its current Python form and now the move to Haskell is well underway. I am very much willing to contribute further to this project, because automated CSV downloads integrate very nicely with other tools like hledger-flow (my current weapon of choice).
Two features that I would love to work on:

native support for multiple accounts (as requested here by @varac)
finer control regarding the transaction data fields to be downloaded. In particular, I am interested in the account balances associated with each booking for automated balance assertions (see relevant doc sections in ledger/hledger).

The only issue is: I, too, have zero prior experience with Haskell (and limited to intermediate Python skills tbh). Hence, I expect the learning curve to be quite steep initially. I have not checked out the Haskell branch yet but can I kindly ask you to map out a rough vision for this project (maybe in a separate issue), @MoritzR? Do you want to go 100% Haskell with this? Is it realistic to keep some legacy support in pure Python for users who prefer it that way (probably not)?

Let me quickly answer here.

Do you want to go 100% Haskell with this?
That's not possible because I don't know any other viable alternative to pyfints. This means that the FinTS part will always be written in Python. You can check the remaining python code after switching to haskell here. This file basically contains everything needed to download FinTS transactions as CSV. Haskell is used for everything else like config parsing and sorting the transactions into a journal file.

Is it realistic to keep some legacy support in pure Python?
Yes, I want to continue to provide support for any important fixes, like migrating to pyfints 4 for example. I won't add features anymore, but let's be real, I also rarely did that on the python version.

I am very much willing to contribute further to this project
That's great to hear 🙂 ! Haskell has a steeper learning curve than Python though, so expect that even smaller changes, like https://github.com/MoritzR/fints2ledger/pull/23 will be noticeably harder. I would also probably need to add at least a bit of documentation and comments to make getting into the codebase easier. Let me know if I should do that.

I don't know any other viable alternative to pyfints. This means that the FinTS part will always be written in Python.

That is music to my ears because I have already toyed around a bit with pyfints and the mt940 library. Sounds like a good entry point for me to find my way around the code base and start working on additional columns/data fields to download.

Yes, I want to continue to provide support for any important fixes, like migrating to pyfints 4 for example.

Sounds great!

I would also probably need to add at least a bit of documentation and comments to make getting into the codebase easier. Let me know if I should do that.

I do not want to make any promises that I can't keep but my schedule for the foreseeable future looks rather empty. I would like to try and fit learning a new programming language in there. I have been on the road these last few days so I have not been able to check out the Haskell branch and see how much of it I (don't) understand. Still, if you find the time to add some comments/documentation to the existing code I'd highly appreciate that! 🙂

I have added a few code comments and some documentation. It's not much but maybe enough to raise additional questions. Feel free to raise an issue regarding any documentation you would like to see. @stewmehr
Regarding this issue (support for multiple accounts), it's probably not a good first issue. I didn't tackle this for a long time because I dread having to set up some migration for existing users, as the config.yml format would change.

Thanks a lot, I will start working my way through the codebase and through "Learn You a Haskell for Great Good". As discussed it will ~probably~ definitely be easier for me to start with some pyfints related coding.
Slightly OT but since you mentioned a possible migration to pyfints 4: do you already know if/how having a fints product ID will be checked and/or enforced in pyfints 4? Are developers supposed to request a project-specific product ID that has to be provided in every API call? Are users supposed to request their own? I am a little confused as to how this is supposed to work with open source software.

Are users supposed to request their own?

No, I requested a product ID myself, which all users of fints2ledger will automatically use.

but where do German banks draw the line?

They specifically mention that product IDs of "libraries" should not be used for API requests, see Question 4 on the ZKA website FAQ.
So for now, users of fints2ledger should not need to worry about all of that at all.


Keep docker container names consistent

I've noticed, when working with pygmy that the container aliases do not all align:

If we could get the mailhog container to get a nice prefixed name like the other ones, would be great for filtering.
Much appreciated!
PS: Not sure if anyone really gets the same degree of meticulousness as I do, but could it be called pygmy-* as opposed to amazeeio-* as well? More descriptive of the task that way too.

https://github.com/pygmystack/pygmy supports this


ReScript 12 compatibility workaround

Workaround for https://github.com/rescript-lang/rescript-compiler/issues/7023 so that I can continue testing v12 in projects using rescript-schema.

I plan to include a few more changes until 8.3.0 release. For now, you can use the package from artifacts https://github.com/DZakh/rescript-schema/actions/runs/11053405533?pr=93


[New game]: Memory Flip

🎮 Game Request
Game logic and basic description
In this game, players flip tiles to find matching pairs. The goal is to match all pairs within a limited number of moves.
Point down the features
Game points

Shuffled Tile Values
The game generates and shuffles tile values so that each game session is unique. This is implemented in the generateTileValues() function, which creates pairs of values, shuffles them, and assigns them to the tiles.

Move Counter
The game tracks the number of moves the player has made. This is displayed on the screen and updated every time two tiles are flipped. The move count is implemented using the moves variable and updated within the flipTile() and checkMatch() functions.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Aditi22Bansal , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#3742

Hey @Aditi22Bansal ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇

Hey @Aditi22Bansal !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Why this issue is closed?
Please clarify
Thankyou


Use HTTP HEAD request for --num-pages instead of GET

We only care about the LINK headers in order to retrieve the number of pages available for the given resource.

hey could you guide me on this i want to work on this issue

yes, basically when a user requests number of pages available in a resource with the --num-pages cli flag in list operations, the code does a GET HTTP request in https://github.com/jordilin/gitar/blob/main/src/remote/query.rs#L48 That is not really necessary as we only care about the link headers in the response. So, technically a HEAD is only needed. For that to happen, we need to add support for a HEAD request in the http module, client struct - https://github.com/jordilin/gitar/blob/main/src/http.rs#L333 Implementation should be close to how the GET is implemented except that the server does not send a response body back. A test would also be required to verify functionality on the dependency being used ureq. That can be done in here https://github.com/jordilin/gitar/blob/main/tests/http_test.rs#L26 - You can take the test_http_runner as a reference which does a GET request. The server mock being used can be set to return a link header like for example
link: <https://api.github.com/repositories/683565078/pulls?state=closed&page=2>; rel="next", <https://api.github.com/repositories/683565078/pulls?state=closed&page=5>; rel="last"

and an empty body.
Make sure to run cargo test and cargo fmt so all tests pass and code is automatically formatted. Thanks!!


Enables output.rreverseAttributes option

Related to https://github.com/microsoft/vscode/issues/110251

Awesome! Can you add a test for this in expand.ts?

Added unit tests


Bump dependencies

I'd also suggest enabling scala-steward that helps keep dependencies up-to-date.

Something's wrong with Travis :thinking:
https://travis-ci.community/t/error-installing-oraclejdk8-expected-feature-release-number-in-range-of-9-to-14-but-got-8/3766

Hi! Thanks for the PR. I'm not sure that Akka Persistence Cassandra `0.9х] is compatible with ]0.6х]

You're right. Migration is required to upgrade... I'll revert it for now. https://github.com/akka/akka-persistence-cassandra/blob/master/docs/src/main/paradox/migrations.md#migrations-to-080-and-later

@guersam I resolved some conflicts with master

Thank you @guersam! Merging now 🚀


[BUG] Preserve last visibility setting in setvisible method to maintain consistent css display values

The setVisible method overrides any "display" value set by CSS with browser default. This method should preserve the last visibility setting on the component, so when visibility is toggled off and back on, the previously set value can be retained.

Fix in BBj 24.11


Webcam.js

Works fine but it saves an empty image

Can you provide a test-case on codepen?

The problem was my code but i got it sorted and its working fine thanks a
lot.
On Thu, Oct 27, 2016 at 9:09 PM, Josh Beckwith<EMAIL_ADDRESS>wrote:

Can you provide a test-case on codepen?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jhuckaby/webcamjs/issues/198#issuecomment-256739865,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVxmvEECmJh_ooAGyh3k95LOR2GoC2qQks5q4PbcgaJpZM4KWuIe
.


Fix simple-solver-logging example

Since we made Dense::convert_to(Dense) a kernel call, the example no longer involves a copy operations for the preconditioner apply, so the vector is empty, which leads to a segfault.
Fixes #1063

I won't run a full pipeline on this, since it's just deleting code and completely independent of the previously merged PR.

rebase!

you can add the skip-full-test to avoid run the full test


Quick-Sort in C++

Changes

Added quick-sort.cpp.

hey, @viralvaghela kindly take a look into it, any suggestion If I can achieve this in some better way or is it cool?

Looks good.

Thanks @viralvaghela for a very quick merge!


Update Contribute guide on Data portal

Description of the task:
The guides on the Data portal need to be updated
The most outdated one was probably the Contribute guide - it has been removed for now because it did not cover managed access datasets and we didn't have time to update before the Genral meeting (29-30 Sept 2024)
Lucia has prepared a draft for the Portal guides which we need to review
draft guide: here
Acceptance criteria for the task:
deadline: ?

[ ] review document
[ ] information for contributors is complete

I've gone thorugh the document quickly
I think the metadata explanation should come before the submission instructions, otherwise it's confusing to see submission instructions for other portals on the dcp
improve the instructions for the data portal - is there an email address? we need to discuss the DCA again (do OA contributors have to sign? it will become very lengthy)
fix some links

We made suggestions, comments and questions on the document. It is now public, but can be edited.


feat: Linked XY Plot Axis Tick Label Synch (PT-184187726)

PT Story: https://www.pivotaltracker.com/story/show/184187726
Most of the features requested in the PT story were already present. There is a small issue, though, where when a table and graph are linked, changing the name of a column in the table does not automatically and immediately update any corresponding label text on the graph's axes. (If you reload the page, the label text does get updated.)
The main problem is that the graph's AttributeLabel component is only looking for a setAttributeName action in its observeAttributeNameChange listener, but the action being reported when the table column name changed is the Attribute model's setName. These changes add a isSetNameAction to src/models/data/data-set-actions.ts, and then update the AttributeLabel's observeAttributeNameChange listener to check for that as well.
There's also a new useEffect added to the graph's AxisOrLegendAttributeMenu component that similarly checks for setName actions and updates the axis label's menu button text.

The main problem is that the graph's AttributeLabel component is only looking for a setAttributeName action in its observeAttributeNameChange listener, but the action being reported when the table column name changed is the Attribute model's setName. These changes add a isSetNameAction to src/models/data/data-set-actions.ts, and then update the AttributeLabel's observeAttributeNameChange listener to check for that as well.

We may end up going this route eventually, but for this case I think it would be simpler to consider having the table use the DataSet.setAttributeName() API rather than changing the attribute directly, which I think would eliminate the need for some of these changes.

The main problem is that the graph's AttributeLabel component is only looking for a setAttributeName action in its observeAttributeNameChange listener, but the action being reported when the table column name changed is the Attribute model's setName. These changes add a isSetNameAction to src/models/data/data-set-actions.ts, and then update the AttributeLabel's observeAttributeNameChange listener to check for that as well.

We may end up going this route eventually, but for this case I think it would be simpler to consider having the table use the DataSet.setAttributeName() API rather than changing the attribute directly, which I think would eliminate the need for some of these changes.

@kswenson Thanks! That did the trick and is simpler, though I found I had to keep the new useEffect in AxisOrLegendAttributeMenu.
I didn't realize there was a different setAttributeName than the TableContent model's (which was being used previously and resulting in the setName action being called), but that makes sense.


Merges Master to Live. See commits for details.

Docs Build status updates of commit 3b9da69:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 828 build(s) for this repository.

Docs Build status updates of commit 3b9da69:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOOrgAssetsLibrary.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-online/Remove-SPOOrgAssetsLibrary.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOOrgAssetsLibrary.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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.

Docs Build status updates of commit d90b974:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 829 build(s) for this repository.

Docs Build status updates of commit d90b974:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOOrgAssetsLibrary.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-online/Remove-SPOOrgAssetsLibrary.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOOrgAssetsLibrary.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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.


toDate() can't parse dates in non-default locale

Hi,
I just encounter this issue:
I'm trying to parse a string to date using Refine, for example:
"01-六月-2013".toDate('dd-MMM-yyyy') is OK!
"01-Jun-2013".toDate('dd-MMM-yyyy') will print: Error: Cannot parse to date
First, I'm in China.

Sorry to bother you here. Just can not find what I need anywhere.
I need to edit the date format so that all the dates in my database are the same. I have some dd/mm/yyyy and some are yyyy/mm/dd. And I need all of them to be yyyy-mm-dd.
Where can I find the expression to filter just one kind (dd/mm/yyyy) so I do not have to include them in the text facet to be able to work only with them and so I can separate in columns and order them in the order I need?
Too many clicks needed. Must be another way.

@mabecabrera the dedicated support forum https://forum.openrefine.org is probably a better place to ask this type of question (where the issue is "how do I do X" rather than a bug or feature request). But to answer briefly, there are a few ways of doing this, but I'd suggest using a text filter on the column, check the 'regular expression' check box and use \d{2}/\d{2}/\d{4} to filter to only dd/mm/yyyy style dates and \d{4}/\d{2}/\d{2} to filter to only yyyy/mm/dd style dates

Thank you very much!!! it helps a lot! I will save that forum link for further questions.
I'm very grateful!!! Thank you very much!


made the DWoC logo smaller

Issue Number
ISSUE #

Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.

Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] The title of my pull request is a short description of the requested changes.
[ ] Participating in SWOC2021.
[ ] Participating in DWOC.

Merge this PR with #28

@Sloth-Panda Make a complete PR for the issue at once.

should i create a new branch then

Hey, I am inexperienced so confused about why this is showing to resolve conflicts above and also make a complete PR means do I need to create a new branch and do the work over there. Please explain a bit .

should i create a new branch then

Go ahead

Hey, I am inexperienced so confused about why this is showing to resolve conflicts above and also make a complete PR means do I need to create a new branch and do the work over there. Please explain a bit .

Corrected that!


Inconsistent alignment in paper-items with different types of non-body content

The icon and the checkbox both align to the right in the drawer, but they are misaligned.
Here is the example:
<paper-drawer-panel>
  <paper-header-panel drawer>
    <paper-toolbar class="top-toolbar">
      <paper-item>Options</paper-item>
    </paper-toolbar>

      <paper-item>
        <paper-item-body two-line>

Show your status

You are visible to everyone

        </paper-item-body>
        <iron-icon icon="warning" class="small"></iron-icon>
      </paper-item>

      <paper-item role="menuitemcheckbox">
        <paper-item-body>

Show your status

        </paper-item-body>
        <paper-checkbox></paper-checkbox>
      </paper-item>

  </paper-header-panel>
  <paper-header-panel main>
    <paper-toolbar class="top-toolbar">Hello</paper-toolbar>
  </paper-header-panel>
</paper-drawer-panel>

This issue was moved to PolymerElements/paper-item#50


Report States "NO DATASET"

reportFile.html.txt
postgresql-2019-08-07_093413.log
postgresql-2019-08-07_141019.log
publishFile.cmd.txt

This is the same as issue #505 , pgbadger is broken on Windows operating system. I'm sorry but I do not have the time to work on it for the moment. Best is to use it on Linux or to use an older version, https://github.com/darold/pgbadger/releases/tag/v9.2 works on window.

Darold
Sorry to create a duplicate issue.
Downloaded and validated that version 9.2 indeed works very well.
Please consider issue remedied.


.NET 6 Preview - Issues Installing

I was not able to get your code running. Not sure what the issues were and I started with installing .NET Preview 6 tried to make it work from there. Not sure if there was some version mismatch or something.
Is the project dependent on the .NET 6 Preview? Or is there a way rollback the project files to work on .NET 5?

Hi, yes, this project uses NET 6 Preview 1, there should be SDK installed. Have you received any error messages? Could you please share them with me?
Theoretically, you can run this code on NET 5, for this you should downgrade the dependencies in the * .csproj files.

Here is what I am seeing. The 6.0 version I am using is 6.0.100-preview.1.21103.13

You should start Blazor.Server project intead of Blazor.Client

Have you solved this problem?


Replace TypeRegistry with specialized class and module spec registries with expanded APIs

Add type parameter for customizing BuildHasher
Expose capacity and build hasher constructors
Expose memory management APIs - reserve, shrink to fit
Add capacity, len, is_empty APIs
Return old values when calling insert
document that TypeRegistry is append only.
add doc comments
derive default, debug, partialeq, eq
Add iterator over items in the registry
Add contains<K>()

This PR enforces that the registries are append only. Duplicate inserts panic. There are no other mutating APIs.


[#1892] Apply type-bind properly to concat fields that are not dynamic arrays

References

Fixes #1892

Description
A bug was discovered with name fields that are not repeatable - they do not obey type bind.
This was found to be because the form control model used is dynamic-concat-model and this model did not specifically set required type bind configuration.
This pull request fixes the issue for concat model, but there may be other form control models around that have the same issue. If more are found in the short term, I will rename this issue / PR and continue committing fixes to other form control models here.
Instructions for Reviewers

See instructions at #1892

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes TSLint validation using yarn run lint
[x] My PR doesn't introduce circular dependencies
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide

Hi @kshepherd,
I tried these changes and they work fine for me as explained in PR 1892

when I change the type they all disappear


feat: Deploy apps in parallel to speedup e2e

What this PR does / why we need it

Run all devenv apps deploy of current app dependencies in parallel to speed up make e2e

Jira ID
[XX-XX]

Notes for your reviewers

This has to be done on devenv side, using --with-deps and supporting parallelization there


update to upstream tendermint 0.24.0-pre.1

this PR unifies the version of tendermint-rs that we are using to 0.24.0-pre.1, across ibc-rs and the main penumbra crates.

@avahowell This looks good, I pushed up an extra commit that removes the hardcoded version from the notes deploy workflow, now that there's just one version.


Istio Echo: Add version string

Add version string in the echo response message. This is to allow using the same image for the example at https://istio.io/v1.12/blog/2021/proxyless-grpc/#in-your-kubernetes-deployment.
I also noticed a major bug where the xDS server was not serving anything earlier.

...
I also noticed a major bug where the xDS server was not serving anything earlier.

Meaning an Istio bug (bug #?) or a bug in grpc (which was not handling the response)?


Bearer plugin doesn't work on Elysia 1.1.0-rc.9

What version of Elysia.JS is running?
1.1.0-rc.9
What platform is your computer?
Linux 6.9.8 x86_64 unknown
What steps can reproduce the bug?
import bearer from "@elysiajs/bearer";

// ...
new Elysia()
    .use(bearer())

What is the expected behavior?
Bearer plugin to work like how it does in 1.0.27
What do you see instead?
2845 | var randomId = () => crypto.getRandomValues(new Uint32Array(1))[0];
2846 | var deduplicateChecksum = (array) => {
2847 |   const hashes = [];
2848 |   for (let i = 0; i < array.length; i++) {
2849 |     const item = array[i];
2850 |     if (item.checksum) {
               ^
TypeError: undefined is not an object (evaluating 'item.checksum')
      at deduplicateChecksum (/home/kainoa/Git/inertia/node_modules/elysia/dist/index.mjs:2850:9)
      at _use (/home/kainoa/Git/inertia/node_modules/elysia/dist/index.mjs:6517:7)
      at /home/kainoa/Git/inertia/packages/backend/src/services/auth.ts:6:3

Additional information
No response

No longer an issue as of the 1.1.0 release!


ofn demo error, can someone help ?

➜  ~ ofn demo --insecure-skip-tls-verify
 -> The OpenFunction demonstration <-
Start launching the cluster.
 ✗ Kind Cluster - Failed!
 ⇲ Error reports:
Failed to download Kind:
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (60) SSL certificate problem: certificate has expired
More details here: https://curl.haxx.se/docs/sslcerts.html

curl performs SSL certificate verification by default, using a "bundle"
 of Certificate Authority (CA) public keys (CA certs). If the default
 bundle file isn't adequate, you can specify an alternate file
 using the --cacert option.
If this HTTPS server uses a certificate signed by a CA represented in
 the bundle, the certificate verification probably failed due to a
 problem with the certificate (it might be expired, or the name might
 not match the domain name in the URL).
If you'd like to turn off curl's verification of the certificate, use
 the -k (or --insecure) option.
HTTPS-proxy has similar options --proxy-cacert and --proxy-insecure.
: exit status 60

This should be a mistake and we need to improve the download process of Kind:

If an available Kind binary already exists, skip the download
Add some parameters to the download process to handle tls and proxy issues
https://github.com/OpenFunction/cli/blob/a4d3be99ab657791b5d5d8e4b120ca93687458bf/pkg/components/linux/linux.go#L202-L216


add allele-specific logs for test

Great stuff, thanks! ref https://github.com/ewels/MultiQC/pull/785


helm: Relax hubble ui image versions validation

This commit is to allow the latest tag for hubble images, as unlike
cilium images, we are still publishing the latest tag.
Related comment https://github.com/cilium/cilium/pull/20036#pullrequestreview-991318933
Relates: #19565
Fixes: #20034
Suggested-By: Joe Stringer<EMAIL_ADDRESS>Suggested-By: Michi Mutsuzaki<EMAIL_ADDRESS>Signed-off-by: Tam Mach<EMAIL_ADDRESS>

/test


Remove PATH from devenv:python:uv task exports.

Removes PATH from the exports of the devenv:python:uv task. PATH is not modified in initUvScript and will conflict with devenv:python:virtualenv PATH export which leads to virtual environments randomly not activating on enterShell
Fixes #1502

See https://github.com/cachix/devenv/pull/1504/files


Indicate where blocks modify the input (e.g. clip max speed)

Assume a block like:
[ Drive forward speed % ] [ 5000 ]

Since the speed value is to be given in percent, we will limit the input to a value from 0...100. It would be nice to indicate the to the user. We could do this with a checker. Ideally we put the warning sign into the number box though.

this is a runtime clipping of values. We can add text in the help like "the value will be clipped to 0...100. But in general, we can't do that with a checker: there may be used an arbitrary expression


Added function to plot PDFs associated with Poly.

@ascillitoe following on from the discourse I added a function to plot all of the PDF's from parameters associated with a polynomial. Please review and clean up as you see fit - no worries if you have other ideas on how to do this and don't use it - I just thought I'd offer this up as I already had most of it written from existing work.
Essentially it does a loop through Polynomial.parameters and calls the existing plot.plot_pdf function. To allow a little bit of customisation I include a cols argument to allow you to specify the number of columns to put the parameters in.
I also noticed a bug in plot_pdf where the defaults were specified in the return making it impossible to change them. I've just removed this so it now gives the expected behaviour.
I also spotted some odd behaviour in the axes from plot_pdf when putting it into a subplot. After some digging this was because of the sns.despine call which was trimming everything. The fix was to add ax=ax into it so it was more targeted.

This pull request has been mentioned on Discourse — equadratures. There might be relevant details there:
https://discourse.equadratures.org/t/variable-naming-in-poly/149/15


test: [M3-7139] - Enable TypeScript typechecking in Cypress directory

Description 📝
This updates the Cypress tsconfig.json file to allow typechecking against files in the Cypress directory, and fixes some TypeScript errors.
Changes  🔄
List any change relevant to the reviewer.

Extend Cypress's tsconfig from Manager's tsconfig, remove a lot of unneeded config
Fix various TypeScript issues

Removed lots of unused imports/consts/etc.
Added missing function param annotations
Fixed a couple issues where incorrect types or properties were specified

Update API-v4 CreateLinodeRequest interfaces property to accept InterfacePayload objects instead of Interface objects

How to test 🧪
We can lean on the automated tests to make sure the e2e tests themselves aren't broken. I did make a really minor change to vite.config.ts, and any issues there should also manifest during the tests, but it might be good to build Cloud and give it a quick look just to make sure there are no unexpected issues.
To run typechecking against the Cypress code, you can run this command from within packages/manager:
yarn tsc -p cypress --noEmit

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Thanks @jdamore-linode! This is exciting. Do you still need help to fix types in here? Happy to jump in in my leisurely time to help with the PR

Wonder if we should add these typechecks to packages/manager/package.json:86:
    "start": "concurrently --raw \"vite\" \"tsc --watch --preserveWatchOutput\"",

This might bring up any new type errors as team members are working on the cloud instead of having to run typechecks manually or relying on test failures.

This is great! 🎉 You may have been use to it @jdamore-linode, but for me it was very frustrating working on the e2es without proper language server help! This is gonna be super nice and help me move faster 🚀

Before
After

I'm not too stressed about any of the "questionable fixes". I think we can iron those out over time.
Regarding what @hkhalil-akamai said about adding tsc to the start command, I'm personally fine without it, but I think I'd be okay with whatever. I think most of the value here for me is the proper language server auto-completion and whatnot as shown in the videos.
I'll continue reviewing and thinking over the changes!

This is great! 🎉 You may have been use to it @jdamore-linode, but for me it was very frustrating working on the e2es without proper language server help!

Interestingly, using Sublime Text with the lsp-typescript plugin, I always had (some) editor annotations which led me to assume that our setup was working as expected -- I'd get imports added automatically when I'd refer to api-v4 types, could see inline type docs when applicable, and so on. I was really surprised by some of the type mistakes that were revealed by this change!

@jdamore-linode  Ahh, very interesting. In VSCode it seemed that things were partially working in general.


Next.js 14: shopify.auth.begin throws TypeError: Cannot read properties of undefined (reading 'getRandomValues

I'm using Next.js 14.0.1 with the app router and get the following error in my auth endpoint if I use the provided next-adapter.ts:
TypeError: Cannot read properties of undefined (reading 'getRandomValues
If I don't use the provided next-adapter.ts and instead use the normal "@shopify/shopify-api/adapters/node" I get the following error:
TypeError: adapterArgs.rawResponse.getHeaders is not a function
Any insight here would be greatly appreciated!
EDIT: this repo is an excellent idea... I wish this was around back when I created my first Shopify app!
// import "@shopify/shopify-api/adapters/node";
import "../../../../lib/adapter";

import { shopifyApi, LATEST_API_VERSION } from "@shopify/shopify-api";
import { NextRequest, NextResponse } from "next/server";

export async function GET(req: NextRequest, res: NextResponse) {
  try {
    const shopify = shopifyApi({
      apiKey: process.env.SHOPIFY_CLIENT_ID as string,
      apiSecretKey: process.env.SHOPIFY_SECRET as string,
      apiVersion: LATEST_API_VERSION,
      scopes: [
        "read_products",
        "write_products",
        "read_orders",
        "write_orders",
        "read_assigned_fulfillment_orders",
        "write_assigned_fulfillment_orders",
        "read_fulfillments",
        "write_fulfillments",
      ],
      hostName: process.env.NEXT_PUBLIC_APP_BASE_URL as string,
      isEmbeddedApp: false,
    });

    const searchParams = req.nextUrl.searchParams;
    const shop = searchParams.get("shop");

    const sanitizedShop = shopify.utils.sanitizeShop(shop as string);

    if (!sanitizedShop) {
      throw new Error("Invalid shop provided");
    }

    return beginAuth(sanitizedShop, req);

    function beginAuth(shop: string, req: Request) {
      return shopify.auth.begin({
        shop,
        callbackPath: "/api/shopify/callbackAuth",
        isOnline: false,
        rawRequest: req,
        rawResponse: new NextResponse(),
      });
    }
  } catch (error: any) {
    console.log(error);

    return new Response(error.message || "An unexpected error occurred", {
      status: 500,
    });
  }
}

well, should've tried updating the shopify-api package before opening this issue, but that seems to have solved it. I'll close this issue if it ends up working.


Macros 1.1 is stable now

See commit https://github.com/nikomatsakis/rust/commit/ce1a38ce20f627e5b6ecd89367102ec786ac30e2
This would be a good chance for num-derive to remove the feature tag.

Care to send a PR that effect?


Legs have object collision but arms have no collision

what would I have to do to make the arms collide with certain objects in the game?
I'm using a quest 2 to test out my game.

Nexus VR Character Model does not affect these properties of the character. This would be a generic question on how to make arms collide with an R15 character, VR or not. Scripting Support on the developer forum would be a better place to ask.


Create a Mixin in your lib

Here my test, but it doesn't works!
Component:
<template>
  <button class="button" type="button" @click="click()">
    <slot>{{ text }}</slot>
  </button>
</template>

<script lang="ts">
import { Component, Prop, Vue } from "vue-property-decorator";

@Component
export default class Button extends Vue {
  @Prop({ type: String, required: true }) readonly text!: string;

  click() {
    this.$emit("click");
  }
}
</script>

Mixin:
import { Component, Vue } from "vue-property-decorator";
@Component
export default class TestMixin extends Vue {
  mixData = "mix data";
}

My Export Index file of package:
export { default as Button } from "./components/Button.vue";

import TestMixin from "@/mixins/TestMixin";

export { TestMixin };

My types definition, generated with: tsc --emitDeclarationOnly -d, Under folder: types/index.d.ts:
import { VueConstructor } from "vue";
export const Button: VueConstructor;
import { Vue } from "vue-property-decorator";
export default class TestMixin extends Vue {
  mixData: string;
}

My package.json line for types:
...
"types": "./types/index.d.ts",
...

In another project, I use my package with:
"dependencies": {
  "vue-component": "file:../vue-component-0.1.0.tgz"
},

I'm able to use component correctly, and it works fine, but if I use mixin, got this error:

Any help???? thanks!

Check out the newly created mixin branch. It seems to work for me.


Statistics: Broken Undo

On Windows 10, Undo does not work when the modal statistics window is active.
To reproduce:

Load any data, open statistics
Change a setting, and try to undo it.  Nothing happens.
Close the statistics window.  Issue an Undo command.
Open the statistics window, and the undo will be applied to the change that was made to the StatisticsParams.

undo/redo is not currently supported for Statistics.


PRF files misinterpreted

The PRF files are "interleaved", but not currently treated as such.  Need to fix!

fixed with v0.1.0


Incorporate Monitor Query Beta 4 UX study feedback

Tracking issue to update the next release to incorporate Beta 4 UX study feedback.
/cc: @joshfree
Action items from 8/24 meeting:

[x] README file changes:

[x] Mention the TenantId column as a way to see the workspace identifier in a multi-workspace Logs query result.
[x] Add samples for the listMetricDefinitions and listMetricNamespaces methods of MetricsQueryClient.
[x] Add a link to the Azure Core PagedIterable documentation, where appropriate.
[x] Update the file's Handle metrics response section so that the casing of property names in the object graph representation match the casing used in the MetricsQueryResult code.
[x] Document the ordering of the results when querying multiple metric names. For example, if I query "SuccessfulCalls" and another metric name, which order do those results appear in?

[x] Eliminate the need to call getName().getValue() to retrieve a metric definition name as a String type. The call to getName() should be sufficient.

[x] Improve the JavaDocs for the context parameter used in the MetricsQueryClient methods, such as queryWithResponse. Nobody knew what the parameter represented.

[x] In the Metric class, update the metricsName parameter to metricName (singular instead of plural).

[x] Add a getMetricByName method to the MetricsQueryResult class. This proposed method would be an alternative to the getMetrics().get(0) call in the following screenshot:

https://github.com/Azure/azure-sdk-for-java/pull/24042


Add motives.

This will resolve #559. The example example/semi-simplicial.prl is not fixed yet, though.

Great! Now it fails properly, as it should. :smile:

@jonsterling One option is to comment out the failing part until we can do them almost automatically. There are instances of #534.

@favonia Yes; I would endorse commenting it out. The proof is very nearly automatic once we deal with the synth stuff.

@favonia (Check out my old exploratory branch which implemented both motives and the improved synthesis, https://github.com/RedPRL/sml-redprl/blob/improve-synth-improved/example/semi-simplicial.prl. I make no guarantees about correctness though)

@jonsterling Hmm I thought I saw some commuting conversions which I doubt can be solved by synth, but the output's so messy and I could not really tell.

Yes, it's really hard to tell... My feeling is that these are the kinds of proofs which we should not be doing: if we really have to reason with output that is this gnarly, we should just change the way that RedPRL works; the synth stuff helps in one respect, and also your idea for improved unfolding of recursive definitions would probably also help a lot.

It may also be that in my experimental branch, I made some other changes to auto that made this work better. I just can't remember, because I was kind of delirious when I wrote it.


Route Planner/Shortest Route Display from/to K-Space

As a future feature, having the ability to see what WH's you have to take to get to a designated location as quickly as possible would be great. Essentially giving you the best exits or shortcuts after factoring in your WH chain to your designated location.

Bumping this since a few people mentioned it to me again. This feature would also go great together with #271 .

Would it be more beneficial to support same or similar API as other mapping tools, so external program like https://github.com/farshield/shortcircuit could use also eve w-space.
Of course, this feature would be good as is too.


Error in handling mouse.

Hi, Saw your post on dev.to, I thought I should try it out. I need converters a lot and google is sometimes just so slow. I am a linux user, so thanks for appimage, I appreciate it very much,
I however have issue mouse input, I discovered when I was trying to switch theme. Mouse click isn't registered on dropdown menu, It just activate window dragging. I can switch theme via keyboard though.
And when I go back from settings, since editor is now out of focus, I can not click to get cursor back on editor. Worse that here, keyboard tabs doesn't always gain focus here (maybe I am counting tabs wrong cause I did get focus once, but never again.
I have a screen recording if I am unclear in explaining.

Hi @Abhinav1217
I will look into it. Please share the recording

caligator-error.zip
There is no sound, But at the end, before closing the caligator, when mouse click didn't give focus to the editor, I was trying to gain focus via tabs, Didn't work here. The theme switching was done by keyboard, not mouse scroll.

I noticed this on Windows as well. After clicking the settings button, it show's the Theme dropdown (which doesn't work) but after clicking the X to close settings you can no longer use the application as you can't get focus back into the entry box

Having the same issue here as well. I'm running Kubuntu 19.04 (KDE), happens both with the AppImage and the deb file.


Any active development?

I'm considering to use your beautiful library in my product. But it seems to be stopped for months. Is it still be developed? Thanks in advance.

:+1:

Development is not active as there is no roadmap of features. I will continue to fix bugs and update dependencies.


chore(deps): jasmine lock sync

v14.4.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Build Errors due to HTTP response code being errors

Here is my cargo.toml file -
[package]
name = "rs-scanner"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
magic = "0.13.0"
rayon = "1.7.0"
snafu = "0.7.4"
vcpkg = "0.2.15"
walkdir = "2.3.3"

[package.metadata.vcpkg]
git = "https://github.com/microsoft/vcpkg"
rev = "4c1db68"
dependencies = ["libmagic"]

Here is what happens when I run cargo vcpkg -v build
PS C:\code-projects\rs-scanner> cargo vcpkg -v build
vcpkg root is C:\code-projects\rs-scanner\target\vcpkg
    Fetching vcpkg
Fetching origin
POST git-upload-pack (165 bytes)
From https://github.com/microsoft/vcpkg
 = [up to date]          master     -> origin/master
    Checkout rev 4c1db68
HEAD is now at 4c1db6840 [python3] Fix dynamic build error on Linux (#11489)
-- stdout --
Your feedback is important to improve Vcpkg! Please take 3 minutes to complete our survey by running: vcpkg contact --survey
Using local portfile versions. To update the local portfiles, use `git pull`.
No packages need updating.

-- stderr --

ExitStatus(ExitStatus(0))
  Installing libmagic libmagic
Your feedback is important to improve Vcpkg! Please take 3 minutes to complete our survey by running: vcpkg contact --survey
Computing installation plan...
The following packages will be built and installed:
    libmagic[core]:x86-windows
   Compiling libmagic (triplet x86-windows)
Starting package 1/1: libmagic:x86-windows
Building package libmagic[core]:x86-windows...
-- Using cached C:/code-projects/rs-scanner/target/vcpkg/downloads/file-file-a0d5b0e4e9f97d74a9911e95cedd579852e25398.tar.gz
-- Using source at C:/code-projects/rs-scanner/target/vcpkg/buildtrees/libmagic/src/9852e25398-4363addce8
-- Acquiring MSYS2...
-- Downloading https://sourceforge.net/projects/msys2/files/Base/x86_64/msys2-base-x86_64-20190524.tar.xz/download...
-- Downloading https://sourceforge.net/projects/msys2/files/Base/x86_64/msys2-base-x86_64-20190524.tar.xz/download... Failed. Status: 22;"HTTP response code said error"
-- Downloading http://repo.msys2.org/distrib/x86_64/msys2-base-x86_64-20190524.tar.xz...
-- Downloading http://repo.msys2.org/distrib/x86_64/msys2-base-x86_64-20190524.tar.xz... Failed. Status: 22;"HTTP response code said error"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:175 (message):

      Failed to download file.
      If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
      variables to "https://user:password@your-proxy-ip-address:port/".
      Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
  scripts/cmake/vcpkg_acquire_msys.cmake:78 (vcpkg_download_distfile)
  scripts/cmake/vcpkg_configure_make.cmake:161 (vcpkg_acquire_msys)
  ports/libmagic/portfile.cmake:9 (vcpkg_configure_make)
  scripts/ports.cmake:90 (include)

Error: Building package libmagic:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: libmagic:x86-windows
  Vcpkg version: 2020.02.04-nohash-external

Additionally, attach any relevant sections from the log files above.
cargo-vcpkg: failed

It tells me to update to latest portfiles. But I cannot run cargo vcpkg update because that is an invalid command. I don't have vcpkg installed globally and I'm using the cargo version to install libmagic for my project and use it only for the project. How do I achieve this?

Hi,
This is a common problem that is often easy to solve, although we don't do a good job of making it obvious how to do so. You probably need to update your Cargo.toml to point to the latest version of the vcpkg repository, changing this line:
rev = "4c1db68"

to the most recent revision ("223b8d5" right now). If I remember correctly you can also change it to
rev = "master"

so that you are always trying to use the latest definitions. This would be less reproducible, since every build would be using whichever version of vcpkg happens to be active, but would potentially be a bit easier to maintain. Empirically, every few months it feels like one needs to update the vcpkg definitions to track changes in the msys2 files.

Yep, this works. Can I use my installed libraries from my code directly after running cargo vcpkg build or do I have to change environment variables?

I used the updated rev field as follows -
[package.metadata.vcpkg]
git = "https://github.com/microsoft/vcpkg"
rev = "master"
dependencies = ["libmagic"]

I can run cargo vcpkg build successfully now, it creates a new directory called vcpkg with appropriate libraries listed.
However, when I try to look for the library using the vcpkg crate as follows -
fn main() -> std::io::Result<()> {
    println!(
        "{:?}",
        vcpkg::Config::new()
            .emit_includes(true)
            .find_package("libmagic")
    );
    Ok(())
}

It prints the following -
Err(NotMSVC)

What is this error? The website for the vcpkg crate describes it poorly.

@nouritsu Are you running your vcpkg code in a build.rs script? The cargo-vcpkg notes.md file mentions as a TODO:

add a better error message than the spurious NotMSVC if find_package gets run from an app rather than in build.rs


Problems Installing XDebug

When installing XDebug this happens. It's been like this for over 10 minutes and nothing happens, now sure how to "debug" this issue.

sorry, but i also no idea for this issue.  Since the command already has -verbose added, brew already outputs the execution details.
You can copy the command and run it yourself in a terminal, and see if you get anything.
The first line in the log is the actual execution command
brew install —verbose<EMAIL_ADDRESS>

Thanks I'll figure it out :)


Adds *_hostname_enabled vars for explicit opt-in to hostnames

what

Adds domain_hostname_enabled and kibana_hostname_enabled var flags to fix an issues with count when dns_zone_id is not being passed static.

why

Implicit opt-in through checking dns_zone_id was causing folks problems as count / enabled need to be calculated during the plan which caused using -target.
This fixes needing a targeted apply when passing dns_zone_id from a resource or module reference.

references

Fixes + Closes #13
Discussion in SweetOps: https://sweetops.slack.com/archives/CB6GHNLG0/p1600294860394500

/test all

/test terratest

/test terratest

/test all

@Gowiem I'm going to update the Terratest to prevent failures like that

@aknysh I believe the failure here was because I added a new flag that is off by default and therefore the previous way of enabling hostnames was ignored, subdomains were not created, and the actual tests that check subdomain output failed. Are you thinking there is another issue outside of that regarding the need for unique subdomain names or something else?

@aknysh I believe the failure here was because I added a new flag that is off by default and therefore the previous way of enabling hostnames was ignored, subdomains were not created, and the actual tests that check subdomain output failed. Are you thinking there is another issue outside of that regarding the need for unique subdomain names or something else?

Yes, we need to add random attributes to the test, because the prev failure was b/c some resources with the same names already existed from the previous failures.
I'll add context.tf as well

@aknysh Gotcha. Do your thing and thanks for the assist.

/test all


update to support latest version of terraform

this commit includes several fixes that I applied in order to get the template working with my version of terraform (1.2.6). specifically:

the "prefix" variable is now referred to with var. syntax instead of local. syntax. without this, npm run plan was unable to resolve the "prefix" variable and failed.
I've updated package.json to add a npm run destroy command, which was mentioned in the readme but seems to have been omitted
I've replaced the "deploy" command with "apply" in  package.json, and changed the terraform command that is run. my version of terraform does not recognize a "deploy" command, perhaps this command has been deprecated?

thanks for creating this template! I found it very helpful.


[feat]: Add dynamic route

Fixes Issue #61
It works but ...

I've never used Chakra UI before so I don't know if I made some sort of error with it

I couldn't use getEvents() in getStaticPaths & getStaticProps

I tried to use next/font for the local fonts but, it didn't work so I just used fonts the CSS way

In _app.tsx, Line 22
<Component pageProps={...pageProps} />

I think this is wrong, and VSCode agrees with me. It should be:
<Component {...pageProps} />

or
<Component pageProps={pageProps} />

but I left it alone sha incase

About the fonts thingy. I think we should use Google fonts, instead of adding local fonts via CSS.
Then you can come to this place and include them as scoped CSS vars
https://github.com/kaf-lamed-beyt/ng-tech-events/blob/b0b5501a0afd9149006afe9e8970b689d6d5b373/frontend/style/globals.scss#L3-L24

These fonts (ClashDisplay and SFProDisplay) aren't Google fonts and I couldn't find a cdn that hosted them. I had to download them from a zip file.

First off @Jemeni11, sorry for circling back this late.
To fix the issue of the fonts and to reduce the way we bloat the repo with local files. I think it'll be nice to have just one font. Let's go in with the "Inter" font, for uniformity across the design.
Sorry that you had to go the stress of setting it up at first. I apologize for not communicating it clearly.

Hi @Jemeni11 can you check the "allow edits by maintainers " checkbox?
I made some changes locally to the PR but, I can't push yet.

Oh wait! It is checked actually.


fix DB connections leak

Every time the plugin is re-deployed, it is leaking the previous DB connections, up to the point where it depletes the available DB connections (especially in a Development mode with a limited number of DB Connections available, but the same would be true for Production), and the DB log would show something like this:

And Keycloak would show the error "We are sorry... An internal server error has occurred":

e.g. Checking the number of connections on a Postgresql DB, running SELECT sum(numbackends) FROM pg_stat_database; would show that 10 connections are leaked on each re-deploy.

Did it actually solved the problem? I think it might have yet some other issue regarding connection cleanup. I will further review the code for connection management issues. Thank you.

Yes, it did solve this specific issue.  I was able to confirm the exact number of connections that were added on each re-deploy without the fix, and to confirm that it (the number of connections increasing) was no longer the case with the fix.  This doesn't mean there aren't other connection cleanup problems to be found, but it fixed this specific one.  Thanks.


unable to build packages

I for some reason cannot run a colcon build without errors anymore.
I have removed the workspace and redownloaded all sourcecode again.
This is the errors I am getting:
Starting >>> dynamixel_sdk
Starting >>> cmake_modules
Starting >>> open_manipulator_msgs
Starting >>> dynamixel_sdk_custom_interfaces
Starting >>> open_manipulator_x_description
Finished <<< cmake_modules [0.08s]
Starting >>> robotis_manipulator
Finished <<< open_manipulator_x_description [0.09s]
Finished <<< dynamixel_sdk [0.12s]
Starting >>> dynamixel_workbench_toolbox
--- stderr: open_manipulator_msgs
make[1]: *** No rule to make target 'open_manipulator_msgs__py/CMakeFiles/open_manipulator_msgs__py.dir/all', needed by 'CMakeFiles/open_manipulator_msgs__python.dir/all'.  Stop.
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:146: all] Error 2
Failed   <<< open_manipulator_msgs [0.13s, exited with code 2]
Aborted  <<< dynamixel_workbench_toolbox [0.28s]
Aborted  <<< dynamixel_sdk_custom_interfaces [0.50s]
Aborted  <<< robotis_manipulator [4.35s]
Summary: 3 packages finished [4.57s]
1 package failed: open_manipulator_msgs
3 packages aborted: dynamixel_sdk_custom_interfaces dynamixel_workbench_toolbox robotis_manipulator
2 packages had stderr output: dynamixel_workbench_toolbox open_manipulator_msgs
6 packages not processed
Any help would be greatly appreciated.

I also get this error:
Starting >>> dynamixel_sdk
Starting >>> cmake_modules
Starting >>> open_manipulator_msgs
Starting >>> dynamixel_sdk_custom_interfaces
Starting >>> open_manipulator_x_description
Finished <<< cmake_modules [0.25s]
Starting >>> robotis_manipulator
Finished <<< open_manipulator_x_description [0.25s]
Finished <<< dynamixel_sdk [0.28s]
Starting >>> dynamixel_workbench_toolbox
--- stderr: dynamixel_sdk_custom_interfaces
make[1]: *** No rule to make target 'dynamixel_sdk_custom_interfaces__py/CMakeFiles/dynamixel_sdk_custom_interfaces__py.dir/all', needed by 'CMakeFiles/dynamixel_sdk_custom_interfaces__python.dir/all'.  Stop.
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:146: all] Error 2
Failed   <<< dynamixel_sdk_custom_interfaces [0.55s, exited with code 2]
Aborted  <<< robotis_manipulator [0.37s]
Aborted  <<< dynamixel_workbench_toolbox [0.37s]
Aborted  <<< open_manipulator_msgs [0.74s]
Summary: 3 packages finished [0.88s]
1 package failed: dynamixel_sdk_custom_interfaces
3 packages aborted: dynamixel_workbench_toolbox open_manipulator_msgs robotis_manipulator
1 package had stderr output: dynamixel_sdk_custom_interfaces
6 packages not processed

Apologies for the delayed response. We have assembled a TF team to address OpenManipulator-x issues and are actively working on resolutions. Please note that we do not support EOL (End-of-Life) ROS versions; our support currently extends to Noetic and Humble versions. We kindly ask that you try again with the latest version. If the same issue arises, please feel free to resubmit it, and we will address it promptly.


[Feature] Support for / searching

Thank you for the awesome plugin!
Are there plans to add support for searching with? and /?
It would complement well with the use of vim.o.cmdheight = 0.
Side node:
I also use rcarriga/nvim-notify to replace notifications, nvim-telescope/telescope-ui-select.nvim to replace selections and stevearc/dressing.nvim to replace inputs/renames.

Telescope already has a built-in feature for search_history and I didn't thought in this scenario. However, it doesn't let you search a new pattern and I would definitely explore this idea. Thanks for sharing.

I've been experimenting with require("telescope.builtin").current_buffer_fuzzy_find(). So far, it's been quite handy, although i've noticed that finding next and previous occurrences isn't working as expected. I've documented this issue here: https://github.com/nvim-telescope/telescope.nvim/issues/2970


[wt] Build error

Package: wt[core,openssl]:x64-windows -> 4.7.1
Host Environment

Host: x64-windows
Compiler: MSVC 19.32.31332.0
vcpkg-tool version: 2022-07-21-a0e87e227afb536c62188c11ad029954f28fdb22
vcpkg-scripts version: 66045de4d 2022-07-29 (30 hours ago)

To Reproduce
vcpkg install wt:x64-windows
Failure logs
-- Using cached emweb-wt-d0e9f2e8096a1acb4558130a851812dd424d8f3e.tar.gz.
-- Extracting source C:/Program Files/vcpkg/downloads/emweb-wt-d0e9f2e8096a1acb4558130a851812dd424d8f3e.tar.gz
-- Applying patch 0002-link-glew.patch
-- Applying patch 0005-XML_file_path.patch
-- Applying patch 0006-GraphicsMagick.patch
-- Using source at C:/Program Files/vcpkg/buildtrees/wt/src/dd424d8f3e-a34424d85e.clean
-- Found external ninja('1.10.2').
-- Configuring x64-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:96 (message):
    Command failed: "C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" -v
    Working Directory: C:/Program Files/vcpkg/buildtrees/wt/x64-windows-rel/vcpkg-parallel-configure
    Error code: 1
    See logs for more information:
      C:\Program Files\vcpkg\buildtrees\wt\config-x64-windows-out.log

Call Stack (most recent call first):
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake:238 (vcpkg_execute_required_process)
  ports/wt/portfile.cmake:46 (vcpkg_cmake_configure)
  scripts/ports.cmake:147 (include)

C:\Program Files\vcpkg\buildtrees\wt\config-x64-windows-out.log
[1/2] "C:/Program Files/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" -E chdir "../../x64-windows-dbg" "C:/Program Files/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" "C:/Program Files/vcpkg/buildtrees/wt/src/dd424d8f3e-a34424d85e.clean" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=C:/Program Files/vcpkg/packages/wt_x64-windows/debug" "-DINSTALL_CONFIG_FILE_PATH="C:/Program Files/vcpkg/downloads/wt"" "-DSHARED_LIBS=1" "-DBOOST_DYNAMIC=1" "-DDISABLE_BOOST_AUTOLINK=ON" "-DBUILD_EXAMPLES=OFF" "-DBUILD_TESTS=OFF" "-DWTHTTP_CONFIGURATION=" "-DCONFIGURATION=" "-DCONNECTOR_HTTP=ON" "-DENABLE_HARU=ON" "-DHARU_DYNAMIC=1" "-DENABLE_MYSQL=OFF" "-DENABLE_FIREBIRD=OFF" "-DENABLE_QT4=OFF" "-DENABLE_QT5=OFF" "-DENABLE_LIBWTTEST=ON" "-DENABLE_OPENGL=ON" "-DENABLE_LIBWTDBO=OFF" "-DENABLE_POSTGRES=OFF" "-DENABLE_SQLITE=OFF" "-DENABLE_MSSQLSERVER=OFF" "-DENABLE_SSL=ON" "-DWT_WRASTERIMAGE_IMPLEMENTATION=Direct2D" "-DCONNECTOR_ISAPI=ON" "-DENABLE_PANGO=OFF" "-DUSE_SYSTEM_SQLITE3=ON" "-DUSE_SYSTEM_GLEW=ON" "-DCMAKE_INSTALL_DIR=share" "-DWTHTTP_CONFIGURATION=" "-DCONFIGURATION=" "-DCMAKE_MAKE_PROGRAM=C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=C:/Program Files/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v143" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=C:/Program Files/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=C:/Program Files/vcpkg" "-D_VCPKG_INSTALLED_DIR=C:/Program Files/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF"
FAILED: ../../x64-windows-dbg/CMakeCache.txt
"C:/Program Files/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" -E chdir "../../x64-windows-dbg" "C:/Program Files/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" "C:/Program Files/vcpkg/buildtrees/wt/src/dd424d8f3e-a34424d85e.clean" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=C:/Program Files/vcpkg/packages/wt_x64-windows/debug" "-DINSTALL_CONFIG_FILE_PATH="C:/Program Files/vcpkg/downloads/wt"" "-DSHARED_LIBS=1" "-DBOOST_DYNAMIC=1" "-DDISABLE_BOOST_AUTOLINK=ON" "-DBUILD_EXAMPLES=OFF" "-DBUILD_TESTS=OFF" "-DWTHTTP_CONFIGURATION=" "-DCONFIGURATION=" "-DCONNECTOR_HTTP=ON" "-DENABLE_HARU=ON" "-DHARU_DYNAMIC=1" "-DENABLE_MYSQL=OFF" "-DENABLE_FIREBIRD=OFF" "-DENABLE_QT4=OFF" "-DENABLE_QT5=OFF" "-DENABLE_LIBWTTEST=ON" "-DENABLE_OPENGL=ON" "-DENABLE_LIBWTDBO=OFF" "-DENABLE_POSTGRES=OFF" "-DENABLE_SQLITE=OFF" "-DENABLE_MSSQLSERVER=OFF" "-DENABLE_SSL=ON" "-DWT_WRASTERIMAGE_IMPLEMENTATION=Direct2D" "-DCONNECTOR_ISAPI=ON" "-DENABLE_PANGO=OFF" "-DUSE_SYSTEM_SQLITE3=ON" "-DUSE_SYSTEM_GLEW=ON" "-DCMAKE_INSTALL_DIR=share" "-DWTHTTP_CONFIGURATION=" "-DCONFIGURATION=" "-DCMAKE_MAKE_PROGRAM=C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=C:/Program Files/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v143" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=C:/Program Files/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=C:/Program Files/vcpkg" "-D_VCPKG_INSTALLED_DIR=C:/Program Files/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF"
CMake Error: The source directory "C:/Program Files/vcpkg/buildtrees/wt/x64-windows-dbg/Files/vcpkg/downloads/wt" does not exist.
Specify --help for usage, or press the help button on the CMake GUI.
[2/2] "C:/Program Files/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" -E chdir ".." "C:/Program Files/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" "C:/Program Files/vcpkg/buildtrees/wt/src/dd424d8f3e-a34424d85e.clean" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=C:/Program Files/vcpkg/packages/wt_x64-windows" "-DINSTALL_CONFIG_FILE_PATH="C:/Program Files/vcpkg/downloads/wt"" "-DSHARED_LIBS=1" "-DBOOST_DYNAMIC=1" "-DDISABLE_BOOST_AUTOLINK=ON" "-DBUILD_EXAMPLES=OFF" "-DBUILD_TESTS=OFF" "-DWTHTTP_CONFIGURATION=" "-DCONFIGURATION=" "-DCONNECTOR_HTTP=ON" "-DENABLE_HARU=ON" "-DHARU_DYNAMIC=1" "-DENABLE_MYSQL=OFF" "-DENABLE_FIREBIRD=OFF" "-DENABLE_QT4=OFF" "-DENABLE_QT5=OFF" "-DENABLE_LIBWTTEST=ON" "-DENABLE_OPENGL=ON" "-DENABLE_LIBWTDBO=OFF" "-DENABLE_POSTGRES=OFF" "-DENABLE_SQLITE=OFF" "-DENABLE_MSSQLSERVER=OFF" "-DENABLE_SSL=ON" "-DWT_WRASTERIMAGE_IMPLEMENTATION=Direct2D" "-DCONNECTOR_ISAPI=ON" "-DENABLE_PANGO=OFF" "-DUSE_SYSTEM_SQLITE3=ON" "-DUSE_SYSTEM_GLEW=ON" "-DCMAKE_INSTALL_DIR=share" "-DWTHTTP_CONFIGURATION=" "-DCONFIGURATION=" "-DCMAKE_MAKE_PROGRAM=C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=C:/Program Files/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v143" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=C:/Program Files/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=C:/Program Files/vcpkg" "-D_VCPKG_INSTALLED_DIR=C:/Program Files/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF"
FAILED: ../CMakeCache.txt
"C:/Program Files/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" -E chdir ".." "C:/Program Files/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe" "C:/Program Files/vcpkg/buildtrees/wt/src/dd424d8f3e-a34424d85e.clean" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=C:/Program Files/vcpkg/packages/wt_x64-windows" "-DINSTALL_CONFIG_FILE_PATH="C:/Program Files/vcpkg/downloads/wt"" "-DSHARED_LIBS=1" "-DBOOST_DYNAMIC=1" "-DDISABLE_BOOST_AUTOLINK=ON" "-DBUILD_EXAMPLES=OFF" "-DBUILD_TESTS=OFF" "-DWTHTTP_CONFIGURATION=" "-DCONFIGURATION=" "-DCONNECTOR_HTTP=ON" "-DENABLE_HARU=ON" "-DHARU_DYNAMIC=1" "-DENABLE_MYSQL=OFF" "-DENABLE_FIREBIRD=OFF" "-DENABLE_QT4=OFF" "-DENABLE_QT5=OFF" "-DENABLE_LIBWTTEST=ON" "-DENABLE_OPENGL=ON" "-DENABLE_LIBWTDBO=OFF" "-DENABLE_POSTGRES=OFF" "-DENABLE_SQLITE=OFF" "-DENABLE_MSSQLSERVER=OFF" "-DENABLE_SSL=ON" "-DWT_WRASTERIMAGE_IMPLEMENTATION=Direct2D" "-DCONNECTOR_ISAPI=ON" "-DENABLE_PANGO=OFF" "-DUSE_SYSTEM_SQLITE3=ON" "-DUSE_SYSTEM_GLEW=ON" "-DCMAKE_INSTALL_DIR=share" "-DWTHTTP_CONFIGURATION=" "-DCONFIGURATION=" "-DCMAKE_MAKE_PROGRAM=C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=C:/Program Files/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v143" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=C:/Program Files/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=C:/Program Files/vcpkg" "-D_VCPKG_INSTALLED_DIR=C:/Program Files/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF"
CMake Error: The source directory "C:/Program Files/vcpkg/buildtrees/wt/x64-windows-rel/Files/vcpkg/downloads/wt" does not exist.
Specify --help for usage, or press the help button on the CMake GUI.
ninja: build stopped: subcommand failed.

The source directory "C:/Program Files/vcpkg/buildtrees/wt/x64-windows-dbg/Files/vcpkg/downloads/wt" does not exist
There's a space in the path. A lot of software still do not properly escape paths, so the general recommendations are:

Don't use paths that contain non-ascii symbols or spaces
Don't use long paths in windows

@sabotzs, please reopen this issue if the solution provided by Osyotr didn't solve your problem.


Documentation Update

This should allow users to quickly set up and understand the project.

[x] Either improve the README.md or add dedicated GitHub Pages based documentation.

Created a documentation config using MKDocs.

The demo config site along with CI workflow is now deployed and available at : https://aquastream.github.io/waterlevels_oker/
Lesss goooo 🎉🎉


Instructions for installing via Scoop

Current State of the Meshery.io site
mesheryctl is available via Scoop as a Windows-based package manager.
The meshery.io site doesn't provide users the option of using Scoop to install Meshery.

Desired State of the Meshery.io site
Scoop should be listed as an option under the Windows section. See this repo as a resource - https://github.com/layer5io/scoop-bucket and the https://docs.meshery.io site.

Contributor Resources

See the Contributing to Meshery.io Website section of the readme.md.

@metonymic-smokey interested in picking this up?

@leecalcote ,
Yes! Give me a week to submit a PR.
Thanks for the consideration!

Sounds real good. You bet. 👍


Change Item on Drag

Similar to issue #265. I would like to change the item being dragged. IE: When dragging a <li> item, change it to a .
I was thinking of a adding a copyFrom option where the copyFrom is the ID of an hidden element? I only need it when the copy=true. I adding this option when the copy=false would be tricky as you would have to swap the item back, if the drop failed.

Hey, so you are saying, you have for e.g. a some draggable <li>s and on drag want to change it to an <img>?
This sounds like a very specific case/edge case to me. Maybe it would be better to somehow do this in the onDragStart event? You could check if the copy option is set to true and than do it, correct? Do you think that would be a good solution as well?
If so, maybe you can add a recipes sections or something to the readme and add this example?
I just want to make sure we don't add to many things and complicate the whole package. 😉
However feel free to provide some arguments why it would be better to add this as a specific option and we can discuss it.

@mfriesen do you think your issue is the same as #289?

Yes, I believe so. Basically I'm creating a toolbar and when you drag the toolbar item, I need it to expand as the type of item it is dropping. I will try and see if onDragStart will work.

onDragStart is the right time but I ran into a problem. The sortstart only fires if the source sortable is connected to the drop sortable. IE: If list1 is acceptFrom: false and list2 is acceptFrom: .list1. The event will not fire. I am not entirely sure why?
For example in the acceptFrom example. If you add
sortable('.js-sortable-oneway')[0].addEventListener('sortstart', function(e) {
    	console.log("sortstart....")
	});

This will never fire.

Hmm, this seems to be because of _dispatchEventOnConnected https://github.com/lukasoppermann/html5sortable/blob/master/src/html.sortable.js#L433-L438 although I don't see the sense in it. I guess we should carefully remove it.

I agree. I think creating a _dispatchEvent function and calling that from sortstart would be fine and I don't think it would break anything. Let me know if you want me to help out.

Hey, @mfriesen if you could help out that would be really neat.

@lukasoppermann Do you think it makes sense to add a placeholderDrag option? Much like the current placeholder option. This would change what you are dragging? I tried to change the icon from the onDragStart but I couldn't get it to work.

Hey @mfriesen are you talking about the ghost? As in the half transparent thing that is shown when you drag something?
This could definitely make sense, however I believe it is a little bit more complicated, as you can not change it directly. What you would need to do is create an item and assign this to the ghost. Basically like this: https://github.com/lukasoppermann/nestable/blob/master/src/nestable.js#L150-L173
However, the linked code is very old and just an inspiration. 😉
If you want to look into it, that would be very neat, but please in a separate PR (I prefer small PRs, as they are much easier & faster to review).
Thank you!

@lukasoppermann yes. that's what I want. When I have time I'll look into it.

@lukasoppermann I'm thinking just overriding the setDragImage to a custom image is enough. Something adding options for
dragImage, dragImageWidth, dragImageHeight.
Thoughts?

Hey, generally that makes sense.
However if I remember correctly you can not change it once it is set in the setDragImage function.
So to achieve this the best would probably be to let the user define a custom function for _makeGhost which returns an element.
You can not influence the opacity as far as I know.
Does this approach make sense to you? Is this what you were suggesting?
I think it is more future prove to let the user define makeGhost(draggedElement, sortable) and do all the magic she wants return an element that will be set to the DragImage. The dragged and the sortable might be useful arguments, what do you think?

Hey, did you start on this already? I was thinking that maybe it would make sense to go all the way: hiding the ghost and add a custom elements that follows the mouse. This would enable to do more things on the future like animations. What do you think?

No I haven't done anything yet. That sounds pretty cool. I think that is beyond my javascript skills. For me just being able to customize the dragimage would be enough.

Hey @mfriesen, no problem. Feel free to implement the custom dragimage and we can enhance it later on if we want. It is probably good to have both options in any case, because there might be some issues with the advance version for some people.

This will fix #271


Fix flaky TestSubscribe test in pkg/util/channel

I have observed the test failing in Github CI, and I was able to reproduce locally by setting GOMAXPROCS=2.
When using the race detector (-race flag), it usually takes 25ms for the condition function in assert.Eventually to run successfully on my laptop. However, it sometimes spikes to 100+ms, and then the test fails, as the timeout is set to 100ms.
My take is that there could be too much contention between the channel logic and the goroutine performing the checks. The issue only occurs when using the race detector.
To avoid the issue, we use a timeout of 500ms and we set the tick period to 50ms. This is more than sufficient on my machine. If we see the failure in CI again, we can always double these values.
We also fix the error log for the test, as the previous log would never print meaningful values: the log arguments are evaluatd when assert.Eventually is called, and not after the assertion fails.

/skip-all


aws quick starts blog preview is much longer than the other previews

noticed the aws quick starts blog has a much longer preview than the rest.. not sure whats going on
https://www.pulumi.com/blog/

@cnunciato do you know whats up with this?

Not offhand, will take a look.


[Addon: info - v5.x] Story Source breaks if using React.forwardRef

Describe the bug
In version 5.0 of the info addon, Story Source is breaking if the component is wrapped in forwardRef
This additionally breaks the parsing of props in TypeScript (and maybe JavaScript, have not been able to test).
Here are some collapsable sections for the files in question:
Component
import React, { forwardRef } from 'react';

export interface TestButtonProps {
  /**
   * Some input prop.
   * @default: "Default value"
   */
  someProp?: string;
  /**
   * Some onClick handler prop.
   */
  onClick?: () => void;
}

export const TestButton = forwardRef<HTMLButtonElement, TestButtonProps>(
  ({ someProp = 'Default value', onClick }, ref) => {
    return (
      <button onClick={onClick} ref={ref}>{someProp}</button>
    );
  },
);

Version 4.x Story Source (GOOD)
<TestButton
  someProp="Hello, World!"
  onClick={onClick}
/>

Version 5.x Story Source (BAD)
<ForwardRef
  someProp="Hello, World!"
  onClick={action}
>
  <button onClick={action}>
    Hello, World!
  </button>
</ForwardRef>

To Reproduce

Create any component using forwardRef in React.

Expected behaviour
Should match up with the result of version 4.x.
System

OS: MacOS
Device: Macbook Pro 2018
Browser: all
Framework: React
Addons: info
Version: 5.0
Language: TypeScript

Note: this result is shown when using TypeScript and react-docgen-typescript-loader. I haven't been able to test with JavaScript, but since the issue seems to be around React.forwardRef, I suspect the same issue will be there

@shilman I think this is addon: info not storysource, unless storysource is used internally by info.

Related: https://github.com/storybooks/storybook/pull/4961
Looks like this needs to get ported to v5

Looks like I have this same issue. 4.1.13 on the left hand side, 5.0.1 on the right hand side.

+1
also props are not generated when i use React.forwardRef

+1

Is this also why import styled from '@emotion/styled' broke?

This is being caused by PR https://github.com/storybooks/storybook/pull/4961 and I'm hoping I have a proposed solution up at the moment.

@elie222 I'm assuming yes, the original reason I started investigating this was due to inconsistencies between v4 and v5 using styled-components.

Yippee!! I just released https://github.com/storybooks/storybook/releases/tag/v5.1.0-alpha.19 containing PR #6222 that references this issue. Upgrade today to try it out!
Because it's a pre-release you can find it on the @next NPM tag.

Works like charm 🎉

Thank you guys 👏

The table component is still not working in my build.
Is this fix also for the table? Or is it my build.. using CRA, Storybook and styled components.
"dependencies": {
"react": "^16.8.4",
"react-dom": "^16.8.4",
"styled-components": "^4.1.3",
"styled-normalize": "^8.0.6"
},
"devDepedencies": {
"@babel/core": "^7.3.4",
"@storybook/addon-actions": "^5.1.0-alpha.19",
"@storybook/addon-info": "^5.1.0-alpha.19",
"@storybook/addon-knobs": "^5.1.0-alpha.19",
"@storybook/addon-links": "^5.1.0-alpha.19",
"@storybook/addon-storyshots": "^5.1.0-alpha.19",
"@storybook/addons": "^5.1.0-alpha.19",
"@storybook/react": "^5.1.0-alpha.19",
"babel-loader": "^8.0.5",
"react-docgen-typescript-loader": "^3.0.1",
"react-scripts": "2.1.5",
"require-context.macro": "^1.0.4",
"typescript": "3.3.1",
...
}

@marcel-k  Yeah I'm noticing the same thing, except that for mine, there just isn't a Prop Types section at all. Hopefully this can be resolved as well. @Krustal @shilman

Whoopee!! I just released https://github.com/storybooks/storybook/releases/tag/v5.0.6 containing PR #6222 that references this issue. Upgrade today to try it out!


nutgram:listen command

Hello.
Thanks for your awesome package. it saved me a lot of time.
nutgram:run command In laravel doesn't get updated upon change on the app. Currently, I have to restart the command each time I make changes in the app when developing. It would be nice if we had a nutgram:listen or nutgram:watch command to run the app while watching changes. that comes in handy when developing.

I have created this feature in my local project. I can make a PR If you agree with this feature.


Fix incorrect usage of flush on encoder

/cc @pranavkm

:shipit:

:watch: for one question

:shipit: