Can't parse pubDate and else info if it is CDATA encoding

Hello
I have a rss feed. It seems to be CDATA encoding. Its pubDate is like <![CDATA[ 2018-12-21 11:27:25 +0800 ]]>.
When I use rssFeed?.items?.first?.pubDate, it returns nil.

Hi, @RayZhao1998
Is this still an issue?

I find it odd that the date is inside a CDATA Block, but even then, that shouldn't stop FeedKit to attempt to parse the feed.

Also, the date doesn't seem to be an RFC822 formatted date. Is this a public feed? If so, could you please share it?

Thanks


rig: fix darwin build

Motivation for this change
ZHF: #122042
cc @NixOS/nixos-release-managers
Hydra failure: https://hydra.nixos.org/build/142942727/nixlog/1
Things done

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

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

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

Result of nixpkgs-review pr 123499 at 558b42cb run on x86_64-linux 1

2 packages built successfully:

rig
tests.trivial

1 suggestion:

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/tools/misc/rig/default.nix:29:5:
   |
29 |     license = lib.licenses.gpl2;
   |     ^


Mathf.SmoothDampAngle()

Great writeup! Do you know if Mathf.SmoothDampAngle() suffers from the same issues?

Yes, all the SmoothDamp functions call down to the same code


Contents of .mat files

Hello Xiangyu-Gao,
I was going through the paper along with the codes. I haven't finished up reading the paper but while going through the data from FMCW I had a question related to it.
When I try to view the contents of .mat files I know it is a dictionary with arrays as one of the following values. After looking at those array values it seems they are mixed characters (not specifically integer or float).
Example :-
[-127.-17.j  -63.-13.j  -34. +3.j ...  -10.-14.j  -10. +8.j  -26. +2.j]
[  81. +3.j   73. +6.j   69.+25.j ...    4. +5.j    3.+23.j  -17.+11.j]
[-129.+68.j  -73.+43.j  -62.+56.j ...    2. +0.j    2.+23.j  -16.+18.j]
Keeping in mind you have knowledge about FMCW after working on it, can you explain what is the output format from FMCW and why the array has mixed characters?

Thanks for reaching out. The data format is the complex number so when you see for example -127.-17.j.  It means a complex number with real part -127.000000 and imag part -17.000000. Note that I keep each part as float double format.
Hope this helps.
Xiangyu


Corrupted colspan makes columns appear in top left corner of document.

        Test
        8 570
        Test

        1 449
        1 187

This also doesn't seem to work properly:

Property

Gateway Complex (O) 1 Macquarie Place, Syd NSW

Permitted Use

Arcade Games

Licensed Area

O-GATE-DS Budget

Site Size

3m x 3m

Licence hours

Standard Hours 8am to 6pm (Monday - Friday)

Commencement Date

Monday, 19 September 2016

Licence Expiry Date

Saturday, 18 March 2017

Set Up Time

Monday, 19 September 2016 08:00

Removal Time

Sunday, 25 September 2016 17:30

Licence FEE

$0.00 (incl GST) payable in advance

Security Bond

($ / NA)

General Acknowledgement

The Licensor grants the Licensee non-exclusive access to use the Licensed Area on the Terms and Conditions of this Licence. This Licence comprises the whole of the agreement between the parties. The Licensee may only access the Licensed Area for the Permitted Use during the Licence Hours or otherwise as permitted by the Licensor at its sole discretion. The Licensee acknowledges that no representation or warranty as to the suitability of the Licensed Area for the purpose or use of the Licensee has been given by the Licensor to the Licensee. This Licence does not confer any estate or interest in the Licensed Area or the Property or create the relationship of landlord and tenant between the parties, and any such implications or inference is hereby expressly denied. The Licensee acknowledges that the rights conferred on the Licensee by this Licence are personal to the Licensee and are non-exclusive to the rights of other invitees of the Licensor.

My W3C-validated table (all the rows have the same number of columns, but it does use colspan) is also rendered as a smear in the corner of the page.


Metadata and readme cleanup

Ok, good to know, thanks!
BTW, I can't approve it either because it's my own PR.


[Keyboard] Derive logical keys from physical keys for non-ASCII characters

This PR changes how logical keys are derived for macOS and Web, so that if the logical key is non-ASCII, it will be replaced by a key of the same name as the physical key. For example, when pressing KeyA on a Russian keyboard, the event's logical key will be KeyA instead of ф.
This ensures consistent behavior cross platform, that logical keys are always ASCII, and that shortcuts work correctly.
For reasoning, see this design doc.
Fixes https://github.com/flutter/flutter/issues/59065 for raw keyboard.
This PR is accompanied by https://github.com/flutter/engine/pull/31714, but does not need it to compile or run.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Closing in search for a better solution.


Crashing with Enchancement

Mod Name
Enchancement
Mod Homepage
https://modrinth.com/mod/enchancement
Minecraft version
1.20.1 (LTS)
Describe the bug
It is
Steps to reproduce

Install latest version of Sinytra Connector, Forgified Fabric API, Connector Extrass, Tooltipfix, and the latest 1.20.1 version of Enchancement
Launch the game
Crash

Logs
https://mclo.gs/ka8Ub3y
Additional context
No response

As our LTS support target, 1.20.1 now only receives critical fixes related to Connector bugs. Please upgrade to the latest version for mod compatibility support.


feat: versioning module

I would like to wait for the export method to be implemented before merging the PR.
By the way, I believe export is not a good name for such a method. The purpose is to convert the Config singleton into a serializable format. Not really to export it. I believe the method should be internal.

I agree, thinking about it now, we should have started with the config serialization. This PR should be merged after the config is done.

I've updated the PR to be compliance with the new serializer on taipy config. I have tested localy and it works for creating, loading and listing versions, for the FS Repositoy. It shouldn't be too hard to expand to other repositories, however I think it's best to first merge all the PRs and do the integration needed for a test end to end.
This PR will fail because the new serializer is not merged yet on taipy config. I wish we had the time to do this before I went on vacation, but I think the code is very simple and Giang will do a great job integrating and improving anything if necessary.
Feel free to do any adjustments to the code as you see fit.
See you guys soon. Cheers!

This PR is merged from https://github.com/Avaiga/taipy-core/pull/359 so it's pretty large.
To summarize the PR:

The "version" property is added to all entities except for Cycle, due to Cycle is attach to Scenario.
The internal _Version class represents the Version entity, with version.config is a JSON serialized string of Config singleton.
The _VersionManager is responsible for create, manage the versions.
The version_cli.py manages how the user will interact with the versioning system via console.

The current use-case:

When the user start Core service, which will block Config update, the version_cli is call (on core/_core.py).
With FS repo, the current version and the development version is save to a file named "version/version.json". This is so that the Manager can quickly access the current or dev version without the need for checking the created/modified date, or check for a flag.
Dev mode: All entities with the development version is deleted. The development version is set to the current version.
Experiment mode: A new version number will be set as the current version.
If switch from experiment mode back to development mode, the old development version will be clean, which keep the experiment version safe.


fix(colors): make whitespace chars visible

