Portfolio: desktop version

[ ] We used media queries in our CSS file to make my desktop version look like the template I selected (Desktop:        >=768px).
[ ]  In order to lay out the elements on the page I used use Grid and Flexbox.
[ ] I did not implement any functionality that requires JavaScript.
[ ] I replicated the design as much as possible (e.g., font, colors, images, text, margins) using the info from the Figma templates.

Good morning.
Please help me check if the design is okay now. Here is the link
https://github.com/Amazinggracee/Portfolio/pull/4
On Thu, 23 Feb 2023 at 00:37, Souad El Mansouri @.***>
wrote:

@.**** requested changes on this pull request.
Hi @Amazinggracee https://github.com/Amazinggracee ,
Good job so far!
There are some issues that you still need to work on to go to the next
project but you are almost there!
[image: 212385281-4797c58a-d358-423a-aa7c-832c22203865]
https://user-images.githubusercontent.com/59707859/220787900-8a6b2d09-dd69-4b75-9c44-dc35e43d0da5.gif
Highlights Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In style.css
https://github.com/Amazinggracee/Portfolio/pull/4#discussion_r1115083144
:

#intro {

display: flex;

flex-direction: column;

align-items: flex-start;
padding: 120px 24px;

gap: 12px;
position: absolute;
width: 100%;
height: auto;

well done @Amazinggracee https://github.com/Amazinggracee on this
project , but this section still needs small changes to be as close as
possible to figma desing as requested by the previous reviewer
the title doesn't have the same font size and I suggest that you use
relative units like rem to make your text also scale with screen size.
figma headline section
[image: figma template2 desktop headline]
https://user-images.githubusercontent.com/59707859/220784502-51c8a41d-4ad9-439c-80dd-e1de94b84272.PNG
your headline section
[image: student template2 desktop headline]
https://user-images.githubusercontent.com/59707859/220784563-c47f12bd-e552-46d1-bc37-765c953e3d18.PNG
also social media section should keep centered when changing screen size
to fix this issue all you need to use is justify-content: center as its
flexbox and has vertical direction and don't use margin-top with pixels as
this won't scale
In style.css
https://github.com/Amazinggracee/Portfolio/pull/4#discussion_r1115098347
:

-.worksDiv .line {

position: absolute;

top: 24%;

height: 1.2px;

width: 70%;

margin-left: 5%;

-}

-#works .real {

background-image: url(./Images/Img\ Placeholder.png);

background-repeat: no-repeat;

background-size: 100%;

width: 80%;

height: 237px;

margin-left: 5%;

+#works .worksDiv img {

display: none;

}
#works h5 {
well done on fixing the overlap issue , but this section still need few
changes to get perfect please pay attention to the spacing between the
element all elements in this section should have the same left and right
margin , I suggest that you use only one section as the content here is
semantically same so projects are meant to be inside works section
figma
[image: space figma]
https://user-images.githubusercontent.com/59707859/220787549-c9a41a21-3a27-4fa3-85a8-4c9d9c5d7045.PNG
your project
[image: scale]
https://user-images.githubusercontent.com/59707859/220787602-4ee07074-8d99-4c17-ad61-bf91b580443d.PNG
—
Reply to this email directly, view it on GitHub
https://github.com/Amazinggracee/Portfolio/pull/4#pullrequestreview-1310417823,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZW47KYNGCHQNIBIVVKHAKLWY2PKNANCNFSM6AAAAAAVEJYJFE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello @Amazinggracee  👋,
This is an additional review as a result of the second opinion request.
I think all the changes requested by @souad988 are clear and easy to understand. If you see that you have implemented all the requested changes, kindly request a new review on your dashboard.
If something specific is not obvious, kindly leave a comment outlining what you did not understand so that we can explain it in more detail. Please remember to tag @souad988 or me in your comment.
Keep rocking 🚀!


Can't get macro script to work! Please help!

Hi. Could you please explain better how I should use the Macro script? I can't get it to work but I can't code. What have I done wrong here?...
/**

@param {String} url - webview url
@param {Boolean} compat - compatibility mode on or off
@param {Number} w - width
@param {Number} h - height
@param {String} name - title of application
@param {String} customCSS - css selectors for elements to be removed, comma seperated
@param {String[]} userList - array of user id's to which the webview gets send, if empty it sends to everyone
@param {String} properties - string of html element properties to add to the iframe element
*/
window.Ardittristan.InlineViewer.sendUrl(https://docs.google.com/spreadsheets/d/1zP9s5kt9CTpOm4qYrzT2LIblNfL28SxqCKlk7KmpBbQ/edit?usp=sharing, compat = false, w = 1000, h = 1500, Conflict Resolve,,,)

I want to display a section of a google spreadsheet in the inline window, to get rid of the menu and stuff in the upper and left corners. Is this possible? If so, what would the macro script for this be?

normal:
window.Ardittristan.InlineViewer.sendUrl("https://docs.google.com/spreadsheets/d/1zP9s5kt9CTpOm4qYrzT2LIblNfL28SxqCKlk7KmpBbQ/edit?usp=sharing", false, 1000, 1500, "Conflict Resolve..." );

with hidden header:
window.Ardittristan.InlineViewer.sendUrl("https://docs.google.com/spreadsheets/d/1zP9s5kt9CTpOm4qYrzT2LIblNfL28SxqCKlk7KmpBbQ/edit?usp=sharing", true, 1000, 1500, "Conflict Resolve...", "#docs-chrome {display: none;}");

strings should be enclosed with quotes, and the = signs suggest that that's the default value

Thanks!! will check

It opens, and looks nice, but I would want to only show the area in the red square in the image below. Is this possible?

best I can get is this:
window.Ardittristan.InlineViewer.sendUrl("https://docs.google.com/spreadsheets/d/1zP9s5kt9CTpOm4qYrzT2LIblNfL28SxqCKlk7KmpBbQ/edit?usp=sharing", true, 1000, 1500, "Conflict Resolve...", "#docs-chrome, div[role=navigation] {display: none;}");

ok thanks!


Let extensions intercept schemas

export const MyExtension: GraphQLExtensionDeclaration = api => {
  api.loaders.schema.use(document => addDirective(document));

  return {
    name: 'my-extension',
  };
};

This is great, thank you!


Parser error: Non-default imports must always have a specifier!

Running Flow 0.13.1:
/* @flow */

import './sideEffect';
alert('lol');

Flow says:
Failure("Parser error: Non-default imports must always have a specifier!")

but it's a valid ES6 import without a binding.

@jeffmo fixed this in a recent commit, but it hasn't made its way into a release yet. Thanks for the report!

Lol, the import was even called the same way in the test.
Thanks!


SIGSEGV much more common in latest builds

#0  0x00007fe7916df2b8 in __cxxabiv1::__si_class_type_info::__do_dyncast(long, __cxxabiv1::__class_type_info::__sub_kind, __cxxabiv1::__class_type_info const*, void const*, __cxxabiv1::__class_type_info const*, void const*, __cxxabiv1::__class_type_info::__dyncast_result&) const () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#1  0x00007fe7919ec1dc in __cxxabiv1::__class_type_info::can_catch(__cxxabiv1::__shim_type_info const*, void*&) const () from /home/jon/.pyenv/versions/3.6.4/lib/python3.6/site-packages/datatable/lib/./libc++abi.so.1
#2  0x00007fe7916ddc65 in ?? () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#3  0x00007fe7916de1bf in __gxx_personality_v0 () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#4  0x00007fe7911ab263 in _Unwind_RaiseException () from /lib/x86_64-linux-gnu/libgcc_s.so.1
#5  0x00007fe7916de90c in __cxa_throw () from /usr/lib/x86_64-linux-gnu/libstdc++.so.6
#6  0x00007fe65dc1dfbe in dmlc::LogMessageFatal::~LogMessageFatal (this=0x7fff0c7fca50, __in_chrg=<optimized out>) at /root/repo/xgboost/rabit/include/rabit/./internal/../../dmlc/./logging.h:302
#7  0x00007fe65dc806e9 in dmlc::ParseFloat<float, true> (nptr=nptr@entry=0x5b7e938 "4920000500.0", endptr=endptr@entry=0x7fff0c7fcc50) at /root/repo/xgboost/dmlc-core/include/dmlc/./strtonum.h:137
#8  0x00007fe65dc80b58 in dmlc::strtof_check_range (endptr=0x7fff0c7fcc50, nptr=0x5b7e938 "4920000500.0") at /root/repo/xgboost/dmlc-core/include/dmlc/./strtonum.h:243
#9  dmlc::stof (pos=0x7fff0c7fcc48, value="4920000500.0") at /root/repo/xgboost/dmlc-core/include/dmlc/./strtonum.h:426
#10 dmlc::parameter::FieldEntry<float>::Set (this=0x5c85e00, head=<optimized out>, value="4920000500.0") at /root/repo/xgboost/dmlc-core/include/dmlc/parameter.h:996
#11 0x00007fe65dcad2a8 in dmlc::parameter::ParamManager::RunInit<__gnu_cxx::__normal_iterator<std::pair<std::basic_string<char>, std::basic_string<char> > const*, std::vector<std::pair<std::basic_string<char>, std::basic_string<char> > > > >
    (option=dmlc::parameter::kAllowUnknown, unknown_args=0x7fff0c7fce70, end=..., begin=..., head=0x5b7e638, this=0x7fe664d8d1a0 <xgboost::LearnerModelParam::__MANAGER__()::inst>)
    at /root/repo/xgboost/dmlc-core/include/dmlc/./parameter.h:404
#12 dmlc::Parameter<xgboost::LearnerModelParam>::InitAllowUnknown<std::vector<std::pair<std::basic_string<char>, std::basic_string<char> > > > (kwargs=..., this=0x5b7e638) at /root/repo/xgboost/dmlc-core/include/dmlc/./parameter.h:160
#13 xgboost::LearnerImpl::Configure (this=0x5b7e600, args=...) at /root/repo/xgboost/src/learner.cc:311
#14 0x00007fe65dc2c796 in xgboost::Booster::LazyInit (this=0x5cb9260) at /root/repo/xgboost/src/c_api/c_api.cc:58
#15 XGBoosterUpdateOneIter (handle=0x5cb9260, iter=0, dtrain=0x5c554b0) at /root/repo/xgboost/src/c_api/c_api.cc:904
#16 0x00007fe79a460356 in ffi_call_unix64 () at /tmp/python-build.20180521125357.1483410/Python-3.6.4/Modules/_ctypes/libffi/src/x86/unix64.S:76
#17 0x00007fe79a45f0b1 in ffi_call (cif=cif@entry=0x7fff0c7fd440, fn=fn@entry=0x7fe65dc2c720 <XGBoosterUpdateOneIter(BoosterHandle, int, DMatrixHandle)>, rvalue=rvalue@entry=0x7fff0c7fd350, avalue=avalue@entry=0x7fff0c7fd320)
    at /tmp/python-build.20180521125357.1483410/Python-3.6.4/Modules/_ctypes/libffi/src/x86/ffi64.c:525
#18 0x00007fe79a45676f in _call_function_pointer (argcount=3, resmem=0x7fff0c7fd350, restype=<optimized out>, atypes=<optimized out>, avalues=0x7fff0c7fd320, pProc=0x7fe65dc2c720 <XGBoosterUpdateOneIter(BoosterHandle, int, DMatrixHandle)>,
    flags=4353) at /tmp/python-build.20180521125357.1483410/Python-3.6.4/Modules/_ctypes/callproc.c:809
