[WIP] Refactor SGObject using mixins

This is still work in progress for a mixins draft, and not intended to be merged. I will continue working on it next Wednesday.

[x] Turn SGObject into one big mixin
[x] Divide the common interface into a mixin (get_name(), global variables (io, version, ...))
[ ] Divide the any parameter/observable interface into a mixin
[ ] Use the parameter mixin for ObservedValue
[ ] Investigate what to do next!

Great initiative, thanks for that! I will have a closer look over the next few days!

This is quite a big PR now, hard to oversee. Would you mind explaining the high level changes you are doing here? This would save us (me at least :D) some time digging through all the code. What shall we look at first design wise?

 Each SGObject is a composition of these mixins, and nothing else. Each mixin can't assume anything about the SGObject containing it but for the other mixins it depends on. To access a function of another mixin you have get a reference of it using the "mutate" function. A mixin can also access the type of the containing SGObject using the "derived_t" type function inside the mutator.
This will require that each function accepting an SGObject as an argument or returning an SGObject has to be templated, and guarded with checks that it contains the mixins the function needs. A problem comes in mind with the class_list 'create' function (which creates an instance of any class and returns an SGObject).
What I did was:

Get rid of "set_generic" specialized template functions inside SGObject, since you can't specialize template functions inside template classes without first specializing the class.
Create two mixins to test things out: HouseKeeper (horrible name) which contains reference counting and global variables (io/version/parallel), and other general purpose functions. And SGObjectBase which contains everything else.
Now SGObject is a composition of these two.

Before you press ahead with this type of design, make sure to look at some of the SWIG interfacing stuff.
In particular, see how SWIG works with templates (messy). There is a few implications, for example if you make the parameter framework a mixin (put/get/etc), then the Shogun base class should have it added (and so all Shogun classes have a common put/get/add). If we don't do that, then the SWIG interface will have to be re-generated for every class where the mixin is used. While it is fine to do that in certain exceptions (we had the example of ObservedValue, where we would like to access "get" without it being a SGObject), we cannot do it for every class as the SWIG compile time will explode.
Also look at a few cases where we actually exploit the fact that all classes in Shogun inherit from SGObject, (class_list for example, used in factories).

IDK....phew this will get pretty mesy.
What about for now we start with something less general and make it work, then make it more general.
Like for example moving all the ParameterMap stuff (self, put/get) into a separate class that SGObject inherits from, and then go from there. That was the main motivation:

to split the code in SGObject into multiple files
to be able to have classes that have the parameter framework without the need to be an SGObject

@theartful which swig have you used? in case you haven't tried it with 4.0 plz do so... it was released very recently.... although idk if variadic template support has been added, but there were a lot of improvements... if it works, it's fine to depend on 4.x

@vigsterkr There are a couple of problems:

separating the get/add mixin is weird. The untyped get functions

CSGObject* get(const std::string& name) const noexcept(false);
CSGObject* get(const std::string& name, std::nothrow_t) const noexcept;

return CSGObject, which depends on the get/add mixin, which in turn depends on CSGObject.
3. relating to 1. The dispatching code in the get/add mixin depends on CSGObject
4. there is no clear division between put/get/add and observe/watch_param mixins. They both depend on each other (since put uses observe) .
5. separating ObservedValue in a different file is difficult. The circular dependency thing, but now with the mixins.

@vigsterkr @karlnapf There are a couple of problems:

separating the get/add mixin is weird. The untyped get functions

CSGObject* get(const std::string& name) const noexcept(false);
CSGObject* get(const std::string& name, std::nothrow_t) const noexcept;

return CSGObject, which depends on the get/add mixin, which in turn depends on CSGObject.

Yes these are problematic anyways. We might add them to the swig c++ code?

relating to 1. The dispatching code in the get/add mixin depends on CSGObject

