Missing config details?

Hello, I implemented this using the suggested setup but I get no traces or events originating from this sdk in Sentry.
Am I missing something, or is there a conflicting order of integrations or?..
    "@sentry/react": "^8.11.0",
    "@sentry/vite-plugin": "^2.20.0",
    "@supabase/sentry-js-integration": "^0.2.0",

export function enableSentryLogging() {
  Sentry.init({
    environment: window.location.hostname.includes("localhost")
      ? "development"
      : "production",
    dsn: "xyz",
    integrations: [
      Sentry.browserTracingIntegration({
        shouldCreateSpanForRequest: (url) => {
          return !url.startsWith(`${projectURL}/rest`);
        },
      }),
      Sentry.replayIntegration(),
      Sentry.reactRouterV6BrowserTracingIntegration({
        useEffect: React.useEffect,
        useLocation,
        useNavigationType,
        createRoutesFromChildren,
        matchRoutes,
      }),
      Sentry.browserProfilingIntegration(),
      // Sentry.captureConsoleIntegration(),
      supabaseIntegration(SupabaseClient, Sentry, {
        tracing: true,
        breadcrumbs: true,
        errors: true,
      }),
    ],
    tracesSampleRate: 1.0,
    tracePropagationTargets: [...],
    replaysSessionSampleRate: 0.1,
    replaysOnErrorSampleRate: 1.0,
    profilesSampleRate: 1.0,
    beforeSend(event) {
       ...
      return event;
    },
  });
}

Hey, have you tried setting debug: true and see whether Sentry SDK is actually producing any spans?
Can you also provide any example event from the beforeSend callback, eg. by calling
const { data, error } = await supabaseClient.from("some-table").select("*").limit(1);
Sentry.captureException(new Error("boop"));

Any repro would be nice as well :)

This is part of the debug: true output in my localhost:

beforeSend config is just translations of the error form:
    beforeSend(event) {
      // Check if it is an exception, and if so, show the report dialog
      console.log("beforeSend", { event });
      if (event.exception && event.event_id) {
        Sentry.showReportDialog({
          eventId: event.event_id,
          title: "Przepraszamy, wystąpił problem!",
          ...
        });
      }
      return event;
    },

Example beforesend event:

only get these on thrown errors
I don't get supabase as category and only see queries when I inspect issues details:

and I cannot really find views similar to the showcase images.
My goal is simple - track all supabase interactions, butI feel like I am missing a key piece of the puzzle - do I need to upgrade from Developer Plan?
I attach my full sentry config, with some sensitive data ommitted.
sentry-config.ts.txt

do I need to upgrade from Developer Plan?

Not at all, it's available for free to everyone. If it collects breadcrumbs, it means that the integration is wired-up correctly.
What you're looking for I assume are traces with all the metadata for database queries, not errors breadcrumbs, and those should be available at https://_.sentry.io/traces/
If that's not the case, there's something off with the React/Sentry SDK interaction itself.
I'm more than happy to take a look at it if you're able to provide some sort of reproduction, as I don't have any react examples setup locally.

I'll try to prepare a reproduction env for you, thanks!

What you're looking for I assume are traces with all the metadata for database queries, - corret, I want all the data i can see in the example screenshots :)
I've created a stripped down version of my app in a new private repo, and send you an invite. You still need to provide your own supabase and sentry project credentials ;) (supa db only has 1 table, profiles with  address_line_1: string | null; address_line_2: string | null;)
I can make it a codesandox, but prefer not to unless necessary.

@krzysztofradomski thanks for the repro. I found the issue with the implementation that is caused by an API change of v8 SDK. Span metadata creation is performed slightly different now.
I'll fix it in ~2 weeks once I'm back home :)

hey @kamilogorek, any progress with this or anything where external help would be appreciated? :)

Hey @wyozi I did some research yesterday for it and already contacted Sentry guys for some help, as I did not follow their v8 SDK transition, and this integration needs some adjustment how span data is stored.
eg. this is working just fine in v7 (see the v7.js), but it fails miserably with v8.
const data = {
  "db.table": table,
  "db.schema": thisArg.schema,
  "db.url": thisArg.url.origin,
  "db.sdk": thisArg.headers["X-Client-Info"],
};

if (query.length) {
  data["db.query"] = query;
}

if (Object.keys(body).length) {
  data["db.body"] = body;
}

span = transaction?.startChild({
  description,
  op: `db.${operation}`,
  origin: "auto.db.supabase",
  data,
});

Easiest way to work on it and test the behavior:
clone https://github.com/supabase-community/sentry-integration-js
cd examples
make start

Go to http://localhost:3000/index-v7 and http://localhost:3000/index-v8, compare first 2 spans in Captured transaction log.
Any help is appreciated :)

Yup! That looks great. The only missing piece is origin, as right now it's listed as manual, wherein we should be able to give it a proper auto.db.supabase name: https://develop.sentry.dev/sdk/telemetry/traces/trace-origin/

Thanks to @wyozi everything works as expected now in SDK v8. Released a new version as 0.3.0. Cheers!
v7 trace (screenshot)

v8 trace (screenshot)


Can I check the type in another table?

Hey,
Instead of using the $childColumn, can i implement an own way of checking the type?
I'm using spatie/laravel-permission and it adds the role in another table.
Would appreciate if you can clarify on this.
Warm regards,
Akmal Hazim

Hey friends! The team at Tighten has taken Parental back over and we're re-opening the issue tracker, but to give us a clean slate, I'm going to close all old issues that remain from when Caleb closed the issue tracker in 2020. If this is still an outstanding concern with the latest version of Parental, please feel free to open a new issue referencing this one.
Thank you!


Reduce blending

Can we reduce the blending effect, to make some images have high contract?
And, if there is a way to generate larger text.
I have tried to change self.min_char_height to more than 8, but I am getting less words in the images; and so many are un-placed!

Yes, all of this can be achieved; it is not supported out of the box, but can modify the code to do so.


Branch naming convention

Name branches to what are and/or what they do

@perarnborg should we add any decisions regarding this to the contribution.md in the docs do you think?

@dennerkrans Yes, good point!


Disabling command aliases in rez env

Hi,
I have a Blender package configuration in my package directory and I would like to have a blender alias in zsh that would run the software in its environment like this :
alias blender="rez env blender -- blender"

However when I resolve an environment, the shell aliases are taken into account and therefore running blender never ends because it keeps calling itself.
Is it the right way to do this? If so, is it possible to disable shell aliases when inside a rez context?
Thanks

As a workaround, maybe this
def commands():
    alias("blender", "unalias blender; blender")
    ...

would work ?

mmh this doesn't work :
# packages/blender/2.93/package.py

name = 'blender'

version = '2.93'

description = 'Blender 2.93'

def commands():
    alias("blender", "unalias blender; blender")
    env.PATH.append("/home/josephhenry/Downloads/blender-2.93.0-linux-x64/blender-2.93.0-stable+blender-v293-release.84da05a8b806-linux.x86_64-release")

$ rez-env blender

You are now in a rez-configured environment.

resolved by josephhenry@pop-os, on Sun Jun 06 19:33:23 2021, using Rez v2.69.6

requested packages:
blender
~platform==linux  (implicit)
~arch==x86_64     (implicit)
~os==Pop-20.10    (implicit)

resolved packages:
blender-2.93  /home/josephhenry/packages/blender/2.93  (local)
$  blender alias blender
blender='rez env blender -- blender'

I believe in bash rez alias becomes a function.
So you probably need to unset not unalias it, right?

@bfloch Do you mean the unset bash function?
I tried:
def commands():
    command("unset blender")
    env.PATH.append(".../blender/bin")

With
alias blender="rez env blender-2.93 -- blender"

But it doesn't work either

I do not have something to test here but what about using the "" to temporarily disable the alias
something like
alias blender="rez env blender-2.93 -- \blender"
if that does not work you can have something along these lines
def commands():
    env.PATH.append(".../blender/bin")
    env.BLENDER_EXE(".../blender/bin/bledner")

then you can have something like
alias blender='rez env blender-2.93 -- $BLENDER_EXE'

@fnaum
Your example just escapes the b letter:

I do not have something to test here but what about using the "" to temporarily disable the alias

Did you meant that?
Also your second solution doesn't really correspond to what I am looking for since putting directly the path of the blender executable is the same as defining a custom shell alias (which I was doing before) and there's no point of using rez then.
But anyway I think I am going to close this issue since I can just launch the command with rez env!
Thanks ;)

Hi @johhnry , I was referring to using the \ to skip any alias and default to the command (picked on the path)
A basic example would be something like
> ls
file1
file2
> alias ls=echo
> ls "now ls is echo"
"now ls is echo"
> \ls
file1
file2

but maybe this is a behavior in bash and not in zsh
For the other suggestion, I just checked and I have some alias defined like this
alias bla='rez env package-version -c "\$TOOL_EXE"'

note that in this case the \ is just escaping the $ and you need double quotes so the env variable does not get expanded until it gets into the rez environment
If TOOL_EXE is the full path to an executable, this should work, as it will have priority over any alias defined


RPC failed with reason :nodedown

Running firezone-ctl create-or-reset-admin gives error:
RPC failed with reason :nodedown

Info:
This happens in AWS, Amazon Linux 2 (kernel 5.15 ) and Centos Stream 9 (Kernel 5.14 ), Arm64 arquitecture.
 This usually happens when the phoenix service is flapping, but it looks like yours is running fine. Could you show the full output of /var/log/firezone/phoenix/current? You should see Erlang BEAM crashes which were causing the :nodedown error.
If you try create-or-reset-admin now, does it work?

Also, can I ask how you're installing Firezone? Using the automatic install script or installing from our package repository?

Could you show (or check) the full output of /var/log/firezone/phoenix/current?

