chore(bzlmod): add trivial pass-through module extension for oci_pull

However it trips over the labels in our generated build files making assumptions about repository names, needs some fixing

I made an example to test this, and there's a bug. Problem here is
ERROR: /home/alexeagle/Projects/bazel-examples/oci_go_image/BUILD.bazel:34:10: no such target '@rules_oci~override~oci~distroless_base//:image': target 'image' not declared in package '' defined by /shared/cache/bazel/user_base/359b45a24cf55cc22380e941fe3bcf54/external/rules_oci~override~oci~distroless_base/BUILD.bazel

which is caused by https://github.com/bazel-contrib/rules_oci/blob/main/oci/pull.bzl#L246
I wanted to have nice syntax sugar for users to choose a base image like just base = "@distroless_static" but one option here is to give that up and require a label like @distroless_static//:image - then we don't need to know the repo name here to create a target with a matching name.

I made an example to test this, and there's a bug. Problem here is
ERROR: /home/alexeagle/Projects/bazel-examples/oci_go_image/BUILD.bazel:34:10:
no such target '@rules_oci~override~oci~distroless_base//:distroless_base': target 'distroless_base' not declared in package ''
defined by /shared/cache/bazel/user_base/359b45a24cf55cc22380e941fe3bcf54/external/rules_oci~override~oci~distroless_base/BUILD.bazel

which is caused by https://github.com/bazel-contrib/rules_oci/blob/main/oci/pull.bzl#L246
I wanted to have nice syntax sugar for users to choose a base image like just base = "@distroless_static" but one option here is to give that up and require a label like @distroless_static//:image - then we don't need to know the repo name here to create a target with a matching name.

I'm missing some pieces here. So you want to be able to pass in base = "@distroless_static" into oci_image? As in you want to pass a string rather than a label and have it do the equivalent of appending the //:image target?

okay the downstream example is green now!


chore: added MIT license

npm is moaning about a missing license. Can you please add on. Thx!

@vojtajina It's your code! So please a short ok releasing it unter MIT!?

Yes.


How can I stop the animation from automatically playing, and turn off looping?

I've just exported using the plugin and can't figure out how to stop the animation from automatically playing and looping?
The best I've been able to do is to call comp.mc.stop(); straight after SVGAnim gets initialised.

You should be able to create a keyframe for the first frame on a new layer in the timeline of the movieclip and call this.stop();  in the actions panel.
For looping the following should work:
comp.mc.loops = false;
The method you're using above should work as well, but it seems to stop before rendering which is a bug.

This is fixed on dev if you update the generated files with the files in the following commit, it should work as expected
https://github.com/cjgammon/SnapSVG-Animator/commit/c7949f8e6d5a67aa2bd3c24d674523fa82ee3c8b


FutureWarning concerning default "rcond" value with numpy 1.16.1

When using pyearth with numpy 1.16.1, I get the following FutureWarinings:
python3.5/site-packages/pyearth/earth.py:802: FutureWarning: 'rcond' parameter will change to the default of machine precision times ``max(M, N)`` where M and N are the input matrix dimensions. To use the future default and silence this warning we advise to pass 'rcond=None', to keep using the old, explicitly pass 'rcond=-1'.
and
 pruning_passer.run() python3.5//site-packages/pyearth/earth.py:1055: FutureWarning: 'rcond' parameter will change to the default of machine precision times ``max(M, N)`` where M and N are the input matrix dimensions.
The second one is easy to fix by just passing rcond=-1 to np.linalg.lstsq in earth.py:1055 to keep using the old behaviour. But I do not know much about cython and was not able to fix the second one myself - even though I thought it might be simple.
Since I use Earth as base estimator in a gradient boosting implementation, it would be great to have less of these warnings issued.

@MissingShoes These warnings are currently nothing to worry about.  They are fixed in the v0.2dev branch.  That branch is not yet ready to be released because of issues with scikit-learn compatibility, but you could use it if you're concerned about the warnings.

@jcrudy thanks for the info. I will use the dev branch instead.

@MissingShoes Sounds good.  As far as I can tell, the dev branch is currently fully functional except for some testing issues.  I'd recommend you point any environment configuration files at the current commit rather than just the branch, since I'll be doing development on it and might break something later.
Also, I'm going to keep this issue open until these warnings are fixed in a release.  Thanks for bringing it up!

This obviously doesn't fix the root problem at hand, but one easy way to make the warnings go away would be to list this line right below your import statements:
import numpy as np
np.warnings.filterwarnings('ignore')

Change the line lstsq(A,b) to lstsq(A,b,rcond=None)


Change Stage() to return the full stage name.

This is a breaking change. The motivation is that it's confusing to have two versions of a stage name floating around -- the full name and the short name.
@viroulep I'd appreciate your thoughts on this!

I think it's a good change!
Assuming the name of the stage is the first "word" of the room name is quite English-centered; I don't see this tool being used for a competition large enough but not using communication in English, but I think it's better to just use the full stage name.
Especially given you can just #define the stage name and use this in stage helpers!
I think you can merge this without any issue; I think I've kept things DRY-ed enough that I can make this change easily :)


Add IAM Support

IBM-Cloud/gp-common#24

gp-ruby-client v0.2.0 has iam support.

gp-ruby-client v0.2.0 has iam support implemented


No xorgxrdp log

I have no .xorgxrdp.%s.log on my system thus cannot check if my connection utilize RemoteFX.

First of all, let me know basic informations.

distro
xrdp version
xorgxrdp version

And make sure you're using xorgxrdp.  Then find -config xrdp/xorg.log line, that's the instance of xrgxrdp. You'll see -logfile .xorgxrdp.%s.log in the line. The log can be found in your home directory.
$ ps uxaww | grep Xorg
Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -noreset -nolisten tcp -logfile .xorgxrdp.%s.log

Also, you can see the Xorg arguments in xrdp-sesman.log.
[20180307-00:10:30] [INFO ] starting Xorg session...
[20180307-00:10:31] [INFO ] calling auth_start_session from pid 70868
[20180307-00:10:31] [INFO ] Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -noreset -nolisten tcp -logfile .xorgxrdp.%s.log
[20180307-00:10:31] [CORE ] waiting for window manager (pid 70870) to exit
[20180307-09:50:09] [INFO ] A connection received from ::1 port 56733

distro
Xubuntu 17.10
xrdp version
0.9.1
xorgxrdp version
0.9.1-9
ps uxaww | grep Xorg gives
xoob       1415  0.2  2.2<PHONE_NUMBER>2 ?        Sl   10:08   0:02 /usr/lib/xorg/Xorg :10 -auth .Xauthority -config xrdp/xorg.conf -noreset -nolisten tcp
I don't see the -logfile addition

Okay, I remember Debian intentionally disables the log. Need to enable it again.
I don't have Debian/Ubuntu VM right now. Give me time.
Also, I think RemoteFX status should be logged xrdp.log, too. I'll do it in future version.

Finally I got RFX capture :)
[   102.423] rdpClientConProcessMsgClientInfo: got RFX capture

And got the working log. But I had to upgrade to 0.9.5 for that. Closing the issue.


Support for Symfony Console Applications

Great package 👏 , as the title says, any plan for symfony console?

@sunel The integration with Symfony 4 is coming real soon.

@sunel Just created https://github.com/nunomaduro/collision-adapter-symfony. Enjoy!

@nunomaduro
The collision-adapter-symfony is for symfony application using bundle.
But for console like the below, how can i integrate this
https://symfony.com/doc/current/components/console.html

@sunel Just add this code (new \NunoMaduro\Collision\Provider)->register(); on your cli file.

Hello,
I've searched for  (new \NunoMaduro\Collision\Provider)->register(); all over the web; there are many sites that have the installation steps but they reference the main docs page - which does not have a file noted.

Just add this code (new \NunoMaduro\Collision\Provider)->register(); on your cli file.

May I please know, what is the cli file you are referencing? So - in a Laravel 7 app where would this file be found?
Thank you much,
Trevor


fix_dh_paramgen_type: Avoid crash with invalid paramgen type

This fixes a segfault on openssl genpkey -genparam -algorithm DH -pkeyopt dh_paramgen_type:4

This pull request is ready to merge

Merged to master and 3.0 branches. Thank you for your reviews.


support SET variable in queries

while I can put SET ROLE in the queries, I cannot do SET variable TO value before queries, which is useful if I have row based security policies that utilize those variables.

@wasaga I haven't used SET variable TO value before, can you provide a bigger example of how it works in practice and how you're using it currently?

Copied from Omesh on SO
User-defined variables (prefixed with @):
You can access any user-defined variable without declaring it or initializing it. If you refer to a variable that has not been initialized, it has a value of NULL and a type of string.
SELECT @var_any_var_name

You can initialize a variable using SET or SELECT statement:
SET @start = 1, @finish = 10;
#OR
SELECT @start := 1, @finish := 10;

SELECT * FROM places WHERE place BETWEEN @start AND @finish;

User variables can be assigned a value from a limited set of data types: integer, decimal, floating-point, binary or nonbinary string, or NULL value.
User-defined variables are session-specific. That is, a user variable defined by one client cannot be seen or used by other clients.
They can be used in SELECT queries using Advanced MySQL user variable techniques.

my usecase is using it together with postgres row-level security, so that every query is executing in the context of currently set database role, authenticated user ID and some other parameters.
I ended up writing a wrapper function that opens a transaction, sets those parameters just for the transaction scope then runs a query.
func Run(ctx context.Context, db *sql.DB, readOnly ReadOnly, uid string, auth AuthProvider, role DBRole, fn func(context.Context, *Queries) error) (finalErr error) {
	conn, err := db.Conn(ctx)
	if err != nil {
		return err
	}
	defer func() {
		finalErr = multierror.Append(finalErr, conn.Close()).ErrorOrNil()
	}()

	tx, err := conn.BeginTx(ctx, &sql.TxOptions{ReadOnly: bool(readOnly)})
	if err != nil {
		return err
	}

	_, err = tx.ExecContext(ctx,
		`SELECT set_config('role', $1, true), set_config('request.uid', $2, true), set_config('request.auth', $3, true)`,
		string(role), uid, string(auth))
	if err != nil {
		return multierror.Append(errors.Wrap(err, "set security context"), tx.Rollback())
	}

	if err = fn(ctx, New(tx)); err != nil {
		return multierror.Append(err, tx.Rollback())
	}

	return tx.Commit()
}

