Use implicit-hie to generate hie-*.yaml

implcit-hie is the project that HLS uses to generate a cradle for a
cabal project when you don't have a hie.yaml. You can also just use it
to generate literal hie.yaml files, which is what I've done here, for both
the Cabal and Stack versions.
This is therefore a halfway house towards not needing the
hie-*.yaml files: they're checked in, so we can edit them if necessary,
or see where the generated output goes wrong (I had to make a few changes
to .cabal files to work around bugs).
Becuase I think we might want to tweak the otuput, I'm not asserting that the
checked-in files are exactly the output of gen-hie in CI, but there is an update-hie
script that will regenerate them. So as long as we keep the structure
broadly the same, you should be able to update them by running update-hie
and doing a bit of judicious git add -p.
TODO: explain this in CONTRIBUTING.

Pre-submit checklist:

Branch

[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[ ] Relevant tickets are mentioned in commit messages

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

If you updated any cabal files or added Haskell packages:

[ ] $(nix-build default.nix -A pkgsLocal.updateMaterialized) to update the materialized Nix files
[ ] Update hie-*.yaml files if needed

If you changed any Haskell files:

[ ] $(nix-shell shell.nix --run fix-stylish-haskell) to fix any formatting issues

If you changed any Purescript files:

[ ] $(nix-shell shell.nix --run fix-purty) to fix any formatting issues

Pre-merge checklist:

[ ] Someone approved it
[ ] Commits have useful messages
[ ] Review clarifications made it into the code
[ ] History is moderately tidy; or going to squash-merge

To elaborate on why I don't just want to delete the files and let implicit-hie do its thing from inside HLS: there is a bug in implicit-hie (https://github.com/Avi-D-coder/implicit-hie/issues/24, fixed in an unreleased version), where comments in hs-source-dirs got into the output. If we didn't have the output checked in, it would be much harder to notice that and some components might just mysteriously not work properly, or you'd have to dig in the HLS logs.

Might just need a few comments where we use plugins, as otherwise the IDE displays large red lines everywhere because plugins crash the IDE. Otherwise: <3.

Might just need a few comments where we use plugins, as otherwise the IDE displays large red lines everywhere because plugins crash the IDE.

As of the newer versions of HLS it seems to work on those components for me now!


Consider only sort_param if the assoc param is not provided, not the drop_param.

The following commit message says it should consider the sort_param, without mentioning the drop_param: https://github.com/elixir-ecto/ecto/commit/afc694ce723f047e9fe7828ad16cea2de82eb217

The tests still pass when removing the check is_list(drop)

Adding the last 2 lines in the tests:

changeset = cast(%Author{}, %{sort: ["new"]}, :posts, opts)
assert Enum.map(changeset.changes.posts, & &1.changes[:title]) == [nil] # passes

changeset = cast(%Author{}, %{drop: [0]}, :posts, opts)
assert Enum.map(changeset.changes.posts, & &1.changes[:title]) == [nil] # doesn't pass

raises with message

** (KeyError) key :posts not found in: %{}

Based on these 3 observations, I suspect the check is_list(drop) shouldn't be there.

Can you please add the failing test that is failing? I suspect we should correct it to not raise, instead of discarding the option.

Could you have a look now @josevalim ?


Cloudflared should be included

Problem/Motivation
Noticed cloudflared is not included in the addon. This is probably one of the easiest ways a user can setup a reverseproxy, especially within homeassistant. While its not required, it would lower the barrier to entry.

Expected behavior

(What you expected to happen)

Actual behavior

(What actually happened)

Steps to reproduce

Install a fresh copy of the addon
setup your admin account
navigate to reverse proxy setting

Proposed changes
Add a working instance of cloudflared

Sorry, not planning no adding that. If you want to run a reverse proxy, one can use an dedicated add-on to do so.
../Frenck


Whisper: Add pip distribution configuration to support pip installations.

Currently the whisper module can only be used in the current project, and there are some difficulties when other projects want to build on it.
For example, some of the possible actions are to copy the source code into their own project or to publish it in a separate package (e.g. lightning-whisper-mlx), which will make it difficult for them to benefit from the changes made to this repository when the whisper module is updated. is updated, it is difficult for them to take advantage of the benefits of the whisper module.
In addition to that, it could be used as a separate command-line tool, like openai whisper.

install

pip install mlx_whisper

use

import mlx_whisper

result = mlx_whisper.transcribe(
    audio="~/Desktop/openai_tts.mp3",
    path_or_hf_repo="mlx_models/tiny_fp16/",
)

print(result["text"])

@awni  Currently my local release works, but there are some paths in the code that are written dead, so there are changes that need to be made if I want to use it. I'd like to know if this part of the current PR changes will be accepted so I can decide if I want to go ahead and commit.

but there are some paths in the code that are written dead,

What do you mean by that?
I think we could accept this change, it's relatively straightforward and seems pretty usefule. We'd (as in MLX maintainers) should manage the PyPi pacakage directly if we are going to distribute it through MLX Examples, so I will put it on PyPi once this is merged.

What do you mean by that?

@awni  After I release mlx_whisper and run the code in another directory, I encounter the following problem:
python mlx-w.py
Traceback (most recent call last):
  File "/Users/madroid/Desktop/pdf/mlx-w.py", line 4, in <module>
    result = transcribe(
             ^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/mlx_whisper/transcribe.py", line 150, in transcribe
    mel = log_mel_spectrogram(audio, n_mels=model.dims.n_mels, padding=N_SAMPLES)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/mlx_whisper/audio.py", line 167, in log_mel_spectrogram
    filters = mel_filters(n_mels)
              ^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/mlx_whisper/audio.py", line 96, in mel_filters
    return mx.load(filename)[f"mel_{n_mels}"]
           ^^^^^^^^^^^^^^^^^
ValueError: [load_npz] Input must be a zip file or a file-like object that can be opened with zipfile.ZipFile

After I located it, I found that the error occurred in the access to the  mel_filters.npz file in the audio.py file as follows:
https://github.com/ml-explore/mlx-examples/blob/5513c4e57dd78bf0d929b49ea2818ee0ac9644b2/whisper/whisper/audio.py#L95
Maybe I should package the contents of the assets file as well, but files like ls_test.flac don't seem to need to be packaged either. I'm not familiar with this part, can you give some advice?

Maybe I should package the contents of the assets file as well, but files like ls_test.flac don't seem to need to be packaged either. I'm not familiar with this part, can you give some advice?

Yes I think we'd need to package the mel filters. I think you can use either MANIFEST.in or package_data to include it in the distribution, I'm not sure which is the preferred route these days 😅

I chose the package_data approach because it looks like he modified less. Now the local package can be run successfully from any directory.


Implement Help() functions for all objects

ExperimentTopology, Node, etc. need help() functions

Looking at it they already have help() support. You can do help() on any one of them and get an extended documentation. I'm not sure what is being asked for @paul-ruth
help(fu.Node)
Help on class Node in module fim.user.node:
class Node(fim.user.model_element.ModelElement)
 |  Node(*, name: str, node_id: str = None, topo: Any, etype: fim.user.model_element.ElementType = <ElementType.EXISTING: 1>, ntype: fim.slivers.network_node.NodeType = None, site: str = None, **kwargs)
 |
 |  A basic node of any topology. In addition to public methods the following calls
 |  return various dictionaries or lists:
 |  node.components - returns a dictionary of components by name
 |  node.interfaces - returns a dictionary of interfaces by name
 |  interface_list - returns a list of interfaces
 |  direct_interfaces - returns a dictionary of direct interfaces, i.e. those
 |  not attached to components (mostly relevant to switches, not servers
 |  or VMs)
 |  network_services - returns a dictionary of network_services (mostly relevant
 |  to switches, not servers or VMs)
 |
 |  Method resolution order:
 |      Node
 |      fim.user.model_element.ModelElement
 |      abc.ABC
 |      builtins.object
 |
 |  Methods defined here:
 |
 |  __getattr__(self, item)
 |      Special handling for attributes like 'components' and 'interfaces' -
 |      which query into the model. They return dicts and list
 |      containers. Modifying containers does not affect the underlying
 |      graph mode, but modifying elements of lists or values of dicts does.
 |      :param item:
 |      :return:
 |
 |  __init__(self, *, name: str, node_id: str = None, topo: Any, etype: fim.user.model_element.ElementType = <ElementType.EXISTING: 1>, ntype: fim.slivers.network_node.NodeType = None, site: str = None, **kwargs)
 |      Don't call this method yourself, call topology.add_node()
 |      node_id will be generated if not provided for experiment topologies
 |
 |      :param name:
 |      :param node_id:
 |      :param topo:
 |      :param etype: is this supposed to exist or new should be created
 |      :param ntype: node type if it is new
 |      :param site: node site
 |      :param kwargs: any additional properties
 |
 |  __repr__(self)
 |      Return repr(self).
 |
 |  __str__(self)
 |      Return str(self).
 |
 |  add_component(self, *, name: str, node_id: str = None, ctype: fim.slivers.attached_components.ComponentType, model: str, network_service_node_id: str = None, interface_node_ids=None, interface_labels=None, **kwargs) -> fim.user.component.Component
 |      Add a component of specified type, model and name to this node. When working with substrate
 |      topologies you must specify the network_service_node_id and provide a list of interface node ids.
 |      :param name:
 |      :param node_id:
 |      :param ctype:
 |      :param model:
 |      :param name:
 |      :param network_service_node_id: network service identifier for substrate models
 |      :param interface_node_ids: interface identifiers for substrate models
 |      :param interface_labels: list of labels for interfaces in substrate models
 |      :param kwargs: additional properties of the component
 |      :return:
 |
 |  add_network_service(self, *, name: str, node_id: str = None, nstype: fim.slivers.network_service.ServiceType, **kwargs) -> fim.user.network_service.NetworkService
 |      Add a network service to node (mostly needed in substrate topologies)
 |      :param name:
 |      :param node_id:
 |      :param layer:
 |      :param kwargs:
 |      :return:
 |
 |  get_component(self, name: str)
 |      Get a component by this name
 |      :param name:
 |      :return:
 |
 |  get_property(self, pname: str) -> Any
 |      Retrieve a node property
 |      :param pname:
 |      :return:
 |
 |  remove_component(self, name: str) -> None
 |      Remove a component from the node (and network services and their interfaces)
 |      :param name:
 |      :return:
 |
 |  remove_network_service(self, name: str) -> None
 |      Remove a network service from the node (and all its interfaces)
 |      :param name:
 |      :return:
 |
 |  set_properties(self, **kwargs)
 |      Set multiple properties of the node
 |      :param kwargs:
 |      :return:
 |
 |  set_property(self, pname: str, pval: Any)
 |      Set a node property
 |      :param pname:
 |      :param pval:
 |      :return:
 |
 |  ----------------------------------------------------------------------
 |  Static methods defined here:
 |
 |  list_properties() -> Tuple[str]
 |
 |  ----------------------------------------------------------------------
 |  Data and other attributes defined here:
 |
 |  __abstractmethods__ = frozenset()
 |
 |  ----------------------------------------------------------------------
 |  Methods inherited from fim.user.model_element.ModelElement:
 |
 |  __eq__(self, other)
 |      Return self==value.
 |
 |  __hash__(self)
 |      Return hash(self).
 |
 |  rename(self, new_name: str)
 |      Rename the model element in the model graph
 |      :param new_name:
 |      :return:
 |
 |  unset_property(self, pname: str)
 |      Unset a property
 |      :param pname:
 |      :return:
 |
 |  ----------------------------------------------------------------------
 |  Data descriptors inherited from fim.user.model_element.ModelElement:
 |
 |  __dict__
 |      dictionary for instance variables (if defined)
 |
 |  __weakref__
 |      list of weak references to the object (if defined)

Maybe this was added already. They asked about "ExperimentTopology". I know I tried it a while back and it didn't have help. Maybe this one is done.

This seems done. closing


TwoPaneLayoutNav can find route for pane1 but not for pane2

Hi team,
long time no see but here I'm with another problem which is properly caused by not understanding the API.
I use the version:
implementation("com.microsoft.device.dualscreen:twopanelayout:1.0.1-alpha06")

My code
Based on the tutorial, I tried to get my TwoPaneLayoutNav working.
class MainActivity : ComponentActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContent {
            val navController = rememberNavController()

            TwoPaneLayoutNav(
                navController = navController,
                singlePaneStartDestination = "regions",
                pane1StartDestination = "regions",
                pane2StartDestination = "regions"
            ) {
                composable("regions") {
                    RegionsScreen(
                        navigateToRegion = { id -> navController.navigate("regions/${id}")}
                    )
                }
                composable("regions/{id}") { backStackEntry ->
                    val id = backStackEntry.arguments?.getString("id")
                    RegionScreen(regionId = id ?: "A")
                }
            }
        }
    }
}

