Uploaded image cannot be displayed

When uploading an image with the function provided by either blogs or thoughts, it will generate a url looks like:
https://raw.githubusercontent.com/username/tinymind-blog/main/assets/images/2024-09-24/1727160262328.jpg

However, this link seems broken and the image cannot be displayed correctly:

The picture itself can be found in the assets dir of the repo, and there the pictures are properly uploaded. If obtain the url from the repo directly, which looks like:
https://github.com/username/tinymind-blog/blob/master/assets/images/2024-09-24/1727159948894.png?raw=true

Then the picture can be displayed correctly.

Thank you, this issue has been fixed.

Sorry, I made a mistake. The problem was not about whether the url is started with raw.githubusercontent.com or github.com. It was because the default name of the main branch. To me, the default name is "master" instead of "main", so the original raw.githubusercontent.com url didn't work. After modifying the branch name of the raw url, the picture can also be displayed properly.

Try now. This commit https://github.com/mazzzystar/tinymind/commit/2b4ed39feeb9c443880d6dbbdf136c394755084d make it supports both main & master branch.

It worked!
One thing I'm not sure is which kind of image url is better? Are they both acceptable urls, or maybe one of them is preferred? I hope I didn't disrupt your original decision there...


Fix parsing ALTER TABLE ... ADD COLUMNS (...)

The current behaviour fails when it encounters ADD COLUMNS and merges everything in the () into one string with no whitespace.
So ALTER TABLE catalog.table.name ADD COLUMNS (apples int) becomes ALTER TABLE catalog.table.name ADD COLUMNS (applesint).
This commit adds an understanding of ALTER TABLE ... ADD COLUMNS to the grammar.

Thank you for your contribution, please fix formatting exception by running './gradlew :spotlessApply' to fix these violations.

Done

Thank you for your contribution!


Ubuntu 17.04 is not correctly setup after running launch, causing apt to fail weirdly

This issue makes it pretty hard to build containers.
Note how in LXC output with delayed apt-get update everything works as expected (I just added sleep 10 before apt-get update).
I haven't found a workaround so far. I tried something like curl --retry 10 https://www.example.org but the network does seem to be working before apt-get update fails.
I'm running Ubuntu Gnome 17.04.
There are replication steps in LXC output.
Distribution:
$ lsb_release -a
No LSB modules are available.
  They have been ignored, or old ones used instead.

 Done

Container log
04:41:23 {lxd} ~/code/box $ sudo lxc info --show-log just-a-test
Name: just-a-test
Remote: unix:/var/lib/lxd/unix.socket
Architecture: x86_64
Created: 2017/09/16 15:38 UTC
Status: Running
Type: persistent
Profiles: default
Pid: 16535
Ips:
  eth0:	inet	<IP_ADDRESS>	vethRJP3ID
  eth0:	inet6	fd42:d2a7:a7b5:d6b:216:3eff:feba:16f3	vethRJP3ID
  eth0:	inet6	fe80::216:3eff:feba:16f3	vethRJP3ID
  lo:	inet	<IP_ADDRESS>
  lo:	inet6	::1
Resources:
  Processes: 29
  CPU usage:
    CPU usage (in seconds): 6
  Memory usage:
    Memory (current): 53.09MB
    Memory (peak): 167.96MB
  Network usage:
    eth0:
      Bytes received: 10.42kB
      Bytes sent: 2.66kB
      Packets received: 89
      Packets sent: 28
    lo:
      Bytes received: 13.17kB
      Bytes sent: 13.17kB
      Packets received: 166
      Packets sent: 166

Log:

I noticed this https://github.com/lxc/lxd/issues/3700.
Is there any planned support for something that waits until the system is in a "ready" state? I'm not even sure how it would work. The test for ubuntu username doesn't really work. It's not that the username isn't setup I don't think.

The behavior you're describing is expected and isn't any different than what you'd get if running code very early on in a virtual machine or similar systems. There is no reliable cross-distro way to know when a system is ready as the definition of "ready" very much varies on what you plan on doing with it.
LXD's interaction with a container finishes when the init system is spawned inside it, which is when "lxc launch" or "lxc start" return. To block until network is functional, I tend to use something along the lines of:
while :; do
     lxc file pull just-a-test/etc/resolv.conf - | grep -q nameserver && break
     sleep 1
done

Which will wait until /etc/resolv.conf contains a DNS server as that's usually a reasonable indication that the network is functional. Note that very recent systemd releases may however cause a false positive there as systemd-resolved may start running before DNS is actually configured.
Assuming your init system has some way of checking if it's done bringing up networking, you could re-use the loop above but instead using some kind of "lxc exec" check.

We did discuss an extension of the /dev/lxd/socket protocol to allow signaling LXD when the container is ready but that's not been implemented yet and when it is, you'd still need to manually craft an init script which will talk to it once your container reaches what you consider to be a ready state.

For the specific apt scenario here, one option is to use cloud-init to update and install any package you need. You can pass a cloud-init configuration through the user.user-data configuration key of the container.

@AndrewVos Cloud-init is probably one of the better ways to handle clouds like lxd, etc.  It has builtin handling for executing command in the right order, apt update after network is up, etc.  If you're writing your own scripts, then you can also use the following to test when cloud-init has finished, which shouldn't complete til the network is up.  Even if not using cloud-init to do actual configs, if you're using the cloud-enabled versions, then the following will let you know when it's safe to continue on.
while [ ! -f /var/lib/cloud/instance/boot-finished ]; do echo 'Waiting for cloud-init...'; sleep 5; done;

I think I might be using the wrong tool for what I'm doing tbh. I'm trying to convert the tests for https://github.com/AndrewVos/box because Lxd seemed like a more lightweight docker. I would prefer to avoid docker for something so simple.

Think of LXD as a lightweight version of a hypervisor, minus the hardware emulation, and bios.  And the kernel is shared, instead of being a separate running instance.  But like anything else, including docker, the container still has to boot, before you can execute commands against it.

Thanks, I'll continue to use Docker for now then.
Thanks so much for your help @jfgibbins, I do appreciate it.


Feature Request: Event Listener

Hi @webyonet first of all, thank you for this great plugin. I'm requesting if you can have a listener that listens when the user turns off their location service because as of now you have to call the function first before triggering the dialog. In my use case it should be triggered whenever the user turns off their location service. Thanks!

1

+1

I'm using React Native GPS State and it working


Merge-On-Read performance degrades for single partition table [SUPPORT]

Describe the problem you faced
Currently we are streaming data, upserting into a Merge-On-Read table.  The total table will contain 350M entities bounded, and we expect the approximate table size to be 10+ TB.  For the first 10M records the batches were completing quickly however we started to see the batch time slowly growing by about 1 minute per 5-10 batches.  We have enable inline compaction for testing and set INLINE_COMPACT_NUM_DELTA_COMMITS_PROP and INLINE_COMPACT_PROP to 12 and are currently running 15 minute batches in Spark.
Expected behavior
With the Merge-On-Read semantics I would expect most writes to be quick and the compaction to take longer and longer. however I see no difference in speed across the 12 batches they all take about the same amount of time
Environment Description
Spark running in Azure, Storage Layer ADLS_V2 and cross tested with GCS
Environment Description
Hudi version : 0.5.3
Spark version : 2.4-1.0.5
Hive version :
Hadoop version : 2.7
Storage (HDFS/S3/GCS..) : ADLSv3 and GCS
Running on Docker? (yes/no) : yes and no, both
Additional context
Add any other context about the problem here.
Stacktrace
No exceptions are thrown, batch times are degrading.
@christoph-wmt

Can you please share spark ui for these slow runs , so we can tune and see what’s going on

@sam-wmt : Pinging to see if you can provide debugging information.

Apologies for the slow response the initial issues appeared to be due to a hanging compaction which was addressed in a newer version which attempts a retry.  We'd done so further work and another devloper has further issues due to a lot of small files which can be seen here.  This Support request can be closed in favor of his: https://github.com/apache/hudi/issues/1852
Kindest Regards,
Sam

Thanks for the update @sam-wmt, closing this.


[BUG] Cannot activate Claude (seems checking availability using non-existent API)

Describe the bug / 描述问题
Cannot use Claude.
The Claude button stays turned off.
To Reproduce / 复现步骤

Click Claude button

The first click will open the login instruction panel, but following clicks have no effect even after logging in.
Expected behavior / 期望行为
Can activate Claude.
Screenshots / 截图

Devtools Info / 开发者工具信息

OS and version / 操作系统版本
macOS 14.4.1
ChatALL version / ChatALL 版本
v1.79.105
Network / 网络
No
Additional context / 其它相关信息
There seems to be a new API.
https://claude.ai/api/account

I have the same issue.

I have the same issue.

I have the same issue.


Add directivesEndMarker to Document type

Noticed this was missing when I tried to create a document in typescript on v1.9.2
Not sure if this is the right place to add or if the .d.ts files are generated in someway

Thank you! That's exactly the right fix for now. I wish that the d.ts files could be generated somehow, but the scripting for that turned out to be more difficult than doing the work manually.


SFR-1064 New Handlers

This adds two new handlers for deployment options for the conversion service. The first provides handlers that can be invoked via the AWS Lambda service and the second provides a generic means of deploying the API as a containerized service. These handlers provide exactly the same method calls as the vercel handlers, differing only in how the responses are constructed/sent.
These handlers are documented in the CHANGELOG and some new dependencies have been added, though these should have no impact in how the service is deployed to vercel.
Some questions to address before approving/merging:

What should the testing strategy be for these handlers? The actual conversion method calls are tested already so it should only need to confirm that each handler accepts the path parameter properly and returns the expected status code and payload
Is additional documentation needed/preferred?
The types in the validateParam method had to be extended somewhat, is this potentially an issue?

It looks like we need to update the github actions scrips to ignore the optional dependency on the private @nypl-simplified-packages/axisnow-access-control-web manually. I will make a commit doing that if that's okay

@kristojorg yeah that makes sense to me, I was wondering why that was failing

Looks like the error that is causing the failures is coming from a test suite that is skipped anyway, could the offending import/require just be commented out? If the tests are enabled and it's still commented the tests will fail, which would ensure that it's not forgotten about.

@mwbenowitz yep that would have been the logical and easiest thing to do 😓  ... if my current change doesn't fix it then I'll just do that

If you wanted to write some tests for the handlers you wrote that would be awesome, but I'll let you make the call on whether it's necessary or not : ). It looked like my fix worked so feel free to merge if you don't think the tests are necessary.

Added the tests and they pass but the ubuntu runs hang, not sure why


fix: fix tests using the new sdk

Description
Fixing e2e and unit tests with the new sdk
Motivation
E2e and unit tests failing after the new sdk migration.
Tests included

