Make GRIB API optional.

NB. This adds a new entry to the test matrix which runs the standard test suite against an installation with a reduced set of dependencies. Currently this is only missing ecmwf_grib but I would expect other dependencies could be removed once the concept is proven.

NB. The examples and doctests are only tested with the full set of dependencies. The examples will certainly fail if ecmwf_grib is missing - I'm not sure about the doctests.

The examples will certainly fail if ecmwf_grib is missing - I'm not sure about the doctests.

Whether this is the behaviour we want is debatable! :wink:

:tada:
Thanks @pelson :grin:


ExternalAssertionLibraryError when assertion gets failed in testcafe

getting assertion on failed assertion
await t.expect(false).ok("response of student school is not in arrayformat");
testcafe 1.7.1

@sachin-funde
Hello,
Please send us a small runnable project so that we can research it on our side, as I wasn't able to reproduce this issue under Ubuntu.
My setup:
test.ts:
import { Selector } from 'testcafe';

fixture `https://google.com`;

test('Test', async t => {
    await t
        .expect(false).ok('message');
});

Command:
npx testcafe firefox test.ts -r allure

I look forward to your response.

if this helps, it happens also if we miss async keyword
in PageModel.ts
  public async login(userName: string, password: string) {
    await t.expect(1).eql(2)

    }

then in test-example.ts
pageModel.login()

Hi, @devmondo
I cannot reproduce the problem again. Could you please provide a complete example that I can run locally without additional set up?

hi @miherlosev,
sure thing :)
as I mentioned before, if we add the missing await the assertion will fail but it won't crash the process, but as soon as you miss the await the process crashes.
you don't necessarily need a page model, just use t.expect(1).eql(2) without await in the test body and it will lead to the same crashing behavior.
page-model.ts
import {Selector, t} from 'testcafe';

export class PageModel {
    public async login() {
    await t.expect(1).eql(2)
    }
}

export const pageModel = new PageModel();

test.ts
import { pageModel } from "./page-model";

fixture(`test`);
test(`test`, async () => {
    pageModel.login();
})

testcafe.js
const createTestCafe = require("testcafe");

let testcafe = null;

createTestCafe('localhost', 1337, 1338,null,true )
    .then(tc => {
        testcafe = tc;
        const runner = testcafe.createLiveModeRunner();
        return runner
            .src(['./test.ts'])
            .browsers(['chrome'])
            .run();
    })
    .then(failedCount => {
        console.log('Tests failed: ' + failedCount);
        testcafe.close();
    }).catch((e)=>{
    console.log(e.stack);});

result error
Make changes to the source files or press Ctrl+R to restart the
 test run.
