Quality umbrella

I'll keep track of some things that are bugging me here.
This is an incomplete list, I'll keep adding stuff here.

[ ] Shell

[ ] First load

[x] Flicker of white https://github.com/bluesky-social/social-app/pull/1759
[ ] Feed jumps around sometimes
[ ] Profile picture in the tab bar disappearing and reappearing during init

[ ] Glimmer after flicking a feed is missing top margin
[x] Top/bottom bar disappearance feels a bit janky

[x] https://github.com/bluesky-social/social-app/pull/1683
[x] https://github.com/bluesky-social/social-app/pull/1691
[ ] The content should “push” the bar up, synchronized with the gesture
[ ] It's also weird when we're already at the top and just scroll down and up a bit — causes blank space
[ ] Switching between feeds while the bar is hidden feels janky due to blank space

[ ] Gesture hell

[ ] Swipe between custom feeds with little distance is interpreted as tap on Android
[ ] Swipe between custom feeds on the left side is interpreted as a drawer swipe on Android

[ ] The feed switching header animation should be synchronized with the swipe gesture
[ ] Navigating to a profile not via preview (e.g. via link click) is missing a push animation
[ ] There is no fade on the background view during back gesture
[ ] There is no header crossfade during back gesture

[ ] Feed

[ ] Feed items

[x] Layout shift due to galleries https://github.com/bluesky-social/social-app/pull/1673
[ ] Layout shift when there's a link attachment
[ ] Tapping a feed item makes the parent post spinner too visible (need to shift it)
[ ] Tapping a reply may cause a content jump just before it scrolls to reply
[ ] Button positions in the glimmer are too off
[ ] Pressing like shouldn't move other buttons
[ ] Back gesture feels unreliable when starting with a long press and then a quick swipe; often interprets as tap
[ ] There's no tap target feedback e.g. on counts
[ ] We're loading too many too large images

[ ] Don't upscale images https://github.com/bluesky-social/social-app/issues/1101
[ ] Load small avatars by default
[ ] Smaller gallery items by default

[ ] "In reply to" should be resolved with initial payload
[ ] Clicking a post in a thread on the web is jumpy
[ ] Clicking on a post should never show spinner for that post’s content

[ ] Profile preview

[ ] Blank background while animating looks broken on Android
[x] There is a jump after navigating on iOS due to safe area https://github.com/bluesky-social/social-app/pull/1693
[x] There is a flash because the modal disappears before the navigation commits https://github.com/bluesky-social/social-app/pull/1693
[ ] The initial state is empty but we should have enough info to show the profile shell immediately
[x] The display name while loading is misaligned https://github.com/bluesky-social/social-app/pull/1693

[ ] Lightbox

[ ] Opening lightbox is laggy / misses target on Android (interferes with showing/hiding chrome?)
[ ] There's no transition between the thumb and the lightbox image and back
[ ] Lightbox doesn't take advantage of the already loaded thumb image (should it?)
[ ] Gestures

[x] Double tap to zoom

[x] https://github.com/bluesky-social/social-app/issues/1275

[x] Fix it on Android
[x] Align iOS behavior with the Android fix

[x] Double tap doesn't work in the header area (even to the left of the close button)

[x] Fix: https://github.com/bluesky-social/social-app/pull/1489

[x] Pinch

[x] https://github.com/bluesky-social/social-app/issues/1485
[x] Images in a single post may end up overlapping on Android https://github.com/bluesky-social/social-app/pull/1624
[x] Make it resistant to gesture spam https://github.com/bluesky-social/social-app/pull/1666
[x] The controls flying away animation is too aggressive https://github.com/bluesky-social/social-app/pull/1687

[ ] Swipe

[ ] Swiping sometimes gets stuck
[ ] Hide alt text during swipe?
[ ] Black screen feels weird
[ ] There's sometimes a spinner behind an already loaded image

[ ] Alt text

[x] Implement "hide on tap" https://github.com/bluesky-social/social-app/pull/1687
[ ] https://github.com/bluesky-social/social-app/issues/1445

[ ] Notifications

[ ] Pressing the notification icon doesn't do the refresh animation properly
[ ] Pressing notifications icon second time should clear it
[ ] Layout is borked when people joined with your invite

[ ] Profile

[x] First profile load jumps https://github.com/bluesky-social/social-app/pull/1690

[ ] Composer

[ ] Cropping needs rethinking on the web

Pressing the notification icon doesn't do the refresh animation properly

For whatever reason, you can't declaratively show the platform refresh animation - hence the fake spinner. Even if you can't use the platform animation, It'd probably look significantly better if you added Reanimated layout animations so that it didn't jump open/closed
Worth noting twitter shows no animation, and threads shows a spinner in the header, top right - I've yet to find an instance where an app opens the pull-to-refresh thing by pressing a different button.

Makes sense. Not saying it needs to animate in particular but something feels janky about it. I guess maybe not showing the spinner (like twitter does) may be better.

@gaearon hey, I finally managed to catch the "jumping up and down when scrolling far down a list" effect that I mentioned recently - I saw this while scrolling on my own profile for posts >1 month ago: https://files.mastodon.social/media_attachments/files/111/371/299/920/026/645/original/29aaba275b20302d.mp4 (starts around ~0:20)
Safari 17.1, macOS 12.7.1

Very interesting, thanks!

I'm not quite sure where to broach this but I wanted to suggest an enhancement to Lightbox gestures on Android to support "doubletap with swipe"* to mimic pinch to zoom. It's a long-standing and one-handed accessible zoom option that I think would be nice when viewing images.
*: https://www.youtube.com/watch?v=6Y7nPhk18W0 (not my video but shows the gesture)


RFC: Extend the sensor API with function for getting the value of a sensor attribute

Introduction
The Zephyr sensor API currently allows for setting attributes for sensor channels, but no way of retrieving the value of a sensor attribute.
Problem description
Extend the sensor API with function for getting the value of a sensor attribute. Sensor attributes should not be write-only, as they provide means for exposing sensor-specific settings that might be of interest for the calling application (calibration values, limits, ...).
Proposed change
Add a new, optional API function for retrieving the value of a sensor attribute.
Detailed RFC
Proposed change (Detailed)
Please see PR #26166.
Dependencies
None.
Concerns and Unresolved Questions
None.
Alternatives
Each sensor could implement a sensor-specific API for retrieving the values of sensor attributes.

Cc @MaureenHelm @gmarull


[azuredocumentdb] Added support for Azure DocumentDB

Support Azure DocumentDB.
scan operation is not supported currently.

Looks like #837 has azure as well. What is the difference between table storage and documentdb?

The binding should probably be something like azuredocumentdb

@k1xme - any plans to update this PR?

@risdenk Sorry! I totally forgot this. Will submit the changes by the end of this week.

@risdenk Sorry for the late update. I've fixed the nits and style problem you mentioned. This PR is ready for integration now.

@risdenk Done!

@risdenk fixed style check errors and exception handling nits per request.


Update documentation for Version 2.0.9

Docs Build status updates of commit 6ea2117:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/mixed-reality, working directory: W:\vann-s, target branch: HEAD, target branch commit: 23be9cef03db02c96936c581e9648ff5437217ba, source commit: 6ea2117f7f2ea95ddba141ff850400fb5a61955f.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 4d9dc4a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-create-player.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-troubleshooting.md
:white_check_mark:Succeeded
View

mixed-reality-docs/holographic-remoting-version-history.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added alerting and ISM to the back and frontend 2.2 manifests.

Description
Added alerting and ISM to the back and frontend 2.2 manifests.
Issues Resolved
https://github.com/opensearch-project/alerting/issues/498
https://github.com/opensearch-project/alerting-dashboards-plugin/issues/293
https://github.com/opensearch-project/index-management/issues/431
https://github.com/opensearch-project/index-management-dashboards-plugin/issues/217
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2427 (c68c4a6) into main (1542759) will increase coverage by 5.37%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #2427      +/-   ##
=============================================
+ Coverage     94.62%   100.00%   +5.37%
=============================================
  Files           210         6     -204
  Lines          4317       105    -4212
  Branches         29        19      -10
=============================================
- Hits           4085       105    -3980
+ Misses          226         0     -226
+ Partials          6         0       -6

Impacted Files
Coverage Δ

...jenkins/jobs/PromoteArtifactsQualifier_Jenkinsfile

tests/jenkins/jobs/Hello_Jenkinsfile

src/test_workflow/test_recorder/test_recorder.py

src/test_workflow/test_recorder/log_recorder.py

..._workflow/input_manifests_opensearch_dashboards.py

...nIntegTestScript_OpenSearch_Dashboards_Jenkinsfile

src/sign_workflow/signer_windows.py

src/test_workflow/integ_test/distribution.py

...bwc_test_start_properties_opensearch_dashboards.py

src/test_workflow/integ_test/distribution_rpm.py

... and 194 more

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

Block until IM is back to the manifests. Waiting for: opensearch-project/index-management#446 opensearch-project/index-management#447

ISM 2.2 branches have been cut, and added back to the manifests in this PR.


Exit during installation of incompatible Python versions

Code copied from https://github.com/ipython/ipython/blob/6a3e2db0c299dc05e636653c4a43d0aa756fb1c8/setup.py#L23-L58
Refs #73
Tested with
sudo python3.5 setup.py install

howfairis supports Python 3.6 and above.

See howfairis `README.rst` file for more information:

https://github.com/fair-software/howfairis/blob/master/README.rst

Python 3.5.2 detected.

echo $?
1

It would be easier to use python-requires to define supported Python versions.
https://packaging.python.org/guides/distributing-packages-using-setuptools/#python-requires

It would be easier to use python-requires to define supported Python versions.
https://packaging.python.org/guides/distributing-packages-using-setuptools/#python-requires

python-requires depends on a recent pip installation, people using an old Python probably also have a old pip so that will not be that portable.

Now refs #212

We have not had anyone with this problem, lets fix it when it occurs. Please reopen if needed.


Pattern Request: aws-https-mqtt-proxy

Use Case
https -> mqtt proxy
Create a pattern that allows https to talk to an mqtt proxy

Proposed Solution

Other

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

I'll be able to help on this request, you can assign it to me.