*The problem *
But as soon as I span the app across both screens, the app crashes with the following error:
java.lang.IllegalArgumentException: Invalid route regions, not present in list of routes []

I tried not just regions but regions/a, too. Both does not work. If I introduce another path like "test" is also does not work. In single screen mode, everything works as expected.
Thanks for your help!

I had the same error, make sure you are importing composable from:
import com.microsoft.device.dualscreen.twopanelayout.twopanelayoutnav.composable
and not from:
import androidx.navigation.compose.composable

Hi @tscholze, thanks for continuing to use our SDK and provide feedback 😄
And thank you @Papes96 for the tip, yes you definitely want to make sure you're importing the correct composable definition so all routes are added to the twoPaneBackStack. I would also recommend that you call navigateTo instead of navigate in your navigateToRegion function so it will work in both single and dual screen mode!
Tobi, if that doesn't fix your problem, please let me know and provide the code for your RegionsScreen and RegionScreen! That way I can help you debug the issue some more.

@Papes96 thanks, that was the fix! <3
@khalp I'm now a step further into the process to adopting the app to the surface duo.
I may come back to the technet community with another question :)


✨ Add a mechanism to connect external devices to a video live

Purpose
We want to allow pairing external devices to Marsha live. External devices are cameras or set-top boxes that can act as video and sound sources for a live in Marsha.
This PR should be merged after #1213

[x] Update frontend actionOne to permit

usage of any request method
usage of empty body request

[x] Add a frontend query to request /api/video/<id>/pairing-secret/
[x] Add a frontend component which:

request a secret
displays the secret
displays a countdown timer initially set to 1 minutes

#1196

