[WIP] ARIMA

Here's what I have so far. Any and all feedback welcomed.

@sryza, added conjugate gradient descent (CGD) option. So now can optimize using BOBYQA (no derivative), or with CGD.

@sryza, updated so that p,d,q terms are broken out in all functions. Some other changes:

improved docs a bit (and corrected given p,d,q change
added a new parameter to fitModel, a set of user provided initial parameters for the optimization
set default optimization to css-cgd, since BOBYQA seems to sometimes get stuck when fitting edge case p = 0, d = 0, q = 0
Added test for test case referenced in (3)
small fix to Hannan-Rissanen name, I had misspelled
I reordered some of the function parameters in fitWith... and Hannan... to be consistent with the order in ARIMA.fitModel

@sryza fixed differencing issue. Changed forecast function accordingly and created 2 functions in UnivariateTimeSeries to handle differencing and inverse differencing appropriately. Added test for both.

added stationarity and invertibility checks (but no transformation of parameters if they checks fail. I'm not sure we want that or not)

OK, this looks ready to go in to me.  Is the WIP labeling just a relic, or are there additional changes you wanted to make?

the WIP marker was just to note that we were still making changes. I will give it a full once-over again, and will remove after that.

also, this patch depends on providing AR parameter for intercept inclusion

@syrza removing the WIP tags, so ready to merge whenever you see fit

Awesome, in that case, merging this in


[AIRFLOW-1495] Fix migration on index on job_id

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
@aoen
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1495

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

There was a merge conflict on the migration hash for down revision
at the time that two commits including migrations were merged.
This commit restores the chain of revisions for the migrations,
pointing to the last one. The job_id index migration was regenerated
from the top migration.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
This is a fix for a migration that was already merged.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

LGTM

Codecov Report

Merging #2524 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2524      +/-   ##
==========================================
- Coverage   70.18%   70.16%   -0.03%
==========================================
  Files         150      150
  Lines       11573    11573
==========================================
- Hits         8123     8120       -3
- Misses       3450     3453       +3

Impacted Files
Coverage Δ

airflow/jobs.py
77.67% <0%> (-0.37%)
:arrow_down:

airflow/models.py
87.41% <0%> (+0.04%)
: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 04bfba3...02c56f4. Read the comment docs.


Support Latin 1 => UTF 32 (AVX)

I thin we already merged this one: https://github.com/simdutf/simdutf/pull/301

I think we already merged this one: https://github.com/simdutf/simdutf/pull/297


Update README.md

Description

Simple update on the README.md so it doesn't say 404 not found.
Checklist for all submissions

[x ] Have you followed the guidelines in our Contributing document?
[ x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ x] Have you tested and reviewed your changes with confidence that everything works?
[ x] Did you branch your changes and PR from that branch and not from your master branch?

If not, why?:

Bump :)

Bump :)

no need to bump this, we see everything that happens ;)


No charts for pageserver and safekeeper?

Why are no charts present for pageserver and safekeeper services? We wanted to run a complete Neon DB cluster on K8s cluster.

Any update on this? Has anyone successfully deployed Neon DB in k8s?

Any update on this?

Hi.
There is a significant demand for adding storage components.
We recognize the importance of addressing this issue promptly for the development of the community.
To that end, we have submitted two proof-of-concept (PoC) pull requests to add Safekeeper, Pageserver, and MinIO.
We hope these will be merged as soon as possible, as they do not affect other charts.
The pull requests have dependencies; do not merge the #91 until the Actions (Release Charts) in the #90 are complete.
The procedure to experience this change is as follows. It is a bit complicated and needs improvement. [1]
helm repo add test-chart https://seinoyuki.github.io/helm-charts/
helm repo update
helm install neon1 test-chart/neon-pageserver
helm install neon2 test-chart/neon-safekeeper --set settings.s3.enabled=false --set settings.broker.enabled=false --set settings.s3.fullnameOverride="neon1-neon-minio" --set settings.broker.fullnameOverride="neon1-neon-storage-broker"

The following issue remains.

[ ] Addition of README and NOTE
[ ] Linkage between Pageserver and Storage Controller
[ ] Linkage between MinIO and Neon Storage Scrubber
[ ] Prepare Chart to deploy compute.

Neon cluster must be able to be built with one helm install. [1]

[ ] Support sharding of Pageserver

Thank you for clarifying the policy.
The PRs I submitted, #90 and #91, have been closed.
I would like to revisit this discussion when Neon is mature enough for k8s deployment.


Use ruby_exe with exception: false

Prepared specs to pass with not merged yet changes in MSpec (https://github.com/ruby/mspec/pull/49).
Changes:

changed every test case where Ruby script (which is run with ruby_exec helper) exits with failed status (expected failure)
specified introduced exception: false option.

I checked specs locally on MacOS with Ruby 2.7 and 2.5 only so there may be some overlooked failing tests targeting other versions/platforms.
I am wondering whether there is a way to run a build on CI with MSpec feature-branch and RubySpec feature-branch. Otherwise master may fail after merging the MSpec's PR.

@eregon Is there any objection to merge the PR now?

Reviewing now.

This looks fine, but I'm wondering if something like ruby_exe(..., exit_code: 1) would better (it would also check the exit code in the non-zero case).
Re merging, we can merge here since that's compatible, and then rebase the MSpec PR which should run latest ruby/spec against it. Might be a good idea to trigger a CI run after the MSpec PR is merged here to ensure it's fine on all platforms.


I've decided to separate the Expo module into another repo. You may follow the link below to keep updated.
https://github.com/Hector-Chong/expo-native-wechat

Finished


Unresolved Question: Who owns Resource objects? When are resource objects deleted? Do consumers of resources own them?

Resource objects are shared ptrs (like Ogre).


The observable interface no longer contains the forEach method

The Observable interface as defined here
No longer contains the method forEach() which is used to determine if an object returned by a task is an observable here.
I'd propose adjusting that line to something like
const isObservable = obj => obj && typeof obj.subscribe === 'function' && typeof obj.constructor.from === 'function'

I don't forsee the from() static method being removed since it's the basis for converting to and from different observable types.
I'd be happy to PR this if it would be accepted.
Thank you for you time :smile:

I guess you're right. PR welcome! :)


Better support for big repos/workspaces

Right now, we have a 100MB-ish limit on workspaces. With a few tweaks, we could drastically improve this. Many of these changes need to be made on the back-end, but we'll also need to increase the limit in our plugins once we've done this. Putting this issue in intellij because that's typically the editor people use with massive java enterprise projects.

Example repo that totally borks: https://svn.kuali.org/repos/ole/trunk/

Is there a process to vote on issues? I would love to see this fixed soon so that I can try it for our team.

I think sparse mode fixes this. It will be released soon.

@peris can you try the new option when sharing to only upload the directory you want to work on and see if that works for you please?

Sparse mode should fix this.


[BUG]: MySQL decimal is inferred as string

What version of drizzle-orm are you using?
^0.28.2
What version of drizzle-kit are you using?
No response
Describe the Bug
const table = mysqlTable('table', {
  column: decimal('column').notNull(),
})
type Table = InferModel<typeof table>
     ^? { column: string }

Expected behavior
The table column should be inferred as a number. It's also not possible to insert floating point numbers.
Environment & setup
No response

This is correct behavior currently, the decimal type was initially designed to be used as string, because it can have very large amount of digits after the decimal point. We plan to change that in the future though, because it's confusing for the users. For now, you can use real or float.


Merge to live!

Docs Build status updates of commit 701f6c2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/privacy-and-security.md
:white_check_mark:Succeeded

docs/excel/custom-functions-json.md
:white_check_mark:Succeeded

docs/excel/custom-functions-parameter-options.md
:white_check_mark:Succeeded

docs/reference/manifest/appdomain.md
:white_check_mark:Succeeded

docs/reference/manifest/iconurl.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/outlook-requirement-set-preview.md
:white_check_mark:Succeeded

docs/testing/debug-add-ins-in-office-online.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Audit & Overhaul of the URL utilities

This is an audit & overhaul issue. We could spend forever on it, the idea is to spend a small amount of time and significantly reduce the number of issues. The long tail of edgecases get cleaned up over time.
The audit centres around the utils/url.js file & how the utilities inside it are structured and used.
First, there are one or two known issues:
Audit:
Review & fix the following bugs:

[x] Fix GhostAuth redirect uri mismatch error for urls with a trailing slash. #7656
[ ] Canonical vs logo blog URLs being output differently i.e. one has a trailing slash and one doesn't  (reported in slack, could have been a theme issue).
[ ] Audit the codebase for use of the urlJoin and urlFor functions:

where they are used, are they used correctly? Consider how they are used to try to identify any small improvements
is there anywhere that they should be used and are not? All internal Ghost urls should be either fetched from urlFor or concatenated using urlJoin never +.

[ ] Review recent url & subdirectory related issues, and check to see if the changes in 1.0 have caused any regressions
[ ] Check the test coverage for the utilities, and any places in the codebase where the utilities are heavily used.

Overhaul:

[ ] if test coverage is found to be lacking, improve this
[ ] make any small improvements that have been identified
[ ] summarise findings & write a new issue to cover any larger changes that you'd like to make

I would like to limit this issue to the following tasks.

[ ] review the existing PR https://github.com/TryGhost/Ghost/pull/7689
[ ] reconsider using urlJoin for social urls  https://github.com/AileenCGN/Ghost/commit/cdc1c552e8ec580604691b9c07452c548770356c#diff-3b9516dc9764f7652261f9383f883d20R5
[ ] QA: ensure we have one true place for url handling and use it everywhere

Regarding this

reconsider using urlJoin for social urls AileenCGN@cdc1c55#diff-3b9516dc9764f7652261f9383f883d20R5

I think this has to be removed again, as we won't use urlJoin for external urls. I will revert this change and update the PR.


✨ 🎨 Conversational Learning page

Description
This PR introduces a well-implemented new feature for our conversation learning page.
Fixes #79
Type of change

[x] New feature (non-breaking change which adds functionality)

Screenshot (optional)
Desktop:

Tablet:

Mobile:

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Kindly review this PR.
I did it with @Vickynjoroge.

Good work. Make sure to only push the required files when creating your pr. And also the styles folder would have looked better separated. Then on the pr name, attach the issue number. All in all. Good work


Remove duplicated websocket client & constants

Summary
Defunct code originally maintained in mattermost-redux, but not used in mobile, in favour of https://github.com/mattermost/mattermost-mobile/blob/master/app/client/websocket.ts
Ticket Link

MM-23798

Device Information
This PR was tested on:

Android 10 emulator
iPhone 11 simulator

@chuttam
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
!!! 'upstream/release-1.31' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

/cherry-pick release-1.31


Is MM compatible with gh-pages (github Projects)?

I am trying to set up a second blog on my github.io page. I followed the instructions here, i.e. I created a gh-pages branch and added baseurl: /travel to _config.yml. As far as I understand, this should result in my new blog being accessible at http://alkamid.com/travel, but it's not (I'm getting 404 — you can check it by opening the link). What might be the cause?

You'll need to modify some bits of the theme to use baseurl properly. Other than that the theme is GH pages compatible. I'm hosting the demo version of the theme there just fine.

Also your site is available just not in /travel as you expected.
https://alkamid.github.io

@mmistakes It's not available. You can tell by "Living in the Cambridge bubble" in the author box, instead of "Learning to undersdand" (see the config file in gh-pages branch).
What bits would I have to modify to use baseurl? permalinks perhaps?

Not sure. Never used it with any projects. https://talk.jekyllrb.com/ is
the place for this sort of help. It might have to do with how your repo is
named. That plays a big part in how it's hosted. GH pages has different
ways of doing it and it's easy to screw up.
On Sunday, August 9, 2015, Adam Klimont<EMAIL_ADDRESS>wrote:

@mmistakes https://github.com/mmistakes It's not available. You can
tell by "Living in the Cambridge bubble" in the author box, instead of
"Learning to undersdand" (see the config file
https://github.com/alkamid/alkamid.github.io/blob/gh-pages/_config.yml
in gh-pages branch).
What bits would I have to modify to use baseurl? permalinks perhaps?
—
Reply to this email directly or view it on GitHub
https://github.com/mmistakes/minimal-mistakes/issues/168#issuecomment-129239453
.

You might have to move the travel blog into its own repo. Might be getting confused with the master branch. For all my Jekyll themes they're hosted as projects in the gh-pages branch. Each in they're own repos v

@mmistakes you're right, this setup was confusing Github indeed. I moved the travel blog into a new repo, added baseurl (but I'm only currently using it for language selector buttons because my blog is bilingual), deleted CNAME, and it seems to work.
For anyone who's asking in the future:
http://stackoverflow.com/questions/31906986/two-jekyll-blogs-on-the-same-github-page
http://stackoverflow.com/questions/31914048/how-to-set-up-two-jekyll-blogs-on-github-pages-with-the-same-domain-cname

