Sort Order and replace_more

Issue Description
It is necessary to change the value of submission.comment_sort before calling submission.comments.replace_more() sorting to be effective; however, any comments added by replace_more() are then sorted by another method (seemingly 'best').
I've confirmed this via thread '5z8dkm'.
Here's the code I used:
import praw
from praw.models import MoreComments
#import pprint
from datetime import datetime
import re

subid = '5z8dkm'

# create Reddit instance here
# insert in your own credentials
reddit = praw.Reddit()

# auto sort by old
submission = reddit.submission(id=subid)
submission.comment_sort = 'old' # default is new for this sub
submission.comments.replace_more(limit = None)
comments = list(submission.comments)

# get some basic information to compare to web browser
# also get datetime stamp for manual sorting
line_list = list()
date_list = list()
for e in comments:
	line = ''
	tab = ''

	# get top level comment information
	date = datetime.fromtimestamp(e.created)
	line1 = tab + str(e.author) + ' [' + str(e.score) + '] ' + str(datetime.fromtimestamp(e.created)) + '\n'
	line2 = tab + e.body.encode('utf-8') + '\n'
	line = line1 + line2 + '\n'

	line_list.append(line)
	date_list.append((date, line))

# write auto sorted version to file
file = open('auto_sorted_by_old.txt', 'w')
for l in line_list:
	file.write(l)
file.close()

# manually sort by old via date
date_list.sort(key = lambda tup: tup[0])
sort_list = list()
for l in date_list:
	sort_list.append(l[1])

# write manually sorted version to file
file = open('manual_sorted_by_old.txt', 'w')
for l in sort_list:
	print l
	file.write(l)
file.close()

I've also attached the files generated, but in summary both files are identical (and correctly sorted) for the original comments ending with this comment:

JadedIdealist [41] 2017-03-14 15:47:57
Thanks for hosting.
Just a note that normally on launch threads the latest information appears at the top of the updates (ie most recent first as you read down, and earlier stuff appears later).
It just makes it a little quicker and easier to grok what's happened most recently - you might want to maybe consider doing it that way round on  Thursday (assuming it's you again).
Cheers.

but after the first top-level MoreComments, the auto-sorted version is as follows:

ethan829 [20] 2017-03-14 19:56:49
It's official:

Targeting Thursday, March 16 for @EchoStar XXIII launch; window opens at 1:35am EDT and weather is 90% favorable.

steezysteve96 [18] 2017-03-15 19:44:55
Are we gonna get a new thread or reuse this one?  Reusing threads would fit the SpaceX spirit, after all
EDIT: never mind, found the update

24 hours until T-0. Weather 90% go. Falcon 9 is vertical. We'll be reusing this launch thread - see you all tomorrow!

while the manually sorted version continues to be correct:

None [2] 2017-03-14 17:47:53
[removed]
ByteStalker [4] 2017-03-14 18:51:14
Aww man I'm going to be in Orlando on the 16th but Im not going to be able to see the launch :(
ethan829 [20] 2017-03-14 19:56:49
It's official:

Targeting Thursday, March 16 for @EchoStar XXIII launch; window opens at 1:35am EDT and weather is 90% favorable.

BEAT_LA [4] 2017-03-14 23:13:26
Is this the reflown booster?

Here are some screenshots to show the correct behavior in the web browser:
Last comment when sorted by 'new' and expanded:

Is the first comment when sorted by 'old':

The last comment when sorted by 'old' before expanding is the last comment correct auto-sorted using praw:

Here are the two output files. The problem starts at line 526.
auto_sorted_by_old.txt
manual_sorted_by_old.txt
System Information
PRAW Version: 4.4.0
Python Version: Python 2.7.11 |Anaconda 4.0.0 (x86_64)
Operating System: macOS 10.12.3 16D32 (Sierra)

So I added print data right before the API call to morechildren in praw/models/reddit/more.py, and every call is sending out sort with a value of best, the default defined in submission._init_.

{'sort': 'best', 'children': u'dexd9xo,deyzpcd,dezfxqo,deyughb,deydvz5,deylh9e,dexsjvf,dexkd9g,deygqzv,dewzact,deyy0fy,dezfpzd,dezc6oa,dezf49f,dezbv4k,dezjbo7,dez4zq5,dez2u76,dexw5ew,deyodpo,dez71pr,dewwfap,dex2m6r,dez0i2d', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex08jc,dex50ri', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex75iu', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dezfq76', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dey5xsr,dexdp7n', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dewvwy0', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex18lq', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex47oh', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex88sh', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dexcwr9', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dezlrxm', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex2gx5', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dexirjc', 'link_id': 't3_5z8dkm'}

Is the sorting supposed to be done locally?

Seems like you found the cause of the bug. Want to try to fix it?
On Mar 15, 2017 7:26 PM, "Katie Daisey"<EMAIL_ADDRESS>wrote:

So I added print data right before the API call to morechildren in
praw/models/reddit/more.py, and every call is sending out sort with a
value of best, the default defined in submission.init.
{'sort': 'best', 'children': u'dexd9xo,deyzpcd,dezfxqo,
deyughb,deydvz5,deylh9e,dexsjvf,dexkd9g,deygqzv,dewzact,deyy0fy,dezfpzd,
dezc6oa,dezf49f,dezbv4k,dezjbo7,dez4zq5,dez2u76,dexw5ew,deyodpo,dez71pr,dewwfap,dex2m6r,dez0i2d',
'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex08jc,dex50ri', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex75iu', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dezfq76', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dey5xsr,dexdp7n', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dewvwy0', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex18lq', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex47oh', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex88sh', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dexcwr9', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dezlrxm', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dex2gx5', 'link_id': 't3_5z8dkm'}
{'sort': 'best', 'children': u'dexirjc', 'link_id': 't3_5z8dkm'}
Is the sorting supposed to be done locally?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/praw-dev/praw/issues/753#issuecomment-286939491, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAC75AsKxzt87BxufUQZZ1STPP5m30JOks5rmJ3XgaJpZM4Mengo
.

@Katiedaisey thanks for filing the issue, and special thanks for providing such detail. I hope it's resolved by the latest master. Could you please confirm, and close the issue if so?

@bboe All fixed.


feat: Allow configuration of the Kubernetes Secret type to be created

We need to create a TLS secret to be used with Ingress.
This PR allows the user to select which type of Kubernetes secret to be created by the operator.
For example, if you want to create a kubernetes.io/tls certificate you will have something like this:
apiVersion: onepassword.com/v1
kind: OnePasswordItem
type: kubernetes.io/tls
metadata:
 name: tls-test
spec:
  itemPath: "vaults/VAULT/items/ITEM"

And it will create something like this:
apiVersion: v1
data:
  tls.crt: REDACTED
  tls.key: REDACTED
kind: Secret
metadata:
  annotations:
    operator.1password.io/item-path: vaults/VAULT/items/ITEM
    operator.1password.io/item-version: "1"
  creationTimestamp: "2021-11-18T00:09:16Z"
  name: tls-test
  namespace: default
  resourceVersion: "641157"
  uid: a972ad1c-74e9-4ee5-b8ea-0807fa0e4de6
type: kubernetes.io/tls

I had to create a custom template for this type of secret to work, so potentially it will be needed for the other type of Kubernetes secrets.
Below is a screenshot of the template for my use case:

Fixes #38

@jillianwilson would it be possible to get someone assigned to this PR? It is getting pretty stale. 🧓🏼

onepassword operator must have such feature

@verkaufer any chance we can get someone to review this? Thanks!

@amontalban I've re-raised the issue with my colleagues - I'll make sure it gets taken care of.

Thank you for the contribution: @amontalban! I've continued working on this PR here: https://github.com/1Password/onepassword-operator/pull/87
It will be merged soon.


React Native doesn't support newest Node (12.11.1)

React Native version: 0.61
(System:
OS: Windows 10
CPU: (16) x64 Intel(R) Xeon(R) CPU E5-2630 v3 @ 2.40GHz
Memory: 10.17 GB / 31.91 GB
Binaries:
Node: 12.11.1 - C:\Program Files\nodejs\node.EXE
Yarn: 1.17.3 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.11.3 - C:\Users\SOMEONE\AppData\Roaming\npm\npm.CMD
SDKs:
Android SDK:
API Levels: 19, 21, 22, 23, 24, 27, 28, 29
Build Tools: 19.1.0, 21.1.2, 22.0.1, 23.0.1, 23.0.2, 23.0.3, 27.0.3, 28.0.3, 29.0.2
System Images: android-19 | ARM EABI v7a, android-19 | Intel x86 Atom, android-23 | Google APIs ARM EABI v7a, android-23 | Google APIs Intel x86 Atom, android-24 | Android TV Intel x86 Atom, android-24 | Android Wear ARM EABI v7a, android-24 | Android Wear Intel x86 Atom, android-24 | ARM 64 v8a, android-24 | ARM EABI v7a, android-24 | Intel x86 Atom, android-24 | Intel x86 Atom_64, android-28 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom
Android NDK: 17.1.4828580
npmPackages:
react: 16.9.0 => 16.9.0
react-native: 0.61.2 => 0.61.2)
Steps To Reproduce

Follow "Getting Started" guide to install latest Node version
Attempt to run "react-native run-android" on Windows using Emulator or physical device
Fails

Describe what you expected to happen:
Sample app works
Snack, code example, screenshot, or link to a repository:
Related issue here was closed prematurely by author: https://github.com/facebook/react-native/issues/26598

After some investigation i found the bug and already created a pull request for the Metro Bundler
https://github.com/facebook/metro/pull/464
hopefully this gets accepted soon.

Is it clearly fixed?

@gluons @NyauIf
Just update metro-react-native-babel-preset to latest 0.59.0 version and issue will be fixed.
I'm on latest NodeJS 12.18.0 and react-native 0.62.2

I believe this has been fixed and can be closed.
In addition to @oleksandr-dziuban 's report above: our project has been using Node 12 for some time, with RN v0.61.5, and we've seen no sign of problems related to the Node version.


Could Not Find USB device..

[✔ ] I have read the FAQ.
[ ✔] I have searched in existing issues.

Environment

OS: [Windows 7]
scrcpy version: [1.25]
installation method: [Windows release]
device model: Samsung Galaxy S9+
Android version: [10]

