Duplicate draft release created when published release with same name exists

Hi there!  I'm trying to use this library to automatically create GitHub releases and upload assets as a CI step.  I've noticed that when reuseRelease and reuseDraftOnly are true, a duplicate draft release gets created if there's already a published release by the same name.
Here's what I'm trying to accomplish:

Create a draft release for version v1.0.0 if one doesn't already exist and there's no published release by the same name
If a draft v1.0.0 release exists, update the release and replace its assets
If a published v1.0.0 release exists, don't create a new draft release for v1.0.0 and don't update existing assets

It doesn't seem possible to do this currently since using reuseRelease and reuseDraftOnly causes a new draft release to be created if one doesn't already exist with the same name.
Is this the intended behavior?  If not, could I send a PR for a new parameter that will skip draft release creation if there's already a published release with the same name?
Thanks!

One thought I had for the name of such an option is skipIfPublished.  Might try prototyping that and send a PR.

We could disable a draft release for v1.0.0 when there is a release v1.0.0 since you will never be able to publish this specific draft release because you can't duplicate tags in a final release/pre release.
But I'm thinking about the users that use the editRelease option to update this draft release tag to a new one before publishing it.

Yeah, that's why I think it's better for there to be a parameter to explicitly opt in to the new behavior.
I've created https://github.com/remixz/publish-release/pull/38 to add a new parameter for this, let me know what you think!


Question: is setting number of pages and page size something that should be configurable or should all be always fetched

We've run into an issue today where with the values we had set for pages and tag_per_pages it wasn't enough to fetch new tags. Doesn't look like the API returns the data in any order so this means we basically have to tweak this parameters over time or set them initially quite large.
Should these even be parameters, should the check resource keep fetching more pages until there is no more?

Note also that the README.md specifies the tags_per_page default as 25 but it's actually 10.


Support order by sql

Related prs:
https://github.com/systemxlabs/bustubx/pull/36
https://github.com/systemxlabs/bustubx/pull/41


Unable to auto-initialize connection

There is issue init connection

Please use the issue template.

@andresesfm did't found about it...

When you create a new issue, select Bug report and fill out all the required info

@mujuappsile How did you solve it?

@SergiOnGit did you used endConnection method ? if you used this then removed it

@mujuappsile Yeah removing endConnection helped. Thank you.


Neither in nor not in a team

I was added to a team before I installed the Keybase app (not sure if that's relevant) and it seems like I'm in some bizarre limbo state where I'm neither in nor not in a team.
λ> keybase team list-memberships
Team    Role    Members
λ> keybase team list-requests
No requests at this time.
λ> keybase team request-access techworkersco
▶ ERROR You are already a member of this team (code 2665)
λ> keybase team leave techworkersco
▶ ERROR You are not a member of this team (code 2623)

Can you keybase log send? Thanks

Success! Your log ID is:

  d03cba30132851af91c4891c

Was the admin of the team online since you tried that?

Looks like the issue has been resolved by the admin updating their keybase client and logging back in


Install flower for celery monitoring

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.03%) to 88.114%

Files with Coverage Reduction
New Missed Lines
%

safe_transaction_service/history/services/collectibles_service.py
2
92.73%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
5664

Relevant Lines:
6428

💛  - Coveralls


Starship, error on install

Error when running /dotfiles/os/linux/distros/ubuntu/20.04/01.install_shartship.sh
-| ** |-  curl --fail --location https://github.com/starship/starship/releases/latest/download/starship-x86_64-unknown-linux-musl.tar.gz --output /tmp/starship-x86_64-unknown-linux-musl.tar.gz
-| ** |-  tar -xzf "/tmp/starship-x86_64-unknown-linux-musl.tar.gz" -C "/usr/local/bin"
-| ** |-  chmod 755 /usr/local/bin/starship
-| ** |-  rm /tmp/starship-x86_64-unknown-linux-musl.tar.gz

I have not been able to reproduce this. I'll try in a few different VMs to see if I can get it to error out.
I assume you ran this as sudo? or root?

I will try again later and confirm. This was on a fresh 22.04 install of Ubuntu Server.

Can confirm,
joel@optiplex:~/dotfiles/os/linux/distros/ubuntu/20.04$ sudo ./01_install_starship.sh
[sudo] password for joel:
-| ** |-  curl --fail --location https://github.com/starship/starship/releases/latest/download/starship-x86_64-unknown-linux-musl.tar.gz --output /tmp/starship-x86_64-unknown-linux-musl.tar.gz
-| ** |-  tar -xzf "/tmp/starship-x86_64-unknown-linux-musl.tar.gz" -C "/usr/local/bin"
-| ** |-  chmod 755 /usr/local/bin/starship
-| ** |-  rm /tmp/starship-x86_64-unknown-linux-musl.tar.gz

I just looked at your latest comment. I don't see any error output. That is all normal output of the script.
You can confirm the existence of the starship binary with
ls -ldah /usr/local/bin/starship

Thanks for letting me test on a system. I can't see any errors happening and confirmed the existence of the starship executable. Everything seems to be working as expected.

Closing the issue since testing confirmed the script is working as expected.


Purpose of API Key?

Does the API Key require you app to have internet connection? Looks like the whole app crashes if it doesn't.

The API key does not require an internet connection. The API key is a soft check, so the renderer will keep running if it’s offline and only fails if the key is proven to be invalid. If your device is offline and we can’t validate the key we allow the app to continue to run.

It doesn't look like that to me right now. It did work offline after I had it connected once. But after a few restarts, the app freezes and the logcat just prints reconnect attempts. The app ends up crashing after a while. And with freezing I mean it blocks the UI thread or react bridge and no JS runs.

That's strange, could you share:
1 - Code sample you were testing with to verify the crash.
2 - Repro steps you took for it to crash.
3 - React native + Viro React version you are using, and as well as the device and model number.
4 - Crash logs if possible.

Can't share the whole code, but here are the important sections, I believe it is well self explanatory.
NOTE: Some times it doesn't freeze/crash, still trying to pinpoint the exact cause. Looking at the log might be related to OOM, but I don't see a reason for it to oom this fast.
Basically, the library is overkill for what I'm trying to achieve, which is just showing a 360 picture in a modal.

function GetViroScene(props){
  return (
    <ViroScene>
      <Viro360Image
        source={{uri: 'https://zinspectordev2.s3.amazonaws.com/usi/211/1555523982eac3d0c0520748118800cddd5a043053.JPEG'}} //props.uri}} //}}
      />
    </ViroScene>
  )
}

modalViewer = (
  <ViroVRSceneNavigator
    apiKey={conf.VIRO_API_KEY}
    vrModeEnabled={false}
    initialScene={{ scene: GetViroScene }}
    passProps={{uri: this.state.modalVisible.uri}}
  />

)

<Modal
    animationType="slide"
    transparent={true}
    visible={this.state.modalVisible != null}
    onRequestClose={this.closeModal}
    supportedOrientations={['portrait', 'landscape']}
>
<View style={{backgroundColor: 'rgba(0,0,0, 0.9)', flex:1}}>
  <Button
    onPress={this.closeModal}
    style={{position: 'absolute', top: MODAL_TOP, left: 5, zIndex: 100}}
    transparent
    light
    rounded
    >
      <Icon name='close' type='AntDesign'></Icon>
  </Button>

  <Button
    onPress={() => this.state.modalConfirmDelete ? this.onDelete(this.state.modalVisible, true) : this.setState({modalConfirmDelete: true})}
    style={{position: 'absolute', top: MODAL_TOP, right: 5, zIndex: 100}}
    transparent
    light
    rounded
    >
      {!this.state.modalConfirmDelete ? <Icon name='delete' type='AntDesign'></Icon> : <Text danger>Delete?</Text>}
  </Button>

  { modalViewer }
</View>

</Modal>

05-02 20:43:35.017 4564-5179/com.zinspector_rn I/Viro: Attempt #3 to fetch api keys failed. Retrying in 8000 milliseconds
05-02 20:43:36.129 4564-5310/com.zinspector_rn I/Viro: Attempt #3 to fetch api keys failed. Retrying in 8000 milliseconds
05-02 20:43:46.556 4564-4564/com.zinspector_rn D/vndksupport: Loading<EMAIL_ADDRESS>from current namespace instead of sphal namespace.
05-02 20:43:46.565 4564-4564/com.zinspector_rn D/vndksupport: Loading /vendor/lib/hw/memtrack.msm8937.so from current namespace instead of sphal namespace.
    memtrack_open: exit 0
05-02 20:43:46.570 4564-4564/com.zinspector_rn W/m.zinspector_rn: type=1400 audit(0.0:6511): avc: denied { read } for uid=10198 name="mem" dev="debugfs" ino=13413733 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:kgsl_debugfs:s0 tclass=file permissive=0
05-02 20:43:46.906 4564-4564/com.zinspector_rn I/chatty: uid=10198(com.zinspector_rn) identical 2 lines
05-02 20:43:46.906 4564-4564/com.zinspector_rn W/m.zinspector_rn: type=1400 audit(0.0:6514): avc: denied { read } for uid=10198 name="mem" dev="debugfs" ino=13413733 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:kgsl_debugfs:s0 tclass=file permissive=0
05-02 20:43:46.911 4564-4564/com.zinspector_rn I/Choreographer: Skipped 43 frames!  The application may be doing too much work on its main thread.
05-02 20:43:47.243 4564-4564/com.zinspector_rn W/m.zinspector_rn: type=1400 audit(0.0:6515): avc: denied { read } for uid=10198 name="mem" dev="debugfs" ino=13413733 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:kgsl_debugfs:s0 tclass=file permissive=0
05-02 20:43:47.590 4564-4564/com.zinspector_rn W/m.zinspector_rn: type=1400 audit(0.0:6517): avc: denied { read } for uid=10198 name="mem" dev="debugfs" ino=13413733 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:kgsl_debugfs:s0 tclass=file permissive=0
05-02 20:43:47.593 4564-4564/com.zinspector_rn I/Choreographer: Skipped 40 frames!  The application may be doing too much work on its main thread.
05-02 20:43:50.073 4564-4564/com.zinspector_rn W/m.zinspector_rn: type=1400 audit(0.0:6519): avc: denied { read } for uid=10198 name="mem" dev="debugfs" ino=13413733 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:kgsl_debugfs:s0 tclass=file permissive=0
05-02 20:43:50.073 4564-4564/com.zinspector_rn W/m.zinspector_rn: type=1400 audit(0.0:6520): avc: denied { read } for uid=10198 name="mem" dev="debugfs" ino=13413733 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:kgsl_debugfs:s0 tclass=file permissive=0
05-02 20:43:50.416 4564-4564/com.zinspector_rn W/m.zinspector_rn: type=1400 audit(0.0:6521): avc: denied { read } for uid=10198 name="mem" dev="debugfs" ino=13413733 scontext=u:r:untrusted_app:s0:c512,c768 tcontext=u:object_r:kgsl_debugfs:s0 tclass=file permissive=0
05-02 20:43:50.434 4564-4564/com.zinspector_rn I/Choreographer: Skipped 44 frames!  The application may be doing too much work on its main thread.
05-02 20:44:05.865 4564-4655/com.zinspector_rn W/cr_ChildProcessConn: onServiceDisconnected (crash or killed by oom): pid=0

