RxJava 1.0.12

@nikitin-da PTAL (really huge release) https://github.com/ReactiveX/RxJava/releases/tag/v1.0.12

ci returns 502

Yep, maintenance…

retest this please


DNM: use ISCSI PV with IPv6

What this PR does / why we need it:
This PR add support for ISCSI target pod and use it in tests
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Related to #4150
Special notes for your reviewer:
Release note:

/test pull-kubevirt-e2e-k8s-1.18

/test pull-kubevirt-e2e-k8s-1.18

/test pull-kubevirt-e2e-k8s-1.18

@ormergi: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Background and border for managing actions

I'm not sure I'm sold on this. What do you think?

I'd say it definitely needs some more padding outside of the container for example above the title. But I quite like it! Especially if we get a row 2 with things like lock & close! I think it looks nice!
Sent with GitHawk

I think what’s definitely off atm is the spacing from the bottom to the title. Think it would help if that was the same spacing as the bottom of the title to the comment.
Sent with GitHawk

+1 on the spacing
Maybe remove the borders?
I also feel like it is kind of weird to split the issue/PR status from the other details. May look better if the status were to slide down with the other details somehow?
Sent with GitHawk

Also now seeing that manage isn’t vertically centered with the merge status. 🙈
Sent with GitHawk

I'm gonna let this simmer for a bit, really not crazy about it. Agree w/ the point about moving the status, but that makes things unnecessarily complicated.

Not feeling this


New design: new behavior for narrow and header

Description
Change the behavior related to the visibility of the left menu and to the header.
Left menu

when landing on the app:

when the screen is narrow (<600px), the menu should not be shown
when in LTI, the menu should not be shown
in the other cases, it should be shown

when in LTI, it should not be possible to show the menu

otherwise, the button to manually show/hide the menu should work (both on small and not-small screen)

when we navigate to a task (not using the left menu) that has to be shown full screen (i.e. a blockly task), the menu is hidden after its loading

when using the left menu to navigate to a content (even blockly tasks), the menu is not hidden

Header

when on a task that has to be shown full screen (i.e. a blockly task), the header is not shown

in all the other cases, the header is shown... the top bar still switch to the title only (instead of breadcrumb) after scroll

To be checked / to be done

[ ] when reducing the window size while the menu is not shown, when we reach 600px, there is an animation to "rehide" the already-hidden menu -> should not happen
[ ] in the content-top-bar : move the "open menu" button so that it may be displayed in front of the breadcrumb as well (so it has to be always shown if showLeftMenuOpener  even in *ngIf="fullFrameContentDisplayed || scrolled" condition does not match)
[ ]  I've removed the condition in src/app/modules/item/components/chapter-children/chapter-children.component.html line 21 because I didn't understand why it was there. check that.
[ ] in src/app/core/app.component.scss, I've removed the mobile as I had no condition anymore to check that... and switch to the usual way to do it with  @media screen and (max-width: 599.98px) , but maybe that's wrong?

Test cases
The cases described above.

How to get open the menu button when full frame mode? https://dev.algorea.org/branch/new-design/narrow-and-title/en/groups/by-id/2713577096475953687;p=

How to get open the menu button when full frame mode? https://dev.algorea.org/branch/new-design/narrow-and-title/en/groups/by-id/2713577096475953687;p=

That's related with the 2nd todo: for now, you need to scroll. But that needs to be fixed.


average_cpu_usage simple_animation_perf_ios regression..

https://flutter-flutter-perf.skia.org/e/?begin=1597878666&end=1597946236&keys=Xdc817274e9c4343e64e486c6ed262323&xbaroffset=21155
Looks like it started on this engine roll. https://github.com/flutter/flutter/pull/64213
cc @chinmaygarde (this week's sheriff) and @liyuqian

Maybe https://github.com/flutter/engine/pull/19842 (Dart_HintFreed) or https://github.com/flutter/engine/pull/20631 (Skia roll)? @dnfield

Revert has landed upstream. Will keep this open until it rolls into framework.

Fixed in https://github.com/flutter/flutter/pull/64593


CI: adding code coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0f6582f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #37   +/-   ##
=========================================
  Coverage          ?   77.64%
  Complexity        ?      572
=========================================
  Files             ?       73
  Lines             ?     1664
  Branches          ?      252
=========================================
  Hits              ?     1292
  Misses            ?      281
  Partials          ?       91

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


release 5.4.16: selecting more than 2 components with ctrl+leftmousebutton will hang Ctrlr

I think it does not matter which components you try to multi-select.
Very simple panel attached with 4 uiLabels. When you select more than 2 labels Ctrlr will hang. This bug has been introduced with v5.4.16.
multipleSelectBugTest.zip

I confirm this issue.

still persists in v5.4.27

Should be fixed in the latest build


Floating step - different results with click and keyboard

Description
When I set step to floating value, like 0.1 - using slider with mouse is okay and everything works as expected.
When I use keyboard to change slider value, there are floating point issues:

The issue is probably because of floating point operations at lines:
https://github.com/zillow/react-slider/blob/master/src/components/ReactSlider/ReactSlider.jsx#L730
https://github.com/zillow/react-slider/blob/master/src/components/ReactSlider/ReactSlider.jsx#L736
I could suggest solution like this:
[...]
const sanitizeFloatValue = (a) => Number.isInteger(a) ? a : parseFloat(a.toFixed(10));
[...]

const newValue = sanitizeFloatValue(oldValue + step);

@stonebk
If you think this solution is acceptable, I will make PR.
CodeSandbox
https://stackblitz.com/edit/react-oeujyh

@viters thanks for the detailed issue report!
I don't think I would add this kind of sanitization logic to the component itself because it is the result of using JavaScript floats and not really the fault of the slider logic.
Perhaps we can leverage our onBeforeChange lifecycle method as a place to run sanitization logic and return a different value.

I don't think I would add this kind of sanitization logic to the component itself because it is the result of using JavaScript floats and not really the fault of the slider logic.

I think libraries do fight with JS sometimes to hide this fight from users.
There may be better solution, but I would not require users to handle this difference themselves. In other words, I think slider should produce the same results, regardless of interaction type (mouse, touch, keyboard).
Maybe there is a way to unify how newValue is calculated for mouse and keyboard?

One example of handling this situation is presented by rc-slider:
https://github.com/react-component/slider/blob/master/src/utils.ts
Especially ensureValuePrecision.
This is proper solution I think, setting the target precision based on step from props and ensuring the output value has the right precision. As as library user I would not expect receiving e.g. 0.1001 value when I have step of 0.1.

Yeah, the more I think about it, it makes sense that we resolve the issue since we are the ones doing the addition/substraction.


Results keeps changing in CameraViewController

I'm using recognizeTextOnDevice in CameraViewController. The problem is the results on overlay view are always changing even I don't move the camera. Any idea on how to make the text stable?

And by the way, why confidence in block is always nil?

There's going to be a substantial amount of changes in the text recognition area within the near future, which will hopefully address these issues. I unfortunately can't give a time frame for it, but since ML Kit is a beta product we're more free to make large breaking changes in shorter time frames.
Confidence is currently not available for on-device text recognition due to technical limitations in the OCR engine.


test: #638 Add cron job for database backup

Open this pr just to run dev deployment and db backup, used for testing

Thanks for the PR!
Any successful deployments (not always required) will be available below.

api
admin
public

Once merged, code will be promoted and handed off to following workflow run.

Main Merge Workflow

close this pr and will reopen later to trigger a clean deployment


Cell fill from a lower row overlapping onto top hline

Generating smaller hlines than normal (missing the bottom half of the stroke).

This might require #2 for a proper solution.

This really needs to get fixed. Without fill is not useable.

I found a workaround. Having a 0cm row that spans all columns and a 0cm column that spans all rows fixes the issue.
This looks correct:
#tablex(
    fill: luma(240),
    columns: (0cm, 1fr, 1fr),
    rowspanx(3)[], [?], [?],
    (), [?], [?],
    (), [?], [?],
)

but this does not:
#tablex(
    fill: luma(240),
    columns: (0cm, 1fr, 1fr),
    (), [?], [?],
    (), [?], [?],
    (), [?], [?],
)

Yep, that will work (as will setting header-rows to a really high value) because it will force tablex to draw all of your cells at the same time (instead of drawing per row), drawing cells first and then lines, which circumvents this problem. However, that will make it impossible to split your table across pages.
I plan on tackling this on 0.1.0 by providing a new renderer which will draw per page instead of per row, which is how native tables do it, but I haven't gotten to it yet (I also haven't yet experimented with the idea to know how much of it is possible).


No validation for unique topic

Currently when adding a topic there's no validation to see if the topic is unique. There is instances where you might want multiple topics with the same name under different topic categories. Because they aren't unique, the filtering of the topic block doesn't work correctly.

Ah, this is why we originally wanted to use node IDs in the URL, rather than node names, but we undid that due to people clamoring that it was bad SEO.
Maybe the best way to fix this would be for topic blocks to see pass through both the topic name and its ID in the URL
/blog/topic/dinner/287
so you can still tell that it's dinner that's being filtered (for search engines) but the actual node that is being searched against is 287.

Has there been any thought to validating if the handle is unique similar to how a page handle works? I'm not sure if there's any plans in the future for adding filtering by topic categories as well. Because the topic categories has the same issue.
I recently built out a fairly comprehensive category/topic hierarchy and built out some custom filtering on the front end for filtering by category handles as well and ran into the same problem. If you we're going to incorporate the node id it could make for a fairly complex url:
eg: category-name/23/topic-name/33 rather then something like: category-name/topic-name
Something to consider I guess.

You wouldn't need that level of complexity in the URL, since node IDs are always unique
So you'd only need /topic/category-name/NODEID, where NODEID is the tree nodeID for that particular topic or category node.
We used to do it this way and had no problem figuring out which nodes to filter by, regardless of their names or whether there were other nodes with the same ID. e.g. /topic/NODEID. People didn't just want the node ID in the URL though, they wanted the category name for SEO.

I don't see any reason the id couldn't/shouldn't be added back in, especially if we leave the url friendly slug in there. I actually seem to see a lot of ID suffixing or prefixing in systems lately due to many times url slugs themselves not being unique. Frequently occurring in patterns like /friendly-url-slug-{id}/ or /{id}-friendly-url-slug/. Regardless as to where it gets put I don't see any way around adding it back other than making the topic/node-name a unique combination (which probably is more work and less flexible ultimately).


