add validation for all url requests

Handle situations when a user sends a bad URL request, for example :

malformed id value.
an id for an entity that doesn't exist.
..............

Done using Spring MVC exception handling mechanisms.
Added an Advice class that contains handlers for types of exceptions that could be fired up.

Done using Spring MVC exception handling mechanisms.
Added an Advice class that contains handlers for types of exceptions that could be fired up.


type '() => Future' is not a subtype of type '() => Future'

URL: https://pub.dartlang.org/packages/async_loader#-readme-tab-
I was using an older version of Dart where httpClient was in use. But when i updated dart, I had to shift to http package. But when I work with it, I am able to get a response but I am unable to load after I get the response. I see an error like
type '() => Future' is not a subtype of type '() => Future'
Stack trace is
I/flutter ( 3802): When the exception was thrown, this was the stack:
I/flutter ( 3802): #0      DetailPage._securityPage. (file:///Volumes/MEM/Flutter/virtualdoor/lib/main.dart:658:32)
I/flutter ( 3802): #1      AsyncLoaderState.build (package:async_loader/async_loader.dart:76:19)
I/flutter ( 3802): #2      StatefulElement.build (package:flutter/src/widgets/framework.dart:3730:27)
I/flutter ( 3802): #3      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3642:15)
I/flutter ( 3802): #4      Element.rebuild (package:flutter/src/widgets/framework.dart:3495:5)
I/flutter ( 3802): #5      BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2242:33)
I/flutter ( 3802): #6      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&RendererBinding&WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:626:20)
I/flutter ( 3802): #7      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:208:5)
I/flutter ( 3802): #8      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:990:15)
I/flutter ( 3802): #9      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:930:9)
I/flutter ( 3802): #10     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:842:5)
I/flutter ( 3802): #11     _invoke (dart:ui/hooks.dart:120:13)
I/flutter ( 3802): #12     _drawFrame (dart:ui/hooks.dart:109:3)

@Arunva28:

Please specify the versions you are using, and double-check that you have strong mode enabled in your project and it has no analyzer warnings.

I'm the author of package:http_client here, and I've published 0.4 a few weeks ago. What was you issue of using it with the new Dart version?

This kind of nonsensical type error is common if you have relative imports in lib/main.dart (imports where the path doesn't start with 'package:...'

PS: I just noted in passing that the type arguments in the initial entry got swallowed during rendering, which makes it look like the type relationship under consideration is Future Function() and Future Function(); it is actually Future<dynamic> Function() and Future<Null> Function(), just like it is in the title, and for those two types it is indeed true that there is no subtyping relationship. I'm not working on pub so I won't make any attempt to go futher, but I just thought that it would be useful to avoid that misunderstanding. ;-)

Not sure what's left to do here, closing.


Classrooms: Alpha version of Classroom list creator

Who is this PR for?
Teachers and principals grades KF-5 as they assign classrooms for the upcoming year.  Specifically, this is for the first pilot tests.
What problem does this PR fix?
This feature is a tool to help teachers and principals create class lists and visualize some information about them in the process.  This adds a new /balancing URL for teachers to use this tool.
What does this PR do?
The UI starts up in ClassroomListCreatorPage which tracks state changes and requests.  ClassroomListCreatorWorkflow renders the different steps of the process and CreateYourClassroomsView is the UI component for the main dragging-dropping and summarizing interaction.
Added dependencies:

multi-select: react-select (requires style-loader and css-loader updates to Webpack config)
dragging and dropping: react-beautiful-dnd (and array#find polyfill which it needs)
percentile computations: percentile

Add components:

<Loading /> factored out from <GenericLoader /> for other uses
<BoxAndWhisker /> for a small box-and-whisker plot
<Button /> factored out from ".btn" className
<Hover /> for rendering on hover
In Ruby, GradeLevels does math with going forward or backward a year in grade levels

Updated user-facing routes:

Rename /equity/principal to /equity/explore clarify
Add /balancing and /balancing/:id

New endpoints:

#available_grade_levels_json > what schools and grade levels can the user create?
#students_for_grade_level_next_year_json > what's the list of students for that grade?
#update_classrooms_for_grade_json > for recording changes while the user works

Changes to existing code:

Change <Bar /> API to take standard style instead of styles, update call sites.  Add story.
Rename <Breakdown /> to <ExploreClassroomComparisons />
Add overflow-x: hidden on index.html to remove scrollbars in IE, add horizontal scroll on /equity/explore page so that's still supported.

Screenshot (if adding a client-side feature)
Choose your grade

Make a plan

Create your classrooms

Clicking on a student
(placeholder for now)

Notes to principal

Submit

Component screenshots
react-select

<Button />

<Circle />

<BoxAndWhisker />

Checklists
Javascript QA

[ ] Author checked latest in IE - Classroom List Creator
[ ] Author checked latest in IE - Student Profile
[ ] Author checked latest in IE - Student Report PDF
[ ] Author checked latest in IE - School Roster
[ ] Author checked latest in IE - Home feed
[ ] Author checked latest in IE - Absences, Tardies, Disciplines

Automated Testing

[x] Author improved specs for code in need of better test coverage
[x] Author included specs for new code

This would be good to double-check but @alexsoble is on vacation :)  We're staging this in a few pilot tests where we can research and support, and then will update this to a beta (https://github.com/studentinsights/studentinsights/issues/1659) as well before a full release.

selfie


initial start to update vue-apollo latest beta .21

This PR will update to latest changes of vue-apollo

@Akryum it would be great if you could check if thats how vue-apollo should be used. Thanks!

There is a problem with fakeInstance, described here
https://github.com/Akryum/vue-apollo/issues/378

@negezor Ok in a separate project I receive the same error:
  TypeError: Cannot read property '$ssrContext' of undefined
  - server-bundle.js:487 Component.options.data
    server-bundle.js:487:14
  - index.js:148 createFakeInstance
    [lumen-cms]/[vue-apollo]/ssr/index.js:14

@Akryum is this an internal error of vue-apollo or is the upgraded code not complete yet? Would be great if we find a fix for this.

@dohomi Of course, I do not know for sure, but maybe should not try to emulate the very Vue instance?

@negezor I updated to the latest .25 and now everything works fine. Could you update your code and re-validate?

@dohomi yes, it works. Thanks 😄

I have to revert - currently I see this error:
 ERROR

  TypeError: Cannot read property 'props' of undefined

  - vue.runtime.common.js:1354 normalizeProps
    [lumen-cms]/[vue]/dist/vue.runtime.common.js:1354:23

  - vue.runtime.common.js:1456 Object.mergeOptions
    [lumen-cms]/[vue]/dist/vue.runtime.common.js:1456:3

  - utils.js:14 exports.getMergedDefinition
    [lumen-cms]/[vue-apollo]/lib/utils.js:14:27

  - utils.js:188 Promise.then.component
    [lumen-cms]/[vue-apollo]/ssr/utils.js:188:19

  - next_tick.js:188 process._tickCallback
    internal/process/next_tick.js:188:7

This error is not persistent though, I couldn't figure out why and when it exactly happens. Usually after few reloads its gone

@Akryum the error is being shown after the change of https://github.com/nuxt-community/apollo-module/pull/158/commits/5ad31177dacc8ca243db9677d62fc48ff0214186
this happens in beta.25

any updates regarding this issue?

@Akryum @Atinux do you guys have any idea how to solve this for this module? Currently the upgrades are not possible due to the SSR problem