Suggest change the pattern name to: aws-apigateway-iot, in line with other patterns in the library e.g. aws-apigateway-lambda, aws-apigateway-sqs and aws-apigateway-kinesisstreams

This pattern is published in the latest release v1.66.0


Configuration details，The configuration object in the "Properties" format has changed and is missing labels

Properties格式的配置对象修改后，缺少“有修改”标签

resolved


swap github.com/gorilla/websocket out to allow building golang wasm gui

I dont know if this is something that interests you but this is useful for me.
This is required in order to build a golang client for benthos in golang using wasm.
There is a working example here: https://git.sr.ht/~whereswaldon/pointstar
It a gioui golang client and a standard golang server.
It does not use benthos, but does use https://github.com/nhooyr/websocket to show the golang server and golang client communicating. Its very easy to use.
The client is compiled to wasm ( for web ) and native ( for Desktop and Mobile).
For example on desktop you can run the client with just "go run ." and it all just works.
For compiling to Web or Mobile you use this: https://github.com/gioui/gio/tree/main/cmd/gogio

Hey @gedw99, in v4 the idea is that components are separate imports, so you'll be able to import alternative implementations at build time. This would enable leaner builds and swapping out certain capabilities and I'm assuming it'd also solve this issue.
What would the outcome of this particular issue be?

@Jeffail ah yeah i remember you mentioning that..  Its a great idea.
But https://github.com/Jeffail/benthos/search?q=github.com%2Fgorilla%2Fwebsocket is used in the core it seems ? If so then it wont help.
https://pkg.go.dev/nhooyr.io/websocket?tab=importedby is used by some battle tested projects btw. Its no toy.
name dropping follows :)
github.com/hashicorp/boundary/
https://pkg.go.dev/github.com/improbable-eng/grpc-web/go/grpcweb


Problem with analog stick

Hi. I detected an issue when using the analog stick. If trigger it gently to any direction (like less than half of range) it goes to the right direction. But if go full range either up or right goes to the opposite direction (If I go full up it goes down; if I go full right it goes left). I've tested with Project64 1.6, 2.3 and 3.0 and with plugin 0.2.2 and 0.2.4, always with the same result.

hi! i actually fixed this in commit 606f8e6 but never made a proper release. i just uploaded a v0.2.5 binary release that incorporates the fix : )


Add tutorial on building unit tests from command line

Proposed changes
Add a small tutorial on building unit tests in the README for the command line.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] Unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Further comments
I only wrote the portion for the command line. While I have a basic grasp on using Visual Studio Code, I figured it'd be best if @AlexanderJDupree wrote that portion.

Codecov Report

Merging #48 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #48   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         408    408
=====================================
  Hits          408    408

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 74ea81b...3ead7f8. Read the comment docs.


MB-16121: If a source bucket is not found, cbft throws authentication error instead

https://issues.couchbase.com/browse/MB-16121

Taking a look again at this after a long hiatus, the issue is REST call to couchbase isn't giving any info that the source bucket isn't there, but just that there's an auth error, so cbft can't tell the distinction.
Closing this, then, as can't fix.


Fix SciPy deprecation warnings

In SciPy v1.6 the names of two functions used in CosmoTransitions were renamed: scipy.integrate.simps was renamed scipy.integrate.simpson and scipy.integrate.cumtrapz was renamed scipy.integrate.cumulative_trapezoid. Until now, the old names of these two functions have been kept around for backwards compatability.
In SciPy v1.14 the old names will be removed; see here and here. Also in SciPy v1.14, the simpson function must have the keyword argument x=... passed with the keyword explicit. In v1.12 there is a deprecation warning for this.
To avoid CosmoTransitions breaking when SciPy v1.14 comes around, in this PR two lines of source code have been changed. Note however, this PR would break CosmoTransitions for SciPy versions older than 1.6.0 (and hence NumPy versions older than 1.16.5 and Python versions older than 3.7, see here). So, the version requirements in the setup.py file have been changed in this PR to reflect this. As a consequence, this PR also bumps the minor version number.

Thanks for doing this! I'm a little wary of killing support for Python 2 just because of a name change in scipy though. Any reason not to do
try:
    from scipy.integrate import cumulative_trapezoid, simpson
except ImportError:
    # scipy.version < 1.6
    from scipy.integrate import cumtrapz as cumulative_trapezoid
    from scipy.integrate import simps as simpson

instead?

Yeah, your suggestion seems a good one to me.

I've updated the PR with your suggestion, and undone the changes to setup.py.

Hey @clwainwright, I just wanted to check in and ask if you had any further suggestions re this PR?

Sorry for taking so long to merge this in! It's been kind of a crazy few weeks.
Yes, this looks good. I'll get it out on PyPI shortly.


Update existing content about correlated subqueries

This PR only updates what currently exists in our docs about correlated subqueries. Follow up PRs will be made by @knz or myself for the following:

Expanding the Subqueries page to include information about what subqueries are not supported
Adding examples of correlated subqueries where necessary

Related to #2976.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/fa071ca9f55149d6260cce1f079ec81219779872/


Uncaught ReferenceError: observer is not defined

Getting this error. Checked it out
ln 60: observer = observers[key];
should be
var observer = observers[key];
?

Mind submitting a diff to fix it? Thanks!

npm install does not install the change.. possible make new release 0.0.2 ? for now I have pinned to the commit hash

can you please submit a patch?

There is not really a patch for release management... ;-)
Read https://docs.npmjs.com/getting-started/publishing-npm-packages#updating-the-package
so something like
npm version minor
npm publish

or use other tools like https://www.npmjs.com/package/npm-bump


Error when trying to rename contract

Hi i cannot rename my contract, i dont know why:
PS C:\Users\jerome\Desktop\nft-erc721-collection-2.0.0\smart-contract> yarn rename-contract jeromescontract
yarn run v1.22.15
$ hardhat rename-contract jeromescontract
An unexpected error occurred:
config/CollectionConfig.ts:2:50 - error TS2306: File 'C:/Users/jerome/Desktop/nft-erc721-collection-2.0.0/smart-contract/lib/Networks.ts' is not a module.
2 import { ethereumTestnet, ethereumMainnet } from '../lib/Networks';
~~~~~~~~~~~~~~~~~
config/CollectionConfig.ts:3:25 - error TS2306: File 'C:/Users/jerome/Desktop/nft-erc721-collection-2.0.0/smart-contract/lib/Marketplaces.ts' is not a module.
3 import { openSea } from '../lib/Marketplaces';
~~~~~~~~~~~~~~~~~~~~~
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
PS C:\Users\jerome\Desktop\nft-erc721-collection-2.0.0\smart-contract>
Any advice? im totally stuck

Hi, can you put rename-contract script here?
thanks in advance


Behaviors don't check their owner.