Glad you got it sorted.


Addition of Mathematical Algorithms

I have added new Algorithms:
Shortest Path : Dijkstra, Floyd-Warshall etc
Minimum Spanning Tree: Prim's etc
Topological Order

#26

Hi @pankaj-bind,
I reviewed this pull request and noticed it's not a level 3 task.
Documentation updates should only be classified as level 1, and the other contributions are simple programs that can be easily found online or generated with AI tools like GPT, so they don't count as feature additions or bug fixes.
Your repo primarily contains README contributions and program submissions, so regardless of the number of lines, assign only level 1. Also, if contributors repeatedly make similar contributions, inform them that the limit for your type of repo is 150. Make sure to monitor this.

@pankaj-bind i am community manager of gssoc-extd please make sure to make this reduction in all the pull requests you have merged so far

@pankaj-bind please reply to me in discord

@pankaj-bind please reduce the level like i asked you to


Rultor hi

@rultor hello

@rultor hello

@viliusl Hi there! I understand a few simple commands: "merge", "deploy", etc. This page explains them briefly. Any questions or bug reports are welcome. Have fun :)


Add default header {x-li-src: msdk} that it works with linkined api

Otherwise access token in header does not work (anymore?)
See here for details:
https://stackoverflow.com/questions/31751441/linkedin-verify-oauth2-access-token-on-server-side
https://stackoverflow.com/questions/28094926/linkedin-oauth2-unable-to-verify-access-token

Coverage remained the same at 100.0% when pulling d4c4433e7b1c416c4bbe486e67d67549dbb2bb52 on freework-gmbh:master into ceb63460b8b45b7e00766ae3a4c3e361d76c6ffc on emorikawa:master.

Coverage remained the same at 100.0% when pulling d4c4433e7b1c416c4bbe486e67d67549dbb2bb52 on freework-gmbh:master into ceb63460b8b45b7e00766ae3a4c3e361d76c6ffc on emorikawa:master.


Bio

👷 Deploy Preview for polite-dolphin-8b4710 processing.

Name
Link

🔨 Latest commit
ceb41511a2c121f137fae46233c7193e1db17577

🔍 Latest deploy log
https://app.netlify.com/sites/polite-dolphin-8b4710/deploys/635a047c315ccc00096cf895


search function  does not work

press enter nothing happen

What version of BitCannon are you using and are you sure MongoDB is running?

BC_Windows_v0.0.5
everything is ok,just no results(which I copy from browse) when searching

What does the bitcannon command prompt say? It should have automatically imported some torrents.
Also, does the Browse page show categories?

http://i.imgur.com/r8Yh5ar.png

What version of MongoDB do you have? BitCannon needs 2.6 or greater for text searches to work.

sorry for bothering,I will try the latest MDB

Don't apologize! It's a common mistake. Let me know if it works!

cool,thx.....

If 2.6 fixes it, feel free to press the close button on this issue. I'll remember to update the instructions to note this.

Were you able to get search to work with Mongo2.6?


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/100438

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/103419

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104146

TagBot fix

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/111545


Consider using a different keywork declaration (use val instead of const)

This is more a question than an issue
Have you considered using a different keyword, like val (meaning value similar to scala) instead of const.
I find it confusing that the same keyword would work completely different from JavaScript (and Typescript) on a perfectly valid ts code base, depending whether we are using ValueScript or not.
It seems like const has been chosen by design, to keep ValueScript syntax compatible with TypeScript, but I think i ends up being really confusing.
On the other hand, having a new and distinct val variable declaration, that declares an immutable value, would be much more clear (in fact, I think JavaScript should directly support such a thing)
Anyway, this is an excellent project, and I think it's meant to fill an obvious and much needed hole in JavaScript spec. I only wish this project is successful enough to make people at CT39 realize how badly JavaScript needs this feature to keep evolving.

Hi @opensas 👋
I think this point actually applies more generally. The downside of this dialect approach where I'm redefining the behavior of the same syntax - that can be confusing. There's also the upside of tooling compatibility and a much easier transition than learning a new language.
I think it's more positive than negative, but it's definitely something to keep in mind as we learn more about what it's like to use ValueScript. It's very possible we'll pivot and deviate from TypeScript a bit more to make it more of its own language than just a dialect.
For the moment, using val is a non-starter because we rely on SWC for parsing and benefit hugely from TypeScript tooling because of 100% syntax compatibility. If this does ever change we'll need to do a language server for intellisense and define syntax highlighting and all that, but for the time being that just takes care of itself.
Personally I'd quite like to adopt rust's mut so that we have constness by default instead of mutability by default.

Anyway, this is an excellent project

Thanks 💪


Improve language in child README.md

Summary
Describe the goal of this PR. Mention any related Issue numbers.
Requirements (place an x in each [ ])

[ ] I've read and understood the Contributing Guidelines and have done my best effort to follow them.

[ ] I've read and agree to the Code of Conduct.

[ ] I've written tests to cover the new code and functionality included in this PR.

[ ] I've read, agree to, and signed the Contributor License Agreement (CLA).

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


Crash when trying to resize grid

I got this Crash when trying to resize a grid, i believe i fixed it by changing this line 56 in LightweightGrids:
array_copy(newDataSize, 0, _data, 0, wh);
to this
array_copy(_newdata, 0, _data, 0, wh);
I believe this was a simple typo, if it was intended and the issue lies elsewhere please let me know:
local variable (-1610511209, -2147483648) not set before reading it.
at gml_Script_anon_Grid_gml_GlobalScript_LightweightGrids_837_Grid_gml_GlobalScript_LightweightGrids (line 56) -                             array_copy(newDataSize, 0, _data, 0, wh);
############################################################################################
gml_Script_anon_Grid_gml_GlobalScript_LightweightGrids_837_Grid_gml_GlobalScript_LightweightGrids (line 56)
gml_Object_oController_Draw_0 (line 17) - depthgrid.resize( 2, max(1,inst_number));

Thank you for reporting this bug. I have fixed this alongside another HTML5-related issue, and pushed up a new maintenance release.


Make Tetris.state a property to automatically manage it

Hey man, want to hear an original saying that I came up with? I'm pretty pround of this one. Here it goes:

Don't do the work using the machine, have the machine do the work for you!

I thought my saying applies nicely to this refactor.

That's a great saying! I think it applies perfectly to this refactor, too.
I also learned a new thing about property.
Thank you very much!

Yeah, I think properties would be a great tool for handling state-related attributes in a class. They are overloaded methods, so you can do whatever kind of hidden logic inside them while using them in a seamless attribute assignment/access fashion.


tests: use basic files in Git for fixtures

It's getting too complicated to dynamically generate all the necessary files in a realistic way, and this allows us to browse the fixture tree (like with find or ls -R).

The next PRs will add more files to merfi/tests/fixtures


Incorrect configuration for size

The documentation and the sample code says the size parameter is String but it seems to be Number in the source code (line 11):
https://github.com/obsidiansoft-io/fa-icon/blob/880a8a2c7ac677cd7e5d2d27bb0d75607f62e2aa/fa-icon.js#L5-L14
If I set the size parameter as Number it doesn't seem to work for me. If I set it as String as shown in the readme file of this repo, it works but my editor (visual studio code) complains that I have a syntax error (passing String instead of Number in the parameter value).

Thanks for report, now size is type String in the version 0.1.6.

Cool!! I'll try it out soon. Thanks for the quick fix.

maybe the eslint you use detects the word "size" by default as if it were a numeric type

I have the following ESLint installed and enabled, but don't have any .eslintrc file or any rules configured in the settings.json of that linter.
I think there's still issue in the code. Just realized that the latest version is still setting the size as a number in the constructor instead of a string.
https://github.com/obsidiansoft-io/fa-icon/blob/af87472d5332cc89c7e052002c867deaa8a252a8/fa-icon.js#L44-L52
If I manually update the source in my node modules and set that value as a string in the constructor, I don't see anymore errors on my element (although I see other linter errors on your source about missing semi-colon)

Can this issue be reopened please?

Thanks for report, now size is type String in the constructor, fixed in the version 0.1.7
I will leave the issue open until you have verified that the change really worked

Yup. It seems to be fixed now. Thanks!!


Use singleton for Laravel 5.2

Replace deprecated bindShared method with singleton for L5.2

Thanks!


DDP/DP training - multigpu

Hi @chrockey, great work!
Can you guide me on how to set up multigpu training? I have only 20GB gpus available, and when using batch size of 2 I obtain poor performance (~6% lower mIoU and mAcc; probably due to the batch norm and batch size).
If I add multigpu support (DDP) according to the example from the ME repository the learning is blocked, i.e. it never starts.
Any help will be appreciated. You commented "multi-GPU training is currently not supported" in the code. Have you had similar issues as I mentioned?
Thanks!

Hi @helen1c,

Have you had similar issues as I mentioned?