Describe the bug
Scrcpy cannot find my usb device no matter what I do. I even tried doing the one with --otg -s
`
Please paste terminal output in a code block.
D:\Downloads\SCRCPY Builds\scrcpy-win64-v1.25>scrcpy --otg -M
scrcpy 1.25 https://github.com/Genymobile/scrcpy
INFO: Killing adb daemon (if any)...
ERROR: Could not find any USB device
D:\Downloads\SCRCPY Builds\scrcpy-win64-v1.25>scrcpy --otg -s RF8K30GM3VA
scrcpy 1.25 https://github.com/Genymobile/scrcpy
INFO: Killing adb daemon (if any)...
ERROR: Could not find any USB device
Please do not post screenshots of your terminal, just post the content as text instead.

What if you disable USB debugging on your device?
See #3050.

still having the same issue even with debug mode off
output :
D:\Downloads\SCRCPY Builds\scrcpy-win64-v1.25>scrcpy --otg
scrcpy 1.25 https://github.com/Genymobile/scrcpy
INFO: Killing adb daemon (if any)...
ERROR: Could not find any USB device

Maybe some drivers are missing. I also encountered a similar problem. I solved the problem through this software, maybe it can solve your problem.https://adbdriver.com/downloads/


Can not see/install 3.0 version in Android Studio

In Android Studio, searching for this plugin only shows old version 2.9.1.
While trying to manually install it from disk gives error;

Plugin 'JS GraphQL' (version '3.0.0') is not compatible with the current version of the IDE, because it requires build 211.0 or newer but the current build is AI-203.7717.56

IDE version

Hi! API incompatibilities require creating a specific build for 2020.* versions, I'll prepare a specific build for 2020.3 a bit later.

Release a new version compatible with 2020.3, should be available to update soon.


Calling section delete delegates after object change delegates.

Currently we do section inserts/deletes and then object insert/delete/move/updates.
Some apps require that the deletes happen after the object changes.

Nice catch, but I want to fully understand this. Can you add a unit test that fails before the change, and now passes? Sorry :(


huggingface_hub.notebook_login fails on AWS Sagemaker Studio

Describe the bug
It looks like there's a compatibility issue between the version of jupyter used by AWS Sagemaker Studio, ipywidgets and/or huggingface_hub
I've installed the latest versions of transformers and datasets and ipywidgets and the output of  notebook_login wont render.
Also note in the System info - Running in notebook ?: No - but I am running in a notebook, it seems the notebook detection code doesn't recognise the AWS Sagemaker Studio notebook variant.
As a workaround I'm using the cell magic
!huggingface-cli login --token $TOKEN
Reproduction
from huggingface_hub import notebook_login
notebook_login()

Error displaying widget: model not found
Logs
No response
System info
- huggingface_hub version: 0.16.4
- Platform: Linux-4.14.314-238.539.amzn2.x86_64-x86_64-with-glibc2.31
- Python version: 3.10.8
- Running in iPython ?: No
- Running in notebook ?: No
- Running in Google Colab ?: No
- Token path ?: /root/.cache/huggingface/token
- Has saved token ?: True
- Who am I ?: david-waterworth
- Configured git credential helpers:
- FastAI: N/A
- Tensorflow: N/A
- Torch: 2.0.1
- Jinja2: 3.1.2
- Graphviz: N/A
- Pydot: N/A
- Pillow: N/A
- hf_transfer: N/A
- gradio: N/A
- tensorboard: N/A
- numpy: 1.25.1
- pydantic: N/A
- aiohttp: 3.8.5
- ENDPOINT: https://huggingface.co
- HUGGINGFACE_HUB_CACHE: /root/.cache/huggingface/hub
- HUGGINGFACE_ASSETS_CACHE: /root/.cache/huggingface/assets
- HF_TOKEN_PATH: /root/.cache/huggingface/token
- HF_HUB_OFFLINE: False
- HF_HUB_DISABLE_TELEMETRY: False
- HF_HUB_DISABLE_PROGRESS_BARS: None
- HF_HUB_DISABLE_SYMLINKS_WARNING: False
- HF_HUB_DISABLE_EXPERIMENTAL_WARNING: False
- HF_HUB_DISABLE_IMPLICIT_TOKEN: False
- HF_HUB_ENABLE_HF_TRANSFER: False

Also note the Error displaying widget: model not found occurs frequently, i.e. in place of the download progress bars when downloading model weights, datasets etc.

Hi @david-waterworth , I'm sorry you're experiencing this issue. As you said, there is most likely a compatibility issue between the installed packages. I tried it myself by just installing huggingface_hub via pip in a AWS Sagemaker Studio session and it worked just fine.
Could you run conda list | grep "jupyter\|widget" and check if maybe you have an missing/outdated package somewhere that breaks the widgets? Here is a screenshot of what I am getting on my side:

Also note in the System info - Running in notebook ?: No - but I am running in a notebook, it seems the notebook detection code doesn't recognise the AWS Sagemaker Studio notebook variant.

This is because you are running !huggingface-cli env in a notebook which under the hood call the actual command line in a terminal. To get the information from a notebook, you should use
from huggingface_hub import dump_environment_info

dump_environment_info()

I admit that this is not really well documented but also not very problematic :)

@Wauplin I'm guessing from your suggestion that you used the Data Science kernel. I switched to that and confirmed it does work fine.
But it doesn't work on the Base Python 3.0 kernel (which isn't based on conda and ships with py310 rather than py37 in Data Science kernel).
The output of the equivalent %pip list | grep "widget\|jupyter on a fresh instance is:
jupyter_client     8.2.0
jupyter_core       5.3.1

After %pip install ipywidgets
ipywidgets         8.0.7
jupyter_client     8.2.0
jupyter_core       5.3.1
jupyterlab-widgets 3.0.8
widgetsnbextension 4.0.8

I restarted the kernel then tried to run notebook_login() and it didn't render properly
What did work (with a clean kernel) was:
%pip install ipywidgets==7.7.5

The output of %pip list | grep "widget\|jupyter  is then
ipywidgets                7.7.5
jupyter_client            8.2.0
jupyter_core              5.3.1
jupyter-events            0.6.3
jupyter-lsp               2.2.0
jupyter_server            2.7.0
jupyter_server_terminals  0.4.4
jupyterlab                4.0.3
jupyterlab-pygments       0.2.2
jupyterlab_server         2.24.0
jupyterlab-widgets        1.1.4
widgetsnbextension        3.6.4

Thanks for investigating this @david-waterworth. I tried locally and for me the notebook login works when I have ipywidgets==8.0.2 installed so I expect the issue to be more tedious (don't know well AWS Sagemaker studio TBH but there has to be a difference compared to a local setup).
In any case, I'm glad that %pip install ipywidgets==7.7.5 fixes your issues. Not the ideal solution but if someone else come across this issue, at least there is a workaround. In the meantime, I'll try to investigate the real issue. I'm reluctant to pin the version of ipywidgets has it could cause problem to other users while being valid in a vast majority of cases (i.e. when not running in AWS sagemaker studio).

I can confirm that I am getting the same error on Pytorch 1.13 Python 3.9 and Pytorch 2.0.0 Python 3.10 Kernels on SM Studio. Using ipywidgets == 7.7.5 on these kernels has not solved the issue.
This is from both kernels for %pip list | grep "widget\|jupyter
ipywidgets                8.1.1                    pypi_0    pypi
jupyter_client            8.2.0              pyhd8ed1ab_0    conda-forge
jupyter_core              5.3.0           py310hff52083_0    conda-forge
jupyterlab-widgets        3.0.9                    pypi_0    pypi
widgetsnbextension        4.0.9                    pypi_0    pypi


[VTA] Fix vta rpc server, refactor launch cond to not depend on sys.argv

cc @areusch @tmoreau89 @vegaluisjose

Fixes https://github.com/apache/tvm/issues/8603


Remove unused variables and correct version string variable names

This change restores the build number to the fourth potential
REPRO_DIR location (removed accidentally in PR 258), and also
includes a number of minor changes to aid maintenance:

Removing unused JEP322_NAME variable
Removing barely-used JEP322_BASE_NAME variable
Correcting product version variables so their names more accurately
reflect their contents

Signed-off-by: Adam Farley<EMAIL_ADDRESS>

Ok, I think all the concerns have been addressed.
This can be merged after the release is done.

Does the release is done ?

Fixes: https://github.com/AdoptOpenJDK/openjdk-installer/issues/264

@douph1 - As far as Windows is concerned, unless there's a respin: yes, Windows is done for the release.
Any objections to merging this?
(Mentioned in Slack).


Invalid results with values larger than MAX_SAFE_INTEGER

Hi,
Thanks for the library, very useful.
After starting to experiment with this library, I ran into problems which seem to originate from how BigInts and Numbers are being handled. I played with GIAI-96 values, but I suspect this issue might apply to many more formats as well. Case example:
// Testing with 34144B5A1C6A94D74F834DC7 / urn:epc:tag:giai-96:0.1234567.30000000004214215

const t = tds.fromTagURI("urn:epc:tag:giai-96:0.1234567.30000000004214215")
console.log(t.toTagURI())  // 'urn:epc:tag:giai-96:0.1234567.30000000004214216'   <-- incorrect assetReference
console.log(t.getAssetReference()) //<PHONE_NUMBER>4214216    <-- incorrect assetReference
console.log(t.toHexString())  // '34144B5A1C6A94D74F834DC8'    <-- incorrect hexString

As far as I could investigate this, this seems to relate to the fact that<PHONE_NUMBER>4214215 > Number.MAX_SAFE_INTEGER. In https://github.com/sergiss/epc-tds/blob/master/epc/utils/bit-array.js#L78 a BigInt is casted to a Number, where things fail:
> var big = BigInt(30000000004214215)
undefined
> Number(big)
30000000004214216

I'm afraid I don't understand all the corners of the library and EPC values well-enough to provide a proper fix, but I believe BigInts should be used everywhere in the library when dealing with the values. Then, when outputting the values, perhaps a bigger question from dev experience point of view is whether e.g. getAssetReference() should output BigInts or strings - I guess BigInt would be more correct.
Here's a simple snippet which could be added to your test.js for validating this case with GIAI-96 values, might be useful:
function withLargeValues() {
    //<PHONE_NUMBER>4214215 > Number.MAX_SAFE_INTEGER
    const tagUri = "urn:epc:tag:giai-96:0.1234567.30000000004214215"
    const epc = tds.fromTagURI(tagUri)
    const resultTagUri = epc.toTagURI()

    if (resultTagUri !== tagUri) {
        throw Error(`Giai96, expected: ${tagUri}, got: ${resultTagUri}`)
    }
}
withLargeValues()

Hello, thank you very much for your contribution, you are correct, in that case, the decoding should be done using BigInt. I have prepared a solution for the problem that I will post between today and tomorrow. Thanks again, regards!

I have updated the library, and now your code should work correctly. If you detect any malfunctions, I would appreciate it if you could notify me. Regards. 😉

Hi, thanks for the superfast fix, appreciated!
However, I'm afraid there's still some issues here with GIAI-96:
> const t = tds.fromTagURI("urn:epc:tag:giai-96:0.1234567.30000000004214215")
undefined
> t.toTagURI()
'urn:epc:tag:giai-96:0.1234567.30000000004214215'
> t.toHexString()
'34144B5A1C6A94D74F834DC7'
> t.getAssetReference()
30000000004214216
> t.getGiai()
'123456730000000004214216'

The toTagURI() and toHexString() produce correct output now, but the getAssetReference() and getGiai() don't. If I'm seeing correctly, the fix was fully applied to GIAI-202 but in GIAI-96 these functions are still using the .getSegment() which produces incorrect results?

😅 I apologize, it was overlooked. I have created a new version (v.1.3.3) with the problem solved, thank you for your contribution.

Thanks! Those getAssetReference() and getGiai() seem to work now with GIAI-96, great.
One observation though: the GIAI-96.getAssetReference() returns a BigInt but GIAI-202.getAssetReference() returns a string now. Not sure which one was your target, but would probably be good to have identical return values in these.
On a related note, I guess these changes to the return types are likely to lead to issues if someone is using these functions and they blindly update the lib version to latest. Just stating this out loud as a heads-up, hopefully no major issues arise from these changes as a side effect.

In the case of GIAI-96.getAssetReference(), a BigInt is returned because it is the only way to represent the values stored with that length. In the case of GIAI-202.getAssetReference(), a string is returned because in this case, alpha-numeric values are allowed. In this case, the user will need to be knowledgeable about the standards and handle the values correctly. Regards!

In the case of GIAI-96.getAssetReference(), a BigInt is returned because it is the only way to represent the values stored with that length. In the case of GIAI-202.getAssetReference(), a string is returned because in this case, alpha-numeric values are allowed. In this case, the user will need to be knowledgeable about the standards and handle the values correctly. Regards!

Ah, of course, thanks for the clarification, the limits of my knowledge about the different formats were already met it seems. But thanks a lot for the quick support with this issue, cheers!


Initial relese

Arduino ESP32 software PWM.

Closing as duplicate of https://github.com/arduino/library-registry/pull/1351


Longest Palindrome Subsequence

Problem statement - Given a string s, find the longest palindromic subsequence's length in s. You may assume that the maximum length of s is 1000. :
Problem Link - https://leetcode.com/problems/longest-palindromic-subsequence/ :
Choose one Cateorgy  - Medium
question category :

Please assign this issue to me.

Please assign this issue to me.

it wasn't assigned to me, but I've done it anyway, I'll appreciate it be being merged. PR #166

Please assign this issue to me...

Assign it to me


DAs entries are not removed on deleting configuration in a project

Added RAG DA to a new project
In a new project, one new configuration(say named as Retrieval Augement Generation) gets added and all 8 DAs were part of this configuration.
Now, I simply tried to delete the configuration. As DAs were part of this configuration, so my assumption is all DAs entry will also be removed automatically on deletion of configuration. But it does not work this way. Configuration gets deleted but not all DAs.

Is this expected behavior?

This is expected behaviour at the moment. I think we can close this one.

It now works as expected, deleting stack deletes the members


Add PagerDuty source field (PD-CEF)

Fix #3079 by adding a new source field to the PagerDuty config. Maintains backward compatibility by defaulting to the client field. Added some tests to ensure behaviour works as expected now and in the future. Updated docs as well.
this is my first ever open source contribution! #hacktoberfest
Signed-off-by: Oktarian Tilney-Bassett<EMAIL_ADDRESS>

Can you sign your commits? the easiest solution is probably to squash all the commits:
git rebase -i main
git amend -s

I opened a new PR with the signing properly setup: https://github.com/prometheus/alertmanager/pull/3106 - sorry!

No problem!


Don't cache host tools failures

See original issue on GitLab
In GitLab by [Gitlab user @BenjaminSchubert] on Oct 24, 2018, 18:50
Summary
When building a project in BuildStream and a host tool is missing or failing (for example, bwrap wasn't installed before running), the error will get cached and subsequent runs (after installing bwrap) will fail in the same way.
This behavior is counter intuitive and has already bitten multiple developers.
Steps to reproduce

uninstall bubblewrap
bst build whatever.bst  # this fails
reinstall bubblewrap
bst build whatever.bst  # this fails... again

What is the current bug behavior?
Host tools failures are cached
What is the expected correct behavior?
Since they are dependent on the external environment, we shouldn't cache host tools failures.
Relevant logs and/or screenshots
Possible fixes
Other relevant information
I added a MR to reproduce the bug (!895) and can fix the bug if the issue is accepted.

BuildStream version affected: /milestone %BuildStream_v1.x

In GitLab by [Gitlab user @BenjaminSchubert] on Oct 24, 2018, 18:50
mentioned in merge request !895

In GitLab by [Gitlab user @cs-shadow] on Oct 24, 2018, 20:08
While I agree that BuildStream should not cache failures due to missing host tools, but I think there is another issue here - even if host tool failures were cached, users should have received a prompt asking what to do next (including a retry option).
So, I think we should investigate why that was happening as well since I wonder if something else is hitting the same codepath causing them to fail and not show up the user prompt.

In GitLab by [Gitlab user @BenjaminSchubert] on Oct 25, 2018, 10:39
Testing with master, BuildStream now always asks to retry, so that bug has been fixed, even though I don't know how.
The only problem that's still there is not caching the error if we get a consensus on this.

In GitLab by [Gitlab user @cs-shadow] on Oct 25, 2018, 13:38
I'm of the opinion that this should not be cached as I don't think these are build failures and doesn't make sense to share them with other people via the cache.

In GitLab by [Gitlab user @BenjaminSchubert] on Oct 25, 2018, 15:03
assigned to [Gitlab user @BenjaminSchubert]

In GitLab by [Gitlab user @BenjaminSchubert] on Nov 19, 2018, 16:29
closed via merge request !895

In GitLab by [Gitlab user @BenjaminSchubert] on Nov 19, 2018, 16:29
mentioned in commit eac7274d23f73dd790b7f74ca55db3ebe0a9dac6


Custom image?

Your library is a great example of modern iOS control, I really appreciate your effort!
Is there an easy way to use a custom images instead of stars?

Hi, thank you for your kind words. There is no ability to use an image unfortunately. The starts are drawn at runtime with a shape. One can use a different shape by using the cosmosView.settings.starPoints property. It contains an array of points, the size of the whole shape is 100 by 100.


feat: move aws/gcp detectors from opentelemetry-js repo

Which problem is this PR solving?
Fixes https://github.com/open-telemetry/opentelemetry-js/issues/1689
Short description of the changes

move aws/gcp detectors from optentelemetry-js repo
migrate @opentelemetry/resources/test/util/resource-assertions.ts to local private project @opentelemetry/test-utils.

@Flarna thanks for reviews. Updated, PTAL again :)

This also requires a companion-PR to remove the code from opentelemetry-js.

is there an easy way to make diff this vs https://github.com/open-telemetry/opentelemetry-js/pull/2392 ?

@obecny suppose we have those two repo in a root directory, and opentelemetry-js is on main branch, we can use diff
> diff --exclude package-lock.json \
  opentelemetry-js/packages/opentelemetry-resource-detector-aws/ \
  opentelemetry-js-contrib/detectors/node/opentelemetry-resource-detector-aws/


GEODE-5228: Stale status file is deleted before waiting for status re…

…sponse file
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

This fix will also fix GEODE-3506.


Reconsider Form description on stories docs page

Forms are described in the stories page at the moment, but we say people should write those with rules, so not sure whether they should be described there at all.

Closing this ticket as Forms exist in the Actions docs section (2.x and 3.x versions).


Update Post “how-often-should-a-termite-treatment-be-done”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
c12e1ba31ca56b8b0c820f4819c58052286d0db8

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/6540ab52f01e810008c69786


DictGymWrapper - obs gets an empty extra dimension

In the DictGymWrapper class, the dict_reset method, the obs tensor gets an extra empty dimension. The obs tensor is produced by from self.reset_wait().

I have a faint memory of looking at this in another repository and it being caused by the vector-environments then. Might remember wrong... it's at least a year ago.


Enable switching booleans anywhere on the line and returning to original cursor position

Great plugin! :+1:
It would be awesome to be able to switch (bools) anywhere on the line and then have the cursor return to the original position. Possibly related to #49 and #14.
I implemented a function to do this for Python. I dunno if it can be used to upgrade the plugin, but it could be useful for someone else, so maybe include it as an example in the documentation?
function! MySwitch()
    let save_pos = getpos(".")
    call search('true\|false', '', line('.'))
    execute 'Switch'
    call setpos(".", save_pos)
endfunction
nmap <Leader>s :call MySwitch()<CR>

I've used this function as a bool only toggle plug-in vim-toggle-bool

That's a good way to handle one-off patterns. The problem is the more complicated the pattern gets, it gets harder to build something compatible on top of them :D.
One thing I'll note is that I think your search only works if ignorecase is on. If you'd like to make sure other people's configs work with this function, maybe search('\ctrue\|false', ... would be better. The \c modifier makes the entire pattern after the \c ignore case. You might also want to check if the result of search is > 0, so the function doesn't activate the Switch command if there's no true or false on the line.

I implemented a function to do this for Python. I dunno if it can be used to upgrade the plugin, but it could be useful for someone else, so maybe include it as an example in the documentation?

I agree it might be useful -- could you add it to the wiki? https://github.com/AndrewRadev/switch.vim/wiki
I've linked to the wiki in the docs under "Customization", although I don't know how visible it is.

The problem seems to be if call search. The code search(...) is an expression, while call search(...) is a command. In the command-line, you need to execute commands, which is why you need :call. But the if-clause expects an expression for its condition.
You can try if search('\c\<true\>\|\<false\>', '', line('.')) > 0 and that should work. Bear in mind though, that the execute calls won't work, because they'll throw errors if there's no match on the current line. And if they do work, the first substitution replaces 1 with 0 and the second replacement replaces it back to 1 :). Try this:
function! ToggleBool()
  let save_pos = getpos(".")
  normal 0
  if search('\c\<true\>\|\<false\>', '', line('.')) > 0
    execute 'Switch'
  elseif search('\<1\>', '', line('.')) > 0
    " decrement 1 -> 0
    execute "normal! \<c-x>"
  elseif search('\<0\>', '', line('.')) > 0
    " increment 0 -> 1
    execute "normal! \<c-a>"
  endif
  call setpos(".", save_pos)
endfunction

The mappings <c-x> and <c-a> decrement or increment numbers, so they're an easy way to work locally on 0 or 1. You need execute "normal! .." with double quotes in particular -- take a look at these two articles for an explanation why:

https://learnvimscriptthehardway.stevelosh.com/chapters/29.html
https://learnvimscriptthehardway.stevelosh.com/chapters/30.html

Awesome! Merci man - it works perfectly :sunglasses: https://github.com/gerazov/vim-toggle-bool/commit/285ff87845a51bef3fbbd1de98a8a9be2ff26897

Awesome :). I'll close this issue for now, but as I said, feel free to add a link to your plugin in the wiki :+1:

I just added a page :wink:


Little problem of design !

Hello ,
How can this small line problem be corrected? And how does one give a name to the column, such as "Actions" for example?
Thanks

Test new version https://github.com/BluesatKV/jquery-tabledit.

Thanks for your reply !
I think the best it's use jquery-tabledit.js, not jquery-tabledit.min.js ?

It does not matter how you use it from markcell
From me:
jquery-tabledit.min.js is version 1.2.6
jquery-tabledit.js is version 1.2.7 development (is better than 1.2.6 and is for tester)

ok i'll try jquery-tabledit.min.js


fix sigma setting

The current example does not reset the initial sigma when the restart with a 'small' population size is performed.
This PR fixes this problem.

@c-bata Can you review this PR?


Move fatty project index controller code to service

Notion card
https://www.notion.so/saeloun/Refactor-controllers-51ad6bf864c448308db7a9c6f65241ad
Summary
View was doing DB query. Moved that to service and all required calls from controller to the service.
Preview

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactor (non-breaking and retains same functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[x] I have manually tested all workflows
[x] I have performed a self-review of my own code
[ ] I have annotated changes in the PR that are relevant to the reviewer
[ ] I have added automated tests for my code

CI Test Suite was passing before merging latest develop,
Now there is no CI, buildkite is failing.

@apoorv1316 Please test this on staging.


very very cool

very very cool

Thanks @andrewmclagan :)

Do you use it successfully?

Yes and no. :-) submitting an issue


TimeoutError: The operation was aborted due to timeout

An error occurs intermittently after upgrading to version 2.2.0.
It seems that an error occurred because of the photo below.

My node version: v20.15.0
If you downgrade to version 2.1.7, the error will not occur.
Thank you for creating the library
(It may be awkward because we used Google Translator)
2024-06-29 12:47:34 error: TimeoutError: The operation was aborted due to timeout
    at node:internal/deps/undici/undici:12502:13
    at runNextTicks (node:internal/process/task_queues:60:5)
    at process.processTimers (node:internal/timers:511:9)
    at async LavalinkNode.rawRequest (/home/ubuntu/alh/node_modules/.pnpm/lavalink-client@2.2.0_bufferutil@4.0.8_utf-8-validate@6.0.4/node_modules/lavalink-client/dist/cjs/structures/Node.js:90:25)
    at async LavalinkNode.request (/home/ubuntu/alh/node_modules/.pnpm/lavalink-client@2.2.0_bufferutil@4.0.8_utf-8-validate@6.0.4/node_modules/lavalink-client/dist/cjs/structures/Node.js:101:38)
    at async LavalinkNode.updatePlayer (/home/ubuntu/alh/node_modules/.pnpm/lavalink-client@2.2.0_bufferutil@4.0.8_utf-8-validate@6.0.4/node_modules/lavalink-client/dist/cjs/structures/Node.js:187:21)
    at async Player.play (/home/ubuntu/alh/node_modules/.pnpm/lavalink-client@2.2.0_bufferutil@4.0.8_utf-8-validate@6.0.4/node_modules/lavalink-client/dist/cjs/structures/Player.js:217:9)

Interesting, someone else also got the error, the error says undici, but i'm not using undici anymore

I did some investings, could you please do the following:
send me the result of npm ls
Temp solution-try:

clear the entire npm cache with npm cache clean --force
rm node_modules -rf
npm i

I assume it's due to some weird global overwriting cause you can see it uses pnpm? don't use pnpm
to clear pnpm store cache you can use pnpm store prune

https://nodejs.org/en/blog/vulnerability/july-2024-security-releases stated here the internal library for fetch is undici, that could be why it's a undici error
you can declare a requestTimoutMS option per node option which will add a AbortSignal.Timeout to the fetch request, so the request was aborted due to a timeout error appears either earlier or later.
The error indicates it tried to do a request and couldn't get a result after X ms.

You are right
Versions prior to 2.2.0 also resulted in intermittent 'connection timeout' errors.
So I blacklisted the "Connection timed out" error.
However, as the 'undici' version changed, the error content also changed, so the error seems to have occurred again.
“Connection timed out” errors can be caused by Internet conditions or temporary server issues.
I wasn't sure. thank you for your help :)

Just an addition:
This library does not use undici, it's just nodejs's internal fetch api, is undici. So when you use nodejs you will see fetch errors occurring as "undici" if it happens during the request.

Node Version: 20.10.0
I've add requestSignalTimeoutMS: 3000 but it's still giving the error

This means the request took longer than 3000ms

Should I put a bigger value?


Implement audit tools for devtools managed repositories based pytest

[ ] clone all managed repos in a similar manner to eco jobs we had on molecule and ansible-lint
[ ] First test to add: Check if twine check line in tox.ini contains --strict on each repo

The reason for suggesting use of pytest on this is that it should allow us to use parametrize.

FWIW some of these can be linted with a tool called nitpick.


Prevent select2 from clearing input-Field

By default, select2 seems to clear the input field every time the input loses the focus. How can I prevent this from happening?

Hi @iFlash,
This issue does not seem to conform to the issue guidelines, so it is being closed.  Perhaps it would be better suited for discussion on the forums?

I don't think so. This is a bug in my book as the behavior is not what any given user would expect.
I reposted this issue with conforming wording.

Ok, but keep in mind that any bug report must also include a minimal, verifiable, and complete demonstration of the behavior in jsbin.
At any rate, this is a duplicate and has been asked many, many times before.


nicotine-plus: init at 1.4.1

Motivation for this change
https://github.com/Nicotine-Plus/nicotine-plus
Would be nice to have it.
Also, maybe it's a good idea to pin dependencies to specific versions?
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build nicotine-plus


Change shell from bash to zsh

Since MacOS now ships with zsh I think it would make sense to change the shell used by ShellOut to match. I was just bitten by an issue where file globbing behaviours were different between the two and debugging it made no sense since it worked when run manually.
Alternatively the shell could be made configurable. Happy to make the change, just wanted to start the discussion first.

I hit this recently too. Agree it would be good to be able to assign or override the launchPath and similar used by the underlying process to better support zsh.
However, in the mean time I have just added an abstracted Process class named ZSHProcess which will ensure it uses the things I need.

Didn't know there was an open issue for this.
I submitted a PR for a quick way to add this option last month. Don't know if it will ever get merged but feel free to fork off my version if it's of any use!


SixLabors.ImageSharp.Web not found at runtime

Hi,
When I install this package via Nuget it installs the dependencies on SixLabors.ImageSharp and SixLabors.ImageSharp.Web (version 1.0.3).
See below from your nuspec file:

.. but then at runtime, I'm getting an error:

I.e., it's looking for version <IP_ADDRESS>, not the version installed by the package manager (1.0.3).
Note, if I manually update your nuspec file to look for 1.0.0 it works... but I'm not sure this is the right fix.
Let me know if I've missed something.
Thanks in advance for you help
Cheers
Dan

Actually, this seems to be an issue with 1.0.3 of SixLabors.ImargeSharp.Web. The dll version installed in that package is <IP_ADDRESS>. Can you update your package to look for 1.0.5? That seems to work

I've uploaded a new version of the package (1.0.6) to the feed that uses ImageSharp.Web 1.0.5. Let me know if it works ok now. Thanks.

Thanks @vnbaaij ! Yes, this is fixed now. Thanks so much.


VERIFY ERROR: depth=0, error=CRL has expired: CN=xxxxxxxxxx

Hi, I just got this error on Raspbian Stretch (raspberry pi) when connecting to my server.
Was connecting fine until today (been using it for 2-3 months now)
How can I fix this problem, tried running the role again to no avail
Thanks !

Cert verification fails if the CRL has expired, but because of a bug there's no way to disable this verification. https://github.com/kyl191/ansible-role-openvpn/pull/59 should fix it definitively, in the meantime you could try removing and re-adding a user. That should force the CRL to be regenerated and be valid for another month.
Perhaps this regeneration should be done on a timer to be more secure, but at least you should be able to turn the check off.

Thanks that's what I did in the end, re-created the user. Sorry for not getting back to you after I resolved it 🙄

No problem, glad you got it working.


Improvement: Add static checks to github actions

see https://github.com/telekom/HPOflow/blob/main/.github/workflows/static_checks.yml

I have added the in my eyes most important checkers (flake8, black, pydocstyle) with the version tagged v0.2.0. Will add more from your example soon.

@RKirchherr added test for mypy, isort, pylint, mdformat. Closing this now.


Pre-build Windows version GPU problem

For the pre-build windows binary for 20160229 and 20160309, when I am finetuning the inception-bn model on a new dataset. The accuracy quickly increases for the CPU mode, however, it fails to increase for the GPU mode.
I also compiled directly from the source, and the GPU version does not encounter this problem.
However, for the pre-build version, for the inception-bn model with batch size = 20, it only costs 1.35G GPU memory, yet for my compiled version (built on 20160229), it costs 1.85G GPU memory. The problem for my compiled version also lies in that the GPU memory consumption is not very stable. Although most of the time it is jumping around 1.85G, occasionally it could cost more than 2G. Is there any explanation for this?

I guess it is caused by different OS/cuda version. Other people have
encountered similar issues. Currently, the binary packages are compiled on
Windows 10, x64, with CUDA 7.5. What are your configurations?
On Mon, Mar 14, 2016 at 10:41 AM, taoari<EMAIL_ADDRESS>wrote:

For the pre-build windows binary for 20160229 and 20160309, when I am
finetuning the inception-bn model on a new dataset. The accuracy quickly
increases for the CPU mode, however, it fails to increase for the GPU mode.
I also compiled directly from the source, and the GPU version does not
encounter this problem.
However, for the pre-build version, for the inception-bn model with batch
size = 20, it only costs 1.35G GPU memory, yet for my compiled version, it
costs 1.85G GPU memory. The problem for my compiled version also lies in
that the GPU memory consumption is not very stable. Although most of the
time it is jumping around 1.85G, occasionally it could cost more than 2G.
Is there any explanation for this?
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/1628.

--
HONG Chuntao
System Research Group
Microsoft Research Asia

It is on Windows Server 2012, x64 with CUDA 7.5 driver. Maybe the system is the problem.
Will you provide the Windows Server version? I believe more people are working on the servers, rather than on their home computers. Since a GPU card is not affordable for everyone.

:-)
I thought more people are working on Windows 7/10. Let's why we switched to
Windows 10. I will consider providing both in the next release.
On Mon, Mar 14, 2016 at 11:01 AM, taoari<EMAIL_ADDRESS>wrote:

It is on Windows Server 2012, x64 with CUDA 7.5 driver. Maybe the system
is the problem.
Will you provide the Windows Server version? I believe more people are
working on the servers, rather than on their home computers. Since a GPU
card is not affordable for everyone.
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/1628#issuecomment-196117658.

--
HONG Chuntao
System Research Group
Microsoft Research Asia

Thanks!

@hjk41 Can we automate this?

let me see if i can set it up on appveyor
On Mon, Mar 14, 2016 at 3:30 PM, Eric Junyuan Xie<EMAIL_ADDRESS>wrote:

@hjk41 https://github.com/hjk41 Can we automate this?
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/1628#issuecomment-196182348.

--
HONG Chuntao
System Research Group
Microsoft Research Asia


Linter for code within unit tests and ng-docs

Implement some form linter for unit tests. Research and possibly do the same for ng-docs to avoid future clean up tasks like #620
Related PR #644

@jeff-phillips-18 & @dtaylor113 input appreciated
This may be a bit larger than simply turning eslint on for the test directory... first run throws back 718 "kinda, but not really problems" such as...
 ...
  315:5   error  All "var" declarations must be at the top of the function scope  vars-on-top

✖ 718 problems (718 errors, 0 warnings)

With the majority of the issues being ```"var" declarations must be at the top````. Can do a couple of things:

continue going in and correcting this manually

then maybe taking a look at some automated form of "beautify" or "prettify"

or/and look at relaxing the eslint config, mostly in the errors to warnings area

I would be ok with relaxing the All "var" declarations must be at the top of the function scope rule

Me too


[BUG] Imath3.1.3 CMAKE_INSTALL_PREFIX Override Not Respected in Solution Build with MSVC

Describe the bug:
When setting the CMAKE_INSTALL_PREFIX variable to a custom directory path (or keeping the default setting C:/Program Files/Imath), the solution builds within the specified build directory (bin, python3_10, and PyIMath 'src' in my case are inserted into the 'cmake build' directory and not my separate custom 'dist' directory).
NOTE: I have confirmed that the generated CMakeCache.txt file does indeed store the correct value I've set. The SLN shows though that this points back to the build directory though.
To Reproduce:

Download Imath3.1.3 source.

Configure with cmake (CLI or GUI), modify the CMAKE_INSTALL_PREFIX variable to another directory other than the 'Build' directory. In my case I have something like this:
-build  <- This contains the SLN and CMakeCache.txt
-dist    <- This is the desired install directory once the SLN is built
-src     <- This contains the Imath3.1.3 source files

In MSVC for building, checking the projects 'Properties/General' for 'Output Directory' points to the original SLN build directory instead.