Due to the color of whitespace characters (spaces and tabs) is same as the background color (#1F1F28), they are not visible unless being highlighted at the current line even if editor.renderWhitespace is not none.

So I changed editorWhitespace.foreground to be slightly lighter than  editorIndentGuide.activeBackground #363646 but darker than
editorLineNumber.foreground #54546D.
This is the result:

I honestly believe this is intentional. Most editors don't have editor whitespace colors, and some people don't like them.

Hm, If it's intentional, that's ok then. Maybe I will create a fork for my own.
But in my opinion, if you don't like whitespaces are visible, that is editor.renderWhitespace setting for.


Slow clear_output() performance in native notebooks

Issue Type: Bug

Create a notebook and add this code to it:

from IPython.display import clear_output

for i in range(10000):
    clear_output()
    print(i)

Run the cell. You'll see flickering in the output of the cell as it seems to go from zero height (from the call to clear_output()? to 1 line in height. If you run the same code in Jupyter classic notebooks, you'll see that the height of the output cell doesn't change due to the call to clear_output().

This is tremendously impacts performance. In native notebooks (in an a run that is different from the simplified case below) it took 1800s vs. 24s in Jupyter on the same machine.
This kind of code is pretty common to show progress during a long training run.
 This is an interesting scenario where a ton of appendOutput and clearOutput requests are queued and the core always try to apply these requests on the model.
https://user-images.githubusercontent.com/876920/124516451-86b87300-dd96-11eb-9ae1-8b9d4f7fb1a4.mov
The challenge for the text model is, we don't know if there is a clear output requests coming next. I'm wondering if we can do the same optimization as Jupyter classic in the Jupyter ext side.

@rebornix this would be a problem for anyone writing to ipynb (anyone building a Python kernel, or even with .NET, julia & the like).
We could go a few ways, my initial plan was to have an API in core that allows users to write to ipynb (on top of the VS Code API). lets will discuss offline.

This is probably due to https://github.com/microsoft/vscode-jupyter/issues/5662


Problem installing node on windows for the first time using nvm windows.

If this is a question about how to use NVM4W, please use stackoverflow instead.
If this is an issue regarding antivirus, make sure you search the existing issues first.
My Environment

[ ] Windows 7 or below (not truly supported due to EOL - see wiki for details)

[ ] Windows 8

[ ] Windows 8.1

[ ] Windows 10

[ ] Windows 10 IoT Core

[ ] Windows Server 2012

[X] Windows Server 2012 R2

[ ] Windows Server 2016

[ ] My Windows installation is non-English.

I'm using NVM4W version:

[X] 1.1.7
[ ] 1.1.6
[ ] 1.1.5
[ ] 1.1.4
[ ] 1.1.3
[ ] 1.1.2
[ ] 1.1.1
[ ] Older
[ ] OTHER (Please Specify)

I have already...

[X] read the README to be aware of npm gotchas & antivirus issues.
[X] reviewed the wiki to make sure my issue hasn't already been resolved.
[X] verified I'm using an account with administrative privileges.
[X] searched the issues (open and closed) to make sure this isn't a duplicate.
[X] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

My issue is related to (check only those which apply):

[ ] settings.txt
[ ] proxy support (Have you tried version 1.1.0+?)
[ ] 32 or 64 bit support (Have you tried version 1.1.3+?)
[ ] Character escaping (Have you tried version 1.1.6+?)
[ ] A standard shell environment (terminal/powershell)
[X] A non-standard shell environment (Cmder, Hyper, Cygwin, git)

Expected Behavior
I installed nvm for windows on a citrix thin client with Windows Server 2012 R2. When I type nvm in the Git Bash terminal that I have installed, I get the nvm help menu. This should confirm that everything is working. When I now do: nvm install latest all, I get the following error message:
Could not retrieve  https://nodejs.org/dist/latest/SHASUMS256.txt.

Get https://nodejs.org/dist/latest/SHASUMS256.txt: dial tcp <IP_ADDRESS>:443: i/o timeout

I also tried nvm install latest all --insecure, but that did not help.
Actual Behavior
I would expect nvm to install the latest npm and node.
Steps to reproduce the problem:
See steps in 'Expected Behavior'.

please use nvm proxy your_proxy_and_port to setup the proxy for nvm

I don't think this is a proxy issue.
It looks like something is blocking your inbound connection at the network level. In this case, NVM4W is just the messenger. Older versions of Windows servers tend to have a pretty tightly locked down firewall, so I'd start by checking there. See if you can even access those  URL's from a browser on the server.
The other possibility is the Windows Certificate Trustchain doesn't recognize the TLS/SSL cert of the remote server. In this case, you can add --insecure to the end of the command to bypass the TLS/SSL validation step. I'm leaning more towards a firewall issue though, because TLS/SSL doesn't typically timeout when there is a problem. It normally says the remote host isn't trusted.

Thanks. I had tried the --insecure option, so I guess the firewall is a
good bet.
Op do 18 apr. 2019 01:05 schreef Corey Butler<EMAIL_ADDRESS>
I don't think this is a proxy issue.
It looks like something is blocking your inbound connection at the network
level. In this case, NVM4W is just the messenger. Older versions of Windows
servers tend to have a pretty tightly locked down firewall, so I'd start by
checking there. See if you can even access those URL's from a browser on
the server.
The other possibility is the Windows Certificate Trustchain doesn't
recognize the TLS/SSL cert of the remote server. In this case, you can add
--insecure to the end of the command to bypass the TLS/SSL validation
step. I'm leaning more towards a firewall issue though, because TLS/SSL
doesn't typically timeout when there is a problem. It normally says the
remote host isn't trusted.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coreybutler/nvm-windows/issues/436#issuecomment-484293869,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABUMADGOGIJMVCBAOK2UILPQ6ULZANCNFSM4HFFZXDQ
.

Not sure what's going on here but even when I try to install an older version of node via nvm - I continue to get this error: Could not retrieve https://nodejs.org/dist/latest/SHASUMS256.txt. shouldn't the url show the version of node i'm trying to install and not latest??
@coreybutler

@terrycoughlan The last line of the output identifies your specific problem. There is no internet connection.
By default, the latest SHASUM is retrieved when NVM4W thinks it needs to retrieve the latest version (https://github.com/coreybutler/nvm-windows/blob/53bc08dfed5a1ec99a6d6fb8f4771f9de993b3de/src/nvm.go#L866). In your case, it does not recognized v14.20.1 as a version because you have no internet connection. It cannot retrieve the list of Node versions.


Make access to FileSystem possible

It would be nice if there would be an easy way to write files to the local file system.
Even a small thing like creating a file with webshot and make it downloadable seems impossible with mupx if you're not using S3 or another extern solution or if you don't want to modifiy the mupx package.
It seems that there is even no temp dir like /temp or /tmp the application is able to write to.
I've found this solution - I think it would work but maybe there is a more recommend way to solve this.
https://github.com/arunoda/meteor-up/issues/914

Might be possible or make a good question or suggestion with the devs of the new version at kadirahq/meteor-up?


[ML][Pipelines]Parse sub graph jobs with try-catch

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

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

General Guidelines and Best Practices

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

Testing Guidelines

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

why add empty file ?


support heater

Pull request recommendations:

[x] Name your pull request your-development-type/short-description. Ex: feature/read-tiff-files
[x] Link to any relevant issue in the PR description. Ex: Resolves [gh-12], adds tiff file format support
[x] Provide context of changes.
[x] Provide relevant tests for your feature or bug fix.
[x] Provide or update documentation for any feature added by your pull request.

Thanks for contributing!
Resolves #3
Introduces Heater type with set_mode options and fills in blanks for not supported options.
I don't know how to introduce tests.

Hey @mepla could you have a look at this PR and approve it?
Your library is natively used by home assistand and it requires me everytime to manually change this.


search_suggestions_section should take content's height only

Currently search_suggestions_section occupy the entire screen even when no suggestion is showing. It should only take at most the height of the suggestion list.
I'm showing a recycle view under the FloatingSearchView when FloatingSearchView is in focus, but when it is in focus the user can not interact with the recycle view.
Pseudo layout:
<FrameLayout>
    <RecycleView />
    <FloatingSearchView />
</FrameLayout>

Sorry for late response, but that's the intended behavior. I would suggest you to clone the repo and customize.
Regards,
Ari


TFT txn naming convention using Stellar > BSC bridge

While using the Stellar > BSC bridge user get the following transaction information:

Even if there is a warning message user can get confused if he doesn’t scan the QR code to fill the transaction fields and decide to do it manually for some reason (e.g. using Desktop version of TFConnect)
Naming convention is different on TFConnect transaction form and on Bridge transaction information
“Message” field on the TFConnect transaction form corresponds to “Memo” on the Bridge transaction information and the correspondence is not obvious.
Suggestions
Bridge transaction information should follow as much as possible the naming convention on TFConnect transaction form
Also it should be clear that both address and memo are required for the transaction to succeed

[ ] Change “Destination” text to “ Destination address”
[ ] Change “Memo” text to “Message / Memo”
[ ] Reinforce warning message saying that “Message / Memo” field is mandatory
[ ] Add a “copy to clipboard” option at the end of both mandatory fields (would help to let it even clearer)

source file:
https://github.com/threefoldfoundation/tft/blob/4c9900950723d165f23268dfb421e504979b6c64/bsc/bridges/stellar/frontend/components/Deposit.tsx

@sabrinasadik
In order to have a consistent naming convention maybe it would also be a good thing to standardize on the TFConnect app side
On the TFT txn form changing the "Message" field to "Memo" or to "Message / Memo" would avoid some confusion

@ken-de-moor pls see request above and create ticket in right repo

@sabrinasadik In order to have a consistent naming convention maybe it would also be a good thing to standardize on the TFConnect app side On the TFT txn form changing the "Message" field to "Memo" or to "Message / Memo" would avoid some confusion

I agree, the term used in stellar is Memo, not message, so we will change it on the TF connect side.

https://github.com/threefoldtech/wallet-next/issues/149


[FEATURE REQUEST] Add a feature to disable being able to retract a RTV vote

Adding in a feature to disable being able to retract a vote would help to give it more variety as to what you could do with the RTV function.

What should happen when people disconnect and reconnect?

Disconnecting and reconnecting would remove your vote, so you would have to RTV again.

Doesn't this defeat the point of your suggestion?

It would somewhat, but it would mean that the person has to reconnect to be able to vote again, which would take some effort to do. My main concern is that if it would be possible to store that the person had voted already without it causing issues (e.g. a person that disconnected will have their vote count still), but if you are able to implement it without causing issues, then I would be fine with doing the other way instead.

I think the best possible way would be a combination of these two:

People who voted to rtv, then disconnect and reconnect keep their vote.
Votes from people who are currently disconnected don't count.

I still don't really see a point in banning people from retracting their vote though. 🤔
I may implement this feature at some point, but I won't prioritise it.


Update README.md to further enhance its flow and readability.

Fix #86

These are tweaks to the fabulous CloudState README.md to further enhance the flow and readability of its narrative. This PR supplants PR #87

@viktorklang - Requesting you to please add yourself as a reviewer of this PR (as I currently don't have permissions to do so), thanks.

Great stuff, thanks @akramtexas!

Great stuff, thanks @akramtexas!

Thanks a ton, @viktorklang 🆒 Working alongside wildly skilled developers is always cool, but when they also happen to be awesome individuals, it's plain awesome!

@akramtexas ^^


Give each ansible-runner task its own virtualenv

We currently share a base virtualenv among ansible-runner tasks, into
which we install ansible, datadog callbacks, ara and requirements for
all ansible-runners.  This is problematic because 1) ansible-runners
that do not run as root require sudo access to update the virtualenv
2) we can potentially end up with version conflicts among dependencies
Instead, this sets up isolated virtualenvs for each runner task, each with
their own callbacks, dependency set, etc.
Closes-issue: BonnyCI/projman#242
Signed-off-by: Adam Gandelman<EMAIL_ADDRESS>

recheck


TL change logs and 'Version changes' content

Original report on BitBucket by Taina Jääskeläinen.

Only SL change logs seem to be copied to TL change logs.

I created a new SL version of the just published vocabulary COVIDCA and made two change. Moved status to ‘Ready to be translated’. Change log was copied to ‘Version changes’ in the pop-up.
I created a new version of the FI TL as ADMIN_TL. Changed a code descriptive term. Moved status to ‘Ready to be published'. The pop-up only copied SL changes from the change log to the TL pop-up ‘Version changes’ field. In the change log, you can see 3 changes, of which one in TL.

@MajaDolinar I am not sure I got the problem. Images are missing. I tried to create new version of SL, with changing the code definition and adding new code and it also shows the later change (add code) in the TL. Can you please elaborate it little bit more?

Yes, I am also having trouble reproducing this and understanding what the issue behind this is. That is why I haven’t provided you with screenshots yet.  I will investigate further and hopefully clarify this in the coming days.

I do not understand what this issue is about. Closing the issue for now, will not be worked on.


Don't attempt to load [dir]meta files in RestoreMultipleItems

Newer versions of Corso can load metadata files inline so we don't need to augment the path set during restore. However the logic for this was never updated
Action items:

remove [dir]meta files from the path set during OneDrive restore for newer versions of Corso

I have a fix for this in https://github.com/alcionai/corso/pull/2908

unassigning as this will have a longer timeline if it does get addressed

closing as we'll still need to pull these in if we want permission inheritance to work out how we expect


mlpnet input layer has one less feature

Hi,
I recently started using AutoPytorch for discovering normal mlpnets for the regression setting. Unfortunately, if my data has 21 features as in the example, the learned network actually takes only 20 input features. I'm not sure what I am doing wrong, but if I try to use the model for further predictions as a standard pytorch model, it gives errors because of the matrix dimension mismatch. Any solutions to this?
This is my discovered network -

For featurized data there is a lot of preprocessing techniques implemented. One of them probably downsamples the data. Could you provide the full configuration?
However, it is probably a good idea to implement a method to extract the preprocessing pipeline in some form for further usage. We should do this in a future release

Thanks for the reply! Turns out it was a preprocessing technique that was downsampling the data. In my configuration, all the options were set active. Setting the preprocessing to none produced the right input size for the architecture. Is there any available documentation other than the example notebook which describes the role of all the options in the configuration?

The instances of the autonet classes have a print_help() function which gives information about some options. This will become more detailed in the future. In terms of preprocessing options we tried adapted the common terminology so that it should be possible to find documentation about it elsewhere (although it would be nice to have some small guide here).

Thanks! I'll look forward to more documentation then, but it's good enough to be used for my purposes right now. Closing the issue!

@GautamSridhar
Hi,
When I use the AutoPytorch for searching network, the data was downsampled. However, I used "include_components" to set the preprocessing none, like this "include_components={'preprocessing': ["NoPreprocessor"]}". But it was not work. So how did you set the preprocessing to none produced the right input size for the architecture?


Show legality of decks in previous PD seasons

Depends on #458

We did this. Woo!

Not quite yet.

I want to include the legalities column on the main page.
I have some ideas as to how to do it neatly, and throw together a prototype 24-48 hours from now

Is it useful enough to take up precious screen width on the main page? A deck is either legal for the thing I care about or not. It's legality in various other scenarios doesn't interest me.
As long as our focus remains PD legal decks there's only really two modes "legal now" and "legal in the past" and I'd be happiest with that being a toggle rather than a column. We can mark currenlty-illegal decks with the same no entry sign we use on cards in the margin on lists?

I did this.


Fix switched local and remote addresses in servlet requests (0.20)

See #2952.

[info] decode should
[error]   x be consistent with utf8Decode (540 ms)
[error]    Falsified after 67 passed tests.
[error]    > ARG_0: "\u0008"
[error]    > ARG_0_ORIGINAL: "魂Ⲷਬ纅ᗀ赕Ὴᔱ鑰跎㎯쾼ዦ㷭"
[error]    > ARG_1: 1
[error]    > ARG_1_ORIGINAL: 1
[error]    > '' != ' (DecodeSpec.scala:24)

Not my fault. :smile:

This should be uncontroversial and we've got enough other things to keep our reviewers busy.  Merging this with 1 so we can release.


Circular Dependency in ChatBubbleAvatar

While getting familiar with the library codebase I realized a message regarding a Circular Dependency when running npm run dist:
Circular dependency: src/index.ts -> src/ChatBubble/index.ts -> src/ChatBubble/ChatBubble.tsx -> src/ChatBubble/ChatBubbleAvatar.tsx -> src/index.ts

This is caused by the ChatBubbleAvatar importing Avatar from src/index.ts instead of src/Avatar.
The current import looks like this:
import { Avatar, AvatarProps } from '..'

When they should be:
import Avatar, { AvatarProps } from '../Avatar'

I am making a PR to fix it right now.

Thanks for finding this, approved the PR!


HMR ( Hot module replacement/ Hot Reload ) Not Working Correctly

To reproduce:

Add an input, and bind it to a counter model.
Add a button to increment the counter, and increment it to something greater than 1.
Add a new variable in any component.
Save the file.
The state is not preserved. The counter goes back to 1.

I don't think it's supposed to keep the internal state of the component beeing reloaded. Can you make this use case work properly with vue-loader ? ( .vue single component files)
IMHO, I think state is preserved for others components in the page, but actually the component beeing reloaded can't keep it's state.

It works if you look at the vue-cli docs. I have tested it myself and it does seem to work.
This would be the ideal situation, as the way it works now is not true Hot Module Replacement.

Ok, so i'll try to implement this :)

Cheers, friend!

I confirm it works with official webpack template when modifying the template only. If script part of the component is modified, it's internal state is lost.

I just tested with both JavaScript and TypeScript using this template, and it works exactly like with official webpack template and single file components .vue files.
When changes are made in the template or the style of the component, it keeps the internal state of the component. But when changes are made in the script code, it's lost.
Vue doesn't provide support to keep internal state of component when modifying the script part from this component.

Thank you for your investigation into the issue.
I was indeed incorrect about it saving state when editing the javascript.
However, when working with this template, I am not able to maintain the state when editing the html only.
Here is a GIF demonstrating the issue (also notice that the compilation takes quite a while for some reason.)

Can you post your whole project in a github repository ? I'll give a look.
Hot Module Reload can be broken because of some babel configuration.

Hi @Toilal
I have just cloned your repo vue-webpack-ts-upgrade in this example, but the current repo does the same thing. I have not changed any settings.

vue-webpack-ts-upgrade ?

Sorry, I have confused another project for yours. I will test out HMR for your project. Apologies if this is the case!!

No problem :) I really think it's working here, but please give it a try !
It has also clarified what's the expected behavior for everybody : It should keep state if changing CSS or HTML, but state is lost when component code changes. Anyway, it should never reload the whole browser, and state from other components in the page should be kept intact.

