Replace extension-module feature with pyo3::main and pyo3::test

This is an experiment to remove extension-module feature, with its known linking problems. See #904 #771 #341
Instead, to enable linking to libpython when needed, I introduce two proc macro attributes
#[pyo3::main] and #[pyo3::test] which force linking using the macro suggested in #771
To sweeten the deal, both of these attributes can automatically acquire the GIL if you make the first argument py: Python. e.g.:
#[pyo3::main]
fn main(py: Python) {
  // use py! :)
}

This implementation is not yet complete - I hard-coded libpython3.8 to test on my laptop. Needs to use proper detection of the python library. It also breaks windows where this hack isn't needed.
I'm happy with #[pyo3::main], though I wonder if adding this link arg to every #[pyo3::test] is overkill - an alternative would be to have a pyo3_test_link_python!() macro or so which could be included once in each test file.

@m-ou-se I'm not quite sure if this solves what you need for inline-python. I think it might - users of inline-python will just have to add #[pyo3::main] where appropriate?

it might be a good idea to be able to apply #[pyo3::main] or similar to a function other than fn main(), since there are systems that can't use the default main function, e.g. for libSDL on Android where the main is essentially provided by libSDL and the Android Java libraries.

Awesome, thank you!
I'll add detailed comments later, but from a glance,

Do we need

            #[link(name = "python3.8")]
            extern "C" {}

for each test function?

We should also allow #[pyo3::main]fn main() {}

another idea is to just merge them both into #[pyo3::entry] or #[pyo3::calls_python] or some other suitable name

Do we need [...] for each test function?

No, we only need this #[link] attribute once per executable really. So an alternative I was considering was that somewhere in a test file, users must add:
#[cfg(test)]
pyo3::link_python();

I think it does make sense for #[pyo3::main] to include this #[link], as that should only ever appear once.

another idea is to just merge them both into #[pyo3::entry] or #[pyo3::calls_python] or some other suitable name

I quite like #[pyo3::main] because it's an obvious name that hits 99% of use cases. I think for the niche use cases you describe above I'd rather recommend they just call pyo3::link_python!() explicitly.

Also, I quite like having #[pyo3::test] separate to #[pyo3::main] because the test macro can then generate the #[test] attribute, as well as acquire the GIL if needed.
But maybe if we prefer just calling pyo3::link_python!() once per test file, and remove #[link] from #[pyo3::test], I should split it into a separate PR.
Let me know what design you all like best 😊

honestly, I'm fine with having #[pyo3::main/test] as long as the link macro is also exposed for the corner cases.