What steps will reproduce the problem?
Attach an AttributeBehavior to Component
What is the expected result?
I expect it to fail since AttributeBehavior only supports being attached to a ActiveRecord.
What do you get instead?
It succeeds silently until I manually call evaluateAttributes. Then it throws an exception.
Additional info
In this specific case one could argue that the call above would not naturally occur (since a component that is not a model will not fire an ActiveRecord::EVENT_BEFORE_UPDATE event.
Still I think it makes sense for a behavior to do some checking at attach time instead of blindly making assumptions that are hard to validate.
One way to implement this would be to add a function to the base behavior:
protected function validateOwner($owner) {
    return true;
}

Which would then be called by attach() before attaching the behavior.
Not sure if that's the way to go, just an option, open to other suggestions.
When using PHP Traits it is common to ensure preconditions by adding abstract public function foo() to the trait.

I can see no benefit of checking owner type - it will simply degrade the script performance (especially for ActiveRecord) without much benefit. I do not consider wrong behavior owner attachment is spead enough mistake.

When using PHP Traits it is common to ensure preconditions by adding abstract public function foo() to the trait.

I do not understand, what do you mean. Can you provide some example for this?

In the context of the most common usage of behavior, AR, I agree with @klimov-paul. Won't be changed for the sake of AR performance.


Flame game in python

Create a program for the FLAME game in Python
@Ishaan28malik please assign me this issue

Here is your python code for FLAMES game. Please do review it and comment for any issues with it.
`firstPerson = input()
secondPerson = input()
flameString = ""
Removing common characters from str1 which are also in str2
for i in range(len(firstPerson)):
if firstPerson[i] not in secondPerson:
flameString += firstPerson[i]
Removing common characters from str2 which are also in str1
for i in range(len(secondPerson)):
if secondPerson[i] not in firstPerson:
flameString += secondPerson[i]
flamesCount = len(flameString)  # Taking the length of the string after removing all the common alphabets
listOfAlpha = ['Frienship', 'Love', 'Affection', 'Marriage', 'Enemy', 'Siblings']
F - Friendship
L - Love
A - Affection
M - Marriage
E - Enemy
S - Siblings
FLAMES game logic
while len(listOfAlpha) > 1:
index = flamesCount % len(listOfAlpha) - 1
if index < 0:
index = len(listOfAlpha) - 1
listOfAlpha = listOfAlpha[index + 1:] + listOfAlpha[:index]
print("Relationship:", listOfAlpha[0])
`

Heyy I want to work on this issue. Please assign it to me.

I have already made this game. Please assign it to me.

I really want to work on this ,please assign this  to me


Expanding cluster when clicking on it

hey,
I am trying to expand cluster when clicking on it. I've forked the repository and managed to get here with my onClusterPress:
const onClusterPress = (e: any, cluster: any) => {

if (!cluster.geometry || !this.superCluster || !this.root || !this.state.currentRegion) {
    return;
}

const coordinates = cluster.geometry.coordinates;
const curZoom = this.getBoundsZoomLevel(
    this.calculateBBox(this.state.currentRegion as any),
    {
        height: h(100),
        width: w(100),
    }
);
const clusterExpansionZoom = this.superCluster.getClusterExpansionZoom(cluster.properties.cluster_id, curZoom);

this.root.animateToRegion({
    longitude: coordinates[0],
    latitude: coordinates[1],

    latitudeDelta: this.state.currentRegion.latitudeDelta / clusterExpansionZoom,
    longitudeDelta: this.state.currentRegion.longitudeDelta / clusterExpansionZoom,
}, 100);

// TODO:
// this timeout is hack,
// need to figure out how to do this correctly without timeout,
// for some reason it does not work without.
setTimeout(() => this.calculateClustersForMap(this.state.currentRegion), 500);
};

for some reason, it does not work exactly like I would hoped for - it expands by random zoom. Does anyone have any recommendations how do I incorporate clusterExpansionZoom with react native maps?

Using this.superCluster is rather bad idea. This module play by itself rules.
The best approach will be to calculate zoom level on your own.
For example you can get all markers in clicked cluster and maybe calculate zoom level.
I know it is a lot of work, but I think this is the most stable way.

thanks.  I will add one gotcha; you need to use map/reduce functions of superCluster in order to "aggregate" all the coordinates that the cluster holds, otherwise you have no access to it.
this.superCluster = new SuperCluster({
    radius: this.props.radius,
    maxZoom: 20,
    minZoom: 1,

    initial: () => ({ payment_count_sum: 0, all_coordinates: [] }),
    map: (props: any) => ({ payment_count_sum: props.payment_count, all_coordinates: [props.marker_coordinates] }),
    reduce: (accumulated: any, props: any) => {
        accumulated.payment_count_sum += props.payment_count_sum;
        accumulated.all_coordinates = [...accumulated.all_coordinates, ...props.all_coordinates];
        },
});

now you can access all_coordinates:
const onClusterPress = (e: any, cluster: any) => {
    const region = regionContainingPoints(cluster.properties.all_coordinates);
    this.root.animateToRegion(region, 100);

   // TIMEOUT hack here?!
}

I am still having some problems clusters not completely expanding on some concrete latitudes/longitudes and sometimes animateToRegion() doesn't trigger calculateClustersForMap() correctly, I tried timeout and it makes it a bit better, but still not good enough. I will do some more investigation.

I released new version of this library.
In new module expanding cluster is set up by default 🎉


Why it can choose new role during training?

Hi, bro
I have a question that I have always wondered. Is it correct that in rode_controller()'s "forward()" function, the role selector you defined chooses new roles for agents no matter in decentralized execution or centralized training?
"""
def forward(self, ep_batch, t, test_mode=False, t_env=None):
agent_inputs = self._build_inputs(ep_batch, t)
    # select roles
    self.role_hidden_states = self.role_agent(agent_inputs, self.role_hidden_states)
    role_outputs = None
    if t % self.role_interval == 0:
        role_outputs = self.role_selector(self.role_hidden_states, self.role_latent)
        self.selected_roles = self.role_selector.select_role(role_outputs, test_mode=test_mode, t_env=t_env).squeeze()

"""
I don't understand why it is necessary to reselect the role of agents when these data are sampled based on ReplayBuffer during training.
I look forward to your answer.

Hi, bro
Are there any hidden parts that can't be told?

Thanks for your question. But I was not was very clear about your question,
maybe I misunderstood your meaning. For now, my thought is that roles
should be selected during both the training and test phases. How can we
train the role selector if training and test settings are different?
The role selector is based on local information and can run in both
centralized training and decentralized execution phases.
There is no hidden part.
On Fri, Nov 18, 2022 at 1:54 AM 98luobo @.***> wrote:

Hi, bro
Are there any hidden parts that can't be told?
—
Reply to this email directly, view it on GitHub
https://github.com/TonghanWang/RODE/issues/12#issuecomment-1319618417,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIX4MOXJKLDWGVQ3Z2CNI5TWI4R2RANCNFSM6AAAAAASBUFDQM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Roles are selected every several (5 in the experiments) time steps. This periodical selection is for better adjusting to the dynamic requirements of the task. The policy are trained under this scheme, and naturally should be tested under the same scheme.

Hi, Bro,
Thank you very much for your reply.
But I still doubt this mechanism. Let's return to the implementation and training of QMIX. Decentralized implementation is OK, but centralized training requires data from ReplayBuffer. For the original QMIX, it uses the stored<O, A, S, Reward>instead of selecting a new A for each agent.
This actually corresponds to the Role Selector you proposed. Selecting a role is equivalent to an agent called a "role selector "making actions. Its training and implementation should also strictly follow the flow of off policy algorithms such as DQN, right?
I try to fix the selected role during training. Specifically, the role information is completely taken from ReplayBuffer, such as<S, O, A, Reward, Role>. But to tell the truth, the effect is very poor.

In my opinion, the role selection is not an action. It is like a latent
variable conditioned on what we have in the replay buffer. We did not use
more information than what is stored in the buffer.
As for your idea, I think it is like use a supervised learning objective to
train the role selector, rather than a reinforcement learning objective.
Maybe you can try to mix these two losses and see what will happen.
On Fri, Nov 18, 2022 at 5:55 AM 98luobo @.***> wrote:

Hi, Bro,
Thank you very much for your reply.
But I still doubt this mechanism. Let's return to the implementation and
training of QMIX. Decentralized implementation is OK, but centralized
training requires data from ReplayBuffer. For the original QMIX, it uses
the stored<O, A, S, Reward>instead of selecting a new A for each agent.
This actually corresponds to the Role Selector you proposed. Selecting a
role is equivalent to an agent called a "role selector "making actions. Its
training and implementation should also strictly follow the flow of off
policy algorithms such as DQN, right?
I try to fix the selected role during training. Specifically, the role
information is completely taken from ReplayBuffer, such as<S, O, A, Reward,
Role>. But to tell the truth, the effect is very poor.
—
Reply to this email directly, view it on GitHub
https://github.com/TonghanWang/RODE/issues/12#issuecomment-1319839918,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIX4MOWXMLUW4ZADOJM3IXLWI5OBLANCNFSM6AAAAAASBUFDQM
.
You are receiving this because you commented.Message ID:
@.***>


Using mutable objects as default parameter values in class methods can lead to unexpected behavior

In the Document class, several methods use {} as a default parameter value for a dictionary-type parameter. However, using mutable objects as default parameter values can lead to unexpected behavior because the object is only created once when the function is defined, not every time the function is called. This means that if the mutable object is modified within the function, the modification persists across multiple calls to the function that omit the parameter.
To avoid this potential issue, I suggest using None as the default parameter value and creating a new instance of the mutable object within the function if the parameter is None. Here's an example of how the find_raw method could be modified to avoid this problem:
@classmethod
def find_raw(
    cls,
    filter=None,
    projection=None,
    **kwargs
):
    if filter is None:
        filter = {}
    if projection is None:
        projection = {}
    # rest of method body

Thank you for raising the issue.
We will solve this issue in the upcoming version after a certain level of R&D.

Hi @artheadsweden, we resolved the potential issue you raised.
You are welcome to raise errors more like this.


Android: does not register device, oneSignalRestClient returns "Invalid device_type format! Expected integer"

Description:
SDK doest not register device.
New device does not appear in dashboard device list.
Initialisation done specifying onesignal_app_id and onesignal_google_project_number in build.gradle.
Following logcat I noticed that there is request failing to onesignal api:
D/OneSignal: OneSignalRestClient: Making request to: https://onesignal.com/api/v1/players/2d9e5e61-46c4-4080-8a81-d47455ecaa9f/on_session D/OneSignal: OneSignalRestClient: POST SEND JSON: {"app_id":"myonesignalappidhere"} D/OneSignal: OneSignalRestClient: Failed request to: https://onesignal.com/api/v1/players/2d9e5e61-46c4-4080-8a81-d47455ecaa9f/on_session W/OneSignal: OneSignalRestClient: POST RECEIVED JSON: {"errors":["Invalid device_type format! Expected integer"]} W/OneSignal: Failed last request. statusCode: 400 response: {"errors":["Invalid device_type format! Expected integer"]}
Notably, after number of launches throughout an hour it returns 200 and registers new device. Without any code change
Environment

What version of the OneSignal React-Native SDK are you using?
3.1.4
How did you add the SDK to your project
npm

Steps to Reproduce Issue:

Install the OneSignal SDK using npm into your project
Initialize the SDK in the android/build.gradle
Launch app, see logcat, check dashboard for new device

Anything else:
In attempt to find a way around the issue, I tried few different sdk versions as well as latest, but ultimately hit #516 or #515 every time

@ivanlyubimov Have you made any customizations to the OneSignal Android SDK...?
Also, can you please completely uninstall your app, then install the app, run it for ~30 seconds, and then send us a link to the logcat?

@Nightsd01 yes, I have deleted device manually from dashboard. Didn't think it would cause problem.
Anyways, If I uninstall app from device, it still appears on the dashboard. And app returns same error in logs. Here is logfile

@ivanlyubimov Yes, deleting a device from OneSignal will cause a problem when you keep trying to use that same device.
This error means exactly the same thing. I would advise that you don't delete users from the dashboard from now on? Why are you deleting them in the first place? Perhaps there is a better way to accomplish what you are trying to do.
Also, I would recommend looking at this troubleshooting section for that custom tabs issue.

@Nightsd01 After all, scenario where dashboard user deletes device is for testing purpose I supposed
I need to make sure that sdk indeed subscribes at the moment when user opens application
What would be right way to do that?

We've made many bug fixes since 3.1.4.
In any case, the main issue you are facing is caused by the fact that you are deleting users so I'll be closing this issue, but we'll be happy to help you fix #516  and #515.
When you update back to the latest version of our SDK, please post on either 515 or 516 depending on what issue you run into. Then in that git issue, please post your app/build.gradle, android/build.gradle, and gradle-wrapper.properties.

@Nightsd01 yet you didn't address how could one test subscription correctly. Does it mean deleted device ids can no longer work with onesignal platfrom?

@ivanlyubimov After the on_session error the SDK will automatically create a new player for your app even on the 3.1.4 SDK. We haven't seen any cases where this wasn't handled correct.
For the other issues you linked to we will follow up there

Above the same issue is have in android also with me and giving same bellow error as i deleted user from dashboard, not app is not uninstall
OneSignalRestClient: POST RECEIVED JSON: {"errors":["not a valid device_type"]}
Failed last request. statusCode: 400
response: null

I'm also unable to register my android devices wether I use a real device or an emulator. No error message tho. I've followed all the setup instructions.


Move import of staging mock data to constants file in app instead of mirage

Since we do not use mirage in staging, importing directly from mirage directory caused an error. Instead, put the mock data to show in staging in a constants folder to keep mock data modular away from code without using mirage.
ember test results:
1..376
# tests 376
# pass  370
# skip  6
# fail  0

# ok

Codecov Report

Merging #1304 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1304      +/-   ##
==========================================
+ Coverage   46.64%   46.68%   +0.04%
==========================================
  Files         181      182       +1
  Lines        3722     3725       +3
  Branches      772      772
==========================================
+ Hits         1736     1739       +3
  Misses       1986     1986

Impacted Files
Coverage Δ

...p/components/datasets/containers/dataset-health.ts
75.75% <ø> (ø)
:arrow_up:

...erehows-web/app/constants/data/temp-mock/health.ts
100% <100%> (ø)

Continue to review full report at Codecov.

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

Critical change, going ahead to merge.

Coverage remained the same at 27.736% when pulling e0b24abf1cec0e6904a9b38715a2bf6b0acde317 on cptran777:fix-staging-mockdata-bug into ac8d93f36e78219f64a5e994eeee4f16d3b71c06 on linkedin:master.


(refactor) some opportunities to use multi-catch

We found some opportunities to use multi-catch in the source code. We changed the source code and assigned to the license statement--- the main problem with the previous pull request

Dear all, not sure whether or not this contribution will be accepted. Should I close it?

Hi @rbonifacio, in general, it is a good one. But we can not apply it at the current moment because it will kill our ability to merge more complex pull requests which are waiting for review now. We will merge it a bit later.

Thanks @Iaa. I will keep it opened.


Fixing wrong sign in route

from atSignIn into signIn.

Thanks, I'll merge this in manually. If you can please make the PR to the devel branch next time :)

@SachaG sorry about that :)