@Toilal
Thank you for your help. I have confirmed it is working, although when I edit in Sublime Text, it does not seem to swap the changes out. I think it has to do with some temporary file it creates on save...
It's a shame though that we can't get it to work like in the following image with React:


Chocolatey installation fails due to thumbprint mismatch

Description
The chocolatey installation script (at https://chocolatey.org/install.ps1) was recently signed with a new certificate (appears to have occurred on 5/17/2024). Therefore, the thumbprint check now fails when attempting to install chocolatey.
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[X] Windows Server 2019
[X] Windows Server 2022

Image version and build link
Issue is not in any released image as it is a build issue
Is it regression?
No
Expected behavior
Chocolatey installs successfully when building the Windows images
Actual behavior
Chocolatey fails to install when building the Windows images with the following message: Signature thumbprint do not match expected.
Repro steps
Attempting to build any of the Windows images will error out when attempting to install chocolatey.

Hey @jhennessey!
Thank you for your effort. Very good example of the first issue for our repo. Should wait for test run before merging PR.

fixed now, please sync your main!


Error when trying to use an Interval for LocalTime

Hello,
Nice library!
I am getting this error when trying to use an Interval for LocalTime:
-- [E008] Not Found Error: ...
23 |  given Domain[LocalTime] = Domain.makeLocalTime(temporal.ChronoUnit.MINUTES)
   |                            ^^^^^^^^^^^^^^^^^^^^
   |value makeLocalTime is not a member of object com.github.gchudnov.mtg.Domain
1 error found
Compilation failed

Maybe it is because I can't download 1.0.1?
Downloading 4 dependencies
[error] .\project.scala:1:16
[error] Error downloading com.github.gchudnov:mtg_3:1.0.1
[error]   not found: ...
[error]   not found: https://repo1.maven.org/maven2/com/github/gchudnov/mtg_3/1.0.1/mtg_3-1.0.1.pom
[error]   not found: ...
[error]   No fallback URL found
[error] //> using dep "com.github.gchudnov::mtg:1.0.1"
[error]

Hi @andrelfpinto
sorry for the breaking the build.
to be honest thought that no-one is using it :)
please wait a bit, I'll finish updating the library today the library and release it.

the build was fixed,

but there still might be problems with method accessibility.
There are still some pending tasks:

to recover examples
update documentation
release the library

can take some more time, but should be done till the end of the week.

@andrelfpinto
The example for LocalTime was added.
Still before publishing the library want to add more examples to see if the public API has any issues.

@andrelfpinto
the library was published https://github.com/gchudnov/mindthegap/releases/tag/v2.0.0
if something is not working or there are any questions, please let me know

I'm closing then, if you have any problems with 2.0 please let me know


JSON tags don't work with Home Assistant

Not specifically a problem with this task, but having a period or colon in the JSON tag causes problems with how these data are unpacked by Home Assistant. The obvious solution is to remove the colons and translate the periods to underscores. I'm pretty sure that a minus character will not fly and it will be nice to preserve the separator. As a reminder, the payload looks like
{"ups.timer.shutdown:": "-60", "driver.version.internal:": "0.47", "t": "1717469101", "driver.version.data:": "CyberPower HID 0.6", "battery.runtime:": "475", "input.voltage:": "120.0", "output.voltage:": "120.0", "ups.timer.start:": "0", "driver.version:": "2.8.0", "battery.voltage:": "14.2", "device.model:": "EC850LCD", "ups.test.result:": "No test initiated", "battery.charge:": "100"}

Fixed by tweaking tags to work with Home Assistant. #3c79cff


recreate the map w/ the new data

Shapefile is now outdated

Done. @gjahansen2 please review the map

Why is Petenwell missing? are there 2148 lakes in the map?

I will look into it

fixed. Can you verify @gjahansen2 ?

looks good to me!


Apparent temperature move to Toggle + translations

Simplified the apparent temperature option, moved it to Toggle and added some initial translations.