No, I haven't. I was able to use DDP with PyTorch Lightning and ME together. However, I found a weird issue: the model's performance gets a little bit worse (~1%). That's why I do not use multi-GPU training in this repo. Anyway, here I provide you a code snippet to support DDP training:
You need to convert BN module into the synchronized BN before this line: https://github.com/POSTECH-CVLab/FastPointTransformer/blob/9d8793a1bb0bb5e2a6175f32dd116f88c8171d23/train.py#L39
as
if gpus > 1:
    model = ME.MinkowskiSyncBatchNorm.convert_sync_batchnorm(model)
    model = torch.nn.SyncBatchNorm.convert_sync_batchnorm(model)

Then, pass the DDP-related keyword arguments here: https://github.com/POSTECH-CVLab/FastPointTransformer/blob/9d8793a1bb0bb5e2a6175f32dd116f88c8171d23/train.py#L61 as
if gpus > 1:
    kwargs["replace_sampler_ddp"] = True
    kwargs["sync_batchnorm"] = False
    kwargs["strategy"] = "ddp_find_unused_parameters_false"

I hope this helps your experiments.

@chrockey Unfortunately, this doesn't help. The same problem again.
Can you provide me versions of torch, cuda and pytorch lightning you are using?
Thanks for the quick reply though! :)

Sorry for the late reply.
Here are the versions:

CUDA: 11.3
PyTorch: 1.12.1
PyTorch Lightning: 1.8.2
TorchMetrics: 0.11.0

If you have further issues, please feel free to re-open this issue.

Hi @chrockey ,
I ran into the same problem, it worked well when I trained with a single GPU, but when I tried multi-GPU training as you suggested, the training process stopped at epoch 0 and no errors were reported. The GPU memory is occupied, but the GPU utilization is 0.

Hi @chrockey ,
I ran into the same problem, it worked well when I trained with a single GPU, but when I tried multi-GPU training as you suggested, the training process stopped at epoch 0 and no errors were reported. The GPU memory is occupied, but the GPU utilization is 0.

Hi @lishuai-97 , I met the same problem as you decribed. Could you please give me some suggestions on how you solve it? Thanks a lot!

Hi @chrockey ,
I ran into the same problem, it worked well when I trained with a single GPU, but when I tried multi-GPU training as you suggested, the training process stopped at epoch 0 and no errors were reported. The GPU memory is occupied, but the GPU utilization is 0.

Hi @lishuai-97 , I met the same problem as you decribed. Could you please give me some suggestions on how you solve it? Thanks a lot!

Hi @Charlie839242, sorry for the late reply, unfortunately I still didn't solve the problem in the end, but I think it may be a problem with the pytorch-lighting setup, and I have now moved to a new point cloud processing repository https://github.com/Pointcept/Pointcept, which is also an amazing work including many SOTA methods.


ansible_args passed to all steps

Issue Type

Bug report

Molecule and Ansible details
molecule 3.2.0a0
    ansible:2.9.7 python:3.8
    delegated:3.2.0a0 from molecule
    podman:0.3.0 from molecule_podman
    vagrant:0.5 from molecule_vagrant

Molecule installation method (one of):

pip

Ansible installation method (one of):

OS package

Desired Behavior
I have a particular use case where I want to test a full playbook and it's roles. However, the playbook has to be called with specified tags. I would call the playbook in production like this:
ansible-playbook site.yml -t install

Therefore, I configured my molecule.yml like this:
---
dependency:
  name: galaxy
driver:
  name: vagrant
platforms:
  - name: opensuse15
    box: generic/opensuse15
  - name: debian10
    box: generic/debian10
provisioner:
  name: ansible
  ansible_args:
   - --tags=install
  playbooks:
    converge: ../../site.yml
verifier:
  name: ansible

I would expect molecule to pass these parameters to the converge step only.
Actual Behaviour
Molecule passes the tags also to create and destroy, which makes my machines never start/stop.
A good create command looks like this
COMMAND: ansible-playbook --diff --inventory /home/t/.cache/molecule/playbook/default/inventory --skip-tags molecule-notest,notest /home/t/.local/lib/python3.8/site-packages/molecule_vagrant/playbooks/create.yml

With ansible_args it looks like that:
COMMAND: ansible-playbook --diff --inventory /home/t/.cache/molecule/playbook/default/inventory --skip-tags molecule-notest,notest --become --tags=install /home/t/.local/lib/python3.8/site-packages/molecule_default/playbooks/create.yml

The create playbook doesn't do anything with just this tag.
I have reprodiced this with both vagrant and podman driver.

My current workaround is to apply tags manually to the converge step, but i would like to avoid this
cat molecule.yml
---
dependency:
  name: galaxy
driver:
  name: vagrant
platforms:
  - name: opensuse15
    box: generic/opensuse15
  - name: debian10
    box: generic/debian10
provisioner:
  name: ansible
  playbooks:
    converge: ../../site.yml
verifier:
  name: ansible

molecule create
molecule converge -- -t install
molecule verify
molecule destroy

Thanks a lot!

I have a "bug ?" linked to this issue... I don't know if it is fixed with the given fix.... (to know if i open something for it or not).
In my case, it is prolbematic that the default "notest" tag is passed to create step.
In my create playbook, i am using a role to create some specific staff.... But this role was tested with molecule... and have some tasks with the given "notest" tag.... so the creation did not work, as theses tasks are not executed.....
For me theses specific tags should be passed only to converge step, no ?
It is the case with this fix or not ? (if not i will open a new issue)

For those who need to pass config options to all steps and found this issue searching for a solution: Use the config_options in the provisioner section of the molecule.yaml file.
I needed to pass a vault-keyring-script to all molecule steps and used ansible_args first, but as stated above that is not intended for this purpose.
Do it like this:
# molecule.yml
provisioner:
  name: ansible
  config_options:
    defaults:
      vault_identity_list: <identity>@<path-to-keyring-scrpt>


Update password doesn't work as it should.

https://github.com/SELab-2/OSOC-4/blob/6db1734fcef4b3fc4d0f3ac9e994bf4a9bd3b22b/backend/app/routers/users.py#L205
I REALLY don't think this endpoint should work like that.
I'm assuming this endpoint is meant for users to be able to change their own password.
First of all, only admins have access to this function so any other user that doesn't have admin access has to go to the admin to change their password.
Secondly an admin can only change the password if they know the old password.
Thirdly the best way for a coach to change their password is to use the forgot password function. So even if this endpoint was intended behaviour it still doesn't work as it should.

@thabils could i get your opinion since you wrote it? If you agree please rewrite it.


Option to delete user and cluster used by context when deleteing it

For my use case, using -d is super annoying, as ~/.kube/config file are left with defunkt entries from previously created clusters.
I'd like to request an option to delete user and cluster used by context when deleting it.

Sadly kubectl doesn’t provide clean cmds to delete user and cluster entries. I have an issue open about this for a long while.
Since we don’t directly manipulate the kubeconfig file, it doesn’t seem currently doable. But we can get to it.

This seems to be possible now:
https://kubernetes.io/docs/tasks/access-application-cluster/configure-access-multiple-clusters/

Is there still interest in this from the project authors? (@ahmetb)
If so, I'm happy to work on adding it.
Also: how would you like to see this being supported?

A separate flag (e.g., -D or -dd)?
A companion flag to -d (e.g., -d -p, "p" is for purge)?
Change behavior of -d

And also: need to consider that the deleted context's cluster/user might still be referenced in other contexts that are still there. In such case, it would be safer to avoid deleting the cluster/user, but possibly provide a way to force the deletion (e.g., -ddd in line with option (1) above, or `-d -p -f in line with option (2)).
I personally am more fond of -dd and -ddd, but I'm also good with -D and no force deletion of non-orphaned clusters/users.

Yeah, I think we can add -D which actually deletes associated orphaned entries. Changing the default behavior of -d is gonna make some folks upset for sure (given there are a lot of course material and content out there explaining people to set up clusters manually, and manually editing the kubeconfig file to make it work).
I recommend adding this feature only to the Go implementation for the sake of simplicity.

@ahmetb please see #409 which adds this as a new -D flag (indeed, only for the Go implementation 😉).


Use common qs bundle ID format in Crashlytics qs

Was using crashlytics qs to debug an issue and got a warning that the bundle ID didnt match with that in the default googleservice-info.plist.
I did this with:
grep -rl 'Google-LLC.CrashlyticsSwiftUIExample' . | xargs sed -i '' 's/Google-LLC.CrashlyticsSwiftUIExample/com.google.firebase.quickstart.CrashlyticsExample/'

Hmm im unsure why the failure is occurring. It's building locally on Xcode 13 and that view modifier is still called task


Consider name collision with packt book

Packt has a book in the works called "Rust Cookbook": https://www.packtpub.com/application-development/rust-cookbook
We might want to rebrand to avoid the collision.
cc @dvigneshwer

The book follows a typical cookbook type format where it has many simple DIY recipes across different chapters focussing on various concepts of Rust and towards the end of the book the reader uses his collective knowledge gained to build simple end to end real-world applications in Rust.
Having different names will definitely help in reaching out to more people.

This is a tricky issue, because the word "cookbook" has such a strong association with what we want to convey here. But perhaps we could include it as a subtitle.
In any case, here are some ideas to maybe kick off a bikeshed:

Rust: Batteries Included (my favorite)
Crates off the shelf: a Rust Cookbook

Variant: Crates off the shelf: a HOWTO guide for Rust

Rust Recipes
Unloading Cargo: a guide to Rust's most important crates

Shame about the name as it started to get traction (I see it suggested on reddit every once and a while along with RBE). I like subtitling idea :+1:
How about:

"Rust Recipes: a Cookbook for oxidized diet" :stuck_out_tongue_closed_eyes: (maybe something shorter)
"Cooking with rust"

A thought:

"Rust by Example" => RBE
"Rust: Batteries Included" => RBI

Nice pair of books there :-)

I liked something @skade said in Why is Rust Successful?.

We are not a batteries-included language; we are a batteries-easily-downloadable language.

But idk how I would translate that to a book title. I feel batteries-included is a more appropriate name for the thing OCaml is using it for: "defines a standard set of libraries which may be expected on every compliant installation of OCaml."

I would propose the Rust Atlas, which I like even better than "cookbook". This characterizes a few important aspects of the book's role:

Given a problem you intend to solve, the atlas gets you headed in the right direction. That may mean it helps you identify the most appropriate crate and even links you to the documentation of the module or function that matches your use case.
The atlas is exhaustive and thorough, but meticulously organized so as to not be overwhelming.
The atlas is something that is valuable to both brand new and experienced Rust users.
The atlas is fun to browse, just to enjoy what all exists in the world.

I think Cookbook is practically a genre of books, and this definitely fits.  I am aligned with 'Rust Cookbook' being the subtitle.
Since finding the batteries (ie knowing which crates to use) is part of the challenge with starting Rust, instead of Batteries Included I was considering:
Where's The Batteries?

A Rust Cookbook

FYI also https://www.packtpub.com/application-development/rust-standard-library-cookbook