I don't think an attribute like that is the way to go.
Some of my projects use Python somewhere in some module, but not directly from main. Attaching the attribute to main seems like the wrong place. It automatically locking the GIL is also not always what I'd want.
It also requires the use of a procedural macro, making the recently introduced macros feature a bit useless, as you'd always need a separate proc_macro crate with all the dependencies (quote, proc_macro2, syn, ..) again. I'd much prefer a simple macro_rules!{} macro like pyo3::link_python!().
But even then, it seems not like a very ergonomic solution. Everybody who forgets this macro (including all existing code) will just get a long linker error with no proper explanation. :(
Maybe this is something to be solved on the side of Cargo/Rustc instead. This is not the only problem when compiling plugins/extensions (whether for Python or other things). There's the problem of -undefined dynamic_lookup on MacOS, there's the problem of the name of the compiled library being libabc.so instead of abc.so, and the problem of only allowing one plugin/extension per package/Cargo.toml, because they are a library and not binaries to Cargo.
Maybe it's time to propose an addition to Cargo, allowing something like
[[bin]]
crate-type = ["plugin"]

and then letting build scripts specify flags for these separately. Or something in that direction.

I agree that the error case for forgetting to link is not very ergonomic.
A cargo plugin sounds great, though I guess it could be a long time before the design and implementation of it is complete.
Maybe a better workaround for now is to keep extension-module for now and add the pyo3::link_python!() macro? That shouldn't break existing code and will let those that run into problem cases have a way to compile.

Maybe a better workaround for now is to keep extension-module and add the pyo3::link_python!() macro? That shouldn't break existing code and will let those that run into problem cases have a way to compile.

@davidhewitt That might be the least surprising option for now. None of the solutions are perfect, but at least this one means that the regular use cases don't change, and the macro only needs to be used in the edge cases.

Yikes, accidentally re-used the same branch name for a new PR... Closing this PR. I plan to revisit the linking macro, but otherwise I think this is dead code.


front end update

adding changes to each page, added landing page, fixed up navbar, added in MintingPress name to index.html (I recommend putting a logo here in place of the default favicon), added instruction to read me

it seems I accidentally uploaded the abi folder as well. shouldn't be an issue since it will be re-created when running npx hardhat compile, if it's an issue I can reupload no problem


Sort a dict by element's value

To demonstrate the use of sorted() function and its 'key' argument.

Thanks!

Cool! I like the idea of this repo! :D

You're welcome :)


Bump kernel version

$subject

PR builder [7.x.x] started
Link: https://github.com/wso2/product-is/actions/runs/5607152789

PR builder [7.x.x] completed
Link: https://github.com/wso2/product-is/actions/runs/5607152789
Status: failure

PR builder [7.x.x] started
Link: https://github.com/wso2/product-is/actions/runs/5607152789

PR builder [7.x.x] completed
Link: https://github.com/wso2/product-is/actions/runs/5607152789
Status: failure

PR builder [7.x.x] started
Link: https://github.com/wso2/product-is/actions/runs/5607421439

PR builder [7.x.x] completed
Link: https://github.com/wso2/product-is/actions/runs/5607421439
Status: failure

PR builder [7.x.x] started
Link: https://github.com/wso2/product-is/actions/runs/5607152789

PR builder [7.x.x] completed
Link: https://github.com/wso2/product-is/actions/runs/5607152789
Status: success


Pull in latest changes from main

I hope it's okay to make this PR! Would like the doc comment fix in my editor.

@kylegoetz anyway to get this merged still?

@zetashift  sorry I forgot about this! I'll look at it tonight or tomorrow and presumably there won't be an issue and I'll merge it in.

I would appreciate it if the license file commit would be added to this PR.


Chunkwm crashes, can only be fixed with reboot

I am on macOS 10.13, and sometimes chunkwm daemon (as brew/launchctl service) crashes.
After starting chunkwm from the terminal, I got the following message and the program exits:
chunkwm: recv signal 11, printing stack trace
0   chunkwm                             0x0000000104173fc8 _Z13SignalHandleri + 36
1   libsystem_platform.dylib            0x00007fff5a7f8f5a _sigtramp + 26
2   ???                                 0x0000000000000400 0x0 + 1024
3   tiling.so                           0x0000000106c4ebea _ZL15ApplyWindowRuleP12macos_windowP11window_rule + 487
4   tiling.so                           0x0000000106c49937 _Z13AddWindowRuleP11window_rule + 143
5   tiling.so                           0x0000000106c48e15 _Z15CommandCallbackiPKcS0_ + 401
6   tiling.so                           0x0000000106c50be5 _Z10PluginMainPKcPv + 2018
7   chunkwm                             0x000000010417390c _Z14DaemonCallbackPKci + 279
8   chunkwm                             0x000000010416effc _ZL16HandleConnectionPv + 99
9   libsystem_pthread.dylib             0x00007fff5a8026c1 _pthread_body + 340
10  libsystem_pthread.dylib             0x00007fff5a80256d _pthread_body + 0
11  libsystem_pthread.dylib             0x00007fff5a801c5d thread_start + 13

The only fix so far is to restart.

Not exactly sure how we'd get a nullpointer dereference there..
Seems to be caused by trying to apply a newly added window rules to the existing windows.
I think this is a bad design decision and rules should only be applied to windows that are created AFTER chunkwm has been started, or AFTER said rules have been added.

Re-added support for newly window rules to apply to existing windows, on master. Fixed it the proper way that it should have worked from the beginning.


[FR]: Is it possible to creat a switch for aliases ?

Describe the new feature or enhancement idea
is it possible to build a switch for the aliases too? Because I have rules with aliases and I could then simply switch off the alias instead of all the rules
Will you be building this?

[ ] I intend to submit a PR to implement this

Additional Details
No response

Why "all" the rules? You can switch off a specific rule. Please explain the use case better.
@Snuffy2: In principle, it's doable, in firewall API there's this endpoint that should toggle the state of the alias:

No, not all rules 😊
I have set up selective wireguard routing for some clients.
https://docs.opnsense.org/manual/how-tos/wireguard-selective-routing.html
But I don't want to use this all the time, so as not to always turn off "all" rules that contain the alias, it would be easier to just deactivate the alias and press apply.
If I deactivate the rules, I think I also have to restart wireguard and unbound.
Is it possible to deactivate the alias and press apply (remotely)?

Is it possible to deactivate the alias and press apply (remotely)?

Technically, there's the API to do it (if I read the docs correctly), then you have to convince the dev, @Snuffy2. :)


官网无法访问

Description
官网无法访问

Some network nodes may be inaccessible. This is a problem between vpn and netlify. Try to replace other nodes for access.


Rescue exceptions from message if raise_on_failure? is false

Move ignored exceptions into Message module
Allow publishers to add to the ignored exceptions, i.e. Publisher::RabbitMq adding bunny specific exceptions

@dmvt you may want to peek at this too. The idea is to allow producing applications to be able to have non-critical messages that could have a publish failure and not disturb the application

I like the concept a lot. The constant approach to the configuration makes me a little nervous though...

So I moved the exceptions into the configuration. Now the Publisher can add their exceptions to the configuration and the message can pull down the exceptions from the configuration.

Thinking more, about maybe de-complicating a bit. what are we gaining with the list of exceptions to handle. Why not just handle StandardError in the publish? Since we're logging it, and the goal is to never fail if we can help it, maybe the list of exceptions isn't needed?

In response to your last comment @ryanwi, I like the ability to override individual error types. Both for the adapter specific reasons @dmccraw mentioned at the top, and for the ability to potentially override MessageNotSent errors, which indicate a service failure I would probably (hopefully) already know about due to monitoring while still raising on InvalidMessageError, which indicates a code or data problem.

Do we even need raise_on_failure? then? Can't we just always raise errors and let the adapters ignore them if they want?

I do like having the raise_on_failure override as well. It's an option on the message class, whereas the ignored_exceptions is system-wide. Having raise_on_failure allows us to say we care about all error messages except the ones in ignored_exceptions normally, but on this one type of message, we don't care at all.

On that note, it may be a good idea to allow the message class to add to the global setting:
rescue *Emque::Producing.configuration.ignored_exceptions => error

Refactored exceptions. Moved default exceptions into read_ignored_exceptions because ignored_exceptions on the class may not get called.

Moved MessageNotSentError as default in Emque::Producing::Configuration#initialize

:shipit: with a version bump

:+1:


In C# code, C# code actions should appear first

From an integration test failure:

or

Feels much more likely users will want C# code actions in C# code.

Just realised, this is not that straight forward. We probably want our code actions that add @using directives to appear first (ie, the ConflictOption - @using System.Data in the screenshot above), and then the C# code actions, and then the Extract to component last.


The .fountain filetype is not supported

I write screenplays using the Fountain markup language and the Better Fountain extension for VS Code.
Grammarly is not working with my screenplays, and when I enter the command "Grammarly > Check Grammar Errors" I get "The .fountain filetype is not supported."

Hey @thegreentimtam,
You can check this issue --> #126.  If you have a windows computer you can disable it.
I have the same problem with js & tsx files but I have macOS so I'm not sure how to override this restriction.
@znck, what is the solution on mac machines?

Search text in C:\Users\user.vscode\extensions\znck.grammarly-0.14.0\dist\extension\index.js
This is an array:
const LANGUAGES = [     'asciidoc',     'git-commit',     'git-rebase',     'json',     'latex',     'markdown',     'mdx',     'plaintext',     'restructuredtext', ];
Edit it to:
const LANGUAGES = [     'asciidoc',     'git-commit',     'git-rebase',     'json',     'latex',     'markdown',     'mdx',     'plaintext',     'restructuredtext',     'fountain', ];
This solved the issue with *.fountain files for me.

Follow https://github.com/znck/grammarly#adding-support-for-new-language to add a new language.
There are no language restrictions now, you have to opt-in by adding file name patterns:


Tooltip stays displayed on top of MenuItem components of a Select component

When using a Tooltip component around a Select component containing MenuItems, when you click the Select (not only Tooltip stays displayed but it also sticks above the MenuItems) and click a MenuItem, the Tooltip stays displayed as if we were still hovering the Select component. Now to make it disappear you can re-hover the Select, or click anywhere else in the DOM.
I feel like these are two separate issues.

The Tooltip having a z-Index above the one of the MenuItem (although this may be specific to my very own use case only)

The other one would be that the tooltip stays displayed after clicking a MenuItem to change the Select's value, when it is not even being hovered anymore.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

When clicking the Select component, the tooltip should disappear. Therefore, when the MenuItems appear, they won't have the tooltip on top of them, and if your MenuItems have a tooltip, it will still be displayed above the other MenuItems as you would expect it and the tooltip would still be invisible after selecting an item in the Select component.
Current Behavior

The tooltip stays displayed on top of everything until the Select component is hovered, or the user clicks anywhere other than the Select component.
Steps to Reproduce (for bugs)

Here is the live example on codesandbox.io.

Click the Select component. (Notice the tooltip stays on top of the MenuItems)
Select any of the MenuItems. (Notice how the Tooltip flickers as you move your mouse)
Click it (The MenuItems disappear but the tooltip is still displayed as if there was a mouse hovering the Select)

Context

Your Environment

Tech
Version

Material-UI
next

React
16.0.0

browser
Chrome 65.0.3325.181

@oliviertassinari I would like to know if this actually an issue or not, if possible! I know that a Button with a Menu/MenuItems has the behavior that I want but I am limited to the use of Select in my case.

I'm experiencing the same issue. I have a tooltip on an icon button that opens a dialog. After the dialog is dismissed, the tooltip appears and will remain active until the button is activated with a mouseover event.

I got my asnwer here.

The controlled answer looks good to me: https://codesandbox.io/s/oxlmqo4m9z.

I am still experiencing this issue.
Currently using a TextField component with the type "select" (the Select component's label is buggy when it shrinks to the top that's why i need it like this) and with the controlled Tooltip open/!open state with the events onMouseEnter, onMouseLeave works just fine but the onClick not.

I also faced same issues while using disableTriggerFocus or defining controlled tooltip as suggested above.
With controlled tooltip, onMouseEnter and onMouseLeave works when select dropdown is opened/closed. But if dropdown is kept open and cursor is moved outside of window and again inside of the open dropdown, tooltip gets shown and overlapped.
Instead, using renderValue prop to wrap the selected value with tooltip worked for me.
const renderValueWrappedByTooltip = (value: any) => {
  return (
    <Tooltip title={title} placement='top'>

{value}

    </Tooltip>
  );
};

return (
  <Select
    ...
    renderValue={renderValueWrappedByTooltip}
  >
    ...
  </Select>
);

https://stackoverflow.com/questions/49198777/material-ui-tooltip-zindex-over-menuitem-in-select-component
try this simple solution with z-index. we have control without making state also


Remove Testnet2 support

Describe your changes

Removed testnet2 from enums
Removed functions with chainId argument
Removed tests for chain-specific testnet

Linked issues

Closes #309
Breaking changes

[x] This issue contains breaking changes

Removed ability to create chain-specific testnet providers

Codecov Report
Patch coverage has no change and project coverage change: -0.24% :warning:

Comparison is base (10d5a64) 69.76% compared to head (6478b23) 69.52%.

:exclamation: Current head 6478b23 differs from pull request most recent head 3be85cd. Consider uploading reports for the commit 3be85cd to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #312      +/-   ##
==========================================
- Coverage   69.76%   69.52%   -0.24%
==========================================
  Files          67       67
  Lines        2510     2487      -23
  Branches      218      216       -2
==========================================
- Hits         1751     1729      -22
+ Misses        686      685       -1
  Partials       73       73

Files Changed
Coverage Δ

...m/swmansion/starknet/data/types/StarknetChainId.kt
100.00% <ø> (ø)

...nsion/starknet/provider/gateway/GatewayProvider.kt
95.65% <ø> (+0.01%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


@External annotation

This PR introduces the @External annotation to replace the (external = true) attribute in @Subscribe, @React, and @Command annotations.

@armiol, PTAL.


Issue 43442: samples: not able to delete a Material not in a sample type

Issue 43442: samples: not able to delete a Material not in a sample type

SampleTypeUpdateServiceDI needs to handle read and delete without a ExpSampleType
Add regression test to verify read, insert, update, delete permissions

Issue 43407: Required boolean fields cannot be set to false in the sample derivation view

The action uses custom form binding of the posted parameters and didn't handle the special @ field prefix for checkbox input parameters.

Issue 43439: LK Server Allows Insert of Samples Outside a Specific Sample Type

Remove the option to insert into exp.Materials
Validate a SampleType was chosen

Bonus Round

Add SqlFragment.appendIf for all those conditional sql woes
Remove obsolete ~checkboxes parameter handling
Clear selection after deleting samples

Moved to #2423 for 21.7 release


fix "Unable to load file" when the exe in a Chinese name"

Use BeginUpdateResourceW and LoadLibraryExW.
I am not C/Win developer, so, I cannot say that it is an ideal solution. But works.

👍


Stabilize [script(…)] attribute

A [script(…)] attribute was added in #2259. See #1479 for discussion.
The attribute is currently unstable.
Before stabilization:

Make sure it's tested and has the right semantics.
Is script the right name?
Consider renaming "shebang recipes" in readme to "script recipes" and consider both shebang recipes and recipes with the script attribute as cases of script recipes.
Add a script-interpreter setting and allow an empty [script] attribute.

Is it possible to add ext name support for script attribute:
[script("pluto")]
[extension: 'lua']
foo:
  print(os.getenv("NICK"))

And following code is clean.
[script("pluto", "lua")]
foo:
  print(os.getenv("NICK"))

If you use JavaScript/Types, lots of extension names:  js, cjs, mjs, ts, mts.
[script("bun", "ts")]
foo:
  let name: string = "jackie"
  console.log(name)

I think that would probably be confusing, and using an additional attribute for the extension isn't much of a burden. But note that an extension is only required if the interpreter requires one. Most don't, in fact the only interpreters I know if that will fail without the correct extension is cmd.exe and PowerShell.

Also, one more reason for not stabilizing the [script] annotation right away: Since this is a new attribute, while it is unstable it is possible in a backwards compatible way to change the semantics of recipes with that attribute. In #1413 I discussed some shortcomings with shebang recipes, and discussed a lot of somewhat radical ideas for changing the semantics of recipes with a [script] attribute. Ultimately I don't think any of those were great ideas, and I don't have any particular ideas for changing the semantics of recipes with the [script] attribute, but I think it's a good idea to let it stew for a bit in case anyone can come up with some.

I wonder if there's a way to conditionally includes settings or variables to recipt like the cfg_attr in rust.
  SH := "sh"

  [cfg_attr(unix, ```
     set shell := ["bash", "-c"]   // setting
     SH := bash                    // variable
  ```)]
  [cfg_attr(windows, ```
     set shell := ["cmd", "/c"]
     SH := cmd
  ```)]
  [cfg_attr(cygwin, ```
     set shell := ["bash", "-c"]   // overwrite windows
     SH := cygwin
     CYGWIN := 1
  ```)]
  [cfg_attr(true, ```
     set unstable
     UNSTABLE := 1
  ```)]
  foo:
    echo {{SH}} // print "cygwin" or "bash" or "cmd"
    echo {{CYGWIN}} // print "1" or ""
    echo {{UNSTABLE}} // print "1"

  bar:
    echo {{SH}} // print "sh"
    echo {{CYGWIN}} // print ""
    echo {{UNSTABLE}} // print "1"

@g9wp This seems unrelated to the [script] attribute.

@g9wp This seems unrelated to the [script] attribute.

This is a more general solution, [script] is just equivalent to [cfg_attr(true, set shell)].


Add cylinder moment of inertia

This included gpkitmodels/moi/moi.pdf. What's our best-practice suggestion for md writeups: should the built pdf be included in the repo, or gitignored?
Also, should these have makefiles to create the pdf?

Included so that visitors to the repository can look at it in all its glory.
Makefiles are still in https://github.com/hoburg/gpkit-models/pull/70, plus some local commits and updates I'm going to polish up...


keyd does not work when window change is detected by keyd-application-mapper

If keyd-application-mapper detects that the Active window has changed, keyd will not work until I release all the keys that are being pressed.
keyd: 2.3.0-rc
gnome: 3.38

I believe this should be fixed in the latest commit.
On Mon, Mar 21, 2022 at 9:53 AM shungo27 @.***> wrote:

If keyd-application-mapper detects that the Active window has changed,
keyd will not work until I release all the keys that are being pressed.
keyd: 2.3.0-rc
gnome: 3.38
—
Reply to this email directly, view it on GitHub
https://github.com/rvaiya/keyd/issues/160, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOP7GJHH2IZANWTEAAGLYTVBB5MNANCNFSM5RHYLHYQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you! It's working.


Demo UI

New:

Demo UI
recipe configs for the new model

Updated:

Added log statement to detector
Added recipe name to QueryTaskGraph

@Purg PTAL
(this was also tested by @alexander-lynch on the live system)


Merge codegen/develop into master for release

The code in codegen/develop is stable and ready for release. This pull request merges those changes into the master branch so that it can be published and released.

I'm going to use my administrator privileges to push this one through. ✊
(All of these changes have already been reviewed and accepted with individual pull requests.)


Add model check for chat stream

Add model check for CreateCompletionStream and CreateChatCompletionStream.
Correct model check in CreateCompletion.

Hey, thank you for the PR!

Hi, I am wondering if it is necessary to have compatibility checks between the endpoint and model, or if it can be removed and let it fail. Here are some points to consider:

OpenAI will regularly release new models, which requires us to update the checks here in a timely manner, or else it will hinder users from using the new models.
If an incompatible model is used in the endpoint, OpenAI API will return a clear error message, which should be enough to inform users of the error.

@j178 What if we check for "incompability" — basically return error only when we are sure that the model is not supported?

basically return error only when we are sure that the model is not supported?

While technically possible, this approach would be very long and difficult to maintain. We would need to explicitly list all unsupported models for each endpoint check, like this:
func checkEndpointSupportsModel(endpoint, model string) error {
	switch endpoint {
	case "/completions":
		switch model {
		case GPT3Dot5Turbo0301, GPT3Dot5Turbo, GPT4, GPT40314, GPT432K, GPT432K0314:
			return ErrCompletionUnsupportedModel
		}
	case "/chat/completions":
		switch model {
		case GPT3TextDavinci003, GPT3TextDavinci002, GPT3TextCurie001, GPT3TextBabbage001, GPT3TextAda001, GPT3TextDavinci001, GPT3DavinciInstructBeta, GPT3Davinci, GPT3CurieInstructBeta, GPT3Curie, GPT3Ada, GPT3Babbage, CodexCodeDavinci002, CodexCodeCushman001, CodexCodeDavinci001:
			return ErrChatCompletionInvalidModel
		}
	}
	return nil
}

@j178 here's a simpler approach:
type endpointModelPair struct {
	endpoint string
	model    string
}

var disabledPairs = map[endpointModelPair]bool{
	{"/completions", "gpt-3.5-turbo"}: true,
}

func main() {
	pairIsDisabled := disabledPairs[endpointModelPair{"/completions", "gpt-3.5-turbo"}]
	fmt.Println("Pair is disabled:", pairIsDisabled)
}

We can also use map[string]map[string]bool to have endpoint->model->disabled flag mapping.

I applied your suggestion, but I also had to introduce a breaking change: I deleted ErrChatCompletionInvalidModel and ErrCompletionUnsupportedModel, and replaced them with a unified ErrUnsupportedModel. Do you have any better suggestions?

The current CI failure is quite peculiar(I was able to pass the test locally). The panic stack trace appears at stream_test.go:167 and stream_test.go:169, even though the stream_test.go file only has 165 lines in total.
=== RUN   TestCreateCompletionStreamError
    stream_test.go:167: CreateCompletionStream returned error: this model is not supported with this method, please use CreateChatCompletion client method instead
--- FAIL: TestCreateCompletionStreamError (0.00s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x88aac1]

goroutine 171 [running]:
testing.tRunner.func1.2({0x8c6f80, 0xbd9fc0})
	/opt/hostedtoolcache/go/1.19.7/x64/src/testing/testing.go:1396 +0x372
testing.tRunner.func1()
	/opt/hostedtoolcache/go/1.19.7/x64/src/testing/testing.go:1399 +0x5f0
panic({0x8c6f80, 0xbd9fc0})
	/opt/hostedtoolcache/go/1.19.7/x64/src/runtime/panic.go:890 +0x262
github.com/sashabaranov/go-openai_test.TestCreateCompletionStreamError(0xc00027fd40)
	/home/runner/work/go-openai/go-openai/stream_test.go:169 +0x581
testing.tRunner(0xc00027fd40, 0x954fa0)
	/opt/hostedtoolcache/go/1.19.7/x64/src/testing/testing.go:1446 +0x217
created by testing.(*T).Run
	/opt/hostedtoolcache/go/1.19.7/x64/src/testing/testing.go:1493 +0x75e
FAIL	github.com/sashabaranov/go-openai	0.196s
FAIL
Error: Process completed with exit code 1.

In terms of panic: I don't even see the TestCreateCompletionStreamError in your fork, could you please sync your form with the master branch?


remembers old directories when serving on previous ports

I once served a static file system locally on port 2002 using -p
Now I've killed the server and navigated to a different port. However, it's still serving from the same old directory if I try to use that port.

@brendajin are you able to reproduce this on the latest version of http-server?

This sounds like some overly-heavy caching by the browser, which can be stopped with the -c-1	option, which was broken but is fixed by #326


i want to run this program

I want to run this program, but I cannot find the header file GL/glew.h

Hey, you must install glew library. If you are using linux debian distro, try with this:
sudo apt-get install libglew-dev

Hey, you must install glew library. If you are using linux debian distro, try with this:
sudo apt-get install libglew-dev


Fractional knapsack Algorithm

Here the problem is solved using the Knapsack algorithm. Hope you like it

Checklist

[x] I've looked at the contribution guidelines.
[x] This pull request is complete and ready for review.

Description

fractional knapsack algorithm

Hey @swatiprajapati08 - we've updated our guidelines and we're no longer accepting interview-type questions.


IDVA6-1467 refactoring

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-1467
Changes:

Unnecessary imports removed
"Unexpected any" issue indicated by Lint fixed
Additional unit tests added

CI: No security warnings found

CI: No security warnings found

CI: No security warnings found


Donald Hebb

yo, where is my book?

https://www.amazon.com/gp/product/0471367273/ref=oh_aui_detailpage_o04_s00?ie=UTF8&psc=1


added prefix

Closes #2

Please have alook @fabiante

Done! @fabiante . Added your suggested documentation.

@krishnaduttPanchagnula I did not notice that I had to approve the github action. I just did and now it shows that the tests have to be modified according to your change.

@krishnaduttPanchagnula I did not notice that I had to approve the github action. I just did and now it shows that the tests have to be modified according to your change.

@fabiante  can you rerun the pipline , i have changed the test

Nice, thank you! I will update the CHANGELOG and bump the minor version.


Remove i686-apple-darwin from CI

This target is no longer supported by rustup.

@bors r+

:pushpin: Commit 389d448c1f3bd4b61384b54a657081928d50c300 has been approved by Amanieu

:hourglass: Testing commit 389d448c1f3bd4b61384b54a657081928d50c300 with merge a22622cb9f83a664629fa34afa5832135bb34038...

:sunny: Test successful - checks-travis
Approved by: Amanieu
Pushing a22622cb9f83a664629fa34afa5832135bb34038 to master...


FollowerService only accepts 1 client

Hey so currently the FollowerService only allows 1 client to be used per instance.
Would be awesome if multiple clients could be used in only 1 instance.

Hey, thanks for creating an issue. Currently debating if it's better design to just create multiple follower service objects, or to have a single follower service object. The reason is because there are a number of instance variables in follower service that the customer may want to differentiate between services (cache, interval, etc). Thoughts?

I think that it should be a single FollowerService object with like optional parameters in a ListenToChannel method so that each channel entry could have its own requirements, and like store them as a dictionary of channel name and last followers response (optionally: alongside a DateTime object to know when the last update occurred).

Ya, you're probably right, I think. Will think it over for a little bit longer though.

This issue was moved to TwitchLib/TwitchLib.Api#35


Bug: File truncated

Hello,
When we are drag'n dropping or opening through a .php file to Brackets 1.6 under El Capitan 10.11.5, the file is truncated as seen in the below screenshots. The same file is not truncated with Xcode editor. The file is in a samba share running Centos. The first screenshot is of Brackets and the other one is Xcode.

Does still reproduce on 1.7? If so, can you share a minimal snippet which reproduce the problem?

The version is Release 1.7 build 1.7.0-16898 and the issue persists.


Week_1_Assignment

The code appears to be well-formed and follows proper syntax.
Here are a few suggestions for improvement:

You might also consider adding some placeholder text to the input fields to provide guidance to users on what kind of information they should enter.
You can try using different fonts to make it more appealing, and give more styling to the buttons.

Well done!


Close webpack-serve

Operating System: macOS 10.13.5
Node Version: 9.10.0
NPM Version: 5.6.0
webpack Version: 4.16.0
webpack-serve Version: 2.0.2

This issue is for a:

[x] bug
[x] feature request
[ ] modification request

Code
Expected Behavior
Return value of serve exposes a close function like<EMAIL_ADDRESS>Actual Behavior
No close function is exposed
How Do We Reproduce?
Start webpack-serve using the node API and try to call close.
const server = serve(...)
serve.close()
// never closes

New Feature Use Case

Can confirm changing to the following works:


missing system references for Gromacs data

When the superclass for atomistic parsers was implemented it changed the way that Gromacs system references are being assigned. Before the calculation and system lists were being matched by time (if available), and now they are matched by step.
The problem is that we are using MDAnalysis to obtain this information for system. The times are assigned correctly by MDA, but the steps do not correspond to the actual simulation step, but rather just an index for the stored trajectory frames.
We need to either go back to first trying to match by times for any data using the MDA util, or for Gromacs data we could alternatively grab the steps directly from the .trr/.xtc files (which are stored correctly).
@ladinesa what do you think? Can you address this?

sorry, here is an example you can look at tests/data/parsers/gromacs/protein_small_nowater/md.log from the nomad tests

sorry for causing the mess.i will fix this asap.

np, thank you!
It is just because the MDAnalysis universe is not storing the steps from the trajectory file. It indexes each from of the trajectory simply with 0,1,2,3,.... But it doesn't take into account the save frequency of the simulation. E.g., in the simulation I referenced, save freq is 100, so the thermo data (read directly from the edr file) has steps 0, 100, 200, 300,... instead.


fix invalid url

i fix a invalid url!thanks!

You can still use the git.k8s.io domain, you'd just need to change the path instead. See - http://git.k8s.io/community/contributors/guide/contributor-cheatsheet/README.md


Remove duplicate Fission keys in secret

Signed-off-by: Chase Martin<EMAIL_ADDRESS>
What type of PR is this?
/kind bug
/kind chart-release

Any specific area of the project related to this PR?
/area falcosidekick-chart

What this PR does / why we need it:
When installing/upgrading the falcosidekick Helm chart, an error is thrown due to duplicated FISSION_* keys in the secrets.yaml file.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: chasestech
To complete the pull request process, please assign leogr after the PR has been reviewed.
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @chasestech! It looks like this is your first PR to falcosecurity/charts 🎉

LGTM label has been added.  Git tree hash: 2db57baed2e8d9e0f286c729c0ecc457000c0a16

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: chasestech, Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


route-converter change

Describe the bug
With the current implementation, when using MapLibre GL JS map.on('click'), the nested object becomes a string.
Expected Behavior
{     "type": "FeatureCollection",     "features": [         {             "type": "Feature",             "properties": {                 "DataSource": "Here",                 "Distance": 6.157,                 "DistanceUnit": "Kilometers",                 "DurationSeconds": 597             },             "bbox": [<PHONE_NUMBER>,                 35.664799,                 139.775649,                 35.702796             ],             "geometry": {                 "type": "MultiLineString",                 "coordinates": [                     [                         [                             139.746602,                             35.664799                         ],                         [                             139.775649,                             35.702796                         ]                     ]                 ]             }         }     ] }
Current Behavior
With the current implementation, when using MapLibre GL JS map.on('click'), the nested object becomes a string.
[     {         "geometry": {             "type": "LineString",             "coordinates": [                 [<PHONE_NUMBER>3413086,                     35.66479283857099                 ],                 [<PHONE_NUMBER>6450806,                     35.70278797825108                 ]             ]         },         "type": "Feature",         "properties": {             "Summary": "{\"DataSource\":\"Here\",\"Distance\":6.157,\"DistanceUnit\":\"Kilometers\",\"DurationSeconds\":597}"         },         "layer": {             "id": "route-result",             "type": "line",             "source": "route-result",             "layout": {                 "line-join": "round",                 "line-cap": "round"             },             "paint": {                 "line-color": {                     "r": 1,                     "g": 0,                     "b": 0,                     "a": 1                 },                 "line-width": 10,                 "line-opacity": 0.5             }         },         "source": "route-result",         "state": {}     } ]
Reproduction Steps
map.on('click', 'route-result', (e) => {         console.log(e.features); });
Possible Solution
Added "flattenProperties" to flatten GeoJSON properties.
https://github.com/aws-geospatial/amazon-location-utilities-datatypes-js/pull/34
Additional Information/Context
No response
Environment details (OS name and version, browser, etc.)
All environments are covered

Thanks @dayjournal, just reviewed your PR. Since we will have 2 places with similar flattenProperties function after your proposed PR. Is it possible to combine them and extract them to a standalone file?

Hi @dayjournal , my teammate @eashuang reviewed your PR and left some comments. We would be able merge it once the comments are addressed.

Hi, @eashuang, @juaneslucero
Changed pull requests to the latest.

Hi, @eashuang, @juaneslucero
Can you check my pull request? And update npm to the latest version.
I will write an article when this is updated.

Hi @dayjournal, npm has been updated with the latest version. Thank you for your contribution.


Consider moving Recently Pushed Branches elsewhere

Following #1106 and #906
How does this position look?

To achieve this, the element would have to be moved between the two elements and floated right:
select('.pagehead-actions').after(select('.js-socket-channel'));

But probably it has to be move inside and positioned absolutely in case the repo name or branch name are too long to fit in one line.
Alternatively there's this position, that expands on hover:

The space between repo name and the buttons can be limited on small screens.
Maybe try with position: fixed, which would allow the user to scroll potentially hidden content into view.

Maybe try with position: fixed

That might make it more annoying (since it floats + it could cover anything on the screen, on whatever border we fix it to)

I personally don't mind it if it overlaps with something and especially if that's the repo name (even though it may hide the commit status link we add there).
I like the first suggestion.

Alternatively there's this position, that expands on hover:

I like this one.

Personally I liked it when it stretched across the top of the page.
Honestly, the green "PR" button needs to be bigger to click on, and would love to see this bar/notification on any page in the repo.
Very rough image:

Did you think about moving out just under fork and star?

Did you think about moving out just under fork and star?

I don't think there's enough space, it will feel cramped if there are tabs under it.

I hope we can fix this also for forks:

Steps to reproduce

Fork project.
Create branch and commit.
Go to your forked project homepage.

Expected
Expected to see the top buttons and the recent-branches banner somewhere else:

See the same issue as @jerone for regular repositories as well:

The size won't actually always fit that area, it depends on the branch name. Also I'm hoping to extend the search box to fill that blank spot.

PR sent

Finally done! You can also disable it now: recently-pushed-branches-enhancements


feat(AlgebraicGeometry/Tilde): add the map from M localising at x to the stalk of M^~ at x

Co-authored-by: Weihong Xu<EMAIL_ADDRESS>Co-authored-by: Sophie Morel<EMAIL_ADDRESS>Co-authored-by: Amelia Livingston<EMAIL_ADDRESS>This contribution was created as part of the AIM workshop "Formalizing algebraic geometry" in
June 2024.

[x] depends on: #15315

I personally think that the added import is fine as the file is a leaf file and the contents added are core properties of tilde M. But any maintainer might very well disagree.
Otherwise LGTM
maintainer merge


feature：support h2 in AT mode

Ⅰ. Describe what this PR did
support h2 DataBase
Ⅱ. Does this pull request fix one issue?
fixes #4122
Ⅲ. Why don't you add test cases (unit test/integration test)?
Test cases have been added
Ⅳ. Describe how to verify it
The SQL file is the same as mysql
As follows:
CREATE TABLE undo_log (
id bigint(20) NOT NULL AUTO_INCREMENT,
branch_id bigint(20) NOT NULL,
xid varchar(100) NOT NULL,
context varchar(128) NOT NULL,
rollback_info longblob NOT NULL,
log_status int(11) NOT NULL,
log_created datetime NOT NULL,
log_modified datetime NOT NULL,
ext varchar(100) DEFAULT NULL,
PRIMARY KEY (id),
UNIQUE KEY ux_undo_log (xid,branch_id)
) ;
Ⅴ. Special notes for reviews
See for details ：
H2DeleteExecutorTest.java
H2InsertExecutorTest.java
H2DeleteRecognizerTest.java
H2InsertRecognizerTest.java
H2UpdateRecognizerTest.java

<EMAIL_ADDRESS>

https://github.com/seata/seata/pull/4096 参考这个pr的标题和内容说明

已经按照要求调整标题和内容

Codecov Report

Merging #4133 (776d46e) into develop (193a112) will increase coverage by 0.31%.
The diff coverage is 46.42%.

:exclamation: Current head 776d46e differs from pull request most recent head 9103ec9. Consider uploading reports for the commit 9103ec9 to get more accurate results

@@              Coverage Diff              @@
##             develop    #4133      +/-   ##
=============================================
+ Coverage      49.09%   49.41%   +0.31%
+ Complexity      4032     3810     -222
=============================================
  Files            730      713      -17
  Lines          25278    24028    -1250
  Branches        3126     2963     -163
=============================================
- Hits           12411    11873     -538
+ Misses         11545    10941     -604
+ Partials        1322     1214     -108

Impacted Files
Coverage Δ

...ta/rm/datasource/undo/h2/H2UndoDeleteExecutor.java
0.00% <0.00%> (ø)

...ta/rm/datasource/undo/h2/H2UndoExecutorHolder.java
0.00% <0.00%> (ø)

...ta/rm/datasource/undo/h2/H2UndoInsertExecutor.java
0.00% <0.00%> (ø)

...ta/rm/datasource/undo/h2/H2UndoUpdateExecutor.java
0.00% <0.00%> (ø)

.../sqlparser/druid/h2/H2OperateRecognizerHolder.java
0.00% <0.00%> (ø)

...qlparser/druid/h2/H2SelectForUpdateRecognizer.java
0.00% <0.00%> (ø)

.../seata/rm/datasource/undo/h2/H2UndoLogManager.java
9.37% <9.37%> (ø)

.../io/seata/sqlparser/druid/h2/BaseH2Recognizer.java
48.78% <48.78%> (ø)

...o/seata/sqlparser/druid/h2/H2DeleteRecognizer.java
54.54% <54.54%> (ø)

.../seata/rm/datasource/exec/h2/H2InsertExecutor.java
58.62% <58.62%> (ø)

... and 202 more

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: HongYan789:x: a364176773You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: a364176773:x: HongYan789You have signed the CLA already but the status is still pending? Let us recheck it.

Due to code conflicts and age, if you are still interested in participating in the community, please resubmit this pr to the 2.x branch after resolving the conflict.
由于代码冲突以及年代久远，如果你还有兴趣参与到社区，请解决完冲突后重新请提交这份pr至2.x分支中


non-dates are often converted to datetime

Described
issue where non-dates are often converted to datetime.
Environment
OS: Windows 10
Python:  3.8.2
IDE: Scite
Package versions
pandasgui==0.2.11
pandas==1.1.4

I changed utility.py line  to

`            return pd.to_datetime(s, format="%b %d %Y")`
and now I only see
_PandasGUI WARNING — pandasgui.utility — In metadata_df, converted columns to datetime: RecordingDate_
which is desired.

It does convert everything else to a string (like float columns), but that is not a big issue for me, visually.
Here are the first two lines and header which are a result of `df.to_csv('metadata_df.csv')` which I do just before `show(metadata_df)`
[metadata_df.txt](https://github.com/adamerose/PandasGUI/files/6625776/metadata_df.txt)

Firstly, when I upgraded to 0.2.11 it seems to need a new module
`>pip install pandasgui==0.2.11`
I got `ModuleNotFoundError: No module named 'qtstylish'`

python
Python 3.8.2 (tags/v3.8.2:7b3ab59, Feb 25 2020, 23:03:10) [MSC v.1916 64 bit (AMD64)] on win32

import pandas
pandas.version
'1.1.4'
import pandasgui
Traceback (most recent call last):
File "", line 1, in
File "C:\Python38\lib\site-packages\pandasgui_init_.py", line 15, in
from pandasgui.gui import show
File "C:\Python38\lib\site-packages\pandasgui\gui.py", line 20, in
import qtstylish
ModuleNotFoundError: No module named 'qtstylish'
import pandasgui
pandasgui.version
'0.2.11'

but it did install fine: `pip install qtstylish`, so now I have

python
Python 3.8.2 (tags/v3.8.2:7b3ab59, Feb 25 2020, 23:03:10) [MSC v.1916 64 bit (AMD64)] on win32

import pandas
pandas.version
'1.1.4'
import pandasgui
pandasgui.version
'0.2.11'

I did see the simple call in utility.py parse_dates_series(), so it appears to me now it is a pandas "thing", but maybe the question was why is pandasgui trying to recast columns at all?

So, it is likely that reading from CSV is not a proper test here.
The data was strings read from report forms, and the df types as created are

...
Side                    object
Location                object
NIHSScore              float64
TICIScore               object
IV_tPA_timing           object
EVT_timing              object
RecordingDate           object
RecordingStartTime      object
RecordingStopTime       object
Gender                  object
Age                     object
SystolicBP              object
DiastolicBP             object

my calling code saves that df to csv, then opens pandasgui on the dataframe for a look

    metadata_df.to_csv('metadata_df.csv')
    try:
        show(metadata_df)
    except ValueError:
        pass

When I call show() I get:
![a](https://user-images.githubusercontent.com/20799053/121825595-3737c900-cc68-11eb-9ac0-550e9e204e55.jpg)

I might need to generate a pickle of metadata_df for you to test, if you'd like; if I "filter" the df  through CSV, I get a result very similar to your screen cap, but with floats instead of ints
![b](https://user-images.githubusercontent.com/20799053/121825591-31da7e80-cc68-11eb-9d1f-594aed5f982c.jpg)

Small data pkl in a zip:
[head.zip](https://github.com/adamerose/PandasGUI/files/6644870/head.zip)

When I run the cleaner:
>>> from pandasgui.utility import clean_dataframe
>>> df = clean_dataframe(df)
PandasGUI WARNING — pandasgui.utility — In DataFrame, converted column names to string: SubjectID, Recording, Sequenze,
Status, StrokeType, Side, Location, NIHSScore, TICIScore, IV_tPA_timing, EVT_timing, RecordingDate, RecordingStartTime,
RecordingStopTime, Gender, Age, SystolicBP, DiastolicBP, HeartRate, Height, Weight, Consciousness, NIHSS_Cons, NIHSS_Con
sQuest, NIHSS_ConsComm, NIHSS_Gaze, NIHSS_VisualFields, NIHSS_FacialParesis, NIHSS_MotorArmLeft, NIHSS_MotorArmRight, NI
HSS_MotorArmTotal, NIHSS_MotorLegLeft, NIHSS_MotorLegRight, NIHSS_MotorLegTotal, NIHSS_LimbAtaxia, NIHSS_Sensory, NIHSS_
Language, NIHSS_Dysarthria, NIHSS_Neglect, HeadsetSN, Coordinator
PandasGUI WARNING — pandasgui.utility — In DataFrame, converted columns to datetime: RecordingDate, Age, SystolicBP, Dia
stolicBP, HeartRate, Height, Weight, NIHSS_Cons, NIHSS_ConsQuest, NIHSS_ConsComm, NIHSS_Gaze, NIHSS_VisualFields, NIHSS_
FacialParesis, NIHSS_MotorArmLeft, NIHSS_MotorArmRight, NIHSS_MotorArmTotal, NIHSS_MotorLegLeft, NIHSS_MotorLegRight, NI
HSS_MotorLegTotal, NIHSS_LimbAtaxia, NIHSS_Sensory, NIHSS_Language, NIHSS_Dysarthria, NIHSS_Neglect, HeadsetSN

View:
> dfh = df.head
> dfh
<bound method NDFrame.head of      UID  Subject  SiteID  ...                 NIHSS_Neglect                     HeadsetSN
          Coordinator
0      0        1       1  ... 1970-01-01 00:00:00.000000000 1970-01-01 00:00:00.000101109      Matthew Mannion
1      0        1       1  ... 1970-01-01 00:00:00.000000000 1970-01-01 00:00:00.000101109      Matthew Mannion
2      0        2       1  ... 1970-01-01 00:00:00.000000000 1970-01-01 00:00:00.000101109          Lena Tabbaa
3      0        2       1  ... 1970-01-01 00:00:00.000000000 1970-01-01 00:00:00.000101109          Lena Tabbaa
4      0        2       1  ... 1970-01-01 00:00:00.000000000 1970-01-01 00:00:00.000101109          Lena Tabbaa
..   ...      ...     ...  ...                           ...                           ...                  ...
151    0       23       2  ... 1970-01-01 00:00:00.000000002 1970-01-01 00:00:00.000101109  Rachel Supple, PA-C
152    0       23       2  ... 1970-01-01 00:00:00.000000002 1970-01-01 00:00:00.000101109  Rachel Supple, PA-C
153    0       24       2  ... 1970-01-01 00:00:00.000000001 1970-01-01 00:00:00.000101101     Shakeel Chowdhry
154    0       24       2  ... 1970-01-01 00:00:00.000000001 1970-01-01 00:00:00.000101101     Shakeel Chowdhry
155    0       24       2  ... 1970-01-01 00:00:00.000000001 1970-01-01 00:00:00.000101101     Shakeel Chowdhry

[156 rows x 44 columns]>

Can you share the file for metadata_df.csv? I can't reproduce this, and your pickle seems to be the object after columns get converted to dates not before. PandasGUI should only convert columns to dates if they are str and if pd.to_datetime succeeds on the column.

Argg, sorry, that was a pkl with the converted columns. The columns being converted with warnings are object type in the pkl, rather than the newer StringDtype. I might need to be explicit in the upstream script that saves the pkl; if I convert to string as below, all is well.
Here is the head data properly:
head.zip
Here is some CMD testing (df is the whole, long file dataframe, dfh is the head):
>>> dfh
   Unnamed: 0  UID  Subject  SiteID  ... NIHSS_Dysarthria NIHSS_Neglect HeadsetSN      Coordinator
0           0    0        1       1  ...              0.0           0.0  101109.0  Matthew Mannion
1           1    0        1       1  ...              0.0           0.0  101109.0  Matthew Mannion
2           2    0        2       1  ...              0.0           0.0  101109.0      Lena Tabbaa
3           3    0        2       1  ...              0.0           0.0  101109.0      Lena Tabbaa
4           4    0        2       1  ...              0.0           0.0  101109.0      Lena Tabbaa

[5 rows x 45 columns]
>>> dfh['Consciousness']
0    NaN
1    NaN
2    NaN
3    NaN
4    NaN
Name: Consciousness, dtype: object
>>> dfh.to_pickle(r'E:\projects\Jan Medical\software\quality\data\head.pkl')
>>> show(dfh)
PandasGUI INFO — pandasgui.gui — Opening PandasGUI...

The columns being converted with warnings are object type in the pkl, rather than the newer StringDtype

Ah yeah you're right, I convert columns to date if they're object type because that's often the type of columns containing strings, and I believe there's no way to differentiate between typical string columns and other types of object column like dict or whatnot, so I just try converting every object column to a date. StringDtype is actually unaccounted for, I'll fix that.
I loaded your pickle and ran parse_dates and these are the columns that got turned into dates on my end.
# This is good
RecordingDate          datetime64[ns]
RecordingStartTime     datetime64[ns]
RecordingStopTime      datetime64[ns]
# This is wrong. These `object` columns are all NaN and pd.to_datetime still converts them to datetime
TICIScore              datetime64[ns]
IV_tPA_timing          datetime64[ns]
EVT_timing             datetime64[ns]
Consciousness          datetime64[ns]

> pd.to_datetime(pd.Series([pd.np.nan, pd.np.nan, pd.np.nan, pd.np.nan]))
Out[3]:
0   NaT
1   NaT
2   NaT
3   NaT
dtype: datetime64[ns]

So here are the issues I see:

parse_dates is too aggressive and sometimes converts things to datetime when not desired.
SOLUTION -  I added an if-statement to check if the whole column is NaN and skip it in that case
StringDtype doesn't get parsed into a date
SOLUTION - Check for this as well as object
Any logic for identifying if a column should be a date will not be perfect
SOLUTION - I'm removing parse_dates from the automatic DataFrame cleaning on load, and instead adding it to the Edit menu as an optional action. I've also added this option to the context menu for column actions

Very nice, thanks!


🔌  Silent WebSocket Disconnects

Issue
SocketTransport handles disconnects (onError or onDone events) silently by marking itself as not connected. When these disconnects occur, a WalletConnectSession appears to be connected but cannot send or receive messages.
Impact
In release and profile mode, when a user puts the app into the background (i.e. to sign a message in their wallet app), the sockets get closed. On return, the session can't send or receive messages despite it looking connected.
Fix
Re-open the socket when onError and onDone events are fired.
I am opening a PR on this and will link it here.

This may viable workaround:
  // add to ScreenState  'with WidgetsBindingObserver '
  // https://api.flutter.dev/flutter/widgets/WidgetsBindingObserver-class.html

  @override
  void initState() {
    // Register observer so we can see app lifecycle changes.
    WidgetsBinding.instance!.addObserver(this);
    super.initState();
  }

  /// The wallet connect client, sometimes loses the webSocket connection when the app is suspended
  @override
  void didChangeAppLifecycleState(AppLifecycleState state) {
    DateFormat dateFormat = DateFormat("HH:mm:ss");
    String dateString = dateFormat.format(DateTime.now());

    logger.d("$dateString AppLifecycleState: ${state.toString()}.");

    if (state == AppLifecycleState.resumed) {
      if (mounted) {
        if (walletConnect.connected && walletConnect.transport.connected == false)) {
          logger.e('$dateString  Wallet connected, but transport is down.  Attempt to recover.');
          appSettings.walletConnectService.walletConnect.transport.open();
        }
      }
    }
  }

  @override
  void dispose() {
    super.dispose();
    // Remove observer for app lifecycle changes.
    WidgetsBinding.instance!.removeObserver(this);
  }

The reconnection logic is now improved in v0.0.8.
We will also introduce a Flutter widget to keep the connection alive, based on @bobwith2bees code.
Thanks @bobwith2bees and @ghunkins


Shorten time till dormant after claimer deletes their message(s)

Idea behind this issue is to lower the time until a channel becomes dormant/closed when original claimer deletes their messages if there are no replies to their question.
Currently channel will only go dormant after 30 mins of inactivity or when someone closes it manually.
There are cases in which some people will claim a channel and find a solution to their problem before anyone will even reply and due to not knowing they can close channel manually, they might delete their messages which makes channel appear as if it was still available, yet it is claimed and is in Python Help: Occupied category, and stays so for 30 minutes of inactivity or until staff member closes it.
Ideally, the bot could check if the messages were deleted and shorten the time till channel closes automatically. Shortening time to 5-10 minutes sounds like a good time in such case as it gives the claimer possibility to send the message again (in case they deleted it to format it better) and if that's not the case, the channel would become dormant sooner, which can be good thing to have in peak hours of help channels usage.

 It would work well with #898 too.

Can I work with this + #898 at once?


New Cluster Setup

As a [dev]
I need [the same cluster settings]
So that [things process as they should]
Details and Assumptions

[This will take a bit of looking back to see what they've done]

Acceptance Criteria
Given [we trying to operate the cluster]
When [we pull it up]
Then [we should have the same settings]

Created the K8's namespaces for Dev/Prod.


Decontracting basis set files with gto.uncontract

Hi,
I've encountered an issue while trying to decontract basis set files downloaded from BSE (https://www.basissetexchange.org/), as
the very last mf.kernel() in the script below yields a numpy.linalg.LinAlgError.
from pyscf import dft, gto
from pyscf.gto.basis import parse_gaussian

basis = {
         'H': parse_gaussian.load('pc1.gbs','H'),
         'O': parse_gaussian.load('pc1.gbs','O')
         }

unc_basis = {
             'H': gto.uncontract(parse_gaussian.load('pc1.gbs','H')),
             'O': gto.uncontract(parse_gaussian.load('pc1.gbs','O'))
            }

atom = [['O',(0, 0, 0)], ['H',(0, 1, 0)], ['H',(0, 0, 1)]]

mol = gto.M(atom=atom,basis='pc1')
mf = dft.UKS(mol)
mf.verbose = 4
e1 = mf.kernel()

mol = gto.M(atom=atom,basis=basis)
mf = dft.UKS(mol)
mf.verbose = 4
e2 = mf.kernel()

mol = gto.M(atom=atom,basis='unc-pc1')
mf = dft.UKS(mol)
mf.verbose = 4
e3 = mf.kernel()

mol = gto.M(atom=atom,basis=unc_basis)
mf = dft.UKS(mol)
mf.verbose = 4
e4 = mf.kernel()

print('delta E contracted: {:.10f}'.format(e1-e2))
print('delta E uncontracted: {:.10f}'.format(e3-e4))

Thanks in advance for your time and help.

There is linear dependence in the unc_basis basis.

https://github.com/pyscf/pyscf/blob/master/examples/scf/11-linear_dep.py  is a workaround at the moment . Commit bfab27b47675a5e7241a19cd7e9ca10b0004d379 will solve the issue

Thank you very much!


Refactor pkg/

There are a few issues with the pkg/ directory:

api, result are not great package names, as it is easy for it to collide/shadow with other names (see https://google.github.io/styleguide/go/decisions#naming)
internal parts should not be in the externally accessible pkgs. internal should depend on pkg not the other way around
Much of the pkg dir is "api" related. This should be better reflected in the naming.

Combining the above problems we could restructure the pkg directory into the following:

pkg/

api/ (this package must remain empty)

notification
analysisrun (includes RunKey (replaces PkgIdentifier), Result, DynamicRunPhase, Status and strace WriteInfo)
ecosystem (includes Ecosystem type)

Sounds good generally!
Regarding specific naming I think we could tweak analysisrun.RunKey and analysisrun.DynamicRunPhase slightly to make them shorter. Could also just name the package analysis instead of analysisrun.
Some suggestions:

(if we use analysis instead of analysisrun) analysis.RunId, analysis.Id; analysis.DynamicRunPhase, analysis.DynamicPhase
(if not) analysisrun.Key, analysisrun.Id; analysisrun.DynamicPhase.

I'd opt for the shortest names: analysis, analysis.Id, analysis.DynamicPhase

I think analysisrun is better than analysis as there is a lot of scope for conflict, and run is more specific than just analysis.
So the suggestions to simplify the structs make sense (i.e. Key, DynamicPhase)

Also, since we have a lot of 'Result' types, perhaps we could make a separate package (analysisresult) to hold all of these?
This may avoid having to have a Result suffix for all of those type names in result.go, especially the prospective analysisrun.DynamicAnalysisResults and analysisrun.StaticAnalysisResults which would be a bit of a mouthful

I think cleaning up "Result" is for a subsequent issue.

Also, I've been trying to implement this and ecosystem is not a great package name due to shadowing.
I'm thinking of using ossecosystem instead of ecosystem and changing EcosystemNPM to NPM. i.e: api.EcosystemNPM -> ossecosystem.NPM.
(any other suggestions are welcome here)

Thinking back to when we did #610 ... one alternative I had come up with was using pkgecosystem as the ecosystem package under pkg and renaming internal/pkgecosystem to something else like pkgmanager pkgmanagers  or pkgmanagement  ?

I like the idea of using pkg/api/pkgecosystem.
Renaming internal/pkgecosystem make sense. Long term, it's really a library of clients for interacting with package repositories.
What about repoclient?
For terminology sake, I consider "ecosystem" the combination of a language, hosting, and other tooling. And a "repository" the online hosting part of that ecosystem.

Yeah using repo sounds good. client may be overstating the functionality though, we could use repoaccess? But happy to go with repoclient if you like.

Actually I might go with pkgmanager from your earlier suggestion. I think that is probably the best name for now

Do we actually need an api top-level directory if almost everything is part of the API anyway?
Having an empty api directory (only containing other directories/packages) doesn't seem like a common pattern in other go libraries that we import.


There are main.dart and main2.dart files in the lib directory. When I open the android project, the main.dart file is executed. How can I modify it to execute the main function in main2.dart?

Such as the title？

Hi @Aaron009
I'm not sure to understand the question,
could you please provide your updated  flutter doctor -v
and a minimal code sample to reproduce the problem?
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Does it  support distillation for large models like Qwen2-72B and LLaMA 3.1-70B?

Does it  support distillation for large models like Qwen2-72B and LLaMA 3.1-70B?

hey @lingq1, it should work with any model. The issue is how much memory you have available. In the tests performed, smaller models were used. I personally haven't tried the 70B. Please change the config and give it a try. Let us know if you hit any blockers.

hey @lingq1, it should work with any model. The issue is how much memory you have available. In the tests performed, smaller models were used. I personally haven't tried the 70B. Please change the config and give it a try. Let us know if you hit any blockers.

ok， ths

@lingq1 You'll probably need to use the distributed recipe for larger teacher models. I've tested the KD distributed recipe on Llama3.1 70B.

You'll probably need to use the distributed recipe for larger teacher models. I've tested the KD distributed recipe on Llama3.1 70B.

Awesome, so how much VRAM was approximately consumed in total and how is the performance?

You'll probably need to use the distributed recipe for larger teacher models. I've tested the KD distributed recipe on Llama3.1 70B.

Awesome, so how much VRAM was approximately consumed in total and how is the performance?

I don't remember the exact numbers. For reference, I used 8 80G A100 GPUs and didn't have any issues with the default distributed recipe set up. Roughly used about 60% of memory according to a recorded experiment, so maybe around 50GB?

You'll probably need to use the distributed recipe for larger teacher models. I've tested the KD distributed recipe on Llama3.1 70B.

Awesome, so how much VRAM was approximately consumed in total and how is the performance?

I don't remember the exact numbers. For reference, I used 8 80G A100 GPUs and didn't have any issues with the default distributed recipe set up. Roughly used about 60% of memory according to a recorded experiment, so maybe around 50GB?

ths, how effective is the final distilled model, and is there anything special to consider in the configuration of the 70B large model distillation recipe?

@lindawangg Could you provide more details on the scenarios where you used the 70B large model as a teacher model for distillation? I have fine-tuned a 7B model for a specific niche in monitoring scenarios, but I'm facing a shortage of GPU resources and would like to compress some GPU usage through distillation techniques.

@lindawangg Could you provide more details on the scenarios where you used the 70B large model as a teacher model for distillation? I have fine-tuned a 7B model for a specific niche in monitoring scenarios, but I'm facing a shortage of GPU resources and would like to compress some GPU usage through distillation techniques.

I haven't done any extensive studies on using the 70B model. We only did some experiments to make sure we're able to run the distributed recipe. I did find the 70B loss to be fairly high, so had to lower the kd_ratio to 0.1. I didn't see any significant improvements though. It could be interesting to try some of these other kd losses, which may yield better results than the default KL divergence loss: https://github.com/pytorch/torchtune/pull/2094.

@lindawangg您能否详细介绍一下使用 70B 大型模型作为蒸馏的教师模型的场景？我已经针对监控场景中的特定领域对 7B 模型进行了微调，但我面临 GPU 资源短缺的问题，因此想通过蒸馏技术压缩一些 GPU 使用量。

ths

Going to close this issue since it seems like the questions have been answered. @lingq1 feel free to reopen if you need any followup here


Add support for Scala 2.10

It seems that this is a useful feature. Please feel free to comment and pick up this issue.
Probable starting point http://docs.scala-lang.org/overviews/quasiquotes/setup#sbt-cross-compile

Closing this as it is more complicated than I had initially thought. Please open another issue in case you find a solution.


Feature : Syntax : Add the final keyword to make a final variable initialization once

Description

Once a final variable has been assigned, it always contains the same value. If a final variable holds a reference to an object, then the state of the object may be changed by operations on the object, but the variable will always refer to the same object this make it non-transitivity. A final variable can only be assign to a value once through out the program.

Type Of Change

[x] New feature: Non breaking change which adds a new functionality ( module, enviroment etc).

Briefly Explain Why This PR Should Be Merged.
Merging the PR will allow the final Keyword to be available throughout the simple-lang to make the lang a more safe and production safe programming language with less vices
How Has This Been Tested ?
Since this is a feature that is part of the language core and it also a syntax addition a final.sim syntax example has been added to the syntax example directory for a lead on how it works.
Also the Console module implements a lot of final variables e.g stdout, stderr, stdin e.t.c to make the variable value the same through out runtime for a more safer module

Checklist:

[ ] I have created a supporting issue for this PR.
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests passed locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

I'll review this.


Update SDK following Axios change

Axios made this change https://github.com/mzabriskie/axios/commit/96d7ac2a0c01b625202aad36ff758e69cf717a07 which broke our progress indicator. This PR fixes this

Published as v6.1.2


apk file is place in same folder and same problem i have attached with screen shots

see i have al ready did that ..........
and then i runed
./backdoor-apk.sh tubemate_2.2.9.apk

2>TERMINAL

its an orignal appliction
backuped my phone apk file

Are u gona reply me or not ,

Go on backdoor-apk.sh open it with edit of text getdit or other afther ctrl + f and text "Owner" he show two line of "Owner" replace it with "Proprietario" Save Rerun and work :)

i allready try it ,
but the result is same ,
if ur are saying so i will try i once more

i think some thing is missing in this code,
plz check this when i open this backdoor apk-sh with gedit its directly take me to this code,
echo -n "[*] Signing recompiled APK..."
$JARSIGNER -verbose -sigalg SHA1withRSA -digestalg SHA1 -keystore $keystore -storepass android -keypass android $compiled_apk signing.key >>$LOG_FILE 2>&1
rc=$?
echo "done."
if [ $rc != 0 ]; then
echo "[!] Failed to sign recompiled APK"
cleanup
exit $rc
fi
echo -n "[*] Verifying signed artifacts..."
$JARSIGNER -verify -verbose -certs $compiled_apk >>$LOG_FILE 2>&1
rc=$?
echo "done."
if [ $rc != 0 ]; then
echo "[!] Failed to verify signed artifacts"
cleanup
exit $rc
fi
mv $compiled_apk $unaligned_apk
echo -n "[*] Aligning recompiled APK..."
$ZIPALIGN -v 4 $unaligned_apk $compiled_apk >>$LOG_FILE 2>&1
rc=$?
echo "done."
if [ $rc != 0 ]; then
echo "[!] Failed to align recompiled APK"
cleanup
exit $rc
fi
rm $unaligned_apk
exit 0

hi, i ecountered the same issue as well, but when I look at the run.log, the line says:
rm: cannot remove 'signing.keystore': No such file or directory.
So I think that's the place keep raising the error.

yes plz post a new  isssue ,
so that they can help us

Run the following via the command line on your Kali Linux instance inside the git cloned directory for backdoor-apk:
shell> git remote show origin
Please share the output.

So, I noticed this in the run.log file you shared above:
Exception in thread "main" brut.androlib.AndrolibException: brut.androlib.AndrolibException: brut.common.BrutException: could not exec: [/tmp/brut_util_Jar_3839462608819321650.tmp, p, --forced-package-id, 127, --min-sdk-version, 7, --target-sdk-version, 18, --version-code, 674, --version-name, 2.2.9, -F, /tmp/APKTOOL7858375429182164987.tmp, -0, arsc, -0, arsc, -I, /root/apktool/framework/1.apk, -S, /root/backdoor-apk/backdoor-apk/original/res, -M, /root/backdoor-apk/backdoor-apk/original/AndroidManifest.xml]
        at brut.androlib.Androlib.buildResourcesFull(Androlib.java:437)
        at brut.androlib.Androlib.buildResources(Androlib.java:371)
        at brut.androlib.Androlib.build(Androlib.java:281)
        at brut.androlib.Androlib.build(Androlib.java:254)
        at brut.apktool.Main.cmdBuild(Main.java:224)
        at brut.apktool.Main.main(Main.java:84)
Caused by: brut.androlib.AndrolibException: brut.common.BrutException: could not exec: [/tmp/brut_util_Jar_3839462608819321650.tmp, p, --forced-package-id, 127, --min-sdk-version, 7, --target-sdk-version, 18, --version-code, 674, --version-name, 2.2.9, -F, /tmp/APKTOOL7858375429182164987.tmp, -0, arsc, -0, arsc, -I, /root/apktool/framework/1.apk, -S, /root/backdoor-apk/backdoor-apk/original/res, -M, /root/backdoor-apk/backdoor-apk/original/AndroidManifest.xml]
        at brut.androlib.res.AndrolibResources.aaptPackage(AndrolibResources.java:436)
        at brut.androlib.Androlib.buildResourcesFull(Androlib.java:423)
        ... 5 more
Caused by: brut.common.BrutException: could not exec: [/tmp/brut_util_Jar_3839462608819321650.tmp, p, --forced-package-id, 127, --min-sdk-version, 7, --target-sdk-version, 18, --version-code, 674, --version-name, 2.2.9, -F, /tmp/APKTOOL7858375429182164987.tmp, -0, arsc, -0, arsc, -I, /root/apktool/framework/1.apk, -S, /root/backdoor-apk/backdoor-apk/original/res, -M, /root/backdoor-apk/backdoor-apk/original/AndroidManifest.xml]
        at brut.util.OS.exec(OS.java:97)
        at brut.androlib.res.AndrolibResources.aaptPackage(AndrolibResources.java:430)
        ... 6 more
Caused by: java.io.IOException: Cannot run program "/tmp/brut_util_Jar_3839462608819321650.tmp": error=2, No such file or directory
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1048)
        at brut.util.OS.exec(OS.java:90)
        ... 7 more
Caused by: java.io.IOException: error=2, No such file or directory
        at java.lang.UNIXProcess.forkAndExec(Native Method)
        at java.lang.UNIXProcess.<init>(UNIXProcess.java:248)
        at java.lang.ProcessImpl.start(ProcessImpl.java:134)
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1029)
        ... 8 more
Forcing cleanup due to a failure or error state!

This is an error being thrown by apktool.

root@localhost:~/backdoor-apk/backdoor-apk# git remote show origin

remote origin
Fetch URL: https://github.com/dana-at-cp/backdoor-apk
Push  URL: https://github.com/dana-at-cp/backdoor-apk
HEAD branch: master
Remote branch:
master tracked
Local branch configured for 'git pull':
master merges with remote master
Local ref configured for 'git push':
master pushes to master (up to date)

Try running the cleanup.sh script before running the backdoor-apk.sh script.

This seems to be a duplicate of another reported issue: #30
Closing as duplicate.


⚠ ClamAV has degraded performance

In 2132517, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 62 ms

Resolved: ClamAV performance has improved in 0901f44 after 16 minutes.


Wrap requests exceptions

📝 Description of the feature
The package might raise requests.RequestException in some places, forcing clients to handle it directly. It would be best if openapi-common returned custom exceptions
💡 Steps for implementing the feature
No response
🔗 Useful links and references
No response

Deferring to a future release.


Does Iban work with Entity Framework

I read in the documentation that Iban class can be used in your domain. I had a few questions about this:

How does the Iban get stored in DB? as a string or as an reference?
Does EF automatically convert it to an instance of Iban?
How do I create an Iban object from a string?

If anyone could point me to the right part of the documentation that would be extremely helpful.
Kind regards,
Nathan T.

You can use the Iban type with entity framework in combination with a value converter.
https://learn.microsoft.com/en-us/ef/core/modeling/value-conversions?tabs=data-annotations
public class MyDbContext : DbContext
{
    public MyDbContext(IIbanParser _ibanParser)
    {
    }

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder
            .Entity<Account>()
            .Property(e => e.AccountNumber)
            .HasConversion(
                iban => iban.ToString(IbanFormat.Electronic),
                ibanStr => _ibanParser.Parse(ibanStr);
    }
}

I wanted to have a List<Iban> and that translates to a List<string> so then I need to create a wrapper class Iban to store an Iban 😞. So I just made my own Iban class with a string and when needed I parse it to an Iban to stringify it.
I will look into replacing the string in my own Iban implementation with the Iban with the value converter.
Thanks for the extra info.


too many request error

If the number of small-sized videos is high, Vimeo API gives the error "You have made too many requests" and stops downloading.

hey @sinan-aydogan, yes they have throttling on their end (you'll get 429 exception) you can't hit them as much as you want, probably adding proxies can solve this issue

What if we had the option to add a delay between downloads or set a limit of downloading a maximum of 20 videos in an hour?
Example:
laradown --dowload-limit=20 --limit-type=hour

I have already added a timeout or delay so you don't hit 429 but it is not exposed as parameter...


DefaultLocalPersistenceService.recursiveDeleteDirectoryContent(File) always returns true

This method is used in DefaultLocalPersistenceService.destroyAll(), which then always records a "positive" action "Destroyed all file based persistence contexts"
Content of this method is:
  private static boolean recursiveDeleteDirectoryContent(File file) {
    File[] contents = file.listFiles();
    if (contents == null) {
      throw new IllegalArgumentException("File " + file.getAbsolutePath() + " is not a directory");
    } else {
      boolean deleteSuccessful = true;
      for (File f : contents) {
        deleteSuccessful |= tryRecursiveDelete(f);
      }
      return deleteSuccessful;
    }
  }

I guess probably a fix could be:
  private static boolean recursiveDeleteDirectoryContent(File file) {
    File[] contents = file.listFiles();
    if (contents == null) {
      throw new IllegalArgumentException("File " + file.getAbsolutePath() + " is not a directory");
    } if (contents.length == 0) {
      return true;
    } else {
      boolean deleteSuccessful = false;
      for (File f : contents) {
        deleteSuccessful |= tryRecursiveDelete(f);
      }
      return deleteSuccessful;
    }
  }

Let me know if you are ok with this fix - I can send a PR.

A better fix would be:
private static boolean recursiveDeleteDirectoryContent(File file) {
  File[] contents = file.listFiles();
  if (contents == null) {
    throw new IllegalArgumentException("File " + file.getAbsolutePath() + " is not a directory");
  } else {
    boolean deleteSuccessful = true;
    for (File f : contents) {
      deleteSuccessful &= tryRecursiveDelete(f);
    }
    return deleteSuccessful;
  }
}

ok! Returns true only if all succeeded.

Re-opening as this needs to be fixed on release/3.0 as well.


How to show / hide toggle particular bar in group bars case?

Thanks to entire team for developing this beautiful and robust chart package. It is very easy to use. Just a request, kindly keep the related document and examples updated.
I have created a grouped bar chart. You can see it's output in attached screenshot. My query is that I did not find any feature which can be used to show/hide particular typed bar in grouped bars case. eg. As per attached screenshot, I want to show/hide all the bars of Male or Female when user clicks on 'Male' or 'Female' legends text.  Do we have such in-built feature or will you need to develop it? There should be a feature of show: true/false for each bar so that we can toggle them by clicking on external button / legend text.
Kindly suggest me how can we do this? Thanks a lot.

Hi. it should be handled outside of the chart functionalities.
I just wrote a sample, check out the code here.


The preprocessor extract_trajectory doesn't create coordinate bounds, and confuses lat/lon coordinates

The preprocessor extract_trajectory is built to interpolate the model data between two points at a prescribed number of points.
This means that it creates a new set of coordinates between those two points. However, the new coordinates do not include the coordinate attribute, bounds.
ie:
The preprocessor
  prep_transect:
    extract_trajectory:
      latitudes: [0., 0. ]
      longitudes: [-5.,  5.]
      number_points: 25

Produces the follow latitude
print(cube.cooord('latitude')
AuxCoord(masked_array(data=[-5.1491704 , -4.4021463 , -4.044439  , -3.6941166 ,
                   -3.3493495 , -3.0086572 , -2.3350284 , -2.0004609 ,
                   -1.6666366 , -1.3331935 , -0.9999002 , -0.33332792,
                    0.        ,  0.33332792,  0.9999002 ,  1.3331935 ,
                    1.6666366 ,  2.0004609 ,  2.3350284 ,  3.0086572 ,
                    3.3493495 ,  3.6941166 ,  4.044439  ,  4.4021463 ,
                    5.1491704 ],
             mask=False,
       fill_value=1e+20,
            dtype=float32)

However, you'll also notice that it actually switches latitudes and longitudes:
https://docs.esmvaltool.org/projects/esmvalcore/en/latest/_modules/esmvalcore/preprocessor/_volume.html#extract_trajectory
        longitudes = np.linspace(minlat, maxlat, num=number_points)
        latitudes = np.linspace(minlon, maxlon, num=number_points)

should be:
        longitudes = np.linspace(minlon, maxlon, num=number_points)
        latitudes = np.linspace(minlat, maxlat, num=number_points)

Whoops!

This should probably be an Core issue, rather than a tool one. I've made this issue again here:
https://github.com/ESMValGroup/ESMValCore/issues/1137

Just as a hint for next time: There is a "-> Transfer issue" link in the side bar on the right that allows you to move issues between repos.


ADD Another reason to why care

Another reason to care is the fallout on the rest of the OSS community as noted in  Massdrivers blog post
CNCF is spending dev cycles removing/replacing Hashi tools. This will impact those CNCF tools features/fixes as time is spent on swapping tools.
Also , CNCF as users of those tools in dev workflows are less likely to commit code to Hashi tools if they don’t use them. Hashi’s change will lead to a vicious cycle of devs ceasing to commit on Hashi tools. Meaning Hashi tool end users will see less and less features and fixes.


Develop

This PR adds error checking to the namelist file read and makes a few other minor changes.  The new namelist is shorter and has a few different labels:  'input_file' is changed to 'forcing_file', one group is removed and one has a name changed.
The error checking finds namelist entries that are not recognized and also missing required namelist entries.  It introduces integerMissing, realMissing, and stringMissing variables that can be used more broadly in the code if needed.
The code was tested against the original develop branch and an NCDIFF on the output showed no differences.   The unit test was run and passed.  The new functionality was tested for missing int, real and string entries.
Lastly, this includes a small fix after it was noticed that the code to handling a run where the user does not supply a namelist did not work.

@andywood It looks good. I just want to make sure that by removing the structure_option, there is no optional keys in the namelists, all keys are mandatory.

It looks like with 'crop_type_option' removed, 'crop_type' could also be removed, and also the Namelist in the run directory should be updated to remove both.   I just resolved conflicts from the develop branch that had my namelist checking, but this was at the same time as some other activity so they may not have stuck.  I'll wait until other activity settles and look at it again.

Andy, I just resolved some conflicts with the base branch and updated the unit testing files. I've committed my updates. You can go ahead and commit your changes now.

Thanks Zhengtao.
I think we've decided that all keys are mandatory.  I haven't checked to
see if there's any key that might never be checked in a model run, but if
there were one, we could change to allow that to be missing.
We do have at least one option that isn't strictly necessary (crop_option)
-- it's left in for now because we might want to add the crop code back in
at some point.
The structure_option seemed unnecessary because it just set up a special
case that can be easily handled by the user using normal entries.
But I wanted to make sure that it had not been added because it was needed
by the framework in some way.
Cheers,
Andy
On Fri, Aug 12, 2022 at 11:04 AM zhengtaocui @.***>
wrote:

@andywood https://github.com/andywood It looks good. I just want to
make sure that by removing the structure_option, there is no optional keys
in the namelists, all keys are mandatory.
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/noah-owp-modular/pull/64#issuecomment-1213331224,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIKAROCKEPUVVTNCNFCFL3VYZ7ZNANCNFSM56JP57QA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Issue #57 - expose work related methods on fflib_ISObjectUnitOfWork

Per discussion in #57 leaving IDoWork in concrete class and exposing in fflib_ISObjectUnitOfWork

Thanks for closing this one for us @jondavis9898 :+1:


Issue with selecting additional port 53 during setup

First off, loving this tool!
During a recent setup I opted to use port 53 in addition to 1194 :
Do you want OpenVPN to be available at port 53 too?
This can be useful to connect under restrictive networks
Listen at port 53 [y/n]: y

When I downloaded the .ovpn profile I wasn't able to connect, so I took a look at open ports on the server
root@crain:~# netstat -ltnup
Proto Recv-Q Send-Q Local Address   Foreign Address    State   PID/Program name
shh-ports blah blah blah ...
udp    0   0 <IP_ADDRESS>:53    <IP_ADDRESS>:*      922/openvpn

It looks like port 1194 isn't attached to anything, but I took a look from my computer just to make sure.
echo -e "\x38\x01\x00\x00\x00\x00\x00\x00\x00" | nc -w 10 -u server.ip 1194 | cat -v

And no response came back, 'pinging' openvpn it should output something like @??e?tg]T^@^@^@^@^@@.....
I'd love to have both ports working, is there anything else I can do to help debug this issue?
Here's my test system info
root@crain:~# lsb_release -a
Distributor ID:	Ubuntu
Description:	Ubuntu 14.04.2 LTS
Release:	14.04
Codename:	trusty

Thanks again!

You are right, this was a regression introduced with the latest commit which has now been reverted.
Thank you for reporting this.


ioregs!() should not use morestack

While building https://github.com/hackndev/zinc/pull/136 I've noticed that the test app doesn't build with -O0 as __morestack is undefined for generated ioreg code. There is nothing in ioreg that would need to use stack, I suppose, so #[no_split_stack] should be dropped into prologues of generated functions.
Alternatively just wait until freestanding target is WAI.

Confirmed that this is no longer happening as of e68f861. Closing.


The commands on the channels are not executed

Good Morning
I will surely be missing some configuration that I cannot find.
I have created a command that simply returns to the channel from which any message was issued, it is only to see if it works.
From the bot's own channel it works correctly, but from a channel where the bot is an administrator it doesn't work.
I have validated that telegram sends the messages, but they do not reach the code of my command.
Any idea what it could be?
Thank you

This is correct behaviour as it is now.
Messages that come in from a channel get handled by the ChannelpostCommand or then the GenericCommand.
I can help more if you can share the part of your code that handles the message :blush:

Good Morning
Thanks for the reply.
It didn't work because as you said it wasn't implementing the ChannelpostCommand class.
I have seen that in the groups managed by bots a button with the enabled commands may appear, can it be done for the channels, if so, how should it be done?
Thank you

You can use the PostchannelCommand for this.
From there you can execute the command that would be called.
Something like this:
$this->getTelegram()->executeCommand($this->getChannelPost()->getCommand());

Be aware that you'll need to adjust your commands, as $this->getMessage() is null when it's called from a channel.
Best use:
$message = $this->getMessage() ?: $this->getChannelPost();


Add next-optimized-images

This package tries to resolve a different problem than next-images does: It doesn't just make images importable, it does also automatically optimize them (using img-loader) because images are often too big or in a too high quality when you download them or get them from a designer. There is also an option to convert images to WebP on the fly.

Cool!


Polyfill updates and maintenance

React Boilerplate
In this PR:

Following @babel/polyfill's deprecation, this enables react-app-polyfill exclusively and removes duplicate polyfills. It contains core-js/stable (v3) and regenerator-runtime plus additional polyfills for compatibility all the way back to IE9. It can be updated very easily for IE11 support (instead of 9) or simply no IE support.
Dependency updates!
Removes unnecessary polyfill from Jest setup.

PS: Ignore the Codacy check. It's just a disagreement on Markdown formatting and I disabled the integration anyway so it shouldn't appear here.

Coverage increased (+0.006%) to 99.675% when pulling e59825371b9e0c9cb4f58deaa8acf144f9a19fb3 on maintenance into fd58cd23c9186bd49d7e8f3aac7536f1f52a9e19 on dev.

Thanks @jwinn!


Option to only use gray-scale

Hi,
In the look of making things more bandwidth-efficient I've been looking around on how to only use gray-scale in this product, but to no avail. Just taking a still picture and opening it in a picture editor and making it gray-scale makes the data about a 1/3 of the size. For me resolution would be more beneficial over color (using a no-ir camera so the colors are skewed anyway). So my request is to have a "pure gray-scale" option somewhere.
Thanks for an otherwise good project :).

Just to clarify where are you aiming to save bandwidth? In the live preview or in viewing the captures? These use separate paths through the camera interface.

Sorry, both would be interesting but live view would be my primary wish. Just used the still picture mode to get some view on how much file size differs when going grayscale (makes sense that it's 1/3 as 3 color channels goes down to one).

This is a good idea. Yes it would reduce bandwidth unfortunately I don't think raspimjpeg supports this. You could obviously set saturation to -100 in the GUI as a simulation of this effect.

I have played around with this a bit but I don't think it is giving much reduction in bandwidth.
The main reason I think is that the colour encoding is actually very efficient in jpeg and mp4. It is not a simple 3 times reduction. The YUV scheme means that all the detail is in the Y luminance channel and the UV chrominance channels are at /2 resolution (/4 data) so that total raw data is maybe 1.5 times and the compression may reduce this factor further.
I did manage to get about a 15% reduction by gray scaling.
You may be able to get a bit more by converting to gray as suggested using by enabling colour effects U:V = 128 and lowering the preview quality a little bit. The gray scale nature may allow a slightly lower quality to be used.


Extended LoadBalance ran 2 times during service call

[x] I have searched the issues of this repository and believe that this is not a duplicate.
[x] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.8
Operating System version: ANY
Java version: 8

Steps to reproduce this issue

Use 2 Registry (zookeeper://<IP_ADDRESS>:2181、zookeeper://<IP_ADDRESS>:2182)
Run 2 Providers.
Implement a loadbalance with SPI (xxx=XxxLoadBalance)
Run a consumer use xxx loadbalance(<dubbo:reference id="demoService" check="false" interface="org.apache.dubbo.demo.DemoService" loadbalance="xxx"/>), and XxxLoadBalance will be called twice. The first time, select a registry, the second time, select a provider.

Here should use random loadbalance only,  when select a registry。

Reproduce demo : https://github.com/zhangyz-hd/dubbo-issues/tree/main/6481
Recreated a PR：　#7192


Sample not working

Hi,
Please, I'm trying to use this sample project but, unfortunately, it's not working.
I just started the project using my bluemix account. I didn't changed the project name, as instructed. I can see the input screen, but when I clicked at "Score Now" button, nothing happen.
I'm using Firefox and Safari in a MacOS.
Thank you.

+1... sample is not working...

Make sure that you have uploaded models from model subdirectory (this git repo) using PA/ML dashboard.


Vouchers

Coverage increased (+0.01%) to 96.93% when pulling 54a58c3ec6f184c863c486ff57e903065e628193 on vouchers into d61dc838c5596b7c4bc05ee944d1e7b2831212f7 on master.


Added OTP register, login, and activate methods.

This PR:

Replaces the autofill presentation anchor in PassageAutofillAuthorizationController (apps crash without it)
Adds OTP register, login, and activate methods to PassageAPIClient
Adds OTP register, login, and activate methods to PassageAuth

I also know we have a generic "login" function on PassageAuth.swift defined as public static func login(identifier: String) that will do a magic link if less than ios 16, which isn't going to work anymore because that would fail if their app is configured to use OTP

Thanks for the callout! I'll be handling that in my next PR, which will reference this ticket. Trying to keep the PRs relatively small, rather than have a single big feature PR. 😊


Prepare cost model for Nucleus - Considering Amazon Managed Airflow (Amazon MWAA)

It was decided to consider the Amazon Managed Airflow (Amazon MWAA) for short term for Nucleus and later consider a more flexible approach such and self managed Airflow in future. This task is to prepare the cost model document considering Amazon MWAA.

Created the draft cost mode (to be reviewed and improved).

DRAFT Nucleus Cost Model
https://docs.google.com/spreadsheets/d/1sLobcbMgigQ0mjNesU-qDQUOj52RWETG0m-V1nUitSY/edit?usp=sharing

this is done for now. to review the trade study, but the cost model is sufficient for the time being

@ramesh-maddegoda to verify when it comes to Nucleus, when we want to execute something like validate on a data products using some files on a file system, do we need to pay for the EC2 or whatever other services the workflow task needs to execute? or is that included in Airflow?

@ramesh-maddegoda to verify when it comes to Nucleus, when we want to execute something like validate on a data products using some files on a file system, do we need to pay for the EC2 or whatever other services the workflow task needs to execute? or is that included in Airflow?

@jordanpadams , that cost is covered as the cost for "Amazon Elastic Container Service (Amazon ECS) with Fargate Launch Type". Because the actual PDS docker containers will be executed as ECS tasks. Airflow will delegate the load to ECS tasks. This is the "Task" related costs shown in line 18 to 20 in cost model .
The details of costs for each PDS Docker container (ECS task) is calculated from line 32 to 40 in cost model

thanks @ramesh-maddegoda sorry for my lack of understanding here. thanks for taking the time to educate me :-)


[3/3 dedupe] Remove restart modal dupe

One #nit, but other than that it looks great!

Tried to rebase for you, but it was a little more involved. Could you resolve the conflicts on this guy?

@rcorral Flicts

Tried it out again, works well :) :shipit: Merge, once tests pass


[DO NOT MERGE] (maint) Sign packages with new signing key

This commit updates the key used to signed packages with the automation
in the packaging repo. We are shipping a new gpg key for all publicly
available packages, so this is a part of that effort.

Currently blocked on getting some releases out the door, and then we'll go and bulk merge everything. I'm not sure which branches are currently in development, so I did everything that could potentially have a release in the future. If for some reason a release is cut from a branch that does not have these changes, we will see failures.

:+1:


add maplist/2..8

Codecov Report

Merging #184 (ea4b205) into main (f202946) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #184   +/-   ##
=======================================
  Coverage   96.05%   96.05%
=======================================
  Files          20       20
  Lines        6208     6208
=======================================
  Hits         5963     5963
  Misses        204      204
  Partials       41       41

Continue to review full report at Codecov.

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