Iceberg REST Catalog using HiveCatalog backed by Hive Metastore

docker-compose up --build

Will this be added? thanks!


Need to tap twice in order to tap on TouchableHighlight when it is within a ScrollView or ListView

perform below action in sample app

input something in the text box
click the button

https://rnplay.org/apps/KjmElQ
expecting:
tap once, keyboard will go away and will trigger "onPress" event on the TouchableHightlight
actual:
have to tap twice. first tap keyboard go away, second tap then trigger "onPress" event on TouchableHightlight

did you try to add keyboardShouldPersistTaps={true} to the scrollview props ?

@FaridSafi thanks for the suggestion. it solve the issue that need to click twice. however it cause another issue. the keyboard won`t go away.

try my solution.
export default FormContainer = React.createClass({
  mixins: [PureRenderMixin],

  render() {
    return (
      <ScrollView
        showsVerticalScrollIndicator={true}
        automaticallyAdjustContentInsets={false}
        contentContainerStyle={styles.content}
        keyboardDismissMode='on-drag'
        keyboardShouldPersistTaps={true}>
        <View
          onStartShouldSetResponderCapture={(e) => {
            const focusField = TextInputState.currentlyFocusedField();

            if (focusField != null && e.nativeEvent.target != focusField){
              dismissKeyboard();
            }
          }}>
          {this.props.children}
        </View>
      </ScrollView>
    );
  }
});

@hufeng this is awesome except if you have multiple text inputs in the form, the keyboard "jumps" switching between them

@marcshilling yes, i have multiple text inputs in the form.

anyone have an idea to fix this globally, not specifically with workaround provided here? I would be happy to provide a PR, but I'm not sure where the problem source is.

I'm experiencing this issue even when the keyboard isn't in play. I have a TouchableHighlight within a scrollview, and only the second tap on the button results in onPress being invoked.

@gre I am using a Dismissable component, wrapping any number of children will hide the keyboard and forward the events.
Here's the source:
import React, {
  View,
  TextInput
} from 'react-native';

import MainEntryContext from '../components/main-entry-context';
import ReactNativeTagHandles from 'ReactNativeTagHandles';

class Dismissable extends React.Component {
  static propTypes = {
    ignoredAreas: React.PropTypes.func,
  };

  static defaultProps = {
    ignoredAreas: () => [],
  };

  componentDidMount() {
    this.ignoredNodes = this.props.ignoredAreas()
      .map(ref => React.findNodeHandle(ref))
      .map(tag => ReactNativeTagHandles.tagToRootNodeID[tag]);
  }

  _dismissKeyboard = (e) => {
    const ignoredTarget = this.ignoredNodes.find(node => e.dispatchMarker.indexOf(node) >= 0);

    if (!ignoredTarget) {
      TextInput.State.blurTextInput(
        TextInput.State.currentlyFocusedField()
      );
    }

    return false;
  };

  render() {
    return (
      <View style={{ flex: 1 }} onStartShouldSetResponderCapture={this._dismissKeyboard}>
        {this.props.children}
      </View>
    )
  }
};

module.exports = Dismissable;

Just wrap any component and optionally pass it an array of refs to ignore (e.g inputs)

@grabbou interesting idea :)

@jcihocki I have the same issue, without any text inputs. Were you able to find out the cause in your situation?

@jcihocki & @alloy I also have the same problem... Any find?

@cyphire My bad, I should’ve updated earlier in my case it ended up being a native module of mine that was using the RCTUIManager to find a view for a react tag in an incorrect way. It works fine for me now.

Thanks for the update... A month ago I had rusty JavaScript.. Now I'm
crushing it!
On Jun 10, 2016 2:36 PM, "Eloy Durán"<EMAIL_ADDRESS>wrote:

@cyphire https://github.com/cyphire My bad, I should’ve updated earlier
in my case it ended up being a native module of mine that was using the
RCTUIManager to find a view for a react tag in an incorrect way. It works
fine for me now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/4087#issuecomment-225275459,
or mute the thread
https://github.com/notifications/unsubscribe/AAUnhk_EF1muDMwIao3SHwuU4y7khkyMks5qKby5gaJpZM4Gg2Ru
.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/need-to-tap-twice-in-order-to-tap-on-touchablehighlight-when-it-is-within-a-scrollview-or-listview
Product Pains has been very useful in highlighting the top bugs and feature requests:
https://productpains.com/product/react-native?tab=top
Also, if this issue is a bug, please consider sending a pull request with a fix.

I am experience same thing but only with Views and buttons (no text input), has anyone found a good solution for this?

This is how i solve it.
I had one input and one button inside view.
<View>
  <Input>
  <Button>
</View>

I wanted it to be scrollable so i added
<ScrollView>
<View>
  <Input>
  <Button>
</View>
</ScrollView>

But then button needs to have a double click in order to work
so i added keyboardShouldPersistTaps={true} in scrollView in order to button work on one click
<ScrollView keyboardShouldPersistTaps={true}>
<View>
  <Input>
  <Button>
</View>
</ScrollView>

But this causes the not hide keyboard when click on anywhere on screen. So i did it by doing this
final code
import { KeyboardAwareScrollView } from 'react-native-keyboard-aware-scroll-view'
import dismissKeyboard from 'react-native-dismiss-keyboard';

<KeyboardAwareScrollView keyboardShouldPersistTaps={true} >
   <TouchableWithoutFeedback onPress={dismissKeyboard}>
       <View>
          <Input>
         <Button>
      </View>
  </TouchableWithoutFeedback>
</KeyboardAwareScrollView>

The above code works for all condition

You can now also user Keyboard.dismiss after importing Keyboard from react-native:
import {Keyboard} from 'react-native';

@ragnorc
Thanks, I will try your solution also :)

@ragnorc is correct. They fixed this over the last two years and now just setting keyboardShouldPersistTaps="handled" on your ScrollView allows your children Touchables to handle the click event when appropriate.
Progress! 🚀

as @joshuapinter pointed out, keyboardShouldPersistTaps="handled" should be enough. If however you have 2 (or more) nested components that all handle this prop, you'll need to provide it to each.
eg:
<ScrollView
    keyboardShouldPersistTaps="handled">
    ...
    ...
    <SwipeableListView
        dataSource={ds}
        keyboardShouldPersistTaps="handled"
    >
        ...
        ...
        ...
    </SwipeableListView>
</ScrollView>

Is anyone trying to use keyboardShouldPersistTaps on a ListView? I have it working on ScrollViews, but I can't press a touchable item in a ListView without first closing the keyboard.

This solved it for me: https://stackoverflow.com/questions/42808294/react-native-have-to-double-click-for-onpress-to-work

Update - The above was definitely not workable, but I was able to get the desired by behavior by modifying the ScrollResponders.js file in /react-native/Libraries/Components as described here: https://github.com/facebook/react-native/issues/4229#issuecomment-174333284
Would love a solution that doesn't require me to modify a react-native file though!


Move some server metrics gauges from internal code to PySOA

Some server gauges exist only in our internal code due to our disparate metrics setup. Now that PySOA uses PyMetrics, move those to PySOA so that they are properly (and more accurately) integrated into the startup and request workflows.

Fixed in #217.


Flutter's local .pub-cache doesn't interact well with separate Dart install

When a developer has separate installs of the Flutter and Dart SDKs, our tools are confused by Flutter's separate .pub-cache.  This can lead to inscrutable errors.  E.g.,:
https://github.com/flutter/flutter/issues/36933

Just to explain why why we have the cache:
The pre-populated cache serves several purposes:

people can just run "flutter run" and it will work right out of the box, with no extra downloading.
People who download flutter when they have a fast connection, but try to use it the first time without a connection, or with a slow one, aren't disappointed (the airport problem), and
People in other parts of the world without access to the pub server (i.e. China) can download our package and use it instead of being stopped by not being able to access the server.

Ideally, we'd do some magic behind the scenes to merge that cache with the user's cache on first run or something.  Even more ideally, we'd have a "real" installer, and be able to just install stuff into the user's cache in the first place.
Another option would be to have pub look in multiple caches (via some sort of PUB_CACHE_PATH var or something), and then we could set it in the flutter tool to look in the user's cache first and the pre-populated one second.

How does pub determine which cache to use?

Via the PUB_CACHE environment variable.  We set that inside of the flutter tool if we notice that the flutter directory has a .pub-cache.
The workaround for this issue is to have people delete the flutter directory's .pub-cache and just use theirs.


Stub generation some times generates empty blocks

Environment data

Language Server version: 2020.8.1
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): 3.8.3

Expected behaviour
Should not generate empty blocks.
Actual behaviour
ci_and_not_headless = False
if sys.platform == "darwin" or sys.platform.startswith("linux") and ci_and_not_headless:
    ...
if sys.platform.startswith("linux") and ci_and_not_headless:
    ...

Logs

XXX

Code Snippet / Additional information

Actual python file that led to the generation of the pyi file above.
import importlib
import os
import sys

from .cv2 import *
from .data import *

# wildcard import above does not import "private" variables like __version__
# this makes them available
globals().update(importlib.import_module("cv2.cv2").__dict__)

ci_and_not_headless = False

try:
    from .version import ci_build, headless

    ci_and_not_headless = ci_build and not headless
except:
    pass

# the Qt plugin is included currently only in the pre-built wheels
if (
    sys.platform == "darwin" or sys.platform.startswith("linux")
) and ci_and_not_headless:
    os.environ["QT_QPA_PLATFORM_PLUGIN_PATH"] = os.path.join(
        os.path.dirname(os.path.abspath(__file__)), "qt", "plugins"
    )

# Qt will throw warning on Linux if fonts are not found
if sys.platform.startswith("linux") and ci_and_not_headless:
    os.environ["QT_QPA_FONTDIR"] = os.path.join(
        os.path.dirname(os.path.abspath(__file__)), "qt", "fonts"
    )

If/else statements at the global level of a file are sometimes important to include in a stub, especially when they're conditionally testing for sys.platform or sys.version_info. When the stub writer is emitting this, it doesn't yet know whether there's anything important to include within the body of the if/else. I could add a bunch of extra complexity to the stub writer to avoid this, but since there's no harm in including it, and I expect that users will need to clean up this sort of thing anyway, I'd prefer to not add the extra complexity.


⚠ File access (beta) has degraded performance

In 82f4fc5, File access (beta) (https://beta.fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 8085 ms

Resolved: File access (beta) performance has improved in 2b58b98.


Implement aten::left_fresh_copy* ops | feat(torchlib)

@justinchuby, is my 2nd commit for adding test ok?

It will likely fail because we need the first name "clone" to be unique in the list. I wouldn't bother adding a test since it's just identity. If we really want we should create an opinfo to use the proper (torch.ops.aten.left_fresh_copy) op.

Ok. Let me revert.

If we really just want to reuse clone, follow this: https://github.com/microsoft/onnxscript/blob/18409c15a8d86c41cbe91753b45234832d1df2ce/onnxscript/tests/function_libs/torch_lib/ops_test_data.py#L1794
ops_test_common.duplicate_opinfo(OPS_DB, "clone", ("ops.aten.left_fresh_copy"))

Thanks. I will use the duplicate_opinfo.

Is this the only change I need?
ops_test_common.duplicate_opinfo(OPS_DB, "clone", ("clone", "lift_fresh_copy"))

Do
- ops_test_common.duplicate_opinfo(OPS_DB, "clone", ("clone", "lift_fresh_copy"))
+ ops_test_common.duplicate_opinfo(OPS_DB, "clone", ("lift_fresh_copy",))

then use lift_fresh_copy in the TorchOpInfo entry.

How about now?


Implement new Metadata API

Implemented the following instructions

revoke_use_authority
accept_use_authority
utilize
set_and_verify_collection
verify_collection
unverify_collection

Transactions for testing both collections and uses are included beneath

set_and_verify -> Tx

unverify_collection -> Tx

verify_collection -> Tx

approve_use_authority -> Tx

revoke_use_authority -> Tx

utilize (with delegate) -> [Tx]
(https://solscan.io/tx/4Q7FBGABrXuctMnoNuhTYndRfJcQVz8SFDFQi4wULwv3VrECSfXF3k4gG8ucU8VEZ9YbL5KHwFkckXygHhfwTSdq?cluster=devnet)

utilize (without delegate) -> Tx

Transactions for testing both collections and uses are included beneath

set_and_verify -> Tx

unverify_collection -> Tx

verify_collection -> Tx

approve_use_authority -> Tx

revoke_use_authority -> Tx

utilize (with delegate) -> [Tx]
(https://solscan.io/tx/4Q7FBGABrXuctMnoNuhTYndRfJcQVz8SFDFQi4wULwv3VrECSfXF3k4gG8ucU8VEZ9YbL5KHwFkckXygHhfwTSdq?cluster=devnet)

utilize (without delegate) -> Tx

Ready for review @samuelvanderwaal :)

I reviewed the collections code and ran tests and everything checked out. I'll get to the uses today or tomorrow and then it will be good for merging. Thanks for your patience!

Sounds good. No problemo ser :)


fixed broken coverage link

Clicking the coverage button on the front page README leads to an improperly rendered page:

This PR fixes the URL link. Test the new coverage button URL here.

Codecov Report

Merging #3058 (39a6656) into main (b3945f8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3058   +/-   ##
=======================================
  Coverage   81.97%   81.97%
=======================================
  Files          55       55
  Lines        6031     6031
=======================================
  Hits         4944     4944
  Misses       1087     1087

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


models: Handle synchronous project builds better

There's a problem with the current logic: A synchronous project with
a build containing multiple runs (say multi-arch docker containers)
will run sequentially.
This query is becoming insane and I think the new approach is probably
going to be more efficient since its so much less stress on the
database coupled with the fact that we are never going to have that
many concurrent runs.
The new approach is to find out everything we need about QUEUED and
RUNNING runs. We can then loop through these results in a single
pass to know if there's a run we can schedule.
Signed-off-by: Andy Doan<EMAIL_ADDRESS>

just deployed upstream and see it working.


Fix log message contain real flow id

1. Why is this change necessary?
In log files you can find:

Could not execute flow with error message: Flow name: Hier könnte dein Flowname stehen Flow id: 0 Typed property ...\CustomerGroupChangedEvent::$mailRecipientStruct must not be accessed before initialization Error Code: 0

Which can happen when you have invalid code or during prototyping solutions. The crucial part is the "Flow id" in the message.
2. What does this change do, exactly?
Change the variable that is used to build the message containing the primary key of the flow so the message reads

Could not execute flow with error message: Flow name: Hier könnte dein Flowname stehen Flow id:<PHONE_NUMBER>abcdef0123456789abcdef Typed property ...\CustomerGroupChangedEvent::$mailRecipientStruct must not be accessed before initialization Error Code: 0

3. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Thanks again for your contribution, your changes are merged now! :tada: :blue_heart:

Awoooo :) thank you for merging


2024-12-19_01-36-10_master-next_python3-boto3

Automatically created.

Backport failed for kirkstone-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin kirkstone-next
git worktree add -d .worktree/backport-10868-to-kirkstone-next origin/kirkstone-next
cd .worktree/backport-10868-to-kirkstone-next
git switch --create backport-10868-to-kirkstone-next
git cherry-pick -x 3cf89ea1743116a184ea9dcf2859a833048535e3

Successfully created backport PR for scarthgap-next:

#10873

Successfully created backport PR for styhead-next:

#10874


Try to insert TVP with None in first row

Hi, I want to know is there any solution to insert data as TVP but with None in first row?
Inspired by https://github.com/mkleehammer/pyodbc/issues/596

There is no fix, but there is a workaround for SQL Server 2016+.
Given …
CREATE TYPE dbo.issue_1229_table_type AS TABLE
(
	id int NOT NULL,
	txt nvarchar(50) NULL,
    PRIMARY KEY (id)
)
GO

CREATE PROCEDURE issue_1229_sp
	@tvp dbo.issue_1229_table_type READONLY
AS
BEGIN
	SET NOCOUNT ON;
	SELECT * FROM @tvp;
END
GO

…
with the latest version of pyodbc — 4.0.39 at the time of writing — this works (no None in the first row)
tvp_data = [(1, "Alfa"), (2, "Bravo")]
results = crsr.execute("EXEC issue_1229_sp ?", (tvp_data,)).fetchall()
print(results)
# [(1, 'Alfa'), (2, 'Bravo')]

but this fails
tvp_data = [(1, None), (2, "Bravo")]
results = crsr.execute("EXEC issue_1229_sp ?", (tvp_data,)).fetchall()
# Invalid SQL data type (0) (SQLBindParameter)

However, with SQL Server 2016+ this works
tvp_data = [(1, None), (2, "Bravo")]
tvp_json = [dict(zip(["id", "txt"], row)) for row in tvp_data]

sql = """\
SET NOCOUNT ON;
DECLARE @tvp dbo.issue_1229_table_type;
INSERT INTO @tvp (id, txt)
SELECT id, txt FROM OPENJSON(?)
WITH (
    id int '$.id',
    txt nvarchar(50) '$.txt'
);
EXEC issue_1229_sp @tvp
"""
results = crsr.execute(sql, json.dumps(tvp_json)).fetchall()
print(results)
# [(1, None), (2, 'Bravo')]

As explained in https://github.com/mkleehammer/pyodbc/issues/596#issuecomment-520024907 , this is a known limitation of the parameter metadata discovery.


unable to generate deps.js

Based on the email of January 20 from Klevin Jin, I tried to generate deps.js for closure library.  Got the following error:
[14:33:32 ~/Documents/Programming/closure-library]$ npm run gen_deps_js

><EMAIL_ADDRESS>gen_deps_js
> node scripts/gen_deps_js > ./closure/goog/deps.js

node:internal/modules/cjs/loader:928
  throw err;
  ^

Error: Cannot find module 'google-closure-deps'
Require stack:
- /Users/erikn/Documents/Programming/closure-library/scripts/gen_deps_js.js
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:925:15)
    at Function.Module._load (node:internal/modules/cjs/loader:769:27)
    at Module.require (node:internal/modules/cjs/loader:997:19)
    at require (node:internal/modules/cjs/helpers:92:18)
    at Object.<anonymous> (/Users/erikn/Documents/Programming/closure-library/scripts/gen_deps_js.js:22:27)
    at Module._compile (node:internal/modules/cjs/loader:1108:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1137:10)
    at Module.load (node:internal/modules/cjs/loader:973:32)
    at Function.Module._load (node:internal/modules/cjs/loader:813:14)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:76:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/Users/erikn/Documents/Programming/closure-library/scripts/gen_deps_js.js'
  ]
}
npm ERR! code 1
npm ERR! path /Users/erikn/Documents/Programming/closure-library
npm ERR! command failed
npm ERR! command sh -c node scripts/gen_deps_js > ./closure/goog/deps.js

npm ERR!      /Users/erikn/.npm/_logs/2021-01-24T22_33_42_113Z-debug.log

Here is output showing my current versions of things.  These are all the latest AFAIK.
[14:33:42 ~/Documents/Programming/closure-library]$ npm -v
7.4.0
[14:44:36 ~/Documents/Programming/closure-library]$ node -v
v15.6.0
[14:44:39 ~/Documents/Programming/closure-library]$ git status
On branch master
Your branch is up to date with 'origin/master'.

nothing to commit, working tree clean
[14:44:43 ~/Documents/Programming/closure-library]$ git pull
Already up to date.
[14:46:24 ~/Documents/Programming/closure-library]$ git pull origin master
From https://github.com/google/closure-library
 * branch                master     -> FETCH_HEAD
Already up to date.

Hi Erik, thanks for bringing this to our attention.
This is something I missed when writing the e-mail -- the first time you update master, you'll need to run npm install again in the root directory to install google-closure-deps, which was added as a dependency.
Let us know if that doesn't work!

Hi Erik, thanks for bringing this to our attention.
This is something I missed when writing the e-mail -- the first time you update master, you'll need to run npm install again in the root directory to install google-closure-deps, which was added as a dependency.
Let us know if that doesn't work!

That worked.   Use @sinonjs/formatio instead
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated in favour of @sinonjs/samsam
npm WARN deprecated<EMAIL_ADDRESS>to-iso-string has been deprecated, use @segment/to-iso-string instead.
npm WARN deprecated<EMAIL_ADDRESS>Legacy versions of mkdirp are no longer supported. Please update to mkdirp 1.x. (Note that the API surface has changed to use Promises in 1.x.)
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Legacy versions of mkdirp are no longer supported. Please update to mkdirp 1.x. (Note that the API surface has changed to use Promises in 1.x.)
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade

><EMAIL_ADDRESS>prepare
> npm run gen_deps_js

><EMAIL_ADDRESS>gen_deps_js
> node scripts/gen_deps_js > ./closure/goog/deps.js

added 285 packages, and audited 288 packages in 28s

25 packages are looking for funding
  run `npm fund` for details

10 vulnerabilities (5 low, 2 high, 3 critical)

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

That worked.   Use @sinonjs/formatio instead
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated in favour of @sinonjs/samsam
npm WARN deprecated<EMAIL_ADDRESS>to-iso-string has been deprecated, use @segment/to-iso-string instead.
npm WARN deprecated<EMAIL_ADDRESS>Legacy versions of mkdirp are no longer supported. Please update to mkdirp 1.x. (Note that the API surface has changed to use Promises in 1.x.)
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Legacy versions of mkdirp are no longer supported. Please update to mkdirp 1.x. (Note that the API surface has changed to use Promises in 1.x.)
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade

><EMAIL_ADDRESS>prepare
> npm run gen_deps_js

><EMAIL_ADDRESS>gen_deps_js
> node scripts/gen_deps_js > ./closure/goog/deps.js

added 285 packages, and audited 288 packages in 28s

25 packages are looking for funding
  run `npm fund` for details

10 vulnerabilities (5 low, 2 high, 3 critical)

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

Great!
Re: warnings -- I believe these are due to our testing setup being somewhat outdated. It shouldn't affect any user-facing functionality in Closure Library (Closure itself has no runtime dependencies, and closure-deps runtime dependencies seem to be up-to-date), but that's a good reminder for us to update these dependencies anyway. I will track this effort internally.
I'll be closing this issue now, feel free to re-open if you have any other difficulties. Thanks!

Great!
Re: warnings -- I believe these are due to our testing setup being somewhat outdated. It shouldn't affect any user-facing functionality in Closure Library (Closure itself has no runtime dependencies, and closure-deps runtime dependencies seem to be up-to-date), but that's a good reminder for us to update these dependencies anyway. I will track this effort internally.
I'll be closing this issue now, feel free to re-open if you have any other difficulties. Thanks!


updated ibm-cpq-prod-3.1.0.tgz and index yaml

This is our release for 3.1.0 for Product IBM CPQ. We don't want to publish charts in /stable for security reasons.
Our previous release 2.0.0 was released on the same lines.
Please attend this PR ASAP.

Please expedite.

@pedjak

Hi @nvaibhav ! Sorry, I understand this went in later than expected.
We'll propose a better process to ensure we get to take in changes on time, and seek feedback on it.

Thank you for acknowledging and looking into it.


Resetting layout alongside view size transition.

Pull request checklist

[x] All tests pass.
[x] Demo project builds and runs.
[x] I have resolved merge conflicts.
[x] I have followed the coding style.

Contributing guidelines confirmation: 💪😎👊
This fixes an issue with/improves #1238
What's in this pull request?
PR #1238 did introduce a layout reset in viewWillTransitionToSize:withTransitionCoordinator:. However the implementation does not use animateAlongsideTransition: of the coordinator - @harlanhaskins, did you have any specific reasons not to? This causes #680 to show when rotating the device. This might also fix #1257.

Current coverage is 64.24% (diff: 0.00%)

Merging #1953 into release_7.3 will decrease coverage by 0.02%

@@           release_7.3      #1953   diff @@
=============================================
  Files              106        106
  Lines             3458       3459     +1
  Methods            866        867     +1
  Messages             0          0
  Branches           195        195
=============================================
  Hits              2222       2222
- Misses            1147       1148     +1
  Partials            89         89

Diff Coverage
File Path

0%
...ewController/Controllers/JSQMessagesViewController.m

Powered by Codecov. Last update fe656ae...77e3c40

Looks good to me. 👍  Thanks @sebastianludwig !
Will wait for @harlanhaskins to comment.

@sebastianludwig -- Do we want to push out a 7.3.5 release for this? (If we can cherry-pick anything major from develop, that would be ideal)

Sorry for not getting back to you earlier. @jessesquires personally I'd be happy if we'd push out 7.3.5 with this fix, but I can't offer any time to cherry pick other fixes atm :-/ So it's your call - I can also keep using my branch, no big deal.

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq

Hey @jessesquires,
that's indeed very sad to hear but also very understandable. I truly admire your courage to let go what you love. It has been an honor for me to be a (very) small part of the whole you created. It was always great fun and very rewarding to work on this project - and with you!
I wish you good luck and great success for all your future projects! Cheers 😃

Thanks @sebastianludwig ! 💯  I'm sure we'll work on something together again in the future!


MSVC compiler gives warning C4190 when include Tensorflow header

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
Yes.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
No
TensorFlow installed from (source or binary):
No
TensorFlow version (use command below):
TensorFlow 1.10
Python version:
None
Bazel version (if compiling from source):
None
GCC/Compiler version (if compiling from source):
None
CUDA/cuDNN version:
CUDA 10.0 / cuDNN 7.5
GPU model and memory:
GTX 1070

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
When include tensorflow header to use the C API in my project, it always gives me warnings warning C4190: 'TF_NewWhile' has C-linkage specified, but returns UDT 'TF_WhileParams' which is incompatible with C. It is a bug introduced by the Tensorflow C API header.
Describe the expected behavior
No warnings should be come from the Tensorflow header.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Got same WARNING but everything seems work fine.
The structure is defined as below
typedef struct TF_WhileParams {
  // The number of inputs to the while loop, i.e. the number of loop variables.
  // This is the size of cond_inputs, body_inputs, and body_outputs.
  const int ninputs;

  // The while condition graph. The inputs are the current values of the loop
  // variables. The output should be a scalar boolean.
  TF_Graph* const cond_graph;
  const TF_Output* const cond_inputs;
  TF_Output cond_output;

  // The loop body graph. The inputs are the current values of the loop
  // variables. The outputs are the updated values of the loop variables.
  TF_Graph* const body_graph;
  const TF_Output* const body_inputs;
  TF_Output* const body_outputs;

  // Unique null-terminated name for this while loop. This is used as a prefix
  // for created operations.
  const char* name;
} TF_WhileParams;

I think it's because C does not support const member and there are a lot of those members in this structure, so TF_WhileParams is treated as a C++ structure. Maybe you could ignore them?

Can you send a PR removing const from this struct?

typedef struct TF_WhileParams {
  // The number of inputs to the while loop, i.e. the number of loop variables.
  // This is the size of cond_inputs, body_inputs, and body_outputs.
  int ninputs;

  // The while condition graph. The inputs are the current values of the loop
  // variables. The output should be a scalar boolean.
  TF_Graph* cond_graph;
  const TF_Output* cond_inputs;
  TF_Output cond_output;

  // The loop body graph. The inputs are the current values of the loop
  // variables. The outputs are the updated values of the loop variables.
  TF_Graph* body_graph;
  const TF_Output* body_inputs;
  TF_Output* body_outputs;

  // Unique null-terminated name for this while loop. This is used as a prefix
  // for created operations.
  const char* name;
} TF_WhileParams;

I removed all top-level consts in this struct and there are no more warnings, and the code seems work, to be tested though.


fix create command's flags typo

#569
What
Change the k3d cluster create --help usage example info.
Why
The contents of Example cannot be used to initialize the cluster:

-l, --label
-e, --env

Hi @Jason-ZW , this is what happens because of copy-paste :roll_eyes:
Thanks a lot for this fix! :)


LittleFS on an Arduino's EEPROM

Hi everyone,
Thanks very much for LittleFS!
I'd very much like to use LittleFS on my Arduino (MEGA)'s EEPROM. I'm already using LittleFS on my ESP8266s and for compatibility reasons would really like to have it on my Arduinos as well.
I couldn't really find anything to start with:

There are issues mentioning LittleFS on an EEPROM, but not quite on an Arduino.
There are issues mentioning Arduinos, but don't provide any examples.
It seems to be possible to port LittleFS to something different than mbedOS (see #382), but no mention of how to do that with an EEPROM on an Arduino.
The ESP8266 Arduino Framework provides a LittleFS port, but that's storing on the ESP8266s Flash memory, not on EEPROM (which it doesn't even have).
There don't seem to be any Arduino-related LittleFS-libraries on the PlatformIO Registry

It surely is possible to use LittleFS on an EEPROM, right? It just seems nobody has ever attempted to do it on an Arduino? Or am I just blind?
Any ideas?
Cheers,
Yann

It's perfectly possible to port LittleFS to pretty much any system capable of compiling the code (as you yourself have highlighted, since you're using it on an ESP8266). At the lower level you just make the appropriate calls to littleFS. (If you look at some of the issues, many if not most of the contributors are running littleFs in a variety of environments; not just mbedOS).
At the hardware level its up to you to write drivers which interface between littleFs and your memory; these are relatively simple (close to the sort of thing you might write to test the hardware setup). And hardware is so variable that the lack of examples should come as no surprise. The documentation of what is required is pretty good (extremely good for an open source project, IMO) and there are further explanations in responses to some of the issues. So I suggest you get reading; here, and in the lfs.h file.
Main thing with EEPROM (as with flash) is that it takes a finite time to program, so you have to manage that appropriately. And you can run with a minimum block size of 128 bytes.

@e107steved those are good highlights.
The documentation is one thing I do want to improve, (especially around porting).

Two extra points:

Look at the struct lfs_config, this is where littlefs gets its runtime configuration including block device functions:
https://github.com/ARMmbed/littlefs/blob/master/lfs.h#L151
If your compiler is special you may also need to modify lfs_util.h. If you don't already know that your compiler is special, I would try compiling as is, and then only change things if it's not compiling. If you're desperate, LFS_NO_INTRINSICS will fall back to software-only implementations of intrinsics.
https://github.com/ARMmbed/littlefs/blob/master/lfs_util.h

It surely is possible to use LittleFS on an EEPROM, right?

How large is the EEPROM?
From my experience with EEPROM it's one of the easiest storage devices to use, but also the least dense. So littlefs should work fine, except if you have less than say ~2KiB, you may run into problems.

From my experience with EEPROM it's one of the easiest storage devices to use,

I actually use FRAM, which is even easier! Basically non-volatile SRAM, so byte at a time access, and no special requirements for writing. But it does cost rather more than EEPROM.

[Here's](Works with EEPROM, see [here](Yes it can, I just tried it for myself, see here.). :wink: :bow:) an example of littlefs working on EEPROM within Arduino :wink: :bow: .


Standardize processes across all plugins

Here is a meta issue for all plugins to track items which have to be taken care of between RC1 and GA for all the plugins.
Create an issue for the plugin you own and take care of the checklist.

[ ] Plugin branching strategy - #35
[x] Backward compatibility testing - #39
[ ] DCO checks - https://github.com/opensearch-project/opensearch-build/issues/53
[x] Broken links check - #29
[x] Standardize labels - #43
[ ] README.md, etc - https://github.com/opensearch-project/.github/issues/8
[ ] #59
[x] #58
[x] #57
[ ] Clean up branches  - https://github.com/opensearch-project/.github/issues/13
[ ] Issue and PR templates - https://github.com/opensearch-project/.github/issues/9
[x] #60
[x] Plugins dependencies from Maven - #45
[x] Test coverage reporting - #46
[x] License header checks - #49
[x] Plugin artifact names - #47
[x] Branch protection rules - #52

I propose we split these into separate issues, and label them for v1.0.0? This way we can track a bunch of things under a label.
Add consistent version labels.

I understand the value of having multiple issues and can be tracked using a label. Would it be cumbersome for folks to track all the items needed?

I understand the value of having multiple issues and can be tracked using a label. Would it be cumbersome for folks to track all the items to put thoughts into?

I do like having a meta issue, but an issue is an issue is an issue, we probably don't want to mix discussions and PRs

@saratvemulapalli @dblock another thing - now that Dashboards defaults to kebab-case for installed plugins, shall we finalize naming conventions for the build zips for Dashboards plugins?
Current zip convention: <pluginName>Dashboards-x.zip (e.g., anomalyDetectionDashboards-<IP_ADDRESS>-rc1.zip`)
New ideas:
Option 1: identical to existing name, but in kebab-case: <plugin-name>-dashboards-x.zip (e.g., anomaly-detection-dashboards-<IP_ADDRESS>-rc1.zip.
Option 2: same convention as OpenSearch plugins, but replacing opensearch with dashboards: dashboards-<plugin-name>-x.zip (e.g., dashboards-anomaly-detection-<IP_ADDRESS>-rc1.zip.
Any thoughts here? I think either make sense. Option 1 will keep consistency with the existing plugin IDs which are following the camelcase convention of <pluginNameDashboards> (note that this could be changed, but will require more code changes & testing). Option 2 will keep consistency with OpenSearch plugin names, in regards to the zips at least.

@ohltyler I opened https://github.com/opensearch-project/opensearch-plugins/issues/47 (you can just open issues next time too!) will add comments there

Looking ahead, I would like it if these issues with the broken down into individual issues. Individual issues so much easier to track￼

Looking ahead, I would like it if these issues were broken down into individual issues. Individual issues so much easier to track￼

They are, aren't they?

I understand the value of having multiple issues and can be tracked using a label. Would it be cumbersome for folks to track all the items to put thoughts into?

I do like having a meta issue, but an issue is an issue is an issue, we probably don't want to mix discussions and PRs
... I don't feel strongly about any of this
we couldnt confirm if everything here was covered as separate issue. I saw a few. So now we had to see this meta issue plus a few others.

No worries, will track only the issues in each repo. thanks!

I'm going to close this, we've done most of it, and original intent was just to collect work up to 1.0 GA.


Errors in raw make file usage when buldiing docs

While the playbooks are nice, they do have an issue in that they can eat the errors that Sphynx generates. I assume these have been here well over a year, probably closer to 2 yrs. Anyhow when running the make file to visually see the STDOUT, i noticed a number of errors: (venv) make clean;make module-doc;make html;make view-html;
updating environment: [new config] 114 added, 0 changed, 0 removed
reading sources... [100%] zhmc-ansible-modules/docs/source/requirements_managed
/Users/ddimatos/git/github/z_ansible_collections_doc/source/Troubleshooting/errorsandmessages.rst:63: ERROR: Unknown target name: "release notes".
/Users/ddimatos/git/github/z_ansible_collections_doc/source/Troubleshooting/errorsandmessages.rst:120: ERROR: Unknown target name: "installation doc".
/Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zos_core/docs/source/modules/zos_apf.rst:125: WARNING: Inline emphasis start-string without end-string.
/Users/ddimatos/git/github/z_ansible_collections_doc/source/index.rst:48: WARNING: toctree contains reference to nonexisting document 'troubleshooting/errorsandmessages'
looking for now-outdated files... none found
pickling environment... done
checking consistency... /Users/ddimatos/git/github/z_ansible_collections_doc/source/Troubleshooting/errorsandmessages.rst: WARNING: document isn't included in any toctree
/Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zos_core/CHANGELOG.rst: WARNING: document isn't included in any toctree
/Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zos_ims/CHANGELOG.rst: WARNING: document isn't included in any toctree
/Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zos_sysauto/CHANGELOG.rst: WARNING: document isn't included in any toctree
/Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zosmf/CHANGELOG.rst: WARNING: document isn't included in any toctree
done
preparing documents... done
writing output... [100%] zhmc-ansible-modules/docs/source/requirements_managed
/Users/ddimatos/git/github/z_ansible_collections_doc/source/zhmc-ansible-modules/docs/source/development.rst:44: WARNING: undefined label: supported environments
generating indices... genindex done
writing additional pages... search done
copying static files... done
copying extra files... done
dumping search index in English (code: en)... done
dumping object inventory... done
build succeeded, 10 warnings.

The HTML pages are in build/html.
====================================================================
Completed HTML generation, see /Users/ddimatos/git/github/z_ansible_collections_doc/build/html; next run 'make view-html'
====================================================================
Display generated HTML 'build/html/index.html' in default browser.
(venv) git status
On branch main
Your branch is up to date with 'origin/main'.

Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
	modified:   .gitmodules
	new file:   source/ibm_zos_cics
	new file:   source/ibm_zos_core
	new file:   source/ibm_zos_ims
	new file:   source/ibm_zos_sysauto
	new file:   source/ibm_zosmf
	new file:   source/zhmc-ansible-modules

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   inventory
	modified:   source/ibm_zos_core (new commits)
	modified:   source/ibm_zos_ims (new commits)
	modified:   source/ibm_zos_sysauto (new commits)
	modified:   source/ibm_zosmf (new commits)
	modified:   source/zhmc-ansible-modules (new commits)

Untracked files:
  (use "git add <file>..." to include in what will be committed)
	build/

After my changes the errors are reduced down to:
1) /Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zos_core/docs/source/modules/zos_apf.rst:125: WARNING: Inline emphasis start-string without end-string.

2)  /Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zos_core/CHANGELOG.rst: WARNING: document isn't included in any toctree
3) /Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zos_ims/CHANGELOG.rst: WARNING: document isn't included in any toctree
4) /Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zos_sysauto/CHANGELOG.rst: WARNING: document isn't included in any toctree
5) /Users/ddimatos/git/github/z_ansible_collections_doc/source/ibm_zosmf/CHANGELOG.rst: WARNING: document isn't included in any toctree