ExternalAssertionLibraryError { code: 'E53', isTestCafeError: t
rue, callsite: CallsiteRecord { filename: 'C:\\projects\\test\\page-model.ts', lineNum: 4, callsiteFrameId
x: 6, stackFrames: [
      [CallSite],  [CallSite],  [CallSite],
      [CallSite],  [CallSite],  [CallSite],
      [CallSite],  CallSite {}, [CallSite],
      CallSite {}, [CallSite],  [CallSite],
      [CallSite],  CallSite {}, [CallSite],
      CallSite {}, [CallSite],  [CallSite],
      [CallSite],  [CallSite],  [CallSite],
      [CallSite],  [CallSite]
    ], isV8Frames: true }, errMsg: 'AssertionError: expected 1
to deeply equal 2' }
(node:25800) [DEP0018] DeprecationWarning: Unhandled promise re
jections are deprecated. In the future, promise rejections that
 are not handled will terminate the Node.js process with a non-
zero exit code.

using windows 10 64x and latest testcafe

i also tested @sachin-funde code above, but it does not crash... weird!
await t.expect(false).ok("response of student school is not in arrayformat");

@devmondo
Thank you for the provided information. I've reproduced the problem.

@devmondo
Thank you for the provided information. I've reproduced the problem.

@devmondo
Thank you for the provided information. I've reproduced the problem.
How to fix this issue . please do the needful .. our tests are failing

The fix is not available yet. We will update this thread once it is ready. You are welcome to submit your PR if you want to fix this issue on your own.

As we upgrade to Node 13  test cafe fails.

Thank you, we will take this into account. Also, please be aware that non-LTS versions of Node.js might have some bugs or regressions. If you face any (not related to this issue), please feel free to open a separate issue.

Is there any update on this? I just went on a wild goose chase and ended up here...
try {
fetch... // network call
await t.expect() // fails
} catch(e) {
e is a ExternalAssertionLibraryError when I spect it to be from the failed fetch
}|

@jasongornall, We have no updates regarding this issue yet.
If the fetch function originated from the node-fetch package, I suggest you place await before calling the function. Also, I recommend you move any TestCafe assertions outside of the try {} block to catch errors from the failed fetch (not TestCafe assertions). This example works fine for me:
import fetch from 'node-fetch';

fixture `GET requests via 'node-fetch' package`;

test(`Request to Github Users API`, async t => {
    const jsonResponse = await (await fetch('https://api.github.com/users/github')).json();

    await t.expect(jsonResponse).contains({ location: 'San Francisco, CA' });
});

test(`Request to invalid domain`, async t => {
    let jsonResponse = '';

    try {
        jsonResponse = await (await fetch('https://domain.invalid/')).json();
    }
    catch (e) {
        // do something with e
    }

    await t.expect(jsonResponse).contains({ location: 'San Francisco, CA' });
});


Add Mechi Multiple Campus

Institution/School Name
Mechi Multiple Campus
Instiution/School Website
Campus Website: https://mechicampus.edu.np/
University Website: http://tribhuvan-university.edu.np/ or http://tu.edu.np/
Domain Used in Email: memc.tu.edu.np
Mechi Multiple Campus is a constituent campus of Tribhuvan University(TU) Nepal.
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[x] Post-graduate research
[x] Graduate School
[x] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@NightmareGaurav Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@NightmareGaurav Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@philipto  the domain mentioned as 'Email domain' [memc.tu.edu.np] is subdomain of [tu.edu.np]. tu.edu.np points/redirects to http://tribhuvan-university.edu.np which is official website of Tribhuwan University, under which Mechi multiple campus runs. From nav bar>academics>campuses, we can get list of campuses that runs under TU(Abbreviation to Tribhuwan University), and then under the  'Constituent Campuses>Pradesh No.1' section third entry is mechi multiple campus, which points to the Mechi multiple campus's official website: https://mechicampus.edu.np/
If that is not enough, please let me know, i will talk to university and campus officials about it...

@NightmareGaurav Thank you for the clarification. Pull request merged.


Feat/dockerize frontend

Should I also add a docker-compose file?

@SID262000 How do you suggest that we document such change?

@idodav Please refer to this section. This will really help to improve the quality of docker files(both existing & new ones).
https://docs.docker.com/develop/develop-images/dockerfile_best-practices/
cc: @Oferlis

@Oferlis Is this ready for review?

@idodav Please take a look at the changes, and proxying works, but ports must be exposed.
A solution is exposing a range of ports and notifying users that these ports must be available for use by Sharkio.
What do you think? is there a better solution?
I could also open this issue to discussion in the Discord channel

It's fine. The EXPOSE keyword is mainly for documentation.
It doesn't actually expose the ports.


Android Support

What would you like to be added:
Support for android Projects.
Why is this needed:
To have SBOMs for Apps, which is quite interesting as a lot of apps already deliver some kind of file mentioning all licenses and dependencies
Additional context:
I tried to scan a public project but the gradle scanner only found the gradle-wrapper
sorry that I don't have much more context on this. Questions are highly appreciated if there are more.

Hi @henrysachs, thanks for the suggestion. We will consider this as a possible new feature in the future. If you'd like to take on work to implement this, we'd be happy to provide some guidance. Please feel free to stop by our Slack channel or our twice-monthly community meetings to discuss. I'll leave this ticket open for tracking. Thanks.

I talked about gradle in this MR. I think it would work for android too then.
#707
I seem to missed out on the slack channel and will have a look there.

Is this a duplicate of #690 -- e.g. if we solve build.gradle support, does it also solve android project support?


(docs): various updates in README.md

Description

correct spelling
correct grammar
add punctuation
make markdown formatting consistent

Related issues
N/A
Notes
N/A

Thanks @vladdoster ! English isn't my primary language ;)


Update sites.json - add lifeofpablo.com to BOTD

The change here is to add lifeofpablo.com to blog of the day.

Thank you for submitting @pmoralesgarcia ! Have a great week!


[user feedback] GO "About" page discoverability

User feedback for consideration:

as a person who hadn't used GO before, I was confused as to what GO was for and how to use it.  It would be good if there was an "About" page with information about what GO was for.

User had not been aware that there was an about page https://go.ifrc.org/resources
[from Clementine prior to starting with IM team]
response from @LukeCaley:  there is an about page, need to make sure it's clearer in the nav, and that users can find the overview info e.g. GO video.
[I am logging on behalf of Luke and Clementine so that this feedback is tracked for consideration for action]

Additional piece of feedback:
Personally, I would rather just read a 5-line paragraph that explains the purpose and scope of GO rather than watch a 3-minute video to get a quick high-level overview. Maybe it can be a useful addition to the page.


Not installing

Hi @BBassi, thanks for reporting the error. I updated the dependencies so that on my machine the theme works with the new release 2.0 of JupyterLab. If you can confirm that you can install it as well I will close the issue.

Resolved


Homepage feed boxes compiled of actual communities

I will first put the available posts like "deneme" title and "şskdfs" description to the thumbnails

Lets render communities instead of posts

Without formatting, it is now implemented

I broke it.

I've fixed it. Again


Add ChromeOptions

This allows passing additional options to chrome. For example to pass --headless and use a different binary, it'd look like this:
	wd := agouti.ChromeDriver(
		agouti.Browser("chrome"),
		agouti.ChromeOptions("binary", "/path/to/chrome"),
		agouti.ChromeOptions("args", []string{"--headless"}),
	)
	if err := wd.Start(); err != nil {
		panic(err)
	}
	defer wd.Stop()
	p, err := wd.NewPage()
	if err != nil {
		panic(err)
	}
	if err = p.Navigate("https://www.google.com"); err != nil {
		panic(err)
	}

While it's possible to pass those values via Capabilities, the Option make it more convenient and clear.

Looks great! Thanks again!

Sorry to bring up an old issue.  With the current implementation of ChromeOptions, am I unable to pass these options to a remote instance of Selenium, in this case the Selenium standalone Chrome Docker container?

@wlaoh does #69 address this?


test: add another test

TL;DR
Added a new test case to ensure the minimum value is less than the maximum value.
What changed?
Introduced a new test in main.test.ts that checks if the function throws an error when the maximum value is lower than the minimum value.
How to test?

Run the test suite
Verify that the new test passes
Try modifying the wait function to allow maximum values lower than minimum values and ensure the test fails

Why make this change?
This change improves the test coverage by explicitly checking for an edge case where the maximum value is incorrectly set lower than the minimum value. It helps ensure that the wait function properly validates its input parameters, preventing potential logical errors in the application.

#386
#385  👈
#384
next

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AliSajid and the rest of your teammates on  Graphite

Merge activity

Oct 11, 12:28 PM EDT: A user started a stack merge that includes this pull request via Graphite.


Python pickle version fix

Added support to allow to compile Godot from the same directory while using different versions of python.
More info:
I compile Godot using different a combinations of dockers and just straight off my main system out of the same directory.
My main system uses bleeding edge version of python - 3.8
while my docker uses an old version of python for stability - 3.5
scons uses the pickle library to persist what files have been built, but it saves it using pickle.HIGHEST_PROTOCOL
this means if I build using my main system, the generate .sconsign.dblite is in pickle format 5 (which was changed in python 3.8 I believe).
then if I went to build on my docker it would fail with a pickle error.
the previous solution was to delete the .sconsign.dblite file to allow the build to run, but the whole project would be recompiled from scratch!
this patch resolves the problem by adding in the pickle version into the filename.

Seems OK to me apart from the couple comments I made.
Might be worth seeing with upstream SCons if that's something they'd want to tackle directly on their end, or if this kind of multi-python support flexibility is something to be done downstream as proposed here (cc @bdbaddog).

issues have been addressed

Seems good now. Could you squash the commits into one?

there we go!

Thanks!

With the introduction of Python Pickle version 5, it may be that highest isn't always the best choice for file serialization.  The pickle.DEFAULT_PROTOCOL for 3.8 is 4 not 5.
Let me see if I can get some advice from Python folks about 4 vs 5 for file serialization.

I had missed that this PR was made against the vulkan branch :(
Please make all PRs against the master branch, there's no reason for this fix to only be vulkan-specific.

Cherry-picked for 3.2.


unable to access 'https://gerrit.opencord.org/manifest/'

Hello! I'm trying to install test CORD using https://github.com/opencord/cord/blob/master/docs/quickstart.md  , but I'm stuck on error
getting repo access: fatal: unable to access 'https://gerrit.opencord.org/manifest/': Operation timed out after 0 milliseconds with 0 out of 0 bytes received

It was related to my MTU settings, sorry!


Video not playing on ios

Video captured using this lib are not playing on IOS device. I am using video tag for playing the video. If I capture the video on android then its working fine.

This is an issue tracker, not a support board. Please read the documentation for how to get support. Please also read the contribution guidelines for how to submit bug reports, as you did not follow those instructions either.
There is nothing that I am going to do specifically about this. I am simply asking the camera hardware to record a video using MediaRecorder. I am not in position to guarantee compatibility with arbitrary programs running on arbitrary operating systems, for videos recorded on arbitrary hardware using arbitrary application code that happens to use this library.
You might wish to track issue #269, where I will look to offer somewhat greater configuration control over the video recording process. You could use those options to see if you can find ones that you prefer.

Hi Mark,
Thanks for the response, I think with grater configuration you mean setting
the more profile parameter such as bitrate,frame width/height then it would
be grate. Will follow the issue #269 you mentioned.
....
On Wednesday, November 9, 2016, Mark Murphy<EMAIL_ADDRESS>wrote:

This is an issue tracker, not a support board. Please read the
documentation for how to get support
https://github.com/commonsguy/cwac-cam2#questions. Please also read the
contribution guidelines
https://github.com/commonsguy/cwac-cam2/blob/master/.github/CONTRIBUTING.md
for how to submit bug reports, as you did not follow those instructions
either.
There is nothing that I am going to do specifically about this. I am
simply asking the camera hardware to record a video using MediaRecorder.
I am not in position to guarantee compatibility with arbitrary programs
running on arbitrary operating systems, for videos recorded on arbitrary
hardware using arbitrary application code that happens to use this library.
You might wish to track issue #269
https://github.com/commonsguy/cwac-cam2/issues/269, where I will look
to offer somewhat greater configuration control over the video recording
process. You could use those options to see if you can find ones that you
prefer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/commonsguy/cwac-cam2/issues/268#issuecomment-259424779,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABuL8ILF_s8KgXQpUszvFoIP4Wvn8ND_ks5q8dZYgaJpZM4KtkTI
.

--
Thanks & Regards,
Mourice Fargose


Allow setting CACHE_DIR and BUMBLEBEE_OFFLINE via Config

Hey,
I'm wondering why CACHE_DIR and BUMBLEBEE_OFFLINE options are loaded directly from the system environment variables and not from the application config as is commonly done in other Elixir libraries.
Replacing System.get_env with Application.get_env would allow greater flexibility. It can default to the env variables for backward compatibility.

This is because they were meant to be used from scripts. See examples/phoenix/README.md that explains how they are used. You can also pass those to as options to Bumblebee itself.

You can pass these and more as {:hf, "...", opts}, see t:repository/0. With that, you are free to use configuration in your app :)


libswoc - replace string_view_util with libswoc version.

This is needed both to remove redundancy and to avoid circular dependencies, as libswoc internals depend on the functions in the string_view utils.

[approve ci fedora]

The builds are failing due to linking issues. Here's a link to the Ubuntu failure:
https://ci.trafficserver.apache.org/job/Github_Builds/job/ubuntu/389/console

Oh, I'm tracking the linking issues. The problem is all of that works perfectly locally. This is part of resolving the linking issues with #9193 .

[approve ci rocky]

Seriously? That was the fix? Gah.

Seriously? That fix worked? Gaaaaaaaaaaaaaaaaaaaaaaah!


Modifying onDelete on LevelInfoLayer causes crash

Geode Issue

[x] I confirm that this crash is NOT related to a mod but directly to Geode Loader itself.

Platform
Windows
Geode Version
v1.3.6
Mods Installed
alphalaneous.testondelete.geode (For me to test the issue, all other mods have been moved out of the mods folder)
Steps to Reproduce
Modify LevelInfoLayer::onDelete(CCObject* sender):
#include <Geode/Geode.hpp>
#include <Geode/modify/LevelInfoLayer.hpp>

using namespace geode::prelude;

class $modify(LevelInfoLayer) {

    void onDelete(CCObject* sender) {
        LevelInfoLayer::onDelete(sender);
    }
};

Crash Report
2023-11-17_07-01-34.log
Geode 17 Nov 07.01.25.log
Additional Information
No response

Binding for onDelete is wrong, in the broma its 0x17a2b0 when it should be 0x17a1b0

Epic, thank you


Trigger guide test to run when subproject is published

Whenever a donejs project is published we need to trigger the tests to run in CI.

https://docs.travis-ci.com/user/triggering-builds

@m-mujica close this if this is complete.


fix: Make OrganizationEditPages' field column wider when screen width is small.

@Wrapping-2000

Plz rebase master onto your branch first.

In my humble opinion, the pr makes the frontend too complex. Why not just modify the value of span in Col according the language or simple setting value of span more large.


Error

Got this error even in compatibility mode.
Log: cleo.log
Version: alpha.33
Mod: https://www.mixmods.com.br/2018/07/saas-san-andreas-ambient-sounds/

Send screenshots of

game folder
Cleo folder
cleo\cleo_plugins folder

Send screenshots of

game folder
Cleo folder
cleo\cleo_plugins folder

also install NewOpcodes.cleo and try
https://www.mixmods.com.br/2020/10/newopcodes-cleo-v2-1/


Change analyzer

I'd need to change the analyzer to whitespace for my URL Slug fields, otherwise, it'll ignore dashes. Any idea how can I solve this quickly and update the index?

For now, I just deleted the index by hand and recreated the model with the necessary changes. Would be nice to have something for this though.


Do not enumerate unavailable digests and ciphers in list -*-commands

Fixes #13594

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

CI minimal failure is relevant, but I don't know how to properly deal with it. The failing test takes the output of -cipher-commands and plans tests according to it.
It can either be fixed either via updating via apps/progs.pl or hard-coding.

The failing test takes the output of -cipher-commands and plans tests according to it.

Can we rewrite the test to not use -cipher-commands, e.g. use -cipher-algorithms instead? We shouldn't be using deprecated stuff in our tests.

Can we rewrite the test to not use -cipher-commands, e.g. use -cipher-algorithms instead? We shouldn't be using deprecated stuff in our tests.

The -cipher-algorithms is not equivalent option. In fact, there is no strictly equivalent option.
May be we should not deprecate these options and just add the checks for presence of the algorithms?

May be we should not deprecate these options and just add the checks for presence of the algorithms?

Hmm. I'm not keen on that. We really don't want to have this hard coded list for any longer than we have to.

Ok. I have some ideas how to eliminate the command from tests and hopefully will be able to do it

My suggestion would be to condition that test out in a deprecated build.  I don't think it is a relevant test anymore -- it was created as a sanity check.

This pull request is ready to merge

Merged. Thanks!

@beldmit I noticed that the following now fails:
./config no-deprecated && make -s && make cmd-nits
Configuring OpenSSL version 3.0.0-alpha10-dev for target linux-x86_64
Using os-specific seed configuration
Creating configdata.pm
Running configdata.pm
Creating Makefile

**********************************************************************
***                                                                ***
***   OpenSSL has been successfully configured                     ***
***                                                                ***
***   If you encounter a problem while building, please open an    ***
***   issue on GitHub <https://github.com/openssl/openssl/issues>  ***
***   and include the output from the following command:           ***
***                                                                ***
***       perl configdata.pm --dump                                ***
***                                                                ***
***   (If you are new to OpenSSL, you might want to consult the    ***
***   'Troubleshooting' section in the INSTALL.md file first)      ***
***                                                                ***
**********************************************************************
/usr/bin/perl ./util/find-doc-nits -c
doc/man1/openssl-list.pod: list does not implement -cipher-commands
doc/man1/openssl-list.pod: list does not implement -digest-commands
make: *** [Makefile:3652: cmd-nits] Error 1

Is this expected?

No, this is not expected. Will take a look.

Fix in #13728

No, this is not expected. Will take a look.

Fix in #13728

Great, thanks for the quick follow-up!


Hot reload

This pull request adds hot-reload support using the webpack dev server

I think we need to get this PR into "ned" branch


Bring up installation instruction in README.MD

Users need to get up and running instruction as soon as possible :)