Thanks, but the buttons are fine as they are, fitting in well with the rest of the pop-ups. Translations should not be edited manually https://github.com/matt8707/ha-fusion/tree/main/scripts/translations


Typescript definitions is not working as it should

This Issue is about (✏ delete as applicable)

An error in the TypeScript defs

Phaser version(s): 2.8.1 and below
What should happen: Type definitions should just work OOTB without the need of third party libraries (typings)
What happens instead: You need to install typings and another dependency

From reading the below, its best practice to include a "types:" section in the package.json to make it work out of the box:
https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html
Quote:
Including declarations in your npm package
If your package has a main .js file, you will need to indicate the main declaration file in your package.json file as well. Set the types property to point to your bundled declaration file. For example:
{
    "name": "awesome",
    "author": "Vandelay Industries",
    "version": "1.0.0",
    "main": "./lib/main.js",
    "types": "./lib/main.d.ts"
}

@dolanmiu have you noticed any difference since v2.8.2?

I will try 2.8.2 tonight, still using 2.8.1


Incorrect usage of emit() or incorrect comment

This block of code does not behave as the comment describes
https://github.com/canonical/opensearch-operator/blob/b0208dc2bcfce66234125f1ab39aed73c070bbae/lib/charms/opensearch/v0/opensearch_base_charm.py#L471-L476
relation-joined will only be emitted on that unit (where https://github.com/canonical/opensearch-operator/blob/b0208dc2bcfce66234125f1ab39aed73c070bbae/lib/charms/opensearch/v0/opensearch_base_charm.py#L466 evaluates to True)
And that unit will immediately return
https://github.com/canonical/opensearch-operator/blob/b0208dc2bcfce66234125f1ab39aed73c070bbae/lib/charms/opensearch/v0/opensearch_base_charm.py#L294-L297
More info:

Note that the emission of custom events is handled immediately. In other words, custom events are not queued, but rather nested. For example:
1. Main hook handler (emits custom_event_1)
2.   Custom event 1 handler (emits custom_event_2)
3.     Custom event 2 handler
4.   Resume custom event 1 handler
5. Resume main hook handler

https://ops.readthedocs.io/en/latest/#ops.BoundEvent.emit

This looks like a bug indeed. Thanks!
In this case, I don't think we unfortunately can get away from emitting the event, in case the current unit is the elected leader unit - which will then not receive a peer-rel-changed event if the data is stored in its unit data bag..
I believe the fix is a mix of both approaches, something along the lines of:
if self.unit.is_leader():
    self.on[PeerRelationName].relation_joined.emit(
        self.model.get_relation(PeerRelationName)
    )
self.peers_data.put(
    Scope.UNIT, "last_updated_at", datetime.now().timestamp()
)

This has been fixed with #346: it generates a peer event when renovating the certificates, hence waking up all its peers.
@Mehdi-Bendriss now, the config-changed is the entry point to update the new IP in our config, we are not changing the unicasts_hosts.txt of each peer to the new IP thou.

Actually, a correction: config-changed does update the unitcast_hosts file:
config-changed > calls self.tls.request_new_unit_certificates() > triggers peer relation change with self.charm.peers_data.delete(Scope.UNIT, "tls_configured") > peer-relation-changed  > calls self._add_cm_addresses_to_conf()


Ease adding latest version of a dependency with given stability level

Allow using a _last_${min_stability_level}+ value for a version property.
Example:
version.kotlin=_last_eap+
version.androidx.lifecycle=_last_beta+
version.androidx.appcompat=_last_stable

(_last_stable doesn't need explicit + because it's the top stability level)
These would used in the artifacts resolution directly, and written straightaway in the versions.properties (instead of having to wait for the refreshVersions task to run).

No longer relevant since we auto add the latest most stable version candidate.


Fix address field in TaskVine manager config.

Previously it looks like this field was ignored, because it was always (in the absence of a project name) overriddel by
parsl.addresses.get_any_address. This PR makes that override only happen when the address field is set to None.
Previously this field had a class level default value of socket.gethostname() That value was never used, because of the above override, and resulted in the hostname of the documentation build host being listed as the default value for this parameter, incorrectly.
This PR removes that default value, allows the address field to be None, and makes the address field be None by default.
Cross-reference this Debian documentation reproducibility issue where the presence of socket.gethostname() breaks binary reproducibility: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1063542
Testing:
Prior to this PR, setting address='<IP_ADDRESS>' does not break anything, even though that is not an IP address of my test environment. After this PR, setting that IP address causes taskvine to hang as I would expect.
Type of change

Bug fix
Update to human readable text: Documentation/error messages/comments
Code maintenance/cleanup

I tried this with different hostnames, the reproducibility bug reported in the above debian issue, and I get what I think is a binary-identical build: the only file that differs is this pickle file, which i think is fine?
$ diff _build/ ~/tmp/b3 -r
Binary files _build/doctrees/environment.pickle and /home/benc/tmp/b3/doctrees/environment.pickle differ

@tphung3 or maybe @colinthomas-z80 can you review this as far as changing taskvine functionality goes?

I believe the manager_config address only serves to override the address given to the factory for the workers to reach out to. So if an erroneous address is set it would only misguide the workers deployed by the factory.

@colinthomas-z80 is that a positive or negative review of this PR?

It is a positive change to error on a bad address and I do not see any issues with functionality.


update refresh-token-grant errors

Two different conditions shouldnt throw the same error

Summary
The main problem is various errors were identical in the Refresh Token Grant file. While working on an express/mongodb adapter I kept erroring out at the client check. Well the client check error is the same error that exist in like 5 other places. Just changing the verbiage alone would help in diagnosing the problem.
Linked issue(s)

Involved parts of the project
[https://github.com/node-oauth/node-oauth2-server/blob/master/lib/grant-types/refresh-token-grant-type.js](Refresh Grant Type)
Added tests?
si
OAuth2 standard
Its just the error verbiage the logic remains untouched.
Reproduction
No new features just changing some text.

@HappyZombies sorry I misread it. It's only the error message that changed while I thought it's the error class that changed. From that point it should be a good improvement. The package.json and package-lock.json should be restored to the previous values and the yarn.lock should be removed then it's good to go imo.

Done!

@chrisfranko dope thanks, but can we remove the sinon update and remove the lock file changes. Then we will approve and merge

Yea man I thought I did that?

https://github.com/node-oauth/node-oauth2-server/pull/140/commits/55e236cb3a3875e730c8bd2d6baf57ffe3f0e6a6
But honestly im not entirely sure how to remove the sinon update because those all came from master and I didnt manually do them. However! It might just be easier for me to push a new pr to the dev branch instead.

https://github.com/node-oauth/node-oauth2-server/pull/143


Video support not working - pi 4 & 3

hello, i trying to do an small app that play videos on raspberry pi, but the game can´t load or work with videos , giving me these errors :

same error, webm or ogv
if requested, i can upload the concept test project, it´s small ( just 55 MB)
all the project are doing with VisualScripting
of course, this error occurs only using the export template binary

Hey, sorry for the late reply! Yeah, it won't work.
The Theora and WebM modules are disabled because I couldn't compile for the Pi with those modules enabled.

For others who run into this limitation: You can use the godot videoplayer on the raspberry pi through FRT.
You'll just have to export the project on a different machine using an official build of Godot


Text Entry Box does not respond to certain keyboard presses like Backspace

Hi,
This bug does not always happen when adding the steps, but once it happens, you need to restart vscode so that keyboard buttons like backspace, delete, home, end work again

Hi there, thank you for letting us know! Can you share what OS / version you are using? Thank you!

I was able to confirm that this is a bug in VS Code, not CodeTour. If you could file an issue on their repo, that would be hugely appreciated. In the meantime, I’m going to close this issue as being external. Thanks!


HDXDSYS-950 Use global HPC HNO dataset(s)

I used a wildcard since we may load multiple global annual datasets
There's a bit of a refactoring in this PR for the output stuff

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

40 of 40 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 93.333%

Files with Coverage Reduction
New Missed Lines
%

src/hapi/pipelines/utilities/logging_helpers.py
1
93.75%

Totals

Change from base Build<PHONE_NUMBER>5:
0.02%

Covered Lines:
1414

Relevant Lines:
1515

💛  - Coveralls


[BUG] Sheet theme not applying

I know I just reported a bug and closed it as solved, but I noticed a new one a couple of moments after the new update (UI v. 3.5.12 // UI v. 1.50)
When selecting a theme for the character sheet, it doesn't seem to apply and defaults to the light theme

I can't reproduce this.

Are you using the starfinder module?
Are you sure a theme is applied at all, or is it just the stock foundry look? (Check the titlebar)

I'm using the Pathfinder module/system

I'm almost sure it's applying the theme (or supposedly to), I've tried changing it to other combinations, but now that you mentioned related to the title bar, it seems somewhat related to the stock foundry look. Not sure what is going on

Yeah so what I'm seeing is that no theme is getting applied for that sheet, which is typically because it's been excluded, or because the theme system is crashing. (Check for red console logs).
I've put out an update, see if that fixes it for you.

I am getting the same issue with the update. Here's the console error that is showing up for me:

It seems to be working for me now, thanks :)
I'll leave the topic open for now to make sure it's also solved for @Ethmaker and others

Thank you!

After updating modules again and reloading the world, the issue seems fixed and themes are properly working again.

Here is the error; I have a similar problem - all updated system and Dorako mods.

I see the issue, should be fixed for next release.


Struck in Kernel

MY system is stuck in kernel im running latest retrorangepi. it works when connected to usb but not when getting 5v from bench psu.

which board? Try a more reliable 5V PSU, usually 2A minimum is required

Orange Pi Zero Plus 2 H3, and it works them i'm just connected to an USB adaptor But it does not work when i connect it to the 5v pins. The Device comes to life, it tries to do the u boot stuff but gets stuck with UBS Bus

I cannot reproduce this issue. My ZeroPlus2 H3 works fine from pins 1 and 2 (5V and GND).

This is the result i get when i connect to pin 1 and 2. waiting does no good...

This behavior is similar to booting with analog TV settings, HDMI  source is active and cable is present
Otherwise, still not reproducible


[10.1.x] ISPN-11840 HotRodMergeTest updated to use numOwners=2

https://issues.redhat.com/browse/ISPN-11840

Integrated into 10.1.x, thanks @ryanemerson !


refresh error with routes

when reloaded on "places/place" the 404 page appears

fixed the routes but non-existing place urls still get stuck at the loading component and other invalid routes must redirect to error page too

nope cant figure this out error page is stuck at vercels default and the other thing still remains


Email unsubscribe path isn't working

My weekly emails have an unsubscribe URL that follows this pattern:
https://www.freecodecamp.org/ue/[unsubscribeId]

This path no longer works.
We will need to fix this before my Thursday email blast.

This is a stability issue. All endpoints are working.

However, I recommend waiting on a confirmation before sending out the blast

The unsubscribe path works, but the resubscribe doesn't -- at least it didn't for me. All I wanted to do was change the email address where I receive, but when I try to resubscribe following the link shown in the screenshot above, I get this:


[StateAccumulator] Remove idempotency from epoch accumulation

Description
The same issue arising in https://github.com/MystenLabs/sui/pull/10160 could be affected by idempotency of epoch accumulation, i.e. if checkpoint builder proposes a bad checkpoint and it happens to be the last of the epoch (since we then immediately accumulate the epoch in order to generate the root state hash), then we would persist a bad checkpoint accumulator and a bad epoch accumulator. Without this fix, the bad checkpoint accumulator would be overwritten in checkpoint executor (using a certified checkpoint), but the bad epoch  accumulator would not, resulting in inconsistent state.
Test Plan
Existing tests

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

I don't think I understand - wouldn't the previous fix prevent a bad checkpoint accumulator from being persisted?

Also, I'm a little hesitant to put more of these fixes in unless there are active test failures, since we should be able to fix the root cause (checkpoint builder bug) this week.

We no longer need this as we fixed the root cause in checkpoint builder.


feat: Securitygroup Drift

Fixes
Description

Security group drift will check drift between AWSNodeTemplate.Spec.SecurityGroupSelector and machines with resolved values returned from EC2

How was this change tested?

Unite Tests
Integration Tests
Manually Tested

Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Related to #1738

Great work on responding to feedback and getting this in!


Allow to not use buffs/pots in city.

Is your feature request related to a problem? Please describe.
It's bad when you die, respawn in city and pot everything before you click in npc Healer.
Describe the solution you'd like
Put a flag to user that represents 2 things (1: Use in all Maps, 2: Don't use in city)

I'm doing it :D

this would require adding a extra client address(for the map) , and a list of cities , plus an extra json file for custom cities names

@noxnoire
I was thinking... this can be a Profile configuration too. What you think?
When you're on a map, you can click in 4R and tells to "Not use autobuff/autopot" in this map. It's a  more customized approach.
This information will be saved in profile, like everything else.

looks good , but still gonna need an base address to actually read the map, im gonna try to make it work on seal , since i alrd have the address for map there

Yeah, we can add a additional field for map address.
I guess that we can't get map address based on hpCurrent address 😢

that only happens to hp/maxhp and sp/maxsp based on current hp and map y based on current x so i guess gonna need to be added

Can you open a PR for this @noxnoire ?

im working on this , still nothing yet

Ok.
I assigned the pr to you.
Any doubts and updates let us know. Maybe we can help :D

left what i had to the moment over at on PR #119  , still a draft for now


[SPARK-35813][SQL][DOCS] Add new adaptive config into sql-performance-tuning docs

What changes were proposed in this pull request?

Add new configs in sql-performance-tuning docs.

spark.sql.adaptive.coalescePartitions.parallelismFirst
spark.sql.adaptive.coalescePartitions.minPartitionSize
spark.sql.adaptive.autoBroadcastJoinThreshold
spark.sql.adaptive.maxShuffledHashJoinLocalMapThreshold

Why are the changes needed?

Help user to find them.
Does this PR introduce any user-facing change?

yes, docs changed.
How was this patch tested?

Test build #139969 has started for PR 32960 at commit accc4f2.

Test build #139969 has finished for PR 32960 at commit accc4f2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139969/

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44496/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44496/

cc @cloud-fan @yaooqinn @HyukjinKwon

can we also update the Coalescing Post Shuffle Partitions section to replace the min partition num config with the new min partition size?

Test build #140875 has started for PR 32960 at commit 080429f.

Test build #140875 has finished for PR 32960 at commit 080429f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140875/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45386/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45386/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/45386/

thanks, merging to master/3.2!

missed one thing: AQE is disabled by default.
Can we correct this in the doc as well?

thank you @cloud-fan , created https://github.com/apache/spark/pull/33295


Add CTAD for __without

I get a build failure due to missing CTAD on __without.

/ok to test

thanks!


Search Box Missing

I keep my browser at 1280px wide (half of a 1440p monitor). If I turn off the add-on, the search box is visible at this size, but it appears to be off-screen when using the add-on. If I make the window somewhat bigger, the search box appears. Same behavior at all feed width sizes.
This seems like a bug?

@eborenst thanks for catching this. I removed the buggy media query, not sure why it was there originally, but this version should work: https://github.com/thomaswang/minimal-twitter/releases/tag/2.40.
Currently pending review on Chrome/Firefox.

I'm actually still having this issue on version 2.5/June 27.

@eborenst The behavior that is currently that is intended, the search box disappears when the tweet button shrinks to a circle. At those window sizes, you can go to the "explore tab" to search. I'll look into a better solution in a future release 😀

Ok, got it. I failed to notice that there's a search box on the Explore tab, but that works. Thanks!


psn-client-secret

HEllo
How i can get psn-client-secret please?

Hi, sorry this project is not actively maintained any longer. Please check closed issues and the DOCS.md file for more information.


Vs code git bash terminal always forgets it's history after re-open

VSCode Version: latest 64bit
OS Version: win 10

Steps to Reproduce:

Use git bash as terminal
Open git bash and type ls
open vs code, type some other command  (in the integrated terminal)
close vs code and open again
do arrow/cursor up. (in the integrated terminal)

The command shown is ls (history from git bash instead of git bash from vs code session).
I Expect either:

History to be remembered globally
Or history of the commands to be remembered per project/path.

All the history is managed by the shell, in this case git bash. If you launch a single shell and exit it, the history will likely be retained. However it can be less reliable when there are multiple shells of the same type, but this happens regardless of the terminal as far as I'm aware.

How come it is only retained when I open git bash directly and is never retainer in vs code?
I'll do some more testing but s far I have never seen vs code => git bash retain anything. although opened manually it always seems to retain history

@joelharkes it's possible there's an issue on our side but it seems unlikely as we're just launching the git bash executable and that's accepting input and telling us the output. Since the git bash executable is the thing handling the input, it should be persisting the history somehow.

Seems to depend on how you close the bash:
https://stackoverflow.com/questions/10488498/bash-history-does-not-update-in-git-for-windows-git-bash
It also seems my projects keep running when i close vs-code (and i started the project running yarn start)
it seems vs-code does not close integrated terminals properly?

A solution would be to either properly close git-bash on vs code close.
or enable the use of executing commands (through settings) when integrated shell closes.
the command that would be needed to execute is: history -a


Remove gulp and browserify from the test setup

Continued from https://github.com/marcules/karma-remap-istanbul/issues/31
This was the only way I could find that would let later versions of remap-istanbul work with this project. It has the nice side effect of reducing a lot of the test complexity by just using webpack for everything and removing gulp, as well as no more temporary test files.
Let me know if this works for you 😃

👍 Awesome thanks a lot for this PR


Invariant Violation error - Git Gateway

I am hosting jekyll site on netlify forked from github(gh-pages).
I am trying to add netlify cms feature.
My admin/index.html
<!doctype html>
<html>
<head>
  <meta charset="utf-8" />
  <meta name="viewport" content="width=device-width, initial-scale=1.0" />
  <title>Content Manager</title>
</head>
<body>
  <!-- Include the script that builds the page and powers Netlify CMS -->
  <script<EMAIL_ADDRESS></body>
</html>

admin/config.yml
backend:
  name: git-gateway
  branch: gh-pages # Branch to update (optional; defaults to master)
media_folder: "images/uploads" # Media files will be stored in the repo under images/uploads
site_url: https://optimistic-poitras-148d9a.netlify.com
collections:
  - name: "post" # Used in routes, e.g. /admin/collections/blog
    label: "Post" # Used in the UI
    folder: "_posts" # The path to the folder where the documents are stored
    editor:
      preview: false
    create: true # Allow users to create new documents in this collection
    slug: "{{year}}-{{month}}-{{day}}-{{slug}}" # Filename template i.e. YYYY-MM-DD-title.md
    fields: # The fields for each document, usually in front matter
      - {label: "Layout", name: "layout", widget: "hidden", default: "post"}
      - {label: "Title", name: "title", widget: "string"}
      - {label: "Author", name: "author", widget: "string"}
      - {label: "Publish Date", name: "date", widget: "datetime"}
      - {label: "Categories", name: "categories", widget: "select", multiple: true, options: ["kerala", "india", "world", "business", "movie", "tech", "job", "sports", "health", "auto", "she", "travel", "homestyle", "index"]}
      - {label: "Audio File", name: "speech", widget: "file", required: false}
      - {label: "Tags", name: "tags", widget: "string", required: false}
      - {label: "Featured Image", name: "image", widget: "image", required: false}
      - {label: "Youtube Video Id", name: "youtubeId", widget: "string", required: false}
      - {label: "Published", name: "published", widget: "boolean", default: true}
      - {label: "Body", name: "body", widget: "markdown"}

When going to  mysite/admin, I get a login screen like below,

On sign in with my netlify account, I get
Error
There's been an error - please report it.

Details
Invariant Violation: Minified React error #31; visit https://reactjs.org/docs/error-decoder.html?invariant=31&args[]=t%3A%20invalid_grant%3A%20No%20user%20found%20with%20this%20email&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings.

Note: I signed in to netlify.com using github.

I need to add me as a user to site in settings or identity settings.

We need to handle these errors much better:

Git Gateway errors should not get tangled up in React errors
Warn when Git Gateway backend is used but identity widget is not on the page

Sent with GitHawk

Is this still an outstanding issue?
I have this issue when accessing /admin locally.
(it works fine when hosted on netlify)

@MildTomato can you share a repo reproducing the problem? Can you post some console logs? The original issue doesn't provide much information

Closing as stale, if someone is experiencing it please re-open a new issue with a reproduction.


Only have a single DefinitionId for each type

Update ApiView tool to have a single DefinitionId for each type.

Ping. I'd like to get the ApiView challenges sorted :).


[FEEDBACK]: Records broken link

Type
Site Bug Report
Feedback
With reference to "Overview" section on this page: https://developer.tbd.website/api/web5-js/dwn/record/

There is a broken link on the word Records, it goes to: https://developer.tbd.website/api/web5-js/dwn/record/records, which is Page Not Found.
The correct link should be: https://developer.tbd.website/api/web5-js/dwn/records
Screenshots
No response
developer.tbd.website URL
https://developer.tbd.website/api/web5-js/dwn/record/
Browser
Chrome
Relevant log output
No response

I would like to pick this up since I am working on a similar issue #825.

@aniketh-varma thank you for noticing this, its been assigned to you. 🥳


DiscreteMovementCommand Attack command breaks all blocks

It would be very convenient if there were some block type that the attack action in the Discrete Movement set was unable to break, for the purposes of setting up permanent maze boundaries and the like. Or perhaps there is a way to configure this, that I was unable to find?
In particular, I was somewhat surprised to find I could break bedrock.
I was able to work around it by using a line of sight check, but it just seems like maybe there should be a cleaner way?

Maybe you can try something like this:
<DiscreteMovementCommands> <ModifierList type="deny-list"> <command>attack</command> </ModifierList> </DiscreteMovementCommands> <ContinuousMovementCommands> <ModifierList type="allow-list"> <command>attack</command> </ModifierList> </ContinuousMovementCommands>
That enables all but the attack command for the discrete command handler and disables all but the attack command for the continuous command handler.
To start attacking, send "attack 1"; to stop, send "attack 0" - it acts like pressing and releasing the attack key, and all the Minecraft attack logic will come into play - so you can hurt pigs, and you won't be able to remove bedrock blocks.

Somewhat surprisingly, that seems to have broken turning. My agents now appear like they start to turn, but they're only ever facing South.
If I keep the deny-list in place for discrete movements, but remove the ContinuousMovementCommands section entirely, this stops happening, and my agents begin to actually turn again.
The commands I'm issuing are "turn 1" and "turn -1".


Notation for arguments

Node declaration can be parametrized by passed argument list:
In -> Dropout(probability = 0.5) -> Out

or shortened D(p=.5).
Parens are probably at the wrong place. What about about these alternatives?

Lispy (weird):

In -> (Dropout :probability 0.5) -> (D :p .5) -> Out

Unixy (familiar):

In -> (Dropout -probability 0.5) -> (D -p 0.5) -> Out

JSONy (meh):

In -> (Dropout probability:0.5) -> (D p:0.5) -> Out

Important is keeping the identifier outside the parens:
In -> d1:(D -p 0.5) -> d2:(D -p 0.5) -> Out

Pros/Cons:

JSON format is nice, but can be confused with identifiers. Argument names can be shortened, identifiers cannot.
Unix style is so far the best. There is also possibility for explicit shortening: --probability vs -p.
Argument list à la LISP is confusing for me. There was idea about argument passing like this:

first:(D :p .5) -> second:(D :p first:p)

Which would look like this:

And it is somehow compelling, but still weird.

In new version of standalone parser I went with Unix style.


Upgrade packages for M1 support

Upgrade packages using the poetry update command. After this upgrade poetry install seems to fetch a working version of torch on M1.

Great news @zabil thanks for checking this


Add VyOS as router platform

Proposed Functionality
Add support to use VyOS as a router platform.
Use Case
When using a router running VyOS.
External Dependencies
A Napalm Library already exists for it.

In 1.4.0 platforms will not longer be a hard coded list. Users will be able to create new platforms and use community NAPALM driver with them (as long as the Python package is installed).


Allow "datetime" data type

Is your feature request related to a problem? Please describe.
The "datetime" data type was added to the flogo core project: https://github.com/project-flogo/core/pull/186
Flogo Web should support it.
Describe the solution you'd like

[x] Importing/exporting apps that reference the datetime type should be supported
[x] Contributions that reference the datetime type should be supported
[x] User should be able to edit the datetime type as a string without any issues (mapper, debugger panel, etc.)
[x] Install datetime functions by default: https://github.com/project-flogo/contrib/tree/master/function/datetime

fixed in PR #1308


Add new methods in different header directories

This is for first-time-contributors :
Add functions which belong to a particular header file in its corresponding .md file inside the header folder.
For example
std::sort belongs to algorithm header file,  so make a new file sort.md inside the algorithm directory while maintaining the CONTIBUTING guidelines and formats.
See todo.txt file in different header directory to see which of the methods have not been implemented yet.

are you saying you want the sort in sort.md moved to algorithm.md?

Is this issue solved? If not, I could try to do it.

@dantedallag this issue always remains open to help contributors .

Can I contribute to this issue?

Can I contribute to this issue?

Yes you can, just see the unimplemented functions in the todo.txt files inside different directories

Hi, I am new to open source in general, and I would like to contribute as a beginner.
I have a few doubts:

How can I differentiate which functions are unimplemented in the README.md file?
I also want to make sure if the process of contributing that I understand is right or not: At the time of writing this, the pop_back() function is not yet implemented yet. So is this what I do - I write a sample code that implements this function (by following the coding conventions of course), and this code will reside in snippet/vector. And I also make a .md file in vector directory (outside snippet) which explains the code. Is there anything that I am missing? If yes, please tell me so that I begin.

Hi, just an update:

I made pop_back.cpp in snippet/vector. I also wrote the corresponding .md file in vector directory, in which I pasted the link to run the code on rextester. Is this enough? Is submitting a PR the job left to do or anything missing?

@saisriram2710 you are right

Can i contribute to this issue ?

Hi, I want to contribute to this issue. Where do i start?

can i contribute to this issue??

Hello! I would like to contribute to this issue. I have reviewed the CONTRIBUTING.md, looked at the Projects, and referenced the Kanban for each directory type. I would like to work on "pop_back" under Vector. If successful, this will be my very first Open Source contribution :)
Thank you @Bhupesh-V for making this opportunity available. So far, the instructions have been very complete and helpful.

Hello! I would like to contribute to this issue. I have reviewed the CONTRIBUTING.md, looked at the Projects, and referenced the Kanban for each directory type. I would like to work on "pop_back" under Vector. If successful, this will be my very first Open Source contribution :)
Thank you @Bhupesh-V for making this opportunity available. So far, the instructions have been very complete and helpful.

@marcinky looks like pop_back is already implemented and only crend is available to contribute.
Don't worry just head over to specific directories like list where pop_back has not been implemented yet (see :heavy_check_mark: and :x:  marks)
Go ahead and implement it and welcome to opensource :hugs:

Ok will do @Bhupesh-V  - thank you! I'll take another look.

Should I create a new issue before starting to work on code or should I just go ahead and open a pull request?

Hi Bhupesh Varshney,
I would like to contribute to this issue.
I've locally created .md file with an example code for the new 'contains' in the unordered_map for C++20.
I've followed your coding guideline and also tested my code. I can create a pull request for review.
Could you please allow me write access to create a PR? Thank you for your help in advance.

Hi Bhupesh Varshney,
I just made my first PR for unordered_map 's at function . Please check it out . :)
https://github.com/Bhupesh-V/30-seconds-of-cpp/pull/579

Hi Bhupesh Varshney,
I would like to contribute to this issue.
I've locally created .md file with an example code for the new 'contains' in the unordered_map for C++20.
I've followed your coding guideline and also tested my code. I can create a pull request for review.
Could you please allow me write access to create a PR? Thank you for your help in advance.

Hi @athulsasi you don't need any write access to the repo.
Everything is listed in CONTRIBUTING guidelines on how & what to contribute.

Hello @Bhupesh-V. I am completely new to open source and would like to contribute to this project, specifically the crend function in the set directory. I really appreciate the opportunity provided for newcomers like me.

@Bhupesh-V I have created a PR please review and suggest changes.
#649

@Bhupesh-V I would like to work on this one .

Hi @Bhupesh-V  I would like to give it a try! :)

