add webpack to documentation

Summary
Add a guide to the official docs on how to integrate Bootstrap with Webpack.
Motivation

Webpack's popularity is growing, but there's a lack of reliable up-to-date resources on how to integrate it with Bootstrap
Following @mdo question in #22196

Background
During the past few months i've been integrating bootstrap into several new and existing projects using webpack version 2. This led me to conduct a thorough research and tests on the best practices to achieve this integration. The results of these efforts are in this guide.
Technical
Check out a sample project which implements the steps detailed in this guide -
https://github.com/IdanCo/webpack-modular/tree/bootstrap4

@ajacksified , i've explicitly mentioned webpack v2 and put more emphasize on custom variables (after all, why even use webpack if you're not planning on customizing?)

Sorry! Must have missed it.
On Apr 23, 2017 2:46 PM, "Idan Cohen"<EMAIL_ADDRESS>wrote:

@ajacksified https://github.com/ajacksified , i've explicitly mentioned
webpack v2 and put more emphasize on custom variables (after all, why even
use webpack if you're not planning on customizing?)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/22423#issuecomment-296490995, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAKtm40LIYB96Dt3nUA_35SSKxUPnROkks5ry8avgaJpZM4M7r3k
.

Disregard that, misread the notification email. Looks good to me!

I would suggest using ExtractTextWebpackPlugin in the example, rather than style-loader. With style-loader, the styles are injected into <style> tags in the html head. With ExtractTextWebpackPlugin, the styles are extracted into a separate css file to be <link>ed in.
The downside is the slightly added complexity, but the upside is less "magic" around importing styles from JS.

@samsch , I agree that using ExtractText in many cases would be the best practice. It's actually used it in the demo project I attached to this PR - https://github.com/IdanCo/webpack-modular/tree/bootstrap4
But I'm not convinced it should be a part of the documentation. There are other best-practices (such as using source maps), but adding them to the documentation might imply that they are essential for compiling Bootstrap, even though they are in no way such. Also, i can think of many use cases in which using ExtractText plugin would be counter productive, such as serving dev server or running tests.
I hope other resources (blogs, stackoverflow, etc.) will expand and elaborate this basic setup. I know I plan to...

Just a quick counter argument. I'm actually less concerned with ExtractText
being a best practice, and more interested in having a separate css file to
be more similar to environments which users may be used to.
I do see your point about it seeming to be "required" when it is not.
Perhaps that could be satisfied with an additional sentence or two in the
guide noting that it's optional.
On Apr 24, 2017 4:44 PM, "Idan Cohen"<EMAIL_ADDRESS>wrote:
@samsch https://github.com/samsch , I agree that using ExtractText in
many cases would be the best practice. It's actually used it in the demo
project I attached to this PR - https://github.com/IdanCo/
webpack-modular/tree/bootstrap4
But I'm not convinced it should be a part of the documentation. There are
other best-practices (such as using source maps), but adding them to the
documentation might imply that they are essential for compiling
Bootstrap, even though they are in no way such. Also, i can think of many
use cases in which using ExtractText plugin would be counter productive,
such as serving dev server or running tests.
I hope other resources (blogs, stackoverflow, etc.) will expand and
elaborate this basic setup. I know I plan to...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/22423#issuecomment-296815953, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJ9AR1QbrxcNvq35zpZn8MmYOoYQ5P95ks5rzQmfgaJpZM4M7r3k
.

Any other feedback before we merge this? (Also, hi @ajacksified! Long time no see! :D)

My only concern about this PR is, is it the role of BS to show how to integrate BS in any plugins, module bundler etc... ?
I know Webpack have a huge community and I love this tool but if we do that we'll have to do for others plugins too so we should create a dedicated page for that on our documentation

My only concern about this PR is, is it the role of BS to show how to integrate BS in any plugins, module bundler etc... ?

I think so, yes. I'd love more in-depth and helpful guides for using, extending, and contributing to Bootstrap—regardless of the package manager or community. Bootstrap is meant to be accessible to everyone, beginner or seasoned veteran, so I'm a fan :).

@mdo totally agree.
@Johann-S i see your point - Bootstrap can't be held responsible for what other bundlers are doing. But on the other hand, the bundlers also can't commit to how Bootstrap is structured (especially while in alfa). So the outcome is that integrations fall "between the chairs". So taking responsibility for documenting may not be the obvious choice, but it will help bring BS to more people, and i think that's worth it.
Regarding which bundlers should be included, IMO as long as there's a demand for integration documentation, it's worth considering adding.

That's ok for me 👍 it's just a choice to do

We are integrating Bootstrap into our web app with Create React App

I think we can add this page on : Write a Learn & Extend or Approach section.
Related to #17021 (docs section)

@IdanCo can you update your branch ? because now we use Popper instead of Tether

Done! and congrats on the huge Popper PR @Johann-S , can't wait to give it a try!

Thank you very much @IdanCo ❤

LGTM 👍
The merge is up to @mdo because he said :

Looking at this, the only question I have is around the placement of this on a separate page—need to think about that a bit more.

Changed the webpack configuration to accommodate popper instead of tether.
@Johann-S , any reason why bootstrap.js in v4-dev is not yet updated and still dependent on thether? I built it manually to do my tests and find the correct recipe, but I'm curious about the repo build process...
BTW, at first glance the new tooltips/popovers/dropdown looks great!

Looking at this, the only question I have is around the placement of this on a separate page—need to think about that a bit more.

no problem, let me know if there's anything i can do to help

Yes we build manualy dist files, currently they are outdated but we will update them soon 👍

Thanks y'all!

@mdo I'd like to point out an issue of compiling the JS with Bootstrap's individual /dist/ files (as hinted here).
I tested @IdanCo's repo and it looks fine when using the bootstrap.js scripts bundle.
However, if you include the /dist/ files individually (and necessarily reference them with ProvidePlugin), the code is duplicated in the resulting JS file (only referencing them in ProvidePlugin and not importing them doesn't work).
The solution for individual imports is, as suggested in the first link, to include the individual /src/ files instead and use a transpiler (Babel etc.).

Thanks @petrpacas , I'll have a look at it this weekend

Hi @petrpacas. Personally, I don't use the Provide Plugin but the Expose Loader instead (see my comment here)
Maybe you could adapt your import with this loader and see what happen ?

For those interested, here's my repo with working example of using the individual source files w/ Babel instead:
https://github.com/petrpacas/webpack-bootstrap-4-setup

Hi.
I followed the guide and for some reason it kept telling me that Tether was needed.  I had to replace Popper with Tether.
I've installed Bootstrap with npm install --save<EMAIL_ADDRESS>and in my package.json the dependency is "bootstrap": "^4.0.0-alpha.6",.
How can I get the dependency to Popper as explained in the documentation?
Thanks.

@ccalvarez, These instructions refer to the develop branch which replaced Tether with Popper.
For version Alpha 6 (which you probably use), You can browse the previous commit, or have a look in the issue discussion.

@IdanCo I understand, thank you very much!

Although the documentation is very good, I still found it incomplete regarding the actual NPM packages that need to be installed and interop with other loader(s) who generate source-maps. Here is a Gist that brings it together:
https://gist.github.com/chrisalexander55/81f3d5db057fbcbd74d62d488adbca8a
Please feel free to use this to update the docs page.


Add File Based support for C

Platform: linux
---How can I add support for C?

We'd need a way to pass in a file and run it for c. Have you tried clang?

Duplicate of #454


common : factorize IPv4/v6 normalize IP/Address

Signed-off-by: Nicolas PLANEL<EMAIL_ADDRESS>

run skydive-scale-tests


Cancellation for YQL queries

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

Build fails with error:
In file included from $(SOURCE_ROOT)/yt/yql/agent/yql_service.cpp:1:
In file included from $(SOURCE_ROOT)/yt/yql/agent/yql_service.h:3:
In file included from $(SOURCE_ROOT)/yt/yql/agent/private.h:3:
In file included from $(SOURCE_ROOT)/yt/yt/core/logging/log.h:8:
In file included from $(SOURCE_ROOT)/yt/yt/core/logging/log-inl.h:8:
In file included from $(SOURCE_ROOT)/yt/yt/core/misc/error.h:9:
In file included from $(SOURCE_ROOT)/yt/yt/core/ytree/public.h:7:
In file included from $(SOURCE_ROOT)/yt/yt/core/rpc/public.h:3:
$(SOURCE_ROOT)/yt/yt/core/actions/callback.h:163:9: error: no matching function for call to object of type 'TCallback<void ()>'
        state->Callback(std::forward<TArgs2>(args)...);
        ^~~~~~~~~~~~~~~
$(SOURCE_ROOT)/yt/yt/core/actions/callback.h:254:26: note: in instantiation of member function 'NYT::TCallableBindState<void (), void (const NYT::TErrorOr<void> &)>::Run' requested here
            &TBindState::Run);
                         ^
$(SOURCE_ROOT)/yt/yt/core/actions/callback.h:211:16: note: in instantiation of function template specialization 'NYT::TCallback<void ()>::CastImpl<void, const NYT::TErrorOr<void> &>' requested here
        return CastImpl<R, TArgs2...>();
               ^