[x] Included for each change/fix?
[x] Passing? (Merge will not be approved unless this is checked)

Documentation

[x] swagger documentation updated [required]
[ ] official documentation updated [nice-to-have]

official documentation info
If you have updated the official documentation, please provide PR # and URL of the pages where the updates are included
Backend version

[ ] Does it require a specific version of the backend
which version of the backend is required:

Summary by Sourcery
Bug Fixes:

Fix tests to use the new SDK by updating service imports and method calls across multiple test files.

Summary by Sourcery
Fix tests to work with the new SDK by updating service imports and method calls. Refactor Cypress commands for better token handling. Update test files to ensure compatibility with the new SDK structure.
Bug Fixes:

Fix tests to use the new SDK by updating service imports and method calls across multiple test files.

Enhancements:

Refactor Cypress commands to use a new method for retrieving tokens, improving code readability and maintainability.

Tests:

Update test files to align with the new SDK structure, ensuring compatibility and functionality.

I tested it locally and it works fine on my machine

Great! Thank you @Junjiequan


Add a check on BTC forks.

Dear Mizzogg!
Please add the verification of Bitcoin forks: BCH, Bitcoin SV
, BTG, etc. in the "Option 7" menu. If possible. Thanks.

Hi Mir0m0v,
I might try and add in next versions. But the main API I was using in older versions of the Tools is now not working and the newer one is limited to results and certain types of Cryptos.
Lets see what happens in next version....


add spread operator for attributes

As we can't remove the wrappers completely for now (#10), let's make them more flexible!
this PR allow the wrapper to have attributes defined to it, e.g. className.
It will solve at least one of the problems described on #5.

As we can't remove the wrappers completely (#10), let's make them more flexible!
this PR allow the wrapper to have attributes defined to it, e.g. className.
It will solve at least one of the problems described at #5.

@krasimir I would appreciate if you could prioritize this PR. The other ones I sent are not so relevant :)

@zvictor 1.2.2 version released containing your change.

awesome, thanks!


Wrong shape dimension check happens in Pooling function, leading to direct crash: src/operator/nn/pooling.cc:109: Check failed: dshape.ndim() >= 3 (-1 vs. 3)

Description
Wrong shape dimension check happens in Pooling function. Will directly crash on both mxnet 1.5.1 and mxnet 1.8.0.
Error Message
Using MXNet backend
lib/python3.6/site-packages/keras/__init__.py:31: DeprecationWarning: MXNet support in Keras is going to be discontinued and v<IP_ADDRESS> is the last release as multi-backend Keras has been discontinued . It is recommended to consider switching to MXNet Gluon. More information can be found here: https://github.com/awslabs/keras-apache-mxnet
  "https://github.com/awslabs/keras-apache-mxnet", DeprecationWarning)
Traceback (most recent call last):
  File "try.py", line 35, in <module>
    x = layers.Conv2D(filters=3, kernel_size=(5, 5))(x)
  File "lib/python3.6/site-packages/keras/engine/base_layer.py", line 470, in __call__
    output = self.call(inputs, **kwargs)
  File "lib/python3.6/site-packages/keras/layers/convolutional.py", line 175, in call
    dilation_rate=self.dilation_rate)
  File "lib/python3.6/site-packages/keras/backend/mxnet_backend.py", line 3709, in conv2d
    padding_mode=padding, data_format=data_format)
  File "lib/python3.6/site-packages/keras/backend/mxnet_backend.py", line 95, in func_wrapper
    train_symbol = func(*args, **kwargs)
  File "lib/python3.6/site-packages/keras/backend/mxnet_backend.py", line 5049, in _convnd
    padding, is_slice, out_size = _preprocess_padding_mode(padding_mode, x.shape,
  File "lib/python3.6/site-packages/keras/backend/mxnet_backend.py", line 4399, in shape
    return self._get_shape()
  File "lib/python3.6/site-packages/keras/backend/mxnet_backend.py", line 4408, in _get_shape
    _, out_shape, _ = self.symbol.infer_shape_partial()
  File "lib/python3.6/site-packages/mxnet/symbol/symbol.py", line 1152, in infer_shape_partial
    return self._infer_shape_impl(True, *args, **kwargs)
  File "lib/python3.6/site-packages/mxnet/symbol/symbol.py", line 1210, in _infer_shape_impl
    ctypes.byref(complete)))
  File "lib/python3.6/site-packages/mxnet/base.py", line 253, in check_call
    raise MXNetError(py_str(_LIB.MXGetLastError()))
mxnet.base.MXNetError: Error in operator average_pooling2d_1/pool2d1: [22:12:42] src/operator/nn/pooling.cc:109: Check failed: dshape.ndim() >= 3 (-1 vs. 3) : Pooling: Input data should be  3D in (batch, channel, x) Or 4D in (batch, channel, y, x)  Or 5D in (batch, channel, d, y, x)
Stack trace:
  [bt] (0) 1   libmxnet.so                         0x000000011249e929 mxnet::op::NDArrayOpProp::~NDArrayOpProp() + 4473
  [bt] (1) 2   libmxnet.so                         0x00000001128dd540 mxnet::op::FullyConnectedComputeExCPU(nnvm::NodeAttrs const&, mxnet::OpContext const&, std::__1::vector<mxnet::NDArray, std::__1::allocator<mxnet::NDArray> > const&, std::__1::vector<mxnet::OpReqType, std::__1::allocator<mxnet::OpReqType> > const&, std::__1::vector<mxnet::NDArray, std::__1::allocator<mxnet::NDArray> > const&) + 750816
  [bt] (2) 3   libmxnet.so                         0x00000001139e535a std::__1::__tree<std::__1::__value_type<unsigned long, mxnet::NDArray>, std::__1::__map_value_compare<unsigned long, std::__1::__value_type<unsigned long, mxnet::NDArray>, std::__1::less<unsigned long>, true>, std::__1::allocator<std::__1::__value_type<unsigned long, mxnet::NDArray> > >::erase(std::__1::__tree_const_iterator<std::__1::__value_type<unsigned long, mxnet::NDArray>, std::__1::__tree_node<std::__1::__value_type<unsigned long, mxnet::NDArray>, void*>*, long>) + 42170
  [bt] (3) 4   libmxnet.so                         0x00000001139dd94e std::__1::__tree<std::__1::__value_type<unsigned long, mxnet::NDArray>, std::__1::__map_value_compare<unsigned long, std::__1::__value_type<unsigned long, mxnet::NDArray>, std::__1::less<unsigned long>, true>, std::__1::allocator<std::__1::__value_type<unsigned long, mxnet::NDArray> > >::erase(std::__1::__tree_const_iterator<std::__1::__value_type<unsigned long, mxnet::NDArray>, std::__1::__tree_node<std::__1::__value_type<unsigned long, mxnet::NDArray>, void*>*, long>) + 10926
  [bt] (4) 5   libmxnet.so                         0x0000000113979a15 MXSymbolInferShapeEx + 2581
  [bt] (5) 6   libmxnet.so                         0x000000011397aa30 MXSymbolInferShapePartialEx + 112
  [bt] (6) 7   libffi.7.dylib                      0x0000000110ab0ead ffi_call_unix64 + 85

To Reproduce
Script to produce such a problem
import os
import warnings

if __name__ == "__main__":
    os.environ["KERAS_BACKEND"] = "mxnet"
    warnings.filterwarnings("ignore", category=DeprecationWarning)
    warnings.filterwarnings("ignore", category=UserWarning)
    warnings.filterwarnings("ignore", category=FutureWarning)
    warnings.filterwarnings("ignore")
    gpu_ids = "1"
    os.environ["CUDA_VISIBLE_DEVICES"] = gpu_ids
    os.environ["MXNET_SUBGRAPH_VERBOSE"] = "0"
    os.environ["MXNET_CUDNN_AUTOTUNE_DEFAULT"] = "0"
    os.environ["MXNET_CUDNN_LIB_CHECKING"] = "0"

    import keras
    from keras import layers

    input_shape = layers.Input(shape=(10, 10, 6))
    # the error will disappear when not using softmax
    x = layers.Conv2DTranspose(filters=3, kernel_size=(3, 3), activation='softmax')(input_shape)

    # when using default setting(channel_last), mxnet will raise another error
    x = layers.AveragePooling2D(pool_size=(2, 2), strides=(2, 2), data_format='channels_first')(x)

    x = layers.Conv2D(filters=3, kernel_size=(5, 5))(x)
    new_model = keras.models.Model(input_shape, x)
    # print("successfully get the prediction result")
    del new_model

Steps to reproduce
directly run above program
What have you tried to solve it?
It seems that this is an inadequate assertion problem.
I looked into the code that trigger such bugs (which lies in line 109 of file src/operator/nn/pooling.cc)

And find that when comparing the dimension of dshape, the Pooling function does not consider the change that the dimension can be -1. I guess it is this problem that leads to MXNet to crash.
Environment
Such issue is tested in both Centos 8, MacOS Big Sur, and Windows 10

Everything seems to work fine on versions >=1.8 - can you provide more information (how you're installing mxnet (pip/form source) and exact library versions (pip freeze)?

Hi bgawrych, thanks for your reply. Yes, I made a mistake previously, mxnet works fine on version 1.8.0.
However, this bug still occurs on version 1.5.1, and even happens on 1.7.0 (using pip instal).
May I know what's the fix or pull request that solves this problem?

@maybeLee seems like this one fixes the issue: https://github.com/apache/incubator-mxnet/commit/6b568fd2c10e8cc14c21d6150ba71933a2af2a41

Thanks for your help, seems that this is the fix related to this bug.


brick and ball added

PR Description 📜
this is simple brick and ball game which   i had coded few months back  and i am very happy to contribute it
back to point :
this is brick and ball game code
i have added game screeenshot,game folder, added in json file and changed README.md file
Fixes #3619

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Yashgabani845 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@Yashgabani845 resolve merge conflicts !

@kunjgit  resolved

@kunjgit  checkout my PR i want to work on another isue of your repo that iassigned to me

Thank you  @Yashgabani845 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

@kunjgit  please add gssoc label otherwise my PR will not  be counted

/assign


Bump serde and serde_json to 1.0

This is blocked on nox/serde_urlencoded#18.

I rebased this myself now that serde_urlencoded has been updated, merged into master with https://github.com/seanmonstar/reqwest/commit/14dbc79da3d8c6027f7196b61534951b51412adb and https://github.com/seanmonstar/reqwest/commit/5947578a28b5d38ce2061c0d8d6c624953ecc1ce
Thanks @TedDriggs!

Great thanks.

OMG, thanx )


Translation wrapper changes suggested by the team

Separate out detected and user-specified source languages
Fix a typo in snippets
Improve documentation for target language in ListLanguages
Give an example with a target language in snippets

