"targetWriter" is a method parameter, and should not be used for synchronization.

Code Path:server/src/main/java/org/apache/iotdb/db/engine/compaction/writer/AbstractCompactionWriter.java

Synchronizing on a class field synchronizes not on the field itself, but on the object assigned to it. So synchronizing on a non-final field makes it possible for the field’s value to change while a thread is in a block synchronized on the old value. That would allow a second thread, synchronized on the new value, to enter the block at the same time.
The story is very similar for synchronizing on parameters; two different threads running the method in parallel could pass two different object instances in to the method as parameters, completely undermining the synchronization.

[WeOpen Star]I would like to help

i will try my best to do it，please assign to me


Check whether the task finishes before deferring the task for RedshiftClusterSensorAsync

Like the SnowflakeOperatorAsync, we need to verify if the task has already completed before deferring it to prevent unnecessary deferring. This way, we can skip deferring the task if it has already been finished. To accomplish this, we can use the poke method found in the sync counterpart of most sensors.


OneDrive policies - Exclude specific kinds of files from being uploaded - Note - *.lnk not synced is incorrect

The recently changed Note to the OneDrive setting Exclude specific kinds of files from being uploaded states that *.lnk files are not synced at all. Which is not the case at least in the deferred ring for now.
If there is a minimum version for this to be the case, please add the version to the box.

File: https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/blob/public/SharePoint/SharePointOnline/use-group-policy.md

Hello @SasStu I am looking into this


fix switch issues

Fixed the problem that 3 wired switches could not be used in the latest version of firmware and plug-ins. 修复了在最新版固件和插件中，3路有线开关无法使用的问题


Voting type: open

Is your feature request related to a problem? Please describe.
Currently there's no option to create a voting with anything other than registered candidates. This makes difficult to vote for things, like the best food or movie, which will never be registered to have a voterId.
Describe the solution you'd like
A new voting type where candidates are not required, where the voter can just cast the vote to a text, without the need of a specific verdict. No limit of how many options a vote can nominate. The winner is the option with most votes.
As an optimal solution words could possibly be grouped by similar spelling.
Describe alternatives you've considered
Registering voting options (not candidates) would create voters that would never vote. It doesn't make sense.
Additional context
voting: {
  ...
  votingDescription: 'Best food'
  ...
}

vote: {
  ...
  choices: ['pizza', 'burger']
}

More than just an open voting this is about votings without candidates.
So 2 new types should be added:

Open: no pre-defined options
Selection: pre-defined options


Move the .pro to the root file

Move the .pro to the root file, possibly rename EasyPaint into src. This adheres more to common build usage.

Added CMakeLists.txt at root for building, later directory will renamed


[FEATURE_REQUEST] Revoke a room access token

Is your feature request related to a problem? Please describe.
If a user's access to a room changes (upgraded to room:write or downgraded to room:read), the previous token used to enter the room is still valid for up to one hour.
There should have a way to revoke a token.

Granted that every token is a JWT bearer token and possessing one is enough to obtain the permissions it has, the only way I see to actively revoke an unexpired token is to keep an administration of revoked tokens in-memory on the server side (at least until they expire), and check that administration on every request.
Does that match with what you would have in mind for this, too?

@nvie Yes it does!


[Enhancement] Make new publish mechanism enabled by default

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Problem Summary(Required) ：

Make new publish mechanism enabled by default
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-2.5


Memory and CPU restriction using rkt CLI

I understand that rkt implements memory and CPU restriction by reading those values from the manifest. Is there any way to set these via the CLI (either per app or per pod)? Is there a way to read those via the CLI?

Unfortunately, parameters in the rkt command line are not implemented yet. Meanwhile, you can use the following workarounds:
If you start rkt via a systemd unit file, you can define the pod-level isolators:
[Service]
CPUShares=512
MemoryLimit=1G

If you start rkt via systemd-run, you can also define the pod-level isolators:
# sudo systemd-run -p MemoryLimit=400M -p CPUShares=800 rkt run coreos.com/etcd:v2.0.10
Running as unit run-10165.service.
# systemctl cat run-10165.service
(...)
CPUShares=800
MemoryLimit=419430400

And to change the limits per app, you would need to modify the ACI:
$ actool patch-manifest --replace --isolators=resource/cpu,request=50m,limit=100m:resource/memory,request=1G,limit=2G

Another option here is to use --from-pod-manifest, which is what we do in the Kubernetes integration /cc @yifan-gu

Thanks @jonboulle and @alban
The use case here is Nomad integration. For the rkt driver, nomad cannot set CPU and memory restrictions currently from the CLI. The nomad scheduler cannot take these into account while scheduling. This also makes the interface different from other drivers. The PR is here: https://github.com/hashicorp/nomad/pull/165
I think irrespective of Nomad, rkt CLI should support these.

depends on the spec: https://github.com/appc/spec/issues/552

Pull request to fix this: https://github.com/coreos/rkt/pull/1851


Move jenkins.. scripts into a jenkins directory off the project root.

This will make the contents of the root of our project more legible, and effectively bring the README into view sooner on the main page of the project.
It will require modifications to most of those scripts and to the netci.groovy script itself.
The build checks will break on the PR for this change because the Jenkins task definitions will be out of sync with the actual locations of the files, so we will need to accept the change despite the breaks and quickly react to breaks following that change.

:+1:

@dilijev how does this look? https://github.com/Microsoft/ChakraCore/compare/master...ianwjhalliday:mvjenkins
I did not test this at all. Just throwing it up there as a rough start.

@ianwjhalliday Great! Exactly what I was planning. I would also like to remove the jenkins. prefix from them since it's redundant. Might as well make the folder name capitalized Jenkins to go with the new scheme.
That should work. Keep in mind your pull request will break the checks because they won't be able to find the scripts where expected (it uses the jobs generated by the current master when netci.groovy was last updated). The subsequent build on master should succeed though, unless something went wrong.

@mmitche FYI does @ianwjhalliday's change in https://github.com/Microsoft/ChakraCore/compare/master...ianwjhalliday:mvjenkins look good to you?

jenkins/jenkins.buildone.cmd looks like it need ~%dp0 in front of the script call?

@mmitche yep, thanks for catching that

For the linux scripts as defined, you'll want to make sure that the scripts are invoked relative to the working directory and not the script directory. I don't know if linux has an equivalent of %~dp0 for that purpose.

In https://github.com/Microsoft/ChakraCore/commit/28c00648ddd98ec48d0fa176351aea32cf103d10 it looks like the execute bit was lost on the shell scripts when you renamed the files.
Please run git update-index --chmod=+x [file] on the shell scripts.
This is required for Ubuntu to be able to execute the scripts.

@ianwjhalliday FYI there are currently several open pull requests regarding changes to the scripts so you'll probably want to reapply the changes you made to move the scripts from scratch after those PRs are resolved.

Yeah, good idea. Thanks for the heads up.

The redirection stub scripts are still in the root directory.  This will be fixed by #531.


#1619: Adds NSError as input to resourceValuesForKeys:error: method t…

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necesarry)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description
This fixes handling of a potential unexpected error in SDImageCache, when cleanDiskWithCompletionBlock: is executed when an implementing app is sent to the background. Instead of passing NULL as the NSError argument, we now pass a NSError reference and continue the loop if an error occurred while reading resourceValues for a random cached image. The crash is only seen happening for iOS10 users.

Looks interesting. I will take a closer look soon

Thanks @jimmaye


feat(updater): Support runtime package bootstrap and runtime versioned package bootstrap

What does this PR do?
Adds support for bootstrap over RC & bootstrap with version over RC
Motivation
Tracer upgrades
Additional Notes
N/A
Possible Drawbacks / Trade-offs
We don't have state yet for bootstrap (nor backend support)
Describe how to test/QA your changes
Start the updater & call the bootstrap method with
curl -X POST --unix-socket /opt/datadog-packages/updater.sock -H 'Content-Type: application/json' http://updater/datadog-agent/bootstrap -d '{"version":"1.21.5"}' (please adapt the version & package with what's available in the catalog)

/merge


[Question] Difficulty Achieving Correct Orientation in 'PickCube-v0' with Pose Control

Hello,
I'm currently working with the PickCube-v0 environment in ManiSkill2 (version 0.5.3) using the pd_ee_delta_pose control mode on Ubuntu 22.04.4 LTS. While the robot successfully reaches the desired position when controlling only the position, it struggles to achieve the correct orientation (rotation) when attempting to control the full pose. The issue is particularly evident when controlling both position and orientation (or even just orientation), as the robot fails to reach the target pose/rotation for most seeds other than env.reset(seed=1).
I'm using the following PDEEPoseControllerConfig settings:
frame = base
use_delta = True
normalize_action = True
interpolate = True
use_target = False
Could you please provide guidance on what might be going wrong with the orientation control and how I can effectively control the robot to achieve a specific target pose? Any suggestions or corrections would be greatly appreciated.
Thank you for your time and assistance!
import mani_skill2.envs
import numpy as np
import gymnasium as gym
from scipy.spatial.transform import Rotation
from sapien.core import Pose

# Initialize environment
env = gym.make("PickCube-v0", obs_mode="state_dict", control_mode="pd_ee_delta_pose", reward_mode="normalized_dense", render_mode="human")
state = env.reset(seed=2)

# Get the target and current poses in the robot's base frame
robot_pose = env.agent.robot.get_pose().inv()
target_pose_robot_frame = robot_pose * env.unwrapped.obj.get_pose()  # Target object pose
current_pose_robot_frame = robot_pose * env.agent.robot.get_links()[-3].get_pose()  # End-effector pose

# Adjust target pose to hover above the object
target_pose_robot_frame.p[2] += 0.05
target_pos = target_pose_robot_frame.p
target_rot_quat = target_pose_robot_frame.q

current_pos = current_pose_robot_frame.p
current_rot_quat = current_pose_robot_frame.q

# Set thresholds and max deltas
threshold_pos = 0.01  # Position error threshold
threshold_rot = 0.05  # Rotation error threshold
max_delta_pos = 0.01  # Maximum position delta per step
max_delta_rot = 0.01  # Maximum rotation delta per step

