FATA[3760] salt-call completed but had failed states

Installing teamdfir/sift ends with errors.
OS is Ubuntu 20.04.6 LTS  and cast version is v0.14.52 (I created the folder /etc/apt/keyrings/)
INFO[3757] state completed                               component=installer duration=0.91 state=sift-config-tools time_begin="23:32:13.331764" time_end="23:32:13.332675"
INFO[3757] state completed                               component=installer duration=5.03 state=sift-config time_begin="23:32:13.343946" time_end="23:32:13.348975"
INFO[3757] state completed                               component=installer duration=0.726 state=sift-desktop-include time_begin="23:32:13.349175" time_end="23:32:13.349901"
INFO[3760] log file location                             component=installer file=/var/cache/cast/installer/logs/saltstack.log
INFO[3760] results file location                         component=installer file=/var/cache/cast/installer/logs/results.yaml
WARN[3760] first failed state                            comment="One or more requisite failed: sift.python-packages.volatility.sift-python-volatility-community-plugins" component=installer run_num=285 sls=sift.python-packages.volatility
INFO[3760] statistics                                    component=installer failed=43 success=630 total=673
INFO[3760] salt-call completed but had failed states     component=installer
FATA[3760] salt-call completed but had failed states

saltstack.log

Yup. Just discovered. Digging into that now

@jofrep this is fixed with 2024.11.01 release of sift.


Fixes for upgrade mode

Uses correct flag for draining with a pod selector
Verifies minimum kubectl version for compatibility

ci check this

/lgtm

/approve
/lgtm


Error when tagInstagram has not been added to config

I installed hexo-tag-instagram,
but didn't add the tagInstagram option to my hexo config
then I got following error in the console:

TypeError: Cannot read property 'captioned' of undefined

I've created fix. Please check it: #2
It checks for config and, if tagInstagram is not defined then sets the default value.

Thank you for your advice. Please update npm version .


The request property 'isdialog [#30174]' is not supported after close and re-open application

Describe the bug
In my program, the application need to be closed and re-open after one task finished then start another task. But some properties such as "IsDialog","IsOffscreen" not support after the application close and re-open.
Code snippets
using FlaUI.UIA3;
Process[] processList = Process.GetProcesses();
foreach (Process process in processList)
{
if (process.ProcessName == processName)
{
process.Kill();
Thread.Sleep(200);
}
}
ProcessStartInfo processInfo = new ProcessStartInfo(appPath);
processInfo.WorkingDirectory = workPath;
application = FlaUI.Core.Application.Launch(processInfo);
var windows = application.GetAllTopLevelWindows(automation);
if (windows.Count() > 1)
{
foreach (var window in windows)
{
Thread.Sleep(200);
if (!window.Properties.IsDialog)
{
......
}
}
}
Screenshots
Add screenshots of FlaUInspect/Inspect to show your controls with issues.

Instead of forcefully terminating the app try to do application.Close() instead. Otherwise FlaUI might still have some state of the old application (just a guess).


When the component unmount

Hi Krasimir,
I found that when components unmounted, the style tag in the head is still there.
I think it will be the problem when the project huge. ( ex. using react router)
Is it possible to clean style tag in the head when component is unmounted?
Thanks :D

Definitely. Thanks for reporting. I'll try resolving this during the weekend.

Sorry for the big delay. It's fixed now in 1.1.1 version.


MWPW-124568: Floodgate Admin Base Page

The floodgate admin page follows the pattern of Milo Localization in terms of code/content setup and processing, especially given the overlap of functionalities and Sharepoint API usage.
Creates a base admin page for Floodgate
Authenticates with sharepoint
Reads the Floodgate config and project excel file and renders the data on the page
Checks for Source and Floodgated page in sharepoint and renders the link and status

Resolves: MWPW-124568
Test URLs:

Before: N/A
After: https://floodgate-base--milo--sukamat.hlx.page/tools/floodgate/index.html

Verified that the floodgate UI reads information from the project files and display status of the source and floodgate content.
Currently only the view of floodgate UI is available with no actions like copy, promote - confirmed the view is rendered.

Validated that there are no regression issues on loc due to the addition of the floodgate param and movement of functions to util file.


Change from blockstream API to mempool.space

Check rate limiting before diving in

@markmhx can we close this issue?

Yep!


[BUG] Got a connection error when I send a request

Expected Behavior
Returns a valid response and not a timeout error
Actual Behavior
Ends up by returning a timeout error. I am unsure if it's a bug or something I am doing wrong, but the example is simple, so maybe I am missing some parameters.  The example is simple, so maybe I am missing some parameters.
Running on Docker Engine 20.10.14 on Mac OS.
Steps To Reproduce
#syntax=ghcr.io/sozercan/aikit:latest
apiVersion: v1alpha1
models:
  - name: uncased-sentiment
    source: https://github.com/lordofthejars/bert-base-multilingual-uncased-sentiment_onnx/releases/download/1.0.0/model.onnx

Then I build:
docker buildx create --use --name aikit-builder

docker buildx build . -t my-model -f aikitfile.yaml --load

Then I start it:
docker run -ti --rm -p 8080:8080 my-model

Finally I curl the model:
curl http://localhost:8080/v1/chat/completions -H "Content-Type: application/json" -d '{
     "model": "uncased-sentiment",
     "messages": [{"role": "user", "content": "It is so good"}]
   }'

