TreeNode problem

Faced with a strange problem. After I minimize my app window, the TreeNode will stop opening, although other elements (buttons, menu bar) continue to work.
Before I collapsed the window:

After:

Hello.
Can you please provide a minimal code example which reproduces the problem? Ideally it should reproduce on a window with only one TreeNode.
My first guess would be to check if this TreeNode has unique id, because widgets not responding is usually caused by label/id clash.

Main loop:
    while(window->isOpen())
    {
        sf::Event event;
        handleEvent(event);// ImGui::SFML::ProcessEvent(event);
        ImGui::SFML::Update(*window, deltaClock.restart());
        if(ImGui::Begin("Tools Window", NULL, window_flags))
        {...}
        update();
        draw();// ImGui::SFML::Render(*window);
    }

How I use TreeNode:
  for(auto& group : groups)
  {
    ImGui::PushID(group_id);
    node_open = ImGui::TreeNodeEx((void*) (intptr_t) group_id, nd, "%s", group->getName().c_str());
    if(node_open)
    {
     for(auto& elem:group)
     {
      ImGui::PushID(elem_id);
      ImGui::Text(...)
      ImGui::PopID();
     }
     ImGui::TreePop();
    }else{...}
    ImGui::PopID();
  }

Absolutely all TreeNode stop working, even in TestWindow, which provides ImGui. I also don't handle the LostFocus and GainedFocus states in any way, could this be an error?
I started the window with only one TreeNode, the problem remained:
  while(window->isOpen())
  {
       sf::Event event;
       handleEvent(event);// ImGui::SFML::ProcessEvent(event);
       ImGui::SFML::Update(*window, deltaClock.restart());
       if(ImGui::TreeNode("adsfjklhasdkufj"))
       {
          ImGui::Text("asdasd");
          ImGui::TreePop();
       }
       draw();// ImGui::SFML::Render(*window);
  }

When I launch a window without focusing on it, absolutely all ImGui elements stop working.

You don't need to handle it yourself, but SFML should send Gained/LostFocus events. Does it do it? Can you check it, please? Maybe it's the problem with your WM which SFML doesn't work correctly with.
I think that ideally I should remove this Gained/Lost event processing stuff, but I'm afraid most people will get confused that ImGui elements would work even if your window is not in focus (e.g. if you click on the text box, you'll be able to write in it even if the window loses focus)

Can you please remove all s_windowHasFocus usage in imgui-SFML.cpp and check how it works for you, please? I feel like we can do fine without this bool, but I'm not sure how ImGui will behave with out-of-focus window without it.

Completely removed s_windowhasfocus now all imgui elements work even if you run the application without focusing on it (but problem in sfml because for some reason, the gainedfocus event is not created when i open the application window for the first time), problem with TreeNode remains.
I don't think TreeNode bug in my case is related to events. After minimizing the window in the ButtonBehavior function g.ActiveId set to zero and the code for checking the mouse click is not executed, but I do not know what this is related to.
I compile the project using g++ 10, tried to compile it using msvc and there the bug looks different. After minimizing the window, the TreeNode opens only by clicking on the arrow.

Does it reproduce when you insert the TreeNode code in the minimal example?

Yes, after minimizing the window, TreeNode do not open.
I tried using the new version of imgui 1.79 (before it was 1.73) and downloaded the latest release of imgui-sfml (i also used it before). The bug remained, but now after minimizing the TreeNode windows open when you click on the arrow.
When i run the application, treenode work perfectly no matter how many times i open them, but after minimizing the window, they open only when you click on the arrow. And if you press alt, just press, then they start working again. This bug does not occur if you just focus on another window with the mouse, but if you minimize the window using Alt+Tab, this bug appears. Even if I focus on another window with the mouse, and return to the window of my application using Alt+Tab everything will be fine.

Please try latest version of ImGui-SFML with ImGui 1.82
I'm using this code for testing and it works well:
#include <SFML/Graphics/CircleShape.hpp>
#include <SFML/Graphics/RenderWindow.hpp>
#include <SFML/System/Clock.hpp>
#include <SFML/Window/Event.hpp>

#include "imgui-SFML.h"
#include "imgui.h"

int main() {
    sf::RenderWindow window(sf::VideoMode(640, 480), "ImGui + SFML = <3");
    window.setFramerateLimit(60);
    ImGui::SFML::Init(window);

    sf::CircleShape shape(100.f);
    shape.setFillColor(sf::Color::Green);

    sf::Clock deltaClock;
    while (window.isOpen()) {
        sf::Event event;
        while (window.pollEvent(event)) {
            ImGui::SFML::ProcessEvent(event);

            if (event.type == sf::Event::Closed) {
                window.close();
            }
        }

        ImGui::SFML::Update(window, deltaClock.restart());

        ImGui::Begin("Hello, world!");
        ImGui::Button("Look at this pretty button");
        if (ImGui::TreeNode("adsfjklhasdkufj")) {
            ImGui::Text("asdasd");
            ImGui::TreePop();
        }
        ImGui::End();

        ImGui::ShowDemoWindow();

        window.clear();
        window.draw(shape);
        ImGui::SFML::Render(window);
        window.display();
    }

    ImGui::SFML::Shutdown();

    return 0;
}

Personally, I still have a problem :). As I understand it, after minimizing the window (alt+tab), an event about the alt key being pressed comes from sfml, but the event that alt was released does not come. And imgui thinks that the alt key is pressed and the interface works differently, for example, treenode will open only when you click on the arrow. Maybe I have some problem with my system.

Oh, so it looks like it's related #88... Okay, I see, I'll try to fix that one first.
You said "minimized", so I clicked on "_" button and didn't think that you mean alt-tabbing.

Okay, I can confirm that Alt+Tab breaks things. When the window is out of focus, SFML doesn't catch any key input events, so the only way to fix it is to poll keyboard events.
Doing this fixes the problem, but introduces some other ones... I'm working on it!

Fixed. Should work now. The problem is described in the commit message.


Create the basic menu outiline of the game

Implement the following menus:
Main menu.
1) Play a new game
2) Load already saved game
3) Exit the game

Save system menu.
1) List all saved games
2) List all saved games for a particular player
3) Show the board of one of the saved games
4) Load a game
5) Return to main menu

Resolved with 33787b9bd4659206f63a11f216f048ce060c3413.


Copy Paste Gadget's settings menu becomes blurry and unusable.

Building Gadgets Version
buildinggadgets2-1.3.7
Describe the bug
When I open the settings menu of the Copy Paste Gadget, the whole screen becomes blurry and none of the buttons can be used.
To Reproduce
I'm playing on ATM10 version 1.1.6. To perform the bug, just open the general settings menu of the copy paste gadget and then press the "settings menu" button with the pencil icon at the bottom right.
Expected behavior
I expect the screen not to blur and the settings to be able to be used properly.
Screenshots
just press this button

and here it is

temporary solution, you can disable Menu Background Blur in the Accessibility Settings.
this solves this issue, and also the same in renaming waystones and such

This should be resolved in the next version, thanks!


go.mod: go 1.18

go.mod's go version should match the oldest entry in github workflow.

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

Comparison is base (85a15a7) 46.59% compared to head (cffe94b) 46.52%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1141      +/-   ##
==========================================
- Coverage   46.59%   46.52%   -0.07%
==========================================
  Files          12       12
  Lines        1526     1526
==========================================
- Hits          711      710       -1
- Misses        672      673       +1
  Partials      143      143

see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@mattn any concerns?

It's generally a good practice to ensure that the version of Go specified in the go.mod file matches the oldest version of Go that's supported by your GitHub workflow. This can help ensure compatibility and prevent potential issues from arising due to version mismatches. It's also important to keep both the go.mod file and the GitHub workflow up-to-date as new versions of Go are released, to ensure that your code remains compatible with the latest tools and libraries.

The statement you provided appears to be a guideline for maintaining consistency between the version of Go specified in a project's go.mod file and the version of Go used in the project's GitHub workflow configuration.
This guideline suggests that the go version specified in go.mod should match the oldest version of Go used in the project's GitHub workflow. This helps ensure that the project's dependencies are compatible and that the code can be compiled and tested using the specified version of Go.
Maintaining consistency between the go.mod file and the GitHub workflow configuration is important for ensuring the reliability and reproducibility of a project's builds and tests.

It seems like you have provided a guideline for maintaining consistency between the version of Go specified in a project's go.mod file and the version of Go used in the project's GitHub workflow configuration.
This guideline recommends that the go version specified in go.mod should match the oldest version of Go used in the project's GitHub workflow. By doing so, the project's dependencies are kept compatible and the code can be compiled and tested using the specified version of Go.
This is an important step for ensuring that a project's builds and tests are reliable and reproducible. Consistency between the go.mod file and the GitHub workflow configuration helps avoid potential compatibility issues that can arise from using different versions of Go in different parts of the project.

The right fix is not to upgrade the version in go.mod but instead to extend the coverage with older versions of Go in the CI.


setup: add setup.py for the ConvLab package

There's currently no setup.py available for the package. This would be nice instead of augmenting the PYTHONPATH all the time e.g. $ PYTHONPATH="../../../:${PYTHONPATH}" python my_script.py

@aatkinson  Thanks for adding setup.py, Adam!


Webb-Solidity package structure

This PR changes the project structure to separate packages:

contracts: types of the latest solidity contracts
fixed-bridge: Wrapper-classes for anchors, bridges, etc. related to fixed-bridge.
tokens: Wrapper-classes for token functionality
utils: Common utilities e.g. toFixedHex and others.
vbridge: UNIMPLEMENTED, wrapper-classes for anchors, bridges, etc. related to variable bridges.

This PR brings in a dependency on lerna for building and publishing the packages.
This PR updates the root-directory package.json to include:

yarn build:packages - needs to be run before tests.
yarn compile - should be run instead of npx hardhat compile as it will run a script to update the src files in the contracts directory of the latest typechain output.

Awesome work, first glance is # of files / lines changed is huge. Maybe we're committing something we shouldn't be. Phone isn't loading commits easily..


Completion is broken for double backticked names

They did not appear at all. After fixing it, they do appear (without backticks, which is OK I think):

back  does not add backtickes:

Same for TPs:

These screenshots are on this PR https://github.com/Microsoft/visualfsharp/pull/2212

@dsyme @vladima could you comment on this? How it worked in pre-Roslyn era?

Any why space is among characters which allowed in operator here https://github.com/Microsoft/visualfsharp/blob/ed948abaec8e2723c4fcafbde969a1071a4d72b1/src/fsharp/PrettyNaming.fs#L117?

I think we should use this http://source.roslyn.io/#Microsoft.CodeAnalysis.Features/Completion/CompletionItemRules.cs,54 to amend backticks-free long ident on commit (). Or maybe not. Ideas?