It seems like the world has room for multiple cookbooks, of any given type.
Should this issue be closed?


feat: support commitlint plugins key

Currently, knip only supports commitlint extends, but not plugins. This pr aims to fix that

Excellent stuff as always

:rocket: This pull request is included in v5.9.0. See Release 5.9.0 for release notes.
Using Knip in a commercial project? Please consider sponsoring me.


New methods to get Array and Map fields without unchecked casts

Is your feature request related to a problem? Please describe.
The DocumentSnapshot class provides methods to retrieve a field by Firestore type, with the exception of 2 types: Array and Map. Retrieving the content of an Array or Map field and assigning it to a variable requires to call get and to do an unchecked cast.
Describe the solution you'd like
For instance, adding 2 new methods such as public <T> List<T> getArray(@Nonnull String field) and public <T> Map<String, T> getMap(@Nonnull String field).
Describe alternatives you've considered
Making an unchecked cast or using get with the second parameter that accepts a value type class but still requires an unchecked cast.
Additional context
I guess there is maybe a reason for not providing such methods (i.e. values for Array and Map are heterogeneous in Firestore and you do not want unchecked cast in your client library?). In that case, a note in the documentation suggesting the recommended approach would be helpful.

Here is a basic utility class I have written to prevent boilerplate code and unchecked casts at several places in different projects:
public final class Firestore {

    private Firestore() {
    }

    public static <T> List<T> getArray(DocumentSnapshot documentSnapshot, FieldPath fieldPath) {
        return getArray(documentSnapshot, fieldPath.toString());
    }

    @Nullable
    @SuppressWarnings("unchecked")
    public static <T> List<T> getArray(DocumentSnapshot documentSnapshot, String fieldPath) {
        Object fieldValue = documentSnapshot.get(fieldPath);

        if (fieldValue == null) {
            return ImmutableList.of();
        }

        if (!(fieldValue instanceof List)) {
            throw new IllegalArgumentException("The field value at " + fieldPath + " is not an array");
        }

        return (List<T>) fieldValue;
    }

    public static <T> Map<String, T> getMap(DocumentSnapshot documentSnapshot, FieldPath fieldPath) {
        return getMap(documentSnapshot, fieldPath.toString());
    }

    @Nullable
    @SuppressWarnings("unchecked")
    public static <T> Map<String, T> getMap(DocumentSnapshot documentSnapshot, String fieldPath) {
        Object fieldValue = documentSnapshot.get(fieldPath);

        if (fieldValue == null) {
            return ImmutableMap.of();
        }

        if (!(fieldValue instanceof Map)) {
            throw new IllegalArgumentException("The field value at " + fieldPath + " is not a map");
        }

        return (Map<String, T>) fieldValue;
    }

}

I am wondering if this could be natively supported by this client library?

Good idea, it's something we can look into!

@lpellegr I've spoken with several folks from the Java OpenSource team here at Google and the consensus we've reached about this issue is: for now we don't want to add a method to the library following the code you have shared.
The reason for this are due to a few reasons:

Due to the nature of arrays in Firestore allowing storage of heterogeneous values, there is no easy way for us to create a method that can safely return List<T> for all scenarios. (We acknowledge that this is not an issue for those Firestore Arrays which do only contain homogeneous values, but there isn't currently a way for us to assert this in a safe and performant way for all users)
Due to the nature of Maps in Firestore also allowing storage of heterogeneous values, there is no easy way for us to create a method that can safely return Map<String, T> for all scenarios.
Given 1 and 2 we do not want to create a method on DocumentSnapshot that can shadow the possibility of a ClassCastException happening for some users. (We acknowledge that this is not an issue where homogeneous values are used, but again we can't guarantee that for all cases)
Given the unsafe nature of casting a full list, we feel it best to allow the compiler to show the warning to users and allow them to choose if they want to suppress it for a method/class or to make a wrapper method as you have done.
The nature of Java's implementation of Generics also leads to difficulty in us trying to provide a safe method to satisfy the heterogeneity of the underlying values
We may be able to look into creating some custom List-like & Map-like data structures that would be able to provide the necessary safety guarantees we would be comfortable with, but I do not know what the timeline for something like that would be.

For now I am going to close this issues, as I don't know when we'll be able to get to the extra work mentioned in 6.


cjs version is not supports --no-color

CJS: https://github.com/ai/nanocolors/blob/main/index.cjs#L3
ESM: https://github.com/ai/nanocolors/blob/main/index.js#L3

Fixed 96a097a

Released in 0.2.1


Template for the JKW

I am willing to contribute to developing a quarto template for the Journal of Kinesiology and Wellness. I have the latex file ready.
I do not have experience as a developer, but with some guidance, I will work hard to get it done.
Any help would be greatly appreciated.

Hello @drfurtado ! Thanks for considering making a new template to use with Quarto !
This repo has been made to help create an extension format. You can start by using this Github template to start. Than documentation is at https://quarto.org/docs/extensions/

I would start understanding what a Quarto extension is at https://quarto.org/docs/extensions/
then specifically focus on Journal article extension, for which you have examples : https://quarto.org/docs/extensions/listing-journals.html
Documentation for journal article is at https://quarto.org/docs/journals/formats.html where we detailled some step

More pratically

First thing will be, based on our LaTeX template to modify the LaTeX partials in Quarto. With Quarto, we specifically allow to tweak only part of the default LaTeX template to ease modification and keep most of Quarto feature. You could also pass a whole custom template but that is not best usage. Those partials are usual Pandoc's template.
For authors part, Quarto offers a specific parsing of metadata to be used in templates, that should allow most of the flexibility that Journal's template requires
For quarto to recognize your format, you need to fill the _extension.yml as shown in quick start example. You can pass default values for you PDF format, as well as any resources.
Than depending on the complexity of your journal's template and its feature, you'll need filters or/and shortcodes that would provide more feature to your document for Markdown syntax to your LaTeX template.

Please give it a try, and ping me in your repo if you need any help.
Hope this helps gets started

Sorry, this is beyond my basic knowledge of latex and Quarto.
I would love to get help from the Quarto community on this matter.
The JKW is a small not-for-profit journal, but we could hire a freelancer to create a repo with the Quarto template based on our latex template.
Any help directing to potential collaborators would be greatly appreciated.

Oh I see - sorry I misunderstood.
I added a label to make this more clear. Hopefully people will pass by. You can also throw then idea on twitter or Mastodon.
I'll keep this in the list of ideas for when I can work more on creating new format. Not very soon unfortunately.

Please place this on your to-do list. Having a template in Quarto will significantly expedite the process of editing papers and allow authors to use the repo for submissions.
Also, Quarto's html output is awesome.
Greatly appreciated.

Please place this on your to-do list. Having a template in Quarto will significantly expedite the process of editing papers and allow authors to use the repo for submissions.

From experience on R package rticles, the real issue is maintaining a format - this is best done by users of the formats or people who knows the template well and what they want in it.
Though, we'll consider it when we can dedicate time to new templates. Thanks

Thanks for considering it. If added, I can volunteer to maintain it.


Besu reports payload as valid from newPayload but invalid from forkChoiceUpdated

Description
When running the BellatrixMergeTransitionAcceptanceTest in Teku (with besu 22.1.3 current develop), the actual transition block is accepted by newPayload with a VALID response but then is INVALID when set as the chain head via forkChoiceUpdated. The same execution payload can't be both valid and invalid.
newPayload call:
{"jsonrpc":"2.0","method":"engine_newPayloadV1","params":[{"parentHash":"0xccd88cdf13f0970314d4a99daada6f6019faf5c91739417d6d20caceeec63288","feeRecipient":"0xfe3b557e8fb62b89f4916b721be55ceb828dbd73","stateRoot":"0x9b06a788c91be3a261a7a9b0d6866530edb4884c6f0f806c1af14957e3bff5e7","receiptsRoot":"0x56e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421","logsBloom":"0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000","prevRandao":"0x20687643f94afae7c01eae4198f4a32430789c21a37e604b80d60e21eec73a00","blockNumber":"0x65","gasLimit":"0x1ffffffffffbff","gasUsed":"0x0","timestamp":"0x624a3adb","extraData":"0x","baseFeePerGas":"0x1","blockHash":"0xf0aa7e775de96cb5e9093161c3be1e93bec2e22b4884612ed5d63c002150402e","transactions":[]}],"id":389}

{
"jsonrpc" : "2.0",
"id" : 389,
"result" : {
"status" : "VALID",
"latestValidHash" : "0xf0aa7e775de96cb5e9093161c3be1e93bec2e22b4884612ed5d63c002150402e",
"validationError" : null
}
}

Then forkChoiceUpdated call:
{"jsonrpc":"2.0","method":"engine_forkchoiceUpdatedV1","params":[{"headBlockHash":"0xf0aa7e775de96cb5e9093161c3be1e93bec2e22b4884612ed5d63c002150402e","safeBlockHash":"0xf0aa7e775de96cb5e9093161c3be1e93bec2e22b4884612ed5d63c002150402e","finalizedBlockHash":"0x0000000000000000000000000000000000000000000000000000000000000000"},{"timestamp":"0x624a3ae1","prevRandao":"0xad601acc23b20b2f23e775d23922f0096a7d184deb470ceea049ebc5f8f38103","suggestedFeeRecipient":"0xfe3b557e8fb62b89f4916b721be55ceb828dbd73"}],"id":396}

{
"jsonrpc" : "2.0",
"id" : 393,
"result" : {
"payloadStatus" : {
"status" : "INVALID",
"latestValidHash" : "0xf0aa7e775de96cb5e9093161c3be1e93bec2e22b4884612ed5d63c002150402e",
"validationError" : "new head timestamp not greater than parent"
},
"payloadId" : null
}
}

Note that the latestValidHash is set to the requested head block suggesting it is in fact valid.

Turns out forkChoice was the correct one. We also updated the logic around latestValidHash around the same time timestamp validation was made consistent across those calls:
https://github.com/hyperledger/besu/blob/277c3e44d21c384725fabfa5a5a9d72c73cd2435/ethereum/api/src/main/java/org/hyperledger/besu/ethereum/api/jsonrpc/internal/methods/engine/EngineNewPayload.java#L145-L152
Does this test in question maybe not update timestamps on next blocks it sent?

Yeah I just realised that when mining with a really low fixed difficulty, besu will create blocks in the future (up to a limit of something like 15 seconds).  That doesn't work too well with the merge because the terminal block then winds up being from after the start of the current slot and this check fails. Would have resolved itself if newPayload had returned INVALID as the beacon block would have been entirely rejected and a new merge block created but the inconsistent response meant things got stuck with an invalid block as head.

Yeah we put a check into Teku to delay accepting a terminal block until it's timestamp isn't in the future which solves that issue - I don't think besu is wrong to be mining "future" blocks and it is something that could be done by miners on mainnet (though fairly unlikely).
Was there a PR to fix the newPayload result recently? We were getting inconsistent results on the latest develop version as of 2 days ago and I don't see anything that has changed since.

Does this test in question maybe not update timestamps on next blocks it sent?

Because Teku gets a VALID response from newPayload, it rejects the INVALID response from fcu because a block can't go from being valid to invalid so we keep trying to build on that block and the system is then stuck.

I can confirm this is still a problem with the latest develop build:
besu/v22.4.0-RC2-dev-92de455a/linux-x86_64/openjdk-java-11
I get VALID from engine_newPayloadV1 but then INVALID from engine_forkChoiceUpdated for the same block.

It turns out I do still have the logs and looking more closely now I think this is actually the issue that Enrico found and fixed in teku where the payloadAttributes we're sending with fcu are invalid.  That's causing Besu to incorrectly tell us the block is invalid - Geth returns a JSON-RPC error in that case not INVALID.
{"jsonrpc":"2.0","method":"engine_newPayloadV1","params":[{"parentHash":"0x68051baceaacdeabb
c6d384cdaf320e44981b2ef39252f953744d0befe7866c7","feeRecipient":"0xfe3b557e8fb62b89f4916b721be55ceb828dbd73","stateRoot":"0xbb0c7d8f66efcecf9c677129818ea89bbf21ac97f20bbb89bf27fa212ccac132","receiptsRoot":"0x
56e81f171bcc55a6ff8345e692c0f86e5b48e01b996cadc001622fb5e363b421","logsBloom":"0x0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000","prevRandao":"0x77aae49450097
576268604d0fd4c08d2d72caee9048c4070873e1b9976ab2c8d","blockNumber":"0x65","gasLimit":"0x1ffffffffffbff","gasUsed":"0x0","timestamp":"0x624d0d3b","extraData":"0x","baseFeePerGas":"0x1","blockHa
sh":"0xac33ef90c62cdf7cc47808ad3803988ffdb1a05e43e91172e3780bb10993a6e2","transactions":[]}],"id":346}

{
"jsonrpc" : "2.0",
"id" : 346,
"result" : {
"status" : "VALID",
"latestValidHash" : "0xac33ef90c62cdf7cc47808ad3803988ffdb1a05e43e91172e3780bb10993a6e2",
"validationError" : null
}
}

{"jsonrpc":"2.0","method":"engine_forkchoiceUpdatedV1","params":[{"headBlockHash":"0xac33ef90c62cdf7cc47808ad3803988ffdb1a05e43e91172e3780bb10993a6e2","safeBlockHash":"0xac33ef90c62cdf7cc47808ad3803988ffdb1a05e43e91172e3780bb10993a6e2","finalizedBlockHash":"0x0000000000000000000000000000000000000000000000000000000000000000"},{"timestamp":"0x624d0d41","prevRandao":"0x89f0d85b01335f671851c45495ad7ec75cf94bf53a455b6ef5de14eb1f24695b","suggestedFeeRecipient":"0xfe3b557e8fb62b89f4916b721be55ceb828dbd73"}],"id":349}

{
"jsonrpc" : "2.0",
"id" : 349,
"result" : {
"payloadStatus" : {
"status" : "INVALID",
"latestValidHash" : "0xac33ef90c62cdf7cc47808ad3803988ffdb1a05e43e91172e3780bb10993a6e2",
"validationError" : "new head timestamp not greater than parent"
},
"payloadId" : null
}
}

Ah yes. This behavior is currently underspecified.  Which one is correct is currently being discussed.

Rough consensus indicates that this should return a JSON-RPC error instead of an INVALID result. Reasoning being there is nothing wrong with the block in this case, just the parameter.


pr-pull: pass --committer to pr-upload

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

@nandahkrishna

Review period will end on 2021-04-02 at 21:47:15 UTC.

Review period skipped due to critical label.

Yes, this should fix it.

Thanks @dawidd6!


Respect Sidebar size preferences from General preferences

I believe that this is not working on Big Sur with 6.0d5.
Adjusting the system preferences sidebar icon size setting does not update in NNW, even after a relaunch.  The fixed size appears to be close to "Medium" when compared to a Finder sidebar.
@vincode-io : should I file a new issue?

I believe that this is not working on Big Sur with 6.0d5.
Adjusting the system preferences sidebar icon size setting does not update in NNW, even after a relaunch.  The fixed size appears to be close to "Medium" when compared to a Finder sidebar.
@vincode-io : should I file a new issue?

Yeah, let's open a new issue to track this.

Yeah, let's open a new issue to track this.


config for mod blacklist does not work

loot_modid_blacklist = [] - List of modids whose loot tables shouldn't be converted (in the format of ["modid", "other_modid"])
I have tried
loot_modid_blacklist = ["apotheosis"]
and
loot_modid_blacklist = ["modid:apotheosis"]
yet it does not work. my server continues to load empty loot chests because apotheosis was removed.
How do i remove loot tables or ignore mods?
Clearly the configuration file is either not working, or EXTREMELY unintuitive and no examples are given.
Can you please provide at least one example for writing your configurations?
Especially when we can't even remove this mod without removing every single loot chest in our servers.

Upon further analysis, This issue only happens in chunks that were generated before apotheosis was removed. Newly generated chunks don't seem to have the issue.
So it's not such a critical issue as i thought before, but i would still like to suggest a "update / refresh" feature.
Perhaps a command that re-selects loot tables for generated chests. This would fix my problem.


Enable parallel computing

take advantage of processors with multiple cores using parallel computing.
a lot of checks in the algorithms are independent and can be parallelized.

during map preparation time:

compute angle representations and distances for each extremity in parallel
check for visible nodes from the each extremity node in parallel (with the optimisation that it is sufficient to check in one direction)

during query time:

check for visible nodes from the start and goal nodes in parallel

Optional parallelism. Off by default.
additional int parameter for the functions map.prepare()
meaning of -1: Max Nr workers according to the available amount of cores
additional bool parameter for the functions map.find_shortest_path()

Consider the GIL limitations: https://superfastpython.com/multiprocessing-vs-gil-in-python/


Default "split-thread-core-pool-size" to 4

See spring-cloud/spring-cloud-dataflow#2279 for more details.

We had a discussion at inception of CTR because, I had defaulted it to 5 (IIRC) when we working on the initial release. It was decided to stay at 1 because :

it was a sensible default.
It prevents users from reporting the issue as a scaling problem, i.e. "I ran 6 and now it fails for some odd reason".  And I will repeat the question I was asked when I defaulted to 5.  Why 5?  in this case why 4?

@mminella: Do you have any recommendations?

Let's discuss.    But the original reason for using 1 was not to choose an arbitrary number of threads.   By selecting the default of 1 users could tune it to their needs.   We can set it to 4, but we will see this issue reported again when the users have 5 concurrent splits.


Delete files task locking sources directory of build

Environment

Server - TFS on-premises

If using TFS on-premises, provide the version:
TFS 2018.3

Agent - Private

Machine OS: Windows Server 2012 R2
Agent.Version: 2.136.1
Agent.OSVersion: 6.3.9600

Task Name - DeleteFiles v1.1.4

Config:

Source folder - $(Agent.BuildDirectory)
Contents - **

Issue Description
I am currently working to upgrade an on-prem installation of TFS from 2015 to 2018. As part of this process, build agents needed to be upgraded. For nearly all our build definitions that exist, there is a delete step included at the end of every build to remove build resources from the agent when finished.
The issue is that all existing builds fail at this final step due to a change in behavior in the Delete Files task. The actual error and behavior is identical to #3628. Both solutions provided there do solve the issue, however, this is at the cost of editing all build definitions that include the aforementioned delete step, which is far from ideal for us.
Judging from logs, the Delete Files process locks the agent\_work\##\s directory as its working directory. This has changed from previous versions, where the Delete Files process would lock  agent\tasks\Deletefiles\<version> as its working directory.
After digging through source code, I believe the process's working directory is being set to a default as part of the agent's NodeHandler logic which sets it to System.DefaultWorkingDirectory since no working directory is specified as part of the task.
I would like to know if there is any way to specify the working directory of all Delete Files tasks, as this should solve my issue. I do see a line in the Delete task logs for loading INPUT_WORKINGDIRECTORY, though I've not found any resources regarding this and I'm not sure whether this line relates to the agent's working directory or the process's. If this isn't possible, are there any alternatives to editing individual build definitions while maintaining existing behavior (resource removal after build)?
Task logs
DeleteFiles task logs:
2019-01-25T20:39:32.7730660Z ##[debug]agent.workFolder=E:\TFS\Agents\2018\agent06_2\_work
2019-01-25T20:39:32.7879789Z ##[debug]loading inputs and endpoints
2019-01-25T20:39:32.7881377Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_SYSTEMVSSCONNECTION_ACCESSTOKEN
2019-01-25T20:39:32.7883682Z ##[debug]loading ENDPOINT_AUTH_SCHEME_SYSTEMVSSCONNECTION
2019-01-25T20:39:32.7885543Z ##[debug]loading ENDPOINT_AUTH_SYSTEMVSSCONNECTION
2019-01-25T20:39:32.7886280Z ##[debug]loading INPUT_CONTENTS
2019-01-25T20:39:32.7886480Z ##[debug]loading INPUT_SOURCEFOLDER
2019-01-25T20:39:32.7886685Z ##[debug]loading INPUT_WORKINGDIRECTORY
2019-01-25T20:39:32.7887070Z ##[debug]loaded 6
2019-01-25T20:39:32.7887552Z ##[debug]check path : E:\TFS\Agents\2018\agent06_2\_work\_tasks\DeleteFiles_b7e8b412-0437-4065-9371-edc5881de25b\1.1.4\task.json
2019-01-25T20:39:32.7887997Z ##[debug]set resource file to: E:\TFS\Agents\2018\agent06_2\_work\_tasks\DeleteFiles_b7e8b412-0437-4065-9371-edc5881de25b\1.1.4\task.json
2019-01-25T20:39:32.7888272Z ##[debug]system.culture=en-US
2019-01-25T20:39:32.7888487Z ##[debug]Contents=**
2019-01-25T20:39:32.7888681Z ##[debug]SourceFolder=E:\TFS\Agents\2018\agent06_2\_work\3
2019-01-25T20:39:32.7888883Z ##[debug]BuildCleanup=null
2019-01-25T20:39:32.7889076Z ##[debug]patterns: E:\TFS\Agents\2018\agent06_2\_work\3\**
2019-01-25T20:39:32.7889290Z ##[debug]find findPath=E:\TFS\Agents\2018\agent06_2\_work\3, options=undefined
...
...
...
2019-01-25T20:39:33.0668468Z ##[debug]rm -rf E:\TFS\Agents\2018\agent06_2\_work\3\s
2019-01-25T20:39:33.0668679Z ##[debug]removing directory
2019-01-25T20:39:33.0668852Z rm: could not remove directory (code EBUSY): E:\TFS\Agents\2018\agent06_2\_work\3\s
2019-01-25T20:39:33.0668993Z
2019-01-25T20:39:33.0669319Z ##[error]Error: Failed rmRF: rm: could not remove directory (code EBUSY): E:\TFS\Agents\2018\agent06_2\_work\3\s
2019-01-25T20:39:33.0669701Z ##[debug]Processed: ##vso[task.issue type=error;]Error: Failed rmRF: rm: could not remove directory (code EBUSY): E:\TFS\Agents\2018\agent06_2\_work\3\s
2019-01-25T20:39:33.0669867Z
2019-01-25T20:39:33.0670077Z ##[debug]rm -rf E:\TFS\Agents\2018\agent06_2\_work\3\a
2019-01-25T20:39:33.0670287Z ##[debug]removing directory
2019-01-25T20:39:33.0670493Z ##[debug]task result: Failed
2019-01-25T20:39:33.0670737Z ##[error]Couldn't delete one or more files
2019-01-25T20:39:33.0670925Z ##[debug]Processed: ##vso[task.issue type=error;]Couldn't delete one or more files
2019-01-25T20:39:33.0685823Z ##[debug]Processed: ##vso[task.complete result=Failed;]Couldn't delete one or more files

Worker logs from agent\_diag of the 2018 agent:
INFO TaskManager] Loading task definition 'E:\TFS\Agents\2018\agent06_2\_work\_tasks\DeleteFiles_b7e8b412-0437-4065-9371-edc5881de25b\1.1.4\task.json'.
INFO TaskRunner] The original input is a rooted path, return absolute path: E:\TFS\Agents\2018\agent06_2\_work\4
INFO ExtensionManager] Getting extensions for interface: 'Microsoft.VisualStudio.Services.Agent.Worker.IJobExtension'
INFO BuildJobExtension] Build JobExtension resolving path use source provide:
INFO BuildJobExtension] The Default Path Root of Build JobExtension is build.sourcesDirectory: E:\TFS\Agents\2018\agent06_2\_work\4\s
INFO BuildJobExtension] After prefix Default Path Root provide by JobExtension: E:\TFS\Agents\2018\agent06_2\_work\4\s
INFO BuildJobExtension] Return absolute path after prefix DefaultPathRoot: E:\TFS\Agents\2018\agent06_2\_work\4\s
INFO TaskRunner] Build JobExtension resolved a rooted path:: E:\TFS\Agents\2018\agent06_2\_work\4\s
INFO NodeHandler] Inspect node_modules folder, make sure vsts-task-lib doesn't overwrite String.startsWith/endsWith.
INFO NodeHandler] This task has already been scanned and corrected, no more operation needed.
INFO HostContext] Well known directory 'Bin': 'E:\TFS\Agents\2018\agent06_2\bin'
INFO HostContext] Well known directory 'Root': 'E:\TFS\Agents\2018\agent06_2'
INFO HostContext] Well known directory 'Externals': 'E:\TFS\Agents\2018\agent06_2\externals'
INFO ProcessInvokerWrapper] Starting process:
INFO ProcessInvokerWrapper]   File name: 'E:\TFS\Agents\2018\agent06_2\externals\node\bin\node.exe'
INFO ProcessInvokerWrapper]   Arguments: '"E:\TFS\Agents\2018\agent06_2\_work\_tasks\DeleteFiles_b7e8b412-0437-4065-9371-edc5881de25b\1.1.4\deletefiles.js"'
INFO ProcessInvokerWrapper]   Working directory: 'E:\TFS\Agents\2018\agent06_2\_work\4\s'
INFO ProcessInvokerWrapper]   Require exit code zero: 'True'
INFO ProcessInvokerWrapper]   Encoding web name: utf-8 ; code page: '65001'
INFO ProcessInvokerWrapper]   Force kill process on cancellation: 'False'
INFO ProcessInvokerWrapper]   Lines to send through STDIN: '0'
INFO ProcessInvokerWrapper] Process started with process id 968, waiting for process exit.