2022-08-06_16:10:02.25593 16:10:01.327 [notice] Application fz_http exited: FzHttp.Application.start(:normal, []) returned an error: shutdown: failed to start child: FzHttp.Conf.Cache
2022-08-06_16:10:02.25601     ** (EXIT) an exception was raised:
2022-08-06_16:10:02.25601         ** (Postgrex.Error) ERROR 42P01 (undefined_table) relation "configurations" does not exist
2022-08-06_16:10:02.25601
2022-08-06_16:10:02.25601     query: SELECT c0."id", c0."logo", c0."local_auth_enabled", c0."allow_unprivileged_device_management", c0."openid_connect_providers", c0."disable_vpn_on_oidc_error", c0."auto_create_oidc_users", c0."inserted_at", c0."updated_at" FROM "configurations" AS c0
2022-08-06_16:10:02.25601             (ecto_sql 3.8.3) lib/ecto/adapters/sql.ex:932: Ecto.Adapters.SQL.raise_sql_call_error/1
2022-08-06_16:10:02.25602             (ecto_sql 3.8.3) lib/ecto/adapters/sql.ex:847: Ecto.Adapters.SQL.execute/6
2022-08-06_16:10:02.25602             (ecto 3.8.4) lib/ecto/repo/queryable.ex:221: Ecto.Repo.Queryable.execute/4
2022-08-06_16:10:02.25602             (ecto 3.8.4) lib/ecto/repo/queryable.ex:19: Ecto.Repo.Queryable.all/3
2022-08-06_16:10:02.25602             (ecto 3.8.4) lib/ecto/repo/queryable.ex:155: Ecto.Repo.Queryable.one!/3
2022-08-06_16:10:02.25602             (fz_http 0.5.2) lib/fz_http/conf/cache.ex:27: FzHttp.Conf.Cache.init/1
2022-08-06_16:10:02.25603             (stdlib 4.0.1) gen_server.erl:848: :gen_server.init_it/2
2022-08-06_16:10:02.25603             (stdlib 4.0.1) gen_server.erl:811: :gen_server.init_it/6

Also, can I ask how you're installing Firezone? Using the automatic install script or installing from our package repository?

In AWS, t4g.nano instance, cloudinit script:
curl -1sLf "https://dl.cloudsmith.io/public/firezone/firezone/setup.rpm.sh" | bash
dnf install -y firezone

Thanks for providing those logs. cc @princemaple -- still getting some sort of rare intermittent race condition here. I'll take a look today.
@jobcespedes -- Does firezone-ctl reconfigure complete successfully for you?

I changed aws instance from t4g.nano to t4g.micro and there were no errors. I don't know why.

Does firezone-ctl reconfigure complete successfully for you?
I think so

Hmm. Resource limits shouldn't been the issue, but perhaps the postgres process is getting killed during migrations and the reconfigure isn't noticing? Possibly related: https://discourse.firez.one/t/cannt-install-firezone-on-new-server-try-ubuntu-and-debian/138/2
Does this instance have swap enabled? Any process killed errors in your kernel log?

No swap and yes there is an error in kernel log:
[  565.586846] Out of memory: Killed process 5241 (beam.smp) total-vm:2354500kB, anon-rss:74224kB, file-rss:0kB, shmem-rss:27844kB, UID:1001 pgtables:472kB oom_score_adj:0

Looks like order has been restored to the universe :-). I'll add a note to update our docs with these findings.

https://docs.firezone.dev/deploy/prerequisites#resource-requirements


Switch to DCO / Add DCO bot

Feature Description
Problem Statement: We have a PR template which includes the language "By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.". Presumably, this is to cover the same risk which DCO can address. Furthermore, there is no automation to verify this check, meaning its up to a reviewer to ensure this text exists in each PR.

Proposed Solution: Add the GitHub DCO bot to this project to enforce that all commits include necessary attestation for DCO. This aligns with wider CNCF and other OSS practices, while also giving us an enforcement mechanism for this attestation.

Alternatives Considered:

Additional Context:

Please vote on this issue by adding a 👍 reaction to the original issue
If you are interested in working on this feature, please leave a comment

Agreed, it's better to automate these with a github bot+++

I'm happy to provide some guidance on how to set this up - I've set it up on some other projects. Someone with admin on this repository will need to click the appropriate buttons to add the application.


"in" support

Is your feature request related to a problem? Please describe.
I'd like to use gazpacho for testing the HTML output of my Django application. A natural way to do this would be to search for a given HTML chunk. Therefore I'd like to be able to use the in operator on a Soup to check for that HTML chunk. This probably has utility outside of tests.
Describe the solution you'd like
I imagine this working something like:
response = self.client.get('/')  # django's test client
assert response.status_code == HTTPStatus.OK
body = response.content.decode()
assert '
Home page
' in Soup(body)

Describe alternatives you've considered
It's possible to reproduce this with find() and making assertions on the contents of the found node, but much more complicated since it requires assertions for each node in the tree.
Additional information
Django already supports a similar assertion called assertInHTML. However this relies on normalizing the HTML text and making text assertions, so it's clunky around matching the actual elements.

Interesting idea!
I don't think I fully understand, though, why would:
assert '
Home page
' in str(Soup(body))

Not work?

Interesting idea!
I don't think I fully understand, though, why would:
assert '
Home page
' in str(Soup(body))

Not work?

If there are two attributes in the HTML, what order will they come out of Soup ? What about whitespace, etc.?

If there are two attributes in the HTML, what order will they come out of Soup ? What about whitespace, etc.?


Data Validation

Do validation of user data (i.e. data sent to update (PUT) endpoints).
Ideally, I see the same logic used on both the frontend and backend with FluentValidation. This means, it would live together with the data models present in the shared PubHub.Common assembly and be used twice; once on the client (for user feedback) and then once again on the server (to ensure data integrity). Using the same validation rules allows for simplicity and consistency, contributing to easier maintenance and an implementation that is more easily kept secure against malformed data submission and potential threats.
I have already written custom components and helpers for property accessor parsing to make FluentValidation compatible with Blazor (only tested on Blazor Server though...). I don't know about MAUI.

Interesting. I agree. Let's try it out.
Also. Does this mean that we want to share View Models across assemblies?
Or do we want replicas in each executable?

Yeah, I might have been a little early on the PubHub.Common stuff. 😅
In commit 61c14b2 i added the assembly with a Models namespace. I though these could be shared between the API and the client applications making both sides agree to use the same classes (same excact structure and easy to maintain) to serialize and then de-serialize by the other.
They are our DTO's in a sense, but they could also be used as the models in FluentValidation. (In FluentValidation you create a validator from a generic class where the generic type you use, is the the model that is to be validated.)