@kernicPanel pair-secret`

@kernicPanel I'm not sure the endpoint /pair-secret and /pair-challenge are good paths. Maybe /pairing-secret and /pairing-challenge are more explicit (because pair makes you think of a pair of things, not about the pairing mechanism).

@kernicPanel I'm not sure the endpoint /pair-secret and /pair-challenge are good paths. Maybe /pairing-secret and /pairing-challenge are more explicit (because pair makes you think of a pair of things, not about the pairing mechanism).

Yes, I'll change that

Once the secret expired the button looks like this. We can maybe use a toast to inform the user that the button has expired ? Here I think there is too much information in the same button.

Yes, I totally agree with you @lunika.
But I'm not sure we should use a toast here, as all visual feedback are located in the button.
Maybe we can display Pairing secret expired for few seconds in the button, and Pair an external device after ?
WDYT ?

Once the secret expired the button looks like this. We can maybe use a toast to inform the user that the button has expired ? Here I think there is too much information in the same button.

Yes, I totally agree with you @lunika. But I'm not sure we should use a toast here, as all visual feedback are located in the button. Maybe we can display Pairing secret expired for few seconds in the button, and Pair an external device after ? WDYT ?

Or we can also move all feedbacks in a toast. I'll try this, if you agree @lunika

PR OK for another review @lunika @carofun @sampaccoud


Addition of serotypes O15 and O17

Serotypes O15 and O17 have been added. Similar to the other serotypes already in this tool, the reference genome for O15 and O17 were determined using the paper from S. Thrane et. al..
The exact location of the O15 and O17 clusters can be found under the Bioinformatics analysis in this paper.
Tests have been added for the new serotypes, and other tests were re-run.
If any additional changes are required, please do let me know.

Hi @StefaanVerwimp
This is beautiful, thank you so much for adding these!
Cheers,
Robert

@StefaanVerwimp would you mind sending me an email (its on my github profile)
I'm working on a paper for camlhmp which will include pasty, I would like to include you for your contributions to pasty (if that is ok)

Hi Robert,
Thanks for such a positive response! I will send you an email a bit later today.


Add beginner friendly web3 project ideas

Hey @kumar-kalyan can i work on this?

Go ahead
On Sun, 2 Oct, 2022, 1:12 pm Claire-me, @.***> wrote:

Hey @kumar-kalyan https://github.com/kumar-kalyan can i work on this?
—
Reply to this email directly, view it on GitHub
https://github.com/hackbeats/complete-web3-bootcamp/issues/3#issuecomment-1264575921,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP7W3ZTNLGMXVPFHRMGA3TDWBE4EPANCNFSM6AAAAAAQ2TZUGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey! I would love to work on this,can I contribute too?

I would like to contribute as well

Go ahead
On Fri, 7 Oct, 2022, 1:18 am Ayushi George, @.***>
wrote:

I would like to contribute as well
—
Reply to this email directly, view it on GitHub
https://github.com/hackbeats/complete-web3-bootcamp/issues/3#issuecomment-1270600378,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP7W3ZRFQPDM35LVXLY63CTWB4UHXANCNFSM6AAAAAAQ2TZUGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @kumar-kalyan can I contribute too?

Go ahead
On Sat, 8 Oct, 2022, 1:08 am Sayyad Azhar, @.***> wrote:

Hey @kumar-kalyan https://github.com/kumar-kalyan can I contribute too?
—
Reply to this email directly, view it on GitHub
https://github.com/hackbeats/complete-web3-bootcamp/issues/3#issuecomment-1272025647,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP7W3ZUJ6UHP5WWXIA5L423WCB33ZANCNFSM6AAAAAAQ2TZUGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Cloud Sync Provisioning Agent and Password Hash shows 20 min interval instead of 2 mins

This documentation is stating that provisioning and password hash syncing should be on a 2 minute interval but running the PowerShell command Get-AADCloudSyncToolsJobSchedule with the AADCloudSyncTools module shows an output of a 20 minute interval. I'm seeing the same output in multiple environments. I'm also seeing changes and passwords taking 15 mins or longer to provision to the cloud object when changed on-prem (with no AD replication delays and in a single DC environment.)
PS > Get-AADCloudSyncToolsJobSchedule |select intervalmins, state
intervalMins state

      20 Active
      20 Active

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

ID: 20a5ec41-cc7d-d0b8-ebe7-6cf12e955ee1
Version Independent ID: 70633e75-f084-4d3b-dada-750aedc40776
Content: Azure AD Connect cloud sync deep dive - how it works - Microsoft Entra
Content Source: articles/active-directory/cloud-sync/concept-how-it-works.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@BRRiley
Thanks for your feedback! We will investigate and update as appropriate.

Thank you for your issue submission. Unfortunately, this appears to be a problem with the cmdlt and a support issue. Issues in this repo are for documentation issues. You can try one of the following links for additional help.
Microsoft Support: https://support.microsoft.com/
Azure AD forums: https://social.technet.microsoft.com/Forums/en-US/home?forum=windowsazureaditpro
Stackoverflow: https://stackoverflow.com/
#please-close


Web App Manifest shipped in Edge 17

https://github.com/MicrosoftEdge/Status/pull/599
https://developer.microsoft.com/en-us/microsoft-edge/platform/status/webapplicationmanifest/

Thanks!


misc/cgo/testcarchive: TestSIGPROF failure with "signal: profiling timer expired" on WSL builder

2021-01-14T22:01:23-e125ccd/linux-amd64-wsl
##### ../misc/cgo/testcarchive
--- FAIL: TestSIGPROF (1.25s)
    carchive_test.go:769:
    carchive_test.go:770: signal: profiling timer expired
FAIL

See previously #19320.
It's not clear to me whether this failure is due to a problem in the test, in the runtime, or in WSL itself.
CC @ianlancetaylor @mengzhuo @oiooj

@bcmills WSL builder’s HTTS PROXY is so slowly now, I'm working it now.

@bcmills WSL builder’s HTTS PROXY is so slowly now, I'm working it now.

The network problem solved and builders online.

The network problem solved and builders online.


Dynamically setting height for carousel (for enforcing a certain aspect ratio)

Is there a way to force the carousel to use a certain aspect ratio? At the moment, the carousel adjusts its height to the height of the active image, so if i have images with varying heights, the carousel will cause the entire webpage to shrink/expand depending on what image is being rendered.
I've seen css solutions where people would add a fixed height, but this is a problem when viewing the images on varying screen sizes. For instance, setting height: 500px would show a lot of empty space on a smart phone screen, but may display fine for desktop browsers.
The only way I can think of to enforce a 16:9 aspect ratio is to do the height calculation myself. In jQuery it might be something like:
$(window).resize(function() {
  var new_height = (current_width * 9)/16;
  $(".carousel").height(new_height);
});

Is there an equivalent way of dynamically setting the height for an angular-ui carousel on a window resize event?

https://github.com/angular-ui/bootstrap#need-help


fix: range expressions must not start with \v

Fixes #3610

The lint failure is expected. We first need to merge https://github.com/coreruleset/crs-toolchain/pull/137.

@superlgn, @airween Could you test the fixed expressions in this PR? \v has now been replaced with \0xb in character classes.

@superlgn, @airween Could you test the fixed expressions in this PR? \v has now been replaced with \0xb in character classes.

I pulled your repo and tried it with my Apache2, everything looks fine. I can take a look in detail tomorrow (test few regexes with my tool).

I run a test with my tool against all regexes, and looks everything is fine.
Please fix the lint error, and I can merge it.

Thanks @airween ❤!

Don't forget to add a remark in CONTRIBUTING.md or somewhere.

Don't forget to add a remark in CONTRIBUTING.md or somewhere.

About what? We simply modified the automation, nothing to be done by devs manually.

Ah, sorry. I thought you had changed the source regexes. But true, the change would include the ra files then. Did not connect the dots.


/config/auth page added support for quickly switching PathAuth status…

/config/auth page added support for quickly switching PathAuth status in the list

Related: apache/shenyu#5488

Please resolve the conflicts. : )

Please resolve the conflicts. : )
resolved


Add message that mentions 2% fee when receiving funds

Note:
Adding this 2% fee is already an experiment - we will have to closely evaluate (by getting into contact with users) if the fee is a blocker for some users.
@bonomat @klochowicz I this fee is an initial blocker for users we should definitely experiment to charge at a later point (e.g. when trading, or if the channel was not used for some time, ...). Please think through if we can derive the fee being a blocker or not from the data we have available.
I think we should be able to correlate the app being opened by a users and the user sending funds. It would be great to also have stats on which users succeeded to get funds in and which users did not.

Let's rather wait for them to complain.


boxy fails to compile

Hello,
While testing out the Bareflank capabilities for possible use in a project, I wanted to compile up Boxy on my Debian 9 (x86_64) system and while it seems that most things were  compiling, I just ran into some errors:
==================================
Scanning dependencies of target boxy_domain
[  4%] Building CXX object src/domain/CMakeFiles/boxy_domain.dir/domain.cpp.o
[  8%] Linking CXX static library libboxy_domain.a
Scanning dependencies of target boxy_hve
[ 17%] Building CXX object src/hve/CMakeFiles/boxy_hve.dir/arch/intel_x64/virt/vclock.cpp.o
[ 17%] Building CXX object src/hve/CMakeFiles/boxy_hve.dir/arch/intel_x64/emulation/cpuid.cpp.o
[ 26%] Building CXX object src/hve/CMakeFiles/boxy_hve.dir/arch/intel_x64/emulation/x2apic.cpp.o
[ 26%] Building CXX object src/hve/CMakeFiles/boxy_hve.dir/arch/intel_x64/emulation/mtrr.cpp.o
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:319:23: error: 'tsc' is not a class, namespace, or enumeration
auto tsc = ::x64::tsc::get();
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:319:10: note: 'tsc' declared here
auto tsc = ::x64::tsc::get();
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:340:23: error: 'tsc' is not a class, namespace, or enumeration
auto tsc = ::x64::tsc::get();
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:340:10: note: 'tsc' declared here
auto tsc = ::x64::tsc::get();
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:374:27: error: 'tsc' is not a class, namespace, or enumeration
if (auto tsc = ::x64::tsc::get(); tsc < next_event) {
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:374:14: note: 'tsc' declared here
if (auto tsc = ::x64::tsc::get(); tsc < next_event) {
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:408:35: error: no member named 'tsc' in namespace 'x64'
m_next_event_tsc = ::x64::tsc::get() + vcpu->rbx();
~^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:572:27: error: 'tsc' is not a class, namespace, or enumeration
if (auto tsc = ::x64::tsc::get(); tsc < m_next_event_tsc) {
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:572:14: note: 'tsc' declared here
if (auto tsc = ::x64::tsc::get(); tsc < m_next_event_tsc) {
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/virt/vclock.cpp:610:9: error: no matching constructor for initialization of
'const vcpu_delegate_t &' (aka 'const delegate<void (bfobject *)> &')
{&vclock_handler::resume_delegate, this}
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfdelegate.h:303:5: note: candidate template ignored: could not match 'bfobject *' against
'bfvmm::intel_x64::vcpu *'
delegate(Ret(C::*memfn)(Args...), C *obj)
^
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfdelegate.h:343:5: note: candidate template ignored: could not match
'void (bfobject *) const' against 'void (bfvmm::intel_x64::vcpu *)'
delegate(Ret(C::*memfn)(Args...) const, C *obj)
^
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfdelegate.h:380:14: note: candidate template ignored: could not match
'void (bfobject *) const' against 'void (bfvmm::intel_x64::vcpu *)'
delegate(Ret(C::*memfn)(Args...) const, const C *obj)
^
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfdelegate.h:268:5: note: candidate constructor template not viable: requires single
argument 'fn', but 2 arguments were provided
delegate(F fn)
^
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfdelegate.h:234:5: note: candidate constructor not viable: requires single argument 'fn',
but 2 arguments were provided
delegate(Ret(*fn)(Args...))
^
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfdelegate.h:396:5: note: candidate constructor not viable: requires single argument
'other', but 2 arguments were provided
delegate(const delegate &other) :
^
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfdelegate.h:405:5: note: candidate constructor not viable: requires single argument
'other', but 2 arguments were provided
delegate(delegate &&other) :
^
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfdelegate.h:209:5: note: candidate constructor not viable: requires 0 arguments, but 2
were provided
delegate() : m_call{nullptr}, m_vtbl{nullptr}
^
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfvmm/hve/arch/intel_x64/vcpu.h:143:61: note: passing argument to parameter 'd' here
VIRTUAL void add_resume_delegate(const vcpu_delegate_t &d) noexcept
^
/home/lonnie/Cloud/Type-2/boxy/bfvmm/src/hve/arch/intel_x64/emulation/x2apic.cpp:45:17: error: no member named 'is_host_vcpu' in namespace 'vcpuid'; did you
mean 'is_host_vm_vcpu'?
if (vcpuid::is_host_vcpu(vcpu->id())) {
^~~~
is_host_vm_vcpu
/home/lonnie/Cloud/Type-2/boxy/build/prefixes/x86_64-vmm-elf/include/bfvcpuid.h:60:27: note: 'is_host_vm_vcpu' declared here
constexpr inline bool is_host_vm_vcpu(type id)
^
6 errors generated.
make[5]: *** [src/hve/CMakeFiles/boxy_hve.dir/build.make:102: src/hve/CMakeFiles/boxy_hve.dir/arch/intel_x64/virt/vclock.cpp.o] Error 1
make[5]: *** Waiting for unfinished jobs....
1 error generated.
make[5]: *** [src/hve/CMakeFiles/boxy_hve.dir/build.make:89: src/hve/CMakeFiles/boxy_hve.dir/arch/intel_x64/emulation/x2apic.cpp.o] Error 1
make[4]: *** [CMakeFiles/Makefile2:182: src/hve/CMakeFiles/boxy_hve.dir/all] Error 2
make[3]: *** [Makefile:118: all] Error 2
make[2]: *** [CMakeFiles/boxy_bfvmm_x86_64-vmm-elf.dir/build.make:113: boxy_bfvmm/x86_64-vmm-elf/stamp/boxy_bfvmm_x86_64-vmm-elf-build] Error 2
make[1]: *** [CMakeFiles/Makefile2:3046: CMakeFiles/boxy_bfvmm_x86_64-vmm-elf.dir/all] Error 2
make: *** [Makefile:84: all] Error 2
lonnie@debian:~/Cloud/Type-2/boxy/build$
==================================
Any ideas on this?

Hello, a previous commit came with changes to the hypervisor and the hypervisor submodule couldn't be updated until the hypervisor repository had the PR merged.
Fix is here: update hypervisor submodule

I think this has been addressed with the latest pushes.


BUG-868079 Add the option to configure keystore and truststore type for Cassandra connection

@wonim2022 we need to capitalize the C in Cassandra in comments and the README.md file.


Project is dead ? No commits since 2 years ago on master branch

Is it abandonned ?

I'm still using Tabli, but it looks like the feature for avoiding the open of a duplicate tab is no longer working.  Using Chrome Version 119.0.6045.123 on the Mac desktop.

@AkechiShiro This project is fairly dormant, but not abandoned. I still use Tabli on a daily basis myself, but between a demanding day job and becoming a parent, I haven't had spare cycles to do more with it. I'd be happy to engage if other developers want to step up and offer to contribute to the project.
@coremania Interesting observation about duplicate tabs -- should probably go in a separate issue.


feat(testing): simulate multipart file upload

Summary of Changes
Draft of tests section for post-ing files as multipart/form-data. Parameter 'files' added to simulate_requests in the testing client, as well as a file test_multipart_formdata_request of tests.
Related Issues
#1010
Pull Request Checklist
This is just a reminder about the most common mistakes.  Please make sure that you tick all appropriate boxes.  But please read our contribution guide at least once; it will save you a few review cycles!
If an item doesn't apply to your pull request, check it anyway to make it apparent that there's nothing to do.

[x ] Applied changes to both WSGI and ASGI code paths and interfaces (where applicable).
[x ] Added tests for changed code.
[ x] Prefixed code comments with GitHub nick and an appropriate prefix.
[ x] Coding style is consistent with the rest of the framework.
[ x] Updated documentation for changed code.

[x ] Added docstrings for any new classes, functions, or modules.
[ x] Updated docstrings for any modifications to existing code.
[ x] Updated both WSGI and ASGI docs (where applicable).
[ x] Added references to new classes, functions, or modules to the relevant RST file under docs/.
[ x] Updated all relevant supporting documentation files under docs/.
[ x] A copyright notice is included at the top of any new modules (using your own name or the name of your organization).
[ x] Changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives.

[ ] Changes (and possible deprecations) have towncrier news fragments under docs/_newsfragments/, with the file name format {issue_number}.{fragment_type}.rst. (Run towncrier --draft to ensure it renders correctly.)

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
PR template inspired by the attrs project.

I used urllib3 and I added it to requirements, but it isn't found in the tests - where does it need to be added to be seen?

Hi again @TigreModerata, and thanks for this improvement :+1:
However, when it comes to urllib3, Falcon is designed to have no hard dependencies on any third part package, so we should ideally rewrite this functionality not to use urllib3 at all...
Alternatively, we could possibly catch an ImportError, and document that this optional functionality requires urllib3.

I'll try to write it out, it's probably good exercise :)

Yeah, it should be easy at least when it comes to basic functionality, check out tests/test_media_multipart.py where I mostly construct these forms by hand.

Yeah, it should be easy at least when it comes to basic functionality, check out tests/test_media_multipart.py where I mostly construct these forms by hand.

Now I  just string a body together with the info coming from files, but it's probably pretty ugly (but it works, at least).

Hi, I'm back, thanks for all the feedback! Getting back to work on this today, sorry for the long delay (holiday + new job).

Hi, I'm back, thanks for all the feedback! Getting back to work on this today, sorry for the long delay (holiday + new job).

Thanks for the update, I'll take a look this week.
(Good luck on your new job!)

I added the data arg to simulate_post, in such a way that if both json and (files or data) are present, a badrequest exception is thrown. If json is absent, then files and data will be processed: if files is also absent, data will be url-encoded.
I'm not sure I got all the requirements right, in particular the "body as an alias of data" (I didn't touch it because I don't know exactly how and I'm not sure if you wanted to leave that for a future version).
I also didn't remove the possibility for nested forms, just because the possibility exists in other multipart test files and I wasn't sure if you guys agreed to remove them... but it's a quick fix, if you confirm.
[I am really a beginner, so some of what you said may have gone over my head... I apologize. Since you guys are so responsive, this is a great help to me to learn/practice, I really appreciate your patience!]

(I'm in the process of terraforming this somewhat... )

Hi, did something happen, can I do anything to help? I just got a notification saying "mentioned" but I'm not sure why. I've not looked at this in ages, but I have time and I'm happy to try to contribute.

Hi @TigreModerata!
I just merged the master branch into your branch to start with.
I was thinking to update your PR, and to change files and data to form, similar to how json is handled. I can make this change, and you can continue from that point!

I was thinking to update your PR, and to change files and data to form, similar to how json is handled. I can make this change, and you can continue from that point!

I'm not totally sure what needs to be done, I need to refresh my memory a bit. But yeah, I'm happy if I can contribute!

I'm not totally sure what needs to be done, I need to refresh my memory a bit.
That's why I thought it would be a good opportunity to restart this from scratch, it doesn't need to be too complex in the first iteration. I was thinking we could simply add a form parameter that can be a dict or a list/tuple of key-value pairs. If the whole form is only simple key-values, we do a URL-encoded form, otherwise encode it as multipart, roughly how browsers do; what do you think?

I have written a simple proof-of-concept implementation, but it needs more validation, tests, documentation, and a newsfragment.

But yeah, I'm happy if I can contribute!
You are very welcome to!


add gdpr support in freewheel bidder adapter

Can you please pull the update to add gdpr support to our bidder adapter documentation ?
(Linke with this Prebid.js PR : https://github.com/prebid/Prebid.js/pull/2522#issuecomment-398064802 )

will be merged as part of 1.15


Generate alias-derived fields

Problem
Alias is a feature that allows you to change the result of a field to any name you want.

https://graphql.org/learn/queries/

This can be used to generate a response with a field that does not exist in Author type.
const query = graphql`
  query ExampleQuery @raw_response_type {
    author(id: "1") {
      id
      name
      name2 # alias
    }
  }
