Version 8.0

Hello,
I have seen that there is already an updated version in the repository of rport.io. How can I use this version (8.0)?
https://github.com/cloudradar-monitoring/rport/releases/tag/0.8.0

I am sure the maintainer wouldn't mind if you propose a PR with the update.

Hello, unfortunately the upstream app has been closed-sourced and its code repository deleted. No more development will be made on this package.


Fix delimiter error message

Avoid "circular structure" error message by avoiding JSON.stringify on a parse structure when getting an invalid delimiter type (e.g. ordgroup). Instead, report type of delimiter when it's invalid.  Fixes #2184.

Codecov Report

Merging #2186 into master will decrease coverage by 5.53%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2186      +/-   ##
=========================================
- Coverage   94.73%   89.2%   -5.54%
=========================================
  Files          83      77       -6
  Lines        5304    4901     -403
  Branches      927       0     -927
=========================================
- Hits         5025    4372     -653
- Misses        256     529     +273
+ Partials       23       0      -23

Flag
Coverage Δ

#screenshotter
89.2% <0%> (-0.04%)
:arrow_down:

#test
?

Impacted Files
Coverage Δ

src/functions/delimsizing.js
92.7% <0%> (-4.11%)
:arrow_down:

src/ParseError.js
4.34% <0%> (-95.66%)
:arrow_down:

src/functions/href.js
8% <0%> (-92%)
:arrow_down:

src/functions/def.js
40.74% <0%> (-55.56%)
:arrow_down:

src/functions/kern.js
43.75% <0%> (-37.5%)
:arrow_down:

src/Settings.js
44.18% <0%> (-34.89%)
:arrow_down:

src/domTree.js
57.6% <0%> (-32.07%)
:arrow_down:

src/mathMLTree.js
67.24% <0%> (-31.04%)
:arrow_down:

src/functions/html.js
60.52% <0%> (-28.95%)
:arrow_down:

src/tree.js
58.82% <0%> (-23.53%)
:arrow_down:

... and 35 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 754e79a...0b557f9. Read the comment docs.

Nice!


Refactor entity-api entity constraints code to consolidate payload, constraints definitions and facilitate usage of ontology api

Currently there are two yaml files being used within entity-api for entity constraints.

Reference: https://github.com/sennetconsortium/entity-api/issues/21


Change opacity/alpha

It would be great if the opacity/alpha of a sharp image instance could be directly changed,
e.g. making it semitransparent, putting out as raw to buffer and overlaying another image with it.
(Also useful for visually debugging overlay compositions).

Hello, a possible workaround for this might be to overlay with a raw pixel buffer containing tiled, semi-transparent pixel value, something like the following (untested):
sharp(input)
  .overlayWith(
    new Buffer([0, 0, 0, 128]),
    { tile: true, raw: { width: 1, height: 1, channels: 4 } }
  )...

Thanks. Though an API method could be more intuitive.

sharp(input)
  .overlayWith(
    new Buffer([0, 0, 0, 128]),
    { tile: true, raw: { width: 1, height: 1, channels: 4 } }
  )

The code works insofar that it adds an alpha channel -
but the output image is now different compared to the input image, much darker.
When I remove tile: true there will be still one pixel changed -
but the output image should be identical to input image plus an alpha channel.

Hello, [0, 0, 0, 128] was an example of adding a semi-transparent alpha channel. If you want it to be fully-transparent, you can use [0, 0, 0, 0].
If you know the image dimensions then an alternative method of adding a fully-transparent alpha channel is by passing a zero-filled Buffer to joinChannel, for example (untested):
sharp(input).joinChannel(Buffer.alloc(width * height), {
  raw: {
    width: width,
    height: height,
    channels: 1
  }
})...

It would be possible to do this with an image preserving its transparency? When I try to change the opacity of a transparent png using this method I end up with a rectangle around the image, like this:

Actually what I need is to dissolve an image into another by a given percent (like IM/GM dissolve).
Is there any way of doing this?

@lovell Could you help me with this? ☝ 😄

I can't get join channel to work for me.  It seems to make the whole image clear, I just want to add an alpha channel.  It would be nice to just have a dedicated method like img = sharp(file).addAlpha();
	let img, meta;
		try {
			img = sharp(file);
			meta = await img.metadata();
			img = await img.joinChannel(Buffer.alloc(meta.width * meta.height), {
				raw: {
					width: meta.width,
					height: meta.height,
					channels: 1
				}
			});
		} catch (err) {
			error(err);
			return;
		}
		const {
			data,
			info
		} = await img.raw().toBuffer({
			resolveWithObject: true
		});
		let imageData = {
			width: info.width,
			height: info.height,
			data: data
		};
		let svgstring = ImageTracer.imagedataToSVG(imageData, 'Detailed');

Rather than dealing with raw data and Buffer allocation, something like the following might be a bit simpler (create was added in v0.17.3).
.joinChannel({ create: { background: 'black', channels: 1, width, height } })

Why even make it that verbose though?  If someone is adding an alpha channel to an image it must be for the entire image.  You can't selectively make one part of the image RGB format and another RGBA format.

@quinton-ashley I've created #1153 for this suggestion; PRs always welcome.

Hello!
I was also looking for a simple method to change the image transparency, but couldn't find one.
Why is this issue closed and why is it classified as a question? I think it's a perfectly valid feature request. @lovell
I would really like to see this in the library. Thank you for your hard work.

@slavafomin This feature request is being tracked at #618


chore(radial_bar_chart_plugin): Version bump

Build succeeded (check pipeline).

LGTM

Sonar scan result
More detail, see in https://sonarqube-gate.intgdc.com/dashboard?id=gooddata-dashboard-plugins-gate-PR70
To scan for vulnerabilities in dependencies and run unit tests (to get coverage report in sonar) please comment your PR with 'extended check sonar'.

Build succeeded (gate pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-dashboard-plugins-sonarqube-scan-zuul-gate/63/ : SUCCESS in 59s
https://checklist.intgdc.com/job/compliance/job/gooddata-dashboard-plugins-fossa-scanning-zuul/40/ : SUCCESS in 49s (non-voting)
https://checklist.intgdc.com/job/compliance/job/gooddata-dashboard-plugins-merge-compliance-check/64/ : SUCCESS in 6s


Questions on positional embeddings

Hi! Can you help with a few questions on positional embeddings:

Did you apply any positional embedding in the Transformer attention?
If so, how did you design the positional embeddings for different types of representation and multi-scale features, and how did you apply them (i.e. how the formula should write)?
If not, what's the consideration behind? Why is it not needed?

Thank you! I look forward to your reply and the code release.

Thanks for your interest in our work. We apologize for answering late.
We don't use positional encoding in our attention module. We believe that positional embedding is very likely to improve the performance, but our work is mainly focused on making transformer as a fusion method, and make the whole architecutre work in the 3D object detection task.


refreshAccessToken deprecated in favor of getRequestHeaders - mistake?

https://github.com/googleapis/google-auth-library-nodejs/blob/c23ae13863f38d43d575b8a930fa701720fcdc88/src/auth/oauth2client.ts#L710-L714
The code documentation shows that refreshAccessToken is deprecated in favor of getRequestHeaders. These two methods don't look seemingly related. Was this deprecation a mistake?

@tjwudi using getRequestHeaders will automatically refresh the access token, I believe this note is probably for implementers like gRPC where they call getRequestHeaders to populate the access token on a request -- just a note that the upstream implementer need no longer call this method.

Thanks @bcoe. The implicit behavior of getRequestHeaders can be quite confusing to API consumers who wants to store updated access token.
Proposal: I found out that getAccessTokenAsync does the actual work that refreshes access token. It updates client.credentials with the new tokens when refreshed. Should it be the alternative to refreshAccessToken as it seems more relevant and straightforward?
https://github.com/googleapis/google-auth-library-nodejs/blob/main/src/auth/oauth2client.ts#L758-L785

@tjwudi I think if you want to store tokens, your best bet is:
client.on('tokens', (tokens) => {
  if (tokens.refresh_token) {
    // store the refresh_token in my database!
    console.log(tokens.refresh_token);
  }
  console.log(tokens.access_token);
});

This should emit an event whenever the token is refreshed, could this work for you?

@tjwudi let us know if there are any issues using the client.on('tokens') functionality to detect and store an updated token. If there are bugs with this interface you're bumping into, we can fix them -- but, going forward this is the preferred interface for detcting a new token has been emitted.


Fix removing lifecycle commands from dashboard

Description
Previous fix used AfterResourcesCreatedAsync to remove annotations. But there could be a race between a resource update being published and AfterResourcesCreatedAsync being called.
This PR changes dashboard to have an annotation to exclude it from getting lifecycle commands added

Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[x] Yes
[ ] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


Add/test training against more ligature forms

Passing --ligatures to text2image gives us st ligatures in Cardo and EB Garamond, as well as ffi ffl (more?) in Garamond. If we could find a way to conditionally turn on the hlig OpenType font feature in Garamond we should be able to get ct ligatures out of it (and maybe Cardo?).
Wyld has ligatures mapped into the ASCII characters:
ÌËÊÉÈÇÅÄÃÂÁÀ
So we'd need to run text2image against these then substitute them back to the characters we want before running training.

For hlig support, it seems like we would need to modify Tesseract's text2image to support a new e.g. --opentype_features argument that could call pango_ot_ruleset_add_feature with the corresponding PangoOTTags (like Pango's syriac-fc.c does).


Database Coverage - explain why some counties are not included in the tables for some  years

E.g. Ayrshire - 100% complete for 41 & 51 but absent from list for other censuses. Do we not have images for these? We should let users know somehow.

That is because I have not added the PARMS files yet.  Have them all for
the missing ones to edit.
It will not change the % as we do not have the images anyway so once added
they will say 0%
I thought this issue was on a waffle board anyway? I know it has been
discussed.
On Thu, 28 Mar 2019 at 16:08, DeniseColbert<EMAIL_ADDRESS>wrote:

E.g. Ayrshire - 100% complete for 41 & 51 but absent from list for other
censuses. Do we not have images for these? We should let users know somehow.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeUKGen/FreeCENMigration/issues/659, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ANC91shJf2RsVkmu4LyYakk5ifN6f85Nks5vbNOhgaJpZM4cQUmV
.

It might be, if it is we can delete this one. Thanks Brenda.

To check for duplicate before closing.

@AnneV-Learn has put in some text, for review in testing.

I think this may be resolved by the new records coverage page - i.e. Catterick, North Riding of Yorkshire:

1841 = no pieces exist
1851 = 1 piece exists, but we have not transcribed it yet
1861 = 1 piece exists, and this is transcribed and online
Suggested text "a zero in the total pieces column means Catterick [or 'this place'] was not the title of this area in that census year".
What is not resolved is if we can / should list here or somewhere which pieces we will never have (destroyed) and which we have but are not yet fully transcribed.

Duplicate is possibley #701


refactor activity

please do go over what you did before i merge

please do go over what you did before i merge and have you tested?

Removed boilerplate in gateway.py for activity class and made it easier to maintain. Yes I tested and all works

merged and closing


Broken documentation link

The following documentation link is broken:
https://github.com/malcommac/SwiftDate/blob/master/Documentation/3.Manipulate_Date

Fixed in #583 and #562.

@malcommac it is still broken


[Feature]Add more info to routine load metrics

What type of PR is this：

[ ] bug
[X] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

add database name, table name, routine load name to  routine_load_jobs  metrics's  label
only master fe expose  routine_load_jobs metrics, because state  not sync to follower,   curl every fe 8040/metircs  have difference value, so remove it.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

PTAL @kangkaisen
unittest maybe have some problems?

run starrocks_fe_unittest

run starrocks_fe_unittest

PTAL @kangkaisen
unittest maybe have some problems?

We will fix the untable UT ASAP.

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 0 / 20 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/metric/MetricRepo.java
0
20
00.00%
[506, 621, 622, 623, 625, 626, 630, 631, 633, 635, 637, 638, 639, 640, 641, 642, 643, 645, 646, 647]


The open editor height is very small for a lot of opened files

The open editor height is very small for using it when I opening more than 5 files in time.
It hard to using it, and hard to close. Can you please make it responsive when more files adding, the height is get bigger?

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog for now. Thanks


requires native node streams when used with browserify + factor-bundle

I get the following stack trace when sending level-js through browserify and factor-bundle:

"browserify": "^6.1.0"
"levelup": "^0.19.0"
"factor-bundle": "^2.3.0"

Closing this due to inactivity, and much has changed since then.


Set BUZZER_PIN to output only before actually buzzing

...as shown by bikemikem here: http://www.rcgroups.com/forums/showpost.php?p=35632071&postcount=4702
Also fixed compiler warning by reordering includes.

cool I was going to add it myself, but it turns out I don't have a buzzer to test it


Dynamic queries doesn't allow to query property of a descendant class

I use Querydsl-jpa  3.7.4 + Hibernate 4.3.11 and Spring Data repository.
The class hierarchy:
@Entity
@Table
@Inheritance(strategy = InheritanceType.SINGLE_TABLE)
public abstract class VObject {
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;
}

@Entity
public class RObject extends VObject {
    @ManyToOne
    private Street street;
}

@Entity
public class Request {
    @Id
    @GeneratedValue
    private Long id;

    @ManyToOne(optional = false)
    private VObject obj;
}

I need different queries for Request for different VObject descendants. E.g. for query containing only request with obj of RObject class I need to filter them with fields specific for RObject.
There is BeanPath.as(Class clazz) method which seems to be the solution.
But when I try to use it:
QRequest vr = QRequest.request;
BooleanBuilder builder = new BooleanBuilder();

QRObject obj = vr.obj.as(QRObject.class);
builder.and(obj.street.fullTitle.containsIgnoreCase(filter.getAddress()));    // NPE here as obj.street is null

I get NPE as obj.street is null.
It doesn't seem to get initialized.
There is a workaround:
Just set obj.street to "new QStreet(new PathMetadata(obj, "street", PathType.PROPERTY))" via reflection and the query appears to work.
But it seems to me like a bug.