It's fine but I feel like we need to discuss these kinda things before we make the choice by ourself as it affects the entire structure of the repository and what I've created (Based on the feedback I got in issue #44) is now to be moved out and into another assembly that I don't have on my branch 🤷


miniengine current updated frame

In various other examples, for example:
https://github.com/Microsoft/DirectX-Graphics-Samples/blob/master/Samples/Desktop/D3D12DynamicIndexing/src/D3D12DynamicIndexing.cpp#L593
A frameindex that points to the frame currently being rendered to is maintained along with a fence to avoid running too far ahead and updating a frame still being processed by the GPU. Is g_CurrentBuffer the same concept in miniengine? I'm not sure because I don't see similar fences, I'm probably missing something.

MiniEngine tracks fences after each command list execution.  These fences are used for resource reclamation.  They are also used for synchronizing the Graphics and Compute command queues.  Though they usually aren't used to synchronize CPU and GPU, they sometimes get used that way during resource creation time.  (E.g. load texture data, upload texture data, wait on fence, destroy original data buffer.)
g_CurrentBuffer is used to keep track of which swap chain buffer is being rendered to.  It's important for presenting the frames in the right order, but it's not used for fencing purposes.  There is a built-in mechanism to stall, however.  When you call Present(), you have a maximum frame latency of 3.  If you have submitted 3 frames already that have yet to be presented, the driver will idle the CPU until one of the frames is displayed.  You can also try using DXGI_SWAP_CHAIN_FLAG_FRAME_LATENCY_WAITABLE_OBJECT, which let's you change the maximum frame latency and also let's you decide when the CPU should be throttled.

Thanks for the reply! A bit of background: I've implemented something like the dynamic indexing example. (I use dynamic descriptors for some things but prefer that approach for the case of high instance counts). So like in the example I have N frames worth of descriptors but I'm uncertain of the best way is to ensure I'm rendering the "correct" frame of descriptors (versus updating them). I think ideally the Update() and RenderScene() would just have the current frame number as context when they call me. Does this makes any sense or is there some other solution you would recommend?

Yes, you can call Graphics::GetFrameCount() to determine the current frame index that the CPU is processing.  You could then somehow associate the frame index with your descriptors if you needed to.  For simplicity's sake, you could have four GPU-visible descriptor heaps (or just rotate through offsets into a single buffer), and then
Update DescriptorTable[ FrameCount % 4 ]
Set DescriptorTable[ FrameCount % 4 ]
Draw objects with DescriptorTable[ FrameCount % 4 ]
DescriptorTable[ FrameCount % 4 ] will persist for three more frames
Present() will increment FrameCount
But for even more simplicity's sake, you could pre-build a very large descriptor table with all of your descriptors.  The indices used to refer to individual resources will remain constant from frame-to-frame.  If a resource goes out of scope (i.e. is released) and thereby makes its descriptor invalid, make sure you don't use that descriptor index for the time being.  But if you stream that resource back in (like texture streaming), you would reuse that descriptor index.
I hope I didn't confuse you.  I'm not totally sure what you're intending, so please excuse me if I'm missing the point.

Thanks James - sounds like something along the lines of Graphics::GetFrameCount() % SWAP_CHAIN_BUFFER_COUNT is what I'm after.


fix: add ignoring increasing oids error for certain hosts

Description
This is a fix for an OID not increasing error returned from the SNMP device. This is being caused by wrong sequence of OIDs and is a problem on the device. This PR adds a way to ignore that problem via HELM configuration.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix
[ ] New feature
[x] Refactor/improvement
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

How Has This Been Tested?
I ran it locally and with Helm dry run. It will be tested on NOVA before merge with main.
Checklist

[x] My commit message is conventional
[x] I have run pre-commit on all files before creating the PR
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

Codecov Report

Merging #376 (418a66b) into develop (6c92353) will increase coverage by 0.04%.
The diff coverage is 100.00%.

:exclamation: Current head 418a66b differs from pull request most recent head e5f1809. Consider uploading reports for the commit e5f1809 to get more accurate results

@@             Coverage Diff             @@
##           develop     #376      +/-   ##
===========================================
+ Coverage    87.52%   87.57%   +0.04%
===========================================
  Files           23       23
  Lines         1347     1352       +5
===========================================
+ Hits          1179     1184       +5
  Misses         168      168

Impacted Files
Coverage Δ

splunk_connect_for_snmp/snmp/manager.py
94.13% <100.00%> (+0.10%)
:arrow_up:

Continue to review full report at Codecov.

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

:tada: This PR is included in version 1.5.5-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add request and response structs for wallet API

@sree-revoori1 yep, I feel you. Here's a draft to address this: https://github.com/tkhq/rust-sdk/pull/4. Hoping having something like this would help? Disclaimer: I haven't tried to use the generated code yet. Just verified that the generation works!

@r-n-o Thank you so much for the quick turnaround on this! This is super helpful, we will build off of the protos instead now.


Routing will not finish when clicking on router-link and the target route component is imported dynamically

Reproduction
https://github.com/BeniRupp/vue-router-async-import-bug
Steps to reproduce the bug
In a vitest test having a router with two routes, a component with a router-link and dynamic component imports for the route components:

setup router in the test
mount a component with a router link
click on that router-link with vue-test-utils' trigger('click')
wait for the route change
expect the new route fullPath

const AboutView =  () =><EMAIL_ADDRESS>// ...

it('should work', async () => {
    const router = createRouter({
        history: createWebHistory(),
        routes: [
            {path: '/', component: HomeView},
            {path: '/about', component: AboutView},
        ],
    })
    await router.push('/')
    await router.isReady()

    const wrapper = mount(
        { template: '<router-link class="link" to="/about">test</router-link>', },
        { global: { plugins: [router] }}
    )

    expect(router.currentRoute.value.fullPath).toBe('/')
    await wrapper.find('.link').trigger('click')
    await flushPromises()
    expect(router.currentRoute.value.fullPath).toBe('/about')
})

Expected behavior
The route fullPath after clicking the router-link is /about instead of /.
Actual behavior
The route fullPath is still /.
Additional information
I assume that the test will finish before the import resolves, because the router changes the route, calls the import function, but the import() statement does not resolve (see reproduction repo). Furhtermore, the click on the router-link causes a correct router beforeEach event but no afterEach event.
This problem is not reproducible if:

... using import AboutView from<EMAIL_ADDRESS>instead of the dynamic import
... router.push('/about') is called instead of clicking the router link

It looks like flushPromises() isn't enough. Adding a longer wait await delay(1000) with a setTimeout, works well. You need a way to await the dynamic import in your test. This is worth asking in Vitest discussions

Hi @posva, thank you for the fast response! 🚀
Damn, I didn't test the behavior with a timeout. Thanks for the hint! This is,at least, a temporary fix. 🙈


Fix typos and inconsistencies in classref

Fixes simple typos.
Wraps true with [code].
Capitalize ID and SDF.
Fixes unchanged description for DisplayServer.set_icon(), was copied from set_native_icon().
Removes hardcoded line wraps in Transform{2,3}D method descriptions.
Makes description of normalized(), lerp(), and cubic_interpolate() consistent across Vector{2,3,4}.

Thanks!


Ingress Create page UI Issues

[x] Name field title in Ingress: Create says "%Name%

[ ] "REMOVE" for Path fields focus highlight is huge and hidden by port field.  Should be like other remove field focuses form workload resources:

Ingress:

Workload resources:

[x] ADD PATH link should really be an Add Path button like the Add Rule.

[x] ADD HOST link in Certs tab should be button

[x] Move first Cert selection down to the REMOVE text isn't so close to the X.

[ ] Port title disappears  when I select a Target service (Default Backend tab)
Before:

After:

needs dev - “Port title disappears when I select a Target service (Default Backend tab)”

@codyrancher do you want a separate issue for the port title?  All Lauren's fixes have been verified.

@tfiduccia if it's more convenient for you that's fine with me. I can update the pr.

The last issue is resolved in the latest.
However I have noticed some new issues around the rule path Target Service and Port. I think these are being caused by changes made to the underlying components. I think this needs to be addressed across the board and not just within ingress.

Fixed - Will do another review of Ingress UI for the issues Cody mentioned and enter in new issue.


-e broken

I have -e but I think that -n is doing an override on this parameter
tv titanic.csv -e
only prints 25 rows

@5tefan @dufferzafar can you verify that tv <file.csv> -e is working on 1.4.5. I missed it on the last release. Just wanted to get a second pair of eyes before I close this.


Locale translation

Hello there, i come from Indonesia. Is there any possibility to make a contribution for the move book? i want to translate this to my locale.

Thank you, i will dot it ASAP

Forked it, if I finish the translation, will open a new issue in the future.

Hey @Qyuriz I'm closing the issue as completed. Feel free to open a new one when ready!


feature: Added Search Function to Navigation

I added a hot-search bar to the navigation component. It now filters out all categories and components by the search criteria.
When attempting to write tests for this feature, I found that the test infrastructure does not support JSX, and the Navigation component is structured differently than the other testing components. To do so, I would have needed to add additional packages to the repo that may have been unwanted.

sorry @kamrinkennedy, looks like the robot got to this before a human could review... I'm going to open it up again and try to recruit somebody to have a look at this

sorry @kamrinkennedy, looks like the robot got to this before a human could review... I'm going to open it up again and try to recruit somebody to have a look at this

looks like we got some travis tests failing...  or did those always fail?

looks like we got some travis tests failing...  or did those always fail?

@HauwaAguillard and I plan to pair on this next week.

@HauwaAguillard and I plan to pair on this next week.

@kamrinkennedy we jammed on this during our internal design meeting today. The search functionality is 💯% a tremendous improvement and works really well. We left a few suggestions and comments on this PR. After you've reviewed the change requests, this should be good to go from a design standpoint. Thank you!

@kamrinkennedy we jammed on this during our internal design meeting today. The search functionality is 💯% a tremendous improvement and works really well. We left a few suggestions and comments on this PR. After you've reviewed the change requests, this should be good to go from a design standpoint. Thank you!

Thank you! I’ll review it this evening.
On Wed, Jan 27, 2021 at 4:26 PM HauwaAguillard<EMAIL_ADDRESS>wrote:

@kamrinkennedy https://github.com/kamrinkennedy we jammed on this
during our internal design meeting today. The search functionality is 💯%
a tremendous improvement and works really well. We left a few suggestions
and comments on this PR. After you've reviewed the change requests, this
should be good to go from a design standpoint. Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/revelrylabs/harmonium/pull/601#issuecomment-768619503,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKNPCB4VGX2XP5FS3P4PXDLS4CHIJANCNFSM4USX5PTQ
.

Thank you! I’ll review it this evening.
On Wed, Jan 27, 2021 at 4:26 PM HauwaAguillard<EMAIL_ADDRESS>wrote:

@kamrinkennedy https://github.com/kamrinkennedy we jammed on this
during our internal design meeting today. The search functionality is 💯%
a tremendous improvement and works really well. We left a few suggestions
and comments on this PR. After you've reviewed the change requests, this
should be good to go from a design standpoint. Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/revelrylabs/harmonium/pull/601#issuecomment-768619503,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKNPCB4VGX2XP5FS3P4PXDLS4CHIJANCNFSM4USX5PTQ
.


Issues playing tunein stations

Not sure why it won't play most stations but heres some output:
https://gist.github.com/interbiznw/374ad9366e5fb139de7db212c3f9a7f8
Also just got this trying to play " CBC Radio One Vancouver" On tune in:
https://gist.github.com/interbiznw/28c62a887f1cdd7e7707aeb0822f3b86

TuneIn seems to be very broken right now I'm even getting errors on my Amazon hardware (TAP and Echo) I think they are moving stuff around ready for the UK/DE launch of the Echo this week.
I'd sit tight and I'll report back when its working on the Amazon HW then we can look into AlexaPi issues.

This project is now deprecated in favor of the new AlexaPi (https://github.com/alexa-pi/AlexaPi) which has all the features of this project and much more, such as:

support for a wide range of devices and platforms (Raspberry Pi, CHIP, Orange Pi, desktops, Magic Mirror and others)
voice activation (great PocketSphinx support and snowboy support soon to be released)
much more robust installation and configuration
better security
Arch Linux support
awesome documentation
and much, much more! You get it - it's just awesome.

Please switch over to the new project and if your issue persists there, file an issue in the new repo's issue tracker. Thank you.


Add ability to "pin" packages to a specific version

Many package managers have the capability to pin a package to a specific version. This allows someone to keep a package at a known working version even if there is a newer version available. It would be nice to have a pinning capability in appget.

Looks like this is a duplicate of #51


Notification drawer

The Notification Drawer is a content delivery system such as events, tasks, and alerts. It is a self contained system that is viewable without having to navigate to another area of the application. Please note, there is currently a bug in the pf3 notification drawer with the dropdown menu (see issue here). We want to ensure to address this issue in this new implementation.
Design folder: https://drive.google.com/open?id=1ZLl8gXAP7n5gkCdS6MN3W_Cf4lC1Ob-j
Updated wireframes: https://docs.google.com/document/d/1UH6J2Mf1Nqx4zUx-TzO6MCgPFq6NJatRb0kagmGN5yE/edit?usp=sharing
Component lifecycle:

[x] Interaction design requirements
[x] Visuals
[x] First checkpoint complete and approved
[ ] CSS
[ ] React
[ ] Second checkpoint complete and approved

Contacts:

https://github.com/patternfly/patternfly-next/pull/36

Working file
Static Screens

Thanks @mceledonia! These look great!

Notification drawer sketch + mockup example
Notification Drawer.zip

@mceledonia Looks good, but just a few comments/questions:

I'm OK with moving the "Mark All Read" and "Clear All" actions from the footer of the expanded section to the header. However the "Delete All" should be changed back to "Clear All" as the action is only to clear these items from the drawer, not to delete them permanently.

I do have concern with using the bright color bands at the left of each notification to indicate severity. While I like this for an alert, it seems like an overwhelming amount of color when the drawer is open, but that's just my opinion. Would be interested in whether others had the same reaction.

Regarding the link in the lower right hand corner, seems unlikely that we would have both the kabob and an action link, or if we did, they should be associated by proximity since the kabob would essentially be a place to hold more links/actions than could be exposed at once.

Curious why only one event includes a date/time stamp. Wouldn't each item have a title, description, and timestamp by default?

Let me know your thoughts.

@mcarrano Thanks for the feedback!

I wasn't certain about the "clear all" action, so I am more than happy to change that back. I initially was considering a confirmation dialogue (are you sure?) but knowing that it only clears and doesn't actually delete, I don't think that's necessary either.
The bright color bands were a choice to align with the alerts, and also to give some more visual prominence to the expanded area. The panel lacked a bit of hierarchy and felt a bit flat, check out the sketch file to see some alternate designs that had that problem. I agree it's a bit much seeing them together and I'm open to discussion around alternatives if we can come up with something better!
This link/button text was moreso included because I saw it in most of the case studies I was looking at for inspiration. The idea here was that each notification entry would have a collapsed and expanded state, and this expanded state revealed a sort of primary action on the notification itself. I wasn't sure what kind of actions or links typically are in the kebab menu, if it's typically just one or two items we might consider bringing them out with the button text instead. However I was assuming it was multiple, in that case the thought was to bring the primary action to the forefront with that button text and leave the leftovers in the kebab menu.
See above regarding expanded/collapsed state. I don't think including the time/date in the collapsed state is a bad idea! More of an oversight than a conscious decision to remove.

@mceledonia Good questions about the need for links vs. kabob actions. I will try to collect some examples from product to see how notification drawers are actually being used. I am curious what actions one can take here other than seeing full details of an alert. I think that having an expanded and collapsed state for a single alert is a good idea, however from looking at the mockup, it's not clear what the interaction is.
Let's continue to work on these details.

@matthewcarleton
Here is the latest updates
Editable File

@matthewcarleton
Latest Sketch file with the updates

Here are some wireframes for the notification drawer based on requirements gathered from product teams: https://drive.google.com/open?id=1S5uwkqLSVbYvckopRYkWEIpXjZJEKb06  Also reference https://github.com/patternfly/patternfly-next/issues/299 for more details about intended interaction design.

Note that complex variation (with categories) is a candidate for deprecation. Simple notification drawer will satisfy most use cases.

I have created a set of updated wireframes that can be found here: https://drive.google.com/open?id=1nU9dpAbGiKSRIhK3GPx52xJQH_578YAB  These should be used as a basis for updating Visual designs. @rachael-phillips @kybaker FYI

Based on @mcarrano 's updated wires and some communication with @kybaker , I updated visuals for this! I have mockups default and on-scroll states for both the 'simple' and 'complex' notification drawer. Those and their respective specs can be found in my prototype here: https://redhat.invisionapp.com/share/7FQDO4RP462#/345357487_Examples
Here's the gist of the simple and complex drawers:
Simple

Complex

@maryshak1996 This is looking good. I left a few comments on the InVision doc.

thanks @mcarrano ! I replied to your comments -- thanks for the feedback so far!

@mcarrano @matthewcarleton I have updated my invision prototype with some updates, based on some great feedback from you guys and others :)
Simple

Complex

And I do want to also call some attention to how we are proposing indicating an unread notification in the masthead (this is also shown within the invision's "Visual Spec' page)

This is looking great @maryshak1996 !  Just one question about icons. In the example I see the you changed the icon for notification settings to the sliders, but in the masthead we show a gear icon. It raises two questions: 1) Which is the proper icon for settings and should we use it consistently? 2) It does seem weird to see these two settings icons in proximity. The purpose of the settings icon in the header of the drawer was for notification settings specifically. But does it make sense for all settings to be accessed globally from the masthead? Or should these be differentiated? Would be interested to hear what others think.
Also, I think we want to start removing real people's names from these mockups because we will eventually publish these to the website. @gdoyle1 did we ever decide what to do about that in our mocks?

