fix: remove requirements

Codecov Report
Patch and project coverage have no change.

Comparison is base (59d90c2) 97.96% compared to head (26ce70d) 97.96%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #93   +/-   ##
=======================================
  Coverage   97.96%   97.96%
=======================================
  Files           7        7
  Lines         246      246
=======================================
  Hits          241      241
  Misses          5        5

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Contextmenu support

Adding contextmenu support
Can you explain the best way to add contextmenu support to rows? I think it is wrap row component with handler. If i think right, please explain how i can manualy render row with some wrapper in right way
I want try to use this library
Thank you!

Please ask this question in the slack channel :) https://react-chat-signup.herokuapp.com/

Has this been answered or support added?  The link in the reply from August 8, 2017 is broken.

Test


Batching - prevent joining of lightmasked items

It turns out lights masking misbehaves when items that are masked are joined. This PR simply disables joining in this case.
Notes

Turns out I had a note already in the code that this might be an issue in future.
It's possible that this case could be handled slightly more optimally, but it requires some time investment to investigate, and it doesn't seem that common a use case.
Everything outside the zone of such a masking light will batch as normal.

Fixes #46154

Thanks!


Rmm v2v h2v discoverytool

This branch contains VMware and HyperV discovery tools.
We are temporarily keeping older files (config.ini,l checksum.txt, discoveryTool of VMware Discovery Tool). Parallel to VMware and HyperV folder.
We will delete those temporarily files as soon as we go GA.
3 Readme files are added: Readme for VMware, Hyper-V, and VPC Tools.

Approved.


Make the size of a browse result page configurable.

Description
Invent a new configuration property to control the number of results per page when displaying browse results.
Instructions for Reviewers
List of changes in this PR:

Create a new configuration property browseBy.pageSize.
Define a default value wherever browseBy is being configured.
Replace hard-coded "20" with the value of this property in the browse-by-metadata component.

Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes TSLint validation using yarn run lint
[x] My PR doesn't introduce circular dependencies
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.

"Error:   121:3  error  Angular will not invoke the ngOnInit lifecycle method within @Injectable() classes  @angular-eslint/contextual-lifecycle"
Suggestions?


VizPanel: Add repeat props to state

Context: https://github.com/grafana/grafana/pull/81818
Bit iffy on this tbh, since the repeat state is ultimately represented within the PanelRepeaterGridItem, but we need some way to store the state during panel edit and before the changes are committed/applied.

📦 Published PR as canary version: 2.4.1--canary.565.7757961436.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Close in favor of https://github.com/grafana/grafana/pull/81818


On PRNG seeding

Hi,
Thanks for these very useful examples. Just a remark on the seeding of the PRNG. instead of:
 ! Each image will have its own RNG seed:
 call rng%seed([ -1337_i8, 9812374_i8 ] + 10*this_image())

I would suggest something like:
call rng%seed([ -1337_i8, 9812374_i8 ])
if (this_image() /= 1) then
  do i = 2, this_image()
    call rng%jump()
  end do
end if

The PRNG sequences on the different images are then  guaranteed not to overlap.
Best,
Christophe

Thanks Christophe,
I will integrate your remark, not the next week but rather at the beginning of July when I have some time for looking into that project.
Best,
Vincent

Done.


IA32_DEBUGCTL.FREEZE_WHILE_SMM does not appear to be honored