Device: Motorola G5, Android 8.1
"dependencies": {
    "@react-native-community/netinfo": "2.0.8",
    "async-mutex": "0.1.3",
    "moment": "2.24.0",
    "native-base": "2.12.1",
    "prop-types": "15.7.2",
    "react": "16.8.6",
    "react-native": "^0.59.5",
    "react-native-background-fetch": "2.5.4",
    "react-native-base64": "0.0.2",
    "react-native-camera": "2.6.0",
    "react-native-color-palette": "2.0.3",
    "react-native-device-info": "1.6.1",
    "react-native-easy-grid": "0.2.1",
    "react-native-gesture-handler": "1.2.1",
    "react-native-image-pan-zoom": "2.1.11",
    "react-native-image-resizer": "1.0.1",
    "react-native-keep-awake": "4.0.0",
    "react-native-loading-spinner-overlay": "1.0.1",
    "react-native-mime-types": "2.2.1",
    "react-native-modal-filter-picker": "1.3.4",
    "react-native-push-notification": "3.1.3",
    "react-native-share": "^1.1.3",
    "react-native-uuid-generator": "5.0.0",
    "react-native-video": "4.4.1",
    "react-native-vitals": "^1.0.2",
    "react-navigation": "3.9.1",
    "react-redux": "7.0.3",
    "react-viro": "^2.14.0",
    "realm": "2.26.1",
    "redux": "4.0.1",
    "redux-logger": "3.0.6",
    "redux-persist": "5.10.0",
    "redux-persist-filesystem-storage": "1.3.2",
    "redux-thunk": "2.3.0",
    "rn-fetch-blob": "0.10.15",
    "underscore": "1.9.1"
  }

In addition, those API checks are piling up. If I close and re open the modal (which will re-build the navigator), it will trigger another api check with re-tries. Since I'm using this to open a viewer from a gallery list, this can escalate very quickly.

Interesting, it sounds like you are constantly destroying + re-constructing the renderer as you close and then re-open then modal - this sounds highly costly and in-efficient.
As @dam00n  has mentioned, the renderer should work even without an app connection. From the above, it sounds highly likely that you are running into memory issues by re-creating a new renderer for every "re-opening" event.
Especially for 3d rendering applications, you would ideally want to construct the 3d renderer view once, then cache it somewhere (or detach it) in hopes of re-attaching it later to the view tree again. Else, you would want a "lite" renderer version, but we don't currently support that at the moment.
I would suggest using Viro in a way in Modal, or alternative controls, where you are not constantly re-creating / destroying the renderer to mitigate the above issue.

Hmm, you are right about it being inefficient. However, I don't think it should be leaking memory or those network request retries.
Is there a better way to achieve what I'm trying with Viro? All I just need is to be able to render 360 pictures, and this library seem to be the only one maintained that links react native and google VR.


Upgrade for Webcomponents V1

In order to avoid a conflict on Webcomponents' version when upgrading  starcounter-layout-html-editor in https://github.com/Starcounter/starcounter-layout-html-editor/pull/16 , juicy-ace-editor needs to use a WebComponents version higher than 1.
I tested it and everything seems ok.

please merge and release patch version.


getTemporaryDirectory call in PathProvider returns 'Library/Caches' directory on iOS

When I call getTemporaryDirectory on iOS it points to the Library/Caches directory. The source comments for getTemporaryDirectory reads that it uses the NSTemporaryDirectory API (tmp directory). Library/Caches corresponds to NSCachesDirectory. NSCachesDirectory seems closer to the getCacheDir API on Android as far as behavior. This may just require an update to the source documentation.
getTemporaryDirectory returns:
/var/mobile/Containers/Data/Application/A6DB6EC4-CA05-4FBE-A8BF-E3FE7F5607F8/Library/Caches/myitems/test0001.jpg

NSTemporaryDirectory returns:
/var/mobile/Containers/Data/Application/A6DB6EC4-CA05-4FBE-A8BF-E3FE7F5607F8/tmp/myitems/test0001.jpg

from Path Provider source:
  /// On iOS, this uses the `NSTemporaryDirectory` API.
  ///
  /// On Android, this uses the `getCacheDir` API on the context.
  Future<Directory> getTemporaryDirectory() async {
    final String path = await _channel.invokeMethod('getTemporaryDirectory');
    if (path == null)
      return null;
    return new Directory(path);
  }

flutter doctor output:
[✓] Flutter (on Mac OS X 10.12.5 16F73, locale en-US, channel master)
    • Flutter at /Users/albertlardizabal/dev/flutter
    • Framework revision 0809a4fcdc (25 hours ago), 2017-05-31 09:00:04 -0700
    • Engine revision 75c74dc463
    • Tools Dart version 1.24.0-dev.3.0

[✓] Android toolchain - develop for Android devices (Android SDK 25.0.3)
    • Android SDK at /Users/albertlardizabal/Library/Android/sdk
    • Platform android-25, build-tools 25.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 8.3.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.3.2, Build version 8E2002
    • ios-deploy 1.9.0
    • CocoaPods version 1.2.1

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Gradle version 3.2
    • Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2016.3.7)
    • Dart plugin version 163.13137
    • Flutter plugin version 12.1

Here is the code:
https://github.com/flutter/plugins/blob/master/packages/path_provider/ios/Classes/PathProviderPlugin.m
FYI @szakarias.  @cbracken or @xster might have opinions as to if this is the "correct" behavior on iOS or not.  In any case, very easy to change!

Marking as milestone 3 since this will be a breaking API change.

cc @cbracken can you help us determine what the "correct" behavoir is?

path_provider seems to be using NSCachesDirectory, I think this was just a documentation issue.
https://github.com/flutter/plugins/commit/79037146b0a70384a9a24c4bec076e0c4551113c#diff-6f3de9136bfae65f73ed7b70915c2c1e

@collinjackson so how do I access the NSTemporaryDirectory API , there are certain ios plugins which create temporary files there and we want to be able to clear them as part of cleanup.

@collinjackson so how do I access the NSTemporaryDirectory API , there are certain ios plugins which create temporary files there and we want to be able to clear them as part of cleanup.

I'm curious on how to achieve this as well.

@collinjackson so how do I access the NSTemporaryDirectory API , there are certain ios plugins which create temporary files there and we want to be able to clear them as part of cleanup.

I'm curious on how to achieve this as well.

This works for me in iOS 14. Not sure if it works in all devices:
final Directory directory = await getApplicationDocumentsDirectory();
String nsTemporaryPath = "${directory.parent.path}/tmp/";


Add Amino signing support to regen-ledger msgs

The current version of @regen-network/api doesn't support amino signing for transactions with regen-ledger msgs. It should be added so that we can sign such transactions through a ledger hardware wallet for instance which only supports legacy amino sign mode for now.
For example, if you try to do so, you'll get this type of error:

Type URL '/regen.ecocredit.v1alpha1.MsgRetire' does not exist in the Amino message type register. If you need support for this message type, you can pass in additional entries to the AminoTypes constructor. If you think this message type should be included by default, please open an issue at https://github.com/cosmos/cosmjs/issues.