`;
const data = useClientQuery(query);
console.log(data);
// output:
// {
//   author: {
//     id: "1",
//     name: "Mikami Komata",
//     name2: "Mikami Komata",
//   },
// }

However, graphql-codegen-typescript-fabbrica cannot generate alias-derived fields. This makes it difficult to build responses for queries that use aliases.
const AuthorFactory = defineAuthorFactory({
  defaultFields: {
    id: dynamic(({ seq }) => `Author-${seq}`),
    name: "Komata Mikami",
  },
});

const author = await AuthorFactory.build();
//    ^? { id: string, name: string }

const dummyResponse: ExampleQuery$rawResponse = { author };
//    ^^^^^^^^^^^^^ error: author.name2 is missing

Solution
Allow defaultFields to accept alias-derived fields.
const AuthorFactory = defineAuthorFactory({
  defaultFields: {
    id: dynamic(({ seq }) => `Author-${seq}`),
    name: "Komata Mikami",
    name2: "Komata Mikami", // alias-derived field
  },
});

const author = await AuthorFactory.build();
//    ^? { id: string, name: string, name2: string }

const dummyResponse: ExampleQuery$rawResponse = { author }; // ok

Drawbacks
In the current implementation of graphql-codegen-typescript-fabbrica, fields not included in type are treated as transient fields. transient fields are not included in the built data.

https://github.com/mizdra/graphql-codegen-typescript-fabbrica/tree/9ac221aaefe886097b2fcc732fc1493f467e49f9#transient-fields

Therefore, to implement this proposal, we have to change the interface of transient fields.

graphql-codegen-typescript-fabbrica excludes properties other than field (__typename, id, name) of its type from the build result.

https://stackblitz.com/edit/playground-graphql-codegen-typescript-fabbrica-nvazzz?file=__generated__%2Ffabbrica.ts%3AL17,src%2Findex.test.ts&view=editor
https://stackblitz.com/edit/playground-graphql-codegen-typescript-fabbrica-nvazzz?file=__generated__%2Ffabbrica.ts%3AL38,src%2Findex.test.ts,node_modules%2F%40mizdra%2Fgraphql-codegen-typescript-fabbrica%2Fdist%2Fhelper%2Ffactory.d.ts,node_modules%2F%40mizdra%2Fgraphql-codegen-typescript-fabbrica%2Fdist%2Fhelper%2Findex.d.ts&view=editor
https://github.com/mizdra/graphql-codegen-typescript-fabbrica/blob/7b14060b5ebd54e33a7b7af62e9b056ce9c86f43/src/helper/factory.ts#L67
https://github.com/mizdra/graphql-codegen-typescript-fabbrica/blob/077a10bf0df0c8a04c78d2009085dd403ae4612f/src/helper/field-resolver.ts#L103

So, if you pass a field such as name2 to the build function, it will not be included in the build result. This limitation is implemented for transient fields.
Therefore, implementing alias-derived fields requires a change in the implementation of transient fields.

This feature supports any additional fields, not just fields for aliases.
I think the feature should be renamed from "alias fields" to "additional fields".


How to calculate PSNR when I call the EncodeFrame (const SSourcePicture* kpSrcPic, SFrameBSInfo* pBsInfo) function?

As encoder does not output reconstructed frame by default, one way to calculate PSNR is to use decoder to get the decoded YUV, and use PSNR tools to compare with the input original YUV data.

close this since answered and no update from asker.


support mounting pvc for job deployer

What this PR does / why we need it:
Support mount pvc to job deployer, so the trained model can be saved in the PVC derectly.
Which issue(s) this PR fixes :
Fixes #324
Special notes for your reviewer:
From muy initial thoughts, that can be implemented by pod_spec_mutators, but checked the adding mechanism for pod_spec_mutators, that's hard to pass the args, but the pvc_name should be pass to job, so I updated the Job class to let user input pvc_name and pvc_mount (optional, defaults to /mnt).
Release note:

support mounting pvc for job deployer

This change is

Please checkout my comments on #328 .

@karthikv2k @abhi-g Any more comments? thanks.

/lgtm
/approve


Require the use of remappings provided via --with

This is a quick first attempt to address #841.
Alternatively I could update the Generator so that finish returns an error to eliminate panicking but that would require updating all the language generators for a change currently isiolated to the rust generator.

Thanks for this! This repository isn't really equipped to test this right now (e.g. via something like the trybuild crate) but that's ok. Can you confirm locally though that the error message looks reasonable when using the generate! macro?

Manually confirmed locally the error message looks reasonable when failing with the generate! macro:


Add aarch

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Seems to work with https://github.com/conda-forge/conda-smithy/pull/1751

One caveat: This will break automerge as that will rerender with the conda-smity release version.

Should we wait until https://github.com/conda-forge/conda-smithy/pull/1751 is released?

Sure, if the conda-smithy PR takes longer than a couple of weeks to merge we can also just setup the swap by hand and not have to manually rerender for eternity.

Nice work <3


ImportError: No module named setuptools

Hi Sandro , i followed those steps
hany@HP-Pro:~/toolz$ sudo git clone https://github.com/sandrogauci/wafw00f
Cloning into 'wafw00f'...
remote: Counting objects: 454, done.
remote: Total 454 (delta 0), reused 0 (delta 0), pack-reused 454
Receiving objects: 100% (454/454), 124.68 KiB | 153.00 KiB/s, done.
Resolving deltas: 100% (252/252), done.
Checking connectivity... done.
hany@HP-Pro:~/toolz/wafw00f$ sudo python setup.py install
Traceback (most recent call last):
File "setup.py", line 4, in
from setuptools import setup, find_packages
ImportError: No module named setuptools
Thanks

you'll need to get setuptools installed. https://pypi.python.org/pypi/setuptools


Cas 13778

Hi Ben, thanks a lot for these large number of fixes. I focused on the rendering of the information within CASA Docs  (https://casadocs.readthedocs.io/en/cas-13778/), and that looks very good. So without fully understanding the details of the sphinx parsing errors (hopefully Ryan can review that), I am signing off my  review on this. Thanks again!!

(...which I now realize we probably already did by email!)


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 92862bb.


Delete cloudflare image

Description:
Suddenly now when i delete the image i'm getting the error
Incorrect CloudFlare Key Or Account ID provided. Please check CLOUDFLARE_IMAGES_ACCOUNT and CLOUDFLARE_IMAGES_KEY to your .env
although  delete image functionality is working. on deleting image is removed from the cloudflare but i don't know why i'm getting above error.
i'm using the dedmytro/laravel-cloudflare-images => 0.1.9

it's fixed after updating the package


x/pool-incentives: Make FundCommunityPoolFromModule call distr.FundCommunityPool

The current FundCommunityPoolFromModule function in pool-incentives should either be upstreamed to the SDK's distribution module, or we should derive the SDK address for this, and call distr.FundCommunityPool.
We derive the SDK address from a module name via:
addr := k.accountKeeper.GetModuleAddress(moduleName)
if addr == nil {
  error or panic
}

@ValarDragon what exactly are you proposing we upstream to x/distribution?

The FundCommunityPoolFromModule function as a whole. The problem we faced was we wanted to send coins to the community pool from a module account, and in this code we basically hackily did updates to the Distr.FeeTokens struct to handle this.

Got it!

Oh true. I guess its a yak shaving question of do we want to get this into the SDK, versus just use more black box functionality  here. (converting sending address to a normal address rather than module address)


Summary table does not match model zoo id format

In sfaira.train.summaries.py line 1163, we create the summary table by parsing the model id:
def create_summary_tab(self):
        metrics = list(self.evals.values())[0]['val'].keys()
        self.summary_tab = pandas.DataFrame(dict(
            list({
                "depth": [id_i.split("_")[self.model_id_len + 0] for id_i in self.run_ids],
                "width": [id_i.split("_")[self.model_id_len + 1] for id_i in self.run_ids],
                "lr": [id_i.split("_")[self.model_id_len + 2] for id_i in self.run_ids],
                "dropout": [id_i.split("_")[self.model_id_len + 3] for id_i in self.run_ids],
                "l1": [id_i.split("_")[self.model_id_len + 4] for id_i in self.run_ids],
                "l2": [id_i.split("_")[self.model_id_len + 5] for id_i in self.run_ids],
                "cv": [id_i.split("_")[-1] if self.cv else "1" for id_i in self.run_ids],
                "model": ["_".join(id_i.split("_")[:self.model_id_len]) for id_i in self.run_ids],
                "organ": [id_i.split("_")[2] for id_i in self.run_ids],
                "model_type": [id_i.split("_")[3] for id_i in self.run_ids],
                "model_gs_id": ["_".join(id_i.split("_")[:(self.model_id_len + 6)]) for id_i in self.run_ids],
                "run": self.run_ids,
            }

This does not work in sfaira-dev because there is a constraint on the model id which does not conform to the above format (sfaira.interface.model_zoo.py, line 125):
@model_id.setter
    def model_id(self, x: str):
        """
        Set model ID to a manually supplied ID.

        :param x: Model ID to set. Format: pipeline_genome_organ_model_organisation_topology_version
        """
        assert len(x.split('_')) == 3, f'model_id {x} is invalid'
        self._model_id = x

We have to change one of these.

The summaries are currently being updated inside of jupyter notebooks with grid search from the new code, I will carry them over as soon as they function again!


Add JetStream metadata to msg parsed from reply string

Hey
This PR adds the sequence and DateTime info to the msg.
Any feedback is appreciated.

Thank you @simonhoss great to see this because it's implementing a missing feature in JetStream.
My initial thoughts are:

We don't want to penalize applications which are not using JetStream but using the reply subject for other patterns of messaging,
The feature really belongs to JetStream project,
NatsMsg is a struct so we need to be careful about enlarging it,
Have you considered calculating it on demand? (I'll also check with other client maintainers see how they're doing it)

Thanks for the feedback. When I started the implementation I didn't realize that there are two msg's in (NatsMsg and NatsJSMsg). With this new knowledge and your feedback, I was thinking about to implement it only in the NatsJSMsg with a lazy implementation. That means that the Seq and DateTime gets only calculated if someone wants to know it.
What do you think?

Brilliant! Is there a way we can retrieve the new properties on NatsJSMsg without storing them on NatsMsg? Because NatsMsg is a struct it means every message needs to be copied so we want to keep it as small as possible 😉
BTW @simonhoss please sign your commits

I added a new approach, which doesn't touch the NatsMsg and only calculates the Sequence and DateTimewhen someone really consumes it.

New version pushed. I think it should now parse both versions correct.
Also, Im not quite sure if struct makes sense for NatsJSMsgMetadata, but NatsJSMsg is also a struct so I took the same.

Also @simonhoss please feel free to join https://slack.nats.io #dotnet channel if you haven't already done so. I'd love to see you there too 💯

Thanks a lot!


charging status notifier

Which files sir?
On Thu, Apr 9, 2020, 6:42 PM prince-09<EMAIL_ADDRESS>wrote:

@prince-09 requested changes on this pull request.
Some files are not added.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hackslash-nitp/habita-android/pull/18#pullrequestreview-390778049,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANJODIG5LCBPRPMPSLUZVBTRLXCU7ANCNFSM4MDAIBLA
.


Fix portals working slow in survival

For some reason git thinks I changed the whole ViaVersionPlugin.java, but I just registered a new listener.

Need to check if nether is enabled/Disabled in spigot.yml

Not going to pull quite yet as I feel this fix feels to complex and there should be a simpler way of fixing as we must be doing something stupid.

+1 you should collapse your catch blocks and also asynchronize your iterating through all players because this heavily blocks the main thread


Don't stomp on *DependsOn properties

This allows the property to be set within a project file without its value being replaced by the imported .targets at the bottom. Particularly in SDK style projects where properties cannot be set after the imported .targets file, this unblocks a key scenario.

/azp run

A few notes here:

This is a breaking change for users that have defined something that has been up 'til now ignored/overwritten. Hopefully that's rare?
It's not true that you can't overwrite these when using SDKs; you can desugar the SDK reference, or you can run an InitialTarget to change dependencies.

I'm willing to try this but we may need to back it out if we get user pain from it.

It's not true that you can't overwrite these when using SDKs; you can desugar the SDK reference

That's not an option that nuget packages that contribute build authoring can make. They get imported wherever they get imported. And when I was testing, my nuget package got imported earlier than this stomping property definition.

or you can run an InitialTarget to change dependencies.

Yes, this is what I ended up doing.

I'm willing to try this but we may need to back it out if we get user pain from it.

Totally reasonable. And thank you.

I stumbled upon this recently when I tried CoreCompileDependsOn for VisualBasic and F# projects. The change in this PR was not applied there.
@AArnott Does it affect VB/F# with NBGV?

@Nirmal4G, no, NB.GV is not adversely impacted by this since we took an execution time workaround to append to the property for all languages.

Does InitialTargets workaround have any issues or gotchas I should be aware of?

You shouldn't set DefaultTargets. That only affects the target to build when the user doesn't specify it anyway.
But your own msbuild file can set InitialTargets without concern. All the InitialTargets set in every imported file gets collected and executed. But these should be very lightweight targets since they run for every single build, even design-time builds.

But these should be very lightweight targets since they run for every single build, even design-time builds.

And the build that happens in command line Restore. For these reasons, we mostly migrated away from them in the common targets. But changing a property is lightweight, and these properties will only matter when getting to the target that DependsOn them, so I don't think those factors apply to this specific case.


I found a working fork for Laravel 7/8

This is working for me on Laravel 8: https://github.com/DariusIII/tmdb-laravel

Oh great! Will this fork integrated in the original project?

Anyone is free to volunteer to maintain "this original" repository, as per https://github.com/php-tmdb/laravel#looking-for-maintainers.


[#139] object: Remove deprecated object.RawObject type

close #139

Please, drop the [#139]  prefix from the commit message.


Prepare for C++ attestation release

Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[X] C++ Guidelines
[X] Doxygen docs
[X] Unit tests
[X] No unwanted commits/changes
[X] Descriptive title/description

[X] PR is single purpose
[ ] Related issue listed

[X] Comments in source
[X] No typos
[X] Update changelog
[X] Not work-in-progress
[X] External references or docs updated
[X] Self review of PR done
[X] Any breaking changes?

Ping. This should be pretty uncontroversial :).


However, the case numbers don't seem very natural.
I just wanted to check this is by design.

This is a mistake from a fix I made earlier. Thanks for reporting it!


feat(designer): Allow operations to specify unknown parameters to not be rendered or serialized

Unfortunately there are some scenarios where the workflowDefinition to be loaded can contain parameters that do not exist in the manifest (and should not exist).
We want to have a way for the workflowDefinition to indicate that we do not want to show these parameters or serialize them.
current behavior:

The plan is for PAuto's workflowDefinitions to always hide and not serialize unknown parameters.
I'm not 100% sure if this will break something, please let me know if there are any specific scenarios I should validate


Dead link

THe link at the top of the page to GUID_BUS_INTERFACE_STANDARD (https://msdn.microsoft.com/library/windows/hardware/ff546561) goes to a non-existent page.

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

ID: d8cdc030-6da5-35cd-604a-445a0109c852
Version Independent ID: fc3feb34-1b08-ab96-0d77-4c42a811857c
Content: _BUS_INTERFACE_STANDARD
Content Source: wdk-ddi-src/content/wdm/ns-wdm-_bus_interface_standard.md
Service: unspecified
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Fixed!  Thanks Peter

It now points to a topic page.  I was expecting that it would point to a reference page, preferably one that would tell me which header file to include to get the definition of GUID_BUS_INTERFACE_STANDARD (turns out it's wdmguids.h but I had to grep the headers to find it)

Thanks for the heads-up. I added the header to the target page, will be live by EOD


Automatic In-Guest Patching for VMSS

Resource Provider
Compute
Description of Feature or Work Requested
Automatic in-guest patching provides customers an easy-to-onboard solution to maintain security compliance for their virtual machines. This feature provides:
Simple onboarding – Enrolling a VMSS requires configuring one property on the VMSS model (process detailed below) with no further onboarding actions needed.
Automatic compliance – Most VMs (including dev/test) have compliance requirements for security/critical vulnerabilities and using automatic in-guest patching keeps the virtual machine compliant without additional overhead.
Availability-first patching – Automatic in-guest patching is orchestrated through an availability-first process (detailed below) which ensures maximum uptime for the customer’s application.
This property is already available for VM:
https://docs.microsoft.com/en-us/azure/virtual-machines/automatic-vm-guest-patching#azure-cli-for-windows-vms
https://docs.microsoft.com/en-us/cli/azure/vm?view=azure-cli-latest#az_vm_create
Reference CLI request for VM:
Windows: https://github.com/Azure/azure-cli/issues/14458
Linux: https://github.com/Azure/azure-cli/issues/16159
To onboard a new VM to automatic in-guest patching, the following property settings must be included in the VMSS model/template either under windowsConfiguration or linuxConfiguration:
"windowsConfiguration": {
"provisionVMAgent": true,
"patchSettings": {
"patchMode": "AutomaticByPlatform"
}
}
"linuxConfiguration": {
"provisionVMAgent": true,
"patchSettings": {
"patchMode": "AutomaticByPlatform"
}
}
Allowed values for Windows  AutomaticByOS, AutomaticByPlatform, Manual
Allowed values for Linux VM: AutomaticByPlatform, ImageDefault
CLI support is requested to set the above flag on the VMSS model when creating the VMSS or modifying it.
Minimum API Version Required
2020-12-01
Swagger Link
Windows case;
Swagger ref: https://github.com/Azure/azure-rest-api-specs/blob/4d1cfdb75ea7ba87f91caf3c943205730c739101/specification/compute/resource-manager/Microsoft.Compute/stable/2020-12-01/compute.json#L12536
This will be mapped to https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.management.compute.models.patchsettings.patchmode?view=azure-dotnet#Microsoft_Azure_Management_Compute_Models_PatchSettings_PatchMode
Linux case:
Swagger ref: https://github.com/Azure/azure-rest-api-specs/blob/4d1cfdb75ea7ba87f91caf3c943205730c739101/specification/compute/resource-manager/Microsoft.Compute/stable/2020-12-01/compute.json#L12558
This should be mapped to LinuxPatchSettings in https://docs.microsoft.com/en-us/dotnet/api/microsoft.azure.management.compute.models.linuxconfiguration?view=azure-dotnet
Target Date
Ignite 2021 (Nov 02, 2021)

@Deepali-Gupta FYI

Compute for Ignite

@mayanknayar  Since we have arranged the work of the next sprint, we may not have enough time to develop and release. Is it acceptable to release in the sprint after the next sprint ( in early December)?

@zhoxing-ms yes, this request is to get the CLI cmdlets for VMSS in line with the implemented CLI for VMs. I included the template samples in the my original request. Can you take a look and let me know which other details you need to get started? Here are some more examples for template from the doc:
https://docs.microsoft.com/en-us/azure/virtual-machines/automatic-vm-guest-patching#rest-api-for-linux-vms
https://docs.microsoft.com/en-us/azure/virtual-machines/automatic-vm-guest-patching#rest-api-for-windows-vms
The same osProfile block applies to VMSS same as I included in my request details.

@zhoxing-ms for timelines, this feature is an Ignite release, so December would be too late. Can you include it for the release immediately after Ignite hopefully by mid-November?

＠mayanknayar Thank you for your reply. At present, our Chinese team is taking the National Day holiday. We will arrange the development and release of this requirement after the holiday

As currently az vmss create cannot add extension while adding health extension to vmss is a prerequisite to set patchMode to "AutomaticByPlatform", so we cannot support set patchMode for az vmss create yet.  @mayanknayar will create another feature request for adding extension during vmss creation (do it after Ignite).
Currently we will only need support updating patchMode for az vmss update using dot syntax (az vmsss update --set virtualMachineProfile.osProfile.linuxConfiguration.patchSettings.patchMode=AutomaticByPlatform) to be consist with 'az vm update`.


