stack broken on kretprobe

May be broken on uretprobe as well. For example:
# ./src/bpftrace -e 'kprobe:__alloc_fd { @entry[stack] = count(); } kretprobe:__alloc_fd { @return[stack] = count(); }'
Attaching 2 probes...
^C

@entry[
__alloc_fd+1
__do_pipe_flags+87
do_pipe2+45
__x64_sys_pipe+16
do_syscall_64+85
entry_SYSCALL_64_after_hwframe+68
]: 1
@entry[
__alloc_fd+1
do_sys_open+273
do_syscall_64+85
entry_SYSCALL_64_after_hwframe+68
]: 1
@entry[
__alloc_fd+1
__do_pipe_flags+73
do_pipe2+45
__x64_sys_pipe+16
do_syscall_64+85
entry_SYSCALL_64_after_hwframe+68
]: 1
@entry[
__alloc_fd+1
do_sys_open+273
do_syscall_64+85
entry_SYSCALL_64_after_hwframe+68
]: 1
@entry[
__alloc_fd+1
do_sys_open+273
do_syscall_64+85
entry_SYSCALL_64_after_hwframe+68
]: 4
@entry[
__alloc_fd+1
do_sys_open+273
do_syscall_64+85
entry_SYSCALL_64_after_hwframe+68
]: 9
@entry[
__alloc_fd+1
do_sys_open+273
do_syscall_64+85
entry_SYSCALL_64_after_hwframe+68
]: 13
@entry[
__alloc_fd+1
__sys_socket+114
__x64_sys_socket+22
do_syscall_64+85
entry_SYSCALL_64_after_hwframe+68
]: 14

@return[
kretprobe_trampoline+0
]: 1
@return[
kretprobe_trampoline+0
]: 2
@return[
kretprobe_trampoline+0
]: 4
@return[
kretprobe_trampoline+0
]: 6
@return[
kretprobe_trampoline+0
]: 9
@return[
kretprobe_trampoline+0
]: 27

The stack shouldn't change between the kprobe (entry) and kretprobe (return). But on the return we just get this kretprobe_trampoline.

I've also seen this if you manually instrument kprobe_events -- $stack on a kretprobe results in a garbage stack (with only the entry [unknown/kretprobe'd] -- which is kretprobe_trampoline). Is there any particular place that you would recommend looking at if I wanted to help fix this issue (I've needed the ability to get a stack trace from a kretprobe in the past so this is quite useful to me).

I looked into this a bit more. It's actually quite conceptually easy to do with the current kretprobe design -- all you have to do is save the stack trace inside pre_handler_kretprobe (kretprobes are actually kprobes that set up a kretprobe once they get triggered -- which means that we can just get a stack trace during the code which sets up the kretprobe and then we can return that if a user asks us for a stack trace during the kretprobe). So getting a stack trace on function entry is very doable.
The problem is how we get that information to our BPF tracer. Getting it to ftrace and all the other kernel tracers (which all appear to be implementing somewhat similar concepts but differently -- presumably because they were all developed separately or because they have strong performance reasons for doing so) appears to be possible -- but getting it to the BPF program is not as easy.
Effectively we need bpf_get_stackid (I believe) to sample from a different stack than the current one -- which means that we have to get a reference to kretprobe_instance. We get this for free in kretprobe_perf_func. But the BPF program doesn't have access to that -- trace_call_bpf doesn't take nor give that information. We'd have to modify what we pass in ctx to allow that -- but given that it's all done with void * I'm quite scared of breaking lots of things by touching it.
Also, bpf_get_stackid uses the perf_events stack trace code which appears to be somewhat different from the /proc/self/stack stack trace code -- they both use the same orc unwinder on x86 but they have different interfaces for some reason. It's not clear to me which we should use for kretprobe_instance given that ftrace also has its own stack-getting code...

Thanks for digging in @cyphar . Sounds like a kernel change, and you know more about fixing this right now than anyone, so if you can I'd try it out! I'm guessing the patch would be submitted via BPF https://github.com/torvalds/linux/blob/master/MAINTAINERS#L2810 , depending on what changes.

Okay, I have a working patch. You can see it in my tree. The basic idea is that we stash away the stack trace on kprobe entry and then use it within the tracing code if the tracing code is executing from within a kretprobe.

Saving the stack trace was NACKed by upstream. However, I've figured out how to make the unwinders produce reasonable stack traces... but unfortunately this doesn't produce identical traces to syscall entry (my original idea was flawed but with the new code could be useful).
So yeah, this requires a bit more work.

I think the kernel issue has been fixed by https://lore.kernel.org/all/163163049242.489837.11970969750993364293.stgit@devnote2/T/#u
Linux v5.16 contains the fix.
For older kernels like v5.4, CONFIG_UNWINDER_FRAME_POINTER=y is likely to be a workaround.

Closing this as it was fixed in 5.16 and we have a workaround for earlier kernels (mentioned above).


[BUG] Unable start VM after upgraded v1.0.1 to v1.0.2-rc2

Describe the bug
In the case of single node upgrade, the VM will be shutdown in Off State and unable to start again.
In the case of 2 nodes upgrade, the VM will stay in Running State, but after Stop it, it would not able to start again.

To Reproduce
Steps to reproduce the behavior:

Install Harvester v1.0.1 with any nodes
Create an image for VM creation
Create the VM vm1 and wait it be Running state.
Perform upgrade to Harvester v1.0.2-rc2
Upgrade should success.
Navigate to Virtual Machines page, Stop vm1 then Start it again
VM must Start without errors and into Running state

Expected behavior
VM should able to start as expected.
Environment:

Harvester ISO version: v1.0.1 to v1.0.2-rc2
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): qemu/KVM single node and 2 nodes

Additional context
There have a workaround to update yaml configuration as following:
spec:
  running: false

to
spec:
  runStrategy: halted

As this bug had a simple workaround work well, I would suggest to fix it in v1.0.3 or later.
(And please help to update priority label and milestone, thanks)

closed as this only affects upgrading v1.0.1 to v1.0.2.


docs: add build-in pipeline greptime_identity doc

What's Changed in this PR

add build-in pipeline greptime_identity doc

Checklist

[ ] Please confirm that all corresponding versions of the documents have been revised.
[ ] Please ensure that the content in sidebars.ts matches the current document structure when you changed the document structure.
[ ] This change requires follow-up update in localized docs.

In my opinion, greptime_identity is more like 'try to recognize the JSON-formatted data's schema and transform them accordingly' rather than a built-in pipeline. The pipeline name is just a way to enter this mode, it's not actually a pipeline.
We can say the greptime_ prefix of in pipeline name is reserved.

In my opinion, greptime_identity is more like 'try to recognize the JSON-formatted data's schema and transform them accordingly' rather than a built-in pipeline. The pipeline name is just a way to enter this mode, it's not actually a pipeline.
We can say the greptime_ prefix of in pipeline name is reserved.

It's true that this is not a pipeline from an implementation point of view, but thinking of it as a pipeline helps reduce the cost of understanding. It's something else that doesn't matter what it's called, but it can be used to transform user-submitted json data as is, just like a pipeline.


fix rtl plugin documentation

Just using rtl() directly appears to throw a type error.

[x] I have followed (at least) the PR section of the contributing guide.

@simoncarpinter Should the TypeScript definitions be fixed instead of the usage? Do we have the same problem with the v4 version? The TypeScript definitions changed.

Just using rtl() directly appears to throw a type

@simoncarpinter  A TypeError at runtime or does it report an error during compilation? Please include the error message. It might be an issue with the typescript types in the jss package.


When I run python run_inter3d.py --verbal=True --instance_counter_id=1 --number_of_instances=1 --cubeedge=0.05 --pretraining_weights='/media/kontogianni/Samsung_T5/intobjseg/datasets/scannet_official/weights/exp_14_limited_classes/weights_exp14_1 4 .pth' --dataset='scannet' --visual=True --save_results_file=True --results_file_name=results_scannet_mini.txt Such an error occurs, what may be the reason? No error was reported when I installed MinkowskiEngine. My cuda version is 11.1, and MinkowskiEngine is version 0.5.4.