For regen-ledger msgs (we could start with the ones from the ecocredit module for now since those are the only ones we're really using in https://github.com/regen-network/regen-web/tree/master/web-registry), we need to add amino converters, like it's done for instance for bank on cosmjs: https://github.com/cosmos/cosmjs/blob/fe34588ff12756d0453c94e792a781f8ecc72315/packages/stargate/src/modules/bank/aminomessages.ts
In particular for the ecocredit v1alpha1 (regen-ledger v3.0), the aminoType's can be found in:

https://github.com/regen-network/regen-ledger/blob/v3.0.0/x/ecocredit/codec.go#L12-L18
https://github.com/regen-network/regen-ledger/blob/v3.0.0/x/ecocredit/basket/codec.go#L13-L17

Side note that as part of regen-ledger v4.0, new versions of existing msgs with different amino type names and new msgs will be added, e.g.:

https://github.com/regen-network/regen-ledger/blob/d5b276015189d335cce7830ead520fdca3df7f5e/x/ecocredit/core/codec.go#L15-L30
https://github.com/regen-network/regen-ledger/blob/d5b276015189d335cce7830ead520fdca3df7f5e/x/ecocredit/basket/codec.go#L13-L17

But those could be added as part of a separate task, probably #36.
Lastly, once those amino converters are set up, we can use them to instantiate the stargate signing client (createStargateSigningClient in /packages/api/src/tx/msgs.ts) by passing aminoTypes as part of SigningStargateClientOptions
A new version of @regen-network/api should be tagged and published on npm once that's done.

Hey team! Please add your planning poker estimate with ZenHub @flagrede @haveanicedavid @mhagel @victordelval

@blushi would you be open to some pair programming on this one? i would like to understand this problem better and be able to contribute on these types of issues.

@blushi would you be open to some pair programming on this one? i would like to understand this problem better and be able to contribute on these types of issues.

Sure let's discuss that during our 1-1 today.

@wgwz I am happy to team up on this with you, since I have a device to test with

check out telescope from osmosis


Finding Success with Multi-Family Recycling

See if a copy of the Webinar is available

Hey all, unfortunately a copy of the webinar was not available. However, I just received an email today with a copy of the PowerPoint slides.

http://kab.leapfile.com/get.jsp?t=ff808081579ac8b90157d34a6fe1418d
IMPORTANT: Save the files first. Once downloaded, they will be automatically deleted.
Helpful Hints
If you have the option of saving the file or opening it in an application, choose save the file.
Download each file individually or get everything at once in a single ZIP file. Use "ZIP ALL" for best results. If you can't open the ZIP files, install WinZip (Windows) or StuffIT (Mac).


speculative changes to support filtering

currently the  only test I've been doing, IndexerWorderIntegrationTest

the last two narrative search tests fail.
checkSearch(1, ImmutableList.of("Narrative"), "markdown", wsid, true);
checkSearch(1, ImmutableList.of("Narrative"), "code", wsid, true);

They are testing searching against the cell_type, and poking around it looks like the values are maybe not being collected (although the other search tests don't fail, but don't know if they are testing the values transform.)
maybe I broke something in the array handling, will look at when have a chance today.

per @scanon, we're going to do something simpler for now and work on a way to run custom code for indexing stuff like this.


[language] add initial draft of translator from stackless bytecode to boogie

Motivation
This is a very drafty draft of the translator from bytecode to Boogie. I have only run it on about two mvir programs so the code is guaranteed to have a ton of bugs. I will constantly make changes to it.
Right now, the best way(in my opinion) to understand how this translator works is to feed it an mvir file and examine the generated Boogie code by running the following command inside bytecode_to_boogie/:
cargo run FILE_NAME

Output Boogie code is stored inside a file creatively named output.bpl.
I have included a few mvir programs to play with inside bytecode_to_boogie/test-mvir/. Right now output.bpl contains auto-generated code for test3.mvir.
Incomplete list of things yet to be handled:

Mutable references passed in as arguments to function calls should be returned as well because they might have been updated in the callee.
Type checking for arguments. For example, in logic operations like And we may want to require is#Boolean(op1) and is#Boolean(op2).
Eq and Neq functions for all types(Is this actually needed? Can Boogie check equality of arrays?).
Instructions related to gas/transactions/account.
Abort/assert.
LdAddr,  LdByteArray and LdStr.

Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
cargo run FILE_NAME
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

Did output.bpl get added to the commit by mistake?

No. I left it here so other reviewers can see a sample output without running the code.


Allow to use custom error codes for unexpected errors

Any unexpected error is translated to ServerError with default error code:
https://github.com/cenobites/flask-jsonrpc/blob/a72a0c4ecaf4adce8086ea277161c35482e7ddcc/src/flask_jsonrpc/site.py#L146
It would be useful to convert, for example, JWTError (https://github.com/mattupstate/flask-jwt/blob/c27084114e258863b82753fc574a362cd6c62fcd/flask_jwt/init.py#L160) to ServerError with specific error code for easier and more deterministic (avoiding string comparison) error handling in frontend.
Changing
jsonrpc_error = ServerError(data={'message': str(e)})

into something similar to:
jsonrpc_error = self.make_unexpected_error(e)

could maybe work, if we then could specify custom handler for unexpected errors.

Maybe here, instead of creating an error handling by scratch, uses the Flask Error Handling: https://flask.palletsprojects.com/en/2.2.x/errorhandling/#error-handlers
I will check out how to use that, ;).

Great, thanks @nycholas !


Restyle tables to match new UX designs

Use buttons for adding criteria instead of clicking on names. Show different text depending on whether the criteria is being added.
Convert add criteria page to a table and tweak styling.
Shrink study list text.
Refactor spacing helper to a shared location.


Need test cases.

Probably okay for now, but it'd be great to have a couple of test cases to test the functions.

I've tried pip install ogh, but I'm not able to import the library. There seems to be an encryption error going on.

@jphuong How did you install ogh? Just pip install?

yes. "pip install ogh". I looked up this error and found this related thread. [https://github.com/Miserlou/Zappa/issues/854] They suggest that it might have something to do with the .pyc, something strange about the python interpreter that could differ between the python 2.7 and python 3.6 versions.

There are many different issues that can be happening, probably related more dependencies, etc. Try installing the dependencies first with conda then do pip install. That works for me.

Gotta figure out what's missing. I can't even complete pip install. What python version are you using?

Python environment:
3.6.4.final.0
conda version:
4.3.29
It seems that you, me, and Christina are each getting different error messages. Christina's error message has to do with line 14 in the ogh.py. After inspecting that error code, I'm just going remove that line as it was a constraint from a different environment that no longer applies. That doesn't resolve your issue, Don.

It's in conda now. Try installing from conda. conda install -c conda-forge ogh

It's in conda. I think the error is just in dependencies. I am having no problem now as long as I have installed all of the dependencies first. Of course installing via conda is the easiest way out. https://anaconda.org/conda-forge/ogh


kbdi needs clear certificate, purpose

When activating a keyboard, e.g.:

the security popup is not very reassuring:

The filename is what it is (or could it have a "surface/UI" name like Keyboard activator or somesuch, while retaining the kbdi.exe CLI name?), what is worse is that the developer is unknown, when in fact it is not.

Fixed.


Cannot launch MATLAB editor after connecting to MATLAB in Visual Studio Code

Issue Description
When I connect to MATLAB in Visual Studio Code and try to open the MATLAB editor, the editor does not launch without any errors. After disconnecting to MATLAB in Visual Studio Code, the editor launch.
Steps to Reproduce

Open Visual Studio Code
Connect to MATLAB using the MATLAB extension
Launch MATLAB editor from Launchpad

Expected Behavior
The MATLAB editor should launch.
Actual Behavior
MATLAB editor does not launch.
System Information

Operating System: macOS 12.6.3
Visual Studio Code Version: 1.77.3
MATLAB Version: R2022b Update 5
MATLAB Extension Version: v1.0.1

Thank you for reporting this!
Workaround: Launch MATLAB via the terminal. For example -
cd /ApplicationsMATLAB_R2022b.app/bin
./matlab

This seems to be related to a behavior on Mac which attempts to limit the number of MATLAB instances at a time - clicking MATLAB from the Launchpad attempts to focus an existing instance instead of spawning a new one. In this case the active instance is hidden, so it appears that nothing has happened.
We are looking into ways of addressing this.

Thank you for providing the workaround. Launching MATLAB via the terminal has proven to be effective.
I understand that this behavior is specific to Mac and is related to the limitation on the number of MATLAB instances. Clicking MATLAB from the Launchpad tries to focus on an existing instance rather than opening a new one, which results in the active instance being hidden.
I appreciate your efforts in investigating ways to address this bug. Please keep me updated on any progress or potential solutions that may arise.

I wanted to provide an update on this issue: We are actively working to resolve this issue. Because the root cause of this is within MATLAB itself, the fix for this will be tied to a future MATLAB release.

I assume this problem also prevents the desktop feature (mentioned here https://blogs.mathworks.com/matlab/2024/03/05/matlab-extension-for-visual-studio-code-now-with-code-execution/ ) from working on mac?

Hi @afugls - I do not believe there should be a problem with that. This issue should only prevent a new instance of MATLAB to be launched on Mac. However, the desktop command should just display the MATLAB GUI for the instance of MATLAB already connected in VS Code.
If you encounter any issues with this, please let us know!

Hi @afugls - I do not believe there should be a problem with that. This issue should only prevent a new instance of MATLAB to be launched on Mac. However, the desktop command should just display the MATLAB GUI for the instance of MATLAB already connected in VS Code.
If you encounter any issues with this, please let us know!

Hi @dklilley - I see that it "works" but maybe not as I expected.
That is I want to:

Open VC Code and a .m file.
Connect to MATLAB
Type desktop in the MATLAB termnial
**Expected: ** A MATLAB GUI should appear in the forefront or in the dock.
Actual: MATLAB GUI opens in the "background". That is, I can see the GUI if I show all open windows (ctrl+up arrow on Mac), but I cannot, for example, cmd+tab to the GUI or click on it in the dock.

I assume this is because the connected MATLAB instance is hidden as mentioned above. However, it is not a very intuitive behavior (at least not as a first time user).

I see - thank you for the clarification!
Yes, I believe this is a side effect of the MATLAB instance being hidden. Would you mind creating a new issue specifically for this behavior with the desktop command?


Documentation in markdown

Merging now to (hopefully) trigger github action

Looks great, Roel!
Thanks a lot :)
Message ID: @.***>

Yes, agree,  excellent work.


Notice to terminate tenancy is coming up in solely rent and possession cases

@tobyfey @nonprofittechy this seems like a bigger issue. What is the solution?


[bug] Refresh browser cause previous connector connection gone

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.3.1
Current Behavior
Previous was working fine. After upgrade to 0.3.1 suddenly refresh will cause connector information become "undefined" from useAccount hook.
I have checked, the wagmi.store still storing the account data.

Expected Behavior
No response
Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

Do you have the autoConnect flag set to true? If this is not set, the wallet will not be automatically connected when the page loads. If this doesn't fix your issue, please re-open with a minimal reproducible example.

Do you have the autoConnect flag set to true? If this is not set, the wallet will not be automatically connected when the page loads. If this doesn't fix your issue, please re-open with a minimal reproducible example.

Yes already set to true. But not working.

Below are the code. Every time I refresh, I'm not able to call mint function. When I printout the useContract hook returned object, the connector is undefined.
App.js
function App() {

    const live = process.env.REACT_APP_LIVE === "true" ? true : false;

    // API key for Ethereum node
    // Two popular services are Infura (infura.io) and Alchemy (alchemy.com)
    //const infuraId = process.env.REACT_APP_INFURA_KEY;
    const alchemyId = process.env.REACT_APP_ALCHEMY_KEY2
    //const infuraId = process.env.INFURA_ID;
    console.log("alchemyId")
    console.log(alchemyId)

    const chains = defaultChains;
    const defaultChain = chain.mainnet;

    // polyfill Buffer for client
    if (!window.Buffer) {
        window.Buffer = Buffer
    }

    // Set up connectors
    const client = createClient({
        autoConnect: true,
        connectors({ chainId }) {
            console.log("chainId: " + chainId);
            const chain = chains.find((x) => x.id === chainId) ?? defaultChain;
            console.log("chain.Id: " + chain.id);
            console.log(chain.rpcUrls);
            console.log("chain.rpcUrls.alchemy: " + chain.rpcUrls.alchemy);
            const rpcUrl = chain.rpcUrls.alchemy
                ? `${chain.rpcUrls.alchemy}/${alchemyId}`
                : chain.rpcUrls.default

            console.log("RPCURL: " + rpcUrl)
            return [
                new InjectedConnector(),
                new CoinbaseWalletConnector({
                    options: {
                        appName: 'Abokado',
                        chainId: chain.id,
                        jsonRpcUrl: rpcUrl,
                    },
                }),
                new WalletConnectConnector({
                    options: {
                        qrcode: true,
                        rpc: {
                            [chain.id]: rpcUrl,
                        },
                    },
                }),
            ]
        },
    })

    return (
        <Provider client={client}>
            <Router>
                <ScrollToTop>
                    <Routes>
                        {live &&
                            <Route>
                                <Route index element={<Home />} />
                                <Route path="provenance" element={<Provenance />} />
                            </Route>
                        }
                        {!live &&
                            <Route>
                                <Route index element={

} />
                            </Route>
                        }
                    </Routes>
                </ScrollToTop>
            </Router>
        </Provider>
    );
}

Minter.js
const Minter = (props) => {

    //State variables
    const [walletAddress, setWallet] = useState("");
    const [status, setStatus] = useState("");
    const [quantity, setQuantity] = useState("");
    const [totalSupply, setTotalSupply] = useState(1);
    const [buttonDisable, setButtonDisable] = useState(true);

    const { data: accountData, refetch, isLoading } = useAccount();
    console.log(accountData);

    const contract = useContractWrite(
        {
            addressOrName: contractAddress,
            contractInterface: contractABI.abi,
        },
        'mint',
    )

   console.log(contract);

    const onMintPressed = async () => {
        console.log("on Mint");
        console.log(quantity);

        contract.write(
            {
                args: [1],
                overrides: { value: ethers.utils.parseEther(0.02 + '') }
            });

    };

    return (
        <button id="mintButton" onClick={onMintPressed} >
            MINT NOW
        </button>
    );
};

export default Minter;

Looks like you need to memoize createClient or hoist oustide of App.

Its working now after move the  outside of

@tmm @bryantoh
I hv exact same issue! And Id like to know why I hv to memoize or hoist to fix it!
Is there any reason why wagmi not working preperly then?
I do know what hositing/memoization is but I cant understand what's the relationship between fixing this issue and hoisting/memoization
Thanks a lot!

@tae100k the wagmi Client is a singleton so must be only create once either initialized in the global scope or memoized in component body so it only runs once.

@tmm Thank you so much! I almost pulled an all-nighter to find out why.
If you don't mind, can I ask you a few more questions by any chance?

Based on what you commented, will refreshing create another instance of the Client? How can refreshing affect a singleton instance? (I don't know so much details about singleton pattern. I know what it is and that's it)

2.It didn't happen in my local environment, but only on the dev server. Do you think is this relevant to the internal implementation of Client or React-Query?

I'm still not sure why hoist is related to this issue...? (I can understand that memo can solve this problem because client is created only once) Maybe we are referring to another hoisting..? not the one from javascript?

I'm very keen to know why this is happening to my code beyond solving this problem!
And please feel free to recommend any post that I can find out what I'm missing!
Thank you so much and have a wonderful Sunday!


Not compatible with Firefox 59 or Chrome

Works on IE9 though.

Safari sucks, IE is the best browser

Open Chrome Dev Tools and does not do anything.

Oh you're right. Ok it is working in Chrome. Thanks for the solution @janikarh

My Boss want me to make it compatible with his father's computer and it so happens that grand-dad is using Netscape Navigator.
My job is on the line. Please help

Here is simple fix:


How do I "uninstall" homebrew-versions?

I did brew tap homebrew/versions but it doesn't have the versions of node that I want (4.3.2 to match AWS Lambda).  So I want to get rid of homebrew/versions to install homebrew formulae "the old way" from the regular formulae not in homebrew-versions.  How do "untap" homebrew-versions to remove it completely and restore homebrew to the state it was in before I tapped homebrew/versions?
Thanks!
Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] If you're seeing permission errors tried running sudo chown -R $(whoami) $(brew --prefix)?

You can erase any parts of this template not applicable to your Issue.
Bug reports:
Please replace this line with a brief summary of your issue AND if reporting a build issue include the link from:
brew gist-logs <formula>
(where <formula> is the name of the formula that failed to build).
Formula Requests:
Please note by far the quickest way to get a new formula into Homebrew is to file a Pull Request.
We will consider your request but it may be closed if it's something we're not actively planning to work on.

brew untap homebrew/versions

Also, please check man brew in future, thanks.


enhance output of clone.js

Here are a few enhancements to clone.js that should help folks identify and fix lint issues in repositories.
This pull request includes:

filter out stdout from the git clone/pull command in order to reduce noise
add github URL to each test message. This should help jump into failing repos quickly.
add pass tests for disabled repositories. This should give some visibility to disabled repositories so no one forgets them!

Here is a sample of the output after this PR (trucated a bit):
TAP version 13
# api: lintFiles
ok 1 no error while linting
ok 2 result is an object
ok 3 should be equal
# api: lintText
ok 4 no error while linting
ok 5 result is an object
ok 6 should have used single quotes
# Disabled Packages
ok 7 DISABLED: hms: broke since v5.0.0 (https://github.com/mafintosh/hms)
ok 8 DISABLED: hyperlog: broke since v5.0.0 (https://github.com/mafintosh/hyperlog)
ok 9 DISABLED: multileveldown: broke since v5.0.0 (https://github.com/mafintosh/multileveldown)
ok 10 DISABLED: node-gyp-install: broke since v5.0.0 (https://github.com/mafintosh/node-gyp-install)
ok 11 DISABLED: pbs: broke since v5.0.0 (https://github.com/mafintosh/pbs)
ok 12 DISABLED: dat-core: broke since v5.2.0 (https://github.com/maxogden/dat-core)
ok 13 DISABLED: npm: broke since v5.0.0 (https://github.com/npm/npm)
ok 14 DISABLED: docs: broke since v5.2.0 (https://github.com/npm/docs)
ok 15 DISABLED: event-bulk-attach: broke since v5.2.0 (https://github.com/yoshuawuyts/event-bulk-attach)
# test github repos that use `standard`
ok 16 har-convert (https://github.com/ahmadnassri/har-convert)
ok 17 echint (https://github.com/ahmadnassri/echint)
ok 18 forwarded-http (https://github.com/ahmadnassri/forwarded-http)
ok 19 har (https://github.com/ahmadnassri/har)
ok 56 simple-websocket (https://github.com/feross/simple-websocket)
ok 57 simple-peer (https://github.com/feross/simple-peer)
ok 58 standard-www (https://github.com/feross/standard-www)
standard: Use JavaScript Standard Style (https://github.com/feross/standard)
  /home/icmpdev/code/standard/tmp/snazzy/index.js:96:1: Multiple blank lines not allowed. (no-multiple-empty-lines)
not ok 59 snazzy (https://github.com/feross/snazzy)
  ---
    operator: error
    expected: |-
      undefined
    actual: |-
      [Error: non-zero exit code: 1]
    at: ChildProcess.<anonymous> (/home/icmpdev/code/standard/test/clone.js:82:21)
    stack: |-
      Error: non-zero exit code: 1
          at ChildProcess.<anonymous> (/home/icmpdev/code/standard/test/clone.js:82:24)
          at emitTwo (events.js:87:13)
          at ChildProcess.emit (events.js:172:7)
          at maybeClose (internal/child_process.js:818:16)
          at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
  ...
ok 60 torrent-discovery (https://github.com/feross/torrent-discovery)
1..140
# tests 140
# pass  139
# fail  1

npm ERR! Test failed.  See above for more details.

Meh, it looks like it's not filtering the git clone stuff... one sec.

Live view of the changes :)
https://travis-ci.org/feross/standard/jobs/87146381
Do folks like this updated output?

I like it. Let's merge when tests pass.

on this note, do you really need to clone these repos for tests? what happens when one of those repos introduce a linting error inadvertently? cant the relevant files perhaps be stored in the standard repository as a test fixture?

The current process is to disable the repository and send a PR to that repo for a fix.
Yeah its more work, but its kinda nice to have a real set of repositories to lint, especially when making potentially breaking changes. It also helps to point out subtle fixes to eslint rules (which is what happened recently I think).


Performance on mobile/edge devices

Hello, thanks for your great work!
I notice that VanillaNet has more channels (512-1024-2048-4096) than traditional deep networks.
As we all know, the bandwidth and computing power of cellphones and some edge devices (for example, Jetson series) are not so sufficient.
However, intuitively a wider network will spend more time on memory access, and the FLOPs of VanillaNet have no advantage on deep networks.
I wonder if you have tried VanillaNet under such conditions and is there any result?

Thank you for your question! Our aim is to maximize the parallelism of the network and reduce the number of layers as much as possible to achieve better speed on the GPU. The current number of channels represents a good trade-off. Reducing the channels may cause a significant decrease in accuracy with relatively little gain in speed, hence our choice of these specific channel numbers. Of course, you're welcome to try different channel numbers and train the network yourself to explore further.

Hello, thanks for your question and the recognition of our work! VanillaNet is mainly designed for GPUs with high parallelism, which is why it has a high number of channels, but also a high speed. On edge devices, where the level of parallelism is not as high, VanillaNet might not have an advantage. For these resource-limited devices, we would recommend GhostNet instead. You can find the GhostNet implementation here: https://github.com/huawei-noah/Efficient-AI-Backbones/tree/master/ghostnet_pytorch. Feel free to ask if you have any other questions!

Thanks for your kind reply and advice.


[FR] drag pdf selection to the markdown file

is it possible to directly drag the text in pdf to the markdown file, then generate a corresponding quote with a backlink?

Thank you for the idea. This is a nice feature to have.
I've done some experiments, and I think I've come close to making this happen. However, I'm having some problems and I can't say when I will be able to release it. I will keep trying.
One thing to note: by default, a drag causes a text selection update. So we will need some modifier key (say, ctrl) to tell PDF++ not to update the text selection and to get ready for a drag.
This step of pressing an extra modifier key might speed down the workflow a little bit.
Will you still find this feature helpful?

Update: The next release of PDF++ will support copying a link to an annotation or an outline item by drag-and-drop.
Text selections will be supported in the future.

This step of pressing an extra modifier key might speed down the workflow a little bit.
Will you still find this feature helpful?

very appreciate your quick move, it's still very useful even if we need to press an extra modifier key
it depends on you whether to implement this feature, what I care more about is maintainability, I occasionally thought asking too many features may decrease your passion for maintaining such a great plugin, hope you can make the balance between your study and open source

Thanks! Yes, maintaining this plugin requires a decent amount of time and energy since I have to deep into Obsidian internals. So it would be nice to keep it simple to ensure maintainability.
I appreciate you cared about that!

the new version's auto-copy action after selection is good enough, this feature request may not be necessary


Cilium agent fails (timeout) to sync kubernetes cache after restart

Support
I have cilium agent deployed through the 1.7.0-rc1 helm chart, the config is at the end. The first time the agent started on each of the nodes (with workload already there because flannel was providing the vxlan fabric) everything was fine. I rebooted the nodes to switch to complete cilium management of the network. Again everything is fine. When cilium restarts though on a node, it never becomes ready. It stops at level=info msg="Waiting until all pre-existing resources related to policy have been received" subsys=k8s-watcher. It eventually times out waiting for this condition, exits and then is restarted.
I'm now trying this on the best of hardware, but the default 3 minutes should have been plenty of time to sync and it does work perfectly on a fresh reboot of the node. To explore the timeout more I pulled the code and extended the timeout to 10 minutes from the hardcoded 3 and made corresponding changes to the liveness probe in the daemonset to let it persist long enough for that timeout.
Again, the process never finished syncing. I then enabled debug to see if any of the individual resource type cache finished sync log lines occurred. None appears, it seems the first informer for network policies never completes which is should be taxing. There are only 2 networkpolicies in this cluster. The full logs from a container with debug on in this case is here.
Since the process connects to the apiserver earlier, I don't think it is a reachability issue although it definitely feels like it. I do have the non cluster network host and port of the apiserver set in the daemonset for the agent. It is not trying to use the internal cluster service address.
For a bit of context, this cluster is very small and has nearly no work occurring in it but it has the exact same configuration as our much larger cluster installs. I'm using it as a testing ground for new infrastructure services and components. Thus I know other informer based caches do work after restart, for example cert-manager and controller-manager itself.
  auto-direct-node-routes: "false"
  bpf-ct-global-any-max: "262144"
  bpf-ct-global-tcp-max: "524288"
  cluster-name: home
  cni-chaining-mode: portmap
  container-runtime: none
  debug: "false"
  enable-host-reachable-services: "true"
  enable-ipv4: "true"
  enable-ipv6: "false"
  enable-node-port: "true"
  host-reachable-services-protos: udp tcp
  identity-allocation-mode: crd
  install-iptables-rules: "true"
  k8s-require-ipv4-pod-cidr: "true"
  masquerade: "true"
  monitor-aggregation: medium
  preallocate-bpf-maps: "false"
  sidecar-istio-proxy-image: cilium/istio_proxy
  sockops-enable: "false"
  tofqdns-enable-poller: "false"
  tunnel: vxlan
  wait-bpf-mount: "true"

Edit:
Updated gist to one with timestamps for each line

After posting this I decided to try removing the overrides for apiserver host and port. The resulted in the pods all becoming ready, but that has the issue on restart the pods can't reach the apiserver because there has not been a route for the in cluster service setup yet.

Dug a bit more into this any my current opinion is that after restart during  a reconfigure the opened and working connection to the apiserver is somehow broken when using the public interface the points to an external load balancer for the apiservers. If I set the kubernetes host to be the kubernetes.default.svc domain, all works on cilium restart because that service endpoints use the private internal ip addresses of each apiserver. If I set the advertise-ip flag on the apiserver to be the public load balancer then restart fail when using kubernetes.default.svc which matches the starting behavior.
If I change the kubernetes hostname to be one of the apiserver host names, restarts work. This would appear to show that DNS not at fault, but it is perplexing because the public IP on the same segment as the private IPs.
More clearly, each of the apiserver hosts has a single private IP of

<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

The load balancer is uses IP <IP_ADDRESS>. The load balancer is on a separate host and not part of the kubernetes system. It is pfsense actually.

Thanks for the issue. A few questions below:
1.Have you tried setting KUBERNETES_SERVICE_HOST to <IP_ADDRESS> instead of k8s.hsv-a.ceh.im?
2. Have you tried running in a clean environment, i.e. without flannel being previously installed?
3. Can you record pcaps with tcpdump for cilium-agent -> LB and LB -> kube-apiserver?

Thanks for taking the time with this.

I had not tried straight IP, but on doing so the behavior is the same as using the DNS. The first iteration actually failed at TLS setup because I didn't have the IP in the certificate, but after resolving that it timeouts on waiting for resource sync.
I haven't tried from a completely from the ground up kubernetes, but all the nodes have been cycled to clean state. The only remnants left are the annotations on the resources from flannel, e.g. flannel.alpha.coreos.com/backend-data: '{"VtepMAC":"fa:c9:c6:35:ee:05"}' on the node resource
I was able to get from cilium-agent->LB
cilium-to-lb.pcap.zip Getting from the LB to apiserver may be a bit more challenging to keep it content only related to this traffic. I'll work on that.

I was attempting to get the LB->apiserver traffic in an isolated manner, and think the problem may lie with the LB in pfsense that I'm using. I created a simple haproxy instance on my laptop to front the apiservers and added my laptop IP as an identity to the apiserver certificate. I verified the proxy was working and then configured cilium to use it. Well everything worked this time, so I guess I'll look into a different LB setup.

Thanks for letting us know.


Fix email tint bug

In #271 @starsirius found that the mail compose view would "lose" the value of the to field on blur. Turns out that this was a side-effect of setting the tint color on Window. This PR addresses the bug by forcing the tint color to be black prior to presenting the mail composer view and then switching it back to whatever it was on dismissal of that view.
Technically speaking all that's required here is 3253c2a, but it felt weird to not extract some helper methods, so that's what I did with 64ac201 and fb37924. Interested in feedback on this!

It's a reasonable hack IMO - probably worth a note of why inside the AREmailComposer.m

Thanks for the quick fix, @jonallured. I will let @orta merge since I still need to get myself familiar with the code.

Cool, fine by me - then this needs a CHANGELOG entry, we're on a new as a release as just shipped

A dumb question: what does the "AR" prefix stand for?

Artsy - Objective-C has a single runtime for all classes (including all of apples) and so everyone has to pick their own prefixes for all classes

Full story

Ohhh thanks for the reference! I was hoping it has something to do with that "AR" in my mind. 😝

I am too afraid of pressing the button. I will let @orta do it. Thanks again, @jonallured!

@orta done!

👍

If you want to give it a shot, try run make distribute to deploy a beta

No dice:
jon@magneto:~/code/energy(master)% make distribute
make: *** No rule to make target `distribute'.  Stop.

Maybe that should be deploy instead?

Sounds right 👍

Well, the good news is that I got a build going, but the bad news is that it failed. :\
Looks like maybe it's a code signing problem:

[07:37:49]: There are no local code signing identities found.
You can run security find-identity -v -p codesigning to get this output.
This Stack Overflow thread has more information: https://stackoverflow.com/q/35390072/774.
(Check in Keychain Access for an expired WWDR certificate: https://stackoverflow.com/a/35409835/774 has more info.)

But maybe not - the error is actually that it timed out. Any ideas @orta?


Socket Exception by using icap-client with File

Hallo,
thank you for providing your icap-Library on Github.
First i used your icap-Library to call my icap-server without uploading any file this works fine. The response Message from server contains status code 204.
But when i added a file with the setHttpRequestBody-Method i getting an exception.
This is my stacktracke:
Exception in thread "main" net.rfc3507.client.ICAPException: java.net.SocketException: Connection reset by peer: socket write error
at net.rfc3507.client.ICAPClient.execute(ICAPClient.java:94)
at net.rfc3507.client.Main.main(Main.java:24)
Caused by: java.net.SocketException: Connection reset by peer: socket write error
at java.net.SocketOutputStream.socketWrite0(Native Method)
at java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:111)
at java.net.SocketOutputStream.write(SocketOutputStream.java:143)
at net.rfc3507.client.ICAPClient.performAdaptation(ICAPClient.java:211)
at net.rfc3507.client.ICAPClient.execute(ICAPClient.java:92)
... 1 more
My Main-Method to use your library was:
    final String SERVER = "myserver";
    final int PORT = 1344;
    final String SERVICE = "AVSCAN";

    File files = new File("C:\\dev\\8473.PNG");
    System.out.println(files.toPath());

    ICAPClient icapClient = new ICAPClient(SERVER, PORT);
    ICAPRequest icapRequest = new ICAPRequest(SERVICE, ICAPRequest.Mode.REQMOD);
    //icapRequest.setHttpRequestBody(files);
    byte[] fileContent = Files.readAllBytes(files.toPath());
    icapRequest.setHttpRequestBody(fileContent);
    ICAPResponse icapResponse = icapClient.execute(icapRequest);
    System.out.println(icapResponse.toString());

Hello!
Thank you for your feedback.
I am going to check your issue.

Hi,
I have tested your snippet against ClamAV icap server on linux, Debian distro, and no exception was thrown.
I have noticed your snipped did not provide a "HTTP Request Header" input; I believe some icap-server implementation require a request header input in order to work together a request body, and some does not.
I would like to ask you to make some changes on your code and try again; please check whether the issue still occurs:
final String SERVER = "myserver";
final int PORT = 1344;
final String SERVICE = "AVSCAN";

File files = new File("C:\\dev\\8473.PNG");
System.out.println(files.toPath());

ICAPClient icapClient = new ICAPClient(SERVER, PORT);
ICAPRequest icapRequest = new ICAPRequest(SERVICE, ICAPRequest.Mode.REQMOD);
//icapRequest.setHttpRequestBody(files);
byte[] fileContent = Files.readAllBytes(files.toPath());
//------------------------------------------------------------------------------
String contentType = "image/png";
String resource = files.getName();
String requestHeader
      		= "POST /"+resource+" HTTP/1.1\r\n"
      		+ "Host: localhost\r\n"
       		+ "User-Agent: Java-ICAP-Client/1.0\r\n"
       		+ "Content-Type: "+contentType+"\r\n"
       		+ "Content-Length: "+fileContent.length+"\r\n"
       		+ "\r\n";
icapRequest.setHttpRequestHeader(requestHeader.getBytes());
//------------------------------------------------------------------------------
icapRequest.setHttpRequestBody(fileContent);
ICAPResponse icapResponse = icapClient.execute(icapRequest);
System.out.println(icapResponse.toString());


Update index.md

minor edits

This index.md was adjusted from the forkless runtime upgrade tutorial, so we might want to apply some of your suggestions to it as well.

@apopiak can you be more specific about the changes that you think should be applied to the the tutorial? I have opened a PR https://github.com/substrate-developer-hub/substrate-developer-hub.github.io/pull/772 that reverts many of the changes in this one but I'm happy to hear dissenting opinions. My apologies for missing this PR and some of the other related ones. @athei please feel free to comment on #772 as well - I would like to try our best to come to some type of consensus on this issue.


java.net.socketexception socket failed eacces (permission denied) android studio

Hi, I'm using your library but i'm trying to connect to OpenERP but i keep getting this error and yes i already added the internet permission in the manifest file
HELPPP

NEver mind fixed it :dancer:


Whenever I join my server with VanishV2 latest plugin version, it gives me this error.

13:29:02] [Server thread/CRITICAL]: Error: "Class 'Ds\Queue' not found" (EXCEPTION) in "plugins/VanishV2.phar/src/superbobby/VanishV2/libs/muqsit/invmenu/session/PlayerNetwork" at line 52
[13:29:02] [Server thread/CRITICAL]: #0 plugins/VanishV2.phar/src/superbobby/VanishV2/libs/muqsit/invmenu/session/PlayerManager(38): superbobby\VanishV2\libs\muqsit\invmenu\session\PlayerNetwork->__construct(object pocketmine\Player, object superbobby\VanishV2\libs\muqsit\invmenu\session\network\handler\ClosurePlayerNetworkHandler)
[13:29:02] [Server thread/CRITICAL]: #1 plugins/VanishV2.phar/src/superbobby/VanishV2/libs/muqsit/invmenu/InvMenuEventHandler(69): superbobby\VanishV2\libs\muqsit\invmenu\session\PlayerManager::create(object pocketmine\Player)
[13:29:02] [Server thread/CRITICAL]: #2 pmsrc/src/pocketmine/plugin/MethodEventExecutor(42): superbobby\VanishV2\libs\muqsit\invmenu\InvMenuEventHandler->onPlayerJoin(object pocketmine\event\player\PlayerJoinEvent)
[13:29:02] [Server thread/CRITICAL]: #3 pmsrc/src/pocketmine/plugin/RegisteredListener(80): pocketmine\plugin\MethodEventExecutor->execute(object superbobby\VanishV2\libs\muqsit\invmenu\InvMenuEventHandler, object pocketmine\event\player\PlayerJoinEvent)
[13:29:02] [Server thread/CRITICAL]: #4 pmsrc/src/pocketmine/event/Event(88): pocketmine\plugin\RegisteredListener->callEvent(object pocketmine\event\player\PlayerJoinEvent)
[13:29:02] [Server thread/CRITICAL]: #5 pmsrc/src/pocketmine/Player(1099): pocketmine\event\Event->call()
[13:29:02] [Server thread/CRITICAL]: #6 pmsrc/src/pocketmine/network/mcpe/PlayerNetworkSessionAdapter(310): pocketmine\Player->doFirstSpawn()
[13:29:02] [Server thread/CRITICAL]: #7 pmsrc/src/pocketmine/network/mcpe/protocol/SetLocalPlayerAsInitializedPacket(45): pocketmine\network\mcpe\PlayerNetworkSessionAdapter->handleSetLocalPlayerAsInitialized(object pocketmine\network\mcpe\protocol\SetLocalPlayerAsInitializedPacket)
[13:29:02] [Server thread/CRITICAL]: #8 pmsrc/src/pocketmine/network/mcpe/PlayerNetworkSessionAdapter(109): pocketmine\network\mcpe\protocol\SetLocalPlayerAsInitializedPacket->handle(object pocketmine\network\mcpe\PlayerNetworkSessionAdapter)
[13:29:02] [Server thread/CRITICAL]: #9 pmsrc/src/pocketmine/network/mcpe/protocol/BatchPacket(130): pocketmine\network\mcpe\PlayerNetworkSessionAdapter->handleDataPacket(object pocketmine\network\mcpe\protocol\SetLocalPlayerAsInitializedPacket)
[13:29:02] [Server thread/CRITICAL]: #10 pmsrc/src/pocketmine/network/mcpe/PlayerNetworkSessionAdapter(109): pocketmine\network\mcpe\protocol\BatchPacket->handle(object pocketmine\network\mcpe\PlayerNetworkSessionAdapter)
[13:29:02] [Server thread/CRITICAL]: #11 pmsrc/src/pocketmine/Player(3325): pocketmine\network\mcpe\PlayerNetworkSessionAdapter->handleDataPacket(object pocketmine\network\mcpe\protocol\BatchPacket)
[13:29:02] [Server thread/CRITICAL]: #12 pmsrc/src/pocketmine/network/mcpe/RakLibInterface(169): pocketmine\Player->handleDataPacket(object pocketmine\network\mcpe\protocol\BatchPacket)
[13:29:02] [Server thread/CRITICAL]: #13 pmsrc/vendor/pocketmine/raklib/src/server/ServerHandler(95): pocketmine\network\mcpe\RakLibInterface->handleEncapsulated(string[18] <IP_ADDRESS> 46273, object raklib\protocol\EncapsulatedPacket, integer 0)
[13:29:02] [Server thread/CRITICAL]: #14 pmsrc/src/pocketmine/network/mcpe/RakLibInterface(109): raklib\server\ServerHandler->handlePacket()
[13:29:02] [Server thread/CRITICAL]: #15 pmsrc/src/pocketmine/network/mcpe/RakLibInterface(99): pocketmine\network\mcpe\RakLibInterface->process()
[13:29:02] [Server thread/CRITICAL]: #16 pmsrc/vendor/pocketmine/snooze/src/SleeperHandler(113): pocketmine\network\mcpe\RakLibInterface->pocketmine\network\mcpe{closure}()
[13:29:02] [Server thread/CRITICAL]: #17 pmsrc/vendor/pocketmine/snooze/src/SleeperHandler(75): pocketmine\snooze\SleeperHandler->processNotifications()
[13:29:02] [Server thread/CRITICAL]: #18 pmsrc/src/pocketmine/Server(2155): pocketmine\snooze\SleeperHandler->sleepUntil(double<PHONE_NUMBER>.4526)
[13:29:02] [Server thread/CRITICAL]: #19 pmsrc/src/pocketmine/Server(1992): pocketmine\Server->tickProcessor()
[13:29:02] [Server thread/CRITICAL]: #20 pmsrc/src/pocketmine/Server(1586): pocketmine\Server->start()
[13:29:02] [Server thread/CRITICAL]: #21 pmsrc/src/pocketmine/PocketMine(291): pocketmine\Server->__construct(object BaseClassLoader, object pocketmine\utils\MainLogger, string[16] /home/container/, string[24] /home/container/plugins/)
[13:29:02] [Server thread/CRITICAL]: #22 pmsrc/src/pocketmine/PocketMine(321): pocketmine\server()
[13:29:02] [Server thread/CRITICAL]: #23 pmsrc(11): require(string[71] phar:///home/container/PocketMine-MP.phar/src/pocketmine/PocketMine.php)

Same error also you said dont use source file use phar instead that doesn't work you gave me dev 87 phar it also not working

update to php 8

Wtf dude my pocketmine verison is 3.22.0

Ok so now this plugin is ded for 3.0.0
pls use this plugin on pocketmine 4.0.0 thanks.

Please don't be toxic

Php version and pmmp version is not the same thing

and yes php 8 is compatible with pmmp 3.0.0

actually I will need your os, pmmp version, php version and plugin version because I can't reproduce the issue on both php 8 and 7.4 on the lastest pocketmine version

Hello!
I don't know who you're talking to but; If you are talking to me, My PMMP version is the latest one from 2 days ago I think. I think certain that I'm using the latest plugin for the Plugin version because I got it from poggit.

can you try this version for me https://poggit.pmmp.io/r/131998/VanishV2_dev-89.phar

can you try this version for me https://poggit.pmmp.io/r/131998/VanishV2_dev-89.phar

Still same error for me :/

tried to reproduce the issue and the only way I found to do it was to run from source with a old InvMenu virion, so make sure you don't have DEVirion installed and the that the source is not in your plugin folder only the phar I sent

I tried to reproduce the issue and the only way I found to do it was to run from source with a old InvMenu virion, so make sure you don't have DEVirion installed and that the source is not in your plugin folder only the phar I sent

But how some plugins needs virions even Auction house needs inv menu virion

no they don't unless you're running from source you don't need to use DEVirion it's already installed in the phar. we are not allowed to publish plugins that require the use of DEVirion on poggit and it should only be used for development purposes

no they don't unless you're running from source you don't need to use DEVirion it's already installed in the phar. we are not allowed to publish plugins that require the use of DEVirion on poggit and it should only be used for development purposes

Ok bro i tested in my test server without virions even a plugin i only have vanish but it sends same error.

try on a fresh server and put this plugin only

try on a fresh server and put this plugin only

It says Invmenu virion not found disabling the plugin

that means you are running the plugin from source

so is the issue fixed? I really like this plugin, hope you guys would be able to fix it
thanks and good luck!

Well as far as I can't tell there's no actual issue, it works for everyone and me.  I'm going to repeat them so you don't have to search.

Delete DEVirion
download the latest version on poggit which is v2.0 and put it in your plugin folder
Delete the folder with the plugin source from your plugin folder
if it still doesn't work reinstall pocketmine and put in your plugin folder the latest version on poggit (v2.0)

Ok so i have tried in a test server i dont have virions too i dont have a single plugin except vanish now i get this error this plugin is from latest update of poggit i think 2.0.0
2021-07-17 [17:43:48] [Server thread/CRITICAL]: InvalidStateException: "Tried to schedule task to disabled scheduler" (EXCEPTION) in "pmsrc/src/pocketmine/scheduler/TaskScheduler" at line 128
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #0 pmsrc/src/pocketmine/scheduler/TaskScheduler(80): pocketmine\scheduler\TaskScheduler->addTask(object pocketmine\scheduler\ClosureTask, integer -1, integer 100)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #1 plugins/VanishV2 (7).phar/src/superbobby/VanishV2/libs/muqsit/invmenu/InvMenuEventHandler(61): pocketmine\scheduler\TaskScheduler->scheduleRepeatingTask(object pocketmine\scheduler\ClosureTask, integer 100)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #2 plugins/VanishV2 (7).phar/src/superbobby/VanishV2/libs/muqsit/invmenu/InvMenuHandler(55): superbobby\VanishV2\libs\muqsit\invmenu\InvMenuEventHandler->__construct(object superbobby\VanishV2\VanishV2)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #3 plugins/VanishV2 (7).phar/src/superbobby/VanishV2/VanishV2(52): superbobby\VanishV2\libs\muqsit\invmenu\InvMenuHandler::register(object superbobby\VanishV2\VanishV2)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #4 pmsrc/src/pocketmine/plugin/PluginBase(116): superbobby\VanishV2\VanishV2->onEnable()
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #5 pmsrc/src/pocketmine/plugin/PluginManager(552): pocketmine\plugin\PluginBase->setEnabled(boolean 1)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #6 pmsrc/src/pocketmine/Server(1785): pocketmine\plugin\PluginManager->enablePlugin(object superbobby\VanishV2\VanishV2)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #7 pmsrc/src/pocketmine/Server(1771): pocketmine\Server->enablePlugin(object superbobby\VanishV2\VanishV2)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #8 pmsrc/src/pocketmine/Server(1584): pocketmine\Server->enablePlugins(integer 1)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #9 pmsrc/src/pocketmine/PocketMine(291): pocketmine\Server->__construct(object BaseClassLoader, object pocketmine\utils\MainLogger, string[16] /home/container/, string[24] /home/container/plugins/)
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #10 pmsrc/src/pocketmine/PocketMine(321): pocketmine\server()
2021-07-17 [17:43:48] [Server thread/CRITICAL]: #11 pmsrc(11): require(string[71] phar:///home/container/PocketMine-MP.phar/src/pocketmine/PocketMine.php)

did you also reinstall pocketmine?

did you also reinstall pocketmine?

It is a new fresh server i am using mutli server so i can able to create unlimited servers

Nvm it works


aws-bedrock agent failing infrequently with failureReason': 'LLM prediction format incorrect'

Hi, I have created an agent and it always works fine when tested with aws console.
Agent contains a lambda function which calls claude for some generation activity. It also has a matching swagger to trigger operations of function.
However, when trigger it(agent) from code(lambda function with python), providing agent and alias id, it works as expected sometimes and fails quite a lot of times with below error -
trace': {'agentId': 'SJ4AHHVXX', 'agentAliasId': 'SZD0OGXXX', 'sessionId': 'SESSMAINT1', 'trace': {'failureTrace': {'traceId': 'd34fe4ba-78b6-4ccf-8aa5-682e74890d31-0', 'failureReason': 'LLM prediction format incorrect'}}}} {'chunk': {'bytes': b'Sorry, I am unable to assist you with this request.'}}
{'chunk': {'bytes': b'Sorry, I am unable to assist you with this request.'}}
sample lambda code -
def invoke_maintenance_agent(searchterm):
try:
response = bedrock.invoke_agent( agentId='SJ4AHHVXX',
agentAliasId='SZD0OGXXX',
sessionId='SESSMAINT1',
inputText=searchterm )
completion = response.get('completion')
if completion:
print('inside completion')
all_records=[]
for event in completion: # loop over to get each record
print(event)
if 'chunk' in event:
chunk = event.get('chunk')
bytes = chunk.get('bytes')
all_records.append(bytes)
agentResponse = str(all_records[0],'utf-8')
else:
agentResponse = 'Some issue in invoking agent'
return agentResponse
except Exception as e:
     raise RuntimeError(f"Exception - {str(e)}")

Kindly assist. Thank You.

I was able to solve this error same evening I posted the issue. Guess I should have posted here earlier.
It actually meant that I was not able to trigger the underlying lambda in the agent. Reason, I suppose is due to some session mgmt issues. Managing the session id fixed this issue.


SMS backend

Current coverage is 56.32% (diff: 70.46%)

Merging #6 into master will increase coverage by 11.19%

@@             master         #6   diff @@
==========================================
  Files            19         28     +9
  Lines           698       1218   +520
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            315        686   +371
- Misses          383        532   +149
  Partials          0          0

Powered by Codecov. Last update 2e8b04b...3f06fcf


yeet ethers-rs

ethers-rs has been superceded by alloy, and is now in maintenance mode. hubble has already migrated to alloy, voyager should as well - this will be done when we integrate the changes from #2965 into voyager.

ethers has been yote


fix(docs): update docs grammar

Current coverage is 98.73% (diff: 100%)

Merging #172 into master will not change coverage

@@             master       #172   diff @@
==========================================
  Files             5          5
  Lines           474        474
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            468        468
  Misses            6          6
  Partials          0          0

Powered by Codecov. Last update 5ace8b8...2369c26

thanks!


Update stereo demo for latest depthai

Fix setting frame timestamp for API change.
To run without using cameras, on static frames from dataset folder, run:
python3 main.py -static

Thanks!


Bump tracing dependency to 0.1.36

This was already required in practice, we just forgot to bump the version bound accordingly.
Fixes #1069.

I'm not sure I'd say this warrants a patch release, since there's a pretty trivial workaround ("just run cargo update"). On the other hand, shipping a version with broken bounds makes life annoying for anyone trying to use -Z minimal-versions, which would be an argument in favour of even yanking the .0.

Having a patch release with this fix would be great

Should it be 0.1.37 ?

Should it be 0.1.37 ?

The change that we depend on was introduced in 0.1.36. This is just the minimum bound, most users will still get 0.1.37 anyway unless they have a reason to lock it to something lower.

Thanks !


What is the starting position of each dial?

Is it straight up? At specific emoji? Random emoji?

Random totally works. Or straight up.
On Tue, Feb 27, 2018 at 3:50 PM, Lou Huang<EMAIL_ADDRESS>wrote:

Is it straight up? At specific emoji? Random emoji?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BadIdeaFactory/the-emoji-compass/issues/13, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAlcOC5w8Upg6Y4wZPI5Fw6cG4JzybOuks5tZGpxgaJpZM4SVnug
.

Random it is!


TypeError: can only concatenate list (not "NoneType") to list

I'm getting the following error when trying to use the example you gave for keract:
Traceback (most recent call last):
  File "new.py", line 139, in <module>
    activations = get_activations(model, test_volumes_mri)
  File ".../.local/lib/python3.7/site-packages/keract/keract.py", line 80, in get_activations
    activations = _evaluate(model, layer_outputs, x, y=None)
  File ".../.local/lib/python3.7/site-packages/keract/keract.py", line 27, in _evaluate
    return f(x_ + y_ + sample_weight_)
TypeError: can only concatenate list (not "NoneType") to list

The code (just the part that crashes. The model itself compiles, trains and evaluates just fine):
from keract import get_activations,display_activations,display_heatmaps,get_gradients_of_trainable_weights,get_gradients_of_activations
activations = get_activations(model, test_volumes_mri)
display_activations(activations, cmap="gray", save=True)
display_heatmaps(activations, input_image, save=True)
get_gradients_of_trainable_weights(model, test_volumes_mri, images_groundtruth)
get_gradients_of_activations(model, test_volumes_mri, images_groundtruth)

What's your model architecture and what is the shape of test_volumes_mri? Also, are keras, keract, TensorFlow etc. up to date on your system?

I'm using the new TF 2.0 RC1 release. test_volumes_mri is a 4 dimensional tensor, with an empty 4th dimension (eg 250x250x250x1). Model architecture is essentially Conv2D alternating with BatchNormalization to get from 250x250x250 to 250x250x1.

I'm using the new TF 2.0 RC1 release. test_volumes_mri is a 4 dimensional tensor, with an empty 4th dimension (eg 250x250x250x1). Model architecture is essentially Conv2D alternating with BatchNormalization to get from 250x250x250 to 250x250x1.

It might be that TF 2.0 is messing with keract, we have already had a report (#58) of tf.keras in TF 2.0 not playing nicely with keract

That's my assumption too. It's a shame, I really wanted to see what keract had to offer

Same issue with TF 1.14
This is the model:
https://github.com/FlorinAndrei/TensorAim/blob/keract_issue/old/model.py
This is the training script:
https://github.com/FlorinAndrei/TensorAim/blob/keract_issue/old/train_real_world_model.py
I tried to do get_activations() / display_activations() in this script:
https://github.com/FlorinAndrei/TensorAim/blob/keract_issue/old/camera_live_test.py

@starkgate @FlorinAndrei work to support newest versions of Tensorflow 1.14 and 2.0 will start soon :)

@FlorinAndrei can you extract the simplest snippet where the call to keract failed?

Fixed in https://github.com/philipperemy/keract/pull/65


Button overflow in blog section

Please assign this task to me

Hello there. Can you assign this task to me as I have a good amount of experience in web development, especially styling them.

hi ,
can you please assign me this task under GSSOC 24 . I would really love to contribute

Hii @akshitagupta15june , i would like to work on this issue. I can fix this issue as i have experience.
can you assign it to me under GSSOC24.


These are popular

I also think we should advertise gitter while maintaining the irc as a
'scratch pad' for off topic/tangential discussion.

I'm curious what the in progress label means in the context of this PR.

Oh, I loaded up the waffle.io page again to see whether I liked it.  I'm still not sure that I do.  That label was automated noise.

Ah, cool.


United Kingdom Output Formatting

I know we're really awkward in the UK but we use Metric units for temperature and imperial for wind speed (so Centigrade and MPH). By default, wttr.in using Metric units for UK (understandable as we're in EMEA) but there's no way to force it into the correct combination.
Could we please either get a way of specifying the Wind and Tempareture units or just specifying the UK formatting?
Thank you!

This issue is more than UK: In Chinese we use Beaufort scale (proposed in #231) for wind speed. It'll be nice to have default formatting per language.

@pnlrogue1 We have one option for wind (?M for m/s; and by default it is km/h),
but we need some option for MPH too. Maybe ?i for miles?
@liyishuai I am not sure that it is a good idea to bind to the languages (at least, not for all languages; for example, it would not work for English). But we definitely need different output formats for wind.
Maybe ?b for Beaufort scale? (#231)

Indeed, I should request "default formatting per locale" rather than "per language".
?b for Beaufort looks good.

The problem is, that the local is not passed by default by the browser; and for the multi-language versions like zh.wttr.in, we have in the server name only the language name, and not the country name.
?b for Beafort accepted.


inspect: missing copyright notice and Boost license info in test/core/image_processing/

Actual behavior

The Boost inspect tool reports the following violations in test/core/image_processing/*:
/mnt/d/boost.wsl/libs/gil$ inspect -text
Boost Inspection Report
Run Date: 19:46:33 UTC, Thursday 08 August 2019
...
Details:
  *Lic* missing Boost license info, or wrong reference text
  *C* missing copyright notice
...
test/core/image_processing/lanczos_scaling.cpp:
    *C*
    *Lic*
  test/core/image_processing/simple_kernels.cpp:
    *C*
    *Lic*

Expected  behavior

The files should have the copyright and license added.

Should I PR straight into develop?


Implement data checks for invalid websites 404 error

This is an enhancement request for data presented on https://map.schnelltestportal.de/
Issue
Some test stations present a URL under "Infos & Termin" which does not work.
Suggested enhancement
Regularly check that all URLs are functioning and resolve any which are not.
Example
https://map.schnelltestportal.de/api/centers?latne=48.13644999876734&lngne=11.584551160296831&latsw=48.13352506185528&lngsw=11.579905574283037&dcc=true
{
    "centers": [
        {
            "uuid": "edf10170-c64d-11ec-aa9f-0a580a82360d",
            "name": "Isartor - München",
            "email"<EMAIL_ADDRESS>            "website": "https://www.vitotest.de/isartor",
            "coordinates": {
                "longitude": 11.5820649,
                "latitude": 48.1351013
            },
            "logo": null,
            "marker": null,
            "address": "Thomas-Wimmer-Ring 1 - 3, 80539 München",
            "openingHours": [
                "Mo - Sa: 07:30 - 11:00 & 11:45 - 16:30"
            ],
            "addressNote": null,
            "appointment": "Possible",
            "testKinds": [
                "Antigen",
                "PCR"
            ],
            "dcc": true
        }
    ]
}

https://www.vitotest.de/isartor gives
404
This page could not be found.

The problem still exists for the example testing station.

Will bring it up.
Mike McCready @.***> schrieb am Do., 18. Aug. 2022,
18:04:

The problem still exists for the example testing station.
—
Reply to this email directly, view it on GitHub
https://github.com/corona-warn-app/cwa-map-backend/issues/4#issuecomment-1219671809,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ7I3PAZW4BIMZLV3YRYPDVZZNHPANCNFSM52S4FZJQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Document building tensorflow-gpu from source?

I have built Tensorflow 1.2 from source with GPU support. However the Python package name is "tensorflow" and not "tensorflow-gpu" as is common. We also have machines that don't have GPUs in them so I'd like to have them both. However I could not find any documentation on how to do this when building from source.

Hi,
It's already here -> https://www.tensorflow.org/install/install_sources#ConfigureInstallation. You need to have bazel, GPU prerequisites in order to execute ./configure. where you will be asked following questions.
Do you wish to build TensorFlow with CUDA support? [y/N] Y
CUDA support will be enabled for TensorFlow
Please specify which gcc should be used by nvcc as the host compiler. [Default is /usr/bin/gcc]:
Please specify the Cuda SDK version you want to use, e.g. 7.0. [Leave empty to use system default]: 8.0
Please specify the location where CUDA 8.0 toolkit is installed. Refer to README.md for more details. [Default is /usr/local/cuda]:
Please specify the cuDNN version you want to use. [Leave empty to use system default]: 5
Please specify the location where cuDNN 5 library is installed. Refer to README.md for more details. [Default is /usr/local/cuda]:
Please specify a list of comma-separated Cuda compute capabilities you want to build with.
You can find the compute capability of your device at: https://developer.nvidia.com/cuda-gpus.

This third party link might help too, https://alliseesolutions.wordpress.com/2016/09/08/install-gpu-tensorflow-from-sources-w-ubuntu-16-04-and-cuda-8-0/

No, that's not what I'm after. I've went through that and confirmed I have a working tensorflow + GPU package. The issue is that the python module is named tensorflow and not tensorflow-gpu.

I misunderstood your question by reading heading only. I think if you try to install a common tensor-flow on non GPU machine . It won't have CUDA , cuDNN. So, it is obvious that will use CPU resources.
./configure ask you CUDA support question which is sufficient for non GPU machines for simplicity.
I believe for making development and distribution compatible and easy with each other they have singular source and for other use cases I guess.
Here are some points in order to modify source : -

Try to experiment by removing third_party/gpus folder.

The easiest way to achieve this by comparing tensor flow-cpu vs tensor flow distribution source side by side and make your own version of tensoflow cpu by making changes in source. It's experimental and not sure will work or not.

@jart any ideas on this one?

At the moment, this is not possible. You will need to conditionally install CPU or GPU packages on each machine based on existence of GPU on them.
CC @martinwicke

I think you just need to  invoke bazel build -- //tensorflow/tools/pip_package:build_pip_package  --gpu to have the --project_name of the setup.py of the package set to tensorflow_gpu:
See:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/pip_package/build_pip_package.sh#L57

Even if the pip packages are built with a different name, when installed they still occupy the same directory with the files with same names under your python library directories. Therefore, If you try to install both tensorflow and tensorflow-gpu you will overwrite the files the first installs with the second installation.
In the end, your installation may even not work.

It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

For anyone following @Mistobaan's advice (https://github.com/tensorflow/tensorflow/issues/11193#issuecomment-326819113), you should invoke bazel build normally, and only then use the --gpu arg:
bazel-bin/tensorflow/tools/pip_package/build_pip_package ${TF_ROOT}/pip/tensorflow_pkg --gpu


Package request: cups-idprt

Project description
CUPS drivers for iDPRT label printers
Metadata

homepage URL: https://www.idprt.com/
source URL: https://www.idprt.com/Driver/ / https://www.idprt.com/downloads/
license: proprietary
platforms: unix, linux, darwin

Add a :+1: reaction to issues you find important.

There is no need to create an issue for a software you are packaging. Create the package right away.

There is no need to create an issue for a software you are packaging. Create the package right away.

I wasn't planning on packaging the software. I opened this issue last week.


Use MathJax in pkgdown site

Instead of including screenshots of rendered TeX, we can just include raw TeX and have pkgdown render it with MathJax (since it supports it)

Oh nice! I didn’t know that was possible.

Somewhere along the line the pkgdown site became broken (probs something I did). Just a note here that we should probably just kind of start over on it. The site was created through a PR I just merged in so I didn't really have anything to do with it and I'm not sure why it doesn't work now... 🤷♂

It might be helpful to go the /docs route instead of always remembering to switch to the gh-pages branch

Yeah that's kind of what I was thinking. I'm familiar with publishing that way, but not the way Jonathan set it up. He was trying to automate stuff for us which is nice, but it becomes burdensome if it's just another new thing to learn. I'm not even sure what the benefit is of the approach he took to the more standard (in my experience anyway) docs folder method.

In part I think it's path dependency—for years, the gh-pages branch was the only way to publish stuff as HTML and so tons of people still use it (that's actually why I've never used github pages; even though I've been using github for like a decade, doing the branch thing was always a horrible hassle). I think GitHub added the docs option more recently so it'd be easier to use.

Ah... okay, cool. That makes sense. Using pkgdown::build_site() will create the docs folder and everything and it seems super straightforward, so we should probably just go that direction. I can do that later. I just hadn't at this point because I didn't want to screw up the current version (which is already screwed up so I'm not sure why that stopped me...).

Cool, that's tons easier. We can probably delete the gh-pages branch and (maybe somehow?) reconfigure the repository settings to look in docs instead of the branch

We can close this since texPreview automatically makes the images of the equations and we don't need to worry about MathJax


"Hidden window" mode

It would be nice to be able to run GUI utilities without having the console window displayed. For example, I've made a shortcut to C:\Windows\System32\bash.exe -c "DISPLAY=localhost:0.0 uxterm", but when I run it I end up with two windows (the extra one being unresponsive).

I've tried start /B bash -c "DISPLAY=localhost:0.0 nohup uxterm &" && exit, but with no avail :(

I've finally succeded with the following:
powershell -windowstyle hidden -command "&{ bash -c 'DISPLAY=localhost:0.0 uxterm' }"

However, there's still a quick window flash when the command starts, but have no idea how to remove it.

I second that. The PowerShell shortcut works but it has usability issues.

From build 17666 release notes

Add --exec option for wsl.exe to invoke a single binary without a shell.

Maybe it's what you are looking for.

>wsl --exec <my application here>
/bin/bash: --: invalid option

I don't seem to have build 17666 yet. :-)

Only Insiders in the fast Ring can get it. Others will need to wait for Fall 2018 Update.

Thank you. I'll wait.

They functionality is will still launch a console. It it mainly a way to launch a particular binary without invoking it through $SHELL -c command. Useful if you hosed your /etc/passwd file or set your default shell to one that doesn't work.

@benhillis I see, thanks for the clarification.

Try this (may cause flicking window):
#include <Windows.h>

typedef HRESULT(WINAPI* LaunchIntDistro)(PCWSTR, PCWSTR, BOOL, DWORD*);

int main() {
	ShowWindow(GetConsoleWindow(), SW_HIDE);
	ULONG ExitCode;
	HMODULE dll = LoadLibraryExW(L"wslapi.dll", NULL, LOAD_LIBRARY_SEARCH_SYSTEM32);
	LaunchIntDistro func = (LaunchIntDistro)GetProcAddress(dll, "WslLaunchInteractive");
	func(L"DistroName", L"DISPLAY=:0 uxterm", TRUE, &ExitCode);
	ExitProcess(ExitCode);
}

Change DistroName with your default distribution name and compile this in Visual Studio (may cause flicking window):
#include <Windows.h>

typedef HRESULT(WINAPI* LaunchIntDistro)(PCWSTR, PCWSTR, BOOL, DWORD*);

int main() {
	ShowWindow(GetConsoleWindow(), SW_HIDE);
	ULONG ExitCode;
	HMODULE dll = LoadLibraryExW(L"wslapi.dll", NULL, LOAD_LIBRARY_SEARCH_SYSTEM32);
	LaunchIntDistro func = (LaunchIntDistro)GetProcAddress(dll, "WslLaunchInteractive");
	func(L"DistroName", L"DISPLAY=:0 uxterm", TRUE, &ExitCode);
	ExitProcess(ExitCode);
}

The former, not the later.

Got it. That makes sense. It is useful for other stuff too 😉.

@therealkenc - Exactly!

Now you just need to fix up the signature of WslLaunch. It looks like:
int execve(const char *filename, char *const argv[], char *const envp[]);

It is interactive unless the caller close()ed fd 0/1/2, naturally.

WslLaunch() and WslLaunchInteractive() functions just open wsl.exe with CreateProcess() and standard I/O handles. There is no Linux functions.


tests: add system tests for python compute

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Hey @georgiyekkert,
Please register your GItHub account at go/github to sort out the CLA. If you're planning on making a lot of contributions to this org (googleapis) you should also join the org at go/github. Kokoro (CI) won't run automatically for contributors who are outside the GitHub organization.

@googlebot I signed it!


Remove cache warning

I'm planning to remove this warning:

I don't remember what problem made me put it everywhere (I think there was one though). Comment here if you think the warning should stay.

+1
Put it just once in the top-level README.
Am So., 3. März 2024 um 12:13 Uhr schrieb Felix Cherubini <
@.***>:

I'm planning to remove this warning:
image.png (view on web)
https://github.com/felixwellen/synthetic-zariski/assets/22154668/d57bfa8a-1271-49b7-9f64-f6e02195cf2e
I don't remember what problem made me put it everywhere (I think there was
one though). Comment here if you think the warning should stay.
—
Reply to this email directly, view it on GitHub
https://github.com/felixwellen/synthetic-zariski/issues/21, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHDTQ7QMRPXLX5AMGRGGKLYWMAWHAVCNFSM6AAAAABED34EQKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3DKMZQGY3DIMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Right! :-)


Use cryptography which is free from CVE-2023-23931

Use cryptography which is free from CVE-2023-23931. See https://www.mend.io/vulnerability-database/CVE-2023-23931.

I am not going to require a specific version of cryptography because of several reasons:

cryptography is most often provided by the system libraries and I don't control that version
there are many users that use a downlevel cryptography due to its Rust dependency in later versions
this is not a really a security issue if you read the CVE description


Why was this fork created ?

It doesn't seem to be compatible with docker-machine or the libmachine driver framework ?

@cfergeau

It was meant to be used in https://github.com/crc-org/vfkit which is in turn used by a machine-driver internal to crc, but it turns out not to be useful for now, this could be removed if this causes confusion.

Just wondered, the whole machine-drivers organization is something of driftwood...
If you are not going to use it for doing any development, you could archive it perhaps.

https://github.com/Code-Hex/vz/pull/159#issuecomment-2399145172
@AkihiroSuda Seems you also share the same frustrations...

@gbraad maybe create it ("vz") in the crc-org organization instead, like "vfkit"?

https://github.com/crc-org/vfkit/issues/10#issuecomment-2282777953

we have shared interests among several projects to use this. bit all code herte is used by all

I hope you realize that the driver you referenced relies on the built binary that we maintain. the issue is that the swift bindings we use are poorly maintained; the maintainer refuses or ignores requests. this has been ongoing for quite a while and would be detrimental also for the visit driver of minikube if we can't get patches or features included, and/or worked on by multiple people.

Oh indeed, but as far as I know there are no docker-machine drivers for vz - so the machine-drivers organization is the wrong place for hosting vz. Especially since it is about to be retired? Here was the issue opened for the collaboration:

https://github.com/crc-org/vfkit/issues/187

And it (vfkit) seems to accept submissions just fine, outside of CRC and Podman Desktop :-)

Also I do think that the original maintainer (of Vz) is looking for funding, in order to start a supporting project for it.

The last remaining fork might continue into 2025, but still... The docker-machine drivers are running on life support.
https://gitlab.com/gitlab-org/ci-cd/docker-machine/-/commit/626d13ee78c1845ea22fe27756463b0c2776cb37

vfkit depends on the vz (swift bindings).

funding

money talks, but does not change attitude. Several people in that repo have raised concerns. We are looking for a reliable way to contribute, with likewise-minded people.
machine-drivers might not even be the right place, but a good place to start the discussion until we find a good home.


Additional preprocessing in the server. Find the mole and zoom in on it and increase the brightness; Add a way to detect brightness of an image

ImageCropBinaryMask.py solved this issue


Compiler Tests

This includes:

Exporting compileJS method for testing
Create test/spec/compiler dir, and reorganize the CLI test

html.js: tests for HTML and expressions (previously a part of compile-cli.js)
scoped-css.js: tests for Scoped CSS (just moved from parent dir)
riotjs.js: tests for riotjs compiling
tag.js: tests for integrated tag file compilation (as a starting point I used test/tag/*)

@GianlucaGuarini , @tipiirai My proposal is a simple fixtures - expect style testing. How about? Thanks.

It looks good to me I also prefer the expect/fixtures folders

This is so great! I love you @cognitom!

Thank you for merging! I love you guys, too!

:neckbeard: weirdos :)


tensorflow-addons fixed version range

tensorflow-addons==0.10.0 is only compatible with tensorflow==2.2.0
Setting lower and higher limit as per https://github.com/tensorflow/addons
(pytest was failing hard looking for random number generator introduced in 2.2.0)

let's bump tensorflow to 2.2.0 instead

As you prefer, I was doing this on the assumption to leave things as-is as much as possible.
Will try and bump it up tomorrow. In any case 0.10.0 supports 2.2.0 only (explicitly lower than 2.3.0) so a similar constraint remains


Build(deps-dev): Bump @testing-library/jest-dom from 5.16.1 to 5.16.2 (#2664)

Pull request opened by the merge tool on behalf of #2664

Merged in 0561938de8724b35a195c94d4e5a255e9fb4de6a


[Table] Migrate TableHead to emotion

[x] I have followed (at least) the PR section of the contributing guide.

#24405
rebase next after #24679

Can the CI be triggered again. For some reason Tooltip is timing out?

Can the CI be triggered again. For some reason Tooltip is timing out?

@natac13 Thanks


Reactions order in MessageReactionUpdated is not preserved?

1-1 chat with the bot, steps are:

put one non-premium reaction (🐳 in the example)
put second non-premium reaction (🤡 in the example)
put third premium reaction (numbers in the example)
remove premium reaction

Expected: order is preserved at all steps
Actual: it is not:

second setup, this time non-premium reactions only

put 🐳
put 👀
put 🤡 - order in old_reaction is not preserved already (swapped)
given ['🐳', '👀', '🤡'] as a current reactions set, remove the middle one (👀) - order is not preserved again

Expected: order is preserved at all steps
Actual: it is not:

Reaction order depends on the number of times the reactions were chosen by other users. Therefore there can be no any order on reactions of other users and it doesn't make sense to pretend that there is any order on reactions of the current user.

so, to summarize: in 1-1 chats there is no way to tell which reaction was set earlier/later, right?

Yes.