An option for IntelliSense in code-fenced examples

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Problem
Provide IntelliSense for the code examples in.mdx
We're using a pattern, where we place our code examples in code-fenced blocks like this:
import { Button } from '…';

## Example

Here is a button:

```jsx
<Button label="I am a button" />
```

Then, we're using react-live to render those examples as live playgrounds.
The issue is that while mdx-analyzer handles the regular code in the MDX (awesome!), it does not treat the code inside the code-fenced blocks as actual code.
Solution
What we would like to see — an option (as I don't this is something that should be available by default, as not everyone is using this pattern of doing things) to treat certain code-fenced blocks as actual code.
Alternatives
Not that I know of. With the requirement that we need to have the examples' source code available, we cannot just use a component wrapper around the code we want — we need the code-fenced blocks specifically to have the access to the actual code, with all the comments etc.
In addition to this, having the example code in code-fenced blocks makes authoring and reviewing markdown changes much better — in various editors and on GitHub we can see the code examples with proper markup, rather than as jsx content.

Hmm, this feels a bit weird. The point of using code is to not evaluate things
How are you evaluating the code inside those code blocks?

in various editors and on GitHub we can see the code examples with proper markup, rather than as jsx content.

Use this project or for other editors use https://github.com/wooorm/markdown-tm-language. Soon also in GH. So I don’t think that’s a solid reason.

How are you evaluating the code inside those code blocks?

We're gathering the imports (and exported consts) from the .mdx, then provide them as the evaluation context to react-live (alongside some additional internal hooks that can be used in the example but which we don't want to import in every .mdx).
The main idea for those code examples — they're their source code first (not evaluated), then as a progressive enhancement they're evaluated and shown alongside an editable source, where we still want to keep all the comments, original formatting etc.
So, if we'd have to do something like
<Example>
    <Button label="I am a button" />
</Example>

instead, we would lose the ability to get the source code without some very serious hacks (like, is it even possible from the mdxast point of view get the source of anything that is nested inside the Example?)

Thanks, I'll investigate this, however I would say that ability to have IntelliSense inside regular code examples would still be helpful — like, we'd want the users of the docs to be sure that the docs are up-to-date, so if we could improve the examples in any way — it would be awesome.

we'd want the users of the docs to be sure that the docs are up-to-date