The NPE happens because of Path initialization limits http://www.querydsl.com/static/querydsl/4.1.3/reference/html_single/#d0e2260
To fix the issue add an explicit join between request and request.obj:
query.from(request).innerJoin(request.obj, obj).where(...

Thanks for your answer.
But the NPE happens when building Predicate object which used as parameter for where() method of query. I.e. before it's possible to use explicit join.
When I building the predicate I can use BeanPath.as() method which seems to work incorrectly.
If I trying to use @QueryInit annotation:
@Entity
public class Request {
    @Id
    @GeneratedValue
    private Long id;

    @ManyToOne(optional = false)
    @QueryInit({"street"})
    private VObject obj;
}

I'm getting the compilation error:
Illegal inits of Request.obj: street not found.
It may be correct as Request.obj is of type VObject which doesn't have such property. Only it's descendant (RObject) does.

The joins are serialized before where, you can refer to joined elements in the where part.

The code:
QRequest vr = QRequest.request;
BooleanBuilder builder = new BooleanBuilder();

QRObject obj = vr.obj.as(QRObject.class);
builder.and(obj.street.fullTitle.containsIgnoreCase(filter.getAddress()));    // NPE here as obj.street is null

JPAQuery query = new JPAQuery(em).from(vr).where(builder.getValue());

Results in NPE at the commented line. I.e. before call to JPAQuery.where().
The line:
QRObject obj = vr.obj.as(QRObject.class);
Doesn't initialize instance of QRObject fully. Fields which belong to VObject are initialized, but not ones which belong to RObject (street in that case).
Initialization of the field via reflection hack solve the NPE and the query work as expected.
But the necessity of this hack seems like a bug. And I'm not sure that such initialization is correct and will always work.

Why this issue is labeled as resolved?
It's definitely a bug as Casting expressions works incompletely.

We're investigating a bit, I see what you mean.

I have the same problem and know that issue has many time, but ¿could you detail the workaround used?

There is a workaround:
Just set obj.street to "new QStreet(new PathMetadata(obj, "street", PathType.PROPERTY))" via reflection and the query appears to work.

I think my PR for treated paths (#2530) can also be considered a fix for this, as the treated paths have all default PathInits sets.


Investigate doing SSR at publish time instead of request time

User story
As an MDN engineer, I want to know how difficult it would be to do server-side rendering of our React UX to HTML at the same time that we pre-render JSON data to S3. That is, can we render HTML when the document is edited and save it to S3 rather than waiting to do SSR when the web page is requested?
This is a timeboxed task to investigate the feasibility of doing this. The question is really whether we have the data we need when the page is published or if there is data required for the rendering that is only easy to obtain during a request. If this turns out to be trivial, then I'll try to produce a prototype PR for it. Otherwise, the task is just to report on what will be involved.
Acceptance criteria

there will either be a pull request with a prototype of the feature, or
comments below explaining what work will need to be done to implement the feature.

One of the complications of doing this at publish time instead of request time is that an english document will be used in any locale that does not have its own translation. So when an english document is published, we'll have to publish the JSON and then multiple HTML files.

Looking at react_document() (in document.py) we don't do much with the request object. It is used for translations, but (as above) we'd have to handle translations differently for the publish flow. It is used for redirects, but we would have to handle redirects in some other way in this pre-render case anyway.
And the request object is used to get the search_url property in the context object, but that property is unused in react_template.html so it is safe to delete that.
So I'm thinking that we could refactor document.py to define a get_jinja_context_for_document() function, and then call that function from react_document() in the same file, and also from the publish task that saves JSON to s3. That object could be simpler than what we're using now, and it ought to allow us to render the template in both places

Turns out that react_document.html includes some files that do use the request object.
Notably, google_analytics.html uses request.user and also waffle flags, so I'll need to get those out of there as part of issue #1452.
lang_switcher.html uses the request object to get query paramters and encode them as hidden input fields in the form along with the select element of languages. Not sure what the point is, but I think we could just take that out. Or even simpler just remove the menu from the footer entirely. It has a whole bunch of languages that are not actually well-supported. Might be simpler to not list them at all.
newsletter.html uses the request object to determine the request url to say "english only" if the request url is not english. For the SSR case we know what language we're rendering to, so we could pass that in via a different variable, or we could create some kind of mock request object.
There are also a bunch of files in kuma/wiki/jinja2 that reference the request object, but since we're removing all the wiki-related functionality from our React pages, so I don't think we're using any of those.
So the update here is that doing SSR to S3 still seems not too hard. But harder than it seemed yesterday. And it now seems clear that I need to tackle #1452 to remove waffle flags first. Then, if time remains in the sprint, I can come back to this and work on it.  Or, if I run out of time, then the investigation I've done here will satisfy the acceptance criteria


Docs(): update ISSUE_TEMPLATE.md - removing angular-translate reference

Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your master!
[x] Make sure you are making a pull request against the canary branch (left side). Also you should start your branch off our canary.
[ ] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.

Description
Please describe your pull request.
💔Thank you!

Oops 😨  Thanks.

Thank you, landed as ce5a65c


About The TLESS result

Hi, I found that the data for AAE and Multipath AAE [38] in the text is different from the original text. What is the reason for this?

Hello. Thank you for your question.
The original Multipath-AAE reports by rendering with the reconstructed T-LESS objects (cf. Multipath-AAE Fig.1, Sec 4.2, and also the demo shown in their github-repo). But in DISP6D, we report and compare with related methods by rendering with the CAD ones (cf. DISP6D, Sec 5.4-Baselines).
The reconstructed objects often suffer from inaccurate geometry, hindering the bounding box comparison for translation estimation (cf. AAE-IJCV, Sec 4.4), resulting in a worse recall rate under e_vsd.

Thanks for your reply！！！


intValue() always throws exception

case .number(value as Int) always fails as Number can not be casted to Int
GraphQL: 1.1.7
Xcode: 12

Please check this https://github.com/GraphQLSwift/GraphQL/pull/71


Detail-Page-Tab-View

I have changed detail page view as per figma design with 4 tabs with data tab instead of view data button like @zackcl shared recently in design group.

hi @Yagnik56 I will try to get a review to you this this week. we are focused on testing out release code at the moment, maybe @zackcl will need to take a look after his vac this week anyway though since he did the design changes. thank you!

Here are few screenshots to get better idea about update.
Data:

Overview:

Design:

Participants:

Metrics:

Sorry for the delay, I will review this soon.

@Yagnik56 I just reviewed this branch, great work!
I notice 2 things that need to be updated:

The "POST RULE" appears at the top of the tabs for some reason, so this should be removed.

The "Data" tab should appear once the experiment status is set to "Enrolling". Until then, there should only be 4 tabs without the "Data" tab (And the default tab should always be the "Overview" tab - the user should be in this tab after creating an experiment and also changing the status to "Enrolling").

Also, there should be an "Aliases" table in the "Design" tab but maybe this can be updated with another PR.

Here you can find the latest design in Figma: https://www.figma.com/file/uYxnCT2dXTftSZKFQ00Q0y/UpGrade-UI-Prototype-(latest). If you have any questions, please feel free to ask.

Sorry for delay @zackcl  and many commits, there were some conflicts in few files.
The "POST RULE" on top section was one of conflict with dev, it's resolved now.
I have made changes for data tab according to your 2nd point and kept overview tab as default.
I have switched between experiment states to verify changes, you can find few images for same below.

let me know if any changes required.

@Yagnik56 I will resolve these conflicts on what is in github today. If you have work that has not been pushed up, let me know and we can work through those merge conflicts as well if need be.

@Yagnik56 Sorry for the delay, I just tried this and it seems to work fine functionally.
But style-wise, I would like to suggest using the same font used for tabs on the other pages in UpGrade (e.g. Logs, Profile) for consistency.

For example, the font should be "Open Sans" with a 22px size and 600 weight. Also, when a tab is clicked, the grey selection box should be transparent. Lastly, the text should not change to light blue when the tab loses focus (by clicking somewhere else).
I think it should be good to be merged after these small changes. Thanks for the great work!

@Yagnik56 Sorry for the delay, I just tried this and it seems to work fine functionally.
But style-wise, I would like to suggest using the same font used for tabs on the other pages in UpGrade (e.g. Logs, Profile) for consistency.

For example, the font should be "Open Sans" with a 22px size and 600 weight. Also, when a tab is clicked, the grey selection box should be transparent. Lastly, the text should not change to light blue when the tab loses focus (by clicking somewhere else).
I think it should be good to be merged after these small changes. Thanks for the great work!

@zackcl I haven't made any changed in Logs, I have only addressed tab view in experiment section only. should I make changes on Logs as well? also I didn't understand grey selection box, can you share pic where change is required!

@zackcl I have made changes you mentioned, let me know if any other changes required otherwise we can merge it with dev.

@Yagnik56 I think it looks good now.
But in terms of hiding/showing the Data tab, we just decided to make it always visible and place it next to the Metrics tab so that it's basically like the current details page but with tabs (because hiding it can be a little confusing).

You can check out the slack post where we discussed this. Steve Ritter also mentioned, "If there’s no data, we could just display no data received yet" but I personally don't think it's necessary for now because the user can still know there's no data if the participant enrollment is 0.

@Yagnik56 I think it looks good now.
But in terms of hiding/showing the Data tab, we just decided to make it always visible and place it next to the Metrics tab so that it's basically like the current details page but with tabs (because hiding it can be a little confusing).

You can check out the slack post where we discussed this. Steve Ritter also mentioned, "If there’s no data, we could just display no data received yet" but I personally don't think it's necessary for now because the user can still know there's no data if the participant enrollment is 0.

@zackcl I have put data tab as it was in last, removed hiding setting when exp.state isn't enrolling.

@Yagnik56 Sorry I was a little busy today, I will review this tomorrow. Thanks!


Add contributed notebooks about LSST simulations with TRILEGAL

Create new directory 05_Contrib/LSST_Sim
Add README_LSST_SIM.md
Add single_and_binary_models.ipynb to combine single and binary catalogs
Add lsst_sim_SGP_plots.ipynb about single stars in the South Galactic Pole
Add plotting functions in utils.py

Hi @gpastorelli-astro , thanks for the PR! I took a look at the notebooks and have a few minor comments.
In lsst_sim_SGP_plots.ipynb:

Under the "Getting useful data" section, the sentence "More details are given in Table 4 of the paper" does not include a hyperlink to the paper. When I try to click on the hyperlink it doesn't work. I assume it should link to: https://iopscience.iop.org/article/10.3847/1538-4365/ac7be6#apjsac7be6t4
Under the same section, change:
umag = R['umag'] # LSST g mag
to
umag = R['umag'] # LSST u mag
Under the "Color-magnitude and color-color diagrams" section, in the cell about 6 panel plots for number counts, you repeat "3." twice.
You repeat "3." twice again in the cell about 6 panel plot - CMDs
... And again in the next 6 panel plot cell!
Spelling error in the cell below the cell explaining "In the following, we plot the color-color diagrams (g - r) vs. (u - g)...". Change statisticistic --> statistic

In single_and_binary_models.ipynb:

Under the "Goals" section, change lsst_sim.dir --> lsst_sim.simdr2 and change lsst_sim.dr2_binary --> lsst_sim.simdr2_binary
Under the "Luminosity functions for single and binary stars" section, you repeat "1." twice.
In the same cell as above, you repeat ax1.set_title("Single stars") twice.
Again in the same cell as above, you have a section for # Axis labels, but set the xlabel for ax1 and the xlabel for ax2 before this section...
For the Hess diagrams color-coded according to stellar density and the color-color plots, I suggest you include only one colorbar instead of three. You could remove these two lines from both cells:
make_cbar(ax1, cc, "# star/pixel",  extend = None)
make_cbar(ax2, cc, "# star/pixel",  extend = None)

Hi @jacquesalice, thanks for reviewing the notebooks!
I addressed your comments in the latest commit c07414b

@gpastorelli-astro awesome, thanks! I'm sure @rnikutta will also want to take a look at the notebooks before merging, but he's on vacation until Friday so probably won't get to these until next week.

Thanks @jacquesalice and @gpastorelli-astro ! I'll go through the NBs myself later today (just back from vacation). Thank you for contributing them!

Thanks @rnikutta! Waiting for your feedback

Thank you @gpastorelli-astro for the contributed NBs and MD file! Below just a few comments and suggestions:
lsst_sim_SGP_plots.ipynb

I recommend to delete the commented-out cell which begins with '# Uncomment the following to check if your token is valid'; the cell above (which uses ac.login()) is sufficient.
Just for consistency, I recommend to replace "lines of the table" with "rows of the table" in this sentence: "With our selected query, we now retrieve the data and print the number of objects and the first lines of the table"
For the very first plot (LSST r vs. (g-r) CMD) I propose to make the figure slightly larger, e.g. (6,6), and the plot marker symbol smaller, e.g. s=0.5 . In addition, an alpha=0.4 or so would make the features of the CMD a little bit more visible.

single_and_binary_models.ipynb

Same as above, recommend to remove the commented-out cell that starts with "# Uncomment the following to check if your token is valid"
Remove the cell that defines the function multimode_gc(); the function is not used in this notebook.
In the very first plot (luminosity functions), right panel title says "stars stars" (duplicate)
In the first code cell in section "CMDs and CCDs", remove the commented-out line #get_2Dhess_param(gmag60-rmag60, rmag60, None, stat='count', bins=bins) (I assume it's an artifact left over from testing).

Thanks again, very nice NBs!

Thanks, @rnikutta! I modified the NBs according to your suggestions.

Thank you @gpastorelli-astro , merged into master branch!


Suggestion to add Traditional Chinese translation

Hi!
I’ve been using json4u and really appreciate the tool. I’d like to suggest adding support for Traditional Chinese translations. If this is something you’re open to, I’d be happy to help with the translation work and submit a pull request. Let me know what you think!
Thanks for your great work! 😊

@PeterDaveHello Thank you, I am open to supporting more languages. However, currently, i18n routing is based on domains. If we add Traditional Chinese, how should we support it? Do you have any suggestions?

I think using subdomains will increase the complexity of maintenance. Perhaps providing a settings option in the top right corner of the header component on the homepage for users to choose their language might be more appropriate.

The purpose is not pure as it contains a political stance; therefore, I will close the issue.

Looks like sometimes even the simplest intentions can be seen through a different lens. I was just hoping to help with language support, but I understand if it’s interpreted otherwise. 😊


339 add synthetic corpus

Added synthetic corpus based on HPOA - single phenopacket/patient for each OMIM disease in the annotation.

@matentzn This is technically ready, but are you happy with the 5000+ files being checked into version control?

Ok, shall we instead just attach the synthetic corpus to the pheval github release?

I think that might be the best way to go about it


Problem to see balance

Hi,
Seens yesterday im trying to see my balance and always appears as "loading".
I sent to my wallet an ammount of ETC and at first a saw them, then a send some more and I couln't see them any more. Almoust 24 hours with this problem.
Thanks!

can you try again now? it seems to be working. for some reason the server is seeing a lot of traffic these last few days, and is getting overloaded.

Thanks
El 22/3/2017 21:16, "Elaine Ou"<EMAIL_ADDRESS>escribió:

Closed #21 https://github.com/ethereumproject/etherwallet/issues/21.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ethereumproject/etherwallet/issues/21#event-1011567161,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZYOVw-L4dib0Ak5-WwXU0w6G-E-Y2S4ks5rocfjgaJpZM4Ml5rP
.


Changes to SDR alert poller to support all SDR types.

Modify ipmitool sdr parser to handle sdr -v output
Change SDR alert poller to publish alert for discrete SDR w/states asserted.

This change will break LTAE compatibility with the SDR poller.  Kranti is working on a fix for this, but the onserve changes should be merged before merging this change.

@RackHD/corecommitters @srinia6 @uppalk1 @dalebremner

nice test coverage! +1

Awesome! :+1: if you fix the semicolon/line too long lint errors.
Also, @VulpesArtificem should take a look.

Also looks good to me :+1:

Thanks for the quick reviews!  I'll clean up the linter errors and post another patch shortly.

Just pushed a new patch that addresses review comments and linter errors.  I did find that houndci was complaining about long lines in weird places.  All of the lines in the diff should be less than 100 lines now.

:+1:

@brianparry should I still hold off on merging, or have the onserve changes gone in (no link?)

@benbp Yeah - please hold off on merging.  There's no PR yet for the onserve changes, I'll post a link here when Kranti submits it.

Ok.

Pull request for onserve changes is here: https://github.com/RackHD/onserve/pull/98

@benbp https://github.com/RackHD/onserve/pull/98 has been merged, so this PR can be merged too.


makefile: dep tools adjusted for linux

for https://github.com/dawenxi-tech/2fa/issues/29
This modifies dep-tools to switch based not the OS and to create the needed stuff for linux.
dep-tools is called from "all" now, so its done once.

Works perfect. Thank you!


Support for metadata in InvocationRequest.trigger_metadata

InvocationRequest message has a trigger_metadata field.
For HTTP requests it contains the following:
trigger_metadata {
  key: "sys"
  value {
    json: "{\"MethodName\":\"function_name\",
            \"UtcNow\":\"2018-02-20T22:28:20.732233Z\",
            \"RandGuid\":\"610a408d-ff49-48ad-9a03-65be1b518114\"}"
  }
}

Do we need to make any of this available via the Context or HttpRequest objects?
cc @asavaritayal @christopheranderson @pragnagopa

Here is an example for  accessing queue trigger metadata. Metadata properties are available by accessing: context.bindingData. We should do the same here.

We've already figured out how to use triggerMetadata for queue bindings, so this PR can be now closed.


remove unused imports in scheduling script

[x] Tests added / passed
[x] Passes black dask / flake8 dask

In this PR, I'd like to remove two imports in docs/source/scripts/scheduling.py.
Thanks for your time and consideration.

Thanks for the PR James! This seems reasonable to me. I suppose we should be running flake8 on .py files in docs?

sure! Yeah I think it should be run on all Python code in the repo. I think that's easier to maintain than picking specific code to run it on.


Getting ParseError when mocking sts calls

I wrote a small script using rusoto_sts and it's working flawless so far (thanks for the library at that point :slightly_smiling_face: ).
However, when I tried to write unit tests with rusoto_mock, I was getting ParseError("Expected StartElement  got None") no matter how I implemented the MockRequestDispatcher (I used with_body and passing a string, I used the MockRequestReader to read from a "real" AWS response that I piped into a file and I'm now using the serialize_structs feature to directly use a AssumeRoleResponse with with_json_body since I thought there might be a typo or issue with my input.
Still I'm getting the same ParseError.
I'm new to rust and might have made some rookie mistake here, but I really don't know where to start debugging the issue (apart from looking for people with similar issues, which I sadly didn't find).
Small code example that generates the error:
let client = StsClient::new_with(
    MockRequestDispatcher::default().with_json_body(AssumeRoleResponse::default()),
    MockCredentialsProvider,
    Default::default(),
);
let request = AssumeRoleRequest {
    role_arn: String::from("arn:aws:iam::123123123123:role/user"),
    ..AssumeRoleRequest::default()
};

eprintln!("{:#?}", client.assume_role(request).await);

The StartElement it's expecting is an opening XML tag, so the body you give to your MockRequestDispatcher must be XML.
The STS API documentation provides an example response for this request, so you can adjust your first statement to be similar to the following:
let client = StsClient::new_with(
    MockRequestDispatcher::default().with_body(r#"
<AssumeRoleResponse xmlns="https://sts.amazonaws.com/doc/2011-06-15/">
  ... rest of your XML response (copied from the link above and edited)
</AssumeRoleResponse>
"#),
    MockCredentialsProvider,
    Default::default(),
);

Oh thanks a lot, having the XML in with_body/1 is working.
But does that mean the with_json_body/1 is broken for STS (or at least not usable), as there doesn't seem to be a way to control those surrounding tags when passing a AssumeRoleResponse to with_json_body/1?

I haven't found a way to use with_json_body with the XML services, no, although I haven't played around with it that much either.


"Discovering Python Interpreters" is occasionally very slow after window reload.

Testing #11755
I'm having a bit of trouble pinning down exactly when this happens, but I had assume that with caching this wouldn't be the case as much any more. Sometimes when I reload the window (no changes to python interpreters) the "Discovering Python Interpreters" step takes 30s or so. The whole wait time there I don't have access to any of the python controllers.
Here is a semi-bad recording that I captured. Just a reload (nothing new added).
https://user-images.githubusercontent.com/812783/197867350-4ef07bf0-642b-4147-bfce-8f3a55810da8.mov

@IanMatthewHuff please could you provide the debug logs

@DonJayamanne Looked to me like the time might be on the Python side, so I captured Jupyter and Python logs for one of these slow instances.
JupyterLog.txt
PythonLog.txt

@IanMatthewHuff you should see a progress indicator contributed by pytohn extension on the bottom left of the status bar (coudln't see it in the video).
Let me know if you see that is visible.
I'm adding some logging that would help us figure out where the delay is.


Python menu

User story:
As a student, calculating grades for each class is tedious. So, I want to see the grades online so that I don't have to do those calculations by hand.
persona: student who does not want to calculate grades
feature: student views the grades online
business value: students do not have to waste professor's time to talk about grades
Task List

[ ]  Make sure the database schemes handle storing and viewing of grades
[ ]  Create HTML view for viewing grades
[ ]  Connect the HTML views to the database/backend to finish the function

Acceptance Criteria

only for the current semester
works only for Chrome

Acceptance Tests

the student should be able to view existing grades for specific courses
the student should be able to view possible "what-if" grades for the course


Add Accept-CH header model

Addresses part of #5010

So what I'm unsure about with this is that it says its not supposed to support
Forbidden header name no CORS-safelisted response header no

So what I'm unsure about with this is that it says its not supposed to support
Forbidden header name no CORS-safelisted response header no

Hi @ChristopherDavenport , sorry I'm sure I fully understand the question - are you saying that we shouldn't model the Accept-CH header? Or that I should use a different parser that accounts for the cases above?


Bump Gitlab-EE version to 9.4.1

I'm aware there's currently an issue open ( https://github.com/kubernetes/charts/issues/1138 ) relating to the ownership of Gitlab charts going forward given Gitlab now publishing official charts being maintained by them.
However the current version used by the published chart is now one unpatched for a serious security vulnerability as well as numerous other improvements.
Matching CE version bump PR: https://github.com/kubernetes/charts/pull/1562
Tested on GKE.

/ok-to-test


Use "gci-resource" to trigger google compute image rebuilds.

Rebuild our heavy-lifter images when the upstream base image has been
updated by the platform owner.

@smgoller please?


refactor mutex locks into hat-trie implementaion

Details:

locking is done inside hattrie operations
implementation is threadsafe except for iterators

please let me know if you have any questions or concerns.


Issue 1826

@nickbattam Is this pull request enough..I remember there being an additional unit test failing...could you add your fix for that one to this pull request

@shroffk Sorry for the delay looking at this, I messed the notification.

I'm still seeing two intermittent test failures:
Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 2.025 sec <<< FAILURE!
exceptionInFunction(org.diirt.datasource.sim.ErrorHandlingTest)  Time elapsed: 1.023 sec  <<< FAILURE!
java.lang.AssertionError: Didn't receive notifications
at org.junit.Assert.fail(Assert.java:93)
at org.diirt.datasource.sim.ErrorHandlingTest.exceptionInFunction(ErrorHandlingTest.java:90)
and
Tests run: 3, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 3.519 sec <<< FAILURE!
activeScanningRate(org.diirt.datasource.ActiveScanDecouplerTest)  Time elapsed: 0.515 sec  <<< FAILURE!
java.lang.AssertionError:
Expected: a value less than or equal to <5>
got: <6>
at org.junit.Assert.assertThat(Assert.java:780)
at org.junit.Assert.assertThat(Assert.java:738)
at org.diirt.datasource.ActiveScanDecouplerTest$1.call(ActiveScanDecouplerTest.java:54)
at org.diirt.datasource.ActiveScanDecouplerTest.repeatTest(ActiveScanDecouplerTest.java:83)
at org.diirt.datasource.ActiveScanDecouplerTest.activeScanningRate(ActiveScanDecouplerTest.java:45)


west.yml: hal_stm32: Generate I3C pins

Generate I3C pin definitions.

@SamP20 I'm sorry, this PR completely went off my radar. Anyhow there were hal_stm32 updates in between and i3c pins definitions are now available. I'm then closing this PR


Issues with service IPs on loopback interfaces

Hi,
we tried to set up Jitsi on boxes which are connected to an IP Fabric and therefore have their system / service IP configured on the loopback interfaces. It turns out for some reason there are explicit checks in the code to ignore IPs configured on the loopback interfaces so the following configuration does not work:
auto lo
iface lo inet loopback
    address <IP_ADDRESS>/32
    address 2001:db8::2342/128

Digging through the code it seems you are check for the usual loopback interface where 127/8 + ::1/128 lives. So it turns out the following config does work (when announcing the IPs into the fabric as well):
auto lo
iface lo inet loopback

auto yolo
iface yolo
    link-type dummy
    #
    address <IP_ADDRESS>/32
    address 2001:db8::2342/128

Is there any reason to not allow IPs configured on the loopback interface? Why does the library care about the network config on the host?
Best
Max

I think the code was meant to avoid needlessly listening on and advertising loopback addresses. We've just never used it in a situation where a loopback interface has useful addresses. A patch to allow use of loopback interfaces would be welcome, just put it behind a property and preserve the current behavior as default.


Nit: omit type="text/css" from

In HTML5 this is the default:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/style

Hmm. All done 👍 I won't release it just yet, as nothing's broken. Thanks though.


Category Ordering

A sellers listings should be ordered alphabetically instead of in the order which they were created. This will help buyers find the category they are looking for faster, especially as sellers stores grow larger and the amount categories they have will increase.

+1 categories should sort alphabetically in the select menu
At some point in the future, we'll probably want to add different sorting options for the listings within a category (Order by date created, order alphabetically, order by most rated, order by highest rated, etc)


[WIP] ECDC-3313: Fix ep01 bug

For p4p the Value class was not in the map, so even if the PV was connected it appeared as unknown.
It is a one line fix, but I added a regression test for it and fixed up the other tests (should have been a separate PR, but ...).
If a PV never connected then it doesn't send anything to Kafka.
It should log this though because it might be a mistyped PV name and we need to know about that.
CA needs fixing too.

Codecov Report

Merging #229 (12c66a4) into main (714c390) will increase coverage by 0.11%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #229      +/-   ##
==========================================
+ Coverage   88.71%   88.82%   +0.11%
==========================================
  Files          25       25
  Lines         886      886
  Branches       99       99
==========================================
+ Hits          786      787       +1
  Misses         75       75
+ Partials       25       24       -1

Impacted Files
Coverage Δ

forwarder/update_handlers/ep01_serialiser.py
100.00% <100.00%> (+2.38%)
:arrow_up:


cmd/go: do not create subdirs of $GOBIN

If you are on a Mac and you run (say) GOOS=linux GOARCH=ppc64 go install cmd/go, it needs somewhere to write the binary, and $GOROOT/bin/go has the Mac copy of the go command, so it uses $GOROOT/bin/linux_ppc64/go. So far so good.
But if $GOBIN is set, it uses $GOBIN/linux_ppc64/go. Since $GOBIN is not under Go's control (not part of $GOROOT's conventions) we should not be creating subdirectories. The install command should report an error instead.
@robpike just ran into this with GOARCH=386 set.

A slightly related problem is that we don't append installsuffix to bin/$GOOS_$GOARCH directory name. See #9737.

Closing this bug means completely killing go install support for cross compilation. I can now probably simulate the previous go install cleverness by using Makefile and go build -o, but it's a sad moment that Makefile is now required where previously go install was everything that was needed.
Can this issue please be revisited. I really liked the way this worked before, native binaries to $GOPATH/bin, cross-compiled to $GOPATH/bin/$GOOS_$GOARCH, it was a great FEATURE!

@zcalusic that is a feature, but this issue is about not creating subdirectories when the GOBIN environment variable is set.  You seem to be describing the case when GOBIN is not set.  If that is not working, that is a bug.  I see you've opened #11778 for that.


No wildcard(s) support for != operator

Hi @massfords ,
we noticed in our project that the != operator does not support wildcard(s) like == does, because it is only assigned to <> in SQL.
Is this restriction by intention?
If not, we can handle it as a bug which can be easily fixed. If yes, maybe we can provide a plugin like this to let developers overwrite the behavior if wanted:
import invariant from 'tiny-invariant';
import type { ComparisonNode } from 'ts-rsql';
import { formatKeyword, formatValue, RsqlOperatorPlugin, RsqlOperatorPluginToSqlOptions } from 'ts-rsql-query';

/**
 * Plugin for a `!=`-overwrite (not-equals) operation to support `NOT ILIKE` if search string contains wildcards.
 * [SQL](https://www.postgresql.org/docs/current/functions-matching.html#FUNCTIONS-LIKE) mapping:
 *
 * - `field!=test` => `field <> 'test'`
 * - `field!=*test*` => `field NOT LIKE '%test%'`
 * - `field!=*test` => `field NOT LIKE '%test'`
 * - `field!=test*` => `field NOT LIKE 'test%'`
 */
export const OverwriteNotEqualsPlugin: RsqlOperatorPlugin = {
  operator: '!=' /* Operator override! */,
  invariant: (ast: ComparisonNode): void => {
    invariant(ast.operands);
    invariant(ast.operands[0]);
  },
  toSql: (options: RsqlOperatorPluginToSqlOptions): string => {
    const { ast, keywordsLowerCase, selector, values, config } = options;
    const { detachedOperators } = config;
    const operand = ast.operands![0];
    const leadingWildcard = operand.startsWith('*');
    const trailingWildcard = operand.endsWith('*');
    if (leadingWildcard && trailingWildcard) {
      values.push(`%${operand.substring(1, operand.length - 1)}%`);
    } else if (leadingWildcard) {
      values.push(`%${operand.substring(1)}`);
    } else if (trailingWildcard) {
      values.push(`${operand.substring(0, operand.length - 1)}%`);
    } else {
      values.push(formatValue({ ast }, config));
    }
    return `${selector}${
      leadingWildcard || trailingWildcard
        ? ` ${formatKeyword('NOT ILIKE', keywordsLowerCase)} `
        : `${detachedOperators ? ' ' : ''}<>${detachedOperators ? ' ' : ''}`
    }$${values.length}`;
  },
};

(I used this already as workaround in our project for now.)
Options Summary:

Treat as bug and fix in code
Provide an out-of-the-box plugin by this lib
Provide no plugin (i.e. not a bug but intended behavior):

Do nothing or
Document such an implementation option for this case in README

What do you think?

I think it's a simple bug. I probably hadn't considered this use case.
If you can fix and update the integration test to exercise this then I'm good merging it.
Thanks

PR is created: https://github.com/massfords/ts-rsql-query/pull/7
BTW, also fixed some other stuff I found on the way.


[test] validate that TLSInputReadyCondition is ready when cert secret…

…s are set
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/1563

recheck

recheck

rebased

recheck

recheck

recheck

recheck

recheck with debug flag

per slack chat the zuul problem here is because of this line https://github.com/openstack-k8s-operators/ci-framework/blob/af461d60e1e22de0e5c37632f8a1fcdf333bce62/zuul.d/base.yaml#L64
which stops the content provider from running in the layout, this one https://github.com/openstack-k8s-operators/ci-framework/blob/af461d60e1e22de0e5c37632f8a1fcdf333bce62/zuul.d/project-templates.yaml#L46-L48

recheck

recheck

recheck

recheck

recheck noop job


Remove Flask login which adds user_id to cookie

What is the context of this PR?
PR #1657 causes the user_id to get stored in the session cookie.
EQ's current encryption model for questionnaire state intends for the user_id to not be exposed to the user, only the user_ik.
This line was only added as an optimisation to save an extra DB hit and therefore I have removed it.
How to review
The cookie can be verified by copying the value of a session cookie to https://www.kirsle.net/wizards/flask-session.cgi and decoding it).
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

This is obviously not needed anymore then either.
https://github.com/ONSdigital/eq-survey-runner/pull/1657/files#diff-4d5ff7e65b71e6875dfead2645e34354R12


Update configuration for frozen-string-literal

In a future Ruby version strings will be frozen by default. You can simulate the new behavior with RUBYOPT="--enable-frozen-string-literal". With this enabled the url from configuration is mutated by gsub!. We can avoid this by marking these strings as mutable.
The error I hit running the test suite with was a string mutation from: https://github.com/DataDog/datadog-api-client-ruby/blob/master/lib/datadog_api_client/configuration.rb#L585

Please say open ❤


Suggestion: support ||= compound assignment

Perhaps there is a good reason for this but I never understood why the compound assignment ||= is not part of js. This would be helpful to ensure a variable to be defined.
Given:
let a = {"key":"value"}

This would let us write:
a.key ||= "default";
a.anotherKey ||= "anotherDefault";

Instead of
a.key = a.key || "default";
a.anotherKey = a.anotherKey || "anotherDefault";

Of course we could have used
a = {...a, key:"default", anotherKey:"anotherDefault"}

But as a general case, this should work on any variable
let a;
a ||= "default";

instead of
let a;
a = a ||"default";

As much as I've personally wanted this, we really can't do this without it being added to ECMAScript itself, since adding syntactic features that aren't strictly on the type side doesn't fit into our design goals.


Allow the jobtype field to be overriden

Currently the worker module name is used for the jobtype field when building a job and sending it to Faktory. This works great when only using the Elixir Faktory client. However, when we want to push jobs from Elixir and process them in one of the other clients (i.e. python, ruby, go, etc...) we may need to set the jobtype to something specific to that client.
This update allows the jobtype field to be set in the following two ways.

When calling perform_async

MyWorker.perform_async(args, jobtype: "run_fn")

As options in the use when defining the worker module

defmodule MyWorker do
  use FaktoryWorker.Job,
    queue: "some_queue",
    jobtype: "run_fn"
end

Codecov Report

Merging #113 (9eef55b) into master (b023a49) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #113   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           27        27
  Lines          364       365    +1
=========================================
+ Hits           364       365    +1

Impacted Files
Coverage Δ

lib/faktory_worker/job.ex
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Looks good. Should this be added to the worker configuration docs? (it looks like at is missing from the options too)


Map colours on Firefox and Chrome

Describe the bug
Colours on Firefox and Chrome are vastly different, using the dark mode and, look ugly on Firefox.
To Reproduce
Steps to reproduce the behavior:
Visit the site using Firefox and Chrome.
Expected behavior
Colours look like they do on Chrome even on Firefox.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Windows
Browser Firefox and Chrome
Version Latest.

This also happens for active, recovered and deceased graphs
ACTIVE

RECOVERED

DECEASED

Seems to be happening post release of latest version of firefox (76.0), works fine for older versions

Was this fixed?

Was this fixed?

Not yet.

I can't seem to reproduce this. Are you sure that this not because of DarkReader?

I'm not sure. I still have the issue in Incognito with all addons disabled. @phoenicopteri was able to reproduce it.

Yes, this is still reproducing without DarkReader
Please note issue is only on firefox, chrome works fine

@phoenicopteri @AzimJaved It's working for me on Mac. Are you both on windows?

Yes this is on Windows 10 for me

Weird, it's working for me on Windows. Which version of Firefox are you guys on?

Windows 10, Firefox 76.0.1 for me.

Firefox 76.0.1, it worked fine in Firefox 75.x

I'm on the same version and it's working fine for me. Do you guys get any logs?

I did not get any logs.
This issue seems to be happening on devices with the default Intel GPU only. Did not hit it on a Windows 7 machine with additional NVIDIA GeForce on it

So, it probably has something to do with the SVG filters that we're using. Can you play around with those and see if changing anything fixes this?

Removing url(#balance-colour) makes it similar, although not exactly the same as on Chrome for me. Removing it from both Chrome and Firefox make the maps identical for me.

So that's a custom SVG filter that we've defined in https://github.com/covid19india/covid19india-react/blob/c030fa8f9d8b3bf83d34cc5c8cdaea7c498e3d8e/src/components/choropleth.js#L474-L486
Can you check if removing the position/height style attributes from this filter helps?
You'll find it here

Nope, no change.

The buttons below got shifted downwards a bit though.

Yeah, ignore that. Looks like the filter is not working properly. This could be related to the GPU thing mentioned earlier. I'll read up and see if I can find anything

Alright.

Can we use cross-browser testing tools like Comparium or LambdaTest?
I am also unable to replicate the bug, please share more details.
Details:
Firefox Version: Mozilla Firefox 77.0.1
OS: Ubuntu 18.04
Graphics Card: Intel Corporation HD Graphics 5500

i think its a browser bug that happens sometimes

Closing due to inactivity


Support option to specify buffer size

Currently, it uses ReadAll() to read from stdin, but that's dangerous and it may cause OOM.
Instead, we'd better directly stream it via HTTP. That way doesn't require to buffer at all.
https://github.com/nakabonne/pbgopy/blob/9e86a266f03feaa369b22e6933687c1a1e704e1e/commands/copy.go#L49

I would like to work on this but you talk about OOM. Wouldn't that still happen if you keep storing the streamed data in the cache? Or do you mean streaming directly to the "pasting device"? What would a perfect implementation be?

@thealamu Sorry for the confusion. It doesn't mean streaming directly to the "pasting device". It means to stream it to the pbgopy server. However, pbgopy supports end-to-end encryption, so I'm beginning to feel it's not appropriate.
iouti.ReadAll() continues to buffer the input from the STDIN until EOF appears. If there is incorrect input, it could eat up the memory on the "copying device". I'm trying to think of something else to do 🤔

I don't know, but maybe capping the number of bytes over the buffered read and dedicating a streaming endpoint for much larger data. With the streaming endpoint, another device must be pasting to avoid OOM on the "copying device". What do you think? Although, I am not sure how encryption fits in here yet.

Let's change it to fail if you try to copy more data than the specified size with the --size(MB) option. By default 500MB is given.

Okay, but that sounds like a 'quick fix' rather than a lasting solution?

Okay, please assign to me.

@thealamu Yes, I'll be happy with that. Your help is essential.

Can this be closed now?

Yes, resolved by https://github.com/nakabonne/pbgopy/pull/10


Refresh Recaptcha

Is there a command to refresh/reaload the recaptcha component?

I had the same problem last week and I solve this using location.reload(). So the will be reloaded and reset de catcha.

Just call the javascript API from recaptcha:
grecaptcha.reset();
It will default to first widget created...
https://developers.google.com/recaptcha/docs/display#js_api
Best Regards


%qiskit_backend_overview returns ZeroDivisionError

Information

Qiskit Terra version: 0.18.3
Python version: 3.8.10
Operating system: macOS Big Sur Version 11.6 (Though running in Quantum Lab also produces this error)

What is the current behavior?
Running the following code results in a ZeroDivisionError:
from qiskit import IBMQ
import qiskit.tools.jupyter
%matplotlib inline

%qiskit_backend_overview

This is the full stack trace returned:
---------------------------------------------------------------------------
ZeroDivisionError                         Traceback (most recent call last)
<ipython-input-5-e9e9700c904d> in <module>
      3 get_ipython().run_line_magic('matplotlib', 'inline')
      4
----> 5 get_ipython().run_line_magic('qiskit_backend_overview', '')

~/opt/anaconda3/envs/qiskit-env/lib/python3.7/site-packages/IPython/core/interactiveshell.py in run_line_magic(self, magic_name, line, _stack_depth)
   2305                 kwargs['local_ns'] = sys._getframe(stack_depth).f_locals
   2306             with self.builtin_trap:
-> 2307                 result = fn(*args, **kwargs)
   2308             return result
   2309

</Users/Matthew.Stypulkoski@ibm.com/opt/anaconda3/envs/qiskit-env/lib/python3.7/site-packages/decorator.py:decorator-gen-134> in qiskit_backend_overview(self, line)

~/opt/anaconda3/envs/qiskit-env/lib/python3.7/site-packages/IPython/core/magic.py in <lambda>(f, *a, **k)
    185     # but it's overkill for just that one bit of state.
    186     def magic_deco(arg):
--> 187         call = lambda f, *a, **k: f(*a, **k)
    188
    189         if callable(arg):

~/opt/anaconda3/envs/qiskit-env/lib/python3.7/site-packages/qiskit/tools/jupyter/backend_overview.py in qiskit_backend_overview(self, line)
     44         backend_title = widgets.HTML(value=_value, layout=widgets.Layout(margin="0px 0px 0px 0px"))
     45
---> 46         build_back_widgets = [backend_widget(b) for b in unique_hardware_backends]
     47
     48         _backends = []

~/opt/anaconda3/envs/qiskit-env/lib/python3.7/site-packages/qiskit/tools/jupyter/backend_overview.py in <listcomp>(.0)
     44         backend_title = widgets.HTML(value=_value, layout=widgets.Layout(margin="0px 0px 0px 0px"))
     45
---> 46         build_back_widgets = [backend_widget(b) for b in unique_hardware_backends]
     47
     48         _backends = []

~/opt/anaconda3/envs/qiskit-env/lib/python3.7/site-packages/qiskit/tools/jupyter/backend_overview.py in backend_widget(backend)
    200                             sum_cx_err += param["value"]
    201                             num_cx += 1
--> 202         avg_cx_err = round(sum_cx_err / (num_cx), 4)
    203
    204     cx_widget = widgets.HTML(value=f"<h5>{avg_cx_err}</h5>", layout=widgets.Layout())

ZeroDivisionError: division by zero

Steps to reproduce the problem
Run the code snippet above
What is the expected behavior?
The backend_overview widget should not error out and should return the expected backend overviews.

Thanks for the report.  Hopefully this should be fixed by #7260 - if you're able to test that out, it'd be great, because by default I don't have access to any of the problematic machines, and our testing infrastructure is a bit lacking in that area.

By the way (and this has no impact on the bug you're reporting), you may want to check your Jupyter environment is what you expect - you said your Python version is 3.8.10, but your packages are being imported from a Python 3.7 environment.  You may just have had the wrong environment activated, or you may not have Jupyter installed in the environment you meant, so when you run it, it pulls from a different environment.


[FEATURE]Add index administrative operations

Is your feature request related to a problem?
Today there is a lack of UI support for basic support functions such as Index CRUD operations, and users have to go through REST APIs and Dev console to change things. While this may work for tech savvy users, this does not cater to the variety of OpenSearch user base that spans across Business Analysts, Operators, Sys Admins
What solution would you like?
Provide a UI based mechanism for common index and data stream administrative operations.

CRUD operations on indexes.

Create, Delete, List Indexes
Open, Close Index
Shrink, Split, force_merge Index
Mappings - define/update mappings for an index
Basic Settings - replica count, apply/remove R/W block,
Advanced settings - Get, Update Settings, setup synonyms, etc.
Add, Update, Remove Aliases
Rollover an index

CRUD operations on Data streams (https://opensearch.org/docs/latest/opensearch/data-streams/)

Create, Delete Data Stream
Rollover Data Stream

CRUD operations on Index templates (https://opensearch.org/docs/latest/opensearch/index-templates/)

Create, Delete Index Template

What alternatives have you considered?
N/A
Do you have any additional context?
The function will be added to index management dashboard plugin first because it already has function to list existing index. It's straight forward to add CUD functions on top of it. Once we have the overall design for admin UI, we should be able to move this page to a new place on dashboard with minimum effort.

The feature should be created in ISM dashboard plugin repo


Support searching for file name.

I have many old gists. They only have file name and no description.
So please support searching for filename.
BTW, I wish to support searching for the content as an option.(Please tell me why you don't add this function)

@Akagi201  Thank you for the ticket:)
Searching for filename is a legitimate request, and I'll add this support in the next release.
Please refer to https://github.com/hackjutsu/Lepton/issues/82 for the reason why content search is not supported. In short, either GitHub Gist API need to provide support for remote content search, or we implement a local content search with huge performance compromise.

Done: https://github.com/hackjutsu/Lepton/commit/6df8467a5c139ab1547aa2249ab704de7ca3e000

@Akagi201 I can build an internal release(v1.1.2) for you. Which platform do you work on? macOS or windows?

@hackjutsu MacOS thanks. I will test it.

@Akagi201 https://share.weiyun.com/f2ceb73753c7e89154ca501ff234e31c

@hackjutsu The filename search function works, and searched very quickly.
Additionally, I hope when I jump to the searched file. The gist list can scroll to the according gist name.

@Akagi201
Just want to clarify. Do you mean the gist list should scroll to the pink box once you jump to the selected gist?

@hackjutsu Yes, That's it. It's like the feature in IDEAs.

Thanks for feedback:)
I created an issue ticket for that https://github.com/hackjutsu/Lepton/issues/111.


Feature request: The color of the title bar for Windows has some drawbacks after the latest fix

Is your feature request related to a problem? Please describe.
I see that now the color of the title bar matches the system theme. However, if a user wants to use a theme different from the system's, such as combining light with dark or dark with light, the title bar will still follow the system theme. This isn't ideal. It would be better if, when a system theme is selected, the title bar also matched that system theme. If a light theme is chosen, the title should also be light (ignoring the system theme). Similarly, if a dark theme is chosen, the title should be dark, even if the system theme is light.
Also, currently, when changing the system theme in Windows from dark to light or vice versa, the title bar in the app only updates to match the new theme after restarting the app (if it was open before). While this isn't critical, it would still be better if the title bar updated automatically to reflect the system theme change without requiring a restart.
Describe the solution you'd like
Described above.
Describe alternatives you've considered
No response
Additional context
No response

I would like to add that this behavior is also relevant on the Linux side, although it differs slightly.
Currently, I am using GNOME 47 (with stock light and dark theme) on Arch Linux with LocalSend Flatpak version 1.15.4 installed, and the LocalSend window decoration (title bar) does not match the system theme, even though the app theme does. Switching the system theme and then restarting the app, as described by @sergd88, does not update the window decoration either. Other Flatpak apps on my system switch the window decoration/app theme according to the system theme as expected.
The video below demonstrates the issue.
Gravação de tela de 2024-09-25 09-07-19.webm


"CONTACT" section name change to " CONNECT US "

As i have noticed in the contatct section,there a  "connect us" form where user can just fill that to add its valueable comments and message to the team.So,i would request to change its name to "CONNECT US"..it would add more value to header. PLEASE ASSIGN ME THIS ISSUE.

please assign me this , i wants to contribute under GSSOC'23

Please assign it to me under GSSOC'23.


Fixes 256 - ValueError mutable default

Fixes issue #256 (https://github.com/EleutherAI/elk/issues/256) . Error message:

ValueError: mutable default <class 'elk.training.train.Elicit'> for field run_template is not allowed: use
default_factory

 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.

If we need to change this for Python 3.11 I would prefer to just use functools.partial or something similar.


Access-Control-Max-Age is never honoured

I have a fetch with CORS, with some custom HTTP headers. Due to that, preflight OPTION fetch is required before the actual fetch.
In the return of that preflight, I have set the Access-Control-Max-Age (https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS). However, in the subsequence fetch, preflight OPTION fetch are still happening. In fact, no fetch happens without preflight.
I have checked the answers from http://stackoverflow.com/questions/23543719/cors-access-control-max-age-is-ignored, seems not related to my case.
Is it possible that within fetch module itself, the preflight fetch is always invoked regardless the Access-Control-Max-Age?

Solved. It turns out preflight cache is per URL, not per domain 😞 .
"In order to reduce the number of preflight requests, CORS has the concept of a preflight cache. However the preflight information is cached for an origin/url pair. That means each unique url has its own preflight cache. An API with the same preflight response across all urls will still receive a preflight request for each unique request."
http://monsur.hossa.in/2012/09/07/thoughts-on-the-cors-preflight-cache.html


Bug : Fix the deposit type URI in the dedup-sites API for nickel.

Reason: data from Inferlink is wrong (contain empty normalized_uri for deposit_type_candidate) - the validator did not run on those files.


added brijrajparmar27 porn hosts

added new hosts file into the stream
please refer to issue #1745

Thank you Brijrajsinh @brijrajparmar27, welcome aboard.
I may call on you occasionally to consider adding new domains that are submitted as issues.  I presume that's okay.
Merging!

Brijrajsinh @brijrajparmar27 you'll notice when you navigate to the readme for any version that contains the porn extension, for example the Unified hosts + porn page you are listed as a source.
Seven of our sixteen bundles include the porn extension.
Thanks again Brijrajsinh.


Applied filters bar + changes in simple/advanced filters

See - https://app.asana.com/0/1202525679663009/1202838947527265/f

I would like to arrange a meeting between me, @RomanMahola and you about the used data structure to save the filters. We believe that it can be simplified and be more developers-friendly

@elador13
May I close this PR now?


Modify PolarAxis

fix clockwise and counterclockwise results of GetCoordinates()
fixed #4310
rename RotationDegrees to Rotation and change type to Angle
refactor GetCoordinates() with PolarCoordinates
synchronizes Rotation with the angular direction of the Cartesian coordinate system
please refer to the following for differences

Default
var polarAxis = plot.Add.PolarAxis();

Rotating polar axis
polarAxis.Rotation = Angle.FromDegrees(90);

Before

After

Thanks!
This also changes fixes the existing cookbook recipe
double[,] values = {
    { 78,  83, 84, 76, 43 },
    { 100, 50, 70, 60, 90 }
};

var radar = myPlot.Add.Radar(values);

string[] spokeLabels = { "Wins", "Poles", "Podiums", "Points", "DNFs" };

before
after


 I use Fontello in almost every of my projects, and it has such a nice and comfortable GUI and this was really over-due. Thanks for making this and keep up the good work !
+1 for everyone who agrees !

Yes there is no "thank you page" so i congratulate you here, Fontello is an incroyable tool for the developpement Frontend. Since 2004 i says that SVG is the futur, with your tool SVG is now so powerful. I will give somes icons fonts, for the community certainly. Thank you .

thanks :)


Add support for horizontal bar charts

Purpose:
This PR adds support for horizontal bar charts in SP5.

Thanks @sghctoma, this looks great! This feature will be in the next release which I plan to publish today or tomorrow 🚀

Thanks @sghctoma, this looks great! This feature will be in the next release which I plan to publish today or tomorrow 🚀

Awesome, thank you!


Improve startup performance. Closes #703

I added a comment re nvm_ensure_version_installed performance in a separate commit; feel free to cherry-pick if you don't want it :)

@joliss ping :-)

Okay, Travis passes now. I had to remove some stuff; see my comment on outdated diff above.

The other issues probably require someone to dig a bit deeper and fix the performance issues properly; I wrote up some findings at #709, but don't want to dig deeper at this point, as I don't know the code well enough.

Thanks! I'll take a look at #709 as well.


assetLoaded event example

derived from the depreciated kontra asset loader example.

Coverage remained the same at 100.0% when pulling 811e4f2adbada4c7f448a7bcb70121e0d5ef2565 on wini3d:master into 5ec8789bcd03391aa3d5d8abfe21abb59fc12b63 on straker:master.

Nice! Glad to see this brought back

Thanks! it is a very handy feature. :)


[Question] FSK synchronization

I have a 4-FSK signal that i want to demodulate. It doesn't have a preamble or anything like that to assist with synchronization. The fskdem object requires perfect symbol timing. How can we realise symbol timing?

This might help: https://liquidsdr.org/doc/symtrack/

Does that work with FSK? Not sure

Yeah symtrack is only designed to work with modem objects which doesn't include FSK

I think you can still use symsync (and symtrack?). Just make sure to pass the samples through the Gaussian filter (LIQUID_FIRFILT_GMSKRX) after FM demodulation. Check this example: https://github.com/jgaeddert/liquid-dsp/blob/master/examples/gmsk_tracking_example.c#L62
Here's more info on what makes GMSK different regarding the matched filter: https://dsp.stackexchange.com/a/74319

GMSK doesn't just use a Gaussian filter instead of a raised-cosine filter. It uses a Gaussian filter on the phase, before applying it to the modulator. This makes it a nonlinear operation.
When a raised-cosine filter is applied to some modulated signal, it is applied after modulation, as a linear operation.

I've tried using symsync for FSK4 and it's not working well at all.

I'm using something similar to freqdem or GNURadio's Quadrature demod block to basically convert FSK4 to ASK4 then I need some kind of symbol synchronization and PLL. In this case the error in the amplitude equals to frequency error, which then needs to be fed to a PLL loop. So i don't think i can simply re-use an ASK4 demod algorithm.


⚠ Bofrost.fr has degraded performance

In 83ea3cc, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 945 ms

Resolved: Bofrost.fr performance has improved in ec41f54.


Can't continue this lesson

This lesson also

This is likely due to the change for biased terminology (moved from master to main). let me check...

Ah an error in setLocalToTrackRemote....

Thanks for the report @hamxn! Should be all fixed up now 🎉
I just pushed the gh-pages site but I would recommend a hard refresh

(There might be other bugs in remote tracking but this one specifically should be fixed)

The first one already fixed. Thanks you.
The second one seem to fix. Please check!

ah its probably a similar issue in the push side of argument parsing. justaec.
also attaching the level names helps a bunch, makes it easier to load it up :)

Btw all of these bugs are related to #751 which I think is a pretty important chunk of work; ideally I caught this with no errors but it's worth fleshing out the few issues to accomplish that task


Ansible request for Scaleway hosts - update sudo version 1.9.5p2 or higher

Please put the name of the software product (and affected platforms if relevant) in the title of this issue

[ ] Missing install
[ ] Bug in ansible playbook
[X] Request for new playbook addition

Details:
Scaleway recommends we update Sudo to the version above.

I believe this relates to https://ubuntu.com/security/notices/USN-4705-1 which is fixed in Ubuntu's sudo version 1.8.16-0ubuntu1.10 which is the one we have on our machines, so no further updates should be needed. Unless there is a belief that this is incorrect (in which case re-open) I'm closing this.


Add Hits-of-Code badge

https://hitsofcode.com/

[![Hits-of-Code](https://hitsofcode.com/github/thomasborgen/storage-bucket)](https://hitsofcode.com/view/github/thomasborgen/storage-bucket)


Bug report-Datalake container cannot be added

Is there an existing issue for this?

[X] I have searched the existing issues

Community Note

Please vote on this issue by adding a :thumbsup: reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Version of the module you are using
v0.0.5
Rover Version
No response
Terraform Version
No response
AzureRM Provider Version
No response
Affected Resource(s)/Data Source(s)
stsga3tmiareporting
Terraform Configuration Files
storage_accounts = {
  datalake1 = {
    access_tier              = "Hot"       # Optional - If not specified defaults to Hot
    account_kind             = "StorageV2" # Optional - If not specified defaults to StorageV2
    account_replication_type = "ZRS"
    account_tier             = "Standard"
    is_hns_enabled           = true
    min_tls_version          = "TLS1_2"
    name                     = "stsga3tmiareporting"
    resource_group_key       = "solution1_region1"

    #### Below block errors in pipeline run, container created manually in Portal
     containers = {
       container1 = {
         name = "landingzone"
       }
     }

Expected Behaviour
A container should be created
Actual Behaviour
Container fails to be created with error:
Error: containers.Client#GetProperties: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="AuthorizationFailure" Message="This request is not authorized to perform this operation.\nRequestId:a78960b2-701e-000b-4813-88c411000000\nTime:2023-05-16T16:29:14.4083162Z"
Further investigation shows that a data_lake_filesystems block should be used instead. But this also errors:
Error: checking for existence of existing File System "landingzone2" (Account "stsga3pdltest"): datalakestore.Client#GetProperties: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: error response cannot be parsed: "" error: EOF
Steps to Reproduce
No response
Important Factoids
No response
References
No response

This appears to be due to network permissions. If a network block exists with bypass = ["AzureServices"] then the container/data_lake_filesystem fails, UNLESS a complete list of uksouth and ukwest IP addresses are added as ip_rules. Then either a container or data_lake_filesystem block can be used


Update README.md

Why if you give an ES6 example just not to use literal templates, which are from ES6 too?

At the moment of writing, I tried to avoid mixing multiple notions to avoid confusing learners.
But since I use a template string right after and that this notion is explained way above, I think I can safely agree with you and approve this PR ! 👍
Thanks !


bug:  min. temperature values changed in 2.21 IronOS

To Reproduce
Steps to reproduce the behaviour:

2.21 IronOS made changes to  the temp  min/maxes due to a typo or conversion error from a long time ago that was only recently noticed.

Turns out the min temp of 60F was incorrect for all  irons  since 10C minimum = 50F and not 60F.   but we only care about pinecil here.

I will submit pr fix  if I have time, but  whoever gets to this first is welcome to fix it and close this ticket. literally this ticket prob took me 100 times longer to write than changing the  60F to a 50F   >.<

This file needs to be updated:
https://github.com/builder555/PineSAM/blob/master/backend/pinecil_setting_limits.py
Screenshots

Original IronOS Issue ticket
Error was reported and I submitted a fix for it here: https://github.com/Ralim/IronOS/issues/1624
Reference this IronOS code file here:
https://github.com/Ralim/IronOS/blob/0a4560efac961e5f5fbe47d5f9d7beb78dc77d4a/source/Core/BSP/Pinecilv2/configuration.h#L132-L137

PineSAM Version: all
IronOS build/date: 2.21

fix in last update.


Issues compiling on OSX Sierra - OpenBLAS by default installs gcc6

Installed OSX Sierra. Tried to build libnd4j, this is the result: https://gist.github.com/crockpotveggies/a4fea0e8cb89aa9c9373f087f5115c9d
I performed a brew install openblas and it installed gcc6 by default. gcc5 will compile just fine, but then there's an issue loading OpenBLAS:
Caused by: java.lang.UnsatisfiedLinkError: /Users/justin/.javacpp/cache/nd4j-native-0.6.1-SNAPSHOT-macosx-x86_64.jar/org/nd4j/nativeblas/macosx-x86_64/libjnind4j.dylib: dlopen(/Users/justin/.javacpp/cache/nd4j-native-0.6.1-SNAPSHOT-macosx-x86_64.jar/org/nd4j/nativeblas/macosx-x86_64/libjnind4j.dylib, 1): Library not loaded: /usr/local/opt/gcc/lib/gcc/5/libgfortran.3.dylib
  Referenced from: /usr/local/opt/openblas/lib/libopenblasp-r0.2.15.dylib
  Reason: image not found
	at java.lang.ClassLoader$NativeLibrary.load(Native Method)
	at java.lang.ClassLoader.loadLibrary0(ClassLoader.java:1937)
	at java.lang.ClassLoader.loadLibrary(ClassLoader.java:1822)
	at java.lang.Runtime.load0(Runtime.java:809)
	at java.lang.System.load(System.java:1086)
	at org.bytedeco.javacpp.Loader.loadLibrary(Loader.java:767)

Note that I manually checked and all of the files above are present.

Try to build OpenBLAS from source, with this script for example:
https://github.com/bytedeco/javacpp-presets/blob/master/openblas/cppbuild.sh
Or don't use OpenBLAS if you won't need it.

It's also possible to compile libnd4j from source with gcc6 :)

gcc6 on mac complaints about missing AVX-512 instructions for some reason :(

libnd4j no longer needs BLAS. ND4J now depends on the build from the JavaCPP Presets for that.


feat(toxicity_measure): Toxicity Extractor

The commits from this PR moved to https://github.com/superwise-ai/elemeta/pull/48


feature(Scanner): Implement first screens for scanner

4-implement-first-screens-for-scanner

Please include screenshots in PRs, it's a lot easier to quickly notice potential issues without having to run the code first.

This does not implement the task correctly. The task says there are two screens:
First:

Second:

As far as I can see only the second one is implemented.

You have not set up pre-commit locally as per instructions, as the validation pipeline errors: https://github.com/ioxiocom/ioxio-tags/actions/runs/6265956028/job/17015897628?pr=2

Oh right, and after installing pre-commit you will need to run pre-commit run --all-files to automatically fix most of the issues it will complain about :smile:

The task specifies there should be a developer flag const DEVELOPMENT = true or something, that we can later disable easily, which leaves the previous "Simulate scan of preset data" -functionality in as a separate button next to "Turn on".

The task specifies that instead of just having console.log for the results, there should be a simple temporary dialog on e.g. the bottom of the screen that shows the result of the scan (including simulated) and goes away after a few seconds.

Unfortunately it seems GitHub closed the PR without asking me after a manual merge of the WIP branch. Please take the comments from here into account while creating a new PR :smile:


feat: add patch contributor feature

implement patch contributor route and logic (stores in DB if not there yet)
remove mock implementation of find contributor
foreign key contributor_id not set yet

Comments welcome

Codecov Report

Merging #151 (1b2e768) into main (1eb8335) will decrease coverage by 2.27%.
 Consider uploading reports for the commit 8b2f4ee to get more accurate results

@@            Coverage Diff             @@
##             main     #151      +/-   ##
==========================================
- Coverage   54.70%   52.42%   -2.28%
==========================================
  Files          82       85       +3
  Lines        2210     2306      +96
==========================================
  Hits         1209     1209
- Misses       1001     1097      +96

Impacted Files
Coverage Δ

src/application/contributor/new.rs
0.00% <0.00%> (ø)

src/application/contributor/update.rs
0.00% <0.00%> (ø)

src/domain/repositories/contributor.rs
100.00% <ø> (ø)

...ure/database/domain_implementations/contributor.rs
0.00% <0.00%> (ø)

src/infrastructure/database/schema.rs
0.00% <ø> (ø)

src/main.rs
0.00% <0.00%> (ø)

src/routes/contributors/patch.rs
0.00% <0.00%> (ø)

src/routes/dto/to_http_api_problem.rs
27.71% <0.00%> (-2.16%)
:arrow_down:

src/routes/dto/u256/mod.rs
38.29% <0.00%> (-1.71%)
:arrow_down:

src/routes/mod.rs
0.00% <ø> (ø)

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Scripts for dedup and filter Common Crawl?

Hi,
I notice that the download URL for the CommonCrawlDataset is http://eaidata.bmk.sh/data/pile_cc_filtered_deduped.jsonl.zst. In other words, this CC dataset is already deduplicated and filtered? However, it doesn't seem like https://github.com/leogao2/commoncrawl_downloader in the README included the scripts for deduplication and filtering. I'm wondering where I can find out exactly how deduplication and filtering for Pile CC is done?
Thanks!

Additional question: it seems like the_pile/pile.py only downloads and interleave the data from various data sources. processing_scripts contains many processing scripts, however, how do we know which script is supposed to be run on which data  source, and how those scripts are supposed to be run?


Does not support current JupyterLab version

Reproduce

Open JupyterLab v3.5.2
Go to Extension Manager
Search for and click Install for jupyterlab-favorites
See error: does not support current version

Context

Python package version:

Extension version:

Operating System and its version: Windows 10 WSL 2
Browser and its version: Chrome 108.0.5359.125 (Official Build) (64-bit)

Nevermind, I was confused, mixing up extensions I was testing, I think.


Spurious versioning.

Latest Version downloaded from Puppet Forge has file name:
liamjbennett-windows_firewall-999.999.999.tar.gz

This is not an issue with the module. Version 999.999.999 was released under the previous namespace to make it's depreciation clear.
The latest version of the module can be found here: https://forge.puppetlabs.com/puppet/windows_firewall


Update Service fixture

Coverage remained the same at 100.0% when pulling 0c41d99403c0d37729abdc8a72b81a6327047b54 on update-service-fixtures into 4d3cda604304e01cfd06473b86acaeaece4025ea on master.


Add ability to link a federated id with the UpdateUser() method.

Note the diffbase. I'll merge+retarget to dev once the bulk_get stuff lands.