Chatbot mode register_search_phrase has to be a full sentence?

Hi, @richardpaulhudson
I have tested this tool with code
import holmes_extractor as holmes
holmes_manager = holmes.Manager(model='en_core_web_lg')
holmes_manager.register_search_phrase('be all ear')
holmes_manager.start_chatbot_mode_console()

I am trying to do "phrase" search, and found out that, once the sentence contains a "be" verb, this will lead to detection of "be all ear".
I suppose this search phrase should be a full sentence, a phrase like stuff is not supported when you saying Ungrammatical expressions.  (Cause I found it a little bit misleading of search phrase and search "phrase")

Hi @zhoudoufu, search phrases do not need to be a full sentence but should be grammatically correct. For some reason, spaCy does not parse 'ear' as a noun in the phrase 'be all ear' so the search phrase does not work. However, the correct idiom is actually 'be all ears', which works as expected.


Update reference-aadsts-error-codes.md

Added missing AADSTS 9002332 error code.
Can be verified through: https://login.microsoftonline.com/error?code=9002332

@ThomVanL : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@ThomVanL  Thanks for the updated.  #sign-off


fix(tag): closable prop disables hover

@chasestarr @nadiia I've re-opened a new PR; instead of a feature I've just opted for a patch - what do you think?


Modify User attributes #39

This uses the previous PRs:

where client information was obtained from the database and displayed
the information was made into a proper table format

and adds the following functionality

The user / System admin can now modify any of the fields and they will be reflected in the table
The changes will also be reflected in the database

This will close issue #39

Codecov Report

Merging #75 into master will decrease coverage by 3.85%.
The diff coverage is 38.88%.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   94.30%   90.45%   -3.86%
==========================================
  Files           6        6
  Lines         246      262      +16
==========================================
+ Hits          232      237       +5
- Misses         14       25      +11

Impacted Files
Coverage Δ

server/user_account/urls.py
100.00% <ø> (ø)

server/user_account/views.py
75.51% <35.29%> (-8.64%)
:arrow_down:

server/user_account/serializers.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 68217ca...4ba7ba6. Read the comment docs.

I will fix lint and add tests tomorrow, but for now should be good to look at for feedback @RobertBeaudenon ,
lmk especially in regards to the form.

Thanks for the good work, we can finally close this PR. I'm merging to master.


Explore "Continue On" support in core

The GitHub Repositories extension exposes a 'Continue On' entrypoint via its package.json schema. This allows extensions to register ways to continue working with a virtual workspace, such as workspaces provided by the GitHub Repositories extension, in a different environment. For example:
"contributes": {
	"virtualWorkspaces": {
		"menus": {
			"continueOn": [
				{
					"command": "remoteHub.clone",
					"group": "0_local@1",
					"when": "!isWeb && remoteHub:hasVirtualFolders"
				},
				{
					"command": "remoteHub.openInCodespaces",
					"when": "remoteHub:virtualFolderProviders =~ /\\bgithub\\b/",
					"group": "1_codespaces@1"
				}
			]
		}
	},

This issue tracks exploring of moving the continueOn menu contribution point into core.

For an initial prototype we will focus only on transferring uncommitted changes, since this yields the highest value for users. The work for this looks like:

Define data format for passing file contents and changes from extensions to core

Initially we will only deal with text editors but eventually we would need to support custom editors as well
Untitled files and hot exit files should also be transferred
Must be able to reconcile file paths e.g. if the source file contents are coming from a different filesystem provider than the destination where they are being applied

Define a workspace identifier which can be used to lookup and retrieve an edit session

UUID -> json blob
Remote git origin

Implement mechanism for extension to provide edit session context to core

Investigate whether the SCM API already exposes working changes

Implement minimal storage service--will meet with Daniel re: extending the settings sync service for our needs here
Implement mechanism for storing and fetching edit session context in core

For a bare bones prototype this could be an explicit gesture like a command to trigger a POST, and another command to trigger a GET

Implement mechanism for applying edit session context in core (we do not want to have to involve extensions in this if possible, since that happens too late in the lifecycle)

Beyond the proof of concept here is a nonexhaustive list of things we will want to tackle:

Passing the edit session ID as a query parameter
Handling the edit session ID in the workbench (desktop) or embedder (web)
Making the UI readonly while an edit session is being applied
Surfacing the new "continue edit session" and "list edit sessions" functionality in the UI
Adopting the contribution point across extensions
Dealing with workspaces being in an inconsistent state (e.g. being several refs behind)

Current status: have a proof of concept in a dev branch and the required changes are deployed to settings sync server. Will work on getting this into main behind an experimental flag in June.

Current status:

You can store and resume edit sessions across VS Code clients (e.g. from vscode.dev to desktop) or devices (e.g. from your laptop to your desktop) by setting workbench.experimental.editSessions.enabled and then using the Store Current Edit Session and Apply Latest Edit Session commands.
You will be prompted to sign in using either GitHub or Microsoft authentication when you first store or resume an edit session on a new device. Make sure to use the same credentials across your devices. You can always sign out later from the accounts menu.
Work to automatically store and resume edit sessions when switching from one device to another is in progress. You can already try out a desktop -> web flow by opening a virtual Azure Repos or GitHub repository on desktop, then using the Remote Repositories > Continue Working On > Open on the Web command to open the same repo and your edits in vscode.dev. You will require the latest pre-release of Remote Repositories, latest VS Code Insiders, and workbench.experimental.editSessions.enabled to try this flow out. GIF:

https://user-images.githubusercontent.com/30305945/175428345-3a5690fd-8c2e-49fe-8484-a9bb137c117a.mp4

@fisher-m that's currently not possible. Please file a feature request if you would like to see this supported, I'd also like to learn more about the use case for this scenario. Thank you!

We have an experimental implementation of Continue On in core as of 1.72: https://code.visualstudio.com/updates/v1_71#_bring-your-changes-with-you-when-moving-across-development-environments. Closing this issue in favor of https://github.com/microsoft/vscode/issues/160047.


Stream subprocess stdout and stderr in heron-executor

The logs for subprocesses launched by the heron-executor were
not actually streamed to the heron-executor log file. Using
communicate to read the output of a subprocess will wait until the
process is closed before actually reading the logs. In the non-blocking
case the stdout and stderr were sent to a pipe but were not actually
read from which results in a deadlock if the subprocess logs enough
information into the stderr buffer
(python docs on subprocess and
pipes).

Do not use stdout=PIPE or stderr=PIPE with this function as that can
deadlock based on the child process output volume.

Heron uses glog to log messages to a file however zookeeper defaults to
using stderr with no easy way to tell it to use glog instead. As such
whenever zookeeper logged a message it would get written to the
subprocess stderr buffer and fill it up ultimately causing a deadlock in
the zookeeper thread in the tmaster trying to write to stderr. This then
manifests itself by the zookeeper client not being able to heartbeat and
having its session expire without being able to callback and restart the
process.
With subprocess.communicate the data is buffered in
memory until end-of-line is reached
and the subprocess is terminated so it is not suitable for long running processes.
One design consideration here for simplicity is the redirection of the
subprocess stderr to stdout. This is to make reading the data easier.
The downside here is that it is not possible to tell if a log message
came from stdout or stderr but I think that information will be clear
with the content of the message. Also some dependencies like zookeeper
log all of their messages on stderr anyway regardless of the message
type.
Fixes https://github.com/twitter/heron/issues/1653

Nice work.
(actually I am happy to see at the first time that not properly piping I/O from subprocess could lead to such bad error...)

Thanks for reviewing. Yeah this was an interesting one to track down, learned a lot about python and its interaction with subprocesses and also about stderr vs stdout buffering.

@ajorgensen last comment: why this dependency?

@objmagic ah sorry I just copied another pex test block and didnt look too closed. I'll remove that dependency if its not actually used.

please fix CI.

@ajorgensen - @srkukarni also encountered this when he was working with C++ api for Heron.

@objmagic I ran the tests locally and they all pass, the failures looked to be caused by
Java HotSpot(TM) 64-Bit Server VM warning: INFO: os::commit_memory(0x00000000eab00000, 40894464, 0) failed; error='Cannot allocate memory' (errno=12)
#
# There is insufficient memory for the Java Runtime Environment to continue.
# Native memory allocation (mmap) failed to map 40894464 bytes for committing reserved memory.
# An error report file with more information is saved as:

Looks like the ci machines may need some more memory or is there some other recourse?

@ajorgensen
I just read this PR again but I failed to understand why memory usage issue could happen. However, I observed that integration tests were running fine for first 3~4 tests, but started having problem afterwards.
Let's try this:

sleep 5 seconds after an integration test finishes
run ps and dump its output on terminal. Just want to make sure processes spawned by test have exited.
sleep another 5 seconds
run next integration test

What do you think?

@objmagic yeah I wonder if these changes impacted the interaction in the integration tests. I'll take a look and see if something jumps out. All of the other prs look like they have passing tests so I think its somewhat safe to say this change introduced some difference in the way the integration tests work.

@ajorgensen Agree. I am quite lost here since I did not expect such non-invasive change could blow up memory... We have to fix this issue to get this important PR merged.

@objmagic I see whats happening, the thread that is streaming the logs is not properly getting closed which is causing the process remain running which is causing the memory pressure. I'll add some better thread management to make sure the threads are getting closed when the process is.

@objmagic it looks like the processes are no longer being properly cleaned up during the integration tests which leaves them around ultimately consuming all the memory available. I'll look into why this is happening.

Latest commit should fix the issue, going to wait for it to go through ci to confirm but I was able to see the issue and the resolution locally as well, it just turned out my machine has more memory than the ci machines did :)

@ajorgensen you are right. nice observation.

This reminds me that I may need to use thread as well in result.py to prevent deadlock.

CI is good now. Feel free to merge @ajorgensen

CI is good now. Merged. Thanks! @ajorgensen


Python 3.4 (Keras): It seems the kernel died unexpectedly. Use 'Restart kernel' to continue using this console