Hey, I was able to debug this and filled Akryum/vue-apollo#402.
Also keep in mind that that new SSR is throwing errors for many apps (see Akryum/vue-apollo#392).
Last thing, in the last line modified in this PR it should be nuxtState.apollo = ApolloSSR.getStates(apolloProvider) instead of nuxtState.apollo = ApolloSSR.exportStates(apolloProvider). With that change this PR will work with simple projects (that doesn't touch any of the above issues).

A side note

nuxtState.apollo = ApolloSSR.exportStates(apolloProvider)
Nuxt state should be serializable. So there should be only values, not a functions or Symbols. So it need to make sure that there no not serializeable data like Symbols


Review "RestProps" spread method for arbitrary attributes

Describe the feature in detail (code, mocks, or screenshots encouraged)
@ryceg has suggested we implement a spread prop that allows for overflow attributes to be appended to the root element in each component. Imagine something like this:
Svelte
let {
    foo = "bar",
    {...rest}
}: ExampleProps = $props();

...

React
export const Switch: React.FC<SwitchProps> = ({
    foo = "bar",
    {...rest}
}) => { /* ... */ };

...

Use Case
Appending ARIA attributes to the root element, for example:
<SomeComponent aria-lablelledby="foo" />

...

What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
We'll need to verify that this works as expected between both Svelte and React.
Potential Gotchas

Duplicating standard props as attributes (ex: per above, foo is a prop, not an attribute)
Folks using the standard class attribute over the base classes convention
Folks thinking these attributes somehow extend into the children elements
We should set contributor guidelines to prevent using rest props. These are for users ONLY.

I don't think it's an unreasonable expectation that class be exposed and implemented on root elements. We're making building blocks, and keeping them to the regular HTML spec makes them predictable. We can always remove the class attribute via Omit<HTMLAttributes<HTMLDivElement>>, 'class'> to keep people using the classes convention if we decide to go all in on that, though.
Regarding duplicating the props, the last one wins- so just make sure that our spread operator is at the top of the element, and it'll be overridden properly.

Yeah we're pretty committed to the classes attribute because it allows for more than just the parent. Any children can follow the same pattern. So the mix of the standard class attribute + classes can and would cause some confusion I'm sure.
https://next.skeleton.dev/docs/resources/contribute/components#props
Though thinking it out, I don't think it would cause any real harm, but again, that's what I'll need to test to verify.

I agree with @ryceg. Using class instead of classes will probably be less confusing. I've already used the techniques decribe above for the Listbox PR: #2754 without realizing this issue was open. I did exactly what was described, spread the attributes and override the one's we use internally. Especially with the zag discussion coming up (#2777) it would be nice if we can resolve this before going all in into porting again.


Fix code block

English pages are OK. However, the Japanese translation misses the end of the code block and stops.

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

@MashaMSFT could you take a look or engage someone from review team to check?

Hello! Thanks for bringing this to our attention! This actually seems to be a localization issue vs an article issue, so I don't believe merging this change is the appropriate method to resolve this. Let me research this a bit further...
Thanks!
Masha from the SQL Docs team

#assign:mashamsft Masha, I'm adding you as an assignee since you're looking into whether there is anything we can do on our end. If it is a localization issue and we've reported it, we probably would close this PR. Thank you!

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

@yoichinak Hello Yoichi, Thank you for these fixes, handily provided by you here in your PR #4738!
I like these changes, and I will now Merge your PR.
cc: @MashaMSFT


pip install takes to long

When I install lambdasoc with pip, It takes too long because it clones submodules recursively, cloning the llvm project which is huge.
Some possible solutions:

Download bios files from its repository when labdasoc is installed or during building process of an soc.
Remove llvm-project from the bios repository
Use another way to track llvm-project. Maybe cloning it in the Makefile

Regards.

Sorry for the delay. I fixed this (once and for all, hopefully) by copying the required builtins to the lambdasoc-bios repo.


[BUG] Using :lcd in other tabs causes other project tests to show in the original summary window

NeoVim Version
Output of nvim --version:
NVIM v0.9.4
Build type: Release
LuaJIT 2.1.1692716794

Describe the bug
Using the :lcd command in other windows or tabs shows discovered tests in the original summary window.
This affects workflows where multiple tabs and windows are used for different projects and the original tab with the summary window starts to get polluted with tests from other project directories.
To Reproduce

Install and add the neotest-python adapter to the standard minimal.lua configuration
Create the  /tmp/parent/{project1,project2} directory structure with separate test files underneath the project1 and project2 directories:

mkdir -p /tmp/parent/project2
cd /tmp/parent/project2
vi test_strings.py

mkdir -p /tmp/parent/project1
cd /tmp/parent/project1
nvim --clean -u /tmp/minimal.lua test_math.py

Run the :NeotestSummary command and view the summary window:

neotest-python
╰╮ test_math.py
 ╰╮ TestMathFunctions
  ├─ test_addition
  ├─ test_subtraction
  ╰─ test_multiply

Create a new tab  with :tabnew
In the new tab run :lcd ../project2 to move to the other project directory
Move back to the original tab and notice how the other tests appear:

neotest-python
╰╮ test_math.py
 ╰╮ TestMathFunctions
  ├─ test_addition
  ├─ test_subtraction
  ╰─ test_multiply

neotest-python
/tmp/parent/project2
╰─ test_strings.py

Expected behavior
The original tab with the summary window for project1/test_math.py shouldn't change when other tabs change their working directory with :lcd.
Logs
The neotest.log is empty during the above testing.

This is the intended behaviour. The summary window will show all tests that it is aware of, it does not filter based on the current directory

@rcarriga I should mention that this behavior also causes duplicates of the same project even when just doing :lcd ../, which gives me the following in the summary window:
neotest-python
project1
╰╮ test_math.py
 ╰╮ TestMathFunctions
  ├─ test_addition
  ├─ test_subtraction
  ╰─ test_multiply

neotest-python
├─ project1

Originally I thought discovery = { enabled = false } would have fixed this, but I understood the parameter wrong.
Do you think there will ever be a setting or toggle that be able to change this default behavior? Or perhaps a command to only explicitly discover tests at a user-inputted path?


add vscode-github integration support

Move existing vscode registry files into a subfolder
When one installs the vscode, copy reg files from bucket rather than download from web
Add registry for vscode-github integration support (inspired by https://github.com/lukesampson/scoop-extras/pull/3924)

Thank you for your feedback. I think it would be better if the changes are split into two pull requests: one for making reg files copied, the other for adding github integration.


Update ec2-amis.nix to add a new ami for eu-west-1

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Welcome to Nixpkgs — this PR should target the master branch and not release-18.03, and also all changes should be squashed into one commit.

Superceded by https://github.com/NixOS/nixpkgs/pull/361106


Please update to KSKrash 1.11.2

Hi,
can you please update the SDK to utilize KSKrash 1.11.2?
A lot of other packages have shifted to 1.11.2 and I can't compile with support
for Bugsnag anymore.
Thanks!
Markus

Thanks for the report, @markuswinkler. Working on it. 1.11.2 includes some breaking changes, which is why I assume other libraries have updated already as well.


[IOTDB-1372] delete TimeIndexLevel class and related methods in other classes

Coverage decreased (-0.03%) to 68.136% when pulling fc1434140db715e939b4c6d8b4f437ace07cd463 on yyt86:remove_device_set into 79e50d651a2b488ac211009947b2a01c5244c37b on apache:master.

What is this PR for?

The motivation?

The motivation and the effect?

After we submitted the PR, we realized those modifications are not valid for the current. I have reverted the changes in another commit of this PR. Thanks.

What is this PR for?

This PR has removed the devices filed of FileTimeIndex. And it will add an TsFileResourceManager class to deal with the degradation from DeviceTimeIndex to FileTimeIndex when the memory of timeIndex is larger than the threshold memory.


add support for Windows 7 build 6.1.7601.24326 (2019-02)

As I can't answer to the existing issue (https://github.com/stascorp/rdpwrap/issues/646):
https://github.com/fre4kyC0de/rdpwrap/blob/master/6.1.7601.24326.txt

https://github.com/stascorp/rdpwrap/issues/646

Here's an interim fix:
https://github.com/stascorp/rdpwrap/issues/669

That's a workaround. The fix is getting rdpwrap to work with latest termsrv.dll. thanks @fre4kyC0de will try this tomorrow. Would love to know how you figured it out. I tried following the YouTube without luck.

Yeah, fix to the lack of access, not to the software. I don't have nearly the knowledge to fix that!

Duplicate of #646.


Join channel?

How do i have the bot join a specific channel of mine?

@Dontmindmes Bots can't do anything on their own. They could only respond to inputs (messages, commands, inline queries)
You must add bot to channel manually from telegram client, like you add regular user or friend


Fix arguments that should be camel case.

Plus spelling fix.
See: https://github.com/qbittorrent/qBittorrent/wiki/WebUI-API-(qBittorrent-4.1)#delete-torrents
Or:
https://github.com/qbittorrent/qBittorrent/blob/5a1dcbae9c9746a0114955c4528a142b3b86f473/src/webui/api/torrentscontroller.cpp#L1035

Thanks for the PR!

No problem. Can you push a new release to crates.io ?

No problem. Can you push a new release to crates.io ?

Done. You should be able to see the new version now.


docs(provider/kubernetes): Add a guide for Patch(Manifest) stage.

@lwander @dorbin
spinnaker/spinnaker#2786

@dibyom
Now that we have a pipeline stages reference, you want to create an entry in there for this?
@nhayes FYI


Flat buttons don't look like buttons

Describe the bug
A flat button looks more like a link, and it might not be clear that they are actually buttons.
How to reproduce
https://clarity.design/documentation/buttons
Steps to reproduce the behavior:

View buttons documentation
View flat buttons

Expected behavior
Buttons should be visually consistent.
Versions
App

Angular: [e.g. 6]
Node: [e.g. 8.10.0]
Clarity: [e.g. 0.12.5]

Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional notes
Add any other notes about the problem here.

If a flat button is considered invalid or inaccessible, then there are other option button choices available.

The flat button is a pretty common component and is used in many products. Changing it will be visually breaking and will likely be significant for some products. Because of this and its common  presence in numerous UI systems, this should remain low priority and be reviewed by a11y - it appears that this is likely not a significant issue.

Closing this. My opinion was that flat buttons would not be visually distinguishable from other text making it difficult for sighted speech recognition users to visually perceive these controls as buttons but I have been assured that these will be styled in such a way that it is visually obvious these are buttons.


There is no way to leave a ipv6 multicast group

Describe the bug
It is possible for the coap_server to join a multicast group, however it is not possible to leave a multicast group. There is no implementation of NET_EVENT_IPV6_CMD_MADDR_DEL in ipv6_addr_event_handler in openthread.c.
Building the coap_server with Openthread support fails because of net_ipv6_mld_join. Replacing this method with net_if_ipv6_maddr_add and  net_if_ipv6_maddr_join seems to solve the problem.
An issue related to the 2 points above has been reported here: https://github.com/zephyrproject-rtos/zephyr/issues/17692.
To Reproduce
Steps to reproduce the behavior:

Build the coap_server sample with Openthread
Build process fails
Change net_ipv6_mld_join to net_if_ipv6_maddr_add and net_if_ipv6_maddr_join to be able to build the sample.
There is no way of leaving a multicast group

Expected behavior
I expected that there were a method to leave a certain multicast group.
Impact
I am not able to leave a multicast group.
Environment (please complete the following information):

Toolchain (e.g Zephyr SDK, ...)
Zephyr: v2.6.99-ncs1-rc2

@AllemanMa - are you able to reproduce this upstream against, e.g. v2.7.0-rc3?

@AllemanMa - are you able to reproduce this upstream against, e.g. v2.7.0-rc3?

I am developing with Nordic Connect SDK which does not have such a recent release yet of Zephyr, that's why I am still on v2.6.99-ncs1-rc2. When I have a look at the Zephyr repository on the master branch, I can see that there is still no implementation for NET_EVENT_IPV6_ADDR_DEL   in https://github.com/zephyrproject-rtos/zephyr/blob/e0f18128bfd23c99a060a5f931c1bcaa6afdab6a/subsys/net/l2/openthread/openthread.c#L117
Als no method like rm_ipv6_maddr_from_ot is provided in https://github.com/zephyrproject-rtos/zephyr/blob/e0f18128bfd23c99a060a5f931c1bcaa6afdab6a/subsys/net/l2/openthread/openthread_utils.c#L336 .
In the coap_server sample, there is no implementation for leave_coap_multicast_group available either.

It seems like join_coap_multicast_group() of the coap server sample can be replaced by otIp6SubscribeMulticastAddress and leaving a multicast address group by otIp6UnsubscribeMulticastAddress. These methods provide the functionality I was expecting.


Exclude some repos

repos such as onboard, confessions, etc shouldn't be shown here because the issues don't reflect action items that any community member can pick up and work on

Hmmmm, I'd kinda like to see onboard listed for this sort of task https://github.com/hackclub/OnBoard/issues/311
Maybe we just don't show the PR count?


Equal sign missing on geohot theme

https://github.com/Mayccoll/Gogh/blob/master/themes/geohot.sh#L31
There's a missing equal sign on this line, nothing special

Thank you


Cant Use My Scraper Credentials : Scraping LImited - no credentials provided Error

Hi Gabriel,
love the work you have done on this scraper and it's really amazing just one thing bothers me and maybe it's a stupid question but, I am not able to use my Screenscraper credentials and get full access to unlimited downloads, maybe i am inputting them in on an incorrect file/place?
Would you mind telling me how i can do that so i don't get the Scraping Limited error?
I am using MuOs Banana.
Thank you in advance for the help, i appreciate it,
1n3tinct5

Hi, I appreciate your words and thank you for opening an issue. Please check the "Adding Credentials" section in the Wiki, make sure they are added to skyscraper_config.ini
https://github.com/gabrielfvale/scrappy/wiki/Configuration#adding-credentials


mCurrentSwiper become null

I'm using SwipeHorizontalMenuLayout on ExpandableListView.
And I'm defining a SwipeHorizontalMenuLayout in GroupView and ChildView of ExpandableListView.
But, Occur NullpointerException when click a SwipeMenu(left or right) in a ChildView.
mCurrentSwiper in follow code got null 😢
public class SwipeHorizontalMenuLayout extends SwipeMenuLayout {

...
     public void smoothCloseMenu(int duration) {
             mCurrentSwiper.autoCloseMenu(mScroller, getScrollX(), duration);
             invalidate();
     }
...

Is this bug? or Not supported for ExpandableListView??
Let me know why, and want to solve this problem.
Best regards m_ _m

Sorry, not supported for ExpandableListView, i'm not test it.

I see, thank you for the response.

I will send pullrequest when i have enough time.


Create landing page design [easy/medium] - Instance #7

Is your feature request related to a problem? Please describe.
Create a design of the landing page using any simple design tool (say Paint)
Describe the solution you'd like
Your design should be appealing to the user and must be related to the idea of the Water monitoring system. You should try to put relevant content as required at different places in the design itself but it's okay if you cannot think of some for certain parts of the website. We can refine this later on.
Compare and create the PR into the feature/frontend branch, and not the master.
Additional context
Please don't include tank divs here. This info is only for the admins.
Time Limit: 3 days
PS: Difficulty level [easy/medium] will depend upon your effort and the look of the design.
PPS: Task will be assigned on an FCFS basis.

@amaaniqbal I would like to work on this issue. Please assign this to me.

@SaranshDawra sure!

Can i work on this issue?

please assign me this issue


Implement a library for building a basis and getting basis functions

This can go under qm7/basis.py

So this is done!


just says loading

hi i get this error in the extension  Errors
Alarm delay is less than minimum of 1 minutes. In released .crx, alarm "booster-json" will fire in approximately 1 minutes.
Context
_generated_background_page.html
Stack Trace
_generated_background_page.html:0 (anonymous function)
Nothing to see here, move along.
Alarm delay is less than minimum of 1 minutes. In released .crx, alarm "owned-games" will fire in approximately 1 minutes.
Uncaught SyntaxError: Unexpected token u in JSON at position 0
it also just says loading when ive added my details after clicking overview

Hi @GrizzlyKt ,
did you installed the extension recently? I'm asking because I updated a lot of files today and I'm not sure if I caused new problems to some users who are starting from scratch.
The extension-errors depending the timing errors can be safely ignored. Depending the overview, I'm just asking to ignore it. I'm trying to migrate to a better webinterface (just replace index.html with index2.html on the extension-tab to see what I mean) and it's useless for me to invest my time in things I want to get rid of. Just open another page but not the overview-page and reload the site and the loading-indicator should disappear.

Alright, quick and dirty fix, since booster-data couldn't be loaded w/o api-key -> replace your functions.js with the one I pushed to this repo now: https://raw.githubusercontent.com/Pandiora/SteamAccountRoboticAssistant/master/js/functions.js


Add port service identification functionality

Extend the script to not only detect open/closed ports but also attempt to identify the services running on the open ports by inspecting banners or sending protocol-specific probes.

please assign this to me

@Rishika631, I am assigning this issue to you. Feel free to submit a PR.

I have raised the PR for this.


Style format remarklint

Description of the change

Screenshots

Checklist

[ ] Added description of the change
[ ] I've read the contributing guidelines
[ ] Search previous suggestions before making a new PR, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes (optional):

This PR is all about switching from markdownlint to remark?

I don't like these changes at all. Markdown Linter is better, IMO.
Let's see what the other maintainers think about this, though. 🙂

I don't like these changes at all. Markdown Linter is better, IMO. Let's see what the other maintainers think about this, though. 🙂

Remark is the recommended linter by codacy, I think the people at codacy know better about styling and stuff..
Plus, if we don't like these changes in the future we can revert them back any time 🙂

Also, note: Codacy build will always fail since settings need to be changed after this pr is merged

Hmm, I'm not aware of remarklint. Maybe some maintainer can also review

Not sure why do we need to care so much about the formatter for markdown 🤔 I think remark is fine, else we can switch to markdown lint but it is a ruby package and just makes the contribution process harder 😕. For @Panquesito7's thought, it is not the linter it is the config, feel free to change the config 🙂

Hmm, I'm not aware of remarklint. Maybe some maintainer can also review

Not sure why do we need to care so much about the formatter for markdown 🤔 I think remark is fine, else we can switch to markdown lint but it is a ruby package and just makes the contribution process harder 😕. For @Panquesito7's thought, it is not the linter it is the config, feel free to change the config 🙂

I have no idea about format remarklint. Can we implement and see changes, incase we don't like the practice, we can revert back? 🤔

Hmm, I'm not aware of remarklint. Maybe some maintainer can also review

Not sure why do we need to care so much about the formatter for markdown thinking I think remark is fine, else we can switch to markdown lint but it is a ruby package and just makes the contribution process harder confused. For @Panquesito7's thought, it is not the linter it is the config, feel free to change the config slightly_smiling_face

I have no idea about format remarklint. Can we implement and see changes, incase we don't like the practice, we can revert back? thinking

Same +1

For me, the Markdown linter is already good, I don't see a need to change. I think we're good as-is, it doesn't affect anything, IMO. 🙂

For me, the Markdown linter is already good, I don't see a need to change. I think we're good as-is, it doesn't affect anything, IMO. 🙂

At first, I was implementing markdown linter but it has some weird requirements :/ You can't make a line have more than 80 chars and more like there can't be inline HTML... That is why I switched to using remark. Moreover, remark is the preferred linter by codacy and most JS projects 🙂

@vinzvinci awaiting your approval...


Update test reporter

Sorry 😬 I clicked the button too quick on this one, I wanted to merge this into my own branch


Task list for DIGR team members for the ARM Summer School

[ ] Create an environment file for a nice Python env for ARM data @scollis
[ ] Create a "How to install a python stack" @scollis

Group 1 tasks

[ ] Create a notebook on KAZR VPT reading and plotting and a little playing around @zssherman
[ ] write code in python for reading in KAZR spectra data. I suggest X-Array. It would be very cool if we could write code to take moments of the spectra data and then populate a Py-ART radar object so we can use the VPT visualization. @rcjackson

Group 5 tasks

[ ] write RST file with instructions on downloading: MWRRET (LWC), TSI, ARSCL and doppler lidar
[ ] Write a notebook visualizing each for a sample day

Group 2 tasks:
* Write a notebook to read in ISDAC microphysical ice size distribution data and CSI bulk water contents from the Convair-580 @rcjackson


How to add EventListener to ui-template?

Sorry if this is been doced somewhere and I didn't find it.
But I would like to add a EventListener to get the current status of a window (browser tab) is activated or not
window.addEventListener('blur', function() {this.send({payload: {browserActive: false}})}), // lost focus
window.addEventListener('focus', function() {this.send({payload: {browserActive: true}})}), // focus

Sorry I have no idea of where to put this code. I've try inside the watch: or mounted() {} section, but nothing works.
Main purpose for this, is I would like to fetch real time data when browser tab is opened (or activated and focused), and stop fetching data when blur (or deactivated). This is useful if I have fetching a lots of data from a server (such as cpu, ram, disk usage info etc..) only if I have open the dashboard page, then start fetching, and when the page is in background, stop fetching from server.
I've already managed to start/stop fetching data using ui-control node when browser tab is created and closed, but I have no idea how to deal with blur/focus event. Please help! Thanks guys!

after I put this into mounted(), now event starts to work
sorry for the spawn, close it


[BUG] 混用langchain的tool和自定义tool时，tests/agent/test_agent_function.py会报错

问题描述 / Problem Description
混用langchain的tool和自定义tool时，tests/agent/test_agent_function.py会报错 ，说langchain的tool不是Tool
复现问题的步骤 / Steps to Reproduce

在server/agent/tools_select.py中增加
Tool.from_function(
func=search_youtube,
name="search_youtube",
description="use this tools to search youtube videos",
args_schema=YoutubeInput,
),
HumanInputRun(input_func=get_input),#-------》增加的tool  from langchain.tools.human import HumanInputRun
]
运行tests/agent/test_agent_function.py
问题出现 / Problem occurs
Traceback (most recent call last):
File "E:\Langchain-Chatchat\tests\agent\test_agent_function.py", line 25, in
prompt=CustomPromptTemplate(
File "E:\Programs\Python\Python38\Lib\site-packages\langchain_core\load\serializable.py", line 97, in init
super().init(**kwargs)
File "pydantic\main.py", line 341, in pydantic.main.BaseModel.init
pydantic.error_wrappers.ValidationError: 1 validation error for CustomPromptTemplate
tools -> 8
init() missing 1 required positional argument: 'func' (type=type_error)

预期的结果 / Expected Result
自定义的tool可以与langchain的tool混合使用
附加信息 / Additional Information
通过修改server/agent/custom_template.py可以正常使用
from langchain_core.tools import BaseTool
from configs import SUPPORT_AGENT_MODEL
from server.agent import model_container
class CustomPromptTemplate(StringPromptTemplate):
template: str
tools: List[BaseTool]#-------》原来是Tool
**最终结果
测试可以使用自定义Tool与langchain的stand tool混合使用
部分结果如下：
text_answer = agent_executor.run("我的朋友有几个？")
'''

Entering new AgentExecutor chain...

Entering new LLMChain chain...
Prompt after formatting:

Answer the following questions as best you can. If it is in order, you can use some tools appropriately.You have access to the following tools:
calculate: Useful for when you need to answer questions about simple calculations
arxiv: A wrapper around Arxiv.org for searching and retrieving scientific articles in various fields.
weather_check:
shell: Use Shell to execute Linux commands
search_knowledgebase_complex: Use Use this tool to search local knowledgebase and get information
search_internet: Use this tool to use bing search engine to search the internet
Wolfram: Useful for when you need to calculate difficult formulas
search_youtube: use this tools to search youtube videos
human: You can ask a human for guidance when you think you got stuck or you are not sure what to do next. The input should be a question for the human.
Please note that the "知识库查询工具" is information about the "西交利物浦大学" ,and if a question is asked about it, you must answer with the knowledge base，
Please note that the "天气查询工具" can only be used once since Question begin.
Use the following format:
Question: the input question you must answer1
Thought: you should always think about what to do and what tools to use.
Action: the action to take, should be one of [calculate, arxiv, weather_check, shell, search_knowledgebase_complex, search_internet, Wolfram, search_youtube, human]
Action Input: the input to the action
Observation: the result of the action
... (this Thought/Action/Action Input/Observation can be repeated zero or more times)
Thought: I now know the final answer
Final Answer: the final answer to the original input question
Begin!
history:
Question: 我的朋友有几个？
Thought:
2023-12-18 11:03:22,335 - _client.py[line:1013] - INFO: HTTP Request: POST http://<IP_ADDRESS>:20000/v1/chat/completions "HTTP/1.1 200 OK"

Finished chain.
这个问题是关于你的朋友的数量，所以我们需要从你的社交媒体账户或者联系人列表中获取这个信息。

Action: human
Action Input: "你能告诉我你有多少个朋友吗？"
你能告诉我你有多少个朋友吗？
Insert your text. Enter 'q' or press Ctrl-D (or Ctrl-Z on Windows) to end.
小李，小王，小林都是我的朋友
q
Observation:小李，小王，小林都是我的朋友

Entering new LLMChain chain...
Prompt after formatting:

Answer the following questions as best you can. If it is in order, you can use some tools appropriately.You have access to the following tools:
calculate: Useful for when you need to answer questions about simple calculations
arxiv: A wrapper around Arxiv.org for searching and retrieving scientific articles in various fields.
weather_check:
shell: Use Shell to execute Linux commands
search_knowledgebase_complex: Use Use this tool to search local knowledgebase and get information
search_internet: Use this tool to use bing search engine to search the internet
Wolfram: Useful for when you need to calculate difficult formulas
search_youtube: use this tools to search youtube videos
human: You can ask a human for guidance when you think you got stuck or you are not sure what to do next. The input should be a question for the human.
Please note that the "知识库查询工具" is information about the "西交利物浦大学" ,and if a question is asked about it, you must answer with the knowledge base，
Please note that the "天气查询工具" can only be used once since Question begin.
Use the following format:
Question: the input question you must answer1
Thought: you should always think about what to do and what tools to use.
Action: the action to take, should be one of [calculate, arxiv, weather_check, shell, search_knowledgebase_complex, search_internet, Wolfram, search_youtube, human]
Action Input: the input to the action
Observation: the result of the action
... (this Thought/Action/Action Input/Observation can be repeated zero or more times)
Thought: I now know the final answer
Final Answer: the final answer to the original input question
Begin!
history:
Question: 我的朋友有几个？
Thought: 这个问题是关于你的朋友的数量，所以我们需要从你的社交媒体账户或者联系人列表中获取这个信息。
Action: human
Action Input: "你能告诉我你有多少个朋友吗？"
-------------------->这里输入------->小李，小王，小林都是我的朋友
Observation: 小李，小王，小林都是我的朋友
Thought:
2023-12-18 11:04:14,263 - _client.py[line:1013] - INFO: HTTP Request: POST http://<IP_ADDRESS>:20000/v1/chat/completions "HTTP/1.1 200 OK"

Finished chain.
看来你的朋友数量是3个。

Final Answer: 3

Finished chain.

Process finished with exit code 0
'''

是可以正常使用的，你新建一个文件写自己的工具然后注册就行，wiki有明确的写法


[Enhancement]: Pipeline and operator updates should be automatically downloaded even if a cached version exists

Is there an existing issue for this?

[X] I have searched the existing issues

What would you like to be added?
If a repository (either pipeline or operator) on the Towhee hub is updated, the changes are not automatically reflected during local runs if a prior version of said repository exists.
Why is this needed?
This seems to be unintended behavior.
Anything else?
No response

Now replace download with git, every time we run a pipeline or call an operator, towhee will automatically get the latest version on your branch.


Refresh the browser if runtime service is unavailable but the http server is

Adding a temporary fix to runtime service being unavailable. This happens only when running start from the cli.
UI is refreshed when it sees that the service is unavailable.

We will ping the grpc server instead.


spring-cloud-gateway  Unified exception handler

I use spring-cloud-gateway in Microservice, sub-service throws exceptions，i want to handle these exceptions in spring-cloud-gateway,What should I do?

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Redirected to MDN (mozilla) from docs.microsoft.com

Describe the bug
I click on the APPLICATIONNAME or applicationName links at the HTA:APPLICATION Element | HTA:APPLICATION Object page (https://docs.microsoft.com/en-us/previous-versions/ms536495(v=vs.85)) and am re-directed to MDN (mozilla)
To Reproduce
Steps to reproduce the behavior:

Go to https://docs.microsoft.com/en-us/previous-versions/ms536495(v=vs.85)
Click on APPLICATIONNAME or applicationName
You are at MDN, not MSDN where the link is supposed to take you

Expected behavior
I expect to see the correct info for APPLICATIONNAME from MSDN.
Desktop (please complete the following information):

Windows 10
Firefox 69

I am guessing someone put a "mdn" where a "msdn" should have been.

Redirection of Web API documentation from MSDN to MDN was announced on October 18, 2017.
However, because MDN does not document the HTA-specific applicationName property, it seems there should be an exception for that, like there already is for commandLine.


Screen Markers: Text boxes and arrows

I been using screenmarkers at the blast furnace recently for the first time but feel that a little more can be used to improve it.
I was thinking of Text boxes and arrows for example at the blast furnace you will be doing a continuos loop and for people like me who have issues remembering what each box is for maybe have an option to also add in on screen text to where you can input for example Ore and Coal above or in the boxes or even wherever you want.
Then there will be arrows much like some paint programs have just draw a line  and boom there's an arrow for for you. which would be useful for helping staying on the path of the the loop of the grind. which can be quite be quite difficult for some people.

I dont think we want to implement anything else other than just square markers, because that is what 99% of people use. for various other stuff you can just use external software

I'd understand if this kept the wontfix tag, but as part of that 1%, just thought I'd chime in on why I'd find this request useful.  I'm still new to using F-keys, and have a bad habit of scrambling to click on the tab I need instead of using them in panic situations (such as when learning a new boss, or defending myself in the wilderness).  To help reinforce the usage of them, I thought I'd throw an opaque screen marker over the tab icons and label them with the corresponding keys.  I can do half of that now and will be, but I was a little disappointed I couldn't label them.  Another niche situation, and I'll hopefully learn it soon regardless, but it is what it is.  Thanks again


Packagist Hook seems to be inactive/broken

Packagist still gives 4b5511e as latest commit for dev-master.
I think your hook might be inactive or broken

well, seems like it was only quite a bit delayed, now it's up to date


Add Dreamdrive64 Support

This adds support for Dreamdrive64 rom cart. It's using a different address space from Picocart64, so I took a different abbreviation.

Hi, thanks for the PR. First thing before review: is it possible to relicense the code using the Unlicense license that libdragon uses? I wouldn't want to bring in code that adds attribution constraints to our users like the BSD license does.

No problem. I'm used to submitting PRs internally at my previous companies 😅

Wow thanks for the though review! It's been a while and I let my code standards lapse 😅 I'll set about getting this fixed up. Not sure what is up with the whitespace, I'll run a formatter and see if I can sort that out.

Weep for the tab to space changes.
Anyway I think I addressed all the feedback. Again, I'm really sorry about the whitespace, I know it's annoying to review code that's basically all whitespace changes.

Hi @khill25, let me update you on this PR.
Your code was sent as two major changes were being worked on. One (just landed) is #366 that is a big improvement to usb.c (coming from upstream). Another one, waiting for last touches, is #371 that basically removes the current SD code in favor of vendoring libcart, which is a newly released library which is more comprehensive that our builtin SD support. Our goal is not having to maintain the SD code anymore in here, and offload that to libcart.
Your PR touches the same areas at the same time, so it is unfortunately due to some rework once also #371 lands. My suggestion would be to wait until #371 lands and then I can point you to the better way of reworking your code with the new overall architecture. That will probably be adding support for Dreamdrive64 to libcart rather than libdragon proper, but let me see where we end with #371 first.
Thanks for the patience!

Hi @khill25, the libcart PR (#371) was merged. This means that SD flashcart code is now maintained at upstream libcart which is where you should send your PR now. I will close this PR now as we won't be able to merge it as-is, as we don't carry SD flashcard code anymore.


Simplify crc status: no virtual machine is equivalent to virtual machine stopped

(opening a discussion)
After a fresh install, crc status fails with the error "Machine does not exist. Use 'crc start' to create it".
More annoying: the tray is showing Unknown status.
I have the feeling we could say that the VM is stopped. This is not fundamentally false after all.
$ crc status
CRC VM:          Stopped
OpenShift:       Stopped
Disk Usage:      0B of 0B (Inside the CRC VM)
Cache Usage:     39.55GB
Cache Directory: /home/guillaumerose/.crc/cache

Perhaps we could introduce a new state ? Like Inexistent, PendingCreation.
What should the API answer ?

Currently the daemon throws 500 errors in the case of no VM. I believe this is wrong, as the daemon is able to process the request, just the success should either be false, with an error message, or success, with a clear state of NoVM or Stopped.

We did that but only for the API.
The CLI now the difference between Stopped and no VM.


Allow sending a query by pressing enter

Sending the query via enter doesn't work anymore (tested in chrome).
@sfakir didn't you already implement this?

@tholor  yes was alraedy implemented, but looks like Julia removed this in the commit adding the brain. Not sure why, I'll add this again

Ah I think this is connected to the autocomplete, since enter selects now the "selected autocomplete" value.
I  can ask Julia if we can detect if an autocomplete value is selected and if not do a regular submit (or try it myself.)

This seems kind of solved, closing it now.


Overhaul FAQ

The FAQ is very outdated at this point and does not actually reflect the most frequently asked question s we get. My quick summary of the actual most commonly asked questions:

How do I set axis ranges?
How do I set color ranges?
How do I export plots to files?
How do I convert plots to bokeh/matplotlib figures?

I'm sure there are others but that's the ones that immediately came to mind.

HoloViews Cheat Sheet #2395 has a list of topics that might find their way here

@ahuang11 did a great job overhauling the FAQ so I consider this addressed.


DBZ-4665 Extend Debezium Schema Generator

finish renaming of API/APIFormat occurences to use Schema instead
add creation of directory structure per connector when creating descriptors: (postgres.json --> postgres/postgres.json)
add configuration to add a schema filename prefix (postgres.json --> debezium-postgres.json)
add configuration to add a schema filename suffix (postgres.json --> postgres-1.9.0-SNAPSHOT.json)
add some minor logging (available schema formats, selected format, processed connectors) and throw errors on failed state (no schema formats found, no connectors found)
stop calling the Schema Generator in a separate Java process when called as Maven Plugin
remove Oracle connector schema generation as it is not annotated yet

closes https://issues.redhat.com/browse/DBZ-4665

@ani-sha, no, not yet.

@ani-sha @gunnarmorling Looks like actually Chris merged the PR last night LOL

@gunnarmorling I addressed your comments!

reomved fat jar build with dependencies
removed logback in favor of java.util.Logging

Downstream schema build works, but is there any other way than adding all connectors plus org.apache.kafka.connect-api as "provided" dependencies in the pom.xml?
I just realize that a change in the schemabuilder module don't trigger connector builds. Should a change here trigger a connector build (as integration test)?

Downstream schema build works, but is there any other way than adding all connectors plus org.apache.kafka.connect-api as "provided" dependencies in the pom.xml?

See inline. Adding connectors seems reasonable, KC API could become a plug-in dependency.

I just realize that a change in the schemabuilder module don't trigger connector builds. Should a change here trigger a connector build (as integration test)?

Yes, that's true indeed.  Should not be needed any longer?


Python 3.4 and pip 19.0.3 fails to install isort 4.3.9

Complete output from command python setup.py egg_info:
/usr/local/lib/python3.4/distutils/dist.py:260: UserWarning: Unknown distribution option: 'python_requires'
warnings.warn(msg)
error in isort setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers

python 3.4
pip 19.0.3

python 2.7 also fails...

I can only reproduce with setuptools==12.0.5 (got idea from https://github.com/jazzband/django-axes/issues/349#issuecomment-433215726). Please, update your setuptools to version 20+ (https://stackoverflow.com/questions/21082091/install-requires-based-on-python-version/32643122#32643122).

Filed a related issue at #853. The install_requires parameter in the setup() call was removed from release 4.3.7 but it has come back in release 4.3.9 which is one of the factors triggering this issue. In issue #853, I have asked if this install_requires parameter was brought back intentionally in 4.3.9.
By the way, we have a different but similar build failure in cloudmarker/cloudmarker#45 due to an old version of setuptools, i.e., setuptools 35.0.2, in Travis CI build environment for Python 3.4. The issue does not occur with setuptools 36.2.0 and later versions. Therefore the issue does not occur in setuptools 40.0.0 and setuptools 38.2.4 in Travis CI build environments for Python 3.5 and Python 3.6, respectively.

Unless I get strong feedback otherwise, I'm going to keep this as invalid and simply require everyone to update their setuptools version - simply because it is not possible to list these optional requirements in a consistent way otherwise when dealing with wheel distribution AFAIK.
@susam, 4.3.7 & 4.3.8 also included the syntax in question, the tags were simply created incorrectly due to a typo on my part. I've updated the tags to be accurate.
Thanks everyone!
~Timothy


split=True on model predictions for GroupRidgeCV does not return predictions for each input feature set?

Hi,
Thanks for this fantastic code. It was really easy for me to get started with it, and it's extremely fast. I couldn't find if this is intended for public use yet and so I apologize if it is not.
For this code:
from himalaya.ridge import GroupRidgeCV
import numpy as np
X=np.random.rand(100,100)
X0=np.random.rand(100,100)
Y=np.random.rand(100,5)
model=GroupRidgeCV(groups='input')
model.fit([X,X0],Y)
print(model.predict([X,X0],split=True).shape)

This returns (100,5). I was expecting a list size 2 of [100x5] predictions, one for each of [X,X0]?

Was able to fix this by installing from source rather than pip.
Could be worth updating the pip version?

Glad that you like this package !
It has been recently made public, so you are very welcome to use it. I will try to make it clearer on the readme.
I will also push a new version to pypi, thanks for the suggestion.
Do not hesitate if you have other comments or questions.


Yet, it has problems with Ngrok.

It has no ngrok yet, you are talking about some invalid app, closing here...


helm/exporter-node : add args and volumes configurability to the node-exporter

i need more configurability at the exporter-node level to enable collectors and ignore filesystem mountpoints.
this PR is not a breaking change, and allows one to add args and volumes to the node-exporter container's spec
a way of using it would be to add something like the following yaml under the exporter-node key of kube-prometheus's values.yaml file
extraVolumes:
- name: rootfs
  hostPath:
    path: /

extraVolumeMounts:
- mountPath: /rootfs
  name: rootfs
  readOnly: true

extraArgs:
- --collector.filesystem.ignored-mount-points="^/(sys|proc|host|dev)($|/)"

you're welcome @gianrubio . thanks for merging it !


[BRD-159] - Primary color

Primary color calculation using tint weight = 89

Pull Request Test Coverage Report for Build 751528650

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 751053355:
0.0%

Covered Lines:
52

Relevant Lines:
52

💛  - Coveralls


NVAccess.NVDA version 2022.1

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@leonardder I think you should re-open these PRs. Currently the publish and validation pipelines are unhealthy and that is most likely why you are seeing these errors. I believe these all will pass the validation test once the pipelines status is healthy.

Ah, I will leave one open and reopen the other ones as soon as one got
validated correctly. Thanks for the heads up!


Add Ruby 3.1 and Rails 7 to the CI matrix

In addition to adding Ruby 3.1 and Rails 7 to the CI configuration, this PR:

Creates a Rails 7 gemfile
Loosens the restriction on activemodel in the gemspec to allow Rails 7
Quotes the instances of 3.0 in the CI configuration.  Unquoted, these are truncated to '3', and the latest Ruby 3 - at this time Ruby 3.1.1 - is loaded.  To ensure a Ruby 3.0.x version is loaded, these should be quoted.

Screenshot showing that Ruby 3.1.x is being loaded for the Ruby 3.0 entries in CI:

@adzap let's merge this

Hi, isn't this a partial duplicate of #213 ?
#213 also contains a fix for a Rails 7 deprecation

@tagliala There's some overlap.  But this PR both adds Ruby 3.1 and fixes the broken Ruby 3.0 entries in the CI matrix.
I've pushed a version of the deprecation fix to this branch.

Is there someone with merging permissions on this repo?
This PR supersedes #218, #214, and #213 and closes #217, #221, and (partially) #222
I'm asking before making a temporary fork with the changes needed to add this gem in a Rails 7 project

@tagliala - I'm in a similar boat; planning to just run off of @petergoldstein's (or rather, a fork from that branch). Will report back with how it works on a Ruby 3.0 + Rails 7.0 production application in a couple weeks.

@adzap would you be willing to designate Peter Goldstein as a backup maintainer here so we can get a fixed release out?

@zspencer feel free to use this branch: https://github.com/tagliala/validates_timeliness/tree/rails-70
I've just protected it and I'm using it in an application
Also, it is based on this PR, plus the usual validates_timeliness approach (= remove 6.x compatibility)

I appreciate the offer, but from a supply-chain security perspective, I tend to prefer deploying my own forked branch over relying on someone else's so that deploys don't accidentally pull in un-audited code.

Thanks but there was too much in this PR and it does not conform to the convention of a major version per ActiveRecord major version, so pushed necessary changes in other commits.


feat(plugins): onInstall hook

TODO

[ ] tests
[ ] runtime validation on 1) the handler type 2) the handler return value.

I will write tests tomorrow morning.

For interface consideration. Here is how nexus-prisma case is looking.
/**
 * Create a nexus-prisma plugin to be passed into the Nexus plugins array.
 */
export const create = (options: Options): Plugin => {
  return hooks => {
    hooks.onInstall(nexusBuilder => {
      return {
        types: Builder.build({ ...options, nexusBuilder }),
      }
    })
  }
}

Alternative Styles
s/hooks/lifecycle:
/**
 * Create a nexus-prisma plugin to be passed into the Nexus plugins array.
 */
export const create = (options: Options): Plugin => {
  return lifecycle => {
    lifecycle.onInstall(nexusBuilder => {
      return {
        types: Builder.build({ ...options, nexusBuilder }),
      }
    })
  }
}

Imperative style write-capable nexusBuilder facade:
/**
 * Create a nexus-prisma plugin to be passed into the Nexus plugins array.
 */
export const create = (options: Options): Plugin => {
  return lifecycle => {
    lifecycle.onInstall(nexusBuilder => {
      Builder.build({ ...options, nexusBuilder })
    })
  }
}

Class Based:
/**
 * Create a nexus-prisma plugin to be passed into the Nexus plugins array.
 */
export class NexusPrismaPlugin extends NexusPlugin {
  onInstall (nexusBuilder) {
    return {
        types: Builder.build({ ...options, nexusBuilder }),
      }
  }
}

Class Based + Imperative style write-capable nexusBuilder facade:
/**
 * Create a nexus-prisma plugin to be passed into the Nexus plugins array.
 */
export class NexusPrismaPlugin extends NexusPlugin {
  onInstall (nexusBuilder) {
    Builder.build({ ...options, nexusBuilder })
  }
}

I think the POJO approach is better. I do not see the benefit of a closure that receives lifecycle. It just makes things a bit more complicated I feel.

@tgriesser happy with how this is looking, and IIRC it fully aligns with your original gist https://github.com/prisma-labs/nexus-prisma/pull/434/files#diff-f41e9d04a45c83f3b6f6e630f10117feR10-R18. So going to mark the above "alternatives" comment as resolved. Feel free to reopen the discussion though!

Milestone reached on sister nexus-prisma pr, fully working atop this version of Nexus.

Hey @tgriesser I think I've responded to all your feedback. There is at least one needing your follow up (the allTypeDefs data).


Configure LED to indicate when servo is active

Description:
Set up the onboard LED to light up whenever the servo motor is running. The LED should turn on when the motor is powered and turn off when the motor is stopped. Also set the LED green when opening and red when closing.
Tasks:

Implement logic for linking LED with servo state.
Ensure the LED turns off when the servo stops.
Test and verify that the LED color and state reflects the servo's state.

Acceptance Criteria:

The LED lights up green when the servo is opening/turning on and red when closing/turning off.
The LED turns off when the motor is stopped.
The system works correctly and LED configuration does not interfere with motor functionality.

The corresponding green and red lighting work with their respective action but the speed of light going off is delayed as the motor has delay code after servo is set to stopping pulse. This is currently set as the motor will not step without this delay code.


Unable to access the local database

Logs: https://gist.github.com/giacgbj/84dd509c2c4916e18b7bfc9827b13d0a

I've run the job many times before posting this issue
I'm using an internal mirror which works
the directory /home/jenkins/workspace/FooProject/dependency-check-data is writeable (775) by the user which launches Jenkins and it's removed each time the job is executed
the dbs permissions are:
-rw-rw-r-- 1 jenkins jenkins 2107392 Aug  5 22:19 dc.h2.db
-rw-rw-r-- 1 jenkins jenkins     116 Aug  5 22:19 dc.lock.db
-rw-rw-r-- 1 jenkins jenkins   43174 Aug  5 22:19 dc.trace.db

sounds a little bit like https://github.com/jeremylong/DependencyCheck/issues/100 to me.
https://github.com/jeremylong/DependencyCheck/issues/100#issuecomment-38585732 -> To recover from your current situation, you can either delete the whole data directory, or you might be able to just delete the .lock file in the data directory.

If the .lock file is in "dependency-check-data" directory, that directory is removed before each build.

Ensure you have auto update enabled in the job. If you're removing the project workspace for each build, autoupdate will need to be enabled so it can rebuild the database each time. Assuming that is done, do you have either of these plugins installed?

https://wiki.jenkins.io/display/JENKINS/Pipeline+Maven+Plugin
https://wiki.jenkins.io/display/JENKINS/Liquibase+Runner

These potentially could cause conflicts as they use beta versions of H2 instead of the stable 1.3 branch that Dependency-Check and the Jenkins H2 plugin uses.

Having a separate job that updates the database regularly is a massive performance Improvement as far as I know.
See E.g. https://medium.com/@PrakhashS/checking-vulnerabilities-in-3rd-party-dependencies-using-owasp-dependency-check-plugin-in-jenkins-bedfe8de6ba8

@stevespringett Autoupdate is enabled (from the gist, i think it's the autoupdate='true' property).
"Unfortunately", I don't have those plugins installed.
@rompic I know I can do that (and I will), but it wouldn't solve this issue. It would be only moved to a single job and all the others would probably get a NoDataException.

It still fails randomly.
Could it be a concurrency issue?
In the logs I see:
#download   threads: 12
#processing threads: 8

Are there a large number of builds on this server? I'm not sure how you have things setup - but is it possible that one job is deleting the database directory while another job is using it?

That's not possible: the builds are sequential. The database directory ("dependency-check-data", I suppose) is different for each job and is always wiped out before each build.

This is believed to be fixed in the 3.0.0-SNAPSHOT - unfortunately, you would have to build that yourself for now to test.

is this part of 3.0.1? still see the same issue

@rompic yes, this should have been fixed in 3.0.1. If after deleting the data directory and re-running ODC if you are still seeing this issue please open a new issue.


Update main README.md

The main README is a bit outdated.

We will change testnet.miden.io, for now I would merge those changes. Can you merge @bobbinth ?


When creating a bundle and then using gulp watch your dist folder is cleaned out

So I've spent the past 3 hours trying to figure out why I couldn't get gulp bundle to work properly.
After struggling with just executing the task, I finally got gulp bundle to run.  And then I would try to see if it worked, I would get a 404 on aurelia.js.
The solution was hosting in IIS, and not using gulp watch to host my aurelia app. Then after I did gulp bundle, I could see my site without gulp watch.
I'm not sure what the solution would be here, but it was super frusterating at very least some better documentation around bundling would be awesome.
thanks!

running gulp watch cleans the dist and all the bundles with it.
just run gulp watch and in another terminal run gulp bundle, refresh the page and see the result.

Sorry for the trouble you have gone through. We have included this in our documentation. Basically, Build -> Bundle -> Deploy is the workflow we tend to follow.


Add jsonpb marshallers

Hi! I think your pull request, but would you mind filling in some context what issue are you fixing here?

Certainly. When I tried to programmatically create the subscription document, I ran into the problem that the core was unable to unmarshall the configuration due to the final json being incompatible. So i added marshallers that make the output json valid for subsequent deserialization by the core.


Duplicate sensitive buffer and buffer length information

This PR supersedes and is a subset of https://github.com/ARMmbed/mbedtls/pull/3434.
The original code is available here: https://github.com/AndrzejKurek/mbedtls/tree/fi-duplicate-buffers.

ROM statistics generated by running:
./scripts/baremetal.sh --rom --gcc:
compared to a run from before this PR:
libmbedcrypto.a: +0 Bytes
   libmbedx509.a: +0 Bytes
       libmbedtls.a: +168 Bytes
                   total: +168 Bytes

@piotr-now , please re-review this PR

@shelib01 - I've done a miscalculation before, not including one of the commits when calculating codesize change.
I'll describe it in detail now:
Stats on commit 98c847 (before this PR) vs 4ed275 (latest commit):

Commit
98c847
4ed275
delta

libmbedcrypto.a
18983
19417
434

libmbedx509.a
4805
4805
0

libmbedtls.a
16801
17005
204

total
40589
41227
638

I can break it down into commits further if you'd like.

This PR will be ready to review once the CI passes.

I fixed the one line mentioned by @shelib01, but left the full context freeing on hmac and sha256 changes. For sha, all of the operations return immediately if an error is found, and for md_hmac (located in md.c) - there are checks for a null ctx.
AES and tinycrypt-related changes only clear buffers, not contexts.
This is now ready for review, @piotr-now, @shelib01.

@shelib01 - I've updated the CCM module with requested changes.


Aborting a request midflight

Any plans to support aborting a request midflight?

jQuery just uses the underlying XMLHttpRequest's abort method:
var jqxhr = $.get('api/foo');

jqhxr.abort();

Angular uses this weird timeout promise dance:
var aborter = $q.defer();

$http({
	method: 'get',
	url: 'api/foo',
	timeout: aborter.promise,
});

// This aborts the request:
aborter.resolve();

Aborting a request midflight is very important in countless scenarios. For example in a typeahead, where - in addition to debouncing/throttling the actual request - you'd want to cancel the existing request when the user starts typing again.

Yes, it's possible to add a abort() function to the promise object, which the HTTP request function returns. This will allow a syntax similar to jQuery.

@steffans that's great news! Always prefer the pragmatic approach over some ideal dream of architectural purity :smile:

The way it currently works will have to be refactored a bit.
We now don't have access to the xhr outside the resolver function. If we are to augment the promise with an abort function, we'll need access to the underlying xhr object.

Any examples about how to use this feature? I'm trying to use it but get abort is not a function error.

You could use the beforeSend callback to access the request object and a add your code to abort the current request.
this.$http.post('url', function (data) {}, {beforeSend: function(xhr) {
    // use XMLHttpRequest and add xhr.abort callback
}});

Nice, it works! I ended with something like this:
beforeSend(xhr) {
    if (globalXhr) {
        globalXhr.abort();
    }

    globalXhr = xhr;
}

@sahibalejandro I try this.The beforeSend's arguments is a request object,But it's property xhr is null

@WilliamDu1981
I've just had the same issue and looked at the request object, it has a cancel function which aborts the request behind the scenes for you. The code below is working for me
beforeSend(xhr) {
    if (this.lastRequest) {
        this.lastRequest.cancel()
    }
    this.lastRequest = xhr
},

@BrockReece I tried your code:
this.$http.get('/users',{},{
        beforeSend(xhr){
          debugger;
          if (this.lastRequest) {
            this.lastRequest.cancel()
          }
          this.lastRequest = xhr

        }
      });

but throw an error below:
App.vue?0940:60 Uncaught (in promise) TypeError: this.lastRequest.cancel is not a function(…)

the xhr object has not a method named cancel

I don't know sorry.
The source shows the object should have a cancel method
https://github.com/vuejs/vue-resource/blob/a27faffb8dbf7a7d7a7aaa8e3792e93f48a8a8c5/src/http/client/xhr.js#L14
Can you console.log the lastRequest obj?

@BrockReece I've watched expression and screenshots

@WilliamDu1981 This is my code that works, what version of vue resource are you using?
this.$http.post(this.url, data, {
    beforeSend(xhr) {
        if (this.lastRequest) {
            this.lastRequest.cancel()
        }
        this.lastRequest = xhr
    },
}).then(response => {
...
}

@BrockReece I install it via npm,It should be the newest version。
I find the code in xhr.js:
var xhr = new XMLHttpRequest(), response = {request: request}, handler;

        request.cancel = () => {
            xhr.abort();
        };

I add breakpoints in this code and beforeSend,I found it execution after beforeSend,so when the beforeSend execution,the cancel method is not defined yet;


Cost Optimization Workbook / Add a new tab for change log

📝 Scenario
Users should be able to tell what are the new COW queries.
💎 Solution
Add a new "Change Log" tab under the Overview main tab.
📋 Tasks
### Required tasks
- [ ] TODO: Create a new tab under the Overview main tab.

### Stretch goals
- [ ] TODO: Add tasks

ℹ Additional context
🙋♀ Ask for the community

We could use your help:

Please vote this issue up (👍) to prioritize it.
Leave comments to help us solidify the vision.

@arthurclares Is there a way we can reuse or enhance the current changelog to support this without duplicating things?


Update to overwrite variables for tests

Signed-off-by: Tim Liu<EMAIL_ADDRESS>

@wjxiz1992

@GaryShen2008 @wjxiz1992  Can you help to review? thanks!


Use external libraries

Hello oskca,
How should we do to use other Vue libraries (like<EMAIL_ADDRESS>with gopherjs-vue?
Thanks in advance!

Sorry, haven't read Gopherjs doc previously to create this issue. For someone with same question:
https://github.com/gopherjs/gopherjs/wiki/Bundle-additional-JavaScript-with-Go-packages


keptn configure monitoring freezes

Environment

Client OS (e.g., Linux, OSX, Windows): mac osx
Keptn Version (keptn version): 0.6.0
Kubernetes Cloud Provider (e.g., GKE, AKS): gke
Kubernetes version (kubectl version):
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.7", GitCommit:"8fca2ec50a6133511b771a11559e24191b1aa2b4", GitTreeState:"clean", BuildDate:"2019-09-18T14:47:22Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.10-gke.17", GitCommit:"bdceba0734835c6cb1acbd1c447caf17d8613b44", GitTreeState:"clean", BuildDate:"2020-01-17T23:10:13Z", GoVersion:"go1.12.12b4", Compiler:"gc", Platform:"linux/amd64"}

Affected Component

[ ] Docs
[x] CLI
[ ] Kubernetes Integration
[ ] Openshift Integration
[ ] Helm
[ ] Istio
[ ] Bridge
[ ] Gatekeeper
[ ] Datastore
[ ] REST API
[ ] eventbroker / distributor
[ ] jmeter

Describe the bug
When executing keptn configure monitoring dynatrace right after installation, the CLI freezes and doesn't provide any output to the user. I assume this is due to the fact that the needed secret is missing. However, no output is created for the user to give a hint why the CLI stalls at this point.
To Reproduce

Install Keptn on a new cluster
Execute keptn configure monitoring dynatrace

Expected behavior
Error message why the command can not be executed.
Current behavior
CLI freezes/stalls.
Screenshots

Stalls at this point.
Additional context

In this context, also the monitoring source/type naming is not consistent. Not sure which is the correct wording.

Duplicate of  #1464.


fix(): replaced old syncano-cli with npx s

Codecov Report

Merging #244 into master will not change coverage.
The diff coverage is 80%.

@@           Coverage Diff           @@
##           master     #244   +/-   ##
=======================================
  Coverage   29.05%   29.05%
=======================================
  Files          93       93
  Lines        3672     3672
  Branches        5        5
=======================================
  Hits         1067     1067
  Misses       2605     2605


Replace .len() < 1 with .is_empty() in generated code

Since this was sitting in my notifications for forever, I just had a look. Seems like this is just a specific case of 1 for an otherwise generic check, and as such I don't think we should change the codegen. We can probably just add an #[allow] to that section of the code.

Agreed, #[allow] would be better. But I don't get how we cannot change codegen. #[allow] should be added by codegen?

Right, I just meant not complicate it with extra branches.

I think the is_empty would be better for the case with < 1. Isn't is possible that that method could be implemented in a more efficient manner than the .len() method?

See: https://github.com/lumeohq/xsd-parser-rs/pull/148


Do not optimize away layers connected to globals

When a layer is connected only to globals, it gets optimized away.
Let's take the swirl shader from the testsuite and break it into two parts
shader
swirl1 (float swirl = 0,
       output float s = 0, output float t = 0)
{
    float s1 = -1 + 2*u;
    float t1 = -1 + 2*v;

    float r = hypot (s1, t1);

    s = s1*cos(swirl*r) - t1*sin(swirl*r);
    t = s1*sin(swirl*r) + t1*cos(swirl*r);
}

shader
swirl2 (string texturename = "grid.tx",
       float s = u, float t = v,
       output color Cout = 0)
{
    Cout = texture(texturename, s, t,
                           "wrap", "periodic");
}

If you run this command
testshade -g 512 512 -od uint8 --o Cout out.tif --param swirl 2.0 --layer alayer swirl1 --layer blayer swirl2 --connect alayer s blayer s --connect alayer t blayer s

connecting outputs from the first layer to inputs of the second layer, you get the expected result.

But if you run
testshade -g 512 512 -od uint8 --o Cout out.tif --param swirl 2.0 --layer alayer swirl1 --layer blayer swirl2 --connect alayer s blayer u --connect alayer t blayer v

writing the outputs of the first layer to (u,v), you get

OSL does not take into account connections to globals and therefore considers that the first layer does nothing.
The change in shadingsys.cpp is not related to this issue but fixes out of range access to m_instoverrides vector. I am unsure if std::vector does not perform this check anyway but it feels safer.

Er, actually, I don't think that it's supposed to be valid to connect anything to a global. Shader parameters may be connected to other layers, but globals are globals, no connections.
I think the real bug is that it should have rejected your attempt to connect the first layer's output params to the second layer's "globals."

There are several reasons for my mistake:

I am familiar with Gelato Shading Language which supports this feature and I assumed that it was the same for OSL (wrongly; when I read the OSL manual, there is nothing that says connections to globals are allowed).
ShadingSystemImpl::decode_connected_param allows connections to globals and the code comments say the same thing.
Connections to globals work if there is also at least one connection to a layer input in the list of connections. So, I assumed it was an oversight. It was only a matter of a few changes to make it work.

Anyway, please close the request if you think this is not a desirable feature for OSL.

````ShadingSystemImpl::decode_connected_param``` allows connections to globals and the code comments say the same thing. Moreover, connections to globals work if there is at least one connection to a layer input. Therefore, I assumed it was an oversight.
Anyway, if you think it is not a desirable feature for OSL, please close this request.

I think decode_connected_param deals with globals because it's possible to connect a global to a shader layer input param. I never intended to allow a shader layer output param to be connected to a global. I think that, in some sense, is the bug -- that it doesn't catch that nonsensical situation.
I think I want to disallow this, and there are basically two reasons that it would be especially problematic:

We have been operating on the assumption that only shader parameters can be connected to things upstream, and (your simple example notwithstanding) I can only assume that making this change would lead to many bugs in places inside the shading system and optimizer where we say if(connected) and thereafter assume it's a parameter and if(global) and thereafter never consider that it might be upstream connected. That would be a nightmare to untangle.

It may have (incorrectly) accepted the command, but the reason it seemed to work to you basically boils down to the fact that at some point in execution of that second layer, it will copy the results of the first layer's outputs into u and v. But there's just one u and v for the whole shader group, so any OTHER layers of the group that run subsequently will see the new values of u,v. It would work out for the trivial two-layer setup you used as an example, but for complex shader networks it sounds like a recipe for subtle shader bugs, especially considering that layer execution ordering is nondeterministic because of lazy evaluation. Literally from point to point in the scene, depending on which variables get pulled on by different conditional branches that may execute or not, the layer execution order may vary, and therefore so will if (or when) the globals get overwritten. Ick.

I think the best thing to do is close this PR (sorry), understand that connecting a shader output to a global is disallowed, and I'll look at the code and propose the right fix that would correctly flag that as an error.

Hmm ... Looks like you made a very compelling case not to do it


Make the "List of Experiments" text smaller for mobile devices

Make the "List of Experiments" text smaller for mobile devices

hello sir , please assign me this issue , i am ready to create a PR .

hello @TheNinza sir , please assign me this task, i have solved this issue will make PR on 1st oct Thank you.

@TheNinza Is this issue still available? I would like to try.


Refactored and created the logic for interpolation

Changes:
Refactor(Pairprogrammed with Savannah):

Added maxGapDistance to the dependent series object
Refactored dspec parser to add the maxGapDistance
Refactored input gatherer
Refactored series provider

Added Interpolation Logic(Pairprogrammed with Beto):
In SeriesProvider:

Created the interpolation function
Created the check_gap_distance function
Created the fill_in_date_gaps function

Notes From Meeting:

change seconds value of maxGapDistance to correspond to points since that's what we use to describe series in dspec and might make the method more generalized
passing a lot to the interpolation method, maybe should only be passing the series
should the method return the full series (create a new series with all the gaps filled) or return only the values that were interpolated
To change the isComplete value we should be passing it to the validate series function NOT marking complete inside the interpolation logic

interpolation function should ONLY do the interpolation

maybe change interpolation object in dspec to mandatory but if not there just sets interpolation to false?

true/false
interpolation method
gapDistance


azure-cognitiveservices-speech for ARMv8 arch

Hi,
Is it possible to build  "azure-cognitiveservices-speech" to support ARM arch ?
I was trying to run my Bot client from Linux on NanoPi A64, which has 64-bit Quad-core Cortex-A53 with no luck.

There are plans for Linux ARM64 support, but no ETA we can promise right now.
Keep in mind, we currently support only these distros for Linux: Ubuntu 16.04, Ubuntu 18.04, Debian 9
It is not clear if others will be available for ARM
Also not clear what HW will be available in our test environment ...


[fix][connector-rocketmq]Fix a NPE problem when checkpoint.interval is set too small(#6624)

Purpose of this pull request

Fix a NPE problem when checkpoint.interval is set too small, which is associated with issue 6624
Does this PR introduce any user-facing change?

No
How was this patch tested?

test with existed test-case
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

A simple analysis of this bug: RocketMqSourceReader. pollNext will start a certain number of RocketMqConsumerThreads. RocketMqConsumerThread will start RocketMqPullConsumer, and the startup time cannot be ignored. In standalone mode, the number of RocketMqConsumerThreads depends on the number of topic partitions. If the checkpoint time is too short, the RocketMqConsumerThread may not have been fully started when the first checkpoint is triggered, resulting in a brief data inconsistency between the set and the consumerThreads set, which in turn triggers NPE

cc @sunxiaojian


Set Modal's Event Prop's Default Value

Fixes #182
Proposed Changes

As a work around to the warning in #182 by default set the Modal's event prop to an empty object.

Build worked successfully however the GitHub action failed to pass lint process....(Working on fixing this I will PING you @kikuomax when this PR is ready)

@kikuomax This issues was solved after gathering an understanding of the following eslint rule https://eslint.vuejs.org/rules/require-valid-default-prop. This PR is now ready for review as all tests/build have passed.


g2p model location

Within run.sh I believe the model location should be $1/ipd_clean_slt2018.mdl not ../ext/ipd_clean_slt2018.mdl
alternatively the user should separately specify it as a parameter

Agreed, parameters could be corpus directory and g2p model.


Hare programming language: init

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Might be made redundant by #170569, can you take a look?

Summarizing: I did not like the code he wrote.

Can this not be marked as broken on darwin? I understand upstream's sentiment but it is free software and we have the right to package it for as many operating systems as we want.

Does it build for Darwin?

@alyssais qbe fails to build so this fails transitively.

Can this not be marked as broken on darwin?

It can, but it will be useless...

I understand upstream's sentiment but it is free software and we have the right to package it for as many operating systems as we want.

Okay, go for it.
[ ] Port QBE to Darwin
[ ] Port the compiler to Darwin
[ ] Port the runtime to Darwin
[ ] Debug and iterate
[ ] Maintain that port (or the patchset) in a separate tree because the upstream will refuse to provide support for a proprietary platform (well, Darwin kernel is open source but the only "distro" really running it is the Cupertinian one :) )
[ ] Provide the patches to this Nix expression
Follow the model:
https://briancallahan.net/blog/20220427.html

@AndersonTorres I understand that the work is non-trivial to maintain on darwin, I guess I was being idealistic with the suggestion 😂

do dependencies need to be specified in both buildInputs and nativeBuildInputs?

strictDeps = true; forces this sometimes.

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

2 packages built:

hare
harec

As a "bug report", I should... report that most recent Hare drivers are not working: they stuck indefinitely trying to recompile harec.

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

2 packages built:

hare
harec

Author of #170569, all fair and I am more than happy to abandon that PR. It would have been nice to receive a poke though so that I could abandon it rather than notice this PR as a rebase conflict post-merge. '^^

Sorry by the terse response.
My expression was almost complete, however I did not upload it because of real life issues.
Would be a waste of time to comment another code and end by copy-pasting my code above another one.

Overall, your expression is better and I think you are more competent with Nix than I am.

Many thanks for the compliments! But I am a novice, I have so much to learn.

The test suite is only running for harec though and not hare, will send a PR.

Initially I have not seen a make test on hare tree, only in harec. Thanks for notice this!

Also, note that hare release relies on OpenSSH and Git being in PATH, whether we want to wrap them I am not sure as they are fairly big and not essential for most Hare usage.

As you have noticed, and also as I have read in this blog post, hare release is completely optional, has two big dependencies and is strongly tied to git (while the author welcomes future VCSes). I was thinking in create a third entry point, something like hare-full containing these extras and such that it does not trigger an unnecessary rebuild.
However it would require a bit more complicated procedure. I can do that later...

I am also not sure why you opt for maintaining a copy config.mk rather than rely on makeFlags

I prefer to follow/emulate most closely the upstream approach when including things to Nixpkgs. This way I can be sure that the build is going as they wanted.
The upstream project suggests to create a config.mk file and edit/populate it according to the local environment.
So I preferred this approach of "write the file" because it emulates that original intention.
In other distros you would do this type of thing with a bunch of sed&awk, but in Nixpkgs you can use those utilities to write files on demand!
Indeed I have done something like this in openmsx before.


⚠ Bofrost.at has degraded performance

In 207f9d9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1146 ms

Resolved: Bofrost.at performance has improved in 4ae7d6b after 16 minutes.


The numbering was out of order. just fixed that

Thanks for the PR. It has been merged.
I will be glad if you can leave a star on this repository.


Broken quotes around link

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
GitHub Pages/Set up site with Jeykll/About GitHub Pages and Jeykll
What part(s) of the article would you like to see updated?
The second link of the second paragraph under the heading "Themes"
Additional information
The link only shows the second quotation mark, and the quotation mark itself is part of the hyperlink.

I have made a pull request regarding this: #30985

@minding-myown-business Thank you for opening an issue and linking it to your PR! ✨