SupplementaryViewFactory for section headers and footers

@icanzilb Updated the PR

@icanzilb Updated the PR

Looks good thank you! Will go about merging it when I have a moment

@alexjameslittle I'm sorry but I merged this by mistake. There were no tests for the new code - could you create a separate PR with those?


Documentation

Description
Add contributors section
Screenshots
No response
Additional information
No response

@Soumya-Kushwaha Go for it...


[BUG][WORKER-9349]fix param priority

Purpose of the pull request
refer to https://github.com/apache/dolphinscheduler/issues/9349
Brief change log
fix param priority
Verify this pull request
I have test this locally.

Codecov Report

Merging #9379 (d13441e) into dev (4a29c6a) will decrease coverage by 0.22%.
The diff coverage is 39.82%.

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

@@             Coverage Diff              @@
##                dev    #9379      +/-   ##
============================================
- Coverage     39.98%   39.76%   -0.23%
- Complexity     4375     4387      +12
============================================
  Files           822      825       +3
  Lines         32924    33174     +250
  Branches       3656     3681      +25
============================================
+ Hits          13165    13191      +26
- Misses        18532    18749     +217
- Partials       1227     1234       +7

Impacted Files
Coverage Δ

...lugin/alert/wechat/WeChatAlertParamsConstants.java
0.00% <ø> (ø)

...uler/plugin/alert/wechat/WechatAppChatMessage.java
0.00% <0.00%> (ø)

...cheduler/plugin/alert/wechat/WechatAppMessage.java
0.00% <0.00%> (ø)

...inscheduler/api/controller/ExecutorController.java
40.00% <ø> (ø)

...cheduler/api/service/impl/ExecutorServiceImpl.java
41.96% <ø> (ø)

...er/api/service/impl/TaskDefinitionServiceImpl.java
24.01% <ø> (+0.05%)
:arrow_up:

...che/dolphinscheduler/common/enums/AlertStatus.java
0.00% <0.00%> (ø)

...nscheduler/service/process/ProcessServiceImpl.java
30.76% <ø> (ø)

...plugin/task/api/parameters/AbstractParameters.java
26.98% <0.00%> (+0.83%)
:arrow_up:

...inscheduler/plugin/task/api/parser/ParamUtils.java
0.00% <0.00%> (ø)

... and 13 more

Continue to review full report at Codecov.

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

data quality will can't use ,because there return null


With landscape rotation on screen, the pushed stream shows distortion as attached.

With landscape rotation on screen, the pushed stream shows distortion as attached.
I've change the init to this.
_captureManager.videoConfig.orientation = UIInterfaceOrientationLandscapeRight;
Attached image is a black pencil on white desk.

It's no need to rotate the image, it will be slower, we just need to set the [AVCaptureConnection setVideoOrientation:AVCaptureVideoOrientationLandscapeRight] or
//屏幕方向
if (self.videoConfig.orientation == UIInterfaceOrientationLandscapeRight) {
    _videoCamera.outputImageOrientation = UIInterfaceOrientationLandscapeRight;
} else {
    _videoCamera.outputImageOrientation = UIInterfaceOrientationPortrait;
}

已解决


Block2D rendering failing with IndexError.

I am currently trying to create an environment where a team of agents (jellies) compete with enemy units (gnomes) to win a shootout game. The gnomes are part of the environment and make use of A* and proximity triggers to fire on the jellies.
So far, I have been rendering the environment only using gd.ObserverType.ISOMETRIC. I tried to switch to gd.ObserverType.BLOCK_2D by specifying the following sections in the GDY file.
# Specifying this block for each object under the Observers key.
Block2D:
   -Shape: e.g. circle
     Color: [ x, y, z ]
     Scale: 1.0

# Specifying in the Environment section also under Observers
Block2D:
   TileSize: 30