def gripper_shortest_angle(angle):
    """Ensure angle is between -π/2 and π/2 for gripper rotation."""
    angle = angle % (2 * np.pi)
    if angle > np.pi:
        angle -= 2 * np.pi
    if angle > np.pi/2:
        angle = np.pi - angle
    elif angle < -np.pi/2:
        angle = -np.pi - angle
    return angle

def quaternion_to_gripper_shortest_euler(q1, q2):
    """Calculate the shortest Euler angle difference between two quaternions."""
    r1 = Rotation.from_quat(q1)
    r2 = Rotation.from_quat(q2)
    r_diff = r2 * r1.inv()
    euler_diff = r_diff.as_euler('xyz')
    euler_diff = np.array([gripper_shortest_angle(angle) for angle in euler_diff])
    return euler_diff

# Control loop to move towards the target pose
pos_error, rot_error = np.inf, np.inf
while (pos_error > threshold_pos) or (rot_error > threshold_rot):
    # Compute position and rotation differences
    pos_diff = np.clip(target_pos - current_pos, -max_delta_pos, max_delta_pos)
    rot_diff = quaternion_to_gripper_shortest_euler(current_rot_quat, target_rot_quat)
    rot_diff_norm = np.clip(rot_diff / (np.pi/2), -max_delta_rot, max_delta_rot)

    # Create the action vector with the gripper action set to 0.9 (open)
    action = np.concatenate((pos_diff, rot_diff_norm, np.array([0.9])), axis=0)

    # Take a step in the environment
    state = env.step(action)

    # Update the current pose from the new state
    current_pose_robot_frame = robot_pose * env.agent.robot.get_links()[-3].get_pose()
    current_pos = current_pose_robot_frame.p
    current_rot_quat = current_pose_robot_frame.q

    # Calculate errors
    pos_error = np.linalg.norm(current_pos - target_pos)
    rot_error = np.linalg.norm(quaternion_to_gripper_shortest_euler(current_rot_quat, target_rot_quat))

    env.render()

# Picking up the object by closing the gripper
grasping_steps = 50
for i in range(grasping_steps):
    # Move downward and then close the gripper
    action = np.array([0, 0, -0.05, 0, 0, 0, 0.9])
    if i == grasping_steps - 1:
        action[-1] = -0.5  # Close the gripper on the last step
    state = env.step(action)
    env.render()

# Lifting the object
lifting_steps = 50
for i in range(lifting_steps):
    action = np.array([0, 0, +0.05, 0, 0, 0, -0.5])  # Move upward with the gripper closed
    state = env.step(action)
    env.render()

# Close the environment
env.close()

You can try turning off use_delta and set normalize_action False and interpolate false. if you keep taking a lot of the same action  / target ee pose, it should eventually reach it

@shayan-ahmadi9 has your issue been resolved?

Thank you for your help!
I applied the recommended changes with the following configuration:
frame = base
use_delta = False
normalize_action = False
interpolate = False
use_target = False

I also adjusted the main control loop as follows:
# Control loop to move towards the target pose
pos_error, rot_error = np.inf, np.inf
while (pos_error > threshold_pos) or (rot_error > threshold_rot):

    # Convert Quaternion to Euler
    target_rot_euler = Rotation.from_quat(target_rot_quat).as_euler('xyz')

    # Create the action vector with the gripper action set to 0.9 (open)
    action = np.concatenate((target_pos, target_rot_euler, np.array([0.9])), axis=0)

    # Take a step in the environment
    state = env.step(action)

    # Update the current pose from the new state
    current_pose_robot_frame = robot_pose * env.agent.robot.get_links()[-3].get_pose()
    current_pos = current_pose_robot_frame.p
    current_rot_quat = current_pose_robot_frame.q

    # Calculate errors
    pos_error = np.linalg.norm(current_pos - target_pos)
    rot_error = np.linalg.norm(quaternion_to_gripper_shortest_euler(current_rot_quat, target_rot_quat))

    env.render()

The robot’s behavior has changed slightly, but it still struggles to reach the desired rotation. The issue persists, especially with achieving accurate orientation.
Any further suggestions or insights would be greatly appreciated!
Thanks again for your time and help.

I find use delta pose as action for EE mode in maniskill, can adjust the rotation nearly  to the target one.


add entrypoint to dockerfile

This change mimics the base-notebook dockerfile to correct issues with
connecting to spark.

@willb ptal, i copied the CMD from the base-notebook. this worked on my local test with openshift, i was able to run the notebooks in local[*] mode.

Nice catch, Mike.  Thanks!


[PackageLoading] Do not call realpath() the base URLs

I am not sure why this was done in the first place. This causes issues when
local packages are symlink.
-- rdar://problem/30119928 Handle /tmp paths being normalized into
/private/tmp on macOS

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


FixResolving Task Status Discrepancies in Query Execution Records

What problem does this PR solve?
When querying the execution records of tasks, the current system has two sources that may describe task status:
Task status in the scheduling module: Represents the processing status of tasks within the scheduling system.
Task status in LoadManager: Specifically refers to the status of Insert-type tasks, containing more detailed execution information.
When the data in the LoadManager is deleted or expires, the completion status will revert to the task status from the scheduling system, which could cause confusion. Therefore, consistency needs to be maintained.
Since these two sources may have inconsistencies in their status information, they can easily cause confusion for users and developers.
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall


Killed when parsing file with an empty automaton

I am using the python binding, libmata version 1.6.9.
Running the following code results in the program being killed due to high memory consumption.
from libmata import alphabets, parser
alpha = alphabets.OnTheFlyAlphabet.from_symbol_map({})
mata_nfa = parser.from_mata("path/to/empty.mata", alpha)

The contents of the file are:
@NFA
%Alphabet 0 1
%Initial
%Final

The same happens for different alphabets.

Thank you for reporting the issue. The issue should be now fixed in Mata 1.6.12. Please, let us know if you encounter this issue again.


✨ [WIP] Remove the Resolved status condition

Description
Fixes #1293
Reviewer Checklist

[x] API Go Documentation
[x] Tests: Unit Tests (and E2E Tests, if appropriate)
[x] Comprehensive Commit Messages
[x] Links to related GitHub Issue(s)

@yashoza19 The PR is still work in progress. Can you please remove it ?


Transformer for regression/time series

Hey guys, thanks for building such a nice GitHub repo. I am working on a problem where I am having tabular data having more than 90 features and one target and all the features are in integers (continuous). I want to use pre-trained BERT, GPT2 but when it comes to the tokenizer the tokenizer is expecting the input in the text format. I can change the integer data in the text format like this;
original_data = [1,2,3,4,5,…,94]
transformed_data = [“1 2 3 4 5 …94”]

Now if I pass the transformed_data to the tokenizer then surely it will work but I wanna know if someone tried to use transformers for this purpose and if yes, then what was the outcome, and how did the results look like?
How can I use huggingface transformer library for this purpose all the tokenizers are trained for the text data so I am kinda lost. Any help will be appreciated. Thanks

@HasnainKhanNiazi i think the latest paper i saw was https://arxiv.org/abs/2211.14730

@lucidrains thanks for your answer, I read the paper but the problem with the paper is that they are using the autoregressive decoder where the input of the decoder is always having the y values in the input which needs to be predicted but in my case, I don't want to pass the true values as in production, I will not be having y values so I padded the y values with zero in inference but results don't look promising. That's why I wanted to use pre-trained models for this problem.

@HasnainKhanNiazi yea, i would say just stick with xgboost, especially if you have a looming deadline and need guaranteed results
making transformers work well for tabular and time series data is still an ongoing research problem

@lucidrains thanks for your useful suggestions. There isn't any tight deadline that's why I am doing experiments with Transformers. I am closing this issue but if anyone else tried to use Transformer for this purpose then please do comment. Thanks


Initial version of infer-schema as cli subcommand

Factor infer schema related code into own crate
Implement cli subcommand

Fix #437
The current version should be seen as proof of concept to start discussion about the implementation. I think before merging

[x] CLI integration tests
[x] support for non default schemas
[x] filtering tables

should be added

Moving this to a top-level comment so it doesn't get lost. In reply to https://github.com/diesel-rs/diesel/pull/577#discussion_r96125450:
Orrrr, rustfmt! 😞 Well, I guess we need to define our own formatting, then. You encoded some rules implicitly. Can you document them? I'd start with:

Seeing { increases indentation level
Seeing } decreases indentation level
Insert newline after {, }, ,, and ;
Don't put spaces:

between ident and !,
between path segments and ::
after ( and before )
before ,

Insert spaces

around arrows (->, and probably also =>)
after keywords like mod, and use

I'd really like to put this whole formatting business into its own module, encode the rules as well documented as possible and add a bunch of tests.

@killercup I've updated the pull request. Now all planed features are implemented. Any idea why the Travis is failing?

thread panicked while panicking. aborting.

Very likely that this happen because you unwrap error in Drop. I encounter with exactly such error some days ago.

@killercup I've implemented a other variant for the error handling.

All tests are passing on beta now. There are failures on nightly, but they are seaming unrelated to this pull request.
If there are no more issues regarding the code style I think this should be ready to be merged.

Thanks, @weiznich!
The nightly failures are from compile-tests, and we need to fix them. It seems to me that either some compiler messages changed because of changes by use (unlikely) or by changes in a new nightly (with nicer/other error messages). I'll have a look this evening.

As promised, I made a few more changes to the print-schema/inference code: https://github.com/weiznich/diesel/pull/2
The most important abstraction is are the new TableData and TableDataWithTokens types that are passed around. I think this makes the code a bit clearer and easier to handle. What do you think?
I'm not sure if this gets run on Travis, but I didn't want to open a new PR on the main repo to not fork the discussion. The tests pass locally though, except for that weird thread panicked while panicking. aborting. failure I haven't investigated further.

Thanks for opening the pull request. I'll try to have a look at it tomorrow.

@killercup I've reviewed and merged your changes (Thanks again). Also I added some small tweaks. I think this should be ready for merge now.

Thanks, @weiznich. I like your changes, very nice! Travis reports some failures in some unit tests, can you have a look? There are some known problems with recent nightlies; I'll try to get it to run on a known good version.