No, the rules are for completely different stuff. I'm out of ideas.
@Pilchie help. What we basically need is to show identifiers like
``long ident``

without the backticks in the completion list, but insert them with backticks into document when completion is committed. I cannot find anything useful in CompletionItem type.

Oh, it seems I've found the right place http://source.roslyn.io/#Microsoft.CodeAnalysis.Features/Completion/CompletionProvider.cs,54

@cyrusnajmabadi and/or @rchande can answer the question above if you still need it.

Your scenario is totally supported.  Basically, you can store whatever data you want on a CompletionItem (like the text you'd like to insert).  Then you implement CompletionService.GetChangeAsync to grab and insert that text if it is available.

Fixed in https://github.com/Microsoft/visualfsharp/pull/2212

@vasily-kirichenko if this is fixed can it be closed? :)


CI for Node

Contains the compiled JavaScript (4 ms)
Initializes with two players (2 ms)
Starts the game with a random player
Sets the current player to be whoever it is not (1 ms)


TypeScript conflict with Webpack 5

When using a TypeScript syntax webpack.config.ts config with webpack-merge 5.2.0 and Webpack 5, the following TypeScript error is generated when trying to use merge(config, {})
TS2345: Argument of type 'Configuration' is not assignable to parameter of type 'Configuration | Configuration[]'.
  Type 'import("node_modules/webpack/types").Configuration' is not assignable to type 'import("node_modules/@types/webpack/index").Configuration'.
    Types of property 'entry' are incompatible.
      Type 'string | (() => string | EntryObject | [string, ...string[]] | Promise<EntryStatic>) | EntryObject | [string, ...string[]] | undefined' is not assignable to type 'string | string[] | Entry | EntryFunc | undefined'.
        Type '() => string | EntryObject | [string, ...string[]] | Promise<EntryStatic>' is not assignable to type 'string | string[] | Entry | EntryFunc | undefined'.
          Type '() => string | EntryObject | [string, ...string[]] | Promise<EntryStatic>' is not assignable to type 'EntryFunc'.
            Type 'string | EntryObject | [string, ...string[]] | Promise<EntryStatic>' is not assignable to type 'string | string[] | Entry | Promise<string | string[] | Entry>'.
              Type 'EntryObject' is not assignable to type 'string | string[] | Entry | Promise<string | string[] | Entry>'.
                Type 'EntryObject' is not assignable to type 'string'.

This is because webpack-merge relies on @types/webpack which is for Webpack 4. Webpack 5 now includes its own types and does not rely on the external typings.
Webpack 4's entry type looks like:
interface Configuration {
  entry?: string | string[] | Entry | EntryFunc;
}

Webpack 5's entry type looks like:
type Entry =
  | string
  | (() => string | EntryObject | [string, ...string[]] | Promise<EntryStatic>)
  | EntryObject
  | [string, ...string[]];

Yeah, that's a great point. I could make it so that Configuration is passed as a generic as doing this would decouple the utility from webpack types. Doing this would also solve another issue that's open.
I wonder if that should be a minor or major release, though, as it's breaking the type definition. Any thoughts on this? I guess a major would be a safer bet.

On 28.10.2020, at 22:20, msheakoski<EMAIL_ADDRESS>wrote:

When using a TypeScript syntax webpack.config.ts config with webpack-merge 5.2.0 and Webpack 5, the following TypeScript error is generated when trying to use merge(config, {})
TS2345: Argument of type 'Configuration' is not assignable to parameter of type 'Configuration | Configuration[]'.
Type 'import("node_modules/webpack/types").Configuration' is not assignable to type 'import("node_modules/@types/webpack/index").Configuration'.
Types of property 'entry' are incompatible.
Type 'string | (() => string | EntryObject | [string, ...string[]] | Promise) | EntryObject | [string, ...string[]] | undefined' is not assignable to type 'string | string[] | Entry | EntryFunc | undefined'.
Type '() => string | EntryObject | [string, ...string[]] | Promise' is not assignable to type 'string | string[] | Entry | EntryFunc | undefined'.
Type '() => string | EntryObject | [string, ...string[]] | Promise' is not assignable to type 'EntryFunc'.
Type 'string | EntryObject | [string, ...string[]] | Promise' is not assignable to type 'string | string[] | Entry | Promise<string | string[] | Entry>'.
Type 'EntryObject' is not assignable to type 'string | string[] | Entry | Promise<string | string[] | Entry>'.
Type 'EntryObject' is not assignable to type 'string'.
This is because webpack-merge relies on @types/webpack which is for Webpack 4. Webpack 5 now includes its own types and does not rely on the external typings.
Webpack 4's entry type looks like:
interface Configuration {
entry?: string | string[] | Entry | EntryFunc;
}
Webpack 5's entry type looks like:
type Entry =
| string
| (() => string | EntryObject | [string, ...string[]] | Promise)
| EntryObject
| [string, ...string[]];
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

You might be able to do it as a minor release if you defaulted the generic to the Configuration from Webpack 4. This would still allow for overriding with the Webpack 5 Configuration.
Something like:
import { Configuration } from "webpack"; // from @types/webpack
declare function merge<T = Configuration>(firstConfiguration: T | T[], ...configurations: T[]): T;

If you decide to decouple the types like moving them to a peerDependency or removing them entirely from the package, I would consider that a major release because it would break builds (especially for those dependency updating bots) until manually adding the generic.

I did a little more experimenting and think you might be able to get away with removing the dependency on @types/webpack while remaining backwards compatible. The only side effect is that it loosens up the accepted types for firstConfiguration and configurations. Anything object-like will work but scalars will trigger an error.
declare function merge<T extends object>(firstConfiguration: T | T[], ...configurations: T[]): T;

Yeah, I think your proposal is the way to go. It's better to loosen the type a little than to break as that still allows a minor release. I'll make sure to document the caveat when implementing. Thanks!


Update ProgressBar.cs

Added Selection Base
So click on scene selects Progress Bar Object instead of its children
My little contribution

Merged the PR. Thanks!