I believe the above snippet may be generalized and be part of sqlc to i.e. validate and safely expose JWT session parameters to the queries - i.e. check out http://postgrest.org/en/v5.0/auth.html
Postgres doesn't have built-in or good 3rd party JWT libraries, thus doing it in Go and propagate down to database engine probably makes a lot of sense to promote best practices securing the data access, and I believe the concept is pretty much same in other database engines.

Useful to store actions history as well
-- name: DeleteUser :exec
SET app.current_user = $1;
DELETE FROM "user"
 WHERE id = $2;

Hello, any update about this issue? I would need to use this type of query.

It would be especially usefull to integrate with Supabase


[CARBONDATA-1222] Residual files created from Update are not deleted after clean operation

Residual files are getting cleaned after adding this fix.

Can one of the admins verify this patch?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/598/Failed Tests: 1carbondata-pr-spark-1.6/org.apache.carbondata:carbondata-spark-common-test: 1org.apache.carbondata.spark.testsuite.dataretention.DataRetentionConcurrencyTestCase.DataRetention_Concurrency_load_date

Thanks for working on this issue. You always can use the same PR by rebasing the code with master and force pushing to this PR branch. Raising multiple PRs for same issue is not encouraged.


Feat/tree view

Linked Issue
Closes #1668
Description
Basic implementation of the tree view.
Changsets
Checklist
Please read and apply all contribution requirements.

[x] This PR targets the dev branch (NEVER master)
[x] Documentation reflects all relevant changes
[x] Branch is prefixed with: docs/, feat/, chore/, bugfix/
[x] Ensure Svelte and Typescript linting is current - run pnpm check
[x] Ensure Prettier linting is current - run pnpm format
[x] All test cases are passing - run pnpm test
[x] Includes a changeset (if relevant; see above)

@Mahmoud-zino unfortunately I'm short on time, but here's a some quick notes based on my initial audit:

The standard name for this component is "Tree View". I'm fine with <TreeView> + <TreeItem> or <TreeViewItem>
The Disclosure element is perfect for this use case. Note that some folks may request animation for this, but it's not possible due to limitations with the elements themselves. We went through this with the Accordion and finally gave in and used non-semenatic elements in order to facilitate this. However, in the case of the Tree View, I think this is PERFECT as is. It helps differentiate it from the Accordion in fact.
Make sure our default padding and gap sizes are p-4 and gap-4 to be consistent with other components
The use of variant-soft is consistent with list styles, which is great, but you might want to wrap the featured demo in a card - it is a bit hard to see on the gradient background (reference)
I think most of your use of ARIA attributes is correct, but I'm not sure if aria-selected="false" is supposed to be hardcoded like that? Definitely follow what the ARIA APG guidelines says here. Review their demos too if you can! (link)
Try to avoid setting dynamic values inline in the reactive classes. This can make the code hard to read. So before:

$: classesBase = `${cBase} ${$$slots.children ? '' : noContentPadding} ${$$props.class ?? ''}`;

After:
$: classesPadding = $$slots.children ? '' : noContentPadding;
// Reactive Classes
$: classesBase = `${cBase} ${currentPadding} ${$$props.class ?? ''}`;

Questions

I'm not sure I understand the use of noContentPadding - can you explain this?
Can we separate the padding from the indentation? Maybe add a specific indent: CssClasses prop specifically for providing a uniform indent style? Margin might be most appropriate for this. Folks might want a more pronounced indentation and not want to have to run the padding offset math in their heads.

@endigo9740
I have applied your suggestions.

Regarding the fixed aria-selected="false", I was forced into it, because having treeitem role requires an aria-selected and since we don't have selection in the basic example I just fixed it to false.

I was not sure if we wanted to keep using noContentPadding, It was there because when a child doesn't have children, the caret will not be displayed and the position of the items will be off. However I changed it now to instead of hiding the carret from the dom (svelte-if) I am now using css-visiblity classes. I think this solution is a lot better but let me know if you have other ideas.

@Mahmoud-zino I've implemented a number of minor changes and updates to this component. Overall I think it's working really well, but there were a few nitpick things I adjusted.
Additionally, my OCD was bothering me that the non-expandable rows had a large empty space when hovering each item, so I've introduced a second hyphen character that takes the place when a caret is not shown. Let me know what you think about this. Users can opt to adjust the opacity on this, even setting it to opacity-0 to make it hidden like your original version.

@endigo9740 yeah It looks really great now and the users have control over displaying it or not, which is always helpful 👍.

I search a UI framework with a Tree View and I just try this PR HERE.
It's a nice component but a lot should be done to be a complete Tree View, with DnD support

@nicolas-albert Skeleton is open source which means folks like yourself can chime in and make feature requests or even contribute your own changes directly as long as you follow our contribution guidelines:
https://www.skeleton.dev/docs/contributing
Given this feature was merged 30 minutes before you posted we can't go back in time and update this pull request specifically. However, we still have time before the next release if you have minor feature suggestions. Larger features might need more consideration and likely wouldn't make it into the first release (just short of 2 weeks from now)
Per your specific requests above:

but a lot should be done to be a complete Tree View

We're not really trying to compete with any dedicated library. Dedicated libraries typically have one or more people focused specifically on a single feature, we're focusing on a whole library. Our components are typically going to be simpler in nature as we treat them as the "primitives" (read: building blocks) for building large features. If there's a dedicated library that offers a lot more features, you're always welcome to substitute that in place of any of Skeleton's features.

DnD support

Do you mean drag and drop? If so I'd need to see a relevant use case for this. It's not immediately clear to me how this would work or what problem it would solve. We were building our Tree View to focus purely on navigation at this time.

(multi-)selection

Multi-selection of what exactly? In a tree view your drill down the tree of expandable options. You can toggle multiple options right now. Maybe an example would help here?

@endigo9740 contributing can be an option. I evaluating alive Svelte UI libraries with a default nice look, a set of components in order to make an IDE like app, including file management and other concepts based on a treeview.
I've done a pure Svelte/HTML basic treeview and I'm just searching if one already exists or if I have to continue to developing it, or maybe contribute to one.
Thank you for your answer!

@nicolas-albert yeah unfortunately I'm not aware of any alternatives for Svelte off hand. Maybe check Flowbite or Tailwind? They provide HTML + Tailwind elements that can be integrated with Skeleton's theme system:

https://www.skeleton.dev/blog/skeleton-plus-flowbite
https://www.skeleton.dev/blog/skeleton-plus-tailwind-ui

These would be for the visible UI, but then allow you full control over the logic and functionality as you see fit.
You could also clone our component and modify as desired locally in your project.
The goal for our component was a simple drill down navigation for a website - so if you're needing something very specific and purpose built, that might mean you would be better off rolling your own solution through a custom component.


Create new projects NumSharp.Python for Iron python

I know it sounds strange. But I come up with the idea that the best way to see how close we are to numpy is to try NumSharp in python.
Which project would be best? Iron Python. I just downloaded Iron python 2.7 and imported NumSharp. Yeah it works.
Just at some points it feels strange since Iron python has generics c python does not have. For this reason I would like say that we should start /src/NumSharp.Python project. With NumSharp iron Python could finally bring numpy to their stack. :)

Ok done.
But think we can change from python Modul to C# class Lib since I chatted with ironpython team and they explained most of python objects implement the common interfaces like IList.
https://files.gitter.im/IronLanguages/ironpython/XAVD/image.png