Traceback (most recent call last):
File "/home/wyy/git_code/InterObject3D_main/InterObject3D/Minkowski/training/run_inter3d.py", line 221, in
main(config)
File "/home/wyy/git_code/InterObject3D_main/InterObject3D/Minkowski/training/run_inter3d.py", line 87, in main
pred, logits = inseg_model_class.prediction(feats.float(), coords.cpu().numpy(), inseg_global_model, device)
File "/home/wyy/git_code/InterObject3D_main/InterObject3D/Minkowski/training/interactive_adaptation/interactive_adaptation.py", line 369, in prediction
sinput = ME.SparseTensor(
File "/home/wyy/anaconda3/envs/me/lib/python3.9/site-packages/MinkowskiEngine-0.5.4-py3.9-linux-x86_64.egg/MinkowskiEngine/MinkowskiSparseTensor.py", line 275, in init
coordinates, features, coordinate_map_key = self.initialize_coordinates(
File "/home/wyy/anaconda3/envs/me/lib/python3.9/site-packages/MinkowskiEngine-0.5.4-py3.9-linux-x86_64.egg/MinkowskiEngine/MinkowskiSparseTensor.py", line 338, in initialize_coordinates
features = spmm_avg.apply(self.inverse_mapping, cols, size, features)
File "/home/wyy/anaconda3/envs/me/lib/python3.9/site-packages/MinkowskiEngine-0.5.4-py3.9-linux-x86_64.egg/MinkowskiEngine/sparse_matrix_functions.py", line 183, in forward
result, COO, vals = spmm_average(
File "/home/wyy/anaconda3/envs/me/lib/python3.9/site-packages/MinkowskiEngine-0.5.4-py3.9-linux-x86_64.egg/MinkowskiEngine/sparse_matrix_functions.py", line 93, in spmm_average
result, COO, vals = MEB.coo_spmm_average_int32(
RuntimeError:  at /home/wyy/MinkowskiEngine/src/gpu.cu:100

Which pytorch version do you use? Python3.9 and Pytorch 1.10.2 should be ok! Do you get any errors when you just import the ME (import MinkowskiEngine as ME)? Another thing to check is what is your "device" and which GPU are you using?

Hello,My pytorch version is 1.9.0, python is 3.9, when I just import the ME, there is no problem, my device uses cuda, and my GPU model is NVIDIA GeForce RTX 4090

Hello, my problem is solved, I uninstalled all the previously installed environments. Now my cuda version is 11.6, python version 11.9, pytorch version is 1.12.1, MinkowskiEngine is 0.5.4.


[Substrate.io Migration] - Polkadot Protocol > Protocol Operations > Off-Chain Operations

This PR aims to migrate this page: https://docs.substrate.io/learn/offchain-operations/

Ok, friends! Did a round of grammar/formatting and editing to this one and added some mermaid diagrams. Let's see what you think!

@eshaben I moved this copy to the Polkadot SDK page for the moment and have this specific file tagged with TODO to delete once the new copy is approved. Needs a grammar and formatting pass from you, please.

@eshaben I moved this copy to the Polkadot SDK page for the moment and have this specific file tagged with TODO to delete once the new copy is approved. Needs a grammar and formatting pass from you, please. WDYT of Further Resources as an interface admonition at the end of each section?

@dawnkelly09 shall we close this PR then?

@dawnkelly09 Also, I moved the TODO under the title and description labels; otherwise, it conflicts with the rendering of the page:

Closing this. If we decide down the road that this info is relevant, we can always refer back to this PR


Add support to version TOSCA definitions

Add support to version and diff TOSCA Definitions. See https://github.com/OpenTOSCA/winery/pull/59 for UI changes.

[x] Ensure that you followed http://eclipse.github.io/winery/dev/ToolChain#github---prepare-pull-request. Especially, we require a single commit
[x] Ensure that the commit message is a good commit message
[x] Ensure to use auto format in all files
[x] Change in CHANGELOG.md described
[x] Tests created for changes
[x] Screenshots added (for UI changes) see https://github.com/OpenTOSCA/winery/pull/59

Can one of the admins verify this patch?

Fixed in https://github.com/eclipse/winery/commit/f6a5c132d60e398566c472338fa14e4aaf18ee0c


[WASAPI] ERole == eMultimedia should not be used

While re-reading some WASAPI docs, I found this:

The client must also specify the appropriate role for the endpoint in the
ERole attribute (eConsole or eCommunications). Do not use eMultimedia.

We're using eMultimedia, and we've never had issues, but maybe we could reconsider.

Yeah, I've run into this several times before when debugging and wondering if some weird behaviour was related to that (but never was).  AFAIK there's no reason we are using eMultimedia, so let's change it.

Fixed in 05abb60b99a46d1d21abe58d9e5f3bdb4f0df4a8.


Inference code

Hi, could you please add code where inference is done on image/video/webcam? what is the output rate or this model?

I am also trying inference, but I can't unrar the downloaded models. any help there?

@tianfanzhu  @MetaDev  You can simply modify test.py and net_modules.py to do inference, there isn't anything new.

Now close this issue, I would re-open it if new questions come out.

I figured it out. I had to use the "result_func" and "debug_vis_patch", the code is pretty clear.

can you tell me how to inference a img?>


Terminal hangs when typing a command

Version: 1.38.1
Commit: b37e54c98e1a74ba89e03073e5a3761284e3ffb0
Date: 2019-09-11T13:30:08.229Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Linux x64 4.19.71

Steps to Reproduce:

Type cachix push --help into terminal and get the following output:

Usage: cachix push [--compression-level [0..9]] NAME ([PATHS...] |
                   [-w|--watch-store])
  Upload Nix store paths to the binary cache

Available options:
  -h,--help                Show this help text
  --compression-level [0..9]
                           The compression level for XZ compression. Take
                           compressor *and* decompressor memory usage into
                           account before using [7..9]! (default: 2)
  -w,--watch-store         Run in daemon mode and push store paths as they are
                           added to /nix/store

Observe that window hangs
Task manager shows that current file memory grows from 300MB to 2GB and CPU goes up to 70%
Window is no longer responding

Does this issue occur when all extensions are disabled?: Yes

/needsMoreInfo
Does this also happen with VS Code Insiders? Download: https://code.visualstudio.com/insiders/

I can not use Insiders, is there something else I can do?

Task manager shows that current file memory grows from 300MB to 2GB and CPU goes up to 70%

What is the current file? Can you show a screenshot of this?

before

after

Does typing in the terminal slow things as well? Does this setting change things?
"terminal.integrated.rendererType": "dom"

I experience the same issue as @domenkozar explained. The open file does not matter. This happens even if there is no one.

Please note, that I typed in console commands blazingly fast, it renders letter by letter with approx. 1 sec delay. Also note, that closing terminal panel makes VS Code fast again. And the last thing to note is that it does not happen all the time. If I restart VS Code it runs OK. I experience this approximately once a day.
Switching to the DOM renderer seems to help. I did this and everything became fine even without restarting VS Code.
.

I suspect this is the electron 4 canvas regression here https://github.com/microsoft/vscode/issues/81047. We're working on updating to v6 but in the meantime you can use the dom renderer.
/duplicate https://github.com/microsoft/vscode/issues/81047


Will android/arm64 be supported in the future?

I just want to use ffmpeg on quest2.

I don't see why it wouldn't compile, but I have no experience building for ARM

This would be easy when switching to CMake see https://github.com/asmaloney/GDExtensionTemplate. Then we could simply use vcpkg and compile ffmpeg ourselves. The following example is for Android but would be similar for iOS:

Download ndk r25c here. The newest 26 does not work for me, and r25c should be totally fine.
./vcpkg install ffmpeg:arm64-android
Add vcpkg to your toolchain file like so:

        "CMAKE_TOOLCHAIN_FILE": "${sourceDir}/../vcpkg/scripts/buildsystems/vcpkg.cmake",

The problem with switching to cmake is that I have no experience with it and I want to keep this repository buildable as an engine module, which is a feature i myself need.

any updates on this? i need this for macos arm64 but i cant do anything...

are there any plans to add compatibility for android in the near future? Im making a file organizing/tagging app in godot and it would be really useful


feat: Explore page with snapshot spaces

Summary
Closes: #297
How to test
will add more details later
To-Do

[ ] Fix Pagination
[ ] Fix issue while switching from snapshot to snapshot x quickly

Can we show the badges too on space preview for Snapshot spaces?

Also can we use abbreviation for proposal and vote counters? With max 3 numbers like 652, 1.25m, 2.51k


If a text is packed in  in the WYSIWYG editor, the text disappears

If you insert and edit a text in the Shopping Experiences with the WYSIWYG editor, everything works. However, if you then switch to the HTML editor and write a  around the text, it disappears.
Steps to reproduce:

Create a Shopping Experiences
Insert a text block
Insert/write a text in this block
Switch to the HTML editor
insert a  before the text and a  after the text

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-38471
[created from NEXT-38471, comment 498183]


Fixed a typo

Under the encrypt a virtual machine scale set one of the prerequisites stated a vm rather than a virtual machine scale set.

@PixelRobots : Thanks for your contribution! The author, @msmbaldwin, has been notified to review your proposed change.

Good catch, @pixelrobots! I'll ask that this PR be merged, and I will update the Encrypt virtual machine scale sets with Azure CLI section as well. #sign-off


button.pressed implementation

button.pressed is a client register, do we have a scheme to implement those?

kind of. we have a way of implementing client commands but currently we have no way of attaching information to roles and also attaching generic event handlers which is what this one would require


[indexerv2] Array<Nullable> for validators on StoredEpochInfo?

Description
Did we want validators array to be Vec-Vec-u8 or Vec-Option-Vec-u8? Noticed that the diesel schema is Array<Nullable> but defined differently in Rust struct
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

@wlmyng this is expected. In the create statement, it's
    validators                      bytea[]     NOT NULL,

diesel translates that to
validators -> Array<Nullable<Bytea>>,

because the item in the array is nullable in postgresql.
But in our application, the validator item is never going to Null, so we don't have the final Option

@longbowlu that makes sense to me, but I could not get the following query to work until I modified the typing:
    pub(crate) async fn fetch_epoch(&self, epoch_id: Option<u64>) -> Result<Option<StoredEpochInfo>> {
        let mut query = epochs::dsl::epochs.into_boxed();

        if let Some(e) = epoch_id {
            let bigint_e = i64::try_from(e).map_err(|_| Error::Internal("Failed to convert epoch to i64".to_string()))?;
            query = query.filter(epochs::dsl::epoch.eq(bigint_e));
        } else {
            query = query.order(epochs::dsl::epoch.desc()).limit(1);
        }

        Ok(read_only_blocking!(&self.pool, |conn| {
            query.first::<StoredEpochInfo>(conn)
        }).map_err(Error::from)?)
    }

It looks like Diesel expects Vec<Option<Vec>> even though as you've explained this will never happen in our actual data

@longbowlu that makes sense to me, but I could not get the following query to work until I modified the typing:
    pub(crate) async fn fetch_epoch(&self, epoch_id: Option<u64>) -> Result<Option<StoredEpochInfo>> {
        let mut query = epochs::dsl::epochs.into_boxed();

        if let Some(e) = epoch_id {
            let bigint_e = i64::try_from(e).map_err(|_| Error::Internal("Failed to convert epoch to i64".to_string()))?;
            query = query.filter(epochs::dsl::epoch.eq(bigint_e));
        } else {
            query = query.order(epochs::dsl::epoch.desc()).limit(1);
        }

        Ok(read_only_blocking!(&self.pool, |conn| {
            query.first::<StoredEpochInfo>(conn)
        }).map_err(Error::from)?)
    }

It looks like Diesel expects Vec<Option> even though as you've explained this will never happen in our actual data

I see, makes sense!


Add Ireland

For information, read and make sure you're okay with the CONTRIBUTING document.

[x] Tests with a significant number of years to be tested for your calendar.
[x] Docstrings for the Calendar class and specific methods.
[x] Calendar country / label added to the README.rst file,
[ ] Changelog amended with a mention like: "Added <country> by @pseudo (#)"

Don't give up! your contribution is close to be accepted, just a few fixes and it's going to be fine.

Excellent! welcome to workalendar, Ireland!


AttributeError: module 'torch_mlir' has no attribute 'compile'

When I tried to run the demo 'examples/bert.py', it reported 'AttributeError: module 'torch_mlir' has no attribute 'compile'. I have installed required packages.

Did you install torch-mlir via pypi instead of from https://llvm.github.io/torch-mlir/package-index/ by chance?  The version on PyPI is just a dummy release right now until we get PyPI publishing working correctly (unfortunately I had to create a dummy release in order to request PyPI to up our per-release file limit; I had a feeling that might be confusing).

There is a colab notebook I tried to run, which is linked in the iree documentation (the pytorch part), and it display that same error as well.
-> https://colab.research.google.com/github/iree-org/iree-torch/blob/main/examples/bert.ipynb
Not sure who owns it and how to fix it.
The install steps use the llvm/torch-mlir way of installing things btw.
pip install -f https://llvm.github.io/torch-mlir/package-index/ torch-mlir

ps: that second line looks really strange to me, the index having an html part. (pip-release-links.html)
!pip install -f https://iree-org.github.io/iree/pip-release-links.html iree-compiler iree-runtime

I also tried to use the daily torch_mlir listed here -> https://github.com/llvm/torch-mlir/releases
After a pip install using this, I can see 'files', but still no torch_mlir.
(venv) ~$ find venv | grep mlir
venv/lib/python3.9/site-packages/torch_mlir-0.0.0.dist-info
venv/lib/python3.9/site-packages/torch_mlir-0.0.0.dist-info/RECORD
venv/lib/python3.9/site-packages/torch_mlir-0.0.0.dist-info/LICENSE
venv/lib/python3.9/site-packages/torch_mlir-0.0.0.dist-info/WHEEL
venv/lib/python3.9/site-packages/torch_mlir-0.0.0.dist-info/top_level.txt
venv/lib/python3.9/site-packages/torch_mlir-0.0.0.dist-info/REQUESTED
venv/lib/python3.9/site-packages/torch_mlir-0.0.0.dist-info/INSTALLER
venv/lib/python3.9/site-packages/torch_mlir-0.0.0.dist-info/METADATA
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlirHlo.cpython-39-darwin.so
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlirLinalgPasses.cpython-39-darwin.so
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/__init__.py
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/__pycache__
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/__pycache__/__init__.cpython-39.pyc
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_ireeDialects.cpython-39-darwin.so
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_ireecTransforms.cpython-39-darwin.so
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlir
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlir/ir.pyi
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlir/__init__.pyi
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlir/passmanager.pyi
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlir/dialects
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlir/dialects/pdl.pyi
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlirDialectsLinalg.cpython-39-darwin.so
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/libIREECompilerAggregateCAPI.dylib
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlir.cpython-39-darwin.so
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/iree-lld
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/iree-compile
venv/lib/python3.9/site-packages/iree/compiler/_mlir_libs/_mlirDialectsPDL.cpython-39-darwin.so
venv/lib/python3.9/site-packages/torch_mlir
venv/lib/python3.9/site-packages/torch_mlir/__init__.py
venv/lib/python3.9/site-packages/torch_mlir/__pycache__
venv/lib/python3.9/site-packages/torch_mlir/__pycache__/__init__.cpython-39.pyc

Thanks. I have actually cloned this repo (iree-torch) and I am installing the deps directly by reading the requirement file, that works better but now I'm getting an error installing pillow ... I'm using the macbrew python (3.9), in recent version of osx I think there's a bundled 3.8 python, will try that one ...

Thanks @bsergean, I'm getting the same error when I try running the notebook too.  Will look into it.

I think this might be because colab rolled the default python version to 3.8.

I've deleted the dummy 0.0.0 install from pypi, so at the very least that confusion should be gone.  I still see it when I'm trying to install, so it might take some time for caches to appear.  @powderluv is going to build new packages for the new Python versions, which should fix this problem.

Thanks for the heads up. That actually won't fix it for you then. I think we need to consider building a generic torch-mlir Python package either in addition to or instead of trying to build for a handful of specific Python versions.  /cc @powderluv

I can install python 3.10 and try with this version. There’s something called py-env which makes switching python version easier those days, but I haven’t tried it on macOS yet.

On Dec 7, 2022, at 9:58 AM, Daniel Ellis @.***> wrote:
Thanks for the heads up. That actually won't fix it for you then. I think we need to consider building a generic torch-mlir Python package either in addition to or instead of trying to build for a handful of specific Python versions. /cc @powderluv https://github.com/powderluv
—
Reply to this email directly, view it on GitHub https://github.com/iree-org/iree-torch/issues/65#issuecomment-1341355311, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC2O6UKCLVK6HCX554OC5ULWMDF5ZANCNFSM6AAAAAASOMUZN4.
You are receiving this because you were mentioned.

I'm currently seeing a similar error in IREE's samples build:
/usr/bin/clang++-9 main.c -flto -O2 -I../../runtime/src/ -L/tmp/iree-build-runtime//build_tools/third_party/flatcc/ -L/tmp/iree-build-runtime//runtime/src/iree/runtime/ -L/tmp/iree-build-runtime//third_party/cpuinfo/ -L/tmp/iree-build-runtime//third_party/cpuinfo/deps/clog -liree_runtime_unified -lflatcc_parsing -lcpuinfo -lclog -ldl -lpthread -o native-training
clang: warning: treating 'c' input as 'c++' when in C++ mode, this behavior is deprecated [-Wdeprecated]
+ python native_training.py /tmp/native_training.vmfb
Traceback (most recent call last):
  File "native_training.py", line 11, in <module>
    from functorch._src.compile_utils import strip_overloads
ModuleNotFoundError: No module named 'functorch._src.compile_utils'

https://github.com/iree-org/iree/actions/runs/3643985501/jobs/6152755452 and https://buildkite.com/iree/iree-samples/builds/563#0184e9f1-f3e6-415f-81d7-d9b8d088cb25/41-5629
@dellis23 are you investigating this?

the function signature has changed in the latest nightly for functorch. A similar fix landed in torch-mlir

I'm currently seeing a similar error in IREE's samples build:
/usr/bin/clang++-9 main.c -flto -O2 -I../../runtime/src/ -L/tmp/iree-build-runtime//build_tools/third_party/flatcc/ -L/tmp/iree-build-runtime//runtime/src/iree/runtime/ -L/tmp/iree-build-runtime//third_party/cpuinfo/ -L/tmp/iree-build-runtime//third_party/cpuinfo/deps/clog -liree_runtime_unified -lflatcc_parsing -lcpuinfo -lclog -ldl -lpthread -o native-training
clang: warning: treating 'c' input as 'c++' when in C++ mode, this behavior is deprecated [-Wdeprecated]
+ python native_training.py /tmp/native_training.vmfb
Traceback (most recent call last):
  File "native_training.py", line 11, in <module>
    from functorch._src.compile_utils import strip_overloads
ModuleNotFoundError: No module named 'functorch._src.compile_utils'

https://github.com/iree-org/iree/actions/runs/3643985501/jobs/6152755452 and https://buildkite.com/iree/iree-samples/builds/563#0184e9f1-f3e6-415f-81d7-d9b8d088cb25/41-5629
@dellis23 are you investigating this?

Yeah, the fix was merged here: https://github.com/iree-org/iree/commit/fcae6d71895c7d13cef0637657219c9ebf931249

I installed torch_mlir by compiling source code and setting environment path, as development.md says.


Hitting enter on results in the find all references tree should move focus to the corresponding line in the editor

Issue Type: Bug
CC: @isidorn

With a keyboard, trigger a find all references in any project. Navigate to the tree view and hit enter on a result.

An editor opens showing the relevant part of the file. However, keyboard focus does not move from the tree view of results to the editor as one would expect.
VS Code version: Code - Insiders 1.46.0-insider (288852d8c26d797974bfb8aed06e1bff9b809223, 2020-05-12T05:43:31.363Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (3.61GB free)

Process Argv

Screen Reader
yes

VM
33%

Extensions (13)

Extension
Author (truncated)
Version

notes
can
1.0.0

vscode-markdownlint
Dav
0.35.1

vscode-eslint
dba
2.1.5

prettier-vscode
esb
4.7.0

rest-client
hum
0.23.2

python
ms-
2020.5.78807

remote-wsl
ms-
0.44.2

cpptools
ms-
0.28.0

Go
ms-
0.14.2

vsnotes
pat
0.7.1

rust
rus
0.7.8

markdown-preview-enhanced
shd
0.5.3

vscode-webhint
web
1.5.5

Can repro. Agree. Enter should move focus. Space should preserve focus (like in the Explorer).
Assigning to @jrieken so he considers changing this

@jrieken also @Neurrone just informed me he is interested in providing a PR to tackle this. So let him know if you are open for a PR in this area and if you can provide some pointers.

@Neurrone Go for it. The references tree is actually an extension that lives here: https://github.com/microsoft/vscode-references-view/blob/3df68dc3f9c25c48ff215169978cd3a671311b54/src/extension.ts#L236

Thanks @jrieken. Where could I refer to to see how I should move keyboard focus to the editor?

https://github.com/microsoft/vscode-references-view/blob/3df68dc3f9c25c48ff215169978cd3a671311b54/src/extension.ts#L236

How do I run Vs Code with the extension from e.g, a checked out copy of that extension instead of the built-in one? The first thing I wanted to try is to add some log statements to see what happens when pressing enter on a result in the find references tree.

@Neurrone just pressing F5 to start debugging should do it. That will launch a new instance of vscode that should have the extension running ideally.

I opened that folder in Vs Code and pressed f5 when my cursor was somewhere inside the extension.ts file.
I get this notification and it fails:
The terminal process terminated with exit code: 1

Do I need to build Vs Code from source and be running Vs Code from source as well? I had some trouble with doing that with Vs Code itself - got some sort of compile error during the install stage.

@Neurrone nope you do not have to run vscode out of source, you could just run regular vs code insiders or vscode stable.
If pressing F5 when debugging the extension does not work then I am not sure what is the cause. I would guess that the extension can not be build for some reason. I would try opening it in the terminal and trying to build it via yarn  or some other script which might be specified in package.json
I can also look into creating this PR if you continue to hit this issue.

Thanks @isidorn. I'll give this another go and will let you know if it still doesn't work.

I got it to work, turns out I forgot to do an npm i.
Where do I see logs being printed out, e.g via console.log?

@Neurrone they should probably be in the debug console in the first window. F1 > view: debug console should show it.

Thanks, this worked.
Looking at this, the simplest way for me to fix this is to modify the references-view.show command being returned in provider.ts to specify true for the second argument of focusEditor.
However, I think I understand why focus doesn't move to the editor by default. There could be situations where you just want to quickly open the file at that point without actually wanting to edit it. In that case since focus stays in the tree, you can still press the arrow keys to rapidly move between references. However, this isn't the same UX that is present everywhere else though such as in the explorer view so I don't know if there will be actual use cases affected by this.
Ideas to fix this welcome. Should I go ahead with the change or introduce a setting to allow configuring this behaviour?
In the course of working on this, I also opened #99563 and #99564.

@Neurrone I do no think there should be a setting.
Like the explorer it should have two commands
space to open the file and not pass focus
enter opens the file and passes focus

@isidorn do I need to create 2 separate commands then to have both behaviour? Space is the current behaviour, and enter is new behaviour so that focus also moves?

@Neurrone I think 2 commands is the way to ge here. It is I think the simplest and cleanest.

Is there any documentation somewhere on the overall structure of Vs Code? I see that there is something called a Tree Provider that I assume is magically called by the extension subsystem to populate the tree.
getTreeItem returns a command attached to each item:
result.command = { command: 'references-view.show', title: 'Open Reference', arguments: [element, true] };

How would I have this support 2 commands so that one can be activated by space and the other bound to enter?

@jrieken shouldn't this be a bug instead of a feature request?

This should work fine now in latest, given https://github.com/microsoft/vscode/issues/85636 landed. @Neurrone can you check in insiders?

Yeah this works for me nicely with latest insiders.
Let's close assing to November and @neurrone can verify


Does not work with Redmine 3.0.2

wanted to upgrade, but nothing happened.
With version 3.0.1 works fine.
redmine-3.0.2]# bundle exec rake redmine:plugins:migrate RAILS_ENV=production                                                                                [175/2455]
rake aborted!
NameError: Couldn't find RedmineBootstrapKitHelper, expected it to be defined in     helpers/redmine_bootstrap_kit_helper.rb
/usr/local/rvm/gems/ruby-2.2.0/gems/actionpack-    4.2.1/lib/abstract_controller/helpers.rb:162:in `rescue in block in modules_for_helpers'
/usr/local/rvm/gems/ruby-2.2.0/gems/actionpack-4.2.1/lib/abstract_controller/helpers.rb:155:in `block in modules_for_helpers'
/usr/local/rvm/gems/ruby-2.2.0/gems/actionpack-4.2.1/lib/abstract_controller/helpers.rb:144:in `map!'
/usr/local/rvm/gems/ruby-2.2.0/gems/actionpack-4.2.1/lib/abstract_controller/helpers.rb:144:in `modules_for_helpers'
/usr/local/rvm/gems/ruby-2.2.0/gems/actionpack-  4.2.1/lib/action_controller/metal/helpers.rb:93:in `modules_for_helpers'

etc

Do you have other plugins installed?

Yes, i have a couple plugins

Duplicate : https://github.com/jbox-web/redmine_git_hosting/issues/498


add correlation metrics to explore module

Somehow inform the user on correlation metrics between target and other factors like gender, mos or age, in the explore module

done


fix: make isBooleanLiteralType a type guard

PR Checklist

[x] Addresses an existing open issue: fixes #80
[x] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview

Waiting on #84

:shipit:


Inputting 0 for an integer gets casted to nil

In the Motor Admin interface, I inputted an integer input field as 0 and saved. Somehow it got casted to nil in Motor Admin. I had to log into Rails console to update this to 0 and it worked from Rails console, just not from Motor Admin interface. Please advise! Thank you!

@vkhang55 thanks for reporting this bug - it has been fixed in the latest release:
https://rubygems.org/gems/motor-admin

Thank you @omohokcoj!


Allow proposing any PlatformAction in the PolicyKit UI

Implements the UI for proposing PlatformActions (Closes https://github.com/amyxzhang/policykit/issues/346)
Changes the PolicyEngine evaluation loop so that....

When an action that was proposed in the UI passes (aka community_origin == False), the engine calls action.execute().

Another way I could have implemented this is to modify the "pass" of All Actions Pass policy to be if not action.community_origin: action.execute(). That would also work and aligns with the goals of https://github.com/amyxzhang/policykit/issues/318, but I'd rather not mess with the StarterKit policies now and just continue as-is adding this logic to the engine. Then when we implement 318 we can lift this logic into policies (if we agree on that - want to confirm approach with @amyxzhang first as well). I think this approach will be less disruptive to current users.

When a constitution action passes, the engine executes it (@rickygv99 this removes the need for "all actions execute" fallback policy, which was removed. you may need to change that starter policy on your server.)

screenshots
All the available actions for the current platform are listed. The "action_codename" is used for now because we don't have readable names defined.

The form is generated based on the model's fields. The choice of fields can be overridden on each model. For example, here we hide the "previous name" field because it's not necessary:

If there are no overrides, all fields are shown (Excluding BaseAction and PlatformAction fields). For example:

On submit, a message shows whether the action passed, failed, or was proposed.

Looks great, thank you for getting this done!
I agree with the decision to hold off on policy structuring changes, though we should do that soon anyway (hopefully).


Updated index.html with new use case for Bower and NPM

What I did not update was the bootstrap-native.min.js link since I never used the delivr CDN. So I'll leave this edit to you. :)

Also I just noticed I forgot to change the "main" entry in bower.json. You may want to fix that from
bootstrap.native.min.js

To
bootstrap-native.min.js

I fixed bower.json. I updated demo page with proper credits. I wish I had more testers to check the implementation before I push the next release to CDN.
Thanks a million for your hard work.

Quick question: https://www.npmjs.com/package/bootstrap.native
If this is your stuff there, please write a small description with link to github demo. I've given you so much work, I realize I was a pain, guilty, but I hope this would be the last one :)
Thanks so much.

I found this https://libraries.io/npm/bootstrap.native/0.9.7 please check repository link
https://github.com/git+thednp/bootstrap.native
should be
https://github.com/thednp/bootstrap.native

As you suggested, I've cooked up an account at npm site https://www.npmjs.com/~thednp
You can transfer project as soon as you see fit :)

There you go! Check https://www.npmjs.com/package/bootstrap.native and see, you are now listed as owner too.
Don't get confused of the version, i had to add the version tag so it would let me update the package. :)
That other listing of bootstrap.native you found comes from a source i haven't seen before. It looks like a service that crawls NPM for packages. So I can not do anything there if their links get generated wrongly.
Also thanks for the crediting. :)
In order to publish to NPM or bower a new version, just navigate into the package and type
npm/bower publish

NPM and Bower share the same set of commands.

Oh by the way, i should mention this. I brought this up for discussion on the Bootstrap Slack chat. They are considering less to no jQuery for bs4. But the bootstrap maintainers saw the project, so it looks like they have an open ear! Just thought I'd let you know. Been hanging out there for a couple of days now.

Also I just realized that my 0.9.7 tag didnt transfer while merging. You need to make a 0.9.7 tag, otherwise Bower won't pick it up too easy.
<EMAIL_ADDRESS>~/W/bootstrap.native $ git tag
0.9.2
0.9.3
0.9.4
0.9.5
0.9.6
0.9.7
<EMAIL_ADDRESS>~/W/bootstrap.native $ cd ../dnp-bsn/; git tag
0.9.2
0.9.3
0.9.4
0.9.5
0.9.6
<EMAIL_ADDRESS>~/W/dnp-bsn $ bower info bootstrap.native
bower bootstrap.native#*        cached git://github.com/thednp/bootstrap.native.git#0.9.6
bower bootstrap.native#*      validate 0.9.6 against git://github.com/thednp/bootstrap.native.git#*

{
  name: 'bootstrap.native',
  homepage: 'https://github.com/thednp/bootstrap.native',
  version: '0.9.6'
}

Available versions:
  - 0.9.6
  - 0.9.5
  - 0.9.4
  - 0.9.3
  - 0.9.2

So you think it's safe to go 0.9.7 release tag on git?

Definitively! I just integrated this into my bigger project. And the change is dramatic:
jQuery + Bootstrap JS + other stuff:	178 Kb minified
Cash + bootstrap.native + other stuff:	60 kb minified
Cash is a 15kb big alternative to jQuery that I use to do some dom manipulation. it really is just for convenience, but its so tiny, that i had to extend it to get $.ready and $.load to work at all. :)
So yeah, the 0.9.7 tag is very safe!

Alright then, we gonna have a CDN very soon.

There you have it, 0.9.7 release tag on CDN, congrats for your wonderful work :)
<script type="text/javascript" src="https://cdn.jsdelivr.net/bootstrap.native/0.9.7/bootstrap-native.min.js"></script>


Tests are not running

The tests that I created in the Kit are not running automatically. Futhermore, there are no runs in history of the "Run Copilot Tests" cloud flow.
I installed the latest solution version with the option to enable plug-ins steps enabled and I also verified that all cloud flows are enabled.  Do you have any tips about what may be causing this behaviour?
Regards

Are you testing with end user authentication enabled? Are you getting any notifications when you save a new test run?

today I installed the solution and  I have the same issue. I'm testing without authentication and tests are not running and I don't see any error/warning.

@mariofontana Were you able to confirm that the cloud flows are enabled?

Thank you @psimolin you pointed to the right direction,
The strange thing is that using the same account I used to install the solution and configure the first tests, it seems I don't have access to the cloud flows. If I try to turn them on, I receive an error
Flow client error returned with status code "Forbidden" and details "{"error":{"code":"ConnectionAuthorizationFailed","message":"The caller object id is 'xxx'. Connection 'cat_CopilotStudioAcceleratorDataverse' to 'shared_commondataserviceforapps' cannot be used to activate this flow, either because this is not a valid connection or because it is not a connection you have access permission for.
That's strange because I created that flow using the copilot studio kit app.
I'm definitely missing something

@mariofontana, please share solution which you are using. We will try to reproduce this issue at our end.

@mariofontana , gentle reminder to update.

@mariofontana, gentle reminder to update.

Closing this issue as no response from user. Thanks.


C# GC not cleaning up InputEvent objects very quickly

Godot version:
3.2.2.stable.mono
OS/device including version:
Retina 5k 2019 Mac, Radeon Pro 570X 4 G, 40GB memory, 3 GHz 6-Core Intel Core i5
Issue description:
When using _Input/_input in C#, the objects count (as seen in the Debugger > Monitors) climbs and climbs until you stop moving your mouse, at which point it pauses, then all of them are cleaned up. In GDScript, this does not climb this way, and the InputEvent objects are cleaned up immediately.

This can cause memory usage to slowly climb, and FPS to slowly drop. While not an issue on my machine, it can be problematic for others.
Steps to reproduce:
Create a C# script that has public override void _Input(InputEvent @event) in it, and run that in a scene.
Minimal reproduction project:
https://github.com/m50/GodotCSharpInputEventGCBug
There is a gdscript scene and a c# scene, both have identical scripts besides language.

I will note: This might be an issue with all Reference objects, it's just VERY obvious with InputEvents.

I think this is by design, that's how garbage collection works. GDScript doesn't exhibit this as it's not GC-ed, but reference counted.
CC @neikeq

GDScript does automatic reference counting.
In C# you can't control when an object is garbage collected. You can manually dispose the input event though.

If this is by design, then excuse me for saying, but the design is bad.
At the very least, the docs need to tell you that you MUST run Dispose on it, or else you will get performance problems, however I would consider this a memory leak, and thus a bug and needs fixing.
I get that C# is supposed to handle the garbage collection, but there should really be something done about this, because it is a memory leak, and a problem.
One of the main reasons to use C# is for the performance benefit, but this can cause the performance to get worse than gdscript.

It's not a memory leak, the memory will be released when the garbage collector finds necessary.
However the fact we have to allocate a new object for each input event can cause issues. I already dislike the fact we're allocating a Reference in C++ for each input event, but it gets worse in C# as it stresses the GC. The latter may be solvable with some form of object pooling, but it would probably still require manual disposal.
I've thought that in the worst case we could add an overload to _Input that takes a struct representation of InputEvent to avoid stressing the GC.
An alternative to manual disposal could be documenting that a reference to the input event passed to _Input must not escape the function call, and as such whoever calls _Input (the engine in this case) takes care of disposing it for you.


Change args to process one ontology at a time, flesh out run a bit

Sorry if I clobbered your edits there, Harshad - I think I tried to push them at the same time

This appears to work now - Jaccard sim output is ~2.6G, while Reznik output is ~497M.

oh no problem ! All I did was formatting which you already did so no conflicts as far as I know.

Resnik test throws a ValueError on the test set, complaining that nodes don't share the same parent. Might just be a sign that the test set has to be larger. Jaccard test works fine on the same set.

@caufieldjh , do you think this is ready to merge?

Yes - I thought about adding a CLI test but if we're going to make this into an OAK module then that may be less important. Otherwise I think it's ready to go.


KBD tag colors are incorrect when switching to dark mode

Issue Summary
KBD macro doesn't look correct with inverse colors (dark mode)
To Reproduce

Switch to dark mode in admin
Add MD snippet with  tag
Here's how it looks like

Technical details:

Ghost Version: irrelevant for CSS issue
Node Version: irrelevant for CSS issue
Browser/OS: Chrome
Database: irrelevant for CSS issue

Hi @ErisDS  we'd like to give it a try to resolve this bug! We are a group of students currently in a software course, and were assigned a task of help contributing to an open-source project!

I tried to reproduce this, but failed. Could you please explain each step to reproduce this behavior?

Was able to reproduce this following these steps:

Install Ghost
Select night-shift toggle on bottom-left of home page after account creation.
Create post, enter markdown block, begin <kbd> tag.
<kbd> tag unfortunately has ghost-dark... css color property set to: #e8ebed, should be something darker, like #394047, which is default for light theme.

Was able to correct issue locally via inspector, but unsure on how ghost-dark-485081d359d3ce1412ccd09a8f5b83d9.css is generated.

I fixed this in admin - Here is the PR: https://github.com/TryGhost/Admin/pull/2176

Thanks for this @danimajo! This has now been merged into main. https://github.com/TryGhost/Admin/commit/8591f893bc27cbdffdb88e2e7d9ac37a0bb86ffc


coreDoMatch fails when comparison types are not exactly identical

e.g. - when a type of String is not equal to roString.

Closed by #26


fixes jquery conflict with other version

This pull request fixes the following issues:

fixes jquery conflict if other jquery version is loaded for example with another theme.

shame that github is not showing the changes in tree_component

We would have to test this extensively but makes sense to implement :+1:

:+1:


add telegram icon.

This PR adds the Telegram icon.
Since the icon from Telegram's website isn't flat enough (it has 2 shades of grey plus white), I used this one from the Android app's source code.
I tested it locally and looks great.

Looks great, thank you for your contribution @piducancore , I just updated the color based on the icon at https://telegram.org/


Fallback for unknown language types

If I set the language type to something unknown (in my case "Plain text", which is the default) for kirby-highlighter (in a block) it returns an error message

Block error: "Unknown language: "text"" in block type: "code"

I would expect instead a fallback to plain code box without specific styling except for dark background and white text.

Good point, should fall back to plain text. Fixed by https://github.com/johannschopplich/kirby-highlighter/commit/45e607b376c27f5ebf90ac65572d79d9092f3ff8.
Could you please re-test with the latest version published?

Thanks, it works with 2.0.1 but I first had to solve a small thing with Composer:


[Site Request] LBRY.tv and hOdysee.com

Checklist

[x] I'm reporting a new site support request
[x] I've verified that I'm running youtube-dlcc version 2020.11.07
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://odysee.com/@lbry:3f/odyseewhatandwhy:9 or https://lbry.tv/@lbry:3f/odyseewhatandwhy:9. Should be - accessible trough lbry://@lbry#3f/odyseewhatandwhy#9
Single video: https://odysee.com/@cryptomining:0/odysee-the-new-and-improved-lbry-tv:f or https://lbry.tv/@cryptomining:0/odysee-the-new-and-improved-lbry-tv:f. As above
Playlist: https://odysee.com/@lbry:3f

Description
LBRY is a free protocol to share files.
LBRY.tv and odysee.com are frontends to LBRY and they mostly work as video platforms.

I shall mention it does try to download the video (which I was confused by, but it's my fault) using the generic extractor, which does not work.
I'll post the verbose log:
https://pastebin.com/hu5DLSVd

you can use the the videodonload helper addon  to download the video
which works.

Just briefly checked lbry.tv and it seems very easy to do. Found the api that fetches the needed data within 1-2 minutes. There are two ways how to obtain the video. Could implement one as a fallback.

Will demonstrate it here (also to write it down for me or anyone else to implement)
Take this as an example https://lbry.tv/@cryptorekt:b/20200222_221757:f
Where the id of the video would be 20200222_221757
The video (returned by player) is located here https://cdn.lbryplayer.xyz/api/v3/streams/free/20200222_221757/f898e93918e68951dcaa1fdbe73e6502641e6bdd/b95f25
BASE_API = https://cdn.lbryplayer.xyz/api/v3/streams/free/
After base api the video id will be appended.
Then the claim id f898e93918e68951dcaa1fdbe73e6502641e6bdd
and last the hash or sd_hash (dunno if there is hd_hash as well).


Specify Which Types For NSwag to Build

Hi, firstly, thanks for an amazing project!
I'm using NSwag to generate a TypeScript file at buildtime as described here.
This is using the nswag.msbuild package with these key lines in the csproj file.
  <Target Name="NSwag" BeforeTargets="PrepareForBuild" Condition="'$(GenerateCode)'=='True' ">
    <Exec Command="$(NSwagExe_Core30) run nswag.json /variables:Configuration=$(Configuration)" />
  </Target>

My issue is that it is only generating TypeScript files for types which are included as either input or output parameters in controller action methods. We have an architectural requirement to generate TypeScript for other classes which are not consumed by the API. Is there a way to tell NSwag to do this? Perhaps include all classes in an assembly with a specific attribute?
Thanks for any help.
DS
Also posted on StackOverflow.

You can implement and add a custom document processor, in the process method you can retrieve the schema generator and add more types

Thanks very much Rico. I'll check it out.

@damiensawyer I'm trying to do the same thing as you are, generate schema for a type that is not referenced as either input or output in controller action methods. I found a tutorial for creating a custom operation processor but I couldn't find anything that shows how to use a custom document processor. Could you post an example of how you accomplished this?
Thanks!

Did you try to put [KnownType(typeof(YourModel))] above one of your Controllers ou Models ?

@jeremyVignelles Thanks for the suggestion, Jeremy. I saw the page you linked but I don't know how to register the document processor. Is there something I need to do for that or is it enough to just add the custom document processor class to my solution? My knowledge of NSwag is very limited so maybe I'm just missing something very simple and obvious. In the custom operation processor example, the custom processor seems to be registered/referenced by adding the SwaggerOperationProcessorAttribute on the controller action. At least, that's how I imagine we're instructing NSwag to use that particular operation processor. Please feel free to correct me wherever I'm wrong.
Regarding using the KnownType attribute, the type I'm trying to add comes from a 3rd party library so I can't annotate it directly with that attribute. I tried extending the type in my solution and annotating the subclass but I'm still not seeing it in the schema.

To add a DocumentProcessor, you must do that in you AddOpenApi() configuration method, generally set in the Startup class of your asp.net core project.
I'm not on a computer right now, so I can't check the names, but it's inside that method that you have a mechanism to configure a DocumentProcessor.
The KnownType must not be on the type that is to be known, but on a type that is already known by NSwag to tell it "hey, here's another type you should consider". I'm not sure if a controller would do the trick, but any Model included in the spec file should. Or maybe I'm missing a caveat and this solution wouldn't work... try it 😉

Hi @mdobric,
After a few hours down the rabbit hole I dropped NSwag and just wrote my own C# POCO to TypeScript generator. As always, there's pros and cons to this. It was an acceptable path for me because I needed some more fine grained customisation of the output.
At the time, I wasn't aware of NSwag Document Processors. It may turn out they might have done the trick.
If you're interested in the approach, it was two C# projects. One was a standard CLI which referenced our POCO assemblies and when called wrote out .ts files. The other project used MSBuild directives in the csproj file to call the CLI project on build. I'd never done this before and it's quite a cool trick to break free from the likes of T4 and Visual Studio plugins (we use Rider). The code to generate the TypeScript was surprisingly simple. Just some recursive functions with reflection.
If you want, I can create gists of the main pieces. The code will have references to our various helper libraries that I can't post, but you'll get the idea.

@jeremyVignelles @damiensawyer Thank you both for your help! I ended up adding another controller method that uses the type I want to add as a parameter and got it into the schema that way. It's a hack but it will do for now. I'm curious to try adding the custom document processor and will attempt to do that next.

I'm trying to do the same thing as you are, generate schema for a type that is not referenced as either input or output in controller action methods.


Go: join of struct fields inserts unnecessary spaces

Again, start with the snippet:
package main

type str struct {
	A, B int
}

func main() {
	_ = str{
		A: 1,
		B: 2,
	}
}

Joining fields of struct str (line 8, cursor on {) produces the following:
_ = str{ A: 1, B: 2 }

After running gofmt the code is formatted to:
_ = str{A: 1, B: 2}

Hm, there's a setting called curly_brace_padding that you could set to 0 to disable it, and it could be set in an ftplugin file as b:splitjoin_curly_brace_padding. But if gofmt changes it, I guess we should simply not use that variable in this case. Do you know if this padding should be removed just when joining structs, or are there other curly-bracket-delimited items that should work the same way? I see that functions on one line should look like this, for instance: func test() { const foo = "bar" }

Actually, I've pushed a change to the improve-go-support branch that should remove the padding, so you could test it out with other things and see what mismatches gofmt finds.
I will say that if it turns out to be too complicated to decide where to add padding and where not to, I might choose not to handle this and leave it to fmt to auto-fix -- if it's handled by it, then there's no need for splitjoin to handle it too, I think.

The issue with extra spaces is gone, thank you!
Joining function brackets to one-liner adds spaces, as it should be. The block case is ok.
I've found the next issues:
type str struct {
	A, B int
}

joins to
type str struct { A, B int }

gofmt formats to
type str struct{ A, B int }

And reversed:
type str struct{ A, B int }

splits to
type str struct{
	A, B int
}

This
type empty struct{}

splits to
type empty struct
,

Joining of
type empty struct{
}

is ok. Quite synthetical case of splitting, because usually I switch to insert mode inside the brackets and hit <CR> (Enter), but it should be handled too, I think.

I've pushed a fix to the main branch. For empty structs, I've fixed this weirdness with ,, although I think the whitespace there is still a bit weird, but as you say, if you're changing an empty struct, you'll enter insert mode and modify anyway. Try it out and let me know how it feels now?

The only last problem is with:
type empty struct {
}

It is joined to
type empty struct{  }

All other cases work fine, thanks!

This should be fixed now as well

Everything is fine, thanks!


Upper part blurred not rendered only first page if vertical single page settings enabled

Upper part of page is not rendered only first page for some pdf files with these settings        .
```
.autoSpacing(true)
.enableSwipe(true)
.pageFling(true)
.swipeHorizontal(false)
.pageSnap(swipeBool)
.enableAnnotationRendering(true)
.pageFitPolicy(FitPolicy.BOTH)
.fitEachPage(true)
https://user-images.githubusercontent.com/32289631/65262000-ddea4680-db22-11e9-8c6a-0fbe7fbe006e.png
https://user-images.githubusercontent.com/32289631/65262002-ddea4680-db22-11e9-8f2c-df09c33d32dd.png

But when we swipe to next page and again swipe to first page the page rendered properly no blur at top like this screen shot
https://user-images.githubusercontent.com/32289631/65262149-1f7af180-db23-11e9-9b6d-bd4b30cd9eb7.png

 So please fix this issue only when pdf is loaded first page upper part is blurred with setting mentioned above

Upper part of page is not rendered only first page for some pdf files with these settings .
.autoSpacing(true)
.enableSwipe(true)
.pageFling(true)
.swipeHorizontal(false)
.pageSnap(swipeBool)
.enableAnnotationRendering(true)
.pageFitPolicy(FitPolicy.BOTH)
.fitEachPage(true)

https://user-images.githubusercontent.com/32289631/65262000-ddea4680-db22-11e9-8c6a-0fbe7fbe006e.png
https://user-images.githubusercontent.com/32289631/65262002-ddea4680-db22-11e9-8f2c-df09c33d32dd.png
But when we swipe to next page and again swipe to first page the page rendered properly no blur at top like this screen shot
https://user-images.githubusercontent.com/32289631/65262149-1f7af180-db23-11e9-9b6d-bd4b30cd9eb7.png
So please fix this issue only when pdf is loaded first page upper part is blurred with setting mentioned above

Have you fixed this problem? please tell me how to fix this.

@rizwan344 have you solved this problem? Please tell me.

no stil problem
On Tue, 23 Feb 2021 at 7:37 AM, Ikhsyan Hasan<EMAIL_ADDRESS>wrote:

@rizwan344 https://github.com/rizwan344 have you solved this problem?
Please tell me.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/barteksc/AndroidPdfViewer/issues/823#issuecomment-783836852,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWLGX5WAPGA37UTPIXTA53TAMIIFANCNFSM4IYNKEOA
.


Password Reset Issue

Steps required to reproduce the problem

Request password reset
Open Password Reset Request link

Expected Result

Password Reset Form

Actual Result

Internal Server Error!

App log output when I open the reset link, don't have much experience with PHP so any help would be appreciated.
[2017-05-19 09:37:16]  opencfp.INFO: Matched route "{route}". {"route":"reset_password","route_parameters":{"_controller":"OpenCFP\\Http\\Controller\\ForgotController::processResetAction","user_id":"1","reset_code":"resetcodegbrgibfiubibvfibfbf","_route":"reset_password"},"request_uri":"https://cfp.website.org/reset/1/resetcodegbrgibfiubibvfibfbf?q=%2Freset%2F1%resetcodegbrgibfiubibvfibfbf","method":"GET"} [] [2017-05-19 09:37:16] opencfp.DEBUG: > GET /reset/1/resetcodegbrgibfiubibvfibfbf [] [] [2017-05-19 09:37:16] opencfp.CRITICAL: Symfony\Component\Form\Exception\UnexpectedTypeException: Expected argument of type "string", "OpenCFP\Http\Form\ResetForm" given (uncaught exception) at /var/www/opencfp/vendor/symfony/form/FormFactory.php line 64 {"exception":"[object] (Symfony\\Component\\Form\\Exception\\UnexpectedTypeException(code: 0): Expected argument of type \"string\", \"OpenCFP\\Http\\Form\\ResetForm\" given at /var/www/opencfp/vendor/symfony/form/FormFactory.php:64)"} [] [2017-05-19 09:37:16] opencfp.DEBUG: < 500 [] []

@dorees Should be all fixed, sorry for any problems it caused you.

@chartjes What was the fix for this? I only see one commit from the 19th and adding that change to /classes/Http/Form/ResetForm.php didn't solve our problem.
I'm probably missing something.


Roundoff error in two stream scheme

Describe the bug
We get incorrect absorbed/reflected/transmitted radiation from plant_absorbed_pfd for both of our radiation schemes when the zenith angle is pi/2 or the sun is below the horizon, and when LAI is very small. This is because we have things like:
K = 1/cos(theta), where theta = zenith angle
and exp(-K*LAI)
We should be able to remove the LAI clipping done in the global run once we do this.
For now, I clip LAI in the TimeVaryingInput so that if LAI < 0.05, LAI = 0 exactly. Then exp(-K*LAI) =1.
To Reproduce
see failing test in kd/radiation_roundoff_bug

We should be able to remove the LAI clipping done in the global run once we do this.

What is the change that will allow us to remove the LAI clipping?


Select stable import name when multiple possible bindings are in scope

Summary
Fixes https://github.com/astral-sh/ruff/issues/12885
The issue with the FURB177 test is that there are two viable bindings for pathlib.Path: pathlib.Path or Path but bindings is a HashMap that has no guaranteed iteration order.
I'm reluctant to change bindings to a BTreeMap. I would suspect that it comes with quiet a performance penalty. That's why this PR fixes this issue locally
by sorting the imports by their range and picking the last import if multiple options exist.
Test Plan
cargo test
Alternatives
Split the test into two files. One that uses import pathlib and one with from pathlib import Path

Tests pass on 32bit architectures: https://gitlab.alpinelinux.org/WhyNotHugo/aports/-/pipelines/252939
Some of the more exotic architectures are not done, but this does fix https://github.com/astral-sh/ruff/issues/12885

@charliermarsh any thoughts on this?


CLICKHOUSE-3817 Try fix distributed CAST with old distributed servers

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

This looks like better fix: https://github.com/yandex/ClickHouse/commit/fe8c4d64e434cacd4ceef34faa9005129f2190a5
Because value is transformed just once during query parsing.
Not all the time during query analysis.


"Sadova" is listed in the drop-down menu

Description:
"Sadova" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Sadova" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Sadova, SUCEAVA".


Personal docker repo

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

Sorry, entered in error


added parent_docs

Can you add a description for this commit,? Just the code isn't obvious to me what this is doing, which makes me think you should add a comment explaining what parent_docs are in the code.


Button tooltipps obscuring other buttons

A button's tooltip should probably go away once they button is not tapped any longer.

Unfortunatly there is no convenient touch support in the Buefy tooltips. Disabled tooltips for devices with no hover capability for now.

Unfortunatly there is no convenient touch support in the Buefy tooltips. Disabled tooltips for devices with no hover capability for now.


publish a new tag to crates.io

hey I was wondering if you could publish a new tag to crates.io, thanks!

Apologies for that!
The is feature was added by a collaborator that I don't use my self so I didn't notice. I see it now when I build with cargo build --features httpcache. I'll add a check for that in ci, fix, and push a release out before the end of the weekend

@jessfraz all set! thanks for raising the issue

Thank you!!
On Mon, Sep 7, 2020 at 8:53 AM Doug Tangren<EMAIL_ADDRESS>wrote:

@jessfraz https://github.com/jessfraz all set! thanks for raising the
issue
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/softprops/hubcaps/issues/269#issuecomment-688406838,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALA23AJTRWHKH4G25HDDSDSET6ZLANCNFSM4QUE7NDA
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3


File picker for plugin assembly

File picker for plugin assembly in environment editor

Thank you @Stitti  for this enhancement!


使用最新版构建报错 0.18.0-beta.2

Describe the bug

Reproduction
依旧如此
System Info
pnpm
win

Used Package Manager
pnpm
Validations

[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Try new version, it works well:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>


float test case: fix referenced issue

@bors r+

:pushpin: Commit 3fdbc0fd3f4e0117d708cac58b35fa841eed19af has been approved by RalfJung

:hourglass: Testing commit 3fdbc0fd3f4e0117d708cac58b35fa841eed19af with merge 3bc1136600fe8c4339ba327e9d5a7d8d09db6a58...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 3bc1136600fe8c4339ba327e9d5a7d8d09db6a58 to master...


How replication control and load balancing being taken care of?

I wanted to create multiple replicas of same docker containers which should be replicated amongst the swarm nodes and a load balancer in place to balance the request to those replicas. Something similar to kubernetes pods, replicationControllers and services.
How can we do this using docker swarm?

For the load balancer part you need something like this:http://jasonwilder.com/blog/2014/03/25/automated-nginx-reverse-proxy-for-docker/
Listen to the swarm events api and rewrite your configuration accordingly.
For the rest, we are waiting for docker compose to add those features, you'll need to do it manually for now.

Thanks @vieux. I noticed if one swarm node goes down, then the container scheduled on it marked as pending but not rescheduled to other available swarm nodes. From the manager node, I'm getting below logs -
ERRO[3121] [3FLV:RF2L:CXDR:PF3Y:QC7S:Y7SY:JLZF:TJH5:HEXA:FH52:YUCP:ASSB/etp-ProLiant-DL180-G6-3] Flagging node as dead. Updated state failed: Get http://<IP_ADDRESS>:2375/v1.15/containers/json?all=1&size=0: dial tcp <IP_ADDRESS>:2375: connection refused
What is expected here? Should it wait forever for the down node to come up or it will try to reschedule the containers to other available nodes?
One more question, may be silly though, I noticed in your demo you were not providing -H switch with docker cli to connect to swarm node and port. Where do I need to configure this for docker so that I do not have to specify -H with swarm ip and port? I tried in /etc/init/docker.conf with DOCKER_OPTS but it is not able to connect while starting the daemon. But with cli, it is able to connect.

@sourav82 for your -H problem the right syntax is DOCKER_OPTS="-H tcp://<IP_ADDRESS>:2375" where <IP_ADDRESS> will answer on every network interfaces. Since it is an http api just issue a curl http://localhost:2375 from your server and check that it answer 404 Not found.

Thanks @david-guenault. Will try with it.


[BUG]: server field not populating in SQLConnects.js

How are you running AnythingLLM?
Docker (local)
What happened?
When trying to connect to MS SQL Server, AnythingLLM is failing with the error:
MSSQLConnector TypeError: The "config.server" property is required and must be of type string
Looking into the code I believe the issue lies here:
#parseDatabase() {
const connectionPattern = new UrlPattern(
"mssql\://:username\::password@\::port/:database"
);
const match = connectionPattern.match(this.connectionString);
this.database_id = match?.database;
this.connectionConfig = {
...this.connectionConfig,
user: match?.username,
password: match?.password,
database: match?.database,
server: match?._[0],
port: match?.port ? Number(match.port) : null,
};
}
// Note how there is no server definition in the URLPattern
mssql://:@:/
the match?._[0] is not picking up the sever name
Are there known steps to reproduce?
No response

Since the server can be any number of valid formats it is wild-carded
#parseDatabase() {
    const connectionPattern = new UrlPattern(
      "mssql\\://:username\\::password@*\\::port/:database*"
      // * => catch all and use this for server endpoint
    );
    const match = connectionPattern.match(this.connectionString);
    this.database_id = match?.database;
    this.connectionConfig = {
      ...this.connectionConfig,
      user: match?.username,
      password: match?.password,
      database: match?.database,
      server: match?._[0],
      port: match?.port ? Number(match.port) : null,
    };
  }


Finished in 14.18 seconds (files took 3.24 seconds to load)
84 examples, 21 failures

This issue is to work through those failing specs and make them pass.

I'll take this! A lot of them appear to be due to a fix I implemented for a different issue anyway, so I should have updated the specs then.

The .import_csv tests were failing bc the Place model is validating lat & and long. Should those be allowed to be blank or do we only want Places to be created with a latitude and longitude?

Should those be allowed to be blank or do we only want Places to be created with a latitude and longitude?

I would say allowed to be blank but I am not sure what the original ticket was asking. @Terrastories/terrastories-stewards pinging to see if you have any input on the preference for lat/long requirements.

Is there anything else that wasn't addressed by that PR, or can this be closed?

@slaloggia This can be closed. I'm so used to them being closed automatically. 🙈
In the future, you can tag a PR with one of the github keywords so it will automatically close this out. In the one linked above, instead of using "Address issue #461" you can use "fixed #461" or "closes #461." Here are all of the documented keywords that will close the associated issue: https://docs.github.com/en/enterprise/2.16/user/github/managing-your-work-on-github/closing-issues-using-keywords
Thank you so much for your work on this!


Strange behavior in pip/virtualenv

I'm not sure if this is a Flask-Babel or Flask issue, but I've noticed something very strange about flaskext importing.
If I install a Flask ext through pip and install another as --editable, I can't import the editable version unless I also install Flask-Babel (non-editable) and import it first.
I've narrowed this down to the fact that Flask-Babel imports from the actual Babel module (from babel import dates, numbers, support, Locale).
Without the import from Babel, any attempts to import editable modules will fail.  What makes this even more perplexing is that this behavior is not observed from within ipython, only through the vanilla Python shell or during the normal running of my application.   Through ipython, all imports work perfectly regardless of which modules I have installed and where.
Sample session (flask-kvsession is installed --editable, flask-babel is installed normally)
Python 2.7.2 (v2.7.2:8527427914a2, Jun 11 2011, 15:22:34)
[GCC 4.2.1 (Apple Inc. build 5666) (dot 3)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from flask.ext.kvsession import KVSession
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/rroemmich/tmp/t/env/lib/python2.7/site-packages/flask/exthook.py", line 86, in load_module
    raise ImportError('No module named %s' % fullname)
ImportError: No module named flask.ext.kvsession
>>> from flask.ext.babel import Babel
>>> from flask.ext.kvsession import KVSession
>>>

I am also having this problem (with flask_mongokit installed as editable). Running tests with nosetests work, but importing directly in python interpreter or running the dev server does not. I am not sure when this issue appeared, but I pip installed/uninstalled several non-related packages since the last time the server worked.


record crew names and hours

As a fisherman I may want to be able to record crew names and hours for each trip worked so that they can divide up wages easily https://github.com/StAResComp/sifids/issues/6

Could


Identifiers validation failed for dataset Fortalecimiento de la Colección de fotocolectas biológicas (CFB): una propuesta del uso de la imagen digital al servicio del conocimiento de la biodiversidad

Identifier validation failed for the dataset Fortalecimiento de la Colección de fotocolectas biológicas (CFB): una propuesta del uso de la imagen digital al servicio del conocimiento de la biodiversidad:

Crawler attempt: 282
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 3000
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 100%, total records count 2994, absent records count 2994
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3073035/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Fortalecimiento de la Colección de fotocolectas biológicas (CFB): una propuesta del uso de la imagen digital al servicio del conocimiento de la biodiversidad : https://doi.org/10.15468/lz93kn.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Fortalecimiento de la Colección de fotocolectas biológicas (CFB): una propuesta del uso de la imagen digital al servicio del conocimiento de la biodiversidad:

Crawler attempt: 285
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 3000
Cause: GBIF IDs hit maximum allowed - 50%, duplicates - 100%, total records count 2994, absent records count 2994
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3094486/step

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Fortalecimiento de la Colección de fotocolectas biológicas (CFB): una propuesta del uso de la imagen digital al servicio del conocimiento de la biodiversidad : https://doi.org/10.15468/lz93kn.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Upgrade angular version in atmosphere package

Hi there.
So, there's no fix for the IOS double tap bug in ionic-sdk npm package for angular-meteor. And the only work around is to use driftyco:ionic atmosphere package. However, our application uses Angular 1.5's component() method and meteor would only let us use the atmosphere package's 1.4.8 version of Angular. So would it be alright to update the Angular version used in the atmosphere package to 1.5.x?

Hello @jvillamor-visp-net ! Thanks for opening an issue with us! Unfortunately this sound's like something that the meteor team will have to resolve. Thanks for using Ionic!


Reference Github repo in helpfiles

I realized that we are not referencing the Github repo in the Stata helpfiles. This would be a good addition for the next release.
@rayhuang11 can you please take a look at the helpfiles (and any other release files) and make sure we reference the Github repo in them?

Summary: we added a reference to the GitHub to the Stata help files.


Add mark files as viewed feature

Whenever I approve a PR I’d love to have all of the changed files marked as viewed.
Test
https://github.com/lukaszklis/refined-github-mark-files-as-viewed/pull/1/files (can grant collaborators privileges upon request)

That’s a good idea 👍

Thank you ^_^


style: feature filters and compact mode

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes

Feature table filters
Compact mode (Applying some suggestions in https://github.com/Flagsmith/flagsmith/pull/2981)
Improve create tag ui
Fix border radius issue on tables

Feature table filters

Compact view

This PR also lead me to believe that the create/edit tag flow is inconsistent with the rest of the design.
Before

After

This PR also fixes a slight issue where the border on tables is not rendered correctly

How did you test this code?

Viewed/filtered features on the features page and user page.

I think we just need to add owners filtering (both users and groups) here?

@dabeeeenster would need API work, I think we should have feature parity then do that as a separate ticket.

@dabeeeenster would need API work, I think we should have feature parity then do that as a separate ticket.

OK sounds good


Readme example does not seem to work

Hello,
I just installed textlint along with this textlint-rule-alex set of rules. I copy-pasted the .textlintrc of the example and tested on a very simple file. Still, the rule is not ignored.
$ cat ~/.textlintrc
{
  "rules": {
    "alex": {
      "allow": ["boogeyman-boogeywoman"]
    }
  }
}

$ cat textlint.md
Are you a boogeyman?

$ textlint textlint.md

/home/tim/local/tmp/textlint.md
  1:11  error  [boogeyman-boogeywoman] `boogeyman` may be insensitive, use `boogey` instead  alex

✖ 1 problem (1 error, 0 warnings)

$ textlint --version
v7.0.1

I tried with several rules, but could not manage to ignore any of them. Am I doing something wrong?

@pixelastic Thanks for report.
I fix it and release<EMAIL_ADDRESS>

Thanks, working well now.


Fix #7765: Fix exceed context length error

Description
I was getting errors of exceeding my context window (see #7765) after a few messages and after some discussion, it seemed like the ChatMemoryBuffer is meant to guard against this and it does actually start to forget some history after a while. But a problem will occur if the system prompt (in the prefix-messages) is quite long and in addition to the chat history, we can actually get an error regarding exceeding the context window.
I should also add that the messages_to_prompts always expect that we have alternating user and assistant interactions and hence I wrote in a line so that when we are forgetting chat history, we delete pairs of messages so we still have these alternating messages. It's also possible to get errors if we forget chat history but we have the chat history starting with assistant messages in the chat history.
In this PR, I made some changes to the ChatMemoryBuffer.get() method by allowing you to pass in an initial_token_count argument which represents a kind of amount of context window to keep available and this gets added to each time we assess the token count of the chat history.
I have made changes to the context and simple chat engines where we first compute the number of tokens in the prefix-messages and pass that into the ChatMemoryBuffer.get() method and this ensures that we do not have a resulting all_messages which exceeds the length.
Happy to add some tests if feel necessary.
Fixes #7765 (this is currently closed, but I think it's still an open issue actually - see https://github.com/run-llama/llama_index/issues/7765#issuecomment-1783084918)
Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

Hello, I'm still running into this issue when using the CondensePlusContextChatEngine. Seems like the above changes haven't been applied there.
Any plans to implement this change to the rest of the chat engines, or do they work differently and I'm doing something wrong?
Thanks!


Issue removing ssl cert from application with spaces in common name

From @msull92 on September 15, 2015 18:14
I am having an issue removing an ssl certificate from an application where the certificate's common name has spaces in it. Is there a way to manually remove this using etcdctl?
Copied from original issue: deis/deis#4487

From @benwilber on September 16, 2015 0:35
@msull92 can you try replacing the spaces with %20
edit:
with etcdctl you should be able to remove /deis/certs/<common name>/cert and /deis/certs/<common name>/key
replace spaces in <common name> with %20.
This is definitely a bug that deis controller doesn't url escape keys (even though etcd itself does.)  deis controller itself can't write url-escaped keys because then they'll be double-escaped by etcd.

From @msull92 on September 16, 2015 15:44
@carmstrong
This causes the log below.

From @msull92 on September 16, 2015 14:57
@benwilber forgot to tag you

From @carmstrong on September 16, 2015 15:9
@msull92 It looks like the certificate has been cleared from etcd (and thus shouldn't be served any longer by the router), but is still there in our database.
Do you see an error when you try to delete it? If you could also paste the logs from the controller component when you try to delete the certificate, that'd be helpful. Thanks!

From @msull92 on September 16, 2015 14:57
I would love to do that, but it's not even showing up in etcdctl, see below:

Yet it still shows up with the client call.

From @msull92 on October 21, 2015 5:9
This has since caused an issue with restarting the controller because it tries to load in the certs from the database into etcd and fails because that cert causes an error. So others can find this later I will attach the stack trace.
Traceback (most recent call last):
 File "./manage.py", line 13, in <module>
   execute_from_command_line(sys.argv)
 File "/usr/lib/python2.7/site-packages/django/core/management/__init__.py", line 399, in execute_from_command_line
   utility.execute()
 File "/usr/lib/python2.7/site-packages/django/core/management/__init__.py", line 392, in execute
   self.fetch_command(subcommand).run_from_argv(self.argv)
 File "/usr/lib/python2.7/site-packages/django/core/management/base.py", line 242, in run_from_argv
   self.execute(*args, **options.__dict__)
 File "/usr/lib/python2.7/site-packages/django/core/management/base.py", line 285, in execute
   output = self.handle(*args, **options)
 File "/app/api/management/commands/load_db_state_to_etcd.py", line 15, in handle
   obj.save()
 File "/app/api/models.py", line 1049, in save
   return super(Certificate, self).save(*args, **kwargs)
 File "/usr/lib/python2.7/site-packages/django/db/models/base.py", line 545, in save
   force_update=force_update, update_fields=update_fields)
 File "/usr/lib/python2.7/site-packages/django/db/models/base.py", line 582, in save_base
   update_fields=update_fields, raw=raw, using=using)
 File "/usr/lib/python2.7/site-packages/django/dispatch/dispatcher.py", line 185, in send
   response = receiver(signal=self, sender=sender, **named)
 File "/app/api/models.py", line 1166, in _etcd_publish_cert
   _etcd_client.write('/deis/certs/{}/cert'.format(cert), cert.certificate)
 File "/usr/lib/python2.7/site-packages/etcd/client.py", line 269, in write
   response = self.api_execute(path, method, params=params)
 File "/usr/lib/python2.7/site-packages/etcd/client.py", line 582, in api_execute
   return self._handle_server_response(response)
 File "/usr/lib/python2.7/site-packages/etcd/client.py", line 600, in _handle_server_response
   raise etcd.EtcdException(resp)
etcd.EtcdException

To fix this, log into the database and remove the offending cert from the api_certificate table.
Note: Not sure if it will matter, but just for completeness; I am running a stateless platform on AWS. v1.11.1

This issue was moved to teamhephy/controller#33


Ensure that the updateActions work for field types that are objects

This is related to some work I'm doing atm.
Wanted to double check that buildActions also works for fields that are objects (i.g Enum, LocalizedEnum, Money) 😉

[x] commit messages are commitizen-friendly
[x] code is unit tested
[ ] code is reviewed by at least one person
[ ] code satisfies linter rules

Coverage decreased (-1.9%) to 92.435% when pulling c1bca9e9132f3f336bc5f44c21009d75e18ff49a on adnasa:category-sync-test-obj into 01b8f78f2f63dcc51f778c5f91316ef4b4d7f0eb on sphereio:rewrite-2.0.

closing...


Querying fixed, fluid, or gatsbyImage nodes of the childImageSharp node of a Craft asset results in an error.

Description
The gatsby-source-craft plugin makes childImageSharp nodes appear in Gatsby's GraphiQL interface, but querying any of the fields within the fixed, fluid, or gatsbyImage nodes results in the following error:
The "path" argument must be of type string or an instance of Buffer or URL. Received undefined

Steps to reproduce

install and configure gatsby-source-craft
run gatsby develop
open GraphiQL on http://localhost:8000/___graphql and query any fields within the fixed, fluid, or gatsbyImage nodes of the childImageSharp of a Craft CMS asset such as:

... on Craft_EXAMPLE_Asset {
  childImageSharp {
    gatsbyImage(layout: FLUID) {
      imageData
    }
  }
}

Additional info

Craft version: Craft Pro <IP_ADDRESS>
PHP version: 7.3.19
Database driver & version:  MySQL 5.5.5
Plugins & versions: Gatsby 1.0.0-beta.1

Until the issue with gatsby-source-craft is resolved, there is a workaround using gatsby-source-filesystem to create remote file nodes based on the public URL of Craft assets.
Steps to implement workaround

install gatsby-source-filesystem
add the following to gatsby-node.js, replacing EXAMPLE with the handle of your Craft volume with working public URLs:

const { createRemoteFileNode } = require('gatsby-source-filesystem');
exports.createResolvers = async ({
  actions,
  cache,
  createNodeId,
  createResolvers,
  store,
  reporter,
}) => {
  const { createNode } = actions;
  await createResolvers({
    Craft_EXAMPLE_Asset: {
      imageFile: {
        type: `File`,
        async resolve(source) {
          return await createRemoteFileNode({
            url: source.url,
            store,
            cache,
            createNode,
            createNodeId,
            reporter,
          });
        },
      },
    },
  });
};

run gatsby develop
open GraphiQL on http://localhost:8000/___graphql
a new imageFile node will appear on any Craft assets, which can be queried for use with Gatsby Image, being sure to query the url field as well or else the createRemoteFileNode() function will fail:

... on Craft_EXAMPLE_Asset {
  url
  imageFile {
    childImageSharp {
      gatsbyImage(layout: FLUID) {
        imageData
      }
    }
  }
}

Steps to reverse workaround
Once the issue with gatsby-source-craft is resolved, gatsby-source-filesystem can be uninstalled, the code from gatsby-node.js can be removed, and the query for images can be shortened to:
... on Craft_EXAMPLE_Asset {
  childImageSharp {
    gatsbyImage(layout: FLUID) {
      imageData
    }
  }
}

Okay, so.
gatsby-transform-sharp adds the childImageSharp child node to anything that has an extension field that contains a file extension that it can transform. Since an Asset describes a file, it's a valid scenario for it to have the extension field, even if it is not the file itself.
I've submitted a bug report in hopes that the current eager behavior can be adjusted since actually removing that field once it's added can get very ugly. Or, well, maybe I just can't figure out how to do it cleanly.
Until then, I've pretty much done exactly what @sandren has done, except I went with a localFile field that gets created automatically on all AssetInterface implementing types.
Since this is a resolver, if you never touch the localFile field, it never gets downloaded, so this should be pretty efficient.
I'll leave this issue open for now because part of the issue is the incorrect childImageSharp field on Assets.

Hi @sandren
I just opened pull request with fix for gatsby-transformer-sharp wrongly adding childImageSharp field to Asset (as described by @andris-sevcenko in previous comment) if you'd like to follow that.
The provided workaround is one that makes sense to me, and maybe the localFile / imageFile resolver could be added by gatsby-source-craft plugin automatically so users don't have to implement this on their own, but this is up to plugin maintainers to decide.

This fix has been implemented in<EMAIL_ADDRESS>

@andris-sevcenko Is the manual resolver still required? So is this closed with a "Won't fix" sort of closed? Or should this be working where asset images are automatically detected and receive the childImageSharp field?

@kara-todd it's not required, not. The sourcing plugin automatically adds a localFile node. The file is actually downloaded from Craft only if you access it in a query.


why are spearman rho values all negative?

I think I'm probably using it incorrectly.

This is addressed now.


Perform PyPi version lookups in parallel.

Use a thread pool to lookup multiple packages concurrently.
Packages that block for a long time are printed to the debug log using a decorator.
Needs at least Python 2.6 for built-in multiprocessing.
I wrote this patch because package name guessing takes a long time on some of my virtualenvs. By using parallel workers, guessing can continue while normal lookups go through unhindered (of course only until all workers are blocked with guesswork).
I don't know if the master branch is still supported, please advise.

Cool!
Would be probably good to have this with pip-compile from the future branch, too.

Closing this since pip-review is gone with the release of 1.0: http://nvie.com/posts/pip-tools-10-released/


GH-35684: [Go][Parquet] Fix nil dereference with nil list array

Rationale for this change
If a column to be written to parquet is a list array with only nulls in it, then the child will be empty and potentially have nil buffers. This case isn't checked in the writeDenseArrow function, so we blindly attempt to grab the bytes from the buffers which might be nil.

What changes are included in this PR?
Adding a check of leafArr.Len() at the top of writeDenseArrow. If the leaf array's length is > 0 we can be sure that the data buffer is non-nil. So we can just bail if the leaf array is empty, nothing to write.

Are these changes tested?
Unit test is added.

Closes: #35684

CC @yevgenypats please give this a try and see that it fixes your issue.

Thanks looks good to me!


SQL Medium Issue

Description
The issue at #925 requested a query to find a country with a population greater than Canada and also less than Poland's population.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!