Oh, and if you merge master into this (or rebase on master if like), the tests should pass again on nightly (#596).

@killercup I've fixed the last failing test and rebased on master. All tests are passing now. I also addressed those small style nitpicks.

Awesome! Merged! 🎉


Nuget Packages

Should Microsoft.Azure.Cosmos.Table be used going forward instead of Microsoft.Azure.CosmosDB.Table + Microsoft.Azure.Storage.Common? Are the later getting deprecated? I see that Microsoft.Azure.Cosmos.Table v1.0.0 was just published yesterday.

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

ID: ef1295f9-eab5-8918-ab99-3a20b8723805
Version Independent ID: 0e551ce1-761a-d379-7b6c-4caf49c5bbb2
Content: Get started with Azure Cosmos DB Table API using .NET Standard SDK
Content Source: articles/cosmos-db/tutorial-develop-table-dotnet.md
Service: cosmos-db
GitHub Login: @wmengmsft
Microsoft Alias: wmeng

Thank you for the question! We are investigating and will update you shortly.

@olufea
Microsoft.Azure.CosmosDB.Table targets .NetFramework.
Microsoft.Azure.Cosmos.Table targets .NetStandard
Please choose as per your requirement.
If you have further questions, please comment and let us know.

Ok, however, the Cosmos.Table provides an important property (TableResult.RequestCharge), which I can't find in CosmosSB.Table. It will be great if they both provide this, as cost estimation/management an important part of migrating to CosmosDB

@olufea, we recommend that you use Microsoft.Azure.Cosmos.Table going forward. Since it targets .NET Standard, it can run in various .NET implementations.

@olufea, We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Escherichia/opaque

Fixes https://github.com/AeneasVerif/charon/issues/206
(charon-ml update still needed)

I'm asking for a few modifications, but overall it works.

Could you add a test for all the new cases you cover? That means adding charon/tests/ui/opaque_attribute.rs that contains a few cases like the atribute on a function, on a trait etc to record the output.

Hm, something is weird: the changes to charon-ml are missing but the CI is passing :thinking:

Oh sorry we skip serializing the field so there's no charon-ml change to do.


tracking supported?

i miss the tracking method to work with text, as described in drawbot://www.drawbot.com/_downloads/tracking.py
are there any plans to implement it?
or is it already implemented under a different name?

Yes, this will likely get implemented, with a bit of luck in this year.
See also #5.

ok ...


Tuttion fee payment system in Finance Module for receiving fee and activating the new user / taking fee of the old user / search using based on name / id / username

Added module to pay fee by student and register by Finance team
https://github.com/rehbarkhan/ms/issues/43


chore(deps): update kubebuilder to 3.5.0

Upgrade of Kubebuilder to version 3.5.0 along with a couple of other
tools that need to be upgraded.
This upgrade will allow to work with multi-group APIs in the future
if we need it and also to new comers that know kubebuilder 3.x to
join to the project in the future.
Closes #482
Signed-off-by: Jonathan Gonzalez V<EMAIL_ADDRESS>

/test

/ok-to-merge


Disable SDPA Attention for gpt-bigcode model

What does this PR do?
Disable SDPA attention until gpt-bigcode model support FusedSDPA attention.

@jiminha It is ok to disable SDPA attention if it is causing problems. But I see a problem with using "_use_sdpa" flag here https://github.com/HabanaAI/optimum-habana-fork/blob/04b07093cdd886448d29740809f16b7933264cd3/optimum/habana/transformers/models/gpt_bigcode/modeling_gpt_bigcode.py#L238
This creates confusion because "_use_sdpa" is used in transformers to refer to torch SDPA which we should differentiate from habana SDPA.  We already fixed 1 problem related to this for Llama in this PR https://github.com/HabanaAI/optimum-habana-fork/pull/73.

Looks good to me. @libinta please go ahead and merge.

https://github.com/huggingface/optimum-habana/pull/771


feat : Added hero animation during screen navigation

I have added a hero animation in a list view separator
The image in the list view gets enlarged on the next screen with the hero animation.
Screen 1

Screen 2

The video is attached below:
https://user-images.githubusercontent.com/86414701/146632783-fa5832b6-37d1-4324-9e1e-88f0de0ee1cf.mp4


Representative message indicator

Bot could represent someone to send messages, e.g. https://github.com/MixinNetwork/supergroup.mixin.one
It should be better to distinguish the messages send by people themselves or represented by a bot.

Done in 0.27.2


use Hash.fetch when accessing version data in helm

Hash.fetch will raise an exception if the key is not present in the
Hash. The current method which uses bracket accessors will silently fail
and return nil (which is emitted as an empty string) if the key does not
exist.
Description

Related issues/PRs

fixes

documents
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit e5a55b6b92373c0fec9405546dbde9e7f70ae3a6
https://deploy-preview-2682--calico.netlify.com


concept2-utility

The Concept2 Utility is free software that allows you to update the firmware on your Performance Monitor and manage the workout data on your USB flash drive or Concept2 LogCard. You can also upload your workout data to your Online Logbook at concept2.com.

After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked the cask was not already refused in closed issues.
[ ] Checked the cask is submitted to the correct repo.

Please submit this to caskroom/drivers


No Requests Sent At All

I used sandbox/client.html in the repository:
Request
{
  "url": "http://www.baidu.com"
  "params": {},
  "method": "GET",
  "headers": {}
}

Response
No Data

And no request is seen in the Chrome Developer Tools.

Also every branch shows "build:error".
What do these mean?

If you are playing with sandbox make sure you run node ./sandbox/server then just open http://<IP_ADDRESS>:3000/ in your browser. This will run sandbox/client.html with the proper dependencies in place. You may also run node ./sandbox/client (with the server running) to test the node client.
The build error is coming from a failing test on Travis CI. I am currently working on sorting that out.

@mzabriskie I did as you tell me but the response is always an empty string with 200 status, even if I can get them in the browser‘s address bar using the same url.

@mzabriskie I did as you told me but I can only GET, I can't POST.

I'll take a look at it.

Can you share your full code for making the request?

@mzabriskie I made the request from webpage client.html without writing my own code here in this issue.

@DIMPLY Are you still having this problem?

Closing this due to inactivity. Please feel free to reopen if needed.


CO2 sensor data not mapped propperly

Describe the bug
The data fields provided by a CO2 sensor are not mapped propperly.
e.g. co2_batt is mapped to co2
So the co2 state always has value 0 although different values are provided by the Sensor/Wheather-Station. Details see logs below.
To Reproduce
nothing specific, simply run latest adapter version
Expected behavior
Each provided data field shall be mapped exclusively to it's exact corresponding object.
pm10_24h_co2 -> pm10_24h_co2
co2 -> co2
co2_24h -> co2_24h
co2_batt -> co2_batt
Root Cause
Inprecise reqexp in constant.js:
...
 id: 'pm10_24h_co2',
...
 wunderground: '^pm10_co2',  --> '^pm10_24h_co2'
 ecowitt: '^pm10_co2',       --> '^pm10_24h_co2'
...
 id: 'co2',
...
 wunderground: 'co2', --> '^co2$'
 ecowitt: 'co2', --> '^co2$'

Screenshots & Logfiles
...
2024-02-12 20:06:31.167	debug	Extracting value for weather.current.co2(co2_batt)
...
2024-02-12 20:06:31.165	debug	Extracting value for weather.current.co2(co2_24h)
2024-02-12 20:06:31.165	debug	Extracting value for weather.current.co2(co2)
2024-02-12 20:06:31.165	debug	Extracting value for weather.daily.pm25_24h_co2(pm25_24h_co2)
2024-02-12 20:06:31.165	debug	Extracting value for weather.current.pm25_co2(pm25_co2)
2024-02-12 20:06:31.165	debug	Extracting value for weather.current.co2(pm10_24h_co2)
2024-02-12 20:06:31.165	debug	Extracting value for weather.current.pm10_co2(pm10_co2)
2024-02-12 20:06:31.164	debug	Extracting value for weather.current.humi_co2(humi_co2)
2024-02-12 20:06:31.164	debug	Extracting value for weather.current.tf_co2(tf_co2)
...
2024-02-12 20:06:31.153	debug	Listener received EW update: {"PASSKEY":"BDE1491A73DF1A2A3779070D90D8F929","stationtype":"EasyWeatherV1.5.9","dateutc":"2024-02-12 19:06:28","tempinf":"72.1","humidityin":"44","baromrelin":"29.811","baromabsin":"28.412","tempf":"43.3","humidity":"90","winddir":"92","winddir_avg10m":"123","windspeedmph":"1.3","windspdmph_avg10m":"0.9","windgustmph":"2.2","maxdailygust":"20.6","rainratein":"0.000","eventrainin":"0.000","hourlyrainin":"0.000","dailyrainin":"0.020","weeklyrainin":"0.071","monthlyrainin":"0.480","yearlyrainin":"2.543","solarradiation":"0.00","uv":"0","temp1f":"70.2","humidity1":"46","temp2f":"70.2","humidity2":"47","temp3f":"70.0","humidity3":"53","temp4f":"46.0","humidity4":"79","temp5f":"64.9","humidity5":"50","temp6f":"58.8","humidity6":"63","temp7f":"71.2","humidity7":"44","temp8f":"70.9","humidity8":"46","tf_co2":"72.7","humi_co2":"52","pm10_co2":"1.9","pm10_24h_co2":"2.0","pm25_co2":"1.9","pm25_24h_co2":"1.6","co2":"695","co2_24h":"675","wh65batt":"0","wh25batt":"0","batt1":"0","batt2":"0","batt3":"0","batt4":"0","batt5":"0","batt6":"0","batt7":"0","batt8":"0","co2_batt":"0","freq":"868M","model":"HP2550A_Pro_V1.7.2"}
2024-02-12 20:06:31.153	debug	Listener body is PASSKEY=BDE1491A73DF1A2A3779070D90D8F929&stationtype=EasyWeatherV1.5.9&dateutc=2024-02-12+19:06:28&tempinf=72.1&humidityin=44&baromrelin=29.811&baromabsin=28.412&tempf=43.3&humidity=90&winddir=92&winddir_avg10m=123&windspeedmph=1.3&windspdmph_avg10m=0.9&windgustmph=2.2&maxdailygust=20.6&rainratein=0.000&eventrainin=0.000&hourlyrainin=0.000&dailyrainin=0.020&weeklyrainin=0.071&monthlyrainin=0.480&yearlyrainin=2.543&solarradiation=0.00&uv=0&temp1f=70.2&humidity1=46&temp2f=70.2&humidity2=47&temp3f=70.0&humidity3=53&temp4f=46.0&humidity4=79&temp5f=64.9&humidity5=50&temp6f=58.8&humidity6=63&temp7f=71.2&humidity7=44&temp8f=70.9&humidity8=46&tf_co2=72.7&humi_co2=52&pm10_co2=1.9&pm10_24h_co2=2.0&pm25_co2=1.9&pm25_24h_co2=1.6&co2=695&co2_24h=675&wh65batt=0&wh25batt=0&batt1=0&batt2=0&batt3=0&batt4=0&batt5=0&batt6=0&batt7=0&batt8=0&co2_batt=0&freq=868M&model=HP2550A_Pro_V1.7.2
Versions:

Adapter version: 0.10.5
JS-Controller version: 5.0.17
Node version: v18.19.0
Operating system:
PRETTY_NAME="Raspbian GNU/Linux 11 (bullseye)"
NAME="Raspbian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

Published 0.11.4 with a fix on this to NPM

closing as no more comments.


This library uses STDERR instead of throwing exceptions when failures are encountered

Describe Design Problem
This issue seems to be pervasive through the library but can be demonstrated with the below (I use pandas_datareader which wraps this library):
import pandas_datareader.data as web

fetch = web.get_data_yahoo( "AACI",<PHONE_NUMBER>,<PHONE_NUMBER>)

It'll return successfully while producing the following output:
1 Failed download:
['AACI']: Exception("%ticker%: Data doesn't exist for startDate =<PHONE_NUMBER>, endDate =<PHONE_NUMBER>")
[*********************100%%**********************]  1 of 1 completed

The error output is printed to STDERR instead of yfinance throwing an exception for the consumer of the yfinance library to handle.  This breaks error handling and pollutes my application's output.
Essentially this means that yfinance currently hijacks STDERR for consuming applications so that it can report errors instead of throwing an exception with the same information (using exceptions allows consuming applications to determine how failures are handled or represented).
A library should almost never write to stdout and stderr for errors -- this also has very big implications for flow control and error handling in consuming logic-- whether by an application that uses this library or another library that wraps it (such as pandas_datareader which is a very visible example).
It seems to be littered all through the library, but this code specifically exemplifies the issue:
https://github.com/ranaroussi/yfinance/blob/1d31e7ca016768c5a3f820c29049d3aa25132b7d/yfinance/base.py#L1687
I think these might help clarify the reasoning here:

the errors that libraries throw are sometimes fatal to the logic of the consuming application and so need to be handled in that logic, which will expect an exception from the called function in a library to catch
In most cases, an application uses stdout+stderr for its own purposes, at its own level of abstraction to talk to the user.  STDOUT and STDERR are categorically for the application to talk, not its libraries.  For example, this current STDERR behaviour will completely break CURSES-based user interfaces.
An exception notifies a consuming developer of a problem, output notifies the user.  The user is often abstracted very far away from the library code so this is an essential practice to incorporate and it will have a huge impact on other projects to do this.  It's good to log errors, but also important to defer that function to whatever the consumer is using.
An exception can have the same data as what you're currently sending to logger, it just lets the application handle how or whether to show that to the user, or decide if it can even continue whatever its trying to do -- and that application may be only using your library for something low level in one piece of it that isn't related to the core function of the application that the developer will care about but that the user won't.

This design problem appears to have always been there, but was exacerbated by the introduction of the use of logger:
https://github.com/ranaroussi/yfinance/issues/1378

Ouch this is true - my previous issue #1378 was aimed at INFO-level and WARNING-level prints, not errors. Errors should be raised as exceptions indeed!

Some context on why exceptions were redirected to logger:

users didn't want failed fetches to abort their loops
yf.download() parallel download. Suppose one symbol suddenly delisted, that would prevent returning other symbols' good data.

Argument raise_errors was added to history() to optionally restore exception raising.

That’s great @ValueRaider thx

I completely forgot to follow up on this:
After reading these responses to what is clarification on a well understood best practice in library behaviour in February, I abandoned this library entirely and migrated to Alpha Vantage and have had no issues since.
I would suggest to any project maintainers that consume this library to consider doing the same or rewriting this project from scratch so that the clone can be used for serious projects without this type of issue, and without this type of reaction to reports of it.
Hijacking STDERR and STDOUT from a consumed library is just completely unacceptable, and the idea that maintainers on a project this visible don't know that isn't inspiring of future confidence sufficient to trust this project in my own code.
I guess this issue can be closed, I just wanted to leave a breadcrumb for any others that need to do more mature things with their code than write a proof of concept.

No need for that immaturity. If the issue is not resolved, then leave it open.


MetricsTest should clear metrics in advance to make test result stable fixes #94

Summary

Currently, MetricsTest.testMetricsLifecycleManagement fails if CompactionProcessorTest is executed first (depending on test execution order), because metrics for CompactionProcessor isn't cleaned up in any place

https://github.com/line/decaton/blob/master/processor/src/main/java/com/linecorp/decaton/processor/processors/CompactionProcessor.java#L58

One possible fix might be creating enclosing class for CompactionProcessor metrics and extends AbstractMetrics and close it upon processor close
But rather, I fixed by clearing metrics in @Before to make testMetricsLifecycleManagement stable even we forget closing metrics in other classes

@kawamuray Good point.
But I found that closing CompationProcessor metrics properly requires extra work, because:

Since CompactionProcessor's metrics are registered per every task to tag with subscriptionId, we can't use AbstractMetrics
Possible solution is to make CompactionProcessor's constructor receives subscriptionId and pass it through ProcessorsBuilder
But it causes breaking changes, and until then, master's test keeps failing.
By the way, even CompactionProcessor's metrics are not cleaned up, it doesn't cause actual problem because it's tagged by subscriptionId (unlike other metrics (partition, thread,)), so the tag will still be valid even after rebalance

So, let's merge this PR as-is, and solve CompactionProcessor's metrics clean-up problem in another PR?


Simultaneous hardware counters

Allow to measure more than once PAPI counter at a time.
This feature has not been considered before because macros inherit from PolyBench code.
Main reason to implement this: efficiency

I think I am going to close this issue, since it is very related with the multiplexing feature.


BufferPool improvement

BufferPool should use a ConcurrentStack instead of a ConcurrentQueue.  When implementing object caches its better to use LIFO than FIFO for numerous reasons.
However... the class is implemented so simplistically it may be worth looking into existing solutions to this problem.  If there are 2 API users of the class that happen to be interleaving requests for different buffer sizes you're probably better off with no pooling at all because it just pops the last buffer and throws it away if it doesn't match the requested length.  If the assumption is that all callers are using the same array size, then the pool should probably take that as a argument to the ctor and the Get/Put methods don't have to check size at all.
Existing stable code that I've used with success to solve a similar problem is Microsoft.IO.RecyclableMemoryStreamManager .  The interface is a via a MemoryStream but you may be able to just use the underlying byte arrays.
Cheers!
edit: CustomBinaryReader.ResizeBuffer() is not returning the resized buffer to the pool.  if you are going to stick with pooling perhaps the Resize() call should be on the pool so it can recover the smaller array.

I agree that using Stack would better take advantage of cache hit in CPU. However the logic assumes that if user changes buffer size after proxy started to run, then outdated buffers will be evicted quickly. This won't happen in a stack implementation until it happens to become empty. But I think the overhead overall would be the same in this case since the number of invalid buffers to throw away would be same. But stack does have an advantage with recently used buffers reused. So I think we should switch it to stack.
I think we overlooked the problem when multiple instances of proxy shares the same buffer pool. It should not have been a static instance.

Maybe we can use this class:
https://www.nuget.org/packages/System.Buffers/4.4.0
And consider dropping the .NET 4.5 compatibility, since it is not supported anymore (since more than 2 years ago):
https://blogs.msdn.microsoft.com/dotnet/2015/12/09/support-ending-for-the-net-framework-4-4-5-and-4-5-1/
4.5.2 and 4.6 is still supported, but I think it would be much easier for us to require 4.6.1, since it is .net standard 2.0 compatible. Maybe with a major version number increase.

I mean in TWP, not only in StreamExtended.

I am fine with it.

i just realized i acutally do have a use case for requiring 4.5.2.  i've got a deployment scenario involving machines running on a default installation of Windows 7 which can't be upgraded.

Lets hold this for now then. May be we can improve existing buffer pool until some time later people stop using Windows 7. :-)
Meanwhile if its possible we can integrate System.Buffers for .Net standard.

