[$250] [HOLD for payment 2024-02-07] Create centralized types for API params

context: https://github.com/Expensify/App/pull/28695#discussion_r1439869279
Problem
Currently, we're defining a type for API params before each use of that API endpoint. This could lead to inconsistent application of API params and might make it difficult to know the params (and associated types) accepted by each API endpoint.
Solution
Create a centralized map mapping API endpoint (string) to the params accepted by that endpoint.
Issue OwnerCurrent Issue Owner: @joekaufmanexpensify
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017c2ec18790b398be
Upwork Job ID:<PHONE_NUMBER>246774784
Last Price Increase: 2024-02-06

While we think this is a good idea in the short-term for its own merits, part of our thinking here is that in the long-term we'll be able to define API params in the API (where they are actually defined), and build some compile-time tool that can generate this map in the front-end so that the front-end is always up-to-date with all of the params accepted by each API endpoint.

cc @blazejkustra @fabioh8010

Commenting for assignment, thanks for creating this one @roryabraham

I got sick and work half hours, I'll get back to this next week, perhaps next week 🤗

@blazejkustra put up a PR for centralized API params. Overall it looks good but it's very large and as such things would likely go smoother if we break it up into multiple PRs and migrate one actions file at a time.
Unless I'm mistaken these PRs should be almost 100% compile-time type changes, so I think we could loosen the screenshot and review requirements for each, unless there are any runtime changes introduced, as is often the case when types are found to be incorrect in TS migration PRs.

Chatted with @blazejkustra, I'm ok with doing a single PR as long as our C+ is thoroughly testing. It's a fair amount of overhead to review such a big PR but ultimately it does look quite safe.

PR is up and ready for C+ review. @s77rt was selected, please have a look on comments above.

@joekaufmanexpensify assigning you back here to help issue C+ payment to @s77rt for their review of https://github.com/Expensify/App/pull/34542

@s77rt offer sent for $250!

@joekaufmanexpensify Accepted! Thanks!

$250 sent and contract ended!

All set!


python3Packages.ibis: 1.6.0 -> 3.2.0

Motivation for this change
Update to latest upstream release 3.2.0
Change log: http://www.dmulholl.com/docs/ibis/master/changelog.html
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.ibis
python39Packages.ibis


Jobs table for unknown Any types

Summary
Implement a queue table to retry unknown Any types
Problem Definition
If a new Msg type is committed to the chain, Explorer has no way of retrying to parse out that Msg object. Adding a job queue table would help with reprocessing automatically vs having to manually go in and ensure everything has been parsed correctly.
Proposal

Add a jobs table to hold errored out msg objects with their identified type
Add in a task to go through that table and reprocess the object once the type has been added to the service

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

closed by #232


Add validation to server-url setting for trailing forward slash

Is your feature request related to a problem? Please describe.

When supplying a url for server-url a trailing slash will break provisioning of clusters

Describe the solution you'd like

On both the setup page and global settings --> settings page add validation to block submission if the value ends in a forward slash

Additional context

It makes sense to tackle https://github.com/rancher/dashboard/issues/5666 at the same time

since this contains sufficient e2e test validation it can be moved to Done. cc @izaac


Update pytorch_forecasting models to handle integer timestamp

🚀 Feature Request
In #178 it was found that pytorch_forecasting models fail on data with integer timestamp. It could be fixed.
Proposal
Fix tests tests/test_models/test_inference/test_forecast/TestForecastOutSample for models:

DeepARModel
TFTModel

Error is withing PytorchForecastingMixin._is_prediction_with_gap which assumes that we are working with datetime. It should be fixed to work also with integer timestamp.
These models should also be checked in other places.
Test cases

Move fixed tests in tests/test_models/test_inference/test_forecast/TestForecastOutSample from fail to success category
Move fixed tests tests/test_models/test_inference/test_predict/TestPredictInSampleSuffix from fail to success category

Additional context
Blocked by #178.

Closed by #208.


enabling sa pkg and automation strategy to add GitOps ZTP policies tests

This PR help us to add following changes
[1] enabling service account pkg usage from eco-goinfra project and
[2] validating test automation strategy/structure to add CNF-6925 feature GitOps ZTP tests.

@mcornea , @shaior - Please review this PR.

LGTM, just rename err where I commented.
thanks.

@shaior , I have addressed your latest review comments.
Please help to merge this PR.  cc/ @mcornea


cmd/compile: OpInlMark can get removed by SSA dead code elimination, resulting in invalid inlining trees

See discussion at https://github.com/golang/go/issues/46234.
The immediate issue is that https://go.dev/play/p/OikFLBmKguY?v=gotip causes the loop at https://github.com/golang/go/blob/7f632f76db65f28038b8797fbfe5e15a9f63e387/src/runtime/traceback.go#L362 to never terminate, because the inlining "tree" has cycles.
The bigger issue is that we're generating trees like this for kube-apiserver, and this is presumably the root cause of #54593.

GOEXPERIMENT=nounified go build -gcflags=-d=pctab=pctoinline net/http produces:
funcpctab net/http.readTransfer [valfunc=pctoinline]
...
    7b      3 00123 (/usr/lib/google-golang/src/net/http/transfer.go:563)       MOVQ    CX, net/http.r+216(SP)
...
-- inlining tree for net/http.readTransfer:
0 | -1 | net/http/internal.NewChunkedReader (/usr/lib/google-golang/src/net/http/transfer.go:563:49) pc=1000
1 | 0 | bufio.NewReader ($GOROOT/src/net/http/internal/chunked.go:32:23) pc=0
2 | 1 | bufio.NewReaderSize ($GOROOT/src/bufio/bufio.go:63:22) pc=0
3 | 2 | bufio.(*Reader).reset ($GOROOT/src/bufio/bufio.go:57:9) pc=0
4 | -1 | io.LimitReader (/usr/lib/google-golang/src/net/http/transfer.go:568:37) pc=1472
5 | -1 | net/http.Header.get (/usr/lib/google-golang/src/net/http/transfer.go:530:47) pc=573
6 | -1 | net/http.bodyAllowedForStatus (/usr/lib/google-golang/src/net/http/transfer.go:550:60) pc=746
7 | -1 | net/http.noResponseBodyExpected (/usr/lib/google-golang/src/net/http/transfer.go:560:28) pc=845
8 | -1 | net/http.bodyAllowedForStatus (/usr/lib/google-golang/src/net/http/transfer.go:560:70) pc=845
--

Note the pc=0 value on entries 1 through 3, and that entry 3 is actually referenced by the MOVQ instruction.
The consequence of this is that when unwinding the stack at the MOVQ instruction, we'll find the bufio.(*Reader).reset call, but treat it as called directly by readTransfer, rather than transitively inlined via internal.NewChunkedReader->bufio.NewReader->bufio.NewReaderSize->bufio.(*Reader).reset.

Interestingly, that's the only function that ends up with a corrupt inline tree with GOEXPERIMENT=nounified that I could find.

Here's a minimized, standalone repro of the nounified failure:
package http

func readTransfer(r *bufioReader) any {
	return NewChunkedReader(r)
}

func NewChunkedReader(r any) any {
	br, ok := r.(*bufioReader)
	if !ok {
		br = new(bufioReader)
		reset(br, r)
	}
	return &chunkedReader{r: br}
}

type chunkedReader struct {
	r *bufioReader
}

type bufioReader struct {
	rd any
}

func reset(b *bufioReader, r any) {
	b.rd = r
}

Compiled with -d=pctab=pctoinline, this input produces:
-- inlining tree for command-line-arguments.readTransfer:
0 | -1 | command-line-arguments.NewChunkedReader (/home/mdempsky/wd/go/src/net/http/dummy.go:8:25) pc=25
1 | 0 | command-line-arguments.reset (/home/mdempsky/wd/go/src/net/http/dummy.go:15:8) pc=0
--

Again, note the bad pc=0 value.

Notably, the minimized test case repros with unified IR too, whereas the original net/http code did not.

Looking at GOSSAFUNC=readTransfer output, the issue with the corrupt inline tree above is that the OpInlMark for the inlined reset call gets removed, because it's determined to be dead code. But when we spill AX to the stack, we end up using the position information from the inlined b.rd = r assignment.
This seems to be an issue with the "expand calls" phase. The inserted OpArgIntReg value seems to be taking the position of its first use, rather than the position of the original OpArg.
/cc @dr2chase

@mdempsky seems related to https://go-review.googlesource.com/c/go/+/309649

@cuonglm Thanks, that seems plausible.
Further minimized repro:
package http

func readTransfer(r *int) {
	NewChunkedReader(r)
}

func NewChunkedReader(r *int) {
	if _, ok := any(r).(*int); !ok {
		// dead code path, but only recognized as such after "decompose builtins"
		reset(r)
	}
	sink = new(int) // force r to be spilled
	sink = r
}

func reset(r *int) {
	sink = r // anything that uses r
}

var sink any

CL 425415 adds a hacky check that we find valid PC offsets for all of the inline tree entries we emit. In particular, it reports the miscompilation of the test case above.