Email assistant webapp (WIP)

Pretty much there. What we need to do:

[ ] Resolve conflicts :/
[ ] Fix some issues around state
[ ] Scroll to bottom
[ ] Add documentation
[ ] Copy into a web-server
[ ] Polish/clean-up examples


Improve impact page

[ ] Animate numbers (count up)
[ ] Link each card to a page

Stories shared -> oa.works/stories
Policies tracked-> SYP permissions (then OArights)
COVID papers -> RSCVD.org
OA searches -> Openaccessbutton.org
ILL searches -> instantILL.org
Guides -> For libraries page? (or should we make a guides resource page on new website)
Made open -> SYP?
Libraries use -> Testimonials?
Authors educated -> NOT SURE

Animate numbers (count up)

Seems a bit weird and a lot less impressive when the numbers aren’t actually accurate/precise & just an estimate.

Yup. I think this can be closed without doing that. Being able to link each card to a page is a nice to have, but not important enough to do right now.
i.e I think this can be closed.

Well I am doing that right now... that’s not a big deal. Just that the page doesn’t feel useful if it’s simply estimates of everything, feels a bit nebulous.

On dev for your approval @JosephMcArthur : https://dev.oa.works/impact/
The bit that says "Many small changes add up to a more just system." doesn’t say much... Could we instead write something like "How we contribute to the Open Access movement  by the numbers"?