Description
Spyder IPython kernel display "It seems the kernel died unexpectedly. Use 'Restart kernel' to continue using this console."
class VGGNet16:
def __init__(self, weights_path=None):
    self.model = Sequential()

    self.model.add(ZeroPadding2D((1,1),input_shape=(3,224,224)))
    self.model.add(Convolution2D(64, 3, 3, activation='relu'))
    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(64, 3, 3, activation='relu'))
    self.model.add(MaxPooling2D((2,2), strides=(2,2)))

    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(128, 3, 3, activation='relu'))
    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(128, 3, 3, activation='relu'))
    self.model.add(MaxPooling2D((2,2), strides=(2,2)))

    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(256, 3, 3, activation='relu'))
    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(256, 3, 3, activation='relu'))
    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(256, 3, 3, activation='relu'))
    self.model.add(MaxPooling2D((2,2), strides=(2,2)))

    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(512, 3, 3, activation='relu'))
    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(512, 3, 3, activation='relu'))
    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(512, 3, 3, activation='relu'))
    self.model.add(MaxPooling2D((2,2), strides=(2,2)))

    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(512, 3, 3, activation='relu'))
    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(512, 3, 3, activation='relu'))
    self.model.add(ZeroPadding2D((1,1)))
    self.model.add(Convolution2D(512, 3, 3, activation='relu'))
    self.model.add(MaxPooling2D((2,2), strides=(2,2)))

    self.model.add(Flatten())
    self.model.add(Dense(4096, activation='relu'))
    self.model.add(Dropout(0.5))
    self.model.add(Dense(4096, activation='relu'))
    self.model.add(Dropout(0.5))
    self.model.add(Dense(1000, activation='softmax'))

    if weights_path:
        self.model.load_weights(weights_path)

    sgd = SGD(lr=0.1, decay=1e-6, momentum=0.9, nesterov=True)
    self.model.compile(optimizer=sgd, loss='categorical_crossentropy')

    self.get_layer_output = K.function([self.model.layers[0].input, K.learning_phase()],
                              [self.model.layers[34].output])

def Test(self, X_Tr):
    # output in test mode = 0
    out = self.get_layer_output([X_Tr, 0])[0]
    return out;

What steps will reproduce the problem?

Initialize a object from the above class
Run Test(..) method for few times (feed forward 1 image at a time), X_tr = [1, 3, 224, 224]
spyder IPython console will display "It seems the kernel died unexpectedly. Use 'Restart kernel' to continue using this console."
I also tried Ipython qtconsole. Same problem exists.

What is the expected output? What do you see instead?
Expected output is the 4096 feature from the VGG net. Instead IPython console crashed after running the Test method few times.
Please provide any additional information below
PC Spec:
Processor: Intel 5820k
RAM: 16GB
HardDisk: 1TB (50GB for page file)
GPUs: Titan X (Pascal), GTX 970
Versions and main components

Spyder Version: 2.3.9
Python Version: 3.4.5
Operating system: Windows 10 x64

Dependencies
IPython: 4.2.0
Spyder Optional Dependencies:
IPython >=1.0      :  4.2.0 (OK)
jedi >=0.8.1;<0.9.0:  0.9.0 (NOK)
matplotlib >=1.0   :  1.5.3 (OK)
pandas >=0.13.1    :  0.18.1 (OK)
pep8 >=0.6         :  1.7.0 (OK)
pyflakes >=0.6.0   :  1.2.3 (OK)
pygments >=1.6     :  2.1.3 (OK)
pylint >=0.25      :  None (NOK)
qtconsole >=4.0    :  4.2.1 (OK)
rope >=0.9.2       :  0.9.4-1 (OK)
sphinx >=0.6.6     :  1.4.1 (OK)
sympy >=0.7.3      :  1.0 (OK)
zmq >=2.1.11       :  15.4.0 (OK)
Please help me out since I tried many solutions that exist in various forums but nothing worked.
*conda install pyzmq=14.7.0 did not work too
Thank you.
Related Issue: #3114

I figure out where the issue was by myself. Im posting the info for others.
It was not about Spyder or IPython console. The error was about the BLAS implementation openBLAS built which was not compatible with the other dependent libs such as NumPY.
I ran the script with python in the prompt: and I observed
** On entry to SGEMM  parameter number  3 had an illegal value
which is infact an issue with the openblas lib that I have linked.

Hello nadith, I came across the same problem working on 64bit windows10 system, but my keras have tensorflow backend. Do you have any idea about it? Thank you so much.

Thanks Nadith.
I was facing a similar issue where Keras would cause the kernel to die when loading a model.
I am sure there must a better way but I was able to clear this issue by refactoring my code and making sure Keras was loaded before numpy.


Update account_id.rs

Fixes #1105

@frol Please review my PR and suggests if there are some changes to be made.

@Asymtode712 Remove the whole account_id.rs file and instead of it, re-export AccountId and AccountIdRef from near-account-id in types/mod.rs. Obviously, you need to at least make the CI passing all the checks.


5 row exp: long press on 3 show but can't insert "-"

The sub-menu will popup, but the small-orange frame won't surround the hyphen symbol (or sharp, or three again whether it has been set to show).
All the ciphers row has this defect. On other rows it works.
(edit) If 5-row experimental keyboard shows it's upper 6th row, then sub-menu of ciphers row suddenly work, but long-press symbols of the added row (Esc to Del) are the non-working ones.
(edit) And if I show back the hint row, even the Esc-to-Del can produce working sub-menus.
So in 5-row configuration and without the hint-row on top, sub-menus of cipher row are not working.
Seen on Fairphone 2, Android 5.1.
HK 1.38.2 from F-Droid.

The last uploaded version on f-droid : v1.39.3 - Added on 2016-12-15 ; fix the issue.
It must be thanks to this brilliantly unanswered bug report, which nonetheless mustn't have been ignored.
Thanks for this great app, and that good fix, agreeable to use.


YJS messages overwhelming gossip & networking

YJS awareness is very chatty, sending a minimum of 12 messages per keystroke.

https://github.com/dxos/dxos/pull/4655


Focus

Pull Request Test Coverage Report for Build 366685016

3 of 48 (6.25%)  changed or added relevant lines in 11 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.2%) to 20.995%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/Counters/Counters.tsx
0
1
0.0%

src/components/charts/PercentageBar/PercentageBar.tsx
0
1
0.0%

src/components/ui-kit/Button/Button.tsx
2
3
66.67%

src/templates/ClauseMainPage/ClauseMainPage.tsx
0
1
0.0%

src/templates/FaqPage/FaqPage.tsx
0
1
0.0%

src/templates/FaqPage/FaqPageUploadFieldsNParameterTree.tsx
0
1
0.0%

src/templates/IndexPage/IndexPageFilter.tsx
0
1
0.0%

src/components/ui-kit/Tooltip/Tooltip.tsx
0
6
0.0%

src/components/charts/Treemap/Treemap.tsx
0
13
0.0%

src/templates/ClauseMainPage/ClauseMainPageFocus.tsx
0
19
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/components/charts/Treemap/Treemap.tsx
1
0%

src/components/ui-kit/Button/Button.tsx
1
62.07%

src/templates/IndexPage/IndexPageFilter.tsx
2
0%

Totals

Change from base Build 355372790:
-0.2%

Covered Lines:
359

Relevant Lines:
1793

💛  - Coveralls


s3: allow use of instance profile

Allows us to specify that we want TFE to use the instance profile to obtain credentials for S3:
https://www.terraform.io/docs/enterprise/install/automating-the-installer.html#aws_instance_profile
The module outputs the instance role, so we can attach additional policies granting S3 access:
https://github.com/hashicorp/terraform-aws-terraform-enterprise/blob/master/outputs.tf#L9-L11
For testing I've been using this the external-services module but it's also looking like I'll need to fork/revise a bit:
https://github.com/hashicorp/terraform-aws-terraform-enterprise/tree/master/modules/external-services

This will be submitted upstream as well, btw

Upstreamed in https://github.com/hashicorp/terraform-aws-terraform-enterprise/pull/65


Requesting optimizations in future versions.Reduce the border size of the collapsible panel to make the display more visually appealing.

Hello, I would like to inquire if there will be any optimizations made to the border in the upcoming versions. Currently, the border width is quite wide and does not look as aesthetically pleasing

Additionally, I have also noticed a blank space at the top when using version v1.7.3.
Your plugin has played a significant role in my work, Thank you very much for your contributions!

Hi you can always adjust the styles yourself to make them look how you'd like. If I make it thinner it becomes harder to click on as the icons become smaller. But again you can change it however you'd like using snippets. I will look into the spacing issue

I think the spacing is actually caused by your theme. Try this with the default theme or the minimal theme and let me know if this is still a problem.

The above is the effect of switching to the default theme

Thank you very much for your response. I have found it. I rewrote the style and achieved the desired effect.

I found it. You have written the styles in the JS file. I can make the changes directly in there. Thank you very much for your response. That's great!

The above is the effect of switching to the default theme

This is just the natural margin between the document and the top of the page I believe

Why does the sidebar float above the content when I use it on a smaller resolution, but not on a larger one? In fact, I don't need this floating effect because it obscures some content. It's not very convenient for reading.

Because on a small resolution there is not enough space for the document and both sidebars to be open at once. So it makes them float so they don't squish the content down tiny if you open both at once.

Also keep in mind that if you change the styles in the js file then all your changes will be overwritten when you update in the future. Just use snippets to override certain styles.

You can disable the floating effect with styles as well. There is a class for floating sidebars. But keep in mind that this will completely break your website on mobile!

Somewhat tangentially, I'd be in favour of a simpler option that has no sidebars at all, with content taking up full width (optionally constrained to some maximum), and then with a top navigation bar with a half a dozen links defined by the user to whatever pages).

Somewhat tangentially, I'd be in favour of a simpler option that has no sidebars at all, with content taking up full width (optionally constrained to some maximum), and then with a top navigation bar with a half a dozen links defined by the user to whatever pages).

In the next update you will be able to generate pages with no extra content other than the document. You will also be able to inlcude custom head content including scripts. This could be used to generate the top navigation bar. Or you could use something like PHP.
I likely will not add specific support for a top navigation bar soon, but thank you for the suggestion.

Sidebar-free multifile output sounds ideal!

Thank you for your reply. I will try to modify the style fragment of the sidebar. The current effect is still very good even when collapsed on larger screens. I am looking forward to your upcoming updates, and I believe there will be some surprises for sure.