@chrisdunelm to review first, then will ask for team review before merging.

Speculatively merging - we'll only be releasing an alpha, so there's still a chance to revise further.


[5.6] Fix a miscompile in existential specialization during SILCombine.

The transformation propagations stack-initialized values. It checks
for any writes to the stack before doing so. OpenExistial instructions
now have a mutable property, so that also needs to be checked.
Add OpenExistentialAddrInst::isRead(SILInstruction).
Replace an isa<OpenExistentialAddrInst>(user) check with
`OpenExistentialAddrInst::isRead(user).
Fixes rdar://88664423 (SR-15791: Miscompile under -O with mutating protocol method)
(cherry picked from commit 07e6cfc8ae9afb8692203133d6d9f34306a5668e)

@swift-ci test

-- CCC ---
Explanation: An existential value is propagated into a specialized type. That value is mutated, but the specialized value sees the original non-mutated value.
Fix: The code already conservatively checked for mutation. However, it did not explicitly check for the "mutable" flag on opened_existential_addr.
Scope: Miscompiles code that passes an existential into a generic function.
Issue: rdar://88664423 (SR-15791: Miscompile under -O with mutating protocol method)
Origin: 2017 time frame. The open_existential_addr instruction was given a "mutable" around the same time that the existential specializer was being worked on. Any changes to the compiler could have exposed the issue recently.
Risk: No tangible risk to this fix. It avoids performing the optimization in cases that were likely to be a miscompile before.
Testing: A new unit test was developed.


Improve Behavior Intervals

LightBehavior calculates the interval that should be used in a rather naive manner. It fires fairly often in very shallow ramps with little delta, despite not having a whole lot to do during most of the ramp.
By improving this, we can reduce power consumption a little in many real world cases. It just isn't a huge priority when without this improvement, it uses <1% CPU on a Pi 3B.

In general, it seems that updates between 1s and 0.01s are the most useful range.
1s minimizes the delay that happens when starting a new schedule in the middle of a behavior, so the user doesn't feel like there is a problem. It papers over issues without putting too much load on the CPU waking up too frequently.
At 0.01s, it handles fast ramps very smoothly. So you can have very short 4-10s ramps that play out without seeing stepping.
I'd like to eventually remove the need to have a behavior queued for flat parts of the ramp, but there's weird interactions involved once you start talking about. It's just too useful to have something there that represents the current light behavior. Currently, it helps in startup cases. In the future, it helps in override cases. A "Lunar Behavior" or "Thunderstorm Behavior" can lookup the LightBehavior to get information about what the brightness is expected to be at a given point in time.
Perhaps to address that, flat ramps can use an interval the length of the ramp, which means it can be looked up by a higher priority behavior in the queue. To handle the startup case, perhaps we always call "DoBehavior" on a behavior that is added to an empty queue?

If we were to start calling behaviors when they are added to the queue, and also happen to be first in the queue after insert, then it is possible (although rare) that they could be called before their StartDate() as written now.
Should either make sure we guard against calling the behavior prior to StartDate() in these cases (preferable), or guard the behaviors themselves against producing nonsense in these cases.

Closing this, as this sort of scaling got implemented almost by accident. There are further improvements, but let's not re-use an issue too much.


bugfix/weekdays-with-string

:bug: Adding an OR condition when moment.weekdays receive a string value as parameters

Coverage increased (+0.09%) to 88.526% when pulling a88f83c621ff34b99946fa85a140b621c4813a64 on guimap:weekdays-with-string into e96809208c9d1b1bbe22d605e76985770024de42 on moment:develop.

I Will open another PR

#5934


Resolve an issue where \r causes parsing errors

Description of Changes
Sometimes \r will get mistaken for a carriage return, causing parsing errors
Raw syslog:
<125>1 2021-02-05T16:31:54.000Z api.localnet WebServer - - [system@444 key="value\\\rap"]

Error
{"level":"error","timestamp":"2021-02-12T17:46:51.364-0500","message":"Failed to process entry","operator_id":"$.5424 tcp.syslog_parser","operator_type":"syslog_parser","error":"expecting chars `]`, `\"`, and `\\` to be escaped within param value [col 91]","action":"send","entry":{"timestamp":"2021-02-12T17:46:51.364796-05:00","severity":0,"labels":{"log_type":"syslog","plugin_id":"5424 tcp"},"record":"<125>1 2021-02-05T16:31:54.000Z api.localnet WebServer - - [system@444 key=\"value\\\\\\rap\"]"}}

Output record
{
  "timestamp": "2021-02-12T17:46:51.364796-05:00",
  "severity": 0,
  "labels": {
    "log_type": "syslog",
    "plugin_id": "5424 tcp"
  },
  "record": "<125>1 2021-02-05T16:31:54.000Z api.localnet WebServer - - [system@444 key=\"value\\\\\\rap\"]"
}

After making this change, the same syslog message returns no errors and the record looks like this:
{
  "timestamp": "2021-02-05T16:31:54Z",
  "severity": 0,
  "labels": {
    "log_type": "syslog",
    "plugin_id": "5424 tcp"
  },
  "record": {
    "appname": "WebServer",
    "facility": 15,
    "hostname": "api.localnet",
    "priority": 125,
    "severity": 5,
    "structured_data": {
      "system@444": {
        "key": "value\\rap"
      }
    },
    "version": 1
  }
}

Please check that the PR fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[x] Add a changelog entry (for non-trivial bug fixes / features)
[ ] CI passes

Log Files
Logs / Second
CPU Avg (%)
CPU Avg Δ (%)
Memory Avg (MB)
Memory Avg Δ (MB)

1
1000
1.8276129
+0.051708817
124.60452
+4.0633087

1
5000
6.4484205
+0.08658457
132.16367
+1.2232056

1
10000
12.9657955
+0.3964777
140.02235
-0.17376709

1
50000
58.157772
+0.8614578
174.23895
-3.5130615

1
100000
115.53361
+2.8932495
231.96214
+9.969147

10
100
2.2932012
+1.835823e-05
128.85156
+2.3945312

10
500
6.6208663
-0.20694113
135.1122
-2.1861572

10
1000
13.379669
-0.034495354
140.6735
-1.0536041

10
5000
63.15695
-0.278656
179.13268
-3.0319214

10
10000
123.89799
+4.3482056
226.42296
+4.9590607

Moving forward with a plugin update until a cleaner solution can be implemented. Turns out the problem being solved is a domain login DOMAIN\user and the username starts with an r.


Code snippets on authorization page aren't highlighted

See https://googlecloudplatform.github.io/gcloud-node/#/authorization -- the snippets with var config = ... don't seem to be using syntax highlighting.

I'll take a look at this again.
Related: https://github.com/GoogleCloudPlatform/gcloud-node/pull/507#issuecomment-96672814


[Document] Design Documentation Improvement with Antora

This PR includes the following features:

Reorg design document structure to support Antora-based document website.
Add Api Completeness Analysis with Neutron v2.0.
Outline design for dataplane manager, network health monitor and node metadata manager.
Add vpc and vm create workflows

LGTM


Set up an S3 inventory report for the storage service

See:

https://docs.aws.amazon.com/AmazonS3/latest/dev/storage-inventory.html
https://www.terraform.io/docs/providers/aws/r/s3_bucket_inventory.html
https://wellcome.slack.com/archives/CN56BRQ5B/p1590499831057100

If you ask CloudWatch Metrics, we have slightly more objects in hot storage than cold, which seems odd.
Would cost a few dollars a month, take maybe ten minutes to set up.

I don't think this is a big change, but I’m not going to pick it up until I’ve finished the bag tracker stuff; I have one too many plates spinning in the air right now.


Creating documentation for [Get|Set]-CsTeamsSipDevicesConfiguration commandlets

Creating documentation for [Get|Set]-CsTeamsSipDevicesConfiguration commandlets

@JohanFreelancer9

Copy review - @anmandav, kindly commit the minor suggested changes. Many thanks.

Done. Committed the suggested modifications.

@yogkumgit could you merge this Pull Request?


Current and Cumulative data doesn't make sense?

Can the Current and Cumulative data for inICU and onVentilator be reviewed/revised?  Cumulative numbers in particular seem suspect ... shouldn't cumulative numbers always either increase or stay the same?

We are having the same issue with the data.  ICU and Vent numbers appear to be swapped between the "cumulative" and "currently" columns

^ bump on this!

Not sure what ^ bump means, but there is some computational error here.  Or I don't understand the column headings.
InIcuCurrently goes up and down, as doe OnVentilatorCurrently.  Makes sense.  But
IncIcuCurrently > InIcuCumulative
OnVentilatorCurrently > OnVentilatorCumulative

Hi folks, this data can indeed be counterintuitive but please see the data documentation in https://covidtracking.com/api


scalogram periods off by factor of sampf

Example code analyzes a simple sine wave with a period of 17.0.  scalogram plots power around 170.0
import numpy as np
import matplotlib as mpl
from matplotlib import pyplot as plt
import cwt

# make signal
P = 17.0 # period, s
D = 100   # duration, s
N = 1000 # number of samples
x = np.linspace(0, D, N) # time, s
data = np.sin(2*np.pi/P*x)
label = 'P=%0.1f sine wave' % P

# plot signal
plt.plot(x, data)
plt.title(label)
plt.xlabel('Time [s]')
plt.ylabel('signal')

# continuous wavelet transform
scales = np.arange(200)
mother = cwt.Morlet(len_signal = len(data), scales = scales)
wavelet = cwt.cwt(data, mother)

# plot with builtin function
wavelet.scalogram()

# plot myself
y = mother.scales / mother.fc / 10.0 # bugfix: divide by 10?
fig = plt.figure()
ax = fig.gca()
contf = ax.contourf(x, y, np.abs(wavelet.coefs)**2)
fig.colorbar(contf, ax=ax, orientation='vertical', format='%2.1f')
ax.set_xlim((x[0], x[-1]))
ax.set_title('scalogram')
ax.set_xlabel('time')
ax.set_ylabel('period')
plt.show()

Actually, "10" is just a result of my sampling frequency.  The correction should be:
y = mother.scales / mother.fc / mother.sampf # bugfix

Greetings @rickyegeland!
First off, thank you for the report. It's good to see someone is looking at the code!
Second, when you created the motherwavelet mother, did you set the kwarg sampf? If sampf, the sampling frequency, isn't set, a default of 1 is used.
I agree the API is a bit fragile in terms of forgetting to set values like sampf.  It's possible that the mother wavelet could be passed into cwt as a class, and initialized within cwt. That would cut down on one step (creating a mother wavelet), while allowing the signal length to be computed automatically. The scales could be computed automatically based on the COI. However, if the signal is long (for example, the sonic anemometer data I worked with had 36000 - 72000 points), it's nice to be able to specify the particular range of scales that you wish to analyze.
sampf is a bit harder to handle automatically without making some assumptions. If the input time, x, was a numpy array or list of datetime objects, then we could do something smarter there. Of course, maybe I'm thinking about it too much, and we could simply compute sampf from x if it was passed into cwt as well as the mother wavelet.
maybe an api like:
cwt(x, data, mother)