Version
0.51.0
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
No response
Other Software
No response
Repro Steps
Set /sys/device/cpu/freeze_on_smi to 1. This should trigger a write to IA32_DEBUGCTL.FREEZE_WHILE_SMM. (I haven't actually verified this because the WSL kernel does not appear to ship the msr module so I can't use rdmsr/wrmsr).
Then run some workload that uses the performance counters and where SMM activity will be visible (in my case I'm running the rr test suite)
Expected Behavior
SMM execution is not visible to the performance counters.
Actual Behavior
SMM execution bleeds through and is visible to the performance counters.
Diagnostic Logs
No response

(With access to the hypervisor source code I think it should be easy for you to verify whether or not writes to IA32_DEBUGCTL.FREEZE_WHILE_SMM actually trigger anything.)


Unexpectedly seeing searchParameters pushed through to child routes

If I have a ROUTES like this:
const TEST_ROUTES = {
  EXAMPLE: route(
    'example',
    { searchParams: { q: string() } },
    { DETAILS: route(':id') }
  ),
};

I would expect to want to navigate to example?q=My+query but not example/1?q=My+query. The typescript is letting this go through. If your application is built correctly, this is going to end up rendering fine but will have an awkward URL (and is a sign that your link is being constructed wrong).
Eg: this does not match the @ts-expect-error I want --
// @ts-expect-error - extra search parameters here.
TEST_ROUTES.EXAMPLE.DETAILS.buildPath({ id: '1' }, { q: 'my query' })

Is this a bug or expected behavior? If it's expected, can you help me understand the scenario it's supporting?

This is an intended feature that matches how nested routes work in React Router.
Each route part usually corresponds to some UI on the screen (check their visualization), which may use some search (and other) params. So the full route has to include params from all its parts.
I see two ways to achieve what you want.

Move search (and other) params to a nested route:

const TEST_ROUTES = {
  EXAMPLE: route(
    'example',
    {},
    {
      INDEX: route('', { searchParams: { q: string() } }),
      DETAILS: route(':id'),
    }
  ),
};

<Routes>
  <Route path={TEST_ROUTES.EXAMPLE.path}>
    <Route index element={<Example />} />
    <Route path={TEST_ROUTES.EXAMPLE.DETAILS.path} element={<ExampleDetails />} />
  </Route>
</Routes>

Unnest the routes entirely:

const TEST_ROUTES = {
  EXAMPLE: route('example', { searchParams: { q: string() } }),
  EXAMPLE_DETAILS: route('example/:id'),
};

<Routes>
  <Route path={TEST_ROUTES.EXAMPLE.path} element={<Example />} />
  <Route path={TEST_ROUTES.EXAMPLE_DETAILS.path} element={<ExampleDetails />} />
</Routes>


Deprecated ContainerProperties#setTransactionDefinition

In scope of #1699, ContainerProperties#setTransactionDefinition was deprecated together with ChainedTransactionManager. The note to this method, which we believe caused the deprecation, states:

Note that this is only generally useful when used with a org.springframework.kafka.transaction.ChainedKafkaTransactionManager configured with a non-Kafka transaction manager.

However, we do not use ChainedTransactionManager and do use ContainerProperties#setTransactionDefinition. Our use-case is implementation of Idempotent Consumer pattern with database storage for processed messages. The rough solution is as following:

Set JDBC/JPA Transaction Manager as ContainerProperties#setTransactionManager - database transaction is open by container when a new record is processed.
Implement RecordFilterStrategy that stores the incoming message id in the database, and filters the record if this id is already present.
This way, @KafkaListener and RecordFilterStrategy are executed in the same database transaction, which guarantees atomic write of business data and Idempotent Consumer records.

Since our @KafkaListeners have different requirements to the database transactions (in particular, timeouts), we use ContainerProperties#setTransactionDefinition to configure those. In case this method is removed after deprecation, we are in big trouble.
Would it be possible to remove the deprecation and update the javadocs of this method, so our services continue to function properly with future updates of spring-kafka?

Yes; your use case makes sense.

Beautiful, thank you!


analysis: Block creation ignores input collections

When creating an Analysis object, you can optionally specify the collections to use, instead of loading them from the Layout
E.g.:
collections = {
    'run': load_variables(bids_layout,
                          derivatives='only',
                          task='some_task',
                          level='run',
                          **entities).get_collections('run')[0]
    }

bids_analysis = Analysis(bids_layout, model, collections=collections)

Looks like currently, the collections argument is ignored, and just saved. When blocks are created, the collections are always loaded up from the layout, which means any "custom" collections are ignored. In my case, this means "derivative" events can't be passed into the Analysis object.

This is a bug, but the solution is probably to drop the collections argument, rather than allowing custom collections to be passed. In principle, it should be possible to add constraints to collection loading as extra kwargs to Analysis.setup.
What's preventing derivative events from being loaded properly? In theory if files are indexed properly at Layout construction, Layout.get_collections should use them no matter where they live. It's possible that's failing for some reason, but if so, that's probably where a fix is called for.

I've tested that and that works. The Layout indexes just fine, this was mostly just leftover code from how you previously 'injected' events into Analysis.
Agree dropping collections argument would be good, as its confusing

Related minor small issue: both the self.sampling_rate and self.scan_length arguments are also ignored

Oh, thanks. Feel free to open a PR, otherwise I'll include it next time I patch anything


remove CircleCI

this is not in use by this repo

removed in 19cff9d16db68c1eb1a1abe43ab234b438f4c98d
I'll be getting better at making these pull requests from now on


feat: add deviceConfigPriorityDir zwave setting

Fixes #1686
cc @raman325

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 18 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 29.764%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ZwaveClient.ts
0
6
0.0%

app.ts
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
3631

Relevant Lines:
12824

💛  - Coveralls

thanks!


HV-1524 Log temporal validation tolerance only if it has been explicitly set

https://hibernate.atlassian.net/browse/HV-1524

Amended the commit message and merged. Thanks!


Print debugging information of the deployed module in the notebook readflow test.

Relates to https://github.com/fybrik/fybrik/issues/1860
This PR prints information that might be useful for debugging the deployed modules before fybrikapplication is deleted at the end of the test:

print the logs of the deployed module pod
print the config-maps in the module namespace

Can this information be printed only in case of errors?

@shlomitk1 I do not know will have to check... I hope to catch failures like in https://github.com/fybrik/fybrik/actions/runs/4483018011/jobs/7881785854

@shlomitk1 I do not know will have to check... I hope to catch failures like in https://github.com/fybrik/fybrik/actions/runs/4483018011/jobs/7881785854

Create a boolean variable "success" set to false which will be set to true at the end of the test function (after all checks). In the function that prints logs + cms, if success is true, exit the function without printing anything.

@shlomitk1 fixed thanks!

Abandon this approach for debugging the test for now...


Image size

There should be a only one image size. Whenever we embed image it uploads with the original resolution but if you can add perfect image size and in center in format that can be could.
Issue :-
The real image size will be big it get's of the real page size and then those scrollbars.

Thanks for reporting. I'll take this one up as well 😊. @rahxuls Thanks for the feedback, if you have any more feedback or feature request feel free to open more issues.

Thanks for reporting. I'll take this one up as well 😊. @rahxuls Thanks for the feedback, if you have any more feedback or feature request feel free to open more issues.

Many coming 😂 coz i in love with this template. I installed in right now just editing some things. Don't know why but i think dependencies need to be upgraded.


Xcode warning in applyRepetitionPenalty

When compiling the LLMEval example app, I get a warning from Xcode about the variable logits. Sorry if this is too basic a question or I'm missing something, but I believe in Swift classes are reference types, but var logits = logits implies that a copy is being created. Isn't this function simply returning a reference to logits, which it mutates?
private func applyRepetitionPenalty(logits: MLXArray, repetitionContext: MLXArray, penalty: Float) -> MLXArray {
    var logits = logits // Xcode warning: Variable 'logits' was never mutated; consider changing to 'let' constant

    if repetitionContext.shape[0] > 0 {
        let indices = repetitionContext
        var selectedLogits = take(logits, indices, axis: -1).squeezed(axis: 0)

        selectedLogits = MLX.where(
            selectedLogits .< 0, selectedLogits * penalty, selectedLogits / penalty)

        logits[0..., indices] = selectedLogits
        return logits
    }

    return logits
}

Right, it is a legit warning.  logits is a reference to an instance of MLXArray, which is a class (reference type).  There is no assignment to logits, that is the reference never changes, so it can be written as a let.
This line:
        logits[0..., indices] = selectedLogits

appears to mutate it, but it actually mutates the inner state of logits and isn't really an assignment.
Here is a case where an MLXArray variable has to be a var:
    func callAsFunction(_ x: MLXArray) -> MLXArray {
        var x = x

        for resnet in resnets {
            x = resnet(x)
        }

        if let downsample {
            x = padded(x, widths: [[0, 0], [0, 1], [0, 1], [0, 0]])
            x = downsample(x)
        }

        if let upsample {
            x = upsample(upsampleNearest(x))
        }

        return x
    }

In particular the x =  are all assignments so the the reference itself is being mutated.
Make sense?

Yes, that makes sense. Is there any reason then to declare let logits = logits in applyRepetitionPenalty? It seems like this could be omitted.

Agreed, it is not needed.  I think in a past version it must have reassigned it and this is what is left over.


Add benchmarks for pure logging speed

Currently, our logging benchmarks measure performance of concurrent transaction execution with logging enabled. We should also measure pure logging performance, or how fast can we log for an execution that has already occurred. This will most likely require adding pause functionality to the log manager. I envision the benchmarking looking like:
log_manager->Start();
log_manager->Pause();
SimulateOLTP();
{
  scopedTimer (&time);
  log_manager->Unpause();
  log_manager->PersistAndStop();
}

This will let all the logs that SimulateOLTP generated accumulate in the log manager, and then we can time purely how long the log manager takes to log everything when we unpause. Note that the pause functionality will need to pause all the LogManager tasks.

Nice


Idea for the new ranchers chapter

Feature Suggestion
 A multiplayer feature
What will this feature do?
You can play multiplayer with your friends who also love this mod it would be super fun to have multiplayer so you can do the stuff in this mod.
How it should integrate into the mod? (Optional)
Viktor will do an experiment on the shapeshifter slime and notices that something weird is happening with the plorts when he has the plorts while near a teleported he decides to experiment with the plorts and and he managed to create a way to travel to different universes he decides to go to a other universe and sees its all the same but there are different ranchers they kinda remind him of Beatrix for some reason
Possible names for the feature? (Optional)
 rangoverse
Additional context

There is already a Multiplayer mod made by Saty. It has a SRML version that supports playing with other mods. Imo the game shouldn't be multiplayer at all, but if you want it I think it's best to leave that to Saty.


Add DotName.packagePrefix() method

I needed this several times in the past when filtering classes by package name. This nicely completes the existing withoutPackagePrefix() method.

2d6ea2f: rebased
@n1hility any chance to review?

2d6ea2f: rebased
@n1hility any chance to review?

Any chance to review, @n1hility?

Looks good, and it can now move forward since next release will be at least 2.3

Thanks!


Scaled Jacobian plot and locations of bad elements [scaled-jacobian-plot-dev]

@tzanio did all the work and gave me the magic to get the center and plot the scaled Jacobian [aka det(J)], this was very helpful for me in finding where I had poor mesh quality.
mesh-explorer now prints the id and center of the first 10 inverted elements:
--> x
subdivision factor ---> 0
det(J) < 0 = -1.59692e-09 in element 23684, centered at: 0.0746917 0.015 9.93346
det(J) < 0 = -6.23481e-09 in element 23688, centered at: 0.0771759 0.015 9.9364
[snip]
det(J) < 0 = -1.00524e-08 in element 23700, centered at: 0.0839758 0.015 9.94575
det(J) < 0 for 157 more elements not printed.

bad elements = 167  --  167 Cube(s)
min det(J)   = -4.94199e-08
max det(J)   = 2.16668e-07
global ratio = -4.38422
max el ratio = 559.434
min kappa    = 16.312
max kappa    = 96606.2

And plots the scaled (unitless) Jacobian values:

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#1835
@wnissen
@tzanio
@vladotomov + @kmittal2 + @tzanio + @v-dobrev
10/24/20
10/29/20
11/03/20

Math note:

In 2D the scaled Jacobian for J with columns v1 and v2 is sin ϕ, where ϕ is the angle between the Jacobian columns, ϕ = ∠(v1,v2).

In 3D, for J = (v1 v2 v3) the scaled Jacobian is sin ϕ sin θ, where ϕ = ∠(v1,v2) and θ = ∠(v3, plane-of-v1-v2).

In both cases, if det J >= 0 the scaled Jacobian is between 0 and 1, with 0 corresponding to a flat element, and 1 to orthogonal Jacobian columns.

@vladotomov, do you want to take a quick look?

Merged in next for testing...

Merged in post v4.2 next.

Re-merged in next for testing ...

Thank you Walt!

Relating this, is there are way to "fix" a mesh when there is negative J element? I was guessing mesh-optimizer may be for it and trying various options. But, could not figure out how....Thank you for your advice.

There are various "untangling" approaches in Mesh Optimizer, but how well they work will depend on the problem. @vladotomov and @kmittal2 will know more.

Thank you @tzanio.
I am attaching one of mesh file I am trying to "fix". I am guessing mid=352 is the one to use. I tried various target_id, but couldn't find the one which works. I tend to see the maximum of metric value increases ( = det (J) becomes even more negative) after the
iteration. Also, sometimes I saw the warning saying that preconditioner is not positive-definite.
fs.msh.txt

Thanks for the mesh @sshiraiwa, we'll run some tests and get back to you this week.

Hi @sshiraiwa,
The best result we could get so far is SS_bc_qo4.mesh.txt, where two of the tets next to the holes are still inverted.
This was generated by
mpirun -np 6 pmesh-optimizer -m SS_bc.mesh -o 2 -rs 0 -mid 313 -tid 1 -ni 100 -ls 2 -li 50 -bnd -qt 1 -qo 4 -vl 1 -fd
in the branch tmop-inv-example, which is still WIP. The SS_bc.mesh has the correct boundary attributes to allow tangential relaxation, except inside holes (which causes the issue).
We'll keep working on it next week and will let you know how it goes.
Vladimir

Hi @sshiraiwa,
Here's a fully untangled version of your mesh SS_bc_Q3_qo4.mesh.txt.
We had to represent it as a 3rd order mesh, as the 2nd order didn't have enough DOFs to move in the restricted elements around the channels (10 nodes per element, 6 of which were fixed for Q2).
The mesh can be generated in the branch tmop-inv-example by
mpirun -np 6 pmesh-optimizer -m SS_bc.mesh -o 3 -rs 0 -mid 313 -tid 1 -ni 100 -ls 2 -li 50 -bnd -qt 1 -qo 4 -vl 1 -fd -no-vis
I hope the mesh is useful,
Vladimir

Nice work  @vladotomov!

OK, that is pretty amazing. I haven't seen the pmesh-optimizer tool before. Are these parameters highly mesh-specific, or would they potentially work for many types of meshes? How much does it change the problem geometry? I see you have a bunch of examples in the source at http://mfem.github.io/doxygen/html/pmesh-optimizer_8cpp_source.html

The parameters are not highly mesh specific, but some of them must be adjusted to the type of optimization we want to achieve. In this case we had a tangled 3D mesh and we wanted to get the best possible shape, hence we used a 3D untangling metric with ideal targets, ie -mid 313 -tid 1.
There is no change in the geometry -- all initial boundaries are preserved exactly. In this case the solver moved some boundary nodes tangentially (the ones on the straight boundaries), and the internal nodes to untangle around the channels. We have several options to preserve mesh features or adapt to solution features, but those were not used in this problem.

@vladotomov Fantastic. This is very very promising. Can you please explain the difference of tid=1, 2, and 3. It is labeled as IDEAL_SHAPE_UNIT_SIZE, IDEAL_SHAPE_EQUAL_SIZE, and IDEAL_SHAPE_GIVEN_SIZE. But not quite clear what these mean.
Also, I have another question. -bnd option uses boundary attribute from 1-4 to enforce the tangential adjustment. Is it correct that the current constraints work only when the surface normal is orienting to a particular direction. How difficult do you think it is to extend it for a general curved surface,?

IDEAL_SHAPE - this part means that the target shape is a regular tetrahedron.

IDEAL_SHAPE_UNIT_SIZE - we should probably rename this to UNIT_SHAPE, because it is not supposed to be used for volume optimization. It's supposed to be combined with shape-only metrics.
IDEAL_SHAPE_EQUAL_SIZE - this will try to produces tets with equal volume throughout the mesh.
IDEAL_SHAPE_GIVEN_SIZE - this will try to produce tets that have the same volume as they originally had (this was named IDEAL_SHAPE_INITIAL_SIZE at some point).

The last two options are combined with metrics that see both shape and size.
For now we can do tangential relaxation only parallel to the axes (we force Dirichlet BC for the displacements). We are exploring methods for general surfaces, but in principle this will always produce some geometric error. We're approaching it as an algebraic method based on level sets, but we don't have anything to show yet.


remove storybook

Description
Storybook and all dependencies related to it are now removed. Added a note in the docs to explain the choice.
closes #4132
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Code is clear and sufficiently documented
[x] Correct, in line with design

Pull request rejected by merge tool. Pull request is not mergeable. Please check for merge conflicts before re-adding the merge-tool-ready label.

Processing this pull request

Merged into branches master, iso5 in efe0d955c702a50359b36e2db0f1610b64d9d938


how to settting email field when after required_if tag?

Package version eg. v9, v10:
V10
Issue, Question or Enhancement:
In this struct ,validator check email format error, how to settting email's validate?
Code sample, to showcase or reproduce:
for example ,in this Struct
type AccountRegisterParams struct {
	RegisterType string `json:"register_type" form:"register_type"  validate:"required,oneof=username email mobile"`
	UserName     string `json:"user_name" form:"user_name"  validate:"required,max=50"`
	Email        string `json:"email" form:"email" validate:"required_if=register_type email,email,max=50"`
	Mobile       string `json:"mobile" form:"mobile" validate:"required_if=register_type mobile,max=50"`
	Password     string `json:"password" form:"password"  validate:"required,min=8,max=20"`
	Captcha      string `json:"captcha" form:"captcha" validate:"required,max=6"`
	ClientId     string `json:"client_id" form:"client_id" validate:"required"`
}

I think the problem here would the fact that you are not using the correct name of the field for required_if to work. It should match the exact name of the field you validating against. In this case it would be
	RegisterType string `json:"register_type" form:"register_type"  validate:"required,oneof=username email mobile"`
	UserName     string `json:"user_name" form:"user_name"  validate:"required,max=50"`
	Email        string `json:"email" form:"email" validate:"required_if=RegisterType email,email,max=50"`
	Mobile       string `json:"mobile" form:"mobile" validate:"required_if=RegisterType mobile,max=50"`
	Password     string `json:"password" form:"password"  validate:"required,min=8,max=20"`
	Captcha      string `json:"captcha" form:"captcha" validate:"required,max=6"`
	ClientId     string `json:"client_id" form:"client_id" validate:"required"`
}