I will close this for now since the specific issue has been resolved


Open member using STRPCCMD

Is your feature request related to a problem? Please describe.
We have a green-screen source management tool, where we can edit the sources with a PDM-like option. It would be nice it we could define an option code that opens the source for editing in vscode-ibmi. (Could also work from PDM itself.)
Describe the solution you'd like
If it was possible from a windows command line to trigger somehow the IBM i: Open File command, then it would also be possible to do this from the IBM i server via STRPCCMD, and a PDM-like option could be defined for this.
Describe alternatives you've considered
I have tried to run code -r member:/lib/file/member.ext, but it did not work.
Additional context
Maybe a small js script could also work by connecting to the running vscode instance and triggering this IBM i:Open File command. I have tried the member: scheme, because I have seen in the source code, that this was registered with vscode.workspace.registerfileSystemProvider, but unfortunately, I am not a JS developer, so I could not figure out how it works.

Hi @ngmhun Thanks for a cool idea!
Sounds very similar to an idea I had before.
Does your idea look something like this video? https://www.youtube.com/watch?v=FtgVTZLD3Hc&ab_channel=LiamAllan

Yes, something like that, but maybe connecting to a specific session is not necessary, with the STRPCCMD method, opening a member from any session would work.

@ngmhun Need some time to think about how this could work. Will get back to you.

Ya, this would be very cool to be able to use. For companies that already have software management and do have sources spread on so many libraries, making a "shortcut command" to be able to use directly inside an emulator to open that source (library/file) in VS Code, would be great.

Thank you! I don't know how this communication with an interactive session works. Did you consider listening to a data queue?

@ngmhun Data queue has and will continue to be considered while I work on this feature.
This will work by the VS Code connection reading the job log and looking for specific commands that start with c4i.

I have it working using named pipe.
I can create a PR once #543 is merged.


bash-language-server: only deploy prod dependencies

Reduces closure size to only
 ➜ du -sh ./result/
7,1M    ./result/

from
 ➜ du -sh ./result/
84M     ./result/

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks!


Class builder should take names and values instead of object literals

So that
class C {
    x() {}
    get y() {}
}

Translates to something like:
var C = _esdown.class(function(__) {
    __("x", function() {});
    __.get("y", function() {});
});

Leaving it as-is for now.


Resolve issue with sending notifications for group membership change

Coverage increased (+0.06%) to 84.802% when pulling 4aacd06190595b55d916639eb7d159f4ce78d39e on bug/webhook_sender_errors into db07d426510144205f814993d3faec2d4e988fc2 on master.


Modules revised for phantompeakqualtools and ngsplot in config file; Code revised for ngsplot in main.nf

Looks great! 👍


Remove dubious arguments from solana node configuration

What does this PR do?
Removes configuration options from the solana node configuration that are likely to lead to poor performance and/or unnecessary support burden
Motivation
a flood of new operators with poorly performing nodes seeking support on the solana tech discord server
More

[ ] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[ ] Mandatory for new node blueprints. Yes, I have added usage example to the README.md file in my blueprint folder
[ ] Mandatory for new node blueprints. Yes, I have implemented automated tests for all stacks in my blueprint and they pass
[ ] Mandatory for new node blueprints. Yes, I have added a reference to my README.md file to website/docs section for this feature
[ ] Yes, I have set up and ran all pre-merge quality control tools on my local machine and they don't show warnings.

For Moderators

[ ] The tests for all current blueprints successfully complete before merge?
[ ] Mandatory for new node blueprints. All pre-merge quality control tools and cdk-nag tools don't show warnings?
[ ] Mandatory for new node blueprints. The deployment test works on blank AWS account according to instructions in the README.md before merge?
[ ] Mandatory for new node blueprints. The website builds without errors?

Additional Notes
please have some support staff keep an eye on https://solana.com/discord

I'm not a reviewer but this looks like a great change to me.

I can only echo my approval as a participant in the Solana ecosystem

Thank you very much for this PR. I'm running "smoke" tests to everything is working well before merging. Should wrap it up by the end of this week, so please stay tuned!

this is actually one of the worst flags to remove. we regularly get folks on discord with nodes not able to connect to the network.  they have copypasta'd that flag from somewhere, misconfigured their firewall and don't get log messages letting them know what's wrong as a result.

which port(s) specifically are violating the restrictions? the tests that flag disables should not be accessing any ports that the validator doesn't need to operate

Yeah, that doesn't make sense to me. --no-port-check removes the sanity check that the ports you declare are actually open and can be used.
Skipping the port check and then trying to use RPC services on a closed port doesn't make a whole lot of sense.

Thank you for clarifications.
I can assure you that it will not be a problem in this setup because firewall rules are controlled in VPC security group by CDK and not the scripts within the EC2 instance (see details below). To limit the room for errors I explicitly specify the port numbers in the startup scripts too.
Here are the ports we control:

Ports open to the Internet (TCP/UDP): 8801, 8812 "P2P protocols (gossip, turbine, repair, etc)"
Ports restricted to internal use from within the VPC (TCP only): 8899, 8900 "RPC ports HTTP /WebSocket" <--- these are causing the problem

Link to the code.
The test initiates connection from the outside to all necessary ports, including 8899 and 8900, which can't be accessed by design. Here is the log message captured during startup:
[2024-01-08T00:29:05.193062671Z INFO  solana_net_utils] Checking that tcp ports [(8899, TcpListener { addr: <IP_ADDRESS>:8899, fd: 87 }), (8900, TcpListener { addr: <IP_ADDRESS>:8900, fd: 88 }), (8801, TcpListener { addr: <IP_ADDRESS>:8801, fd: 89 })] are reachable from <IP_ADDRESS>:8001
Opening these ports to public without extra protection exposes nodes to security risks caused by undesirable traffic to RPC. This will not pass our security review. What are your usual recommendations for protecting ports 8899 and 8900?

This will not pass our security review. What are your usual recommendations for protecting ports 8899 and 8900?

you can pass --private-rpc. this will affect a few things

omits the rpc/ws ports from the startup port checks
omits the rpc/ws ports from being published with gossip contact info
changes the default rpc/ws bind address to localhost, rather than adopting the gossip bind address (override with --rpc-bind-address ADDRESS)

note that this does not necessarily restrict foreign addresses from connecting to the service! that remains the job of a firewall and/or reverse proxy
does this achieve an acceptable result?

Yes, that should work in theory.
Let me test it today and I'll get back to you.

All right. The test is not successful so far. With --private-rpc and --rpc-bind-address ADDRESS but without --no-port-check the validator process starts syncing while also gradually increasing the amount of consumed RAM. Eventually RAM it runs out of memory and crashes without fully syncing. I tested on version v1.16.25. Will try switching to v1.16.26 tomorrow.

nothing between those two versions would affect memory usage. nor, removal of any of the proposed cli flags.
what are the specs of the test machine?

I upgraded to version 1.17.16 and ran another test. The memory consumption problem has not repeated itself. I also made a few more changes and bug fixes, pushed to another branch and opened PR #32 . Closing this PR and let's move our conversation to PR #32 . @t-nelson will be great if you can have one more look and give your feedback.


Enhancement: Attachment Sequence

The Attachment Format was assigned as ${notename} in the plugin settings.
Eg. Note named "try-attachment"
Actual
The attachment seq. generated was try-attachment, try-attachment-1, try-attachment-2....
Expected
The attachment seq. to be try-attachment-1, try-attachment-2, try-attachment-3....
It would be neat if the sequence number (1) is assigned from the very first attachment.

Thanks for a fantastic utility plugin 👍

Hi @hrxkr , the suffix number was added for duplicate file name if there is already an attachment with name try-attachment. But, we don't konw whether there will be more same file add to one folder.
So, there is no reason to add a number 1 to the attachment that be firstly placed. Hope you understand.


Connection SDK library is not using shared scripting engine

This feature was added later because every call into Photoshop was executing a fresh new script engine which also means executing all startup scripts and this causes about 80ms delay per each call. But connection SDK is slower anyway per one call. CEP panel is a bit faster.
You can compare python photoshop protocol with connection made by adobe.
Script type of message has constant 2 and script in shared engine 10. And in python library, this is missing.
https://github.com/kyamagu/photoshop-connection/blob/master/src/photoshop/protocol.py#L17
https://github.com/adobe-photoshop/generator-core/blob/24a2c4e38aef79adb42869f200778ab52de4ef36/lib/photoshop.js#L67

Thanks a lot for the heads up. Adding a photoshop tag in case there's someone able to improve the connection to Photoshop.

One more thing... using not shared engine also means that all variables in the global PS scripting context will be wiped out once call is done. Otherwise, you could cache there something useful for all your calls.


Fixed unauthorized error

Fixes issue with unauthorized error after login.
Issue
Chrome sends authorization TOKEN to ui before it is sent to APIs. So the UI loads but the user is not authorized in APIs so the requests will throw an error.
Changes
Adds a workaround. UI waits on auth token from chrome (the same that is sent to API) and shows placeholder until auth token is received. This should be solved in chrome , but because they don't own the auth service they cannot do this right now. Maybe they will come up with something in a future.

@Hyperkid123 Is this how other apps on the platform resolved the issue?

@gmcculloug - yes, this is the recommended way -for now, wait until the token is received.


Distinct until changed

Why not use distinctUntilChanged here https://github.com/devxoul/RxTodo/blob/master/RxTodo/Sources/Rx/RxOperators.swift#L43
The first publish will set, the second won't as it's not distinct.

@seivan, I couldn't understand what you expected. Can you provide me some detail?

I'll make a pr as soon as I can. Otherwise ping me in slack.
Sent from an iPhone on the go.

On 3 Aug 2016, at 17:46, Jeon Suyeol<EMAIL_ADDRESS>wrote:
@seivan, I couldn't understand what you expected. Can you provide me some detail?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Ah completely slipped my mind. But I can see the code has changed since then.


[HW] Biondi Septian S

Name: Biondi Septian S
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/biondiseptian/
Attach the homework screenshots below for both step II and step III:

Hey there Biondi, thank you for your submission. Nice job getting Through the workshop materials. The only thing we are missing is step 3 of the homework.
3. Complete the following short courses: Cassandra Data Modeling and Cassandra Query Language. Take screenshots of the "Congratulations" page from each course.
Please post an image of the "completed" screen once finished.