When I run the program though, what I get is a couple of time steps rendering and then the following error:
Traceback (most recent call last):
  File "/Users/tekier/uni/diss/smac_lite/aliens/aliens.py", line 31, in <module>
    obs, reward, done, info = env.step(env.action_space.sample())
  File "/opt/anaconda3/envs/smac_lite/lib/python3.8/site-packages/gym/wrappers/order_enforcing.py", line 37, in step
    return self.env.step(action)
  File "/opt/anaconda3/envs/smac_lite/lib/python3.8/site-packages/gym/wrappers/step_api_compatibility.py", line 52, in step
    step_returns = self.env.step(action)
  File "/opt/anaconda3/envs/smac_lite/lib/python3.8/site-packages/griddly/GymWrapper.py", line 363, in step
    self._player_last_observation[p] = self._get_observation(self._players[p].observe(),
IndexError: map::at:  key not found

Hopefully, this is case of me just missing something from the GDY file, but I've had a look at some examples and it seems like I've got everything I need.
To Recreate
aliens.zip

Download and unzip the attached folder.
Run aliens.py

My environment:
MacOS Monterey 12.2
Python 3.8.13
Griddly version 1.4.3

So this is something thats handled poorly in Griddly.
when you have something set up like this:
Isometric:
        - Image: oryx_iso_dungeon/fire-pink-up.png
          Scale: 0.5
        - Image: oryx_iso_dungeon/fire-pink-right.png
          Scale: 0.5
        - Image: oryx_iso_dungeon/fire-pink-down.png
          Scale: 0.5
        - Image: oryx_iso_dungeon/fire-pink-left.png
          Scale: 0.5
      Block2D:
        - Shape: circle
          Color: [ 0.8, 0, 0.1 ]
          Scale: 0.2

And you are using setTile commands... if there is only one tile for the block observer, it's basically crashing because its looking for the nth tile (which is set in the isometric observer but not the block observer)
A quick workaround would be to add 4 block shapes (that are the same).
In Griddly this should almost certainly throw a warning or an error (and just use the single tile that exists)

Workaround seems to work fine.


Update for release of runtime-php 1.16.0

fine except for the one missed 1.15 to 1.16 change I flagged

Doh!


Mock up front-end design

User story
As a developer working on this project, I should be able to reference UI mockups so that I have a clear vision to work toward and can separate development concerns from UX tasks.
Sub-tasks

[x] Research existing tools’ UI for inspiration and direction
[x] Mock up relevant screens and basic user flow
[ ] ~Export assets as needed for specific components~

Marking this as closed with the understanding that for MVP, we can narrow scope to the low-fidelity mockup Joel created from our whiteboard session. I think as we move beyond MVP we can take time to create higher-fidelity mockups and decide on a look and feel together.
Mockup
Frontend mock-up -2.pdf


Moves derive and match definitions to a new module.

The new module in derive_def.v can then be imported by both setoid.v and derive.v
Also adds Existing Instance for regex_setoid and associated morphisms so
they can be used in modules that import setoid. (By default instances inside a
Section are not exported).
@awalterschulze This is what you had in mind in https://github.com/awalterschulze/regex-reexamined-coq/pull/42
I couldn't figure out how to make the setoid instance and morphisms visible in modules that import setoid. I eventually worked out Existing Instance does the trick.

I am going to merge this, so we can use it for the streaming session.


Added Sieve Of Eratosthenes [C] and minor changes in PR Template

By submitting this pull request I confirm I've read and complied with the below requirements.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm name} [{Language}], not Update README.md or Added new code.

@aashutoshrathi: Please create two different pull requests for the two issues.

@singhpratyush should i close this pr and create a new one which only add Sieve Of Eratosthenes [C]

I think that would be the better way. @aviaryan : Views?


Unable to connect to openshift after installation

Hi,
First of all, thanks for this useful way of installing openshift. I appreciate it!
Secondly, I ran into the docker image issue which I was able to bypass by ignoring the docker checks. However, after the installation, I am not able to connect to openshift by running the:
open $(terraform output master-url) command.
I logged into the master node and could see the pods:
> $ oc get pods
NAME                       READY     STATUS    RESTARTS   AGE
docker-registry-1-jdsvq    1/1       Running   1          29m
registry-console-1-d2chx   1/1       Running   1          29m
router-1-ggncv             1/1       Running   1          29m

When I try to connect, I get this standard error:
This site can’t be reached

<ip>.xip.io refused to connect.
Try:

Checking the connection
Checking the proxy and the firewall
ERR_CONNECTION_REFUSED

Any idea what might be wrong here?

Same here, I cannot connect to the master from outside VPC.


Recipient selector

Is it possible to create a recipient selector using magic form?  ie, if I have three divisions (sales, parts, service) the user could select from a dropdown menu and the email would be sent to the appropriate email address (choose sales, email sent to sales@company.com)?  Sorry, I'm new to OctoberCMS

Hello @jleinbaugh
I like the idea, i will see if can be part of the next release (no estimated date yet).

Hello @jleinbaugh
After spend some time i found this features requires some refactor so i found a quicker approach.
Using the extend feature of the plugin (more info here) you can easily override the recipients email address.
You need to create a new plugin and work on the boot method of Plugin.php
Here is an example:
public function boot() {
    \Event::listen('martin.forms.beforeSaveRecord', function (&$formdata, $component) {

        // Get department input value
        $department = $formdata['department'];

        // Get department email address
        switch ($department) {
            case 'marketing':
                $email =<EMAIL_ADDRESS>                break;
            case 'sales':
                $email =<EMAIL_ADDRESS>                break;
            case 'support':
                $email =<EMAIL_ADDRESS>                break;
            default:
                $email =<EMAIL_ADDRESS>                break;
        }

        // Override component mail recipients
        $component->setProperty('mail_recipients', [$email]);

    });
}

Of course your form will need a new department input:
<select id="department" name="department" class="form-control">
    <option value="marketing">Marketing</option>
    <option value="sales">Sales</option>
    <option value="support">Support</option>
</select>

Also, i made an empty plugin to be used for this pourpouse: https://github.com/skydiver/october-plugin-formsextender
Simply replace the boot method with previous code.


[HOLD for payment 2022-12-27] [$1000] Tooltip not shown for emoji picker button on hover the outer part, it only shown for hovering the icon

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Hover on the emojipicker button icon surrounding pressable area
Expected Result:
Tooltip should shown for entire button
Actual Result:
Tooltip only shows on hover the icon
Workaround:
unknown
Platform:

Where is this issue occurring?

Web

Version Number: 1.2.37-2
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
ff5be293-b6dd-4ede-8cde-239f39379a3f.webm
Expensify/Expensify Issue URL:
Issue reported by: @Pujan92
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1670532553163569
View all open jobs on GitHub
https://user-images.githubusercontent.com/43996225/206594156-1c8b2031-d564-43b2-ac46-d88917dbd0e3.mp4
ff5be293-b6dd-4ede-8cde-239f39379a3f.webm
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e647d4e751c0a2f6
Upwork Job ID:<PHONE_NUMBER>775242752
Last Price Increase: 2022-12-12

Proposal :-
We need to wrap Pressable in Tooltip instead of the icon
diff --git a/src/components/EmojiPicker/EmojiPickerButton.js b/src/components/EmojiPicker/EmojiPickerButton.js
index 32af3ed59..f199850be 100644
--- a/src/components/EmojiPicker/EmojiPickerButton.js
+++ b/src/components/EmojiPicker/EmojiPickerButton.js
@@ -28,6 +28,7 @@ const defaultProps = {
 const EmojiPickerButton = (props) => {
     let emojiPopoverAnchor = null;
     return (
+        <Tooltip text={props.translate('reportActionCompose.emoji')}>
         <Pressable
             ref={el => emojiPopoverAnchor = el}
             style={({hovered, pressed}) => ([
@@ -39,14 +40,15 @@ const EmojiPickerButton = (props) => {
             nativeID={props.nativeID}
         >
             {({hovered, pressed}) => (
-                <Tooltip text={props.translate('reportActionCompose.emoji')}>
+
                     <Icon
                         src={Expensicons.Emoji}
                         fill={StyleUtils.getIconFillColor(getButtonState(hovered, pressed))}
                     />
-                </Tooltip>
+
             )}
         </Pressable>
+        </Tooltip>
     );
 };

After Fix :-
https://user-images.githubusercontent.com/81307212/206596002-448e07f9-1491-4642-97a3-320f9e0d103f.mov

Proposal
Wrap Pressable inside the Tooltip.
https://github.com/Expensify/App/blob/216bbe9ff3995264e08a542d618bcc1db12d946f/src/components/EmojiPicker/EmojiPickerButton.js#L31-L49
     <Tooltip text={props.translate('reportActionCompose.emoji')}>
            <Pressable
                ref={el => emojiPopoverAnchor = el}
                style={({hovered, pressed}) => ([
                    styles.chatItemEmojiButton,
                    StyleUtils.getButtonBackgroundColorStyle(getButtonState(hovered, pressed)),
                ])}
                disabled={props.isDisabled}
                onPress={() => EmojiPickerAction.showEmojiPicker(props.onModalHide, props.onEmojiSelected, emojiPopoverAnchor)}
                nativeID={props.nativeID}
            >
                {({hovered, pressed}) => (
                    <Icon
                        src={Expensicons.Emoji}
                        fill={StyleUtils.getIconFillColor(getButtonState(hovered, pressed))}
                    />
                )}
            </Pressable>
        </Tooltip>

Proposal:
You need to wrap whole pressable component with Tooltip. At this time, Tooltip is wrapping a Icon component.
That's why the tooltip is showing when mouse hover the Icon, not whole pressable.
Code is same as above.
My Upwork profile is :
https://www.upwork.com/freelancers/~01a1a69d7da842b796

Proposal
It is a simple problem to fix we need to just wrap the tooltip component into the pressable component.
This is a simple solution to this problem.
Here is my Upwork profile: https://www.upwork.com/freelancers/~01cf836885e7c7a016

IMO @Pujan92 should be hired, it's the first (and probably best) working solution after Help Wanted and External labels are applied.

@s77rt Just to be clear there is no such rule that you can't propose solution before External label according to Contributing Guidelines. We are allowed to post proposals before External Label.

As @Pujan92 is duplicate proposal to mine. I don't think they should be hired.
This concern was also raised in another issue https://github.com/Expensify/App/issues/13435#issuecomment-1343468927 they were advised to bring there concerns in slack channel by the assignee. I would also advise you to do the same for any concerns.

@s77rt Just to be clear there is no such rule that you can't propose solution before External label according to Contributing Guidelines. We are allowed to post proposals before External Label.

I think one of the latest comment by @JmillsExpensify suggested locking the issue till the external label won't be added which indirectly point to the same thought according to me. Let the team decide and I am fine with their decision.

Let the team decide and I am fine with their decision.

Yes, it would be the best let the team decide I will also be okay with there decision.

Hi, I am struggling to set up this project locally. I need help, I am not here to steal anyone's work
This is the error I am getting while just running 'npm install'. I am using the same node version, can you help?
**npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js.
npm ERR! code 1
npm ERR! path C:\Users\asimi\OneDrive\Desktop\work-practice\App\node_modules\shellcheck
npm ERR! command failed
npm ERR! command C:\Windows\system32\cmd.exe /d /s /c ./install.sh
npm ERR! '.' is not recognized as an internal or external command,
npm ERR! operable program or batch file.** ###

@asimimamabdullah Please direct all setup and environment problems not related to the issue to #expensify-open-source here
We're aware of the lack of clarity around proposals before the External label, and have a couple fixes that will be coming out soon. But yes, contributors should not be posting until the external label is applied.  I'm going to minimize all conversation that is not related to the issue.  If you have additional questions, please bring it to #expensify-open-source

1 minor issue I am seeing for the above proposals is due to the vertical margin applied to the emoji button showing tooltip even though we hover on the top and bottom part.
Issue
4734cc5f-15ea-45aa-af1f-d70ed50a66f8.webm
Updated Proposal
Wrap Tooltip with View and remove vertical margin from this emoji button.
<View style={[styles.justifyContentCenter]}>
    <Tooltip text={props.translate('reportActionCompose.emoji')}>

https://github.com/Expensify/App/blob/ed3e2dc507e0f02a30b22a012412fbecc0be2290/src/styles/styles.js#L1482
remove marginVertical property
9c0cb7b4-9cbf-4adc-b389-ef96cd1ebf05.webm

I don't think it is intentional and according to me, it might not be a good user experience. I have raised the same here. To make the item center I think it's better to use justifyContent instead of a fixed vertical margin.

@aimane-chnaif any thoughts on the above?

IMO, we should keep it consistent with the send button and so when you hover between them it has the same height
so 4px margin between emoji button and tooltip (similar to send button)

Proposal:
Previous proposals didn't handle responsive with multiline text
https://github.com/Expensify/App/blob/main/src/components/EmojiPicker/EmojiPickerButton.js
    return (
+     <Tooltip containerStyles={styles.alignSelfEnd} text={props.translate('reportActionCompose.emoji')}>
        <Pressable
            ref={el => emojiPopoverAnchor = el}
            style={({hovered, pressed}) => ([
                styles.chatItemEmojiButton,
                StyleUtils.getButtonBackgroundColorStyle(getButtonState(hovered, pressed)),
            ])}
            disabled={props.isDisabled}
            onPress={() => EmojiPickerAction.showEmojiPicker(props.onModalHide, props.onEmojiSelected, emojiPopoverAnchor)}
            nativeID={props.nativeID}
        >
            {({hovered, pressed}) => (
-               <Tooltip text={props.translate('reportActionCompose.emoji')}>
                    <Icon
                        src={Expensicons.Emoji}
                        fill={StyleUtils.getIconFillColor(getButtonState(hovered, pressed))}
                    />
-               </Tooltip>
            )}
        </Pressable>
+     </Tooltip>
    );

Keeping it consistent with the send button makes sense to me, but ultimately I would think this one would also behave like other buttons, for example the mini menu:

Updated Proposal :-
To handle edge case of multiline text in the composer we can wrap EmojiPickerButton with a view just like we are doing for send button as passing self align property to tooltip container doesn't sound a good idea.

diff --git a/src/components/EmojiPicker/EmojiPickerButton.js b/src/components/EmojiPicker/EmojiPickerButton.js
index 32af3ed59..237af6c43 100644
--- a/src/components/EmojiPicker/EmojiPickerButton.js
+++ b/src/components/EmojiPicker/EmojiPickerButton.js
@@ -28,6 +28,7 @@ const defaultProps = {
 const EmojiPickerButton = (props) => {
     let emojiPopoverAnchor = null;
     return (
+        <Tooltip text={props.translate('reportActionCompose.emoji')}>
         <Pressable
             ref={el => emojiPopoverAnchor = el}
             style={({hovered, pressed}) => ([
@@ -39,14 +40,13 @@ const EmojiPickerButton = (props) => {
             nativeID={props.nativeID}
         >
             {({hovered, pressed}) => (
-                <Tooltip text={props.translate('reportActionCompose.emoji')}>
                     <Icon
                         src={Expensicons.Emoji}
                         fill={StyleUtils.getIconFillColor(getButtonState(hovered, pressed))}
                     />
-                </Tooltip>
             )}
         </Pressable>
+        </Tooltip>
     );
 };

diff --git a/src/pages/home/report/ReportActionCompose.js b/src/pages/home/report/ReportActionCompose.js
index 7811d1c25..ad94fd561 100644
--- a/src/pages/home/report/ReportActionCompose.js
+++ b/src/pages/home/report/ReportActionCompose.js
@@ -686,11 +686,13 @@ class ReportActionCompose extends React.Component {
                         )}
                     </AttachmentModal>
                     {canUseTouchScreen() && this.props.isMediumScreenWidth ? null : (
+                        <View style={[styles.justifyContentEnd]}>
                         <EmojiPickerButton
                             isDisabled={isBlockedFromConcierge}
                             onModalHide={() => this.focus(true)}
                             onEmojiSelected={this.addEmojiToTextBox}
                         />
+                        </View>
                     )}
                     <View style={[styles.justifyContentEnd]}>
                         <Tooltip text={this.props.translate('common.send')}>

https://user-images.githubusercontent.com/81307212/207423409-a58747db-77d0-4237-8424-3dd5f456e719.mov

can you elaborate more detail about the reason?

@aimane-chnaif My bad at first glance I thought it might mess up the tooltip to show when setting tooltip container to flex-end so proposed the change https://github.com/Expensify/App/issues/13455#issuecomment-134953956 . On testing it also works pretty good and fixes the issue on multiline comment. Now, it is up to you to go with whose ever proposal. Both proposals do the same job and fix the issue. Only mine follows the same approach we are using to keep send button icon to flex-end

Removing the Help Wanted label since we have plenty of proposals to choose from that will solve the issue

Thanks for the review, I agree with your analysis.
In this specific scenario, it might not matter too much since the text is constant and unlikely to break into multiple lines but I'll move forward with @situchan's proposal since since they considered other edge-cases here and like you mentioned found a fix to a possible regression

Thanks @thienlnam!! I've sent upwork contracts to @aimane-chnaif and @Pujan92.
@situchan please apply on Upwork or let me know your name in Upwork so we can ensure you're paid :)

I'm going OOO for a family emergency, if this needs action while I'm away please re-add the bug label to assign someone.

@situchan please apply on Upwork or let me know your name in Upwork so we can ensure you're paid :)

@kadiealexander applied. thank you

I extended the contract while Kadie is out. In the meantime, we're still waiting on the regression period to pass.

I'm not assigned here but I can help. Assigning myself.

It looks like we have the following:

Issue reporter: @Pujan92 $250
Contributor: @situchan $1,500 (incl. 50% bonus)
Contributor+: @aimane-chnaif $1,500 (incl. 50% bonus).

Upwork job is here. All contributors have been paid out.

That means we only lack the BZ checklist. @thienlnam do you agree with @aimane-chnaif that we're good on your items? If so, let's get those checked off.

Yeah I agree, checked those items off

Thank you for your help @JmillsExpensify, I was unfortunately OOO sick. I've proposed a regression test update in Slack here :)

Awesome thank you!

Closing this out, from the discussion here there is no regression test needed https://expensify.slack.com/archives/C01SKUP7QR0/p1673912860873499


[bug] Vue2.0.0-rc.8 v-if AND 'slot'

https://jsfiddle.net/myronliu347/mj7xmfjn/3/embedded/
when open change more ,   name is test slot to 'default'

Hello @myronliu347
Thanks for filing this issue, it seems to be a bug indeed.

it seems to be a bug in Android Webview, still


OpendTect External Attributes

Hello,
I wonder if the development of python scripts through the External Attribute plugin is limited to working only with scalar parameters and seismic volumes? Well, I would like to have access to other inputs as well: access to horizons to perform operations, wavellets and seismic wells.
How can I access these entries using the External Attribute?
Thank you very much in advance.
Regards,
Charlan.

The plugin only works with seismic volumes but you can use other features in OpendTect to generate seismic volumes from horizons (Horizon Attribute )and wells (Create Seismic from Well) to use with it. Not sure how you would get wavelets into a seismic volume.
Regards,
Wayne

Hello,
I am very grateful for your return.
I have one more question about how the "extattrib" script works. I see that there is a function called "doCompute" where calculations are made. In this function there is the call of other functions like "doInput and Output" that prepare the input and output data. My question is: If the "doInput and Output" functions define the inputs and outputs respectively, what is the need for the "doInput and Output" functions inside a "while" loop if I already have this data loaded in memory? I would like to understand how the access of the information of my inputs happens, because this has a direct impact on the calculations. I'd like to have more control than is occurring in my scripts.
Thank you very much in advance.
Regards,
Charlan

Have you had a look at the plugin documentation? In particular the section "Structure of a Conforming Application".
The External Attribute plugin provides a mechanism to communicate between the OpendTect attribute processing system and a completely separate program, in this case a script running inside a Python interpreter. Information is passed between the script and the External Attribute plugin in OpendTect by using stdin and stdout pipes. OpendTect determines what data is sent there is no control over this from the Python script. From the scripts point of view everything is very atomic. In each pass through the script's "while loop" enough input traces are provided (as specified by the stepouts) to calculate a single output trace. The doInput function reads the set of traces from its stdin pipe. The doOutput function writes the result to the scripts stdout pipe.
To use this plugin you will have to adapt your algorithm to operate with how the OpendTect attribute calculation system delivers data. It is a bit inefficient but it is also very flexible allowing the same attribute code to output data to elements in the 3D scene (inlines, crosslines, random lines, horizons), crossplots and to new seismic volumes via batch processing.
If you want to control the order of traces being processed you need to look into the OpendTect volume builder or batch processing facility which require delving into C++.
Wayne


Please bump Keras

Please bump Keras again or change the requirement to >=

It's bumped in PR #115 , which should get merged sometime in the next few days. Feel free to give that branch a shot if you'd like it sooner. That branch should be totally functional, all that's left is some docs I want to add before merging. Would appreciate any feedback if you hit any issues using the new version

We've been burned in the past with >= when keras was breaking backward compatibility with every version. Would be open to reconsidering now though

Requirement is now tensorflow>=1.1.0 in the current master


Clean-up Python #4815 - P1.3: Chem\Draw

This is the sub-PR breaking from the large PR 4815: https://github.com/rdkit/rdkit/pull/4815

Updated Changes:

[New]: Clean-up unused import in IPythonConsole.py
[New]: Clean-up unused import in UnitTestSimilarityMaps.py
[Q&A]: In line 434, symbolLength += 1 + len(str(nHs)) confused me as the number of hydrogen for every atoms is limited at most to one digit only, usually from 0 to 4 (probably at most 7). The number of hydrogen which is larger than 10 simply needs a further checking, I guess. So, the 'symbolLength' variable should be incremented by 2 rather than 1 + len(str(nHs)).

Thank you for your review. I have changed as requested.


Prepare to require authentication for unsubscribe

https://trello.com/c/sgJIkObA/662-authenticate-before-unsubscribing-from-digest-emails
This simplifies the existing unsubscribe controller and associated
tests, so that it's easier to add more code: we want to require the
user is authenticated in some way, in order to prove they own the
subscription they are unsubscribing from.
This PR makes a couple of minor behavioural changes:

We always redirect to the latest subscription, irrespective of frequency.
We always show/redirect appropriately if the user is signed in.

I decided to cut this PR here, since the volume of change is already
quite large. Please see the commits for more details.

Just a minor comment that it's a blocker.

@kevindew thanks for the review - is there a blocking comment I'm missing?

Just a minor comment that it's a blocker.

@kevindew thanks for the review - is there a blocking comment I'm missing?


aiohttp server behind a reverse proxy: url_for() needs to emit the "real" URL

I am running an aiohttp server behind an SSL reverse proxy, so that https://real_server:80/foo/bar/baz is served by http://localhost:8080/bar/baz.
How do I teach my aiohttp server to return the "real" URL from url_for()?
(This is an enhancement issue for aiohttp's documentation. Also a feature request if this isn't yet possible. ;-)

url_for() returns a relative URL, isn't it?

I'd rather have one option (or a small snippet of code) in my app to prefix the URLs with, than to be required sprinkle every url_for in my templates with {{host_prefix}} variables. That doesn't scale well, and if these are missing anywhere nobody will notice until it's deployed and customers complain.
aiohttp can already do nested applications, so I just need a way to specify the prefix manually.

aiohttp does prepending prefixes for nested application but doesn't automatic adding these URLs absolute by adding scheme/host/port info.
E.g. for /name/{to} url_for() might return something like /name/John but never https://example.com/name/John.
You could specify secure_proxy_ssl_header and do something like req.url.join(req.app.url_for('name')) though.

maybe we should introduce some general rewriting rules for url generation,
that would help with virtual hosts.

For virtual host we need something like app.router.add_virtual_host('virtual.host.com', subapp) I think.
Implementation looks to be pretty easy but I personally have issues with higher priority in my virtual todo list.
The same for subdomains.

you don't want improvement issues to remain open?

What exact improvement are you talking about?
.url_for() should return a relative URL (as it does right now)

Hmm, maybe we need to resurrect aio-libs/aiohttp_jinja2#23 instead of changing url_for() behavior.
@smurfix what do you think?

Something like that, though I'd need a configurable fixed prefix instead of calculating it from request params, as those might have been mangled by my front-end server – the URL in the response needs to be the pre-mangled version.

Sounds good.
Your PR for aiohttp_jinja2 is welcome.
I believe your issue should be solved by template renderer


UI tweaks for open in web

@ehmadzubair please review. Just a couple minor tweaks I noticed later:

The cancel button needs to have .Cancel style so it gets separated out at the bottom
We don't want to hardcode a button width since it will take up different widths in different languages. This gives a standard margin no matter what the text is.

:+1:


Feature/subjects

I don't know how to tell you this, but this PR already has merge conflicts.

I have some concerns that this implementation locks us into a narrow definition of a taxonomy, but we need something to go forward with integration, so merging as is for now.
Also, in the future, please add a PR description to help with review. (tips for generating nested test data etc)


PList Requirements for inclusion in a Hardened Runtime

Hi socsieng!
Question:
This is an outstanding library that works quite well in development but when I attempt to build and notarize as a hardened runtime, it won't fully start. The window opens but nothing load. I have tried adding all of the Runtime Exception PList entitlements (https://developer.apple.com/documentation/security/hardened_runtime) but that does not help. Are there any hardened runtime/notarized apps that include this library as a dependency you know of? It would be super helpful to get some tips as to what I need to do to continue releasing my apps as hardened runtime/notarized, despite needing the functionality of sendkeys-macos.
When launching the app after building as a hardened runtime, the system console has errors like, "failed to lookup DARWIN_USER_CACHE_DIR" (subsystem com.apple.runningboard for my app) and "Failed to create LSApplicationRecord for ... 'The operation couldn't be completed.  (OSStatus error -10814.)"
Thank you!

Are there any hardened runtime/notarized apps that include this library as a dependency you know of?

No, not that I know of.
Unfortunately, I have no experience with app notarization.
How are you building/packaging your app? I wonder how hardening works with external app dependencies. In this case, the library has dependencies on NodeJS and osascript. Is that compatible with hardening?

This is an electron app and I use electron builder to package it. I tried removing the notarization step and told the builder not to build it as a hardened runtime and the same thing occurs. I sure wish I could get it working as it is absolutely flawless in the development environment.
I have many other NodeJS libraries so I know it will work with that. I wonder about osascript though. Is that compatible with Mac OS Big Sur?
Thanks so much for the quick reply by the way!

I wonder about osascript though. Is that compatible with Mac OS Big Sur?

I'm not currently on Big Sur but I've been told that it works.

K - it was not this library. Sorry for taking your time but I appreciate your support!
The issue was that I was loading things too early in my program so before electron was done with it's startup, I was trying to load this library even though I was not going to use it until later on.
It is now working exceptionally well. Thanks again for this AWESOME library and your super responsiveness!!

No problem. Glad the library is of use to you. Curious about how you are using it, are you able to share any details about the electron app you are building?

FYI, I've ported this tool over to Swift which includes support for mouse commands: https://github.com/socsieng/sendkeys


Mintomic on arm64

Hey there,
i'm trying to run mintomic on amr64 (iphone5s) ... i'm missing something?
in "platform_detect.h"
I got  #error Unrecognized target CPU!
and of course if add
#elif defined(arm) || defined (__arm64)
i got
#error Unrecognized ARM CPU architecture version!
I'll try to add another if
in order to define
#define MINT_CPU_ARM_VERSION 8
#define MINT_PTR_SIZE 8
but i don't have any chance to know if it will works properly :)
Thank you for your reply in advance :D

:+1: In general in would be nice to know, what the future of mintomic is. I think I started using mintomic in march, and as my luck would have it started to stale right from there :(
I put this into platform_detect.h
    #elif defined( __arm64__)
        // ARM
        #define MINT_CPU_ARM  1
        #define MINT_PTR_SIZE 8

        #if defined(__ARM64_ARCH_8__)
            // ARMv7
            #define MINT_CPU_ARM_VERSION 8
        #else
            // Could support earlier ARM versions at some point using compiler barriers and swp instruction
            #error Unrecognized ARM CPU architecture version!
        #endif
        #if defined(__thumb__)
            // Thumb instruction set mode
            #define MINT_CPU_ARM_THUMB 1
        #endif

But it doesn't compile.

If anyone succeeds in adding ARM64 support, I'd merge it!
In the meantime, C11 and C++11 are a lot more ubiquitous now than they were when I started Mintomic... I'm curious, what prevents you from using those?

Good to know that C11 (I don't do C++)  support is coming together. Last time I checked around for atomic operations, and that's admittedly a year ago, Xcode 6, did know about C11 but not about <stdatomic>. Apparently (having just googled this) it's C11 support is still only partial. But you are right, if mintomic is basically staling, I should migrate now. Quite frankly I also always assumed that  isn't inlining, but I just checked it, and at least with clang it does inline.
I did like mintomic and your articles though and used in successfully in my ABA solution. So thanks for all the fish ;)

"I'm curious, what prevents you from using those?"
Because using mintomic we know what's under the hood :D.
so do I've to give up about arm64?
thanks anyway.

Hi guys, as I mentioned elsewhere, Mintomic is nearing the end of its life. I've basically moved all the atomic C code to Turf: https://github.com/preshing/turf/tree/master/turf/c
I won't make any promises, but native ARM64 support is somewhat more likely to happen there!


Fix x-amz-expires header value

What this PR does / why we need it:
While working on #736, I noticed that the current authenticator is setting x-amz-expires header to 60 nanoseconds instead of 60 seconds.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
This value is ignored by STS, but this changes the signed header value to not be 0, but now be 60 seconds.

/lgtm

/approve


Literally.


Add RequestType and ResponseType to Endpoints and refactor test helpers ...

...to not use testify.TestSuite

:ship:


Bug: Remove constraint for phone numbers

Since we don't use phone numbers for authentication, they don't need to be unique as this causes an issue due to them also being nullable, something to fix if we ever implement phone auth.

Closing as the project is being archived. https://github.com/caendesilva/mutual-aid-app/issues/48


how to use this model to train and evaluate on Imagenet?

Thank you for providing this network!
I want to use this model "ViT" to classify Imagnet,but the accuracy is not good.I try it on mini-Imagenet first.
I use the network like this:
net = ViT(
image_size = 224,
patch_size = 16,
num_classes = 64,
dim = 1024,
depth = 6,
heads = 8,
mlp_dim = 2048,
dropout = 0.1,
emb_dropout = 0.1
).cuda(device)

But the accuracy just stops increasing.

But the accuracy just stops in

creasing.

@Alexanzhuo Hi Alex, you won't see any positive results in the small data regime.
Or you can just use Ross' version with the pretrained weights released by Google


TypeError: make_grid() got an unexpected keyword argument 'range'

TypeError: make_grid() got an unexpected keyword argument 'range'

Hi @wanlipeng ,
Can you please refer to the specific file and line that causes the error?

@wanlipeng Not sure if this is still relevant, but replace "value" with the kwarg "value_range" instead wherever the torchvision.utils.save_image function is called. You probably have a newer torchvision version.


Ibnopcit/average loop length

A solution for average loop length

Thanks this looks good :) Thanks for including tests.


Implement a way to add your score to the leaderboard

This should start an event at the end of a match (@ Game over phase) and ask the user their username and added it to the leaderboard.

No leaderboard needed, game should stay simple as it is


sidr causing body scroll when open in mobile view

while opening the sidr in mobile it causing a horizontal scroll in body. The toggle menu is functioning but a scroll in body failing all its purpose.

It's work for me.
Thanks

this doesnt work with iOS 8.4 for some reason


AppsFlyer events not showing up on dashboard

Report
Plugin Version
5.4.40
On what Platform are you having the issue?
ℹ Please replace these two lines with the SDK version.
e.g. ios and android
What did you do?
ℹ Please replace these two lines with what you did.
e.g. Run pod install
What did you expect to happen?
ℹ Please replace these two lines with what you expected to happen.
e.g. Event to be logged
What happened instead?
ℹ Please replace these two lines with of what happened instead.
e.g. No uninstalls on my dashboard
Please provide any other relevant information.
ℹ Please replace these two lines with more information.
e.g. The issue started when we upgraded the plugin to the latest version

hi @amitpatil77 ,
In order to help you in the best way, please contact our support team<EMAIL_ADDRESS>When you opening the ticket, please attached code snippets of your implementation and AppsFlyer debug logs of a session that you sending events to our servers.


Fix tag opening a blank page on block page.

Task description
On clicking on the Tag in Tagged Data Payload on block page it leads to the page with no content.
Requirements
N/A
Acceptance criteria
Disable the click on tag in tagged data payload.
Creation checklist
I have assigned this task to the correct people
I have added the most appropriate labels
I have linked the correct milestone and/or project

Needs eslint fix from https://github.com/iotaledger/explorer/pull/537


Inline action button not showing up in IE, only Chrome

I have a user which reported inline button is not showing up in IE but only in Chrome.  I am not sure what could be the issue.  Just in case anyone already ran into this issue, please share your solution.  I am going to do a site visit also and see what the problem is and report back.
    protected getColumns() {
        var columns = super.getColumns();
        var fld = MyRow.Fields;

        columns.unshift({
            field: 'MyButton',
            name: '',
            format: ctx => '' +
                '',
            width: 24,
            minWidth: 24,
            maxWidth: 24
        });
    }

It looks to be a font awesomeness issue with IE11.  The columns are there bit icons are now appearing. Trying different solution found online to no avail...

Try playing with CSS, or other rules that may be changing your inline buttons.


ERROR: Installation aborted, plugin 'logstash-input-jdbc' is already provided by 'logstash-integration-jdbc'

Description:

Can not build logstash
Expected Behavior:

Can build
Context information:
Output of git rev-parse HEAD
df4ccf5eb7857d459e2b1d15c17ac07655d030bf

Output of docker version
Client:
 Cloud integration: 1.0.17
 Version:           20.10.8
 API version:       1.41
 Go version:        go1.16.6
 Git commit:        3967b7d
 Built:             Fri Jul 30 19:55:20 2021
 OS/Arch:           darwin/amd64
 Context:           default
 Experimental:      true

Server: Docker Engine - Community
 Engine:
  Version:          20.10.8
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.16.6
  Git commit:       75249d8
  Built:            Fri Jul 30 19:52:10 2021
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.4.9
  GitCommit:        e25210fe30a0a703442421b0f60afac609f950a3
 runc:
  Version:          1.0.1
  GitCommit:        v1.0.1-0-g4144b63
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Output of docker-compose version
docker-compose version 1.29.2, build 5becea4c
docker-py version: 5.0.0
CPython version: 3.9.0
OpenSSL version: OpenSSL 1.1.1h  22 Sep 2020

System info: Mac, Windows or Linux. Include which disto/version
macOS Monterey Version 12.2.1

Steps to reproduce the issue:

run docker-compose build logstash

Stacktrace & Additional info:
docker-compose build logstash                                                                                 5.74     4.55G   3.05     06:44    03.12.22 
Building logstash
[+] Building 84.0s (11/12)
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 37B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/logstash:7.9.1                                                                                3.7s
 => [auth] library/logstash:pull token for registry-1.docker.io                                                                                0.0s
 => [1/7] FROM docker.io/library/logstash:7.9.1@sha256:e2caec02269f054d3381583a6dbdb30b050e77c65172047143f300004eb0df31                                                                                0.0s
 => [internal] load build context                                                                                0.0s
 => => transferring context: 129B                                                                                0.0s
 => CACHED [2/7] RUN rm -f /usr/share/logstash/pipeline/logstash.conf                                                                                0.0s
 => CACHED [3/7] RUN curl -L -o /usr/share/logstash/lib/mysql-connector-java-5.1.47.jar https://repo1.maven.org/maven2/mysql/mysql-connector-java/5.1.47/mysql-connector-java-5.1.47.jar                                                 0.0s
 => CACHED [4/7] ADD ./pipeline/ /usr/share/logstash/pipeline/                                                                                0.0s
 => CACHED [5/7] ADD ./config/ /usr/share/logstash/config/                                                                                0.0s
 => ERROR [6/7] RUN logstash-plugin install logstash-input-jdbc                                                                                80.1s
------
 > [6/7] RUN logstash-plugin install logstash-input-jdbc:
#11 0.417 OpenJDK 64-Bit Server VM warning: Option UseConcMarkSweepGC was deprecated in version 9.0 and will likely be removed in a future release.
#11 7.339 WARNING: An illegal reflective access operation has occurred
#11 7.339 WARNING: Illegal reflective access by org.jruby.ext.openssl.SecurityHelper (file:/usr/share/logstash/vendor/jruby/lib/ruby/stdlib/jopenssl.jar) to field java.security.MessageDigest.provider
#11 7.339 WARNING: Please consider reporting this to the maintainers of org.jruby.ext.openssl.SecurityHelper
#11 7.339 WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
#11 7.339 WARNING: All illegal access operations will be denied in a future release
#11 12.06 Validating logstash-input-jdbc
#11 80.04 ERROR: Installation aborted, plugin 'logstash-input-jdbc' is already provided by 'logstash-integration-jdbc'
------
executor failed running [/bin/sh -c logstash-plugin install logstash-input-jdbc]: exit code: 1
ERROR: Service 'logstash' failed to build : Build failed

ERROR: Installation aborted, plugin 'logstash-input-jdbc' is already provided by 'logstash-integration-jdbc'

Having the same issue, working on the fix.


Admin Logs register every attack even if it doesn't hit.

1: Spam click attack someone.
2: Open Admin Logs.
3: Recoil in horror as you see the amount of logs for "attacked" despite only being hit a couple times.

I wonder if there's a way to add log stacking for multiple that happen in quick succession in the same area or if that'd be too vague when trying to recreate the scene.


Don't check for updates after having checked for updates

When I'm notified of package updates in the bottom right corner, it's always an anti-climax to click it and see a "Checking for updates..." hourglass appear.
If Atom has already checked for updates and it knows there are and X number of package updates, just show those package updates straight away without checking again and making the user wait needlessly.

Thanks for the suggestion! This is a duplicate of https://github.com/atom/settings-view/issues/409 – please subscribe there for updates.

Thanks, you're right, I missed that one when searching for existing issues.


ZeroDivisionError: float divmod()

Dear Author,
When I run the last part of the code from frame import FRAME
file_path =  "./videos/nice_road.mp4"# <== Upload appropriate file
video_out = "videos/output11.mov"
frame =  FRAME(
ego_vehicle_offset = .0,                       # offset for self vehicle in frame
yellow_lower = np.uint8([ 20, 50,   110]),      # LOWER YELLOW HLS THRESHOLD
yellow_upper = np.uint8([35, 255, 255]),        # UPER YELLOW HLS THRESHOLD
white_lower = np.uint8([ 0, 140,   0]),         # LOWER WHITE THRESHOLD
white_upper = np.uint8([180, 255, 100]),        # UPPER WHITE THRESHOLD
lum_factor = 110,                               # NORMALIZING LUM FACTOR
max_gap_th = 0.45,                              # MAX GAP THRESHOLD
YOLO_PERIOD = 2,                              # YOLO PERIOD
lane_start=[0.2,0.5] ,                        # LANE INITIATION
verbose = 3)                                  # VERBOSITY
frame.process_video(file_path, 2,video_out = video_out,pers_frame_time =398, t0  =398 , t1 =698)#None)                  , I am getting the following error: "ZeroDivisionError                         Traceback (most recent call last)
 in ()
14     lane_start=[0.2,0.5] ,                        # LANE INITIATION
15     verbose = 3)                                  # VERBOSITY
---> 16 frame.process_video(file_path, 2,video_out = video_out,pers_frame_time =398, t0  =398 , t1 =698)#None)
/content/drive/MyDrive/copilot/frame.py in process_video(self, file_path, fps_factor, video_out, pers_frame_time, t0, t1)
232         frame_h_out =  int(frame_h*(1-self.ego_vehicle_offset))
233         print("{:s} WIDTH {:d} HEIGHT {:d} FPS {:.2f} DUR {:.1f} s".format(
--> 234             file_path,frame_w,frame_h,fps_actual,nb_frames//fps_actual
235             ))
236
ZeroDivisionError: float divmod()". Kindly help me out in the same.

Instead of the line:     file_path = "./videos/nice_road.mp4"
Use:    file_path = "./videos/nice_road"
This solved the error for me!


Add config option for Browserstack to support native automation

What is your Scenario?
When running latest version of TestCafe with nativeAutomation enabled and using browserstack remote browser provider, the following error occurs:
Error: The "browserstack" do not support the Native Automation mode. Remove the "native automation" option to continue.
What are you suggesting?
Add a configuration option to support this new mode, or release a new module that supports it.
What alternatives have you considered?
AWS DeviceFarm, Docker localhost headless
Additional context
I have created another issue here:
https://github.com/DevExpress/testcafe-browser-provider-browserstack/issues/199
I felt it was necessary to mention, however, since the code to throw the error and block native automation in browsertstack was added to testcafe and not the browser provider itself.

Hello,
At present, TestCafe supports the Native Automation mode only for local Chromium-based browsers. Please refer to the following help topic to learn more about Native Automation mode limitations: https://testcafe.io/documentation/404237/guides/intermediate-guides/native-automation-mode#limitations.

This is a new feature request. Are you saying that you are not going to support native automation in remote browsers moving forward? If you plan on supporting this feature then you should probably leave the issue open.

Currently, we do not have plans to support remote browsers. We are fully focused on Chrome-based native automation mode. Our plans can change in the future. However, at present, there is no need to keep this issue opened.

We discussed the issue with the team once again and decided to keep it open in order to track user interest.

The lack of remote browser support further complicates the visual regression testing story in TestCafe. To test reliably, you need a 100% reproducible environment, both hardware and software (i.e. the Docker image). Right now, the options are managing your own browser pool or using a third-party browser pool, like Browserstack. Lack of the remote browser support makes the second option impossible. TestCafe users have to choose between improved test stability brought by the NA mode, supporting more infrastructure or forgoing the visual regression testing,

any updates about planning to support remote browsers? we are using selenium grid for testcafe tests run , we have several issue related to proxy and using native automation would be really  important to us

We also need native automation for remote browsers, specifically Browserstack.

@wombatka
At present, we cannot give any precise estimates on when TestCafe will support any other browsers except Chromium-based.

Any updates? Any actions will be taken? or there are still no plans at all to support native automation in remote browsers?

We have not researched the issue yet, since we were focused on other tasks.

Wanna second the request for remote browsers (we use Saucelabs instead of BrowserStack, but still).


Add "frozen" warning

LGTM as soon as the README for ripple-lib is updated with the new docs.

+2

@clark800 @geertweening Can I use ripple-lib in react native? Its only support for node.js


Add moment.inspect() for friendly console output

This gets used automatically by node's util.inspect function and a number of similar libraries. This also means that it will be used by the NodeJS REPL.
As discussed in #3229.
Please be as brutal as you like with code-review 😄 .
Couldn't think of a better way to test the round-trippable nature of this format than eval, so I've kept it constrained to that small helper function.
Is there any other property of a moment that's important to include in the representation?

@glenjamin the code looks good. If you had just said nothing I'd probably merge it. But you asked what else you can add, so here you go ;-)
Invalid moments
> moment('Invalid date').format()  // prints an exception and then works
> moment.invalid()  // this is the best way to produce an invalid moment

Also note that for some reason node REPL will not be very happy with moment.invalid(), or any other invalid moment, or new Date(NaN).
local time or fixed zone offset
> moment() // produces local time
> moment.utc() // produces fixed utc offset == 0
> moment.utc().utcOffset(4) // produces fixed utf offset +04:00
> moment.utc('2016-06-12T04:44:33+01:00').format()  // '2016-06-12T03:44:33+00:00' note the zone
> moment.utc('2016-06-12T04:44:33+01:00').parseZone().format()  // '2016-06-12T04:44:33+01:00' note the zone
> moment('2016-06-12T04:44:33+01:00').parseZone().format()  // '2016-06-12T04:44:33+01:00' same as above

So basically if its offset 0 you can do moment.utc(fmt), and if its not offset 0 you can do moment(fmt).parseZone(). If moment-timezone is enabled, and the moment is zoned then you have to do something like http://momentjs.com/timezone/docs/#/using-timezones/parsing-in-zone/
locales
You can pass locales to the constructor, the slight catch is that there is also a global locale, so to be extra safe you should always specify it in the constructor (if you want copy-paste to give you the exact same moment).
> moment('2016-06-11T21:03:28-07:00', moment.ISO_8601, true, 'en')  // longer but safer
> moment('2016-06-11T21:03:28-07:00').locale('en')  // shorter but might not work

The second one won't work if you locale has preparse and postformat. But if you use toISOString, your code will be shorter and you'll avoid this issue.

@ichernev I think you know a good amount more about node than me. The one concern I had with this PR is that we might have support issues if there isn't a good way to get a screenshot of all the properties on the object in Node. I know I frequently need the values of _pf or _i or _f to figure out what someone was doing when troubleshooting. Does this make things more complicated?  And also fix invalid moment inspect by just presenting the parsing flags and saying its invalid. We could add a way moment.locale.debug = true that makes locale inspect verbose again, if you happen to be debugging the locale, but I'm rarely using that anyway, so not sure how useful it is.

I've expanded the coverage of inspect() a bit more, quick link to examples
I've not attempted to tackle locales, as I wasn't entirely clear on the advice. Is the suggestion to always specify the 8601 format and the locale? Would it be feasible to push locale support back to a later addition if it turns out to be desired?

@glenjamin sorry to come back to this so much later, but yes, there is a difference between parseZone and moment().
I'm on pacific time, which is UTC-7 right now because DST is in effect. Observe the following code:
moment("2016-07-30T07:41:32-07:00").add(6, 'months').format()
"2017-01-30T07:41:32-08:00"

moment.parseZone("2016-07-30T07:41:32-07:00").add(6, 'months').format()
"2017-01-30T07:41:32-07:00"

As you can see, parse zone actually doesn't explicitly set a time zone. Instead, it sets an offset that does not change, even when the offset of my time zone has changed.

If i'm following that correctly, one is a local time, and the other is a UTC time combined with an offset? (rather than a named timezone).
Is it possible to portably serialise that without the timezones extensions?
The scenario I'm thinking of is spitting out a time on one computer, and then evaluating it on another that might be in a different timezone.
Is it preferrable to keep the time as a local time, or keep the offset but lose the local-ness?

@glenjamin you can just use format without Z token.

@glenjamin now I see what's your question -- if its local it should be printed as one -- moment(ISO FORMAT). If you copy paste that to another timezone ... not my problem :)

@glenjamin this is almost ready, would you finish it up?

Yeah, sorry!
I thought it had merged - forgot about the last bit

On 29 Oct 2016, at 02:45, Iskren Ivov Chernev<EMAIL_ADDRESS>wrote:
@glenjamin this is almost ready, would you finish it up?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@ichernev - that should be all sorted now.
I did end up using isLocal, which I notice isn't documented - I wasn't sure whether it it supposed to be part of the public API or not, but I guess that doesn't matter heavily for this implementation.

Yes, isLocal is the right thing to use. Our documentation can be improved :/
Thank you for finishing it up!

Heads up, because we came under JS Foundation management, we need a CLA before this can be merged. Should be pretty easy to do.

Sorted.

Very nice. Thank you @glenjamin for working on this. It will be merged in next release.

Merged in 70c0f76ff0864b84e6b5d6e020c39f4843d72324


RSA-Decrypt Request body middleware

This was working for Argon2 password hash, the other routes that are currently being called from the C# SDK need to be implemented.

Ran into an issue with the byte size being too long to encrypt with RSA encryption. Need to AES. Can implement Hybrid encryption with AES256 since we already have the key pairs generated.

AES key should not be generated on the server. It should be generated via client side. Do not store public key in JWT send in request body when asking for a token.


fixed querying for safe address

Fix for the 500 server error that happened when address was not specified.
2020-08-11 09:35:55,797 [ERROR] [MainProcess] Internal Server Error: /api/v1/notifications/devices/799eb559-0128-4013-b356-f4c5f908cc06/safes/0x7348C4a88858Bc3E105150D50860FbcaA0327c2a/

Traceback (most recent call last):

  File "/usr/local/lib/python3.8/site-packages/django/core/handlers/exception.py", line 34, in inner

    response = get_response(request)

  File "/usr/local/lib/python3.8/site-packages/django/core/handlers/base.py", line 115, in _get_response

    response = self.process_exception_by_middleware(e, request)

  File "/usr/local/lib/python3.8/site-packages/django/core/handlers/base.py", line 113, in _get_response

    response = wrapped_callback(request, *callback_args, **callback_kwargs)

  File "/usr/local/lib/python3.8/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view

    return view_func(*args, **kwargs)

  File "/usr/local/lib/python3.8/site-packages/django/views/generic/base.py", line 71, in view

    return self.dispatch(request, *args, **kwargs)

  File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 505, in dispatch

    response = self.handle_exception(exc)

  File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 465, in handle_exception

    self.raise_uncaught_exception(exc)

  File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 476, in raise_uncaught_exception

    raise exc

  File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 502, in dispatch

    response = handler(request, *args, **kwargs)

  File "/usr/local/lib/python3.8/site-packages/rest_framework/generics.py", line 217, in delete

    return self.destroy(request, *args, **kwargs)

  File "/usr/local/lib/python3.8/site-packages/rest_framework/mixins.py", line 91, in destroy

    self.perform_destroy(instance)

  File "/app/safe_transaction_service/notifications/views.py", line 37, in perform_destroy

    safe_contract = SafeContract.objects.get()

  File "/usr/local/lib/python3.8/site-packages/django/db/models/manager.py", line 82, in manager_method

    return getattr(self.get_queryset(), name)(*args, **kwargs)

  File "/usr/local/lib/python3.8/site-packages/django/db/models/query.py", line 419, in get

    raise self.model.MultipleObjectsReturned(

safe_transaction_service.history.models.SafeContract.MultipleObjectsReturned: get() returned more than one SafeContract -- it returned more than 20!

Change makes sense, but I would not merge it as we cannot deploy it and uxio should be aware of this change

I would not merge it as we cannot deploy it and uxio should be aware of this change
ok

I would propose to add a test for this
I'll need Uxio's help to do this

I added a test for this, thank you @DmitryBespalov for your fix


Bluetooth: host: Fix Limited Advertising timeout cancel

Timeout cancel should only be done for connections established in
peripheral role.
Enhanced connection complete event could still be delivered without
extended advertising support (i.e no advertising set terminated event)
so this handling should be moved to the common conn complete function.
Signed-off-by: Michał Narajowski<EMAIL_ADDRESS>

Thanks @michal-narajowski, could you add the fixes #issue as well to link the issue?


Make form responsive on Mobile

It's a CSS thing to make current moz-connect form responsive on mobile

Can mentor this.


Right-click on variable getter should only show delete option in the palette

Expected Behavior
Right-clicking on the variable getter in the palette should show "rename" and "delete".  Right-clicking on the variable getter in the workspace should only show the names of other variables.
Actual Behavior
The delete and rename options are always available.
Steps to Reproduce
Drag a variable getter block into the workspace and right-click on it.
Right-click on a variable getter block in the palette.
Operating System and Browser
All

(..I accidentally interpreted this title as "variable getters in the palette" "should only show delete option" 🙃 ..)

Quick note: When right clicking a variable in the getter block in the palette, nothing shows up in the context menu.

Update: Rename Variable currently is not an option for either of these context menus.


Stripe js submit another form in the page

In the same page I present to the user the previous "Pay With PayPal" button and the new "Pay With Stripe" button.
The PayPal button is a very simple form that post to PayPal.com.
Randomly, when i click on the "Pay With Stripe" button it submit the "PayPal" form and redirect me to PayPal payement page.
PS
I have an adblocker in my browser as many of users now have.
Please make a way to integrate stripe server side without relayng on a client javascript.

Hi @marcostaff, for help with your integration please contact Stripe Support. We leave this issue tracker open for bugs in the @stripe/stripe-js loader library only. Thanks!


/dining/hours 404 Error

The most recent changes are not yet deployed to production, deploying to production fixes these issues.

Should work now, for example /dining/hours/1 produces the correct output.


Errors running watchify with node 0.12.0

When I run watchify commands with latest node, I get these errors:
dyld: lazy symbol binding failed: Symbol not found: __ZN2v86Object3SetENS_6HandleINS_5ValueEEES3_NS_17PropertyAttributeE
  Referenced from: /usr/local/lib/node_modules/watchify/node_modules/chokidar/node_modules/fsevents/build/Release/fse.node
  Expected in: dynamic lookup

dyld: Symbol not found: __ZN2v86Object3SetENS_6HandleINS_5ValueEEES3_NS_17PropertyAttributeE
  Referenced from: /usr/local/lib/node_modules/watchify/node_modules/chokidar/node_modules/fsevents/build/Release/fse.node
  Expected in: dynamic lookup

dyld: lazy symbol binding failed: Symbol not found: __ZN2v86Object3SetENS_6HandleINS_5ValueEEES3_NS_17PropertyAttributeE
  Referenced from: /usr/local/lib/node_modules/watchify/node_modules/chokidar/node_modules/fsevents/build/Release/fse.node
  Expected in: dynamic lookup

dyld: Symbol not found: __ZN2v86Object3SetENS_6HandleINS_5ValueEEES3_NS_17PropertyAttributeE
  Referenced from: /usr/local/lib/node_modules/watchify/node_modules/chokidar/node_modules/fsevents/build/Release/fse.node
  Expected in: dynamic lookup

sh: line 1: 17268 Trace/BPT trap: 5       watchify bg.js -o build/bg.js -v -d

Switching back to 0.11.13 gets rid of the errors, though.

Whatever version of watchify you are/were running was installed (and thus built) with a different version of node. When you switch node versions you run the risk of breaking binary modules. Just delete the old watchify and reinstall.


Document debugging JavaScript

Debugging JavaScript requires steps different from that of debugging PHP. Make that distinction and document how to debug clinet andd server side JavaScript.

Also added a wiki page for this:
https://github.com/apolopena/gitpod-laravel-starter/wiki/Debugging-JavaScript


Complex numbers: hermitian and unitary

Hello,
Thanks for the great library!
Do you plan to support cases on complex numbers like hermitian, skew-hermitian and unitary?
Thank you

There is no plan to support those yet, but I'd happily take a PR adding them.
Here you have an example of a PR that adds a new manifold to the library:
https://github.com/Lezcano/geotorch/pull/31


add string to number conversion function

cleaned up version of #3782
[ch1619]

@rintcius I'm on mobile and I can't find the approval button, but I approve this. Can you (or anyone else) merge this? Thanks :)


Dockerfile in the project root directory not used

If the user has its own Dockerfile in the project root, that file is not used when building the app:
$ ./gradlew dockerBuild
...

> Task :dockerBuild
Building image using context '/home/ivan/workspaces/micronaut-graal-tests/micronaut-basic-app'.
Using Dockerfile '/home/ivan/workspaces/micronaut-graal-tests/micronaut-basic-app/build/docker/Dockerfile'
...

The file build/docker/Dockerfile is used instead of the one in the project root directory.
Note the same applies when running dockerBuildNative, although in this case I'm not really sure. I mean, if the user wants to use their own dockerfile and put it in the project root directory, they should run the task dockerBuild and once this bug is fixed, the right file will be used.
$ ./gradlew dockerBuildNative
...

> Task :dockerBuildNative
Building image using context '/home/ivan/workspaces/micronaut-graal-tests/micronaut-basic-app'.
Using Dockerfile '/home/ivan/workspaces/micronaut-graal-tests/micronaut-basic-app/build/docker/DockerfileNative'

I can't reproduce this personally

Steps to reproduce the issue:

git clone https://github.com/micronaut-graal-tests/micronaut-basic-app
cd micronaut-basic-app
git checkout micronaut-gradle-plugin
There is no Dockerfile in the project root directory.
./gradlew dockerfile
cp build/docker/Dockerfile .
./gradlew dockerBuild

> Task :dockerfile

> Task :dockerBuild
Building image using context '/tmp/micronaut-basic-app'.
Using Dockerfile '/tmp/micronaut-basic-app/build/docker/Dockerfile' <-------------
Using images 'basic-app'.
Step 1/7 : FROM openjdk:14-alpine
 ---> 8273876b08aa
...

You can see that the build is using /tmp/micronaut-basic-app/build/docker/Dockerfile instead of /tmp/micronaut-basic-app/Dockerfile

This seems to be erroneous output coming from the Docker gradle plugin. If you add:
RUN echo "BUILDING WITH DOCKER!!!"

To the Dockerfile it prints, so the correct Dockerfile is being used.


Include requester ip address in ForwardHandler received request debug log

Expected behavior
As a bosh-dns user

in order to easily identify DNS client that make unexpected DNS request to a bosh-dns process
I need the IP address of the remote machine that made this request

Observed behavior
The logs don't yet include the source IP address of the received DNS request
[ForwardHandler] 2021-10-15T12:53:07.404484142Z DEBUG - handlers.ForwardHandler Received request id=14476 qtype=[AAAA] qname=[api.github.com.]
[ForwardHandler] 2021-10-15T12:53:07.407406989Z DEBUG - handlers.ForwardHandler Received request id=30384 qtype=[A] qname=[api.github.com.]

Context
A bosh-dns job is acting as a DNS resolver for multiple client vms. Client vms are expected to use http proxy for internet traffic and only resolve intranet FQDNs through the bosh-dns resolver. Identifying client vms that incorrectly request internet FQDNs is time consumming.
Workaround
Use tcpdump traces

Hi Guillaume!
It seems that the work submitted by SAP in #78 could be a good base for your need. You would just have to add the source IP address to the produced log lines.
Would that fit your requirement?

@bgandon The requirement to log the source ip address seems independent to me of the logging configuration options. I'm however happy if this ends up being bundled within a PR offering a new "recursion-logging" opt-in

Hi @gberche-orange,
I was trying to understand when this additional information can help but I can't find an useful use case. Usually, you have following two situations to look into bosh-dns logs

on a bosh VM itself and there the request ip is clear
bosh-dns logs forwarded to a logging service. In this case the forwarder can enrich the log entries with the VM ip from where the logs are coming

Do you have a different use case?

But wait a minute, is a Bosh-DNS job really able to act as a DNS resolver for multiple client VMs?
Possibly you've made an invalid assumption in the issue description above @gberche-orange, because Bosh DNS server is only available for queries performed on localhost. This means that when you see a DNS query log line, then you don't need the IP address of any remote client that has made the request. The client cannot be remote. That IP address can only be <IP_ADDRESS>, so you can conclude that a process on the localhost is responsible for sending the DNS query.
Indeed, when you inspect the TCP/UDP sockets on some Bosh instance, the DNS server is only bound to the <IP_ADDRESS> IP address as shown below:
# netstat -ntulp | sort -n -t: -k2
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN      151/sshd
tcp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*               LISTEN      2125/bosh-dns
tcp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*               LISTEN      2125/bosh-dns
tcp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*               LISTEN      2125/bosh-dns
tcp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*               LISTEN      2125/bosh-dns
tcp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*               LISTEN      2125/bosh-dns
tcp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*               LISTEN      2125/bosh-dns
tcp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*               LISTEN      2125/bosh-dns
udp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*                           2125/bosh-dns
udp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*                           2125/bosh-dns
udp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*                           2125/bosh-dns
udp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*                           2125/bosh-dns
udp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*                           2125/bosh-dns
udp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*                           2125/bosh-dns
udp        0      0 <IP_ADDRESS>:53          <IP_ADDRESS>:*                           2125/bosh-dns
tcp        0      0 <IP_ADDRESS>:2822          <IP_ADDRESS>:*               LISTEN      2178/monit-actual
tcp        0      0 <IP_ADDRESS>:2825          <IP_ADDRESS>:*               LISTEN      56/bosh-agent
tcp        0      0 <IP_ADDRESS>:8853            <IP_ADDRESS>:*               LISTEN      2101/bosh-dns-healt
tcp        0      0 <IP_ADDRESS>:53080         <IP_ADDRESS>:*               LISTEN      2125/bosh-dns

Plus, the <IP_ADDRESS> is assigned to the loopback lo network device, as show below:
# ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/32 scope global lo
       valid_lft forever preferred_lft forever
...

Having said that, my understanding could possibly be wrong, or you could also need to track bad clients that manage to send DNS queries remotely for some reasons that I'm not aware of.
Guillaume, could you detail further your requirement considering the details shown above?

@bgandon @beyhan missed to follow up. In our use case, we use bosh-dns exposed publicy, to enable bosh dns resolution from other directors (we have chained architecture micro-bosh => bosh master).


Block Stage 1 - Room 4

Level 1.zip
@vitormaduro uploading the level blocking till room 4 here just for backup purposes


Update wascap dependency to 0.6.0 to support wasmcloud:* contract IDs

Once https://github.com/wasmCloud/wascap/pull/32 is merged, and wascap 0.6.0 is published to crates.io, wash should incorporate this update as the shorthand flags for wash claims sign still have the wascc:* prefixes for contract IDs.

Fixed by #69


Mail Verification Template

Describe the Change and Related Issue Number :

I worked on the mail template

Type of change :

[ ] Bug fix
[x] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Preview (Screenshots) :

Checklist:

[x] I have read the Code Of Conduct document.
[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] All new and existing tests passed.

@PraveenMalethia


Heading anchors don't work for headings with code snippets

For example, see the dyn* heading in https://theincredibleholk.org/blog/2022/12/19/async-fn-in-trait-object-update/

It'd probably be good to fall back onto something like #anchor1 if for some reason we can't extract any text from a heading. We might want a way to deduplicate anchors as well.


clean the usage of "nothing" and "missing"

@claytonpbarrows What do you mean?

@claytonpbarrows I think this issue can be closed. The codebase uses nothing instead of missing, and based on my understanding from our discussions, that is by design.  Do you agree?

agreed


fixed analysis page figure

changed the figure on the analysis page so it is the updated version.

Codecov Report

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

@@           Coverage Diff           @@
##              dev    #2578   +/-   ##
=======================================
  Coverage   94.42%   94.42%
=======================================
  Files         160      160
  Lines       19477    19477
=======================================
  Hits        18391    18391
  Misses       1086     1086

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 1cc4315...387c1f4. Read the comment docs.


png: Separate and unbundle libpng from Godot loader

Related to #6157. Also added the libpng LICENSE file.
Basically what this PR does is:

move all upstream libpng files from drivers/png to drivers/png/thirdparty/libpng, thus making it clear which files are Godot's, and which aren't
adapt the SCsub accordingly
allow to build against the system-wide libpng version (on Linux only for now, other platforms would likely error out if png=system is passed)

Feedback welcome.

Ideally this should be moved to a module as proposed in #6157, but I'm not sure yet how to do it exactly, will have to experiment a bit.

Closing as this was more a proof of concept, and we settled on an even better scheme IMO in #6157 (with all external code moved to a module-like thirdparty/ folder).


Create closable pill

Description
The Pill should support a closeable action. This should be achieved by allowing the addition of a close button to the end of the Pill.
Requirements:

When clicking the close button it should allow for deletion to be recognised.
The decisions made on the API/structure should allow the label to have a clickable action in the future (with the correct semantics)

Dependency of #2541
Tasks
### Checklist
- [ ] Density (HD, MD, LD, TD)
- [ ] Mode (Light, Dark)
- [ ] Fill (Primary, Secondary, Tertiary)
- [ ] State (Default, Hover, Selected/Active, Disabled, Read-only, Other)
- [ ] Controlled API
- [ ] UITK accessibility parity (screen reader + keyboard interactions)

### Deliverables
- [ ] Characteristics spec
- [ ] Interactions spec, where necessary
- [ ] In Code & Figma (Component + Guide)
- [ ] Supporting site documentation

23 Oct - wip, api review?

26 Oct - Storybook example discussion with @Fercas123

When using keyboard to navigate, the parts that are focused with focus ring should be hover state
The example can be displayed as Pill group and some can show icons

Push target date by 1d, to align with OOO holidays

PR ready to merge, follow up PR for documentation


atomic_xchg doesn't behave as expected

The kernel below dead locks (when the lock matrix is big enough), and my understanding is that it should not.
If you comment the else branch part and print the lock matrix, you will notice that it's full of 1 and 2s, where it's expected to have it full of 2s.
import torch
import triton
import triton.language as tl

@triton.jit()
def dead_lock(
        locks,
        size: tl.constexpr,
        ):
    pid = tl.program_id(0)
    pid = pid % size  # every lock position got 2 CTAs accessing it
    if tl.atomic_xchg(locks + pid, 1) == 0:  # original value is 0
        tl.atomic_xchg(locks + pid, 2)
    else:
        while tl.atomic_xchg(locks + pid, 3) != 2:  # supposed to return the old value (2 at this point)
            pass

lock = torch.zeros(800, dtype=torch.int32, device='cuda')

dead_lock[(lock.numel()*2,)](lock, lock.numel())
print(lock)

Test ran on 3090 RTX and triton main branch.
related to https://github.com/openai/triton/issues/1393 and https://triton-lang.slack.com/archives/C050BRSAH4Z/p1680168403507209

I cannot reproduce this branch using triton/main. Can you double check?

Also the PTX is a bit strange, at the end there are 2 consecutive calls to barrier sync, the second one seem to be redundant. Does it matter or is it something that disappear at SASS level?
//
// Generated by LLVM NVPTX Back-End
//

.version 8.0
.target sm_86
.address_size 64

	// .globl	dead_lock_0d
.extern .shared .align 1 .b8 global_smem[];

.visible .entry dead_lock_0d(
	.param .u64 dead_lock_0d_param_0
)
.maxntid 128, 1, 1
{
	.reg .pred 	%p<6>;
	.reg .b32 	%r<18>;
	.reg .b64 	%rd<6>;

	ld.param.u64 	%rd3, [dead_lock_0d_param_0];
	mov.u32 	%r6, %ctaid.x;
	mul.hi.s32 	%r7, %r6,<PHONE_NUMBER>;
	shr.u32 	%r8, %r7, 31;
	shr.s32 	%r9, %r7, 8;
	add.s32 	%r10, %r9, %r8;
	mul.lo.s32 	%r11, %r10, 800;
	sub.s32 	%r12, %r6, %r11;
	mul.wide.s32 	%rd4, %r12, 4;
	add.s64 	%rd5, %rd3, %rd4;
	mov.u32 	%r1, %tid.x;
	membar.gl ;
	setp.eq.s32 	%p4, %r1, 0;
	mov.u32 	%r3, 1;
	mov.u32 %r2, 0x0;
	@%p4 atom.global.gpu.exch.b32 %r2, [ %rd5 + 0 ], %r3;
	mov.u32 	%r4, global_smem;
	@%p4 st.shared.b32 [ %r4 + 0 ], %r2;
	bar.sync 	0;
	ld.shared.u32 	%r13, [global_smem];
	bar.sync 	0;
	setp.ne.s32 	%p3, %r13, 0;
	@%p3 bra 	$L__BB0_2;
	bar.sync 	0;
	membar.gl ;
	mov.u32 	%r15, 2;
	mov.u32 %r17, 0x0;
	@%p4 atom.global.gpu.exch.b32 %r17, [ %rd5 + 0 ], %r15;
	@%p4 st.shared.b32 [ %r4 + 0 ], %r17;
	bar.sync 	0;
	bar.sync 	0;
$L__BB0_2:
	ret;

}

Just a suggestion, probably this is what you want?
@triton.jit()
def dead_lock(
        locks,
        size: tl.constexpr,
        ):
    pid = tl.program_id(0)
    pid = pid % size  # every lock position got 2 CTAs accessing it
    if tl.atomic_cas(locks + pid, 0, 1):  # original value is 0
        tl.atomic_xchg(locks + pid, 2)

Thank you for your point, I think I now understand, even if ops are atomic, the design of this snippet do not provide any guarantee on the order of execution of the ops (the first xchg of the second visit can be executed before or after the second xchg of the first visit), meaning it's expected to have a mix of 1 and 2s.
There is still the 2nd question about the double sync barrier in PTX, is it expected?

There is still the 2nd question about the double sync barrier in PTX, is it expected?

We know about it. It's not a big deal, since (1) ptxas may remove them; (2) even if not, two barriers back-to-back shouldn't have much overhead since the threads are already synchronized by the time they reach the second barrier.

Thank you, it makes sense


Feature Request: Message filtering with AI

The title of this request is pretty self-explanatory. I was thinking of using a pre-trained model with real time message filtering done in the browser through Tensorflow.js (click send button > message is filtered w/ AI in browser > data is sent to server). This is a pretty tough challenge that won't really have a visible effect on this app at this moment, so it's probably low priority.

This sounds like a good idea, but what text do you think the AI algorithm should filter? If it's simple enough it can probably be done with regular JavaScript.

I was thinking the same thing as I posted the message - personally, I that that a blacklist with fuzzy string matching will do the job perfectly fine. AI is a super cool tool, but in this case, it's like using a sledgehammer to drill in a screw, if you know what I mean.

@Tloru Any new information on this?

Closing because of inactivity.


Office key does not remap correctly. Deleting remap does not work.

Microsoft PowerToys version
0.36.0
Running as admin

[ ] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
My keyboard has the office key which I do not like and I wanted to remap it to the app/menu key (the one which opens with the mouse right click). I used the powertoys remapper, however it also remapped my windows key. I deleted the entry but the remapping did not change back. I had to disable the remapper.
✔ Expected Behavior
No response
❌ Actual Behavior
My keyboard has the office key which I do not like and I wanted to remap it to the app/menu key (the one which opens with the mouse right click). I used the powertoys remapper, however it also remapped my windows key. I deleted the entry but the remapping did not change back. I had to disable the remapper.
Other Software
No response

This is multiple items in one issue.  The primary tracked by #3194  for not detecting office key and is a known limitation.
Can you please use the "report bug" feature and upload the zip?   Also take a screen of your KBM settings screen?

After uninstalling and reinstalling powertoys the problem has stopped.


Ebean run jar error is NOT an Entity Bean registered with this server

i use maven build java jar app. buu run it on java  wrapper services  has error like this
javax.persistence.PersistenceException: Model.MonUser is NOT an Entity Bean registered with this server?
No Entities found in ClassPath using ClassPathReader [com.avaje.ebeaninternal.server.util.DefaultClassPathReader@7674c011] Classpath Searched[[file:/opt/eagle/lib/wrapper.jar, file:/opt/eagle/lib/annotations-13.0.jar, file:/opt/eagle/lib/avaje-agentloader-2.1.2.jar, file:/opt/eagle/lib/avaje-ebeanorm-6.10.3.jar, file:/opt/eagle/lib/avaje-ebeanorm-agent-4.7.1.jar, file:/opt/eagle/lib/commons-lang3-3.4.jar, file:/opt/eagle/lib/commons-pool2-2.3.jar, file:/opt/eagle/lib/drc.client-1.0.0.jar, file:/opt/eagle/lib/lib/commons-lang3-3.4.jar, file:/opt/eagle/lib/lib/avaje-ebeanorm-6.10.3.jar, file:/opt/eagle/lib/lib/annotations-13.0.jar, file:/opt/eagle/lib/lib/persistence-api-1.0.jar, file:/opt/eagle/lib/lib/slf4j-api-1.7.7.jar, file:/opt/eagle/lib/lib/jackson-core-2.5.3.jar, file:/opt/eagle/lib/lib/avaje-agentloader-2.1.2.jar, file:/opt/eagle/lib/lib/avaje-ebeanorm-agent-4.7.1.jar, file:/opt/eagle/lib/lib/jedis-2.8.0.jar, file:/opt/eagle/lib/lib/commons-pool2-2.3.jar, file:/opt/eagle/lib/lib/mysql-connector-java-5.1.18.jar, file:/opt/eagle/lib/lib/drc.client-1.0.0.jar, file:/opt/eagle/lib/jackson-core-2.5.3.jar, file:/opt/eagle/lib/jedis-2.8.0.jar, file:/opt/eagle/lib/junit-3.8.1.jar, file:/opt/eagle/lib/mysql-connector-java-5.1.18.jar, file:/opt/eagle/lib/persistence-api-1.0.jar, file:/opt/eagle/lib/lib/commons-lang3-3.4.jar, file:/opt/eagle/lib/lib/avaje-ebeanorm-6.10.3.jar, file:/opt/eagle/lib/lib/annotations-13.0.jar, file:/opt/eagle/lib/lib/persistence-api-1.0.jar, file:/opt/eagle/lib/lib/slf4j-api-1.7.7.jar, file:/opt/eagle/lib/lib/jackson-core-2.5.3.jar, file:/opt/eagle/lib/lib/avaje-agentloader-2.1.2.jar, file:/opt/eagle/lib/lib/avaje-ebeanorm-agent-4.7.1.jar, file:/opt/eagle/lib/lib/jedis-2.8.0.jar, file:/opt/eagle/lib/lib/commons-pool2-2.3.jar, file:/opt/eagle/lib/lib/mysql-connector-java-5.1.18.jar, file:/opt/eagle/lib/lib/drc.client-1.0.0.jar, file:/opt/eagle/lib/slf4j-api-1.7.7.jar]]

buu run it on java wrapper services

It is not clear what that means.  Are you building a runnable jar using spring boot or something like that?

No Entities found in ClassPath using ClassPathReader

There are 2 configuration settings to control the searching.  The jars that are searched (if the entities are in an included jar) and the packages that are searched.
These 2 options can be set in ebean.properties or via serverConfig.  Are you setting either of these options?
Is the Model.MonUser entity bean in a included jar?  It looks like the package name is Model with an uppercase "M" - that normally isn't the java convention - just pointing that out.

i fix it. must package  exclude ebean jar to one jar.

Ok, closing it then.


Parsing TSX files

Nice work you did with tile-soup!
I was playing with tiled and I was wondering if it would make sense to the tileset files (.tsx) if declared in the tilemap.
Having both the tileset and the tilemap in the same map would be nice for tile custom properties, for example.
I could try to implement that if I have a bit of time. I just wanted to know what you were thinking about the idea.
I'm new to Clojure, I have no clue about spec, so don't expect the best work ever, if I do it :smiley:

I probably won't have tile-soup automatically retrieve external files like tsx because the way you retrieve it will depend on your environment. In clojurescript, you will need to do an ajax request, and in clojure you'll probably either be reading it from resources with clojure.java.io/resource or just slurping it from the disk. So it's probably best not to put that in tile-soup itself.
But i think i can make it easier to parse with a small change to tile-soup. Right now, the parse function is hard-coded to use the ::map/map spec, but maybe i can make it possible to pass any spec there. That way, you can make it parse with the tileset spec, which is what tsx files need. I'll experiment with this and see if it works.

OK that was easier than i thought. I released 0.4.0 with the change to the parse function. I added an example near the end of the docs, where it says "parsing a TSX file":
https://oakes.github.io/tile-soup/
Like i said, you'll still need to retrieve it yourself, but that should allow you to parse it afterwards.

It makes sense. Thanks a lot!

BTW i had another thought: with clojurescript you don't necessarily need to retrieve it with ajax. In the super koalio example, the tiled map is being read by a macro which runs at compile time, so it's running on the JVM, not in the browser. That means you could actually just read the TSX file in that macro, parse it, and add it somewhere in the overall hash map (right before the pr-str). That way, when the game loads, it has everything it needs and doesn't need to retrieve anything else.

It's exactly what I wanted to do. It's how I handled loading levels in my sokoban game. The downside is you can't pass a symbol or a procedure because the macro doesn't eval the argument. Other than that, it works well.
Thanks for the tip anyway :) I went through super koalio many times to understand how everything is working.


A bug in the container edit fucntion / the ownership of private volumes is changed to "adminstrators" once it's attched to a container

Description

Steps to reproduce the issue:

in portianer 2.0.0 -> login as normal user (I used Oauth/keycloak)
go to an endpoint
create a container e.g. nodered/node-red:latest
Go to volumes -> create a new volume e.g. nodered-volume-1
Go back to the created container -> click on edit/duplicate
Go down to the Volumes section and mount the already created volume (nodered-volume-1) to /data in the container
click on deploy the container
you sure confirm the pop-up message (click on replace)
go back to the volumes section -> the volume will disappear.
Login as admin and check the volume -> the volume exists with ownership "administrators"

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
@ncresswell confirmed a bug in the edit function of the container on the slack chat!
Technical details:

Portainer version: 2.0.0 CE
Target Docker version (the host/cluster you manage):  19.03.8
Platform (windows/linux): Linux
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):
Target Swarm version (if applicable):
Browser:

I have performed additional diagnostics on this report... it ONLY occurs when using images with health-checks (eg /nodered/node-red:latest); it DOESNT occur if i follow the exact steps and use a simple image such as nginx.


Infinite loop in Truncate

As of this revision, Truncate assumes that the scrollHeight of an unpadded element containing only text will be an exact multiple of its line-height property:
const visibilityArea = LINE_HEIGHT_VALUE * Number(lines);
let newElementTextHeight = newElement.scrollHeight;
while (newElementTextHeight > visibilityArea) {
   //... code to truncate text
  newElementTextHeight = newElement.scrollHeight;
}

This assumption is incorrect.  Independently of a parent element's line-height,  if a child inline element is vertically aligned to baseline (the default), it is possible for the parent's scrollHeight to be greater than the sum of the heights of all lines it contains.  This is because the browser will pad each line's box vertically with enough space for a font's descenders with a value that varies according to font, browser zoom level, etc.
In particular, consider this usage of Truncate, where a single line is desired across three inner s.   No matter what value of line-height is set for the parent, it was observed in at least two browser implementations (Firefox 107.0 (64-bit) and Chromium 107.0.5304.121 (64-bit), both on Ubuntu Linux 22.04.1) that when finally truncated to one line, visibilityArea would always be one pixel larger than newElement.scrollHeight, leading to an infinite loop.
A workaround, as suggested here, is to vertical-align: bottom all child s so they conform to the parent's line-height.  In practice, this successfully avoids the infinite loop because when its down to a single line, newElementTextHeight will finally be equal to the expected visibilityArea.
Truncate should be smart enough to avoid this  without any expectations regarding the vertical alignment of child elements.
(Nowadays browsers actually support CSS-only line clamping, so Truncate might optionally use that.  Clamp.js, for example, checks for support and uses it, if enabled.)

@viktorrusakov, I see you'll soon start working on Truncate v3.  Can we take this issue in consideration during its development?

@arbrandes working on it, thanks for reporting the issue!


Replace ROS test macros with normal methods

The original point of the macros was so that, when tests failed, we'd have useful line numbers to diagnose the problem. The macros have caused more problems than they solved, though, and it's time to replace them.

Failing jobs are all failing because of CI issues. The 8.3.3 object server test failure is something unrelated that I've never seen before, so I think it's very likely just a transient error.


Error on keyword arguments to parallel functions

Reporting a bug

[x] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

ParallelAccelerator only considers positional arguments to parallel functions (such as here), resulting in unexpected errors. Supporting kws properly is necessary to avoid several similar issues.
import numpy as np
import numba

@numba.njit(parallel=True)
def f(n):
    return np.arange(n, dtype=np.int32)

print(f(11))

    return func(*args, **kwargs)
  File "/Users/ehsan/dev/numba/numba/core/compiler_machinery.py", line 289, in _runPass
    mutated |= check(pss.run_pass, internal_state)
  File "/Users/ehsan/dev/numba/numba/core/compiler_machinery.py", line 262, in check
    mangled = func(compiler_state)
  File "/Users/ehsan/dev/numba/numba/core/typed_passes.py", line 255, in run_pass
    preparfor_pass.run()
  File "/Users/ehsan/dev/numba/numba/parfors/parfor.py", line 1373, in run
    self._replace_parallel_functions(self.func_ir.blocks)
  File "/Users/ehsan/dev/numba/numba/parfors/parfor.py", line 1439, in _replace_parallel_functions
    if guard(replace_func):
  File "/Users/ehsan/dev/numba/numba/core/ir_utils.py", line 1452, in guard
    return func(*args, **kwargs)
  File "/Users/ehsan/dev/numba/numba/parfors/parfor.py", line 1427, in replace_func
    new_blocks, _ = inline_closure_call(self.func_ir, g,
  File "/Users/ehsan/dev/numba/numba/core/inline_closurecall.py", line 601, in inline_closure_call
    args = _get_callee_args(call_expr, callee, block.body[i].loc, func_ir)
  File "/Users/ehsan/dev/numba/numba/core/inline_closurecall.py", line 713, in _get_callee_args
    return numba.core.typing.fold_arguments(
  File "/Users/ehsan/dev/numba/numba/core/typing/templates.py", line 217, in fold_arguments
    ba = pysig.bind(*bind_args, **bind_kws)
  File "/opt/anaconda3/envs/DEV/lib/python3.8/inspect.py", line 3025, in bind
    return self._bind(args, kwargs)
  File "/opt/anaconda3/envs/DEV/lib/python3.8/inspect.py", line 3014, in _bind
    raise TypeError(
TypeError: Failed in nopython mode pipeline (step: Preprocessing for parfors)
got an unexpected keyword argument 'dtype'

Thanks for the report. I suspect the root issue is the inliner opposed to parfors.

Thanks for the report. I suspect the root issue is the inliner opposed to parfors.

I'm sure there's another ticket about this somewhere, same sort of issue but no parfors involved, but I unfortunately can't find it.

So this is still an open issue? I am getting bitten by this lately and just now finding out it was because of parallel (wouldn't have guessed that...).

So this is still an open issue? I am getting bitten by this lately and just now finding out it was because of parallel (wouldn't have guessed that...).

Probably, yes. If no-one has closed this yet it is most likely still applicable.

Probably, yes. If no-one has closed this yet it is most likely still applicable.

Yeah in retrospect it was a silly question 🙂
In my case I was just trying to use np.arange with dtype=np.int32 and I can get around with np.arange(n).astype(np.int32), so it works out.

Probably, yes. If no-one has closed this yet it is most likely still applicable.

Yeah in retrospect it was a silly question 🙂

Not necessarily, we have a lot of issues in Numba, sometimes they do get fixed but not closed, so that question is often actually worthwhile. But mostly, users end asking about issues because they are not yet resolved, as in your case.