See the difference here is instead of required_if=register_type email it's required_if=RegisterType email

thanks

thanks


Request for Async Support and Dependency Update

Hi all.
In my work we're using the  databricks-sql-connector with sqlalchemy-databricks  but it currently lacks support for async operations, which is causing performance issues for us. It would be appreciated if you could consider adding async support in a future release.
We have noticed that one dependency of the sqlalchemy (https://github.com/crflynn/sqlalchemy-databricks) dialect it is  pyhive. The problem here is that pyhive  is not being maintained anymore by dropbox. This raises security concerns. There are any additional considerations about changing that?
I  would appreciate if you could let me know if there is anything I can do to help with these requests.
https://github.com/dropbox/PyHive/issues

Hi @devmessias, tudo bem?
Just to clarify a bit, what exactly do you mean by async? Separate thread for parallelism/concurrency, or the ability to save operation handle and come back to it later in a potentially different process?

Ditto Andre's questions.
Separately:

We have noticed that one dependency of the sqlalchemy (https://github.com/crflynn/sqlalchemy-databricks) dialect it is pyhive. The problem here is that pyhive is not being maintained anymore by dropbox. This raises security concerns. There are any additional considerations about changing that?

sqlalchemy-databricks is a community project. We are building our own SQLAlchemy dialect which does not rely on Pyhive and will be available soon (there is an open pull request that implements it). We don't officially support the community project and once ours is available you will be able to connect with SQLAlchemy without installing any separate packages.

sqlalchemy-databricks but it currently lacks support for async operations

We don't have plans to implement the SQLAlchemy asynchronous API at this time but are open to considering this in the future. SQLAlchemy considers asyncio support to be a beta feature so I don't think it makes sense to prioritise it at this time.

which is causing performance issues for us.

I'm very curious to know what performance issues you're seeing. If you're interested we can share a dev build of databricks-sql-connector with you that includes the first-party SQLAlchemy connector -- this very well may solve the performance issues even without async.

I would appreciate if you could let me know if there is anything I can do to help with these requests.

The best thing we can get is feedback on our new connector :) https://github.com/databricks/databricks-sql-python/pull/57

SQLAlchemy support is now built-in to databricks-sql-connector as of v2.4.0.


Hard to benchmark the operation in the repo

Hi, thanks for your work! I recently wanna benchmark each step's latency of this repo, and I found if I use torch.cuda.synchonize() and time.time(), I cannot get the actual data copy time.
For example, I believe the data copy time is those two lines.
    device_expert_buffer.storage.copy_(self.offloaded_storages[info_to_load.index], non_blocking=True)
    offloaded_storage_buffer.copy_(self.main_modules[info_to_evict.index].storage, non_blocking=True)

And time.time gives me 1e-5s, which I believe is far faster than real data transfer latency. I think the reason might be there exist multiple process/threads and would lead to wrong latency. Could you help me solve this problem?
Many thanks!

Hi! In this case the .copy_ operation is non-blocking. Meaning the python thread can proceed as soon as the copy operation is submitted. You might want to look into torch's profiler. I recommend you export your traces into json and view them using perfetto or chrome://tracing.


Favor our choice of Imath includes over OIIO's.

New changes in OIIO guards the contents of its Imath.h with the
OIIO_IMATH_H_INCLUDED preprocessor symbol. This lets us in OSL
pre-define it to absolutely ensure that OSL headers are including
exactly the Imath headers they want, not accidentally getting the ones
that OIIO includes.
See https://github.com/OpenImageIO/oiio/pull/3322
This present PR is harmless for versions of OIIO prior to that PR being
integrated.
Signed-off-by: Larry Gritz<EMAIL_ADDRESS>

Does this seem ok?


The vpc_security_group_ids gets ignored once use_custom_launch_template =false

As mentioned in this comment(see here) the security groups do not get correctly propagated to the instances, once not using the custom launch template.
Here the code snippet that causes it:
 vpc_security_group_ids = length(var.network_interfaces) > 0 ? [] : local.security_group_ids
see this PR- here.
This results into assigning only the default security group to all of the instances in the EKS managed nodeGroup:
$aws ec2 describe-instances| jq -r '[.Reservations[].Instances[]|{"id":.InstanceId, "securityGroups":.SecurityGroups}]' [   {     "id": "i-045a8e0315252ac37",     "securityGroups": [       {         "GroupName": "eks-cluster-sg-projectname-eks-1916997484",         "GroupId": "sg-0321e7706adb006be"       }     ]   },   {     "id": "i-0782a4971b14ec3a8",     "securityGroups": []   },   {     "id": "i-0905a580c6295c2de",     "securityGroups": []   },   {     "id": "i-064b61114fdafd0b8",     "securityGroups": [       {         "GroupName": "eks-cluster-sg-projectname-eks-1916997484",         "GroupId": "sg-0321e7706adb006be"       }     ]   },   {     "id": "i-051be7cf0f2e71bc6",     "securityGroups": []   },   {     "id": "i-0bd6fc3c287fbfa64",     "securityGroups": [       {         "GroupName": "eks-cluster-sg-projectname-eks-1916997484",         "GroupId": "sg-0321e7706adb006be"       }     ]   } ]
@bryantbiggs  - could you please explain what is then the correct way to add a security group i.e. to grant access to a specific NodePort-Service on the instances?
Thanks in advance!

I don't follow the question(s) - can you please use the issue template provided and fill in the requested information so we can properly triage and provide guidacne

I don't follow the question(s) - can you please use the issue template provided and fill in the requested information so we can properly triage and provide guidacne

Thanks for the reply, actually the issue is already described, that is why I was just referencing it - https://github.com/terraform-aws-modules/terraform-aws-eks/issues/2651
Basically, once you decide to not use the default template, the instances created in the managed EKS node group
do not get assigned any other security groups ( provided over  - vpc_security_group_ids input), except the "default cluster security group "(see here)
Hence, I was asking:
what is now the correct approach to attach other security groups i.e. for a specific NodePort to the instances created in the managed node group?

Here the versions information:
$terraform version
Terraform v1.0.9
on linux_amd64

provider registry.terraform.io/hashicorp/aws v4.64.0
provider registry.terraform.io/hashicorp/cloudinit v2.3.2
provider registry.terraform.io/hashicorp/http v3.1.0
provider registry.terraform.io/hashicorp/kubernetes v2.20.0
provider registry.terraform.io/hashicorp/local v2.4.0
provider registry.terraform.io/hashicorp/null v3.2.1
provider registry.terraform.io/hashicorp/time v0.9.1
provider registry.terraform.io/hashicorp/tls v4.0.4
provider registry.terraform.io/terraform-aws-modules/http v2.4.1

...
$cat .terraform/modules/modules.json | jq -r '.Modules[]|select(.Version !=null)|"(.Key) (.Version) (.Source)"'
amaieks.eks.kms 1.1.0 terraform-aws-modules/kms/aws
amaieks.eks 19.13.1 terraform-aws-modules/eks/aws
network.vpc 3.19.0 terraform-aws-modules/vpc/aws

Thanks for referencing this example!
A few lines above your reference one finds an easier way to define a managed eks nodegroup
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/771465be280450fc96d889ef9e15f191bb512849/examples/eks_managed_node_group/main.tf#L98-L110
As you see there on the line 101, one has set the use_custom_launch_template = false
which thus allows to use the larger disk_size = 50 without any additional block_device_mappings, but rather get the instances with initially larger fs.
That disk_size gets evaluated under
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/0f9d9fac93caf239386f47b0f117706ea78c2bec/modules/eks-managed-node-group/main.tf#L331-L332
and the instances with the larger disk are created.
However, as stated in the beginning of this discussion, with using the use_custom_launch_template = false the created instances get only attached the default EKS security group( AWS minimum recommendations ), and not the "additional" security group, as denoted by you in the module's documentation on network-connectivity
So an attempt to use the example with default_node_group denoted above, results into a set of EC2 instances, that are only attached the default EKS security group( AWS minimum recommendations ), but not the "additional" one. That "additional" security group gets created(incl. the NodePort rules added), but not attached to the instances of the NodeGroup.
Same goes for creating externally a security group with those rules for NeodePort grants, and trying to attach them through the vpc_security_group_ids attribute within the managed nodegroup definition.

I'm lost as to what solution you are trying to get to. When using the default launch template created by EKS managed nodegroups, there are restrictions - tags aren't propagated to the instances, you have to use the cluster primary security group, etc. The module creates a custom launch template which is transparent to users, for the most part. This is what provides the full breadth of features and customization. Please refer to the examples and links shared to see what is possible

I am aiming at creating the simple EKS cluster with one managed node group, with as small additional settings to the module as possible. I have however experienced that the default variant does not allow to increase the size of the volume though the disk_size attribute. The documentation states one has to then set use_custom_launch_template = false, see here. Which I have followed but the security groups have not got attached, now only in your last answer you state

there are restrictions - tags aren't propagated to the instances, you have to use the cluster primary security group,

which directly corresponds to the "issue" I was referring to, but is confusing as the documentation is written.
I personally do not follow, how the user of the module after reading this section https://github.com/terraform-aws-modules/terraform-aws-eks/blob/master/docs/faq.md#setting-disk_size-or-remote_access-does-not-make-any-changes will know all of the implications, incl. cluster security group only.
I will for sure take further look at the more advanced examples, but please enhance the documentation accordingly.


I will be able to save the text content of an uneditable element to later use that for asserting. (between the time I saved and the assertion time, the text content of the element may change)
await Selector('#policy-manager-header')
    .find('mcr-functional-drop-down')
    .find('span')
    .innerText;

error:
Selector cannot implicitly resolve the test run in context of which it should be executed. If you need to call Selector from the Node.js API callback, pass the test controller manually via Selector's `.with({ boundTestRun: t })` method first. Note that you cannot execute Selector outside the test code.

await testController.selectText(Selector('#policy-manager-header')
        .find('mcr-functional-drop-down')
        .find('span'))
        .innerText;
error:
The action element is expected to be editable (an input, textarea or element with the contentEditable attribute).

and:

A call to an async function is not awaited. Use the "await" keyword before actions, assertions or chains of them to ensure that they run in the right sequence.

You can save text content of an element for further usage. Please refer to the following article: http://devexpress.github.io/testcafe/documentation/test-api/selecting-page-elements/selectors/using-selectors.html#dom-node-snapshot

I've prepared a sample example:
import { Selector, ClientFunction } from 'testcafe';

fixture `fixture`
    .page `http://example.com`;

const fn = ClientFunction(() => {
    document.querySelector('h1').innerHTML = 'changed header';
});

test('test', async t => {
    const selector = await Selector('body').find('h1');
    const snapshot = await selector();

    console.log(snapshot.innerText);

    await fn(); // change element's content

    console.log(snapshot.innerText); // snapshot has previous value

    console.log((await selector()).innerText); // Note that after changing the text content you need to
                                               //  execute selector again.
});

You can use the SelectText command only for inputs, textareas, and contenteditable elements.

Please try to modify your code in this way:
const selector = await Selector('#policy-manager-header')
  .find('mcr-functional-drop-down')
  find('span'));