Hi @Bhupesh-V Can I contribute to this issue a function to unordered_map?

@Bhupesh-V,Hey! I would like to contribute to this, please assign this to me.


Unit test for pretrained checkpoints

Run pytest -q ROOT_DIR/tests/unittests/test_pretrained_checkpoints.py to verify the correctness of loading pretrained checkpoints.

Shall we close https://github.com/amazon-science/earth-forecasting-transformer/pull/14?

Shall we close #14?

Done


tickets/DM-33000

Switch to GH workflow from Travis.

Add GH workflow.
Remove .travis.yml file.
Replace Travis references with GH ones.
Replace master branch reference with main

Yep. Nothing to see here.


Sample Startup Portfolio Website

_ No description provided. _

@itssiddhantjain make star the repp first 🌟

@thecyberbuzz I have starred this repo and also merged. kindly check it and merge under hacktoberfest.


Fixing issue with AlertRuleGroupExport data model

Fixing issue with AlertRuleGroupExport data model
For more context: https://github.com/grafana/grafana/issues/94936

The same needs to be done for the AlertRuleExport.For field as well.

The same needs to be done for the AlertRuleExport.For field as well.

Updated.  Thanks for calling that out.

Any thoughts on getting merged this in? objections or otherwise?

/deploy-to-hg