Thanks @omidfi - I appreciate the PR!


Added Hi-DPI support for Retina, scaled or mobile displays

Modified the canvas creation routine slightly to allow matching device DPI and thereby avoiding blurring scaling artifacts on Retina, hi-dpi displays.
Also including the latest build...

Latest checkin correctly detects the "options" for values of a checkbox field type. The default behavior ignored that the on/true/Yes value is controlled via the export value of the PDF and instead set 1 for the .value attribute of the field.

Extracting values for radio buttons from the appearances dictionary and putting them in the HTML element value attribute means that radios now set and use the correct values (instead of indices).

I've added you as a collaborator. You're commits seems fine, and given the pace you're going I don't have the time to test them all. Feel free to commit whatever enhancements you want, as long as you don't break default functionality!

Most of my changes should be minor now, I think it's pretty stable and I've been testing against a wide range of PDFs and browsers. The only things left that I may want to investigate is matching fonts and sizes a bit better, and perhaps an improvement to multiselect/listboxes (which are just dropdowns right now). Otherwise this pretty much meets needs now, thanks for the work you put into getting it as far as you did!


Demo has bug leading to negative weak/dead branches

Overhauled demo with compounding, no longer getting negative weak/dead.


Returning to NULL doesn't seem to trigger a prefetch abort

Might be a Unicorn bug. Need investigating.

Well surprise surprise setting until to NULL terminates the emulator when it tries to execute NULL.


Visitor API cleanups

Fixes #171.
Partially fixes #181 (the first two bullets).

Make Visitor protocol class-constrained.
Simplify protoFieldNumber argument label to just fieldNumber. (We should do the same across the code base.)
Remove side effects from visitor initializers, except JSONEncodingVisitor (needs more work to cleanly handle special cases).
Reformat some files.
Delete MirrorVisitor and remove CustomReflectable conformance from Message.
Remove some explicit tab width settings that have collected in the PBX file.

@tbkka Anything else you want to see on this?

Sorry for the long delay; I was on vacation over the holidays.
Summary:  This looks great.  Let's merge it.
Random observations:

THANK YOU for documenting the Visitor protocol.   That was greatly needed.
Removing the initializer side effects is  nice code cleanup.

It recently occurred to me that there's an interesting trade-off between using a reference type for things like this vs. using an inout struct type:  The former reduces casting woes, but the latter makes the lifetime more explicit.  I think in this case, it turns out to be a win, but I'm starting to prefer inout structs in general.  Something to think about...


How can I just test one image?

I just want to test one image, and show its mask result....
Can you tell me what should I do??
Maybe "tests/test_data_samplers.py", but I don't known how to run it?
Thanks!

I 'm also Confusion about it, maybe the author didn't provide the code to test. Have you solved it?

@stoneeve415 No.... /(ㄒoㄒ)/~~

@Jason-xin @stoneeve415 I want to know how to use the test_net.py too. The results I got is clearly wrong.
2019-03-21 17:19:01,079 maskrcnn_benchmark.inference INFO: OrderedDict([('bbox', OrderedDict([('AP', 9.300105363128172e-09), ('AP50', 6.022500060225e-08), ('AP75', 0.0), ('APs', 9.522348740812624e-09), ('APm', 0.0), ('APl', 0.0)])), ('segm', OrderedDict([('AP', 1.4857428119762759e-08), ('AP50', 5.942971247905103e-08), ('AP75', 0.0), ('APs', 1.519675543192826e-08), ('APm', 0.0), ('APl', 0.0)]))])

I have put the model after training in the corresponding link which can be found in README. For testing one image, you can follow the original maskrcnn-benchmark https://github.com/facebookresearch/maskrcnn-benchmark/tree/master/demo

I have put the model after training in the corresponding link which can be found in README. For testing one image, you can follow the original maskrcnn-benchmark https://github.com/facebookresearch/maskrcnn-benchmark/tree/master/demo

thank you! i got it

hello,Can you tell me how to do it? @stoneeve415

@zhuliqian https://github.com/XiaoLaoDi/maskscoring_rcnn here I provide the demo.py and pretrained models, hoping can help you.


[MNT] Review all_extras for soft dependencies.

Describe the feature or idea you want to propose
I suspect we dont need all of these.

mne is a tool for modelling neuro data, see #1108, #1110
h5py was in old benchmarking, can I think go
Is scikit_posthocs used anywhere?
tensorflow-probability, see #1077
xarray and dask are only used in the datatypes to convert back and forward. Cant see them actually used by an estimator

there are several other single use softdeps,
all_extras = [
"cloudpickle",
"dask",
"esig>=0.9.7,<<IP_ADDRESS>; python_version < '3.11'",
"filterpy>=1.4.5",
"h5py",
"hmmlearn>=0.2.7",
"gluonts>=0.12.4",
"keras-self-attention",
"kotsu>=0.3.1",
"matplotlib>=3.3.2",
"mne",
"pmdarima>=1.8.0",
"prophet>=1.1.0",
"scikit_posthocs>=0.6.5",
"seaborn>=0.11.0",
"statsforecast>=0.5.2",
"plotly-resampler>=0.9.0", # from statsforecast, needed for pandas2
"statsmodels>=0.12.1",
"stumpy>=1.5.1",
"tbats>=1.1.0",
"tensorflow<2.16.0",
"tensorflow-probability",
"tsfresh>=0.20.0",
"tslearn>=0.5.2",
"xarray",
"mlflow<2.4.0", # see https://github.com/mlflow/mlflow/issues/8629
"imbalanced-learn",
]
dl = [
"keras-self-attention",
"tensorflow<2.16.0",
"tensorflow-probability",
]
Describe your proposed solution
remove unnecessary soft dependencies, reduce code bloat
Describe alternatives you've considered, if relevant
leave it all in there
Additional context
No response

closing this as all_extras has moved on, still dont think we need "h5py". Will revisit after forecasting goes


Updated ARCore to 1.12.0

This change:

Updates ARCore bindings to version 1.12.0.
Fixed ARCore NuGet warnings: NU5125, NU5128

Fixed NU5125 by including the license file in the nupkg.
Fixed NU5128 by adding a dependency group for MonoAndroid7.0 to the nuspec.

"Fixed" the cake build for ARCore

I made the cake build work similar to the SceneForm project, which appears to be a more modern formula and happens to also work on Windows (the existing formula did not). This makes the instructions the same on both macOS and Windows.

Updated the ARCore README.md

This change updates the README with updated instructions on how to build the ARCore component and samples.

I have run the HelloAR sample app and verified that it works correctly.
I realize that ARCore 1.13.0 is out and will follow-up with that change as well, but I think it's valuable to have packages for 1.11.0 as well.
Since the SceneForm projects depend on the ARCore NuGet package, i'll have to follow up with updates to the SceneForm bindings separately once the ARCore 1.12.0 NuGet package is available.

@Redth


Add multithreading to the ceph medic for faster outputs

/assign @andrewschoen can you please review this PR


[FLINK-19649][sql-parser] Fix unparse method of sql create table. Keep syntax the same for tables without columns

In Flink's module flink-sql-parser, the sqlCreateTable.class unparse method the create table statement does not match the new rules of the print connector.
The print connector supports this:
CREATE TABLE table_print WITH ('connector' = 'print') LIKE table (EXCLUDING ALL)
When a print table is created without any columns, the parentheses must be omitted.
However, the unparse generated statements in SqlCreateTable contain parentheses. This will affect the SqlCreateTable.toString() ,SqlCreateTable.toSqlString() methods.
 CREATE TABLE table_print() WITH ('connector' = 'print') LIKE table (EXCLUDING ALL)
Might need to add the following to the unparse method for columnList.size > 0.

Please add some unit tests.

Please add some unit tests.

Okay, I'll add some unit tests.

Please add some unit tests.

Okay, I'll add some unit tests.

Hi, Kurt. I've added some unit tests.


Greenkeeper/sequelize 4.13.15

Coverage remained the same at 98.034% when pulling 411bb30b4b783f8543964ac78d1e1d6ad28b7569 on greenkeeper/sequelize-4.13.15 into af51cb5921a20ab1b36a9d27da17dde80ae9bc31 on master.


glue together firm module and binary generation CLI

work started by @reiner-dolp , I added changes to have a statically linked mj-rt lib on top.
reiner says some patches from @flip1995  are still missing?
EDIT
Oh, apparently @reiner-dolp  rebases this one on #134 .

bors try

bors r+

the only thing missing from #134 is commit 959ffdbfa. Will add this in a later commit since it conflicts heavly with what we have implemented in this PR.

I 🍒 ed that one!

oh, thanks!

ah, but it didn't work. https://github.com/comprakt/comprakt/pull/137/files#diff-3fccd914d4013b283c3a1df6d630e173R64 should be something collision free ('.main')

@problame mjrt-rs fails to compile in release mode:
thread 'cli_ast_0_empty_class_mj' panicked at 'called `Result::unwrap()` on an `Err` value: CargoError { cause: Some(CargoError { kind: CommandFailed, context: Some("   Compiling mjrt-src v0.1.0 (/home/travis/build/comprakt/comprakt/mjrt-src)\n   Compiling mjrt-bin v0.1.0 (/home/travis/build/comprakt/comprakt/mjrt-bin)\nerror: failed to run custom build command for `mjrt-bin v0.1.0 (/home/travis/build/comprakt/comprakt/mjrt-bin)`\nprocess didn\'t exit successfully: `/home/travis/build/comprakt/comprakt/target/release/build/mjrt-bin-25aded626b1728f1/build-script-build` (exit code: 101)\n--- stderr\nthread \'main\' panicked at \'libmjrt_src-*.a not exactly one match: []\', mjrt-bin/build.rs:32:9\nnote: Run with `RUST_BACKTRACE=1` for a backtrace.\n\nwarning: build failed, waiting for other jobs to finish...\nerror: build failed\n"), cause: None }) }', libcore/result.rs:1009:5

so what broke the tests, exactly?

bors try

retrying with warm cache:
bors try

Looks like the hack described in https://github.com/comprakt/comprakt/pull/137/files#diff-58ceb2fdbcf4280270d9729caaef9a85R4 doesn't work after all. I'll fix that.