maybe just switch to ConcurrentStack for now?

Yeah. We can create a beta version with the change to Stack.
In MS implementation of array pool in above nuget package, they are using an array of stacks for each specific buffer size. Its just a more robust solution. Our implementation would not be that different than MS implemention if we use ConcurrentStack, except that if user is constantly changing buffer size then a 2D array stack implementation of MS would do better with separate stacks for each buffer size.
https://github.com/dotnet/corefx/blob/master/src/System.Buffers/src/System/Buffers/DefaultArrayPoolBucket.cs

I have exposed the buffer pool to the API user with the IBufferPool interface. If users want to use custom buffer pools they can always provide one by implementing the interface. If not they can just pass in DefaultBufferPool . So buffer pool implementations can be customized by API user in Titanium Web Proxy as well with this change.

@jgilbert2017 Do you still need .net 4.5.2 support?

@honfika no, i don't (thanks for asking).


The sum of browsers audience coverage in 29 regions greater than 100%

Hi!
I'm involved in updating the browserslist website with caniuse audience coverage data by regions.
We noticed that in 29 region audience coverage is more than 100% (Antarctic 103.9%, North Korea 105% etc.)
I created demo with latest caniuse-db version from CDN. Large values are shown in bold
https://codepen.io/nibach/pen/QWmYMOP
Why is this happening? Can we fix it?

We found this issue during Browserslist website development