Expected behavior:
Imath installs into the directory set by the CMAKE_INSTALL_PREFIX.
Platform information:
Imath3.1.3
CMake 3.22.1
Windows 10 x64
MSVC (14.2) 16 2019 Community

Apologies on this issue report - I realized that the MSVC configuration was trying to build outside the INSTALL project and thus was bypassing the CMAKE configuration specified.


Use server2server (two-legged) Auth to connect to SubmissionHistory API

User Story:
As an automated sender, I want to use two-legged auth so I can connect to the SubmissionHistory API
Description/Use Case
As an automated sender, I need to use two-legged auth so I can connect to the SubmissionHistory API.  Currently the SubmissionHistory API Azure functions do authorization by requiring a human to log in.   We need to also allow connectivity to the History API functions via two-legged auth, so that automated senders can connect.
Risks/Impacts/Considerations

If we don't do this, automated users will have to do a manual login prior to pulling data from the API.

Dev Notes:

Detailed notes on how to use two-legged auth are here:  https://github.com/CDCgov/prime-reportstream/blob/master/prime-router/docs/playbooks/how-to-use-token-auth.md
Detailed notes on how we implemented two-legged auth are here:  https://github.com/CDCgov/prime-reportstream/blob/master/prime-router/docs/proposals/0001-authentication.md
We have already implemented two-legged auth (aka "FHIR AUTH") for the waters submission endpoint, so this is a case of mimicking that code.   Look for the code in
function fun report aka @FunctionName("waters") in ReportFunction.kt