How do you mean? How doesn’t my above solution not do that?
What is an example of out-of-date code that isn’t up-to-date?
Or are you saying that all ```js/jsx/ts/tsx/md/mdx code blocks should be (optionally) handled by this project?

I think #287 comes very close to what you want in a generic matter.
Your code requires some domain specific logic though. What you might want is to use a custom remark plugin for this. This is a more complex feature and is tracked in #297.

I’ve closed this in favor of #287 and #297.


can you give me an exmple of how to create a frame around the QR code like this?

@codemasher you've been awesome with the examples you've provided with this repository.
I'm trying to create something like this with such kind of frames as shown in the pictures.
https://i.imgur.com/B16LP5r.png
Any guidance or examples you cn provide on how to achieve the same kind of effect with this library?

If you want to do this in SVG, you can start from the examples given in #150, for raster images like png or jpg, you will need to create the frame by yourself in GD or ImageMagick and embed the QR Code in it (i can assist you with the latter in that case).
If you need examples for any of, you can check out the following links:

http://apex.infogridpacific.com/SVG/svg-tutorial-contents.html
https://developer.mozilla.org/en-US/docs/Web/SVG
https://phpimagick.com/Tutorial
https://www.php.net/manual/en/book.image


VZ-685  Add OCI DNS/DYN DNS Support

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

acc test exempt


Enable gradient method for Gridded(Constant()) and Gridded(Linear())

This was mostly an exercise to confirm that I understand how all the meta-programming fits together.
One thing I don't like is that when I tested the gradient on a non 1-spaced grid I had to raise the multiplicative factor on the "equality tolerance" from 0.1 to 0.5. Does this bother anyone else or suggest I have made a mistake somewhere?

Oh oops, looks like I left a docstring change in this branch. If you are ok with that then I'll leave it, otherwise I can cherry-pick the gradient stuff and push to this pr branch

Nah, I'm not going to be that nitpicky.

@spencerlyon2, sorry not to notice this earlier.

One thing I don't like is that when I tested the gradient on a non 1-spaced grid I had to raise the multiplicative factor on the "equality tolerance" from 0.1 to 0.5. Does this bother anyone else or suggest I have made a mistake somewhere?

For writing tests, my personal preferred style is to pick test functions for which interpolation is exact, so that there is no question about the source of any discrepancies. An example is shown here, where for testing quadratic interpolation I chose an "underlying" function that was, in fact, quadratic.
For example, for Gridded{Linear} you could do something like
x = sort(rand(5))
y = 0.2*x

and then test that the gradient is approximately 0.2 (with high precision).


Added stripe payment into wix but while testing it i am getting Charge ID - undefined Error

I am not coder i am new to coding, i wanted to add stripe payment integration into my wix website, and i integrated stripe payment into wix but while testing it i am getting Charge ID - undefined Error
below i have given my Home Page Code, Public/Frontend Code and Backend Code so please have a look and advise any help would be appreciated Thanks in advance
Home Page code -
import {createToken, encodeCard} from "public/stripeAPI.js";
import {charge} from 'backend/stripeProxy';
var payment;
export function payNow(event) {
createToken(encodeCard(createCard()))
.then((token) => {
console.log("Card token: " + token);
charge(token, payment)
.then((chargeResponse) => {
console.log("Charge ID: " + chargeResponse.id); -THIS IS CHARGE ID ERROR I AM GETTING
});
});
}
function createCard() {

return {
"name": $w("#cname").value,
"nuumber": $w("#cnumber").value,
"exp_month": $w("#month").value,
"exp_year": $w("#year").value,
"cvc": $w("#cvc").value,
};

}

function chnageState() {
payment = {
"amount": ($w("#amt").value*100),
"currency":"INR",}
}
export function chargeCard_click(event) {
payNow(); }

FrontEnd/Public Code -
import {fetch} from 'wix-fetch';
export async function createToken(card) {
//Go to stripe.com to create a test key and replace the one in the example
const apiKey = "YOUR_API_HERE";
const response = await fetch("https://api.stripe.com/v1/tokens", {
method: 'post',
headers: {
"Content-Type": "application/x-www-form-urlencoded",
"Authorization": "Bearer " + apiKey
},
body: card
});
if (response.status >= 200 && response.status < 300) {
const json = await response.json()
return json.id;
}
const responseText = await response.text();
console.log(responseText);
return response.status;
}
export function encodeCard(card){
let encoded = "";
for (let [k, v] of Object.entries(card)) {
encoded = encoded.concat("card[", k, "]=", encodeURI(v), "&");
}
return encoded.substr(0, encoded.length - 1);
}

BackEnd Code -
import {fetch} from 'wix-fetch';
export async function charge(token,payment) {
const cart = payment;
//Go to stripe.com to create a test key and replace th eone in the example
const apiKey = "YOUR_API_HERE";
const response = await fetch("https://api.stripe.com/v1/charges", {
method: 'post',
headers: {
"Content-Type": "application/x-www-form-urlencoded",
"Authorization": "Bearer " + apiKey
},
body: encodeBody(token, cart)
});
if (response.status >= 200 && response.status < 300) {
return await response.json();
}
return await response.text();
}
function encodeBody(token, cart){
let encoded = "";
for (let [k, v] of Object.entries(cart)) {
encoded = encoded.concat(k,"=", encodeURI(v), "&");
}
encoded = encoded.concat("source=", encodeURI(token));
return encoded;
}

Hi @gatakraj, I don't see Cloud Functions code here, so I'm not sure we'll be much help. I'd suggest asking on Stack Overflow instead. One thing I did notice is that you may not have included a Stripe API key in your code (it just says "YOUR_API_HERE", although you may have taken it out to post in this issue). But that could be a place to start!
If you want to see our sample for communicating with Stripe from Cloud Functions, it lives here


[Question] how can I get the error.message in errorComponent

Vue version
all
Link to minimal reproduction
https://codesandbox.io/p/sandbox/demo-load-remote-vue3-comp-lazy-forked-93nsyn
Steps to reproduce
visit the demo
What is expected?
I can get err ref or err message string in the ErrComponent
What is actually happening?
currently I can not get err ref or err message string in the ErrComponent by reading defineAsyncComponent doc
System Info
Browsers

Any additional comments?
No response

errorComponent receives an error prop. You should modify the code accordingly:
export default {
  name: "ErrorComponent",
    props: {
      error: Error,
    },
};

playground

it works, thx a lot


should calc-size and interpolate-size features be combined?

#2110 added a feature for interpolate-size and #2148 added a feature for calc-size.  However, these are very closely related things, defined in a single spec section, that will hopefully be shipped at the same time in all engines that support them (and was certainly the case when they both shipped in Chromium).
I'm not quite sure what the standard is for when things should be part of a single WebDX feature... but I would have thought these should be a single feature rather than two separate ones.
cc @vwallen @ddbeck

Thank you, David, for filling this.
For reference, here are the two features in the explorer:

https://web-platform-dx.github.io/web-features-explorer/features/interpolate-size/
https://web-platform-dx.github.io/web-features-explorer/features/calc-size/

To me, the question is less about whether the features are part of the same spec, or whether they ship at the same time. We need to think about it from a developer stand point:
Would a web developer always use those two features together to achieve a certain scenario? Or, are there cases when they might use them separately, for different things?

Would a web developer always use those two features together to achieve a certain scenario? Or, are there cases when they might use them separately, for different things?

I'm not sure that question makes sense, because I think the best way to think about them is as two interfaces to the same underlying feature:  calc-size() is the lower-level primitive and interpolate-size is the much easier-to-use higher-level interface that covers the main use case in a single line (and uses calc-size() underneath).

In which case grouping them makes sense to me.

I'm not opposed to this change, but I think it's likely to be confusing to developers. If we make this change, then we should take some steps to alert the responsible parties that this relationship is poorly communicated. Otherwise, I think we're likely to see a "where's the calc-size() feature?" issue.
What I was thinking
As a reviewer on both PRs, the close relationship between these features was not at all obvious. I considered other groupings and the one proposed here was not one of them. I turned to the spec and docs:

The calc-size() section in the spec opens with a comparison with calc(), implying that is the thing it's most-closely related to. Interpolation isn't highlighted until the third subsection, which doesn't exactly scream "this is why this exists."
The MDN page for calc-size() opens with a comparison to calc(). That page takes four examples before it reaches one involving animations.

Meanwhile the interpolate-size spec section requires the reader to be pretty eagle-eyed to notice the mention of calc-size(). Likewise on MDN, where the only calc-size() mentions are in a note at the end of the page and a See also bullet.
Finally, the existing features have distinct support data. It doesn't look like they shipped together in Edge.
What we should do next
If we're going to merge these features, then I'd expect the following:

Someone to check whether Edge is or isn't shipping interpolate-size. Right now browser compat data says that it's not. This is probably wrong and it'd be great for someone to open a PR to fix it.
(If it's correct—that Edge opted-out of interpolate-size but not calc-size(), then that seems like a big enough deal to derail this proposal.)

An issue opened against the spec, asking to make this relationship explicit:

calc-size() is the lower-level primitive and interpolate-size is the much easier-to-use higher-level interface

This would be especially helpful for our spec links. Being able to link to one spec section that mentions both of these things in the same breath would great.

An issue opened against MDN, also asking to make the relationship explicit on both pages.

The description of calc() to be expanded with "not to be confused with" text.

Someone to check whether Edge is or isn't shipping interpolate-size. Right now browser compat data says that it's not. This is probably wrong and it'd be great for someone to open a PR to fix it.
(If it's correct—that Edge opted-out of interpolate-size but not calc-size()—then that seems like a big enough deal to derail this proposal.)

This is a mistake. interpolate-size seems to work for me in Edge.
I think this is a probably due to the bcd collector somehow. Chris Mills had properly "mirrored" Edge's status in https://github.com/mdn/browser-compat-data/pull/23863 but the PR got closed in favor of this bcd collector one which didn't mirror Edge to Chrome: https://github.com/mdn/browser-compat-data/pull/24210

They're currently in the same spec section already (also see the table of contents for the structure).  And the explainer for the feature is perhaps clearer about the relationship than the spec is.
I've been doing a significant part of the work driving these features to being specified and to ship, and basically all of the communication around them (since interpolate-size was added) has talked about them together, from the single spec section and explainer (both linked above), the chromestatus and intents, to the web.dev blog post.  Part of the reason for this is that we don't want developers to reach for calc-size() when interpolate-size would address their needs.
So I filed this mainly because I was a bit surprised to see them separate, and also because it's going to be at least a little bit of extra work to correctly tag the WPT tests to the two separate features rather than a single one.
That said, I'm ok with leaving them separate if you think that's better.

That said, an argument for keeping them separate is that it probably is interesting to track their usage levels separately.

I'm going to just go ahead and add the test annotations and use counters for separate features, so I guess I'll close this.


Invalid Config

My integration says invalid configuration an it does not loss
Log Details (ERROR)
Logger: homeassistant.setup
Source: setup.py:215
First occurred: 09:00:24 (1 occurrences)
Last logged: 09:00:24
Setup failed for custom integration swemail: Unable to import component: Exception importing custom_components.swemail
This error originated from a custom integration.
Logger: homeassistant.loader
Source: custom_components/swemail/init.py:20
Integration: Svensk Postutdelning (documentation, issues)
First occurred: 09:00:24 (1 occurrences)
Last logged: 09:00:24
Unexpected exception importing component custom_components.swemail
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 813, in get_component
ComponentProtocol, importlib.import_module(self.pkg_path)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1206, in _gcd_import
File "", line 1178, in _find_and_load
File "", line 1149, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/config/custom_components/swemail/init.py", line 20, in
@asyncio.coroutine
^^^^^^^^^^^^^^^^^
AttributeError: module 'asyncio' has no attribute 'coroutine'

@asyncio.coroutine ^^^^^^^^^^^^^^^^^ AttributeError: module 'asyncio' has no attribute 'coroutine'
Looks like the problem that was solved in 1.0.7. Try update!

It’s working now

Duplicate of #23


[BEAM-10921] Add the UserPipelineTracker to track user pipelines for derived pipelines

This code is a data structure to track which derived pipelines came from which user pipelines. Previous code to calculate this relationship erroneously used pipeline-specific PCollection ids. This makes the relationship explicit.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @KevinGG

R: @davidyan74

Thanks, Sam! LGTM. Does this mean we need to call the add_derived_pipeline every time we do a from_runner_api(to_runner_api()) and similar conversions? The DerivationTree is like a 2-way multimap.

Yes, I have a follow-up PR adding this.

R: @pabloem

Run Python PreCommit

Run Portable_Python PreCommit

thanks Sam!

(in case it wasn't clear: LGTM)


New Extension: Library Genesis

Description
An extension that allows searching books on Library Genesis and quickly opening the link in the browser, using the libgen.js API.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@yz3440 all looks good, the only thing missing is removing the screencasts folder, since it is redundant considering the metadata one. Besides that you can change the date in the changelog to today's date and then I can go ahead and merge it 🙂

@danielsequeira thanks! I had kept the screencasts folder because the image on top of this PR references the URL. I did it anyway. Now that I've removed it, the image src shows 404.


Update site not found

I'm getting the following issue with the update site when trying to install cppcheclipse on Luna.
"No repository found at http://cppcheclipse.eclipselabs.org.codespot.com/svn/update/."
I don't know of anywhere else that I can report this but, if it all looks ok, please respond.

Unfortunately Google seems to have closed down that page. I will set up a new site which is tracked in #72 .

Thank you for your reply. I hadn't realised the loss of the existing site and the "Provide p2 update site" issue were related.


Code indentation

For Bug Reports

BookStack Version: v0.13.1
PHP Version: 5.6.28
MySQL Version: Unknown

There is a indentation problem in the first line of all code blocks:

You can see the first line #define is indented slightly.

Hi @unknowndomain, Thanks for highlighting this issue. This issue looks to be the same as #264. Please could you update BookStack to the latest version and check if the issue persists?

This appears to be resolved.

@unknowndomain Glad to hear it. I will close this issue.


Add commeent

Why
This code looks logic-bug ...
https://github.com/rust-analyzer/rust-analyzer/blob/ce86534e1cb22685e83c3f91ea89025edbfcbc98/crates/syntax/src/syntax_node.rs#L50
However, this code has been intentionally disabled.
It's a good idea to write a comment
ref: https://github.com/rust-analyzer/rust-analyzer/pull/10357
What

I added comment

Ouch, bors closed this because it was included, then reverted, in #10504.

@k-nasa I can't reopen this PR, feel free to file a new one (but see my suggestion above).


[BUG]  XrmApp.Lookup.SwitchView(viewName) selenium exception.

Bug Report
Issues should only be created for items related to covered functionality.
Not covered functionality, feature requests, and questions should use the Feature Request or Question templates.
EasyRepro Version

[ ] Microsoft Dynamics 365 Online Version 8.2 (8.2.x) (DB 8.2.x) online
[ ] Microsoft Dynamics 365 Online Version 9.0 (9.0.x) (DB 9.0.x) online
[ ] Microsoft Dynamics 365 Online Version 9.0.2 (9.0.2) (DB 9.0.2) online
[x] Microsoft Dynamics 365 Online Version 9.1 (9.1.x) (DB 9.1.x) online

UCI or Classic Web

[x] UCI (XrmApp)
[ ] Classic Web (XrmBrowser)

Online or On Premise

[x] Online
[ ] On Premise

Browser

[x] Chrome
[ ] Firefox
[ ] IE
[ ] Edge

Describe the bug
When switching view on a lookup field, Selenium not found exception thrown. I believe it is a known issue as the test has been marked as [TestCategory("Fail - Bug")] in the sample tests.
Special formatting / display

Code to reproduce

Expected behavior

Screenshots

Additional context

** Code to fix it **
          //Driver.WaitUntilAvailable(By.XPath(AppElements.Xpath[AppReference.Lookup.ChangeViewButton])).Click(true);
          //Driver.WaitUntilAvailable(By.XPath("//li[contains(@role,'treeitem') and contains(@class,'bu')]"));
          //Driver.WaitUntilAvailable(By.CssSelector("li[aria-label='" + viewName + "']")).Click();

I am having an issue with the switch view but my error is a different. It clicks on the switch view drop down but then when attempting to click on the option from the drop down list it then errors with the following:
 Message:      Test method B_Microsoft.Dynamics365.UIAutomation.Sample.Tests.IndividualTests.ContactTimeline.SearchForContactAndAddAppointmentOnTimeline.test_SearchForContactAndAddAppointmentOnTimeline threw exception:      System.ArgumentException: An item with the same key has already been added.
Regards
Shehzad


Upgrade Ruby to 2.2.2

Reopened version of #2215
A few gem upgrades:

kgio 2.8.0 wasn't compatible
raindrops 0.11.0 wasn't compatible
babosa had a duplicated key in a hash, which Ruby 2.2 has a warning for

In one of our tests, we were defining a key twice in the same hash, causing a warning. This has been fixed.

:+1:


option for certain projects or tasks to not require notes

Inspired by the break UX (see #40, etc). For breaks, you want the project to be "break", and I could imagine different tasks (talking to friends, meditation, reading articles, etc). But you don't really need notes.
I think I'd like something where if you try to submit a task without a note (see #39), rather than just rejecting it, we'll give a little popup that explains that a note is required, but gives you an option to make it not required for either this task, or for all tasks under this project.

Won't do. See comment in #39.


Upgrade to Python 3.13 and Django 5.1

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Can't install

I'm trying to install Huginn on my Raspberry Pi 2. I followed the Novice setup guide, then the Quick start.
I had to solve a few minor issues, e.g., I had to install ruby-dev and libmysqlclient-dev, to pass the first "bundle" run.
But now I can't get through the step where I run "bundle exec rake db:migrate". The error I get is
==<PHONE_NUMBER>0107 SetCharsetForMysql: migrating ===============================
rake aborted!
StandardError: An error has occurred, all later migrations canceled:

undefined local variable or method `acts_as_mappable' for #<Class:0x391c4a0>/var/lib/gems/1.9.1/gems/activerecord-4.2.2/lib/active_record/dynamic_matchers.rb:26:in `method_missing'

Any suggestion on how to solve the error?

Hi @gsportelli, if your raspberry linux distribution is based on debian 6 or 7 would you mind trying the new manual installation guide? The pull request is still in the works, but the installtion worked well for me on stock debian jessie and wheezy installations.

Closing since we haven't heard back.  Hope it's working for you!


Is password change URL correct?