Done - now (merge with #99 ) NumSharp for Ironpython is totally C# classlib and seems it works.


Shape Population Viewer on Linux will not display color pick dialog

When using SPV on Linux as an integrated widget, the color pick dialog for the color bar will not show up when double clicking the circle within the colorbar.
cc: @jcfr

Update, it does not work for MacOS either.
It does work for Slicer, not for SlicerSALT

"Problem" has been identified, a fix is being implemented.
In a nutshell, in the standalone SPV, the GPL version of the Qt widget is used, in the one integrated, we rely on VTK chart and the "double click on point" event wasn't wired.

After enabling the widget on top of the gradient, here is how it looks:

The next step is to make sure the change are propagated to the loaded models without having to click on "Reset range" button

Is this ready for testing? If so what version?

Change integrated in ShapePopulationViewer and ExtensionIndex updated


feat(uppercase) Be able to choose number of uppercase letters

Some software have checks for number of upper/lower letters.
This Pull Request let users choose the number of upper letters.

Hi @holyhope thank you for the Pull Request. This feels pretty specific and also changes our public API, so I'm not sure we'll be able to accept this contribution as-is. Can you tell me more about the use case?

Hello @sethvargo,
About the public API
IMO updating the public API was pretty much nice since it goes in the same direction that how it was already implemented. I mean it seems normal to have the expected number of upper letters when the feature to have expected digits and specials characters count is already available.
But if you want, I can move it to a /v2 folder to have a transition time of the public API?
Use case
I am working on a Kubernetes operator: https://github.com/goharbor/harbor-operator
I want to generate password for users.
Since users may choose their password, Harbor checks security level and requires 1 uppercase, 1 lowercase, 1 number and 1 special char. Otherwise it denies the password and I need to retry.

Got it - thanks for clarifying. I don't think we need a /v2 - we can use git tags with Go modules. However, I think the current API is limiting and would like to change the entire thing at once as a single breaking change. In short, instead of:
Generate(var, var, var, var...)

I'd like to switch to:
Generate(&GenerateConfig{
  Var: ...
  Var: ...
  Var: ...
})

What do you think? To be clear, I'm not asking you to do that work. I think it would provide flexibility for these types of enhancements in the future.

It will be really great. Event more based on a interface
interface {
    GetDigitsCount(context.Context) int32
    GetSpecialsCount(context.Context) int32
   ...
}

Probably with a BasicGeneratorOption with wrap it.
What do you think about something like
type Pickup interface {
    // PickupSequence returns the slice on byte allowed for password generation,
    // and the number of bytes to pickup from this sequence
    PickupSequence(context.Context) ([]byte, int32, error)
}

So the Generate() function would look like:
func Generate(ctx context.Context, pickups ...Pickup) ([]byte, error) {
    // TODO iterate over pickups and generate a mixed password from all sequences
}

Written with my phone, so I may be unclear, just let me know, I will rephrase it

Hey sorry - this was auto-closed when I renamed master -> main.


TypeError: not enough arguments for format string in rez-cp _copy_variant_payload

I get the following error when running rez-cp as part of a release procedure. It appears as though it should raise a PackageCopyError, but instead breaks due to a bad format string.
Traceback (most recent call last):
  File "/rel/third_party/rez/2.106.0/bin/rez/rez-cp", line 8, in <module>
    sys.exit(run_rez_cp())
  File "/rel/third_party/rez/2.106.0/lib/python3.7/site-packages/rez/cli/_entry_points.py", line 127, in run_rez_cp
    return run("cp")
  File "/rel/third_party/rez/2.106.0/lib/python3.7/site-packages/rez/cli/_main.py", line 191, in run
    returncode = run_cmd()
  File "/rel/third_party/rez/2.106.0/lib/python3.7/site-packages/rez/cli/_main.py", line 183, in run_cmd
    return func(opts, opts.parser, extra_arg_groups)
  File "/rel/third_party/rez/2.106.0/lib/python3.7/site-packages/rez/cli/cp.py", line 168, in command
    dry_run=opts.dry_run
  File "/rel/third_party/rez/2.106.0/lib/python3.7/site-packages/rez/package_copy.py", line 206, in copy_package
    verbose=verbose
  File "/rel/third_party/rez/2.106.0/lib/python3.7/site-packages/rez/package_copy.py", line 248, in _copy_variant_payload
    "be present on disk (%s)." % src_variant.uri, variant_root
TypeError: not enough arguments for format string

Environment

OS linux
Rez version 2.106.0
Rez python version 3.7.7

To Reproduce

change a package.py's variants so that they no longer match the file path.
rez-cp the resulting package to trigger the PackageCopyError that's supposed to occur. You will instead get the above TypeError.

I'm seeing something similar on Windows, though it's related to the shortened variant symlinks not working on Windows.


BUG: Fix multilaunch spam

This should fix the issue with #144.
The issue was caused by that the run stack was never cleaned and the termination check included finished runs. Fixed by cleaning the run stack periodically and also now it should not terminate finished runs.

Codecov Report
Base: 95.36% // Head: 95.36% // No change to project coverage :thumbsup:

Coverage data is based on head (2fae76a) compared to base (7dbd1c0).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #160   +/-   ##
=======================================
  Coverage   95.36%   95.36%
=======================================
  Files          80       80
  Lines        4660     4660
=======================================
  Hits         4444     4444
  Misses        216      216

Impacted Files
Coverage Δ

rocketry/core/schedule.py
93.63% <100.00%> (+0.02%)
:arrow_up:

rocketry/core/task.py
92.23% <100.00%> (-0.02%)
:arrow_down:

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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix missing kafka max metrics in Atlas

Only pass numbers which are finite to servo for reporting to Atlas.


Contextual narrowing not narrowing on generic map.

Bug Report

🔎 Search Terms

contextual narrowing
type narrowing 4.3
4.3

🕗 Version & Regression Information

This is something that I expected to work after 4.3's Contextual Narrowing for Generics

⏯ Playground Link

Playground link with relevant code
💻 Code

declare const eventType: keyof DocumentEventMap;

document.addEventListener(eventType, (event) => {
  event; // Here `event` is `Event | UIEvent | AnimationEvent | MouseEvent | InputEvent | FocusEvent | CompositionEvent | ... 9 more ... | ClipboardEvent
  if (eventType === `click`) {
    event; // Expected: `event` to be type `MouseEvent`. Actual: It's the union from above.
  }
});

🙁 Actual behavior
Second reference of event is type Event | UIEvent | AnimationEvent | MouseEvent | InputEvent | FocusEvent | CompositionEvent | ... 9 more ... | ClipboardEvent.
🙂 Expected behavior
Second reference of event to be of type MouseEvent.

Generic functions like document.addEventListener() have their type parameters specified when you call them.  In the case above, the K type parameter is inferred to be keyof DocumentEventMap and therefore the type of event in the callback is the full union; it is no longer "generic" in the way that #43183 (the contextual narrowing feature) would have any bearing on.
It seems like what you really want is something like #27808 where the type parameter K is treated as some unknown single member of the keyof DocumentEventMap union... although then the compiler might not even let you call document.addEventListener() with a union-typed eventType.
This also feels like eventType and event are supposed to be correlated unions but the compiler has no way to track that; which is #30581.  (This is where #25051 would have helped, if you could wrap the call to document.addEventListener() in a hypothetical type switch (eventType) {...} block.  Oh well!)

Thanks @jcalz  for the detailed response! I agree that this is similar to #30581.
Unlike #30581, I was under the impression that the if check would be sufficient to narrow the type to MouseEvent.

Right - the compiler doesn’t understand that eventType and event are interrelated for the purposes of type narrowing.


🐛 BUG:

Which Cloudflare product(s) does this pertain to?
Wrangler core
What version(s) of the tool(s) are you using?
wrangler2
What version of Node are you using?
v18.16.0
What operating system are you using?
Fedora 38
Describe the Bug
Installed Wrangler via yarn global add wrangler
yarn global v1.22.19
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...
success Installed<EMAIL_ADDRESS>with binaries:
      - wrangler
      - wrangler2

wrangler --version returns error:
bash: wrangler: command not found...
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

Let's see if this helps:
This will output the path to the Wrangler binary.
Open your .bashrc or .zshrc file. This file is located in your home directory.
Add the following line to your .bashrc or .zshrc file:
export PATH=$PATH:`which wrangler`
Save your .bashrc or .zshrc file.

Reload your .bashrc or .zshrc file by running the following command:
source ~/.bashrc
Save your file and run source ~/.bashrc to reload it. Now you should be able to run wrangler without any errors.
To test that the wrangler command is working, run wrangler --version.

hi @Boffice :) did @JacobMGEvans 's recommendation above solve your issue?

Closing this. If it continues to be an issue, please reopen.


Use slog

Replace log with slog, add some optional debug log messages and verbosity

Really great stuff, some small comments


Failed integration in version 2023.3

Describe the bug
Logger: homeassistant.helpers.frame
Source: helpers/frame.py:77
First occurred: 09:08:54 (1 occurrences)
Last logged: 09:08:54
Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups; this will fail in version 2023.3. Please report issue to the custom integration author for miwifi using this method at custom_components/miwifi/init.py, line 100: hass.config_entries.async_setup_platforms(entry, PLATFORMS)
Please complete the following information:

HASS version: 2023.2.1
Component version v2.9.0
Router model AX3600
Router firmware version 3.0.22 EU

Good evening! I have same error
Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups; this will fail in version 2023.3. Please report issue to the custom integration author for miwifi using this method at custom_components/miwifi/init.py, line 100: hass.config_entries.async_setup_platforms(entry, PLATFORMS)
AX3600

Fixed in release https://github.com/dmamontov/hass-miwifi/releases/tag/v3.0.0
@edudele thanks.


the repository does not hava a release file

when i execute the scripts:setup-apt-repo,it got a error:the repository  'https://packages.groonga.org/ubuntu focal Release' does not hava a release file,had you ever met such error?how can i fix it? Zulip version:4.5
thanks everybody !
CZO thread

@wuyz-harder - Thanks for the report!
Reporting the issue in the #production help stream in the Zulip development community is a good way to get more interactive help with production issues. Please provide details there in code block format.

This smells like a networking problem, or the 4.5 release is old enough that pgroonga has changed their release repository organization.
Please upgrade to 7.x.  I'm also not sure why you need to run setup-apt-repo by hand.  In either case, I'm closing this issue -- feel free to discuss with us on CZO, as @laurynmm described in the above comment.


Add feature to view profile on tapping @mention for a user

Closes #3879.
This is a common pattern in messaging apps, and a feature I felt was missing.

I also created a selector  - getUserForId and wrote tests for it.
Tested that it works in varoius narrows, such as stream narrows, topic narrows, PM narrows, starred messages narow.

Here's how the Alert dialog looks for an invalid user_id

@ray-kraesig have pushed some changes.

Incidentally, I'd suggest holding off on this PR for now. There are plans to use IDs, rather than emails, almost everywhere as user identifiers; after #3789 is merged, it should be easy to convert navigateToAccountDetails in particular, which should save a fair amount of trouble.

@ray-kraesig https://github.com/zulip/zulip-mobile/pull/3789 has been merged. What changes should I make now?

Something similar to that. Refactoring navigateToAccountDetails to use user IDs should probably go hand-in-hand with refactoring getUserIsActive and AccountDetailsScreen to do the same.
That's enough to be worth submitting on its own as a separate PR, though. (We don't have an issue for it specifically, but it's part of #3764.)

https://github.com/zulip/zulip-mobile/pull/3968 is now merged :tada:! I've made some changes to this PR to use its results. Additionaly, the first commit addresses https://github.com/zulip/zulip-mobile/pull/3968#discussion_r398221311.
@ray-kraesig @gnprice please review.

Merged – and thanks! (I've taken the liberty of amending the first commit to strike `[nfc]' from the summary and apply the mentioned grammar nit.)

Thanks for merging!

and apply the mentioned grammar nit

Which nit are you referring to here?

... what on earth, GitHub. :disappointed: Hopefully, it's been posted now.
(My other comment in the review was about [nfc] not being applicable, since the error paths were very different. It's gone, apparently, but that's no great loss.)


Try to find all leaked widget that causes failing tests

Description
This PR is a next PR to clean up bugs in tests that are causing segfaults for some test run. Previous one was #5373
In this PR I have focused on finding widgets that are not properly teardown.
Next to the improving of tests:

change part of the test to use QtViewer instead of full Viewer
add missed widget to qtbot
use before_close_func instead of custom teardown code in fixture

This PR contains a code for detecting of leaked widgets. This code contains two parts.
The own subclass of QtBot that ensures that object, with is hadled using addWidget name is set to 'handled_widget' affter the test end.
And _find_dangling_widgets fixture that is executed after the test end (and after qtbot teardown) and looks for all top-level widgets that do not have parents and check if this widgets are either:

_QtMainWindow - we handle it using make_napari_viewer fixture, without using qtbot.
some widget form qtconsole (I do not have time to investigate if we could make a PR to qtconsole to avoid it).
is not CanvasBackendDesktop that is leaking in test_sys_info.py and may require PR to vispy to fix this. (or some monkeypatching)
is not handled by qtbot already by having name "handled_widget"
Is not having __qtbot_handled__ set to true. This option requires more investigation to make this working deterministic. Maybe need to be removed after review.

If the widget does not fit to any above category, it is reported as an error.
There is no guarantee that all problematic widgets are found. So if this PR will be merged then some random test failures need to be observed, as some widget not too often leak from tests.

Thank you @Czaki, this is great work! I am not knowledgable enough in Qt to review but there is some very useful information here that would be great to get into our docs (probably in testing). I can think of these items:

advise when to use make_napari_viewer and when QtViewer is adequate
advise on the autouse fixtures we add when qapp is used (I had no idea of this)
how _find_dangling_widgets checks for widgets (essentially your bullet list above, it's very useful)

advise when to use make_napari_viewer and when QtViewer is adequate

I have put this in "Use the smallest possible widgets for tests" section of https://czaki.github.io/blog/2024/09/16/preventing-segfaults-in-test-suite-that-has-qt-tests/ feel free to take information from there to docs.

advise on the autouse fixtures we add when qapp is used (I had no idea of this)

I do not want to advertise it too much. It is a kind of magic, that may introduce a confusion to contributors. I much more prefer split on Qt and non Qt tests based on simple directory structure and using @pytest.fixture(autouse=True) in non-top level conftest.py. But in napari it will require to keep tests outside the code. That means a big refactor.

how _find_dangling_widgets checks for widgets (essentially your bullet list above, it's very useful)

I would rather put this in a comment in code. It may outdated easy, and if it is documented in code (like _dangling_qthreads) it is much easier to spot this during code review.
However, experience from writing _dangling_qthreads shows that my initial comments were totally rewritten by @jni to be clear by less experienced user.

I have put this in "Use the smallest possible widgets for tests" section of https://czaki.github.io/blog/2024/09/16/preventing-segfaults-in-test-suite-that-has-qt-tests/ feel free to take information from there to docs.

I mean specifically for napari case. For dummies, can you say what does Viewer have that QtViewer does not (and thus in what case you would need to use full Viewer) ? I'm happy to add a section on this if you give me a rough guide?

I would rather put this in a comment in code.

Sure, would be great to see this in the code.

I mean specifically for napari case. For dummies, can you say what does Viewer have that QtViewer does not (and thus in what case you would need to use full Viewer) ? I'm happy to add a section on this if you give me a rough guide?

Based on my knowledge, the main difference is the option to dock widgets (especially all plugin widgets), and has menus.
It is used, also, to detect active window (but it may be a problem in tests).
Fast checking of code does not show more usecases.
Using a pair of viewer = ViewerModel(); qt_viewer = QtViewer(viewer) makes also viewer.window path unavailable, so do not allow testing this interface.

I have updated doc strings. I have added fixtures for qt_viewer and viewer_model

that we should include some of this stuff in our testing docs, and maybe even do a little "crash course in napari testing" workshop for core devs or something

I hope to have time to do this in not so long time

@Czaki I've updated the docstrings and the OwnQtBot class name to be more descriptive. Please check the docstrings especially the pytest_runtest_setup to make sure I haven't mangled something. Otherwise thanks again for this and this is ready to merge from me! 🙏 🚀


upgrade zuul-core to 2.1.6

I noticed that is using an old and unsupported version of zuul-core 1.3.1 (Nov, 2017) which could be a security concern for Zuul users.
I suggest we upgrade zuul-core to the latest version 2.1.6 (Nov, 2019)

We are not supporting zuul 2, it is not backwards compatibe. We created spring cloud gateway instead.


Can't make hours work

Hi!
First: good job, it's a very nice card, so useful and beautiful.
I had it when it didnt hide seconds and i updated it today with the new feature "show_hours: auto".
It does it, hides automatically to HH:MM (seems to...) but when I click on supposed hours, it's minutes. Timer tells that.
I checked it several times, in several ways, but i cant make it do it well :(
By the way, a little typo in your readme.md:  you have to put "show_hours: auto" instead of "show_hour: auto" to make this work.
Thanks a lot for this card, its so beautiful!

Can you explain the problen after setting show_header: true ?
Thanks.

This code:
          - type: custom:flipdown-timer-card             entity: timer.olla             show_hours: auto             show_title: true             show_header: true             theme: dark             duration: '00:00:00'             styles:               rotor:                 width: 50px                 height: 70px               button:                 width: 60px                 location: right
Shows minutes and seconds, not hours and mins :(

There was a confusion on show_hour setting because of my english skill.
finally, show_hour : auto is right on this card.
Maybe gramatically wrong? I don't know.

Oh! show_hour works!
So sorry, now it works beautifully. Thanks for your excellent work!


add amd support

I want to use jquery terminal in amd module.But I don`t know how to do that.help me please

I can add UMD loader but I don't know how to export more than one entity,  I have few plugins and included dependencies. I'll ask on Stack Overflow and let you know. 07.08.2017 9:08 AM fazelfarajzade<EMAIL_ADDRESS>napisał(a):I want to use jquery terminal in amd module.But I don`t know how to do that.help me please
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

I've added UMD it will be in next version 1.6.0 soon.

version 1.6.0 is out.

thanks a lot  jcubic ♥♥♥♥♥


Need Docs on Adding Env Variables in MGW

Description:

Add missing content to MGW docs based on [1].
Related to MGW 3.2.0

[1] https://github.com/wso2/product-microgateway/issues/2671

Update
Waiting on SMEs in order to get more information with regard to the questions that I had based on the grey areas that I came across when putting this document together.

The content merged via the mentioned PR can be found in the following pages.
[1] https://mg.docs.wso2.com/en/latest/reference/configurations/configuring-micro-gw-conf-for-docker/
[2] https://mg.docs.wso2.com/en/latest/reference/configurations/configuring-micro-gw-conf-for-kubernetes/


agent: enable unlimited allowance for FillManager

following #353, we can introduce the same change to the agent, saving on transaction per fill by approving an unlimited amount to the fill manager contract.

@fredo this one should be trivial, right?
Check if the token allowance for the agent's address is UINT256_MAX and, if not, approve the max amount.

@fredo ping... :-)

I'd say we should make sure that this is optional. If agents wants to enable it and do it, then fine, but it's a bit of bad practice, so maybe giving the agent / users the choice is the best practice.

please note, that every transfer deducts the amount from the contract's allowance. If we make this value configurable with any real number (which IMO is desirable), we need to check the current allowance before any call.
It then gets a bit tricky. We must decide how the agent reacts when the allowance decreases towards 0. When should the agent re-update the allowance and call approve()? It involves a strategy which is not quite trivial. But as always we should go ahead with an easy approach and optimize later.
One suggestion would be:
Upon every fill check whether current allowance is sufficient (RPC call). If not, call approve to update allowance. I think this is a suboptimal solution but good for now. Note, that the agent will most likely lose the race for that particular fill against someone where allowance is high enough.

Additionally, please note that agents which have an unlimited allowance have an advantage over other agents which do not, because such agents don't need the RPC call to check the allowance before the fill transaction.
This might give them a couple of hundred milliseconds advantage and are more likely to win the "fill race".
If it becomes financially unviable to run an agent with limits set, must yet to be seen.

Upon every fill check whether current allowance is sufficient (RPC call). If not, call approve to update allowance. I think this is a suboptimal solution but good for now. Note, that the agent will most likely lose the race for that particular fill against someone where allowance is high enough.

I do not perceive that as a problem, because it arises from the very same thing that would be the solution in this case: configurable allowance.
The extra call is not nice in general, however, it is good enough for now.

Not saying that this is a problem. But it needs to be handled by the agent correctly. And again, tehre are various ways to handle it.

Picking this up again, as we see now inefficiency in the agents' performances on mainnet.
Currently, agents see a request and most of them start approving. While only one agent will be able to fill, the allowance remains.  Additionally, the agent who fills has one more transaction in total per transfer.
By allowing to approve a bigger amount, the successful agent will have one transaction less. Additionally, all other agents will try to fill directly. The interesting part here is, that if the agent's RPC has the fill of another agent in its state upon filling, the transaction will not be send at all. It will already fail on estimating gas.
This does not happen for approves as they will never fail. This means that we have an upper bound of n-1 failing transactions (where n is the number of agents) but it could and probably will be less on average.
Implementation
In the past we discussed that the agent should be able to configure the allowed value (MAX_INT could be configured as -1).
Additionally, I realized that the agent can keep track of the allowance in memory without doing an RPC.

Upon start up check the current allowance for FillManager
Never do allowance until a specific fill would fail because insufficient allowance
Call approve for the configured value
For every successful fill, deduct the filled amount from internal allowance


[Feature]: Fetch data from opensource APIs (ex: Twitter)

Slack Post

[X] The team agrees on the need for this feature

Describe the Feature
This could be a module to fetch data from various opensource APIs that the team frequently uses. The first function could be to fetch data from Twitter.
Examples of this Feature
Twitter api tutorial

Closing for now based on discussions if fetching specific datasets belongs in this package


Feat/request literal verb

Request param - verb
Creating a Literal type to Request param verb to improve the developer experience and to avoid any mistake when using the Request object to make requests.

It also makes the autocomplete better

@pedroimpulcetto thanks for the contribution again, if you want to fix the conflicts we can get this one merged 👍

fixed @drish
thank you!


parameters javadocs description

PR-Codex overview
This PR adds documentation to Kafka attribute wrappers in the code.
Detailed summary

Added documentation for client.rack, batch.size, client.id, linger.ms, bootstrap.servers, auto.offset.reset, and acks Kafka attributes.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@h1alexbel thank you for your Pull Request. I'll assign someone to review it soon.
If this PR solves a todo from the code, please don't forget to remove it.

@l3r8yJ please review this Pull Request. Deadline (when it should be merged or closed) is 2023-06-01T09:42:20.000568.
You should check if the requirements have been implemented (partially or in full), if there are unit tests covering the changes and if the CI build passes. Feel free to reject the PR or ask for changes if it's too big or not clear enough.
Estimation here is 30 minutes, that's how much you will be paid. You will be paid even if this PR gets rejected.

@zoeself deregister

@zoeself deregister

@h1alexbel ok, I've removed this task from scope. I'm not managing it anymore.

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 3min)


Screenshots grabbed with driver tests have 'slow mode' banner

If you grab a screenshot with driver.screenshot() it has the slow mode banner. It's not clear how to remove this.

cc @Hixie I think I heard of a way to turn off the banner programmatically? Is there a service protocol extension for this?

Check the flutter_tools code, its screenshot logic shows how to temporarily turn off the banner.

@mit-mit Hello, I've been looking on how to grab the screenshots taken with the driver, I found you opened an issue to add an example for this here, but still no one has provided an example.
In this issue I see that you managed to do it. would you mind sharing how you got the screenshot saved?
Thanks!

Yup, see https://github.com/mit-mit/flutter_testing/blob/master/test_driver/tabs.dart !

The slow mode banner can also be turn-off, without modifying the app, using:
  WidgetsApp.debugAllowBannerOverride = false; // remove debug banner

in the flutter driver's test_driver/main.dart
See https://github.com/mmcc007/screenshots/blob/master/example/test_driver/main.dart for a working example.

There's nothing really actionable here since there's already ways to programmatically turn the debug banner off.


[Fix #6242] Fix an incorrect autocorrect for Style/EmptyCaseCondition

Fixes #6242.
This PR fixes Style/EmptyCaseCondition auto-correction removes comment between case and first when.
This change keeps the following source code comment.
def foo
  case
  # when bar ...
  when bar
    2
  end
end

The following is the result of auto-correct.
Before
def foo
  if bar
    2
  end
end

After
def foo
  # when bar ...
  if bar
    2
  end
end

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

Nicely done! 🙇


feat: authorization

Adds an authorization module.
Scopes organization staff teams and roles to an organization since that's what they are.
Adds a Guard to check that a member has one of the required actions. Acts as an OR.
Adds a decorator to require members to have permissions for an action.

This looks good to me, but this is important enough that I'm going to say it needs tests before we can merge it.


+![𝐅𝐑𝐄𝐄*]* IDM Crack 6.42 Build [2025] Patch +Activation key (latest)

+![𝐅𝐑𝐄𝐄*]* IDM Crack 6.42 Build [2025] Patch +Activation key (latest)
Internet Download Manager 6.42 Build 25 IDM Crack with Patch free download is a powerful and fastest application that can increase the download speed upto 5 times. Now you can download any files you need using the IDM Crack free Download application. You can download IDM for free from IDM’s official website. But you only get a trial version from the official website.


Bug in Bweight::from_position

Distance calculation is wrong for any z != 0:
let dist = (pos[0] * pos[0] + pos[1] * pos[1] + pos[1] * pos[2]).sqrt()
                                                    ^ should be 2

Same bug in doppler_rate (2x)


test against tcp transport

Hi,
I have just tested this, and it works with tcp transport. I did the the test locally (all services running on localhost), based on the multicast-discovery example (https://github.com/senecajs/seneca-mesh/tree/master/examples/30-multicast-discovery). I just changed the api, hex and rgb services to set the transport to tcp as below:

Hope this helps!

Probably the example can be extended to include the transport as an argument (default http), so tcp can be easier to test. I can do it in a PR later if you do not mind? Please let me know


Connection breaks after an hour or so

I am using SAS key for device authentication (ESP32), and after 60 minutes or so, the connection closes. I know that the token expires after 3600 seconds, but shouldn't it be automatically refreshed as the pxTokenRefresh function is initialized in the SetSymmetricKey function call. Do I explicitly need to call Connect method again after the expiration?

Hi @nexstem-siddhant thanks for the question. At this time, the refresh is not done automatically. Application code can take account of this scenario by reissuing a connect when it expires and disconnects.
We can look into doing this automatically, which would still be a disconnect followed by another connect. But no promises at this point.
It would be great if this could be done without disconnecting, and that's exactly what is possible with MQTT 5 :)
https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-mqtt-5#reauthentication


feat(@ngtools/webpack): redirect ngcc errors and warnings to webpack

Currently blocked, as I am using the snapshot


fix: Resolve version before checking arch

Fixes a bug where gimme module reports the following error
on macOS with ARM64.
error: arm64 is not supported by this go version
try go1.5 or newer

This occurs because the check for architecture
is performed before the module version
turns into a real version number with _resolve_version.
Fix by resolving version number before checking the architecture.

This fix works for me from 1.16 (when darwin/arm64 support was added). But the behaviour if the go directive specifies <1.16  changes from a partially incorrect error message (should say 1.16 on darwin):
% gimme module
error: arm64 is not supported by this go version
try go1.5 or newer

to an unhelpful error message:
% gimme module
I don't have any idea what to do with '1.13.15'.
  (using download type 'auto')

Can we restore the original error message but correct the version number for darwin? 1.5 was when linux/arm64 support was added.

Also this needs to change, perhaps:

To install the release specified in the go.mod file, or the most recent patch when no patch version is present:

Sounds good!
Updated the arm64 version matching.
Agreed about the README change too.
Should've done that in #25, but late is better than never.
Updated to reflect new behavior.

New error message looks good but it still doesn't actually show up:
% uname -sm
Darwin arm64
% cat go.mod
module github.com/urfave/gimme/test

go 1.13
% ../gimme module
I don't have any idea what to do with '1.13.15'.
  (using download type 'auto')

That's weird.
❯ uname -sm
Darwin arm64

❯ head -n3 go.mod
module github.com/urfave/gimme

go 1.13

❯ ./gimme module
error: arm64 is not supported by this go version
try go1.16 or newer


add indication of genome size to peptidome clustering

It might be handy to have an estimation of the genome size (measured as the number of peptides) together with the peptidome clustering. This could help to spot false "complete genomes" that were initially included in the analysis. Now this information is available from the Unique Peptide Finder view, but it requires swapping back and forth.
As an example, make a clustering of all Mycobacterium genomes:

the genome of Mycobacterium yongonense 05-1390 is far from complete (or the annotation of the genome is wrong), since it has only 2508 peptides, compared to > 50.000 for the other genomes; this entry should clearly be removed from the cluster analysis
the two Mycobacterium tuberculosis RGTB genomes have 15-20.000 peptides less than the other Mycobacterium tuberculosis genomes, which could hint on the fact that these are draft genomes, not complete genomes (or they should have a highly reduced genome). This could explain why these genomes do not cluster together with the other Mycobacterium tuberculosis genomes.

To make this kind of inferences immediate, I would recommend adding a plot of the genome sizes below the similarity matrix (instead of the duplicated labels). This would avoid the need to switch back and forth between the clustering and the unique peptide finder.
Original issue by @pdawyndt on Sun Jun 15 2014 at 11:30.

As a somewhat related issue, the fact that the two "draft" Mycobacterium tuberculosis still cluster closely with the other Mycobacterium tuberculosis genomes, gives an indication that having the possibility to add high-quality draft genomes to the analysis might be interesting. However, instead of using the size of the union in the denominator of the similarity computation, we might use instead the minimum or average size of both genomes.
We could make this a parameter that can be set by the user, and add a discussion on this in the article. For whole-genome clusterings of closely related organisms we could recommend using the size of the union in the denominator. For draft genomes, more remotely related organisms, or related organisms with varying genome sizes (e.g. Clostridium) we could recommend using the minimum or average size of both genomes.
If we define d_T(A,B) as interection(A, B) / T(A, B), with T in {min, avg, max, union}, then it always holds that intersection(A, B) <= min(A,B) <= avg(A,B) <= max(A, B) <= union(A, B), and thus 0 <= d_min(A, B) <= d_avg(A, B) <= d_max(A, B) <= d_union(A, B) <= 1.
Literature references where the minimum size of both genomes is used in the denominator of the distance function

Snel B, Bork P, Huynen MA (1999). Genome phylogeny based on gene content. Nature Genetics 21, 108–110.
Henz SR, Huson DH, Auch AF, Nieselt-Struwe K, Schuster SC (2005). Whole-genome prokaryotic phylogeny. Bioinformatics 21(10), 2329-2335.

Literature references where the average size of both genomes is used in the denominator of the distance function

Auch AF, Henz SR, Holland BR, Göker M (2006). Genome BLAST distance phylogenies inferred from whole plastid and whole mitochondrion genome sequences. BMC Bioinformatics. 7, 350.
Deloger M, El Karoui M, Petit MA (2009). A genomic distance based on MUM indicates discontinuity between most bacterial species and genera. Journal of Bacteriology 191, 91-99.

These last two references also have a discussion why using the average is better than using the minimum. Apparently the average allows greater sensitivity to variations due to gene loss and gene acquisition (which is necessary between close relatives). The union even gives greater sensitivity, since it does not "average out" gene loss by gene acquisition. So minimum-average-maximum-union would give different sensitivities to gene loss and gene acquisition, either naturally or due to the fact that some genomes are still high-quality drafts.
Original comment by @pdawyndt on Sun Jun 15 2014 at 11:41.

For now, the genome size is also shown in the tooltip of the similarity matrix
Original comment by @bmesuere on Sun Jun 15 2014 at 11:42.

@bmesuere : yes, this is how I spotted the differences initially, but having a clear overview over all sizes would seem like a helpful feature
Original comment by @pdawyndt on Sun Jun 15 2014 at 11:48.


Feature/#25

Not sure if the new test is good, but it's passing.

Cool.  Yeah, I guess I had changed my mind on where this was going to be implied...thanks for the patch.  I'll take a closer look in the A.M.


fix - remove check for embedding model when using azure open ai

Removes an options check that ensures an embedding model is configured when using Azure OpenAI llm type.
PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[x] Describes the source of new concepts.
[x] References existing implementations as appropriate.
[x] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

@vivekrao1985 Great work, merging this.


GEODE-1728: Http session filter should only apply to REQUEST

The docs should not tell users to add apply the session filter to
FORWARD, INCLUDE, and ERROR dispatchers. It should just be the default,
REQUEST.

@joeymcallister @davebarnes97 - can you review?

+1 LGTM, Dan. Thanks!


Repartitioning browser gene model Hail Table?

Hi team,
Thank you so much for releasing the browser gene model Hail Table! It's so helpful to have all the relevant transcript and coding sequence annotations pulled together and harmonized in one place.
I wanted to note that for a relatively small dataset (~60k rows), there are a lot of partitions (~2k), and that's causing even simple, small joins on the table to run relatively slowly. It's not prohibitively slow, but it is a noticeable inefficiency, so I was wondering if there was a deliberate design decision around the number of partition in the release, or if this is something you could adjust.
Thank you!
Grace

Hey @gtiao! Hope all is well. Thanks for the suggestion, and for filing this issue.
This is certainly something we can adjust, we'll look into a rule of thumb for a better amount of partitions for this table with respect to its number of rows/columns.
Since it sounds like this isn't blocking at the moment, we'll likely add this repartition step to the end of our pipeline, and not re-run the pipeline immediately. As far as I know, the next time we anticipate updating the gene model tables would be to include GTEx v10 when it releases, so at that time the released tables will have less partitions. Does that seem reasonable?

Hi Riley! Nope, not a blocking issue, just a nice-to-have suggestion -- and no particular rush. Thank you!


[Tooling] BLE Cancel Connection

Reproduction steps / Feature
Build chip-tool using a linux/arm64 based Docker container. After installing the necessary build tools and sysroot:

export SYSROOT_AARCH64=/opt/ubuntu-22.04.1-aarch64-sysroot
checkout source code
./scripts/checkout_submodules.py --shallow --platform linux
source ./scripts/bootstrap.sh
./scripts/gn_build_example.sh examples/chip-tool out

chip-tool builds then is copied to the R Pi and used normally. The question is why does the chip-tool that is packed with the test harness image not produce the following errors (these appear to be benign, but still inconsistent). Are these being suppressed somehow?
[1688578543.970738][6727:6729] CHIP:BLE: CancelBleIncompleteConnection() failed, err = src/platform/Linux/BLEManagerImpl.cpp:732: CHIP Error 0x0000002D: Not Implemented
.
.
.
 CHIP:BLE: CancelBleIncompleteConnection() failed, err = src/ble/BleLayer.cpp:372: CHIP Error 0x00000003: Incorrect state

Platform
raspi
Platform Version(s)
No response
Type
Manually tested with SDK
(Optional) If manually tested please explain why this is only manually tested
No response
Anything else?
No response

The question is why does the chip-tool that is packed with the test harness image not produce the following errors

Probably because it's built from a version that predates https://github.com/project-chip/connectedhomeip/pull/27304 ?

Duplicate of #27607


Feature template picker

Closing in favor of https://github.com/Esri/arcgis-runtime-toolkit-java/pull/28


Motion dataset: How to find if two objects are on the same lane?

Hello!
Currently I am using the Waymo motion dataset to extract car-following behaviors. In the dataset, there are some columns about the coordinates of the objects. But I wonder how to tell whether two vehicles are driving on the same lane.

Hi.  There is not a direct way to determine this.  One way you could approach this is to find the nearest lane points to the centers of the vehicles and compare the feature IDs of those closest points to see if they are from the same lane feature.

Thanks for your reply!
Following your advice, if two vehicles are both closed to the same lane center, is that safe to say that they are in the same lane? The premise is that vehicles are keeping centered in their lanes.

@18627242758 Did you get any closer to a solution to this task? I'm trying to solve the same problem and would be glad if you could share your progress.

Hello!
I just have one more question: no take-over/disengagement events are included in both perception and motion parts, correct?

Is there any update on this task?


scrcpy will prevent windows turning off screen after some time

win10 17134.
windows cant turn off screen/display after the time user setted.
DISPLAY:
[PROCESS] \Device\HarddiskVolume5\scrcpy-win64\scrcpy.exe

SYSTEM:
[DRIVER] NVIDIA High Definition Audio (HDAUDIO\FUNC_01&VEN_10DE&DEV_0083&SUBSYS_73771121&REV_1001\5&12d02362&0&0001)
An audio stream is currently in use.

hope fix

Thank you for your report.
For reference, I think it will be sufficient to enable SDL_HINT_VIDEO_ALLOW_SCREENSAVER here.


refactor: perception_field_point_analysis - allow reprocessing

What

[ ] New feature
[ ] Bug fix
[ ] Breaking change
[x] Refactoring
[ ] Change code style
[ ] Update test
[ ] Update document
[ ] Chore

Test performed
Doesn't apply
Notes for reviewer
This PR is a part of refactoring done in perception_eval/test to allow reprocessing loaded data e.g. in jupyter notebooks.

https://github.com/tier4/autoware_perception_evaluation/pull/195


Choose signal setting interface/behaviour for model classes

Currently the interface for setting signals of HamiltonianModel and LindbladModel uses the setter for the .signals property. This is a bit awkward in the case of the LindbladModel, which has two collections of signals for Hamiltonian and dissipator terms. A result is that, the signals property for Lindblad is now a Tuple of these two terms, and setting it requires setting both, though this is a bit awkward given that a huge use case for the Lindblad model will be when the signals are constantly 1.. It may be good in general to change the signal setter for GeneratorModel/HamiltonianModel to a method set_signals, and Lindblad model should maybe have two separate properties/setters for hamiltonian_signals and dissipator_signals.

Closing this as I think the current convention is actually fine and we can revisit if there are usability issues.


Assassin's Creed Origins: blurry textures

Running the game with DXVK enabled causes textures to appear blurry, which can be seen on the screenshot. Two pics attached for comparison (1-with, 2-without dxvk). It started after the GPU driver was updated. Before version 20.12.1 it was fine. All driver versions afterwards cause this issue. I rolled back to the old driver, 20.9.1, and there were no issues, with or without dxvk. Then I switched to the newer driver, version 22.3.1, and the issue came back. With DXVK, textures become blurry, regardless of the DXVK version. So apparently the problem emerges if dxvk is used with latest AMD drivers.
ACOrigins_d3d11.log
ACOrigins_dxgi.log

Software information
Assassin's Creed Origins, latest patch, medium settings (although all settings were tried).
System information

GPU: Radeon RX480
Driver: 22.3.1
OS version: Windows 10 21H2
DXVK version: 1.10 (issue persists regardless of the version)

Probably the same issue as https://github.com/doitsujin/dxvk/issues/2540.
TL;DR DXVK will no longer work correctly on Windows AMD drivers in situations where anisotropic filtering is used, and we can't really do anything about it.

Probably the same issue as #2540.
TL;DR DXVK will no longer work correctly on Windows AMD drivers in situations where anisotropic filtering is used, and we can't really do anything about it.

Works perfectly fine on the same hardware and drivers but with different titles tho, e.g. Far Cry 3, Call of Duty WW2, NFS HP 2010.

Different games do different things.

Probably the same issue as #2540.
TL;DR DXVK will no longer work correctly on Windows AMD drivers in situations where anisotropic filtering is used, and we can't really do anything about it.

Clarification:
Actually, it seems that the issue is not texture related. Take a look at the screenshots. It's something wrong with rendering.

looks like the issue is not texture related. Take a look at the screenshots. The ga

me looks like a retro pixel game.

Not sure what to tell you, the game works fine on other drivers.

It looks more like an internal resolution issue.
I don't have any such bug on Vega + driver 22.3.1 + latest DXVK build.

Vulkan features are not the issue here, Polaris supports everything we need. DXVK would just not work otherwise.

@Adibo88 Can you try again with the latest driver.

I no longer own that VGA, but I have tried using the R7 260X instead, latest legacy driver 22.6.1 and dxvk 1.10.3, on the same computer, OS and same graphics settings. Graphics seem to be ok without any visible glitches.

Im having the same rendering issue with RX 580 on the newest driver and with dxvk 1.10.3
Does anyone fix this issue?


Mypy: unexpected behavior with orm_mode and classmethods

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
❯ python -c "import pydantic.utils; print(pydantic.utils.version_info())"
             pydantic version: 1.8.2
            pydantic compiled: True
                 install path: /home/tammo/unbind/monorepo/.venv/lib/python3.9/site-packages/pydantic
               python version: 3.9.2 (default, Mar 15 2021, 12:27:27)  [GCC 9.3.0]
                     platform: Linux-5.4.72-microsoft-standard-WSL2-x86_64-with-glibc2.31
     optional deps. installed: ['typing-extensions']

❯ mypy --version
mypy 0.812

from typing import Optional

import pydantic as pyd

class DBModel:
    id: str
    name: Optional[str]

class Model(pyd.BaseModel):
    id: str
    name: Optional[str]
    something: Optional[str] = None

    class Config:
        orm_mode = True

    @classmethod
    def from_db(cls, model: DBModel) -> "Model":
        result = cls.from_orm(model)
        result.something = "assign something here"
        return result

When running mypy I get the following error:
❯ mypy issue.py
issue.py:21: error: The pydantic mypy plugin ran into unexpected behavior: ctx.type: Type[issue.Model] (of type TypeType)
Please consider reporting this bug at https://github.com/samuelcolvin/pydantic/issues/new/choose so we can try to fix it!
Found 1 error in 1 file (checked 1 source file)

Workarounds:

Removing the orm_mode = True and manually creating result fixes the mypy issue.
using result = Model.from_orm(model) fixes the issue

orm_mode is removed from V2

(For a bit of additional context — you'll now set from_attributes = True in the config, instead of orm_mode = True, and then you'll be able to just call cls.model_validate(model) and it will do the right thing. If you run into more mypy plugin issues after making this change, please let us know.)


Add new  tutorial

Adding code for the new  tutorial. Please give feedback!
Note: this is based on https://github.com/CrystalOnScript/vanilla-js-amp-script.

Let's push this out at the same time as the new tutorial. I'll link that PR here when it's ready.


Email Callback URL Not Set Correctly in Vercel Deployment

Environment

Operating System: Darwin
Node Version:     v21.2.0
Nuxt Version:     3.8.2
CLI Version:      3.10.0
Nitro Version:    2.8.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      modules, ui, auth, devtools
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
N/A
Describe the bug
I've added a simple EmailProvider to use a magic link to sign in. I have no special callback handling. I have a simple app with a protected page. When I try to access this page, I correctly am redirected to the sign-in page:
https://.vercel.app/api/auth/signin?callbackUrl=%2Fcategories
I use the email provider to send the default email with a link. Locally, it works and includes the callback URL properly. In my Vercel deployment, instead of the expected callback, the URL is:
https://.vercel.app/api/auth/callback/email?callbackUrl=https%3A%2F%2F.vercel.app%2Fsw.js&token=&email=
Then I get a 404 error on the sw.js page not found.
I'm not sure if it's related, but the documentation on AUTH_ORIGIN and baseUrl is a bit confusing. If I don't set an environment variable for AUTH_ORIGIN, the app crashes horribly. The docs state that baseUrl must include the full path, but if I include the "/api/auth" path portion along with my app URL in AUTH_ORIGIN, that causes the "/api/auth" portion to be duplicated. I've tried even hardcoding the "baseUrl" in my nuxt.config.ts file to include the /api/auth path, but it seems like it's overwritten by AUTH_ORIGIN?
nuxt.config.ts:
auth: {
isEnabled: true,
baseUrl: 'https://.vercel.app/api/auth',
provider: {
type: 'authjs'
},
globalAppMiddleware: {
isEnabled: true
},
},
export default NuxtAuthHandler({
secret: process.env.AUTH_SECRET,
adapter: PrismaAdapter(prisma),
providers: [
// @ts-expect-error You need to use .default here for it to work during SSR. May be fixed via Vite at some point
GithubProvider.default({
clientId: process.env.GITHUB_AUTH_PROVIDER_CLIENT_ID,
clientSecret: process.env.GITHUB_AUTH_PROVIDER_CLIENT_SECRET
}),
EmailProvider.default({
server: process.env.EMAIL_SERVER,
from: process.env.EMAIL_FROM
})
],
session: {
strategy: 'jwt'
}
})
In vercel environment var:
AUTH_ORIGIN=https://.vercel.app/
Additional context
No response
Logs
No response

Got this to work finally. Two things:

I added typescript checking, which flagged that "baseUrl" should have been "baseURL". I set this to baseURL: AUTH_ORIGIN + "/api/auth"
Took a while, but kind of figured out the sw.js part. I think this was a holdover from some experimental deployments with SSR set to false, then my browser had registered a service worker for that site and was making a GET request for it. Using Developer Tools (under Application) to stop and de-register the service worker made the 404 error go away, which allowed the callback URL generation to work correctly.


Update documentation about default Driver

Prerequisites

[X] This was not already reported in the past (duplicate check)
[X] It does reproduce it with code from main branch (latest unreleased version)
[X] I include a minimal example for reproducing the bug
[X] The bug is not trivial, as for those a direct pull-request is preferred
[X] Running pip check does not report any conflicts
[X] I was able to reproduce the issue on a different machine
[X] The issue is not specific to any driver other than 'default' one

Environment

molecule --version

molecule 6.0.2 using python 3.11
    ansible:2.15.6
    default:6.0.2 from molecule

system info

DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=23.10
DISTRIB_CODENAME=mantic
DISTRIB_DESCRIPTION="Ubuntu 23.10"

What happened
I was working with molecule few years back and now I needed to work with it one more time. I was surprised that a lot old working drivers (for example vagrant) has been removed. And there is default driver which there is little information about. Even displaying following command not giving any hints:
molecule drivers

# Result
default

There is no extra information what default means and there is little information on documentation. Once I found 'somewhere' that this is default ansible driver. However not too much information what this means.
I was really surprised to see that running command molecule test is really running ansible on my machine (not sandbox, not VM, not docker) - directly running on my machine. I was lucky that I was working on very simple role for testing - and nothing broke, however I don't think that is right behavior.
For example working on role which installing new version of software - will cause that this software will be installed on local machine. This is not always recommended way - for example contribution for open source role is dangerous now, because default driver is running on local developer machine.
I was not able find any documentation / information to explain this before run.
What I would expect

Clear information about driver when running command molecule drivers - Maybe some command molecule drivers info default? Where there will be CLEAR information what will happen.
Easy to find documentation about default driver - also with CLEAR information how this is working

Reproducing example
- Create simple role with default driver for molecule: `molecule init scenario new-role`
- Run `molecule test` --- with Default driver - this is running on local machine ????!

So if role is having any command such as `install package / apt / yum / etc` - this will run on local machine not any sandbox / container / VM.

@marcinpraczko There is a pending change that will make the driver listing more informative.
Your remark regarding security is bit misleading because running any software locally is unsafe  and it was already possible for anyone to put some playbooks that do nasty stuff regardless which driver was the default. Basically this behavior is not less secure than before.
We would welcome any PRs that also improve the docs.


[Qualcomm feedback] 7.2.1. Secure SPDM Session

Reference: https://lists.riscv.org/g/tech-ap-tee-io/topic/103498833#47

With trusted I/O, the TSM establishes SPDM sessions with both the ROT and the physical devices
DSMs.

Why make SPDM between TSM and ROT mandatory? In PCIe standard this is an implementation issue. Host may as well assign (protected via MTT) an inherently trusted device to TSM which is used for IDE key transport to root port.

between the TSM
and either PCIe root ports or devices is a mandatory requirement

For ROT, SPDM should not be mandatory.

Yes. I think it is an implementation choice, if we want to have a SOC specific TSM.
If we want to use a generic TSM, then SPDM is a good choice.


daemon: remove unneeded Daemon field from RuleReactionEvent

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-me-please

Coverage decreased (-0.01%) to 45.554% when pulling 1d1fce3c63c156865b9653c484734b3318ca7305 on pr/ianvernon/remove-daemon-rule-reaction into 28d75e4c40c0652cb6a91fdd18b503e601097035 on master.


Graph examples: delete allocator usage in example

deleting allocator usage in examples, beautifying example output

/intelci: run


Swift2.2

fixes some compiler issues with the new open source swift 2.2 beta
Fixed a crash related to promise lock synchronization.
fixed the link to the playgrounds.

looks good.


Weekly meetings

I would like to be part of the weekly meetings. Can someone add the required information in the Readme.md?
Also, if possible enable Discussions for this repository.

Hey!
We're working to schedule the first meeting right now. I just opened #6 to track that.
GitHub Discussions are enabled - I've never used them before so perhaps some permissions are configured. I can see them right now:

Meetings will monthly, not weekly. I think we should try to find a recurring time every month, instead of sending out a doodle every time.

Did the meeting schedule get finalized? I'm interested in joining in.

I suggest closing this issue. Meeting schedule has long been finalized.


How to save tree?

How to get all data of tree after change add,edit,delete,checked node?

Hi!
You can use toJSON method:
this.$refs.tree.toJSON()

To get more examples clone the repo and try:
$ npm run dev

Then open Exporting page.

Hi! Thank support.
But, I have test and get lost data ?
Need click expanded:true all level of tree, it will have all data for toJson()
else not click expanded true data just level can see.
How to get all data of tree?

Could you show me an example? You can use https://codesandbox.io/ for it

Sorry, current i need more time make demo in codesandbox,
I just make demo it http://jsfiddle.net/e0h2yvxp/4/

Check the console. By clicking the button it is showing the correct data


Android : How to use https://github.com/ypresto/android-transcoder with this utility?

Hi Yuya Tanaka,
I am working on an Android project, where requirement is to transcode a video. So I am using https://github.com/ypresto/android-transcoder, but this utility does not place moov atom at beginning of file.
So I went through this utility and wanted to use it.
I did not find any documentation regarding how can I use it with https://github.com/ypresto/android-transcoder. Can you please add some documentation? Steps would also be fine, later I will add documentation for this utility and will share pull request.
Thank you.

I saw your comment in a closed issue https://github.com/ypresto/qtfaststart-java/issues/2
So what I understood is

Step1 : Transcode the video
Step2: Use utility to add "moov" in transcoded video

And my implementation looks like
MediaTranscoder.Listener listener = new MediaTranscoder.Listener() {
                        @Override
                        public void onTranscodeProgress(double progress) {
                        }

                        @Override
                        public void onTranscodeCompleted() {
                            Log.d(TAG, "transcoding took " + (SystemClock.uptimeMillis() - startTime) + "ms");
                            onTranscodeFinished(true, "transcoded file placed on " + file, parcelFileDescriptor);

                            Log.d(TAG, "Adding MOOV at beging of this file...");

                            File videoToPlay = null;

                            try {
                                if (QtFastStart.fastStart(file, moovFile)) {
                                    Log.d(TAG, "Added MOOV at beging of this file...");
                                    // Use output file path to play video
                                    videoToPlay = moovFile;
                                } else {
                                    Log.d(TAG, "MOOV present in this FILE");
                                    // Use input file path, as it has moov already so output file will not exists.
                                    videoToPlay = file;
                                }

                            } catch (QtFastStart.MalformedFileException m) {
                                Log.e(TAG, m.toString());
                            } catch (QtFastStart.UnsupportedFileException q) {
                                Log.e(TAG, q.toString());
                            } catch (QtFastStart.QtFastStartException e) { // catches QtFastStartException other than MalformedFileException and UnsupportedFileException
                                Log.e(TAG, e.toString());
                            } catch (IOException e) {
                                Log.e(TAG, e.toString());
                            }

                            Log.d(TAG, "Playing video " + videoToPlay.getAbsolutePath());

                            Uri uri = FileProvider.getUriForFile(MainActivity.this, FILE_PROVIDER_AUTHORITY, videoToPlay);
                            startActivity(new Intent(Intent.ACTION_VIEW)
                                    .setDataAndType(uri, "video/mp4")
                                    .setFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION));
                        }

                        @Override
                        public void onTranscodeCanceled() {
                            onTranscodeFinished(false, "Transcoder canceled.", parcelFileDescriptor);
                        }

                        @Override
                        public void onTranscodeFailed(Exception exception) {
                            onTranscodeFinished(false, "Transcoder error occurred.", parcelFileDescriptor);
                        }
                    };
                    Log.d(TAG, "transcoding into " + file);
                    mFuture = MediaTranscoder.getInstance().transcodeVideo(fileDescriptor, file.getAbsolutePath(),
                            MediaFormatStrategyPresets.createAndroid720pStrategy(8000 * 1000, 128 * 1000, 1), listener);

Can you please confirm If I am doing it in correct way?

@PankajChunchun does one have to wait for the transcoding to complete? In my use case, I want to start playing the stream immediately. This solution will not work for me.


Join

How can I join to HP Organization so that I can contribute with OS ideas for this project?

Hello @FMCalisto, thanks for your interest in contributing to Grommet.  You do not need to join the HP organization to contribute.  You can follow the content in this video to contribute to Grommet:
https://vimeo.com/129681048


Thorough review of OSO docs to ensure they reference latest data model and API

I did a first pass last week to reflect the updated OSSD schema, but it's worth a thorough page-by-page review to make sure everything is up-to-date.
Most important is to check that anything that references the API is visible in the Hasura explorer and has the V1 postfix and all SQL queries still work in BigQuery.

@ryscheng per our discussion today, will remove pages like this one and instead add links to dbt-generated docs on models.opensource.observer. See also #959


[PSM Interop] Set canonical server via canonical-* tag, not commit sha

While this is less transparent, it makes it significantly easier to deliver images with latest security patches.

[ ] grpc/core/v1.56.x/branch/linux/grpc_xds_k8s_lb
[ ] grpc/core/v1.56.x/branch/linux/psm-security

Note: URL map is already deployed with this changes, see https://github.com/grpc/grpc/pull/33587#issuecomment-1616859672.


Add doctests

Expected Behavior

All public methods should have informative docstrings with sample usage presented as doctests when appropriate.

Actual Behavior

Steps to Reproduce the Problem

Specifications

Closing stale issue.


failed to load assets in subdomain app

Describe the bug
hi
I have project that uploaded on subdomain url in server
the problem is when I use multiple theme for admin and user panel  css file doesn't load in admin panel
I don't know why but all the filament assets load correctly except css file
Your minimal, reproducible example
--
Steps to reproduce
I dont know
Expected behavior
I expected to load assets correctly
How often does this bug happen?
Every time
Screenshots or Videos
https://cdn.discordapp.com/attachments/883085267383226478/1009380663276884028/unknown.png
Platform

os: nginx
browser: chrome

tailwindcss-themer version
2.0.1
Additional context
No response

Hiya! Thanks for opening this issue.
Without a reproduction repo, I'm unable to help you with the problem.


The base image in the Dockerfile seems to be wrong

Environment Details

Helidon Version: 3.0.0
Helidon MP
JDK version: 17.0.4
OS: Oracle Linux
Docker version (if applicable): 19.03.11-ol

Problem Description
The base image in the Dockerfile included in the quickstart generated from the Helidon CLI seems to be wrong.

jdk version should be 17+

# 1st stage, build the app
FROM maven:3.6-jdk-11 as build

WORKDIR /helidon

# Create a first layer to cache the "Maven World" in the local repository.
# Incremental docker builds will always resume after that, unless you update
# the pom
ADD pom.xml .
RUN mvn package -Dmaven.test.skip -Declipselink.weave.skip

# Do the Maven build!
# Incremental docker builds will resume here when you change sources
ADD src src
RUN mvn package -DskipTests

RUN echo "done!"

# 2nd stage, build the runtime image
FROM openjdk:11-jre-slim
WORKDIR /helidon

# Copy the binary built in the 1st stage
COPY --from=build /helidon/target/quickstart-mp.jar ./
COPY --from=build /helidon/target/libs ./libs

CMD ["java", "-jar", "quickstart-mp.jar"]

EXPOSE 8080

Steps to reproduce
Firstly, I created skelton via helidon cli.
$ helidon init --version 3.0.0

| Helidon Flavor

Select a Flavor
  (1) se | Helidon SE
  (2) mp | Helidon MP
Enter selection (default: 1): 2

| Application Type

Select an Application Type
  (1) quickstart | Quickstart
  (2) database   | Database
  (3) custom     | Custom
Enter selection (default: 1): 1

| Media Support

Select a JSON library
  (1) jackson | Jackson
  (2) jsonb   | JSON-B
Enter selection (default: 1): 1

| Customize Project

Project groupId (default: me.opc-helidon):
Project artifactId (default: quickstart-mp):
Project version (default: 1.0-SNAPSHOT):
Java package name (default: me.opc.mp.quickstart):

Switch directory to /home/opc/git/quickstart-mp to use CLI

Start development loop? (default: n): n

Then, build the container image.
$ cd quickstart-mp/; docker build -t quickstart .
[+] Building 9.8s (13/16)
 => [internal] load build definition from Dockerfile                                                                               0.0s
 => => transferring dockerfile: 846B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 107B                                                                                0.0s
 => [internal] load metadata for docker.io/library/openjdk:11-jre-slim                                                             0.0s
 => [internal] load metadata for docker.io/library/maven:3.6-jdk-11                                                                0.0s
 => [build 1/7] FROM docker.io/library/maven:3.6-jdk-11                                                                            0.0s
 => [stage-1 1/4] FROM docker.io/library/openjdk:11-jre-slim                                                                       0.0s
 => CACHED [stage-1 2/4] WORKDIR /helidon                                                                                0.0s
 => [internal] load build context                                                                                0.0s
 => => transferring context: 19.00kB                                                                                0.0s
 => CACHED [build 2/7] WORKDIR /helidon                                                                                0.0s
 => CACHED [build 3/7] ADD pom.xml .                                                                                0.0s
 => CACHED [build 4/7] RUN mvn package -Dmaven.test.skip -Declipselink.weave.skip                                                  0.0s
 => CACHED [build 5/7] ADD src src                                                                                0.0s
 => ERROR [build 6/7] RUN mvn package -DskipTests                                                                                9.8s
------
 > [build 6/7] RUN mvn package -DskipTests:
#13 4.570 [INFO] Scanning for projects...
#13 5.720 [INFO] ------------------------------------------------------------------------
#13 5.722 [INFO] Detecting the operating system and CPU architecture
#13 5.724 [INFO] ------------------------------------------------------------------------
#13 5.733 [INFO] os.detected.name: linux
#13 5.735 [INFO] os.detected.arch: x86_64
#13 5.735 [INFO] os.detected.version: 5.4
#13 5.736 [INFO] os.detected.version.major: 5
#13 5.743 [INFO] os.detected.version.minor: 4
#13 5.748 [INFO] os.detected.release: debian
#13 5.748 [INFO] os.detected.release.version: 10
#13 5.748 [INFO] os.detected.release.like.debian: true
#13 5.750 [INFO] os.detected.classifier: linux-x86_64
#13 5.784 [INFO]
#13 5.784 [INFO] --------------------< me.opc-helidon:quickstart-mp >--------------------
#13 5.785 [INFO] Building quickstart-mp 1.0-SNAPSHOT
#13 5.786 [INFO] --------------------------------[ jar ]---------------------------------
#13 8.029 [INFO]
#13 8.029 [INFO] --- maven-resources-plugin:2.7:resources (default-resources) @ quickstart-mp ---
#13 8.482 [INFO] Using 'UTF-8' encoding to copy filtered resources.
#13 8.502 [INFO] Copying 5 resources
#13 8.531 [INFO]
#13 8.534 [INFO] --- maven-compiler-plugin:3.8.1:compile (default-compile) @ quickstart-mp ---
#13 9.094 [INFO] Changes detected - recompiling the module!
#13 9.115 [INFO] Compiling 5 source files to /helidon/target/classes
#13 9.635 [INFO] ------------------------------------------------------------------------
#13 9.637 [INFO] BUILD FAILURE
#13 9.643 [INFO] ------------------------------------------------------------------------
#13 9.651 [INFO] Total time:  5.101 s
#13 9.655 [INFO] Finished at: 2022-07-27T01:41:46Z
#13 9.656 [INFO] ------------------------------------------------------------------------
#13 9.660 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project quickstart-mp: Fatal error compiling: error: release version 17 not supported -> [Help 1]
#13 9.668 [ERROR]
#13 9.668 [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
#13 9.669 [ERROR] Re-run Maven using the -X switch to enable full debug logging.
#13 9.669 [ERROR]
#13 9.669 [ERROR] For more information about the errors and possible solutions, please read the following articles:
#13 9.670 [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
------
failed to solve with frontend dockerfile.v0: failed to build LLB: executor failed running [/bin/sh -c mvn package -DskipTests]: runc did not terminate sucessfully

Indeed ; it should be maven:3.8.4-openjdk-17-slim instead of maven:3.6-jdk-11

Files that are impacted:
mp-tutorial.adoc
Dockerfile.mustache

./archetypes/helidon/src/main/archetype/common/files/Dockerfile.jlink.mustache:FROM maven:3.6.3-jdk-11-slim as build
./archetypes/helidon/src/main/archetype/common/files/Dockerfile.mustache:FROM maven:3.6-jdk-11 as build
./docs/mp/guides/mp-tutorial.adoc:FROM maven:3.6-jdk-11 as build <1>


pageSize & totalCount returning wrong values

What steps will reproduce the problem?
Similar to https://github.com/yiisoft/yii2/issues/17093.
        $dataProvider = new ActiveDataProvider([
            'query' => $query,
            'pagination' => [
                'pageSize' => 1,
                'totalCount' => 1,
            ],
        ]);

        $dataProvider = new ActiveDataProvider([
            'query' => $query,
            'pagination' => [
                'pageSize' => 1,
            ],
            'totalCount' => 1,
        ]);

The pageSize doesn't work, neither totalCount.
What is the expected result?
Page Size having the size set in the Model Search $dataProvider attribute pageSize, and the totalCount models as the Model Search $dataProvider attribute totalCount
What do you get instead?
The Widget (ListView and/or GridView) loads all the models found and the page size is according with the total of models found.
No matter if I set totalCount to 1 or more, if the query gets more than the totalCount, the totalCount will be it instead of configured value, and if in the pageSize I set 1, the number of pages will be by default 20.
Additional info

Q
A

Yii version
2.0.45

PHP version
7.4

Operating system
openSUSE TW/CloudLinux

If in the controller, I set $dataProvider->pagination->pageSize = 1; the pageSize will respet this value.
If in the controller, I set $dataProvider->totalCount = 1; the totalCount will respet this value.
If in the same controller I set $dataProvider->pagination->totalCount = 1; it neither will work

For paginator to work u need to give ur dataprovider an id.
That could solve the issue with pagesize, not sure.


[Custom Speech Containers] Python SDK speech_recognizer.recognize_once returns CancellationReason.Error

We're trying to recognize text in audio, against a brand new Custom Speech Container running
containerpreview.azurecr.io/microsoft/cognitive-services-custom-speech-to-text:latest. This works in CRIS on Azure, not locally via containers.
Platform: Windows 10 1909 (18363.720)
Audio sample is 21 seconds of PCM S16 LE (s16l) @ 16000 Hz.
Error details:
Timeout: no recognition result received. SessionId: 086f74c1bd2f4e1cb13146578c4a0f2b

I'd be happy to share container logs but the documented logging parameters do not seem to work. (I tried Logging:Disk:Format=json.)

Looks like this is related to recognize_once only supporting 15 seconds of audio. It's not clear why this limitation exists. May also be worth putting in the FAQ https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/speech-container-faq?tabs=stt


_post/0000-01-02-Cookie419.md

So far I created a branch,  opened a file and getting ready to do a pull request.

Somewhere I got lost I would like to start over again.


Error when creating role in China

https://github.com/terraform-aws-modules/terraform-aws-rds-aurora/blame/ec99016a364fe7abee822f78a11b8ccfccaa70d9/main.tf#L240
Error:
│ Error: creating IAM Role (terraform-20240314084919599800000001): MalformedPolicyDocument: Invalid principal in policy: "SERVICE":"monitoring.rds.amazonaws.com.cn" │       status code: 400, request id: 607f6a7e-6557-4b0d-90d3-822172f306c4 │  │   with module.rds.module.rds_cluster.aws_iam_role.rds_enhanced_monitoring[0], │   on .terraform/modules/rds.rds_cluster/main.tf line 245, in resource "aws_iam_role" "rds_enhanced_monitoring": │  245: resource "aws_iam_role" "rds_enhanced_monitoring" {
On opening a role that works in China it's
{     "Version": "2012-10-17",     "Statement": [         {             "Sid": "",             "Effect": "Allow",             "Principal": {                 "Service": "monitoring.rds.amazonaws.com"             },             "Action": "sts:AssumeRole"         }     ] }
So it seems to me in china the suffix should be .com for ht service, unless I'm missing something. Can anyone else confirm this is the case?

This issue has been resolved in version 9.3.1 :tada:


Add tests for !MERGE and !SPLIT

These need to be tested before the next version of the gem is released.

This was done way back in 2b99bf2db514683f7b2ed65c89bff45904036b00