I verified that https://github.com/comprakt/comprakt/pull/137/commits/14f5b2082fccb2aae3c4295782f497e327028ab0 does not depend on what we have in the build cache by building it in a clean /tmp checkout.
@reiner-dolp  feel free to merge if the tests pass!

bors try

attempting merge with cold/emptied cache.
bors r+


Address bar can become unresponsive

Operation System: n/a
Beaker Version: 0.6.0
Minor UX glitch:

Start SAFE Browser
Login to account
While login is proceeding, type a SAFE URL without the safe:// prefix, hit return before the tab has finished the login process. Call me impatient, go on :wink:
This URL never loads, user can click, hit return etc but as it lacks the safe:// prefix the browser does nothing.
as soon as you edit what's in the address bar the problem goes away and the prefix is applied

This seems to happen because if the user types into the address bar on a clean tab, the safe:// prefix is automatically even though it is not shown in the address bar until you hit return, but that this fails to happen if the tab is busy.

@theWebalyst this is most likely due to the way we're handling safe interactions at the moment (as it's occurring as you login).
Current interactions occur in the main electron thread, which we're discovering is bad practice. It causes a lot of hangs etc. We're aware of this and have some ideas to tackle which we'll be implementing soon.
So I'd hope that would address this. I'm going to leave this open to track it once we've got that nailed down.

Verify if this is also an issue in Peruse v0.6.0, otherwise let's close it as we won't be maintaining beaker.

No longer an issue following the steps to reproduce with peruse branch of this repo.


fix(peer): creating PeersState so there will be no deadlocks on peer_…

…mutex
Purpose
So this should resolve all issues with deadlocks on peers_mutex_. Main issue was that we locked for reading peers_mutex_ and then called sealAndSend that is also trying to lock for reading that mutex. This should work, but in case another thread try to lock for wrting peers_mutex_ in between those two shared_locks we end up with deadlock as unique_lock has priority and when anything try's lock after unique_lock it needs to wait.
So basic concept what was happening:
Execution                                T1                          T2
    |                                    |                            |
    |             std::shared_lock lock1(peers_mutex_)                |
    |                                    |              std::unique_lock lock2(peers_mutex_)   <--waitng for lock1
    |             std::shared_lock lock3(peers_mutex_) <--waitng for lock2
    V                                    -                            -

resloves https://github.com/Taraxa-project/taraxa-node/issues/1029

prnet cleaned-up successfully


use unique id also in wrapper runner

forgot the wrapper runner when replacing incrementing integers with unique_ids

@chbiel in case you didn't notice: the tests are failing

Yeah I saw. I will have time at the weekend to fix this
Am 11.02.2015 23:38 schrieb "Julian Seeger"<EMAIL_ADDRESS>
@chbiel https://github.com/chbiel in case you didn't notice: the tests
are failing
—
Reply to this email directly or view it on GitHub
https://github.com/brianium/paratest/pull/152#issuecomment-73980367.

@julianseeger please delete this pull request due to https://github.com/brianium/paratest/pull/156 resulting from the discussion in https://github.com/brianium/paratest/commit/ec7216b9cdbefa0d6c358913152d33117f01c23e

There is no delete and so it will exist forever in the endless space of www ;)
But on the buttom, there is a "Close" button


Are there any plans to release paintkit as an npm package?

Hi @seanchas116
I'm very impressed with your excellent work!
I'm working on a graphic editor with a PDF library called pdfme.
And I was hoping to have exactly this kind of UI component...!
because creating such a UI component is a bit difficult for me...
Do you have plans to release paintkit as an npm package?
If so, it would be great if you could share something like a roadmap.
I'd like to contribute if there is something I can help with. (testing, writing docs, etc.)
Thanks!! You are great!!✌

I haven't released paintkit as a "ready-to-use" component library because:

It has many dependencies (styled-component for every components, MobX for some components etc)
It has utility functions/classes that are not relevant to UI components
I have no time to write docs
I'm thinking of migrating it to Web Components (so it can be used in Macaron documents)

I'd like to release it as a npm package when ↑ are resolved!

Thanks for replying.
I see... I understand.
so, I'll close this issue.


django-simple-history plugin doesn't work correctly with django 1.10 admin UI

When django-simple-history was added to replace default admin UI in django 1.10 the following exception was raised: Exception Value: render() got an unexpected keyword argument 'dictionary'
Please use the following link for details: http://stackoverflow.com/questions/39179559/django-simple-history-plugin-render-got-an-unexpected-keyword-argument-dictihttp://stackoverflow.com/questions/39179559/django-simple-history-plugin-render-got-an-unexpected-keyword-argument-dicti

I've corrected this issue by replacing 2 calls of render function in simple-history.admin module:
1:
78        return render(request, template_name=self.object_history_template,
79                      context=context)