6) /Users/ddimatos/git/github/z_ansible_collections_doc/source/zhmc-ansible-modules/docs/source/development.rst:44: WARNING: undefined label: supported environments

is a known and accepted issue, its because of the escaping in the module that causes this, I have not found a solution
2 - 5) are new because of the unused changelog RST that is required for the collection, a skip list in the config probably can handle these thus for now are accepted
there is a git issue opened on it

After more changes we are down to just issues (1) and (6), 2 - 5 are gone.


Fix strlen issue with null in binary changes

Similar like https://github.com/phacility/arcanist/pull/274

https://secure.phabricator.com/book/phabcontrib/article/contributing_code/


Case insensitive LIKE filters

Add ILIKE, NILIKE, LILIKE, RILIKE operators (case insensitive search)
add en and fr translation for ilike,rilike nilike,lilike
Reference #679
It's just an implementation suggestion, the other option would be to add a $caseSensitive boolean field in Filter to avoid duplication of operators;

These add some BBC but it's ok. Can you complete the documentation too ?

I thought about BC, I could change the code to make LIKE filter case insentitive by default and change ILIKE to SLIKE (for Sensitive Like). I think it will reduce the impact because most of users expect LIKE to be insensitive (I may be wrong). What do you think ?
Ok for the doc :)