Yes, that code is another candidate to maybe be removed. We could instead offer something in swig like get_kernel which maps to c++ `get so we would have
k = m.get_kernel("kernel")
k.init(....)

instead of (what we currently have)
k = as_kernel(m.get("kernel"))
k.init(....)

It has the disadvantage that we need swig get- functions for all base types. But is has the advantage that there is no more dynamic dispatching in c++. And more importantly, it doesn't really make sense as in most use cases, there will be a cast afterwards (see lines above)
You could try to do this in a separate sub-PR

there is no clear division between put/get/add and observe/watch_param mixins. They both depend on each other (since put uses observe) .

Ah that is true. I was initially thinking that the observable stuff simply depends on put/get, but I then forgot that put actually calls observe. I guess the current design means that we need to group them. But I wonder whether that is the best idea ...

separating ObservedValue in a different file is difficult. The circular dependency thing, but now with the mixins.

I think this is solved once 3. is sorted?


Add timestamp to optimize windows in qt3scan application

It would be helpful to add a timestamp to the output optimization plots in the qt3scan application. This would help experimenters know which of the open windows was the most recent

Update Title of window: https://github.com/qt3uw/qt3-utils/blob/main/src/applications/piezoscan.py#L268
Probably make the window title similar to window title defined here: https://github.com/qt3uw/qt3-utils/blob/main/src/applications/piezoscan.py#L377


Changing the shape of node

I want to change the shape of node to any logic gate like AND gate OR gate etc. Is there any way to do that?

https://rete.js.org/#/docs/plugins/react-render#lang=en&tosearch=component%3A MyNode

https://codesandbox.io/s/retejs-react-render-t899c

Ok so if I put my svg code in return is it suppose to work?

Yes, why not?


CSS selectors and properties are consolidated and organized to follow semantic structure.

User Story
AS A marketing agency
I WANT a codebase that follows accessibility standards
SO THAT our own site is optimized for search engines.
Acceptance Criteria
The website must meet accessibility standards.
CSS selectors and properties are consolidated and organized to follow semantic structure.

done


Idea: magic broomsticks

Another type of vehicle with a different sitting pose that allows for flying x acceleration

https://docs.webaverse.com/docs/create/mounts
https://docs.webaverse.com/docs/create/hoverboards

Could be a variation of a hoverboard that allows for flying, might want characters to stand on broomsticks too

This was in the original design document. Standing is much more dynamic and interesting for this IMO.


Advanced filter for Player Rivalries

Vs Filter
Vs Country
Date Filter
Best Of
In/Out

Done all except Vs Height


Merge main branch into dev-1.32

This PR syncs the dev-1.32 branch with the main branch to keep the dev-1.32 branch up to date.
cc: @chanieljdan @AnshumanTripathi @rdalbuquerque @spurin @hacktivist123 @rdalbuquerque

/lgtm

/retitle Merge main branch into dev-1.32

/approve


Automated cherry pick of #8809: fix: baremetal usage should exclude converted hypervisor

Cherry pick of #8809 on release/3.5.
#8809: fix: baremetal usage should exclude converted hypervisor

Codecov Report

Merging #8810 (fe1c6aa) into release/3.5 (e3fe6c4) will decrease coverage by 1.81%.
The diff coverage is 0.28%.

@@              Coverage Diff               @@
##           release/3.5   #8810      +/-   ##
==============================================
- Coverage         6.03%   4.22%   -1.82%
==============================================
  Files              795     162     -633
  Lines           124282   24786   -99496
==============================================
- Hits              7503    1047    -6456
+ Misses          116114   23647   -92467
+ Partials           665      92     -573

Flag
Coverage Δ

aFlag
4.22% <0.28%> (-1.82%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/cloudcommon/db/domainresource.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/modelbase.go
4.51% <0.00%> (-0.08%)
:arrow_down:

pkg/cloudcommon/db/namevalidator.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/orderby.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/virtualresource.go
0.00% <0.00%> (ø)

pkg/cloudprovider/securitygroup.go
0.00% <ø> (ø)

pkg/compute/regiondrivers/aliyun.go
0.33% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/regiondrivers/base.go
1.03% <0.00%> (-0.04%)
:arrow_down:

pkg/compute/regiondrivers/esxi.go
6.38% <0.00%> (-36.48%)
:arrow_down:

pkg/compute/regiondrivers/huawei.go
0.21% <0.00%> (-0.01%)
:arrow_down:

... and 576 more

Continue to review full report at Codecov.

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


Fix Bug in TwitterConverter

This PR fixes a bug in TwitterConverter, an operator that converts a single field - raw JSON string of a tweet into multiple fields.
Previously in PR #599 , the DateField is refactored using the new Java 8 Date API, and a new DateTimeField is added. In this PR, the create_at attribute generated by TwitterConverter is changed from the original String type to a more precise DateTime type.
However, in PR #602 , where the transformToOutputSchema function is added to every operator, the actual field object generated by TwitterConverter is changed from DateTimeField (correct) to StringField (incorrect) for unknown reasons. This PR causes the mismatch between the Schema DateTime type and the actual field type StringField.  We didn't notice the bug in that PR.
This PR:

fixed the bug and improve code quailty
adds a test case for the operator (The essential reason that we didn't find out the bug is the lack of testing for TwitterConverter. Previously there's no test case for the operator)
adds lots of comments and documentation to TwitterConverter and related classes
improves the design of TwitterConverter and related operators. (see detail below)
changes TwitterSample class (we ingest a sample twitter table by default) to use the TwitterConverter class instead of parsing the JSON data on its own
adds a new AsterixTwitterIngest class to be used to ingest data into texera from AsterixDB

The operator was an operator created in hurry to achieve the task of ingesting twitter from AsterixDB into Texera and it was poorly designed. Previously it must be chained to AsterixSource operator, because it retrieves the tweet from a hard-coded column named rawData. The AsterixSource operator also writes the result into the hard-coded column rawData. The rawData attribute is also hard-coded into TupleSink to be removed in the end.
The old design violates the principle of compose-able operators. This PR improves it by allowing the user to specify the name of the column for the twitter JSON data.
During adding the test case for TwitterConverter, a minor bug in Schema.Builder is found, and it's also fixed.

Just curious about "However, in PR #602 , where the transformToOutputSchema function is added to every operator, the actual field object generated by TwitterConverter is changed from DateTimeField (correct) to StringField (incorrect) for unknown reasons. "  Do we know the reason for that change so that we can avoid such mistakes in the future?

@chenlica That PR is Ming’s work to change every operator to add the function transformToOutputSchema. I’m not sure the reason of the change, but it’s quite normal and understandable to make a mistake when changing about 30 operators.
As I stated in the description, the solution is to test cases and ensure good test coverage. Previously the operator has no test case. If there was a test case, we could easily spot and avoid the bug.

Understood!

@avinash0161 I've taken care of your comments. I've replied the new changes under your original comments, please check.


Improve default compiler options and clear output from compiler

What does it do?

Switch to use clang by default and use options to enable C++17, warnings and boost.
Clear compiler output of ASCII colors

How can this be tested?
Before warnings/errors would be with ASCII symbols
After warnings/errors are clean

What does this do? You mean it removes all the ANSI color escape sequences? Wouldn't it be better to actually interpret them correctly?

@Leandros Aside from compiler adjustments, yes, I remove escapes which ain't handled at all.
At the moment I'm not sure how to handle them in HTML as it would require us to build quite complex one.
As far as I'm aware there are no solutions to map ASCII into HTML/CSS so I think for now, at least, it would be better to leave nice output instead of unreadable one.

Something like that should allow to convert ANSI escapes into colored HTML output: https://github.com/drudru/ansi_up


pack_pipeline_e2e test is failing transiently

Here is a gist showing the transient issue
https://gist.github.com/nirvedhmeshram/5deeaaa94729053794aa2dd9fbf223e0

Prefetch lowering config before Packing and set later solves this issue.
Reason for the failure -  the lowering_config was non-deterministically not being carried forward after the first application of pack-and-transpose and due to cascading effect tile-and-fuse couldn't perform tiling from 2nd level onwards.
CC: @MaheshRavishankar @nirvedhmeshram @yzhang93


[MKLDNN] Use MKLDNNRun

Description
MKLDNNRun will handle ndarray view and enhance code quality.
@PatricZhao @TaoLv
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

LGTM but let's wait for the performance testing reports.

@TaoLv @ciyongch any comments?

Do we fix any specific issue with these changes or they're just for code refactoring?

Do we fix any specific issue with these changes or they're just for code refactoring?

Not for any particular issue, but a code refactoring to improve code quality. Currently, ndarray view check doesn't cover all mkldnn operators, which may cause trouble in rare case. This is to prevent that happen.

@TaoLv is this refactor OK?

@ciyongch Yes. MKLDNNRun has 2 versions. One is for unary ops, and other one is for ops with multiple inputs & outputs.  elemwise_add will use the second version, and all inputs are well handled.

@ZhennanQin please rebase the code


Update of Accumulated CO2 data

This PR provides a new indicator file for the Accumulated CO2 per citizen, including data for 2015 till 2020

The deployment looks good to me. I think we can merge it @olavurellefsen

I left the score calculation based on the accumulated Emissions between 2020 and 2050.
Maybe I should mention that somewhere?

I left the score calculation based on the accumulated Emissions between 2020 and 2050. Maybe I should mention that somewhere?

Yes, you can mention it in the README.md file (or some new documentation file available from there) or in the /about page.

I left the score calculation based on the accumulated Emissions between 2020 and 2050. Maybe I should mention that somewhere?

Yes, you can mention it in the README.md file (or some new documentation file available from there) or in the /about page.

Will prepare something for this in the afternoon and create a new PR.


IC-1387: Add manual deployment to user research environment

What does this pull request do?
Adds a manual deployment "gate" to CircleCI.
There will be a new "approve_research" and "deploy_research" button, similar to the preprod ones:

What is the intent behind these changes?
To enable user research.

🙇♂ Thank you


I need help!

I have learned vert-Mqtt Server long time.but at recently I found some problem,because too many so I can't list here.
Hope one can help me who are the mqttServer's author or developer.
Very thanks.

I am a Chinese developer ,my Email is 751957846@qq.com,if who can help me please send e-mail to this address.

@wwhai, I think the best way to solve these problems is to create an issue for each of them in the related repository.

@wwhai, I think the best way to solve these problems is to create an issue for each of them in the related repository.

Thank you,last night I read the document of vertx-mqtt by careful,I found this vert-mqttServer dosn't for my design,It isn't a standard mqtt broker,so it can't suit a broker's function ,or maybe I should consider other mqtt broker such Activemq or HiveMq,etc.

@wwhai right, this module is for building your own broker, not a ready to use one.


fx relate to DNNs failure to exhibit numerosity?

https://arxiv.org/pdf/1802.05160.pdf
(thanks Gautam)

closing, besides the point of the paper


[TEST] Have pipeline to build engine test images

What's the test to develop? Please describe
https://github.com/longhorn/longhorn-tests/pull/1012 can generate engine test images
Need to have a pipeline build engine test images on demand to replace run in local.
Describe the items of the test development (DoD, definition of done) you'd like

[x] Pipeline work well

Additional context
N/A

Close ticket because now we have private pipeline to build necessary test engine upgrade images and arm64 pipeline have no failures related to engine upgrade
Build 182
Build 181


Chained requests works in desktop client but not via Inso CLI

Expected Behavior
Chaining requests works in CLI as well as on desktop
Actual Behavior
The same as here https://github.com/Kong/insomnia/issues/3366
I have checked the same chaining requests on inso cli v.2.4.0 and 9.2.0 and the results are different. Everything works on v.2.4.0 and the same requests don't work on v.9.2.0.
v.2.4.0:

v.9.2.0:

Reproduction Steps

Make a request that returns something
Make a second request that needs something from the first
Make a test suite that contains those two requests, each as "tests"
Verify Run Tests works
Try and run inso test and select that suite

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information

Installation Method: Mac App

Version: Insomnia 9.2.0
Build date: 5/15/2024
OS: Darwin arm64 23.4.0
Electron: 29.3.0
Node: 20.9.0
Node ABI: 121
V8: 12.2.281.27-electron.0
Architecture: arm64

Insomnia Version
9.2.0 / Inso CLI 9.2.0
What operating system are you using?
macOS
Operating System Version
mac OS Sonoma 14.4.1
Installation method
Inso CLI
Last Known Working Insomnia version
3.8.0

Hi @0erty,
Are you using the Response Body tag from another request or do you store the value in the environment?
The second option is not supported (related Issue)
Otherwise you might want to check if the Trigger behavior in the Response Body tag's settings is not set to Never as this could run fine in Insomnia because you have a previous response but not on the cli

Hi @gatzjames, I tried it in both options and it doesn't work

`  Collection
[plugin] Loading
[response tag] Resending dependency
Error rendering template Template render error: (unknown path) [Line 1, Column 1]
attempted to output null or undefined value
at Object._prettifyError (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at Template2.root [as rootRenderFunc] (eval at _compile (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js), :18:3)
at Template2.render3 [as render] (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at Environment4.renderString (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Failed to render element url {{ _.base_url }}/client-api/v1/auth/login
Error rendering template Template render error: (unknown path)
Error: attempted to output null or undefined value
at Object._prettifyError (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at eval (eval at _compile (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js), :11:11)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Failed to render element Environment af={% response 'body', 'req_9a7e36ca9dc54432b5253d61fad75936', 'b64::JC5tb25vbGl0aC5jb29raWU=::46b', 'always', 60 %}; Expires=Tue, 19 Jan 2038 03:14:07 GMT; Domain=appfollow.io; Path=/
[response tag] Resending dependency
Error rendering template Template render error: (unknown path) [Line 1, Column 1]
attempted to output null or undefined value
at Object._prettifyError (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at Template2.root [as rootRenderFunc] (eval at _compile (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js), :18:3)
at Template2.render3 [as render] (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at Environment4.renderString (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Failed to render element url {{ _.base_url }}/client-api/v1/auth/login
Error rendering template Template render error: (unknown path)
Error: attempted to output null or undefined value
at Object._prettifyError (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at eval (eval at _compile (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js), :11:11)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Failed to render element Environment af-cli={% response 'body', 'req_9a7e36ca9dc54432b5253d61fad75936', 'b64::JC5jbGllbnRfYXBpLmFjY2Vzc190b2tlbg==::46b', 'always', 60 %}; Expires=Tue, 19 Jan 2038 03:14:07 GMT; Domain=appfollow.io; Path=/
[response tag] Resending dependency
Error rendering template Template render error: (unknown path) [Line 1, Column 1]
attempted to output null or undefined value
at Object._prettifyError (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at Template2.root [as rootRenderFunc] (eval at _compile (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js), :18:3)
at Template2.render3 [as render] (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at Environment4.renderString (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Failed to render element url {{ _.base_url }}/client-api/v1/auth/login
Error rendering template Template render error: (unknown path)
Error: attempted to output null or undefined value
at Object._prettifyError (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at eval (eval at _compile (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js), :11:11)
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Failed to render element headers[0].value cookie1={% response 'body', 'req_9a7e36ca9dc54432b5253d61fad75936', 'b64::JC5tb25vbGl0aC5jb29raWU=::46b', 'always', 60 %};cookie2={% response 'body', 'req_9a7e36ca9dc54432b5253d61fad75936', 'b64::JC5jbGllbnRfYXBpLmFjY2Vzc190b2tlbg==::46b', 'always', 60 %}
1) Check collection
0 passing (25ms)
1 failing

Collection
Check collection:
Error: attempted to output null or undefined value
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at /snapshot/insomnia/packages/insomnia-send-request/dist/index.js
at RawTask.call (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at flush (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at Timeout.handleTimer (/snapshot/insomnia/packages/insomnia-send-request/dist/index.js)
at listOnTimeout (node:internal/timers:573:17)
at process.processTimers (node:internal/timers:514:7)`

@0erty from the logs it looks like there might be a missing variable in the selected environment?

[response tag] Resending dependency

Request chaining seems to work since inso is trying to send the request but fails to find an env variable?

@gatzjames This variable is in the environment, but it seems that Insomnia does not write the received value from the previous request to it. I downgraded to version 3.8, everything works fine there

Yep storing a chained request result to the environment is not supported by inso since it would need to recursively evaluate the environment. In that version I think we did a double pass but that led to other substantial issues and was removed.
I'd suggest using the response tag directly in the places you want to use the response instead of the environment.
We are also working on pre-request/after-response scripts atm so once these are available in inso you will be able to manipulate the env from there.

Closing this since saving responses on environment data is not supported and will be possible later when pre-request/after-response scripts become available in Inso-CLI.


Extend validation errors with line and column

Map jsonpointer and rfc5147 (character position) to line:column

Implemented in dev


with SET WINDOW/ASPECT, trying to fill entire window, see gaps at sides or top

Reported by Ansley B Manke on 3 Sep 2002 19:13 UTC
!Subject: viewport/window edges
!Date: Tue, 22 May 2001 20:08:37 -0400 (EDT)
!From: Andrew Wittenberg<EMAIL_ADDRESS>!To<EMAIL_ADDRESS>!
!Dear Ferreters,
!
!I'm having trouble getting a viewport to fill the entire window area in
!Ferret v5.22 for Linux.  "GO magnify" almost works, but for some reason
!this gives small gaps at the right/top edges of windows with low/high
!aspect ratios. The following demo distills my script and magnify.jnl
!down to their basics (I think).  Note the gap between the right edge of the viewport and the window. The
!gap is absent when when the window aspect is near 1.  For wide windows
!there's a gap on the right, and for tall windows there's a gap at the
!top.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/429

Comment by<EMAIL_ADDRESS>on 21 Oct 2003 20:15 UTC
Moving from the<EMAIL_ADDRESS>user to the<EMAIL_ADDRESS>user.

Comment by<EMAIL_ADDRESS>on 5 Nov 2003 22:58 UTC
reassign to Ansley

Adding @AndrewWittenberg


Implement SettableHttpHealthCheckService

-Add SettableHttpHealthCheckService implementation
#511

A few overall comments:

It seems like 'do not override' command is missing. See #511 to read about three modes.

We could probably represent the state using an Optional<Boolean> where an empty Optional signifies 'do not override'.

Question: What would be better, 1) sending the command as the content of a POST request or 2) Using path as implemented in this PR?

How about providing the sensible default behavior in a protected method and let a user override it his/her subclass? We won't have to introduce PathMatcher this way. e.g. protected Optional<Boolean> mode(HttpRequest req) ?

Using `Optional for represent on/off/illegal state is good idea. I'll fix it. THX:)
Use path for updating health status is more familiar to me. So, I would use path(../on|../off) as default. But PathMatcher implementation looks not good. Choose use opinion. THX!

How about the name ExternallySettableHealthCheckService - took me a while to figure out what's being set, but realized it's server health being set from the outside.

@anuraaga Cool name. Thanks:)

Isn't it too long? Would love to have a better name.

illegal state

I actually meant Optional.empty() for 'do not override and respect the automatically determined healthiness'. Given that a user may send an invalid request, perhaps it's better defining an enum like you initially suggested?

@trustin
If mode() method is member of HttpHealthCheckService, that looks capable.
But this is inherits implementation for make health status externally settable.
So I think, BAD_REQUEST is capable for not mapped request.

I think, remaining concern is only class name.
ExternalSettableHttpHealthCheckService looks little long. It makes server builder code ugly.
But also, SettableHttpHealthCheckService is hard to understand what this class is.
Which one do you like? @anuraaga , @trustin

Maybe ManagedHealthCheckService? Managed implies a little that something external is controlling.

Managed also good name. I hope to choose.

@yuinacor Any progress on this? :-)

@trustin Sorry for late.

@yuinacor Please fix Javadoc errors. Removing </p> should fix them. To test locally, run ./gradlew site:javadoc

com.linecorp.armeria.server.http.HttpServerTest > testTooLargeContent[0: h1c] FAILED
java.util.concurrent.ExecutionException: com.linecorp.armeria.common.ClosedSessionException
at java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:357)
at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1895)
at com.linecorp.armeria.server.http.HttpServerTest.testTooLargeContent(HttpServerTest.java:515)
Caused by:
com.linecorp.armeria.common.ClosedSessionException

Test failed by reason above.
It looks not related with this PR... Is it know issue?

I tried to build and test with the master branch, but there was no error on my local environment. Did you always got the failure?

My local env also same.
But AppVeyor fails test during yesterday.
Seems repeat.
Can I manually retest this?

Can I manually retest this?

You can amend your commit and force-push to trigger a rebuild.

Thanks a lot, @yuinacor !


Allow copyright statement in footer to be set via Nunjucks macro

Currently the copyright statement in the footer component is hardcoded, which means it can’t be set by the Nunjucks macro. The guidance on the page suggests that the copyright can vary:

The NHS website is (c) Crown copyright but the NHS generally does not have Crown status. The service manual footer, for example, says (c) NHS England.

To allow for both options, we add an additional option to the Nunjucks macro, with NHS England remaining as the default option.
This would follow the approach of the GOV.UK Design System footer component.

This was done in #975.


bug in command line versions

What command was run:
di_vgp.py -f tmp.txt
What happened:
Traceback (most recent call last):
File "/Users/ltauxe/PmagPy/programs/di_vgp.py", line 94, in
main()
File "/Users/ltauxe/PmagPy/programs/di_vgp.py", line 87, in main
output = pmag.dia_vgp(data)
File "/Users/ltauxe/PmagPy/pmagpy/pmag.py", line 1175, in dia_vgp
(decs, dips, a95s, slats, slongs) = list(zip(*largs))
TypeError: zip argument #1 must support iteration
And there are probably more programs with this problem.
What I expected to happen:
that it would spit out the vgp
Data file(s):
tmp.txt
Screenshot(s):
PmagPy version:

The PmagPy-cli.ipynb example still seems to work:
di_vgp.py -f data_files/di_vgp/di_vgp_example.dat

It looks like the problem is with having a single line of data in tmp.txt.
I believe I fixed that, but I found another issue if you have multiple lines with 5 values each (a95s included).
Traceback (most recent call last):
  File "/Users/nebula/Python/PmagPy/programs/di_vgp.py", line 97, in <module>
    main()
  File "/Users/nebula/Python/PmagPy/programs/di_vgp.py", line 81, in main
    output = pmag.dia_vgp(inlist)
UnboundLocalError: local variable 'inlist' referenced before assignment

Looks like you just need to set inlist in this code if data.shape[1]==5:
    if len(data.shape)>1: # 2-D array
            N=data.shape[0]
            if data.shape[1]==4:   # only dec,inc,sitelat, site long -no alpha95
                data=data.transpose()
                inlist=numpy.array([data[0],data[1],numpy.zeros(N),data[2],data[3]]).transpose()

I am not sure how the 5 values should be ordered...

with the alpha95 after the dec inc and before the lat lon

On Jun 12, 2019, at 2:23 PM, Lori J<EMAIL_ADDRESS>wrote:
The PmagPy-cli.ipynb example still seems to work:
di_vgp.py -f data_files/di_vgp/di_vgp_example.dat
It looks like the problem is with having a single line of data in tmp.txt.
I believe I fixed that, but I found another issue if you have multiple lines with 5 values each (a95s included).
Traceback (most recent call last):
File "/Users/nebula/Python/PmagPy/programs/di_vgp.py", line 97, in
main()
File "/Users/nebula/Python/PmagPy/programs/di_vgp.py", line 81, in main
output = pmag.dia_vgp(inlist)
UnboundLocalError: local variable 'inlist' referenced before assignment
Looks like you just need to set inlist in this code if data.shape[1]==5:
if len(data.shape)>1: # 2-D array
        N=data.shape[0]
        if data.shape[1]==4:   # only dec,inc,sitelat, site long -no alpha95
            data=data.transpose()
            inlist=numpy.array([data[0],data[1],numpy.zeros(N),data[2],data[3]]).transpose()

I am not sure how the 5 values should be ordered...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/585?email_source=notifications&email_token=AABKWDK76K3LT757VCMPT63P2FSLRA5CNFSM4HXE6FXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXR234Y#issuecomment-501460467, or mute the thread https://github.com/notifications/unsubscribe-auth/AABKWDJSIA6SGXQBWHP3423P2FSLRANCNFSM4HXE6FXA.

@ltauxe, this seems to work correctly.  Reopen this issue if you need to!


Issue with contextual shortcuts can lead to loss of data

Godot version:
3.2.2
OS/device including version:
Any OS
Issue description:
In the editor, some shortcuts do more than one thing depending on context. For example Ctrl+D can duplicate nodes, keyframes or lines of code. Delete can delete keyframes, files or nodes.
Sometimes Godot fails to identify the right context. Some examples:
Ctrl-D to duplicate a keyframe and a line of code gets duplicated. Easy to miss if it doesn't immediately break the script, can be hard to debug otherwise.
Ctrl-D to duplicate a keyframe and a node gets duplicated. Again this this easy to miss and can cause trouble.
Delete to detele a keyframe and a file/node gets deleted.
It also happens for other shortcuts and in other context but I'm not sure how to reproduce it consistently

Duplicate of https://github.com/godotengine/godot/issues/36205.


Add option for maximum line separation

As discussed in #33, remove line separation between blocks of code.
For example sake, lets say the variable is set as max_line_seperation = 1 and is the only option that is set.
The formatter would modify the file like so:
Unformatted file:
local M = {}

--- Utilities ---

-- Convert string to title case
function M.firstToUpper(str)
  return (str:gsub('^%l', string.upper))
end

-- Math operations
-- Check if number is prime
function M.is_prime(n)
  local max_divider_checked = math.sqrt(n) + 1
  for i = 2, max_divider_checked do
    if n % i == 0 then
      return false
    end
  end
  return true
end

return M

File diff:
+local M = {}
+
-
-
-
+--- Utilities ---
+
-
+-- Convert string to title case
+function M.firstToUpper(str)
+  return (str:gsub('^%l', string.upper))
+end
+
-
+-- Math operations
+-- Check if number is prime
+function M.is_prime(n)
+  local max_divider_checked = math.sqrt(n) + 1
+  for i = 2, max_divider_checked do
+    if n % i == 0 then
+      return false
+    end
+  end
+  return true
+end
+
-
-
+return M

Final Formatted file:
local M = {}

--- Utilities ---

-- Convert string to title case
function M.firstToUpper(str)
  return (str:gsub('^%l', string.upper))
end

-- Math operations
-- Check if number is prime
function M.is_prime(n)
  local max_divider_checked = math.sqrt(n) + 1
  for i = 2, max_divider_checked do
    if n % i == 0 then
      return false
    end
  end
  return true
end

return M

I'll take a look this weekend

this has been implemented by other way.
set all below
keep_line_after_if_statement = maxLine:1
keep_line_after_do_statement =maxLine:1
keep_line_after_while_statement = maxLine:1
keep_line_after_repeat_statement = maxLine:1
keep_line_after_for_statement = maxLine:1
keep_line_after_local_or_assign_statement = maxLine:1
keep_line_after_function_define_statement = keepLine:1
keep_line_after_expression_statement = maxLine:1

there are also some statements that are not in settings, but I don't think they are set up necessary

Fell through the cracks, it definitely works and thanks.


adding additional adjacency state telemetry to isis/feature.textproto

Adding additional adjacency state telemetry paths to isis/feature.textproto:

.../circuit-counters/state/adj-changes
.../levels/level/adjacencies/adjacency/state/up-timestamp

Pull Request Functional Test Report for #1728 / d0a558efb864267c5c7314bdf3ca7f921b41663f
No tests identified for validation.
Help


[Bug] Spec specifies access paths are "sorted", but does not specify sort order

I would consider this a bug in the spec.
On page 15 of the spec https://developers.libra.org/docs/assets/papers/the-libra-blockchain.pdf , section 4.4, it specifies the contents of an "account". It says:

An account is treated as an ordered map of access paths to byte array values. An access path is a delimited string similar to a path in a file system.
In the first iteration of the protocol, we serialize an account as a list of access paths and values sorted by access path

In order to participate in the protocol, it is necessary to sort the access paths, because this is needed to compute the authenticator (hash) for the account state. However, the spec does not specify how access paths are to be sorted. A nice person on Twitter from the Libra/Move team tells me that in the implementation, strings are 8-bit bytes and the strings are sorted by 8-bit byte value:
https://github.com/libra/libra/blob/7c5de4d161e096648dba8ac54328d1568ae2d2e3/types/src/access_path.rs#L255
Which is fine, but the spec should be specific about this. Sorting bytestrings is trivial but sorting human-readable strings is not.
Expected behavior:

The spec throughout refers to "strings" but does not define this term. The spec should specify that "strings" means strings of 8-bit bytes (if this is true).
The reference to "sorted" in 4.4 should be explicit it means unsigned byte-string sort.

Vague suggestions— I wouldn't call these "bugs" but addressing them could improve the spec:

The "access paths" in 4.4 are implied— by the analogy to filesystems— to be intended to be human-readable, things like "libracoin/balance". Maybe you want to support raw binary values as access paths— that could be convenient sometimes! But if you intend access paths to be usually human readable, it would be best to explicitly specify a normative text encoding (UTF-8, UTF-16, ASCII-7) (hopefully not ASCII-7). It would also be best to specify a normative normalization form. If this is left up to individual Move developers, it is probable they will make different decisions about how to encode Unicode text, which could lead to interoperability problems with user tools or each other later on. This will be especially important once user modules are allowed in the main Libra database.
There is a history of security-relevant bugs arising from different software making different assumptions about unicode normalization in path-like strings (although this would be alleviated if the Libra spec were explicit that string equivalence and sorting is based on raw bytes).

4.4 says the access paths are "delimited" but doesn't specify a delimiter. I interpret this to mean the delimiter is arbitrary and Move programs may choose any delimiter they wish. If this is true, this could (similar to failing to suggest a unicode encoding) lead to confusion among Move writers later if different groups choose different delimiters. If this is not true and a specific delimiter is intended, the spec should be changed to specify the delimiter.

Thanks for the report and engaging on Twitter @mcclure! It's great to have detail-oriented people like you look through the paper and find all places it's ambiguous. Having worked on source control for a long time where these concerns tend to be front and center, I'm also personally bothered by the lack of detail here.
So I had some discussions with folks involved and here's what's up:

The paper is not meant to be a spec (and certainly not a final one). There are a number of things that are un- or underspecified at the moment, and that's because we don't know the answer to them.
We're still figuring out the exact encoding scheme. At the moment it's just whatever UTF-8 representation we get (not normalized) plus slashes as delimiters. You've accurately pointed out the security risks and tradeoffs involved, and we still need to make a decision about exactly what we're going to do. (We'll most likely choose UTF-8, but there are more decisions to be made on top of that.)
Separately, the storage model for access paths is also not totally specified yet. For example, we're currently considering hashing access paths before storing them. This doesn't help us bypass the encoding problem of course.

I'm going to keep pursuing this and make sure it's addressed soon. Thanks again!

(note for myself: this is more general, since it also applies to strings in the bytecode string pool)

I did a bit of digging today and it seems like NFKC normalization would be the most appropriate. Any thoughts?

Hey! Just wanted to provide an update about this.
We're going to define a spec for identifiers that's going to be a fairly restricted subset of ASCII, similar to the Rust spec, for V1.
We may introduce Unicode identifiers in the future, and any such proposal will be developed with community input.
We're going to enforce this within both the source language and bytecode. For this implementation we're going to enforce the identifier spec through the type system -- hooray for newtype wrappers!
Once we have that, we should be able to define a proper spec for access paths as well. There are still some details to be figured out there, such as whether access paths should be hashed.

At this point, we're using hashed access paths, right? Is it still important to have a well-defined sorting order for them? @runtian-zhou , can you follow up on this?


Updated to be compatible with current version of cloud logger

Import path is updated to logging_v2
Updated definition of _worker_enqueue

Hi @rakeshmukundan!
I have tried your PR:
pip3 install --user git+https://github.com/snickell/google_structlog.git@refs/pull/6/head
However, it is not working as expected. Every time I send a log (info, warn, etc.), the library enters into an infinite loop, and it sends the log again and again.... I can only stop it by killing the python process.


M1 Capability

Issue description
Currently the repository dependencies like electron (arm based) won't work if you're using a Mac with a m1 processor. I would love to fix all the dependencies errors so that the Desktop / iOs will build successfully.
Wallet version
Which version of wallet are you using?

develop branch

System specification
Macbook Air - macOs Big Sur v11.3.1 - Apple M1 Chip
Is it a repeatable issue?
Y
Steps To reproduce the issue
Just try to setup the repository using an M1 Device.
yarn
Expected behaviour
Install all dependencies and setup the project.
Actual behaviour
Dependencie errors...
Errors
[4/4] 🔨  Building fresh packages... [1/2] ⡀ wallet-nodejs-binding error /Users/ingokopp/Documents/Workspace/firefly/node_modules/electron: Command failed. Exit code: 1 Command: node install.js Arguments:  Directory: /Users/ingokopp/Documents/Workspace/firefly/node_modules/electron Output: HTTPError: Response code 404 (Not Found) for https://github.com/electron/electron/releases/download/v9.4.4/electron-v9.4.4-darwin-arm64.zip     at EventEmitter.<anonymous> (/Users/ingokopp/Documents/Workspace/firefly/node_modules/got/source/as-stream.js:35:24)     at EventEmitter.emit (events.js:376:20)     at module.exports (/Users/ingokopp/Documents/Workspace/firefly/node_modules/got/source/get-response.js:22:10)     at ClientRequest.handleResponse (/Users/ingokopp/Documents/Workspace/firefly/node_modules/got/source/request-as-event-emitter.js:155:5)     at Object.onceWrapper (events.js:483:26)     at ClientRequest.emit (events.js:388:22)     at ClientRequest.origin.emit (/Users/ingokopp/Documents/Workspace/firefly/node_modules/@szmarczak/http-timer/source/index.js:37:11)     at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:647:27)     at HTTPParser.parserOnHeadersComplete (_http_common.js:126:17)     at TLSSocket.socketOnData (_http_client.js:515:22)

I've created a branch to fix this issue.
It's ok if you think this fix is unnecessary.  I just want to make it work for me so i can contribute more stuff :)

+1

Closing issue due to inactivity, please upgrade to the latest app version and reopen the issue if the problem persists 🌷


mouse bug

the custom mouse's position is not equal to the real mouse, ive had this issue happen in alot of scripts but with some theres no issue

disable custom cursor

disable custom cursor

could you tell how


[21100] Include v2.13.5 release notes in master

Description

This PR includes the release notes of the recently release Fast DDS v2.13.5 in master

@Mergifyio backport 2.14.x

Contributor Checklist

[x] Commit messages follow the project guidelines.
N/A Code snippets related to the added documentation have been provided.
[x] Documentation tests pass locally.
[x] Applicable backports have been included in the description.

Reviewer Checklist

[x] The PR has a milestone assigned.
[ ] The title and description correctly express the PR's purpose.
[ ] Check contributor checklist is correct.
[ ] CI passes without warnings or errors.

@richiprosima please test this


Tag Helper

The views used for adding a new app or for editing it has a bug.
This is how it should work: you set the app Confidential using the combobox then you should see all the available roles for choosing which will be used when that app logs in. However, if you choose a public app roles has to be hidden because for that apps what is logged in is the user of the app, not the app itself, and the user has its own roles assigned so no there is no need of selecting any role.
Problem is it doesn't work as described because of tag helper provided by Orchard 2: <script at="Foot"> doesn't load an inline script in Edit.cshtml https://github.com/xkproject/Orchard2/blob/00cdd7dff42dac4b7ab7cc7d5a45b4969c667601/src/Orchard.Web/Modules/Orchard.OpenId/Views/Admin/Edit.cshtml
<script at="Foot">
     $("#Type").change(function () {
         $("#RoleGroup").toggle("slow");
     });
</script>

Same problem happened at Create.cshtml https://github.com/xkproject/Orchard2/blob/00cdd7dff42dac4b7ab7cc7d5a45b4969c667601/src/Orchard.Web/Modules/Orchard.OpenId/Views/Admin/Create.cshtml but there I removed at="Foot" in that way inline script is included but not in the right order so it doesn't work because of its dependencies.

This is what I use in my views:
<script at="Foot" type="text/javascript">
//<![CDATA[
    $(function(){
        Sortable.create(document.getElementById("fields"));
        Sortable.create(document.getElementById("parts"));
    })
//]]>
</script>

And you need to ensure this is added to your _ViewImports.cshtml :
@addTagHelper *, Orchard.ResourceManagement


Use TypeScript instead of JavaScript

It will be pretty annoying to declare types for the various HTML elements, but it has to be done.
Still, if we do so, what bundler should we use rollup or webpack?

Yep, Typescript is neccessary for developing, change the repo into Typescript was listed in TODO list, however, it is limitied by two problem

I haven't learn how to use Typescript yet
Using Typescript may bring some barrier for adding new features, thoght the editor started to get stable

Perhaps you would like to organize it around a framework like React or Angular?


Handling mal-formated output in scoring functions.

What seems like a good approach?

raising errors
log errors and give score = 0 back?

I think we want to be able to do both, what if our solution sometimes does not produce a valid output, we might not want everything to crash..

This is an important question. I believe that a reasonable solution would be to let the  score function raise an exception that is caught and handled in main.py but maybe just allowed to crash when the score module is run.
However is there a reasonable scenario where the output would be sporadically incorrect? If not, we probably want to fail quickly so that any errors can be found and valuable time isn't wasted.

Maybe we are overengineering this, but it looks good. I'll merge and we'll have to test if we actually want to use it :)


Normalise provider values

AAT give us provider values such as Vodacom, CellC, MTN, Telkom_8ta. We need to map these to the values we expect, vodacom, cellc, mtn, telkom.

Ready for review.

Weird travis failure, seems like the cryptography python package isn't working properly with pypy or something. Seems irrelevant to this PR though.

:+1:
And, yeah, Vumi is unhappy on PyPy from Travis right now.


mappings and arrays can also cause recursive struct fields

Closes https://github.com/xermicus/fuzzy-sol/issues/31

@seanyoung I think you are right

The original problem is that fn contains_builtins() does not deal with a mapping that contains itself.

Thanks for the pointer, I'll go over it again

@seanyoung I think we need change the way how fn find_struct_recursion works. Right now, we always error out if a recursive field occurs. Instead, we should have a another (dedicated) check for infinite size? If so, I think my original fix could be correct (but not complete).

On this branch solang goes into a infinite loop with this input:
contract C {
	struct S {
		int f1;
		S[] f2;
	}
}

Found another case which still does not work:
contract C {
        struct S {
                int f1;
                S[] f2;
        }

        function foo(S s) public {}
}

More findings:
contract Test {
	struct S {
		int foo;
		S[] s;
	}
	function test(S calldata s) public {}
}

We compile this. But we should not, solc does not allow it:
Error: Recursive type not allowed for public or external contract functions.
 --> /tmp/test.sol:5:16:
  |
5 | 	function test(S calldata s) public {}
  | 	             ^^^^^^^^^^^^

contract Test {
	struct S {
		int foo;
		S[] s;
	}

	function test(int foo) public returns (S) {
        S ss = S(foo, new S[](2));
        return ss;
    }
}

I have a stack overflow in your branch when I try this example.

@LucasSte your example produces a stack overflow in emit. In order to not bloat this PR and move on (it already contains a bunch of fixes), I'd rather fix this another time. Anyways thanks for the example, I added it to the issue for this problem :+1:

contract Test {
	struct S {
		int foo;
		S[] s;
	}

	function test(int foo) public returns (S) {
        S ss = S(foo, new S[](2));
        return ss;
    }
}

I have a stack overflow in your branch when I try this example.

The overflow here is in the encoder. It enters an infinite loop when calculating the struct size. Maybe this is something for other PR.

contract Test {
	struct S {
		int foo;
		S[] s;
	}

	function test(int f, S[] ss) public returns (S) {

		return S(f, ss);
	}
}

This code causes a stack overflow in sema: fn to_signature_string is called recursively.

My fix for find_struct_recursion is not working for all cases. Need to go over it again. A good way to do this is constructing a graph


1513 configurable activate events

What kind of change does this PR introduce? (check one with "x")

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
This pr fixes this issue
What is the new behavior?
Listeners for activate events are dynamically registered to avoid additional change detection triggers.
User can choose what type of events are registered and emitted via NGX_DATATABLE_ACTIVATE_EVENTS token.
e.g  {provide: NGX_DATATABLE_ACTIVATE_EVENTS, useValue: ['click', 'checkbox']}
By default all events are registered.
Now activate event emitter has proper interface.
Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

Appears no one needs this, and I'm not working angular projects for now

@surya-pabbineedi Can you merge this in? Issue https://github.com/swimlane/ngx-datatable/issues/989 still exists and none of the fixes appear to work.


BIG IMPORTANT QUESTION

What is your question?
What is the var for doTweenAlpha? i looked in the wiki, but there was nothing there for it.

it's a function
doTweenAlpha() --this happens first
function onTweenCompleted(t)
    --all the code in here happens once a tween is completed, just make sure you have an 'if t ==' in here, otherwise, it could lead to some problems from doing a bunch of tweens in songs
end

no, doTweenAlpha(tag:String, vars:String, value:Dynamic, duration:Float, ease:String)

and i need the var.

The variable t will be whatever you set tag to. If you set tag to "Bruh", when the tween is completed t will be "Bruh".

There is no var to get if it was an alpha tween.


Update config.rb to require fileutils

Without fileutils, I get the following error when trying to set an API key with config-forecast:
[ERROR: alfred.workflow.output.script] /Users/kj/Dropbox/Alfred/Alfred.alfredpreferences/workflows/user.workflow.AA22CE0C-25A1-44E5-B3E8-263CAB430CD4/vendor/bundle/ruby/2.0.0/gems/alphred-1.2.1/lib/alphred/config.rb:24:in `update!':
uninitialized constant Alphred::Config::FileUtils (NameError)
	from -e:1:in `<main>'

Requiring fileutils in config.rb fixes this.

Thanks for the PR, but I fixed this upstream in Alphred: https://github.com/kejadlen/alphred/commit/dfa3388e58394d3cb216fe9bb5c9c4fc7d0f7d6b. I'll push an update to this workflow shortly!


Implement advanced keyboard options

Clicking on "Advanced" in the keyboard options menu should bring up a modal allowing the user to toggle things like enabling the developer console.

Might push towards this feature over the weekend unless it's a 1.0.0 thing.

We can push it in with v1.0.0 if you'd like.

If you're interested in being listed on /planimeter for your contributions, give me a heads up, too.

Yeah, sure. I'm a resident of Oslo, Norway.

So, the panel should have developer console checkbox + ?

Cool, welcome to Planimeter.
Also yeah, pretty much just a checkbox for the developer console at the moment. Pretty bleak, but yeah.

Sorry for the delay! This is merged and ready for v1.0.0. Dropping the beta tag after this release.

Pending code drop downstream from vertex-adventure.

Fixed in build 806


Results on DOTA

Thanks for your awesome work. Your work is reallllllly helpful to me.
I did several experiments based on your project, but the results were not quite good. By the way, would you mind releasing the results on DOTA dataset?

I feel very lucky to be able to help you.this project is just the basis of my work on remotesensing images includes DOTA dataset.I suggest you make some improvements according to the basis results and i am doing this too.I will releasing the results on DOTA dataset after a while.

I really appreciate it that you could make your work open-sourced. Look forward to your brilliant paper. Thank you for your suggestions.

Can you give the code converting four points coordinate to bbox content?

Can you give the code converting four points coordinate to bbox content?

You can simply use cv2.minAreaRect(Points) to get what you want.

Why the angle is all smaller than 0.The source in your repository rs_devkit is available?Another problem is that why don't load pretrained resnet model weight and directly load weight from faster rcnn.It is so strange that training in other projects using mmdetection is not like this.The loss cannot convergence in a plane detection problem when I directly use minAreaRect function and cannot convergence if I load resnet weight and don't load faster rcnn weight.The loss is very big always bigger than 1.

bbox content like DOTA2COCOROTATED.py in you rs_devkit project?

All code is tested and works.You can refer README.md to experiment.Data preprocess and postprocess script is in rs_devkit.


remove default (pre-configured) nic from vm

Describe the bug
The challenge vulnerability exploitation has been used to access internet through reverse shell, so apparently default network interface card (nic) has not been removed while setting bridges. It enables users to have internet access through windows xp with existing vulnerability CVE-2003-0352,  for this reason it should be removed from while setting bridges.
Expected behavior
No internet access at all.
Screenshots

Disabled through Haaukins.


Dropdown

The dropdown have stopped working :/ it works for everything else but socialite in the titan bar.
Any ideas?

When mousing over the module, the following lua error is thrown:
Interface\AddOns\TitanSocial\tooltip.lua:208: attempt to call method 'GetBackdrop' (a nil value)

Message: Interface\AddOns\TitanSocial\tooltip.lua:208: attempt to call method 'GetBackdrop' (a nil value)
Time: Sat Jan 22 14:00:46 2022
Count: 1
Stack: Interface\AddOns\TitanSocial\tooltip.lua:208: attempt to call method 'GetBackdrop' (a nil value)
[string "@Interface\AddOns\TradeSkillMaster\LibTSM\Service\ErrorHandler.lua"]:761: in function <...Ons\TradeSkillMaster\LibTSM\Service\ErrorHandler.lua:717>
[string "=[C]"]: in function GetBackdrop' [string "@Interface\AddOns\TitanSocial\tooltip.lua"]:208: in function Clear'
[string "@Interface\AddOns\TitanSocial\TitanSocial.lua"]:1251: in function `TitanPanelSocialButton_OnEnter'
[string ":OnEnter"]:1: in function <[string ":OnEnter"]:1>
Locals: errMsg = "Interface\AddOns\TitanSocial\tooltip.lua:208: attempt to call method 'GetBackdrop' (a nil value)"
isBugGrabber = nil
tsmErrMsg = nil
oldModule = nil
private =  {
globalNameTranslation =  {
}
FormatErrorMessageSection =  defined

@b0tt3dk based on the error message I'm assuming this is in a classic client -- season of mastery or TBC? What other addons are installed?

It is TBC yes. With ONLY titanbar and sociolite active.

@solocommand
It broke when Blizzard launched the pre patch for phase 3.

Hmm make sence. The GB logs dont work either, dunno if sociolite uses any inputs from that.

If you comment out line 208 in tooltip.lua, the error goes away and the dropdown works.  However, it has no backdrop.


[mesheryctl] Add unit tests for utils/sse-client.go

Description
This PR is related to #3982
Have added tests for utils/sse-client.go
Also removed unused get() function, and added return after channel close statements.
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #4176 (6547de5) into master (2227a06) will decrease coverage by 6.98%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #4176      +/-   ##
==========================================
- Coverage   26.91%   19.93%   -6.99%
==========================================
  Files          59       59
  Lines        5031     5027       -4
==========================================
- Hits         1354     1002     -352
- Misses       3262     3730     +468
+ Partials      415      295     -120

Flag
Coverage Δ

gointegrationtests
?

unittests
19.93% <50.00%> (+0.75%)
:arrow_up:

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

Impacted Files
Coverage Δ

mesheryctl/pkg/utils/sse-client.go
92.50% <50.00%> (+92.50%)
:arrow_up:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-62.83%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
0.99% <0.00%> (-53.98%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.85% <0.00%> (-36.76%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.78% <0.00%> (-32.15%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
0.00% <0.00%> (-21.60%)
:arrow_down:

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 8fab9be...6547de5. Read the comment docs.

Will use golden files and reopen PR later

@piyushsingariya hope you could review it whenever you're free :)

This is so great, @krithikvaidya 😃

CITY! Congrats for the PR. and Thanks for the contributions. We expect more btw 😉

Thanks! A little caught up with personal and other technical work these days, so haven't been able to contribute much. Definitely gonna contribute more once I get free :)
And also, what does CITY mean 😅

@krithikvaidya "CITY" : Cool I Trust You. Big deal to get one of those on your PR, Good stuff.


SMS terms and conditions page

https://www.caesars.com/careers/sms-terms-and-conditions

Reopening issue to keep it until https://github.com/hlxsites/caesars-careers/issues/22 is completed for QA


Can't install or run "npm install discord.js"

Hey guys. So I am having some trouble trying to create a personal discord bot because when I run the cmd prompt and do "npm install discord.js" I get some errors.
+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>`-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>No repository field.
I've tried googling this and looking at articles but nothing seems to do anything. I did "npm init" and "npm i --save discord.js --no-optional" but it didn't do anything and I couldn't run it. Halp!

if you don't need music these can be safely ignored.

I would like to use music.

just run
npm i discord.js opusscript
npm i -g ffmpeg-binaries

These dependencies take almost no storage -- why wouldn't these just be included?

We're planning on changing this. 👍

any update on this?


refactor(es): listing change

AMLOS-532
Description
Make Elasticsearch Updates more efficient for Listing Modifications
Right now every time a listing gets updated it calls Elasticsearch multiple times via the Model Save Hook.  Have it so that it only calls elasticsearch once for each listing modification via the Observer Listing Hook (only gets called once)
Acceptance Criteria
Code to make Elasticsearch Updates more efficient for Listing Modifications has been Refactor.
How to test code
make test_parallel

Please Review
@aml-development/ozp-backend-team
Checklist

[x] Read CONTRIBUTING.md
[x] Write code to complete task
[x] Python Code is PEP8 Compliant
[x] Add unit tests for new code
[x] All tests must pass before merging the pull request
[ ] At least 2 people reviewed code

:+1:

Tests passed
:+1:


Fix typo in protected getContainerForConfiguration tests method name

It's protected and only used in a single unit test file... so no BC break here :+1:

:+1:

:metal:


feature request for single nvm command for precmd or PROMPT_COMMAND

It would be a nice feature to add a single recommended nvm command to automatically check for a .nvmrc file and call nvm use if needed.
For example, I created this precmd:
precmd (){
  if [ -s ".nvmrc" ] \
     && [ -n "$(command -v nvm)" ] \
     && [ -n "$(command -v node)" ] \
     && [ "v$(cat .nvmrc)" != "$(node -v)" ]
  then
    nvm use;
  fi
}

which might be shortened to something like this:
precmd(){
  nvm use --auto
}

Obviously, both the nvm command and recommend script are up for debate/refinement but just posing the idea for a feature of this type.

nvm use does this by itself, and is the recommended command for it.
If you want it done automatically, https://github.com/creationix/nvm#deeper-shell-integration has a few options.


With Firefox some framenavigated events are not arriving

Steps to reproduce
Tell us about your environment:

Puppeteer version: 10.1.0 (FIREFOX)
Platform / OS version: Macos 11.4
URLs (if applicable): https://www.youtube.com and any video (e.g. https://www.youtube.com/watch?v=H1HlUe886Hs)
Node.js version: v14.17.0

What steps will reproduce the problem?
Use the following code to log the visited URLs
import {Frame, launch, Target} from "puppeteer";

(async () => {
    const browser = await launch({headless: false, product: "firefox", defaultViewport: null});

    const pageOld = (await browser.pages())[0];

    browser.on("targetcreated", async (target: Target) => {
        const page = await target.page();
        page?.on("framenavigated", (frame: Frame) => {
            if (frame.parentFrame() === null) {
                console.log(`URL Frame = ${frame.url()}`);
            }
        });
        console.log(`URL = ${page?.url()}`);
    });

    await browser.newPage();
    await pageOld.close();

    await new Promise(resolve => setTimeout(resolve, 100000));
})();

Run the script and visit https://www.youtube.com , then after landing on the page with the videos, open one video (e.g. https://www.youtube.com/watch?v=H1HlUe886Hs)
What is the expected result?
The script should log at least both:

https://www.youtube.com
https://www.youtube.com/watch?v=H1HlUe886Hs. (the selected video)

What happens instead?
The selected video (e.g.: https://www.youtube.com/watch?v=H1HlUe886Hs) is never logged. Therefore, I suppose no framenavigated was fired for the video URL.
Is there something missing with Firefox? This problem doesn't happen with Chrome.
Thanks in advance,
Regards
Luis

I'm having exactly the same issue

Please fix it, I have the same issue.


distutils removed in Python >= 3.12

Apache Iceberg version
None
Please describe the bug 🐞
So this crashes in latest Python https://github.com/apache/iceberg-python/blob/a6cd0cf325b87b360077bad1d79262611ea64424/pyiceberg/utils/config.py#L19C28-L19C37

cc @MehulBatra

Adding a function with the same name in the types.py to replace the distutil dependency, as Python 3.12 no longer support the distutil package.


Disable fetching content by default

As a user, I want to get a list of assets from collection from a STAC endpoint.  Currently, when I run the method getAssetFromItem, content is automatically fetched from IPFS.
Additionally, when the Asset object is instantiated, it automatically starts fetching content from IPFS even though a the cid property may be an empty string.
What to fix:
Ensure the default value for any methods containing the parameter fetch_data is false.

I've also introduced a bug that needs to be fixed
        is_pinned = self._is_pinned_to_local_node()
        if fetch_data and not is_pinned:
            self.fetch()

# should probably be

        self.is_pinned = False
        if fetch_data:
            if not self._is_pinned_to_local_node():
                self.fetch()

We would then need to handle the is_pinned' property in the pin` method.
We can also revise the error handling in _is_pinned_to_local_node if we do end up staying with the current logic.

Closing since it's been resolved.


fix after

in reddit scanner.go you have the after var getting set to before. This is breaking pagination
func (s *scanner) Listing(path, after string) (Harvest, error) {
	return s.r.reap(
		path, map[string]string{
			"raw_json": "1",
			"limit":    "100",
			"before":    after,
		},
	)
}

This is intentional. Not necessarily defending the decision, but I believed
at the time that Reddit was incorrect. “after” in the graw API means
“chronologically after”.
At this point it would be a bad idea to change since users and the graw
package depend on these semantics.
You can use ListingWithParams to set whatever parameters you need.
On Fri, Apr 3, 2020 at 8:10 AM Justin Thompson<EMAIL_ADDRESS>wrote:

in reddit scanner.go you have the after var getting set to before. This is
breaking pagination
func (s *scanner) Listing(path, after string) (Harvest, error) {
return s.r.reap(
path, map[string]string{
"raw_json": "1",
"limit": "100",
"before": after,
},
)
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/turnage/graw/issues/71, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWR64GBEPX4REYADF6IOC3RKX37RANCNFSM4L4ESEXQ
.

I’ll re-open this until the a comment about this is added to clear up confusion in the future. Unfortunately I think that’s the best that can be done without breaking users.

I just pulled in 200 mentions using pagination with after and it pulled in the second page with all comments createdUTC earlier than the first page

With which meaning of the parameter “after”? If you pass “after” to Reddit, you will get what is chronologically before. If you pass “after” to reddit.Scanner.Listing, I believe you will receive what comes chronologically after.
This is the way all of the streams in graw work.

i used the listingWithParams and got
end of page 1.             beginning of page 2
1585763973<PHONE_NUMBER>
1585763157<PHONE_NUMBER>
1585763134<PHONE_NUMBER>
1585763072.<PHONE_NUMBER>
1585763016.<PHONE_NUMBER>
1585763009.<PHONE_NUMBER>

If you use ListingWithParams, you are talking to directly to Reddit with
your parameters, so that is expected.
On Fri, Apr 3, 2020 at 9:05 AM Justin Thompson<EMAIL_ADDRESS>wrote:

i used the listingWithParams set to after and got
end of page 1.             beginning of page 2
1585763973<PHONE_NUMBER>
1585763157<PHONE_NUMBER>
1585763134<PHONE_NUMBER>
1585763072.<PHONE_NUMBER>
1585763016.<PHONE_NUMBER>
1585763009.<PHONE_NUMBER>
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/turnage/graw/issues/71#issuecomment-608525323, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWR64B3XEFL77N6PT7ZBF3RKYCK3ANCNFSM4L4ESEXQ
.


Add to meta lists

https://github.com/sindresorhus/awesome
https://github.com/bayandin/awesome-awesomeness
https://github.com/jnv/lists
Dead:

https://github.com/pshah123/awesome-lists

https://github.com/sindresorhus/awesome/pull/1789

https://github.com/bayandin/awesome-awesomeness/pull/411

https://github.com/jnv/lists/pull/154

sindresorhus/awesome#1796


Makefile + @license

@license to help the comment from being stripped by scripts.  Helpful to know license when found randomly in a project (for lawyer's sake).

OK, thanks!

 :)
On Sun, Jan 18, 2015 at 1:34 PM, katspaugh<EMAIL_ADDRESS>wrote:

OK, thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/katspaugh/wavesurfer.js/pull/331#issuecomment-70424528
.


s3ql: 3.3.2 -> 3.7.2

At least one test fails if ran under ZFS.
https://github.com/s3ql/s3ql/issues/249
Motivation for this change
ZHF: #122042
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

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

1 package built:

s3ql

It failed on aarch64. Which filesystem grahamc-aarch64-community-8 uses on /tmp?

/tmp is typically a tmpfs. However, grahamc may have configured it differently for builders

either way, The update still seems worth while. Aarch64 can be done in another PR.


how long can a video be for transcription?

i've read whisper processes videos in 30-second chunks.
my question is how long can the video be transcribed by whisper? 1-hour long, 8-hour long? should a video be used or audio for faster transcription?
i used your code on https://huggingface.co/spaces/sensahin/YouWhisper on a 15 mins video & it accurately transcribed it into ~4117 words.
is there an upper limit? does the program youwhisper cut it into chunks 30-second chunks? i haven't used python in like 5 years so not totally sure about the code.

I think this question need to be asked at Whisper official forum. I couldn't find an exact answer for this as well. I tried 1 hour 30 minute long video and it did work well. So i am not sure what the limit is.
Here is a similar question on Whisper discussion page:
https://github.com/openai/whisper/discussions/136

Since this is related with official Whisper library, i close this issue.

haha the issue you linked to was asked by me only. honestly didn't understand what that guy bcz of jargon said & my lack of knowledge so i thought you had an answer.


Update s2i image from library for ppc64le

What type of PR is this?
/kind bug
What does does this PR do / why we need it:
Similar to PR #3861 . The upstream library updated the link for s2i imagestreams. There are errors when running odo ci script on OCP/P. Errors of following types are observed.
++ oc apply -n openshift -f https://raw.githubusercontent.com/openshift/library/master/arch/ppc64le/official/nginx/imagestreams/nginx-rhel7.json
error: unable to read URL "https://raw.githubusercontent.com/openshift/library/master/arch/ppc64le/official/nginx/imagestreams/nginx-rhel7.json", server reported 404 Not Found, status code=404
++ oc apply -n openshift -f https://raw.githubusercontent.com/openshift/library/master/arch/ppc64le/official/php/imagestreams/php-rhel7.json
error: unable to read URL "https://raw.githubusercontent.com/openshift/library/master/arch/ppc64le/official/php/imagestreams/php-rhel7.json", server reported 404 Not Found, status code=404
++ oc apply -n openshift -f https://raw.githubusercontent.com/openshift/library/master/arch/ppc64le/official/python/imagestreams/python-rhel7.json
error: unable to read URL "https://raw.githubusercontent.com/openshift/library/master/arch/ppc64le/official/python/imagestreams/python-rhel7.json", server reported 404 Not Found, status code=404

Which issue(s) this PR fixes:
Fixes #?
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:

/area system-pz

/retest

@mohammedzee1000 @cdrage please review the PR and leave an /ok-to-test message.

/ok-to-test

/retest

@cdrage @mohammedzee1000 Need approved, lgtm labels.

Looking good
/lgtm

@cdrage awaiting your approval.

/approve

/retest

/retest

/retest


Event and Exception needs to parse on UTC

It's expecting values in UTC, we need to convert values into UTC.
Wiki: https://github.com/NubeIO/flow-framework/wiki/Schedule

We are going to have Local timezone in all.


Include staticcheck Github action

Resolves: #62

This execution is an example that demonstrate that the Github action works appropriately:
https://github.com/latchset/tang-operator/actions/runs/5037358957


Documentation

Hey there!
Thanks for making this lib. I'm missing a real documentation tho.
Would it be possible to request one?
I would suggest using something like docfx :heart:

I don't have the time to invest in a full documentation set. The library is fairly self-documenting and there are plenty of samples that I've included with how to use a lot of common scenarios.
What specifically are you having difficulty understanding?


Support build time environment variables

There are several things I'm forced to do in my dockerfile at "boot" rather than build... simply because I don't have environment variables at build time.
This looks like its being proposed: https://github.com/docker/docker/pull/9176
Is there a work around for now?  Maybe something that echo's the apps environment variables with ENV into the dockerfile right before build?  Maybe our own custom {{ENV}} string that gets replaced before build?  Is this a bad idea? :)

It's always problem. I even though that it could be possible to inject ENVs during ONBUILD steps, but did not succeed.
I need to rethink that.

How about this, right after you clone the repo to /tmp OR right before build... run a script like: https://gist.github.com/that0n3guy/e392d763df4b21c04fd3
It looks simply takes line by line takes the ~dokku/app/ENV file (one line looks like export MYSQL_USER=whatever) turns them into one long string like: ENV MYSQL_USER=whateve FOO=bar MYSQL_DB=thinger.
Then it looks in $DOCKERFILE for the text #dokkuEnv#... if found, it replaces it with the string.
So a dockerfile that looks like:
FROM that0n3guy/baseimage-nginx-phpfpm

#dokkuEnv#

ADD . /app/
RUN rm /app/Dockerfile && mv /app/.docker /build/

RUN chmod +x /build/.docker/*.sh && chmod +x /build/.docker/*/*.sh
RUN bash /build/.docker/addSSHkey.sh
RUN bash /build/.docker/gitCheckout.sh

RUN composer self-update

# Give web server permissions to access app folder
RUN chown -R www-data:www-data /app

becomes:
FROM that0n3guy/baseimage-nginx-phpfpm

ENV MYSQL_USER=whateve FOO=bar MYSQL_DB=thinger

ADD . /app/
RUN rm /app/Dockerfile && mv /app/.docker /build/

RUN chmod +x /build/.docker/*.sh && chmod +x /build/.docker/*/*.sh
RUN bash /build/.docker/addSSHkey.sh
RUN bash /build/.docker/gitCheckout.sh

RUN composer self-update

# Give web server permissions to access app folder
RUN chown -R www-data:www-data /app

What do you think?

That can work. I'm looking through docks and looking for the best approach.

Is there a way to pass environment variables to the buildpack step?  I had an issue with the standard ruby buildpack timing-out during the fetch process on a slow VPS.  In fetcher.rb, the call to curl has timeout set to ENV['CURL_TIMEOUT'] || 30 but I'm not sure how to set the env var.
It was easy enough to fork the buildpack and set a higher default, but I'd rather just pass a var if I can.

Yes. Simply pass configs to app. They should get injected into app during build.

@ayufan Any more thoughts on this?


Removes dist folder from repository.

As mentioned in #596 this PR removes the dist/ folder with generated bundles from the repository.
Only files listed in package.json > files will be published to the npm registry (as well as the LICENCE file).
This will break bower support for now. I deliberately did not consider npmcdn's bower.zip feature because it will be removed in Jan 2017.

Actually, I think we should wait on 2.0 for this change just because it could break things for certain people, but I'll get it merged soon.
Should this wait for 2.0 or is it safe to merge now, @thgreasi?

That's fine with me.
I guess that this would affect people who npm install the git repo directly.

Yeah. I'm partially assuming they'd have a build system or something anyway, I wish I had numbers on them but sadly I don't 😢

So what do we do if we install using bower?

So what do we do if we install using bower?

Honestly bower support is something I think we've chatted about removing and I've certainly thought of. They have stats like npm (see: https://github.com/bower/registry/issues/39) so I don't know how many people use it, but anecdotally npm usage amongst frontend developers seems to be the norm these days. I'd like our npm support to be better; I am less interested in bower.
That said I would be okay waiting on 2.0 to merge this with a notice that 2.0 would drop bower support.

Well I definitely wouldn't support removing bower support. I can work around it and add a build step in my process, but I hope you get real numbers before this happens.

That’s the problem though :-( (see the issue I linked to)
There are no numbers on bower installs short of asking GitHub. I guess I can try it as a one-off though; if they’re way high then you’re right: I won’t nix it (though it’s annoying to maintain when we want to keep dist/ out of the repo).

+1 for a build step. That's what we are doing in angular-ui-sortable.
Also Polymer/webcomponent guys seem to be in favor of Bower as well.
Thodoris Greasidis


Building & publishing MAUI Blazor hybrid apps with GitHub Actions

Adding a doc request placeholder here based on customer feedback from Blazor hybrid survey.
cc: @guardrex
Coverage concepts...

Individual targets (webasm, Android, iOS, Mac, Windows)
All targets
YAML example scripts
Reducing build times
Cost considerations (maximizing ROI on Win/Linux/Mac runners)

We may want to enhance the MAUI docset and link to hybrid as appropriate as this already talks about each of the platforms distribution / store requirements: https://learn.microsoft.com/en-us/dotnet/maui/deployment/
Metadata:

Physical path: [e.g., /blazor/hybrid/topic-url-segment]
Title: Typical limit is 59 characters + the technology name ("ASP.NET Core Blazor Hybrid")
Author: Author's GH handle
Description: Preferred range 115-145 characters long
Framework version:
UID: Usually, matches the physical path [e.g., blazor/hybrid/topic-url-segment]

@Sweekriti91 would appreciate any input on this topic from you :-)


armV6 support

Hi,
kind of sorry for the question but ... armV6 support was dropped intentionally I assume. Any chance I can still run this on an old Raspi? Compile the docker myself - any instructions?
I have seen issue 6 but the instructions there give me an error ...
Tested an old docker that still had V6 support, works fine.
Thanks :)

Docker build fails with old arm versions, unfortunately. I'll look further into the issue, but until then there are binaries for arm starting with armv5:
https://github.com/aceberg/WatchYourLAN/releases


Question - what is a reasonable box configuration for a GROBID REST endpoint

I've been trying to write a quick PDF->XML Spark conversion job leveraging GROBID.  I am remotely reading the PDFs and since the GROBID Java library doesn't support passing in ByteArrays, I decided to spin up a separate server with GROBID that I can send REST transform requests to instead of writing and reading tmp files on the Spark Nodes.  While the code appears to work fine during simple testing, when scaling things up, the GROBID REST server quickly bogs down considerably and I see many Jetty timeouts, very long processing times, etc.  The documentation doesn't really address many details about configuring the server and there are a number of moving parts (Jetty, Grobid, pdf2xml, etc.) in play, so I was hoping you might be able to provide some recommendations based on your experience.
For instance, assuming I have a virtual box with 8 virtual CPUs and 16GB of memory:
How many concurrent connections would be to set in the grobid.properties?
Would you lengthen pool max wait?
Would you modify any Java settings for memory allocation?
Modify any of the Jetty settings?
Etc.
Thanks in advance for any insight you can provide.

Hello @curtkohler
Thanks for the issue!
Your issue made me realized that a change has been made when moving to Dropwizard leading to incorrect status code in the service responses for the PDF processing in version 0.5.0 and 0.5.1 (I overlooked this change when merging, the service for text processing was not changed and OK).
Normally GROBID send a status 503 (service unavailable) when all the threads are used so that the client can wait a bit that some threads become available before re-sending the query. This is the way the service can scale, avoiding mega-queues of PDF queries at the server. The change made a runtime error for this case and accumulating queries is very likely the reason of these problems.
I've corrected this in the current master version of GROBID. In addition I completed the documentation:
https://grobid.readthedocs.io/en/latest/Grobid-service/
I describe for each service the response status codes. I also added some more explanation here:
https://grobid.readthedocs.io/en/latest/Grobid-service/#parallel-mode
In addition, I've written 3 clients that uses the service in the foreseen scalable way:

Java client: https://github.com/kermitt2/grobid-client-java
python client: https://github.com/kermitt2/grobid-client-python
node client: https://github.com/kermitt2/grobid-client-node

Normally with a machine with 8 CPU you should get good performance with the default setting of GROBID and of these clients, you don't need to increase the size of the thread pool or max wait. 16GB memory is enough for exploiting all your available threads.
In the next weeks, I will perform more large scale tests with these clients (with millions PDF) and I will report if everything is fine.

It's not "official", but I can describe a box set up that has been working ok for some time:
Single large worker host, running in a virtual machine: 30 cores, 2.1 GHz, 50 GByte RAM, slow spinning disk (not SSD).
Python workers GET (HTTP) PDFs from remote storage and POST them to GROBID worker, then POST XML response elsewhere, so as little hits disk locally as possible. Run 50x python worker processes (synchronous, single threaded processes; plenty of RAM to do so). Not using the (new) python library, just requests.
org.grobid.max.connections=40
org.grobid.pool.max.wait=1
grobid.temp.path=/run/grobid/tmp
org.grobid.service.is.parallel.execution=true (default)
As an environment variable: TMPDIR=/run/grobid/tmp/
Process called as: ./gradlew run --gradle-user-home .
File logging is disabled; console logging (at WARN level) goes through syslog and does end up on disk. With a slow spinning disk, the fact that pdftoxml could be impacting performance, but doesn't seem too bad.
In aggregate, it takes about 3 core-seconds per CPU to do fulltext extraction, and we can do a million PDFs in about 28 hours.


Determine this repo's direction and policy for 2019

There are a few teams which use MOE, and which need fixes, outside of Google.  Some teams inside google use MOE, others have migrated to Copybara (github.com/google/copybara).  Officially, Copybara is the successor to MOE, but has a different set of use-cases and assumptions.
We should decide whether this open-source repo is going to be updated, and accept patches, or whether it is officially sun-setted, as its earlier maintainer (@cgruber) is willing to continue to patch and update it, but is not at Google, and needs either to spin up a fork or to be re-empowered to make merges on this repository.

cf. @ngzhian @ronshapiro

I think we can continue to keep this in maintenance mode and accept patches as necessary. I don't think we necessarily need to sun-set as there are still users of MOE that aren't ready to use Copybara as you say. This has been relatively stable and when patches come in (which is infrequent) we can figure out how to address them. As for making official releases, I'm happy to do that as well as it should be relatively easy.
So in sum: keep alive, keep changes minimal where possible but still fix bugs and add important features. Encourage Copybara or other tools unless there's a good reason to desire otherwise.

Not sure how I missed this, but it makes sense. I may evolve my fork as a personal hobby, but I can see maintenance mode as being reasonable here.


add withCredentials = false header

Possibly solves https://github.com/spatie/ray/discussions/605

@AdrianMrn Can you see why the tests are failing? Thanks! :+1:


Remove scale restriction on businesses map layer

Just return up to maxResults results.

This is done but not yet testable as we have no live occupant data.

Try STN 309 ANAHIM LAKE -- 6637 Marten Rd, Anahim Lake, BC

Something else is preventing occupants from being displayed at lower zoom levels (eg. full province view) but some occupants do display at some (but not all) lower zoom levels. Further investigation forthcoming...

It appears that the failure is being caused by some small number of occupants not having a valid business category in the current data. If any such occupants are within the bbox, the display fails. There are no such problems against 3.2 with the latest data.

This is working in delivery now.

Verified


Update circe-generic, circe-literal to 0.14.8

About this PR
📦 Updates

io.circe:circe-generic
io.circe:circe-literal

from 0.14.6 to 0.14.8
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
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!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.circe" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.circe" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #307.


Add license

README mentions MIT.
META.json mentions PostgreSQL.

I am not familiar with it, can I use MIT? I added a LICENSE, but not sure if correct.

I like the MIT license.
You should also update this line to MIT: https://github.com/alitrack/duckdb_fdw/blob/62724d27efc2a7ba65a0c40d7af5e8d34a7c42d1/META.json#L7

I like the MIT license.
You should also update this line to MIT:
https://github.com/alitrack/duckdb_fdw/blob/62724d27efc2a7ba65a0c40d7af5e8d34a7c42d1/META.json#L7

done.

Thanks.


Strict templates should report unknown events

Strict templates reports unknown components and unknown props.
I would also like it to report unknown events.
<router-link
    to="/contact"
    @eventDoesNotExist="doSomething"
>
    Text here
</router-link>

In this example vue-tsc should report an error which states that eventDoesNotExist is not defined on router-link.

@so1ve
I would love for this to get some attention, but i also suspect that there are many other more important issues.
I wanted to look into the possibility of fixing this myself, but have not touched this code base before.
Would you be able to give me some pointers on where in this code base that i could start in order to fix this?

You can install the Volar labs extension then you can inspect the generated virtual code in your sidebar.

I've tested locally and this seemed to work

tested locally and this seemed to work

Thanks a lot!
Are you planning to merge this into the next version?

Sorry, I couldn't explain it clearly. I mean vue-tsc has currently reported unknown events (with strictTemplates: true 💦

I tested it now,
you're right, this actually works when no props are present.
And then it stops working when props are assigned.
So it seems like the current behavior is bugged.

Maybe we should remove the 'enhancement' tag,
replace it with a tag indicating that this is a bug.

I'd hate to bump this again, but would be great if someone could look into this bug.
It's an important feature and it's broken.
We have a working reproduction.
Is there anything else i can provide?

Hey @so1ve
Did you figure out a solution here?

Unfortunately I didn't, my solution breaks vue 2.7 tests.


[4.0][BUG] The select2_from_ajax field does not save a null value when the column is nullable and the old value is removed

Bug report
What I did
I have a Product that has a select2_from_ajax field, which is linked to a nullable column. This field had a value set, but I wanted to change it to null by clicking on the "Clear" button that appears at the end of the field. I clicked "Clear" and then saved the form.
What I expected to happen
After saving the form, this column should have null as the value, and on the panel it should display a '-' on the table.
What happened
After saving the form, the column had the same value as before (didn't change).
What I've already tried to fix it
Looked through the select2_from_ajax.blade.php to see if I could find something that can give me a hint at what's going on, but I had no luck there.
However, I saw that after clicking on "Clear", the old <option> of the <select> element was still there, and had the property selected as true (same as the first <option> which has no value). When dumping the request to see what is being sent, the field had the old value still.
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
Laravel Framework version: 6.5.2
Backpack CRUD version: 4.0.22
PHP version: 7.2.9
MySQL version: 5.7.23
Hopefully this made sense. Thank you for the hard work!

Hello @nahuelmino
Thank you very much for reporting this.
I submited a PR #2267  that I think will solve this issue and some other minors.
If you have time, give it a look, test in your app and give us feedback. (It's just one file changed, just copy the whole PR file to your APP to test).
It will really help us on testing more user scenarios. :+1:
I am going to close this issue so any discussion should be addressed in the PR.
If you feel this should be re-opened please be our guest, sometimes we are wrong :)
Best,
Pedro


App crashes when syncing subreddits in the manage subreddits view

I'm not able to reproduce this

Fixed


4.0.2 is not in maven central

Just check this
https://mvnrepository.com/artifact/gs.mclo/api

It can take a couple of hours until a version is synced to maven central. Unfortunately, we can't change that.


example server stays at "connecting..."

Hey,
I get this error when I try to run the server:
Could not subscribe: [object Object]
I'm a ShareJS 0.6.X user and I don't have a good understanding of 0.7+ at the moment so I can't figure out what's wrong, would you mind checking this?

FYI, this still happens on master.

Looks like share.uncompressed.js is missing from the main directory and is being called from index.html.
Go to  /node_modules/share/dist and copy either the minified or regular js file over to the root directory and reference correctly in index.html
This should the demo working.

@christopherdro Thanks, that worked for me but I wasn't happy with using a workaround. When I went into the code to figure out the actual problem, I realized that the scripts in share/dist were still being hosted but they must have been renamed at some point by a ShareJS dev.
All it takes to fix this is renaming share.uncompressed.js to share.js in example/index.html. I'll submit a PR in a bit.


Please get rid of redundant commits

@indexzero
When merging the pull request for fixing isolate to also support coffee script you also included some redundant commits I did make and which have now been merged to master.
How about getting rid of the following commits:
6652a90f699753073979c295a4d4da6e59cbd9cd
6fc9a3aca99a36549a6961035aca9280c9793578
262512586fda3c8b9ab3b946e172e57a74e92363
These commits contain unrelated commit messages as well as dummy changes that were made to retrigger travis builds that were failing at that time.
Sorry for that bogus PR.
Here is a provable solution for getting rid of these commits.
> git rebase -i 4b8362ec38d40937b50b56eafb6f01176d578401^

in the editor you will then have to delete the following lines
pick 6652a90 - felt free to add myself to the list contributors in order to restart the fric$
pick 6fc9a3a - previous build failed due to npm being unable to resolve existing packages, t$
pick 2625125 Update package.json

And finally you simply git push -f master to get rid of these commits once and for all.
Of course, this will leave existing clones to be out of sync and may require the owners of these clones to fix their repositories as well.
In fact, I just tried with my own clone, I did also need to push -f master.
Will now try with a fresh clone of vows and pulling the changes from my clone.
Now, this being done leaves the unwanted commits in my own clone but it will merge just fine. Perhaps the other users need to recreate their clones... sigh, or fix them using the above steps as well.

I have discontinued using vows in favor of mocha as I am transitioning to babel/ES. Closing this now.


Missing tourney

/mjg/ vs may is missing

Will add eventually.

no it's in "others"

maybe it deserves its own spot? The plans are to make it a recurrent thing

I guess I am blind seems okay to me to put it in the others. If it becomes a regular thing yeah we should probably make it it's own thing.