Indicating runtime location on the system

cc @JeroMiya
Would be good to indicate runtime locations on the system. The simple way is to set an environment variable, to say "I am the OSVR install.". The more complete way would be to essentially add itself to a list of OSVR installs, to permit choosing between installs. (These are not mutually exclusive: a new install might add itself to the list of installs, and may set itself as a system-wide default)
I've seen two models for this:

Unreal engine - they've got a few tools to work with it, but it was a real pain and took some serious powershell scripting juju to just iterate through the available engine installs and find one of the right version with a desired tool: https://github.com/OSVR/OSVR-Unreal/blob/master/generate-project-files.ps1  Might not be super long, but it was very painful.
CMake exported packages (package database/registration): https://cmake.org/cmake/help/v3.0/manual/cmake-packages.7.html#user-package-registry  This, in addition to being neatly documented and cross-platform, is pretty easy to script - much quicker than the Unreal stuff was. Sample script that is actually included with every OSVR snapshot, to let you register it and its included packages - note that this is a totally generic script that traverses a tree looking for anything resembling a CMake package and registering it, which makes it longer: https://github.com/OSVR/OSVR-Core/blob/master/devtools/add_sdk_to_registry.ps1

Obviously, if we decide that keeping track of multiple locations of OSVR on the system is something we want to do, you know which general technique has my vote ;)

cc: https://github.com/OSVR/OSVR-Core/pull/380
The current implementation uses the OSVR_SERVER_ROOT environment variable. If the installer set that environment variable (preferably user level environment variable) on Windows, then the auto-launch functionality would work.

First step with indicating runtime location is done - https://github.com/OSVR/OSVR-win-installer/commit/40257ee8c3ee7965a2714220e66199bd4913203b
It adds (and removes on uninstall) user lever environment variable - OSVR_SERVER_ROOT which points to bin folder with all "exes", when you install runtime or sdk.
@rpavlik , not sure if I understood completely the adding to a list of OSVR installs?
Do you mean maintaining an environment variable like "OSVR_ROOT" and append install locations when you install SDK and Runtime?

@rpavlik, @JeroMiya
Which direction do we want to go here. Appending multiple locations to OSVR_SERVER_ROOT will break things for OSVR-Config and auto launch. I recently fixed issue #17 where it appended multiple values and I think it was causing some problems. We can of course setup a new variable OSVR_INSTALL_LOCATIONS, if needed?


How to avoid inf value while feature extracting in Matlab?

I’m trying to extract features from some EEG signals. One of the trails in my for loop creates inf value and emd doesn’t accept it so it causes an error! Would you please tell me how I can fix it?!
Thanks a lot in advance.
My code:
fs = 250;
idSubject = 1:9;
idTraining = 1:3;
for ii = idSubject
disp(ii)
for jj = idTraining
disp(jj)
[signal,H] = sload(['B0' num2str(ii) '0' num2str(jj) 'T.gdf']);
    s = 1;
    for ch = 1:3
        Signal(:,s) = signal(:,ch);
        s = s + 1;
    end

    Signal = Signal';
    SignalMean = nanmean(Signal);
    SignalMean = SignalMean';

    LabelLeft = find(H.Classlabel == 1);  %Signal of left hand
    LabelLeft = LabelLeft';
    LabelRight = find(H.Classlabel == 2); %Signal of right hand
    LabelRight = LabelRight';

    s = 1;
    for i = LabelLeft
        SignalL = SignalMean(H.TRIG(i)+fs:H.TRIG(i)+(3*fs)-1,1);
        SignalLeft{s,:} = emd(SignalL,'MAXMODES',4);
        s = s+1;

        clear SignalL
    end
    SignalLeft = cell2mat(SignalLeft);
    sLeftall{jj,:} = SignalLeft;

    s = 1;
    for i = LabelRight
        SignalR = SignalMean(H.TRIG(i)+fs:H.TRIG(i)+(3*fs)-1,1);
        SignalRight{s,:} = emd(SignalR,'MAXMODES',4);
        s = s+1;

        clear SignalR
    end
    SignalRight = cell2mat(SignalRight);
    sRightall{jj,:} = SignalRight;

    clear SignalLeft SignalRight signal Signal SignalMean H LabelLeft LabelRight
end
sLeftall = cell2mat(sLeftall);
sRightall = cell2mat(sRightall);

signalLeftAll{ii,:} = sLeftall;
signalRightAll{ii,:} = sLeftall;

clear sLeftall sRightall

end

Hi @jemappelleMarie -- I don't see how this EEG-specific Matlab code is in any way related to my Python code for deep learning.
This is not a bug in this code repo. I'm closing the issue.

Perhaps try StackOverflow?