Yes SLIKE would be great, It can also mean "Strict" too.
Plus, we avoid some legal issues with Apple!

Is it ok for you ?

Yep good for me, thanks.


Add binding to execute current cell

The "execute command in terminal" well fine, but I don't want to touch the mouse. Is there any way to add a binding to execute the entire markdown cell where the cursor is?

Hi @jpambrun
According to documentation:

Alternatively, use cmd+f1 (mac) / ctrl+f1 (windows) shortcut to execute current selection.

Did you try that?

I am not 100% sure since Ctrl+F1 doesn't work for me, see #3, but my
understanding it that it first requires to select the text and choose a
runtime. I want a shortcut that run the block where the cursur is and the
runtime be determined by the markdown block syntax, just like when I press
the action button.
On Sat, Sep 3, 2022, 2:59 AM Hans @.***> wrote:

Hi @jpambrun https://github.com/jpambrun
According to documentation:
Alternatively, use cmd+f1 (mac) / ctrl+f1 (windows) shortcut to execute
current selection.
Did you try that?
—
Reply to this email directly, view it on GitHub
https://github.com/HansKre/markdown-execute/issues/1#issuecomment-1236063064,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFPMMAVZZQKOPIS4IBUYH3V4LZMXANCNFSM6AAAAAAQDPMIJQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @jpambrun, I implemented a logic for that. There are 3 cases to consider:

cursor is at opening backticks of a codeblock
cursor is at closing backticks of a codeblock
cursor is somewhere inbetween

For first tow cases, the codeblock inbetween is selected, runtime is detected and codeblock is exectued.
For the third case, I saw two different approaches: one would have been to try to go up and down from cursor-position and try to expand the selection. Another, also valid approach is to execute exactly the line at the cursor. From a user perspective, I found both to be pefectly valid approaches, but the last one significantly easier to implement, which is why it is the one that I chose.


Merge fee cache and fix sample conf

Merge #8484 to master and add the missing [fee] section.

LGTM, thanks! Can you rebase to get rid of the merge commit though?

Done!


If the spynet is fixed during training？

Good work and thanks for sharing the code. But one question is  if the spynet is fixed  during training？ It seems that the spynet is not updated during training as  'spynet_path=None'  and  'self.basicvsr.spynet.requires_grad_(False)'.

Thanks for letting me know. Indeed I made a mistake here. We should initialize the pre-trained SPyNet, and keep it fixed throughout training.

Thank you for your quick response！I got it.


[Suggestion] Add Issue Labels

Since there are a lot of issues being opened recently, it can be useful to add labels to them to sort things out. And inform the issue author about the state of their issue. though this is not necessary, just saying. IMO this can be useful