Question

Hi,
Your plugin is just perfect ! Exactly what i search. (This similar plugin give the same functionality https://github.com/somewebmedia/hc-sticky with more features.)
In your doc, you don't write what is the compatibility with browser (IE version, mobile device etc ...?)
It will be very interesting if your add callback and few "command" like this :
stop 	: Stops the sticky while preserving its current position.
off 	:Turnes off the sticky completely.
on 	:Turns the sticky back on after above two commands.
reinit 	:Recalculates sticky size. Useful after altering DOM elements inside sticky.
destroy 	:Completely destroys sticky and reverts element to original state.
Thanks !

@usb248 thank you for feedback and proposals

In your doc, you don't write what is the compatibility with browser (IE version, mobile device etc ...?)

My missing, updated this info in README.md

It will be very interesting if your add callback and few "command" like this :
stop : Stops the sticky while preserving its current position.
off :Turnes off the sticky completely.
on :Turns the sticky back on after above two commands.
reinit :Recalculates sticky size. Useful after altering DOM elements inside sticky.
destroy :Completely destroys sticky and reverts element to original state.

Good point, will think about additional functionality for next release, added ticket for myself.
Btw if you want to speed up process and contribute some of this features, you are welcome.

Yes thanks, if i have time. Why not.
On tablet/mobile the scroll event is not fired continuously like it is on desktop. It is fired only when the scroll actually ends.
The resulting behaviour is that the element does not reposition immediately, but just at the end of the scroll.
Have you detect this little bug ?

@usb248 yeah, saw this, sometimes u even need to do additional scroll to make fixes element stay in right position.
Didn't think about solution, rarely sidebar stays sidebar on mobile. Need to investigate it, thanks.

Yes rarely on mobile device but often on tablet.

@usb248 added bug ticket


Change default matching strategy to RougeChunkMatch

https://github.com/relari-ai/continuous-eval/blob/63cf7d8176f09117b2c678949beee272e8a36179/continuous_eval/metrics/retrieval_precision_recall_f1.py#L8C62-L8C77

Also in the Ranked Retrieval metrics


(BKR-767) Only use Solaris major version for pkg

Puppet only publishes packages under the major version of a Solaris
release. Currently, if Beaker is passed a Solaris host with a minor
version (eg. solaris 11.2), it fails to find and install package.
This commit truncates the Solaris version to its minor component
in the solaris_puppet_agent_dev_package_info and
solaris_install_local_package  methods.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2379/
Test PASSed.

👍


Dark theme instability

I have just followed through instructions

[X ] The issue is present in the latest release.
[ X] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
While in Dark mode, some components update (routine state change) Cards switch back to light and then to black after a while (non-deterministic, sometimes it's 1 second, sometimes it's when I change the state again or click on an empty space on the page).
https://user-images.githubusercontent.com/32850517/103481128-b6b29980-4da6-11eb-98cc-e5cd0bdeb08c.mp4
Before and after clicking on update. This style sheet "flickers".

Codebase

Your Environment 🌎
  System:
    OS: macOS 11.1
  Binaries:
    Node: 14.15.3 - /usr/local/bin/node
    Yarn: Not Found
    npm: 6.14.9 - /usr/local/bin/npm
  Browsers:
    Chrome: 87.0.4280.88
    Edge: Not Found
    Firefox: Not Found
    Safari: 14.0.2
  npmPackages:
    @material-ui/core: ^4.11.0 => 4.11.2
    @material-ui/icons: ^4.9.1 => 4.11.2
    @material-ui/lab: ^4.0.0-alpha.55 => 4.0.0-alpha.57
    @material-ui/pickers: ^3.2.10 => 3.2.10
    @material-ui/styles:  4.11.2
    @material-ui/system:  4.11.2
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react: ^17.0.0 => 17.0.0
    react: ^16.14.0 => 16.14.0
    react-dom:  16.14.0
    typescript:  3.5.3

Please provide a minimal reproduction test case with the latest version (v5). This would help a lot 👷 .
A live example would be perfect. This codesandbox.io template may be a good starting point. Thank you!

Please provide a minimal reproduction test case with the latest version (v5). This would help a lot 👷 .
A live example would be perfect. This codesandbox.io template may be a good starting point. Thank you!

Note: This is just a quick check, please follow @oliviertassinari recommendations.
This behavior reminds me of something that happened to me a while ago, perhaps is related to what @eps1lon once help me out with, replace your media query line with this one:
 const prefersLightMode = useMediaQuery('(prefers-color-scheme: light)', {noSsr: true});

By the way, if you are using MUI v5, change the prop type to mode in your palette object.

Note: This is just a quick check, please follow @oliviertassinari recommendations.
This behavior reminds me of something that happened to me a while ago, perhaps is related to what @eps1lon once help me out with, replace your media query line with this one:
 const prefersLightMode = useMediaQuery('(prefers-color-scheme: light)', {noSsr: true});

By the way, if you are using MUI v5, change the prop type to mode in your palette object.

@luminaxster Regarding your very point, we track this problem in #15588.

@luminaxster Regarding your very point, we track this problem in #15588.


xCluster: Fix cluster creation on Server 2012

Pull Request (PR) description
xCluster\Set-TargetResource builds a hashtable for splatting against New-Cluster. It now checks if Force is an accepted parameter on that command before adding it to the hashtable.
I added a stub of the Server 2012 version of New-Cluster named New-Cluster_NoForce which does not have the Force switch defined.
There is one applicable test, "Should call New-Cluster cmdlet". I've copied this into another block, "Should call New-Cluster cmdlet without -Force if it is not supported". This block swaps the New-Cluster stub for the New-Cluster_NoForce stub by using Set-Alias.
The test confirms that, when the Server 2012 version of New-Cluster is in place the -Force option will not be passed along.
I've tested:

Removing my change results in the new test failing.
Adding my change results in all tests passing.
And that the change works for me on a Server 2012 VM.

It seems that the stub file had incorrect line endings which changed when I edited it. But you can view in VS Code etc that it's only one stub function added.
This Pull Request (PR) fixes the following issues

Fixed #188

Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: codykonior sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #189 into dev will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff         @@
##            dev   #189   +/-   ##
===================================
  Coverage   100%   100%
===================================
  Files         8      8
  Lines       486    487    +1
===================================
+ Hits        486    487    +1

I've generated the stub for Server 2012 and then modified each test file to run twice - once against the 2012 stub and once against 2016. Getting the quorum tests to pass on 2012 required some additional fixes to the xClusterQuorum resource to use 2012 compatible aliases.
In the tests there's a new global:moduleVersion. This is because having all of the tests inside the InModuleScope block hides that variable and it is not possible to access it in any other way. That variable is needed within the tests to:

Label the Describe blocks with the version number so that the Pester output is clear.
Switch between different parameter filters within some of the Quorum tests. These filters operate against the parameter name and don't support the 2012 compatible aliases.
Skip one version of the Quorum test that uses a Cloud (not applicable to 2012).

Style changes submitted.


Unable to add account in Windows 11

Hi, I recently switched to Windows 11, whenever I try to add an account the spinning loading icon appears, then after a minute it stops but never prompts me to enter anything. Is there a solution for this?

Please, contact me using email.
Right-click on Calendar Flyout icon in notification area > Settings > About > Email support.

What were you testing with your calendars? Your description means nothing without any details about what you did ;)