console.log(selector.innerText);


Add batch page

I created an endpoint to get id of the user by UID, which is needed to create the batch.

Hi @zkwsk, it worked with me :thinking_face: I mean I can see the page http://localhost:3000/addbatch when I'm logged in with<EMAIL_ADDRESS>

Hi @zkwsk  I fixed that 401 issue and other issues which were reviewed by @gizemcandemir  (i fixed all the issues) but i have a git problem it won't allow me to push it. i tried in many ways and @cecastosic also tried to help ....it didn't work out.......plz help me to fix it

I am closing this PR #198 because I have created a new PR #207 for ADD BATCH


Authentication Problem

Expected behavior

Current behavior

Possible solution

Steps to reproduce (for bugs)

Context

Steps for resolving the issue

[x] Reauthenticate on Last.fm

[ ] Try to use unpacked extension from latest master

[ ] Check other issues for duplicates

Environment

Version used:
Browser name and version:

Please describe the issue properly.

Closed as outdated.


Feature/launch icons

rename:  logo to primary_onboardin_logo
added:    secondary_onboardin_logo (for use on different color layouts)

Android 13:
adaptive icons
1 layer: image: color [108x108]
2 layer: image: color [108x108] ilusration [72x72]+ (better add 16 padding)
Android <13:
icons  image: color [432:432]  for max resolution drawable-xxxhdpi


20&oauth2State=dbab06a2620b11eda9a534e12d3c2fc6&show_select_country=N

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Close because incomprensibile.


postgres, redis: Can not pull images by version tags

Which image/versions are related to this issue/feature request?

cgr.dev/chainguard/postgres:15
cgr.dev/chainguard/postgres:15.4
cgr.dev/chainguard/redis:7

Issue/Feature description
I can pull chainguard/postgres by latest tag, but 15, 15.4, or other version tags are not working.
$ docker pull cgr.dev/chainguard/postgres:latest
latest: Pulling from chainguard/postgres
Digest: sha256:5d3af7b0ed95cd5f032cd08403568f44728cfb79ecade74cb02f97fc8a6f2200
Status: Image is up to date for cgr.dev/chainguard/postgres:latest
cgr.dev/chainguard/postgres:latest

$ docker pull cgr.dev/chainguard/postgres:15
Error response from daemon: manifest for cgr.dev/chainguard/postgres:15 not found: manifest unknown: Unknown manifest

$ docker pull cgr.dev/chainguard/postgres:15.4
Error response from daemon: manifest for cgr.dev/chainguard/postgres:15.4 not found: manifest unknown: Unknown manifest

Docker Version
Client:
 Version:           24.0.5
 API version:       1.43
 Go version:        go1.20.6
 Git commit:        ced0996
 Built:             Fri Jul 21 20:32:30 2023
 OS/Arch:           darwin/arm64
 Context:           orbstack

Server: Docker Engine - Community
 Engine:
  Version:          24.0.5
  API version:      1.43 (minimum version 1.12)
  Go version:       go1.20.6
  Git commit:       a61e2b4
  Built:            Fri Jul 21 20:35:24 2023
  OS/Arch:          linux/arm64
  Experimental:     false
 containerd:
  Version:          v1.7.2
  GitCommit:        0cae528dd6cb557f7201036e9f43420650207b58
 runc:
  Version:          1.1.8
  GitCommit:        82f18fe0e44a59034f3e1f45e475fa5636e539aa
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

I just found https://www.chainguard.dev/unchained/scaling-chainguard-images-with-a-growing-catalog-and-proactive-security-updates 😓

@siketyan apologies you ran into this. We tried to communicate these changes extensively over the past few months to avoid breaking folks but it's hard to reach everyone.
Here's another post from today https://www.chainguard.dev/unchained/important-updates-for-chainguard-images-public-catalog-users.
Please reach out if we can help with anything like advice on migrating. My GitHub username at chainguard dot dev.


Issue when running on local machine

In the development environment, running the app in emulator crashes due to the below error:
6303-6320/com.areuin.uin E/MixpanelAPI.ViewCrawler﹕ Error parsing URI wss://switchboard.mixpanel.com/connect/XXX for editor websocket
java.net.URISyntaxException: Illegal character in path at index 39: wss://switchboard.mixpanel.com/connect/"123"
at libcore.net.UriCodec.validate(UriCodec.java:63)
at java.net.URI.parseURI(URI.java:402)
at java.net.URI.(URI.java:204)
at com.mixpanel.android.viewcrawler.ViewCrawler$ViewCrawlerHandler.connectToEditor(ViewCrawler.java:452)
at com.mixpanel.android.viewcrawler.ViewCrawler$ViewCrawlerHandler.handleMessage(ViewCrawler.java:296)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:148)
at android.os.HandlerThread.run(HandlerThread.java:61)

Thanks for writing in!
Can you please provide me more details? Is it crushing in the same way on a real device? Are you sure your emulator has internet connection? What was the device you were emulating? What's the lib version?

Closing. Please re-open if you still have this issue.


I WANT THE EXACT CODE 🙏🏻

I’m new to programming. So plz can you provide me the Full exact code. I can’t even change a line so plz provide me and make sure that it’s compilable
AND IS THE CODE AVAILABLE FOR ESP8266
PLZ WHATSAPP ME<PHONE_NUMBER>

Can u plz provide me the code ?


Prototype pollution (Fixed)

Hi,
This package is vulnerable to prototype pollution.
POC
var {JsonPointer } = require("json-ptr")
var obj = {}
console.log("Before : " + obj.polluted);
JsonPointer.set(obj,'/__proto__/polluted','Yes! Its Polluted', true);
var obj1 ={}
console.log("After : " + obj1.polluted);

Fixed prototype pollution in util.ts.
Thanks

Thanks for the PR. I added a test; since pollution is now disallowed, I test that the error is thrown.

Will be v2.0.0 shortly; seems it is potentially a breaking change if someone was using such pointers.

Glad to be of help.

Hello @luci-m-666 , I'm a little curious as it how you found prototype pollution in the library and how you came with this exact lines:
https://github.com/418sec/json-ptr/pull/1/files#diff-3294a832ea2276e554177e0b3007cc2d401c082912c7fbde49fa09141bf1aed1R175-R177


feat: replaced dlc-wasm-wallet with @dlc-link/dlc-tools

Try out this version of Leather — download extension builds.

We've replaced the dlc-wasm-wallet package with the @dlc-link/dlc-tools package. While it maintains the same core functionality, we've added some new features to the wasm code. Additionally, we've moved the package from a temporary user account to our official company account.

@Polybius93 please rebase and see if this fixes CI


Wireless Model D-

I purchased a Wireless Model D-, before the purchase I asked Glorious if it had the same internals as the model O Wireless, and pointed them to your GitHub repo. They said it would likely work with your utility, but of course gave me a bunch of boilerplate about how they don't support it. I took the chance and bought one, however it obviously has a different USB ID than the MOW, and as such running mow just says it can't find a supported device:
./mow report battery
Error: No matching device found!

How do I go about adding the wired/wireless ID's to your code in order to try and get it recognized? I'm not a developer, and I have zero Rust experience, but I'd be willing to take a crack at it if it's just a matter of changing the ID in the code. It would be nice if there was a command line option to explicitly specify the ID, for testing with other models from the same vendor, or a config file where you can list the IDs you'd like it to recognize.
Below is the dmesg output when plugging this mouse in.
Wired:
[Sun May 22 12:52:49 2022] usb 3-1.2.4: Manufacturer: Glorious
[Sun May 22 12:52:49 2022] input: Glorious Model D- Wireless as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.0/0003:258A:2014.001F/input/input50
[Sun May 22 12:52:49 2022] hid-generic 0003:258A:2014.001F: input,hidraw17: USB HID v1.11 Mouse [Glorious Model D- Wireless] on usb-0000:03:00.0-1.2.4/input0
[Sun May 22 12:52:49 2022] input: Glorious Model D- Wireless Consumer Control as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2014.0020/input/input51
[Sun May 22 12:52:49 2022] input: Glorious Model D- Wireless System Control as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2014.0020/input/input52
[Sun May 22 12:52:49 2022] input: Glorious Model D- Wireless Keyboard as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2014.0020/input/input53
[Sun May 22 12:52:49 2022] input: Glorious Model D- Wireless as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2014.0020/input/input54
[Sun May 22 12:52:49 2022] input: Glorious Model D- Wireless as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2014.0020/input/input55
[Sun May 22 12:52:49 2022] hid-generic 0003:258A:2014.0020: input,hiddev96,hidraw18: USB HID v1.11 Keyboard [Glorious Model D- Wireless] on usb-0000:03:00.0-1.2.4/input1
[Sun May 22 12:52:49 2022] hid-generic 0003:258A:2014.0021: hiddev104,hidraw19: USB HID v1.11 Device [Glorious Model D- Wireless] on usb-0000:03:00.0-1.2.4/input2