Looks like @Dar9586 accepted this.


Implement Gemini API

Use Google's API to generate or summarize text in our HTML, CSS, and JavaScript webpage.

Example 1: you can add a text input that takes the user's input and generates or summarizes the text
Example 2: You can add a file import to extract text from a PDF, Image, or other files
Example 3: You can add a text input that takes the user's input and generates a quiz (e.g., quiz generator)

Do whatever you'd like as long as you are using Google's API

Chatbot

Chatbot
Enter your question:

Ask
Response will appear here
<script type="module">
    import { GoogleGenerativeAI } from "@google/generative-ai";

    document.getElementById('askButton').addEventListener('click', askQuestion);

    const genAI = new GoogleGenerativeAI("AIzaSyD60yb7EjCaeOszNSfYvfBcArtrXo67sSM");

    async function askQuestion() {
        const userInput = document.getElementById('userInput').value;
        if (userInput.trim() === "") {
            document.getElementById('responseText').innerText = "Please enter a question.";
            return;
        }

        const model = genAI.getGenerativeModel({ model: "gemini-1.5-flash" });

        try {
            const result = await model.generateContent(userInput);
            const text = await result.response.text();
            console.log(text);
            document.getElementById('responseText').innerText = text;
        } catch (error) {
            console.error("Error generating content:", error);
            document.getElementById('responseText').innerText = "Sorry, there was an error generating the response.";
        }
    }
</script>

Code for the Gemini API Implementation

`

Chatbot

Chatbot
Enter your question:

Ask
Response will appear here
<script type="module">
    import { GoogleGenerativeAI } from "@google/generative-ai";

    document.getElementById('askButton').addEventListener('click', askQuestion);

    const genAI = new GoogleGenerativeAI("AIzaSyD60yb7EjCaeOszNSfYvfBcArtrXo67sSM");

    async function askQuestion() {
        const userInput = document.getElementById('userInput').value;
        if (userInput.trim() === "") {
            document.getElementById('responseText').innerText = "Please enter a question.";
            return;
        }

        const model = genAI.getGenerativeModel({ model: "gemini-1.5-flash" });

        try {
            const result = await model.generateContent(userInput);
            const text = await result.response.text();
            console.log(text);
            document.getElementById('responseText').innerText = text;
        } catch (error) {
            console.error("Error generating content:", error);
            document.getElementById('responseText').innerText = "Sorry, there was an error generating the response.";
        }
    }
</script>

`

Implemented a better UI for the Gemini API website, and everything other then the input and summarize button are placeholders for now

Implemented a better UI for the Gemini API website, and everything other then the input and summarize button are placeholders for now

const {VertexAI} = require('@google-cloud/vertexai');

/**
 * TODO(developer): Update these variables before running the sample.
 */
async function analyze_pdf(projectId = 'PROJECT_ID') {
  const vertexAI = new VertexAI({project: projectId, location: 'us-central1'});

  const generativeModel = vertexAI.getGenerativeModel({
    model: 'gemini-1.5-flash-001',
  });

  const filePart = {
    file_data: {
      file_uri: 'gs://cloud-samples-data/generative-ai/pdf/2403.05530.pdf',
      mime_type: 'application/pdf',
    },
  };
  const textPart = {
    text: `
    You are a very professional document summarization specialist.
    Please summarize the given document.`,
  };

  const request = {
    contents: [{role: 'user', parts: [filePart, textPart]}],
  };

  const resp = await generativeModel.generateContent(request);
  const contentResponse = await resp.response;
  console.log(JSON.stringify(contentResponse));
}

It's hard to find a pure JavaScript approach for PDF uploads, but I found this Node.js approach that seems to work pretty well. This one logs the response to the console but it easy to update the website with it. A similar approach can be used to upload the regular prompts too.


bug: object_type_id is not included

The object_type_id property is not included in the mapped types. We use this frequently.

Woah, I need to stop drinking so much whiskey. I think it's actually there 🤣 I'm so sorry!


Fixed build errors

Xcode9.2 fails to build the project because of SourceKittenFramework. Like:
Clang+SourceKitten.swift:81:52: error: cannot use optional chaining on non-optional value of type 'String'
        return rootXML["Declaration"].element?.text?

Changes:

fixed build errors;
fixed swiftlint warnings;
updated project to use recommended settings;
updated project's dependencies to the latest versions.

This is awesome. Thank you!
Would you be able to switch to Xcode 9.2 in .travis.yml too?

@fastred  Done

Thanks!


added issue #1142

@iamrahulmahato i have added the project , please review this PR.

i am closing this pr as all check are failed and files are missing,please make a new pr


[BEAM-7390] Add combineglobally code snippets

Add CombineGlobally code snippets.
R: @aaltay
R: @rosetn
Staged: http://apache-beam-website-pull-requests.storage.googleapis.com/12252/documentation/transforms/python/aggregation/combineglobally/index.html

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

(Please ping me once the open comments are addressed.)

@aaltay I've addressed the comments

I also noticed that the output tags were a little misleading now since we are showing the stdout instead of an output PCollection


Add tests for string and f-string literals

Summary
This PR provides additional test cases for string and f-string parsing.