Hi, I was just trying to import my Google calendar to my MS account (which worked fine).
And yes, I did see the window where I select either the MS account I'm logged into, or login to a new account. Tried both ways (but using the same MS account). The process went fine, my credentials were approved, the window closed -- and nothing. The loader icon spinned for a sec -- and no account was added.
And no, I'm not using a VPN.

I have the exact same problem. First login worked fine. Then I logged out and tried to login again, but when I press next in the popup window it disappears and nothing happens. Reset and reinstall of the app does not help.

Hi, I was just trying to import my Google calendar to my MS account (which worked fine).
And yes, I did see the window where I select either the MS account I'm logged into, or login to a new account. Tried both ways (but using the same MS account). The process went fine, my credentials were approved, the window closed -- and nothing. The loader icon spinned for a sec -- and no account was added.
And no, I'm not using a VPN.

Did you follow this article?
https://support.microsoft.com/en-us/office/see-your-google-calendar-in-outlook-c1dab514-0ad4-4811-824a-7d02c5e77126#PickTab=New_Outlook

Yes, I actually have multiple calendars linked. I added some in between the login attempts.

Yes, I actually have multiple calendars linked. I added some in between the login attempts.

Please, contact me using email.
Right-click on Calendar Flyout icon in notification area > Settings > About > Email support.


[python] Python 3.12 support

For issue #1849 -- which has a long history.
See also #2999 and [sc-53002].
This needs https://github.com/single-cell-data/SOMA/pull/222