@mcarrano I never ended up updating those in the master sketch file but I can do that. When we finish the symbol library, we will definitely use fake names!

@mcarrano I was looking through what we had in the master sketch file icons page, and that page is calling for 'Settings' to use that icon. I'm not sure if that is just out of date, or if there is a greater consistency issue here.
@kybaker do you have any thoughts on that icon topic?

@mcarrano @kybaker @gdoyle1 I did a little bit more iterating and I think that I can put the done stamp on the visual side of this! The invision prototype is up to date and here is my latest updated designs to take a look at:

Visual spec can be found on the invision link or in this PDF
notification-drawer_visual-spec.pdf

https://redhat.invisionapp.com/share/7FQDO4RP462

@maryshak1996 I like the additional background color change for unreads! Looks great

Overall this looks great. I like the changes you made @maryshak1996 . I just had one question about the location of the footer in the complex drawer that I left in InVision.

Thanks @gdoyle1 ! and @mcarrano thanks for pointing that out, what you suggested is what I had in mind in the first place and it was just an oversight on my part :)
https://redhat.invisionapp.com/share/7FQDO4RP462
@mcoker I think that the design on this is complete, and here is the most up-to-date spec
notification-drawer_visual-spec.pdf

Looks good to me @maryshak1996 . @mcoker can you take a look and let us know if you see any issues that would affect implementation of this? @rachael-phillips pending that, I think this can move to CSS as soon as someone can take it.

Thanks for letting me know! I'll be on the lookout for what @mcoker has to say!

Thanks @maryshak1996, this looks great! A couple of questions:

It looks like the individual notifications are what scrolls - whether there are just notifications in the drawer, or those notifications are in a notification group. What happens if the number of groups overflows the height of the drawer? Would that add an additional scrollbar, so you have an outer scroll to scroll the groups and an inner scroll to scroll notifications within a group? FWIW, looking at PF3, if the notification "tabs" (groups) overflow the window height, they're just hidden and you can't access them. (The scrollbar on the right in the screenshot is just for the overall page, not for the notification drawer)

If you click the kebab icon and open the dropdown of an alert in the drawer, is the dropdown menu expected to display inside of the drawer (if it's not in a group) and/or group (if it's in a group)? If the dropdown is expected to display outside of either, that might complicate things as it's difficult to get children of a scrollable area to display outside of the scrollable area, though I think that would be possible to do with the react component (I would want to double check to make sure). Looks like PF3 displays the dropdown within the drawer/group as I would expect, which is easy to do.

Let me try to answer these questions @mcoker .

* What happens if the number of groups overflows the height of the drawer?

The likelihood of this is small. If groups are used at all (in most cases they won't be) it would normally be a small number (less than 5). In most cases this will work with the current accordion approach. I would not be in favor of extra work to add an additional scrollbar outside the group.

* If you click the kebab icon and open the dropdown of an alert in the drawer, is the dropdown menu expected to display inside of the drawer (if it's not in a group) and/or group (if it's in a group)?

You are correct about the PF3 behavior, but this had actually been raised as a bug. We deferred it because of the difficulty to fix. I do want to do something better for PF4. If there is not enough room to display the menu inside the boundary of the drawer, is it possible to just pop the menu up (above) rather than down? How do we handle this in similar situations, i.e. in a list or table when you open the menu from the last row and there's not enough space to display it?

If there is not enough room to display the menu inside the boundary of the drawer, is it possible to just pop the menu up (above) rather than down? How do we handle this in similar situations, i.e. in a list or table when you open the menu from the last row and there's not enough space to display it?

We have a variation of the dropdown type components that aligns the menu above the toggle, but whether the component is smart enough to detect the available space and whether the menu should open on the top or bottom of the toggle, I'll need to defer to react devs. @tlabaj do you know?
If we aren't going to have a scrollbar to scroll groups, and the notification item dropdown will open inside the notification drawer and/or group, then this looks like it's ready to build to me. Seems like the dropdown could possibly open outside of the drawer/group, too, if we did something in react that allowed the dropdown menu to exist outside of the notification drawer in the DOM. Maybe something similar to the appendTo prop for tooltips.

@mcoker The Dropdown component does not detect space and automatically position it above if there is not enough space.
@jschuler worked on tooltip positioning do you.  Do you think we could do something similar  for Dropdown (see @mcoker comment above)?

@mcoker after some discussion with a larger group, there are a few concerns that were raised about the current design that we will likely require some design updates that I wanted to document here, so that you are aware before getting too knee-deep in the development. Here is a break down of the feedback given, and the solutions that I am proposing to resolve the issues can be seen on my updated invision prototype:

we need a visual indicator associated with the bell icon in masthead to show that the drawer is open
There is inconsistencies with the badging of 'read' vs 'unread' notifications
The 'close' icon may not be necessary
Should we consider relocating the "Mark as read" and "Clear all" actions into the gear? Should the gear instead be a kebab?
How do we deal with long titles? Truncation? Stacking?
https://redhat.invisionapp.com/share/7FQDO4RP462#/345385129_Notification_Drawer_-_Complex_Example3

@mcoker @rachael-phillips I think that all of the feedback from product groups has been considered in these latest mocks above, and we can consider [again] this done from the visual side. sorry for the confusion!

a few designers have asked for the sketch file for this, so here she is in all her glory along with the visual spec
Notification-Drawer.zip
:

@maryshak1996 This is looking good. I like your changes. I only had one comment about how we are reflecting the state of the drawer (open vs closed). Did you also pass your Sketch assets along to @gdoyle1 for integration with the master file?
@mcoker please look over this design when you have a chance. I'm wondering how we will approach this from an implementation perspective. I think the drawer itself can be a component. But this also involves integration with an element living in the masthead. Should we also create a demo to show the drawer in context?

@mcarrano yes I did! @gdoyle1 has my sketch assets for this

This looks great! Re: implementation, the toggle looks like a button with a badge. Do we want to build something that serves as the badge that can be used elsewhere in the app, or build a notification drawer toggle that has the icon and badge built in?
Also, how is the drawer supposed to be positioned relative to the bottom of the page? Should it extend to the bottom of the viewport? The PF3 notification drawer nearly extends to the bottom of the page with a 20px gap.
We'll definitely want a demo showing it context. That may need to live in the react workspace, too, since it looks like the positioning of the drawer is going to need to rely on javascript to compute the height.

@mcoker Addressing your questions...

Do we want to build something that serves as the badge that can be used elsewhere in the app, or build a notification drawer toggle that has the icon and badge built in?
Good question. It's possible that there is a general case for adding a badge to any icon (although I can't think of one offhand), in which case we should make this a component unto itself or a variation of the icon button. @kybaker do you have thoughts on this? The badge applied to an icon is not something we currently have as part of the component set and the existing Badge component does not really work for this.