A question and maybe a feature request

He
I just stumbled upon your plug-in and really like it. By now I was always using ''vim-scripts/CmdlineComplete.git' to allow for a completion on command line. The problem with that one however is that the completion in a search and replace is only working before the / not in the second part. This seems to be the same for your plug-in? Or can I achieve that with the right options set? Any help with that would be appreciated.
My setup now is:
let g:Cmd2_options = {
       \ '_complete_ignorecase': 1,
       \ '_complete_uniq_ignorecase': 0,
       \ '_complete_fuzzy': 1,
       \ }

 cmap <expr> <Tab> Cmd2#ext#complete#InContext() ? "\<Plug>(Cmd2Complete)" : "\<Tab>"

 set wildcharm=<Tab>

Thank you for using and sorry for the big delay in replying.
The cmap <expr> <Tab> checks that Cmd2#ext#complete#InContext() is true to trigger the completion. The current implementation is quite simple, checking if it is just before the first s/. Hence it is possible to replace the check if a function of your own, or even just default to always true. Check the implementation in the source code. The regex would need to be changed to something a bit more complicated as now it would have to check for possible escaped /s in the search string.
If you need further help I might be able to write the regex depending on your use case.


Error: unable to verify the first certificate

Some links have this error, although it still opens fine in the browser.
Code:
const probe = require('probe-image-size');
(async () => {
    await probe("https://www.nylabone.com/-/media/project/oneweb/nylabone/images/dog101/activities-fun/10-great-small-dog-breeds/maltese-portrait.jpg")
})();