From the build log:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/__w/TileDB-SOMA/TileDB-SOMA/venv-soma/lib/python3.10/site-packages/tiledbsoma/__init__.py", line 103, in <module>
    from ._collection import Collection
  File "/__w/TileDB-SOMA/TileDB-SOMA/venv-soma/lib/python3.10/site-packages/tiledbsoma/_collection.py", line 3[7](https://github.com/single-cell-data/TileDB-SOMA/actions/runs/10895777089/job/30234556256#step:18:8), in <module>
    from ._dataframe import DataFrame
  File "/__w/TileDB-SOMA/TileDB-SOMA/venv-soma/lib/python3.10/site-packages/tiledbsoma/_dataframe.py", line 17, in <module>
    from tiledbsoma import _new_shape_feature_flag_enabled
ImportError: cannot import name '_new_shape_feature_flag_enabled' from partially initialized module 'tiledbsoma' (most likely due to a circular import) (/__w/TileDB-SOMA/TileDB-SOMA/venv-soma/lib/python3.[10](https://github.com/single-cell-data/TileDB-SOMA/actions/runs/10895777089/job/30234556256#step:18:11)/site-packages/tiledbsoma/__init__.py)

This is because at the point this line is executed, the symbol _new_shape_feature_flag_enabled is not yet available. The line
https://github.com/single-cell-data/TileDB-SOMA/blob/a96c72d0854e535a79063562391498a77dbcead4/apis/python/src/tiledbsoma/__init__.py#L138
needs to be moved above all the imports. (Alternately, it might make sense to create a tiledbsoma._flags module so that there’s no circular dependency between submodules and the tiledbsoma/__init__.py file.

needs to be moved above all the imports
@thetorpedodog thank you!!! I got some 'new style lint' here -- I don't know why, I didn't change any linter versions -- and was forced to move imports :(

There's something weird to debug here -- I'll see if I can make everything in the CI toolchain happy simultaneously -- thank you!!!

@thetorpedodog

#3004
#3005

And re

There's something weird to debug here -- I'll see if I can make everything in the CI toolchain happy simultaneously -- thank you!!!

I have traced that to here:
https://github.com/single-cell-data/TileDB-SOMA/pull/3004/files#r1763424424

Hi guys, thanks for your amazing package.
I understand this is in flux as we write - as a power user I would kindly like to ask if there's anything I can do to help push this through - I really need it and workarounds are a decent pain.
(As I understand, now there is a buggy typechecker stalling CI stalling 3.12 adoption, which itself was released 12 months ago. If inaccurate feel free to correct)

@iosonofabio thank you! Your assessment is correct -- current status is here: https://github.com/single-cell-data/TileDB-SOMA/issues/1849#issuecomment-2358420564 and @ryan-williams will be moving this forward

I can't add you as a reviewer since you opened this PR, @johnkerl, but I would like your review on the latest changes that I've pushed here 🙏
Depending on typeguard@HEAD may trip up SOMA developers who have existing typeguard 4.2.1 or 4.3.0 installations, but I'm not sure what the other option is. As soon as https://github.com/agronholm/typeguard/pull/490 makes it into a release, we can move to that. Could also try to only depend on typeguard@HEAD in Python 3.12, but I thought that was more complex than it was worth.

Depending on typeguard@HEAD may trip up SOMA developers who have existing typeguard 4.2.1 or 4.3.0 installations, but I'm not sure what the other option is

Agreed. And it only affects folks on 3.12, so any dev who really wants to avoid this can use 3.11.


Virtualbox doesn't work

After some attempts I still am unable to use Virtualbox.

https://github.com/mocaccinoOS/desktop/commit/11fa6db4f5c8b1d74caad83d210182cb96c81737


Fix indentation of Webhook NetworkPolicy matchLabels

Pull Request Motivation

Configuring webhook podLabels using webhook.podLabels causes helm to fail trying to render the chart because indentation is not configured correctly in the networkpolicy-webhook.yaml file.
Kind

/kind bug
Release Note

Fix indentation of Webhook NetworkPolicy matchLabels in helm chart.

/approve
/lgtm


Muting Notifications & Notification Sound

As any user
I want to have a sound when I receive a notification
so that I know when I received a new notification

Assumptions & Scope
What are the assumptions for this story?
What is IN scope?
Sounds for receiving new notification
Ability to mute and just have number accumulate with no sound
What is NOT in scope?

Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

check in with users at a future demo


Access semantic scene mesh directly

Motivation and Context
We have implemented the ability to directly access mesh data as discussed in issue #1081.
This required adding methods to the ResourceManager class and the Simulator class. We added python bindings
to the methods in the Simulator class.
How Has This Been Tested
We added two new tests to tests/test_simulator.py, test_mesh_point_extraction and
test_mesh_point_extraction_when_not_instance. The first one uses a datafile not
present in the repository and is marked as skipped. It tests that the methods run and
return vectors with the correct length.
The other test check that the methods throw an exception when the meshes are not
of the GenericInstanceMeshData type.
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Sorry for loosing track of this for a long time. I'm updating it now to the current main.

You are going to need pull / rebase from master. Assigning @jturner65 to review since we now support more than just semantic  meshes, but semantic textures / vertexes etc.

I rebased on main before making the requested changes.
Unfortunately I see some changes in the submodules in my PR branch. Any ideas on what I can do about this?

When you rebased you committed your local changes to the submodules, which reverted them to an older state.  Always discard submodule changes when you rebase unless you explicitly are changing the submodules :).

@rantahar Hello, I also encounter the problem of accessing mesh directly. After cloning your repo https://github.com/AaltoRSE/habitat-sim, unfortunately, I cannot use APIs like sim.get_instance_mesh_keys() shown in test_simulator.py. I wonder how the pr is going(it is not in the master branch yet) and also how I can use your code to access mesh. I will really appreciate it if you can provide some help!


What are the functions to read the incoming data from a keyboard

I tried using the read function
But this happened
from hid import * vid = My VID pid = My PID try:     h = Device(vid, pid)     product_type=str(h.product)     is_usb=str(h.manufacturer)     if (product_type == "USB Keykoard" and is_usb == "USB"):         print("Found Your Keyboard...") except HIDException:     print("Connect your device") h.read(size = 64)
It throws me this error
hid.HIDException: Access is denied.
Please help me

A normal hid keyboard will get a keyboard class driver loaded by your operating system that takes full control. If you want to read the keyboard input reports in userspace you need to prevent the keyboard driver from loading on the vid/pid of the keyboard in question.

How would i do that

tell me

This is not your personal support forum, your question does not directly relate to the library of subject here, and nobody owes you answers - especially if you are being impatient/rude.


[Issue 7787][pulsar-client-cpp] Throw std::exception types

Change throws of const char* to use std::exception types instead

Fixes #7787
Motivation
Throw std::exception instead of const char*
Modifications
Should be self explanatory.
Verifying this change
This change should be covered by existing tests, such as BatchMessageTest.  Not sure about the Auth test which catches ... however.
Does this pull request potentially affect one of the following parts:
Yes

Operations which formerly threw const char* now throw a type derived from std::exception

Documentation
The methods that can throw should be documented as such.  This was not implemented.

Seems some tests are failing, but I didn't touch the Java code (CPP/Py tests are OK).

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


Thread: make Thread.name return the current thread name

This makes the name property use the CoreFoundation SPI to query the
thread name to avoid having to have this interface be used directly in
the tests.

@swift-ci please test

@swift-ci please test and merge

ugh, need to make this a SPI - can't mix get and didSet

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test Linux platform

@swift-ci please test

@swift-ci please test


FINAL SPRINT : ANNOUCEMENT DATE IS CONSTANT

admin side: date posted

user side : announcement tab -> date does is not the same with the one presented above

Fixed #299
Commit ID: 0b1887c7c824df82913dd5f2d13c410c47887797


RPC fails with large contract class size

Issue
My compiled_contract_class.json generated with snfoundry fails when i try to declare it.
The size of file is 28 megabytes and when snfoundry tries to send it to juno it errors following msg:
command: declare
error: Unknown RPC error: data did not match any variant of untagged enum JsonRpcResponse: data did not match any variant of untagged enum JsonRpcResponse

But i expect to see:
command: declare
error: Contract class size is too large

The log-level trace is not yet available as mentioned here https://github.com/NethermindEth/juno/issues/1767
I cannot provide much more info then this (no important logs in debug log-level) but i am happy to help with reproduction of this issue.
There is no such issue when i send 10 megabytes contract class to juno it behaves as expected.
Do u know how to solve this issue? And why it happens?
Versions
snfoundry v0.20.1
scarb nightly-2024-03-16
nethermindeth/juno:v0.11.4

I’d like to resolve this.

Can I work on this, please?

Can I jump on this task?


Removes the numpy_usm_shared module from numba_dpex.

The PR removes the numpy_usm_shared module from numba_dpex. All related unit test cases are also removed.

accidentally closed.


The basic implementation of tokens on utxo

This basic token system allows to create, send and view a list of tokens.
Execute the extrinsic tokensCreate for creating new tokens. It allows making a new transaction with an output that includes a new tokens amount:
utxo -> tokensCreate

- pubkey of the creator
- An input for taking a creation fee.
- tokenName
- tokenTicker or short name
- supply amount

For sending tokens it's able to use spend just check TokenId in TxOutputHeaders.
For request a list of tokens:
curl http://localhost:9933 -H "Content-Type:application/json;charset=utf-8" -d   '{
    "jsonrpc":"2.0",
    "id":1,
    "method":"tokens_list",
    "params": []
}'

Returns the array with TokenIds and their names:
[
    {1, "My test token"},
    {2, "Global universe super token"},
]

Closed due to unverified commit


Run husky install as prepare script

Describe the bug
Yarn tries to run husky install as part of postinstall script when installing this package as a dependency.
To Reproduce
yarn add vue-use-popperjs

Expected behavior
The package being successfully installed.
Screenshots
$ yarn add vue-use-popperjs
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
[1/2] ⠄ vue-demi
error /app/node_modules/vue-use-popperjs: Command failed.
Exit code: 127
Command: husky install
Arguments:
Directory: /app/node_modules/vue-use-popperjs
Output:
/bin/sh: husky: not found

ERROR: 127

Desktop:

yarn v1.22.17

Additional context
https://docs.npmjs.com/cli/v9/using-npm/scripts#npm-install
postinstall is executed as a part of install by npm and yarn. Husky is a development dependency of this package which might not be true for project where this package is used.
Now Husky project recommends to use prepare script to install hooks. See explanation at https://blog.typicode.com/husky-git-hooks-autoinstall/

I tried prepare as the way to setup Husky (cloned this repo, adjusted package.json and pushed dist directory to be able to use it as a dependency) but it looks like it breaks yarn install in our project when it's executed on clean project checkout (no node_modules/) with empty yarn cache.
Some people suggest it's due parallel fetching + git repo as a package source + "prepare" script combination, see https://github.com/yarnpkg/yarn/issues/6312#issuecomment-422806004
If I run yarn install  --network-concurrency 1 then it succeed. So seems like "concurrency issue" is valid explanation.
Probably prepare script will work if the package installed from the npm registry.