Initially (when I submitted the bug) I wasn't setting sampf.  Now I am, however in the current version sampf is not used to properly set the periods in scalogram().
I think the API can be improved, either in the way you suggest, or by implementing cwt() as a method of MotherWavelet.  Also, we could allow for the user to set the scales explicitly or allow them to be calculated.
# Ex. 1: explicitly set scales
mother = Morlet()
result = mother.cwt(t, data, scales=myscales)

# Ex. 2: automagically calculate scales
mother = Morlet()
result = mother.cwt(t, data)
# or
result = Morlet().cwt(t, data)
# or
Morlet().cwt(t, data).scalogram()

In any case, parameters like len_signal and sampf should never be necessary as they can be extracted from a time axis, which I expect should always be available to a user.  In my current tests I'm setting them up like this.
# t, time axis and data, the signal prepared above...
N = t.size
T = t[-1] - t[0]
dt = 1.*T/N
sampf = 1./dt
fc = 0.849 # taken from code...
Nscales = int(T * fc * sampf) # compute Nscales to scan from P=0 to P=T
mother = cwt.Morlet(len_signal=N, scales=np.arange(Nscales), sampf=sampf)
wavelet = cwt.cwt(data, mother)

Although as you mention, if a cone of influence is used it doesn't make sense to calculate scales all the way up to T.  However, that should be optional too, because if the signal is known to be cyclical there is no need for a cone of influence.
I'm willing to work on these changes if you agree.  I would also make a simple unit test and document the usage of this module.  I need python wavelet analysis for my research, and although there are alternatives out there, this code is the cleanest implementation I've seen.  Maybe if it is improved and tested enough it can make its way into scipy, where I previously wasted many hours trying to figure out how their cwt and Morlet are supposed to work.  (They don't.  At least not together.)
By the way, I am also working at NCAR (HAO).

Hi Ricky,
I'd love to work together on this. You are at HAO (Center Green?). Maybe we
could get together sometime soon and talk in person about the API.
Thanks!
Sean
On Fri, Jan 30, 2015 at 2:33 PM, Ricky Egeland<EMAIL_ADDRESS>wrote:

Initially (when I submitted the bug) I wasn't setting sampf. Now I am,
however in the current version sampf is not used to properly set the
periods in scalogram().
I think the API can be improved, either in the way you suggest, or by
implementing cwt() as a method of MotherWavelet. Also, we could allow for
the user to set the scales explicitly or allow them to be calculated.
Ex. 1: explicitly set scales
mother = Morlet()
result = mother.cwt(t, data, scales=myscales)
Ex. 2: automagically calculate scales
mother = Morlet()
result = mother.cwt(t, data)# or
result = Morlet().cwt(t, data)# or
Morlet().cwt(t, data).scalogram()
In any case, parameters like len_signal and sampf should never be
necessary as they can be extracted from a time axis, which I expect should
always be available to a user. In my current tests I'm setting them up like
this.
t, time axis and data, the signal prepared above...
N = t.size
T = t[-1] - t[0]
dt = 1.*T/N
sampf = 1./dt
fc = 0.849 # taken from code...
Nscales = int(T * fc * sampf) # compute Nscales to scan from P=0 to P=T
mother = cwt.Morlet(len_signal=N, scales=np.arange(Nscales), sampf=sampf)
wavelet = cwt.cwt(data, mother)
Although as you mention, if a cone of influence is used it doesn't make
sense to calculate scales all the way up to T. However, that should be
optional too, because if the signal is known to be cyclical there is no
need for a cone of influence.
I'm willing to work on these changes if you agree. I would also make a
simple unit test and document the usage of this module. I need python
wavelet analysis for my research, and although there are alternatives out
there, this code is the cleanest implementation I've seen. Maybe if it is
improved and tested enough it can make its way into scipy, where I
previously wasted many hours trying to figure out how their cwt and Morlet
are supposed to work. (They don't. At least not together.)
By the way, I am also working at NCAR (HAO).
—
Reply to this email directly or view it on GitHub
https://github.com/Unidata/pyCWT/issues/1#issuecomment-72274003.


Enrich cs cr status

display all installed bedrock operators' version & status in cs cr

@horis233 @Daniel-Fan this pr is ready for final review, thanks!


[BUG] Cannot upload file larger then memory.

Describe the bug
Command Name
az storage fs file upload
Errors:
Traceback (most recent call last):
lib/python3.8/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
azure/cli/core/commands/__init__.py, ln 654, in execute
    raise ex
azure/cli/core/commands/__init__.py, ln 717, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
azure/cli/core/commands/__init__.py, ln 709, in _run_job
    return cmd_copy.exception_handler(ex)
azure/cli/command_modules/storage/__init__.py, ln 342, in new_handler
    raise ex
azure/cli/core/commands/__init__.py, ln 688, in _run_job
    result = cmd_copy(params)
azure/cli/core/commands/__init__.py, ln 325, in __call__
    return self.handler(*args, **kwargs)
site-packages/azure/cli/core/__init__.py, ln 784, in default_command_handler
    return op(**command_args)
cli/command_modules/storage/operations/fs_file.py, ln 94, in upload_file
    data = stream.read(count)
MemoryError

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

get a large file
az storage fs file upload -s {} -p {} -f {} --account-name {} --auth-mode login
failure

Expected Behavior

file to upload

Environment Summary
Linux-5.4.0-1031-azure-x86_64-with-glibc2.29
Python 3.8.5
Installer:

azure-cli 2.15.1

Additional Context
This is an azure storage optimize instance has 64gb of ram (mostly free). I am sad i can't upload file this way.

Storage


FileNotFoundError: [Errno 2] No such file or directory: './balance_split/random-50_crop-250/val_val_250m/coordShift.json'

FileNotFoundError: [Errno 2] No such file or directory: './balance_split/random-50_crop-250/val_val_250m/coordShift.json'
Can you release the coordShift.json in this code?

I fixed the pre-processing script.

hello,
I am getting this issue I think due to CoordSHif.json file issue. Can you please guide?

because it looks like this,

thanks


Update to version 0.9.0

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Add Upstash Embeddings Support

This PR adds Upstash Embeddings support, which allows users to send their data to Upstash Vector Database without creating the embeddings. Upstash Embeddings will handle the embedding operation for them.
More context on Upstash announcement

Sorry for the delay!

Hey @jacoblee93 any chance for review again?

Hey @jacoblee93 any chance for review again?

Hey, sorry for the delay!
I left a note about using FakeEmbeddings and documenting it rather than the string. While I do agree that a string is more descriptive, we must adhere to the base class interface for now otherwise we'll have problems with types etc. We have some refactoring of vectorstores planned in the future and we can apply a better fix then.

Hey @jacoblee93 any chance for review again?

Hey, sorry for the delay!
I left a note about using FakeEmbeddings and documenting it rather than the string. While I do agree that a string is more descriptive, we must adhere to the base class interface for now otherwise we'll have problems with types etc. We have some refactoring of vectorstores planned in the future and we can apply a better fix then.

Hey, it's ready as you asked.

Thank you! I renamed a couple of things but re-ran the integration tests and they passed. Will get this released today.


Improved fish support

Hi, I was able to improve the fish shell support, but wanted to check with you if that is an approach you would support before creating a PR:
Basically I took over your "init" stub and named it shell. It shall output an "eval"-able result:
elif [[ "shell" == "${1}" ]]; then
  # TODO: This will be multi-shell support like rbenv, e.g. fish
  # Supports "assume-role shell [account] [role]" | <translate syntax> | source in your favourite shell
  set -eo pipefail
  OUTPUT_TO_EVAL="true"
  shift
  assume-role "$@";

BUT to enable the interactive mfa prompt (which I really want to have with fish) I had to redirect the prompt to stderr:
    if [ -z "$mfa_token_input" ]; then            # removed: && [ -z "$OUTPUT_TO_EVAL" ]
      echo -n "MFA Token: " >> /dev/stderr  # redirected user prompts to stderr - should this be used everywhere to enable interactive prompts when caledl via eval or source pipe?
      read -r -s mfa_token

You can now have a function like this in fish:
function role2
   assume-role shell $argv | sed 's/^export/set -x --global/' | sed 's/=/ /' | source
end

Is that an approach worth to follow for you? Any suggestions for improvements?
Jan

I like that. I know some people that use fish that would love more support for it.
I think that you are right and that echo -n "MFA Token: " >> /dev/stderr should probably be there. I am not sure about the consequences for people using OUTPUT_TO_EVAL as it might start prompting them.
What is the difference between calling with the shell first argument and calling it normally, the only difference I see is the shift.
I also love the idea creating a function that wraps the piping to source, might steal that trick.
Please submit a PR we can work on it from there.


[Request] Structure bounding box

Hi!
This is a very handy utility. Would it be possible to add a feature to show the bounding box of the structure? This would be handy for building mob farms that rely on specific structures, and for helping differentiate structures that have intersected.
I doubt this is as easy as it sounds, but it never hurts to ask!

Been thinking about it and WITS is more about being a serversided mod that even vanilla clients can use. Showing structure bounds on client side would require wits to no longer be serversided and would need to be locked behind a config for server owners that think showing bounds could be cheating.
I think this feature is better done by another mod as it is a bit out of scope of what WITS is for. Maybe Structure Gel API already has this feature?


Swift Version

Thanks for great library. But need swift version

Selene works great with Swift projects. No need to rewrite it in Swift.


Feat: Add typescript declarations and Prettier, resolves #40

Describe your changes

Add dev dependencies: Prettier, prettier-plugin-astro for *.astro file support, and .prettierrc for config.
Format components with Prettier configs, removed extraneous empty path tags <> from formatting.
Add Typescript declaration and component Props types.

Issue ticket number and link
Issue #40
Checklist before requesting a review

[X] I have performed a self-review of my code

Specific areas needing review

Notification.astro: I've based the role prop off MDN accessibility requirements but I'm not an accessibility expert so I would appreciate these being double checked for accuracy. Left in a comment that region could possibly be added as a role but there is no ID currently on the component. If I'm correct and this is easy to add this way, I figured it might be better suited to a separate pull request.

Pagination.astro: Astro.props default values such as firstPage = '#', appear to be purely descriptive. It might be worth removing these in favor of the type declarations so that users do not need to explicitly pass falsy values.
Pagination.astro SVG formatting was also heavily impacted by Prettier. .prettierrc. config changes may or may not be desired.

index.d.ts: It would be good to have a second set of eyes on @param descriptions to check for accuracy and legibility.

@david-abell wonderful PR!! Great job on being so descriptive about your work and the work itself (me not being a TypeScript expert in this one) looks great and understandable 🙂 The index.d.ts looks very good, only made some small spelling changes. It's a nice little documentation for how to use the components! I'll merge this one in soon 😄
Thanks again! 🚀

@markteekman Oh god did really submit documentation for a Dorkmode component :anguished: :rofl: . I'm feeling the sudden urge to actually build that now

@david-abell hahaha yes I would love to see a DorkMode 🤣🤣 Please make it 😋


feat: support removing fields from response

support removing fields from FaissLMDBSearcher response

Hey @Ahmad-Nouh
Would you have time to polish this for merging?

Hey @Ahmad-Nouh
Would you have time to polish this for merging?

Hey @cristianmtr
I couldn't find a way to implement this feature because it seems that the final response fields are being added from somewhere else outside the FaissLMDBSearcher.
I also tried to create an executor on top of the FaissLMDBSearcher to slim the response.
Although the executor output was free of extra fields, the extra fields were present in the last output,
It seems that they are being added from somewhere else.

Hey @Ahmad-Nouh
Would you have time to polish this for merging?

Hey @cristianmtr I couldn't find a way to implement this feature because it seems that the final response fields are being added from somewhere else outside the FaissLMDBSearcher. I also tried to create an executor on top of the FaissLMDBSearcher to slim the response. Although the executor output was free of extra fields, the extra fields were present in the last output, It seems that they are being added from somewhere else.

I see. Do you have some code where this happens? I would expect that removing those fields would work.


Add basic club info

[ ] location
[ ] description

I plan to make a separate about page, but I have added location, basic description, and pictures to the main page.


docs: review for #14

Duplicated PR with #15
Since there need to make changes in unchanged part, I made some changes and open PR.
If @sm-stack thinks it's ok to push to https://github.com/kroma-network/kroma-specs/pull/14, please merge this.
This PR do not require 2 approvals from code owner. Author can merge this.

I take over the #14, so applied the changes in this PR to #14 as a separate commit. So I close this PR.


Mouse without Borders Chinese translation

Microsoft PowerToys version
0.75.0
Utility with translation issue
Mouse Without Borders
🌐 Language affected
Chinese Simplified
❌ Actual phrase(s)
无边框鼠标
✔ Expected phrase(s)
无界鼠标
ℹ Why is the current translation wrong
the current translation is "无边框鼠标", while "无边框" is often used to literally describe some boxed or closed area that has no outlined border, which maybe a confusion to describe the mouse pointer itself is borderless.
To describe the mouse can fly across different devices, the "无界鼠标" suggest freedom to move the mouse across computer boundaries without any restrictions, it may be a better translation for "mouse without borders"

/dup https://github.com/microsoft/PowerToys/issues/27995


I can't understand how we can transform all handlebars files to jsx templates

Can you please make this theme closer to what the default theme should look like?

@Gerrit0 please give more documentation for converting a custom theme to functional JSX template theme

TypeDoc supports partial patching of theme partials - you generally aren't supposed to replace every template since then your theme won't benefit from any fixes introduced in TypeDoc.
https://github.com/TypeStrong/typedoc/tree/master/src/lib/output/themes/default/partials contains (for the most part, somewhat refactored) the JSX equivalent as the original handlebars.

I thought that once we have a custom theme(which is an npm package) that has a load method with typedoc-theme as a keyword we can set it through the --theme option by passing the name of the registered theme e.g customTheme:
typedoc --theme customTheme
It turned out that this way was not including the styles I have defined in the assets  folder. I made it work by passing the path to the dist folder of my theme:
typedoc --theme node_modules/customTheme/dist
Is this the right way for passing a reference to a custom theme?
Because I read in the API doc that you can only pass the name of the theme that gets registered by the library itself. But that is not how the styles get integrated into the build

For custom themes, I'd recommend adding a listener to app.renderer.on(RendererEvent.END, which will receive a RendererEvent, you can then copy data to event.outputDirectory after checking that the app.renderer.theme instanceof YourTheme


Map Presets created, added to menu

#564 map selection includes an option which generates a random map
Now hiding the map options such as size, seed and clutter under an advanced tab
added map and gamemode to lobby browser

Skulle vilja ha kvar kommentarerna då det är WIP


Gateway timeout

ability to pass through deadline value in "x-google-backend" (closes #224 )

Codecov Report

Merging #225 (3d7f8a8) into main (84ebba2) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #225      +/-   ##
==========================================
+ Coverage   87.64%   87.74%   +0.10%
==========================================
  Files          36       36
  Lines        2897     2921      +24
==========================================
+ Hits         2539     2563      +24
  Misses        358      358

Impacted Files
Coverage Δ

goblet/tests/test_openapispec.py
94.35% <ø> (ø)

goblet/resources/routes.py
86.01% <100.00%> (+0.47%)
:arrow_up:

goblet/tests/test_routes.py
98.43% <100.00%> (+0.11%)
:arrow_up:

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


WIP: [ci skip] Build verify container images

CIRRUS: TEST IMAGES
Depends on: #3754 #3901

Closing this for now, possibly temporarily, to work on more important items.  May re-open in the future as priorities dictate.


Fix zero time interval in example

This was intended as a corner case. Does it break something?

The visualisation is suboptimal for our toy example. It is difficult to see an utterance at 0. Was the corner case zero interval at zero or just zero interval?

The corner case was just zero duration for any segment. You can also add an empty segment with zero duration

Ok, I moved the zero duration segment now to 1-1. There it is better recognizable. I also added an empty segment.


Dual SIM: Respect default sim

Hi!
So i have been using the program for a few weeks, and found an issue.
The Android Messages app on the phone is respecting the default sim being set to 1, and rebooting the phone or doing anything keeps the setting to 1.
On the desktop program however, i need to change from sim 2 to sim 1 each time i start the program.
The program should either have an option to remember the last selected sim, read the default from the phone, or let the user set the default.

Hi @chrisknepper
You are right, seems to be a google issue. The desktop and web both have the same issue.
Here is how it looks either way:

Looks the same on the desktop app. Maybe it would be possible for the desktop app to have a setting in options to set the default sim, and when the page loads, change it using javascript for the user automatically?


allow negative offset

See issue #8458 for comments

It seems that your branch is out of sync with master and has conflicts.
Additionally, I'd suggest you start of the following branch/commit and add the necesarry feature flag that was missing into that pull request: https://github.com/prometheus/prometheus/pull/6283
Feature flags are managed in main.go (look for remove-write-receiver there.)
Thanks!

cc @dgl for awareness

Ok, rebasing here...

@roidelapluie Please let me know how this looks for a negative offset.

Can we moved the unrelated docs change in a separate pull request? Thanks!

Yes, we can.
All the best!

promql/parser/generated_parser.y.go is conflicting.
Can you please add a feature flag?

Please note that the file generated_parser.y.go is the output of goyacc, so it isn't a breaking change.  The source file is generated_parser.y.
I can remove generated_parser.y.go if you would like as it will be compiled again by the deployment process.  I provide the compiled file only as a courtesy as it was pre-compiled when I cloned the repo.  :)

I can remove generated_parser.y.go if you would like as it will be compiled again by the deployment process.

We actually need the file for the builds and tests to pass. Not sure why the conflict, can you try running goyacc again and push?

Can you please rebase your pull request on top of master and regenerate it?
Le mar. 16 févr. 2021 à 07:37, Ganesh Vernekar<EMAIL_ADDRESS>a
écrit :

I can remove generated_parser.y.go if you would like as it will be
compiled again by the deployment process.
We actually need the file for the builds and tests to pass. Not sure why
the conflict, can you try running goyacc again and push?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/8487#issuecomment-779620816,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACHHJX53OUOSZ7NB3OAERLS7IHDJANCNFSM4XTRVWAA
.

Can you please rebase (docs/disabled_features.md) is conflicting?
Also, can you please update promql/parser/printer_test.go with a test ? It seems not to work out of the box.

Can you please rebase (docs/disabled_features.md) is conflicting?
Also, can you please update promql/parser/printer_test.go with a test ? It seems not to work out of the box.

yup, done

I will look at this later, can you please fix the DCO?
cc @codesome do you want to review this as well?

Thanks!


[DO NOT MERGE] Test new agent pools

DO NOT MERGE
This is a test of the new Azure Scale Set Agent Pools for dynamically creating and destroying VM's.
DO NOT MERGE
Signed-off-by: Brett Logan<EMAIL_ADDRESS>

/azp run

/azp run


Artifact search form

With all those artifacts we have and their big names, it's hard to find the one you are looking for in the github release page.
Our naming now is more structured and is based solely on the flavors.json and the naming.sh script. For example with a script like the following, all artifact names can be generated:
~/workspace/kairos/kairos (master)*$ cat all-artifacts.sh
#!/bin/bash
# https://stackoverflow.com/a/58009121

cat .github/flavors.json | jq -c '.[]' |
while IFS=$"\n" read -r c; do
    FAMILY=$(echo "$c" | jq -r '.family') && export FAMILY
    FLAVOR=$(echo "$c" | jq -r '.flavor') && export FLAVOR
    FLAVOR_RELEASE=$(echo "$c" | jq -r '.flavorRelease') && export FLAVOR_RELEASE
    VARIANT=$(echo "$c" | jq -r '.variant') && export VARIANT
    TARGETARCH=$(echo "$c" | jq -r '.arch') && export TARGETARCH
    MODEL=$(echo "$c" | jq -r '.model') && export MODEL
    REGISTRY_AND_ORG=quay.io/kairos && export REGISTRY_AND_ORG
    KAIROS_VERSION=$(git describe --always --tags --dirty) && export KAIROS_VERSION
    ./naming.sh container_artifact_name
done | sort

We could generate a static page with every release that allows users to search for an artifact using drop down menus for each "input" field. E.g.
select box -> family
select box -> flavor
select box -> type (iso, img etc)
...
and the form would output all mathing artifacts in a text area box of a ul or something like that.

something that would make our life easier and has nice filters is datatables which can provide filters by field even so if we export the data correctly to a file we could have a nice way of filtering by version, flavor, variant, etc..
https://datatables.net/examples/api/multi_filter_select.html

It would be great if it was integrated in out website, no need to go to gh for it


Pets disease predictor using CNN

I want to submit a project on Pets image disease classification using CNN. Please assign me to this.

Assigned @akhilesh1709

Generated pull request


Pytest runs slowly in ci

Pytest completes quickly locally but takes a long time to run during the commit build

fixed by limiting db rows, so that it does not try to build a 1000 row db


Fail to createChatThreadId

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [x ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 10 20H2

Versions

Mention any other details that might be useful

How to generate jwtPrivateKey in communication-services-contoso-med-app-main\contoso-node-api\config.json ?
"mongodbConnection":"<COSMOS_DB_OR_MONGODB_CONNECTION_STRING>",
"dbName": "<DATABASE_NAME>",
"connectionString": "<AZURE_COMMUNICATION_SERVICES_CONNECTION_STRING>",
"jwtPrivateKey": "<JWT_PRIVATE_KEY_FOR_AUTHENTICATION>",
"endpoint": "<AZURE_COMMUNICATION_SERVICES_ENDPOINT>",
"smsLogicAppEndpoint": "<AZURE_COMMUNICATION_SERVICES_SMS_LOGIC_APP_HTTP_TRIGGER_ENDPOINT>",
"qnaMakerEndpoint": "<QNA_MAKER_ENDPOINT_URL>",
"qnaMakerEndpointKey": "<QNA_MAKER_ENDPOINT_KEY>"

Fail to createChatThreadId  => token refresh of refresh args undefined

Which @azure/communication SDK version are you using ?

Thanks! We'll be in touch soon.

Hi ddematheu2,
I downloaded the latest contoso-med-app example from
kushan-developer/communication-services-contoso-med-app but it still can't
create chat thread. It showed error messages below. Could you help me ?
Thanks.
[image: image.png]
[image: image.png]
Best Regards,
Gigi
On Wed, Mar 24, 2021 at 3:14 AM ddematheu2 @.***> wrote:

Closed #5
https://github.com/Azure-Samples/communication-services-contoso-med-app/issues/5
via #6
https://github.com/Azure-Samples/communication-services-contoso-med-app/pull/6
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Azure-Samples/communication-services-contoso-med-app/issues/5#event-4498070843,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRTYQO5V6MVRUB663VHRITTFDSAPANCNFSM4ZOEIPGA
.


Include supported?

Is it possible to include one CDDL file into another CDDL file?

No, but you can pass multiple CDDL files to zcbor in a single call.

Solved by just using c preprocessor. Always works :D


Obtain the hash of a PDF document for offline signing

The signing process of a document is done using a very slow process therefore i would like to:

Calculate the hash of a document
Take my time to obtain the signed hash of the document (it may take up to 1 day)
Add the signed has to the PDF (I can achieve this by implementing an HSM interface tin with an externally populated hash)

An example is pdf-sign-cms-hash.py and pdf-sign-cms-hash-sign.py.
pdf-sign-cms-hash.py creates a .json file with the data to be signed, but ... the correctness of the signature depends on the signature method, you may need to encode the hash as https: // datatracker. ietf.org/doc/html/rfc8017# section-9.2

It works and it answers my question. Great. Thank you for this.
The API sign call is a bit counterintuitive when you first read it and doesn't immediately signals you get the has as result. Running it makes it clear.
    cls = pdf.cms.SignedData()
    datas = cls.sign(datau, dct, None, None, [], "sha512", clshsm, mode="sign", timestampurl=None)

I've run multiple times [examples/pdf-sign-cms-hash.py] and for the second run, that attempts to sign the PDF file given the content of the JSON object i get an assert error related to the content of the PDF file that changed which is not true.
If I delete the JSON file and pin the signed_time to a fixed datetime the hash and content seems to be changing each run. Can you please elaborate a bit why this is the case?
Traceback (most recent call last):
  File "create-hash.py", line 111, in <module>
    main()
  File "create-hash.py", line 99, in main
    datas = cls.sign(datau, dct, None, None, [], "sha256", clshsm, mode="sign", timestampurl=None)
  File "/usr/local/lib/python3.9/site-packages/endesive/pdf/cms.py", line 734, in sign
    contents = signer.sign(
  File "/usr/local/lib/python3.9/site-packages/endesive/signer.py", line 176, in sign
    signed_value_signature = hsm.sign(keyid, tosign, hashalgo)
  File "create-hash.py", line 23, in sign
    assert self.tosign == data

The first run of pdf-sign-cms-hash.py creates a json file, the second run signs the file and saves the result in a new pdf file if there is a signed hash (signed_bytes) in the opened json file, otherwise it creates an entry (tosign) and saves the json file. If the pdf file has not changed, I always have the same hash, regardless of the number of pdf-sign-cms-hash.py runs. Manually, you can only change signed_bytes in the json file, changes to any of the parameters generate a different hash (tosign).
The input pdf file is never modified by endesive.
my terminal is just empty:
$ ./pdf-sign-cms-hash.py
$ ./pdf-sign-cms-hash.py
$ ./pdf-sign-cms-hash-sign.py
$ ./pdf-sign-cms-hash-sign.py
$ ./pdf-sign-cms-hash.py
$ ./pdf-sign-cms-hash.py

json ['when'] is part of the signature hash, each time a json file is created a when value is also created, same is for certificate if it changes

Correct. This is why i set the 'when' and the certificate to constants and the hash keeps changing.

I tried again to replicate the problem from scratch. Here are the steps to reproduce:

Check out the latest version from this repo.
Updated the certificate and PDF file in 'examples/ 'pdf-sign-cms-hash.py' because I couldn't find any of the test certificate and 'pdf.pdf' file.
Updated the line https://github.com/m32/endesive/blob/bf58ed13d63735effb59c7347efa0276ea9ba3e0/examples/pdf-sign-cms-hash.py#L99 to read
datas = cls.sign(datau, dct, None, None, [], "sha256", clshsm, mode="sign", timestampurl=None)
Run 'examples/ 'pdf-sign-cms-hash.py' --> OK
Run again 'examples/ 'pdf-sign-cms-hash.py'
then I get the error:

Traceback (most recent call last):
  File "/Users/user/Downloads/endesive/examples/pdf-sign-cms-hash.py", line 111, in <module>
    main()
  File "/Users/user/Downloads/endesive/examples/pdf-sign-cms-hash.py", line 99, in main
    datas = cls.sign(datau, dct, None, None, [], "sha256", clshsm, mode="sign", timestampurl=None)
  File "/usr/local/lib/python3.9/site-packages/endesive/pdf/cms.py", line 734, in sign
    contents = signer.sign(
  File "/usr/local/lib/python3.9/site-packages/endesive/signer.py", line 176, in sign
    signed_value_signature = hsm.sign(keyid, tosign, hashalgo)
  File "/Users/user/Downloads/endesive/examples/pdf-sign-cms-hash.py", line 23, in sign
    assert self.tosign == data
AssertionError

certificates are created by cert-make-hsm.py, run this script after fetching repo (incompatible change in cryptography package)
pdf.pdf is created by pdf-make.py
each run of pdf-sign-cmd-hash.py produces no errors, cant reproduce this step

Upgrading from endesive=2.0.6 to endesive=2.0.9 solved the problem with pinning the timestamp and forcing timestampingurl=None. Sorry for this.
Thank you for clarifications. Tested with the pdf.pdf file created by pdf-make.py + my test certificate and it work correctly as described.
Having tested with other PDF files for some it works correctly and for others it fails with the error mentioned above.
Since one of the PDF file is not private I shared it with you below. Hope you can reproduce.
law.pdf

endesive/pdf/cms.py line 689: ID = prev.trailer.get("/ID", None)
The PDF signature has its own requirements, there is no document ID in your file, so it is generated on each run and affects the hash

Thank you so much for clarifying this.
So for such files that already exist, the signing process must happen in one go using an HSM implementation that signs the file in one go instead of this split approach. Thanks again.

Please check if the repo version now works for you with different documents, including those without ID

I've tried the latest master and documents without ID still don't work as they produce a different fingerprint each run and the assertion fails in the same place. I did notice the pinned id that is calculated and saved in the JSON as well as the other changes. Can't get it to work.

for clear pull, cert-make-hsm.py, pdf-sign-cms-hash.py produces each time the same hash, after signing acrobat reports no errors
law.pdf.json.txt
law-signed-cms-hash.pdf
law.pdf

I confirm the latest version from master fixes the problem of signing documents without and ID.
I didn't realise at first the code was picking up the version that was installed as a pip package. Tested with the master code and it works.
Thank you for this quick fix.


Copy/cut operations not working

Explain the bug
Sometimes when I copy/cut then paste it doesn't copy the files it will do one of the folders and then show it completed with no files or other folders. Click and drag will also do that same.
Expected behaviour
It should copy all files/folders
Logfile
debug.log
debug_fulltrust.log
System Information

Version: <IP_ADDRESS>
OS Version: 10.0.19043.1110

Hi, do you find this still an issue in 1.9.21?

Haven't seen it happen again yet

Thanks! Please let us know if it does.


Setting getter property should return error

Bug Report
🕗 Version & Regression Information
4.6.4
⏯ Playground Link
Playground link with relevant code](https://www.typescriptlang.org/dev/bug-workbench/?ts=4.6.4#code/KYDwDg9gTgLgBAYwDYEMDOa4GEIFswpTABiAlksALIQAmwScA3gFDNxxgCuARkqQnADmweKRoAKAJQAuOGhhRSAO0FM27OERicoSuDAAWpNADoAZuWABJGiZgQAyguWCpJhBCUIUMcQHIAWj8AGn0jUyIAN2NSTzgAH3i4Pz9JAG51AF9Wdi5efjgLChtZJU5cbmAoOABeOAAGDNyePgEomM9ZeUUVWuS-DPU81sRPbs4EeyhxZVIYAH5ZAAVCGFIUJAAeHHxCEktqOiQAPkk1DTgAeW4AK2BJk3Q0UkElcUNjUNmYdKycjhaBWQnmAUlkOwIRDIFEO9HOGi0Oj0SmAAHdsHhIfsYbR6OIWBd2EVrDRZB9TMSbMF1BcxLI-Gg-DS4JlfuxstkgA))
💻 Code
export class CompareFileModel {

  public get id(): string {
    return this.fileId.toString().concat('-', this.revision || '');
  }

  public fileId: number = 0;
  public revision: string = '';

  public constructor(init?: Partial<CompareFileModel>) {
    Object.assign(this, init);
  }

  public clone(): CompareFileModel {
    return new CompareFileModel({
      fileId: this.fileId,
      // id is getter property. It should not allow setting getter property
      id: 's'
    });
  }
}

Workbench Repro
🙁 Actual behavior
Typescript reported there is no error.
🙂 Expected behavior
Code should not compile, as we're setting getter property

It's a design limitation. TypeScript does not differentiate between properties with accessors and without.

Yeah, duplicate of #9726, even though it doesn't look like it at first.  The lack of a setter is irrelevant at the point you expect an error since you're just initializing an object literal, which wouldn't call it anyway.  Even if there was a setter for id you'd probably still want to exclude that property from init if the setter did anything more complex than this._id = val.

Even if #9726 were implemented, mapped types like Partial would still include non-enumerable properties. Really you'd want an entirely new kind of distinction here, since I don't think the non-enumerability of id is the real crux -- if getters were enumerable tomorrow, the code would still be wrong. You'd want something like Settable<T> that was a mapped type that only produced settable properties, but we don't have write semantics anywhere in relatability.

but we don't have write semantics anywhere in relatability.

So https://github.com/microsoft/TypeScript/issues/21759, then.


remove kokoro related scripts and assets

As discussed in #1873.

@umarcor - Snap - it looks like i created #1945 at the same time.....


[Self-hosted Bug]: 406 (Not Acceptable) / Nuget "Part URI cannot start with two forward slashes."

Impact
I'm unable to use this version
Describe the bug
Pushing the Package "mongodb.driver.core.2.14.1.nupkg" runs into the Error:
406 (Not Acceptable) / Nuget "Part URI cannot start with two forward slashes."
The Bug corresponds to this Issue: https://github.com/dotnet/aspnet-api-versioning/issues/637
Repro Steps

Download mongodb.driver.core.2.14.1.nupkg
Try to push the Package on Self-Hosted NuGet.Server v<IP_ADDRESS>

Expected Behavior
Successfull Push
Screenshots
No response
Additional Context and logs
No response

I think you have diagnosed the root cause accurately. This appears to be a corrupted package w.r.t. to OPC (the stringent rules on how packages should be shaped that NuGet.Core -- NuGet 2.x -- follows closely. NuGet.Server uses this library for package processing.
I would recommend notifying the author (perhaps via GitHub or Contact Owners on NuGet.org) of this problem so that the package can be fixed.
If you are interested in attempted a bug fix to handle this edge case, you can consider opening a PR against the NuGet.Server repository: https://github.com/NuGet/NuGet.Server
It may be difficult to fix this problem however since it's not clear to me whether it's a problem specific to NuGet.Server or if all NuGet processing code (e.g. NuGet.Core and the newer NuGet.Packaging) would have the same problem.


Make it work without requiring Symbol polyfill

The best would be to have babel to transpile Symbol itself.

Same for Reflect

so babel will add local polyfill but target browsers could be Symbol-ready. so we loose in performance and module size in that case

Why use the symbol at all? Just change:
return {
  configurable: true, // must be true or we could not be changing it
  get() {
    if (!this.hasOwnProperty(_key)) {
      this[_key] = fn.bind(this);
    }
    return this[_key];
  }
};

to
return {
  configurable: true, // must be true or we could not be changing it
  get() {
    var fn = this[key].bind(this);
    Object.defineProperty(this, key, {
      configurable: true,
      writable: true,
      value: fn
    });
    return fn;
  }
};

The current way is slower too since the overhead of the getter is going to be there every single time.

@sebmck you are right, that's actually how it was implemented initially https://github.com/andreypopp/autobind-decorator/commit/fb29744229f06bec8575e089f7c5fe28a407c6e5#diff-1fdf421c05c1140f6d71444ea2b27638


The Ink Cartridge

It's tooltip is "May be refilled useing dyes to use for 3D prints" however its now also used for the Wide Format Printer, which is not a 3D printer but a 2D Printer,
I propose the tooltip is changed to, "May be refilled useing dyes to use for all kinds of prints." or something similar.

Whoops this one was a bug.


Initial commit for amd transformer upgrade path

Hi @guybedford,
here is the promised PR. I've published the transformer plugin in an initial 0.0.1 version to npm.
Best regards

Won't be merged until all unit tests will pass.
@guybedford Can you help with investigating the problems? You can use my forked builder repo
https://github.com/jrauschenbusch/builder/tree/upgrade-amd-transformer.

It seems that the loading operation of several files fails as they are loaded from the wrong path:
test/output  **should be test/fixtures/test-tree**
  L jquery.js
    global.js
    plugins.js
    text_plugin.js
    first.js
    second.js

If the missing files are copied to test/output, then the following errors occur:
AMD 1, 3:
Error: (SystemJS) XHR error loading /builder/test/fixtures/test-tree/traceur
Unable to load transpiler to transpile /builder/test/output/first.js

AMD 4, 7:
Error: (SystemJS) Module /builder/test/output/amd-4.js interpreted as global module format, but called System.register.

AMD 11:
Error: (SystemJS) Multiple anonymous System.register calls in module /builder/test/output/amd-11.js. If loading a bundle, ensure all the System.register calls are named.

UMD:
Potentially unhandled rejection [16] Error: (SystemJS) Module /builder/test/output/umd.js interpreted as cjs module format, but called System.register.


Fix accidental erase of paper contents by spamming save action

About the PR
RunOnSaved() now disables the save button, making it unspammable.
Why / Balance
Fixes https://github.com/space-wizards/space-station-14/issues/26024
Technical details
As the text rope processing takes a while, it was possible to click (or hit ctrl+enter) the save button multiple times during this process and launch multiple  RunOnSaved() event handlers.
One of the step in the text processing is clearing the textbox, which makes last handler to save the empty contents.
Added button disabling to RunOnSaved()  to prevent this from happening. As Ctrl+Enter also checks for button disabled, this fixes both accidental keyboard shortcut spam and click spam.
Media
None required
Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Breaking changes
Changelog
:cl:

fix: Fix accidental erase of paper contents by spamming save action.

Hey, could you add a changlog entry for this? Have a look at the PR guidelines to see how it needs to be formatted so that the bot can read it. It will then be automatically added to the in-game changelog and your name will show up in the game's credits :)

Wasn't sure this warrants a changelog. Added!

I would say HOP players will love to hear that this got fixed :)
I gave it an in-game test and works as intended.
Thank you for the fix and your first contribution!

I would say HOP players will love to hear that this got fixed :)

slarticode is definitely correct, I jumped in glee when I read the CL
(I'm exaggerating, my legs are too sore for that. But I am definitely thrilled :D)


MultiplayerSynchronizer node order is significant

Godot version
4.0.2.stable
System information
Ubuntu 22.04
Issue description
While trying to recreate the multiplayer scene replication code as demonstrated in https://godotengine.org/article/multiplayer-in-godot-4-0-scene-replication/ I discovered that the order of the MultiplayerSynchronizer nodes is significant. If the player input synchronizer (that sends input to the server) appears before the player position synchronizer (that sends position from server to clients) then the replication fails. The other way around it works.
In the failing configuration, it appears that the player property on the Player node spawned on the connecting client does not get set, which leads to the multiplayer authority not being set on the player input.
Notably, I tried changing the node order on the example project in the article linked above and that does not replicate this bug. The primary difference is that my project is 2D and the example is 3D.
Steps to reproduce

Run two instances of the attached project.
Press "Host" in the first instance.
Note that the arrow keys allow the player square to move.
Press "Connect" in the second instance.
Note that the arrow keys fail to move the player square to move in the second instance but that the first instance continues to work/move and that the position is replicated to the second instance.
Stop the project and swap the order of the two MultiplayerSynchronizer instances under the Player node.
Repeat steps 1-5 but note that the second instance now allows the player square to move.

Minimal reproduction project
godot_4_multiplayer.zip

While this order is done consistently in the example, it isn't obvious so there should be documentation for this

Just a little side-question ... looking at this MRP. Is this still in Godot supported
https://godotengine.org/qa/137477/how-to-use-godot-4-setget?show=139941#a139941
or a bug somewhere? E.g. with
extends Node
@export var bug := 1 :
	set(v):
		pass

I get on tip 8784f95e1f5b7112beefeaf8ef6bdd2ca6c907f2
ERROR: Class name cannot be empty.
   at: get_class_icon (editor/editor_node.cpp:4471)

When I run your code i get this error
E 0:00:07:0926   on_replication_start: The MultiplayerSynchronizer at path "/root/Multiplayer/Level/FirstLevel/Players/2014463641/ServerSynchronizer" is unable to process the pending spawn since it has no network ID. This might happen when changing the multiplayer authority during the "_ready" callback. Make sure to only change the authority of multiplayer synchronizers during "_enter_tree" or the "_spawn_custom" callback of their multiplayer spawner.
  <C++ Error>    Condition "pending_sync_net_ids.is_empty()" is true. Returning: ERR_INVALID_DATA
  <C++ Source>   modules/multiplayer/scene_replication_interface.cpp:242 @ on_replication_start()

I think this is related to your issue


[MCTF] Filter strength bug fix

When Bitsperpixel default value is different to 0, it locks MCTF to run in bitrate analysis mode, overriding the filter strength set by the user.
Issue: MDP-52518
Test: Manual testing
Signed-off-by: Sebastian Possos<EMAIL_ADDRESS>

This pull request is for mss2018_r2 branch, this bug is not present on master.

@onabiull We scoped the fix into PV5 HF3. Can you help merge it into mss2018_r2 branch?


could not set up shops,a exception thrown while click at a sign after click a chest.

see exception report at gist:
https://gist.github.com/Tollainmear/5beb507f7c9aa8b5594c740b9331dede#file-gistfile1-txt
while i create a shop(while i hold a redstonedust to click at a sign),the exception thrown.

btw may i transport this nice plugin to chinese minecraft forum 'MCBBS'?

thanks for your reply，i've test this latest version at my localhost server and i forget to install the currency plugin,that's my mistake.
I have post Carrot-Shop at HERE with links to your git repo,and i also leave a link to make users could only download CS. from your ore page.
: )

Fin, I'm going to deal with it~

I have already translated the language file, I found two problems with CarrotShop:
1.CarrotShop dont have config file, how could we choose the language file?
2.Could you describe what is "PRICE_TRANSLATION_HINTS" ? I dont understand.
😀

You can modify the language file you find in the config folder directly.
Maybe you wanted each player to be able to use a different language? If that is the case, it is not possible but would be a nice feature. I can make that hapen
The price hint is not used. It is just to inform the translators of the available placeholders for the price format

language.zip
OK, and Im not mean that, the word 'we' in 1st problem means us server owners, it seem like no config file to make Chinese server owners choose the Chinese language file. So maybe we could just replace the language file with which is translated. however, I think its dosent matter~ thanks for updating carrotshop! here is the translated language file in Chinese, and you can use this anywhere~
BTW, I have transport CS to MCBBS with a detailed tutorial in Chinese, considering there some server owners was not good at English (just like me haha), you could add that Link to '/carrotshop help' to guide Chinese users to lean this plugins

Cool, I'm glade to see that~~ 😆

This is now possible with 1.17 if you edit WIKI_URL in language file


add counter when generating files for multiple lstm cells

When we have multiple lstm cells, we also have multiple weights and bias vhdl files. The idea is to add an counter at the file names or create a dictionary for each of them.

obsolete


Sort declaration cleanup

Removes name from basic sorts.
Removes unconditional get_sort.
Adds Presort trait.
Removes basic sorts from being stored in primitives.
Removes unused and undocumented macros.
Makes Rational set lazy static.

I had to cache the sort names to fix a performance regression with cykjson, where the lookup in the symbol table was noticeable. This PR can be merged with the cached names, and I'm going to investigate whether the Symbols are actually helping with performance.

@yihozhang The ~20% speedup on eggcc-extraction is what we expect to see; it matches the speedup in #441. (It's lower than you might be expecting because Saul changed run 5 to run 2 to speed up CI in #443.)

Wow nice, that's a huge win!


Intermittent: "Failed to parse CAR file: early eof"

via Windows run https://github.com/subconsciousnetwork/noosphere/actions/runs/6487283518/job/17619038228?pr=671

2023-10-11T21:04:26.8911296Z test latency::clients_can_sync_when_there_is_a_lot_of_content ... FAILED
2023-10-11T21:04:26.8911884Z
2023-10-11T21:04:26.8912819Z failures:
2023-10-11T21:04:26.8912868Z
2023-10-11T21:04:26.8914396Z ---- latency::clients_can_sync_when_there_is_a_lot_of_content stdout ----
2023-10-11T21:04:26.8916647Z Error: Internal error: Some("Failed to parse CAR file: early eof")

On [nightly linux]:
2023-10-19T18:38:01.0409863Z ---- latency::clients_can_sync_when_there_is_a_lot_of_content stdout ----
2023-10-19T18:38:01.0410211Z Error: Internal error: Some("Failed to parse CAR file: early eof")

Windows failure

2023-10-18T19:00:44.9719954Z ---- latency::clients_can_sync_when_there_is_a_lot_of_content stdout ----
2023-10-18T19:00:44.9720833Z Error: Internal error: Some("Failed to parse CAR file: early eof")

We get an error from iroh_car::Error::Parsing with string "early eof". Possible sources that match this error all deconstruct into io::Error, but io::Error::UnexpectedEof stringifies as "unexpected end of file", not "early eof".
Possible sources:

From<cid::Error>: cid::Error::Io(io::Error)
From<multihash::Error>: multihash::Error { kind: Kind::Io(io::Error) }
ld_read(): unsigned_varint::io::ReadError::Io(io::Error)
reader.read_exact(): io:Error
CarHeader::decode() -> DagCborCodec::decode(): dag_cbor::CborError::{Io(io::Error), Cid(cid::Error::Io(io::Error)}

This failure is occuring more often than not.


Task #2 Compile and Execute a Program

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

/question


Added concurrency guide.

The only thing I worry about this doc is it's getting pretty deep into Baseten-specific features. We should def ship it for now as is, and once I have more Baseten docs up I might move some of the information around or add some redirects.

for sure -- going to merge for now, but feel free to move this to the baseten side if you feel it belongs there.
Another thing to note is that I am not thrilled with the way all of this works today. We don't want to block documentation on future feature development, but in my ideal world, both of these types of concurrency are configured in the Truss config


Limit search to current profile (restore feature)

Organization pages already limit searches to the current organization:

User profiles should behave the same:

This issue replaces #548
2018 edit: this type of search has been dropped from GitHub, this might not be easily possible

Reopening because this feature was dropped in the last GitHub update

@bfred-it It's unclear what the actionable thing to do here is?

Restoring the feature, somehow. This is on your profile:

"Search this user" could appear up there, like "In this repository" appears when searching from a repo:

Mockup:

That makes sense. 👍

https://blog.github.com/changelog/2018-11-26-searching-by-user-from-a-profile-page/


df plugin doesn't support ValuesPercentage

Newer versions of collectd support ValuesPercentage on the df plugin:
https://collectd.org/wiki/index.php/Plugin:DF
Looking at the code, it doesn't look like the module supports this. I find reporting by percentage to be more useful than absolute values.

Sorry, looking at the wrong thing. DF plugin clearly has ValuesPercentage.


Intermittent work / fail behavior of OCP CAD Viewer

Using

a recently updated Manjaro (arch linux based)
tested with both python 3.12.5 and python 3.10.14
vscode 1.93.1
Installed extensions
code --install-extension ms-python.python
code --install-extension bernhard-42.ocp-cad-viewer  # ver 2.4.1
Used the "Quick build123d" install button

After following the installation instructions, when opening the auto created demo file "ocp_vscode_demo.py" the viewer pane intermittently does not initialize properly.
It behaves like an initialization race condition.  Changing the way I load the file changes the timing and the failure frequency, e.g. opening a folder (workspace) which contains the demo file vs opening the demo file directly.  When opening a folder the viewer fails to initialize most of the time, whereas when opening the file directly the viewer pane works most of the time.  But both cases have intermittent work/fail behavior.
My test system is on the slow side (running in VirtualBox) so it probably is exposing this race condition where a faster system would not see it.
Originally posted by @WayneSherman in https://github.com/bernhard-42/vscode-ocp-cad-viewer/discussions/104

I found one problem that could be related to it. The viewer tries to respond to a resize window event and call viewer.resize when viewer is not ready initialized. Hence the viewer crashes and does not work properly any more.
On fast machines, the viewer might be initialized before the resize event hits. The fix is to check whether viewer is not null before responding on the resize event.
But, I did not check this against a "slow system" - due to not having a slow system :-)

I've published v2.5.0 which fixes the potential race condition. Could you please test it since I can't reproduce it

I testet a fresh install (windows OS) and the hanging empty viewer window on first start remained. I will make a seperate issue in the following days.

@CePeU Yes please, since there was no issue open I forgot to deal with it

I've published v2.5.0 which fixes the potential race condition. Could you please test it, @WayneSherman, since I can't reproduce it

Thanks for the updates.   I tested v2.5.0 and the behavior is unchanged.  When I "Open Folder..." with the demo file folder, most often the viewer window does not display properly.  When I "Open File..." with the demo file, the viewer window usually displays correctly.

I have no idea how to reproduce this


Querstion regarding MFA Token integration

Hey,
i wanted to clarify regarding the usage of using Cognito with MFA token.
mfa_configuration        = "ON"
allow_software_mfa_token = true

Does this require the block for software_token_mfa_configuration?
software_token_mfa_configuration {
enabled = true
}

If i add this i keep running into:
Error: Unsupported block type
Blocks of type "software_token_mfa_configuration" are not expected here.
Is this a version issue with my TF or where might i have to potentially look there to enable this feature?

hi @jaystary,
just checked and our documentation is a tiny bit misleading here (it is a copy&paste of the providers' documentation that missed adjustment)
to clarify:

allow_software_mfa_token = true  adds the software_token_mfa_configuration block and sets enabled=true
allow_software_mfa_token = false does not add the software_token_mfa_configuration

You do not set the software_token_mfa_configuration block but use allow_software_mfa_token = true instead in the module arguments.
please let me know if this helps you or let me know how I can help you better.
some background information: blocks are not (yet?) supported for modules so blocks are always represented as variables and rendered conditionally in the resource in the code of the module.

I tried this and despite Cognito setting being set to Required in MFA it still wouldnt trigger the software MFA, in fact it ignored it completly and just let me log in via Email/Password - please see the code snippet (some things are removed though). It works fine for SMS MFA though if i add the SMS configuration block.
module "cognito_user_pool" {
  source  = "mineiros-io/cognito-user-pool/aws"
  version = "~> 0.4.0"

  name = "${local.eks_cluster_name}-userpool"

  allow_admin_create_user_only = true

  enable_username_case_sensitivity = false
  advanced_security_mode           = "ENFORCED"

  alias_attributes = [
    "email",
    "phone_number"
  ]

  auto_verified_attributes = [
    "email"
  ]

  # If invited by an admin
  invite_email_subject = "..."
  invite_email_message = "...
  invite_sms_message   = "..."

  default_email_option  = "CONFIRM_WITH_LINK"
  email_subject_by_link = "Your Verification Link"
  email_message_by_link = "Please click the link below to verify your email address. {##Verify Email##}."
  sms_message           = "Your verification code is {####}."

  challenge_required_on_new_device      = true
  user_device_tracking                  ="ALWAYS"

  #MFA
  mfa_configuration        = "ON"
  allow_software_mfa_token = true
  sms_authentication_message = "..."

  # Password
  password_minimum_length    = 8
  password_require_lowercase = true
  password_require_numbers   = true
  password_require_uppercase = true
  password_require_symbols   = true
  temporary_password_validity_days = 3

  # App Client
  clients = [...]

}

looks like your code is correct:
when running terraform plan with your configuration I get the following (partial) plan:
  # module.cognito_user_pool.aws_cognito_user_pool.user_pool[0] will be created
  + resource "aws_cognito_user_pool" "user_pool" {

      [...REMOVED..]

      + mfa_configuration          = "ON"

      [...REMOVED..]

      + sms_authentication_message = "Your temporary password is {####}."

      [...REMOVED..]

      + sms_configuration {
           [...REMOVED..]
        }

      + software_token_mfa_configuration {
          + enabled = true
        }

      [...REMOVED..]

which matches what is documented in the provider itself under:
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/cognito_user_pool#enabling-sms-and-software-token-multi-factor-authentication

So let's just assume everything is correct on the terraform side... what does your userpool show? is MFA required or optional? seems like you can't set it to required if the user pool was created initially with off or optional.
never tried to upgrade it and not sure if this will be a silent fail in the provider.. can validate tomorrow.
can you check the following:

what is the current state of the cognito pool (e.g. check in AWS console under mfa) is it optional or required?
did you try to change it to required from being off/optional in the past? (first deploy optional, then changed to required?)

just validated with the settings provided and running aws-cli:

set cognito pool to require MFA for users
use TOPT (software token challenge) only
created a user
1st challenge encountered: NEW _PASSWORD
set a new password
next challenge encountered: MFA_SETUP  "ChallengeName": "MFA_SETUP",
  "ChallengeParameters": {
      "USER_ID_FOR_SRP": "cafffbc3-ffff-ffff-ffff-7ba3fffc92a9",
      "MFAS_CAN_SETUP": "[\"SOFTWARE_TOKEN_MFA\"]"
  },

so on first login you are forced to set up MFA and can't just continue.

I did more research on this topic, and i had the misconception that the Totp would be accessible via Cognito hosted UI, but seems i have to integrate an additional Auth flow on top of that (e.g. via Amplify). From your end, my issue is solved. Thanks a lot for the fast reply and the help!


Update search-the-audit-log-in-security-and-compliance.md

Since the table has been removed, updated the document accordingly.

@JohanFreelancer9 Please copy review

issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/9988

Learn Build status updates of commit 434e99e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/search-the-audit-log-in-security-and-compliance.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@robmazz Please merge this PR. Thanks

@robmazz Gentle reminder

Closing as per the comment in #9988