Hi
Oops, I'm so sorry, my bad! I didn't want to create the issue here!
So sorry for that :(
Yeah, I did.
 Thanks a lot
On Thursday, May 16, 2019, 6:21:57 PM GMT+4:30, Jon Krohn<EMAIL_ADDRESS>wrote:

Perhaps try StackOverflow?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

haha no problem 😄


docker doesn't use gid/uid

docker run memcached complains that it cannot run as root user without the -u flag. It looks like the uid + gid aren't being filled in the OCI spec.

due to similar issue MySQL image is not able to start MySQL database.
even when passing -user option in docker run command.


This repo is impossible to install locally for developing purposes

I have tried almost everything and I have follow the few instructions you provide.
I think the list of requirements should be something like this:
numpy
scipy
pillow
torch==1.12.0 # This is a requirement from you
torchvision==0.13.0
decorator
attrs
tornado
psutil
#xgboost # Just in case you want to optimize a xgboost model
cloudpickle
onnx
onnxoptimizer
tqdm
#time # There's no a package called like this

# Other packages
nvidia-pyindex  # TensorRT index
nvidia-tensorrt<8.5 # I couldn't install the tensorrt version your repo require
PyYAML
pandas
opencv-python
matplotlib
seaborn
transformers
onnxruntime
apache-tvm

I tried everything and I am still getting the following error:
ModuleNotFoundError: No module named 'transformer_deploy'
I think this should be installed using pip install sentence-transformer, but I am still getting some packages are not installed.

Hi @hdnh2006. It is easier to pull our docker and work on it. If you try to install it locally, there will be a lot of compatibility issues. Any issues with working with the docker?

@VoltaML I am stsill trying, no problems since I started but this repo should be easier to install locally for development purposes, it is not easy to develop inside containers.
Any help would be appreciated, I am open to collaborate with you if necessary.

@VoltaML I think there are some issues like a lack of certain files in the repo, but I think I finally could do it.
Your code can be easy integrated into the original yolov5 code. I tried with a private rtsp camera in order to see the improvements of your optimization.
These are my results for yolov5x using just tensorRT:

And these are the results for yolov5x optimized by voltaML:

In the case of yolov5s, I don't see improvements, maybe the model is too small and I am trying with bs=1.
These are my results for yolov5s using just tensorRT:

And these are the results using your optimization:

As I told you, I'll try another batch size and I'll let you know

@VoltaML I think there are some issues like a lack of certain files in the repo, but I think I finally could do it.
Your code can be easy integrated into the original yolov5 code. I tried with a private rtsp camera in order to see the improvements of your optimization.
These are my results for yolov5x using just tensorRT:
And these are the results for yolov5x optimized by voltaML:
In the case of yolov5s, I don't see improvements, maybe the model is too small and I am trying with bs=1.
These are my results for yolov5s using just tensorRT:
And these are the results using your optimization:
As I told you, I'll try another batch size and I'll let you know.
If you need help to improve the documentation, I would love to collaborate.

Hi, @hdnh2006. Great to see that you're benefiting with our repo.  Usually small models may not give that big of a difference. But have you compared the performance of voltaML vs vanilla PyTorch for YoloV5s?
Sure, we'd love to collaborate with you on the documentation part. Please shoot me an email at<EMAIL_ADDRESS>


Meaning of pooling in pre-training step

Hi  Xiaolei, thank you for your work. I'm interested in various works with knowledge graph.
Anyway, I have a question.  What is the meaning of the pooling in the pre-training step?
$$ h_{u} = Pooling[f(\tilde{C}{pre} | \Theta{plm} ; \Theta_{fuse})] $$  in this equation, I cannot understand the meaning of pooling, and find corresponding code in github.
It would be very grateful indeed if you can give me anyhelp.
Best regards.
Yongtaek

Hi, thanks for your attention on our work.
Pooling means that we will merge the encodings of each token, for the DialoGPT model here, we will take the last token encoding as commonly done.


Integration tests do not format

Anything under src gets the formatting but it does not work for tests directory. To give an example, I have a very, very long line, which does not format as below:
    let assertion = binary.args([
        "bot", "send", "poll", "foo", "--receiver", &env::var("TELEGRAM_RECEIVER").expect("TELEGRAM_RECEIVER environment variable could not be found. Please create .env file and define it.")
    ]);

...which should format to multiline but it does not.
It does not work in VSCode. I have also tried it as below and it does not still format:
cargo fmt
cargo fmt -- tests/
cargo fmt -- tests/my_test.rs

Environment

Ubuntu 21.04
Cargo 1.56.0
If relevant, VSCode 1.62.3 with Rust Extension Pack (incl. Rust Analyzer instead of RLS)

Thanks for the report. However, the integration tests piece is a red herring because you're running into the same issue as reported in #3863 which will occur regardless of the target type/file directory location.
Standard workaround advice applies around either:

create a local for the long string lit
manually break up the long string lit so that it can be fit within the max_width limit
(if you're able to use nightly) enable format_strings to give rustfmt permission to try to break up the long string lit on your behalf


PageDown key selects last one when left items count is short

current behavior

new behavior

I guess the same change might be relevant for PageUp.

didn't notice that. will check out.

@akien-mga updated

Thanks!


feat(metrics): add fragment_id into udf metrics via task local variable

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
In #15278, we try to add actor_id and fragment_id for udf metrics. After discussion, we decide only use fragment_id and use task local to avoid intrusive way.

Set FRAGMENT_ID in task local variable
We can ony get the fragment_id and actor_id in compute node. So I try to update fragment_id in the express context when update actors information.
Will try to get the express_context.fragment_id info when call udf.
Final prometheus will looks like:

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Another things we need to be careful is that the value created from with_label_values will be long existing in the metric registry even if the fragment has been dropped, which may result in collecting redundant prometheus metrics and increase its workload.
To avoid this we may consider using LabelGuardedXxx metric instead. If we choose to use it in this PR, we should be careful that we should store the variable returned from with_label_values and later use this variable to report the metric, and otherwise the label will be frequently created and then dropped.

Thanks for reminding this (again?). We can fix this usage in the following PR.

And please also update the dashboard json. You can check the README.md under the grafana directory for guidance.

And please also update the dashboard json. You can check the README.md under the grafana directory for guidance.

Added. Test locally, the last number means fragmen_id


Add option to use jemalloc for Debian

Use "ceph_jemalloc_enabled" to enable jemalloc as the default memory
allocator for Ceph on Debian installs. This defaults to false, so no
change to existing deploys, but can be set to install and configure
jemalloc as the default memory allocator.
If the LD_PRELOAD line is changed in /etc/default/ceph it will cause a
restart of the ceph services so that jemalloc is in use.

Should be fixed now!
One note - the script makes some other changes to the rhcs.yml.sample, i reverted those changes, so the only changes here are to do with the ceph_jemalloc_enabled var.

No worries, leave the modification as is, the rhcs.yml.sample should stay in sync :).
Thanks!

jenkins test luminous-ansible2.3-centos7_cluster

@guits Thanks for the review - I changed it up to use one task and ternary instead. We can avoid the with_items loop in that case (and the ignore_errors: True).

Ahh nice spot! will fix that up - i'm looking into adding RHEL/CentOS support for this.

I reworked it slightly - it now works for CentOS/RHEL too - and I included a conditional on bluestore.
Lastly, I'd like to add a follow-up (if this is accepted) to adjust tcmalloc thread cache size, which is why I've called the task file "configure_memory_allocator", I'll just add an additional task in there if that works.
The ordering works well too, since we don't need to stat /etc/default/ceph anymore since it will have run that already (tags: always).

Note Sage just dropped support for this from /etc/default/ceph in Ceph master/Mimic, because jemalloc crashes the OSD if Bluestore is enabled. https://github.com/ceph/ceph/pull/18486 https://github.com/ceph/ceph/pull/18487
That will probably be backported to Luminous too.

By the way, does jemalloc override tcmalloc when using the default builds from download.ceph.com? From talking with Kyle Bader a few years ago, the --with-tcmalloc build configure option overrides LD_PRELOAD=....jemalloc.so, so it might appear to work, but Ceph still uses tcmalloc.

@ktdreyer - good feedback, let me test, I thought I had run it with the download.ceph.com packages - and it was using the LD_PRELOAD, since jemalloc was being used instead of tcmalloc. I'll do another test deploy just to confirm that. Otherwise I think the bluestore issue is resolved in that the conditional avoids that.
Just as a point of clarity - the default is still to not deploy jemalloc, you would have to actively go and enable jemalloc.

It would be a good idea to check with the developers on the best way to ensure that jemalloc is in use and tcmalloc is truly disabled. When Kyle last looked at this (2015) the environment variable is not enough. It makes me wonder if jemalloc ever really worked with the main download.ceph.com builds.

@ktdreyer you can check using lsof. it'll show which memory allocator is being used. In hammer and before I don't think you could just enable it, although I haven't tested that. In jewel+ it definitely does work, I'll try confirm with the download.ceph.com packages but I think that is the case for those too.

You can also use ldd on the binary to see what libraries its using.
The ceph.download.com packages are compiled with tcmalloc only - with that in mind I'm going to close this although I think there is use, if you have to build your own packages for this to become useful it makes it much harder to see a point of putting it in ceph-ansible. I'll post the second half of what I had planned to add in the "tc_malloc" settings in a separate PR! Thanks for the feedback @ktdreyer

Thanks Andy for researching this!


Painfully slow image loading

Refile serve the assets from the cache instead of directly from S3 url and the loading times are really low for some reason. Trying to investigate why this is happening.
Question @jnicklas : Why Refile doesn't link directly to assets on S3 and use cache instead?

Because that's how it is designed. Please re-read the README.

@jnicklas why this issue is closed?
I understand this is the way the refile works but there is still issue with slow image loading. I will add more info once I find why this is happening.


Do not spam Created directory to stderr on each get_filename4code

Hi. Each time the get_filename4code is called, it prints Created directory name, see https://github.com/timofurrer/pandoc-plantuml-filter/issues/29 and https://github.com/timofurrer/pandoc-plantuml-filter/issues/25
It will work as well, when the imagedir is symlink
In [3]: os.path.isdir('test-symlink')
Out[3]: True

In [6]: ls -l test-symlink
lrwxrwxrwx 1 felixoid felixoid 8 Jun 23 16:49 test-symlink -> examples/

I see it was changed in the https://github.com/jgm/pandocfilters/commit/4da2c73331c94d1227ea5104b0bdcad4c1e1ffda
But now it's very noisy. I think it's supposed to check for the existing directory, indeed.

No, it's not a hick-up https://github.com/jgm/pandocfilters/actions/runs/9648939829/job/26611289299?pr=109
But it is something that needed to be addressed in the actions https://github.com/actions/setup-python/issues/866


manifold docs

Description

Added references of riemman algo
Related Issues

Checklist:

[ ] My code follows the code style of this project, as detailed in our contribution guidelines.
[ ] I have added tests.
[ ] I have updated the documentation accordingly.

Thanks @miguelgfierro for changes in the doc.


Update versions

PULL REQUEST
Summary
What changes are being proposed?

Updating the versions for ecr-viewer and orchestration charts.

Related Issue 1272

@nickclyde yes, I can update this in this pr for the serviceAccountRoleArn for the ecr-viewer (ecr-viewer-s3-role-arn) and orchestration (orchestration-s3-role-arn) service.


give create docx example

i am  a newer in c++ could you give me a example about how to create a docx example

Hi @lugeluge
Currently DuckX is under heavy development and it can't create docx file.
You can just read, write and modify docx files.
I'll try to add this feature soon.
Good luck! :smiley:


Made template responsive and added transition in dialing icon

Description
Items aligned properly in template and made responsive ,also introduced transition in telephone icon
Fixes:  #(issue no.)https://github.com/arpit456jain/Cool-Front-End-Templates/issues/460
460
Type of change

[ x] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)