Logs:
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

Error: unable to verify the first certificate
    at TLSSocket.onConnectSecure (node:_tls_wrap:1550:34)
    at TLSSocket.emit (node:events:514:28)
    at TLSSocket._finishInit (node:_tls_wrap:967:8)
    at ssl.onhandshakedone (node:_tls_wrap:743:12) {
  code: 'UNABLE_TO_VERIFY_LEAF_SIGNATURE'
}

Node.js v18.17.1

How to fix:
Add options:
{ rejectUnauthorized: false }


Add WASM file extensions

Pull Request
Thank you for contributing to @dvcs/gitignore and https://www.gitignore.io.
New or update
Select the appropriate check box for this pull request. This helps when merging to ensure there are no conflicts with other templates or misunderstandings of how thee template list works.
New

[ ] Template - New .gitignore template
[ ] Composition - Template made from smaller templates
[ ] Inheritance - Template similar to an existing template
[ ] Patch - Template extending functionality of existing template

Update

[x] Template - Update existing .gitignore template

Details
Web Assembly file extensions are missing from the Web gitignore.
Web Assembly Text Format: https://webassembly.org/docs/text-format/
Web Assembly Binary Encoding: https://webassembly.org/docs/binary-encoding/

Does web assembly need its own template?

It still depends on other Web components. If it starts to become it's own environment it would be interesting to consider it but for the moment, no, I don't think so.


Bugfix/mui tooltip

When merged this should resolve #101

Coverage remained the same at 84.47% when pulling 9c259da9f6e9bd68955eec4ee8501fa95d83e2a1 on bugfix/mui-tooltip into 460489d136de32d277648c14a3ecf86665258de5 on master.


add pywis-pubsub exercise

