Update sbt-microsites to 1.3.0

Updates com.47deg:sbt-microsites from 1.2.1 to 1.3.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.47deg", artifactId = "sbt-microsites" } ]

labels: sbt-plugin-update, semver-minor

Codecov Report

Merging #149 (33eaf17) into master (cee719a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   71.59%   71.59%
=======================================
  Files          12       12
  Lines         426      426
  Branches       21       21
=======================================
  Hits          305      305
  Misses        121      121

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 cee719a...33eaf17. Read the comment docs.


Fixed: removed support for BASE_URL entry from the app

(#234)

Closes #
Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

Contribution and Currently Important Rules Acceptance

[ ] I have read and followed contribution rules

Closing as we have handled this in the upcoming version of app.


Fix hardcoded paths

Closes #12

It's been over a year -- is there anything blocking this from being merged?

So sorry! I think it just slipped my mind!


fix: allow hyphens in parameter names

Fixes #976:
openapi-python-client fails to generate endpoints and throws incorrect path templating warnings when the path has a parameter with a hyphen.

Sorry, was too quick to suggest the PR: after testing, it needs more work, as the hyphenated parameter names are identified, but not correctly replaced with their python_name in the generated code. Please leave it open, unless you prefer that I make a new PR.

Closed in favour of #986 and #987.


Allow Users to Specify Trained Slice for Model Runs

Allow users to specify the slice id which was used to train a model.

@vayunalapati remember to bump the version before merging!


CSAT Host ASP.NET Core on Windows with IIS

URL

Since the topic hasn't changed much but the score has been steadily falling from 70-75 range, have you been able to draw a conclusion as to why the score has fallen so much?

I agree with your assessment when you wrote:

The topic hasn't hardly changed. The audience might be changing as a more diverse group of devs and devs with no Core experience are coming in. ... Anyway ... splitting off the Troubleshooting into a new standalone topic might be helpful.

Xref https://github.com/aspnet/Docs/issues/4685 ... if diagnosing startup errors can be addressed (the IIS portion of that issue) at the same time.


Incorrect result for "test.jpg"

which file you executed? (classifier_2_v2.py or classifier_5_v2.py)

classifier_2_v2.py
Checked in Ubuntu, got the result as below:

So, no issue on it now.
But getting the wrong result in windows as displayed in the last comment.
How can we classify the grains in 5 classes using "classifier_5_v2.py", as the main file "cmd_wheat_quality_detector_v2" is classifying grain and not grain ?

you have to make changes in "cmd_wheat_quality_detector_v2"
at line 20:
"model = keras.models.load_model('weights_results_2out/weights_01234567.pkl')"
by
"model = keras.models.load_model('weights_results_5out/weights_01234567.pkl')"
.

If your issue is resolved Can you please close it.


Bug

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

### Tasks
- [ ] https://github.com/microsoft/vscode/issues/222295
- [ ] https://github.com/microsoft/vscode/issues/222288

/needsMoreInfo

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!


Starting Allsky via WebUI does not work

I tried to stop and start allsky by web browser, instead of the "reboot pi" but that gives me no image, but a screen with
Error
Allsky Stopped!

RPi camera command not found!

Originally posted by @zanco in https://github.com/AllskyTeam/allsky/discussions/3970#discussioncomment-11362833

Can you reproduce this?
If so, please do the following:
sudo systemctl stop allsky
sudo truncate -s 0 /var/log/allsky.log

Then in your browser press F12 and on the right side go to the Console tab.  Press the "Start Allsky" button.
Attach /var/log/allsky.log and a screenshot of any lines in the Console tab.

The next release of Allsky will be out December 6, and I have successfully used that button several times.  In the meantime you can start it via the command line:~sudo systemctl start allsky~Just to make sure the command exists on your Pi, please run:~~~which libcamera-still ~~~From: Ben aka PE2BZ @.>Sent: Monday, November 25, 2024 2:14:34 AMTo: AllskyTeam/allsky @.>Cc: Eric Claeys @.>; Author @.>Subject: Re: [AllskyTeam/allsky] Starting Allsky via WebUI does not work (Issue #3978)Hi, can reproduce. Followed instructions, Console has no output lines when pressing "Start Allsky"Output of allsky.log:2024-11-25T09:10:19.136200+01:00 allsky allsky[9591]:      ***** Starting AllSky ***** 2024-11-25T09:10:20.961501+01:00 allsky allsky[9647]: Can't determine what command to use for RPi camera. 2024-11-25T09:11:00.823904+01:00 allsky allsky[9757]:      ***** Starting AllSky ***** 2024-11-25T09:11:02.639374+01:00 allsky allsky[9813]: Can't determine what command to use for RPi camera.Tried once, then stopped again, and tried second time. Pasted as text because only 4 lines where in the log.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.

@zanco, Ben,
Have you been able to upgrade to v2024.12.06 and try the buttons?


Rendering Multiple Markers

May you please give an example on how to render multiple markers with infoWindows? I have tried but it does not want to render any marker with infowindow wrapped around a div. It only works if I render a collection of a single component, that is, a collection of just markers. But if I extend this and add infoWindows too, it won't render anything.
This is my Map Component:
     <Map google={this.props.google}
            zoom={16}
            initialCenter={{
            lat: -26.107567,
            lng: 28.056702
          }}
      enableEventPropagation={true}
      mapTypeControl={false}
      streetViewControl={false}

      disableDefaultUI={true}
      centerAroundCurrentLocation={false}
      onClick={this.onMapClick}
      >
      {this.state.markerCollection}
      <Search setMarkersData={this.setMarkers} updateMarkersData={this.updateMarkers}/>
      </Map>

The state has a marker collection which is simply an array of divs with Marker and infoWindow components inside. I also get this warning: Warning: Unknown props `map`, `google`, `mapCenter` on <div> tag. Remove these

Don't have enough information for me to debug the code, please provide a minimal reproducible repository so that others can help.

I understand this is frustrated, but this is not a support form on how to use Google Maps API. react-google-maps is simply a wrapper around it. Unless there's a specific issue on the react-google-maps itself, please post your questions on StackOverflow instead of creating an issue here.
The maintainers only have limited time on open source projects. Please help us keep the health ecosystem. Thanks!

Hi can you please tell how to use initial center and two markers at a time on the map


Remove Scotland and Wales from Employment visualisations

Ideally to be done by Thursday 5th October
After feedback from YFF - they're not interested in data from Scotland and Wales as they are England focussed so let's remove this data from the hexmaps on the Employment page.
https://data.youthfuturesfoundation.org/maps/employment/

Filtering to keep only geography code starting with 'E'

Thanks Giles - closing as completed


Xcode 9 template changes

Hi Vokal team,
I follow your work to understand Xcode templates in our team since 3 months. Good point to had open your work 👍
Did you test on the first Xcode9-beta version ?
Now Xcode links folders with finder by default, so it produces a lot of group duplications when using  your Vokal templates for instance.

I am currently testing and adjusting your templates to see which changes and impacts we have to do.

Thanks for pointing this out 👍 . We have not had a chance to test the template against the new Xcode 9 beta yet but we may create a separate Xcode 9 branch for this purpose.

This is now fixed


Standardize  analytics nomenclature

Related to https://github.com/NebraLtd/hm-dashboard/issues/1084
source_scope-action
app_hotspot-onboarded

sources:  app
scopes: home, hotspot, support, dashboard
actions: new, opened, location_asserted,  transferred, antenna_updated, address_copied

Acceptance Criteria

[x] The standard name scheme is source.scope_subscope.action
[x] Actions: For every action button and page view

@marvinmarnold https://github.com/NebraLtd/hm-dashboard/issues/1244 is a separate issue.

You are right, thanks for pointing that out.

App v1.0.6 submitted to both stores


don't attempt to merge table cell values

while we want to merge the props it doesn't make sense to merge values, particularly when they are React elements. merging values that are React elements will generally lead to a browser hang / crash in non-trivial examples. this is easily observed by running the demo locally on v.0.11.1.

Thanks a lot! Fixed in 0.11.2.


Loading z-index is higher than slideout panel

Looks weird. The panel should be on top.

@robbieaverill can we close this based on the interaction on #475?

Nah it still needs to be resolved. This screenshot is an example of standard component use from the UI library and it shouldn't overlap like this. I closed #475 because there was a difference of opinion on how to fix this, but haven't spent any time on investigating whether we can remove the z-index from the Loading component yet. I think that's the way forward though - let the implementor decide what z-indexes will work for each use case (if they need to at all)

Ultimately we should probably have a documented z-index strategy. Different levels should have different purposes. For example:

z-0 (none): Normal content
z-10: Sectional loading indicators
z-20: Panels and modals
z-30: Toasts and banner alerts
z-40: Full page loading indicators

We could even rename the z- options in tailwind to be clear on the use of each z index, so you'd do something like @apply z-modal for modals.

As Robbie mentioned, sectional loading indicators shouldn't actually have z-index, but it might be a bit of a job to do that, as then the loading indicator MUST appear after the content it overlays in the DOM.


Find References doesn't find lazy loaded React components

Does this issue occur when all extensions are disabled?: Yes
Does this issue occur in the Insiders build?: Yes
VS Code Version: 1.71.2
OS Version: MacOS 12.5.1 (MacBook Pro 2019)
The problem is that Find References doesn't find React components that are lazy loaded.
Steps to Reproduce:

Export a React component.

Lazy load it somewhere.

Check that VS Code doesn't find any reference for the component.

Not sure if it's a bug or intended behavior (probably intended), but nonetheless it would be nice that finding references would also find lazy-loaded ones. This forces me to always use standard search (with all the downsides of it) to see if I've used the Component somewhere that references can't find.

Thanks @buondevid
This is not related to react. Minimal example:
// a.ts
export default function foo() { }

// other.ts
const a = import('./a');

Find all references doesn't pick up the default export referenced through  the import extension

I have the same problem in js file.

Thanks @buondevid
This is not related to react. Minimal example:
// a.ts
export default function foo() { }

// other.ts
const a = import('./a');

Find all references doesn't pick up the default export referenced through the import extension

Only if the import has that format. The following is working for me:
import a from './a';

See repro https://stackblitz.com/edit/react-ts-wtgage?file=App.tsx

Only if the import has that format. The following is working for me:
import a from './a';

See repro https://stackblitz.com/edit/react-ts-wtgage?file=App.tsx

But
import a from './a';

is not a lazy load import, isn'it? The issue is on lazy-load imports so import().

const a = import('./a'); is not a reference to the default export.

Well my own example was with react components, shouldn't VSCode support finding references for lazy loaded ones? @RyanCavanaugh
I'm gonna test tomorrow if it find references for a.default as in @mjbvz example.

@mjbvz hey, they closed the issue here but the problem I opened the ticket for is actually not solved, could we move this back in VSCode? references still doesn't find React lazy-loaded components

Not really. This is the expected behavior because there's no programmatic way to know that React.lazy's behavior is to unwrap default exports. It's the same as if you had written
// a.ts
export function foo() { }
export function bar() { }

const p = blah(() => import("./a"));

Is the blah call a reference to foo, bar, or neither? Who can say?

why close this issue, the problem is not resolved!

@RyanCavanaugh I understand your point: from a JS point of view the lazy React abstraction hides away the reference I guess. My ticket was more: is there some VSCode can do to recognize that and start supporting it? Or it's not VSCode duty because it's a React-specific thing?

VS Code asks TypeScript for the references, so the discussion best belongs here.
Unless TypeScript starts hardcoding the behavior of various library functions, which isn't something we're keen to do, there's not a feasible way to do this.

VS Code asks TypeScript for the references, so the discussion best belongs here.
Unless TypeScript starts hardcoding the behavior of various library functions, which isn't something we're keen to do, there's not a feasible way to do this.

I see, however, I notice that the lazy loaded component knows about the props on the component definition:

So somehow TS knows about the original component definition and their connection (probably through a generic in the React types), but it doesn't enter the references tho... maybe TS could implement a low-level abstraction that Libraries like React could implement to make a step further and say "this is a reference to that function"? Just reasoning out loud 😄

This problem has troubled me for a long time

This problem has troubled me for a long time

It makes refactoring so much harder

Is there any solution?

So should it be a cake of Vscode? OR Javascript? OR React?
Maybe we the React.lazy should be improved........or abandoned?

Found a workaround.
Example
// do named export
export const MyComponent = () => <>hello</>;

// import using `lazily`

import { lazily } from "react-lazily";
const { MyComponent } = lazily( // right click and 'Find all references' on this works
  () => import("path_to_file")
);

Found by pure accident.
if u're OK with doing this ritualistic named exports for sake of DX.


Restriction on reused modules might be too strict

It seems that the reuse of module is banned by THOP tools
https://github.com/Lyken17/pytorch-OpCounter/blob/1f4ddb7fb51c3b1a49d60708f9b857535e5dc4e1/thop/profile.py#L49-L51
However, it is not quite reasonable. The re-use of ReLU-type module is very common, it can make your codes neater and causes no harm. e.g.
class Model(nn.Module):
  def __init__(self):
    super().__init__()
    self.dropout = nn.Dropout(0.5)
    self.relu = nn.ReLU()
    self.lin1 = nn.Linear(4096, 4096)
    self.lin2 = nn.Linear(4096, 4096)
    self.lin3 = nn.Linear(4096, 4096)
  def forward(self, x):
    output = self.relu(self.lin1(x))
    output = self.relu(self.lin2(output))
    output = self.dropout(self.relu(self.lin3(output)))
    ....

Besides, I think the network legality check is not THOP's work to do. This kind of restriction is unnecessary and will cause troublesome Error raise as I can see.

And if you truly want to show your attitude toward module re-use that much, warnings is more recommended as it won't interrupt the program. 😄
By raising it you're generating error, rather than real warning.

Question, does the example code you provide trigger the warning? Since though ReLU is forwarded several times, the registration should only happen once.

@Lyken17 Sorry that I didn't dig deep into your code's logic, see the following NN design instead.
class SomeNet(nn.Module):
    def __init__(self, reduction=4):
        super(SomeNet, self).__init__()
        self.activation = nn.LeakyReLU(negative_slope=0.3, inplace=True)

        self.encoder_feature = nn.Sequential(OrderedDict([
            ("conv3x3_bn", conv3x3_bn(in_channel, 2)),
            ("activation", self.activation),
        ]))
        ...

The problem has nothing to do with module reuse in forward function. THOP will recursively enter each leaf nn.Module and detect if there exist any "re-defined" module.
However, the aforementioned re-defined is legal, since I want to define a globally same activation function and plant it in many sub-module. And THOP stop me from doing that.
Like I said, even my definition of network is wrong, it is not THOP's job to point it out. And even if THOP would like to point it out, warnings rather than Raise Error should be used.
As a flops counting tools, it should not do extra judgement or security check. A correct flops and params output is the only thing to focus on.

It is a weird usage, I have to say. For your code example, each activation should have its own nn.Module. The proper case should be a recurrent structure where one layer is used in different depth. I have relaxed the check: now it only prints the warning instead of raises it.

Yet a legal one and a convinient one, too. Still unable to get your points of checking user's network...


Support descriptions for params

Support descriptions for params
Feature Request
It would be nice to be able to support adding descriptions (like helper text) to example params.  Currently we do this via the notes section but that involves toggling between the 2 tabs. Not terrible but could be better.
Since @param is a normal YARDoc tag but Lookbook seems to be using it in a slightly different way I'm sure this feature is much more difficult to implement than it sounds but I'm happy to help out with this if you'd like.

Hey @DRBragg yes that is a nice idea, being able to add a description/helper text to the params that shows up in the same panel would definitely be useful.
I'll have a think about how best to implement this, the @param tag is already fairly overloaded but I'm sure there is a way to get a description in there without the syntax being too messy!
Another possibility would to add an @param_info tag (or something like that) but it would probably be better to keep it within the @param tag itself if it doesn't get too messy.

Yea, it may end up being a breaking change based on the current implementation. @param_info isn't a bad idea tho.

@DRBragg I've just shipped v1.1.0 that has support for @param descriptions (plus lots of other @param improvements!) so I'm going to close this issue down now. Let me know what you think if you get a chance to have a play :-)


Duplicate H1s Titled 'Azure Service client library for JavaScript - Version 3.0.0'

The following pages in Docs are all using the same H1 title heading 'Azure Service client library for JavaScript - Version 3.0.0'. First level headings must be unique within a docset.
https://github.com/MicrosoftDocs/azure-docs-sdk-node/blob/main/docs-ref-services/latest/arm-avs-readme.md (13,1)
https://github.com/MicrosoftDocs/azure-docs-sdk-node/blob/maindocs-ref-services/latest/arm-vmwarecloudsimple-readme.md (11,1)
These appear to come from: sdk/avs/arm-avs/README.md, and sdk/vmwarecloudsimple/arm-vmwarecloudsimple/README.md

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.17418817,Docs:0.0584932,Service Bus:0.056028176'

@sima-zhu The links have been fixed in the issue description.

Consolidating into #22197


How to run websocket server automatically when http server start

Im using ratchet in a restaurant food ordering system,
is there any way without running socket server by cmd or terminal ???

You can run it using supervisor or systemd which launches it in the background and keeps it running

User supervisor and setup as service.


Remove bitfields

Hello,
I'm trying to write a wrapper for mruby that would allow you to embed it into a zig program.
Unfortunately because many of the structs in mruby use bitfields, the structs can only be opaque types. This is a huge limit since it means you can't access any fields on mrb_state or many other important structs.
Would it be possible to remove the usage of bitfields in the mruby codebase?
Thank you,
Dante

I've been able to work around this by writing C wrapper functions

Removing bitfields are too much pain for us. I close this issue since you have a workaround.


Extent log() tests to 64-bit

Hi Matt (@mborland) as we chatted offline, I'm in the process of identifying the biggest test blocks and reducing their depths when using sanitize/debug. I missed a few on the previous run, so let's see if this recent push makes the time cut or still times out, ...


Get correct version by git describe

When the pdc module is imported into another git repo, the version
reported is the version from that repo, not from PDC.

@erichuanggit I will update the version. Since the first tag I create was not annotated, I also created a new test release with annotated tag.

:+1:


events.playerInteract not firing when right clicking air

events.playerInteract doesn't seem to fire when right clicking air, despite bukkit behaving like this by default.
I am aware the client does not send any packets when right clicking air while not holding anything, but it doesn't fire despite holding items.
EDIT: May be related to the fact playerInteractEvent is cancelled by default, should still allow us to do stuff with it. I'm not a java developer, so I wouldn't know.

I've learned a lot about the Bukkit API since opening this, and I managed to write my own version of your events.on function which does not have this issue:
exports.listen = function(eventType, func, priority) {
	var scPlugin = scPlugin || server.getPluginManager().getPlugin('scriptcraft');
	var RegisteredListener = org.bukkit.plugin.RegisteredListener;
	var EventExecutor = org.bukkit.plugin.EventExecutor;
	var EventPriority = org.bukkit.event.EventPriority;

	priority = EventPriority[ priority || 'NORMAL' ];

	// We have to use this method of getting the .getHandlerList
	// method because nashorn is dumb.
	var eventClass = eventType['class'];
	var handlerList = eventClass.getMethod('getHandlerList').invoke(null);

	// This map is used for assigning extra variables to the
	// listener's execute() function's `this` scope.
	var extraBinds = {};

	// Construct the lsitener's executor.
	var executor = new EventExecutor({
		execute: function(plugin, e) {

			// Let's create a function for safely canceling any
			// cancel-able event
			extraBinds.cancel = function() {
				if(e instanceof org.bukkit.event.Cancellable) e.setCancelled(true);
			};

			// We'll assign this map to `func`'s `this` scope
			var bind = {};
			// Append extra variables from outside this scope
			// (currently just this.unregister & this.cancel)
			for(var key in extraBinds) bind[key] = extraBinds[key];

			func.call(bind, e);
		}
	});

	var listener = new RegisteredListener(scPlugin, executor, priority, scPlugin, false);
	handlerList.register(listener);

	// Pass an unregister event to the listener
	extraBinds.unregister = function() {
		handlerList.unregister(listener);
	};

	return extraBinds;
};

Seems like the trick was your version was passing true as the last argument to the org.bukkit.plugin.RegisteredListener constructor. I'll admit I don't know exactly what this means, so I'm not sure if this should be changed in master as well, I'll let you decide that.


feat(stdlib,interp): Implement autoboxing and roString (ifStringOps only)

roString — specifically the methods described in ifStringOps — is one of the most often-used non-SceneGraph types in the BrightScript language.  It's not necessarily directly created by BrightScript authors, but it gets created whenever foo.split("/") or bar.encodeUriComponent() are called due to the reference implementation's automatic boxing rules.  Basically, if you use a boxable primitive like an object, it'll get automatically boxed for you, and the temporary boxed value will be destroyed at the end of the operation.
This PR implements the ifStringOps functions on a new roString type, and includes that automatic boxing behavior.

Ignore the commit history; it'll get squashed


Peer to peer code review

Great job👍👍😊😊 adding the live demo. No issues raised at this point.

Your work looks amazing. No issues found


test: adjust memory limit for macOS

This test result 1332 on mac with -c opt. Also fixed typo.
Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>

/azp run


Out of date documentation

The documentation for .association, more specifically it's options parameter, says:
:blueprint (Symbol) — Required. Use this to specify the blueprint to use for the associated object.

I think it's missing the fact that one can pass a lambda to dynamically evaluate the blueprint to use.
Link to the doc: https://www.rubydoc.info/gems/blueprinter/Blueprinter/Base#association-class_method
I was lucky to look at old issues, and see that a PR had been merged for this feature that I needed.
Let me know if I missed something.

Thank you for bringing this to our attention, Robin.

You do indeed, in Supporting Dynamic Blueprints For Associations! I missed that. I focused on the doc only.

No problem -- We could stand to clarify this a bit more. Thanks for contributing!

Good callout, we'll update that portion of the documentation and leave this issue open until that's resolved.  Thanks!

If I could piggy back on this issue, the render_as_hash documentation section says its the same as render which implies that its an alias but they return different things. Maybe some clarification or just remove the "Same as render" part or reword it differently. This also goes for the render_as_json documentation.
Speaking of the render method, do you think that maybe it would be better to have something like:
# Could name the option 'adapter' or 'style', something to that effect.
PostBluePrinter.render(post, adapter: :json)
PostBluePrinter.render(post, style: :hash)

Adding it as an option instead of creating three different render methods seems more "coding best practice" to me. If this is something even worth considering, let me know and I'll create a new issue for this and even try to do a PR for it as this is a "found a problem, create the solution" kind of thing.


react hooks: add base interfaces Listenable, StatusParmas and ChatStatusResponse

Context
Adds interfaces as they are defined in CHADR-030. Closes CHA-383. They will be used for future hooks.
Checklist

[x] QA'd by the author.
[ ] Unit tests created (if applicable).
[ ] Integration tests created (if applicable).
[x] Follow coding style guidelines found here.
[ ] TypeDoc updated (if applicable).
[ ] (Optional) Update documentation for new features.
[ ] Browser tests created (if applicable).
[ ] In repo demo app updated (if applicable).

Testing Instructions (Optional)
n/a

@AndyTWF done both

Approved (pending the fix in Stevens PR and rebase)

Does anyone know why the test coverage fails here? There are no new functions added, just interfaces and their imports.
cc @AndyTWF @splindsay-92

Does anyone know why the test coverage fails here? There are no new functions added, just interfaces and their imports.
cc @AndyTWF @splindsay-92

I've noticed that for whatever reason, v8 isn't detecting that some files are just declarations. We might need to just adjust the thresholds (as soon as we start adding more hooks they'll go back up again)


welcome screen latest design

new image, different buttons in main section, different links in footer

let's


feat(client): generate challenge data as static files

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Ref - https://github.com/freeCodeCamp/freeCodeCamp/pull/50355#issuecomment-1544400365
Instead of relying on the generate page data by Gatsby, I'm generating the challenge data and storing them as individual files in the clients static folder so that mobile app can make use of them.

This isn't necessarily blocking, and if you can work with the weird directories that's fine.
But it looks like the 2022/responsive-web-design is being parsed as a nested directory:

Yes, we're fine with this and actually I prefer it this way


Trouble running the example

Hi, and congratulations for the software. I want to give a try, and I managed to successfully install it (the conda environment and the step-by-step process is much appreciated, but there are few inaccuracies in the readme, such as FIND instead of find when running, or where is install.sh or the folder where the compressed files for the external software has to be downloaded).
When running the example as it's written in readme, I'm getting the following errors, like it's missing some files:

mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_0.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_1.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_2.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_3.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_4.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_5.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_6.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_7.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_8.gtf': No such file or directory
mv: cannot stat '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_vote.gtf': No such file or directory
Traceback (most recent call last):
File "/bin/finder/finder", line 626, in
main()
File "/bin/finder/finder", line 587, in main
orchestrateGeneModelPrediction(options,logger_proxy,logging_mutex)
File "/bin/finder/finder", line 411, in orchestrateGeneModelPrediction
findTranscriptsInEachSampleNotReportedInCombinedAnnotations(options,logger_proxy,logging_mutex)
File "/bin/finder/scripts/findTranscriptsInEachSampleNotReportedInCombinedAnnotations.py", line 17, in findTranscriptsInEachSampleNotReportedInCombinedAnnotations
combined_transcript_info=readAllTranscriptsFromGTFFileInParallel([combined_gtf_filename,"combined","combined"])[0]
File "/bin/finder/scripts/fileReadWriteOperations.py", line 202, in readAllTranscriptsFromGTFFileInParallel
fhr=open(gtf_filename,"r")
FileNotFoundError: [Errno 2] No such file or directory: '/bin/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/combined.gtf'

Can you please provide some support?
Thanks

Thank you so much for trying out the software. I will look into this and get back to you.
Thanks

Hello @josruirod,
I have made some changes to the scripts and updated the install.sh file. I tested on my end, and it works now. Please give it a try and let me know if you face any issues. Also, please send the progress.log file. The install.sh file is under the main directory and the companion softwares need to be downloaded inside the dep directory.
Thank you.

Hello again, thanks for taking the time.
The step by step guide is awesome, thank you very much for that (in the readme you may want to change "cd finder" instead of "cd Finder" after git clone, and step 11 and 14 are not in the src sub-folder but dep, right?)
The installation run without error, also STAR and olego indexes. However, the example is not working. I'm attaching the progress.log and the .error file (extension changed so github accepts it). The .output is empty. Any ideas? Thanks
FINDER_test_ARATH.error.txt
progress.log

Hello @josruirod,
Thanks for checking out the updates. I really helps to improve the software on our end.
I decided to keep it as cd Finder and not cd finder because the name of the head directory is Finder. The name of the program is finder. I know this is a bit confusing and I wanted to create a bioconda package but Finder depends on softwares that are not allowed to be redistributed. Let me know if the head directory name is in fact Finder when you download it. On my system it downloads as Finder and not finder.
I checked the error file and I think that, for some reason, finder does not have execution permissions. File permissions could have been altered when you download from GitHub. I have updated the install.sh script so that it explicitly provides execution rights to all files under the main directory.
I went through the log file and I realized where the problem is. You probably executed the install.sh file the second time. This created another entry in the ~.bashrc file. Hence, finder was reading in two locations of the program instead of just one. The best practice would be to remove all previous installations from the .bashrc file which could mess things up. Hence, I have changed the install.sh script to install.py script. Now the installation will attempt to locate previous installations of finder and remove them before installing it anew. It will inspect previous installations and remove them only if those are finder installations. So if you have a different software called finder installed on your system, it will be left touched.
Could you please send me the contents of the ~/.bashrc file?
Please give this a try. And I would recommend that you remove the previous download and start afresh. Also, since you had previous installations of finder it might be a good idea to log out of the system and log back in after you have completed the new installation. Let me know if you run into any other problems.
Thank you.

Hi, thanks to you for the software and the support.
So, in my system in my system, Ubuntu 16.04, git clone is indeed creating the folder "finder". But if that's variable, guess it's not important then.
Regarding bashrc, when I rerun, I did start fresh and removed the previous installation. However, you are right that bashrc had twice the same entry (not different locations). Not sure if that could affect, bu I have restored.
So with a new installation, now it's working until it doesn't find some psiclass output I think. I attach again progress jand error files. According to the progress file, psiclass ended, but the folder where finder it's looking for some files is empty. I also attach a file I've seen inside the psiclass folder, maybe an error with my perl version? Thanks
FINDER_test_ARATH.error.txt
progress.log
combined.error.txt

Hello,
Thanks for rerunning finder. It is weird that ubuntu would download the folder as finder and not Finder. I will update the README about this issue.
About the bashrc, now it should not matter at all. You can install finder as many times you want. The current installation will remove all the previous installations and preserve the latest one. finder runs some dependent softwares for which the path needs to be provided. Hence, it looks for the installation directory. If there are multiple installations (or multiple entries of the same installation) then it can mess things up. Actually, I am glad you reported this issue. Thanks!
I checked the progress.log file and combined.error.txt. I do think it is a perl issue. It is weird that your system is having trouble with multi-threaded support since the environment explicitly install a library for thread support in perl (line 147 of environment.yml). Could you please run which perl and perl -V|grep threads? In the output you should see useithreads=define. In fact psiclass was unable to run without multi-threaded support. I will get in touch with the developers of psiclass to see if they can help us.
Thank you.

Could you also post the output of perl -v?
Thank you.

Hi, so I decided reboot and git clone/install fresh everything again. So I got during install.sh many errors, operattion not permited for chmod (i tried to execute install.py with sudo but it didn't work). I rerun the example and I'd say now it failed in other step, in braker?
I attach again the progress and the error files. Sorry for the trouble, let's see if we get it solved. Thanks
progress.log
FINDER_test_ARATH.error.txt

Hello @josruirod,
It's really great that you are testing finder out on your system. It gives us a chance to fix things that are system dependent.
Do not run the install.sh directly. Execute the install.py file. Could you elaborate on the errors you encountered during this step? chmod should not have encountered errors with permission since you are the owner of the files whole permission you wish to change. I will try to install this on different systems to check if I can reproduce the error.
From the progress.log file I can see that all the steps have finished properly. In fact, the problem with perl too seems to have been resolved. Could you please attach the /mnt/data/i7_HDD_4TB/Temp/example/FINDER_test_ARATH/braker.output and /mnt/data/i7_HDD_4TB/Temp/example/FINDER_test_ARATH/braker.error file?
From now on you do not have to remove the output directory. Just run the same command. finder will go through the files and determine the last point of failure and will launch the execution from that point onwards.
Thank you.

Hi, happy to help.
Sorry for the typo, I meant I executed the install.py file. So, I attach the braker.error. Braker.output is empty. I removed the finder directory, git cloned and installed again, and I attach a log of the errors on the new installation (this log is 400mb, so I attach a link to a cloud).
Nice to know I don't have to remove the output directory
braker.error.txt
https://saco.csic.es/index.php/s/x75wDFHdLWTEpNX

Hi @josruirod,
Sorry for the late reply and Thanks for your feedback. I had made some changes to the dep directory and I forgot to make the required alterations in the accompanying install.sh file. Sorry for that!! I have removed install.sh and have migrated all the installation commands to install.py. I realized having two different installations file could be a source of confusion.
The reason why you were getting 400MB worth of error messages is because python was trying to change the permissions of all the files and folders on your system. That should not happen anymore. There was a single line that was failing in install.sh which led to the error cascade.
Since, there were errors in the installation process, GeneMark was not properly installed. Please repeat the installation process and you should be all set.
I have included another option in finder to reduce the size of the downloaded fastq files. This will make the test run finish quicker.
Thank you.

Hi, and thanks again for the support.
So I executed a fresh installation, no errors, and it ran for a longer time. I now see files in the "final_GTF_files" folder. But I think braker is still failing. I'm attaching the logs, could you check what's wrong please? Braker.gtf is empty.
Thank you
progress.log
FINDER_test_ARATH.output.txt
braker.error.txt
braker.log
braker.output.txt
FINDER_test_ARATH.error.txt

Hello,
Thanks for running the pipeline again. I am glad to note that it completed running this time. I too was running finder on my end and I was able to reproduce the exact same error as you encountered with braker2. I figured that it is a dependency issue caused by depreciation of some functions in BioPython. The error seems to have occurred in a python script from Augustus. There might be more of such cases. I am attending to those now. I will let you know when everything runs smoothly so that you can give it a try.
Thank you.

Thanks for the explanation! Looking forward to the refined version then!

Hello. Thank you both for raising and troubleshooting these issues. I am keen to run Finder and have been running into the same stumbling blocks you report (i.e. now with final gtf files written but an empty braker gtf). I'm looking forward to  the next iteration. Thank you for addressing this so quickly.

Hello @AnnabelWhibley and @josruirod,
Thank you very much for trying out finder. The braker2 run has completed and is now producing the .gtf files. Luckily, the deprecation issue was confined to a single python script in Augustus which I have now replaced with the latest one. Please go ahead and give it a try.
Thank you.

HI there, happy to hear that. I'd say I've reinstalled and reran as previous times, but I'm getting the following error, even when trying just to execute finder:

File "/home/i7_station/bin/finder/finder", line 263
cmd=f"touch {options.temp_dir}/dummy_protein.fasta"
^
SyntaxError: invalid syntax

Nothing is done due to this error, any input?
Thanks again

My re-install was running OK but crashed out with a memory error during the braker2 phase so I have relaunched it with more resources. Fingers crossed. As an aside, I did find that I needed to manually install the ruffus module into the conda environment
.

My latest problem was a silly mistake, I had not activated the conda environment. It's running now and I'll confirm when it fnishes the example and also a run test with my own data. Thanks

Hi @josruirod,
Happens to the best of people!! I am glad you figured it out. Let me know if you run into any issues.
Thank you

Hi @AnnabelWhibley,
Yes, braker can take quite a bit of memory to operate. I did not encounter any issues with the ruffus package while setting up finder in conda. It could possibly be a system issue. Did the installation complete? Or did it give any errors? Could you also send the configuration of the system where you are running finder? I will try to replicate the issue on my end to troubleshoot.
Thank you.

Thank you @sagnikbanerjee15.  Re: ruffus conda package issue, I can't replicate this and it was easily fixed. I see no errors on installation either time, and the package is clearly there in the environment list. I don't think it is worth your time troubleshooting, it might be some strange SLURM incompatibility at my end.
Run completed (with bug reported in issue#2). Run with 16 threads, max mem usage ~30Gb.

Hello @AnnabelWhibley,
I am glad to know that the run has completed. I have fixed the bug reported in issue#2.
Thanks.

@sagnikbanerjee15. Thank you again for all your support and for developing this tool.

@AnnabelWhibley It's my pleasure.

Hi, I can confirm the example is now running fine, including the issue #2. Thanks for the work. The gff files from braker seem to be now fine. I'm now proceeding to apply this to my own data. A small side note, in the readme it is said that the columns Description, Read Length (bp), Date, are not necessary nor used, but if they are removed the program fails. It seems they are necessary, even if they are empty or with dummy values. You may specify or fix that.
Thanks

Hi @josruirod,
That's awesome! I am glad that the example is up and running now. Thanks for pointing out the issue with metadata. I have updated the README.
Thank you.

Hi, I canc onfirm finder has now worked with my own datasets and organism. Thanks. Just a side note, it seems to be important that  all the columns in the metadata are kept, and not any dummy value can be used or the program will fail.
Thanks for the support

Hi @josruirod,
Thank you so much for trying finder out on your own dataset. I will test finder with the metadata issue you mentioned. Could you please tell me on which step finder is failing when you leave fields vacant in the metadata file?
Thank you.

Hi, I have not kept those runs, but just try to remove the columns that are said to be not essential, such as description, try a dummy name in Project name other than its't not following the ID format (PRBXXX...), try to use "-" in Description... I think it would be needed to improve that documentation because I had to spend some time tuning the metadata.
Thanks again for the support

HI, just fyi. I've found that in another system I had to manually change the limit of files with ulimit. Otherwise I think STAR was failing, and the error message was not too explicative.
Regards

Hello @josruirod,
Thank you for reporting this issue. Yes, it is an issue with STAR. I will add a check in finder to verify the ulimit and report warnings.
Thank you.

Did you do conda activate finder_conda_env? Could you please check?
Also in the command it's not going to be $CPU=30 just put $CPU
Thanks.

Hi, I just came across this software as I have been struggling with MAKER. Running the example data and my data, I have encountered this same isssue. It says it can't find the genome but it's right there.
EXITING: Did not find the genome in memory, did not remove any genomes from shared memory
This is what is on the log.out file;
cat Log.out
STAR version=2.7.7a
STAR compilation time,server,dir=Mon Dec 28 13:38:40 EST 2020 vega:/home/dobin/data/STAR/STARcode/STAR.master/source
Command Line:
STAR --runThreadN 30 --genomeLoad Remove --genomeDir /mnt/nfs/home/b9017460/finder/example/star_index_without_transcriptome
Initial USER parameters from Command Line:
All USER parameters from Command Line:
runThreadN                    30     ~RE-DEFINED
genomeLoad                    Remove     ~RE-DEFINED
genomeDir                     /mnt/nfs/home/b9017460/finder/example/star_index_without_transcriptome     ~RE-DEFINED
Finished reading parameters from all sources
Final user re-defined parameters-----------------:
runThreadN                        30
genomeDir                         /mnt/nfs/home/b9017460/finder/example/star_index_without_transcriptome
genomeLoad                        Remove

Final effective command line:
STAR   --runThreadN 30   --genomeDir /mnt/nfs/home/b9017460/finder/example/star_index_without_transcriptome   --genomeLoad Remove
Number of fastq files for each mate = 1
Finished loading and checking parameters
Reading genome generation parameters:
STAR   --runMode genomeGenerate   --runThreadN 30   --genomeDir star_index_without_transcriptome   --genomeFastaFiles Arabidopsis_thaliana.TAIR10.dna_sm.toplevel.fa      --genomeSAindexNbases 12
GstrandBit=32
versionGenome                 2.7.4a     ~RE-DEFINED
genomeType                    Full     ~RE-DEFINED
genomeFastaFiles              Arabidopsis_thaliana.TAIR10.dna_sm.toplevel.fa        ~RE-DEFINED
genomeSAindexNbases           12     ~RE-DEFINED
genomeChrBinNbits             18     ~RE-DEFINED
genomeSAsparseD               1     ~RE-DEFINED
genomeTransformType           None     ~RE-DEFINED
genomeTransformVCF            -     ~RE-DEFINED
sjdbOverhang                  0     ~RE-DEFINED
sjdbFileChrStartEnd           -        ~RE-DEFINED
sjdbGTFfile                   -     ~RE-DEFINED
sjdbGTFchrPrefix              -     ~RE-DEFINED
sjdbGTFfeatureExon            exon     ~RE-DEFINED
sjdbGTFtagExonParentTranscripttranscript_id     ~RE-DEFINED
sjdbGTFtagExonParentGene      gene_id     ~RE-DEFINED
sjdbInsertSave                Basic     ~RE-DEFINED
genomeFileSizes               120586240   985722733        ~RE-DEFINED
Genome version is compatible with current STAR
Number of real (reference) chromosomes= 7
1       1       30427671        0
2       2       19698289        30670848
3       3       23459830        50593792
4       4       18585056        74186752
5       5       26975502        92798976
6       Mt      366924  119799808
7       Pt      154478  120324096
Started loading the genome: Fri Jun 18 10:22:22 2021
Genome: size given as a parameter = 120586240
SA: size given as a parameter = 985722733
SAindex: size given as a parameter = 1
Read from SAindex: pGe.gSAindexNbases=12  nSAi=22369620
nGenome=120586240;  nSAbyte=985722733
GstrandBit=32   SA number of indices=238963086
EXITING: Did not find the genome in memory, did not remove any genomes from shared memory
Jun 18 10:22:22 ...... FATAL ERROR, exiting
Please help. I have been trying to annotate a genome for a very long time now but keep encountering errors.
Best,
Lenshina

Hi there,
I've been trying to run Finder based on the test data prior to running it on my own data, but unfortunately I've been unable to get it to run to completion.
The most recent run:
Jun 28 13:25:24 ...... FATAL ERROR, exiting
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_0.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_1.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_2.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_3.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_4.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_5.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_6.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_7.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_8.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_9.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_sample_10.gtf': No such file or directory
mv: cannot stat '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/psiclass_output_vote.gtf': No such file or directory
Traceback (most recent call last):
File "/scratch2/software/finder/finder", line 668, in
main()
File "/scratch2/software/finder/finder", line 629, in main
orchestrateGeneModelPrediction(options,logger_proxy,logging_mutex)
File "/scratch2/software/finder/finder", line 442, in orchestrateGeneModelPrediction
findTranscriptsInEachSampleNotReportedInCombinedAnnotations(options,logger_proxy,logging_mutex)
File "/misc/scratch2/software/finder/scripts/findTranscriptsInEachSampleNotReportedInCombinedAnnotations.py", line 17, in findTranscriptsInEachSampleNotReportedInCombinedAnnotations
combined_transcript_info=readAllTranscriptsFromGTFFileInParallel([combined_gtf_filename,"combined","combined"])[0]
File "/misc/scratch2/software/finder/scripts/fileReadWriteOperations.py", line 243, in readAllTranscriptsFromGTFFileInParallel
fhr=open(gtf_filename,"r")
FileNotFoundError: [Errno 2] No such file or directory: '/scratch4/gseaton/FINDER_test3/FINDER_test_ARATH/assemblies_psiclass_modified/combined/combined.gtf'
I've tried reinstalling it, as well as making sure the appropriate lines were present in my bashrc file, but haven't had any luck yet. Any guidance you could give will be greatly appreciated.

Hello @GSS-Investigator,
Thank you very much for trying out finder. I think there might be an issue with psiclass. Could you please send me the progress.log file?
Thanks.

progress.log
Here you go @sagnikbanerjee15

Thank you @GSS-Investigator for sending the progress.log file. It seems that finder had difficulty starting the program. Could you please check if the gzipped files were expanded? The best would be to zip the output folder and email it to<EMAIL_ADDRESS>I can inspect it on my end.
Thank you.

Hi I am having the same issue as GSS-Investigator and my progress.log file looks the same. Did this get resolved?
I also found in FINDER_test_ARATH/assemblies_psiclass_modified/combined.error this error:
_sh: line 1: 299248 Segmentation fault      (core dumped) Finder/finder/dep/psiclass_terminal_exon_length_modified/trust-splice Finder/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/splice/psiclass_output_splice_0.list Finder/finder/example/FINDER_test_ARATH/alignments/SRR5197913_for_psiclass.bam > Finder/finder/example/FINDER_test_ARATH/assemblies_psiclass_modified/combined/splice/psiclass_output_bam_0.trusted_splice
Which might relate to this - https://github.com/splicebox/PsiCLASS/issues/12?


added regression test for subsetting with group and cv with group-based metrics

Added regression test for #1646.

Seems that VSTS check is not required?..

@StrikerRUS I will enable it


Fix issue with C++ endf float converter

This closes #6 - at least for me, I noticed someone having similar issues in #4.
The crash seems to be a seg fault that occurs when the endf float being converted is negative, i.e. has a leading minus sign.
Positive numbers have leading whitespace so hit the continue statement and j is not incremented. Therefore, the first digit is placed at arr[0]. However, negative numbers had the leading minus sign placed at arr[0] and first digit at arr[1]. This means when the null terminator is placed at the end, for negative numbers this is at arr[12] so the crash occurs.
EDIT: my initial commit was not the correct solution, I think it is simply a case of increasing arr to be length 13 so it has space for both the e and null terminator if the input buffer is length 11. Apologies, some how my initial incorrect fix ran (I have no idea how!) but on restarting it failed.
This would seem to me to be an issue across systems.

@paulromano
Sure, in ENDF/B-VIII.0 incident neutron data for deuterium there is -2.225002+6 at the beginning of the MF = 3, MT = 16 record. This is the QM/QI value.

@aidancrilly Would you be up for making a similar PR to OpenMC? The cfloat_endf function was originally borrowed from there:
https://github.com/openmc-dev/openmc/blob/develop/openmc/data/endf.c
It would be good to also add the example you mentioned (-2.225002+6) to the tests here:
https://github.com/openmc-dev/openmc/blob/develop/tests/unit_tests/test_endf.py


Rerender and rely on run_exports

Did feel like there was an issue with freetype but it is somewhere else in the stack but as I'm already at it, let's clean it up a bit.

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge/freetype This is ready for review/merge.

@conda-forge/freetype This is ready for review/merge.

Thanks @xhochy!

Thanks @xhochy!


feat(icons): Add ethernet-port and wired-connection icons

Closes #1816
What is the purpose of this pull request?

[x] New Icon
[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other:

Icon use case
To indicate a network connection in a router, network switch, or similar interfaces.
Can also be used as a system icon, next to the existing system icons (wifi, Bluetooth, battery and audio etc)
Alternative icon designs
See #1816
Icon Design Checklist
Concept

[x] I have provided valid use cases for each icon.
[x] I have not added any a brand or logo icon.
[x] I have not used any hate symbols.
[x] I have not included any religious or political imagery.

Author, credits & license

[x] The icons are solely my own creation.
[x] The icons were originally created in # by @
[x] I've based them on the following Lucide icons:
[x] I've based them on the following design:

Naming

x ] I've read and followed the naming conventions
[x] I've named icons by what they are rather than their use case.
[x] I've provided meta JSON files in icons/[iconName].json.

Design

[x] I've read and followed the icon design guidelines
[x] I've made sure that the icons look sharp on low DPI displays.
[x] I've made sure that the icons look consistent with the icon set in size, optical volume and density.
[x] I've made sure that the icons are visually centered.
[x] I've correctly optimized all icons to three points of precision.

Before Submitting

[x] I've read the Contribution Guidelines.
[x] I've checked if there was an existing PR that solves the same issue.

What about arcifying the eathernet-port icon?
Open lucide studio

wired-connection feels an oddly specific name for an abstract symbol that to my best knowledge is not a standard representation of said concept, so maybe arrow-double(-headed)-horizontal-dashed or something similar would be better?

@karsa-mistmere Kinda true. It's more common than you might think.

Wiki: https://en.wikipedia.org/wiki/Ethernet
So not sure if we should choose a generic name since it is possibly recognized as a symbol for ethernet.
But I get you, I still have some doubts about the name. 🤔

@karsa-mistmere Kinda true. It's more common than you might think.
Wiki: https://en.wikipedia.org/wiki/Ethernet So not sure if we should choose a generic name since it is possibly recognized as a symbol for ethernet. But I get you, I still have some doubts about the name. 🤔

Still feel like it is describing the use case rather than what is actually depicting.

@jguddas @karsa-mistmere I've renamed it to chevrons-left-right-ellipsis


DefaultTimestampFormat -> defaultTimestampFormat breaks API backward compatibility

In 325575f18110626a8fd7f027f899aa8c8f5169c6 you unexported defaultTimestampFormat in formatter.go commenting that you fixed a linter suggestion.
I know these damn linters, but sometimes you have to keep a strong opinion about your decisions.
You eventually broke API backward compatibility.  I know of at least one program that suffered from that: github.com/toorop/tmail. Please fix that issue.
I saw your comment on that PR, and there are two sides to that coin. If people use your code you should prevent breaking their stuff,  regardless if you like or dislike the outcome.
Linters are not the source of all wisdom.

Petty sure the golint complaint was just that it was an exported value without a comment.  The solution to that is to comment that value, not to unexport a previously exported (and useful) value.

@morriswinkler we have not been very good at doing semver until now, but we are trying to do better. Besides that, this issue is almost one year old with not so many complain. I think we can just move on instead of exporting again an internal constant. wdyt ?

I feel like the cost of re-exporting it is essentially non-existent, no?

https://github.com/FactomProject/logrustash/releases/tag/0.7.0 is also broken.

@dgsb look in 3 days I will have posted that issue one year ago, and since it is still going on. I will give you my 30 cents of personal opinion on the issue.
I personally don't really care if this constant is gone or not, I think golang developers should keep in mind that the original package design strategy was to build stable API's and keep them backward compatible. While the community has moved on and adapted versioning years ago, we still should uphold that goal.
So please someone take a stand on that issue, either reexport it ( not for me by the way ) or just say you won't and close that issue.


i32 being rendered as #/components/schemas/i32"

Hi.
Using the following example
#[derive(OpenApi)]
#[openapi(components(schemas(Transactions)))]
struct ApiDoc;

#[derive(ToSchema)]
pub enum Transactions {
    Transaction(EntryAlias),
}

pub type EntryAlias = Entry<i32>;

#[derive(ToSchema)]
pub struct Entry<I> {
    pub entry_id: I,
}

Integers are being rendered as #/components/schemas/i32"
"components": {
    "schemas": {
      "Entry": {
        "type": "object",
        "required": [
          "entry_id"
        ],
        "properties": {
          "entry_id": {
            "$ref": "#/components/schemas/i32"
          }
        }
      },
      "Transactions": {
        "oneOf": [
          {
            "type": "object",
            "required": [
              "Transaction"
            ],
            "properties": {
              "Transaction": {
                "$ref": "#/components/schemas/Entry"
              }
            }
          }
        ]
      }
    }
  }

Expected result

"entry_id": {
  "type": "integer",
  "format": "int32",
}

If Enum is specified like this
#[derive(ToSchema)]
pub enum Transactions {
    Transaction(Entry<i32>),
}

Then it renders correctly.
Also I have tried using utoipa-config
use utoipa_config::Config;

fn main() {
    Config::new()
        .alias_for("EntryAlias","Entry<i32>")
        .write_to_file();
}

But this did not help either (assuming I've done it correctly..)

When you change the config in build.rs try cargo clean so that next time you build the app wont even accidentally use old cached Config. This might be happening and is common even with other libs like tonic.

I've tried cargo clean and still generating as "#/components/schemas/i32"

Okay, I have to investigate this a bit and see is there something that can be done. Maybe I have missed something or it did hit the limit of possibilities with automatic resolution.

The build.rs config should not let that happen.
n
   Compiling utoipa-config-test v0.1.0 (/home/juha/dev/repos/utoipa/utoipa-config/config-test-crate)
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.72s
     Running `target/debug/utoipa-config-test`
{
  "openapi": "3.1.0",
  "info": {
    "title": "utoipa-config-test",
    "description": "",
    "license": {
      "name": ""
    },
    "version": "0.1.0"
  },
  "paths": {},
  "components": {
    "schemas": {
      "Entry_i32": {
        "type": "object",
        "required": [
          "entry_id"
        ],
        "properties": {
          "entry_id": {
            "type": "integer",
            "format": "int32"
          }
        }
      },
      "Transactions": {
        "oneOf": [
          {
            "type": "object",
            "required": [
              "Transaction"
            ],
            "properties": {
              "Transaction": {
                "$ref": "#/components/schemas/Entry_i32"
              }
            }
          }
        ]
      }
    }
  }
}
{
  "type": "object",
  "required": [
    "name",
    "my_type",
    "my_value",
    "date"
  ],
  "properties": {
    "date": {
      "type": "string"
    },
    "my_int": {
      "type": [
        "integer",
        "null"
      ],
      "format": "int32"
    },
    "my_type": {
      "type": "boolean"
    },
    "my_value": {
      "type": "string"
    },
    "name": {
      "type": "string"
    },
    "optional_date": {
      "type": [
        "string",
        "null"
      ]
    }
  }
}

main.rs
#[derive(OpenApi)]
#[openapi(components(schemas(Transactions)))]
struct ApiDoc;

#[derive(ToSchema)]
pub enum Transactions {
    Transaction(EntryAlias),
}

pub type EntryAlias = Entry<i32>;

#[derive(ToSchema)]
pub struct Entry<I> {
    pub entry_id: I,
}
#[allow(unused)]
struct MyDateTime {
    millis: usize,
}

fn main() {
    let api = ApiDoc::openapi();

    println!(
        "{}",
        serde_json::to_string_pretty(&api).expect("api must be JSON serializable")
    );
}

build.rs
fn main() {
    utoipa_config::Config::new()
        .alias_for("EntryAlias", "Entry<i32>")
        .write_to_file()
}

And likewise if I have this:
#[derive(OpenApi)]
#[openapi(components(schemas(Transactions)))]
struct ApiDoc;

#[derive(ToSchema)]
pub enum Transactions {
    Transaction(EntryAlias),
    TransactionEntryString(EntryString),
}

pub type EntryAlias = Entry<i32>;
pub type EntryString = Entry<String>;

#[derive(ToSchema)]
pub struct Entry<I> {
    pub entry_id: I,
}

build.rs
fn main() {
    utoipa_config::Config::new()
        .alias_for("EntryAlias", "Entry<i32>")
        .alias_for("EntryString", "Entry<String>")
        .write_to_file()
}

the output is
{
  "openapi": "3.1.0",
  "info": {
    "title": "utoipa-config-test",
    "description": "",
    "license": {
      "name": ""
    },
    "version": "0.1.0"
  },
  "paths": {},
  "components": {
    "schemas": {
      "Entry_String": {
        "type": "object",
        "required": [
          "entry_id"
        ],
        "properties": {
          "entry_id": {
            "type": "string"
          }
        }
      },
      "Entry_i32": {
        "type": "object",
        "required": [
          "entry_id"
        ],
        "properties": {
          "entry_id": {
            "type": "integer",
            "format": "int32"
          }
        }
      },
      "Transactions": {
        "oneOf": [
          {
            "type": "object",
            "required": [
              "Transaction"
            ],
            "properties": {
              "Transaction": {
                "$ref": "#/components/schemas/Entry_i32"
              }
            }
          },
          {
            "type": "object",
            "required": [
              "TransactionEntryString"
            ],
            "properties": {
              "TransactionEntryString": {
                "$ref": "#/components/schemas/Entry_String"
              }
            }
          }
        ]
      }
    }
  }
}

Ugh, not again 😅
I will try again and report back. Thanks for investing

Unfortunately I still experience the same issue even after cargo clean and cargo update. Tested both of your examples and behavior is the same as I reported.
It is likely that its something wrong with cargo.lock as in the issue that I previously reported. I trust that it will work fine after release.
So for now the issue may be closed if you wish so. Thanks

Right super weird, hope it will start working then at least as it should. When I tried with the test crate and modified the build.rs file I had to sometimes to build multiple times or change some files in the codebase in order it to recognize that there was any changes so it could use the fresh config.
Nonetheless closing for now.


oneOf throws ERROR Error: Cannot find control with unspecified name attribute

Description
I am getting error from oneOf json schema type
ERROR Error: Cannot find control with unspecified name attribute

Full Reproduction
https://stackblitz.com/edit/angular-yz9cv3?file=src%2Fassets%2Fjson-schema%2Fwith_issue.json
There is my schema with_issues.json, addded as first on load.
Expected
Presented schema does not throw errors.
Stackblitz Environment
"@angular/core": 7.2.16",
"@ngx-formly/core": "^5.9.1",
My Project Environment
"@angular/core": "~8.2.14",
"@ngx-formly/bootstrap": "^5.7.2",
"@ngx-formly/core": "^5.7.2",

@aitboudad I can see underneeth another problem, when I patched your implementation, or even without it I noticed that form is populated with all properties from oneOf with null values.
Only after change of every property from oneOf field valueChanges emits proper values from form so on init and after one change or one element added You submit wrong values cause all properties are set from oneOf so schema does not validate as oneOF
Also form first emit is after inintialization and it presents wrongly populated value
Should be there first emit?
https://stackblitz.com/edit/angular-ngx-formly-oneof-playground

I have updated schema to hierarchy in option to result in
   {
      id: {
              ...oneOf options
      }
   }

Stackblitz
Also after this changes I am able to get correct model when I log it on changes but form values are incorrect.
changes_for_oneOf.zip

Hi @Adam-Michalski,
I'm not able to track all the mentioned issues, could you please check with v5 branch and let me know if there are some remaining issues to fix (it would be better to split them in a separate issue).
Thank you!

Ok, I would prepare separate examples in stacblitz if issues are still there.

This issue has been fixed and released as part of v5.10.0 release.
Please let us know, in case you are still encountering a similar issue/problem.
Thank you!


How to change the settings.json file location ?

Description of the new feature/enhancement

Proposed technical implementation details (optional)

By default, the settings.json is in C:\Users\xxx\AppData\Local\Packages\Microsoft.WindowsTerminal_8wekyb3d8bbwe\LocalState\settings.json.
How to change the default location ?

I don't think you can do that now. See #2933 .

/dup #2934 thanks Chester!

@DHowett, you mistyped the issue number. Does that affect any automation?

Nope, just my flawless sense of style. Thanks :)


MINOR: Adding -daemon option to zookeeper-server-start.sh USAGE, similar to kafka-server-start.sh

Output after fix:
#satul# ./kafka-server-start.sh
USAGE: ./kafka-server-start.sh [-daemon] server.properties
#satul# ./zookeeper-server-start.sh
USAGE: ./zookeeper-server-start.sh [-daemon] zookeeper.properties

could add a config item at server.properties and zookeeper.properties ?
It's same with redis config file . Maybe looks better :)

Thanks for the patch @atulsm. It looks good. I think the config item that @darionyaphet suggested can be discussed separately since it is a more substantial change (it modifies existing behavior).
Do you mind opening a JIRA, so this change can make it to our release notes?

@darionyaphet  & @gwenshap . Thanks for your suggestions.
I have created JIRA https://issues.apache.org/jira/browse/KAFKA-3119 for this. I am not able to assign this issue to my name though.

Thanks @atulsm. I'll assign the JIRA to you. Can you please edit the issue title to replace MINOR with KAFKA-3119? It makes the automated merging a bit easier on me :)

@gwenshap Sure, i have already renamed the pull request name to contain the JIRA.

Perfect. LGTM.

Merged. Thank you for your contribution!


Fixes to README and styles

Summary

Fixed cloning instructions : "TypeScript-Docs" to "TypeScript-Website"
Fixed Star link to TypeScript's GitHub not being displayed
Before :

After :

Fixed footer not appearing in the bottom (for our friends that are IE users)
Before :

After :

Tested in IE, Edge, Firefox and Chrome

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

Kudos! Nice work ❤


Move the instructions for installation of Code Editor earlier in the set up process

What
Move the instructions for the installation of Code Editor earlier in the set up process
Why
As a new user, I want to be able to complete all the installations in one go, so that I know it is complete and I can get on to creating a prototype
Who needs to work on this

Content designer

Done when

[x] The installation of Code Editor is at a similar stage to the other installations in the tutorial

This has been included within the branch: https://github.com/alphagov/govuk-prototype-kit-docs/tree/operating-systems


Support for attachment upload

From the doc: https://gitlab.adhoc-gti.com/help/api/projects.md#uploads
How python does it: https://github.com/gpocentek/python-gitlab/pull/239/files
This is effectively a requests call for MultiPart: http://docs.python-requests.org/en/master/user/quickstart/#post-a-multipart-encoded-file

:+1:

I'll take it @NARKOZ !


UICollectionViewFlowLayout

Hello! This isn't an issue but a "help wanted".
I need to replicate this custom UICollectionViewFlowLayout layout (image).
Using:

collectionView(_:layout:insetForSectionAt:)

collectionView(_:layout:sizeForItemAt:)
I'm able to replicate the layout.

I'm not able to replicate the layout using PulsarKit and in particular I tried all the suggested layouts.
The elements are not on the same row even if I set a little width size.
Do you have any suggestions?
Thanks

Hi @Bellaposa, apologies for the delayed response.
With PulsarKit 1.4.0 you can use the new SplittedSize.
If you want a fixed number of blocks in width and height
SplittedSize(numOfElementsInWidth: 2, numOfElementsInHeight:  4)
If you want a fixed number of blocks in width and dynamic height (autolayout)
SplittedSize(numOfElementsInWidth: 2)
If you want a fixed number of blocks in width and your specific sizable for height
SplittedSize(numOfElementsInWidth: 2, heightSize: FixedSize(height: 100))

Hi @Bellaposa, apologies for the delayed response.
With PulsarKit 1.4.0 you can use the new SplittedSize.
If you want a fixed number of blocks in width and height
SplittedSize(numOfElementsInWidth: 2, numOfElementsInHeight:  4)
If you want a fixed number of blocks in width and dynamic height (autolayout)
SplittedSize(numOfElementsInWidth: 2)
If you want a fixed number of blocks in width and your specific sizable for height
SplittedSize(numOfElementsInWidth: 2, heightSize: FixedSize(height: 100))


Ansible scripts for Stream Processor 4.2.0

Description:
Add Ansible scripts to deploy single node WSO2 Stream processor 4.2.0
Suggested Labels:
Type/Improvement
Affected Product Version:
Stream Processor 4.2.0
OS, DB, other environment details and versions:
Ansible 2.6.2

Closing because https://github.com/wso2/ansible-sp/pull/2 fixed this issue.


Add '--keep' option for update

Add '--keep' command line option for update.
If command line option is not specified, use preference stored in the database (this is what should be happening now, but may not be working properly).

Fixed in 7b526dce41fe865464d16ae9e22783ecfbf92346.


Added IE11 Hash change Event

You're saying spawnPopState only supports changing the hash? As far as I can see using popstate supports changing any part of the URL, not only the hash. The solution in https://github.com/mjackson/history supports both as it listens to both events, not just one or the other as it looks to me this PR does.

Please, test it, and report the results.
You can pull my branch to try it out.
On Jan 8, 2017, 12:16 PM +0100, David Ed Mellum<EMAIL_ADDRESS>wrote:

You're saying spawnPopState only supports changing the hash? As far as I can see using popstate supports changing any part of the URL, not only the hash. The solution in https://github.com/mjackson/history supports both as it listens to both events, not just one or the other as it looks to me this PR does.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm sorry, but I don't have the time to create a test case and test this. I can't see how it's supposed to work correctly. Do you have a minimal test case that shows it working with both hashchange and popstate in IE11 and Chrome?

My friend,
Who here is bearing the burden of proof?
If you don't have time to test it, then obviously you don't need it, so why bother speculating about two line change?
You really expect that I will try convince that this is working?
On Jan 12, 2017, 12:40 PM +0100, David Ed Mellum<EMAIL_ADDRESS>wrote:

I'm sorry, but I don't have the time to create a test case and test this. I can't see how it's supposed to work correctly. Do you have a minimal test case that shows it working with both hashchange and popstate in IE11 and Chrome?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Your PR doesn't solve the issue, because You delete the whole popstate event from IE, and the issue is that IE doesn't trigger the popstate event only when the hash is changed in the url, but the rest of events are working fine in IE. With your "fix" the rest of navigation will not work in IE.

@popara, to the question:

Who here is bearing the burden of proof?

It's on you. A high quality PR (1) has a description at all, (2) it outlines the key concerns and anticipates questions that may arise, and (3) provides as much evidence as possible to make the review an easy process.
This is what @process-bot says to everyone.
I need to look into this more on my own to figure out what everyone is trying to say here.

I agree with @edmellum and @daymosik's analysis here. This will not listen to non-hash changes.


Synchronize SemanticDB spec and Symbol.apply

At the moment, the specification doesn't comprehensively describe how the Symbol.apply parser works. It would be great to get to the bottom of this for the sake of https://github.com/twitter/reasonable-scala/issues/10.

Fixed in https://github.com/scalameta/scalameta/pull/1255.


Air quality related variables and classes should be placed in its own file

The configuration file has been quite bloated. Furthermore, the air quality data isn't exactly configuration - it should be a module.
This might also help reorganize the clutter within.

Closed as of recent commit.


DOCS-1040/3.24: Add K8 v1.25 support to docs

Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

@lwr20 @ctauchen Can you please review this one?

/sem-approve


[Image Captioning] states in lstm()

Hello,
I have a quick question on calling of the lstm() in model.py.
In the training step, you simply run lstm layer as: "hiddens, _ = self.lstm(packed)", without assigning the states.
Differently, in the sampling step, you send the features of EncoderRNN to the lstm layer as: "hiddens, states = self.lstm(inputs, states)".
I wonder that why we send the state parameters in sampling step. Because it seems to me that the returned states are not used in sample() function, can we just do "hiddens, _ = self.lstm(inputs)"?
Thanks.

Training phase
If we don't assign hidden state, nn.LSTM assigns zero state as initial state in default. It is desirable to set the initial state value of nn.LSTM to zero.
Test phase
In hiddens, states = self.lstm(inputs, states), you should note that returned states are used for the next time step. If you do not assign the states such as self.lstm(inputs), it assigns zero states internally and it performs wrong operation.
In test phase, the shape of inputs is (batch_size, 1, embed_size) so that only 1 time step forward propagation is performed when self.lstm(inputs, states) is called.

It makes much sense to me. Thanks for the answer!