Checklist:

[ x] My code follows the style guidelines of this project

[x] I have performed a self-review of my own code

[x ] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@VidhiBhatt01 please review my PR  request...

 Just fix it and then I will approve it.

Also, add this project in style_links.json

None of the images are being loaded. Just fix it and then I will approve it.

Also, add this project in style_links.json

ok ,i think i forget to add that.

@VidhiBhatt01 Please check now,I tried uploading files properly.


How to cross database query？

How to cross database query？

Sorry, as far as I know, that's not possible.

I think it is not possible, if you want to work on an implementation , or design this feature, open a new Issue with a description of your solution/proposal
thanks 👍


[contracts] Multisig restricted withdrawals

The Graph will need to restrict withdrawals from non-node multisig owner to be a specified contract address.
Need to update the multisig and add tests around this.
Will be important to consider how this will impact the deployment on withdrawals and other channels without this restriction. Should also verify this will not change the calculated multisig address (would hate to run into those issues again...)

This will be handled here: https://github.com/ConnextProject/indra/issues/824


Update "Week in Openverse" script to support monorepo

Problem

The current week in openverse script splits PRs by repository:
https://github.com/WordPress/openverse/blob/main/automations/js/src/last_week_tonight.js
Description

We'll need to update this to split the monorepo PRs by their stack.
Alternatives

it should work as-is but will have empty sections for the frontend and api repos now.
Additional context

Since the JS script uses data/github.yml as the source for repo names, this was fixed in #763 by @AetherUnbound.

We haven't had a "Week in Openverse" Make blog post since the monorepo merge so this is unsolved.


Dark mode - does not work if you apply Ionic docs

https://forum.ionicframework.com/t/how-to-force-a-dark-theme/227535/5
https://ionicframework.com/docs/theming/dark-mode
I just tried adding dark class to body, which does not work. So I reckon something is going on that needs to be fixed.

Initial post - https://github.com/Tommertom/svelte-ionic-app/issues/54

closed - cleanup


apt-get update && apt-get upgrade fails

Muscle-memory had me entering an
apt-get update && apt-get upgrade

Which had some issues with udev, systemd, and initramfs-tools.
For mitigation, is there a way to reset the bash environment on Windows 10?

Issue #4 ultimately had the commands to completely uninstall Bash (allowing you to reinstall cleanly).
To summarize, as they were spread over a couple comments, you ultimately end up with:
Run the following from an Adminstrator (i.e. "elevated") Command Prompt
lxrun /uninstall /full
sc stop lxssmanager
rmdir /S "\\?\%LOCALAPPDATA%\lxss"

lxrun /uninstall should be enough to get you out of most states.
lxrun /uninstall /full should only be required if you need to uninstall your Linux home directory.
Note that lxrun /uninstall will not work correctly if you have any files open.  In that case the sc stop and rmdir will force your system to clean up.


services: fix TestNotary

Close #3366
The initial idea was that we have here similar to the #3369 situation. But maybe need additional checks

Replaced by https://github.com/nspcc-dev/neo-go/pull/3390.


suite: pass dev-commit:  to ceph-deploy

Use the new ceph-deploy install --dev-commit  option to inform the
install.
Signed-off-by: Sage Weil<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1810/
Test PASSed.

Any reason to care about old ceph-deploy versions here?

I don't think so..w e don't test them!
note that the ceph-deply pr hasn't merged yet, tho..
https://github.com/ceph/ceph-deploy/pull/348

@liewegas Oh! Glad I didn't merge yet then :)

:+1:
Are there things in the suites that will have to change to figure out a SHA1 instead of a branch name?  That's probably already done somewhere...

On Fri, 28 Aug 2015, Travis Rhoden wrote:

Are there things in the suites that will have to change to figure out a SHA1 instead of a branch name? That's probably already done somewhere...

the only change to teh suite will be to remove the install task, i
believe.  once this slow-ass dist-upgrade finishes i can test and verify!

tested this maually and dev-commit: sha1 is working!


Error when using scp_recv when the filename has spaces

I have a problem using scp_recv when the filenames have spaces in them
e.g. I have the following code to download all the files in a given path
let tcp = TcpStream::connect(&host)?;
let mut session = Session::new()?;
session.set_tcp_stream(tcp);
session.handshake()?;
session.userauth_password(&username, &password)?;
let sftp = session.sftp()?;
for (filepath, _entry) in sftp.readdir(&data_path)? {
    let (remote_file, stat) = session.scp_recv(&filepath)?;
    // .... do file download
}

As soon as it hits a file with a space in the name, i get the following error
Error { code: -28, msg: "Failed to recv file" }
To work around the problem I had to add the following at the start of the for loop
    let filepath = if let Some(filepath) = filepath.to_str() {
        PathBuf::from(&filepath.replace(" ", "?"))
    } else {
        panic!("could not get {:?} as a string", filepath);
    };

Also, I first tried to use "\\ ", and "\\\\ " but that didn't seem to help (but i don't think it should be necessary to do this replace at all from paths coming from the readdir method).
Am I doing something wrong, or is there a bug somewhere?
Thanks!
system: archlinux
system libssh2 version: 1.9.0
connecting to a sftp server with username/password

I noticed that you're mixing sftp with scp.  I'd suggest doing something like this to get a handle on the file contents using your established sftp channel:
let remote_file = sftp.open(&filepath)?;

(see https://docs.rs/ssh2/0.8.0/ssh2/struct.Sftp.html#method.open)

Thanks for pointing that out @wez, using sftp.open works without my .replace(" ", "?") hack :tada: .
I also took the time to check this using a plain ssh server, and I don't have the same problem
e.g. this works fine for me
use std::io::prelude::*;
use std::net::TcpStream;
use ssh2::Session;
use std::path::Path;

fn main() -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
    let host = "<IP_ADDRESS>:22";
    let tcp = TcpStream::connect(&host)?;
    let mut sess = Session::new()?;
    sess.set_tcp_stream(tcp);
    sess.handshake()?;
    sess.userauth_agent("tristan")?;

    let (mut remote_file, stat) = sess.scp_recv(
        Path::new("A file with spaces.txt")
    )?;
    println!("remote file size: {}", stat.size());
    let mut contents = Vec::new();
    remote_file.read_to_end(&mut contents)?;
    println!("{}", String::from_utf8(contents)?);

    Ok(())
}

So, this is not a problem with scp_recv but something to do with how sftp servers treat filenames, which I don't know enough about to comment on :sweat_smile: .


MongoDB no longer starts on Windows by default

As runner images changed for Windows (MongoDB is stopped+disabled by default), the action no longer works there: https://github.com/ankane/setup-mongodb/blob/v1/index.js#L35
See: https://github.com/actions/runner-images/issues/5949
For now, I overcome the issue by adding this step before setup-mongodb (based on suggestion in the runner-images issue):
      - name: Start MongoDB (Win service)
        if: matrix.os == 'windows-latest'
        run: |
          Set-Service mongodb -StartupType Automatic
          Start-Service -Name mongodb

Hey @MarekSuchanek, thanks for reporting! Just pushed a fix.


Help: Check if more info can be retrieved from Kite

Is it possible to get the format of the information  (MD vs rst vs... ?)

@dalthviz is this something we need to ask to the Kite team, or something we can investigate ourselves?

I think there is an endpoint that currently we aren't using which we could investigate (I think is the one the Copilot retrieves the parsed info). Also, I think Kite can provide an HTML version of the documentation but yes I think we can check first ourselves :+1:

This issue no longer applies, so I'm going to close it.


MOCK: Plan usage alerts

Description
This PR is a POC mock for the upcoming Plan Usage Alerts.
Screenshots

Design Spec
Possible now (See sandbox page)

Items to discuss:

[x] DESIGN: Final color choices need to be added to Sage 3
[x] DESIGN: Are text and icon colors going to change to Charcoal within SageAlert or is this custom for this alert?
[x] GENERAL: Secondary button style needs to be adapted to use with the Alert, probably as a prop that can be exposed.

