LocationEmbed function option request

Function to choose MAP of streetview or combination of both in a box

Streetview needs the exact coordinates of the location and I don't think people will be writing something like @(12.12,14.54) in the normal text.

Mabey you can use this to convert adres to coordinates:

[code]
var latitude;
var longitude;
/*

Get the json file from Google Geo
*/
function Convert_LatLng_To_Address(address, callback) {
url = "http://maps.googleapis.com/maps/api/geocode/json?address=" + address + "&sensor=false";
jQuery.getJSON(url, function (json) {
Create_Address(json, callback);
});
}

/*

Create an address out of the json
*/
function Create_Address(json, callback) {
if (!check_status(json)) // If the json file's status is not ok, then return
return 0;
latitude = json["results"][0]["geometry"]["location"]["lat"];
longitude = json["results"][0]["geometry"]["location"]["lng"];
callback();
}

/*

Check if the json data from Google Geo is valid
*/
function check_status(json) {
if (json["status"] == "OK") return true;
return false;
}

var address = "streetname number city";
Convert_LatLng_To_Address(address, AlertLatLng);
function AlertLatLng() {
var result = "The latitude is " + latitude + " and longitude is " + longitude;
document.getElementById('result').innerHTML=result;
}
[/code]

maybey you can use this converter to get the coord's.
var latitude;
var longitude;
function Convert_LatLng_To_Address(address, callback) {
url = "http://maps.googleapis.com/maps/api/geocode/json?address=" + address + "&sensor=false";
jQuery.getJSON(url, function (json) {
Create_Address(json, callback);
});
}
function Create_Address(json, callback) {
if (!check_status(json)) // If the json file's status is not ok, then return
return 0;
latitude = json["results"][0]["geometry"]["location"]["lat"];
longitude = json["results"][0]["geometry"]["location"]["lng"];
callback();
}
function check_status(json) {
if (json["status"] == "OK") return true;
return false;
}
Convert_LatLng_To_Address(address, AlertLatLng);
function AlertLatLng() {
var result = "The latitude is " + latitude + " and longitude is " + longitude;
document.getElementById('result').innerHTML=result;
}
var address = "adres number city";

How will you resolve the issue of multiple place with same name ? I don't think anyone is going to write the whole address.

var address = "adres number city COUNTRY/REGION"; can be used too...

now you can check them and report any bugs that you find.

thnx. I updated the embed.js and added the options, but can't see a diference in the map display.

what does your option look like ?

i tried: 'place' or 'streetview' or 'view'

are you using the latest code from the canary branch ?

yes from this location
https://github.com/ritz078/embed.js/commit/6cdb951c8ae15563613f99b995c21d317a425907

any error on the console ?