And in the container console I got the following error:
rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing: dial tcp <IP_ADDRESS>:39863: connect: connection refused"
rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing: dial tcp <IP_ADDRESS>:39839: connect: connection refused"
rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing: dial tcp <IP_ADDRESS>:33679: connect: connection refused"
```

Any idea why this is happening? The example is a simple, not complicated lifecycle.

### Are you willing to submit PRs to contribute to this bug fix?

- [ ] Yes, I am willing to implement it.

@lordofthejars I am not sure if onnx format is supported by LocalAI. You can check the compatibility here: https://localai.io/model-compatibility/

Thank you very much. One question, do you know if any of these files https://huggingface.co/nlptown/bert-base-multilingual-uncased-sentiment/tree/main are supported by LocalAI. I end up loosing a bit between so many formats. Thanks.


Accuracy analysis of the results of tests

Here we can discuss our conclusions about the accuracy of the results obtained from each test.
Passed Tests
List of tests for which the results accuracy is as expected for the formal order of the numerical scheme employed:

[ ] Lorenz' test:

[ ] explicit:

[ ] Adams-Bashforth;
[ ] Euler;
[ ] Leapfrog;
[ ] Runge-Kutta:

[ ] TVD/SSP;
[ ] Low Storage;

[x] Oscillation test:

[x] explicit:

[x] Adams-Bashforth;
[x] Euler;
[x] Leapfrog;
[x] Runge-Kutta:

[x] TVD/SSP;
[x] Low Storage;

[ ] Burgers' test:

[ ] explicit:

[ ] Adams-Bashforth;
[ ] Euler;
[ ] Leapfrog;
[ ] Runge-Kutta:

[ ] TVD/SSP;
[ ] Low Storage;

[x] 1D Euler test:

[x] explicit:

[x] Adams-Bashforth;
[x] Euler;
[x] Leapfrog;
[x] Runge-Kutta:

[x] TVD/SSP;
[x] Low Storage;

Issues
Complete the accuracy analysis

Hi all,
I have refactored 1D Euler and Oscillation tests in order to facilitate yhe errors analysis for accuracy assesment. I have added some scripts (pytjon snd bash) for automating tbe errors analysis. Essentially I have completed the analysis for the oscillation test (that you csn easily reprofuce with scripts provided, tomorrow I will add more instructions): all seems reasonably, only the tvd/ssp runge-kutta solvers are not good as I exepected they were. I have introduced some results into the manuscript draft.
https://github.com/Fortran-FOSS-Programmers/FOODiE/blob/master/papers/announcement/errors_analysis/errors_analysis.sh
https://github.com/Fortran-FOSS-Programmers/FOODiE/tree/master/papers/announcement/errors_analysis/utilities-oscillation
https://github.com/Fortran-FOSS-Programmers/FOODiE/blob/master/papers/announcement/manuscript.tex#L215


Bad link in Challenge 80 page

In challenge 80, I found a legacy link in the videos tab that points to /learning/programming-with-text.

Is this a missing track? Do we have a replacement URL or should this be removed?

The programming with text track is not ported to the website, so the link should point to the YouTube playlist: https://www.youtube.com/playlist?list=PLRqwX-V7Uu6YrbSJBg32eTzUU50E2B8Ch

Perfect, thank you. I'll replace the link with the YouTube playlist in an upcoming PR with other content fixes.


Marketing Experiment - 13 July - Self Hosted vs Open Source

Experiment
Context
For the second marketing experiment, running a basic a/b test on Reddit Ads to see which performs best.
In the red corner we have the sauciest message in the northern hemisphere..."PostHog is Open Sourcccceeee!!"
And in the blue corner is a contender that needs no introduction (but will get one anyway)..."PostHog can be self-hosteddd!"
Which message will win out when focused on identical Reddit targeting? Tune in two weeks from today to discover who wins this heavyweight high-level messaging test!
We have a number of USPs that set us apart. Two of the biggest are self-hosting and open source. This experiment tests those messages using otherwise identical imagery and targeting. The test uses Reddit Ads, targeting users rather than search keywords.
Experiment
Goal
We are trying to see which of these two messages has the most impact with a developer audience. Open source or self-hosting.
Hypothesis
One of these messages should have greater impact than the other. My suspicion is that it will be self-hosting, as this originates from a user need. However, open source also evokes strong opinions and there's only one way to be sure.
Update: At Kunal's suggestion, have added a third option merging both ideas.
Campaign details

Title: Analytics, session recordings, feature flags, heatmaps and more. All in a single product you can host yourself. vs Analytics, session recordings, feature flags, heatmaps and more. All in a single, open source product.
CTA: Learn more
Channel: Reddit Ads.
Destination: Posthog.com (no bespoke landing page)
Bidding: $5 per click (may increase as needed)
Audience
Devices: Desktop
Locations: Worldwide
Placements: In feed only
Interests: Programming, Computing, Programming, Business
Communities: productmanagement, prodmgmt, opensource, coding, programming, dataisbeautiful, analytics, dataanalytics, postgresql, aws
Resources and timeline
Like other ads in this series of experiments, the ads will run for two weeks with a £1000 budget each.
Metrics
Tracking
UTM parameters:
CampaignID = open-vs-selfhost
CampaignSource = redditads
CampaignMedium = cpc
CampaignTerm 1 = openandself
CampaignTerm 2 = opensource
CampaignTerm 3 = selfhost

Additional ad tracking through Reddit Ads. Will export a report at end of experiment.
Primary
CTR
Why
CTR tracks which message in the ad resonates best with the audience.
Pre-experiment baseline:
No baseline as these ads haven't run before. Once we have more ads running in this channel we can aim to formulate an average.
Secondary
We can potentially go further and look at conversion rate for each group, but we can only optimise conversion for each group once we know which group performs best.
Results
To be filled out later.
What are the top learnings from this experiment?
What impact was there on the primary metric or any secondary metrics?
What are the next steps based on the learnings of this experiment?

@kpthatsme Used the new template you provided. Thanks! Feedback is that it felt a little repetitive to fill out in a few places, but that may also be down to this being a simple test.

@joethreepwood – it looks like the "Tracking" section of the template was removed? Could we please get that section added back in with information on how we're tracking clicks?
Also – one idea: thoughts on running a third variant with both open source and self-hosted included? I feel like we should be able to combine both of those concepts into a single ad and I would love to see how that performs versus the individual inclusions.

@joethreepwood – it looks like the "Tracking" section of the template was removed? Could we please get that section added back in with information on how we're tracking clicks?

Added back in. I removed it because the answers were feeling a little repetitive (we're tracking CTR through Reddit Ads' platform directly) and wanted to cut it shorter. Let me know if there's extra info you need?

Also – one idea: thoughts on running a third variant with both open source and self-hosted included? I feel like we should be able to combine both of those concepts into a single ad and I would love to see how that performs versus the individual inclusions. Something like Open source product analytics that you can host in your own infrastructure.

This is a good idea and one I should have thought of. Will tweak the budgets and spin up a variant today, add to this thread.

New ad. Dropped budgets for each variant to $550, accounting for 20% overspend to equal original overall budget.

Added back in. I removed it because the answers were feeling a little repetitive (we're tracking CTR through Reddit Ads' platform directly) and wanted to cut it shorter. Let me know if there's extra info you need?

Thanks! think we're still missing information. re: repetition - the prompt for the this section was: "How are we tracking results of this experiment? Include any UTM parameters or other relevant details." – I don't think that's actually covered elsewhere in the spec?
This section is meant to capture specifically, how are we tracking traffic to this specific campaign. If we wanted to look at pageviews in PostHog that originate from this campaign, how can we find that?
Typically, for each campaign we run, I'd expect us to be setting some UTM terms on the links and capturing those terms in these specs.

Ah, understood. With this campaign I checked against Charles' campaign before the template was created and didn't set up any third party tracking or UTMs. We can add this in still if needed, but intent was to export a report directly from Reddit Ads once the campaign had finished.

Wouldn't be a blocker, but related question: do we have any standards around UTMs currently?

No, we don't have any standards (yet!) – I think Charles missed them by mistake.
We should definitely get some created as part of the work this week so we can tie this back into product data.
As a suggestion, what do you think about these?
utm_source=reddit
utm_medium=cpc
utm_campaign=open-vs-selfhost
utm_term= either opensource, selfhost, openandhost - based on the specific AD

I'm not great at naming so feel free to suggest others :) I think the most important things here are the source and medium. Those will let us measure $$. The others are important to help us measure longer-term impact of specific campaigns and variants.

Thanks Kunal -- I've set some UTM parameters and added above.

And the results are in!
Summary Results: The three campaigns reached a total 194,446 impressions, with 128 clicks at an average $2.40 CPC. The average CTR was 0.066% and the amount spent of a $2000 max budget was $307.85.
Results: The results are a little skewed because one option (the combined message) went live a few days later. The difference in performance for all three was only slight. OS option had the highest number of impressions, the highest number of clicks and the lowest CPC, however.
Full performance report:
t2_d9a2w6mg___2021-07-19_2021-07-25 (1).csv
Learnings:

There isn't much difference between self-host and open source options. The best approach is probably to combine the messages going forward, though if a choice is forced then open-source seems to get more attention initially.
We can afford to bid very aggressively. Each campaign was set at a max bid of $5 per click, but achieved approximately half of this. In two weeks, none of the campaigns hit their budget limits. Setting a higher big limit will likely create more, faster results.
Targeting developers and data analysts on Reddit is hard. For this campaign I selected the following broad categories: Business, Computing, Programming. And the following specific communities: Coding, Analytics, ProductManagement, Programming, AWS, Opensource, Dataanalytics, ProdMgmt, PostgreSQL, DataIsBeautiful and was only able to reach a 7.6-9.5m audience size.

Recommendations
If we're going to continue experimenting on Reddit, it would be worth compiling a bulk community list of subreddits which we believe would be worthwhile to target.

We can afford to bid very aggressively.

I found this as well - I think for the next experiment I'll try doubling to $10. I think it's worth running the same experiment again maybe, as $307 is probably too low to learn from definitively?

If we're going to continue experimenting on Reddit, it would be worth compiling a bulk community list of subreddits which we believe would be worthwhile to target.

Yeah, we'll need to keep adding to this list, this is a good start though!

@piemets @kpthatsme
Went ahead and used a few community research tools I've tried before to throw together a list of 70+ subreddits which we can use for the next Reddit test. Looked into it and we can just bulk upload Column A from here when creating ads. There's arguably a lot more subreddits we can add, but beyond these the tools I've used suggest either we'll hit dilution or diminishing returns.

Great summary @joethreepwood! I think I agree with your takeaway, assuming all else relatively the same it's probably best to float the version w/ both phrases.
One question re: the Ad copy, we aren't really highlighting "open source" and "self-host" as keywords in the joint ad. Only the "host in your own infrastructure" part really jumps out at me. Could be worth a treatment where all key words are highlighter a special color?
In general I'm eager to see how these experiments net out with some more money and compare against other channels – specifically those CTRs. I think Reddit can provide some niche communities but I feel most are not the type to 1) browse without an adblocker 2) interact with an ad whilst using reddit

Yeah, highlighting the copy with different colours or something was something I considered, but as we weren't doing that with the other ads I felt we'd end up biasing the results in the other direction.
My understanding is that Marketing will be leading further experiments for now. @piemets do you have plans for what sort of experiments are coming up next? It could be interesting in the coming weeks to start thinking about a landing page strategy based on results.

Went ahead and used a few community research tools I've tried before to throw together a list of 70+ subreddits which we can use for the next Reddit test. Looked into it and we can just bulk upload Column A from here when creating ads. There's arguably a lot more subreddits we can add, but beyond these the tools I've used suggest either we'll hit dilution or diminishing returns.

Thanks! I've added a link to the Paid Ads page in the Handbook (and also updated the page generally a bit)

My understanding is that Marketing will be leading further experiments for now. @piemets do you have plans for what sort of experiments are coming up next? It could be interesting in the coming weeks to start thinking about a landing page strategy based on results.

Yep that's right - cadence is that we launch an experiment every Tuesday, and then report back on the previous experiment every Thursday. I'll tag you both in the next issue I create - I want to do a Twitter ad this week as we haven't done one yet.


Fix issue with multiselect and submit on blur

Fix issue with multiselect and submit on blur
Closes #168

@ckosloski thanks for the contribution


Form if minMatchingChars == 0 show all data

Pull Request Test Coverage Report for Build 81

2 of 3 (66.67%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.8%) to 68.627%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/VueBootstrapTypeaheadList.vue
2
3
66.67%

Files with Coverage Reduction
New Missed Lines
%

src/components/VueBootstrapTypeaheadList.vue
1
84.21%

Totals

Change from base Build 38:
-0.8%

Covered Lines:
48

Relevant Lines:
70

💛  - Coveralls

I don't think I'm understanding this request fully. When I pulled the code down it didn't do what I expected. Can you please explain the use case in a little more detail?

I merged in functionality that I think resolves this issue. Check out the new version under it's new name:  https://www.npmjs.com/package/vue-typeahead-bootstrap
Let me know if you have issues here: https://github.com/mattzollinhofer/vue-typeahead-bootstrap/issues


Topological Sort

Is your feature request related to a problem? Please describe.
Topological Sort is a graph algorithm
Describe the solution you'd like
I would like to add an algorithm for topological sort to this repo in C++ language.
Kindly assign me this task

Assigned!
@Piyush-onGIT : C++

Assign it to me in Java @Kumar-laxmi

I can code it using C language. Assign this to me.

Please assign this issue in Python to me.


Inaccurate details on using the Automation Job Operator and Automation Runbook Operator roles in combination

A principal with the "Automation Job Operator" role can start any runbook (create or write a new job for any runbook) through the API, even without being given the "Automation Runbook Operator" on specific runbooks. That is, it can start a runbook just by knowing its name and without having read permissions on it.
This is in contrary to what stated in the #automation-runbook-operator section: "This role combined with the Automation Job Operator role enables the operator to also create and manage jobs for the runbook."

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

ID: 0db74982-1ecc-bfed-5cd1-e68450963636
Version Independent ID: 75190263-b53d-6346-b3ad-0c0c2715bc5b
Content: Manage role permissions and security in Azure Automation
Content Source: articles/automation/automation-role-based-access-control.md
Service: automation
Sub-service: shared-capabilities
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

Thanks for the feedback! I have assigned the issue to the doc author to update appropriately.

@oribesser & @SamaraSoucy-MSFT - Thanks for bringing this to my attention.  I've created a work item to correct the article and review the other details to ensure it is correct.  #please-close


The repo needs clean

Hi,
This is a valuable effort but many of the papers in the repo have nothing to do with medical. So either the title should change or many papers should be removed. Below is just one example:
https://arxiv.org/pdf/2302.00268.pdf

Thank you for your advice. I will remove some papers that have nothing to do with medical :)


Port all tests for PlainOption to avajs

Depend on https://github.com/karen-irc/option-t/pull/310

This change is

bors r+


Issue with running main.py

Hey,
I have been running some back testing and was ready to test a few strategies but am having some problems using the main.py. When I try to execute it I get:
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib64/python2.7/threading.py", line 811, in __bootstrap_inner
self.run()
File "/usr/lib64/python2.7/threading.py", line 764, in run
self.__target(*self.__args, **self.__kwargs)
File "/home/andy/Oanada/oandapybot/exchange/oanda.py", line 44, in _start
self.start(accountId=self._account_id,instruments=self._instrument,ignore_heartbeat=False)
File "/home/andy/Oanada/oandapybot/exchange/oandapy.py", line 315, in start
data = json.loads(line.decode("utf-8"))
File "/usr/lib64/python2.7/json/init.py", line 338, in loads
return _default_decoder.decode(s)
File "/usr/lib64/python2.7/json/decoder.py", line 365, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib64/python2.7/json/decoder.py", line 381, in raw_decode
obj, end = self.scan_once(s, idx)
ValueError: Expecting object: line 1 column 1 (char 0)
Which I guess means its not getting any data back from OANDA.  I am using a practice account and have set my AccountID, Access Token and Environment.
Any help would be greatly appreciated.

I was trying to a use a v20 API account, requested a legacy account, updated the API code and its working perfectly.

Good it works now!
Let me know if you want to do backtesting - I can share some historical data with you

Hi Peter,
I have been downloading data from HistData and been running some testing. Looking at the code it was using OHLC's and I have been able to test combinations of 1 to 100, on a 1 minute candle chart in SMA for EURUSD for last month.
I have nearly finished back testing the strongest pairs for the last year.   I have started trading in a practice account now with the updated strategy and hoping to do more back testing over the weekend.
Many thanks for coming back to me,
Cheers,
Andy


Add checks for namespace access operators (:: and :::) in cleanClosure.

Thanks @hlin09 for looking into this. Can we also add a test case where ::: and :: are used ?

Thanks for the test case and thanks @elbamos for helping us find this bug.
Merging this


v1.8 backports 2020-07-28

#12640 -- backporting: Report progress in set-labels.py (@pchaigno)
#12626 -- .travis:fix up TestSpanStatRaceCondition failure (@Jianlin-lv)
#12632 -- hubble/relay: split server code into a relay/observer and relay/server (@Rolinh)
#12654 -- hubble/relay: fix report of unavailable nodes (@Rolinh)
#12651 -- bpf: change tcp slot selection to prandom instead of cookie (@borkmann)
#12652 -- api: add missing audit mode enum values for EndpointPolicyEnabled (@tklauser)
#12659 -- doc: Clarify post-release steps (@christarazi)
#12521 -- health: Differentiate between unknown and unreachable state (@mrostecki)
#12683 -- Use the global Azure key (@gravis)

Skipping due to conflicts; please take a look @Rolinh

#12655 -- hubble/relay: report nodes connectivity status on ServerStatus (@Rolinh)
#12685 -- hubble/relay: fix unavailable nodes count on ServerStatus (@Rolinh)

Once this PR is merged, you can update the PR labels via:
$ for pr in 12640 12626 12632 12654 12651 12652 12659 12521 12683; do contrib/backporting/set-labels.py $pr done 1.8; done

test-backport-1.8

test-backport-1.8

@christarazi I resolved the conflicts for #12655 and #12685 and updated the PR.


[api-docs] add fields limit (#6585)

A new commit was made into the api-docs repo: https://github.com/discord/discord-api-docs/commit/b86c0295c77ac97e46457684be3d8b20fa344799
This is a bot created issue.

@MatthewSH the EmbedBuilder already has the check in the validation, right? if so this could be closed, i don't think we have any other place that needs an edit

yeah it does: https://github.com/discordeno/discordeno/blob/5a12def591931d6edfceae7179181e779e2f255e/packages/utils/src/builders/embeds.ts#L316
No other changes needed.


Please add Houston, Texas

Mayor Sylvester Turner<EMAIL_ADDRESS>Amy Peck<EMAIL_ADDRESS>Jerry Davis<EMAIL_ADDRESS>Abbie Kamin<EMAIL_ADDRESS>Carolyn Evans-Shabazz<EMAIL_ADDRESS>Dave Martin<EMAIL_ADDRESS>Tiffany Thomas<EMAIL_ADDRESS>Greg Travis<EMAIL_ADDRESS>Karla Cisneros<EMAIL_ADDRESS>Robert Gallegos<EMAIL_ADDRESS>Edward Pollard<EMAIL_ADDRESS>Martha Castex-Tatum<EMAIL_ADDRESS>Mike Knox<EMAIL_ADDRESS>David Robinson<EMAIL_ADDRESS>Michael Kubosh<EMAIL_ADDRESS>Letitia Plummer<EMAIL_ADDRESS>Sallie Alcorn<EMAIL_ADDRESS>"My name is _____. I am a constituent of District _____. I am writing in support Councilmember Letitia Plummer's police reform policies and to decrease the next annual budget for HPD. Thank you, __________"
Letitia Plummer At-Large Budget Amendments 200604.pdf

working on this now

I can take a look at the differences and add the language if that works. I think there is at least one missing official


access: github tells me I can merge on golang/tools bypassing gerrit

Here is my view of https://github.com/golang/tools/pull/551

Here is how I think it should look taken from https://github.com/golang/go/pull/70983

I guess if I pressed the button it would only corrupt the mirror and gopherbot would :crossed_fingers: force push over me or loudly error somewhere.

cc @dmitshur

I've checked all the repos under the golang github org that are marked with [mirror] description prefix.
Here is all the incorrectly configured ones I could find:

https://github.com/golang/tools
https://github.com/golang/sys
https://github.com/golang/crypto

I'm sorry I had to find this a Dec 25th.

See #59048


[Question]: trustedSigning Identity validation public vs private

Question
Hi,
In the release 6.1 there is mentioned that we have the possibility to add trustedSigning linked to the Azure Trusted Signing.
On the Directions in Vienna, there was mentioned that we should setup a new public Identity validation.

Is it possible to have a private identity for this setting as well?
I ask this because in the Microsoft Learn documentation it is mentioned the Trusted Signing at this time can onboard only legal business entities that have verifiable tax history of three or more years.
Since I only started at the beginning of this year with my business, this will not be possible for me to create a public identity validation.

I don't know what a private identity means - it is not requirements we (Business Central Team) sets.
You can try to reach out to azure support to see what you can do.
In order to do proper codesigning - we do need a trusted certificate (not self-signed or test)

I think you should go for public (I did)
https://learn.microsoft.com/en-us/azure/trusted-signing/concept-trusted-signing-resources-roles#certificate-profiles

Hi,
According the Microsoft Learn documentation , I should have verifiable tax history of three or more years.
Unfortunately, 3 years is not something I can magically create.

Certificate profile was approved... so the remark have verifiable tax history of three or more years is not that strict.


Unclear error: "Inconsistency: all regressors must have the same type of HRF (3 and 2 found)"

In a first attempt to explore nidmfsl I got this error:
% nidmfsl 1stlvl_glm.feat
Exporting NIDM results from /tmp/tmp.U5HOVWLTMP/localizerdemo/1stlvl_glm.feat
Traceback (most recent call last):
  File "/home/mih/env/datalad3-dev/bin/nidmfsl", line 6, in <module>
    exec(compile(open(__file__).read(), __file__, 'exec'))
  File "/home/mih/hacking/nidm/nidmresults-fsl/bin/nidmfsl", line 46, in <module>
    fslnidm.parse()
  File "/home/mih/hacking/nidm/nidmresults-fsl/nidmfsl/fsl_exporter/fsl_exporter.py", line 143, in parse
    super(FSLtoNIDMExporter, self).parse()
  File "/home/mih/hacking/nidm/nidmresults/nidmresults/exporter.py", line 97, in parse
    self.model_fittings = self._find_model_fitting()
  File "/home/mih/hacking/nidm/nidmresults-fsl/nidmfsl/fsl_exporter/fsl_exporter.py", line 185, in _find_model_fitting
    design_matrix = self._get_design_matrix(analysis_dir)
  File "/home/mih/hacking/nidm/nidmresults-fsl/nidmfsl/fsl_exporter/fsl_exporter.py", line 655, in _get_design_matrix
    ' found)')
Exception: Inconsistency: all regressors must have the same type of HRF (3 and 2 found)

It seems that this is a limitation of nidmfsl, and not an error in the analysis, but the exception would allow for both interpretations.
Is it not possible to represent different HRF models in NIDM results, or is this merely a temporary "didn't get to it yet" issue?

Thanks for letting us know Michael!  BTW, what is the use case?  I can’t think of anytime that I’ve mixed HRF types, and won’t normally recommend it to anyone.
@cmaumet: is this single-HRF restriction an implementational convenience or part of the NIDM Spec?

There is no real use case. I just clicked together a toy analysis for: https://github.com/ReproNim/ohbm2018-training/pull/3 -- but this particular choice was unintentional. The underlying use case is to build a metadata extractor for datalad (https://github.com/datalad/datalad-neuroimaging). We have started working on capturing as much of NIDM as possible, but I completely missed that this library for FSL existed (@yarikoptic pointed it out to me).  And for this effort we would naturally aim for something that is as robust as possible given the to-be-expected variability of configurations in the wild. In other words, we cannot really expect sanity as a precondition ;-)

I think mixing HRFs in one model is an edge case if not bad practice.  I'd change your design.fsf to have all fmri(convolveX) values to be the same.
Looking at the spec, HRF is an instance of Convolution Basis Set, which is an attribute of a Design Matrix.  (Excuse me if I've mangled the Semantic Web semantics). So I'm afraid that we don't currently allow a column-specific HRF.
There are a constellation of different analysis options and in building NIDM Results we tried to capture the typical ones, but if you think it's important use case, create an [ENH] issue on https://github.com/incf-nidash/nidm-specs/issues

done: https://github.com/incf-nidash/nidm-specs/issues/463

@mih, @yarikoptic: thank you for your feedback and thank you @nicholst for your answers! Let's see how we can update NIDM-Results to include this usecase!


Add an option to disable SSL verification

The --no-verify-ssl option was added to allow skipping SSL verification when POSTing to GitLab. It adds a VERIFY_SSL option to the config object, which defaults to True.
This is useful when setting up the integration inside of SSL-enabled intranets and other similar internal infrastructure where self-signed certificates are likely to be in use.

Coverage increased (+0.02%) to 89.951% when pulling 6628ab717009128807add1af262c91120ac2dc73 on eabruzzese:master into 4454bd45cf78a82fd0c58b7a06bcb41a42ae2be9 on NotSqrt:master.

Thanks !

how to add this option into a Gitlab install ?
the default gitlab mattermost integration doesnt offer the option to disable SSL verif in the UI


net: arp: Zephyr does not use MAC address from unicast ARP reply with wrong target MAC

If Zephyr receives unrequested unicast ARP reply with wrong target MAC it does not update translation table. So, instead of using destination MAC address from request packet (ICMP, TCP, UDP) in response, it should use MAC from unrequested ARP reply.
According to RFC 5227:

An incorrect ARP implementation may ignore ARP Replies where 'ar$tha' doesn't match its hardware
address.

unreq-arp-reply-wrong-tgt-mac.pcap.gz

So Zephyr drops unrequested unicast ARP reply and that's a bad thing? Ok, sounds weird.
Could you explain "So, instead of using destination MAC address from request packet (ICMP, TCP, UDP) in response, it should use MAC from unrequested ARP reply."
On arp request we don't know the MAC address, so there is none to use at this point (thus the ARP request). Do you mean we should blindly update the table from the info read on the unrequested ARP reply and use such info for pending packets? (ones that generated an ARP request on our side)

Do you mean we should blindly update the table from the info read on the unrequested ARP reply and use such info for pending packets? (ones that generated an ARP request on our side)

Yes, that is what I mean.


-Status parameter does not filter on status

API documentation of Snipe IT states the following:

Optionally restrict asset results to one of these status types: RTD, Deployed, Undeployable, Deleted, Archived, Requestable

When I use the -Status parameter to filter "Archived", it does not work. I just get the results as I would get without any filters. When I do an API call without the PS wrapper (?status=Archived), it does work as intended, proving the issue does not lie in the instance.
Using the verbose data, I see this being called:
VERBOSE: [ConvertTo-GetParameter] Making HTTP get parameter string out of a hashtable
VERBOSE: [Invoke-SnipeitMethod] Invoking method Get to URI http://server/api/v1/hardware?Invocation=System.Management.Automation.InvocationInfo&BoundParameters=System.Management.Automation.PSBoundParametersDictionary&DefaultExcludeParameter=id+url+apiKey+Debug+Verbose
VERBOSE: GET http://server/api/v1/hardware?Invocation=System.Management.Automation.InvocationInfo&BoundParameters=System.Management.Automation.PSBoundParametersDictionary&DefaultExcludeParameter=id+url+apiKey+Debug+Verbose with 0-byte payload

Using Powershell Core 7.1.0 on MacOS.

Actually, I saw issue 67 (https://github.com/snazy2000/SnipeitPS/issues/67#issuecomment-769958633) and @PetriAsi seems to have fixed it. @snazy2000, hoping you can merge his pull request.

fix  is  pulled to develop branch


Consider upgrading to Jackson 2.13.0 or later

Due security analysis reported on users' channel

For Helidon 2, we can consider 2.12.6 (Dec 2021) if 2.13.X has compatibility risks.
For Helidon 3, Jackson 2.13.X has some new Jakarta variants -- need to determine if/how those impact us.

In 2.x we have upgrade to 2.12.6. In 3.x we have upgraded to 2.13.1.


Add git lfs link or description to "Adding a file to a repository" page

Code of Conduct

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

What article on docs.github.com is affected?
https://docs.github.com/en/repositories/working-with-files/managing-files/adding-a-file-to-a-repository
What part(s) of the article would you like to see updated?
Adding a link to the top of the page that takes the user to this page: https://docs.github.com/en/repositories/working-with-files/managing-large-files/about-large-files-on-github
This would be helpful because it provides easier access to the git LFS documentation in case users want to upload files greater than 100 MB in size.
Right now the "managing large files" section is a tab on the left sidebar, but it may not be intuitive for people to look their if they are wondering how to add files greater than 100mb to Github
Additional information

@vedantsinghania05 Thanks so much for opening an issue and providing great detail! I'll triage this for the team to take a look :eyes:

Thank you!

Hi @vedantsinghania05, thanks for opening this issue! I agree that it would be helpful to include a link to the "About large files on GitHub" article.
You or anyone else is welcome to open a PR to add the following text at the end of the first paragraph in "Adding a file to a repository" (in the section "Adding a file to a repository on GitHub"):

To add files larger than 100 MB, you must use Git Large File Storage. For more information, see “About large files on GitHub.”

So the full paragraph would read:

Files that you add to a repository via a browser are limited to 25 MB per file. You can add larger files, up to 100 MB each, via the command line. For more information, see "Adding a file to a repository using the command line." To add files larger than 100 MB, you must use Git Large File Storage. For more information, see “About large files on GitHub.”

💡 Tip for whoever opens this PR: You will need to use variables.


OpenXML-Dll defines System.Collections.Generic.IReadonlyList in conflict with mscorlib

Description
System.Collections.IReadonlyList added in #400 conflicts with mscorlib
Information

..NET Framework 4.5.2
DocumentFormat.OpenXml Version: 2.8.1

Repro
Observed

Upload DLL as part of web application using .NET 4.0, dynamically loading this DLL.
Get exception when accessing service:

Unhandled Exception: Aras.Tools.SolutionUpgrade.CItemException: Failed login to Innovator; original error: Line number -35, Error Number: CS1685, Warning as Error: The predefined type 'System.Collections.Generic.IReadOnlyList' is defined in multiple assemblies in the global alias; using definition from 'c:\Windows\Microsoft.NET\Framework64\v4.0.30319\mscorlib.dll'
Line number -35, Error Number: CS1685, Warning as Error: The predefined type 'System.Collections.Generic.IReadOnlyCollection' is defined in multiple assemblies in the global alias; using definition from 'c:\Windows\Microsoft.NET\Framework64\v4.0.30319\mscorlib.dll'

Verified it exists in GitRepo and via Reflection.

Expected
System.Collections.Generic.IReadOnlyList should not be defined in this assembly under the same name

Backing to 2.7.2 fixed the issue, btw.

@marcusl  Can you include a repro? This type is internal and I'm surprised it's showing up anywhere. I'll switch the namespace to something internal regardless, but I'm interested in what's causing this

Not easily, as it showed up when I included the DLL on some customizations on the Aras Innovator PLM web application, so nothing small and local. Easy to repro for me, not easy to export for someone else.
Maybe the dynamic assembly loading done by the PLM web app circumvents the internal modifier somehow.  I'm not sure, as that is a bit above my current .NET competence level, but I can investigate and see if it is possible.

Ok. If you find anything, let me know. I've submitted a fix for this, and it'll be available in the CI shortly after building. We probably won't have a stable 2.9.0 release until after .NET Core 2.1 comes out (there's a fix in System.IO.Packaging that we want to incorporate), so you can stay on 2.7.2, use the prerelease version, or we can create a 2.8.2 release with this fix. Since you're targeting .NET Framework 4.0, you shouldn't be missing too much staying on 2.7.2.

Will do. 2.7.2 is fine for quite some time.

I'm not familiar with the Aras Innovator PLM web application, but it may be good to see if you can log an issue there that they are trying to load internal types.

I'll do that!
ons 7 mars 2018 kl 22:39 skrev Taylor Southwick<EMAIL_ADDRESS>
I'm not familiar with the Aras Innovator PLM web application, but it may
be good to see if you can log an issue there that they are trying to load
internal types.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OfficeDev/Open-XML-SDK/issues/426#issuecomment-371294454,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACp4PWEBGiScmXDDzZjGCALIsdNq2mDks5tcFOogaJpZM4SgBhJ
.

--
/Marcus

Just following up that a better place for a link is probably their community forum, as the official (paid) support does not have an open issue tracker, AFAIK.
http://community.aras.com/en/forums/topic/dlls-in-method-config-are-loaded-with-internals/

I just updated to OpenXML.DLL version 2.8.1 and am getting this warning. I was using version 2.5 previously and made no other dependency changes, so I'm operating under the assumption that the OpenXML.DLL upgrade is where my issue started. I see this is closed, is there somewhere I can go for more information?

The fix is in, but hasn't been released yet. We're in the process of doing so. Are you also using the aras tools?

Just to clarify, this is a bug with Aras (or whatever you are using to host the app) as it is incorrectly not honoring visibility rules. You can use the latest CI build of 2.9.0 stable (we haven't pushed it to NuGet but it is supported as a stable release) which has this fix, but also please contact the developers of the hosting framework to notify them they have a potentially serious bug. Not  honoring visibility should be fixed on their ends ASAP.


Mirage: You're trying to create a order model and you passed in "8" under the payMethod key, but that key is a BelongsTo relationship. You must pass in a Model or null.

#question
I've updated to 0.4.9 from 0.1.0
I'm getting an error:
Mirage: You're trying to create a order model and you passed in "8" under the payMethod key, but that key is a BelongsTo relationship. You must pass in a Model or null.
fixtures/orders.js
  {
    id: 'MTYUT47',
    payMethod: 8,
    status: 'pending',
    type: 'new'
  },

fixtures/payment-methods.js
  {
    id: 8,
    name: 'DKK Bank Transfer',
    namespace: 'dkbank',
    days: 1,
  },

config.js
  this.get('/orders/:id', (schema, request) => {
    let order = schema.orders.find(request.params.id);
    return order;
  });

app/models/order.js
var Order = DS.Model.extend({
  payMethod: DS.belongsTo('paymentMethod', { async: true }),
  status: DS.attr('string'),
  type: DS.attr('string'),
});

app/models/payment-method.js
var PaymentMethod = DS.Model.extend({
  name: DS.attr('string'),
  namespace: DS.attr('string'),
  days: DS.attr('number'),
});

after update, I'm getting an error:
Mirage: You're trying to create a order model and you passed in "8" under the payMethod key, but that key is a BelongsTo relationship. You must pass in a Model or null.
If I update my fixtures to payMethodId instead of payMethod:
fixtures/orders.js
  {
    id: 'MTYUT47',
    payMethodId: 8,
    status: 'pending',
    type: 'new'
  },

Then I get this warning:
warn.js:45 WARNING: Encountered "data" in payload, but no model was found for model name "datum" (resolved model name using DS.RESTSerializer.modelNameFromPayloadKey("data"))
I guess I'm missing something, can't find what exactly..

@samselikoff Thank you very much!!
I had JSONAPISerializer, changed to RestSerializer.
now seems like works with new payMethodId


[CARBONDATA-3298]Remove Log Message for Already Deleted Segments in old stores

Problem:
In old store, Create table and perform one Insert operation. Now, update and delete that record, which marks that segment as  "MARKED FOR DELETE". Now run "clean files command" to delete the segment.
Note: In this case, Metadata folder doesn't  contain segment file.
In new store, Refresh the table and again perform IUD operation. Now, when "clean files" command is executed,  We will check if physically segment file exists or not, if not present, then we will log a warning message, as file not present. If old store contains more segments, then for each segment, this log message will be getting printed, which is not required.
Solution:
Removed log message for already deleted segments

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[x] Testing done
Testing done manually

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2587/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10847/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2818/

In new store, do we need this log ?

@qiuchenjian In new store also, this log is not required.

@Indhumathi27
Sorry that I didn't understand the problem clearly. Do you mean that 'clean files' did clean the files but didn't clean the segment folder, so in the new store, carbondata will complain that the are no files in that segment folder?
If it is so, I'd suggest that we should delete the segment folder in the old store, this will solve the problem at the root.

@xuchuanyin No,  Clean files will delete the segment folder and files. In new store, when  clean files command is executed, we will take LoadMetaDetails from TableStatus file and delete Segment folder if Physically exist or if Segment file in MetaData folder exist. In this case, LoadMetaDetails will contain Details for already deleted segments in old store, where Segment folder physicaly does not exist, hence we will log the message for those Segments.

@Indhumathi27 So the root cause is that while marking the segment as ‘marked as deleted’, the tablestatus file does not get updated, right?
I‘m wondering whether 'delete segment' command update the tablestatus file?

@xuchuanyin we cannot decide on SegmentStatus "MARKED_FOR_DELETE" to check if physically segment folder exist or not. Both already deleted segments and segments ready for clean using "clean files" will be marked as "MARKED_FOR_DELETE" only. And i dont think this log is needed, if any error occurs during clean files, then there are other logs to display the error message

LGTM


'comb' from 'scipy.misc' is deprecated

If I want to import suftware in python, I get the following import error:
"cannot import name 'comb' from 'scipy.misc'"
...apparently, 'comb' is deprecated (see also documentation of scipy)
Is there a way how I can still use suftware, altough I'm using the newer scipy version?

Hello ykarrer,
Thanks for bringing this to our attention. An updated version of suftware will be deployed next week to fix this comb issue.
In the meantime, here are some alternatives that should temporarily get suftware working until the new release:

use 'virtualenv'; this python package will let you use suftware with an older version of comb without affecting your system installation of scipy.

you can use suftware on google colab with python 2.7. Since google colab packages are hosted online, you can install any package you want (e.g. an older version of scipy to get suftware running) and it should not affect you system's python.

Suftware's behavior is the same for python 2.7 and 3.6. As a temporary solution, you could use suftware with python 2 (if previously you were use python 3) with an older version of scipy?

Thanks again,
Ammar

Perfect, thanks a lot!!

Hey @ykarrer, I just released suftware 0.15 which contains the fix for this comb issue. Please update your suftware via pip and let me know if you still have the issue.
Thanks!
-Ammar

Works perfectly fine now. Thanks!


V1.0 patch

Pull fixes from master into v1.0 branch

Codecov Report

Merging #296 into v1.0 will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             v1.0     #296      +/-   ##
==========================================
+ Coverage   52.95%   52.96%   +0.01%
==========================================
  Files          40       40
  Lines        4406     4407       +1
==========================================
+ Hits         2333     2334       +1
  Misses       1878     1878
  Partials      195      195

Impacted Files
Coverage Δ

enforcer/datapath_tcp.go
62.8% <100%> (ø)
:arrow_up:

supervisor/iptablesctrl/acls.go
69.69% <100%> (+0.07%)
:arrow_up:

Continue to review full report at Codecov.

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


Make sending of master state to nodes be asynchronous

ApplyClusterStateFromMaster seems to copy the state from the master to the target node synchronously, guaranteeing that the target always receives the current state. It strikes me it might be more realistic to represent this as an asynchronous message, allowing for the received state to be somewhat stale, which is what I'm trying to do with this change.
This leads to a violation of the GlobalCheckPointBelowLocalCheckPoints invariant.
(It's quite likely there's a good reason that how it was originally modelled is correct - I'm just exploring the spec a bit)

Thanks for the PR, @DaveCTurner. The reason that the GlobalCheckPointBelowLocalCheckPoints  invariant is violated is because the change here allows cluster state updates to be applied out-of-order. The trace that leads to the violation is as follows:

Document is indexed into primary on node1 and replication messages sent to node2 and node3
Replicated document is indexed into replica on node2, and replication response sent back to node1
Master sends initial cluster state CS1 (primary on node1, and replicas on node2 and node3) to node1
Fault detection on master kicks node3 out, updates cluster state on master removing replica on node3 (= CS2)
Master sends updated cluster state CS2 to node1
Node1 now applies the cluster state CS2 first.
Node1 then handles the replication response from node2. It can now increase the global checkpoint as it knows that the replica on node3 has been failed.
Node1 then applies the cluster state CS1. This means that node1 thinks again that the replica on node3 is not failed anymore. This step should not happen, as cluster state updates cannot be applied out-of-order (guaranteed by consensus algorithm, not modelled here).
Another document is indexed into primary on node1 and replication messages sent to node2 and node3, now containing the updated global checkpoint
Node3 handles the replication request that has as global checkpoint "1" but node3 has not indexed the previous document yet. This violates the condition.

Fixing this would require introducing an order on the cluster states (e.g. by attaching a version number to the cluster state, incremented with each change on the master), and adapting HandleReceiveMasterState not to apply older states.
Even with a fix, I'm a bit hesitant on integrating the change. What this change adds is the possibility for a node to apply an older cluster state while the master (and other nodes) are already on a newer cluster state. Note that the current spec already allows a node to be on an older state (just not apply an older state). The main reason I went with the current approach was to deal with state space explosion issues. I did not explore the specific solution here though, so I'm open to integrating this if it does not have too much of a negative impact on the model checker running times (or maybe we can extend the StateConstraint to somehow bound the impact).

Good, ok, this ties in with my understanding of what's going on in the model which is an encouraging sign.
I was particularly looking at this with a view to proving theorems about the model, for which state-space explosions shouldn't matter, but I appreciate that this might not be compatible with with using the model-checker. Feel free to close without merging if it makes things too much worse.

... indeed I wasn't expecting it to be merged at all, so didn't put much effort into trying to fit into your naming style etc.

Tidying up by closing some of my stale PRs. Sorry for the noise.


graphql-java-extended-validation ValidationSchemaDirective doesn't get registered

Hi, I am using  https://github.com/graphql-java/graphql-java-extended-validation library along with graphql-java-tools spring boot.
I tried registering SchemaWiring Directive in SchemaParser builder's  directiveWiring method but still it fails to pick that directive which has been provided in the library above.
Basically

ValidationRules validationRules = ValidationRules.newValidationRules()
                .onValidationErrorStrategy(OnValidationErrorStrategy.RETURN_NULL)
                .build();
        //
        // This will rewrite your data fetchers when rules apply to them so that validation
        ValidationSchemaWiring schemaWiring = new ValidationSchemaWiring(validationRules);

This schemaWiring is not getting added to directive in SchemaParser.
Can you help?

You're using the graphql-spring-boot-starter since you've registered the issue here. Instead of passing it in manually, can you expose a bean for it and see if that works. E.g.
@Bean
public ValidationSchemaWiring() schemaWiring(){
    ValidationRules validationRules = ValidationRules.newValidationRules()
                .onValidationErrorStrategy(OnValidationErrorStrategy.RETURN_NULL)
                .build();
    return new ValidationSchemaWiring(validationRules);
}

Tried the approach that you suggested. Unfortunate to say, it didn't work either.
Added screenshot of debug point. You can see that 3 directives get registered but there should be fourth one too which had an annotation "@NotEmpty" in SDL file.

As discussed in other issue this is supported starting with version 7.0.2-SNAPSHOT of graphql-spring-boot (although the next official release will be 7.1.0 btw).
However... This library is using graphql-java 14.0, while graphql-java-extended-validation is not yet compatible with it. That's why you'll run into problems trying to run it. You'll have to wait until they merged this PR and released a new version to provide support for graphql-java 14.0

Yeah till then i have used snapshot version and its working fine till now.
On Mon, 6 Apr, 2020, 12:25 AM Michiel Oliemans<EMAIL_ADDRESS>wrote:

As discussed in other issue this is supported starting with version
7.0.2-SNAPSHOT of graphql-spring-boot (although the next official release
will be 7.1.0 btw).
However... This library is using graphql-java 14.0, while
graphql-java-extended-validation is not yet compatible with it. That's why
you'll run into problems trying to run it. You'll have to wait until they merged
this PR
https://github.com/graphql-java/graphql-java-extended-validation/pull/11
and released a new version to provide support for graphql-java 14.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/graphql-java-kickstart/graphql-spring-boot/issues/381#issuecomment-609464501,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL37CYOMBTY4IPEWMRYIE43RLDHYFANCNFSM4L6Q6FXA
.

If you really want to get this working, you can do the following. Create the the bean as @oliemansm suggested,
@Bean
public ValidationSchemaWiring() schemaWiring(){
    ValidationRules validationRules = ValidationRules.newValidationRules()
                .onValidationErrorStrategy(OnValidationErrorStrategy.RETURN_NULL)
                .clearValidations()
                .addRule(new BlankValidationRule())
                .build();
    return new ValidationSchemaWiring(validationRules);
}

Then you'll need to define your own NotBlankRule as:
public class NotBlankRule extends graphql.validation.constraints.standard.NotBlankRule {
    protected boolean isOneOfTheseTypes(GraphQLInputType inputType, GraphQLScalarType... scalarTypes) {
        GraphQLInputType unwrappedType = Util.unwrapNonNull(inputType);
        for (GraphQLScalarType scalarType : scalarTypes) {
            if (scalarType.equals(unwrappedType)) {
                return true;
            }
        }
        return false;
    }
}

Though you may just want to wait for the validation library to be updated.

Thanks for throwing a light in right direction.
Actually the extended-validaiton library and java-tools library are so closely associated and people are using it as if they both are from the same source.
That's why it is becoming difficult to figure out if an issue is related to java-validation library or it has something to do with java-tools
Like 1 update from java-tools could make/break validation library.
Vice-versa


Webhook messages from another bot get weird username on matrix

[I'm coming from t2bot's help channel. They say they didn't modify anything related to this and are in sync with upstream's release version, so it's something with upstream - aka you.]
We have Pluralkit users in a bridged discord server. Pluralkit is kinda a proxying bot in itself - it takes discord messages with set prefix/suffix and writes them (still on discord) as set profile using webhooks.
Bridge sees those new messages and tries to send them to Matrix... And kinda fails with username.
Showing visual example. First message is Pluralkit, everything else is from Matrix bridged to Discord. I replied on Matrix to the weird username, so now you can see both correct and weirded out usernames on discord.

Middle of username (only small letters) gets written normally, so I censored it for user's privacy. First character (big A) and end (space, parentheses) get encoded. What is interesting is that emoji doesn't get encoded.
Pluralkit's webhook "user" as it shows on discord: Acensored (⭐)
Display name for the same message after bridging to Matrix: _discord_928017576183873567_=41censored=20=28⭐=29 (username is the same, but of course with @ and bot's domain)
92... is Pluralkit's discord id. But normally proxy users on matrix are @_discord_<userid>:<botdomain> (without anything else), so it's probably both how id and name of webhook "user" are supplied. Normal bot messages (not webhooks) show correctly.

For me it is a bit inconsistent: Some bridged names from bots get good names, some get bad names. Maybe it is related to #689. The name stays good/bad for subsequent messages from the same user.
Example of good name:

Example of bad name:

Maybe a workaround would be to kick the bridged bot with a wrong name and hope that the rename of the bot works in matrix next time.

Duplicate of #610 and #709.


fix(taos-ws): connect to taosadapter,retry wait time, max 30s

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 71.25%. Comparing base (249f194) to head (5aa3f21).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

taos-ws/src/lib.rs
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #358      +/-   ##
==========================================
- Coverage   71.25%   71.25%   -0.01%
==========================================
  Files          98       98
  Lines       32594    32595       +1
==========================================
- Hits        23225    23224       -1
- Misses       9369     9371       +2

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


"RedissonShutdownException: Redisson is shutdown" error during graceful shutdown (deployment)

Redis version
7.0.7
Redisson version
3.32.0
Redisson configuration
Default against AWS Elasticache, Cluster Mode disabled
What is the Expected behavior?
No exception
What is the Actual behavior?
During deployments of two-instance services using Redisson - when the old instances are shut down - we sometimes see different errors on the instance that either has the lock or doesn't have the lock. So far we are not able to reproduce it during deployments of single-instance services. Deployment includes graceful shutdown which includes a lock.unlock(). We do not see any exceptions related to a failing unlock call.
This sometimes happens on the instance that has the lock:
org.redisson.RedissonShutdownException: Redisson is shutdown
	at org.redisson.command.RedisExecutor.execute(RedisExecutor.java:128)
	at org.redisson.command.CommandAsyncService.async(CommandAsyncService.java:658)
	at org.redisson.command.CommandAsyncService.evalAsync(CommandAsyncService.java:601)
	at org.redisson.command.CommandAsyncService.evalWriteAsync(CommandAsyncService.java:449)
	at org.redisson.command.CommandAsyncService.syncedEval(CommandAsyncService.java:992)
	at org.redisson.RedissonBaseLock.evalWriteSyncedAsync(RedissonBaseLock.java:214)
	at org.redisson.RedissonBaseLock.renewExpirationAsync(RedissonBaseLock.java:184)
	at org.redisson.RedissonBaseLock$1.run(RedissonBaseLock.java:145)
	at io.netty.util.HashedWheelTimer$HashedWheelTimeout.run(HashedWheelTimer.java:713)
	at io.netty.util.concurrent.ImmediateExecutor.execute(ImmediateExecutor.java:34)
	at io.netty.util.HashedWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:701)
	at io.netty.util.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:788)
	at io.netty.util.HashedWheelTimer$Worker.run(HashedWheelTimer.java:501)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Thread.java:840)

This sometimes happens on the instance that does not have the lock:
org.redisson.RedissonShutdownException: Redisson is shutdown
	at org.redisson.command.RedisExecutor.execute(RedisExecutor.java:128)
	at org.redisson.command.CommandAsyncService.async(CommandAsyncService.java:658)
	at org.redisson.command.CommandAsyncService.evalAsync(CommandAsyncService.java:601)
	at org.redisson.command.CommandAsyncService.evalWriteAsync(CommandAsyncService.java:449)
	at org.redisson.command.CommandAsyncService.syncedEval(CommandAsyncService.java:992)
	at org.redisson.RedissonBaseLock.evalWriteSyncedAsync(RedissonBaseLock.java:214)
	at org.redisson.RedissonLock.tryLockInnerAsync(RedissonLock.java:219)
	at org.redisson.RedissonLock.tryAcquireAsync(RedissonLock.java:193)
	at org.redisson.RedissonLock.lambda$tryAcquireAsync0$0(RedissonLock.java:160)
	at org.redisson.connection.ServiceManager.execute(ServiceManager.java:514)
	at org.redisson.connection.ServiceManager.execute(ServiceManager.java:509)
	at org.redisson.RedissonLock.tryAcquireAsync0(RedissonLock.java:160)
	at org.redisson.RedissonLock.tryAcquire(RedissonLock.java:156)
	at org.redisson.RedissonLock.tryLock(RedissonLock.java:278)
	at org.redisson.RedissonLock.tryLock(RedissonLock.java:320)
	at org.factcast.factus.redis.AbstractRedisProjection.acquireWriteToken(AbstractRedisProjection.java:107)
        ...

FYI: this is the acquireWriteToken method mentioned in the stack trace. RedisWriterToken is an AutoCloseable which includes an unlock() call during close(). maxWait is set to 5 minutes.
  @Override
  public WriterToken acquireWriteToken(@NonNull Duration maxWait) {
    try {
      if (lock.tryLock(maxWait.toMillis(), TimeUnit.MILLISECONDS))
        return new RedisWriterToken(redisson, lock);
    } catch (InterruptedException e) {
      // assume lock unsuccessful
    }
    return null;
  }

Additional information
Had no such problems with <=3.27.1.
Started to see issues with 3.27.2 (with an NPE as cause). Seeing this issue's behavior (or very similar behavior) since 3.28.0 up until 3.32.0
With 3.32.0 errors seem to occur less frequent compared to 3.31.0.

this error may happen only when RedissonClient.shutdown method is called

Yes, this happens while the application (old instance during deployment) is shut down. IMHO the exceptions are unexpected because nobody can react to them anymore (instance shutting down).
This produces errors and triggers alarms and developers "become error-blind".

I currently have the same problem in prod with basically the same setup. Are there any updates regarding this issue?

Fixed. Thanks for report

Thank you @mrniko you are the best.


reporter-web-app: Revert changes to for-loops

These changes from 1d3f2d9 broke the report. A difference to using
forEach is that it does not enumerates properties in the prototype
chain, which probably caused the breakage.

This change is

Merging prematurely as the reporter-web-app is not covered by tests anyway yet.


Fix CUDA device tests (fix #989)

@johko, it's still your commit, but I got rid of the change to .gitignore and tweaked the message to make github happier... :-)

Codecov Report
Patch coverage has no change and project coverage change: -0.04% :warning:

Comparison is base (4a1946e) 78.41% compared to head (11a7d51) 78.38%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #993      +/-   ##
==========================================
- Coverage   78.41%   78.38%   -0.04%
==========================================
  Files         100      100
  Lines        6209     6209
==========================================
- Hits         4869     4867       -2
- Misses       1340     1342       +2

see 1 file with indirect coverage changes

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


Speed button

For some exercises in which the entity need to do lots of steps it would be usefull to speed up the simulation

A slider would be cool.


feat: vue 3 compatibility

Fixes #4
Changes proposed in this pull request:

@samuveth Nice! Can you make a new file vue3.ts and keep vue.ts original file?

@samuveth Nice! Can you make a new file vue3.ts and keep vue.ts original file?

@samuveth Nice! Can you make a new file vue3.ts and keep vue.ts original file?

Yes, that will be better

@samuveth Nice! Can you make a new file vue3.ts and keep vue.ts original file?

Yes, that will be better


Newline characters in cells of written workbooks

When values v of cells contain unix line endings \n they do not become line breaks in the corresponding cells of the written workbook. However, when the Windows carriage return newline combos \r\n are used Excel appears to properly break the lines when a cell containing them is double clicked (edited).

So there appear to be two things to potentially address here:

Unix line endings \n do not become new lines at all in the written file
Windows line endings \r\n only become newlines in Excel when the cell is edited. This may be a shortfall in Excel for Mac 2011 and may not apply to the Windows version.

Using booSST:true seemed to work for me when opening xlsx file on Linux and LibreOffice but didn't work on Excel on Win 10.

bookSST:true doesn't work for Excel on Mac (Office 365) either.

@tophsic @dferer can you share either a code sample or a bad file generated by the library?

@reviewher here is a code example.
It works fine with macOS numbers. I didn't test it with LibreOffice yet. But Excel 15.17 under macOS ignores the line breaks at first, but adds them as soon as you double click on the cell.

Did someone find a workaround? We the same issue on Excel macOS and Excel Windows -- text is shown in a single line and changes on double click.
Apple Numbers shows the file correctly.

I have the same problem. Did anyone find a workaround?

I could only get this working correctly using the Pro edition where the saving of cell styles is supported. It needed: \n in the string value, bookSST: true, cellStyles: true in XLSX.write, and cell.s = { alignment: { wrapText: true } } in the cell style.

To summarize: merely placing a newline character is insufficient.  Excel requires the text wrapping style attribute to be applied to the cell, otherwise it is rendered like a whitespace character.  This is a part of our Pro builds.
@fozcode I don't believe bookSST: true is required, if you have a bad case email us.

@SheetJSDev I've tested again and you are correct, bookSST: true is not required for Excel.
In LibreOffice a multi-line string is displayed on one line unless you set bookSST: true, which I guess is a LibreOffice bug. When you click into the cell the lines are then shown correctly.

it seems to not be correct tag in xlxs.js file :line 19974
.replace(/\n/g, "text:line-break/")
instead of
.replace(/\n/g, "</text:p>text:p")

@jefleponot Reviewing the spec http://docs.oasis-open.org/office/v1.2/os/OpenDocument-v1.2-os-part1.html
<table:table-cell> (9.1.4) has <text:p> (5.1.3) children
<text:line-break> (6.1.5) is a valid child of <text:p>, which was originally interpreted to be a newline character.
You're proposing we create separate paragraphs for each logical line, which is fine but I'm curious why the line break doesn't work

Could I propose a PR ? Could you accept it ?

https://bugs.documentfoundation.org/show_bug.cgi?id=43107

To summarize: merely placing a newline character is insufficient. Excel requires the text wrapping style attribute to be applied to the cell, otherwise it is rendered like a whitespace character. This is a part of our Pro builds.
@fozcode I don't believe bookSST: true is required, if you have a bad case email us.

@SheetJSDev is there a solution for the community edition?

a solution for the commu

same issue

Anybody ever found a solution for community edition? We can not access the pro edition because we have not received replies yet when we reach out...

Why is something as simple as a newline behind a paywall (Pro)?

Same issue

Add more VBA Code to resolve this
wsSrc.Cells.WrapText = True


Set argument types to fix issue #25

Explicitly set the argument and return types of the clipboard functions used in the module. This fixed the error described in issue #25 on my computer. The problem seems to be due to larger 64-bit pointers and handles used while running in 64 bits on 64-bit Windows, as opposed to the smaller size ints that ctypes uses by default.

I fixed it by similarly to this PR by argument types and return value types, but my fix is not important now more. I confirm that it solves the issue also on a Windows Server 2012 R2 with Python 3.5. All tests passed.
If you apply the fix, you will see that handles are really longer than 32 bits on x64 with Python 3.5 (even on a system with 1 GB memory - in VirtualBox) and that upper bits were stripped without fix. So because the error "6 - invalid handle" is missed in your app, you get a null pointer reference on the next line usually.

Thank you for this PR. I'm sorry I couldn't get to it in a timely manner. The fix for this issue is based off of this code, I believe. Even though this particular PR won't be pulled in, I'll add your name to the contributors list. Thank you very much!


How to improve the map result

The map that we produced looks like this, it shows that there are some issues with the alignment. The imu is mounted on top of the car,and the speed of the car is around 7-10 km/hr. How can we improve to get a better result?

Please provide configurations + bags. Otherwise we cannot help. Closing till necessary data is provided.

This is the lua file
include "map_builder.lua"

options = {
  map_builder = MAP_BUILDER,
  map_frame = "map",
  tracking_frame = "base_link",
  published_frame = "base_link",
  odom_frame = "odom",
  provide_odom_frame = true,
  use_odometry = false,
  use_laser_scan = false,
  use_multi_echo_laser_scan = false,
  num_point_clouds = 1,
  lookup_transform_timeout_sec = 0.2,
  submap_publish_period_sec = 0.3,
  pose_publish_period_sec = 5e-3,
}

TRAJECTORY_BUILDER_3D.scans_per_accumulation = 160 -- google default 160, 160 packets

MAP_BUILDER.use_trajectory_builder_3d = true
MAP_BUILDER.num_background_threads = 7
MAP_BUILDER.sparse_pose_graph.optimization_problem.huber_scale = 5e2
MAP_BUILDER.sparse_pose_graph.optimize_every_n_scans = 320 -- google default 320
MAP_BUILDER.sparse_pose_graph.constraint_builder.sampling_ratio = 0.03
MAP_BUILDER.sparse_pose_graph.optimization_problem.ceres_solver_options.max_num_iterations = 10
-- Reuse the coarser 3D voxel filter to speed up the computation of loop closure
-- constraints.
MAP_BUILDER.sparse_pose_graph.constraint_builder.adaptive_voxel_filter = TRAJECTORY_BUILDER_3D.high_resolution_adaptive_voxel_filter
MAP_BUILDER.sparse_pose_graph.constraint_builder.min_score = 0.62

return options

Please provide configurations + bags. Otherwise we cannot help.


Volume not working

Hi SeydX, all good? Thx for all the hard work! It's the absolute best plugin I'm using. Just wanna report that since the 4.0 my volume (from the remote control in the Control Center of my iPhone) is not working... Everything else working fine and the homebridge is sending the commands to the TV ([11/25/2020, 13:51:39] [BraviaOSPlatform] Sony 75: Reducing volume by -1). My TV is a Sony XBR75X855D. Let me know if you need me to send any other info. Thx!

Hey @fcoura
Have you tried changing the audio output in config?
You can try audio output "others". Maybe this will work

yes! perfect.


Social Recovery Wallet Challenge

Closes #23
Description
Creates the Social Recovery Wallet Challenge.
@escottalexander I think you'll need to create a social-recovery branch to merge this into
Some potential extra credit features

Ability to add/remove guardians
Hide identity of guardians
Add expected wallet functionality (send/receive tokens, call other contracts, etc.)

@escottalexander I think you'll need to create a social-recovery branch to merge this into

I just created the new branch. You can point this PR to that one now instead of main.

I just created the new branch. You can point this PR to that one now instead of main.

Done

@escottalexander thanks for the review! I believe I've addressed all your comments

@escottalexander thanks for taking the time for the further review! I made all the changes you commented about


GLTF Texture Filter Importing

The GLTF/GLB importer does not seem to be taking into account filter, wrapping, and mipmap settings built into the file.
This feature implements a fix on the 3.x branch by creating and parsing the texture sampler objects from the GLTF file and applying its properties as flags to the created textures.

@TheTophatDemon What's the status of this PR?

I believe @fire made a separate pull request to continue working on this, since I ran out of time in my schedule.

I believe @fire made a separate pull request to continue working on this, since I ran out of time in my schedule.

I don't see any PR made by @fire related to glTF filter in 3.x.
Nonetheless, I've labeled this as salvageable in case another contributor wishes to continue this work.

Superseded by https://github.com/godotengine/godot/pull/66856. Thanks for the contribution nonetheless!


add per identity Always Bcc

I'm new to android development, and not sure if this builds all OK.
But if it does, it should close #1648 and add a feature I'd love to see in K9mail.

Good start. It looks like it builds on Travis :)
There's a few bits missing from an initial look through.
Needs changes to com.fsck.k9.Account#loadIdentities and saveIdentities to correctly retrieve the value from storage. Will need to be written in a backwards compatible way. Right now every time your device reboots (or K-9 is otherwise restarted) the setting will be lost I believe.
Needs changes to com.fsck.k9.preferences.SettingsExporter#writeIdentity to correctly write the preference and SettingsImporter to write the preference.
It would be nice to get some unit testing on the new code (I dislike us merging new features un-tested - the code base is fragile enough without it). I have some stuff for SettingsImport and Export. Ideally we'd get to the point where not having handled SettingsExport and Import would cause a test failure of course :)

@philipwhiuk I added changes as requested.
I also tried to add test in separate PR  https://github.com/philipwhiuk/k-9/pull/3
Feel free to ask for improvement.

I like the feature. But the hard part is migrating the setting from the account level to the identity level. I created issue #2268 to keep track of this.


Chopper: add flatten and to_callgraph

This PR is a WIP of certain high-level API functions.

@ocnkr The "merge" commit just came from merging the upstream hatchet:develop branch into my forked repo to keep it updated.


daemon: Correct toFQDNs to update all IPs on change

The toFQDN implementation's DNSPoller would update only IPs that changes. This was done in a way where unchanged IPs would be considered missing, and thus dropped. We now use all available IPs in the poller to populate toCIDRSet.

This change is

test-me-please

test-me-please


Add Global page charts

Add the two charts in the global page
Filter options will be add in diffrent PR
ChartJS: https://www.chartjs.org/docs/latest/

I will add tests for the query soon

I added the tests


Doesn't work with Django v2.0

It doesn't work with the version 2.0 of Django.
It gives the following error
    from django.core.urlresolvers import reverse, resolve, Resolver404
ModuleNotFoundError: No module named 'django.core.urlresolvers'

According to https://docs.djangoproject.com/en/2.0/releases/2.0/#features-removed-in-2-0
"The django.core.urlresolvers module is removed in favor of its new location, django.urls."

It seems like the project is not alive.

There is a pending PR to resolve this issue at #29.

Try this version:
pip install -e git+https://github.com/Hikaon/django-debug-panel.git#egg=debug_panel

change the middleware file

As a workaround, to install the PR that has the patch that fixes this issue:
pip install git+https://github.com/Naddiseo/django-debug-panel.git@patch-1

I think there is also a bigger problem: https://micropyramid.com/blog/understanding-middleware-functionality-in-django2/
Django requires to change the syntax of middleware.
@recamshak are you planning to make changes to this project at all? I understand you may not want to still develop it, but it would be beneficial if somebody would at least bring it to the state it works. Maybe consider adding somebody to contributors?

This is a good replacement to track ajax calls:
https://github.com/djsutho/django-debug-toolbar-request-history


feat(app node): add rest api endpoints and integrate control-plane grpc

App Node: Represents a CSI node or a node where io app/initiator would be situated.
Goals:

App node resource is needed to issue commands to the node for operations like fsfreeze, unfreeze.
App nodes will now register itself to the control-plane via the rest API.
These app nodes would be used to issue fsfreeze, unfreeze as a part of snapshot creation to ensure filesystem consistency.

Changes in this PR:

This adds the rest api endpoints for registration and operations for app nodes

bors merge


Debugging workspace packages

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
I have modified the "basic" turborepo starter example here to demonstrate the issue.
Which canary version will you have in your reproduction?
2.3.1
Enviroment information
CLI:
   Version: 2.3.1
   Path to executable<EMAIL_ADDRESS>   Daemon status: Running
   Package manager: pnpm9

Platform:
   Architecture: aarch64
   Operating system: macos
   WSL: false
   Available memory (MB): 10547
   Available CPU cores: 12

Environment:
   CI: None
   Terminal (TERM): xterm-256color
   Terminal program (TERM_PROGRAM): vscode
   Terminal program version (TERM_PROGRAM_VERSION): 1.95.3
   Shell (SHELL): /bin/zsh
   stdin: false

Expected behavior
The official Turborepo recommendation is to use compiled workspace packages for internal packages. Debugging these compiled packages should require minimal effort. However, since the packages are bundled with your primary applications (e.g. Next.js), there is no documented guidance on how to properly set up debugging of these compiled workspace packages.
Actual behavior
Compiled workspace package source code breakpoints do not bind on default examples and with turborepo docs guidance.
To Reproduce

Clone the repository.
Open examples/basic/packages/lib/src/index.ts and set a breakpoint at line 2.
From the examples/basic directory, start the development server by running turbo dev.
Attach the debugger by selecting "Next.js: attach server-side" from the dropdown, then click the green "Play" button.
In your browser, navigate to http://localhost:3000/api/auth, or alternatively, run curl http://localhost:3000/api/auth -i from your shell of choice.
The breakpoint you set should be triggered, but it is not.

Additional context
I've found that adding "sourceRoot": "../../packages/lib/src" to the relevant workspace package's tsconfig.json will sometimes allow breakpoints to bind correctly. However, if you have multiple workspace packages, breakpoints may occasionally hit random lines in the dist (JS) files instead of the src (TS) files, even when the sources are fully built and up-to-date. I'm not sure why this occurs; I hope it's just a bug with the JavaScript Debugger plugin or something else that will resolve itself eventually. I am also unsure if setting sourceRoot like this affects anything else.
No response

This sounds like something that would be nice to add, but I'm trying to figure out Turborepo's role here. This sounds more like something that lies at the intersection of VSCode, its built-in debugger, TypeScript, and how to use all of those in a monorepo.

It feels like 90% of time in web development is spent wrangling configuration files, dealing with conflicting tsconfigs, ensuring linting rules are set up correctly, figuring out es modules vs commonjs, configuring bundlers, and in-general, fighting with tooling. From my perspective, one of Turborepo's core strengths is its ability to streamline these sorts of issues in a monorepo environment. By adopting a more opinionated approach and simplifying how these tools work together through transparent documentation, Turborepo helps developers focus less on boilerplate configuration and more on building actual features. I understand that there's a balance of how much to include to keep things simple, but IMO debugging is a core necessity of advacned development like that done in a monorepo. Including this kind of documentation would reinforce the idea that Turborepo isn’t just another monorepo tool; it’s a framework designed to smooth out those rough edges so that teams can spend more time coding and less time fiddling with setup.

It feels like 90% of time in web development is spent wrangling configuration files, dealing with conflicting tsconfigs, ensuring linting rules are set up correctly, figuring out es modules vs commonjs, configuring bundlers, and in-general, fighting with tooling. From my perspective, one of Turborepo's core strengths is its ability to streamline these sorts of issues in a monorepo environment. By adopting a more opinionated approach and simplifying how these tools work together through transparent documentation, Turborepo helps developers focus less on boilerplate configuration and more on building actual features. I understand that there's a balance of how much to include to keep things simple, but IMO debugging is a core necessity of advacned development like that done in a monorepo. Including this kind of documentation would reinforce the idea that Turborepo isn’t just another monorepo tool; it’s a framework designed to smooth out those rough edges so that teams can spend more time coding and less time fiddling with setup.

It feels like 90% of time in web development is spent wrangling configuration files, dealing with conflicting tsconfigs, ensuring linting rules are set up correctly, figuring out es modules vs commonjs, configuring bundlers, and in-general, fighting with tooling. From my perspective, one of Turborepo's core strengths is its ability to streamline these sorts of issues in a monorepo environment. By adopting a more opinionated approach and simplifying how these tools work together through transparent documentation, Turborepo helps developers focus less on boilerplate configuration and more on building actual features. I understand that there's a balance of how much to include to keep things simple, but IMO debugging is a core necessity of advacned development like that done in a monorepo. Including this kind of documentation would reinforce the idea that Turborepo isn’t just another monorepo tool; it’s a framework designed to smooth out those rough edges so that teams can spend more time coding and less time fiddling with setup.

FYI I scoured the Next.JS source code and found these comments in webpack-config.ts which are directly relevant to the issue at hand:
// if `sources[number]` is not an absolute path, it's is resolved
// relative to the location of the source map file (https://tc39.es/source-map/#resolving-sources).
// However, Webpack's `resource-path` is relative to the app dir.
// TODO: Either `sourceRoot` should be populated with the root and then we can use `[resource-path]`
// or we need a way to resolve return `path.relative(sourceMapLocation, info.resourcePath)`


Where is buffers.h?

I cannot compile the project because of the lack of buffers.h
where should that be located?

In my case I was using a docker tensorrt image nvcr.io/nvidia/tensorrt:22.07-py3
it was located in /opt/tensorrt/samples/common/buffers.h


Modelling Card Correlations

Hey fashia,
I recently read your ebisu article again and really enjoyed it! I would really be interested, whether it really reduces the review load empirically?
Card Correlations
But to the main matter of this issue, one aspect which should influence recall of cards, are correlations between them. For example an

increase/decrease of a cards interval/recall probability could also increase/decrease another cards intervall/recall probability (positive correlation).
an increase/decrease of a cards interval/recall probability could also decrease/increase another cards intervall/recall probability (negative correlation).

I am no export in memory, but I believe one such correlation effect is called interference.
Card Correlations within native Ebisu
So my first question would be, whether I understood it correctly, that Ebisu could not model such correlation events, because a correct answer to a card B solely due to a correlation with a previously reviewed card A, would wrongly solely attributed to the card B?
Later review of the card B without the previous cue of card A would lead to an unexpected failure.
Bayesian model of card correlations
Therefore, I was wondering, whether it would be possible to model card correlations within your bayesian framework?
Correlation Modelling
A very rough sketch of this model could work like this:
For each (directed) card pair a prior zero mean distribution is initialized and when a cards changes its estimates (interval/recall probability) the previous changes of other cards are used to try to estimate posterior correlation distributions to these respective cards. Obvisiously larger time differences between these interval/recall probability changes should produce smaller changes to the prior.
Another way to see this would be, that cards form nodes in a graph, and the correlation distributions are directional edges. As the previous update method effectively only updates a few correlation distributions, whilst there are roughly n^2 correlations for n cards in a deck, another strategy would be needed to densely connect the graph.
The following simple example provides a good idea: Assume our priors for the cards 1 and 2 and for the cards 1 and 3 are well estimated and have non zero means with the same sign. As cards 2 and 3 were never reviewed in close timely fashion, our prior for the cards 2 and 3 (and vice versa) still has zero mean.
For this correlation graph to be consistent, we would have to update our 2,3 and 3,2 prior correlation distributions towards the non zero mean and identical sign.
Nicely enough both approaches to updating the correlation distributions can be done for past review data, thus this model would be backwards compatible.
Updating Card Estimates
Such correlation model could be used in the following way:
If the estimate (interval/recall probability) of card A changes due to a review, all connected cards (connected means the correlation distribution has a non zero mean) should also update their intervales/recall probabilities weighted by the mean of the respective correlation distribution.
Advantages
Once the estimates of the correlations between cards are good enough, the review load for positively correlated cards would be reduced. Furthermore, negatively correlated cards are reviewed more often, giving the student the opportunity to address interference issues.
Question
So what do you think about this as an expert in bayesian spaced repition models? Would this feasible mathematically as well as implementation wise?
And thinking of compatibility to Ebisu: You mentioned in another issue, that Ebisu is built on binary cards answers (passed/failed instead of Again/Hard/Good/Easy), could non binary correlations still be used as an input to update Ebisu's parameters?

Hey, I wanted to let you know I haven't forgotten about this and am thinking about this thoughtful comment!

Thats good to know! I am excited to hear your opinion, whenever you find the time.
I have continued to think about this and found an issue, for which I have not yet found an intuitive solution:
Timely decay of correlations
In my rough sketch above I described how the correlation distributions are updated (and our estimate of the correlation hopefully is improved) due to correlating review results of cards. But this model has the build-in assumption, that the true correlation is stationary with respect to time.
I believe that is not true, but I am still uncertain, whether this assumption would be harmful in any real world application. It may very well be the case, that the effect of card correlation induced parameter change is much smaller, than the parameter change due to reviewing. If most correlations means are around the single digit percent size, their effect on the cards scheduling is really more fine tuning.
As I dont think correlations are stationary, and modeling this decay and deriving its parameters from the reviews could be hard, I think it would be the best to ignore this issue at the beginning.
Again I would be interested in your opinion, whether the assumption of stationary correlations could be harmful?


Add a utility function to split a VCS URL into the base repo and a path

Please have a look at the individual commit messages for the details.

This change is

r2: ..to keep them...

Reviewed 1 of 1 files at r1, 2 of 2 files at r2, 2 of 2 files at r3, 1 of 1 files at r4, 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

util/src/main/kotlin/Utils.kt, line 162 at r3 (raw file):

            return Pair(repo, path)
        }
    }

Is there a generic way to do the same for Gerrit/Bitbucket/GitLab URLs?

Comments from Reviewable

Done.

Review status: all files reviewed at latest revision, 1 unresolved discussion.

util/src/main/kotlin/Utils.kt, line 162 at r3 (raw file):
Previously, mnonnenmacher (Martin Nonnenmacher) wrote…
Is there a generic way to do the same for Gerrit/Bitbucket/GitLab URLs?

For Gerrit, no, as it supports arbitrary nesting of projects. We'd need to query Gerrit for valid project names, which is ugly as it requires a network connection. I'd prefer to limit this function to static parsing. For Bitbucket / GitLab / Sourceforge etc., I wanted to add these later once we need them. This commit was more or less for you to agree that we make this a separate function instead of adding it to normalizeVcsUrl().

Comments from Reviewable

Reviewed 3 of 3 files at r6, 2 of 2 files at r7, 1 of 1 files at r8, 1 of 1 files at r9.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


FR: Integration of web3signer

Is your feature request related to a problem? Please describe.
Using the web3signer of Consensys will make setups more flexible: https://github.com/ConsenSys/web3signer
Describe the solution you'd like
Integration of web3signer as a validator service.
Describe alternatives you've considered

Docs can be found here: https://docs.web3signer.consensys.net/en/latest/