Closing. Per Joe, these are either integrated or being handled elsewhere 👍


Feature request: Option to control quote-reply behavior

as per matrix conversation in the heisenbridge room
Currently quote-replying on the matrix side turns the message into a single tag. This might be fine for translation between matrix conventions and IRC conventions, however seasoned IRC users using heisenbridge might know about the IRC conventions, and want to intentionally quote when quote-replying (like when replying to a message from a while ago). I'd like to propose an option for changing quote-reply translation behavior:

Turn reply-quotes into tags (default)
Keep reply-quotes as is
Turn reply-quotes into tags only if the last message of the person being replied is the quoted one (a middle ground)

Also would be nice to have an option to control quote style:

Matrix / email style:

> blah blah message being replied to
whoever: reply content

IRC style:

<whoever> blah blah message being replied to
reply content

As I commented on Matrix this sounds like a good idea as long as the default is non-quoting.


Loading local images

Is it possible to load local images, such as those in src/assets/ images?
Using the code here: https://stackoverflow.com/a/50515442 for the source, <lazy-img<EMAIL_ADDRESS>it doesn't seem to load the image properly.

It should be possible. I’ll check that little bit later.
Are you sure your path to the image is correct?

If I change the code to <img<EMAIL_ADDRESS>/> (just remove the lazy part), the image shows.
The image is an SVG if that matters.

<img<EMAIL_ADDRESS>/>
Translates to (hash at the end is autogenerated):
<img src="/static/img/image.d79ab85.svg">

```

```

I’ll update you on that.

By the way, have you tried to do something like this?
<EMAIL_ADDRESS>with : before the path?
And another way is maybe to import the image like this:
import image from<EMAIL_ADDRESS>Then in your data():
testImage: image,
And then:
:src=“testImage”

Ok, so finally I came up with results.
It turns out, that this is not our package's bug or limitation. Actually, vue-loader itself does not support lazy-img tag by default. And you can easily fix that.
Basically vue-loader's default tag/attribute combinations for transforming assets urls are:
transformAssetUrls: {
  video: ['src', 'poster'],
  source: 'src',
  img: 'src',
  image: 'xlink:href'
}

All you have to do is to add our lazy-img tag with src attribute to the vue-loader's transformAssetUrls option. So you would have:
transformAssetUrls: {
  video: ['src', 'poster'],
  source: 'src',
  img: 'src',
  image: 'xlink:href',
  'lazy-img': 'src'
}

And then, everything just magically works.
Source: https://vue-loader.vuejs.org/options.html#transformasseturls
By the way, if you're using older vue-loader's version <=14 the option is called transformToRequire and not transformAssetUrls.


Update download URI

💁  The download URIs for Go have changed. This changeset updates the base URL and removes the unrequired version branching.
Closes #12

👋  @wfarr: For your consideration.

This doesn't work for me.
$ bin/goenv install 1.6.2