These functions need to accept two legged auth:

getOrgSubmissions
getReportHistory
Both are in SubmissionFunction.kt

The authorization to do an activity is determined by the "Scope" value.   I think this can use the same scope as the existing waters submission endpoint. I believe we do not need a new "scope" for this work.  That is, if you have the right to submit data, you equally have the right to see its lineage/history data.
Acceptance Criteria

An automated user can follow the two-legged auth onboarding procedure as outlined in the playbook, and is able to use the two Submission History endpoints listed above.
Okta auth continues to work without issues

Further work is in #5290.


add a new example -- read_sexp.rs

I made this PR to thank you for helping me on this issue :: https://github.com/murarth/ketos/issues/39
Hopefully I will keep PR-ing about docs and examples from a new users point view.

I do believe there should be an example to demonstrate that users can override the functionality of the Enter key command, but I feel that implementing Lisp-style expression parsing adds a lot of noise that obscures the linefeed API usage involved.
So, I'm rejecting this PR, but I've added a new example, called enter_function, that shows the API functionality with minimal noise.


chore(build): add license & remove litestar from dependencies

Description
There have been a number of complaints about circular dependencies with this package.  This change make litestar only required for tests.
Litestar will be updated to have a minimum pin of 0.4.0 here.

Closes
Summary by Sourcery
Remove 'litestar' from the project dependencies, addressing circular dependency issues, and update the project version to '0.4.0'. Update the 'ruff-pre-commit' hook version in the pre-commit configuration.
Build:

Remove 'litestar' from the project dependencies in 'pyproject.toml', making it only required for tests.

Chores:

Update the version of the 'ruff-pre-commit' hook from 'v0.7.0' to 'v0.8.1' in the pre-commit configuration.

Closes #7


Conflict with CIT Resewn infinite looping reloading

so when i install CIT Resewn and cpm this is making my game infinite looping reloading the resource pack manager
mod: https://www.curseforge.com/minecraft/mc-mods/cit-resewn

I'm also getting this glitch


Not using navigationcontroller

New Issue Checklist

I have read the guidelines for contributing and I understand:

[ x] My issue is happening in the latest version of SideMenu.
[ x] My issue was not solved in the README.
[ x] My issue can not be answered on stackoverflow.com.
[ ] My issue is not a request for new functionality that I am unwilling to build and contribute with a pull request.
[ ] My issue is reproducible in the demo project.

Issue Description

Awesome Library, I want to ask if it is possible to use this library without using Navigation controller for the MainViewController. In my application, I do not want to use a Navigation controller. is this possible and how do I go about it?

Use a navigation controller. Turn off the navigation. It looks the same.


JsonResponse and TextResponse is removed in 0.7.0?

error TS2305: Module '"./node_modules/routing-controllers/index"' has no exported member 'JsonResponse'.
error TS2305: Module '"./node_modules/routing-controllers/index"' has no exported member 'TextResponse'.

This is not documented in your changelog. What is it's replacement?
I'm using this feature for a controller which should return JSON in some endpoints, but plain text or Handlebars rendered HTML in others. I do not want to split up the controller into a JsonController and a regular Controller, as all endpoints fit logically under the same url path for my use.

Yeah they are removed, sorry forgot to mention that in changelog. JsonResponse and TextResponse bring lot of issues and confusion into the framework thats why it was decided to remove them. Unfortunately you have to split your controllers. But its not that bad...

Hmm... Wonder if it's perhaps easier to specify content-type manually and return a JSON.stringify on each endpoint required to return JSON. Trying that first.


chore: Add m365 PnP CLI reference

Workaround for MicrosoftDocs/office-docs-powershell#1709 MicrosoftDocs/office-docs-powershell#10271 where it does not work on macOS and Azure Cloud Shell

@StephenRiceMSFT @MikePlumleyMSFT
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@StephenRiceMSFT can you review?

Seems fine to me.