#19 _ctypes_callproc (pProc=pProc@entry=0x7fe65dc2c720 <XGBoosterUpdateOneIter(BoosterHandle, int, DMatrixHandle)>, argtuple=argtuple@entry=0x7fe68b6edb88, flags=4353, argtypes=argtypes@entry=0x0, restype=0x174d078, checker=0x0)
    at /tmp/python-build.20180521125357.1483410/Python-3.6.4/Modules/_ctypes/callproc.c:1147
#20 0x00007fe79a44d6d9 in PyCFuncPtr_call (self=self@entry=0x7fe689ac78e0, inargs=inargs@entry=0x7fe68b6edb88, kwds=kwds@entry=0x0) at /tmp/python-build.20180521125357.1483410/Python-3.6.4/Modules/_ctypes/_ctypes.c:3962
#21 0x00007fe79d8c18ca in _PyObject_FastCallDict (func=0x7fe689ac78e0, args=<optimized out>, nargs=<optimized out>, kwargs=0x0) at Objects/abstract.c:2331
#22 0x00007fe79d9b19b8 in call_function (pp_stack=pp_stack@entry=0x7fff0c7fd6c0, oparg=oparg@entry=3, kwnames=kwnames@entry=0x0) at Python/ceval.c:4848
#23 0x00007fe79d9b612b in _PyEval_EvalFrameDefault (f=<optimized out>, throwflag=<optimized out>) at Python/ceval.c:3322
#24 0x00007fe79d9b1854 in _PyEval_EvalCodeWithName (_co=0x7fe68b6b1390, globals=globals@entry=0x7fe68b60a090, locals=locals@entry=0x0, args=<optimized out>, argcount=4, kwnames=kwnames@entry=0x0, kwargs=0x5cd20d0, kwcount=0, kwstep=1,
    defs=0x7fe68b6ae3e0, defcount=1, kwdefs=0x0, closure=0x0, name=0x7fe79df3a5a8, qualname=0x7fe689aff870) at Python/ceval.c:4153
#25 0x00007fe79d9b1b07 in fast_function (kwnames=0x0, nargs=<optimized out>, stack=<optimized out>, func=0x7fe689ae0d90) at Python/ceval.c:4965
---Type <return> to continue, or q <return> to quit---

This is with hash c76d993681d8486decc43d4bd4b81d86762ed434 , while older hashes from 3 months ago had no such issues.
I don't have repro yet, but I hope the back trace from the seg fault will help.

Looks like the code attempted to initialize a float parameter with value<PHONE_NUMBER>.0. Do you know where this number comes from? The value is larger that 16777216 and cannot be represented exactly by float, so dmlc::Parameter is throwing an error. We may want to submit a fix so that error checking is not so aggressive (the value will be silently truncated).

@pseudotensor Actually I think the behavior is strictly a mistake, since<PHONE_NUMBER>.0 is simply 4.920005e9, which is totally fine value for a float. My bad. I will submit the fix right away.

@pseudotensor dmlc/dmlc-core#486 should fix the issue. We'll update the dmlc-core submodule after dmlc/dmlc-core#486 is merged.

Great, thanks!

No problem. And thanks for taking time to test it. Good thing we found the bug early and fix it right away.


fix(card): avoid clipping data state renderer error tooltip

Closes #1198
Summary

Change the direction that data state tooltips are rendered on cards

Change List (commits, features, bugs, etc)

items_here

Acceptance Test (how to verify the PR)

view https://deploy-preview-1278--carbon-addons-iot-react.netlify.app/?path=/story/watson-iot-valuecard--data-state-error-small
also view the other data state stories
the tooltip should render right instead of bottom

This will cause issues for RTL. Perhaps you could to use left side when in RTL?

Ah, good observation @bjornalm. I think I can make this configurable so it can at the very least be handled at the application level.

@bjornalm @sstone2423 PR has been updated to allow tooltip position to be configurable at the application level. Please re-review.

:tada: This PR is included in version 2.89.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: stale pageSize cause of missing useEffect deps

docs: add MinimalExample
Closes : none, I didnt open an issue
Description
adding missing useEffects deps, you might want to add the react-hooks/exhaustive-deps eslint rule for that, I noticed this isnt the only place with missing deps
Current Tasks
none
Live Demo Link
i did add a MinimalExample that was buggy before that PR, all the current examples use withPagination which solved the problem
basically it goes like this, with an initial state like:

before the PR:

after the PR:

looks good, i added eslint-plugin-react-hooks and fixed the hooks with missing deps


IncludePathsMatch issue when features path isnt supplied

Describe the bug
I think this bit of the code will evaluate differently for windows machines. this bug has been hiding here since version 3.x i think.

I'm running a windows machine
So what happens is, when i run behavex in parallel mode without supplying a features path argument.
Internally, FEATURES_PATH will resolve to a default value joining the absolute BEHAVEX_PATH and features:

in windows, this is most likely gonna be under DRIVE:\foo\bar\path\features - notice the : right after DRIVE

Result:

i get 0 tests when i run:

behavex -t one -t this --parallel-processes 2 --parallel-scheme scenario

but it works when i do:

behavex -t one -t this --parallel-processes 2 --parallel-scheme scenario features, which is essentially the same.

Note that behavex -t one -t this works just fine.
Desktop (please complete the following information):

OS: Windows

I'd be happy to help with the fix but I'm not really sure what the check for : is for.

Hi @iamkenos. First of all, thank you for taking time for reporting that kind of issues, it is very important for us to tackle the bugs as soon as possible. We are working on fixing this issue, and we will have a new release available very soon, most probably early next week.
Keep you posted on this.
Thanks!

Your solution seems to be working, to I have tagged the PR as fixed. Thanks!!


Create new ASP for users in basic creation

Fixes https://github.com/Microsoft/vscode-azureappservice/issues/795 (kind of)
For some reason, the method we call to list all the asp's today does not retrieve the property numOfSites.  In order to get that information, I had to retrieve each one with its own API call and it delayed the quickPick prompt considerably.  So I'm thinking it might not be worth it to display that in the UI.
If a user is in basic creation, I feel like rather than prompting them about performance issues, we should just create the new ASP for them when they're creating their fourth app.  I don't really see a downfall for doing that for them.

I was hoping you weren't going to mention that...

As I've been refactoring, I've realized that the asp and rg names are OS reliant, and we can't guarantee we will have that information until the user has been prompted for it (since we don't pick a runtime for the user if we don't see the appropriate files) so I think this PR will have to remain in the appservice package.

Take a look at createFunctionAppSettings https://github.com/Microsoft/vscode-azuretools/blob/master/appservice/src/createAppService/IAppCreateOptions.ts#L13
Maybe we could add a property to creationOptions like:
afterPrompt?(context: WizardContext): Promise<WizardContext>;

I'm not saying you have to do this refactoring now, but we've had several PRs affecting this code recently and it would be a lot easier (IMO) if it was in the app service directly

Closing this due to lack of activity


fix: replace Ogre deprecated header

Description
OgreVector3.h is deprecated in Jazzy, so this PR replace it
Tests performed
colcon test passed in both Humble and Jazzy.
Effects on system behavior
Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Note
This change does not involve any semantic changes to the source code because the only job of OgreVector3.h is to include OgreVector.h
https://github.com/OGRECave/ogre/blob/master/OgreMain/include/OgreVector3.h
On the other hand, OgreVector3.h says to migrate to Ogre.h, but it seems like an appropriate small change here because it would increase unnecessary includes and increase build times.


Please add a FOSS license

For inspiration see: https://choosealicense.com/

was added in https://github.com/InsaneTechnologies/elasticsearch-plaso-pipelines/commit/3fdeaac6029d26a8b4aa2de6709cd8acd4830d71


[r2.12-rocm-enhanced] Update golden api, enable api_compatibility_test

https://github.com/ROCmSoftwarePlatform/frameworks-internal/issues/3420

retest gpu-non-pip-large please


very slow with many files

Use example code and add ckeditor files to public folder, then run pkger, when i use packr, it works fine.
package main

import (
	"log"
	"net/http"

	"github.com/markbates/pkger"
)

func main() {
	if err := run(); err != nil {
		log.Fatal(err)
	}
}

func run() error {
	dir := http.FileServer(pkger.Dir("/public"))
	return http.ListenAndServe(":3000", dir)
}

I have the same issue. I have a newly created gobuffalo application. If I add one action and use pkger.Open in it just listing the one file with pkger on the command line takes > 30 seconds on my machine.

I generally like this solution and it works predictably. But it is extremely slow for moderately sized applications (~1m:35s on my app). I wonder if the issue is that it is walking every dependency's tree as well?

It takes a minute or so to parse for just small include.. (on a 12core 4gh machine...)
Ok the app is large but packr need one or some second or so...
First I tought the execution hangs due an internal bad-logic...
Sadly this lib is unusable with this performance


up to v0.2.0: reorganise & clean

actually we are up to v0.3.0 now.