```2:

181        return render(request, template_name=self.object_history_form_template,
182                      context=context)

django-simple-history isn't compatible with 1.10 yet. I have a branch in progress, but work isn't complete.

What other incompatibilities with Django 1.10 are there, besides the admin integration and the middleware settings?

There's also a change in the way management commands are defined, we may be dropping 1.6 support due to that. But it's not too bad over-all, I just haven't had time to work on it yet.
Here is what I have in progress right now: https://github.com/treyhunner/django-simple-history/compare/django110

Hey @macro1 is there anything I can do to help you with this? What remains to be done? I can jum on finishing it up if you want (and if I'm able...)

Any update on this issue? I really need it to be compatible with 1.10...

I took @macro1's django110 branch and made a slight change to middleware.py, available here. It works for 1.10 and the current dev branch of Django, at least, but may not be backward (or forward) compatible from there.

I'm still new to python django. @nessalc, could you please elaborate how to install your fix? I'll need it to be working on heroku too.

@achillis1: If you run the following:
git clone -b django110 https://github.com/nessalc/django-simple-history.git
pip install django-simple-history/

it should work. I just needed a quick-and-dirty fix—no idea if it is compatible with heroku. I'm hoping @macro1 can wrap this up, because I'd prefer something that's less slap-dash than my approach.

Hi guys, thanks for this nice module that I'd like to use for a project based on Django 1.10. Do you have an estimation about when simple-history will be fully compatible with 1.10? Thanks!

@nessalc How has your fix worked out? Have you encountered any issues in practice?

@dalberto My fix has been functioning admirably in my particular application, and no, I haven't encountered any issues thus far. I don't claim to have tested it extensively--there are likely cases/situations that my fix doesn't work for--but at least for "simple" history, it's doing the job.

File "/home/trusty/django/local/lib/python2.7/site-packages/simple_history/models.py", line 9, in
from django.db.models import loading
ImportError: cannot import name loading
Any help please?

1.10 support is now in master. I need to update the documentation, but a release will be coming soon. Thanks for your interest and patience everyone!

Thanks @macro1 for all your great work. 👍


FastForward breaks National Australia Bank banking website

Body
When trying to access https://ib.nab.com.au/nabib/index.jsp?browser=correct nothing loads except some text with this addon enabled and without it disabled the website loads properly.
Version
0.0.0
What browsers are you seeing the problem on?
Firefox
What OS are you seeing the problem on?
Windows
Anything else?
I saw there was a whitelist feature being developed so maybe before that you could tell me how to whitelist nab so it works with this addon.

Seems to work, please test again.

Closing this as OP has not responded in a while; if you can find evidence to reproduce this error, please make another issue (disable all other extensions).


Need to decide what asserts to use for boolean rules

We have a lot of rules with double assertion: first with typeof, and the second with === true.
As mentioned in many PRs it's overkill to use typeof x === 'boolean' and x === true.
The last PRs with double check:
https://github.com/jscs-dev/node-jscs/pull/1023/files#diff-3c958cdc2aa18d8507669afbc4372ee5R76
https://github.com/jscs-dev/node-jscs/pull/1028/files#diff-eb24b2408e3575c79b3b3bd2a596f8aaR40
https://github.com/jscs-dev/node-jscs/pull/1059/files#diff-572c6f5a08392c7fff2ce006ebf948ebR34
etc.
I propose to drop typeof === 'boolean' asserts in all rules.
What you think?
/cc @markelog @mikesherov @mdevils @mrjoelkemp

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/8575922-need-to-decide-what-asserts-to-use-for-boolean-rules?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Yes, do it. However, we should be working on the declarative options feature anyway so this becomes an unnecessary discussion.

Oh I guess https://github.com/jscs-dev/node-jscs/commit/9f09ee0b5389e2f4db4fe83175653b8e9f5306cf closes this? Didn't see this before.

Yup. Thanks!


Infinite Loading spinner on Login screen

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open App on Mobile E.cash.
Enter username, press continue.
Enter the password, press continue.

Expected Result:
Users should sign in to the app.
Actual Result:
Loader is shown the password page and it hangs there forever.
Workaround:
Close the app and restart.
Platform:
Where is this issue occurring?

[x] Web - Possible
[x] iOS - Possible
[x] Android - Found here
[x] Desktop App - Possible
[x] Mobile Web - Possible

Version Number: latest
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @greg-schroeder (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

This is a serious issue that is aware of for a month but It just occurred again to me and thus it will come for many users. Thus need resolution.
Explanation
During app uses, we run many uncontrolled apis calls which require Auth token. When a user logs out, it is possible that many of these requests are still running and due to this reason, our queue will be paused.
But as we are repeating the queue, it will not resume until we get the token back which causes this issue.
To permanently fix this, we have to flush the queue when the user is logged out.

Just faced the same issue on Desktop App.

Triggered auto assignment to @MitchExpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Triggered auto assignment to @AndrewGable (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

I have personally experienced this issue, can confirm it's happening, so skipping Triage.

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Adding deploy blocker as this is only occurring on staging. I can reproduce on a fresh install as well.

To fix this. the best option is to flush the queue when the user logs out. As there won't be any call that uses auth token and should continue after logout.

Do we flush the queue here? https://github.com/Expensify/App/blob/main/src/libs/actions/SignInRedirect.js#L52

No. Here https://github.com/Expensify/App/blob/e511b1f70082ad965b5dc85117d18f4dce54287d/src/libs/API.js#L62.
But this is not really clearing the queue which some requests are made after this statement has run. As per the image attached above, there was a request for Timer.

@AndrewGable Ah, I think I found it. Maybe we should unpause the queue when we authenticate. Which does not have any impact if the queue was already unpaused.
We do this for reauthenticate https://github.com/Expensify/App/blob/e511b1f70082ad965b5dc85117d18f4dce54287d/src/libs/API.js#L278
We can follow the same for https://github.com/Expensify/App/blob/e511b1f70082ad965b5dc85117d18f4dce54287d/src/libs/actions/Session.js#L207-L225.

Maybe if I am correct, this can be triaged and assigned to me.

Plus, We should also flush the offline queues when we flush the in-memory queue. I will try to experiment with this a bit.
 Onyx.set(ONYXKEYS.NETWORK_REQUEST_QUEUE, []);

Triggered auto assignment to @jboniface (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@parasharrajat as mentioned in slack, the issue is yours. We will get it configured in Upwork.

posted to upwork and sent invite to rajat https://www.upwork.com/jobs/~01fede9fe7086b3d75

@AndrewGable @jboniface  Be sure to fill out the Contact List!


Support adding release branch changes to master's changelog

Hi!
For the https://github.com/mozilla-neutrino/neutrino-dev repository, after we released v8.2.0, we switched to treating master as the next breaking release (v9; as yet unreleased), and have since released a v8.2.1 and v8.2.2 from a separate release/v8 branch.
auto-changelog successfully updated CHANGELOG.md in that branch (run via a package.json version entry), however I've tried to find a way to update master's changelog (since that's what people will look at when visiting the repository on GitHub), and can't find a way to do so.
It seems like as-is, when we release v9 the master changelog and will end up skipping straight from v8.2.0 to v9.x.x.
Is this something that's possible to achieve at the moment? If not, what changes do you think might be required to do so?
Many thanks :-)
cc @eliperelman

As it stands, auto-changelog will just use the currently checked out branch to generate the changelog for that branch. I suppose we could add a --branch option to specify a branch to generate a log for? Then with master checked out you could auto-changelog --branch release/v8
What is your plan when releasing v9? Merge in release/v8 and bring 8.2.1, 8.2.2, etc across to master?

What is your plan when releasing v9? Merge in release/v8 and bring 8.2.1, 8.2.2, etc across to master?

Ah that's the catch, we likely won't be merging it in, since it only has cherrypicked fixed from master, and master doesn't need anything from it.

So the real solution for what you need would be to include commits from a certain branch as well as the current one? Which would capture any extra releases from the included branch... although I'm not sure how well the two sets of commits would consolidate into one log? And then what happens when that branch is deleted?

I have finished but not yet pushed an --include-branch option that I think solves your use case. Once the tests and stuff are sorted I'll push it up. Sorry for the delay!

@edmorley I've added my attempt at an --include-branch option to 1.6.0. Would be great to see if it's at all useful for your use case (or if it even works properly) as I don't have any decent repos to test it with.

Sorry for the delayed reply -- I've finally had a chance to try this out (mozilla-neutrino/neutrino-dev#970), and it works great. Many thanks!


Bug: Context menu clipped by the edge of window

Description
Context menu flyouts could be clip by the edge of the window, when they are placed at the top/bottom of the window, or simply be taller than the window.

Steps To Reproduce
Resize the window and right tap anywhere.
Requirements
Fix the bug by limiting the height of the menu or any other ways.
Files Version
<IP_ADDRESS>
Windows Version
Windows11 22H2 22621.1265
Log file
debug.log

This is a WinAppSdk limitation

But Gallery's flyouts would collapse automatically, which looks better I thought?

Which property controls that? It might not be available for the command bar flyout.

Which property controls that? It might not be available for the command bar flyout.

Seems you are right. And the issue is caused by CommandBarFlyout.Placement. when it is set to Full, the CommandBarFlyout would exceed the lower boundary of the window

And whatever the value is, it might exceed the left or right boundary. We may have to set Placement to Left or Right manually to fix this bug.

I too am having a similar problem. For example, when I want to remove the "Tags" category the drop-down menu shows only part

It is now fixed by WASDK 1.4.

@hishitetsu  Thanks! Can you link if you are convenient?


feat: landing page (code)

Need to create the landing page using HTML and CSS (don't use any CSS libraries or frameworks)
Design: https://imgur.com/a/AFQR3HL

I would like to work on this issue under JWOC

I want to work on this issue under JWOC

Assigned you @AtriSukul1508
I will create a few new issues, please have a look at them @tanya-gupta25


Adding Any Type, Encode, and Decode impl for chrono time types

Hello,
This PR would add  impl of AnyEncode, AnyDecode, and Typecrate::any::Any for chrono time types when all the SQL types included in compilation features support these types.

Works for me, thanks. We are planning a large rewrite of how Any works but this is a nice bump in the meantime.


Implement graph tooltip.

Added an option to show sub graph as tooltip. Toggleable from settings.

The comment bubble can not scale when the event graph scales, could we fix that or provide a PR to the engine or just use the "comment box (hold C + left click)"


DetourUpdateThread with non-pseudo current thread handle will freeze the thread.

When running the following code, detours will suspend the current thread, with no way of resuming it, possibly resulting in a process deadlock if this is done in DllMain.
HANDLE hThread = OpenThread(THREAD_SUSPEND_RESUME, false, GetCurrentThreadId());
DetoursUpdateThread(hThread);
// no code will execute after this point.

This is caused by https://github.com/microsoft/Detours/blob/master/src/detours.cpp#L1854 only checking against the pseudo-handle instead of also checking if the TID of the handle matches the current thread's.
This if check can be safely replaced by GetThreadId(hThread) == GetCurrentThreadId() to handle both real and pseudo handles to the current thread (although there might be a slight performance gain due to less API calls by still explicitly checking if it's a pseudo handle)

Yes, that's a problem

after official merge my PR#127, DetourUpdateAllOtherThreads will instead of the old DetourUpdateThread  API,  and it will exclude currrent thread automaticall. and DetourUpdateAllOtherThreads is a safer solutuon for HOOK operation.


Error verifying module

Hello!
I followed your Readme, but when I try to start the master with the hook module TestHook, the execution fails with this message:
Error loading modules: Error verifying module 'org_apache_mesos_TestHook': Unknown module kind: Hook
*** Error in `/home/my_home/mesos-0.24.0/build/src/.libs/lt-mesos-master': double free or corruption (fasttop): 0x0000000001fa9950 ***
Aborted (core dumped)
Can you please help me understand where I am wrong? It seems that it doesn't recognize the Hook module kind!
Thanks

Could you provide us the commandline that you used to launch the master?
Also, if you don't mind, can you share the bits related to the module
declaration part in you module sources (the stuff starting with
"mesos::modules::Module ....")?
That will help us get started on debugging this issue.
Best,
Kapil
On Sun, Oct 25, 2015 at 7:00 AM, alepatacca<EMAIL_ADDRESS>wrote:

Hello!
I followed your Readme, but when I try to start the master with the hook
module TestHook, the execution fails with this message:
Error loading modules: Error verifying module 'org_apache_mesos_TestHook':
Unknown module kind: Hook
*** Error in `/home/my_home/mesos-0.24.0/build/src/.libs/lt-mesos-master':
double free or corruption (fasttop): 0x0000000001fa9950 ***
Aborted (core dumped)
Can you please help me understand where I am wrong? It seems that it
doesn't recognize the Hook module kind!
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/mesos/modules/issues/8.

Hi!
I'm launching the master with this command:
./bin/mesos-master.sh --ip=<IP_ADDRESS> --hostname=<IP_ADDRESS> --work_dir=/var/lib/mesos --quorum=1 --modules="file:///home/my_home/modules/hook/modules.json.in" --hooks=TestHook
The folder /modules is the one i cloned from github, and I left the source files unchanged, so the declaration of the hook module is the following:
mesos::modules::Module<Hook> org_apache_mesos_TestHook(
    MESOS_MODULE_API_VERSION,
    MESOS_VERSION,
    "Apache Mesos",
<EMAIL_ADDRESS>    "Test Hook module.",
    NULL,
    createHook);

The only thing i changed is the path of the shared library in modules.json.in:
{
  "libraries": [
    {
      "file": "/home/my_home/modules/build/.libs/libtesthook.so",
      "modules": [
        {
          "name": "org_apache_mesos_TestHook"
        }
      ]
    }
  ]
}

Instead of --hooks=TestHook, you need --hooks=org_apache_mesos_TestHook. Looks like we need to fix the error message as well.

Nope, I'm receiving the same error!

This doesn't make sense. For sanity check, can you replace mesos::modules::Module<Hook> with mesos::modules::Module<Isolator> and try again?

I replaced it and make fails with this error (and many others related):
../hook/test_hook_module.cpp:158:24: error: 'Isolator' was not declared in this scope
 mesos::modules::Module<Isolator> org_apache_mesos_TestHook(

Aah, you need #include <mesos/module/isolator.hpp> as well.
On Tue, Oct 27, 2015 at 5:03 AM, Alessandro Patacca <
<EMAIL_ADDRESS>wrote:

I replaced it and make fails with this error (and many others related):
../hook/test_hook_module.cpp:158:24: error: 'Isolator' was not declared in this scope
mesos::modules::Module org_apache_mesos_TestHook(
—
Reply to this email directly or view it on GitHub
https://github.com/mesos/modules/issues/8#issuecomment-151419804.

Still not working. Same error.


Update pytorch dependecies

fix #286

Test fail looks unrelated. May you restart this tests?

Test fail looks unrelated. May you restart this tests?

Yes, this is a known issue for the tensorflow 1 legacy build. We can ignore it.
@FynnBe could you please review this?

yes, we need to correct the dependencies for pypi. In order not to break the conda package we need to update the conda meta.yaml as well though...


NXP Link based on LPC11U35 or LPC4322

The link to NXP website mentioned the NXP LPC LINK and LINK2, but I found that they are based on LPC3154 and LPC4370, why we write LPC11U35 and LPC4322 here?

Because DAPLink is designed to run on boards with an on-board debug probe using LPC11U35 and LPC4322, not the LPC-LinkI/II boards. LPC-LinkI/II has its own firmware. (Though DAPLink could be ported rather easily.)

Closing this issue. @tz-arm please reopen if you have more questions.

Sorry to jump into that but I really think that any mention to  the LPC-LinkII board should be removed from this site - This is very misleading  - I thought the LPC-LinkII was officially supported by the DAPLink team (which is apparently wrong) and EmbeddedArtists/NXP claims support for DAPLink without mentioning that their software is actually custom, not available on their web site  and locked to debug NXP devices only.  (something that Embedded Artists does not mentioned on their website).  Now, I need to port  DAPLink to this board which means having access to a KEIL license....:-(

Sorry to jump into that but I really think that any mention to the LPC-LinkII board should be removed from this site

@rbarzic Can you add here references here? We shall review them.

If you are in a bind, and can't wait for a preferable solution.
Mbed Studio Comes with ARMC6.
Daplink doesn't use Mbed OS so building it with Mbed Studio's AC6 might not be within the license.


Bug Report: Random right arrow that doesn't show up in chart

Hello, a group of my friends and I are working on a mod, but just when I finished my chart for a song I encountered an issue I can't seem to get rid of. I'm using v1.4.2. The problem is, the first few sections are empty in the chart, yet there's a right arrow that the enemy has show up. It isn't anywhere in the chart, and I even tracked down where exactly on the grid it thinks there's a right arrow and I placed one there, saved, removed it, and saved again, but it still won't go away. I'm not good at inspecting .json files, but my friend who does know what he's doing had a look and says it doesn't even show up in the .json file but it's still there in game. How do I get rid of it? Thanks :)
Here's a link to a video of the problem:
https://www.youtube.com/watch?v=JyE6n6x70dk

have you tried clearing that section? also probably a Discussion

have you tried clearing that section? also probably a Discussion

Just tried clearing the section, it's still there :/

(not converting to discussion since this might actually be a bug with kade engine @prokube)

I'm pretty sure this isn't a bug, I believe that some charts have hardcoded notes. (but I can't back this up I've just noticed that some charts have notes that show up and you can't remove them) I'll have to look it later

@Viath1 does this happen if you move the chart to a different song that isn't bopeebo?

Can confirm, this also happens for me with bopeebo.


parsedmarc: init at 6.10.0

Motivation for this change
Parsedmarc is not yet available in nixpkgs.
I plan to provide a service module soon.
Things done

Added missing python dependencies
Added parsedmarc

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

There is some activity on #96533. Basically the lack of a new enough elastic held me back to push forward on this. I intend to pick it up again soon-ish as I have a renewed need for parsedmarc.

Made obsolete by #125404 :)


Number of trainable params in network

Hi,
print(net) is giving the overview of the network, but doesnt give me the number of params that are being updated in each layer. for example, keras net.summary() gives the overview of shapes and number of params in each layer that could be trained/updated.
is there a way to figure this out from gluon API ?

Gluon now supports a summary view too. https://mxnet.incubator.apache.org/versions/master/api/python/gluon/gluon.html?highlight=summary#mxnet.gluon.Block.summary
The input to the method is the typical input such as a batch.

this does require an input i wud assume ?
Since we wud be doing .hybridize(), is there a simpler way for us to view the summary from the computed static graph?

yes, it requires an input. in order to get intermediate sizes, an input would always be needed even if it's from a static graph since gluon allows partial shape and deferred initialization.

Closing now. Let me know if you have more questions along the line.


⚠ MAS Portal 4 - /api/chart has degraded performance

In 27f54f3, MAS Portal 4 - /api/chart (https://mas-portal-api.yodo1.com/api/chart) experienced degraded performance:

HTTP code: 200
Response time: 21771 ms

Resolved: MAS Portal 4 - /api/chart performance has improved in 9619a9a.


Adding linting to Chart and values yaml files

This does not lint yaml templates as they are templates rather
than valid yaml files. They cannot be linted with a normal linter
yamllint is used for linting. This is an existing Python project
https://github.com/adrienverge/yamllint
The rules are not the default rules and are stored in their
entirity so they can be controlled over time
The existance of a Chart.yaml file and values.yaml file is checked
and an error is thrown if one is missing. Helm lint will not
detect a chart if Chart.yaml is missing and if a values.yaml file
is missing it is noted as info.
The run function is introduced to enable running all the linters,
capturing non-zero exit codes, and exiting with a non-zdero code
if any of them fail. This is used instead of exiting when the
first failure happens to provide more feedback to chart developers.
/cc @prydonius @viglesiasce @unguiculus
You can see an example of a failure at https://circleci.com/gh/mattfarina/charts/41
There are docs on configuring yamllint including the rules. I tried to tailor our rules to the existing charts. Though, many of the charts fail the rules and are inconsistent.
You can take a look at all the lint failures we have in the existing stable and incubator charts in this gist based on the rules in the config file here.
Once we agree on a set of rules I can create separate PRs to clean up the existing charts.
References #2373

@unguiculus Sorry about that. Thanks for catching those. I'm too used to my editor fixing issues like these automatically. I see I don't have that setup for shell scripts.
I've updated with a fix for the formatting.

/lgtm


Can't add app service metric chart to the workbook

Hi team,
I found it's not able to add app service's metric chart to the workbook. Looks like only AI supported to do that.
Is this by design limitation?

@xuegao112 ,  could you please share the document you were referring to here for further insights?

@xuegao112 , Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Docker Daemon Config tests on CoreOS when using early-docker

I'm running Kubernetes on CoreOS and using early-docker to start flannel (network overlay for k8s) before starting docker.
The 'Docker Daemon Config' test searches for the oldest running docker process (https://github.com/docker/docker-bench-security/blob/master/helper_lib.sh#L44) and picks up early-docker instead of docker and ends up running config tests on it. Any reason why we look for oldest running docker process instead of newest?

I have a workaround for this by replace this line with:
for PID in $(pgrep -x -o "$PROC")
where $PROC = "docker daemon" instead of just docker

I have a fix for this by replace line with:
for PID in $(pgrep -f -n "$PROC")
where $PROC is "docker daemon" instead of "docker"

Hi @dhawal55, and thanks for reporting this and creating a PR, but it seems to be duplicate of #124 and #127.

Oh ya, thanks for pointing it out. I love that i ended up with the same solution as you did. Any reason why the fix is not merged yet?

Well, I didn't get any response (https://github.com/docker/docker-bench-security/issues/124#issuecomment-197487316) and it hasn't been brought up since.
I'll go with your https://github.com/docker/docker-bench-security/pull/153, since there isn't any conflicts.

#153 merged. Closing.
Thanks @dhawal55 for your contribution.


error with flax < 0.8

Just a heads up, with jax==0.4.26 and flax==0.7.* I was getting errors when importing sax installed recently with pip install --upgrade sax
>>> import sax
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/opt/homebrew/lib/python3.11/site-packages/sax/__init__.py", line 15, in <module>
    from flax.core.frozen_dict import FrozenDict as FrozenDict
  File "/opt/homebrew/lib/python3.11/site-packages/flax/__init__.py", line 19, in <module>
    from .configurations import (
  File "/opt/homebrew/lib/python3.11/site-packages/flax/configurations.py", line 93, in <module>
    flax_filter_frames = define_bool_state(
                         ^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/flax/configurations.py", line 42, in define_bool_state
    return jax_config.define_bool_state('flax_' + name, default, help)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'Config' object has no attribute 'define_bool_state'

It fixed when pip install --upgrade flax installed 0.8.*.
Not sure if this is something you might need to know for setting requirements but thought I'd let you know / create a paper trail in case others see this, feel free to close

im having the same issue
@flaport


Interface call signature

In Typescript, we can have callable interface signatures with additional properties, such as
interface Callable {
  (): void;
  someProperty: string;
}

Closure Compiler does not yet support this, but there is an open issue addressing it. Currently, there is only one proposal, suggesting this syntax (source):
/** @interface @extends ICallable<function(?):?> */
ko.Observable = function() {};

This solution would be restricted to work under Closure Compiler and not in JsDoc in general (see Special Types in the Closure Type System to understand why).
In VSCode / Typescript, intersection types are properly parsed, so this is possible:
/** @typedef {function(): void} CallSignature  */
/** @typedef {{ someProperty: string } & CallSignature} Callable */

But firstly, intersection types will probably not be supported by JsDoc in near future, and secondly, they are different from a type system perspective.
I could not find an open issue addressing this, has there been a decision if this should be supported? If so, this could be the thread to collect syntax suggestions. Personally, I would suggest either putting it directly into the interface definition:
/** @interface @returns {void} */
function Callable() { }

But I think this syntax is confusing, especially here using a parameterless, void signature.
Another possibility is to override the prototype as a function, which I personally think would make more sense:
/** @interface */
function Callable() { }
/** @returns {void} */
Callable.prototype = function() { }
/** @type {string */
Callable.prototype.someProperty;

For compatibility with comment-type interfaces, perhaps just
/**
 * @interface Callable
 */
/**
 * @function
 * @name Callable // or Callable#
 */

Or possibly:
/**
 * @function
 * @interface Callable
 */

Well, crud.  I was hoping to come here and find a solution, but it looks like we don't have one.  I'm trying to document a function that has additional known properties.  I thought the simplest way might be to have a class merged with a callable interface, as in this SO answer, but if I can't actually describe that in JSDoc, I may need another approach.


updated to elm 0.16

Hi, I updated the code to 0.16 and tested basic functionality.
Can you please publish this as an Elm package? Thanks.

Hi badri,
Thank you for the pull request, and interest in this project. I stopped development a few month ago after running into problems with how Elm handles key events from the dom. Because of these problems I found it very hard to accurately detect combos.
Perhaps the situation has changed since then. I'll look back into it and document the problem with more detail.


feat(tags)

PR for this feature https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/issues/552
All states and use-cases are implemented.
Review and discussion open ;)

@julien-deramond Just pushed a new commit on my feat-tags branch, extending badge component.

Just got some information from the designer. A tag is always clickable - always linked to an action; we can't use a tag to just display an information. So @vogloblinsky, nothing more to do concerning non-actionable tags.
I still need to check your latest commit during the week and ask for a design review.

@julien-deramond Rebase done, and others remarks too.

@julien-deramond Rebase done, and others remarks too.

Don't know why exactly but there is an issue with the PR. Indeed you have 48 commits including commits not coming from you.
Try to:

Update the main branch of you fork
Squash all your commits into 1
Rebase to the main branch

If it doesn't work, I think you have the correct rights to not to use a fork. You could recreate your branch with only the content needed and open a new PR.


[BEAM-1585] Add ability to attach a bootstrap script to the pipeline

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @aaltay PTAL

Coverage increased (+0.01%) to 70.679% when pulling 2e0fc969a7f9bc89972f484bc4878e673777eea5 on sb2nov:BEAM-bootstrap into 2df9dbd24f6fd3bcfb14316859f4b38c38d95eca on apache:master.

Retest this please

Coverage decreased (-0.05%) to 70.614% when pulling 2e0fc969a7f9bc89972f484bc4878e673777eea5 on sb2nov:BEAM-bootstrap into 2df9dbd24f6fd3bcfb14316859f4b38c38d95eca on apache:master.


Fix typo (my -> by)

Docs Build status updates of commit 24f2923:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/errors-and-warnings/NU1605.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Model what's given to GENMODEL

This is needed to make this all play nicely with the build tool.

Actually, hold off on this for now. I'm not crazy about the way it's turning out.


Line Templates from spells are too thin

When casting spells with an area of Line (such as Grim Tendrils and Lightning Bolt), the template placed from the chat card is only 1ft wide instead of 5ft wide. This means that when placing the template on non-45 degree increments, it doesn't highlight the squares properly.
Tested on both an active world with mods and a completely fresh world.

I also get this and there's no way to increase the width of the line in Details


Add configurations options for HTML5 Indexed Databases

Configure IndexedDB from plugin configuration file:

IndexedDB JS api availability
storage path
disk usage

Disable IndexedDB by default
Change default storage dir to g_get_user_cache_dir()/wpe/databases/indexeddb
(WebKit default was g_get_user_data_dir()/databases/indexeddb)

As this is more on the webkit behaviour, @modeveci could you please have a alook?

Got it! Thank you!


add objc initialization support for dynamic color

Platforms Impacted

[ ] iOS
[ ] visionOS
[x] macOS

Description of changes
I want to create customized color badge in objective C but I can't do that because I can't create new dynamic color in objective C. add objc support for Dynamiccolor initialization.
Binary change
(how is our binary size impacted -- see https://github.com/microsoft/fluentui-apple/wiki/Size-Comparison)
Verification
no impact on test app. Generated Fluentui-swift.h include
SWIFT_CLASS_NAMED("DynamicColor")
@interface MSFDynamicColor : NSObject
- (nonnull instancetype)initWithLight:(NSColor * _Nonnull)light dark:(NSColor * _Nonnull)dark OBJC_DESIGNATED_INITIALIZER;
@end

Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] iPad Pointer interaction
[ ] SwiftUI consumption (validation or new demo scenarios needed)
[x] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow

An aside, as of macOS 10.15, I think Appkit has an API to make dynamic colors directly https://developer.apple.com/documentation/appkit/nscolor/3294481-colorwithname

[heart]         Mike Schreiber (HE/HIM) reacted to your message:

From: Saad Najmi @.>
Sent: Monday, August 12, 2024 9:47:58 PM
To: microsoft/fluentui-apple @.>
Cc: Mike Schreiber (HE/HIM) @.>; Review requested @.>
Subject: Re: [microsoft/fluentui-apple] add objc initialization support for dynamic color (PR #2083)
An aside, as of macOS 10.15, Appkit has an API to make dynamic colors directly https://developer.apple.com/documentation/appkit/nscolor/3294481-init
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/fluentui-apple/pull/2083#issuecomment-2284961596, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABFUYP7DKTDZI7UENBIFQN3ZREUQ5AVCNFSM6AAAAABMM4D4YSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBUHE3DCNJZGY.
You are receiving this because your review was requested.Message ID: @.***>


Pass request into the group conditional

We'd like to be able to check the request itself inside our registered group, for example check the IP address is internal:
Flipper.register(:internal_admins) do |actor, context|
  admin = actor.respond_to?(:admin?) && actor.admin?
  internal_ips = %w[x.x.x.x x.x.x.x ::1 <IP_ADDRESS>]
  internal = internal_ips.include?(context.remote_ip) || Rails.env.development?
  internal && admin
end

However it doesn't seem like you can access the request from the context... or pass the request itself to the enabled?
We looked at using a custom helper e.g.
def internal_admin?(feature, actor)
  internal_ips = %w[x.x.x.x x.x.x.x ::1 <IP_ADDRESS>]
  internal = internal_ips.include?(request.remote_ip) || Rails.env.development?
  Flipper.enabled?(feature, actor) && internal
end

But that would allow someone to add a feature to another group and it be internal... which isn't what we want. And if we were to force the check for the admin? into the helper as well like so:
Flipper.enabled?(feature, actor) && actor.admin? && internal

It would feel a bit hacky... it would be awesome if we could avoid a custom helper and just use:
Flipper.enabled?(feature, actor)

And it automatically do all the necessary checks under the group.

One idea we had was to expect a hash in the group:
Flipper.register(:internal_admins) do |actor, context|
  return false unless actor.is_a?(Hash) # enforce hash for this group

  user = actor[:user]
  request = actor[:request]
  admin = user.respond_to?(:admin?) && user.admin?
  internal_ips = %w[x.x.x.x x.x.x.x ::1 <IP_ADDRESS>]
  internal = request.respond_to?(:remote_ip) && (internal_ips.include?(request.remote_ip) || Rails.env.development?)
  internal && admin
end

And then pass it like so to the enabled?:
Flipper.enabled?(:our_feature, { user: current_user, request: request })

Yep, CustomActor is the correct way to do this. I'll often create a FlipperSession or ActorWithRequest class like this in my apps. Nice job making it work for you!


Intellisense

Type: Bug
At some point, Intellisense suggestions are not displayed anymore
VS Code version: Code 1.77.3 (704ed70d4fd1c6bd6342c436f1ede30d1cff4710, 2023-04-12T09:16:02.548Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E3-1505M v5 @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.79GB (8.41GB free)

Process Argv
--crash-reporter-id c95ea6bc-917d-471a-928e-f452d74b2d72

Screen Reader
no

VM
0%

Extensions (35)

Extension
Author (truncated)
Version

gitignore
cod
0.9.0

doxdocgen
csc
1.4.0

latex-workshop
Jam
9.10.0

better-cpp-syntax
jef
1.17.2

dbc
lha
2.0.0

language-matlab
Mat
1.0.2

vscode-docker
ms-
1.25.0

python
ms-
2023.6.1

vscode-pylance
ms-
2023.5.10

jupyter
ms-
2023.3.1201040234

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

remote-containers
ms-
0.292.0

remote-ssh
ms-
0.100.0

remote-ssh-edit
ms-
0.84.0

remote-wsl
ms-
0.77.0

cmake-tools
ms-
1.14.31

cpptools
ms-
1.15.4

cpptools-extension-pack
ms-
1.3.0

remote-explorer
ms-
0.4.0

vsliveshare
ms-
1.0.5864

vscode-print
pdc
0.11.1

awesome-vhdl
puo
0.0.1

java
red
1.18.0

cmake
twx
0.0.17

vscodeintellicode
Vis
1.2.30

vscode-java-debug
vsc
0.50.0

vscode-java-dependency
vsc
0.22.0

vscode-java-pack
vsc
0.25.11

vscode-java-test
vsc
0.38.2

vscode-maven
vsc
0.41.0

vscode-todo-highlight
way
1.0.5

markdown-all-in-one
yzh
3.5.1

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyonecf:30548226
3biah626:30602489
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
a9j8j154:30646983
showlangstatbar:30730054
vsctsb:30705553
azdwalk:30721579
pythonms35:30701012
pythonfmttext:30731395
pythoncmvfstr:30731969
fixshowwlkth:30730052
hidesbindicator:30730055

I also see this issue with the most recent releases (all of a sudden).
Here's what the TypeScript server is spitting out:
2023-05-09 12:37:21.700 [info] Using tsserver from: /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js undefined
This path does exist, though (perhaps escaping the spaces is the issue?):
~  ls /Applications/Visual\ Studio\ Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js
/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js

My version info:
Version: 1.78.1
Commit: 6a995c4f4cc2ced6e3237749973982e751cb0bf9
Date: 2023-05-04T10:01:22.349Z
Electron: 22.5.1
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 21.6.0
Sandboxed: No

I did disable all extensions as well.

@LeonardoAiroldi-Polimi It looks like the issue is caused by one of your installed extensions. All extensions execute together on a single nodejs process, so in order to narrow down the possible problematic extension, please try the following:

run F1 > Help: Start Extension Bisect
(this will restart VS Code with half of your extensions disabled)
try to do the steps which reproduce the problem
if the problem reproduces, click "This is bad" (this narrows down the problematic extension to the half which was enabled)
if the problem does not reproduce, click "Good now" (this narrows down the problematic extension to the half which was disabled)
these steps will repeat until the problematic extension is found

initial
step
final

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Increase specificity for tooltip title stylings.

Explanation About What Code Achieves:

.title stylings are (in my opinion pretty common) and my tool tip titles were not displaying because of my existing .title styles. I'm using Bulma, but I think there are many css libaries that have stylings for titles. Adding the relevant styles to the more specific selector ensure they get used.
Screenshots/GIFs:

Before:

After:

Steps To Test:

A quick test involves adding the following styles to your html:

<style type="text/css">
    .graph-svg-tip > .title {
    font-size: 12px;
    color: #959da5;
    text-align: left;
    }
</style>

TODOs:

There might be more stylings that should be added in, or a better way to add specificity. I'm not sure if the goals are minimum size, or something else.

So we've namespace the styles in the current code base, which will be reflected in the upcoming release, so this would be unnecessary.

even better!

Exact same issue, also using Bulma :).


An error occurs when using an access policy to generate a SAS for one ADLS Gen2 blob container

Storage Explorer Version: 1.17.0
Build Number: 20201216.1
Branch: main
Platform/OS: Windows 10/ Linux Ubuntu 16.04/ MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to Reproduce

Expand one ADLS Gen2 storage account -> Blob Containers.
Create a blob container then right click it -> Click 'Manage Access Policies...'.
Add one access policy and save it -> Right click the blob container.
Click 'Get Shared Access Signature...' -> Select one access policy -> Click 'Create'.
Check there is no error.

Expected Experience
No error occurs.
Actual Experience
An error occurs.

Verified this issue on build 20210113.4. Fixed.

Verified this issue on build 20210113.4. Fixed.


Allow configuration of zod optional/null type

I'd love to use this library but since we redefine inputMaybeValue: 'T | undefined' in the graphql-codegen configuration, it's incompatible with the .nullish() that gets added to the zod schema when using graphql-codegen-tyespcript-validation-schema.
It would be great if we could have a configuration such as zodOptionalType that allowed us to specify optional or nullish.

@peniqliotuv Hi 👋
I'm sorry to delay this reply.
Could you tell me how to do this?

we redefine inputMaybeValue: 'T | undefined' in the graphql-codegen configuration

Hey @Code-Hex ! No worries.
Yea - I'm looking for an ability to customize the nullish values generated by graphql-codegen-typescript-validation-schema.
An example API that would be nice:
...
    config:
      maybeSchemaValue: 'nullish' | 'null' | 'undefined'

This value would default to nullish.
The behavior of nullish would be akin to what's currently in the plugin today.
z.nullish()

Providing null
z.null()

Providing undefined
z.optional()

I omitted the yup equivalents for simplicity's sake, but hopefully this makes it clearer!

I think instead of providing null, this should be nullable > .nullable().
Also to provide undefined, one needs to write optional, so that renders to .optional().

I'm also having this issue, my current workaround is to run sed -i 's/\.nullish()/.optional()/g' in afterAllFileWrite but it feels very hacky and prone to errors.


[core-amqp][event-hubs] browser support

The rhea package was updated in version 1.0.17 to better support bundling.
This change updates the minimum version of rhea used in core-amqp to ensure it can be bundled.
I also made some updates to the code in core-amqp and event-hubs that sets the user agent string. Previously it assumed it was running in a node.js environment in some cases.
Edit: I have tested these changes by in a front-end app built using webpack and parcel (separately). The browser live tests use rollup already.

/azp run js - eventhubs-client - tests

/azp run js - eventhubs-client - tests

/azp run js - eventhubs-client - tests

I'll update the pnpm-lock.yaml after @bterlson approval to prevent too much churn.

/azp run js - eventhubs-client - tests


Add Custom bookmarks

Just Do It!

Ok I will do it once I wake up

Ok I will do it once I wake up

The question is: will you wake up?

whoops it looks like this is too hard for me to do!

This is low but -> https://www.w3schools.com/howto/howto_js_todolist.asp


connectors: Add Unittests for opaconnector

This PR addresses #124 .
This PR does the following changes to opa connector:
(1) Minor redesign of opa connector to make it suitable for unit tests
(2) Preliminary unit tests for opa connector
To run the unit test, execute the following:
(1) cd to connectors/opa
(2)In the Makefile in this dir, set/unset NEED_TEST_COVERAGE_STATISTICS and NEED_TEST_COVERAGE_STATISTICS_IN_HTML as per requirement.
(3) Execute make test
Note: This PR is a revised version of PR https://github.com/IBM/the-mesh-for-data/pull/47 which was closed due to some errors due to DCO changes.

Added some comments. I'm still struggling with testutil/helper.go. It includes too much and actually having it hides what the test does (looking at the test file all you can tell is we send a request and expect some results, go to helper to see what)

basically testutil/helper.go has functions to (1) to start mock catalog connector (2) to start mock opa server (3) implements GetDatasetInfo on mock catalog connector (4) function to return mock OPA server response (5) have GetExpectedOpaDecisions() which returns pb.PoliciesDecisions object which has OPA decisions that are expected based on what are configured in the mocks.  (6) additional helper functions which help in implementing (1) - (5) above.
At a high level, testutil/helper.go is designed to help us to configure different test cases based on different scenarios which may arise when working with opa connector.
Please let me know if this is ok. Thanks.

Replied to your comments, let me know if you want to address them before merging.

Addressed most of your comments. pls let me know if I need to do anything about this comment. https://github.com/IBM/the-mesh-for-data/pull/125#discussion_r529367702. Thanks.

Replied to your comments, let me know if you want to address them before merging.

Addressed most of your comments. pls let me know if I need to do anything about this comment. #125 (comment). Thanks.

Actually what the description means is “redact columns with name nameOrig and nameDest  in datasets which have been tagged with Finance” ..The purpose could be marketing or fraud-detection or any other purpose..I have edited the description to the above to avoid confusion. Thanks.


Added static_comments plugin.

A plugin which allows to add static comments which don't require JavaScript. Needs theme support to actually show the comments.

LGTM. My blogs still compile and produce the same output.

Sure, fine for me. Merge whenever you are done.


Select component numeric value issue

The select react component has an issue when passing value as numeric. when we pass numeric 0 as value into Select
<Select
  className={styles['times']}
  value={0}
>
// ....

This logic will flag an empty class, which is not desired.

We should do null/undefined check instead of using falsy negative operator.


fix: data-race in badger hook

Potential fix for https://github.com/mochi-co/mqtt/issues/265

I will let others speak on this  as I am not super familiar with the storage implementation, but them seems to be a safe way to prevent said race conditions.

@gsagula This looks like it would work, as far as I can see. With the change in place, do you find it fixes any race conditions you are seeing?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.876%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5279

Relevant Lines:
5339

💛  - Coveralls


unpack API call

This should call get_tx_info2() (but passing the tx serialization) and then the 'unpacking' parts of send.parse(), order.parse(), etc., which should become separate functions.
The unpack method should return a dictionary of the message parameters (e.g. quantity, asset_name, destination).

is the "get_tx_info" API already available in master branch? I was unable to get this to work

@mtbitcoin, yeah: https://github.com/CounterpartyXCP/counterpartyd/blob/master/lib/api.py#L665-L668
What was the problem?

This is what I am doing (escaped because I am running on windows)
curl http://x.x.x.x:4000/api/ --user user:pass -H "Content-Type: application/json; charset=UTF-8" -H "Accept: application/json, txt/javascript" --data-binary  "{"jsonrpc":"2.0","id":0,"method":"get_tx_info","params": {"tx_hex":"0100000001ad369b446173cea81b150b856cea7bb2bab37595703ee3b2fe0c9e33bd416038000000008a4730440220591be7c1ef5b300eb1f72c755a28d54c34a11826e9a5d77dd310a987fd721cb30220765ce88463e00662d803e22b6457b14db0df5640376187a7efd30c9607d886d70141040286a47e8b11624c510881521b73a02929b9615d219100300c651771c0cf46c04b14cfa790e3f66940d11b5ab3d8ae81a3577473fbd36e2fc2b3d69e91d9af62ffffffff03781e0000000000001976a914b80c64c20a65646ae8238eeb019cc2380ca0617f88ac781e00000000000089512103e70605eeb731a017ad895d071893f3a9f2a69f1ff79106b2c7784608517d9870210381221a27700043f77357cd1becdd1c6f869c052b4960326ca23627fd713109ba41040286a47e8b11624c510881521b73a02929b9615d219100300c651771c0cf46c04b14cfa790e3f66940d11b5ab3d8ae81a3577473fbd36e2fc2b3d69e91d9af6253ae505f0000000000001976a914fa0692315a9f730b403e4be216b9b5d05492e81888ac00000000"}}"
And the response that I am getting is
--> {"id": 0, "error": {"message": "Invalid params", "code": -32602}, "jsonrpc": "2.0"}
The "unpack" method however works as expected using the sample above

@mtbitcoin, fixed: bafc122

Works. Thanks

Does the "unpack" work for anything other than sends? I tried to unpack an order and an asset issuance but received the following
(for getrawtransaction 140a2b99d960b828fe395cbc5d781bba31b117d3c3100db36a27db3147c7f6e8)
C:>curl http://localhost:4000/api/ --user x:X -H "Content-Type
: application/json; charset=UTF-8" -H "Accept: application/json, txt/javascript" --data-bi
nary  "{"jsonrpc":"2.0","id":0,"method":"unpack","params": {"data_hex":"000
000140000002bec05ea4e000000000000000000000000000000000000214d696461732052657a6572766520446
56d6f20436f696e7320284d52434f494e29"}}"
{"id": 0, "error": {"message": "Server error", "data": {"args": ["'module' object has no a
ttribute 'unpack'"], "message": "'module' object has no attribute 'unpack'", "type": "Attr
ibuteError"}, "code": -32000}, "jsonrpc": "2.0"}

@mtbitcoin, not at the moment. Right now, it works for all messages whose modules have implemented unpack().

Is there an API to decode the data_hex for non-send transactions ?

@mtbitcoin, it's all done with that method, it's just the other messages haven't had it implemented yet. Is it a priority for you?

ok. so eventually we will be able to decode the data_hex with the unpack method? Its not an ultimate priority now but I've seen this requested a couple of times and can be useful for reading raw  transactions. I was planning to use this for a CP transaction builder and decoder on blockscan.

Definitely


Is variable input size possible?

Is it possible to use rknn models with variable input sizes?
Like onnx dynamic_axes or somewhat.
As for now, it seems that when rknn model was built with some input_size_list, it will expect same input_size_list during inference.

I have the same issue at the moment


Cleanup sorting commit

I fixed a bug where both game objects and assets couldn't be renamed
at the same time. I also removed some unnecessary sorting ops.

@uvivagabond feel free to read over my notes if you're curious how I cleaned up the PR.


argparse: 2.9 -> 3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/argparse/versions.
meta.description for argparse is: Argument Parser for Modern C++
meta.homepage for argparse is: https://github.com/p-ranav/argparse
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0 with grep in /nix/store/2b054ars50sifp1b5lr7dw1s33iawizq-argparse-3.0
found 3.0 in filename of file in /nix/store/2b054ars50sifp1b5lr7dw1s33iawizq-argparse-3.0

Rebuild report (if merged into master) (click to expand)

11 total rebuild path(s)

11 package rebuild(s)

First fifty rebuilds by attrpath
airlift
argparse
ebusd
python310Packages.dataprep-ml
python310Packages.mindsdb-evaluator
python310Packages.pydateinfer
python310Packages.tabcmd
python311Packages.dataprep-ml
python311Packages.mindsdb-evaluator
python311Packages.pydateinfer
python311Packages.tabcmd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2b054ars50sifp1b5lr7dw1s33iawizq-argparse-3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A argparse https://github.com/r-ryantm/nixpkgs/archive/cc52251f9bc2db5e6c0417b783621db825f2a008.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/cc52251f9bc2db5e6c0417b783621db825f2a008#argparse

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2b054ars50sifp1b5lr7dw1s33iawizq-argparse-3.0
ls -la /nix/store/2b054ars50sifp1b5lr7dw1s33iawizq-argparse-3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

11 packages built:

airlift
argparse
ebusd
python310Packages.dataprep-ml
python310Packages.mindsdb-evaluator
python310Packages.pydateinfer
python310Packages.tabcmd
python311Packages.dataprep-ml
python311Packages.mindsdb-evaluator
python311Packages.pydateinfer
python311Packages.tabcmd

Maintainer pings
cc @2gn for testing.

Result of nixpkgs-review pr 266006 at cc52251f run on aarch64-darwin 1

10 packages built successfully:

airlift
argparse
python310Packages.dataprep-ml
python310Packages.mindsdb-evaluator
python310Packages.pydateinfer
python310Packages.tabcmd
python311Packages.dataprep-ml
python311Packages.mindsdb-evaluator
python311Packages.pydateinfer
python311Packages.tabcmd