(I need to rebase the parser stack for the ecosystem check to be in sync, I'll do it later.)


Improving implementation of max_line_length

It turned out that even the EditorConfig-specification requires max_line_length being applied by inserting (hard) wraps, a raising user base treats that as unexpected if they explicitly disabled "Soft Wraps" in Atom.
How should the implementation of max_line_length being improved to fit the user expectations better?

The overall Atom-configuration of "Soft Wraps" should be respected.
The gutter should be shown
2.1 Therefore the default wrap-guide should still be hidden

and
❓ The EditorConfigs-wrap-guide should turn red if any line exceeds the max_line_length

I proposed a change of the official max_line_length-property -- support would be appreciated: https://github.com/editorconfig/editorconfig/issues/280

I'm not sure if this is related, but I noticed that enabling max_line_length = 80 in my (global) .editorconfig causes the wrap-guide to disappear. And removing this line causes the guide to appear again. Looks like a bug to me.

Hi @mik01aj, thank you for reporting this. It might be that the wrap-guide implementation is faulty due to the current transition of Atom towards 1.13. To take care about your issue i'd like to ask you to open a new issue containing all necessary information. Thanks in advance! 👍

Ok, see #163.

IMHO max_line_length should position the wrap-guide, and possibly highlight any characters that are past it, e.g. with a red background. As Atom doesn't have a built in text wrap command, it can't do much else, unless it implemented wrap functionality itself, or worked in cooperation with another plugin.
Soft wrapping is almost the exact opposite of what it should do. It makes it look like the file complies with the preferred width when it doesn't.

softwrap, in my opinion, makes absolutely no sense here. The editorconfig spec says hardwrap - hardwrapping is not possible so using softwrap instead is just "for the user's eyes". Setting the wrap-guide to max_line_length should be fine to indicate that the code should wrap just there.
The user's softwrap setting(s) should be taken into consideration here. If the user has softwrap enabled, atom-editorconfig should softwrap at max_line_length. If it's disabled the wrap-guide should be moved to max_line_length (as it was a few weeks ago iirc).

its so annoying that i can disable softwrap, that i had to disable editorconfig plugin

The max_line_length will respect at least the SoftWrap setting in the next minor-release.


Handle stale attach code

Removed code in handle_stale_attach() to reset KV if the VM is powered off. This code is not relevant any more with the handling of VM events in the service.

The check for the volume attached to a live VM is needed and in a cluster
scenario the service doesn't assume the other node is running a version
that handles hostd events

The check for the volume attached to a live VM is needed

Why? There are 2 places where handle_stale_attach is called and I don't see a reason why that function should exist. In both cases, we check KV status and return error if attached.
In removeVMDK,
kv_status_attached, kv_uuid, attach_mode = getStatusAttached(vmdk_path)
if kv_status_attached:
         // ret = handle_stale_attach(vmdk_path, kv_uuid)
         logging.info("*** removeVMDK: %s is in use, VM uuid = %s (%s)", vmdk_path, kv_uuid, ret)
         return err("Failed to remove volume {0}, in use by VM uuid = {1}.".format(
                vmdk_path, kv_uuid))

In cloneVMDK
        attached, uuid, attach_as = getStatusAttached(src_vmdk_path)
        if attached:
            // if handle_stale_attach(vmdk_path, uuid):
           return err("Source volume cannot be in use when cloning")

and in a cluster scenario the service doesn't assume the other node is running a version
that handles hostd events

Do you mean some ESX running 0.13 or earlier version and some running 0.14 or later? Firstly, I don't see a reason to optimize code for this corner case. Secondly, I don't understand how keeping handle_stale_attach in upcoming release helps.

If the KV says its attached and we treat it as attached no matter what the actual status of the volume. What happens if the event handler misses an event for some reason? Do we go with what the KV has in it? OTOH the function itself can be removed and instead log based on the status in the KV. Or, leave the function as is  as its a backup for cases where for the event handler has missed the event or doesn't get one and this function resets the KV so the volume is usable.

Leaving the function as-is doesn't help since it only works if VM using volume is on same ESX.
Good point on not failing the request based on KV status. So, we should proceed with remove and clone and return appropriate error?

Sorry, missed this over the weekend. Yes, I'd say attempt the removal/clone/detach and handle the failure. I'll post the change and with the tests for that.

Testing, volume is already in use:
$ docker volume inspect test-attached
[
{
"Driver": "vsphere:latest",
"Labels": {},
"Mountpoint": "/mnt/vmdk/test-attached",
"Name": "test-attached",
"Options": {},
"Scope": "global",
"Status": {
"access": "read-write",
"attach-as": "independent_persistent",
"attached to VM": "2",
"attachedVMDevice": {
"ControllerPciSlotNumber": "224",
"Unit": "0"
},
"capacity": {
"allocated": "15MB",
"size": "100MB"
},
"clone-from": "None",
"created": "Mon May 29 10:06:22 2017",
"created by VM": "2",
"datastore": "bigone",
"diskformat": "thin",
"fstype": "ext4",
"status": "attached"
}
}
]

Clone of an in-use volume on the ESX host
docker volume create --driver=vsphere --name=test-attached-clone -o clone-from=test-attached
Error response from daemon: create test-attached-clone: VolumeDriver.Create: Failed to clone volume: Unable to access file [bigone] dockvols/11111111-1111-1111-1111-111111111111/test-attached.vmdk since it is locked

Remove a volume in use on the node,
docker volume rm test-attached
Error response from daemon: unable to remove volume: remove test-attached: volume is in use - [ca783cfd2969c425cded3cd07323d936f0d3004fa9d9954250ece119ac50ff08]

Attach a volume already attached to the docker host,

$ docker run -it --volume-driver=vsphere -v test-attached:/vol1 --name ub1 ubuntu
root@b260255e3518:/#
root@b260255e3518:/#

The fix doesn't reset the volume KV just because we don't find the volume on the local node, using the same volume on multiple nodes is disallowed and if a volume is in use one node, its locked and can't be used on another node.

Swarm test seems to have timed out on a poll and errored out, otherwise build is good.
FAIL: swarm_test.go:91: SwarmTestSuite.TestDockerSwarm
swarm_test.go:141:
c.Assert(status, Equals, true, Commentf("Volume %s is still attached", s.volumeName))
... obtained bool = false
... expected bool = true
... Volume swarm_test_volume_1496839264 is still attached

CI failed with the issue in #1372

Depends on #1368 to handle the swarm test error

CI is in pogress and once its complete.

CI failing due to #1371

@govint Please triage CI failure and re-push if needed. This PR has been open for a while.

@pdhamdhere the CI failed with #1371. Can I merge this change since its not my change thats causing the issue?

@pdhamdhere the CI failed with #1371. Can I merge this change since its not my change thats causing the issue?

Test that was causing #1371 is fixed. We should get a green CI.

Green CI.

Closing.


Updated README.md after #53

Coverage remained the same at 69.629% when pulling 4211008a262d2913220da1179a744e4f3bbeb102 on Slacki:master into 132dc5c4ed66db03be31f820ef32fcebfdff43bd on gernest:master.


chore(ast): add some documentation links

Current dependencies on/for this PR:

main

PR #2399   👈

This stack of pull requests is managed by Graphite.

Merge activity

Feb 12, 12:20 AM EST: @Boshen merged this pull request with Graphite.


Created blog page with proper responsiveness and darkmode feature

Title: Add responsive Blog page layout with cards,  issue #99
Description: This PR adds a responsive blog page layout featuring three blog cards in a row, each containing a title, content, and a blue background div in place of an image. The layout adjusts across different screen sizes using Tailwind CSS.
Changes:
Implemented BlogPage component in React.
Used a responsive grid layout for the blog cards.
Added placeholder data for the blog posts.

kindly review this , and merge this PR @jahnvisahni31

when you click on blog link it shows 404 not found error

when you click on blog link it shows 404 not found error

yes , i haven't linked that to the blog page i guess , let me correct that @jahnvisahni31

@jahnvisahni31 , blog button is working now


Auto imports does not work for extensions

From @bpasero on October 31, 2017 13:37
Testing: https://github.com/Microsoft/vscode/issues/37177

create a hello world JS extension
open extension.js
start to type RelativePattern
complete it

=> no import is added
Imho if there is no import action after accepting a proposal, we should also not show it in the IntelliSense window.
Copied from original issue: Microsoft/vscode#37282

@mjbvz can you elaborate on the issue. i am not sure i understand

Repo

Run yo  code to create a vs code extension
Select and generate a new js extension
Open the generated extension in VS Code insiders and then open the extension.js file
At the top of the file, type out Relative so that RelativePattern is shown as a completion item

Expected
Selecting this item should add an import for RelativePattern from 'vscode'
Bug
Selecting this item only does the completion. It does not add the import
With TS 2.6.1 I see this for the error being printed when we request the details:
[Trace  - 4:56:58 PM] Sending request: completionEntryDetails (33). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/test/test-js-ext/extension.js",
    "line": 4,
    "offset": 9,
    "entryNames": [
        {
            "name": "RelativePattern",
            "source": "vscode"
        }
    ]
}
[Trace  - 4:56:58 PM] Response received: completionEntryDetails (33). Request took 6 ms. Success: false . Message: Error processing request. Debug Failure. False expression.
Error: Debug Failure. False expression.
    at getImportDeclaration (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:75926:30)
    at /Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:75913:98
    at Object.mapDefined (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:1610:30)
    at getImportDeclarations (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:75912:72)
    at Object.getCodeActionForImport (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:75882:32)
    at getCompletionEntryCodeActions (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:65309:31)
    at Object.getCompletionEntryDetails (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:65283:35)
    at Object.getCompletionEntryDetails (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:79782:35)
    at Object.proxy.(anonymous function) [as getCompletionEntryDetails] (/Users/matb/.vscode-oss-dev/extensions/eg2.vscode-ts-tslint-0.0.1/node_modules/tslint-language-service/out/src/index.js:42:33)
    at /Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:86221:64
    at Object.mapDefined (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:1610:30)
    at IOSession.Session.getCompletionEntryDetails (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:86219:27)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:85317:61)
    at /Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:86669:88
    at IOSession.Session.executeWithRequestId (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:86660:28)
    at IOSession.Session.executeCommand (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:86669:33)
    at IOSession.Session.onMessage (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:86689:35)
    at Interface.<anonymous> (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:87881:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)

This also isn't working in an empty TS project with two files a.ts and b.ts where b.ts contains export const bbb = 0;.
It looks like the list of entryNames we're getting from vscode doesn't have source properties set.
Info 105  [12:59:34.526] request:
    {"seq":17,"type":"request","command":"completionEntryDetails","arguments":{"file":"/home/andy/sample/ts/src/a.ts","line":1,"offset":3,"entryNames":["bbb"]}}

Should be: "entryNames": [{ "name": "bbb", "source": "/home/andy/sample/ts/src/b.ts" }]


fix #11

Sorry for not being clear on the issue description.

I wanted something like this so the user can see that there is more content on the homepage below the landing page.
I'll close this PR for now.

You can add a new PR. It should be very simple, just add the same arrow to the middle bottom of the screen on the homepage. No connection to footer required.


Documentation mistake

The method is marked async because the Bot Builder makes use of the C# facilities for handling asynchronous communication. It returns a Task which is the reply to the passed in Message.
https://docs.botframework.com/en-us/csharp/builder/sdkreference/dialogs.html
It no longer returns "Task". This is making it difficult to understand whats going on.

Yes, I agree.  We should fix.

this Commit should solve this issue.


id should be on the data, so invariant

node ids should be on the data rather than on the node, so that they are invariant.
eg, currently node ids in fg and node ids in bg are different, I think?

See https://github.com/torch/nngraph/blob/99919fa260dd2ca33347c06da04dce5931f9fcc8/gmodule.lua#L108
The node id s are coming from underlying graph class in torch/graph so they need to be on the node. As you said, the forward and backward graphs are separate graphs so nodes have different ids, but of course the data is shared between separate graphs.

graph is aware of data object, and could inject an id into it.  It would obviously have to be careful not to overwrite some existing property also called 'data.id'.

I agree with you, we might want to inject id into id. Just for information, gModule constructor does this by injecting a filed named forwardNodeId into data. If you are trying something, you can rely on that now.

Ah, good point about forwardNodeId
=> closing


Unclear if docker group is auto-created in post-install (CentOS 7)

File: engine/installation/linux/linux-postinstall.md
I am researching Docker as part of a class project. While working through the Post-installation steps for Linux, I noticed that the docker group was already present on my CentOS 7 VM. Is the docker group created during the install?

@thaJeztah @andrewhsu can you please verify the expected behavior? I thought for CentOS and RHEL, no group was created automatically.

Closed via #4069


"Gottlob" and "Vizejdia" are listed in the drop-down menu

Description:
"Gottlob" and "Vizejdia" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Gottlob" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Gottlob, TIMIȘ (Gottlob)" and "Comuna Gottlob, TIMIȘ".

Step 3
Type "Vizejdia" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Vizejdia, TIMIȘ (Gottlob)".


InternalServerError: column "f68496a2-eca2-11ec-b62e-d34bce7c0a12" is of type uuid but expression is of type record

EdgeDB Version: 1.4
OS Version: Windows 10.0.19044

edgedb migrate
Connecting to an EdgeDB instance at localhost:10701...
edgedb error: InternalServerError: column "f68496a2-eca2-11ec-b62e-d34bce7c0a12" is of type uuid but expression is of type record
Hint: This is most likely a bug in EdgeDB. Please consider opening an issue ticket at https://github.com/edgedb/edgedb/issues/new?template=bug_report.md
Server traceback:
edb.errors.InternalServerError: column "f68496a2-eca2-11ec-b62e-d34bce7c0a12" is of type uuid but expression is of type record
Project stucture

The culprit as to why things are going wrong is this expression for populating the newly created logo field:
          SELECT
              default::Logo {
                  url := 'some_url'
              }
          LIMIT
              1

We generate bogus code for it because of the presence of the shape, but the shape doesn't accomplish anything even if we didn't mess up
I think it should produce an error


Cluster sporadically showing wrong number of markers

Summary
Have a cluster manager on a map with only 9 markers. Most of the time, it shows cluster with "9" as expected, but occasionally, it will show 10+, and this morning even got it to say 20+. I can't find a pattern as to why it's happening. Will have a streak of all 9s, but then will revert to 10 or 20 once or twice.
Expected behavior
What did you expect?

Observed behavior
Did not change any code between taking these two pictures, just re-ran.

Environment details

Nexus 5X API 29 x86 Emulator, with Library 1.2.1.