refactor(home): simplify refresh method

Simpler and more performant refresh method for the home component.
CC @PhPrinz

I am in support of any PR that removes LOC. 👍


Modal Sheet on Mobile - Aligning items to bottom

Opening this issue to follow up on https://discord.com/channels/909986013848412191/974145843919716412/1095928073653858314
Basically it seems like Modal Sheet on mobile is height is actually bigger than the displayed sheet, this means that if one tries to align items within the sheet to the bottom, those items would often render outside of the screen.
As an example, open https://tamagui.dev/docs/components/sheet/1.9.18 on iOS, make sure the type is modal and click Open. Then if you manually add "justify-content: flex-end" to the sheet node, you'll see the bottom button and half of the input is rendered outside of screen (at least on iPhone 14 Pro Max, I assume different device heights affect what you'll see)
On the surface of it, it seems like the sheet is set to height: 100% (i.e. the full screen height) but is then translated down, but I'm not sure if that's actually the issue.

Hey kzlar 😊  Thanks for the PR
I still wonder how this fix fixed this issue
I am still encoutering issues upon updating, I struggle to make a simple modal={true} view and justify-content space-between
I upgraded to last version tho


Are browsers supported?

Similar to cordova-plugin-file-opener2, but without browser support?

Browser is not supported


[install_script] Set proper config ownership and permissions even if config already existed

What does this PR do?
This is necessary for usage with the BeanStalk template, which creates the file and sets its ownership to root.
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

On a fresh system without the Agent installed, create /etc/datadog/datadog.yaml. Make it owned by a user other than dd-agent and set permissions other than 0640.
Invoke the install script.
Ensure that /etc/datadog/datadog.yaml is owned by root:root and its permissions are 0640.

Reviewer's Checklist

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

The version at https://s3.amazonaws.com/dd-agent/scripts/install_script.sh, which is used by the instructions on this page, doesn't have this fix.

Hi @jacobweberbowery,
Thanks for the reminder! We are planning on releasing a new version of the install script, including this change, this week.