Wireless:
[Sun May 22 12:53:01 2022] usb 3-1.2.4: Manufacturer: Glorious
[Sun May 22 12:53:01 2022] input: Glorious Model D- Wireless as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.0/0003:258A:2025.0022/input/input56
[Sun May 22 12:53:01 2022] hid-generic 0003:258A:2025.0022: input,hidraw17: USB HID v1.11 Mouse [Glorious Model D- Wireless] on usb-0000:03:00.0-1.2.4/input0
[Sun May 22 12:53:01 2022] input: Glorious Model D- Wireless Consumer Control as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2025.0023/input/input57
[Sun May 22 12:53:02 2022] input: Glorious Model D- Wireless System Control as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2025.0023/input/input58
[Sun May 22 12:53:02 2022] input: Glorious Model D- Wireless Keyboard as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2025.0023/input/input59
[Sun May 22 12:53:02 2022] input: Glorious Model D- Wireless as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2025.0023/input/input60
[Sun May 22 12:53:02 2022] input: Glorious Model D- Wireless as /devices/pci0000:00/0000:00:01.1/0000:01:00.2/0000:02:00.0/0000:03:00.0/usb3/3-1/3-1.2/3-1.2.4/3-1.2.4:1.1/0003:258A:2025.0023/input/input61
[Sun May 22 12:53:02 2022] hid-generic 0003:258A:2025.0023: input,hiddev96,hidraw18: USB HID v1.11 Keyboard [Glorious Model D- Wireless] on usb-0000:03:00.0-1.2.4/input1
[Sun May 22 12:53:02 2022] hid-generic 0003:258A:2025.0024: hiddev104,hidraw19: USB HID v1.11 Device [Glorious Model D- Wireless] on usb-0000:03:00.0-1.2.4/input2

When replacing with the IDs 0x2014 and 0x2025 in main.rs on line 31 and 48, it builds, but running the tool, I get the following errors (I tried multiple reports and config settings and got the same error):
~/S/N/D/m/t/release (master) [2]> ./mow config scroll invert
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: HidApiError { message: "hid_error is not implemented yet" }', src/main.rs:51:52
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

~/S/N/D/m/t/release (master) [101]> RUST_BACKTRACE=1 ./mow config scroll invert
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: HidApiError { message: "hid_error is not implemented yet" }', src/main.rs:51:52
stack backtrace:
   0: rust_begin_unwind
             at /rustc/1.60.0/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/1.60.0/library/core/src/panicking.rs:143:14
   2: core::result::unwrap_failed
             at /rustc/1.60.0/library/core/src/result.rs:1749:5
   3: mow::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

~/S/N/D/m/t/release (master) [101]> RUST_BACKTRACE=full ./mow config scroll invert
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: HidApiError { message: "hid_error is not implemented yet" }', src/main.rs:51:52
stack backtrace:
   0:     0x55cec60d22bc - std::backtrace_rs::backtrace::libunwind::trace::h4c82c30effa1ec8b
                               at /rustc/1.60.0/library/std/src/../../backtrace/src/backtrace/libunwind.rs:93:5
   1:     0x55cec60d22bc - std::backtrace_rs::backtrace::trace_unsynchronized::h04f51b94feda398c
                               at /rustc/1.60.0/library/std/src/../../backtrace/src/backtrace/mod.rs:66:5
   2:     0x55cec60d22bc - std::sys_common::backtrace::_print_fmt::h83c260b7e0b0e807
                               at /rustc/1.60.0/library/std/src/sys_common/backtrace.rs:66:5
   3:     0x55cec60d22bc - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h4c69548d53cef23e
                               at /rustc/1.60.0/library/std/src/sys_common/backtrace.rs:45:22
   4:     0x55cec60f070c - core::fmt::write::he1f7c3e09b6d48b1
                               at /rustc/1.60.0/library/core/src/fmt/mod.rs:1190:17
   5:     0x55cec60cee31 - std::io::Write::write_fmt::h7f5865fb08a452b9
                               at /rustc/1.60.0/library/std/src/io/mod.rs:1657:15
   6:     0x55cec60d4325 - std::sys_common::backtrace::_print::hd7f28ad4b7c7f649
                               at /rustc/1.60.0/library/std/src/sys_common/backtrace.rs:48:5
   7:     0x55cec60d4325 - std::sys_common::backtrace::print::hda6514e610876808
                               at /rustc/1.60.0/library/std/src/sys_common/backtrace.rs:35:9
   8:     0x55cec60d4325 - std::panicking::default_hook::{{closure}}::hec9b2f020ff35128
                               at /rustc/1.60.0/library/std/src/panicking.rs:295:22
   9:     0x55cec60d3fe4 - std::panicking::default_hook::hd76a5748cebf86d7
                               at /rustc/1.60.0/library/std/src/panicking.rs:314:9
  10:     0x55cec60d4a58 - std::panicking::rust_panic_with_hook::h372a0c94f5d85a2f
                               at /rustc/1.60.0/library/std/src/panicking.rs:698:17
  11:     0x55cec60d4767 - std::panicking::begin_panic_handler::{{closure}}::h6bef164c07c5cd1d
                               at /rustc/1.60.0/library/std/src/panicking.rs:588:13
  12:     0x55cec60d2754 - std::sys_common::backtrace::__rust_end_short_backtrace::ha06d828e2bf3aa2d
                               at /rustc/1.60.0/library/std/src/sys_common/backtrace.rs:138:18
  13:     0x55cec60d4489 - rust_begin_unwind
                               at /rustc/1.60.0/library/std/src/panicking.rs:584:5
  14:     0x55cec6047f23 - core::panicking::panic_fmt::h95a16302ac706f92
                               at /rustc/1.60.0/library/core/src/panicking.rs:143:14
  15:     0x55cec6047fb3 - core::result::unwrap_failed::h7182d5bb0014fa58
                               at /rustc/1.60.0/library/core/src/result.rs:1749:5
  16:     0x55cec605b57f - mow::main::h19257dd5480f69b5
  17:     0x55cec605ee03 - std::sys_common::backtrace::__rust_begin_short_backtrace::h672290ab32783fa0
  18:     0x55cec6066469 - std::rt::lang_start::{{closure}}::h841e16ea26ed5a0a
  19:     0x55cec60d1a6e - core::ops::function::impls::<impl core::ops::function::FnOnce<A> for &F>::call_once::he81739e93730d87f
                               at /rustc/1.60.0/library/core/src/ops/function.rs:259:13
  20:     0x55cec60d1a6e - std::panicking::try::do_call::haab80c588b564f17
                               at /rustc/1.60.0/library/std/src/panicking.rs:492:40
  21:     0x55cec60d1a6e - std::panicking::try::h44df61bdf16d5e2d
                               at /rustc/1.60.0/library/std/src/panicking.rs:456:19
  22:     0x55cec60d1a6e - std::panic::catch_unwind::h5a4755568a112d4c
                               at /rustc/1.60.0/library/std/src/panic.rs:137:14
  23:     0x55cec60d1a6e - std::rt::lang_start_internal::{{closure}}::hd1cd8cace1e688e2
                               at /rustc/1.60.0/library/std/src/rt.rs:128:48
  24:     0x55cec60d1a6e - std::panicking::try::do_call::h2a95c2966749f5f5
                               at /rustc/1.60.0/library/std/src/panicking.rs:492:40
  25:     0x55cec60d1a6e - std::panicking::try::hd43011e2f8131e56
                               at /rustc/1.60.0/library/std/src/panicking.rs:456:19
  26:     0x55cec60d1a6e - std::panic::catch_unwind::hd291d3c47494d6cf
                               at /rustc/1.60.0/library/std/src/panic.rs:137:14
  27:     0x55cec60d1a6e - std::rt::lang_start_internal::h97283ffe4f4e7a74
                               at /rustc/1.60.0/library/std/src/rt.rs:128:20
  28:     0x55cec605b922 - main
  29:     0x7fc4d8983310 - __libc_start_call_main
  30:     0x7fc4d89833c1 -<EMAIL_ADDRESS>  31:     0x55cec60486f5 - _start
  32:                0x0 - <unknown>

First of all, sorry about the late reply! I love your enthusiasm, I am sure we can figure this out. Pretty sure the internals are indeed close enough that only a proper id change should be sufficient in theory. Are you on Linux or macOS? There are some extra hoops to jump for Linux that I know of, so that might be the issue you are running into.

Maybe you are running into the same issue as discussed here https://github.com/korkje/mow/issues/1

Hej,
No problem on the time frame, we all have lives to deal with. Your software is pretty nice and it could easily become the go-to for these mice with broader support. I'm fairly sure the O and O- is basically the same mouse except for ID, same with D and D-. Let's try and get easy support for them all?
I'm on Linux, so what else should I be looking for to try and make the code work?

Hej,
No problem on the time frame, we all have lives to deal with. Your software is pretty nice and it could easily become the go-to for these mice with broader support. I'm fairly sure the O and O- is basically the same mouse except for ID, same with D and D-. Let's try and get easy support for them all?
I'm on Linux, so what else should I be looking for to try and make the code work?
Pretty sure I ran this as root, so it shouldn't be a permission issue. I'll double check some time this weekend.

Please try "sudo ./mow report battery" with the changes again, your error is consistent with what is expected without sudo or root.

Hi,
I have also just purchased a Glorious Model D Wireless, and following this I have amended the IDs (although mine are different) - 0x2012 for wired and 0x2023 for wireless.
Installed and executed as root, so -
sudo ./mow report battery from within the release folder and it does successfully report battery usage.
report firmware (as root) displays -
<IP_ADDRESS>
Before I try and change some settings, is there a way to output all of the rest of its config to check everything is being queried correctly?
Also, what group/permissions would we need to grant to run without root/sudo?
Thanks!

I see, thank you and apologies!


Standard quickstart

Description

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

@Pketaki @akshay2985 can you resolve conflicts? Then I'll merge and maybe we can create a 0.0.1 release?

/run pipeline

/run pipeline

