Add support for using PeriodicVestingAccount for the relayer account

Summary of Bug
unable to create a channel due to unknown protobuf account

Version

hermes 1.0.0+ed4dd8c8
Steps to Reproduce
hermes create channel --a-chain phoenix-1 --b-chain columbus-5 --a-port transfer --b-port transfer --new-client-connection

and it returns
ERROR error raised while creating client for chain phoenix-1: failed sending message to dst chain : Failed to deserialize account of an unknown protobuf type: /cosmos.vesting.v1beta1.PeriodicVestingAccount
Acceptance Criteria

it can create the channel

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Thank you for opening an issue!
Continuing the discussion from Discord here.

my initial thought was this was due to columbus-5 being a old chain, and maybe I need to use a older version of hermes?

I doubt it's due to columbus-5. It's unlikely related or specific to a version of Hermes.
Like I mentioned earlier, I think we don't support at the moment PeriodicVestingAccount, only cosmos.auth.v1beta1.BaseAccount or *.EthAccount. This error is related to the wallet you've configured in Hermes for using with chain phoenix-1. The config param is key_name (ref https://github.com/informalsystems/hermes/blob/41aec7154e4fee51fba803d021c4541ef8902228/config.toml#L128). Specifically, this wallet cannot be a PeriodicVestingAccount.
We might be able to support this.
Action items for you

Can you help us by providing the output for this command:

  hermes keys balance --chain phoenix-1

I don't see public RPCs available for phoenix-1 in the chain registry. Do you know any RPC node for that network we can use?

$ hermes keys balance --chain phoenix-1
2022-10-19T13:28:11.031623Z  INFO ThreadId(01) using default configuration from '/home/hermes/.hermes/config.toml'
SUCCESS balance for key `terrakey`: 1505247 uluna
$

RPC keys

https://terra2-rpc.dalnim.finance/ -- phoenix-1
https://terra-rpc.dalnim.finance/ -- columbus-5

$ hermes keys list --chain phoenix-1
2022-11-01T14:47:50.000557Z  INFO ThreadId(01) using default configuration from '/home/hermes/.hermes/config.toml'
SUCCESS
- terrakey (terra14cpqjy6tfryrqlwyycyy7hca5uzrdyqh0leked)

as for why? I didn't specifically create it that way. It could be a hold-over from pre-depeg luna (in may) and it was just migrated over this way.

I found this info:

https://docs.cosmos.network/v0.45/modules/auth/05_vesting.html#intro-and-requirements
The account you're using is this one https://cosmosrun.info/terra-phoenix-1/account/terra14cpqjy6tfryrqlwyycyy7hca5uzrdyqh0leked

Not all accounts on phoenix-1 are of type cosmos-sdk/PeriodicVestingAccount, for instance this one is cosmos-sdk/BaseAccount https://cosmosrun.info/terra-phoenix-1/account/terra1l4v02k8azrekgzjahjdgaaplr4tlnmlc4eelrx

aah..ok.. so it's due to it being a migrated account.
I'll adjust and retry.
do you want to close this, or are you planning on adding the protobuf entry?
(ie. feel free to close, as my issue is mitigated)

aah..ok.. so it's due to it being a migrated account.
I'll adjust and retry.
do you want to close this, or are you planning on adding the protobuf entry?
(ie. feel free to close, as my issue is mitigated)

I see. Thanks for the quick update! Since your issue is mitigated, I'll close it.
If you feel like you'd like support for this in the future, please let us know.
Technical details: Adding support for PeriodicVestingAccount would need Hermes to import the SDK's vesting proto files, which means patching ibc-proto-rs to include those files. So this feature requires an upgrade to crate ibc-proto-rs in addition to upgrade to Hermes. This is not  difficult, but needs a bit of wrangling, so we will do it if the need for it arises.


vkGetMTLDeviceMVK populating target pointer with garbage

I'm attempting to write a proof of concept for sharing OpenGL textures with Vulkan on OSX, using the guidelines here.  Right now my main stumbling block is accessing the MTLDevice associated with my Vulkan VkPhysicalDevice.  I'm trying to use the vkGetMTLDeviceMVK function, but although no validation logging is triggered and the function returns without a problem, after calling the function the id<MTLDevice> pointer I've passed is not a valid pointer.  It contains the value 0x02.
This then crashes when I next use the metal device in CVMetalTextureCacheCreate.
Note that when I call  MTLCreateSystemDefaultDevice();, I get a valid MTLDevice pointer that I can see in the debugger is of type MTLDebugDevice*, and I can pass it to the CVMetalTextureCacheCreate without problems.
My current code is here
My Objective C skills are pretty low, so it's possible that I'm doing something wrong in the declaration or the function call, but I can't find anything different I should be doing (that compiles)

I tried it a number of different ways....  my current code, linked above,
@interface GLInterop : NSObject
- (nonnull instancetype)initWithVulkanDevice:(VkPhysicalDevice) vkDevice
                           vkPixelFormat:(VkFormat)vkPixelFormat
                                       size:(CGSize)size;
...
@property (readonly, nonnull, nonatomic) id<MTLDevice> metalDevice;
...
@end

and
- (nonnull instancetype)initWithVulkanDevice:(VkPhysicalDevice) vkDevice
                           vkPixelFormat:(VkFormat)vkFormat
                                       size:(CGSize)size
{
        ...
        vk::mvk::vkGetMTLDeviceMVK(vkDevice, &_metalDevice);
        ...

where vk::mvk::vkGetMTLDeviceMVK is defined as
namespace vk { namespace mvk {
        static PFN_vkGetMTLDeviceMVK vkGetMTLDeviceMVK{nullptr};

void init(const vk::Device& device) {
  if (!vkGetMTLDeviceMVK) {
    vkGetMTLDeviceMVK = PFN_vkGetMTLDeviceMVK(device.getProcAddr("vkGetMTLDeviceMVK"));
    vkSetMTLTextureMVK = PFN_vkSetMTLTextureMVK(device.getProcAddr("vkSetMTLTextureMVK"));
    vkGetMTLTextureMVK = PFN_vkGetMTLTextureMVK(device.getProcAddr("vkGetMTLTextureMVK"));
  }
}

} }

I've been able to side step the problem currently by directly fetching the system metal devices and using it directly, but I'm still not able to use vkGetMTLDeviceMVK.  I will investigate the behavior of the cube demo on my system.

I've confirmed the same behavior as you're seeing when running the cube demo.  Whether I use the implicit declaration of vkGetMTLDeviceMVK as you do, or I use the explicit loading of the function via vkGetDeviceProcAddr, I get a valid MTLDevice pointer back.
I also tried changing DemoViewController.m to DemoViewController.mm to see if there was some issue with C vs C++ linkage, but still can't reproduce the issue.
I'm currently examining the project settings in the Cube demo and comparing them to the project settings in my own CMake based Xcode project to try to find some important distinction.  I'll also examine the sequence of Vulkan commands that my app is executing compared to the demo.

Further testing has demonstrated that if I call the vkGetMTLDeviceMVK function immediately after selecting my vk::PhysicalDevice, I do get a valid pointer.   I'm trying to figure out if the issue is heap corruption, if so, when it occurs (i.e. when does vkGetMTLDeviceMVK go from giving me good data to garbage) and whether it's in my code or MoltenVK.

OK, I completely re-wrote my sharing layer and it seems to work all right now, though I'm still getting intermittent crashes I don't quite understand.  However, I'm going to chalk it up to my rudimentary ObjC skills for now until I can get a more consistently reproducible bug or figure out what I'm doing wrong.  If I find something more compelling I will re-open or file a new issue.  Sorry for the inconvenience.


Logging proto3

Proposed change:
Logging class will be replaced by particular logging classes depending
on the component doing the logging.  Instead of doing
"Logging.Enter(Logging.Socket, socket, "method", "parameters")" the new
call will look like this: "SocketsLogging.Enter(socket,  "method",
"parameters")".
This is done so each project don't have to include every logging related
file and only the ones related to that component. A few logging lines
are replaced using this proposed new system in this prototype.
The original logging class is being left for now for 3 reasons:
-So old calls using shims can still work (at least until they get
replaced by the new format, which will be the next step after this
prototype is approved).
-Some common methods related to hashing are in logging.cs, until a
better place is found for them.
-The enum value related to the eventIDs will be there also until a
better fit place is found.

@cipop , @davidsh PTAL.

I added a LoggingUtilities generic (template) class that will have all the common code between logging events.
In order to do that, each EventSource has to implement an interface (ILogging, defined for now in the LoggingUtilities class) with the common logging methods (as  FunctionStart and FunctionStop).
Since interfaces can not have any static elements, and our eventsources follow the singleton pattern,  it was not possible to have any code related to initialization in the LoggingUtilities class. This includes validation, checking for initialization, and the property On. Those will need to remain repeated in each Logging class.
@CIPop  @davidsh  PTAL

Only look at the changes done under Net/Logging and System.Net.Sockets. I didn't have time to clean the code under other projects.

@CIPop  I cleaned the code, and put the skeleton and examples we talked about.


Client Registry Workflow Issues

[ ] When importing a patient on server #2 , an extra patient record is created in OpenCR. The 2nd server is likely sending two consecutive requests, and OpenCR is not updating the previous one because it doesn't exist. This needs to be investigated.
[ ] The decision rules do not always seem to correctly result in automatic vs. potential matches, and need to be built out and tested. All scenarios should be put into the Postman collection for automated testing.

@ckemar @ccwhite333 @jeandaboul add any issues that you run into while testing.


Expand should work only when we click on the expand indicator

Describe the bug
Hi I am using the custom expand indicator to implement the collapse only functionality  in the  expandHeaderColumnRenderer
as per the storybook example i have implemented the custom indictors to show the collapse all (...) when i click on that and i gave the other condition as null  (not to perform expand)
  expandHeaderColumnRenderer: ({ isAnyExpands }) => {       if (isAnyExpands) {         return <b>-</b>;       }       return null;     },
As per the above condition i will not see any indication to expand all., which is correct But  the issue is when i click on that area which null i am seeing  the expand is working, which should not be the case in this scenario. Is there any way I can restrict only expand functionality in such a case.
Apologies if this is not a bug
Screenshots
If applicable, add screenshots to help explain your problem.
codesandbox
https://codesandbox.io/s/react-bootstrap-table-next-basic-example-3fkwz?file=/src/index.js:969-1107
@AllenFang  @chimericdream

@sonusandeep if you want to expand the row by clicking expand indicator, you should enable expandByColumnOnly
Please open another issue if the problem still remain, thanks

Thank you so much @AllenFang , for the reply, i dont need the expand to be performed in table header, all i need is only the collapse should be performed, for that i have the indicator for only the collapse case and the other case i  gave it as null. But when in case an empty indicator if I click on the expand column, the rows are expanding, in my case i don't need to perform  the expand, is there any way i can restrict this. Thank you once again. #1422


Allow for limiting the number of results returned when querying an event stream

When synchronizing append-only event streams between different systems, it's helpful to have control over the number of events returned when querying.
Add a limit parameter to the IEventStore.FetchStream and IEventStore.FetchStreamAsync methods.

I think we're good to just query for IEvents with Skip/Take


AgentAquaticus enhancements

[ ] Integration tests
[ ] Use the same mission in all examples
[ ] Misc bug fixes

See old issue here https://github.com/CarterFendley/moos-ivp-agent/issues/24


feature request: easy display of full gen info if present

If i find something cool in a large grid I can save it again, or work backward to derive where it lives on disk but it would be nice if there were an easier way to see full gen info (for a selected image) as well as path on disk (perhaps in pop out/over suggested in #5.
Thanks!

After experimenting, the last idea actually totally works, and works well! Metadata is now visible when you click an image to view it.
It is also optional on the in-WebUI generation options, for users that want to hide it for privacy reasons.
It's output specifically in a format that matches how the WebUI formats metadata, to make it easy to copy-paste the metadata to the WebUI.
Because of the hacky way its implemented, it will be missing data that isn't available as an option in the grid generator, or data that gets added to the WebUI's format in the future (until the grid gen is updated to include it).


Split actions and activations DB

This PR splits the Database into an activations and an actions (and other entities) DB.

PG2#1363 is green.

PG1#1370 is green

rebased and merged in 7b5a64481aedfd060f3ccaab72a2b6197de47df2


[SPARK-10973][ML][PYTHON] Fix IndexError exception on SparseVector when asked for index after the last non-zero entry

See https://github.com/apache/spark/pull/9009 for details.

Can one of the admins verify this patch?

Test build #1874 has started for   PR 9062 at commit 962e1ab.

This patch merges cleanly.

This patch includes updates to a lot of pom files.  Can you please remove those?
(Thanks for sending all of these backports, btw!)

Test build #1874 has finished for   PR 9062 at commit 962e1ab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

@jkbradley Should be OK now.

Test build #1879 has started for   PR 9062 at commit c209b5f.

This patch merges cleanly.

Test build #1879 has finished for   PR 9062 at commit c209b5f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #1884 has started for   PR 9062 at commit c209b5f.

This patch merges cleanly.

I'll wait for the test to finish, but regardless, I'll merge this into branch-1.3.  I tested it locally, and it worked.

Test build #1884 has finished for   PR 9062 at commit c209b5f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merging with branch-1.3
Thanks for all of the patches!


Cannot return to lxde after returning to ChromeOS in Xorg

I have a trusty chroot installed with lxde, xorg, and xiwi. Running with xiwi fails, another issue, but running with xorg lets me use lxde in its own screen until I page back to ChromeOS, when attempting to page back will drop me into the developer console. The following is the output that the crosh window I ran it from gave before I terminated it:
X.Org X Server 1.15.1
Release Date: 2014-04-13
X Protocol Version 11, Revision 0
Build Operating System: Linux 3.2.0-67-highbank armv7l Ubuntu
Current Operating System: Linux localhost 3.10.18 #1 SMP Tue Jul 14 23:44:39 PDT 2015 armv7l
Kernel command line: cros_secure console= loglevel=7 init=/sbin/init cros_secure oops=panic panic=-1 root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 2506752 verity payload=PARTUUID=a47bac0c-0d25-064d-aab7-12d0e61b30bb/PARTNROFF=1 hashtree=PARTUUID=a47bac0c-0d25-064d-aab7-12d0e61b30bb/PARTNROFF=1 hashstart=2506752 alg=sha1 root_hexdigest=5b555294b7e577d41c7de4fc52a0910623ba3769 salt=cadb7c2b91f1de700dbccb20c599894313edf982ca48c597bf7226c320694daf" noinitrd vt.global_cursor_default=0 kern_guid=a47bac0c-0d25-064d-aab7-12d0e61b30bb
Build Date: 12 February 2015  02:55:07PM
xorg-server 2:1.15.1-0ubuntu2.7 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.30.2
Before reporting problems, check http://wiki.x.org
to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
(++) from command line, (!!) notice, (II) informational,
(WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(==) Log file: "/var/log/Xorg.1.log", Time: Wed Jul 29 20:32:47 2015
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
Initializing built-in extension Generic Event Extension
Initializing built-in extension SHAPE
Initializing built-in extension MIT-SHM
Initializing built-in extension XInputExtension
Initializing built-in extension XTEST
Initializing built-in extension BIG-REQUESTS
Initializing built-in extension SYNC
Initializing built-in extension XKEYBOARD
Initializing built-in extension XC-MISC
Initializing built-in extension SECURITY
Initializing built-in extension XINERAMA
Initializing built-in extension XFIXES
Initializing built-in extension RENDER
Initializing built-in extension RANDR
Initializing built-in extension COMPOSITE
Initializing built-in extension DAMAGE
Initializing built-in extension MIT-SCREEN-SAVER
Initializing built-in extension DOUBLE-BUFFER
Initializing built-in extension RECORD
Initializing built-in extension DPMS
Initializing built-in extension Present
Initializing built-in extension DRI3
Initializing built-in extension X-Resource
Initializing built-in extension XVideo
Initializing built-in extension XVideo-MotionCompensation
Initializing built-in extension SELinux
Initializing built-in extension XFree86-VidModeExtension
Initializing built-in extension XFree86-DGA
Initializing built-in extension XFree86-DRI
Initializing built-in extension DRI2
Loading extension GLX
crouton: version 1-20150716185530~master:aba2c948
release: trusty
architecture: armhf
xmethod: xorg
targets: xorg,lxde,xiwi
host: version 7077.95.0 (Official Build) stable-channel nyan_big
kernel: Linux localhost 3.10.18 #1 SMP Tue Jul 14 23:44:39 PDT 2015 armv7l armv7l armv7l GNU/Linux
freon: no
Agent pid 27163
** Message: main.vala:99: Session is LXDE
** Message: main.vala:100: DE is LXDE
** Message: main.vala:131: log directory: /home/jack/.cache/lxsession/LXDE
** Message: main.vala:132: log path: /home/jack/.cache/lxsession/LXDE/run.log
croutonwebsocket error: Error timeout
Error: target STRING not available
Error: target STRING not available
The following is the output from running sudo enter-chroot -n trusty croutonversion
Entering /mnt/stateful_partition/crouton/chroots/trusty...
crouton: version 1-20150716185530~master:aba2c948
release: trusty
architecture: armhf
xmethod: xorg
targets: xorg,lxde,xiwi
host: version 7077.95.0 (Official Build) stable-channel nyan_big
kernel: Linux localhost 3.10.18 #1 SMP Tue Jul 14 23:44:39 PDT 2015 armv7l armv7l armv7l GNU/Linux
freon: no
Not unmounting /mnt/stateful_partition/crouton/chroots/trusty as another instance is using it.

Make sure to use Ctrl+Alt+SHIFT+F1/F2 to switch.

And that seems to do it. Thanks!

Cool; if your satisfied you can close this issue...


không biết lỗi gì

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[X] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?

Bloxstrap Log
N/A

System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\Windows 10 Pro\AppData\Local\Bloxstrap\Versions\version-d2bde6b0a05e4840\content\fonts\families'.

Send your log file


check the access permission to the photo gallery

When using the application for the first time, check the access permission to the photo gallery

@cnjsstong
How about merge and make 2 releases on NPM?

@0nlyoung7
Thank you for the pull request.
While we are still on RN 0.39, we may need some discussion on the pull request internally.
I'll update on this pull request when we are ready to merge and release package on NPM.

Has manually merged. Currently, the up-to-date #import <React/***.h> version will be at the rn40 branch, and the master branch will continue to use the old import path. Will switch to the new path later.


Add a tool to check for test regressions

Example usage:
python utils/check_regressions.py \
--new="new_report.json" \
--old="old_report.json" \
--report-file="comparison.md" \
--perf_tol_regression=0.01 \
--perf_tol_progression=0.01

Or -r=0.01 -p=0.01 for shortened tolerance flags.
Here is a manufactured sample from the shark-test-suite.txt models (fake data):
Test Run Comparison Report
Performance Comparison
regression tolerance: 1.0%
progression tolerance: 1.0%

model name
exit_status
analysis
old_time_ms
new_time_ms
change_ms
percent_change

ConvNeXt_vaiq_int8
Numerics
progression
1730.5054
1647.7364
-82.7691
-4.78%

2 Regressions Found:

model name
old_status
new_status

KeypointRCNN_vaiq_int8
compilation
preprocessing

retinanet_resnet50_fpn_vaiq_int8
compilation
preprocessing

One Progression Found:

model name
old_status
new_status

RAFT_vaiq_int8
compiled_inference
Numerics

Thanks! Not related this to PR, but in general, we only run benchmarking on a test if it fully passes right?

Did a test run with the changes, and it looks great (check out https://github.com/nod-ai/e2eshark-reports/tree/main/2024-10-14/ci_reports_onnx). Thanks Zach!


Add some interface for sessions

This PR allows for inserting a session programmatically, and retrieving a list of all sessions alive in memory, is that correct?
If those two were intended, it looks good to me


Reworking the processor implementation.

Instead of, what can become, a complicated original implementation migrating
filebeat processors to it's own defined type. There it's much easier to perform
some value checking before generating the configuration files in the config
dir location.
Orignally processors can only be declared using a class parameter like the following example
 class{"filebeat":   processors => [     {       "drop_fields" => {         "params" => {"fields" => ["input_type", "offset"]}       }     }   ] }
Now they can be declared using defined type filebeat::processor, or still through $filebeat::processors but with a breaking format. Examples:
class{"filebeat":   processors => {     "my_processor" => {       "processor_name" => "drop_fields",       "params" => {"fields" => ["input_type", "offset"]}     }   } }
filebeat::processor{"my_processor":   processor_name => "drop_fields",   params => {"fields" => ["input_type", "offset"]} }
Have had issues with rake testing on my development environment which I cannot figure out. Travis CI testing may or may not be successful with the new test cases.

This provides a more reliable and permanent fix for #79

I agree - I haven't had a chance to test this at all yet to be honest (everyone's been on vacation). I'm hoping to get it tested in the next few days and pushed up.

I'll fix the conflict on templates/filebeat5.yml.erb at my earliest convenience.

I'm going to test this out and make sure I understand it (we don't use processors yet), and will hopefully merge this week to get it in for the next release. I put a note in the changelog that a breaking change will be showing up in the next feature release.
Thanks for taking the time on this.


Display model timing tab next to "Lineage View"

Describe the feature
Display a timeline chart similar to the one in the dbt Cloud UI that shows how long each model ran during the last invocation of dbt from the command line. All of the info to build the chart is available in target/run_results.json. If that artifact doesn't exist display a friendly message indicating there aren't any run results to use.
Describe alternatives you've considered
No response
Who will benefit?
Everyone who wants to know how long their models took when invoking a complex sub DAG during local development
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

@dluftspring : I see you are willing to submit a PR. Is there anything I can do to get you started?
You can always come around to #tools-dbt-power-user on dbt Slack if you wanna chat about this.


Remove unused macro (configure_shell)

Also i've removed some unused lints on sha256.rs

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+
Thanks!

:pushpin: Commit 2141f13 has been approved by alexcrichton

:hourglass: Testing commit 2141f13 with merge aab09be...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing aab09be84b0f458d131bd1ee34636675ce65f114 to master...


Maybe serious bug in JedisCluster.subscribe

I got ERR only (P)SUBSCRIBE / (P)UNSUBSCRIBE / QUIT allowed in this context when using subscribe.
As we all known a connection with CLIENT_PUBSUB flag cannot process with other command like get,set ,etc..... (server.c:2448)
Usually , we get connection subscribe and then unsub it.
When we use subscribe in JedisCluster we call runWithAnyNode .
public T runWithAnyNode() {
        Jedis connection = null;
        try {
            connection = connectionHandler.getConnection();
            return execute(connection);
        } catch (JedisConnectionException e) {
            throw e;
        } finally {
            releaseConnection(connection);
        }
    }

we release connection (return pool or desert it ) by isBroken tag in Connection.java
Code Example
@Test
    public void subscribe() {
        JedisCluster c = cluster.getClient();
        try {
            c.subscribe(new JedisPubSub() {
                public void onMessage(String channel, String message) {
                       throw new RuntimeException("sss");
                        unsubscribe();
                }
                public void onSubscribe(String channel, int subscribedChannels) {
                    assertEquals("foo", channel);
                    assertEquals(1, subscribedChannels);

                    // now that I'm subscribed... publish
                    publishOne("foo", "exit");
                }

                public void onUnsubscribe(String channel, int subscribedChannels) {
                    assertEquals("foo", channel);
                    assertEquals(0, subscribedChannels);
                }
            }, "foo");
        } catch (Exception e) {

        }

         for (int i = 0; i < 100; i++) {
                cluster.get(RandomUtil.getRamdomString());
          }
    }

if any Runtime Exception in onMessage it is hard to process with this situation because there is no exceptions with jedis subpub connection and connection will be return to pool....
if we unsubscribe , but there will still contains somedata in RedisInputStream and its buf[]
try {
          throw new RuntimeException("sss");
 } finally {
         unsubscribe();
 }

we will got java.lang.ClassCastException: java.util.ArrayList cannot be cast to [B
Expected behavior
should be no exception in Test.
it never happend in single Jedis due to we can controll how to process with used jedispool .(but now we can't right? only .close() )
Solution
Two solution we can process with it

do not return connection to pool . mark connection as broken . (it is confusing , connection is actually not broken)

using   unsubscribe(); in finally but we need to clear buf in RedisInputStream . so we need some modifications in RedisInputStream and Connection

Steps to reproduce:
see example above
Redis / Jedis Configuration
set JedisPool maxtotal=1 for test
Jedis version:
All
Redis version:
3.0+
Java version:
All

@marcosnils  please review this~

@zts1993 nice finding. IMHO this should be handled in the resetState method of BinaryJedis https://github.com/xetorthio/jedis/blob/master/src/main/java/redis/clients/jedis/BinaryJedis.java#L1821 basically unsubscribe before returning the connection to the pool as it makes no sense for other client to use that.
@HeartSaVioR ?

@marcosnils
Would just clearing buffer of inputstream and outputstream be safe? I'm not sure, since we're reusing socket. Still the safest way is discarding that instance.

@HeartSaVioR I agree that discarding the pool instance would be the safest, but we're penalizing the user performance by forcing a new resource to be created unnecessarily. I'm trying to understand why unsubscribe on @zts1993 example onUnsubscribe is not being called. I believe the problem lies there as the unsubscribe response never gets read back.

@marcosnils  i have working on this today . i found reason .
we send  unsubscribe(); and not waiting for response .
usually we wait unsubscribe() response in JedisPubSub.process() . but if we throw Exception from onMessage , the do-while loop in process() do not working again . so response of  unsubscribe command remains in input buffer area.
it necessary to discard this jedis instance and create new one , but it is a safe way.
it is not safe that we just empty the buffer area because unsubscribe command should be send then blocking and wait for response.


SPIKE: investigate Versioning in openMetadata

User Story
As a data consumer
I want to understand the different versions of a data product that are available
So that I can tell if I'm accessing an old version and a new version is available
Proposal
Treat this as a spike?
We need to understand:

how OpenMetadata represents versions of tables / schemas in the UI
whether we can override OpenMetadata versions with our own versions when updating tables and schemas

Based on this we can decide how we represent versions of a data product within OpenMetadata, e.g.

do we create a single schema per data product, or do we create schemas for every major version?
do we update OpenMetatadata's version attributes whenever something changes in the data platform, or do we store the data platform version as a custom attribute?

Definition of done

[ ] We know how we will change entities in OpenMetadata when there is a major update to a data product
[ ] We know how we will change entities in OpenMetadata when there is a minor update to a data product
[ ] At least one other member of the team has reviewed the suggested approach

No longer relevant


feat: update subscribe section to show toast and intl props

Update component to show toast on success
Add successMessage prop
Add dismissLabel prop
Change revalidation onBlur (this felt better for me)

As a side note: 1password is trying to save this field.
https://github.com/user-attachments/assets/c65ca487-d4fd-424d-b351-3e0bd76fcb19

As a side note: 1password is trying to save this field.

We can add data-1p-ignore to disable it

I don't particularly like how the field shifts as well when there's an error. Seems like this needs some design love @andrewreifman

I don't particularly like how the field shifts as well when there's an error. Seems like this needs some design love @andrewreifman

Agreed about the shift @migueloller. I think for that particular section we just need to align the items to the top instead of center. As far the as the red error text, you could be right in that it's hard to read, but I also wonder if the video makes it look worse. I need to test it out myself to see it, but maybe an easy solution is just tweaking the error color variable some as well.

Can we decide if we want to still use a toast for success or show a message like what @apledger is doing in the auth forms?
This is an example:

However, the form still shifts when these messages are shown, even if we top align. cc @andrewreifman @migueloller

Can we decide if we want to still use a toast for success or show a message like what @apledger is doing in the auth forms?
This is an example:

However, the form still shifts when these messages are shown, even if we top align. cc @andrewreifman @migueloller

I'm good to use these messages in a container like the screenshot displays. Yeah, it will be impossible to avoid any shifting without absolute positioning, but you shouldn't really see 2 messages like that on top of each other in that section, right?

I'm good to use these messages in a container like the screenshot displays. Yeah, it will be impossible to avoid any shifting without absolute positioning, but you shouldn't really see 2 messages like that on top of each other in that section, right?

Yeah, both statuses will never be shown at the same time. I updated the PR to use this instead of toast!

I like using those messages 👍🏻
How do we want to avoid the layout shift? Maybe we can anchor it below the input but absolute position? Or have a space there for it? The other forms should also have a space where the message goes to avoid the shift


Please add below note/Pointers in the Technet with the redirection link attached

Under the sections

""Block the following URLs" list for Safe Links"
"Configure the "Block the following URLs" list in the Microsoft 365 Defender portal"
"Do not rewrite the following URLs" lists in Safe Links policies"

"You can now manage block entries in the Tenant Allow/Block List. (https://docs.microsoft.com/microsoft-365/security/office-365-security/tenant-allow-block-list)
The Global Settings Block List will be turned off in the coming months".
Note: Organizations will no longer have the ability to add to the URL/Domain entries to the Safe Links Block List in the Global Settings flyout and we will attempt to migrate all the entries in an organization's Safe Links Block List to TABL on their behalf. Any entry migrated from the Safe Links Block List to the Tenant Allow/Block List will adopt the behavior of TABL. This means that any message with the URL present will be moved to Quarantine.

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

ID: 8112b7a9-8f21-18b3-03da-f8c02b05f239
Version Independent ID: a441f1d1-5844-ecb9-ba44-95b94714e4e8
Content: Complete Safe Links overview for Microsoft Defender for Office 365 - Office 365
Content Source: microsoft-365/security/office-365-security/safe-links.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

Thanks Chris for addressing all the 3 in this case..
One small correction .Under the section "Do not rewrite the following URLs" lists in Safe Links policies", we need to add below
NOTE : The intended purpose of "DoNotRewrite" is to give tenants the ability to skip the wrapping of URLs. So, With the deployment of the Tenant Allow Block List, it is expected that all tenant allows (ex Detonation and Verdicts) shall be managed there. Ref: https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/allow-block-urls?view=o365-worldwide


Add RFC for Deref in web-sys

This proposal is a formal alternative to RFC 3 which has evolved over time through the comments, hopefully now all written down in one place!
Rendered

This looks great to me, and I think it is the right set of trade offs given the choices we have.

Oh, by the way, it might be worth pointing out that web-sys can continue to use final methods, since it's known that methods aren't overridden.
So regardless of what the default is, it won't affect web-sys, so the default is for the sake of non-web-sys APIs (which are generally created by hand).

Excellent write-up. It explains the situation as I see it very well. I agree with essentially all of it.
I'm strongly in favor of making functions structural by default.  Structural is at best a minor optimization but at worst a giant foot-gun with a silencer.
I'm weakly in favor of Deref versus traits. If we go with structural unless explicitly marked otherwise, then both solutions do the right thing. The trade-offs between the two are pretty minor and mostly subjective, and I could live with either solution.

@Pauan ah yes, that was my intention but I forgot to explicitly say it. When final is faster we can switch everything in web-sys transparently.

An article was just posted that is very relevant to the benchmarks in this RFC: Calls between JavaScript and WebAssembly are finally fast 🎉.
A brief summary: Firefox just implemented some optimizations that allows wasm to call JIT-ed JavaScript directly, and vice versa, without going through a C++ "trampoline". The optimizations made wasm->JS calls about 2x faster, and both JS->wasm and wasm->built-ins about 10x faster.
It may be worth re-running the Firefox benchmark using the latest Firefox nightly to see if these optimizations dramatically affect the performance trade-off between structural and final.

A good point! Some testing shows, however, that there's not much difference. On today's nightly Firefox build structural is 5% faster, roughly where it was before. As the RFC mentions though the benchmark is available publicly for anyone to tinker with it

Bikeshedding the attribute name:

static_dispatch
statically_dispatched
no_proto_walk

I'd like to now propose that we enter the Final Comment Period for this RFC.
Disposition: merge
@rustwasm/core members to sign off:

[ ] @fitzgen
[ ] @ashleygwilliams
[x] @alexcrichton

I think the remaining issue of attribute names can be figured out during FCP, but if changing I'd lean towards the name of no_proto_walk or something like that.

I am in favor of merging this RFC, but I (also) think it is worth bike shedding the name of the non-structural attribute a little more (and am also happy to do this in FCP time).

final in C++ (I don't know about other bracket OO languages: Java, C#) really means it is impossible to override the method. Where as what we are dealing with here is a situation where it is still possible for other JS classes to override the method, but our bindings won't ever call the overridden method. I think the "final" terminology has the potential to confuse folks coming from a C++ background (and maybe C# and Java?). I don't think we should bend over backwards for this group, since I don't think it is either of our core audiences of Rust programmers targeting the Web and Web programmers learning Rust. However, it would be easy to choose another name in this case, so renaming seems like a win/win.
In C++ land, structural is like virtual methods, so how about we call this non_virtual?
Also happy with no_proto_walk since that accurately describes exactly what is not happening and doesn't come with preconceived baggage from other languages.

Since the semantics of JS are weird (prototypal inheritance, etc.) I'm in favor of a JS-specific name.
Many good options have been suggested, I don't have a strong opinion about them, but I do like static_dispatch or static_prototype_dispatch.
Since it should be used rarely, it's okay if the name is weird, long, and unwieldy.

I'm also in favor of this RFC. At least the technical aspects of it. As far as the name of the attribute goes, I don't care very much one way or the other (it's just a name after all), but I do want to present a few more points in favor of final:
final is used to mark imported functions, not exported functions. When you mark an imported type or function with any attribute, that always means that it's a description of what that type/function is/does on the JavaScript side. There is not a single attribute on an imported item that somehow changes the behavior of that type or function on the JavaScript side. That would be weird. Only attributes on exported items change the behavior of things on the JavaScript side. And I think people intuitively understand this.
For example, if a JavaScript class extends another one, you mark the imported type as extends = .... Marking the imported type with extends = Foo does not somehow force the class to extend Foo in JavaScript. It merely describes that you believe that the class extends Foo. I haven't heard of anyone being confused about this and mistakenly thinking that the extends attribute on an imported type somehow forces the JavaScript class to extend the given class, so why would they think that marking a function final somehow forces the function to become non-overridable on the JavaScript side?
Say (hypothetically) that a future EcmaScript version would add "final" as a keyword to mark methods as non-overridable (they almost did, in the (abandoned) ES4 proposal). Then any method that is final in JavaScript, you would mark final in your Rust bindings, which is very consistent with how all the other attributes on imported types/functions work:

An imported class extends another? You mark it as extends = ....
An imported function is a constructor? You mark it as constructor.
An imported method is a getter? You mark it as getter.
An imported method is final? You mark it as final.

All @rustwasm/core team members have signed off. Entering the 7 day :sparkles: Final Comment Period :sparkles:!

For those interested I have an implementation of this at https://github.com/rustwasm/wasm-bindgen/pull/1019 where I've proposed renaming final to host_binding


sesman: auth session before fork

So, the home directory can be created by "pam_mkhomedir".

+1

The change looks good. But I want you to include "why" in commit message for future source code reading. "Why" is always more important than "What".  In this change, to create home directory, right?  It's OK to do force push in unmerged PRs.

Correct session  management is necessary for more things than just to create homedirectory. Here some reasons:
-> Create Homedirectory on demand
-> Correctly setup umask  (pam_umask)
-> Correct setup of selinux contexrt
-> Handling of limits (pam_limits)
and so on .....

I'm a big fan of good comments in the code, however...
The reader of the code would not see that auth_start_session was called elsewhere. So a comment in the code would not be very useful. We cannot annotate every line of code with an explanation why it's there and not elsewhere.
Those who want to dig deeper should be able to find the commit and it's comment. That would provide information about the change and the reason behind it.
Of course, if somebody can come with a useful comment about the auth_start_session that is not duplicating the code, I'll be glad to see it. But the lack of a comment should not be a blocker.

I think most of correct session management procedures can be done in PAM.
For example, suggested in #688. This fixes things done in PAM wasn't working.
Of course I don't intend to block merging this PR. We can add comments whenever later.

#688 describes the correct setup of the pam configuration, but that doesn't help as long as auth_start_session is called in the wrong place.
So just write as comment something like:
"Calling auth_start_session in the correct place, otherwise pam session management fails"

Does it also fix the issue if auth_start_session is moved above wait_for_xserver?
I'd rather do that.
There is not matching auth_start_session / auth_stop_session now.

This patch don't work with pam_krb5 on Debian 8.
I moved auth_start_session higher to the same process as auth_userpass.
That works, but some auth_end closes session too early (but retain_after_close=true on pam_krb5 solves temporary).
I'm attaching patch and log files from successful session (pam_krb5 with debug=true).
auth_start_session2.txt
auth_start_session2_log.txt

@rolnas can you add your pam.d/common-* files to understand the problem ?
I'm using pam_krb5 or pam_ldap with xrdp since several years without problem.

Am 16.03.2017 um 18:29 schrieb Blindauer Emmanuel:

@rolnas https://github.com/rolnas can you add your pam.d/common-* files to
understand the problem ?
I'm using pam_krb5 or pam_ldap with xrdp since several years without problem.

the problem is not authorization as you need for pam_ldap or pam_krb5. It is
session management. For example pam_limits was not working (pam_limits has
only the session module).
Sincerly,
Klaus
--
Rechnerbetriebsgruppe / IT, Fakultät für Physik
Klaus Steinberger
FAX: +49 89 28914280
Tel: +49 89 28914287

I create #696 that can replace this PR.  Hopefully it works for everyone.

Use CVE-2017-6967.
Thanks

Hello, I would like to confirm if PR #704 and #697 are the fix to this CVE issue?
Thanks!

It's just #704 for the fix.  Can someone else confirm it;s fixed?

I confirm it. I thing the pam process is now ok: before, it pam_krb5 was used the env variable was't available in the session an the file was owned by root. Now the variable is set and file owned by user.


Clarify Pig Latin input strings are all lowercase

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts either fix/
[x] You have only one commit
[x] All new and existing tests pass the command npm run test-challenges.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)

Checklist:

[x] Tested changes locally.
[x] Closes #9576

Description
Clarify Pig Latin instructions to indicate that input strings are guaranteed to be English words in all lowercase to address concerns about handling case when solving the challenge.

@BKinahan Can you please repeat for the translations as well? I think a google translate would be sufficient :+1:

@Bouncey I'd rather not, I know firsthand how poorly google translate works for certain languages 😄
Is that how changes are normally implemented in translations? I'd think a separate issue for translators would be more effective.

@BKinahan Certainly.
Can you raise the issues for each respective translation and reference this PR please.
If the new text isn't complicated, then yes. For whole challenges / complicated sentence structure / concept descriptions then they are translated by a human.

@Bouncey 👍

Thanks for contributing :tada:


Question

emb_cost = None if (trk_embs.shape[0] == 0 or det_embs.shape[0] == 0) else det_embs @ trk_embs.T
ValueError: matmul: Input operand 1 has a mismatch in its core dimension 0, with gufunc signature (n?,k),(k,m?)->(n?,m?) (size 3 is different from 512)

Thanks for your interest! Have you ensured you’re running with the —grid_off flag? I’ll change that to be the default.

python3 main.py --exp_name 1 --post --grid_off --new_kf_off --dataset mot17 --w_assoc_emb 0.75 --aw_param 0.5. when I run DeepOCSORT on MOT17, I met this question.


OpenGL deprecation issues

A lot of the code shown here is really great, but some things, like display lists have been deprecated in OpenGL 3.1 (WGL fonts etc)
That means a lot of this code appears to work in some OpenGL programs, as long as you haven't kicked your OpenGL state machine into GL 3.1+ mode
Once you kick your OpenGL state machine into GL 3.1+ mode, it seems a lot of the deprecated features just stop working and you get a blank screen
Deprecation warnings should be added to this code if possible so people know that parts of this code just won't work on later releases of OpenGL

I found adding an extra glDisable( GL_TEXTURE_2D ) call actually made my display lists work (even though there isn't any corresponding glEnable( GL_TEXTURE_2D ) in the program!). I think my program is crossing into some OpenGL UB and it's probably going to be untrustworthy if using any deprecated features


links are corrupted in preview

with a link [[self\-study notes|p.selfstudy notes]] I should see text "self-study notes" linking to file "selfstudy notes.md", but instead, I see "self\ study notes" (dash is changed to a space... bug #1) linking to "selfstudy-notes.md" (space is changed to a dash, bug #2... the link is broken)

I tried to use your extension with Dendron, because Dendron swaps in their own MD preview and it's a disaster (they know it, but it's going to be a while to get a proper fix)... if you're telling me that changing spaces to - in a file name is "normal" then sorry, I'll just uninstall & move on... if my filename has a - in it, I'll type a - in my wikilink
apologies if that sounds snarky, it's not my intent... your use case & mine are just different enough to be incompatible, as a dev I see that often

Yeah, fair. I had a feeling that you were using dendron. Apologies for the inconvenience.


Upgrade libp2p from 0.51.1

Issue summary
The recent upgrade https://github.com/ChainSafe/forest/pull/2732 most likely introduced a regression https://github.com/ChainSafe/forest/issues/2755
Reverting this upgrade seems to mitigate the issue (all my mainnet sync trials passed for Forest pinned to 0.51.1, whereas none passed with the updated libp2p). That was around ten trials.
The revert https://github.com/ChainSafe/forest/pull/2759 is a temporary workaround, the library should be updated, and the deprecated suppressions should be removed. These can be found by gripping this ticket link in the codebase.
:warning: when updating libp2p, make sure it syncs to mainnet.
If it turns out that the issue is back again without the libp2p update, this one should be closed and the original bug ticket should get re-opened.
Other information and links

https://github.com/ChainSafe/forest/pull/3219

Let us (@mxinden & myself) know if we can help you with the update! :)

@thomaseizinger Thanks! I really appreciate it, I will reach out once I contrive a minimal reproducible example of the issue.

Update: the issue is likely to be related to https://github.com/libp2p/rust-libp2p/pull/4250  @LesnyRumcajs has forest running well by raising the connection limit

There were some other changes that may have helped, e.g., not banning peers when they abruptly close the connection.

https://github.com/ChainSafe/forest/pull/3267


Stable order in generated schema's SDL (#1134)

Resolves #1134
Synopsis
See #1134 for details:

Currently, the order persists between compilations as long as there are no changes to schema, but a change to one item commonly results in multiple unrelated items changing their position.

Solution
The solution is based on the https://github.com/graphql-rust/juniper/issues/1134#issuecomment-1704425425 approach, proposed and implemented by @michael-groble for the graphql-parser directly.
We ship it without waiting for a graphql-parser, and later can switch to the solution provided by the graphql-parser out-of-the-box.
Additionally
This PR removes the graphql-parser Cargo feature of juniper, because it effectively exposed the RootNode::as_parser_document() method only, hardly overlapping with the schema-language Cargo feature exposing RootNode::as_schema_language() method only.
Since they're very similar, now it's only the schema-language Cargo feature is left, exposing both RootNode::as_sdl() and RootNode::as_document() methods.
Checklist

[x] Documentation is updated.
[x] Tests are updated.
[x] Changelog entry is added.

@LegNeato please look whether you have any comments on this.


improve readability of the code

improve readability of the code in schedule.hbs , rooms.hbs , tracks.hbs
Partially fixes #1297

@shubham3211 please check the Travis build.

@shubham3211 please fix travis.


Update Gradle Crowdin plugin

Use latest version to allow filtering translated files.

@kingthorin @ricekot and this one :)


Centering image on zoom in/out

Whenever I zoom (wether its by buttons or by the slider) the image won't stay in the middle of the parent div, causing me to drag every time I zoom. Is there someway to fix this?
Demo:  http://wiki.appnetwork.nl/test/

+1 having the same issue. Any fix found?

Try adding "contain: false"

Tryed and not working

Did you ever find a solution for this?

I have same issue image does not zoom in and out from center of parent DIV. So If you have found any workaround for this then please answer here.

Having the same issue on mobile when you are doing pinch/pull any workaround?

No still not get any workarounds for this.
On Mon, Mar 14, 2016 at 8:23 PM, david.truchet<EMAIL_ADDRESS>wrote:

Having the same issue on mobile when you are doing pinch/pull any
workaround?
—
Reply to this email directly or view it on GitHub
https://github.com/timmywil/jquery.panzoom/issues/148#issuecomment-196349064
.

I'm trying to do this as well.

transform-origin normally takes care of this if the image is already centered. If not, set the focal option manually to the center of the div...
http://jsbin.com/sojulof/edit?html,js,console,output

Thanks.
On Sat, Jul 16, 2016 at 12:39 AM, Timmy Willison<EMAIL_ADDRESS>wrote:

Closed #148 https://github.com/timmywil/jquery.panzoom/issues/148.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/timmywil/jquery.panzoom/issues/148#event-724779473,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH11SHEIirF2nhFlqBbRYfeYR5pd6gTEks5qV9regaJpZM4DGiKF
.


Find unattached NICs

Story:
During a cleanup process that I was performing on our Azure infrastructure, I found a useful article on how to delete unattached disk https://docs.microsoft.com/en-us/azure/virtual-machines/linux/find-unattached-disks
Issue:
Also, during this process, I discovered several unattached NICs which are the remainders of deleted VMs. Unfortunately, I did not find a similar article to the above to delete unattached NICs.
Suggestion:
I created an article similar to the above where I describe how to delete unattached NICs.
https://github.com/gpetrousov/azure-docs/blob/gpetrousov-patch-1/articles/virtual-machines/linux/find-unattached-nics.md
The integration of this suggestion was long overdue to technical issues in the automation system in Azure docs. Now that it got resolved, I would appreciate some feedback on the usefulness of this addition before creating a PR.

@gpetrousov Thanks for the feedback! We are currently looking into what you have suggested.

@gpetrousov I think this is a great idea. I also took a look at your doc and it is very well written. I have assigned to this to @tysonn to present it to the content team

@cynthn + @iainfoulds - Hey, could you please review this contribution proposal?

Thanks for the contribution, @gpetrousov . For the most part, I think that this is a good addition. We typically need both an Azure CLI 2.0 version for the Linux VMs documentation set, and an Azure PowerShell version of the Windows VMs doc set. Would it be possible for you to also create a PowerShell version of this doc?
From then a doc formatting perspective, each document requires a final H2 for 'Next steps'. There's nothing super obvious for your proposed doc, but maybe something as simple as:

Next steps
For more information about virtual networks in Azure, see Azure Virtual Networks overview

Also cc @jpconnock for visibility.
#in-progress

@iainfoulds - Thanks for reviewing my article. Unfortunately, I cannot add the powershell version for the windows VMs as I am no longer working with my previous company and thus don't have an azure account to test it out. Also, I'm not so familiar with powershell and windows VMs.
What I can do is add a "Next steps" header which leads to networking concepts.
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/network-overview

Okay, thanks. Please create a PR against the azure-docs repo and I'll work with @tysonn to figure out how we best publish the contribution.
For now, I'll close out this issue and we can work in the PR itself. Please reference issue #5886 and tag @iainfoulds. Thanks!
#please-close

Where i can mention Subscription ID or name in above script?
i want to list unallocated NICs from multiple Subscription where i can mention all subscription name or IDs?


Handle Jest v20's changes to jsonOutputFile

See https://github.com/skovhus/jest/commit/804a802f455e385e75cdf6f15091657b8cc5d20a

There's also the changes around: yarn jest -- --showConfig

This should be good now.


Emulate the signature verification for classic accounts in recording mode

We only emulate loading the account in order to get the correct footprint, we should also emulate the authentication check as well. Probably just charging a signature verification should suffice (we can't really emulate the whole flow because we don't have a real valid signature for the account yet - but we'll just be missing a couple of small metered_clones).

This has been implemented a while ago via emulate_authentication helper.


Reset Password Fails on Second Attempt

Second attempts at password resets update the previous deleted request record, instead of creating a new record.
Probably has been broken since the soft deletes schema change.

Looks like this should just be a matter of dropping the ON CONFLICT portion of the INSERT statement in controller/store/password_reset_request.go (CreatePasswordResetRequest).


Polish the crypto identity reset

Hide the button on the account page under an advanced section
Rework the button to make it non-destructive & reword it like "Looking to reset your crypto identity?"
Make the page match the design: https://www.figma.com/design/jvZ7TIyd7V9Vn8sIDB9vMd/Compound-iOS-Templates?node-id=0-1&t=SzBJoQP4C1n1t6P1-0

I am trying to make sense if/what is there to be done here on EX side, e.g:

Rework the button to make it non-destructive & reword it like "Looking to reset your crypto identity?"
Which button? Is it calling out that the implementation does not correspond to what's in Figma or suggesting to change things that are in Figma?

Make the page match the design
Which page?

This is not for EX or any other client. It is for pages served by MAS.


choose to render weekends or not

Currently it's not possible to choose witch days to render. I would like to hide Saturday and Sunday for example

Hey. I like the idea :+1:. Should be pretty easy to add.

I finally found a tricks in css. But I saw you release to do that more niecely. Thanks a lot !  🥳


Default Content before Resolution

I'm re-working a website that has a very nested interface requiring several child views. Think of a shopping site, with paging, results and filters. What I'd like to do is render default content for said children views while the results are being resolved from the back-end. However, I can't find a way to insert default content past the first <ui-view>, which of course, makes sense.
To get around this, we are currently using $broadcast in the child controllers. We moved the resolution out of the resolve event into the controller, which is working, but requires us to make all of our directives use $broadcast as well, or they don't work since the data isn't loaded before they are. It also seems like a very inelegant solution to the issue.
What's weird, is that when I move the resolve function into a child view, neither the parent nor it's siblings views load before the results child view loads. I can't understand that whereas I can understand children views not loading before the parent is resolved.
Is there a way we can work around this? Building in broadcasting into all aspects of our code base seems like an extremely poor practice. Is there a way to show children default content before the parent is resolved, or even render child states' views before their controllers are instantiated?

https://gist.github.com/christopherthielen/b3fc0304b737a4bf8ec9


How to identify what method we followed for istio installation and get the same settings for upgrade

Hi Experts,
I am very new to ISTIO and  would like to get clarified with my following doubts.
We are planning to upgrade Istio version from 1.7 to 1.8 in our AKS cluster of 1.18.14 in our production
But I am not sure about the  proper method of upgrade to follow in production since there are multiple m,ethods are given by Istio.
I dont have any clue about the how the curent istio setup is done in my environment and what profilesettings we used  as its done long before. Is there any method to export all the existings setting (the one currently running) and do the upgrade
So I am looking for a production ready approach to upgrade Istio with all existing settings in placed.

If you installed using istioctl do kubectl -n istio-system get iop to get the configuration used to do the install.
You can then feed that back into Istio, for example
kubectl -n istio-system get iop installed-state-install -o yaml
istioctl install --dry-run -f /tmp/iop.yaml

(Remove the --dry-run to actually install)

@esnible thanks for your quick guidance. What i understood from your answer that..
-> we can convert the iop details which we are getting from the command "kubectl -n istio-system get iop" to a yaml file.
-> and use the same yaml file to flag to the istioctl install command
Is this method can be applied for Production enviroment? Because Istio is offering multiple ways to upgrade, not sure which one should select
have few doubts on this that... we are not looking for install from a existing configuration, but looking to upgrade the version.

@esnible When we tried to do canary model upgrade, we are getting below error.
./istioctl8 install --set revision=1-8 --set profile=default -f /home/admin/upgrade/iop.yml
Error: failed to get profile and enabled components: failed to read profile: unknown field "telemetry" in v1alpha1.IstioComponentSetSpec:
Where the /home/admin/upgrade/iop.yml is the exported iop installed-state yaml file from the existing istio version 1.73
kubectl -n istio-system get iop installed-state-install -o yaml > /home/admin/upgrade/iop.yml


api: creating record containing specific recid

This problem has already being mused a bit IRL but not on GitHub.
Currently if one tries to load records having specified recid invenio-records crashes with:
-------------------------------------------------------------------------------
ERROR in api [.../invenio_records/api.py:94]:
Problem creating a record.
--------------------------------------------------------------------------------
Traceback (most recent call last):
  File ".../invenio_records/api.py", line 88, in create
    db.session.commit()
  File ".../sqlalchemy/orm/scoping.py", line 150, in do
    return getattr(self.registry(), name)(*args, **kwargs)
  File ".../sqlalchemy/orm/session.py", line 790, in commit
    self.transaction.commit()
  File ".../sqlalchemy/orm/session.py", line 392, in commit
    self._prepare_impl()
  File ".../sqlalchemy/orm/session.py", line 372, in _prepare_impl
    self.session.flush()
  File ".../sqlalchemy/orm/session.py", line 2004, in flush
    self._flush(objects)
  File ".../sqlalchemy/orm/session.py", line 2122, in _flush
    transaction.rollback(_capture_exception=True)
  File ".../sqlalchemy/util/langhelpers.py", line 60, in __exit__
    compat.reraise(exc_type, exc_value, exc_tb)
  File ".../sqlalchemy/orm/session.py", line 2086, in _flush
    flush_context.execute()
  File ".../sqlalchemy/orm/unitofwork.py", line 373, in execute
    rec.execute(self)
  File ".../sqlalchemy/orm/unitofwork.py", line 532, in execute
    uow
  File ".../sqlalchemy/orm/persistence.py", line 174, in save_obj
    mapper, table, insert)
  File ".../sqlalchemy/orm/persistence.py", line 748, in _emit_insert_statements
    execute(statement, multiparams)
  File ".../sqlalchemy/engine/base.py", line 914, in execute
    return meth(self, multiparams, params)
  File ".../sqlalchemy/sql/elements.py", line 323, in _execute_on_connection
    return connection._execute_clauseelement(self, multiparams, params)
  File ".../sqlalchemy/engine/base.py", line 1010, in _execute_clauseelement
    compiled_sql, distilled_params
  File ".../sqlalchemy/engine/base.py", line 1146, in _execute_context
    context)
  File ".../sqlalchemy/engine/base.py", line 1341, in _handle_dbapi_exception
    exc_info
  File ".../sqlalchemy/util/compat.py", line 199, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb)
  File ".../sqlalchemy/engine/base.py", line 1139, in _execute_context
    context)
  File ".../sqlalchemy/engine/default.py", line 450, in do_execute
    cursor.execute(statement, parameters)
  File ".../MySQLdb/cursors.py", line 205, in execute
    self.errorhandler(self, exc, value)
  File ".../MySQLdb/connections.py", line 36, in defaulterrorhandler
    raise errorclass, errorvalue
IntegrityError: (_mysql_exceptions.IntegrityError) (1452, 'Cannot add or update a child row: a foreign key constraint fails (`inspire`.`record_json`, CONSTRAINT `record_json_ibfk_1` FOREIGN KEY (`id`) REFERENCES `bibrec` (`id`))') [SQL: u'INSERT INTO record_json (id, version_id, json) VALUES (%s, %s, %s)'] [...]

The implicit desired behavior would be that the record is well uploaded and the given recid is stored correctly in pidstore/bibrec tables.

This problem have been solved in 1.0.0aX. Records module now never touches the JSON being stored in anyway. It's the responsibility of the one creating the record to include any recid/control numbers etc into the JSON (some PIDStore minters do this for you).
Each record is now identified by a UUID (not stored in the JSON). This UUID can either be generated by the one creating the record or automatically generated by the underlying model:
# Set by you
Records.create({'title': 'My title'}, id_=uuid.uuid4())
# Automatically set by records:
Records.create({'title': 'My title'})

If a UUID provided by the user already exists, the record creation will fail with an IntegrityError.


Base Install should include KKM for AWS so that there's only 1 set of instructions.

Base install should probably include KKM install for AWS, so that we have only 1 set of instructions for AWS vs other targets.

Let's keep KKM separate from base nested install as the requirements are very different.  Mixing the two would cause confusion.


Manually hiding and showing drawer-panel in core-scaffold using dart

Originally opened as dart-lang/sdk#22077
This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?
Create a CoreScaffold component, find it in the dom using:
CoreScaffold e = shadowRoot.querySelector('core-scaffold');
Try to close / toggle the drawer:
e.closeDrawer();
What is the expected output? What do you see instead?
I'm expecting the drawerPanel to close, instead it doesn't move.
Please include the following information:

Dart SDK version: 1.8.3
polymer package version: 0.15.1+1
Operating system: OSX 10.10.1
Browser (if any): Dartium and Chrome

StackOverflow: https://stackoverflow.com/questions/27957771/manually-hiding-and-showing-drawer-panel-in-core-scaffold-using-dart
One of the responses is somebody else experiencing the same problem.

<img src="https://avatars.githubusercontent.com/u/984921?v=3" align="left" width="48" height="48"hspace="10"> Comment by jakemac53

duplicate of https://github.com/Polymer/core-drawer-panel/issues/43

Added MovedToGithub label.


Build error in npm version 1.11.0

Hi,
I seem to get the following build error after a npm install semantic-ui:
Plumber found unhandled error:
Error in plugin 'gulp-less'
Message:
variable @selectPreInitHeight is undefined in file /code/semantic-ui/semantic/src/definitions/modules/dropdown.less line no. 351
Details:
type: Name
filename: /code/semantic-ui/semantic/src/definitions/modules/dropdown.less
index: 7546
line: 351
callLine: NaN
callExtract: undefined
column: 10
extract: select.ui.dropdown {,  height: @selectPreInitHeight;,  padding: @selectPadding;
lineNumber: 351
fileName: /code/semantic-ui/semantic/src/definitions/modules/dropdown.less
I don't get the same build error if i do a bower install semantic-ui, nom install and then build.

Is this in the LESS repo? I'm about to patch. Looks like a variable from next got in somehow.

This is in the standard repo (when doing: "npm install semantic-ui")

Had the same problem. Cleaning the npm cache via npm cache clean solved it for me. Seems like npm does not pull from the latest tag, but from next

Just published a one fiesta 1.11.1

Perfect. It works now.
And BTW thanks for some really good work with semantic. Definitely one of the best and most interesting dev frameworks currently out there.

You're welcome! :smiley_cat:


Add unit test for pkg/chart/chart.go

Signed-off-by: Hu Shuai<EMAIL_ADDRESS>

PTAL

PTAL

I've changed it. PTAL, Thanks.


Regression problems in OmniPred.

I want to ask if, when fine-tuning a regression task in the materials field using OmniPred, I only need to place the corresponding data into vz.ProblemAndTrials, or do I need to create a new vocabulary for chemical elements?

Take a look at this test: https://github.com/google-research/optformer/blob/main/optformer/omnipred/omnipred_test.py#L28
You can inject your own:

Featurizer (i.e. a converter from any object into a TensorDict of string values) - you don't need to use any of the VIzier APIs.
Vocabulary file (I recommend using default T5X vocabulary file). The vocabulary will be amended by custom y-representation tokens, e.g. <+>,<->,<0>,<1>,...

Because at the end of the day, you need a tf.Dataset where elements are of the form {"inputs": x, "targets": y} where x is some string representation of your features, and y is of form e.g. <+><3><4><5><E-2> which can be obtained using our [DigitByDigitFloatTokenSerializer](https://github.com/google-research/optformer/blob/main/optformer/common/serialization/numeric/tokens.py#L27-L129)
Hope this helps,

Richard


fix) imports broken

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #7   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          10     12    +2
=====================================
+ Hits           10     12    +2

Impacted Files
Coverage Δ

appDataPath.go
100% <100%> (ø)

appDataPath_linux.go
100% <100%> (ø)

Continue to review full report at Codecov.

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


set proxy when connecting to cc server

HI
I want to be able to set a proxy address when connecting to cc, and route all of my traffic to cc through that address
can I do this in my app?
thanks

A fetch is used under the hood for HTTP requests.
If you have  a code snippet for fetch re how to set a proxy - please share, we will point further

I think you can use this lib https://github.com/touv/node-fetch-with-proxy
and overwrite a standard fetch implementation


How to run jeeliz on a static image or custom video stream?

I noticed that using JEEFACEFILTER.init() only allows for detection of the face on the video stream accessed through getUserMedia(). As far as I understand there is no way to control the image/video which jeeliz analyses. I found out however that your sunglasses example allows to upload a custom picture.
Is it possible to input custom image/video stream to jeelizFaceFilter ? If yes then how can I do it ? I've looked through the readme and tutorials but with no luck :(

This is possible: https://jeeliz.com/demos/faceFilter/demos/canvas2D/fromVideoFile/
and the source code: https://github.com/jeeliz/jeelizFaceFilter/blob/master/demos/canvas2D/fromVideoFile

Thanks!

This link is not working any more
https://jeeliz.com/sunglasses/?sku=rayban_new_wayfarer_havane_marron_clair_degrade
as this server is not accessible
https://fallbackglassesdb.jeeliz.com/
Is there any other accessible link with this example of using an image as a source intead of a webcam?

Hi,
There was a problem with the image rendering server.
I have restarted it and

Thank you for the quick turnaround.
Where I can read about licensing of the module?

Hi,
Licensing of FaceFilter (this repo) is free since it is released under Apache2 licence.
However we provide developement services (integration/neural network improvement/specifics needs) using our libraries ( https://github.com/jeeliz ) to fit specific needs.
The sunglasses virtual tryon module used in https://jeeliz.com/sunglasses or https://github.com/jeeliz/jeelizGlassesVTOWidget is a specific case, because for this we provide the whole solution (hosting, glasses modelization, server side fallback rendering).


[Feature Request] Mediainfo add toggle keybind

At present the info stays on screen for 5 seconds. I would prefer if a toggle keybind can be added.

This cannot be done at present. This is the limitation of script implementation.
https://github.com/dyphire/mpv-scripts/blob/3fe0551c723232f18b51e97675c00138ab12cbcf/mediaInfo.lua#L79-L82
I am not interested in implementing it for now.


With a menu on a node, passed element is often a different empty node.

When right button clicking on a node I get the correct menu for a node, yet sometimes the element passed is for an unknown node.
It appears that an empty node element is generated on the first right button click, located next to the selected node, and then subsequently that element is often returned rather than the correct node for subsequent menu selections.

Just to add to this, my implementation uses two layouts, "cose" and "dagre" this only happens with the "cose" layout.

Yet another update. Further investigation shows that the nodes that are selected in place of the original are those generated by the edgehandles project.
I have, what must be a common choice, cytoscape.min.js,  cytoscape-cxtmenu.js, and cytoscape-edgehandles.js running in my application, with a couple of layouts.  I was wrong previously: this defect occurs with all the layouts I have tried.
The bug I have found is that inside cxtmenu, the element passed to the selection function can be either the genuine node, or the transitory node generated by the edgehandles code to represent the starting point of an edge drawing process.
I would have thought that most implementers would have used these three cytoscape projects together, so it surprises me that this bug has not occurred before.
Knowing more fully what has caused this gives me a chance to build a workaround - but I'm surprised one doesn't exist already.

See https://github.com/cytoscape/cytoscape.js-edgehandles/issues/143
If you're interested in contributing to this issue, you're welcome to create a pull request in the edgehandles repo.
I'm closing this issue, since it doesn't look like it's specific to the context menu.
All the best with your project.

Don't know if it still helps, but we found a workaround for this exact problem. https://github.com/cytoscape/cytoscape.js-edgehandles/issues/166#issuecomment-931268439


Support GRUs of cuDNN (GRU/BiGRU)

cuDNN RNN supports various types of RNNs.
I create GRU/BiGRU which use NStepRNN common class. (#2467 )

[x] NStepGRU
[x] NStepBiGRU

LGTM


UIHostingViewController not working correctly

Hi, thanks for making this available.
I just tried using the SwiftUIX.UIHostingViewController but find that it does not work quite like it should.
I have a SwiftUI View than displays a number of items in a subview, however if this view changes size then the animations to the new size are not correct.
One can see that when the subview changes size it is expanding upwards (if it is getting taller) before the hosting view is getting resized - so there is this visual animation of things moving upwards and then moving back down when the hosting view itself is performing its layout and resizing itself.
It seems like somehow the view needs to be told to animate the change in size at the same time as the subview is changing its size.

@duncangroenewald - please answer @vmanot, thanks.

Closing due to inactivity, @duncangroenewald please feel free to reopen if you have more info/a repro :)


No documentation and how to define behaviour

There's no documentation that I can see on how to actually define  if behaviour is scroll to top  or preserve the scrolling

Thanks for opening an issue with me.
Right, This document needs to be improved. I added the following instructions:

vue-scroll-behavior allows you to customize the scrolling position on route navigation. Especially hash mode. You just need to tell it which routes need to be processed, and it will do that automatically

If you have some special scenes, we also provide some options, and you can manually use them to save or restore the scroll position

By default, It will scroll to the saved position.  If you have some special scenes, we also provide some options, and you can manually use them to save or restore the scroll position.
But, some options, I haven't finalized yet. You can check the below issue: https://github.com/jeneser/vue-scroll-behavior/issues/3
If you have any good ideas, please let me know.
Thanks.


Using xml-config file for dynamic loading of plugin dll doesn't work in Net6

I have been using Windsor container with config via new WindsorContainer("path_to_config") for a while in NetFramework till we've decided migrate to Net6. And in Net6 it doesn't work anymore. I can load class from dll only if that dll is referenced from my program.
If I remove reference - I get error
Exception: Castle.MicroKernel.SubSystems.Conversion.ConverterException: Could not convert string 'ClassLibrary.TestClass, ClassLibrary' to a type. Assembly was not found. Make sure it was deployed and the name was not mistyped.
at Castle.MicroKernel.SubSystems.Conversion.TypeNameConverter.GetType(String name)
at Castle.MicroKernel.SubSystems.Conversion.TypeNameConverter.PerformConversion(String value, Type targetType)
at Castle.MicroKernel.SubSystems.Conversion.DefaultConversionManager.PerformConversion(String value, Type targetType)
at Castle.MicroKernel.SubSystems.Conversion.DefaultConversionManager.PerformConversion[TTarget](String value)
at Castle.Windsor.Installer.DefaultComponentInstaller.GetType(IConversionManager converter, String typeName)
at Castle.Windsor.Installer.DefaultComponentInstaller.SetUpComponents(IConfiguration[] configurations, IWindsorContainer container, IConversionManager converter)
at Castle.Windsor.Installer.DefaultComponentInstaller.SetUp(IWindsorContainer container, IConfigurationStore store)
at Castle.Windsor.WindsorContainer.Install(IWindsorInstaller[] installers, DefaultComponentInstaller scope)
at Castle.Windsor.WindsorContainer.Install(IWindsorInstaller[] installers)
That is a client code:
    try
    {
        var container = new WindsorContainer();
        container.Install(Configuration.FromXmlFile("config.xml"));
        var driver = container.Resolve<IDriver>();
        driver.Start();
    }
    catch (Exception ex)
    {
        Console.WriteLine($"Exception: {ex}!");
    }

That is a config.xml file:

<components>

    <component id="ViaQuatroIntegrationDriver" service="CommonClassLibrary.IDriver, CommonClassLibrary" type="ClassLibrary.TestClass, ClassLibrary">
      <parameters>
      </parameters>
    </component>

</components>

If I add reference to ClassLibrary.dll - all works as expected.
I'm using Castle.Windsor 5.1.2 from nuget
I've tried to examine source code and didn't find any tries to load dll from config dinamically.
So, is that a changed behavior? Or, may be, I missed some special option/property/whatever? Is it still possible to load dll dynamically via config file, without reference to dll?

Ok, if I understand all correctly from #619 #622 and other requests about FEATURE_ASSEMBLIES that trouble should be fixed in 6.0.0 version


xFailOverCluster: Opt-in for module files common tests

Pull Request (PR) description

Changes to xFailOverCluster

Opt-in for module files common tests (issue #119).

Removed Byte Order Mark (BOM) from the files; CommonResourceHelper.psm1 and FailoverClusters.stubs.psm1.

This Pull Request (PR) fixes the following issues:
Fixes #119
Task list:

[x] Change details added to Unreleased section of CHANGELOG.md?
[ ] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[x] New/changed code adheres to Style Guidelines?
[x] Unit and (optional) Integration tests created/updated where possible?

This change is

@johlju,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #120 into dev will not change coverage.
The diff coverage is n/a.

@@        Coverage Diff         @@
##           dev   #120   +/-   ##
==================================
  Coverage   90%    90%
==================================
  Files        7      7
  Lines      379    379
==================================
  Hits       343    343
  Misses      36     36

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Should relude export mask, onException, finally?

mask and friends are key to writing good exception safe code.  Its absence from relude entirely could be easily, and without disturbance, be rectified with a Relude.Extra.Exception module.  Agreed? What about exporting these operations by default from Relude?

@TomMD I agree that these functions are useful. But one goal of the relude is not give any preferences to competing abstractions in the ecosystem, giving users the most freedom.

Minimalism (low number of dependencies). We don't force users of relude to stick to some specific lens or text formatting or logging library.

Regarding functions like mask: there are several libraries that export this function under different types:

base — mask :: ((forall a . IO a -> IO a) -> IO b) -> IO b
exceptions — mask :: MonadMask m => ((forall a . m a -> m a) -> m b) -> m b
lifted-base — mask :: MonadBaseControl IO m => ((forall a . m a -> m a) -> m b) -> m b
unliftio — mask :: MonadUnliftIO m => ((forall a . m a -> m a) -> m b) -> m b

Sticking to one option will conflict with intentions of other users. They will need to hide this function and export from another library. Also, reexporting mask from any other place than base will introduce new dependencies.

I see.   Thanks for the clarification.


CVE-2024-32650

Would you mind requiring a newer version of rustls due to: https://github.com/rustls/rustls/security/advisories/GHSA-6g7w-8wpp-frhj

Sure! Thanks for bringing this up.
sa


CHANGELOG: update from #9845

Update CHANGELOG-3.4 for #9845

@gyuho Sorry! I will be more careful... Thanks!


Bug: Other mods' structures spawning in the Aether

What happened?
I seem to be getting Vanilla Mineshafts and the cabins from Subterrestrial spawning in the Aether.

Mod Version
1.6.6
Fabric API Version
0.48.0
Relevant log output
No response
Other mods
Subterrestrial 1.0.2
Additional Information
No response

Please forward this to Subterrestrial

Locked and pinned for others with similar issues

This guidance also applies to other mods' mobs spawning in the aether. Please forward it to the mods in question.


Fixed typos, grammatical errors, etc.

Description
Summary of the change (What have you documented? What have you corrected?): Fixed typos, grammatical errors, rephrased sentences, etc.
Type of change

[x] Typo/grammar correction
[ ] Documented an existing feature
[ ] Documented a new feature
[ ] Documented a new supported device

Checklist:

[x] I have performed a self-review of my own text
[x] I have checked my text and corrected any misspellings.
[x] I have read the CONTRIBUTING document.

Thank you for the PR but please do not update the project docs (CONTRIBUTING, issue templates, CoC, LICENSE etc.)

@SincerelyFaust In that case would you mind merging that into the template repository? These do seem to be quite severe grammatical issues.

I've seen a few, yes, and I will apply fixes, correct.

Manually merged, I'll be sure to help out and roll this out into repos asap.

Thank you for the PR but please do not update the project docs (CONTRIBUTING, issue templates, CoC, LICENSE etc.).
We will do those ourselves.

I don't understand how fixing the project docs harms the documentation but fine, I will not edit them anymore. I was just trying to help.

@Daksh777 Sorry, it was just a simple misunderstanding, we will manually merge this into our template, these files are replicated across all repositories.


QR Code Size

Hi, using my pass in wallet app is fantastic, thank you so much. However I have a little issue, basically sometime the QR code is difficult to read, usually on low-end smartphones. Is it possible to make the QR code bigger or is it standard for general passes on the apple passkit? Thank you so much.

Duplicate of #75.


nicer docstrings

How can we render better docstrings?
See for example
https://gdsfactory.github.io/gdsfactory/components.html#mzi1x2-2x2
@flaport

Do you mean docstrings or call signatures? For me the call signature looks a lot more scary than the docstringe...
I personally like autodoc_typehints='description' to relieve some of the pressure on the call signature.

Also, if the type hint becomes too big it's worth aliasing your type hint for a shorter version.

nice, autodoc_typehints='description'  worked great!
how could we also show a better function name for partials?
add_pins=functools.partial(<function add_pins_siepic>

autodoc_typehints='description' merged
i also added a css to do one parameter per line
https://github.com/gdsfactory/gdsfactory/pull/510


S3 module

This is the new module. I’m pushing it because it works now but there are still quite a lot of caveats and TODOs.
Please disregard directory and file structure, I will fix them later.
The main things to review right now is the general architecture, the way state is advanced (this is the specialized, time based version) and especially how do we pass the Transducer.
Finally things that I could not solve are related to the default behavior: the key and the value in the simplest case are wrapped by case classes which are awkward, any ideas in that area is appreciated.

Ok, there are still a few things off (no parallelism) but this could be an mvp.

Ok, there are still a few things off (no parallelism) but this could be an mvp.

Never mind, I just realized it continuously publishes the latest state even if unchanged.

Never mind, I just realized it continuously publishes the latest state even if unchanged.

@sirocchj actually I don't know if this is by design. Should the state be always republished when unchanged? I can't find the bit of code that would prevent this.

@sirocchj actually I don't know if this is by design. Should the state be always republished when unchanged? I can't find the bit of code that would prevent this.

I need to actually look into this PR. You're correct saying it's "by design". But the design is also that only the consumption of a new state triggers invocation of the fetch... so I'm not sure how this is supposed to work here

I need to actually look into this PR. You're correct saying it's "by design". But the design is also that only the consumption of a new state triggers invocation of the fetch... so I'm not sure how this is supposed to work here

One proposal could be that the state fold function becomes blocking. I doesn't do anything until the state changes. This will trow a wrench in the gears of the kafka loop.

One proposal could be that the state fold function becomes blocking. I doesn't do anything until the state changes. This will trow a wrench in the gears of the kafka loop.

Anything more we need to address here? Are we good to merge?

Anything more we need to address here? Are we good to merge?

Do you think we should have a more involved example app?

Do you think we should have a more involved example app?

Probably worth it

Probably worth it

Probably worth it

done in #235


zstandard-verify: ignore error.UnknownContentSizeUnsupported

Thanks! Might also make sense to ignore error.DictionaryIdFlagUnsupported but decodecorpus might not ever generate files with that set.


getting error while running npm install on mac m1 - 'ERR_DLOPEN_FAILED'

I'm on an m1 mac and i've installed node using homebrew. When i run npm install using the terminal, i don't get any issues but when i run npm install in vscode, i get vulnerabilities error. After doing an npm audit check, it's asking me to use different dependencies.

Because whenever i use npm run generate inside vscode, i get this error:

Should i downgrade to node v14 or something? How do i fix this?

Solution


fix: fix import on demand mode

PR Checklist

[x] Fix linting errors
[x] Label has been added

Change information
fix #74
fix #75

WIP
testing

效率真高，点赞


Updates

Solved daily puzzles
Added title metadata to JSON output
Added command line interface for each puzzle
Updated the daily helper
Various bug fixes

Codecov Report

Merging #9 (1ec64ce) into main (310fce8) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              main        #9    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           47        52     +5
  Lines         1772      2187   +415
==========================================
+ Hits          1772      2187   +415

Impacted Files
Coverage Δ

advent_of_code/year2016/day12.py
100.00% <ø> (ø)

advent_of_code/year2016/day13.py
100.00% <ø> (ø)

advent_of_code/year2016/day14.py
100.00% <ø> (ø)

advent_of_code/year2016/day15.py
100.00% <ø> (ø)

advent_of_code/year2016/day16.py
100.00% <ø> (ø)

advent_of_code/app.py
100.00% <100.00%> (ø)

advent_of_code/year2015/day10.py
100.00% <100.00%> (ø)

advent_of_code/year2015/day11.py
100.00% <100.00%> (ø)

advent_of_code/year2015/day12.py
100.00% <100.00%> (ø)

advent_of_code/year2015/day13.py
100.00% <100.00%> (ø)

... and 36 more

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


Support browser PNG

16-bit PNGs don't display well in the browser, so it's helpful for testing and demoing to be able to generate 8-bit PNGs as well. This adds that functionality, as well as cleaning up a few other things noticed / fixed while deploying to EC2, notably changing the single zoom output to a list of zooms.
Connects to #9.
@rmarianski could you review, please?

:+1:


how to compute tasks' metrics by their sub-tasks

As the picture shows below, the lm-evaluation-harness computes the metrics of sub-tasks (such as leaderboard_gpqa_diamond/extended/main), how can i get the metrics of leaderboard_gpqa?

There are other examples, I want know the metrics of bbh, but I got only the sub-tasks score.

Hugging Face Blog discusses the idea behind normalization of  scores -
https://huggingface.co/spaces/open-llm-leaderboard/blog
The exact method and scripts can be found here
https://huggingface.co/docs/leaderboards/en/open_llm_leaderboard/normalization


boards: Fix device UUID logging for nrf52

Fixes: https://github.com/Nitrokey/nitrokey-3-firmware/issues/498

No significant changes.

Insignifcant changes

metric
value

change

binary-size-nk3am
1423270
:white_circle:
+0 (+0.00%)

binary-size-nk3am-test
2022174
:white_circle:
+0 (+0.00%)

binary-size-nk3xn
510272
:white_circle:
+0 (+0.00%)

binary-size-nk3xn-test
547156
:white_circle:
+0 (+0.00%)

binary-size-nkpk
736421
:white_circle:
+0 (+0.00%)


Fix show command only

This change addresses item #3381
This fixes a bug where showing configure or build commands, would run these commands. This is caused by passing wrong arguments to the function configureInternal

@microsoft-github-policy-service agree

This change will be publicly available on our pre-release channel in the VS Code Marketplace within 24 hours and will be included in our upcoming 1.16 release which is currently scheduled for mid-November. Thanks again for your contribution!


HHH-17322 Allow custom generators to determine timing based on current entity state

https://hibernate.atlassian.net/browse/HHH-17322

This change looks good!


graphiql schema editor not rendering properly

Instead of rendering in a schema pane iit is rendering in a result pane

strange, can you show your component implementation code by chance?

strange, can you show your component implementation code by chance?

I am getting results as expected. the problem is only with the Document Explorer rendering.
I am using custom endpoint like /abc/v1/graphql  and implementing using controllers and routing in ASP.net core.
In startup configuration
app.UseGraphiQl("/graphql/ui", "/abc/v1/graphql");
service registration as below
services.AddSingleton<IEntityService, EntityService>();
services.AddSingleton();
services.AddSingleton();
services.AddSingleton();
services.AddSingleton();
services.AddSingleton<ISchema, ABCSchema>();
services.AddSingleton(
c => new FuncDependencyResolver(type => c.GetRequiredService(type)));
controller.txt

strange, can you show your component implementation code by chance?

I am getting results as expected. the problem is only with the Document Explorer rendering.
I am using custom endpoint like /abc/v1/graphql and implementing using controllers and routing in ASP.net core.
In startup configuration
app.UseGraphiQl("/graphql/ui", "/abc/v1/graphql");
service registration as below
services.AddSingleton<IEntityService, EntityService>();
services.AddSingleton();
services.AddSingleton();
services.AddSingleton();
services.AddSingleton();
services.AddSingleton<ISchema, ABCSchema>();
services.AddSingleton(
c => new FuncDependencyResolver(type => c.GetRequiredService(type)));
controller.txt

Solved 👍 Added : app.UseGraphQL("/abc/v1/graphql"); in configuration
and changed ISchema to Schema in controller file.

ah ok! sorry i couldn't help you with this implementation, I haven't used ASP.net in years, glad you resolved it!


winget upgrade winget

Description of the new feature/enhancement

Would be very cool if I could run winget upgrade winget
Proposed technical implementation details (optional)

Many other similar tools can upgrade themselves.

Duplicate of #505


Permit combing secrets with raw data

For example,
---
apiVersion: digitalis.io/v1
kind: ValsSecret
metadata:
  name: my-secret
spec:
  ttl: 3600       # Optional, default is 0. The secret will be checked at every "reconcile period". See below.
  type: Opaque    # Default type, others supported
  data:
    username:
      ref: ref+vault://secret/username
      encoding: text
    password:
      ref: ref+vault://secret/password
      encoding: text
    server:
      plain: my-server.com

I just realised this already works:
  data:
    username:
      ref: plain-text-username
      encoding: text

As the ref+ prefix is not set, it won't be process by the underlying vals.
@rgooding Would you happy with just documenting the feature or do you think we should create a separate config key for plain text/raw data?

I think documenting this would be enough for now, we can always review it later if it causes problems or if people find it confusing


Convert the column index writer projector

Codecov Report

Merging #5037 into batch-iterator will decrease coverage by -0.07%.
The diff coverage is 92.85%.

@@                 Coverage Diff                  @@
##             batch-iterator    #5037      +/-   ##
====================================================
- Coverage             82.34%   82.28%   -0.07%
+ Complexity            10953    10949       -4
====================================================
  Files                  1325     1325
  Lines                 45156    45162       +6
  Branches               5685     5685
====================================================
- Hits                  37185    37160      -25
- Misses                 5300     5326      +26
- Partials               2671     2676       +5

Impacted Files
Coverage Δ
Complexity Δ

...ate/operation/projectors/IndexWriterProjector.java
78.94% <100%> (+0.76%)
9 <2> (+1)
:white_check_mark:

...eration/projectors/ColumnIndexWriterProjector.java
83.33% <100%> (+1.11%)
10 <2> (+2)
:white_check_mark:

...tion/projectors/IndexWriterCountBatchIterator.java
84.5% <100%> (+9.5%)
23 <0> (+3)
:white_check_mark:

...eration/projectors/BatchConsumerToRowReceiver.java
90% <75%> (+0.29%)
24 <5> (ø)
:x:

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
80% <0%> (-20%)
2% <0%> (ø)

...te/blob/pending_transfer/PutHeadChunkRunnable.java
67.12% <0%> (-9.59%)
11% <0%> (-1%)

core/src/main/java/io/crate/types/ArrayType.java
73.84% <0%> (-7.7%)
24% <0%> (-2%)

...src/main/java/io/crate/jobs/UpsertByIdContext.java
77.5% <0%> (-7.5%)
8% <0%> (ø)

...va/io/crate/protocols/postgres/types/JsonType.java
77.41% <0%> (-6.46%)
8% <0%> (-1%)

.../java/io/crate/blob/TransportDeleteBlobAction.java
75% <0%> (-5%)
5% <0%> (-1%)

... and 26 more

Continue to review full report at Codecov.

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


force resending states

When you change your configuration or restart http://www.openhab.org/ the current states of your device might get lost (Yes, you can prevent this).
But it would be nice to send a mqtt message to let the device resend the initial messages.
devices/15b638e0/$*
An alternative workaround would be to reset the device remotely (https://github.com/marvinroger/homie-esp8266/issues/33)

I think a feature to resending specific values like temperature (rather than wait for the next TEMPERATURE_INTERVAL) need to be implemented by the user.

This is already handled by MQTT, and the initial messages are sent with the retain flag set to true, which means that when a new client connect it receives all the latest values sent by the device.

This would not help in my case, since openhab can lose the values while still keep up the connection to the mqtt broaker.
But like I said, this is not an issue when openhab ist configured correctly.
And I dont want you to break the mqtt standards. :)

I am not in favor of that, I don't want to workaround a bad behavior of a 3rd party software. openHAB should fix the issue on its side, don't you think? And as you said, this is not an issue if it's configured correctly. ;)