docs: remove redundant headings (box/box-openapi#442)

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

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 41.914%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/schemas/fileOrFolderScope.generated.ts
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/schemas/fileOrFolderScope.generated.ts
1
45.31%

Totals

Change from base Build<PHONE_NUMBER>3:
0.01%

Covered Lines:
12842

Relevant Lines:
23849

💛  - Coveralls


[$500]  Android - Keyboard does not open automatically when replying to message in a task report

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

Action Performed:

Open any report
Create a task
Send any message in task report
Select message and press reply in thread
Observe keyboard does not open automatically

Expected Result:
The keyboard should be opened automatically
Actual Result:
The keyboard is not opened automatically
Workaround:
Unknown
Platforms:

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

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/38287caf-399c-495b-a6c1-2530bf0983c9
https://github.com/Expensify/App/assets/93399543/8413e93e-35e7-4011-9ed0-cd54701bcaef
Expensify/Expensify Issue URL:
Issue reported by: @krishna2323
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1694366917362669
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016cb6134fb786c7cd
Upwork Job ID:<PHONE_NUMBER>235686912
Last Price Increase: 2023-09-13

Re-opened other GH, since it was reported first!


Hidden channels list becomes blank

In a team that has channels set to hidden (not private), the hidden channels should be visible in the flyout list when clicking the "n channels hidden    >" link in the Teams sidebar.
Often this listing starts to appear empty, despite many channels that should appear there. Of course this makes it impossible to go to those channels.
This appears to happen irrespective of the number of hidden channels. I happens both for teams with 2 hidden channels and also for teams with ~70 hidden channels. It happens for all teams shown in the sidebar, at the same time (i.e. it doesn't happen to just one team at a time, but to all teams shown in the sidebar)
A restart of the application usually resolves this; this is a workaround rather than a fix though.
Teams version <IP_ADDRESS>55 (64-bit)
Windows 1809 (build 17763.1457)

So there's no way for users to provide feedback. Great. 👎


Deleting .godot folder make some meshes disappear from the scene

Godot version
4.1
System information
Godot v4.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated GeForce GTX 1050 Ti () - Intel(R) Core(TM) i5-9300H CPU @ 2.40GHz (8 Threads)
Issue description
Godot version:
4.1
OS version:
Windows 10
Issue:
Here's the baseline scene:

After closing Godot I deleted my .godot folder to see if I could add it to the .gitignore file. On restarting the engine I noticed that one of the meshes in the scene had disappeared (see MeshInstance3D2 Mesh field):

.godot folder was automatically created without explicit errors
After that I restarted Godot again and the mesh spontaneously reappeared:

I'm not sure ifthis behaviour is intended, as multiple restarts do not seem to be a good practice (and UX).
Steps to reproduce

Download the project
Launch it, open the levels/TestLevel1.tscn scene to see if MeshInstance3D2 has Mesh assigned to it
If it is not, then close Godot and open it again. MeshInstance3D2 should probably get Mesh assigned.
Then delete the .godot folder and restart the engine, notice the MeshInstance3D2 behavior.
Restart Godot again and check the MeshInstance3D2 .

Minimal reproduction project
hovercraft_issue.zip

Could you explain your process for creating that MeshInstance?
Its not surprising that deleting the .godot folder while the editor is running would cause issues. Resources that are imported are stored in the .godot folder, since the engine relies on those imported resources, things break.
What is especially concerning though is that the mesh itself appears to disappear and then it reappears after a reload. That probably shouldn't be happening. But to diagnose this issue further, we need to know how the MeshInstance was created, i.e. was it imported directly, was the mesh copied from another source, etc.

I did not delete the .godot folder with the engine still running. First I closed it, then deleted the folder. My bad, I didn't mention that.
My process for creating MeshInstance is:

Select root node in the scene
Press Ctrl+A anf add new MeshInstance
Place it in 3D space
Model some meshes in Blender 3.6, import them via .glTF official importer as .glb file (res://assets/gltf_files/test_scene_1.glb)
In Advanced Import Settings, press Actions... → Set Mesh Save Paths → Save them to the same folder as .glb file
Actions... → Extract Materials → Save them to assets/materials folder
Assign desired Mesh (in this case, test_scene_1_Thing.res) to the MeshInstance3D2
The same assignment procedure was done for another Mesh, with ewrewr name (test_scene_1_Cube.res → ewrewr, sorry for that naming), but there was no problem with it

Does it also happen with a .tres mesh? Possibly related https://github.com/godotengine/godot/issues/77978

I wonder if this is a more general issue related to deleting the .godot folder. Another user and I had both noted in a Discord discussion yesterday that the first editor launch after deleting the .godot folder results in many errors and scenes not opening correctly, but, after restarting again, the second launch works as expected. Errors are gone and scenes open normally.
The editor is closed at the times I delete the folder (which The last time this occurred for me, I was working in 2D/Godot 4.1-stable/Windows 11. I don't know about the other user.

I also suspect that the auto reload functionality is triggering in correctly.

Do you mean 4.2.1? Or a custom 4.2.2-dev build?


Add @_disfavoredOverload attribute to affect overload resolution.

Introduce an attribute @_disfavoredOverload that can be used to state
that a particular declaration should be avoided if there is a
successful type-check for a non-@_disfavoredOverload. It's a way to
nudge overload resolution away from particular solutions.

@swift-ci please smoke test

@swift-ci please test source compatibility

Oh no 😭

@swift-ci please smoke test

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please smoke test

When I went down this path for Self, it didn’t work out well. Hope it’s better in this context.


wrap user messages

User messages were not wrapping long lines:

This PR fixes this:

Excellent UI catch!


Some little improvements

Hope you like it

Sorry but I see no value in these changes. I want to check that enough arguments were provided, performance of editing one string is negligible, and I like the current docs formatting.
Anyway thanks for the effort.


chore: gracefully handle returning servers when dns not configured

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1335
#1334  👈
#1333
#1332
#1329
#1328
#1327
#1322
#1321
#1320
#1319
#1318
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Suggestion to be able to select Bluetooth radios to use

Just deployed yesterday and It is working great!
Just one challenge:
I have four ESP32 proxys around my house, plus the Pi4 Bluetooth radio. The issue that I am facing is that the Pi4 captures most of the activity from the BLE devices nearby.
For example, while my smart watch is 1-2 meters from the Kitchen ESP32, the Pi4 (8-10 meters away) picks up a stronger signal  and provides a false location.
It would be helpful to be to select which radios to utilize.
Thank you

Hmmm good point.
I have an issue to implement independently calibrating both proxies and beacons at #45 that will let you effectively define the receiver sensitivity by proxy - so you'd offset the bluetooth adaptor so that it reported accurate distances.
This would solve the particular problem you are having, but I think the ability to disable a scanner completely from calculations is a good idea - we'll need it if/when we support scanners that are not fixed in location (like mounted on a robot vacuum etc).
Right now I'm tossing up where to put the configuration - whether to put it on the device page as an entity or whether to put it into the options_flow when you click the "Configure" button for Bermuda. The latter is starting to get a bit crowded, but it might be the better place for it.
I'll make a decision on that soon, and get a fix sorted. My bt dongle also gets a stronger signal than the esp32's, so yeah I get warped into the garage more often than I should, too! 😅

This is exactly my issue. I have mainly two types of ESP32 boards that would require independent calibration of RSSI strength/sensitivity to achieve accurate placement. Looking forward to the independent calibration.

While you're adding individual base station calibration, I would wonder if the area of each room could be specified as well. Some rooms are small and others are large. Shrinking area radius of the smaller rooms would be even more important when the adjoining room does not have an ESP32 base station.

Hello, I'd like to express my gratitude for the great project. It's working better for me than other projects of this type that I've tried. I suggest placing it on the device page as an entity, as putting it on the configuration page might make it easy to miss. As someone who has other integrations with multi-page configurations, it's hard to remember what's on each page and where it is.

Duplicate of #45

I'm going to close this one out and track progress on the original issue #45 which will include both per-proxy offset for ref_power and a toggle to select whether to use the given proxy for calculations or not.


[WIP] identify missing functions in MLX backend

This PR is part of a learning process to understand how to send PRs to the mlx branch.
During this process, I have identified missing functions in the MLX backend and have added them as TODOs. I will be happy to address them later.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.36%. Comparing base (fb6244b) to head (910c748).

Additional details and impacted files
@@           Coverage Diff           @@
##              mlx   #19540   +/-   ##
=======================================
  Coverage   52.36%   52.36%
=======================================
  Files         377      377
  Lines       42838    42838
  Branches     8346     8346
=======================================
  Hits        22433    22433
  Misses      18817    18817
  Partials     1588     1588

Flag
Coverage Δ

keras
52.36% <ø> (ø)

keras-numpy
52.36% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Hi @fchollet,
Since there are conflicts, and to make it easier for you, I'm thinking of deleting this PR and creating a new updated one.
Sound good?

Since there are conflicts, and to make it easier for you, I'm thinking of deleting this PR and creating a new updated one.
Sound good?

Sure, sounds good!


Invalid semantic version in hpa template

First check

[X] I added a descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the Prefect documentation for this issue.
[X] I checked that this issue is related to Prefect and not one of its dependencies.

Bug summary
hpa.yaml is specifying a semantic version comparison of '<1.23-0"` here:
https://github.com/PrefectHQ/prefect-helm/blame/abe3dd1ce7979102b0f7209d77d04ad57f5c961a/charts/prefect-server/templates/hpa.yaml#L27-L39
This is an invalid semantic version.
Reproduction
Here's an example of trying to validate the helm charts when you have the following property set to true:

prefect-server:
  server:
    autoscaling:
      # -- enable autoscaling for server
      enabled: true

> helm template . --values deployments/values-prod.yaml --debug
install.go:194: [debug] Original chart version: ""
install.go:211: [debug] CHART PATH: /tools/prefect

Error: template: prefect/charts/prefect-server/templates/hpa.yaml:2:15: executing "prefect/charts/prefect-server/templates/hpa.yaml" at <include "common.capabilities.hpa.apiVersion" .>: error calling include: template: prefect/charts/prefect-server/charts/common/templates/_capabilities.tpl:133:7: executing "common.capabilities.hpa.apiVersion" at <semverCompare "<1.23-0" (include "common.capabilities.kubeVersion" .context)>: error calling semverCompare: Invalid Semantic Version
helm.go:84: [debug] template: prefect/charts/prefect-server/templates/hpa.yaml:2:15: executing "prefect/charts/prefect-server/templates/hpa.yaml" at <include "common.capabilities.hpa.apiVersion" .>: error calling include: template: prefect/charts/prefect-server/charts/common/templates/_capabilities.tpl:133:7: executing "common.capabilities.hpa.apiVersion" at <semverCompare "<1.23-0" (include "common.capabilities.kubeVersion" .context)>: error calling semverCompare: Invalid Semantic Version

### Error

```python3
Error: template: prefect/charts/prefect-server/templates/hpa.yaml:2:15: executing "prefect/charts/prefect-server/templates/hpa.yaml" at <include "common.capabilities.hpa.apiVersion" .>: error calling include: template: prefect/charts/prefect-server/charts/common/templates/_capabilities.tpl:133:7: executing "common.capabilities.hpa.apiVersion" at <semverCompare "<1.23-0" (include "common.capabilities.kubeVersion" .context)>: error calling semverCompare: Invalid Semantic Version
helm.go:84: [debug] template: prefect/charts/prefect-server/templates/hpa.yaml:2:15: executing "prefect/charts/prefect-server/templates/hpa.yaml" at <include "common.capabilities.hpa.apiVersion" .>: error calling include: template: prefect/charts/prefect-server/charts/common/templates/_capabilities.tpl:133:7: executing "common.capabilities.hpa.apiVersion" at <semverCompare "<1.23-0" (include "common.capabilities.kubeVersion" .context)>: error calling semverCompare: Invalid Semantic Version

Versions
2.10.3

Additional context
No response

@achordia20 What version of Helm are you running? I'm running the latest version of Helm and it works fine for me:
$ helm version
version.BuildInfo{Version:"v3.11.2", GitCommit:"912ebc1cd10d38d340f048efaf0abda047c3468e", GitTreeState:"clean", GoVersion:"go1.20.2"}

$ helm template server charts/prefect-server/ --set prefect-server.server.autoscaling.enabled=true --set postgresql.auth.password=test
---
# Source: prefect-server/templates/serviceaccount.yaml
apiVersion: v1

Ah, I can reproduce the problem now (deleted my previous comment to avoid confusion):
$ helm template server charts/prefect-server/ --set server.autoscaling.enabled=true --set postgresql.auth.password=test --set server.resources.requests.cpu=1 --set server.resources.requests.memory=1Gi
Error: template: prefect-server/templates/hpa.yaml:2:15: executing "prefect-server/templates/hpa.yaml" at <include "common.capabilities.hpa.apiVersion" .>: error calling include: template: prefect-server/charts/common/templates/_capabilities.tpl:133:7: executing "common.capabilities.hpa.apiVersion" at <semverCompare "<1.23-0" (include "common.capabilities.kubeVersion" .context)>: error calling semverCompare: Invalid Semantic Version

Use --debug flag to render out invalid YAML

We'll investigate a fix, thanks for the report!

We intend to resolve this issue by dropping support for Kubernetes version 1.22, as it predates the general availability of HorizontalPodAutoscaling.


CI: Workaround YAML gotcha in Actions

To avoid unexpectedly stop testing Ruby 3.0 when Ruby 3.1 is released.
See actions/runner#849
At https://github.com/Shopify/graphql-batch/runs/1691443768?check_suite_focus=true#step:3:4 we can see that the setup-ruby action ran with just 3 as the input and not 3.0.

Good ol' yaml, thanks!


python310Packages.proxmoxer: 2.0.0 -> 2.0.1

Diff: https://github.com/proxmoxer/proxmoxer/compare/refs/tags/2.0.0...2.0.1
Changelog: https://github.com/proxmoxer/proxmoxer/releases/tag/2.0.1
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 207017 run on x86_64-linux 1

3 packages built:

prometheus-pve-exporter
python310Packages.proxmoxer
python39Packages.proxmoxer


🌱 Enable linting on pull requests

What this PR does / why we need it:
🌱 Enable linting on Pull Requests
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #838
Special notes for your reviewer:
Checked this on a fork here
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:

[ ] squash commits
[ ] include documentation
[ ] add unit tests

@kranurag7 We use pull_request_target everywhere, this needs to be changed accordingly. Also, please add me as a reviewer next time.


Loadoff Failed (Uitladen Mislukt)

Hi,
No real bug to be honest, however I do get the message "uitladen mislukt", which I think is the raw translation of loadoff failed.
As Home Assistant does not display any additional information regarding this alert (yellow alert sign is displayed), I am curious why this is visible and how it impacts the working of this excellent integration.
Kind regards,
Guus

Hi Guus, this is probably "Unload failed". This integration supports unloading. But to be completely honest, I am not sure when Home Assistant would try to unload an integration. Is the error always there? Even after restarting Home Assistant? Is there anything in the logs about this?

Hi Vincent,
Thank you for the fast reply, appreciated!
I agree, would not expect the need for unloading. The integration is running non stop and seems to function as designed.
Yes, even after a reboot of the RPi, the error returns. Maybe I should just ingnore it, however the exclamation mark triggers attention :)..
Normally I would check for a seperate button ‘log’, do you have a quick how-to where this located in this integration?
Kind regards,
Guus

The error/message, if there is any, should be in your System logbook.

Hi Vincent,
Thanks, I think this is the error related.
I hope, I’ve redacted all ‘personal’ data :)..
Deze fout is ontstaan door een aangepaste integratie.
Logger: homeassistant.config_entries
Source: custom_components/enphase_envoy/envoy_reader.py:579
Integration: Enphase Envoy (Installer) (documentation, issues)
First occurred: 14 augustus 2023 om 20:14:32 (1 occurrences)
Last logged: 14 augustus 2023 om 20:14:32
Error unloading entry Envoy redacted for privacy for enphase_envoy
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/anyio/streams/tls.py", line 131, in _call_sslobject_method
result = func(*args)
^^^^^^^^^^^
File "/usr/local/lib/python3.11/ssl.py", line 921, in read
v = self._sslobj.read(len)
^^^^^^^^^^^^^^^^^^^^^^
ssl.SSLWantReadError: The operation did not complete (read) (_ssl.c:2576)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/httpcore/_backends/anyio.py", line 34, in read
return await self._stream.receive(max_bytes=max_bytes)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/anyio/streams/tls.py", line 196, in receive
data = await self._call_sslobject_method(self._ssl_object.read, max_bytes)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/anyio/streams/tls.py", line 138, in _call_sslobject_method
data = await self.transport_stream.receive()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/anyio/_backends/_asyncio.py", line 1203, in receive
await self._protocol.read_event.wait()
File "/usr/local/lib/python3.11/asyncio/locks.py", line 213, in wait
await fut
asyncio.exceptions.CancelledError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/httpcore/_exceptions.py", line 10, in map_exceptions
yield
File "/usr/local/lib/python3.11/site-packages/httpcore/_backends/anyio.py", line 32, in read
with anyio.fail_after(timeout):
File "/usr/local/lib/python3.11/site-packages/anyio/_core/_tasks.py", line 119, in exit
raise TimeoutError
TimeoutError
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/httpx/_transports/default.py", line 60, in map_httpcore_exceptions
yield
File "/usr/local/lib/python3.11/site-packages/httpx/_transports/default.py", line 239, in aiter
async for part in self._httpcore_stream:
File "/usr/local/lib/python3.11/site-packages/httpcore/_async/connection_pool.py", line 347, in aiter
async for part in self._stream:
File "/usr/local/lib/python3.11/site-packages/httpcore/_async/http11.py", line 325, in aiter
raise exc
File "/usr/local/lib/python3.11/site-packages/httpcore/_async/http11.py", line 317, in aiter
async for chunk in self._connection._receive_response_body(**kwargs):
File "/usr/local/lib/python3.11/site-packages/httpcore/_async/http11.py", line 186, in _receive_response_body
event = await self._receive_event(timeout=timeout)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/httpcore/_async/http11.py", line 200, in _receive_event
data = await self._network_stream.read(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/httpcore/_backends/anyio.py", line 31, in read
with map_exceptions(exc_map):
File "/usr/local/lib/python3.11/contextlib.py", line 155, in exit
self.gen.throw(typ, value, traceback)
File "/usr/local/lib/python3.11/site-packages/httpcore/_exceptions.py", line 14, in map_exceptions
raise to_exc(exc) from exc
httpcore.ReadTimeout
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 544, in async_unload
result = await component.async_unload_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/enphase_envoy/init.py", line 293, in async_unload_entry
await task
File "/config/custom_components/enphase_envoy/init.py", line 251, in read_realtime_updates
result = await envoy_reader.stream_reader(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/enphase_envoy/envoy_reader.py", line 579, in stream_reader
async for chunk in response.aiter_text():
File "/usr/local/lib/python3.11/site-packages/httpx/_models.py", line 948, in aiter_text
async for byte_content in self.aiter_bytes():
File "/usr/local/lib/python3.11/site-packages/httpx/_models.py", line 927, in aiter_bytes
async for raw_bytes in self.aiter_raw():
File "/usr/local/lib/python3.11/site-packages/httpx/_models.py", line 985, in aiter_raw
async for raw_stream_bytes in self.stream:
File "/usr/local/lib/python3.11/site-packages/httpx/_client.py", line 146, in aiter
async for chunk in self._stream:
File "/usr/local/lib/python3.11/site-packages/httpx/_transports/default.py", line 238, in aiter
with map_httpcore_exceptions():
File "/usr/local/lib/python3.11/contextlib.py", line 155, in exit
self.gen.throw(typ, value, traceback)
File "/usr/local/lib/python3.11/site-packages/httpx/_transports/default.py", line 77, in map_httpcore_exceptions
raise mapped_exc(message) from exc
httpx.ReadTimeout

The error seems to be the same as in #81. Do you have real-time updates enabled (See configure button at the integration)?

It seems indeed to be related.
Yes, real-time updates is enabled.
I read that you advise to disable it, and see if that results in no more errors, will try that this week.
For the record:
Account type: [x ] Installer
[x ] Metered with CTs installed
FW version: D7.6.175
Amount of micro inverters connected: 10
Home Assistant Version: 2023.8.2

I’ve disabled real-time updates, and the error is gone. If you prefer, this issue may be closed. However I will watch issue #81 . However if you would like to check/test things, let me know!

@mnederlof, any idea about this one? It seems to throw an exception during unloading the stream endpoint. Not easy to debug for me with my Envoy Standard.

It has to do with logic I added previously for handling the realtime task, but I can debug in a few days for sure

@gritcalculator can you please check if 0.3.4-beta1 fixes your issue?

@gritcalculator can you please check if 0.3.4-beta1 fixes your issue?

Hi Vincent,
Sorry for waiting, was abroad a few days.
I've installed the 0.3.4 non-beta relaase. It seems that error messages do not populate directly, which was the case previously.
If things change, I'll let you know. For now case closed (saw that this has already been done).
Thanks for helping me out! Also a thank you to the other programmer/author which stepped in to debug!


Periodic polling of megad states

Need for the case when we missing events from megad for some reason, restart device etc.

done


Error when first loading dashboard page: Cannot read properties of undefined (reading 'TextHighlightRules')

What happened:

This error appears whenever the Grafana dashboard page first loads. All datasources are broken. On a page refresh this error goes away and things work as expected.
What you expected to happen:
This error should not pop up and the TwinMaker datasources should work properly on the first load of the page.
How to reproduce it (as minimally and precisely as possible):
I am unable to produce from the dev team. This is happening with a customer's Grafana self-managed server.
Anything else we need to know?:
Environment:

Grafana version: 9.5.2
Plugin version: 1.7.1

Any update ?

We're waiting for more info in order to be able to reproduce this. Will update when we know more.

@hwandersman is this issue still happening for your customer? Do we have repro steps for this one yet? If not maybe we can close as stale

Closing as stale

Likely resolved on TM side.


Specify embedded object types in json rules

Sometimes it is not possible to modify incoming JSON in a way that it contains __rec or __type attributes in the objects (existing public APIs). At that case json rule should contain type information.
-json({connection, port, {type, protocol, "protocol"}}).
-json({protocol, {atom, name}}).

decode(Input) ->
  from_json(Input, connection).

In the from_json I can specify what data type I expect. The protocol attribute of the JSON will be a protocol record type (at least named tuple).

This has been implemented by specifying the type in the field option list of author-list.


Why does submit form?

Hi, I have got a bootstrap modal form with datetimepicker as input. If i leave the datetimepicker field the background form will be submited. Why?

You have to provide us with some code in order to help you.


[enhancement]: Two-way communication for Doorbell

Describe the solution you'd like
Current Situation
When someone rings the bell, I need to open eufy app to respond, ideally using Apple Home app would be way better
Proposed Change
Implement Two-way audio communication, and maybe if possible also some "fake switches" to play the "doorbell pre-recorded answers" that are available in eufy app

Hi @cpvbruno
This is already planned.
Unfortunately the needed functionality has just been added to eufy-security-client in version 2.1.0 and this version has some bugs that prevent it from being used straight away with this plugin. There's an issue about this on the repo.
Nonetheless we'll try to implement it as soon as possible.

I enabled it but the option to disable “RTSP” disappeared, do you experience the same?

I just tried on my doorbell and I couldn't hear anything at all, unfortunately :(

I've tested talkback and it's working great. Thanks for this awesome implementation.

I've tested talkback and it's working great. Thanks for this awesome implementation.

Did you have to tweak anything to make it work?

Just updated and enabled talkback in settings. Both my doorbell and indoor cam are working great!

I just tried on my doorbell and I couldn't hear anything at all, unfortunately :(

@cpvbruno what is your exact homebridge setup? As mentioned above I had trouble with talkback on windows which looked kinda like your screenshot: the talkback process just quits after approximately 20-30 seconds with code 0 and signal null.

I've tested talkback and it's working great. Thanks for this awesome implementation.

Did you have to tweak anything to make it work?

Hi, just checked the talkback option in the settings (found in the ui)

@thieren
homebridge.log.txt

@cpvbruno Please don't mix up the issues. ;-)
Here the last question was about your exact homebridge setup. If you're running homebridge on windows, this might explain why talkback is not working. If it's another installation method, I'd have to try it maybe.
The log was for #114 I think. And no. The storage path to the plugin specific log-lib.log file will be written at startup of the plugin since the latest beta. Please get me this.

Oh sorry @thieren , my setup is a MacBook Pro M1

Didn't try it with this, but your luck is, that I have the same device. Will try to reproduce later.
Did you install with this method?

Yes, this exactly method

I was accessing my Mac remotely so I didn't see this bunch of messages, it could definitely be the problem, I'll check now

I allowed this permission and now I don't see errors in the logs at least, I'll soon test with my real doorbell to see if I can hear something coming from he speaker

Yeah that definitely looks like a possible cause. :-D
Let me know if this was it.

It worked like a charm ❤

Is implemented with release of 2.1.0

I've got problems with talkback audio. Hearing is quite fine, but if I want to speak to the cam just a lot of weird noise comes out. I use Raspi4, Video works fine at 640x480p.


add phi nodes

Description
This PR contains the second part of add phi nodes.
There are still 4 functions that needs to be removed to finish this.
Related issues
Please mention any github issues addressed by this PR.
Checklist

[ ] I have performed a self-review of my code.
[ ] I have commented hard-to-understand parts of my code.
[ ] I have made corresponding changes to the public API documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have updated the changelog with any user-facing changes.

@cqc-alec Feel free to take a look at this now, I will add the last part in the next days.
I will replace all the four functions for setting and getting bits from the i64 in the last step.

I would like to merge this PR without adding the zext and trunc operations to have separate release candidates and make sure we can test this individually.


🍒 Cherry pick PR #10136 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/10136 to staging 🍒

🚀 Cherry-picked to staging by @OSBotify in version: 1.1.86-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @yuwenmemon in version: 1.1.86-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Implement all available preliminary APIs

[x] Alliance Selection
[x] Ancillary
[x] Awards
[x] [Match Results] (http://docs.frceventsprelim.apiary.io/#matchresults) - Currently being worked on. Implemented function is getMatchResults()
[x] Rankings
[x] Schedule Added in intial commit: 39dbef361c179c2e73daa9c56361df2d83a93766
[x] Season Data

[x] Season Summary

[x] Event Listings

[x] District Listings

[x] Team Listings - currently implemented so you can only get the info of one team. Might use a separate function to get the team listing of an even/district.

Added season summary in commit b68f18864a65100bea419fd9e7aabd771690c0c0

Added event listings in 15a0c4572bdff620dcedf10e004ca2b40e5cf475

Added event alliances in 6d50b3f50fecd94214e1ee319547d8d481c86d2e

Added event rankings: 0ab409a00171f7bb247e6cea721c012d0230e241

Added event awards in c3a468b01312c2eaf1802d1b81517d131267c53c


Make rewards contract able to handle multiple concurrent factories

Refs: #597
Depends on https://github.com/keep-network/keep-core/pull/2158

Closing without merging as we changed the approach for ECDSA staker rewards. The new approach is described in https://blog.keep.network/a-new-rewards-mechanism-deef3412c3e1 and work can be tracked in https://github.com/keep-network/keep-ecdsa/issues/602.


[v6.0.0-alpha.15]  FieldArray move and removeAll undefined?

I'm not sure if maybe something isn't updating properly, but here is what my "inserts" array looks like:

This is after running npm i -S<EMAIL_ADDRESS>Is anyone else seeing this issue with alpha.15?
Thanks!

@tnrich, yep seeing this as well!

Confirmed. This is a bug. The following screenshot is from the published Field Arrays example.

Thanks, @tnrich!

This has been fixed. I will comment again here when the fix has been released.

Published in v6.0.0-rc.1.


[ITensors] Add RXY gate, fix sqrt(iSWAP), add tests

Description
Please include a summary of the change and which issue is fixed (if applicable). Please also include relevant motivation and context. List any dependencies that are required for this change.

Added tests for various ops on Qubits
Added Rxy gate for Qubits
Made Rz's angle input consistent with Rx and Ry
Fix bad division factor for sqrtISWAP gate
Added various tests elsewhere for sitetype and lattices

How Has This Been Tested?
New tests pass locally.
Checklist:

[x] My code follows the style guidelines of this project. Please run using JuliaFormatter; format(".") in the base directory of the repository (~/.julia/dev/ITensors) to format your code according to our style guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have added tests that verify the behavior of the changes I made.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.

Codecov Report

Merging #1080 (71c2bcf) into main (7148b0e) will decrease coverage by 25.54%.
The diff coverage is 28.57%.

:exclamation: Current head 71c2bcf differs from pull request most recent head 4a45fde. Consider uploading reports for the commit 4a45fde to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #1080       +/-   ##
===========================================
- Coverage   67.81%   42.28%   -25.54%
===========================================
  Files          95       94        -1
  Lines        9999     9950       -49
===========================================
- Hits         6781     4207     -2574
- Misses       3218     5743     +2525

Impacted Files
Coverage Δ

src/mps/mps.jl
0.00% <0.00%> (-90.38%)
:arrow_down:

src/physics/site_types/qubit.jl
60.81% <100.00%> (+2.06%)
:arrow_up:

src/adapt.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/adapt.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/projmpo.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/mps/observer.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/matelem.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/qnmatelem.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/opsum_to_mpo.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/physics/autompo/opsum_to_mpo_qn.jl
0.00% <0.00%> (-97.62%)
:arrow_down:

... and 36 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thanks!


Sync up supported ops for 24.02 plugin release

Signed-off-by: Ahmed Hussein (amahussein<EMAIL_ADDRESS>Fixes #795, Fixes #721, Fixes #712
This code change is to sync the supported-ops in the tools with the Plugin release 24.02.0 in addition to disabling JsonTuple

@kuhushukla this includes fix for

ResultQueryStage should be marked as "IgnoreNoPerf"


primeira tradução

traduzindo a primeira linha

Thank you for working on this, @raphaelpe!

This is awesome, @raphaelpe! Is it finished and ready to be merged into the project, or are you still working on it?

Hello Nrotstan, Good evening, I made the translation of every file more appeared in red lines and others in green, what would it be? Does some kind of review need to be done? If you need to do I do, did you do any revision? I await your response! I would like you to send me an e-mail from your staff to send you some requests when it comes to the maproulette project?

Good Night nrotstan, I wonder if a revision of the translation I did was made?

Hi @raphaelpe, I have looked at it briefly, but have not yet tested it. I will do that soon. :-) The current plan is to include it in the next release (v3.2.4)

nrotstan, a comunidade brasileira agradece! /  nrotstan, the Brazilian community thanks!


setting data-value different versions

setting data-value in processElements("data-value",.. doesn't work in
version 1.10 jquery, chrome.
adding:
//before jQuery 1.6
$elem.attr('defaultValue', formatted);
//after jQuery 1.5.2
$elem.attr('value', formatted);
seems to get it working.

fixed in version 1.5.7


Add flag to install the --pre wheels for PyTorch, etc.

Having a flag for easier nightly wheel installation would be nice.

Add ==version as a variable maybe?


Error

I tried to run code in riviera
It’s throw some errors
ERROR VCP5190 “cannot find match for key imm in structure. “ “ ../riscv_vip/src/./instruction_unit_test.sv” 137 1
ERROR VCP5190 “cannot find match for key imm in structure. “ “ ../riscv_vip/src/./instruction_unit_test.sv” 138 1
ERROR VCP5190 “cannot find match for key funct7 in structure. “ “ ../riscv_vip/src/./instruction_unit_test.sv” 139 1
Could you help?

Thanks Alvin for trying the riscv-vip with Riviera.  I have not tried this and I'm not sure if I can get access to that simulator.  I'm not sure if it supports the same subset of the SystemVerilog LRM that Cadence and Mentor simulators do.  Riscv-vip does work with Cadence and Mentor simulators and I develop and test with those.  I'm on vacation currently, but will look into Riviera and the issue you are mentioning when I'm back and have some time.   You may also want to check with Aldec support and point them to this repository to see what they say.  In the mean time, if you have any more information you can provide or if you find some work-arounds, please report back.

I explored this code using EDA playground.  And I may have a workaround for the build error.  I can look at adding this code as a commit, but it would need to be accepted as a pull request, so I offer it here as a quick workaround.
If the following code at lines 137-139:
      insts[0] = i_inst_t'{imm:99    ,rs1:2,   funct3:3,   rd:1, op:LOAD};
      insts[1] = i_inst_t'{imm:'hFF  ,rs1:1,   funct3:2,   rd:5, op:SYSTEM};
      insts[2] = r_inst_t'{funct7:0, rs2:1, rs1:1, funct3:2, rd:2, op:OP};

Are replaced with:
      insts[0] = create_i_inst(._imm(99), ._rs1(2),
                               ._funct3(3), ._rd(1), ._op(LOAD));
      insts[1] = create_i_inst(._imm('hFF), ._rs1(1),
                               ._funct3(2), ._rd(6), ._op(SYSTEM));
      insts[2] = create_r_inst(._funct7(0), ._rs2(1),
                               ._rs1(1), ._funct3(2), ._rd(2), ._op(OP));

And then the following two functions are added to the end of the file (within the module, so before endmodule)
function i_inst_t create_i_inst(imm_low_t _imm, regsel_t _rs1, funct3_t _funct3, regsel_t _rd, opcode_t _op);
    i_inst_t _i_inst;

    _i_inst.imm = _imm;
    _i_inst.rs1 = _rs1;
    _i_inst.funct3 = _funct3;
    _i_inst.rd = _rd;
    _i_inst.op = _op;

    return _i_inst;
  endfunction

  function i_inst_t create_r_inst(funct7_t _funct7,
                                  regsel_t _rs2,
                                  regsel_t _rs1,
                                  funct3_t _funct3,
                                  regsel_t _rd,
                                  opcode_t _op);
    r_inst_t _r_inst;

    _r_inst.funct7 = _funct7;
    _r_inst.rs2 = _rs2;
    _r_inst.rs1 = _rs1;
    _r_inst.funct3 = _funct3;
    _r_inst.rd = _rd;
    _r_inst.op = _op;
    return _r_inst;
  endfunction

There will be 6 compile errors when running the riscv_vip in VCS even in the latest release version.
Error-[NYI] Not Yet Implemented
instruction.svh, 254
  Feature is not yet supported: expression other than coverpoint label as
  coverpoint bin value.

   i32s_insts_cp : coverpoint inst {       bins s_insts[] = S_INSTS;    }
I put all the stuffs at (https://www.edaplayground.com/x/5reM).


Nix: inability to resolve some ~wrapped / impure setuid executables such as sudo

Nix has to special-case some setuid executables, and this disrupts resholve's ability to resolve them to absolute paths. (There are a number of interlocking issues here, and I suspect this will take some time--and some willingness to be squeaky wheels--to get this fixed in Nixpkgs. I vaguely plan to document these issues--but for now I'm just outlining.)
I don't have a lot of the Nix(OS)/nixpkgs system-level perspective to have the best handle on all of this. I get the impression there isn't a canonical list, but guessing from the run wrappers on my own NixOS system, this seems like a fair list:

chsh dbus-daemon-launch-helper fusermount3 fusermount kcheckpass kwin_wayland mount newgidmap newgrp newuidmap passwd ping pkexec polkit-agent-helper-1 sg start_kdeinit sudoedit sudo su umount unix_chkpwd

In the near future, I'll update resholve to raise the following error for a cross-platform subset (ping chsh newgrp passwd su sudo mount umount) of these whenever NIX_BUILD_TOP is in the environment:
There is not yet a good way to resolve 'sudo' in Nix builds. Your feedback may help me (and the Nix community) understand what the best course of action is here.

See https://github.com/abathur/resholve/issues/29 for info, feedback, and potential workarounds.

In the short term, your best bets for working around this are:

add a fake directive via the CLI or the Nix API. here's an example of what this would look like for sudo:

CLI: --fake 'external:sudo'
Nix:fake = {
  external = [ "sudo" ];
};

Use resholve's prologue option to inject (at the head of the script) some refinement based on your context:

A run-time check that will abort execution if the lookup fails.
Add/change the PATH to ensure the lookup will succeed.
Define a function or alias that executes any specific absolute path you need.

In some more limited cases, you may know that you have access to an executable that doesn't actually need a setuid wrapper and you really just need resholve to get out of your way. If you're really sure, you can tell it to back off by adding fix directive via the CLI or the Nix API. Here's an example of what this would look like for sudo:

CLI: --fix 'sudo'

Nix:
fix = {
    sudo = true;
};

There's now a real nixpkgs case here:

https://gist.github.com/r-rmcgibbo/cd7ae74a0029886c15c8071e0e7c9a76
https://github.com/archlinux/arch-install-scripts
https://github.com/NixOS/nixpkgs/pull/138080/files#diff-724a80eaed950d49b1a3c49a83efda4498d574f4be58a130f66c675e6a4fafd2R56-R60

Additional context-dumping for later:

https://github.com/NixOS/nixpkgs/pull/124536#issuecomment-849097368
https://gist.github.com/Infinisil/3366e7dfc9a01f6eeb25b5cb475cc585
https://search.nixos.org/options?channel=21.05&show=security.wrappers&from=0&size=50&sort=relevance&type=packages&query=wrappers

I have a use case this is biting me in. The scripts are running as root already under systemd.
My approach has been to use substitute in place:
  (substituteAll {
    ...
    mount = "${util-linux}/bin/mount";
    umount = "${util-linux}/bin/umount";
  })

for mountloc in $(@mount@ \
   ...
   @umount@ "$mountloc" || true
done

and then passing resholve:
keep = [
    "${util-linux}/bin/mount"
    "${util-linux}/bin/umount"
  ];

but I would prefer to tell resholve that I do want it to resolve these paths even if it is troublesome.

msmtpq is broken because of ping (see NixOS/nixpkgs#195532). The approach here doesn't appear to work.
I'd be grateful for some help — perhaps the resolution could then be added to the documentation?

Just in case you're watching here but not there, I reached out to what I assume is your handle on Matrix earlier this evening.

I have a use case this is biting me in. The scripts are running as root already under systemd.
My approach has been to use substitute in place:
  (substituteAll {
    ...
    mount = "${util-linux}/bin/mount";
    umount = "${util-linux}/bin/umount";
  })

for mountloc in $(@mount@ \
   ...
   @umount@ "$mountloc" || true
done

and then passing resholve:
keep = [
    "${util-linux}/bin/mount"
    "${util-linux}/bin/umount"
  ];

but I would prefer to tell resholve that I do want it to resolve these paths even if it is troublesome.
I guess the main reason I prefer it is because I accidentally made a mistake the first time I made this adjustment: I passed an absolute path to umount as unmount and my script broke.

There's a similar case in arch-install-scripts. When using NixOS's setuid wrappers, its unshare mode doesn't work (https://github.com/archlinux/arch-install-scripts/issues/38). I think simply resolving mount/umount would fix it.

@SamLukeYes the last workaround in the first post may help here. Since 0.8.3 you can use fix.<command> = true to bypass the error here and force it to resolve from inputs.
(This assumes that you've verified that the unwrapped commands will run correctly for the invocations in your script.)

It looks like systemd is adding a new sudo replacement named run0 that doesn’t use setuid. I bet that you could workaround this problem by using run0 instead of sudo.


CPI TokenAccount Init errors

I'm trying to initialize a new TokenAccount in an instruction's Accounts struct. If I comment out the token::mint and token::authority the following error shows up.

If I uncomment the token information then there are all these obscure compiler errors.

Cargo dependencies:
anchor-lang = "0.13.0"
anchor-spl = "0.13.0"
solana-program = "=1.7.8"
spl-token = { version = "3.2.0", features = ["no-entrypoint"] }

Can you share the entire Accounts struct?
I believe the mint need to be an account provided as a CpiContext<'info, Mint>
The mint could be checked with has_one = option_mint on option_market

It turns out the macro I was using wasn't in anchor 13 and was released in anchor 14. I'm going to close this issue


Example Notebook/script to access forced sources parameters from the database.

@jchiang87 I am looking for an example of how we can find parameters relating to all point like forced sources in the database. I would love to start with SN only, but would be OK if other classes of objects were there as well. I am trying to figure out how to generally associate objects quickly between the reference and outputs of DM.

@rbiswas4 Which forced source parameters were you interested in searching on?  For the comparisons I made in #29, I'm just doing a positional query on RA, Dec, e.g., here.

Yes, that is the kind of thing I would like, let me look more carefully after the meeting:

I wanted to get RA, Dec, sequence of MJD where they were measured to have 'significant' non zero fluxes, or SNR values.
I want to iterate over all such objects with various thresholds. So what I am really looking for is the sequence of ID for  all forced sources.
How do you figure out if they are like SN and not AGN for example?

How do you figure out if they are like SN and not AGN for example?

This might be a useful thing to discuss during the meeting.  If you histogram the peak normalized light curves from the Level 2 data, the distributions clearly show three families: constant sources, AGNs and SNe.  The AGNs tend to have a broad distribution of fluxes while the SNe have mostly quiescent values and a tail of higher fluxes.  (There is actually a fourth class: constant sources near the edges of the sensors from some visits.)

Ok. When you say broad distribution of fluxes do you really mean that AGN are long lived and SN are short lived so most of the SN fluxes are consistent with 0 ? That is true and an easy way to distinguish over long time periods.


transformation context

Transducers are all about compose and reuse :) The fact that we have to define context on the transform makes them less reusable. Could we find a way to execute a composition of transforms with a given context ? Like a ctx argument on transduce method ?

The ctx variable in each transform is mostly there when you use the transforms directly like map([1, 2, 3], function() {}, context). You are right that transduce should take a context.


Updated pypi version

Are there any plans to publish a new pypi version?
Latest is 0.7.0
https://pypi.org/project/flask-talisman/#history

A few of the fixes since the last release would be nice to have. @theacodes would you be willing to push a new release to PyPI?

A few of the fixes since the last release would be nice to have. @theacodes would you be willing to push a new release to PyPI?

As per #78 this has now been forked, released as v8.0.0 and uploaded to PyPi.


Advanced concepts article

:warning: Validation status: warnings

File
Status
Preview URL
Details

LauragraDocset0317235153/excel/excel-add-ins-advanced-concepts.md
:warning:Warning

Details

LauragraDocset0317235153/excel/excel-add-ins-core-concepts.md
:warning:Warning

Details

LauragraDocset0317235153/excel/excel-add-ins-advanced-concepts.md

Line 9: [Warning] Invalid file link:(~/../reference/excel/excel-add-ins-reference-overview.md?product=excel).
Line 11: [Warning] Invalid file link:(~/../reference/add-ins/javascript-api-for-office.md?product=excel).
Line 15: [Warning] Invalid file link:(~/../reference/shared/context.md).
Line 17: [Warning] Invalid file link:(~/../reference/shared/document.md).
Line 21: [Warning] Invalid file link:(~/reference/requirement-sets/excel-api-requirement-sets.md).
Line 38: [Warning] Invalid file link:(~/reference/manifest/requirements.md).
Line 54: [Warning] Invalid file link:(~/reference/requirement-sets/office-add-in-requirement-sets.md).
Line 134: [Warning] Invalid file link:(~/reference/excel/worksheet.md).
Line 224: [Warning] Invalid file link:(~/../reference/excel/excel-add-ins-reference-overview.md?product=excel).

LauragraDocset0317235153/excel/excel-add-ins-core-concepts.md

Line 3: [Warning] Invalid file link:(~/../reference/excel/excel-add-ins-reference-overview.md?product=excel).
Line 129: [Warning] Invalid file link:(~/../reference/excel/excel-add-ins-reference-overview.md?product=excel).
Line 130: [Warning] Invalid file link:(~/../reference/excel/loadoption.md?product=excel).
Line 273: [Warning] Invalid file link:(~/../reference/excel/excel-add-ins-reference-overview.md?product=excel).

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


fetch data for _InnerSourceMetadata

Implements most of the keys for _InnerSourceMetadata based on recommendations here.
Closes #7

tested and it works great! Thanks @jasonmacgowan!


Bring your own key and LLM

:tada: This issue has been resolved in version 0.63.0 :tada:
The release is available on:

v0.63.0
GitHub release

Your semantic-release bot :package::rocket:


Update pubspec.yaml

A few packages require intl to be up-to-date.

@appsup-dart any idea when this could be merged?

@appsup-dart or @rbellens  any idea when this could be merged?


Image upload widget rename css class

When migrating to image_upload_widget.hbs  some names of CSS class
are unchanged. This PR is to rename some of the CSS class to be more generic.
Testing Plan:
Tested manually in the browser

@timabbott ready for review

I've searched for the renamed css elements and didn't found any remaining renames to be done. I've also tested this locally and it's working fine.

@clarammdantas can you test this again?

Sure! Everything is good now, thanks @Jagansivam28!

@timabbott ready for review

@Jagansivam28 every commit should pass prettier, you can't do it as a fixup at the end.

Merged as dac8f4f20f27d1286875571f27c248dd9d62dcff after running lint --fix and amending each commit, thanks @Jagansivam28!

ok sure tim next time i'll correct it


Docs: Move async thunk middleware example to own section

I think if this is going to warrant its own section in the docs, it should be more than a copypasta. There should be some supporting documentation and a clear breakdown of which files are to be created in this example.

I agree. Right now it doesn't fit with the docs flow (every section leading onto “Example” with code from previous chapters).
A better idea perhaps would be to create “Asynchronous Control Flow” in Recipes and rework this example into a tiny tutorial.

Yeah.
This PR, if anything, is more about showing how perhaps this doesn't belong
in the API section. Not a great PR, but illustrates what I think.
On Thu, Sep 17, 2015 at 1:22 PM, Dan Abramov<EMAIL_ADDRESS>wrote:

I agree. Right now it doesn't fit with the docs flow (every section
leading onto “Example” with code from previous chapters).
A better idea perhaps would be to create “Asynchronous Control Flow” in
Recipes and rework this example into a tiny tutorial.
—
Reply to this email directly or view it on GitHub
https://github.com/rackt/redux/pull/747#issuecomment-141158039.

@dminkovsky Did you want to take on writing that up?

@timdorr I would have as part of the PR but I've never used that module.
I would be down to edit other sections of the docs, where I am familiar with the material. Let me know I can provide a list of suggestions, or PRs.

I'm going to close this out, but feel free to continue to make suggestions. I'll try to get to a full recipe in another PR soon, so we can all discuss the changes.


Update documentation for redirect URI and native app registration

According to this https://docs.microsoft.com/en-gb/azure/active-directory-b2c/tutorial-desktop-app?tabs=app-reg-ga, when we define a redirect uri for a native app, for production purposes we should define a unique scheme. However, for system browsers we need to always (both dev and prod) use http://localhost.
From the documentation one understands that localhost should only be used for development purposes. It is not possible to define and use a custom scheme eg. in a .net application which uses a b2c native app and system browser - only localhost is possible to use.
The documentation needs to be adjusted to reflect the above.
In the app registration, we should add the http://localhost redirect uri and from the app we should either use a specific port or .WithRedirectUri("http://localhost") or .WithDefaultRedirectUri()
`

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

ID: 90602ccd-184e-bb74-504c-5b06eb8eb4ab
Version Independent ID: 3d4865d9-f91f-9aeb-16a8-9789e647c068
Content: Tutorial: Authenticate users in a native client application - Azure AD B2C
Content Source: articles/active-directory-b2c/tutorial-desktop-app.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@nataliahering Thank you for your feedback . We will investigate and update the thread.

@nataliahering You have already pushed a change to the sample. We will review the changes in the linked issue and update the details on this document .

@shashishailaj I don't think I have pushed any changes. If you think it will accelerate the process, happy to create a PR though

@nataliahering I've assigned it to the author to review and update.

Dear @nataliahering, thanks for the feedback. I'm updating the article with the relevant information. A new version will publish later today, and this GitHub issue will be closed automatically (before the doc is published). If you have any questions or need further information, please open new GitHub issue.