From: Mike Plumley @.>
Sent: Monday, October 2, 2023 3:54 AM
To: MicrosoftDocs/OfficeDocs-SharePoint @.>
Cc: Stephen Rice @.>; Mention @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SharePoint] chore: Add m365 PnP CLI reference (PR #4056)
@StephenRiceMSFThttps://github.com/StephenRiceMSFT can you review?
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-SharePoint/pull/4056#issuecomment-1742801233, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALOEECBMHWQ4L4RLEYJLAE3X5KMPRANCNFSM6AAAAAA5I4QGWE.
You are receiving this because you were mentioned.Message ID: @.***>

Hi - any updates on this?


sync: map.Range has started allocating as of Go 1.20

What version of Go are you using (go version)?

$ go version
go version go1.20.4 linux/amd64

Does this issue reproduce with the latest release?

The latest versions of go runtime added a 16Byte allocation to sync map Range function. This was not the case in prior releases.
I have tested go 1.19.5 (which does not do the allocation) and go 1.20.4 and go 1.21.0 (both of which do the allocation).
package smap

import (
	"sync"
	"testing"
)

var myMap sync.Map

func BenchmarkRange(b *testing.B) {
	b.ReportAllocs()
	b.ResetTimer()

	for i := 0; i < b.N; i++ {
		myMap.Range(func(k interface{}, v interface{}) bool {
			return true
		})

	}
}

func init() {
	for ind := 0; ind < 100; ind++ {
		myMap.Store(ind, (ind*2 + 1))
	}
}

What did you expect to see?
[murthy@testbuild] 12:35:32 $ /usr/local/go1.19.5/bin/go test -bench Range
goos: linux
goarch: amd64
pkg: smap/go1195
cpu: AMD EPYC 7352 24-Core Processor
BenchmarkRange-16    	  840378	      1418 ns/op	       0 B/op	       0 allocs/op
PASS
ok  	smap/go1195	2.041s
[murthy@testbuild] 12:35:39 $

What did you see instead?
[murthy@testbuild] 12:36:03 $ go version
go version go1.20.4 linux/amd64
[murthy@testbuild] 12:36:05 $ go test -bench Range
goos: linux
goarch: amd64
pkg: smap/go1204
cpu: AMD EPYC 7352 24-Core Processor
BenchmarkRange-16    	  782552	      1512 ns/op	      16 B/op	       1 allocs/op
PASS
ok  	smap/go1204	2.029s
[murthy@testbuild] 12:36:12 $

CC @bcmills

Seems like this allocation comes from the change from atomic.Value to atomic.Pointer in this CL https://go-review.googlesource.com/c/go/+/426074

CC @dsnet

Specifically, what is allocating memory is the empty value for the return:
func (m *Map) loadReadOnly() readOnly {
	if p := m.read.Load(); p != nil {
		return *p
	}
	return readOnly{}
}

Ah, probably escape analysis. What we need here is some manual memory management. 🙃
m.read.Store(&read)

should be changed to instead make a copy of the variable, and store a pointer to that copy so that it only escapes in that one code path. (I guess the compiler doesn't do live-range splitting for local variables.)


Add tutorial how to create icons and contribute

Would like to create icons myself, just don't know how to get started

In the meantime, I managed to create clean .svg icon files and to fork the main repo into Android Studio.
How can I now create the .xml file with the  property? Is this a manual process or can I generate it automatically somehow?
Any hint is appreciated :)
(Is my assumption correct that it suffice to submit the three changes:

app\src\main\res<icon>.xml
svgs<icon>.svg
add a line in app\src\main\res\xml\grayscale_icon_map.xml ?)

Look at the CONTRIBUTING.md file


docs: Clarify "python" vs "python3"

It looks like PPCI requires Python3, which is great. It looks like "python -m ppci" syntax is used somewhere in the docs (as suggested in https://github.com/windelbouwman/ppci-mirror/issues/11), which is also great: https://ppci.readthedocs.io/en/latest/reference/lang/java.html#compile-java-ahead-of-time
However, that uses exactly python, and for most users nowadays that will lead to stack trace ending with:
AssertionError: Needs to be run in python version 3.x

Not exactly user friendly (especially for people not familiar with Python, and #11 suggests to keep those in mind).
So, suggested to use python3 consistently in the commands everywhere.

@windelbouwman: Do you agree that this makes sense, should I make such a change if see a bare "python" command usage?

Hmm, I don't know, since python2 is officially dead (for some hours ;)), we might assume the default is python3 now? I propose to leave the docs as it is, and improve the error message, which I agree is poor, and should be more explaining.

I added a more explanatory error message 726dea002d8d54afcf2aa966a130e55fbaac1294 do you mean something like this?

Hmm, I don't know, since python2 is officially dead (for some hours ;)), we might assume the default is python3 now?

I personally don't think we can. While "official" deadline has been passed, my system still has "python" pointing to "python2". And it's the same for millions of other users, for months to come, and for sizable numbers still, for years to come. And I expect the echo of confusion to roll around for the next 10-20 years. While "python3" is already there, will be there, and even was there all the time.

I propose to leave the docs as it is, and improve the error message, which I agree is poor, and should be more explaining.

Ok, but then I'd suggest to keep this ticket open, and revisit again in some time.

I added a more explanatory error message 726dea0 do you mean something like this?

Yes, that's detailed enough, thanks.


Update default runtime-images to latest versions

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

@akchinSTC - is curl our only prerequisite for images?   I'm assuming you checked the others as well (thanks).

@kevin-bates yes just curl and python3. I checked the other images

Looks good. Thanks.
It might make sense to have script or something that validates images - ensuring they have the various commands. Something like:
docker run -it --rm <image> which curl
or mount a directory that contains a general validation script to run.

this would be a good test to run on all image_name under etc/config/metadata/runtime-images/*.json


Knitro interface

Artelys Knitro interface for linear problems
Adding:

ortools/knitro directory with 'environnment' and CMakeLists.txt files
ortools/linear_solver/knitro_interface*.cc files (interface and test files)

Modifying:

ortools/linear_solver/CMakeLists.txt file

Pipelines:

Linux: OK
Windows: needs fix
Other: not developped so far

what is the relation with the other PR ?
Can you rebase somehow ? 1350 files changed is unrewierable ?

To answer your question:

Yes it is the same after a squash of our commits (I have deleted the other one)
Yes sure! PR into the 'main' branch is OK for you?

Yes.
Le jeu. 24 oct. 2024, 16:04, yojvr @.***> a écrit :

To answer your questions:

Yes it is the same after a squash of our commits (I have deleted
the other one)
Yes sure! PR into the 'main' branch is OK for you?

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/pull/4416#issuecomment-2435389996,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3JGSCQUEY7B3F57H7TZ5D47DAVCNFSM6AAAAABQJNUAFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZVGM4DSOJZGY
.
You are receiving this because you commented.Message ID:
@.***>


[FLINK-15298][docs] Fix dependences in the DataStream API Tutorial doc

What is the purpose of the change
This pull request fixes the maven dependence issues in the DataStream API tutorial doc where:

The scope should be 'provided' for flink-java and flink-streaming-java
The dependency flink-client is not needed

Brief change log

Added provided
Removed the flink-client dependency

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

sorry for not re-reviewing it earlier - since 1.9 is a bit too old now, let me close this PR


Add episode shortcode URLs

Adds short URLs which end up being rendered as the full SEOed URLs. Will make it easier to directly link to episodes.
For example:
https://seetee.io/podcast/S1E1
will render as:
https://seetee.io/podcast/S1E1/gigi-introduction-to-closing-the-loop
I also added the guest name to the long URL.
Background
We're not redirecting (that's not possible for a statically built site). Instead we're dynamically replacing the short URLs with the long ones without re-rendering the page. You'll see what I mean when trying it out in the deploy preview (notice that the site doesn't reload when opening a "short URL" but the URL changes anyway). Not sure if this is the best thing for SEO but I'd say it's ok. Eventually we might want to switch to a server-rendered deployment where we can do normal redirects (should be an easy task with NextJS). But for now I'd say we're fine.

Fixed conflict 👍


Update region list and moved to helper

Nobody will suspect a thing
#218

All according to keikaku.
TL Note: keikaku means plan
I now need to go find a new minor change to make sure I get the first commit in the dev branch. I think we are missing the config sample. Hehe :)

thank god admin has his priorities straight :p

:+1:

Do you want me to make the same PR into the dev branch as well?

Already done

iOS capitalised it without me catching it though. I'll bring back the config sample tomorrow when I have a keyboard.

are you reverting the capitalisation as well? Just so I don't need to re-link my local copies to the correct upstream :p

UNSUBSCRIBE

Fine... Fixed. Pretty sure you still need to delete your local branches and re-pull origin (upstream).

nah, just need to do a pull and delete the Dev remote branch

How to invite other people to join the game?

Open a new issue @mjxtw1989
The answer is to have a look at the readme. It's very explicitly explained there.


Reuse cf-xarray for CF conventions

cf-xarray (https://github.com/xarray-contrib/cf-xarray) provides a number of convenience methods to fetch (multiple) variables with specified roles.
Currently, the ugrid_io does this -- haphazardly, as it cannot deal with multiple topologies.
It may be possible to re-use a lot of cf-xarray's machinery, but let it search for other (ugrid) roles?

Done in 9563e83eeb93346bdae4a3ef9441a4ab3c9ebb4e
Couldn't use cf-xarray directly: the UGRID conventions are fundamentally nested through the topology dummy variable attrs, while cf-xarray mostly looks at the variable directly. Part of the setup, including an xarray accessor work fine, however.


Support import of CSS in SCSS

Nativescript 3.4.0
try to import a theme @import 'nativescript-theme-core/css/core.dark.css';
Actual: The theme is not going to be applied
The generated code will be:
@import url(/Users/vhristov/Downloads/NSPlayground/app/nativescript-theme-core/css/core.dark.css);

quotes are missing
it seems that the full path is not working

is there any workaround ?
using @import '~nativescript-theme-core/css/dark' gives below error
Error: File to import not found or unreadable: E:\nativescript\HelloWorld
node_modules\nativescript-theme-core\css\dark.
Parent style sheet: E:/nativescript/HelloWorld/app/app.scss
on line 1 of app/app.scss
any workaround available?

this fails with the file not found error :
@import '~nativescript-theme-core/scss/dark';
this works but it is useless from within a scss file :
@import '~nativescript-theme-core/css/dark.css';
I am only working in scss so i need the first one to work.

@bhavincb @T0T4R4 The sample demo in the plugin's repo imports the light theme without any issues using your approach. Can you provide a demo that reproduces the issues you experience?
Notice that v1.5.0 of the plugin has been released.

hii @zbranzov ,
mine problem is solved by installing node-sass globally.


Fixes issue #12

Issue that this pull request solves
Closes: #12
Proposed changes
Description of what is fixed or changed

Header.js

Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Documentation update (Documentation content changed).
[ ] Other (please describe).

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project.
[x] I have made corresponding changes to the documentation.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface.

responsive header center

@piyush97 It doesn't fix the problem. I can still not see navbar items centered!

@piyush97 It doesn't fix the problem. I can still not see navbar items centered!

@imskr check now please :)


Setup a real documentation

The README is big enough, it's time to think about writing a real documentation (I'm for leaving a few exemple in the README, though, juste to give a taste of the package).
I've already some experience using Read The Docs, but if you have other suggestions, your input is welcome :)

Would prefer RTD over anything else :+1:

RTD it will be then ;)

Okay, a very basic documentation has been pushed to RTD: http://lifter.readthedocs.org
Contributions are welcome :)


[WebPubSub] Increase isolated package version and fix a few issues

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

@jsquire Can you help merge the PR?

API change check
API changes are not detected in this pull request.


Issue with .create

Hi, first let me tell you great package, congrats to the authors.
I have read your docs multiple times but I can't find enough examples about the join setup.
I am not sure how you set up the services postService and commentService.
I am trying to create a child-parent join structure and then I am trying to add items just like your example:
I have a mainservice and I can make a find query but sadly I can't make a create one.
const mainService = service({
    Model: new elasticsearch.Client({
        host: 'localhost:9200',
        apiVersion: '6.0'
    }),
    paginate: {
        default: 10,
        max: 50
    },
    elasticsearch: {
        index: 'find',
    },
    esVersion: '6.0'
    });
   mainService
        .find({
            query: {
                $all: true
            }
        }) //this works fine
Result:

{"total":2,"skip":0,"limit":10,"data":[{"_meta":{"_index":"find","_type":"_doc","_id":"c9552b9b-1152-4d3b-b6f6-b9a0eb22f285","_score":1},"type":"item","uuid":"c9552b9b-1152-4d3b-b6f6-b9a0eb22f285","title":"Test title","softTitle":"Test title","snippet":"Snippet string","thumbnailURL":"http://image.com/1","content":"Content String","author":"Jon Doe","siteName":"http://example.com","description":"Description string","find_join_field":"item","_id":"c9552b9b-1152-4d3b-b6f6-b9a0eb22f285"},{"_meta":{"_index":"find","_type":"_doc","_id":"c9552b9b-1152-4d3b-b6f6-b9a0eb22f296","_score":1,"_routing":"1"},"type":"map","uuid":"c9552b9b-1152-4d3b-b6f6-b9a0eb22f296","name":"Map name","find_join_field":{"name":"map","parent":"c9552b9b-1152-4d3b-b6f6-b9a0eb22f285"},"_id":"c9552b9b-1152-4d3b-b6f6-b9a0eb22f296"}]}
const itemService = service({
        Model: new elasticsearch.Client({
            host: 'localhost:9200'
        }),
        paginate: {
            default: 10,
            max: 50
        },
        elasticsearch: {
            index: 'find',
            type: '_doc'
        },
        esVersion: '6.4',
        parent: 'parent',
        routing: 'routing',
        join: 'mapitem',
        meta: 'meta'
        })
itemService..create({
            "_id": "222",
                "type":"item",
                "uuid":"222",
                "title":"Test title",
                "softTitle":"Test title",
                "snippet":"Snippet string",
                "thumbnailURL":"http://image.com/1",
                "content": "Content String",
                "author" :"Jon Doe",
                "siteName":"http://example.com",
                "description":"Description string",
                "mapitem":"item"
        })
// this gives the following error

Cannot read property 'query' of undefined     at createCore (/home/bogi/Work/Find/findES/node_modules/feathers-elasticsearch/lib/core/create/core.js:17:16)
What am I missing from the itemService declaration to make it work with joins?
Thank you for your help in advance.

Hey @boboci9,
Thanks for using the package! I've gone through your code example, but the definition of the itemService seems to be missing. Could you provide a complete example, preferable along with an excerpt from Elasticsearch mapping type?
Thanks,
Jacek

Hi @jciolek,
Not sure what is the reason for the error but I managed to fix it with a second empty object parameter, so id I do itemService..create(doc, {})  i don't have the error anymore.

I am trying with the following parameters:
parent = {
  "_id": "11",
  name: "I am the parent"
 };
  itemService.create(
        parent,{}
    );
parentId = "11";
doc = {
        name:"Insert test child",
        _id: id,
        _parent: parentId,
        routing: parentId,
        aka: 'mapitem',
        meta: { _id: parentId
        }
      };
    itemService.create(
        doc,{}
    );

The issue is both items are inserted as parents, the child link is not created. Can you tell me how to set up the service to insert a child?

I edited, with this final solution I managed to create a child item.


ENH: Add pl_limit_batches to TrainerConfigs.

Enable pytorch lightning flag "fast_dev_run" in our pipeline to be able to limit the number of train/test/val batches at once for fast debugging.

Great spot that we can use this parameter to simplify our command line args. Since we aren't removing the component params (e.g. --pl_limi_train_batches etc, I wonder what is the behaviour if --pl_limit_train_batches (e.g.) and --fast_dev_run are specified with different values? Do we want to keep all 4 of these params? If so, can you add some tests to determine which parameter overrides the other? (In fact adding tests for this new param is a good idea in general!)


Running gotty top gives a blank screen in browser

Doing gotty top gives me a black screen with no text on browser

And logs state : Failed to parse init message unexpected end of JSON input

Go version is 1.5.1 on linux/amd64 and ubuntu 12.04

Could you provide me your version and/or commit id of gotty?

gotty version 0.0.10

Could you try the v0.0.11? It has some bug fixes and they might fix your problem.

Yeah the update fixed it. Thanks :)


Better End music doesn't play when Music Control is installed

When Music Control and Better End are installed and an end biome has custom music, music control plays music and is cut off. The end biome music never plays
I'm not sure if this is a bug with Music Control or Better End, since the logs mention nothing about the issue
Here's a video of the issue
https://user-images.githubusercontent.com/39657565/147793141-a57f6226-5042-46dc-91d8-889a7753fbc9.mp4
Music control scrolls through music without playing it

Looks like this is Music Control issue


Fixed image_tag call parameters, introduced with HAML conversion.

This was introduced with https://github.com/ManageIQ/manageiq/pull/1594
@dclarizio please review/test

@roliveri can you test/verify the fix.

Tested it. Looks like it fixed the problem.


Sporadic server errors on API calls

In the past I've been told by Pantheon support to post issues like this here.  Let me know if that's changed.
terminus site backups create --site=new-site05 --env=test --element=all
[2015-12-16 19:29:04] [error] Server error: 502

Attempt 5:
Wed Dec 16 11:28:53 PST 2015
terminus sites aliases
[2015-12-16 19:29:53] [error] Server error: `GET https://dashboard.pantheon.io/api/users/a5814357-ecde-46aa-af6e-82336848c660/drush_aliases` resulted in a `504 Gateway Time-out` response:

<title>504 Gateway Time-out</title>

504 Gateway Time-out
< (truncated...)

Attempt 9:
Wed Dec 16 11:32:53 PST 2015
terminus sites aliases
[2015-12-16 19:33:54] [error] Server error: `GET https://dashboard.pantheon.io/api/users/a5814357-ecde-46aa-af6e-82336848c660/drush_aliases` resulted in a `504 Gateway Time-out` response:

<title>504 Gateway Time-out</title>

504 Gateway Time-out
< (truncated...)

Attempt 10:
Wed Dec 16 11:33:55 PST 2015
terminus sites aliases
[2015-12-16 19:34:56] [error] Server error: `GET https://dashboard.pantheon.io/api/users/a5814357-ecde-46aa-af6e-82336848c660/drush_aliases` resulted in a `504 Gateway Time-out` response:

<title>504 Gateway Time-out</title>

504 Gateway Time-out
< (truncated...)

In the past I've been told by Pantheon support to post issues like this here.

Sorry your being told by Pantheon Support to open an issue here as you're seeing Server error rather than a Terminus client error. These issues should be escalated through Support rather than opened as issues here.
Sorry again for the support runaround; I'll make sure that the internal Pantheon support process is better defined.

So noted. Apologies.


Aether/regular farmland cannot get wet

Self-explanatory, I think i've narrowed the issue down to the weird cloud in a bottle liquid, it does wet soil, so something broke and regular water doesn't count as water

was tag issue


Updating dependencies

general cleanup and updates of existing dependencies.
It's probably worth reviewing whether some of these are still dependencies...

merged with latest version. i removed the @hapi/joi dependency since it was just used to validate the config, and I think that can be checked relatively easily without the dependency.


undefined method `increment' for []:Cequel::Record::RecordSet

I have a counter table, given below.
`require 'cequel'
module ASD
class postviews
include Cequel::Record
belongs_to :blog, class_name: 'ASD::Blog'

key :postid, :text
key :anotherPrimaryKey, :text

column :views, :counter

end
end`
I just want to increment the value of views atomically, so that in case I face really heavy traffic I do not miss out on increments.
After going through the Metal Class, I realise it functions over DataSet, whereas a .which on Cequel::Record will return a Record Set.
After going through the example, I see it works over models created by the Metal class.
How does it not interact with each other? And what is the way around this?
Thanks :)

Counter tables are really not supported by Cequel at this time. Personally, I usually generate raw CQL statements by hand and then execute them directly via Cequel::Record.connection.client.execute_async(my_counter_batch) where i need counters.
I'd love to have proper counter table support. If you have an idea about how to integrate such a thing into the API please submit a PR.

So I found a way around it, its good to go until Cequel officially supports Counter Tables.
args = {postid: "abc", anotherPrimaryKey: "pqr"}
options = {views:1}
ASD::postviews.where(args).data_set.increment(options)

where returns a record set object
data_set returns a data set (Cequel::Metal::DataSet) underlying this record set
increment acts on the data set above and increments/decrements the counter columns in options


IDAKLU solver sensitivities using IDAS

Summary
Extend the IDAKLU solver to return sensitivities of y wrt the parameters p
Motivation
It is useful to calculate sensitivities in, for example, model fitting to data. The IDAKLU solver already using IDA in sundials, and IDAS is another part of the sundials package which can calculate sensitivities, so in theory this should be easily achievable.
Additional context
Work on incorporating sensitivities for the casadi solver is already underway in #1100

@tinosulzer: were you planning that the API for sensitivities you have implemented in #1100 is just for the casadi solver, or should I also follow this for the idaklu solver? I'm worried that this API seems rather casadi specific, what are your thoughts on this?

Ideally the API should just be something like
sol = sim.solve(t_eval, inputs={"p": 2, "q": 3}
sol["variable"].data # returns the values
sol["variable"].sensitivity["q"] # returns the sensitivities wrt q

which then works with any solver. This involves getting rid of the ProcessedSymbolicVariable class which is casadi-specific

Should we put a flag or something going into sim.solve which indicates if sensitivities are required?

Should we put a flag or something going into sim.solve which indicates if sensitivities are required?

IIRC in the 1100 branch I made it a flag in the Solver class (so you can decide between explicitly defining the SDAEs or letting the solver do its own fancy forward sensitivity / adjoint stuff)

is there any reason that the inputs arg to solve has keys of strings, rather than InputParameters? I feel that something like this would be more natural:
a = pybamm.InputParameter("a")
# ...
sol = solver.solve(model, t_eval, inputs={a: 1.0})

It also helps when I'm calculating the sensititivites of rhs and algebraic wrt the input parameters using Jacobian.jac, as I don't have to re-create the InputParameter. @tinosulzer : what do you think?

also need to add flag to solve function to say which sensitivities will be required:
# calculate sensitivities for "a" input
sol = solver.solve(model, t_eval, inputs={"a": a_value}, calculate_sensitivites=["a"])
# calculate sensitivities for all input parameters
sol = solver.solve(model, t_eval, inputs={"a": a_value}, calculate_sensitivites=True)

I've got a draft PR for this, a couple of issues remaining:

at the moment you give a list of variables that you want to calculate sensitivities for in the solve method of the sover. But the set_up function uses this information to generate the correct equations, so if you change this list for a second solve then it isn't going to work. Perhaps the calculate_sensitivities should to in the solver __init__?
need to add integration tests to ensure that sensitivity calculations work for all models ok, and all the relevent parameters that we want to use (e.g. are there both scalar and vector parameters in the pybamm models)


bug/group-click

Summary
Clicking the result icon (or empty circle) in the navigation tree now also uses the selection handler.

Codecov Report
Base: 73.70% // Head: 94.21% // Increases project coverage by +20.51% :tada:

Coverage data is based on head (f70208b) compared to base (5c30f5a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #308       +/-   ##
===========================================
+ Coverage   73.70%   94.21%   +20.51%
===========================================
  Files         191      115       -76
  Lines        9299     3078     -6221
  Branches      771      357      -414
===========================================
- Hits         6854     2900     -3954
+ Misses       2037      178     -1859
+ Partials      408        0      -408

Flag
Coverage Δ

backend
94.21% <ø> (ø)

frontend
?

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

Impacted Files
Coverage Δ

client/src/components/_common/TreeItem.tsx

...ite/TestSuiteDetails/TestListItem/MessagesList.tsx

client/src/store/testSession.ts

client/src/components/InputsModal/styles.tsx

...src/components/_common/__mocked_data__/mockData.ts

client/src/components/TestSuite/TestSession.tsx

...ponents/TestSuite/ConfigMessagesDetails/styles.tsx

...c/components/SuiteOptionsPage/SuiteOptionsPage.tsx

client/src/components/SuiteOptionsPage/index.ts

client/src/components/Footer/Footer.tsx

... and 66 more

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

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


Remove deprecated qiskit.IBMQ alias object

Summary
This commit removes the depercated qiskit.IBMQ alias object. It was marked as deprecated in 0.23.0 and now that the qiskit-ibmq-provider package has been retired and is no longer supported it truly has no more meaning. Realistically it probably should have been removed as part of 0.25.0 which corresponded with the retirement of the qiskit-ibmq-provider library, but this was forgotten as part of the 0.25.0 release so we'll have to do it as part of 0.45.0.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
5
90.89%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
73071

Relevant Lines:
85039

💛  - Coveralls

So looking at the list of what Luciano highlighted, I think the path forward here is a bit more expansive than I realized at first. There are chunks of qiskit in dark corners that still have a hard dependency on qiskit-ibmq-provider. Mainly the backend overview, most of the juypter widgets, and some of the algorithms/quantum instance helpers. I'm going to defer this from 0.45 and then we can just remove all of those en masse early in the next release cycle.

This is superseded by: https://github.com/Qiskit/qiskit/pull/11513


Minor notebook cleanups to make TPI nb more consistent with OECM nb

Minor notebook cleanups to make TPI notebook more consistent with OECM notebook.
Also commit changed to XLSX benchmark data (which is not formally used by ITR project, though it could be).

@ModeSevenIndustrialSolutions can you take a look at the failing black-nb and pydocstyle problems?


chore(main): release 0.2.0

:robot: I have created a release beep boop
0.2.0 (2022-12-05)
Features

add option to keep gaps in nucleotide sequence during aminoacid translation (3c2daa8)

Documentation

update function docstrings (fea7643)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/RIVM-bioinformatics/AminoExtract/releases/tag/v0.2.0 :sunflower:


🔮 V5: Milestones, Ideas, Suggestions, ...

Pull request and beta versions: https://github.com/react-spring/react-three-fiber/pull/423

[ ] It would be nice if r3f could also be used without react-dom (saves ~30kb)

import React from 'react'
import { render } from 'react-three-fiber'

render(<mesh />, document.getElementById('canvas'), { shadowMap: true, ... })

[ ] Rewrite reconciler, make it type-safe, optimize hot paths
[ ] Inline react-reconciler (again, faster hot paths) via closure-compiler
[x] Remove bloat (<Dom />, { __$ } = useLoader, sRGB
[x] colorManagement=true by default
[x] auto-attach geometries and materials

<mesh>
  <planeBufferGeometry />
  <meshBasicMaterial />
</mesh>

[x] https://github.com/react-spring/react-three-fiber/issues/429 viewport(), calculates precise viewport bounds + distance

current viewport is responsive to screen resize, that makes no sense because the viewport is reliant on the camera position. if the camera changes/moves, the last viewport is obsolete.
const { viewport } = useThree()
const { width, height, factor, distance } = viewport(optionalTarget)

[x] Fix crash with createPortal https://github.com/react-spring/react-three-fiber/pull/435

There doesnt seem to be many helpers for animating the camera. I've had to step out of the framework and jsx to tween the camera position and it feels pretty hacky. It'd be amazing if the camera could be set and animated in JSX, so declarative animations and react-spring can be used.

@likethemammal I've had similar struggles with animating the camera.

using react-three-fiber without react-dom would be huge! I can imagine plenty of use cases for this.

@likethemammal you can animate the camera declaratively with react spring, but you need to set updateprojectionmatrix. Nothing in react should be animated via props though, that is an anti pattern.

Wasn't too sure if I should put this here or create a new issue for it but having an onProgress callback for useLoader would be great! Having a loading bar for big model loading is nicer than waiting for it to pop up.

i think loading should eventually be part of suspense, it doesnt seem right that the component should handle it. but either way, i have added it. errors get thrown as well finally. it's up in v5-

There doesnt seem to be many helpers for animating the camera. I've had to step out of the framework and jsx to tween the camera position and it feels pretty hacky. It'd be amazing if the camera could be set and animated in JSX, so declarative animations and react-spring can be used.

Check out https://www.npmjs.com/package/camera-controls
I think we should make a convenience wrapper component in drei I just haven't gotten around to it yet.

@drcmda are you still interested in making react-dom optional for v5? any help needed on that front?

Yes, very much, could need help for sure. I've completed many of the tasks I wanted to tackle and this this the big one that would need quite the rewrite.

@drcmda what's your high-level vision for refactoring this? what happens to ResizeContainer for example? (happy to fork this discussion to a separate issue)


firefox-beta-bin-unwrapped: 106.0b2 -> 106.0b5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for firefox-beta-bin-unwrapped is: "Mozilla Firefox, free web browser (binary package)"
meta.homepage for firefox-beta-bin-unwrapped is: "https://www.mozilla.org/firefox/"
meta.changelog for firefox-beta-bin-unwrapped is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 106.0b3 with grep in /nix/store/3xf4n4rzg0ai7l5qm7hmcs9mj3qx2b71-firefox-beta-bin-unwrapped-106.0b3
found 106.0b3 in filename of file in /nix/store/3xf4n4rzg0ai7l5qm7hmcs9mj3qx2b71-firefox-beta-bin-unwrapped-106.0b3

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

Outpath calculations were skipped for this package; total number of rebuilds unknown.

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3xf4n4rzg0ai7l5qm7hmcs9mj3qx2b71-firefox-beta-bin-unwrapped-106.0b3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3xf4n4rzg0ai7l5qm7hmcs9mj3qx2b71-firefox-beta-bin-unwrapped-106.0b3
ls -la /nix/store/3xf4n4rzg0ai7l5qm7hmcs9mj3qx2b71-firefox-beta-bin-unwrapped-106.0b3/bin

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

2 packages built:

firefox-beta-bin
firefox-beta-bin-unwrapped

Maintainer pings
cc @taku0 @lovesegfault for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for firefox-beta-bin-unwrapped is: "Mozilla Firefox, free web browser (binary package)"
meta.homepage for firefox-beta-bin-unwrapped is: "https://www.mozilla.org/firefox/"
meta.changelog for firefox-beta-bin-unwrapped is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 106.0b4 with grep in /nix/store/g0pk44ff6xqpj2jrzxfx2pq88n0i8n80-firefox-beta-bin-unwrapped-106.0b4
found 106.0b4 in filename of file in /nix/store/g0pk44ff6xqpj2jrzxfx2pq88n0i8n80-firefox-beta-bin-unwrapped-106.0b4

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

Outpath calculations were skipped for this package; total number of rebuilds unknown.

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/g0pk44ff6xqpj2jrzxfx2pq88n0i8n80-firefox-beta-bin-unwrapped-106.0b4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/g0pk44ff6xqpj2jrzxfx2pq88n0i8n80-firefox-beta-bin-unwrapped-106.0b4
ls -la /nix/store/g0pk44ff6xqpj2jrzxfx2pq88n0i8n80-firefox-beta-bin-unwrapped-106.0b4/bin

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

2 packages built:

firefox-beta-bin
firefox-beta-bin-unwrapped

Maintainer pings
cc @taku0 @lovesegfault for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for firefox-beta-bin-unwrapped is: "Mozilla Firefox, free web browser (binary package)"
meta.homepage for firefox-beta-bin-unwrapped is: "https://www.mozilla.org/firefox/"
meta.changelog for firefox-beta-bin-unwrapped is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 106.0b5 with grep in /nix/store/1r7qq0awgqr4f3wx1jzgqrjpjyn71db3-firefox-beta-bin-unwrapped-106.0b5
found 106.0b5 in filename of file in /nix/store/1r7qq0awgqr4f3wx1jzgqrjpjyn71db3-firefox-beta-bin-unwrapped-106.0b5

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

Outpath calculations were skipped for this package; total number of rebuilds unknown.

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1r7qq0awgqr4f3wx1jzgqrjpjyn71db3-firefox-beta-bin-unwrapped-106.0b5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1r7qq0awgqr4f3wx1jzgqrjpjyn71db3-firefox-beta-bin-unwrapped-106.0b5
ls -la /nix/store/1r7qq0awgqr4f3wx1jzgqrjpjyn71db3-firefox-beta-bin-unwrapped-106.0b5/bin

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

2 packages built:

firefox-beta-bin
firefox-beta-bin-unwrapped

Maintainer pings
cc @taku0 @lovesegfault for testing.