i think i found the problem :P i was running script on old backup folder.
it works now.
sorry :(

nice that ur problem is solved now :smile:

yeah yeah! next time i look 4times before reporting something :P

push the button to close this :P

its already closed.


Significant Wave heights

How about adding data on wave height e.g. from the NOAA Wavewatch III model  http://polar.ncep.noaa.gov/waves/

I see that waves have appeared on earth.nullschool.net, but not in source. Is it possible to have patch?


Throttling, IP bans, and account deletion coming to yt-dlp users reportedly

Maybe turn off yt-dlp by default or have a "force" arg required.

https://www.reddit.com/r/youtubedl/comments/1c4e3y0/is_it_possible_to_be_permabanned_from_youtube_for/
https://github.com/yt-dlp/yt-dlp/issues/10085

Yt-dlp is already not installed by default. The youtube argument also has to be set to True manually for yt-dlp to activate as there is no automatic detection.

Ok, that seems to be all to be done for now, thanks.


Numpy add numpy linalg norm and numpy min

Description
(Brief description on what this PR is about)
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 http://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
@haojin2

@gyshi Please separate this into 2 PRs: one for linalg.norm, the other for np.min.

@gyshi Please separate this into 2 PRs: one for linalg.norm, the other for np.min.

OK， because i use min in my op linalg.norm, so should I write np.min and test it again in a new PR?

done


Updated nix dependency version to 0.14

This fixes the compilation error on a Raspberry Pi Zero W (and presumably other armhf7 devices).

Attempts to resolve #13.

nix::poll::EventFlags was renamed to nix::poll::PollFlags in upstream. See this pull request.

Hi @drinkdhmo, thanks for the change.  Would you mind confirming that this is working and clean up the branch to remove the merge commit.  Probably just squash all this down to a single commit.
P.S. Repo just moved to rust-embedded org.

Can confirm, this compiles and gpio works on a RPi zero W, with kernel 4.19.42+.

Excellent, thanks @drinkdhmo.  Going to try bors but I think it isn't quite fully setup.

bors r+


Height isn't easy to define

I can't set the height of the slider, I'm loading images with 570px but the slider's wrapper is just 370px, Im using the 'adaptiveHeight' parameter in true. But is not working.
You can see it in:: http://dev.gardel.com.mx
There is a setting to fix it? thank you

Hello,
to me is seems like the height is right, what browser are you using?
adaptiveHeight = true is only needed if the content has different heights, otherwise keep it to false.


Use DAB value as fallback value for service path setting

I kept hitting this error even when it should have been fixed:
  File "/app/venv/lib64/python3.11/site-packages/ansible_base/resource_registry/signals/handlers.py", line 103, in sync_to_resource_server_post_save
    sync_to_resource_server(instance, action)
  File "/app/venv/lib64/python3.11/site-packages/ansible_base/resource_registry/utils/sync_to_resource_server.py", line 73, in sync_to_resource_server
    client = get_resource_server_client(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/venv/lib64/python3.11/site-packages/ansible_base/resource_registry/rest_client.py", line 26, in get_resource_server_client
    return ResourceAPIClient(
           ^^^^^^^^^^^^^^^^^^
  File "/app/venv/lib64/python3.11/site-packages/ansible_base/resource_registry/rest_client.py", line 66, in __init__
    self.base_url = f"{service_url}/{service_path.strip('/')}/"
                                     ^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'strip'

My finding here is that this is (fortunately) a one-off problem in the eda-server settings processing for just this setting. I don't have any problem allowing the dynaconf setting override, so I made this patch to continue allowing that. But if there is not an override present, then we need to fall back to the DAB value, and the traceback above is because it incorrectly gives the None value, after the DAB defaults are already loaded about 100 lines earlier in this file.

Closing, re-opened, against devel.

Re-opening, because we want this against the main branch

"e2e tests on demand" failed with:
2024-09-27 12:50:35,287 Pod 5e58db8a2a6f7abe74ac022b68416137de8e799c86c9ee7f5d848b5eae91f953 has successfully exited. Activation completed. It will attempt to restart in 5 seconds according to the restart policy always.It may take longer if there is no capacity available.
Traceback (most recent call last):
  File "/home/runner/work/eda-server/eda-server/eda-qa/eda_qa/utils/activations.py", line 62, in wait_for_activation
    wait_for(
  File "/opt/hostedtoolcache/Python/3.9.20/x64/lib/python3.9/site-packages/wait_for/__init__.py", line 278, in wait_for
    raise TimedOutError(timeout_msg) from exc
wait_for.TimedOutError: Could not do 'Activation 5 did not reach the expected state(s) ['running'] within 150 seconds. Look at the activation output for more details.' at /home/runner/work/eda-server/eda-server/eda-qa/eda_qa/api/activations.py:76 in time

flaxy test, rerunning @AlanCoding  ;)


remove language and culture

let the browser and the website figure out whether /de-de/ or /en-US/ or any other combination is preferable

Absolutely! Thanks for the fix!


Struggling using the -css option

I'm trying to use the -css option but every time I run it I just get the help screen.
I'm running: azuredevops-export-wiki.exe -css "D:\Development\Projects\Sandbox\DevopsTest.Wiki\DevopsLearning.wiki\releasenotes.css"
Is the path formatting incorrect? How should I be specifying the path.


One-click deployment solution based on WeQing delivery system, contributing to the community

【Screenshot to Code】这个简单的应用程序将截图转换为代码（HTML/Tailwind CSS，或React或Vue或Bootstrap）。它使用GPT-4 Vision生成代码，并使用DALL-E 3生成类似的图片。现在您还可以输入URL来克隆一个活动网站！
安装地址：https://console.w7.cc/api/deploy/thirdparty_cd/redirect?route=/zpk-install?path=https%3A%2F%2Fzpk.w7.cc%2Frespo%2Finfo%2Fchatgpt_screenshottocode
说明：安装地址为一键部署方案，服务器环境基于k3s，微擎交付系统是管理k3s的控制面板，可视化一键管理基于k3s部署的应用
【Screenshot to Code】This simple application converts screenshots into code (HTML/Tailwind CSS, React, Vue, or Bootstrap). It utilizes GPT-4 Vision to generate the code and DALL-E 3 to generate similar images. Now, you can also input a URL to clone an active website!
Installation Link: https://console.w7.cc/api/deploy/thirdparty_cd/redirect?route=/zpk-install?path=https%3A%2F%2Fzpk.w7.cc%2Frespo%2Finfo%2Fchatgpt_screenshottocode
Note: The installation link provides a one-click deployment solution. The server environment is based on k3s, and the Micro-WeChat delivery system is a control panel for managing k3s, allowing for the visual one-click management of applications deployed on k3s.

Is the deployment system open source?

k3s open source,but this control system(WeQing delivery system) is saas service

Not going to promote it here then.


Overlapping of navbar elements

In the provided image, there is an overlap between "Opportunities" and "Agents" in the navbar section. I would like to address and resolve this issue. Can I proceed with resolving it?

can you assign me plz?

I am waiting for the maintainer's approval and I will work on it. Please find some other issue.

Please do open website related issues in https://github.com/PolyPhyHub/PolyPhy-Website
Thanks!


gui-wm/labwc: Correct PATCHES to work with 9999

The PATCHES variable was added to the 9999 ebuild. Because there wasn't a labwc-9999-meson_doc_path.patch, the live version of the ebuild was failing.
I've renamed the PATCH file to labwc-meson_doc_path.patch so that the 0.7.4 ebuild as well as the 9999 ebuild can make use of it. I've also ammended the 0.7.4 and 9999 ebuild to make use of this renamed file via ${PN}-meson_doc_path.patch.
Running pkgcheck scan was complaining about incorrect spacing on the PATCHES array so I also corrected that.

Good point. Thanks for the PR.


conversational-retrieval-agent   using  AgentTokenBufferMemory : Still cannot prevent maximum content length errors

System Info
Based on https://github.com/hwchase17/conversational-retrieval-agent/blob/master/streamlit.py, did few changes to code.
Since the beginning of the tests could not prevent the max content length error.
the best change was using:
memory = AgentTokenBufferMemory(memory_key="history", llm=llm, max_history=5, max_token_limit= 3000)
llm = ChatOpenAI(temperature=0, streaming=True, model="gpt-3.5-turbo",max_tokens=500)
It took longer but still got the error message. All the conversation was logged using LangSmith , username<EMAIL_ADDRESS> ,, https://smith.langchain.com/projects/p/2e2c3411-5910-4785-985d-a4b31702e6c7/r/2893c334-9650-4125-99df-af6e18481b12
It seemed that the context was under control,  but the latest question broke it.
Python version - 3.11.3
Langchain  - 0.0.252
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[X] Memory
[X] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
memory = AgentTokenBufferMemory(memory_key="history", llm=llm, max_history=5, max_token_limit= 3000)
llm = ChatOpenAI(temperature=0, streaming=True, model="gpt-3.5-turbo",max_tokens=500)
Expected behavior
All the conversation was logged using LangSmith , username<EMAIL_ADDRESS> ,, https://smith.langchain.com/projects/p/2e2c3411-5910-4785-985d-a4b31702e6c7/r/2893c334-9650-4125-99df-af6e18481b12

Dear Dosu,
Thanks for your answer. However, it did not work for me. Some comments:
1- I think that the command is wrong:
if self.llm.get_num_tokens(buffer[0]) <= self.max_token_limit:
should me replaced by
if self.llm.get_num_tokens_from_messages(buffer) <= self.max_token_limit:
2- Command buffer.pop(0) does not work, it does nothing,   I ended with an endless loop
Cheers,
Rubens

modified parameters, and it worked so far.

modified parameters, and it worked so far.


Adding demand queue splay tree class and test cases

Rename of #22 !
Great work @RKoulikova. I've added some tests and simplified a bit. Check your email for code review!
This adds the OrderBook and Order classes w/unit and simple usage tests.

@RKoulikova See below and let's be sure to review tomorrow. A few small items and this is a substantial, solid, and clean contribution to the project!
Refactoring remaining

[ ] Combine the Node and Order classes, keep Order as the class name.
[ ] Docstrings for classes see pycmc for examples.

here's what that renders to

[ ] Docstrings for class methods

These can be much simpler such as """Delete the given node from the tree.""".

[ ] In OrderBook._delete_node_helper let's come up with more descriptive names for temporary variables
x, t, and s.

If you see variables elsewhere that need the same treatment please take the liberty to fix.


After Datasette 1.0a9 respect alter-table permission

See comment:

https://github.com/simonw/datasette-edit-schema/issues/55#issuecomment-1935170976

I think renaming a table should require both create-table and drop-table since it has the same impact as removing a table entirely (dropping it) and then creating a brand new one.

This is no longer blocked, 1.0a9 is out now: https://docs.datasette.io/en/latest/changelog.html#a9-2024-02-16

This work will happen in:

https://github.com/simonw/datasette-edit-schema/issues/56


📝 Add details to the usage section of README.md

Add a sub-section to the usage section per main function.

Codecov Report

Merging #16 (c588903) into main (b6dd11d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #16   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           40        40
=========================================
  Hits            40        40

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


[REQ] Auto-Cancel Match if Players Don't Join Within Specified Time

Feature Description
This feature request is for the addition of an auto-cancel functionality in MatchZy that automatically cancels a match if players do not join within a predetermined time frame.
Suggested Implementation

Configurable Time Limit: Allow server admins to set a custom time limit for how long the server should wait for players to join before cancelling the match.
Notification System: Implement a notification system to inform players about the remaining time before the match is cancelled.
Percentage/Number Based Cancellation: Option to set the cancellation criteria based on either the percentage of players needed or a fixed number of players required to start the match.

I believe this feature would be a valuable addition to MatchZy and greatly enhance the experience for both players and server administrators.
Looking forward to hearing your thoughts on this.

Hey!
Thanks for the suggestion. Having a configurable time limit would be great
But could you please help me with understanding what would happen after a match has been auto-cancelled?
For example, the plugin is in pug mode, and match gets cancelled. So should we kick all the players out of the server, or simply shut down the server?
Thanks!

Hey!
Thanks for the suggestion. Having a configurable time limit would be great But could you please help me with understanding what would happen after a match has been auto-cancelled? For example, the plugin is in pug mode, and match gets cancelled. So should we kick all the players out of the server, or simply shut down the server?
Thanks!

Thanks, for reply.
I think like default get5, kick all players (it's depends on autokick on no match or not) and calls cvar get5_endmatch to cancel match.

Got it, I've noted this feature for future and I'll explore more on what get5 used to do!
Thanks! :D

Thank you too!

Any progress on this? I would love a post to the endpoint aswell when a match gets canceled. 😁

Update on this?


Bug Report: Clicking console window pauses application

Per https://serverfault.com/questions/204150/sometimes-powershell-stops-sending-output-until-i-press-enter-why
Legacy console did not pause. However, the new console does pause (I thought randomly...but seems like user input is the cause) while executing long-running processes like npm install.
Could Quick Edit Mode and Insert Mode be deselected by default?
Environment
Windows 10 Enterprise
Steps to reproduce
Click on the window, the console pauses
After

Edit Mode and Insert Mode inside console properties is checked
Running a long-running process
Clicking the console window (text not scrolling)
Leaving for lunch
Return, unlock computer, make console active
Hit enter and progress continues

No progress was being made. This was not a production issue. However, without changing these settings, it could cause production issues.
Expected behavior
Any process that is started continues running
Actual behavior
Process pauses mid-run

This happens with me on legacy console. Running, say node app.js and going into the select mode, the server stops sending response and the browser is stuck on 'waiting for localhost...'. It is only after I click to exit the select mode, the response is successfully received by browser.
This may have something to do with how CMD handles.

Hey @tamusjroyce! Sorry about the delay here. This is, unfortunately, one of those big polarizing features we can't change in the legacy console. There's a couple reasons:

User behavior. The people who love this feature really love it. They'll go to the mats over this feature.
Since console applications get fairly intimate control over the windows they're hosted in (which in hindsight was certainly the wrong decision), some applications actually use this to pause their own output. There's also an API for querying whether output is currently paused (I think.)

We're not going to turn off Quick Edit mode by default, either, as that was hailed as a huge advance in the usability of the console some four and a half years ago.
This one's just going to have to be "by design." Sorry!

My python program was paused accidentally. How can I resume my program?

Try hitting esc


⚠ Bofrost.fr has degraded performance

In 59e2261, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1950 ms

Resolved: Bofrost.fr performance has improved in 5d9f2cd after 7 minutes.


Header component - long service names continue onto a new line

Context
When supplying a long service name to the header component, the service name may "overflow" onto a new line.
See screenshot for example:

Though not a problem in itself, it can make the header and service name look untidy and harder to read.
It also seems the GOV.UK part of the header won't centre vertically in the header, and sits towards of the top instead which can look odd.
A few options that might be useful for such scenarios are:

Be able to change the size of the service name through Nunjucks macro options so it can fit on one line (maybe similar to how heading element text can be given a large, medium or small class size for example)
Be able to make the service name appear under the GOV.UK part instead, similar to how the component displays for smaller screen sizes  to help make it easier to read e.g.

Provide other formatting options (maybe some margin-right or centred text rather than left-aligned) so that more of the words are spread over two lines (rather than just one or two hanging spilling onto the second line) or to make it easer to read over two lines.

Related component
This relates to the existing header component here

Hi @Chirag-Bhatti,
Thanks for the feedback, but I don't think that this is something we want to change at the moment.
To go through it with a finer comb:

When supplying a long service name to the header component, the service name may "overflow" onto a new line.
Though not a problem in itself, it can make the header and service name look untidy and harder to read.

It's expected that the header change in height in response to its contents, including particularly long service names, navigational elements, search forms, and so on.

It also seems the GOV.UK part of the header won't centre vertically in the header, and sits towards of the top instead which can look odd.

This is intentional, so that the GOV.UK logo and link is consistently located in the same position on every page.

Be able to change the size of the service name through Nunjucks macro options so it can fit on one line (maybe similar to how heading element text can be given a large, medium or small class size for example)

This would mess with the hierarchy of the header's contents. The GOV.UK text is largest, followed by the service or product name, followed by navigation.
We don't—and other teams shouldn't—aim to optimise the viewing experience favouring any particular screen size. Shrinking the service name to fit on one line on one viewport size just means there's a slightly narrower viewport or slightly higher zoom level where it now wraps instead. Embrace that text can wrap, rather than try to fight it.

Be able to make the service name appear under the GOV.UK part instead, similar to how the component displays for smaller screen sizes to help make it easier to read

I don't think this is something we want to do for consistency reasons. Service names and navigation changing places between different services isn't ideal, we'd want them to be consistently in the same place.
Some of our current ongoing work involves potentially moving the service name out of the header, likely to appear below the GOV.UK logo instead, though that isn't a certainty at the moment.

Provide other formatting options (maybe some margin-right or centred text rather than left-aligned) so that more of the words are spread over two lines (rather than just one or two hanging spilling onto the second line) or to make it easer to read over two lines.

We advise against using centre-aligned text for accessibility reasons. It can make content harder to read for users with literacy difficulties (e.g. dyslexia) or using screen magnifiers to read.
I don't think adding the ability to apply additional as an option in the component itself is something we would do, as it's too situational and subjective to be of widespread use. This could be a situation that the recent text-wrap: balance value would be a good candidate for, though.
Hopefully our stance here makes sense. Feel free to ask any follow up questions you might have.


Resove dependencies bom with properties on ChangeParentPom

This just demonstrate an issue we are facing with jenkins modernization.
The ChangeParentPom recipe wrongly add a dependency into dependency management when it should not
Such dependency is still managed by the bom.
    +            <dependency>
    +                <groupId>org.jenkins-ci.plugins</groupId>
    +                <artifactId>scm-api</artifactId>
    +                <version>698.v8e3b_c788f0a_6</version>
    +            </dependency>

I think it cause issue because the ${jenkins.baseline} on the artifact id.
But I don't know how and the best way to fix it
Any help would be appreciated
Checklist

[ ] I've added unit tests to cover both positive and negative cases
[ ] I've read and applied the recipe conventions and best practices
[ ] I've used the IntelliJ IDEA auto-formatter on affected files

Yes 2nd commit is the fix. I had some failure locally, but was probably related to my environment.
CI looks good so we can move forward


Add basic React dev docs to dev-guide

Doesn't get into too much detail, but hopefully helps some folks to run React views

Looks good. Would also suggest adding instructions to install nodejs and yarn packages. It seems nodejs >= 10 is required, or else you get
$ yarn install
yarn install v1.22.5
[1/4] Resolving packages...
[2/4] Fetching packages...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=10". Got "8.10.0"
error Found incompatible module.

Done! Thanks for the quick reviews y'all - happy holidays!


review font sizing

Fix #1001

the secondary sidebar was still using a non-existing variable for its size. after removing it it was still piloted by bootstrap
edit font sizing in general to use rem instead of em (secondary and primary were not rendering --pst-sidebar-font-size the same way)
in genreal tried to fix alignement between left and right sidebar

As a rule of thumb I think we should stop using em and px everywhere and stick to rem. px is a absolute value that scale very poorly and em can lead to very complicated set up (like this PR where I just wanted to use the same size on left and right sidebars), it requires to be a very skillful frontend dev to use it and we are not 😄

Looks great in general - one thing i think you missed is the sidebar headings for captions. They are a bit smaller:

I was so close to making it work first try ...

the remaining error seems unrelated to this PR. It's logo related, I think it will be solved by #1131


"auth" is not allowed when running seed script via docker

I haven't tested it outside of docker  to confirm if it is a universal issue with the code
I'm getting this error after setting up appy via docker and then running the seed script
[1] "auth" is not allowed
ERROR    seed.js:629:9 seed —     at Object.exports.process (/backend/node_modules/hapi/node_modules/joi/lib/errors.js:201:19)
ERROR    seed.js:629:9 seed —     at internals.Object._validateWithOptions (/backend/node_modules/hapi/node_modules/joi/lib/types/any/index.js:751:31)
ERROR    seed.js:629:9 seed —     at module.exports.internals.Any.root.validate (/backend/node_modules/hapi/node_modules/joi/lib/index.js:146:23)
ERROR    seed.js:629:9 seed —     at module.exports.internals.Any.root.attempt (/backend/node_modules/hapi/node_modules/joi/lib/index.js:176:29)
ERROR    seed.js:629:9 seed —     at module.exports.internals.Any.root.assert (/backend/node_modules/hapi/node_modules/joi/lib/index.js:171:14)
ERROR    seed.js:629:9 seed —     at Object.exports.inject (/backend/node_modules/hapi/node_modules/shot/lib/index.js:52:17)
ERROR    seed.js:629:9 seed —     at internals.Server.inject (/backend/node_modules/hapi/lib/server.js:309:32)
ERROR    seed.js:629:9 seed —     at seed (/backend/scripts/seed.js:313:31)
ERROR    seed.js:629:9 seed —     at <anonymous>
ERROR    seed.js:629:9 seed —     at process._tickDomainCallback (internal/process/next_tick.js:229:7)

This is the block of code where the issue is happening
    injectOptions = RestHapi.testHelper.mockInjection(request)
    let result = await server.inject(injectOptions)

Specifically failing at server.inject.
If i replace the auth key with credentials and use the same value as request.credentials. the script completes. If this is only an issue with hapijs . please go ahead and close this issue

I am having same issue, is there any update on this?

Apologies, I've been swamped and haven't been able to visit issues in a while. I believe this is due to an update in rest-hapi moving to hapi v18:
https://github.com/hapijs/hapi/issues/3871#issue-377743660
See the section Migration Checklist -> server.inject() Authentication
The fix should be simple and I believe @anurupr mentioned it. I will try to push an update soon.

A quick update here. The issue seems to step from rest-hapi >1.8 updating to hapi 18 while appy is still using other hapi libraries that depend on hapi 17.
A quick fix for now is to set your rest-hapi version to be explicitly v1.7

bump!


Incremented version to 1.2.5

Incremented version to 1.2.5 in 1.2 following from https://github.com/opensearch-project/opensearch-build/issues/1600 to prepare for the next possible release.
Backport the change to higher version branches

@abhinavGupta16 - Could we know the targeted dates for 1.2.5?


Enable bundler caching for travis

Would be interested to know why bundler caching of Travis hasn't been enabled in this repository. Thank you.

Hi @nov. Any reasons why bundler caching is not used in this project? I am doing research and would appreciate any justifications. Thanks.


[Question]: Secure authentication login support?

Describe your problem
Right now, the registration form allows anybody who has access to the tool to register. I would like to know if there is a way to use secure authentication methods that limit the registration to certain type of emails (allowing certain @domain.xxx) or third party methods such as google, microsoft or github authentication.

All these kind of features are easy to implement as long as a slight alteration of front end code.
Github authentication is supported. But you need to alter the front end code to show that button.

Thank you very much for your reply.


Taskbar as a Zone

Summary of the new feature/enhancement
I have a super wide monitor and having the task bar spread so wide is difficult.  I use display fusion to concentrate my task bar in a middle zone.  Were FancyZones able to add the task bar as a definable zone then I could choose the location and width of the task bar within the monitor or other monitors.
Proposed technical implementation details (optional)
When setting up zones have a zone area that can be assigned to be the Taskbar zone.  This might be a specific taskbar zone or just the bottom of another defined zone, depending on how this might be implemented.

Hi @tre4b
thank you for your feedback.
I'm closing this issue in favor of https://github.com/microsoft/PowerToys/issues/503
Please fell free to comment there.
Thank you.


All workers unpredictably hang

After running Celery for a few days, my tasks unpredictably hang.
In the log, I don't see workers exiting, I see notices of the workers having received or completed previous tasks, then nothing. Often the last log message indicates that the MainProcess got a task from the broker, and I see log messages form the individual tasks indicating that tasks were in progress but just stopped dead. The tasks quit at different points, so they're not all stuck waiting on the same resource.
Using celeryctl inspect, I see no active tasks, but a long list of reserved tasks.  Rabbitmq, which I am using as my queue, seems to be up and working fine.  Celeryev shows "Info: events:2 tasks:0 workers:1/1" even though there are 4 worker processes running. celeryctl status reports "OK, 1 node online" as expected.
In gdb the workers show with a stacktrace like this:
(gdb) bt
#0  0xf57fe416 in __kernel_vsyscall ()
#1  0xb775d245 in sem_wait@@GLIBC_2.1 () from /lib/tls/i686/cmov/libpthread.so.0
#2  0xb72137c6 in semlock_acquire (self=0xa01f3c0, args=0xb73d702c, kwds=0x0) at /build/buildd/python2.6-2.6.5/Modules/_multiprocessing/semaphore.c:310
#3  0x080e0a21 in PyEval_EvalFrameEx ()
#4  0x080e2807 in PyEval_EvalCodeEx ()
...
My versions are
python=2.6.5
celery==2.2.6
django-celery==2.2.4
My django-celery settings are
CELERYBEAT_SCHEDULER = 'djcelery.schedulers.DatabaseScheduler'
CELERY_TRACK_STARTED = True
CELERYD_MAX_TASKS_PER_CHILD = 500
CELERYD_TASK_TIME_LIMIT = 320
CELERYD_SOFT_TASK_TIME_LIMIT = 300
CELERY_RESULT_BACKEND = 'database'
BROKER_HOST = "localhost"
BROKER_PORT = 5672
BROKER_USER = "..."
BROKER_PASSWORD = "..."
BROKER_VHOST = "..."
When I shutdown celery via supervisor, it's able to shut down the main celeryd process but none of the worker processes shut down and supervisor gets stuck.  When I kill all celeryd worker processes, and restart the main celeryd process, everything immediately starts working again and celery resumes processing all backed-up tasks.

Could this be related to this issue?
https://github.com/ask/celery/issues/496
I also posted about it previously in https://github.com/ask/celery/issues/492 but didn't go ahead with an upgrade because version 2.3.3 would cause the workers to lock up after around 20 minutes, rather than after several days.

I set that up so they will show the task titles, we'll see what happens when they hang again today.
This happened again last night, after I had increased the max tasks per worker setting, so it's probably not the same issue as the other ticket.

OK, this ran for about 15 hours and now has been hung for 1.5 hours.  Upgrading to version 2.4.6 didn't solve the problem.  Here's what I see under ps:
21358 ?        Sl    11:00 [celeryd@muckrack:MainProcess] -active- (-E -l INFO -v 1)
24340 ?        S      9:39 [celeryd@muckrack:PoolWorker-39] -idle-
24740 ?        S      6:04 [celeryd@muckrack:PoolWorker-42] -idle-
24778 ?        S      3:42 [celeryd@muckrack:PoolWorker-44] -idle-
24814 ?        S      3:16 [celeryd@muckrack:PoolWorker-45] -idle-
24822 ?        S      3:50 [celeryd@muckrack:PoolWorker-46] -idle-
25036 ?        S      0:11 [celeryd@muckrack:PoolWorker-49] -idle-
25060 ?        S      0:00 [celeryd@muckrack:PoolWorker-50]
25061 ?        S      0:00 [celeryd@muckrack:PoolWorker-51]
Here are the last few lines from the celery log
7: WARNING/MainProcess] Skipping revoked task: alerts.tasks.run_alert[1f7fb930-50bd-40d4-a893-75e0f6e4cba4]
[2012-01-24 00:44:56,057: WARNING/MainProcess] Skipping revoked task: alerts.tasks.run_alert[d135a361-1747-4fa5-8b41-90aed74df0b3]
[2012-01-24 00:44:59,063: WARNING/MainProcess] Skipping revoked task: alerts.tasks.run_alert[377db3f3-6e25-41b3-89a3-b97b510ee6da]
[2012-01-24 00:45:00,010: INFO/MainProcess] Got task from broker: explore.tasks.snapshot_trending_terms[305e2bce-8d73-4081-aeb7-489e542bb6f0]
[2012-01-24 00:45:00,012: INFO/MainProcess] Got task from broker: directory.tasks.download_statuses[92217215-d2b9-48d5-a804-0363d76aaf43]
[2012-01-24 00:45:00,016: INFO/MainProcess] Got task from broker: directory.tasks.download_spreadsheet[7e6acee6-cfeb-414f-b810-3430ef8be825]
[2012-01-24 00:46:00,012: INFO/MainProcess] Got task from broker: explore.tasks.update_trending_terms[2d120043-589b-44cc-adf5-a4e40fdbecf4]
[2012-01-24 00:46:00,169: INFO/MainProcess] Got task from broker: directory.tasks.download_statuses[0aa561d9-e1f8-4fc1-bd73-4d15960f385b]

Anything else I can do to help figure out what's going on?

OK, I think this probably not an issue intrinsic to Celery. Celery processes are being killed by the kernel.
cat kern.log| grep Killed -B 10
Jan 25 03:09:28 muckrack kernel: [ 1765]    33  1765    25392    11722   2       0             0 apache2
Jan 25 03:09:28 muckrack kernel: [ 1794]    33  1794    26429    13152   1       0             0 apache2
Jan 25 03:09:28 muckrack kernel: [ 1897]  1002  1897    89528    56232   0       0             0 [celeryd@muckra
Jan 25 03:09:28 muckrack kernel: [ 1899]  1002  1899   101747    54236   0       0             0 [celeryd@muckra
Jan 25 03:09:28 muckrack kernel: [ 1901]  1002  1901    65455    47342   0       0             0 [celeryd@muckra
Jan 25 03:09:28 muckrack kernel: [ 1904]  1002  1904    50568    32384   0       0             0 [celeryd@muckra
Jan 25 03:09:28 muckrack kernel: [ 1906]  1002  1906   138325   120272   0       0             0 [celeryd@muckra
Jan 25 03:09:28 muckrack kernel: [ 1910]  1002  1910   112863    94962   0       0             0 [celeryd@muckra
Jan 25 03:09:28 muckrack kernel: [ 1982]  1002  1982    51348    33481   0       0             0 [celeryd@muckra
Jan 25 03:09:28 muckrack kernel: Out of memory: Kill process 1906 ([celeryd@muckra) score 109 or sacrifice child
Jan 25 03:09:28 muckrack kernel: Killed process 1906 ([celeryd@muckra) total-vm:553300kB, anon-rss:480988kB, file-rss:100kB

I see that a Celery worker process is using up a lot of memory and being killed.
I think the underlying issue is that if any Celery worker child process is killed, Celery itself seems to block entirely until it's manually restarted.  It'd be great if Celery could detect if one of its worker child process ended and not end processing completely.

On 31 Jan 2012, at 16:19, Lee Semel wrote:

I see that a Celery worker process is using up a lot of memory and being killed.
I think the underlying issue is that if any Celery worker child process is killed, Celery itself seems to block entirely until it's manually restarted.  It'd be great if Celery could detect if one of its worker child process ended and not end processing completely.

It should detect lost worker processes and replace them with new ones.
However there is a bug in multiprocessing, or really it is an issue with using
fork combined with threads, that may lead to a deadlock.
This issue is currently being worked on, and a preliminary patch is in
the using_execv branch (#496).
--
Ask Solem
twitter.com/asksol | +44 (0)7713357179

Closed by 363a56cfe10c698a879b91dd2393f9916c2df98c


Implement JpqlSerializer for JpqlIsNotEmpty

Backgrounds

In Kotlin JDSL 3.0.0, the Query objects no longer know how to draw themselves as SQL String.
The Query object is drawn in JPQL String by JpqlSerializer.

Requirements

Please implement  JpqlSerializer that draws JpqlIsNotEmpty as a SQL String.
Please implement a unit test for the implemented class.

Contribution

Please check if there is an assignee in this issue. And if it don't have an assignee, please change the assignee to yourself.
Please check the commit convention through the commitlint.config.js file. The commit  convention also described in the PR template.
When you use intellij, please set the code style through the .editorconfig file.
Please create the PR for develop/3.0.0 as the base branch.
If you have any questions, please leave comment in Kotlin-JDSL Discord Channel in LINE Open Source

저에게 할당 부탁드립니다

This issue is closed by #430


SC4S does not recognized correctly VMWare vSphere logs

Was the issue replicated by support?
No
What is the sc4s version ?
sc4s version=3.22.2
Is there a pcap available?
No
Is the issue related to the environment of the customer or Software related issue?
Unknown
Is it related to Data loss, please explain ?
Seems there no data loss, but data is parcialy classified. The unclassified data are being indexing under sc4s:fallback
Last chance index/Fallback index?
Not classified data are being sending to fallback
Is the issue related to local customization?
No
Do we have all the default indexes created?
Yes
Describe the bug
We are integrating logs from VMWare vSphere, Some logs are correctly sent to the defalt index: index=infraops sourcetype=vmware:esxlog:
Configuration files:
env_file
# AUTOMATIC PARSER CONFIGURATION
#REF: https://splunk.github.io/splunk-connect-for-syslog/main/sources/vendor/VMWare/vsphere/
#Do not enable with a SNAT load balancer
SC4S_USE_NAME_CACHE=yes
#Combine known split events into a single event for Splunk
SC4S_SOURCE_VMWARE_VSPHERE_GROUPMSG=yes
#Learn vendor product from recognized events and apply to generic events
#for example after the first vpxd event sshd will utilize vps "vmware_vsphere_nix_syslog" rather than "nix_syslog"
SC4S_USE_VPS_CACHE=yes
# Enable a TLS port for this specific vendor product using a comma-separated list of port numbers
SC4S_LISTEN_VMWARE_VSPHERE_TLS_PORT=514

app_parser
application app_vps_vmware[sc4s-vps] {
	filter {
		host("<IP_ADDRESS>") or host("<IP_ADDRESS>")
	};

	parser {
		p_set_netsource_fields(
			vendor('vmware')
			product('vsphere')
		);
        };
};

Screenshot of categorized VMWare events under vmware:esxlog:

Screenshot of uncategorized VMWare events:

@ivanfr90 can you copy paste one example for fallback and one for vmware:syslog MESSAGE=* sample for me to check?
It looks like framed events sent to 514[tcp|udp], I want to check 601/tcp on my side
for context please see examples in this test for another vendor: https://github.com/splunk/splunk-connect-for-syslog/blob/428d0d6e3c897310e25e14ad50773966bfc022d1/tests/test_netwrix_epp.py

Hello, @ivanfr90 !
First of all problem here:
SC4S_LISTEN_VMWARE_VSPHERE_TLS_PORT=514
You need to provide non-default port for VMWARE_VSPHERE, in your case all the logs that going to 514 will be identidied as as VMWARE_VSPHERE, but according to RFC is default port.
Second thing:
application app_vps_vmware[sc4s-vps] {
	filter {
		host("<IP_ADDRESS>") or host("<IP_ADDRESS>")
	};

	parser {
		p_set_netsource_fields(
			vendor('vmware')
			product('vsphere')
		);
        };
};

Please check hostname, I think it shouldn't be IP, it should be hostname. You can read here about netmask and hostname.
Second thing
I tried also the following:
netmask(<IP_ADDRESS>/32) or netmask(<IP_ADDRESS>/32)
but seems ignored also.

 For support please open a Splunk support ticket, and for bugs or enhancements please open a new issue, in both cases feel free to refer to this issue

Hi @mstopa-splunk
I see that for your test you are using standard ports: 514 and 601.
In the current SC4S logs are sent using TLS to port 514, but following the recommendations of @ikheifets-splunk now customer is trying to switch to another random port different to standard, ¿could be this configuration affecting in son way to the classification of events?
thanks!!

Hi @mstopa-splunk
It's possible share the SC4S complete config used in your automated test?
In our system part of messages continues being not classified and are being indexed in fallbak and sincerelly I cannot find where the problem is (I have other sources working fine).
My current config:
env_file
SC4S_DEST_SPLUNK_HEC_DEFAULT_URL=https://splunk.instance.com
SC4S_DEST_SPLUNK_HEC_DEFAULT_TOKEN=435345345-344c-42ca-836c-32323434
# Uncomment the following line if using untrusted SSL certificates
#SC4S_DEST_SPLUNK_HEC_DEFAULT_TLS_VERIFY=no
SC4S_SOURCE_TLS_ENABLE=yes
#SC4S_ARCHIVE_GLOBAL=yes

# AUTOMATIC PARSER CONFIGURATION VSPHERE | REF: https://splunk.github.io/splunk-connect-for-syslog/main/sources/vendor/VMWare/vsphere/
#Do not enable with a SNAT load balancer
SC4S_USE_NAME_CACHE=yes
# Combine known split events into a single event for Splunk
SC4S_SOURCE_VMWARE_VSPHERE_GROUPMSG=yes
# Learn vendor product from recognized events and apply to generic events
#for example after the first vpxd event sshd will utilize vps "vmware_vsphere_nix_syslog" rather than "nix_syslog"
SC4S_USE_VPS_CACHE=yes
# Enable a TLS port for this specific vendor product using a comma-separated list of port numbers
SC4S_LISTEN_VMWARE_VSPHERE_TLS_PORT=4514

local/config/app_parsers/app_vps_vmware_v2.conf
application app_vps_vmware[sc4s-vps] {
	filter {
		(netmask(<IP_ADDRESS>/32) or netmask(<IP_ADDRESS>/32))
	};

	parser {
		p_set_netsource_fields(
			vendor('vmware')
			product('vsphere')
		);
        };
};

local/context/splunk_metadata.csv
vmware_vsphere_nix_syslog,index,infraops

hi @ivanfr90 I used a basic configuration: SC4S_DEST_SPLUNK_HEC_DEFAULT_URL, SC4S_DEST_SPLUNK_HEC_DEFAULT_TOKEN, SC4S_DEST_SPLUNK_HEC_DEFAULT_TLS_VERIFY=no. Please open a Splunk support ticket, because it's not an SC4S bug or enhancement request

Hi @ivanfr90 thank you for your help here, in fact SC4S should have been extended with new programs: https://github.com/splunk/splunk-connect-for-syslog/pull/2462/files .
To process octet-counting events please create a port dedicated to Vmware vsphere but on RFC6587:
SC4S_LISTEN_VMWARE_VSPHERE_RFC6587_PORT=4514

Please use it only for framed events. They should no longer go to fallback.
Some of them will be still classified as nix:syslog in the released messages. Please try the PR image: ghcr.io/splunk/splunk-connect-for-syslog/container3lite:pr-2462 which should classify all vsphere logs correctly:

Please let me know if you find any missing cases, else we will include it in the release in 2 weeks.
Also, it should be available to turn off octet counting on the source side

It will be released with next feature release.


"Auto-reflow" on editing

I think it might be useful to have an option to reflow paragraphs after every edit (typing, insertion, deletion, ...), so the user doesn't need to remember to reflow his buffer after writing.
Because this might not make sense for every kind of document, I think it would be reasonable to allow the user to select on which document kinds (identified by their grammar) this feature should be enabled.

:+1:

I like this idea too; any chance of it making it into a new release?

This issue was moved to atom/atom#18033


Rare hanging bug when waiting for wikipedia

Was only reported once, but caused a 10h hang for a validator.
10|s1_vali | 2024-02-16 10:44:49.761 |       INFO       | 📋 Selecting task... from ['summarization', 'qa', 'debugging', 'math', 'date_qa'] with distribution [0.5, 0.5, 0.0, 0.0, 0.0]
10|s1_vali | 2024-02-16 10:44:49.761 |       INFO       | 📋 Creating summarization task...
10|s1_vali | 2024-02-16 20:22:38.092 |     SUCCESS      | Validator killed by keyboard interrupt.

We will address this by running api calls in background threads with timeouts.

Closing since we never saw this again in 2 months


batch-size is incorrectly set to source count when client explicitly sets to less than source count

The PipelineBintr LinkAll function logic to check if the client has explicitly set the batch-size is incorrect (see below).
// If the batch size has not been explicitly set, use the number of sources.
if (m_batchSize < m_pPipelineSourcesBintr->GetNumChildren())
{
    SetStreamMuxBatchProperties(m_pPipelineSourcesBintr->GetNumChildren(), m_batchTimeout);
}

Merged into v0.22.alpha


[logstash] Add option loadBalancerIP to service

Fix https://github.com/elastic/helm-charts/issues/1031

[X] Chart version not bumped (the versions are all bumped and released at the same time)
[ ] README.md updated with any new values or changes
[X] Updated template tests in ${CHART}/tests/*.py
[ ] Updated integration tests in ${CHART}/examples/*/test/goss.yaml

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

jenkins test this please

jenkins test this please

@jmlrt or @ebuildy, could you backport this fix to 7.x and 6.8 versions please ? This is needed for our production release scheduled at the first week of May, 2021.

Hello @jmlrt and @ebuildy, we are also looking forward for this backport on our side for a few weeks now, when do you plan to backport this change?


Version 3

Version 3 is about rewriting Scientist.net for .NET 6 & rejuvenating the project
Some actions to start with:

New Version 3 branch (feature/version-3/net-6 maybe?)
Issue for GitHub action to deploy to NuGet with 3.x.x-alpha
Potential 1.x.x branch for maintance

Great to see all the momentum with the new version.
I don’t think this should be a direct port to .net 6. We should take our time to consider if there improvements or changes etc we’d like to make and what new things .net 6 can give us.

Should we have a discussion about how we will approach the code rewrite to .net 6. Are we planning on porting everything from the current 2.x.x release to 3.0.0?

I've been thinking about this, and IMO for the 3.0.0 release we should strive to have no effective breaking changes behaviorally.
If we need/want to tweak the API that would be fine, but the user should not have to think about the changes much.
Afterwards we can start setting up an obsoletion policy and make deeper changes

Sounds good, I think we port everything across to the new solution with the upgraded .net and get everything setup, finish off all the fluff around docs, actions etc. Then take it from there perhaps?
We already have kinda a policy in place if you check #143

Yeah, sounds like what I had in mind 👍

@JoshHiles I see alot of tickets open for the fluff around the docs/actions etc but what about the port to latest .net version?.
I am working through the wiki. I see alot of new actions created. What about getting started on the version migration?

Apolgies @marblekirby lots on at work!
I think it has already started with the new solution been created but as far as that im not sure anything else apart from the supporting community stuff has been done.
I'll make a start moving it over to the new solution when i have a spare minute

👋 I followed the breadcrumbs from other issues to get here. Am I correct that the re-write ran out of steam and the project is unmaintained, at this point?

@JoshHiles Great! I just got permission from my company to contribute, so I'll probably pull this into a spike I'm working on next week. Is there a discord or something like that for informal collab?

@JoshHiles Great! I just got permission from my company to contribute, so I'll probably pull this into a spike I'm working on next week. Is there a discord or something like that for informal collab?

If you want to use the discussions area that could work?


Run shell provider in a clean bundler environment

execute_and_log eventually forks a child that runs the deployment_command. Unfortunately Heaven's Bundler environment leaks to that process, so deploy scripts are not able to use their own instance of Bundler.
Running the deployment_command inside Bundler.with_clean_env should fix the issue.

@Mumakil any objections? Seems sane.

Looks good! :shipit:


tests: Pinecone - fix test_serverless_index_creation_from_scratch

Related Issues

fixes errors like this: https://github.com/deepset-ai/haystack-core-integrations/actions/runs/9474487185/job/26104185768

Since we are deleting, then creating, then deleting an index with the same name, testing in different environments (python 3.9 and 3.10) may interfere.
Proposed Changes:
I initially thought of dynamically setting the name of the index to be created.
Then I remembered we have a limit of 5 indexes in Pinecone's free tier, which we can easily hit if the index is not deleted (think of test interruptions...).
So, I am using a fixed index name, but I run the test only once in our test suite.
How did you test it?
CI
Checklist

I have read the contributors guidelines and the code of conduct
I have updated the related issue with new insights and changes
I added unit tests and updated the docstrings
I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.

Massi made me notice that I was introducing a tight coupling between CI and code and suggested a better way to fix the problem.
Now I am defining the index name to use in the workflow matrix.


Documentation suggests using a method that appears not to exist

All of the documentation suggests using the run() method as the safe alternative. However, it appears that method was removed in https://github.com/Clikengo/foundationdb-rs/commit/c53f7f67411f5a23f65aacd238f428b6ed348474. It's unclear to me what the suggested API is in 0.5.

In rust, non unsafe methods cannot cause undefined behavior. If found ways to cause UB with the run method.
So unfortunately, I had to rollback to the 0.3 / 0.4 way of starting the network thread which now requires an unsafe block.
This is due to the network thread MUST be stopped before the program exits.
I see there is still references to the now removed run() methods in the documentation, they should be removed.


"No quick fixes available"

Is this no quick fixes available expected? I tried setting "xo.format.enable": true in my VSCode config but it didn't have an effect on this (I don't know if it should?)
Thanks

Seeing the same thing - but given that this plugin hasn't been updated in a year I'm wondering if it's dead.

+1
this affects my DX badly


Variables are wrongly assumed to be always in snake_case

Document:
$foo = 1
$fooBar = 2

What You Are Seeing?

The syntax coloration does not support camelCase for variables.
What is Expected?
camelCase is supported for variables.
How Did You Get This To Happen? (Steps to Reproduce)
Copy paste the document in a file

Thanks for the bug report. The syntax grammar works like that because we use the recommended Puppet style guide for variable naming https://puppet.com/docs/puppet/5.5/style_guide_variables.html#variable-format

Sure, though IMO syntax colorators shouldn't force a style guide, but instead follow puppet's own grammar (which does allow camelcase). To my knowledge, no other syntax colorator does this: I can have camelcase in python, or snakecase in java, and they still are colored correctly.

Thanks for fixing this! However I just found out there is still a minor bug regarding variables in strings:


Do not use ?override=1 when uploading to Bintray

The ?override=1 parameter is only needed to overwrite already published
artifacts. You can overwrite unpublished artifacts without it.

👍 thanks @sjackman!


[WIP] Getting started guide

closed in favor of https://github.com/redhat-developer/ocdev/pull/144


The Repository pattern should decouple data access from the domain

Writing the hypotheses of the "hate" articles as issues and defending them should help us improve as professionals, but also to get to the bottom of all this hate! ;)
Let me start with Rob Conery's article. He starts

It's generally believed that by using the Repository pattern, you can (in summary) "decouple" your data access from your domain and "expose the data in a consistent way".

And refutes it with an MSDN code sample.

If you look at any of the implementations of a Repository working with a UnitOfWork (EF) – you'll see there's not all that much "decoupling":
... (code snippet)...
This class can't exist without the SchoolContext – so what exactly did we decouple here? Nothing.

He makes the following claim for named command/query classes over Repository.

Less Abstraction == Clarity. We've lost our Repositories which didn't have a reason to exist other than to abstract an existing abstraction. Our Command/QueryObject approach is cleaner and the intent of each one is clearer.

A few questions...

Do we accept the Repository decoupling premise?
Do we accept the sample as a prototypical implementation of Repository over EF?
What is the definition of decoupling?
Do we agree that decoupling is a "good thing"?
Does the CQS pattern decouple? How?

Let's pull in Jimmy Bogard's argument about Repository being an unnecessary abstraction.

I don’t think creating an abstraction over your ORM provides much value... Instead of bloating a data access god class, why not just create an isolated class that encapsulates our query?

His solution, named command/query classes, invokes the terms "encapsulation", "isolation", and "dependency injection".
Are these related to coupling?

Ayende and Jimmy both oppose over abstraction. I'm sympathetic to this to some degree, but I refuse to take a direct dependency on DbContext.
I understand the arguments against bloating repositories. The interface keeps growing as the app matures and we need to talk to the DB in more and more ways. I think the MSDN article is fairly representative. It's not much different than what I've been doing.
It's interesting that, to me, my command and query objects are more abstract than the former IRepository<T> I used to use. That doesn't mean I don't like them, but they are farther removed from calling the DB directly.
The thing that I really like about this approach is the explicit unit of work and how few methods live on the Dispatcher. Everything else is in the command and query objects which, thanks to their decoupling from DbContext, can live close to the code that uses them.

I was working my way through the "hate" list, and preparing a response to the first one as I was of a like mind with Alan et al. While reading Rob Conery's article, it seemed easy enough to reject what he actually "hated" was a repo that violated the contract of accepting a Unit of Work, not that a repo was using one. That is, nothing that accepts a Unit of Work should ever start/stop it, abstracting the transaction is kind of its whole purpose for being. I thought he was right be to against that usage, but misplaced the root cause of the issue.
But then I came across something that just made no sense to me. I couldn't for the life of me figure out where he got the idea that a repository "should be atomic". So I decided to go back to the source, PoEAA. And I mean the physical book, not the summary on Fowler's website.
Wow. I don't think my memory of the specifics of a Repository could be that far off.
As best as I can tell, my memory of the Repository pattern is tainted from simplified examples in Evan's DDD book and some of the DDDExample projects that surfaced after it. Continuing my reading, I found that Jimmy Bogard's second article touched on this. In fact, almost every repo I've seen in the wild followed this same pattern, which naturally leads to an IRepository<T>, because DRY.
As a recap, the PoEAA chapter on the Repository pattern clearly shows it as only accepting variations on the Query Object and/or Specification patterns. (Ironically, the example code actually used a base repository class). The concept of having Add/Update/Delete just weren't there. Though I saw nothing that would preclude it getting a Unit of Work either. The Repo's purpose is to abstract the execution of the Query Object instead of the Query Object doing that itself. The Repo does the actual execution by swapping out RepositoryStrategies. The strategies would be the things actually connected to EF, NH, ADO.NET, or some Web API.
As everything begins to come together in that Repository chapter, I find myself thinking that the ultimate culmination of that section of the book is an ORM, and using the abstractions over them for purposes other than DRY for reused queries/commands really is just an accidental implementation of the inner platform effect. That begins to align me to Ayende, Bogard, Smith, et al.
That is, if you're using an ORM, then you already have super powerful mature implementations of generic Specification (ICriteria, ISession.Query, IQueryable<T>), Unit of Work (ISession, DbContext), and Metadata Mapping (DbContext), and the Repository pattern is redundant. However, introducing the Query Object (or Command and Query Objects, or extension methods like suggested in some of the "hate" list) can help with reuse of common queries and/or complex queries.
The crux of the issue then becomes whether or not your architecture will allow composing Queries and Commands into a single Unit of Work or whether each one starts and stops its own.
I'm still not sure I'm comfortable letting my domain know about my ORM though. Having actually swapped out an ORM mid-project, I know it can happen, but for most projects I know it doesn't happen. But something about my domain having access to run db queries or manually manipulate the Unit of Work implementation just feels dirty. Given that 80% of the devs on any one project do not have the rigor of the other 20%, I feel that over time the project would just end up collapsing under the ignorance of the 80% and start looking a lot like ActiveRecord + Transaction Script. Even Microsoft's P&P Group managed to recommend an anemic domain model antipattern as the reference implementation even with the Repository pattern. I have little faith in the commons. I don't want to "protect the developer", but I do at least want to make it easier to do the right thing than it is to do the wrong thing.

Jason, thanks for the terrific commentary. I have always felt that I was missing something essential in the arguments against repositories. From your comments, it looks like I was fundamentally mistaken on what a repository actually is. If I follow your comments correctly, it looks like the IDispatcher in my code is a Fowler repository.
Once I get over being butt hurt over people criticizing the way I've been writing code, I've started to like the approach in my code sample. I still don't know if it holds up to criticism because nobody has commented directly about the code. I do like that it has an explicit unit of work in the IDataContext.
I have tried to make the abstraction around DbContext as thin as possible without taking a direct dependency on EF. This wouldn't allow for switching ORMs but, as you said, that is an edge case. All in all, your comments make me feel better about this approach. I think I will continue to pursue it and see if I can make it more production ready.

I have a hunch the reason "anemic domain model anti-pattern" becomes a common implementation is b/c OO is hard and data access is really very conducive to a function style of list processing and data structures.
The domain model, repository, and other related patterns are an attempt to model things according to OO. However, I have come to find those approaches more of a burden than a help. You won't really reuse any of those things. Why then bother with an OO approach? On the contrary, what you generally want is to retrieve some data structure from a data store. Data access queries typically the process of retrieving a stream / list of data and mapping those into desired data structures. When viewed this way, you are just discussing standard functional programming techniques, not an anemic domain model or transaction script OO patterns.
It's probably worth thinking about use of OO for the boundaries of modules where encapsulation actually matters. Within a module, encapsulation is much less useful. Across modules, you will typically expose only methods for those things that cross boundaries. I tend to think of OO as a paradigm that best fits scenarios where the Actor Model would be a good fit.
I usually tie my domain modules to their data access at a very high level so that the majority of the domain module is just processing lists of data structures provided from elsewhere. These functions are very easy to test. The modules expose an interface of behaviors only, so I still get good encapsulation without having to resort to weird tricks for testing. I also don't rely on or require dependency injection. Data access happens at the same top-level where I first start calling into my domain module functions to produce results.
On Twitter, you noted that this reminded you of how you used to do things, @alanstevens. I started in .NET around the time LINQ to SQL and EF were on the scene, so I never really did a lot of ADO.NET, but I found the micro-ORMs really eased a lot of the problems I had with some of the L2S and EF code I had used earlier. Perhaps it is a return to earlier styles, but I'm not convinced that's a bad thing. It may require a paradigm shift away from the "anti-pattern" noise from OO documents of the past, but the approach is really quite reasonable, simple, and very easy to follow, test, and reason about.

Ryan, thanks for your comments. I'm sympathetic to your arguments. I've made similar ones in the past. For years, I would take a data reader and convert it into a list of objects.
A couple of things motivate my pursuit of the approach under discussion here. First, EF is the elephant in the room. People make that their default data access approach without actually exploring others. That said, I do think there's some value to Identity Map (http://martinfowler.com/eaaCatalog/identityMap.html).
The arguments you make are arguments against ORMs in general. I don't hear nearly enough of those. In the spirit of strong opinions held lightly, I wish there were more debate on that topic. As it stands, I'm left trying to deal with EF in a sane manner. That's my narrow goal here. I don't think this is a complete solution, but it's a start.

I apologize for not making it more clear: I didn't intend to say anything for or against EF. I should have clarified that

found the micro-ORMs really eased a lot of the problems I had with some of the L2S and EF code I had used earlier
was related to the various abstractions I tried to apply around EF / L2S to follow the OO paradigm, including repositories, repository strategies, units of work, etc. I think the larger problem is in trying to make ORMs do far more work for you than they probably should. I disliked EF 1.0 b/c I read the vote of no confidence written by people I looked to for wisdom. I find I now disagree with that "vote." Mapping data into data structures and providing a direct query abstraction directly in the language is actually a very, very useful tool. Just don't let it go too far.

The same may be said for the size of most models I've seen. At a previous company, we had our entire database loaded into a single .edmx. This caused all sorts of problems for us, especially when we wanted to update it. We eventually started using little, targeted .edmx and .dbml files to create bounded contexts and modeled what we needed for a specific domain module.
In short, I don't think the data access tool you use is as important as how you use it. EF is a perfectly valid option. You just have to think about it more as a tool for getting the data you want for the purposes of your module, including direction of relationships, etc., than as a single representation of your database. This of course assumes you follow a modular architecture with bounded contexts.

I would also like to point out I don't "hate" repositories. I find them a less useful pattern for satisfying my goals with data access.

I hadn't heard the idea of making a bounded context by creating a DbContext with a small subset of the data. I find this idea attractive. Modeling a large, complex DB in one DbContext is part of why I hate DbContext. A smaller, targeted DbContext would function well as a repository.

@alanstevens, yes, I was surprised the first time I saw it, as well. We had about 5-7 such DbContexts when I left. Most had some representation of a User that typically varied based on need. We found it worked really well.

You guys have hit on some pain my main project has been feeling. We have a single massive domain model with some 200+ "logical aggregates" in it and poised to continue growing (it's a legacy migration). It's just too big for the tools, much less our heads. We've been looking at Bounded Contexts and Core Domain as solutions but been procrastinating in actually attempting them. We've actually had massive success in the basics of DDD allowing us to get where we are with little pain. Our previous attempts with BO/DO, ActiveRecord, and DataSet as ORM fell apart long before this point.
Something I've found is all of those patterns work so long as you have a "small" context. Small being a context that a single person can grok in about a week on their own. Perhaps theajor solution is to focus on keeping things small, and what you do inside there doesn't really matter. Maybe that's why  Microservices and Evan's push for Bounded Context seem to be taking off as of late.

Check out this feedback from @Ashic:
Part 1: http://www.heartysoft.com/ashic/blog/2015/2/command-query-example-part1
Part 2: http://www.heartysoft.com/ashic/blog/2015/2/command-query-example-part2

Useful feedback. Curious to see where it goes when a Domain model gets added to the mix.
Moving the abstraction of the query/command up a level was unexpected, but I see the benefit. Could be especially useful in a situation where your services component might have polyglot persistence or partial BaaS. Services component at that point would almost act like a mashup layer and the query/command could coordinate across them if needed without building a shared infrastructure all over the place. Helps put the focus on reusable libraries instead of framework integrations.
Current state of it lends itself to having a single command per transaction as well, which I like as it more naturally ends up on a parallel with dispatching events afterward, forces the abstraction of "subcommands" into your domain and/or decouples events from those "subcommands" instead of ending up in the endless event loop, or dealing with unintentional sagas thanks to events being too fine grained for the business domain once you go past basic CRUD.

Going to resurrect this after its three year hiatus. I'm wondering a few things:

Where are we at, as a community, now with CQS vs. Repositories (or Repositories in general) specifically around dealing with DbContext
Have we learned anything new that someone has some concrete examples on where things have evolved to
Testing DbContext seems to have got better (with things like In-Memory databases) so is removing the dependency on DbContext with an interface the answer for that?

How do we test command and query objects?

I don't know what others are doing, but I've adopted Jimmy Bogard's MediatR library for message passing. In the handler, I use a small targeted interface for data access. It's an old school DAO, I suppose, but it works well. It may be on a full repository, but I only see what is specific to my handler. This gives me all the convenience of a repository but without the bloat. This is hardly CQRS, but it meets my needs.
I don't agree with all of Jimmy's conventions, but the core ideas that I use are there: https://github.com/jbogard/ContosoUniversityDotNetCore


IE8+IE7 race condition leads to error when checking for webp support

The newest version of the code errors on older IE browsers.
If a browser doesn't have webp support the picturefill method is called too soon (it's triggered from detectTypeSupport when onerror is triggered creating the image). Error message is
"Object doesn't support this property or method"
because getAllElements is not yet attached to the pf object.
We were further confused because the version number of the latest code is the same as working example found here...  http://scottjehl.github.io/picturefill/dist/picturefill.js

We had the same issue with getAllElements and <= IE8, there's an issue here for it #450 with link to a fork that I created as a workaround


Im quite confused

I was using the library for packet analysis and needed a way to scan for FTP, SSH, and telnet. But it seems you dont have it in here
is that correct?

You may use Gopacket to capture and parse individual packets and then use reassembly package to reconstruct a TCP session.
FTP, SSH, Telnet are application protocols which use TCP as a transport layer. Reconstructed TCP session data may then be used to parse application layer.


CLI: pack release binaries with upx

Compressing with upx seems to bring the binary size down from ~15MB to ~6MB.

there may be drawbacks to consider here as well - online reading indicates this may cause false positives for antivirus programs and may also have performance implications (presumably startup time) on constrained devices.

I don't think 15MB -> 6MB is worth any extra startup time

yeah, I agree - closing


Fix semantic NeRF config bug

Seems that there has been an update to the property naming in the NerfactoModelConfig class, which wasn't propagated into semantic_nerfw.py.
The culprit was num_proposal_network_iterations changing to num_proposal_iterations.

This fixes #821


Docs need meta_html values

On plone6-docs branch, the html_meta values are missing from every page. These values are used to improve SEO. Omitting them results in errors when building the docs with Sphinx.
Note that the values for description and property=og:description should be identical.
html_meta:
  "description": ""
  "property=og:description": ""
  "property=og:title": ""
  "keywords": ""

@stevepiercy I think I can work on this issue, may I?
Should I create a new branch for this or should I start pushing on the updates in the docs-lang branch itself?

@stevepiercy as the plone/documentation is a separate repository, should I create a pull request in that repository after making the modifications?

No. These changes should be made in the volto project only. The main documentation repository imports volto and plone.restapi repositories.


New client-side commands in v21.1

Fixes https://github.com/cockroachdb/docs/issues/9133.
Fixes https://github.com/cockroachdb/docs/issues/9134.
Fixes https://github.com/cockroachdb/docs/issues/9135.
Fixes https://github.com/cockroachdb/docs/issues/9136

This change is

@knz Friendly ping on review :)

@knz TFTR!


FIDO key as MFA method

When using cloud-only authentication, I can use the FIDO key as an MFA method.
When using ADFS, I cannot see that method when signing in.
Is that not supported, or do I need to configure additional settings?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: b8f14188-98ec-5c94-f408-162ee1baa076
Version Independent ID: 41ed4874-e1e6-56d3-4de4-39c024d1e06c
Content: Authentication methods and features - Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/authentication/concept-authentication-methods.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@BakkerJan
Thanks for your feedback! We will investigate and update as appropriate.

@BakkerJan
Yes, you are correct. Currently, the FIDO2 security key sign-in is only
supported for cloud-only authentication and not for ADFS.To use the FIDO2 security key with ADFS,
you would need to configure additional settings.You can find more information on how to
do this in the Azure Active Directory documentation.The link you provided explains
the different authentication methods available in Azure Active Directory and how each method works.
For you reference you can follow below links:
https://learn.microsoft.com/en-us/azure/active-directory/authentication/howto-mfa-adfs
https://learn.microsoft.com/en-us/azure/active-directory/authentication/howto-mfaserver-adfs-2
if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Thanks!
The urls shared do not provide any guidance on how to configure additional settings to support FIDO for ADFS. Is there any guidance around that?

Hi @BakkerJan
Thank you for your response!
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
Thank you for your time and patience throughout this issue.


Logical error in "2016-01-6-Swift-One-Liners-Playground" file

On prime number generation using Sieve of Erathostenes,  you wrote:
//:#10 Sieve of Erathostenes

// With Side-effects
var n = 50
var primes = Set(2...n)

(2...Int(sqrt(Double(n)))).forEach{
     primes.subtract(Set(stride(from:2*$0, to:n, by:$0)))
}
primes.sorted()

// Without Side-effects
var sameprimes = Set(2...n)

sameprimes.subtract(Set(2...Int(sqrt(Double(n))))
                   .flatMap{stride(from:2*$0, to:n, by:$0)})
sameprimes.sorted()

which gives

[2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37, 41, 43, 47, 50]

number 50 shouldn't be here. And this is because of stride.
According to apples documentation stride excludes the end. Therefore, subtract fails to eliminate 50 from prime set.
Please fix the code by including last element in the subtraction sequence. One possible solution is:
sameprimes.subtract(Set(2...Int(sqrt(Double(n))))
                   .flatMap{stride(from:2*$0, to:n+1, by:$0)})

Thanks for pointing this out!
The playground was originally created for Swift 2.2 and this error is the result of a small Swift change happened in 3.0.
Previously, the stride method was part of the Strideable protocol and returned the sequence of values (self, self + stride, self + stride + stride, ... last) where last is the last value in the progression less than or equal to end, this was later modified as you describe.


Migrate Discourse integration to the new framework

Changes proposed ✍
What

🤖 Generated by Copilot at 35081c1
Refactored the discourse integration logic to use the new service SQS queue and the integrations library. Removed the old node worker SQS queue and the DiscourseIntegrationService class. Added new functions and files to handle the discourse API calls, stream generation, data processing, and webhook processing. Commented out the discourse webhook endpoint temporarily.

🤖 Generated by Copilot at 35081c1

We are the masters of discourse
We integrate with force and skill
We use the service queue and library
We comment out the route until

Why
How

🤖 Generated by Copilot at 35081c1

Refactor the webhook processing logic to use a new service SQS queue instead of the node worker SQS queue (link, link, link, link)
Remove the discourse integration service and the integration run creation logic as they are no longer needed after the refactoring (link, link, link, link)
Implement the integrations library for the discourse integration, which defines the type, member attributes, generate streams, process stream, process data, and post process functions for the integration (link)
Implement the functions to fetch the categories, topics, posts, and user from the discourse forum using the discourse API, which are used by the integrations library to process the data for the integration (link, link, link, link, link)
Implement the handler functions to generate the streams, process the streams, process the data, and process the webhook streams for the discourse integration, which are used by the integrations library to initiate and execute the integration logic (link, link, link, link)

Checklist ✅

[ ] Label appropriately with Feature, Improvement, or Bug.
[ ] Add screenshots to the PR description for relevant FE changes
[ ] New backend functionality has been unit-tested.
[ ] API documentation has been updated (if necessary) (see docs on API documentation).
[ ] Quality standards are met.

Great work.
Quick question: When will the docs be updated so anyone can add new integrations?
cc @garrrikkotua @themarolt


CocoaPods spelling in README

Changes in this pull request
Just a nitpick.

This is amazing, and lightning fast! Once the CLA is signed & confirmed I'll merge this. We're pushing the last updates to the readme right now, so this is super helpful!

I just signed it 💯


[Bug] Match Cast Missing After Legalization

When we were trying to get Expand operator supported, we tried to legalize broadcast_to op and found that the operator would fail well formed check after legalization. The cause is match cast went missing during legalization.
The reproducible script is here: https://gist.github.com/zxybazh/5b09bad2cf13ea7d15329a86b03a6abf
CC @MasterJH5574 @tqchen

LegalizeOps does a step of RemoveAllUnused here. https://github.com/tlc-pack/relax/blob/ce5c7f4117e4fcc33894a1449f751f32f8e1460e/python/tvm/relax/transform/legalize_ops.py#L777
It looks to me that the RemoveAllUnused regards the MatchCast as an unused binding, and thereby removes it. While in fact, the MatchCast is a binding with side-effect (where the side-effect is “defining x_0 and x_1).
As a workaround, you can probably try removing the step of RemoveAllUnused. In the long term, I think RemoveAllUnused should be updated to skip the bindings with side-effects, or we introducing a better DCE pass.

This should be a bug of remove unused not considering dep through symbolic variables and we should fix it


Improve match for services of problematic webhook detector

How to categorize this PR?

/area usability
/kind enhancement
What this PR does / why we need it:
This PR limits the scope of services matched by the problematic webhook detector to those in the kube-system (usually managed by Gardener) and default (mainly because of kubernetes service) namespaces. Without this change it was not possible to define and failurePolicy: Fail webhooks on services.
Earlier the code said that services in all namespace should be considered for the KAS bootstrap-controller. This controller however mutates objects directly in the object store (etcd), see both repair controllers.
Release note:

Gardener refined the scope of the problematic webhook matcher for `service` objects. Earlier, shoot clusters were assigned a constraint reporting a problem with a `failurePolocy: Fail` webhook acting on these objects. Now, only `service`s in the `kube-system` and `defaults` namespaces are considered for this check.

/assign

/lgtm

/test pull-gardener-e2e-kind-operator

/approve


Could not find 'rubocop'. The program may not be installed

Description
The results of beautification are not what I expect.
I'm work with rbenv and I did gem install robucop and it did not work.
GIST
Input Before Beautification
This is what the code looked like before:
class BookInStock
def initialize(isbn, price)
@isbn = isbn
@price = Float(price)
end
def to_s
"ISBN: #{@isbn}, price: #{@price}"
end
end

Expected Output
The beautified code should have looked like this:
class BookInStock
  def initialize(isbn, price)
    @isbn = isbn
    @price = Float(price)
  end

  def to_s
    "ISBN: #{@isbn}, price: #{@price}"
  end
end

Actual Output
The beautified code actually looked like this:

Could not find 'rubocop'. The program may not be installed.
Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist:
GIST
Checklist
I have:

[ ] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[ ] Reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[ ] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[x] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

having the same issue. When I try and run
gem install rubocop
I get the following:
`// ♥ gem install rubocop
Building native extensions. This could take a while...
ERROR:  Error installing rubocop:
ERROR: Failed to build gem native extension.
current directory: /Users/rogerperez/.rvm/gems/ruby-2.5.3/gems/jaro_winkler-1.5.2/ext/jaro_winkler

/Users/rogerperez/.rvm/rubies/ruby-2.5.3/bin/ruby -r ./siteconf20190201-95719-di5gax.rb extconf.rb
creating Makefile
current directory: /Users/rogerperez/.rvm/gems/ruby-2.5.3/gems/jaro_winkler-1.5.2/ext/jaro_winkler
make "DESTDIR=" clean
xcrun: error: invalid active developer path (/Library/Developer/CommandLineTools), missing xcrun at: /Library/Developer/CommandLineTools/usr/bin/xcrun
current directory: /Users/rogerperez/.rvm/gems/ruby-2.5.3/gems/jaro_winkler-1.5.2/ext/jaro_winkler
make "DESTDIR="
xcrun: error: invalid active developer path (/Library/Developer/CommandLineTools), missing xcrun at: /Library/Developer/CommandLineTools/usr/bin/xcrun
make failed, exit code 1
Gem files will remain installed in /Users/rogerperez/.rvm/gems/ruby-2.5.3/gems/jaro_winkler-1.5.2 for inspection.
Results logged to /Users/rogerperez/.rvm/gems/ruby-2.5.3/extensions/x86_64-darwin-17/2.5.0/jaro_winkler-1.5.2/gem_make.out`

A SOLUTION:
xcode-select --install
Then try running
gem install rubocop


[Critical!] Add a confirmation box before downloading from chrome plz!

Is your feature request related to a problem? Please describe.
what if i enter a shady website and they auto download some shit, and i don't even get a notification cuz "focus assist, don't disturb or whatever" is on
Describe the solution you'd like
when the extension grabs the download a box should appear with information about the download (file name & link) + 2 buttons to either confirm or cancel the download
Describe alternatives you've considered
this is pretty much what needs to happen ( "free download manager" implementation )

any plan on this??


feat: the homology sequence of a distinguished triangle in the derived category

In this PR, we define homologyFunctor C n : DerivedCategory C ⥤ C, show they are homological functors and construct the long exact sequences associated to distinguished triangles in the derived category.

Thanks!
maintainer merge

Thanks!
bors merge


User nonroot commands while installing GH action

User reported from slack:
Hi. I just tried running digger and stumbled over this line which produces an error in my workflow:
https://github.com/diggerhq/digger/blob/15ccff47be4602502cd98b55922e751f79610224/action.yml#L247. Is this action
supposed to run as user root? The path is usually not accessible as a regular user and running just everything as root
seems awefully weird.

Proposed solutions are:

appending sudo to the mv command to make sure it is run as previlliged user
instead of using a mv command to append it to $PATH variable however still not sure if this will be correctly properly propagated in custom workflows

it should be fixed in the next release


Throw exception: More bytes written than specified in the Content-Length header

Hi, I build the self-hosting, but the service keeps receiving error logs. What could be the problem?
Version: Exceptionless-7.0.1-pre13
Number of organizations: 13
Number of projects: 600+
ERROR LOG:
2020-11-12 10:39:45.313 +08:00 [ERR] HTTP GET /api/v2/projects/config?v=0 responded 500 in 1.1874 ms
System.InvalidOperationException: More bytes written than specified in the Content-Length header.
at Microsoft.AspNetCore.Server.HttpSys.ResponseBody.CheckWriteCount(Nullable`1 count)
at Microsoft.AspNetCore.Server.HttpSys.ResponseBody.WriteAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
at Microsoft.AspNetCore.Server.HttpSys.ResponseStream.WriteAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
at System.IO.Pipelines.StreamPipeWriter.FlushAsyncInternal(CancellationToken cancellationToken)
at Exceptionless.Web.Utility.ProjectConfigMiddleware.Invoke(HttpContext context) in C:\Localhost\Exceptionless-7.0.1-pre13\src\Exceptionless.Web\Utility\Handlers\ProjectConfigMiddleware.cs:line 62
at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.HttpOverrides.HttpMethodOverrideMiddleware.Invoke(HttpContext context)
at Serilog.AspNetCore.RequestLoggingMiddleware.Invoke(HttpContext httpContext)
2020-11-12 10:39:45.320 +08:00 [ERR] ProcessRequestAsync
System.InvalidOperationException: More bytes written than specified in the Content-Length header.
at Microsoft.AspNetCore.Server.HttpSys.ResponseBody.CheckWriteCount(Nullable`1 count)
at Microsoft.AspNetCore.Server.HttpSys.ResponseBody.WriteAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
at Microsoft.AspNetCore.Server.HttpSys.ResponseStream.WriteAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
at System.IO.Pipelines.StreamPipeWriter.FlushAsyncInternal(CancellationToken cancellationToken)
at Exceptionless.Web.Utility.ProjectConfigMiddleware.Invoke(HttpContext context) in C:\Localhost\Exceptionless-7.0.1-pre13\src\Exceptionless.Web\Utility\Handlers\ProjectConfigMiddleware.cs:line 62
at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.HttpOverrides.HttpMethodOverrideMiddleware.Invoke(HttpContext context)
at Serilog.AspNetCore.RequestLoggingMiddleware.Invoke(HttpContext httpContext)
at Exceptionless.Web.Startup.<>c__DisplayClass5_0.<b__1>d.MoveNext() in C:\Localhost\Exceptionless-7.0.1-pre13\src\Exceptionless.Web\Startup.cs:line 204
--- End of stack trace from previous location where exception was thrown ---
at Joonasw.AspNetCore.SecurityHeaders.Csp.CspMiddleware.Invoke(HttpContext context)
at Foundatio.Extensions.Hosting.Startup.WaitForStartupActionsBeforeServingRequestsMiddleware.Invoke(HttpContext httpContext)
at Microsoft.AspNetCore.Builder.Extensions.MapWhenMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.Builder.Extensions.MapWhenMiddleware.Invoke(HttpContext context)
at Microsoft.AspNetCore.Server.HttpSys.MessagePump.ProcessRequestAsync(Object requestContextObj)
at Microsoft.AspNetCore.Server.HttpSys.MessagePump.ProcessRequestAsync(Object requestContextObj)
Thanks.

Hello,
Can you try upgrading to the latest release which may resolve this issue (we'll also be cutting a release shortly with .NET 5.0 RTM support from the last rc). Also are you able to determine what the user agent of the client making this request?

Hello,
Can you try upgrading to the latest release which may resolve this issue (we'll also be cutting a release shortly with .NET 5.0 RTM support from the last rc). Also are you able to determine what the user agent of the client making this request?

Thanks for your reply, I checked the code of Exceptionless-7.0.1-pre13 and the latest version of GITHUB, there is no difference in ProjectConfigMiddleware.cs. I try running to debug mode.
GET /api/v2/projects/config?v=0&access_token=XXXXXXXX

Result:
2020-11-13 12:05:18.851 +08:00 [ERR] HTTP GET /api/v2/projects/config?v=0&access_token=XXXXXXXX responded 500 in 6135.7303 ms System.InvalidOperationException: More bytes written than specified in the Content-Length header.
But when I comment out "context.Response.ContentLength = json.Length;", I can respond normally.

Is this an issue of unexpected content length output? Thanks.

The latest release version is 7.0.7 and it contains .NET Core 5.0 RTM, the build you are on is a nightly build. It could very well be. Is there any chance you could add a unit test for this as well as put that data exclusion as a string here (non image)? Can you see if the latest version resolves this? I don't think it will after your last evidence but it's worth verifying. Is there a better way to set the content length that you know of?

OK, I will try after upgrade my development environment, thank you.

The latest release version is 7.0.7 and it contains .NET Core 5.0 RTM, the build you are on is a nightly build. It could very well be. Is there any chance you could add a unit test for this as well as put that data exclusion as a string here (non image)? Can you see if the latest version resolves this? I don't think it will after your last evidence but it's worth verifying. Is there a better way to set the content length that you know of?

Hi, I used version 7.0.7 for testing, but the same issue exists. I am not sure how to write a test case for Middleware, but I give two examples as follows.
https://github.com/exceptionless/Exceptionless/blob/cc472b1c8161d45dec0867de4d7f6f72de92ab82/src/Exceptionless.Web/Utility/Handlers/ProjectConfigMiddleware.cs#L57
// Contains Chinese
var jsonStr1 = "{\"version\":10,\"settings\":{\"@@UserAgentBotPatterns\":\"*bot*,*crawler*,*spider*,*aolbuild*,*teoma*,*yahoo*\",\"@@DataExclusions\":\"*unregistered*,*取得登入认证发生错误*,*This account unregistered!*\"}}";
context.Response.StatusCode = StatusCodes.Status200OK;
context.Response.ContentType = "application/json";
context.Response.ContentLength = jsonStr1.Length;
await context.Response.WriteAsync(jsonStr1); // Error: More bytes written than specified in the Content-Length header.

// Without Chinese
var jsonStr2 = "{\"version\":10,\"settings\":{\"@@UserAgentBotPatterns\":\"*bot*,*crawler*,*spider*,*aolbuild*,*teoma*,*yahoo*\",\"@@DataExclusions\":\"*unregistered*,*This account unregistered!*\"}}";
context.Response.StatusCode = StatusCodes.Status200OK;
context.Response.ContentType = "application/json";
context.Response.ContentLength = jsonStr2.Length;
await context.Response.WriteAsync(jsonStr2); // No exception

Because the project configuration may contain Chinese, maybe we should encode the content first.
byte[] jsonBtyes = Encoding.UTF8.GetBytes(_serializer.SerializeToString(project.Configuration));
context.Response.StatusCode = StatusCodes.Status200OK;
context.Response.ContentType = "application/json";
context.Response.ContentLength = jsonBtyes.Length;
await context.Response.Body.WriteAsync(jsonBtyes);

In order to be able to calculate the length correctly, do you have any suggestions? Thanks.

I'm not sure, we will need todo some research on that. Would you mind looking into this and reporting your feedback. Can you check to see if the runtime will automatically set the content length for us? I'm wondering if we even need to set it.

I checked the aspnetcore source code and found that encoding is used to calculate the length inside WriteAsync.
https://github.com/dotnet/aspnetcore/blob/6de6ba907271a652e77c98a262e7ab6409c27930/src/Http/Http.Abstractions/src/Extensions/HttpResponseWritingExtensions.cs#L103
So when the final check is sent, the content-length of JSON string length will be inconsistent with the actual encoding length.
https://github.com/dotnet/aspnetcore/blob/f490dd24c924e32acb46d0dee6b838237172d5e8/src/Servers/HttpSys/src/RequestProcessing/ResponseBody.cs#L511
I think there is no need to set the content-length, because asp.net core will respond to chunked. Thanks.

Thanks again for bringing this to our attention and investigation! I just fixed this issue and added some test coverage for this. I also confirmed we don't need to set the length and it will be calculated (as talked about here https://weblog.west-wind.com/posts/2020/Mar/29/Content-Injection-with-Response-Rewriting-in-ASPNET-Core-3x). Please use a nightly ci build to confirm this is fixed for you. We will be pushing a release very soon.


fix: bypass Date for AV._decode

Current coverage is 65.47% (diff: 100%)

No coverage report found for next at a80bacb.

Powered by Codecov. Last update a80bacb...1aa747c


update dependencies in json files

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (9997946) 98.23% compared to head (ba0ff4b) 98.23%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   98.23%   98.23%
=======================================
  Files           1        1
  Lines         113      113
=======================================
  Hits          111      111
  Misses          2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add family "similar to" key in target.json

@bogdanm @autopulated

+1
what about "kinetis-k64" ? That one is in keywords :-) I wonder how you are using kinetis

We use it here (at the moment it is still kinetis-k64).
Porting to uVisor is now made a family at a time (for reference: https://github.com/ARMmbed/uvisor/pull/142), so we need to know the family of a given target -- kinetis-k64-gcc is too granular

Is this ready to merge? @0xc0170


get_latest_bundle now uses latest_in_channel field

@apaplaus PTAL
For now it's just a WIP since I have to fix the tests yet - had a bit of a problem to run them since I don't have rpm based distro.

Maybe it would be better to squash all those commits into one, to have a cleaner history?


[Infra] Add Release Note for ONE-vscode v0.3.0

This commit adds release note for ONE-vscode v0.3.0
ONE-vscode-DCO-1.0-Signed-off-by: yunjay-hong<EMAIL_ADDRESS>

Let's wait till the documents has been landed.

@lemmaa PTAL :-)