$(SOURCE_ROOT)/yt/yql/agent/yql_service.cpp:52:36: note: in instantiation of function template specialization 'NYT::TCallback<void ()>::operator TCallback<const NYT::TErrorOr<void> &>' requested here
        context->SubscribeCanceled(BIND([=, this, this_ = MakeStrong(this)] {
                                   ^
$(SOURCE_ROOT)/yt/yt/core/actions/bind.h:235:19: note: expanded from macro 'BIND'
#define BIND(...) BIND_IMPL(true, __VA_ARGS__)
                  ^
$(SOURCE_ROOT)/yt/yt/core/actions/bind.h:230:39: note: expanded from macro 'BIND_IMPL'
    #define BIND_IMPL(propagate, ...) ::NYT::Bind<propagate, ::NYT::TCurrentTranslationUnitTag, __COUNTER__>(FROM_HERE, __VA_ARGS__)
                                      ^
$(SOURCE_ROOT)/yt/yt/core/actions/callback.h:231:7: note: candidate function not viable: requires 0 arguments, but 1 was provided
    R operator()(TArgs... args) const
      ^
1 error generated.

Oops, fixed


Changed $exitCodes to a reference

Without this it always says Command was executed on zero tenants.

What if we actually showed on how many tenants the command was run? Maybe even a progress bar while running?

I like the progress bar idea.  Shouldn't be that difficult.

We could do $query->count() before chunking for the progress bar, but that would result in additional query. Is there a way to do it with just a single query?

@Plytas good point, can you create an issue or PR for the count?
Please note I removed this from the original PR of the run command to lower the overhead. If there's a decent way to get it done nevertheless I'd love to hear so.


Change z-index to public

I'm not sure why, but the current version of Engo has a zIndex as private value.
The only way to set ZIndex is by using SetZIndex, at the current version. That makes it impossible to set the value in advanced (at the &RenderComponent{}). In my case doesn't make any sense to set the ZIndex using the SetZIndex, since some elements have always the same ZIndex (like: background is 0 and player is 1).
I'm not aware of any issue of that change, but I don't know what wasn't public before.

It does this because it has to set rs.sortingNeeded so that the system can sort the textures properly. If it's public and you set it yourself, it may not actually do anything, since the textures still draw in the same order every pass.
I do see what you mean, though. I think maybe there can be another public variable, such as StartZIndex, that sets the starting Z-Index. Then near the top of the render system's Add function, you just call SetZIndex(render.StartZIndex). Would that solve your problem?

Yes, I don't thought that someone will set something.ZIndex = 3 after already use the render.Add(...). 😅
The StartZIndex is better.

I think it's fine now, the rendering is working as expected. I didn't test on GopherJS. I'm going to sleep now. 😅

Looks good, thank you for your contribution!


FEAT Add option to disable timeout

An option to disable the timeout would be great (--timeout -1 for instance).

Adding and option --no-timeout would be nice indeed.


Not running with Xcode 9 and Swift 4

Using Xcode 9, the compile failed with
Command failed due to signal: Segmentation fault: 11

Thanks for reporting this! I'd been meaning to check, but hadn't gotten around to it.
All fixed up by #30!


Can we change code distributed for angular-winjs as it is under MIT license?

We have an application that uses AngularJS 1.4 and we are trying to upgrade it to 1.5. This application uses library angular-winjs which had its latest release on Oct 2016 and doesn't support 1.5(because of the issue)
Is it okay to alter the code of the library angular-winjs as it is licensed under the MIT license?

At this point, I’d suggest a complete rewrite of the front end in VueJS.
On Wed, Nov 10, 2021 at 12:23 AM shrhegde @.***> wrote:

We have an application that uses AngularJS 1.4 and we are trying to
upgrade it to 1.5. This application uses library angular-winjs which had
its latest release on Oct 2016 and doesn't support 1.5(because of the
issue https://github.com/winjs/angular-winjs/issues/88)
Is it okay to alter the code of the library angular-winjs as it is
licensed under the MIT license?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/winjs/angular-winjs/issues/108, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHVQHEPH7WWYCJZOJ2KFC3ULIFWFANCNFSM5HXCGGNQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

The application is in sustenance mode so we don't want to move to a new framework.


fix: allow object options to share the same label

Closes #216
Keys can be objects, though somewhat discouraged.

The key can be any object, but strings and numbers are recommended since they allow identity to persist when the objects themselves change.

I thought keys have to be primitive types?

Ah, I was wrong. From the Svelte docs:

You can use any object as the key, as Svelte uses a Map internally — in other words you could do (thing) instead of (thing.id). Using a string or number is generally safer, however, since it means identity persists without referential equality, for example when updating with fresh data from an API server.

Maybe we should still default to the option label as key but allow overriding this behavior by exposing a new prop get_key: (option: Option) => unknown = get_label?

That's a an option too, but having two different values with the same name would still break svelte-multiselect

That's true but that was by design (until now anyway). Maybe it's not a good default. I think I made that decision at the time because I couldn't think of a strong use case for duplicate options. That was before allowing user-created options. But even now, maybe it's better to show a warning to users rather than allowing them to make duplicate options?

That's right, shall I update this PR with a new get_key option? Or are you ok with option being used as an index?
The only thing that might break with the current PR is animate:flip when selected is updated by copy and not reference
https://svelte.dev/repl/60e7b5893b304b8f9e0ea094fe6bb293?version=3.57.0

The only thing that might break with the current PR is animate:flip when selected is updated by copy and not reference

I didn't realize the downside was this small. Nice REPL btw. Very educational.

That's right, shall I update this PR with a new get_key option?

No, I'm sold. I think we should just add a test that covers selecting two identical object options.

Thanks for the quick merge!


preview in browser

Is it possible to preview the html version of the markdown that I'm currently editing in a browser using your plugin?  If not, would you like to include that (feature request) or are there other vim plugins that you could recommend for that (which do not conflict with vim-markdown)?

Note: I currently use iamcco/markdown-preview.vim which does not seem to conflict with vim-markdown...


drivers/video: Support spot position setting

Summary
Add new control id V4L2_CID_EXPOSURE_METERING_SPOT_POSITION to support spot position setting in spot exposure metering.
Impact
video driver.
Testing
Tested with spresense

@SPRESENSE  please rebase your change.

@pkarashchenko @xiaoxiang781216 Thanks for your comments. I updated this PR as suggested.

@SPRESENSE does the camera support work with NuttX mainline? BTW, I think there is not official documentation to Sony Spresense's chip in our mainline: https://nuttx.apache.org/docs/latest/platforms/arm/index.html could you please submit it?

@acassis
Yes, the camera works with NuttX mainline.
Thank you for your suggestion. I will try to submit a separate pull request for documentation.


Fix producer ConnectionFactory

The RabbitMessageChannelBinder creates an internal distinct
ConnectionFactory instance based on the RabbitProperties for the
non-transactional producers to avoid dead locks when connection
is blocked.
In case of Cloud Connectors the ConnectionFactory bean is overridden,
but not RabbitProperties.
Therefore the original ConnectionFactory for consumers is good,
cloud-based, but for producers it is still based on the RabbitProperties
from Spring Boot, in most cases with default options.

Add one more producerConnectionFactory @Bean to the
CloudConnectors configuration to obtain a distinct ConnectionFactory
instance from the cloud provider
When we ara not in the cloud profile, create a fresh ConnectionFactory
based on the RabbitProperties
Inject the extra producerConnectionFactory instance into the
RabbitMessageChannelBinder instead of the internal non-stable solution

Cherry-pick to 1.3.x
Conflicts:
spring-cloud-stream-binder-rabbit/src/main/java/org/springframework/cloud/stream/binder/rabbit/RabbitMessageChannelBinder.java
spring-cloud-stream-binder-rabbit/src/main/java/org/springframework/cloud/stream/binder/rabbit/config/RabbitMessageChannelBinderConfiguration.java
spring-cloud-stream-binder-rabbit/src/test/java/org/springframework/cloud/stream/binder/rabbit/RabbitTestBinder.java
spring-cloud-stream-binder-rabbit/src/test/java/org/springframework/cloud/stream/binder/rabbit/integration/RabbitBinderModuleTests.java

Merged as https://github.com/spring-cloud/spring-cloud-stream-binder-rabbit/commit/11adfa80ed17c4d2c63ab763284400fbfcc44657 by @olegz


feat: orElse callback when retries is exhausted

reference by https://github.com/google/dart-neats/pull/167

I could use this as:
final msg = await retry(
  getMessage,
  retryIf: (e) => e is TimeoutException,
  orElse: (e) => 'failed to get data',
);

But I could also do:
final msg = await retry(
  getMessage,
  retryIf: (e) => e is TimeoutException,
).catchError((e) => 'failed to get data');

Maybe, instead of creating a new orElse parameter, we should improve:

examples,
tests, and,
API documentation,
to clearly help users discover that you can elegantly combine this with .catchError().
If you want to produce a value, when retry fails.

I think a section in the README.md would be fantastic.
Maybe, even a hint in the documentation for the retry() function would be fine. Perhaps something like:
/// Call [fn] retrying so long as [retryIf] return `true` for the exception
/// thrown, up-to [maxAttempts] times.
///
/// Defaults to 8 attempts, sleeping as following after 1st, 2nd, 3rd, ...,
/// 7th attempt:
///  1. 400 ms +/- 25%
///  2. 800 ms +/- 25%
///  3. 1600 ms +/- 25%
///  4. 3200 ms +/- 25%
///  5. 6400 ms +/- 25%
///  6. 12800 ms +/- 25%
///  7. 25600 ms +/- 25%
///
/// ```dart
/// final response = await retry(
///   // Make a GET request
///   () => http.get('https://google.com').timeout(Duration(seconds: 5)),
///   // Retry on SocketException or TimeoutException
///   retryIf: (e) => e is SocketException || e is TimeoutException,
/// );
/// print(response.body);
/// ```
///
/// If no [retryIf] function is given this will retry any for any [Exception]
/// thrown. To retry on an [Error], the error must be caught and _rethrown_
/// as an [Exception].
///
/// To always produce a value, even when retries are failed or an exception not
/// matched by [retryIf] is encountered, combine [retry] with [Future.catchError].
/// ```dart
/// final body = await retry(
///   () => http.read('https://google.com').timeout(Duration(seconds: 5)),
///   retryIf: (e) => e is SocketException || e is TimeoutException,
/// ).catchError((e) => 'failed to get data');
/// print(body);
/// ```


Error: Failed to parse declaration "flex: 0 1"

The shorthand property
flex: 0 1;

Should be parsed to
flex-grow: 0;
flex-shrink: 1;

reproduction on the node repl:
require('css-to-react-native').default([['flex', '0 1']])

Thanks for finding this! It turned out to be zeros showing up as falsy values—and would have actually effected a lot more shorthands!


finish Restore Home page to design as in Figma

What I did:

Resolve padding around top yellow header. Needs more room above it
Restore proper spacing between buttons Tag Your Projects and Learn More
Ensure white and blue bg colors extends full width on wide desktops

Generally, we prefer Material-UI components to HTML elements. And we always want to reference the MUI theme, when possible.
So instead of h1 tags, please use Typography tags with variant="h1" and instead of p tags, please use Typography tags with variant="body1". Etc.
Also, colors (hex values) are all defined in the global theme so they should not be hardcoded anywhere else, as a general rule. Theme colors can be referenced in a variety of ways, including theme.palette.x.y as you use. But also using color="textSecondary" and similar attributes on Typography components. Please see the "Guides" pages for some examples.
Small pieces of styling for a single component are fine, but when it extends to many lines, such as 35 lines in Home/index.js, that is what is called a "code smell" by some in the industry. It is an indication that we are probably not using Material-UI and our theme and our best practices.
Please be sure to read the CONTRIBUTING.m document, the UI-THEME-STYLESmd document, and also be sure to inspect the JSX code and the resulting output of the 4 guides pages. You have to type in the URL manually to link to them. Just type "/guides" after localhost 3000 or civic tech index dot org.
Thanks!

Wow , it is a really nice comments, I will work on my issue tomorrow morning, I will definitely read closely about the document. Thanks again for the comments.

Thanks again for the code review, I do read through both contributing.md and UI-THEME-STYLES.md  Hopefully, this time I can meet our project coding requirements, Feel free to give me any suggestions , In this pull request, I reorganized the UI to match up with figma design, I also find out the footer  layout have some trouble in md size. I am not sure if i am allow to modify footer file or not.  Thanks for your time.


Highlight current window

environment

vim:
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Jul 27 2016 18:56:30)
MacOS X (unix) version
Included patches: 1-1831

vim-airline: 7b9b68f

OS: OS X El Captain
if you are using terminal:

terminal: iTerm2 / Zsh

$TERM variable: screen-256

color configuration (:set t_Co?): 256

actual behavior
I am using Molokai theme and I am happy with it most time. When I have more than a few windows, it appears that the airline status bar has almost identical main color (blue) which makes it difficult to have the current window standing out.

expected behavior
I want to change the inactive window's status bar to a more darker color. Is there such a way to do so? Thanks,

By forking the airline-theme and makes changes to the inactive part I have solved the problem


dictionary arguments are turned into strings

>>> jsonpickle.loads(jsonpickle.dumps({1: 'a'}))
{'1': 'a'}

This does not seem right?
If this is an intentional or known limitation I am sorry I missed it but I do believe even in that case it should raise an error.
This is python3.7 running jsonpickle-3.0.1

Hello, and thanks for your report! While I understand this is confusing behavior, it would be a backwards-incompatible change to raise an error. Also, the output you want to achieve is already possible with keys=True, as per the note in https://jsonpickle.readthedocs.io/en/latest/api.html#jsonpickle-high-level-api.
>>> jsonpickle.loads(jsonpickle.dumps({1: 'a'}, keys=True), keys=True)
{1: 'a'}

Thank you - it is indeed explained in the documentation.


Still crash on load due to missing image file

@mpaperno  @LorenzMeier
I have a problem in anting the project jmavsim.  I also get a crash on boot because the image is not found:
Exception in thread "main" java.lang.IllegalArgumentException: input == null!
at javax.imageio.ImageIO.read(ImageIO.java:1388)
at me.drton.jmavsim.Visualizer3D$CustomCanvas3D.(Visualizer3D.java:707)
at me.drton.jmavsim.Visualizer3D.(Visualizer3D.java:114)
at me.drton.jmavsim.Simulator.(Simulator.java:209)
at me.drton.jmavsim.Simulator.main(Simulator.java:611)
I also used followed code, but it still report this errors.
java -Djava.ext.dirs= -cp lib/*:out/production/jmavsim.jar me.drton.jmavsim.Simulator
In aternate build method. the terminal report  that the error is followed:

BUILD FAILED
Target "creat_run_jar" does not exist in the project "jMAVSim".

OS:Ubuntu 14.04 64bit
FIle:/home/user

BUILD FAILED Target "creat_run_jar" does not exist in the project "jMAVSim".
                         ^

Please check your spelling.
I don't know why the images don't work when trying to start it the "old way."  It shouldlook in environment/ folder relative to the executing directory.
Someone could try to debug the line throwing the error -- perhaps the way the image is being loaded (the classLoader) is causing the problem.  There are quite a few ways to load a URL/path in Java... one of them should work (as an example, there's another way used in KinematicObject.modelFromFile()).
// load and scale compass image for overlay
Image img = ImageIO.read(getClass().getClassLoader().getResource(TEX_DIR + COMPASS_IMG));

-Max

Thank you, Max. I will try it.  I just copied the code" ant create_run_jar copy_res" to the terminal.  I am a newer of Ubuntu,since the jMAVSIm is work in WIndows.

@mpaperno Thank you very much for your work. the code is work after you fixed in my somputer

in my computer

Good to hear, and thanks for letting me know!
-Max
On Mon, Apr 04, 2016, at 04:31 AM (-0400), muyangren499<EMAIL_ADDRESS>wrote:

@mpaperno https://github.com/mpaperno Thank you very much for your work. the code is work after you fixed in my somputer
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/PX4/jMAVSim/issues/23#issuecomment-205193677

@mpaperno  Great job! I will test controller  in the next period of time


three dots are replaced by … (unique character) in commands

URL for broken page
https://git-scm.com/docs/gitrevisions#Documentation/gitrevisions.txt-Theem82308203emthree-dotSymmetricDifferenceNotation
Problem
In short code sample imported from git documentation, '...' are transformed into a single character three dots instead of the actual three separate dots that should be used for the three-dot symmetric difference notation.
As far as I can tell, it's not a bug with original documentation https://github.com/git/git/blob/master/Documentation/revisions.txt#L286 which never use this symbol (it is also an issue in some translations however) but with the way the documentation is compiled to this website.
Operating system and browser
Latest ubuntu and chrome
Steps to reproduce
open the browser and look at the command example, not three characters are replaced by a single one
Other details
I suppose the problem is in one of the tool you use to compile documentation into html. So this issue will probably simply be forwarded to another project. But I was not able to find which tool is doing the compilation in this particular case

Fix sent upstream<EMAIL_ADDRESS>

I'm going to close the issue, as I think there's nothing to be done in this repo. When the fix makes it into the next Git version, we'll pick it up on the site. Thanks for reporting!

Thanks for taking it into account. Congrats for finding how to solve it, I admit I did take a quick look and decided not to try to understand how to install and test the whole pipeline.


[core] Add support for new platform: Oculus Rift

Add support for PLATFORM_OCULUS using Oculus SDK 0.7 (DK2).
Related functions [core.c]:
static void InitDisplay(int width, int height);
static void InitGraphics(void);
void BeginDrawing(void);
void EndDrawing(void);

Process goes as follow (draft):

Initialize OVR device
Initialize OpenGL FBOs with textures for render (every eye)
On render, draw every eye independently (multiple render passes)

Note that OpenGL 3.3 es required. Probably new helper functions will be required on rlgl.c.

Work on this issue is already on the go.
I'm using Oculus SDK 0.7 and the samples included as the main reference code. Unfortunately, the samples are C++ and come with custom C++ helpers. I'm rewritting them to pure C.
Samples also mix OVR device code with OpenGL related code, I'm trying to separate it between core module and rlgl module.
Right now I'm working in local, if someone wants to help me with this, please let me know and I can create a new branch on github.

Noticed that Oculus SDK 0.8 introduced breaking changes so I decided to postpone Oculus Rift support until Oculus SDK becomes a more stable version (maybe 1.0?).
On the other side, I couldn't get an Oculus DK2 before it get sold out (no oficial distribution to Spain). I'll better wait for the device to be on a more stable (and fortunately commercial) state.

Oculus PC SDK 1.3.0 has been released. Maybe it's a good time to re-consider this feature for raylib.

Just a quick update on this issue: I'm working on it.
A first sample of Oculus SDK 1.3 integration with raylib has been uploaded here: https://github.com/raysan5/raylib/tree/develop/examples/oculus_glfw_sample
Still not working properly; Oculus Rift device is detected and initialized correctly but there seems to be some connection problem between GLFW3 window/context  and Oculus window/context attachment.
Right now looking at this sample to solve that issue: https://github.com/jherico/OculusMinimalExample
Some internal redesign will also be required on raylib drawing system to accomodate to Oculus Rift; right now, all 2D drawing is done at EndDrawing() but 3D models drawing is done by model, those draw calls will require some batching.

Finally, it works! Right now, only window/context initialization with GLFW3 and blank screen. I managed to draw a simple red rectangle with rlgl:

Now I'm working in a simple 3D scene (a cube); I'm dealing with math (using raymath instead of ovr math library) to track head position and orientation.

Still working on it but almost finished! raylib rlgl module required a deep redesign to accomodate to the stereo rendering...
Just to keep track, new drawing system stores draw calls (models-drawing) in an array and everything (independent model buffers + internal default buffers) is processed at EndDrawing().

Some recap on this issue:
Still working on rlgl module redesign, lots of changes done... and still some work required...
A working Oculus Rift sample has been developed. This sample uses rlgl module and includes a set of functions to deal with Oculus data.
Now comes the hard part, integrating this set of Oculus functions from the sample to raylib.
Like I did with raylib, my intention was separating GL related stuff (placed in rlgl module) from device initialization stuff (placed in core module) but with Oculus Rift it's not that easy because library itself (libOVR) already mixes those two parts and separating them would imply using some libOVR functions in core module and using some other libOVR functions in rlgl module.
Finally I decided to place all Oculus stuff into core module, including OpenGL related calls required by Oculus functions. Maybe in a future all Oculus stuff could be moved to rlgl module... need to think about that...
Let's see how Oculus integration works, stereo drawing could be an issue...

Recap on this issue: still working on it.
Finally, all Oculus Rift related functionality has been moved to rlgl module. After some thinking, I decided to move Oculus functions to rlgl module for two reasons:

Most of the functions listed requires OpenGL functions. As I said, all OpenGL stuff is contained inside rlgl so it makes more sense to be there. The only functionality that shouldn't be there would be Oculus device initialization/closing and tracking data update... but I decided to move everything for the following reason.
rlgl module can be used as a standalone module including Oculus Rift support. So, advance users can use this self-contained module to add Oculus Rift support together with a simple and easy-to-use OpenGL layer.

Right now, Oculus functionality has been integrated into raylib and it works very well, check this usage example for details.
So, what's next?
Simplify rendering mechanish for raylib users.
Oculus Rift (and any other VR system) requires stereo rendering; it means, rendering two eyes separately considering some eye parameters (position, orientation) for every render. I'm still looking for an easy way to accomplish that without forcing the user to implement the for loop in their code.
That's not easy because user can change drawing states frequently:
Begin3dMode() / End3dMode()           // State change: projection, modelview
Begin2dMode() / End2dMode()           // State change: projection, modelview
BeginShaderMode() / EndShaderMode()   // State change: currentShader
BeginBlendMode() / EndBlenMode()      // State change: blendingMode

...and every state change internal buffers are processed and reseted (together with projection/modelview matrices in most cases). That's a problem for the stereo rendering, specially for the eyes view setup...
Solution: draw-calls batching
The best solution that comes to my mind is batching draw calls, it means storing in a list all the required draw-calls with the required state (matrices, shader, texture, blending, buffers...) and launch internally the stereo rendering at EndDrawing(), applying the required eyes-view transformations.
Right now, that's not easy to do and requires some rlgl redesign. So, I'm looking for a simpler (temporal) solution.

Final recap:
Stereo rendering is working pretty good, right now, using a for loop in the user side (oculus example). This would be the implementation for the first version of this feature.
It has also been added support for Oculus simulator, it means, if no Oculus device is detected or raylib is compiled with no Oculus support, it's still possible to simulate Oculus stereo rendering on the display... with configurable parameters!
Only a couple of points left before clossing this issue:

Setup left/right eyes matrix in simulator for a given IPD.
Embed Oculus distortion shader into code with support for OpenGL ES
Adjust distortion shader parameters to match Oculus device (configurable)

This new feature has become a monster and required lot of raylib redesign (specially for the module rlgl) and, probably, in the future more redesign will be required to support any generic VR device.

Update:
No for loop in the user side! Instead, managed internally in rlgl, in DrawDefaultBuffers() and rlglDrawMesh(). This improvement simplifies a lot user code, now VR experience could be easily enabled.
Now deciding the best way to begin/end stereo drawing... should it be integrated into BeginDrawing() and EndDrawing()? Thinking about it...

VR related functions have been renamed to be HMD independent! Now, desired VR device can be controlled only with 3 functions:
InitVrDevice(HMD_OCULUS_RIFT_CV1);  // If deviced not detected, it fallbacks to simulator
UpdateVrTracking();  // Read tracking data (head position/orientation) to be used at rendering
CloseVrDevice();  // Unload internal stereo buffers and safely close VR device (or simulator)

Finally, begin/end stereo drawing has been moved to Begin3dMode() and End3dMode(), that way, users can only render in stereo 3d content (actually, it makes sense). Any "2d" content (GUI, text...) must be rendered into a 3d billboard...
Now working on HMD physical device parameters translation to stereo rendering configuration (including distortion shader) to allow simulation of any HMD device, just setting up its parameters. ;)

Ok, VR is ready! :)


Handling Output Saving on Ctrl+C Interruption (Question)

HI @xnl-h4ck3r
I'm running scans in huge number of urls, but sometimes I find myself needing to interrupt the process using Ctrl+C before it completes. I'm wondering what happens to the output in such cases. Does knoxnl save any partial results, or is all progress lost upon interruption?

Hi @0xPugazh, thanks for using the tool and taking the time to raise an issue. Knoxnl doesn't currently save partial resilts unfortunately :(
I will add it to the TODO list, but I'm not sure when that will get completed

Thanks for your reply.


Data Details When Finetune

Thanks for your excellent work. I encountered a small problem. When finetune, do I need to retrain a lora for each test class such as color, or should I mix all the fake images together to train a lora?

Thank you for your kind words! In our work, we fine-tune a separate LoRA for each specific category, such as color, shape, etc.
Of course you may mix all the generated images to train a lora.


feat(vitest): support clearScreen cli flag

Description

Closes https://github.com/vitest-dev/vitest/issues/5185

todo

[x] docs
[x] tests

manually tested in examples/basic by adding clearScreen: true/false in vite.config.ts and using --clearScreen=true/false from cli

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[x] Ideally, include a test that fails without this PR but passes with it.
[ ] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[ ] Run the tests with pnpm test:ci.

Documentation

[x] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[x] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

@hi-ogawa you're missing a "r" in your title cleaScreen 😉

@ghiscoding Good catch :smile: I fixed it now!


gitlab-ci workflows reference token.txt which is no longer being created

I referenced the issue on the relevant commit from a few weeks ago:
https://github.com/GoogleCloudPlatform/cloud-foundation-fabric/commit/47daeaafe132730cab7b164c239e988854329c77#r124479639
{
  "type": "external_account",
  "audience": "xxx",
  "subject_token_type": "urn:ietf:params:oauth:token-type:jwt",
  "token_url": "https://sts.googleapis.com/v1/token",
  "credential_source": {
    "file": "token.txt"
  },
  "service_account_impersonation_url": "xxx",
  "service_account_impersonation": {
    "token_lifetime_seconds": 3[60](https://gitlab.com/xxxxx)0
  }
}

The issue seems to be related to the generated credentials.json referencing token.txt as the credential_source but that file is no longer being created in the defaults of the pipeline. This causes the pipelines to fail on rm token.txt as well as any stage after the gcp-auth stage.
ERROR: (gcloud.alpha.storage.cp) There was a problem refreshing your current auth tokens: File 'token.txt' was not found.

Ouch. Thanks for spotting this, I can try getting to the bottom of this tomorrow, but do send a PR if you get there first.


Update TCA from 0.3.0 to 0.4.0

This causes tests to fail, and I'm not sure if it's due to problems with my tests or with TCA itself.

I'm not sure if these were bugs that were present in TCA 0.3.0 and fixed in 0.4.0, or if they're bugs in TCA introduced in 0.4.0.
What do you think @stephencelis @mbrandonw ?

Hey @jpsim, that's strange, but I think from 0.3.0 to 0.4.0 we fixed some bugs with effect cancellation, which you not only use explicitly with .cancellable operator but also implicitly with debounce.
For example, this test:
store.assert(
  .send(.delete([0])) {
    $0.milestones = []
  },
  .send(.persistToDisk)
)

was passing because you explicitly sent the .persistToDisk action to do the persisting action and it seems to have coincidentally canceled the debounced action that started with delete. I say coincidentally because the effect you return from .persistToDisk does not say to cancel in flight effects, yet somehow due to a cancellation bug it was actually cancelling.
So what I think you want to do is make sure that when you force persistence you also cancel the in-flight debounced effect:
persistEffect = Effect.fireAndForget { environment.persist(newMilestones) }
  .cancellable(id: PersistID(), cancelInFlight: true)

Let me know if that works for you.

Ah yes that makes a lot of sense. Tests seem to pass with that change, and it seems like the right thing to do.
Thank you so much for your help!


Wrong peer dependency when installing with Yarn

UnoCSS version
0.50.6
Describe the bug
When installing unocss with Yarn, it warns that vite was requested by @unocss/vite, but unocss didn't provide it.
~/w/rezero develop• 2.3s ❱ yarn install
➤ YN0000: ┌ Resolution step
➤ YN0002: │ @unocss/astro@npm:0.50.6 doesn't provide vite (pd448d), requested by @unocss/vite
➤ YN0002: │ unocss@npm:0.50.6 [7eda2] doesn't provide vite (pb07dc), requested by @unocss/vite
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed in 0s 232ms
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed in 1s 409ms
➤ YN0000: Done with warnings in 1s 851ms
~/w/rezero develop• 2.1s ❱

Possible solution: add vite as optional peer dependency to unocss and @unocss/astro.
Possible workaround: add following directives to .yarnrc.yaml makes warnings go away.
packageExtensions:
  unocss@*:
    peerDependencies:
      vite: ^2.9.0 || ^3.0.0-0 || ^4.0.0
    peerDependenciesMeta:
      vite:
        optional: true
  "@unocss/astro@*":
    peerDependencies:
      vite: ^2.9.0 || ^3.0.0-0 || ^4.0.0
    peerDependenciesMeta:
      vite:
        optional: true

Reproduction
Having package.json fields with
"devDependencies": {
    "unocss": "^0.50.6",
    "vite": "^4.2.1",
}

System Info
Yarn v3.5.0
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

To me, I personally think Yarn's rule to unnecessary too strict, but I am open to have peerDeps in the vite and astro package, if anyone want to send a PR.


add @tanjunchen to /zh reviewer

pr: https://github.com/kubernetes/website/pulls?q=is%3Apr+author%3Atanjunchen+is%3Aclosed
review: https://github.com/kubernetes/website/pulls?q=is%3Apr+reviewed-by%3A%40me+is%3Aclosed
Could I become the reviewer of zh/ website.
/cc @tengqm
/cc @chenrui333
/cc @markthink

/lgtm
@jimangel

/lgtm
/approve
@tanjunchen Thanks for your continued support

/lgtm
/approve

This PR needs approval from chairs ... @chenrui333

@tengqm got it.

/approve


openapi.yaml not standing in specifications.

The openapi.yaml not standing in https://github.com/OAI/OpenAPI-Specification/blob/main/schemas/v2.0/schema.json specifications.
[{\"code\":\"ValidationError\",\"target\":\"representation\",\"message\":\"Parsing error(s): JSON is valid against no schemas from 'oneOf'. Path 'paths['/{connectionId}/api/v1/executions/{id}/result'].get.responses.200', line 806, position 18.\"},{\"code\":\"ValidationError\",\"target\":\"representation\",\"message\":\"Parsing error(s): JSON does not match any schemas from 'anyOf'. Path 'definitions.AliasExecution.properties.actionalias.properties.formats.items', line 1123, position 24.\"},{\"code\":\"ValidationError\",\"target\":\"representation\",\"message\":\"Parsing error(s): Invalid type. Expected Object but got String. Path 'definitions.AliasExecution.properties.execution.properties.$ref', line 1199, position 45.\"},{\"code\":\"ValidationError\",\"target\":\"representation\",\"message\":\"Parsing error(s): Invalid type. Expected Array but got Boolean. Path 'definitions.PacksInstall.properties.skip_dependencies.required', line 1620, position 27.\"},{\"code\":\"ValidationError\",\"target\":\"representation\",\"message\":\"Parsing error(s): The input OpenAPI file is not valid for the OpenAPI specificate https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md (schema https://github.com/OAI/OpenAPI-Specification/blob/master/schemas/v2.0/schema.json).\"}]}}","Target":null,"Details":[],"Innererror":null}
Also recieving alot of errors uploading it to: https://editor.swagger.io/

Thanks for the report.
Sounds like a bug indeed.
If you see any low-hanging fruits to fix the validation here and there, - contributions would be very welcome!

@armab
I also saw this relevant issue:
#4488
I'm not sure from where to start.
For example I got:
Semantic error at paths./api/v1/executions/{id}/re_run.post.parameters.1.schema.properties.tasks
Schemas with 'type: array', require a sibling 'items: ' field

For:
              tasks:
                type: array
                default: []

But I'm not sure what type is required for items (In other words what the type the tasks array should contain):
          items:
            type: string


Is there a way to access gradients of each layer?

I want to display the distribution of gradients of each layer. How can I do it?

Gradients are handled by Theano, and there is no practical way to access them.

Would that be true even when using Keras on top of Tensorflow?

Even I am looking for the same thing. Is it true even if we use keras on top of tensorflow


NPM publish & CI/CD

first successful pipeline: https://github.com/michael-gee/gee-ui/actions/runs/9319820587/job/25655207499


nuxt 3 local - signIn path not matching

Environment
(static website made on nuxt 3)
  "devDependencies": {
    "@nuxtjs/i18n": "^8.0.0-beta.10",
    "@nuxtjs/tailwindcss": "^6.6.5",
    "@pinia/nuxt": "^0.4.8",
    "@sidebase/nuxt-auth": "^0.6.0-alpha.1",
    "@vite-pwa/nuxt": "^0.0.7",
    "@vueuse/core": "^9.13.0",
    "nuxt": "^3.3.3",
    "pinia": "^2.0.34",
  }

Reproduction
in nuxt config:
  ssr: false,
  auth: {
    provider: {
      type: 'local'
    },
    pages: {
      login: '/'
    },
    baseURL: '....amazonaws.com/uat/api',
    endpoints: {
      signIn: { path: 'users/login', method: 'post' }
    }
  }

Describe the bug
when calling signIn the path called is wrong and also method "post" is not used:
curl '........amazonaws.com/uat/api/login' \
  -H 'sec-ch-ua: "Chromium";v="112", "Microsoft Edge";v="112", "Not:A-Brand";v="99"' \
  -H 'DNT: 1' \
  -H 'sec-ch-ua-mobile: ?0' \
  -H 'User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/112.0.1722.48' \
  -H 'content-type: application/json' \
  -H 'accept: application/json' \
  -H 'Referer: http://localhost:3000/' \
  -H 'sec-ch-ua-platform: "Windows"' \
  --data-raw $'{"email":"test@xxx.xxx","password":"xxxxx"}' \
  --compressed

what is expected here is signIn path to be appended after baseURL
Additional context
I'm using a login component where code is taken from your example:

<template>

Status: {{ status }}

Data: {{ data || 'no session data present, are you logged in?' }}

Last refreshed at: {{ lastRefreshedAt || 'no refresh happened' }}

JWT token: {{ token || 'no token present, are you logged in?' }}

    <form @submit.prevent="signIn({ email, password })">
      <input v-model="email" type="text" placeholder="email">
      <input v-model="password" type="password" placeholder="Password">
      <button type="submit">
        sign in
      </button>
    </form>

    <button @click="signIn({ email, password }, { callbackUrl: '/' })">
      sign in (with redirect to /)
    </button>

    <button @click="signOut({ callbackUrl: '/signout' })">
      sign out
    </button>

    <button @click="getSession({ required: false })">
      refresh session (required: false)
    </button>

    <button @click="getSession({ required: true, callbackUrl: '/' })">
      refresh session (required: true)
    </button>
    <NuxtPage />

</template>

Logs
No response

just to be sure: according to this: http://sidebase.io/nuxt-auth/v0.6/configuration/nuxt-config baseURL is a property of root config, not the providers section.
For me, it is working like in the snippet I pasted above.

just to be sure: according to this: http://sidebase.io/nuxt-auth/v0.6/configuration/nuxt-config baseURL is a property of root config, not the providers section. For me, it is working like in the snippet I pasted above.

yes, but for me is not working in any way
and since I also faced an error with signInResponseTokenPointer i decided to give up with this package and write my own storage

This is my auth config in nuxt.config,js, and currently it's working except for the signOut because it's not sending the bearer token in the backend:
``
auth: {
provider: {
type: 'local',
endpoints: {
signIn: {path: '/login', method: 'post'},
signUp: {path: '/register', method: 'post'},
signOut: {path: '/logout', method: 'post'},
getSession: {path: "/profile", method: "get"},
},
        token: {
            signInResponseTokenPointer: '/data/token',
            type: 'Bearer',
            headerName: 'Authorization'
        }
    },

    pages: {
        login: '/login'
    },
    baseURL: process.env.API_BASE_URL,

    session: {
        enableRefreshOnWindowFocus: false
    }
},

``

Hey!
Just did a quick test and I can confirm that if you change the path inside sign-in the composable also uses it for trying to sign in:

@manuel-84 we need a reproduction repository from you to vberify this further!

except for the signOut because

ah, good one! I fixed this in #347 -> the token is now also sent on sign out (:
I'll do a release in the next hour or so that should adress this (:

closing as reproduction is missing - feel free to reopen with a repro, thanks!


Data distribution

https://github.com/chenfengxu714/SqueezeSegV3/blob/346e0fc371940754e633101982db80b54e051257/src/tasks/semantic/config/arch/SSGV353.yaml#L89-L94
https://github.com/chenfengxu714/SqueezeSegV3/blob/346e0fc371940754e633101982db80b54e051257/src/tasks/semantic/config/arch/SSGV321.yaml#L90-L95
Hi, I am confused by the ordering of range and z. Why do range and z swap places in SSGV3-53.yaml and SSGV3-21.yaml?

Hi, thanks for your kind pinpoint. I make a mistake here, the right one should be range: 12.12 and z: -1.04, I believe the right one will lead to a better performance for SSGV3-53, we will retrain it soon.


Add missing checks for nullptr.

We got crash reports from LLDB where protocolList is a nullptr when demangling a symbolic reference. While we're also investigating the root cause of the issue, the code in MetadataReader should also not just crash with such input.
rdar://122698966

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves #NNNNN, fix apple/llvm-project#MMMMM.

@swift-ci test

@swift-ci test

LGTM. Really makes you miss Swift's optional chaining.


Add a job to publish the package in GPR

This PR adds a job to publish the relative-time-element package to GPR (GitHub Private Registry) because having the package only on NPM under @github scope causes an issue for some repos (memex, dotcom) whose automation dependency management are configured to install these private packages from GRP in the @github scope not NPM @github  scope.
For more info, see this slack thread

:wave: Hello and thanks for pinging us! This issue or PR has been added to our inbox and a Design Infrastructure first responder will review it soon.

:art: If this is a PR that includes a visual change, please make sure to add screenshots in the description or deploy this code to a lab machine with instructions for how to test.
:fast_forward: If this is a PR that includes changes to an interaction, please include a video recording in the description.
:warning: If this is urgent, please visit us in #primer on Slack and tag the first responders listed in the channel topic.


More miscellaneous fixes

Coverage decreased (-0.5%) to 94.675% when pulling 9cb903d5e0fe3b87f53d3086b4c15c7a50b4b573 on Juanlu001:more-misc-fixes into 2db28ca896974cbfc21e8f2d4c7c036dc6fbfa41 on satellogic:master.


Add GitHub to Slack

Output github stuff into the sleeves channel in Slack for the contributors to follow along
Maybe Heroku as well to follow builds and deploys.
Maybe not, could be too chatty. @ardeay - Thoughts?

GitHub - https://mtgmarket.slack.com/apps/A0F7YS2SX-github
Heroku - https://mtgmarket.slack.com/apps/A0F7VRF7E-heroku
DoorBell.io - https://mtgmarket.slack.com/apps/A0G4ZP5GV-doorbell


Skip Iceberg tests on Spark 3.4.0

Since Apache Iceberg tests should be skipped for Spark 3.3 or above, this updates the version check to account for Spark 3.4 in addition to Spark 3.3

build


[EDQ-498] Make Logic for Data Quality issue Reporting Consistent

Fixed issue with understanding the distribution behind erroneous weights.
Ensured all f-strings were present in the visualization notebooks.
Got rid of target low information in metrics_over_time and email_generator - not needed now that all of the upstream report-outs are correct.
Made small changes to data quality issue tracker. Issue mostly lied with the difference in calculation methods for some of the 'error metrics'.
Variable changes for the updated unit success rate name.

@nishanthpp93 Just some bug fixes to integrate.

@nishanthpp93 my bad. extra space eliminated.


Drop "service" suffix, and standardize names

When running the demo, it is composed of 25+ individual containers/components/microservices. The names of these components are carried over from the demo's original roots, and several additional services and external components were added to support the demo's goal of showcasing OpenTelemetry across various languages and frameworks.
Currently, most services in the demo have a service suffix as part of their image and OpenTelemetry service name. Containers will have a -service suffix. Some services, such as the product catalog service, have a space in their name. The container name will be hyphenated, but the service/pod and OpenTelemetry definitions will have this as a single word.
A well-understood, concise, and consistent naming for all services would significantly improve the usability of the demo for users in observability tooling as well as contributors that support this effort. I propose dropping the service suffix from all services and using a hyphen (-) for spaces in names. The names should be consistent across the docker compose service, container name, OpenTelemetry service name, and Kubernetes deployment name.
The following table describes the current and suggested name for each Demo component:

Current Name
Suggested Name

accountingservice
accounting

adservice
ad

cartservice
cart

checkoutservice
checkout

currencyservice
currency

emailservice
email

flagd
flagd

flagd-ui
flagd-ui

frauddetectionservice
fraud-detection

frontend
frontend

frontendproxy
frontend-proxy

frontend-web
frontend-web

grafana
grafana

imageprovider
image-provider

jaeger
jaeger

kafka
kafka

loadgenerator
load-generator

opensearch
opensearch

otelcollector
otel-collector

paymentservice
payment

productcatalogservice
product-catalog

prometheus
prometheus

quotesservice
quote

recommendationsservice
recommendation

shippingservice
shipping

valkey-cart
valkey

I welcome feedback and suggestions on this proposal. This is something we can also discuss during the next SIG meeting.

Great suggestion!
+1 to that.
I'd just keep the valkey-cart. But fine in going to only valley as well.

Hello @puckpuck and @julianocosta89,
I’m interested in contributing to this topic. Would it be okay if I start with the AdService component?

Hello @puckpuck and @julianocosta89,
I’m interested in contributing to this topic. Would it be okay if I start with the AdService component?

yes please help where you can here

Is the helm chart being updated to match this work?

@pessoa not yet.
The Helm chart will only be update with a new release of the demo that should happen after all rename is done


[Feature request] - Create a code sample using Svelte Framework - Page 04

This is a suggestion to include a sample code using Svelte Framework in this 2 pages:
Link 1: https://docs.microsoft.com/en-us/azure/javascript/tutorial-vscode-static-website-node-04?tabs=angular
Since the Svelte Community has been growing every day among developers around the world. As you can see in the image below.

Closing as resolved because there's an active PR and we don't need the issue tracking.


Select items inherit colour

Prerequisites

[X] I have not leaked any internal company information.
[X] I have not posted internal links or code snippets.

What happened?
Hi,
I noticed that the items in the select component inherit the colour from the surrounding container.

This requires manually applying the colour for each dropdown, if it is contained in a container that has a different colour.
What type of frontend frameware are you seeing the problem on?
Angular
Which version of iX do you use?
2.0.4
Code to produce this issue.
-

Internal ref [IX-894]


how can i get imei number of android device using flutter

Added in https://github.com/TheDOme6/device_id/commit/7d43af33ee991b64b0b3836e8de059eb3a822e9d


Type double does not exist on Postgres

Hi,
I have a class like this
case class Geocoding(city: String, latitude: Double, longitude: Double)

and this SORM initialization:
object Db extends Instance (
  entities = Set(
    Entity[Geocoding](unique = Set() + Seq("city", "latitude", "longitude"), indexed = Set() + Seq("city") + Seq("latitude", "longitude")),
  url = "jdbc:postgresql://localhost:5432/common_data",
  initMode = InitMode.Create,
  poolSize = 20
)

using Postgresql (driver version: 9.1-901.jdbc4)
SORM try to create the table with this SQL code:
| CREATE TABLE "geocoding"
|   ( "id" BIGSERIAL NOT NULL,
|     "longitude" DOUBLE NOT NULL,
|     "latitude" DOUBLE NOT NULL,
|     "city" VARCHAR(255) NOT NULL,
|     PRIMARY KEY ("id"),
|     UNIQUE ("city", "latitude", "longitude") )

with version <= 0.3.8 the geocoding table is not create silently
with version 0.3.13 i get
java.lang.ExceptionInInitializerError: null
...
Caused by: org.postgresql.util.PSQLException: ERROR: type "double" does not exist

The error is caused by "DOUBLE" type,
Postgres have instead "DOUBLE PRECISION" type.

We use Slick 3.1 and it generate wrong create statements for Double PostgreSQL column, it generate DOUBLE instead of double precision !

seeing the same problem as @zarinfam any suggestions?

nevermind! resolved our issue.
@zarinfam: one possible gotcha that tricked us up and you might want to watch out for -- the source of our bug was namespace collisions between an import of slick's H2 DSL (import slick.driver.H2Driver.api._) and its Postgres driver (import slick.driver.PostgresDriver.api._) which caused calls to column[Double]in ourTabledeclaration code to pick up onH2Driver.api's notion of Doubles rather than Postgres`'s. Not sure if this might be the cause of your bug, but perhaps worth a quick sanity check just to make sure?

@aguestuser You were right and the bug was for namespace collisions. thanks.

Thanks @aguestuser made the same mistake today - thanks for reporting back the root cause.


*: update vendor, clean up test results

Fix https://github.com/coreos/dbtester/issues/42
Fix https://github.com/coreos/dbtester/issues/46

@heyitsanthony only minor changes in control package (just use capnslog to be consistent).
And bunch dep updates to sync with current etcd master. And remove all bunch of unnecessary files in the repo. Will update other upstream packages (consul, zk ones) later.


Update vendored test-infra

Also update update-deps.sh to remove the unused test-ifnra/devstats dir and OWNERS files.
Part of #knative/test-infra#829

Although this is focused on test-infra, update-deps.sh changed some other stuff in Gopkg.lock. Since verify-codegen.sh passed, I assume these changes are necessary.

/cc @yt3liu

/lgtm

Does this supersede #428?

Yes. I'll close 428.


biggest warnings reduce and typing.

Before: 200 warns
Now: 114 warns

Also i wanna convert Brain and anothers to dataclasses, to reduce memory-use with slots.

It's looking good!

Ready to merge

Problem with static typing in Python is circular dependency.
Now we have project->vectorbd.tools->project :(

Problem with static typing in Python is circular dependency. Now we have project->vectorbd.tools->project :(

oh...

sorry for it


Please correct DefaultFailureHandlerImpl log content

https://github.com/alibaba/fescar/blob/develop/tm/src/main/java/com/alibaba/fescar/tm/api/DefaultFailureHandlerImpl.java
begin  should be rollback~~~~~
    @Override
    public void onRollbackFailure(GlobalTransaction tx, Throwable cause) {
        LOGGER.warn("Failed to begin transaction[" + tx.getXid() + "]", cause);
    }

please close this issue as the related bug has been fixed.


[BUG] Hotkeys not working

Describe the bug
Hotkeys not working, Enhance Cmd+A not working
Expected behavior
Expected to be able to move blocks up and down, hit Cmd+A twice to select the current block.
Desktop (please complete the following information):

OS: MacOS Big Sur
Obsidian Version: 0.12.4

Additional context
Have tried turning off all other plugins, have tried uninstalling and reinstalling obsidian. No luck.
Had a friend who is a more experienced obsidian user look over my obsidian settings and nothing about them stuck out as odd to him.

Hey Guys
I can confirm hotkeys stopped working for me either.
Not sure but it might be a regression after  recent 0.12.15 release  ?

FWIW -- I realized that some of the hotkeys that weren't working for me
were because I wasn't following the stricter markdown rules for creating
lists (as I was previously using Roam which makes lists with less strict
rules). Once I was following the markdown rules (i.e., start a list with a
dash followed by a space) the move up and move down hotkeys were working
within those lists.
The Cmd+A hotkey only works with the advanced functionality that this
plugin provides if I'm within a list. If I try to use it within a
paragraph that is not a list, it just selects the full page. I think that
this is how the plugin is supposed to work, though. If not, then that's
still a problem I'm having. But if so, then from my end this is resolved.
On Sun, Jun 27, 2021 at 7:15 AM Zeebee Siwiec @.***>
wrote:

Hey Guys
I can confirm hotkeys stopped working for me either.
Not sure but it might be a regression after recent 0.12.15 release ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vslinko/obsidian-outliner/issues/161#issuecomment-869170787,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEKILLOBU5T76JBHXL4PEQLTU4XADANCNFSM464PJRUQ
.

Hi!
Could you please provide some examples of documents where the hotkeys doesn't work?

Right now the double Cmd+A hotkey isn't working for me across all the pages
in my database unless I use it within a properly formatted list. But if I
try to use it while my cursor is in the middle of a paragraph, for
instance, it will not work (i.e., in this case when I hit Cmd-A it just
selects the full page, and if I hit it a second time nothing different
happens).
I've tried starting fresh databases and the problem still happens for me.
Maybe this is a functionality limit and I'm just not aware that having it
work from within paragraphs isn't supported?
On Sat, Jul 3, 2021 at 5:45 AM Viacheslav Slinko @.***>
wrote:

Hi!
Could you please provide some examples of documents where the hotkeys
doesn't work?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vslinko/obsidian-outliner/issues/161#issuecomment-873402174,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEKILLM7WS3WFE4MNTLPNYLTV4A6FANCNFSM464PJRUQ
.

CmdA should only work in properly formatted lists, this is true. Do you have any other problems with the hotkeys?

@byniutek Could you please describe more about your problem?

Ahh, okay. Then all is working as intended. I'm not having any other
problems with the hotkeys.
On Sat, Jul 3, 2021 at 2:17 PM Viacheslav Slinko @.***>
wrote:

CmdA should only work in properly formatted lists, this is true. Do you
have any other problems with the hotkeys?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vslinko/obsidian-outliner/issues/161#issuecomment-873471637,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEKILLOFQP3S34SMS656LQ3TV55AJANCNFSM464PJRUQ
.


Remove go-etcd

As of https://github.com/coreos/etcd/commit/7d172bb1142eee5ee3bb6b2b4f08e2390217e01f, etcdctl uses the etcd/client pkg and github.com/coreos/go-etcd. The official etcd/client pkg needs to be rich enough to drive all of etcdctl, and go-etcd needs to be removed from etcd altogether.

@xiang90 We talked about adding this to the v2.1.0 milestone. But we need to consider #2604 too.

progress at #2913

Fixed by #3083


fix: centered to-do list wrapper

Description:
This Pull Request addresses issue #9, where we needed to center-align the to-do list wrapper and make it mobile-responsive to enhance the visual aesthetics and usability of our project.
Screenshot

Great job on addressing issue #9, The changes look fantastic. Thank you for your hard work and attention to detail.


Get DTA working on android again

@swbsi is this still a valid PR?

@swbsi is this still a valid PR?

No, we can get rid of it.


[Feature][Kanbanize] Add kanbanize support

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Use case
Hey all,
Our team currently uses Kanbanize for issue tracking/project management. We are currently looking for a solution similar to DevLake that can ingest our data and provide insights into DORA metrics to measure DevOps improvements across our engineering org.
Having a Kanbanize plugin to ingest issues would be very beneficial for measurement.
Description
Using the Dev Manual, we could integrate Kanbanize into DevLake via the Kanbanize REST API under the Issue Tracking domain for data sources.
Kanbanize has a very simple API key for authorization, I believe this would be entered by end users during connection.
Related issues
#6180
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

We have a discussion here: https://github.com/apache/incubator-devlake/discussions/6619


Segnet Datasets

In the FCN-AlexNet-PASCAL-VOC directory there is an original.prototxt along with the trained model, and the other files created for this model.
But, the original.prototxt in there doesn't match up with the fcn_alexnet.prototxt that a person would normally use on the Pascal-VOC dataset as part of the semantic-segmentation example tutorial.
The key difference seems to be the fcn_alexnet.prototxt has a "group: 21" setting under the convolution_param for the upscore layer. This is missing on the original.prototxt and wasn't used to build the model.
I don't know what this setting does, but it seems to make a big difference. I'm also not able to run the segnet-console app with a model that has been trained with this setting. One reason for this is I can't train a model successfully unless I have this setting along with the changing the layer names to match what original.prototxt has. So any prototxt/model I test has both changes.
Now I know the segnet inference is likely still being worked on, but is there any limitation we should be aware of? The included examples seem to work fine.

Hi, the issue was that 'group' setting was not available in TensorRT 1.0.   The group allows for more than 21 classes to be represented (it derives from cityscapes which has ~30+ classes by default).  Hence I removed it for the segmentation models used in this TensorRT-based tutorial.

Also I remembered now that I totally remove the Deconvolution 'upscore' layer from my final segmentation networks, because it has learning-rate (LR) of zero (i.e. represents a simple linear transform) and runs slowly.  Hence I just perform it myself as post-processing.  After training in DIGITS, remove the Deconvolution and Crop layers near the end of the prototxt.  Also you change 'pad: 100' parameter of 'conv1' layer to 0 to get it to load in TensorRT.

The dataset I'm using has less than 21 classes so the group parameter isn't really needed. But, the problem is when I remove it I lose the ability to train the model. Where it won't work with the pre-trained FCN_Alexnet model that net_surgery.py in Digits 5.0 generates.  Either it complains about the layer name conflict or it won't learn (accuracy stays at zero).
What model did you use as a pretrained model?
 I removed both of those after training. I don't recall this impacting anything. I started off with the CityScape which has both of those removed.

Initially, when I was still using the deconv layer with TensorRT, I was using this devbranch of DIGITS as a workaround:   https://github.com/gheinrich/DIGITS/tree/dev/groupless-deconv
However, now that I am removing deconv layer entirely when use with TensorRT, the special DIGITS is no longer required.  You can train like normal.   Then after the model is done training, remove the deconv layer from prototxt before loading with TensorRT.   Hence the deconv group parameter is no longer relevant.
I just confirmed that my Pascal-VOC model still has the deconv layer listed, this must have been left over from before.  I am removing it like the others and re-uploading Pascal model.
From: Jason Mecham<EMAIL_ADDRESS>Sent: Tuesday, April 11, 2017 12:45 PM
To: dusty-nv/jetson-inference
Cc: Dustin Franklin; Comment
Subject: Re: [dusty-nv/jetson-inference] Segnet Datasets (#67)
The dataset I'm using has less than 21 classes so the group parameter isn't really needed. But, the problem is when I remove it I lose the ability to train the model. Where it won't work with the pre-trained FCN_Alexnet model that net_surgery.py in Digits 5.0 generates. Either it complains about the layer name conflict or it won't learn (accuracy stays at zero).
What model did you use as a pretrained model?
 I removed both of those after training. I don't recall this impacting anything. I started off with the CityScape which has both of those removed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/dusty-nv/jetson-inference/issues/67#issuecomment-293323956, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOpDK0MDqZ4viBBtIG0FWSEzQZ40fOiKks5ru64MgaJpZM4M3UFz.

This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.

Thanks. That took care of that issue.
I'm able to successfully train the model in Digits 5.0 by leaving in the Group setting. The only thing I had to do to get it to work with segnet-console is to retrain using the layer names segnet expected (score_fr_21classes).
Once I trained the model I simply deleted the upscore layer, and the crop layer from the deploy.prototxt and changed the PAD parameter of conv1 to from 100 to 0 (gave a segmentation fault when I forgot this).
So it now seems to load successfully, but it's not giving me results like I expect. But, then I tried FCN-AlexNet-Pascal-Voc and it's not either.
I noticed that for the overlay colors file it's set to null, and there isn't a file like there is for the other ones like fcn-alexnet-aerial-fpv-720p-21ch that works fine. I did try a file, but it didn't give me anything that resembled the results I expected.

Here is what the output looks like using the AlexNet-Pascal-Voc model by default.

Here is the original file from the Semantic example.

Here is the what the output looks like using the latest AlexNet-Pascal-VOC that was updated a couple days ago. I noticed it added the color table file.


SIFT Install Issue on AWS using ami-04505e74c0741db8d (64-bit x86)

I've been trying to install SIFT on vanilla Ubuntu 20.04 running on AWS.
Ubuntu Server 20.04 LTS (HVM), SSD Volume Type - ami-04505e74c0741db8d (64-bit x86)
--command--
$ sudo sift install --mode=server
Issue: Hangs right at the following line every time

Running: cpanm --no-man-pages install DBD::SQLit

Is this a know issue or is there a workaround?
Thanks
Theo

Hi @theocaines , any chance you can provide your saltstack.log so we can review?

Doing some additional checking as I was able to go to completion on using a larger instance type (t2.xlarge). All previous tries were on t2.micro and I noticed the CPU being pegged at 100%.  I will update you on this either way.

There are prebuilt AMIs already. See the main SIFT repo's README


Feat/timegrid/2.0

Please check if the PR fulfills these requirements

[x] It's submitted to right branch according to our branching model
[x] It's right issue type on title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[x] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has description for the breaking change

Description
The Times component supports  various expression type of times like 'minute', 'hour', 'date', 'month', 'year'.
You can see the UI from Storybook page.
Breaking Changes of templates
src/ts/template/default.ts

timezoneDisplayLabel
timegridDisplayPrimaryTime
timegridDisplayTime
timegridCurrentTime

Next feature of Times component

Theme feature will be implemented after making whole Week View'
ScrollToNow function will be implemented when TimeGrid component.

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

재택 중엔 카르마 테스트가 안되네요.

[3/3] 리뷰 완료합니다. 고생하셨습니다~


Move back to finding bash with env

Unclear why https://github.com/oogali/ebs-automatic-nvme-mapping/commit/80b4f7d71716c3022c154417c934997b07e0b68b moved from the more flexible env?

https://github.com/oogali/ebs-automatic-nvme-mapping/pull/4 moved to using env - I think a PR that got merged in didn't rebase

I verified that fully qualified paths are needed for udev on CentOS 7.8 (which the script in this repo was before); moving back to that as well


[T000462] Rep. Pat Tiberi

Rep. Pat Tiberi:
http://tiberi.house.gov
Likely contact form URL to start with:
http://www.house.gov/writerep

YMAL file not found. I am working on it.


NETOBSERV-1995: Use global map, spinlock, split maps

Also:

remove MAC and eth_protocol from map key (they're moved to flow
metrics ie. map values)
in every hook, we don't update flow counters, even when the flow doesn't exist and is created. Previously, we were
adding bytes/packet counters from hooks, which doesn't seem correct as
counting bytes/packets is the role of the TC hook only - else it can
lead to overcounts I guess
remove code that sets MIN_RTT on TCP flows (not sure why we had that)
small refactoring of the "Accumulate" functions to adapt to new model

Perf tests (https://docs.google.com/spreadsheets/d/1qakBaK1dk_rERO30k1cSR4W-Nn0SXW4A3lqQ1sZC4rE/edit?gid=48358889#gid=48358889) show ~ -25% CPU usage with the global map

cc @tohojo

removing packed attribute gives another 20% CPU gain
I'm currently at about -40% CPU and -10% memory
And haven't done the in-kernel dedup yet ...

Looked through the BPF code, and it basically LGTM :)

I appear to have broken DNS somehow ...

Perf-scale test results: comparison, full data sheet
Comparison is done with https://github.com/netobserv/netobserv-ebpf-agent/pull/466
=> -43% agent CPU ; other metrics are roughly stable (~+2 to 5% in everything, including workload throughput, number of flows, memory used, etc.)

Perf-scale test results: comparison, full data sheet Comparison is done with #466
=> -43% agent CPU ; other metrics are roughly stable (~+2 to 5% in everything, including workload throughput, number of flows, memory used, etc.)

its very strange that memory shows tiny increase moving from perCPU to global for the same number of flows I would expected to see ~ 1/ncpu memory drop is the cpu drop mostly in lookupAndDelete() or somewhere else ?

"Mohamed S. Mahmoud" @.***> writes:

=> -43% agent CPU ; other metrics are roughly stable (~+2 to 5% in everything, including workload throughput, number of flows, memory used, etc.)

its very strange that memory shows tiny increase moving from perCPU to global for the same number of flows I would expected to see ~ 1/ncpu memory drop is the cpu drop mostly in lookupAndDelete() or somewhere else ?

Are you actually measuring the memory and CPU usage of the BPF programs
and maps, or is this just numbers for the userspace side? If it's the
latter I guess it makes sense that the memory usage doesn't change, and
the drop in CPU would be because the time spent de-duplicating things
goes away?

"Mohamed S. Mahmoud" @.***> writes:

=> -43% agent CPU ; other metrics are roughly stable (~+2 to 5% in everything, including workload throughput, number of flows, memory used, etc.) its very strange that memory shows tiny increase moving from perCPU to global for the same number of flows I would expected to see ~ 1/ncpu memory drop is the cpu drop mostly in lookupAndDelete() or somewhere else ?
Are you actually measuring the memory and CPU usage of the BPF programs and maps, or is this just numbers for the userspace side? If it's the latter I guess it makes sense that the memory usage doesn't change, and the drop in CPU would be because the time spent de-duplicating things goes away?

we still doing dedup in this PR but we no longer aggregate we probably need to compare memory using bpftool to also check kernel usage

Yes exactly we don't track today the BPF memory :-/ We definitely should

Joel Takvorian @.***> writes:

Yes exactly we don't track today the BPF memory :-/ We definitely
should

The BPF map memory usage is accounted to memory cgroup of the process
that creates the map, so one way to do this would be to just record
memory usage at the cgroup level; then you won't have to do anything
special for BPF :)

few nits but I am more less done with review, will be good to run features testing with this PR pre merge
/lgtm

pls update the arch doc reference to perCPu hashmap and make sure to run https://github.com/netobserv/netobserv-ebpf-agent/pull/172#issuecomment-1698837448 to make sure we have no concurrency issues again

btw added a diagram for kernel space https://github.com/jotak/netobserv-agent/blob/global-map/docs/architecture.md

btw added a diagram for kernel space https://github.com/jotak/netobserv-agent/blob/global-map/docs/architecture.md

can u share the tool used to draw as we add new hooks on a regular basis so that diag will be quickly become out of date

@msherif1234 you don't necessarily need a tool you can directly edit script:
flowchart TD
    A[TC/X Hooks] -->|Accumulate packet data| M1(Global map: aggregated_flows)
    D{If DNS} -->|Req: store req info| MD(Global map: dns_flows)
    D -->|Resp: compute latency| MD
    A -->D
    D -->|Store DNS info| M2(PerCPU map: additional_flow_metrics)
    B[Drops Hook: kfree_skb] -->|Accumulate drop data| M2(PerCPU map: additional_flow_metrics)
    C[RTT Hook: tcp_rcv_established] -->|Extract & store sRTT| M2(PerCPU map: additional_flow_metrics)
    E[Events Hook: psample_sample_packet] -->|Accumulate net events| M2(PerCPU map: additional_flow_metrics)
    A -->F{If busy map / error}
    F -->|Single-packet flow| RB(RingBuffer)
    M1 --> |Polling|U[User space]
    M2 --> |Polling|U
    RB --> |Push|U

So you can easily add a new hook by adding something like:
F[My new hook] --> |any interaction written here| M2

(M2 stands for the additional metrics map)
Or if you prefer there's also online tools such as this: https://mermaid.live/

I took some measurements on the ebpf side, written down there: https://docs.google.com/spreadsheets/d/1gdwX4ZLjKicaJzANVq-r-lAW6uZxefqjBsIw3Orx4bo/edit?gid=0#gid=0
That's on a single (worker) node, running my usual stress tests. I looked at the sum of each data structure size, then removed a constant 16MB that's taken by ring buffer to make the gains on maps more apparent.
For CPU I used bpftop, with very approximative manual measurement (don't rely too much on my CPU measurements!)
Between -28% and -54% memory used depending on scenarios. And between -4% and -36% CPU.
The next PR improves that a little bit further with between -42% and -61% memory, and around -20% CPU.

/label qe-approved

thanks @msherif1234  @tohojo @Amoghrd !
/approve


Fix Selenium waits, use JS clicks, and update image resampling method

Summary:

Improved element detection and interaction in change_tone function.
Modified image thumbnail generation method.
Removed redundant functionality.

Details:

Element Detection and Interaction:

Wait Condition: To ensure robustness, a wait condition using WebDriverWait was added. This prevents the script from proceeding before the required element (cib-serp) is fully loaded.
JavaScript Click: The direct click method occasionally caused the ElementClickInterceptedException because of browser overlays. Switching to a JavaScript-based click ensures that the element is interacted with directly, bypassing any potential overlays.
Updated Clicks: For consistency and reliability, other element clicks in the code were updated to use the JavaScript execution method.

Image Processing:

Resampling Method Change: The Image.ANTIALIAS attribute was causing an AttributeError due to version discrepancies in the Pillow library. By switching to Image.LANCZOS, which is synonymous with ANTIALIAS in newer versions, the code becomes compatible with recent Pillow versions while maintaining the desired image quality.

Cleanup:

Removed unnecessary functionality to reduce code complexity and improve maintainability.

@Arti787 Please address the commens in review. Outstanding work!
I have some additional concerns too:

When I was running my old script, it often typed only the latter portion of the text prompt accompanying the image. For example, if i told it to type "How can AI benefit us?" it would miss the first 4-6 keys and type "an AI benefit us?" instead. Are you aware of this issue? Do your changes address this?
Have you tested the script in cases where it returns a very long response? Does the script terminate at the specified wait_time and read the response then only, or does it check for the response to finish? Is there some solution or way to stop it from reading the response too early?

@Arti787 Please address the commens in review. Outstanding work!
I have some additional concerns too:

When I was running my old script, it often typed only the latter portion of the text prompt accompanying the image. For example, if i told it to type "How can AI benefit us?" it would miss the first 4-6 keys and type "an AI benefit us?" instead. Are you aware of this issue? Do your changes address this?
Have you tested the script in cases where it returns a very long response? Does the script terminate at the specified wait_time and read the response then only, or does it check for the response to finish? Is there some solution or way to stop it from reading the response too early?

Answer to the first question - yes, the current implementation fixes the problem of not writing the whole message to the text box, but now there is a possibility of captcha and I don't know how to fix that yet
The answer to the second question - it is not implemented now, but you can track the state of the /html/body/div[2]/cib-serp/cib-action-bar//div/cib-typing-indicator element which has aria-hidden="true" parameter.
As long as this parameter is false, bing is currently responding to the user. as soon as this parameter is true, bing has stopped responding, and you can start parsing its response


Apim 3945 mapi v2 add create theme endpoint

Issue
https://gravitee.atlassian.net/browse/APIM-3945
Description
User can create portal and portal next themes with endpoint /themes in mapi-v2
Additional context

📚  View the storybook of this branch here

Will be addressed by other PRs


auto suspend/resume review apps

This PR enables Fly's auto suspend/resume feature for review apps.
This will shut review app machines down if they've been idle for 5 mins and then restart them on demand on an incoming request (pretty much how the old heroku review apps used to work).
This should save us some money on review apps.
We could also consider moving the cleanup review apps job to run less frequently so we don't have to keep re-deploying the review apps if we have long-running reviews, although I have not done it in this PR.

The review app at https://pr-10699-badges-shields.fly.dev/ will sleep after 5 mins of inactivity and resume on an incoming request.


Help needed to coding tool selection - also all other advices are gold..

Meaning, what physical tool must I use ?
Have in-house :
VCDS hex2
PEAK PCAN-USB (IPEH-002021)
misc no-name K+DCAN usb interface (bmw originated, used with INPA&ISTA)
With VCDS I can contact directly to the ESM module (without can-gateway module or car) though address C0 and see that ok... but it doesn't allow anything else to be modified then the sound volume.
Might be able to other things if i would have security code for it, dunno.
I have 2 different ESM's lying around that I could do some coding testing / parameter reading - most intruiging would be unit from a 2019 Porsche 982, with OE numbers (both stamped in device) :
Porsche 982 907 160 K
Audi 4G0 907 160 B
What i recon from Youtube, this Porsche module would give out nice sounds, but honestly I don't know how to start this.
As electrical engineer, working in automotive fieId for few years (for work & hobbies), I do know some basics, but at this point I'm stucked on quite low level issues.
https://repsa.kuvat.fi/kuvat/Avo2/activesound/20220611_102307.jpg POC for direct connection on image.
And, if i would need new HW, e.g. some ODIS compatible or VCP+K or other, I could use some hints&tips on how-to code.
As I'm not saying that I'm stupid, but these aren't actually my normal work related issues... Sorry about that.
But, In advance, thank you all helping - and jilleb & NumberOneBot naturally. Truly amazing work on the reverse engineering.

The work both of you are involved in is beyond me... but I am using ODIS-E (offline) to flash firmware.  Slow and annoying to use, but works just fine.

Here's the board depotted.

Is a f3371 nec v850 processor, I have a c0 fucked because of uploading wrong firmware and I have opened it. Will try to read with vvdi prog tool which supports that processor. Will update this thread after successfully read it and try to rewrite to c0 again…


[Feature] add biometric auth and import any type of file support like zip apk etc

Biometic auth is already planned (#45)
Videos are planned for the future (#36). Other file types may follow, but the main focus will be media files. Photok is not a encrypted file manager.

Thank you for your contribution, I will close this. Please look at the referenced issues.

Photok is not a encrypted file manager.
But it could be, photoc literally stores picture files in zip like archive container, it's not probem to add file support, if the tumbnail is the problem for the other files types, it can be default file icon thumb, its very useful feature, there's no other like app in f droid to support this simple encrypted archive container feature, so you can add and make easier for users to use less apps, and make devs to not make thousands of forks, also it would be nice if you add folder support to organise your encrypted files, please don't ignore this.

So.
Photok is designed and made to be a gallery, and I will keep it like that.
But, I could imagine to support the type "file" in the future. If so, it won't become the main focus of this project.

there's no other like app in f droid to support this simple encrypted archive container feature

You could use Cryptomator (paid app, it is in IzzyOnDroid F-droid repo) for files. Or just create a veracrypt container and open it with EDS Lite.
In my opinion, there should not be files in Photok.

there's no other like app in f droid to support this simple encrypted archive container feature

You could use Cryptomator (paid app, it is in IzzyOnDroid F-droid repo) for files. Or just create a veracrypt container and open it with EDS Lite.
In my opinion, there should not be files in Photok.

I second that I think photok should only support Photos, Videos and Audio files nothing more (Multimedia files) but it's true that we need a veracrypt like app for fdroid like EDS Lite but with the premium feautures!

there's no other like app in f droid to support this simple encrypted archive container feature

You could use Cryptomator (paid app, it is in IzzyOnDroid F-droid repo) for files. Or just create a veracrypt container and open it with EDS Lite.
In my opinion, there should not be files in Photok.

Eds lite is bad and useless app compared  to photok,
The reason i prefer photok over eds is for very nice simplicity

I second that I think photok should only support Photos, Videos and Audio files nothing more (Multimedia files) but it's true that we need a veracrypt like app for fdroid like EDS Lite but with the premium feautures!

Yea sadly thats true.. EDS even the premium version has to many bugs and Cryptomator is expensive.
We just need another app which can mount gocryptfs, for this, a filemanager would be necessary due to this necessarity files would not fit into Photok in my opinion.
To combine the structure of a filemanager and a simple gallery would not be easy I think. Try to search and open photos in a filemanager, thats not comfortable. The other way round you would need to hassle with files in a gallery, thats not comfortable either.
Due to this, I do not use EDS Lite (which is comparable with a filemanager) anymore. It was not comfortable for photos.


Link to several extensions

Hi, I've add so many folks adding extensions to the list, that there are a bunch of merge conflicts. Would you mind fixing them?

@ExtendVS done.


Update microsoft-365-vpn-implement-split-tunnel.md

Added a question Faq on Teams support scenario with browser and desktop client.

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/2784

@JohanFreelancer9  Please copy review

@kelleyvice-msft Please merge this PR. Thanks.


Can't compile dependency mongodb

I have a phoenix app that I'm trying to use with mongodb. When I try to start the app or run mix deps.compile, I get the following error:
== Compilation error in file lib/mongo/protocol.ex ==
** (CompileError) lib/mongo/protocol.ex: internal error in beam_record;
crash reason: {case_clause,{'EXIT',{{badmatch,false},
                      [{beam_utils,defs_list,4,
                                   [{file,"beam_utils.erl"},{line,1251}]},
                       {beam_utils,defs,3,
                                   [{file,"beam_utils.erl"},{line,1109}]},
                       {beam_utils,defs,3,
                                   [{file,"beam_utils.erl"},{line,1158}]},
                       {beam_utils,defs,3,
                                   [{file,"beam_utils.erl"},{line,1167}]},
                       {beam_utils,defs,3,
                                   [{file,"beam_utils.erl"},{line,1158}]},
                       {beam_utils,defs,3,
                                   [{file,"beam_utils.erl"},{line,1184}]},
                       {beam_utils,defs,3,
                                   [{file,"beam_utils.erl"},{line,1110}]},
                       {beam_utils,defs,3,
                                   [{file,"beam_utils.erl"},{line,1184}]}]}}}

  in function  compile:'-select_passes/2-anonymous-2-'/3 (compile.erl, line 585)
  in call from compile:'-internal_comp/5-anonymous-1-'/3 (compile.erl, line 349)
  in call from compile:fold_comp/4 (compile.erl, line 376)
  in call from compile:internal_comp/5 (compile.erl, line 360)
  in call from compile:'-do_compile/2-anonymous-0-'/2 (compile.erl, line 177)
  in call from compile:'-do_compile/2-anonymous-1-'/1 (compile.erl, line 190)
    (stdlib) lists.erl:1338: :lists.foreach/2
    (stdlib) erl_eval.erl:677: :erl_eval.do_apply/6
could not compile dependency :mongodb, "mix compile" failed. You can recompile this dependency with "mix deps.compile mongodb", update it with "mix deps.update mongodb" or clean it with "mix deps.clean mongodb

Running the commands suggested by the output does not address the issue.
My Elixir version:  1.7.2
Here's my mix.exs file:
defmodule MyApp.Mixfile do
  use Mix.Project

  def project do
    [
      app: :my_app,
      version: "0.0.1",
      elixir: ">= 1.5.3",
      elixirc_paths: elixirc_paths(Mix.env),
      compilers: [:phoenix, :gettext] ++ Mix.compilers,
      build_embedded: Mix.env == :prod,
      start_permanent: Mix.env == :prod,
      aliases: aliases(),
      deps: deps()
    ]
  end

  # Configuration for the OTP application.
  #
  # Type `mix help compile.app` for more information.
  def application do
    [
      mod: {MyApp.Application, []},
      extra_applications: [
        :logger,
        :runtime_tools,
        :comeonin,
        :timex,
        :faker,
        :poolboy,
        :mongodb,
        :uuid,
        :httpoison,
      ]
    ]
  end

  # Specifies which paths to compile per environment.
  defp elixirc_paths(:test), do: ["lib", "test/support"]
  defp elixirc_paths(_),     do: ["lib"]

  # Specifies your project dependencies.
  #
  # Type `mix help deps` for examples and options.
  defp deps do
    [
      {:phoenix, "~> 1.3.4"},
      {:phoenix_pubsub, "~> 1.0.2"},
      {:phoenix_ecto, "~> 3.3"},
      {:postgrex, "~> 0.13.5"},
      {:phoenix_html, "~> 2.12.0"},
      {:phoenix_live_reload, "~> 1.0", only: :dev},
      {:gettext, "~> 0.11"},
      {:faker, "~> 0.10"},
      {:comeonin, "~> 4.1.1"},
      {:bcrypt_elixir, "~> 1.0.9"},
      {:cors_plug, "~> 1.5"},
      {:guardian, "~> 1.1"},
      {:timex, "~> 3.3"},
      {:timex_ecto, "~> 3.3"},
      {:tzdata, "~> 0.5.17"},
      {:cowboy, "~> 1.0"},
      {:mongodb, ">= 0.0.0"},
      {:poolboy, ">= 0.0.0"},
      {:uuid, "~> 1.1" },
      {:httpoison, "~> 1.2", override: true},
    ]
  end

  # Aliases are shortcuts or tasks specific to the current project.
  # For example, to create, migrate and run the seeds file at once:
  #
  #     $ mix ecto.setup
  #
  # See the documentation for `Mix` for more info on aliases.
  defp aliases do
    [
      "ecto.setup": ["ecto.create", "ecto.migrate", "run priv/repo/seeds.exs"],
      "ecto.reset": ["ecto.drop", "ecto.setup"],
      "test": ["ecto.create --quiet", "ecto.migrate", "test"]
    ]
  end
end

Any suggestions are appreciated!

Duplicate of #234. You will need to upgrade upgrade your OTP version.


remove trigger markup for non-scrolly formats

It'd be nice if a closeread doc could "fail gracefully" when rendered to a different format like pdf or html. Right now it does so fairly well with the exception of the @cr-mysticky which remain visible in the narrative blocks after rendering. A few options:

Appeal to Quarto devs to add cr- as a protected prefix. Then we could have Quarto scrub them.
Switch closeread to a filter that could be run even on different formats. When run on those formats, all it'd run is a trigger scrubber.
Add options to closeread-html to also produce an alternate (non-scrolly) format.

Mmmm, good idea! I still love the idea of getting Closeread all the way down to a filter, but I can't remember now what our blocker on that was.

Haha, I was trying to remember why we decided to hold off on the filter for now and ... I really couldn't remember our rationale for sticking with a format for the time being!