Actually I added the cra-config.yaml in https://github.com/terraform-ibm-modules/terraform-ibm-zvsi/pull/168
so now that your PR is rebased with master, the only change required is to update the value of the CRA_TARGET in cra-config.yaml to be solutions/standard. You will also have to add any required TF_VARs in that file too. the following all seem required for standard: prefix, region, ssh_public_key, cert_common_name
For ssh_public_key you can just generate any random one (like we did here), as it won't actually be used in an apply, just a terraform plan

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@akshay2985 have you guys run any of the tests yet? They don't seem to work. Pipeline fails with this:
TestRunCompleteExampleStandard 2024-05-29T10:08:18Z retry.go:99: Returning due to fatal error: FatalError{Underlying: error while running command: exit status 1; ╷
│ Error: Value for undeclared variable
│
│ A variable named "ssh_key" was assigned on the command line, but the root
│ module does not declare a variable of that name. To use this value, add a
│ "variable" block to the configuration.
╵
╷
│ Error: No value for required variable
│
│   on variables.tf line 29:
│   29: variable "ssh_public_key" {
│
│ The root module input variable "ssh_public_key" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵}

Also you will need to skip the upgrade test for this initial PR since code doesn't yet exist in main branch. To do this, push a commit to the branch with the string SKIP UPGRADE TEST in it

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Determine the use-case for a JS-packaged rs-ucan

We need to determine the scope and use-case for wielding WASM-targeted rs-ucan from JavaScript. Please share your use-case for a JS-packaged rs-ucan in this issue.
In #17 , we have seen the first pass at making WASM-targeted rs-ucan legible to JavaScript. The use case of wielding rs-ucan from JavaScript has always been in-scope for this project, but the plan for how to do it has been vaguely defined.
There are many technical and logistical details to consider for this endeavor. JS packaging is a complex topic on its own, and when we add the detail that there is a Rust-compiled WASM module in the mix, we get an additional layer of complexity.
With infinite time and attention we could address many different use cases. However, in the interest of being somewhat tactical, let's figure out how folks would actually like to use such a package in practice. What is your use case for a JS-packaged rs-ucan?
Please note: it is a non-goal to decide specific tools and/or API designs in resolving this issue (although feel free to hypothesize about such things). We should strive to figure out how folks want to use the package in order to know what to build to support them, and that can guide our tooling / API choices.

@matheus23 @dholms @fabricedesre @expede @bmann @hugomrdias CC'ing you all since you were either participating or mentioned during relevant Discord discussions.

Some things to consider:

The JS package use case is currently served by the actively maintained project https://github.com/ucan-wg/ts-ucan
Fission is working on a Webnative SDK in Rust

They will expose rs-ucan in user-facing API in this Rust SDK
The SDK will also be offered as a JS package

The JS package use case is currently served by the actively maintained project https://github.com/ucan-wg/ts-ucan

At the moment ts-ucan should cover UCAN use cases from JS/TS.
That said, @fabricedesre mentioned he really didn't like dealing with bundlers & packaging, which was a point in favor of js-bound rs-ucan.

Fission is working on a Webnative SDK in Rust

They will expose rs-ucan in user-facing API in this Rust SDK
The SDK will also be offered as a JS package

Yeah. And once we're getting to that, we'll likely only need rs-ucan to have a rust API. I'm not sure whether it's possible to share JS bindings across rust projects? I'm guessing that'd be possible, and that'd probably be useful for us? But it's all still unclear. We don't really know what exactly that'll look like, and we'll need a bit until we'll get to that.

One use case for a JS-packaged version is for the validator in the UCAN Community website: https://ucan.xyz/validator/
The validator does the following:

Decodes from a token
Checks that a UCAN is not too early or expired
Checks for a valid signature
Checks that proofs are valid and prove what is claimed

The validator currently uses ts-ucan and works with 0.8.x UCANs.
We want to use the validator as a test case for using rs-ucan in a web app. It won't be the most complete test case because it doesn't need to generate any UCANs, but it could be a good first step because it is less complicated than other use cases (no need for key support or signing).
We have an issue in the community website repo: https://github.com/ucan-wg/ucan-community-web/issues/52

Would it be alright if I made an attempt to add JS packaging support? Perhaps I could start by adding verification methods only?
I saw in #17 there was a discussion around using the Rapier approach to packaging. I'd be happy to replicate their approach for rs-ucan.


[3.6.rc1] Existing meshes don't use vertex cache optimization?

Tested versions

Reproducible in 3.6.rc1.

System information
Windows 11 - Godot v3.6.rc1 - GLES3 - dedicated
Issue description

In short, i have a ton of meshes saved as .res files. That helps to save scene file sizes, helps to manipulate materials of the surfaces, etc. However, as far as i understand, vertex cache optimization affects only reimported meshes (which i did btw, i removed .import and reimported all assets). In this case, meshes saved in .res files may stay unaffected by this change. There are no methods in Mesh nor ArrayMesh to optimize the vertex cache with GDScript. So, the question is, how can i apply this optimization method on existing mesh resources? Or is it applied automatically and is not documented to be that way?
Steps to reproduce

Have a project that you have developed in 3.5.3.
Save mesh resources in any format you'd wish (.res in my case)
Convert to 3.6.rc1.
Remove .import folder, painfully wait for it to import back because you should not have made that much assets.
Play the game and notice no difference in framerates in any scenes.
... OR ...
Open MRP with 3.6.rc1, it was made with version 3.5.3.

Minimal reproduction project (MRP)
MeshOptimizationMRP.zip

To clarify here what should be discussed here is how to document this, for support questions please ask in the other community channels, not here

To clarify here what should be discussed here is how to document this, for support questions please ask in the other community channels, not here

Excuse me if i didn't say it correctly, but the problem here is also related to resource files. Meshes that were saved in .res files seem to have no vertex cache optimization.

Yes and that's what needs to be discussed and documented if the current behavior is expected, but the question of "how can i apply this optimization method on existing mesh resources?" is a support question and not a bug report, at least in the way it's phrased

that's what needs to be discussed and documented if the current behavior is expected

I didn't expect that behavior when i switched to 3.6.rc1 to test how would vertex cache optimize my game and it is not documented anywhere as far as i know, please, correct me if i'm wrong, but i really could not find any information, hence why i thought it would be worthy to report because, in my opinion, Lawnjelly may need about this

Have just seen this, it is fine to report as an issue imo.
The situations with already imported resources didn't occur to me. I'm not sure there is an easy way around this, as we don't want to be applying it to already imported resources "just in case".
Perhaps it is possible to write some kind of upgrade script, I'll have a think.
Bear in mind also this is a "nice to have" optimization, there are no promises for a dramatic difference, depending on your game and asset type (it gives more benefit with higher poly models, which are more likely to be bottlenecked by vertex processing).


Rewrite option wrappers using cmake_parse_arguments

This addresses #236 I think it would be better to move the CMake snippet to a module that is always downloaded, but I am not sure how to best achieve that.

Good point. I think we should make it much easier to place modules into a "standard library". I will follow up with an issue on that: #238.

Thanks for the rewrite!


Not being able to run the server code even after successfully building it

I went through the commands as you said and a vs solution folder was created on opening that I pressed run option(Local windows debugger)) but there was one problem , no errors one warning but on running the code wanted a file called ALL_BUILD in the server/Debug folder but there was no such file or exe there.
Can you please help me out?

The ALL_BUILD target is automatically generated by CMake and is not the target for the server executable itself. The target you want to run is called "server". You can set this as the default target for debugging by right-clicking on it and selecting "Set as StartUp Project":

Sorry for not replying earlier, I will check back with if I have further issues

I followed the steps, and set server as the startup project but I got the following error.
I built the files by going to Developer Command Prompt for Visual Stdio
went to the server folder and typed the command:
cmake .
cmake --build .
one after the other
do I need to add any path variables before doing that?
I am using cmake version 3.14.19050301-MSVC_2
Please could you give it a look

Error on the client side

The error message from the server tells you exactly what is wrong: asio listen error: asio.system:10048 (Only one usage of each socket address (protocol/network address/port) is normally permitted.)
Another process is already listening on port 8080, which is preventing the server from binding to that port. You will need to identify and terminate the other process to free the port for use by the server.

Cool sorry I will check it out

Hey, I killed the process that was using up port 8080 - TSLNSR.exe using taskkill.
Then, I ran VS and compiled built the code having server as startup project.
But now what happens is that the exe files runs and stops instantaneously and it gets deleted from the debug folder by itself. After which on compiling again I get the error:
LNK1104	cannot open file.

@adamrehn  sorry to bother you once again, but I really need this to work.

Hey it worked , I was initially having some problems with the exe file cause of Avast Antivirus on stopping it I got the following output:

So the sockets are connecting and server is sending messages to the clients.
But on hitting send message on the client side  I am not able to see if I have recieved  {"a":1, "b":2, "c":3} or not. How will I know the above?
@adamrehn sorry for bombarding you with problems,  but I really need this to work.
Thanks a lot for your help till now.

It's been years since I last looked through the code for this demo, so on initial inspection I was actually just as confused as you regarding the lack of feedback for sent messages. Having looked through the code now, I've realised that there's actually a discrepancy between the code in this repository and the workshop that it was written for.
In this repo, I've hard-coded the server to listen for messages with a type of myMessage, but in the original workshop the server listens for messages with a type of message, which is what the client sends by default. Messages with this type are ignored, but if you enter myMessage in the text field on the client then the messages will be acknowledged correctly and echoed back to the client.
I'm not entirely sure how this discrepancy crept in. Presumably I must've made that change at some point during the workshop to demonstrate the handler functionality for the students, and then forgotten to change it back before uploading the code.

I've pushed a fix for this now, which realigns the code in this repository with the workshop notes. Thanks for helping to uncover this oversight!

Yeah no problem, and thanks to you too. Without your demo code it would have been impossible to build the code from the websocketpp repo as I  had no idea of what files to link to make the repo examples work.

Hi, @adamrehn  ,had a query:
Do you have any idea how to build the projects in the https://github.com/zaphoyd/websocketpp directory just wanted to build those examples to see how to configure your demo, or is there a way to compile those using your demo?

@rishavghosh605 that question is best directed to the maintainers of the WebSocket++ library itself, since my demo doesn't use their examples.

Hey @adamrehn  had a question simple question bout your demo, when I use cmake and build the project I am able to access the exe file. But if I copy this same exe to another computer which does not have Visual Studio. On  opening the ex file on another latop I get the error : the code execution could not be done because MSVCP140D.dll, VCRUNTIME!$)D.dll and ucrtbased.dll was not found. So I copied all of these files from my computer, and also downloded the redistributable C++  packages, but still the problem did not go.
Is it because I have not downloaded anything, or I have to build the files on the other laptop using cmake?
I was hoping you could shed some light on it.
Sorry, for disturbing you once again, but I promise this will be the last time.