Worker logs from agent\_diag of the 2015 agent:
10:16:11.131976 JobExtensionManager.GetDefaultPathRoot
10:16:11.131976 JobExtensionManager.GetLocalPath
10:16:11.131976 JobRunner.Run - Running task DeleteFiles.
10:16:11.131976 BaseLogger.LogConsoleMessage(scope.JobId = 8b63bd21-1eb8-4eb3-9850-fed1d1886bd7, message = ##[section]Starting task: Clean Build Files)
10:16:11.134977 JobExtensionManager.OnBeforeExecuteTask
10:16:11.134977 JobExtensionManager.OnExecuteTask
10:16:11.145977 BaseLogger.LogStatus(scope.JobId = 8b63bd21-1eb8-4eb3-9850-fed1d1886bd7, scope.TimelineRecordId = 38bce15d-090c-4e4f-8978-b0e6a4005049, record.Name = Clean Build Files)
10:16:11.145977 BaseLogger.LogConsoleMessage(scope.JobId = 8b63bd21-1eb8-4eb3-9850-fed1d1886bd7, message = Set workingFolder to default: E:\TFS\Agents\agent06_01\tasks\DeleteFiles\1.0.7)
10:16:11.147976 Entering ProcessWrapper.CreateStartInfo
10:16:11.147976    workingFolder = E:\TFS\Agents\agent06_01\tasks\DeleteFiles\1.0.7
10:16:11.147976    filename = E:\TFS\AGENTS\AGENT06_01\AGENT\WORKER\HANDLERS\NODE\node.exe
10:16:11.147976    arguments = "E:\TFS\Agents\agent06_01\tasks\DeleteFiles\1.0.7\deletefiles.js"
10:16:11.147976 Leaving ProcessWrapper.CreateStartInfo
10:16:11.147976 ProcessWrapper.Start - enter
10:16:11.147976 ProcessWrapper.Start - starting process
10:16:13.884115 ProcessWrapper.Start - process started
10:16:13.884115 ProcessWrapper.Start - leave
10:16:16.198231 Process logging event with task handler.

Error logs
From DeleteFiles task:
[error]Error: Failed rmRF: rm: could not remove directory (code EBUSY): E:\TFS\Agents\2018\agent06_2\_work\3\s

As you've noticed, the build agent now runs tasks from within the source folder, rather from within their own folders. This was done to provide a measure of consistency to each run. Unfortunately, this does seem to break this use case.
One workaround is to instead delete $(Build.StagingDirectory), $(Build.BinariesDirectory), $(Build.SourcesDirectory), and $(Common.TestResultsDirectory) individually. This will cover removing the contents of those folders but not attempt to delete the folders themselves.
I'll defer to @vijayma or @azureDaveOps on whether or not we should revert the working folder change, or add a custom working folder setting to the Delete task.

Exactly this is what I did as a workaround, to delete the directories individually.
In this case only the empty source folder remains after the deletion. It works and acceptable, but hmmm not that nice as it was in TFS 2015 ;-)

Why closing this? This is an open issue and the workaround only deletes folder contents, not the actual folders.


Create 0000-01-02-djibo648.md

informatique

informatique

https://docs.github.com/articles/github-community-guidelines

https://docs.github.com/articles/github-community-guidelines


Cannot read property 'Qa' of null from zone.js

I am getting Cannot read property 'Qa' of null from zone.js when added this lib.
any idea of cause?
regards
Sean

I have this to still got no idea why this pops up

same issue here

source code: https://github.com/born2net/studioDashboard/blob/hot-reload/src/comps/app1/dashboard/StationsMap.ts
error:

@born2net thanks! Seems like it's a bug in the latest google maps release.
https://code.google.com/p/gmaps-api-issues/issues/detail?id=7991
Can you try to configure the 3.18 version and test it again? thanks

yes will do...

@born2net that's really cool Sean! Let me know if you need help.

I m using Webpack...How can i configure to use the 3.18 version?

@imichelakos see: https://github.com/SebastianM/angular2-google-maps/issues/177#issuecomment-190868795

I discovered that this must be a Zone.js bug. When using the plain Google Maps API and adding Zone.js to the page, the described error occurs. Demo: http://plnkr.co/edit/hbjkks?p=preview
I will create an Issue for Zone.js shortly.

We track this in #355


Bugfix/wrong lable

if use doc.index() to find text position, when this entity is appeared before this words, we find wrong position.

@Alan-buaa Please tell me some specific case.


Implement Jaeger Propagator

@fhwang showed interest in this on gitter. If you comment here, I can assign this to you.

@fhwang showed interest in this on gitter. If you comment here, I can assign this to you.

🙋

🙋


Cluster improvement

Implemented in multiple commit most of the improvement suggested in the "northbound api" PR

bytebuffer is a standard class (well tested) and wrap the byte[] an provide the needed methods for array.
I think you are prematurely optimizing. If we reimplement a class from the std lib I think we should have strong rationales.

Array.hashcode() and Array.equals is standard class too (well tested too).

I integrated this changes on master
(commit 37496e72fec9f38e5bcd149433ab217fb34299c2,  0e8518139beddb8dc5b2aea81f6439df7ca95435, ae582948f9800c99e36ec3355ba34d1b5ad792ff, 91b115d5f14914b32ac9bc1f220b8723253bf344,  61b9a9c66ceeddd2f23ffc35f5ace9b9fa46844c )
except for bytebuffer.wrap,  removing env var, unified lh/lp and  slh/slp.

Sorry, not clear if this is actually merged into master, seems some conflicts remain. Thanks.

The comment above seems clear.
All is integrated excepted : bytebuffer.wrap, removing env var, unified lh/lp and slh/slp for reason exposed above.
I think removing env var, unified lh/lp and slh/slp could be done in another PR. (changing this for all Leshan command line tools) I create  issues about this #208, #209
I close this issue.
Please feel free to reopen it or comment if it's still not clear


Support initial stx balance

Currently, the VM is started with 0 balance for all accounts. (I think)
For tests, it would be beneficial if the initial balance could be set.