Also, how is the drawer supposed to be positioned relative to the bottom of the page? Should it extend to the bottom of the viewport?
Yes, the drawer should size to the height of the viewport. I have created an additional issue to create a demo that reflects what the drawer looks like in the content of a page: #1522

quick update here: after some stakeholder feedback, I updated the indicator for opened vs unopened drawer in the masthead
Previously the only only masthead indicator was a white outline around the blue 'unread' circle, but the point was raised that the user should see a visual indicator that the drawer is opened even when there is not an unread notification.
What I am proposing now is having the color of the bell itself changing to pf-blue-50. This is a subtle change from the default masthead icon color and we see opened/expanded items turn blue when selected elsewhere (however the selected-blue color is not compliant over the masthead color, hence using the lightest shade of blue). Here's what it would look like:

My invision above is updated to reflect this

This is pretty subtle @maryshak1996 . Not sure that I would have noticed a difference if you did not point it out. What do others think?

I agree with @mcarrano
Is the intention for the circle to be present when the drawer is open? I could see removing it and changing the bell icon to blue.
If I've opened the drawer and "seen" all of the new notifications but haven't explicitly marked them as read does that circle remain when I close the drawer?
I could see an argument for removing it and then adding it again if another notification is present. Alternatively the circle could be removed when the drawer is open and replaced when it is closed again, which would allow more flexibility on the design of the bell icon.

@mcarrano @matthewcarleton
I do agree that it's very sublte, and I think that this something that we may want to consider addressing more globally in reference to the 'selected vs unselected' behavior of all utility icons (so the cog as well, launcher, or other icons that various products might use up there). What do you think?
With that said, I think that we should have the 'selected/unselected' visual indicator be seperate from the blue dot, since that is not something that other icons would have nor something that would appear when there are 0 unread notifications -- thinking about the 'selected/unselected' visual treatment should probably be considered externally from that blue dot for these reasons in order to have a standard that is consistent across utility icons

I'll keep tweaking how we represent 'selected/unselected' with the bell for now and reach out to a few other visual folks too :)

Hi @maryshak1996 how is this issue going?

@rachael-phillips hey sorry about the hold up on this! I have a couple of options for this last piece (opened vs unopened utility item), that I'm just working to refine :) should have the final design recommendation here shortly!

@rachael-phillips I also have this in my queue to work on an interaction spec. But once Mary is done, I think there will be enough to hand over to CSS if/when there's a resource to take it.

@rachael-phillips here's what we'd like to do!

That sounds great. Thanks for the update!

This component looks good! I really like the guidelines you are working on here, however let's not tangle together icon, drawer and notifications together.
Let me propose an idea and let's think about it to see if it's something reasonable or not.
Notification icon/text
I really like the idea of icon that indicates some changes has been done or happend since user last went to specific part of application, so I would really like to see a component that allows me to add icon ro text and I'd see this icon with blue circle at top to indicate that user should click on it and see what is there.
Drawer
Please do not think about drawer as notification drawer specific, this is really good design guideline, but not good component. We should have a component that is just static drawer opened on any place with static width and some other properties [1]. What the consumer is going to add to this component is really not our business, we can guide them and tell them which components are good, but they can choose whatever they want in there.
Noification drawer
With these things said I think that the best approach to implement notification drawer would be to implement it using these components

Notification icon - new component
Drawer on right side - new component

Accordion [2] - different header, but same component

Notification group [3]

Why these changes you might ask? What if someone doesn't realy likes bell icon, but would rather use inbox icon? What if they want to position the drawer on left side? What if someone would like to use sortable table with collapsible items as notification groups? All these questions might come in future we'd have to write specific components, or consumers will have to write specific component based on what they really want. Let's give developers freedom of choice and provide them with guidelines, but allow them to break these guidelines.
[1] https://material-ui.com/demos/drawers/
[2] https://pf4.patternfly.org/components/Accordion/examples/
[3] https://pf4.patternfly.org/components/AlertGroup/examples/

@karelhala I like this proposal. While I'm not in favor of having many variations of the Notification Drawer pattern, I agree that, in general, flexible/modular design is a good thing. I can see more generalized components like a notification icon and a sliding drawer as applicable to a variety of use cases, and therefore it makes sense to build in that way.
@mcoker, @matthewcarleton  and @tlabaj I'd be interested to know what you think? Ultimately I see this as a development decision, but I can easily factor the design into more modular pieces if you think that makes sense. The Notification Drawer would then be more like a demo that puts these pieces together.

I see a lot of value in breaking this out since we will have use for these things on their own. My biggest question is can we implement this in a demo if we have specific needs on the accessibility side? @jgiardino based on the work you've done so far with alerts do you have any thoughts on this?

I'm also in favor of @karelhala's proposal.
@matthewcarleton The main a11y thing with dynamic alerts is handling the live region. But the notification drawer shouldn't need to include live regions. The a11y patterns that I think apply here are the relationship of the icon in the masthead with the drawer, and what role the drawer has. It isn't clear to me if the notification drawer displays besides other contents or on top of other contents. If it's beside contents, we would probably use role="aside". But if it's on top of contents, the approach I've been taking is to use a similar pattern that we use for modals or popovers, which uses role="dialog" and traps focus.

thanks @jgiardino That makes a lot of sense.
I think we'd just have to figure out how this differs from the current "alert group". I can see some confusion there if we have a component for the notifications inside the draw and then another one for the alerts outside of it when they are grouped. Probably just needs some brainstorming :)

+1 to @karelhala's idea of separating into components, sounds like a smart approach.

Based on the ideas included here and some follow-up discussion, some other items should be considered. This include:

Whether we need a way to restore an alert that was cleared accidentally.
How to display a neutral alert state. Issue #1749
Mobile considerations?
Filtering drawer contents

Thanks for the suggestions on how to break this component down @karelhala
@mcoker I've created, linked and assigned you those issues.

Latest designs from @maryshak1996 https://redhat.invisionapp.com/share/7FQDO4RP462#/345357488_Visual_Spec

I have added the demo tag to this issue and will close #1522 as a duplicate now that we have broken this into smaller components. I agree with the decomposition but we will still want to have a demo that shows these pieces working together.

I've updated the invision and visual spec based on some collaboration with @mceledonia

https://sketch.cloud/s/y50lz

With the switch to marvel, the most up to date stuff can be found here: https://marvelapp.com/project/4090736/

Draft of usage document can be found here: https://docs.google.com/document/d/1kccmdwXgnkLZdH5OY_sr6C91Gsm0gCtF6zp9OBwC-dE/edit?usp=sharing

I really want to stress out that we can't limit ourselves to drawer itself. I should be able to place notifcation accordion wherver I want - dialog, full page w/e. And drawer itself can contain whatever we want. I guess that everybody already know how drawer will look like side component that is over other component. The drawer should be flexible enough to allow app devs to set its width (either percentage of page or fixed doesn't matter all the way to full page view). And anything can be part of drawer it doesn't really have to be notifications. We should start with drawer itself and once it is done we will understand much more other parts of this notification drawer demo.

@karelhala Thanks for your detailed comments on the design/usage doc that I posted above. @mcoker @dlabaj please take note as they mostly address how the drawer component is implemented.

@karelhala thanks for the feedback. Have you seen the drawer component yet? Also here is a demo of the drawer in the page component.
The notification list is something that will be placed inside of the existing drawer component. Do you see any problems with that?

@mcoker was not aware of the drawer component, nice! Yes, that is absolutely correct to place notification list in the drawer, just bear in mind that it can be placed anywhere else.

closing in favor of https://github.com/patternfly/patternfly/issues/3186


glyph class as list and tuple instead of set and frozenset

Fixes #504

Coverage decreased (-0.04%) to 70.488% when pulling 4ec59b27dd0a09f1e1a123bbfcfd160377d6d2ae on moyogo:feaLib-glyphclass into 4c92c4cd62f7dc2fcb75efee910350754a08f882 on behdad:master.

Coverage decreased (-0.009%) to 70.515% when pulling 84e7423a6e2e0187b7934dfea3e44b00f389771b on moyogo:feaLib-glyphclass into 4c92c4cd62f7dc2fcb75efee910350754a08f882 on behdad:master.

👍


flask calculator incorrectly averages flask charge gain

for example, a 14 second duration flask is incorrectly assumed to gain 14 seconds of flask charges, when in fact, it will gain:
12 seconds of every 3 seconds flask charge gain mod (pf)
10 seconds of every 5 seconds flask charge gain mod (jewel)
a 15 second duration flask gains:
15 seconds of every 3 seconds flask charge gain mod (pf)
15 seconds of every 5 seconds flask charge gain mod (jewel)
(i learned this by looking at pob source code)

@banana-isad  correct, this is a low effort calculator

@banana-isad  this is actually fixed now.


UIScrollView - add the ability to hide scroll bars

It would be good to be able to hide the appearance of scroll bars in UIScrollView.  Or by default have them be not visible.

We'll address this in the coming months but this is not high-priority for now.

Thanks for the feedback. We checked and confirmed it's already possible to call UIScrollview or its subclass using set ShowsHorizontalScrollIndicator:NO or set ShowsVerticalScrollIndicator:NO or using property syntax access to hide horizontal/vertical scrollindicator.
Let us know if you're running into any issues using this.


pull request for e2e test at 01/30/2017 04:45:12

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


fix(sbb-header-action): updated readme.md in sbb-header-action

fixed typos in the readme.md-file.
Preflight Checklist

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)

Changes
Changes in this pull request:
fixed typo in readme.md file (sbb-header-action)
Pull request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] Documentation content changes
[ ] Other (please describe):

Does this introduce a breaking change?

[ ] Yes
[x] No

Codecov Report