Code example
Private project so can't post the code, but let me know if there is any other information needed.
This is not super urgent, as I am probably going to add custom cluster icons anyway, but just want to make sure nothing is fundamentally wrong with my use of CM.

Thanks for reporting @queden. I'm not able to repro this on the demo app. Can you share a code sample that reproduces this problem?

@queden , this seems to be an old issue. Are you still able to reproduce it? Could you provide us with a code sample?


Support custom structs

Currently Rudi works only/best with pure map[string]any, generic datastructures. However it would be nice if one could also just use any kind of Go struct, similar to how Go templates also allow to access fields in structs using reflection.
Structs can of course contain pointer fields and those must be taken care of separately, as most funtions in Rudi do not take pointers (like concat), but Rudi also offers no language elements to dereference a pointer, nor do I intend to add such language elements. My plan is to automatically dereference pointers when evaluating a JSON path, so that if a field is nil, nil is returned, otherwise the dereferenced value.
In general, pointers are discouraged in Rudi and can lead to many hard to troubleshoot side effects. Support for them is on a best effort basis.

Work on this is slowly progressing in the jsonpath2, jsonpath3 and multipath branches.


Error from backend docker image (lodash)

When attempting to run docker-compose up, I encounter the following error during the build process:
 => [backend  9/11] RUN npm install                                                                                113.3s
 => [backend 10/11] COPY . .                                                                                0.1s
 => ERROR [backend 11/11] RUN npm run build                                                                                5.1s
------
 > [backend 11/11] RUN npm run build:
0.342
0.342 ><EMAIL_ADDRESS>build
0.342 > tsc
0.342
4.990 node_modules/@types/lodash/common/object.d.ts(1026,46): error TS1005: '?' expected.
4.991 node_modules/@types/lodash/common/object.d.ts(1031,46): error TS1005: '?' expected.
4.991 node_modules/@types/lodash/common/object.d.ts(1041,46): error TS1005: '?' expected.
------
failed to solve: process "/bin/sh -c npm run build" did not complete successfully: exit code: 2

add "@types/lodash": "4.14" to package.js [devDependencies]

PR: https://github.com/canove/whaticket-community/pull/645


Replace csv storage formats

This project so far has relied on csv files as storage of outputs and inputs into following processes, and that's been working, but as the variety and volume of listings and publishers grow, we're starting to see issues with encoding, line endings, quoting, arrays etc.
This issue needs to consider replacing csv files as storage for:

web scrapers output (extract)
merge_data.py output (aggregate and clean)

JSON has been suggested but we're not closed to other options.
As a secondary outcome, we'll still want to provide a .csv as output for public users to download, but this should be published output only.
See process flow for current system and PR #160

just adding to this - the current merge_output.csv is so riddled with git conflicts, it's not currently usable by the public anyway.

Started branch 163-replace-csv-storage-formats for this.

[x] replaced merged_output.csv
[x] replaced all pipeline references to it (export2jkan.py and main.sh)
[ ] create alternative csv output for end users
[ ] create ticket in jkan to replace any frontend references to .csv files (resources)
[ ] create ticket to replace csv in ckan api
[ ] create ticket to replace csv in sparql api
[ ] create ticket to replace csv in argis api
[ ] create ticket to replace csv in usmart api
[ ] create ticket to replace csv in dcat api
[ ] create ticket to replace csv in web-scrapers
[ ] update docs

note: having merged_output.csv doesn't actually lose us any data because export2jkan was correctly reading linebreaks. looks like our issues with .csv are just aesthetic problems, functionally it's fine. But worth it to change to .json anyway because it is easier to traceback than in .csv given the rendering issues.


[Q]: wandb: WARNING Disabling the wandb service is deprecated as of version 0.18.0 and will be removed in version 0.19.0.

Ask your question

When I use the 0.17.8 version, "wandb: WARNING Disabling the wandb service is deprecated as of version 0.18.0 and will be removed in version 0.19.0.", should I use "os.environ.setdefault (" WANDB_MODE "," disabled "if not config.wandb" else offline ") ", what code should I use to implement this feature after this feature is discarded

Hi @39c5bb-miku! thank you for writing in.
could you please talk a but about your workflow and why you are interested in disabling wandb service?

I also have this. When debugging, I want to disable WANDB.

Also have this warning, actually I have a if statement when debugging, it will disable the wandb service:
if "debugpy" in sys.modules:     print("Debugger detected, disable wandb")     wandb.init(project=config["wandb_project"], config=config, mode="disabled") else:     wandb.init(         project=config["wandb_project"],         config=config,     )

@zzccppp @wcarvalho  sorry it is a small bug we introduced, should be fixed in the next release.

@zzccppp @wcarvalho @39c5bb-miku just to let you know, wandb is indeed disabled, the fact that you see the warning message is just a bug in our condition to print it but wandb won't be doing anything, we will fix it so the message won't show up.

@kptkin Thank you for your response.

Using the WANDB_DISABLED environment variable is deprecated and will be removed in v5.？with 0.18.1，how to use environment variable？

Any solution for this yet?

Hey @datalee, Thank you for the follow-up. You can still use the environment variable as needed. The warning is false. Our sdk team is still working on a fix. We'll let you know when it gets resolved.


Item taking too long to load

Hello,
I followed the BasicDemo and I have URLs for 2 videos on the server. I am using Toro with Exo player
I noticed that the recycler view takes long time show the contents (upon start). And when it shows the content, the video starts playing.

Is it trying to download the whole video before playing it? Can I change that?
Shouldn't the contents such as texts  and thumbnail be shown right away? I have all the contents as part of VideoViewHolder ( didn't separate it into two).

Thank you so much

Figured it out. I need to use values for LoadControl to enable buffering

android4canada I also face the same issue how you solve the problem please help


Build error when bundle exec pod install

How to sovle this ?

"Open source, but not open API", but can not build success

Take a look at .env.example and turn that into a .env file, and then you'll need to follow the README.  Install the gems, run pod install and rake generate:keys (even though they'll just be empty strings, you can still compile).


Android Build Issue

Android NDK: Found platform level in ./project.properties. Setting APP_PLATFORM to android-23.
Android NDK: WARNING: APP_PLATFORM android-23 is higher than android:minSdkVersion 19 in ./AndroidManifest.xml. NDK binaries will not be comptible with devices older than android-23. See https://android.googlesource.com/platform/ndk/+/master/docs/user/common_problems.md for more information.
Android NDK: Your APP_BUILD_SCRIPT points to an unknown file: ./jni/Android.mk
Android/Tools/android-sdk-windows/ndk-bundle/build/core/add-application.mk:101: *** Android NDK: Aborting...    .  Stop.
Error building project!

Some Android build files were missing. Please try again.


Containerize your microservice using Docker

As a developer
I need to containerize my microservice using Docker
So that I can deploy it easily with all of its dependencies

Given the Docker image named accounts has been created
When I use docker run accounts
Then I should see the accounts service running in Docker


Build Autoscaler / Instrumentor / Scheduler images using Ko

We should leverage ko for building images which are standard Go applications (everything except Odiglet and UI images)
Read our blog post about the advantages of using ko: https://keyval.dev/how-we-use-ko/

/assign

Hey, I would like to work on this.

Can I work on this?


Introduce a CSS preprocessor

Many modern projects feature a CSS preprocessor like Sass. This allows for several useful features, like variable definitions, mixins, functions, iterations, etc.
One benefit in this project would be that it would make theming much more feasible and easier, because we could define default variables, which users of this module could override easily in their own projects.
This ideally would not require much more than introducing another Gulp/npm task, and converting existing CSS files to SCSS syntax.
If the project owner(s) find this useful, I would gladly help implement this. Any thoughts on this?

Note that latest SCSS version complains about importing *.css file without a custom importer:
DEPRECATION WARNING on line 52, column 8 of /home/user/dev/project/sass/theme.scss:
Including .css files with @import is non-standard behaviour which will be removed in future versions of LibSass.
Use a custom importer to maintain this behaviour. Check your implementations documentation on how to create a custom importer.

the styles of v2 are written in scss. Why now in v3 only css?


numpy.flip

numpy.flip

https://docs.scipy.org/doc/numpy/reference/generated/numpy.flip.html
Please also past the python link, thanks.


Auto-adapt replication tests

.. they do test if the replica set is available and do NOT fail when it's not available.

for #1


Fixing "Submit a job" link on contribute page fixes #336

Fixes "Submit a job" link so it links to the correct place (fixes #336 )

@JFernando122 Can you also comment on the issue in the repo that this fixes so I can close that issue too please?


Error message not corresponding to real error

Here inside the linuxExecute function, the launched error message (User did not grant permission) may has nothing to do with the real error.
I'm using electron-sudo to request user permissions before installing a .deb package. Due to some errors from mine, when trying to execute the command "/usr/bin/pkexec" --disable-internal-agent dpkg -i /tmp/app.deb, I was getting always that error message, but the real error was that the .deb package was corrupt, so dpkg was failing. This was driving me crazy for a long time.
I think it's better to simply use something like error = new Error(error.message || error);.
I can make a pull request if you find this reasonable enough (I do!!).

@cgalvarez

I can make a pull request if you find this reasonable enough (I do!!).

Yeah, it would be awesome!


Add AlwaysPresent for AudioContainer and fix AlwaysPresent not being taken into consideration when computing isMaskedAway or ShouldBeAlive

used for ppy/osu#29215

Presence has nothing to do with being masked away or being alive. I'm fine with the property, if that's all that you need to get things working.

I did not notice i pushed the ShouldBeAlive, i removed it and it still works.

Yeah, that check is supposed to not pass. It has caused problems in the past but I don't think presence is the way to fix this, or at least I'm not ready to do it yet.
There are two solutions I can mention without having looked at the osu!-side code:

Make the container not be masked away in general.
Override UpdateSubTreeMasking() and return false.

Also agree here.
If anything, rather than exposing this as a setting it should be implicitly applied to AudioContainer without the user having control.

I'm not sure about that... Means if at some point the entire game is wrapped in an AudioContainer no masking calcs would ever take place. Would prefer it to be applied to a local container where the intent is explicit, for now.


IP Cameras

Do you have plans to support IP Cameras?
I would think that the most Cameras are installed seperatly without a connected Raspi.
the code is easy, just call
video_capture =<EMAIL_ADDRESS>#sample
insted of
video_capture = cv2.VideoCapture(0)

I will add this remark in the description !