Issue scanning signed transaction QR from Specter-DIY

When co-signing in a multi-sig with a Specter-DIY I can't read the signed PSBT.
The Specter-DIY signs the PSBT fine whether it is transmitted via QR or SD card.
When I try to scan the QR code into Sparrow, the blue bar fills up, but the camera window doesn't disappear.
When I try to load the PSBT from the SD card, I get this error:

I have tested QR signing with this device on Specter and Blue Wallet and it works in both cases.
Specter-DIY has a simulator, which might help with this issue.
CC @stepansnigirev

Support for Specter DIY's trimmed PSBTs has already been added in https://github.com/sparrowwallet/sparrow/commit/600a77da3ae33bd283980c490cbf7b771707cb8e (not yet in a binary release).


PSA: passenger from Ubuntu 16.04 repo does not provide /usr/bin/passenger

I just stumbled over this:
The Ubuntu native package passenger (5.0.27-2) doesn't provide a passenger binary. This is relevant, if fetch(:passenger_restart_with_touch) is nil, i.e. you don't have an explicit
set :passenger_restart_with_touch, false # or true

in your Capistrano configuration. Otherwise capture(:passenger, '-v') will be executed and fail with
SSHKit::Runner::ExecuteError: Exception while executing on host xxx: passenger exit status: 127
passenger stdout: Nothing written
passenger stderr: /usr/bin/env: 'passenger': No such file or directory

Duplicates #39.  PR welcome, if you know what the proper command would be to restart passenger in this case.

Oh, I didn't see #39, sorry for the noise.
Restarting is not a problem, as /usr/bin/passenger-config does exist.

Is there a guarantee as to what version of passenger is running in this situation?
Is there a command we could run to detect this situation?

No, not really.

Ubuntu ships passenger (5.0.27-2) since Xenial (16.04 LTS)
Debian ships

ruby-passenger (4.0.53-1) in Stable (Jessie/8)
passenger (5.0.27-2) in Testing (Stretch/9)

Neither has a passenger binary, but a passenger-config (except Debian Stable, of course).
I've made a conscious decision and set :passenger_restart_with_touch in the config/deploy.rb.

I mean, one could first check with which passenger, whether the passenger binary exists and if not, decide whether or not to use touch-restart by checking the existance of passenger-config.
But I don't know how well that would work, if sudo is involved...


Allow to customize disabled background color of MacosTextField

Allow to customize disabled background color of MacosTextField
Pre-launch Checklist

[x] I have incremented the package version as appropriate and updated CHANGELOG.md with my changes
[x] I have added/updated relevant documentation
[x] I have run "optimize/organize imports" on all changed files
[x] I have addressed all analyzer warnings as best I could

@nbonamy Thanks for your contribution! Please merge in the latest changes from dev, and please post some screenshots demonstrating your change. Thank you!

Branch updated with latest dev. PR ready.

Branch updated with latest dev. PR ready.

Please provide a gif or screenshot of your change in action.

@nbonamy since you've not provided a GIF or screenshot of your PR's changes, I am closing this PR. You may reopen it when you are ready to provide what I've asked for. Thanks!


Ava dataloader

AVA dataloader

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: dreamer121121:x: ssssssss seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Compiler generates mismatched text maps for "t:=b ifTrue: [x] ifFalse: [y]. ^t"

From @blairmcg on October 5, 2016 18:11
The compiler is able to spot and optimise away a redundant temp that is assigned to and then its value immediately returned. It can also optimise away unconditional jumps to a return message bytecode, by inlining a copy of the return bytecode in place of the jump. In combination this particular code form causes the compiler to generate text maps with different numbers of entries for release (optimized) vs debug (non-optimized) code. It is an invariant of the debugger that the text maps should have the same number of entries as these are used to map between debug/release representations of methods when de-optimizing methods on the stack for debugging.
Copied from original issue: dolphinsmalltalk/DolphinVM#95

Another slightly different case is TestCase>>#runCase, defined as follows:
runCase

	[self setUp.
	self performTest] ensure: [self tearDown]

If a breakpoint is set inside the setUp method and then a step occurs so the debugger converts the setUp frame and the runCase (block) frame to debug frames, the IP is remapped incorrectly in the runCase frame so that when returned to it sends the setUp message again. The issue relates to the optimisation of 'push self; send' into a single instruction in the release build of the method vs separate instructions in the debug build.
This can be worked around by changing the pair of sends in the runCase block to a cascade.


Theme for new docs homepage

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-docs-theme-oss-check/50/ : SUCCESS in 3s

Created from the wrong branch


Players usually can't use the rtv command to switch maps

Players usually can't use the rtv command to switch maps when there's only one person inside

Issue pending ...


[move-prover] Track Abort instruction location for execution traces.

While we tracked aborts coming via the abort_flag, we did not for explicit calls to the Abort instruction, so those did not surface correctly in execution traces.
Motivation
Bug fix.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Existing tests.
Related PRs
NA

@bors-libra r=shazqadeer

:pushpin: Commit 95bfacc has been approved by shazqadeer

:hourglass: Testing commit 95bfaccd22295eea493f4785bca92944fa76841f with merge ad4f45a1bef93be62895990262908f17ee348992...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: shazqadeer
Pushing ad4f45a1bef93be62895990262908f17ee348992 to master...


Update version: Jellyfin.Server version 10.8.10

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/115002&drop=dogfoodAlpha

@wingetbot run

Automatic Validation ended with:


Need help mapping Jira custom email fields into ADO

I need to map Jira custom fields with sub-email addresses into ADO WIs email fields.
I have read the FAQ, but can't seem to generalize it to my problem.
I'm using  v3.0.139
In Jira, my custom field looks like this:
"customfield_10012": {
    "self": "https://jira.myorg.com/rest/api/2/user?username=databaseadmins",
    "name": "databaseadmins",
    "key": "databaseadmins",
    "emailAddress"<EMAIL_ADDRESS>    "avatarUrls": {
        "48x48": "https://jira.myorg.com/secure/useravatar?avatarId=10232",
        "24x24": "https://jira.myorg.com/secure/useravatar?size=small&avatarId=10232",
        "16x16": "https://jira.myorg.com/secure/useravatar?size=xsmall&avatarId=10232",
        "32x32": "https://jira.myorg.com/secure/useravatar?size=medium&avatarId=10232"
    },
    "displayName": "Database Admins",
    "active": true,
    "timeZone": "America/Los_Angeles"
},

And my target ADO WI field needs to look like this:
"Custom.DBA": {
    "displayName": "databaseadmins",
    "url": "https://spsprodcus1.vssps.visualstudio.com/guid/_apis/Identities/guid",
    "_links": {
        "avatar": {
            "href": "https://dev.azure.com/myorg/_apis/GraphProfile/MemberAvatars/aad.reallylongstring"
        }
    },
    "id": "524189c0-5085-6c3c-b784-922a01ca2d5d",
    "uniqueName"<EMAIL_ADDRESS>    "imageUrl": "https://dev.azure.com/myorg/_apis/GraphProfile/MemberAvatars/aad.reallylongstring",
    "descriptor": "aad.reallylongstring"
},

My (non-working) mapping in config.json looks like this:
{
    "source": "customfield_10012",
    "target": "'Custom.DBA'.uniqueName",
    "source-type": "emailAddress",
    "mapper": "MapUser"
},

I've tried quoting and un-quoting Custom.DBA.uniqueName in the config file, but no change.

I see that you would need a mapper that accesses the emailAddress property of the customfield_10012 field then? Could that be correct?
Writing your own mappers is relatively straight forward. Until then we should think of finding a general solution to this problem in the form of a mapper where you can specify the name of the property that you want to access. I will create a feature request for this so that we can track it.

Writing your own mapper is relatively straightforward, if you wish to have a crack at it yourself! Simply add a new case to the switch at https://github.com/solidify/jira-azuredevops-migrator/blob/master/src/WorkItemMigrator/JiraExport/JiraMapper.cs#L87C27-L87C27, and write your own logic


Hover issue on Support Us

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Hover on Support Us

Expected behavior
A clear and concise description of what you expected to happen.
it should stay pointer
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@akshitagupta15june Please review the PR #1183

@akshitagupta15june  Please assign this issue to me

@akshitagupta15june

I can still the issue
Please merge this PR

Helo, I am currently a GSSOC contributor, kindly assign the issue to me


Add README.md for time service [ECR-2896]

Overview
Add README.md for time service.
Do not merge, as it's needed to rename the time-service module first.