@rishavghosh605 if you're building any C++ project with Visual Studio for use on another computer then you should build the Release configuration and ensure the target machine has the Visual C++ Redistributable package installed that corresponds to the version of Visual Studio that you used to build the executable. You can find the relevant documentation here: https://docs.microsoft.com/en-us/cpp/windows/determining-which-dlls-to-redistribute
Again, this is a question that is not specific to my demo code, and is therefore best asked in a more appropriate venue. I'm locking this thread now to prevent any further discussion. If you have any further questions that are specifically related to the WebSocket demo code then feel free to open separate issues to address those questions.


Wrong placement in Noto sans arabic & UI

Character: ڕ
Name: ARABIC LETTER REH WITH SMALL V BELOW
Kurdish
UTF-8: 0xDA 0x95
UTF-16: 0x0695
C octal escaped UTF-8: \332\225
XML decimal entity: ڕ
https://www.unicode.org/charts/PDF/U0600.pdf

I agree this is not optimal, but I'm sure was done intentionally.  Makes sense for UI version. Not sure why done for non-UI.
@marekjez86 can you ask Kamal to review if this is acceptable form?

I think even in the UI version it should not be there
(Noto Naskh Arabic UI)

(Noto Naskh Arabic)

(Noto Naskh Arabic, Noto Naskh Arabic UI, Noto Sans Arabic, Noto Sans Arabic UI)


Can't we have the nispor python bindings available in Pypi ?

I remember seeing an older version of nispor on pypi, but I don't see it now. Can't we have the nispor python package on pypi ?

@sreerajkksd I failed to find a good way to compile nispor rust code or ask user to install nispor-libs during python setup.py build stage. So we remove it.
Personally, I dislike pip, so I would suggest you to use make install of nispor tarball or use OS shipped rpm.

Closing as won't fix.


docs.trafficserver.apache.org web server issue

G'day,
Sorry, not a bug with the code, but something seems to be wrong with the docs web severing, which I think is hosted by GoDaddy.  Wasn't sure the best way to let somebody know.
When browsing to the guide, some pages don't load, or they take a very long time.
https://docs.trafficserver.apache.org/en/latest/admin-guide/
Just now browsing to: https://docs.trafficserver.apache.org/en/latest/preface/index.en.html#other-resources
Request Timeout
Server timeout waiting for the HTTP request from the client.
host docs.trafficserver.apache.org docs.trafficserver.apache.org has address <IP_ADDRESS>
Thanks,
Dave

I'm facing the same issue, it happened to me a lot of times

Sorry for the inconvenience. We were testing the latest ATS with HTTP/3 and QUIC enabled, and I think that caused the issues. I just turned off a setting and I think it makes the server less unstable.
The test is to make sure next version of ATS is stable by the release, and the server is probably less stable until we make 10.0 release but it should not cause much trouble. If you are continuously having trouble with accessing the server, please reach out on our slack channel. Somebody should be able to check the server and take care of it.

Thanks so much!


wrong link for hawkeye

Discussed in https://github.com/Sunbird-Obsrv/Community/discussions/6

Originally posted by pramodkvarma April  8, 2022
@rhwarrier page https://obsrv.sunbird.org/learn/product-and-developer-guide/report-configurator points to hawkeye PDF with DIKSHA logo etc. You need to move that to a proper Obsrv doc. Please track and fix.

The document has been updated with a generalised SB doc. Thanks.

That's great! 👍
Btw while doing that we should move those PDFs as markdowns (native docs pages). Navigation and then links from within the PDF etc takes me out of context a bit. See if you can move those into native docs. Shouldn't take too long I suppose.


Tracing: stress test tracing services

I did a fair amount of stress testing with tracegen before the release. In the future, we can open issues for identified bottlenecks.


tcp proxy: add default idle timeout

We should did 1 hour for HTTP so let's do 1 hour for tcp proxy also.
cc @alyssawilk

related to https://github.com/envoyproxy/envoy/pull/8678/files (make sure to update FAQ entry)


feat: add ImagePullSecrets to PodSpec

What this PR does / why we need it:
Add ImagePullSecrets to PodSpec in order to support pulling EnvoyProxy and RateLimit images from private registries.
Use cases:

Custom images
Pulling images from a private mirror

Which issue(s) this PR fixes: N/A

I think we can learn from istio project, introduce K8SObjectOverlay, no need to add all the fields one by one.

@zirain the UX for https://github.com/envoyproxy/gateway/pull/2337#issuecomment-1866229984 is not great.
I do agree we need some generic way to patch k8s fields we dont have native support for
Pulling Envoy and RateLimit from air gapped & private registries is a common use case imo

@zirain the UX for #2337 (comment) is not great. I do agree we need some generic way to patch k8s fields we dont have native support for
Pulling Envoy and RateLimit from air gapped & private registries is a common use case imo

it's hard to say which one is common or not.


LAB_06-Implement_Network_Traffic_Management.md

Current Translation:
menu drop-down
Suggested Translation:
menu menurun

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run the bugfix tool and waiting for result. ICMS QUERY

This issue has been fixed, so I close the PR.


INVALID_CLIENT: Invalid redirect URI

When clicking the Log in button and log in with Spotify appears 'INVALID_CLIENT: Invalid redirect URI' error.

Is this issue still open , I'm Interested to work on .


Bot occassionally comments twice.

Add a check whether the bot has already commented before posting the reply.

Doesn't seem to happen anymore


extract visual data from report in python script and not in jupyter notebook widget.

Hi,
I have tried this library to embed report in jupyter notebooks. But I want to extract the data from report visuals in python script and run through commandline. Currently report is getting embedded after calling Report() on jupyter notebooks only. When i try to run the same thing in python script commandline it says report is not embedded.
Could you please suggest me some resource which i can use for reference to extract data from power report  visuals in python .py script.

I am getting ready to start using this package to try and pull data, if I can, I will let you know.

I am getting ready to start using this package to try and pull data, if I can, I will let you know.

Did you find the way to how to do it ?

I'm facing sort of similar challenges - I've stayed with the notebook but trying to use Run All to work through the rendering and then interacting with reports. But it seems quite flaky in that mode.  Visually I can see there's a lag between the "report" cell that renders the report and the actual report output becoming visible.  So I tried adding a wait at the start of the next cell, but I still often stalls, errors or gives incomplete results.
Here's my project fwiw - I wrote it to be generic for any report.
https://github.com/Mike-Honey/powerbi-export-all-visuals

Thank you for reporting this issue. We're closing GitHub issues on 01/01/2025. For further assistance, we kindly request that you direct your inquiries to the following support channels:

Bug Reports and Technical Assistance: Visit the Fabric Developer Community Forum. Our team and community experts are ready to assist you.
Additional Support: Contact your account manager or reach out to the Fabric Support Team.

Thank you,
The Power BI Embedded Team


Chinese Training using List Trainer

Hi. I'm having problems trying to use the ChatterBot to talk to me in Chinese.
In the terminal I get:
/usr/local/lib/python2.7/site-packages/chatterbot/conversation/statement.py:46: UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal   if response.text == self.in_response_to[index].text: Type something to begin...
for this first case (talk to english and respond in chinese works ok):
Hello
你好。 你好吗?
but it doesn't work for talking in chinese to the bot.
我很好你呢
我很好你呢
I also get this error:
/usr/local/lib/python2.7/site-packages/chatterbot/adapters/logic/closest_match.py:35: UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal   if input_statement.text in text_of_all_statements:
The code of what I'm using:
# -*- coding: utf-8 -*-
from chatterbot import ChatBot
from chatterbot.training.trainers import ListTrainer

chatterbot = ChatBot("Terminal",
    storage_adapter="chatterbot.adapters.storage.MongoDatabaseAdapter",
    logic_adapters=[
        "chatterbot.adapters.logic.ClosestMatchAdapter"
    ],
    input_adapter="chatterbot.adapters.input.TerminalAdapter",
    output_adapter="chatterbot.adapters.output.TerminalAdapter",
    database="chatterbot-database"
)

chatterbot.set_trainer(ListTrainer)

chatterbot.train([
    "你好。 你好吗?",
    "Hello",
])

chatterbot.train([
    "我很好太感谢",
    "我很好你呢",
])

print("Type something to begin...")

while True:
    try:
        bot_input = chatterbot.get_response(None)

    # Press ctrl-c or ctrl-d on the keyboard to exit
    except (KeyboardInterrupt, EOFError, SystemExit):
        break

Thanks!

Hi, I made a pull request you might want to check out, I think that is same issue I encountered with Croatian language, so I just added utf-8 support for corpus. I tried ListTrainer with your dataset, and it seems to work fine.
You want to check this commit https://github.com/gunthercox/ChatterBot/pull/216/commits/9ea48eac3954fd818947dbb0742d1d545b4f8c08

I have the same problem with @Nykho ,and I have merged the commit from @mbarisa said,but it still didn't work for me !

Hi @Nykho, I've been able to confirm this issue on my end. I will attempt to issue a patch as soon as I can.

Thanks @gunthercox!

Hi, I met this issue for unicode in python 3.5 in the training phase :
File "/usr/lib/python3.5/json/decoder.py", line 357, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Hope for help !


Linking a ClaimsMappingPolicy to an App causes AADSTS50146 upon authentication

[Environment]
AzureAD
.NET Core 3.1
OpenID Connect
[Summary]
Goal is to have AzureAD issue AAD non-basic schema properties as claims in the ID Token. Following the documentation and linking a newly created policy object to an existing app breaks the authentication flow. Upon successfully authenticating against AAD (directly or via a RP) the application would fail validating the ID Token with error AADSTS50146
[Error details]
”AADSTS50146: This application is required to be configured with an application-specific signing key. It is either not configured with one, or the key has expired or is not yet valid. Please contact the application’s administrator.”
[Repro steps]

Register a new application using dotnet new webapp --auth SingleOrg --tenant-id  or do it manually and link it to any existing application.
Use OpenID Connect as protocol and the Authorization Code flow with openid scope (ID Token).
Verify the auth is working and the received claims in the ID Token are the default ones
Create a new ClaimsMappingPolicy as in the examples:
New-AzureADPolicy -Definition @('{"ClaimsMappingPolicy":{"Version":1,"IncludeBasicClaimSet":"true", "ClaimsSchema": [{"Source":"user","ID":"employeeid","SamlClaimType":"http://schemas.xmlsoap.org/ws/2005/05/identity/claims/employeeid","JwtClaimType":"employeeid"}]}}') -DisplayName "Custom claims test" -Type "ClaimsMappingPolicy"
Link the policy to the application service principal
Add-AzureADServicePrincipalPolicy -Id  -RefObjectId
Re-run the authentication process and verify the error is returned upon authentication and redirection to the application