Fixed "pragma-patch.diff" on MacOS Ventura 13.1 / Xcode 14.2 (#925)

Work-around the problems that are most probably caused by the use of Windows new-line characters in the patch file.

Hi @openvmp ,
Thanks for tracking down the malformed patch issue. I was able to reproduce it and applying your diff helps to resolve it. However I encountered additional errors which I've documented here https://github.com/ros2/rviz/issues/943 and was able to resolve with https://github.com/ros2/rviz/pull/944.
Are you able to comment on whether you also encountered those issues and if so, does my PR resolve them?

Hi Yadunund,
Sorry, can't help there.
I was not able to get far enough in the build process to get to the other packages in question due to issues with Python installation on that workstation.
Thanks
Roman

Hi @openvmp,
I tested your update on macOS Sonoma (ARM) and it help to succesfully build rviz_ogre_vendor
Thanks!
K.


Recreate db after containers are up

When you execute 'make start' after running composer update it gives an error because the container aren't up.
docker-compose exec php sh -lc './bin/console d:d:d --force'
ERROR: No container found for php_1

This could be solved from 2 ways. The way i did it or just using 'docker-compose run' instead of 'docker-compose exec'  in the db command.

It totally make sense. Thanks for the fix


fix(query-documentation): typo range query

Description
Fix a typo in this documentation: https://quickwit.io/docs/get-started/query-language-intro
duration:[0 TO 1000] instead of duration:[0 TO 1000}

That's not necessarily a typo: [ is for inclusive, { is for exclusive. I'd be better to leave the examples as is and enhance the description.


Windows: If "Start app on login." is set to True, start the app automatically, but keep the window minimized in the taskbar until the user has set focus to the app

I confirm (by marking "x" in the [ ] below):

[X] This is not a feature idea. Feature ideas go here: http://www.mattermost.org/feature-ideas/
[X] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/
[X] This is a product defect issue that can be reproduced using the steps listed below.
[X] I have read CONTRIBUTING.md.

Summary
If "Start app on login." is set to True, start the app automatically, but keep the window minimized until the user has set focus to the app
Steps to reproduce
On

Windows 10,
Mattermost Desktop v1.3,
Mattermost Server v3.2

In Desktop Settings, set Start app on login. to True.
Restart your machine.

Expected behavior
App launches automatically, but remains minimized in the taskba until the user has set focus on the app.
Observed behavior
App launches automatically, and is focused on launch.
This means that if the user has opened another program while Mattermost desktop is launching, the desktop app will open on top. This can be annoying and interrupt user's work.
Possible fixes

Auto-starting was implemented with node-auto-launch. @Razzeee Would you check this?


Web Application Proxy: Publish an Application that uses HTTP Basic: Client User Certificates

In the image here it is suggested that it is possible to use client certificates:
https://github.com/MicrosoftDocs/windowsserverdocs/blob/6e8dae6474c6d9c1393f2387471325ceada4d7c7/WindowsServerDocs/remote/remote-access/web-application-proxy/Publishing-Applications-using-AD-FS-Preauthentication.md?plain=1#L256-L258
However when making client certificates required in ADFS, the WAP fails to authenticate the request, it doesn't seem to be possible to receive client certificates in this way:
Pre-authentication is 'ADFS For Rich Clients'. The relying party rules require additional authentication.
The additional rules of either the target relying party of the WAP relying party are needed to be modified.
Error:

Client Certificate Issuer: <Not Found>

Update: Perhaps the certificate in the diagram actually refers to a device registration certificate?

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Enhance Privacy by concealing API Key once set and allow for removal.

To enhance privacy, I have customized the APIPopup and APIMenu components by concealing the API Key field after it is set. While testing this customization, I'm encountering a warning while determining whether the field should be visible or hidden. As my expertise lies mainly in JavaScript and not TypeScript, I'm still searching for alternatives to mitigate this obstacle.
Additionally, I have added a "remove key" functionality to facilitate any future changes to the API key.
Once I have resolved all the issues, I will be happy to initiate a pull request.

I also need it.

I also need it.

my fork contains this, original maintainer never bothered to reject or approve my PR


TwammHook: Address math imprecision on single sell pools

Component
Pool Interaction, Hooks
Describe the suggested feature and problem it solves.
Because single-side sell TWAMM pools math is calculated slightly differently, math imprecision leads to selling a few wei more than what's available. Must address imprecision.
https://github.com/Uniswap/v4-periphery/blob/main/test/TWAMM.t.sol#L132
Describe the desired implementation.
No response
Describe alternatives.
No response
Additional context.
No response

Hey ewilz, one math imprecision in single sell pool is the amount to token to swap:

the current code ignore this parameter by using type(int256).max
poolManager.lock(abi.encode(key, IPoolManager.SwapParams(zeroForOne, type(int256).max, sqrtPriceLimitX96)));

the amountIn backout from sqrtPriceLimitX96 can be larger than the total amount to sell on the single side. this will lead the code to revert. we can cap this number by calculate the exact amount to be swapped.

I have developed this code and need a right to new branch for open PR.

closing this issue for now, since example hooks do not live in the main branch anymore.
i've added a revisit tag as a reminder

fixed in slightly different manner

Sorry, fixed ben's suggestion slightly differently. Still might be worth looking into the math imprecision because it is off by more than just some wei.


Namespace Change

Ran a few code snippets with the new Vonage namespace, seemed to work fine. Once the changelog is fixed, it's a :+1: from me


Update to Node20

Notice the CI runs still say node16? I think you have to recompile the dist.js

Oh I might be wrong. The warnings are mentioning the other actions in the CI.

Yeah it is quite likely the login action. That's why I added the dependabot configuration too. Let me update that one quickly inline here.

Even the latest login action still has that issue :(
https://github.com/Azure/login/issues/357


Question: forwarding selectively messages to clients from another broker

Hi,
this is a question not really an issue. Hope this is ok to place it here also for future reference
I have an use case where I want aedes to forward messages from another broker, selectively forwarding messages based on some client details.
I would like to expose aedes to internet with custom auth/authz and send to specific clients only the messages that they can receive.
I have already implemented it by filtering subscriptions topic based on pattern, but from an user perspective get complex as one has to know exactly what to subscribe for ahead of time eg.app/1 is ok but app/+ is not
Example use case:

topics are like app/[id]
client1 subscribe to # but can access only app/[id=1,2]
magic happens
client1 receive just those messages

What I did so far

authenticate() perform a login and fetch locally all the required details for a client and store it in the client object.
an mqtt client for broker1 and one for aedes that publish based on the details in the client.
authorizeForward() lookup in the cache by client id and decide if the message can be forwarded or not

I tried also to use aedes.publish({...}) or aedes.mq.emit({ ... }) but those seems not to work as I need (publish doesn't  show up in the auth* callback, emit just publish to all clients)
Is there a better approach in this case?
Thank you!

I am a bit lost, it’s not clear how you would like to achieve this.
I would also recommend that you should not violate the MQTT spec in your implementation.

When a msg from an internal broker is received it is published to aedes. On authorizeForward the client details (stored at login) are used to decide whether to allow or not the forward.
Do you see a violation of MQTT  spec in this approach?
Thank you

That would also be very slow, however it's probably the closest thing you can do to what you want to do.

I see, thank you


Dataset.with_format behaves inconsistently with documentation

Describe the bug
The actual behavior of the interface Dataset.with_format is inconsistent with the documentation.
https://huggingface.co/docs/datasets/use_with_pytorch#n-dimensional-arrays
https://huggingface.co/docs/datasets/v2.19.0/en/use_with_tensorflow#n-dimensional-arrays

If your dataset consists of N-dimensional arrays, you will see that by default they are considered as nested lists.
In particular, a PyTorch formatted dataset outputs nested lists instead of a single tensor.
A TensorFlow formatted dataset outputs a RaggedTensor instead of a single tensor.

But I get a single tensor by default, which is  inconsistent with the description.
Actually the current behavior seems more reasonable to me. Therefore, the document needs to be modified.
Steps to reproduce the bug
>>> from datasets import Dataset
>>> data = [[[1, 2],[3, 4]],[[5, 6],[7, 8]]]
>>> ds = Dataset.from_dict({"data": data})
>>> ds = ds.with_format("torch")
>>> ds[0]
{'data': tensor([[1, 2],
        [3, 4]])}
>>> ds = ds.with_format("tf")
>>> ds[0]
{'data': <tf.Tensor: shape=(2, 2), dtype=int64, numpy=
array([[1, 2],
       [3, 4]])>}

Expected behavior
>>> from datasets import Dataset
>>> data = [[[1, 2],[3, 4]],[[5, 6],[7, 8]]]
>>> ds = Dataset.from_dict({"data": data})
>>> ds = ds.with_format("torch")
>>> ds[0]
{'data': [tensor([1, 2]), tensor([3, 4])]}
>>> ds = ds.with_format("tf")
>>> ds[0]
{'data': <tf.RaggedTensor [[1, 2], [3, 4]]>}

Environment info
datasets==2.19.1
torch==2.1.0
tensorflow==2.13.1

Hi ! It seems the documentation was outdated in this paragraph
I fixed it here: https://github.com/huggingface/datasets/pull/6956

Fixed.


kernel crashes with ipywidget interact in demo nb

Hi!
Thank you for your work!
Trying the xeus-python kernel with binder using your project badge, I noticed a cell causing kernel repeated crashes:
from ipywidgets import interact
@interact
def foo(x = ['a', 'b'], n=(1, 10)):
    print(x * n)

I'm not able to help so it's just a simple report.

Thanks a lot for the report! I'll have a look at it :)

After some investigation, I found that the kernel dies in the xmock_kernel::parent_header method call. The xeus::get_interpreter().parent_header() seems to return a reference to an nl::json object that does not exist, because whenever you try to print this object or operate on it, the kernel silently dies...
@JohanMabille, I tried replacing the xeus::xkernel_core::parent_header method by:
const nl::json xkernel_core::parent_header(channel c) const noexcept
{
    return get_parent_header(c);
}

(I am basically returning a copy of the parent header instead of a reference to it)
And that fixed the issue. Do you have an idea why the current implementation could fail?

That is really weird. You don't have any message in the console where the kernel is launched?

No I've got nothing. It dies silently...

Does the following work?
inline py::object xmock_kernel::parent_header() const
{
    nl::json tmp = xeus::get_interpreter().parent_header();
    return py::dict(py::arg("header")=tmp.get<py::object>());
}

No it does not, that is what I don't understand. It's like the reference returned by parent_header is not valid

can you activate core dumps and see what you get?

I'll try

I could display the segfault:
Thread 1 "xpython" received signal SIGSEGV, Segmentation fault.
0x00007ffff7794a7e in nlohmann::basic_json<std::map, std::vector, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool, long, unsigned long, double, std::allocator, nlohmann::adl_serializer>::basic_json(nlohmann::basic_json<std::map, std::vector, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool, long, unsigned long, double, std::allocator, nlohmann::adl_serializer> const&) [clone .lto_priv.307] ()
   from /home/martin/miniconda3/envs/xeus-python/lib/libxeus-python.so.0

So we set the parent_header in the xkernel_core as being a reference to the message's parent_header. Is the message object still alive when we try to access the parent header? Maybe it's not

So we set the parent_header in the xkernel_core as being a reference to the message's parent_header.

The header is copied in  xkernel_core::m_parent, which is an std::array. xkernel_core::parent_header returns a constant reference on this object while xkernel_core::get_parent_header returns a copy. In both cases the original header must be a valid object.

This should be fixed with the latest release of xeus. Many thanks to @adriendelsalle for the report.
This was a dangling reference bug which has been around for a while but did not impact us before for some reason.


environments missing from admin center

hi, when I tried to access my environment from admin center, it says there are no environments. However I do have an environment but it is missing, can you please let me know how to fix?

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

ID: b6bf79b0-5e31-ebc4-2397-c7274967dd8f
Version Independent ID: 9b67d821-bfc9-eb04-c10d-31187c655984
Content: Manage environments in the Power Apps Admin center - Power Platform
Content Source: power-platform/admin/environments-administration.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@Karenzhang7717
Please see if you can get your question answered here:
https://powerusers.microsoft.com/t5/Power-Apps-Community/ct-p/PowerApps1
Or create a support request:
https://powerapps.microsoft.com/support/


Runtime error after update to Angular 5.2 (iOS)

After updating to angular 5.2 it's throwing the runtime error:
App doesn't crash buy navigation takes up to 30 seconds while processing this error.
Error happens when navigating through side drawer.
Please help - it's impossible to decode this error stack..
CONSOLE ERROR [native code]: ERROR Error: Invalid value: 
CONSOLE ERROR [native code]: ERROR CONTEXT {
"view": {
"def": {
"nodeFlags": 521650183,
"rootNodeFlags": 452984835,
"nodeMatchedQueries": 14,
"flags": 0,
"nodes": [
{
"nodeIndex": 0,
"parent": null,
"renderParent": null,
"bindingIndex": 0,
"outputIndex": 0,
"checkIndex": -1,
"flags": 402653184,
"childFlags": 0,
"directChildFlags": 0,
"childMatchedQueries": 0,
"ngContentIndex": -1,
"matchedQueries": {},
"matchedQueryIds": 0,
"references": {},
"childCount": 0,
"bindings": [],
"bindingFlags": 0,
"outputs": [],
"element": null,
"provider": null,
"text": null,
"query": {
"id": 1,
"filterId": 2,
"bindings": [
{
"propName": "drawerComponent",
"bindingType": 0
}
CONSOLE ERROR [native code]: ERROR Error: Invalid value: false
CONSOLE ERROR [native code]: ERROR CONTEXT {
"view": {
"def": {
"nodeFlags": 521650183,
"rootNodeFlags": 452984835,
"nodeMatchedQueries": 14,
"flags": 0,
"nodes": [
{
"nodeIndex": 0,
"parent": null,
"renderParent": null,
"bindingIndex": 0,
"outputIndex": 0,
"checkIndex": -1,
"flags": 402653184,
"childFlags": 0,
"directChildFlags": 0,
"childMatchedQueries": 0,
"ngContentIndex": -1,
"matchedQueries": {},
"matchedQueryIds": 0,
"references": {},
"childCount": 0,
"bindings": [],
"bindingFlags": 0,
"outputs": [],
"element": null,
"provider": null,
"text": null,
"query": {
"id": 1,
"filterId": 2,
"bindings": [
{
"propName": "drawerComponent",
"bindingType": 0
}

My package.json
"nativescript": {
    "id": "org.nativescript.ExchangeApp",
    "tns-android": {
      "version": "4.0.0"
    },
    "tns-ios": {
      "version": "4.0.1"
    }
  },
  "scripts": {
    "lint": "tslint \"app/**/*.ts\""
  },
  "dependencies": {
    "@angular/animations": "~5.2.0",
    "@angular/common": "~5.2.0",
    "@angular/compiler": "~5.2.0",
    "@angular/core": "~5.2.0",
    "@angular/forms": "~5.2.0",
    "@angular/http": "~5.2.0",
    "@angular/platform-browser": "~5.2.0",
    "@angular/platform-browser-dynamic": "~5.2.0",
    "@angular/router": "~5.2.0",
    "ccxt": "^1.12.137",
    "crypto-js": "^3.1.9-1",
    "email-validator": "^1.0.7",
    "nativescript-angular": "^5.3.0",
    "nativescript-async": "^1.0.2",
    "nativescript-iqkeyboardmanager": "^1.1.0",
    "nativescript-nodeify": "^0.7.0",
    "nativescript-numeric-keyboard": "^3.0.3",
    "nativescript-pro-ui": "^3.4.1-2018-2-26-1",
    "nativescript-theme-core": "1.0.4",
    "nativescript-webview-interface": "^1.4.2",
    "qs": "^6.5.1",
    "reflect-metadata": "0.1.10",
    "rxjs": "^5.5.5",
    "rxjs-tslint-rules": "^3.14.0",
    "tns-core-modules": "^4.0.0",
    "tns-platform-declarations": "^3.4.1",
    "zone.js": "^0.8.4"
  },
  "devDependencies": {
    "@angular/compiler-cli": "~5.2.0",
    "@ngtools/webpack": "~1.9.4",
    "@types/node": "^9.6.4",
    "babel-traverse": "6.26.0",
    "babel-types": "6.26.0",
    "babylon": "6.18.0",
    "clean-webpack-plugin": "~0.1.19",
    "codelyzer": "3.2.2",
    "copy-webpack-plugin": "~4.3.0",
    "css-loader": "~0.28.7",
    "extract-text-webpack-plugin": "~3.0.2",
    "lazy": "1.0.11",
    "nativescript-dev-sass": "^1.5.0",
    "nativescript-dev-typescript": "0.7.0",
    "nativescript-dev-webpack": "^0.10.1",
    "nativescript-worker-loader": "~0.8.1",
    "raw-loader": "~0.5.1",
    "resolve-url-loader": "~2.2.1",
    "sass-loader": "~6.0.6",
    "tslint": "5.8.0",
    "typescript": "~2.7.2",
    "uglifyjs-webpack-plugin": "~1.1.6",
    "webpack": "~3.10.0",
    "webpack-bundle-analyzer": "^2.9.1",
    "webpack-sources": "~1.1.0"
  }

@morzyns I've noticed that you are using nativescript-pro-ui suite for your RadSideDrawer.
Please update to the split packages and use nativescript-ui-sidedrawer (migration guide here).

That was super fast resp. Thanks. Checking up now.

@NickIliev is there any repo with side drawer for Angular? It looks like a lot of breaking changes...

I followed the migration guide and get this now

ERROR Error: Uncaught (in promise): Error: StaticInjectorError(AppModule)[MarketsComponent -> DrawerTransitionBase]:
StaticInjectorError(Platform: core)[MarketsComponent -> DrawerTransitionBase]:
NullInjectorError: No provider for DrawerTransitionBase!

So the SideDrawer component can't be migrated just via changing imports.
There are more structural changes.
For others migrating to Angular 5 i recommend to read the code that's available:

I think the ticket can be closed. Thank you.

@morzyns unless you are not planning to use RadSideDrawer as root element you should be able to migrate with no changes related to RadSideDrawer (e.g. I am using the drawer with latest NativeScript here)
Migration steps describing how to migrate your application with RadfSideDrawer as a root (meaning reused everywhere in your application with a single code base) can be found here.
I am also working on implementing the root side drawer in this POC application  where the side-drawer is used as root element

Thanks again, for some reasons i got issues with DrawerTransitionBase, as i found later new version doesn't import DrawerTransitionBase (explicitly) at all. After changing the imports and removing DrawerTransitionBase attribute i got it working.
That might be some specifics with my code base of course.


Search bar blinking cursor should appear far left

Position of blinking cursor should be consistent for "Search" and "Explore" prompts (currently "Search" cursor is centered, "Explore" cursor is far left)

Yes agree... "Search" should be left aligned in this case.

The text in the input field would need to be fully left aligned for the cursor to appear to the left of the placeholder text. Happy to make the change, can you confirm that is preferable?

@rdunk with this being the case I think it makes sense to keep it as is for usability reasons. Aligning left in this instance would make the text more difficult to see, especially on mobile. @laramplus -- are you okay with this?

Closing issue.


Slugify thinks apostrophes are significant

Given a title of "Baby's new shoes", slugify ought to return "babys-new-shoes" but instead returns "baby-s-new-shoes" which isn't right.  Seems related to changes made for #17.

I don't think it's not a bug : it's the default behavior (check the content of issue #17 and the tests in test.py.
Maybe you lack a boolean parameter in slugify() to specify if a quote mark should be replaced by '-' or ''.

Yeah, my preference would indeed be to have a boolean indicating what I'd prefer to have happen.
Of course I'd prefer the boolean to default to English's stance of apostrophes being insignificant but that's probably unnecessarily provoking to non-English speakers.  :)

I wish this was a one size fits all situation here, but it is not.  I am open to a boolean that defaults to the current behavior as I don't see a point in breaking backward compatibility.  I'll wait for a PR, or will roll it out on the next release cycle.

Closing as per https://github.com/un33k/python-slugify/issues/30.


New Problem sets

How do I extend the problem class. I need the WFG problem sets and the MaF  as well?
Is there any plans to incorporate these problem sets in the jMetalPy?

Hi Claudio.
We don't have plans to incorporate the WFG and MaF problems immediately into jMetalPy.
To add new problems you can follow as example the implementation of some of the problems already provided. For instance, the code of the ZDT1 is the following:
class ZDT1(FloatProblem):
    """ Problem ZDT1.

    .. note:: Bi-objective unconstrained problem. The default number of variables is 30.
    .. note:: Continuous problem having a convex Pareto front
    """

    def __init__(self, number_of_variables: int=30):
        """ :param number_of_variables: Number of decision variables of the problem.
        """
        super(ZDT1, self).__init__()
        self.number_of_variables = number_of_variables
        self.number_of_objectives = 2
        self.number_of_constraints = 0

        self.obj_directions = [self.MINIMIZE, self.MINIMIZE]
        self.obj_labels = ['x', 'y']

        self.lower_bound = self.number_of_variables * [0.0]
        self.upper_bound = self.number_of_variables * [1.0]

    def evaluate(self, solution: FloatSolution) -> FloatSolution:
        g = self.eval_g(solution)
        h = self.eval_h(solution.variables[0], g)

        solution.objectives[0] = solution.variables[0]
        solution.objectives[1] = h * g

        return solution

You can use it as a template. Anyway, if you have any question about this issue, please feel free to comment it to us.


OAuth logging in feature prototype.

I also tidied up the code, suppressing warning and removing parts that are not used (e.g. unnecessary wrappers of Grid tag).
I can do it in additional PR, but it's so little change, that I hope it can stay in this bigger feature PR.
Why it's not fully working feature? It has to make queries to the Firestore to check if the user is admin/inviter/member. It's not yet set up, so changes will come in two batches. First one is only logging-in and reading user data from Google id token.

@googlebot I consent.

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.
We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.
Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).
ℹ Googlers: Go here for more info.

@googlebot I consent

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.
We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.
Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).
ℹ Googlers: Go here for more info.

@googlebot I consent.

@googlebot I consent.

@leon-kastler-google looking over this I realise most of the comments are about code that I'm currently writing for the queries to the database I need on the admin side. @LightFelicis needed one of my queries to try one of her auth features, so I told her to re-base her branch on firestore-setup, the one I'm currently working on. If she feels comfortable with it, I can switch to her branch and add the suggestions you mentioned.

Of course, you can commit here. Copying all the comments to another firestore DB seems to be an overkill, so let's rename this PR to "Backend prototype", resolve all of the comments and merge it to master. Authentication needs only read access to the DB, co current firestore code (which is stable but has some style issues) is enough. Other firestore features like ADD, DELETE, CHANGE can be added in other PR, to make this one smaller. What do you think? @semiluna @leon-kastler-google

Yeah, keep the database reading part in there. Otherwise it will be a pain to untangle it. :+1:

Lots of warnings from flow.
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/components/admin/Dialogs/ActionInfo.js:18:30
Cannot use Promise [1] with fewer than 1 type argument. [missing-type-arg]
 src/components/admin/Dialogs/ActionInfo.js
  15│   action: ActionType,
  16│   open: boolean,
  17│   onClose: () => void,
  18│   onConfirm: (ActionType) => Promise<>,
  19│   tab: 'active' | 'solved'
  20│ };
  21│

 /tmp/flow/flowlib_38d5d604/core.js

[1] 667│ declare class Promise<+R> {
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/components/admin/Dialogs/PendingInfo.js:17:31
Cannot use Promise [1] with fewer than 1 type argument. [missing-type-arg]
 src/components/admin/Dialogs/PendingInfo.js
  14│   user: PendingType,
  15│   open: boolean,
  16│   onClose: () => void,
  17│   onConfirm: (PendingType) => Promise<>
  18│ };
  19│
  20│ const PendingInfo = (props: PropsType): React.Node => {

 /tmp/flow/flowlib_38d5d604/core.js

[1] 667│ declare class Promise<+R> {
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/components/admin/Dialogs/UserInfo.js:30:35
Cannot use Promise [1] with fewer than 1 type argument. [missing-type-arg]
 src/components/admin/Dialogs/UserInfo.js
  27│   user: UserType,
  28│   open: boolean,
  29│   onClose: () => void,
  30│   saveNote: (UserType, string) => Promise<>
  31│ };
  32│
  33│ const groups = [

 /tmp/flow/flowlib_38d5d604/core.js

[1] 667│ declare class Promise<+R> {
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/components/admin/Tables/ActionTable.js:62:42
Cannot get event.target.value because property value is missing in EventTarget [1]. [prop-missing]
 src/components/admin/Tables/ActionTable.js
 59│   };
 60│
 61│   const handleChangeRowsPerPage = (event: SyntheticEvent<>) => {
 62│     setRowsPerPage(parseInt(event.target.value, 10));
 63│     setPage(0);
 64│   };
 65│

 /tmp/flow/flowlib_38d5d604/react-dom.js

[1] 27│   +target: EventTarget;
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/components/admin/Tables/PendingTable.js:46:42
Cannot get event.target.value because property value is missing in EventTarget [1]. [prop-missing]
 src/components/admin/Tables/PendingTable.js
 43│   };
 44│
 45│   const handleChangeRowsPerPage = (event: SyntheticEvent<>) => {
 46│     setRowsPerPage(parseInt(event.target.value, 10));
 47│     setPage(0);
 48│   };
 49│

 /tmp/flow/flowlib_38d5d604/react-dom.js

[1] 27│   +target: EventTarget;
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ src/components/admin/Tables/UsersTable.js:83:42
Cannot get event.target.value because property value is missing in EventTarget [1]. [prop-missing]
 src/components/admin/Tables/UsersTable.js
 80│   };
 81│
 82│   const handleChangeRowsPerPage = (event: SyntheticEvent<>) => {
 83│     setRowsPerPage(parseInt(event.target.value, 10));
 84│     setPage(0);
 85│   };
 86│

 /tmp/flow/flowlib_38d5d604/react-dom.js

[1] 27│   +target: EventTarget;


Change emoji from spring to summer

This pr changes the default scrappy emoji reaction from :spring-of-making: to :summer-of-making: because it's summertime.

bump!

bump x2!

I agree that the emoji needs to be changed but I don’t agree that it should be the original :summer-of-making:. One thing that that emoji provides is an archive of the 2020 summer of making, and I think it should stay that way. Could we design one that’s similar but like orange or something? 👀

I agree that the emoji needs to be changed but I don’t agree that it should be the original :summer-of-making:. One thing that that emoji provides is an archive of the 2020 summer of making, and I think it should stay that way. Could we design one that’s similar but like orange or something? 👀

True. I'll think of some ideas too :)

I made a really quick drawing and left it on scrapbook for ideas.

I tried my hand at a similar design but orange. Let me know what you think! (the second one is currently on slack as :som-21: but i can change it if need be)

could we make it just :summer:?

Actually, there's some exciting stuff coming up re summer so let's give this a bit more time

sounds good; looking forward to whatever is happening this summer! 👀

I’m actually in favor of merging this with the new summer-21 emoji if you’re okay with it @sampoder because I think scrappy really needs to stop replying with spring-of-making when it’s almost July lol

Ya, that's fair. Online stuff seems to blocked so let's merge this


compiling viewer.less fails

Hello everyone,
until now i use angular-pdfjs-viewer 0.7.4 and everything works fine.
Now i was upgrading it (tryed it with both 0.8.0 and 0.8.1) and my build fails.
At the start of my build process i recompile the viewer.less with the global variable as it stands in the advanced configuration in the readme:
"lessc --global-var=\"pdfjsImagePath='../pdfjs'\" bower_components\\pdf.js-viewer\\viewer.less bower_components\\pdf.js-viewer\\viewer.css
As i said with 0.7.4 everything was ok. Now i get this:

SyntaxError: expected ')' got '/' in  on line 2, column 4312:

And this fails:
<EMAIL_ADDRESS>

This fails for all references to @pdfjsImagePath they all need to be enclosed in curly braces (78 occurance)
<EMAIL_ADDRESS>
needs to become
<EMAIL_ADDRESS>


API: Extend the blockstore/block to retrieve full transaction details

The current blockstore/block API method only returns transaction hash, even though the complex type is already populated with all the transaction details from the block store.
To reduce the amount of queries needed to show blocks and the transactions for local API clients, I propose that the block method includes a boolean property (query) named "Verbose" that can be set to include transaction details.
A related question, that might be relevant to this feature. This feature should maybe be turned off, if txindex=0 in the configuration (which it is by default). The node/getrawtransaction API method returns null if txindex is turned off in the configuration. Even with txindex turned off, at least some, if not all, transaction details are included in the block index (from my local debug testing). I don't know if txindex=0 results in some pruning eventually and might not always have block data available, hence I'm asking the question if this feature suggestion should throw error if txindex is not enabled.
Current API result:

API result with verbose turned on:

Done in https://github.com/stratisproject/StratisBitcoinFullNode/pull/1913


[configgen] Fix comment indentation.

Reduces the indentation of the interface name comment by one level. This
conforms the gapic config to Google style for yaml files.
Fixes #2653

haha I had been getting these YAML linter errors in google3! Thanks!


Allows deserialization of serialized requests with omitted http method

Per my comment on #89, I'm not sure this makes sense. Lack of an HTTP method means that the request is in an invalid state, and de/serialization of invalid messages probably should not be supported, as it makes parsing impossible.

Closed in favor of https://github.com/zendframework/zend-diactoros/pull/117


Upgrade simapp to 0.40.0

Closes #607
Closes #608
Closes #610

Merging now since we only have this set of test data

Merging now since we only have this set of test data


CVC4: non-terminating verification problem + Boogie SMTlib output not recognized

The attached Boogie file when run with cvc4 leads to non-termination with rapid memory growth. It's unclear whether this is caused by cvc4 or by Boogie (or both).
Trying to isolate the problem by using the SMT output Boogie generates and feed it directly into cvc4 did not work because cvc4 rejects this output with (error "Cannot push when not solving incrementally (use --incremental)"). (An independent bug.)
Repro (careful you may need to interrupt it can freeze your machine):
boogie -doModSetAnalysis -printVerifiedProceduresCount:0 -printModel:1 -enhancedErrorMessages:1 -monomorphize -proverOpt:SOLVER=cvc4 -proverOpt:PROVER_PATH=/home/wrwg/bin/cvc4 -proverOpt:O:smt.QI.EAGER_THRESHOLD=100 -proverOpt:O:smt.QI.LAZY_THRESHOLD=100 -vcsCores:12 -proverLog<EMAIL_ADDRESS>output.bpl

To reproduce the smtlib problem, take the generated .smt file and pass it to cvc4.
@barrettcw
bug.zip

The error message from cvc4 is not a bug, it's a feature.  cvc4 does not by default accept push and pop commands (because in general it is worse for performance to enable this feature).  Just pass the -i flag (short for --incremental) and it should work fine.

The error message from cvc4 is not a bug, it's a feature. cvc4 does not by default accept push and pop commands (because in general it is worse for performance to enable this feature). Just pass the -i flag (short for --incremental) and it should work fine.

Great with that input I could nail the issue down to cvc4. Now I have the smtlib file which lets cvc4 quickly grow to gigabytes. Should I file an issue on cvc4 github or how do I report?

Yes!  Please file an issue on cvc4's github issue tracker.

Opened https://github.com/CVC4/CVC4/issues/6495 and closing this one.


[KED-944] adding contrib.NetworkXDataSet

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Motivation and Context
Why was this PR created?
To add native support for graphs to Kedro (via NetworkX).
How has this been tested?
What testing strategies have you used?
I've used this in a number of personal projects. I had to refactor slightly due to changes in Kedro's develop branch.
I also ran make test. All of the tests for NetworkXDataSet failed, so I'll need to familiarize myself with parametrized unit tests (and fixtures) so I can resolve those issues.
Checklist

[X] Read the contributing guidelines
[X] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[X] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes
[ ] Assigned myself to the PR

f-string f"{self._filepath}" is not supported by Python 3.5. I would suggest str(self._filepath).


Colab runs forever

Hi.
The colab runs forever, even after setting block.launch(debug=False) and produces no result.

Solved the problem.


Backup EFI to Desktop does not work as expected

Backup EFI to Desktop "Backup EFI"  does not work as expected
Thanks for this excellent tool and for the frequent updates! It generally works very well and is easy to learn. I just came across this small issue when wanting to backup my currently active EFI.

Reproduce

Launch OCAuxilaryTools in Monterey (or Mojave)
Mount ESP Partition and Open Config.plist using OCAuxilaryTools
Click Backup EFI to Desktop "Backup EFI" button or select from Menu
nothing happens, no folder or zip file is created, no errors, no log found
tested on macOS Monterey 12.2 and macOS Mojave 10.14.6
tested with V202200091

Expected Result

A "Backup EFI" folder is created with a zipped up EFI inside
the EFI to be backed up is expected to be the currently open EFI in OCAuxilaryTools, so it can be backed up before changes are made

What does work?

Menu > Generate EFI on the Desktop
Then click Backup EFI to Desktop "Backup EFI" button or select from Menu
This actually creates a backup in the Backup EFI folder, but not the one I want, just a backup from the newly created EFI on the Desktop

This is either a bug or a problem of lack of documentation (with user expectations being different from developer intentions).

In fact, OCAT has already made a backup for you in the background, it is done silently, you can right click on the "Backup EFI" icon in the toolbar and select "Open backup directory", you will find the files already backed up there.
This backup is done silently, please note!

"Open backup directory", you will find the files already backed up there.

Thanks for your reply! With version V202220094 on Monterey, Backup EFI to Desktop actually works and it creates a "Backup EFI" folder with the zipped EFI inside after clicking the button. Therefore, on Monterey this feature now works in the way I would expect. - On the other hand, no backup was created silently without clicking the button first, which I checked with "Open backup directory". - Is there another hidden backup directory?
On macOS 10.14.6 none of this works:  "Open backup directory" as well as "Backup EFI to Desktop" does nothing at all (just as detailed in my initial bug report). No errors either. Also, "Synchronize OC Main Program" is greyed out (see screenshot) and cannot be started. I tried starting the app from the terminal, but still no error messages.

I understand that this is a well designed GUI application which is mostly self-explanatory. Nevertheless, if you do have some documentation explaining how each feature is expected to work, please link me to it. I think I have read all that there is (linked from the main Readme), but I may have missed something.

On macOS 10.14.6 none of this works: "Open backup directory" as well as "Backup EFI to Desktop" does nothing at all (just as detailed in my initial bug report). No errors either. Also, "Synchronize OC Main Program" is greyed out (see screenshot) and cannot be started. I tried starting the app from the terminal, but still no error messages.

The reason why you can't backup is that the configuration file you opened is not in a complete EFI directory, so OCAT can't complete the backup.
The prerequisite for an OCAT backup is that the EFI target must be complete, otherwise the backup will not take place.

For details you can refer to this question, which has a description of EFI directory integrity: https://github.com/ic005k/QtOpenCoreConfig/issues/77

Please forgive me for not having much time to write the documentation. In the next version, OCAT will improve this and there will be a prompt telling the user why they can't backup.

The reason why you can't backup is that the configuration file you opened is not in a complete EFI directory, so OCAT can't complete the backup. The prerequisite for an OCAT backup is that the EFI target must be complete, otherwise the backup will not take place.

(Google Translate of #77 ) OCAT will automatically check the integrity of each directory under EFI, otherwise the sync button will be grayed out. Main check:
EFI/OC
EFI/BOOT
EFI/OC/Drivers

Ok, I finally found out why my fully functional and complete OpenCore installation would not backup with OCAT: I used the new Launcher Option in OpenCore, which does not require a BOOT directory or boot files. Therefore I had deleted the BOOT directory on my main OC booter. Only the debug OC booter still had the boot file and directory.

If everything is set up correctly, the first boot will have OpenCore create a new boot option in our BIOS (pointing to EFI/OC/OpenCore.efi) and future boots will update the entry making sure it's correct and ensuring it is still present. This now allows us to remove BOOTx64.efi and not worry about other OSes overwriting the EFI/BOOT/BOOTx64.efi path.
Setting up the LauncherOption | OpenCore Post-Install

SOLUTION: I recreated the EFI/BOOT  directory on my main OC booter, and now OCAT recognises it as valid.
A user prompt or error message from the UI would definitely be helpful when OCAT cannot find a valid OC directory structure.

The new version of OCAT has excluded EFI/BOOT, so now you don't have to do something extra for EFI/BOOT.


Fix mapPage method in PagedIterable for sync stack

Fixes https://github.com/Azure/azure-sdk-for-java/issues/34148

API change check
API changes are not detected in this pull request.

/azp run java - core - ci


cancel render

Is there a way to cancel a render?
once i've started a job, say i realise i've made a mistake, rather than have to wait for the render to finish is there a way to cancel/stop it?

Hey @jonnymangothinkjam
The only way to stop a running job right now, that comes to my mind is to essentially kill a process that does the rendering.
Unfortunately, at this moment there are no other ways of doing that, since AE does not give us an API through which we could do that within aerender, as far as I know at least.
Cheers!

I'd like to be part of investigating this further, eg by finding a way of discovering the processes started by aerender (like aerendercore) and store them on the job, and adding a method on the job to kill the whole tree and do the cleanup.
Anyone with experience in this?
I've outlined a crude approach that we're using at DR in a discussion here:
https://github.com/inlife/nexrender/discussions/739


Finish deprecating the fs argument

See #5385 for some discussion on this
The fs= arg was depcrecated from Dataset.save_to_disk and Dataset.load_from_disk in 2.8.0 (to be removed in 3.0.0).  There are a few other places where the fs= arg was still used (functions/methods in datasets.info and datasets.load).  This PR adds a similar behavior, warnings and the storage_options= arg to these functions and methods.
One question: should the "deprecated" / "added" versions be 2.8.1 for the docs/warnings on these?  Right now I'm going with "fs was deprecated in 2.8.0" but "storage_options= was added in 2.8.1" where appropriate.
@mariosasko

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks for the deprecation. Some minor suggested fixes below...
Also note that the corresponding tests should be updated as well.

Thanks for the suggestions/typo fixes.  I updated the failing test - passing locally now

Nice thanks !
I believe you also need to update _load_info and _save_info in builder.py - they're still passing fs=self._fs instead of storage_options=self._fs.storage_options
This should remove the remaining warnings in the CI such as
tests/test_builder.py::test_builder_with_filesystem_download_and_prepare_reload
tests/test_load.py::test_load_dataset_local[False]
tests/test_load.py::test_load_dataset_local[True]
tests/test_load.py::test_load_dataset_zip_csv[csv_path-False]
tests/test_load.py::test_load_dataset_then_move_then_reload
  /opt/hostedtoolcache/Python/3.7.15/x64/lib/python3.7/site-packages/datasets/info.py:344: FutureWarning: 'fs' was deprecated in favor of 'storage_options' in version 2.9.0 and will be removed in 3.0.0.
  You can remove this warning by passing 'storage_options=fs.storage_options' instead.

re: docstring, I assume passing in storage_options=s3.storage_options is correct/necessary to pass the secrets?

what about
https://github.com/huggingface/datasets/blob/5b793dd8c43bf6e85f165238becb3c64f6cd3ed0/src/datasets/filesystems/__init__.py#L43-L54


Implement Root-style Reputation Tracks - [closed]

In GitLab by @n1xx1 on Nov 30, 2022, 10:17
Merges master -> master
I implemented something similar to Root's RPG reputation tracks. Basically it tracks 3 values, the actual bonus you get (.value) and two other values (.negative.value and .positive.value) that represent how close you are from changing the actual bonus. It's marked as draft because I didn't handle the diffing yet, and maybe something else I forgot.

In Root it works like this: you have a Reputation (.value), Notoriety (.negative.value) and Prestige (.positive.value). Basically Notoriety and Prestige are two clocks that go in different direction. Every time you reach a certain threshold with your Notoriety/Prestige you increase/decrease the Reputation, and clear the Notoriety/Prestige. This threshold depends on the actual Reputation. If the Reputation is 0, then this threshold would be the steps of the Notoriety/Prestige tracks (so 3/5), but, if it is +1, then the thresholds become 3/10, if it's +2 it's 3/15. Likewise if the value is -1 the thresholds are 6/5. This behavior is automated.
The default configuration is:
  [character.attributesTop.reputationField]
    type = "Track"
    label = ""
    default = 0
    [character.attributesTop.reputationField.negative]
      label = ""
      steps = 3
      max = 3
      default = 0
    [character.attributesTop.reputationField.positive]
      label = ""
      steps = 5
      max = 3
      default = 0

Hopefully it's common enough to make it into the system, but to be honest I don't know of any other game that has a similar subsystem.
Also, in Root you have multiple of these tracks for every faction you interact with. So eventually I may make a new pull request to support Lists of Tracks.

In GitLab by @n1xx1 on Dec 1, 2022, 03:28
added 1 commit
e1e3f4e3 - Track diffing
Compare with previous version

In GitLab by @mclemente on Nov 28, 2023, 22:25
mentioned in commit bd5ad70f1f8663fb26f4e1bba13dc41647c7e66f

In GitLab by @Asacolips on Dec 2, 2022, 08:54
@n1xx1 thanks for submitting this! I've had a few requests for a tracker that supports Root, so even though it's pretty game specific, I think it's worth including in the system. At a quick glance the approach looks good overall, but I'll need to pull this down and do some local testing of it along with reading it more closely. It will probably be a week or so before I can do that, but I'll add comments on here if/when I find anything.

In GitLab by @mclemente on Nov 28, 2023, 22:27
Thank you for your contribution, it has been a while and this MR wasn't set for members to edit, so I've edited and sent a MR myself with you as co-author (!51).

In GitLab by @mclemente on Nov 28, 2023, 18:33
marked this merge request as ready

In GitLab by @mclemente on Nov 28, 2023, 18:33
changed target branch from master to reputation-tracks

In GitLab by @mclemente on Nov 28, 2023, 22:09
changed target branch from reputation-tracks to master


Strange behavior with headers in FastAPI

When your running local wrangler dev based on example in documentation here https://developers.cloudflare.com/workers/languages/python/packages/fastapi/
I see strange response headers, contains b'' strings

PS. I am using npx<EMAIL_ADDRESS>dev bc latest wrangler version not working at all.

I think this is fixed by https://github.com/cloudflare/workerd/pull/2125


win-sshproxy.tid created before thread id is available

this commit fixes a potential race condition that prevented the tests to succeed when running in a github workflow.
Basically the thread id was not actually available before writing it on the file, resulting in a thread id equals to 0 written in it. So, when the tests were trying to retrieve the thread id to use it to send the WM_QUIT signal, they failed.
This patch adds a check on the thread id before writing it on the file. Now, if the thread id is 0, it keeps calling winquit to retrieve it. If, after 10 secs, there is no success it returns an error.
it resolves #432

@n1hility fwiw, a small race in win-ssh-proxy/winquit.

I've created https://github.com/containers/winquit/issues/2 for the underlying winquit issue.


solr & fedora configs for wrappers

Hyrax should manage it's own Solr/Fedora configs for the solr_wrapper and fcrepo_wrapper.
Changes proposed in this pull request:

add .fcrepo_wrapper

copied some values from .internal_test_app/.fcrepo_wrapper

add .solr_wrapper

copied some values from .internal_test_app/.solr_wrapper

This makes it possible to call the wrappers from the hyrax repo checkout, rather than cd into the internal test app, to run Solr/Fedora in a separate window.
@projecthydra-labs/hyrax-code-reviewers

Some changes pushed up, leaving commits to see changes, to be squashed when we arrive at the final versions.

I'd like some justification in the commit message. Why do we want this change?

This PR seems to encourage yet another way to do testing/dev.  Will that add confusion to an already confused topic?

I see these files in other projects and they allow running solr_wrapper and fcrepo_wrapper from the project root directory.  In hyrax, these files do not exist yet, they only appear to be auto-generated in the engine-cart .internal_test_app.  So when I want to run the solr/fcrepo wrappers, I cd into that directory.  I get comments from Chris that this is a weird practice and he seemed to suggest that it's strange these files are not in the hyrax repo already.  When they are added here, they conform to an expectation on how to configure these wrappers in the code repo.  So, now that this is being questioned too, I'm totally confused.   I thought this is the best practice for hyrax to take control over these settings for the wrappers.

There's no extensive commit message because we use github.  If you use github features to track the blame for a change and also note the merge PR number, you get back to this commentary etc. and the description in this PR.

@darrenleeweber I don't want to tie us to github.  It's nice when you're away from the network to see the rational in the commit messages.  Furthermore, we can't guarantee that github is going to offer us a free account (or even exist) as long as we need it.  Putting everything in git would enable us to go elsewhere.

FWIW, I'm fine with adding .solr_wrapper and .fcrepo_wrapper to Hyrax, as long as we follow up by editing the appropriate wrapper-related sections of https://github.com/samvera-labs/hyrax/wiki/Hyrax-Development-Guide such that we now have a less confusing state of affairs. Sorry for the twists and turns here, @darrenleeweber.

Cannot edit the development guide until we merge this.  When we do, it becomes something like these patches:

https://github.com/samvera-labs/hyrax/wiki/Hyrax-Development-Guide#solr

- solr_wrapper -d solr/config/ --collection_name hydra-development
+ solr_wrapper # using .solr_wrapper config

https://github.com/samvera-labs/hyrax/wiki/Hyrax-Development-Guide#fedora

- fcrepo_wrapper -p 8984
+ fcrepo_wrapper # using .fcrepo_wrapper config


[HELP] [Multiple Host] workflow suggestion for multiple devices

I am using workflow where i have default settings in dotfiles, and host specific config in dotfiles_local. (same as the author is doing).
i moved polybar in dotfiles_local, as on some system temperature module has different thermal package ID then others, and the solution is working fine.
But now i want to add new module to polybar config for all hosts, how can i do it? currently i have to go in to each branch of dotfiles_local and applying changes.

came up with jinja solution with existing setup.
https://github.com/dhruvinsh/dotfiles/blob/master/patches/jinja.patch - patch available is here.
i have forked this repo and applied the changes and my reference is currently using it, at the time of writing, polybar is using it.


Add turn restrictions

Implements #94 by adding a new turn restriction frontier model which loads a set of edge id tuples that each represent a turn restriction.
This also adds the ability for multiple frontier models to be provided and iterates over each model, excluding an edge from the search if any of the models result in an invalid frontier.

the recursive approach is more general, it supports both of these kinds of collections. whatdyathink?

Yeah, that is a much more general way to do it, good call. I'll take a pass at that and update this PR!

@robfitzgerald - Okay, I've gone in and introduced a new CombinedFrontierModel that will execute the same logic as before, should be ready for another review.

dig it!


[FEATURE] reduce(wide, plus) optimisations

Arm has pairwise additions/ full additions, we should expose those from reduce(wide, plus)

Discussion: let's reduce(x) be sum(x) and optimize from here

We discussed that sum will be in detail also.


how to use command hello with two params?

when i tried command with "!hello 1 2", and bot response Hi 1 2

Sorry for the delayed response. Sorry but it looks like I put a weird corner case in the examples so that file doesn't work without modification, I have now updated them.
The simplest fix is to change hello to something else in one of the commands. The below snippet should work.
@slackbot.command('hi <name>')
def hello(name):
    '''Simple command with argument which replies to a message'''
    return Response(text="Hi {0}".format(name), channel=command.channel)

@slackbot.command('hello <name> <from_>')
def hello_from(name, from_):
    '''Simple command with two arguments which replies to a message'''
    return Response(text="Hi {0}, from {1}".format(name, from_),
                    channel=command.channel)

The reason this is needed is due to inner workings of phial. When commands are registered their 'patterns' (the strings passed into the command decorator) are converted to regex and added to list. When a slack message is detected phial loops through the list until a match is found which it then executes.
An alternative solution would be to register the more specific commands first
@slackbot.command('hello <name> <from_>')
def hello(name, from_):
    '''Simple command with two arguments which replies to a message'''
    return Response(text="Hi {0}, from {1}".format(name, from_),
                    channel=command.channel)

@slackbot.command('hello <name>')
def hello_from(name):
    '''Simple command with argument which replies to a message'''
    return Response(text="Hi {0}".format(name), channel=command.channel)

Sorry for the confusion this caused, when I get some free time (a rarirty) documenting phial is my number one priority. If you have any more questions going forward please just raise another issue and I'll do my best to get back to you in a timely manner

Going to close this now, if the issue persists feel free to reopen


Add interface for external signals not using Arrow data streams

Can be used for platform specific stuff

Sorry, this breaks the Python wheels, but we'll solve this later.


tab key does not set

Your program is awesome, thank you for it. I got your code and build in my Raspberry Pi for test it.
When I created a Makefile for do anything in C, I saw this, lite defined tab as two spaces, then, I came here to view any your answer.
That is an error?!
Excuse me for my English, I'm a Brazilian and beginner in that language.

Your program is awesome, thank you for it. I got your code and build in my Raspberry Pi for test it.
When I created a Makefile for do anything in C, I saw this, lite defined tab as two spaces, then, I came here to view any your answer.
That is an error?!
Excuse me for my English, I'm a Brazilian and beginner in that language.

You can edit the size of tab in user config (data/user/init) changing the config.indent_size and config.tab_size (I don't know exactly what this last one does, I just edited the first and worked for me).
But no, it's not a bug.

Thank you for your answers but this is not the problem... When I save some code, lite define tab as spaces (0x20) instead Horizontal Tabulation (0x09 or \t).
Try to write a Makefile and run it.

Thank you for your answers but this is not the problem... When I save some code, lite define tab as spaces (0x20) instead Horizontal Tabulation (0x09 or \t).
Try to write a Makefile and run it.

I don't know the problem with this but ok. You can make a plug-in to replace (gsub) every "\s" * tab_size in your code that isn't inside a string

Putting config.tab_type = "hard" in your data/user/init.lua file should switch you to hard tabs.

Thanks, it's this. I didn't found other value for this field, resolved!


Watch Time Duration

Hello MShawon , firstly I wanna say thank you so much for this project.
I think its working for me I just cant see the views on my youtube videos.
I want to ask how can we change the watch duration time and make the bots watch videos longer ? Also is there any proxy provider which you suggest? Because I bought one and almost %70 of proxies are failing

First of all youtube take some times to show increased views in YouTube. So you need yo check video analytics in YouTube Studio.
Second to avoid a pattern I randomize watch duration between 85% to 95% . So certainly this duration won't hurt your YouTube average watch duration statistics.
Third I never tried any paid proxies service. But you can take a look at this free proxies list https://github.com/MShawon/YouTube-Viewer/issues/11#issuecomment-772421530


Tab Image should collapse on fragment scroll itself.

I have a scrollview in my fragment. If the Image is still on the screen and i scroll,complete content of the that fragment doesnt appear. I think on scrolling,the image should collapse by itself.

You can see the cut at the end... that is the case.

Also, what to do , if i dont want the title bar?


Register User API and Postgres DB Addition

Added API code in separate file for now since init.py file use different DB. API code works locally.
In docker-compose, added container with pgadmin image. PgAdmin is a postgres client. The database content is viewable by going to pgadmin link in docker and logging in.

@JPrinceUTD could please refactor the code by moving the registration functionality into user-management

Jignesh,
I refactored the code locally but it looks like the psycopy2 library (postgres adapter) won't install with python alpine. So, I have temporarily changed it to python latest. I am not able to try this change today since I exceeded my pull limit in docker, but will try tomorrow.
Jibin Prince

From: Jignesh Satam @.>
Sent: Sunday, March 26, 2023 2:39 PM
To: jigneshsatam/Photo-Album-Generator @.>
Cc: Prince, Jibin Chirayil @.>; Mention @.>
Subject: Re: [jigneshsatam/Photo-Album-Generator] Register User API and Postgres DB Addition (PR #12)
@JPrinceUTDhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJPrinceUTD&data=05|01|Jibin.Prince%40UTDallas.edu|e58164b30c0b4259bb0508db2e31c243|8d281d1d9c4d4bf7b16e032d15de9f6c|0|0|638154563497111663|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=TomY6K8pIR3%2F1Y3QIo6n1cj9foKmxwd2iqv3fRdgb8M%3D&reserved=0 could please refactor the code by moving the registration functionality into user-management
—
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjigneshsatam%2FPhoto-Album-Generator%2Fpull%2F12%23issuecomment-1484201158&data=05|01|Jibin.Prince%40UTDallas.edu|e58164b30c0b4259bb0508db2e31c243|8d281d1d9c4d4bf7b16e032d15de9f6c|0|0|638154563497111663|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ZRger8r%2FE8zTPNzx%2FJy8dNag5pHlEq4ZLi72FW3S6go%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVSUX2GTAALABN3HGSYBYELW6CLNRANCNFSM6AAAAAAWDETSNU&data=05|01|Jibin.Prince%40UTDallas.edu|e58164b30c0b4259bb0508db2e31c243|8d281d1d9c4d4bf7b16e032d15de9f6c|0|0|638154563497111663|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=7Tlsj32%2FMH2RgiOlTUe%2BGmqTjnQXLHQgU0w0qJIJoHM%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Cursor disappears in Vim in build 18252

Your Windows build number:  (Type ver at a Windows Command Prompt)
10.0.18252.1000

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
Attempt to navigate a text file using vim. As soon as any of the primary navigation keys are pressed ('h', 'j', 'k', 'l'), the cursor disappears. It will reappear if you search for something or change modes (e.g. visual or insert), but it then disappears after any further keystrokes. The cursor will also reappear if the console is deactivated/activated (e.g. alt-tab away and back).

Cursor visible:

Cursor no long visible after hitting 'l':

What's wrong / what should be happening instead:
The cursor should remain visible.

I think this issue is specific to WSL as I cannot reproduce it in either a standard command prompt or powershell instance.

I believe this is the same as  https://github.com/Microsoft/console/issues/269.


Feature: Flash Message Styles and Animation

PR connects to #415
Background
Initial styles and some animation for flash components.

Flash that fades is show here:

Added props and vars to the bottom of the example page

@CheetoMao when you get the chance, would love to get a code review. Thanks!


#6 Join: Add new styling to selected consultant

Update the styling to what is displayed below when the consultant is selected:

Ticket:  https://razoyo.atlassian.net/browse/TST-183

Looks good to us:

need Bug #299 fixed 1st

bug # 299 was fixed.....
and here is what I see now

closing- bug #107


setWidgetId method

Firstly Awesome directive...
I am using angular-recaptcha and i want to multiple widgets in an single application....
As told in documentation i used setWidgetId('widget1'); but in the console i got an error as invalid client Id....
Can u please share a fiddle or plunker explaining how to render multiple widgets using this directive..
Thanks,waiting for ur response...
sam....

Here is the same question: https://github.com/VividCortex/angular-recaptcha/issues/76
http://plnkr.co/edit/r6msW5sXV7QzoWIwxY28?p=preview

The documentation which uses setWidgetId is not documenting setWidgetId, but showing how one would use the on-create listener to listen for the create event which will provide the ability to get the widgetId (created by recaptcha).  The setWidgetId in the example is a custom function allowing the developer to store the id (created by recaptcha, passed through the listener callback) for future use.
You do not get to set the ID of the widget, but you can get and set/store the value so that you can reference specific widgets when there are multiple of them.

why the plunker u shared is not displaying recaptcha???

@ssameer07 Sorry, try now http://plnkr.co/edit/r6msW5sXV7QzoWIwxY28?p=preview
you're can do that:
$scope.response1 = '';

$scope.submit = funtion()
{
  if($scope.response1.length === 0){
    $scope.alerts = [ { type: 'danger', msg: 'Please complete the captcha.' } ];
    return;
  }
}

@jackkum  wooooh...that worked for me...
it took ma whole day solving this....thanks a lot man!!!


expose proof state in proof checking

r? @patrickxb
@chrisnojima  needs this plumbed through for doing proofs in the UI

LGTM

./prove_check_test.go:63: assignment count mismatch: 3 = 4

LEBTM (looks even better to me)

Sorry I was letting CI do the testing since my laptop is quite puny


Error: It looks like you are using Playwright Sync API inside the asyncio loop. Please use the Async API instead.

Hello,
I am working on implementing the schab-api package with my trading strategy, and have gotten the error:
"Error: It looks like you are using Playwright Sync API inside the asyncio loop. Please use the Async API instead."
After some investigation, it looks like there is an issue with using the schwab-api within spyder. Is there a workaround for this? I saw that there was a way to get things to run on google colab, but the solution does not work locally for either spyder or jupyter notebook.

Seeing this also, when running in Jupyter notebook.

Seeing this also, when running in Jupyter notebook in Colab.

still seeeing this one in JupyterLab. Any workarounds?

The only workaround I could see is using async playwright to login to Schwab since the sync_playwright library is inherently incompatible with Jupyter

@itsjafer could this be added with an option when creating using api = Schwab(). Like doing api = Schwab(async=True) or something like that? Can have the option default to False but be there for Jupyter or Google Colab.

Could one of you provide an example Colab notebook that faces this error so I can debug?

Brief update on this: I tested out an asynchronous version and it seemed to work well with Jupyter/Colab, though it wasn't actually able to login as expected. I'll continue looking into offering an async solution in the meantime.

I made an async login a few weeks ago that I was running on streamlit.
I'll try to see if I can retrieve it...

Cool!
Thanks
On Mon, Mar 4, 2024 at 10:00 PM 4rumprom @.***> wrote:

I made an async login a few weeks ago that I was running on streamlit.
I'll try to see if I can retrieve it...
—
Reply to this email directly, view it on GitHub
https://github.com/itsjafer/schwab-api/issues/11#issuecomment-1977357799,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6L6M5AKUN7U5INDH6NXFTYWTHHDAVCNFSM6AAAAAAQRUB7UWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGM2TONZZHE
.
You are receiving this because you commented.Message ID:
@.***>

So I found the code! I'll create a new pull requests as soon as my current one for option strategies went through.

Hi version 0.4.0 is now compatible with asynchronous environments. Thank you @4rumprom for your contribution!


[GHSA-c72p-9xmj-rx3w] Archive package allows chmod of file outside of unpack target directory

Updates

References

Comments
Adding patch link for v1.4.8: https://github.com/containerd/containerd/commit/22e9a70c71eff6507be71955947a611f2ed91e6c
Adding patch link for v1.5.4: https://github.com/containerd/containerd/commit/7ad08c69e09ee4930a48dbf2aab3cd612458617f
The GHSA-ID is in the commit message: "Merge pull request from GHSA-c72p-9xmj-rx3w Use chmod path for checking symlink"

Hi there @dmcgowan! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory


[INFINITY-1012] Make tests go faster ( parallelize)

Goals:

run tests faster in parallel
Better logging (timestamps!)
Render failures unambiguously via junit xml (not done yet)
Try to reduce the number of environment variables floating around.
New test runner, parallelizes, junit for test actions, verifies requirements.

So this seems like it will reduce test runs by about an hour (33% reduction); but I can't get a test run to pass for reasons unrelated to this change.  Maybe by tomorrow.
That's not a very exciting improvement, sorry.  However, most of the work was figuring out where all the problems were running overlapping tests.  We could get a larger improvement in a variety of ways, and I'll continue to work on that mixed in with other tasks.

Reviewers, I welcome critique.
I haven't addressed Susan's existing criticism that we should fetch the dcos command line from 1.9, but I plan to do that in the next few working days, by fetching the cluster version and fetching the dcos command line that matches.

How are the junit changes related to the parallelization?  If it's a separate task, can you put that in a different PR?

This PR description seems to be the same as that of the last commit: https://github.com/mesosphere/dcos-commons/commit/bf496380964403d33bd3f9b4c290eef9ef87fe2c
What's different about this?  Does it resolve INFINITY-1012?  If not, can you create a JIRA for what this resolves?

The last commit was intended as wip but art was.having significant trouble with build dependency that were addressed in the wip. Probably I should have pointed him to the branch to copy the script out of.


Package request: vscode-extensions.vue.volar

Project description
A VSCode extension that adds Vue.js language syntax highlighting.
Metadata

homepage URL: https://marketplace.visualstudio.com/items?itemName=Vue.volar
source URL: https://github.com/vuejs/language-tools
license: mit
platforms: unix, linux, darwin

Add a :+1: reaction to issues you find important.

Whoops, that was the typescript + vue support plugin, not the main VueJS one.

Just open a PR adding the extension, much easier than opening an issue...

Please see https://github.com/NixOS/nixpkgs/issues/290471#issuecomment-2088628067


Marathon fix close connection

It looks good, but another fix would be to close the connection factory. The fixes in #570 will make these changes void i think.


Create CUDAMatrix from device pointer

hi all,
Is there any C API for CUDAMat? I want to create a matrix using CUDAMat in python from an already allocated GPU memory and I can only access the memory by a pointer in c. Is there any c/c++ API for CUDAMat to accomplish this?
Great thinks in advance!

Is there any C API for CUDAMat? I want to create a matrix using CUDAMat in python from an already allocated GPU memory and I can only access the memory by a pointer in c.

CUDAMat is a C shared library accessed from Python via the ctypes module. So the CUDAMatrix is defined in Python and there is no C API to create it. However, if you can get your pointer into Python, you can create a CUDAMatrix accessing this pointer. See this example:
https://github.com/Theano/Theano/blob/7a97cfe/theano/misc/cudamat_utils.py#L67-L86
In that example, it is assumed that the memory belongs to the already existing object x, so the matrix is marked as not owning the data, and the object x is attached as a property to the CUDAMatrix object so Python will not delete x before deleting the CUDAMatrix. If instead, you want the memory to be managed by cudamat (and freed with cublasFree), just set owns_data to True.
Hope this makes things clear -- if you want to make this use case easier, I'd be open for a PR that adds a factory method CUDAMatrix.from_memory or something, taking a device pointer, shape and an optional parent object.

@f0k Thanks for your answer! It's clear.
But he python module I'm to add that feature to uses boost.python to wrap C++ code and doesn't expose the GPU memory pointer to python. So I have to start from C++ and it seems that boost.python doesn't have an elegant way to expose a pointer such as float * to python. So I'm wondering whether can I use cudamat  to handle this along with boost python?

Or what type of object should I expose in python managing the GPU memory pointer?

So I have to start from C++ and it seems that boost.python doesn't have an elegant way to expose a pointer such as float * to python.

There should be a way. In the worst case, you can expose it as a Python long and then cast it to a pointer within Python using ctypes, as in the example code I linked to.
You can also create the cudamat struct in C/C++, but then you still have to pass it to Python -- this won't make things easier.

@f0k I will try it. Thanks for your thorough explanation.


Add save_freq to CAPQL.train()

When training the CAPQL agent, I noticed a drop in the simulation speed when setting checkpoints = True. In the code, if checkpoints = True, the weights of the nets are stored after every step taken by the agent. A parameter such as save_freq could be added to save the net's weights only every save_freq number of global steps.
Another improvement would be to save the weights in different files as its being done in the GPI-PD agent, for example.

Thanks for spotting this! I will open a PR to fix it :)


Tag a release?

I wonder if this project is now mature enough to tag an alpha 0.0.1 release? That might help if we want to use it on the tskit tutorials pages.

Looks like you are addressing this, thanks @kitchensjn. If you are packaging for PyPI, then there are some relatively simple tskit projects such as https://github.com/tskit-dev/tsdate which have files from which you can copy/paste if you need some examples.

Ah, yes! I meant to follow up on this issue. I've been working through a few of the other issues that I think are important to have for the initial release. It should be very close! Thanks for directing me to the tsdate files; very helpful to have an example.
I have my graduate qualifying exam next week, but after that I'll have more time to focus on getting this released.

Great, no hurry here. Good luck for your exam!

Release 0.0.1 is available on GitHub and PyPI. This is my first time releasing a package onto either site, so please let me know if I missed anything obvious or if you have any issues installing and running the visualizer.

Amazing. Thanks!


Speech / Persuasion Broken

After an indeterminate amount of time playing, speech challenges eventually break. Every speech option fails, no matter the difficulty, regardless of your ranks in persuasion and other modifiers.

Details
It appears some effect sets your speech modifier to 0 persistently, giving every attempt a 0% chance, as it's a multiplier. This can be fixed via console: player.setav 2d873b [x]. Sometimes it has to be set to 2 to balance out whatever is modifying it to 0. forceav probably works too.
Game Version
1.12.30
Language
English (EN)
New Game
No
Reproduction Steps
Play normally, taking as many speech challenges as possible. Best way is to join the trackers alliance and find roaming bounty targets in cities. Pass these challenges, rank up your persuasion skill, and eventually you'll start consistently failing every attempt, no matter the difficulty.

I've observed that NG+'ing restores the referred AV to it's default value of 1.00.
Does anyone know how the Persuasion perks affect persuasion chances? Using player.getAVInfo 2d873b with four ranks of persuasion returns the following:
GetValueInfo: PlayerPersuasionSkillSuccessChanceMult on Vincent Nemo
...Current Value: 1.25 Computed Base: 1.00
...Base Value Components:
......Reference Base Value: 0.00
...Modifiers: Temp: 0.25 Perm: 0.00 Damage: 0.00
...Level-up Value: 1.00

I've got Paramour affecting my character at that moment and four ranks in Persuasion. When I NG+'s, I had three ranks in persuasion.
Shouldn't the Perm modifier be at 0.50 with four ranks of persuasion? Or does the persuasion perk effect apply to some other AV related to speech checks?


bump version in package.json to v2.20.2

release version v2.20.2

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Import 3db3ecfac5: Add flow en/disable/ to menu, API. Listing in menu as 'dynamic bone flow ' (by Caitlyn)

This is an automated proposal to look at a commit made by Caitlyn and import it into Overte

Commit:
3db3ecfac5b18584315dd3472976d669ea31373a

Author:
Caitlyn

Date:
Mon, 13 Apr 2020 14:43

Add flow en/disable/ to menu, API. Listing in menu as 'dynamic bone flow '

Stats:

Filename
Stats
Lines
Added
Removed
Lines in blame

interface/src/Menu.cpp

23
21
2
14

interface/src/Menu.h

2
2
0
1

interface/src/avatar/MyAvatar.cpp

2
1
1
1

interface/src/avatar/MyAvatar.h

19
19
0
19

4 files
-
46
43
3
35

To work on this, please assign the issue to yourself, then look at the commit and decide whether this would be a good addition to Overte.

If the commit is useful, tag it with "Tivoli: Keep", and keep it open until it's merged.
If the commit is not useful, tag it with "Tivoli: Discard", and close it.
If the commit is not useful right now, but might be later, tag it with "Tivoli: Maybe later", and close it.
If it's hard to decide, tag it with "Tivoli: Discuss", and keep it open.

Useful commits should be submitted as a PR against Overte. Tag this issue in PR, so that it's automatically closed once the PR is merged.
You can cherry-pick this issue with this command:
git cherry-pick 3db3ecfac5b18584315dd3472976d669ea31373a

possibly useful?


Raw data low/high since midnight

The updates look good, I noticed the raw data shows Fahrenheit temps for current but is showing Celsius.

Good catch.  Thanks!


ngrok not working in latest release

What is the issue?
ngrok http 11434 --host-header="localhost:11434"
OS
Linux
GPU
Nvidia
CPU
No response
Ollama version
latest-v0.3.12

Merging this with https://github.com/ollama/ollama/issues/3269


Override the doSetLastModifiedTime in FileObject

The method doSetLastModifiedTime in AbstractFileObject throws exception so anyone who tries to use it get an error. Please override this method.

Just merged your PR