See: https://jira.bf.local/browse/ECR-2896
Definition of Done

[ ] There are no TODOs left in the code
[ ] The continuous integration build passes

@dmitry-timofeev Shall we add a separate LICENSE file (similar to exonum-time - https://github.com/exonum/exonum/tree/master/services/time ) or reference the EJB one?

Coverage increased (+0.07%) to 85.703% when pulling 16e43338c9987e769acd6d54dddb89d3f9ff6c7a on ECR-2896 into 20a5610857dafd271955744d27cacc4f36060515 on master.

@dmitry-timofeev Shall we add a separate LICENSE file (similar to exonum-time - https://github.com/exonum/exonum/tree/master/services/time ) or reference the EJB one?

EJB one as they are the same project.


Render CSVlint messages in standard way

Return the validation JSON in matter that correlates to the CSVLint translations.yaml file that contains ordered messages to go with errors warnings and info_messages.
Something like this but longer
document.getElementById("message-panel").innerHTML = JSON.stringify(errors)+JSON.stringify(warnings)+JSON.stringify(info_messages);
});
}

#38 adds validation messages to the pane, but they are not quite descriptive as only their types are shown. More descriptive messages would be nicer in the future, but not a priority right now.


Last modified date in Azure Storage Explorer explorer view Azure File Share

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Problem
No Last Modified Data for files and folder for Azure File Share
Desired Solution
Last Modified Date field in explorer view storage explorer
Alternatives and Workarounds
No response
Additional Context
No response

Next to this I would like to add date and sort on it


Domain-Driven Design

What Is It?

Ubiquitous Language - Have your code use the same words as the business
Model-Driven Design - Entities, Value Objects, Services, Modules, Aggregates, Factories, Repositories
Refactoring Toward Deeper Insight - ie. make it hard to code bugs.
Bounded Context - i.e. Some words might mean different things in different contexts. They should be broken out into different domains.

TODO Tasks
Refactor to:

[ ] Entities
[ ] Value Objects
[ ] Services
[ ] Modules
[ ] Aggregates
[ ] Factories
[ ] Repositories


[DoNotMerge]Add ContainerService track2 config

/azp run automation - sdk

Can one of the admins verify this patch?

hi @changlong-liu could  you pls help to review?

Hi @yungezz Yes, I'll review.
Hi @00Kai0 , looks more details are needed in the example content:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=482496&view=logs&j=8d2f9c49-cd83-5a9d-f3fe-2fd651afa742&t=43324f1f-6724-5d61-785f-f0ac9fd256a1&l=11


[feat] Ability to open internal ref in new tab

Thanks @dgarcia360 for the fix on my last issue.  Though of a feature request, not sure what the scope would be:
I'd like to be able to open an internal reference in a new tab

We don't have a use case to support this functionality in this theme.
If you want to modify the theme's behavior, here are the lines that process all the page links and determine if they are external or not to open them on a new window: https://github.com/scylladb/sphinx-scylladb-theme/blob/master/src/js/index.js#L9-L21


Support parent of top-level abstract clafers

The following model
abstract FeatureModel
abstract Feature
    [ parent in Feature, FeatureModel]

produces an error
Compile error at line 3 column 7...
Function 'in' cannot be performed on
TClafer {_hi = ["root"]}
in
TClafer {_hi = ["c0_Feature","c0_FeatureModel"]}

the type checker incorrectly assumes that the parent of an abstract clafer is the root.

The following model
abstract FeatureModel
abstract Feature
    [ parent in Feature, FeatureModel]

FM : FeatureModel
    F1 : Feature
       F2 : Feature

// It should be incorrect to have a Feature at the top-level
// F3 : Feature

produces a correct Choco output with a statement
c0_Feature.addConstraint($in(joinParent($this()), union(global(c0_Feature), global(c0_FeatureModel))));

for which Choco errors with
Exception in thread "main" org.clafer.ast.analysis.TypeException: Cannot join abstract c0_Feature . parent

Also, the model produces an incorrect Alloy output
abstract sig c0_Feature
{}
{ (this.~@r_c0_Feature) in (c0_Feature + c0_FeatureModel) }

but r_c0_Feature is not defined. Actually, it should be a derived union of the parents of the concrete clafers:
r_c0_Feature = r_c0_F1 + r_c0_F2