[Solution]
In order to avoid this issue, the app manifest should be edited in the application registration panel, so that the property acceptMappedClaims is set to true
{
"id": "XXXX",
"acceptMappedClaims": true,
"accessTokenAcceptedVersion": null,
"addIns": [],
"allowPublicClient": false,
...
[Request]
The desired is to have this whole process automated at some point (at least in GA).
However, mentioning the manifest change in the docs page would suffice for a Preview feature.

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

ID: 195c79a3-ccc5-8c6a-233e-95e344a9b4df
Version Independent ID: 401c7557-5817-bbd3-5bd8-8c97ef5a28cc
Content: Customize Azure AD tenant app claims (Powershell) - Microsoft identity platform
Content Source: articles/active-directory/develop/active-directory-claims-mapping.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

looks like #5111 but context might be different

@albigi Thanks for the feedback! We are currently investigating and will update you shortly.

@albigi, We are going through an internal discussion for this. Do allow us sometime and will get back to you soon on this thread.

@albigi Thanks for your feedback !!  I have assigned this issue to content author to investigate and update the document as appropriate with the manifest file updates.  Also, for your automation feature request please post this as a feedback at Uservoice.
@rwike77 can you please look into this and update the documentation as required.

Hi @rwike77 , @SaurabhSharma-MSFT
Is there any update?

I used to face this issue when trying to acquire a token for my own Web API.
To overcome it, I changed the identifierUris value in the API manifest for a verified domain of my tenant. For example, something like:
"identifierUris": [
"https://secorrecloudlab.onmicrosoft.com/120021025001298api"
],
Instead of the usual api://08ae2945-287b-4e62-8582-b1a1f2ac89c4

@albigi Sorry for the slow reply.  Thanks for the repro steps.  The error you receive is by design, you need the custom signing key first.  We'll get the error message improved and I've created a work item to improve the content.  #please-close

Hi @rwike77 ,
I actually cannot get it to work even if I create a custom signing key (powershell New-AzureADApplicationKeyCredential). My tokens still include a  which does not match any of the listed keys within the jwk_uri, no matter if I got with or without ...well-known...?appid=
Any ideas?


Labels for setting tags in tracing

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
For now, few informations are available in traces : method, host, status, backend name.
Backend name is heavily normalized, something like [a-zA-Z][a-zA-Z0-9\-]+.
When you put a traefik in front of lots of projects (the mutualized pattern), even if your backends are cleanly named (something like : group, project, service), you can't revert the information from opentracing tags. Host is not enough when you route with path patterns, backend name is squashed, and - can be used in your slugs.
A label pattern, like traefik.tracing.attribute.<key>=VALUE can be useful to tag traces.

I need this feature for https://github.com/factorysh/jaeger-traefik, a jaeger server that maintains prometheus counters.

I am not sure I understand the issue, do you want to add more tags to spans related to traefik? If that is the case, you have that information in the next span which is produced by the actual service, isn't?

Traefik traces are enough for me, services can be black box.
I Just need to link services to its project, just like docker-compose does.
Using jaeger and providing project name can't be mandatory : the information is already here, in the service name (before its normalization).

I see two use cases, you can have a service name for the a given service:
traefik.http.services.Service0.tracing.service_name=MY_APP_NAME
and also default tags for each service, through labels:
traefik.http.services.Service0.tracing.default_tags.tag0=<key>:<value>
(an alternative would be traefik.http.services.Service0.tracing.default_tags.<key>=<value>)
Both are usual use cases for tracing. Does that work for you?
What do you think @ldez?

Ping @ldez


Minimum Falling Path Sum II

Problem Link
https://leetcode.com/problems/minimum-falling-path-sum-ii/
Coding platform
Leetcode
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

Additional context (OPTIONAL)
I want to solve it in Java

I would like to do this in c++

I want to solve it in python
Please assign me this issue as a part of Cross Woc


tod and yes should be aliases in SPIRES syntax

Originally by valkyrie (@valkyriesavage) on 2011-07-07
tod == today
yes == yesterday

Please re-open in Invenio Query Parser if still needed.


Feature/6636 deploy using dacpac

Story: AB#6636
Task: AB#7573

Needs more work, parking it for now

Ready for review


Service Request: Spellcheck (microsoft/playwright)

Service Request: Spellcheck Automation
ℹ This is an archival item to capture a past request that I implemented according to these standards.
Prerequisites
This service is currently only a good fit in certain situations.

[x] The docs are in Markdown or similar format.
[x] The docs are written in English.

Request Information:

Repository URL: https://github.com/microsoft/playwright
Are you one of the maintainers of this repository? N/A

Or, link to where spellcheck is discussed for the repository: N/A

About Your Docs Setup

For Implementers or Would-be Implementers

[x] Implement according to the implementation guide
[x] Paste the PR link in the description or comments here
[x] Be sure you comment here along the way if you find anything challenging; we can use that to improve our process.

https://github.com/microsoft/playwright/pull/17919


Question: Should this work on a Teensy LC?

I am trying to create the Audio Guestbook with the old phone.  I purchased a Teensy LC by mistake and time is running out.
When I compile I get several errors for :usb_mtp_status not declared

Note: Better to ask these questions on the forum,
Simple answer is, I am not sure. @PaulStoffregen @mjs513  ?
Sorry I am not on my main machine, on Ubuntu test machine.
But if I edit the sources and add it to the couple of #if in MTP_Teensy.h/.cpp
like:
#if defined(__MK20DX128__) || defined(__MK20DX256__) ||                        \
    defined(__MK64FX512__) || defined(__MK66FX1M0__) ||                        \
    defined(__MKL26Z64__)

2 places in .cpp 1 in .h
It mostly builds.  But even the most simple SD example, then currently fails to link,
/home/kurte/.arduino15/packages/teensy/tools/teensy-compile/11.3.1-beta1/arm/bin/../lib/gcc/arm-none-eabi/11.3.1/../../../../arm-none-eabi/bin/ld: region `FLASH' overflowed by 6016 bytes
/home/kurte/.arduino15/packages/teensy/tools/teensy-compile/11.3.1-beta1/arm/bin/../lib/gcc/arm-none-eabi/11.3.1/../../../../arm-none-eabi/bin/ld: region `RAM' overflowed by 4748 bytes
collect2: error: ld returned 1 exit status

Not enough memory.  So would need to go through and see what things could be stripped..
And probably a lot would need to be fixed up to make room for the Audio Guestbook which probably needs a lot more space.

Thanks for the reply, didn't know where the forum was. First time user and in a panic :-/  Think this is going to have to go on the shelf and order a Teensy proper.
Thanks again.


nominating rusty-machine

(no further commentary)

(Oh, actually: disclaimer: I'm a contributor.)

Thank you


Disable new registrations

Implemented feature and added Cucumber tests.
Right now the string value "1" is considered "enabled", and anything else is "disabled". All we need is true/false, but Settings are saved as strings.
Fixes #24.

Nice implementation! LGTM!

You got a big-gollum installation running with users and stuff somewhere? Didn't think anyone was still using this :D pretty cool.
Happy Holidays!

Yes haha. I'm using it for a documentation server at my day job. Many software companies don't have much documentation (unfortunately), so this is usually one of the first things I do at a new company. I also want to throw an instance up on my personal server at some point.
Thanks, and Merry Christmas!


feat: add the env variable KITTY_SCROLLBACK_NVIM

:tada: This PR is included in version 3.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


:latest should not point to a pre-release

Describe the bug
Github releases show v0.40 as pre-release, but when using the official container, tfsec/tfsec:latest gives you v0.04 instead of official latest release. Using :latest in your CI builds for example makes sense, but then you definitely do not want to get a pre-release.
To Reproduce
docker run --rm -it -v "$(pwd):/src" tfsec/tfsec --version
0.40.0

Expected behavior
I expect docker hub :latest tag to point to latest official release, not a pre-release. I expect the github release status to match docker hub tags.

thanks @vertti  - release state of v0.40 has been corrected


(WIP) Proposal to simplify FocusTrap

This is a proposal to replace FocusTrap with a root Focus instead, in order to:

More closely match native behavior.  Only web should lose focus when clicking the background, not desktop.
Allow overriding this behavior, like buttons that don't remove focus from text fields (as discussed in https://github.com/flutter/flutter/pull/76072#issuecomment-902187533).

TODO

Read through https://github.com/flutter/flutter/issues/86972 and make sure nothing was missed here.
Tap support or just clicks?

Flutter (this PR)
Native web

Related issues
Fixes https://github.com/flutter/flutter/issues/86972
Unblocks https://github.com/flutter/flutter/pull/76072
Original FocusTrap PR: https://github.com/flutter/flutter/pull/82575
Follow up that fixed clicks on prefix/suffix: https://github.com/flutter/flutter/pull/86041

Gold has detected about 10 new digest(s) on patchset 3.
View them at https://flutter-gold.skia.org/cl/github/90240

Gold has detected about 7 new digest(s) on patchset 4.
View them at https://flutter-gold.skia.org/cl/github/90240

Gold has detected about 10 new digest(s) on patchset 6.
View them at https://flutter-gold.skia.org/cl/github/90240

Gold has detected about 1 new digest(s) on patchset 7.
View them at https://flutter-gold.skia.org/cl/github/90240

Gold has detected about 4 new digest(s) on patchset 7.
View them at https://flutter-gold.skia.org/cl/github/90240

Closing this until I have time to return to it.

@justinmc do you remember if this was looking promising or if it ran into some problems?
FWIW, I would definitely support efforts to remove FocusTrap entirely.

I'm going to update this PR and see if tests pass, etc.  I remember really doubting myself that I could make this work with such a simple change, but I can't remember if I specifically ran into any major blockers.

@Hixie Here's the major blocker: the tests don't pass on web because the onTapDown is swallowing gestures.  For example, for iOS's swipe back gesture, the onHorizontalDragUpdate is never called when the onTapDown is in the tree above it.
I imagine this can be solved with a custom tap gesture detector or something like that?  If I get more time maybe I can figure out something that works.


fix: clippy errors

This PR should fix the CI issues. Basically making clippy happy again.
When this is merged could you perhaps make a new release?

Hey, thank you for spotting this originally.
Finally, this is fixed in https://github.com/vorot93/tokio-tar/pull/21 and now we can think of a new release.
I see to lack the credentials for that, so a separate issue for that would be good to create?

Finally found the place where a release was discussed: it's unfortunate that this repo does not have issues for that.
There's a new crate version released recently.