Can we add pip3 install pywis-pubsub to the student VM setup (not sure if it's in this repo)?

Yes I can add
pip3 install pywis-pubsub
to the student VM
(sorry I didn't add an install-script for the student VM to this repo TBD)


Generate base object for patterns in schema

This enhancement would be an option to generate the object for patterns in the schema.  It would be optioned off by default.  If turned on, then a base class is generated for the item.  The programmer would need to add in the actual pattern names in the appropriate pojo for their particular instance.  The object generated would have a TODO mark and a comment field describing the purpose of the object and how to tie it in.

@Smitty010 In the case you're describing here, are you saying that patternProperties is used simply as a shortcut to describe many property names with one definition? It sounds like the class you create by manually implementing 'the actual pattern names' would not be valid for some valid input data.
Using the example you linked to on #301 (http://www.iptc.org/std/ninjs/ninjs-schema_1.1.json), the renditions object may have any properties with a name that match ^[a-zA-Z0-9]+. So what does it mean to manually implement the actual names?

I'm going to close this one as a wontfix, since as I mentioned on #301 I'm concerned that there isn't a significant minority of users that would use this and hence I don't think it's worth the time to implement or maintain in the core of this plugin. This feels like a bit of a special case rather than a more generally applicable solution to patternProperties.


Add two new components for displaying a resource's details

Work is related to manifoldco/engineering#8689
Reason for change
The product and the plan component will display the resource's product and plan respectively thanks to the resource-container component. These can be used by platforms to build a resource details page with their own styling thanks to the state tunnel in resource-container.
For testing purposes, I also added a mock-resource component that allows us or platforms to fake a resource-container for testing purposes.
Testing
Test with storybook

I tested in Storybook and it looks great - the Resource Product Details component is unstyled but I assume that's intentional. And does the as-card="" attribute require the empty string?

For the styling, it's not intended, that's a storybook issue. If you look at the docs deploy, it should look right. For the as-card="", it's only required in pure HTML in storybook and in our docs I think, it works fine when used in react.

And just since it's a lot to read over and I'm trying to understand - the manifold-service-view is something you're abstracting away because it can be used in both manifold-service-card and manifold-resource-product, is that correct?

That's exactly it! I am trying to abstract display components and data components so we can reuse them in various context, in our current case, the resource-container context.


Communication: Allow instructors to activate communication and messaging features separately

Warning: Deploying this PR on a test server will break it for other PRs. Contact someone with admin access to the testservers before deploying it.
Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.

Client

[x] I followed the coding and design guidelines and ensured that the layout is responsive.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context
Artemis supports two information-sharing features. Communication(Previously called question and answers) and Messaging. Currently, the instructor can only deactivate both features at once. This PR allows instructors to activate/deactivate the two features separately.
Closes #6136.
Description
After discussing with @krusche we decided to change the existing postsEnabled flag in the course entity to an enum with four possible values:
    @Enumerated(EnumType.ORDINAL)
    @Column(name = "info_sharing_config")
    @JsonView(QuizView.Before.class)
    private CourseInformationSharingConfiguration courseInformationSharingConfiguration;

package de.tum.in.www1.artemis.domain.enumeration;

/**
 * NOTE: Ordinal mapping in property of Course.java do NOT change the order of the existing values.
 */
public enum CourseInformationSharingConfiguration {
    /**
     * Both Communication and Messaging are disabled VALUE = 0
     */
    DISABLED,

    /**
     * Both Communication and Messaging are enabled VALUE = 1
     */
    COMMUNICATION_AND_MESSAGING,
    /**
     * Only Communication is enabled VALUE = 2
     */
    COMMUNICATION_ONLY,
    /**
     * Only Messaging is enabled VALUE = 3
     */
    MESSAGING_ONLY,
}

For performance reasons, we save this using ordinal mapping on a tiny int column in the db.
Steps for Testing
The easiest way to check this from an end-user perspective is to activate/deactivate the respective feature by updating the course and then checking if the navigation tabs 'Communication' or 'Messaging' are hidden in the course overview page.

Review Progress
Code Review

[x] Review 1
[x] Review 2

Manual Tests

[x] Test 1
[x] Test 2

Test Coverage
Adapted the existing tests.

Component
Statement Coverage
Branch Coverage

ChannelResource
100%
75%

ConversationManagementResource
100%
100%

ConversationResource
100%
71%

GroupChatResource
100%
74%

OneToOneChatResource
100%
81%

Screenshots
Updated course form:

Tooltip:

Tooltip:

ideally, we have 2 more approvals

also server tests and e2e tests should pass

also server tests and e2e tests should pass

The failing tests seem to be unrelated flaky tests. @krusche


Renaming API_Version and project Parameter.

Google doesn't support v1beta1/2 anymore.
'projectId' parameter is no longer supported, new parameter is 'project'

{
"error": {
"errors": [
{
"domain": "global",
"reason": "turnedDown",
"message": "Version v1beta2 of this API is no longer available. Please try again using JSON API v1. To request temporary reinstatement for your project, please visit https://docs.google.com/forms/d/1isIxBZg3rsQbDN_TOalZaz1WT_ebJchsrlv-Qr_r9mY/viewform?entry.244568692=38914250655&entry.176324201=v1beta2&entry.1071661541-Qr_r9mY/prefill",
"extendedHelp": "https://cloud.google.com/storage/docs/json_api/v1/how-tos/migrate"
}
],
"code": 400,
"message": "Version v1beta2 of this API is no longer available. Please try again using JSON API v1. To request temporary reinstatement for your project, please visit https://docs.google.com/forms/d/1isIxBZg3rsQbDN_TOalZaz1WT_ebJchsrlv-Qr_r9mY/viewform?entry.244568692=38914250655&entry.176324201=v1beta2&entry.1071661541-Qr_r9mY/prefill"
}
}
ArgumentError: Missing required parameters: project.
from /Users/adihochmann/work/lab/ruby/2.1.0/gems/google-api-client-0.8.2/lib/google/api_client/discovery/method.rb:311:in `validate_parameters'

dupe of #4


flow-init Shadows Parent Scope

This could be a bug -- or perhaps a lack of documentation.
It seems that flow-init shadows parent scope. For example:
Controller:
$scope.myVariable = "Hello world!";

HTML:
{{ myVariable }} <!-- this comes from $scope.myVariable and is not updated by the input below-->
<div flow-init>
   <input ng-model="myVariable" />
   {{ myVariable }} <!-- this references myVariable within the flow-init scope -->
</div>

This is not an issue, flow-init creates a new scope. Google more about angular scope inheritance.


feat: add target option

Add target option.
This allows to skip publish or publish the extension as public or private.
This implementation is similar to what one of semantic-release plugins does.
<EMAIL_ADDRESS>supports trustedTesters - https://github.com/fregante/chrome-webstore-upload/tree/v0.2.2#publish-extension

@kovalskyi I'm not sure I can be of any help - I'm not familiar with this project at all.

Hey @kovalskyi, sadly I don't work for IBM anymore and as such have lost write access to this repository. You're probably better off maintaining your own fork unless someone at IBM can grant access. 😞


Question about Dataset Standardization

Hi,
I appreciate your paper and code.
Could you confirm if you standardized the datasets (applied z-scores) before using RevIN?
Thank you,
Huy

Hi,
I appreciate your paper and code.
Could you confirm if you standardized the datasets (applied z-scores) before using RevIN?
Thank you, Huy

Hi there,
I've got the same question with yours, I read some source code using RevIN and found out that they used the StandardScaler to standardize the data before sending them into the model.
I'm still wondering if there is conflict between these two standardization methods, but doing this can improve the model's performance on my private dataset.


Implement skipping strict enums during parsing

Fixes #40
Added the strict_enum flag to the parse function, which is forwarded to the underlying builders. The default value is True to keep the current default behaviour.
The Loose<EnumType> classes mimick the Enum API through the value attribute, so downstream users don't need to check the type of Schema.format or Content.type.

I'll apply the feedback later this week - probably it will be in the weekend. Thanks for the review!

Rebased on master and PR feedback is processed!

Published in v1.1.6 release, thanks 👍


Show a confirmation popup when deleting a list

This fixes the issue #53

In addition, there is a new message to translate


Fix/162 - volunteers calendar

Volunteers can register for events, lesson events (those belonging to a specific class), and classes they can attend.

Could you provide more info how to try this funcionality?. Right now the creation of class and events from the frontend it is been fixed because with the last backend changes it is not working

In this functionality volunteer can join a lesson, just like volunteer can join an event. All events and lessons that exist after the current date appear on the calendar and you select the one you want to join.


All fixes needed for NCS 2.4.1 release.

This is cumulative PR to collect all needed bug fixes for NCS 2.4.1 release.

[x] Cherry-Pick: https://github.com/project-chip/connectedhomeip/commit/8db0d655a074c8b3d5539c7847fc54f8448cb5a3
[x] Cherry-Pick: https://github.com/project-chip/connectedhomeip/commit/0f8fac1f86a91fd3fe4e1056d546ef00de72e46f
[x] Cherry-Pick: https://github.com/nrfconnect/sdk-connectedhomeip/pull/295/commits/39c99f79e34f2b0ebfaeb70e9800d45a2abf14be
[ ] Optionally fixes for Sheliak
[ ] Known issues to v2.4.0 are added for the important bugs (like clusters configuration, wrong links, etc).

Closing, we're going to update the 2.4 branch first.


Loading models into CLP takes more time than solving them

Load time of the model into CLP is of the same order a solving the model in CLP. Loading the model in CLP is 3 times slower than in GLOP for the model shown below. Comments state that code uses CoinModel to build the model and CoinBuild is 5x faster. Also, each call to set a coefficient involves a hash lookup in CoinHash. These should be optimized.
ClpInterface::SetCoefficient
/*
 * Clp model can be built using CoinBuild, CoinModel, CoinPackedMatrix and then
 * uploaded efficiently into ClpModel. CoinBuild is extremely lightweight and
 * best suited for adding entire column or row vectors. CoinModel has more
 * flexibility, like adding an element at a time, for the cost of being five
 * times slower than CoinBuild but four times faster than ClpModel. This
 * interface uses CoinModel. Add methods require row and column indices, and
 * CoinModelHash is used to convert names to indices.
 * /
bool ClpInterface::SetCoefficient(const char *row_name, const char *column_name, double value) {
    int row_index = row_hash_.hash(row_name);
    int column_index = column_hash_.hash(column_name);
    if (row_index == -1 || column_index == -1) {
        return false;
    }
    coin_model_.setElement(row_index, column_index, value);
    return true;
}

Usage in LPSharp:
foreach (var rowName in model.A.Indices) {
  var row = model.A[rowName];
  foreach (var colName in row.Indices) {
     this.clp.SetCoefficient(rowName, colName, row[colName]);
  }
}

This is fixed.


Bad display with --stacked

The display in column doesn't work properly with the --stacked option.
Example1: ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 22.00
Longertext example : ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 36.00

I have to use column -t -s ':' to clean it up.

Have you tried this?
termgraph data/ex7.dat --color {yellow,magenta} --stacked --title "Stacked Data"
Also please tell me the situation in detail.

Here are two full examples :
The good one without --stacked.
echo -e "LOOOOOOOOOOOOOOOOOOOOOOOOONG LINE,10,20\nSHORT LINE,10,20"|termgraph --color {blue,red} --title BEAUTIFUL
And the bad one with --stacked.
echo -e "LOOOOOOOOOOOOOOOOOOOOOOOOONG LINE,10,20\nSHORT LINE,10,20"|termgraph --color {blue,red} --stacked --title UGLY

To solve this problem, PR #40 was submitted by nnist.
Plz wait for mkaz to review it.


After import data to table Schema Table/View list Vanish and transfer complete with cancel button

Description
Import data using the Table right click import selection for the CSV file format. After completion of the import the schema table/view list on the connection window is vanished.

Select the CSV file

After selecting the file

After table mapping Data load settings

On the confirm Window

Here need to click on the stop button on the right hand side

After click on stop gives message transfer complete and data actually transferred.

But on the left hand side DB connection project window schema table/view list is vanished

DBeaver Version
Community Edition 22.3.3
Operating System
Windows 11
Database and driver
DB2 12 and com.ibm.db2:jcc:RELEASE <IP_ADDRESS>

Steps to reproduce
Follow the steps which is given in the description for the CSV file import on table/view.
Additional context
No response

I tried again with restarting the Computer and everything is working as expected. Closing this one.


switch to argparse for CLI

Before submitting

[ ] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
JSONARGPARSE for CLI adds a extra dependency. This PR switches to argparse to keep the library lean.
Before

After

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in GitHub issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@aniketmaurya let's use Fire as we already have it for SDK so it would be zero dependency

@Borda Yes, but if someone wants to dockerize their server and deploy it, the Docker image shouldn't include any external dependencies.


ESP logging library - how to remove colorization characters?

The ESP logging library https://esp-idf.readthedocs.io/en/v1.0/api/log.html seems to decorate log traces with escape characters, which ensure proper colorization if displayed with Putty (and maybe other clients). It is pretty cumbersome to decipher those logs if displayed on an "ordinary" serial monitor
Example (this is still pretty good readable, but usually the traces are worse):

I (28) boot: ESP-IDF v2.0-rc1-257-g4745895 2nd stage bootloader
I (28) boot: compile time 16:45:51
I (55) boot: Enabling RNG early entropy source...
I (56) boot: SPI Speed      : 40MHz

Is there a way to get rid of those "I" and friends?

Hi Neil,
If you run make menuconfig, there's an option under Component Config -> Log output to "Use ANSI terminal colors in log output". If you disable this then the logging output will be clean ASCII.
Angus

Thanks. Worked


Please provide runid dropdown option too on grafana dashboard

Hi,
We have a requirement to see results on Grafana based on runid provided in influxdb. We need runid dropdown also on dashbaord.We are using this plugin to see live data.
Regards,
Amit

Hi, sorry was very busy. Will try to take a look.
BR,
Mike

Hi Mike, thanks for replying me from your busy schedule.
Is there any luck ..
Regards,
Amit

@amitngm Haven't time sorry. I Hope will take a look this week at the capability that we have. By the way what Grafana version are you using? I'm going to update the board as well to use the latest.
BR,
Mike

It's 9.1.1
Regards,
Amit

Also, we don't find any metric for "All" sample

Hello, @amitngm I started adding the runid to the board. So we will have the capability to select one run.
So for example you have two runs with ids 1 and 2  executed in 10 minutes. You will be able to select only run with id 1 or 2 to see the data. To see runs 1 and 2 at the same time will not be possible on the same chart. Any comment?
"All" sample - what do you mean? describe the feature, please.
BR,
Mike

Hi @amitngm implemented in https://github.com/mderevyankoaqa/jmeter-influxdb2-listener-plugin/releases/tag/v2.5
BR,
Mike

Thank You Mike.
Also, All is the collection of request count defined in the suite or executed samples, avg hits, avg responses etc. are the total count or average of execution results.


Create lore.ts

creating a command template that can be added to.

seems great!
I was thinking of a level tier, so it would be 'p' as 'p' instead