Is this where password change should be redirecting to for a given user?

It seemed like it was the original intention to do this, I just hooked it up to make the links work (or maybe the custom page just wasn't implemented in the first place) I can create an integrated password change form if you want.

Yeah I'd prefer the interface to be consistent throughout the site. I think all the necessary templates are already in the account folder.
Perhaps we could sort #122 at the same time?


Transmit (GFIR) filtering seems to be Inoperative

I have set up a test rig using a Raspberry Pi 4 and the latest version of Bookworm Lite.  I have both LimeSuite and LimeSuiteNG (commit 626a781) installed and am doing comparitive tests using a LimeSDR Mini (firmware 1.30) prior to migrating my application software on to a LimeNET Micro 2.0 DE.
The application that I am using is a DVB-S2 TV transmitter set to transmit QPSK  with a symbol rate of 4MS/s on a frequency of 437 MHz.   This is the same application that I have been using in the Portsdown ATV transmitter for many years.
Using LimeSuite, the transmitted spectrum is what would be expected, a flat pedestal with 0.35 roll-off at the edges.  Using LimeSuiteNG, the spectrum is a sin x/x shape with the first nulls at +/- 4 MHz.  So there is clearly some digital signal processing that is not being applied in LimeSuiteNG.  It is possible to demodulate valid data from the sin x/x spectrum.
The test above was performed with no oversampling and no GFIR3 coefficients applied.  If I try to apply GFIR3 coefficients in LimeSuiteNG (with 2x opersampling), I just get a low level carrier output.  I suspect that this might be a related issue.
Spectrum views attached.

Thanks, Dave

Are you using LMS_SetGFIRCoeff to set the coefficients directly? What was used to generate the coefficients? How many coefficients is being set?

I am using (LMS_SetGFIRCoeff(device, LMS_CH_TX, 0, LMS_GFIR3, xcoeffs, 119) to set the coefficients (119 of them).  xcoeffs is an array of (double) coefficients that I have been using in this application with legacy LimeSuite for 5 years.
I have in the past (with commit https://github.com/myriadrf/LimeSuiteNG/commit/626a781f2f1d20e25b3e731f3fe8fb350a7adc97) checked that the coefficients are being entered using:
LMS_GetGFIRCoeff(device, LMS_CH_TX, 0, LMS_GFIR3, readout);
for (int i = 0; i < 119; i++)
{
  printf("Coeff[%d] = %.10f\n", i, readout[i]);
}

and they were printed correctly.
I am now unable to check anything due to Issue #91
Thanks
Dave

GFIR clock divider configuration was missing.
There are nuances with the GFIR coefficients configuring. Even though the absoulte maximum count of coefficients for GFIR3 is 120, the actual maximum number of coefficients used is determined by oversampling.
oversampling of 2, allows for 30 coefficients.
oversampling of 4, allows for 60 coefficients.
oversampling of 8 and higher, allows for 120 coefficients.
Another catch is that coefficients order in chip memory layout is depending on used coefficients count.
Therefore GFIR coefficients should be set after sampling rate/oversample is configured.
In legacy LimeSuite, if 2x oversample is used and 119 coefficients are being set, they all are written into chip memory, but actually only 30 coefficients would be used (not the first 30 as in the uploaded array, but 30 out fo 120 depending on chip settings).
LMS_GetGFIRCoeff returns all 120 coefficient registers, but they don't necessarily reflect which coefficients and in which order are being used. It's complicated.
Which is why I was interested what was used to generate those 119 coefficients. If they were generated with standard math tools for GFIR design, then 119 coefficients would work as expected only when 8x oversampling is used.

As alternative for uploading custom GFIR coefficients and all that complications I mentioned, there is LMS_SetGFIRLPF which generates and configures coefficients for desired bandwidth.


uv run should not require an existing virtual environment

Currently uv run fails if a virtual environment is not present
❯ uv run --with anyio -- python -c "print('hi')"
warning: `uv run` is experimental and may change without warning.
error: Failed to locate a virtualenv or Conda environment (checked: `VIRTUAL_ENV`, `CONDA_PREFIX`, and `.venv`). Run `uv venv` to create a virtualenv.

We should not require a virtual environment to exist to run commands.
You can currently bypass this with --isolated
❯ uv --isolated run --with anyio -- python -c "print('hi')"
warning: `uv run` is experimental and may change without warning.
Resolved 3 packages in 195ms
Downloaded 3 packages in 78ms
Installed 3 packages in 8ms
 + anyio==4.3.0
 + idna==3.7
 + sniffio==1.3.1
hi

Are you sure this is true on main? It works fine for me.

I'm thinking of: https://github.com/astral-sh/uv/pull/3499

Ah I was running the last version, presuming you fixed this then.


use pkg_resources to check for distributions

This changes the implementation of pip-grep to use pkg_resources.get_distribution instead of parsing requirements.txt manually. This makes the buildpack more accurate for some project configurations.
This change is backwards compatible, scripts calling this with a requirements file will continue to work, since "requirements.txt" is not a package.
fixes #359

✨🍰✨


Handle resizing and add scrolling

@livibetter I'm so, so sorry for dropping the ball on this and not merging it. Your code works beautifully, and I'm really grateful for your contribution. I know this comes a year later, but thank you!


Add AutoPrint option

This PR adds an AutoPrint config option.

Codecov Report

Merging #1065 into master will decrease coverage by <.01%.
The diff coverage is 65.52%.

@@            Coverage Diff             @@
##           master    #1065      +/-   ##
==========================================
- Coverage   63.26%   63.25%   -<.01%
==========================================
  Files         180      180
  Lines       10065    10091      +26
==========================================
+ Hits         6367     6383      +16
- Misses       2905     2915      +10
  Partials      793      793

Impacted Files
Coverage Δ

pkg/config/store_config.go
76% <ø> (ø)
:arrow_up:

pkg/ctxutil/ctxutil.go
82.64% <0%> (-3.56%)
:arrow_down:

pkg/config/legacy.go
100% <100%> (ø)
:arrow_up:

pkg/action/generate.go
62.92% <100%> (ø)
:arrow_up:

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 ebab2ab...0039bc6. Read the comment docs.

The show action already prints the password, without any special options. So I currently fail to see how that would apply there.

You're right, I was mislead by the case for multiple matches, where you can select what to do.


[BEAM-9078] Pass total_size to storage.Upload

Goal
It's possible for the tarball uploaded to GCS to be quite large. An example is a user vendoring multiple dependencies in their tarball so as to achieve a more stable deployable artifact.
Before this change the GCS upload api call executed a multipart upload, which Google documentation states should be used when the file is small enough to upload again when the connection fails. For large tarballs, we will hit 60 second socket timeouts before completing the multipart upload. By passing total_size, apitools first checks if the size exceeds the resumable upload threshold, and executes the more robust resumable upload rather than a multipart, avoiding
socket timeouts.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.
@pabloem @charlesccychen

For clarity of the underlying problem, this is the function that sets the upload strategy in apitools: https://github.com/google/apitools/blob/88a05fb5a6826eb553e49dfccaf989eb27b850fa/apitools/base/py/transfer.py#L715. You can see that since we pass a None value as total_size to the Upload instance, we default to a simple upload, not resumable.

Thank you!


Set up initial transforms; use Koza for ingests

Downloads appear to work as expected.
Setting up transforms.

OBOs are all from KG-OBO now so they require only passthrough parsing - KGX can accept the .tar.gz of each node/edgelist as input, or we can decompress, but either way they should be (mostly) ready to use.
DrugCentral and GO-CAMs are node and edge lists already (though the latter will need a specific query to retrieve)
Orphanet and OMIM are n-triples
Reactome, TCRD, and HPA need their own new transform processes.

Setting up the Orphanet transform as per KG-COVID-19 is feeling awkward, so I'll move toward using Koza for ingests of anything not immediately KGX-compatible.

Renamed branch so I will close this one and start a new PR


Set images at correct positions

closes many issues related to the wrong position of applets
closes #108

closes #48


Images are downloaded after upload

A better approach could be copy the images to the local storage and upload it on the background.

localstorage size is ~5mb according to awesome-web-storage, so adding images to localstorage is not an option.


CocosSharp Windows Store template has error when created

MonoGame.Framework\Themes\generic.xaml' does not exist

This is a problem with the NuGets that are being generated.

Closed by https://github.com/mono/CocosSharp/commit/b777916285815a36eb5e5169694d684dd64e745f


[Bug]The duplicate options is displayed in radio widget

Description
The duplicate option is displayed  in the radio widget when the user edits the value option from the property pane
Inside the Radio button property pane Edit the value in the options section the same label option is displayed twice in the UI
Steps to reproduce the behaviour:

Important Details

Version: [Cloud ]
OS: [MacOSX]
Browser Chrome
Environment [production, release,]

There is no changes in the widget for some time now, The last change 5 months ago might be causing this, We see that the evaluation of the options is also correct.
cc @riodeuno

@somangshu This is happening because we are using the options value as the key of each Radio component.
Now when we remove all the values from options, more than 1 Radio component gets the same key i.e empty string(value part of the option), and this confuses react and makes it go haywire.
Having unique keys should fix this up.

Good find @ashit-rath 👏 , Please let me know if you wont be able to find time to fix this issues.


feat(github-action-plan-storage): added

what

terraform plan storage for github actions

why

supports github-action-terraform-plan-storage

references

Closing as stale


DwC-A Import task - Autocomplete search

Implement autocomplete search to be used for both Import Dataset, but also from DwC-A Import task.

[ ] Implement backend side [@LocoDelAssembly]
[ ] Add to Data > Import Dataset (all import dataset types) [@LocoDelAssembly]
[ ] Add to task index page (@jlpereira)

@jlpereira let me know if I should scaffold the branch (@LocoDelAssembly there is a generator that does this in one command, just FYI).

@mjy for this task? its already created

The index page though? Not the core app.

I think the index page its the same task but only visible if there is not file selected @LocoDelAssembly can you clarified this?

The first two items I mean here:

For the third item for @jlpereira I'm hoping the same backend code can be used.
PS: "New" will be disabled as each import type will be created from their own interfaces (e.g. moving some of the existing batch loaders to this feature, but for now this is DwC-A exclusive).


Split the maximum fragment length into two - an input and output MFL

Since the server might want to have a different maximum fragment length for the outgoing messages than the negotiated one - introduce a new way of computing it. This commit also adds additional ssl-opt.sh tests ensuring that the maximum fragment lengths are set as expected.

I believe that all of the comments have been addressed.

CI passed on https://github.com/ARMmbed/mbedtls/pull/3163 which is a superset of these changes.
So far this CI run failed only on DTLS client reconnect from same port: reconnect.

Shall we close this in favor of https://github.com/ARMmbed/mbedtls/pull/3163 and merge from there?

Either way is fine. It's up to you how these changes will be seen - as one or two merges.

Closing in favor of https://github.com/ARMmbed/mbedtls/pull/3163


Disjunct names RCC_CFGR_SWS_HSISYS vs RCC_CFGR_SWS_HSI and  RCC_CFGR_SWS_PLLRCLK vs RCC_CFGR_SWS_PLL

the headers use RCC_CFGR_SWS_HSISYS and  RCC_CFGR_SWS_PLLRCLK  while all other families use RCC_CFGR_SWS_HSI  and
RCC_CFGR_SWS_PLL. This is unneeded complication.

Hello @UweBonnes,
The naming convention you mentioned (RCC_CFGR_SWS_HSISYS and RCC_CFGR_SWS_PLLRCLK) comes directly from the reference manual. To ensure consistency and accuracy, we align our headers with the terminology used in the reference manual.

Please allow me to close this issue. thank you for your understanding.
Best regards,


Couldn't reach https://raw.githubusercontent.com/huggingface/datasets/1.2.1/datasets/csv/csv.py

Hi,
When I load_dataset from local csv files, below error happened, looks raw.githubusercontent.com was blocked by the chinese government. But why it need to download csv.py? should it include when pip install the dataset?
Traceback (most recent call last):
  File "/home/tom/pyenv/pystory/lib/python3.6/site-packages/datasets/load.py", line 267, in prepare_module
    local_path = cached_path(file_path, download_config=download_config)
  File "/home/tom/pyenv/pystory/lib/python3.6/site-packages/datasets/utils/file_utils.py", line 343, in cached_path
    max_retries=download_config.max_retries,
  File "/home/tom/pyenv/pystory/lib/python3.6/site-packages/datasets/utils/file_utils.py", line 617, in get_from_cache
    raise ConnectionError("Couldn't reach {}".format(url))
ConnectionError: Couldn't reach https://raw.githubusercontent.com/huggingface/datasets/1.2.1/datasets/csv/csv.py

I temporary manually download csv.py as custom dataset loading script

I temporary manually download csv.py as custom dataset loading script

Indeed in 1.2.1 the script to process csv file is downloaded. Starting from the next release though we include the csv processing directly in the library.
See PR #1726
We'll do a new release soon :)

Indeed in 1.2.1 the script to process csv file is downloaded. Starting from the next release though we include the csv processing directly in the library.
See PR #1726
We'll do a new release soon :)

Thanks.

Thanks.


Tools

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

tools/office-cmdlet-updater/config/default.json
:white_check_mark:Succeeded

tools/office-cmdlet-updater/constants/commands.js
:white_check_mark:Succeeded

tools/office-cmdlet-updater/db/index.js
:white_check_mark:Succeeded

tools/office-cmdlet-updater/services/log.parse.service.js
:white_check_mark:Succeeded

tools/office-cmdlet-updater/services/log.store.service.js
:white_check_mark:Succeeded

tools/office-cmdlet-updater/services/markdown.service.js
:white_check_mark:Succeeded

tools/office-cmdlet-updater/services/power.shell.service.js
:white_check_mark:Succeeded

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


Switching MudTab from 1 tab to multiple

Bug type
Component
Component name
MudTabs
What happened?
Hi.
I'm trying to dynamically create tabs for in my control and also set the ActivePageIndex, according to what has been specified and passed to control as parameters. Now I'm facing one strange issue, When the newly rendered Tab control has more tabs then the previous one and new ActiveTabIndex is pointing to the newly added tab, then the tab context is rendered empty and visual active tab indicator remains unchanged.
(examples: switch from "2nd on 2 tabs" and then switch to "3rd on 3 tabs" or switch from "1st on 1 tabs"  to "2nd on 3 tabs")

If I try to increase tab count and switch active tab index to the value which is less or equal previous tab count, then everything works just fine. (example "2nd on 2 tabs" and then switch to "2nd on 3 tabs")
When tab count decrease control always works as expected.
Thank you in advance.
BR, Oleg
Expected behavior
Tab Control rendered with relevant tab page content and corresponding active page highlighted.
Reproduction link
https://try.mudblazor.com/snippet/QuGHuRcvAgrqiJer
Reproduction steps

Run the snippet - 1 tab is shown with the content of the tab visible on the screen

Press "2nd of 2 tabs" button
EXPECTED : control with 2 tabs is rendered, the second tab is active and content is visible.

ACTUAL:  control with 2 tabs is rendered, but the first tab shown as active and tab content is empty

Tab panel is displayed as expected only after manually selecting corresponding tab.

Relevant log output
No response
Version (bug)
6.2.0
Version (working)
No response
What browsers are you seeing the problem on?
Chrome, Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Perhaps the Dynamic Tabs would be a better choice?

Make sure to take a good look at the example code regarding StateHasChanged().

Thanks you @Yomodo !
I was able to get things spinning after following your advice about StateHasChanged().
In my case I had to add the following code:
protected override void OnParametersSet()
{
        shouldRender = (TabIndex.HasValue && TabIndex.Value != tabIndex)
            || (TabCount.HasValue && TabCount.Value != count);
        tabIndex = TabIndex ?? 0;
        count = TabCount ?? 1;
}

protected override void OnAfterRender(bool firstRender)
{
        base.OnAfterRender(firstRender);
        if (shouldRender)
        {
            shouldRender = false;
            StateHasChanged();
        }
}

But is this some kind of the trick? As for my perspective, it is a bit weird to call re-render by StateHasChanged() just after the current render is competed?
Regards, Oleg


Update LOCALIZATION_CONTRIBUTORS

Changes

Added my name under the language "Korean" section.

Implementation details

How to read this PR

Screenshots

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

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


Feature request: please add another setting to just to sanitise dsn hidden rolls whilst keeping attack rolls visibile.

what currently happens is I either have the option to hide all rolls from players rather than just hidden/secret rolls from players and still display the dice from dice so nice.
what i would like is a setting to just sanitise dsn rolls when its hidden/secret (pf2). so that players can see my attack rolls but not my secret rolls.

I'm not quite sure I follow. If you leave the option to sanitize DSN rolls disabled, I think it will already do what you want, since DSN should already respect private rolls?

I'm not quite sure I follow. If you leave the option to sanitize DSN rolls disabled, I think it will already do what you want, since DSN should already respect private rolls?

if i roll for a secret roll i cannot roll without players seeing a dsn roll. so i have to have the setting to sanitise all rolls. i cannot just roll an attack roll with dsn rolls on.

i have these settings enabled but im forced to so that dsn doesnt roll the secret checks.

what id like is to have dsn sanatise only on secret/hidden checks. and not on public rolls.

what id like is to have dsn sanatise only on secret/hidden checks. and not on public rolls.

This is what DSN should do by default - if DSN is showing dice to other players (you'll need to log in as a player to see) for a private roll, then either some other module is causing it, or DSN has a bug. Make sure your DSN version is up-to-date, and if you can reproduce with only DSN enabled, you should probably lodge a bug with them as rolling dice for other players on private rolls is not expected behaviour.
I'm going to close this out here, but feel free to re-open if you find that there's something this module should be doing to help, though that doesn't seem likely based on the description.

what id like is to have dsn sanatise only on secret/hidden checks. and not on public rolls.

This is what DSN should do by default - if DSN is showing dice to other players (you'll need to log in as a player to see) for a private roll, then either some other module is causing it, or DSN has a bug. Make sure your DSN version is up-to-date, and if you can reproduce with only DSN enabled, you should probably lodge a bug with them as rolling dice for other players on private rolls is not expected behaviour.
I'm going to close this out here, but feel free to re-open if you find that there's something this module should be doing to help, though that doesn't seem likely based on the description.

i can invite you to the game i think its the way secret rolls are treated in pf2.

I'm not sure what "secret rolls" are here, if that's a technical term used by pf2e, or just how you usually refer to private rolls - in Foundry, there's Public Roll, Private GM Roll, Blind GM Roll, and Self Roll - all chat message types other than Public Roll cause their content to be hidden from other players, and should cause DSN not to show dice. If pf2e doesn't use the built-in Foundry functionality to perform private rolls, then neither this module nor DSN will/can do the right thing.
It would surprise me if that is actually the case, but if it was I'd be asking them to reconsider and do things the normal way because I personally don't have much interest in supporting hacks for all the possible systems out there, especially for standard functionality that's included in core.

I'm not sure what "secret rolls" are here, if that's a technical term used by pf2e, or just how you usually refer to private rolls - in Foundry, there's Public Roll, Private GM Roll, Blind GM Roll, and Self Roll - all chat message types other than Public Roll cause their content to be hidden from other players, and should cause DSN not to show dice. If pf2e doesn't use the built-in Foundry functionality to perform private rolls, then neither this module nor DSN will/can do the right thing.
It would surprise me if that is actually the case, but if it was I'd be asking pf2e to reconsider and do things the normal way because I personally don't have much interest in supporting hacks for all the possible systems out there, especially for standard functionality that's included in core.

secret rolls are just blind gm rolls. and dsn activates on those. as a standard thing with the mod. but i have to sanatise that to stop the blind rolls. as some rolls when exploring require secret perception checks but use the blind gm rolls.  so id like it if it were possible to hide blind gm rolls with the rolls from dsn. but still do public.

As I said earlier - if you're certain that DSN is activating for your players when you roll a blind GM roll, and your DSN is up-to-date, and it happens with no other modules enabled, that sounds like a DSN bug and you should report it, though I cannot reproduce what you're describing under those conditions.

As I said earlier - if you're certain that DSN is activating for your players when you roll a blind GM roll, and your DSN is up-to-date, and it happens with no other modules enabled, that sounds like a DSN bug and you should report it, though I cannot reproduce what you're describing under those conditions.

"
ive done it and disabled all modules but i run on the forge and players can still see it as ive tested.

i can invite you to my game to test if you like

GuitarGuyNick#7895

As I said earlier - if you're certain that DSN is activating for your players when you roll a blind GM roll, and your DSN is up-to-date, and it happens with no other modules enabled, that sounds like a DSN bug and you should report it, though I cannot reproduce what you're describing under those conditions.

i found the issue you were right. currently there is no way to seperate the permissions between gm and player for blind gm rolls to show as "ghosted" rolls.  so i wrote an issue to ask to split them.


JSX support?

Any chance for JSX support?
I tried adding a JSX-compatible wrapper-function myself, but so far just found new and interesting ways to make the H function explode ;-)
This got me some of the way, but doesn't work for text nodes yet...
/** @jsx make*/

const make = (type, props, ...children) => H(type, props || {}, ...children);

Any particular reason the H function signature doesn't resemble a "normal" virtual DOM factory function?

Hello there @mindplay-dk !

Any particular reason the H function signature doesn't resemble a "normal" virtual DOM factory function?

Do you have some reference in mind? I modelled it (as close as it can get in 512 bytes) having react's createElement in mind : https://reactjs.org/docs/react-api.html#createelement
Let me give it a try with JSX myself and I might have some more feedback for you ;)

Oh I think I found the issue. JSX uses null if there are no properties defined. This library cannot handle this. It can do undefined, {} or even [] just fine, but null will cause issues.

JSX uses null if there are no properties defined. This library cannot handle this.

This should be on the list of caveats maybe?
If supporting JSX is a non-goal, maybe this should be stated in the documentation as well? Many people will see the JSX-like H factory function and the word "VDOM", and will assume it'll work, as it does with almost any library of this type.
I expect JSX-support is just a simple one-liner though - even if the library doesn't support it, perhaps an example wrapper-function could be added to the documentation?

This should be on the list of caveats maybe?

For now, totally, but I would also like to somehow support it...
JSX was not my first priority so I never tested it to be honest... I based my initial API on React when I started, but due to the size constraints I decided to sacrifice some of the features. (Don't forget that the manifesto of this library is to never exceed the 512 bytes in size 🙂)

The devel/0.3.1 branch should now also support null 💪

Wow. This just works?! With one line of code?! 😂
https://codesandbox.io/s/jsx-with-dot-dom-jjdzy
Props, state, state-updates, updates to a parent component, everything just works.
That's CRAZY cool.
I think maybe now you can put this JSX one-liner in the docs and show off JSX support? 😀

Technically you should be able to do just
/** @jsx H */

Apparently, yep. (I don't recall why I thought a function was necessary before...)

Okay, so I recall now, #31 for one is something I wanted to address - as well as support for other literals like true and false.
So I would maybe hold off on announcing JSX support, as it's probably going to be a bit too confusing for someone to use in practice right now.


 IF YOU DO NOT FOLLOW THE FORMAT OR MAKE A GENERIC TOKEN REQUEST YOUR ISSUE WILL BE DELETED WITHOUT COMMENT
YOUR JUSTLIST MUST FOLLOW THE JSON SPECIFICATION
https://github.com/justswaporg/justlists/example.justlists.ts
Checklist

[ ] I understand that this is not the place to request a token listing.
[ ] I have tested that my JustList is compatible by pasting the URL into the add a list UI at justswap.org.
[ ] I understand that filing an issue or adding liquidity does not guarantee addition to the justlists website.

Please provide the following information for your token.
JustList URL must be HTTPS.
JustList URL:
JustList Name:
Link to the official homepage of the JustList manager:

YES

Sorry, your issue will be closed as you did not submit your information in the correct format.


isBlocking ? could not find that in source angular-block-ui.js in dist

Instead I found something like
state().blocking
to check if blocking is on or off.

isBlocking is not contained in a release yet (it was introduced in march, while the last release 0.2 was from february). If you wan't to use it you need to clone the repo.
@McNull Do you have any plans for an upcoming release?

This is included from v0.2.1 onwards. I think this issue can be closed now.


Unable to connect to Inertia SSR server and 404. How to debug?

I've managed to set it up with Svelte and everything seems to work fine when using bun run dev 👍 , but facing issues when using SSR. Building works fine.

When I run: php ./bin/hyperf.php inertia:start-ssr --runtime=bun it shows Unable to connect to Inertia SSR server but it actually works fine.

I can see app is serving SSR-ed content but client side hydration does not work because of 404s.

.env contains
INERTIA_IS_SECURE=false
INERTIA_SSR_ENABLED=true

@vmikhav On ssr mode app the app not correctly serving public directory. How can I fix?

@harryqt Provide your  .env

https://github.com/harryqt/hyperf-inertia-test/blob/main/.env

@harryqt Do you have nodejs and npm installed on your server?  If not, install them

npm run build builds the app successfully but upon browsing the js files throws 404.
Such as http://<IP_ADDRESS>:9501/public/build/assets/app-ClBELM2y.js ---> this 404
Which is expected, because it actually stored in /storage/public/build/assets/app-ClBELM2y.js

I think vite.config.js is the culprit here. It serving correct directory on npm run dev but wrong upon npm run build
import { defineConfig } from "vite";
import laravel from "laravel-vite-plugin";
import { svelte } from "@sveltejs/vite-plugin-svelte";

export default defineConfig(({ command }) => {
    const isBuild = command === "build";
    return {
        base: isBuild ? "/public/build/" : "/",
        plugins: [
            laravel({
                publicDirectory: "storage/public",
                input: [
                    "storage/inertia/css/app.css",
                    "storage/inertia/js/app.js",
                ],
                ssr: "storage/inertia/js/ssr.js",
                refresh: true,
            }),
            svelte({
                compilerOptions: {
                    hydratable: true,
                },
            }),
        ],
    };
});

@harryqt Give me the permissions to create a PR. In you repo

Invite sent.


make a shared account for managing flux

i.e. shared, but private, mnemonic

Done, details found in pw manager


NumberformatException may need to be catched in sql connector

Some places like [1] may need to catch NumberFormatException
[1] - https://github.com/ballerinalang/ballerina/blob/master/modules/ballerina-native/src/main/java/org/ballerinalang/nativeimpl/connectors/data/sql/client/SQLConnectorUtils.java#L62

this is created in wrong repo, will create the issue in correct one


[TimeLine] Add task through selected area

✅ Closes: #176

:tada: This PR is included in version 1.27.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Order notification crash: Value was "object" expected "int" at Lunar\Base\Casts\TaxBreakdown:27

Lunar version: 0.1.4
Laravel Version: 9.30.1
PHP Version: 8.1.11
Database Driver & Version: MySQL 8.0.28

Expected Behaviour:
Sending mail when changing order statut from admin hub doesn't crash.
Actual Behaviour:
Crash even for a very basic mailable
Steps To Reproduce:

From website, submit and pay an order
Go to admin hub, view order
Change status to "payment received" (yes, same status as current)
Select the mailable to use
Select recipients
Send email
Mailable crashes, and order cannot be viewed anymore without crashing

Error report: https://flareapp.io/share/lm2BqaYm#F113
Lunar order config in config\lunar\orders.php :
use App\Mail\OrderPaymentReceived;

    'statuses' => [
...(snipped)...
        'payment-received' => [
            'label' => 'Payment Received',
            'color' => '#6a67ce',
            'mailers' => [
                OrderPaymentReceived::class
            ],
            'notifications' => [],
        ],
...(snipped)...
    ],

Mailable class in app\Mail\OrderPaymentReceived.php :
namespace App\Mail;

use Illuminate\Bus\Queueable;
use Illuminate\Mail\Mailable;
use Illuminate\Queue\SerializesModels;

class OrderPaymentReceived extends Mailable
{
    use Queueable, SerializesModels;

    public function build()
    {
        return $this->subject('ORDER PMT RVC')
            ->view('emails.order_payment_received');
    }
}

Mailable view in resources\views\emails\order_payment_received.php :
PMT RCV: {{ $order->reference }}

Closing as this refers to an older version of Lunar, if it's still a problem in the latest version feel free to reopen

For anyone that stumbles on this same issue (like me); This is caused by Laravel Telescope when it tries to serialize certain data. When I disabled telescope no error would show. (Even when the error shows, the mail is sent)


Rate limit Keycloak API access

The ssh-portal-api and ssh-token services should rate limit access to the Keycloak API to avoid excessive load.

closed in #354


feat(ledger): smart schema and credential definition registration

Signed-off-by: Moriarty<EMAIL_ADDRESS>closes #859

Codecov Report

Merging #900 (439ea29) into main (bcbed35) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #900   +/-   ##
=======================================
  Coverage   87.65%   87.65%
=======================================
  Files         470      470
  Lines       11172    11172
  Branches     1781     1868   +87
=======================================
  Hits         9793     9793
+ Misses       1375     1313   -62
- Partials        4       66   +62

Impacted Files
Coverage Δ

packages/node/src/NodeFileSystem.ts
37.14% <0.00%> (ø)

packages/core/src/crypto/JwsService.ts
90.90% <0.00%> (ø)

packages/core/src/utils/transformers.ts
79.16% <0.00%> (ø)

packages/core/src/agent/MessageSender.ts
84.61% <0.00%> (ø)

packages/core/src/agent/MessageReceiver.ts
80.76% <0.00%> (ø)

packages/core/src/modules/ledger/IndyPool.ts
82.92% <0.00%> (ø)

packages/core/src/modules/proofs/ProofsModule.ts
79.19% <0.00%> (ø)

packages/core/src/transport/WsOutboundTransport.ts
11.59% <0.00%> (ø)

...ckages/core/src/modules/routing/RecipientModule.ts
61.85% <0.00%> (ø)

...ckages/core/src/transport/HttpOutboundTransport.ts
11.62% <0.00%> (ø)

... and 23 more

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 bcbed35...439ea29. Read the comment docs.

Still need to add tests for new functionality. Old stuff works

Note to self @morrieinmaas  & @blu3beri & @TimoGlastra still need to add tests.

@TimoGlastra @blu3beri would be great to get som efeedback before I add the tests - thanks in advance

thanks for the feedback @blu3beri. all addressed.

@TimoGlastra can this be merged then?


vitest unit test coverage seems to be broken in sonarcloud

Same date as this PR was merged https://github.com/josephrkramer/batuu-scanner2/pull/165

https://www.npmjs.com/package/vitest-sonar-reporter

https://community.sonarsource.com/t/sonarqube-generic-coverage-xml-successfully-generated-but-code-coverage-not-shown/115967/3
https://community.sonarsource.com/t/currently-sonarcloud-does-not-show-coverage-analysis-on-each-pull-request/114064/12


Fixes #116, add modal usage docs

@Gregoor and @malqinneh,
I would appreciate it if you have a moment to have a look at this. The layout etc. is still in early draft form so, what I am mostly interested in is whether the content is in the right direction.
Is it too technical? Not technical enough? Too much information? Not enough information?
I want to focus initially on getting some good content put together around design elements, components, usage etc, instead of focusing too much on the technical aspects off how this will be delivered.
With that said, at the moment I am leaning heavily towards using Hugo with a completely custom MDN Web Docs theme going forward. I look forward to here your thoughts.
Running locally
You will need

Nodejs
Yarn

# once you have the above installed, clone the repo
$ git clone https://github.com/mdn/mdn-fiori.git

# change into the project directory
$ cd mdn-fiori

# fetch this branch and check it out so you are this working branch
$ git fetch origin issue#116-add-modal-usage-docs:issue#116-add-modal-usage-docs && git checkout issue#116-add-modal-usage-docs

# run yarn
$ yarn

# startup the server
$ npm start

Point your browser at: http://localhost:8080/components/modal.html

Running this gives me an error page I haven't seen before (very exciting!):

Wrt content:
I'd vote for less information, or more precisely: more technology-agnosticism. Basically my hope would be that each component has an API consiting of:

the required HTML structure, maybe similar to how frameworks like Bootstrap do it
the required CSS & JS file(s) for this component

And then each codebase would be responsible for figuring out how to include and bundle that themselves, which would hopefully reduce the scope of what you(/we) need to do here.
I hope this is roughly the feedback you were looking for and not too in-the-weeds.

Thanks for sharing @schalkneethling! I just pinged you to get your help setting up local host if you don't mind :)

Thanks for the feedback @Gregoor - I am going to merge this one in for now and we can work on rewriting the content to be more in the vein of "Just the facts mam" :)


Prisma Engine Hangs on end of test

Bug description
Prisma Engine hangs on test with Jest.
Used (https://www.npmjs.com/package/leaked-handles) to detect the open handles
child process handle leaked at one of:
    at Object.fakeSpawn [as spawn] (C:\Users\Khris\Documents\git\KoalaBotFrontend\node_modules\leaked-handles\stacks.js:46:21)
    at C:\Users\Khris\Documents\git\KoalaBotFrontend\node_modules@prisma\client\runtime\index.js:27213:40
child process handle {
  pid: 29860,
  cmd: 'C:\Users\Khris\Documents\git\KoalaBotFrontend\node_modules.prisma\client\query-engine-windows.exe',
  args: [ '--enable-raw-queries', '--port', '24765' ]
}

How to reproduce
https://github.com/ChrisF999/prisma-testing-hang
run the tests in the minimal app above: npm run test, after installing decencies.

Expected behavior
Prisma engine to close down gracefully and not hang.

Prisma information
Can be seen in the repo above

Environment & setup

OS: Windows
Database: SQLight
Node.js version: v14.16.1
Prisma version: 2.21.2

I can confirm this observation.
To use leaked-handles I needed to:

npm install leaked-handles
Add to express\__tests__\verify.test.js: require("leaked-handles");

Then I get the following output:
C:\Users\Jan\Documents\throwaway\prisma-testing-hang>npm run test

><EMAIL_ADDRESS>test
> jest

 PASS  express/__tests__/verify.test.js
  Verify tests
    √ responds with json (396 ms)
    √ responds with 201 (136 ms)

Test Suites: 1 passed, 1 total
Tests:       2 passed, 2 total
Snapshots:   0 total
Time:        4.13 s
Ran all test suites.
Jest did not exit one second after the test run has completed.

This usually means that there are asynchronous operations that weren't stopped in your tests. Consider running Jest with `--detectOpenHandles` to troubleshoot this issue.

no of handles 6

tcp stream {
  fd: 1,
  readable: false,
  writable: true,
  address: {},
  serverAddr: null
}

tcp stream {
  fd: 2,
  readable: false,
  writable: true,
  address: {},
  serverAddr: null
}

tcp stream {
  fd: 0,
  readable: true,
  writable: false,
  address: {},
  serverAddr: null
}

child process stdout stream handle leaked at one of:
    at C:\Users\Jan\Documents\throwaway\prisma-testing-hang\node_modules\@prisma\client\runtime\index.js:27126:40
child process stdout stream handle { fd: -1, readable: true, writable: false }

child process stderr stream handle leaked at one of:
    at C:\Users\Jan\Documents\throwaway\prisma-testing-hang\node_modules\@prisma\client\runtime\index.js:27126:40
child process stderr stream handle { fd: -1, readable: true, writable: false }

child process handle leaked at one of:
    at C:\Users\Jan\Documents\throwaway\prisma-testing-hang\node_modules\@prisma\client\runtime\index.js:27126:40
child process handle {
  pid: 20680,
  cmd: 'C:\\Users\\Jan\\Documents\\throwaway\\prisma-testing-hang\\node_modules\\.prisma\\client\\query-engine-windows.exe',
  args: [ '--enable-raw-queries', '--port', '58030' ]
}

The reproduction starts another PrismaClient instance when it starts, additionally to the one that is created in the server. Replacing that with const prisma = require('../util/prisma.js'); makes the main leak go away.


npm install<EMAIL_ADDRESS>fails on windows 7 even though I have latest nodejs and npm

C:\Users\sunil\Documents\git\react-native-navigation\example>npm install --save<EMAIL_ADDRESS>
<EMAIL_ADDRESS>postinstall C:\Users\sunil\Documents\git\react-native-navigation\example\node_modules\detox
scripts/postinstall.sh

'scripts' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install" "--save"<EMAIL_ADDRESS>npm ERR! node v6.11.1
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: scripts/postinstall.sh
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'scripts/postinstall.sh'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the detox package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     scripts/postinstall.sh
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs detox
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls detox
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\sunil\Documents\git\react-native-navigation\example\npm-debug.log

Detox only supports iOS currently, hence it only runs on macOS. We are in process of adding support for Android. When we do, it will make more sense of supporting Windows as well.

When is support for windows expected?

Detox is being used internally at Wix, and currently being integrated in our build process. This process reveals bugs and missing features that we are addressing as our first priority.
The second priority is to develop Android support (which are are already half way through I believe). When that is done we can address Windows/Linux support for Detox (sorry I have no estimates for you)

No problem :) That's good to know!


Move bulk action x of y to avoid extra flicky flicking

The row of bulk actions reacts to the selection of rows via the selection text to the right of the row
Normally this is a small and consistent x selected text which is taken into account when working out the row width
However some tables (workspaces) can have actions that are only applicable to certain rows
In those cases on mouse over of an action the selected text changes to a much bigger Available for {actionable} of the {total} selected
This is again taken into account for row width which creates a lot of flickering when the cursor moves over/out of an action
To resolve this move the x of y text into a tooltip
Also removed the source of flickering when moving the cursor between actions

#4873

Note - Failing drone run was due to an incorrect repo push (i'd pushed changes to origin instead of my fork.. so i deleted the branch in origin but not before drone kicked off). As normal PR run passed I'll merge


Gosund Switch dead after flash

I did my first OTA flash of tasmota today and I fear I bricked my device.  Everything seemed to go fine but now it does not work.
This is my first tasmota flash and I am very much a newbie to using Python as well.  I used a pi 3 to flash the device and all was successful, I was able to connect to the device's WiFi and go to the screen to set my home network credentials.  I scanned the WiFi, found my network, entered my password and hit save.
Then nothing.
My switch shows no power at all.  I cannot get any response from it at all.  It is not showing on my network so I cannot connect to it so I cannot connect to the tasmota settings.
I have no control on the device at all.  When I press the switch nothing happens, there are no lights flashing or appearing at all.  I have pressed and held the switch to try to reset it and no response.
I fear it is bricked but I have no experience here.
I apologize, I am not sure how to find any logs to show for this.

Just guessing here -- what if you entered the wrong WiFi password?
You definitely got Tasmota flashed alright or you would not have been able to connect to the device's WiFi.
Have you tried the following Tasmota reset procedure?
https://github.com/arendst/Tasmota/wiki/Fast-power-cycle-device-recovery
It's worth a try.

This is not a tuya-convert issue, as you have stated Tasmota was flashed successfully. I would agree with Leif's advice, failing this, get in touch with Tasmota discord support. Good luck!


Nearest to avg

Define 10 length sized integer array
Read 10 integer number from console.
Find the mean of array
Find the nearest element to mean in the array.
Print nearest element and element  index to screen.

For example: [1,2,3,4,5,6,7,8,9,10]
Output : Index : 4, element : 5
Difficulty : 10/4

Eta: 30 minute

Ata: 1 hours


No tests

This module is included in Rakudo * - the lack of tests make it very hard to insure we haven't broken anything in a new Star release.

Looks like tests were added since the creation of this issue.


Fixed formatting of LogFast

Before:
[019:48:03] Test
Test Async
[019:48:05] Test
Test Async

After:
[19:49:55] Test
[19:49:55] Test Async
[19:49:57] Test
[19:49:57] Test Async

I think the method in async module is obsolete. Since same implementation exists in none async package. Thats also why i didn't notice it had that mistake as well.

I think the method in async module is obsolete. Since same implementation exists in none async package. Thats also why i didn't notice it had that mistake as well.

@FabianTerhorst you are right the Async version is exactly the same as the normal one. I would suggest to remove the Async version (I did already in this PR).
But the format issue for the normal version is still there and should be fixed 😄

@FabianTerhorst you are right the Async version is exactly the same as the normal one. I would suggest to remove the Async version (I did already in this PR).
But the format issue for the normal version is still there and should be fixed 😄


broken link in Readme.md

seems mathworks moved the page about font issues that was linked in the Readme.md
the old URL was:
http://www.mathworks.com/access/helpdesk/help/techdoc/creating_plots/f3-103191.html#f3-96545

Fixed - thanks for reporting.


Saving a file and the cursor goes to the end of the file

Issue Type: Bug
Everytime I hit CTRL+S to save, the cursor goes to the end of file..
 Does it happen in all file types? Do you perhaps have an extension which modifies the file on save?

@cduris I cannot reproduce this in a fresh install. Perhaps there are more steps than just saving?


Crash with Cards Flipping sample project

Steps to Reproduce

Open Cards Flipping sample project
Click on the 3D transform from card_clubs_05
Crash

Log

Click to expand!
___________________________________________
############################################################################################
ERROR in
action number 1
of Draw Event
for object o_main:
bool argument is array############################################################################################
gml_Script_is_surface (line 0)
gml_Script_anon_Node_3D_Transform_gml_GlobalScript_node_3d_transform_1962201_Node_3D_Transform_gml_GlobalScript_node_3d_transform (line 57)
gml_Script_drawNodeOverlay_Panel_Preview_gml_GlobalScript_panel_preview (line 306)
gml_Script_drawContent_Panel_Preview_gml_GlobalScript_panel_preview (line 325)
gml_Script_draw_PanelContent_gml_GlobalScript_panel_data (line 413)
gml_Script_drawPanel_Panel_gml_GlobalScript_panel_data (line 354)
gml_Script_draw_Panel_gml_GlobalScript_panel_data (line 297)
gml_Script_draw_Panel_gml_GlobalScript_panel_data (line 308)
gml_Script_draw_Panel_gml_GlobalScript_panel_data (line 308)
gml_Script_draw_Panel_gml_GlobalScript_panel_data (line 308)
gml_Script_draw_Panel_gml_GlobalScript_panel_data (line 308)
gml_Object_o_main_Draw_64 (line 4)

Versions

Pixel Composer 8.1

Sample projects are created in .6 version and a lot changed since then. I'll update the sample projects after version .9 (since that version will also change lot of things) so I will mark these issues as closed for now. In the next release (.8.2) I will just remove sample projects that doesn't work. Thank you for reporting in.


API to draw diagram

Is your feature request related to a problem? Please describe.
As a dev-ops, I want to draw the diagram automatically. For example, when one switch or one node is added, I want to automatically update corresponding diagrams.
Describe the solution you'd like
I can use any network discovery tool or integrate with inventory system to get notified when new network device is identified, then I can call API to update diagrams. Thus people can always get real time view.
Describe alternatives you've considered
do it by our own. but I do like the simplicity of this software and think it achieves our network topology purpose.
Additional context
Also, is there one specification about the diagram store format? maybe we can manipulate the diagram source file to achieve the purpose.

You can use this library if you want to update programatically your diagrams.
https://github.com/pablomarle/networkmaps/blob/master/networkmapslib.js
If you are not going to use javascript (which I can imagine), you can use it as a base to understand how the api works.
I'll try to add some more extensive documentation on how to use the api.


fix(dsp): prevent cache instantiation when cache is disabled

The environment variable DSP_CACHEBOOL does not completely disable the filesystem cache.
As a result, joblib.Memory is still instantiated, which will try to create a directory on the filesystem.
In an environment where a user does not have write access to the DSP_CACHEDIR, this will throw a surprising exception despite DSP_CACHEBOOL being set to false.
File "/var/lang/lib/python3.11/site-packages/dsp/modules/cache_utils.py", line 27, in <module>
    CacheMemory = Memory(location=cachedir, verbose=0)
  File "/var/lang/lib/python3.11/site-packages/joblib/memory.py", line 1020, in __init__
    self.store_backend = _store_backend_factory(
  File "/var/lang/lib/python3.11/site-packages/joblib/memory.py", line 132, in _store_backend_factory
    obj.configure(location, verbose=verbose,
  File "/var/lang/lib/python3.11/site-packages/joblib/_store_backends.py", line 452, in configure
    mkdirp(self.location)
  File "/var/lang/lib/python3.11/site-packages/joblib/disk.py", line 61, in mkdirp
    os.makedirs(d)

Thanks @stevenhoelscher for the PR! This looks good and maintains the cache correctly. Is the PR ready to merge or pending the change you've mentioned here?


x64: use constant pool for u64 constants rather than movabs.

Another isel improvement from staring at disassemblies of spidermonkey.wasm compilation results: I noticed quite a few uses of constants of the form
   0xC0DD259:  movabsq $-9223372036854775808,%r9

where r9 is used exactly once. This is a consequence of the "emit constants right before use" logic in the x64 backend that is meant to reduce register pressure (otherwise, we get one movabs at the top of the function and then a spill/reload).
There's a better way though: a bunch of these uses are in reg/reg instructions that could be mem/reg, so we could load directly from a constant pool. So instead of orq %r9, %rax after the above, we could do orq OFFSET(%rip), %rax. This avoids the need to use a temporary register, reducing register pressure, and also deduplicates constants (because our constant pool is deduplicated) vs movabs-per-use-site.
This PR improves performance on meshoptimizer by 1%. Surprisingly, and frustratingly, it does not actually make SpiderMonkey faster. I am starting to realize that SpiderMonkey.wasm on the current compilation is cache-bound, and so further improvements in isel may not translate to further performance improvements (at least at the individual lowering level); we need better mid-end opts to advance further. But its hot blocks are a good source of examples of awkward codegen regardless :-)
Builds on top of #4080.
Also fixes an emission bug in pinsrd where RIP-relative addresses were off by one (due to extra byte at end of instruction, not accounted for in the relocation). We'll never hit this currently because no lowerings use pinsrd with RIP-relative data (only used for the constant pool) but with this PR it's now relevant.

(For context for those not already deeply lost in the depths of the x64 backend, the movabs lowering for immediates happens when the value doesn't fit in a signed 32-bit immediate field that most reg/mem/immediate operands support. So this sort of code would not appear when doing arithmetic on small integers, but appears all the time when 64-bit bitmasks are involved, e.g. in SpiderMonkey's NaN-(un)boxing sequences.)

How will this affect disassemblers? I presume they would try to interpret the constant pool as instructions and due to the varying size of x86 instructions completely mess up disassembly of instructions following the constant pool, right? Would it be an option to allow disabling the constant pool usage?

How will this affect disassemblers? I presume they would try to interpret the constant pool as instructions and due to the varying size of x86 instructions completely mess up disassembly of instructions following the constant pool, right? Would it be an option to allow disabling the constant pool usage?

@bjorn3 most disassemblers will sync to function boundaries at least (if they have that metadata, e.g. with an ELF image), so the constant pool at the end will at worst become garbage instructions after the final ret.
I do agree it'd be better to put the constants somewhere else if possible (in .rodata or at least between functions, outside of the ranges/function body sizes specified by metadata) -- but two factors work against this:

Our per-function compilation strategy means that we would need a separate pass at the end to collect all constant pools, and do fixups for rip-relative references at that point. Right now with per-function constant pools each function body is completely independent and has no relocs for constant references.
On some platforms (e.g. aarch64) the available range for constant references is limited (+/- 128MiB, IIRC), so we need at least the possibility of constant pools among/between functions as well as in a separate .rodata. And as long as we're doing that, it's simpler to adopt a strategy that always works (constants per function, with MachBuffer's constant-island handling) rather than choose between strategies and have a rarely-used path (i.e., less well-tested).

And finally I'll note that we have constant pools already, for SIMD constants; this PR is expanding their use, not introducing them at all.

@bjorn3 most disassemblers will sync to function boundaries at least (if they have that metadata, e.g. with an ELF image), so the constant pool at the end will at worst become garbage instructions after the final ret.

This PR doesn't put the constant pool in the middle in case of large functions like it does for AArch64?

It doesn't change the behavior at all; the MachBuffer will do whatever it did before for SIMD constants. On x86-64 the available PC-relative range is +/- 2GiB, so realistically the constant pool will always be at the end of the function.

@abrown or @fitzgen: rebased and ready for review (stacks on #4080, which stacks on #4091).

How about we review this after we figure out #4080?

Rebased now that #4080 merged.

@abrown or @fitzgen, I think this should be OK to review now (thanks!).

I was poking around and it looks like this PR fails tests on main due to what looks like a "merge conflict" in test added after this PR was created, but I suspect the test probably just needs an updated output

Urgh, we really need a bors-like thing... I'll take a look, sorry!


[mgmt] networkanalytics release

https://github.com/Azure/sdk-release-request/issues/4733

/azp run prepare-pipelines

/azp run js - networkanalytics - mgmt - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-arm-networkanalytics


Add jdk18 variants

Closes: #258

With as many variants and versions that we already have, we do not plan on basing the tomcat images on any non-LTS version of the JDk.
https://www.oracle.com/java/technologies/java-se-support-roadmap.html

Hi. Thanks for the info, makes sense.
It might make sense to support the latest version in addition to the LTS versions, but that might add unnecessary maintenance overhead for the maintainers.
I will be closing this, thanks again for the response.

Linking to the very relevant https://github.com/docker-library/tomcat/pull/248#issuecomment-1329908153 (sorry for not typing this up sooner :bow:).


Publish 8.1-20230330100852+0000

Publish 8.1-20230330100852+0000

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.


bug: Interactions with USDC proxy contract in mainnet

In #155 we are fixing open zeppelin proxy detection, but we then have some issues on the interactions with these kind of contracts.
How to reproduce
We'll reproduce it trying to read the USDC balance on mainnet (CA: 0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48) for Vitalik's address (vitalik.eth - 0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045)
In etherscan read as a proxy you can read the balance of an account nicely (0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045) but not in abi ninja
It seems like if we are querying directly the implementation address, like etherscan does here
Also if you open USDC mainnet in abi ninja and select a method, the URL reloads to the implementation address

Fixed in #155 !


Bump to version 0.2

Replaced GlobalSettings with SiteSettings (Plugin configuration is now from admin > settings > login).
Internationalization ready.
Using the oAuth2UserInfo database with providers.
Dealing with existing accounts.
Added Icon to login button.
Fixed README.

This has fallen between cracks, @davidtaylorhq does this look good to merge, seems fine to me.

Unfortunately I think we've left this a little too long to merge cleanly. It looks like this PR was written before all the new Authentication changes, so it's using the 'old method' of storing associations in the database.
There are a few other out-of-date things, such as using SCSS variables for the fontawesome icon. And also, the migration from GlobalSettings to SiteSettings will break the plugin for any existing users.
A few people have reported success using the OpenID connect plugin for Azure AD integration, so my preference would be to deprecate this plugin in favour of that. What do you think @SamSaffron?

@davidtaylorhq I can update the out-of-date packages and review it again.

@ahmader if you'd like to, that would be great. However, I'd strongly recommend that you try out the OpenID-connect plugin, and see if that works for you. It's used by a lot of people, and is being actively maintained/developed.
If you want to continue working on this one, the main change would be to migrate it to our new "manager authenticator" system: https://meta.discourse.org/t/adding-a-new-managed-authentication-method-to-discourse/106695

@ahmader  I am closing this off for now, feel free to open another PR with the new managed auth system support!


Parameter Defaults See Scope of Function Body

TypeScript Version:
1.8.10
Code
namespace A {
	"use strict";
	let foo = "out";
	export function bar(func = () => foo) {
		let foo = "in";
		console.log(func());
	}
}
A.bar();

Expected behavior:
I read http://www.2ality.com/2015/02/es6-scoping.html (section: 5.3. Parameter default values don’t see the scope of the body) and then tried out:
(() => { "use strict"; let v = "out"; ((f = () => v) => { let v = "in"; console.log(f())})()})()

in node v6.0.0 and got the value 'out'.
Actual behavior:
What TypeScript generates gives you the result 'in'.
Doesn't matter if you swap var for let.

FYI: ~same issue
If you read this and try:
// Doesn't work! Throws ReferenceError.
function f(a = go()) {
  function go(){return ":P"}
}

in the [playground] (http://www.typescriptlang.org/play/index.html), the error shown isn't because it does not "see" the function go (which it shouldn't), it's complaining that the default parameter is a reference to something that isn't declared yet.

the error is what i would expect. and we should be showing the same error in the case in the OP.
The emitted output will evaluate these initializes in the body of the function, i.e. the function go is what the name will point to at runtime, it is probably not what you intended, so it is an error.

@mhegazy your point is based on how TS implements codegen for ES6->ES5 translation.
However, in ES6 the original example returns "out", and the latter example fails with ReferenceError: go is not defined. In other words ES6 differs from TSC approach.
That may lead to confusion and obscure issues — I wonder whether the right thing is to keep TSC way of erroring in such cases, but make sure the error message is very clearly tailored to this special case of default initializers.
Also the original example doesn't seem to be reflected in the tests. I think it's a significant case, because unlike the trivial fail/succeed tests, it results in different value being returned based on some obscure treatment. Would be great to make sure the behaviour is well-defined and pegged to one specific outcome.

In other words ES6 differs from TSC approach.

that is correct. we are being more restrictive than what ES6 allows.

That may lead to confusion and obscure issues

there is an error reported, not sure i see how that would cause confusion.

Also the original example doesn't seem to be reflected in the tests.

Test PRs are always welcomed.

there is an error reported, not sure i see how that would cause confusion.
Initializer of parameter 'func' cannot reference identifier 'foo' declared after it.

That error in the original use case would cause confusion the reader of the code (even though it is technically correct as per TS spec). Hence my suggestion: treat this case specially, generating bespoke error. Something less academic, clearer and more actionable
Initializer of parameter 'func' cannot reference local 'foo' from the body of the function, declared at XXX.
P.S. Sure I'll submit a test PR in the meantime :-)

A PR for a better error message is also welcomed.


Add sorting of subject sets

Fixes issue #1727
Allow display_name sorting for subject sets

Thanks @alexbfree nice work!

@camallen How can i know if this has been deployed to production? it should have autodeployed right?
A quick test of the PFE testcase in #1727 is still showing unsorted values in the sorted by display_name subject_sets Panoptes request in the network monitor tab... not sure if this is because this fix hasn't yet been deployed or if something else odd is going on...

It's not deployed yet, lets do it first thing monday.


[BUG] Ping shows player's pos instead of ping's target

Hello there !
In src/Handlers.cs line 249 FinalizeFormattingAndSend(peer, playerHostName, preFormattedMessage, Plugin.StaticConfig.ChatPingPosEnabled, ev); should be FinalizeFormattingAndSend(peer, playerHostName, preFormattedMessage, Plugin.StaticConfig.ChatPingPosEnabled, pos, ev); to allow the message to contains the position of the ping instead of the player's.

Pull Request : #242

Thanks, released in 2.2.0


Ensure all projects relying on the rules widget are using the latest version

With the package being upgraded every so often, we need to ensure all our projects are using the latest version.
### Tasks
- [ ] Ensure Rules is running the latest package version
- [ ] Ensure People is running the latest package version

This PR should update Rules to the latest version of the widget (1.6.0) https://github.com/SSWConsulting/SSW.Rules/pull/1123

This PR should update People to the latest version of the widget (1.6.0) https://github.com/SSWConsulting/SSW.People/pull/551

latest is 1.6.2

This PR should update People to the latest version of the widget (1.6.2) https://github.com/SSWConsulting/SSW.Rules/pull/1141


failed to read the model file.

Traceback (most recent call last):
File "E:\code\ai\shap-e\shapetest\shapetest\shapetest.py", line 11, in
model = load_model('text300M', device=device)
File "e:\code\ai\shap-e\shap_e\models\download.py", line 110, in load_model
model.load_state_dict(load_checkpoint(model_name, device=device, **kwargs))
File "e:\code\ai\shap-e\shap_e\models\download.py", line 99, in load_checkpoint
return torch.load(path, map_location=device)
File "C:\ProgramData\Anaconda3\lib\site-packages\torch\serialization.py", line 797, in load
with _open_zipfile_reader(opened_file) as opened_zipfile:
File "C:\ProgramData\Anaconda3\lib\site-packages\torch\serialization.py", line 283, in init
super().init(torch._C.PyTorchFileReader(name_or_buffer))
RuntimeError: PytorchStreamReader failed reading zip archive: not a ZIP archive

solved the problem by re-download the file. but met another error:
File "E:\code\ai\shap-e\shapetest\shapetest\shapetest.py", line 40, in
display(gif_widget(images))
NameError: name 'display' is not defined

The original issue seems to be a duplicate of https://github.com/openai/shap-e/issues/2


Kernel plotting raising an AttributeError

When I define a RBF kernel using
K = GPy.kern.RBF(input_dim=1, variance=1, lengthscale=1)
and try to plot the kernel using
K.plot()
it raises an AttributeError as follows:
AttributeError: 'numpy.ndarray' object has no attribute 'cache'
The entire traceback is as follows :
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 K.plot()
/home/anirudh/.virtualenvs/dev/local/lib/python2.7/site-packages/GPy/plotting/init.pyc in deprecate_plot(self, *args, **kwargs)
94             import warnings
95             warnings.warn(DeprecationWarning('Kern.plot is being deprecated and will not be available in the 1.0 release. Use Kern.plot_covariance instead'))
---> 96             return self.plot_covariance(*args, **kwargs)
97         Kern.plot = deprecate_plot
98         Kern.plot_ARD = gpy_plot.kernel_plots.plot_ARD
/home/anirudh/.virtualenvs/dev/local/lib/python2.7/site-packages/GPy/plotting/gpy_plot/kernel_plots.pyc in plot_covariance(kernel, x, label, plot_limits, visible_dims, resolution, projection, levels, **kwargs)
108     elif isinstance(x, Number):
109         x = np.ones((1, kernel._effective_input_dim))*x
--> 110     K = kernel.K(Xgrid, x)
111
112     if projection == '3d':
/home/anirudh/.virtualenvs/dev/local/lib/python2.7/site-packages/GPy/kern/src/kernel_slice_operations.pyc in wrap(self, X, X2, *a, **kw)
82     @wraps(f)
83     def wrap(self, X, X2 = None, *a, **kw):
---> 84         with _Slice_wrap(self, X, X2) as s:
85             ret = f(self, s.X, s.X2, *a, **kw)
86         return ret
/home/anirudh/.virtualenvs/dev/local/lib/python2.7/site-packages/GPy/kern/src/kernel_slice_operations.pyc in init(self, k, X, X2, diag, ret_shape)
51         if (self.k._all_dims_active is not None) and (self.k._sliced_X == 0):
52             self.k._check_active_dims(X)
---> 53             self.X = self.k._slice_X(X)
54             self.X2 = self.k._slice_X(X2) if X2 is not None else X2
55             self.ret = True
/home/anirudh/.virtualenvs/dev/local/lib/python2.7/site-packages/paramz-0.5.6-py2.7.egg/paramz/caching.pyc in g(obj, *args, **kw)
274             obj = args[0]
275             if not hasattr(obj, 'cache'):
--> 276                 obj.cache = FunctionCache()
277             cache = obj.cache
278             if self.f in cache:
AttributeError: 'numpy.ndarray' object has no attribute 'cache'

I am using numpy version 1.12.0

Could you try updated paramz? The underlying parameterized package for GPy? It seems to be somehow connected with this. If this does not work, we will ahve to figure out what is going on. Habe you recently updated GPy? There is a new version out :)

On 05 Aug 2016, at 16:27, Anirudh Vemula<EMAIL_ADDRESS>wrote:
I am using numpy version 1.12.0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I am using the most recent version of paramz (0.5.6) and the most recent version of GPy (the development version - 1.2.1).
I still get the same error when I try to plot the kernel.

I tried building both GPy and paramz from the source (from respective github repositories), essentially ensuring that they are both in their most recent versions. But I still get the same error.

The automatic builds are in place, so you do not need to build things yourself anymore, no hassle there ;) I will have a thorogh look, maybe there is some other thing going on there. Thanks for the update!

On 12 Aug 2016, at 19:57, Anirudh Vemula<EMAIL_ADDRESS>wrote:
I tried building both GPy and paramz from the source (from respective github repositories), essentially ensuring that they are both in their most recent versions. But I still get the same error.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks. Let me know if you figure something out.

I cannot reproduce the error.
I am running numpy 1.11, though maybe that is the problem?

I tried reinstalling everything again.
Numpy = 1.11.0
Scipy = 0.18.0
Paramz = 0.6.6
GPy = 1.4.0
(Python = 2.7.6)
I still get the same error.
/home/anirudh/.virtualenvs/dev/local/lib/python2.7/site-packages/paramz/caching.pyc in g(obj, *args, **kw)
274             obj = args[0]
275             if not hasattr(obj, 'cache'):
--> 276                 obj.cache = FunctionCache()
277             cache = obj.cache
278             if self.f in cache:
AttributeError: 'numpy.ndarray' object has no attribute 'cache'

On a side note, GPy version 0.8.8 works perfectly (but doesn't have the plotting functionality of GPy 1.4.0 and other new functionality, which I need)

Okay final update: I resolved the error by creating a new virtualenv (clean without any site packages) and reinstalled everything from numpy to GPy. It's working now.
Thanks for the response and I hope I knew what caused the error before, but unfortunately I don't.


[Issue 63] Increase the size of the receive buffer from 64 to 81 whic…

…h will allow a complete response to be received

@peterj43, thanks for raising this issue in #63!
@Kevin-M-Smith, thanks for your responsiveness. Are you able to do the review before we merge?

May want to add an #ifndef bracketing the definition so that someone can change the buffer size at will