Thanks for pointing this out, bit hard to tell what caused this, possibly an issue with the source StatCounter data at the beginning of the month. I've regenerated the files and all percentages are < 100% now. Will keep an eye out again next month.

Relevant thread by @developit: https://twitter.com/_developit/status/1367169135977902082

@Fyrd: Thanks for pointing this out, bit hard to tell what caused this, possibly an issue with the source StatCounter data at the beginning of the month. I've regenerated the files and all percentages are < 100% now. Will keep an eye out again next month.

Thanks for the answer! Now the stats look normal again.

@simevidas: Relevant thread by @developit

Thank you for sharing other issues in StatCounter statistics. It seems that one should be able to check these values.

Found the problem after updating the database.
At the beginning of the month, Chome for Android 104 in Russia was used by 29.4% of users. Today 43.1%.
I prepared a demo to find problems in stats.
Before. Previous version (1.0.30001379) with stats bug
https://preview-450---browserslist-vw3nqlhdtq-ue.a.run.app/?q=>+0.5%25&region=RU
After. Current version (1.0.30001380)
https://browsersl.ist/?q=>+0.5%25&region=RU

@Fyrd: Will keep an eye out again next month.

Thank you! This is very important for audience coverage queries in browserslist.
I think that we need to prepare tests (validations) to check regional usage data. Something like diffcheker between previous and current StatCounter statistics.
Ideas for tracking:

The sum of browsers audience coverage >100%
Changing in browser version usage statistics by >10% per month

No issues were found for the August import so I'm going to close this ticket. Will continue monitoring in the future.


feat:helm upgrade

Description

Added helm upgrade feature in helm test

HI @shijinrajbosch,
this looks good to me and would also fix #165. Could you please resolve the merge conflicts, so we can merge this?

Hi @SebastianBezold ,
Thanks for the input. I have resolved the merge conflict.

Hi @SebastianBezold ,
Thanks for the input. I have resolved the merge conflict.

Hi @SebastianBezold  ,
Thanks for the input. I have resolved the merge conflict.


💡 [ADD]: ANIMATED CONFIRMATION BUTTON

Details

This issue is about adding  a new animated confirmation button
The confirmation is conformed once the user clicks the button
This button has animation for both states during hover & after it's click

Screenshots
https://github.com/Spyware007/Animating-Buttons/assets/92252895/c898b138-f2a9-457e-8c68-2f7e636536cb
Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

I would like to work on this project. Please assign it to me

Please check #1346


Failed to load slider when using on Dynamic Component

I am loading component dynamically like <component :is="currentView"></component> but slider range left and right stuck to 0.
I also use :real-time property but doesn't worked. How can i reinitialize vueslider?
<vue-slider ref="slider1" v-bind="price" v-model="price.value" :real-time="true"></vue-slider>

this.$ref.slider1.refresh() can refresh the component.

Error in mounted hook: "TypeError: Cannot read property 'slider1' of undefined"

oops, I am careless, it should be this.$refs.slider.refresh().
https://vuejs.org/v2/guide/components.html#Child-Component-Refs

This should be inside mounted()?

After the component is rendered, then call this method.

No man not working

You can see here can give you inspiration: https://jsfiddle.net/9pp3dqkq/54/

Dude onclick of refresh is not going to 100

I am using getuikit and showing slider inside dropdown but dropdown failed to refresh vue slider

Maybe you can try to listen for the shown event and call refresh later to refresh the component.

UIkit.dropdown(element).shown({     this.$refs.daysSlider.refresh();      ^ });
Syntax Error: this is a reserved word

I also have updated your jsfiddle to work with dropdown of uikit
Snippets

https://jsfiddle.net/9pp3dqkq/94/

thanks its working

what if i want to do it like UIkit.dropdown(element).shown({ this.$refs.daysSlider.refresh(); });

That's not the UIkit binding of events.
look it: https://getuikit.com/docs/javascript#programmatic-use
$('#offcanvas').on('show', function () {
    // do something
});


Update applicationview_tryresizeview_229045332.md

Fix syntax error resulting in markdown asterisk visible instead of bolding "true"

Hi @seanmcr, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


go-graft: 0.2.11 -> 0.2.13

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mzz2017/gg/releases.
meta.description for go-graft is: "A command-line tool for one-click proxy in your research and development without installing v2ray or anything else"
meta.homepage for go-graft is: "https://github.com/mzz2017/gg"
meta.changelog for go-graft is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.12 with grep in /nix/store/fs2s2ciaga4n8m1hsaz5w5i4kjkryfxg-go-graft-0.2.12
found 0.2.12 in filename of file in /nix/store/fs2s2ciaga4n8m1hsaz5w5i4kjkryfxg-go-graft-0.2.12

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
go-graft

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fs2s2ciaga4n8m1hsaz5w5i4kjkryfxg-go-graft-0.2.12 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A go-graft https://github.com/r-ryantm/nixpkgs/archive/dc91fe4d4538ec4a9aea96044dba3e68bb750e7f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fs2s2ciaga4n8m1hsaz5w5i4kjkryfxg-go-graft-0.2.12
ls -la /nix/store/fs2s2ciaga4n8m1hsaz5w5i4kjkryfxg-go-graft-0.2.12/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

go-graft

Maintainer pings
cc @XYenon for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mzz2017/gg/releases.
meta.description for go-graft is: "A command-line tool for one-click proxy in your research and development without installing v2ray or anything else"
meta.homepage for go-graft is: "https://github.com/mzz2017/gg"
meta.changelog for go-graft is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.13 with grep in /nix/store/x1i1nf2x3sgsbbf1dxplnjmakhd9bis2-go-graft-0.2.13
found 0.2.13 in filename of file in /nix/store/x1i1nf2x3sgsbbf1dxplnjmakhd9bis2-go-graft-0.2.13

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
go-graft

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/x1i1nf2x3sgsbbf1dxplnjmakhd9bis2-go-graft-0.2.13 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A go-graft https://github.com/r-ryantm/nixpkgs/archive/6d16e066b14bf96d24d26229650a4e278faa6757.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/x1i1nf2x3sgsbbf1dxplnjmakhd9bis2-go-graft-0.2.13
ls -la /nix/store/x1i1nf2x3sgsbbf1dxplnjmakhd9bis2-go-graft-0.2.13/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

go-graft

Maintainer pings
cc @XYenon for testing.

Result of nixpkgs-review pr 193313 run on x86_64-linux 1

1 package built:

go-graft


ISPN-10535 productize server readme

https://issues.jboss.org/browse/ISPN-10535
It would be ideal to productize the server readme. It contains reference information that should be maintained in source rather than duplicated in docs.
To productize, we need to convert from md to adoc. From there I can sync the adoc into the downstream repo and replace "Infinispan" and version info with attributes then include it in the product docs.

Merged, thanks


SC-6104: Update React Build Args

Scope of changes
Updates the GDS configuration with the new Auth0 react build args.
Type of change

[ ] bug fix
[ ] new feature
[ ] documentation
[x] other (technical debt and configuration)

Acceptance criteria

[x] ensure no private information has been committed
[x] ensure all template variables are correct
[x] ensure variables are spelled correctly
[x] ensure no variables are missing

Author checklist

[ ] ~I have manually tested the change and/or added automation in the form of unit tests or integration tests~
[ ]  ~I have updated the dependencies list~
[ ]  ~I have recompiled and included new protocol buffers to reflect changes I made~
[ ]  ~I have added new test fixtures as needed to support added tests~
[x]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report

Merging #567 (ef333a2) into main (bf53113) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head ef333a2 differs from pull request most recent head 2368866. Consider uploading reports for the commit 2368866 to get more accurate results

@@           Coverage Diff           @@
##             main     #567   +/-   ##
=======================================
  Coverage   45.07%   45.07%
=======================================
  Files         489      489
  Lines       14153    14153
  Branches     1146     1146
=======================================
  Hits         6380     6380
  Misses       6793     6793
  Partials      980      980

Continue to review full report at Codecov.

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


Associate vscode-gitlens#1135 with microsoft/vscode#37947

The ticket #1135 is closed for conversation but It need to have reference to currently open microsoft/vscode issue #37947.

I added the comment to the original issue


add topics

I suggest adding the topics performance, benchmark, benchmarks in the About section at https://github.com/attractivechaos/plb2

Added. Thanks for the suggestion.


Cleanup handlers and test handlers

[x] Test 404 route tests.
[x] Test 500 route tests.

You might need to add another sample route to the API to throw the bad response e.g.
// Bad route
app.get('/bad', (req, res, next) => {
  next('error message to send')
})

Done and merged.


feat: Configure which fields to add to index

Describe your changes
Secondary indexes are now configurable via the schema. Adding index: true to a top level field will add that field to the secondary index. The schema rules have been updated to make sure that only top level fields can be indexed.
How best to test these changes
All tests should pass. I've updated some of the update and delete test's so that the queries will use the secondary index
Issue ticket number and link
Fixes #875

Codecov Report
Patch coverage: 54.34% and project coverage change: -1.05 :warning:

Comparison is base (40f2fdd) 30.34% compared to head (9d76b8a) 29.30%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #889      +/-   ##
==========================================
- Coverage   30.34%   29.30%   -1.05%
==========================================
  Files         176      176
  Lines       19905    19958      +53
==========================================
- Hits         6041     5849     -192
- Misses      13154    13425     +271
+ Partials      710      684      -26

Impacted Files
Coverage Δ

query/filter/logical.go
0.00% <0.00%> (ø)

server/services/v1/database/search_runner.go
0.00% <0.00%> (ø)

...ver/services/v1/database/secondary_index_reader.go
0.00% <0.00%> (-58.34%)
:arrow_down:

server/services/v1/search/runner.go
0.84% <0.00%> (ø)

schema/rules.go
21.90% <30.43%> (+1.42%)
:arrow_up:

server/services/v1/database/base_runner.go
55.89% <50.00%> (+1.02%)
:arrow_up:

server/services/v1/database/secondary_indexer.go
41.22% <57.14%> (-17.41%)
:arrow_down:

query/filter/filter.go
31.11% <66.66%> (+4.88%)
:arrow_up:

schema/fields.go
67.48% <75.00%> (+0.24%)
:arrow_up:

query/filter/selector.go
18.55% <100.00%> (-5.16%)
:arrow_down:

... and 18 more

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

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


Allow recipe to specify IO_Generator_Args

Add another optional property to the test definition that lets the user specify additional arguments for the IO generator per-test.  For example, the following test would run a 4k random read test with IOs aligned to 8k boundaries:
test(
description         => "4k Random Reads",
write_percentage    => 0,
access_pattern      => 'random',
block_size          => '4K',
queue_depth         => 1,
warmup_time         => 60,
run_time            => 3600,
io_gen_args         => '-r8k ',
);
Currently, the only way to loop through multiple values for a generator's parameter is to build a script that calls storscore with different values for --io_generator_args.  This makes it difficult to name each test and manage their results as a set.

I think this is easily doable, but it breaks the abstraction around the IO generator -- it would lead to recipes that only work with DiskSpd and not SQLIO, for example.  (Or perhaps more realistically, if someone ports StorScore to linux and adds fio support).  Maybe that's fine.

It might help to have a flag to explicitly mention which IO_generator is needed for the recipe.  Something like
...
io_gen_args => "-r8k ",
io_gen => diskspd
...
This way the code could fail, saying "IO generator inconsistency" instead of whatever the IO generator does with a different generator's args.
I'll have to think about how the IO-gen on the command line vs. in the recipe will work...

You could just wrap the test in "if( $cmd_line->io_generator eq 'diskspd' )" and it would be silently ignored in the other cases.  I think we once had some recipes that did this.

ah, I like it!


Making levels are not dependent on only C and SDL

Making levels are currently also dependent on xqilla, maybe the game could integrate an XML-parser directly so that we remove that 3rd party dependency.

@TheKnarf I think it's kinda limiting. What if we will need to generate levels not only from SVG? I like the idea of a native level format that you can generate from pretty much anything.

While I don't disagree on having some sort of native level format, couldn't that be a flavor of XML? A lot of tools can output XML.
Tiled is a populare map editor for 2D games that also uses an XML format. Converting from SVG to Tiled-xml should be fairly straight forvard.
Or one could invent a more fitting xml-format for your engine, and convert both SVG, Tiled-xml and other format into it.

S-expression would be preferred, especially now that you have the parser implemented.
But can't I wish to have it both ways? To have my cake and eat it, too, so as to speak.

After #451 the game does not depend on xqilla anymore


Windows UTF-8 encoding

A student posted on the forums that they were getting the following UNICODE error when checking TMA02 with allowed on Windows resulting in the file not being checked.
UNICODE ERROR: 'charmap' codec can't decode byte 0x9d in position 96375: character maps to <undefined>

I was able to replicate the Error on Windows 10 running in Virtualbox using the same TMA02 .ipynb file.
After changing  line 500 in allowed.py from
 with Path(filename).open() as file:

to
 with Path(filename).open(encoding='utf-8') as file:

the error was no longer present and the notebook was checked.
One can only assume that windows is opening the file with an encoding other than UTF-8 ?

From the Python3.10 interpreter running in Windows 10 Powershell:
>>> import sys
>>> import locale
>>> print(sys.getdefaultencoding())
utf-8
>>> print(locale.getpreferredencoding())
cp1252

Thanks for this, but forcing a fixed encoding may lead to problems for other files. Can you try without encoding but instead errors='surrogateescape'? I got it from here.
If that fixes the issue, i.e. finds the same errors as with UTF-8 encoding, then please make a PR.

I already created a branch for this.

I can confirm that on Windows 10  with the errors='surrogateescape' argument added to .open(), there were no UNICODE errors reported. This was using the same TMA02 .ipynb file that caused the original errors to be shown.

Good to know that the Unicode errors go away, but does it report the same disallowed constructs as with utf-8? We don't want the surrogates to lead to a different list of reported violations...

@mwermelinger  the last commit seems to have changed the way the dev environment is working and I cannot seem to test allowed in the same way.
When using the normal make install then make test command the dev environment does not seem to install ipython and pytype Is there a different way to set up the environment now?
Also to note is that pylint does not like the .open() method to be used without an explicit encoding. Its giving me the dreaded yellow squiggly line ! :smile:

Thanks, poetry does need extra arguments to set up dev environment now. I used poetry install --extras all which seemed to do the job (although I don't know if that is the best way). The tests are reporting no strange behaviors with the errors='surrogateescape' argument.
See unspecified encoding for details on the pylint warning.

After reading pylint and PEP 597, I guess it is best to explicitly use utf-8 and surrogate any error characters. Thanks for the detective work! (And thanks to Steve for reporting the issue.)


Added add sale and edit sale command parsers. Basic GUI layout done without image icons yet.

Using commands still does not show up on sale page yet.

@eugenelim97 can help resolve conflict? Which one did you use?

ok resolved


How to express project dependencies in Traversal SDK?

In a Visual Studio solution, I can explicitly express dependencies between projects. This is useful when projects implicitly depend on the output of other projects, so that Visual Studio can calculate a compatible build order.
How can I describe project dependencies in a Microsoft.Build.Traversal SDK proj?

Unfortunately, build ordering in a solution file is not a very good best practice.  Since the ordering is only respected when building the solution file, this means that when you build projects instead, the build ordering is not guaranteed.  The better solution is to declare dependencies directly in each project.  This will have your build order respected in Visual Studio and the command-line, whether you're building a solution or a project.
Are you able to add <ProjectReference /> items in the projects to explicitly declare a dependency?  You can set metadata like ReferenceOutputAssembly to not actually pass the reference to the compiler and just have it declared as a dependency:
<ItemGroup>
  <!-- Declare a build dependency on a project but don't actually depend on the assembly at compile time -->
  <ProjectReference Include="..\SomeDependency\SomeDependency.csproj" ReferenceOutputAssembly="false" />
</ItemGroup>

If you run into target framework incompatibility, you can also set SkipGetTargetFrameworkProperties=false  to bypass the check and just have the dependency declared for build ordering.


Support ESM-based functions

We have a number of dependencies that are migrating from commonjs to ESM. ESM is supported in Node 12 and it can import CommonJS modules, but not vice versa. We should evaluate what it takes to deploy an example function written using ESM to the four supported clouds.
Chance is that the webpacking step can take care of some of the complications, but not all of them.

note, that esm is still in the experimental state for node 12. even for node 14:

As of Node.js 14 there is no longer this warning when using ESM in Node.js. However, the ESM implementation in Node.js remains experimental. As per our stability index: “The feature is not subject to Semantic Versioning rules. Non-backward compatible changes or removal may occur in any future release.” Users should be cautious when using the feature in production environments.

So running pure ESM code might be "risky". I think a better alternative is to use babel+webpack. this combination is tested very well.

see https://github.com/adobe/helix-home/discussions/193

esm+webpack solution works with #229


Listening Mode's settings' description text cut off on iPhone devices

Description:
On iPhones with smaller screen sizes (height wise) or when rotating the device to landscape, the description text for the Listening Mode's settings is cut off at the bottom. The screen is not scrollable, so you cannot read the rest of the text.
Steps:
One a smaller iPhone, or while any iPhone is in landscape:

Navigate to Settings > Listening Mode
Note the description text under the Hey Vocable shortcut setting

Expected behavior:
All of the description text should be visible; either by scrolling or the text resizing dynamically.
Actual behavior:
The description text is cut off. Since the screen is not scrollable there is no way to read it.
Screenshots:

Device Information:

Device: Any; but specifically iPhone 8 Plus is cut off in portrait; any iPhone will be cut off in landscape
OS: any

This fix looks good! 💸  💸


Load a template, modify it, and then save it

The documentation provided is not clear how one can proceed through reading, editing and saving a template to a new file.
This feature was present and working correctly in version 0.3

Is this problem solved?

I've tried to use it, but my Excelfiles contains several charts and PHPExcel have some problems with that. So no success from my point.

@wiesson Did you try to enable the charts in the config export.includeCharts ?

Is this working now? When I run download or export I get the updated values.

Four years later... I'm trying to do exactly this, now in version 3.1. Can't seem to find out how. The link to the documentation that @MaatwebsiteSupport provided above doesn't seem to work any longer. Anyone got any working pointers?


chore: print agent logs in addition to sending over grpc

Had to also introduce --log-fd mechanism for agents, so we don't have a spaghetti of Log message logic in both server and agent (e.g. we were capturing USER logs through logged_io in server, but BRIDGE ones were in agent), which was not allowing this PR to be just  a oneliner print(log).
Note that --log-fd mechanism doesn't change anything for the user, same messages are getting sent out, but now from only from server instead of from both server (this is where we were sending out USER logs) and agent (this is where we were sending BRIDGE logs).

Some more test failured, looking into that.

Works in https://github.com/fal-ai/isolate-cloud/pull/1765 , but there are some fixes for previous changes that are already merged. So merging this for now.


Allow server to serve multiple channels easily

The lower-level axtix implementation allows a concept of “channels” each with independent state, but the ServerBuilder does not expose a way to do this. There should be an easy way to allow channel creation from the server.

Accomplished with Jamsocket integration.


Fix a crash when trying to run Godot debugger on a release build.

The GDScriptLanguage::enter_function is wrapped in #ifdef DEBUG but the exit_function is not, resulting in a stack underflow error.

Thanks!

Cherry picked into 3.0.3


[v2-rc8] Type: Object isn't updated

Astro Class
TempClass = Astro.Class.create({
  name: 'TempClass',
  collection: new Mongo.Collection('tempclass'),
  fields: {
    _id: {type: String, index: 1},
    name: {type: String},
    options: {type: Object, default() { return {}; }
  }
});

Usage
var myTempObj = new TempClass({name: 'Testing', options: {target: true}});
myTempObj.save(); // Everything works perfectly.

myTempObj = new TempClass({_id: 'someOldId', name: 'Testing'});
myTempObj._isNew = false; // To make astro update old document
myTempObj.save(); // Everything else is saved properly but options remains unchanged.

Yes in deed there is some bug. I will take a look at it.

Ok bug fixed. If you really want have this feature right now then just clone repository. I'm not going to release next RC version. I will release 2.0 before the end of April.

I'll patch it locally thanks alot.

No problem


Phasing out use of n1 machines in favor of n2 machines on GKE

I've researched to conclude that n2 machines seems like an almost pure win with:

n2-highmem is more cost effective per RAM
n2-highmem is more cost effective per CPU performance
n2-highmem has 1:8 CPU:RAM ratio, while the n1 stands out with the unique ratio of 1:6.5

Proposal

To change anything we touch anyhow to n2 from n1
Exception: we need to retain n1 nodes for GPU machines as they may only come as n1 variants
[x] To update templates so new clusters are bootstrapped with n2 machines by default

Current use of n1 machines

[x] 2i2c-uk core nodes
[x] 2i2c-uk user nodes
[ ] cloudbank core nodes
[ ] cloudbank user nodes
[x] cloudbank dask nodes (*use only one node pool with n2-highmem-16)
[x] m2lines core nodes
[ ] m2lines user nodes
[x] m2lines dask nodes (*use only one node pool with n2-highmem-16)
[x] meom-ige core nodes
[ ] meom-ige user nodes
[x] meom-ige dask nodes (*use only one node pool with n2-highmem-16)
[x] pangeo-hubs core nodes
[ ] pangeo-hubs user nodes
[x] pangeo-hubs dask nodes (*use only one node pool with n2-highmem-16)
[x] two-eye-two-see core nodes
[x] two-eye-two-see user nodes
[x] two-eye-two-see dask nodes (*use only one node pool with n2-highmem-16)
[x] two-eye-two-see neurohackademy nodes

*In https://github.com/2i2c-org/infrastructure/issues/2687 I'm proposing we standardize on n2-highmem-16 nodes for dask workers.
Related
The following issues relate to this smaller and more tightly scoped issue.

#2687
#2212
#2121

Not much changes remains, closing in favor of an issue that is cloud provider agnostic as I want to convey the same strategy for all cloud providers.


Fix race condition in helm list

supercedes #4608 by writing a unit test to confirm the change fixes the issue.

Thanks for adding the test and getting it merged.
It looks like -race isn't enabled in the circleci  test step. Without it, the test may not detect the bug. (The build log reports it is running HELM_HOME=/no/such/dir go test  -run . ./cmd/... ./pkg/...)
I see make test and make docker-test both enable -race, is there any reason it isn't in CI?

In Circle, each of the test steps are split into separate containers calling make test-unit directly, whereas it looks like the -race flag is only enabled when you run make test. I don't think there's any technical reason why it's not enabled in CI.


Please clarify 'Guest invite settings' when Entitlement Management is used

Hi,
please clarify in the docs how the 'Guest Invitation Settings' in the Azure AD External Collaboration Settings were used when using entitlement management to invite guests.
From my experience everybody is able to approve new guest invitations (as long as he is defined as an approver in a AP Policy) as long as the settings are not more restrictive than 'Only users assigned to specific admin roles can invite guest users' (This limitation seems not to have any relevance for the access package assignment request handling. Also users not in the guest inviter role can approve new guests ) .
Only when the option 'No one in the organization can invite guest users including admins' is set, the Access Request is not delivered with an error code specific for this setting.
This is not really straight forward.

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

ID: 7bd8b41a-26fd-936b-5773-3eb158de253a
Version Independent ID: eac9538f-02c5-efec-77ed-1059777c294d
Content: Govern access for external users in Azure AD entitlement management - Azure Active Directory
Content Source: articles/active-directory/governance/entitlement-management-external-users.md
Service: active-directory
Sub-service: compliance
GitHub Login: @ajburnle
Microsoft Alias: ajburnle

@J-a-k-o-b
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

#please-close


Update limits-and-config.md

Updating concurrency behavior.

@llistephanie : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@llistephanie : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MSFTMan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MSFTMan : Thanks for your contribution! The author(s) have been notified to review your proposed change.


feat: add missed variables, replace dash in resource names by underscore

Hey @SweetOps,
thanks for taking the time to add this and apologies for the very long wait. I'm merging this PR to make sure the HCL 1.0 version of this is up to date.
Later this week, a 2.0 rebuild will be released (pending approval of #14) - if you're on Terraform 0.12.0 or later, and you feel like testing. let me know!


Student life

Added info for STUDS:

Updated contact email address

Changed details (STUDS are maximum 30 students, and other students can sign up to participate in the events)

Added the sign up form

Updated README for npm install (to install node_modules)

Changed hover color of the sidebar to bright yellow

Currently unreadable because all the previous commits get in the way. Can you merge main into the branch? You might need to reapply your changes, I suppose they are not too large.


Current visualization does not support multiple users

added dynamic request proxying. TODOs

[ ] dynamically assign port numbers for tensorboard process
[ ] record tensorboard process port number in job results
[ ] check when the tensorboard server is up before connecting to it


Install v17 redistributables

The latest VC redistributables support VS2015 through VS2022, so we only need to install one version.

Codecov Report

Merging #2943 (7f61d1b) into main (3adb2fe) will increase coverage by 0.97%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2943      +/-   ##
==========================================
+ Coverage   28.51%   29.49%   +0.97%
==========================================
  Files         125      272     +147
  Lines       13602    30519   +16917
==========================================
+ Hits         3879     9001    +5122
- Misses       9723    21518   +11795

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

Validated in check-pr


netty epoll native throw java.lang.IllegalArgumentException in linux(debian)

Netty version: 4.1.5.Final
When start application, the following exception is thrown from netty
Caused by: java.lang.IllegalArgumentException
at sun.misc.Unsafe.allocateMemory(Native Method)
at io.netty.util.internal.PlatformDependent0.allocateMemory(PlatformDependent0.java:627)
at io.netty.util.internal.PlatformDependent.allocateMemory(PlatformDependent.java:262)
at io.netty.channel.epoll.IovArray.<init>(IovArray.java:64)
at io.netty.channel.epoll.EpollEventLoop.<init>(EpollEventLoop.java:62)
at io.netty.channel.epoll.EpollEventLoopGroup.newChild(EpollEventLoopGroup.java:130)
at io.netty.channel.epoll.EpollEventLoopGroup.newChild(EpollEventLoopGroup.java:35)
at io.netty.util.concurrent.MultithreadEventExecutorGroup.<init>(MultithreadEventExecutorGroup.java:84)

The reason is caused by another library which contains the android-2.3.3.jar. Netty EPOLL native try to detect the system by checking the functions & class. In the PlatformDependent::isAndroid0(), because of the android jar, it will believe itself is running in android.
private static boolean isAndroid0() {
boolean android;
try {
    Class.forName("android.app.Application", false, getSystemClassLoader());
    android = true;
     ...

So the function hasUnsafe0() will wrongly believe there is no sun.misc.Unsafe.
private static boolean hasUnsafe0() {
if(isAndroid()) {
    logger.debug("sun.misc.Unsafe: unavailable (Android)");
    return false;

Because of this, the exception is thrown when it is calling the native memory allocation. Its parameter is negative value!!!

The EPOLL transport assumes that all Linux platforms will support Unsafe. The allocateMemory method has no "non unsafe" alternative ... and it looks like your platform doesn't support Unsafe. To be safe we should likely check that Unsafe is supported in Epoll.java in order for the EPOLL transport to make itself available ... @normanmaurer - WDYT?

The mentioned issue is caused by the isAndroid() check. Because it check whether there is android app class in the class path. That is why, the hasUnsafe0() return false even my application is running in debian OS.

Based upon your description I'm not sure how hasUsafe0() has any impact on the exception being thrown .. can you elaborate? Actually looking a bit deeper we do already restrict the EPOLL module's availability based upon Unsafe's availability here. So it looks like hasUnsafe() returns true, and then sun.misc.Unsafe.allocateMemory throws an unexpected exception when called (IIRC android only partially implements Unsafe).

In the PlatformDependent class, the IS_ANDROID flag is result of this function. So it check whether the class "android.app.Application" is in the class path.
private static boolean isAndroid0() {
    boolean android;
    try {
        Class.forName("android.app.Application", false, getSystemClassLoader());
        android = true;
    } catch (Throwable ignored) {
        // Failed to load the class uniquely available in Android.
        android = false;
    }

    if (android) {
        logger.debug("Platform: Android");
    }
    return android;
}

In the function  io.netty.channel.epoll.IovArray::IovArray(). It will allocate the memory with the size CAPACITY. The mentioned exception is caused by this "CAPACITY". It is negative value.
IovArray() {
    memoryAddress = PlatformDependent.allocateMemory(CAPACITY);
}

The reason for this negative value is because of the result value of   PlatformDependent.addressSize()
/** The size of an address which should be 8 for 64 bits and 4 for 32 bits. */
private static final int ADDRESS_SIZE = PlatformDependent.addressSize();

/**
 * The size of an {@code iovec} struct in bytes. This is calculated as we have 2 entries each of the size of the
 * address.
 */
private static final int IOV_SIZE = 2 * ADDRESS_SIZE;

/**
 * The needed memory to hold up to {@link Native#IOV_MAX} iov entries, where {@link Native#IOV_MAX} signified
 * the maximum number of {@code iovec} structs that can be passed to {@code writev(...)}.
 */
private static final int CAPACITY = Native.IOV_MAX * IOV_SIZE;

The PlatformDependent.addressSize() -> PlatformDependent.addressSize0(). In later function, it will call PlatformDependent.hasUnsafe(), will return -1 if it is true. While the function PlatformDependent.hasUnsafe() just return the flag HAS_UNSAFE, which is initialized by the function hasUnsafe0().
So that is why the allocation of memory will throw the exception.

As stated above if the platform is detected as android the epoll module shouldn't be available ('Epoll.isAvailable()'). You should check the result of this method before you try to use the epoll transport.

it is not android platform. it is debian, just my system get a android jar file in the class path. My point is the adroid detection is misled. Anyway, The issue is reported here to respond the recommendation in stackoverflow.


Update plugins.md (Leaflet.Signposts)

Submitting new Leaflet plugin under User Interface: Leaflet.Signposts

This is my first plugin so feeling my way as I go here.
IE: Fixed a HTML1506 Unexpected token error, believe a remaining DOM7011: The code on this page disabled back and forward caching error is not an issue.
IE10: Error here appears to be related to Turf.js code and I'm struggling to diagnose. Does this need to be fully resolved?

The errors don't come from you. Ignore them. You have only modified a docu file, they are not included in the tests.


Robustness against stochastic failure in test CI

I note that #314 got bumped from the merge queue due to an occasional stochastic failure... despite this PR not being concerned with EpiAware.
what is the best way to build resilience here without causing false pass checks? Maybe fixing the rand seed?

Yes I think fixing the seed here and generally makes a lot of sense

Good point. Is there really no ability to run across test setup code?

Looks like this is coming https://github.com/julia-vscode/TestItemRunner.jl/pull/57
AFAIK its blocked by integration into VS-code test suite.

Things moving here
https://github.com/julia-vscode/TestItemRunner.jl/pull/57#issuecomment-2212396718

Alternate suggestion from f2f discussion is an EpiAware submodule with test utility functions.

This is covered by #419 .


Updated computed change detection documentation

As discussed:
https://github.com/vuejs/vue-devtools/issues/393
Feedback welcome.

I agree that a note about this would be helpful in the Vue Devtools docs, but I don't think it belongs here in the guide. This is more of an under-the-hood optimization than something the Vue users have to keep mind when building apps. The edge case of debugging a computed property that's not even being used is also quite rare I think.

@chrisvfritz

I agree that a note about this would be helpful in the Vue Devtools docs

Absolutely, well IMO, it should be fixed in devtools so that devtools actually updates.

This is more of an under-the-hood optimization

It is an over-the-hood optimization because I notice it.  When you can see something and it is constantly visible...well...thats above the hood as it gets.

than something the Vue users have to keep mind when building apps.

It is something I have to constantly keep in mind when building apps.  Maybe your workflow is different, that doesn't make mine (and many others') invalid.

The edge case of debugging a computed property

What?  Is debugging an edge case now?
Verifying that a computed property works before consuming it is not an edge case.  By nature, you cannot consume a function until it is written (and written correctly).  As an analogy I check to see if my pan is hot before I put food in the pan.  Knowing if my pan is hot is not an edge case.  It's part of the dev flow, developer experience (DX).  Observing that a function runs even once is quite common before consuming the return value (or in this case consuming via dom).  In fact, it is half the point of dev-tools.  This is js after all, not precompiled code...

quite rare I think.

Took me two years and many days of debugging with no solution to learn about this.  Ran into this almost every week.
I am giving vue the best kind of user feedback, really, pure user feedback from actual experience using the lib and what I am hearing is basically "I didn't have that issue, so, it doesn't matter".
Sorry for ranting on you.  This is like the 7th thing in a row I have suggested that has been closed for a less-than-adequate reason.

@FEA5T Honestly, I'm hesitant to respond at all with your combative tone and deliberately taking my words out of context in this post - if your issues are being closed, that might be a big part of it. And please remember, the Vue team is looking at a much bigger picture that we don't always have time to fully explain, so please try to give us the benefit of the doubt. 🙂


Suggestion: Add indenting to nested list comprehension inside dict comprehension to match dict formatting

Operating system: MacOS 10.14 (beta)
Python version: 3.7
Black version: 18.6b4
Does also happen on master: Yes
We have written a dict comprehension that is generated using list comprehension, and when black formats it, it feels like it is treating it all as list comprehension. I would expect it to handle the list comprehension like as if it were a dict assignment.
So, we are writing code that looks like:
dict_list_comprehenstion = {
    m['email']: m for m in comprehension_data if m['kind'] == 'internal'
}

Which black formats to:
dict_list_comprehenstion = {
    m["email"]: m
    for m in comprehension_data
    if m["kind"] == "internal"
}

The black formatted version reduces the clarity of what is going on by putting the list comprehension lines on the same indentation as the key argument. List comprehension can already be easy to misread.

It looks like there's no list comprehension involved in your example, only a dict comprehension.
Black will only split up the elements of the dict comprehension if it doesn't fit in one line otherwise. For example:
$ black --diff dcomp.py
--- dcomp.py	2018-09-10 15:43:52.548892 +0000
+++ dcomp.py	2018-09-10 15:43:54.490608 +0000
@@ -1,2 +1,4 @@
-very_very_very_very_long_name_so_it_doesnt_fit_in_one_line = {x: y for x, y in d.items() if x}
+very_very_very_very_long_name_so_it_doesnt_fit_in_one_line = {
+    x: y for x, y in d.items() if x
+}

But if the condition is itself long Black will split it up:
$ black --diff dcomp.py
--- dcomp.py	2018-09-10 15:45:16.958302 +0000
+++ dcomp.py	2018-09-10 15:45:19.145213 +0000
@@ -1,2 +1,6 @@
-very_very_very_very_long_name_so_it_doesnt_fit_in_one_line = {x: y for x, y in d.items() if very_long_condition_that_doesnt_fit_in_one_line_either(x)}
+very_very_very_very_long_name_so_it_doesnt_fit_in_one_line = {
+    x: y
+    for x, y in d.items()
+    if very_long_condition_that_doesnt_fit_in_one_line_either(x)
+}

That behavior all seems reasonable to me.

Putting elements of the comprehension on the same indentation level with "for" and "if" is a typical thing to do. If you see "for" inside a bracket pair, it tells you right away it's a comprehension. There is no other syntax that makes this keyword valid within a bracket pair. Since Black always dedents closing brackets on a separate line, it's visually obvious when you are inside a bracket pair.


Fix checkbox & radio input elements

Bugs found:

nested <label> tags
duplicate id attribute for each control
missing control classes: d-block, my-2
adding events on labels
text label missing

@hisk this looks pretty legit, when are you planning on testing/merging?

This is a bug indeed, thanks @ZsharE

@hisk bump. We bought everything from Shards, which is supposed to include support. I reached out via support ticket to see if you could merge this PR. Any thoughts?

Thanks for merging, @hisk!


Add PerkinsPostProgramPlacementIndicator to DimCteOutcomeIndicators

This is for capturing needs not currently supported by the CEDS model. Please do not send or share actual data as examples in this issue or in attachments.
Author(s)
Duane Brown
Authoring Organization(s)
AEM on behalf of Mississippi Department of Education
Email Address
Use Case Title
Add PerkinsPostProgramPlacementIndicator to DimCteOutcomeIndicators
Use Case Overview
The existing CEDS element Perkins Post-Program Placement Indicator is an indication of the post-program status of a CTE Concentrator in the second quarter after exiting from secondary education. This would be considered an an outcome. The proposal is to add this option set to the DimCteOutcomeIndicators table. The table contains only two elements presently.
Pull Request Number(s) (If applicable)
Use Case Background
Provide information related to why these changes/additions are needed.
Components Affected
Delete everything from the list below except the components affected:
CEDS Integrated Data Store (complete IDS Design Overview Table)
CEDS Data Warehouse (complete DW Design Overview Table)
CEDS Data Migrations (complete Data Migration Impact Analysis Table)
CEDS Element Gap Analysis
Provide link to CEDS Align Map or Connection.

INTEGRATED DATA STORE DESIGN OVERVIEW
Description of Proposed Changes
Provide a concise description of the proposed changes.
As-Is Diagram
Add as an attachment in this issue.
Proposed To-Be Diagram
Add as an attachment in this issue.

DATA WAREHOUSE DESIGN OVERVIEW (Includes Dim, Fact, and Report tables)
Description of Proposed Changes
Provide a concise description of the proposed changes.
As-Is Diagram
Add as an attachment in this issue.
Proposed To-Be Diagram
Add as an attachment in this issue.

DATA MIGRATION IMPACT ANALYSIS
Description of Proposed Changes
Provide a concise description of the proposed changes.
New Business Rule
Provide a concise and comprehensive description of the new business rule.
Existing Business Rule
Provide a concise and comprehensive description of the existing business rule.
Changed Business Rule
Provide a concise and comprehensive description of the changes to be made to the business rule.

Completed with feature/Issue-85


fix(post): Fixes issues with post

Auth added
Sends data as json

Resolves #46

Resolves #46

Codecov Report

Merging #49 (848050c) into master (57412f5) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #49   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          124       124
=========================================
  Hits           124       124

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pysolcast/base.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #49 (848050c) into master (57412f5) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #49   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          124       124
=========================================
  Hits           124       124

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pysolcast/base.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thank you so much

Thank you so much


Help me please, the same week is scrolled twice

If you open the monthly view of July, it will end in the first week of June. If we scroll a month ahead, there will again be the first week of the month. Maybe this is the reason for this calendar behavior.

Yes this is your reason.
Have you taken a look at the setup for a typical 1 row calendar? -> https://patchthecode.com/jtapplecalendar-home/week-view-month-view/ ?
here is the code for 1 row calendar setup
let parameters = ConfigurationParameters(startDate: startDate,
                                         endDate: endDate,
                                         numberOfRows: 1,
                                         generateInDates: .forFirstMonthOnly,
                                         generateOutDates: .off,
                                         hasStrictBoundaries: false)

Put that in your configure calendar setup
let me know if it works for you,

Thank you very much!


Update the manual with MPCal content

Updates the manual introduction to mention the new user story (don't mention editing the compiled output as primary strategy)
Updates the usage section with how to enable MPCal compilation (TODO: how do you set up MPCal -> Go compilation?)
Adds a section called ModularPlusCal that takes from the Wiki article and adds some extra prose / content (like pattern mapping macros)
TODO: the example outputs have moved, so I commented out a lot of the example Go output in order to the LaTex to compile. We need to figure out how we want to present the example sections (and add the MPCal examples)

I've fixed the TODO mentioned in the description of the PR. Is there anything else you want to fix, @fhackett?

@mrordinaire (see my code comment)
There's also the issue of adding the MPCal examples - maybe the simplest way is to just inline those in the example section without Go codegen for the moment?
Aside from my two issues/questions, looks good to me.


cargo clippy fails on the master branch

Describe the bug
After update rustup to the ;latest version rustc 1.59.0, running cargo clippy on master branch fails.
To Reproduce

update rustup to the latest version  rustc 1.59.0
run cargo clippy --all-targets --workspace -- -D warnings on master branch
build failed

Expected behavior
No error.
Additional context
Found the bug from #1878

@houqp

This happens on almost every rust release :D @HaoYang670 please feel free to send a PR to fix those warnings.