Merging #1775 (4c88b9f) into master (ab917b0) will increase coverage by 0.47%.
The diff coverage is 57.16%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master    #1775      +/-   ##
==========================================
+ Coverage   54.85%   55.33%   +0.47%
==========================================
  Files          49      162     +113
  Lines        1659     6414    +4755
  Branches      406     1759    +1353
==========================================
+ Hits          910     3549    +2639
- Misses        671     2638    +1967
- Partials       78      227     +149

Impacted Files
Coverage Δ

...ts/sbb-accordion-item/sbb-accordion-item.events.ts
100.00% <ø> (ø)

...mponents/sbb-accordion-item/sbb-accordion-item.tsx
0.00% <0.00%> (-37.21%)
:arrow_down:

...onents/sbb-autocomplete/sbb-autocomplete.events.ts
100.00% <ø> (ø)

src/components/sbb-calendar/sbb-calendar.events.ts
0.00% <0.00%> (ø)

src/components/sbb-checkbox/sbb-checkbox.events.ts
0.00% <0.00%> (ø)

...components/sbb-datepicker/sbb-datepicker.events.ts
0.00% <0.00%> (ø)

src/components/sbb-datepicker/sbb-datepicker.tsx
49.18% <ø> (ø)

src/components/sbb-dialog/sbb-dialog.events.ts
100.00% <ø> (ø)

src/components/sbb-dialog/sbb-dialog.tsx
44.96% <ø> (ø)

src/components/sbb-divider/sbb-divider.tsx
100.00% <ø> (ø)

... and 30 more

... and 133 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


NodeId-bound cluster connections

A connection, that is obtained for a particular nodeId, should be stable and survive cluster reconfigurations.

Renamed issue from slot-bound to node-id-bound. My bad for the initial wrong name

Motivation: Redis cluster nodes can be identified by three-and-a-half means:

NodeId
Host and Port
Slot

The half: Master/slave state for a certain slot

The identification details can be moved/changed at runtime. Most prominent examples are slots and master/slave state. A certain nodeId can be moved as well from one host/port to another one. The previous implementation did not care too much about that fact; all connections were identified by host and port.
While moving a certain nodeId from one host to another is quite unlikely, it still might happen. The connection pool, therefore, distinguishes now between host and port-bound and nodeId-bound connections. Host and port-bound connections stick to the particular host/port. NodeId-bound connections are reconfigured once the cluster topology changes.


[FEATURE] Refactor map clusters to use tailwind instead of css modules

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
https://github.com/EddieHubCommunity/BioDrop/tree/main/components/map
Cluster.js uses css modules for styles.
Proposed Solution
We can refactor it to use tailwind classes for consistency?
Screenshots
No response
Do you want to work on this issue?
No
If "yes" to above, please explain how you would technically implement this
No response

great idea!

Hi @amandamartin-dev , can I work on this issue?

Hi amandamartin-dev , can I work on this issue?

@Apurv428 thank you for your enthusiasm to work on this project.  Please avoid tagging a specific maintainer, as this creates unnecessary extra notifications.  All maintainers receive notifications and can assign an Issue.
I can see that you have not yet contributed to this project.  We ask new contributors to start off with issues between 1-3 Points (this is 5 points).

Sure, how can I identify a problem within a range of 1 to 3 points? Also, sorry for tagging!

Sure, how can I identify a problem within a range of 1 to 3 points? Also, sorry for tagging!

The Issues ready for dev have a label called "Point: [number of points]" in the issues tab

All issues identified with points 1, 2, and 3 have been allocated to a contributor for resolution.

All issues identified with points 1, 2, and 3 have been allocated to a contributor for resolution.

@Apurv428 I suggest every once in a while you take a look at the unallocated issues as we regularly have new ones coming up.  Please come and chat to us about these types of topics on the EddieHub Discord, rather than in the comments to a specific Issue - as it is best to leave the comments here dedicated to the Issue.

Thank you for your suggestion, and I apologize for not being aware of the existence of a Discord community. I will be more cautious moving forward.

So in Cluster.js, wherever we are using styles, we have to use tailwind components? Basically, we have to get rid of the Clusters.module.css?

So in Cluster.js, wherever we are using styles, we have to use tailwind components? Basically, we have to get rid of the Clusters.module.css?

Yep

Hello, can i work on this. I have some experience in tailwind and i've contributed to biodrop before

Hello, can i work on this. I have some experience in tailwind and i've contributed to biodrop before

@bikrantjajware thank you for your enthusiasm, but we do ask contributors to have worked on a few Issues between 1-3 Points (beyond creating their BioDrop profile) before assigning Issues with 5 Points, like in this case.

hello, can i be assigned to this issue?


Documentation is outdated, and nothing compiles

#include <reactive/ObservableProperty>

is no longer valid. Now it's
#include <reactive/ObservableProperty.h>

Just one line

#include <reactive/ObservableProperty.h>

already breaks the build process.
This is yet another so-called 'reactive' library I found, and none simply 'works'.

Sorry for this inconvenient. What compiler do you use, and what is compiler output?
P.S. I just tried latest mingw-w64 gcc 7.1, compilation is now broken. It seems this is due to apply disambiguation (gcc sees it as if it be in global namespace). VS 2017 and clang 5 compiles fine.

Please try now, if my assumption was correct, it should fix your problems.

I actually rolled out my own implementation for reporting back then in September, so this issue can be closed :)

Is it in public domain? Can it be watched?

Yeah. It's part of 'mjin-report', part of the game tech I'm working on. Here's a sample test of its usage: https://bitbucket.org/ogstudio/mjin-report/src/tip/include/mjin-report/Test.h?at=default&fileviewer=file-view-default
Though, I ended up with reporting only without passing values, because passing values breaks when we deal with arrays: many questions arise, many corner cases, so I just limited the application to reporting only.

Yep.


Feature/spatial view change list split

Description
Refactored the way ops are processed to have an explicit intermediate format.
This is mostly moving things that touch ops directly from ViewDelta into another class. Mostly to make ViewDelta.cpp a little less difficult to read and remove some of the bloat.
This also slightly decouples the exact nature of engine ops from the view delta construction, as the view delta cares about some synthetic format to describe changes to the view.

Is this mostly for breaking things up (which is great) or are there other benefits? Is the idea that we're going to use this intermediate type elsewhere, for e.g. callbacks? or creating changes internally to be short circuited?


[4.2] Export Window Sized outside of screen, cannot be resized when export template is selected

Godot version
v4.2.stable.official [46dc27791]
System information
Godot v4.2.stable - Windows 10.0.19045 - GLES3 (Compatibility) - NVIDIA GeForce RTX 3080 (NVIDIA; <IP_ADDRESS>17) - AMD Ryzen 5 5600X 6-Core Processor (12 Threads)
Issue description

Steps to reproduce
Just converted my 4.1 project to 4.2, wanted to export immediately as the given project is just a gateway for my server.
Minimal reproduction project
n/a

Fixed it by clicking "Sources", which allowed me to resize the window, had to scroll resize it multiple times until it fitted my screen, then went back to "Options" and now I can resize all export windows in other projects too.

I can't reproduce it, so exact conditions are unknown. It's probably another sizing bug with the window's contents, but to debug it we would need to figure out how to make this repeatable.

I have been consistently getting this issue by doing the following, even on a completely blank project :

Go to Project > Export...
Add a Linux export (no need to even download the templates)
Go to Resources
Change the Export Mode to Export as dedicated server
The export screen gets huge, and when in Single Window Mode there is no way to access the buttons all the way down.

Work around : From the editor settings, disable the Single Window Mode setting. It becomes easier to resize the Export window. You'll need to drag the top of the window down several times to bring it to a reasonable size.

I'm also facing this problem on Android. How to fix? I could use it just fine before


'portable' --> 'not version-specific'

Fixes #20562
Related to https://github.com/dotnet/core/issues/3554
"Portable" remained in the doc only as the group title for a group of RIDs that aren't OS-version/distribution-specific; I changed these subheadings to:

Windows, not version-specific
Linux, not-distribution-specific
macOS, not version-specific

@ericstj @agocke Who are the right people to look at RID graph documentation changes?


Error in Documentation

I think there is an error in the documentation in chapter "Create a new web part project".
"What is your Web part name?" should be "jQuery" and not " jQuery Web Part" otherwise the code samples aren't correct.

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

ID: 9f5a35c2-9914-fc67-1550-15d4f63c918c
Version Independent ID: 8418c398-1a93-622d-4df8-84c866fef67f
Content: Add the jQueryUI Accordion to your SharePoint client-side web part
Content Source: docs/spfx/web-parts/get-started/add-jqueryui-accordion-to-web-part.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Mainly wrong description here:

Should be src/webparts/jQueryWebPart if the web part name is jQuery Web Part

Sorry, but the error is still in there.

This should be updated to: What is your Web part name?: jQuery

It's fixed, it's just not live on the docs site. The live branch reflects what's currently on the .COM public site. The PR applied the changes to the master branch which will get merged into the live branch. This is usually done on a scheduled basis. The person at Microsoft who usually does this is currently on vacation, so it might take a bit longer as only urgent things will likely get pushed out.
The issue was automatically closed because the person who submitted the PR linked the issue (correctly) to the PR so it was automatically closed when the PR was merged.

Ok, I'm sorry. It was the first time I've reported an issue. :-)


SSL issue for streamtip.

http://pastebin.com/ViHByjPJ
PotatoOrgy was having this issue. I don't know of any other streamtip users to test with to see if it's local or not. Perhaps you know more.

From StackOverflow

In most cases, this is because the root CA's certificate is absent in the trust store, leading to the situation where a certificate trust path cannot exist; the certificate is essentially untrusted by the client. Browsers can present a warning so that users may ignore this, but the same is not the case for SSL clients (like the HttpsURLConnection class, or any HTTP Client library like Apache HttpComponents Client).

Aka he's computer does not trust the certificate. It may be invalid, close to expiring.
OR his antivirus is intercepting the connection and tries to apply its own certificate to the connection. If that certificate isn't accepted by java, it will fail. Perhaps adding an exception to not scan https://streamtip.com/api/tips?*

Alright. Ill tell him.


AttributeError: 'method_descriptor' object has no attribute 'today'