curl: (22) The requested URL returned error: 404 Not Found
goenv: unable to install Go `1.6.2' from binary, download not available at https://storage.googleapis.com/golang/go1.6.2.darwin-amd64-osx10.8.tar.gz

@spraints: Looks like the packages have changed markedly since "install" was implemented.  https://storage.googleapis.com/golang/go1.6.2.darwin-amd64-osx10.8.tar.gz should be https://storage.googleapis.com/golang/go1.6.2.darwin-amd64.pkg or https://storage.googleapis.com/golang/go1.6.2.darwin-amd64.tar.gz

Looking at the Go downloads page, the format change in OS X package naming happened between version 1.4.2 and 1.4.3.

👋  @spraints: goenv install 1.6.2 should work with you as of 640b2a7363de38f3241bb19d4ee588c10926bd74. ♻

Looks great! Thanks!


Swift 4 support

Convert to swift 4
Remove warnings

Coverage remained the same at 98.784% when pulling 5fe90a15e0f40ed63a65eca1d76163eb3cae7019 on AYastrebov:master into 566eb51e994af33fdb0b050025e141f790e65647 on KrakenDev:master.

@hectormatos2011 tests are passing now. I've updated travis config also

Again, thank you! If I remember someone today I'll make sure to update CocoaPods to reflect the new version


[Documentation] Add contribution guidelines

Make a contributing.md file and add the guidelines for contributing to this project. You can refer this for the same ad make the required changes in here.

Hey!
I would love to work on this.  I prepared contributing guidelines this morning itself.

Sure @kanishkaa24 go ahead with this issue.

hey, Can I work on this issue? @Kajol-Kumari

hii
i would like to work on this issue

Hey @singhsaurav23 @chehak123 we can't assign this issue to more than one people so please look at some other issue or wait patiently as we will be opening new issues very soon :)

Hello
I have made a pull request for the same.

Closing via this PR

@kanishkaa24 Can you please incorporate Commit message guidelines from this

Sure

@jackfrost13
I have made the changes.

Thanks, I will have a look ;)

Is it fine or do I need to make any other changes?

Is it fine or do I need to make any other changes?

Yes, I have requested for a minor change. Please have a lot

@jackfrost13
I have made the suggested changes.

@kanishkaa24 Can you please confirm, in response to which event you have contributed?

@kanishkaa24 Can you please confirm, in response to which event you have contributed?

I'm participating in DWOC and SWOC.

Thanks for the update!


[BUG]: In expo project, run:ios is ok, but build error, Can't merge pod_target_xcconfig for pod targets: ["expo-dev-menu", "Main", "ReactNativeCompatibles", "SafeAreaView", "Vendored"]. Singular build setting DEFINES_MODULE has different values.

Describe the bug
[BUG]: In expo project, run:ios is ok, but build error, Can't merge pod_target_xcconfig for pod targets: ["expo-dev-menu", "Main", "ReactNativeCompatibles", "SafeAreaView", "Vendored"]. Singular build setting DEFINES_MODULE has different values.
To Reproduce
build
Environment
System:
OS: macOS 14.6.1
CPU: (14) arm64 Apple M3 Max
Memory: 228.61 MB / 36.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 20.11.1
path: ~/.nvm/versions/node/v20.11.1/bin/node
Yarn:
version: 1.22.1
path: ~/.volta/bin/yarn
npm:
version: 10.2.4
path: ~/.nvm/versions/node/v20.11.1/bin/npm
Watchman:
version: 2<IP_ADDRESS>
path: /opt/homebrew/bin/watchman
Managers:
CocoaPods:
version: 1.15.2
path: /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 23.5
- iOS 17.5
- macOS 14.5
- tvOS 17.5
- visionOS 1.2
- watchOS 10.5
Android SDK:
API Levels:
- "33"
- "34"
Build Tools:
- 33.0.1
- 34.0.0
- 35.0.0
System Images:
- android-34 | Google APIs ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2024.1 AI-241.15989.150.2411.11948838
Xcode:
version: 15.4/15F31d
path: /usr/bin/xcodebuild
Languages:
Java:
version: 17.0.11
path: /Users/shanks/.jenv/shims/javac
Ruby:
version: 2.6.10
path: /usr/bin/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native:
installed: 0.74.5
wanted: 0.74.5
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: true
newArchEnabled: false
iOS:
hermesEnabled: true
newArchEnabled: false
Additional context

expo version (ex: 51) : 51
using firebase : no
using static build : no
routing : (ex: basic / expo-router / react-navigation): expo-router
package version (ex: 1.2.0) : x.x.x
platform target : iOS
device : simulator / real device
type: local build

cannot reproduce, reopen if it still happen


internal error!

crash-2022-12-14_<IP_ADDRESS>96-quilt_loader.txt

java.lang.ClassNotFoundException: java.util.Map
...it couldn't find a standard Java library class, oh dear

What version of Java are you using and where did you install it from?

That's not the problem, although I haven't looked into what could cause this yet.

Using Java 20? Maybe also try using a stable JVM and not some Early-Access Builds ...


kickoff-react

Adding this here for transparency

@ciaranpark, @jpdriver, @denzBrade & I will be creating a Kickoff/react package. It will include all current components so they can be imported into any React project.
I would also like to extend the Yeoman generator so it can scaffold a simple React app, include kickoff-react and potentially react-router, redux and anything else that we deem useful.
More info and thoughts to come soon..

I've been looking at other React UI libraries for reference and https://github.com/callemall/material-ui looks to be very good. The project structure and DX would be something I'd like to emulate. Does anyone else have any suggestions?

New repo for this project: https://github.com/TryKickoff/kickoff-react. Further comms should probably be posted there...


Window context when accessing this

When i call this in a function in the Element script i get the window context?!
<link rel="import" href="../../../bower_components/polymer/polymer.html">

<dom-module id="channel-view">
  <style></style>
  <template>
    <template is="dom-repeat"
              items="{{channels}}"
              as="channel">
      <span>{{channel}}</span>
    </template>
    <button on-tap="_onTap"></button>
    <input on-add-channel="addChannel">
  </template>
  <script>
    Polymer({
      is: "channel-view",
      properties: {
        channels: Array,
      },
      _onTap: () => {
        console.log(this); //<-- this prints the global window context
        this.fire('add-channel')
      },
      addChannel: (event) => {
        this.push('channels', {})
      }
    });
  </script>
</dom-module>

I tested it on FF (latest), Chrome and Electron they all have the same context

Looks like that's some sort of bug specific to arrow functions.  When you do it this way, the bug disappears:
_onTap: function () {
	//...
}

This solves my problem, thank you so much!!

@luii Yeah, it looks like arrow functions work this way by design.
"Arrow functions capture the this value of the enclosing context".
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions


TMP

Manually copy files to simplify the review of https://github.com/ome/omeroweb-install/pull/33
When https://github.com/ome/omeroweb-install/pull/33 the files will be updated via the autogen job
DO NOT MERGE THIS PR
cc @pwalczysko

Testing of https://github.com/ome/omeroweb-install/pull/33 is finished, which I think was the purpose of this PR.
@jburel : Let me know if I can be of any help with copying of these files.

Closing this PR since I had to manually copy the files
in https://github.com/openmicroscopy/ome-documentation/pull/1761 to make travis happy


I can't delete a bot.

Describe the bug
I have a EmptyBot-01 turd in my documents folder (see #2973 ) and I can't delete it. I don't know where it put it (see #2973 ).  And now there is no way for me to delete it from composer.
Version
0.0.1 (2 days ago)
Browser

[ ] Chrome
[ ] Safari
[ ] Firefox
[x] Edge (chrome)

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce

open compose
try to get rid of EmptyBot-01

Expected behavior

Right Click gives menu to "Delete Bot"
Opening the bot gives me menu to "delete the bot"

@tomlm this has been discussed in the past, and is a very fair requirement. The work is not hard to plumb this through.  To implementer:
When deleting the bot directory, we also need to remove any application level state referencing the deleted bot. The bot field from data.json needs to be removed and any information in LocalStorage pertaining to the bot needs to be removed. composer:DialogSetting and composer:LuFileStatus has this state in LocalStorage.

@cwhitten @benbrown  do we need to delete the runtime as well. Runtime maybe hosted in local or customized path ? Meanwhile, need the placeholder for delete action. @mareekuh need your suggestion.

@hibrenda yes the bot directory needs to get blown away and if they have injected a /runtime is will also get removed.

We should definitely NOT delete anything outside of the main bot project folder. That is, if the custom runtime path is pointing at a totally different folder, it should NOT delete these files.

Quick screenshot here -
Marc may do some cleaning up but this is the idea (below).
Lives in the toolbar of bot settings page.

https://www.figma.com/file/LlTlh5vXwq91zjGnFtrUUR6l/Composer-master-design-spec-UI-library?node-id=1514%3A656
@DesignPolice and @liweitian

refs #919

@cwhitten @benbrown Talk about it, Do we need a design for stop runtime too?  We need to stop runtime before delete a running bot.

Good points @VanyLaw and @yeze322 - I will look at those this morning as well... Ben and Chris fill in any other info I might need.

@DesignPolice @mareekuh  We already have a 'delete' button on that tool bar used to delete node in the visual editor. So we probably need another design.

I have never seen a delete button that acted like a dropdown like that. Seems error prone.

I do not think we need to explicitly stop a bot before deleting it. It should automatically stop if you delete (or close) the project.

Agreed, the delete dropdown with options looks weird.

Hmm, so do we just want to have a stand alone "Delete Bot"  - I think the other options could moved into the settings drop downs - and if I don't need to Stop the runtime to delete would free up some options.
I will make some options and we can discuss in the standup, rather than this thread blowing up everyone all morning.

@liweitian and @DesignPolice
Our buttons in the toolbar are contextual to the content of the page.
So in the Design view, when the user has selected a node, delete applies to that node.
In the Settings page, there are no nodes visible and delete applies to the content on bot settings or app settings.

delete a language  (only available >2)
delete the bot
etc;

So what we do for add, when there are multiple things that can be added in the context of the page, we do a dropdown. We should do the same for Delete...
If this is concerning to you all, we can add the 'delete bot' and 'delete language' and 'Delete runtime' all next to one another in the short term.
@benbrown @cwhitten

This is why I suggested we add it to the Homepage.

when adding it to the home page we'd be making it possibly more prominent than needed for a task like this and we'd have to have a 'picker' page: which bot to delete.
In the settings page you're already 'in' a bot - you're already managing the bot...

Yes but you just contradicted yourself saying the operations in the Settings page need to apply to the content on bot settings or app settings so I'm questioning the system here...

We have 2 settings pages: Bot settings - which is where 'delete bot' fits, and App settings. (composer app - neither deleting language or bot fits here)

We only have one Settings page at the moment.

Right.
Today the settings for the current bot as well as the ones for Composer as the app are on one page. So buttons related bot either go on that page.

Updating some info after a busy day.
I know what I have designed on this page is more robust than we have time for - but the basic need is a single button to delete the entire bot. That also gives plenty of extra warnings that you are making a very permanent decision.

https://www.figma.com/file/LlTlh5vXwq91zjGnFtrUUR6l/Composer-master-design-spec-UI-library?node-id=1514%3A1499
Let me know if you want to make any tweaks - thanks everyone.
m

The functionality is done waiting for review. Let me know if the UI is finalized. We need a checkbox right?

Thanks @liweitian I will get with @beyackle today and get you a mock-up or we will fix it... thanks so much for all the hours! M

Yeah, the functionality there looks exactly like what I had in mind (given that the box starts unchecked). It needs a design mockup, which I think @DesignPolice is already on.

@beyackle can you go ahead and take this over once Marc gets the design mock to you?

@beyackle here you go.. the only change I made was to the modal - Should we make the toolbar button say Delete Bot? @mareekuh @cwhitten - not a strong opinion, but if I'm scanning the page there are lots of things that might be deletable
here is the Figma:
https://www.figma.com/file/LlTlh5vXwq91zjGnFtrUUR6l/Composer-master-design-spec-UI-library?node-id=2193%3A1035

@DesignPolice How does this look to you?

(-: loud and clear
From: Ben Yackley<EMAIL_ADDRESS>Date: Friday, May 15, 2020 at 12:06 PM
To: microsoft/BotFramework-Composer<EMAIL_ADDRESS>Cc: Marieke Iwema<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/BotFramework-Composer] I can't delete a bot. (#2974)
@DesignPolicehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FDesignPolice&data=02|01|mariekei%40microsoft.com|bd9de974026943b94ea608d7f90321a4|72f988bf86f141af91ab2d7cd011db47|1|0|637251664160381762&sdata=3kHfOP4qd8fzLeR6pthx5SWbMK9gOxcbQmLtC03s42o%3D&reserved=0 How does this look to you?
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F61990921%2F82087020-30750600-96a4-11ea-8425-ee7b590ce156.png&data=02|01|mariekei%40microsoft.com|bd9de974026943b94ea608d7f90321a4|72f988bf86f141af91ab2d7cd011db47|1|0|637251664160391755&sdata=2YHc8a6UYVSLYR2id3HpXBfbmLjpM6Fr75ATUYb55WU%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FBotFramework-Composer%2Fissues%2F2974%23issuecomment-629428032&data=02|01|mariekei%40microsoft.com|bd9de974026943b94ea608d7f90321a4|72f988bf86f141af91ab2d7cd011db47|1|0|637251664160391755&sdata=Aq%2FEp0NecVlQdZFJa5V6cyhtr3%2FLzjoEeYfcLk3%2FUhg%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALSBVCWLEGUZ4D2NMDYG3BTRRWHE3ANCNFSM4M43HGPA&data=02|01|mariekei%40microsoft.com|bd9de974026943b94ea608d7f90321a4|72f988bf86f141af91ab2d7cd011db47|1|0|637251664160391755&sdata=kF5kUC18wc70JeNmSgMiLXhNaDFTsTbhCLt6UHNM4Sk%3D&reserved=0.

Awesome.
This is goofy, but can we shorten the text limit a little bit to get "Going" to drop down to the next line and hopefully get another word to the bottom.. so we don't have folder alone?
If not - push it -
thanks!

@DesignPolice

Wooot!
just the little stuff, appreciate the help - IPAs on me - ship it!


⚠ Bofrost.be has degraded performance

In 713fc6c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1247 ms

Resolved: Bofrost.be performance has improved in bd639ff after 6 minutes.


Updated to the September 29 code of the main fork.

Updated to the September 29 code of the main fork.

Updated to the September 29 code of the main fork.


Merge master for release

Name release version in pr title?

A review is not required for mergin... :-)

I made the release branch protected and allowed only you and I to push to it for now.


DDLS, the objectdependencies filename does not follow specification

https://github.com/SAP/abap-file-formats/blob/main/doc/specification.md#file-names
also see #115

What is wrong with the filename z_aff_example_ddls.ddls.objectdependencies.json?
A content type is specified as mentioned in https://github.com/SAP/abap-file-formats/blob/main/doc/specification.md#content-type.
Do I miss anything?
BTW: I just saw that the filename pattern is specified wrong. I will open a PR to fix it.
<object_name>.<object_type>.
  [<sub_object_name>.<sub_object_type>.]
  [<content_type>.<language>.]
<file_extension>
´´´

It should be be:

<object_name>.<object_type>.
[<sub_object_name>.<sub_object_type>.]
[<content_type>.][.]
<file_extension>
´´´

In #311 we removed this file. So this is not longer an issue.


Update c.ts

Pull Request Template
Please ensure all items below are checked before creating a pull request:

[x] My changes were made in the resources folder, not in the auto-generated README.md file or db folder
[x] My submission is formatted according to the guidelines in the contributing guide
[x] My submission is ordered alphabetically based on the resource name
[x] My submission has a useful description
[x] I have searched the repository for any relevant issues or pull requests

Thank you for contributing, but it's not really a tool for developers specifically.


⚠ ClamAV has degraded performance

In 399b129, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 1b21717.


use el.scrollHeight to calculate offset

see #26

This still works on my project, so it looks good. Changes like this one are why we need #2 (Implement formal testing)


feat: support generic interface generation

Description
This addresses issue #173 regarding mock generation for interfaces with generic types. It works (almost) with the full scope of what is possible to define with golang 1.18. The test file shows the usage of interfaces with multiple type parameters, any constraints, constraints with embedded interface and basic type constraints, as well as aliasing of a generic interface.
Limitation:
An Interface with a generic type that has both a type constraint and additional interface functions defined cannot be generated accurately. In order to ensure the correct interface implementation, the mock needs to be cast to the interface with a specific type parameter. Example:
var _ GenericInterface[string] = GenericInterfaceMock[string]

Consider this type parameter:
type Constraint interface{
   ~int32 | string
   DoSomething() error
}

It is not even guaranteed that an implementation exists, so templating cannot simply pick one. These are the options to handle this case:

Fail to generate such a constraint correctly
Remove the check against the interface in this case
Generate a mock for the Constraint interface

I don't think skipping the interface check without the user specifying it would be the right way to go and option 3 is a more involved implementation. This PR should cover the vast majority of use cases, so I chose to not implement this edge case for now and discuss it before going down that route.

@matryer let me know how I can help to get this change merged!

Not sure why the build did not run here but this broke support for Go versions older than 1.18. We should indicate the same when we release it @matryer

Released in v0.3.0


Trying to compile version 2.20

user@localhost:~/zzz/newsboat/newsboat-2.20$ make newsboat podboat mo-files
cargo build --package libnewsboat-ffi --verbose --release
       Fresh semver-parser v0.7.0
       Fresh cfg-if v0.1.6
       Fresh lazy_static v1.2.0
       Fresh cc v1.0.29
       Fresh ucd-util v0.1.3
       Fresh utf8-ranges v1.0.2
       Fresh autocfg v0.1.2
       Fresh pkg-config v0.3.14
       Fresh matches v0.1.8
       Fresh lazy_static v0.2.11
       Fresh rand_core v0.4.0
       Fresh version_check v0.9.1
       Fresh nodrop v0.1.13
       Fresh smallvec v0.6.10
       Fresh static_assertions v0.3.4
       Fresh unicode-width v0.1.5
   Compiling percent-encoding v2.1.0
       Fresh rustc-demangle v0.1.13
       Fresh bitflags v1.0.4
       Fresh once_cell v1.3.1
     Running `rustc --crate-name percent_encoding /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/percent-encoding-2.1.0/lib.rs --color always --crate-type lib --emit=dep-info,link -C opt-level=3 -C metadata=a15566e7fb442dd4 -C extra-filename=-a15566e7fb442dd4 --out-dir /home/user/zzz/newsboat/newsboat-2.20/target/release/deps -L dependency=/home/user/zzz/newsboat/newsboat-2.20/target/release/deps --cap-lints allow`
       Fresh natord v1.0.9
       Fresh xdg v2.2.0
   Compiling libc v0.2.69
   Compiling memchr v2.1.3
   Compiling semver v0.9.0
     Running `/home/user/zzz/newsboat/newsboat-2.20/target/release/build/libc-5016c61b64efa195/build-script-build`
     Running `rustc --crate-name semver /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/semver-0.9.0/src/lib.rs --color always --crate-type lib --emit=dep-info,link -C opt-level=3 --cfg 'feature="default"' -C metadata=6cd8040965a36448 -C extra-filename=-6cd8040965a36448 --out-dir /home/user/zzz/newsboat/newsboat-2.20/target/release/deps -L dependency=/home/user/zzz/newsboat/newsboat-2.20/target/release/deps --extern semver_parser=/home/user/zzz/newsboat/newsboat-2.20/target/release/deps/libsemver_parser-d1593d6b8e3c62f7.rlib --cap-lints allow`
     Running `/home/user/zzz/newsboat/newsboat-2.20/target/release/build/memchr-4300ed1a69a2ae05/build-script-build`
   Compiling thread_local v0.3.6
     Running `rustc --crate-name thread_local /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/thread_local-0.3.6/src/lib.rs --color always --crate-type lib --emit=dep-info,link -C opt-level=3 -C metadata=6bba36cc1bb90f49 -C extra-filename=-6bba36cc1bb90f49 --out-dir /home/user/zzz/newsboat/newsboat-2.20/target/release/deps -L dependency=/home/user/zzz/newsboat/newsboat-2.20/target/release/deps --extern lazy_static=/home/user/zzz/newsboat/newsboat-2.20/target/release/deps/liblazy_static-20dbc14b9d327b99.rlib --cap-lints allow`
   Compiling gettext-sys v0.19.8
     Running `rustc --crate-name build_script_build /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/gettext-sys-0.19.8/build.rs --color always --crate-type bin --emit=dep-info,link -C opt-level=3 --cfg 'feature="gettext-system"' -C metadata=299e771ed19a136a -C extra-filename=-299e771ed19a136a --out-dir /home/user/zzz/newsboat/newsboat-2.20/target/release/build/gettext-sys-299e771ed19a136a -L dependency=/home/user/zzz/newsboat/newsboat-2.20/target/release/deps --extern cc=/home/user/zzz/newsboat/newsboat-2.20/target/release/deps/libcc-552d437370fadf96.rlib --cap-lints allow`
error: local variables in const fn are unstable
  --> /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/percent-encoding-2.1.0/lib.rs:72:13
   |
72 |         let chunk = self.mask[byte as usize / BITS_PER_CHUNK];
   |             ^^^^^

error: aborting due to previous error

error: Could not compile `percent-encoding`.

Caused by:
  process didn't exit successfully: `rustc --crate-name percent_encoding /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/percent-encoding-2.1.0/lib.rs --color always --crate-type lib --emit=dep-info,link -C opt-level=3 -C metadata=a15566e7fb442dd4 -C extra-filename=-a15566e7fb442dd4 --out-dir /home/user/zzz/newsboat/newsboat-2.20/target/release/deps -L dependency=/home/user/zzz/newsboat/newsboat-2.20/target/release/deps --cap-lints allow` (exit code: 1)
warning: build failed, waiting for other jobs to finish...
error: build failed
Makefile:108: recipe for target 'target/release/libnewsboat.a' failed
make: *** [target/release/libnewsboat.a] Error 101

Do I miss something?
Regards,
Piotr

Hi! It looks like you're running an older Rust compiler. Newsboat 2.20 requires Rust 1.40 or newer (and we'll be bumping that each release, see #709).
Can you update and see if the error goes away?

I am on the server which I planned to migrate to newer version, but it did not happen yet.
Would it be possible to compile on other machine and deploy on the older version?

Would it be possible to compile on other machine and deploy on the older version?

Yes, as long as the older machine has compatible versions of dynamically linked libraries.

Any possibility to add support for creating the AppImage in the make process?

We already support DESTDIR, so just mkdir AppDir && make DESTDIR=AppDir install, and you should be good to go. Is there anything else that we need to do on our end? (I only skimmed the doc you provided, sorry.)
We also have a Snap package, which I believe does essentially the same thing as AppImage does, albeit somewhat differently.

I will try to compile it on my newer version of Debian and package then. If I face some issues then I will ask for some help here. :)
I tried the Snap, but is not supported on my older machine (I still runs there Debian 8.1 but snapd is available from 9 onwards).

Another thing you can do is install Rust via Rustup.

I managed to package it into AppImage using following steps:
make
make DESTDIR=AppDir install
mkdir AppImage
convert logo.svg -resize 32x32 newsboat.png
wget https://github.com/linuxdeploy/linuxdeploy/releases/download/continuous/linuxdeploy-x86_64.AppImage
chmod +x linuxdeploy-*
./linuxdeploy-x86_64.AppImage  --appdir AppDir -e newsboat -i newsboat.png --create-desktop-file --output appimage

This will generate the file newsboat-x86_64.AppImage which can be run standalone on the another platform.
So if you want to add another possibility for your build you can use these steps.
The only problem is that I was compiling on 64 bit system and the target is on 32 bit. How can I compile from 64 bit system 32 bit target binaries?
Regards,
Piotr

So if you want to add another possibility for your build you can use these steps.

Thanks, but it's complex enough already.

How can I compile from 64 bit system 32 bit target binaries?

Easy way: follow the comments from docker/ubuntu_18.04-i686.dockerfile (but you'll need Docker installed for that). Harder way: follow the commands from that Dockerfile (but you'll need to understand the Docker file syntax for that). No matter which path you choose, run make distclean beforehand, to remove the 64-bit objects from your local copy.

OK. I will try this. Thanks!


[release-2.7] upgrade addon framework to handle manifestworks larger than 50k

Also includes updates to upgrade controller runtime to v0.14.4 for compatibility with OCP 4.14

We'll tackle the test coverage for the component as a separate issue.


handshake error when connecting to https, works fine with http

HandshakeException: Handshake error in client (OS Error:
CERTIFICATE_VERIFY_FAILED: ok(handshake.cc:354))
is there any configuration that I need to do on client side ?
can we have a badCertificateCallback like in HttpClient ?
static HttpClient client = new HttpClient()
..badCertificateCallback = (_certificateCheck);
static bool _certificateCheck(X509Certificate cert, String host, int port) =>
host == 'local.domain.ext';

Hi @LtotheWT, this library was built to be used on the server side. It does not work on Browsers or Flutter. You will find alternatives to it on https://pub.dev.

i added badCertificateCallback and its working now, thanks for the repo mate


Support v1.4.0 Spec Update

Changes

Add support for v1.4.0 of the specification (with support for the new Construction API)

TODO

Update asserter/construction.go
Update asserter/request.go
Update fetcher/construction.go

Pull Request Test Coverage Report for Build 1881

45 of 187 (24.06%)  changed or added relevant lines in 4 files are covered.
21 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-4.6%) to 62.102%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fetcher/mempool.go
0
1
0.0%

asserter/block.go
15
21
71.43%

asserter/construction.go
0
67
0.0%

asserter/request.go
30
98
30.61%

Files with Coverage Reduction
New Missed Lines
%

asserter/block.go
3
89.3%

asserter/request.go
18
50.69%

Totals

Change from base Build 1821:
-4.6%

Covered Lines:
1483

Relevant Lines:
2388

💛  - Coveralls


Start Native Python REPL errors: UNKNOWN notebook type: jupyter-notebook

Discussed in https://github.com/microsoft/vscode-python/discussions/24590

Originally posted by ChiNoel-osu December 11, 2024
When I try to start the REPL it shows this error message:

Smart send (Shift+Enter) gives the same error message:

This was working before, I was just using it, and I have no idea why it suddenly stopped working.
2024-12-12 11:55:02.185 [error] Error: UNKNOWN notebook type: jupyter-notebook
    at VUe.f (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1886:34374)
    at VUe.createUntitledNotebookTextModel (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1886:35512)
    at Y6e.$tryCreateNotebook (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:780:43024)
    at Fdt.S (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:3145:41680)
    at Fdt.Q (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:3145:41458)
    at Fdt.M (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:3145:40546)
    at Fdt.L (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:3145:39650)
    at aee.value (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:3145:38444)
    at x.B (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:30:748)
    at x.fire (vscode-file://vscode-app/c:/Users/victor1.wang/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:30:967) python.startNativeREPL {"value":"ms-python.python","_lower":"ms-python.python"}

I don't have Jupyter extension installed. It was working without that extension too. I tried installing it now but still this error throws.
Terminal REPL works, but native REPL doesn't.

It somehow managed to fix itself today, really had no clue what's going on.
Here's my troubleshooting process:
If I delete all user data and install the Python extension alone, the REPL will work.
But as soon as I turned on setting sync and have all the settings and extensions synced, the REPL stopped working and throws that error message to me again.
I tried doing a clean reinstall of VS Code, same issue.
I tried disabling all extensions except the Python ones, the problem still persists.
I tried installing the jupyter extension, same issue. (REPL was working without this extension too)
I tried downgrading Python extension from v2024.22.0 to v2024.20.0, same issue.
Then I tried going into setting.json, and it doesn't seem to have anything related to REPL. But still I cleared up some grayed out or unnecessary settings. After that the REPL still doesn't work. I also don't think setting.json is causing the issue because REPL was working with this setting.json before.
And then I tried numerous of things including upgrading the Python extension back on and again deleting user data. Then in a session, the native REPL suddenly starts working again. Have no idea why.
Now everything is back to normal. But I think we should leave this issue open because this is quite bizarre.
I was using remote Python REPL (ssh to a server in my LAN) when the issue first happened, but this issue persists locally.


ImGui in Z# (Intergrated)

I've been kind of wondering if this is possible but something like a toggle for ImGui would be nice, I kind of want it integrated instead of something like Importing it constantly!
Could there be an Internal ImGui function which could be used for game development?
I'm not sure if it is implanted but would like too see it, I haven't used this for quite some time so I don't know.
Just a simple line like; enable ImGui;
this is probably never going to be used but I'll probably use it since its quite major for games, ImGui is used in a load of big games in companies like Ubisoft so like would be neat to see!

just edit the files of the GUI.


ScalaMock does not work with ScalaTest's TestNG/JUnit Support Traits

ScalaMock currently does not integrate with ScalaTest's TestNGSuite or JUnitSuite traits.  I am using ScalaMock_2.9.1-2.2 and ScalaTest_2.9.1-1.6.1.
Here's a sample test, using TestNGSuite:
import org.scalamock.ProxyMockFactory
import org.scalamock.scalatest.MockFactory
import org.scalatest.testng.TestNGSuite

class ScalaMockTestNG extends TestNGSuite with MockFactory with ProxyMockFactory

The following error is produced:
error: overriding method runTests in trait TestNGSuite of type (testName: Option[String], reporter: org.scalatest.Reporter, stopper: org.scalatest.Stopper, filter: org.scalatest.Filter, configMap: Map[String,Any], distributor: Option[org.scalatest.Distributor], tracker: org.scalatest.Tracker)Unit;
 method runTests in trait MockFactory of type (testName: Option[String], reporter: org.scalatest.Reporter, stopper: org.scalatest.Stopper, filter: org.scalatest.Filter, configMap: Map[String,Any], distributor: Option[org.scalatest.Distributor], tracker: org.scalatest.Tracker)Unit cannot override final member;
 other members with override errors are: runTest
class ScalaMockTestNG extends TestNGSuite with MockFactory with ProxyMockFactory
      ^
one error found

It appears that ScalaTest's TestNGSuite does not use the runTest method.
From the ScalaTest TestNGSuite source (http://code.google.com/p/scalatest/source/browse/tags/release-1.6.1/src/main/scala/org/scalatest/testng/TestNGSuite.scala ):
 The main purpose of this method implementation is to render a compiler error an attempt
 to mix in a trait that overrides runTests. Because this
 trait does not actually use runTests, the attempt to mix
 in behavior would very likely not work.

So it sounds like this is not an issue with ScalaMock's ScalaTest integration - instead it's the fact that ScalaMock doesn't have any JUnit or TestNG integration. See this discussion for details:
http://groups.google.com/group/scalatest-users/browse_thread/thread/34c251f7298ac538
JUnit and TestNG integration are on my todo list, but if you (or anyone else) fancies having a go and contributing the changes, I'd be delighted to accept a pull request. I'd be very happy to help in any way if you do fancy taking up the challenge.

Do let me know if you need anything on the TestNG side.

Cedric - is that an offer to take on its development? :-)
If so, yes please!
If not, I'll certainly get in touch for help when I look at it myself.
Thanks!

Not really, no :-)
Just offering to help if there's anything that needs to be done on the TestNG side.

Cool - I'll certainly be in touch when I get around to looking at TestNG. Thanks!

Can reproduce this compiler error, but I don't think there is much desire to implement special integration with the TestNGSuite in Scalatest at this point


Refined Storage API for OpenComputers not working

Issue description:
Can not access the fields of an Itemstack in oc.
What happens:
After retrieving the items of the system you are unable to access any information of them because everything returns null.
What you expected to happen:
Steps to reproduce:

Make an opencomputers setup
Get the items of the refined storage system
Try to access the fields of an itemstack (name, label, etc)

Version (make sure you are on the latest version before reporting):

Minecraft:   1.12
Forge:
Refined Storage: latest

Does this issue occur on a server? [yes/no]
yes

Fixed in #ac1a8d4f614040645eef1c6e57455a132a4d77bf


Fixups in CHANGES.md

Fixed one ambiguous line and removed a few unnecessary backticks.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

The Travis passed here just the reporting back to github is somehow stuck.


configurable speed format

It would be good if library allows configuration of the format of the reported speed. Currently it's reported in meters/second but it would be valuable if is allowed and km/h to be reported.

You can easily convert units in your code. Don't feel it should be plugin's job.


Stream event logs to configurable channels

It would be useful for kube-monkey to have the ability to send events ("killed pod foo for app bar") to channels like Slack, HipChat etc.
In the event that an action taken by kube-monkey causes an issue, the ability to quickly correlate that could be valuable.

Fixed by #175


getParentViewModel throwing exception

when tried to access parent viewmodel using getParentViewModel from the BaseViewwModel,
got an exception.
debugIsInInheritedProviderUpdate': Tried to listen to a value exposed with provider, from outside of the widget tree.
as of this issue
#https://github.com/rrousselGit/provider/issues/313
had to add listen: false to get it fixed

Are you sure the provider falls under the same tree?

I was trying to access viewmodel of a view with ListView from viewmodel of ListTile.
please, check the test project in this repo
getParentViewModel

You're not supplying your viewmodel anywhere in that example. Please see how to use the ViewModelBuilder widget and the re-write your code to use that.
You need to use the ViewModelBuilder.reactive / non-reactive constructor to supply the viewmodel

I'm using it as in here.
and it's working fine, but the problem is when I try to get the parentViewModel in here


Add colored output

Added README section

Thanks for this contribution.  It looks really nice with colored output.


Fix CLS in DropDown / Popper

Closes #245
📑 Description
Just using visible as a class does not remove the space it needs from the dom. Well, it somehow does for some reason, but not before half a second or so, which causes half a second of CLS.
Using "hidden" removes it from the dom, but breaks the animation.
Solution: use "absolute". Keeps the animation & removed the space from the dom => no CLS anymore.
This is a simple fix, but I'd still be interested to know why that space from the dom is removed after like half-a-second? Is it adding some kind of hidden element after a delay or so to keep the animation? Because if so, that could be removed

The delay in removing the dropdown comes from the hideHandler:
	const hideHandler = (ev) => {
		if (activeContent)
			setTimeout(
				() =>
					(!clickable && (triggerEl.matches(':hover') || contentEl.matches(':hover'))) ||
					contentEl.contains(document.activeElement) ||
					(open = false),
				100
			);
		else open = false;
	};

This is introduced to manage the cases when you move the mouse from the triggering button onto the dropdown. Dropdown gets hidden when button or dropdown loses focus/hover, and doing that move the cursor is in-between the two for a while causing the dropdown to close.

The delay in removing the dropdown comes from the hideHandler:
	const hideHandler = (ev) => {
		if (activeContent)
			setTimeout(
				() =>
					(!clickable && (triggerEl.matches(':hover') || contentEl.matches(':hover'))) ||
					contentEl.contains(document.activeElement) ||
					(open = false),
				100
			);
		else open = false;
	};

This is introduced to manage the cases when you move the mouse from the triggering button onto the dropdown. Dropdown gets hidden when button or dropdown loses focus/hover, and doing that move the cursor is in-between the two for a while causing the dropdown to close.

That one doesn't seem to affect the hiding of the space occupied by the dropdown after a page load, though. Even when removing the hide handler, the CLS disappears after half a second

On top of the above you have a default animation ‘duration-300’. This should explain.

On top of the above you have a default animation ‘duration-300’. This should explain.

Hm even removing the animation completely doesn't fix it. Here's a screen recording to showcase the CLS I'm talking about:
https://user-images.githubusercontent.com/70709113/186657611-b172bf62-73a7-4527-8fb0-0052107baa18.mov
.

Alright so I've further looked into it, and one main problem seems to be that this style: property seen in the image is being set too late:

Because that's what hides it. Generating this style: property earlier, right when the page loads, might be an even cleaner solution than adding the "absolute" class name manually

This should be a good solution now! createPopper() creates the style: attribute, but it doesn't run on the server and only when the client hydrates (it seems). So until that runs, I just added a manual style: attribute which hides the dropdown until createPopper() runs, thus eliminating the CLS.
So before client finished hydrating:

After:

Should be good to go now, though maybe that popper stuff could be improved in the future to not cause a CLS by default and run on the server during SSR as well

As for that modification, I'm not comfortable with adding that style via style property, as you proposed in that PR. We should achieve the same result by adding absolute class to divClass. Would that work? Or will we experience some SSR issues?
	let divClass: string;
	$: divClass = classNames(
		'z-10 absolute',
		animation && `transition-opacity ${animation}`,
		open ? 'visible opacity-100' : 'invisible opacity-0',
		$$props.class
	);

That seems to work as well, but are you sure that's the better solution? The createPopper thing adds in the style: attribute anyways. If we add absolute to the classname like you proposed, it would be a duplicate once createPopper runs (there would be an absolute class name AND a position: absolute style rule). While if I add it as a style (like it is currently), it simply gets overridden by createPopper once not needed anymore and is thus not duplicated.

Fair point. Let’s go with your proposal in the PR.

Merged. Thank you for your contribution.

Can you check Dropdown navbar