so, the correct Alloy code should be
abstract sig c0_Feature
{}
{ (this.~@(r_c0_F1 + r_c0_F2) in (c0_Feature + c0_FeatureModel) }


fix(Pagination): add pagination props to toolbar

Please check if the PR fulfills these requirements

[x] The commit(s) message(s) follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Please can you describe the purpose of this change what enhancement does it bring or what does it fix ?

We use Pagination with it's props in Toolbar but doesn't list them in Toolbar.props

Will be fixed in #171


Push notification after tapped is clicked not working

Hello All,
I’m working with push notifications in one of my Ionic 2 Application.
Just stucked when the app is not running in the foreground that means when the app is closed and push notification triggers, its showing up properly but when its tapped the application starts up but the code after push is not working :s
pushObject.on(‘notification’).subscribe((data: any) => {
console.log(‘Data returned’, data);
//if user using app and push notification comes
if (data.additionalData.foreground) {
console.log(“app is running”);
} else {
//if user NOT using app and push notification comes
//TODO: Your logic on click of push notification directly
console.log(‘Push notification clicked’, data);
}
});
I’m unable to see this ‘Push notification clicked’ log, how to check my after push code is working properly?
Any help will be highly appreciated!
Thanks.

Ok I got it but I'm using FCM, and sending notification from firebase console.
What is the notId and surveyID?

Can you please help me?
This is not working on Android -_-

@ShafiqEssani
Hello,
Have you found a solution?
Thank you

@ShafiqEssani  excuse me, the noteId and surveyId are dummy data taken from the documentation.
After in my app.component.ts  I capture the events like this.
    pushObject.on('notification').subscribe((data: any) => {         if(data.additionalData.foreground) {           this.onConfirmNotification(data.additionalData.news)         } else {           this.showNewsModal(data.additionalData.news);         }     });
Remember to send the push in format Data Payloads
Goodbye.

@StefanoMartella I'm still looking for it :/
If anyone can help me :(


Okta sso function v2

2.1.2

fixed a bug with the extraction and insertion of the domain name into the record set to ingest into Log Analytics
Changed the value used for the RowKey to a config name instrad of the Okta API, Okta API can be changed on demand by admins

2.1.0

Modified Okta Event log tracking to use next URI link from header
Fixed Total record counter to record correctly the total number of records ingested

2.0.2

Added  ACN_CD_OktaIssue925 resolution to add domain to output records

2.0.1

Added CHANGELOG.md to track future code changes
Complete rewrite of Azure Function code to address the following

Code needs to loop requests to Okta as each response from Okta is limited to max of 1000 records
relying on Timetrigger time minus (-) 5 minutes is unreliable and may result in duplicates and missing records
Okta communicates via HTTP Headers details about log event pagination orginal version did not utilise this information

Added using a table on Azure Storage account of function to maintain state of latest event time provided in OKTA Response
Added logic to use OKTA time detials in records to track last record returned from OKTA
Added Loop to continue getting paged results from OKTA until either all records retrieved or getting close (260 seconds) to Azure Function default timeout (300 seconds)
Added dependency on Powershell modules 'AZTable' and 'Az.OperationalInsights'
Fixed Timestamp to corret field from OKTA "published" field for Azure Log Analytics Time Generated header in Post HTTP
Fixed HTTP Request to OKTA, we need the Headers and 'Invoke-RestMethod' won't provide them
Fixed HTTP request to OKTA to use limit, since and after parameters to make sure we get all records and also don't get duplicates
Removed need for Environment setting of 'TimeTrigger' Function is now self managing event log times
Cleanup of Azure Function Log messages, consistent format of output
Added inline comments to explain code logic

@preetikr Created new pull Request with the Domain issue fixed


starsector: 0.96a-RC10 -> 0.97a-RC7

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for starsector is: Open-world single-player space-combat, roleplaying, exploration, and economic game
meta.homepage for starsector is: https://fractalsoftworks.com
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.97a-RC7 in filename of file in /nix/store/ip5bl73k4k93n1qwb1nb6dnhy6m3xc20-starsector-0.97a-RC7

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
starsector

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ip5bl73k4k93n1qwb1nb6dnhy6m3xc20-starsector-0.97a-RC7 \
  --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 starsector https://github.com/r-ryantm/nixpkgs/archive/3c4058a471ea0cc7dec9b375ddd53786573066db.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/3c4058a471ea0cc7dec9b375ddd53786573066db#starsector

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ip5bl73k4k93n1qwb1nb6dnhy6m3xc20-starsector-0.97a-RC7
ls -la /nix/store/ip5bl73k4k93n1qwb1nb6dnhy6m3xc20-starsector-0.97a-RC7/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:

starsector

Maintainer pings
cc @bbigras @rafaelrc7 for testing.

Fails to launch with
[nix-shell:~/.cache/nixpkgs-review/pr-286561]$ ./results/starsector/bin/starsector
OpenJDK 64-Bit Server VM warning: ignoring option PermSize=192m; support was removed in 8.0
OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=192m; support was removed in 8.0
Conflicting collector combinations in option list; please refer to the release notes for the combinations allowed
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

It seems that manual intervention will be required, fixing some of the JVM flags.

Close in favour of #286586 .


What is the intention of the controllers?

I see this project already has a set of boilerplate controllers that seems to be no-op. What is the intention of the controllers.  Can I propose that this project focus just on the API and only as a last resort create controllers. I think if you design with that mindset you will have a stronger API.  In the end there will most likely be one dominate implementation, but that should minimally be in a separate repo from "service-apis."
Related but different, a validating webhook would make sense to be in scope for this project to ensure the types are sane.

The controllers are generated automatically by the kubebuilder framework.
It is not in the plan currently to create a "default" controller here.
The controller stubs will likely be used to test the validation logic and support library code, though.

So the answer to your question is yes, we are focused on the API here, not a controller.

I added your question to the FAQ, otherwise not sure how to close this issue...

Thanks!


created a gcd example file using euclid's algo

Thought it could be nice to have more example vivaldi files.

Absolutely— the more the better! Thanks!


Email Templates

Hi,
It would be good to get further explanation of the new email template requirements.  I've upgraded from 0.1 and now nothing quite works.  I've had to put the templates directly in the config, as the modules aren't found at the time of the config parsing.  Also, adding <%= email %> raises a token error:
** (exit) an exception was raised:
** (FunctionClauseError) no function clause matching in EEx.Tokenizer.tokenize/5
    (eex) lib/eex/tokenizer.ex:27: EEx.Tokenizer.tokenize(#Function<20.50752066/0 in :erl_eval.expr/5>, 1, [trim: false], [], [])
    (eex) lib/eex/compiler.ex:16: EEx.Compiler.compile/2
    (eex) lib/eex.ex:191: EEx.eval_string/3
    lib/addict/mailers/mail_sender.ex:11: Addict.Mailers.MailSender.send_register/1

as though email isn't known when parsing the template.  Any ideas?
Thanks,
Lee

Hey @Lazarus404,
Sorry for the trouble with the breaking changes. Have you checked this? Does it help?

No wholly.  I need to revisit.  Currently, I'm still updating my controller
functions so that they replace the ones supplied.  Mostly because when
registering / logging in, the user has more fields and relationships than a
simple email / pass.
It's all good, though :-)
On Mon, Apr 11, 2016 at 7:54 PM, Nizar Venturini<EMAIL_ADDRESS>wrote:

Hey @Lazarus404 https://github.com/Lazarus404,
Sorry for the trouble with the breaking changes. Have you checked this
https://github.com/trenpixster/addict/blob/master/docs/configs.md#e-mail-templates?
Does it help?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/trenpixster/addict/issues/69#issuecomment-208211829

Alright! If you feel there's something that Addict could provide to ease the pain with relationships feel free to speak out :)
I've added the concept of Post Hooks where you can do extra processing after an action happens (maybe post_login and post_register are useful) and one other handle for extra validations that might also fit your flow, given you have relationships on the User model.
I'll close this issue for now, if you still have any issues with the e-mail templates after reading the docs please re-open it :+1:


download cmdstan as artifact

are there plans to download cmdstan as an artifact rather than having to manually install it ?
It would be so much easier for a beginner to just do ] add Stan and start coding.

Hi @filchristou
That has been attempted many times without success. Stan needs a compiler (written in Occam) to translate Stan Language programs to C++ and subsequently compiles and links that newly generated C++ code into the cmdstan executable.
On which platform do you work? For Linux and MacOS I suggest using git, on windows the coda route.
Rob

I work in Linux and it was just needed to download cmdstan-2.30.1.tar.gz from https://github.com/stan-dev/cmdstan/releases

Yip, that's the easier case.
Let me know if you need further help.

No help needed. Just wondering if that could be automated as an artifact in the first place.


[Expo][Realm] Cannot read property 'Helpers_get_table' of undefined, js engine: hermes

Description
I was trying to use realmDB in my react native application, but it had some problems in the current version, so I downloaded the "^12.0.0-alpha.2" version, apparently it solved it, but I have an error appearing, and I don't know how to solve...
TypeError: Cannot read property 'Helpers_get_table' of undefined, js engine: hermes
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:102:17 in reportException
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:148:19 in handleException
at node_modules/react-native/Libraries/Core/setUpErrorHandling.js:24:39 in handleError
at node_modules/expo/build/errors/ExpoErrorManager.js:25:19 in errorHandler
at node_modules/expo/build/errors/ExpoErrorManager.js:30:24 in
at node_modules/@react-native/polyfills/error-guard.js:49:36 in ErrorUtils.reportFatalError
at node_modules/metro-runtime/src/polyfills/require.js:225:40 in guardedLoadModule
at node_modules/metro-runtime/src/polyfills/require.js:127:23 in metroRequire
at http://<IP_ADDRESS>:19000/index.bundle?platform=android&dev=true&hot=false&strict=false&minify=false:null in global
React Native Version
0.71.6
Output of npx react-native info
info Fetching system and libraries information...
System:
OS: Linux 5.19 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
CPU: (8) x64 Intel(R) Core(TM) i5-1035G1 CPU @ 1.00GHz
Memory: 7.03 GB / 15.32 GB
Shell: 5.1.16 - /bin/bash
Binaries:
Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node
Yarn: 1.22.19 - /usr/bin/yarn
npm: 8.19.2 - ~/.nvm/versions/node/v18.12.1/bin/npm
Watchman: Not Found
SDKs:
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Languages:
Java: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.6 => 0.71.6
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
. npm install<EMAIL_ADDRESS>. Error "Cannot read property 'Helpers_get_table' of undefined, js engine: hermes"
Snack, code example, screenshot, or link to a repository

Please also consider reporting this to Realm here:
https://github.com/realm/realm-js


[Task]: Deduplicate text component in CE & EE

Closing it coz change in release plan. Tracking it in epic https://github.com/appsmithorg/appsmith/issues/20460


Convert unsupported expressions to keyframes?

I notice that certain expressions, like wiggle(), don't play nice with bodymovin…
Why not automatically convert unsupported expressions to keyframes as in Animation > Keyframe Assistant > Convert Expression to Keyframes?
Maybe taking that a step further… it could then delete the generated keyframes after export and re-enable the expressions (since Convert Expression to Keyframes disables them).

Yes, it's a possibility, but I prefer to leave that decision to the user at the moment.
Not all expressions are simple to identify if they will work.
And those that are, it might be better to use another way of animating, instead of converting to keyframes.


implement gradient checkpointing

This brought huge memory saving at a little compute cost in other places
I think it could be really helpful here too

@rom1504 i don't think it'll do that much in this setting, but you can try it out with https://github.com/lucidrains/DALLE2-pytorch/commit/be3bb868bfa428822b892709ecd79c70ad6f73fd#diff-038ecede954c29266888cb88e37cc06f61ba2433f8b5142c7b2b2cefde5ed0edR1724

ok this should be done for resnet blocks, we can add it for attention blocks at some future date


error running elastalert

bash-4.4$ elastalert
Traceback (most recent call last):
  File "/usr/local/bin/elastalert", line 11, in <module>
    load_entry_point('elastalert==0.1.29', 'console_scripts', 'elastalert')()
  File "/usr/local/lib/python3.6/site-packages/pkg_resources/__init__.py", line 587, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/local/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2800, in load_entry_point
    return ep.load()
  File "/usr/local/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2431, in load
    return self.resolve()
  File "/usr/local/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2437, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 951, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 894, in _find_spec
  File "<frozen importlib._bootstrap_external>", line 1157, in find_spec
  File "<frozen importlib._bootstrap_external>", line 1131, in _get_spec
  File "<frozen importlib._bootstrap_external>", line 1112, in _legacy_get_spec
  File "<frozen importlib._bootstrap>", line 441, in spec_from_loader
  File "<frozen importlib._bootstrap_external>", line 544, in spec_from_file_location
  File "/usr/local/lib/python3.6/site-packages/elastalert-0.1.29-py3.6.egg/elastalert/elastalert.py", line 1246
    raise EAException("Error querying for dashboard: %s" % (e)), None, sys.exc_info()[2]
                                                               ^
SyntaxError: invalid syntax

#1210

I see on v.0.2.1


Sort visited NFTs by time user has visited them

I visited Kusama bots last but still, appear as third and fourth.
Would be nice to sort it by the time the user has visited it.
I guess now it's sorted by the time being minted or actions?
now I check on the far right is 6 months ago on the very left 3 days and middle 1 day :D

Ill check this

@yangwao isnt the time mentioned in these cards referencing to minting date? do you want to sort them by minting date?

@vikiival what is the default order for nftEntities as the ids in query is being sent in the right order but returned in a different one, even when no value in orderBy is provided

@vikiival 👀

what is the default order for nftEntities as the ids

Not sure what you mean. Could you (@prachi00) rephrase your q?

Sure,
So what I mean is we store these in the localstorage and that is stored in the correct order that is by when the user visited them and the ids are also sent to the api in the correct order that is recent one is at the top
However, the response returned from the api is not in order

However, the response returned from the api is not in order

Imagine that you pass your shopping list to a shop assistant.
Shop assistant will go from the beginning of the store to the end and puts items to the basket if the item is in the shopping list.
After the shopping assistant go to the end of the store you may have probably all items however they won't be sorted in the correct order.
It means that the api can't guarantee you correct order (see the stackowerflow Q). The solution is to create sort of the matching function or do multiple queries 🤷

yeah makes sense, I guess creating a sorting function is then the ideal way


'System.Reflection.ReflectionTypeLoadException' occurred in mscorlib.dll

Steps to reproduce
I'm trying to create a user control with DotSpatial.

I build the Windows Form Control Library Project and add the .dll file to the reference of the Windows Form Application project.

DotSpatial version: 1.9
Expected behavior
No exception occurs.
Actual behavior
An unhandled exception of type 'System.Reflection.ReflectionTypeLoadException' occurred in mscorlib.dll
What should I do? Does DotSpatial control unable to be used as Windows Form Control Library?

Please attach sample project to reproduce.

Never mind, it was my fault by adding DotSpatial references with different version and thus make the exception occurred. Closing this issue now.


No master-latest-daily release available

On gcr.io/istio-release, there is no daily release of the master available.
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[X] Developer Infrastructure
Expected behaviour
There should be such a release every day.
Note: The images for all releases (1.0,1.1,1.2) are there. "Just" master is missing. So this might be related to https://github.com/istio/istio/pull/14737 which just got merged. Or is this just a coincidence.
How to get any information what went wrong in the release process?

@utka I have tried (I am still trying) to get a local release to check if this is related to the whole distroless change. I have seen you have created a local.sh for purpose. Thanks. Until now, I am struggling with this script and maybe you are interested in issues:

The directory istio/output/local did not exist the first time, so the rm -rf failed
The script several times because I have not the necessary repos cloned (e.g. proxy or tools)
The was a problem with some check, so I changed CB_VERIFY_CONSISTENCY=false
create_linux_archive failed on my mac because --owner is not a valid option for the installed tar

OK, I succeeded with a local release. Or better said, I failed with a distroless related error. So I am quite sure that the same occured in the release of master-daily. I will have a closer look.


change default hashtag to NSBCon

ping @Particular/hashbus-committers

Sorry for the self merge. I can't deploy a branched version of the website for NSBCon like I have with some of the back end components :worried:
Incidentally, this is "data in code" - https://github.com/Particular/HashBus/issues/124 :unamused:


Tensorboard support

Add support for tensorboard.
This commit add support for tensorboard and edits the following:
1. config.yml: add boolean flag wether to save in tensorboard or not
2. main.py and train.py: main writers for tensorboard
3. requirments.txt: added the tensorboard to be installed with the requirments (no version specified yet!)

This pull request will be merged after #27, #29, #32


ATmega32u2 Support

I just found your project. I'm very interested in using it.
However, I have some boards based on an ATmega32u2, a slightly simplified version of the ATmega32u4, that I'd like to use.
I'm wondering what special features of the ATmega32u4 you might have used that might not exist on the u2. I'm personally rather familiar with both of these processors and the only 2 major differences I know about are the existence of an ADC (and internal temperature sensor) and the 3-phase PWM output.
If I don't intend to use the internal temperature sensor, do you foresee any issues with using the ATmega32u2 instead?
Thanks!

If these are the only differences it should be possible to use the ATmega32u2 as well. I didn't list it currently, because I didn't come across any board that uses an ATmega32u2 chip.

Yeah, the main reason I'm thinking of this is that I'm planning to make a custom board and the ATmega32u2 is available in a smaller package. I happen to have other boards to test with as well.
Thanks for the reply. I'll close this for now. When I try it, and especially if I have issues, I'll report back here, assuming I remember!


Internal Javascript Error with Push/Pull functions

TypeError: draftname is undefined
Uncaught TypeError: Cannot read property 'replace' of undefined

Should be fixed with latest update.


setup.py: Remove deprecated pkg_resources in favor of importlib.util

Remove deprecated pkg_resources in favor of importlib.util.
Fixes this warning at install-time:

DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html

Closing in favor of

#2435


Stack crash when global variables are reset

Has anyone encountered the same problem?

Try using no global variables.


Remove deprecated and confusing thread configuration methods

IIRC this originally came to existence as a workaround for "you can't just remove methods from public API" but this is even worse, since upgrading infinispan in a project does compile but fails with NPE at runtime when trying to use these configuration methods...

I always enjoy removing deprecated methods. Pulling...

Actually I will add a JIRA to track this as well.

https://issues.jboss.org/browse/ISPN-5660

Integrated into master, thanks @rhusar !

Thanks @wburns !
(I didn't open a Jira originally thinking that since it never worked nobody would need to know that its gone but you are right, lets be concise so thanks for opening one.)

I'm just missing the word "deceitful" in the JIRA! :-)

Browsing the thesaurus at night.. couldn't resist! :-)

Haha, writing code and browsing the thesaurus? Take it easy, Rado! :-)


connect tracking events

connect tracking events to letters
remove some old files that got missed in previous PRs (doh!)
refactor address_editable model to DRY it a bit
remove example from individual attributes (and readOnly when I noticed it since it will go away in v3.1)

It looks like models/letter.yml is missing the url field which is described in the docs. The URL for the custom envelope field is present, but there's also a URL associated with the letter object (which is how the customer downloads it).

nice catch!

I'm wondering whether the examples in the response files should contain every field, or only required ones? I saw that in the examples within the responses for letter and all_letters, not every field is listed out.

I'm wondering whether the examples in the response files should contain every field, or only required ones? I saw that in the examples within the responses for letter and all_letters, not every field is listed out.

We should work towards having multiple examples. At least one that contains every field, and at least one that has only the required fields. For now, we're doing a first pass through the documented endpoints. The examples aren't going to be ideal.
Please add a section (maybe even a page?) to the markdown docs discussing examples:

where to put them (in requests/responses and models)
why (3.1 compatibility)
best practices (multiple examples, at least one with all, at least one with only required, cover cases of interest)
use cases: link to prism mock docs on using named examples


Key based authentication is not permitted on this storage account in assets workflow

Steps to reproduce
The last few runs of the assets workflow (triggered by a merge to main) is failing with the following error:
ERROR: Key based authentication is not permitted on this storage account.
Observed behavior
assets workflow is failing in the Upload install.sh step: https://github.com/radius-project/radius/blob/main/.github/workflows/assets.yaml#L36.
There is a connection string passed into the<EMAIL_ADDRESS>action. We receive the following error:
ERROR: Key based authentication is not permitted on this storage account.
Desired behavior
Connection to the storage account should work fine, thus making assets workflow run successfully.
Workaround
Not sure if this is something that we could handle in the storage account (the one that we use in this workflow) level. I am thinking that there might be an update to the storage account resources in general that would block some types of authentications if a permission was not given.
rad Version
edge
Operating system
No response
Additional context
Link to a failing run: https://github.com/radius-project/radius/actions/runs/10205813601/job/28237404786.
Would you like to support us?

[ ] Yes, I would like to support you

AB#12930

Same issue was reported a few days ago in another repository: https://github.com/OfficeDev/teams-toolkit/issues/12092.

The assets workflow is now deleted. Closing this issue.


importContractFromAddress() Usage problem

A simple contract API ping() is defined in gjavactestjavacontract/src/test/java/gjavac/test/simple/simpleContractTest.java file.
When this method gets called in gjavactestjavacontract/src/test/java/gjavac/test/simple2/simpleContractTest.java like this:
`MultiOwnedContractSimpleInterface multiOwnedContractSimpleInterface = importContractFromAddress(MultiOwnedContractSimpleInterface.class, arg);
        return multiOwnedContractSimpleInterface.ping("");`

when the simple2 contract API is invoked for testing the interaction among contracts, an exception is raised:

So far what we have got:

The internal API importContractFromAddress() is commonly used for the interaction of XWC smart contracts, and it is widely used in the LUA smart contracts which all work well on the XWC chain, a simple assumption is that there is a improper use in the Java-written smart contract. We have also tried comparing the Java compiled .ass with Lua compiled .asm for the same simple logic, but failed to find the root cause.

As the compiler works fine and the generated smart contract binary code can be deployed on the XWC chain without any problem, the issue happens during the runtime inside the UVM. We may need to check the source code of UVM and see what exactly this error refers to.

Even for a simple case of calling importContractFromAddress() between smart contracts, it doesn't work as expected. Considering this is an important API used in Lua, we may need to find out what differences between Java and Lua compiled binary code.

Please help on this. Thanks a lot!

It appears that the problem comes from the contract that gets called.
We have made a test to use a LUA compiled smart contract, and a Java smart contract imports it and calls its API, it works as expected.
We need to go deeper on why UVM reports the error when a Java smart contract gets called.

I have built a new UVM with some debug info and tried calling importContractFromAddress(), and found something new as described below:
In the scenario that contractB calls API of contractA, the reported error saying that "this contract byte stream not matched with the info stored in uvm api", it appears that on the UVM, contractA has an unexpected API "storage" for the contract written in JAVA. Compared with LUA contract, there is no such issue.
The related code of UVM is lauxlib.cpp: int luaL_import_contract_module_from_address(lua_State *L):
When fetching APIs from the lua_State, the unexpected API is found and therefore an error is returned:
 if (stored_contract_info->contract_apis.size() != apis_count)
    {
        char error_msg[LUA_COMPILE_ERROR_MAX_LENGTH];
        snprintf(error_msg, LUA_COMPILE_ERROR_MAX_LENGTH - 1, "this contract byte stream not matched with the info stored in uvm api, need %d apis but only found %d", stored_contract_info->contract_apis.size(), apis_count);
        if (strlen(L->compile_error) < 1)
            memcpy(L->compile_error, error_msg, LUA_COMPILE_ERROR_MAX_LENGTH);
        global_uvm_chain_api->throw_exception(L, UVM_API_SIMPLE_ERROR, error_msg);
        uvm::lua::lib::notify_lua_state_stop(L);
        return 0;
    }

Next we need to find out why the "storage" is considered as an API.

This problem occurs because，we code contract.setStorage(new Storage()); in ContractEntrypoint class,
then,we remove the code,the smart contract can perfect execution

The probelm can be solved by removing the improper setStorage() function. However, need to figure out if an error/warning should be raised for such kind of improper usage. It is quite confused if everthing seems to work well and then an exception happens at runtime.
If it is confirmed that the function contract.setStorage() is useless, maybe we should consider removing it to avoid this potential issue.

The issue has been fixed by changing the type of UvmContract.storage.
var storage: T? = null to val storage: T? = null
The root cause is that. in kotlin, 'var' refers to 'variable', which by defaults has two methods: set() and get(), while 'val' refers to 'value' which by default has get() only.
The problem happens when the variable 'storage' defines the set() method implicitely. So changing 'var' to 'val' fixes this issue.
Now importContractFromAddress() works well and passed all testcases. So close this issue.


Feat: Change in UI designs of the virtual assistant

Describe the feature
The Virtual Assistant page appears inconsistent with the overall design due to a mismatch in the color theme between the main page and the chatbot section.
Aligning the color scheme will create a more cohesive and visually appealing experience.
Add ScreenShots

Record

[X] I agree to follow this project's Contributing.md
[X] I'm a SWOC contributor
[X] I want to work on this issue

@vansh-000 Issue assigned to you.


[T3A3][F10-A2]Shi Xiyue

Ready for review

Some comments added. Please close the PR after reading comments:
:+1: Hi Xi Yue, Well done, simple and clean execution, however for this round of execution since this task is simple it is ok to do the changes and push all in 1 commit however, in future commits do take note to commit after each significant changes instead of all in one :)


Linux ASP

Is there new URL's for Linux ASE Support? I can not find any new url's for docker hub or any other stuff that may be used by Linux ASP

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

ID: 29246f12-744e-08a1-c222-b0eae866b401
Version Independent ID: 6e925f96-dc31-d400-2681-2e387b40ed74
Content: Lock down outbound traffic - Azure App Service Environment
Content Source: articles/app-service/environment/firewall-integration.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@dhruvb14  Thanks for the question! We are investigating and will update you shortly.

@dhruvb14 can you elaborate further on your question? Are you asking if ASE supports Linux ASPs? Or are asking if there are different/new base images that are used in Linux ASPs that are in an ASE?

Specifically asking if there is any new endpoints needed when this is running in azure government which our firewall teams need to unblock because currently deploying a Linux ASP fails.

@dhruvb14 Can you provide a snippet of those deployment errors? Are you getting errors when pulling images from docker hub into your ASE app or when you're pushing your code the ASE?

@dhruvb14, since we're working with you offline, I'll proceed to close this thread and update once we have a resolution.

To provide an update, Linux App Service Plans in ASEs is not currently supported in Azure Government. Support will be added in the future; however, there isn't an available time frame for when support will be added.


Make Java NIO standard options available in fs.open()

The Java NIO API provides useful standard open options[1] for file operations. At the moment Ringo only supports READ, WRITE, APPEND. Ringo could also provide shortcuts / options for CREATE, CREATE_NEW, DELETE_ON_CLOSE, SPARSE, TRUNCATE_EXISTING.
[1] http://docs.oracle.com/javase/7/docs/api/java/nio/file/StandardOpenOption.html

Would be nice, but if it isn't supported in common js, we should provide it
in some other way.  On the other side: if it doesn't brake the standard,
why not...
Am 21.10.2015 5:14 nachm. schrieb "Philipp Naderer" <
<EMAIL_ADDRESS>
The Java NIO API provides useful standard open options[1] for file
operations. At the moment Ringo only supports READ, WRITE, APPEND. Ringo
could also provide shortcuts / options for CREATE, CREATE_NEW,
DELETE_ON_CLOSE, SPARSE, TRUNCATE_EXISTING.
[1]
http://docs.oracle.com/javase/7/docs/api/java/nio/file/StandardOpenOption.html
—
Reply to this email directly or view it on GitHub
https://github.com/ringo/ringojs/issues/306.

:+1: these would be just additional options, so i don't see any harm offering them.

Then it will be a useful addition. We should give a hint in the
documentation that these aren't standard, tough.
Am 21.10.2015 8:15 nachm. schrieb "Robert Gaggl"<EMAIL_ADDRESS>
[image: :+1:] these would be just additional options, so i don't see any
harm offering them.
—
Reply to this email directly or view it on GitHub
https://github.com/ringo/ringojs/issues/306#issuecomment-149982694.


Implement channel mechanism in preload script

Doc: https://docs.google.com/document/d/1rI078gHjmR1cL1cjKtrGoDAsJoDVLkX_aF64pc8hgOs/edit#heading=h.7nki9mck5t64
script.addLoadScript should be able to communicate back to client.
Add the binding mechanism similar to #294, #295.
Probably, only binding names can be provided, and the created script will have bindings with provided names as an argument:
{
  method: "script.addLoadScript",
  params: {
    function: "(binding1, binding2)=>{binding1('some_event')'}"
    bindings: ["BINDING_NAME_1", "BINDING_NAME_2"]
  }}

Research is needed, as CDP bindings don't support WebDriver serialisation.
Tracking

[x] add arguments to add preload script
[x] implement channel mechanism in preload script
[x] make WPT tests pass

Implementation details are described in http://go/chrome-devtools:bootstrap-bindings-proposal

Done in #857


Unexpected error when running an UPDATE command on a shared dataset with write permissions

Context
Going through smoke tests (Smoke tests BUILDER onrpremise (2016/12/05) - OG09 in particular).
Steps to Reproduce

Login to an on-premise version of the BUILDER.
Open a shared dataset with read-write permissions (you'll need to share a dataset between two accounts if you don't have one).
Open the SQL editor.
Try to run an UPDATE SQL statement (something simple like UPDATE carto.barrios SET nombre='CARTO' WHERE cartodb_id=1).

Current Result
I'm getting the following error: permission denied for function _cdb_userquotainbytes.

Expected result
Since I have write permissions on the dataset, I should be able to run an UPDATE.
Browser and version
Chrome Version 54.0.2840.98 (64-bit) for Mac.
.carto file
You can test with the following dataset:
barrios.csv.zip
Additional info
I couldn't run the UPDATE command on the the shared dataset, but a DELETE command worked fine.
#SMOKE_OG09

I've reproduced it
/cc @xavijam

I cannot reproduce it 🤔

Ok, this only happens when the dataset is owned by the admin, and the subuser tries to run the UPDATE, not the other way around.

Replicated in dev environment using the same user creation tasks as onpremise:
SUBDOMAIN="oadmin" PASSWORD="12345678"<EMAIL_ADDRESS>ORG="oorg" bundle exec rake cartodb:db:setup_user
bundle exec rake cartodb:db:create_new_organization_with_owner \
    ORGANIZATION_NAME=oorg \
    ORGANIZATION_DISPLAY_NAME=Org \
    ORGANIZATION_SEATS=100 \
    ORGANIZATION_QUOTA=$((1000 * 1024 * 1024)) \
    BUILDER_ENABLED=true \
    USERNAME=oadmin

Also happens when creating users via superadmin API

Just waiting for 🍏

:clap: @javitonino


Update information-management-policy-planning.md

Add link for the SPO version of this article.

Tagging #1481

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.


adding an azure queue section

As Azure queue is an option visible in the UI but not mentioned in this document. two new images would be added to the media/scaler content

@craigshoemaker
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks, @yodobrin!
This article is meant to explain scalers from a high level rather than detail each option.
I am going to #please-close this PR for now.


add tool-tip and shortcut keyboard functionality to the swap handler

I try in windows it's not working
ctrl+shift+s
Please check in mac it's ok

UnelmaSupport
Hello!
Thanks for getting in touch! This is an automatic email just to let you know that we've received your request and your ticket reference is #700.
One of our support agents will get back to you shortly. Please do not submit multiple tickets for the same request, as this will not result in a faster response time.
View Help Center: https://unelmasupport.com/hc
Regards,
UnelmaSupport
If you’re having trouble clicking the "View Help Center" button, copy and paste the URL below into your web browser: https://unelmasupport.com/hc
© 2024 UnelmaSupport. All rights reserved.

UnelmaSupport
Hello!
Thanks for getting in touch! This is an automatic email just to let you know that we've received your request and your ticket reference is #705.
One of our support agents will get back to you shortly. Please do not submit multiple tickets for the same request, as this will not result in a faster response time.
View Help Center: https://unelmasupport.com/hc
Regards,
UnelmaSupport
If you’re having trouble clicking the "View Help Center" button, copy and paste the URL below into your web browser: https://unelmasupport.com/hc
© 2024 UnelmaSupport. All rights reserved.


Add Members to ignore

I've ran into something that had non-standard properties.  Rather than implementing the handling of those, I've added the ability to have ignoring of strings to stop the generation of things.
They can be reimplemented on the partial interface via _Instance if needed.
Hope this is good for you

@adamhathcock
Thank you for your work.
I'll abandon this PR and continue on https://github.com/StefH/ProxyInterfaceSourceGenerator/pull/77


Converting the CSS to a SCSS structure

Introducing a clearer SCSS structure for the addon.

All the CSS is now compiled to vendor.css
Vendor prefixing is handled by ember-cli-autoprefixer

One problem I was unable to solve was the separation of the theme stylesheets and the import based on config/environment.js. I've removed that as part of this initial commit. I'm not sure if this is something that you want to remain @knownasilya, but it would be nice if we could get the best of both implementations.

I prefer Less, but Sass (not that familiar with it) is fine I guess :smile:, although I do want to keep it configurable, so I'm not going to merge this just yet.
If you have ideas on how to keep it configurable, that would help move this along.

@knownasilya I'll keep at it 👍

Hi Folks, I'm just helping @ynnoj here with the build system and i'm trying to make use of the treeForStyles hook in the ember addon. The plan was for it to decide which theme to include in the index.js file so that you could define a configuration entry in the consuming App to specify which theme to include.
Unfortunately my ember-cli-fu has failed me this time :sob: Maybe it's time to get some external help from someone on the ember-cli team to see if this is even the right avenue to be taking. Maybe reach out to @rwjblue ?

I'm going to close this, since there hasn't been any movement. Feel free to submit a fresh PR (or ask me to reopen) if you are still working on this.


Changes to suit Exile & make better gameplay

I have discussed these changes in a message to Janski on the Exile Forums and am pushing them. Please consider them as they're mostly fixes ;)
Love the work! Just a few tweaks i find beneficial.

@GamingAtDeathsDoor  Cool, thx for the contribution


Time Rail isn't showing up on media player & volume slider in wrong place

The time rail/scrubber isn't loading on my player, regardless of browser, when I press play.
Additionally, the volume slider seems to show up over the pause/play button.
ANy advice on how to fix this would be greatly appreciated.
My site with player is here: http://thetvdudes.com/index.php/features/podcasts/564-the-tv-dudes-podcast-084-comics-issues

Still can't seem to figure this one out. Any suggestions?

@baronvongrant I only see the Podcasts section and everything seems to be working fine. Can you point a section where you are experiencing the issue you describe? If it is not an issue, can you close this ticket, please?

Closing this issue since no answer was posted in more than 2 weeks. Let us know if you need to reopen this issue. Thanks!


Thanos backend

Add thanos backend for kube-prometheus-stack
By default, Prometheus uses an in-memory database, and restarting the pod leads to the loss of metrics. Also, the standard installation does not allow you to store metrics for longer than 14 days. You need to add a time series database for storing metrics with the ability to configure the collapse of metrics after a certain time

sak-prometheus


For Hacktoberfest 2023

Changes
Add the correct logo
Submitter Checklist

[ ] Includes tests if functionality changed/was added
[ ] Includes docs if changes are user-facing
[x] Set a kind label on this PR
[x] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes
NONE

/approve
/lgtm


Update fabric-sdk-java to 2.2.19

Updates org.hyperledger.fabric-sdk-java:fabric-sdk-java from 2.2.5 to 2.2.19.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.hyperledger.fabric-sdk-java", artifactId = "fabric-sdk-java" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.hyperledger.fabric-sdk-java", artifactId = "fabric-sdk-java" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #260.


Gradient do not fill entire screen

On large screens the background gradient don't fill the entire cover image
Example: http://d.pr/i/MIql

This issue is still visible for me