Checking local filename...
Traceback (most recent call last):
  File "PixivUtil2.py", line 687, in process_image
    PixivHelper.writeUrlInDescription(image, __config__.urlBlacklistRegex, __config__.urlDumpFilename)
  File "/home/nho/workshop/PixivUtil2/PixivHelper.py", line 636, in writeUrlInDescription
    filename = datetime.date.today().strftime(filenamePattern) + ".txt"
AttributeError: 'method_descriptor' object has no attribute 'today'
Error at process_image(): (<type 'exceptions.AttributeError'>, AttributeError("'method_descriptor' object has no attribute 'today'",), <traceback object at 0x7f1b359afa28>)
Dumping html to: Error medium page for image 59621214.html
Stuff happened, trying again after 2 second ( 2 )
Traceback (most recent call last):
  File "PixivUtil2.py", line 403, in process_member
    result = process_image(mode, artist, image_id, user_dir, bookmark, title_prefix=title_prefix)  # Yavos added dir-argument to pass
  File "PixivUtil2.py", line 687, in process_image
    PixivHelper.writeUrlInDescription(image, __config__.urlBlacklistRegex, __config__.urlDumpFilename)
  File "/home/nho/workshop/PixivUtil2/PixivHelper.py", line 636, in writeUrlInDescription
    filename = datetime.date.today().strftime(filenamePattern) + ".txt"
AttributeError: 'method_descriptor' object has no attribute 'today
'```

It's normal UI.

try the latest commit.

It appears to be working.


[Bug]: Custom endpoints with same Base URL use first key [LiteLLM]

What happened?
When using custom endpoints with different keys but the same base URL, only the first keys is used. This baseUrlToNameMap object groups by base URL only .This breaks down when using a service like LiteLLM, where the base URL is the same, but the API keys allow access to different models.  I would expect name or apiKey to be the unique key for the map here.
https://github.com/danny-avila/LibreChat/blob/2ea6e8c18a65ccb9c3aa93ea3e3ba9db9ad4adb5/api/server/services/Config/loadConfigModels.js#L45-L69
Steps to Reproduce

Create multiple keys in LiteLLM with access to different models
Create associated custom endpoints in LibreChat with different keys but same base_url

    - name: "LiteLLM"
      apiKey: "${LITELLM_ALL_MODELS}"
      baseURL: "${LITELLM_HOST}"
    - name: "OpenAI"
      apiKey: "${LITELLM_OPENAI_MODELS}"
      baseURL: "${LITELLM_SECOND_HOST}"
    - name: "Google"
      apiKey: "${LITELLM_GOOGLE_MODELS}"
      baseURL: "${LITELLM_SECOND_HOST}"

What browsers are you seeing the problem on?
No response
Relevant log output
No response
Screenshots

Code of Conduct

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

Thanks for investigating this. Merging a fix soon from #1985

Let me know if that does the trick for you!

Works perfect, thanks!


After installation stuck on loop

Hey,
First of all thank you for EFI. I used your EFI to install macOS Sonoma 14.5, and the installation went smoothly. However, I'm encountering a boot loop issue right after installation (before reaching the macOS setup menu where you select language and Apple ID). I've tried booting with -v for verbose mode but haven't had any luck.
My specs are similar:
Asus ROG Strix Z790-A Gaming Wi-Fi D4
Intel Core i7-13700KF (13th Gen)
AMD Radeon RX 6600
Any suggestions on how to resolve this?

Try clearing NVRAM and keep trying, also updated to 1.0.2. Should be a lot of fixes for your system there :)


Licence key

I want Licence key please

License key please

the website not working and can't get License key !!!

license key pls

Hello, may I kindly ask you for license key provision?
Thank You!

license key pls

Can i get a key pls, website is down

The license key 🗝 to start the app even trying purchasing doesn't work

I can provide you working licence key for $10.


Callback Error syncAccounts - typeError: Cannot read property 'migratedTransactions' of undefined

Issue description
App Version: 1.3.2
platform: macOS
platformVersion: 11.6.2
platformArchitecture: x64
cpuCount: 4
totalMem: 8192.0 MB
freeMem: 95.6 MB
Error Type: Callback Error syncAccounts
TypeError: Cannot read property 'migratedTransactions' of undefined
at Ws (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:2115031)
at file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:2110216
at Array.forEach ()
at onSuccess (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:2110203)
at Function.t.. (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:2108132)
Firefly version
1.3.2
OS details
platform: macOS platformVersion: 11.6.2 platformArchitecture: x64 cpuCount: 4 totalMem: 8192.0 MB freeMem: 95.6 MB
Can the issue be reliably reproduced?
No
Steps to reproduce the issue

...
Expected behaviour
staking available
Actual behaviour
staking not available
Errors
sync wallets do not work

Hello, can you try to create a new profile and restore it with your stronghold backup or your 24 words?

This might be due to not using the optional chaining in processMigratedTransactions in wallet.ts (see link).
Let us know if MiKeF05's suggestion help, since this api will change in the future.
https://github.com/iotaledger/firefly/blob/e66be3b316a14d4c626e537b6c164537261b0248/packages/shared/lib/wallet.ts#L1030

6 months old issue and no replies from the author yet, I will close it


[WIP] Isolate zk client config for suppressing authentication

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4504/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4507/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4509/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4510/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4511/
Test FAILed.

Jenkins, test this please.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4517/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4521/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4520/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4522/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4525/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4526/
Test PASSed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4527/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4530/
Test FAILed.


documentation: include CMake installation in Linux dependencies

name: pull request
about: submit changes to the project
title: "[pr] "
labels: ''
assignees: ''

description
brief description of the changes in this pr.

Neglecting to install cmake leads to error: failed to run custom build command for 'alsa-sys v0.3.1' when running cargo build --release on linux (sidenote: I'm using WSL2 running Ubuntu 22.04).

reorder apt-get install command for enhanced readability
docs: add missing 'cmake' installation step to apt-get instructions for linux

related issue: none
type of change

[ ] bug fix
[ ] new feature
[ ] breaking change
[x] documentation update

how to test
docs update, no tests to run
checklist

[x] i have read the CONTRIBUTING.md file
[ ] i have added the custom cursor AI prompt to my settings as mentioned in CONTRIBUTING.md and used to write this PR
[x] my code follows the project's style guidelines
[ ] i have performed a self-review of my code
[x] i have updated the documentation if necessary
[ ] my changes generate no new warnings
[ ] i have added tests that prove my fix is effective or that my feature works
[ ] all tests pass locally with my changes

additional notes
none

@braxton-mills thx!
any chance you can test #477 ? trying to support wayland but need one tester to merge


Warning using external.js "Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience"

I'm using the external.js in a reveal.js presentation with some external sources. I get the follow warnings quite a lot in the debugger:
Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.
They relate to the line of code:
xhr.open( 'GET', url, false );
Line 54 in external.js
Any help always appreciated...

 Requesting files from a local directory will break it if you're not.


[Help] Editable field not update value

i have table included editable column but when i focus and editing field without enter or select other section or row the value not update and not trigger onchange event.
it possible to detect onchange onkeyup something or trigger onchange or update this field when focus out ?
thank you for response

Facing the same issue here, is there any solutions?


Example for "Copy Files Over SSH" task is not working

The example under "contents" does not work:
- Example: **/*.jar \n **/*.war includes all jar and war files (including sub folders) under the source folder.

The newline character breaks it, and nothing is matched.
What worked for me was to instead use:
**/*.*(jar|war)

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

ID: 71788382-82ac-c878-0768-fb602d9be9df
Version Independent ID: 6651ef8d-5a5f-2785-e36b-5238fcbdc510
Content: Copy Files Over SSH task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/copy-files-over-ssh.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

@RoopeshNair -- Roopesh, please review this update in private PR 4591.

Facing the same issue. Using the exact examples from the documentation doesn't work. Seems like white spaces break the task.

After going through a lot of trials and errors I'm proud to bring you the solution.......... use double quotes.
e.g.
- task: CopyFilesOverSSH@0
  inputs:
    sshEndpoint: 'DEV'
    contents: "!.git/**\n!.vscode/**\n!api/**"
    targetFolder: '~/some/path'
    readyTimeout: '20000'

Fixed


Fixed return statement for SimpleXML

There is a possibility to instantiate custom classes, which extends SimpleXMLElement by providing class name in each of these functions:
simplexml_import_dom
simplexml_load_string
simplexml_load_file
Extending custom classes explained here

Example to test with:
$xml = '<?xml version="1.0" encoding="UTF-8"?>
<resultset>
  <row>
    <name>
        <lastname>Harry</lastname>
    </name>
    <age>20</age>
  </row>
  <row>
    <name>
        <lastname>Potter</lastname>
    </name>
    <age>25</age>
  </row>
</resultset>';

$response = @simplexml_load_string($xml, 'SimpleXMLElementEnhanced');

print("\nVar dumping the response:\n");
var_dump($response->xpath('row'));

class SimpleXMLElementEnhanced extends \SimpleXMLElement
{

    /**
     * @param string $path
     * @return SimpleXMLElementEnhanced[] <------- PhpStorm complains here .. fixed by this pull request (you can see in var_dump output that it's really this class, not SimpleXMLElement)
     */
    public function xpath($path)
    {
        return parent::xpath($path);
    }
}

I'm against mixed return types.
Your test case looks not related and wrong, because problem in this case is on line return parent::xpath($path);.
You defined SimpleXMLElementEnhanced[] return type in phpdoc and return not that type.

About functions return types: PhpStorm should respect that parameters and I think it can be done with some internal .phpstorm.meta.php magic. AFAIK @neuro159 is expert in that.

Thank you for contribution. I've created this ticket https://youtrack.jetbrains.com/issue/WI-37860 . We are going to fix it soon in a proper way. I will close this pull-request.


Support to add new schemas and delete existed schema for orchestration

Considering the suggestions from ShardingSphere community, I plan to add a new feather for Sharding-orchestration, which basically includes

User can add or delete scheme configurations in registry, e.g. zookeeper or etcd.
Registry pushes those changed events to all of applications online.
Applications online refresh their schema configurations to make those changes effective

It seems not to work in 3.1.0. We tested with a 2 instances cluster and Zookeeper for orchestration.
Steps:

Run 2 instances with same config
Add a new schema in first sharding proxy and restart. New shema could be seen on zookeeper soon.
Connect to second proxy, run 'show databases', but no new schema.
Any bug?


Update Composer dependencies (2022-01-28-00-02)

Visual regression test passed!
View the visual regression test report


Migrate to Bzlmod.

This change is

Depends on https://github.com/world-federation-of-advertisers/common-jvm/pull/224


Improve in Nav bar section for good design

I can make a good nav bar which can be helpful for see nav bar section if any white background or blackground comes it can be visible
can u assign me in this issue?
im from Gssoc contributer.

@justynigam please explain the issue clearly with Screenshoot.

here the nav bar is transparent
sometimes making multiple page can make issue to see clearly what is written in nav because it is transparent if we can make little blur with nav bar background then it can be more visible

@justynigam go ahead!

Update Navbar from Transparent to Visible
fixes https://github.com/Anjaliavv51/Retro/issues/10
Updates the navbar from a transparent background to a visible one, improving the overall user experience and readability of the navigation menu.
Changes:
Modified the navbar background from transparent to a visible color, making it more prominent and easier to use.
retro redesign navbar

@Anjaliavv51 successfully done ,
check #44 if any issue i will solve it


Statistical analysis in two conditions and 2 bioreplicates

Hello,
I require some suggestions to compare the differences in protein expression between 2 conditions (P vs C). I have 3 bio-replicates and 2 conditions. Please find the snapshot. I would want to analyze the differences in protein expression in condition P versus C by taking together P in expr2 and expr3 and comparing against C in expr2 and expr3. How to do it

Thank you

Helly @priyambial123,
Sorry for only replying now. We somehow missed the mails from gh.
Are you still struggling with this analysis?


BDD strings in step are concatenated in the 1st parameter

What are you trying to achieve?
I'm using codecept in BDD mode and I have an issue when a step has more than 1 string.
If I define a feature step like : When I say "hello" and "good afternoon"
in the step file I have : When('I say {string} and {string}', (str1,str2) ...
I want :
str1 = "hello"
str2 = "good afternoon"
What do you get instead?
when I run the test
str1 = "hellogood afternoon"
str2 = undefined

Provide console output if related. Use --verbose mode for more details.

feature file:
When user sets attribut "name" to the value "Comprehensive attribut with Text constraints"
step.js:
When('user sets attribut {string} to the value {string}', (name,value) => {
...
});

Provide test source code if related

Given user sets attribut "name" to the value "Comprehensive attribut with Text constraints"
I fill field "#nameComprehensive attribut with Text constraints", undefined
-- FAILURES:

First Tests
[...]
chrome not reachable

Scenario Steps:

I.fillField("#nameComprehensive attribut with Text constraints", undefined) at Object.obj.(anonymous function).obj.(anonymous function) [as fillField] ([...]\AppData\Roaming\npm\node_modules\codeceptjs\lib\actor.js:28:26)

Details

CodeceptJS version: 1.3.2
NodeJS Version: 8.11.3
Operating System: windows 10
WebDriverIO: 4.13.1
Configuration file:

Issue comes from 2 steps syntaxicaly too close and not correctly discriminated.


Qwester Tests Seem to Depend on Running Dummy App Tests First

The only way I seemed to be able run the Qwester tests was to run the dummy application's tests first.
git clone git@github.com:reggieb/qwester.git
cd qwester
bundle
cd test/dummy
bundle exec rake db:create db:migrate
cd ../..
bundle exec rake

Qwester tests fail: ActiveRecord::StatementInvalid: Could not find table 'admin_users'
cd test/dummy
bundle exec rake
cd ../..
bundle exec rake

Qwester tests pass
It seems strange that I needed to run the dummy app's tests to get Qwester's tests to pass. What is the correct way to get the tests running?

The problem is the test database. It is not automatically updated from the dev database, when the root tests are run. I've got into the habit of running:
rake db:migrate RAILS_ENV=test

To keep the test database up to date when new migrations are added.
I think the duality of the engine test environment can be confusing - what needs to be run at root, against what needs to be run at test/dummy. But the good things engines bring outweigh the small inconveniences.
If you can suggest a better way of doing things, that would be splendid, but I don't really want to change too much just to avoid the occasional rake db:migrate.
I'll add a note to the readme for now.

I've added the note to the README. Please feel free to reopen this issue if you don't think that adequately deals with the issue.

Thank you very much! The explanation that you put in the README is exactly what I needed.

Excellent.


Add Mockae to Public REST APIs To Use In Tests

Add Mockae (My website) to Public REST APIs for Testing. This open-source API allows customization of API responses, including headers, bodies, and status codes, using Lua

Thanks!


How to use EOI ?

In several grammars they refer to EOI in the lexer, something like:
<xq,xe,xus><<EOF>>		{ yyerror("unterminated quoted string"); }
...
<xb><<EOF>>		{ yyerror("unterminated bit string literal"); }
...
<*><<EOF>>                              return 'EOF';
...
<<EOF>>			{ SET_YYLLOC(); yyterminate();	}

How to use/access the EOI through a grammar ?

I once added <> support, but it didn't work well. Harmut suggested removing it as 'nobody needs that feature' or some-such.
Let me think about it again with the benefit of greater experience.

Can you point it out in git history ?

This pre-github. The first proper release was in 2006 (I think it was on CodeProject originally) and it was committed to github in August 2012.


device_util.cuh problem, diffuse texture not working

Hi @sketchbooks99!
I am reopening the previous issue #11. I created a details error gist here.
I assume the error may caused by static_assert and trying different solutions also.  Because your code base in C++20 and the function is something look like C++11
// tested this assumption, still similar error in both cases
static_assert( constant-expression, string-literal ); // C++11
static_assert( constant-expression ); // C++17 (Visual Studio 2017 and later)

Apparently the error looks like this:
/home/alpha/Documents/rt_mpi_viewer/rt_mpi_viewer/cuda/../../prayground/core/ray.h(73): warning #940-D: missing return statement at end of non-void function "prayground::getWorldpRay"

/home/alpha/Documents/rt_mpi_viewer/prayground/optix/cuda/device_util.cuh(69): error: expected a "("

/home/alpha/Documents/rt_mpi_viewer/prayground/optix/cuda/device_util.cuh(71): error: expected a "("

/home/alpha/Documents/rt_mpi_viewer/prayground/optix/cuda/device_util.cuh(73): error: expected a "("

/home/alpha/Documents/rt_mpi_viewer/prayground/optix/cuda/device_util.cuh(75): error: expected a "("

/home/alpha/Documents/rt_mpi_viewer/prayground/optix/cuda/device_util.cuh(77): error: expected a "("

/home/alpha/Documents/rt_mpi_viewer/prayground/optix/cuda/device_util.cuh(79): error: expected a "("

/home/alpha/Documents/rt_mpi_viewer/prayground/optix/cuda/device_util.cuh(81): error: expected a "("

/home/alpha/Documents/rt_mpi_viewer/prayground/optix/cuda/device_util.cuh(83): error: expected a "(

@bipul-mohanto
I might find the cause of your reported error.
At first, please check your compilation options of optix/cuda code. It's can be found in CMakeLists.txt in root directory of this project. The reason of your errors seems to be a c++ standard  at a cuda compilation. If you compile cuda codes with ~c++14, you will fail because if constexpr expression is only supported after c++17.
And when I compiled my codes with -std=c++14, I got same errors as you reported.
https://github.com/sketchbooks99/PRayGround/blob/4bf899d00d78e22ab905de3513c811d61c30e74c/CMakeLists.txt#L160-L211

Thank you so much, It is working perfectly. I just overlooked that, and thought the OptiX codebase is C++11 standard, so no need to change that.


Convert an Entered text to a Cipher text in java

Please Assign me this task for Hactoberfest-2021

Assign this task to me as soon as possible so that I can work on it

Assign this task to me as soon as possible so that I can work on it

Sorry for late assign😅

Please, assign to fast ⏩🙏🏻...


Test Dart 2.18.7 base image created via GHA

NO MERGE : Testing Dart 2.18.7 dart2_base_image GHA
This is a test PR to run CI with the base image created via github actions
instead of Workiva Build.
For more info, visit #support-frontend-dx in Slack.
Created by Sourcegraph batch change Workiva/test_dart_218_gha.

Security Insights
No security relevant content was detected by automated scans.
Action Items

Review PR for security impact; comment "security review required" if needed or unsure
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.


"lp.draw_box()" is not able to displays the result in .py file

Hi everyone! Thanks for developing this package. It was quite impressive to see the work you guys have achieved. I have a lot of fun when I was messed around with it.
However, would it be possible for the draw_box function to be working in a normal .py file? I was originally writing all the code in a normal python file but, the draw_box function just couldn't display the result to me somehow. I am on Ubuntu 20.04 and Python 3.8 is the version I am using. Then, I switched to the Jupyter Notebook extension in Visual Studio Code. Everything was up and running fine. I noticed in your examples, you guys are using the .ipynb file to show the example as well. I don't know it's a bug on my end or it just couldn't display the result in a normal python file. I don't have a strong reason why we should use the .py file instead of the .pynb. I just noticed a lot of other packages support both, so I guess it would be good if this one also supports them.
Thanks!

Thanks! The command will return an PIL image object, and you could try to use the .show command to have it displayed:
lp.draw_box(xx).show(). It's definitely not the most elegant solution - let me figure out a better one and incorporate it into the library.
You might also want to check more about this issue here.

I will try that solution later today. I didn't realize you guys are using PIL for image processing. Would it be possible for you guys to transfer the PIL image object into an actual image (like the ones that could be displayed using OpenCV or matplotlib)? I don't know the details to achieve this but I think this could be a valid solution. And then whenever we call the draw_box function. It would return the actual image. This way users can save the image for later or do other image processing if they need.


Provisioning dialogs are clearing text fields

Enter VM provisioning dialogs
Fill in all required fields, specifically:

Name fields on Requestor tab
VM name field on Catalog tab

Switch to a different tab and go back to the original, your text fields are now blank

Submitting the request also shows validation errors on the 3 fields above.

@miq-bot assign @dclarizio
cc @gmcculloug

@bdunne this should be fixed after #8061, please close if so