With this patch, above test case pass, ./make.bash run ok:
diff --git a/src/cmd/compile/internal/ssa/expand_calls.go b/src/cmd/compile/internal/ssa/expand_calls.go
index 90ea2d5040..d1a742c014 100644
--- a/src/cmd/compile/internal/ssa/expand_calls.go
+++ b/src/cmd/compile/internal/ssa/expand_calls.go
@@ -1773,7 +1773,7 @@ func (x *expandState) newArgToMemOrRegs(baseArg, toReplace *Value, offset int64,
                toReplace.Type = t
                w = toReplace
        } else {
-               w = baseArg.Block.NewValue0IA(pos, op, t, auxInt, aux)
+               w = baseArg.Block.NewValue0IA(aux.Name.Pos(), op, t, auxInt, aux)
        }
        x.commonArgs[key] = w
        if toReplace != nil {

But still fails for test in go/internal/gcimporter:
--- FAIL: TestImportTypeparamTests (2.58s)
    --- FAIL: TestImportTypeparamTests/issue47775b.go (0.01s)
        gcimporter_test.go:58: /home/cuonglm/sources/go/test/typeparam/issue47775b.go:21:5: inline marker #0 still poisoned
        gcimporter_test.go:59: go tool compile issue47775b.go failed: exit status 1
    --- FAIL: TestImportTypeparamTests/issue49432.go (0.01s)
        gcimporter_test.go:58: /home/cuonglm/sources/go/test/typeparam/issue49432.go:13:6: inline marker #0 still poisoned
        gcimporter_test.go:59: go tool compile issue49432.go failed: exit status 1
FAIL
exit status 1
FAIL	go/internal/gcimporter	3.347s

Sounds like we still mess the pos somewhere else.

What's the status of this? I see the CL was merged. Should we rebase and re-trybot CL 425415 to see if it reports any other failures?

What's the status of this? I see the CL was merged. Should we rebase and re-trybot CL 425415 to see if it reports any other failures?

See my comment in https://github.com/golang/go/issues/54625#issuecomment-1227421349
Those tests still failed with current tip.

Should we rebase and re-trybot CL 425415 to see if it reports any other failures?

Done. Still reporting failures: https://storage.googleapis.com/go-build-log/eeb08b68/linux-amd64_bc78596a.log
Retitled to better reflect the current understanding of the issue, and bumping to 1.21.

@cuonglm points out that CL 468415 addresses the failures reported by CL 425415. So I'll look into polishing that CL up to mail out for inclusion, at least during the dev cycle to help smoke out more failure cases.
I think the underlying issue around ssa's deadcode elimination pass and OpInlMarks is still applicable though.

Hey @mdempsky, doing a sweep of the Go 1.21 milestone. Any updates here?
Pushing to Backlog for now, feel free to move it to the Go 1.22 milestone if you plan to work on it. Thanks!


[WFLY-16657] moves EE Concurrency Transaction Resources to EE Subsystem

Issue: https://issues.redhat.com/browse/WFLY-16657

Please note, all was moved except the TransactionSetupProvider service, which I fully removed, that existed only to plug the TransactionSetupProvider on EE Concurrency services.

retest this please

Squashed RTS-jakarta dependency changes by @jamezp, thank you.

Thanks @emmartins!


Inconsistent types of controller prop

Release 6.3 branch
In geojson example [1] controller prop is a boolean but according to API docs [2] prop is a MapController
[1] https://github.com/uber/deck.gl/blob/6.3-release/examples/website/geojson/app.js#L113
    const {viewState, controller = true, baseMap = true} = this.props;
    return (
      <DeckGL
        layers={this._renderLayers()}
        initialViewState={INITIAL_VIEW_STATE}
        viewState={viewState}
        controller={controller}
      >
     ...

[2] https://github.com/uber/deck.gl/blob/6.3-release/docs/api-reference/map-controller.md
import DeckGL, {MapController} from 'deck.gl';

<DeckGL
controller={{type: MapController, dragRotate: false}}   // <---
viewState={viewState}
onViewportChange={v => this.setState({viewport: v})}
/>

To Do List

[ ] Add label and assign to milestone
[ ] Coding
[ ] Test

This prop is currently documented in
https://github.com/uber/deck.gl/blob/6.3-release/docs/api-reference/view.md#controller-function--boolean--object, but not in the Deck class doc.

Thanks.  Coming from OOP this will take a bit to get used to controller (Function | Boolean | Object) :)


prepare release v1.4

The following issue captures the overall intent to release v1.4 of the sphinxcontrib-confluencebuilder module. This includes issues and more found under 1.4 milestone.
initial comments
This release aims to be a quick release primarily to fix the configuration issues introduced in v1.4 (#423). Additional changes related to numerical figures are desired to be added, since a full-stop builder error can be observed.
This issue will be updated to reflect progress as the release is made. Feel free to comment if desired.

phase 01 -- "what to complete"

[x] For 1.4 milestone, ensure all pull requests have been reviewed/merged and issues ready for closure (marked as "pending next release").

phase 02 -- "system testing"
With appropriate changes merged in, validation tests will need to be performed under the following environments:

[x] Confluence Cloud (Organization Space)
[x] Confluence Cloud (User Space)
[x] Confluence Server

[x] Confluence Server v6.13
[x] Confluence Server v7.10

Several server revisions will be skipped as the subset of changes introduced in this release are small and are believed to not be a significant
phase 03 -- "release"

[x] Update CHANGES.rst.
[x] Ensure version values in the implementation and incremented and tagged appropriately.
[x] Push release cycle commits and tag to origin/master.
[x] From release tag, build and publish to PyPI (using twine).

phase 04 -- "sanity checks and cleanup"

[x] Verify Read the Docs space reflects the most recent documentation.
[x] Un-tag any issues with "pending next release" and close the issues in 1.4 milestone.
[x] Update online validation set demonstration site to include the v1.4 release.
[x] Close this issue.

v1.4 release is now complete.

v1.4 release is now complete.


Queue::fake() inconsistencies between assertPushed and assertPushedOn

Laravel Version: 8.61.0
PHP Version: 8.0.8
Database Driver & Version: MySQL 8.0.25

Description:
Queue::assertPushed() accepts an integer as the last argument, to assert that the job was pushed that many times. Works fine!
The docblock for Queue::assertPushedOn() states that it also accepts an integer as the last argument, but when I try that I get: Error: Value of type int is not callable.
* @method static void assertPushed(string|\Closure $job, callable|int $callback = null)
* @method static void assertPushedOn(string $queue, string|\Closure $job, callable|int $callback = null)

I think that assertPushedOn should match assertPushed and accept an integer as the last argument.
Steps To Reproduce:

https://github.com/adevade/queue-assert-bug
Run PHPUnit and see it fail due to assertPushedOn not accepting an integer as the last argument.

Fixed, thanks.


(WIP) Add Encryption instructions for LocalPV and cStor volumes

Taking these changes with the current content, as FAQ results in 404 without these pages.


Added sample script explaining how to check for updates, download and install

Hi,
I had some trouble trying to figure out how to check for updates, download and install them.
So I read the Atom source to figure out how the update process on windows works and I've been able to successfully implement auto update on my app.
I extracted a piece of code from Atom source to explain in the README how to do this process. I hope this can help the new users of this project.
Thank you.

/cc @kevinsawicki for his Thoughts™

@sergiovilar hey, the things are changed, so we're using ES2015 right now, also the main code has been moved to electron/windows-installer. Would you like to update your PR? I'll be glad to review it and merge in electron-winstaller. BTW, I had a terrible time with understanding of squirrel events :)

@havenchyk I'll be glad to update the PR if someone commit to review because there's almost a year I sent this one and the people simply don't care.

@sergiovilar I'll push @paulcbetts to do this :smile:  It will be cool to have an example of using squirrel events, I spent a week to realize how to implement it.

@sergiovilar well, as @kevinsawicki said, this functionality already in the electron. There is no any sense in this PR, sorry.


Sym-links in nvcc C compiler path causes "undeclared inclusion(s) in rule '//tensorflow/core:lib_hash_crc32c_accelerate_internal'"

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Redhat Enterprise Linux 7.x
TensorFlow installed from (source or binary): source
TensorFlow version: 1.14.0
Python version: 3.7.4
Installed using virtualenv? pip? conda?: virtualenv
Bazel version (if compiling from source): 0.24.1
GCC/Compiler version (if compiling from source): 4.9.2
CUDA/cuDNN version: 10.0.130/<IP_ADDRESS>
GPU model and memory: Tesla P100

Describe the problem
We have compilers installed in /shared/ucl/apps/gcc/4.9.2.  Unfortunately /shared is a sym-link to /lustre/shared.  Attempting to build Tensorflow with Cuda support results in this sym-link being inconsistently de-referenced meaning that some rules refer to the /shared location while the system include paths refer to /lustre/shared.  This seems to be an issue only with the C compiler used by nvcc, not with the one used to build the rest of the code.
Telling the configure script to use /lustre/shared/ucl/apps/gcc/4.9.2/bin/gcc as the nvcc c compiler works around the problem but is less than ideal as this path is different on different clusters. It culd also be an issue if a user on a mult-user system uses their own install of GCC in /homewhere /home is a sym-link. This appears to be a bug in the way the configure script and/or bazel deals with sym-links.
Provide the exact sequence of commands / steps that you executed before running into the problem
The script at https://github.com/owainkenwayucl/install-scripts/blob/master/scripts/tensorflow/tensorflow-1.14.0-py37-gpu_install
But effectively:
CONFIG_OPTS="--config=cuda --copt=-march=broadwell --cxxopt=-D_GLIBCXX_USE_CXX11_ABI=0 --copt=-O3"
export TF_CUDA_PATHS=/shared/ucl/apps/cuda/10.0.130/gnu-4.9.2,/shared/ucl/apps/cudnn/<IP_ADDRESS>/10.0/cuda
./configure

bazel build --verbose_failures $CONFIG_OPTS //tensorflow/tools/pip_package:build_pip_package

Answering "yes" to build with cuda and selecting defaults for compute capability, nccl etc.

Actions I took at configure stage:
You have bazel 0.24.1 installed.
Do you wish to build TensorFlow with XLA JIT support? [Y/n]:
XLA JIT support will be enabled for TensorFlow.

Do you wish to build TensorFlow with OpenCL SYCL support? [y/N]:
No OpenCL SYCL support will be enabled for TensorFlow.

Do you wish to build TensorFlow with ROCm support? [y/N]:
No ROCm support will be enabled for TensorFlow.

Do you wish to build TensorFlow with CUDA support? [y/N]: y
CUDA support will be enabled for TensorFlow.

Do you wish to build TensorFlow with TensorRT support? [y/N]:
No TensorRT support will be enabled for TensorFlow.

Found CUDA 10.0 in:
    /shared/ucl/apps/cuda/10.0.130/gnu-4.9.2/lib64
    /shared/ucl/apps/cuda/10.0.130/gnu-4.9.2/include
Found cuDNN 7 in:
    /shared/ucl/apps/cudnn/<IP_ADDRESS>/10.0/cuda/lib64
    /shared/ucl/apps/cudnn/<IP_ADDRESS>/10.0/cuda/include

Please specify a list of comma-separated CUDA compute capabilities you want to build with.
You can find the compute capability of your device at: https://developer.nvidia.com/cuda-gpus.
Please note that each additional compute capability significantly increases your build time and binary size, and that TensorFlow only supports compute capabilities >= 3.5 [Default is: 3.5,7.0]:

Do you want to use clang as CUDA compiler? [y/N]:
nvcc will be used as CUDA compiler.

Please specify which gcc should be used by nvcc as the host compiler. [Default is /shared/ucl/apps/gcc/4.9.2/bin/gcc]:

Do you wish to build TensorFlow with MPI support? [y/N]:

Build failure:
ERROR: /dev/shm/tensorflow/tmp.6ooBek4IW8/tensorflow/tensorflow/core/BUILD:2499:1: undeclared inclusion(s) in rule '//tensorflow/core:lib_hash_crc32c_accelerate_internal':
this rule is missing dependency declarations for the following files included by 'tensorflow/core/lib/hash/crc32c_accelerate.cc':
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/stddef.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/stdint.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/nmmintrin.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/smmintrin.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/tmmintrin.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/pmmintrin.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/emmintrin.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/xmmintrin.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/mmintrin.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/mm_malloc.h'
  '/shared/ucl/apps/gcc/4.9.2/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/include/popcntintrin.h'
Target //tensorflow/tools/pip_package:build_pip_package failed to build
INFO: Elapsed time: 108.730s, Critical Path: 1.51s
INFO: 42 processes: 42 local.
FAILED: Build did NOT complete successfully

If I change the answer at Please specify which gcc should be used by nvcc as the host compiler. [Default is /shared/ucl/apps/gcc/4.9.2/bin/gcc]:  to /lustre/shared/ucl/apps/gcc/4.9.2/bin/gcc then Tensorflow build successfully.

Can we close this issue since the query is been resolved.Thanks!

It's not been resolved.  There's a workaround which I found in this case but that's only because I've a great deal of familiarity with the system.
If I were a user of a system where I were trying to install things in my home directory and i did not know how things were set up underneath it would cause problems.  Googling the error finds other people stuck at this point (e.g. https://github.com/bazelbuild/bazel/issues/4573 where it's clear that the path of the user's conda tools is likely to be under a sym-link) so it should really be resolved properly.

Just to be clear, you think this would be solved if our ./configure script resolved symlinks, right?

I think so.  However it works it out, it needs to be consistent with how Bazel writes its rules (which I'm really not an expert in!).

Having looked at configure.py I believe it's as simple as adding:
gcc_host_compiler_path = os.path.realpath(gcc_host_compiler_path)

at line 842.
I'm happy to do this and do a pull request if you prefer?

Yes, that would be very helpful, thank you!
On Fri, Oct 4, 2019 at 6:57 AM Dr Owain Kenway<EMAIL_ADDRESS>wrote:

Having looked at configure.py I believe it's as simple as adding:
gcc_host_compiler_path = os.path.realpath(gcc_host_compiler_path)
at line 842.
I'm happy to do this and do a pull request if you prefer?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/32776?email_source=notifications&email_token=AHXWEQBFMDOXYSA35UDXGK3QM5DTHA5CNFSM4IZ6XHP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEALWP3A#issuecomment-538404844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHXWEQH4HSLPYQ2FOZLRZX3QM5DTHANCNFSM4IZ6XHPQ
.

I face this problem too. After I use the patch, I still face a similar problem with CUDA includes path.

Ok, that's weird - our CUDA includes are also under the sym-link and we don't get that problem.  What error do you get?

Ok, that's weird - our CUDA includes are also under the sym-link and we don't get that problem. What error do you get?

ERROR: /cache/home/jz748/.cache/bazel/_bazel_jz748/9a5804611367d5a3465d7ad3decb2864/external/nccl_archive/BUILD.bazel:67:1: undeclared inclusion(s) in rule '@nccl_archive//:device_lib':
this rule is missing dependency declarations for the following files included by 'external/nccl_archive/collectives/device/max_all_gather.cu.cc':       '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/cuda_runtime.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/host_config.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/builtin_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/device_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/host_defines.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/driver_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/vector_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/surface_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/texture_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/library_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/channel_descriptor.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/cuda_runtime_api.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/cuda_device_runtime_api.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/driver_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/vector_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/vector_functions.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/common_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/math_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/math_functions.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/cuda_surface_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/cuda_texture_types.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/device_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/device_functions.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/device_atomic_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/device_atomic_functions.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/device_double_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/device_double_functions.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_20_atomic_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_20_atomic_functions.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_32_atomic_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_32_atomic_functions.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_35_atomic_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_60_atomic_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_60_atomic_functions.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_20_intrinsics.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_20_intrinsics.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_30_intrinsics.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_30_intrinsics.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_32_intrinsics.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_32_intrinsics.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_35_intrinsics.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_61_intrinsics.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/sm_61_intrinsics.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/sm_70_rt.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/crt/sm_70_rt.hpp'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/surface_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/texture_fetch_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/texture_indirect_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/surface_indirect_functions.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/device_launch_parameters.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/cuda_fp16.h'
  '/scratch/jz748/cuda-10.1/targets/x86_64-linux/include/cuda_fp16.hpp'

Do you think it is a different issue? If so, I'll create another issue.

@njzjz facing the same error and i think this error is related to this issue only

@njzjz facing the same error and i think this error is related to this issue only

Downgrading from CUDA 10.1 to CUDA 10.0 solves the issue. Something wrong with CUDA 10.1?

@owainkenwayucl, 58b236b235ce57808c917feb19b9f895f628710c should solve the original problem here. Thank you for your patience with the CLA issue in your PR.
For others -- please file a separate issue if you're still running into problems.

No problem - thanks for solving the issue.

@njzjz facing the same error and i think this error is related to this issue only

Downgrading from CUDA 10.1 to CUDA 10.0 solves the issue. Something wrong with CUDA 10.1?

I guess it is because in CUDA 10.1, /usr/local/cuda-10.1/include is now a symlink to /usr/local/cuda-10.1/targets/x86_64-linux/include and /usr/local/cuda-10.1/lib64 is a symlink to /usr/local/cuda-10.1/targets/x86_64-linux/lib. This is different from CUDA 10.0, where both are real directories.
Ping @buchgr who seems to work on this recently. I hit the same issue setting up RBE builds with NCCL complaining missing dependency declaration. Might worth taking a look.

@njzjz facing the same error and i think this error is related to this issue only

Downgrading from CUDA 10.1 to CUDA 10.0 solves the issue. Something wrong with CUDA 10.1?

I guess it is because in CUDA 10.1, /usr/local/cuda-10.1/include is now a symlink to /usr/local/cuda-10.1/targets/x86_64-linux/include and /usr/local/cuda-10.1/lib64 is a symlink to /usr/local/cuda-10.1/targets/x86_64-linux/lib. This is different from CUDA 10.0, where both are real directories.
Ping @buchgr who seems to work on this recently. I hit the same issue setting up RBE builds with NCCL complaining missing dependency declaration. Might worth taking a look.

See here https://github.com/tensorflow/tensorflow/issues/35122#issuecomment-589858889. I have a temporary solution: resolving symbolic links for all CUDA paths.

@njzjz You are welcomed to submit a PR and ping @angerson to see if he'd like to review.


[Bug]: toBeCalledWith incorrectly succeeds when matching {} with foo: undefined

Version
27.3.1
Steps to reproduce
it("matches `{foo: undefined}` (although it shouldn't)", () => {
  hello = jest.fn();
  hello({});
  // passes although it should fail: `{}` is different from `{ foo: undefined }`
  expect(hello).toBeCalledWith({ foo: undefined });
});

Full working reproduction available here: https://replit.com/@nicolasrouanne/Jest-toBeCalledWith-undefined-match

Detailed code sample
describe('toBeCalledWith', () => {
  const hello = jest.fn()

  describe('when called with `John`', () => {
    beforeEach(() => { hello('John'); })

    it('matches `John`', () => {
      expect(hello).toBeCalledWith('John')
    });
  })

  describe('when called with `{}` argument', () => {
    beforeEach(() => { hello({}); })

    it('matches `{}`', () => {
      expect(hello).toBeCalledWith({})
    });

    it("matches `{foo: undefined}` (although it shouldn't)", () => {
      // passes although it should fail
      // {} is different from { foo: undefined }
      expect(hello).toBeCalledWith({foo: undefined})
    });

    it("doesn't match `{foo: null}` (as expected)", () => {
      // fails, as expected
      expect(hello).toBeCalledWith({foo: null})
    })

    it("doesn't match `{foo: undefined}` with stricted `objectContaining` (as expected)", () => {
      // fails, as expected
      expect(hello).toBeCalledWith(expect.objectContaining({foo: undefined}))
    })
  });

  afterEach(() => {
    jest.clearAllMocks();
  })
});

Expected behavior
I expect toBeCalledWith to fail when the argument is {} and the expected is { someRandomKey: undefined }

to me these are different

Actual behavior
toBeCalledWith passes when the argument is {} and the expected is { someRandomKey: undefined }.
Additional context
No response
Environment
System:
    OS: Linux 5.11 Debian GNU/Linux 9 (stretch) 9 (stretch)
    CPU: (4) x64 Intel(R) Xeon(R) CPU @ 2.30GHz
  Binaries:
    Node: 12.22.6 - /usr/local/bin/node
    Yarn: 1.22.5 - /usr/local/bin/yarn
    npm: 6.14.15 - /usr/local/bin/npm
  npmPackages:
    jest: ^27.3.1 => 27.3.1

Yeah, I think this is the same thing that we added toStrictEqual for... Not sure how to solve this without a major breaking change

Following @SimenB comment, I found related issues/PR:

https://github.com/facebook/jest/issues/8386#issuecomment-487767740
https://github.com/facebook/jest/pull/11026

TL;DR toBeCalledWith (or toHaveBeenCalledWith) uses toEqual to check equality. And for toEqual {} and { foo: undefined } are the same.
Closing since it seems to be the intended behaviour; though it's a bit confusing IMO. https://github.com/facebook/jest/pull/11026 might help in clarification.

It would be nice to find in the code where this toBeCalledWith is implemented using toEqual BTW. I spend 10 minutes searching in the (giant) codebase, not really finding. Don't really know where to look too 🙈


Same error message

I'm getting the same error regardless of what I do.
 python fetch_pdfs.py -pmf example_pmf.tsv
Trying to fetch pmid 28514316
** Reprint 28514316 cannot be fetched as ovid is not supported by the requests package.
 python fetch_pdfs.py -pmid 30374447
Trying to fetch pmid 28514316
** Reprint 28514316 cannot be fetched as ovid is not supported by the requests package.
the pubmed ID it is even requesting is incorrect....

In the latest update, 6 or 7 lines were uncommented out that set the args reference to some sample PubmedID (28514316). After re-commenting those lines, it fixed it.

Sorry about that, pushed a version that had those commented out now.
Thanks!


I have aggregated these unlisted potential tracking domains as i come across them

<IP_ADDRESS> 	googleoptimize.com
<IP_ADDRESS> 	gstatic.com       # Google tracker found on many sites
<IP_ADDRESS>         facebook.net
<IP_ADDRESS> 	spotify.com       # Found on many sites' embedded
<IP_ADDRESS> 	cloudflare.com
<IP_ADDRESS> 	cloudflareinsights.com
<IP_ADDRESS> 	improving.duckduckgo.com     # Search collection on duckduckgo
<IP_ADDRESS> 	boards.4channel.org  # Popular Image board, can be in Social media
<IP_ADDRESS> 	activehosted.com
<IP_ADDRESS> 	addtoany.com
<IP_ADDRESS> 	andbeyond.media
<IP_ADDRESS> 	associates-amazon.com
<IP_ADDRESS> 	automatad.com
<IP_ADDRESS> 	bbvms.com
<IP_ADDRESS> 	cbsistatic.com
<IP_ADDRESS> 	chmod777.ltd
<IP_ADDRESS> 	cookielaw.org
<IP_ADDRESS> 	cookiepro.com
<IP_ADDRESS> 	crmservice.eu
<IP_ADDRESS> 	easypolls.net
<IP_ADDRESS> 	flourish.studio
<IP_ADDRESS> 	fontawesome.com      # Found on many sites, can potentially track activity
<IP_ADDRESS> 	gravatar.com      # Found on many sites, can potentially track activity
<IP_ADDRESS> 	gumlet.com
<IP_ADDRESS> 	hooptalk.net
<IP_ADDRESS> 	hubspot.net
<IP_ADDRESS> 	kinja-static.com
<IP_ADDRESS> 	kinja.com
<IP_ADDRESS> 	knightlab.com
<IP_ADDRESS> 	kostprice.com
<IP_ADDRESS> 	licdn.com
<IP_ADDRESS> 	lolsefti.com
<IP_ADDRESS> 	mailchimp.com
<IP_ADDRESS> 	mailerlite.com
<IP_ADDRESS> 	mashable.com
<IP_ADDRESS> 	mntzrlt.net
<IP_ADDRESS> 	msecdn.net
<IP_ADDRESS> 	netdna-cdn.com
<IP_ADDRESS> 	nsstatic.net
<IP_ADDRESS> 	ntv.io
<IP_ADDRESS> 	pjtra.com
<IP_ADDRESS> 	polyfill.io
<IP_ADDRESS> 	scroll.com
<IP_ADDRESS> 	sentry-cdn.com
<IP_ADDRESS> 	sharethis.com
<IP_ADDRESS> 	sp-prod.net
<IP_ADDRESS> 	statcounter.com
<IP_ADDRESS> 	staticflickr.com
<IP_ADDRESS> 	statuspage.io
<IP_ADDRESS> 	stripe.com
<IP_ADDRESS> 	subscribers.com
<IP_ADDRESS> 	thaickoo.net
<IP_ADDRESS> 	truste.com
<IP_ADDRESS> 	videoplayerhub.com
<IP_ADDRESS> 	vlitag.com
<IP_ADDRESS> 	vntsm.com
<IP_ADDRESS> 	waweb.com
<IP_ADDRESS> 	windows.net
<IP_ADDRESS> 	wunderground.com
<IP_ADDRESS> 	yandex.ru
<IP_ADDRESS> 	ziffstatic.com
<IP_ADDRESS>         ams-ix-2.microsoft.com
<IP_ADDRESS>         ae29-0.icr02.ams30.ntwk.msn.net      # Additional domains on duckduckgo.com, works fine without them
<IP_ADDRESS>         be-102-0.ibr01.ams30.ntwk.msn.net
<IP_ADDRESS>         be-7-0.ibr01.dub07.ntwk.msn.net
<IP_ADDRESS>         ae124-0.icr03.dub08.ntwk.msn.net

I don't remember the names of all the sites I found these potential trackers in, I just look at my uBlock Origin network requests on every new site and block domains

Is this a joke?

@llacb47 Of course not, why would I? I have seen these trackers on multiple websites and saw that these were not listed in the hosts, so I thought of reporting it and getting other contributors' opinion on it.

Thank you for this @LordRishav.  I've decided to decline this suggestion.  The list includes domains foundational to the internet, finance, identity, and others.
Closing.

Oh man,the first five are the BOMB 💣
@LordRishav,just a super friendly advise that i'd recommend you to follow it:
add github.com to your list and include it in your hosts file. Thanks for doing it,much appreciated 👍

Dan @dnmTX please try to be nice to our visitors 🙃.

@StevenBlack Maybe you could select and add the domains which are not problematic? I suggested the domains I have blocked on my side and are not included in the list, you're free to choose whichever domains you think should be in the list :)

I filtered out OP's request lol
Here are the at least somewhat harmful domains:
Most of them are just 3rd party cookies, trackers, spam sites, but there are 3-4 malware/phising sites too
<IP_ADDRESS> activehosted.com
<IP_ADDRESS> addtoany.com
<IP_ADDRESS> andbeyond.media
<IP_ADDRESS> automatad.com
<IP_ADDRESS> bbvms.com
<IP_ADDRESS> flourish.studio
<IP_ADDRESS> hubspot.net
<IP_ADDRESS> knightlab.com
<IP_ADDRESS> lolsefti.com
<IP_ADDRESS> mailchimp.com
<IP_ADDRESS> msecdn.net
<IP_ADDRESS> netdna-cdn.com
<IP_ADDRESS> ntv.io
<IP_ADDRESS> pjtra.com
<IP_ADDRESS> sharethis.com
<IP_ADDRESS> statcounter.com
<IP_ADDRESS> statuspage.io
<IP_ADDRESS> thaickoo.net
<IP_ADDRESS> truste.com
<IP_ADDRESS> videoplayerhub.com
<IP_ADDRESS> vlitag.com
<IP_ADDRESS> vntsm.com
<IP_ADDRESS> windows.net
<IP_ADDRESS> wunderground.com
<IP_ADDRESS> ziffstatic.com

@kekmacska , please avoid necrobumping. Submit a new issue.


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/instalator/ioBroker.mikrotik

[ ] [E019] Invalid repository URL: https://github.com/instalator/iobroker.mikrotik. Expected: git+https://github.com/instalator/ioBroker.mikrotik.git

I have also found warnings that may be fixed if possible.

[ ] [W113] Adapter should support compact mode

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Thanks, that all bugs have been fixed.


Model  addition

Model introduction
This model is trained on around 23000+ set of codes. Each set having 2 conversations. This conversation is in Vicuna/ShareGPT format. Each set, along with code, has detailed explanation. I have released the data.
Training: Entire dataset was trained on Azure 4 x A100 80GB. For 3 epoch, training took 42 hours. DeepSpeed codebase was used for training purpose. This was trained on Llama-1 by Meta.
Model URL
ajibawa-2023/Python-Code-33B
Data type
bfloat16
Additional instructions (Optional)
No response
Author
Yes
Security

[X] I confirm that the model is safe to run which does not contain any malicious code or content.

Integrity

[X] I confirm that the model comes from unique and original work and does not contain any plagiarism.

Sorry, I was supposed to add other model. I have opened other Request under: https://github.com/evalplus/evalplus/issues/73
Thank you very much!


Remove "da.random" from random array keys

We tend not to preface keys with the module name.
The long names also make diangostics slightly less pleasant.

[ ] Tests added / passed
[ ] Passes flake8 dask

One small comment. Otherwise LGTM.


CrashReporter improvement : IssueReporter ?

Contains
Fixes #2765
How to test

Rename the button to "Issue Reporter".

However I still don't know why my other text on button disappear. Appreciate badly someone can tell me the reason 👍
I just changed the text value in .lang file but haven't changed others, like "crash-reporter": "Issue Reporter", I'm wondering if it's better change all "crash reporter" to "issue reporter", like "Issue-reporter": "Issue Reporter"

I've moved this button under Developer Tools menu

I'm wondering if it's necessary rename Developer Tools button to Extras

Outstanding before merging

[ ] Release CrashRreporter v4.1.0 and bump the Terasology dependency to match it
[ ] I've tried to set crash reporter window to non modal windows simply by setting set jDialog to modeless type, sadly log doesn't update @Cervator  What's more, in the non modal windows, you can reopen this windows, and there can be more than one CR windows :( Other than making this window to update log, I think we should think about how to make sure there is only one CR window, like if CR window is open, press the CR button to make this windows in front of all other windows.
[ ] I think that there are some texts to be changed from "Crash Reporter" to "Issue Reporter" :

Ah it's mentioned in MovingBlocks/CrashReporter#35

Can one of the admins please verify this patch?

Problem found ! It's from system language setting, seems that Chinese support doesn't work well ! :(

@GabrielXia just responded with a big comment over on #2677 - just to clarify, you picked Chinese yourself under the Player Settings, right? It didn't get picked by the game automatically from simply running on your system? I'd think it would start out English in any case :-)

Why are you removing the button from the in-game menu?

Uh oh, something went wrong with the build. Need to check on that

Why are you removing the button from the in-game menu?

@msteiger that one is on me, I raised it as an idea over in #2765 but it was honestly more of a stray thought / suggestion I had hoped to get some more opinions on first :-)
To be clear the button just moved one level deeper, it didn't go away entirely. But I am curious what the most ideal setup there would be. Should reporting an issue be right there as a top level button users can easily spot, or grouped with other stuff under "Extras" ?
I would love to instead see the thing I mentioned over in #2765 about what @iojw did in the CR, so we can have a little warning icon in a bottom corner instead of a full button. Make it highlight if anything has occurred we think might be worth reporting. Otherwise it is just a small icon hiding in a corner, not adding much to clutter.

Hello :-) I've moved this button to the Developer Tools. I think the issue reporter might rather be a tool for developer, but you are right, put it in in-game menu is more convenient for user to check. Maybe we can ask @Cervator here to discuss about where we put this button :-)
p.s. Why there is a red cross ? Scared ...

The red x is because the build in Jenkins failed - in this case that's just the unfortunate reality of making a change that requires a new version of an external project like the CrashReporter. Since the engine build test uses the latest CR without your change there is a simple syntax error where the new thing is missing :-)
Poking @rzats for an additional opinion on the button since he does UI stuff!

Uh oh, something went wrong with the build. Need to check on that

Uh oh, something went wrong with the build. Need to check on that

Uh oh, something went wrong with the build. Need to check on that

https://github.com/MovingBlocks/CrashReporter/pull/36 has been merged some time ago. Is there something outstanding here except the new release and version bump?
The conflict in the language file seems to be pretty small.

Thanks for keeping on this, I think it's ok for the moment. :-)
The enhancement of CR could be:

Show warning icon somewhere in the game when there is an error
Based on the telemetry system, maybe a new page showing all the information sent to the server

I've just released a new version of CR (4.1.0). We can probably merge this after the next Alpha release.
@GabrielXia could you please update the dependency in this PR?

Uh oh, something went wrong with the build. Need to check on that

I'm so glad to see this to be merged :-)
@skaldarnar which dependency do you refer to? Is that the exclusion of jackson in CR: https://github.com/MovingBlocks/CrashReporter/pull/39 or the one to import snowplow: https://github.com/MovingBlocks/Terasology/pull/2968 ?

I went ahead and did the version bump (just the CR itself in the engine's build.gradle from 4.0.1 to 4.1.0) while merging. Conflicts were benign as expected and it tested out fine :-)

@Cervator There seems to be a problem with my build. Although you bumped the version of CR, do i need to do anything else manually like download the new version jar for CR?

@0shine0 running gradlew idea or gradlew jar should put you on the safe side and download the updated dependency. Be sure that you don't have the Crash Reporter checked out as source in case you run into problems.


Compilation errors when running create or replace command

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
There has been multiple instances where i run a create or replace procedure/package command. After running there are no apparent error messages. I do not realize until i refresh the procedure list that there was in fact an error when creating the procedure
Describe the solution you'd like
A clear and concise description of what you want to happen.
There should be the same error message when you run create or replace as if you would right click and compile the object. Right now there is no way to discern whether the command was successful or not.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

seems issue is obsolete


Mac/Linux Install?

While I am not an experienced developer, this project appears to just be a web site running on a SQL database. Is there a reason why it is packaged in a Windows installer with no options for Mac or Linux installs? Wouldn't it be much simpler to run on a system that has built-in support for these kinds of projects?

While it does utilize SQL I think the entire stack is primarily Microsoft based, with front-end components such as IIS and ASPX that would be much more difficult to implement in Linux or Mac environments. The standalone IIS and SQL Express installer for example, is much quicker and easier than deploying a full blown MySQL instance with Apache, for example.

We are working on this.   A new version to come out in the winter will run on MAC environments (not phone yet, but want that too).


Reference Yubico Linux Instructions

Sorry this has been sitting for so long! I think adding the support page link could be helpful, but I don't think the headers make all that much sense (Python/Windows/Linux). I'll merge this and make the adjustments I think are needed, hope you don't mind.


docs(slide-toggle): migrate examples to MDC

Migrate slide-toggle example to use mdc-based components.

brought back the inline demo
fixed component harness example
This PR is ready for review again.


Find motorway_link with missing oneway tags

The recent OSRM update https://github.com/Project-OSRM/osrm-backend/pull/2491 does not automatically consider every motorway_link as oneway anymore.
Motorway links with missing oneway tags in OSM may now create incorrect routes. We need a linter to detect such links so that mappers can add the oneway=yes if required.
Sketch

Find ways tagged highway=motorway_link without oneway tag
If the link is connected to a highway=motorway with a oneway, it might also be a oneway

We can load these into to-fix for later review.
cc @Rub21

@planemad  can you check on the detection http://osmlab.github.io/to-fix/#/stats/missingonewaysmotorwaylink
also the validator https://github.com/osmlab/osmlint/tree/missingoneway/validators/missingOneway.

@Rub21 looking fantastic! Most of the incorrect detections are two way links connected to other one way links. These can be dropped.
False positives - Not an error

Links between two motorways https://www.openstreetmap.org/way/248995749
Links with access=no
Link connected to another oneway link https://www.openstreetmap.org/way/21288171

@planemad  I fix the code to avoid false positive, could you check  http://osmlab.github.io/to-fix/#/task/missingonewaysmotorwaylink ?

@planemad ^^

@Rub21 sorry, missed this. Results look excellent, error rate is 1 in 10.
False positives:

https://www.openstreetmap.org/way/421273035

I think the task is ready to hit, if you can fix ^ even better.
cc @maning @andygol

I think the task is ready to hit

Nice!
Next action:

[ ] @rub21 add missingOneway validator in connectivity linting. This  will detect every two days.
[ ] @planemad  create a master ticket to start working on these issues.

@planemad ^^

@planemad  still missing a master ticket to start working on this task - http://osmlab.github.io/to-fix/#/task/missingonewaysmotorwaylink, can you create it.?

done!


Home page: Show progress bar towards goal

Add a progress bar to the home page which shows the amount of funds raised and the goal.

[ ] HTML/CSS visual progress bar
[ ] Real current amount of funds in bitcoin address of project

I've done progress bar but I'm not able to push my code to git
git push origin progress_bar
ERROR: Permission to Learn-by-doing/bitstarter.git denied to msirovy.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

That's alright.
What you should be doing is pushing to YOUR github account
(msirovy/bitstarter) and making a PULL REQUEST from there to Learn-by-doing
Hope this helps.
Boris
On Mon, Jan 16, 2017 at 10:20 PM, Marek Sirovy<EMAIL_ADDRESS>wrote:

I've done progress bar but I'm not able to push my code to git
git push origin progress_bar
ERROR: Permission to Learn-by-doing/bitstarter.git denied to msirovy.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Learn-by-doing/bitstarter/issues/102#issuecomment-272964540,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMU6L9B2TV8oNHOu15oPBsfdhxAF6lqxks5rS98UgaJpZM4Lk3qU
.

That's strange, I've got an email notification form @msirovy but can't see it in the thread here. That's what I've been replying to above:

"I've done progress bar but I'm not able to push my code to git
git push origin progress_bar
ERROR: Permission to Learn-by-doing/bitstarter.git denied to msirovy.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists."

Hi,
I familliar with gitlab and I used to work on one repository and push to
own branche, and doing merge requests.
I deleted my post to issue, because I have found answer in README.
Thanks for your help. Problem was that I had started to code before red
REDME:-)
On Mon, 16 Jan 2017 at 22:42, bokova<EMAIL_ADDRESS>wrote:

That's strange, I've got an email notification form @msirovy
https://github.com/msirovy but can't see it in the thread here. That's
what I've been replying to above:

"I've done progress bar but I'm not able to push my code to git
git push origin progress_bar
ERROR: Permission to Learn-by-doing/bitstarter.git denied to msirovy.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists."
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Learn-by-doing/bitstarter/issues/102#issuecomment-272968613,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC5FdAqLS190eQJ4J4-rAvvQ68lYFp7Wks5rS-Q-gaJpZM4Lk3qU
.

--
Marek Sirovy


UserDefault out of date

Please update dependency, really need this package .

This is a duplicate of https://github.com/juliansteenbakker/mobile_scanner/issues/1165


fix(server): remove force-clone of options

In porting over amended changes from https://github.com/pact-foundation/pact-js/pull/489, my tests were routinely failing because the suite couldn't start the server with logLevel provided.

Then I discovered I had the same behavior in master when running tests locally.
Then I discovered the latest CI build for master was failing for the same reason.
Removing this line fixes the build locally for me. Marking this as a draft PR to see what CI does when it's submitted, or if anyone else has ideas about why this is failing. 🤷♀
cc @TimothyJones

This is awesome! You're smashing through my todo list!
I'll merge this in and release immediately.

@neezer It wasn't worth holding up the merge, but for the next one, commit messages with fix or feat go straight into the release notes. If it doesn't need to go in to the notes, I usually name them chore, even if fixing something.
This reminds me that I've been meaning to add a note explaining this in the CONTRIBUTING doc. I'll do that now for all three JS repos.

Releasing as 10.10.1, build is here

@neezer It wasn't worth holding up the merge, but for the next one, commit messages with fix or feat go straight into the release notes. If it doesn't need to go in to the notes, I usually name them chore, even if fixing something.

Sorry, force of habit.

This reminds me that I've been meaning to add a note explaining this in the CONTRIBUTING doc. I'll do that now for all three JS repos.

👏


How to use Language property in WinUI TextBox controls

How to use Language property in WinUI TextBox controls and what will be the behavior of Language property?

duplicate,
https://github.com/microsoft/WindowsAppSDK/discussions/1042


Review Namespace Concept

Especially:

should we switch to domain / endpoint focused subnamespaces in the "model" Namespace?
generally start with domain / endpoint?
How to handle the reusable models that aren't bound to an endpoint (e.g. Address)

done with 12f6221 (we decided to move more stuff into domain namespaces instead of functional namespaces)


Pattern matching errors for dynamic-import-chunkname

Webpack has various output templates that do not match chunkname regex

Duplicate of #1181. See also #1163, #1130.


Auto-generated builtin comments with type hints + smaller fixes

Great! A lot of improvements I see :)


Sinatra application streaming always returning Content-Type 'application/octet-stream'

First of all, thanks for the hard work on Carrierwave and Refile. Carrierwave has been great in the past, and I'm looking forward to the benefits of a newer library incorporating lessons learned.
Now to the issue:

I've used the default instructions on setting up refile on a new Rails app. (Example app can be found here).
It looks like the Sinatra app is failing to accurately determine Content-Type. I've even tried adding the attachment_name_content_type column to my Model and can see that it is correctly storing that value in the database.
After a little debugging, it looks like the Sinatra app is trying to determine the Content-Type by looking at ::File.extname(request.path) (https://github.com/elabs/refile/blob/master/lib/refile/app.rb#L106), and since refile uses tokens for path names and not the actual filename, the extension is "".
Example:

This results in a Content-Type of application/octet-stream. This is preventing IE9 from properly displaying images.
Example:

It looks like it might take a significant refactoring to be able to get ahold of the ActiveRecord object from within the Sinatra app in order to be able to use the stored image_content_type or even look at the original filename.
If you can point me in the right direction, I'd be happy to attempt a fix.

Unfortunately the README doesn't explain very well how this is supposed to work. I'll see if I can work this into the docs somehow.
Basically, the Sinatra app should not be aware of anything to do with the model, or the data that was stored in it. But we still want to serve the file with the appropriate content-type. What we do is specify the content-type in the URL instead, via the file extension. So we want a URL like:
/attachments/store/xyz/image.png

This way, Sinatra can infer that this is a PNG file and serve it with the appropriate content type.
Unfortunately the URL in your case did not include the file extension. There are three ways a URL can get an extension:

It's explicitly set. As in attachment_url(..., format: "jpg"). The image processing system will actually use this information to convert to the appropriate format automatically.
It is inferred from the stored content type
It is taken from the stored filename

My guess as to what went wrong in your case is that while you added the  attachment_name_content_type column, you did not retroactively add data to the files that had already been uploaded without a content type, and so the already uploaded files did not have a content type, and therefore no extension in the URL. Am I right?
I've already tweaked the quick start example to suggest option (1) for now, and I'll work on further improving the documentation so that this doesn't trip up more people.

Thanks for the excellent response!
I actually tried the attachment_name_content_type and it was failing for new records, so I don't think that was the specific issue. It seems the Sinatra app wasn't inferring it from the stored content type (since it doesn't have access to the model).
This makes a lot more sense now and I'll go ahead and update my attachment_url or update the way the file URL is stored.
You can consider this closed from my end. I've verified that supplying the format like attachment_url(..., format: "jpg") does indeed fix my problem.


fix for peerDependencies requirements!

downgraded to 0.14.8
formsy is making some bad trouble here better use the nice bootstrap forms? :) please?
thanks :)

peerDependencies warning has already been addressed in latest devel branch.
No need to downgrade React, formsy version 0.18 will now use<EMAIL_ADDRESS>git clone https://github.com/TelescopeJS/Telescope
cd Telescope/
git checkout devel
npm install
`meteor'
If you prefer the bootstrap forms feel free to create a Telescope package which replaces the functionality of NovaForm component using bootstrap forms and issue a pull request.
This would be an awesome contribution and allow users to choose between either formsy or bootstrap.

@justintime4tea well Formsy itself uses Bootstrap, so I don't know if we need another Bootstrap version. Also I'm not sure if React-Bootstrap by itself provides the same kind of form wrapping logic that things like Formsy or Redux-Forms do. Although I'll admit I'm still pretty new to React forms so I could be wrong.

Aw, makes sense. I wasn't familiar at all with the Bootstrap Forms the person above us was talking about. Good to know.

me too I don't know this person above lol :D

The deps issue got fixed earlier 🎉 🎋


Importlib_metadata pinned at very old version

I've noticed that importlib_metadata is pinned at version 1.* when version 2 came out already a 2 years ago
Would it make sense to update this pint or make it less strict?
https://importlib-metadata.readthedocs.io/en/latest/history.html

https://github.com/bioconda/bioconda-recipes/blob/dab3e65f7a3073d0abe3ecf22f249a36b21bfac7/recipes/snakefmt/meta.yaml#L31
https://github.com/snakemake/snakefmt/blob/f136a7ec300310862e501823604f04bffa519c4c/pyproject.toml#L21

I was just having problems with this same point. It is starting to make it hard to install snakefmt in pipelines with new versions of other software.

Will have a look at this in the next 7 days :+1:


move_and_slide makes bodies overlap

Godot version:
First appeared in Godot 3.1 Beta 6, still the same on Godot 3 RC 3
OS/device including version:
Arch Linux
Issue description:
When I'm using the move_and_slide function with changing linear_velocity parameter, the body tends to go "inside" of other bodies in an unexpected manner. It never happened to me in Godot versions before Beta 6.
Steps to reproduce:
Use keys WSAD to control player. When the player lands on the ground press S to increase velocity.y, player should then fall a bit into the ground.
Minimal reproduction project:
move_and_slide_minimal.zip

Also having this issue, game physics worked fine on 3.0.6 and 3.1 beta 5, doesn't work on 3.1 stable or any later beta versions. Any updates on this?

Maybe related with this bug also ? Please take a look at 2D and 3D Kinematic overlapping issues here also. https://github.com/godotengine/godot/issues/27652


Upgrade to stable React Email version

Issue:
Currently using a canary version of React Email due to the issue outlined in https://github.com/resend/react-email/issues/1245.
Fix:
Upgrade to next React Email release version when available.
Next Steps:
As per the link we can see that the fix has been merged into canary and pending stable release.
Therefore, next steps are;

Wait for new release version
Upgrade to new version
Test export and verify working

Already fixed


Basic authentication support

Please consider adding basic authentication support.

https://github.com/yahoo/kafka-manager/issues/111 duplicate?

The other ticket mentions oauth2. This one is IMO less demanding, could be a good start. Preferably both get supported.

+1!

1 !

+1

Fixed by https://github.com/yahoo/kafka-manager/pull/217


Adds config for GitHub Settings app

Adds config for GitHub Settings app

https://github.com/apps/settings

This change was executed automatically with Shepherd. 💚🤖

@relistan and I discussed about this repo a few days ago and came to the conclusion that it's an open source project, hence it shouldn't have any of the stuff that we have for our other repos (it doesn't have Concourse for example). Closing this for this reason 🙃

cool, sounds good!


Incorrect canonical Zed for "join"

Here's some Zed I show off in the join cookbook:
$ cat join-uid-conn-fused.zed
split (
  => filter _path=conn | sort uid
  => filter _path=spl | sort uid
) | left join uid=uid orig_spl,resp_spl,orig_spt,resp_spt | fuse

It runs fine in that form with zq commit dd946d3.
$ zq -z -I join-uid-conn-fused.zed conn.log spl.log
{_path:"conn",ts:2020-09-01T20:48:11.39799Z,uid:"C09v4A1ThuoXr4Ucdc" (bstring),id:{orig_h:<IP_ADDRESS>,orig_p:49836 (port=(uint16)),resp_h:<IP_ADDRESS>,resp_p:443 (port)} (=0),proto:"tcp" (=zenum),service:null (bstring),duration:153.774ms,orig_bytes:0 (uint64),resp_bytes:0 (uint64),conn_state:"REJ" (bstring),local_orig:null (bool),local_resp:null (bool),missed_bytes:0 (uint64),history:"Sr" (bstring),orig_pkts:1 (uint64),orig_ip_bytes:52 (uint64),resp_pkts:1 (uint64),resp_ip_bytes:40 (uint64),tunnel_parents:null (1=(|[bstring]|)),orig_spl:null (2=([uint64])),resp_spl:null (2),orig_spt:null (3=([float64])),resp_spt:null (3)} (=4)
...

However, if I send the Zed on a round trip through the canonical Zed generator, it no longer parses.
$ zast -C $(cat join-uid-conn-fused.zed) | tee busted.zed
split (
  =>
    filter _path="conn"
    | sort uid
  =>
    filter _path="spl"
    | sort uid
)
| join on uid=uid
  join-cut orig_spl,resp_spl,orig_spt,resp_spt
| fuse

$ zq -z -I busted.zed conn.log spl.log
zq: parse error: error parsing Z at line 9, col 11:split (
  =>
    filter _path="conn"
    | sort uid
  =>
    filter _path="spl"
    | sort uid
)
| join on uid=uid
      === ^ ===
  join-cut orig_spl,resp_spl,orig_spt,resp_spt
| fuse

Verified in Zed commit 60c70f2.
Revisiting the original Zed, on is now a required part of invoking join, to be more in line with what users may be familiar with from SQL. Modifying the original Zed, now it looks like:
$ cat join-on-uid-conn-fused.zed
split (
  => filter _path=conn | sort uid
  => filter _path=spl | sort uid
) | left join on uid=uid orig_spl,resp_spl,orig_spt,resp_spt | fuse

When turned this into canonical Zed and sent back through zq, it no longer creates a parse error.
$ zc -C $(cat join-on-uid-conn-fused.zed) | tee canonical.zed
split (
  =>
    filter _path="conn"
    | sort uid
  =>
    filter _path="spl"
    | sort uid
)
| join on uid=uid orig_spl,resp_spl,orig_spt,resp_spt
| fuse

$ zq -z -I canonical.zed conn.log spl.log
Zed query requires a single input path

As the last command shows, there's a different, non-parser-related problem with this older Zed script. In brief, wiring up the multiple data sources as a prerequisite for joining data is more easily performed today by using new data pools and from, though in a pinch, @mccanne taught me the trick that adding pass can make this still work with zq.
$ cat pass-join-on-uid-conn-fused.zed
pass | split (
  => filter _path=conn | sort uid
  => filter _path=spl | sort uid
) | left join on uid=uid orig_spl,resp_spl,orig_spt,resp_spt | fuse

$ zq -z -I pass-join-on-uid-conn-fused.zed conn.log spl.log
{_path:"conn",ts:2020-09-01T20:48:11.39799Z,uid:"C09v4A1ThuoXr4Ucdc" (bstring),id:{orig_h:<IP_ADDRESS>,orig_p:49836 (port=(uint16)),resp_h:<IP_ADDRESS>,resp_p:443 (port)} (=0),proto:"tcp" (=zenum),service:null (bstring),duration:153.774ms,orig_bytes:0 (uint64),resp_bytes:0 (uint64),conn_state:"REJ" (bstring),local_orig:null (bool),local_resp:null (bool),missed_bytes:0 (uint64),history:"Sr" (bstring),orig_pkts:1 (uint64),orig_ip_bytes:52 (uint64),resp_pkts:1 (uint64),resp_ip_bytes:40 (uint64),tunnel_parents:null (1=(|[bstring]|)),orig_spl:null (2=([uint64])),resp_spl:null (2),orig_spt:null (3=([float64])),resp_spt:null (3)} (=4)
...

Thanks @mccanne!


warning while intalling requirements - depreciation

Issue Type
Others
Source
source
Keras Version
3.6.8
Custom Code
No
OS Platform and Distribution
Linux Ubuntu 20.04 focal
Python version
3.18.9
GPU model and memory
NVIDIA RTX A6080 rev a1
Current Behavior?
I was installing in ubuntu from requirements and part of that was the following error:

DERECATION: promise is being installed using the legacy setup.py install' method, because it does not have a 'pyproject.toml' and the wheel package is not installed.  Pip 23.1 will enforce this behavior change.  A possible replacement is to enable the --use-pep517 option.  Discussion can be found at ...
Standalone code to reproduce the issue or tutorial link
ubuntu --> venv --> activate --> git clone repo

pip install -r requirements.txt

Relevant log output
na.  pip doesn't naturally log, and by the time I wrote this, the installation process had completed.

Could you please clarify on what package you are trying to install in Keras and which requirements.txt you are referring to?
Keras does not have 3.6.8 package yet, latest available as of now is 3.6.0


Kafkaoutput binding for azure function : Java

We are receiving message from event hub and perform some processing in azure function. The output need to be publish to kafka. We also want to preserve the order (order by partition key) of the input event hub while sending to kafka. What option is available to @kafkaout annotation to provide the partition key?
The azure function is written in Java

@bikramjit84 You can use the partition or the key(from the headers) of the incoming messages as shown in the sample here. This can then we used to the partition directly or provide a key as a part of the headers as in the sample here.


Replaces uses of 'Prepass' with 'Deferred'

Goes and replaces the references/names that use Prepass to be Deferred, since we're actually using deferred.

Na... this makes too much sense.

Verified via basegame+FPSGameplay module.


Allow nullable for CarbonAware.csproj

Description
CarbonAware.csproj is the original project and it does not have the flag set to allow nullable objects.
Acceptance Criteria

[x] Ensure that the EnableNullableOjects flag is set in the CarbonAware.csproj
[x] Ensure that all warnings that an object is not typed are fixed by adding a type
[x] Unit Tests are completed and code passes tests

Dependencies
Task List

[ ] List of tasks (punchlist) needed to complete the user story

Sprint-Ready Checklist

[ ] Acceptance criteria defined
[ ] Acceptance criteria is verifiable / testable
[ ] External / 3rd Party dependencies identified
[ ] Engineering team understands acceptance criteria
[ ] Is the user story small enough to be implemented in a short amount of time, but large enough to provide value?

PR: https://github.com/Green-Software-Foundation/carbon-aware-sdk/pull/237

PR merged


storage: introduce elasticsearch client shim

In order to support elasticsearch on AWS (and other *aaS vendors) that
choose not to expose the ES binary protocol, we must have an option for
clients to use the native transport or REST.
Towards that end, this change refactors the existing storage client
interactions behind an interface that can be replaced by a different
transport. Almost all the changes were non-functional, with the
exception of three methods:

getRawTrace
getServiceNames
getSpanNames

Which now use the leinientExpandOpen strategy for their index selection.
We anticipate this is a positive change, as skipping unavailable (or
closed) indexes will at worst present a partial span for the period
of time that the index is unavailable, which seems be prefereable to
a hard error.
Also, when using the same elasticsearch cluster for multiple Zipkin
instances (i.e. running tests + a local development server) the
instances will race to claim the zipkin_template with a pattern
that will match only their indices – so if the tests are run first,
then the local server's indices will have only an implicit schema, or
vice versa.
This change modifies the name of the template on a per-index-pattern
basis, so multiple templates can happily coexist. For the default
zipkin index prefix, users should not observe any change in
behavior. NB: It may be necessary to delete unstructured indexes or
clear out old zipkin_template templates in some cases to get
multiple instances working together.
This commit is the first step in support of #1302

here's the follow-up. thanks again @sethp-jive!
https://github.com/openzipkin/zipkin/pull/1304


Limit handler calls to respective host modes

Otherwise, when /etc/default/mesos is changed, it fires off both handlers.

Not sure how you guys feel about it, but these conditionals are a little more succinct and accomplish the same thing:
when: '"master" in mesos_install_mode'

when: '"slave" in mesos_install_mode'

I agree, it reads a lot easier.

Funny, I got similar changes in this PR #34 . But yeah that is better.

Oh whoops... didn't see your changes there @ninthnails. I'll close this for now, and perhaps open another PR for changing all those conditionals to the shortened version after #34 is merged.


Local Development Setup: Host

Goal: Develop on a completely local setup.
Previous Steps: Setup tira-application locally and mount the toy model locally.
TODO:

[x] follow wiki to setup local development host

If current local host setup does not work, implement local setup as follows

[x] instructions how to install salt locally
[x] derive more generic (not betaweb specific) salt script from the betaweb salt script
[x] run it
[x] fix this procedure in documentation
[ ] add default virtual machine (alpine) for dummy user to the toy model
[x] switch from two makefile targets to one: specify tira model host path in a variable

Trying to get the host running locally, by following wiki, it seems like I did not get it to work:
PREREQUISITES FOR LOCAL SETUP OF TIRA HOST

I have ensured to have the tira-model (toy version) mounted at /mnt/nfs/tira (with mount --bind /tira-model/src /mnt/nfs/tira/)

RUNNING THE TIRA APPLICATION

I have changed tira_root: /mnt/nfs/tira in tira-application/config/settings-dev.yml
then run the application by using make run-dev in tira-application/

FOLLOWING THE LOCAL SETUP INSTRUCTIONS FOR THE HOST IN THE WIKI (Deploying the Backend)

I have installed virtualbox 6.1.16 + extensionpack and docker locally
I have run "sudo chown -R 1010:1010 build/initial-docker-tira-model" as recommended
I have run "make build-tira-docker"
and then "make run-tira-docker-dev"

There was no error in running the container, but I can see the host is not up, because navigating the interface of the app to http://<IP_ADDRESS>:8080/task/sample-task/user/pan21-av-baseline shows "internal server eror" for the host and the commandline log prints:
INFO 2021-08-31 16:02:02,802 basehttp: "GET /public/tira/js/software.js HTTP/1.1" 200 10397
ERROR 2021-08-31 16:02:02,948 actions: /grpc/pan21-av-baseline/vm-info to localhost failed with <_InactiveRpcError of RPC that terminated with:
status = StatusCode.UNAVAILABLE
details = "failed to connect to all addresses"
debug_error_string = "{"created":"@1630418522.946520627","description":"Failed to pick subchannel","file":"src/core/ext/filters/client_channel/client_channel.cc","file_line":5396,"referenced_errors":[{"created":"@1630418522.946518495","description":"failed to connect to all addresses","file":"src/core/ext/filters/client_channel/lb_policy/pick_first/pick_first.cc","file_line":397,"grpc_status":14}]}"

Traceback (most recent call last):
File "/home/theresa/Desktop/TIRA/TIRA/tira/tira-application/src/tira/actions.py", line 315, in vm_info
response_vm_info = grpc_client.vm_info(vm_id)
File "/home/theresa/Desktop/TIRA/TIRA/tira/tira-application/src/tira/grpc_client.py", line 45, in vm_info
response_vm_info = self.stub.vm_info(tira_host_pb2.RequestVmCommands(vmId=vm_name))
File "/home/theresa/Desktop/TIRA/TIRA/tira/tira-application/venv/lib/python3.8/site-packages/grpc/_channel.py", line 923, in call
return _end_unary_response_blocking(state, call, False, None)
File "/home/theresa/Desktop/TIRA/TIRA/tira/tira-application/venv/lib/python3.8/site-packages/grpc/_channel.py", line 826, in _end_unary_response_blocking
raise _InactiveRpcError(state)
grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
status = StatusCode.UNAVAILABLE
details = "failed to connect to all addresses"
debug_error_string = "{"created":"@1630418522.946520627","description":"Failed to pick subchannel","file":"src/core/ext/filters/client_channel/client_channel.cc","file_line":5396,"referenced_errors":[{"created":"@1630418522.946518495","description":"failed to connect to all addresses","file":"src/core/ext/filters/client_channel/lb_policy/pick_first/pick_first.cc","file_line":397,"grpc_status":14}]}"

ERROR 2021-08-31 16:02:03,002 log: Internal Server Error: /grpc/pan21-av-baseline/vm_info
ERROR 2021-08-31 16:02:03,002 basehttp: "GET /grpc/pan21-av-baseline/vm_info HTTP/1.1" 500 49
ERROR 2021-08-31 16:02:03,002 basehttp: "GET /grpc/pan21-av-baseline/vm_info HTTP/1.1" 500 49
I don't know what went wrong at this point and I am open for hints! :)

Is the container running or stopped?
-$ docker ps -a
can you connect to the container?
`-$ docker exec -ti  bash'
Are all files there (/mnt/nfs/tira)? Is the python app running?
can you run the tira command while connected to the container?
can you create/start/stop vm's while connected to the container? Can you log into one of the created machines?
The application probably can't connect to a working host because I changed the API and have not had the time to also update the hosts.

Thanks, Matti. I have the following answers to your questions:
Is the container running or stopped?->
The container is not running, it "exited (2)"

can you connect to the container? -> no because it is not running
Are all files there (/mnt/nfs/tira)?  ->
/mnt/nfs/tira has the tira-model/src folder mounted, and shows the directories data, /log, /model, /state

Is the python app running?->
By python app you mean the application? -> yes:

The following questions relate to the host-container, which is not running i guess?!
"can you run the tira command while connected to the container?
can you create/start/stop vm's while connected to the container? Can you log into one of the created machines?"
Any idea how i get the host-container running? Or continue from here?

Is the container running or stopped?-> The container is not running, it "exited (2)"
All of these questions are for the host container. You first want to figure out why the container is not working.

Nikolay and I have gotten the container to run. It was a permissions issue, that has been fixed in the documentation.
(The mounted Tira-Model did not have tira user and group permissions)
Now, we have proceeded to the next hickup: The Ubuntu kernel 5.11 does not support virtualbox 6.1.16.
-> We are installing 6.1.27 and see if it works.

The new virtualbox version 1.27 works.
The next hickup is caused by this recent commit, removing initial-docker-tira-model:
running make run-tira-docker-dev runs into an error:

I "fixed" this locally, by commenting out the corresonding line in the Dockerfile
The container is running after this, but I still get an error in  step 19:

This is the reason, I did not commit yet, because I am not sure, just commenting out was the right fix.
Any suggestions?

(a) When you start the container, mount (-v) the tira/tira-model from the outside to /mnt/nfs/tira inside the container.
(b) add the necessary mkdirs and touches to the build script (meh)
(c) move the dockerfile out of the tira-host scope into the tira scope, so both tira-host and tira-model are in the context for copying. If we do this, we should move all build utility from all modules in the tira scope.

14890986b2dff70c2787f782c474cdb342c08678
Deleting the copying-step of the model to /mnt/nfs/tira was ok, because we already mount the locally mounted tira model from outside he container via Makefile to inside the container.
the error in step 19 is still not fixed, but does not stop the container from working: we tested vm create within the local container and logged in via the interface (the application needs a restart to accept the new user's login).

We have decided to leave the hosts dev setup as it is right now.
A completely local setup is currently difficult due to dnsmasq and the routing of ssh requests into the VM. Without an update to the networking of the VMs, the host's local setup cant be simplified.
We recommend developing the tira-host on a tira-host machine (see the host salt state or contact devs to get it).


action after template will render

How can i track when template is ready to do some action after that?
For example:
FlowRouter.route("/time", {
    action: () => {
        mount(Layout, {
            content: (<Time />)
        });
        console.log(document.getElementById('react-root'));
    }
});

Always return null

With the lifecycle of your <Layout> if it's shared with all routes or the <Time> if it's specific to your time route. Check componentDidMount.


a better way to wait for uninstall or undeploy

@sschneid looking for a review

👍 , this looks good aside from the minor indenting comments.


drop very old versions, add GHA workflow to build & test images

Related to #56.

Drops pre 6.4 builds. We could go as far as to drop builds up to 7.4, but I'm curious what you think, @jackfirth.
Builds images in parallel by series (6x, 7x, and 8x), reducing the build time.
Changes the base image to debian:stable-slim, which reduces the total size of each final image by about 500MB.
Adds a GHA workflow to build, test & deploy the changes. Deployments only happen on pushes to the master branch, and you'll have to add the DOCKER_USER_USERNAME and DOCKER_USER_PASSWORD secrets to the repo for this to work.
Drops the Circle CI config.

There's one potential breaking change here in the switch to debian:stable-slim from buildpack:stable. The latter has more "stuff," so some users of the images may depend on that stuff. IMO, it's still worth trimming the images down.

I'm broadly in favor of all of these changes. I want to keep the 6.4-7.4 builds since older versions can help determine when some change in Racket's behavior occurred. I've created a deployment environment with the DOCKER_USER_USERNAME and DOCKER_USER_PASSWORD secrets in it. Do you need to update the action configuration to get them to run in that environment?

Sounds good. I've split the workflow into one that deploys merges to master using the env you created and one that builds & tests PRs. I think this should be good to go now.


Package.json: why dependencies?

I'm submitting a ...
[x ] question about the decisions made in the repository

What is the current behavior?
The package.json file lists project dependencies.
I noticed that a lot of the "project" packages, for example @angular/*  are  listed in the dependencies section.

I assume that when a project is built, all dependencies are put into the bundle files.
Therefore I would assume that all the files in the "dependencies" section really should be in the "devDependencies" section.
Let's for example say that you want to add a node backend (which is reasonable to think).
You would install packages for the backend, and when you deploy the application you would copy your backend source to the destination server, and then run "npm install --production"
This would effectively install all the @angular/* packages in the production environment. A behaviour you don't want. Npm install --production installs all packages listed in the "dependencies" section.
So my question is:
Is there something I've completely missed - or should we not move all packages into the "devDependencies" section?
If not, how would you distinct packages to be installed in a production environment, when adding a node backend?
Cheers, guys, really appreciate this project, it has helped me tremendously in setting up the exact environment I need for my projects.

Hi, i think is more about concept than about actual use, off course you can put @angular dependencies in dev since most users just install in production by uploading the bundled files and installing the server deps.
The concept i talk is that devDeps are the deps you need during development (mostly webpack, browserify, gulp...) and in normal deps goes the ones your app really use (like @angular/core) so it's about stating in the package that deps are the ones your app requires and devDeps are the ones you use to bundle, compile or deploy your app.

I understand this - however once you try to setup a node backend in the same project, you'll be in trouble.
This is mostly and information to newbies: if you use this excellent repo as a starter for your project, and you also need a backend as part of the project, do yourself a favor and move all dependencies to the "devDependencies" section.
Otherwise you end up installing a lot of modules in your production environment that you won't need.
I will close this issue now ;)


add log() for battery voltage, RIO voltage, and battery current draw

The Rev PDP should be used. Use its classes to read in the values and use log() to record them to file and to dashboard

Unlikely to malfunction :+1:


Modern Desktop Lab Kit fails to extract and complete

The lab kit fails to fully extract and complete the automated setup. Despite reaching 100% only a sample of the VM's were created. The setup wizard is displaying the following constant error as it retries.
"Cannot validate argument on parameter 'ParameterComplete'. The 129 argument is greater than the maximum allowed range of 100. Supply an argument that is less than or equal to 100 and then try the command again."

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

ID: 21195f1c-0f99-502b-c6e7-a87d35237a81
Version Independent ID: 1848a7e1-4919-8e28-27ea-cfdbef125609
Content: Windows and Office Deployment Lab Kit - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/modern-desktop-deployment-and-management-lab.md
Service: o365-solutions
GitHub Login: @JGruszczyk
Microsoft Alias: jogruszc

Hi - it appears the host machine either has too little CPU or storage, probably storage. I've verified this works on fine on our end, however we’ll add a ‘troubleshooting’ section to the next version of the guide to help with this. Thanks!


Buggy DatePicker inside of Dialog

Description
When I place DatePicker inside of a Dialog and I click on the DatePickerTrigger the layout will render. If I select prev/next or click the trigger that should change layout view to month the DatePicker will automatically close.
This weird behaviour is only in Safari, but in other browsers there's another bug that's related. Setting the min/max dates and clicking on disabled "prev/next" action or clicking anywhere near the action that changes layout view (from day to month for example) will auto close the DatePicker as well.
This behaviour is only present when DatePicker is used within Dialog. Not when DatePicker is used directly without component nesting.
Link to Reproduction (or Detailed Explanation)
https://stackblitz.com/edit/vitejs-vite-qaltgb?file=src%2FApp.vue
Steps to Reproduce

Click "Click to Open"
Click "Show datepicker"
Click on "September 2024" to change layout view

In Safari, DatePicker will auto-close.

Click near "September 2024" on white background

In Chrome, DatePicker will auto-close.

Ark UI Version
3.13.1
Framework

[ ] React
[ ] Solid
[X] Vue

Browser
Chrome, Safari
Additional Information
No response

Temp. A fix for this is to assign tabIndex to the date picker content. We'll fix that in Zag.js shortly

I just pushed a fix for this in Zag.js. We'll release an update shortly.
If the issue persists after upgrading, I'll re-open it.


webpack devserver的after配置，rspack有相关功能的实现吗？目的是加入自定义中间件

What problem does this feature solve?

What does the proposed API of configuration look like?
webpack.devserver.after


Script looping start when add '.' between decimal values

Script looping start when add '.' between decimal values
e.g.
if add 1000 and try to add '.' after 10, then its start looping
but its working fine while if we add like 10.00 continuously
i m using this script
$('.floatval').inputmask("decimal",{digits:2});

@ankurdhrusoft ,
I will have a look. Thx for reporting.

Yeah, I noticed this as well.
It occurs at the following code in the setValidPosition function of inputmask when vps[i].input is ".":
if (void 0 != t) for (var posMatch = j; posMatch < getMaskLength() && (null == t.match.fn && vps[i] && (vps[i].match.optionalQuantifier === !0 || vps[i].match.optionality === !0) || null != t.match.fn); ) {
    if (null == t.match.fn || !opts.keepStatic && vps[i] && (void 0 != vps[i + 1] && getTests(i + 1, vps[i].locator.slice(), i).length > 1 || void 0 != vps[i].alternation) ? posMatch++ : posMatch = seekNext(j), positionCanMatchDefinition(posMatch, t.match.def)) {
        valid = isValid(posMatch, t.input, !0, !0) !== !1, j = posMatch;
        break;
    }
    valid = null == t.match.fn;
 }
```

Thank u very much for very nice support...
Your input validation plugin is awesome.... [?]
But do not understand, how can i fixed this [?]
On Thu, Apr 23, 2015 at 3:17 AM, jmanos<EMAIL_ADDRESS>wrote:

Yeah, I noticed this as well.
It occurs at the following code in the setValidPosition function of
inputmask when vps[i].input is ".":
if (void 0 != t) for (var posMatch = j; posMatch < getMaskLength() && (null == t.match.fn && vps[i] && (vps[i].match.optionalQuantifier === !0 || vps[i].match.optionality === !0) || null != t.match.fn); ) {
if (null == t.match.fn || !opts.keepStatic && vps[i] && (void 0 != vps[i + 1] && getTests(i + 1, vps[i].locator.slice(), i).length > 1 || void 0 != vps[i].alternation) ? posMatch++ : posMatch = seekNext(j), positionCanMatchDefinition(posMatch, t.mat
ch.def))
{
valid = isValid(posMatch, t.input, !0, !0) !== !1, j = posMatch;
break;
}
valid = null == t.match.fn;
}
—
Reply to this email directly or view it on GitHub
https://github.com/RobinHerbots/jquery.inputmask/issues/870#issuecomment-95347510
.

--
Love & Prayer
Ankur

I will have a look at it next week.
My dauhter Tess is born past thursday, so I have other priorities for the moment :-)

Robin- I just want to point you in the right direction with this issue as it is something that we have had a problem in our company with as well and we have done our best to do a quick fix before you are able to fix it yourself- I expect you know the code pretty much inside out afterall!
So, I found that there is an endless loop (line 238) which appears to be caused because the code "positionCanMatchDefinition(posMatch, t.match.def)" (line 242) does not return TRUE and so the for loop does not break.
To reproduce the issue:

Set decimal masking on a field
Enter 2 digits, e.g. 56
Then enter a decimal between the 2 digits e.g 5.6
It should then enter a continuous loop and the page freezes

I hope this help.

Thx for the info.  A fix will follow soon.

@ankurdhrusoft, @jpontet ,
Fixed in the latest commit

@RobinHerbots thanks!


(maint): fix lookup option spelling to be consistent with code and documentation

Can one of the admins verify this patch?

Hi @keeckle could you please edit your commit message to start with (maint) so that our tests pass? Thanks!

Complete.

Should be good to go.
On Mon, May 6, 2019 at 12:56 PM Jorie Tappa<EMAIL_ADDRESS>wrote:

Hi @keeckle https://github.com/keeckle could you please edit your
commit message to start with (maint) so that our tests pass? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/puppetlabs/puppet/pull/7493#issuecomment-489692502,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHBUKCVQKDEG7KG73ZZCJLDPUBPKPANCNFSM4HIEZGOQ
.

Looks like the three commits just need to be squashed down into one, if you need help going through that process, let me know!

Hi @keeckle, if you look at https://github.com/puppetlabs/puppet/pull/7493/commits it appears there's still three separate commits there. Let me know if you'd like help with this.


upgrade packages in go.mod for KMC

This PR bumps the dependencies and the replaces in the go.mod of KMC to prevent potential security vulnerabilities.

/retest

/retest

/retest

/retest

/retest

/retest

Closed in favour of
https://github.com/kyma-project/control-plane/pull/1608


refactor: Clean up Account Settings Object Storage

Description 📝

Refactors the Object Storage section of the https://cloud.linode.com/account/settings page 🔁
Adds a React Query mutation called useCancelObjectStorageMutation so we don't have to manage our own loading and error states 🧼 😃
Renames the component from EnableObjectStorage to  ObjectStorageSettings ✏

EnableObjectStorage was very confusing. This component's main purpose is to allow users to disable object storage

Adds a new toast for when you successfully disable object storage 🍞
Updates the unit tests to account for changes 🧪

Preview 📷

[!note]
UI should not look any different. All changes were to just clean up the code and use React Query for the mutation

https://github.com/user-attachments/assets/d1c11ff9-cbe8-40fe-8665-d0a236011554
How to test 🧪

Test the disable Object Storage flow on http://localhost:3000/account/settings

The enable-object-storage.spec.ts covers this flow so ensure that test passes ✅

As an Author I have considered 🤔

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[ ] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Renames the component from EnableObjectStorage to ObjectStorageSettings ✏
EnableObjectStorage was very confusing. This component's main purpose is to allow users to disable object storage

Thank you! 🙏🏼 This has tripped me up more than once.


Merge to live!

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/OfficeDev/office-js-docs-pr, working directory: W:\nnsz-s, target branch: live, target branch commit: 7edf455853f20136c51bab61e79ecf90b0fb8cb9, source branch: master, source branch commit: .

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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

README.md
:white_check_mark:Succeeded

docs/develop/implementation-details.md
:white_check_mark:Succeeded

docs/develop/office-js-versioning.md
:white_check_mark:Succeeded

docs/develop/office-versions.md
:white_check_mark:Succeeded

docs/excel/custom-functions-overview.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.


i can see the stats,quest,log from my friend but mine doesnt upload

pls help me out my stats dont update , been busy with it for 24 hours , i reloaded page removed downloaded twice chek the token and everything !

No need to open multiple issues for this.

Make sure you have the correct plugin installed: https://runelite.net/plugin-hub/show/group-ironmen-tracker
Make sure the Group Name and Group Token fields in the plugin settings are correct. Copy them from https://groupiron.men/setup-instructions
Remove anything from the Self Hosted Config section in the plugin settings
Your in game name and the character name on the website must match exactly. This is case sensitive

No need to open multiple issues for this.

Make sure you have the correct plugin installed: https://runelite.net/plugin-hub/show/group-ironmen-tracker
Make sure the Group Name and Group Token fields in the plugin settings are correct. Copy them from https://groupiron.men/setup-instructions
Remove anything from the Self Hosted Config section in the plugin settings
Your in game name and the character name on the website must match exactly. This is case sensitive

ty<3 sorryyy


Sema: Allow @objc enums without -enable-objc-interop or importing Foundation [5.5]

We used to allow @objc enums on Linux, where Objective-C interop is disabled,
because the check first looked for an import of the Foundation module
(which could be the non-Objective-C corelibs-foundation) before checking
if Objective-C interop was enabled.
Since @objc enums don't actually depend on Foundation or Objective-C and
only have a different layout in IRGen, let's bring back and formalize the
old behavior.
Fixes https://bugs.swift.org/browse/SR-14548 / rdar://problem/77325078.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 75a9bfccde12a510b31fcccc9e95b8f0146832b9

@swift-ci Please test Linux