Now that clarity-cli supports setting initial balances via a json file (see https://github.com/blockstack/stacks-blockchain/pull/1662), the sdk should allow you to either pass your own JSON file, or provide an API to generate that JSON file when calling the initialize target on clarity-cli.
JSON file should look like:
[
  {
    "principal": "S1G2081040G2081040G2081040G208105NK8PE5",
    "amount": 1000
  },
  {
    "principal": "S1G2081040G2081040G2081040G208105NK8PE5.names",
    "amount": 2000
  }
]

Usage:
clarity-cli balances.json path-to-db

What should the API look like?
I suggest that each provider (NativeClarityBinProvider, JsonRpcProvider, ..) handles their own initialization process. Users have then to register their provider manually.


[supereditor_robot] placeCaretInParagraph from the second time doesn't place the caret

When using the SuperEditorRobot.placeCaretInParagraph the second time, it doesn't place a new caret.
Consider this modified test case in supereditor_robot_test.dart
testWidgets("enters text with hardware keyboard", (tester) async {
    // Configure and render a document.
    await tester //
        .createDocument()
        .withSingleEmptyParagraph()
        .forDesktop()
        .autoFocus(true)
        .pump();

    // Tap to place the caret in the first paragraph.
    await tester.placeCaretInParagraph("1", 0);
    // await tester.placeCaretInParagraph("1", 0);

    // Type some text by simulating hardware keyboard key presses.
    await tester.typeKeyboardText("Hello, world!");

    await tester.placeCaretInParagraph("1", 13);

    await tester.typeKeyboardText("ABC");

    // Verify that SuperEditor displays the text we typed.
    expect(SuperEditorInspector.findTextInParagraph("1").text, "Hello, world!ABC");
  });

Result
It fails to insert the ABC
Test log

Logs
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following TestFailure was thrown running a test:
Expected: 'Hello, world!ABC'
  Actual: 'Hello, world!'
   Which: is different. Both strings start the same, but the actual value is missing the following
trailing characters: ABC

When the exception was thrown, this was the stack:
#4      main.<anonymous closure>.<anonymous closure> (file:///Users/thang/Workspace/super_editor/super_editor/test/super_editor/supereditor_robot_test.dart:139:7)
<asynchronous suspension>
<asynchronous suspension>
(elided one frame from package:stack_trace)

This was caught by the test expectation on the following line:
  file:///Users/thang/Workspace/super_editor/super_editor/test/super_editor/supereditor_robot_test.dart line 139
The test description was:
  enters text with hardware keyboard
════════════════════════════════════════════════════════════════════════════════════════════════════
Test failed. See exception logs above.
The test description was: enters text with hardware keyboard

✖ SuperEditor robot enters text with  hardware keyboard
Exited (1)

FYI, when placing the caret 3 times in a row, e.g
await tester.placeCaretInParagraph("1", 0);
await tester.placeCaretInParagraph("1", 0);
await tester.placeCaretInParagraph("1", 0);

it throws an error
══╡ EXCEPTION CAUGHT BY SERVICES LIBRARY ╞══════════════════════════════════════════════════════════
The following RangeError was thrown while processing the key message handler:
RangeError (end): Invalid value: Only valid value is 0: -1

This is instead a not typical use case, so I'm just gonna report, but it doesn't necessarily need to be resolved.

I am working on it


Fix issue where requests with a query string would show up under the wrong view with -q.

Certain requests show up under the requests view even when --no-query-string is used.

This has been fixed. Will be deployed in the upcoming version.


Replace array indexOf checks with includes

Closes #898
Changes:
Replaced indexOf calls to check presence or absence with includes (with negation operator when checking absence).
Decisions:

There are some indexOf calls which use the index value to both check presence and use it to remove some elements at that position. They can be potentially refactored with separate calls to includes and indexOf, but I decided not to do that as indexOf alone can serve both the purposes here.

In testcases, replaced assert.equal + indexOf check with -1 by assert.ok + includes. This will probably show less information during error, as the actual index value won't be printed. That seems like a small price to pay.

Observations:

Most of the tests don't seem to fail if I change the presence or absence logic (includes or !includes) with the opposite one. I often did this mistake when working on this initially, but none of the test cases failed as they should have.
Maybe we should add some more tests around that?

Sorry for the delay. These changes look great!
If you are having trouble running the tests on windows, could you please open another issue so we can get that fixed separately. Thanks!


Form\Collection: allow remove all objects

There is no way to remove all objects from collection at this time. According to the logic I need to pass an empty array to Zend\Form\Element\Collection::populateValues. But empty arrays are ignored https://github.com/zendframework/zf2/blob/master/library/Zend/Form/Element/Collection.php#L185
The next possible upcoming issue is that there is not way to create the html element in view script that would generate an empty array. May be some type conversion should be implemented to handle this.

fixed with https://github.com/zendframework/zend-form/pull/14


Zappa never finds domain name in route 53 and will always delete api gateway domain when calling certify again

Description
Zappa is not properly checking if there is an existing route 53 record for the domain. Record names end with a period so that has to be stripped when comparing to the domain name.
I've also removed the deletion of the api gateway domain when no route 53 record is found because deletion takes several minutes to take effect and you'll get an error trying to create another one right away (I've run into this problem multiple times). A better way would be to use the update_domain_name method in the api gateway client.

Coverage increased (+0.04%) to 78.762% when pulling 3bb68a66c10e98efdbf49a8489d4422d86569f69 on kwellman:get_domain_name into b5b22fe2b5b0fee531ed0e49568c6dcc8427d83c on Miserlou:master.

Can you elaborate a bit more about why you took out the delete_domain_name call? That was added for a reason. I'd certainly like to see that code replaced by a faster/smarter update call, but I don't just want to remove functional code for a performance increase.
I also think this section of code needs to be tested.

I removed the delete_domain_name because I would get this error when that code is executed:
An error occurred (BadRequestException) when calling the CreateDomainName operation: The domain name you provided is already associated with an existing CloudFront distribution.

It seems like the domain was being deleted, but immediately trying to recreate it (as Zappa currently does) would cause this exception. After waiting a few minutes it would work when running Zappa again. Based on posts like this https://forums.aws.amazon.com/thread.jspa?threadID=222468, it seems like it's expected behaviour and therefore the logic of deleting the api gateway domain to recreate it doesn't work unless a delay is added (but I wouldn't know what amount to use).
With the changes in the pull request, you would have to delete the api gateway manually if you ever get into a situation where the api gateway wasn't created correctly. I prefer this for now because:

Currently the exception above occurs
The method get_domain_name is potentially very dangerous if called when your site is live. As it is now, my api gateway domain gets deleted when certify is called again and this brings down my site. This is because the method also fails to find the route53 records (because of not removing the trailing '.'). Of course that is fixed easily, but I don't think that situation should ever be close to possible. Maybe I'm being too neurotic that bugs could be introduced again.
A method named get_domain_name shouldn't delete anything, but that's just personal style.

Of course I could be wrong and misunderstand something, but the changes work for me. I agree that tests need to be made. I could work on that when I have the time.

Okay, I've finally merged this but I'm still finding the behavior to be funky. One particularly annoying piece is that it seems like there is no way to check which server certificates are active in the IAM panel.


API: start, stop, kill server

Hi,
I was trying to use you API to start/stop/kill server but I have problem when I'm trying to do this.
I created new user that is not admin, and I granted him only to start/stop/kill and send command to minecraft server. When I'm calling /auth/login/ endpoint with json:
{
    "username": "the_user",
    "password": "some_password"
}

I'm receiving following:
{
    "session": "eyJhbGciOiJFUzI1NiIsInR5cCI6IkpXVCJ9.eyJhdWQiOiJzZXNzaW9uIiwiZXhwIjoxNjA0MTc1NDU4LCJpYXQiOjE2MDQxNzE4NTgsInN1YiI6IjEiLCJwdWZmZXJwYW5lbCI6e319.ZLLIkd1uL5rG8pzwQo56ommPvLBgmhJ2BzgOpiUU-bGQ8KD7_qEisN1zb-9ICVvyi1fjgJx0qCEKul_mF8VHZg",
    "scopes": [
        "servers.admin",
        "servers.create",
        "nodes.view",
        "nodes.deploy",
        "nodes.edit",
        "templates.view",
        "users.view",
        "users.edit",
        "servers.view"
    ]
}

Then I trying to call /daemon/server/e6bdac4a/start
I'm setting puffer_auth or Authorization header (I tried many possibilities) but I always receive error:
{
    "error": {
        "msg": "missing scope",
        "code": "ErrMissingScope",
        "metadata": {
            "scope": "servers.start"
        }
    }
}

but when I'm connecting to Websocket /proxy/daemon/socket/e6bdac4a with puffer_auth cookie, and I'm sending command like pufferpanel frontend I can start/stop/kill server without any problem.
It is hard to find any api doc, so I was reading pufferpanel code to achieve anything (maybe this is a problem).
Am I missing something or am I doing something wrong?

The token you get for login is not applicable to the /daemon endpoint due to differences. You should use /proxy anyways because it handles stuff for you.


Roll Engine from 073263e39d1d to 99cc50dfffab (35 revisions)

Roll Engine from 073263e39d1d to 99cc50dfffab (35 revisions)
https://github.com/flutter/engine/compare/073263e39d1d...99cc50dfffab
2020-10-29<EMAIL_ADDRESS>[ios] Surface factory holds the canonical reference to the external view embedder (flutter/engine#22206)
2020-10-29<EMAIL_ADDRESS>Update painting.dart (flutter/engine#22195)
2020-10-29<EMAIL_ADDRESS>Add "input shield" to capture pointer input for reinjection (flutter/engine#22067)
2020-10-29<EMAIL_ADDRESS>Fix FlTextInputPlugin tear down (flutter/engine#22007)
2020-10-29<EMAIL_ADDRESS>Fix typos in FlValue docs (flutter/engine#21875)
2020-10-29<EMAIL_ADDRESS>Refactor make_mock_engine into fl_test (flutter/engine#21585)
2020-10-29<EMAIL_ADDRESS>Use preTranslate when applying offset to matrix (flutter/engine#21648)
2020-10-29<EMAIL_ADDRESS>Add a proc table version of embedder API (flutter/engine#21813)
2020-10-29<EMAIL_ADDRESS>Set strut font to Roboto if the given font hasn't been registered (flutter/engine#22129)
2020-10-29<EMAIL_ADDRESS>bring back build_test to ensure we validate licenses (flutter/engine#22201)
2020-10-29<EMAIL_ADDRESS>Fix for firefox custom clipping (flutter/engine#22182)
2020-10-29<EMAIL_ADDRESS>[profiling] Handle thread_info to account for killed threads (flutter/engine#22170)
2020-10-29<EMAIL_ADDRESS>Standardize macro names for dartdoc macros (flutter/engine#22180)
2020-10-29<EMAIL_ADDRESS>Reland [ios] Refactor IOSSurface factory and unify surface creation (flutter/engine#22016)
2020-10-29<EMAIL_ADDRESS>[web] Fix scroll wheel line delta on Firefox. (flutter/engine#21928)
2020-10-29<EMAIL_ADDRESS>[web] Fix transform not invalidating path bounds causing debugValidate failure (flutter/engine#22172)
2020-10-29<EMAIL_ADDRESS>Migrate runZoned to runZonedGuarded (flutter/engine#22198)
2020-10-29<EMAIL_ADDRESS>[web] Assign default natural width/height for svgs that report 0,0 on firefox and ie11 (flutter/engine#22184)
2020-10-29<EMAIL_ADDRESS>Invalidate the cached SkParagraph font collection when a new font manager is installed (flutter/engine#22157)
2020-10-29<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from PY5hNI-wY... to Usec4YBzR... (flutter/engine#22197)
2020-10-29<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from xHjtLQPQ5... to ICK_JlnKJ... (flutter/engine#22188)
2020-10-29<EMAIL_ADDRESS>Roll Skia from 5567a6091ceb to f548a028ce70 (7 revisions) (flutter/engine#22155)
2020-10-29<EMAIL_ADDRESS>[web] Clean up unused previousStyle logic (flutter/engine#22150)
2020-10-29<EMAIL_ADDRESS>Fix possible use of std::moved value in Rasterizer (flutter/engine#22125)
2020-10-29<EMAIL_ADDRESS>Add plumbing for command line arguments on Windows (flutter/engine#22094)
2020-10-29<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from UKgKCjxSA... to PY5hNI-wY... (flutter/engine#22175)
2020-10-29<EMAIL_ADDRESS>Includes roles for links, checkboxes, and radio buttons in semantics (flutter/engine#22061)
2020-10-29<EMAIL_ADDRESS>Allow parse_and_send to use access tokens (flutter/engine#22019)
2020-10-28<EMAIL_ADDRESS>Smooth window resizing on macOS (flutter/engine#21525)
2020-10-28<EMAIL_ADDRESS>Update constraint to latest integration test (flutter/engine#22169)
2020-10-28<EMAIL_ADDRESS>Manual Dart SDK roll 6e015bd9cddb to 9c6e76468ca4 (6 revisions (flutter/engine#22153)
2020-10-27<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from GKPwGj1Ux... to xHjtLQPQ5... (flutter/engine#22151)
2020-10-27<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from XYN02FThN... to UKgKCjxSA... (flutter/engine#22147)
2020-10-27<EMAIL_ADDRESS>Roll Skia from 7737a5bd2510 to 5567a6091ceb (8 revisions) (flutter/engine#22146)
2020-10-27<EMAIL_ADDRESS>Add dart-lang/sdk's new package:clock dependency (flutter/engine#22142)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from XYN02FThN03P to Usec4YBzRUKB
fuchsia/sdk/core/mac-amd64 from GKPwGj1UxJOh to ICK_JlnKJ9AX
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
...

Commit queue failed; closing this roll.

Reopneing, ignoring cirrus flakes.

Merging, Windows tool_tests passed: https://ci.chromium.org/p/flutter/builders/try/Windows tool_tests/4927?
All other LUCI passed in previous run before this was initially closed.


Support for multipart field in HTTP output client

With multipart support, we can divide a single batch of message into multiple HTTP request parts and send the POST request.
No need of having multiple messages to create a multipart request. Each part of the request will have its own headers and the fields of multipart array support interpolation
A sample benthos config for the same: set the parts in the bloblang pipeline and refer those in data of multipart
pipeline:
  processors:
    - bloblang: |
        meta AttachmentName = filename
        root.part1 = "ABC"
        root.part2 = "XYZ"

output:
  http_client:
    headers:
      Content-Type: multipart/form-data
    url: ""
    verb: POST
    multipart:
      - content_disposition: form-data; name="bin"; filename='${! meta("AttachmentName")
        content_type: application/bin
        body: ${! json("part2") }
      - content_disposition: form-data; name="data"
        content_type: application/json; charset=utf-8
        body: ${! json("part1") }

Note: this is only valid for HTTP output client

Ace, thanks @YachikaRalhan this looks great.


Add value in the documentation file about SonarCloud AB#225

Other information

Step 23 - specify which workflow needs to be manually run.

Step 23 - specify which workflow needs to be manually run.

@kulov  updated

@kulov  updated


Template does not build – but just warnings, no errors –

The only warning what could lead to the reason is the following one.

Does anyone manage to build this current version?

I am not sure whether GitHub actions counts as "anyone"?
See https://github.com/latextemplates/scientific-thesis-template/runs/1423965865 and https://latextemplates.github.io/scientific-thesis-template/main-minted-german.pdf for the compiled version.
Without knowing your concrete build setup, I cannot really help. But this is not what you asked for ^^.
I know that someone had issues with Mac OS X and this template. We managed to get docker running on his machine and executing the docker command. With someone with more patience - and using Mac OS - we could analzye the errors.
The warning above is not releveant at all. I would use "pdflatex main-german.tex" and see the line where it stops. pdflatex has the feature that the compiler stops at the first error. This is not well-supported at UIs, but very well supported at command line prompts.
It compiled one week ago "on my machine".
Today, on another machine:
With lualatex, I get ! error:  (type 1): cannot find encoding file 'ntx-ot1-tlf.enc' for reading. Unfixable for me.
With pdflatex, I get ! TeX capacity exceeded, sorry [main memory size=3000000].. I added a new FAQ.
Then, I get the lualatex error, too.
I submitted https://github.com/MiKTeX/miktex/issues/663.
Thereby, I found a work-around: Download following three files and place them into the folder of the .tex file

http://mirrors.ctan.org/fonts/newtx/enc/ntx-ec-tlf-pc.enc
http://mirrors.ctan.org/fonts/newtx/enc/ntx-ec-tlf.enc
http://mirrors.ctan.org/fonts/newtx/enc/ntx-ot1-tlf.enc

Building from current master on my arch linux I can reproduce the error message from the original report, but I also get a proper output for the template.
pdflatex main-german.tex also surfaces the reported warning, but also completes.
Additionally there's the following warnings:

"Command \@parboxrestore has changed."
"csquotes should be loaded after fvextra"
filecontents is obsolete
eq:test multiply defined
eq:test@cref multiply defined
footnotehyper warns about: "The footnote environment will not be fully functional, sorry."
pgfplots wants to be updated to compat 1.17 from 1.14
Use of \deftripstyle should be replaced with \deftriplepagestyle
center environment in figure
4 nag warnings and missing font shapes.

These are consistent with the warnings from running make. It seems like this is a MiKTeX issue?

Note that an issue to MikTeX seems to have been submitted in parallel to my (lengthy) report at https://github.com/MiKTeX/miktex-packaging/issues/209. It is fixed. It should build.
I also updated the FAQ http://latextemplates.github.io/scientific-thesis-template/#q-i-get-font-latinmodernmath-regular-not-found-what-can-i-do
Finally the hint to ask the MikTeX maintainer to improve auto-installation at https://github.com/MiKTeX/miktex/issues/127 (or submit a MiKTeX fix via a PR).

@Vogel612 Thank you for listing the warngings. I added them to https://github.com/latextemplates/scientific-thesis-template/issues/135.
I am still trying to get the template-generator finished. However, other projects require my energy, too.

scrwfile is marked as important to avoid "No room for a new \write". To support that usecase (although preliminary testing on my machine reveals it's not strictly necessary to have scrwfile) another option might be to use morewrites, which seems to be significantly newer and was not available when srcwfile was originally added.

I updated my latex packages with MiKTeX Console today on Window and now the template doesn't compile anymore. My CI pipeline, which runs on linux, still compiles the template.

what is the output of lualatex main? It stops at the first  error. Please show us this first error! Replace "lualatex" by "pdflatex" (if you use pdflatex) and "main" by the main document filename.

The error was TeX capacity exceeded, sorry [main memory size=3000000].
I fixed it by increasing the memory limit in MiKTeX to 5000000.

Thank you for the update. We also have that in the FAQ: http://latextemplates.github.io/scientific-thesis-template/
Not sure what changed in LaTeX's packages.
I am working at the LaTeX template generator, but I do not know if it helps if some features are omitted as default.

The FAQ page does not redirect to HTTPS/is HTTPS only.


Wanchain wallet password

Hi, i unwittingly sent my wan to the second address on my wallet which has the same name. I can see the amount but cannot access because I do not know the password. Can you please let me know if there is a way to retrieve my Wan? I have saved my keystore file.
The wan were sent by Bianance to this address 0x0B0c64d6c677f2A4908b6c2e07b97B333f8d6960
While on my 1password that I use to save my passwords II see that the username is saved as 0x022c038d19a711a9b69f1762f688091f9f0e0402242087aec4ec8dfb76e896ca83022a08de317b3d932f5b268da46b3f152fd444677afa0eef37d87481275a948e5a
which is the private address of the account which has 0 wan.
I am sending you the screenshot. Kindly help me through this very difficult time. I would appreciate that. Thanks.
https://imgur.com/60mdYpm

I saw you had transfer out your wan successfully, congratulations

Thanks a lot. I took suggestions from people like you and it worked.

Glad to have invested in Wanchain!
Good luck


More units

Is it possible to display the time as x hours y minutes ago .i.e using both hours and minutes as a single unit?

I have the same question, but the issue was closed after 2 hours. @FarooqAR did you end up finding an answer? I see another issue (https://github.com/nmn/react-timeago/issues/115#issuecomment-453783647) suggesting a custom formatter, so maybe you went with that?

@butters16 It's been almost 4 years so I don't remember, but seems like I would've used a custom formatter.


www/nginx: Reverse sorting of logs

When analysing logs, typically the most recent entries are of interest. Therefore the output of log display through the UI should be changed to display newest first.

@fabianfrz Please review

Retiring PR in favour of #2165

Retiring PR in favour of #2165

@8191 PR retired but still open? I’m not the maintainer and reviewing/testing #2165 is out of my scope due to volume.


odo login overwrites the current active project/namespace

/kind bug
What versions of software are you using?
Operating System:
MacosX
Output of odo version:
odo v2.0.4 (ffb7d79aa)
How did you run odo exactly?
odo login
odo project create my-project
odo project get
odo login
odo project get
Actual behavior
The outcome of the last odo project get is the first project listed by odo project list command.
Expected behavior
The outcome of the last odo project get should be the active project (the project I just created) ie my-project
Any logs, error output, etc?

Marking closed since the PR is merged.
/close


Add WriteProgress and OpenProgress methods

Hi!

According to build-script-interaction-with-teamcity.html#reporting-build-progress, there should be support of
##teamcity[progressMessage '<message>']

and
##teamcity[progressStart '<message>']
...some build activity...
##teamcity[progressFinish '<message>']

service messages.  This PR add this support.

Also I've noted that build targets somewhat obolete and I've added netstandard2.0, net6.0 and net8.0 targets.

I've also notices that WriteBuildProblem has required parameter named identity, bu documentation says that identity should be optional:
identity (optional): a unique problem ID. Different problems must have different identity, same problems — same identity,
which should not change throughout builds if the same problem, for example, the same compilation error occurs. It must be
a valid Java ID up to 60 characters. If omitted, the identity is calculated based on the description text.

So, I've added additional method without identity parameter to not break compatibility with existing code

Please, review this PR.
Cheers, Aleksei.

Hi @halex2005! The implementation looks good to me, I just have a question regarding the target frameworks and some minor comments on wording.


Server parent disk Win2022Core_G2.vhdx not found

Got that message while deploying phase
 Restarting service RemoteAccess...

Creating DSC config to configure DC as pull server
Processing $LabConfig.VMs, creating VMs
Creating VM S2D1
Looking for Parent Disk
Server parent disk Win2022Core_G2.vhdx not found.
Press enter to continue ...

Ende der Windows PowerShell-Aufzeichnung
Use Windows Server 2019 ISO with CU patched.
What´s the trick ?

I have the same problem.. what is the solution? Pls. help!