@MarekSuchanek : what do you mean by 'identifier'?
Also, yes, sorry about the git mv... I forgot... :(

@MarekSuchanek : what do you mean by 'identifier'? Also, yes, sorry about the git mv... I forgot... :(

I meant the variable - I will fix it and merge this one. No problem with git mv, just for next time 😉

Thanks!!!


Maybe don't morph internally

Since we use an ID system to match nodes now, maybe we can go back to not morphing internally. Or maybe we do, since its more contained that way.

I tried it, but there are benefits to morphing internally:

didUpdate hook
Standalone, choo-free components


k4abt_sensor_orientation_t not working anymore

Hello,
The sensor_orientation parameter seems to not be taken in account anymore.
No matter the value we set, bodies are only detected with the camera beeing on its normal position.
Could you check if you have the same issue please ?
Thank you.

Hello,
Any update on the subject ?
Thank you.

Please provide more info:
BT SDK version
OS version
GPU model
tracker_config.processing_mode
Do you use default model file or you use other model files?

BT SDK: 1.1.0
OS version: Windows 10.0.19042
GPU model : Nvidia Geforce GTX 1050
tracker_config.processing_mode : K4ABT_TRACKER_PROCESSING_MODE_GPU
For model file we use dnn_model_2_0_lite_op11.onnx and dnn_model_2_0_op11.onnx
The error appears with both of them.

Dear @qm13, is this issue is corrected ? If so, when do you plan to release the new version.
Regard,
Pierre

@Moovency
Could you please provide more detailed information on how exactly are you reproducing this issue?
Also, it's interesting whether bodies are detected only partly, sometimes or they are not detected at all.

Hi @fkavache
To reproduce this issue :

call k4abt_tracker_create with k4abt_tracker_configuration_t config and set sensor_orientation with K4ABT_SENSOR_ORIENTATION_FLIP180  (this is reproducible also with the other options https://microsoft.github.io/Azure-Kinect-Body-Tracking/release/1.1.x/group__btenums_ga8e5fb3391addee8fadaa809c601e223e.html ).
Flip the Kinect Camera 180°
Try to track the skeleton

-> In our case, the body is not tracked at all when using 180° (rotation is not taken in account)
Let me know if you need more details.

Hi @Moovency
In my case body tracking is working fine.
In the worst scenario (flipped camera without any orientation parameter) at least there should be partial body recognition.
Can you try to:

record flipped .mkv video using k4arecorder tool and use it as an input?
(you can use the PlayFile function from simple_3d_viewer) sample as reference.
flip camera 180° without any orientation parameter.

Hi @fkavache
You can check in attachment the scenarios we have tested :

Pic01 : k4abt_sensor_orientation_t set to K4ABT_SENSOR_ORIENTATION_DEFAULT , camera in place -> body tracking ok
Pic02 : k4abt_sensor_orientation_t set to K4ABT_SENSOR_ORIENTATION_FLIP180, camera in place -> body tracking ok
Pic03 : k4abt_sensor_orientation_t set to K4ABT_SENSOR_ORIENTATION_FLIP180, upside down camera -> body tracking ko

Thanks !

Hi @Moovency
I rechecked just in case, I can confirm that BT works correctly with FLIP180 option and camera upside down.


bug @import layer syntax will use scss version

The @import for the layer option (soon to release officially) breaks if used without extension
@import "@fylgja/button/layer";

It will import the scss version even if used with postcss-import;
The way the file is built needs to change so that there is no scss version of the same name.

On hold for any patch release, this will break the import syntax for any SCSS users
Will be part of the v1.3 release

Note this rare case that will alos popup with older Sass versions
The solution is to move the variants to a src folder

Closing this for


adjust the check parameter order for uninstall

Adjust the  check parameter order in uninstall processing, move  the   regex  and length check before
get  release locker.  #2529

Yeah, it makes sense to move the lock down for this.


down coverage

test

Codecov Report

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

@@             Coverage Diff             @@
##           master       #3       +/-   ##
===========================================
- Coverage   91.36%   78.63%   -12.73%
===========================================
  Files          12       12
  Lines         220      220
===========================================
- Hits          201      173       -28
- Misses         19       47       +28

Impacted Files
Coverage Δ

FWRouter/Classes/RouteNode.swift
60.52% <0%> (-34.22%)
:arrow_down:

FWRouter/Classes/Router+Method.swift
72.72% <0%> (-27.28%)
:arrow_down:

FWRouter/Classes/PathComponent.swift
75% <0%> (-25%)
:arrow_down:

FWRouter/Classes/TrieRouter.swift
76.31% <0%> (-23.69%)
:arrow_down:

FWRouter/Classes/URLComponentsRepresentable.swift
76% <0%> (-4%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f470f67...59f7d79. Read the comment docs.


Provide documentation on how to make setup production ready

Doc: https://github.com/OraOpenSource/oraclexe-apex/blob/master/docs/connect_oracle.md
Blog post: http://www.oraopensource.com/blog/2015/4/16/how-to-connect-to-oracle
Decided to put this in a document so that it can be updated as part of the project and others can contribute to it.


First Close File Stream in NativeLibraryLoader's Load Method

When I used pushy which  depends on the netty-handler 4.1.5.Final lib, and ran the Sample Code in README.md on Windows 7 x64, I got the lock exception of the netty-tcnative-windows-x86_64.dll file: “Another program is using this file, the process can not access”.
Finally I found the cause in the class' load method:
......
out.flush();
loadLibrary(loader, tmpFile.getPath(), true);
} catch (Exception e) {
throw (UnsatisfiedLinkError) new UnsatisfiedLinkError(
"could not load a native library: " + name).initCause(e);
} finally {
closeQuietly(in);
closeQuietly(out);
......
Because the out stream is not close before invoking the loadLibrary method, the loadLibrary throws the exception.
Please fix it, many thanks!

It was fixed with 4.1.6.Final. See https://github.com/netty/netty/pull/5776

I see, Thanks!


gitAndTools.hub: 2.12.1 -> 2.12.2

Motivation for this change
Update gitAndTools.hub to the latest release.

https://github.com/github/hub/releases/tag/v2.12.2

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Looks like upstream uses go modules, should we update the expression to use buildGoModule?

Looks like upstream uses go modules, should we update the expression to use buildGoModule?

Not sure, they still appear to be vendored.


Config is Not Loaded in leads_vec_rc

Changing "leads_vec_rc/config.json" does not really have an effect.

The configuration is managed by a global registry since commit f76a48e1 though, it does not affect leads_vec_rc. I am still not sure about the mechanism behind this bug. It is likely to be caused by Uvicorn.


support sending Signals with "/signal"

Since Camunda Platform 7.8 there is an additional route "/signal" to send signals to specified executions or to all subscribed handler. It would be super cool, if you can manage to implement this. Unfortunately i don't have time for that yet.
greets Ephraim

@flyingpie does this means that your repo at MyGet is more updated than this one in Github? I mean which one should I use (MyGet or Github)?
Thanks...

@asalhani Currently, the NuGet package at MyGet contains all the changes proposed in the PR here: https://github.com/jlucansky/Camunda.Api.Client/pull/24. You can use that one until the PR is completed and the main package from NuGet gets updated.

@flyingpie thanks for the clarification...

Avaiable in 2.1.0


Guest room additions

It would be great to add some things to the guest room for people who stay for several days to feel more at home (specifically my parents, since they are here often and with mom's chemo coming up they'll likely be here a lot more often).
Right now they just set their suitcase on the folding chairs, and dad throws his clothes on the ground. He mentioned having somewhere to hang his clothes would be nice (so maybe a wall rack like I have in our closet?), and I think having a dresser where they could put stuff and/or store the suitcase on top would be nice.
Also, @amglovier you mentioned putting a TV in there for them, which i think they would really appreciate. :tv:
Let's price out some items, budget for them, and start cozifying!
:construction: :hammer: :womans_clothes: :shirt: :computer:

Here's a decent dresser from Ikea for $99:

This one is closer to $200, but shipping is free. It's also really gorgeous.

This one is only $100 and has free shipping, though it's not as fancy

And of course Walmart has this joint for $66, and it's not bad looking.

Heh, that exact same $100 one from Overstock is on Amazon for $80 and Prime eligible (but not in stock).

Talked to Mom, and based on the height and dimensions, she like's this one from Walmart best:

Found this one at Walmart and it looked way better:

We got the one from Walmart, and some wall artwork/mirror, and a coat rack. I'd call that a great start, and enough for now.


Video Overlay in CameraSubsystem

Pull Request Details

Added a camera subsystem that overlays a black line over the robot's video stream
Description

Creates CameraSubsystem
Intended to help with alignment of arm when placing game pieces - may be obsolete with new intake designs
Related Issue

Which does this affect?

CameraSubsystem
Testing Done

Checklist

[x] My code follows the code style of this project.
[x] My code has been tested on the Robot OR includes simulation code to prove functionality.

Checked comments, and other changes


[BEAM-8815] Define the no artifacts retrieval token in proto

Is there a good way to avoid repetition of extend google.protobuf.EnumValueOptions for this purpose?

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.


BQ Data Transfer: Add quickstart sample

A sample showing how to use the BigQuery Data Transfer API to list
possible data sources.

Travis is failing on go get, but seems unrelated to this change.
github.com/go-sql-driver/mysql (download)
# cd /home/travis/gopath/src/github.com/go-sql-driver/mysql; git pull --ff-only
You are not currently on a branch.
Please specify which branch you want to merge with.
See git-pull(1) for details.
    git pull <remote> <branch>
package github.com/go-sql-driver/mysql: exit status 1


feat: added statuses to generic actor message queue loop processing

Implementation of stop process function for actor concurrency model.
You can stop actors from any thread using #stop_actor method. After stopping the actor currently running(got from message queue tasks) stay running but they can't send the results to the caller. Also if you try to call stopped actor it will raise GenericActor::AbuseStoppedActorException.
Note: After the changes mechanic, specs became slower

Still not clear, why do you want to stop actors?
What problem are you trying to solve?

Still not clear, why do you want to stop actors?
What problem are you trying to solve?

The problem I want to solve is control of the life flow of actors. If you can create and start an actor you should stop and delete the actor. Resource control.

Deleting an actor object does not help?

Deleting an actor object does not help?

How you can do it if it has a running loop at another fiber?

Deleting an actor - means assigning all variables that point to nil.
Probably in actor deinitialization of an channels ad fibers should stop.
But I would not put an explicit method to stop an actor.
And it is still not clear for me - what problem are you trying to solve?

Is it going to improve performance?
Does it fix a certain bug?
Do we have a need to stop actors in the scope of certain functionality in the engine?
Do we have resource leaks because of unstopped actors? (if yes - we need comparative analysis)

I can see that it makes specs slower. WHy? Will it make code slower?

Deleting an actor - means assigning all variables that point to nil. Probably in actor deinitialization of an channels ad fibers should stop. But I would not put an explicit method to stop an actor.
And it is still not clear for me - what problem are you trying to solve?

Is it going to improve performance?
Does it fix a certain bug?
Do we have a need to stop actors in the scope of certain functionality in the engine?
Do we have resource leaks because of unstopped actors? (if yes - we need comparative analysis)

I can see that it makes specs slower. WHy? Will it make code slower?

That is finishing to be correlated with the actor model. Now it won't help to project cause little number of exploit mechanics use GeneticActor mechanics. But to make code more assyncronious you need implement this functions to free resources gracefull


Demo Doesn't work

Describe the bug
https://myriad-dreamin.github.io/typst.ts/preview.html this doesn't work
To Reproduce
Steps to reproduce the behavior (Library test):

Go to https://myriad-dreamin.github.io/typst.ts/preview.html
Click "Export to PDF"

Console Output
Uncaught (in promise) LinkError: import object field '__wbg_call_3bfa248576352471' is not a Function
Uncaught (in promise) LinkError: import object field '__wbg_iterator_695d699a44d6234c' is not a Function

I have fixed it in https://github.com/Myriad-Dreamin/typst.ts/pull/623 and purged caches on jsdelivr. Now it works on my computer. Please invalid your browser caches and check it again.

Works :1st_place_medal:  thanks


net/http: Custom Error handlers

In the http package Error displays just an error message. When building a web application you will often want to have style error pages matching the site or application. In order to get this with Go you need to implement your own Error handling along with any code that calls it.
For example, if you want custom error pages (including 404 not found pages) and want to serve files you need to bypass the built in file server and re-implement much of it to change the error page handling.
It would be far simpler to allow the Error to provide a default but be capable of being replaced.

It's hard to get excited about this.
Something like this?
var ErrorHandler func(w http.ResponseWriter, error string, code int) = func(w ResponseWriter, error string, code int) {
    w.Header().Set("Content-Type", "text/plain; charset=utf-8")
    w.WriteHeader(code)
    fmt.Fprintln(w, error)
}

func Error(w ResponseWriter, error string, code int) {
    ErrorHandleR(w, error, code)
}

Seems a bit gross TBH. I don't know that we want to add yet another hook.

It so easy to write your own functions.

@rsc how would one write an error handler for the not found case and use that in the built-in file server instead of the NotFound response? Something so I wouldn't need https://github.com/Masterminds/go-fileserver

Give ServeContent or whatever a ResponseWriter which intercepts the 404/403/500/etc and writes its own message.

@bradfitz that's an interesting take I had not thought of. It should work.
My issue with it is that it's not a good separation of concerns. It's a hack to work around this limitation. Something that's responsible with content now needs to look for certain types of responses and override the output to something custom.
It would be nice to have a clean solution.

There's also this and this that seem to bypass the ResponseWriter altogether.


Issue resolving

Issues resolved:

#76
#136
#119

Pull Request Test Coverage Report for Build 222

0 of 46 (0.0%) changed or added relevant lines in 7 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.1%) to 65.478%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/logic/parser/AddressBookParser.java
0
2
0.0%

src/main/java/seedu/address/model/ModelManager.java
0
3
0.0%

src/main/java/seedu/address/model/AddressBook.java
0
3
0.0%

src/main/java/seedu/address/ui/CommandBox.java
0
5
0.0%

src/main/java/seedu/address/logic/parser/DeleteJobCommandParser.java
0
6
0.0%

src/main/java/seedu/address/logic/CommandCorrection.java
0
8
0.0%

src/main/java/seedu/address/logic/commands/DeleteJobCommand.java
0
19
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/address/logic/CommandCorrection.java
1
0.0%

src/main/java/seedu/address/ui/JobListPanel.java
2
90.91%

Totals

Change from base Build 221:
-1.1%

Covered Lines:
1946

Relevant Lines:
2972

💛  - Coveralls


Question about the voting module architecture

Hi,
According to the paper description of the voting module:
The voting module MLP has output sizes of 256, 256, 259 for its fully connected layers. The last fully connected layer does not have ReLU or BatchNorm.
However, it seems that the FC layers in voting_module.py are replaced by convolutional layers :
self.conv1 = torch.nn.Conv1d(self.in_dim, self.in_dim, 1)
self.conv2 = torch.nn.Conv1d(self.in_dim, self.in_dim, 1)
self.conv3 = torch.nn.Conv1d(self.in_dim, (3+self.out_dim) * self.vote_factor, 1)
Is either known to perform better than the other? And how does the parameter sharing work in case of the Conv1d layers?
Thanks

actually here 1x1 Conv is just FC, i.e., MLP


Repo has disappeared and won't auto-import on godoc.org

My spellcheck respository was removed from godoc.org, and when I try searching for it, it won't auto-import it. Any thoughts?

Hm, when I load [the godoc page] I see this in the logs:
2015/10/07 03:21:47 web   fetch: 480 notfound: no Go files or subdirs github.com/nathanjsweet/gohun

Not sure why this is happening, as there does appear to be Go source code in that repo. Did you change anything around the same time as it became delisted?

Yeah, a little:

I changed main.go to use the CGO ${SRCDIR} feature that's new to 1.5+, moving away from pkg-config.
I changed the makefile, as it didn't have as much work to do (creating the pkg-config file, etc).

Other than those two things I can't think of anything, other than fixing a typo I found in the doc.go file.

I cloned your repo and observed the same "Not found" error for godoc.org on
that repo.
Then I deleted all go files except doc.go
and the documentation reappeared.
I restored the _test.go file and it still worked. So, something about the
contents of gohun.go is tricking godoc.org into thinking the package
doesn't exist.
On 7 October 2015 at 14:28, Nathan Sweet<EMAIL_ADDRESS>wrote:

Yeah, a little:

I changed main.go to use the CGO ${SRCDIR} feature that's new to
1.5+, moving away from pkg-config.
I changed the makefile, as it didn't have as much work to do
(creating the pkg-config file, etc).

Other than those two things I can't think of anything, other than fixing a
typo I found in the doc.go file.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/gddo/issues/332#issuecomment-146068796.

Well, I think that the site isn't handling the new ${SRCDIR} directive very well. Try using this gohun.go, to see if it works.

I don't see why it should care about the SRCDIR directive. It shouldn't be
parsing cgo comments at all.
On 7 October 2015 at 16:27, Nathan Sweet<EMAIL_ADDRESS>wrote:

Well, I think that the site isn't handling the new ${SRCDIR} directive
very well. Try using this gohun.go
https://raw.githubusercontent.com/nathanjsweet/gohun/2929e8a0d9e1f60186385e7c9611b876566376e7/gohun.go,
to see if it works.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/gddo/issues/332#issuecomment-146080262.

Well, unfortunately, it seems to be the case, because I took it out and now its working.

I don't see why it should care about the SRCDIR directive. It shouldn't be
parsing cgo comments at all.

My guess is that the dollar sign and or brackets are throwing it off.

I don't see why it should care about the SRCDIR directive. It shouldn't be
parsing cgo comments at all.

Builder.go does seem to care.

FYI, I reverted my repo, so that it's not broken. It no longer is in godoc.org, as before.

The server running on my Linux desktop can load the package. I built the server with Go 1.5.1. Hmmm.

I'm guessing that 1.5 is currently not in use on the live site? That would be my guess.

Yup, breaks on 1.4. I submitted #333 to help diagnose problems like this the future.
@adg Please build with 1.5.1.

Updated to 1.5.1. Now it works.

Thanks.


Simplify machine tee writer

The MultiMachineWriter better compliments the behavior we're modelling given Tee only outputs to 2 things and is a CLI tool. Similalry, the stdlib has a MultiWriter function so we better align.

/hold

/unhold

/lgtm


Update alias

Docs Build status updates of commit 41bd844:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch modify_install has been deleted or has been force pushed: fatal: Couldn't find remote ref refs/heads/modify_install

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


using sdl2: can't load .so/.DLL for: mingw32.dll

Hi it's me again with a similar problem as in #5
I'm trying to use the sdl2 package on windows (mostly for developing in (quine)[ekmett/quine]). After some pitfalls I'm able to build the sdl2 package from source and from hackage. My current build flow:

unpack the SDL2-devel-2.0.3-mingw.tar.gz from https://www.libsdl.org/download-2.0.php into C:\SDL2-2.0.3
cabal get sdl2
cd sdl2
cabal configure --extra-include-dirs="C:\\SDL2-2.0.3\\include" --extra-lib-dirs="C:\\SDL2-2.0.3\\x86_64-w64-mingw32\\lib"
cabal build

the build is successful but using the lib in an other project results in a build error:
Loading package sdl2-1.3.0 ...
<no location info>:
    <command line>: can't load .so/.DLL for: mingw32.dll (addDLL: could not load DLL)
[49 of 51] Compiling Quine.Geometry.Frustum ( src\Quine\Geometry\Frustum.hs, dist\build\Quine\Geometry\Frustum.o )

<no location info>:
    ghc.exe: panic! (the 'impossible' happened)
  (GHC version 7.8.4 for x86_64-unknown-mingw32):
        Dynamic linker not initialised

Please report this as a GHC bug:  http://www.haskell.org/ghc/reportabug

the relevant lines from the SDL pkg-config:
Libs: -L${libdir}  -lmingw32 -lSDL2main -lSDL2  -mwindows
Libs.private: -lmingw32

I believe that there is no mingw32.dll in MinGW. A wild guess: something tries to link dynamically while it should linked statically?
related on SO: http://stackoverflow.com/questions/15996796/haskell-sdl-cant-find-mingw32-dll
I'm a bit lost here. Is this really a ghc bug? Should I report it there? Or is this a bug of this bundle. I'm using the GHC 7.8.4 (64-bit) version of MinGHC.

okay sorry for the noise, it's a known bug. https://ghc.haskell.org/trac/ghc/ticket/3242
so no windows for my project :(

Sorry to hear that, not much for us to do then.

Note that this bug is specific to 64-bit GHC (and the reason I'm using 32). If you can make that switch, also see here for how to avoid a bug on the SDL2-devel library side.

@tejon thank you very much for this information. I will consider now to switch to 32bit!

sorry for the noise again:
I switched to the x32 version and ran into another bug...
like above:

cabal get sdl2
cd sdl2
cabal configure --extra-include-dirs="C:\\SDL2-2.0.3\\include" --extra-lib-dirs="C:\\SDL2-2.0.3\\x86_64-w64-mingw32\\lib"
cabal build

results in this error:
cabal get sdl2
Building sdl2-1.3.0...
Preprocessing library sdl2-1.3.0...
running dist\build\Graphics\UI\SDL\Types_hsc_make.exe failed (exit code -1073741701)
command was: dist\build\Graphics\UI\SDL\Types_hsc_make.exe  >dist\build\Graphics\UI\SDL\Types.hs

and a windows error dialog pops out with the error code 0xc000007b

The x86_64 mingw lib path looks suspicious.
Cheers,
Darren
On Feb 9, 2015 1:15 PM, "Jan-Philip Loos"<EMAIL_ADDRESS>wrote:

sorry for the noise again:
I switched to the x32 version and ran into another bug...
like above:

cabal get sdl2
cd sdl2
cabal configure --extra-include-dirs="C:\SDL2-2.0.3\include"
--extra-lib-dirs="C:\SDL2-2.0.3\x86_64-w64-mingw32\lib"
cabal build

results in this error:
cabal get sdl2
Building sdl2-1.3.0...
Preprocessing library sdl2-1.3.0...
running dist\build\Graphics\UI\SDL\Types_hsc_make.exe failed (exit code -1073741701)
command was: dist\build\Graphics\UI\SDL\Types_hsc_make.exe  >dist\build\Graphics\UI\SDL\Types.hs
and a windows error dialog pops out with the error code 0xc000007b
—
Reply to this email directly or view it on GitHub
https://github.com/fpco/minghc/issues/28#issuecomment-73591631.

@Kludgy sorry was a c&p error, I corrected it just a minute after hitting the comment button.


Failed to execute 'define' on 'CustomElementRegistry': this name has already been used with this registry

If there's more then 1 component library / element loaded then storybook fails to build :(
Trying to trace down if it's just in Polymer resolutions or what

import<EMAIL_ADDRESS>reference is what was failing. New issue for that but that's more a "nice to have"


fix: strip dev from version when semver comparing for migrations

This is to address #820 - creating the PR as I'm hitting this issue and now cannot upgrade versions without wiping the DBs.
Also, rc can be in the version too. I haven't covered that in this PR though because then dev1 and rc1 are the same version. Perhaps we can ignore it and just test rc versions in release environments? Open to suggestions!

I addressed this in a different way my migration PRs here.
Thanks for catching the second version check, I only had the first. My migrations PR is updated to include both.

Nice @kentbull that looks like a better approach!
I wonder if we should capture the dev or rc using it too. On the other hand, if there's a migration within the same patch version at the dev or rc level, maybe this means the minor or patch should be being bumped!

if there's a migration within the same patch version at the dev or rc level, maybe this means the minor or patch should be being bumped!

That makes sense to me. Anything significant enough to require a database migration qualifies for a patch bump.

Cool - I prefer your approach to my quick hack, so I'm going to close this


Adding cargo clippy

I would like to run cargo clippy as well as cargo check. How would I go about adding that?

There is https://github.com/neomake/neomake/blob/4cf0b39777728652f13eed32b1667a4412e1a286/autoload/neomake/makers/clippy.vim already, so Neomake! clippy should work.

Cool, i missed that..


Cvdz

NOTE: New updated data is intended to be released once weekly on Tuesday until further notice.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Additional context
Add any other context or screenshots about the feature request here.

I like to help anyone needs me


Audaspace: Update PulseAudio sync functions to be more accurate

As requested in https://projects.blender.org/blender/blender/pulls/126091, here is the pulseaudio changes.

I just wanted to give you a heads up that I'm currently sick with covid. (So things might go a bit slower)
I guessing that it might be because I'm a bit brain dead right now, but I'm having a hard time to grasp exactly what changes to the interface you are proposing to the interface and synchronizer.
I do think that having some interpolation even in the underlying API doesn't have it is good. One of the issues we had on the Blender side of things was the people switched off from using A/V sync as the playback wasn't as smooth as without it.
IIRC I implemented something in the SDL backend to smooth out the position reporting.
It is also the reason I've been working on the other audio backends to make it as good as regular V-sync.

You don't want to make the ring buffer unnecessarily big. Sure, if it's just the scene playing it would work, but again, the UI could also want to play back sounds in some cases, maybe in plugins and that would then have a 1 minute delay, which is of course unacceptable.

Would we want some kind of secondary API for these kinds of "play sound NOW" cases?
It feels a bit weird to me how things are now intertwined in that the audio device has to be in a "playing == true" state for these kinds of impromptu sounds to be played back.
For example. Lets say that Blender did have UI notification sounds and I have queued up sounds on the timeline in the VSE off Blender but it is paused. If a notification sound wanted to playback, and I simply switch on the device to play the notification sound (playing(true)), it would also start playing back the audio timeline.
Again, perhaps I'm just too braindead and missing something obvious.

I just wanted to give you a heads up that I'm currently sick with covid. (So things might go a bit slower) I guessing that it might be because I'm a bit brain dead right now, but I'm having a hard time to grasp exactly what changes to the interface you are proposing to the interface and synchronizer.

No worries, I also take long to respond sometimes, when I'm busy with other stuff. Get well soon!

I do think that having some interpolation even in the underlying API doesn't have it is good. One of the issues we had on the Blender side of things was the people switched off from using A/V sync as the playback wasn't as smooth as without it. IIRC I implemented something in the SDL backend to smooth out the position reporting.
It is also the reason I've been working on the other audio backends to make it as good as regular V-sync.

Interpolation is another thing that could be added to smooth it out, but you have to be careful there to not cause jumps again, i.e. let the position only increase monotonically.

I think would we want some kind of secondary API for these kinds of "play sound NOW" cases? It feels a bit weird to me how things are now intertwined in that the audio device has to be in a "playing == true" state for these kinds of impromptu sounds to be played back.
For example. Lets say that Blender did have UI notification sounds and I have also queued sounds on the timeline in the VSE of Blender but it is paused. If a notification sound wanted to playback, and I simply switch on the device to play the notification sound (playing(true)), it would also start playing back the audio timeline.
Again, perhaps I'm just too braindead and missing something obvious.

I think you are mixing up two things here. First, there is the playback state of the device. This is mostly internal to the device to be able to shutdown/pause/sleep the backend whenever there is actually nothing playing. This can of course not be changed.
The other playing state is that of the synchronizer and that one is independent of the playing state of the device. That is the one that should be used for synchronization in Blender with the timeline. It's of course not used in the case of notification sounds and the like.

I think I understand now. Thank you so much for you explanation and patience!
I realized that my current implemention would break if the end user didn't regularly call getPosition. (As the m_timeline_pos variable only got updated when calling getPosition.
I've updated the code to get rid of m_timeline_pos and as a bonus is should now accurately report the position even if the user didn't call getPosition during playback. (Of course this doesn't take into account the ring buffer latency.
Note that I only test it briefly as I just wanted to check if you are ok with the changes.
Regarding the latency and general API discussions:

As for the latency that is introduced by the ring buffer, we could simply store the ring buffer read size when play or seek is called and keep the reported position at the start/seek time until we can reduce that to 0. I guess this is what you mean by stop counting up the clock in that case, right?

Yes, that is what I had in mind.
However I feel it might be best to do all of this later, right?


[DRLK] High level PICS has to be updated as per test plan changes

Description:
High level PICS has to be updated as per latest test plan for below test cases:

TC-DRLK-2.5
TC-DRLK-2.7

Script Reference:

Test plan Reference:

@KishokG fixed in PR 31060 please verify it and close the ticket.

Issue resolved


Use git remote to avoid PAT tokens

Extends upon #5
This uses git as remotes so that you don't run into rate limiting issues and avoids the need for a GitHub PAT token.

Could you help me understand how this works? Why does this avoid the GitHub download limit?

Could you help me understand how this works? Why does this avoid the GitHub download limit?

Remotes seems to use the GitHub API to download and inspect files when you specify the repo as "github". When you mark the remote as a git repository, it should clone it as a regular repository and you not use the API (thus not triggering the rate limit and avoiding the need for a PAT).
https://remotes.r-lib.org/reference/install_git.html#examples
I'm still testing this. I'll confirm when it works.

Remotes seems to use the GitHub API to download and inspect files when you specify the repo as "github". When you mark the remote as a git repository, it should clone it as a regular repository and you not use the API (thus not triggering the rate limit and avoiding the need for a PAT).
https://remotes.r-lib.org/reference/install_git.html#examples
I'm still testing this. I'll confirm when it works.

Does not seem to work

Does not seem to work


How to exclude folder?

I have public folder with images uploaded by users? After deploy this folder is overwritten. There are some ways to avoid this?

Originally got the idea here: https://github.com/tomitrescak/meteor-uploads/issues/235#issuecomment-228618130
Added issue for feature upgrade here: https://github.com/kadirahq/meteor-up/issues/160

#160


Splash Display Fix + Code Clean-up

Splash display is not respecting the default time properly
We need to look at the logic for #define SPLASH_DISPLAY
Additionally, we started having Get* functions in the StorageManager.h which breaks our current coding conventions. We need to remove these and make the StorageManager.h get functions single option only rather than option.
This means we need to get rid of
int Storage::GetButtonLayout()}
int Storage::GetButtonLayoutRight()
int Storage::GetButtonLayout()
int Storage::GetButtonLayoutRight()
int Storage::GetSplashMode()
int Storage::GetSplashChoice()
int Storage::GetSplashDuration()
int Storage::GetSplashMode()
int Storage::GetSplashChoice()
int Storage::GetSplashDuration()
and start being more strict about PRs that do this.

This has been addressed.  Thank you @deeebug and @arntsonl


Gracefully catch HTTP errors

Improved the exception handler to catch exceptions from async or sync functions, and to then report that exception to the command line and abort the operation.

LGTM!


Source and destination overlap in memcpy_chk errors with Valgrind

Running valgrind shows these errors:
==20160== Source and destination overlap in memcpy_chk(0x416818, 0x4169d0, 3520) ==20160==    at 0x4A4A310: __memcpy_chk (vg_replace_strmem.c:1617) ==20160==    by 0x376BDF: memmove (string_fortified.h:36) ==20160==    by 0x376BDF: gfxd_execute (gfxd.c:746) ==20160==    by 0x2B968D: ZDisplayList::ProcessGfxDis(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /home/louist103/decomp/oot/tools/ZAPD/ZAPD.out) ==20160==    by 0x2B7B5B: ZDisplayList::GetSourceOutputCode(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /home/louist103/decomp/oot/tools/ZAPD/ZAPD.out) ==20160==    by 0x2CAA95: ZFile::GenerateSourceFiles(std::filesystem::__cxx11::path) (in /home/louist103/decomp/oot/tools/ZAPD/ZAPD.out) ==20160==    by 0x2C8C17: ZFile::ExtractResources() (in /home/louist103/decomp/oot/tools/ZAPD/ZAPD.out)

Having overlapping source and destination is perfectly safe when using memmove, that is the reason why memmove is used in this instance instead of memcpy. This is a superfluous warning by valgrind. Closing.


details finding edit direction

I wonder how many sentences do you use to find the edit direction? How about the modifying task w and w/o glasses, do you use sentences include and do not include the word 'glasses'?

I generated more than 10000 sentences with the source and target words using gpt-3.5, finding that the extracted editing direction performed worse than directly swapping the words :(
Hoping the authors to complete the implementation details about finding editing direction.

Hi @nancy6o6,
Could you mention what exact editing task you're trying to implement and some more details about your setup?


Support for default verb?

 Issue by alexreg
Monday Oct 16, 2017 at 17:20 GMT
Originally opened as https://github.com/gsscoder/commandline/issues/494

Is there any support for some sort of "default" verb? That is, a verb that gets executed when no verb is actually specified. Perhaps an existing verb can be marked as default, even?

 Comment by alexreg
Monday Oct 16, 2017 at 17:38 GMT

Furthermore, is it possible to have global options while still having a verb?

 Comment by nemec
Monday Oct 16, 2017 at 17:46 GMT

This is a duplicate of #490. Unfortunately, no it isn't possible. With the way verbs work in the current version (e.g. Parse<VerbA, VerbB>(args)) I'm having a hard time thinking of a way to add a default verb and global options.

 Comment by alexreg
Monday Oct 16, 2017 at 18:06 GMT

Thanks for the quick reply, @nemec. What about a simple IsDefault property of the Verb attribute. The name of the verb should also be optional in this case. As for global options, what about a special Options generic argument that inherits from a (dummy) GlobalOptions class, or something similar?
Update: Actually, quite a nice solution that already works is simply inheriting from a GlobalOptions class!

 Comment by alexreg
Monday Oct 16, 2017 at 18:46 GMT

I'm currently trying to come up with a workaround for this, but I can't get the NoVerbSelectedError, nor the BadVerbSelectedError. It just runs my (one and only) options handler instead. Using the latest beta here. Any known issue with that right now?

 Comment by alexreg
Monday Oct 16, 2017 at 20:28 GMT

Update: it seems I have to specify at least two options types to ParseArguments for these errors to kick in! I see why, though perhaps this should be documented.

 Comment by TheFanatr
Saturday Oct 21, 2017 at 07:16 GMT

Just to clarify, is there a workaround by using GlobalOptions?

 Comment by nemec
Tuesday Oct 17, 2017 at 00:38 GMT

You're right - it is not obvious that one generic argument is interpreted as an Options class but multiple are considered Verbs.

I try to find a workaround. I use solution like this and it seems to work.
    [Verb("add", HelpText = "Add file contents to the index.")]
    class AddOptions
    { //normal options here
    }
    [Verb("commit", HelpText = "Record changes to the repository.")]
    class CommitOptions
    { //normal options here
    }
    class CloneOptions
    { //normal options here
    }

    class Program
    {
        static void Main(string[] args)
        {
            Parser parser = new Parser(o =>
            {
                o.HelpWriter = null;
            });
            var result = parser.ParseArguments<AddOptions, CommitOptions>(args);

            if ((result is NotParsed<object>) &&
                ((NotParsed<object>)result).Errors.Any(o => o.Tag == ErrorType.NoVerbSelectedError || o.Tag == ErrorType.BadVerbSelectedError))
            {
                parser.ParseArguments<CloneOptions>(args)
                    .WithParsed<CloneOptions>(opts => Console.WriteLine(opts))
                    .WithNotParsed(errs => Console.WriteLine(errs.First().Tag));
            }
            else
            {
                result
                  .WithParsed<AddOptions>(opts => Console.WriteLine(opts))
                  .WithParsed<CommitOptions>(opts => Console.WriteLine(opts))
                  .WithNotParsed(errs => Console.WriteLine(errs.First().Tag));
            }
        }
    }

I have some legacy options (without a verb) and need to add some verbs. I came up with a similar workaround like @tkouba. The problem with such an approach is that help texts are no longer automatically generated.
So built-in support for default verbs would be much appreciated.
The CLAP library (https://github.com/adrianaisemberg/CLAP) seems to support default verbs, but I'm hesitating to move to that library, because it does not look like it's still actively maintained.
Is supporting default verbs on the roadmap for this library?

V2.8 support default verb


:bug: no doctests running!

There's no unit(/integration) test file to actually run doctest on the docs! Easy fix, but needs to be done.

Resolved in #18


Keep and report stats on texture calls and handles

I did this to help me debug something (which turned out not to be a bug), but it's still a useful item to keep in the stats output.

LGTM


Memory corrupt when running in distributed mode with CUDA 8.0 and Tesla M40

We have implemented tested the distributed TensorFlow applications. It works in CPU environment and the GPU environment with CUDA 7.5 and Tesla K40. But it fails with CUDA 8.0 and Tesla M40.

Not sure if it's related to CUDA 8.0. The test code is here and 100% re-produce if I run it with CUDA 8.0 and Tesla M40.
Environment info
Operating System: CentOS 7.0
CUDA and cuDNN: CUDA 8.0 and cuDNN v5
TensorFlow version: 0.11.0rc0

We have tested and it works with CUDA 7.5 and M40.
It seems to be the problem of TensorFlow 0.11.0rc0 with CUDA 8.0.

Supporting CUDA 8.0 is released by v0.11.0rc1. I test with the latest release version and everything works with CUDA 8.0 and Nvidia M40.
Thanks for the contribution 😃

This seems like a lot of work for maintainers to reproduce, it would be
productive to isolate the issue further -- ie, finding which ops/conditions
give an unexpected result
On Fri, Oct 21, 2016 at 4:30 AM, tobe<EMAIL_ADDRESS>wrote:

We have implemented tested the distributed TensorFlow applications. It
works in CPU environment and the GPU environment with CUDA 7.5 and Tesla
K40. But it fails with CUDA 8.0 and Tesla M40.
[image: screen shot 2016-10-21 at 19 10 37]
https://cloud.githubusercontent.com/assets/2715000/19597024/9882b2de-97c4-11e6-930a-3c373524d21b.png
[image: screen shot 2016-10-21 at 19 11 16]
https://cloud.githubusercontent.com/assets/2715000/19597035/ab8502ec-97c4-11e6-9028-80d10347dbf2.png
Not sure if it's related to CUDA 8.0. The test code is here
https://github.com/tobegit3hub/distributed_tensorflow and 100%
re-produce if I run it with CUDA 8.0 and Tesla M40.
Environment info
Operating System: CentOS 7.0
CUDA and cuDNN: CUDA 8.0 and cuDNN v5
TensorFlow version: 0.11.0rc0
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/5113, or mute the thread
https://github.com/notifications/unsubscribe-auth/AABaHGOvzEjkp4_ucNaS3LXT-slfLNelks5q2KJmgaJpZM4KdIrA
.


Swagger generation is incomplete when simple content has

Description:
Some model definitions are missed when simple content has "." in its name.
For example:
<xsd:complexType name="test">
        <xsd:simpleContent>
          <xsd:extension base="test.Content">
            <xsd:attribute name="code" type="code"/>
          </xsd:extension>
        </xsd:simpleContent>
</xsd:complexType>

Steps to reproduce:
Affected Product Version:
2.6.0, 3.1.0
Suggested Labels:

Suggested Assignees:

Have a "." character in an element name is not a best practice. But it is better to improve this to generate API even though "." is defined.

This issue is only reproducible when the simple content extension name contains ".". For other complex types it is working fine.


pkg/template_binding_test started failing on ie9 checked with revision 30299

I have compared dart2js output of r30299 and r30298 and both are the same.
log: http://build.chromium.org/p/client.dart/builders/dart2js-ie9-win7-all-5-6-be/builds/888/steps/dart2js-ie9 tests --checked/logs/stdio

There is no more template_binding_test.


feat(data): Import Sarabloh Granth

#Minor
Sanitization:

[x] In-place bindi ˆ vs stand-alone bindi N

[x] yˆ to yN
[x] Yˆ to YN
[x] oˆ to oN
[x] Oˆ to ON
[x] Iˆ to IN
[x] auˆ to auN

[x] Confirm extended accents ü/¨

[x] Ru to Rü
[x] RU to R¨
[x] HU to §

[x] Convert wN to W
[x] Check ~ vs ` (Adhak)
[x] Check nUM vs nUµ
[x] Check R vs ®
[x] Check ´ vs Ï (Pair yayya)
[x] Check N vs ˆ
[x] Check for any whitespace before vishraams
[x] Check for apostrophes/hyphens/pair-bindi akhar in gurmukhi and act accordingly (have a discussion if unsure)

I approve these assuming we're going to be proof reading and fixing them later as well. There are already some discrepancies with gurmukhi and english translation.


Assignments to superglobals are analyzed like creating local variables in scope

This is low priority because assigning to superglobals is extremely uncommon.
Noticed this while working on test cases for https://github.com/etsy/phan/pull/393
<?php
function f1() {
    $_POST = 42;
    return intdiv($_POST, 2);
}

function f2() {
    $a = intdiv($_POST, 2);  // only 1 issue is emitted here
    $b = count($_POST);
    return $a + $b;
}
f1();
f2();

Before #393, this would emit no issues.
After #393, this would emit only 1 issue
PhanTypeMismatchArgumentInternal Argument 1 (numerator) is string[]|string[][] but \intdiv() takes int
However, I think phan should also emit some sort of issue for the line $_POST = 42;, and ignore that line for analysis of uses of $_POST.
(General case: assigning incompatible type to a global or a field of a global or a field of $GLOBALS)

Hi @TysonAndre.
Types assigned to superglobals are meaningful in small snippets, but IMHO will be noise in any real working codebase. Most codebases are going to have tons of entry point that run in isolation and as such will have sloppy assignments to superglobals that will collide in very confusing ways when the codebase is analyzed as a whole.
My guess is that a fix for this would be worse than leaving it as it is, despite a fix being possibly more correct.
I'm going to close this issue as will-not-fix. Feel free to disagree if you think thats a bad idea.


If no primary key is set, use entire row as condition

And just add LIMIT 1

Almost have this working, but LIMIT 1 doesn't seem to be applying to the UPDATE statement..asking in gitter

Turns out this isn't possible in postgres:
https://www.postgresql.org/docs/current/static/sql-update.html
So dataface will require that tables have primary keys.


Small issue with SQL beautifier (":=" becomes ": =")

For named parameters in functions, like this:
select somefunc("param" := 'value');

in sample queries listed in "Top ** queries" we see:
select somefunc("param" : = 'value');

which is not a valid SQL for Postgres because of extra space.

Commit 095a355 replaces SQL::Beautify with v3.0 of pgFormatter::Beautify, this should fix the issue.


How to get embeddings for time series longer than 512 timesteps

I need to get embeddings for time series that are way longer than 512 timestamps.
Is it possible to do that with MOMENT ?

Hi, Thanks for your interest in MOMENT!
Depending on the length of the time series, here are some suggestions:

Down sampling longer time series -- The MOMENT research repository (https://github.com/moment-timeseries-foundation-model/moment-research) has some way to down sample time series such as sampling the rightmost, left most or a random subset of 512 time steps, cubic spline interpolation, and uniform down sampling.
Modeling using recurrence: MOMENT may be used in a sliding window fashion for long time series. A long time series can be split into disjoint chunks of 512 time steps, and embeddings from each chunk can be concatenated (or averaged) for predictions.

A combination of these tricks may work better for your use case.
This is a hard problem in general, and our team is doing research to extend the context length of MOMENT. Stay tuned for updates, and thank you so much for your interest!
Best,
Mononito

I will close this issue for now, but please don't hesitate to re-open the issue or Open another one should you have a question!


Adding ARO SRE notification banner controller

Which issue this PR addresses:
Fixes https://msazure.visualstudio.com/AzureRedHatOpenShift/_workitems/edit/9396114/
What this PR does / why we need it:
This PR allows ARO SRE to set a web console banner which asks the customer to contact Azure or Red Hat support. The wording of the message isn't set in stone yet. The code is ready for review.

Test plan for issue:
Start a dev cluster, edit the Cluster cluster object.
spec:
features:
reconcileBanner: true
banner:
content: 1
the banner in Spec is a separate struct to include additional information via specific fields if we add more messages (ticket number, desired version, deadline date etc.).
Is there any documentation that needs to be updated for this PR?
https://msazure.visualstudio.com/AzureRedHatOpenShift/_wiki/wikis/ARO.wiki/63811/Escalation-guidelines?anchor=[draft%2Fnot-on-prod-yet] - SOP proposal, I'll put it into a PowerShell script after it rolls out to production.

The wording sounds good. It's enough to capture their attention, and having a standard message makes it easy to flip on/off. The code changes are very straightforward, too. Nice job.

Changing to Ready for Review with the caveat that the text of the message may change.
For now please don't use this feature, message review is in progress in parallel.

Is the text in the banner selectable, so they can highlight and copy the resource ID?
Just wanted to check - sometimes GUI designers choose to make UI elements like that not selectable (you could get the value by inspecting, but that is not convenient).

Is the text in the banner selectable, so they can highlight and copy the resource ID?

Yes, the whole notification message is selectable in the browser.

Im tempted to change the message to suggested one and merge. We gave enough time for review...

@mjudeikis I think it is important here to have good unit test coverage. https://github.com/Azure/ARO-RP/pull/1589#discussion_r673087288


DOC: Update pull request review guidelines

Currently, there are snippets of information about how to review pull requests and use the github workflow to do so in two different pages in the SciPy docs:
- https://scipy.github.io/devdocs/dev/contributor/reviewing_prs.html -> this is an orphan doc, as you can see by the lack of left sidebar. There is also a broken link to a script-generated page that I'm not sure is still useful?
- https://scipy.github.io/devdocs/dev/core-dev/index.html#pull-request-review-workflow
I suggest updating and consolidating some of this content. One starting point may be the NumPy documentation for pull request reviews.
We could also include the saved replies as examples of how to communicate changes (this is entirely optional, and unfortunately at this point GitHub saved replied are only configurable on a per-user base, meaning every user needs to manage their own saved replies).

+1 to clean up things. For saved replies, we could use the approach of scikit-learn and have them in the doc instead. Otherwise I thought we could save replies for an org?


DCOS-13054: click container in pod takes you to container config

Clicking a container in a pod now takes you to that container's config page. This was brought up in user testing here.
After:
https://cl.ly/2O1W453N072V
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[x] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

This is great! really liked the UX.
I just left a couple comments @ahoskins

Awesome! Good idea!

Thanks for the review @weblancaster. I changed those names to match the others (onClickItem, etc), and put the noPropagate inline as suggested in the anonymous function!

retest this please


Update deseq2_clustering.R

Allow comment lines in R scripts too
What problem is the PR solving / What's new?
R scripts couldn't handle the newly allowed comment lines.

oops :angel:

instant service! wow :D


add TSC examples

Examples of how to use blocking/polling TSC for stm32f303 and stm32l073.
Example of how to use async TSC for stm32l073, inspired by https://github.com/embassy-rs/embassy/blob/main/examples/stm32u5/src/bin/tsc.rs.

I would have liked to add a corresponding async example for stm32f303, but I am having problems with registering the EXTI2_TSC interrupt with the usual bind_interrupts! macro, because I get an ldd conflict. My guess is that I get the conflict because of this line in the exti.rs. I am quite new to Rust though, so it could be something else.

thanks for the examples, looking great!

problems with registering the EXTI2_TSC interrupt with the usual bind_interrupts! macro, because I get an ldd conflict. My guess is that I get the conflict because of this line in the exti.rs. I am quite new to Rust though, so it could be something else.

oof yes. this is unfixable with the current design: embassy-stm32 assumes EXTI interrupts are used for EXTI only, so it unconditionally registers them. This means you can't also register it yourself to use it with TSC.
It should work if you disable the exti Cargo feature so it doesn't register the irq, but that's not great because then you can't use exti.
The ideal solution would be to change EXTI to use bind_interrupts!, so you can bind the same irq to both exti and tsc. That's a big refactor tho...

thank you @Dirbaio

The ideal solution would be to change EXTI to use bind_interrupts!, so you can bind the same irq to both exti and tsc. That's a big refactor tho...

Do you know if anyone is already working on a refac like that?
I am tempted to look into it. Even if I don't succeed and it becomes way over my head, -I will at least gain more embedded Rust experience... If I do look into it, though, it might be quite on and off in between other stuff I am currently working on.

no one's working on it, no.


Electron icon is incorrect on final .exe

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

VS Code Version: 1.81.1
OS Version: Windows 10 22H2

Steps to Reproduce:

Build vscode via instructions in the contributing docs (I have made a successful build many times before)
Installer and archive build OK, and they open OK, but the icon is wrong.
The icon of the installer .exe and Code - OSS.exe is the default Electron one, rather than the code.ico one.
Otherwise, the app runs OK, and the icon is correct within the UI, it is just the .exe. Taskbar icon is incorrect, and using resourcehacker, I can verify via the RC header data that the .exe still has the Electron icons.

@VSCodeTriageBot The issue remains

@sergei-dyshel I see you have an active fork. Have you had this issue?

https://github.com/microsoft/vscode-gulp-electron/issues/16


Prevent creating agreement as duplicate of one that is UnderReview

 2.0: Prevent creating agreement as duplicate of one that is UnderReview


PMM-11111 upgrade backup tests

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-upgrade-tests/detail/pmm2-upgrade-tests/23378/pipeline/
@yurkovychv @puneet0191 Test works locally - I've doublechecked. I don't know why we have disable schedule after upgrade on pipeline

UI way: https://pmm.cd.percona.com/job/pmm2-upgrade-tests/23525/
Docker way: https://pmm.cd.percona.com/job/pmm2-upgrade-tests/23524/allure/#suites/edeec0c1fcab5cabaaddd2f661aab654/35b252b5f86c19d7/


Unable to connect to DB with token, it keeps throwing Bad connection : Unknown host error

Type: Bug
Try to connect to DB with user name and password
generate the token by 'az login' and following command
Azure Data Studio version: azuredatastudio 1.47.0 (c7c2b2f21505562d21972d4c135119d00806db4f, 2023-11-07T17:17:45.669Z)
OS version: Windows_NT x64 10.0.19045
Restricted Mode: No
Preview Features: Enabled
Modes:

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz (8 x 1805)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.73GB (2.52GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

azuredatastudio-postgresql
Mic
0.6.0

Are you connecting using Connection Dialog via Microsoft Entra Id MFA Authentication or somewhere else?
You don't need to az login for connecting to database with this option:

You can clear token cache using "Azure Accounts: Clear Azure Account Token Cache" command.


Could not get 0.4.0 version

I am able to use 0.3.5, 0.3.6 versions of the library but version 0.4.0 could not be downloaded by Gradle. Need to check that it properly published.

0.4.0 hasn't been published, that was a typo.

Sorry, it has actually

Still unable to get the 0.4.0 version off gradle, is it going to be fixed soon?

The latest is 0.3.6.

The github repository says that there is an available 0.4.0 release available, I guess it should be fixed then?


fix ice gathering statechange event name

Codecov Report

Merging #114 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #114   +/-   ##
=======================================
  Coverage   99.37%   99.37%
=======================================
  Files           4        4
  Lines         963      963
=======================================
  Hits          957      957
  Misses          6        6

Impacted Files
Coverage Δ

rtcicegatherer.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77656d5...9b1e9ab. Read the comment docs.


fix: update go path for major version update

The upgrade to v2.0.0 was not done properly and now v2 is not importable by any other programs.
See similar issue here.
See Go wiki here
After merging this PR, you will need to make a patch release (v2.0.1) so that this library is importable again.
My main use-case is that I have written middleware in Go for hoverfly, and have imported the middleware request/response structures into my middleware, but now can't upgrade to v2 because of this issue.
This PR addresses https://github.com/SpectoLabs/hoverfly/issues/1060

Oh no, my bad. @joobus I'm thinking to remove the v2 release and re-do it as v1.5.0 instead, because it's still backwards compatible. What do you think?

@tommysitu I would support that.  Sticking to semantic versioning is a good idea.

thanks, I will fix the versioning issue.

this is fixed now: https://github.com/SpectoLabs/hoverfly/releases/tag/v1.5.0
sorry for the confusion.


localizedCallerName added to displayIncomingCall

Added localizedCallerName for displaying a callerName, so that handle can be used for storing caller id/number. Please, review the PR.
Thanks!

My bad😬 Just fixed it!

Thanks! react-native-callkit 1.3.2 released.

@aarkalyk Can you please add localizedCallerName support to startCall method too?

@sujameslin Sure 👌

@aarkalyk Did you manage to add localizedCallerName to startCall method?

@sujameslin turns out there's no localizedCallerName in CXStartCallAction class. There's contactIdentifier instead, which seems like to be used for same purposes. Here's what apple's docs say about it:

The identifier is displayed in the native call UI, and is typically the name of the call recipient.


Doesn't support icons

I have installed a package to show icons. But now it is not showing.

Which icon package are you using? I am using "A File Icon" and it works just fine for me:

Yes, I am using A File Icon only.
Have you done any kind of special modifications in your settings?

I had to uninstall A File Icon and install it again after installing this theme. :) It works now.

I will give it a shot and get back to you.

proof:

Yup. It works. Thanks

A did what @kud said but icons wont show up

But that doesn't make any sense, Sublime and Atom packages are different.


ux: Support command docker extension update --install

Is your feature request related to a problem? Please describe.
Many times during iterative development of an extension one may manually uninstall the extension. After that the next command has to use
docker extension install ...
and the repeated:
docker extension update  ...
I keep forgetting to up arrow in the shell and change update to install from the previous repeat cycle and then next time switch back 'install' to 'update' again.
Describe the solution you'd like
Support option like
docker extension update --install  ...
Especially because the syntax is pretty much the same for install and update.
This request is somewhat analogous to the helm commands option:
`
Upgrade helm release but if absent install
helm upgrade --install ...
`
Hope this makes sense.
Describe alternatives you've considered
Well I can always have two different aliases and remember to use the correct one.
Additional context

I am using something like this for now:
docker extension $(docker extension ls | grep --silent kubernetes-dashboard ; if [  $? -eq 0 ] ; then echo 'update' ; else echo 'install' ; fi)  sandipchitale/kubernetes-dashboard:1.0.0 -f


Update testing, types, 0.4 compatibility

To turn on Coveralls, all you have to do is sign in to coveralls.io with your github credentials, click 'sync repos' if you're not seeing this one, flip the switch next to this repo to the 'on' position.
I really like how the package was organized so the definitions flowed straight from the spec, very helpful.

Thanks @garborg, this is great! (:

No problem, I'm excited about this package!

@garborg Sorry for the high churn; maybe I should have transferred ownership to JuliaGeo before registering on METADATA. You think it's good to go?

Don't be sorry -- you're getting things done. I do think it's good to go. I'll open an issue with migration todos from when I migrated Geodesy.jl.

Checklist in issue #4.


Updated ru.coffee

Coverage remained the same at 56.107% when pulling 31c6a9676952d722b78f133be3342c28ae56211b on CatSkald:ru-more-translation into d237d7e917cbe958a49218a506f83748adeff7c1 on codecombat:master.

Coverage remained the same at 56.107% when pulling 92476e0eba8c8eb49a92339ddd6225067e67acf8 on CatSkald:ru-more-translation into d237d7e917cbe958a49218a506f83748adeff7c1 on codecombat:master.

Спасибо!


Enable MapIt in staging

New MapIt is currently enabled in integration, however it requires extra
configuration to be able to work in staging.

:+1: Merge when ready.


Fix spelling of verification in loadingVerificationStatus

fixed typo for "verification"
Description
Just a minor typo fix for the word verification in displayed text (line 63).
Tested
Double-checked the spelling of verification on Google.

Thanks for the fix, @Felicious !


Write something more in this Project.

Write something more.

Changed the abc.py file.

good work!!


fix multi-arg docker push

since buildx does not support our workflow we rebuild and push using buildx
see https://github.com/docker/buildx/issues/733

currently the multi-arg build no longer runs in a pr as this would complicate things even more


A sample of policy to perform a switch from a sign up flow to a sign in flow.

The page mentions possible scenario “policy is able to perform a switch from a sign up flow to a sign in flow for that user.”
Does  a sample of such policy exist?
if yes, it will be good to provide a link to such policy.
If not, can such example  be created?

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

ID: fa334514-803c-a2dc-3a05-af10070c46d4
Version Independent ID: aa2ebc9b-bb2a-1fae-1f24-5494acc07005
Content: Sub journeys in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/subjourneys.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@MNF
Thanks for your feedback! We will investigate and update as appropriate.

@MNF
We'll proceed to close this issue. However, if you find our response unsatisfactory, or if you have additional questions, please feel free to contact us again. #please-close.


Error running any commands: no such file or directory './drivers'

Here is a sample (import) command that fails on master:
node --nouse-idle-notification \
     --expose-gc /tmp/node_modules/elasticsearch-exporter/exporter.js \
     --source file \
     --sourceFile /tmp/api-es \
     --targetHost localhost \
     --targetIndex seatgeek-1 \
     --max-old-space-size=700

stacktrace
Elasticsearch Exporter - Version 2.0.0
Caught exception in Main process: Error: ENOENT, no such file or directory './drivers'
Error: ENOENT, no such file or directory './drivers'
    at Object.fs.readdirSync (fs.js:666:18)
    at Object.exports.find (/tmp/node_modules/elasticsearch-exporter/drivers.js:130:20)
    at /tmp/node_modules/elasticsearch-exporter/options.js:196:17
    at /tmp/node_modules/elasticsearch-exporter/node_modules/async/lib/async.js:125:13
    at Array.forEach (native)
    at _each (/tmp/node_modules/elasticsearch-exporter/node_modules/async/lib/async.js:46:24)
    at Object.async.each (/tmp/node_modules/elasticsearch-exporter/node_modules/async/lib/async.js:124:9)
    at Object.exports.read (/tmp/node_modules/elasticsearch-exporter/options.js:195:11)
    at Array.exports.readOptions [as 0] (/tmp/node_modules/elasticsearch-exporter/exporter.js:70:13)
    at /tmp/node_modules/elasticsearch-exporter/node_modules/async/lib/async.js:487:38

This used to work (as of march) but doesn't anymore. Using the latest stable release results in a different bug - Caught exception in Main process: SyntaxError: Unexpected token u - that I think was since fixed but since I can't get npm to install a specific tarball (grr), I'm stuck with master or stable :(

have you tried running the script in the exporter directory?
Otherwise there's also an option to specify the drivers directory in which to look for.
In any case this can be made more robust and path independent so thank you for reporting.

Sorry, how was this fixed?

commit 931d155 should fix the problem.


Dialogs do not appear when editor is inside a React MUI Dialog component

I’m having some issues though with the display of dialogs when my MDXEditor is already inside a React  component.
In the example shown below, when the user clicks the toolbar 'Link' button, the create link dialog is not shown, or is perhaps shown underneath the existing dialog.

import {  Dialog,  DialogContent,  DialogTitle } from "@mui/material";

      <Dialog>
        <DialogTitle>My Dialog</DialogTitle>
        <DialogContent
            <MDXEditor
            plugins={[
                linkPlugin(),
                linkDialogPlugin({linkAutocompleteSuggestions: []}),

                toolbarPlugin({
                toolbarContents: () => (
                    <>
                    {" "}
                    <CreateLink />
                    </>
                )
                })
            ]}
            />

        </DialogContent>
      </Dialog>

Most likely a matter of zIndexing. The editor appends an element to the body and uses it as a container for all dialogs. Many of them include forms, and if they are kept within the editor, the dialog buttons are "caught" by form management tools like Formik for example.
I will take a look and see what kid of workaround is needed. Most likely you will have to zIndex it.

Thanks - for anybody who needs it, here's the CSS workaround until your investigation is complete:
div[data-radix-popper-content-wrapper] {
    z-index: 2000 !important;
}

(FYI: MUI seems to give its dialogs a z-index of 1300)

Yep, you can create a new stacking context in the editor like so:
.editorRoot {
  /* ... */
  position: relative;
  z-index: 202;
}

This essentially gives you a baseline z-index of 202. All z-index: auto components (the link dialog, paragraph dropdown, etc.) will render on top of it. So you don't need to target individual components.
It would of course be nice to have support for this by default, although you'd have to come up with an elegant way to make the z-index customizable.

Adding global styles for an attribute known to be on the wrapper seems preferable to creating a new stacking context for the editor. Setting the z-index for the editor itself is likely to put it in the wrong stacking order with respect to the UI elements that one might want the link dialog to display over, introducing more stacking problems.
I think it would be beneficial for the linkDialogPlugin to take options for the display of the dialog, such as a classname to add to the popper content wrapper.

Update, there's a CSS class added especially for that purpose: mdxeditor-popup-container.
Something like:
.mdxeditor-popup-container {
z-index: 9999;
}

should solve the issue.


Nuxt build fail rc4

When running run build it fails with this error:
ERROR in vendor.51c885f8f18426da3f13.js from UglifyJs
Unexpected token: operator (>) [./node_modules/vue-video-player/ssr.js:105,0]
[vendor.51c885f8f18426da3f13.js:37548,37]

same problem

The v4.0.2 fix it. thx!


Using Hide Middlescroll Opponent Notes disables the opponent notes when not using middlescroll.

Describe your bug here.
Nothing else needs to be said, I guess.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No, this is the base OS Engine.

If anything this is a good thing. Might be good to rename the option to "Hide Opponent Notes" though.


[Catalog] Unresponsive Modal

Currently Behavior:
Our modals for showing more info about catalog item is unresponsive.

Expected Behavior:
Supports responsiveness for these catalog modals and improves UX
Contributor Resources
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
See Meshery site designs in this Figma designfile. Fill-in the Layer5 Community Member Form and join the Layer5 Community Slack for access.

Hey @theBeginner86 can i work on this but I may require some help as I am new to Meshery.

hello @theBeginner86, I want to work on the issue. Will you assign me.

@Yana-Gupta @EraKin575, thanks for volunteering, this issue will be taken care of by @Ghat0tkach, and we'll help you get assigned other potential bug fixes or enhancement tasks.

Thank you @thisiskaransgit  , I'll fix this together with the catalog design box

closed as completed via https://github.com/meshery/meshery.io/pull/1360


CAS004

Checks to see if the RC4 cipher is enabled (should be disabled)

This looks nearly identical to CAS003

Wrong test got copied up.  Will fix this now.

Code is fixed now.

Line 73 has a typo:
$succes = $false

Also, emailed you some notes from CAS003 that will be relevant to this test as well, since they share similar code and logic.

Fixed the typo at line #73.

Closing stale PRs. Resubmit after testing if still valid.


i donr like grandle .always error

Please .i want .jar library

I'd suggest you learn gradle or maven.


Module imported with use at function scope not available to nested try block

This is the test case:
fn wtf {
    use re
    try {
        e:false
    } except e {
        try {
            local:matches = (re:find ' exited with (\d+)''' (echo $e))
            print $matches[groups][1][text]
        } except e2 {
            put 're:find failed' $e2
        }
    }
}

When I run that function it throws ?(fail 'search re:find: not found'). I have to move the use to within the innermost except { } block for it to work. That seems wrong. It doesn't matter if I use re at the top-most scope. Shouldn't it have walked up the enclosing scopes till it found that the module had been imported and only fail in that manner if it hadn't been imported?

I feel like this is expected behavior and I have misunderstood something fundamental about how module references are resolved since this simpler example fails in the same way:
fn wtf {
    use re
    { re.find 'hello' 'abc hello def' }
}

But it is still counterintuitive is not wrong.

Thanks for reporting! What you described is a bug. Module imports should be lexically scoped, like variables. (Lexical scoping of module imports was only introduced recently, it seems I should introduce some tests for that.)

Interestingly, this seems to only affect function calls.


Using Hotstar without valid login (Not for VIP or Premium content)

You will able to use this app without valid login details. First you have to click "Login Free" from the setting before entering the addon. Once done, play any video. First play gives an issue (Need fixing). Just return from playing and start playing another video.There after it work smoothly.

Also you have to handle logged in error explicitly here. Because isLoggedIn function will allow to play but if user is not logged in then hotstar will give error with some response code and on that particular response we can prompt user for login

I tried this method. This way we can skip additional tab in the setting but it gives an error in the first playback. Then again additional changes in code is required to tackle this error. I have added self._refreshToken() in the free login code. now everything is working fine.

What if the stream is not for guest and requires login.

if stream is not for the guest then it will raise subscription error. In that case, user will have to logout and do valid login using Login option.


Selection tools (incl. Parse) should gradually light up whatever is being selected.

A problem: how does this interact with the "real" selection? Maybe we don't want to be selecting things along the way in the same way that they are "really" selected.
What if the Views (both CropObjectView and for EdgeView) implement some highlight, which gets called separately from select(), and the selection tools just interact with the objects in this way?
It is problematic to guarantee un-highlighting (e.g. on interruption of the tool flow). Easiest way: implement this inside the tool.

Implement this as ActiveSelectionBevaior?

Work on this: handled by a "slow mode" in the active selection mechanism, which only does its thing once every X on_touch_move events. The X is computed based on how long the selection processing takes, so that the amortized time per on_motion_touch event is less than some threshold (set by BaseListItemViewsOperationTool._active_selection_target_time_per_event property).


How long of a video clip ?

Length per gb ?

It is nonlinear with a large constant (from model loading I assume). Some data points reported by torch.cuda.max_memory_allocated():
8s: 5.89G
16s: 6.66G
32s: 7.19G
64s: 8.23G
But I don't recommend running on videos much longer than the training length (8s). The quality will degrade.


chore(master): Release 0.8.3

:robot: I have created a release beep boop
aztec-packages: 0.8.3
0.8.3 (2023-10-04)
Bug Fixes

Do not depend on npx for check rebuild script (#2681) (20ffbbc)
Remove package json properties whitelist (#2680) (ef499a0)

Miscellaneous

Update authwit computation (#2651) (fdbe2b2), closes #2448

barretenberg.js: 0.8.3
0.8.3 (2023-10-04)
Miscellaneous

barretenberg.js: Synchronize aztec-packages versions

barretenberg: 0.8.3
0.8.3 (2023-10-04)
Miscellaneous

barretenberg: Synchronize aztec-packages versions

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/aztec-packages-v0.8.3 :sunflower:

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/barretenberg.js-v0.8.3 :sunflower:

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/barretenberg-v0.8.3 :sunflower:


Now handling targetType of Version in TryConvertInternal

Added handling of Version type in TryConvertInternal.

Done here https://github.com/JamesNK/Newtonsoft.Json/commit/57f4f9816204230bdb2fc356c497f993bf25d86f